COG support

Add support for Cloud Optimized GeoTIFF (COG).
CLI
martin  /path/to/cogs/file.tif  /path/to/directory

Conf file
...
cogs:
  paths:
    # scan this whole dir, matching all *.tif files
    - /dir-path
    # specific cog file will be published as cog source
    - /path/to/file1.tif
  sources:
   # named source matching source name to a single file
    cog-src1:  /path/to/file.tif

It seems that the conf sections about PMTiles and MBTiles are very similar.  I don't know maybe we could merge them into a single section?
files:
  paths:
    - /dir1
    - /path/to/file1.ext
  sources:
    source1: /path/to/file.ext

I like the idea, thanks!  One aspect that I am not too certain about is combining things into files -- one of highly requested features for files it to serve them from http -- in order words, pmtiles and i suspect COG files will reside in some cloud bucket, and Martin will access them with the HTTP range requests.
This means that we would need to tell the type of that file without relying on file extension. We could try to auto-detect it - e.g. get first few bytes and detect the type, but it won't be relevant to mbtiles, so in short - much easier to keep things clean and separate.

I've built a simple C# prototype without gdal, and am working on to port it to Rust.
The config would be separated as @nyurik suggested.

Things to track.
Supported colortype and bits per sample

colory type
bits per sample
supported
status

rgb/rgba
8
🛠
PR 1590

rgb/rgba
16/32...
🛠
working on

gray
8/16/32...
🛠
working on

Tif with Non-Square pixel
Square
On X-Axis Or Y-Axis 1 pixel = 10 meters
Pixel Size = (10.000000000000000,-10.000000000000000)
Non-Square
On X it's 10, On Y it's 11
Pixel Size = (10.000000000000000,-11.000000000000000)
If it's non-square, we should adjust the output PNG size based on the resolution of X or Y and we should tell the user the resolution we picked in our TileJSON.
PseudoCode
let (resolution_x, resolution_y) = get_tiff_resolutions();
let ratio = resolution_x / Math.abs(resolution_y); // we always pick resolution_x as the resolution
let output_tile_size = (tif_tile_size[0], tif_tile_size[1] / ratio);

I'm not sure how often we encounter TIF files with non-square pixels. We will handle this when needed.

The tile-schema and tile-json
Clients needs to know the tile-schema(like min_zoom, max_zoom,tile-size, extent, resolutions array...) in tiff file to display them.
We must add these  in tile json and it's the most high priority.
And Another to be noted is:
Should we handle the google-schema specially?
The tile-schema could be GoogleMapsCompatible(see gdal doc).
Should we do a transformation internally, like if we have 5 layers in a GoogleMapsCompatible tiff file:
Add a google compatible in tile-json when it's tiff file.
{
    ...,
   "min_zoom": 10,
    "max_zoom": 14,
    "googleMapsCompatibled": "true",
    ...
}

sub-file
not-specially
specially

sub-file 0
user should call /4/x/y
user should call /14/x/y

sub-file 1
user should call /3/x/y
user should call /13/x/y

....
....
....


My calendar bank holidays

When my calendar page shows only four upcoming months and some of them are in new year, bank holidays from those months are not shown, even though when year is expanded to full they are shown.

It is not an issue.


