Test portback of fix #15637

Signed-off-by: Fodor Zoltan<EMAIL_ADDRESS>
Attempts portback for #15536 . Cherry picked from #15637
Context
Need this in 6 as 7 is too far still. This is my first PR to gradle, also I am quite new to gradle, and I am not a java developer :D.
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[x] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass sanity check: ./gradlew sanityCheck
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

Initial purpose here is to check if tests pass. I don't see how that can be triggered, or whether there is pipeline for such.

OK, I've already triggered ReadyForNightly build for you.

@archfz I've updated the configuration for release6x pipeline, you should be able to access it now. The failures are not your fault, but some stale configurations on the branch. I've fixed it and merged it into your branch.

@bot-gradle test RFN

Re

Thanks for the PR, we are going to do a 6.9.2 but this looks like quite a large changeset and I am not sure it is worth the risks.
Can you rebase it on the latest release6x so we can run tests in an up-to-date context?
In parallel, I am looking at the code and asking others internally to assess whether we take this contribution and risk or not.

Updated from upstream. Sounds good if it could get included.
This is important, see in https://github.com/gradle/gradle/issues/15536 all the attempts to have hotfix, none of those work properly for our case.

@bot-gradle test RFN

Sorry but I'm afraid you're not allowed to do this.

@bot-gradle test this

Sorry some internal error occurs, please contact the administrator @blindpirate

@bot-gradle test this

Sorry some internal error occurs, please contact the administrator @blindpirate

This branch needs some recent changes on release6x to work. @bot-gradle test this

Sorry some internal error occurs, please contact the administrator @blindpirate

@bot-gradle test this

Do I need to help?

Sorry some internal error occurs, please contact the administrator @blindpirate

@archfz No there's nothing your fault, release6x is an old branch and we have some problems triggering community builds. I'm taking care of it.

@bot-gradle test this

Sorry some internal error occurs, please contact the administrator @blindpirate

@bot-gradle test this

Sorry some internal error occurs, please contact the administrator @blindpirate

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@archfz Looks like in order to get a build, your PR needs to be rebased again the latest release6x. Can you please do that when you have a chance?

@ljacomet Updated.

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

No milestone for this PR. Please set a milestone and retry.

@bot-gradle test and merge

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

OK, I've already triggered a build for you.


AttributeError: 'DummyMapping' object has no attribute 'w_avg'

When I try to generate_video I get the Following error:
AttributeError: 'DummyMapping' object has no attribute 'w_avg'

Duplicate of:

#56

@woctezuma apologies, but I am not sure how to resolve this issue. Thanks!

can you post the command you're using ?


Preprocessor-free implementation of single binary for sp/dp versions

This draft PR aims at demonstration of the possible technical solution to the challenge of selecting precision of computations at runtime.
Main code modifications are:

Renaming of JPRB to a desired precision at the library creation time (thus exploiting exisiting cmake ${prec} definition}  via PRIVATE_DEFINITIONS. Expecting removal of dependence on PARKIND1 module, it is renamed to EC_PARKIND the same way.

Creation of  stub ectrans module that imports actual routines from trans/external as submodules. aiming at modern Fortran importing. The ectrans module relies on the existing interface headers of trans/include/ectrans, slightly preprocessed to offer "MODULE PROCEDURE" if needed.

The benchmark program is augmented with a set of wrappers to setup_trans(0), dir_trans, inv_trans etc. that are imported by a a benchmark program instead of their legacy trans/external counterparts. The latter (apart from specnorm) are not even compiled directly, but only when embedded into ectrans_mod submodules. The wrappers are equipped with an extra argument that is being resolved at the wrapping level, so the original trans external routines are not modified in this respect. At the moment, the wrappers sit in the program directory to minimise mixing with the legacy trans code.

Minor technical adaptations, namely introducing JPRC to serve in assignments like a=1.0_JPRC were introduced to satisfy the needs of the NVHPC preprocessor, which apparently can't identify macro names being a part of such statement(s). On the occasion, JPRB was removed when imported but not used in several files.

Code specialisation is realised by cmake function generate_trans_build that:
a) creates directory structure to host processed files mimicking the source tree
b) executes define_code_spec cmake function that performs sed edit on the source files using sed script include/ectrans/sedrenames.txt that annotates module names with the description of a desired variant, eg. _dp or _sp.
c) creates modified source files list for add_library by prepending the name with path to the specialised source code tree in CMAKE_BINARY_DIRECTORY
d) propagates modified source file list name to parent scope

Note that the sed script itself is edited to specify the designated variant (eg. _sp or _dp).

The source code was sorted using extra subdirectories to separate fully precision-independent modules that do not need specialisation. They could potentially be cast in a separate "common" module.

The auxiliary construction that provides transparency of the new solution to the legacy code seems to me a bit complicated. For a given trans/external file "a", we have programs/wrappers/a.F90, a.inc; src/trans/modules/a_mod.F90 in addition to existing trans/external/a and trans/include/ectrans/a.h .This means that five files need to be synchronized and interface is actually a duplicated code. Debugging is not perhaps very hard, but still could be a bit confusing. If this solution is here to stay, perhaps a small  file generator could be helpful to keep the files in sync.


[WIP][RFC]Device control features

Reviving #24511 (See api channel in discord)
See #22941 as well for more background information.
This is a WORK IN PROGRESS
Current status:

it breaks I²C, Ethernet, etc... statistics. I know, I'll come with a solution within this PR (on the first commit since it is the one breaking it)
it is not using a k_mutex but a k_sem: besides the fact that k_mutex are not isr proof (which is a problem in itself), this has been already discussed here #8496 and looks like priority inheritance is not such a good idea.
Not all device will need such mechanism: I'll make the lock (and any control structures in the context) optional in order to save so ram. In general I'll try to expose as much flexibility as possible.

More features to come

Something's off: I get 0 device_context in the .map file in devcontext's section, though __used is set to all of them, same for the locks. Need to investigate this.

Last update: finally got a clean way to make the device lock optional per-device (thanks @mbolivar-nordic for the proposal!). Introducing a generic device property "zephyr,no-lock" which permits to disable the lock allocation on an instance basis. Current drawback: not all device instance in the system are DTS based, so these ones by default do not get any lock anyway.

After running some numbers, I feel this is not going to be the best solution for limited RAM targets.

it unilaterally grows the device_context structure for all devices: lock, sync, else... pointers are always going to be present for all, even if a fraction only are going to find it useful
many devices do no need this: all chose exposing an API which resolves to atomic operation (interrupt controllers, many gpios etc...), thus where colliding operation are unlikely to happen
I could not get a clean way of reinstating the can/i²c stats specific object without - again - growing the device_context, loosing then the ROM gain I made by removing the device_state pointer from device...

Anyway,I am addressing that from a different stand point on devices, keeping the improvements while solving the above issues.


[Bug] Bincode deserialization of tuples containing enums is broken in c++

🐛 Bug

To reproduce
Code snippet to reproduce

#[derive(Clone, Debug, Eq, PartialEq, PartialOrd, Hash, Serialize, Deserialize)]
pub enum Key {
    Id,
    Custom(String),
}

#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
pub enum Value {
    NotSet,
    Integer(i32),
    String(String),
}

#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
pub struct MyStruct {
    pub a: (Key, Value),
    pub o: Key,
}

fn make_attribute() -> MyStruct {
    MyStruct {
        a: (Key::Custom("abc".into()), Value::String("bar".into())),
        o: Key::Id,
    }
}

#[no_mangle]
pub unsafe extern "C" fn test_in(data: *const u8, len: usize) {
    if !data.is_null() {
        let slice = std::slice::from_raw_parts(data, len);
        let data = bincode::deserialize::<MyStruct>(slice).unwrap();
        assert_eq!(data, make_attribute());
    }
}

#[no_mangle]
pub unsafe extern "C" fn test_out(data: *mut *mut u8) -> usize {
    let p = make_attribute();

    let binary = bincode::serialize(&p).unwrap();
    let b = binary.into_boxed_slice();
    let len = b.len();
    let out: &mut *mut u8 = &mut *data;
    *out = Box::into_raw(b) as *mut u8;
    len
}

#include <iostream>
#include "include/testing.hpp"

using namespace std;

extern "C" size_t test_out(unsigned char** out_ptr);
extern "C" void test_in(const unsigned char* data, size_t len);

int main(int argc, char *argv[]) {

    uint8_t* out_ptr1 = nullptr;
    size_t len1 = test_out(&out_ptr1);
    test_in(out_ptr1, len1);
    std::cout << "Just in and out works fine" << std::endl;

    uint8_t* out_ptr = nullptr;
    size_t len = test_out(&out_ptr);

    std::vector<uint8_t> serialized_result(out_ptr, out_ptr + len);
    auto res = testing::MyStruct::bincodeDeserialize(serialized_result);

    auto serializer = serde::BincodeSerializer();
    serde::Serializable<testing::MyStruct>::serialize(res, serializer);
    std::vector<uint8_t> bytes = std::move(serializer).bytes();

    test_in(bytes.data(), bytes.size());

    std::cout << "Everything is fine" << std::endl;

    return 0;
}

