Improve oxc_traverse

Various improvements to make to oxc_traverse:

[x] Implement ScopeFlags (ctx.scope()).
[ ] Reduce the size of ancestor.rs by using generics instead of repetition for *Without* types.
[ ] Add TraverseCtx::has_ancestor method where closure returns true or false
(or maybe Some(true), Some(false) or None)
[x] Check that TraverseCtx::find_ancestor can return an ancestor node that it found.
[ ] Set enum discriminants for Ancestor so that is_* can use bitwise ops + retag_stack can write 1 byte only.

Compiler does not auto-optimize matches!() in is_* functions. Need to do it manually.
See ancestor_type2 branch (on my local machine).
Need to handle big endian systems where bytes are in reverse order.

That's easy, but how to run tests on big endian? Miri?

[ ] Implement Debug for Ancestor and *Without* types

Implement intermediate as_ref method which creates a struct of references, which Debug can use.

[ ] Improve API for Ancestor::is_via_*
[ ] API to read siblings in a Vec.
[ ] API to get which index current node is in a Vec.
[ ] API to allow mutating other branches of AST

I think all that's required is to:

Pass &mut TraverseCtx to enter_* and exit_*
Add parent_mut, ancestor_mut methods to TraverseCtx
Add span_mut, directives_mut etc to all *Without* types

Mutable borrow on TraverseCtx and Ancestor prevents creating more than 1 mut ref at a time
Mutable borrow on TraverseCtx unfortunately also blocks calling ctx.alloc().

Can solve that with e.g. ctx.ancestry.parent_mut() + ctx.ast.alloc() - separate properties
can be mut borrowed at same time.

[ ] Move Span to constant position in all structs, for fast lookup?

We've hit a case where APIs to read siblings in a Vec and to get which index current node is in a Vec would be useful.
#4725 needs to check if true is in
Object.defineProperty(exports, 'Foo', {
  enumerable: true,
  get: function() { return Foo_1.Foo; }
});

Currently this is implemented by checking when entering a CallExpression whether it matches this pattern and setting a flag if so. It would be preferable to do this check by reading up AST in enter_boolean_literal instead.
fn compress_boolean(&mut self, expr: &mut Expression<'a>, ctx: &mut TraverseCtx<'a>) -> bool {
    let Expression::BooleanLiteral(lit) = expr else { return false };
    if Self:: bool_is_in_define_export(lit, ctx) { return false; }

    let num = self.ast.expression_numeric_literal(
        SPAN,
        if lit.value { 0.0 } else { 1.0 },
        if lit.value { "0" } else { "1" },
        NumberBase::Decimal,
    );
    *expr = self.ast.expression_unary(SPAN, UnaryOperator::LogicalNot, num);
    true
}

/// Check if `BooleanLiteral` is `true` in:
/// ```js
/// Object.defineProperty(exports, 'Foo', {
///   enumerable: true,
///   get: function() { return Foo_1.Foo; }
/// });
/// ```
fn bool_is_in_define_export(lit: &BooleanLiteral, ctx: &mut TraverseCtx<'a>) -> bool {
    // We are only interested in `true`
    if !lit.value { return false; }
    // We are only interested in this construct at top level
    if ctx.ancestors_depth() != 5 { return false; } // TODO: Is 5 correct?

    // Check `true` is in an object property `enumerable: true`
    let Ancestor::ObjectPropertyValue(prop) = ctx.parent() else { return false; };
    let PropertyKey::StaticIdentifier(key) = prop.key() else { return false; };
    if key.name != "enumerable" { return false; }

    // Check that object is param of a call expression.
    // Skip over checking parent's parent - it must be an `ObjectExpression`.
    let object_parent = ctx.ancestor(3).unwrap();
    let Ancestor::CallExpressionArguments(call_expr) = object_parent else { return false; };

    // We want to check that:
    // 1. First arg is `exports`.
    // 2. Object is 3rd arg.
    // But we can't because `Ancestor` does not give us access to `arguments`.
    // TODO(@overlookmotel): Make this possible.

    // Check callee is `Object.defineProperty`
    // Use tighter check than `call_expr.callee.is_specific_member_access("Object", "defineProperty")`
    // because we're looking for `Object.defineProperty` specifically, not e.g. `Object['defineProperty']`
    if let Expression::StaticMemberExpression(callee) = call_expr.callee() {
        if let Expression::Identifier(id) = &callee.object {
            if id.name == "Object" && callee.property.name == "defineProperty" {
                return true;
            }
        }
    }
    false
}

This isn't currently possible as cannot access call_expr.arguments. It would be safe to access other elements in arguments except for the current one, but this is not supported currently by Traverse's API.


Add SRR support by passing getServerSnapshot param

Fixes https://github.com/zoontek/react-atomic-state/issues/12

Thanks, I will publish a new version today!


Running the Rust NFT example contract throws errors

I am on Ubuntu and have Rust already installed.
Here are the steps I followed:

Cloning the repository
Going into the rust folder
Following the README.md
running npm run build:rs throws the following errors

I then went to the prerequisites as detailed here but they are for when Rust is not installed so not helpful in this particular case

@janedegtiareva told me to run the following: rustup target add wasm32-unknown-unknown then running npm run build:rs worked like expected.

Circling back, need to add this link:
https://docs.near.org/docs/tutorials/intro-to-rust#1-install-rustup

As solution has been shared, and building contract is now performed without npm, I will proceed closing this issue


Refactoring of blending classes into a module and separating out the …

…Triangular Blending across adjacent points
#189
Refactoring ahead of adding new functionality.
Testing:

[x] Ran tests and they passed OK
[ ] Added new tests for the new feature(s) - N/A

Happy to get feedback on names of files.


Range to until

What's done:

[x] Implemented Rule, added Warnings
[x] Added tests on checks
[x] Added tests on fixers
[x] Updated diktat-analysis.yml
[x] Updated available-rules.md
[x] Closes #979

will be merged after 1.0.0 release

Also we need to keep in mind the following:
https://detekt.github.io/detekt/performance.html#foreachonrange


Add possibility to Bypass Emote restriction

I have a few server where you can post images but you cannot send emote from outside server.
could it be possible to bypass the lock ?
Or do you wish to keep it like this ?

Done in 1.13.0


[AIRFLOW-1158] Increment multiplart s3 upload chunks by 1

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

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

https://issues.apache.org/jira/browse/AIRFLOW-1158

Description

[x] Here are some details about my PR, including screenshots of any UI changes:
When I try to upload a file of say 104MBs, using multipart uploads of 10MB chunks, I get 10 chunks of 10MBs and that's it. The 4MBs left over do not get uploaded.

Tests

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

Commits

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

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

Codecov Report

Merging #2263 into master will decrease coverage by 0.03%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2263      +/-   ##
==========================================
- Coverage   68.88%   68.84%   -0.04%
==========================================
  Files         142      142
  Lines       10966    10966
==========================================
- Hits         7554     7550       -4
- Misses       3412     3416       +4

Impacted Files
Coverage Δ

airflow/hooks/S3_hook.py
22.27% <0%> (ø)
:arrow_up:

airflow/jobs.py
74% <0%> (-0.41%)
:arrow_down:

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 4e79b83...cd57a02. Read the comment docs.

Something is out of whack here.  Closing this for now, will reopen after resolution.


[feature] Researchers database

position_th: อ./ผศ./...
position_en: -/Assoc. Prof./...
title: ดร./นพ./พญ.
suffix: Ph.D./M.D.
fullname_th
fullname_en
departmentId
divisionId แต่ละ department จะมี division ย่อย
area_of_interests: (multiple)
keywords: (multiple)
emails: (multiple)
links: (multiple -> title, link) เช่น [{ title: Scopus, link: www.scopus...com }]
image_url

ตัวอย่าง create (POST)
{
  "position_th": "c",
  "position_en": "d",
  "title": "k",
  "suffix": "l",
  "fullname_th": "l",
  "fullname_en": "l",
  "published_at": "2021-02-02T17:04:30.250Z",
  "created_at": "2021-02-02T17:03:57.724Z",
  "updated_at": "2021-02-02T17:04:30.283Z",
  "emails": [
    {
      "email"<EMAIL_ADDRESS>    }
  ],
  "links": [
    {
      "url": "kk",
      "title": "kk"
    }
  ],
  "area_of_interests": [],
  "keywords": [2]
}

CREATE API
Area of Interests
fetch("https://mdcuresearchclub-strapi.docchula.com/aois", {
  method: "POST",
  body: JSON.stringify({"title":"pharmacology"}),
  headers: {
    "Authorization": "Bearer TOKEN",
    "Content-Type": "application/json; charset=utf-8"
  },
})

Keywords
fetch("https://mdcuresearchclub-strapi.docchula.com/keywords", {
  method: "POST",
  body: JSON.stringify({"title":"pharmacology"}),
  headers: {
    "Authorization": "Bearer TOKEN",
    "Content-Type": "application/json; charset=utf-8"
  },
})

Researchers
fetch("https://mdcuresearchclub-strapi.docchula.com/researchers", {
  method: "POST",
  body: JSON.stringify({
  "position_th": "Position_th",
  "keywords": [
    1,
    3
  ],
  "fullname_en": "Fullname_en",
  "suffix": "Suffix",
  "area_of_interests": [
    1,
    3,
    2
  ],
  "emails": [
    {
      "email"<EMAIL_ADDRESS>    }
  ],
  "title": "Title",
  "fullname_th": "Fullname_th",
  "links": [
    {
      "title": "Title",
      "url": "Title.com"
    }
  ],
  "position_en": "Position_en"
}),
  headers: {
    "Authorization": "Bearer TOKEN",
    "Content-Type": "application/json; charset=utf-8"
  },
})

CREATE API
Departments
fetch("https://mdcuresearchclub-strapi.docchula.com/departments", {
  method: "POST",
  body: JSON.stringify({title: "Microbiology"}),
  headers: {
    "Authorization": "Bearer TOKEN",
    "Content-Type": "application/json; charset=utf-8"
  },
})

Divisions
fetch("https://mdcuresearchclub-strapi.docchula.com/divisions", {
  method: "POST",
  body: JSON.stringify({title: "Immunology"}),
  headers: {
    "Authorization": "Bearer TOKEN",
    "Content-Type": "application/json; charset=utf-8"
  },
})