undefined method `thumb' for #AvatarUploader:0x128508f10

Hello,
I just followed this link https://github.com/blueimp/jQuery-File-Upload/wiki/Rails-setup-for-V5
and I am getting this issue of undifined method thumb :)
I have rmagick gem in my gemfile
any help would be appreciated thank you

This is because .thumb is a method for carrierwave. If you are using paperclip or another just use their respective methods. I use paperclip and therefore:
has_attached_file :asset,
:styles => {
:square_thumb => '95x85#',
:large => '600x600>',
:overview_square => '300'
},
:default_url => '/images/pixel.gif',
:path => "public/system/images/:style/:id/:filename",
:url => "/system/images/:style/:id/:filename"
validates_attachment_content_type :asset, :content_type => %r{image/.*}
validates_attachment_presence :asset
def to_jq_upload
{
"name" => read_attribute(:asset),
"size" => asset.size,
"url" => asset.url,
"thumbnail_url" => asset.url(:square_thumb),
"delete_url" => delete_asset_path(:id => asset.id),
"delete_type" => "DELETE"
}
end


Add feature to extract English texts along with Bangla

After the text extraction code is pushed to the dev branch (#22), we should add English text extraction capability. The code currently extracts Bangla texts only, but some books have both Bangla and English texts, as noted by @svefn-g-englar.
The lang parameter may need to be updated:
extract_text_from_scanned_pdf(pdf_path, lang='ben'...)
To Do

Update the code to extract text in two modes: lang='ben' and lang='eng+ben', generating two text files.
You can generate both text files or provide the user the option (interactive input) to extract text with ben, eng+ben, or both. It'd be your discretion to choose what to go for.

Why
It might make QAing easier by giving us different baselines to compare issues with text extraction.

@Irtizaya updated the issue description

Closing this because it's redundant. Work was already done in #25.


update.bat - error in the work log

Recently I decided to run an update script, and an error message appeared in the work log:
Download Models...
Traceback (most recent call last):
  File "runpy.py", line 187, in _run_module_as_main
  File "runpy.py", line 110, in _get_module_details
  File "D:\nunif\nunif\waifu2x\__init__.py", line 1, in <module>
    from .utils import Waifu2x
  File "D:\nunif\nunif\waifu2x\utils.py", line 3, in <module>
    import torch
  File "D:\nunif\python\lib\site-packages\torch\__init__.py", line 148, in <module>
    raise err
OSError: [WinError 126] The specified module was not found. Error loading "D:\nunif\python\lib\site-packages\torch\lib\fbgemm.dll" or one of its dependencies.
Error

File fbgemm.dll is present at the specified path. The error may be related to update "torch-2.4.0+cu118"

Have you installed
"Microsoft Visual C++ Redistributable Packages" https://aka.ms/vs/16/release/vc_redist.x64.exe
?

I reinstalled VC++, but the problem persisted.

7 days ago, the update took place without errors. Today, after updating Torch from version 2.3.1 to version 2.4.0, this error appeared.

I could not reproduce this error in my nunif-windows package environment.
However, a similar issue is posted in pytorch repo
https://github.com/pytorch/pytorch/issues/131662

In this case, I will return the old files from the backup. Probably a problem with pytorch. Should I close this incident?

For now, You can revert to pytorch 2.3.1 with the following steps

run nunif-prompt.bat
python -m pip3 install torch==2.3.1 torchvision==0.8.2 --index-url https://download.pytorch.org/whl/cu118

Note that if you run update.bat, the latest pytorch(2.4.0) will be installed again.

@cynodesmus
No need to close this issue yet because it is possible that the same issue will be posted.

Oh, thank you. pytorch rollback to version 2.3.1 was successful.

I have temporarily updated requirements-torch.txt to use torch==2.3.1 on Windows.
I will revert this change when torch==2.4.1 is released.


Tag sequencing

Hi there,
At the moment our custom events which we would like to send to braze are only transferred to braze when we load the Braze Init tag before the actual event via the tag sequencing function within GTM.

If we do not load the Base tag before, the custom event is not pushed to braze. Do you know why? The Braze Base Init tag loads on every page_view so it doesn't seem correct to us that we need to load it again when pushing a custom event.
Any help is as always highly appreciated.

Hi @mariusrudolph you might have to configure the tag's priority so that you guarantee the Initialization Tag fires before your custom event.
If that doesn't fix it, please reach out to our Braze Support team<EMAIL_ADDRESS>and include steps to reproduce, verbose logs, and any other relevant screenshots or information.


Disable Comment Node Connection Semantics

As part of making this more functional, Luna performs some automatic connection of nodes from input to output where possible. It operates based on the following guidelines:

Luna is an Object-Oriented language, and hence each node has a special 'self' port. This self port is used as part of the 'main-line' of processing, often made visually distinct in the graph. In text, this is a group of variables:
a = foo1 ...
b = a.foo2 ...
c = b.foo3 ...
d = c.foo4 ...

In this case, disabling a node means 'take no further action', and hence we can safely connect the node chain. If ## c = b.foo ... is disabled, it has the effect of c = b.

If we disable a line that does not bind variables, it does not have any special meaning. ## print "foo", for example, just makes this node non-executable.

When disabling a line that does not use the 'self' port, we connect the first-used variable through if the types would match. In other cases, we cannot make it work, and execution is halted. For example, if we have ## x = foo a b, and the types of x and a are the same, there is an implicit creation of x = a so execution can continue.

Regarding point 3, we can even not check types atm. We can just connect to the first input and if types dont match we will get compile time error. In fact I believe this is the desired final behavior too.


Use background color for input toggle active state

Fixes #36439 as discussed in https://github.com/microsoft/vscode/issues/36439#issuecomment-428749511
Dark

Light

This adds inputOption.activeBackground on active toggle states (search, find widget, etc.). This also sets inputOption.activeBorder to transparent as will only use outline on toggles for focus, as discussed in https://github.com/microsoft/vscode/issues/36439#issuecomment-428749511. The new color is re-using focusBorder so it works well with other themes. Sample below:

cc @Tyriar @sandy081 @roblourens @bpasero @aeschli

Update the colors for high contrast mode:

@roblourens doesn't affect other checkboxes, just the toggle ones like exclude and search widget.
Here's what it looks like in other themes, some explicitly set inputOption.activeBorder so it doesn't look like much of a change for them:

Ok cool, how about the checkboxes in the settings editor? (sorry I could build it myself but my repo is in the middle of something else)

@roblourens doesn't affect checkboxes in the settings editor as it's not a toggle:

@misolori  I was out on vacation, sorry for not responding.
This looks good, I will provide more feedback once I selfhost on it for a couple of days.


Reconcile core module duplication

Logging, specifically, but we can foresee this being more things in the future.

:+1: tnx

Fixed by #31.


test_download_remote_layers_api and similar: don't start safekeepers to validate on-demand downloads

I've recently found the following procedure to prevent a bunch of flakiness.
We should follow it in older tests, such as the original on-demand download tests (test_download_remote_layers_api).

fire up the full stack
insert data using endpoing / pgbench
shutdown endpoint
shutdown safekeepers
do whatever needs to be tested in pageserver
if we need to validate contents, start endpoint in read-only mode
BUT: never start the safekeepers again :D

For example, the test_download_remote_layers_api doesn't do this.
At the end, it starts SKs again just to make a read-only query that is supposed to do layer downloads.
(This is follow-up to https://github.com/neondatabase/neon/pull/4024#pullrequestreview-1401395814 )

I don't immediately see how that helps; the source of flakiness I've mostly encountered is logic to wait until data from compute up to some LSN arrives to pageserver, but in the above steps no such waiting is happening. Probably it is just about something different though.

It's for tests that want to control layer creation.
These tests need to prevent a new layer from being created.
At least when I started at Neon, I thought it'd be sufficient to stop the endpoint.
But obviously it's not sufficient, because the safekeepers act as a buffer.
By stopping the safekeepers, and never starting them again, we ensure that no new WAL will arrive at pageserver.
So yes, it's for a different flakiness than the one you fixed in #4024

#3697 works like this.

Team is aware of this as a best practice and we nowadays have better monitoring of flaky tests & find issues early/earlier.


CssBuilder, StyleBuilder: Fix method with optional parameter is hidden by overload

Description
The problem with this is that if you have overloads:
StyleBuilder AddStyle(string style)
StyleBuilder AddStyle(string style, bool when = true)

And someone of them has an optional value bool? when = true this two start to conflict with each other and code analyzers will tell you to fix it.
To fix it, there is two ways:

Remove the one with least overloads and combine it with the optional one
Remove optional value

I went for second option.
More info: https://stackoverflow.com/questions/16546831/overloaded-methods-give-method-with-optional-parameter-is-hidden-by-overload-w
Type of Changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

Improved missing xml comments and improved the wording in general, since it's basically a core part for building components.

I think it would be interesting to add an implicit operator to not call build, what others think about it?

Improved coverage, I'm done with this

Nice, looks like a lot of work went into the xml docs.


Block-level, intra-segment bloom filters

This idea is not mine, but rather was mentioned by @npawar during the '22 RTA Summit video covering Tiered Storage[1]. The context in the video was using block-level bloom filters to enable loading of blocks in parallel / knowing which blocks would (likely) contain the sought after data.
Has this idea been explored for "local" pinot as well (coupled compute and storage), or would its overhead not provide query processing speed improvements?
[1] https://www.youtube.com/watch?v=nEAvHAvh0tY

cc @klsince

Yes. This idea has been implemented as sparse index, an extension available in StarTree Pinot: https://dev.startree.ai/docs/startree-extensions/sparse-index for more details. It works well with both “local” and “remote”, particularly for columns with very high cardinality.
Loop in @gortiz the author of this feature to shed more light if needed.

Thanks all 👍  I'll close this issue out so it doesn't appear as outstanding work etc.


Loading theme on the official page.

Hi. Thanks for this awesome plugin. I'd like to know if I can get my hands on the theme you guys have used in the official page here. The theme in which the white color seems to fills the text "PACE" while the page loads. I need to do it with a text of mine. Thanks! :)

@wannesdebacker That makes sense but inserting HTML markup is a challenge so a downloadable theme would be great

This has been open for far too long. If there is still interest, feel free to open a new issue.

This has been open for far too long. If there is still interest, feel free to open a new issue.


Improved sign in forms

In this update:

Add label to email field to improve accessibility of sign in form
Add support for generating sign in forms for credentials providers.

You can now generate custom sign in forms that request any details in any order.

Published as<EMAIL_ADDRESS>


Mismatch with hashes - local and Github

I've noticed recently that my repos in Github aren't displaying headings as they used to. Their official docs now say that there should be a space between # and the heading.
####Example1
Example2

Here's a screenshot of what Grip shows in grip-4.3.2:

And on Github:

I'm going to have a read now and see what I can do.

So here's an update - I've tried sending requests to the Github API via Postman. Requests to the normal /markdown endpoint are fine, but requests to /markdown/raw do return headings for #heading, which Github actually doesn't render.
Not sure, maybe that's something they should fix? Maybe they don't know?

Sent them a message see what they say I guess.

OK so Github says that's expected behaviour with the raw endpoint. Maybe we should make a way for the user to specify if they want to see raw markdown or GFM markdown?
At terminal:
grip -raw
At browser
http://localhost:6419/raw
Then we could disable the raw markdown otherwise, since it doesn't help the project's objective of being as close to Github as possible.

Thanks for all the details here!
I just tried it out and looks like you're right.
###Usage

renders differently in a README file than in Grip.
Looks like this may be from some of the recent Markdown changes they made. (I'm planning on opening an issue to track all those changes and fix them in one place. Will get to that soon!)
Since you have a conversation going, any chance you could respond and ask whether their statement that /raw "renders it as plain Markdown without a repository context (just like a README.md file is rendered)" is going to remain true, or if they're going to remove that phrasing. Because as of right now, like you point out, /raw is rendering differently than what appears on GitHub's README.md.
Thanks for digging in and getting the ball rolling on this 😄

If you're going to do that separately, I can close this issue if you like.
Here's the email I sent:
Hello,

This concerns the Github API, and in particular the markdown/raw endpoint. I've made an issue on this repo about this. FYI the project uses the Github API to preview Github readme files.

As my comment explains, posts of #heading to the Github API return a heading element, but, without a space on a Github README.md, a #heading will appear as a paragraph.

Wondering if this is a known issue, or an issue at all - is this intended behaviour?

Best regards,

And their response:
Hi Jason,

Thanks for getting in touch! The GitHub Flavored Markdown Spec was recently updated to use CommonMark spec. You can read more about the what, the why, and the how here:

https://githubengineering.com/a-formal-spec-for-github-markdown/

What you're seeing is expected behavior. I think you may find this section of our documentation of interest: https://github.github.com/gfm/#atx-headings

At least one space is required between the # characters and the heading’s contents, unless the heading is empty.

https://github.github.com/gfm/#example-34

I hope that helps, but let me know if you have any other questions!

All the best,

I've sent them an email to clarify that point.

OK, Github eventually acknowledged the issue and made a patch for it. Now the raw/ endpoint behaves as expected. My previous message about raw/ behaving as expected was wrong.

That's great news!
Thanks for following up with them and reporting back here with the results 😃
I'll mention this in CHANGES.md in the next release.


Add disable note in README.md about AI Completions

Added a note under the OPENAI_API_KEY specifying that to disable AI features (as this came up a lot in issues) just enter some random string (in this case disable) so the ++ hotcut and the AI button doesn't work.
Was just asked by some users in issues. Simple readme fix.

Hey thanks, for the PR, but with new update AI featurs are opt-in.
If you would like to update the readme about the new changes, please make a new PR
https://novel.sh/docs/quickstart#anatomy


Viewer toolbar buttons should be display:none when hidden

Please see this example -- mouse over the red area and you should see a crosshair.  Note that when the mouse moves under the scene mode selector, there are two small areas where the crosshair turns back to a normal pointer.  This is because the scene mode buttons for 2D and Columbus are not actually hidden, they are still there with an opacity of 0.
I understand why this is being done -- display:none doesn't play nice with CSS animations -- but once the animation is done, these buttons need to be removed from the document flow because they can obscure items below the toolbar.  I've contrived a simple injected element in this example, but it can also prevent clicks within e.g. the InfoBox widget in some situations.
I don't think I understand how your animations line up well enough to contribute a patch, but hopefully there's enough information in this ticket that your Animation Expert can quickly fix it.

Thanks @thw0rted!  I'm not sure what the best solution is here either.  Maybe we can play around with pointer-event: none when the dropdown is transitioning/hidden.  Using display is a little tricky because there isn't a way to delay it until after the animation finishes using CSS.  We would need to use a setTimeout in the Javascript, and I don't think that's something we want to do.

I just tried adding pointer-events: none for .cesium-sceneModePicker-hidden, and it still prevents clicks on stuff under it.  The cursor property is important too, and wouldn't be effected. I initially discovered this because I'm injecting some form fields into the InfoBox, and sometimes clicks wouldn't register in the topmost input.  It's the same bug.
I figured out a more relevant-to-me example, if it helps.  Open the Sandcastle demo for "GeoJSON and TopoJSON", select a state, then move your mouse over the value (date) box for "Statehood".  In the middle of the date, the cursor will change from "selection" (I-shaped) to a regular pointer, as it moves over the transparent-but-still-in-the-way button.
When I was working on a similar slide-out animation for my application, a common suggestion was to play with max-height.  That was no help to me, because my content was dynamic, and you can't animate an auto height, but your container height is fixed.  You might be able to do something with a delayed max-height transition, so the buttons fade out, then their container shrinks, all without using any Javascript.

Thanks for the ideas!  Using max-height might do the trick.  @emackey might have some suggestions too.
For using pointer-events, the current structure of the buttons is
<button>Main button</button>
<button>scene mode 1</button>
<button>scene mode 2</button>
<button>scene mode 3</button>

I think if we changed it to
<button>Main button</button>
<div>
    <button>scene mode 1</button>
    <button>scene mode 2</button>
    <button>scene mode 3</button>
</div>

and put pointer-events: none on the div wrapping the drop down buttons that would work.
I probably won't have a chance to look at this in the next few weeks, but I'd be happy to review a pull request if you have a solution for this!


Support Ruby 2.7's pattern matching for Layout/IndentationWidth

This PR supports Ruby 2.7's pattern matching for Layout/IndentationWidth.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Looks good!


hi I want to get  voice data: commentary in pro matche

Thanks in advance :)
I don`t see it in the example repository.
Can anybody help me getting voice data?

