boards: lpcxpresso54114: Combine m4/m0 boards into one directory

Combines the lpcxpresso54114_m4 and lpcxpresso54114_m0 boards into one
directory containing both board configurations. This eliminates some
duplicate code and documentation, and better represents the actual
hardware.
Signed-off-by: Maureen Helm<EMAIL_ADDRESS>@MeganHansen This will change the url for the board document. We currently have two board documents (one for each core):

https://docs.zephyrproject.org/latest/boards/arm/lpcxpresso54114_m4/doc/lpcxpresso54114.html
https://docs.zephyrproject.org/latest/boards/arm/lpcxpresso54114_m0/doc/lpcxpresso54114_m0.html

This change will combine them into one:
https://docs.zephyrproject.org/latest/boards/arm/lpcxpresso54114/doc/lpcxpresso54114.html

Codecov Report

Merging #10921 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #10921   +/-   ##
=======================================
  Coverage    53.3%    53.3%
=======================================
  Files         215      215
  Lines       25900    25900
  Branches     5708     5708
=======================================
  Hits        13805    13805
  Misses       9776     9776
  Partials     2319     2319

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 f2b9cc6...ad9a0ee. Read the comment docs.


The tests don't pass on main on macOS

I started to look into it, but the TestPushup function is pretty hard to understand

$ go test ./...

On my MBP running Ventura 13.1, it takes about 10 seconds. (Most of the time, ~9 seconds, is the end-to-end test TestPushup that creates a server and makes a request for each subtest).  I'm going to try to use this lib and see if I can get any useful info out of it

no luck with that

hangs after:
$ go test -count=1 -v .
=== RUN   TestPushup
=== RUN   TestPushup/$name.up
=== RUN   TestPushup/$name.up/0.conf
=== RUN   TestPushup/$name.up/#00
=== RUN   TestPushup/attributes.up
=== RUN   TestPushup/attributes.up/#00
=== RUN   TestPushup/empty.up
=== RUN   TestPushup/empty.up/#00
=== RUN   TestPushup/escape.up
=== RUN   TestPushup/escape.up/#00
=== RUN   TestPushup/handler.up
=== RUN   TestPushup/handler.up/#00
=== RUN   TestPushup/if.up
=== RUN   TestPushup/if.up/0.conf

just like in the full buffered output

I've started instrumenting the function, and it seems that which particular request it hangs on is random

I'm pretty sure it's this line where the process ultimately hangs: https://github.com/adhocteam/pushup/blob/main/main_test.go#L182 , but I suppose that's not extremely exciting news

I pushed my instrumentation code to the instrument-test-failure branch. When a test succeeds, you'll see something like:
    main_test.go:166: /var/folders/cj/3fpctnd52vv37y9gpdgg7c200000gn/T/TestPushup3617381832/001/pushup.exe run -build-pkg github.com/adhocteam/pushup/build -page testdata/attributes.up -unix-socket /var/folders/cj/3fpctnd52vv37y9gpdgg7c200000gn/T/pushup282877214/sock
    main_test.go:195: launching read output
    main_test.go:229: waiting to kill
    main_test.go:247: making request
    main_test.go:288: waiting
    main_test.go:297: checking error
    main_test.go:185: waiting for build process to finish, 60590
    main_test.go:213: read output, needle found
    main_test.go:284: request completed
    main_test.go:233: done, killing -60590
    main_test.go:238: killed
    main_test.go:191: pushup process done
    main_test.go:291: wait completed
    main_test.go:294: closes completed

When it fails, you get something like:
=== RUN   TestPushup/empty.up/#00
    main_test.go:166: /var/folders/cj/3fpctnd52vv37y9gpdgg7c200000gn/T/TestPushup3617381832/001/pushup.exe run -build-pkg github.com/adhocteam/pushup/build -page testdata/empty.up -unix-socket /var/folders/cj/3fpctnd52vv37y9gpdgg7c200000gn/T/pushup4194445339/sock
    main_test.go:195: launching read output
    main_test.go:229: waiting to kill
    main_test.go:247: making request
    main_test.go:288: waiting
    main_test.go:297: checking error
    main_test.go:185: waiting for build process to finish, 60614
    main_test.go:213: read output, needle found
    main_test.go:284: request completed
    main_test.go:233: done, killing -60614
    main_test.go:238: killed

You can see that the pushup process done log - which fires right after the cmd.Wait for the pushup run process - never happen.
So it seems like Wait isn't returning, despite the fact that process 60614 is dead, and the process just sits there for 10 minutes until the timeout finally throws.
$ ps aux | grep 6061
llimllib         63105   0.0  0.0 408626880   1312 s009  S+    3:11PM   0:00.00 grep 6061

@llimllib Can you double-check that you pushed your changed to instrument-test-failure? If I look at https://github.com/adhocteam/pushup/commits/instrument-test-failure I don't see any new commits. I might be doing something wrong.

doh, I failed at git... one sec

ok, pushed a redo of that code, with slightly better output now:
$ go test -count=1 -v .
=== RUN   TestPushup
=== RUN   TestPushup/$name.up
=== RUN   TestPushup/$name.up/0.conf
    main_test.go:166: /var/folders/cj/3fpctnd52vv37y9gpdgg7c200000gn/T/TestPushup1917556785/001/pushup.exe run -build-pkg github.com/adhocteam/pushup/build -page testdata/$name.up -unix-socket /var/folders/cj/3fpctnd52vv37y9gpdgg7c200000gn/T/pushup4028056865/sock
    main_test.go:285: closer: waiting
    main_test.go:179: pushup: launching
    main_test.go:184: pushup: waiting, pid 51513
    main_test.go:209: ready listener: needle found
    main_test.go:263: test runner: making request
    main_test.go:273: test runner: sending done
    main_test.go:229: process killer: recvd done -51513
    main_test.go:280: test runner: complete
    main_test.go:233: process killer: killed -51513
    main_test.go:189: pushup: process done
    main_test.go:287: closer: wait complete
=== RUN   TestPushup/$name.up/#00
    main_test.go:166: /var/folders/cj/3fpctnd52vv37y9gpdgg7c200000gn/T/TestPushup1917556785/001/pushup.exe run -build-pkg github.com/adhocteam/pushup/build -page testdata/$name.up -unix-socket /var/folders/cj/3fpctnd52vv37y9gpdgg7c200000gn/T/pushup1160732222/sock
    main_test.go:285: closer: waiting
    main_test.go:179: pushup: launching
    main_test.go:184: pushup: waiting, pid 51537
    main_test.go:209: ready listener: needle found
    main_test.go:263: test runner: making request
    main_test.go:273: test runner: sending done
    main_test.go:280: test runner: complete
    main_test.go:229: process killer: recvd done -51537
    main_test.go:233: process killer: killed -51537
<process hangs here, with the "pushup" process hung on `Wait()`>

Interestingly, I just got a test run where the test hung on the cmd.Wait in the process killer goroutine, instead of the pushup goroutine:
=== RUN   TestPushup/$name.up/#00
    main_test.go:166: /var/folders/cj/3fpctnd52vv37y9gpdgg7c200000gn/T/TestPushup3674165444/001/pushup.exe run -build-pkg github.com/adhocteam/pushup/build -page testdata/$name.up -unix-socket /var/folders/cj/3fpctnd52vv37y9gpdgg7c200000gn/T/pushup1529436637/sock
    main_test.go:285: closer: waiting
    main_test.go:179: pushup: launching
    main_test.go:184: pushup: waiting, pid 52688
    main_test.go:209: ready listener: needle found
    main_test.go:263: test runner: making request
    main_test.go:273: test runner: sending done
    main_test.go:280: test runner: complete
    main_test.go:229: process killer: recvd done -52688
    main_test.go:189: pushup: process done

pushup wasn't checking the error on the Wait inside the process killer goroutine, I added a handler for it and got an error:
=== RUN   TestPushup/if.up/2.conf
    main_test.go:166: /var/folders/cj/3fpctnd52vv37y9gpdgg7c200000gn/T/TestPushup943142027/001/pushup.exe run -build-pkg github.com/adhocteam/pushup/build -page testdata/if.up -unix-socket /var/folders/cj/3fpctnd52vv37y9gpdgg7c200000gn/T/pushup1722349862/sock
    main_test.go:288: closer: waiting
    main_test.go:179: pushup: launching
    main_test.go:184: pushup: waiting, pid 56902
    main_test.go:210: ready listener: needle found
    main_test.go:266: test runner: making request
    main_test.go:276: test runner: sending done
    main_test.go:283: test runner: complete
    main_test.go:230: process killer: recvd done -56902
    main_test.go:234: process killer: wait caught an error wait: no child processes
    main_test.go:236: process killer: killed -56902

Removing cmd.Wait from the process killer goroutine seems to solve the issue, PR incoming


edit multiple social_link attribs on same page

This could solve #4782
(Social Link Attribute not saving multple values)
I know that the above editable_attribute.php also includes this same id. But I'm not sure if that will always be wrapped around the attrib form, and if it will stay the same on updates.
So, I choose to only make changes in the social_links map.
in form.php, I created a new wrapper, with it's own data-id (akID)
This id is given to the js init() at the end of the form.php
The js will handle this and will only fire on a click with the right akID
ps. I'm pretty new to using github, so, hope this is the correct way to do things.

This pull request needs to be rebased and reopened because we did some history rewriting, if you need help with this head into #dev in our slack channel
#4814


Add endpoint to deploy action

I have create a new action of category deploy.
action DoMongoDBInstall {   deploy(repository: "GlobalRepo", path: @'software', pattern: "installers/mongodb-linux-x86_64-rhel70-3.6.3.tgz");   deploy(repository: "GlobalRepo", path: @'software', pattern: "scripts/ase/mongodb-install.sh");   ExecMongoDBInstall;   return; }
And i am running this task like this:

and then:

But it is not asking for endpoint where to run.

What is the ExecMongoDBInstall?  Can you send me the details on it?

It's a procedure to execute the shell script on the endpoint.

Ok, got it.  Make an Action (ExecMongoDBInstallAction) to call the ExecMongoDBInstall procedure.  Then for your application version add the MongoDB component first in the layout, then the MongoDB Install Script component.
Make sure the MongoDB component has no pre/post/custom actions assigned.  For the MongoDB Install Script component set pre and custom actions to blank and post action to ExecMongoDBInstallAction.
The DoMongoDBInstall task and action are not needed.
DeployHub will send out the MongoDB tar file first to the end point, then send the install sh to the end point.  The post action on the MongoDB Install Script component will get executed after everything is on the end point.
An application is assigned to and environment and that environment has end points (servers) defined to it.  You will deploy the application to an environment and that is how DeployHub know which end points are being deployed to.
Send me the log output if you run into issues.

Yes, this way we are able to push the files and execute the script. But wanted if it's possible by creating custom actions. Is it not possible?

