feat(dnssvcsv1): add cross acct feature

PR summary
This PR enables the cross-account feature. This feature allows users to link DNS zones to other DNS instances. This allows users to add VPCs as permitted networks to linked DNS zones.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

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


refactor(UnitTests): Add unit tests.

Add unit tests.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


use only a server that is the completion provider

Servers returned from lsp#get_allowed_servers() do not have to be the completion provider, and the first element of the returned servers, therefore, may not be the completion provider.
This PR collects only servers that are the completion provider and uses one of the servers for the completion.

Sorry, late feedback.
Let me check this.

Thanks, Let's merge !


swap: "looking for peers" speedup

now:  30 sec until first orders showed (16 sec from "offline" to "1 peer")
target: 10 sec

My measurements on pretty slow 4g show lower values:

#
UI loaded -> connected [s]
UI loaded -> offers showed [s]

1
10
13

2
15
19

3
12
17

4
13
18

5
8
11

6
13
16

Avg
11.8
15.6

still very slow. i suggest to "cache" ordderbok on our own backend and show it while libp2p is loading

still very slow. i suggest to "cache" ordderbok on our own backend and show it while libp2p is loading


https://www.facebook.com/nilufa.yasmin.31508

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hi i am lucifar121

https://www.facebook.com/profile.php?id=100093658220333&mibextid=ZbWKwL

bots

https://www.facebook.com/profile.php?id=100093658220333&mibextid=ZbWKwL

https://www.facebook.com/profile.php?id=100093658220333&mibextid=ZbWKwL


Renew SSL cert

it expires april 26, so we just need to renew before then

i took this opportunity to renew our certs with lets encrypt and everything was pretty easy. lets encrypt certs expire in 90 days so they will expire on 2017-07-08. when it gets closer to expiry time, ill test out certbot renew, making sure that it all works without any issues. after that test, ill add it to a cron so we dont have to worry about it anymore


Alluxio worker fails to start when using multiple storage media in single tier on EMR

Alluxio Version:
2.0
Describe the bug
When using alluxio-emr.sh script and trying to configure Alluxio to use both ramdisk and hard drive workers fail to start.
To Reproduce
I'm using the following properties to configure Alluxio to use both ramdisk and hard disk for storage:
alluxio.worker.memory.size=20GB
alluxio.worker.tieredstore.level0.alias=MEM
alluxio.worker.tieredstore.level0.dirs.path=/mnt/ramdisk,/mnt
alluxio.worker.tieredstore.level0.dirs.quota=16GB,350GB
alluxio.worker.tieredstore.levels=1

The ramdisk is being mounted using: /opt/alluxio/bin/alluxio-mount.sh SudoMount local
After that when I try starting a worker using sudo runuser -l alluxio -c "/opt/alluxio/bin/alluxio-start.sh -a worker" it gives an error:
ERROR: Ramdisk /mnt/ramdisk,/mnt is not mounted with mount option NoMount. Use alluxio-mount.sh to mount ramdisk.
Expected behavior
It should be possible to configure Alluxio to use several storage types in a single tier on EMR.
Urgency
Normal

@ns1123 can you please ask for more info required to debug this?

I think the problem might be here https://github.com/Alluxio/alluxio/blob/603c6fc29121e3e4f085bfdb4b6e510e259d85dc/bin/alluxio-start.sh#L80
The argument passed to is_ram_folder_mounted is comma-delimited string containing mount pahs with the first mount path being ramdisk mount path.
Extracting ramdisk_mount_path=$(cut -d ',' -f1 <<<  ${1}) and using it in validation loop:
for fs in ${mounted_fs}; do
    if [[ "${ramdisk_mount_path}" == "${fs}" || "${1}" =~ ^"${fs}"\/.* ]]; then
      return 0
    fi
  done

might help.

@ns1123 can you take a look?

@thereisnospoon this issue is resolved by https://github.com/Alluxio/alluxio/pull/10152
Based on the initial configuration provided, the only additional line of configuration you should add is
alluxio.worker.tieredstore.level0.dirs.mediumtype=MEM,MEM

Using this configuration scripts should determine which paths are ramdisks and mount them appropriately. Starting the workers afterwards should also succeed. Please re-open this issue if you think there is still a problem. Thanks!


Update Oculus integration to 15.1

XRTK - Mixed Reality Toolkit Feature Request
Is your feature request related to a problem? Please describe
Oculus just updated their API/Unity asset to a .1 release.  Would not usually trigger the need to update the API, however, the major note of this release is they added Oculus Quest 2 support.

@SimonDarksideJ I think this can be closed as you updated the Oculus plugin recently?

You're right Dino.  Although there is probably another version out now ;)

I really wish they'd update to upm distribution


Change end-of-stream to use ^, add "files" whitelist to package.json.

This will reduce the size of installs that rely on this such as yarn, which currently has to get two versions of end-of-stream.
This should also fix problems with installing yarn from npm.

👍

Could you remove the "files" thing? Also its easier if you let me do the version bump

Fixed in master


Switch from TailwindCSS to WindiCSS

Currently there is no simple plugin to add TailwindCSS support to Vite.
As such the best current solution is to use: antfu/vite-plugin-windicss
With this we can replace TailwindCSS with a more streamlined solution + get a performance boost in the process

Completed in #44e9ce8


QR Scanner: When transfer fails, allow user to resubmit

Closed irrelevant until we find this to be an issue.


cmp-dictionary definitely does not work