Thanks for reply :)
is there any parser can get commentary voice data?

There's no open source one that I am aware of.

@spheenik I've taken an interest in this as well. Do you know of a place to start to be able to decode the voice data?


Update README.md

To help make it clear how to use Shortest.

Could probably still be clearer, e.g what to name the file and where to place it and how does shortest determine the route (it always starts from / or does it use the folder it's in to determine it "magically"?)

For now it uses the testDir property in shortest.config.ts to find the test file. Maybe we could add a new section for configurations.


API Problem

Using "account" does not filter results in terms of account number such as in the example case of http://schoolido.lu/api/cards/?account=1

The account parameter can be used to populate the owned_card array within the card objects. It's not a filter.
Such filters might be added later in the API. Is this something you'd be interested in?

"account" is currently only a placeholder. Proper account
access/authentication in the API had not been implemented yet, though it is
on the TODO list.
2015/08/30 午前5:59 "saltedbread"<EMAIL_ADDRESS>
Using "account" does not filter results in terms of account number such as
in the example case of http://schoolido.lu/api/cards/?account=1
—
Reply to this email directly or view it on GitHub
https://github.com/SchoolIdolTomodachi/SchoolIdolAPI/issues/141.

@diethnis Thanks for the info! Looking forward to its completion.
@db0company Thanks for the quick reply! Pardon me on my usage of the word "filter". I actually have no idea what you call those things after the ?; like "../?account=x". Was thinking either filter or parameter but chose to call it the former.
The feature I was mainly looking for was to do a quick request on /api/cardids/ and use the account parameter to narrow down that list to the cards that said user has. The purpose is that I'm trying to make more complete Team Builder app that integrates with your site rather than relying on excel sheets, Was planning on hitting you up with that later on when its actually closer to completion, but I guess I'll bring it up now since its related to the topic at hand.

I was looking for a similar function: access and edit account cards from
API, but as it is yet to be implemented, I'm writing a python3+JSON+YAML
command line application to make a personal database. Check it out on my
Github if you're interested. It's still a Work-in-Progress. Is your project
also on Github? We may be able to help each other out!
By the way, I'm not (yet) a developer working on SchoolIdolAPI, so I don't
fully understand the inner workings of it. :P
2015/08/30 午後0:03 "saltedbread"<EMAIL_ADDRESS>
@diethnis https://github.com/diethnis Thanks for the info! Looking
forward to its completion.
@db0company https://github.com/db0company Thanks for the quick reply!
Pardon me on my usage of the word "filter". I actually have no idea what
you call those things after the ?; like "../?account=x". Was thinking
either filter or parameter but chose to call it the former.
The feature I was mainly looking for was to do a quick request on
/api/cardids/ and use the account parameter to narrow down that list to
the cards that said user has. The purpose is that I'm trying to make more
complete Team Builder app that integrates with your site rather than
relying on excel sheets, Was planning on hitting you up with that later on
when its actually closer to completion, but I guess I'll bring it up now
since its related to the topic at hand.
—
Reply to this email directly or view it on GitHub
https://github.com/SchoolIdolTomodachi/SchoolIdolAPI/issues/141#issuecomment-136076001
.

I just realized my message sounded rude, sorry. I just ment to say that the account parameter adds information in the existing objects, but doesn't "filter" the results in the sense that the total objects returned is still the same. But you probably know that already ;)
Anyways, @saltedbread if you're interested in making a team builder, since this is something we planned to work on, would you be interested in adding it directly to the website itself?
Also, since lots of people are asking for a way to edit users information, I might do that earlier. The reason why it's been delayed is that we don't want to let third parties get the users passwords, so we're thinking about being an OAuth provider, which requires us to secure a good scenario.
Since it doesn't require a lot of work on my side, would you ( @diethnis + @saltedbread ) be interested in getting the users information, and not edit them (for now)?

Sure, that would be great. If I'm not mistaken, returning a public
account's album cards or deck cards should be fairly simple if given the
account ID. Private account cards could probably work in a similar way but
require a private hash instead.
I'm no security expert, but is OAuth necessary? From what I've heard, it
seems rather difficult to add into the system.
2015/08/30 午後0:43 "deby"<EMAIL_ADDRESS>
I just realized my message sounded rude, sorry. I just ment to say that
the account parameter adds information in the existing objects, but
doesn't "filter" the results in the sense that the total objects returned
is still the same. But you probably know that already ;)
Anyways, @saltedbread https://github.com/saltedbread if you're
interested in making a team builder, since this is something we planned to
work on, would you be interested in adding it directly to the website
itself?
Also, since lots of people are asking for a way to edit users information,
I might do that earlier. The reason why it's been delayed is that we don't
want to let third parties get the users passwords, so we're thinking about
being an OAuth provider, which requires us to secure a good scenario.
Since it doesn't require a lot of work on my side, would you ( @diethnis
https://github.com/diethnis + @saltedbread
https://github.com/saltedbread ) be interested in getting the users
information, and not edit them (for now)?
—
Reply to this email directly or view it on GitHub
https://github.com/SchoolIdolTomodachi/SchoolIdolAPI/issues/141#issuecomment-136082998
.

@diethnis I still haven't started anything major yet since I have just been spending my time messing around. Once I start getting important things done is when I'll most likely start putting them out to github. I'll be sure to check out what you currently have and see if I am able to contribute to anything.
@db0company No worries! Learning the actual terminologies is definitely useful to me and I didn't take any offense to the first comment. As for adding the team builder to the website itself, definitely would prefer for that to happen. I was initially going to propose to you the idea once I have functioning code to show, but I think it would be better to be working with you guys directly from the beginning.
For obtaining user info, that would be excellent! Getting the info from public accounts would be fine for now. Private accounts can be added later once a safe way of passing the info is found.

I updated the API: http://schoolido.lu/api/ with some details + added methods to get users, accounts and owned cards. I still have to add the activities and event participations, tell me if you need that sooner.
I tried to think of all the search/filters/ordering parameters you guys might need, but tell me if you need more!


[Strapi 4] Cannot update a User collection item

Bug report
Describe the bug
When using<EMAIL_ADDRESS>with Strapi 4, I am unable to update a User collection with strapi.update().
Steps to reproduce the behavior
I have a User collection that, among other fields, has these:

I am using the SDK in Nuxt 2, so I followed your instructions to integrate it. So far auth methods working well.
On a settings page that I have, I'm doing this:
async saveFirstName() {
  await this.$strapi.update('users', this.user.id, {
    firstName: this.user.firstName,
  })
},

That sends this payload:

However, the field isn't updated and the returned object also contains it as null:

However, if I manually PUT to the /api/users/19, it works just fine:

Can it be because of the nested data object that is sent when updating a resource?

Hi @cossssmin
Thanks for this issue! Indeed Users & Permissions plugin has not be updated in order to follow Strapi v4 CRUD format, so the nested data object doesn't work here. (See the U&P update method: here
In order to update the user correctly you can use strapi.request as below:
const user = await this.$strapi.request('put', `/users/${this.user.id}`, {
   data: {
      firstName: this.user.firstName,
  },
})

Sorry again for this issue but it's something regarding Users & Permissions plugin ☹

Thanks for the quick reply and the alternative solution, works great 👍


✨ feature  - Remove some additional kcp references

Summary
Remove some additional kcp references, as per Mike's comment:
https://github.com/kubestellar/kubestellar/issues/848#issuecomment-1666147382
Related issue(s)
#848
Fixes #
#848

/lgtm
Thanks!
Just FYI: Tests have also passed
$ make test-syncer
go test  `go list ./... | grep "/pkg/cliplugins\|/pkg/syncer"`
?       github.com/kubestellar/kubestellar/pkg/syncer   [no test files]
?       github.com/kubestellar/kubestellar/pkg/syncer/clientfactory     [no test files]
?       github.com/kubestellar/kubestellar/pkg/syncer/shared    [no test files]
?       github.com/kubestellar/kubestellar/pkg/syncer/syncers   [no test files]
ok      github.com/kubestellar/kubestellar/pkg/cliplugins/kubestellar/syncer-gen        0.827s
ok      github.com/kubestellar/kubestellar/pkg/syncer/controller        14.542s

$ make e2e-test-kubestellar-syncer
...
kcp is not ready. wait for 1s...
kcp is not ready. wait for 1s...
kcp is not ready. wait for 1s...
kcp is not ready. wait for 1s...
Starting test(s)
ok      github.com/kubestellar/kubestellar/test/e2e/kubestellar-syncer  156.216s

@yana1205 Thanks, now I just need /approve

/approve

@yana1205 thanks!

@MikeSpreitzer would you /approve this?


[8.x] Make use of new PHP8 string functions

In PHP 8, functions str_starts_with(), str_ends_with() and str_contains() were introduced. These are considered superior to userland implementations.
This PR reimplements Str::startsWith, Str::endsWith, and Str::contains using the new built-in functions, while keeping backwards compatibility.
Note: in laravel/helpers, function str_contains() is already defined.To avoid infinite recursion for users using PHP 7 and the helpers package, in this PR for  Str::contains PHP version is checked rather than the existence of str_contains().