Area of Interests
fetch("https://mdcuresearchclub-strapi.docchula.com/aois", {
  method: "POST",
  body: JSON.stringify({"title":"pharmacology"}),
  headers: {
    "Authorization": "Bearer TOKEN",
    "Content-Type": "application/json; charset=utf-8"
  },
})

Keywords
fetch("https://mdcuresearchclub-strapi.docchula.com/keywords", {
  method: "POST",
  body: JSON.stringify({"title":"pharmacology"}),
  headers: {
    "Authorization": "Bearer TOKEN",
    "Content-Type": "application/json; charset=utf-8"
  },
})

Researchers
fetch("https://mdcuresearchclub-strapi.docchula.com/researchers", {
  method: "POST",
  body: JSON.stringify({
  "position_th": "Position_th",
  "division": 1,
  "keywords": [
    1,
    3
  ],
  "fullname_en": "Fullname_en",
  "suffix": "Suffix",
  "area_of_interests": [
    1,
    3,
    2
  ],
  "emails": [
    {
      "email"<EMAIL_ADDRESS>    }
  ],
  "title": "Title",
  "fullname_th": "Fullname_th",
  "links": [
    {
      "title": "Title",
      "url": "Title.com"
    }
  ],
  "position_en": "Position_en"
}),
  headers: {
    "Authorization": "Bearer TOKEN",
    "Content-Type": "application/json; charset=utf-8"
  },
})


Adding list of rotation CA Support

Adding list of rotation CA Support

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

@PPacentMSFT : Thanks for your contribution! The author, @mattbriggs, has been notified to review your proposed change.

Hi Matt this looks good. Please include “1803 & Later” for the From Enterprise to Enterprise Row. Other than that it is good to publish
Thanks,
Peter
From: Matt Briggs<EMAIL_ADDRESS>Sent: Tuesday, May 15, 2018 11:53 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Peter Pacent<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Adding list of rotation CA Support (#8368)
Importance: High
https://review.docs.microsoft.com/en-us/azure/azure-stack/azure-stack-rotate-secrets?branch=pr-en-us-41238https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Freview.docs.microsoft.com%2Fen-us%2Fazure%2Fazure-stack%2Fazure-stack-rotate-secrets%3Fbranch%3Dpr-en-us-41238&data=02|01|ppacent%40microsoft.com|31dbf1da8c534ca79c7a08d5ba950a65|72f988bf86f141af91ab2d7cd011db47|1|0|636620071649699972&sdata=g9d%2BgA9CXniK%2FjJRq9GnVP8C54n5S8a6kOzs7AmoMtA%3D&reserved=0
#Please-close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F8368%23issuecomment-389274711&data=02|01|ppacent%40microsoft.com|31dbf1da8c534ca79c7a08d5ba950a65|72f988bf86f141af91ab2d7cd011db47|1|0|636620071649709977&sdata=%2FzE69lmoV9sfP0UrPl95vc8dmI804cmJf3SsxciXOTw%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAWMFu4AjsEYmtnalfG8qlAqqV1Dd_6EZks5tyyP6gaJpZM4T4uPc&data=02|01|ppacent%40microsoft.com|31dbf1da8c534ca79c7a08d5ba950a65|72f988bf86f141af91ab2d7cd011db47|1|0|636620071649709977&sdata=jzONlIRLhOx3ChpGdyotNsk%2BVuQ33RViL%2BT%2FncJADOI%3D&reserved=0.


String TLV Attribute max size should include length byte(s) (CON-1070)

Fixes #857

@JonathanWitthoeft Thanks for the PR, we had identified this sometime back and wanted to fix it in a different way. We will push it out soon.

@shubhamdp Any idea on a time frame? Will this fix be back-ported to release 1.2?

@shubhamdp Any idea on a time frame? Will this fix be back-ported to release 1.2?

Yes, this will be backported to v1.2 and v1.1 branches.

@JonathanWitthoeft the fix is out, PTAL https://github.com/espressif/esp-matter/commit/d481fa9cf1d09001b1bbd9cd59dbb7836bbb13fe

@JonathanWitthoeft Fix for v1.2: 19f4775ff3cbf41305303f48114c1ee1410f3cbc. Closing this.


Getting security error.

Here is the error, How can I fix this

Hey! That seems to be an error with the interpolated URL inside src parameter.
Anyway, I have just released a new version with Angular 11 support, please upgrade your ngx-lazy-load-images dependency in package.json to ^2.0.0.
If you encounter any problems while using it, please open a new issue.


expr: Support json append replace

What is changed and how it works?

Issue Number: Close  #50601

What's Changed:

Related changes

[x] PR to update pingcap/docs/pingcap/docs-cn:
[] Need to cherry-pick to the release branch

Check List
Tests

[x] Unit test
[ ] Integration test
[x] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

/test


Cosmetic/pretty printing of compiler output

Fixes #144. (edit: Now with circuit drawings.) E.g.,
import qiskit_superstaq as qss
import qiskit
qc = qiskit.QuantumCircuit(1)
qc.h(0)
qc.draw()

provider = qss.SuperstaqProvider()
compile_result = provider.ibmq_compile(qc, target="ibmq_lagos_qpu")
print(compile_result.__repr_pretty__())
# CompilerOutput(
#          ┌─────────┐┌─────────┐┌─────────┐
#     q_0: ┤ Rz(π/2) ├┤ Rx(π/2) ├┤ Rz(π/2) ├
#          └─────────┘└─────────┘└─────────┘,
#     {0: 0},
#     Schedule(
#         (0, ShiftPhase(-1.5707963268, DriveChannel(0))),
#         (0, ShiftPhase(-1.5707963268, ControlChannel(1))),
#         (
#             0,
#             Play(
#                 Drag(
#                     duration=160,
#                     sigma=40,
#                     beta=-1.053810766518306,
#                     amp=0.1987049754148708,
#                     angle=0.022620855474399307,
#                     name="X90p_d0",
#                 ),
#                 DriveChannel(0),
#                 name="X90p_d0",
#             ),
#         ),
#         (160, ShiftPhase(-1.5707963268, DriveChannel(0))),
#         (160, ShiftPhase(-1.5707963268, ControlChannel(1))),
#         name="sched25",
#     ),
#     None,
#     None,
#     None,
# )

I bet preprocess_compiler_output and postprocess_compiler_output can be replaced by clever regex -- I would be glad to nuke as much of it as possible, but wanted to get something in since I've been dragging my feet.

black needs to be moved from dev-requirements to requirements

black needs to be moved from dev-requirements to requirements

Done.

Low pri and not enough bandwidth. Closing.


Imager can’t manipulate remote images stored as variables

I have a client project that is trying to grab various YouTube videos from their channel. Most have a high res thumbnail, but a few do not. I was attempting to find a middle ground by using the medium size thumbnail from YouTube and crop them, but it appears that a remote file that is stored as a variable is inaccessible to the plugin:
{% set externalImage = video.video_mediumres %}

{% set transformedImage = craft.imager.transformImage(externalImage, {
    width: 480,
    height: 270,
    mode: 'crop' }) %}

This results in this error:

Internal Server Error
An unknown image object was used.

However, using the value that’s stored in video.video_mediumres, which is https://i.ytimg.com/vi/axBJlbq3m4o/hqdefault.jpg, instead of actually using video.video_mediumres causes the plugin to function as expected.
Is there any way to manipulate these remote images?

What kind of object is externalImage? If you do {{ dump(externalImage) }}, what does it say? Is it a string or some kind of other object?

Ah, it’s indeed “some kind of other object.”
object(SimpleXMLElement)[655]
  public 0 => string 'https://i.ytimg.com/vi/axBJlbq3m4o/hqdefault.jpg' (length=48)

Right, and Imager just takes AssetFileModels and strings. Think you can "cast" it to string with something like {{ externalImage ~ '' }} or maybe {{ externalImage | t }}.

That did it! Thanks for the explanation.


handle exponential floats

Fix for #15
Hope it won't affect performance significantly

Good catch! However, I don't think we can regex evaluate all numbers to cover the scientific notation case. Is there another way to distinguish them?

I haven't done any benchmarks, but i think it's valid concern...
According to this reference

All numbers in Javascript are 64bit (8 bytes) floating point numbers which yields an effective range of 5e-324 (negative) to 1.7976931348623157e+308 (positive) at the time this article was written (this may eventually change to 128 bits in the future as 64 bit processors become commonplace and the ECMA standards evolve).

JS numbers seems to be platform agnostic, but i wouldn't rely on internal Number implementation because it may change.
There is Number.prototype.toExponential method that seems to return the same result as toString for exponential numbers
So using something kinda if (Number.isInteger(obj) && obj.toString() !== obj.toExponential()) seems reasonable. I'll update the PR

added few optimizations

any more concerns here?


Cloning entity with non initialized collection

Describe the bug
Atm it is not possible to simply clone (lodash clone) entity which has not initialized collection (for example entity loaded without population). I know entity should not be cloned when you deal with ORM. However in this case I want to serialize entity to send it as plain json so I want operate on clone - not original entity. My serialization process is a way complicated so I can't use toJSON or any other solution.
To fix this I used more complicated cloning process - first detect collection / entity with cloneWith and then create new instance of Collection with original collection items
Stack trace
Error: Collection<AssetPrice> of entity Asset[4] not initialized
    at Collection.checkInitialized (.../api/node_modules/@mikro-orm/core/entity/Collection.js:289:19)
    at Collection.count (.../api/node_modules/@mikro-orm/core/entity/Collection.js:141:14)
    at Collection.get length [as length] (.../api/node_modules/@mikro-orm/core/entity/ArrayCollection.js:139:21)
    at isArrayLike (.../api/node_modules/lodash/isArrayLike.js:30:42)
    at keys (.../api/node_modules/lodash/keys.js:34:10)
    at baseAssign (.../api/node_modules/lodash/_baseAssign.js:14:39)
    at baseClone (.../api/node_modules/lodash/_baseClone.js:123:32)
    at clone (.../api/node_modules/lodash/clone.js:33:10)

To Reproduce
Steps to reproduce the behavior:

load entity from db with collection not populated
clone entity with lodash/clone

Expected behavior
I think getter from ArrayCollection get length() should not trigger errors.
So I suggest to add check = true to Collection.count() the same as getItems(check = true) and use version without check in getter.
Versions

Dependency
Version

node
16.15

typescript
4.7.4

mikro-orm
5.3.0

your-driver
mysql

I dont really like this idea, and I would first like to hear why you think you cant use toJSON or similar. Converting to POJO first is the right approach, not removing invariants from the ORM and making it less safe/correct.

So I suggest to add check = true to Collection.count() the same as getItems(check = true) and use version without check in getter.

What you suggest is changing behaviours, you want to have the default set to false unlike with getItems().
And more importantly, why would you want to clone an entity is my main question. If you dont want to track changes on it, you should fetch it via a separate fork, or with disableIdentityMap: true.

To fix this I used more complicated cloning process - first detect collection / entity with cloneWith and then create new instance of Collection with original collection items

You might easiily fall into issues with persistence with such collections, if you break the internal state, or worse, if you let one entity be managed by multiple EMs.

In case of simple objects (without nested entities) toJSON is good enough. However sometimes even is such case it would be better to have control over that process - for example some properties may require additional processing when serializing / deserializating.
But when I have nested entities I don't want to get final JSON in one call - cuz I may want to serialize objects on lower levels in different way. For example - I may want to convert collection of entries to collection of ID strings (like "@/asset/23"). That process depends on object constructor type so I don't want to lose it after toJSON call.
I want to traverse entire nested object tree and process it manually.
For that purpose I want clone of EM managed object which will not be managed (EM lose track of it) anymore but keeps all typings (class constructors).
Check class-transformer for example. My serializer is similar to that library, however it also doesn't provide enough control over process and that is why I implemented my own.
Like I wrote before I fully understand clone is not adviced when one deal with managed objects and want to use cloned object later as ORM entity. But in my case cloning is done to:

lose reference to managed object
keep object typing

If propsed adjustment breaks something - or even you only think it may break something - that is ok for me, I will preserve special clone version for ORM entities.

However sometimes even is such case it would be better to have control over that process - for example some properties may require additional processing when serializing / deserializating.

You have toPOJO for cloning purposes, it ignores populate hints. You also have property serializers.
Anyway, what about using https://lodash.com/docs/#cloneDeepWith instead, and handling not initialized collections that way?
Because such collections shoudn't be mapped to empty arrays in the first places. Uninitialized collection is not the same as empty one. You should probably skip such fields instead of turning them to empty arrays.

Because such collections shoudn't be mapped to empty arrays in the first places. Uninitialized collection is not the same as empty one. You should probably skip such fields instead of turning them to empty arrays.

That makes sense. But as long as this behavior is described it is not so bad to return empty array for non initialized collection.
cloneWith is just enough if I pass proper handling function - I have it already working.
My suggestion was only to simplify it so clone would also work. I thought it will not be large change.
If you meant
count() {
    return this.isInitialized()
        ? super.count()
        : undefined;
}

it works well.

That makes sense. But as long as this behavior is described it is not so bad to return empty array for non initialized collection.

Let me disagree. One of the use cases for serializing entity is caching, with your changes the entity restored from cache would be invalid - it would suddently have initialized empty collections, instead of uninitialized ones. Their state would change. This is the part I dont like, this is why I even consider it breaking.

it works well.

So it results in the property being omitted or it just produces empty array regardless of the length value?
I'll be happy to add some config toggle to allow this, but making it default behaviour seems dangerous/wrong. And I would much rather return undefined in there, especially if it would mean the collection property will be omitted from the cloned result.

add some config toggle to allow this

Ouu in that case it is overkill. I will stay with custom cloning for ORM entities.
I will close this. Thx for responsiveness.


Central SSL is not supported without a hostname

win-acme runs successfully here with command line options; certificates are generated and stored in the Central Certificate Store as intended. However, part of the auto-renewal task fails - the error in the subject line of this message appears in the log file:
*[ERR] Error updating binding :443:...Central SSL is not supported without a hostname

The friendly name of the certificate IS a hostname, in fact the hostname of the machine on which win-acme and the renewal task are being run, and the server on which Exchange server is installed to which the certificate is to be bound to IIS. (Everything relevant is on this server only). I thought that is what it means "with a hostname"; and I believe that when I ran wacs with equivalent proper command line options, the certificate was properly generated, stored in the Central Store, and bound to IIS with no errors.

The log says: [INF] Removing certificate from the Central SSL store; and
[ERR] Renewal for [Manual] servername.prefix.domain.com failed, will retry on next run; however the updated certificate appears to have been generated and properly stored in the Central Certificate Store; there is a PFX file there with a favorable name, size, and timestamp.

Binding of IIS to the generated certificate did fail.

How can I get the auto-renewal to a) for sure save the generated certificate to the Central Certificate Store, b) bind properly to IIS, and c) not generate errors?
Thanks.
-Frank

