Remove InsertBatch in gRPC message

What type of enhancement is this?
API improvement, Performance, Refactor, Tech debt reduction, User experience
What does the enhancement do?
before:
message InsertExpr {
  string schema_name = 1;
  string table_name = 2;

  message Values {
    repeated bytes values = 3;
  }

  oneof expr {
    Values values = 4;
    string sql = 5;
  }

  uint32 region_number = 6;
  map<string, bytes> options = 7;
}

after:
message InsertExpr {
  string schema_name = 1;
  string table_name = 2;
  repeated Column columns = 3;
  uint32 row_count = 4;
  uint32 region_number = 5;
  map<string, bytes> options = 6;
}

benifit:

get rid of "sql" in InsertExpr, which was introduced before as a temporary solution to insert data via frontend's grpc interface, because we cannot get table schema at that time
much simpler to use in grpc cli tool, otherwise we have to serialize InsertBatch to bytes then to base64 manually, which is rather tedious
no ser/de costs of InsertBatch on both grpc client and server
a lot easier to write grpc related codes

first step to #490
Implementation challenges
No response

after this is done, we should update doc https://docs.greptime.com/user-guide/reading-writing-data

You will also need to refactor common_insert::insert::build_create_expr_from_insertion's insert_batches argument to InsertExpr


[Bug] "psycopg2" Dependency fails building Docker Images

Is this a new bug?

[X] I believe this is a new bug
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
Hi all,
Building Docker image fails while installing dbt-postgres in integration with cosmos.