(I'm aware that this leaks memory and unwinds through the ffi boundary.)
Stack trace/error message
terminate called after throwing an instance of 'serde::deserialization_error'
  what():  Unknown variant index for enum

See here for an example project reproducing the error. Assuming a linux environment with g++ installed you only need to clone that repository and execute make to run all required steps to see this error message
Expected Behavior
I would expect that the c++ code executes without throwing an exception.  That would imply that it prints the following messages:
Just in and out works fine
Everything is fine

System information
Please complete the following information:
serde-reflection: 0.3.6
serde-generate: 0.25.0
rustc: rustc 1.67.0 (fc594f156 2023-01-24)
Additional context
Slightly changing the layout of the MyStruct type and/or the Value enum changes the generated error message. For example removing this line removes the c++ exception but causes this assert to fail with the following message:
thread '<unnamed>' panicked at 'assertion failed: `(left == right)`
  left: `MyStruct { a: (Custom("bar"), String("abc")), o: Id }`,
 right: `MyStruct { a: (Custom("abc"), String("bar")), o: Id }`', src/lib.rs:35:9

Which implies that there is something wrong with how enum variants are handled inside of a tuple.

@ma2bd Any ideas how to fix that?

Oh I just saw this. Let me have a look

Thanks @weiznich for the high quality bug report. I cloned your project and running make actually works fine on my Macbook, with g++ = "Apple clang version 13.1.6".
I'm a bit puzzled. Let me try to add a unit test for the Linux CI.

oops didn't mean to close the issue just yet

Once we have a clean repro, I suspect the issue is related to https://stackoverflow.com/questions/43959229/does-stdapply-provide-a-guarantee-for-order-of-evaluation

FWIW using g++-12 from homebrew's gcc also works on my laptop.

Given the fact that it seems to switch key/value with the following patch to the test case:
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -10,20 +10,20 @@ pub enum Key {
 #[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
 pub enum Value {
     NotSet,
-    Integer(i32),
+//    Integer(i32),
     String(String),
 }

 #[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
 pub struct MyStruct {
     pub a: (Key, Value),
-    pub o: Key,
+    //pub o: Key,
 }

 fn make_attribute() -> MyStruct {
     MyStruct {
         a: (Key::Custom("abc".into()), Value::String("bar".into())),
-        o: Key::Id,
+        //o: Key::Id,
     }
 }

results in
thread '<unnamed>' panicked at 'assertion failed: `(left == right)`
  left: `MyStruct { a: (Custom("bar"), String("abc")) }`,
 right: `MyStruct { a: (Custom("abc"), String("bar")) }`', src/lib.rs:35:9

I guess that the issue might be here: https://github.com/zefchain/serde-reflection/blob/7facc0412d060a7cf20d1da17d75b014b1218fab/serde-generate/runtime/cpp/serde.hpp#L613 ?

@ma2bd I've opened #32 to fix this.

@weiznich Thanks for the fix! I suspected an issue at this line but couldn't find the right explanation.

Released in serde-generate v0.25.1


Fixed warnings

Which warnings did this fix? All of the hidden overloads?

On Oct 15, 2015, at 5:10 PM, Vinzenz Bildstein<EMAIL_ADDRESS>wrote:
Merged #521 https://github.com/GRIFFINCollaboration/GRSISort/pull/521.
—
Reply to this email directly or view it on GitHub https://github.com/GRIFFINCollaboration/GRSISort/pull/521#event-436890571.

I see that it wasn't so I will continue working

No, I just fixed the warnings on the linux side (gcc 4.9.0), you'll have to fix the MacOs warnings.


Fix GHA invoking cargo audit

We recently got a major version update of the audit-check GHA.
This version invokes:
cargo audit --json --file ./Cargo.lock

Which fails, because we don't have a Cargo.lock file.
This commit makes sure the Cargo.lock is present prior to
invoking cargo audit.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 29.02%. Comparing base (57570dd) to head (76872a1).
Report is 57 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #237      +/-   ##
==========================================
+ Coverage   26.61%   29.02%   +2.41%
==========================================
  Files          24       26       +2
  Lines        2788     3159     +371
  Branches     1516     1671     +155
==========================================
+ Hits          742      917     +175
- Misses        948     1021      +73
- Partials     1098     1221     +123


jshint incorrectly supports the module x from 'y' format?

I noticed this is allowed...
module x from 'y';

Is that right? I haven't seen it in es6 or am I wrong? I'm not a spec expert.
should it be removed from jshint or put into mozilla javascript extensions, which has other non es6 stuff in like the strange let statement.

It was previously defined at http://wiki.ecmascript.org/doku.php?id=harmony:modules. That page's content is now defined at https://people.mozilla.org/~jorendorff/es6-draft.html#sec-scripts-and-modules but ModuleDeclaration is not specified, so it is deprecated.
(Also both babel and traceur don't support it)

Yep. Either if you care to send a patch getting rid of it?

I can do it,  but if someone is enthusiastic, shout and i will leave you to
it,  my work fixing used label bugs is showing no sign of finishing.

I wouldn't mind tackling this.
I'm pretty sure this one block needs to be altered through throw a warning or be removed all together. What's the expected behavior?

yes, thats the code i came across. If I were you I'd try taking it out and making sure you get a ok error message in a test if the old module format is used - but I'm not a maintainer, @caitp etc might have more specific requirements.

i'd rather just remove it from the parser

Okay I'm authoring a pull request that removes it completely from the parser and also removes the associated test. Let me know if I'm missing anything!


Spring Boot 2 compatibility

help
feign.codec.EncodeException: Could not write request: no suitable HttpMessageConverter found for request type [org.springframework.web.multipart.support.StandardMultipartHttpServletRequest$StandardMultipartFile] and content type [multipart/form-data]

need to be annotated with @Configuration...


HBASE-26300 Skip archived master wals during incremental backups

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 56s
Docker mode activated.


⚠ Bofrost.fr has degraded performance

In b6b9f59, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1115 ms

Resolved: Bofrost.fr performance has improved in a5a8a50.


Request: Add Elemental Spell as an option for Loremaster prerequisites

Tabletop specifies "any 3 metamagic or item creation feats" so it seems like an oversight that Elemental Spell shouldn't qualify for Loremaster?
Cheers!

It is because of how it was setup as a selection, I will see if I can adjust to make this possible


Getting $handle not found error

Handcash 2.0 no longer supports the old api.  All handcash handles resolve to<EMAIL_ADDRESS>A quick fix:
 //HandCash

async function callHandCash(id) {

  id = id.toLowerCase();

  id = id.replace('$','');

  id =<EMAIL_ADDRESS>
  return axios.get('https://api.polynym.io/getAddress/' + id);

}

you will also have to return x.data.address from callHandCash(id) rather than x.data.receivingAddress`

Cheers dude. I've patched this on server, npm and github :)


Make the editor's 'CheckButton' icon be smaller

CheckButtons are very tall UI components due to their icon's size, this makes so that they stretch any other Control that happens to be together with them. This PR shrinks the icon for the editor a little, so they now appear less out-of-place.
Before:

After:

Thanks!

Cherry-picked for 3.2.3.


Log in functionality via Wikidata

As a user of a WikidataComplete Web UI, I want to log in using my Wikimedia account.
Remark: An OAuth2 mechanism would be preferred.

@lalit97 I suggest you ask in the Wikimedia chat/forum about how to do such an login mechanism.

Thanks for the suggestion @anbo-de will reach out to the community :)

meeting notes: we need to implement Oauth via Wikidata instead of MediaWiki

This is now deployed on prod. Can be tested by visting https://datacompletewiki.toolforge.org/accounts/login for now.
Closing with #58


Add paths config for Intellisense

Issue
The TypeScript engine config doesn't list the webpack mappings for the common modules (utils etc.). The config should be extended to allow the usage of e.g. Go To Definition with those imports.

Resolved in PR #273 and #284


Remove dataset

Currently, Roddy requires that there is a patient ID or dataset ID is located as directory name in the inputBaseDirectory. This occasionally comes up a restriction on the interface.
Remove the dataset dependency from the Roddy core.

This may become important as soon as OTP starts using UUIDs for input and output directories. A fix should be accompanied by communicating workflow configuration/metadata via JSON or YAML.


Could Not Load Wallet

Hi, I have done MBC Mining and earned approx. 20,000 MBCs
I am using https://microbitcoinorg.github.io/wallet/#/ to check my balances
But after few days, When I check my current balance then it shows 0 MBC
And I am getting a warning message: "We couldn't load wallet balance!".
Does anyone knows about this problem?
Had I lost my MBCs or it is due to Server problem at the moment when i checked?
Please suggest

I have a same probs here, my wallets always 0

Make sure you using correct address type.


The tutorial about the forgae shape command

@Pegahbit whats your oppinion?

Sorry guys for the delay! I just saw it now! Looks very good to me as well! IMO would be nice to have always one complete and working source code example at the end :) What do you think?

Sorry guys for the delay! I just saw it now! Looks very good to me as well! IMO would be nice to have always one complete and working source code example at the end :) What do you think?

@Pegahbit I have added a reference link to the aepp source code.

following the tutorial doesn't work for me

@thepiwo, @Pegahbit I have updated the tutorial, but we should wait until the new forgae release, because I have made changes to the local network config file.

@Pegahbit or @maptuhec can you review?


Register - Birthdate datepicker

The birthdate datepicker does not allow selecting the year/month.

Docs:

https://nextui.org/docs/components/date-picker
https://ui.shadcn.com/docs/components/calendar
https://daypicker.dev/

This may be useful - https://www.youtube.com/watch?v=R_7FX2gGiSY


Need to provide container version 1.7.9 / 1.6.25

What would you like to be added:
Need to provide EKS Optimized AMI which contains containerd version 1.7.9 or 1.6.25
Why is this needed:
As you can see https://github.com/containerd/containerd/issues/9309 and https://github.com/containerd/containerd/issues/7496,
Under high host disk IO pressure, containerd shim leak occurs.
This issue finally fixed in containerd version 1.7.9 or 1.6.25.
The latest containerd version in AL2 repository (amzn2extra-docker) is still 1.7.2.

duplicate of https://github.com/awslabs/amazon-eks-ami/issues/1526


[Bug]: Error uploading Parquet file larger than 7MB: s3 error: missing ETag

What happened?
I have encountered an issue while trying to upload Parquet files larger than ~7MB or over a certain duration using the LakeFS UI. The issue seems to arise particularly when uploading a file larger than 10MB from a specific source. When I attempt the upload, I receive an error message in the UI, and no logs are generated for further insight.
Steps to Reproduce:

Run docker-compose up to start LakeFS.
Create a new repository.
Attempt to upload a file larger than 10MB from the following source: https://beta.source.coop/cholmes/overture/places-geoparquet-country

The UI displays the error message: s3 error: missing ETag, and no logs are generated for debugging.

Additional Information:

I am using contabo object storage and enabled all CORS and ETag …etc. with AWS CLI
I suspect the issue may be related to either the file size or the duration of the upload, but I am not entirely certain about the exact cause.
There is no additional log information available to provide more context on this error.
The issue persists regardless of the browser used.

the docker-compose.yml :
version: '3.8'  # updated to version 3.4 to use healthcheck condition in depends_on
services:
  lakefs:
    image: yharby/lakefs:latest
    container_name: lakefs
    ports:
      - "8000:8000"
    environment:
      - LAKEFS_DATABASE_TYPE=postgres
      - LAKEFS_DATABASE_POSTGRES_CONNECTION_STRING=postgres://secret:secret@postgres:5432/secret
      - LAKEFS_AUTH_ENCRYPT_SECRET_KEY=ca7a41c47b06bf3762af3df69e2a98d5847a6e22c7b85d6f17e5801e3397040e9f6306ed8c10b5ccd02723caa0b2fe67071fbb9b86c4702b3a872ab9444b1806
      - LAKEFS_BLOCKSTORE_TYPE=s3
      - LAKEFS_BLOCKSTORE_S3_FORCE_PATH_STYLE=true
      - LAKEFS_BLOCKSTORE_S3_ENDPOINT=https://eu2.contabostorage.com
      - LAKEFS_BLOCKSTORE_S3_DISCOVER_BUCKET_REGION=false
      - LAKEFS_BLOCKSTORE_S3_CREDENTIALS_ACCESS_KEY_ID=secret
      - LAKEFS_BLOCKSTORE_S3_CREDENTIALS_SECRET_ACCESS_KEY=secret
      - LAKEFS_GRAVELER_REPOSIORY_CACHE_TTL=1h
      - LAKEFS_GRAVELER_REPOSIORY_CACHE_SIZE=9999999999
      - LAKEFS_AUTH_CACHE_JITTER=1h
      - LAKEFS_AUTH_REMOTE_AUTHENTICATOR_REQUEST_TIMEOUT=20m
      - LAKEFS_STATS_ENABLED
      - LAKEFS_AUTH_CACHE_TTL=20m
      - LAKEFS_GRAVELER_COMMIT_CACHE_JITTER=1h
      - LAKEFS_GRAVELER_COMMIT_CACHE_TTL=1h
    depends_on:
      postgres:
        condition: service_healthy  # depends on postgres being healthy
    command: run

  postgres:
    image: kartoza/postgis:15-3.4
    container_name: postgres
    environment:
      - POSTGRES_USER=secret
      - POSTGRES_PASSWORD=secret
      - POSTGRES_DB=secret
    ports:
      - "5432:5432"
    volumes:
      - postgres-data:/var/lib/postgresql
    healthcheck:  # healthcheck directive
      test: "PGPASSWORD=secret pg_isready -h <IP_ADDRESS> -U secret -d secret"
      interval: 15s
      timeout: 5s
      retries: 3

volumes:
  postgres-data:

Expected behavior
The file should be uploaded successfully to the newly created repository.
lakeFS version
v0.112.1 / latest
How lakeFS is installed
Docker
Affected clients
No response
Relevant log output
No response
Contact details
<EMAIL_ADDRESS>

@Youssef-Harby, S3 compatible storages sometime exhibit different behaviors and unfortunately I don't think we have much experience with contabo.
To understand the issue better - please provide the following:
1.Please set the following env var to enable debug logging: LAKEFS_LOGGING_LEVEL=debug
2. How did you setup CORS? Please provide the configuration
3. Have you tried uploading a different file of that size? Did you see the same behavior?
4. Please attach the lakeFS logs during the upload attempt

@Youssef-Harby closing this issue due to inactivity. Please reopen and provide requested information if this is still relevant


test(blockchain): add missing paths test

Should bring coverage back up to 100%
What kind of change does this PR introduce?

[x] Tests

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Does this PR release a new version?

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the develop branch, not the master branch
[x] All tests are passing
[x] New/updated tests are included

Codecov Report

Merging #106 into develop will increase coverage by 2.4%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop   #106     +/-   ##
========================================
+ Coverage     97.6%   100%   +2.4%
========================================
  Files           15     15
  Lines          250    250
========================================
+ Hits           244    250      +6
+ Misses           6      0      -6

Impacted Files
Coverage Δ

src/api/paths.cpp
100% <0%> (+3.72%)
:arrow_up:

Continue to review full report at Codecov.

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


Use footer landing page

dependent on https://github.com/acteng/govuk-svelte/pull/13

Demo link https://acteng.github.io/atip/use-footer-landing-page/
Will review more later, but one more request please: let's move the About to a separate page like privacy/accessibility. It doesn't make sense as a modal anymore.


LoadData - MariaDB - does not work with BLOB column type

Environment
Liquibase Version: 4.7.0
Liquibase Integration & Version: spring boot 2.7.0-M1, but is reproducible by cli
Liquibase Extension(s) & Version:
Database Vendor & Version: 10.5.11-MariaDB
Operating System Type & Version: does not matter
Description
When we use loadData and there exists a BLOB column, then no data are populated in the DB at all. Even no exception is thrown, but in databasechangelog table the changeset is set as run.
Steps To Reproduce
lets have test.changelog.yaml:
databaseChangeLog:
  - changeSet:
      id: "create-table"
      author: test
      changes:
        - createTable:
            tableName: person
            columns:
              - column:
                  autoIncrement: true
                  constraints:
                    nullable: false
                    primaryKey: true
                  name: sys_pk
                  type: INT
              - column:
                  constraints:
                    nullable: false
                    unique: true
                  name: sys_id
                  type: BINARY(16)
              - column:
                  name: email
                  type: VARCHAR(320)
  - changeSet:
      id: "populate-data"
      author: test
      changes:
        - loadData:
            #            usePreparedStatements: false # cannot use this, because BLOB has needsPreparedStatement = true (see LoadDataChange::generateStatements)
            commentLineStartsWith: '#'
            encoding: UTF-8
            file: test.csv
            quotchar: '"'
            separator: ','
            tableName: person
            relativeToChangelogFile: true
            columns:
              - column:
                  header: sys_id
                  name: sys_id
                  type: BLOB
              - column:
                  header: email
                  name: email
                  type: STRING

and test.csv file:
sys_id,email
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Values of sys_id contain BASE64 encoded binary UUIDs.
Actual Behavior

changeset populate data is marked as successful in databasechangelog table
no data are pupulated in database
no exception or warning can be found in log

Expected/Desired Behavior

changeset populate data is marked as successful in databasechangelog table
data are pupulated in database

Additional Context

in https://github.com/liquibase/liquibase/issues/1551 there is suggestion to use usePreparedStatements: false but this does not help because BLOB type sets needsPreparedStatement = true (see LoadDataChange::generateStatements)
this issue is probably caused by https://github.com/liquibase/liquibase/pull/2215 - a50e90b4c526521b516e548250c0fdeb415c3d92
the logic in LoadDataChange::generateStatements collects data in preparedStatements list, but in the end the logic returns value based on variable statements (which is empty list), so all the prepared statemenets are "ignored" and nothing gets executed later.

            if (anyPreparedStatements) {
                // If we have only prepared statements and the database supports batching, let's roll
                if (databaseSupportsBatchUpdates && statements.isEmpty() && (!preparedStatements.isEmpty())) {
                    if (database instanceof PostgresDatabase) {
                        // we don't do batch updates for Postgres but we still send as a prepared statement, see LB-744
                        return preparedStatements.toArray(new SqlStatement[preparedStatements.size()]);
                    } else {
                        return new SqlStatement[]{
                                new BatchDmlExecutablePreparedStatement(
                                        database, getCatalogName(), getSchemaName(),
                                        getTableName(), columns,
                                        getChangeSet(), Scope.getCurrentScope().getResourceAccessor(),
                                        preparedStatements)
                        };
                    }
                } else {
                    return statements.toArray(new SqlStatement[statements.size()]); //##### all the statements are in variable "preparedStatements" but here variable "statements is used"
                }
            } else {
             ...

Probably it may be sufficient, to extend the if (database instanceof PostgresDatabase) by the excluded MySQLDatabase from above.

The problem still occurs in 4.9.1 on MariaDB. Statements collected to the variable preparedStatements but in the end the logic returns result from variable statements.

databaseChangeLog:
  - changeSet:
      id: "1"
      author: usu
      changes:
        - createTable:
            tableName: test
            columns:
              - column:
                  constraints:
                    nullable: false
                    primaryKey: true
                  name: sys_pk
                  type: INT
              - column:
                  constraints:
                    nullable: false
                  name: sys_when_created
                  type: DATETIME(6)
              - column:
                  constraints:
                    nullable: false
                    unique: true
                  name: sys_id
                  type: BINARY(16)
  - changeSet:
      id: "2"
      author: usu
      changes:
        - loadData:
            commentLineStartsWith: '#'
            encoding: UTF-8
            file: 0013.csv
            quotchar: '"'
            separator: ','
            tableName: test
            relativeToChangelogFile: true
            columns:
              - column:
                  header: sys_pk
                  name: sys_pk
                  type: NUMERIC
              - column:
                  header: sys_when_created
                  name: sys_when_created
                  type: DATETIME
              - column:
                  header: sys_id
                  name: sys_id
                  type: BLOB

and 0013.csv:
sys_pk,sys_when_created,sys_id
1,NOW,bIgBzXJhdUOy1Y6dSD/RyQ==
2,NOW,Zt4QmCBWNUmIOSW6yw5/1A==


Add windows_service deprecation of configuring run_as properties

Description
Steps towards cleaning up the windows_service resource.
Issues Resolved
N/A
Check List

[x] New functionality includes tests
[x] All tests pass
[ ] RELEASE_NOTES.md has been updated if required (not required for bugfixes, required for API changes)
[x] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

@jasonwbarnett You're going to need to make this deprecation 28 now since we just merged 27. You also have a pile of failures in appveyor and travis.

@jasonwbarnett You're going to need to make this deprecation 28 now since we just merged 27. You also have a pile of failures in appveyor and travis.

I totally winged this and have no idea if it's even right. Which PR was for 27 so I can use it for a reference?

@tas50 ready for 👀

@tas50 ready for review!

So the problem here is that philosophically Chef should never have had verbs at all other than "action :fixit" and it should only have properties which are nouns (more or less -- ignoring flags that determine how something should get fixed like atmoic_update on the file resource).
So, looking at the english verb "start" and trying to "make that make sense" is going to be digging the hole even deeper here.
Ultimately if you have a resource which declares start and declares a run_as user and the run_as user is ignored, then that's what nobody would expect.  If the run_as user is specified on start then you need to leave the actual configured resource on the system with the correct run_as user and to fix it so that it is running (action :start means something like "must be left in a running state with all the correct properties that I've specified").
It sounds like this comes from a situation where the run_as user is incorrectly taking a run_as user of System so that a simple action :start with nothing specified will cause the service to reconfigure to the System user in windows.  That's a bug.  The run_as_user property should be nil.  When the run as user is nil then when trying to reconfigure a system the run_as_user should be ignored.  When trying to create a user the run_as_user should still be nil, and that nil should mean System.  It sounds like from discussion that windows sets up services to run_as_user as System so that could be just left up to the O/S which sounds most correct -- whatever the O/S default is we should probably trust that (unless it is literally insane for some reason).
The bug here is that someone decided to be explicit about the System default in the run_as_user which is wrong.  That is the correct behavior but specifying it on the property and making that the Chef property default is incorrect.  Properties should default to nil whenever possible to just leave it up to the O/S.  Asserting the O/S default as a property default leads to bugs like this one.

No reason to leave this open. Thanks for the information @lamont-granquist


Feature/202210 institutional storage control/fixedbug/35815

Purpose
Remove logic of permission check when exporting and restoring
Changes
No DB changes
QA Notes

Documentation

Side Effects

Ticket

Closing for creating the wrong PR.


Crash in 1.9.4 either on world load or startup

I have two crash reports for you. The first is with Quantum Storage installed. I thought it might have been the culprit but I still crashed with it removed so the second crash report is without Quantum Storage installed. All other mods stayed the same. No crashes since removing Refined Storage.
Report 1:
https://gist.github.com/Dragnier/65eeddc17ffbd4f242c9883e069683f8
Report 2:
https://gist.github.com/Dragnier/01a089e6787f084d6fb15fe61832145e

Some other mod is interfering with the RF api and is possibly having an outdated version of it installed.

To explain this further:
Caused by: java.lang.IllegalAccessError: tried to access field net.minecraft.item.ItemStack.field_77990_d from class cofh.api.energy.ItemEnergyContainer

This means some other mod is straight up trying to access field_77990_d (which is stackTagCompund) without using the getter.
The RF API bundled with RS is doing it correctly, using the getter:
 @Override
    public int getEnergyStored(ItemStack container) {

        if (container.getTagCompound() == null || !container.getTagCompound().hasKey("Energy")) {
            return 0;
        }
        return container.getTagCompound().getInteger("Energy");
    }

Ah
* CoFHAPI (1.7.10R1.0.2) from [1.9.4] MachinePowerCraft-1.1.2.jar

MachinePowerCraft is using the CoFH 1.7.10 API...

Thanks for letting me know.


Can't install via composer

Hi. Can't install 3.12.0 via composer. PHP 7.2, Git <IP_ADDRESS>, Composer 1.6.5, PHPStorm 2017.2.4. Composer hanging up for some minutes and then:

C:\ProgramData\ComposerSetup\bin\composer.bat update --no-interaction
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 1 install, 0 updates, 0 removals

Installing joomla-projects/selenium-server-standalone (v3.12.0): Loading from cache
Invalid zip file, retrying...

Installing joomla-projects/selenium-server-standalone (v3.12.0): Loading from cache
Invalid zip file, retrying...

Installing joomla-projects/selenium-server-standalone (v3.12.0): Loading from cache
Failed to download joomla-projects/selenium-server-standalone from dist: 'E:\xampp_php7.2\htdocs\test\vendor/joomla-projects/selenium-server-standalone/415b7dd9c9412aa4760a414ebf7e1b75' is not a zip archive.
Now trying to download from source

Installing joomla-projects/selenium-server-standalone (v3.12.0): Cloning aa98fd550e from cache
aa98fd550e1b3281da3ed72f15a487588670e39c is gone (history was rewritten?)

[RuntimeException]
Failed to execute git checkout "aa98fd550e1b3281da3ed72f15a487588670e39c" -

&& git reset --hard "aa98fd550e1b3281da3ed72f15a487588670e39c" --

fatal: reference is not a tree: aa98fd550e1b3281da3ed72f15a487588670e39c

Composer can't clone. This problem occurs with your repo only. Thx.

the same :(

@wenny611  what OS are you using? I am able to install via composer the latest version without any problems.

Win7. I think it may be because of our idiotic bans of Telegram networks (and many others).

try: composer clear-cache


[DT] Changelog date

if you have any concerns or questions please to Follow or contact with any concerns.


TypeError in version 8.4.1

Updating from version 8.4.0 to 8.4.1 causes an error in our tests. Maybe related to the changes in #362 ?
cc @ArnaudRinquin

Yes, it's likely related. JSDom doesn't implement document.fonts - that alone I don't classify as a problem. The runtime code of the library doesn't have to support JSDom quirks. However, the README currently states that:

This module supports IE9 and above.

And document.fonts is not available in IE. So I would accept a PR fixing this.

+1
I have a same issue.

Sorry folks, only noticed this issue now. Proposed something that should fix this.


[PkgEval] DataStructures may have a testing issue on Julia 0.4 (2014-10-28)

PackageEvaluator.jl is a script that runs nightly. It attempts to load all Julia packages and run their tests (if available) on both the stable version of Julia (0.3) and the nightly build of the unstable version (0.4). The results of this script are used to generate a package listing enhanced with testing results.
On Julia 0.4

On 2014-10-27 the testing status was Tests pass.
On 2014-10-28 the testing status changed to Tests fail, but package loads.

Tests pass. means that PackageEvaluator found the tests for your package, executed them, and they all passed.
Tests fail, but package loads. means that PackageEvaluator found the tests for your package, executed them, and they didn't pass. However, trying to load your package with using worked.
This issue was filed because your testing status became worse. No additional issues will be filed if your package remains in this state, and no issue will be filed if it improves. If you'd like to opt-out of these status-change messages, reply to this message saying you'd like to and @IainNZ will add an exception. If you'd like to discuss PackageEvaluator.jl please file an issue at the repository. For example, your package may be untestable on the test machine due to a dependency - an exception can be added.

ERROR: test error in expression: sort(collect(keys(d))) == ['a':'z']
`isless` has no method matching isless(::Char, ::Char)
 in sort! at sort.jl:247
 in sort! at sort.jl:291
 in sort! at sort.jl:296
 in sort! at sort.jl:332
 in sort at sort.jl:337
 in anonymous at test.jl:83
 in do_test at test.jl:47
 in include at ./boot.jl:242
 in include_from_node1 at ./loading.jl:128
 in anonymous at no file:17
 in include at ./boot.jl:242
 in include_from_node1 at loading.jl:128
 in process_options at ./client.jl:293
 in _start at ./client.jl:362
 in _start_3B_3769 at /home/idunning/julia04/usr/bin/../lib/julia/sys.so
while loading /home/idunning/pkgtest/.julia/v0.4/DataStructures/test/test_defaultdict.jl, in expression starting on line 45
while loading /home/idunning/pkgtest/.julia/v0.4/DataStructures/test/runtests.jl, in expression starting on line 14

INFO: Testing DataStructures
===========================[ ERROR: DataStructures ]============================

failed process: Process(`/home/idunning/julia04/usr/bin/julia /home/idunning/pkgtest/.julia/v0.4/DataStructures/test/runtests.jl`, ProcessExited(1)) [1]

================================================================================
INFO: No packages to install, update or remove
ERROR: DataStructures had test errors
 in error at error.jl:21
 in test at pkg/entry.jl:719
 in anonymous at pkg/dir.jl:28
 in cd at ./file.jl:20
 in cd at pkg/dir.jl:28
 in test at pkg.jl:68
 in process_options at ./client.jl:221
 in _start at ./client.jl:362
 in _start_3B_3769 at /home/idunning/julia04/usr/bin/../lib/julia/sys.so

>>> end of log

I believe this is fixed.


server.py doesn't gracefully check channels format

It might make sense to implement graceful checking and error message if tuple is passed as channel when used meter is not in multichannel mode (device type 52 for instance)
server log sample:
ptd-server 2021-02-23 18:45:49,435 [INFO] Sending to ptd: 'Stop'
02-23-2021 18:45:49.435ptd-server 2021-02-23 18:45:49,435 [INFO] Reply from ptd: 'Stopping untimed measurement'
: Response to client sent: Stopping untimed measurement
ptd-server 2021-02-23 18:45:49,446 [ERROR] Got an exception
Traceback (most recent call last):
  File "C:\cygwin64\home\lab\mlperf\power-dev\ptd_client_server\lib\server.py", line 528, in handle_connection
    reply = self._handle_cmd(cmd, p)
  File "C:\cygwin64\home\lab\mlperf\power-dev\ptd_client_server\lib\server.py", line 594, in _handle_cmd
    return unbool[self.session.stop(Mode.RANGING)]
  File "C:\cygwin64\home\lab\mlperf\power-dev\ptd_client_server\lib\server.py", line 804, in stop
    start_channel = int(self._server._config.ptd_channel[0])
ValueError: invalid literal for int() with base 10: '(2'

List of multichannel devices: 48 (Yokogawa WT500), 59 (Hioki PW3336), 61 (Hioki PW3337), 77 (Yokogawa WT332/WT333)

Thank you for the report. Could you check the changes?

verified it works
ptd-server 2021-02-25 07:42:15,207 [CRITICAL] server.cfg: 'channel' value should consist of one number or be disabled for a 1-channel device 52.
PS C:\cygwin64\home\lab\mlperf\power-dev\ptd_client_server> python server.py -c server.cfg
ptd-server 2021-02-25 07:42:33,648 [CRITICAL] server.cfg: 'channel' value should consist of two numbers for a multichannel device 77.

thanks


Publish GetConn and friends again

In issue #28 the GetConn type was supposedly published, but as of version 0.19.0 it is not published.
Besides the GetConn type, there's also a DisconnectPool future, which similarly deserves to be published.

Ah! Sorry for that. Will publish asap.

Published as v0.19.1


Quicksort

Coverage remained the same at 100.0% when pulling c615ea13db6c5a7242b6750b87568d29335e903f on quicksort into db9da1f498cb1db9c24982bdb68d122da4314569 on master.


IssueID:1890:Loongson1C support

[Detail]
Loongson1C support.
[Verified Cases]
Build Pass: helloworld_demo@qemu_loongson1c
Test Pass: helloworld_demo@qemu_loongson1c

Security Check : c64d78504a7e04cfe57b9a248b324e6dc6f9366b

auto build fail,number:263


How to run this app?

Hello, I cloned the repo and built from master:

yarn build
yarn start

I have latest yarn.
app is available at 3001, because I am locally running grafana.
I am able to connect to my neo4j instance (running inside Neo4jDesktop) using my safari browser.
I am getting white screen with the following text:
MATCH (n) RETURN count(n) AS count
There are 29457 nodes in the database.
And nothing else. What could be the problem?
Thanks.

This is just a starter kit with a basic app that shows the database count to start with.  You can start to build your application by editing App.jsx.

This is just a starter kit with a basic app that shows the database count to start with.  You can start to build your application by editing App.jsx.

Basically what is at http://charts.graphapp.io/
There, btw I have a problem setting the default database, so in query explorer, the incorrect one is introspected.

Basically what is at http://charts.graphapp.io/
There, btw I have a problem setting the default database, so in query explorer, the incorrect one is introspected.

You can find the code for the Charts app at https://github.com/neo4j-labs/charts

You can find the code for the Charts app at https://github.com/neo4j-labs/charts

great, cloned charts, built everything, and started, but no login form is displayed.
I can create dashboard, but queries & reports obviously don't work throwing client auth failure.
What could be a problem?

great, cloned charts, built everything, and started, but no login form is displayed.
I can create dashboard, but queries & reports obviously don't work throwing client auth failure.
What could be a problem?

If you aren't seeing a login screen then it's taking the driver configuration in .env and passing a driver instance to Neo4jProvider in index.tsx.
Do you have REACT_APP_NEO4J_SCHEME in .env?

If you aren't seeing a login screen then it's taking the driver configuration in .env and passing a driver instance to Neo4jProvider in index.tsx.
Do you have REACT_APP_NEO4J_SCHEME in .env?

no, but I figured out that I need to edit index.tsx. Thanks

no, but I figured out that I need to edit index.tsx. Thanks

No issue. Thanks for help!


Java tutorial 1 has a broken external link

Describe the bug
While following the tutorial for Java I ran into an error while sending message on the queue. The webpage asks to check configuration file documentation. Which returns a 404 error

Reproduction steps

Go to java tutorial
Scroll down to : Sending doesn't work!
Click on configuration file documentation
...

Expected behavior
Display the configuration file documentation page.
Additional context
No response

Thank you for providing these details.


fix: use-upstream-rbac-image

CC: @lance @osmman

RE: https://github.com/securesign/secure-sign-operator/pull/331#issuecomment-2063507288

/override ci/prow/tas-operator-e2e


chore: bdd-gh-1578660330 to 0.0.1

chore: Promote bdd-gh-1578660330 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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


Error inflating class

Error inflating class android.constraintlayout.Contrainlayout and android.support.design.widget.CoordinatorLayout

Fixed in 1.5.0


Fix Menu Button Visibility

This pull request fixes the issue of the main menu button disappearing since ChatGPT moved the profile button to a new location. This PR implements a robust solution for relocating the menu button to the profile button div.
Changes Made:

Main Menu Button Placement: The export menu button now resides as the first child of the profile button div. This visually groups it with the profile button.
Optimized Injection: The injectMenu() function utilizes setInterval to actively check if the menu button already exists in the profile button div. If not, it injects it. The setInterval loop is immediately stopped after successful injection to minimize unnecessary checks and improve performance.
Minor Aesthetic Optimizations: Since the menu button now shares a container with the profile button, the extra border and divider line have been removed for a cleaner look.

Testing:

Tested these changes in MS Edge by updating the userscript manually.
Performed a hard refresh (Ctrl+Shift+R or Cmd+Shift+R) on ChatGPT to clear the cache and ensure a clean reload.
Confirmed the export menu button consistently appears next to the share chat and profile button in the header.

Benefits:

Improved Visibility: The menu button is visible again after ChatGPT moved the profile button.
Improved Performance: The optimized injection logic, including the setInterval termination, reduces unnecessary DOM manipulation for better performance.

Screenshot:

Closing Issues
This PR closes issue #251.

Do you still have the team workspace button on the left side? If so, I would still prefer to put the menu there.

the team workspace button is still on the left side. The improvement moves the export button to the top right, alongside the profile and share buttons. This does solve the issue of the menu disappearing caused by some extensions. I noticed that enabling certain scripts can still cause the menu to disappear, but it’s not a major problem.

Thanks for the contribution. But I still want to keep it in the same place. We can work on fixing the disappearing issue. I will merge the PR if we can't find or fix the problem.

Thanks for the contribution. But I still want to keep it in the same place. We can work on fixing the disappearing issue. I will merge the PR if we can't find or fix the problem.

Update:
The menu disappeared again today. I’m not sure what happened, but the only thing I’m certain of is that it conflicts with some scripts or extensions—sometimes with extensions, sometimes with scripts. The most challenging part is that it doesn’t conflict with any specific extension or script.

Thanks for feedback 👍. An additional observation is that the menu button
in my PR didn't survive switching from desktop to mobile to desktop view.
Op do 5 sep 2024 om 19:11 schreef kgsjtl @.***>:

Thanks for the contribution. But I still want to keep it in the same
place. We can work on fixing the disappearing issue. I will merge the PR if
we can't find or fix the problem.
Update:
The menu disappeared again today. I’m not sure what happened, but the only
thing I’m certain of is that it conflicts with some scripts or
extensions—sometimes with extensions, sometimes with scripts. The most
challenging part is that it doesn’t conflict with any specific extension or
script.
—
Reply to this email directly, view it on GitHub
https://github.com/pionxzh/chatgpt-exporter/pull/252#issuecomment-2332244834,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKTLWPE62XOE5I5G3VERKVTZVCGC3AVCNFSM6AAAAABNQSKGMSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZSGI2DIOBTGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Latest version restores the menu button visibillity provided that another user script doesn't break it.


Fix missing initializations

@mbelluzzo check it out. Some trivial stuff, please check if the values set to initialize private members of mav_vehicle make sense.

:+1: Merged!


Fix ItemQueue/BlockingItemQueue tests

Problem
Have been seeing intermittent build failures in travis, this is a stab at fixing these errors.
Solution
ItemQueueTest

adding a sleep in the Items causes the error to be manifested, it shows the items aren't getting drained out of the queue.  The theory is that in travis sometimes the latch takes longer than 100ms to close.  Thus increasing timeout to 5s, which should cover most cases.

BlockingItemQueueTest

If you add a sleep after the initial fill(), then test adding an overflow item, you see that the test fails.  I believe the queue sometimes drains too fast, thus we never overflow.  Switching to use a countdown latch which sync's all the items to exit at the same-ish time.  Similar to ItemeQueueTest's usage.

fixes #566

added longer timeouts for all Awaits in ItemQueueTest, due to failures in tracis

thanks!


[BUG] status code 500 while trying to login

http://localhost:8000/auth/login
I am trying to login. Account has 2FA . First I filled username and password after that I get the 2FA code via email and filled the input and send another request but still getting 500 error.
I am using the docker environment.

shown in the above picture Is mock data. However, I have tried with the real account but, it did not work either.
see below:

@fatihozdil send your fix as PR please

@adw0rd I will try to solve, but I am not very familiar with python. I just wanted to try api.

I faced with this 500 error when I used to install above commands for installation.  The problem is 2fa, it does not wait for me to write 2fa code, throws "EOFError: EOF when reading a line" error. I tried other installation methods. I did not face with the same problem it let me write 2fa code, but I faced another error ChallengeRequired. I don't know what that means.

@fatihozdil https://adw0rd.github.io/instagrapi/exceptions.html

is instagrapi-rest as powerfull as instagrapi ?
i mean.. does it has the same features?..
I'm about to start a project with instagram and I want to use "REST" but I don't know if "REST" receives updates.
I can see that "Instagrapi" is more active. Should I use instagrapi instead of REST?  (( I'm sorry if this has nothing to do with the post, but I couldn't find where to put my question.))


Rule Ideas: A11y Rules

Rule(s) related to or rule(s) missing
A11y (Accesibilty) is getting a bigger thing, and has some new Laws in Germany following the EU 2019/882 that have to be followed, or big Lawsuits can happen.
It would be cool to have some rules that could find a11y related stuff in your twig templates and possibly even fix some of them automatically
The A11y Spec: https://www.a11yproject.com/checklist/
There was a WIP RuleSet for the old FriendsOfTwig CS Fixer: https://github.com/NielsdeBlaauw/twigcs-a11y
Expected behavior
Twig cs fixer can tell me about stuff that I should fix related to accessibility (like aria labels)
Actual behavior
It cant yet :(

Hi, having perfectly implemented rules would be hard because of how twig works and this is certainly why there is almost no rule in the NielsdeBlaauw/twigcs-a11y repository.
You can write things like
{% if image %}
<img
{% else %}

and it's kinda complicated to understand if the img has all the required attributes.
Also this lib doesn't parse HTML, only twig.
 is considered as

Text

which doesn't help writing such rules.

I don't expect a perfect implementation, but Developers are just humans.
Having automated pipelines running first, and then taking time to check stuff, after they are saying that everything is right, can highly simplify such work, and how often accessibility needs to be tested.
Looking at the Technical Case, I would also say, that an extra experimental repository for experimental extensions like this would make sense.
Alone having it checking for the possibility of there being aria-tags and labels, would already be an improvement, because the fixer can be used locally for linting, and in the CI.
I dont exspect any tool to completely replace a full WAST test, but it means caching problems earlier.

I'd be interested to see two or three concrete examples.. with the twig code that would be marked "invalid" and the reason.
I fear this can only cover a very limited subset of use case, as an important part of aria attributes are runtime based.

Here are some:
Invalid: Image doesn't have an alt text

Valid:

Invalid: There should always be an alt Tag

Valid: We don't 100% know that the alt_text var is set, but at least we know that there is an alt tag on that image. Verifying exact content shouldn't be the job of the Fixer

Invalid: Headers should be in consecutive order. The Start of that order (there should always be an h1) isn't important, though, because it can't be checked for by twig cs fixer.

A

B

C

Valid:

A

B

C

Invalid: HTML Tags should have a language defined for screen readers

Valid:

Invalid: a should use href instead of onclick

Valid:

Invalid: Videos shouldn't have autoplay, when not muted
<video autoplay>

<video autoplay {% if should_mute%}muted{% fi %}>

Valid:
<video autoplay muted>

<video  {% if should_autplay%}autoplay muted{% fi %}>

I really think these rules could only (and more easily) checked by looking at the generated HTML.
I fear every exemple you are giving would be silenced with a simple "include" or any Twig tag/filter/function ... so the end result would  only give a very partial vision of the problems.
And i'm not talking about the difficulty to parse HTML mixed with Twig tags ..  🤷♂

Having automated pipelines running first, and then taking time to check stuff, after they are saying that everything is right, can highly simplify such work, its complexity and the time needed.

Some basic functionnal tests could be perfect place for this!
If you work with Symfony, you can implement this kind of checks with the DomCrawler provided by the WebTestCase client: https://symfony.com/doc/current/testing.html#testing-the-response-assertions

Guess, I will just have to pull full E2E in all projects, but I get the Points.
the DomCrawler is just a worse Version of real E2E, and out of my experince buggy.

I will just have to pull full E2E in all projects

@snoweuph I don't think full E2E are needed, as JS should not be required to comply with Ally.
So a simple WebTestCase with no real DOM execution will be enough (and a lot quicker to execute).
You can look at zenstruck/browser that offers a really nice DX to test that kind of things.
In particular, you coul create your own set of Browser Components (like an ImageComponent for instance) to have some custom assert with domain value, easily re-usable along your project tests/pages.
  // ...
  ->use(function(ImageComponent $image) {
      $image
          ->assertAltIsDefined()
          ->assertWidth()
         ->assertHeight()
  })
  // ...

Using this with the KernelBrowser offers you very good coverage without the hassle of E2E clients, and could fullfill your need :)


cleanup(unit_tests): try making test_configure_interesting_sets more robust

What type of PR is this?

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

/kind bug
/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

/kind release

Any specific area of the project related to this PR?

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

/area build

/area engine

/area tests

/area proposals

/area CI

What this PR does / why we need it:
@jasondellaluce still getting errors locally with current master, e.g.
[----------] Global test environment tear-down
[==========] 30 tests from 9 test suites ran. (3009 ms total)
[  PASSED  ] 28 tests.
[  FAILED  ] 2 tests, listed below:
[  FAILED  ] ConfigureInterestingSets.selection_not_allevents
[  FAILED  ] ConfigureInterestingSets.selection_allevents

 2 FAILED TESTS
[]$ git status
nothing to commit, working tree clean
[]$ git log
commit 0b6e243582c0e467efb6c2355259c4eefaa28c91 (HEAD -> master, upstream/master)

Have been reading more about it and everywhere people warn that gtests can influence each other. SetUp and TearDown didn't do anything, oddly just changing some variable names per test did help :woman_shrugging: ... now everything is passing again locally. Unsure if it's now also just random or really helped, at the same time these changes won't hurt at all. Let's see if the CI remains intact.
Also experimented with adding a generic syscall such as init_module to the s_sample_filters ... let's try adding it after the final libs patch to make sure we get consistent new / final behavior throughout.
Which issue(s) this PR fixes:

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

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: incertum
Once this PR has been reviewed and has the lgtm label, please assign mstemm for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

LGTM label has been added.  Git tree hash: d9bae7e53c6864c570c64906025fe0b8cb737dc6

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

Needs approval from an approver in each of these files:

OWNERS [FedeDP,jasondellaluce]

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


Error when running dendron publish export (selfcontained)

Please select the area your bug applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[x] Publish.  External Site Publish

Describe the bug
I have a self-contained remote vault (just migrated) and if I run yarn dendron publish export --target github --yes I get an error Cannot read properties of undefined
After the error occurs, it just hangs and has to be stopped manually to prevent it running indefinitely.
Look at the end for complete log.
To Reproduce
I have no idea, I don't know what's wrong.
Expected behavior
Exporting should work.
Screenshots
not applicable
Desktop (please complete the following information)

OS: Windows and Ubuntu (Desktop and Github Actions)
Browser (if with published websites; include version): /
Versions (all that apply to bug):

Dendron Extension: /
VS Code version: /
Dendron CLI: 0.96.0

Additional context
The Error
(cleaned up long paths and linebreaks)
yarn dendron publish export --target github --yes
🌱 checking if NextJS template is initialized
🌱 NextJS template is already initialized.
⠋ generating metadata for publishing...
⠧ error {"stack":"TypeError: Cannot read properties of undefined (reading 'dendron.defaultTimestampDecorationFormat')
    at Function.getFM (...\\node_modules\\@dendronhq\\engine-server\\lib\\markdown\\utilsv5.js:123:102)
    at Function._procRemark (...\\node_modules\\@dendronhq\\engine-server\\lib\\markdown\\utilsv5.js:187:53)
    at Function.procRemarkParse (...\\node_modules\\@dendronhq\\engine-server\\lib\\markdown\\utilsv5.js:342:21)
    at Function._procRehype (...\\node_modules\\@dendronhq\\engine-server\\lib\\markdown\\utilsv5.js:284:35)
    at Function.procRehypeFull (...\\node_modules\\@dendronhq\\engine-server\\lib\\markdown\\utilsv5.js:364:27)
    at NextjsExportPod._renderNote (...\\node_modules\\@dendronhq\\pods-core\\lib\\builtin\\NextjsExportPod.js:174:48)
    at NextjsExportPod.renderBodyToHTML (...\\node_modules\\@dendronhq\\pods-core\\lib\\builtin\\NextjsExportPod.js:278:32)
    at ...\\node_modules\\@dendronhq\\pods-core\\lib\\builtin\\NextjsExportPod.js:334:22
    at arrayMap (...\\node_modules\\lodash\\lodash.js:653:23)
    at Function.map (...\\node_modules\\lodash\\lodash.js:9622:14)","message":"Cannot read properties of undefined (reading 'dendron.defaultTimestampDecorationFormat')"}
{"level":50,"time":1653578126547,"pid":19928,"hostname":"spectre","name":"publish <cmd>","stack":"TypeError: Cannot read properties of undefined (reading 'dendron.defaultTimestampDecorationFormat')
    at Function.getFM (...\\node_modules\\@dendronhq\\engine-server\\lib\\markdown\\utilsv5.js:123:102)
    at Function._procRemark (...\\node_modules\\@dendronhq\\engine-server\\lib\\markdown\\utilsv5.js:187:53)
    at Function.procRemarkParse (...\\node_modules\\@dendronhq\\engine-server\\lib\\markdown\\utilsv5.js:342:21)
    at Function._procRehype (...\\node_modules\\@dendronhq\\engine-server\\lib\\markdown\\utilsv5.js:284:35)
    at Function.procRehypeFull (...\\node_modules\\@dendronhq\\engine-server\\lib\\markdown\\utilsv5.js:364:27)
    at NextjsExportPod._renderNote (...\\node_modules\\@dendronhq\\pods-core\\lib\\builtin\\NextjsExportPod.js:174:48)
    at NextjsExportPod.renderBodyToHTML (...\\node_modules\\@dendronhq\\pods-core\\lib\\builtin\\NextjsExportPod.js:278:32)
    at ...\\node_modules\\@dendronhq\\pods-core\\lib\\builtin\\NextjsExportPod.js:334:22
    at arrayMap (...\\node_modules\\lodash\\lodash.js:653:23)
    at Function.map (...\\node_modules\\lodash\\lodash.js:9622:14)","type":"Error","msg":"Cannot read properties of undefined (reading 'dendron.defaultTimestampDecorationFormat')"}
unknown error [object Object]

dendron.yml
version: 5
dev:
    enablePreviewV2: true
commands:
    lookup:
        note:
            selectionMode: extract
            confirmVaultOnCreate: true
            vaultSelectionModeOnCreate: smart
            leaveTrace: false
            bubbleUpCreateNew: true
            fuzzThreshold: 0.2
    randomNote: {}
    insertNote:
        initialValue: templates
    insertNoteLink:
        aliasMode: none
        enableMultiSelect: false
    insertNoteIndex:
        enableMarker: false
    copyNoteLink: {}
workspace:
    vaults:
        -
            fsPath: .
            remote:
                type: git
                url: https://github.com/joshuajeschek/dendron-tuc
            name: tuc
            selfContained: true
    journal:
        dailyDomain: daily
        name: journal
        dateFormat: y.MM.dd
        addBehavior: childOfDomain
    scratch:
        name: scratch
        dateFormat: y.MM.dd.HHmmss
        addBehavior: asOwnDomain
    task:
        name: ''
        dateFormat: ''
        addBehavior: childOfCurrent
        statusSymbols:
            '': ' '
            wip: w
            done: x
            assigned: a
            moved: m
            blocked: b
            delegated: l
            dropped: d
            pending: 'y'
        prioritySymbols:
            H: high
            M: medium
            L: low
        todoIntegration: false
        createTaskSelectionType: selection2link
    graph:
        zoomSpeed: 1
    enableAutoCreateOnDefinition: false
    enableXVaultWikiLink: false
    enableRemoteVaultInit: true
    enableUserTags: true
    enableHashTags: true
    workspaceVaultSyncMode: noCommit
    enableAutoFoldFrontmatter: false
    enableEditorDecorations: true
    maxPreviewsCached: 10
    maxNoteLength: 204800
    enableFullHierarchyNoteTitle: false
    workspaces: {}
preview:
    enableFMTitle: true
    enableNoteTitleForLink: true
    enableFrontmatterTags: true
    enableHashesForFMTags: false
    enableMermaid: true
    enablePrettyRefs: true
    enableKatex: true
    automaticallyShowPreview: false
publishing:
    enableFMTitle: true
    enableNoteTitleForLink: true
    enableMermaid: true
    enablePrettyRefs: true
    enableKatex: true
    copyAssets: true
    siteFaviconPath: favicon.png
    siteHierarchies:
        - tuc
        - semester
        - betriebssysteme
        - mathematik-iii
        - theoretische-informatik
        - softwareengineering
        - rechnerorganisation
        - mediengestaltung
        # - media-retrieval
    enableSiteLastModified: true
    siteRootDir: docs
    siteUrl: 'https://tuc.jeschek.eu'
    enableFrontmatterTags: true
    enableHashesForFMTags: false
    enableRandomlyColoredTags: true
    writeStubs: false
    seo:
        title: TUC - Joshua Jeschek
        description: Joshs Uniaufzeichnungen
        author: Joshua Jeschek
    github:
        enableEditLink: true
        editLinkText: Edit this page on GitHub
        editBranch: main
        editViewMode: tree
    enablePrettyLinks: true

dendron.code-workspace
(this should be irrelevant, right?)
{
    "folders": [
        {
            "path": "notes",
            "name": "tuc"
        }
    ]
}

Dendron Log file
not applicable

Taking a look at this. Can you confirm the following?

This is what your self contained vault looks like

- dendron.yml
- dendorn.code-workspace
- notes/

You are running dendron publish export --target github --yes in the same folder as dendron.yml?

I can confirm all of the above

0.97.0 does not fix the issue, sadly.
if you want to take a look, here is my repo ^^

This should not have been closed! 😲

This should not have been closed! 😲

Apologies, our bot was not set up properly when this issue was set to status.info-needed (which will auto-close after a certain amount of time unless there are more comments). We fixed the bot today and is now catching up and closed the issue without seeing your comments.

hey @joshuajeschek
took a look and it looks like we're failing to render because dendron.code-workspace doesn't have the following setting (this is used when we are calculating note references)
"settings": {
        "dendron.defaultTimestampDecorationFormat": "DATE_MED_WITH_WEEKDAY"
}

if you add the above, it should work

Thanks for the help, it's working after I added that option! I suspect that I could remove it again after #3114 has been merged and released?

yep

just merged. will go out in the release next tuesday (8 days from now)

this just went out. in 0.101 :)

works as intended! thanks as always!


Mod Config not saving [1.3]

The Mod Config is not saving between sessions (Restart the Game).

Fixed in release v1.0.4


Prepare for v1.0.1 release.

This PR updates the change log in preparation for the release of version 1.0.1.

the summary of new developments look exactly what has happened after v.1.0.0

Thanks for confirming!
:tada:


interop classes 1

move Prelude.fs into Escalier.Compiler
Create a Prelude class

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

Comparison is base (5dabc84) 82.19% compared to head (a7026d0) 82.24%.

Files
Patch %
Lines

src/Escalier.Compiler/Prelude.fs
94.71%
14 Missing :warning:

src/Escalier.Interop/Infer.fs
25.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #147      +/-   ##
==========================================
+ Coverage   82.19%   82.24%   +0.05%
==========================================
  Files          20       20
  Lines        6901     6907       +6
  Branches      946      946
==========================================
+ Hits         5672     5681       +9
+ Misses       1229     1226       -3

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


My bug report

The command---- >  Get-Command -Module HardwareCertification  is not able to list the HLK Automation tool cmdlets in the powershell with latest version which is required for execution of test cases. The command is not working in version of  HCK -2.1 and HLK -1607 . The same command works with HLK-1903 and HLK-1809 version.
Expected results: It should display the following cmdlets:
Export-HwCertTestCollectionToXml
Import-HwCertTestCollectionFromXml
Merge-HwCertTestCollectionFromPackage
Merge-HwCertTestCollectionFromXml
New-HwCertProjectDefinitionFile
New-HwCertTestCollection
New-HwCertTestCollectionExcelReport
Actual results: It is not displaying anything.

If you load the module in PowerShell 7.0 and the module doesn't work please report the issue to the module owner.


Space as default leading trivia for left brace

https://github.com/apple/swift/pull/59980
The next approaches didn't work:

Pass space trivia as the build expression argument − it leaves property/subscript accessors untouched because there is AccessorBlock/CodeBlock choice and build expression has no context regarding actual syntax type and related traits.
set default value of leadingTrivia for syntax types with the Braced trait − it breaks on the SwitchStmt, which doesn't start with the brace.

I don’t fully understand the tradeoffs of the different designs from your PR description but setting requires_leading_space=True  for the LeftBrace token seems like a simple solution to me that doesn’t require any new logic. From the top of my head I can’t come up with a situation where you wouldn’t want to have a space in front of {.

https://github.com/apple/swift/pull/59980
@swift-ci Please test

https://github.com/apple/swift/pull/59980
@swift-ci Please test

https://github.com/apple/swift/pull/59980
@swift-ci Please test

@ahoppen could you please make sure it is merged? Otherwise I'll need to rebase it again after more generated files are moved to the SwiftSyntaxBuilderGeneration

https://github.com/apple/swift/pull/59980
@swift-ci Please test

Sorry, I forgot to merge the PR. Feel free to ping me on other PRs as well if tests passed and I didn’t merge them.

Curious, could we do this for arrow tokens (->) as well? Having a leading space along with the trailing space would look a bit nicer in generated function return type annotations.

Curious, could we do this for arrow tokens (->) as well?

This is the next thing I want to do


Use HolidayBase::add_holiday* family methods [c]

Proposed change

This PR continues https://github.com/vacanza/python-holidays/pull/1368. Update countries starting w/ c.
Type of change

[ ] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[x] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency update (version deprecation/upgrade)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] This PR is filed against beta branch of the repository
[x] This PR doesn't contain any merge conflicts and has clean commit history
[x] The code style looks good: make pre-commit
[x] All tests pass locally: make test, make tox (we strongly encourage adding tests to your code)
[x] The related documentation has been added/updated (check off the box for free if no updates is required)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

148 of 148 (100.0%)  changed or added relevant lines in 13 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 99.796%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
9026

Relevant Lines:
9036

💛  - Coveralls


fix(driver/modern-bpf): solve some verifier issues and improve debbuging capabilities

What type of PR is this?
/kind bug
/kind feature
Any specific area of the project related to this PR?
/area driver-modern-bpf
/area libscap-engine-modern-bpf
Does this PR require a change in the driver versions?
No
What this PR does / why we need it:
This PR fixes some verifier issues in the modern bpf probe and improves the debugging phase when needed.
Verifier issues:

In recent kernel versions (https://github.com/torvalds/linux/commit/f1a7941243c102a44e8847e3b94ff4ff3ec56f25) struct mm_rss_stat doesn't exist anymore.
In some images like cos_containerd the task struct has no loginuid field

Debugging improvements:

modify signal_deliver test that causes some strange failures on some machines.
expose a parameter to modify the libbpf verbosity when opening the engine. Could be useful to detect relocation issues.
add a dockerfile to create a docker image for scap-open executable. Could be very useful to deploy custom pods on clusters during the development :)

Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

/milestone 0.11.0

Left a comment, otherwise this LGTM! I am not sure about the dockerfile (i mean, it seems a bit over-engineered for a small example :) ) but no strong opinions on that.

I used it to debug these issues so in my opinion it could be useful :) Of course, you are not forced to use it :)

LGTM, i'll approve once the ci is :green_circle:

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

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

LGTM label has been added.  Git tree hash: bd6077b06f65cb94c190ca4c9f69ac092963509a

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,jasondellaluce]

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


Bind vs addParameter

I can't seem to use .bind(myObj) to update the myObjs table. All of my POJOs data members have the same exact names as their corresponding SQL Server database fields. The POJO I'm using has private data members and public get/set functions.
If I set each parameter individually with .addParameter() this works fine. Can .bind() not access private data members via public getter functions?
None of my function or field names start with "is". All of the "get" function names start with the word get followed by the name of the database field/POJO-private-data-member exactly as they are typed. For instance a DB field called Active corresponds to a private data member called Active which is accessible via a public method called getActive().
Is there a way to troubleshoot bind?

I think I have the problem that, my POJO does not work with bind properly. I have setters and getters matching the POJO field names, and the :fields, and when the attributes are set to private, it just gives sql null values.


Docker mounts increasing as we enable download to host option for models and datasets

The number of docker mounts in the app-mlperf-inference script is increasing as we enable the download to host option for models and dataset. Is this a good practice? Alternatively, we could use a common environment variable like CM_ML_MODEL_PATH to mount the path. However, this would create an issue when the script is called with the _all-models variation, as it attempts to download multiple models for a single Docker container, causing the common environment variable to be substituted during the downloads.
Edit:
I think providing the docker mounts inside the variations solves the issue. An example is here

I agree that it's better not to grow global mounts for all models but to use variations for extra mounts.


Migrate to CircleCI

Could you try pushing to this project? Anything will do and you can delete the commit later. It should be hooked up to Circle now.


Add newline support for commit

Adds support for newlines in commits, either by passing a message with newlines or an array of messages with the option multi.
Closes #122

Please add documentation and examples as well as change the opts.multi to opts.multiline

Thanks!


ipcache reference count underflow

Occurs in latest master but problem has existed in 1.3 and possibly before that.
2018-10-28T14:29:12.134973434Z level=error msg="BUG: attempt to release ipcache entry while refcnt == 0" key=cilium/state/ip/v1/default/<IP_ADDRESS>/0 subsys=ipcache
2018-10-28T14:29:12.135006306Z level=error msg="BUG: attempt to release ipcache entry while refcnt == 0" key="cilium/state/ip/v1/default/::/0" subsys=ipcache
2018-10-28T14:29:12.135011117Z level=error msg="BUG: attempt to release ipcache entry while refcnt == 0" key=cilium/state/ip/v1/default/<IP_ADDRESS>/0 subsys=ipcache
2018-10-28T14:29:12.135014758Z level=error msg="BUG: attempt to release ipcache entry while refcnt == 0" key="cilium/state/ip/v1/default/::/0" subsys=ipcache
2018-10-28T14:29:12.135190256Z level=error msg="BUG: attempt to release ipcache entry while refcnt == 0" key=cilium/state/ip/v1/default/<IP_ADDRESS>/0 subsys=ipcache
2018-10-28T14:29:12.135212862Z level=error msg="BUG: attempt to release ipcache entry while refcnt == 0" key="cilium/state/ip/v1/default/::/0" subsys=ipcache
2018-10-28T14:29:12.135165356Z level=error msg="BUG: attempt to release ipcache entry while refcnt == 0" key=cilium/state/ip/v1/default/<IP_ADDRESS>/0 subsys=ipcache
2018-10-28T14:29:12.135176748Z level=error msg="BUG: attempt to release ipcache entry while refcnt == 0" key="cilium/state/ip/v1/default/::/0" subsys=ipcache

It is always the default gateway block <IP_ADDRESS> or ::0 (cc @joestringer )

Duplicate of https://github.com/cilium/cilium/issues/6017?


FlowRouter.reactiveCurrent() causes 2 run, when first time access

When access directly (not clicking on anchor), autorun that includes FlowRouter.reactiveCurrent(), runs two times. Ideally, it should be run once.
FlowComponents.define('hello', function() {
   this.autorun(function() {
     console.log(FlowRouter.reactiveCurrent());
   });
});

I want to make sure there is an id param. Something like that:
var comp = Components.define('postDetail', function() {
    this.autorun(function() {
       var id = FlowRouter.getParam('id');
       if (!id) {
          return FlowRouter.go('/');
       }
       console.log(id);
    });
});

but first time id is undefined.

Is the first run when the autorun is registered and the second one when FlowRouter has run and changed the value?
What if you use computation.firstRun? Docs: computation#firstRun.
Do this instead?
var comp = Components.define('postDetail', function() {
    this.autorun(function(computation) {
       var id = FlowRouter.getParam('id');
       if (!(computation.firstRun || id)) {
          return FlowRouter.go('/');
       }
       console.log(id);
    });
});```

@elidoran Thanks for the workaround. It did the trick.
But it feels wrong, I think it should be assigned for first time.
I guess #95 it is also related to it.

I think I found the source of the problem in #95. If approved I'll generate a PR. It's about the order of storing the new values in the ReactiveDict in relation to calling the route's action.
In your example above you're calling autorun and that is called immediately by FlowComponents here. I haven't used FlowComponents but I briefly looked at its docs and it seems we're supposed to write the components in a script file. So, is the example code you have above in a component script file? If so, then that code is run at app loading time, not in an action like #95.
Where is that call to FlowComponents.define() being made?
When is the autorun executing and what's triggering it?

Oh, I forgot to change code. It is exactly same as
var comp = FlowComponents.define('postDetail', function() {
    this.autorun(function() {
       var id = FlowRouter.getParam('id');
       if (!id) {
          return FlowRouter.go('/');
       }
       console.log(id);
    });
});

it called when access to page/route.

Released version 1.8.0 with fixes for this bug.

:+1:


Over layer on slidebar

When sidebar is open an over layer should be on the main contain so when user click on over layer sidebar will be closed.

Merged with version next


Figure out how python imports work.

Right now everything is in the root directory because I can't seem to get anything to run otherwise.

Got it! :D


support for unified cgroups (cgroup2)

This implements support for cgroups v2 along support for legacy version.
I took #2785 and fixed the conflicts.
Half of #1757
Based on work from: Michał Świętek<EMAIL_ADDRESS>
/cc @mike1729 @lucab

Pushed some fixes.

I've left some comments and questions inline.
Something I'm not completely sure about is that we seem to have have settled on the "legacy"/"unified" dichotomy, but in some signatures we just omit the "unified" part (not sure if for brevity or just an overlook). I'd personally go for using "v1"/"v2" everywhere, instead: it makes declarations shorter and adds uniformity. (The latter is close to bikeshedding, but the former requires some renaming anyway)
Moreover, this will conflict with https://github.com/coreos/rkt/pull/3027. I'd take that one before this, so you'll need to rebase.

Something I'm not completely sure about is that we seem to have have settled on the "legacy"/"unified" dichotomy, but in some signatures we just omit the "unified" part (not sure if for brevity or just an overlook). I'd personally go for using "v1"/"v2" everywhere, instead: it makes declarations shorter and adds uniformity. (The latter is close to bikeshedding, but the former requires some renaming anyway)

Heh, ok, I just read that. I think your suggestion makes sense.

Rebased to master
Fixed a couple of review comments
Renamed functions to v1/v2

PTAL.

Can we have a VM in Jenkins with cgroup2? fedora-rawhide-cgroup2 maybe?

Can we have a VM in Jenkins with cgroup2? fedora-rawhide-cgroup2 maybe?

I'll get one

This needs to wait for https://github.com/coreos/rkt/issues/2843 because we need https://github.com/systemd/systemd/pull/2903

@iaguis bumping to next release, since it is still WIP

WIP, hence bumping to the next release. @alban Do you think we'll make it for v1.16.0?

I'll rebase and fix the outstanding issues for v1.16.0

Updated. PTAL.


Add mixers to hyper_params_factory

Move the QAOA mixers in QAOA-CV example to the factory module.

Thanks @qbarthelemy !


add a way to serialize normalization

save the featurewise mean and stdev, the ZCA, and whether or not we are doing samplewise mean and stdev

You mean, serializing ImageDataGenerator?

Yes

please reopen. this is still an issue. i missed the stale tag.


[Bug] don't drop temporary table o$pt

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
The global temporary table is not deleted after running the model due to an
[dbt-oracle/blob/main/dbt/include/oracle/macros/adapters.sql](https://github.com/oracle/dbt-oracle/blob/7348c2b21883c156583adc8282876c7a993eb13d/dbt/include/oracle/macros/adapters.sql#L259)
example:
create global temporary table TEST (object_id number, object_name varchar2(30)) on commit PRESERVE rows;
Such tables get bound to the session with the first insert and then this binding is ONLY removed when the session exits or by truncate. If the binding exists DDL cannot be performed on these temporary tables and if you try a DDL on such tables you would receive ORA-14452 attempt to create, alter or drop an index on temporary table already in use.
add before
https://github.com/oracle/dbt-oracle/blob/7348c2b21883c156583adc8282876c7a993eb13d/dbt/include/oracle/macros/materializations/incremental/incremental.sql#L63
{% for rel in to_drop %}
{% do adapter.truncate_relation(rel) %}
{% endfor %}
Expected Behavior
drop table
Steps To Reproduce

create model materialized='incremental'

Relevant log output using --debug flag enabled
No response
Environment
- OS:MacOs
- Python:3.9
- dbt-core: 1.1.0

What Oracle database version are you using dbt with?
12.1.0
Additional Context
No response

Hi @AlexMorti
Thank you for the detailed bug report. I would like to understand how to reproduce the error ORA-14452
In dbt-oracle, temporary tables are named in the following format:
    {% set dtstring = dt.strftime("%H%M%S") %}
    {% set tmp_identifier = 'o$pt_' ~ base_relation.identifier ~ dtstring %}

So, unless you have multiple dbt sessions at the same time (precisely the same second), you wouldn't get ORA-14452

test model
 {{config(materialized='incremental')}} select 1 as user_id,        'test' first_name,        'test2' last_name  from dual
if you remove the exception condition
https://github.com/oracle/dbt-oracle/blob/461f58cd5d079d2ee4eac72dfc9ff09969d8fd7c/dbt/include/oracle/macros/adapters.sql#L262-L263
then there will always be an error:
ORA-14452: attempt to create, alter or drop an index on temporary table already in use
logs attached
dbt.log

Current Behavior
The global temporary table is not deleted after running the model due to an

Hi @AlexMorti, you did not finish this sentence regarding the current behavior.
Are you getting an error, if yes which one?
I had the same issue that the temporary table was not dropped caused by the following issue #8 as the incremental materialization failed due to the ORA errors mentioned in the ticket.

@AlexMorti
Thanks for attaching the logs. I am checking this

Hi @AlexMorti
Could you run dbt without any code changes for an incremental model and share the log ? I would like to understand the ORA errors you get without any code changes.
Thank you

Hi @AlexMorti
I could reproduce the issue and confirm that O$PT tables are only truncated at the end of the dbt session but not dropped. This will be fixed using a TRUNCATE statement before DROPPING the table as you proposed.
Thank you.


feat: adding aria-label alias for accessibilityLabel

Summary

Adding aria-label alias for accessibilityLabel

https://github.com/facebook/react-native/issues/34424
Changelog
[General] [Added] - Add aria-label prop to Button, View, Pressable component
Test Plan
<Button
     onPress={() => onButtonPress('cancelled')}
     testID="cancel_button"
     color={theme.SystemRedColor}
     title="Cancel Application"
     aria-lable="Press to cancel your application!"
/>

Hi @necolas, I Just Wanted to know if this is the right way. I will add other components if it's good to go. Open to suggestions.

Hi @Viraj-10, could you rebase your branch, please?

Hi @Viraj-10, could you rebase your branch, please?

@cipolleschi, okay

@Viraj-10 I have to ask if you can rebase this PR again... Also thanks for applying the suggestions!

@Viraj-10 I have to ask if you can rebase this PR again... Also thanks for applying the suggestions!

Sure, No worries.

There are some errors in CI, could you have a look at them, please?

Hi @Viraj-10, there are still errors. You can see them here: https://app.circleci.com/pipelines/github/facebook/react-native/15336/workflows/555b206f-388d-4dbd-868e-4727297563be/jobs/283063
An example of those is:
/home/circleci/react-native/Libraries/Components/Pressable/Pressable.js
  226:5  error  Duplicate key 'accessibilityLabel'  no-dupe-keys
  227:5  error  Duplicate key 'accessibilityState'  no-dupe-keys

could you have a look at them?

Hi @cipolleschi @necolas, Sorry for the delay. I guess I have added an alias in all the components possible. Please review Once and let me know.

Hi @cipolleschi, All test cases are passed and I have resolved all the merge conflicts.


Integration Test

Integration test latest uppdate https://github.com/Sneakycloud/Priority-calculator/commit/0bd285cb24a5760c64997eccf0fb8c9bf01f5b93
PR https://github.com/Sneakycloud/Priority-calculator/pull/110


there is no package called ‘topography’

Hi
I'm trying to recreate some of your nice examples but I'm missing the ‘topography’, and cannot seem to find it to install.
Can you please advise me where to find this package?
Regards
Martin

Hi Martin,
Sure thing, here is the repo: https://github.com/hypertidy/topography
Let me know how you get on!
Cheers,
Hugh


Cross compile builds fail

When generating binaries to use during the build (build tools), they should be built for the host, not the target.
As of now, the build fails with the execution of tools/depsgen.tmp, which is built for the target.
This hack seems to make it go further:
 --- a/depsgentool.mk
+++ b/depsgentool.mk
@@ -3,6 +3,7 @@ $(call setup-stamp-file,DEPSGENTOOL_STAMP)
 # variables for makelib/build_go_bin.mk
 BGB_PKG_IN_REPO := tools/depsgen
 BGB_BINARY := $(DEPSGENTOOL)
+BGB_ADDITIONAL_GO_ENV := GOARCH="amd64"

 CLEAN_FILES += $(DEPSGENTOOL)

Closing this since we now have a PR that fixes it https://github.com/coreos/rkt/pull/1186.

Usually we leave the issues open and they get closed when the PR with the fix is merged (github does this automatically if your PR includes something like "Fixes #issue_number").
In this way, the issue is closed iff the problem is fixed :)

Fixed via #1186


refact(operator): add validatingwebhookconfig list API in clusterrole

To delete older webhook configuration requried to have
list API allowed.
Signed-off-by: prateekpandey14<EMAIL_ADDRESS>

@vishnuitta openebs.io/version label will be added by code itself