We shouldn't be using these version checks in the framework. We can make use of these functions once PHP 8 is the minimum version. This doesn't really improve anything atm.


[FEATURE]: Optional joins

Describe what you want
Similar to the where function, I would like to be able to join optionally, based on a condition.
 db
  .select()
  .from(table)
  .where(condition ? eq(...) : undefined)

Proposed api (but of course it can be anything else that makes sense and allows for optional joining):
 db
  .select()
  .from(table)
  .leftJoin(anotherTable, condition ? eq(...) : undefined)

Check this comment for a workaround


🗑 fix ebs_volume_size deprecation

what

change ebs_volume_size to storage_info.ebs_storage_info.volume_size
lock terraform version to >= 4.15.0

why

ebs_volume_size is deprecated in 4.15.0 https://registry.terraform.io/providers/hashicorp/aws/4.15.0/docs/resources/msk_cluster#ebs_volume_size

references

https://registry.terraform.io/providers/hashicorp/aws/4.15.0/docs/resources/msk_cluster#ebs_volume_size
seems there's already a PR relating to this https://github.com/cloudposse/terraform-aws-msk-apache-kafka-cluster/pull/62

/test all

hmm how do i fix the Summary check error

We have been running into this. There are multiple pr's opened by folks in the community and I was wondering if there was a plan to address this issue.
For now, we are forced to maintain a private repo with patch for the fix.

Bump

@Shuliyey can you please try running
make init
make github/init

and commit the result?


Settings/Storage shows encrypted partition

When using community encryption implementation, Settings/Storage shows /dev/sailfish/home_open as an encrypted SD card. Gives handy options:

format
open

Format - didn't test. Open does nothing as it is open already.

Fixed by https://github.com/sailfishos-sony-tama/nemo-qml-plugin-systemsettings/commit/775259af1f622fb141c26303d8f540d1923a9e26


Add TLS certificate check

removed "ssl_check" bypass options
replaced by an optional Adapter option
created a secure Adapter to force certificate check and force secure TLS cipher
added 3 tests with default adapter, forced CA list adapter with the root certificate of DEGIRO, and a faulty CA list adapter

Will try to check that the following days.
Thanks !

Great PR @facoptere !
By any chance do you have Discord or a way to join you ?
Thanks


Updated Command descriptions

Notes for Reviewers
This PR fixes #7603
Replaced Service Mesh with Cloud Native, in all instances where the changes were required.
Signed commits

[X] Yes, I signed my commits.

Codecov Report
Patch coverage has no change and project coverage change: +4.11 :tada:

Comparison is base (8b67fe0) 11.95% compared to head (585b7a3) 16.06%.

: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
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7605      +/-   ##
==========================================
+ Coverage   11.95%   16.06%   +4.11%
==========================================
  Files         120      120
  Lines       16702    16702
==========================================
+ Hits         1996     2684     +688
+ Misses      14384    13530     -854
- Partials      322      488     +166

Flag
Coverage Δ

gointegrationtests
5.58% <ø> (?)

unittests
11.95% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/app/app.go
43.33% <ø> (ø)

mesheryctl/internal/cli/root/filter/filter.go
38.46% <ø> (ø)

mesheryctl/internal/cli/root/pattern/pattern.go
38.46% <ø> (ø)

mesheryctl/internal/cli/root/root.go
65.68% <ø> (+40.19%)
:arrow_up:

mesheryctl/internal/cli/root/system/start.go
54.93% <ø> (+53.39%)
:arrow_up:

... and 10 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Thanks for this, @thisiskaransgit


Staff member list

Most servers have something like /STATS p or whatever to list opers, or a way to list opers who have specifically been marked as 'helpers'. Maybe we could come up with something similar for Ora. Interesting to see whether it should just keep that same format, whether we could possibly extend the output in some way or display more information, etc.

Some idea about the output of /STAFF  :
Hello <nickname>,
[Servername] tehnical staff is composed of :
*[Administrators]

    1. <nickname1>
    2. <nickname2>

*[Global Operators] <- (This isn't needed atm, since no federation available)

    1. <nickname1>
    2. <nickname2>

*[Local Operators] <- (maybe you should only use Operators, without global and local)

    1. <nickname1>
    2. <nickname2>

*[Helpers]

    1. <nickname1>
    2. <nickname2>
    3. <nickname3>
    4. <nickname4>

If you need assistance, feel free to /staff help <nickname>, or join the channel #CHANNAME (maybe Help, or Assistance? <- channel defined in ircd.yaml)
For further assistance, please visit https://www.somesitename.org
*End of /STAFF
Now, are these people all defined in ircd.yaml?
Are they getting the privs, via OperServ/Services?


[C++] Remove original registration ID from ExclusivePublication.

As exclusive publications are guaranteed to be original.

It is best leaving this in place as the publication API can be used interchangeable. What is the rationale for removing it?

Just code cleanup and maybe very tiny perf gain. I kept the public API fully compatible. Though, I am not sure if conductor's onNewExclusivePublication callback is used by someone.

Also wanted to confirm this statement.

As exclusive publications are guaranteed to be original.

Exclusive publications will always be original.
The change I'm OK with except for the change to the onNewExclusivePublication callback which is breaking. Maybe @tmontgomery has a view on the value of the callbacks for publications?

Is it possible to keep the same callback and just not save the original registration ID?

I've reverted the change on onNewExclusivePublication leaving the constructor of ExclusivePublication unchanged from the initial commit.

The test does not compile.

I had reverted the call to the ExclusivePublisher constructor in the test. That has been fixed.


[Snyk] Fix for 3 vulnerabilities

Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

superset-frontend/package.json
superset-frontend/package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

658/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS) SNYK-JS-LODASH-1018905
No
Proof of Concept

753/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 7.2
Command Injection SNYK-JS-LODASH-1040724
No
Proof of Concept

551/1000  Why? Recently disclosed, Has a fix available, CVSS 5.3
Improper Input Validation SNYK-JS-URIJS-1078286
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: lodash
    The new version differs by 1 commits.

c6e281b Bump to v4.17.21

See the full diff

Package name: urijs
    The new version differs by 3 commits.

46c8ac0 chore(build): bumping to version 1.19.6
a1ad8bc fix(parse): treat backslash as forwardslash in scheme delimiter
d7bb4ce chore(build): bumping to version 1.19.5

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Codecov Report

Merging #89 (db28620) into master (af3a460) will increase coverage by 0.61%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
+ Coverage   76.48%   77.09%   +0.61%
==========================================
  Files         865      865
  Lines       44979    44980       +1
  Branches     5421     5421
==========================================
+ Hits        34400    34677     +277
+ Misses      10447    10180     -267
+ Partials      132      123       -9

Flag
Coverage Δ

cypress
58.51% <ø> (+3.99%)
:arrow_up:

javascript
62.20% <ø> (ø)