On this specific step, RUN python -m venv dbt_venv && source dbt_venv/bin/activate && \   pip install --no-cache-dir dbt-postgres && deactivate (adding this in a docker file) and building using astro dev start , I'm receiving the following error
9.396 Collecting psycopg2<3.0,>=2.9 (from dbt-postgres)
9.418   Downloading psycopg2-2.9.9.tar.gz (384 kB)
9.475      ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 384.9/384.9 kB 7.4 MB/s eta 0:00:00
9.542   Installing build dependencies: started
12.74   Installing build dependencies: finished with status 'done'
12.74   Getting requirements to build wheel: started
12.94   Getting requirements to build wheel: finished with status 'error'
12.95   error: subprocess-exited-with-error
12.95
12.95   × Getting requirements to build wheel did not run successfully.
12.95   │ exit code: 1
12.95   ╰─> [21 lines of output]
12.95       running egg_info
12.95       writing psycopg2.egg-info/PKG-INFO
12.95       writing dependency_links to psycopg2.egg-info/dependency_links.txt
12.95       writing top-level names to psycopg2.egg-info/top_level.txt
12.95
12.95       Error: pg_config executable not found.
12.95
12.95       pg_config is required to build psycopg2 from source.  Please add the directory
12.95       containing pg_config to the $PATH or specify the full executable path with the
12.95       option:
12.95
12.95           python setup.py build_ext --pg-config /path/to/pg_config build ...
12.95
12.95       or with the pg_config option in 'setup.cfg'.
12.95
12.95       If you prefer to avoid building psycopg2 from source, please install the PyPI
12.95       'psycopg2-binary' package instead.
12.95
12.95       For further information please check the 'doc/src/install.rst' file (also at
12.95       <https://www.psycopg.org/docs/install.html>).
12.95
12.95       [end of output]
Upon Searching, I guess this is doing something with Postgres and psycopg2 packages. But since that package is used in dbt we're receiving this error. I'm very noob to dbt,airflow,astro and cosmos. So can anyone help me sort this issue out?

Expected Behavior
astro dev start should be building the Airflow along with DBT Adaptor mentinoed in the Docker file.
Refer Here:
Steps To Reproduce

Install Astro CLI
Create a folder and cd into it.
run astro dev init to create an empty airflow project
Add this
# install dbt into a virtual environment RUN python -m venv dbt_venv && source dbt_venv/bin/activate && \  pip install --no-cache-dir <your-dbt-adapter> && deactivate in the Docker file
Add astronomer-cosmos in the requirements.txt file
run astro dev start to build the Docker Image and Spin up the Airflow container

Relevant log output
No response
Environment
- OS:Windows
- Python:3.10.9
- dbt-postgres:latest

Additional Context
No response

Hey @sridharstreaks - I think this is a duplicate of:

https://github.com/dbt-labs/dbt-postgres/issues/96

Please see this comment for a full explanation and discussion of the potential paths forward:

https://github.com/dbt-labs/dbt-postgres/issues/96#issuecomment-2113020011


SCP-1861 Marlowe Playground Client: Implement holes in blockly

This PR implements blockly holes, which is a way to suggest to the user what blocks could fit into the selected block.
I had to upgrade blockly to the latest version (and while I was at it modify the source to the official one) as it was necessary for defining the Toolbox as JSON. Then I re-implement the toolbox from XML to Json and finally added the mechanism to modify the Toolbox with the hole information.
As a "weird" usability issue, the names of the toolbox categories correspond to the name of the holes rather than the name of the block. But trying to change that broke the CodeToBlocklyToCode quick check, as the names of the holes is tightly coupled to the block definition, and the quick check verifies that the names of the holes are preserved.
The code is deployed here: https://hernan.marlowe.iohkdev.io/

Pre-submit checklist:

Branch

[x] Commit sequence broadly makes sense
[x] Key commits have useful messages
[x] Relevant tickets are mentioned in commit messages

PR

[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

If you updated any cabal files or added Haskell packages:

[ ] nix-shell shell.nix --run updateMaterialized to update the materialized Nix files
[ ] Update hie-*.yaml files if needed

If you changed any Haskell files:

[ ] nix-shell shell.nix --run fix-stylish-haskell to fix any formatting issues

If you changed any Purescript files:

[x] nix-shell shell.nix --run fix-purty to fix any formatting issues

Pre-merge checklist:

[x] Someone approved it
[x] Commits have useful messages
[x] Review clarifications made it into the code
[x] History is moderately tidy; or going to squash-merge

Yes, basically thats the reason. After doing this PR I saw that we can change the shape with foreign-generic using stuff like this. But I need to check how it would work with optional fields (the catMaybe parts).
In any case, argonaut-core was already included in the dependencies and it supposedly ready to be used by servant-purescript. We should check if it's worth moving code to foreing-generic/argonaut ecosystem, or if its fine to have a mix. It would also be nice to see what direction is the community heading to.


Degradated performance in recent versions

Installations take a long time in projects with few dependencies, this goes better when the project has many dependencies.

[ ] Related to a RAM issue?

Looks like this problem is related to duplicated dependencies with different versions.

[ ] Compare installed versions and skip if the package is suitable.
[ ] Cache manifests to speed up fetching.


ci: Add SonarCloud integration and configuration

Pull Request
Description
Integrates SonarCloud code quality scanning into the CI pipeline. Added a new SonarCloud scan step to the code-test workflow and configured source path overrides to ensure correct coverage reporting. Created a new sonar-project.properties file that defines the organization, project key, Python version, and coverage reporting settings. Test files are excluded from coverage analysis.
The integration will provide automated code quality metrics, security vulnerability scanning, and coverage reporting through SonarCloud's platform.
fixes #102

Merge activity

Oct 24, 8:44 AM EDT: A user merged this pull request with Graphite.


Re: issue 52 - creating constructors that inherit from Chart

(I accidentally created this in my own repo yesterday--still figuring github out!)
I was going to take a stab at making a donut chart, then thought it might be useful first to make part of the Chart constructor reusable for different kinds of charts. I've extracted a separate constructor for the moving block chart and added some basic inheritance (all in prototype-1a.js).

That sounds good. Thanks, @ptgott .  Can review and resolve conflicts at tonight's work session. Will you be there? I get the gist of what you've done but would have some questions.

Sounds good. I'll be there!
On 31 January 2017 at 10:58, John Osterman<EMAIL_ADDRESS>wrote:

That sounds good. Thanks, @ptgott https://github.com/ptgott . Can
review and resolve conflicts at tonight's work session. Will you be there?
I get the gist of what you've done but would have some questions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codefordc/housing-insights/pull/64#issuecomment-276404236,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYFLbOkJrAocdgLHpG5pSZOhWfBFrOSsks5rX1odgaJpZM4Lywvy
.

Merged 52-chart-inheritance.


HTML Iframes

HTML IFrames 📜
I would like to contribute in the documentation part of HTML iframes with explanations and examples of how to use it .

Domain of Contribution 📊

[x] Frontend Dev (HTML, CSS, JS)

/assign

/assign

/assign


write command format to request for RPM and Speed

I have tried RxAndroidble library for connecting to ble device but i dont know how to send command to get value from ble device.Right now i am sending the value of 0105 for engine coolant temprature value and it gives me write success message but i dont know how to get value of this parameter.does anybody help me with this?

Duplicate of #446


Added contribute.json to generated project

Tried to fix #65. Hope I am on a good track.

Thanks for contributing @tyler46. I'll review tomorrow

Can one of the admins verify this patch?

ok to test

Nice work @tyler46 . Please update and ping me here for another pass.

Oh and please add a simple test here
https://github.com/mozilla/sugardough/blob/master/{{ cookiecutter.project_name }}/{{ cookiecutter.project_name }}/base/tests.py
that hits /contribute.json and makes sure that content is returned and that the application/json Content-Type header is set

thank you for reviewing my pr. I will update it and ping you for another review.

@glogiotatidis can you review it again please?

Squashed and merged here. Thanks @tyler46  :cake:


Saber/uhf

Added Microsoft UHF and moved Announcement bar to below navbar

@microsoft-github-policy-service agree company="Catalysis"

@microsoft-github-policy-service agree

@microsoft-github-policy-service rerun


Make Spinning Animations Quicker

Hi, Is it possible when calling progressDemoHelper(success: true), to make the spinning animations quicker before proceeding to the next step, it's taking too long, I would like to speed it up if possible
Thanks

What kind of method is that?


[FR] option to disable notification

hi, thanks for the great plugin!
the notification is particularly annoying - especially in a place like obsidian, which is one of the few without context switching and attention problems

I wanted that for some time too. #56 provides that.

@gforien Thanks for the contribution! This feature will be available in the next release.


-ZapEnabled option not found?

Hello
We are using Powershell to implement quick SPAM filter setting inside Office365, but recently we encountered the problem that the "-ZapEnabled" option is not found.
Is this option replaced with another option? On the document page here, the option is still available, but inside the command not anymore.
Thanks for feedback about this case.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 164c3337-1862-be67-7e1d-ea8055b88791
Version Independent ID: 3cb06add-3cb6-7cb8-779b-a681e03ab4fb
Content: Set-HostedContentFilterPolicy (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/Set-HostedContentFilterPolicy.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@chrisda Thank you very much for sharing this explanation. @duracellkonijn22 Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Tiler module

Tiling module that generates 256x256 tiled xarray datasets from the larger scene-level datacube generated in datacube.py.
To do:

[x]  add cloud filter
[x]  refine extent to tile within to valid data region
[x] add main script to call the datacube module followed by the tile module
[ ] decide on method for storing the tiled xarray datasets
[x] add some docstrings
[x] plot tiles to make sure they look good

FYI I am using the following snippet to plot the merged data for visual cross-checks:
def plot(merged):
    """
    Plot a tile in RGB and vh, and DEM
    """
    import numpy
    from matplotlib import pyplot as plt

    # data = merged.coarsen(x=100, y=100, boundary="trim").mean()
    data = merged

    fig, axs = plt.subplots(3)

    rgb = (
        numpy.array(
            [
                data.get("B04"),
                data.get("B03"),
                data.get("B02"),
            ]
        ).clip(0, 3000)
        / 3000
    )
    axs[0].imshow(rgb.transpose(1, 2, 0))
    axs[1].imshow(numpy.log(data.get("vh")))
    axs[2].imshow(data.get("DEM"))

    plt.show()

The tiler function seems to work nicely. Here is a plotted result [RGB, VH, DEM] in order from top to bottom.

Added a catch for sampled S2 reference dates which don't have S1 scenes within the +/- 3 day surrounding interval. This should prevent any failures due to unavailable matchup as it just triggers a new sampling for S2.

Going to merge this to keep things moving. Will address the tile saving method in a follow-up PR.


Update youtube-dl.json

Updated version to 2015.07.07

Cool :+1:

What if we write something that scrapes the version number from respective websites automatically? There is manual work involved for every program under scoop, but it would keep all software up to date. Can I start writing?

There's already a checkver script that does what I think you want to do. There's some usage info here, and the app manifest reference has some more info about how to add this to your manifest.

What I meant was,
say gcc, the current url in manifest is,
http://ufpr.dl.sourceforge.net/project/mingwbuilds/host-windows/releases/4.8.1/64-bit/threads-posix/seh/x64-4.8.1-release-posix-seh-rev5.7z
This 4.8.1 version number affects the url once a new update for gcc comes out.
Why did I create this pull request to update the manifest? What if all manifests can be updated automatically?

Thanks, found it.

Oh also, did you see @guillermooo's script to update Dart manifest's automatically?


[Schema Registry Avro] Replace AvroError with Error with cause

Packages impacted by this PR
@azure/schema-registry-avro
Issues associated with this PR
Describe the problem that is addressed by this PR
Replacing AvroError with Error and using the cause proposed feature.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
N/A
Are there test cases added in this PR? (If not, why?)
N/A
Provide a list of related PRs (if any)
N/A
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check for @azure/schema-registry-avro
API changes have been detected in @azure/schema-registry-avro. You can review API changes here
API changes
- export declare class AvroError extends Error {
-     constructor(message: string, options?: {
-             innerError?: unknown;
-             schemaId?: string;
-         });
-     innerError?: unknown;
-     schemaId?: string;
- }


[UI] Update ISO permissions

Description
Add UI support for the existing updateIsoPermissions API.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):

How Has This Been Tested?
Register ISO -> Update ISO permissions

@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔debian. JID-1133

@shwstppr do we have this for Primate as well? @nvazquez can you check and send a PR for Primate as well?

@rhtyd already there for Primate https://github.com/apache/cloudstack-primate/issues/278

Thanks @shwstppr!
@rhtyd yes, I have tested manually the Primate PR # 285
@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔debian. JID-1150

@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔debian. JID-1154

@blueorangutan package

@vladimirpetrov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔debian. JID-1188

@blueorangutan test

@vladimirpetrov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan test

@vladimirpetrov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-1470)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 36755 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4017-t1470-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_iso.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_supported_versions.py
Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Smoke tests completed. 80 look OK, 3 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_04_extract_Iso
Failure
128.30
test_iso.py

test_01_add_delete_kubernetes_supported_version
Error
636.22
test_kubernetes_supported_versions.py

test_02_vpc_privategw_static_routes
Failure
235.84
test_privategw_acl.py

test_03_vpc_privategw_restart_vpc_cleanup
Failure
270.75
test_privategw_acl.py

test_04_rvpc_privategw_static_routes
Failure
338.94
test_privategw_acl.py

Looks like has been tested. LGTM, let's merge.


[FEATURE] Create DenyList.yml

Is your feature request related to a problem?
We currently have AllowList.yml. However this isn't generally feasible for many use cases. A more reasonable use case is when a cluster admin wants to explicitly deny permission to a certain few endpoints. We should provide a mechanism to allow for this, instead of blocking all endpoints except for a few.
What solution would you like?
An equivalent to AllowList.yml, but for denial.
What alternatives have you considered?
None
Do you have any additional context?
No

[Triage] Thank you for filing this issue @derek-ho. Can you provide an example configuration?


Migrate androidx

should migrating androidx.

no need for that, android does that for the publisher app: https://developer.android.com/jetpack/androidx/migrate


When transforming new data using an existing UMAP object, output array is entirely NaNs

I have two UMAP transform objects that I've "trained" on two separate datasets, one 8000x512 and one 8000x1024 in dimension. Both of these transform objects produce a successful and reasonable embedding in 2 dimensions using the training set. I have an additional 160k rows of data for each of the two feature sizes (512 and 1024), which I am using as "test" data. When I transform the 160k x 512 array into 160k x 2, there is no issue. However, the content of the 160k x 2 array that is produced when I transform the 160k x 1024 input array is entirely NaNs. I have already verified that the inputs do not contain any NaNs or infinities. Additionally, taking a smaller subset of the input set, such as 10 rows, also produces a 10x2 output that is full of NaNs. Is there any reason why this might be the case? I'm wondering if perhaps there is an inherent size limit to the number of features for new data that is not a constraint on the original data.

I also tried using a new input numpy array of random floats in the same range as my test input, and encountered the same issue as below:
test_rand_input = np.random.uniform(low=0.0, high=np.max(umap_test_set_dense), size=(100, 1024)) test_rand_output = umap_model_25nn_dense.transform(test_rand_input) print(test_rand_output)
Getting this as the output:
[[nan nan]  [nan nan]  [nan nan]  [nan nan]  [nan nan]  [nan nan]  [nan nan]  [nan nan]  [nan nan]  [nan nan]]

There were definitely some bugs in earlier versions of UMAP that could cause this to happen. Do you know what version you are using? Can you reproduce the problem on a build of umap directly from the current master?


OSMBright TM2Source improved

Completely functioning OSM Bright TM2Source style.
Check also:
https://www.mapbox.com/developers/vector-tiles/

The #25 is duplicate to this ticket now.


Update release notes for 2.0.17 release

FYI: @jasonrshaver @sptramer @mayurid

@derekbekoe,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

latest/docs-ref-conceptual/release-notes-azure-cli.md
:white_check_mark:Succeeded
View (azure-cli-latest)

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


Add Homer uniqmap

Generating uniqmap
Unfortunately, generating your own uniqmap files is a clunky procedure. However, you can do it by following these steps:

Place the genome FASTA files for the genome of interest in a directory. There MUST be only one entry per file (i.e. chr1.fa, chr2.fa, chr3.fa). Unfortunately, the programs will not accept a single FASTA file will all of the chromosomes.
Next, run the getMappableRegions command. This will eat up a ton of memory, which can be scaled by adjusting the first argument which sets the number of sequences that are analyzed in parallel. The program takes a while on mammalian genomes (~day). The second argument is the length of the reads you want to consider unique: "getMappableRegions<PHONE_NUMBER> 50 *.fa > out.50nt.txt"
Next run a utility program to create the uniqmap directory: "homerTools special uniqmap uniqmapDirectory/ out.50nt.txt". The "uniqmapDirectory/" can then be used with the findPeaks command.

Source
Might be useful for Nascent, I don't know if anything besides GRO-Seq uses it.

Implemented in https://github.com/Functional-Genomics-Lab/homer-uniqmap-nf
Probably a better place for it, as it's pretty niche. If anyone needs any help, feel free to comment here, open an issue at homer-uniqmap-nf or find me in the Nascent channel of the nf-core slack!


massrename

Description
Svedberg_Ornithoscopy
Change List
Please include a summary of the change and which issue is fixed.
Reasons for the PR
Please also include relevant motivation and context.
New Dependencies and Reason
List any dependencies that are required for this change and the reason for the dependency add.
Issues List
Closes # (issue)
Additional context
Add any other context or information about the problem here.
List of Developers
This feature will be implemented by the following Developers:

@FeeeDz

Type of changes
Please check relevant options.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings

NOTE: the PR have to be labelled in-progress until the work is complete, the PR will be checked, reviewed and merged only after being labelled as complete

La PR deve essere fatta sul branch hackathon, non main


Add overview of element children shortcuts

We now have an overview of the shortcuts active from the point of view of a single element. Essentially, these are the shortcuts defined in the element or in any of its parents.
This is nice when we want to learn what's possible to do within the current context. At the same time, to improve discoverability, we can benefit from an overview of the shortcuts of all children. We need a view for that.


Powershell ps1 filetype not working

Although it already has a set commentstring to #%s

It is fixed now. Please use master.


Add support for selecting the runner version

Hi! Nice to meet you! First of all thanks for creating this tool and GH Action ❤
I'm trying to push for the adoption of this GH Action at @stackbuilders for some of the repos we maintain. I was having a slightly unrelated conversation with regards to the macOS runners that GitHub offers, and wanted to test the macos-14 one since that uses Apple Silicon (macos-latest is based on macos-12 which is currently x86-64 based AFAIK). But I noticed that this tool doesn't currently support that, and that it has the *-latest versions hardcoded.
I think it would be nice to support choosing the runner version for each OS. Adding a new input to the action that allows one to set an explicit list of the OS runners one wants to support could be a possible approach.
I am willing to try to work on this, unless you prefer doing that. Let me know if you have any ideas/preferences with regards to how you would like to see this reflected in the action.

@DavidMazarro hi David, glad to hear this is action is getting traction! I will create a PR for this. Do you mind if I ask a review from you once it's ready?

@Kleidukos By all means! Thanks for taking a look at this ^^

@DavidMazarro Turns out it was simpler than expected. Would you mind trying out v<IP_ADDRESS>? The instructions are in the README.

It works! Feel free to close this issue. I might have other ideas for potential ways to make the tool/GH Action more expressive in terms of runner/GHC version possible configurations, but I'll create issues for those (before that, I'll consider if they are feasible without going against the philosophy of the tool).
Thank you so much!


JSON data per page

What needs to be done to have the data be available per page instead of for the entire site?
I would like to access page-specific data from an API, so I tried to add this to the pages' frontmatter. E.g. a page for every group has data defined in a GitLab group, so I need to access the group-specific JSON URL per page:
jekyll_get_json:
  - data: group
    json: 'https://gitlab.domain.org/api/v4/groups/162'

This does not seem to work. I guess it would need a change in this line:
  config = site.config['jekyll_get_json']

