Refactor run-dev and sbt commands

Description
Please explain the changes you made here.
Release notes:
Describe the change as it should be communicated to partners in the release notes.
Checklist
General

[ ] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >
[ ] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

User visible changes

[ ] Wrote browser tests using the validateAccessibility method
[ ] Manually tested at 200% size
[ ] Manually evaluated tab order

New Features

[ ] Add new FeatureFlag env vars to server/conf/application.conf
[ ] Conditioned new functionality on a FeatureFlag
[ ] Wrote browser tests with the feature flag off and on, etc.

Issue(s) this completes
Fixes #<issue_number>; Fixes #<issue_number>...

Codecov Report

Merging #3809 (cefcbe2) into main (7a01d95) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #3809   +/-   ##
=========================================
  Coverage     80.63%   80.63%
  Complexity     2441     2441
=========================================
  Files           296      296
  Lines          9642     9642
  Branches        659      659
=========================================
  Hits           7775     7775
  Misses         1613     1613
  Partials        254      254

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


hw-3 Going Live

I keep getting the Error 404 message after committing my changes successfully and adding hw-3 to the end of the url.   The index.html is in the right place and I can preview it and the color is right and the text is placed correctly.  My directories seem to be organized correctly

More information:  I tried copying index.html directly into hw-3 directory instead of having it live in the subdirectory 01-hello-world which is inside of hw-3 and contains the project, including index.html.  Now when I click on index.html inside of hw-3 it opens to a blank page, but when I open the original still inside of the subdirectory 01_hello-world it still opens up to the proper image.  I tried adding /hw-3/01_hello-world to the end of the url and it comes up with a blank page.  Adding just /hw-3 to the url still returns the Error 404

If you throw another /at the end of /hw-3, in your address bar  does it work?

Nah that was the first thing I tried :(

When I originally was supposed to create an index.html file in the very beginning, how is that process supposed to go?  I added the file and gave it that name, but I just realized the icon beside it is not an html icon, and I'm wondering if it won't convert and is still a .txt?  Would that matter or is the source just the html code I wrote in Atom?  That part looks fine and shows up fine

And now adding hw-3 to the end of the url yields a blank page, no more Error 404

Might be making progress with that last bit.  Don't forget to put thep5_lib folder in the same folder as your index.html and sketch.js files.

Yep they are all three inside hw-3/01_hello-world   When I click on index.html in the 01_hello-world subdirectory inside of hw-3 it brings me to the correct image

I'm also getting the blank page when I add the hw-3.

Yeah that's what's happening to me anyway

Okay, so for others who may see this let's break down the GitHub live site a bit.
The formatting is:
Username
.
github
.
io
/
Repository name
/
Folder
/
Subfolders
/
Ad Nasuem.
So for example, my live site for hw-3 is dvsnell.github.io/120-work/hw-3/  but what the browser is actually seeing is https://dvsnell.github.io/120-work/hw-3/index.html  My username is DVSnell, my stuff is hosted on github.io and my repo is 120-work.  My homework for week 3 is in hw-3 and that folder contains my index.html.
After looking at your repository, yours would be jolenetiffanyg.github.io/course-work/hw-3/01_hello-world/.  Again, the index.html is implied.
A 404 means there is no index at all, but since yours is returning a blank page it means the browser can see it and display it.  Looking at your index.html, I can see everything is in order.  The sketch.js and readme.md is also good, but there is no copy of the p5_lib folder and subfiles in your repository.
So the index is there, the sketch has code, but the index can't display that code since the p5 files are missing.  Try putting the p5_lib folder with it's three child files from the original example or course-work inside your 01_hello-world folder.  And don't forget to commit and push the changes to GitHub!

So it looks like the p5_lib folder with its subfolders are showing in both Atom and GitHub Desktop, as well as when I look into the folder directory.  Am I missing something?

yeah, you're good, but it's not on Github (the site).  The green + means the file(s) have changed, but haven't been committed or published.  Go ahead and make a commit, publish and let's see if that works.
I realize the language is a little clunky.  GitHub, GitHub Desktop, GitHub the site... It's a bit annoying.

Like That.

Yeah, I successfully committed everything but can't get the "HW-3" addition to work when I add it to the url.

Right, that's because your repository flow is 120-Work -> HW-3 -> empty-example  so the last part of the URL should be /empty-example  Adding the hw-3 only works if the last step in your repository is hw-3'.  Basically, https://erinflint.github.io/120-work/HW-3/empty-example/hw-3/` points to a non-existent location.
The fault here is the instructions not taking into account the possibility of different repository structure and names.
All that being said, even with the index.html there, I can't access it outside the raw code on your repository.  Are you sure you've made the 120-work repository settings a Github Pages site?

https://erinflint.github.io/120-Work/HW-3/empty-example/ this is the link that ended up working for me, thank you so much for your help. I was really overthinking it.

@erinflint Yep!  That one works perfectly.  If you want it to match Prof. Musick's you can basically move everything in /empty-example/ back one step to HW-3 and drop the last part of that URL you linked above, but if not, your Hello World is looking good!

 Thank you so much. This issues forum is invaluable to someone such as myself who has never even looked at code before. Thanks!

I have committed and published a million or so times with the index.html in the right place and working, and the p5.lib has been there the whole time.  So I'm not sure where the problem could be...  Right now though when I try to commit, I'm getting that Commit failed. Exit code 1 message.  All I did is change something in my readme so I don't think it will affect the main problem I'm having, but do you remember what that error message is about?

Ok the error message stopped happening on its own without me changing anything.. So I guess I'll worry about that next time it happens.  But it's still just blank when I navigate to hw-3/01_hello-world which is where my index.html lives and it works perfectly fine when I open it from the folder.  I've committed and pushed multiple times to make sure it's all current..

Yeah, look inside the p5_lib folder in Atom and delete the .git and .gittattributes items from there.  Exit code 1 tends to happen because there is a git inside another git.  After deleting them, commit and publish and it should work.
If not, you can manually add the files from the repository webpage.

Ok deleting the extra git. files again.  What do you mean by adding the files manually?

So, when you're looking at your repository on the github website, you can click through to the folder that has your working index.html and sketch.js and click the Add Files button on the right hand side.  Drag and drop the p5_lib folder from Windows Explorer into the designated space and put a commit comment at the bottom of the page before clicking submit.
Make sure the .git and .gitattributes items are NOT in that folder.

In my experience, 7% of all negative computer interactions are resolved by magic, so that's what I'll chock this one up to.

Hahaha mine have been resolved by magic like 30% of the time so far


How to convert NBT stored in drawers

This is an item stored in a drawer before 24w09a :

This is the same item but in the new format :

It will be especially hard with others NBT components, like enchantments, custom data from others datapacks, copying them to the new format will be so hard to do.

This is very annoying. Because new server versions will no longer be able to handle the old format at some point and all plugins and data packs that work with the old format will have to be rewritten. I'm already having a problem with it, especially because I want to export the nbt data. Now I have to convert the new data into an old format. If I find a way to ensure that it is recognized automatically and can also be applied to datapacks, I will be happy to let you know here.

Yeah everything will be in the "minecraft:custom_data" component, but I have to mimic the way items are transform at runtime.
Maybe by making a lib to do so ? idk

Writin something for my plugin at this moment for it. And yes there will be a lib with some usual formats.

The drawers will not update from 1.20 to 1.21, this is too hard to do in datapacks


Fix issue #670 (Destructuring struct returned from func call bug)

Related issue(s)
Resolves #670
Description
Reference issue for details of the bug.
Summary of changes
In code_gen, when we are returning a struct from a function we have a flattened vector of fields. Struct fields are stored as an ordered list, thus when we map the NodeIds returned from a func call in the IRGenerator we are associating NodeIds with an ordered list of struct fields. When we construct an ExtractTupleMember expression in the monomorphisation, we unpack a struct pattern with several let statements for each struct field in the way they were declared by the user. Thus, if the user does not deconstruct their struct in alphanumerical order when the struct is returned from a func call this could lead to an incorrect object being accessed.
I altered the patterns iterator used in the struct case of unpack_pattern to use a BTreeMap. This will match the order of fields in the StructType which is being unpacked. When calling unpack_tuple_pattern we enumerate over the patterns iterator and use the current index at each iteration to construct an ExtractTupleField expression. This has not changed, except now that the struct pattern iterator uses a BTreeMap we are accessing the correct index.
Dependency additions / changes

Test additions / changes
I added a regression test to our existing struct test that uses the snippet from the linked issue this PR is solving.
Checklist

[X] I have tested the changes locally.
[X] I have formatted the changes with Prettier and/or cargo fmt with default settings.
[X] I have linked this PR to the issue(s) that it resolves.
[X] I have reviewed the changes on GitHub, line by line.
[X] I have ensured all changes are covered in the description.

Additional context

👍

Will merge when CI finishes


Adding Using() to select bmc-lib drivers for client

Description
bmc-lib tries to establish a connection to check compatible protocols (redfish, ipmi etc) for all supported implementations. These changes modify Rufio to only use Redfish and IPMI protocols for all hardware.
Why is this needed
Rufio opens a new connection via bmc-lib for every reconcile loop. In the latest version of bmc-lib support for IntelAMT protocol was added. It was noticed that on older hardware when users try to run Rufio, IntelAMT takes too long to establish a connection and hence blocks the reconcile thread. This was causing timeouts of Rufio Tasks when it exceeded 3 mins to hit the connection timeout.
Fixes:
Timeout issues on older hardware due to Rufio reconcile threads getting blocked when bmc-lib provider open connection calls hang and timeout.
How Has This Been Tested?
make docker-build and tested the controller.
How are existing users impacted? What migration steps/scripts do we need?
Unblocks existing users who are running Rufio for older hardware.

I don't think we want to disable IntelAMT. An alternate to creating a better UX could be to set the per provider timeout. There's probably some improvements we could make in bmclib also.
@joelrebel

I noticed we simply don't specify any timeout in Rufio for establishing connections. I raised #87 to illustrate how we can implement a configurable timeout spread equally across providers, however it doesn't scale with more bmclib providers particularly well. The overview has some details on how further improvements could be made.

Hey @pokearu , I agree with @chrisdoherty4. We should not disable IntelAMT. We should have a context with a timeout for the open call. We can discuss longer term solutions like concurrent provider calls for open, and/or exposing provider filtering, and/or sharing a registry, and others.

Lets close this in favor of more discussion on some of the other options mentioned here. We could raise an issue for anything async as needed.


PersonAvatar Contact card hidden when used in AgGrid cell

Describe the bug
Contact Card is being rendered bellow AgGrid / hidden behind AgGrid
To Reproduce
Create a CellRenderer with a PersonAvatar within and enable trigger on hover. Hover over with mouse and see that the contact card is not shown properly
Expected behavior
That the contact card is shown above aggrid
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Browser Chrome
Version Chrome Version 121.0.6167.85 (Official Build) (64-bit)

Additional context
Add any other context about the problem here.

We are planning of creating separate component for AGGrid that will allow user to use person components and it will also include sort and filter.
In the meantime, there is a workaround with using <PersonCard /> as tooltipComponent in AGgrid.
You can find working example in Person cell definition - Contract Personnel

That's good to know, I'll take a look at the workaround in the meantime  👍
Thanks for the response

Hello, @Jonathanio123
good news :sparkles:  now we have working component that solves this issue :tada:
@equinor/fusion-react-ag-grid-person-cell
You can see working example in Contract Personnel - Delegate table


Portainer hasn't made any 3rd Party Auth-Methods "Business Only"

It appears that Portainer already got in touch with you, I'm mainly opening this issue for others to read about this too and be informed about what's going on :)