python
80.70% <ø> (-0.12%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/db_engine_specs/presto.py
82.47% <0.00%> (-5.99%)
:arrow_down:

...set-frontend/src/dashboard/util/getDropPosition.js
90.90% <0.00%> (-1.52%)
:arrow_down:

superset/models/core.py
88.55% <0.00%> (-0.28%)
:arrow_down:

superset-frontend/src/components/ListView/utils.ts
85.95% <0.00%> (+0.82%)
:arrow_up:

...dashboard/components/FiltersBadge/DetailsPanel.tsx
61.36% <0.00%> (+2.27%)
:arrow_up:

superset-frontend/src/chart/chartAction.js
78.17% <0.00%> (+2.53%)
:arrow_up:

...t-frontend/src/dashboard/actions/dashboardState.js
68.87% <0.00%> (+5.29%)
:arrow_up:

...rontend/src/views/CRUD/dashboard/DashboardList.tsx
80.18% <0.00%> (+5.40%)
:arrow_up:

...eFilters/FiltersConfigModal/FiltersConfigModal.tsx
98.63% <0.00%> (+5.47%)
:arrow_up:

.../src/components/Select/WindowedSelect/windowed.tsx
94.11% <0.00%> (+5.88%)
:arrow_up:

... and 31 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 af3a460...db28620. Read the comment docs.


Feature curveball

Don't merge yet.
Curveball is a edge randomization scheme similar to the Edge Swapping Markov Chain approach already implemented as part of the LFR generator (and also exposed to Python). Instead of switch single edge, in each step Curveball "trades" the neighborhoods of two random nodes. Experimentally this yields to faster convergence.
In https://arxiv.org/abs/1804.08487 (to appear in ESA 2018), we study GlobalCurveball which additionally improves the mixing time and is has algorithmic benefits. This PR adds a new NetworKit module randomization and the two Algorithms GlobalCurveball as well as Curveball. This first implementation should be treated as the default, which the second one is useful if a fixed trade sequence is required.

Just FYI: I removed an old deprecated todo, added an option for parallel graph building after the Curveball algorithm has run and fixed an issue in the docstrings for the TradeGenerators where the order of the class documentation and the cdef was not correct.

Thank you for your review. I will fix all comments to which I did not answer directly.

The PR should now comply to NetworKit's naming conventions, and I removed the _t types, and speed-up tests. Thus, I currently see nothing in the way of merging the PR.

Yes. Thank you for your work! I will merge the PR before the release.


Training Cancel button and Model List Management

The training needs a cancel or stop button to stop the training process, since training can easily be resumed, I see no reason why a simple stop button is too much.
There also needs to be a way to manage the model list. Either on its own tab or just a button that opens a text doc to change the listed models

good idea

I +1 this as well ontop of the other one. Same thing, additionally, I would appreciate the inclusion of this feature in the standard build. I have a particular interest in periodically evaluating my voice models, approximately every 50 epochs. However, due to the excessive consumption of VRAM during training, my graphics card is unable to simultaneously handle both training and voice evaluation processes. Moreover, I have encountered difficulties restarting the training if I close the program. Therefore, I kindly request guidance on how to successfully restart the training process in such cases.

did you at least add the button?


Automatically increase padding between Event Time and Event Title when using larger font size

Increasing the font size for Event Time and Event Title, the text starts to overlap.
Expected behaviour
Would like the spacing between time and title to increase as font size increases.
Current behaviour
With larger fonts, the event time and event tile overlap
Screenshot

Possible solutions
Unsure how to resolve.
Context and reason
Checklist

[x] I searched the current issues and the feature is not known.
[x] I am mostly sure that the feature will bring more good than bad.

Hey, I will see if I can amend this to auto grow, however it could be tricky to achieve, I may be able to do some math based off the font size that is set.

Hey, I will see if I can amend this to auto grow, however it could be tricky to achieve, I may be able to do some math based off the font size that is set.

Thanks! Would it work better to just have a config options for setting the padding? I was able to edit the JS file manually and add padding as a work around. Trying to setup a dev environment so I can try and help also.

Thanks! Would it work better to just have a config options for setting the padding? I was able to edit the JS file manually and add padding as a work around. Trying to setup a dev environment so I can try and help also.

I'm trying to avoid adding nay options for changes to padding and colours etc. The existing ones were done by the previous developer.
The issue with adding this is that it makes the already complex config even more complex and is something that can easily be modified with card-mod which is what I would recommend.

I'm trying to avoid adding nay options for changes to padding and colours etc. The existing ones were done by the previous developer.
The issue with adding this is that it makes the already complex config even more complex and is something that can easily be modified with card-mod which is what I would recommend.

No worries. I'm testing out the card-mod card. Also, this calendar is working great! So thanks again for all your effort.

closing as i feel card mod is the best way to modify the looks of the card without adding too many options to this card.


fix launchpad readme

move main readme to launchpad package, and add symlink at root

ref #43

Thanks @claytercek ! Anything else you think we should account for before we merge?

No, I think it's good to go!


Compatibility with newest Mac hardware/software?

I am trying to install CellRank 2.0 on my Macbook with a Apple Silicon M2 chip and MacOS 14.0 Sonoma, but unfortunately it is unable to do so.

I created a conda environment with python=3.12 and simply tried installing through the default method
 % conda install -c conda-forge cellrank

A similar problem occurs when I use pip or when I create a conda environment with OSX64 architecture (old Intel chip) instead of the ARM64 architecture.

Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): | WARNING conda.models.version:get_matcher(546): Using .* with relational operator is superfluous and deprecated and will be removed in a future version of conda. Your spec was 1.9.0.*, but conda is ignoring the .* and treating it as 1.9.0
WARNING conda.models.version:get_matcher(546): Using .* with relational operator is superfluous and deprecated and will be removed in a future version of conda. Your spec was 1.8.0.*, but conda is ignoring the .* and treating it as 1.8.0
done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: \
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed

UnsatisfiableError: The following specifications were found to be incompatible with each other:

Output in format: Requested package -> Available versionsThe following specifications were found to be incompatible with your system:

  - feature:/osx-arm64::__osx==14.0=0
  - cellrank -> matplotlib-base[version='>=3.5.0'] -> __osx[version='>=10.9']
  - python=3.12 -> ncurses[version='>=6.4,<7.0a0'] -> __osx[version='>=10.9']

Your installed version is: 14.0

Note that strict channel priority may have removed packages required for satisfiability.

Never mind, downgrading to python 3.10 solved the issue.


Global InsecureSkipVerify does not work

Forwarding https to my backend seems to fail:

I tried adding InsecureSkipVerify = true to the global section in traefik.toml, but it still won't validate my backend. I'm using 1.0.3 release binary.

This option is not available in 1.0.x, it will be in 1.0, please refer to this issue and this PR.

@emilevauge Wait what do you mean not available in 1.0.x but in 1.0?

@ralphtheninja  This option: InsecureSkipVerify does not work in the current 1.0.x release, but will work as expected in 1.1.0 when it is released...
I think that 1.0 was a typo by @emilevauge ...

Thanks! ;)

Indeed, v1.1, not v1.0 ^^ My bad.


Direct HTML files?

Hello, this is a feature request. Can you please directly post the HTML files?

If you want HTML files it builds from the src folder for reading docs .


xarray bump from 2023.10.1 fails tests

Failure in dependabot PR #174. I'm not sure if the issue is with dscim or the new xarray 2023.11.0 release.
I could use someone's help triaging this and confirming what the issue is.
There is one failing test. Looks like an issue with "OECD Env-Growth" vs "OECD Env-" when it compares coordinates. The full error message from CI:
=================================== FAILURES ===================================
_______________________ test_concatenate_energy_damages ________________________

tmp_path = PosixPath('/tmp/pytest-of-runner/pytest-0/test_concatenate_energy_damage0')
econvars_fixture = <dscim.menu.simple_storage.EconVars object at 0x7fb21566ceb0>
energy_in_netcdf_fixture = None

    def test_concatenate_energy_damages(
        tmp_path,
        econvars_fixture,
        energy_in_netcdf_fixture,
    ):
        """
        Test that concatenate_energy_damages correctly concatenates separate energy damages by batches across SSP-RCP-GCM-IAMs and saves to separate netcdf file by batches
        """
        concatenate_energy_damages(
            input_path=os.path.join(tmp_path, "energy_in_netcdf"),
            save_path=tmp_path,
            ec_cls=econvars_fixture,
        )

        batch = ["batch" + str(i) for i in range(0, 15)]

        for b in batch:
            ds_out_expected = xr.Dataset(
                {
                    "histclim_rebased": (
                        ["batch", "rcp", "gcm", "model", "ssp", "region", "year"],
                        np.float32(np.full((1, 2, 2, 2, 2, 2, 2), 2 * 1.273526)),
                    ),
                    "delta_rebased": (
                        ["batch", "rcp", "gcm", "model", "ssp", "region", "year"],
                        np.float32(np.full((1, 2, 2, 2, 2, 2, 2), 2 * 1.273526)),
                    ),
                },
                coords={
                    "batch": (["batch"], [b]),
                    "rcp": (["rcp"], ["rcp45", "rcp85"]),
                    "gcm": (["gcm"], ["ACCESS1-0", "GFDL-CM3"]),
                    "model": (["model"], ["IIASA GDP", "OECD Env-Growth"]),
                    "ssp": (["ssp"], ["SSP2", "SSP3"]),
                    "region": (["region"], ["USA.test_region", "ZWE.test_region"]),
                    "year": (["year"], [2010, 2099]),
                },
            )

>           xr.testing.assert_equal(
                ds_out_expected,
                xr.open_dataset(os.path.join(tmp_path, f"rebased_{b}.nc4")),
            )
E           AssertionError: Left and right Dataset objects are not equal
E
E           Differing coordinates:
E           L * model             (model) <U15 'IIASA GDP' 'OECD Env-Growth'
E           R * model             (model) <U9 'IIASA GDP' 'OECD Env-'

tests/test_input_damages.py:875: AssertionError

This is from this run.
Archived log from the failed CI run is here: logs_689.zip

#229 should address this.

Resolved by #229


ATT Notification buffer not released after reconnection

Describe the bug
A buffer allocated by Bluetooth ATT (and passed on to conn) but not yet sent when a disconnect occurs is never freed.
Please also mention any information which could help others to understand
the problem you're facing:
-Added BT_DBG messages to tx_meta_data_alloc and tx_meta_data_free to track buffer allocation count
To Reproduce
The issue occurs on a custom application which is sending several Notifications per second in an environment where disconnects occur every few minutes (perhaps electrically noisy environment).
Expected behavior
Either the buffer should be freed upon disconnect, or else freed after reconnect and send operation.
Impact
Annoyance. Workaround is currently to reboot system on disconnect.
Logs and console output
In boflo2.log, the first 2 disconnects occur when no Notification data is pending. In this case the buffer alloc and free counts are either 0 or 1 as expected (see log lines after 'Client MTU 512'). Subsequent 2 disconnects occur when Notification data is pending. In these cases, the buffer alloc count increases.
In boflo3.log (after more disconnects), the system eventually hangs (no response to ATT Client MTU request).
Environment (please complete the following information):

nRF52840DK
Commit  2a8e3fe82daf43fe9e52ae25507e8e0ef72f4d59

boflo2.log
boflo3.log

@carlst thanks for reporting. Is this a regression? Or you haven't tried older Zephyr versions?

I have not tried older Zephyr versions. I have reviewed parts of the conn module handling disconnects, and will add further logging to gather more details.

I added further logging to bluetooth/host/conn.c and captured a send_buf operation followed by a disconnect in log file boflo4_47649
The added log entries (see send_frag 2 failed and send_buf: failed) indicate failure to completely send the buffer. As the connection proceeds through disconnect state, it is marked for cleanup (the added log entries bt_conn_set_state: set cleanup : old_state disconnect-complete and conn_cleanup: enter 1) but no callback to ATT to free its buffers (e.g. conn_tx_destroy?) occurs.
Please let me know if I can provide any further information.
boflo4_47649.log
.

Hmm I thought it was resolved, but I guess I didn't look hard enough. This very much looks like #43718

Hmm I thought it was resolved, but I guess I didn't look hard enough. This very much looks like https://github.com/zephyrproject-rtos/zephyr/issues/43718