So what can I exchange this with, to be able to configure the json URL separately on every page? Will this work at all or are pages' frontmatters only parsed after the plugin?
Sorry, I am not very well-trained in writing Jekyll plugins or even Ruby, and so I am only guessing…

https://github.com/avillafiorita/jekyll-datapage_gen


Correcting a Time#gmtoff test in optional/capi/time_spec.rb

This pull request corrects a test that can fail when performed during summer time (daylight-saving time). Time#gmtoff values match only when both or neither of the Time objects are in summer time. The original test compares Time.now.gmtoff with the gmtoff of a Time object representing "2015-11-09", so it succeeds or fails, depending on today's date, if performed in a place which observes summer time. A more robust test is to use the Time objects derived from the same time (Time.now in this case) on both sides of the comparison.

Thank you for your contribution and the detailed explanation!


brctl is obsolete and should be replaced in code

Description
brctl command is obsolete and bridge-utils package is not in REHL8, Oracle Linux 8 and in upcoming CentOS 8. There is also no package in EPEL-8.0 release.
brctl calls in VNM drivers should be replaced by ip link commands.
Details

Affected Component: Network
Hypervisor: KVM, LXD
Version: All

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

Also related to brctl #3522

PRs:

one https://github.com/OpenNebula/one/pull/3654 to master
docs https://github.com/OpenNebula/docs/pull/736 to master

just note, bridge-utils dep have to be removed from packing spec files in this repo https://github.com/OpenNebula/packages


Output of %#v is not gofmt clean

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
go version go1.6.3 linux/amd64

What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/hhecker/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

What did you do?
If possible, provide a recipe for reproducing the error.
The following is used to generate code via go run example.go > output.go:

package main

import "fmt"

func main() {
        data := map[string]interface{}{"key": "value"}
        fmt.Printf("package main\n\nvar data = %#v\n", data)
}

Running gofmt -d on the output shows
diff output.go gofmt/output.go
--- /tmp/gofmt472599174 2016-08-09 08:24:53.951200879 +0200
+++ /tmp/gofmt828973101 2016-08-09 08:24:53.951200879 +0200
@@ -1,3 +1,3 @@
 package main

-var data = map[string]interface {}{"key":"value"}
+var data = map[string]interface{}{"key": "value"}

What did you expect to see?
Nothing :)

What did you see instead?
the above whitespace changes

I don't see why every piece of output a go program produces has to be as go
fmt would produce it. We have a tool that can do this for us so we don't
have to nickle and dime every single piece of output.
On Tue, 9 Aug 2016, 16:27 Hanno Hecker<EMAIL_ADDRESS>wrote:

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
go version go1.6.3 linux/amd64
2.
What operating system and processor architecture are you using (go env
)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/hhecker/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"
3.
What did you do?
If possible, provide a recipe for reproducing the error.
The following is used to generate code via go run example.go >
output.go:
package main
import "fmt"
func main() {
data := map[string]interface{}{"key": "value"}
fmt.Printf("package main\n\nvar data = %#v\n", data)
}
Running gofmt -d on the output shows
diff output.go gofmt/output.go
--- /tmp/gofmt472599174 2016-08-09 08:24:53.951200879 +0200
+++ /tmp/gofmt828973101 2016-08-09 08:24:53.951200879 +0200
@@ -1,3 +1,3 @@
package main
-var data = map[string]interface {}{"key":"value"}
+var data = map[string]interface{}{"key": "value"}

What did you expect to see?
Nothing :)
2.
What did you see instead?
the above whitespace changes
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/16645, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAcA5LOaNjHw2CLgVr4LmPXaCVc6Ahfks5qeB3dgaJpZM4JfwZa
.

I agree with Dave. We have go/printer, so if you want gofmtted output, you
could use that. Embedding the full gofmt rule into the fmt package seems
unnecessary.

I agree that the general goal is pointless.  And this specific case is not due to the fmt package at all, but is simply a reproduction of the type string generated by the compiler, which would be somewhat painful to change.  Closing.  Sorry.


Change to use internal interface->enp0s1f0d4(d4), on ACC, for host<->ACC communication

Change to use internal interface->enp0s1f0d4(d4), on ACC, for host<->ACC communication. Previously d2 interface was used on ACC, which was meant for external network interface. Using external interface D2 also caused issue when fixed(unique) MAC was set thro fixboard script, since VSP checks for vportid in MAC. Since above change would reduce the number of APFs returned by GetDevices by 1, value of ->acc_apf in node-policy has been increased to 16.

@vkannapp thanks, I have removed one of the debug print. Current VSP code uses fmt.* mostly, will have a separate PR, to convert them to log.* wherever applicable.

Approved. The changes looks good. Please remove the debug printf if it is not intended.
@vkannapp thanks, I have removed one of the debug print. Current VSP code uses fmt.* mostly, will have a separate PR, to convert them to log.* wherever applicable.


Add backend to this app

It would be great if we add backend to this app coz data.json file is not scalable option.
@vijayyprajapati add hacktober label to this issue coz this issue seems relevant for hacktober.

I would like to work on this issue. Assign it to me.

Go ahead @deepakgohil009

Thanks @vijayyprajapati for assigning me during development of backend I may require to create couple of issues. So I hope you approve that as well.


More informative message for missing backend dependencies

Currently, if you don't have the Python package redis intalled
then you get the same message if you type:
requests_cache.install_cache(backend='redis')

as if you type:
requests_cache.install_cache(backend='nonsense!!!!')

Thanks!


routing: improve equal cost route comparison

When the (virtual) payment attempt cost is set to zero, probabilities are no longer a factor in determining the best route. In case of routes with equal costs, we'd just go with the first one found. This commit refines this behavior by picking the route with the highest probability. So even though probability doesn't affect the route cost, it is still used as a tie breaker.

Test added. Also ran into the heap comparison, which I updated as well.


Read-Only TxnContext Interface; Read-Only (single-statement select, txn) optimizations

#1396

Removed default read_only=false from TransactionManager::BeginTransaction
Removed TransactionContext::SetReadOnly
Added read-only param to TransactionContext constructors
Transactions with no modifying queries treated as read-only
4.1 assumption: if transaction is marked read-only, then READ transactions are not added to ReadWriteSet

#1395
Single-statement selects are marked as read-only

Currently, I am only failing copy_test (1/181).
If I add traffic_cop.SetStatement(statement) here, I will pass. The question is, should I do that?
I'm looking into what else is using SetStatement right now.

Coverage decreased (-0.5%) to 76.452% when pulling 5711dc221ac81af21ba76b45d93c745d057c8be2 on lmwnshn:txnctx/readonly into fa6b48c5622becb4ae51a7357b23d42f714e86f4 on cmu-db:master.

The more I look at this, the more it seems like we're building a hack on top of an unstable foundation. We're trying to find a shortcut if the RWSet only contains Reads/is empty. However, we shouldn't even be putting Reads in the RWSet in the first place. @yingjunwu says as much. We only add READ_OWNs for the sake of releasing the SpinLatch at Commit/Abort.
Commit and Abort do nothing if an element is READ when they iterate through the RWSet, so they shouldn't really be there in the first place.
This PR looks fine on its own, but it seems like a workaround for what a larger design issue that should be addressed.

I am not sure what you guys are currently working on, but I wanna mention that the original rwset was design to be general enough -- We didn't assume the underlying CC protocol, and the rwset was supposed to be compatible with any protocol, like OCC and 2PL. While our experiments show that timestamp ordering performs best, it seems that some people (e.g., Phil Bernstein) have a strong faith in 2PL :-)

So that’s sort of emblematic of the issues I feel like I’m finding in the current system. Things like maitaining a doubly-linked version chain, reads getting added to the RWSet, etc. seem to be preventing us from making the best MVTO system possible.
I definitely see why it was necessary in the context of comparing CC systems, but if MVTO is what we’re sticking with I think we can improve it.

@mbutrovich Do we want to merge this PR?

@apavlo Yes. @lmwnshn might need to do a rebase first, but I still think it should be merged.

@mbutrovich Undid the formatting changes


Fix filter no features in duckdb index cache

fix https://huggingface.co/datasets/fka/awesome-chatgpt-prompts/viewer/default/train?f[act][min]=4&f[act][max]=8&f[act][transform]=length

This was caused by RuntimeError: The indexing process did not store the features.

Codecov Report
Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (a0a75be) 90.90% compared to head (6b97c2a) 80.61%.
Report is 24 commits behind head on main.

Files
Patch %
Lines

services/search/src/search/routes/filter.py
14.28%
6 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #2347       +/-   ##
===========================================
- Coverage   90.90%   80.61%   -10.29%
===========================================
  Files         188        9      -179
  Lines        9541      454     -9087
===========================================
- Hits         8673      366     -8307
+ Misses        868       88      -780

Flag
Coverage Δ

jobs_cache_maintenance
?

jobs_mongodb_migration
?

services_admin
?

services_search
80.61% <25.00%> (+0.04%)
:arrow_up:

services_sse-api
?

services_worker
?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Do you know why there might be no features? I remember having asked when creating the /filter endpoint and I was told that we could safely assume there were features (only very exceptionally we would not find and we could fail in that case)...

Because the first version of the "config-parquet" job didn't store the features and now it does.
So old datasets may have features: None for subsequent processing jobs like split-duckdb-index
It's not a big deal since all the image and audio datasets (those that actually require the "features" info in the cache) were refreshed at that time. So for all the datasets with missing "features" we can infer the features from e.g. the pyarrow schema

maybe add a comment in the code to make it clear?

I found the conversation: https://github.com/huggingface/datasets-server/pull/1418#discussion_r1334517809

You can raise an error if it's None imo.
It can only happen in very rare cases (parquet dataset processed with old version of datasets for example)


GROUP BY mutiple columns

The group by only support 1 column, array desired .

It's now supported.

Pull request #190 is probably supported too.
Actually I moved to respect relational. This project look to be abandoned

In version 1.2.1, multiple GROUP BY is still not working for me. Still getting the following error:

expects parameter 2 to be string, array given

As far as I understand you have to fork this repo because it only support
owner projects' requirements
El dom., 30 de abr. de 2017 10:41, sebam91<EMAIL_ADDRESS>escribió:

In version 1.2.1, multiple GROUP BY is still not working for me. Still
getting the following error:
expects parameter 2 to be string, array given
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/catfan/Medoo/issues/191#issuecomment-298233034, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJ5pQco4GkqyDEmk_2BLb3-18IdnbceXks5r1I9rgaJpZM4DVyZK
.

@sebam91 Oh, sorry. I mixed this issue up with ORDER by.
It's added on develop branch now, and will be release soon. 8c796f48eb


[BUG]: - Copywrite section should be attached to the bottom

Assign this task to me and I will fix it @samratghosh291?

Also add "developed by

Done.


add language q to languages.yml

sample search for in the wild repositories:
user:AquaQAnalytics extension:q
more examples see:
http://kxsystems.github.io/