Hi Frank,
The IIS does not support bindings with blank host names that refer to the Central Certificate Store for TLS. You can try to create one for yourself: the dialog will simply disable the Save button until you specify the host name.
Typically the program will match new https bindings that it need to create to existing http bindings. If you're using the CCS and the best matching binding doesn't have a host name specified, it's supposed to recognize that and create a new binding instead.  I'm not able to reproduce this issue with the latest version of the program. If you are using an older release, please try that one first. Otherwise, please provide log output with the --verbose switch on so that we might see what's happening behind the scenes.
The way to work around this is to manually set up a binding, you'd only have to do it once. I'd be helpful for others though if you could help us get to the bottom of this.

I continue to see this error and I have to manually intervene each time I need to fetch a new certificate.
Installation step 1/2: IIS...
 Error updating binding *:443:
 Error installing
 (InvalidOperationException) Unable to install certificate: Central SSL is not supported without a hostname
 Renewal for [Manual] server.fqdn.com failed, will retry on next run
 Install failed: Central SSL is not supported without a hostname

My set up is as follows:
I have an on-prem Exchange server that is bound to mail.domain.com, autodiscover.domain.com, mail.domain2.com, autodiscover.domain.com, etc. Additionally, it is bound to *:443 such that requests without SNI and just hitting port :443 would receive a certificate error, but the same Exchange virtual host would still service requests.
How can I have a single virtual host pull certificates from the central certificate store AND still have a certificate for non-SNI requests?

Do you require requests without SNI - are you supporting Windows XP?


Clarify the BYOT scope and next action items

We decided to go with an implementation of a simple stake/unstake functionality that we can include on the HookedTokenManager. A first draft was implemented here: https://github.com/1Hive/gardens/blob/gardens-token-generator/packages/hardhat/contracts/GardenTokenGenerator.sol


test: test root user setup

A continuation of https://github.com/osbuild/bootc-image-builder/pull/357 - the root user is used to do a bootc status inside the booted container to ensure the setup is correct. This is nicer then the previous sudo approach and it has the added benefit of testing the root user creation/setup during the build too.

Almost merged. Missed this fixup commit.

I removed the fixup commit now and fixed the bug in the PR that prevented the tests from working - it turns out we wrote invalid kickstart files because the keyfile read from disk contained a (not stripped) newline at the end and with the kickstart file looked somehting like:
sshkey --username foo "ssh-key
"

I opened https://github.com/osbuild/osbuild/pull/1772 with one idea


Update dashboard docs for queuing

Closes https://github.com/dask/distributed/issues/7306.
Also some driveby fixes of typos/small inaccuracies in the docs (not exhaustive).

[x] Tests added / passed
[x] Passes pre-commit run --all-files

https://dask--9660.org.readthedocs.build/en/9660/dashboard.html#progress

WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f26ec47bbe0>: Failed to establish a new connection: [Errno 101] Network is unreachable')': /packages/b2/6c/9ccb5213a3d9fd3f8c0fd69d207951901eaef86b7a1a69bcc478364d3072/pre_commit-2.20.0-py2.py3-none-any.whl
  WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f26ec47beb0>: Failed to establish a new connection: [Errno 101] Network is unreachable')': /packages/b2/6c/9ccb5213a3d9fd3f8c0fd69d207951901eaef86b7a1a69bcc478364d3072/pre_commit-2.20.0-py2.py3-none-any.whl
  WARNING: Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f26ec47bf70>: Failed to establish a new connection: [Errno 101] Network is unreachable')': /packages/b2/6c/9ccb5213a3d9fd3f8c0fd69d207951901eaef86b7a1a69bcc478364d3072/pre_commit-2.20.0-py2.py3-none-any.whl
  WARNING: Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f26ec4a4040>: Failed to establish a new connection: [Errno 101] Network is unreachable')': /packages/b2/6c/9ccb5213a3d9fd3f8c0fd69d207951901eaef86b7a1a69bcc478364d3072/pre_commit-2.20.0-py2.py3-none-any.whl
  WARNING: Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f[26](https://github.com/dask/dask/actions/runs/3465838377/jobs/5789036313#step:4:28)ec4a42b0>: Failed to establish a new connection: [Errno 101] Network is unreachable')': /packages/b2/6c/9ccb5213a3d9fd3f8c0fd69d207951901eaef86b7a1a69bcc478364d3072/pre_commit-2.20.0-py2.py3-none-any.whl
ERROR: Could not install packages due to an OSError: HTTPSConnectionPool(host='files.pythonhosted.org', port=443): Max retries exceeded with url: /packages/b2/6c/9ccb5213a3d9fd3f8c0fd69d207951901eaef86b7a1a69bcc478364d3072/pre_commit-2.20.0-py2.py3-none-any.whl (Caused by NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f26ec4a4460>: Failed to establish a new connection: [Errno 101] Network is unreachable'))

for linting: some transient pip stuff setting up pre-commit

Yep, looks like a connection blip -- restarting that CI build

@gjoseph92 Thanks for this, will merge this in!


Storm control with port-channels

Issue Type

Bug Report
Documentation Issue Report
EOS Configuration error

Summary
When we have port-channels and we call port profiles with storm control, it doesn't take into account the structured config and add it under port-channel.
It works well with physical interfaces.
  Ethernet5:
    peer: server03
    peer_interface: Eth1
    peer_type: server
    description: server03_Eth1
    type: switched
    shutdown: false
    mode: trunk
    vlans: 210-211
    storm_control:
      broadcast:
        level: 10
      multicast:
        level: 20
      unknown_unicast:
        level: 15
    channel_group:
      id: 5
      mode: active
    profile: TENANT_B

interface Port-Channel5
   description server03_PortChannel1
   no shutdown
   switchport
   switchport trunk allowed vlan 210-211
   switchport mode trunk
   mlag 5

Working example
  Ethernet4:
    peer: server02
    peer_interface: Eth0
    peer_type: server
    description: server02_Eth0
    type: switched
    shutdown: false
    mode: trunk
    vlans: 210-211
    storm_control:
      broadcast:
        level: 10
      multicast:
        level: 20
      unknown_unicast:
        level: 15
    profile: TENANT_B

interface Ethernet4
   description server02_Eth0
   no shutdown
   switchport
   switchport trunk allowed vlan 210-211
   switchport mode trunk
   storm-control broadcast level 10
   storm-control multicast level 20
   storm-control unknown-unicast level 15

Role or Module Name

OS / Environment
EOS Version

Cloudvision version (optional)

OS running Ansible

Steps to reproduce

Expected results

$

Actual results

Fixed by PR #615

Fixed by PR #615


Occasional randomized test failure in CI

The indexing unit test test_scatter_gather_tensor fails occasionally in CI. It's not a big deal since it usually passes after re-running the CI build, but we investigate and should make sure there's not an underlying bug in scatter and gather causing the failures.
Example of failing CI run with test and error message below: https://github.com/BasisResearch/chirho/actions/runs/6443034118/job/17494507123?pr=241
=================================== FAILURES ===================================
________ test_scatter_gather_tensor[True-(2,)-(2,)-(2, 2, 3)-(2, 3, 2)] ________

enum_shape = (2,), plate_shape = (2,), batch_shape = (2, 2, 3)
event_shape = (2, 3, 2), use_effect = True

    @pytest.mark.parametrize(
        "enum_shape,plate_shape,batch_shape,event_shape", SHAPE_CASES, ids=str
    )
    @pytest.mark.parametrize("use_effect", [True, False])
    def test_scatter_gather_tensor(
        enum_shape, plate_shape, batch_shape, event_shape, use_effect
    ):
        cf_dim = -1 - len(plate_shape)
        name_to_dim = {f"dim_{i}": cf_dim - i for i in range(len(batch_shape))}

        full_batch_shape = enum_shape + batch_shape + plate_shape
        value = torch.randn(full_batch_shape + event_shape)

        world = IndexSet(
            **{
                name: {full_batch_shape[dim] - 2}
                for name, dim in name_to_dim.items()
                if full_batch_shape[dim] > 1
            }
        )

        with contextlib.ExitStack() as stack:
            if use_effect:
                stack.enter_context(IndexPlatesMessenger(cf_dim))
                for name, dim in name_to_dim.items():
                    add_indices(
                        IndexSet(**{name: set(range(max(2, full_batch_shape[dim])))})
                    )
                _name_to_dim = None
            else:
                _name_to_dim = name_to_dim

            actual = gather(
                value, world, event_dim=len(event_shape), name_to_dim=_name_to_dim
            )
            actual = scatter(
                actual,
                world,
                result=value.new_zeros(full_batch_shape + event_shape),
                event_dim=len(event_shape),
                name_to_dim=_name_to_dim,
            )

        mask = indexset_as_mask(
            world,
            event_dim=len(event_shape),
            name_to_dim_size={
                name: (dim, full_batch_shape[dim]) for name, dim in name_to_dim.items()
            },
        )
        _, mask = torch.broadcast_tensors(value, mask)

        expected = value
        assert (actual == expected)[mask].all()
>       assert not (actual == expected)[~mask].any()
E       assert not tensor(True)
E        +  where tensor(True) = <built-in method any of Tensor object at 0x7f4afa5f6f40>()
E        +    where <built-in method any of Tensor object at 0x7f4afa5f6f40> = tensor([False, False, False, False, False, False, False, False, False, False,\n        False, False, False, False, Fals...alse, False, False, False, False, False, False, False,\n        False, False, False, False, False, False, False, False]).any

tests/indexed/test_internals.py:2[85](https://github.com/BasisResearch/chirho/actions/runs/6443034118/job/17494507123?pr=241#step:7:86): AssertionError

value is random:
value = torch.randn(full_batch_shape + event_shape)


hello

Windows こんにちは for Business が散見されます。
Windows Hello for Business に統一して下さい。

ドキュメントの詳細
⚠ このセクションを編集しないでください。 docs.microsoft.com で必須です ➟ GitHub の問題のリンク。

ID: 13c400ea-f52d-b45b-a871-31c94262084d
Version Independent ID: 9465e043-990f-cb8b-2d4b-33dc8d3c21bf
Content: Active Directory の前提条件の検証 (Windows Hello for Business)
Content Source: windows/security/identity-protection/hello-for-business/hello-cert-trust-validate-ad-prereq.md
Product: w10
GitHub Login: @MikeStephens-MS
Microsoft Alias: mstephen

@samanthaw @MollyMalone
cc: @AndreaBarr

@shiomr : Unfortunately, I can't read kanji or speak Japanese, so I put some of the issue text into Google Translate. This is the output:

Windows Hello for Business will be seen here and there.
Please unify with Windows Hello for Business.
Document Details
⚠ _ Do not edit this section. Required at docs.microsoft.com リンク Link of GitHub problem. _

I would like to mention what @DaniHalfin commented in a previous issue concerning Japanese content, https://github.com/MicrosoftDocs/windows-itpro-docs/issues/2761#issuecomment-466838921

Our localized content doesn't offer the ability to directly contribute [ ... ]

If you are looking for ways to contribute to the content of a localized language page, the Microsoft Docs team will have to forward your request for changes and improvements to other Microsoft technicians.
(You may want to add more details about your issue so the MS techs will fully understand what the issue is about.)

@AndreaBarr - This issue can be closed. Thank you.


Refactor puts_columns to allow $stderr.puts_columns.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew tests with your changes locally?

puts_columns currently only outputs to $stdout. This change makes it an instance method of the IO class, so it can be used with any IO object.
Also added Kernel::puts_columns which outputs to $stdout by default to be in line with Kernel::puts.
Also modified it so it doesn't output trailing whitespace anymore.
--
Fixes https://github.com/caskroom/homebrew-cask/issues/24642.

@reitermarkus Thoughts on this?

@MikeMcQuaid, I moved the columns method into the existing file but left the test as-is, because we will need to merge the test suites anyways at some point.

@reitermarkus As the tests are already written and working I'll make an exception this time but definitely don't want to be adding any more Homebrew functionality tests primary in Cask tests in future.

definitely don't want to be adding any more Homebrew functionality tests primary in Cask tests

Then I guess I now have an incentive to merge the test suites. 😄

@reitermarkus Yes 😀


is the connect could be cancel?

i use wisefy to connect wifi, but when i switch wifi quickly and uninterrupted，it got failed,so i think if i can cancel the last connection when i click the new one ?

I'm sorry, but I will not be implementing this.  You may bump down the time to connect if you want to try to switch faster but you should wait for the callback before continuing.


rather than blatting files all over the pwd, write outputs to a specific folder named by the job

currently bacalhau docker run --download or bacalhau get blat the files (shards, stderr, stdout, volumes) all over the current working directory, which, say, if you do it in a directory with existing files in, makes it hard to see what it's done.
Instead, it should write the files to ./bacalhau-job-${job_id} and also tell you in the CLI output what the name of that folder is
this will improve the UX and make it less confusing when downloading the results of a second job also

Hang on, I think it already does that:
cid=$(bacalhau docker run ubuntu echo hello)
sleep 5
bacalhau get $cid
cid=$(bacalhau docker run ubuntu echo hello)
sleep 5
bacalhau get $cid

Results in:
❯ tree -L 1 shards
shards
├── job-071c7cb6-100a-47d6-8ef9-c121a53487d8-shard-0-host-QmYgxZiySj3MRkwLSL4X2MF5F9f2PMhAE3LV49XkfNL1o3
└── job-797c3b86-3ef6-44ac-a5f9-51c3319634bb-shard-0-host-QmdZQ7ZbhnvWY1J12XYKGHApJ6aufKyLNSvf8jZBrBaAVL

I.e. the job ID is already included in the directory name.
Also the log messages say:
11:52:12.752 | INF ipfs/downloader.go:195 > Combining shard from output volume 'outputs' to final location: '/Users/phil/source/filecoin-project/docs.bacalhau.org'

So the second request to adding the output to the CLI is valid, if that's what you want.

It shouldn't write shards, stderr, stdout, volumes to the current working directory though. It should put all of the things inside a uniquely named directory.  It's the shards dir that's the issue then.
Can you suggest a preferred format then? Something like:
> tree .
- outputs
| - job-071c7cb6-100a-47d6-8ef9-c121a53487d8
|  | - shard-0-host-QmYgxZiySj3MRkwLSL4X2MF5F9f2PMhAE3LV49XkfNL1o3
|  |  | - stdout
job-071c7cb6-100a-47d6-8ef9-c121a53487d8
├── shard-0-host-QmYgxZiySj3MRkwLSL4X2MF5F9f2PMhAE3LV49XkfNL1o3
│   ├── exitCode
│   ├── stderr
│   └── stdout

BTW in case this issue was unclear, when I say "outputs" I mean all the files that the CLI writes in bacalhau get not just the output volumes.

I've heard feedback that someone broke because we were writing to /tmp and then moving it to the final directory. I have fixed this to just write directly to OutputDir (unless anyone objects)

@aronchick on the CLI side? Please link to your change. Thanks!


Refactor ast

The goal here is to change all recursive structures to have the expression as a template argument. Once we do that, we can much more easily switch out expression under the hood, and for example, add a source reference to the expression or remove enum variants.


build: update the location of gyp

Google moved the repository

It seems to still be at the old place too: https://git.chromium.org/gitweb/?p=external/gyp.git;a=summary where did they say they are moving it permamently?

Ah, I see now, sorry.

Thanks! Landed in 5abb33d and an ammended change to the README.


Fix passing of shell arguments with spaces to WP-CLI

Fixes #158.

@westonruter thanks for tackling this. Could you share an example of what a command with spaces looks like before and after this change? That way I have a record for an eventual unit test.

@bensheldon here are a couple examples that fail without the patch:
$ terminus wp --site=example --env=dev option set foo "bar baz"
Running wp option set foo bar baz  on make-xwp-dev
Error: Too many positional arguments: baz

$ terminus wp --site=example --env=dev user update 1 --display_name="Weston Ruter"
Running wp user update 1 --display_name=Weston Ruter  on make-xwp-dev
Warning: Invalid user ID, email or login: 'Ruter'
Success: Updated user 1.

@westonruter this looks great. Thank you so much for tackling this.


Work on WebAssembly

https://docs.google.com/document/d/131vjr4DH6JFnb-blm_uRdaC0_Nv3OUwjEY5qVCxCup4/edit#

The goal is to upstream it to the official Go repository in the current development cycle, targeting Go 1.11.

On my local machine, I’ve succeeded to compile example/sprites to Wasm https://gyazo.com/9ec8cdfb5ac61ee806d66d9ecbdff0fd
This requires some dirty hack and I can't upload this to master branch yet.

Now Ebiten works with WebAssembly port, except for audio :-)

https://github.com/hajimehoshi/oto/commit/e9c8c6db1bc1e2c85f9bf960537761d626a7b9ee Finished!!


Self generated elements

Hello,
I am working with BAM and BPM domains and after adjusting my models it works. However, I can see these elements.

These elements are not part of my model and I know I can delete it from them by using
    JMSServer:
        # TO DELETE JMS SERVERS CREATED BY DEFAULT
        '!BamReportCacheJmsServer_auto_1':
        '!BamReportCacheJmsServer_auto_2':
        '!BamPersistenceJmsServer_auto_1':
        '!BamPersistenceJmsServer_auto_2':
        '!BamAlertEngineJmsServer_auto_1':
        '!BamAlertEngineJmsServer_auto_2':

However, suffixes  such as auto_[9,10,11,12,13] look totally random for my domain whose BAM cluster only has two servers.
Is there any way to delete elements by using wildcards or regular expressions? For example,
    JMSServer:
        # TO DELETE JMS SERVERS CREATED BY DEFAULT
        '!BamReportCacheJmsServer_auto_*':
        '!BamPersistenceJmsServer_auto_*':
        '!BamAlertEngineJmsServer_auto_*':

Best regards,
TAM

To be reviewed for enhancement

Hello,
Here you can see an example of the element that are generated in my domain. I have two managed servers in a BamCluster and then I do not understand how these numbers are generated.
        '!BamCQServiceJmsFileStore_auto_1':
        '!BamCQServiceJmsFileStore_auto_7':
        '!BamCQServiceJmsFileStore_auto_8':
        '!BamCQServiceJmsFileStore_auto_9':
        '!BamCQServiceJmsFileStore_auto_10':
        '!BamCQServiceJmsFileStore_auto_11':
        '!BamCQServiceJmsFileStore_auto_12':
        '!BamCQServiceJmsFileStore_auto_13':
        '!BamCQServiceJmsFileStore_auto_14':

Best regards,
TAM

@textanalyticsman Can you help me understand what you are doing with WDT and these BAM/BPM domains?  For example, are you just using createDomain from a model that you are maintaining?  Are you trying to use discoverDomain?
I am just trying to make sure I understand what you are trying to do so that I can scope the work needed.  For example, the createDomain case would suggest that you need the wildcard syntax in the model itself whereas the discoverDomain case would require putting the wildcard syntax in the system-elements portion of the typedef file.

@textanalyticsman As it turns out, the system-elements portion of the typedef file already supports that due to the elements being treated as regular expressions.  There are several tricky elements of implementing this wildcard syntax, such as how to detect when a name is a name versus a pattern since there is one (or more?) properties folder that supports names that end in an asterisk (see PR 563).

Could you provide an example about using regular expressions?

@textanalyticsman The system-elements section of the typedef files support the use of regular expressions for filtering out "system components," which is primarily used for excluding model definitions of these components during discovery since these components are created by the WLST domain extension templates applied to a JRF domain.  See https://github.com/oracle/weblogic-deploy-tooling/blob/main/core/src/main/typedefs/JRF.json for examples.


Mv 280 adding pinning

Draft for pinning the user

LGTM, but wait for approval from @kamil-stasiak & @alexmaar


Value of CPPWINRT_PROJECTION_ROOT_PATH is not exported from generate_winrt_projection

I think this line needs to be duplicated with PARENT_SCOPE so the caller of generate_winrt_projection can access the value of CPPWINRT_PROJECTION_ROOT_PATH. At least, when I tried this locally, it didn't work without that. (That said, the more holistic changes proposed by #27 sound like a better investment and may supersede this.)
https://github.com/MarkSchofield/Toolchain/blob/c789b4b58f995cb401f2f290447c3d0c4fd6c91b/Support/CppWinRT.cmake#L54

Ah, yes, you're right. Whenever I'm using generate_winrt_projection, I explicitly set CPPWINRT_PROJECTION_ROOT_PATH, so already know it outside of generate_winrt_projection. If you're relying on the default, then it would need to be set.

Yeah, that's probably more useful than relying on the default anyway; I was following along with the example project to get bootstrapped and tripped on this. I suspect the example project may inadvertently be compiling against SDK winrt headers rather than the ones generated by CMake... (haven't checked out this repo and tried it personally though).

The example code did a heavy-handed:
include_directories(BEFORE SYSTEM ${CPPWINRT_PROJECTION_ROOT_PATH})

To force the C++/WinRT projection into the start of the SYSTEM includes for everything. So I believe it was working, just heavy-handedly.
I've added add_cppwinrt_projection which creates a target to link to, and that gives better control and better performance.


filePath must be defined when using 'file' format and be a non-empty string

Current default is '' (empty string). Expecting for it to work with default value.

Do you get this error using type: base64 ?

@stoffern when using type: file

ah, well a default filename would be default.png 😉
Want to create a PR or should i?

@stoffern going for it 👍

👍 by the way. set it to default.jpg as jpg is default extension.

@stoffern if I just set it to default.jpg by default, it'll produce:

Could not write file: You don't have permission to save the file "default.jpg"

Should I go and add react-native-fs as a dependency to be able to use TemporaryDirectoryPath? Even this solution is iOS only. Would love your input on that 😉

Well then i think we should let it be like it is.
The error description is precise and enough to work with.


video features

sorry to bother you，could you tell me some details about negative samples for the video features？

I am not totally sure I understand your question here.
The video features are extracted for all frames using a sliding temporal window, i.e., a single feature vector is extracted from a local window using a consecutive chunk of frames, and features from all local windows (potentially overlapping) forms a sequence as the input to the model. The assignment of positive and negative samples will be based on the action annotations, as we described in our tech report.

I am not totally sure I understand your question here.
The video features are extracted for all frames using a sliding temporal window, i.e., a single feature vector is extracted from a local window using a consecutive chunk of frames, and features from all local windows (potentially overlapping) forms a sequence as the input to the model. The assignment of positive and negative samples will be based on the action annotations, as we described in our tech report.

i mean using different sliding temporal window when training？

For training, we use a fixed sliding window, e.g., 2304 for THUMOS14 and EPIC-Kitchens. During training, we will random select a fixed length of input features from each input feature sequence to form the batch. For ActivityNet, since all videos are resized into equal length, we directly feed the whole sequence into the model.

For training, we use a fixed sliding window, e.g., 2304 for THUMOS14 and EPIC-Kitchens. During training, we will random select a fixed length of input features from each input feature sequence to form the batch. For ActivityNet, since all videos are resized into equal length, we directly feed the whole sequence into the model.

thx,  vvvvvvvvvery much！


stdlib: avoid posix sched_yield in favour of C++11

What's in this pull request?

Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

Use the C++11 std::this_thread::yield to abstract away platform specific
differences for yield execution time.  This makes the code more portable to
non-POSIX environments (i.e. Windows).  NFC.

CC @gribozavr

@swift-ci please test

NFC on Darwin since std::this_thread::yield() calls sched_yield().


Investigate if author is referenced correctly when latest state of a Records is not authored by the initial record author

Investigate if author is referenced correctly when latest state of a Records is not authored by the initial record author.
Add tests and fix any issues.
A couple of scenario to kick start the investigation:

Alice is the author for initial write, then later Bob modified it. Alice should remain as the author for the record in reads and queries.
In protocols, if a 3rd party modifies a record, the record author should still be the initial record author. Make sure there is no escalation of privilege such as the 3rd party can act as the author once they modify the record.

Will need to coordinate with #800 because there is deviation in terminology/concept between the two.


Data.eachOrphanedPost is very inefficient

The forum I'm importing has 813833 posts and is currently 529194 / 813833 on fixPostsToPidsStart.

Here are the last two lines of mongo's log file:
2016-02-26T16:14:05.504+0000 I COMMAND  [conn14] command 0.objects command: find { find: "objects", filter: { _key: "_imported:_posts", value: "912903" }, projection: { _id: 0, value: 1 }, limit: 1, batchSize: 1, singleBatch: true } planSummary: IXSCAN { _key: 1, value: -1 } keysExamined:1 docsExamined:0 cursorExhausted:1 keyUpdates:0 writeConflicts:0 numYields:1 nreturned:1 reslen:141 locks:{ Global: { acquireCount: { r: 4 } }, MMAPV1Journal: { acquireCount: { r: 4 } }, Database: { acquireCount: { r: 2 } }, Collection: { acquireCount: { R: 2 }, acquireWaitCount: { R: 2 }, timeAcquiringMicros: { R: 16955 } } } protocol:op_query 112ms
2016-02-26T16:15:31.786+0000 I COMMAND  [conn20] command 0.objects command: find { find: "objects", filter: { _key: "posts:pid" }, sort: { score: 1 }, projection: { _id: 0, value: 1 }, skip: 601455, limit: 101 } planSummary: IXSCAN { _key: 1, score: -1 } keysExamined:601556 docsExamined:601556 cursorExhausted:1 keyUpdates:0 writeConflicts:0 numYields:6829 nreturned:101 reslen:2832 locks:{ Global: { acquireCount: { r: 13660 } }, MMAPV1Journal: { acquireCount: { r: 6832 }, acquireWaitCount: { r: 1 }, timeAcquiringMicros: { r: 61283 } }, Database: { acquireCount: { r: 6830 } }, Collection: { acquireCount: { R: 6830 }, acquireWaitCount: { R: 2 }, timeAcquiringMicros: { R: 164 } } } protocol:op_query 86148ms

I don't know much about MongoDB, but I do know that looking at hundreds of thousands of things spread across a file about an order of magnitude larger than the physical RAM of the machine is going to be very slow.
Is there anything that can be done in this situation?

true .. we could just iterate over them all and check each one of them.. then just use eachPost.
Instead of handing over the querying to mongo, which I hoped to be faster but it isn't.
the reason I have to fixPostToPids is because some posts have a _toPid value, but the posts are not guaranteed to imported in order unless you order by timestamp in your adapter.. if that's the case, then we can avoid this whole phase, fixPostsToPids and set the correct new toPid during each importPosts iteration.

im gonna assume that the posts ordered all the time by timestamp, and avoid this whole fixPostsToPids phase. I will change it this weekend

Is this something we need to order by in our importers now?
-Brian

On Feb 26, 2016, at 8:49 AM, Aziz Khoury<EMAIL_ADDRESS>wrote:
im gonna assume that the posts ordered all the time by timestamp, and avoid this whole fixPostsToPids phase. I will change it this weekend
—
Reply to this email directly or view it on GitHub.

@bdharrington7 well, you need to make sure your getPaginatedPosts() always orders by _timestamp or _pid - in most cases, that's the database's default behavior, but just in case.

closing, thanks @BenLubar


"Try It" does not load Az.ManagedServiceIdentity module

Running the code sample above for Get-AzUserAssignedIdentity throws an error:
Get-AzUserAssignedIdentity : The term 'Get-AzUserAssignedIdentity' is not recognized as the name of a cmdlet, function, script file, or operable program.
Get-Modules -ListAvailable yields a list that skips from Az.MachineLearning to Az.ManagedServices. Az.ManagedServiceIdentities is not listed.
Running 'Install-Module -Name Az.ManagedServiceIdentity -RequiredVersion 0.7.1' will install the module.
It appears that the ManagedServiceIdentity module is not imported by default with the rest of the Az module. I experience this issue both locally and in the Cloud Shell.

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

ID: 048f07a8-7858-86df-1095-318cc7910e40
Version Independent ID: 6ba8aaa8-53c9-3b4d-d158-3090416cdd90
Content: How to create, list and delete a user-assigned managed identity using Azure PowerShell
Content Source: articles/active-directory/managed-identities-azure-resources/how-to-manage-ua-identity-powershell.md
Service: active-directory
Sub-service: msi
GitHub Login: @MarkusVi
Microsoft Alias: markvi

@SaurabhSharma-MSFT did you receive any feedback?

Is there feedback available on this?

I have created a backlog item to track this.
#please-close


Need ability to set custom picture for node.

Ehlo!
We have nodes of several types in our tree. I'm interested in setting custom picture for every type of node. Is it possible?

if you are looking to do something like a file system tree, you should use the label templates to include those images.
The pictures for the tree nodes here are like the [+] and [-] buttons you have in a file system tree, and are not instead of the file icons.

http://wix.github.io/angular-tree-control/#file-style


Hero()s don't animate when routing using a GlobalKey<NavigatorState>()

Steps to Reproduce

Run flutter create bug.
Create a Material App and give it a GlobalKey<NavigatorState>()
Create a Widget that returns a Navigator() and give it a different Globalkey<NavigatorState>()
Create two pages with Hero() widgets and have a way to navigate between them.
Navigate with Navigator.of(context).pushReplacement()
Then Navigate with secondKeyName.currentState!.pushReplacement()

Expected results:
They both should look the same
Actual results:
The hero widgets don't animate when using the the key's current state.
https://user-images.githubusercontent.com/63734405/119223489-d1de3580-bb0a-11eb-9194-27478cb0ba32.mp4
https://user-images.githubusercontent.com/63734405/119223492-d4408f80-bb0a-11eb-9222-535afa3586ae.mp4

Logs

info • 'accentColor' is deprecated and shouldn't be used. Use
          colorScheme.secondary instead. This feature was deprecated after
          v2.3.0-0.1.pre. • lib/main.dart:31:15 • deprecated_member_use
   info • 'accentColor' is deprecated and shouldn't be used. Use
          colorScheme.secondary instead. This feature was deprecated after
          v2.3.0-0.1.pre. • lib/main.dart:154:15 • deprecated_member_use
   info • This class (or a class that this class inherits from) is marked as
          '@immutable', but one or more of its instance fields aren't final:
          ResetPassword.email • lib/screens/authentication/log_in.dart:464:7 •
          must_be_immutable
   info • The declaration '_getColor' isn't referenced •
          lib/screens/authentication/log_in.dart:489:9 • unused_element
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/authentication/log_in.dart:560:57 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/authentication/log_in.dart:567:55 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/authentication/log_in.dart:822:46 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/authentication/log_in.dart:830:44 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/authentication/log_in.dart:846:46 •
          deprecated_member_use
   info • Unused import: 'package:releaf/screens/authentication/veryify.dart'
          • lib/screens/authentication/register.dart:4:8 • unused_import
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/authentication/register.dart:151:46 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/authentication/register.dart:159:44 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/authentication/register.dart:175:46 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/authentication/veryify.dart:103:52 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/authentication/veryify.dart:112:54 •
          deprecated_member_use
   info • The value of the local variable '_ratio' isn't used •
          lib/screens/authentication/welcome.dart:18:11 •
          unused_local_variable
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/authentication/welcome.dart:81:58 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/authentication/welcome.dart:124:58 •
          deprecated_member_use
   info • Unused import: 'package:releaf/screens/home/settings.dart' •
          lib/screens/home/dashboard.dart:6:8 • unused_import
   info • Unused import: 'package:releaf/services/auth.dart' •
          lib/screens/home/dashboard.dart:8:8 • unused_import
   info • Unused import: 'package:releaf/services/database.dart' •
          lib/screens/home/dashboard.dart:9:8 • unused_import
   info • The value of the local variable 'theme' isn't used •
          lib/screens/home/dashboard.dart:41:11 • unused_local_variable
   info • 'accentColor' is deprecated and shouldn't be used. Use
          colorScheme.secondary instead. This feature was deprecated after
          v2.3.0-0.1.pre. • lib/screens/home/dashboard.dart:62:42 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/home/dashboard.dart:73:40 • deprecated_member_use
   info • Unused import: 'package:animations/animations.dart' •
          lib/screens/home/home.dart:1:8 • unused_import
   info • Unused import: 'package:firebase_storage/firebase_storage.dart' •
          lib/screens/home/home.dart:2:8 • unused_import
   info • Unused import: 'package:provider/provider.dart' •
          lib/screens/home/home.dart:4:8 • unused_import
   info • Unused import: 'package:releaf/services/auth.dart' •
          lib/screens/home/home.dart:6:8 • unused_import
   info • Unused import: 'package:releaf/services/database.dart' •
          lib/screens/home/home.dart:7:8 • unused_import
   info • Unused import: 'package:provider/provider.dart' •
          lib/screens/home/journal.dart:5:8 • unused_import
   info • Unused import: 'package:releaf/screens/home/dashboard.dart' •
          lib/screens/home/journal.dart:6:8 • unused_import
   info • Unused import: 'package:releaf/shared/const/app_theme.dart' •
          lib/screens/home/journal.dart:8:8 • unused_import
   info • Unused import: 'package:releaf/services/auth.dart' •
          lib/screens/home/journal.dart:9:8 • unused_import
   info • Unused import: 'package:releaf/services/database.dart' •
          lib/screens/home/journal.dart:10:8 • unused_import
   info • 'accentColor' is deprecated and shouldn't be used. Use
          colorScheme.secondary instead. This feature was deprecated after
          v2.3.0-0.1.pre. • lib/screens/home/journal.dart:60:38 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/home/journal.dart:143:52 • deprecated_member_use
   info • Unused import: 'package:releaf/screens/home/dashboard.dart' •
          lib/screens/home/meditation.dart:4:8 • unused_import
   info • Unused import: 'package:releaf/services/auth.dart' •
          lib/screens/home/meditation.dart:7:8 • unused_import
   info • Unused import: 'package:releaf/services/database.dart' •
          lib/screens/home/meditation.dart:8:8 • unused_import
   info • The value of the local variable 'theme' isn't used •
          lib/screens/home/meditation.dart:40:11 • unused_local_variable
   info • 'accentColor' is deprecated and shouldn't be used. Use
          colorScheme.secondary instead. This feature was deprecated after
          v2.3.0-0.1.pre. • lib/screens/home/meditation.dart:75:42 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/home/meditation.dart:86:40 • deprecated_member_use
   info • Unused import: 'dart:ui' • lib/screens/home/setting_popup.dart:1:8 •
          unused_import
   info • Unused import: 'package:email_validator/email_validator.dart' •
          lib/screens/home/setting_popup.dart:4:8 • unused_import
   info • Unused import: 'package:releaf/services/auth.dart' •
          lib/screens/home/setting_popup.dart:5:8 • unused_import
   info • Unused import: 'package:releaf/shared/assets/custom_form_field.dart'
          • lib/screens/home/setting_popup.dart:7:8 • unused_import
   info • The value of the local variable '_theme' isn't used •
          lib/screens/home/setting_popup.dart:16:11 • unused_local_variable
   info • Unused import: '../../wrapper.dart' •
          lib/screens/home/settings.dart:15:8 • unused_import
   info • 'accentColor' is deprecated and shouldn't be used. Use
          colorScheme.secondary instead. This feature was deprecated after
          v2.3.0-0.1.pre. • lib/screens/home/settings.dart:277:42 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/home/settings.dart:288:40 • deprecated_member_use
   info • The value of the local variable '_theme' isn't used •
          lib/screens/home/settings.dart:458:11 • unused_local_variable
   info • Unused import: 'package:releaf/screens/home/dashboard.dart' •
          lib/screens/home/tasks.dart:5:8 • unused_import
   info • Unused import: 'package:releaf/services/auth.dart' •
          lib/screens/home/tasks.dart:8:8 • unused_import
   info • Unused import: 'package:releaf/services/database.dart' •
          lib/screens/home/tasks.dart:9:8 • unused_import
   info • The value of the local variable 'theme' isn't used •
          lib/screens/home/tasks.dart:44:11 • unused_local_variable
   info • The value of the local variable 'width' isn't used •
          lib/screens/home/tasks.dart:45:11 • unused_local_variable
   info • 'accentColor' is deprecated and shouldn't be used. Use
          colorScheme.secondary instead. This feature was deprecated after
          v2.3.0-0.1.pre. • lib/screens/home/tasks.dart:75:40 •
          deprecated_member_use
   info • 'accentIconTheme' is deprecated and shouldn't be used. No longer
          used by the framework, please remove any reference to it. This
          feature was deprecated after v2.3.0-0.1.pre. •
          lib/screens/home/tasks.dart:86:38 • deprecated_member_use
   info • Unused import: 'package:releaf/services/database.dart' •
          lib/services/auth.dart:4:8 • unused_import
   info • 'accentColor' is deprecated and shouldn't be used. Use
          colorScheme.secondary instead. This feature was deprecated after
          v2.3.0-0.1.pre. • lib/shared/assets/themed_button.dart:94:55 •
          deprecated_member_use
   info • 'accentColor' is deprecated and shouldn't be used. Use
          colorScheme.secondary instead. This feature was deprecated after
          v2.3.0-0.1.pre. • lib/shared/assets/themed_button.dart:96:27 •
          deprecated_member_use
   info • 'accentColor' is deprecated and shouldn't be used. Use
          colorScheme.secondary instead. This feature was deprecated after
          v2.3.0-0.1.pre. • lib/shared/assets/themed_button.dart:240:55 •
          deprecated_member_use
   info • Unused import: 'package:releaf/screens/home/home.dart' •
          lib/wrapper.dart:7:8 • unused_import
   info • Unused import: 'package:releaf/screens/home/journal.dart' •
          lib/wrapper.dart:8:8 • unused_import
   info • Unused import: 'package:releaf/services/auth.dart' •
          lib/wrapper.dart:9:8 • unused_import

[✓] Flutter (Channel master, 2.3.0-13.0.pre.59, on macOS 11.3.1 20E241
    darwin-x64, locale en-AU)
    • Flutter version 2.3.0-13.0.pre.59 at
      /Users/hadymashhour/Developer/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 1e35f4c0c6 (2 days ago), 2021-05-20 03:14:01 -0400
    • Engine revision 67c537d41b
    • Dart version 2.14.0 (build 2.14.0-129.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version
    30.0.2)
    • Android SDK at /Users/hadymashhour/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.56.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.22.0

[✓] Connected device (2 available)
    • iPhone 12 Pro Max (mobile) • BD55A391-EEF1-45D8-831D-3FC41EF96918 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • Chrome (web)               • chrome                               •
      web-javascript • Google Chrome 90.0.4430.212

• No issues found!

This solved it https://stackoverflow.com/questions/51760916/hero-animation-not-working-inside-nested-navigator
Just add a HeroController to the Navigator()'s observers and that should fix the problem.


mongodb-backup/restore: updates

<EMAIL_ADDRESS>introduce stream cursor
https://github.com/hex7c0/mongodb-backup/issues/11

meteor does not support tilde operator.
this PR is just an update of deps

Thank you so much :)