Yes, most likely missing a call to conn_tx_destroy() somewhere. I am also wondering why the buffers are not freed on disconnect. conn_cleanup() should be called on disconnect and free all buffers.

@carlst I'm having some trouble reproducing it, do you mind uploading the build/zephyr/.config file so I can use your exact configuration ? If you have proprietary Kconfigs you can just grep any lines that contain CONFIG_BT, that will be enough.
Also, do you do anything on (re)connection before sending the notifications ? e.g. MTU exchange, encryption, stuff like that.

@carlst thanks to the config, I think I managed to reproduce it (using babblesim, which is pretty neat btw) here: https://github.com/zephyrproject-rtos/zephyr/compare/main...jori-nordic:zephyr:att-buffer-leak .
I opened a proper PR to fix it. Thanks for the bug report !

@jori-nordic I can no longer reproduce the issue with #48395 applied. And I verified the patch code was called (and the in-flight buffer freed by ATT). A nice, clean fix.
Babblesim is indeed quite good; I have not used it much where timing is important but may do so in the future.


fix: Migrate ports command from packngo to metal-go client

Issue Task as part of migrating metal-cli from packngo to metal-go client, added the support of Two Fa to use metal-go
Fixes: https://github.com/equinix/metal-cli/issues/333
Discussion:
As of metal-go 0.22.2 there is 1 issue which needs api support

All the APIs such as FindPortById, BondPort, DisbondPort, CreatePortVlanAssignmentBatch etc needs to support Exclude([]string) parameter as well. It only supports Includes([]string)

@aayushrangwala There are a few error logs and test failures related to these changes in the most recent E2E run:
...
=== RUN   TestPorts_Convert/convert_port_layer-3_bonded_true
2023/10/23 12:37:55 Converting port a6f37738-6cb9-4c8b-80ad-f88874f33ba8 to layer-3 with addresses [{0xc0002b750c 0xc0002b7520 map[]}]
Error when calling `DevicesApi.FindDeviceById`: 404 Not Found
...

=== RUN   TestPorts_Retrieve/retrieve_port
Error when calling `DevicesApi.FindDeviceById`: 404 Not Found
--- PASS: TestPorts_Retrieve (146.11s)
    --- PASS: TestPorts_Retrieve/retrieve_port (0.18s)
=== RUN   TestPorts_VLANs
Error when calling `VLANsApi.CreateVirtualNetwork`: 404 Not Found
    vlans_test.go:29: 404 Not Found
=== RUN   TestPorts_VLANs/vlan_assignment_port
Error: Could not update port VLAN assignments: 422 Unprocessable Entity
    vlans_test.go:67: Could not update port VLAN assignments: 422 Unprocessable Entity
    vlans_test.go:75: Timeout exceeded for vlan assignment with port ID: e0f3e6f8-5bbf-4d24-ae9d-567647c5167c
=== CONT  TestPorts_VLANs
    vlans_test.go:34: 422 Unprocessable Entity
...

Rerunning since the failed test (createdevice) could be a flake. If the helper hasn't been modified I would definitely assume this was a flake.

Rerunning since the failed test (createdevice) could be a flake. If the helper hasn't been modified I would definitely assume this was a flake.

@displague Yes it was flaky test. Tests passed in this run

Re: this note in the PR description:

All the APIs such as FindPortById, BondPort, DisbondPort, CreatePortVlanAssignmentBatch etc needs to support Exclude([]string) parameter as well. It only supports Includes([]string)

@sbhatnagar-equinix did some focused work earlier this year to align include/exclude parameter declarations in the spec with actual parameter support in the API.  Surbhit can confirm whether these missing methods are expected to be missing.
If we'd like to validate the API behavior ourselves, one thing to try is to make a curl request to the affected endpoints, then try the request again with an excludes parameter, to see if the response changes:
# Run commands like the following and see if the request changes due to the addition of exclude param
$ curl -v -H 'X-Auth-Token: <some_token>' https://api.equinix.com/metal/v1/<path_to_endpoint>
$ curl -v -H 'X-Auth-Token: <some_token>' https://api.equinix.com/metal/v1/<path_to_endpoint>?excludes=<something>

If we do find differences we can put those details in a ticket for the API team.

@aayushrangwala Re: this note in the PR description:

All the APIs such as FindPortById, BondPort, DisbondPort, CreatePortVlanAssignmentBatch etc needs to support Exclude([]string) parameter as well. It only supports Includes([]string)

@sbhatnagar-equinix did some focused work earlier this year to align include/exclude parameter declarations in the spec with actual parameter support in the API. Surbhit can confirm whether these missing methods are expected to be missing.
If we'd like to validate the API behavior ourselves, one thing to try is to make a curl request to the affected endpoints, then try the request again with an excludes parameter, to see if the response changes:
# Run commands like the following and see if the request changes due to the addition of exclude param
$ curl -v -H 'X-Auth-Token: <some_token>' https://api.equinix.com/metal/v1/<path_to_endpoint>
$ curl -v -H 'X-Auth-Token: <some_token>' https://api.equinix.com/metal/v1/<path_to_endpoint>?excludes=<something>

If we do find differences we can put those details in a ticket for the API team.

@ctreatma I remember we discussing this and as you correctly pointed out, there was some work on this but still after that only include was supported.
Checked the Developer API doc for the support and even the HTTP API doesnt seem to support the exclude
https://deploy.equinix.com/developers/api/metal/#tag/Ports
Although, when I tried to run via curl, exclude also works for ports service. @sbhatnagar-equinix Can you please check here
curl -v -H 'X-Auth-Token: API_TOKEN' https://api.equinix.com/metal/v1/ports/1d7b102b-a955-4aea-9662-c4ab742fae9b\?exclude\='virtual_networks' | grep virtual_networks

Response:

{"id":"1d7b102b-a955-4aea-9662-c4ab742fae9b","type":"NetworkBondPort","name":"bond0","data":{"bonded":true},"network_type":"layer2-bonded","native_virtual_network":null,"virtual_networks":[{"href":"/metal/v1/virtual-networks/561319f2-1ba2-43ea-9337-12cb409c24c9"}],"disbond_operation_supported":true,"href":"/metal/v1/ports/1d7b102b-a955-4aea-9662-c4ab742fae9b"}

curl -v -H 'X-Auth-Token: API_TOKEN' https://api.equinix.com/metal/v1/ports/1d7b102b-a955-4aea-9662-c4ab742fae9b\?include\='virtual_networks' | grep virtual_networks

Response:

{"id":"1d7b102b-a955-4aea-9662-c4ab742fae9b","type":"NetworkBondPort","name":"bond0","data":{"bonded":true},"network_type":"layer2-bonded","native_virtual_network":null,"virtual_networks":[{"id":"561319f2-1ba2-43ea-9337-12cb409c24c9","description":"","vxlan":1000,"created_at":"2023-10-30T14:13:43Z","tags":[],"metro":{"href":"/metal/v1/locations/metros/b1ac82b2-616c-4405-9424-457ef6edf9ae"},"metal_gateways":[],"facility":null,"instances":[{"href":"/metal/v1/devices/d6a80ef5-8c92-4502-a22b-482bc89ea5b1"}],"assigned_to":{"href":"/metal/v1/projects/ad7bb41a-810e-4a06-b12b-97697bf9e2cd"},"facility_code":null,"assigned_to_virtual_circuit":false,"metro_code":"fr","href":"/metal/v1/virtual-networks/561319f2-1ba2-43ea-9337-12cb409c24c9"}],"disbond_operation_supported":true,"href":"/metal/v1/ports/1d7b102b-a955-4aea-9662-c4ab742fae9b"}

I can't seem to use this command to convert a port to Layer 3.
PACKNGO_DEBUG=1 go run ./cmd/metal ports convert -i d5acf76f-329b-4534-96f4-b28d9cfa6173 --bonded -4 -6

^ If I omit -b, the CLI only attempts a GET request and doesn't attempt any conversion. When I include -b, the CLI only attempts to bond the port.
Fortunately, for this PR, I can't seem to do that with the existing 1.16 binary either 😬
While looking at the --help docs, I noticed:

there is no option or instruction on how to --disbond (and -b false does not do this either, not sure which I would prefer) https://github.com/equinix-labs/metal-go/blob/main/docs/PortsApi.md#disbondport
the notes on adding a public ipv4/ipv6 address say "Layer-2" when they mean "Layer-3"
there should be an option to convert to layer3, this would imply only a private ipv4 address. -3 | --layer3. adding -4 and -6 would add the optional ipv4 and ipv6 addresses to the request.
-2 should not assume layer-2 unbonded, that should depend on the -b true|false argument

@displague https://github.com/equinix/metal-cli/issues/374 Created this issue

@displague Can you please re-trigger the E2E tests. I think was a flaky scenario

One E2E test failed:
=== RUN   TestPorts_VLANs
Error when calling `DevicesApi.FindDeviceById`: 403 Forbidden
    helper.go:302: 403 Forbidden
    helper.go:305: Timeout while waiting for device: 467ae367-bb48-469d-b910-8e71533cf42f to be active
Error when calling `DevicesApi.FindDeviceById`: 403 Forbidden
    helper.go:310: 403 Forbidden
--- FAIL: TestPorts_VLANs (65.85s)
panic: runtime error: index out of range [2] with length 0 [recovered]
	panic: runtime error: index out of range [2] with length 0

goroutine 36 [running]:
testing.tRunner.func1.2({0xca9f80, 0xc0001cb338})
	/opt/hostedtoolcache/go/1.19.13/x64/src/testing/testing.go:1396 +0x24e