Marking as stale as this appears to have been WIP since inception.
If you're like to continue this PR, please review Adding a Language.

ok, yes, it’s been hanging for a while - sorry about that
I’ve read the adding a language doc, but it seems very very complicated to
add a language that isn’t in
the whole textmate grammar compatible world. Is it possible for me to ask a
couple of questions of someone
on the linguist team to sort out what a minimum profile would be to get the
language added?
On 27 April 2017 at 03:21, Colin Seymour<EMAIL_ADDRESS>wrote:

Marking as stale as this appears to have been WIP since inception.
If you're like to continue this PR, please review Adding a Language
https://github.com/github/linguist/blob/master/CONTRIBUTING.md#adding-a-language
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/github/linguist/pull/3398#issuecomment-297584906, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA1FQvR0Qlb3oz-W6yUGb5Kod8fqo9MQks5rz-2BgaJpZM4LR-ei
.

Hey @simongarland,
For a language to be formally recognised on GitHub, it needs to meet the usage criteria. I'll highlight the relevant snippet:

We try only to add new extensions once they have some usage on GitHub. In most cases we prefer that extensions be in use in hundreds of repositories before supporting them in Linguist.

Part of adding a new language is proving that it's being used by enough users for it to warrant official recognition.

yep, the hundreds of repositories isn’t the problem - we can drag that up
for you
it’s the requirement for a textmate grammar/syntax file, as these languages
are in the vim/emacs corner
and nobody’s needed to get these things set up and blessed/included by the
various editor vendors
On 28 April 2017 at 16:02, John Gardner<EMAIL_ADDRESS>wrote:

Hey @simongarland https://github.com/simongarland,
For a language to be formally recognised on GitHub, it needs to meet the
usage criteria. I'll highlight the relevant snippet:
We try only to add new extensions once they have some usage on GitHub. In
most cases we prefer that extensions be in use in hundreds of
repositories before supporting them in Linguist.
Part of adding a new language is proving that it's being used by enough
users for it to warrant official recognition.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/github/linguist/pull/3398#issuecomment-298006015, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA1FQk81asrwpgSorSnJr9ZZqIOdfQi5ks5r0fFjgaJpZM4LR-ei
.

Those are optional. There's no requirement to include a language-grammar. =) Languages can still gain formal recognition on GitHub without highlighting.

ah great, then I must have misunderstood / mis-submitted the request
I’ll retry and let you know when it’s done - if you’d have the time to
quickly look over
what I’ve done in case it’s easily corrected that’d be very much
appreciated.
On 28 April 2017 at 16:46, John Gardner<EMAIL_ADDRESS>wrote:

Those are optional. There's no requirement to include a language-grammar.
=) Languages can still gain formal recognition on GitHub without
highlighting.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/github/linguist/pull/3398#issuecomment-298017869, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA1FQngxcT6ICqPj7xMdxzCWPw3WpTK8ks5r0fu4gaJpZM4LR-ei
.

so, just to check that I understand what a minimal submission includes,
please correct if I still haven’t got it:

I need to submit an amended <languages.yml> with an entry for Kdb:

Kdb:
type: programming
color: "#3F3F3F"
extensions:

".q"
".k"
interpreters:
q
tm_scope: none
ace_mode: text
language_id: NNN

where I generate the NNN and fill it in using set-language-ids

I should add a couple of samples of .q and .k files

I should point you at a big enough list of existing repositories that
you can accept that there is a
community of users who’d benefit from having  recognised as a language

that’s it..
On 28 April 2017 at 16:46, John Gardner<EMAIL_ADDRESS>wrote:

Those are optional. There's no requirement to include a language-grammar.
=) Languages can still gain formal recognition on GitHub without
highlighting.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/github/linguist/pull/3398#issuecomment-298017869, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA1FQngxcT6ICqPj7xMdxzCWPw3WpTK8ks5r0fu4gaJpZM4LR-ei
.

@simongarland Yup, that's pretty much it.  You may also want to substantiate your choice of colour too just for clarity.

should be all ready now ..

From: Colin Seymour<EMAIL_ADDRESS>Sent: Wednesday, May 10, 2017 5:08:28 PM
To: github/linguist
Cc: Simon Garland; Mention
Subject: Re: [github/linguist] add language q to languages.yml (#3398)
@simongarlandhttps://github.com/simongarland Yup, that's pretty much it. You may also want to substantiate your choice of colour too just for clarity.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/github/linguist/pull/3398#issuecomment-300512620, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA1FQkWqrZsqT-buLBiSqnFGoB2c2lZNks5r4dLsgaJpZM4LR-ei.


feat: add dark mode toggle button

feat: add dark mode toggle button

@ssarkar551 resolve merge conflicts

resolved merge conflicts

Re-request review after applying the changes @ssarkar551


[WIP]feat: support xa oracle connection

What this PR does:
Which issue(s) this PR fixes:

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

Codecov Report

Merging #391 (4604d46) into master (334879e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #391   +/-   ##
=======================================
  Coverage   43.25%   43.25%
=======================================
  Files         123      123
  Lines        7146     7146
=======================================
  Hits         3091     3091
  Misses       3830     3830
  Partials      225      225

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: jasondeng1997:x: raspberry-huYou have signed the CLA already but the status is still pending? Let us recheck it.


production names

Glyphs can store production names in the .glyphs file now. This is mainly useful i you need to work with different clients and different name requirements.
This should be supported by glyphs2ufo. The biggest problem its that the feature code needs to be converted. I have some code in Glyphs that I could port over to python but maybe there is something already there.

@schriftgestalt has just published a subset of GlyphData.xml (just the basic stuff i.e. Unicode-to-Glyphs-glyphname mapping and some categorization, without the additional metadata) under the MIT license:
https://github.com/schriftgestalt/GlyphsInfo
This can be used as an auxiliary source for converting the internal Glyphs glyphnames into production glyphnames.

The biggest problem its that the feature code needs to be converted. I have some code in Glyphs that I could port over to python but maybe there is something already there.

All the feature code in a UFO is stored in a single plaintext file, so maybe just some simple find+replace operations would be sufficient? Another concern is that components are referenced by glyph name. Not sure if there's anything else.

Renaming at the end sounds more robust to me.  find+replace sounds scary.

Is there an easy way to do renaming with fontTools?

The basic principle is: UFO uses glyph names which are not necessarily PostScript glyph names at all. The FEA code “sort of” uses PostScript glyph names, but the assumption is that they’re not necessarily that (especially since AFDKO now accepts characters in glyph names used within FEA which are not valid characters within PostScript glyph names).
There is currently no way to store PostScript glyph names in UFO. I opened a discussion 19 months ago:

https://groups.google.com/forum/#!topic/ufo-spec/pJnk5x1_6pE
https://github.com/unified-font-object/ufo-spec/issues/9
but it hasn’t gone very far.

AFDKO uses its own GOADB file which allows glyph renaming after the GSUB/GPOS tables have been compiled.

Is there an easy way to do renaming with fontTools?

It's not complete, but can be done.  See https://github.com/behdad/fonttools/issues/149

I recently needed a way to set production names in UFO files as well, may be we can agree on something in the glif lib and then ufo2ft and other tools would use it?

Maybe glif > lib > public.productionname?

Or maybe postscriptName instead of productionname similar to the suggestion in https://groups.google.com/forum/#!topic/ufo-spec/pJnk5x1_6pE and to be an indicator that it needs to be a valid PostScript name?

I did like public.postscriptName suggested by @typesupply — it fits with other postscript* entries in fontinfo and makes it obvious what its purpose is.

Would you call the names in a TrueType file postscript, too? And postscriptsname sounds like one of the entries in the name table. And it is even longer.

...purpose but also heritage and set of restrictions etc. And after all in OT fonts, this name lives in the “the PostScript table” (post) or in the CFF  table which in its spec makes heavy references to PostScript.

@schriftgestalt Of course I would. They are only needed for PostScript and PDF anyway, and are optional in TrueType-flavored fonts. And they live in the “PostScript table” (official spec name)

I'm fine with both.

So I think we want to populate public.postscriptName using  1) the "production" field if present in the .glyphs data, or else 2) the production name in GlyphData.xml for this glyph.
I wonder how best to use info from GlyphData.xml? Should the file just be copied to glyphsLib?

I suggest we add the @schriftgestalt GlyphsInfo repository as a submodule for glyphsLib, and install the GlyphData.xml file though setuptools' "package_data".

And then we could use pkg_resourced.resource_filename() to retrieve the path to the packaged GlyphData.xml.

Hmm. I wonder how valuable the data in GlyphData.xml really is, and whether that's worth doing. As far as I can tell it only specifies production names for glyphs with unicode values, using a "uniXXXX" pattern that's easy enough to do manually. Though, it also only specifies production names past uni0180 (with some seemingly arbitrary exceptions?)

Entries in the GlyphData without a production name are most likely in AGLFN. The glyph list is helpful to map names to unicodes, e.g. If you need to find the Unicode of the base name for glyphs with suffixes.

The glyph list is helpful to map names to unicodes, e.g. If you need to find the Unicode of the base name for glyphs with suffixes.

That's a good point. Though, this is possible with loaded UFOs too. Something like
basename, suffix = name.split('.')
prodname = glyphData[basename]['production'] + '.' + suffix
# could be swapped out with
prodname = 'uni' + hex(ufo[basename].unicode) + '.' + suffix
# or even
prodname = ufo[basename].lib['public.postscriptName'] + '.' + suffix

Anyways, it's obvious we should at least use production names as they appear in Glyphs sources, so I'll open a PR with that for now.

Thanks Sascha!
So I guess the next steps will be tagging a v1.3.0 release for glyphsLib, bumping the minimum install_requires  of fontmake to "glyphsLib>=1.3", and finally release a new fontmake v1.2.0.
I can do that, unless you have other stuff to push to glyhpsLib.


Generalize token to owned resources

The transaction functions initialize, finalize, transfer, split, merge, send, supply-dependent resource logic functions,  and getBalance projection functions are more generally applicable.
They can be generalized by

using traits
renaming and restructuring the folders and files
renaming variables

Implemented in #32 .


bf_crc: Add final XOR and refout to example impl

This commit makes the following changes to the example implementation of
the CRC.

Fix the refin bug when FEEDING is wrongly set as STANDARD_FEEDING
when reflected input is activated under the feeding type of AUTO.
Add refout by adding the function reflect_output when reflected
output is activated. Fixes #7.
Add final xor in the crc function before returning the crc.

Signed-off-by: Solomon Tan<EMAIL_ADDRESS>

Replaced my tabs with your four spaces in the force-pushes to keep indentation constant

The CRC output should now match when copied and pasted as is and compiled with GCC. It solved my issue #7. Not sure how to test it for other cases like CRCs with irregular widths


added missing javadocs

Coverage remained the same at 95.584% when pulling d4e2d28b79e061fa92e100ac8c4483bf91b45ea5 on add-javadoc into 1967107ef89be1bd37a15e595033fe4f815d6b83 on master.


