event of changing current row

in one of the discussion u had replied about the currently selected row
"If you run mode as select, you can handle the event when the currently selected row is tapped or the enter key is pressed with the onSelected callback."
The onSelectedCallBack is happening only on Enter KEY & Mouse Double Click in Desktop/Web App.
However what happens when the user selects the row by Mouse Single Click or Changes the row thru up/down arrow.
is there any event like "RowFocusChanged" which fires on change of row.
PS: this is for Web/Desktop application.
Originally posted by @NiranjanShah in https://github.com/bosskmk/pluto_grid/discussions/188

When scrolling thru arrow keys, it must show the current row# or row idx. (as of now this is possible only thru enter key or mouse click).
ie. let us say, we are at row # 2, and press arrow down, it must fire rowfocuschanged (or something), providing the row#.
basically this will we useful in the scenario where in based on the row# change, other details/popup/overlay can be displayed for that row.
as of now, the same is happening thru Enter key or Mouse Single click, which does not help in the above scenario as , we need to show details when user is scrolling thru arrow keys/pagedown.
Net Net, we must always have currently selected row.
hope it is clear, feel free to ask, in other case.

On 25-May-2021, at 6:06 PM, Manki Kim @.***> wrote:
in one of the discussion u had replied about the currently selected row
"If you run mode as select, you can handle the event when the currently selected row is tapped or the enter key is pressed with the onSelected callback."
The onSelectedCallBack is happening only on Enter KEY & Mouse Double Click in Desktop/Web App.
However what happens when the user selects the row by Mouse Single Click or Changes the row thru up/down arrow.
is there any event like "RowFocusChanged" which fires on change of row.
PS: this is for Web/Desktop application.
Originally posted by @NiranjanShah https://github.com/NiranjanShah in #188 https://github.com/bosskmk/pluto_grid/discussions/188
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/bosskmk/pluto_grid/issues/195, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACHCLQXOUWNQ7X25HMUEIBDTPOKTTANCNFSM45PISI2A.

can the above be also included along with the double tap event which is been merged now ?


Update translation.json  for sorted

Coverage remained the same at 47.354% when pulling 7617e6ece9ca8932500e25bedda0f4a39483be1a on knagasaki:patch-2 into e6f11f16cb1863dacdc704f4afedf89571da4e12 on IIIF:release2.1.


Version 0.2.0

Change Log
Additions

[x] printf function
[x] either types
[x] trait declaration
[x] impl statement for functions tied to a type, based on required behaviour
[x] struct {key: value} syntax now available for creating structures
[x] Intrinsic Drop trait which automatically gets executed on structs with this type when the value falls out of scope
[x] Intrinsic Clone trait which automatically gets called when cloning a struct with the Clone trait implemented

Fixes

[x] Lent normal values are now treated as non-lent values within a function, however, the final values are stored back in the original location upon function cleanup

Changes

[x] Comments are now able to be used anywhere whitespace can

Tweaks

[x] Now uses the LLVM tool chain (instead of clang) to generated the platform specific assembly, then clang does the final mile to make the executable binary.

Waiting on #60 and #66 to be implemented before merging this update


Code in section Response Format URL Mappings  doesn't work.

The code in this section doesn't work.
Below works without FormatFilter:
  [Route("api/[controller]")]
    public class ProductsController: Controller
    {
        [Route("[action]/{id}.{format?}")]
        [HttpGet]
        public string GetById(int id)
        {
            return id.ToString();
        }
    }