testing.tRunner.func1()
	/opt/hostedtoolcache/go/1.19.13/x64/src/testing/testing.go:1399 +0x39f
panic({0xca9f80, 0xc0001cb338})
	/opt/hostedtoolcache/go/1.19.13/x64/src/runtime/panic.go:884 +0x212
github.com/equinix/metal-cli/test/e2e/ports.TestPorts_VLANs(0xc0004f4000)
	/home/runner/work/metal-cli/metal-cli/test/e2e/ports/vlans_test.go:28 +0x4dc
testing.tRunner(0xc0004f4000, 0xd6b4f0)
	/opt/hostedtoolcache/go/1.19.13/x64/src/testing/testing.go:1446 +0x10b
created by testing.(*T).Run
	/opt/hostedtoolcache/go/1.19.13/x64/src/testing/testing.go:1493 +0x35f
FAIL	github.com/equinix/metal-cli/test/e2e/ports	596.680s

Going to merge this to unblock other work. Ahead of the next release (ahead of merging any feat or fix PRs) we should address these two issues.

#386
#387


Prevent HttpClient modify default header settings

@poying I've just merged this in to master - thanks for your contribution!
I added a test for this behaviour too in #162


Enhancements following 5.0

Tickets we have decided won't be in 5.0 but we should be addressed as a follow-up in 5.1

The open tickets have been prioritized and added to other epics. Closing this.


ParseTypeArguments calls parseExpression()

Migrated from https://bugs.webkit.org/show_bug.cgi?id=189022:
At 2018-08-27T23:05:20Z<EMAIL_ADDRESS>wrote:
It calls parseConstexpr(), which calls parseTerm(), which calls parseExpression(). We don't want to allow things like
int x = 2;
vector<float, (x + 1)> v;
We need to either change how the parser works, or add validation that typeArguments can only be certain things.

This was only true of the js parser.


Fix Issue 26680 in blazor image documentation

Fixes #26680

... and I just set up a PR for this on the dotnet/blazor-samples repo. I'll sit on that until Tanay reviews this.
https://github.com/dotnet/blazor-samples/pull/32


[MM 23785] Show confirmation dialogue when mention groups

REVIEW ORDER
1st: https://github.com/mattermost/mattermost-mobile/pull/4118
2nd: https://github.com/mattermost/mattermost-mobile/pull/4156
3rd: https://github.com/mattermost/mattermost-mobile/pull/4250
Last commit should be the one reviewed! All other commits are from the branch I've built off of
Things to do before merge:

[ ] Make min server check from 5.22 to 5.24 before merge.
[x] Fix the actions to not use bind function!
[x] ADD TESTS!

Summary

Show the confirmation modal for group mentions
Ticket Link

https://mattermost.atlassian.net/browse/MM-23785
Related Pull Requests

https://github.com/mattermost/mattermost-server/pull/14068
https://github.com/mattermost/mattermost-mobile/pull/4118
Screenshots

I haven't looked at the code but my guess is that this PR will conflict with https://github.com/mattermost/mattermost-mobile/pull/4122#pullrequestreview-406171634 that is soon to be merged

@enahum
Hey Elias,
I had added some test for this confirmation modal (https://github.com/mattermost/mattermost-mobile/pull/4250/commits/9178ebd8083dd28a32e7fce90ab34e665179c018#diff-fedc8dc3780078490a33a110f81a72cbR162) in the post_textbox_base file however after the refactoring I noticed that that entire test file had been deleted. I also noticed that some of the tests such as https://github.com/mattermost/mattermost-mobile/pull/4250/commits/9178ebd8083dd28a32e7fce90ab34e665179c018#diff-fedc8dc3780078490a33a110f81a72cbR190 where not included anywhere in the refactor. Was this done on purpose or was it possibly a mistake? How should I go about re-adding my tests to the refactored files? Thanks!

Yes that why I said this PR will have conflicts with the refactor. Probably we missed the test, so if you can re-add it that would be awesome

@enahum
Okay I'll add in the missing test. I just need some help due to the fact that I need to call the function handleTextChange which I noticed was moved to post_input but need to test things in post_draft. I'm not very familiar how to call the function in that file but then test things within post_draft. This is my only blocker when I was trying to re-add the tests back in.

Just ask in the mobile channel

Due to the mess this PR has become with the history and the conflicts, I'm deciding to close it in favor of a fresh PR made from master https://github.com/mattermost/mattermost-mobile/pull/4548 .


Changing the package structure

Moved all the stylesheet files into a new css directory to address #13 and #7, as well as improving compatibility with other projects.

Looks good. Can you please squash a little and remove the version bumping? I will do that before merging into master.

Did the squashing for you. Merged this with 372fa6e695d59b7b14b1ed714557f6a2e9825aab

Released as v0.4.0

Thanks :)

You're welcome! Thanks for the PR


What is the rationale for freezing the Solidity version?

Using a frozen Solidity version makes it easy to bump into issues when integrating Permit2, e.g. https://github.com/Uniswap/permit2/issues/206
What is the rationale for this design choice? Why not allow any versions greater than or equal to 0.8.17, like this?
pragma solidity ^0.8.17;

Or like this:
pragma solidity >=0.8.17 <=0.9.0;

perhaps the sol v0.8.17 lobbyists have an angle here...

A solution for this was merged in #225


vLEI Beta - Get Started no longer works

I'm trying to complete Technical Setup with fresh docker images and containers, but the Get Started command now does nothing.
With a previous set of containers Get Started did work, a credential appeared in my wallet and I could complete Technical Setup.
Please advice.

@psteniusubi Thanks for opening the issue, we have restarted one of the agents as it appears that stopped responding and so your subsequent requests were failing.
You should be able to restart your containers and complete technical setup.

Thanks for your response. However Get Started still does nothing.
First I tried Get Started command after simply starting my stopped container.
Then I tried Get Started command from a fresh image.
Nothing happens. There are no errors or messages shown in the browser or in any of my containers console logs.

We're continuing to look into your issue.
I will update here with further information.

@psteniusubi My co-worker and I have been through the flows now, we can both successfully receive the QVI credential after clicking get started.
Can you try stopping, deleting and re-pulling the containers.
Once the QVI container is running can confirm in Settings -> Address Book the GLEIF AID is EZ921Tv_rdgIiHURHiWwyG8vW6R69QN2sCOYUDUvfCN0 please?

Ran into the same issue today and following the recommendation to stop, delete and re-pull the containers.  I was able to receive the QVI credential after clicking "get started".

Now I think this works
First I tried with existing images by only removing containers. This did not work
Then I remove images and try again. This worked.
Was there an update to the images in between? The tag version has not changed

@psteniusubi there have been no changes to the docker container.


keybindings to cut/copy/paste to/from clipboard

Is there a keybinding for clipboard pastes from external sources such as the browser....atomic-emacs supports local cut/copy paste using regular emacs keybindings but not able to paste stuff from outside (without using the right click mouse->paste)
The way Emacs seems to handle it is through M-x clipboard-yank and M-x clipboard-kill-region.

update: am able to copy text from atom to outside, and have updated the title to reflect this. The issue is with copying text from external sources to atom buffer.

Which OS are you on? What do you see in the key binding resolver (cmd-. on mac) when you use the standard OS paste key (e.g. cmd-v on mac)?

Ubuntu 14.04
On the key binding resolver and pressing Ctrl-v, the first 2 rows I see are

Sorry for the slow response here.
That explains it, though: the problem is that emacs' ctrl-v (scroll-up) conflicts with what you'd expect of a regular Linux desktop app (paste). I just tested with Emacs on Ubuntu (16.04), and it too favors scroll-up for ctrl-v, so I'm inclined to keep this behavior, at least as the default.
If you're just after direct a substitute for M-x clipboard-yank, you can do alt-x core:paste.
If you want to permanently override ctrl-v to the Atom default, you can add this to your personal keymap:
'atom-text-editor:not(.autocomplete-active):not(.mini)':
  'ctrl-v': 'core:paste'

In my experience, in current versions of Emacs, when I cut or copy something in another program, it is inserted at the top of the kill ring, so C-y pastes it.  The kill ring here needs to work the same way.

I agree with @MartyGentillon
Even when I start emacs with emacs -Q it integrates with system clipboard and I can copy and paste text from other apps to emacs (and vice vesa) seamlessly.
@oggy please consider adding this feature to atomic-emacs

Tomorrow I should have a pull request for this.

An initial implementation https://github.com/avendael/atomic-emacs/pull/130.  There are a couple concerns: first, when yanking to multiple cursors, a multi-line system clipboard should be split between the cursors based on lines.  (The current behavior is similar to my complaint in https://github.com/sublime-emacs/sublemacspro/issues/176)  Second, I am not certain that the "kill-ring per cursor" design is good.  It might be better to have one unified kill ring that is aware of multiple cursors, and multiple cursor yanks.

So, the basic feature is now available.  Todos include possibly rethinking the kill-ring-per-cursor feature such that cuts from other programs integrate the way they would with the normal cut / paste feature (specifically where if you paste two lines of text with two cursors, each line goes to one cursor.)

Closing this, as the requested functionality is there now - thanks @MartyGentillon .

Just found atomic-emacs from this issue. Thanks for having a fix for one of my biggest complaints of atom.

Hi,
I'm using version 0.15 right now, and the external copy doesn't insert into the kill ring. Am I missing something?


image optmizations and css compressed

Did a few things:

Compressed svgs
Switched main logo to svg, reduced ~100b and is responsive for hidpi
Used inline svg with jekyll include. Reduces requests, also allows for color variation in the header logo with a single file.
Compressed CSS (sass output)

Tests fixed. Ready to be reviewed and pulled.