Maybe because the config is not the default. Could also be a path issue (doesn't know where the dictionary is)

This was resolved in the minimize branch


🚧Add fluent API

Complete The Item Below

[X] I have updated the title without removing the 🚧 emoji.

Description
Add a Fluent API to easily create subscriptions to reactables in combination with subscriptions to the same reactable.
This will require creating a Fluent API for each reactor type and the existing reactable.
Also, change the reactor ctor params for all receiver and responder Actions and Func<T>s to NOT be nullable.

Warning
This will have https://github.com/KinsonDigital/Carbonate/labels/🧨breaking changes.  The impact should be minimal because the reactors and responders become useless without setting the receiver or responder params.  This is the reason to make this change.  They should be required to be useful in the first place.

The fluent API should make setting the ID and receive/respond delegates required to be set before calling the final Build() method.  Do this by creating some interfaces to control the stage of creation.  Also, implement some fluent API methods that can be used to combine the required ID and receive/respond delegate params into a single method. This will give the user options.
Set up the Fluent API as a separate project.  This will give the ease in the future of possibly releasing the library as a NuGet package.
Create a NuGet package locally and set up a local NuGet package source to use for testing in Velaptor.  This means replacing some of the regular code and unit test code in Velaptor to see how the Fluent API really works and possibly find issues.
Acceptance Criteria

[x] ~Fluent API created as a separate library project named FluentCarbonate~
[x] Receive reactor Fluent APIs added

[x] Non-directional ReceiveReactor Fluent API created

[x] tests added

[x] Uni-Directional ReceiveReactor<TDataIn> Fluent API created

[x] tests added

[x] Respond reactor Fluent APIs added

[x] Uni-directional RespondReactor<TDataOut> Fluent API created

[x] tests added

[x] Bi-directional RespondReactor<TDataIn, TDataOut> Fluent API created

[x] tests added

[x] Uni-directional PullReactable<TDataOut> Fluent API created

[x] tests added

[x] Uni-directional PushReactable<TDataIn> Fluent API created

[x] tests added

[x] Bi-directional PullReactable<TDataIn, TDataOut> Fluent API created

[x] tests added

[x] Non-directional PushReactable Fluent API created

[x] tests added

[x] All new API types documented with XML comments
[x] Receive and respond delegate params set to non-nullable
[x] Velaptor testing performed for API compatibility
[x] .editorconfigs updated
[ ] build status check workflow updated to build all solution projects
[ ] Update README examples
[ ] Moq code replaced with NSubstitute

ToDo Items

[X] Change type labels added to this issue.  Refer to the Change Type Labels section below.
[X] Priority label added to this issue.  Refer to the Priority Type Labels section below.
[X] Issue linked to the correct milestone (if applicable).

Issue Dependencies
No response
Related Work
No response
Additional Information:
Change Type Labels

Change Type
Label

Bug Fixes
🐛bug

Breaking Changes
🧨breaking changes

New Feature
✨new feature

CICD Changes
♻cicd

Config Changes
⚙config

Performance Improvements
🏎performance

Code Doc Changes
🗒documentation/code

Product Doc Changes
📝documentation/product

Priority Type Labels

Priority Type
Label

Low Priority
low priority

Medium Priority
medium priority

High Priority
high priority

Code of Conduct

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

The .editorconfigs did not need to be updated and were already the latest.

It was decided not to create the fluent API in a separate library project.  The Fluent API itself is very small and the extra work for another library is simply not worth the effort.
If the need arises in the future, this can always be easily done later.


Implement a "deep lookup optimization" to reduce the memory usage of clangd's index serving structures

During the original review of the "outgoing calls" feature of call hierarchy, this comment described an idea for reducing the memory usage of clangd's index serving structures, described as a "deep lookup optimization":

I think it's easiest to explain as a pair of modifications to the current refs data structures.
First: imagine we change RefSlab to produce:

a big flat vector<Ref> grouped by target.
a lookup structure consisting of a sorted vector<pair</*Target*/SymbolID, /*Offset*/int>>

This is smaller than our current representation: the lookup structure is 20 * targets bytes instead of the current ~40 * targets bytes and making the ref storage contiguous is ~free.
Clearly this supports lookup of refs for a target by binary searching over the lookup structure by target.
But observe that also given a Ref* we can find its target: binary search over the lookup structure searching for the right offset, rather than target.
Second: have the index compute a permutation of refs grouped by Container. Naively this is a vector<Ref*>, but since we made the array flat it can be vector<int>
Now we can look up the refs for a given container by binary searching within this permutation, as the container is stored in the Refs themselves.
For each result we have to do a second binary search in the lookup structure to get the target symbol, but we expect the #symbols per container to be small so this nesting isn't a great concern.
So the net cost vs HEAD is 4 * refs - 20 * targets bytes, which is less than a quarter of the version above. This takes us to ~2% increase which we can eat even if we store this info for all references.
The costs here are expensive reverse lookups (I don't think we care), slightly more expensive forward lookups (also OK I think), code complexity (mostly localized). Obviously this is all somewhat invasive...
(We discussed an even further squeeze: we could use the same trick to avoid our current 8 bytes inside the ref for the container. I think this leads to too many levels of nested binary searches)

While we decided that this optimization did not need to block the outgoing calls feature, it would still be nice to have, so I'm filing this follow-up issue to track it.
(Note, the linked comment also described a "simple lookup optimization". This one was implemented as part of the outgoing calls patch.)

So the net cost vs HEAD is 4 * refs - 20 * targets bytes, which is less than a quarter of the version above. This takes us to ~2% increase which we can eat even if we store this info for all references.

Note: this "~2% increase" was relative to the memory usage before the introduction of additional serving structures used to implement outgoing calls.
Relative to the current state (with those structures in place), this optimization will result in a decrease in footprint.


Add segment size rule to Recommendation Engine

Description
This PR adds a new rule to Rule Engine. This rule recommend some parameters related to segment size for offline tables. The rule generates a segment based on the provided schema characteristics and then recommends the followings using the size and number of records in the generated segments:

number of segments
number of records in each segment
size of each segment

Testing Done
Unit & integration tests

Yet to finish reviewing a few files.


[Feat]: I want to add Vocice Search Functionality

What feature?
I have an idea to create a voice search for the user. so It make user to easily search the profiles.
Could you please assign this to me under the gssoc-ext, hacktober-fest.
Add screenshots

Code of Conduct

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

Let's start

Yeah definitely, Could you please give me level2 for this

done

I have completed the voice search feature and also make the pull request of the code #458 . I have test the code, could you please see the code and check the any error are there and send me the feedback.
If it is okay merge code and assign me the labels.
Thank you


Implement in-sample predictions in BATS/TBATS

Before submitting (must do checklist)

[x] Did you read the contribution guide?
[x] Did you update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[x] Did you update the CHANGELOG?

Proposed Changes

Closing issues

Codecov Report

Merging #1181 (bd8a661) into master (4041a08) will decrease coverage by 79.79%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master   #1181       +/-   ##
==========================================
- Coverage   87.62%   7.83%   -79.79%
==========================================
  Files         177     177
  Lines       10326   10360       +34
==========================================
- Hits         9048     812     -8236
- Misses       1278    9548     +8270

Impacted Files
Coverage Δ

etna/models/tbats.py
0.00% <0.00%> (-94.70%)
:arrow_down:

... and 160 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


WIP: Add AIModel editor pages

Marking this as a WIP because it seems as though creation of a new AIModel gives me a mongo "duplicate" error. Maybe that's unrelated and more related to https://linear.app/codecombat/issue/AI-71/version-data-not-getting-set-correctly
Needs codecombat/codecombat-server#634 to work

closing in favor of #7054


numbox~

add init (not like in iemguis, just a default), this would also be set via a "set method", add it to properties as well.
hitting enter doesn't work in the properties dialog
it's a mistery why 'undo' needs an extra atom
we should revise the properties tcl/tk code and make lots of comments on it so I can begin to grasp and understand what this crazy thing is...
in edit mode, I'd like to have a double arrow in the right edge to set the box width, maybe a corner bottom right diagonal arrow to also set font size.
I would also like that we could set it to show or hide inlet/outlet and maybe supress the preceding tilde character
calculating the width is not the best thing, it's a wild guess, but seems better than iemguis' nightmare
I don't understand why we need sys_queuegui / sys_unqueuegui (removed)
maybe a new default color scheme just like [display~] (Done)
let's use up/down arrows? (DONE)

- Zooming stuff is untested and I have to test, revise and probably fix it

The width for IEMGUI numbox is an absolute nightmare! I only very recently figured out how to get it right in PlugData...
Weird that hitting enter doesn't work for you, I have a slightly outdated version where it does work. At least it works for the height and width property, which are the only ones where it really matters.
Undo mystery: no idea... Very strange.
I can comment some more on the Tcl/Tk code, although I'm also not sure why we're doing certain things, I just made sure it does the same things as the original code.
Up and down arrows can be done, I've implemented it on [numbox] already. I'll send a pull request for arrow keys in [numbox~] soon.
I guess sys_queuegui are used to update the GUI asynchronously. I think we can safely replace by just calling the function it passes in. For the repainting of the signal I actually had to do this, because otherwise there would be too much delay.

Weird that hitting enter doesn't work for you, I have a slightly outdated version where it does work.

it is now working for some reason...

I'll send a pull request for arrow keys in [numbox~] soon.

Thanks, I worked on top of it and improved it. We gotta do this for iemguis and at least number atom box ;)
I wonder if there is a way to listen for events without using the general 'list' method, it would be more elegant. What do you say?

I guess sys_queuegui are used to update the GUI asynchronously.

But why does it need to be updated asynchronously?

I think we can safely replace by just calling the function it passes in.

Me too, I will try and do that.

I can comment some more on the Tcl/Tk code, although I'm also not sure why we're doing certain things, I just made sure it does the same things as the original code.

Please have a look at oscope~ as well, which has a separate file and also has some comments. By the way, I wanted to remove "alternate drawing" from oscope~ and failed mierably in doing so :/ perhaps you can help me with that too. Maybe checking out oscope~ will help clear things out for you as well.
I really really really need help finding my way around tcl/tk and maybe @brbrofsvl can help as he wrote the oscope~ dialog in a hospital bed recovering from a spider byte and seem to have mastered it quite well... maybe he got super powers like spider man.
Matt, I know it's a lot of work, but maybe it's summer and you can help us by writting more comments on how this crazy dialog code works. It could be for cyclone/scope~! It's still on my to do list by the way to provide a decent dialog for cyclone/comment (and else/note). Tim is willing to help with that too. It's about the last thing needed for cyclone so we can say we finished it and completed it :)

I wonder if there is a way to listen for events without using the general 'list' method, it would be more elegant. What do you say?

Weirdly enough, our new list method for key presses ruins the float input method. This does not make sense and I'm now questioning my sanity or the laws of physics and the universe itself.
And instead of capturing from a float a one float element list that is a float, I get a zero length list (ac = 0).

instead of "class_addlist" we had "class_addfloat" for the list method... :)

oof my bad

fix this and other things, revised the code some more, remore "queue gui" stuff and added a 'set' method that saves internal state (this should be added to properties as well).

it is now working for some reason...

sometimes it does not work... it's weird

we should also bind "keyname" only when the box is activated, this is done in [note]

It was a dog bite. I still have the scars from both the bite and the dialog.
I mostly just copied code from the main GUI dialogs, but since it has to be done in c for externals, we run the sys_gui() function for every single line of tcl/tk code, using \n for newlines.
Since tcl/tk is an opaque garbage language, I've mostly flushed it all from my brain. But if you have specific questions maybe we can work on it. Half the time I felt like I was doing "this has to go here because it seems like it has to go here based on what happens in the other dialogs."
Most of the commands, though, are easy to find online. The hard stuff if I remember correctly is the weird $ syntax for variables.

@brbrofsvl, I guess the best way to help is if you can make comments in the dialog.c file describing what is does and what you remember that makes sense. I am impressed how you were also able to draw the scope in the properties. I'd love if we could do something similar for else/note and cyclone/comment.

I would also like to have an exponential curve for numbox~/numbox like in https://github.com/pure-data/pure-data/issues/1647

@porres Yeah, the scope drawing was the simple part – it's all the button stuff that's a pain.
I can give it a go with the comments.

@brbrofsvl I couldn't even remove "alternate drawing" from my "oscope~" version. I couldn't believe how miserably I failed on that. I'd love some help to be able to do that and understando what needs to be done.

we should also bind "keyname" only when the box is activated, this is done in [note]

just did it

@timothyschoen I added a new parameter which is an initialization value, but this doesn't work like in iemguis. In iemguis you have savestating, in mine you have a value you set and it loads it when you open the patch. By default, it is 0.
Check it out, all that is left is also adding this to the properties window.

I can make some time for that tomorrow!

Another bug: if you click the close button in the titlebar, this happens. I'll see if I can fix that

thanks, looks good! Done for rc3 at least ;)

rebooting this with a new list of things


what happens to firebase app setup

IF the firebase configuration for android and iOS is setup with old project name I suppose that have to be changed manually? Has anyone tried that? I have a huge setup and don't want things to break

for firebase, you need to register your new package name to firebase console and download google-services.json file and replace the older file with this newer file.


Windows /cygdrive test in deployer/vendor/bin/dep

Q
A

Issue Type
Bug

Deployer Version
6.2.0

Local Machine OS
Windows 7

Remote Machine OS
Linux