wait :)
I'm writing a new commit for fix must declare exact version of dependency

Could you please make a new PR thanks :) I reverted other back


Log window is to wide

I am not sure in which version this started, but when viewing a log screen in virtualmin, like the Apache Error Log, or Acces it does not fit on the screen.
In previous version you could scroll to the right, but now that is impossible.

It's done on purpose not to wrap long lines.
Do you have Side Slider fixed or you have a button with bell to open it?

I see what the source of the issue. It's quick and easy to fix. I'll do it tomorrow.
Thanks for reporting.

thanks

I can confirm it is fixed in 19.04


Fix self intersecting polygon warning for PianoModel.fbx

Overview
Address warnings related to a FBX when importing MRTK into project

Changes

Fixes: # .

Verification
Opened HandInteractionExamples scene and was able to play the piano.
You can also get this commit, right click on the PianoModel.fbx and re-import it from the menu. The console should not print any warnings.

/azp run


SUBMARINE-315. Fix run_submarine_mnist_tony_rpc.sh run error

What is this PR for?
Change classpath in run_submarine_mnist_tony_rpc.sh
What type of PR is it?
[Bug Fix]
Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/SUBMARINE-315
How should this be tested?
https://travis-ci.org/pingsutw/hadoop-submarine/builds/622624260
Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