Problem/Motivation

In the latest Changelog it was said that Portainer now moved the previously used Auth method to their "Business" model. This seems to have been a bit of a miscommunication.

Expected behavior

X

Actual behavior

Portainer only deprecated the -no-auth option due to misuse by their users.
More infos can be read here: https://twitter.com/portainerio/status/1433914478534430720

Steps to reproduce

X

Proposed changes

X

I've read through a bunch of other issues now to see what's been going on and saw that apparently "Portainer lies there"? Is this for real?

I would love to know what features we moved from CE to Business, we have not moved ANY features across at all. Sure, we deprecate features from CE (--no-auth), but we didnt do that to move it to Business (Business doesnt have that functionality). If someone can point to a feature we moved, i would happily investigate, but we do not lie, never have, never will.


CSS bundling / refactoring (SASS plugin?)

Type of Change

[ ] New Feature Request
[ ] Improvement / Suggestion
[ ] Bug
[x] Other (please clarify below)

Summary
Currently just to get things through in #17 , compiling SASS was bypassed (see #15 ) inn favor of getting things through.  As a result, full bundles of bootstrap.css and font-awesome.css are bundled into every WC, which is definitely not ideal.
Details
As part of https://github.com/ProjectEvergreen/greenwood/discussions/763 would like to evaluate ways to improve this