feat: provide functionality to transform TrivyV2Dtos into RawValueKpis.

The use case for this extension is that we run an external service producing and storing TrivyDtoV2s. We want to have the possibility to transform these TrivyDtoV2 objects into RawValueKPIs. Currently, we only support the transformation of TrivyDtos, which is not sufficient for us as we want to store the richer information included in TrivyDtoV2.

I have read the CLA Document and I hereby sign the CLA. Further, I agree to the storage of my GitHub account information in the signature file in https://github.com/fraunhofer-iem/spha-cla.

I have read the CLA Document and I hereby sign the CLA. Further, I agree to the storage of my GitHub account information in the signature file in github.com/fraunhofer-iem/spha-cla.

recheck


Added apostrophe in users name while generating streak embed

Earlier streak embed was generated like BadalAryas Hard75 Grind,
Now it should be BadalArya's Hard&5 Grind.

comment misleading.


git-open: init at 1.3.0

Motivation for this change
Add git open
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

How does git-open capture the in-store git binary?

@joachifm  Thanks for your feedback, I updated the PR to remove the .... About your question, git-open doesn't really capture the git binary. In short, when you execute git foo, git looks for git-foo in your PATH and calls it. A better explanation from the official git documentation:

Git subcommands are standalone executables that live in the Git exec path, normally /usr/lib/git-core. The git executable itself is a thin wrapper that knows where the subcommands live, and runs them by passing command-line arguments to them. (If "git foo" is not found in the Git exec path, the wrapper will look in the rest of your $PATH for it. Thus, it’s possible to write local Git extensions that don’t live in system space.)

For even more information:

How to integrate new subcommands
git builtin API

What I mean is, git-open assumes git is in PATH, which may or may not hold (i.e., it is a runtime impurity). I expected the reason you added git to build inputs was to ensure its availability at runtime.
For optional features i think it's okay to fail at runtime if it's clear to the user how to resolve it, but if it's a basic requirement for the program to work at all it should be patched in/wrapped.

@Mic92 sure, it's a command I use regularly, I'll add myself as maintainer
@joachifm I think now I understand what you mean. You are right, I added git to ensure its availability, but the correct way to do it would be:
  installPhase = ''
    mkdir -p $out/bin
    cp git-open $out/bin
    wrapProgram $out/bin/git-open \
      --prefix PATH ':' "${git}/bin"
  '';

Right? I'm new to NixOS, so if you see room for improvement, corrections are welcome.

yes the fix is correct.

@jlesquembre LGTM. Since this is a bash script, you could patch the source directly to avoid the indirection, but your solution is pretty standard.
I notice that the script also has implicit dependencies on xdg-open and (e)grep. The latter can be safely assumed, but xdg-open may need to be added as well. You can use the makeBinPath helper (from stdenv.lib) for convenience.

@joachifm Thanks again for the feedback, today I learned a lot about nix packages.
I added xdg_tools and gnugrep as dependencies, I hope that I didn't miss anything.

Thanks!


Logo misalignment and UI improvement on SCholoarship page

Description
The logo when schloarship page is loaded not aligning in navbar

Steps to Reproduce
If you'll click the link, you'll come to know
Expected Behavior
It should be aligned
Actual Behavior
No response
Screenshots

I would like to improve the way each scholarships' info is shown
Additional Information
Kindly assign this issue to me under hactoberfest, gssoc-ext and relevant tags.
What browser are you seeing the problem on?

[ ] Firefox
[X] Chrome
[ ] Safari
[ ] Microsoft Edge

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

@aditya-bhaumik I want to resolve these issues, please assign it to me.

this issue has been solved, so we are closing this issue.


Anvil is incorrectly caching responses

Component
Anvil
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
anvil 0.1.0 (5a257ef 2023-04-13T00:15:45.229644000Z)
What command(s) is the bug in?
anvil
Operating System
Linux
Describe the bug
I am fetching logs and I get a log from an older block than fromBlock param on the second request. Here is the log from tcpflow:
<IP_ADDRESS>.65151-<IP_ADDRESS>.09000: {"jsonrpc":"2.0","id":10,"method":"eth_getLogs","params":[{"address":"0xe7f1725E7734CE288F8367e1Bb143E90bb3F0512","topics":["0xb72b85ef2a843244f5ea1955248b0ac363732d2e6a98cc3641084dd5718ad8b5"],"fromBlock":"0x0"}]}
<IP_ADDRESS>.09000-<IP_ADDRESS>.65151: HTTP/1.1 200 OK
content-type: application/json
content-length: 1429
access-control-allow-origin: *
vary: origin
vary: access-control-request-method
vary: access-control-request-headers
date: Thu, 13 Apr 2023 13:36:03 GMT

{"jsonrpc":"2.0","id":10,"result":[{"address":"0xe7f1725e7734ce288f8367e1bb143e90bb3f0512","topics":["0xb72b85ef2a843244f5ea1955248b0ac363732d2e6a98cc3641084dd5718ad8b5","0x0000000000000000000000000000000000000000000000000000000000000001","0x000000000000000000000000f39fd6e51aad88f6f4ce6ab8827279cfffb92266"],"data":"0x0000000000000000000000000000000000000000000000000000000000000020000000000000000000000000000000000000000000000000000000000000015400000150588876ebdf5339b8ce4fd13d4c4faf6a7ec069df98f36d16611ea4a781506c16b0869c1cdeb643977de92cb3ab1ae8dc253ca726156580d7f0f435f884f81e4a914d46e4fad88842f125fa579d2be5fbbec7c29943e32a30d2b708062f8ad98749f7629053a9b2f6f4191ac8ae4befa536a4be8fbf2e10537509bfe746ff4ca3d11336ccfc7dacd3d0dcd93f2cf8487060441a7a6cb16f84f39ca65e876498142e011498032e1972700ebefe4fe9bb45e340b1415a04e25f6c85a80030bd8470895d11e735763b5ac6012596de1aa3e2ab0b73967bc2140ed8bd07f548d7a19662126d7c98cbd30d607b36da3d6a697e0696eda204d4029c4b1ac0c67d8ce2613b05f82811ad25cfe148e20657b19411223101418982f59ea0369854cabd2e272e4b5ac71384aea27512fd15414cdc1b2b51f50299056c43d40c6b9d55030c0fe672df5fb763cd844528b7735e1857c9000000000000000000000000","blockHash":"0x3c90e734d4da46ef80431a8547975d111c19ce43be961b24f8029523073a76ee","blockNumber":"0x4","transactionHash":"0xfd6b1d333e0bf0efb0de3fc22a5e02e1931a3e436619bef5ccea080043a01cb8","transactionIndex":"0x0","logIndex":"0x0","transactionLogIndex":"0x0","removed":false}]}
<IP_ADDRESS>.65151-<IP_ADDRESS>.09000: POST / HTTP/1.1
host: localhost:9000
connection: keep-alive
Content-Type: application/json
accept: */*
accept-language: *
sec-fetch-mode: cors
user-agent: undici
accept-encoding: gzip, deflate
content-length: 141

<IP_ADDRESS>.65151-<IP_ADDRESS>.09000: {"jsonrpc":"2.0","id":13,"method":"eth_getLogs","params":[{"address":"0xe7f1725E7734CE288F8367e1Bb143E90bb3F0512","topics":["0xb72b85ef2a843244f5ea1955248b0ac363732d2e6a98cc3641084dd5718ad8b5"],"fromBlock":"0x5"}]}
<IP_ADDRESS>.09000-<IP_ADDRESS>.65151: HTTP/1.1 200 OK
content-type: application/json
content-length: 1429
access-control-allow-origin: *
vary: origin
vary: access-control-request-method
vary: access-control-request-headers
date: Thu, 13 Apr 2023 13:36:04 GMT

{"jsonrpc":"2.0","id":13,"result":[{"address":"0xe7f1725e7734ce288f8367e1bb143e90bb3f0512","topics":["0xb72b85ef2a843244f5ea1955248b0ac363732d2e6a98cc3641084dd5718ad8b5","0x0000000000000000000000000000000000000000000000000000000000000001","0x000000000000000000000000f39fd6e51aad88f6f4ce6ab8827279cfffb92266"],"data":"0x0000000000000000000000000000000000000000000000000000000000000020000000000000000000000000000000000000000000000000000000000000015400000150588876ebdf5339b8ce4fd13d4c4faf6a7ec069df98f36d16611ea4a781506c16b0869c1cdeb643977de92cb3ab1ae8dc253ca726156580d7f0f435f884f81e4a914d46e4fad88842f125fa579d2be5fbbec7c29943e32a30d2b708062f8ad98749f7629053a9b2f6f4191ac8ae4befa536a4be8fbf2e10537509bfe746ff4ca3d11336ccfc7dacd3d0dcd93f2cf8487060441a7a6cb16f84f39ca65e876498142e011498032e1972700ebefe4fe9bb45e340b1415a04e25f6c85a80030bd8470895d11e735763b5ac6012596de1aa3e2ab0b73967bc2140ed8bd07f548d7a19662126d7c98cbd30d607b36da3d6a697e0696eda204d4029c4b1ac0c67d8ce2613b05f82811ad25cfe148e20657b19411223101418982f59ea0369854cabd2e272e4b5ac71384aea27512fd15414cdc1b2b51f50299056c43d40c6b9d55030c0fe672df5fb763cd844528b7735e1857c9000000000000000000000000","blockHash":"0x3c90e734d4da46ef80431a8547975d111c19ce43be961b24f8029523073a76ee","blockNumber":"0x4","transactionHash":"0xfd6b1d333e0bf0efb0de3fc22a5e02e1931a3e436619bef5ccea080043a01cb8","transactionIndex":"0x0","logIndex":"0x0","transactionLogIndex":"0x0","removed":false}]}
<IP_ADDRESS>.65151-<IP_ADDRESS>.09000: POST / HTTP/1.1
host: localhost:9000
connection: keep-alive
content-type: application/json
accept: */*
accept-language: *
sec-fetch-mode: cors
user-agent: undici
accept-encoding: gzip, deflate
content-length: 48

The issue is present in version from 2023-04-04 as well as in the most recent version. I tested the issue both on M1 Mac and x86_64 Linux.

I glanced the linked discussion,
how is this related to caching?
this looks like an off by one?

ty!
can you please share the anvil logs:
RUST_LOG=api,backend,node anvil ...
this should include the filter object, if this is parsed correctly we have a bug in filter handler impl