What you initially setup will execute a custom action but its outside the context of environments and servers.  You need to tie the custom action to a component so it has a context to work within.
See the following link for details on the deployment logic.
https://www.deployhub.com/helpdh/NetHelp/DeploymentProcessing.html

FYI, custom actions at the component level are used for components that require an alternate method of delivery, such as REST Post (Salesforce and IBM Data Power work this way).

Is there a way to just run some commands on an endpoint through application action without deploying anything?

There is no direct way to do that.   There is a work around in which you create a component with no component items in it.  Then associate what you want to run to the components custom action.   The component will need to be associated to an application.  Running the Deploy Task against the application will in turn run the components custom action.


refactor: rename state to localStateExperimental

Fixes #82
As discussed during our f2f hackathon, local state is not our primary focus at this point, hence moving it to experimental. It generally works, but it has this limitation of requiring explicit key to be set. The API may change in the future as well as implementation.
However, we're keeping it there as its quite useful (in experimenting)

[x] Rename state as experimental
[x] Explicitly require unique key to be set (will be fixed at some point, but we have other priorities for now)

@ericvicenti, question: shall we create a simple "experimental" docs page for state? I know API may change, but we have quite a few nice abstractions already. Maybe it's worth having a skeleton just for our future selves.

Lets not write the docs for state yet, maybe once we have finished all the other docs 😂


OCPBUGS-37934: Reverting ETCD cluster rebuild

This PR reverts:

https://github.com/openshift/hypershift/pull/4354
https://github.com/openshift/hypershift/pull/4419

This is quite destructive and automatic approach, and could have side-effects in clusters already existent.
We are working in another approach which is less destructive, meanwhile we look for the root cause of the issue.

/approve
/lgtm

/jira refresh

/retest

/retest

/hold cancel

/retest-required


Metaflow Pluginify our KFP Plugin

Benefits:

We fit into the Metaflow click paradigm getting access to context object giving us access to

        obj.graph,
        obj.flow,
        obj.package,
        obj.metadata,
        obj.datastore,
        obj.environment,
        obj.event_logger,
        obj.monitor,

We are also now packaging the Flow rather than passing around a Flow python file to curl download.
Tested that @conda decorator works

I've refactored how the graph is created.  A recursive DFS is the path towards binding foreach mapper subgraphs to the foreach node.
Foreach is a work in progress, this is because KFP doesn't support aggregating outputs into a join/reducer node.  Hence, we will need to save the step taskIds in a KV store.  Ideally, this is Dynamo to avoid consistency issues.  However, we could use S3 and only store step_name -> taskId, and for the foreach node we'd store step_name -> [generated mapper taskIds].  The join or reducer node will have access to the parent node/step name at compile time, but not the taskIds which are dynamic.
As part of this is for taskIds to not be set at compile time but at runtime using KFP/Argo taskIds, except for foreach mapper split nodes which may have taskId=step_name_ or more robustly taskId=<parent_task_id>step_name where the <parent_task_id> is generated by KFP/Argo).  This would only work when we have a KV store of step or node name -> taskId.  We'd persist and load that info much like the AWS Step Functions plugin does.

Hmm that seems like quite the additional complexity for a short term fix given we'd need to provision dynamo databases, set up access permissions
This PR may be confusing given it has TODO skeleton code to enable the foreach scenario.  That is out of scope of this PR.

Dynamo is one solution, and another with less infrastructure is to use S3 directly as KV store for taskIds (we will not use S3 list operations which have low consistency guarantees).
The suggestion to build directly to Argo would be taken if and only if storing in S3 becomes unfeasible, because writing our own compiler to Argo is a large work item because it must also adhere to what KFP expects as annotations and tags to the Argo yaml, and this will change between KFP versions.  Nevertheless, if we build our own compiler as was the original approach, then we'd ideally compile to an intermediate representation rather than Argo YAML https://github.com/kubeflow/pipelines/issues/3703, and see https://gitlab.com/datadrivendiscovery/d3m/-/issues/458#note_414060723


Unclear how to create embeddings index

Mousing over the x icon says "repository github.com/datastax/cassandra is not indexed and has no embeddings," but clicking on it does not help and there is nothing relevant in Settings.

What is the status of embeddings for private repos for Jetbrains at the moment? The Jetbrains plugin became GA just recently and I'm keen to switch to it over my vs code setup (Cody Pro user here, not enterprise). Is vs code still the only environment for non enterprise users that generates and uses embeddings for private repositories?

hey @serefarikan, we're working on supporting embeddings and it's still on the roadmap. Apologies for taking it so long on jetbrains. Going to close this as we have more information coming out later


use SparseArrays machinery for printing

Types here could hookup into SparseArrays show implementation for a nice printing.
julia> using SparseArrays, SuiteSparseGraphBLAS

julia> x = sprand(10, 10, 0.5)
10×10 SparseMatrixCSC{Float64, Int64} with 39 stored entries:
  ⋅       0.359801   ⋅         ⋅          ⋅          ⋅         ⋅        0.579892  0.592154  0.088285
  ⋅        ⋅         ⋅        0.522135    ⋅         0.840658  0.561799  0.158865   ⋅         ⋅
  ⋅       0.84019    ⋅         ⋅         0.189342    ⋅         ⋅         ⋅        0.889942  0.435336
  ⋅        ⋅        0.793216  0.0387143  0.295225    ⋅         ⋅        0.159388  0.97282   0.630283
 0.06779  0.784768   ⋅         ⋅          ⋅          ⋅         ⋅         ⋅        0.71467    ⋅
  ⋅       0.452141  0.5673     ⋅          ⋅         0.497747  0.892377   ⋅        0.624008   ⋅
  ⋅       0.461338   ⋅         ⋅         0.0744113   ⋅         ⋅         ⋅        0.817912   ⋅
  ⋅       0.685151   ⋅         ⋅          ⋅         0.4771     ⋅         ⋅         ⋅        0.110999
  ⋅        ⋅        0.671881   ⋅          ⋅          ⋅        0.848146  0.205909   ⋅         ⋅
  ⋅       0.775386   ⋅         ⋅         0.453898    ⋅        0.76114    ⋅         ⋅        0.478859

julia> GBMatrix(x)
10x10 GraphBLAS double matrix, bitmap by col
  39 entries, memory: 1.1 KB

    (5,1)    0.06779
    (1,2)    0.359801
    (3,2)    0.84019
    (5,2)    0.784768
    (6,2)    0.452141
    (7,2)    0.461338
    (8,2)    0.685151
    (10,2)    0.775386
    (4,3)    0.793216
    (6,3)    0.5673
    (9,3)    0.671881
    (2,4)    0.522135
    (4,4)    0.0387143
    (3,5)    0.189342
    (4,5)    0.295225
    (7,5)    0.0744113
    (10,5)    0.453898
    (2,6)    0.840658
    (6,6)    0.497747
    (8,6)    0.4771
    (2,7)    0.561799
    (6,7)    0.892377
    (9,7)    0.848146
    (10,7)    0.76114
    (1,8)    0.579892
    (2,8)    0.158865
    (4,8)    0.159388
    (9,8)    0.205909
    (1,9)    0.592154
    ...

This is what CUSPARSE types do in CUDA.jl https://github.com/JuliaGPU/CUDA.jl/blob/583b948ef4f1ab739047d19e7f65657a354ec28b/lib/cusparse/array.jl#L503
julia> using SparseArrays, CUDA

julia> x = sprand(3,3,0.5)
3×3 SparseMatrixCSC{Float64, Int64} with 7 stored entries:
  ⋅        0.662806   0.34835
 0.422577  0.0506328  0.932217
  ⋅        0.649229   0.753374

julia> xd = x |> cu
3×3 CUDA.CUSPARSE.CuSparseMatrixCSC{Float32, Int32} with 7 stored entries:
  ⋅         0.66280556   0.34835035
 0.4225766  0.050632793  0.9322167
  ⋅         0.649229     0.7533741

Yep I plan on doing this soon, thanks for the pointer on CuSparseMatrixCSC that will help.


fix(e2e): screenshot changes will display proper error message

closes #191
closes #192

@jelbourn @hansl

Can address commit in a follow-up if necessary. LGTM for getting the build green.

@jelbourn No real reason, to be honest, other than when I was looking at what others were doing in Node, it seemed more common to just throw a string.


Issue with Ratchet Deadlocked (PS2)

Hi I'm running Ratchet Deadlocked on PCSX2 v1.7.0-dev-1177-gedeb0d7bd and everything was working really well until I reached the second planet in the game (Catacrom Four) and the mouse injection suddenly stopped working. So I went back to the first planet (DreadZone Station) and it worked again. Then when I went back to Catacrom Four it still wouldn't work. So I guess the issue really is with Catacrom Four? I can't progress without using the mouse so I can't test this much further.
I've tried restarting PCSX2 and KAMI and the problem persisted - I'm not much of a coding guy so I'm not sure what other tests to run.
Otherwise it was working really well and you've done a cracking job, thanks! I'm really excited to get into this in the future!
Cheers,
Ben
(Edit: Described what "it" was in the first paragraph)

yes the ps2 single player is problematic, probably won't look into it again for a while.


Yarn Install

I have created yarn workspaces from scratch, but I am having trouble with this one. I cloned your repo & I created a Cloudinary account and input my settings in the .env file & I ran yarn install from the root, which I thought would install all dependencies and and got the following....
$ yarn workspaces info
yarn workspaces v1.17.3
{
"gatsby-transformer-cloudinary": {
"location": "packages/gatsby-transformer-cloudinary",
"workspaceDependencies": [],
"mismatchedWorkspaceDependencies": []
},
"site": {
"location": "site",
"workspaceDependencies": [],
"mismatchedWorkspaceDependencies": [
"gatsby-transformer-cloudinary"
]
}
}
Done in 0.06s.
Environment
System:
OS: Windows 10
CPU: (8) x64 Intel(R) Core(TM) i7-7700K CPU @ 4.20GHz
Binaries:
Yarn: 1.17.3 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
npm: 6.8.0 - C:\Program Files\nodejs\npm.CMD
Browsers:
Chrome
I appreciate your work on this, it looks like the future!

Maybe try yarn workspace site add "gatsby-transformer-cloudinary@*" — that should pick up the local package and resolve the mismatched dependency.

has this been resolved for you? @doakheggeness ?

