Login - Unable to log in manually with locked gmail but able to log in with Google

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

To create a locked Gmail:

Go to Settings > Profile > Contact method > New contact method.
Add an invalid email address (eg<EMAIL_ADDRESS>Save and log out.
Log in with primary contact method.
Steps:
Go to staging.new.expensify.com.
Log in with Google witth the locked accout.

Expected Result:
Since user is blocked from manually logging in with the locked Gmail, login with Google should be blocked as well
Actual Result:
User is able to login with Google when manually logging in with the locked Gmail fails
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome
[x] Windows / Chrome

Version Number: 1.3.56-7
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/d6f61016-8314-4432-bb0e-4612464d3bdf
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

@marcochavezf, this looks like maybe a server-side issue with Google Sign-In?

Yeah, this is a backend issue from our side. I don't know exactly how we're triggering the suspended email message, but I think we'd need to hook up that logic to the SignInWithGoogle API command. So this can only be fixed internally.
Since this is introduced with the new Google sign-in functionality, I don't think this should be a blocker.

@marcochavezf could you take over this issue in that case?

Since this is a back-end issue, we're not going to block NewDot deploy on this. But we'll keep it as a daily

No update

No update, prioritizing wave7 atm

No update, moving it to weekly until we get collect approvers done at the end of the offshore

No update, focused on wave7 and traveling back from Offshore

No update, wrapping other priorities for wave7

No update

No update

No update

No update

No update

No update, focused on wave bugs

No update

Focused on wave issues, no update

No update

No update, focused on wave work

No update, posted in wave9 to ask if it should part of it https://expensify.slack.com/archives/C05NJ4SLBMF/p1707159629690479

No update, focused on critical wave issues and older tasks

No update, focused on critical and high wave tasks

After some investigation, I found out this error is only shown when our email providers can't send the validation code for some reason (has the nvp expensify_emailDeliveryFailure). Since the user is trying to sign in via Google, we don't send a validation code. Also, we don't show this error in OldDot when the user has the nvp expensify_emailDeliveryFailure and sign in via SSO, so I'm going to close it out.


libstreaming on Android Lollipop encoding issue

The encoding with MediaRecorder, on Nexus 4 with Android Lollipop, gives:
11-18 09:50:21.028: W/System.err(15783): net.majorkernelpanic.streaming.exceptions.ConfNotSupportedException
11-18 09:50:21.028: W/System.err(15783): 	at net.majorkernelpanic.streaming.video.VideoStream.encodeWithMediaRecorder

Did anyone solved this ConfNotSupportedException issue?


GET_MANY does not return with id in mapResponse

Recording to aor docs
A {Record} is an object literal with at least an id property, e.g. { id: 123, title: "hello, world" }.

Currently there is no case to handle GET_MANY in mapResponse, which seems to break <ReferenceField> for my project.
A simple fix would be to add case GET_MANY:  to get same treatment as GET_LIST?
// src/restClient.js
const mapResponse = (response, type, resource, params) => {
    const idKey = getIdKey({resource, options})
    switch (type) {
      case GET_ONE:
      case UPDATE:
      case DELETE:
        return {data: {...response, id: response[idKey]}}
      case CREATE:
        return {data: {...params.data, id: response[idKey]}}
      case GET_MANY:  // added GET_MANY
      case GET_LIST:
        response.data = response.data.map(item => {
          if (idKey !== 'id') {
            item.id = item[idKey]
          }
          return item
        })
        return response
      default:
        return response
    }
}

I think you are right on this, we forgot to include it.
Can you create a PR on this? I will be glad on merging it ;)

@josx I have spotted the same issue with GET_REFERENCE, I will submit PR with fix for both case.
Is that ok for you?

ok! great merged ;)

Cheers!


📘 EPIC: New DPG Registry page design

Theme
Design
Implement

I delete as https://github.com/DPGAlliance/digitalpublicgoods-nextjs/issues/9 duplicates it.


(raw_vehicle_cmd_converter): want unit test for pid controller

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I've agreed with the maintainers that I can plan this task.

Description
currently raw vehicle cmd converter has unite test for map conversion but there is no unit test for pid feed-back control.
Purpose
to test without real vehicle
Possible approaches
make unit test for pid controller class
https://github.com/autowarefoundation/autoware.universe/blob/main/vehicle/raw_vehicle_cmd_converter/src/pid.cpp
Definition of done
added unit test

resolved by https://github.com/autowarefoundation/autoware.universe/pull/2212


cargo tree -i --workspace doesn't work even if you specify a package

Problem
cargo tree -i can't be used in a workspace.
Steps
$ cargo tree --workspace -p winapi -i
error: The `-i` flag requires a package name.

The `-i` flag is used to inspect the reverse dependencies of a specific
package. It will invert the tree and display the packages that depend on the
given package.

Note that in a workspace, by default it will only display the package's
reverse dependencies inside the tree of the workspace member in the current
directory. The --workspace flag can be used to extend it so that it will show
the package's reverse dependencies across the entire workspace. The -p flag
can be used to display the package's reverse dependencies only with the
subtree of the package given to -p.

Note that both cargo tree -i -p winapi and cargo tree --workspace work fine individually.
Notes
Output of cargo version: cargo 1.49.0-nightly (d5556aeb8 2020-11-04)

I'm not sure what your trying to view specifically, but I think the command you are looking for is cargo tree --workspace -i winapi.  The first line lets you know that the -i flag requires a package name.  Perhaps the error message could be clarified to say something like error: The argument `-i <SPEC>` requires a package name, but none was supplied. to match the clap style?

@ehuss I'm confused - if -i requires a package, then why does it work without --workspace?
(-bash@build-server) ~/.../src/librustdoc ▶ cargo tree -i -p serde_json
serde_json v1.0.59
└── rustdoc v0.0.0 (/home/joshua/rustc/src/librustdoc)
(-bash@build-server) ~/.../src/librustdoc ▶ cargo tree -i -p serde_json --workspace
error: The `-i` flag requires a package name.

Backwards compatibility.  That was the old way of specifying a package with -i.  There is an ambiguity of which packages are the roots (those passed to -i) and what the "context" is (those passed to -p), so it was changed so that -i takes a package name.  If no package name is given to -i, it treats the -p flags as-if they were passed to -i to retain backwards compatibility.

Would it be possible to extend that compatibility to --workspace as well? Or maybe add a warning to -i -p without --workspace that says the old behavior is deprecate?

I'm a little concerned that a warning would be a little noisy, since in most cases it just works, but it would probably be ok.
The error message definitely could stand some clarification, though.


Let users join by servername instead of IP or FQDN

https://github.com/overload-development-community/olmod/blob/6750f788cd1c96b24080855f2c19ee1128e9d611/GameMod/MPInternet.cs#L26
We had this issue on our latest LAN at the weekend. The referenced check prevents to join via "servername" only, as a dot has to be given, like "servername.localnet".
Also, the screen to join a game only mentions to enter an IP address, but a host name would be valid, too.

Is there actually a password? How is this set up?

Originally, there was only a password. Olproxy/olmod internet matches worked by stuffing the IP (or FQDN) into the password field. Yuo can always add and underscore after the IP to get a passowrd-protected game. As a side effect, the IP / hostname you chose to connect to the server was still part of the password.
However, I just changed that behavior. PR #225 will ignore the IP/hostname part from the password field and only treat the actual password part as a password now. Furthermore, this patch also allows a hostname without dots, so it should resolve this issue.
Note that your "LAN" server hast to be started with the -internet parameter for this to work. I was able to join my local server via 3 clients using 3 different addresses: <IP_ADDRESS>, <IP_ADDRESS> and the local hostname ixtlilton. However, it blew up when I tried using localhost, but that was because localhost is resolved to an IPv6 here and the game code can't deal with IPv6 unfortunately. But that is an entirely different issue.

So olmod took lan games logic as a base and widened it, to make internet games useable?
Wow, what a mess for a newly developed game.
In the PR you separate ip and pwd and inject the pure pwd to the right location? Sounds good.  ;)
In the GameMod folder: Is there a way to identify what classes are intended for client or server?
As far as I can tell, we only used Internet Match on the LAN-Party.

So olmod took lan games logic as a base and widened it, to make internet games useable? Wow, what a mess for a newly developed game.

The development focus clearly was single-player, with multiplayer being an afterthought. The internet multiplayer was based on automatic, skill-based matchmaking on the Revival servers (which are shut down by now). The LAN match they implemented is basically a huge hack around that matchmaking. And olmod is a hack around that to allow it for internet by replacing the LAN broadcasts with directed matches.

In the PR you separate ip and pwd and inject the pure pwd to the right location? Sounds good. ;)