[ ] Creating a SASS plugin - #15
[ ] something else more idiomatic to WCs?  (constructible or adopted stylesheets or using Lit's static styles?)
[ ] CSS Modules

Don't need font awesome for everything?


Symbol links with custom authored titles are rendered in code voice

Description
Symbol links with custom authored titles are rendered in code voice
If I write a doc link with a title like
Check out [this great function](doc:Sloth/eat(_:quantity:)).

the rendered output is
Check out `this great function`.

With this great function in code voice. The custom title should respect the formatting that was provided in the link title.
Checklist

[X] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue.

Swift-DocC Version Information
88d0fd6676f2a2ee2d169d813420645cf6bfd52d

CC: @Kyle-Ye maybe you're interested in taking a look at this? It looks related to the work for https://github.com/apple/swift-docc/issues/271.

rdar://108515663

CC: @Kyle-Ye maybe you're interested in taking a look at this? It looks related to the work for #271.

Sorry @ethan-kusters, I'm not sure what the specific question you raised is. Does it mean that we need to respect the format in the original title, but it is not?
Could you give me an expected behavior and actual behavior (Adding some code snippet or screenshot would be appreciated.)

CC: @Kyle-Ye maybe you're interested in taking a look at this? It looks related to the work for #271.

Looking at the screenshot of #376, I can't see what the problem is for
Check out [this great function](doc:Sloth/eat(_:quantity:)).

Looking at the screenshot of #376, I can't see what the problem is for
Check out [this great function](doc:Sloth/eat(_:quantity:)).

That screenshot demonstrates the issue I believe:

The "Hi" link text in "Check out Hi" is rendered with the default font while the "this great function" link text is rendered in monospace.
Monospace should only be used if the user actually specifies it, for example:
Check out [`this great function`](doc:Sloth/eat(_:quantity:)).

So expected behavior for:
Check out [this great function](doc:Sloth/eat(_:quantity:)).

would be seeing that "this great function" rendered with the default styling. Instead, I'm seeing it rendered in monospace code font.

I'll check and try to fix it this weekend.

The generated data/documentation/slothcreator.json on swift-docc side is OK.

But the corresponding reference doc://SlothCreator/documentation/SlothCreator/Sloth/eat(_:quantity:) is determined as an internal symbol by swift-docc-render.
So currently swift-docc-render will add "CodeVoice" style to it. "CodeVoice" will use "code" tag defined in "src/styles/base/_typography.scss" which set font-family to mono family.
The same is true before #376.

https://github.com/apple/swift-docc-render/blob/adadcb3e5784f899720315915640a453da3f11f6/src/components/ContentNode/ReferenceInternalSymbol.vue#L13-L15
https://github.com/apple/swift-docc-render/blob/adadcb3e5784f899720315915640a453da3f11f6/src/components/ContentNode/CodeVoice.vue#L11-L13
https://github.com/apple/swift-docc-render/blob/adadcb3e5784f899720315915640a453da3f11f6/src/styles/base/_typography.scss#L200-L204

So what would be your expected behaviour? Should 4 and 5 both show non-code style text?
4 [doc:Sloth/eat(_:quantity:)](doc:Sloth/eat(_:quantity:))
5 [this great function](doc:Sloth/eat(_:quantity:))

cc @ethan-kusters

TL,DR:
The custom title did respect the formatting that was provided in the link title in swift-docc side(The json content).
But if it was internal symbol, swif-docc-render will add custom css effect to it. Maybe this should fall into a swift-docc-render issue. 🤔

I think you're right – DocC-Render should be respecting the formatting provided in the overridingTitleInlineContent value instead of forcing monospace. I'll transfer this there.


Implement np.tril_indices_from

Implement np.tril_indices_from in jax
Part of #70

#3346 implements the same thing; since that one was first we will probably go with that PR.


Decrease review interval

Every month is a bit too much, let's decrease it to every third month.

Also fixed the comment


Flash Player Doesn't Work For Me

Hello,
I have been trying to make a proxy site today and I was testing out a lot of stuff (videos, games, etc.) and I can not get flash player or other scripts to work. Can anyone offer any assistance?

What site was it? Also, if the site is "too complex", the script won't work writing a special plugin. Similar problem here:
https://github.com/Athlon1600/php-proxy/issues/80#issuecomment-333273213

Are you asking for my site or the site that im trying to access?
My site is http://superskull.us
The site i was trying to use was http://www.nintendoemulator.com/nes/3100 which displays a message to download flash player.
Thanks, ill try to make a plugin, but I'm not great at php so ill probably have a little trouble with it
P.S. Your YouTube plugin is good 👍


Get auth-secret from other namespace in kubernetes.

Do you want to request a feature or report a bug?
Feature
What did you expect to see?
nginx ingress controller seems to supports nginx.ingress.kubernetes.io/auth-type of the form "namespace/secret". Traefik gives me the following error:
failed to load auth credentials: secret \"ingress_namespace\"/\"namespace/secret\" not found"

It looks like it insists on using the ingress namespace. Should traefik mimic nginx's behavior?

Hello @jpambrun,
I don't think that Traefik should reference auth credentials from a different namespace for a number of reasons:

Namespacing is used explicitly for isolation purposes. If you don't want isolation, namespacing is not required.
You can have the same secret duplicated across namespaces. If your application cannot function without configuration from another namespace, I think the decision to split into multiple namespaces may have not been correct.
Pods may only mount secrets within their own namespace (https://kubernetes.io/docs/concepts/configuration/secret/#restrictions). Although you (as a sysadmin) can configure traefik to monitor for secrets outside of the namespace, allowing end users to submit ingresses that reference other namespaces seems like a security risk, when they can't access those secrets via their own pods.

I would be interested in hearing @dduportal and @timoreimann's opinions on this subject

I figured the current approaches made sense after submitting the issue and
went the duplicated the secret route. I left it open because nginx seems to
support that and figured you might want to look into it.
Thanks for the prompt and comprehensive response.
On Wed., Feb. 20, 2019, 5:58 p.m. Daniel Tomcej<EMAIL_ADDRESS>wrote:

Hello @jpambrun https://github.com/jpambrun,
I don't think that Traefik should reference auth credentials from a
different namespace for a number of reasons:

Namespacing is used explicitly for isolation purposes. If you don't
want isolation, namespacing is not required.
You can have the same secret duplicated across namespaces. If your
application cannot function without configuration from another namespace, I
think the decision to split into multiple namespaces may have not been
correct.
Pods may only mount secrets within their own namespace (
https://kubernetes.io/docs/concepts/configuration/secret/#restrictions).
Although you (as a sysadmin) can configure traefik to monitor for secrets
outside of the namespace, allowing end users to submit ingresses that
reference other namespaces seems like a security risk, when they can't
access those secrets via their own pods.

I would be interested in hearing @dduportal https://github.com/dduportal
and @timoreimann https://github.com/timoreimann's opinions on this
subject
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/containous/traefik/issues/4497#issuecomment-465789703,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAr2MAU9dUlITIF-rHIdXMUydr-qpN3rks5vPdMOgaJpZM4a-d1q
.


Is there a way to make width of chart responsive?

At the moment if we are on a mobile or on screen the width of the table is of set length.
Is there a way to make the width of the chart responsive? So if on mobile the view of the width gets shortened?
Does the above make sense?
Thanks

Yes, it makes absolutely sense! Even if it is not yet implemented. At least, it doesn't refresh width when resizing window. Yet, if you refresh your browser, it should work out of the box.

Thanks @jpetitcolas. Got it. Not an issue. Just a followup question. Is it possible to insert an image in the left side where the labels are instead of using text?
For example where the text says 'admin-on-rest' I'd like a svg/png icon? Is that possible?

Currently not. But that's a nice addition (too much to be done! (-:). Can you create a separate issue for a better follow up?


Breadcrumb category

https://github.com/OpenBazaar/OpenBazaar-Client/issues/408

This closes #408


Refactor Active Storage migration into a job

Our nice-and-simple approach didn't work, so this makes it a bit more
"enterprise".

Move migration of documents into a job, and change the original Rake
task to simply kick off one job per document
Add integrity checks in the job to make sure we've actually
successfully migrated the document
Add a new Rake task verify_migration to verify documents and
supporting documents match after migration
Add EventContext#suppress_events! to make sure we don't fire events
when all we're doing is programatically updating something

Review app deployed to https://teaching-vacancies-review-pr-3510.london.cloudapps.digital


Allow SVG images for product/service image

Currently we can only use JPG or PNG. It would however be nice to use vector-images (which are usually far superior to these pixel images) - in particular SVG.

Gratipay team, please keep in mind that SVG files can contain code and therefore we should be very careful if we decide to incorporate this feature.

Security is the reason we don't support SVG already. What's the state of the art on securely handling SVGs from users? Does Camo help?

From what I read Camo is a (TLS) proxy. Why should it help?

I would rather we do not allow SVG files. The amount of CVEs I have seen related to SVG files bothers me.
In answer to your question @whit537, I believe most websites blacklist possibly malicious code in SVGs. Unfortunately, this is a very complicated and error prone procedure.

I suggest running the SVG through an optimizer like svgcleaner. That process should remove any malicious code, although I can't guarantee that (someone would need to take a deeper look if it hasn't been done yet).

FYI here is a good presentation about security issues with SVG files.


enhancement(poc): custom controller inside k8s to tell when there is a need for scaleup or scaledown

Why do we need it?
we are planning for custom controller

#251

What it will bring?
this issue tries to bring a working controller which can tell when the scaleup or scaledown should be triggered!
Additional context
Check Contribution's guidelines

not sure if it helps
https://github.com/kubernetes/autoscaler/tree/master/cluster-autoscaler/core


Create a contact info block

Add links to social networks (LinkedIn, Facebook, Instagram), phone number, email, Telegram username, etc.

https://www.facebook.com/nei.li.566
<EMAIL_ADDRESS>


Add more information to errors and add an exponential retry

Signed-off-by: Tibor Vass<EMAIL_ADDRESS>

yays for exponential backoff!

LGTM


Enhancement: Added copilot alias to chatbot commands

Closes #6261
Enhancement:
Updated below commands,
[☑]  Rename pp chatbot get to pp copilot get
[☑]   Add alias pp chatbot get to renamed command
[☑]   Rename pp chatbot list to pp copilot list
[☑]   Add alias pp chatbot list to renamed command
[☑]   Rename pp chatbot remove to pp copilot remove
[☑]   Add alias pp chatbot remove to renamed command

Hi @ktskumar, could you fix the error in the docs? Seems like you'll have to update the navigation nodes to the pages.
   These sidebar document ids do not exist:
  - cmd/pp/chatbot/chatbot-get
  - cmd/pp/chatbot/chatbot-list
  - cmd/pp/chatbot/chatbot-remove

I have updated the navigation code. Pls review it

Hi @milanholemans, I have updated the navigation code. Pls review it

Seems like there are still broken links in the release notes, could you have a look at it?

Merged manually, thank you!


Backend user with "User" role cannot reset password

The update route is not accessible for "user" roles which means they end up in a loop when they need to (force) reset password:
https://github.com/nodes-vapor/admin-panel-provider/blob/82f16e0531cddcb77b52ef6a0d42426a7a6a0642/Sources/AdminPanelProvider/Controllers/AdminPanelUserController.swift#L143

Fixed in https://github.com/nodes-vapor/admin-panel-provider/pull/25


Delete CODE_OF_CONDUCT.md

deleting this file in favor of the org wide coc in the .github repo. having this file in place means that this repo opts out of those org wide benefits.

@swift-ci Please test

@swift-ci Please test


Foxy Implementation

Hello,
Just curious how to use this with ros2 foxy, I run into issues when cloning the repo and building from source.
best regards

@OsamaThakur This package was backported after Foxy release.
Therefore no guarantee that it will be compiled with Foxy branch.
@emersonknapp please correct me if I am wrong.

This package was backported after Foxy release.

This package was originally written after Foxy release.  There is no expectation that it would be able to compile on Foxy. If you would like to create a Foxy-compatible branch and submit that to this repo, there would be no problem with that. However, Foxy's end-of-life is in about 100 days from now, so it may not be worth the effort

@MichaelOrlov @emersonknapp Thank you guys for your feedback, I will take them into consideration


Language Grammar - generalized-identifier Syntax

Hello,
If I'm understanding the language specification correctly, a generalized-identifier that starts with a decimal digit must be followed with a letter or underscore. However, both Power BI and Excel do not enforce the "must be followed by" rule--they both allow a single decimal digit to be used by itself:
[9 = true][9]

To align the specification with this behavior, would it make sense to update the spec by adding (opt) as noted below:
generalized-identifier-part:
      generalized-identifier-segment
      decimal-digit-character generalized-identifier-segment (opt)

The definition of generalized-identifier is ... not great. This is largely because the implementation doesn't really lend itself to a usable grammar. For instance, [9=1] and [9.0=1] are legal, while [9.=1] is not.

@CurtHagenlocher, would there be any chance of the current implementation being tweaked so that it aligns with a usable grammar? If not, any chance you could share the logic currently used, even if it doesn't express well in formal grammar syntax.
I'd like to align the Data Mashup Cmdlets code validation to the current implementation, but of course that's about impossible without a grammar. :-) It also looks like Microsoft's Power Query Parser's behavior doesn't align with the current implementation--which could possibly (???) be due to the lack of a good rule here.
So some clarity here, or a tweak to the existing implementation, could potentially help a couple of projects.
Thanks!

Characters which are always valid in generalized-identifier are in the Unicode categories Uppercase Letter, Lowercase Letter, Titlecase Letter, Modifier Letter, Other Letter, Nonspacing Mark, Spacing Mark, Decimal Number, Connector Punctation and Format. The one character which is valid only situationally is the period (code point 46). Periods are not valid as the first or last character of the identifier, and it is not legal to have two consecutive periods. They are otherwise permitted.

Thank you, @CurtHagenlocher!
Does this look right?
space:
    literal space character (code point 20)
period:
    literal period character (code point 46)
generalized-identifier-always-valid-character:
    any character from Unicode categories Uppercase Letter, Lowercase Letter, Titlecase Letter, Modifier Letter, Other Letter, Nonspacing Mark, Spacing Mark, Decimal Number, Connector Punctation and Format
generalized-identifier-inner-valid-character:
    generalized-identifier-always-valid-character | space
generalized-identifier:
    generalized-identifier-always-valid-character ((period? generalized-identifier-inner-valid-character)* generalized-identifier-always-valid-character)?

Please ignore (I'm using bot automation to add labels). #please-open

This issue seems to be about this page:
URL: https://docs.microsoft.com/en-us/powerquery-m/m-spec-lexical-structure#generalized-identifiers
Source: https://github.com/MicrosoftDocs/query-docs/blob/main/query-languages/m/m-spec-lexical-structure.md
@bgribaudo if you have time and are willing, feel free to propose the change in the public repo. Thanks much, Jason
#please-open

@CurtHagenlocher, do you time to look at the proposed grammar definition in my last comment (above)?

Thanks, @egorelik93! How about the following?
space:
    literal space character (code point 20)
period:
    literal period character (code point 46)
generalized-identifier-always-valid-character:
    any character from the following Unicode categories: Uppercase Letter, Lowercase Letter, Titlecase Letter, Modifier Letter, Letter Number, Other Letter, Nonspacing Mark, Spacing Mark, Decimal Number, Connector Punctuation and Format
generalized-identifier:
    generalized-identifier-always-valid-character (space* period? space* generalized-identifier-always-valid-character)*

@bgribaudo No dice; this gross thing apparently is valid M:
[9 ab.  .   .   c= 0]
Yours I believe forces every intermediate 'space dot space' piece to end in an always-valid-character, so it doesn't cover the above.

@bgribaudo Possibly this? I'd need to think about it a bit longer to check for errors.
space:
    literal space character (code point 20)
period:
    literal period character (code point 46)
generalized-identifier-always-valid-character:
    any character from the following Unicode categories: Uppercase Letter, Lowercase Letter, Titlecase Letter, Modifier Letter, Letter Number, Other Letter, Nonspacing Mark, Spacing Mark, Decimal Number, Connector Punctuation and Format
generalized-identifier-inner-valid-character:
    generalized-identifier-always-valid-character | space
generalized-identifier:
    generalized-identifier-always-valid-character ((period? generalized-identifier-inner-valid-character)* period? generalized-identifier-always-valid-character)?

Oooh! Good catch on period-space-period.

@bgribaudo @egorelik93
What needs to be done to the documentation to resolve this issue?

Hi @DougKlopfenstein,
I think what should happen here/if anything should happen is dependent on the outcome of this PR: https://github.com/MicrosoftDocs/query-docs/pull/167. In essence, this PR is blocked by that one.

Hi @bgribaudo
This issue has been open for almost 4 years. Since the issue is being handled in the PR you mention above, I'm closing this issue so it doesn't continue to skew our internal reports.
#please-close

Thank you, @DougKlopfenstein.


[SPARK-348] get option bug

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

cc @susanxhuynh  for review

Looks good!


Fix the Netlify hosting

Description

Fix the Netlify hosting

Netlify installs twice cause the packages.json file in the root isn't the one used my the website itself,
Netlify automatically installs the root package.json, but thats isn't needed
Instead of running 'yarn build' I have to run cd /website && yarn && yarn install
That increases deploy times

Fixed, changed base dir to website/


Proposal for composite resource type

I would like to propose a new composite resource type to move HydroShare towards the model of file types separate from resource types.  Separate file and resource types have been discussed in #965 and #1034, but these are complex and intimidating (and in my opinion slowing development).  I think that a composite resource type can circumvent the difficulties and get us functionality that we need quickly.
The need for apps to act on individual files within a resource has come from both RHESSys and CyberGIS use cases and discussions related to the HydroShare GIS app.
I suggest we define a composite resource as a resource holding one or more files of a recognized type. For the first implementation this could be identical to a generic resource with the added metadata of an identifier giving the file type for each file.  There would need to be a table in HydroShare that lists acceptable file types.  For simplicity I suggest we start with just raster, shapefile and netcdf, then add others late.  The table would need to also list mime type and possibly a file type description.  It may be that mime type can be file type.  Then we can incrementally add functionality such as

Ability of HydroShare GIS app to look at the files in this resource type and load them based on type
Ability to have additional metadata such as is currently associated with a typed resource with the corresponding typed file.  This would reuse code already in place for raster, feature and netcdf resources (but without having to touch those resource types)
Ability to associate an app with a recognized file type just as we do now for a resource.  Extend the app linkage model for this.
Ability to move files from an existing resource into a composite resource.

While a composite resource has elements similar to collections, it is distinct from a collection.  A collection is comprised of fully formed resources, while a composite resource just holds files of a known type.  A collection could hold composite resources (composite resource just being a new resource type).  A composite resource could not hold a collection (or any other resource for that matter).  It holds files, not fully formed resources.
By building this functionality in the context of a new resource type I think we can defer refactoring existing resource functionality.  I do not think we need complex validation for the initial implementations, nor the celery tasks and registry for validators mentioned in #1034, at least initially.  I think that considerable functionality can be implemented just relying on mime type.  HydroShare would thus not protect users from misleadingly named files that have an extension (mime type) that does not match the content.
Files that are held in composite resources may need to be generalized to include "folders of content" to accommodate shapefiles and rasters, because for both of these there are multiple files involved in a single object.  The functionality for this should be identical to how multiple files are handled for geographic feature and raster resources.

Not sure how this is different than what we have already with generic resource? Each generic resource already has a set of files - some of which have recognized types. Mime types for files can already be recorded in the metadata for the generic resource. What's missing is the ability for an app to look inside a resource and pull out an individual file (or set of files).  Not sure how a new resource type would solve that problem.
I also think it would be somewhat confusing for users - e.g., you can view a file of this type in this app, but only if it is part of a "composite" resource and not any other kind of resource.  And - it seems like it needs to be consistent. If I can open a raster file from a generic resource in the GIS App, I should also be able to open a raster file from a raster resource or a model instance resource, or any other type of resource that contains a raster file of the right type.

This is proposed as an incremental solution to avoid what seems to be big refactoring of existing resources to achieve this.  If we can achieve this for generic/all resources without an incremental approach I would support that, but this is an attempt to unbog what seems to be bogged down.

I tend to agree with Jeff on this. We need to articulate the use case for a
composite resource a bit better. In fact, in our current system, we can
write apps to read files out of resources and act on them. Shawn is
currently working to extend HydroShare GIS to open any resource type,
inspect it's contents, and load any known layer types. This will all work
with our current resources - just requires a clever app. Other apps could
behave similarly. So let's think about what use case would not be supported
by our current model.  I'm sure there is one, but I'm not quite seeing it
yet. - Dan
On Thu, Apr 21, 2016 at 12:57 PM David Tarboton<EMAIL_ADDRESS>wrote:

This is proposed as an incremental solution to avoid what seems to be big
refactoring of existing resources to achieve this. If we can achieve this
for generic/all resources without an incremental approach I would support
that, but this is an attempt to unbog what seems to be bogged down.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/hydroshare/hydroshare/issues/1126#issuecomment-213067049

If we can get this done with existing resources I am all for it.  The immediate use case I want to support is that CyberGIS returns multiple TauDEM results in a single generic resource.  I want to be able to visualize these.  We could have CyberGIS return individual resources, but that requires an abstract and keyword for each - which seems overkill and loses the grouping of them together.
Also we have designed resource specific metadata for things like rasters and shapefiles.  This adds value to the files.  I'd like this metadata to be able to be associated with individual files in a resource, rather than held at the resource level.  This would support the CyberGIS TauDEM use case allowing metadata to say what the rasters are (elevation, contributing area etc, and give the units).
An App should be able to get a list of files in a resource and could infer type from its extension.  This is not robust, but probably OK.  An App resource should be able to declare the file extensions it can work with (small extension to the app resource model), and then any resource landing page could enable an open with that app button next to that file in the resource (an extension to the resource landing page model).  Complications arise for logical files that are comprised of multiple physical files (such as shapefiles and .vrt raster files that we are already using).  We could accommodate this perhaps by requiring these all be encapsulated in directories so they are logically together.  The resource data model already lists file and mime types for the files in a resource, but as currently implemented this listing does not map to specific files.  I do not know how complicated changing this would be, or if it is necessary if we just use file extensions.

Perhaps CyberGIS could output a single generic resource with all the
generated rasters AND a HydroShare GIS project file with the symbology,
ready for viewing in HydroShare GIS. - Dan
On Fri, Apr 22, 2016 at 8:18 AM David Tarboton<EMAIL_ADDRESS>wrote:

If we can get this done with existing resources I am all for it. The
immediate use case I want to support is that CyberGIS returns multiple
TauDEM results in a single generic resource. I want to be able to visualize
these. We could have CyberGIS return individual resources, but that
requires an abstract and keyword for each - which seems overkill and loses
the grouping of them together.
Also we have designed resource specific metadata for things like rasters
and shapefiles. This adds value to the files. I'd like this metadata to be
able to be associated with individual files in a resource, rather than held
at the resource level. This would support the CyberGIS TauDEM use case
allowing metadata to say what the rasters are (elevation, contributing area
etc, and give the units).
An App should be able to get a list of files in a resource and could infer
type from its extension. This is not robust, but probably OK. An App
resource should be able to declare the file extensions it can work with
(small extension to the app resource model), and then any resource landing
page could enable an open with that app button next to that file in the
resource (an extension to the resource landing page model). Complications
arise for logical files that are comprised of multiple physical files (such
as shapefiles and .vrt raster files that we are already using). We could
accommodate this perhaps by requiring these all be encapsulated in
directories so they are logically together. The resource data model already
lists file and mime types for the files in a resource, but as currently
implemented this listing does not map to specific files. I do not know how
complicated changing this would be, or if it is necessary if we just use
file extensions.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/hydroshare/hydroshare/issues/1126#issuecomment-213445651

That would be cool.  It is certainly well within the realm of possibility once you get the GIS project figures out.

@pkdash I am putting this on your plate initially with a lower priority milestone. This needs more discussion before implementation.

@dtarb Does this issue need to stay open until we implement all file types?

I am happy to close and handle additional types, or additional issues associated with this separately.


mentioning Personal Access Tokens for Azure setup

Proposed mention of Personal Access Tokens in the extra section at the bottom of this page: https://docs.getdbt.com/docs/collaborate/git/setup-azure
What are you changing in this pull request and why?

Recently wrote a writeup for internal usage, based on a number of customers writing in about this.
https://www.notion.so/dbtlabs/382bbd560dfc4065950a1c153c58bd72?v=24968eb725ef4ec0bdc76c9c90bca62f&p=07df647a0c774d8fb9e796d7e3d00ed4&pm=s
Maybe some proactive info on the website will prevent them from reaching out!

thanks PAT - i'll take a look at this and revert back 😄

Slight change requested by Sharif. Making some updates!


list mode selection should be keep through navigation

Issue

close #311

Overview (Required)

Provide mode selection with ViewModel.
Save mode selection on DataStore.

Screenshot
https://user-images.githubusercontent.com/26338889/189364426-26569af2-c782-4c49-a978-f556e84e3276.mp4

I didn't mean to hold the state in datastore. 👀 I wanted you just to hold it in ViewModel. 🙇


MUMUP-2337 : Change colors to match uw-system theme

Using https://www.wisconsin.edu/digital-communication/download/styleguide-5.27.15.pdf as a reference, changed the theme to look like this:

Note the colors that are weird in the body will be fixed in another issue

:+1:

:+1:

late to the game but :+1: may tweak that system logo at some point but not now

Sounds good @keirserrie , I snagged there logo from https://www.wisconsin.edu/digital-communication/download/styleguide-5.27.15.pdf


use fetch-everywhere to get horizon working in more places

Right now node has an issue with fetch-isomorphic

This was fixed


[Query] Java createAndInitializeObjectMapper can I override the default ObjectMapper?

Question
I am using kotlin with the CosmosAsyncClient and as such many of the data classes I'm using will not deserialize. If I was to use the https://github.com/FasterXML/jackson-module-kotlin module then this wouldn't be an issue. So I'm wondering if there is a way to add additional configuration to the ObjectMapper used by the CosmosAsyncClient? So that in my case I can add the lines
objectMapper.registerModule(new KotlinModule());
Why is this not a Bug or a feature Request?
For all I know it might be possible to do this already, so I'll ask a question first then if the answer is NO it's not possible. then I'll submit a feature request with a PR.
Setup

OS: MacOs
IDE: IntelliJ
Library/Libraries:  com.azure:azure-cosmos:4.44.0

Information Checklist

[x] Query Added
[x] Setup information Added

@CraftyFella - this is not possible at the moment, as we don't expose custom serialization to Cosmos DB Java SDK. This is in our backlog, but we don't have any timelines for it yet.


[fix postcommit] Rename tests to remove them from dataflow test suite

These tests use a fake bigquery service, so they don't belong as integration tests.

R: @Abacn


Coverage with custom jacocorunner jar file - what am I doing wrong?

Trying to get coverage working for a repository after upgrading rules_scala to a40063ef97688f056824b22b9e49fae6efd1df0f and bazel 5.1.1.

Checked out rules_scala at 73f5d1a7da081c9f5160b9ed7ac745388af28e23 and built the custom deploy jar as follows:
I then edited the test.sh file to use the bazel 5 version:
 build_local_jacocorunner() {
-    $root_dir/scripts/build_jacocorunner/build_jacocorunner.sh
+    $root_dir/scripts/build_jacocorunner/build_jacocorunner_bazel_5.0+.sh
     cp /tmp/bazel_jacocorunner_build/JacocoCoverage_jarjar_deploy.jar $root_dir/manual_test/coverage_local_jacocorunner
 }

$ cd manual_test/coverage_local_jacocorunner
$ ./test.sh

With the custom jar built, packed it into a zip file:
$ jar -cfM jacoco-coverage-73f5d1.zip ./JacocoCoverage_jarjar_deploy.jar

And uploaded that zip file here:
jacoco-coverage-73f5d1.zip

The zip file can be consumed as an http_archive as follows:
    http_archive(
        name = "rules_scala_jacocorunner",
        sha256 = "1e21d640ac5e39bff30c6255b2de389b0a53adf2338aeeeb5a656b13de11ba2b",
        urls = ["https://github.com/bazelbuild/rules_scala/files/10109719/jacoco-coverage-73f5d1.zip"],
        build_file_content = """
filegroup(
    name = "jar",
    srcs = ["JacocoCoverage_jarjar_deploy.jar"],
    visibility = ["//visibility:public"],
)
        """,
    )

A custom scala_toolchain was setup:
load("@io_bazel_rules_scala//scala:scala_toolchain.bzl", "scala_toolchain")

scala_toolchain(
    name = "jacocorunner_toolchain_impl",
    jacocorunner = "@rules_scala_jacocorunner//:jar",
    unused_dependency_checker_mode = "error",
    visibility = ["//visibility:public"],
)

toolchain(
    name = "jacocorunner_scala_toolchain",
    toolchain = "jacocorunner_toolchain_impl",
    toolchain_type = "@io_bazel_rules_scala//scala:toolchain_type",
    visibility = ["//visibility:public"],
)

The toolchain was registered in the WORKSPACE:
register_toolchains(
    "//bazel_tools/scala:jacocorunner_scala_toolchain",
)

Salient .bazelrc flags:
build --java_language_version=17
build --tool_java_language_version=17
build --java_runtime_version=remotejdk_17
build --tool_java_runtime_version=remotejdk_17
coverage --combined_report=lcov
coverage --coverage_report_generator="@bazel_tools//tools/test/CoverageOutputGenerator/java/com/google/devtools/coverageoutputgenerator:Main"

Despite all this, coverage.dat files are always empty:
bazel coverage //common/auth:tests
INFO: Invocation ID: 9a653f7d-2864-4208-80cd-aa4bdb558f10
INFO: Using default value for --instrumentation_filter: "^//trumid/ats/common/auth[/:]".
INFO: Override the above default with --instrumentation_filter
INFO: Analyzed target //common/auth:tests (0 packages loaded, 0 targets configured).
INFO: Found 1 test target...
INFO: LCOV coverage report is located at /private/var/tmp/_bazel_i868039/f9bbdd40f3eca77219d10972abb94836/execroot/foo/bazel-out/_coverage/_coverage_report.dat
 and execpath is bazel-out/_coverage/_coverage_report.dat
INFO: From Coverage report generation:
Nov 28, 2022 8:58:32 PM com.google.devtools.coverageoutputgenerator.Main getTracefiles
INFO: Found 1 tracefiles.
Nov 28, 2022 8:58:32 PM com.google.devtools.coverageoutputgenerator.Main parseFilesSequentially
INFO: Parsing file bazel-out/darwin-fastbuild/testlogs/trumid/ats/common/auth/tests/coverage.dat
Nov 28, 2022 8:58:32 PM com.google.devtools.coverageoutputgenerator.Main getGcovInfoFiles
INFO: No gcov info file found.
Nov 28, 2022 8:58:32 PM com.google.devtools.coverageoutputgenerator.Main getGcovJsonInfoFiles
INFO: No gcov json file found.
Nov 28, 2022 8:58:32 PM com.google.devtools.coverageoutputgenerator.Main getProfdataFileOrNull
INFO: No .profdata file found.
Nov 28, 2022 8:58:32 PM com.google.devtools.coverageoutputgenerator.Main runWithArgs
WARNING: There was no coverage found.
Target //common/auth:tests up-to-date:
  bazel-bin/common/auth/tests.jar
  bazel-bin/common/auth/tests.sh
INFO: Elapsed time: 0.528s, Critical Path: 0.14s
INFO: 2 processes: 1 disk cache hit, 1 internal.
INFO: Build completed successfully, 2 total actions

Executed 0 out of 1 test: 1 test passes.
INFO: Build completed successfully, 2 total actions

$ du -sh /private/var/tmp/_bazel_i868039/f9bbdd40f3eca77219d10972abb94836/execroot/foo/bazel-out/_coverage/_coverage_report.dat
0	/private/var/tmp/_bazel_i868039/f9bbdd40f3eca77219d10972abb94836/execroot/foo/bazel-out/_coverage/_coverage_report.dat

bazel version
Build label: 5.1.1- (@non-git)
Build target: bazel-out/darwin-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Sat May 21 09:38:24 2022<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

cc @gergelyfabian any suggestions here?

Did you try playing with the --instrumentation_filter parameter?

rules_scala is on version 73f5d1a7da081c9f5160b9ed7ac745388af28e23 in my example.

Indeed things are working in that example, with the custom JacocoCoverage_jarjar_deploy.jar uploaded to this issue.
bazel-scala-example % git status
On branch jacoco_coverage_fix_new
Your branch is up to date with 'origin/jacoco_coverage_fix_new'.

Untracked files:
  (use "git add <file>..." to include in what will be committed)
	tools/JacocoCoverage_jarjar_deploy.jar

lcov --list bazel-out/_coverage/_coverage_report.dat
Reading tracefile bazel-out/_coverage/_coverage_report.dat
                                   |Lines       |Functions  |Branches
Filename                           |Rate     Num|Rate    Num|Rate     Num
=========================================================================
[example-lib/src/main/]
java/mypackage/Greeter.java        | 0.0%     19| 0.0%     6|    -      0
scala/mypackage/Foo.scala          | 100%     20|52.6%    38|    -      0

[example-maven/src/main/scala/mypackage/]
Maven.scala                        | 100%      2|66.7%     3|    -      0
=========================================================================
                             Total:|53.7%     41|46.8%    47|    -      0

Clearly something different about the repository I'm using it within (not sure yet what it is) but at least this is a good sanity check so thank you @gergelyfabian for setting up this example.

Just leaving a trail here for my future self, or someone else...  Seeing an exception from the jacocorunner, not sure what it means yet.
In the repo I'm working with, copying the example-lib/ into it and running coverage works for those simple targets.  But running it for "real" targets always yields empty coverage.dat files.
Here's what I see with --test_env=VERBOSE_COVERAGE=1 flag:
bazel coverage \
  //common/utils/caseclass/... \
  --experimental_fetch_all_coverage_outputs \
  --instrumentation_filter=//common/utils/caseclass \
  --test_output=all \
  --experimental_split_coverage_postprocessing \
  --test_env=VERBOSE_COVERAGE=1
  --spawn_strategy=standalone

INFO: Invocation ID: 612902eb-19b6-48b3-99c8-541a00a179dd
Loading:
Loading: 0 packages loaded
Analyzing: 2 targets (0 packages loaded, 0 targets configured)
DEBUG: /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/external/io_bazel_rules_scala/scala/private/phases/phase_coverage.bzl:57:14: running jacoco instrumenter! //trumid/common/utils/caseclass:scala
DEBUG: /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/external/io_bazel_rules_scala/scala/private/phases/phase_coverage_runfiles.bzl:27:18: coverage runfile: tools/JacocoCoverage_jarjar_deploy.jar
DEBUG: /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/external/io_bazel_rules_scala/scala/private/phases/phase_coverage_runfiles.bzl:27:18: coverage runfile: ../remote_coverage_tools/Main.jar
DEBUG: /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/external/io_bazel_rules_scala/scala/private/phases/phase_coverage_runfiles.bzl:27:18: coverage runfile: ../remote_coverage_tools/Main
DEBUG: /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/external/io_bazel_rules_scala/scala/private/phases/phase_coverage_runfiles.bzl:27:18: coverage runfile: trumid/common/utils/caseclass/scala-offline.jar
INFO: Analyzed 2 targets (0 packages loaded, 0 targets configured).
INFO: Found 1 target and 1 test target...
[3 / 6] [Prepa] BazelWorkspaceStatusAction stable-status.txt ... (2 actions, 0 running)
FAIL: //trumid/common/utils/caseclass:tests (see /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/test.log)
INFO: From Testing //trumid/common/utils/caseclass:tests:
==================== Test output for //trumid/common/utils/caseclass:tests:
+ [[ -z bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files ]]
+ export ROOT=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity
+ ROOT=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity
+ [[ bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files != /* ]]
+ export COVERAGE_MANIFEST=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files
+ COVERAGE_MANIFEST=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files
+ export COVERAGE_DIR=bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
+ COVERAGE_DIR=bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
+ [[ bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage == /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity* ]]
+ COVERAGE_DIR=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
+ mkdir -p /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
+ COVERAGE_OUTPUT_FILE=bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat
+ [[ bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat == /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity* ]]
+ COVERAGE_OUTPUT_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat
+ export JAVA_COVERAGE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/jvcov.dat
+ JAVA_COVERAGE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/jvcov.dat
+ export COVERAGE=1
+ COVERAGE=1
+ export BULK_COVERAGE_RUN=1
+ BULK_COVERAGE_RUN=1
+ [[ -z '' ]]
+ export GCOV_PREFIX_STRIP=3
+ GCOV_PREFIX_STRIP=3
+ export GCOV_PREFIX=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
+ GCOV_PREFIX=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
+ export LLVM_PROFILE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/%h-%p-%m.profraw
+ LLVM_PROFILE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/%h-%p-%m.profraw
+ [[ ! -z '' ]]
+ [[ 0 == \0 ]]
+ cd /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.sh.runfiles/unity
+ [[ 1 == \0 ]]
+ /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.sh.runfiles/unity/trumid/common/utils/caseclass/tests.sh
Discovery starting.
Discovery completed in 58 milliseconds.
Run starting. Expected test count is: 1
CaseClassFactoryTest:
- should instantiate a case class from it's corresponding tuple (7 milliseconds)
Run completed in 100 milliseconds.
Total number of tests run: 1
Suites: completed 2, aborted 0
Tests: succeeded 1, failed 0, canceled 0, ignored 0, pending 0
All tests passed.
+ TEST_STATUS=0
+ [[ 0 -ne 0 ]]
+ [[ 1 == \1 ]]
+ [[ 0 == \0 ]]
+ exit 0
+ [[ -z bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files ]]
+ export ROOT=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity
+ ROOT=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity
+ [[ bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files != /* ]]
+ export COVERAGE_MANIFEST=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files
+ COVERAGE_MANIFEST=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files
+ export COVERAGE_DIR=bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
+ COVERAGE_DIR=bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
+ [[ bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage == /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity* ]]
+ COVERAGE_DIR=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
+ mkdir -p /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
+ COVERAGE_OUTPUT_FILE=bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat
+ [[ bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat == /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity* ]]
+ COVERAGE_OUTPUT_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat
+ export JAVA_COVERAGE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/jvcov.dat
+ JAVA_COVERAGE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/jvcov.dat
+ export COVERAGE=1
+ COVERAGE=1
+ export BULK_COVERAGE_RUN=1
+ BULK_COVERAGE_RUN=1
+ [[ -z '' ]]
+ export GCOV_PREFIX_STRIP=3
+ GCOV_PREFIX_STRIP=3
+ export GCOV_PREFIX=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
+ GCOV_PREFIX=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
+ export LLVM_PROFILE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/%h-%p-%m.profraw
+ LLVM_PROFILE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/%h-%p-%m.profraw
+ [[ ! -z '' ]]
+ [[ 1 == \0 ]]
+ [[ 1 == \1 ]]
+ [[ 1 == \0 ]]
+ [[ 1 == \1 ]]
+ [[ 1 == \1 ]]
+ touch /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat
+ cd /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity
+ [[ -n '' ]]
+ [[ -z bazel-out/darwin_arm64-fastbuild/bin/external/remote_coverage_tools/Main ]]
+ for name in "$LCOV_MERGER"
+ [[ ! -e bazel-out/darwin_arm64-fastbuild/bin/external/remote_coverage_tools/Main ]]
+ [[ 1 == \1 ]]
++ resolve_links /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
++ local name=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
++ '[' -e /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage ']'
++ '[' -L /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/_coverage ']'
++ echo /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/_coverage
+ COVERAGE_DIR=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/_coverage
++ resolve_links /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files
++ local name=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files
++ '[' -e /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files ']'
++ '[' -L /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files ']'
++ echo /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files
+ COVERAGE_MANIFEST=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files
+ LCOV_MERGER_CMD='bazel-out/darwin_arm64-fastbuild/bin/external/remote_coverage_tools/Main --coverage_dir=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/_coverage   --output_file=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/coverage.dat   --filter_sources=/usr/bin/.+   --filter_sources=/usr/lib/.+   --filter_sources=/usr/include.+   --filter_sources=/Applications/.+   --filter_sources=.*external/.+   --source_file_manifest=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files'
+ [[ -n '' ]]
+ [[ -n '' ]]
+ JAVA_RUNFILES=
+ exec bazel-out/darwin_arm64-fastbuild/bin/external/remote_coverage_tools/Main --coverage_dir=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/_coverage --output_file=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/coverage.dat --filter_sources=/usr/bin/.+ --filter_sources=/usr/lib/.+ --filter_sources=/usr/include.+ --filter_sources=/Applications/.+ '--filter_sources=.*external/.+' --source_file_manifest=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files
Dec 17, 2022 6:12:20 PM com.google.devtools.coverageoutputgenerator.Main getTracefiles
INFO: Found 1 tracefiles.
Dec 17, 2022 6:12:20 PM com.google.devtools.coverageoutputgenerator.Main parseFilesSequentially
INFO: Parsing file /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/_coverage/jvcov11223020425242003820.dat
Dec 17, 2022 6:12:20 PM com.google.devtools.coverageoutputgenerator.Main getGcovInfoFiles
INFO: No gcov info file found.
Dec 17, 2022 6:12:20 PM com.google.devtools.coverageoutputgenerator.Main getGcovJsonInfoFiles
INFO: No gcov json file found.
Dec 17, 2022 6:12:20 PM com.google.devtools.coverageoutputgenerator.Main getProfdataFileOrNull
INFO: No .profdata file found.
Dec 17, 2022 6:12:20 PM com.google.devtools.coverageoutputgenerator.Main runWithArgs
WARNING: There was no coverage found.
java.io.IOException: Permission denied
	at java.base/java.io.UnixFileSystem.createFileExclusively(Native Method)
	at java.base/java.io.File.createTempFile(File.java:2170)
	at com.google.testing.coverage.JacocoCoverageRunner.getUniquePath(JacocoCoverageRunner.java:345)
	at com.google.testing.coverage.JacocoCoverageRunner$2.run(JacocoCoverageRunner.java:519)
================================================================================
INFO: LCOV coverage report is located at /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/_coverage/_coverage_report.dat
 and execpath is bazel-out/_coverage/_coverage_report.dat
INFO: Elapsed time: 0.933s, Critical Path: 0.67s
INFO: 2 processes: 1 internal, 1 local.
INFO: Build completed, 1 test FAILED, 2 total actions
//common/utils/caseclass:tests                                    FAILED in 0.5s
  /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/test.log

Executed 1 out of 1 test: 1 fails locally.
INFO: Build completed, 1 test FAILED, 2 total actions

It seems your test runner reported a success, but then Bazel reports a
failure. It seems there is a permission error, right at the end of the log.
Check maybe whether Bazel can create coverage report files.
In general coverage support works with rules_scala. The last case I saw and
fixed in my project was some Scala files did not have a matching location
(in the file system) and package name (as in the Java convention). In that
case no coverage was reported, but without a failure. It was fixed by
adjusting the folder structure to the package names.
Paul Cody Johnston @.***> ezt írta (időpont: 2022. dec.
17., Szo 19:13):

Just leaving a trail here for my future self, or someone else... Seeing an
exception from the jacocorunner, not sure what it means yet.
In the repo I'm working with, copying the example-lib/ into it and
running coverage works for those simple targets. But running it for "real"
targets always yields empty coverage.dat files.
Here's what I see with --test_env=VERBOSE_COVERAGE=1 flag:
bazel coverage
//common/utils/caseclass/...
--experimental_fetch_all_coverage_outputs
--instrumentation_filter=//common/utils/caseclass
--test_output=all
--experimental_split_coverage_postprocessing
--test_env=VERBOSE_COVERAGE=1
--spawn_strategy=standalone
INFO: Invocation ID: 612902eb-19b6-48b3-99c8-541a00a179dd
Loading:
Loading: 0 packages loaded
Analyzing: 2 targets (0 packages loaded, 0 targets configured)
DEBUG: /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/external/io_bazel_rules_scala/scala/private/phases/phase_coverage.bzl:57:14: running jacoco instrumenter! //trumid/common/utils/caseclass:scala
DEBUG: /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/external/io_bazel_rules_scala/scala/private/phases/phase_coverage_runfiles.bzl:27:18: coverage runfile: tools/JacocoCoverage_jarjar_deploy.jar
DEBUG: /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/external/io_bazel_rules_scala/scala/private/phases/phase_coverage_runfiles.bzl:27:18: coverage runfile: ../remote_coverage_tools/Main.jar
DEBUG: /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/external/io_bazel_rules_scala/scala/private/phases/phase_coverage_runfiles.bzl:27:18: coverage runfile: ../remote_coverage_tools/Main
DEBUG: /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/external/io_bazel_rules_scala/scala/private/phases/phase_coverage_runfiles.bzl:27:18: coverage runfile: trumid/common/utils/caseclass/scala-offline.jar
INFO: Analyzed 2 targets (0 packages loaded, 0 targets configured).
INFO: Found 1 target and 1 test target...
[3 / 6] [Prepa] BazelWorkspaceStatusAction stable-status.txt ... (2 actions, 0 running)
FAIL: //trumid/common/utils/caseclass:tests (see /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/test.log)
INFO: From Testing //trumid/common/utils/caseclass:tests:
==================== Test output for //trumid/common/utils/caseclass:tests:

[[ -z bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files ]]
export ROOT=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity
ROOT=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity
[[ bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files != /* ]]
export COVERAGE_MANIFEST=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files
COVERAGE_MANIFEST=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files
export COVERAGE_DIR=bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
COVERAGE_DIR=bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
[[ bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage == /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity* ]]
COVERAGE_DIR=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
mkdir -p /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
COVERAGE_OUTPUT_FILE=bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat
[[ bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat == /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity* ]]
COVERAGE_OUTPUT_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat
export JAVA_COVERAGE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/jvcov.dat
JAVA_COVERAGE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/jvcov.dat
export COVERAGE=1
COVERAGE=1
export BULK_COVERAGE_RUN=1
BULK_COVERAGE_RUN=1
[[ -z '' ]]
export GCOV_PREFIX_STRIP=3
GCOV_PREFIX_STRIP=3
export GCOV_PREFIX=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
GCOV_PREFIX=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
export LLVM_PROFILE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/%h-%p-%m.profraw
LLVM_PROFILE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/%h-%p-%m.profraw
[[ ! -z '' ]]
[[ 0 == \0 ]]
cd /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.sh.runfiles/unity
[[ 1 == \0 ]]
/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.sh.runfiles/unity/trumid/common/utils/caseclass/tests.sh
Discovery starting.
Discovery completed in 58 milliseconds.
Run starting. Expected test count is: 1
CaseClassFactoryTest:

should instantiate a case class from it's corresponding tuple (7 milliseconds)
Run completed in 100 milliseconds.
Total number of tests run: 1
Suites: completed 2, aborted 0
Tests: succeeded 1, failed 0, canceled 0, ignored 0, pending 0
All tests passed.

TEST_STATUS=0
[[ 0 -ne 0 ]]
[[ 1 == \1 ]]
[[ 0 == \0 ]]
exit 0
[[ -z bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files ]]
export ROOT=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity
ROOT=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity
[[ bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files != /* ]]
export COVERAGE_MANIFEST=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files
COVERAGE_MANIFEST=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/trumid/common/utils/caseclass/tests.instrumented_files
export COVERAGE_DIR=bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
COVERAGE_DIR=bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
[[ bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage == /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity* ]]
COVERAGE_DIR=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
mkdir -p /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
COVERAGE_OUTPUT_FILE=bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat
[[ bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat == /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity* ]]
COVERAGE_OUTPUT_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat
export JAVA_COVERAGE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/jvcov.dat
JAVA_COVERAGE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/jvcov.dat
export COVERAGE=1
COVERAGE=1
export BULK_COVERAGE_RUN=1
BULK_COVERAGE_RUN=1
[[ -z '' ]]
export GCOV_PREFIX_STRIP=3
GCOV_PREFIX_STRIP=3
export GCOV_PREFIX=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
GCOV_PREFIX=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
export LLVM_PROFILE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/%h-%p-%m.profraw
LLVM_PROFILE_FILE=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage/%h-%p-%m.profraw
[[ ! -z '' ]]
[[ 1 == \0 ]]
[[ 1 == \1 ]]
[[ 1 == \0 ]]
[[ 1 == \1 ]]
[[ 1 == \1 ]]
touch /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/coverage.dat
cd /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity
[[ -n '' ]]
[[ -z bazel-out/darwin_arm64-fastbuild/bin/external/remote_coverage_tools/Main ]]
for name in "$LCOV_MERGER"
[[ ! -e bazel-out/darwin_arm64-fastbuild/bin/external/remote_coverage_tools/Main ]]
[[ 1 == \1 ]]
++ resolve_links /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
++ local name=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage
++ '[' -e /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/trumid/common/utils/caseclass/tests/_coverage ']'
++ '[' -L /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/_coverage ']'
++ echo /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/_coverage
COVERAGE_DIR=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/_coverage
++ resolve_links /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files
++ local name=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files
++ '[' -e /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files ']'
++ '[' -L /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files ']'
++ echo /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files
COVERAGE_MANIFEST=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files
LCOV_MERGER_CMD='bazel-out/darwin_arm64-fastbuild/bin/external/remote_coverage_tools/Main --coverage_dir=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/_coverage   --output_file=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/coverage.dat   --filter_sources=/usr/bin/.+   --filter_sources=/usr/lib/.+   --filter_sources=/usr/include.+   --filter_sources=/Applications/.+   --filter_sources=.*external/.+   --source_file_manifest=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files'
[[ -n '' ]]
[[ -n '' ]]
JAVA_RUNFILES=
exec bazel-out/darwin_arm64-fastbuild/bin/external/remote_coverage_tools/Main --coverage_dir=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/_coverage --output_file=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/coverage.dat --filter_sources=/usr/bin/.+ --filter_sources=/usr/lib/.+ --filter_sources=/usr/include.+ --filter_sources=/Applications/.+ '--filter_sources=.*external/.+' --source_file_manifest=/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/bin/common/utils/caseclass/tests.instrumented_files
Dec 17, 2022 6:12:20 PM com.google.devtools.coverageoutputgenerator.Main getTracefiles
INFO: Found 1 tracefiles.
Dec 17, 2022 6:12:20 PM com.google.devtools.coverageoutputgenerator.Main parseFilesSequentially
INFO: Parsing file /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/_coverage/jvcov11223020425242003820.dat
Dec 17, 2022 6:12:20 PM com.google.devtools.coverageoutputgenerator.Main getGcovInfoFiles
INFO: No gcov info file found.
Dec 17, 2022 6:12:20 PM com.google.devtools.coverageoutputgenerator.Main getGcovJsonInfoFiles
INFO: No gcov json file found.
Dec 17, 2022 6:12:20 PM com.google.devtools.coverageoutputgenerator.Main getProfdataFileOrNull
INFO: No .profdata file found.
Dec 17, 2022 6:12:20 PM com.google.devtools.coverageoutputgenerator.Main runWithArgs
WARNING: There was no coverage found.
java.io.IOException: Permission denied
at java.base/java.io.UnixFileSystem.createFileExclusively(Native Method)
at java.base/java.io.File.createTempFile(File.java:2170)
at com.google.testing.coverage.JacocoCoverageRunner.getUniquePath(JacocoCoverageRunner.java:345)
at com.google.testing.coverage.JacocoCoverageRunner$2.run(JacocoCoverageRunner.java:519)
================================================================================
INFO: LCOV coverage report is located at /private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/_coverage/_coverage_report.dat
and execpath is bazel-out/_coverage/_coverage_report.dat
INFO: Elapsed time: 0.933s, Critical Path: 0.67s
INFO: 2 processes: 1 internal, 1 local.
INFO: Build completed, 1 test FAILED, 2 total actions
//common/utils/caseclass:tests                                    FAILED in 0.5s
/private/var/tmp/_bazel_pcj/d7e82b68b84ece06dc82c3639a1eabcf/execroot/unity/bazel-out/darwin_arm64-fastbuild/testlogs/common/utils/caseclass/tests/test.log

Executed 1 out of 1 test: 1 fails locally.
INFO: Build completed, 1 test FAILED, 2 total actions
—Message ID: @.***>

Having the custom JacocoRunner build script, you could go and modify
JacocoRunner and add some debugging around the code that you see in the
trace, to check accessing which exact file results in a permission denied
error.

Message ID: @.***>

Circling back to this to note that, some time ago, I created https://github.com/stackb/bazel-jacocorunner.


Force bc to return ints, no decimals for 'shift'

The result from bc's calculation will return verious
decimal places due to bc's default scale (20).
sh's 'shift' doesn't like decimals.
Force scale=0 to cause bc to return integers.

Did you encounter any cases when it was actually causing issues?
The result of $(echo "($azimuth + 11.25)/22.5 % 16" | bc) is always an integer for me, never encountered any errors. However, I'm considering merging this to be on the safe side, just being curious.

Oops, forgot to add the results which prompted this:
@xenon:~/projects/ansiweather$ ./ansiweather
./ansiweather: 264: shift: Illegal number: .00000000000000000005
And I just realized WHY this is happening.  I use bc quite often, so I've set the environment variable BC_ENV_ARGS to cause it to start with the '-l' option and others.
This forces bc to default to 'scale=20' (among other things), and since it's in the environment, it's always active, waiting, stealthily.  Oops.
Adding 'scale=0;' to the calculation forces integer values from the calculation, regardless of how bc might be optioned.
As an aside, this may need to be added to other calculations, but I believe this is the only interaction with sh commands in which a non-integer might cause issues.

Thanks for the detailed explanations, makes sense. Merged this yesterday.
The other place where bc is used is to convert pressure to a different unit system, so this would not cause issues.


Limiting uploads to specific filetypes

I'm using regex on the server to only allow specific files to be uploaded. This works, and results in an error when the client tries to upload.
If I try to use the "fileTypes" option for the uploader (client-side), I don't see it behaving any differently?
What would be the best way to trigger an error to the user before they even get a chance to press the upload button?

It works for me with the semantic-ui form implementation — in the file picker on OSX, I can only choose image files, everything else is greyed out, which I assume is the intended behaviour. However, the current drop zone implementation doesn't appear to validate filetypes, and instead anything dragged&dropped is uploaded. Which version are you using?

I'm using Bootstrap + both types of uploaders. I did actually get the non-dropzone version to validate client-side (it was a regex error on my part), but not the Dropzone, so I guess we're on the same boat there.

There is a validate event, in which you get File object that you can work on with javascript. Look at the validation section of the documentation in here.

This is how validate method is used in the uploader on client:
https://github.com/tomitrescak/meteor-tomi-upload-jquery/blob/master/uploader.js#L201-L206

Closing for now .. if need be reopen please.


Page icons all display wrong icons

Info:

OS: macOS Catalina (also happens on Windows 11)
Logseq ver: 0.9.3
Plugin ver: v1.15.16
Theme name: none

Description:
When using page icons that I copied from nerd font and paste it into Logseq, Logseq does not display the icon that I actually copied.
To Reproduce:
Create a page type with an icon:: property.
Go to https://www.nerdfonts.com/cheat-sheet and click on copy icon for the "group" icon.

Now paste this icon into the icon:: property
Actual behavior:
The icon looks totally different in Logseq.

Expected behavior:
The icon looks exactly like the the one we copied from nerd font
This happened ever since I updated Logseq and the plugin. Now all my previously setup icon::s look wrong.

Same issues for me =(

I think this problems comes from the fact that v1.15.7 made the new Tabler icon set the default, which seems to be a breaking change for everyone who had the plugin installed already.
To solve this you need to change the config from Tabler back to Nerdfont:

@bitbonk Thank you! It helps 🙏


Fix #909, reorganize ES public API + msg definitions

Describe the contribution
Put all ES typedefs which are shared across API and telemetry messages into the cfe_es_extern_typedefs.h file.
Put all ES typedefs which define the telemetry interface into the cfe_es_msg.h file.  Also include structures which define the output of commands that write data files into this group (query all apps, query all tasks, query all CDS).
Removes some localized definitions and replace with MISSION scope definitions where appropriate/necessary.
Fixes #909
Testing performed
Build and sanity check CFE
Run all unit tests
Expected behavior changes
No impact to behavior
(Should not change any binary formats)
System(s) tested on
Ubuntu 20.04
Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

Adding commits to fix two other minor issues
Fixes #964 via commit d21d94a
Fixes #370 via commit f9f18a7

CCB 2020-10-28 APPROVED

Consider moving bool in cfe_es_msg
Discuss with Walt

CCB 2020-10-28 APPROVED

Discussed at splinter, moving bool didn't sound like it would be an issue.


BrowserContextExtensions should return BrowserContext for chaining

Current:
BrowserResponse response = Browser.Post(
    "form",
    with =>
    {
        with.Accept("text/plain");
        with.JsonBody<Form>(form);
    });

Could be better:
BrowserResponse response = Browser.Post(
    "form",
    with => with.Accept("text/plain").JsonBody<Form>(form));

The first one reads properly, the second one doesn't - first one reads "with this accept value and this json body",  second one reads "accept text plain and a json body" - that doesn't make sense.

I think that is only naming issue. That could be like this:
BrowserResponse response = Browser.Post(
    "form",
    ctx => ctx.WithAccept("text/plain").WithJsonBody<Form>(form));

or
BrowserResponse response = Browser.Post(
    "form",
    ctx => ctx.WithAccept("text/plain").And.WithJsonBody<Form>(form));

The first one is a syntax change for the sake of it, and also falls down when one of the extensions wants to return it's own builder type - the second one is just replacing "with" with "and", and is still less expressive imo.
Sorry, just don't see any value in this.

The value here is keep the syntax same as (or at least similar with) ResponseExtensions.


Feature/complex example

@grabbou New pull request based on #428

Closing, unable to pass es-lint


road.py: fix the error in close_vehicles_to();observation.py: adapt KinematicObservation and add see_behind

I propose some modifications related to the issue #66 :
I added both directions exo-vehicles observation in the KinematicObservation class, and set the ego-vehicle X position to zero in the observation when absolute == False

Codecov Report

Merging #68 into master will increase coverage by 0.10%.
The diff coverage is 62.50%.

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
+ Coverage   80.17%   80.27%   +0.10%
==========================================
  Files          40       40
  Lines        3137     3144       +7
==========================================
+ Hits         2515     2524       +9
+ Misses        622      620       -2

Flag
Coverage Δ

#unittests
80.27% <62.50%> (+0.10%)
:arrow_up:

Impacted Files
Coverage Δ

highway_env/envs/common/observation.py
77.36% <62.50%> (-0.78%)
:arrow_down:

highway_env/road/road.py
92.16% <0.00%> (-1.21%)
:arrow_down:

highway_env/envs/intersection_env.py
99.10% <0.00%> (+0.89%)
:arrow_up:

highway_env/envs/summon_env.py
96.66% <0.00%> (+1.66%)
:arrow_up:

highway_env/road/regulation.py
76.92% <0.00%> (+9.61%)
: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 d8de149...19e7e1c. Read the comment docs.

Ok after the last modifications I get:

with order = "sorted" and see_behind = True:

[[ 1.          1.          0.25        0.40740741  0.        ]
 [ 1.          0.00603194  0.          0.          0.        ]
 [ 1.          0.15726268  0.5        -0.00208349  0.        ]
 [ 1.         -0.17417744  0.5        -0.00469832  0.        ]
 [ 1.         -0.43574158 -0.25       -0.02057853  0.        ]]

with order = "sorted" and see_behind = False:

[[1.         1.         0.         0.36987187 0.        ]
 [1.         0.05157368 0.         0.01950613 0.        ]
 [1.         0.28685791 0.25       0.02730386 0.        ]
 [1.         0.40236949 0.         0.01630911 0.        ]
 [1.         0.62910764 0.75       0.03575808 0.        ]]

with order = "shuffled" and see_behind = True:

[[ 1.          1.          0.5         0.36987187  0.        ]
 [ 1.          0.19228631  0.25        0.03802319  0.        ]
 [ 1.         -0.1110027   0.         -0.02882871  0.        ]
 [ 1.         -0.23701134 -0.5         0.02480767  0.        ]
 [ 1.          0.35311491  0.          0.03228953  0.        ]]

with order = "shuffled" and see_behind = False:

[[ 1.          1.          0.75        0.36987187  0.        ]
 [ 1.          0.644497    0.          0.0444848   0.        ]
 [ 1.          0.24557883 -0.5        -0.00167471  0.        ]
 [ 1.          0.09860658 -0.25        0.03482058  0.        ]
 [ 1.          0.52330299 -0.5         0.03502487  0.        ]]

I think I'm good ?

Great work.
Just a little suggestion: Using interactive rebase before final merge to master to keep the commit history clean.