Looks good to me. +1

Thanks @tangzhankun and @liuxunorg for the review and commit


React Functionality

Description
( this can be split so people are working on different components, i.e. signup/login forms, navigation, reactionList, etc )

Add functionality to components/pages
Update JSX in all components

What is the issue to be addressed

Hi Pavan.  I'm sorry but who are you exactly?  This is a final group project for a full stack developer boot camp, we wrote these issues before anything had been coded as a way to track our progress.  We didn't really expect for non-contributors to participate in these issues.  The four contributors to this project will handle any actual issues that arise and we respectfully reserve the right to delete any comments from users outside of our team.  This will be part of our upcoming job application efforts and we don't need to have immaterial comments in the issues section of our repository.

Sorry about that, I was just surfing GitHub and stumbled across this. Could you perhaps make the issue private so that only the participants of the final group project can access it? Thank you very much.


Update Navbar.tsx

Pull Request
Description
Added link to the github svg in the navbar
Related Issues
No link for the github svg in the navbar
Checklist

[x] I have tested these changes locally.
[x] My code follows the project's coding guidelines.
[x] I have updated the documentation, if necessary.

@Rakshit-gen make sure the checks pass

Closing the PR since made against main make a fresh PR to develop branch.


Improved delete ACLs logic and execution

This PR add an improved logic for deleting acls, a more clean one where only the desired ACLs are deleted. As well it include a better execution and plan print when executed as dry run.
fixes #76