Yeah. There is only one place in the code where the passwords are matched (somewhere in a delegate in TryLocalMatchmaking, where the password is used as a key for a dictionary, so that the same password would be matched to the same dictionary entry. I added a transpiler to modify the key that is used for that dictionary. If an underscore is present, I just use the rest of the string (including the underscore) as the password, otherwise I just use EMPTY without an underscore as the password (without an underscore, so a game without password can only entered without password). This is all server-side, the password  field in the jason-based match presets is not modified at all.

In the GameMod folder: Is there a way to identify what classes are intended for client or server?

No, not really. We group by feature/functionality. The multiplayer-related stuff is usually in files beginning with MP. However, the original overload multiplayer code is a huge mess. There are some Server and Client classes, but the separation isn't really there. Often you see code which branches depending on whether it thinks it is a server or a client. And olmod has to deal with all of that, so we also have no good separation between client and server.

However, it blew up when I tried using localhost, but that was because localhost is resolved to an IPv6 here and the game code can't deal with IPv6 unfortunately. But that is an entirely different issue.

OK, I made a small workaround for that Issue in the PR: if a host has both an IPv4 and IPv4 address, the V4 one is preferred. That also makes my localhost work.

@derhass looks resolved by #225, please let me know if it's not.  v0.5.5 should be out tomorrow and that patch is included.

@derhass looks resolved by #225, please let me know if it's not. v0.5.5 should be out tomorrow and that patch is included.

Yeah, this issue can be closed now.


Please expose internal variants that are not safety opinionated

https://github.com/Plutonomicon/plutarch-plutus/blob/93833546d61d7a463d3ec25e8fbd16ba1abd0ec1/Plutarch/ByteString.hs#L240C1-L291C43
The unique value proposition of Plutarch over other languages targeting PlutusCore is that it is not opinionated. It gives the developer full low-level control over everything from strictness to inlining vs hoisting to binding and so forth. This is important because if you are very knowledgeable in the domain of Cardano smart contracts, you can write extremely efficient code that you cannot write in the other languages where the compilers are more opinionated (ie the compiler makes choices on your behalf regarding strictness or whether something is inlined or not, etc).
Please don't include any opinioned logic in builtin internal functions, if you truly want such logic please include it a builtin external module so that developers who don't want that logic can still use the internal module (which I expect is at-least 95% of the developers using Plutarch since it is a hyper-efficiency focused DSL).
For example,
preplicateBS :: forall (s :: S). Term s (PInteger :--> PByte :--> PByteString)
preplicateBS = phoistAcyclic $ plam go
  where
    go :: forall (s' :: S). Term s' PInteger -> Term s' PByte -> Term s' PByteString
    go len w8 =
      punsafeBuiltin PLC.ReplicateByte
        # pif (len #< 0) 0 len
        # punsafeCoerce @_ @_ @PInteger w8

pzeroesBS :: forall (s :: S). Term s (PInteger :--> PByteString)
pzeroesBS = ...

ponesBS = ...

All of these functions have this pif (len #< 0) 0 len to check that the value is non-negative. There is no reason for this check. Optional types and so forth for delayed failure are an anti-pattern in onchain code, why would you ever want to pass this a negative integer and not have it error? I'd imagine this might even cause some vulnerabilities if the caller is not expecting negative integers to be passed. Even if you do want this check in some weird edge-cases, why should you make that the default behavior?
Please just expose all PLC.ANYTHINGHERE directly as a Plutarch binding with no additional opinionated logic.

@colll78 you can still use punsafeBuiltin directly I think, but obviously this code is wrong. Missed in review I suppose.

You could always make a PR.

You could always make a PR.

Lol I did make a PR, I was the one who introduced these functions to the Plutarch codebase in the first place. Then my PR was reworked to add type safety and suddenly when I upgrade to the newer version my ex-unit benchmarks went up a ton.

...lol. I really think the core Plutarch implementation should be entirely separate from it's pseudo-standard-library anyway. It should only provide the most core things and everything else should be in plutarch-extra or similar.

https://github.com/Plutonomicon/plutarch-plutus/pull/767 this exposes raw builtin functions.
It is restructures such that Plutarch.Builtin.Xyz holds interfaces to raw builtins and builtin types


Gantt

Hi everyone,
First of all, thank you very much for these resources. It's very appreciated.
I just wonder if it could be possible to improve 2 things in the gantt chart JSON:

Include subtask. i.e I could add a column type with the task type [task, subtask]. Then we could sort by task. This would require editing the JSON to manage groupview
I don't know if this is possible to find a way to avoid having the field projectStart/projectEnd in every row…

Best,
Bastien

Hi Bastien,
Thank you for your suggestions.
"to avoid having the field projectStart/projectEnd"
This would be very interesting indeed, and would have been preferable in the first place. Unfortunately the format is always row based, you can not refer to values stored in another row. So I personally don't see a solution there.
The projectStart and projectEnd dates are used in every line to calculate the position of bar, so those values really need to be available in every row.
"Include subtask. i.e I could add a column type with the task type [task, subtask]. Then we could sort by task. This would require editing the JSON to manage groupview"
I do see what you are suggesting. But the way grouping is working, what you really need to do is add a column called "projectPhase" (or something) and group by these values. Unfortunately we will loose the display of the projectStart and projectEnd for every group because those are only displayed in the first row of the table.
Second, the group view itself doesn't really have access to the information in the rows below accept for aggregations like count, average and probably maximum... bur there could be a way of displayin the start +enddate of that phase....
I need some time to get this in a new version... but surely worth the try
Grt Geert

Hello,
I have just test project-gantt-chart in my Sharepoint list but unfortunately I realized it doesn't work if dates are in dd/MM/yyyy format.
I also found that I had to use  toString insted toLocaleDateString in order to read data from variables. ( e.g. "txtContent": "=toString([$ProjectStart])"
Finally, if wrong dates are converted to Numbers, formulas crash all the time.

I am trying to adapt this (very, very nice) list format script to column names that are standard for "Activity" cloumn in the content types in my sites. I have been trying changeing all the occurances of variables in the script, but as soon as I change the first one, the whole page turn to "blank"; what I don't understand is that if I misspell the variable name, only that variable disapperars from the page. I solved with the workaround of creating calculated columns with the original names, made equal to my names, but I'd like to solve the root cause of the problem....
thank you for your help
paolo

HI @paobrunelli ,
The reason why it most of the time breaks is because the script (like all List-formatting) refers to the internal name of the column and not the display columnname. You can find tha name in de addressbar when you try (or pretend) to change the (display)name of the column. See screenshots, where it mentions Field=Title

Because of the calculations it indeed can happen that everything turns weird at the moment that column names don't match.
If you tell me what your internal names are for the columns I can make it work on my end.
geert

thank-you geert! with your suggestion I solved the issue (it derived from language setting in the site column definition).
I am now planning to insert this view in our project management flow, using well structured sharepoint list made available to the team members, that do not have MS Project. What should be great, would be to have the ability to define, if not task-subtask hierarchy, at least the ability to define a sort of "main task" type, that have a different task bar color and task name in bold. I perfectly understand that with this simple implementation the plan view might easily loose consistency in case of re-ordering the list items, and also main task duration and progress could be inconsistent with the other task, but this task-subtask differentiation at presentation level only, wold be in any case very useful
thank you again
paolo (paobrunelli)


Check outbound healthz endpoint on sidecar wait

Feature request
Similar to dotnet sdk check the outbound healthz endpoint for checking if sidecar is ready.
https://github.com/dapr/dotnet-sdk/blob/master/src/Dapr.Client/DaprClientGrpc.cs#L1790
Even if the port is open, Dapr might not be ready yet.
Current behavior
Python only waits to check if the port is available and socket connection can be created.
https://github.com/dapr/python-sdk/blob/master/dapr/clients/grpc/client.py#L1431
RELEASE NOTE:

@dapr/maintainers-python-sdk For this one, can I modify the existing wait method in the gRPC clients (sync and async), to directly check against the HTTP endpoint "/v1.0/healthz/outbound" instead of the the socket check that we do right now.
This will change the behavior of wait method, but will not be a breaking change since this was supposed to be the actual health check and current implementation is considered a bug.

If the appconfig Dapr configuration is not found in default namespace for some reason, the dapr.io/config parameter will not be set and skipped emitting a warning:
⚠  Dapr configuration "appconfig" not found in namespace "default". Skipping annotation "dapr.io/config"

Additionally, you also need to use this in the Dapr Actor Client and Dapr Service Invocation Client.

Based on the current usages, I only see the wait method being called in examples. Can you tell me why we need to use this in Actor Client and Service Invocation Client?

Additionally, you also need to use this in the Dapr Actor Client and Dapr Service Invocation Client.

Based on the current usages, I only see the wait method being called in examples. Can you tell me why we need to use this in Actor Client and Service Invocation Client?

It might be nice for anything which talks to the sidecar to first wait to make sure Dapr is ready. So if it's missing it should be added I think.

Done in https://github.com/dapr/python-sdk/pull/670


Add option to specify what encoding to use in the database field

I've been having some trouble retrieving cleartext from this module when persisting to Postgres. Everything seems to work in isolation but when it runs together I'm getting some kind of automatic transcoding happening. Text appears in the database in what looks like hex, but isn't quite and then errors when I try to decrypt it. The problem seems to be in Postgres' handling of binary blobs but I haven't managed to pin it down.
To work around the problem I've added an option to specify what encoding to use in sequelize-encrypted, so that I can persist the ciphertext as base64 or hex which sidesteps the problem.
var encryptedFields = require('sequelize-encrypted')(
    sequelize,
    key,
    {
        db_encoding : 'base64'    //optional
    }
);

I've made the option optional, so this should be a non-breaking change with respect to the arguments passed when you instantiate it... but couldn't find any documentation for cypher.end() and cypher.read() so used cypher.final() which seems to do the same job. I'm a bit concerned that I can't find documenation for those method calls though, I looked at node 0.12 all the way back to 0.4. Normally I would have left it alone and kept the diff line count down but couldn't find the docs for those calls. Expecting you to tell me I've missed something obvious.

After updating the PR, please squash the commits.

My interest in this PR has now waned. Your call what to do with it. I'll close for now but feel free to use it if you're bothered.


Word break rewriter: move protected words config under 'decompound'

Currently the words protected from decompounding are configured as a direct property of the config object in Querqy for Solr.  It should be a child of decompound as it is not applicable to compounding.
{
    "class": "querqy.solr.rewriter.wordbreak.WordBreakCompoundRewriterFactory",
    "config": {
         "dictionaryField" :  "dictionary",
         "lowerCaseInput": true,
         "decompound": {
             "maxExpansions": 5,
             "verifyCollation": true
         },
         "morphology": "GERMAN",
         "reverseCompoundTriggerWords": ["for"],
         "protectedWords": ["slipper"]

    }
}

I would think a "protected word" would also be a false positive for the compounder, wouldn't it?

Indeed, thank you! - Closing this issue


Add text to the error event return value

An error occurred difficult to import the copied text.
In a more simple way, The error event will be stoked text.
example)

How should you think?

Merged, thanks @kimorkim!


Support for 0-Dimensional Tensors in Burn

Motivation
Currently, the Burn deep learning framework in Rust lacks support for 0-dimensional tensors (scalars). Adding support for 0-dimensional tensors would enhance the framework's capabilities and provide several benefits:

Completeness: Supporting 0-dimensional tensors would make Burn more complete and consistent with other deep learning frameworks that already support scalars. Notably, ONNX (Open Neural Network Exchange) format, widely used for interoperability between frameworks, often deals with 0-dimensional tensors.

Simplified Operations: Many deep learning operations, such as loss functions and regularization terms, often involve scalar values. Loss functions, in particular, are crucial for training models and are typically represented as 0-dimensional tensors. Having native support for 0-dimensional tensors would simplify the implementation and usage of such operations, making it easier to compute and optimize losses during training.

Interoperability: Seamless integration with other libraries and frameworks that utilize 0-dimensional tensors would be improved, enabling smoother interoperability and data exchange. This is particularly important when working with ONNX models that frequently incorporate 0-dimensional tensors.

Reduced Workarounds: Without 0-dimensional tensor support, users may need to resort to workarounds like using 1-dimensional tensors with a single element, which can be less intuitive and efficient.

Avoiding Unnecessary Data Copying: By supporting 0-dimensional tensors directly, Burn can avoid unnecessary data copying from the device (e.g., GPU) to the host (CPU) and vice versa. This can lead to improved performance and reduced memory overhead, especially when dealing with large-scale models and datasets.

Proposed Solution
To address this limitation, we propose the following:

Extend the Tensor struct in Burn to support 0-dimensional tensors.
Implement necessary methods and traits for creating, manipulating, and operating on 0-dimensional tensors.
Update relevant functions and operations to handle 0-dimensional tensors correctly, with a focus on loss computation and optimization.
Ensure proper broadcasting and type promotion rules are followed when mixing 0-dimensional tensors with higher-dimensional tensors.
Add comprehensive unit tests to verify the correctness and consistency of 0-dimensional tensor support, including tests specifically related to loss functions.
Update the documentation and examples to showcase the usage of 0-dimensional tensors, particularly in the context of loss computation and ONNX interoperability.

Benefits
By implementing support for 0-dimensional tensors, Burn will:

Provide a more complete and consistent API for tensor operations, aligning with ONNX and other frameworks.
Simplify the implementation of common deep learning operations involving scalars, especially loss functions.
Enhance interoperability with other libraries and frameworks, particularly when working with ONNX models.
Improve usability and reduce the need for workarounds.
Optimize performance by avoiding unnecessary data copying between devices.

Potential Challenges

Ensuring backward compatibility with existing code and models.
Handling edge cases and maintaining consistency with broadcasting rules.
Optimizing performance for operations involving 0-dimensional tensors, especially in the context of loss computation.

Next Steps

Discuss and refine the proposed solution with the Burn community.
Create a detailed implementation plan and allocate resources.
Implement the necessary changes and additions to support 0-dimensional tensors, with a focus on loss computation and ONNX compatibility.
Conduct thorough testing and address any issues or edge cases, including tests for loss functions and ONNX interoperability.
Update the documentation and examples, highlighting the usage of 0-dimensional tensors in loss computation and ONNX scenarios.
Release a new version of Burn with 0-dimensional tensor support.

We believe that adding support for 0-dimensional tensors will significantly enhance the capabilities and usability of the Burn deep learning framework in Rust, particularly in the context of loss computation and ONNX interoperability. We look forward to feedback and collaboration from the community to make this feature a reality.

CC @nathanielsimard , @laggui , @louisfd

@LaurentMazare has confirmed Candle supports 0D tensors:

Zermelo Fraenkel: Scalars values (tensors with 0 dimension) should be supported. Empty tensors (multiple dimensions but with one of them being zero) should also be supported but only to some extent. Certainly interested if you find places where this doesn't work properly.

@laggui has said NDarray does not support it:

laggui: NdArray doesn't though

PyTorch supports 0D tensors

cc @ashdtu

@laggui found that Ndarray supports 0dim arrays: https://docs.rs/ndarray/latest/ndarray/type.Array0.html

@nathanielsimard and I had an offline conversation.
Here's a summary of the conversation for others:
We discussed the need to support scalar tensors in the Burn deep learning framework. While scalar values can be encoded as rank-1 tensors, the main issue is the lack of an automatic broadcasting API in Rust stable due to limitations with const generics.
As a better long-term solution, we proposed introducing a new Scalar type, which would be an enum that can hold either a native value (e.g., f32) or a rank-1 tensor. This explicit Scalar type would provide more security and avoid unnecessary broadcast operations. It would also be beneficial for exporting to other formats like ONNX, since all operation can be tracked in a computation graph.
We plan to modify the burn_tensor module to include this Scalar type, with variants like Scalar<Int>, Scalar<Float>, and Scalar<Bool>. This change would not introduce any breaking changes to the existing API.
Overall, while the naming and exact implementation details still need to be finalized, we agreed that introducing a dedicated Scalar type is a good idea to handle scalar values properly in the Burn framework.

While most of the current discussion has been around 0-dim tensors for scalar support, we also had two other supporting arguments around that on discord this morning:

Sparse tensors (when a tensor has only zeros in a dimension, that dimension would become 0 when converted to its sparse representation)
Current ops like argwhere() already return 0-d tensors for inputs that are all zeros (#2212 fixed the crash for the operation results, but using that tensor with other operations would lead to other problems)


Guides

Description of the new feature / enhancement
Display vertical and horizontal guides that float on top of all else on the screen.
Scenario when this would be used?
A screen guide system, useful to help align objects in a multitude of applications and to display the position in pixels on the x and y axes.
Supporting information
No response

/dup #24944


Fix formatting in the charts readme page

Currently, the page is rendered as shown below

with this PR, it will be displayed as follows.

@sridhargaddam
I did the sh blocks indentation just because it appears like that in the bundle docs - https://raw.githubusercontent.com/istio-ecosystem/sail-operator/refs/heads/main/bundle/README.md
But feel free to modify it as you prefer.

But feel free to modify it as you prefer.

The bundle readme was getting rendered properly. But anyways, just to keep it consistent, I updated the bundle readme as well. Thanks for pointing.


Use database helpers from mariadb-operator/api

The lib-common/modules/database is moved to mariadb-operator/api to remove a circular dependency.

/retest
Init container ci-scheduling-dns-wait not ready: (state: waiting, reason: "PodInitializing", message: "") Init container clonerefs not ready: (state: waiting, reason: "PodInitializing", message: "") Init container initupload not ready: (state: waiting, reason: "PodInitializing", message: "") Init container place-entrypoint not ready: (state: waiting, reason: "PodInitializing", message: "")


Add defaults on forms and fields

Closes #103

Codecov Report

Merging #104 (51f19a9) into main (92b4cb4) will increase coverage by 0.69%.
The diff coverage is 87.27%.

@@            Coverage Diff             @@
##             main     #104      +/-   ##
==========================================
+ Coverage   90.65%   91.34%   +0.69%
==========================================
  Files          14       14
  Lines         931      971      +40
  Branches      125      132       +7
==========================================
+ Hits          844      887      +43
+ Misses         66       65       -1
+ Partials       21       19       -2

Impacted Files
Coverage Δ

formlessness/constraints.py
86.74% <63.15%> (+0.51%)
:arrow_up:

formlessness/base_classes.py
90.36% <100.00%> (+0.36%)
:arrow_up:

formlessness/fields.py
99.26% <100.00%> (+0.03%)
:arrow_up:

formlessness/forms.py
86.66% <100.00%> (+6.93%)
:arrow_up:

formlessness/utils.py
84.61% <100.00%> (+1.28%)
:arrow_up:

tests/test_forms.py
100.00% <100.00%> (ø)

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 92b4cb4...51f19a9. Read the comment docs.


Detect spectrum wavelength input units

Provide a way to allow the wavelength to be in different units, like micrometers, angstrom, nanometers, etc. Maybe automatically detect the units in "read_spectrum" according to the order of magnitude of the spectral axis.

Also must include some estimation of the order of magnitude of the wavelength elements and compare it to an expected order of magnitude. Include a Raise error if the orders of magnitude do not match.
Also, include a warning when the dispersion is not linear. For this there must be some calculation of the differences between pixels.


stan: Error in system(cmd, intern = !verbose) : 'C:/Program' not found

from user:
setwd("e:/hmad")

emad_dat <- list(mad = 50)

library(rstan)
fit <- stan(file = 'e:/hmad/EMAD1.stan', data = emad_dat,
            iter = 10000, chains = 4)

I get the error:

Error in system(cmd, intern = !verbose) : 'C:/Program' not found

We don't use system anywhere, so it looks like a Stan bug - probably due to installation path having spaces? I'd ask to see if this repros with Rgui.

he said it works in RS.

What's actually failing here is inline::cxxfunction. Easier repro:
fx <- cxxfunction(signature(x = "integer", y = "numeric"), 'return ScalarReal( INTEGER(x)[0] * REAL(y)[0] )')

The root cause is that this code assumes that R.home() never has spaces in it, and so doesn't quote it. It works in Rgui and RS, because they use short path. In Rgui/RS:
> R.home()
[1] "C:/PROGRA~1/R/R-32~1.3"

In RTVS:
> R.home()
[1] "C:/Program Files/R/R-3.2.3"


diary: note refs can be arbitrarily long

Going to mention here since this seems to be an issue for references to notebook posts. The reference that I experience when looking at a notebook post that is just plain text (because it is pure markdown that isn't being parsed, see #1610), the reference is to the entire notebook post and that makes the 'continue reading' button redundant.
basically channel has a nearly endless scroll that looks like this for a recent post:

Originally posted by @thelifeandtimes in https://github.com/tloncorp/landscape-apps/issues/606#issuecomment-1369381508

Addressing this as part of #1664


l3cam_ros: 0.0.3-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository l3cam_ros to 0.0.3-1:

upstream repository: https://github.com/beamaginelidar/l3cam_ros
release repository: https://github.com/beamaginelidar/l3cam_ros.git
distro file: noetic/distribution.yaml
bloom version: 0.11.2
previous version for package: null

l3cam_ros
* Added allied Wide and Narrow cameras
* Implemented default parameters
* Bug fixes
* Disable dynamic network configuration
* Contributors: Adrià Subirana

Superceded by #36578
Please close duplicate PRs so that they don't conflict and require manual fixes.


Machine agent install seems to look for db-agent path

Hello - I'm trying to install the machine agent on linux, but it appears to look for the db-agent path.  Am I missing something here?
#ansible-playbook -i hosts-linux machine.yaml
.
.
.
TASK [appdynamics.agents.logger : Copy the log4j file down to update it localy] *************************************************************************
fatal: [<IP_ADDRESS>]: FAILED! => {"changed": false, "msg": "file not found: /opt/appdynamics/db-agent/conf/logging/log4j.xml"}

You won't face that issue if you try to run the automation using latest version ansible

FYI @v-zhuravlev

hi @pk-z1 , thanks for reporting. It Would be great if you can provide more info in order to replicate this issue:
ansible version
appdynamics-ansible collection version
machine.yaml contents if it is custom playbook
any machine agent related variables you may set elsewhere( in inventory, vars file etc)
@jpadmin , do you mean ansible version or collection version?

Can confirm this error for ansible 2.9

as a workaround, please upgrade to 2.10 + or remove agent_log_level: debug from vars or change to info

Thanks @v-zhuravlev - I also validated that agent_log_level: "info" works as well.

@v-zhuravlev , I can confirm that the automation works perfectly using latest ansible software with python3 as the interpreter

Found what was the issue in 2.9. Fixed in #72 . Will be available in the next release.

available in 21.6.0


Modal transition & animation settings

Description
This branch adds a default fade transition to all modals along with options for animated movement. Screens attached below, though the effect should be easier to see on-page.
Transitions are controlled through data attributes. Using this implementation as a test case for an extended Sage animation library.
Notes:

huge thanks to @philschanely for assistance with the React version 🙇🏼
added ability to disable background blur: in testing, the css filter tends to increase GPU/CPU usage when animating, resulting in dropped frames

Screenshots

Before
After

Test notes
In order to implement these transitions, the inactive modal CSS was changed from display: none to instead use visibility: hidden and a negative z-index. This should still be sufficient to "hide" the modal from accidental user interaction, but we'll need to ensure the stacking order is working as expected depending on placement of the modal's markup. In Rails, if SageModal is nested within a <% content_for :modal do %>, there should not be an issue.
All transitions are disabled when "reduce motion" is enabled on a user's OS, or on browsers that support the feature. Modal backdrop opacity is also increased for additional contrast.
Steps to test

Navigate to http://localhost:4000/pages/object/modal
Open some modals
(optional) Enable "reduce motion" in your MacOS system preferences -> Accessibility (screen below). All transitions should now be disabled

Estimated impact

(HIGH) Modal CSS display property has been altered to allow for transitioning effects. Confirm that modal is displayed on page as expected, and cannot be accessed via user interaction or keyboard navigation. Examples of use:

[x] Beta features modal
[x] Offer price modal
[x] Create site modal
[ ] Coupons: delete Coupon modal
[ ] Coupons: add Offer modal

Related

Is the fading out of the modal something we'd be able to handle in a followup? I know there's some complexity around incorporating requestAnimationFrame, so it may be more trouble than worth currently.

Great call, that's something I'd like to see as well. I considered adding an new event before sage.modal.closeAll to help set off a closing timer, but it seemed better suited for a follow-up.


Fix node modules not working properly in EnderApps

Re-write the modules manager in EnderInstaller

https://github.com/EnderAdel/EnderFramework/pull/165 https://github.com/EnderAdel/EnderFramework/pull/166 https://github.com/EnderAdel/EnderFramework/pull/167


Fixes ENYO-527

Issue
Need to provide API to determine which controls are visible for a DataList and DataGridList.
Fix
Add getVisibleControlRange() to enyo.DataList which returns the start and end index of the visible controls. For a control to be considered visible, a certain percentage (established by the visibleThreshold property) must be visible.
Enyo-DCO-1.1-Signed-off-by: Ryan Duffy<EMAIL_ADDRESS>

I can see the problem on garnet is fixed. It works well on garnet DataList


Question: how to disable Introspection query.

related to #118
How can i set NoIntrospectionGraphFieldVisibility?
schemaBuilder.fieldVisibility has been deprecated, so I should set it via graphql.schema.GraphQLCodeRegistry.Builder#fieldVisibility(graphql.schema.visibility.GraphqlFieldVisibility) instead.
I tried to set below code, but failed because codeRegistry already initialized.
        graphQLSchemaGenerator.withSchemaProcessors(
            object : GraphQLSchemaProcessor {
                override fun process(schemaBuilder: GraphQLSchema.Builder, buildContext: BuildContext): GraphQLSchema.Builder {
                    return schemaBuilder.codeRegistry(
                        GraphQLCodeRegistry.newCodeRegistry()
                            .fieldVisibility(NoIntrospectionGraphqlFieldVisibility.NO_INTROSPECTION_FIELD_VISIBILITY)
                            .build()
                    )
                }
            }
        )

I think if GraphQLSchemaGenerator provides getCodeRegistry() method, i can do it.
like below.
schemaGenerator.getCodeRegistry().fieldVisibility(NoIntrospectionGraphqlFieldVisibility.NO_INTROSPECTION_FIELD_VISIBILITY)

This did the trick for me:
GraphQLSchema schema = new GraphQLSchemaGenerator()
...
.withSchemaProcessors((schemaBuilder, buildContext) -> schemaBuilder.fieldVisibility(new NoIntrospectionGraphqlFieldVisibility()))

BuildContext allows access to GraphQLCodeRegistry.Builder, so you can use that from a GraphQLSchemaProcessor.
.withSchemaProcessors((schemaBuilder, buildContext) -> {
    buildContext.codeRegistry.fieldVisibility(NoIntrospectionGraphqlFieldVisibility.NO_INTROSPECTION_FIELD_VISIBILITY);
    return schemaBuilder;
})

Thank you @codelizrd @kaqqao .
resolved.


LP Callback Rejects...Info and backup pools

Still seeing LP callbacks being sent to miners when the block is reassigned to a pool, also the work being submitted is rejected.  This also happened to btcg when a block is reassigned, work being submitted is rejected.  Showing 3 / 3 100.00% for Eligius which is on info (shouldn't even have shares...) and 4 / 4 100.00% for Ars which is on backup.  Rejects only occur on reassignment, regular LP pushing has no issues.
http://pastebin.com/sNwU5dGQ

I think I just fixed this.

Deleting stats.db, git pull'ing and re-running.

Looks like the rejects are gone, and it's not trying to submit shares to info pools...but I'm not seeing the "LP Callback to miner:" anymore.  Also, a getwork to polmine occurred right after block assigned to slush.  Is this intended ?
http://pastebin.com/vx58XfME

I'm going to close this as it appears to be resolved.


Crew Elo system + Ranking Command

If the custom crews could be ranked using an Elo system, that'd be great.
If this happens, I'd also like a "!crew rankings" | Displays the top 5 crews(by Elo)
"Crew Rankings: crewname, crewname, crewname, crewname, crewname"
"[crewname] has just reached [1st, 2nd, 3rd, 4th, or 5th] place in the crew battle rankings! Congratulations!"
If a crew is already put in the system and hasn't played in a battle in over 2 weeks, it should have some kind of slow decay.
Should, be a setting.
I added it to the doc, but
!crewelodecay | Options: # of days, off | !elodecay
The elo should decay slowly. Probably once every 2 days after that decay point. Not sure the exact amount...
Also I was researching Elo systems and some have different numbers as different levels of "skill", if this kind of thing is done and you have an idea of what level is in relation to what; could you let me know?
Ex) for USCF: Class B = 1600-1799

https://en.wikipedia.org/wiki/Elo_rating_system for reference on mister Elo


Vendoring libnetwork 31139cdb513aea5ad1ed08b60d4350a68b4c96db

Closes #15081 #14859 #14738
(More test code present in libnetwork)
Signed-off-by: Alessandro Boch<EMAIL_ADDRESS>

@aboch sorry to be annoying, but would you mind putting the vendoring in a separate commit than the test?

@tiborvass Sure, I can do that. Just from my understanding, why is that needed ?

@aboch it's to keep things clean. That way we understand that a vendoring commit cannot have changed the docker codebase and vice-versa. Just makes things nice and clean.
Also makes it very easy to review what changes in docker when the vendoring is big.

@tiborvass Moved test code addition to a separate commit.
@calavera Took care of your comment.

LGTM

LGTM

cherry picked into #15091

Fixes #14908


const keyword in JavaScript

Could JerryScript support const keyword?
for example,
const ctx = wx.createCanvasContext('myCanvas')

// Create linear gradient
const grd = ctx.createLinearGradient(0, 0, 200, 0)
grd.addColorStop(0, 'red')
grd.addColorStop(1, 'white')

// Fill with gradient
ctx.setFillStyle(grd)
ctx.fillRect(10, 10, 150, 80)
ctx.draw()

The lexical environment rules for const and let makes it very hard to implement it in an Abstract-Syntax-Tree less environments. At least a somewhat efficient way. Until somebody came with a clever idea how to do this it will probably won't happen.

or just simply let 'const' as 'var'? Or add a read-only flag in var?

Possible but will work in a different way that people expect.

Yes. But better than parse failed.
How about the specification of 'const' key word in JavaScript?

I agree with @zherczeg. Non-standard behavior is something we should avoid. People usually expects standard behavior. You can transcript your ES2015 code to ES5.1 with babel before you run JerryScript. It does what you want (changes 'const' to 'var').

OK. I see. It's the feature of ES2015/ES6. Thank you!


Is the BadPVName exception enforcing an EPICS naming rule or just a convention?

I'm getting the BadPVName exception
BadPVName: SR05ID01MIR01:t.Y.RBV
raised when attempting a get() on an EpicsMotor.
Although most comments from colleagues suggest that this seems to be a poorly named PV, is it actually illegal in some way?
The original implementation comment suggests it's more of a guideline than a rule https://github.com/bluesky/ophyd/commit/1e973e162e926d67fbf73b1e03113888258da8e6
Our legacy code based on caget, which we still need to use, is quite happy to read it and I can't find any rule in the EPICS docs about dots in PV names being disallowed.
I think the current behaviour should be able to be bypassed if necessary if it is enforcing a convention rather than a rule.
We will try to create an alias of the PV for this case.

This code is improperly informed: https://github.com/bluesky/ophyd/blob/030c668da64443c54e17679f3342df371c52a620/ophyd/utils/epics_pvs.py#L74-L75
While very unusual, EPICS allows more than one . in an EPICS process variable name.
Thanks for uncovering this little gem.  I had not come across it before.  The last common instances of double-dot PVs was excised from the APS control systems many years ago, before ophyd was conceived.
I'm searching for the naming rules in the EPICS documentation.  Last time I asked this topic, I recall the answer was not as clear as I hoped (hoped for a regular expression, got prose instead).

@anjohnson : Can you provide a reference here to the rules about naming of EPICS process variables?  If it makes a difference this issue is discussing Channel Access PV names.

CA and PVA servers have their own different limitations and rules for PV names (also called channel names, the terms mean the same thing); the rules for an IOC server are different than for a PCAS or PVA servers. Thus the names you can use depend on the server that publishes the PV, and a generic client application should generally accept and send any string that the user enters (unless it only supports talking to an EPICS motor record say).
A motor record is an object that lives in an IOC, so the characters up to the first . specify the record name, and are subject to the limitations described here. A . character is optional at the end of the PV name, in which case the channel will connect to the record's VAL field.
When pointing to a specific field of an IOC record the . is required as a separator and followed by the field name; field names have to follow the same rules as a C-99 identifier, see here for more information.
When connecting to an IOC running EPICS 3.15 or later, the field name (or the . which is required in this case) may be followed by a field modifier, see here for the modifiers supported by EPICS 7.0.8.
A JSON field modifier may include . characters, although only when inside a JSON value (string or number). Thus the PV name that @gazzar is trying to connect to above couldn't belong to an EPICS motor record, but it might refer to a legal channel on some other CA or PVA server.
HTH!

@anjohnson Thanks!  That helps me.  We're only talking here about CA (not PVA).
From the first link, notice that a "record instance" does not include the . (dot) character.  (Note also that { and } are absent from this list.)  A "PV name" can include both the record instance name and .+field name.  When the latter is not provided, the CA server defaults to .VAL.

@anjohnson - I notice there is no advice about "field names". (Perhaps that advice is here?)  I assume that such names do not have . is a convention?  I recall some IOC software that preceded the EPICS areaDetector provided fields with embedded . characters.

Field names are set by the record type. I wrote

field names have to follow the same rules as a C-99 identifier, see here for more information.

The standard rules for C identifiers can be found here, but apparently I should have said C-89 instead of C-99 because even that version allowed unicode characters to be used in identifiers (who knew?). On that page you have to set the "Standard revision" pull-down at the top to C-89 to see our definition as we haven't tried using unicode characters in field names, and even if the Perl code processes them correctly the result will almost certainly not be portable to all our target compilers.

Note that I'm reserving the use of subsequent . characters as field modifiers in case we want to add support for DBF_STRUCT field-types and allow substructure access to the IOC in the future. That would be years away though, don't hold your breath…

Thanks for picking up the discussion around this @prjemian. @anjohnson is correct that this isn't a motor record but a field of a table record (specifically the y translation drive field), which is implemented as a pseudomotor. It has turned out that I can safely redefine my Ophyd device as an EpicsSignalRO in this case, because that position is only being read from for now in my code. However I did try the monkeypatching approach, which works and will be useful if I decide to redefine the device as an EpicsMotor; Thanks @ZLLentz for a practical workaround.


Add generic create_spoken_forms function

Let's create one version of create_spoken_forms that can be used in many places.  This would be a function that you can pass in a string, and it returns a list of possible ways to speak the given string.  I think @rexroni's speakify is a great place to start.  Here are a few things I can think of to support off the top of my head:

[x] numbers decomposed into single digits
[ ] for numbers in the range 1900–2100, support "nineteen eighty five", "two thousand five", and "twenty nineteen"
[x] pull abbreviations from abbreviate.py.  That should prob use the new csv stuff fwiw
[x] file extensions, which should maybe also be a list and use csv support.  I made an initial pass at making them a list fwiw
[x] split camel / snake into words
[x] support saying subsequences of the given phrase rather than the whole thing
[ ] symbols, ideally pulling from your symbol map in keys.py, which could prob use a csv re-work as well, though I think that's prob going to cause a merge conflict for every single knausj user 😅
[ ] do we want the prefix fanciness?
[ ] potentially need to toy with some of this type of replacement, though maybe let's start without it and see how necessary it is with something like conformer

The places I can think of offhand to use this function are:

[x] focusing an application
[ ] launching an application
[ ] file manager files and folders
[ ] #387
[ ] @Gauteab's tree-sitter identifiers in https://github.com/Gauteab/talon-tree-sitter-service
[ ] I could use them for tags and lists in my things3 grammar

In terms of naming, I've seen the following names:

create_spoken_forms
create_spoken_form
make_speakable
speakify

I think I'd vote for create_spoken_forms but they all seem fine to me

One small note from #399: it'd be nice to preserve Dragon's number parsing for those that might stick with it.

From testing around this, these are my thoughts:

I don't see an immediate need for the prefix fanciness(?), but I do think we should generate the variants with and without symbols provided it's not too expensive.

some-file -> some file, some dash file

I would probably vote in favor of generating the variants with all specified spoken forms of a symbol/digit, but maybe that's a bad idea in practice? we might want some form of "preferred spoken form" thing otherwise

I've got an initial pass of this working with the filemanager etc in my personal branch that I can merge into the revelant PR eventually. Generally, this is working very well save for the few known issues.


fix pturning init_process_group error

fix error: Default process group has not been initialized, please make sure to call init_process_group

Are you using win or Linux?

Are you using win or Linux?

Linux


Automatic remediation

Is the deployIfNotExists policy able to do the remediation automatically or does the remediation task can be created only manually?

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

ID: 99d433a1-4e81-e226-e835-639daa90dab4
Version Independent ID: ab3aa1ad-f438-e2a1-5556-9f9b6269ba76
Content: Remediate non-compliant resources with Azure Policy
Content Source: articles/governance/policy/how-to/remediate-resources.md
Service: azure-policy
GitHub Login: @DCtheGeek
Microsoft Alias: dacoulte

@diwsun Thanks for the feedback! We are currently investigating and will update you shortly.

@diwsun Currently, performing remediation is only done with manual steps.  This lets you use the remediation against a specific set of resources as both testing, and controlled roll-out of the fix.  It might be possible in the future to do this automatically.  If this is a feature you want, please post it over on the UserVoice for Azure Governance. Since this is a product question / feature request and not an issue with the docs, I'm going to have the team close this.  Thanks again!
@MicrosoftDocs/azure-cxp-triage Please assign to me and #please-close.

Hi all,
is auto remediation possible now?
Thanks!

is auto remediation possible?

Hi @bhushan7337 ,
auto remediation for "deployIfNotexist" as well as "modify" policies ist enabled by default, however just for resources created after policy enablement. All resources that you have created before, you can remediate through the Azure Policy Option "Remediate" (within the Azure Portal -> Policy). Hope that helps.

Hi @dani3lheidemann , we are working on policy and noticed that remediation task only works once when it deploys. We are getting non compliant status after 24 hours however "deployifnotexist" remediation is not working. We need to manually create a remediation task again to remediate the same.
Is it still manual?

Hi @paggarwalmsft, as far as I know, auto remediation only works on new resources.
@YutongTie-MSFT do you have any update for us on auto-remediation of existing resources?

Hi @dani3lheidemann  Policy do not get retrigger after 24 hours. We applied policy and created a remediation task. It fixes all existing with reference to time we created remediation task. After 24 hours we get a new non compliant data however remediation do not trigger itself. It even allow us to create resources with non complaint configuration and do not remediate after 24 hours

@paggarwalmsft yeah that's exactly what I know about auto remediation so far. There's no retrigger based on events or time configured. Remediation only works on new resources. I hope that Microsoft will fix this.

@YutongTie-MSFT or @dani3lheidemann is this still a manual work. Do we need to create tasks if the resource goes non-complaint after 24 hours again.

Thanks, David. I followed up with Neha to confirm, after which I'll update the page. Have a great day, Tim
Tim Warner
(he/him)
Senior Content Developer, Azure Governance
Microsoft Content & Learning
<EMAIL_ADDRESS>Based in Nashville, TN (CDT)
[Microsoft Logo]
From: David Coulter @.>
Sent: Monday, June 20, 2022 10:29 AM
To: MicrosoftDocs/azure-docs @.>
Cc: Tim Warner @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Automatic remediation (#17764)
The Azure Policy integration with Event Gridhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fazure%2Fgovernance%2Fpolicy%2Fconcepts%2Fevent-overview&data=05|01|timwarner%40microsoft.com|b536969e313b4fa0264f08da52d19f29|72f988bf86f141af91ab2d7cd011db47|1|0|637913357521028823|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=P6VqH13jl%2B0YsJR7pseAYSQ8APm4vJ0qBCzyShBW4nY%3D&reserved=0 should cover the ability to trigger remediation on non-compliant resources. Tagging @timwarner-msfthttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftimwarner-msft&data=05|01|timwarner%40microsoft.com|b536969e313b4fa0264f08da52d19f29|72f988bf86f141af91ab2d7cd011db47|1|0|637913357521028823|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=UOWpk%2F2Whhb9MzRJ%2FTMt69IoPN%2FPJpv%2FWPuLIbrvGw0%3D&reserved=0, who now owns this content, for awareness of the discussion and possible improvements to the content.

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F17764%23issuecomment-1160587325&data=05|01|timwarner%40microsoft.com|b536969e313b4fa0264f08da52d19f29|72f988bf86f141af91ab2d7cd011db47|1|0|637913357521028823|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=JFpwDhUDosI5so3CSKKqqclRsSqQnx6P4mU6%2BVrVDPc%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAYECLA7JO2EM6IQSWAKZB6TVQCE4LANCNFSM4F7XGE5Q&data=05|01|timwarner%40microsoft.com|b536969e313b4fa0264f08da52d19f29|72f988bf86f141af91ab2d7cd011db47|1|0|637913357521028823|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=V8ID%2Fw2wRCFgAGN2H0OTv1cxf7SpUFfjOuHD%2BVprb6Q%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@DCtheGeek Thanks for the real quick turn around. So right now we don't have auto remediation to handle this scenario directly in azure policy and the only possible way it to use event grid to react on events happening in azure policy. Is my understanding correct?

@IaminAzure I'll defer to @timwarner-msft for that since I don't own this content anymore. As he mentioned, he's going to check with the PM and then will respond back to the thread and update the content as needed.

Hi @IaminAzure, thanks for the follow up! I am a PM on the Azure Policy PG team, and @timwarner-msft got in touch with me about this thread.
Your understanding above is correct - Policy does not support automatic remediation for brownfield resources because is could introduce risk of harm to the environment. Remediation tasks should be explicitly and intentionally triggered by the user, so event grid integration is our guidance for doing so in a non-manual way. Remediation can be automated and managed in a controlled manner through event grid integration, by creating a pipeline to trigger remediation tasks through Azure Functions, Azure Logic Apps, etc. when policy state change events occur. See Reacting to Azure Policy state change events - Azure Policy | Microsoft Docs
Do you feel you have the information you need to set up this integration? We are happy to improve our content based on feedback on the documentation.

Hello @shanhix1, would this approach also work when managing policy compliance for resources in multiple tenants through Azure Lighthouse?
I've assigned a policy in the customer tenants at the root management group level that is intended to enroll all subscriptions into our offering in Azure Lighthouse. The policy has deployIfNotExists and a remediation defined for enrolling subscriptions into Azure Lighthouse.
The hope is that any new subscriptions would be out of compliance with the policy and they would be automatically enrolled into our offering in Azure Lighthouse through the "deployIfNotExists" - remediation task relationship.

@brfabia thanks for the response! I got around that limitation by writing a deployment/onboarding script for customer tenants in PowerShell. The onboarding script uses the "owner" permissions of the customer executing the script to create the Lighthouse enrollment policy and assign it at the Root Management Group level. This policy has "deployIfNotExists" specified and the script starts a compliance scan after assignment, defines a remediation task, and initiates it.


kube-aws: don't append trailing dot to externalDNSName

We previously automatically appended trailing dots to externalDNSName
in the config struct, which was then reflected in kubeconfig.
This caused some problems with people using /etc/hosts, since they would
add the externalDNSName to their hosts file without the trailing dot,
and /etc/hosts lookups don't append one.
Now we only add trailing dots on the fly, whenever they're necessary to
interact with the AWS API, never anywhere persistent.

LGTM


Vue mode: support more style and script types

I added the second commit mostly to support <script type="text/ecmascript-6"> which, from what I understood, was necessary in some old version of Webstorm.
Feel free to merge only the first commit.

Looks harmless enough. Merged as 60f39516

Thank you.


Sample Update - Quick fixes to the script sample

What's in the pull request

Quick fixes to the script sample: Hide the 'See All' Button in the Highlighted Content Web Part

Awesome quality checks and fixes, thank you @ganesh-sanap - this will be live shortly 😁✨


Make fuzzing runs introspectable/debuggable

I forgot what I had in mind here, probably check stuff like "live-coverage" and where a fuzzing campaign is stuck. Will close for now. If wanted/needed we can create a new ticket


allow relative paths in meta loop

🚀 Feature Proposal
(A clear and concise description of what the feature is.)
if I want to run a node command pointing to a javascript file I have to use the fully qualified path when I issue: meta loop "node /some/fully/qualified/path", it would be nice if one - I didnt have to quote the thing at all and two if relative paths could be used and have the plugin sort out the diff in the relative path from where the target repo is so that the node command wont fail when changing directories.
Motivation
Simply more useful.  The whole point of this tool is to be able to run command over multiple dirs.  So this would make it much simpler to use rather than just passing the whole quoted cmd line to commander or whatever.
Example
Say I have a scripts dir in one of my repos I want to run against all repos.
I would like to be able to say meta loop node ./bin/scripts/dosomething.js,
Rather than meta loop "node /Users/foo/long path to js file" since the script path is not relative to my other projects paths

This is not the case. meta-loop runs with the cwd set to the child repo's directory. Run meta loop pwd to illustrate.

I will retest


Autocompletion suggests labeled args that were already assigned

(Followup to #175)
This was already solved for JSX props, but is still an issue for labeled args.
Example (only _to should be shown in the list as from was already assigned a value):

About what's involved:
The backwards parser for function calls currently only deals with nested ( and ), so it goes back to substring( and completes from it. So it never parses from.
What's needed is to parse all the arguments between substring( and ), with all the possible forms they can take, including args included in parents brackets etc, optional args, type annotations, punning.

Here: https://github.com/rescript-lang/rescript-vscode/blob/master/analysis/src/PartialParser.ml#L38


Bring BBspec back

cc @CheukHinHoJerry

This probably needs a PR at EQM?

Not necessarily. I think we can just read it off A2Bmap. Maybe I can try tonight.

Right - I just realized that should be easier. I am halfway so maybe I can take care of that?

Done by #205


Issues with integrating flowchart.js

I am seeing this error show up in the logs from rapheal.js on rendering the example you guys have. I have included rapheal.js already.
Error: Invalid value for  attribute preserveAspectRatio="meet"
Is this something you guys have noticed? Also I am seeing weird behavior where the same content is being rendered differently (non deterministic). See below. I will dig deeper into this issue but if you have any pointers it will be great.

Here goes the source:
renderSeqDiagram: function(){         if (this.state.content.flowDescription) {             try {                                         if(this.dia)                     {                         this.dia.clean();                     }                    var options = {'x': 0, 'y': 0};                     this.dia = flowchart.parse(this.state.content.flowDescription);                     this.dia.drawSVG(this.refs.canvas.getDOMNode(),options);                  } catch (error) {                     console.log(error);                     this.refs.canvas.getDOMNode().innerHTML = error;             }         }     },
I am building a react component.

@sobanhameed
I've been looking for a React library that wraps around flowchart.js and wonder if you have anything you might be able to share or point me to any open source project
So far this issue is the only thing that comes up in search
Any help/guidance greatly appreciated!

dmk23, see https://github.com/alwinn1977/react-simple-flowchart

If someone is still interested I have created this React wrapper for flowchart.js: react-flowchart.js
It has built-in TypeScript support and should support all the same features.

If someone is still interested I have created a React wrapper for flowchart.js: jln-dk/react-flowchart.js
Thanks to Adriano Raiano for creating this library! :)


[KYUUBI #1613] Record the engine's applicationId in KyuubiSessionEvent

Why are the changes needed?

Record the engine's applicationId in KyuubiSessionEvent. #1613
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[X] Add screenshots for manual tests if appropriate

[X] Run test locally before make a pull request

Codecov Report

Merging #1622 (24e001f) into master (54a1b88) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1622      +/-   ##
============================================
+ Coverage     58.23%   58.30%   +0.06%
  Complexity      164      164
============================================
  Files           258      258
  Lines         12803    12823      +20
  Branches       1608     1608
============================================
+ Hits           7456     7476      +20
  Misses         4717     4717
  Partials        630      630

Impacted Files
Coverage Δ

...ngine/spark/SparkThriftBinaryFrontendService.scala
81.69% <100.00%> (+4.10%)
:arrow_up:

.../apache/kyuubi/client/KyuubiSyncThriftClient.scala
83.19% <100.00%> (+0.73%)
:arrow_up:

.../org/apache/kyuubi/events/KyuubiSessionEvent.scala
92.59% <100.00%> (+0.28%)
:arrow_up:

.../org/apache/kyuubi/session/KyuubiSessionImpl.scala
84.48% <100.00%> (+0.27%)
:arrow_up:

Continue to review full report at Codecov.

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

thanks, merged to master


including jquery on a static html file results in the live server showing a blank page

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Other:

Current behavior
When including jquery, the live server shows a blank page with nothing contained in the  tag.
Expected behavior
When including jquery, everything in the  tag should appear. :)
Environment
Browser:

- [x] Chrome (desktop) version 111.0.5563.147
- [x] Firefox version 111.0.1
- [ ] Safari (desktop) version XX
- [ ] IE version XX
- [x] Edge version 111.0.1661.62

For Tooling issues:

- Live Server: 5.7.9
- Platform:  Windows 10
- Visual Studio Code: 1.77.0

Others
Note that I have attempted the same with several versions of jquery, including 2.0.0, 3.0.0 and the latest 3.6.4.
Example HTML that exhibits this behavior:
<!DOCTYPE html>

        <meta charset="utf-8">
        <meta name="viewport" content="width=device-width">
        <title>Live Server Test</title>
         :)


Make assemble changelog script enforce line length

Description
As I descovered the hard way, a changelog entry with a line length greater than 80 characters would still pass CI, although technically being invalid. This is a quick change to the script to make it detect these descrepancies and fail them.
Status
READY
Requires Backporting
YES
Migrations
NO
Todos

[x] Tests
[ ] Backports done

Steps to test or reproduce
scripts/assemble_changelog.py should now enforce the 80 character line length limit

The CI failure is not related to this PR. Good to go.


A question about code and data

Hello, I am very interested in your excellent work on AAAI-21. I would like to ask when the code and data set will be released on github?  Thank you very much!

Thanks for being interested in our paper. The code and data will be uploaded Sept, well, hopefully early Sept:).  Just too busy recently.

Datasets and codes are updated.


Nsk/7 19 19 synth lib improv

the commit history here is a little weird, not quite sure what happened. I waited like a year almost to merge from the last PR so who knows what happened. See #20


#470 fix cannot create product with product variant

root cause: product repository cannot save product variant with remainingQuantity null
2023-03-21T04:04:04.335Z ERROR [product,b646db7d6a0d639549eb057c6b4e9852,ada19929e2683be6] 1 --- [p-nio-80-exec-8] o.h.engine.jdbc.spi.SqlExceptionHelper   : ERROR: null value in column "remaining_quantity" of relation "product" violates not-null constraint
Fix: add remainingQuantity field to Product Variant

Actually we should remove the column remaining_quantity, and replace by the stockTrackingEnabled (boolean). And all the variations (if any) should inherit this value from parent product


Put node version limit in package.json

Consider adding it as a Shield.io badge in the Readme file too.

Related to #703
@aaronfrost should we go from node 12-13?

I think we should stay on whatever the version the Angular CLI supports. If they went to 13, then let's do it.

I think we should stay on whatever the version the Angular CLI supports. If they went to 13, then let's do it.

@SanderElias^^

@aaronfrost the cli demands a version like this:
{
  "engines": {
    "node": ">=10.13.0 <13.0.0",
  }
}

However, we have some dependencies in Scully (jsDom at least) that don't work properly anymore with version 10.
The polyfill proposed in #703 has serious TS issues, that might take at least a couple of hours to resolve
I would propose to go to:
{
  "engines": {
    "node": ">=12.0.0 <13.0.0",
  }
}

I'm not sure why the CLI doesn't include 13, as it seems to work fine.

Tested support for node 12 13 and 14, Scully works with all 3.
Node 10 isn't supported. Some of our dependencies demand globalThis, which is not in V10, and TS is making it really hard to polyfill.
Then we use fromEntries all over the place, and that is part of the es2019 library but does not get polyfilled by TS when we compile down to ES2018 anymore. (this worked with previous TS versions
Then we need this TS version for proper symbol support, so its not easy to go to an older TS version.
Supporting V10 would take at least 2 full day's of work


[Snyk] Fix for 1 vulnerabilities

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

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

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

768/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 7.5
Regular Expression Denial of Service (ReDoS) SNYK-JS-NODEFETCH-2964180
Yes
Proof of Concept

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

Commit messages

Package name: node-fetch
    The new version differs by 217 commits.

2880238 fix: ReDoS referrer (#1611)
e87b093 fix(Headers): don't forward secure headers on protocol change (#1599)
bcfb71c chore: remove triple-slash directives from typings (#1285) (#1287)
95165d5 fix spelling (#1602)
11b7033 fix: possibly flaky test (#1523)
4f43c9e fix: always warn Request.data (#1550)
1c5ed6b fix: undefined reference to response.body when aborted (#1578)
a92b5d5 fix: use space in accept-encoding values (#1572)
0f122b8 docs: fix formdata code example (#1562)
6ae9c76 docs(readme): response.clone() is not async (#1560)
043a5fc Fix leaking listeners (#1295) (#1474)
004b3ac fix: don't uppercase unknown methods (#1542)
c33e393 Fix Code of Conduct link in Readme. (#1532)
6875205 docs: Fix link markup to Options definition (#1525)
6425e20 fix: handle bom in text and json (#1482)
a4ea5f9 fix: add missing formdata export to types (#1518)
61b3b5a fix: cancel request example import (#1513)
5e78af3 Replace changelog with valid url (#1506)
9014db7 types: support `agent: false` (#1502)
2e1f3a5 chore: fix typo in credential error message (#1496)
4ce2ce5 docs(readme): fix typo (#1489)
ba23fd2 docs: remove the changelog (#1464)
8fedc1b core: move support and feature to discussion (#1471)
0b43b9f docs: update formdata example (#1465)

See the full diff

Package name: zx
    The new version differs by 4 commits.

7977cb5 Drop node 14.x, 15.x from tests
5578d67 Update deps and remove ts compile
0109e06 feat: add yaml dependency (#274)
1ad81de fix(test): add a pre-test script to build (#275)

See the full diff

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

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

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

D


Making JS mentions Node-specific

Also added a link to our security / data handling policy

:tada: This PR is included in version 0.98.0 :tada:
The release is available on:

v0.98.0
GitHub release

Your semantic-release bot :package::rocket:


Status subresource of BMH is getting big to fit in annotation

As part of pivoting operation, we are storing bareMetalHost status subresource data in the baremetalhost.metal3.io/status annotation so that BMO in the target cluster can continue reconciliation from point it was before pivoting. In some cases, where there are quite many multipath disks attached to a BMH, e.g, 800 then the BMH metadata.annotations field becomes too big, because it stores storage entry for each and every disk regardless of multipath. The problem with that is - there is a limit on how much data can be stored in metadata field and is currently 262144 bytes.
In one of our deployments we are hitting the issue with this limited size because of big number of multipath disks
metadata.annotations: Too long: must have at most 262144 bytes
Any thoughts on how to avoid it?
A couple of things we were thinking:

Don't store all the disks entries for multipath disks
Increase the data limit for the metadata field during CRD generation (not sure if possible to do it via some flags). I've started a discussion with Kubebuilder folks on this

As a workaround, based on https://github.com/argoproj/argo-cd/issues/5704#issuecomment-792337798

kubectl create -f  can be used instead if kubect apply -f   if the resource is being created 1st time
 kubectl replace -f can be used instead of kubect apply -f   if the resource is being created not 1st time

Ref: https://github.com/metal3-io/baremetal-operator/blob/master/docs/statusAnnotation.md
/cc @kashifest @maelk @zaneb @hardys @andfasano

Increase the data limit for the metadata field during CRD generation (not sure if possible to do it via some flags). I've started a discussion with Kubebuilder folks on this

Apparently it's not possible because it is a global limit

Don't store all the disks entries for multipath disks
+1

I'm copy-pasting @zaneb's comment here from the closed issue but the same topic.
As discussed in the meeting today, I think in the long term the answer is to move this data to a separate CRD so that we don't have to include it in the status annotation.
In the meantime, collapsing the multipath disks (whatever those are) seems like a reasonable approach. You could make an argument that that should be done in ironic-python-agent even /cc @dtantsur

I've tried to keep only status.hardware.nics, status.provisioning in the annotation, but that didn't help and nodes went for another inspection. How feasible it is to start thinking and working on another CRD to store inspection data? @zaneb @dtantsur ?

Proposal is opened in https://github.com/metal3-io/metal3-docs/pull/208 and ready for review.

/assign @fmuyassarov

/triage accepted

//lifecycle active

/lifecycle active


Baml exception error

WpfApp1.zip
<Window x:Class="WpfApp1.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:local="clr-namespace:WpfApp1"
        mc:Ignorable="d"
        Title="MainWindow" Height="450" Width="800">
    <Grid>
        <TextBlock>[123456</TextBlock>
    </Grid>
</Window>

Thank you. I am able to reproduce the error. I'll fix that asap.

@bprg Could you please confirm that this build fixes your issue?

This issue is fixed, Thank you.

Another issue with Comma and Close Bracket.
<TextBlock>,]</TextBlock>
I think the PropertyPath Parser should be rewritten.
https://referencesource.microsoft.com/#PresentationFramework/src/Framework/MS/Internal/Data/PathParser.cs

I agree. The thing is that I am not sure why it is even executed for those text blocks. I have yet to find reason why there would be a reference to a property in a plain text entry in the XAML.

I found the reason... there are specific cases when a path to the property is stored in the inner text, that occur for some control if they are directly data bound to XML data.
I am ripping of the implementation of the parser by Microsoft now to get parsing this stuff working properly.

I hope it works significantly better now. I basically copied the original parser from the .NET Framework. So this one should be able to handle the paths properly without problems. The new version will be available in this build once it is done.
@bprg Can you verify that the new version works for you?

Works properly, Thanks.

Very well. Merged it into the master.


Right click to run goal on target

This is based on some ability to determine what goals a target runs under.

Currently appear to be missing the Pants functionality to allow this to work. There doesn't seem to be a ./pants --filter-only=package list :: (or equivalent).

Dupe of #18


Reingest records that are on exhibition with a due date

See #2474
Victoria will provide a list.

Everything has been reingested since I raised this, so there is no longer any need to do so for a specific subset


Share code in parsers of committee commands

Changelog
- description: |
    Share code in parsers of committee commands. This doesn't share the CLI's behavior.
# uncomment types applicable to the change:
  type:
  # - feature        # introduces a new feature
  # - breaking       # the API has changed in a breaking way
  # - compatible     # the API has changed but is non-breaking
  # - optimisation   # measurable performance improvements
  - refactoring    # QoL changes
  # - bugfix         # fixes a defect
  # - test           # fixes/modifies tests
  # - maintenance    # not directly related to the code
  # - release        # related to a new release preparation
  # - documentation  # change in code docs, haddocks...

Context

While doing PRs recently in this part of the codebase, I realized that some places had code duplicated, without a good reason. Sharing will also fixes to affect multiple commands at once in the future, and also helps keeping different commands consistent.
Fixes https://github.com/IntersectMBO/cardano-cli/issues/811

More sharing is possible (e.g. hot VS cold), but I don't want to stack too much in one PR.
How to trust this PR

Golden files are not changed
The tight types means that if the code still compiles, and the types didn't change; then probably the runtime behavior didn't change either.
Each commit is standalone and makes sense on his own, to ease review. So it's advised to review commit per commit.
It's twice more deletions than additions

Checklist

[X] Commit sequence broadly makes sense and commits have useful messages
[X] Self-reviewed the diff

Thanks @palas, I'll look at your suggestions in another PR, I don't like big PRs :wink:


xy grid broken in Safari 10

On Safari 10 we get an empty page. It seems like a cell takes the whole screen dimension and the other cells are not visible.
The issue is not reproducible in Safari 11 and Chromium.
Is this a known issue and are there solutions?
We use Foundation Sites 6.4.3.

It seems Safari 10 needs some height: 100% for some cells as Safari 10 does not apply it automatically like it is in other browsers and after the spec.
In Safari 11 this is not needed anymore and it works directly without this workaround.

Sounds good @DanielRuf. I'll assign this one to you so you can decide what needs to happen.

Probably related to https://bugs.webkit.org/show_bug.cgi?id=137730 so I close this for now.


DF-457:Alt tag amend

Related ticket(s):
https://national-archives.atlassian.net/browse/DF-457
About these changes

Fixed missing divs
Removed problematic alt value, added alt without value to grid template
Restore and Update condition for teaser image
To look into further to refactor constants using bucket keys

How to check these changes
ALT value should not appear for any search results.
Before assigning to reviewer, please make sure you have

[x] Checked things thoroughly myself before handing over to reviewer.
[x] Included the ticket number in the PR title to help us keep track of changes.
[x] Ensured that PR includes only commits relevant to the ticket.
[x] Waited for all CI jobs to pass before requesting a review.
[x] Added/updated tests and documentation where relevant.

For Reviewer
Once PR is merged, check and arrange to deploy on platform-sh.

@mozaki1 / @jamesbiggs - Please can you review this PR. Thanks.

@JohnHeeryTNA , Please can you review FE side of this PR. Thanks.

Hi @TNA-Allan , I think we can completely remove the alt tags on these so the code changes would be:
templates/search/blocks/search_results__list-card--grid.html line 32   templates/search/blocks/search_results__list-card.html line 27

@sdwilkes : I understand that currently using description is not an accessible value. Will this be replaced with suitable one when available?  Under this scenario, I thought alt tag is a necessary element. Please can you confirm for this change - removal of the alt tag element or removal of alt tag value?


ci: add default shell to workflow draft-binaries.yml

Issue #3996
Current behavior
Need to repeat default shell declaration in the workflow
Proposed changes
Set a global default so that it is available across all jobs in the workflow.
Checks

[X] All commits in this Pull Request are signed and Verified by Github.
[X] All commits in this Pull Request follow the Ockam commit message convention.
[X] I accept the Ockam Community Code of Conduct.
[X] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository. The easiest way to do this is to edit the CONTRIBUTORS.csv file in the github web UI and create a PR, this will mark the commit as verified.

Not sure why the workflow failed, don’t think it’s relevant to this PR

@saputkin we have some timeout issues on some tests that we need to fix. Thanks for your help and this PR, it should be merged very soon now.


Short solution needed: "How to load data frame from Excel (XSL)" (python-pandas)

Please help us write most modern and shortest code solution for this issue:
How to load data frame from Excel (XSL) (technology: python-pandas)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to load data frame from Excel (XSL)


[Add Car] The "Brand", "Models", "Colors" dropdown items are displayed under the dropdowns.

Environment:  iPhone 12 Pro Max, OS 14.2
Reproducible: always
Build found:  build 1.0.77
Preconditions
The "Car" bug is opened
The 'My Profile' page is opened
Steps to reproduce

Go to the "Add Cars" page
Select the "Brand" dropdown
Pay attention to the dropdown items

Actual result
The "Brand", "Models", "Colors" dropdown items are displayed under the dropdowns.
Expected result
The "Brand", "Models", "Colors" dropdown items are displayed over the dropdowns.
User story and test case links
E.g.: "User story #33
Labels to be added
"Bug" ("bug"), Priority ("pri: ")

Fixed https://github.com/ita-social-projects/Car-Front-End/pull/586


Question: Catalog of all available trainings in the portal

Is there a catalog that list all of the available trainings in the Learning Portal? I'd like to recommend specific trainings for a customer's custom playlist. It would be helpful to provide the customer direct links or recommend trainings I know are available within the portal.
Question

[X ] Question
[ ] Bug
[ ] Content

@Celestero  -- no, the best way to get a listing of all the playlists is to review the administration view... CustomLearningAdmin.aspx


Add support for macOS

Would it make sense to add support for macOS?
This would require some type aliases, similarly as I've done in my BezierPath+Length project.
To be clear: I'd like to take this task upon me, but would like a green light before I start 🙂 to make sure it fits the vision for the VectorBoolean project.

That sounds like it'd be worthwhile if you're still interested. I've been extremely busy working on the project for which I created this port, so I'm only now able to get back to you. Sorry.


Why a fill rate ?

Why a filling rate when we define the number of population? If we define the population at 80 on a 10x10 grid, that's already 80%.
For the moment, if we define the population at 80 (10x10 grid) and a fill_rate at 0.6, int he "initilisation()" function we have:
nb_cell_dep = round(grid.size * fill_rate) #so equal to 60
nb_cell_dep //= len(total_population) #but here equal to 1
So I dont understand why we need a fill_rate when initializing the population at 80 make it 80%, and if we want to change this rate by 60%, finally we have 1% and then a loop :
for j in range(nb_cell_dep):
looping inly one time for the value : 0
Afterwards, I don't know this game, maybe I misunderstood something.

So the idea is that you chose the quantity of population usually between 1 and 4, because each individual only interacts with other individuals from the same population. Then filling rate is for the board, where each population starts with equal weight.
So for example 2 populations at 0.6 filling rate make a board with 30% tiles occupied by the first population, 30% occupied by the second population, and 40% of the board stays empty.
I suggest to check the wiki page that I will add a link for in the readme to understand how the game functions.


WIP: Minor inconsistencies discovered during the smithy migration

[ ] Sometimes we use a map for passing environment variables (https://build-server-protocol.github.io/docs/extensions/jvm#test-environment-request), sometimes we use a list (https://build-server-protocol.github.io/docs/extensions/scala#scala-specific-data-kinds-for-debugging)
[ ] JVM environment requests include an originId. Why?
[ ] Scala test classes request also has an originId (both in params and respons)
[ ] framework field in ScalaTestClassesItem is marked as optional, but with an annotation "It's optional in order to maintain compatibility, however it is expected from the newer implementations to not leave that field unspecified.". Let's make it obligatory then.
[ ] OnBuildShutdown should be named OnBuildExit
[ ] buildTarget/inverseSources in should be called textDocument/inverseSources
[ ] split TaskData into TaskStartData, TaskProgressData and TaskFinishData

Related: #562


No symbols archive for aspnetcore

Describe the Bug
After building the .NET 8 VMR, I see this:
$ find -iname '*symbols*tar*'
$ find artifacts -iname '*symbols*tar*'
artifacts/obj/x64/Release/blob-feed/assets/dotnet-runtime-symbols-fedora.38-x64-8.0.0-rc.2.23426.4.tar.gz
artifacts/x64/Release/dotnet-runtime-symbols-fedora.38-x64-8.0.0-rc.2.23426.4.tar.gz

This only shows runtime symbols. No symbol archive is produced for ASP.NET Core.
It looks like some symbols are produced for ASP.NET Core. I can see files like src/aspnetcore/artifacts/source-build/self/src/artifacts/bin/Microsoft.AspNetCore.Mvc/Release/net8.0/Microsoft.AspNetCore.Antiforgery.pdb on disk, but I don't know how to go from those pdb files to something that can be installed/shipped on a distro.
Steps to Reproduce

git clone https://github.com/dotnet/dotnet -b release/8.0.1xx
./prep.sh
./build.sh
find artifacts/ -iname '*symbols*tar'

Other Information
There's a related issue, https://github.com/dotnet/source-build/issues/3547, about how the symbols are produced and packaged. I think whatever approach we pick, we should make sure all symbols are made available post-build in the same format.

This issue has been resolved with the linked PRs.  Symbols will be available in RC2.


Improve SQLX_OFFLINE / cargo sqlx prepare workflow

I've transitioned to a workflow where SQLX_OFFLINE is the default because it improves compile times substantially. However, it feels like this workflow is not well supported with cargo sqlx prepare writing the cache from cache data if SQLX_OFFLINE is true. Luckily, after #732, there's an easy way to overwrite SQLX_OFFLINE (before, you could only unset the env variable but that wouldn't help when it was set in .env).
The last commit should allow running multiple cargo commands (e.g. IDE-spawned cargo check + cargo sqlx prepare) without the risk for broken query metadata files, as long as only one of them is running in online mode.

CI failure seems unrelated.

Looks good, thanks!


Change Text with Click Events - Test should also accept $('message').text('xxx').  Currently only accepts .html('xxx')

Challenge Change Text with Click Events has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/48.0.2564.109 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

Cat Photo Finder

      The message will go here

      <button id = "getMessage" class = "btn btn-primary">
        Get Message
      </button>

@dougwells Can you further explain what the issues are? I tried the above code and it appears to work.
I'd also recommend using another browser (Chrome or Safari) over Firefox.

Hi Kris,
My experience is that the code below is not accepted by the test engine.  Note, I am using .text() not the .html() that the test apparently prefers.  No biggie either way.  Just trying to help out as a member of the FCC community.  Thanks for you work helping to make FCC better with each passing month ...
Doug Wells

``

``

```

@ltegman is this valid? AFAIK .html() is for HTML elements and .text is for both HTML and XML. So theoretically, .text should be allowed but is this a good practice?

@raisedadead Yeah, either solution to this is valid. Since both work in HTML documents and we're not creating a child HTML element (which would require using .html) this should accept both methods.

My guiding principle when using .html() vs .text() hinges on whether I need to include an HTML tag or not.  If I need to include "Another line item then I use .html().  If not, I use .text().
In this challenge, I would actually argue that .text() is the "better" answer as no HTML tags are needed.
Just my opinion.  Fun to be involved in this positive discussion.  FCC is AWESOME!

Here is the difference from jQuery's docs:

.html() Get the HTML contents of the first element in the set of matched elements or set the HTML contents of every matched element.
.text() Get the combined text contents of each element in the set of matched elements, including their descendants, or set the text contents of the matched elements.

More specifically:

if you need to get/change the HTML tag and contents from an element, AND only need the first one found, use .html()
if you need to get/change the text AND are okay with getting all decedents of the selector, use .text()

In this case, I would conclude that since only the text of the selector is being updated, .text() would be the correct answer; however, technically each should pass this test.


Fix the Navbar

Your task is to fix and improve the navbar of the Forum website.
You have to route the navbar and also make the current open page's button primary.
Search icon is also missing in the navbar search input.
Also, you have to make the navbar responsive. In mobile view, only Forum name, a search icon(on clicking it, it should expand to open a search input box) and a hamburger like icon should be there.
On clicking that icon, options like home, communities, chat, profile, signout should be shown.
Guidelines

The code should be clean and reusable.

Note

Make sure you follow the given template before making the pull request.

claim

Issue open for claim

Claim

if no one claims i would claim it at night as i already have 3 issues

Claim

Go ahead @GammaMicrowave