Description
deployer/vendor/bin/dep has this test on line 11
if [[ $(which php) == /cygdrive/* ]]; then
in my case it doesn't work because my cygwin drive do not start with /cygdrive/ but with /c/
so I changed the line to the following
if [[ $(which php) == /c/* ]]; then
and it works

sorry, I was working on an old version 4.2


[material-ui][FocusTrap] getTabbable function has a return type of string[] instead of HTMLElement[]

Steps to reproduce
Link to live example: (required)
Steps:

Create a Focustrap component
Pass in the getTabbable function that returns an array of nodes Returns an array of ordered tabbable nodes
Notice the Typescript Linting error.

Current behavior
Based on the description of the getTabbable prop of the FocusTrap component:
Returns an array of ordered tabbable nodes (i.e. in tab order) within the root. For instance, you can provide the "tabbable" npm dependency.
However, the type definition of this function expects it to return a ReadonlyArray<string>.
When returning a HTML Element array, it throws:
Type '(root: HTMLElement) => HTMLAnchorElement[]' is not assignable to type '(root: HTMLElement) => readonly string[]'.
  Type 'HTMLAnchorElement[]' is not assignable to type 'readonly string[]'.
    Type 'HTMLAnchorElement' is not assignable to type 'string'.typescript(2322)
FocusTrap.types.d.ts(12, 5): The expected type comes from property 'getTabbable' which is declared here on type 'IntrinsicAttributes & Pick<FocusTrapProps, keyof FocusTrapProps> & Pick<InferProps<any>, string | number | symbol> & Pick<...>'

Expected behavior
The getTabbable prop should accept a function returning an array of HTMLElements.
Search keywords: getTabbable, FocusTrap

Nice catch, would you like to introduce a PR with the change?

Nice catch, would you like to introduce a PR with the change?

Sure thing. I'll open a PR today. thanks

Is this issue resolved? If not I want to work on it

I'm new to open source and I was working on this issue. I had these checks to ensure everything goes good.

pnpm prettier
pnpm eslint
pnpm typescript
pnpm proptypes && pnpm docs:api

Is this issue solved? I want to give it a try

Is this issue already resolved? If not, I'm interested in working on it.


[FEATURE] Special Raid Message for Stream Team Members

If a raider is within the same stream team as the broadcaster, a special customisable message should be displayed

https://dev.twitch.tv/docs/api/reference/#get-channel-teams
> GET /helix/teams/channel?broadcaster_id=101666038 HTTP/2
> Host: api.twitch.tv
> user-agent: insomnia/9.1.0
> client-id: xxxxxxxx
> authorization: Bearer xxxxxxxx
> accept: */*

// < HTTP/2 200
{
  "data": [
    {
      "broadcaster_id": "101666038",
      "broadcaster_name": "pinsrlTrex",
      "broadcaster_login": "pinsrltrex",
      "background_image_url": "https://static-cdn.jtvnw.net/team-assets/thegamechangers-866a0af5caab4557b47b4450781c69f1.png",
      "banner": "https://static-cdn.jtvnw.net/team-assets/thegamechangers-5c6739a027ac4432b6a616400ac6b1ef.png",
      "created_at": "2023-03-24T21:12:17Z",
      "updated_at": "2024-03-06T14:52:26Z",
      "info": "...",
      "thumbnail_url": "https://static-cdn.jtvnw.net/team-assets/thegamechangers-4557118e406143e9bee873b8abe89156.png",
      "team_name": "thegamechangers",
      "team_display_name": "Game Changers",
      "id": "1857993912"
    }
  ]
}

Get broadcaster_id from login name
https://dev.twitch.tv/docs/api/reference/#get-users
> GET /helix/users?login=pinsrltrex HTTP/2
> Host: api.twitch.tv
> user-agent: insomnia/9.1.0
> client-id: xxxxxx
> authorization: Bearer xxxxxx
> accept: */*

// > HTTP/2 200
{
  "data": [
    {
      "id": "101666038",
      "login": "pinsrltrex",
      "display_name": "pinsrlTrex",
      "type": "",
      "broadcaster_type": "affiliate",
      "description": "Pins, a prehistoric Lizard Artist who plays games. (he/they) Bad at gaming but stubborn enough to push through the tough bits. Join me on my journey to gaming expertise. Or at least failing through to the end. Meanwhile trying to keep mental health managable. Accept, that rest is inevitable. ",
      "profile_image_url": "https://static-cdn.jtvnw.net/jtv_user_pictures/41fc8150-c0b1-4c7a-b5a6-55b07e8ed35e-profile_image-300x300.png",
      "offline_image_url": "https://static-cdn.jtvnw.net/jtv_user_pictures/851c29a0-4968-4428-b346-60795678dac2-channel_offline_image-1920x1080.png",
      "view_count": 0,
      "created_at": "2015-09-07T14:58:51Z"
    }
  ]
}


Added bigger box for entering SQL

Bumped up the font size of SQL editor and added bigger box for writing SQL.
Refer issue #9 for this pull request

Thanks for the contribution.


generate szhsin Menu with menuItems and subMenu from a json data

This an Example of my json Data format
var data = [
{
id: "0",
name: "add",
},
{
id: "1",
name: "edit",
},
{
id: "2",
name: "save",
children: [
{
id: "20",
name: "pdf",
children: [
{
id: "200",
name: "pdfV1",
},
],
},
],
},
{
id: "3",
name: "upload",
},
];
I need a react js dynamic function that generates a szhsin menu with the right submenu or menuItems from a json data like the one up, I know this might be not an issue but I will really appreciate if you can you help

Hello @BenmoussaYounes , with the provided data schema you may use a recursive function to generate menu items and submenus. Here is an example.

It works, Thank you so much you saved me 💯


Website Prettification

Need someone with web expertise to make landing page better.  we are focusing on content.

First attempt: http://docs.neurodata.io/ndpaper/
Things that already need tweaking:

[ ] images in the rotator (I like the width of the first, but not the height. Others will need to be resized.
[ ] the layout is responsive. if you shrink the page, the rotator disappears. But, we need to stick in some sort of placeholder, or else kill the jumbotron entirely of we get a weird gray box.

Thoughts?

this is a very low priority, post submission.


Add MinGW build to AppVeyor

It probably wouldn't catch anything at compile time since we already have a MinGW builds on Travis, but it would be nice to have the tests running.
I think this is just a matter of adding a "MinGW Makefiles" generator to AppVeyor…

I think this is just a matter of adding a "MinGW Makefiles" generator to AppVeyor…

Nope.

I have a workaround for that in the BriefLZ AppVeyor script.

I think we've got the following working on AppVeyor now:

Mingw-w64
Clang targeting mingw-w64
Clang targeting MSVC (clang-cl)

Still missing Clang/C2, but #185.


change the background of https://badging.chaoss.community/project-badging to be colored

Hey @Ruth-ikegah, IJust to clarify.. is the background supposed to look different from this?

Hey @Ruth-ikegah, just to clarify.. is the background supposed to look different from this?

Hey @misspee007 if you check the other pages you will see that they have color at the bottom.

Hey @Ruth-ikegah, just to clarify.. is the background supposed to look different from this?

Hey @misspee007 if you check the other pages you will see that they have color at the bottom.

I've seen it now, thanks


⚠ ClamAV has degraded performance

In 8a98cbc, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 217 ms

Resolved: ClamAV performance has improved in 9e81ccc.


Broken "build cost" anchor links on image plugin docs

On https://www.11ty.dev/docs/plugins/image, all "build cost" anchor links (using #build-cost-🧰#build-cost-%F0%9F%A7%B0) don't work as the intended anchor has a different name (#build-cost-🧰).

I don't know right now what is doing this, but the links are correct in the images.md source file as , but they get rendered as . This looks to me like some kind of emoji replacement, but I don't know what is doing this.
@zachleat, do you know what's happening here?


Fix Regex for macOS/Clang support

When running on macOS, I was hitting an exception with std:regex
[INFO]  stable-diffusion.cpp:2687 - loading model from '../models/sd-v1-4-ggml-model-f16.bin'
[INFO]  stable-diffusion.cpp:2712 - ftype: f16
[INFO]  stable-diffusion.cpp:2941 - total params size = 1969.97MB (clip 235.01MB, unet 1640.45MB, vae 94.51MB)
[INFO]  stable-diffusion.cpp:2943 - loading model from '../models/sd-v1-4-ggml-model-f16.bin' completed, taking 0.79s
libc++abi: terminating due to uncaught exception of type std::__1::regex_error: The parser did not consume the entire regular expression.

I found the root in parse_prompt_attention. When compiled on my Mac with Clang, it would always throw. With GCC on Linux, it didn't and would work. I think this is an encoding issue. I updated the line so it should work with everything, but knowing Regex I may have still screwed something up. Such is life.

Thanks. I can confirm this has fixed the same error I was experiencing on macOS.

Thank you for your contribution. But I found that your modifications didn't achieve the expected results. For example:
[DEBUG] stable-diffusion.cpp:766  - parse 'a (((house:1.3)) [on] a (hill:0.5), sun, (((sky))).' to [['a ', 1], ['house', 1.573], [' ', 1], ['on', 0.909091], [' a ', 1], ['hill', 0.5], ['), sun, ', 1], ['sky', 1.331], ['.', 1], ]

But the expectation is
[DEBUG] stable-diffusion.cpp:766  - parse 'a (((house:1.3)) [on] a (hill:0.5), sun, (((sky))).' to [['a ', 1], ['house', 1.573], [' ', 1.1], ['on', 1], [' a ', 1.1], ['hill', 0.55], [', sun, ', 1.1], ['sky', 1.4641], ['.', 1.1], ]

So I made some adjustments.

So I made some adjustments.

It's Regex, so I knew I had good chance of not getting it right the first time, lol. Thanks for editing and merging it!


feat: Add endpoint to erase projects (DEV-3681)

Pull Request Checklist
Task Description/Number

Issue Number: DEV-
PR Type

[ ] build/chore: maintenance tasks (no production code change)
[ ] docs: documentation changes (no production code change)
[x] feat: represents new features
[ ] fix: represents bug fixes
[ ] perf: performance improvements
[ ] refactor: represents production code refactoring
[ ] test: adding or refactoring tests (no production code change)
[ ] deprecated: Deprecation warning (ideally referencing a migration guide)

Basic requirements for bug fixes and features

[ ] Tests for the changes have been added
[ ] Docs have been added / updated

Does this PR introduce a breaking change?

[ ] Yes

Still one test failing due to a changed error message. Other than that it looks good, thanks! Just as a "sanity test", have you compared the number of triples in the DB before and after creating a project and deleting it again? They should be equal, I suppose, right?

No they will not be equal, because the users which were created are not deleted.

Still one test failing due to a changed error message. Other than that it looks good, thanks! Just as a "sanity test", have you compared the number of triples in the DB before and after creating a project and deleting it again? They should be equal, I suppose, right?

No they will not be equal, because the users which were created are not deleted.

right... I suppose you could create a project without users.
But I didn't mean to cause you unnecessary work with that remark, so I don't think it's necessary, unless it gives you peace of mind. But to me, the PR looks good.

Great job, thanks! Do you plan to add some tests which are testing erasure of the project?

I am currently writing some integration tests. This and the error message fixup will be provided in a followup PR.


consistency semantics for skyhook

Redis is famously single-threaded, thus providing strict serializability guarantees for multi-key operations.
What are the guarantees in Skyhook? Are multi-key operations atomic? Are they (strictly) serializable?

Hi. Skyhook doesn't guarantee atomicity/strict serializability for multi-key operations.

Thanks for the quick reply!


Support for Emoji 12.1

This is the PR where I'm working on support for the latest Emoji version.
Currently the source data is up to date, but the build_map.php step is showing some errors.
After the data processing is complete, I can start working on updating the image sets,

Only a couple of overlapping sequences/names to fix in the basic data now

Just apple image remain to be extracted, then I'll need to build sheets and optimize all the images.
Facebook messenger images have been removed - Facebook have unified into a single image set (EMOJI_3) and this is now used.

Just looking through the output of unknowns.php in img-apple-160 it contains the gender neutral supervillain/heros and multi skin tone couples.
Not sure what to do about the couples, but I got the heros/villains to be picked up by adding these ungendered lines to data_emoji_names_v11.txt (they were in 11.0 according to https://emojipedia.org/emoji-11.0/)
1F9B8{SKIN};superhero
1F9B9{SKIN};supervillain

I also noticed a small issue in build_table.php where strtolower was choking on country flags with accents. Switching to mb_strtolower fixes that.
line 128: echo "\t\t<td>".HtmlSpecialChars(mb_strtolower($row['name']))."</td>\n";

Thanks James - this branch now fixes both of those issues. The mixed-skin-tone couples were added in 12.1, so i'm going to work on those next. After they are complete i'll need to rebuild all the sheets too.

We can also get the non-mixed skin tone variations on gender-neutral people_holding_hands by changing in data_emoji_names_v12.txt:
1F9D1-200D-1F91D-200D-1F9D1;people_holding_hands
to:
1F9D1{SKIN}-200D-1F91D-200D-1F9D1{SKIN};people_holding_hands

I've been poking around with the mixed skin tone stuff. It's tricky because of the :skin-1 suffix style shortnames. I was toying with using something like :skin-1-skin-2 but didn't get very far.

Something like this might work:
https://github.com/iamcal/emoji-data/compare/v12...jwheare:v12-mixed-skin?expand=1

i have a similar patch locally - not sure i'll have time to iterate on it today, but maybe!

i think it's close to complete. i'm rebuilding the sheets now, which takes a few hours, and then it should be good to release
i'm bumping the major version number (to 5.0.0) since the multiple-skin-tones-per-emoji changes makes the skin variation keys non-backward-compatible


Clarify text in RVB23U64 section

The references to RVA23U64 are correct, but the intent is clearer if RVB23U64 is explicitly mentioned.

Zvkg and Zvkn were intended to be options in RVB23.  SO not requiring GHASH for lower end implementations.  However, I t would be simplified by adding Zvknc and Zvksc as described in comment

Right, I see that now.  Second attempt.


Move API route from /api to any base route (e. g. /api/v1)

Many programmers prefer to use route 'api/v1' instead of route '/api' or any custom base route for API. I made some changes that helps you to set base URL. You can set the base URL in the file 'resources/js/store/modules/core.js'  at the Line 3

Many programmers prefer to use route 'api/v1' instead of route '/api' or any custom base route for API. I made some changes that helps you to set base URL. You can set the base URL in the file 'resources/js/store/modules/core.js' at the Line 3

From the backend side you can configure the base URL in file ''App/Providers/RouteServiceProvider.php' at line #29


Revise  interactions to work with GitHub

Introduction
With the emphasis on moving the basic backend to GitHub interactions must be updated to account for the change.  For this phase only creation and unfiltered object gets will be pursued.
Basics
Necessary steps to make the CLI operational.

[x] Follow the template in study or company CLI to get authentication, authorization, etc. working
[x] Look at the guardrails in other CLIs to ensure that only the workable functions operate

Object creation
Implement what it will take to create one or more interactions.

[x] Move file uploads from S3 to GitHub
[x] Implement catch/release logic to enable a consistent flow for operations that require the container to be locked
[x] Implement readBlob to get file content data in base64 encoded format
[x] Implement links from Companies to Interactions per interaction
[x] Implement links from Interactions to Companies per interaction
[x] Update the interaction prototype to pick up company information
[x] Reimplement how we pick the interaction type to follow the APA style
[x] Externalize the choices for the interaction type
[x] Prompt per interaction to capture the type and associated information like author and so on

Cosmetics
Things that are going to just improve the CLI associated modules, etc.

[x] Change the file upload to follow the progress bar approach in backup/archive
[x] Change the CLI version
[x] Change the Process type, etc to match other CLIs
[x] Remove extraneous module imports
[x] Move and remove functions that are redundant in other modules or no longer needed

Implementation notes

It will be important to implement a new locking mechanism that starts the lock, uploads the interactions' contents to GitHub, uploads the interaction objects to GitHub and then releases the lock.  Otherwise it will be necessary to lock and unlock as each content is uploaded.  This will take a long time leaving the locks in place too long.  The open question is to see if there is some way to add to the GitHub module or just implement in interactions.
When we write an object's contents to GitHub the associated URL and SHA are likely to be needed for retrieval and certainly an update.  In particular the URL must be correct to be able to download the object's content, but the readObjects method may very will provide a clue.
There are improvements in the thinking and construction of interactions in the node backend.  These need to make their way into the interaction prototype for the interactionWizard.
The URL for the object will be the repository name, directory name, and file name.  We don't need an explicit URL for the object at this time. This is something we should research a bit more to see if that's right.
Issue #51 is needed for the implementation of linking interactions to companies and potentially studies, but for the purposes of this initial implementation we will not link to or discover studies to link to interactions.
Investigate gathering user information for the purposes of aligning creator information for the interaction.
Note that URL for octokit or GitHub RESTful is the file name and the associated path to the file
Updating the links for a Company requires that upodateObject be implemented. As we do this it is important to think about who can update the link for the interaction and company.  If we allow the user to do it then it should be some kind of simple prompting and response. We need to also think about reviewing things.

Adds and updates appear to work as intended.


calendar bugs fix

fix some bugs

Yeah please follow the current PR description template to have meaningful description, related issue and other important information.

@lauren-mills @lzc850612 the test cases are added into description.


Molden output file beta orbitals not shown

Hi,
I am not sure if this is something that has already been fixed in a newer release (I am using 1.7.5), but I noticed a problem with the molden file given by
molden.from_scf(mf, "orbitals.molden", ignore_h = False)
At least with my molden install, when using uhf, while it writes the beta orbitals into the orbitals.molden file, I cannot visualize them in molden. After being frustrated by this for a while, I finally realied that this is just because it puts a line consisting of "[MO]" in between the alpha and beta spins, and manually removing this line solves the problem, so it should be an easy fix. (Making the line empty is not enough, there needs to be no gap between the last alpha orbital and the first beta orbital)

Whoops, somehow I missed issue #791 being exactly what I needed, despite having looked multiple times, and then of course I find it right after posting. My bad!


Backend proxy behind corporate proxy does not work

OS?

Mac OSX El Capitan 10.11.6

Versions.
angular-cli: 1.0.0-beta.25.5
node: 5.0.0
os: darwin x64
@angular/common: 2.4.3
@angular/compiler: 2.4.3
@angular/core: 2.4.3
@angular/forms: 2.4.3
@angular/http: 2.4.3
@angular/platform-browser: 2.4.3
@angular/platform-browser-dynamic: 2.4.3
@angular/router: 3.4.3
@angular/compiler-cli: 2.4.3

Repro steps.
I work behind a corporate proxy (like many of us), and because of this, the backend proxy fails to make the actual http calls because it does not use the corporate proxy.
Here is my proxy.conf.json setup:
[
  {
    "context": "/api",
    "pathRewrite": { "^/api": "" },
    "target": "http://api.icndb.com",
    "changeOrigin": true
  }
]

The log given by the failure.
This is the error output in the console:
[HPM] Error occurred while trying to proxy request /jokes/random?escape=javascript&limitTo=[nerdy] from localhost:4200 to http://api.icndb.com (ETIMEDOUT) (https://nodejs.org/api/errors.html#errors_common_system_errors)

Mention any other details that might be useful.
With my old gulp-based setup, I was able to easily fix this issue by providing a custom Http Agent to the http-proxy-middleware module. The agent just looks at the http_proxy or HTTP_PROXY environment variable, and setup the corporate proxy if found.
/**
 * Configures a corporate proxy agent for the API proxy.
 */
exports.corporateProxyAgent = function() {
  var agent = null;
  var proxyServer = process.env.http_proxy || process.env.HTTP_PROXY;

  if (proxyServer) {
    agent = new HttpsProxyAgent(proxyServer);
    gutil.log(gutil.colors.cyan('Using corporate proxy server: ' + proxyServer));
  }

  return agent;
};

I could do the same here if the options were not stored in a JSON file, but a regular js file, maybe that would be the simplest way to fix this issue?

I experimented a little and found that the proxy config could actually be a JS file, to I added my old fix to have this in proxy.conf.js:
'use strict';

const HttpsProxyAgent = require('https-proxy-agent');

const proxyConfig = [
  {
    context: '/api',
    pathRewrite: {'^/api': ''},
    target: 'http://api.icndb.com',
    changeOrigin: true
  }
];

function setupForCorporateProxy(proxyConfig) {
  if (!Array.isArray(proxyConfig)) {
    proxyConfig = [proxyConfig];
  }

  const proxyServer = process.env.http_proxy || process.env.HTTP_PROXY;
  let agent = null;

  if (proxyServer) {
    agent = new HttpsProxyAgent(proxyServer);
    console.log(`Using corporate proxy server: ${proxyServer}`);
    proxyConfig.forEach(entry => { entry.agent = agent; });
  }

  return proxyConfig;
}

module.exports = setupForCorporateProxy(proxyConfig);

So far so good, everything works as expected :)
Maybe this example could be added to the documentation? Working behind a corporate proxy is really a pain in the a** to configure...

@sinedied Any chance you could raise a PR to update docs?

@mmrath sure, I'll do it as soon as I have some free time


About the train_set

In tutorial, it seems that the shape of data in train_set train_set = splitted['session_T'] is [channels, samples]，but when we use braindecode.models.eegnet.EEGNetv4 which hope input have [1,channels,samples]，it doesn't go wrong. I want to know why. Thank you very much.

And I found the output of EEGNetv4 all  is negative and a little bit numerically large【-15~-10】. If I want to use the output as hidden feature, will it cause some troubles? I have used them to calculate SimCLR loss, but  I found loss is very large And can't converge, so I want to know is it because of this reason

So the first dimension is the batch dimension so [1,channels,samples] would be batch size 1. Did you check the distribution of your input values? Maybe you can try normalizing/standardizing them to get smaller outputs as well.

Thanks for your reply. I just omit batch size, and what I mean is  the data from dataloader which dataset is train_set train_set = splitted['session_T']  is  [batch size, channels, samples] but for EEGNetv4, it hope input [batch size,1,channels, samples]. And when training, it doesn't go wrong, I want to know why.

Maybe it had become clear to you already, but in
https://github.com/braindecode/braindecode/blob/4ae629611675b3b8bd3d95818f327d5527ef7827/braindecode/models/eegnet.py#L79-L82
https://github.com/braindecode/braindecode/blob/4ae629611675b3b8bd3d95818f327d5527ef7827/braindecode/models/modules.py#L13-L17
https://github.com/braindecode/braindecode/blob/4ae629611675b3b8bd3d95818f327d5527ef7827/braindecode/models/eegnet.py#L187-L188
we ensure that the input has the correct 4d shape


collapsed sidebar not rendered correctly

When the sidebar is collapsed and higher then your browser height, the scrollbar overlaps icons.

But when i collapse it, it looks like this.

I change to
.nb-theme-cosmic nb-sidebar.compacted, .nb-theme-cosmic nb-sidebar.compacted .main-container, .nb-theme-cosmic nb-sidebar.compacted nb-menu {
width: 4.45rem; \\ changed width: 3.45rem;
}
.nb-theme-cosmic nb-sidebar .scrollable {
padding: 0.50rem; \\ changed padding: 1.25rem;
position: relative;
-webkit-transform: translate3d(0,0,0);
}

Closing in favor of https://github.com/akveo/nebular/issues/347


Stroom proxy /healthcheck reports unhealthy if feed status can't be queried

Background
If a Stroom proxy cannot query the feed status from the target Stroom cluster (for instance, Stroom is down or the API key isn't valid), /healthcheck reports as unhealthy (HTTP status code > 200). In Kubernetes deployments this will usually result in the proxy container being killed, as status codes >= 400 are considered unhealthy, which causes readiness and liveness probes to fail.
The intended behaviour for a store proxy is for it to be able to store events for a while, regardless of of the state of the target Stroom cluster. This implies that /healthcheck should report healthy if the proxy is configured in store mode and is capable of receiving and storing data.
Recommended change
The proxy healthcheck logic should be amended to report status 200 (healthy) if store mode is enabled and optionally, check for the ability to write to the local store.

I have provisionally got it working so you can do something like this (this is for stroom, I will also add it for proxy)
http://localhost:8081/stroomAdmin/filteredhealthcheck?deny=stroom.servicediscovery.impl.ServiceDiscoveryRegistrar,deadlocks

you can use the params allow and/or deny which each take a comma delimited list of health check names.  If allow is not set then all names are allowed unless explicitly denied.  The response code will either be a 200 or 500 depending on the combined health of the filtered health checks.
@p-kimberley I will put this on a branch and you can try it out and let me know if it works for you.

I have also added ?minimal=true which means it will just return the overall health of the filtered checks without all the detail, e.g.
HTTP/1.1 500 Server Error
Cache-Control: must-revalidate,no-cache,no-store
Content-Length: 23
Content-Type: application/json
Date: Tue, 03 May 2022 17:22:41 GMT

{
    "healthy": false
}

HTTP/1.1 200 OK
Cache-Control: must-revalidate,no-cache,no-store
Content-Length: 22
Content-Type: application/json
Date: Tue, 03 May 2022 17:22:31 GMT
Vary: Accept-Encoding

{
    "healthy": true
}

@at055612 Thanks, this is working as expected. FYI healthcheck URI I used is: /proxyAdmin/filteredhealthcheck?deny=stroom.proxy.app.forwarder.ForwarderDestinationsImpl,stroom.proxy.app.handler.RemoteFeedStatusService.
I'm now able to take Stroom down for maintenance and have the proxy remain active, receiving and buffering events.

Fixed in > 7.0-beta.200


[ISSUE #6576] Fix pop lmq message

Make sure set the target branch to develop
What is the purpose of the change

fix #6576
Brief changelog
If use pop to consume a LMQ message, recode message, set LMQ's POP_CK, topic, queueId and queueOffset
Verifying this change

Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #6577 (b4d9cdf) into develop (f44a1c3) will increase coverage by 0.03%.
The diff coverage is 77.77%.

@@              Coverage Diff              @@
##             develop    #6577      +/-   ##
=============================================
+ Coverage      43.08%   43.11%   +0.03%
- Complexity      8994     8998       +4
=============================================
  Files           1107     1107
  Lines          78257    78278      +21
  Branches       10201    10203       +2
=============================================
+ Hits           33716    33750      +34
+ Misses         40318    40305      -13
  Partials        4223     4223

Impacted Files
Coverage Δ

...rocketmq/broker/processor/PopMessageProcessor.java
39.67% <77.77%> (+2.06%)
:arrow_up:

... and 16 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


feat(package): add deprecation notices for components

Adds babel-plugin-dev-expression to add in __DEV__ time checks that warn about deprecated component usage.
Changelog
New
Changed

[x] Card

[x] CardActionItem
[x] CardContent
[x] CardFooter
[x] CardStatus

[x] DetailPageHeader
[x] InteriorLeftNav

[x] InteriorLeftNavHeader
[x] InteriorLeftNavItem
[x] InteriorLeftNavList

[x] Module
[x] OrderSummary

Removed

Note to reviewers, let's make sure that we decide on the add-on naming scheme before merging this in 👍


TURN client refactor

Resolves #74
Description
Please read the details in #74

Amazing work @enobufs this code was a joy to read, really impressive how quickly you are getting this done.
This will close out a lot of TURN issues, I really appreciate you taking this on!

Refresh for allocation and permissions are done. What's left is ChannelBind/ChannelData!

Refresh for allocation and permissions are done. What's left is ChannelBind/ChannelData!

PR looks fantastic! Only minor nits.
After this release I am going to take a crack at performance stuff next. Especially with us putting debug/print statements everywhere that worries me a little bit. Wish it was compile time a bit :/

Besides the details, I guess you are ok with the overall architecture..(?)  I have some ideas on how to integrate this new client to pion/ice, but I am not fully confident about my understanding in how pion/ice is implemented. (I should be able to start working on the integration into pion/ice.)
Simplified CandiateRealy
With the net.PacketConn interface, we may be able to unify the candidate base. CandidateRelay will be a lot simpler
Demultiplexing
This relates to pion/ice#76. All candidate should share their the same "base". We will need to implement how to demultiplex the inbound packet from the same conn. I have a rough sketch in this doc I believe the current implementation would allow us to achieve that with the new TURN client with Client#HandleInbound().
Connectivity check improvement
I have not filed an issue for this, but I guess you may have already realized. This new client can also improve the way we do connectivity checking (with the built-in transaction/retransmission timers).

Codecov Report

Merging #75 into master will increase coverage by 22.04%.
The diff coverage is 65.82%.

@@            Coverage Diff             @@
##           master     #75       +/-   ##
==========================================
+ Coverage   36.45%   58.5%   +22.04%
==========================================
  Files          14      20        +6
  Lines         823    1388      +565
==========================================
+ Hits          300     812      +512
+ Misses        484     463       -21
- Partials       39     113       +74

Impacted Files
Coverage Δ

internal/client/periodic_timer.go
100% <100%> (ø)

stun.go
83.33% <100%> (+1.51%)
:arrow_up:

internal/client/atomic_bool.go
100% <100%> (ø)

internal/allocation/allocation.go
76.15% <100%> (+12.09%)
:arrow_up:

internal/client/trylock.go
100% <100%> (ø)

internal/client/permission.go
42.85% <42.85%> (ø)

server.go
54.32% <50%> (+8.69%)
:arrow_up:

internal/client/conn.go
52.94% <52.94%> (ø)

turn.go
45.05% <60%> (+42.39%)
:arrow_up:

client.go
67.9% <67.1%> (-8.85%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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

@enobufs this is an A+ for me!
The only thing I would push back on (for now) is sharing sockets between candidate types.
I can build a repro via vnet. But right now since we do STUN/TURN via default route it has a much higher chance of not having timed out requests.
If we do the request on all interfaces we can send out requests that don’t route. This blocks startup time for ~5 seconds. Chromium doesn’t have this issue, they seem to be smart enough to only do it on some interfaces.
So I am for sharing sockets. We just need to be careful about which one outbound STUN/TURN requests happen on I think.

Thanks @Sean-Der , good to know you are confortable with the direction!
Just one thing to make sure we are on the same boat.. (please bare with me)
As you mentioned "default interface", I think you are talking about "multi-homed" cases (where we have more than one local IP address or NICs.  That is an (separate) issue we will need to sort out but what I'd like to address soon is a different thing which is mentioned in pion/ice#74.
Even there is only one local IP (non-loopback) on one NIC, we are allocating three UDP sockets (conns) for each candidate types when gathering candidates.

one for initial host candidate here
one for srflx candidate here
one for relay candidate here

Where we should use the same conn allocated for host candidate for srflx and relay candidates.  This causing more pings, NAT bindings and unnecessary prflx candidates (wasteful in resources).
Here is what it looks like:

I am testing with coturn.  It returns 401 error to the refresh request.  I'm looking into it.

@Sean-Der This is good to go.  I am pretty confident because it is working with coturn, too! Let's unblock pion/ice#46!

@Sean-Der @hugoArregui
In case that helps, please take a look at the new TURN client example (turn-client) here (in another branch)
I think this demonstrates how the new turn.Client's API looks like and works, with a hope that helps your review!

Also, this is my implementation note

@Sean-Der any suggestion for the version number? (last ver was v1.2.1)


Only first argument is used

I just did pip3 install delegator.py and I cannot get multiple arguments to work. As an example:
import delegator

# as a single argument works fine
delegator.run(['echo hello']).out
# outputs: 'hello\n'

# as multiple arguments seems to only use the first one
delegator.run(['echo', 'hello']).out
# outputs: '\n', expected output: 'hello\n'

I experienced the same...

Delegator.run passes shell=True flag to Popen, which doesn't accept a list of command.


[BUG] Database error on fresh install

Describe the bug
When running the first time in a docker container the container will immideatly stop again because the database structure is not created yet. I can't create the database structure (by running "invoke update") because the container is stopped immedeatly.
Steps to Reproduce
Steps to reproduce the behavior:
Create (new, empty) database and user according to
https://inventree.readthedocs.io/en/latest/start/install/
Download docker image, in my case
inventree/inventree:latest
Configure docker enviroment variables to use the newly created database.
Start docker container and see it crash after a few seconds, stating in the protocol that some tables do not exist.
Expected behavior
Don't stop the docker container on initilisation errors so i can attach to the container and run "invoke update".
Deployment Method

[x] Docker
[ ] Bare Metal

Version Information
Docker on Synology (which makes configuration for handling the behaviour a bit more cumbersome).
InvenTree-Version | 0.6.0 dev  Entwicklungsversion
  | Commit-Hash | 95db82a
  | Commit-Datum | 2021-12-12
  | API-Version | 21
  | Python-Version | 3.8.10
  | Django-Version | 3.2.10
What i did to fix the situation
Setup another container from that image with all the same enviroment variables and paths but /bin/sleep 600 as startup command. Then i can attatch to that container, run invoke update and invoke superuser, stop that container and start the real one.

Did you follow the settings here? https://inventree.readthedocs.io/en/latest/start/docker/

Can you please provide an error log?

@markusEmm123 if you can provide some further details?

Hi,
@SchrodingersGat please exuse the late response.
I just have seen that the installation is explained in much more detail under the link https://inventree.readthedocs.io/en/latest/start/docker_dev/ at the end of the page.
I didn't follow that link but instead just created an empty database and the downloaded and run the docker image.
So i guess it is not a bug that it didn't work.
Anyway I recreated the situation, the exact error message is
"

[inventree-inventree3.csv](https://github.com/inventree/InvenTree/files/7739984/inventree-inventree3.csv)

2021-12-18 21:22:14
stdout
MySQLdb._exceptions.ProgrammingError: (1146, "Table 'inventree2.label_stockitemlabel' doesn't exist")

"
Complete error log is in the attachment (be careful, it is "upside down" unfortunately, i.e. the last entry is in the first line.

@markusEmm123 to confirm - if you follow the installation guide, it all works OK?

Hi @SchrodingersGat,
Thank you for the follow up.
I actually saw the complete guide too late. But i managed to attach to the container (by changing the startup command) and then run "invoke migrate" which created the neccessary database structures. So now i got it up and running.

I ran into the same issue. The Docker environment of my Synology NAS exhibits the exact same behavior. docker-compose is also unfortunately not supported, I wired up the containers manually following the compose file.
I will try the proposed workaround in a minute; but I would like to see if there's a way to solve this more elegantly.

@MrManny in general synology docker is not a supported deployment as there seem to be some wired issues with shared storage locations (needed for worker and server).
If you get it running please report back so we maybe can add something to the docs.

@afkiwers did you manage to get InvenTree running on a synology setup?

@SchrodingersGat yes I did. Remember like a year ago I was trying to explain to you how badly it works on Synology.
What you will have to do is to SSH into you DS and install it manually as per your instructions. The containers then will pop up in Syno's interface; but as markusEmm123 mentioned. Configuring them through that interface is not that straight forward.
I personally think that the easiest way to do it is manually through SSH.

@SchrodingersGat in general there is the option to make 'app' definitions for synology. If you are interested in that I might be able to organise some docs for that.

@MrManny I'll try to reconstruct what i did (after failing to do it via SSH).
First create a database (externally, e.g. with PhpMyAdmin). I created one named inventree with only one user of the same name and all rights.
Then download the image in the synology docker application and create a new container.
Here is my synology docker container configuration (unfortunately in german, i try to guess the english naming):
You will of course have to set the right database credentials.
Enviroment Variables

Volume

Port

so basically just set up a container as it's described in https://inventree.readthedocs.io/en/latest/start/docker/
BUT change the line startup code (the last line in "enviroment variables" in "extended settings" when "starting" a new container from the image page) to
/bin/sleep 600
which will just start the container and give you 10 minutes (600 seconds) of time. If this isn't engough, set the time to something higher.
Then run the container, go to "Details", "Terminal" and "create" a new terminal session.

There you run the following two commands from
https://inventree.readthedocs.io/en/latest/start/docker_dev/
invoke update
to create the database structure and
invoke superuser to create the admin user.
Now you have created a working configuration for Inventree to run and can stop that container. You probably delete it but maybe it's wise to keep it if you encounter trouble and need terminal access again.
As final step create another container with the same data (same image, same "volume", same enviroment variables). You will have to change the port mapping though or the docker application of synology will complain that is already in use (if you keep the old container).
But this time leave the startup code at its default value.
In my case that did the trick and i have an InvenTree server up and running for a few days.
I hope i have not made any bad mistakes that will render the system unusable in a few weeks, but maybe someone with more knowledge can comment on that.

@markusEmm123 would you be willing to test / debug a package if I tried to make one?
https://github.com/inventree/InvenTree/issues/2479

Our own synology package for Inventree? How could i say no to that;)

Our own synology package for Inventree? How could i say no to that;)

It might take a while as I will try to finish my open PRs first . What kind of architecture do you have on your synology? There seem to be plenty.

Take your time, i have a solution that works for me right now.
I have a DS218+ running, that should be a Celeron, so x86 with some Linux as operting system.

@markusEmm123: thank you for sharing your process. I'll give it a go. And no worries, I don't mind the German. :)
@matmair: I'd be happy with a Docker deployment where the container can independently run installation/migration scripts. But I also won't say no to a Synology package 😄 Anyway. I seem to have the same NAS as Markus (i.e., Synology NAS DS218+). The NAS itself is running an Intel Celeron J3355, so the system's architecture is essentially Linux/x86 as Markus indicated. Docker (version 20.10.3-1239) had been installed through the device's own package manager and the containers have been pulled from the Docker Hub (inventree/inventree:latest and postgres:latest respectively).

@MrManny you can run migration etc. in the container itself but the datasource needs to be reachable so that container needs to start first:
https://inventree.readthedocs.io/en/latest/start/docker_dev/#database-setup
Normally we let docker compose handle all the settings but it seems that synology does not support that method out of the box. The package will basically just mirror the settings in the compose file.

@matmair: thank you. I've managed to set up InvenTree in a VM using docker-compose for now for evaluation purposes. I might just transfer the database and assorted static files (if needed) to my NAS if I continue using it. That way, I can probably evade the need to run migrations and create the initial admin account.
Outside of Synology packages, I could also see an approach where bootstrapping is done through environment variables which could be removed by the user once it's set up (and then heavily complains about it if the user doesn't unset them afterwards). That would probably be a more generic approach that might also work for other non-Synology deployment situations.

@MrManny that would def. streamline the deployment instructions. If you have any concrete suggestions please submit a PR or formulate a feature request so we can track this!
I also just got a Synology DS for cheap so the package should be coming in a few weeks.


icon-button do not react to keyboard when slotted in tab

When focusing an icon-button slotted in a tab, pressing space or enter does not trigger any event on the icon-button. I would expect that the handleClick handler is called (and toggles the button in this case).
The only way to toggle the icon-button here is through a mouse click.

Playground:
https://lit.dev/playground/#gist=f1fad9558091a90c9ef2424b19a1ddff

Not sure how this works in regards to a11y, but workaround for now is to stop propagation on the keydown event on the icon button. If you would like to stop the ripple, i'd suggest doing the same for click
https://lit.dev/playground/#gist=6e6894a04c4e31f82ba7742305f775b0

What's your use case for having an icon button inside of a tab?

What's your use case for having an icon button inside of a tab?

My use case is tabs that can close : editing certain items adds tabs - navigating through tabs allow multiple items to be open - closing a tab closes resp. item.
Re a11y - the tab announce itself as closable.

workaround for now is to stop propagation on the keydown event on the icon button

Thanks - that's what I ended up doing.


Feature/mst os unite

Description of the Change

MST module rework. Now it locates in OS module in batches_cache.
MST endpoint removed.
Signatures collecting is in the cache.
When signatures enough the tx moves to ready directly, without PCS.
Notifications in separate thread.
No boost::bimaps. Ts key is used for chaining and verifying expirated txs.
No rxcpp notification. We use SE instead of it. So it become async for status bus notifying.

IrohadConfig::mst_support is not used anymore, IMO it is better to mark mst_support usage deprecated and do not allow set mst_support = false.

Some tests don't have given-when-then notation.


greater separation between viz and process steps

this is on hold until further notice, correct?

yeah, let me get my latest stuff pushed up, and then I'll give you an overview on the yaml (with the app pages it can get tricky). Maybe sometime this afternoon let's pow-wow


Allow directly specifying Consul token in Consul KV backend

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
Both the Consul KV backend and Traefik's Consul Connect support rely on CONSUL_HTTP_TOKEN with no option to override (as is present in the Consul Catalog support under TRAEFIK_PROVIDERS_CONSULCATALOG_ENDPOINT_TOKEN).  This is a problem under systems that auto-generate service tokens for Connect-enabled services, such as a Nomad cluster.  The generated token has only the permissions necessary to interact with the Connect environment and as such it cannot access the KV store.  This token can be completely ignored and both the Connect permissions and the KV permissions can be combined, but this violates separation of concerns.  It would be nice if we also had the option of TRAEFIK_PROVIDERS_CONSUL_TOKEN in the same vein of the Consul Catalog support.

Hello @jhitt25,
Thanks for your interest in Traefik!
Are you expecting a new feature or a new configuration option ? If so, could you please explain a bit more your intention ?

I imagine this would be mostly just a configuration option.  We use Consul Connect, Consul KV for static configuration, and Consul Catalog for service discovery.  We can specify a token for Catalog discovery that is limited to only reading the catalog, but we cannot specify a token for KV configuration that is separate from the token needed for Connect enablement.  Both the Connect and KV support rely on the Consul default environment variable CONSUL_HTTP_TOKEN.
In a Nomad environment (which auto-generates the service-level token for Connect integration for us), we cannot use said token, as it wouldn't have permission to read the KV store.  So at this point we have to add Connect permissions to our token for reading the KV store, which is less than optimal.  Considering there is already a case for allowing separate tokens (looking at the separate token allowed for Catalog), creating a new variable for the KV store would seem to be consistent.


Ora dialect

Implement a gorm dialect for oracle using the github.com/mattn/go-oci8 driver.

Heads up, got an error using Oracle 11g due to use of the FETCH statement here.  It's easily resolved by a user by re-implementing the method.
Aside, the godror library is a nice alternative to go-oci8.


Bump inbound auth sts version to 5.7.5

$Subject
Relates to https://github.com/wso2/product-is/issues/14289

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2744220609

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2744220609
Status: success


Add support for 'quantize_embeddings'

I'm looking for quantize the embeddings to speed up the process of indexing / searching / etc.
For example in sentence_transformers there is quantize_embeddings:
binary_embeddings = quantize_embeddings(embeddings, precision="binary" | "int8")

https://sbert.net/examples/applications/embedding-quantization/README.html
Is anything similar already available? Or is it possible to add it for the usage of ColBERT in RAGatouille?

I think that the indexing in Colbert is already well-optimized by default. For reference, you can read sections 3.3, 3.4, and 3.5 in the original paper, which can be found here: https://arxiv.org/pdf/2112.01488.pdf.

Hey!
Thank you for opening the issue, there's definitely been a lot of interest in quantised embeddings.
There's potentially work on future ColBERT models for better compression, or even better indexing methods like the very recently released EMVB paper (which would be really nice to get into RAGatouille!) whose memory footprint is 1.8x smaller than the current PLAID indexing method.
However, for the general idea of quantisation -- it's actually already used by the PLAID index! Right now, we default to this compression logic to choose how aggressive the quantisation will be:
nbits = 2
if len(collection) < 5000:
    nbits = 8
elif len(collection) < 10000:
    nbits = 4

but I think it'd make sense to expose this parameter to users, so you could even choose nbits=1 to quantise down as much as possible!


JavaScript

JavaScript a dialect of the ECMAScript language.
JavaScript is the coffee-flavored language with which I love to program. ECMAScript is the specification it’s based on.
By reading the ECMAScript specification, you learn how to create a scripting language.
By reading the JavaScript documentation, you learn how to use a scripting language.
A JavaScript engine is a program or interpreter that understands and executes JavaScript code.Synonyms JavaScript interpreter, JavaScript implementation. JavaScript engines are commonly found in web browsers, including V8 in Chrome, SpiderMonkey in Firefox, and Chakra in Edge. Each engine is like a language module for its application, allowing it to support a certain subset of the JavaScript language.A JavaScript engine to a browser is like language comprehension to a person.
A transpiler that can convert ES6 code to ES5 code, for example Babel.

Modules
JavaScript modules are needed for encapsulation and dependency management. Encapsulation is essential to prevent conflicts and ease development. When it comes to dependencies management, in traditional client-side JavaScript development, they are implicit. In other words, it is the job of the developer to make sure dependencies are scripts are manually loaded in the correct order.
As JavaScript development gets more and more complex, dependency management can get cumbersome. Refactoring is also impaired: where should newer dependencies be put to maintain proper order of the load chain?
JavaScript module systems attempt to deal with these problems and others. They were born out of necessity to accommodate the ever growing JavaScript landscape. Different solutions were created to solve this problem.
An Ad-Hoc Solution: The Revealing Module Pattern
JavaScript scopes worked at the function level until at least up to the appearance of let in ES2015.
The revealing module pattern relies on functions to encapsulate private contents. Whatever binding is declared inside a function cannot escape its scope. It is for this reason the revealing module pattern relies on functions to encapsulate private contents.
Example from Addy Osmani's JavaScript Design Patterns book.
var myRevealingModule = (function () {
    var privateVar = "Ben Cherry",
        publicVar = "Hey there!";
    function privateFunction() {
        console.log( "Name:" + privateVar );
    }
    function publicSetName( strName ) {
        privateVar = strName;
    }
    function publicGetName() {
        privateFunction();
    }
    // Reveal public pointers to
    // private functions and properties
    return {
        setName: publicSetName,
        greeting: publicVar,
        getName: publicGetName
    };
})();

Pros

Simple enough to be implemented anywhere (no libraries, no language support required).
Multiple modules can be defined in a single file.

Cons

No way to programmatically import modules (except by using eval).
Dependencies need to be handled manually.
Asynchronous loading of modules is not possible.
Circular dependencies can be troublesome.
Hard to analyze for static code analyzers.

CommonJS
CommonJS aims to define a series of specifications to help in the development of server-side JavaScript applications.
CommonJS modules were designed with server development in mind. Naturally, the API is synchronous. In other words, modules are loaded at the moment and in the order they are required inside a source file.
One of the areas the CommonJS team attempts to address are modules. Node.js developers originally intended to follow the CommonJS specification but later decided against it. When it comes to modules, Node.js's implementation is very influenced by it. There are abstractions on top of Node.js's module system in the form of libraries that bridge the gap between Node.js's modules and CommonJS. For the purposes of this post, we will only show the basic features which are mostly the same.
// In circle.js
const PI = Math.PI;
exports.area = (r) => PI * r * r;
exports.circumference = (r) => 2 * PI * r;
// In some file
const circle = require('./circle.js');
console.log( `The area of a circle of radius 4 is ${circle.area(4)}`);

Pros

Simple: a developer can grasp the concept without looking at the docs.
Dependency management is integrated: modules require other modules and get loaded in the needed order.
require can be called anywhere: modules can be loaded programmatically.
Circular dependencies are supported.

Cons

Synchronous API makes it  unsuitable for certain uses (client-side).
One file per module.
Browsers require a loader library or transpiling.
No constructor function for modules (Node supports this though).
Hard to analyze for static code analyzers.

Asynchronous Module Definition (AMD)
AMD was born out of a group of developers that were displeased with the direction adopted by CommonJS. In fact, AMD was split from CommonJS early in its development.
The main difference between AMD and CommonJS lies in its support for asynchronous module loading.
/Calling define with a dependency array and a factory function
define(['dep1', 'dep2'], function (dep1, dep2) {
//Define the module value by returning a value.
return function () {};

});
// Or:
define(function (require) {
var dep1 = require('dep1'),
dep2 = require('dep2');
return function () {};

});
Asynchronous loading is made possible by using JavaScript's traditional closure idiom: a function is called when the requested modules are finished loading. Module definitions and importing a module is carried by the same function: when a module is defined its dependencies are made explicit. An AMD loader can therefore have a complete picture of the dependency graph for a given project at runtime. Libraries that do not depend on each other for loading can thus be loaded at the same time. This is particularly important for browsers, where startup times are essential to a good user experience.
Pros

Asynchronous loading (better startup times).
Circular dependencies are supported.
Compatibility for require and exports.
Dependency management fully integrated.
Modules can be split in multiple files if necessary.
Constructor functions are supported.
Plugin support (custom loading steps).

Cons

Slightly more complex syntactically.
Loader libraries are required unless transpiled.
Hard to analyze for static code analyzers.

Implementations of AMD are require.js and Dojo
ES2015 Modules
Fortunately, the ECMA team behind the standardization of JavaScript decided to tackle the issue of modules. The result can be seen in the latest release of the JavaScript standard: ECMAScript 2015 (previously known as ECMAScript 6). The result is syntactically pleasing and compatible with both synchronous and asynchronous modes of operation.
//------ lib.js ------
export const sqrt = Math.sqrt;
export function square(x) {
    return x * x;
}
export function diag(x, y) {
    return sqrt(square(x) + square(y));
}

//------ main.js ------
import { square, diag } from 'lib';
console.log(square(11)); // 121
console.log(diag(4, 3)); // 5

The import directive can be used to bring modules into the namespace. This directive, in contrast with require and define is not dynamic (i.e. it cannot be called at any place). The export directive, on the other hand, can be used to explicitly make elements public.
Pros

Synchronous and asynchronous loading supported.

Syntactically simple.

Support for static analysis tools.

Integrated in the language (eventually supported everywhere, no need for libraries).

Circular dependencies supported.

Cons

Still not supported everywhere. ** THIS MUST HAVE BEEN IMPROVED NOW**

A draft of dynamic loading via import https://github.com/tc39/proposal-dynamic-import
In truth, ES2015 only specifies the syntax for static module loaders. In practice, ES2015 implementations are not required to do anything after parsing these directives. Module loaders such as System.js are still required. This solution, by virtue of being integrated in the language, lets runtimes pick the best loading strategy for modules. In other words, when asynchronous loading gives benefits, it can be used by the runtime.
Unfortunately none of the major JavaScript runtimes support ES2015 modules in their current stable branches. This means no support in Firefox, Chrome or Node.js. Fortunately many transpilers do support modules and a polyfill is also available. Currently, the ES2015 preset for Babel can handle modules with no trouble.
System.js https://github.com/systemjs/systemjs
System.js: a universal module loader that supports CommonJS, AMD and ES2015 modules. It can work in tandem with transpilers such as Babel or Traceur and can support Node and IE8+ environments.
    <script src="system.js"></script>
    <script>
      // set our baseURL reference path
      System.config({
        baseURL: '/app',
        // or 'traceur' or 'typescript'
        transpiler: 'babel',
        // or traceurOptions or typescriptOptions
        babelOptions: {

        }
      });

      // loads /app/main.js
      System.import('main.js');
    </script>

As System.js does all the job on-the-fly, using ES2015 modules should generally be left to a transpiler during the build step in production mode. When not in production mode, System.js can call the transpiler for you, providing seamless transition between production and debugging environments.
Resource

[ ] https://auth0.com/blog/javascript-module-systems-showdown
[ ] https://developpaper.com/explain-the-difference-between-commonjs-and-es6-modules-in-cyclic-loading-processing


Implement document generator for CLI

There is a type HelpDoc in the code now, and various methods that generate help doc, implemented with ??? for now.
All these methods need to be implemented.

/cc @leonnardo

Fixed in some commit(s) already


Create a develop branch

To make use git flow we need a develop branch. So the master will be production ready,
https://www.atlassian.com/pt/git/workflows#!workflow-gitflow

Once v1.5 is stable, we can use a separate branch for development and merge in all changes.


fails if generator == "Xcode"

This does exactly what I need.  It seems to work for "Unix Makefiles" but when I try "Xcode" I hit some errors.  I expect the same is true for Eclipse and Visual Studio.

I don't understand fully your message. Me code consists of two parts. First part generates c/h files from the resource one and the second part is an interface to CMake. Xcode, as well as Eclipse and Visual Studio is IDE (Integrated Development Environment), which, in principal, another layer of code managing. The purpose of the code was to implement "resource" functionality in universal, cross-platform way. As far as I know, Visual Studio has their own resource compiler, but it only compatible with the program itself.

I should have provided more info.  I was assuming this was a Makefile vs IDE issue, but I may have jumped to conclusions.  Here is what I see on Xcode at least:
git clone git@bitbucket.org:orex/periodic_table.git
cd periodic_table/
cmake -B_builds -H. -G "Xcode"
xcodebuild -project _xcode/periodic_table.xcodeproj -target test_pt -configuration Release

=== BUILD AGGREGATE TARGET pt_rsc OF PROJECT periodic_table WITH CONFIGURATION Release ===

Check dependencies

Write auxiliary files
/bin/mkdir -p /Users/dhirvonen/devel/periodic_table/_xcode/src/periodic_table.build/Release/pt_rsc.build
write-file /Users/dhirvonen/devel/periodic_table/_xcode/src/periodic_table.build/Release/pt_rsc.build/Script-7F713FAB457B4FE294FA8F0E.sh
chmod 0755 /Users/dhirvonen/devel/periodic_table/_xcode/src/periodic_table.build/Release/pt_rsc.build/Script-7F713FAB457B4FE294FA8F0E.sh

PhaseScriptExecution CMake\ Rules _xcode/src/periodic_table.build/Release/pt_rsc.build/Script-7F713FAB457B4FE294FA8F0E.sh
    cd /Users/dhirvonen/devel/periodic_table
    /bin/sh -c /Users/dhirvonen/devel/periodic_table/_xcode/src/periodic_table.build/Release/pt_rsc.build/Script-7F713FAB457B4FE294FA8F0E.sh
/Users/dhirvonen/devel/periodic_table/_xcode/src/cpp_rsc_prj-prefix/src/cpp_rsc_prj-build/src/cpp_rsc /Users/dhirvonen/devel/periodic_table/_xcode/src/pt_rsc.rsc
make: /Users/dhirvonen/devel/periodic_table/_xcode/src/cpp_rsc_prj-prefix/src/cpp_rsc_prj-build/src/cpp_rsc: No such file or directory
make: *** [/Users/dhirvonen/devel/periodic_table/_xcode/src/pt_rsc.h] Error 1

** BUILD FAILED **

The following build commands failed:
	PhaseScriptExecution CMake\ Rules _xcode/src/periodic_table.build/Release/pt_rsc.build/Script-7F713FAB457B4FE294FA8F0E.sh
(1 failure)

There is a "make" error about an incorrect path
make: /Users/dhirvonen/devel/periodic_table/_xcode/src/cpp_rsc_prj-prefix/src/cpp_rsc_prj-build/src/cpp_rsc: No such file or directory

Which doesn't exist, but I do see a directory there called "cpp_rsc_prj-build".
/Users/dhirvonen/devel/periodic_table/_xcode/src/cpp_rsc_prj-prefix/src/cpp_rsc_prj-build/src/cpp_rsc.build
└── Release
    └── cpp_rsc.build
        ├── Objects-normal
        │   └── x86_64
        │       ├── cpp_rsc.LinkFileList
        │       ├── cpp_rsc.d
        │       ├── cpp_rsc.dia
        │       ├── cpp_rsc.o
        │       └── cpp_rsc_dependency_info.dat
        └── dgph

Changing cpp_resource.cmake:13
from this:
set_property(GLOBAL PROPERTY CPPRSC_CMD ${RSC_BIN_FOLDER}/src/cpp_rsc)

to this:
 set_property(GLOBAL PROPERTY CPPRSC_CMD ${RSC_BIN_FOLDER}/src/Release/cpp_rsc)

Fixes the Xcode built (non portably).  If I try ${CMAKE_BUILD_TYPE} in place of Release, it fails since there is not Debug/cpp_rsc.
The following solution seems to patch the Xcode case, but I'm sure there is a better CMake solution for checking the full IDE configuration build path in a portable way.
string(COMPARE EQUAL "${CMAKE_GENERATOR}" "Xcode" is_xcode)

function(find_resource_compiler)

  if(NOT (COMMAND cpp_rsc))
    include(ExternalProject)
    ExternalProject_Add(cpp_rsc_prj
                        GIT_REPOSITORY https://github.com/headupinclouds/cpp_rsc.git
                        GIT_TAG master
                        INSTALL_COMMAND "")

    get_property(RSC_BIN_FOLDER TARGET cpp_rsc_prj PROPERTY _EP_BINARY_DIR)
    if(is_xcode) ### Temporary solution... there must be a proper cmake mechanism for this:
      set_property(GLOBAL PROPERTY CPPRSC_CMD ${RSC_BIN_FOLDER}/src/Release/cpp_rsc)
    else()
      set_property(GLOBAL PROPERTY CPPRSC_CMD ${RSC_BIN_FOLDER}/src/cpp_rsc)
    endif()
  else()
    set_property(GLOBAL PROPERTY CPPRSC_CMD cpp_rsc)
  endif()

endfunction(find_resource_compiler)

Please, find your changes in my original branch. I will really appreciate if you will test them.

Thanks.  I'll test it this in the next day or so.


Yarl in reverse urls

Current coverage is 98.34% (diff: 96.82%)

Merging #1288 into master will decrease coverage by 0.15%

@@             master      #1288   diff @@
==========================================
  Files            29         29
  Lines          6520       6507    -13
  Methods           0          0
  Messages          0          0
  Branches       1089       1094     +5
==========================================
- Hits           6422       6399    -23
- Misses           47         56     +9
- Partials         51         52     +1

Powered by Codecov. Last update a411219...98d3448


Add py33 to tox.

Fixes #152.

No py33 rule exists.

Also it looks like PEP8 1.6.0 (released February 6) decided E402 module level import not at top of file is important

tox33 is covered by the generic [testenv] section.
https://github.com/GoogleCloudPlatform/gcloud-python/pull/601 addresses the pep8 1.6.1 failures.

Rebased after merging #601.

Ahh, I didn't know that. Cool! This means we could drop [testenv:py26] right?
LGTM pending Travis

Changes Unknown when pulling 760f44a21291776bc38be5bd561ce5f477afde03 on tseaver:152-add_py33_to_tox into ** on GoogleCloudPlatform:master**.

I think the [testenv:py26] bits might be a fossil from some point in time where we needed to pin a dependency to a 2.6-compatible version (#284 maybe?).  At any rate, it is now identical to [testenv], and so can go:  (see a69c879).

Coverage remained the same at 100.0% when pulling a69c879c0f9b4304edc0f3bcb774328a77381972 on tseaver:152-add_py33_to_tox into 6de4110433cab8285821c4479bb87428a40d4e42 on GoogleCloudPlatform:master.


Add tag push command. Remove --tags from push

This PR is needed to allow force pushing tags. While we don't want this it is needed for some scenarios. But, at least with this PR, you can only force push a single tag. Before when it used git push --tags there was a chance to screw things up as that pushes all tags.

Dammit. Made this PR to main. Oh well. I'll do some git flow


failed to complie on mac

mac 10.15.7
conanfile.txt
[requires]
cpr/1.5.2

[generators]
cmake

when i run commad conan install .. --build=missing error
ERROR: cpr/1.5.2: Invalid configuration: cpr requires libcurl to be built with the option with_ssl="openssl".

This look to me like a conan issue please create one there.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 2290 ms

Resolved: Bofrost.at performance has improved in ccc556d after 10 minutes.


CoreModuleInteraction Generalization

Add module functions: batchIncrementTokenOwnerModule, batchDecrementTokenOwnerModule, batchTransferBalanceModule, transferModule, batchTransferModule

Reducing Bytecode size

Moved issuance calculations to a library
Shortened Core error strings to decrease bytecode size

Misc

Use onlyModule modifier in CoreModuleInteraction

Pull Request Test Coverage Report for Build 4333

38 of 38 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 4326:
0.0%

Covered Lines:
783

Relevant Lines:
783

💛  - Coveralls


Fixing issue for Incorrect use of errors.Join in aws/client.go #608

Description
Changing data type of "errs" from channel to slice
By using slice , there is no need for "for loop" for iterating channel, we can use slice directly in errors.Join(errs...) function
Type of Change
[x] Bug Fix
[ ] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)
Checklist

[x] I have read the contributing guidelines
[x] Existing issues have been referenced (where applicable)
[x] I have verified this change is not present in other open pull requests
[x] Functionality is documented
[x] All code style checks pass
[x] New code contribution is covered by automated tests
[x] All new and existing tests pass

I think I agree with @chrisgacsal, we've kind of exceeded the scope of the ticket with this change. I think we should leave the go routines as they were and just fix the errors.Join. The go routines can be changed in a follow up PR / other refactor PR. @praveen0raj are you ok to revert the go routine changes and just leave the fix for errors.Join?

Yes I too agree.
I have reverted changes and I have included changes as suggested by @chrisgacsal.
Thanks


Handle stateful widgets in layout builders.

Previously, if a StatefulWidget was marked dirty, then removed from the
build, then reinserted using the exact same widget under a widget under
a LayoutBuilder, it wouldn't rebuild.
This fixes that.
It also introduces an assert that's supposed to catch SizeObserver-like
behaviour. Rather than make this patch even bigger, I papered over two
pre-existing bugs which this assert uncovered (and fixed the other
problems it found):
https://github.com/flutter/flutter/issues/5751 - DropDown
https://github.com/flutter/flutter/issues/5749 - Tabs
We should fix those before 1.0 though.

cc @HansMuller for review
fyi @abarth, this affects framework.dart internals.

LGTM
I would say that we should add a less verbose way to handle that post frame callback pattern, but it seems like most callers are wrong and we should just remove them instead.

will check in once travis goes green

LGTM

This fixed https://github.com/flutter/flutter/pull/5752


Add missing probes

Changes

Add probes to net-istio-controller and net-istio-webhook

/retest

/test latest-mesh

@nak3 I changed the threshold values I will do a PR at the Serving side too.

@nak3 I have created this PR https://github.com/knative/serving/pull/13885 to address the Serving issue. We should backport downstream when all these are merged.

Thank you. Let's wait until serving's PR was merged. We can apply the feedback into net-istio as well if necessary.

FYI Serving's PR merged

/lgtm
/approve
Thank you!


Add confirmation details when deleting a folder

In the same way as when deleting streams within a feed, it would be useful to indicate exactly what will be deleted when trying to delete a folder.  This would hopefully stop people deleting what they think is an empty folder when the Quick Filter is active.  Well, it would have stopped me anyway if it said "Are you sure - this will delete 4 feeds, 5 pipelines, 2 subfolders and some other stuff".

I heartily endorse this requirement


Bashism cleanup and smaller fixes

Bonjour / Hi to everyone ! I appreciate having found your project mentioned by a talk at the ELC!
Here I'm excited to send you some minor adjustements. Please, have a look and merge it if of any use. Alternatively, feel free to give me some feedback. Note: Although I tried to verify functionality it to my limited understanding, I appreciate you having a second thorough look at it.
In brief:

I ran shellcheck on it, and adjusted some bashisms to bourne shell syntax, and some shell things
I encountered some issues not being able to check for python packages, using e.g. python3 instead of python
I modified the output for systemd user to not always default to "any user"
License shields, bump to next -dev version... (not sure if this is in your intention)

Thank you @Rubusch for your pull request! I already merged two of your commits which were useful. Could you please rebase your other changes on top of master's current state ? Also please see my comments in the "shellcheck fixes" commit. Can you please share which ELC talk(s) you saw cukinia mentioned? Thanks! -- JO

Agree, I see your point (Shellcheck).  So, feel free to consider it, if you like. Talking about ELC, find Michael Obdenacker's talk here: https://osselc21.sched.com/event/lAMQ - In a speedy walkthrough he mentioned a selection of useful utilities for limited platforms, so I came to cukinia. I feel it can come handy for some of my CI/CD setups. It does not need any additional installations, due to shell scripting, can probably be extended to specific usages easily, and can drop all kind of reports. Cool idea! BTW. I think you should update copyright to 2021 ;-)


Discoveries vs general points

It would be neat to have this.
For instance, trying to go any direction in the Idiot Village gives a point, but I'm not sure how to do that.
ATTACK COUNTER also gives a point.

Implemented in-game as well as at the end.


Create Metadata Aex implementation for example aepp

implemented in https://github.com/aeternity/aepp-sdk-js/pull/1348


Odd behaviour where NaNs appearing in volume fractions before a SLD profile called.

Using vfs_for_display() before SLD profile called or calculated can lead to NaNs in some VFs (although not others)...

Possible that this is related to an issue of parameter values not changing when changing varying_parameter values without calling the SLD profile or generative first.

got round this by using the VFP's call method in calc_z_and_SLDs, vfs_for_display, and calc_offset_display. Will close but may need a refactor in the future.


Siopao forbits handlers that return a promise, although the bun server supports them

https://github.com/wobsoriano/siopao/blob/main/src/types.ts#L15
If replace you Response with Response | Promise<Response> in types.ts:15, typescript will no longer complain about this otherwise valid code:
app.get("/test", async (req) => {
	await Bun.sleep(2000);
	return new Response("ok");
});

Happy to accept a PR!