These are the logs:
Listening on <IP_ADDRESS>:9000
2023-04-13T13:57:21.359796Z  INFO node::user: eth_getBlockByNumber
2023-04-13T13:57:21.359815Z TRACE backend: get block by number Latest
2023-04-13T13:57:21.359800Z  INFO node::user: eth_chainId
2023-04-13T13:57:21.359805Z  INFO node::user: eth_getTransactionCount
2023-04-13T13:57:21.359894Z TRACE backend: get nonce for 0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266
2023-04-13T13:57:21.414855Z  INFO node::user: eth_sendRawTransaction
2023-04-13T13:57:21.415131Z TRACE node: Added transaction: [0x24d2e8f4382ae9ed1f7dbbeafda042d191cd5c438e1c230563d6f4283be6eee7] sender=0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266
2023-04-13T13:57:21.415176Z TRACE backend: creating new block with 1 transactions
2023-04-13T13:57:21.415189Z TRACE backend: [0x24d2e8f4382ae9ed1f7dbbeafda042d191cd5c438e1c230563d6f4283be6eee7] executing
2023-04-13T13:57:21.416198Z TRACE backend: [0x24d2e8f4382ae9ed1f7dbbeafda042d191cd5c438e1c230563d6f4283be6eee7] executed with out=Create(b"", Some(0x5fbdb2315678afecb367f032d93f642f64180aa3)) exit_reason=Continue gas_used=851598
2023-04-13T13:57:21.416210Z TRACE backend::executor: transacted [0x24d2e8f4382ae9ed1f7dbbeafda042d191cd5c438e1c230563d6f4283be6eee7], result: Continue gas 851598
2023-04-13T13:57:21.416216Z TRACE backend: New contract deployed: at Some(0x5fbdb2315678afecb367f032d93f642f64180aa3)
2023-04-13T13:57:21.416367Z TRACE backend: Mined block 1 with 1 tx [0x24d2e8f4382ae9ed1f7dbbeafda042d191cd5c438e1c230563d6f4283be6eee7]
2023-04-13T13:57:21.416375Z  INFO node::user:
2023-04-13T13:57:21.416377Z  INFO node::user:     Transaction: 0x24d2e8f4382ae9ed1f7dbbeafda042d191cd5c438e1c230563d6f4283be6eee7
2023-04-13T13:57:21.416379Z  INFO node::user:     Contract created: 0x5fbdb2315678afecb367f032d93f642f64180aa3
2023-04-13T13:57:21.416381Z  INFO node::user:     Gas used: 851598
2023-04-13T13:57:21.416384Z  INFO node::user:
2023-04-13T13:57:21.416387Z  INFO node::user:     Block Number: 1
2023-04-13T13:57:21.416389Z  INFO node::user:     Block Hash: 0x7d6b0ee4a64e41d3aa7ec3e10a3bad879f3c3b7cd021e4d91e5afc984d2c1fbf
2023-04-13T13:57:21.416392Z  INFO node::user:     Block Time: "Thu, 13 Apr 2023 13:57:21 +0000"

2023-04-13T13:57:21.416416Z TRACE backend::fees: updated base fee 882096650
2023-04-13T13:57:21.416420Z TRACE node: mined block 1
2023-04-13T13:57:21.416776Z  INFO node::user: eth_blockNumber
2023-04-13T13:57:21.418209Z  INFO node::user: eth_getTransactionReceipt
2023-04-13T13:57:21.458018Z  INFO node::user: eth_chainId
2023-04-13T13:57:21.458082Z  INFO node::user: eth_getTransactionCount
2023-04-13T13:57:21.458086Z TRACE backend: get nonce for 0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266
2023-04-13T13:57:21.458103Z  INFO node::user: eth_getBlockByNumber
2023-04-13T13:57:21.458108Z TRACE backend: get block by number Latest
2023-04-13T13:57:21.499527Z  INFO node::user: eth_sendRawTransaction
2023-04-13T13:57:21.499749Z TRACE node: Added transaction: [0x395b60efd8162a8f82d82f71111113f0aa5ddd1305dad703f44f0e5e49926d74] sender=0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266
2023-04-13T13:57:21.499785Z TRACE backend: creating new block with 1 transactions
2023-04-13T13:57:21.499795Z TRACE backend: [0x395b60efd8162a8f82d82f71111113f0aa5ddd1305dad703f44f0e5e49926d74] executing
2023-04-13T13:57:21.499831Z TRACE backend: [0x395b60efd8162a8f82d82f71111113f0aa5ddd1305dad703f44f0e5e49926d74] executed with out=Create(b"", Some(0xe7f1725e7734ce288f8367e1bb143e90bb3f0512)) exit_reason=Continue gas_used=182809
2023-04-13T13:57:21.499838Z TRACE backend::executor: transacted [0x395b60efd8162a8f82d82f71111113f0aa5ddd1305dad703f44f0e5e49926d74], result: Continue gas 182809
2023-04-13T13:57:21.499842Z TRACE backend: New contract deployed: at Some(0xe7f1725e7734ce288f8367e1bb143e90bb3f0512)
2023-04-13T13:57:21.499954Z TRACE backend: Mined block 2 with 1 tx [0x395b60efd8162a8f82d82f71111113f0aa5ddd1305dad703f44f0e5e49926d74]
2023-04-13T13:57:21.499959Z  INFO node::user:
2023-04-13T13:57:21.499961Z  INFO node::user:     Transaction: 0x395b60efd8162a8f82d82f71111113f0aa5ddd1305dad703f44f0e5e49926d74
2023-04-13T13:57:21.499963Z  INFO node::user:     Contract created: 0xe7f1725e7734ce288f8367e1bb143e90bb3f0512
2023-04-13T13:57:21.499964Z  INFO node::user:     Gas used: 182809
2023-04-13T13:57:21.499966Z  INFO node::user:
2023-04-13T13:57:21.499969Z  INFO node::user:     Block Number: 2
2023-04-13T13:57:21.499971Z  INFO node::user:     Block Hash: 0x39b8c7a3288e61f27f6559b83bcef87bf2a4f06c3f994564f512cd5d1ba25c45
2023-04-13T13:57:21.499973Z  INFO node::user:     Block Time: "Thu, 13 Apr 2023 13:57:22 +0000"

2023-04-13T13:57:21.499977Z TRACE backend::fees: updated base fee 773178362
2023-04-13T13:57:21.499979Z TRACE node: mined block 2
2023-04-13T13:57:21.500730Z  INFO node::user: eth_blockNumber
2023-04-13T13:57:21.501912Z  INFO node::user: eth_getTransactionReceipt
2023-04-13T13:57:21.504345Z  INFO node::user: eth_blockNumber
2023-04-13T13:57:21.522521Z  INFO node::user: eth_getLogs
2023-04-13T13:57:21.522531Z TRACE backend: get logs [Filter { block_option: Range { from_block: Some(Number(0)), to_block: None }, address: Some(Value(0x5fbdb2315678afecb367f032d93f642f64180aa3)), topics: [Some(Value(Some(0x655779015b9b95c7fd18f01ea4619ab4c31289bbe134ba85c5b20bcdeb1dabf3))), None, None, None] }]
2023-04-13T13:57:21.526614Z  INFO node::user: eth_getLogs
2023-04-13T13:57:21.526625Z TRACE backend: get logs [Filter { block_option: Range { from_block: Some(Number(0)), to_block: None }, address: Some(Value(0xe7f1725e7734ce288f8367e1bb143e90bb3f0512)), topics: [Some(Value(Some(0xb72b85ef2a843244f5ea1955248b0ac363732d2e6a98cc3641084dd5718ad8b5))), None, None, None] }]
2023-04-13T13:57:22.859666Z  INFO node::user: eth_getLogs
2023-04-13T13:57:22.859687Z TRACE backend: get logs [Filter { block_option: Range { from_block: Some(Number(0)), to_block: None }, address: Some(Value(0x5fbdb2315678afecb367f032d93f642f64180aa3)), topics: [Some(Value(Some(0x655779015b9b95c7fd18f01ea4619ab4c31289bbe134ba85c5b20bcdeb1dabf3))), None, None, None] }]
2023-04-13T13:57:22.868055Z  INFO node::user: eth_getLogs
2023-04-13T13:57:22.868076Z TRACE backend: get logs [Filter { block_option: Range { from_block: Some(Number(0)), to_block: None }, address: Some(Value(0xe7f1725e7734ce288f8367e1bb143e90bb3f0512)), topics: [Some(Value(Some(0xb72b85ef2a843244f5ea1955248b0ac363732d2e6a98cc3641084dd5718ad8b5))), None, None, None] }]
2023-04-13T13:57:25.085338Z  INFO node::user: eth_getLogs
2023-04-13T13:57:25.085353Z TRACE backend: get logs [Filter { block_option: Range { from_block: Some(Number(0)), to_block: None }, address: Some(Value(0x5fbdb2315678afecb367f032d93f642f64180aa3)), topics: [Some(Value(Some(0x655779015b9b95c7fd18f01ea4619ab4c31289bbe134ba85c5b20bcdeb1dabf3))), None, None, None] }]
2023-04-13T13:57:25.090164Z  INFO node::user: eth_getLogs
2023-04-13T13:57:25.090196Z TRACE backend: get logs [Filter { block_option: Range { from_block: Some(Number(0)), to_block: None }, address: Some(Value(0xe7f1725e7734ce288f8367e1bb143e90bb3f0512)), topics: [Some(Value(Some(0xb72b85ef2a843244f5ea1955248b0ac363732d2e6a98cc3641084dd5718ad8b5))), None, None, None] }]
2023-04-13T13:57:26.979876Z  INFO node::user: eth_getLogs
2023-04-13T13:57:26.979891Z TRACE backend: get logs [Filter { block_option: Range { from_block: Some(Number(0)), to_block: None }, address: Some(Value(0x5fbdb2315678afecb367f032d93f642f64180aa3)), topics: [Some(Value(Some(0x655779015b9b95c7fd18f01ea4619ab4c31289bbe134ba85c5b20bcdeb1dabf3))), None, None, None] }]
2023-04-13T13:57:27.520048Z  INFO node::user: eth_getLogs
2023-04-13T13:57:27.520058Z TRACE backend: get logs [Filter { block_option: Range { from_block: Some(Number(0)), to_block: None }, address: Some(Value(0xe7f1725e7734ce288f8367e1bb143e90bb3f0512)), topics: [Some(Value(Some(0xb72b85ef2a843244f5ea1955248b0ac363732d2e6a98cc3641084dd5718ad8b5))), None, None, None] }]
2023-04-13T13:57:27.699508Z  INFO node::user: eth_estimateGas
2023-04-13T13:57:27.740068Z  INFO node::user: eth_chainId
2023-04-13T13:57:27.740131Z  INFO node::user: eth_getBlockByNumber
2023-04-13T13:57:27.740135Z TRACE backend: get block by number Latest
2023-04-13T13:57:27.740216Z  INFO node::user: eth_getTransactionCount
2023-04-13T13:57:27.740224Z TRACE backend: get nonce for 0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266
2023-04-13T13:57:27.778938Z  INFO node::user: eth_sendRawTransaction
2023-04-13T13:57:27.779189Z TRACE node: Added transaction: [0x1dcbfbcd925a7efe7c3ad34a1b3011a463c84f8386169244c52738991c6264a7] sender=0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266
2023-04-13T13:57:27.779225Z TRACE backend: creating new block with 1 transactions
2023-04-13T13:57:27.779236Z TRACE backend: [0x1dcbfbcd925a7efe7c3ad34a1b3011a463c84f8386169244c52738991c6264a7] executing
2023-04-13T13:57:27.779380Z TRACE backend: [0x1dcbfbcd925a7efe7c3ad34a1b3011a463c84f8386169244c52738991c6264a7] executed with out=Call(b"") exit_reason=Stop gas_used=78956
2023-04-13T13:57:27.779387Z TRACE backend::executor: transacted [0x1dcbfbcd925a7efe7c3ad34a1b3011a463c84f8386169244c52738991c6264a7], result: Stop gas 78956
2023-04-13T13:57:27.779532Z TRACE backend: Mined block 3 with 1 tx [0x1dcbfbcd925a7efe7c3ad34a1b3011a463c84f8386169244c52738991c6264a7]
2023-04-13T13:57:27.779538Z  INFO node::user:
2023-04-13T13:57:27.779540Z  INFO node::user:     Transaction: 0x1dcbfbcd925a7efe7c3ad34a1b3011a463c84f8386169244c52738991c6264a7
2023-04-13T13:57:27.779542Z  INFO node::user:     Gas used: 78956
2023-04-13T13:57:27.779544Z  INFO node::user:
2023-04-13T13:57:27.779547Z  INFO node::user:     Block Number: 3
2023-04-13T13:57:27.779549Z  INFO node::user:     Block Hash: 0xa8513b7b78c4bc03c63d6094d88e5cefb559b831ee27d5e21d993ffbc134299d
2023-04-13T13:57:27.779552Z  INFO node::user:     Block Time: "Thu, 13 Apr 2023 13:57:27 +0000"