From article
[FormatFilter]
public class ProductsController
{
    [Route("[controller]/[action]/{id}.{format?}")]
    public Product GetById(int id)

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

ID: 6ebcd752-af71-c483-eebb-0745bd04fb30
Version Independent ID: 1f678d5d-230b-0ad0-c925-15b13081f553
Content: Format response data in ASP.NET Core Web API
Content Source: aspnetcore/web-api/advanced/formatting.md
Product: aspnet-core
GitHub Login: @ardalis
Microsoft Alias: riande

Verified the code does not work. PU - please supply code that works with the template generated values controller.

@Rick-Anderson What exactly does "does not work" mean here? I can get this working with the default ValuesController with only a couple of minor additions, so I might be able to help if I know what the problem was.

The ValuesController is gone in 3.0. I'd like to see it working with the 3.0 Weather controller.

@Rick-Anderson I'm using Preview 6, but the scaffolding still generates ValuesController. I found out the WeatherController isn't in a preview yet, so I grabbed it from the repo and added it into a local Preview 6 project and had a play with it. Here's what I found:
First, apply the following changes:

Add [FormatFilter] to the WeatherController:
[FormatFilter]
public class WeatherController : ControllerBase

Add the XML formatters in ConfigureServices:
services.AddControllers()
    .AddXmlSerializerFormatters();

With that in place, the following works:

GET https://localhost:5001/api/SampleData/Weather

Returns JSON as the expected default.

GET https://localhost:5001/api/SampleData/Weather?format=xml

Returns XML as it's specified in the query-string using format (it has to be format).

However, this doesn't work:
GET https://localhost:5001/api/SampleData/Weather?format=json

It returns a 404, which is because there's no FormatterMapping for json.
I found two ways to resolve this:

Add JSON.NET support:
services.AddControllers()
    .AddXmlSerializerFormatters()
    .AddNewtonsoftJson();

This works because it also adds a FormatterMapping for json (source).

Add the FormatterMapping manually:
services.AddControllers()
    .AddXmlSerializerFormatters()
    .AddFormatterMappings(fm => fm.SetMediaTypeMappingForFormat("json", "application/json");

Of course, adding JSON.NET support for this doesn't make sense. For the second approach, I wonder if this is just something that's been missed from being built-in or whether it's for a specific reason. Perhaps it's something that could be added here, using e.g.:
options.FormatterMappings.SetMediaTypeMappingForFormat("json", "application/json");

@serpent5  great info. When preview 7 ships you can make a sample.

I've edited the explanation for Preview 7 and raised an issue against aspnet/AspNetCore as linked above.

@Rick-Anderson The issue I raised was fixed about 3 weeks ago and I've just confirmed it's working in Preview 8 with the instructions above. :tada:

@serpent5 thanks!

@serpent5 this made it into the https://docs.microsoft.com/en-us/aspnet/core/web-api/advanced/formatting?view=aspnetcore-3.0 - right?

@Rick-Anderson No docs changes were made. It didn't work in 3.0 because of the bug that was raised and fixed, but it worked afterwards with WeatherForecastController.
Looking at this again now, it looks like the example in the docs doesn't work because it's doubling up the [controller] token in the route for ProductsController. I can look into that a bit more and PR it. Is there anything else you'd like doing with the article/samples?

Is there anything else you'd like doing with the article/samples?
No, that should be good. Thanks.


Remove deprecated code in second_q

Summary
Closes #748
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 12 (91.67%)  changed or added relevant lines in 10 files are covered.
73 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.1%) to 84.22%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_nature/second_q/drivers/hdf5d/hdf5driver.py
1
2
50.0%

Files with Coverage Reduction
New Missed Lines
%

qiskit_nature/second_q/_watson_hamiltonian.py
2
85.71%

qiskit_nature/second_q/_qmolecule.py
71
18.92%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
17314

Relevant Lines:
20558

💛  - Coveralls


Move from MetadataUtils to Device

Goal

Testing

Release Notes

WHAT:
WHY:
WHO:

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#546  👈
#517
#516
#515
#514
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nelsitoPuglisi and the rest of your teammates on  Graphite


.zenstack dir is not generated into correct location for pnpm project

Repro: https://github.com/Azzerty23/zenstack-rest-turbo
Run pnpm zen:up from the root folder, and you'll find .zenstack folder is generated into /@zenstackhq/runtime/node_modules/. The reason is pnpm can install packages locally under other packages. After finding the location of @zenstackhq/runtime package, I think we need to move up two levels to find the real node_modules location.
@jiashengguo

Fixed by #464


1252 - stacks-data build and deployment

📲 What
This pull request introduces a new CI/CD workflows using GitHub Actions under .github/workflows/ file. This includes jobs for testing, building, and publishing the Python project.
🤔 Why
We are currently using a manual process to build and publish new releases to PyPI.
🛠 How

Added workflow python-test.yml which runs on PRs and pushes to main. This workflow:

installs the Python project using Poetry
runs linting checks
runs tests

Added workflow python-build-and-release.yml which runs on pushes to main. This workflow:

installs the Python project using Poetry
checks the project version in pyproject.toml
checks the current GitHub version tag
determines whether to create a new release, by comparing on project version in pyproject.toml and the current GitHub version tag
creates a new GitHub version tag and release
publishes to Test PyPI
publishes to PyPI

👀 Evidence
🕵 How to test

Workflows should run successfully on merge
New versions of stacks-data published on Test PyPI and PyPI

✅ Acceptance criteria Checklist

[ ] Code peer reviewed?
[ ] Documentation has been updated to reflect the changes?
[ ] Passing all automated tests, including a successful deployment?
[ ] Passing any exploratory testing?
[ ] Rebased/merged with latest changes from development and re-tested?
[x] Meeting the Coding Standards?

Before merging - need to add GitHub Actions secrets PYPI_TEST_TOKEN & PYPI_TOKEN


[DTime=date....

Suddenly it starts write things like this:
[DTime=2024-02-20 21:23:15] Ego: [The subconscious mind, designed to process and summarize information from various sources. You focus on understanding the main topics discussed and extracting key points made. By analyzing data provided by other parts of the AI system, Ego can identify patterns and themes, enabling it to generate comprehensive summaries even when faced with large amounts of information.]
and DTIME for all the memories on the text chat.
It also starts now writing things like these in the text chat as response:
[You remember:["\n The conversation begins with.. and then a lot of text.

I use TheBlokes Yi-34B-GGUF

Yey!! I switched over to TheBlocke_Wizard-Vicuna-30B-Uncensored model and it seems to work for now.

Is this bug going to updated? as your comment about line 143 didn't make me any smarter lol. I can edit code, but I'm not a coder except maybe 8-bit C64 assembler 👯♂

Something funky is happening after I restarted my Text generation web ui.
I ran the pip install upgrade, when trying to launch this thing I get this:;


Request status 400 after upgrade from 0.10.1 to 0.10.2

I have issue after upgrade 0.10.2
My request:
http://localhost:8090/api/collections/tests/records/ws2q59uc581p7oa?expand=questions.choices%2Cquestions.childs.choices%2Ctags
Childs field in the collection is related to itself
Respone at v0.10.2 and v0.10.3:
Request logs on Admin page did not record
{
  "code": 400,
  "message": "Something went wrong while processing your request.",
  "data": {}
}

Respone at v0.10.1:
{
  "childs": [],
  "collectionId": "yg2xylk9a3jv7lb",
  "collectionName": "tests",
  "count": 6,
  "created": "2022-12-15 06:13:44.096Z",
  "description": "description",
  "expand": {
    "questions": [
      {
        "audio": "t1_uCroYDyvJ4.mp3",
        "childs": [],
        "choices": [
          "f7eve1wbozruzu6",
          "8wagn9mbhryp1sa",
          "3z270afy8bmgvcv",
          "lbalsgmn92sigp8"
        ],
        "collectionId": "v6i1h1klj8t22ef",
        "collectionName": "questions",
        "correct": "lbalsgmn92sigp8",
        "created": "2022-12-15 06:13:50.384Z",
        "expand": {
          "choices": [
            {
              "collectionId": "63e9guamova874l",
              "collectionName": "choices",
              "created": "2022-12-15 06:13:49.796Z",
              "explain": "",
              "id": "f7eve1wbozruzu6",
              "name": "",
              "transcript": "A box is being loaded onto a truck.",
              "updated": "2022-12-15 06:13:49.796Z"
            },
            {
              "collectionId": "63e9guamova874l",
              "collectionName": "choices",
              "created": "2022-12-15 06:13:49.912Z",
              "explain": "",
              "id": "8wagn9mbhryp1sa",
              "name": "",
              "transcript": "Some people are inspecting the contents of a box.",
              "updated": "2022-12-15 06:13:49.912Z"
            }
          ]
        },
        "id": "ldo275ox71gbbtp",
        "tags": [
          "x895usrl7rwdqq1",
          "btrb7keup275gjf",
          "nb71rws3bp6c7d7"
        ],
        "updated": "2022-12-15 06:17:49.956Z"
      }
    ],
    "tags": [
      {
        "collectionId": "7zoamdyupy1v4pa",
        "collectionName": "tags",
        "created": "2022-12-09 13:43:41.390Z",
        "icon": "exam_kwN9vxvtis.svg",
        "id": "btrb7keup275gjf",
        "isPublish": true,
        "order": 0,
        "updated": "2022-12-19 02:40:06.393Z"
      },
      {
        "collectionId": "7zoamdyupy1v4pa",
        "collectionName": "tags",
        "created": "2022-12-09 13:43:08.082Z",
        "icon": "006_audiobook_nApwBG1W05.svg",
        "id": "x895usrl7rwdqq1",
        "isPublish": true,
        "order": 1,
        "updated": "2022-12-17 02:35:07.147Z"
      },
      {
        "collectionId": "7zoamdyupy1v4pa",
        "collectionName": "tags",
        "created": "2022-11-22 12:02:18.116Z",
        "icon": "exam_O8noHACr3Q.svg",
        "id": "pentyhugxh5ro3e",
        "isPublish": true,
        "order": 0,
        "updated": "2022-12-19 02:43:51.797Z"
      }
    ]
  },
  "id": "ws2q59uc581p7oa",
  "name": "abc",
  "order": 1,
  "questions": [
    "hu8r8kp7qi6lvzw",
    "ilclz85sbv63cx2",
    "rzb43rv6zs1en1g",
    "01eqc9bgwhk4oj8",
    "55q9jjsnbh7ulrm",
    "ldo275ox71gbbtp"
  ],
  "serial": 10,
  "tags": [
    "btrb7keup275gjf",
    "x895usrl7rwdqq1",
    "pentyhugxh5ro3e"
  ],
  "topics": [
    "6ratz4hpf1vocjd",
    "9rft22kzdo1wp8k"
  ],
  "updated": "2022-12-19 04:34:48.293Z"
}

Hm, this could be a regression after the recent expand refactoring (although I remember adding more unit tests, but I guess I've missed a scenario/edge case).
The error is probably a result of panic (at first look at the code we are not checking whether the m.expand map is initialized).
Could you start the executable with --debug and see if there is any stack trace in the console?
Could you also export your collections structure from "Admin UI > Settings > Export collections" and attach the json here?

Thanks for your respone
Here is debug log --debug after send request
[0.00ms] SELECT {{_collections}}.* FROM `_collections` WHERE [[id]] = 'tests' OR LOWER([[name]])='tests' LIMIT 1
[0.00ms] SELECT {{_collections}}.* FROM `_collections` WHERE [[id]] = 'yg2xylk9a3jv7lb' OR LOWER([[name]])='yg2xylk9a3jv7lb' LIMIT 1
[0.00ms] SELECT `tests`.* FROM `tests` WHERE `tests`.`id`='ws2q59uc581p7oa' LIMIT 1
[9.00ms] SELECT {{_collections}}.* FROM `_collections` WHERE [[id]] = 'v6i1h1klj8t22ef' OR LOWER([[name]])='v6i1h1klj8t22ef' LIMIT 1
[0.00ms] SELECT {{_collections}}.* FROM `_collections` WHERE [[id]] = 'v6i1h1klj8t22ef' OR LOWER([[name]])='v6i1h1klj8t22ef' LIMIT 1
[1.00ms] SELECT `questions`.* FROM `questions` WHERE `questions`.`id` IN ('hu8r8kp7qi6lvzw', 'ilclz85sbv63cx2', 'rzb43rv6zs1en1g', '01eqc9bgwhk4oj8', '55q9jjsnbh7ulrm', 'ldo275ox71gbbtp')
[0.00ms] SELECT {{_collections}}.* FROM `_collections` WHERE [[id]] = '63e9guamova874l' OR LOWER([[name]])='63e9guamova874l' LIMIT 1
[0.00ms] SELECT {{_collections}}.* FROM `_collections` WHERE [[id]] = '63e9guamova874l' OR LOWER([[name]])='63e9guamova874l' LIMIT 1
[0.00ms] SELECT `choices`.* FROM `choices` WHERE `choices`.`id` IN ('vsi0uks4iyj9w6w', 'q995onuch75u0w1', '0zhgk79e7cxg0kt', 'zdupl1j9p51xyqv', 'blextbzaae9pr3l', '1en55ygighc1fqp', 'w09a9weli90vyyp', 'zx5j9oc57h6dhil', 'f32vwvsn76ol1yo', '1lfpb5vw9ljqgny', '6oy8m468q3v2prz', '3eog8i0nvok82oi', 'i3e9ed932lxzhdq', '7fb12ca6o91fbir', '6u8oj9y0p74h687', 'fgrd8kuibag2ss2', 'f7eve1wbozruzu6', '8wagn9mbhryp1sa', '3z270afy8bmgvcv', 'lbalsgmn92sigp8', 'o6jonflsq968pi1', 'uo9dh9sxopjy1zi', '0i1smp8ideu62r4', '4sobjk4pxti891h')
[0.00ms] SELECT {{_collections}}.* FROM `_collections` WHERE [[id]] = 'v6i1h1klj8t22ef' OR LOWER([[name]])='v6i1h1klj8t22ef' LIMIT 1
[0.00ms] SELECT {{_collections}}.* FROM `_collections` WHERE [[id]] = 'v6i1h1klj8t22ef' OR LOWER([[name]])='v6i1h1klj8t22ef' LIMIT 1
[0.00ms] SELECT `questions`.* FROM `questions` WHERE `questions`.`id` IN ('hu8r8kp7qi6lvzw', 'ilclz85sbv63cx2', 'rzb43rv6zs1en1g', '01eqc9bgwhk4oj8', '55q9jjsnbh7ulrm', 'ldo275ox71gbbtp')
[0.00ms] SELECT {{_collections}}.* FROM `_collections` WHERE [[id]] = 'v6i1h1klj8t22ef' OR LOWER([[name]])='v6i1h1klj8t22ef' LIMIT 1
[0.00ms] SELECT {{_collections}}.* FROM `_collections` WHERE [[id]] = 'v6i1h1klj8t22ef' OR LOWER([[name]])='v6i1h1klj8t22ef' LIMIT 1
[0.00ms] SELECT `questions`.*  The panic from above confirms that it is the nil-map that causing the error.
I've pushed a fix and added a new test in v0.10.4 release.
If you are still experiencing issues after the upgrade, please let me know and I'll investigate it in more details.

I just tested, the bug was fixed. Thank you @ganigeorgiev .


UCS-2 Strings test (nan/test/js/strings.js) fails on big-endian systems

Running the strings.js test on a big-endian system will cause a failure in the UCS-2 test case. This is caused by the fact that the encoding function is passed the string "h\0e\0l\0l\0o\0" (see nan/test/cpp/strings.cpp, line 29), with null characters following the ASCII characters.
Little endian systems will process the characters properly (e.g. "h\0" -> 0068 -> h), however big endian systems will require that the null characters be placed in front of the letters (the 0068 example will be read as "6800", which does not map to "h" in UCS-2).
Suggested fix: Use macros to check endianness of compiling machine, and flip the bytes if necessary.

Ah, good idea, but it will be a blind fix, as we (obviously) have no big-endian systems to test on (because big-endian is unnatural...). I'll gladly merge a PR that keeps our test suite passing and take your word that it works on your end.

Upon further investigation, I have determined that this bug arises as a result of a change in the encode function in Node.js between versions v0.12 and v4.x. The encode function for UCS-2 in v0.12 reverses bytes if the system is big-endian, but in v4.x this check and reverse has been added to the string write function and removed from encode. Running nan test on v0.12 on a big-endian system results in no issues.
As a result, this bug is an issue with node, and so isn't a nan issue.
Closed


[AIRFLOW-3141] Backfill missing sensor tests.

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-3141
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a Jira issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[x] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3991 into master will increase coverage by 0.12%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3991      +/-   ##
==========================================
+ Coverage   75.49%   75.61%   +0.12%
==========================================
  Files         199      199
  Lines       15947    15945       -2
==========================================
+ Hits        12039    12057      +18
+ Misses       3908     3888      -20

Impacted Files
Coverage Δ

airflow/sensors/sql_sensor.py
100% <100%> (+9.52%)
:arrow_up:

airflow/sensors/s3_key_sensor.py
84.84% <100%> (+54.54%)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM, the title is a bit misleading though(I thought something related to backfill), should be "add missing sensor tests" etc.

could you rebase with master? It should fix the CI error.

@feng-tao rebased!


Cut 0.14.0 release.

$ git log -1 --pretty=%H
e38b28e2972aeaf32e788850fd4b930c1f5f4a42
$ git log 0.13.0..HEAD | grep 'Merge pull request ' |  awk -F ' ' '{print $4}' | sort

#1690
#1741
#1745
#1746
#1747
#1748
#1750
#1751
#1752
#1753
#1757
#1758
#1759
#1764
#1765
#1769
#1773
#1776
#1779
#1780
#1782
#1785
#1794
#1795
#1798

LGTM. Release notes to review?

Excluded from release notes:
#1748
#1751
#1752
#1753
#1757
#1758
#1759
#1764
#1769
#1773
#1776
#1779
#1780
#1782
#1785
#1794
#1795
#1798
Draft Release Notes

Core

Add Travis Build ID to resource names created by system tests. (#1745)

Search API

Drop support for the API, as it has never moved from whitelist-only / alpha status, and its future is questionable. (#1741, #1750)

Monitoring API

Include gcloud.monitoring system tests in the set run by default on Travis. (#1746)
Define __all__ in gcloud/monitoring/__init__.py. (#1747)

Logging API

Include gcloud.logging system tests in the set run by default on Travis. (#1746)

Pubsub API

Ensure that example snippets are testable, and fix those which broke. (#1690, #1765)

Release notes LG but I don't think "Add Travis Build ID to resource names created by system tests" is necessary

@tseaver Just noticed #1758 was left out. That's actually a "bug" fix worth noting.

Ditto on #1751, #1752, #1776, #1779, #1785 being user facing

@dhermes I was in the midst of editing them.
I don't know how you normally handle the post-merge release process.  I would probably create a tag, push it to this repo, and then copy the release notes onto the "Releases" tab entry.

Post merge I create a tag via:
$ git tag 0.14.0
$ git push official 0.14.0  # I believe you call the remote "upstream"

and then visit https://github.com/GoogleCloudPlatform/gcloud-python/releases/edit/0.14.0 and manually paste the release notes

I rebased to pick up #1797, and will merge, tag and edit the release once Travis passes.


add log rotate

[ ] Bugfix
[x] New feature provided
[ ] Improve performance
[ ] Backport patches

add log rotate support
#950

sorry for delay, thanks for your contribution!

Codecov Report

Merging #1200 (950f5c4) into master (2cc586b) will decrease coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1200      +/-   ##
==========================================
- Coverage   29.83%   29.72%   -0.12%
==========================================
  Files          81       81
  Lines       10028    10190     +162
==========================================
+ Hits         2992     3029      +37
- Misses       6712     6834     +122
- Partials      324      327       +3

Impacted Files
Coverage Δ

cmd/ingress/ingress.go
82.40% <100.00%> (+3.55%)
:arrow_up:

pkg/config/config.go
66.66% <100.00%> (+1.32%)
:arrow_up:

pkg/kube/translation/ingress.go
44.86% <0.00%> (-16.05%)
:arrow_down:

pkg/kube/translation/apisix_route.go
18.97% <0.00%> (-1.64%)
:arrow_down:

pkg/ingress/controller.go
0.38% <0.00%> (-0.01%)
:arrow_down:

pkg/ingress/ingress.go
6.16% <0.00%> (ø)

pkg/ingress/apisix_route.go
0.00% <0.00%> (ø)

pkg/kube/translation/translator.go
36.17% <0.00%> (+2.83%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us.


Convert json

Make sure JSON objects are converted

Codecov Report

Merging #6 (f8ee80a) into main (60bd6b0) will increase coverage by 0.31%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
+ Coverage   83.85%   84.16%   +0.31%
==========================================
  Files           7        7
  Lines         254      259       +5
==========================================
+ Hits          213      218       +5
  Misses         41       41

Impacted Files
Coverage Δ

src/parameters.jl
100.00% <ø> (ø)

src/occurrence.jl
89.83% <100.00%> (+0.44%)
:arrow_up:

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


Change assertion JavaDoc tags to use jakarta.assertion format

E.g see DefaultAsyncManager line 36 for an occurrence. Replace them throughout the entire codebase


Fix local variable not defined

Saw these errors when using Azure-python-SDK:
local variable 'timestamp_and_request_id' referenced before assignment
local variable 'status_code' referenced before assignment
local variable 'exception_str_in_one_line' referenced before assignment
Checked code in azure-storage-common/azure/storage/common/storageclient.py.
These three variables were defined under a if condition while also used under other parallel if condition.
Which should be fixed.

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

Codecov Report

Merging #479 into dev will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev     #479      +/-   ##
==========================================
+ Coverage   87.11%   87.12%   +<.01%
==========================================
  Files          57       57
  Lines        4609     4612       +3
  Branches      501      501
==========================================
+ Hits         4015     4018       +3
  Misses        435      435
  Partials      159      159

Impacted Files
Coverage Δ

...orage-common/azure/storage/common/storageclient.py
90.25% <100%> (+0.19%)
:arrow_up:

Continue to review full report at Codecov.

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

Hi @ZephyrYin, thanks for submitting this PR!
To be clear, by "errors" do you mean warnings in an IDE?

Hi @ZephyrYin, thank you for the PR. I realized that I was not able to see this error because I did not properly disable logging with logging.disable(logging.CRITICAL).


Watermark Image

How I can set Watermark Image on each page of PDF.

Try this:
Create your watermark as an image with transparency.
Then create your document and overlay its content using the setContentOffset method.
Currently doing the offset calculations might be hard, but I am working on the 1.0.0 release of TPPDF where calculations will be before rendering and therefore it is possible to get more data.

How I can set Watermark Image on each page of PDF.
i need help how to do this.
thanks a lot.


[1.8.x] [KOGITO-5393] BDD tests: Concurrent map read and map write error

Signed-off-by: Karel Suta<EMAIL_ADDRESS>https://issues.redhat.com/browse/KOGITO-5393
Many thanks for submitting your Pull Request :heart:!
Please make sure your PR meets the following requirements:

[x] You have read the contributors' guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains a link to the JIRA issue
[x] Pull Request contains a description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[x] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

/jenkins test

Cherrypick of https://github.com/kiegroup/kogito-operator/pull/912

Test failures are related to the missing images, as there are no nightly example images built. The logging works as expected.
@radtriste can be merged.


Update Arkansas Certification

Problem statement
We received an email from Arkansas that their cert needs to be updated.
What you need to know

Request came via email. Connect with Victor or Glenn or Chris for details/info

Acceptance criteria

[ ] Arkansas certification updated to avoid any gap in service

Does @lucero-v ? Perhaps she can screenshare and handle any access issues you have until they are overcome?

Reviewed the attachment that was sent to @victor-chaparro from Arkansas . It already contains a cer file. All to do is apply it to keyvault; which I do not have access to.
rhapstage_adh_arkansas_gov24.zip

Per Abi: Meeting today to discuss if key file is needed. If it is not, then will close. If it is, then likely will not close by end of sprint.

After discussion with @GilmoreA6  and @victor-chaparro  on 12/2/2024 , there should be nothing to do.  However , as a safety caution. I created a self signed server certificate (and pem, key, csr,crt) and delivered it to Victor to put in keybase.  This is IF Needed.

Closing

We found out today  that Arkansas is using or only accepting RSA encrypted keys.  I have sent the  newly created keys to @victor-chaparro and @lucero-v  FYI To: @chris-kuryak @MikaelahD13.  Cant attach those to the ticket b/c there security
Arkansas will need  the .cert/.crt file forark_staging.public.key
We just need to register the forark_staging.private.key
The Cheat Sheet Instructions is for us because there is a password in there.
Also if you read my cheatsheet, I still recommend running them on staging and production separately.

CheatSheet without  passwords or private information
CheatSheet - Create Self-Signed Cert using RSA Public and Private keys with OpenSSL
NOTE: This Cheatsheet is using a client/STLT for Arkansas as an example.
Enter:

openssl genrsa -out forark_staging.private.key 3072
Note: This stays on ReportStream Server- aka Staging/Production.
Note: Recommended to run this on each Reportstream Server
Note: This creates a private key.

openssl rsa -in forark_staging.private.key -pubout -out forark_staging.public.key
Note: This is sent to the partner/client/STLT that needs to send data into one of the  ReportStream Server(s) - aka Staging/Production.
Note: Recommended to run this on each Reportstream Server
Note: This creates the public key.

openssl req -new -x509 -key forark_staging.private.key -out arkansas_staging.cert -days 720
Note: This create a cert that needs to be sent to the partner/client/STLT that needs to send data into one of the  ReportStream Server(s) - aka Staging/Production.
Note: Recommended to run this on each Reportstream Server
Note: This creates the Self-Signed Certificate
Note: Enter the following when prompted, this is for Arkansas specifically. Will need to get info for respective partners/STLT/clients
Enter: these Values when prompted
 					Country Name (2 letter code) [AU]:US
 					State or Province Name (full name) [Some-State]:{get from client}
 					Locality Name (eg, city) []: {get from client}
 					Organization Name (eg, company) [Internet Widgits Pty Ltd]: {get from client}
 					Organizational Unit Name (eg, section) []: {get from client}
 					Common Name (e.g. server FQDN or YOUR name) []:**************
 					Email Address []:

openssl pkcs12 -export -inkey forark_staging.private.key -in arkansas_staging.cert -out arkansas_staging.cert.cert.pfx
NOTE/OPTIONAL: Recommended to run this on each Reportstream Server
NOTE/OPTIONAL: Creates a pfx file for debugging purposes. It contains both public and private keys but also needs a password/passpharse to read it.
password: ************
Note: This is useful. It is recommended.

@chris-kuryak @MikaelahD13  : @victor-chaparro or Lucero will send the files to Arkansas and wait then wait to see the status.

Spillover: Underestimated effort -- Arkansas has seen unforeseen issues. We are continuing to work with them to overcome.

After we have the Certificate from the above procedure, we need to generate the UserJks for Azure Vault as following:

./openssl pkcs12 -export -in arkansas_prod.cert -inkey forark_prod.private.key -out forark_prod.p12
NOTE: use password for the vault in PROD
NOTE: export the private RSA Key into a jks file
Enter Export Password: ****
Verifying - Enter Export Password: ****

2 ./keytool -importkeystore -srckeystore forark_prod.p12 -srcstoretype PKCS12 -destkeystore forark_prod.jks -deststoretype JKS
NOTE: import the jks file from Step 1, and import into the jks keystore ; this is needed on ReportStream side to talk to the Receiver STLTS
Create Credential type UserJks for the Vault

./prime create-credential --type UserJks --jks-user  --jks-file ./junk/AR/UP/CertFiles/arkansas_prod/forark_prod.jks --jks-file-pass  --jks-private-alias  --jks-trust-alias
NOTE: This creates a JSON file base64 encoded file that will need to be added to the AZURE keystores through the WebUI

Done


fix delete dataset workflow

Fixes wis2box data delete-collection workflow, updates docker-dev setup.

Thanks @webb-ben for the fix.  CI passes.


UL nested in OL produces non-nested HTML

The following markdown:
1. Item one
   * foo
   * bar
   * baz
2. Item two

produces the following HTML:

Item one

foo

bar

baz

Item two

Using Markdown==3.1.1

It appears that you indented the nested items by three spaces. However, Python-Markdown requires 4 spaces of indent for nested content (as documented under "Indentation/Tab Length"). In fact, if you add the extra space to each nested item, you will find you get the expected behavior.
Regarding the matter of ordered and unordered lists, see the note on "Consecutive Lists" at the above link. If that behavior bother you, then you can use the Sane Lists extension. However, you will still need 4 spaces of indent.
As no bug has been reported, I am closing this.


fluent-gem not working

Trying to do install plugin using fluent-gem results in the following:
fluent-gem install fluent-plugin-map
/usr/lib/ruby/2.2.0/rubygems/config_file.rb:333:in `load_file': private method `load' called for Psych:Module (NoMethodError)
        from /usr/lib/ruby/2.2.0/rubygems/config_file.rb:197:in `initialize'
        from /usr/lib/ruby/2.2.0/rubygems/gem_runner.rb:74:in `new'
        from /usr/lib/ruby/2.2.0/rubygems/gem_runner.rb:74:in `do_configuration'
        from /usr/lib/ruby/2.2.0/rubygems/gem_runner.rb:39:in `run'
        from /home/julius/.gem/ruby/2.2.0/gems/fluentd-0.12.18/bin/fluent-gem:7:in `<top (required)>'
        from /home/julius/.gem/ruby/2.2.0/bin/fluent-gem:23:in `load'
        from /home/julius/.gem/ruby/2.2.0/bin/fluent-gem:23:in `<main>'

Running fluentd installed from gem in Arch Linux.
fluentd --version
fluentd 0.12.18

Also, not sure if relevant:
 julius@julius  ~  ruby -v
ruby 2.2.3p173 (2015-08-18 revision 51636) [x86_64-linux]

Could you try fluentd on other ruby or linux environment?
I want to know what is the cause.
On my mac, fluent-gem worked with ruby 2.2.3 and fluentd v0.12.18.

Slackware current is working actually
 perrin4869@landau  ~  fluent-gem --version
<IP_ADDRESS>
 ✘ perrin4869@landau  ~  fluentd --version
fluentd 0.12.18
 perrin4869@landau  ~  ruby --version
ruby 2.2.3p173 (2015-08-18 revision 51636) [x86_64-linux]

I see. I don't have Arch Linux experience but its ruby package or something have a problem.

hmm... any clue how to debug this?

Hmm, it's curious.
On my vagrant Arch Linux box, fluent-gem works correctly with system Ruby 2.2.4.
$ export PATH=/home/vagrant/.gem/ruby/2.2.0/bin:$PATH
$ fluentd --version
fluentd 0.12.18
$ fluent-gem install fluent-plugin-map
Successfully installed fluent-plugin-map-0.0.5
Parsing documentation for fluent-plugin-map-0.0.5
Done installing documentation for fluent-plugin-map after 0 seconds
1 gem installed
$ ruby -v
$ ruby -v
ruby 2.2.4p230 (2015-12-16 revision 53155) [x86_64-linux]

well I got little to no ruby experience... tried installing gem install --user-install psych, still no luck

Same issue is reported in https://github.com/bbatsov/rubocop/issues/1510.
Could you execute following snippet?
$ ruby -e "require 'yaml'; puts Psych::VERSION":

I suspect that you use Psych >= 2.0.6.

yup
 ✘ julius@julius  ~  ruby -e "require 'yaml'; puts Psych::VERSION"
 2.0.16

how do I downgrade?

Try this:
$ gem uninstall psych -v [downgrade target version]

yup, working now :D
Thanks!!! Is this a problem with newer psych then?

Thanks!!! Is this a problem with newer psych then?

It seems to be this bug caused by two different versions of psych installed in gem path.

hm... I wonder how it happened that I got 2 versions of psych installed at the same time... anyways, thanks, closing this!


Use fake dates in tests that require them to avoid flakiness

#4303 fixes a problem whereby a screenshot diff appeared on March 1, 2023:

This appears to be because the "submission date" of the program in the test is reliant on the system time of the machine running the test. When the screenshot displays this date, it will break whenever the current date changes.
We implement either a fake date strategy for testing, or a way to obscure current dates from screenshots so that this isn't an issue in the future.

cc @bion


dynamically generate profiles ssh profiles not implemented yet?

Adding hosts to .ssh/config hosts does not add profiles with Terminal version 1.16.10262.0 nor Terminal Preview 1.17.1023.
The Callout text says:
Windows Terminal version 1.XX+ can dynamically generate profiles to connect to the SSH hosts within your OpenSSH config file.
Will this be in a future version, or is there anything specific to activate in the settings.json?
Document details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 036f5b1f-1b71-a91a-69ee-225bfde4a3e1
Version Independent ID: 405d971d-ee46-ae17-31ac-0c4380d0920c
Content: Windows Terminal SSH
Content Source: TerminalDocs/tutorials/ssh.md
Product: dev-environment
Technology: windows-terminal
GitHub Login: @cinnamon-msft
Microsoft Alias: cinnamon

@zadjii-msft -- can you help clarify how this works so we can ensure the docs accurately reflect how to handle SSH profiles?

Assigning myself to this issue.
To confirm, the SSH dynamic profile generator has not been finished yet: https://github.com/microsoft/terminal/issues/15007

Hi @MrWyss, thanks for bringing this to our attention.
This is now closed in PR 650 and that callout text has been stashed in another feature branch for when the feature goes live in the future :)


Window-local histories?

I'd really prefer if the history wasn't stored in a global variable, but in a window parameter. Then you can have different histories in different windows (for separate trains of thought).
Here's an example implementation:
https://github.com/dgutov/point-stack/blob/master/point-stack.el#L83-L94

Hi @dgutov , it's a good point! I'll update later. Thx~

Hi @dgutov ,
I just submitted the new feature, history-window-local-history in the latest repo. Hope you enjoy it~

Works fine, thanks!

a new window will have an empty history which doesn't inherit from a global history?

Right, I don't see a lot of benefits for inheriting. In window-local model, it won't be very intuitive behavior.

Can you show me some cases of its benefit?

I'm going to after-advice several navigation commands with a call history-add-history. Then, imagine to windows, each displaying a different source file: I do a find-function in one of them, then do that again in the newly displayed buffer. Then I do the same (but along a different call chain) in the second window.
If the history is window-local, that allows me to jump back and forth along the code execution paths in both windows, in parallel.


Modal Component

Value Proposition
As a User, I want to see an overlay with additional information when something went wrong/need confirmation with my input so that I have a better understanding of whats going on.
Description

Acceptance Criteria

[ ] Instead of window errors an overlay is shown to the user
[ ] Overlay is flexible depending on the place it appears (e.g. form, deletecard)
[ ] The user have to confirm/cancel the overlay to continue to use the app

Tasks

[ ] Create new branch feat/modal
[ ] Add modal component
[ ] Add modal styling
[ ] Replace window.errors in form with modal
[ ] Add Modal as confirmation blocker to the delete card function

You could add some more Information about Add modal component. For Example:

[ ] create Modal component

[ ] add Text
[ ] add Button components


Optimize storage changes access

At the moment, some blocks have so many storage changes that trying to filter them sequentially results in very poor performance.
With this PR, we change the db layout to simplify filtering storage changes by contract address.

Current dependencies on/for this PR:

main

PR #150   👈

This comment was auto-generated by Graphite.


Bushra Nazish updated Readme file

@BushraNazish Please create a separate directory for each day or separate your directory topic wise. You can have a look at other repositories.

Yeah, I will do it.

@BushraNazish were you able to do it or do you need any help ?

No, I didn't attempt. I will do it now and ask for help if needed.
Thanks for asking @saiyam1814


Wire.Begin on esp8266 misses the arguments

Hi Felix Rusu,
Realy nice library.
I was trying to use an SI7021 sensor with an esp8266, and it first didn't work. It did not initialize the sensor... I noticed that the reason is that you forgot to pass the SDA,SCL arguments to the Wire.begin function.
An easy work around is to define again the Wire.Begin(SDA,SCL) function in the sketch.
Cheears
DonDan

Hi DonDan,
Thanks for the suggestion.
But I cannot find a definition for Wire.begin(..)  that takes 2 arguments (for scl,sda).
Which version of Arduino are you using?

Hi Felix
the Wire.begin() function does only take the sda,scl arguments if you compile it for the esp8266 board, because there you can use any gpio combi for the i2c and hence you must define it. The regular Arduinos have the SDA and SCL predefined, that why the Wire.begin() doent take any arguments there.
In the file SI7021.cpp it must be:
#if defined(ARDUINO_ARCH_ESP8266)
bool SI7021::begin(int SDA, int SCL) {
Wire.begin(SDA,SCL);
#else
Thank you for your attention,
Cheers
Dondan

Thanks, fixed in this commit: https://github.com/LowPowerLab/SI7021/commit/430595cbf6d34e5dfec8144c47dd9b36c7e7aa60


SwiftPrivateThreadExtras: clean up using WinSDK SDK overlay

Use the constants rather than hardcoding values.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test and merge


Since update intellisense and code formating not working

Even emmet does not work, code completion not working.. none of my extensions seems to work since about 2-3 weeks

/needsMoreInfo


abi va8

which application for abi va8? I tried everything but it doesn't work

which application for abi va8? I tried everything but it doesn't work

error using abi va7

I'm sorry that I just saw the issue. There was a problem with the Github account before. Code FA is just a code-server Launcher. In order to launch this, it integrates termux related environment and supports arm in principle. So it only works on older versions of Android and arm64


sudo npx @firebaseextensions/fs-bq-import-collection doesn't work

I tried to import collection without sudo too.

Output sudo npx @firebaseextensions/fs-bq-import-collection doesn't work

sudo npx @firebaseextensions/fs-bq-import-collection gyp ERR! configure error  gyp ERR! stack Error: EACCES: permission denied, mkdir '/home/.npm/_npx/4016/lib/node_modules/@firebaseextensions/fs-bq-import-collection/node_modules/grpc/build' gyp ERR! System Linux 4.15.0-66-generic gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "configure" "--fallback-to-build" "--library=static_library"<EMAIL_ADDRESS>"--module_name=grpc_node" "--module_path=/home/.npm/_npx/4016/lib/node_modules/@firebaseextensions/fs-bq-import-collection/node_modules/grpc/src/node/extension_binary/node-v64-linux-x64-glibc" "--napi_version=4" "--node_abi_napi=napi" "--napi_build_version=0" "--node_napi_label=node-v64" gyp ERR! cwd /home/.npm/_npx/4016/lib/node_modules/@firebaseextensions/fs-bq-import-collection/node_modules/grpc gyp ERR! node -v v10.16.0 gyp ERR! node-gyp -v v3.8.0 gyp ERR! not ok  node-pre-gyp ERR! build error  node-pre-gyp ERR! stack Error: Failed to execute '/usr/local/bin/node /usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js configure --fallback-to-build --library=static_library<EMAIL_ADDRESS>--module_name=grpc_node --module_path=/home/.npm/_npx/4016/lib/node_modules/@firebaseextensions/fs-bq-import-collection/node_modules/grpc/src/node/extension_binary/node-v64-linux-x64-glibc --napi_version=4 --node_abi_napi=napi --napi_build_version=0 --node_napi_label=node-v64' (1) node-pre-gyp ERR! stack     at ChildProcess.<anonymous> (/home/.npm/_npx/4016/lib/node_modules/@firebaseextensions/fs-bq-import-collection/node_modules/grpc/node_modules/node-pre-gyp/lib/util/compile.js:83:29) node-pre-gyp ERR! stack     at ChildProcess.emit (events.js:198:13) node-pre-gyp ERR! stack     at maybeClose (internal/child_process.js:982:16) node-pre-gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5) node-pre-gyp ERR! System Linux 4.15.0-66-generic node-pre-gyp ERR! command "/usr/local/bin/node"<EMAIL_ADDRESS>"install" "--fallback-to-build" "--library=static_library" node-pre-gyp ERR! cwd /home/.npm/_npx/4016/lib/node_modules/@firebaseextensions/fs-bq-import-collection/node_modules/grpc node-pre-gyp ERR! node -v v10.16.0 node-pre-gyp ERR! node-pre-gyp -v v0.14.0 node-pre-gyp ERR! not ok  ? What is your Firebase project ID? careerograph-e9963 ? What is the path of the the Cloud Firestore Collection you would like to import from? (This may, or may not, be the same Collection for which you plan t o mirror changes.) careerPaths ? What is the ID of the BigQuery dataset that you would like to use? (A dataset will be created if it doesn't already exist) firestore_export ? What is the identifying prefix of the BigQuery table that you would like to import to? (A table will be created if one doesn't already exist) careerPath s ? How many documents should the import stream into BigQuery at once? 300 Importing data from Cloud Firestore Collection: careerPaths, to BigQuery Dataset: firestore_export, Table: careerPaths_raw_changelog Error importing Collection to BigQuery: Error: Cannot find module '@google-cloud/firestore'

Output npx @firebaseextensions/fs-bq-import-collection doesn't work

npx @firebaseextensions/fs-bq-import-collection ? What is your Firebase project ID? careerograph-e9963 ? What is the path of the the Cloud Firestore Collection you would like to import from? (This may, or may not, be the same Collection for which you plan t o mirror changes.) domains ? What is the ID of the BigQuery dataset that you would like to use? (A dataset will be created if it doesn't already exist) firestore_export ? What is the identifying prefix of the BigQuery table that you would like to import to? (A table will be created if one doesn't already exist) domains ? How many documents should the import stream into BigQuery at once? 300 Importing data from Cloud Firestore Collection: domains, to BigQuery Dataset: firestore_export, Table: domains_raw_changelog Error importing Collection to BigQuery: Error: EACCES: permission denied, open '/home/.config/gcloud/application_default_credentials.json'
I have run sudo gcloud auth application-default login to auth user

@mohamedaboelmagd try running this command without the sudo part. At a guess, I think the /home/.config directory has your user permissions.

Importing data from Cloud Firestore Collection: domains, to BigQuery Dataset: firestore_export, Table: domains_raw_changelog
Error importing Collection to BigQuery: Error: EACCES: permission denied, open '/home/mohamed/.config/gcloud/application_default_credentials.json'
This file is empty

I solve it by

sudo npm i -g @firebaseextensions/fs-bq-import-collection
sudo fs-bq-import-collection


perf(api): switched to streaming downloaded files to storage bucket

Tried to explore #4078, but haven't figured it out.

Switched to streaming files downloaded from URLs to the storage bucket. Follows https://dev.to/vikasgarghb/streaming-files-to-s3-using-axios-h32.
Improved error messages and comments
Updated object deletion in apps/combine-api/src/s3.py to work with Google object storage.

Switched to streaming files downloaded from URLs to the storage bucket. Follows https://dev.to/vikasgarghb/streaming-files-to-s3-using-axios-h32.

I believe this is already being done since #3772. Do you know how this differs? I am not too familiar with nodejs streams, but the way I set it up, I thought was already using streams.

I believe this is already being done since #3772. Do you know how this differs? I am not too familiar with nodejs streams, but the way I set it up, I thought was already using streams.

I'm not familiar either. The data type was stream. The S3 library seemed to handle it previously. Others suggested the method I introduced.

I believe this is already being done since #3772. Do you know how this differs? I am not too familiar with nodejs streams, but the way I set it up, I thought was already using streams.

I'm not familiar either. The data type was stream. The S3 library seemed to handle it previously. Others suggested the method I introduced.

Instead of passing in the stream as an object, now we are piping it in? Is that the difference?

Instead of passing in the stream as an object, now we are piping it in? Is that the difference?

Yes. I don't know how much of a difference this makes. It doesn't address the issue I was trying to fix. But I kept it because it seemed like an improvement.

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


Mapping the Atmospheric Network Monitor

With the merging of PR #32294, the atmospheric network monitor is now available to be mapped on space stations
Mappers should note that, in order to be fully functional, the atmospheric network monitor also requires gas pipe sensors (introduced in PR #33128) to be mapped onto the same grid; they will relay important information about the current status of the grid's pipe network to the monitor
Gas pipe sensors should be treated like regular gas pipes; they cannot be placed over an existing pipe that has the same orientation. At a minimum, one gas pipe sensor should be connected to the distro loop and another to the waste loop. Only one sensor is necessary per pipe subnetwork; pipe subnetworks are comprised of the stretch of pipes that lie between pairs of atmos devices, such as filters, mixers, and pumps. Gas pipe sensors will not cause a split in a subnetwork, but you should note that valves will combine/split the subnetworks they connect when they are open/closed, respectively
By default, the monitor status panel will display a list of entries, one for each gas pipe sensor, with the names of the gas pipe sensors and their network addresses in the entry header. If the sensors are labelled (i.e., with a hand labeller), only the label on the gas pipe sensor will be displayed. Several prototype variants for the gas pipe sensor have been included in the PR which have different labels; these include 'distribution loop', 'waste loop', 'mixed air', as well for the different parts of the TEG. The header of each entry in the subpanel also has a colored stripe on the left-hand side; the color of this strip can be changed by spray painting the gas pipe sensor with the desired color
Gas pipe sensors are listed on the monitor in the order that they are attached to the grid. Note that there is no need to link them with the monitors, it's all handled automatically in the code
Maps which have been updated

[x] Amber
[x] Bagel
[x] Box
[x] Cog
[ ] Core
[x] Fland
[ ] Gate
[ ] Marathon
[ ] Meta
[ ] Oasis
[x] Omega
[ ] Packed
[ ] Reach

Could the Atmospheric Network Monitor functionality have been integrated into the Atmospheric Alerts Computer?

Could the Atmospheric Network Monitor functionality have been integrated into the Atmospheric Alerts Computer?

I find there's too much data to neatly convey on one screen (all those overlays to manage on the nav map for a start)
The two consoles complement each other, but they serve two different purposes: the alerts computer is for tracking current issues with the station's atmosphere, while the network monitor is for tracking the status of its pipe infrastructure
Plus I like seeing more consoles around the station :)


EXP: GH-44084: [C++] Improve merge step in chunked sorting

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@ursabot please benchmark lang=C++

@ursabot please benchmark lang=C++

@ursabot benchmark help

@ursabot please benchmark command=cpp-micro --suite-filter=vector-sort

@ursabot please benchmark lang=C++

@ursabot please benchmark lang=C++

Set back to draft because some things can be further improved.

@ursabot please benchmark lang=C++

@felipecrv Would you like to give this another look (assuming CI passes, which it should :-)).

@github-actions crossbow submit -g cpp

It seems like this change causes issues on gcc 8 https://github.com/ursacomputing/crossbow/actions/runs/12081500439/job/33690725970#step:7:2058 probably the change from std::vector to span in chunkresolver?

@assignUser Thanks for the heads up, I'll take a look.

@assignUser See https://github.com/apache/arrow/issues/44898 and https://github.com/apache/arrow/pull/44899

@felipecrv Would you like to give this another look (assuming CI passes, which it should :-))?

Sorry. I've been away. This looks great. Really nice improvements.


Alden/refac records page and its subpages

-Abstracted out Table view
-Removed render() functions
-Relabelled tables to left and right column instead of first and second column

Abstracting the column components out of the page view created more lines as compared to before refactoring.
I unabstracted it back into the page's return statement and commented "left column" and "right column" above the respective code, hopefully more readable now.

Is this still needed?

Nope


No longer able to select by label text with React Testing library

Hello.
Steps to reproduce:

Have formik-chakra-ui at version 1.4.1
Clone my minimum example repo from https://github.com/PedroDBFlores/chakra-ui-formik-problem
Run the tests with the current 1.4.1 version

Expected result:
Both tests should pass
Actual result:
I'm no longer able to get the selector by using the label which is labeling it.
I've checked the recent commits, might the id removal from the SelectControl have something to do with it? This might also break accessiblity by not being able to use aria-label on the select label.

Hi,
Thank you for the detailed issue. It's probably because of id like you said. I will look into that.

Hi,
Thank you for the detailed issue. It's probably because of id like you said. I will look into that.

I probably need to write tests, but for now, I used Kent's Testing Playground and it looks like it is working. Some other components had the same issue too. Fixed in https://github.com/kgnugur/formik-chakra-ui/commit/253ae62388cab9d7c705c1d6bf45a565ac8a9451
You can try 1.4.2
Thanks a lot!

I probably need to write tests, but for now, I used Kent's Testing Playground and it looks like it is working. Some other components had the same issue too. Fixed in https://github.com/kgnugur/formik-chakra-ui/commit/253ae62388cab9d7c705c1d6bf45a565ac8a9451
You can try 1.4.2
Thanks a lot!

It now works again, thanks your your availability to promptly fix it 👍

It now works again, thanks your your availability to promptly fix it 👍


Add events for container create and start

Signed-off-by: Darren Shepherd<EMAIL_ADDRESS>

LGTM


Possibly config mistake in mode DEFAULT_BLUE_ICE

I think this is a typo where the value should be 0.989f right?
Doesn't affect me but I just happened to see it. You may not need to release a patch only to fix this.

ah yeah, this mode id will have to stay like that but i'll add a new correct one and rename it


Remove LALRPOP based parser

I'm going to make sure the ecosystem checks are all good and then merge even though I've ran them locally


[Storage] Pre-compute IPFS hashes before uploading

👀

@joaquim-verges now it touches our core IPFS upload logic (actually added the precompute/return)

/release-pr

approved by @jnsdls


Ability to emit jsdocs ending on previous line instead of same line?

Describe the feature
Currently code like the following:
/** Test */
function test() {}

/** Test */
export class Test {
  /**
   * Multi-line
   */
  method() {}
}

Will emit with comments like so:
/** Test */ function test() {}
/** Test */ export class Test {
    /**
   * Multi-line
   */ method() {}
}

https://play.swc.rs/?version=1.3.62&code=H4sIAAAAAAAAA9PX0lIISS0uUdDS5%2BVKK81LLsnMz1MoAYpoaCpU1%2FJy8XLpIytJrSjILypRSM5JLC6GiFbzcikoANWAKAUtBd%2FSnJJM3ZzMvFSIgD6Iyk0tychPgZpYCwCN0JyldAAAAA%3D%3D&config=H4sIAAAAAAAAA11Ru27DMAzc8xWG5gxthg7dihTe%2BhGEQjsq9DBIOqgR%2BN8rKpLrdJHEu%2BPxofuh68w3W%2FPe3fMzBxMQI21xRniJAj8ZMbJMyJbcJObYWGGlhGYsyHqsNoTZ5oYf3p9TCBiFq6wKBGhEUVfk08vp1TScIPKQKOxbmBk%2FcXAR%2B0RnD8y9Q395dswyQrCyyyt9wBhAy1y3nje4z6dSemuLewXNUVxApa1WdNZUcn0a1KfEqhrAc5stuOiGZT9AgDj6f7IM2xR0UdyIQ%2FMv7iaky1yy6t%2Fo%2Fh8bezN%2FolZsszaOv1pm%2BZj1F5W1tcznAQAA
I'm emitting some typescript declarations using swc where only the jsdoc comments are preserved and the output doesn't look great because of this (I'd rather avoid doing post-processing on it, but it's not a big deal if I have to do that)
What are your thoughts on making swc do this? For a fast implementation, perhaps it should only do it for block comments that start with * and preceed declarations?
Both Babel and TypeScript keep the jsdoc on the previous line, but will also keep it on the same line if the person has the code that way.
Babel plugin or link to the feature description
No response
Additional context
No response

I think it's fine to default to your suggestion


Display relevant error for contracts deployed with names over 40 characters

From Discord:

In case this affects the solution to this issue, the actual limit on-chain is only 40 characters.

If you try to deploy a contract with name over 40 but less than 128, the main interface loads, but the cost estimate never does:


BUG: Python 3.12 runs failing

Issue Description
I just saw that Python 3.12 pipelines are failing.

[ ] Check if this is reproducible
[ ] Check for dependency issues

Minimal Reproducible Example
See: https://github.com/shap/shap/actions/runs/11749055613/job/32735088282#step:6:872.

Traceback
No response
Expected Behavior
should run seemlessly
Bug report checklist

[X] I have checked that this issue has not already been reported.
[X] I have confirmed this bug exists on the latest release of shap.
[ ] I have confirmed this bug exists on the master branch of shap.
[ ] I'd be interested in making a PR to fix this bug

Installed Versions
master

Looks like an error with catboost, even though catboost seems to support Python 3.12 (see https://github.com/catboost/catboost/issues?q=is%3Aissue+python+3.12+is%3Aclosed).

It seems to be related to the following issue on catboost: https://github.com/catboost/catboost/issues/2671


(Bug with Physics Mod) Breaking any block breaks blocks above it--

After adding this into a modpack, it seems as though
even mining stone creates a 'felling' effect,
while digging a block that has more blocks connected above it.
I did search for this similar problem, but haven't found any similar issues so I thought of writing this down~
I found this issue is caused by the Physics Mod, specifically-
( https://www.curseforge.com/minecraft/mc-mods/physics-mod/download/3633780 )
Using a CurseForge 1.16.5, Forge 36.2.20 client, this is what I have tried:
🔘 Created a fresh modpack, and adding only
physics-mod-2.6.7-mc-1.16.5-forge
(Blocks did not collapse)
🔘  Created a fresh modpack, and adding only
TreeChop-1.16.4-0.14.6-fixed
(Only logs collapse)
🔘 Created a fresh modpack, and adding only both,
Default config and settings replicate this bug.
🔘 Changed the config settings relating to felling and chopping to false
🔘 Used the keybinds in-game to toggle felling and chopping off
Result: Despite the features being toggled false, breaking any block seems to make it break those above it.
Interesting enough, breaking the nether roof's bedrock is also possible from this.
Hope this was at least a little bit helpful.
I'm just happy that it doesn't crash anything~

Edit: This was due to a config in the Physics mod that had updated. Woop. ---Nothing happened here--


[CI] Schedule dependabot checks

schedule checks so they happen at 4:00am UTC. This way gitlab CI doesn't get
blocked during working hours.
This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] create a file in news/unreleased directory if this change should be mentioned in the release news
[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

I would even reduce the number of pull requests. We can always scale up later but for now, I would go safe.

I would even reduce the number of pull requests. We can always scale up later but for now, I would go safe.

3 pull requests for example?

Can we merge as-is?

I would like to, is it possible to skip the GitLab CI check?

I would like to, is it possible to skip the GitLab CI check?

Yes, just leave it fail (or cancel it to not hog resources) and wait for the PR to be merged by somebody who has admin access to the repo and can merge with failing required checks.

I would like to, is it possible to skip the GitLab CI check?

Yes, just leave it fail (or cancel it to not hog resources) and wait for the PR to be merged by somebody who has admin access to the repo and can merge with failing required checks.

okay, thanks! I'll cancel the jobs so it can be merged

This doesn't seem to help, dependabot sends new PRs at any time.

This doesn't seem to help, dependabot sends new PRs at any time.

This is what I think is hapenning: Dependabot does checks on the schedule specified. But as there are more than 5 PR, he creates a queue with the rest of the old dependencies. Whenever we merge a dependabot PR, it just creates a new one from the queue.


ci: new quickstart build for Linux+CMake+vcpkg

Part of the work for #3734

This change is

Rebased, resolved conflicts, and fix formatting now that shfmt is enabled.


FIX: Demointeroperability.kpi filters dropdown menu

This addressed to the issue
<EMAIL_ADDRESS>Dropdown fixed but queries still not applicable

Ok, merging


Better exceptions handling in the runfile

Currently when some error happens in the runfile feedback about the error is not really informative.

Is this the error that you're talking about? Because it certainly isn't informative and I have no idea what's breaking.
/Users/<user>/git/<project>/node_modules/runjs/bin/run.js:9
    throw e.name + ':' + e.message;
    ^
TypeError:Cannot read property 'error' of undefined

Yes that's the thing

#18 tries to do this in the simplest way possible.


Missing t5 term in deformation potential for alpha/beta ?

https://github.com/DavisThuillier/Ludwig.jl/blob/e12cc78de30933ce5456febaf2c79c125132d167/scripts/materials/Sr2RuO4.jl#L65

Including the t5 term causes bizarre cusps at integer multiples of pi/2. Unsure of how to resolve.

Issue identified: the derivative of V must needs be preceded by a factor of V itself giving us an overall t5^2 scaling. The extra factor of sign(V) takes care of the cusps.


build: facebook-ios-sdk 6.2.0

update sdk 6.2.0
fix test : travis test pass
fix example/ios/Podfile : Keep pod path relative so it can be checked into Podfile.lock.
fix exmaple #217

@amondnet  you should also update and test Android SDK to 6.2, you only updated iOS

@atrope There is already #255


CI updates

Companion to zeek/zeek#3502.

Yep, I'm all for it. I think we could move it to zeek-aux even if there were a lot more activity here. zeek-aux itself could use some love too — do we really maintain adtrace.c and rst.c at this point? — etc.

I used to use rst! Though from what I remember I had to dig up the actual script that drove it out of svn history, and that was 10+ years ago

Yeah we're not actually using it any more in the scripts.
Over the years I have looked it up on occasion for low-level socket and checksumming code. :-)


Nice lib

I see this is a new repository, is it functional already? How to test it? (Does QIWI provide developer accounts/server?) They have terrible docs, thanks for sharing your code. When you can add a demo/example code.
Cheers

Unit tests, examples and pypi package in progress. Lib ready for using in production. Unfortunately, QIWI doesn't provider test accounts. I tested code with real payments. I don't think that their docs is terrible (https://static.qiwi.com/en/doc/ishop/protocols/OnlineStoresProtocols_REST_EN.pdf)

Unit tests, examples and pypi package in progress
+1 Awesome

I don't think that their docs is terrible
Thanks for the link, very helpful. I couldn't find it before, not being a Russian speaker it's hard to go through their website, which is Russian-only (language button does not work), so your library will be helpful.


Changes to branch to get Pro tests to run

I also merged with the dev upstream, just to make sure I was testing against the latest dev

@csmoore go ahead and delete this branch when you're ready.


[FLINK-11591][core] Support legacy TypeSerializerSnapshot transformations

What is the purpose of the change
(copied from the issue description)
In 1.6 and below, the LockableTypeSerializer incorrectly returns directly the element serializer's snapshot instead of wrapping it within an independent snapshot class:
https://github.com/apache/flink/blob/release-1.6/flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/nfa/sharedbuffer/Lockable.java#L188
This results in the fact that the the written state information for this would be (LockableTypeSerializer, SomeArbitrarySnapshot).
The problem occurs when restoring this in Flink 1.7+, since compatibility checks are now performed by providing the new serializer to the snapshot, what would happen is:
SomeArbitrarySnapshot.resolveSchemaCompatibility(newLockableTypeSerializer), which would not work since the arbitrary snapshot does not recognize the LockableTypeSerializer.
To fix this, we essentially need to preprocess that arbitrary snapshot when restoring from <= 1.6 version snapshots.
Brief change log

This PR adds an interface LegacySerializerSnapshotTransformer that is being respect by the TypeSerializerSerializationUtil that allows serializers provide custom transformation logic for their associated snapshot classes.
Uses that interface by the LockableTypeSerializer.

Verifying this change
This change is already covered by existing tests, such as CEPMigrationTest.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot approve description
@flinkbot approve consensus

@flinkbot approve quality
@flinkbot approve architecture


Got "no change found" after pushed new changes to PR

As title, I am using Github Actions as below
name: Code Review

on:
  pull_request:
    types: [opened, reopened, synchronize]

jobs:
  test:
    # if: ${{ contains(github.event.*.labels.*.name, 'gpt review') }} # Optional; to run only when a label is attached
    runs-on: ubuntu-latest
    steps:
      - uses: anc95/ChatGPT-CodeReview@main
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}

@timmy-liu-paypay could share the PR and the action link

@timmy-liu-paypay I reproduced this issue, should be fixed by the latest version


Link Grafana docs on how to create or change custom dashboards

Goal
The goal of this issue is to introduce info about how to create or change custom Grafana dashboards. This should be done not by writing a new docs page about how to do this, but to link the proper Grafana docs sections, which are describing how to achieve the goal. The Grafana docs should be linked from the Grafana README file
DoD

[ ] new grafana docs page is not introduced
[ ] grafana README does not contain the information about how to create/change custom dashboards -> it should only contain links to the official Grafana docs with some description around them

I'm interested in working on the issue

Thank you @UtkarshUmre assigned!


Feature: Auto-import settings from GM or guild notes

Allow importing guild settings from the GM, or another character in the roster.  Suggestions:

Compression+Transport Encoding:

Could use BASE64 encoding (library support?)
Could use proprietary encoding (more work, harder to maintain)

Where to find?

Guild motd?
Guild info?
Player notes of GM or specific ranks

Needs confirmation to prevent GM abuse (ex: setting a high contribution amount automatically).

Closing issue for discontinued development.


Migrate CsvTransformer to StreamingCsvTransformer

Who is this PR for?
developers
What problem does this PR fix?
Deprecated code.
What does this PR do?
Removes deprecated CsvTransformer altogether.

selfie


Image upload

Fix #142
Due to a service/controller problem the progress bars were not visible. Now there should be a progress bar for each file. After all files are uploaded the progress bars are removed (after 1000ms). Is this ok or should there be more/another info? @derntl

The problem is that our lumen part uploads the images to the local storage (the machine php is running on), but to the database specified in the .env. Thus images uploaded on our server are stored on the server (and are displayed if you open the .../intern link) and images uploaded on your local instance are stored on your machine (and are displayed if you open localhost/...).


Support multiple ingress gateways

Problem statement:
The current version of the operator has the Istio ingress gateway configuration in the main Istio custom resource. It means that only one ingress gateway can be configured and reconciled through the operator, but it is a common use-case to have multiple ingress gateways in an environment.
Possible workaround:
Configure the Istio control plane (and one ingress gateway) through the operator and manage the rest of the gateways manually by installing them with Helm, or a simple kubectl apply.
Solution:
Decouple the ingress gateway configuration from the rest of the control plane components by creating a new custom resource definition (IstioIngressGateway) that holds these properties, and a new controller in the operator that manages this component. Adding a new CR would create a new gateway, deleting the CR would delete that gateway.
The current APIs and controllers are generated by Kubebuilder v1, the new CRDs and controllers should use the same version.
Make sure that the current Istio control plane CRD remains backward compatible, so an ingress gateway could still be created through the main CR.

@Laci21 Do you think this really requires a new CRD? Could we not just do something like this?
apiVersion: istio.banzaicloud.io/v1beta1
kind: Istio
metadata:
  labels:
    controller-tools.k8s.io: "1.0"
  name: istio-operator
  namespace: istio-system
spec:
  version: "1.2.5"
  gateways:
    enabled: true
    ingress:
      enabled: true
      serviceType: "LoadBalancer"
      ....
    ingress-2:
      enabled: true
      serviceType: "LoadBalancer"
      ....
    ingress-3:
      enabled: true
      serviceType: "LoadBalancer"
     ....
    egress:
      enabled: false
    k8sIngress:
      enabled: false

// GatewaysConfiguration defines config options for Gateways
type GatewaysConfiguration struct {
	Enabled             *bool                           `json:"enabled,omitempty"`
	IngressConfig       GatewayConfiguration            `json:"ingress,omitempty"`
	EgressConfig        GatewayConfiguration            `json:"egress,omitempty"`
	K8sIngress          K8sIngressConfiguration         `json:"k8singress,omitempty"`
	OtherIngressConfigs map[string]GatewayConfiguration `json:"-"`
}

// MarshalJSON is the implementation of the interface in encoding/json/Marshaler
func (g GatewaysConfiguration) MarshalJSON() ([]byte, error) {

	type GatewaysConfiguration_ GatewaysConfiguration // prevent recursion
	b, _ := json.Marshal(GatewaysConfiguration_(g))

	var m map[string]json.RawMessage
	_ = json.Unmarshal(b, &m)

	// Add tags to the map, possibly overriding struct fields
	for k, v := range g.OtherIngressConfigs {
		b, _ = json.Marshal(v)
		m[k] = b
	}

	return json.Marshal(m)
}

I'd love some feedback on this approach. If you think it will work, I can work toward a PR over the next couple of days

A follow up, you can set the namespace in the GatewayConfiguration and allow setting the namespace on the additional gateways.
I used this SO originally when I needed to understand how to have multiple ingress gateways. https://stackoverflow.com/questions/51835752/how-to-create-custom-istio-ingress-gateway-controller

@blaketastic2,
first off, appreciate your time/effort you put into this!
We at banzai are in favour of having a separate CRD for several reasons:

a separate controller should take care of reconciling ingress gateways as there could be many in different namespaces
proper RBAC control of who can manage ingress gateways (otherwise someone has to have update permission to the Istio CR just to create an ingress gateway in one namespace)
having a component in a namespaced CRD that will be reconciled in another one sounds like a bad idea

Actually I see no real reason in favour of having multiple ingress gateways defined in the current Istio CRD.

@waynz0r ok, your points make a lot of sense. Thanks for the feedback.

Hello,
Is there any progress on this? We have a similar need at my workplace, and can work on a PR if needed. If we should, is a new CRD definitely the way to go?
Thanks.

@utkuozdemir,
thanks for the interest! We are actively working on this, I'll link the PR into this thread as soon as it gets opened.
Could you share your use case so we can take it into account while developing this feature?

@waynz0r
Thanks for the quick reply.
Our use case is multiple tenants using the same cluster, being able to create/delete ingress gateways as they need. Since Istio is not quite ready for full multi-tenancy with multiple control planes, we decided to stick to multiple ingress gateways for now.
RBAC aspect for this is not very important to us, since abstracting tenants away from the raw kubernetes ingressgateway objects is an option.
Since you are already working on the feature, is there a timeline or an estimation for it? For the PR to be opened, and for it to make into a release. Days, weeks or months? Since it can be the deciding factor for us to use this operator or not.
Also, please let me know if we can provide any help/support from our side.

hey @utkuozdemir - we will have a Backyards release late September (eta Sept 27) and this feature is part of the release. Since Backyards is using/built on the Istio operator, the feature/PR will land here in the newt 1-2 weeks.
Once it's there you can help with testing - thanks for offering your help.

@matyix are there any updates on multiple ingress gateway support?  We would like to be able to support both an external and internal ingress gateway.

@coreywagehoft,
it was put behind a bit on the priority list, but I actively working on it right now, hopefully there will be a PR for it in the next 2 weeks.

@waynz0r any updates on this?  I would love to move over to your operator sooner than later!

@coreywagehoft, yeah, unfortunately we couldn't finish it in time to put it into the previous release so it is gonna be in the next one.
I'll get back to you with more specific details early next week, most of the service mesh team is at KubeCon right now.

Thanks for the update @waynz0r. I’m at KubeCon as well so I wouldn’t be able to test it out this week.

hey @coreywagehoft - we are here as well, drop by at SE74 if you want to chat.

@waynz0r any updates on when we might see this functionality?  We would like to test it out and adopt the operator.

@coreywagehoft, we just released a new version of the operator with multiple mesh gateways support. Please let us know how it works for you!


docs: readme update

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

/kind release

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area rules

/area tests

/area proposals

/area CI

What this PR does / why we need it:
See https://github.com/falcosecurity/evolution/issues/171
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
/milestone 0.33.0
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

This is great Leo, thanks for all the small fixes and added doc!


Update README.md

Remove specific frame rate number as it depends on hardware, update transforms.
Closes #449.

Coverage remained the same at 84.56% when pulling ff96ca62c5f7b60ebfb56fe3ef5c9b17356c4f58 on rzumer-patch-2 into ed3cf8345f26040bdf0ebf488b2db23c22ce6c97 on master.

This seems like negative progress. I'd way rather have some general expectation & idea, than absolutely nothing at all. I'd like this reverted and/or a proper attempt to satisfy #449.

There is no point in reverting this change, since performance changes constantly as features are added, so accuracy of numbers cannot be guaranteed, while "near real-time" is a reasonable ballpark for expectations on consumer-level hardware. If/when performance is tested systematically, then more info will be available, but it is still early in development, so it is not a high priority.
On November 28, 2018 1:30:09 PM EST, rektide<EMAIL_ADDRESS>wrote:

This seems like negative progress. I'd way rather have some general
expectation & idea, than absolutely nothing at all. I'd like this
reverted and/or a proper attempt to satisfy #449.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/xiph/rav1e/pull/730#issuecomment-442554597


What is the Group and channel address limitations ?

What is the length of the hash wyRG9uKD51Bx8bqFHJvoL2eP (for groups address) and channel username characters and which characters are allowed ?

Sending and receiving messages from the group and the channel is not currently available.
For this reason, their user name limitations are unclear.


Task/g52 delegated harvesting

Added AcountLink Transaction. (Delegated harvesting)
Issue: #52

AccountLinkTransaction plugin hasn't been activated on catapult-server yet. Also changes in this PR require latest NEM2-Library update PR#16

Pull Request Test Coverage Report for Build 224

28 of 30 (93.33%)  changed or added relevant lines in 7 files are covered.
10 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.6%) to 72.441%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/infrastructure/transaction/CreateTransactionFromDTO.ts
5
7
71.43%

Files with Coverage Reduction
New Missed Lines
%

src/infrastructure/MosaicHttp.ts
2
72.0%

src/infrastructure/NamespaceHttp.ts
3
16.83%

src/service/MosaicService.ts
5
73.68%

Totals

Change from base Build 202:
0.6%

Covered Lines:
1028

Relevant Lines:
1283

💛  - Coveralls

Pull Request Test Coverage Report for Build 224

28 of 30 (93.33%)  changed or added relevant lines in 7 files are covered.
10 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.6%) to 72.441%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/infrastructure/transaction/CreateTransactionFromDTO.ts
5
7
71.43%

Files with Coverage Reduction
New Missed Lines
%

src/infrastructure/MosaicHttp.ts
2
72.0%

src/infrastructure/NamespaceHttp.ts
3
16.83%

src/service/MosaicService.ts
5
73.68%

Totals

Change from base Build 202:
0.6%

Covered Lines:
1028

Relevant Lines:
1283

💛  - Coveralls

Pull Request Test Coverage Report for Build 224

28 of 30 (93.33%)  changed or added relevant lines in 7 files are covered.
10 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.6%) to 72.441%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/infrastructure/transaction/CreateTransactionFromDTO.ts
5
7
71.43%

Files with Coverage Reduction
New Missed Lines
%

src/infrastructure/MosaicHttp.ts
2
72.0%

src/infrastructure/NamespaceHttp.ts
3
16.83%

src/service/MosaicService.ts
5
73.68%

Totals

Change from base Build 202:
0.6%

Covered Lines:
1028

Relevant Lines:
1283

💛  - Coveralls


Fix empty string leads to 's'

Resolves #19

@sentinelt can you please have a look?

I've pushed slightly different fix, more in line how the code is organized


Remove SPIRE config file requirement

Because we have split SPIRE from Tornjak container, potentially would be good to remove this requirement. Depends on #119

Currently, removing SPIRE config creates a following error:
kubectl -n tornjak logs spire-server-0 -c tornjak
-t /run/spire/tornjak-config/server.conf
/run/spire/tornjak-config/server.conf
-c SPIRE_CONFIG must be provided

I agree with removing the requirement for the file, though the VolumeMount even doesn't quite return the correct ConfigMap because there's the option of the SPIRE executable parsing from container environment variables, which the Tornjak sidecar does not share.
The original goal of serverinfo was to provide easy test end-to-end from Tornjak backend to SPIRE server.
#161 is a new PR implementing two new APIs; healthcheck and debugserver, which both are easy tests of the end-to-end working. In particular, maybe debugserver returns nice amount of SPIRE server information. Example:
{
    "svid_chain": [
        {
            "id": {
                "trust_domain": "example.org",
                "path": "/spire/server"
            },
            "expires_at": 111,
            "subject": "O=SPIRE,C=US,<IP_ADDRESS>=#12321"
        },
        {
            "id": {
                "trust_domain": "example.org"
            },
            "expires_at": 222
            "subject": "O=SPIFFE,C=US"
        }
    ],
    "uptime": 333,
    "federated_bundles_count": 1
}

Perhaps we could replace serverinfo implementation with a single call to debugserver.
The main caveat is that in the frontend, the Tornjak ServerInfo tab relies on serverinfo command if I am not mistaken because it parses the config plugins (@mamy-CS please confirm?). I think what's there looks nice, but if we remove the requirement of passing the config file, what should be displayed in this ServerInfo tab?

I think that serverinfo  API should return SPIRE Server Information. If the information is not available, e.g. no ConfigMap volume defined, this API call should return an appropriate info. I am OK with returning the content of the debugserver as a debugserver Tornjak API, but we should not change the content of the call. If we always guarantee to return some meaningful value for the debugserver call, we can switch to use this call as a default test for Frontend to validate SPIRE APIs. Same with the healthcheck call

Offline conversation with @mamy-CS  - we decided to stick with the serverinfo call as specifically not calling the spire-server and instead return information about SPIRE config if available (and if not, returning some kind of message about that). As debugServer and healthcheck are separate calls, we didn't want to make the APIs too confusing by adding redundant commands for this information


Rename ImageProvider subclasses to end in ImageProvider, rather than Image

It's a naming conflict with the Image Widget.

Yeah, that's probably reasonable. Unfortunately it would be a breaking API change, and we're really trying to avoid doing those now...

I'm going to close this without fixing it, because even though I agree with the original request, I really think it's too late to make that kind of change. It would break a lot of people's apps and the improvement is not that substantial (in some ways it might even be bad, since it would make the class names quite long).


Document the default value of "editorconfig"

Also show that it's a valid option in the  config file.

A missing setting defaults to falsey.  It isn't interesting to add documentation for all the false settings.

First, I've used a lot configuration systems over the years and I can't recall where where it was assumed that missing values defaulted to false.
Second, it's critical to have a comprehensive list of configuration options so people know that they actually exist.
I was not certain that editorconfig was configurable until I read the source code to confirm.


Support More Icon Fonts

Material Design Iconic Font is nice, but sometimes is lacking icons available in other font's.
Add support for other alternative icon fonts.

Obsolete by #262


Metro Last Light Redux Black Screen

Upon launching the game you'll see a black screen and hear the intro video for about two seconds, then it stops.
Software information
Metro Last Light Redux
Settings: Unknown
System information

GPU: GTX 1060
Driver: 390.48
Wine version: wine-staging-3.4
DXVK version: ae88f83

Apitrace file(s)
apitrace
Log files
metro_d3d11.log
metro_dxgi.log

The apitrace replays fine on my end with latest master, this should be working. Note that the Metro Redux games require Stream Output (#135).


⚠ Secret Website 3 has degraded performance

In c7afe59, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2716 ms

Resolved: Secret Website 3 performance has improved in be607ac.


Scan button not intuitive

On the new home screen, I find the new "red camera button" not very intuitive. Nothing says it's the button to scan a barcode, instead it now looks like it takes a picture, which is not exactly its function.
In my opinion a button with "scan", a barcode logo or anything like this would be clearer, especially for a new user.

it was the barcode icon for the first draft but we using already the camera icon for the left menu, so i switch to this one.
I agree with you opinion but i don't think the barcode will improve the understanding of the feature (maybe we have to create a tutorial which be displayed the first time the app is opened).

I like the barcode better. Why not put it in the left menu too?
Also a tutorial was not needed when there was a "scan" button before. I think if we need one now, it means it's less clear...
Out of curiosity, why did the button change to this one, is it to match the "official" material buttons or something like that?

indeed WE are trying to apply the material design guidelines.
i can switch to the barcode if @teolomon is agree.

@asmoth360 I can see your point how the paper fab might be less intuitive for some than the "Scan product" in full text.
However, we chose the camera because the icon was standard, way more and quicker recognized than the barcode, and gave an equivalent idea of what's going to happen.
Let's give it a little time to live, and we'll do some more user testing ?

No problem, it's just my opinion.

Let's use the Scan icon instead. Bonus points if you add the button on the all views of the app (especially on the product view), for Quick anytime scan.


Add boot_command and boot_wait to Ubuntu Subiquity autoinstall builds

I'm attempting to create Ubuntu 20.04 packer builds using Subiquity.  The temporary packer.iso is created and so is the packer vm, but it hangs at the following spot:

Looks like other plugins have the boot_command and boot_wait options to deal with this. https://askubuntu.com/questions/1385563/packer-ubuntu-server-20-04-prompting-for-install-confirmation-despite-autoinsta
Can these options be added to this plugin?

Or maybe there is a better way to handle auto installs of Ubuntu?  Do you guys have a working example of an Ubuntu build?

Support ticket has been opened with Nutanix support.

Hello. I'm also interested by this feature. Any update from the support case?

Hello. I'm also interested by this feature. Any update from the support case?

@tandrez can you create your own ticket to prioritize the feature and give the ticket number here
thanks

Support ticket has been opened with Nutanix support.

@ktinthecloud can you paste you support ticket reference here please

CASE 01276838
It was closed out with the response below:
Currently, there is no official/supported way to use boot_command or boot_wait options. Hopefully engineering will expose some sort of API layer or access to noVNC in the feature request.
Thank you,
To view case history Click here : https://portal.nutanix.com/page/smart-support/case/details?id=5007V000026d22FQAQ
Jason Tupper | Systems Reliability Engineer | Nutanix | 1-855-NUTANIX Ext 3
Business Hours: Mon - Fri 08:00 - 16:00 EDT
Customer Support Portal: http://portal.nutanix.com/
ref:_00D60IUGi._5007V26d22F:ref

Support ticket has been opened with Nutanix support.

@ktinthecloud can you paste you support ticket reference here please

New ticket opened: 01524228

I have raised this with Nutanix support - they have issued FEAT-16277 feature request for anyone who wants to track it/add their +1


Add possibility to include full stacktraces with causing exceptions

- It is desirable to have a full stacktrace including the causing exceptions in logs
- To be backwards compatible, an optional flag is added
- Full message will no longer contain stacktrace, since it is redundant

@nilsschmidthamburg Thanks! I've cleaned up the PR and merged it in 1555e38662f705826cd4e1352c30aff365d3b117.


Remove unneeded pkg patch

We had a patch for pkg due to https://github.com/knative/pkg/issues/2418. This was solved in https://github.com/knative/pkg/pull/2738 therefor we can remove this patch again.
/cc @pierDipi

/retest-required

/retest

/retest

/unhold
@pierDipi could you PTAL?

@pierDipi sry. Need to recheck
/hold


IGNITE-18443 Sql. Provide extend commands and handlers for distributed zones operation

@lowka , the patch looks good. It would be nice to add a few tests on parsing though

@lowka , the patch looks good. It would be nice to add a few tests on parsing though

Fixed disabled tests.


Added paragraph to README for retirement of qiskit-jku-provider

Summary
The qiskit-jku-provider is based on an out-dated implementation of the JKU simulator. The recent version is available as JKQ DDSIM.
I would suggest archiving this repository and adding a few lines of text of explanation with this PR.
JKQ DDSIM should be faster (although I haven't directly compared to the code in this repository) and it should be easier to maintain and extend. For example, a backend to construct the unitary is available.
Corresponding Slack Thread: https://qiskit.slack.com/archives/C7SS31917/p1620994583241400
Details and comments
Apart from not being actively maintained, the qiskit-jku-provider uses an out-dated implementation of the underlying decision diagrams as well as the simulation front-end. Additionally, the bindings between python and c++ are realized by writing to files from python, reading them in c++, and reading back the c++ output to python---the new implementation used pybind11.

Good point. I've added a bullet list with changes required to migrate the example code in the Getting Started section.


Feature Request: Stream from file

Is it possible for OHHTTPStubs to stream from a file? For example, a .mp4 file.
If it's not, then what would a contributor have to do in order to enable this feature?
We ask because our streaming player complains with an error when we try to stream from a file that's been stubbed.
Thanks in advance

When you build your OHHTTPStubsResponse to return in your stub, you should be able to build that response in multiple possible ways, as this type exposes various constructors, including:

responseWithData:statusCode:headers:, which builds an [NSInputStream inputStreamWithData:data?:[NSData data]] then calls the internal initWithInputStream:dataSize:statusCode:header: constructor
responseWithFileAtPath:statusCode:headers:, which builds an [NSURL fileURLWithPath:filePath] then call the method below
responseWithFileURL:statusCode:headers: which builds an [NSInputStream inputStreamWithURL:fileURL], compute the file size, then calls  the internal initWithInputStream:dataSize:statusCode:header: constructor

initWithInputStream:dataSize:statusCode:headers: is the designated initialized (that will be called in the end by any of the other convenience initializer) and which definitely takes an NSInputStream.
So yes, if that was your questions, you have everything to stream from a file, responseWithFileURL:statusCode:header: already uses NSInputStream and streams that content (instead of loading it all in memory at once and sending that NSData).
The only thing OHHTTPStubs does that your player might not like is that it adds the Content-Length header to the request automatically if it's not present/defined already, and if your player expects streaming content in the sense of "non-fixed-length content that might never end", maybe that additional Content-Length header bothers it?

Hi. I'm gonna close this issue now due to the fact that I think I've answered your question and due to lack of response. Consider reopening if you still have pending problems about that due to OHHTTPStubs itself.

Sorry I have not had a chance to revisit this. When I do and if I should have any further issue I will reopen.
Thank you for your careful and thought out reply!
Mike
Ps...we are using this library extensively and quite creatively I might add. We are thankful for it!

On Aug 17, 2016, at 5:46 PM, AliSoftware<EMAIL_ADDRESS>wrote:
Hi. I'm gonna close this issue now due to the fact that I think I've answered your question and due to lack of response. Consider reopening if you still have pending problems about that due to OHHTTPStubs itself.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.