Yes, thanks Sent from Mail for Windows 10 From: WilliamSent: Tuesday, December 24, 2019 3:19 PMTo: cloudinary-devs/gatsby-transformer-cloudinaryCc: Doak Heggeness; MentionSubject: Re: [cloudinary-devs/gatsby-transformer-cloudinary] Yarn Install (#8) has this been resolved for you? @doakheggeness ?—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.

Great to see all is solved for you @doakheggeness 👍


Remove slot-related code and address linting warnings

Changes Made in this PR
In this PR, I've removed all slot-related code from the project. This includes the removal of slot usage and associated dependencies, effectively eliminating the slot functionality.
Reason for the PR
The reason for this PR is to streamline and simplify the codebase by removing the slot functionality, which is no longer required for the project's current goals. This simplification enhances maintainability and reduces the complexity of the code.
Status
Code changes have been tested against prettier, or there are no code changes
I know how to update typings and have done so, or typings don't need updating
Semantic Versioning Classification
This PR should be considered a Major version update since it involves the removal of a significant feature (slots) from the project.
/claim #567

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

what the hell is going on

sorry i remove i was doing a deployment test


bug: Validator doesn't support format: email

Seems, that format email it's just string.
I have schema:
{
    "type": "object",
    "properties": {
        "email": {
            "type": "string",
            "format": "email"
        }
    }
}

I'm trying to validate data:
{"email": "just string"}

And validator said that's ok.

Have you read the documentation?
http://python-jsonschema.readthedocs.io/en/latest/validate/#validating-formats
You haven't included your code, so I don't know what you're running, but I suspect you haven't passed in a FormatChecker. Note also that this is pretty much the only instance where jsonschema deviates from the spec (intentionally), in that the email validator basically just does "@" in email.

Sorry, i don't take put a FormatsChecker.


Config JSON is invalid - empty hideSelectors array

The empty hideSelectors array is breaking the config file.
Need to remove the array if it's empty.

Fixed in #13


please upload the architecture picture

https://github.com/Azure/Industrial-IoT
There is an architecture section, the picture is a broken link

fixed with PR #97

Thanks @lucarv for raising the issue and @cristipogacean for getting it fixed

Let's keep this issue open to also track updating the diagrams with the latest pieces and in particular dependencies.


Update-DbaInstance - fixing a typo

Type of Change

[ ] Bug fix (non-breaking change, fixes # )
[ ] New feature (non-breaking change, adds functionality, fixes # )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Fix a typo discovered in #6742

🙇 good to see you, Kirill!


chore: replica alert typo

Dear Owner,
I have found a typo and submit this PR to fix
On the other hand, I wish to ask why master number MUST be larger than 3 for redis cluster?
In our use case, we wish to create a redis cluster with minimum resources for development environment, so I feel the requirement is too strict
Thank you

@Kaggggggga, Thanks for your contribution. can you please sign-off the commit?

This has to be fixed in the source repo https://github.com/kubedb/redis/pull/166


Fix parsing class properties as readonly

The readonly property was always false.

Thank you for the pr!!


Adding GarlandISD

Adding Garland ISD
Domain: https://garlandisd.net/

@la-ch3ferino Thank you for your patience while waiting for a review and processing your request. Pull request merged.


How to specify the mtl file in my obj files?

I have a .obj file an mtl file and an image.
However, when I try to convert into a gltf, it seems the mtl file is simply not recognized. Could you guys help me to figure out if there is something missing in the .obj file that causes the tool to not recognize the mtl?
The zip file contains a obj, a mtl and a image file for texture
Handle1.zip

Hi @liuzhen2008
I added mtllib Handle1.mtl to Handle1.obj and added file extensions to the images in Handl1.mtl. That fixed it for me.
Handle1.zip

Yes sir, that has indeed fixed it for me.  How can I repay you.


[Bug] [HomePage] HomePage miss some workflow instance status.

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Homepage miss workflow ready-to xx status.
What you expected to happen
xx
How to reproduce
xx
Anything else
No response
Version
dev
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

After #15181 merged, I will submit pr to fix.

closed by #15193


Add RabbitMQ production config

I assume the env var is set in the environments? If so, 👍

Not yet, was planning to set it if nobody was too offended by this pull request.

Does this mean we'll lose the magic file that's inserted as part of deployment? if so 🎉

Once https://github.com/alphagov/govuk-puppet/pull/4876 is in production this can be merged.

This is ready, you can verify it with:
fab staging class:backend do:'cat /etc/govuk/publishing-api/env.d/RABBITMQ_PASSWORD'
fab production class:backend do:'cat /etc/govuk/publishing-api/env.d/RABBITMQ_PASSWORD'


Use sharedpreferences to store darkmode state

Related Issue

Fixes #406
Checklist:

[x] I have read the contributor's guide.
[x] I linked an issue in the previous section
[x] I have commented on the linked issue
[x] I was assigned the linked issue (not required)
[x] I have tested the change to the best of my ability against the sandbox or a local build.

Optional items:

[ ] My change adds new text and requires a change to translations.
[ ] My change requires a change to the documentation.
[ ] I have submitted a PR to the documentation repo.
[ ] I was not able to test... (explain below, e.g. you did not have permissions to test a specific feature)
[ ] This change depends O-FISH Realm repository changes (explain below)
[ ] This change depends O-FISH Web repository changes (explain below)

Optional: Add any explanations here

Now the dark mode status is being saved in the shared preference, I have reverted all the changes related to realm

Optional: Add any relevant screenshots here

Congratulations! 🎉 🙌 🎆 ㊗
This PR is great and has been merged into the code. Thanks so much for contributing! If you'd like, you can request to work on another issue. We really appreciate your effort!


Basic HTML List types

I added all the Basic HTML List types

Thanks @Lahirutech for contributing.

@Lahirutech
further you can add css for these lists.

yes sure!!!


06 Error with pretrained Model

When I try to train the pre-trained model in chapter "06 transfer learning", I get the following error:
raise RuntimeError('''
RuntimeError:
An attempt has been made to start a new process before the
current process has finished its bootstrapping phase.
    This probably means that you are not using fork to start your
    child processes and you have forgotten to use the proper idiom
    in the main module:

        if __name__ == '__main__':
            freeze_support()
            ...

    The "freeze_support()" line can be omitted if the program
    is not going to be frozen to produce an executable.

    To fix this issue, refer to the "Safe importing of main module"
    section in https://docs.python.org/3/library/multiprocessing.html

I copied everything right, I guess. Does anyone know where my mistake lies?

In chapter 05 I ran into the same issue. By setting the number of workers in the DataLoader to 0, it solved this issue for me.


Can't install torch due to 503 during download.

Describe the bug

My deploy process runs a pip install during image build time on AWS.  This process has now failed a couple times in a row with the following error:
ERROR: THESE PACKAGES DO NOT MATCH THE HASHES FROM THE REQUIREMENTS FILE. If you have updated the package versions, please update the hashes. Otherwise, examine the package contents carefully; someone may have tampered with them.
    torch==2.1.1 from https://files.pythonhosted.org/packages/72/d0/8e7157fe416f657e38736a42d9b0b82ef7f7af00398516200b59ebb5995e/torch-2.1.1-cp311-cp311-manylinux2014_aarch64.whl (from -r requirements.txt (line 84)):
        Expected sha256 61b51b33c61737c287058b0c3061e6a9d3c363863e4a094f804bc486888a188a
             Got        db7f567ef4ee64ffdb28fe1cc71206584bdddc70e1e4a92e26b3671de6f9e32b

I tried fetching the file locally and got the correct SHA.  So, I spun up an instance in EC2, connected to it, and fetched the file.  That fetch failed after precisely 39MiB (a bit under half the expected size) with a 503 error.  The resulting file had the hash db7f567ef4ee64ffdb28fe1cc71206584bdddc70e1e4a92e26b3671de6f9e32b.
All subsequent attempts to fetch the file from that instance succeeded.
I don't have enough samples to be conclusive about it, but it appears to be an oddly deterministic failure in which the first attempt to fetch this file from an EC2 node results in the same incomplete file being produced consistently.
I did get a successful download on the third image build attempt, so it's not perfectly consistent (thankfully).  Honestly, I wouldn't bother reporting it except that it happened 3 times and produced the same incorrect hash every time.
Expected behavior
I would expect the server to not be sending me just under half the expected file on the first attempt.  Alternatively, I'd expect pip to properly retry (help screen says it should make 5 attempts by default, and I'm not overriding that).
To Reproduce
pip3.11 install --requirement requirements.txt

Where requirements.txt contains the line torch=2.1.1, and the command is performed from a Linux instance on EC2 in Amazon's us-east-1 region.
My Platform

We're using Debian 11, with Python 3.11.4 (SHA 85c37a265e5c9dd9f75b35f954e31fbfc10383162417285e30ad25cc073a0d63) built from source.
Additional context

This appears to be highly similar to https://github.com/pypa/pip/issues/11153 - which discusses the issue surfacing as an interrupted download - which you've confirmed as likely, seeing about half of the download size before the error surfaces.
There's also some discussion there as to retry logic.
Without some more details on the actual HTTP connections from the output, such as with python -m pip install -vvv ... it's hard to make a determination as to why this may have happened.
Do you have those kinds of logs or context?

I'm afraid I do not.  I will add -vvv to our image build process, and if there's another failure will provide the relevant data.

I have not seen it come up again, but will keep an eye out!  TY for the update!


"SQL Server Configuration Manager is installed with your SQL Server installation"

The statement "SQL Server Configuration Manager is installed with your SQL Server installation" is at least not always correct.  I have "SQL Server Management Studio" installed on Windows 10, can connect to "(LocalDb)\MSSQLLocalDB" and can see (and use) several databases there.
None of the "C:\Windows\SysWOW64\SQLServerManager1?.msc" are present.
I would suggest that this page is updated to contain information about how the relevant "SQL Server Configuration Manager" .msc file can be added if it missing.
[Enter feedback here]

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

ID: 767ab6f3-216d-6646-daf0-b95028c3aa2e
Version Independent ID: 631179d6-0647-a525-cea2-582e2c019f6b
Content: SQL Server Configuration Manager - SQL Server
Content Source: docs/relational-databases/sql-server-configuration-manager.md
Service: sql
GitHub Login: @markingmyname
Microsoft Alias: maghan

Duplicate of https://github.com/MicrosoftDocs/sql-docs/issues/8773.


e2e tests for update scenarios

Most (all?) of our current e2e tests are related to the deployment of CRs. We recently added support for instance updates and we do not have e2e tests for that.
Scenarios that would be nice to have as e2e tests:

[ ] Simple replica size update -- deploy a simple prod (Elasticsearch) with the collector's replica size to 2, check that spans can be stored/retrieved, update the replica size to 3, wait for the available replica size to be 3, and check again that spans can be stored/retrieved
[ ] Same as above, but deploying first a simplest (in memory) and then a simple-prod (ES).
[ ] Add volumes
[ ] Change operand's image (ie, custom jaeger-query)
[ ] Enable/disable ingress
[ ] Update the UI config map

@jpkrohling I may have time to work on this shortly, so I was wondering if you could:

Update the list of scenarios?  I can probably come up with some but would love more ideas than just changing the number of replicas.
Let me know what you think of combining things in test cases.  Ideally it would be nicer to have separate tests to say, test the simple-prod deployment, and then test changing the number of replicas in the simple-prod deployment.  However, this could end up adding to the time required to do PR builds.  (Although I suppose this depends on how many instances of minikube Github actions will let us run at a time.
)

I added a few extra cases to the list.

@iblancasa  Don't we have update tests now?

No, we haven't. This is something I have to work on

Update.
Covered scenarios right now:

[x] Simple replica size update -- deploy a simple prod (Elasticsearch) with the collector's replica size to 2, check that spans can be stored/retrieved, update the replica size to 3, wait for the available replica size to be 3, and check again that spans can be stored/retrieved: covered in the elasticsearch E2E test suite.
[x] Same as above, but deploying first a simplest (in memory) and then a simple-prod (ES): covered in the elasticsearch E2E test suite.
[x] Add volumes: covered in the examples E2E test suite.
[ ] Change operand's image (ie, custom jaeger-query)
[x] Enable/disable ingress: covered as part of the ui E2E test suite.
[x] Update the UI config map: covered as part of the ui  E2E test suite.


Allow for more than 2 splits per node

Is your feature request related to a problem? Please describe.
At the moment, a node can only be split horizontally or vertically once, with a fraction indicating how far apart the two are.
In the scenario where I split horizontally twice, I would love for it to instead split into 3 equally-sized chunks [A | B | C], rather than
[A | [B | C]

Describe alternatives you've considered
We could hack around this by twiddling with fraction values based on the tree as a whole, but it's probably easy enough to migrate into a non-binary tree structure. probably.
Additional context
I'm planning to work on this over the weekend, but making an issue for it just incase this is something you're interested in.

Closed by #145

@Adanos020 In my understanding this PR issue is not closed by #145.
This issue wants to allow more than 2 nodes in the tree children.
The PR adds the possibility do disable split in a certain

@lvaroqui Oh, you're right. Sorry about that!


Slack invite link broken in README

When trying to join the community Slack linked in the README, you receive a page with the following message:
This link is no longer active
To join this workspace, you’ll need to ask the person who originally invited you for a new link.

How to reproduce the bug

Go to 'README.md'
Click the "Join our community's Slack" link
See error

Expected results
Get an invite to the Slack community
Actual results
Get an inactive link error message
Environment
(please complete the following information):

Chrome 99.0.4844.84
superset version: master branch
python version: N/A
node.js version: N/A
any feature flags active: no

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Seems similar to this older issue: https://github.com/apache/superset/issues/8613

Thanks @MCBoarder289 for the catch! I will fix this ASAP


Feature: mountable prebuilt generic installation images

Imagine offering prebuilt generic Qt versions in a mountable disk image, similar to older offline Qt installers for macOS (example). This generic version of Qt would include most popular modules (e.g., qtbase, qtdeclarative, qtmultimedia). However, instead of a large installation executable, the image file would contain a directory with an already-installed Qt, ready to mount and use immediately. We could mount the image file as a read-only file system (e.g., via FUSE on Linux) and use it immediately. This mountable prebuilt installation would significantly enhance overall installation speed.
If there is already an option for this and I missed it, I apologize in advance.

Seems like this is out-of-scope for this simple aqtinstall wrapper, and would be better implemented as a separate action.

Does the Cache mechanism ^1 help you? I don't remember the official offline installer clearly, but I think its installation will not be largely different from the one installed by aqtinstall ^2 , which is the backend of this action.

Yes, cache helps a lot. This was just a suggestion, anyway. Thanks.


Refresh CSV Automatically

Is your feature request related to a problem? Please describe.
The query results from CSV are not refreshed automatically.
(In my memory, the previous version was fine……)
Describe the solution you'd like
Automatically refresh
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Should be easy via cache-invalidation on CSV file changes.

Can I expect this issue to be implemented in the next update?
(a request from someone who restarts Obsidian dozens of times a day😭)

Will be next update, or the update after the large section metadata update.


Custom optiont for BelongstoMany

Please, help!
Can I make custom options for BelongstoMany select? And How?

Hi! If you want custom options, you can just provide a custom endpoint using ->api() and populate the relationship using ->fillUsing().


Cache emoji icons

The emoji pictures are currently downloaded on request and it makes them to "pop in" when they first appear (either in the channel messages or emoji preview).
The emojis should be cached locally in the repository in client/src/icons/emojis.
Before doing this, we need to check the license for the emojis we currently use (https://github.com/banyan/react-emoji), ie. can we package them with the app?

react-emoji is MIT. The emojione assets it uses are under creative commons. I think we can package them with attribution

Excellent. There's no rush to this. It's more of an enhancement/performance optimization.

these should be packaged directly through ipfs-- there should be a bundle of assets that ships with orbit-- similar to the readme files that ship with go-ipfs

@haadcode you are currently using the http version for jsdelivr, but it is also available on https:

http://cdn.jsdelivr.net/emojione/assets/svg/1F642.svg
https://cdn.jsdelivr.net/emojione/assets/svg/1F642.svg

that would at least make chrome less sad about insecure content

Thank @dignifiedquire! This was done here: https://github.com/haadcode/orbit/commit/3a4fc96ce17b28824ce7a959424af50d6933c3fc. It fixes emojis in orbit-inside-orbit case where they didn't load previously.
👍
Next: cache them locally or load from IPFS.


[fix] potential vulnerabilities

Coverage decreased (-14.3%) to 70.122% when pulling 709b33732bd59c177fad79aa77047e43c94c4afd on sunny4381:fixes/potential-vulnerability into c52eaffded14edde9ac3b38138bd3f860da4c7ac on shirasagi:master.


Pros and cons of using this dynamic approach vs hard coded signals.

As the title says. What are the pros and cons of using this dynamic approach vs defining all the signals in an auto load.
I could think of a pro that you don't need to open the file every time you add or change something in the game. But also a con would be that I need to document all the events and don't have auto complete (making it more error prone).

I'd say main advantage is loose coupling of events and nodes. Any node can send any event everywhere without explicitly wiring them together.
Another advantage is ability to have multiple event buses and not mixing events in them.
Disadvantage? I'd say it's less performant than signals

I see. Thanks for the quick answer. But would you mind sharing how you keep track of the events you are using? Do you document them in a readme? Or somewhere else? Having to use strings is one of the biggest cons which are giving me a hard time deciding between both approaches.

Just a quick note. I've actually just tried implementing a simple spelling mistake check using the levenshtein distance algo. That way I can at least get an error message logged suggesting the most probable event name / correcting spelling mistakes.

Also another question. Why are you processing the events "tick based" or on update rather than just calling every subscribed Callable instantly whenever an event is invoked?

Also another question. Why are you processing the events "tick based" or on update rather than just calling every subscribed Callable instantly whenever an event is invoked?

if there are many subscibers and their functions take long to complete, calling them "by tick" will give game some breathing room to run


Added root redirect to github wiki docs

Closes #222

Added redirect to wiki in the root path

Pull Request Test Coverage Report for Build 659

0 of 4 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 78.569%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/mod.rs
0
4
0.0%

Totals

Change from base Build 653:
-0.06%

Covered Lines:
3864

Relevant Lines:
4918

💛  - Coveralls

Pull Request Test Coverage Report for Build 659

0 of 4 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 78.569%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/mod.rs
0
4
0.0%

Totals

Change from base Build 653:
-0.06%

Covered Lines:
3864

Relevant Lines:
4918

💛  - Coveralls


UI Color Issues

List of Color Issues:
Main Stack screen

[x] Sidekick grey background color same as grey part of grid color
[x] the Blue line that underlines Applications on main menu and the highlight for Stack/Catalog sub-menu, makes it seem like there is a highlight bug.  This would be true for the other main menu titles and submenus. (added up arrows instead lv)
[x] The color for a degraded Stack is the same "peachy" color that we are using to show the number of Containers.  Maybe degraded stack symbol should be red?

Stack Details Screen

[x] Degraded text is kinda hard to see because it's a light peach color.
[x] Can barely see the line between the Stack Name and the view icons.
[x] The view icons (tiny dots, containers, etc...) are all light grey and look like they are disabled
[x] + Scale up text is too light (can barely see it and it look like it's disabled)
[x] The little green outline circle that says a service is running, it's hard to tell it's green and running because it's not solid green circle.

Main Host Screen & Host Details screen

[ ] Main - The little green outline circle that says a service is running, it's hard to tell it's green and running because it's not solid green circle.
[ ] Main - Hard to see circle that tells you how many sidekicks you have.  I think it might be a light green outline circle here as well.  But hard to tell and see because of grey background.
[x] Main - + Add Container text is too light and hard to see.
[x] Details - Container & Storage tabs when unselected is almost same color grey as background.

Other Screens

[x] All Screens - the light grey Title text for input fields make it look like this fields are unavailable?  This is especially noticeable on LDAP access control screen.  I'm not sure you would want to change this as it would have to be changed on every single screen, but thought I would bring it up.
[x] All screens - Can barely see the grey line between the screen name and options.  Example: On the main stack screen, the line between the "Stacks" and the "Add Stack" button.
[x] Environments & Setting dropdown from Main menu - The grey title text (All Environments, Your Accounts, This Environment) are too light.  Can barely see them.
[ ] Grids - Grey outline colors for grids is also very faint.
[ ] Alerts - The color on Alerts is orange, maybe it should red

State colors should be some form of
Active/Running: green
Degraded: orange/yellow
Inactive/Stopped: red
Transitioning: something else (currently blue, which I like because not red or green)

Running is a hollow circle vs solid to provide an alternate way to tell which is which for color-blind people.

Version - Master with latest reskin UI
Verified fixed


blockchain_market_* api calls are way too complicated

It is almost impossible to extract a simple order book for a given market using RPC API calls. Where "simple" means an ordered list of currently executable bid/ask offers, each with volume and price.
Currently, various API calls exist for retrieving partial order books etc., which can be combined into a "simple" orderbook by applying complicated (and mostly undocumented, and changing) market logic. Examples:

blockchain_market_order_book doesn't include shorts.
not all shorts are executable at all times, they depend on the feed price.
not all cover orders are executable at all times, they depend on the feed price and on their expiration time.

Things will get even more complicated with the upcoming relative order types.

I agree. @valzav What are all the commands you use to construct the order book shown in the frontend?

He does it the same way I do: by avoiding the use of blockchain_market_order_book :)
We both pull the individual orders using list_asks, list_bids, list_shorts and list_covers and then construct the order book ourselves.


EXCEPTION: No provider for CookieOptions!

For AOT compilation I'm getting such error.

I've tried to fix this for myself, feel free to take a look here:
https://github.com/sebastianhaas/angular2-cookie/tree/fix-aot-compile
I added a CookieModule which you can import in your module. It will provide the CookieService.  If you want to try, run npm pack on my branch and install from there.
However, I don't have much experience with gulp nor time to figure out the rather complex setup of this project to prepare it as a PR atm :(

@sebastianhaas thank you, you helped me a lot with link to same issue.
I've implemented this approach:
export function cookieServiceFactory(): CookieService {
  return new CookieService();
}

I still would be nice to have solution for box.

It will be in the next version, which I plan to release this weekend.

@salemdar I really appreciate if you could fix this sooner.

You may want to check out salemdar/ngx-cookie.

Thanks @sebastianhaas! salemdar/ngx-cookie solved the problem.

@salemdar @sebastianhaas  does this mean that 'ngx-cookie' will be supported in future and we can move from 'angular2-cookie' to 'ngx-cookie'?  (indicating 'angular2-cookie' is dead )

I am using 'ngx-cookie' instead of 'angular2-cookie' and i get same error.

I have this error when testing with Karma/Jasmine.

Typescript Error
Module '"node_modules/angular2-cookie/services/cookie-options-args.model"' has no exported member 'CookieOptions'.


Reformat the processing report

Paul wrote
Could the error list be produced as a CSV file? This would make it easier to work with than text in an email, with columns for error type, line no etc
@geoffj-FUG wrote
Kirk. This is your domain.

Given the several hundred messages to reformat individually this will take about a week of work to go through each message.

Kirk
Another ‘nice to have’
I am not sure that the gains outweigh the effort involved.
It is not essential for implementation and migration.
Geoff
From: Kirk Dawson<EMAIL_ADDRESS>Sent: Thursday, September 10, 2020 6:15 AM
To: FreeUKGen/FreeCENMigration<EMAIL_ADDRESS>Cc: geoffj-FUG<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [FreeUKGen/FreeCENMigration] Reformat the processing report (#955)
Given the several hundred messages to reformat individually this will take about a week of work to go through each message.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/955#issuecomment-689796879 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFIDZ3W34U2LOG3JK5DSE7O3HANCNFSM4Q3QZBTQ .  https://github.com/notifications/beacon/AKCPIFNLTKR3WFDQJ3YRYCDSE7O3HA5CNFSM4Q3QZBT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFEOXODY.gif

Will leave on the back burner for a future release

@geoffj-FUG
wrote:
How difficult is it to get a text version of the Error report
I ask because I have been helping a relatively new transcriber through sorting out her errors.
To get a text error report I copied the Warnings and Errors from the email, amended all the manual page breaks to paragraphs, sorted the results so that errors were at the top and copied and pasted into an email.
It is something that Coordinators need to do regularly.
If a Coordinator could get a text version of the Errors (only) then it would make life much easier. Not all Coordinators have the knowledge to do what I did either.
To which @Captainkirkdawson responded:
Are you talking about the processing report?
If you recall we have story Reformat the processing report#955 which was put on the back burner
The processing report is purely a sequential processing log and likely should remain that way. As you know the processor will crash from time to time and the sequential log is essential in error tracing.
I ask myself why we are not encouraging people to use the online system to view the error and warning reports. Even if the corrections are being done off line one could still have the error list on the screen.
We are trying to move to more modern processes.
Rather than jumping into the reformatting of the existing processing report. Let's go back and ask ourselves what is the requirement and then look for the best solution. There are many ways if  I know the actual requirements

Kirk
I will have to think that one out.
Geoff
From: Kirk Dawson<EMAIL_ADDRESS>Sent: Sunday, October 18, 2020 5:37 AM
To: FreeUKGen/FreeCENMigration<EMAIL_ADDRESS>Cc: geoffj-FUG<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [FreeUKGen/FreeCENMigration] Reformat the processing report (#955)
@geoffj-FUG https://github.com/geoffj-FUG
wrote:
How difficult is it to get a text version of the Error report
I ask because I have been helping a relatively new transcriber through sorting out her errors.
To get a text error report I copied the Warnings and Errors from the email, amended all the manual page breaks to paragraphs, sorted the results so that errors were at the top and copied and pasted into an email.
It is something that Coordinators need to do regularly.
If a Coordinator could get a text version of the Errors (only) then it would make life much easier. Not all Coordinators have the knowledge to do what I did either.
To which @Captainkirkdawson https://github.com/Captainkirkdawson  responded:
Are you talking about the processing report?
If you recall we have story Reformat the processing report#955 which was put on the back burner
The processing report is purely a sequential processing log and likely should remain that way. As you know the processor will crash from time to time and the sequential log is essential in error tracing.
I ask myself why we are not encouraging people to use the online system to view the error and warning reports. Even if the corrections are being done off line one could still have the error list on the screen.
We are trying to move to more modern processes.
Rather than jumping into the reformatting of the existing processing report. Let's go back and ask ourselves what is the requirement and then look for the best solution. There are many ways if I know the actual requirements
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/955#issuecomment-711069653 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFJAKFUCFZ72NQX3LE3SLHW67ANCNFSM4Q3QZBTQ .  https://github.com/notifications/beacon/AKCPIFIEPIDQMMSVVQCJ3EDSLHW67A5CNFSM4Q3QZBT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFJRA7VI.gif

Kirk
Download Message report does what I wanted. If it was in .csv format it can be filtered in a spreadsheet and the Coordinator can comment against those messages that they need to.
Geoff
From: Kirk Dawson<EMAIL_ADDRESS>Sent: Sunday, October 18, 2020 5:37 AM
To: FreeUKGen/FreeCENMigration<EMAIL_ADDRESS>Cc: geoffj-FUG<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [FreeUKGen/FreeCENMigration] Reformat the processing report (#955)
@geoffj-FUG https://github.com/geoffj-FUG
wrote:
How difficult is it to get a text version of the Error report
I ask because I have been helping a relatively new transcriber through sorting out her errors.
To get a text error report I copied the Warnings and Errors from the email, amended all the manual page breaks to paragraphs, sorted the results so that errors were at the top and copied and pasted into an email.
It is something that Coordinators need to do regularly.
If a Coordinator could get a text version of the Errors (only) then it would make life much easier. Not all Coordinators have the knowledge to do what I did either.
To which @Captainkirkdawson https://github.com/Captainkirkdawson  responded:
Are you talking about the processing report?
If you recall we have story Reformat the processing report#955 which was put on the back burner
The processing report is purely a sequential processing log and likely should remain that way. As you know the processor will crash from time to time and the sequential log is essential in error tracing.
I ask myself why we are not encouraging people to use the online system to view the error and warning reports. Even if the corrections are being done off line one could still have the error list on the screen.
We are trying to move to more modern processes.
Rather than jumping into the reformatting of the existing processing report. Let's go back and ask ourselves what is the requirement and then look for the best solution. There are many ways if I know the actual requirements
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/955#issuecomment-711069653 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFJAKFUCFZ72NQX3LE3SLHW67ANCNFSM4Q3QZBTQ .  https://github.com/notifications/beacon/AKCPIFIEPIDQMMSVVQCJ3EDSLHW67A5CNFSM4Q3QZBT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFJRA7VI.gif

@geoffj-FUG the penny just dropped at this end.
You were talking about the extension .log; takes 1 minute to make that change
I had thought you were asking for a reordering and reformatting of the report!!
It never crossed my mind it was the extension.
Any test editor will open a log extension so its never an issue for me. But I can see that for some people it might be an issue.

Kirk
Can we add it to the To Do list please? It is not urgent as I am probably the only person who is using the system to manage their volunteers.

Geoff
From: Kirk Dawson<EMAIL_ADDRESS>Sent: Monday, October 19, 2020 7:03 AM
To: FreeUKGen/FreeCENMigration<EMAIL_ADDRESS>Cc: geoffj-FUG<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [FreeUKGen/FreeCENMigration] Reformat the processing report (#955)
@geoffj-FUG https://github.com/geoffj-FUG  the penny just dropped at this end.
You were talking about the extension .log; takes 1 minute to make that change
I had thought you were asking for a reordering and reformatting of the report!!
It never crossed my mind it was the extension.
Any test editor will open a log extension so its never an issue for me. But I can see that for some people it might be an issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/955#issuecomment-711422984 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFPTUDXHPKXMQJNFY2TSLNJZRANCNFSM4Q3QZBTQ .  https://github.com/notifications/beacon/AKCPIFIM766CKUAOFPYY6FTSLNJZRA5CNFSM4Q3QZBT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFJTXICA.gif

Done on test3.
Will look at adding the download of the warning messages in txt and csv formats as well as downloading the error messages again in csv and txt formats


Disabled media buttons on my keyboard

Works great but it disabled functionality of my media buttons on keyboard.
that's my keyboard if it helps
https://snipboard.io/20dnyV.jpg

Cant reproduce this. Media buttons seems to work for me while audio-band is running, though i dont have the same keyboard.
Does anything happen when you use the media keys, perhaps control switched to a chrome tab running Youtube or something?

I forgot to reply here...i had no clue why this was happening but i've fixed my issue. It had something to do with my keyboards firmware. Now i've updated it and it works great.


react-dom is an implicit dependency in order to support<EMAIL_ADDRESS>

Hi, I'm trying to run a very simple test using mocha:
Spec file:
import React from 'react';
import { shallow } from 'enzyme';
import { expect } from 'chai';

import Header from './index.js';

describe('Header', () => {

  let wrapper;

  beforeEach(() => {
    shallow(<Header name="Hello" />);
  });

  it('mounts correctly', () => {
    expect(wrapper.is('header')).to.be.truthy();
  });
});

Output:

➜  component-library git:(master) ✗ npm test

><EMAIL_ADDRESS>test component-library
> mocha --require ./test-setup.js src/**/*spec.js

react-dom is an implicit dependency in order to support<EMAIL_ADDRESS>Please add the appropriate version to your devDependencies. See https://github.com/airbnb/enzyme#installation
component-library/node_modules/enzyme/build/react-compat.js:113
      throw e;
      ^

Error: proxies not supported on this platform. On v8/node/iojs, make sure to pass the --harmony_proxies flag
    at new global.Proxy (component-library/node_modules/harmony-reflect/reflect.js:2116:13)
    at Object.<anonymous> (component-library/node_modules/react-dom/lib/SyntheticEvent.js:188:22)
    at Module._compile (module.js:410:26)
    at Module._extensions..js (module.js:417:10)
    at Object.require.extensions.(anonymous function) [as .js] (component-library/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (component-library/node_modules/react-dom/lib/SyntheticCompositionEvent.js:13:22)
    at Module._compile (module.js:410:26)
    at Module._extensions..js (module.js:417:10)
    at Object.require.extensions.(anonymous function) [as .js] (component-library/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (component-library/node_modules/react-dom/lib/BeforeInputEventPlugin.js:16:33)
    at Module._compile (module.js:410:26)
    at Module._extensions..js (module.js:417:10)
    at Object.require.extensions.(anonymous function) [as .js] (component-library/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (component-library/node_modules/react-dom/lib/ReactDefaultInjection.js:14:30)
    at Module._compile (module.js:410:26)
    at Module._extensions..js (module.js:417:10)
    at Object.require.extensions.(anonymous function) [as .js] (component-library/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (component-library/node_modules/react-dom/lib/ReactDOM.js:16:29)
    at Module._compile (module.js:410:26)
    at Module._extensions..js (module.js:417:10)
    at Object.require.extensions.(anonymous function) [as .js] (component-library/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (component-library/node_modules/react-dom/index.js:3:18)
    at Module._compile (module.js:410:26)
    at Module._extensions..js (module.js:417:10)
    at Object.require.extensions.(anonymous function) [as .js] (component-library/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at component-library/node_modules/enzyme/build/react-compat.js:109:18
    at Object.<anonymous> (component-library/node_modules/enzyme/build/react-compat.js:192:5)
    at Module._compile (module.js:410:26)
    at Module._extensions..js (module.js:417:10)
    at Object.require.extensions.(anonymous function) [as .js] (component-library/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (component-library/node_modules/enzyme/build/Utils.js:62:20)
    at Module._compile (module.js:410:26)
    at Module._extensions..js (module.js:417:10)
    at Object.require.extensions.(anonymous function) [as .js] (component-library/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (component-library/node_modules/enzyme/build/MountedTraversal.js:31:14)
    at Module._compile (module.js:410:26)
    at Module._extensions..js (module.js:417:10)
    at Object.require.extensions.(anonymous function) [as .js] (component-library/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (component-library/node_modules/enzyme/build/ReactWrapper.js:35:25)
    at Module._compile (module.js:410:26)
    at Module._extensions..js (module.js:417:10)
    at Object.require.extensions.(anonymous function) [as .js] (component-library/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (component-library/node_modules/enzyme/build/index.js:6:21)
    at Module._compile (module.js:410:26)
    at Module._extensions..js (module.js:417:10)
    at Object.require.extensions.(anonymous function) [as .js] (component-library/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (component-library/src/components/Header/spec.js:2:1)
    at Module._compile (module.js:410:26)
    at loader (component-library/node_modules/babel-register/lib/node.js:144:5)
    at Object.require.extensions.(anonymous function) [as .js] (component-library/node_modules/babel-register/lib/node.js:154:7)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at /usr/local/lib/node_modules/mocha/lib/mocha.js:219:27
    at Array.forEach (native)
    at Mocha.loadFiles (/usr/local/lib/node_modules/mocha/lib/mocha.js:216:14)
    at Mocha.run (/usr/local/lib/node_modules/mocha/lib/mocha.js:468:10)
    at Object.<anonymous> (/usr/local/lib/node_modules/mocha/bin/_mocha:403:18)
    at Module._compile (module.js:410:26)
    at Object.Module._extensions..js (module.js:417:10)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Function.Module.runMain (module.js:442:10)
    at startup (node.js:136:18)
    at node.js:966:3
npm ERR! Test failed.  See above for more details.

I've uploaded a gist with my dependencies.
Any ideas what I'm doing wrong here?
Thank you.

What version of node are you using? it looks like the error is actually in harmony-reflect, a dependency of yours (and since using --harmony is a horrifically bad idea, you probably don't need that dep)

Hi, I'm using v2.4.6.

Um, node v2? That'd be io.js - are you sure?

Oh, sorry, mixed up my digits here - v4.2.6.

Oh, and just for the record, here is the required test-setup.js

And what version of jsdom? What's your package.json look like? Do you use harmony-reflect in your own code?

I've added my dependencies in the OP: https://gist.github.com/doque/57fe0cbbb0d57a76cca2f515eca786fb
I'm not using harmony-reflect anywhere.

And just to be absolutely sure - can you also provide a gist of your npm ls output?

Sure, but that command is failing with this error message:
npm ERR! extraneous<EMAIL_ADDRESS>component-library/node_modules/archy
npm ERR! extraneous<EMAIL_ADDRESS>component-library/node_modules/caching-transform
npm ERR! extraneous<EMAIL_ADDRESS>component-library/node_modules/default-require-extensions
npm ERR! extraneous<EMAIL_ADDRESS>component-library/node_modules/istanbul-lib-hook
npm ERR! extraneous<EMAIL_ADDRESS>component-library/node_modules/istanbul-lib-instrument
npm ERR! extraneous<EMAIL_ADDRESS>component-library/node_modules/istanbul-lib-report
npm ERR! extraneous<EMAIL_ADDRESS>component-library/node_modules/istanbul-lib-source-maps
npm ERR! extraneous<EMAIL_ADDRESS>component-library/node_modules/istanbul-reports
npm ERR! extraneous<EMAIL_ADDRESS>component-library/node_modules/merge-source-map
npm ERR! extraneous<EMAIL_ADDRESS>component-library/node_modules/node-pre-gyp
npm ERR! extraneous<EMAIL_ADDRESS>component-library/node_modules/resolve-from
npm ERR! extraneous<EMAIL_ADDRESS>component-library/node_modules/spawn-wrap
npm ERR! extraneous<EMAIL_ADDRESS>component-library/node_modules/test-exclude

Yields this output before the error message:
https://gist.github.com/doque/d96297bc322652416d48ad0a167cd4ae

That output can be resolved by running npm prune, but that shouldn't impact this issue either way.

We're encountering the same thing as of today.
> mocha server/test-helpers/jsdom-browser.js "src/**/*.spec.js*"

react-addons-test-utils is an implicit dependency in order to support<EMAIL_ADDRESS>Please add the appropriate version to your devDependencies. See https://github.com/airbnb/enzyme#installation
/home/pmetzdorf/IdeaProjects/datavis/node_modules/enzyme/build/react-compat.js:132
      throw e;
      ^
Error: Cannot find module 'react-dom/lib/ReactTestUtils'
    at Function.Module._resolveFilename (module.js:455:15)

Any recent updates on the test utils maybe?

Yes, react 15.4 removed ReactTestUtils from react-dom. However, react-addons-test-utils (and enzyme) handles this just fine, unless your dependencies aren't correct.

@ljharb Well, it worked just fine last week, so I am assuming my dependencies should be as normal.
I have Enzyme 2.4.1, react-addons-test-utils 15.3.2 (and React and react-dom 15.3.2 as well), so I don't think I'm encountering the 15.4 issue.

I've found the solution. We're using mock-css-modules, which requires mocha to run with --harmony-proxies parameters. Once added, tests run through fine.

Ideally, I'd like to have this re-opened, please. I am not using mock-css-modules.

Did you try passing the parameter to your test runner?

Thanks for re-opening, @doque .
My error varies slightly in that it doesn't complain about proxies or adding the harmony flag. So I think it's caused by something else.
And since it all worked fine until the TestUtils got upgraded to 15.4, it's an issue that should be fixed either by Enzyme, which doesn't handle that update properly, or by the Test Utils project if the error originates there.

@batjko if you check package.json, npm ls, and the contents of node_modules, what versions are react, react-dom, and react-addons-test-utils? Do you have more than one copy of any of them?

Yes, I did check npm ls and, as mentioned above, I found that react and react-dom were on 15.3.2 while react-addons-test-utils was suddenly on 15.4.
For now I had to revert the latter back to 15.3.2. Should I instead look at upgrading react and react-dom to 15.4?

Ah, sorry, I'd missed your edit. Yes, all three need to be on the same version at all times. If you want to upgrade one, upgrade the other two as well.

Just as a side note - Using the latest version of node, I did not need to add the --harmony-proxies flag anymore.

@ljharb Thanks for confirming.
I am surprised that only the test utils were updated then, as in our package.json all three are at saved as @^15.3.2, so I would have thought if Facebook updated the whole lot, they would all upgrade when you do an npm install.
Either way, I'm good. Happy to close this one off.

I'm getting this same error, tried to solve it in different ways; seems to have started happening randomly, tried to revert back to previous versions from weeks ago and the error is triggered. Nuked the node_modules and installed (when reverted back). Not sure what I'm missing.
the actual error in the console.log:
`eact-dom is an implicit dependency in order to support<EMAIL_ADDRESS>Please add the appropriate version to your devDependencies. See https://github.com/airbnb/enzyme#installation
events.js:160
throw er; // Unhandled 'error' event
^
Invariant Violation: EventPluginRegistry: Cannot inject event plugin ordering more than once. You are likely trying to load more than one copy of React.
at invariant (/Users/Helder/www/foobar-kiosk/node_modules/fbjs/lib/invariant.js:44:15)
at Object.injectEventPluginOrder (/Users/Helder/www/foobar-kiosk/node_modules/react-dom/lib/EventPluginRegistry.js:151:66)
at Object.EventPluginHub.injection.injectEventPluginOrder (/Users/Helder/www/foobar-kiosk/node_modules/react-dom/lib/ReactTestUtils.js:361:29)
at Object.inject (/Users/Helder/www/foobar-kiosk/node_modules/react-dom/lib/ReactDefaultInjection.js:49:33)
at Object. (/Users/Helder/www/foobar-kiosk/node_modules/react-dom/lib/ReactDOM.js:27:23)
at Module._compile (module.js:571:32)
at Module._extensions..js (module.js:580:10)
at Object.require.extensions.(anonymous function) [as .js] (/Users/Helder/www/foobar-kiosk/node_modules/babel-register/lib/node.js:152:7)
at Module.load (module.js:488:32)
at tryModuleLoad (module.js:447:12)
at Function.Module._load (module.js:439:3)
at Function.Module._load (/Users/Helder/www/foobar-kiosk/node_modules/mock-require/index.js:23:22)
at Module.require (module.js:498:17)
at require (internal/module.js:20:19)
at Object. (/Users/Helder/www/foobar-kiosk/node_modules/react-dom/index.js:3:18)
at Module._compile (module.js:571:32)`
the package.json:
{   "name": "zxxzzxzx",   "version": "1.0.0",   "description": "zxzxzzxzxzx",   "main": "gulpfile.js",   "scripts": {     "test": "echo \"Error: no test specified\" && exit 1"   },   "repository": {     "type": "git",     "url": ""   },   "author": "Helder O.",   "license": "ISC",   "bugs": {     "url": ""   },   "homepage": "",   "devDependencies": {     "babel-core": "^6.13.2",     "babel-eslint": "^6.1.2",     "babel-loader": "^6.2.5",     "babel-polyfill": "^6.23.0",     "babel-preset-es2015": "^6.13.2",     "babel-preset-react": "^6.11.1",     "chai": "^3.5.0",     "cheerio": "^0.22.0",     "compression-webpack-plugin": "^0.3.2",     "css-loader": "^0.23.1",     "enzyme": "^2.4.1",     "extract-text-webpack-plugin": "^1.0.1",     "file-loader": "^0.9.0",     "gsap": "^1.19.1",     "gulp": "^3.9.1",     "gulp-mocha": "^3.0.1",     "gulp-plumber": "^1.1.0",     "gulp-s3": "^0.3.0",     "gulp-util": "^3.0.7",     "html-webpack-plugin": "^2.22.0",     "jsdom": "^9.5.0",     "lodash": "^4.15.0",     "mocha": "^3.0.2",     "mocha-jsdom": "^1.1.0",     "mock-require": "^2.0.1",     "node-sass": "^3.8.0",     "nodemon": "^1.11.0",     "open": "0.0.5",     "react": "^15.3.1",     "react-addons-test-utils": "^15.3.1",     "react-dom": "^15.3.1",     "react-hot-loader": "^1.3.0",     "react-redux": "^4.4.5",     "react-router": "^2.7.0",     "react-router-redux": "^4.0.5",     "redux": "^3.5.2",     "redux-mock-store": "^1.2.0",     "redux-promise": "^0.5.3",     "redux-thunk": "^2.1.0",     "s3": "^4.4.0",     "sass-loader": "^4.0.0",     "sockjs-client": "^1.1.1",     "style-loader": "^0.13.1",     "testdom": "^2.0.0",     "webpack": "^1.13.2",     "webpack-dev-server": "^1.15.0",     "webstomp-client": "^1.0.3",     "yargs": "^6.6.0",     "zombie": "^4.2.1"   },   "dependencies": {     "express": "^4.14.0"   } }
and the test spec example below:
`require('testdom')('')
import React from 'react'
import { expect } from 'chai'
import { shallow, mount, render } from 'enzyme'
import configureStore from 'redux-mock-store'
import InitialScreen from '../../src/js/containers/initialScreen'
import Keyboard from '../../src/js/containers/Keyboard'
import mock from 'mock-require'
// mock gsap
mock('gsap', {
TimelineLite: function () {
return null
},
TweenLite: {
fromTo: function () {
return null
}
}
})
// provide the store (mock store)
const mockStore = configureStore()
const getState = {
reservation: {},
user: {
user_typed_code: ''
},
whoami: {
carrierBagPrice: '',
carrierBagSku: '',
kioskCollectionPoint: '',
storeName: '',
storeNumber: ''
}
} // replace this with necessary state
const store = mockStore(getState)
const props = {
store: store
}
describe('The reservation number input box', function () {
it('should exist', function () {
const wrapper = mount(<InitialScreen {...props} />)
expect(wrapper.find('[name="collect-reservation-code"]')).to.have.length(1)
})
it('should start empty', function () {
const wrapper = mount(<InitialScreen {...props} />)
expect(wrapper.find('input[name="collect-reservation-code"]').props().value).to.equal('')
})
})
describe('The Keyboard container', function () {
const totalKeys = 37
it('should have a total of ' + totalKeys + ' keys', function () {
const wrapper = mount()
expect(wrapper.find('.row span')).to.have.length(totalKeys)
})
})`

@heldrida using "mock require" functionality often breaks things in unanticipated ways. Can you reproduce the error without using that?


Delete Unwanted Automate Sections

:nut_and_bolt: Description: What code changed, and why?

:chains: Related Resources
https://chefio.atlassian.net/browse/CHEF-2999
:+1: Definition of Done
:athletic_shoe: How to Build and Test the Change
:white_check_mark: Checklist
All PRs must tick these:

[ ] I have read the CONTRIBUTING document.
[ ] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[ ] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[ ] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[ ] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[ ] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[ ] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[ ] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable

Closing this PR as there are lot of conflicts with the main.
The updated PR is: https://github.com/chef/automate/pull/8607


switch: support migration when cluster is scrubbing

Similar to c13a3c3 we must allow scrubbing when running this playbook.
In cluster with a large number of PGs, it can be expected some of them
scrubbing, it's a normal operation.
Preventing from scrubbing operation force to set noscrub flag.
This commit allows to switch from non containerized to containerized
environment even while PGs are scrubbing.
Closes: #3182
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>

@leseb we can backport 1bc4d94b51182db28848e9c2a0adc8d6989ff0db without 05751b525c10380c6f68f5a036fa0b3ba5f6177a

jenkins test this please


init_by_lua error: /usr/local/openresty/lualib/ngx/semaphore.lua:126: no request found

Is there an existing issue for this?

[X] I have searched the existing issues

Kong version ($ kong version)
3.0.1
Current Behavior
I was using kong 2.8.3 CE and upgrading to 3.0.1 recently and try to use the cool function vault.
Then I set PostgresSQL connection information via vault like {vault://env/DATABASE/password}, everything works well.
However, after deploy it to AWS ECS, it crashed and I got below errors
After digging it out, I found it relates to resty-core party (see https://github.com/openresty/lua-resty-core/blob/master/lib/ngx/semaphore.lua#L124-L127) and I have no idea how to resolve it.
2022/11/29 06:54:53 [error] 221#0: init_by_lua error: /usr/local/openresty/lualib/ngx/semaphore.lua:126: no request found
stack traceback:
[C]: in function 'error'
/usr/local/openresty/lualib/ngx/semaphore.lua:126: in function 'wait'
/usr/local/share/lua/5.1/kong/pdk/vault.lua:511: in function 'connect'
.../share/lua/5.1/kong/db/strategies/postgres/connector.lua:521: in function 'query'
.../share/lua/5.1/kong/db/strategies/postgres/connector.lua:289: in function 'init'
/usr/local/share/lua/5.1/kong/db/init.lua:141: in function 'init_connector'
/usr/local/share/lua/5.1/kong/init.lua:546: in function 'init'
init_by_lua:3: in main chunk
nginx: [error] init_by_lua error: /usr/local/openresty/lualib/ngx/semaphore.lua:126: no request found

Expected Behavior
No response
Steps To Reproduce
1. Use kong version 3.0.1 (image :https://hub.docker.com/layers/library/kong/3.0.1-ubuntu/images/sha256-70b9313e4f8c89b35b12d3a964a883840d75a19a46255a26ca1f8bac57c65ebf?context=explore)
2. Set `KONG_PG_HOST`, `KONG_PG_USER`, `KONG_PG_PASSWORD`, `KONG_PG_DATABASE` to vault expression, like `{vault://env/DATABASE/password}`
3. Test it on local, it works
4. Deploy it to AWS ECS (container solution), it crashes.

Anything else?
I temporary commented out the RETRY_SEMAPHORE related codes in https://github.com/Kong/kong/blob/3.0.1/kong/pdk/vault.lua#L511-L514, and it works.

Not sure if it is related to https://github.com/Kong/kong/issues/1349#issuecomment-229803113

More findings:
Regarding to the lua-resty-core document: (https://github.com/openresty/lua-resty-core/blob/master/lib/ngx/semaphore.md#wait), the wait API does not work in init_by_lua  context.
However, the initialization of PostgreSQL connector is in init context ( I debug it by printing ngx.get_phase() )
I guess that's why this issue happens. One weird thing,  I can't reproduce this issue in local docker environment but it DID happens on the cloud environment with same docker image.

Yes, we can skip using semaphores in init phase.

PR #9851 has fixed it.


Modify System settings

When initializing the latern object using the below code
lantern = new Lantern(this)
.checkAndRequestSystemPermission()
.observeLifecycle(this);
its takes me to the settings page when it says "Can modify system settings". I dont know why we need this and how can this be avoided ?

Because when you add .checkAndRequestSystemPermission()  then it basically sends an intent message to open the system settings.
Check the source code here: https://github.com/nisrulz/lantern/blob/master/lantern/src/main/java/github/nisrulz/lantern/DisplayLightControllerImpl.java#L104
If you are not going to be using the Display as a flashlight i.e., the screen going full bright, then you can remove it from the init call.  Without this method being called this is how your init code will look like:
lantern = new Lantern(this)
.observeLifecycle(this);

Assuming the question was answered, closing this issue.


[api-docs] Add some periods to Group DM doc

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/a4f02380d29d5a436a3faade499ed54d7d7fed46

ncn


Minor UI fixes

Display a message when there are no new statements/applied/passed tests. Depends on change in ui_util templates.

This PR now includes several minor improvements to the UI, namely:

Ensuring model type names are formatted everywhere
Adding links to Elsevier and evidence text for WM model
Showing a message for empty results
Sorting the results in the table
Fixing some minor bugs

👍


There are some mistakes in GANomaly paper

In paper 3.2 Model Training, the Encoder Loss  part :
1)In last sentence which is write "'Lcon is formally defined as Lenc=...."  ,It's a mistake i think,the 'Lcon' should be corrected to  'Lenc'.
2) In second sentence "encoder loss Lcon to minimize the..."  should  corrected  'Lcon' to Lenc  too.

Thank you for spotting this. I'll be updating this in the final version of the paper.


Label and organize Grafana dashboards

We currently deploy all dashboards at the 'root'. Adding labels, and putting them in folders, would be good.

Suggested grouping:

The folder creation and management is only available through the http api, we can't manipulate them from the json we are importing.

Thanks for reporting back, @alxf. Let's stick with adding labels to the various dashboards for now, then. I'd prefer to stay away from the complexity of interacting with APIs of deployed services as much as possible. If something can't be configured statically, we may want to avoid doing it at all :)


Terminal suggest: ".." should be offered as completion when navigating multiple directories

This should present ../../ as a completion:

This seems to have regressed

The terminal completion ../../ is now offered in the suggest widget but not in the first position (I had to scroll down the suggest to see it). Would it make sense to position it higher up in the list?

Exception:
OperationStopped: D:\microsoft\vscode\out\vs\workbench\contrib\terminal\browser\media\shellIntegration.ps1:392:43
Line |
 392 |  … Prefix -match "(\.\.$([System.IO.Path]::DirectorySeparatorChar))+") {
     |                          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Invalid pattern '(\.\.\)+' at offset 8. Not enough )'s.

Not seeing it on the latest Insiders on Windows

@rzhao271 did you restart the terminal? This is part of the terminal session:

I've verified this personally as seen above. @rzhao271 let's create a new issue if you still see problems


[bug] Starting a container with exit value 11 should not return 0.

Ⅰ. Issue Description
Starting a container with exit value 11 should not return 0.
Ⅱ. Describe what happened
Starting a container with exit value 11 returns 0.
In docker, the operation returns 11.
Ⅲ. Describe what you expected to happen
Starting a container with exit value 11 returns 11.
Ⅳ. How to reproduce it (as minimally and precisely as possible)
#pouch create reg.docker.alibaba-inc.com/base/busybox:latest sh -c "exit 11"
container ID: d496b381e6937c26b7333029a02ad213f316b231749d03431d45e304b7e41ae8, name: d496b3

<EMAIL_ADDRESS>/root/pouchtest/testcases/api_cli]
#pouch start -a d496b3

<EMAIL_ADDRESS>/root/pouchtest/testcases/api_cli]
#echo $?
0

<EMAIL_ADDRESS>/root/pouchtest/testcases/api_cli]
#pouch ps
Name   ID   Status   Created   Image   Runtime

Ⅴ. Anything else we need to know?
NA
Ⅵ. Environment:

pouch version (use pouch version):
#pouch version
Os:              linux
Version:         0.3.0
APIVersion:      1.24
Arch:            amd64
BuildTime:       Fri, 30 Mar 2018 11:50:18 +0800
GitCommit:       69e66f0e6a2dfc6e8c3f56b6f97325523b746da7
GoVersion:       go1.9.1
KernelVersion:   3.10.0-327.ali2015.alios7.x86_64

OS (e.g. from /etc/os-release):
#cat /etc/os-release
NAME="Alibaba Group Enterprise Linux Server"
VERSION="7.2 (Paladin)"
ID="alios"
ID_LIKE="fedora"
VERSION_ID="7.2"
PRETTY_NAME="Alibaba Group Enterprise Linux Server 7.2 (Paladin)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:alibaba:enterprise_linux:7.2:GA:server"
HOME_URL="https://www.aliyun.com/"
BUG_REPORT_URL="http://bbs.aliyun.com/"

ALIBABA_BUGZILLA_PRODUCT="Alibaba Group Enterprise Linux 7"
ALIBABA_BUGZILLA_PRODUCT_VERSION=7.2
ALIBABA_SUPPORT_PRODUCT="Alibaba Group Enterprise Linux"
ALIBABA_SUPPORT_PRODUCT_VERSION=7.2

Kernel (e.g. uname -a):
#uname -a
Linux e100069194102.zmf 3.10.0-327.ali2015.alios7.x86_64 #1 SMP Sun Feb 11 16:23:43 CST 2018 x86_64 x86_64 x86_64 GNU/Linux

Install tools:

Others:

pouch exec also has such problem, I will fix in another pr.


Full Citation wrong content

Hello, I've got this kind of content for the "Full Citation" field in notion instead of the expected value. Note that I have a few plugins installed in Zotero that may interfere: Zutilo, Zotfile, MdNotes, Scite.
"Key","Item Type","Publication Year","Author","Title","Publication Title","ISBN","ISSN","DOI","Url","Abstract Note","Date","Date Added","Date Modified","Access Date","Pages","Num Pages","Issue","Volume","Number Of Volumes","Journal Abbreviation","Short Title","Series","Series Number","Series Text","Series Title","Publisher","Place","Language","Rights","Type","Archive","Archive Location","Library Catalog","Call Number","Extra","Notes","File Attachments","Link Attachments","Manual Tags","Automatic Tags","Editor","Series Editor","Translator","Contributor","Attorney Agent","Book Author","Cast Member","Commenter","Composer","Cosponsor","Counsel","Interviewer","Producer","Recipient","Reviewed Author","Scriptwriter","Words By","Guest","Number","Edition","Running Time","Scale","Medium","Artwork Size","Filing Date","Application Number","Assignee","Issuing Authority","Country","Meeting Name","Conference Name","Court","References","Reporter","Legal Status","Priority Numbers","Programming Language","Version","System","Code","Code Number","Section","Session","Committee","History","Legislative Body"
"R6VEK5U9","journalArticle","2015","Steffen, Will; Richardson, Katherine; Rockström, Johan; Cornell, Sarah E.; Fetzer, Ingo; Bennett, Elena M.; Biggs, Reinette; Carpenter, Stephen R.; Vries, Wim de; Wit, Cynthia A. de; Folke, Carl; Gerten, Dieter; Heinke, Jens; Mace, Georgina M.; Persson, Linn M.; Ramanathan, Veerabhadran; Reyers, Belinda; Sörlin, Sverker","Planetary boundaries: Guiding human development on a changing planet","Science","","0036-8075, 1095-9203","10.1126/science.1259855","https://doi.org/10.1126/science.1259855","The planetary boundaries framework defines a safe operating space for humanity based on the intrinsic biophysical processes that regulate the stability of the Earth system. Here, we revise and update the planetary boundary framework, with a focus on the underpinning biophysical science, based on targeted input from expert research communities and on more gener

Hi @cmjn14, thanks for reaching out!
The Full Citation field is generated based on the setting you have selected in Zotero's preferences under Export > Quick Copy > Item Format. Based on the snippet you shared, it looks like you may have CSV selected. If you change the setting and then sync the item again, you should see a different format saved into Notion.
Let me know if this helps!

Yes you were right, it works now with the right settings!
Thank you very much for the quick reply :-)


[AutoRelease] t2-redis-2024-12-16-68682(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/5787
Recording test fail, detailed info is in pipeline log(search keyword FAILED)!!!
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1022

issue link:https://github.com/Azure/sdk-release-request/issues/5787


Adding arialive to flex and div container

This PR fixes #19581
The UI should alert the description text at the bottom. For that, we need to add aria-live attribute to the description container. This PR adds that attribute to flex/div modelview container.

There was an issue with how the withProps was defined. Due to it being in the base class, we could not use withProps with withLayout method.

Example of usage:
	return view.modelBuilder.flexContainer().withProps({
			ariaLive: 'off'
		}).withLayout({
			flexFlow: 'row',
			justifyContent: 'space-between'
		});

aria-live is for dynamic text content, in your case, it seems to be static text, right?

aria-live is for dynamic text content, in your case, it seems to be static text, right?

@alanrenmsft , the text changes when you click a link on the feature name (1st) column in the table.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 4 (0.0%)  changed or added relevant lines in 2 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.008%) to 42.108%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/api/common/extHostModelView.ts
0
2
0.0%

src/sql/workbench/browser/modelComponents/componentBase.ts
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.13%

Totals

Change from base Build<PHONE_NUMBER>:
-0.008%

Covered Lines:
28152

Relevant Lines:
62382

💛  - Coveralls

Thank for fixing this @Charles-Gagnon .

Closing this PR as I have opened a new one #19770

@aasimkhan30 Did you mean to close this one? It's fixing a completely different issue than the one you linked, right?

Oops, I mean to close the other one


fix heisenbug with apollo-client caching

This fixes an issue with apollo-client's caching and fetching system. Our current method of extracting object IDs does not also include the type of the object in its return value.
Consequently, apollo thinks that a show object of ID 1 is the same as the entry object of id 1. The only side-effect of this I have observed is that the __typename of an embedded Entry is occasionally 'Show', which is clearly incorrect.
This fixes the issue by prefixing an object's ID with its typename.

Neat. Looks like they updated the docs since I implemented this several months ago.
That's what we get for living on the bleeding  edge.
🚀


feat: add a basic visitor for Substrait Relations

The main goal of this PR is to start building Substrait's c-sharp client. The commit adds a visitor to process the SQL logical operations represented in Substrait. The visitor's design is influenced by the Substrait-java client. As such, this commit adds classes corresponding to various Substrait relations in protobuf. The intention is to build the client iteratively, and hence this commit makes a tiny improvement.
In addition, the change also updates the Substrait submodule. It also deletes the old bindings since there is no need to track the bindings in source control (the binding will be generated at build time). Similarly, this commit adds a project structure which follows c-sharp project naming conventions. Hence a new project is added, which replaces the old core project in the solution.

Hi @westonpace. Thanks for your interest and comments.

This looks good to me. I have a few questions but nothing prohibitive. Are you intending to consume Susbstrait plans? Transform them? What utilities do you hope to eventually be able to create?

The idea of the core project borrows from Substrait-java:core, which IMO helps decoupling the specification from protobuf and makes it much easier to consume and produce Substrait objects. The first utility I am building extract data lineage from Substrait plans. This utility will take a dependency on the core project instead of protobuf objects. We have a few more ideas. The csharp client would help with reuse.

Disclaimer: I have been toying around with the idea of using Substrait as an interface between LINQ and Acero. I explored it a bit last Christmas break and am hoping to play around with it a bit more this coming break.

LINQ and Acero sounds really interesting, I would love to learn more.

I'm satisfied with this.  I'll leave it open for a day or two to see if @jcamachor has any input and then merge.  I notice that this repo doesn't have any PR checks.  A good next step, if you add some functionality with a unit test, might be to setup a CI job to run the unit tests as a PR check.

Thanks for your contribution @ashvina and @westonpace for the timely review!
I left a minor comment in the PR, other than that LGTM.

I notice that this repo doesn't have any PR checks. A good next step, if you add some functionality with a unit test, might be to setup a CI job to run the unit tests as a PR check.
@ashvina , could we create an issue to keep track of this task? It would be great to have asap and other contributors might pick it up. Thanks

Thanks for your contribution @ashvina and @westonpace for the timely review!
I left a minor comment in the PR, other than that LGTM.

I notice that this repo doesn't have any PR checks. A good next step, if you add some functionality with a unit test, might be to setup a CI job to run the unit tests as a PR check.

@ashvina , could we create an issue to keep track of this task? It would be great to have asap and other contributors might pick it up. Thanks

Thanks for the suggestion, #3 tracks this task.


WIP: Refactor type map

Codecov Report

Merging #27 (e2c76a2) into master (87d5b4e) will increase coverage by 0.19%.
The diff coverage is 88.46%.

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
+ Coverage   51.06%   51.25%   +0.19%
==========================================
  Files          15       15
  Lines        1036     1036
  Branches      191      190       -1
==========================================
+ Hits          529      531       +2
  Misses        493      493
+ Partials       14       12       -2

Impacted Files
Coverage Δ

cimsparql/model.py
26.58% <0.00%> (-0.12%)
:arrow_down:

cimsparql/type_mapper.py
97.77% <97.87%> (+3.27%)
:arrow_up:

Continue to review full report at Codecov.

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


Disable devise mailer

How to disable mailer when reset password. I want to send password reset intructions by other mail service.

Please use the mailing list or StackOverflow for questions/help, where a wider community will be able to help you. We reserve the issues tracker for issues only.


Removed 3rd party dep on thrift

Merging without waiting for tests in order to fix said tests.

No significant performance differences

@vjpai, that would be fine with me, and would simplify things a bit for us. Will the repo be grpc-ecosystem/grift?

We should probably do it like that. I will look into setting up the repo next week. I'll create a new issue here for now.

I suspect the following two test failures are due to this change:
https://grpc-testing.appspot.com/job/gRPC_pull_requests_linux/5076/testReport/junit/(root)/sanity_linux_dbg_native/tools_run_tests_sanity_check_sources_and_headers_py/

https://grpc-testing.appspot.com/job/gRPC_pull_requests_linux/5076/testReport/junit/(root)/sanity_linux_dbg_native/tools_run_tests_sanity_check_submodules_sh/

Could you confirm/disconfirm?