2023-04-13T13:57:27.779555Z TRACE backend::fees: updated base fee 677039792
2023-04-13T13:57:27.779695Z TRACE node: mined block 3
2023-04-13T13:57:27.780812Z  INFO node::user: eth_blockNumber
2023-04-13T13:57:27.781436Z  INFO node::user: eth_getTransactionReceipt
2023-04-13T13:57:27.782785Z  INFO node::user: eth_estimateGas
2023-04-13T13:57:27.820614Z  INFO node::user: eth_chainId
2023-04-13T13:57:27.820677Z  INFO node::user: eth_getTransactionCount
2023-04-13T13:57:27.820682Z TRACE backend: get nonce for 0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266
2023-04-13T13:57:27.820717Z  INFO node::user: eth_getBlockByNumber
2023-04-13T13:57:27.820720Z TRACE backend: get block by number Latest
2023-04-13T13:57:27.860024Z  INFO node::user: eth_sendRawTransaction
2023-04-13T13:57:27.860260Z TRACE node: Added transaction: [0xdf071201e2e964b723765547dd35f40bf3ed41da83e4be5e584e2f65fc4779e0] sender=0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266
2023-04-13T13:57:27.860300Z TRACE backend: creating new block with 1 transactions
2023-04-13T13:57:27.860310Z TRACE backend: [0xdf071201e2e964b723765547dd35f40bf3ed41da83e4be5e584e2f65fc4779e0] executing
2023-04-13T13:57:27.860337Z TRACE backend: [0xdf071201e2e964b723765547dd35f40bf3ed41da83e4be5e584e2f65fc4779e0] executed with out=Call(b"") exit_reason=Stop gas_used=32513
2023-04-13T13:57:27.860342Z TRACE backend::executor: transacted [0xdf071201e2e964b723765547dd35f40bf3ed41da83e4be5e584e2f65fc4779e0], result: Stop gas 32513
2023-04-13T13:57:27.860464Z TRACE backend: Mined block 4 with 1 tx [0xdf071201e2e964b723765547dd35f40bf3ed41da83e4be5e584e2f65fc4779e0]
2023-04-13T13:57:27.860469Z  INFO node::user:
2023-04-13T13:57:27.860471Z  INFO node::user:     Transaction: 0xdf071201e2e964b723765547dd35f40bf3ed41da83e4be5e584e2f65fc4779e0
2023-04-13T13:57:27.860473Z  INFO node::user:     Gas used: 32513
2023-04-13T13:57:27.860475Z  INFO node::user:
2023-04-13T13:57:27.860479Z  INFO node::user:     Block Number: 4
2023-04-13T13:57:27.860480Z  INFO node::user:     Block Hash: 0xcc704f871ff6a567e2d378c5a89d91fbde93a2e34f894bb1313491b50d3894d5
2023-04-13T13:57:27.860484Z  INFO node::user:     Block Time: "Thu, 13 Apr 2023 13:57:28 +0000"

2023-04-13T13:57:27.860487Z TRACE backend::fees: updated base fee 592593256
2023-04-13T13:57:27.860490Z TRACE node: mined block 4
2023-04-13T13:57:27.861103Z  INFO node::user: eth_blockNumber
2023-04-13T13:57:27.861661Z  INFO node::user: eth_getTransactionReceipt
2023-04-13T13:57:28.526432Z  INFO node::user: eth_getLogs
2023-04-13T13:57:28.526464Z TRACE backend: get logs [Filter { block_option: Range { from_block: Some(Number(0)), to_block: None }, address: Some(Value(0x5fbdb2315678afecb367f032d93f642f64180aa3)), topics: [Some(Value(Some(0x655779015b9b95c7fd18f01ea4619ab4c31289bbe134ba85c5b20bcdeb1dabf3))), None, None, None] }]
2023-04-13T13:57:28.535263Z  INFO node::user: eth_getLogs
2023-04-13T13:57:28.535285Z TRACE backend: get logs [Filter { block_option: Range { from_block: Some(Number(0)), to_block: None }, address: Some(Value(0xe7f1725e7734ce288f8367e1bb143e90bb3f0512)), topics: [Some(Value(Some(0xb72b85ef2a843244f5ea1955248b0ac363732d2e6a98cc3641084dd5718ad8b5))), None, None, None] }]
2023-04-13T13:57:28.541144Z  INFO node::user: eth_getTransactionByHash
2023-04-13T13:57:28.541160Z TRACE backend: transaction_by_hash=0x1dcbfbcd925a7efe7c3ad34a1b3011a463c84f8386169244c52738991c6264a7
2023-04-13T13:57:29.551986Z  INFO node::user: eth_getLogs
2023-04-13T13:57:29.552007Z TRACE backend: get logs [Filter { block_option: Range { from_block: Some(Number(4)), to_block: None }, address: Some(Value(0x5fbdb2315678afecb367f032d93f642f64180aa3)), topics: [Some(Value(Some(0x655779015b9b95c7fd18f01ea4619ab4c31289bbe134ba85c5b20bcdeb1dabf3))), None, None, None] }]
2023-04-13T13:57:29.555424Z  INFO node::user: eth_getLogs
2023-04-13T13:57:29.555435Z TRACE backend: get logs [Filter { block_option: Range { from_block: Some(Number(5)), to_block: None }, address: Some(Value(0xe7f1725e7734ce288f8367e1bb143e90bb3f0512)), topics: [Some(Value(Some(0xb72b85ef2a843244f5ea1955248b0ac363732d2e6a98cc3641084dd5718ad8b5))), None, None, None] }]

And this last log from tcpflow output:
<IP_ADDRESS>.65506-<IP_ADDRESS>.09000: {"jsonrpc":"2.0","id":13,"method":"eth_getLogs","params":[{"address":"0xe7f1725E7734CE288F8367e1Bb143E90bb3F0512","topics":["0xb72b85ef2a843244f5ea1955248b0ac363732d2e6a98cc3641084dd5718ad8b5"],"fromBlock":"0x5"}]}
<IP_ADDRESS>.09000-<IP_ADDRESS>.65506: HTTP/1.1 200 OK
content-type: application/json
content-length: 1429
access-control-allow-origin: *
vary: origin
vary: access-control-request-method
vary: access-control-request-headers
date: Thu, 13 Apr 2023 13:57:29 GMT

{"jsonrpc":"2.0","id":13,"result":[{"address":"0xe7f1725e7734ce288f8367e1bb143e90bb3f0512","topics":["0xb72b85ef2a843244f5ea1955248b0ac363732d2e6a98cc3641084dd5718ad8b5","0x0000000000000000000000000000000000000000000000000000000000000001","0x000000000000000000000000f39fd6e51aad88f6f4ce6ab8827279cfffb92266"],"data":"0x00000000000000000000000000000000000000000000000000000000000000200000000000000000000000000000000000000000000000000000000000000154000001509a1d0fed0b06532c9b464ae4a7085c2ea9f4b3f086faaf1f4c9f29cad779d6de317825893495fb5c4ff3b3752d8891459305c0a377773ed67a22c5652f2605d268d9be28c88022754f2b40fd6d9b557b34f425ac24ba96a7c74484712e8a9e7694ef02f79aad95046f878cc41410cb03a0405820c8f1630def1e525acdd11a452ec88de44696496c57b9e20c9317562fac7af2e4b050560c59f322e536a92a56581d54d28ab5862c63fe711960115355abee0916382c91b34919732471e08b15856b3bae6a67028a946aed43720128a43e3f5ccda194754e5e4f9f5569e21faee7b8bd749475417281bd6b4e4367646fa82d1f391f7b9ff43a1c5b797e81d929c5b1413eb95a8945cc66a504739f443223be5ad9ff4c031e509143d468092a362faf76fbbde61b49a7e55127f108b48a29941b6fa73243babf8c6078e1ef547acd2bf262e7dcda7393fe8ee649da75fd000000000000000000000000","blockHash":"0xcc704f871ff6a567e2d378c5a89d91fbde93a2e34f894bb1313491b50d3894d5","blockNumber":"0x4","transactionHash":"0xdf071201e2e964b723765547dd35f40bf3ed41da83e4be5e584e2f65fc4779e0","transactionIndex":"0x0","logIndex":"0x0","transactionLogIndex":"0x0","removed":false}]}

2023-04-13T13:57:28.526432Z  INFO node::user: eth_getLogs
2023-04-13T13:57:28.535263Z  INFO node::user: eth_getLogs
2023-04-13T13:57:29.551986Z  INFO node::user: eth_getLogs
2023-04-13T13:57:29.555424Z  INFO node::user: eth_getLogs

looks like there are 3 eth_getLogs calls?

ah if I read the logs correctly then you're requesting logs for [5..]
but current height is:

2023-04-13T13:57:27.860490Z TRACE node: mined block 4

we have this check
https://github.com/foundry-rs/foundry/blob/5a257efcab2fedb5ebe090355164a366b699a77d/anvil/src/eth/backend/mem/mod.rs#L1165-L1165
that resets the lower bound, which is probably wrong?

yeh this is a bug, because would return logs from the best block even if not requested


Create dependabot.yml