Nice @purbon :-) I can try to give you some feedback on this. But today/tomorrow I´m rather busy with other stuff and since I´m still rather new to the code base it will take a little bit of time to do a proper review. :-)
One question. This PR fixes the delete part of it but wouldn´t it be best to have the same logic for creation as well; only try to create new acls? At least for having a proper log of what is actually changed.

Looking forward to your help and contribution @akselh , in regards to your questions I just pushed a commit doing that, few unnecessary rounds trips are always welcome for performance ;-)

Hi @purbon , due to circumstances I have unfortunately not been able to follow up on this or other contributions to the project. However, I should have some time from now.

@akselh I know, life is hard! all contributions are highly welcome! Looking forward to get your thoughts here!


Throw error when component is undefined or null (#2038)

Add assertion when a route component is null or undefined, a fix for issue #2038.
Really appreciate for a review.

Oh sorry I missed that. Will try to work on that.

Keep in mind you can run the tests locally too, it will save you some time 😄

Just now, I've tried to trace through what might be throwing error when it shouldn't. Then I found in Route Matching that the routes does not have any components nor any other options like redirections, aliases. So I think it is probably better to just warn instead of asserting error.
You might want to review it again. Anyway thanks for the guide.


Use goLineStartSmart for cmd-left in Sublime keybindings

OSX users of Sublime will be used to this as the default behavior. Right now CodeMirror with Sublime bindings jumps all the way to pos 0 on cmd-left. In actual Sublime, it jumps to start of text before going to pos 0. Luckily there's already a handy command for that!

Is there an equivalent on non-Mac platforms? If so, could you add it, and if not, could you make this line conditional with if (mac) ...?

There is not. I updated the PR.

Thanks. Merged as 90ef4bd


Large drift and rotation upon initializing the mapping processs in ROS2

Hello,
Thank you for creating this repo. I am working on mapping KITTI360 with LIO-SAM in ROS2 humble.
For the extrinsic, I am using the KITTI extrinsic posted on readme.

The green trajectory is the GPS trajectory(gt). The pointcloud reconstruction failed at the beginning when pc are projected then it started to spin and fly around, similar to #433 . I also noticed that the pc loading speed is much slower than GPS. I have already set the loading speed to -r 0.1.
I get error messages like "Lookup would require extrapolation into the past." "Large velocity (38.120705), reset IMU-preintegration!"

Interestingly, same thing also happened with another dataset I have, which was tested on another PC with same ROS2 yaml setup and produced perfect mapping results.

So I wonder if my problem is related to #437 . Help will be much appreciated.

Hello,
I am having similar issue applying the code for a dataset that I recorded on a ROS2 instrumented car having VLP16 and hesai pandar, and used the Hexagon Novatel pwkpak7 for the imu, odom and gps topics.
The params.yaml is the following :
/**:
  ros__parameters:

    # Topics
    pointCloudTopic: "/velodyne/velodyne_points"                   # Point cloud data
    imuTopic: "/novatel/oem7/imu/data"                        # IMU data
    odomTopic: "/novatel/oem7/odom"                    # IMU pre-preintegration odometry, same frequency as IMU
    gpsTopic: "/novatel/oem7/fix"                    # GPS odometry topic from navsat, see module_navsat.launch file

    # Frames
    lidarFrame: "velodyne_frame"
    baselinkFrame: "base_link"
    odometryFrame: "novatel_frame"
    mapFrame: "map"

    # GPS Settings
    useImuHeadingInitialization: false           # if using GPS data, set to "true"
    useGpsElevation: false                       # if GPS elevation is bad, set to "false"
    gpsCovThreshold: 2.0                         # m^2, threshold for using GPS data
    poseCovThreshold: 25.0                       # m^2, threshold for using GPS data

    # Export settings
    savePCD: false                               # https://github.com/TixiaoShan/LIO-SAM/issues/3
    savePCDDirectory: "/Downloads/LOAM/"         # in your home folder, starts and ends with "/". Warning: the code deletes "LOAM" folder then recreates it. See "mapOptimization" for implementation

    # Sensor Settings
    sensor: velodyne                               # lidar sensor type, either 'velodyne', 'ouster' or 'livox'
    N_SCAN: 16 #64                                   # number of lidar channels (i.e., Velodyne/Ouster: 16, 32, 64, 128, Livox Horizon: 6)
    Horizon_SCAN: 1800                            # lidar horizontal resolution (Velodyne:1800, Ouster:512,1024,2048, Livox Horizon: 4000)
    downsampleRate: 1                            # default: 1. Downsample your data if too many
    # points. i.e., 16 = 64 / 4, 16 = 16 / 1
    lidarMinRange: 1.0                           # default: 1.0, minimum lidar range to be used
    lidarMaxRange: 1000.0                        # default: 1000.0, maximum lidar range to be used
...

But I am getting a very weird spiral accumulation of the data more than the creation of the map.

@wxyLuna2024 were you able to resolve the issue?
@TixiaoShan can this be related to a calibration issue, or is it another problem?
Thank you in advance for your input.

Hello,
I am having similar issue applying the code for a dataset that I recorded on a ROS2 instrumented car having VLP16 and hesai pandar, and used the Hexagon Novatel pwkpak7 for the imu, odom and gps topics.
The params.yaml is the following :
/**:
  ros__parameters:

    # Topics
    pointCloudTopic: "/velodyne/velodyne_points"                   # Point cloud data
    imuTopic: "/novatel/oem7/imu/data"                        # IMU data
    odomTopic: "/novatel/oem7/odom"                    # IMU pre-preintegration odometry, same frequency as IMU
    gpsTopic: "/novatel/oem7/fix"                    # GPS odometry topic from navsat, see module_navsat.launch file

    # Frames
    lidarFrame: "velodyne_frame"
    baselinkFrame: "base_link"
    odometryFrame: "novatel_frame"
    mapFrame: "map"

    # GPS Settings
    useImuHeadingInitialization: false           # if using GPS data, set to "true"
    useGpsElevation: false                       # if GPS elevation is bad, set to "false"
    gpsCovThreshold: 2.0                         # m^2, threshold for using GPS data
    poseCovThreshold: 25.0                       # m^2, threshold for using GPS data

    # Export settings
    savePCD: false                               # https://github.com/TixiaoShan/LIO-SAM/issues/3
    savePCDDirectory: "/Downloads/LOAM/"         # in your home folder, starts and ends with "/". Warning: the code deletes "LOAM" folder then recreates it. See "mapOptimization" for implementation

    # Sensor Settings
    sensor: velodyne                               # lidar sensor type, either 'velodyne', 'ouster' or 'livox'
    N_SCAN: 16 #64                                   # number of lidar channels (i.e., Velodyne/Ouster: 16, 32, 64, 128, Livox Horizon: 6)
    Horizon_SCAN: 1800                            # lidar horizontal resolution (Velodyne:1800, Ouster:512,1024,2048, Livox Horizon: 4000)
    downsampleRate: 1                            # default: 1. Downsample your data if too many
    # points. i.e., 16 = 64 / 4, 16 = 16 / 1
    lidarMinRange: 1.0                           # default: 1.0, minimum lidar range to be used
    lidarMaxRange: 1000.0                        # default: 1000.0, maximum lidar range to be used
...

But I am getting a very weird spiral accumulation of the data more than the creation of the map.  @wxyLuna2024 were you able to resolve the issue? @TixiaoShan can this be related to a calibration issue, or is it another problem?
Thank you in advance for your input.

Hi, I did what was mentioned in #437 , didn't really work out. I would say try to debug your extrinsic matrix (ROT and RPY specifically) first, and then check your IMU publishing rate to see if it is matching the pc. However me personally is still struggling in getting the first few frames of the mapping to work correctly. If you found the solution towards this please don't hesitate to post here

Hi, I have encountered this issue as well.  I use lio sam in ros2-humble, and there is persistent positional deviation during the localization process, but this error does not occur every time. Are there any suitable solutions available now?
Thank you for any response.

@wxyLuna2024  I have found that there might be instances where pointcloud messages cannot be successfully received during the operation, leading to trajectory drift. By modifying the QoS settings, I was able to resolve this issue, and you might want to give it a try.
rmw_qos_profile_t qos_profile_lidar{     RMW_QOS_POLICY_HISTORY_KEEP_LAST,     5,     RMW_QOS_POLICY_RELIABILITY_RELIABLE, # pre is BEST_EFFORD     RMW_QOS_POLICY_DURABILITY_VOLATILE,     RMW_QOS_DEADLINE_DEFAULT,     RMW_QOS_LIFESPAN_DEFAULT,     RMW_QOS_POLICY_LIVELINESS_SYSTEM_DEFAULT,     RMW_QOS_LIVELINESS_LEASE_DURATION_DEFAULT,     false };


GDPR: Marketing emails

Next to the GDPR related email that we'll send in the beginning of May, marketing emails will be sent to inform users of our new features: Regional news, Joyn, Payconiq, ...
In the emails, we'll ask the user's permission to keep him up to date with emails with news. There will be a button that should link to an unauthenticated request handler in OCA. OCA should save this setting somewhere.
The customer export should include the user's permission to send emails (such that we can hide the GDPR part if the user already granted us permission)

I suppose we need to ask permission before sending an email instead of sending emails to ask permission...

I suppose we need to ask permission before sending an email instead of sending emails to ask permission...

Right now, we are still allowed to do this. After May 25th we aren't

Yes i know but like its described right now we are violating GDPR.
We should add a checkbox in the customer signup.

https://ec.europa.eu/info/law/law-topic/data-protection/reform/rules-business-and-organisations/legal-grounds-processing-data/grounds-processing/does-consent-given-25-may-2018-continue-be-valid-once-gdpr-starts-apply-25-may-2018_en

just had a look at our marketing tool and suggest to do the following.
When customers register we add a checkbox if they want to receive marketing updates otherwise we won't send a smart email.
Create and use 1 list (marketing newsletter) for all our customers, when sending a smart email make sure to add that recipient to the list as well.
Then for that 1 list we create a webhook to add/remove consent to that list
Still need to validate with legal if we need to add a way to unsubscribe/subscribe in the dashboard as well
https://www.campaignmonitor.com/api/transactional/#smart-email-listing
https://www.campaignmonitor.com/api/lists/#list-webhooks
https://www.campaignmonitor.com/api/webhooks/#whats-a-webhook


cda.pl

Match found in list.34.hosts.ubuntu101.co.za.domains:
cda.pl
same as #299 and #102

Duplicate of #299 which is on hold.


ic won't work in jupyter

p9 = use("plotnine", version="0.7.1")
ic| Error: Failed to access the underlying source code for analysis. Was ic() invoked in an interpreter (e.g. python -i), a frozen application (e.g. packaged with PyInstaller), or did the underlying source code change during execution?

Not really a justuse issue, is it?


Basic freebie Feats: Point-Blank Shot

Hello,
For Basic freebie feats. It looks like Point-Blank Shot is gained automatically. I understand if this is because of coding as it's a prerequisite for a lot of feats. In the Elephant Handbook, Precise Shot replaced Point-Blank Shot for all prerequisites and Point-Blank Shot is now a stand-alone feat.
Thanks for the mod!
Cheers,

Yes. I didn't want to change all the prerequisites. Would also create issues with other people's mods.


Pop-ups sometimes hide behind their parent

Follow-up to https://github.com/manjaro-sway/manjaro-sway/issues/370
For example in Firefox, the dict.cc and MasterPassword extensions are unusable as their pop-ups move behind the browser window right after opening.

I think this can only be solved on an application level. Searching through the mozilla issues, there are several ones related to this.

For me this just works :tm::
recording_20230102-204934.webm
Disclaimer: Running sway 1.8 rn.

It always works for me on non-sway setups. dict.cc extension works again, mpw never worked

Can you give me a link to the masterpassword extension you have been using? for this one nothing hides. ffel free to reopen!

yep, I am using that one, it keeps positioning behind firefox. Maybe related to my sway config?
assign [app_id="firefox"] workspace 2

No permission to reopen ;)

No permission to reopen ;)

sorry, overread that one.

by the by, maybe you can give me triage permission on the repo so I can support actively :)

don't see how this fails. for me this just works:

works fine now, but with above line disabled

Nice!

I could imagine that assign [app_id="firefox"] workspace 2 somehow "moves" the window to the workspace and thereby positions the additional window moves behind it. Perhaps also focus the window? does something like assign [app_id="firefox"] workspace 2, focus


feat(cb2-9288):  repair dynamo call and remove marshaller

Ticket title
Remove dynamo marshaller call
Checklist

[ ] Code has been tested manually
[ ] PR title includes the JIRA ticket number
[ ] Branch is rebased against the latest develop
[ ] Squashed commit contains the JIRA ticket number

Placing on hold as found a different issue

Will raise this again when the time is right, a ticket is being generated to capture the work


Support a multi-writer option for volume create

This is perhaps long pending and easily implemented as a volume create option to set the "multi-writer" flag on a VMDK in the VMX config scsix:y.sharing = "multi-writer". The only requirement is that the vmdk must also be created as a thick disk as dynamic growing is disallowed for a multi-writer disk. Otherwise this VMDK can be shared across VMs.
Also, state for a VMDK with this option set will be ignored and the attached-to field in the KV will be a list of VMs vs. a single VM. VMDK can be removed only if this list is empty.

We already discussed in in #193 . The customer issue there was a regression  which  we fixed in 0.11.
but the issue has been raised there, and the explanation about why we should not do that was also there. What has changed ? Why are bringing it up again ?
To save the click, here is the point from  #193:

multi-write VMDKs are used for cluster file systems only and require in-VM cluster software that synchronize access to essential shared block device. There is no use case for enabling this until somebody may decide to put Oracle RAC or CFS into a container without admin access to do it statically.
It is super dangerous and pretty much guarantees data corruption if multiwrite is allowed and disk attached / regular filesystem is mounted on 2 VMs.

And yes.  you refer to some "clustering"  app which could use it.  So,    if we enable it, which speciific "clustering" app would use it without data corruption ?
Unless there is a clear answer we should close it to minimize distractions

What is the suggested option to share a volume across multiple containers hosted on different vm sharing the same datastore ?
Currently we can attach the volume in multiple containers only if those containers are started on the same vm (same swarm node).

From my discussions with the team that supports multi-writer disks, these VMDKs can be used simultaneously from multiple VMs (for both read and write IOs). Any collaboration on how the data is managed is up to the applications thats using these disks.

Any collaboration on how the data is managed is up to the applications thats using these disks.

to clarify - only clustered apps doing their own distributed locks will be able to use it (e.g. .Oracle RAC). Any attempt to access these disks as a mounted FS will either fail on mount (if a FS marks superblock as mounted) or will corrupt the FS

Native multi-writer disks are exactly to be used with apps that manage data level locking, no doubt about that. Can surely be used by apps that do support such capability.

I understand, as the vmdk volume created by this plugin is formatted using a traditional file system, it's not meant to be shared across / mounted from multiple machine.
I'm looking for the most simple way to store binary object in the already redundant and distributed vmware vsan that I have. I don't need a filesystem actually. The goal is to be able to scale our application inside a docker swarm and access those binary object from every container.
It's a shame vmware doesn't offer any API to store binary object (S3 like) directly.
I could create a vmdk for each object (millions) but I'm not sure it's that scalable.
@msterin Any suggestion on this ?

yeah, S3 API in VSAN directly would be the way to go. Unfortunately I don't know if it's planned.
For now, this may be a good  workaround, depending on your IO patterns.

Yup. I'm evaluating minio too ;-)
I will either use this, or simply setup one VM with a NFS Server (because it seems not possible to run it inside a swarm for now, missing privileged mode).

@blop - yes, server file systems (NFS, CIFS) would still do wonders :-).
Samba does not require privileged mode and fails over fine in swarm, you can feed it a docker-volume-plugin volume and this might cover your use case too. We are currently working on"multiple-containers shared access to vSphere storage" using this approach.