Didn’t test yet, but looking through the code and improvements it looks good! Also good on removing the superfluous alt text. :+1:

Looks good!

Great change!


Fix buttons styles

Thanks for submitting a PR! Please check the boxes below:

[ ] I have run pre-commit to check linting
[ ] I have filled in the "Changes" section below?
[ ] I have filled in the "How did you test this code" section below?

Changes
⋅ added common button props to _new-variables
⋅ fixed modals
How did you test this code?

Please describe.

Labels - old boostrap added a margin bottom of $spacer

List Item Links - they seem to be using the button link component which has a lighter font weight, these are normally bold

Outline button is 50px in height, regular is 48 - probably need a border on all buttons and to reduce the height by 2px


Contour lookups

In this PR:

If we have a contours data source we can process a file, and the results seem to make sense.
A decent start on a useful README

To come in future PRs:

Ability to use SRTM where we don't have contours
A useful CONTRIBUTING.md

Today's work sped up contour lookups considerably: processing the montlake sample data file now takes ~4 minutes on my laptop, from 18-19 yesterday.

Nice! 4 minutes is still a bit slow, but I think there are ways to mitigate that on my end. I also imagine parallelizing the queries could be an easy win, since there's no mutations to the geopandas data happening.

I asked this in the comments on https://github.com/eldang/elevation/pull/3 but I'll repeat it here lest that get lost in a closed PR: does it make more sense for me to add SRTM next, or do  the parallelising of lookups?


MD-ATP/Alerts: copy-paste error correction

Description:
As reported in issue ticket #6541 (lastUpdateTime), the description text for lastUpdateTime is identical to the entry for firstEventTime.
This is not a correct description for lastUpdateTime.
Thanks to @harveywargo2 (Harvey Wargo) for reporting this.
Changes proposed:

Replace the misplaced copy-pasted text with a suitable description.
Remove 5 end-of-line blanks (whitespace correction).

The page author is, of course, invited to improve the phrasing.
Ticket closure or reference:
Closes #6541

#sign-off


[SPARK-47292][SS] safeMapToJValue should consider null typed values

What changes were proposed in this pull request?

Additional null check to the safeMapToJValue. Normally we won't create a StreamingQueryProgress with map fields as null. It is also very unlikely in Spark Connect, but it is theoretically possible because we send the json directly in Spark Connect, so add this check for additional safety to not crash the server.
Why are the changes needed?

Minor bug fix
Does this PR introduce any user-facing change?

No
How was this patch tested?

Added unit test
Was this patch authored or co-authored using generative AI tooling?

No

CC @HeartSaVioR PTAL, thank you!

Merged to master.

Late +1. Thanks @WweiL !


build upload does not handle paths and binary files correctly

prelude build upload <path> returns an error when pointing it at a path or a binary file.
Expected:
The cli complains that the file type is not supported or a path to a file must be used.
Actual:
utf-8 error mesage.

This should be solved when the new "wizard" PR is merged in (shortly).

@VVX7 can you retest?

Yep, working as expected!


inmanta-dev-dependencies bump breaks inmanta-support

Seems to be a timeout in tests/test_cli.py::test_collect_from_server

Hey team! Please add your planning poker estimate with Zenhub @arnaudsjs @FloLey @Hugo-Inmanta

I suspect it is this:
tests/test_cli.py::test_collect_server_status
  /home/jenkins/workspace/nmanta-extensions_support_master/project/inmanta-support/.tox/tests/lib/python3.11/site-packages/pytest_asyncio/plugin.py:451: DeprecationWarning: pytest-asyncio detected an unclosed event loop when tearing down the event_loop
  fixture: <_UnixSelectorEventLoop running=False closed=False debug=False>
  pytest-asyncio will close the event loop for you, but future versions of the
  library will no longer do so. In order to ensure compatibility with future
  versions, please make sure that:
      1. Any custom "event_loop" fixture properly closes the loop after yielding it
      2. The scopes of your custom "event_loop" fixtures do not overlap
      3. Your code does not modify the event loop in async fixtures or tests

    warnings.warn(


Restart Stream If RTSP back online

Hi I'm a beginner . I want to make reconnection if RTSP back online without restarting node server, i've tried with PM2 and forever but it restart the node server and ffmpeg terminal keeps popping out. Help pls

this is working out of box for me. if i kill the server and run again the stream continues automatically.

This seems to be resolved so I am closing. Feel free to open another issue if need be.

hello @kyriesent
I tested with the situation, while node server was processing so RTSP was down(about 2-3 minutes) after that it back online node server still working, I can see log information in terminal but jsmpeg can not show anythings on the clients
pls help! thanks


feat: Add initcode limit to be double of bytecode limit

Set initcode limit from EIP-3860 to be double of ma bytecode size if custom max bytecode size is set inside env.cfg.limit_contract_code_size.

I guess that makes sense, this would also prevent a breaking issue

This looks a little bit nicer to me, but if needed we can add an additional limit.


NimBLE AFR: Fix Notification/Indication tx handling

NimBLE AFR: Fix Notification/Indication tx handling
Description
The handling of notifications/indications is done in BLE_GAP_EVENT_NOTIFY_TX where the appropriate status is returned. So, it is not required in prvBTSendIndication
Checklist:

[ x] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ravibhagavandas Can you please take a look

/bot run checks


Docs?

Hello, I could not find the project documentation, api or even a quickstart
Is there a document?

Yes, please add at least basic documentation beyond using with docker, as we do not use docker.

Docker is the best way to package something like ceryx what sort of documentation are you after?
Currently there is a quickstarter guide for running with docker
But I do agree there should be some documentation as I found it quite difficult to get up and running, there seem to be some confusing stale branches and docker image tags.

API

path
methods
params and body request
body response
Auth?
Port?

Quickstart

There are a list of parameters, but all are required? https://github.com/sourcelair/ceryx#configuration
It is not clear to me if I use the dockercompose available in the repository root can be used in production
Is required a host, but if I will test in a local machine? Will work?
The Development comments about the initial state of the application, but not about the steps required for a people configure a local development

Hello everyone. I will reply to each topic separately.
Docs
I see that a confusing part of our documentation is that we do not mention that you have to set up the DNS records pointing to Ceryx, in order to get things working.
We will update the docs within the coming days, since the project has already matured internally for us.
For a quick-start guide take a look at: https://github.com/sourcelair/ceryx#quick-bootstrap.
Packaging
We can definitely distribute ceryx-api as a Python package, but I have no good idea about how we could distribute the server which is based on NGINX, OpenResty and Lua, besides as a Docker image.
Any thoughts? I can't promise we will do so, but I would love to hear any alternatives.

Perhaps we could start with the prerequisites to install for those of use that are not looking to use with docker containers. i.e... versions for OpenResty, Lua, Python, and any required modules. I have gotten it partially running outside of docker, however getting errors that appear to be related to incompatible versions and install locations.
Thank you in advance, this platform has amazing potential.

Could we please in the meantime, get more detail on the setup required for the DNS?

The DNS is required only if you're willing to use domain names as backend targets. Currently, we're using the Docker DNS server since we're using internal DNS records, but if you just use normal DNS records setting something like <IP_ADDRESS> should be fine.

Can you please at least supply some usage examples, how redirect works etc?
Like this one:
http://openresty.org/en/dynamic-routing-based-on-redis.html
I believe this project should even add more abstractions on top of what openresty supplies.
Would it be something you will be considering?
Thanks!

Thanks for the comments everybody. I prepared a PR to improve the documentation in the README: https://github.com/sourcelair/ceryx/pull/57.
It should land in master branch soon, after we are confident with the content.


feat: add property sync functionality for group components

This PR allows <sbb-action-group> and <sbb-link-list> to define properties, which will be applied to the nested elements.
With this it is no longer necessary to apply certain properties to all nested elements as the parent element will take care of it.
<sbb-action-group>:

variant="block" is applied to any nested <sbb-link>
size is copied to nested <sbb-button>

<sbb-link-list>:

variant="block" is applied to any nested <sbb-link>
textSize and negative is copied to nested <sbb-link>

Codecov Report

Merging #1629 (ed94104) into master (ab917b0) will decrease coverage by 1.76%.
The diff coverage is 51.39%.

: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    #1629      +/-   ##
==========================================
- Coverage   54.85%   53.09%   -1.76%
==========================================
  Files          49       79      +30
  Lines        1659     3573    +1914
  Branches      406      959     +553
==========================================
+ Hits          910     1897     +987
- Misses        671     1548     +877
- Partials       78      128      +50

Impacted Files
Coverage Δ

...mponents/sbb-accordion-item/sbb-accordion-item.tsx
0.00% <0.00%> (-37.21%)
:arrow_down:

src/components/sbb-icon/sbb-icon.tsx
91.66% <ø> (ø)

src/components/sbb-image/sbb-image.helper.tsx
62.06% <ø> (-2.45%)
:arrow_down:

src/components/sbb-image/sbb-image.tsx
69.09% <ø> (+6.12%)
:arrow_up:

...mponents/sbb-journey-header/sbb-journey-header.tsx
88.23% <ø> (+1.27%)
:arrow_up:

...onents/sbb-journey-summary/sbb-journey-summary.tsx
80.55% <ø> (ø)

src/components/sbb-link-list/sbb-link-list.tsx
81.57% <ø> (-6.66%)
:arrow_down:

src/components/sbb-link/sbb-link.tsx
60.41% <ø> (-31.02%)
:arrow_down:

src/components/sbb-logo/sbb-logo.tsx
0.00% <ø> (ø)

src/components/sbb-menu-action/sbb-menu-action.tsx
65.21% <ø> (ø)

... and 85 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more