unrelated to this commit, but should we consider a dependabot for jekyll-resize?

unrelated to this commit, but should we consider a dependabot for jekyll-resize?

probably.
Dependabot wasn't flagging the npm audit warning for braces.  Though maybe that's just because it will happen later...


Add averaging operator to postprocessing

Eli's next request is to add an operator to postprocessing for averaging data from multiple aimmdb entries. This operator should take an arbitrary number of entries as input and return a new entry with the averaged data, and metadata that points to the original unaveraged data.
@x94carbone I noticed all the currently defined operators use the UnaryOperator class. Is there any abstraction defined for operators that take multiple inputs?

@CharlesC30

I noticed all the currently defined operators use the UnaryOperator class. Is there any abstraction defined for operators that take multiple inputs?

Nope! We need to make one 👍

@x94carbone I came up with this MultiOperator abstraction by modifying the UnaryOperator class. I am still working on the averaging operator itself but let me know if this looks alright so far. The one thing I am wondering is if this should also be able to operate on the Node class.
https://github.com/CharlesC30/aimmdb/blob/3664eaa08f9e876dd1d947e06157a5f2977a63b7/aimmdb/postprocessing/operations.py#L105-L163

@CharlesC30 great, thank you! Can you open a PR for this? We can discuss there. Totally fine if it's still a WIP.

@x94carbone sure! I just created the PR. Not sure why but it included several commits from when I was working on the NormalizeLarch scheme, so just the most recent few commits are related to this.


generic proxy: virtual host support for the generic proxy routing

Commit Message: generic proxy: virtual host support for the generic proxy routing
Additional Description:
In the previous implementation, a generic matching tree is used to as the route table of generic proxy.
However, in the practices of mesh or gw, a layered route table would be useful and necessary in some cases.
Specifically, a utility or abstraction (virtual host for HTTP) that represent a group of related routes is still necessary. Mesh use this abstraction to represent a service and gw use this abstraction to isolate different projects or tenants. Some common attributes could be attached to the abstraction.
To achieve this target, this PR add a virtual host layer for route table of generic proxy.
Risk Level: low. Contrib only update.
Testing: unit/integration.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

Q&A:  Why couldn't we use the generic matching tree to mock the layed route table directly? Although generic matching tree is powerful and flexible, but there are still some shortage make we can't mock the layed route table directly.
First, we cann't attach common attributes of a group of related routes to the tree. For mesh/gw, this will result in that we cannot implement service-level/project-level traffic management.
Second, with the single generic matching tree, we cann't index same utility/abstraction (virtual host) with multiple keys.

Will wait for @soulxu first pass.
/wait-any

@wbpcode re: FAQ, what does "we cann't attach common attributes of a group of related routes to the tree." mean? Should we add the ability to attach metadata to subtrees? @snowp

@wbpcode re: FAQ, what does "we cann't attach common attributes of a group of related routes to the tree." mean? Should we add the ability to attach metadata to subtrees? @snowp

Yeah, metadata, per_filter_config, etc. are absent in the subtrees. But I think it's ok from the  perspective of generic matcing API. They are unnecessary in many use cases of generic matching API. And if we add the metadata to the subtrees, then it would be very complex. And It will also very complex to iterate all nodes' metadata when we match a leaf node.

OK, up to you folks, but I think the hope is that generic matcher would be able to at some point be an effective substitute for RouteConfiguration for HTTP, so if we can't do these things, I think we're falling short on the generic matcher API side @snowp

OK, up to you folks, but I think the hope is that generic matcher would be able to at some point be an effective substitute for RouteConfiguration for HTTP, so if we can't do these things, I think we're falling short on the generic matcher API side @snowp

I think the root issue is that the generic matching tree is a layered searching struture, but isn't a layered route table. Because all the inner nodes are stateless and just be used for searching. So, from the perspective of a specific scene, there are only one layer in the generic matching tree: the final matched action layer.
But it most of cases, single layer is enough. For the route table, two layer also be good. Not sure should we introduce new complexity to the tree to make the inner nodes be stateful. 🤔

cc @mattklein123 for second review or merge.

main merged and conflicts resolved.

/retest

I think the integration test is flaky. But it not related to this pr, I will look into it and fix it with a separated pr.


Made jake [lint, build, test] work on windows

Use csslint.cmd and jshint.cmd on Windows host.
Sort the list of files before building ripple.js; otherwise
ripple.js files built on Linux and Windows don't compare well.
Added .css files below lib/client/ui to the csslint input.
Used path.sep instead of computing a "slash" variable.
Finally, fixed misspelling of the great Yogi Berra's name!

First pull request bringing in Ripple improvements made in the Intel XDK Device Emulator.
This one just makes it possible to do Ripple development on a Windows host.

Thanks for the updates, Julian! LGTM, apart from one question (which I've put inline).

LGTM, as well (sorry for late comment), aside from the one thought. :-)

I'd much rather use exec() instead of spawn() to fix the issue. exec() launches a shell process (/bin/sh on Unix and cmd.exe on Windows) which, in turn, executes the command. spawn() creates a process directly bypassing shell (this is why it needs a concise executable name). so instead of doing shell's job of executable lookup in our script, we could simply implement _spawn() (e.g. _spawn()) via exec(). The code will not have all these platform-specific ifs.

@jchorn - if you're happy with the suggestion by @busykai (he has a later PR here with his suggested changes), could you close this PR, or alternatively pull out the bits that are no longer needed. Thanks!

Hey @jchorn - are there still pieces of this that we should take? If so, could you update it as needed. If not, could you close the PR? Thanks!


 I'd like to pick this up. If that's ok.
It looks pretty straight forward, but it would be my first PR to this project, or any open source project. Also will allow me to start to get familiar with the code base and grow from there.

@Xavier-J-Ortiz , ok. And I will write some tests.
Don`t forget to read #20481.

@shvabuk - I'm going to let this one go. I checked the generator library, I think that there's some F# stuff there that is over my head.
I'll let someone else work on this. Sorry about that. :(

According to searchDuplicates script results https://gist.github.com/shvabuk/ad6b9974794dc7595c5832722df443fa more duplicates was found

Duplication in file: lib/lib.dom.d.ts on lines: 776 & 14373 (ProgressEventInit)
Duplication in file: lib/lib.dom.d.ts on lines: 3735 & 14684 (EXT_frag_depth)
Duplication in file: lib/lib.d.ts on lines: 4874 & 18471 (ProgressEventInit)
Duplication in file: lib/lib.d.ts on lines: 7833 & 18782 (EXT_frag_depth)
Duplication in file: lib/lib.es2016.full.d.ts on lines: 779 & 14376 (ProgressEventInit)
Duplication in file: lib/lib.es2016.full.d.ts on lines: 3738 & 14687 (EXT_frag_depth)
Duplication in file: lib/lib.es2017.full.d.ts on lines: 784 & 14381 (ProgressEventInit)
Duplication in file: lib/lib.es2017.full.d.ts on lines: 3743 & 14692 (EXT_frag_depth)
Duplication in file: lib/lib.es2018.full.d.ts on lines: 778 & 14375 (ProgressEventInit)
Duplication in file: lib/lib.es2018.full.d.ts on lines: 3737 & 14686 (EXT_frag_depth)
Duplication in file: lib/lib.webworker.d.ts on lines: 1882 & 1900 (dispatchEvent)
Duplication in file: lib/lib.esnext.full.d.ts on lines: 780 & 14377 (ProgressEventInit)
Duplication in file: lib/lib.esnext.full.d.ts on lines: 3739 & 14688 (EXT_frag_depth)
Duplication in file: lib/lib.es6.d.ts on lines: 6585 & 20182 (ProgressEventInit)
Duplication in file: lib/lib.es6.d.ts on lines: 9544 & 20493 (EXT_frag_depth)

@RyanCavanaugh
Looks like this issue may be closed and appropriate issue should be opened in https://github.com/Microsoft/TSJS-lib-generator .
May you give advise? May I add searchDuplicates script to tests? If yes, than in wich directory? It looks like generator (not compiler) issue.

I will have free time to work on this issue (or the new one that @shvabuk may create) starting from Monday, 18th. But, if someone is willing to work on this before, please don't hesitate 😃

I will be working on this issue.
I will spend some time first familiarising myself with F#


KafkaCluster validation fails when custom webhook port is provided

Describe the bug
KafkaCluster webhook is not accessible when custom webhook port is provided.
The validations fails because the webhook is not reachable.
Steps to reproduce the issue:
Set the webhook.serverPort in the chart's values yaml.
webhook:
  enabled: true
  serverPort: 8444
#  tls:
#    certDir: ""
  certs:
    generate: true
    secret: "kafka-operator-serving-cert"

Deploy the operator with helm.
helm install kafka charts/kafka-operator

Edit attempt to an existing KafkaCluster resource fails, because the webhook is unreachable.
k edit kafkacluster kafka -n kafka
error: kafkaclusters.kafka.banzaicloud.io "kafka" could not be patched: Internal error occurred: failed calling webhook "kafkaclusters.kafka.banzaicloud.io": Post "[https://koperator-kafka-operator-operator.kafka.svc:443/validate-kafka-banzaicloud-io-v1beta1-kafkacluster?timeout=10s](https://koperator-kafka-operator-operator.kafka.svc/validate-kafka-banzaicloud-io-v1beta1-kafkacluster?timeout=10s)": dial tcp <IP_ADDRESS>:443: connect: connection refused
You can run `kubectl replace -f /var/folders/8l/6jws9zmx1ng710v4v252b9jm0000gr/T/kubectl-edit-664393602.yaml` to try this update again.

Expected behavior
The webhook shall be reachable when custom webhook port is provided.
Additional context
version: 0.22

Resolved by #896, thanks.


collect images and content for the website

Downloaded the resources provided in the semester project brief, wich include text and images we can use in our design.


Implement audit fixes

This PR implements all of the audit fixes in 1 PR. This simplifies merging since each merge requires updating the respective gas arguments.
resolves #39, #38, #37, #36, #35, #34, #33, #32

Revert error NotEnoughGasExeuction is misspelt.

Which line?


Fix -Wmissing-braces warning under clang (for SFML 2.6)

Thanks a lot for making a contribution to SFML! 🙂
Before you create the pull request, we ask you to check the follow boxes. (For small changes not everything needs to ticked, but the more the better!)

[ ] Has this change been discussed on the forum or in an issue before?
[x] Does the code follow the SFML Code Style Guide?
[x] Have you provided some example/test code for your changes?
[ ] If you have additional steps which need to be performed list them as tasks!

Description
Cherry-picked this commit because SFML 2.6 had the same problem with building with Clang..
This PR is related to the issue #2046 and PR #1914.
Tasks
This PR only effects Windows.

[ ] Tested on Linux
[x] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

How to test this PR?
Please check the original commit and the #2046 issue.

SFMLBuildMaster: Build this please.

Thanks for this fix! 🙂