Status table

Will there be some sort of status table where we that provide test data can get some feedback?
I think that that is pretty important in order to get as many as possible to supply data.

What do you mean with "status table"? We create a table like that:
https://github.com/eu-digital-green-certificates/dgc-testdata/blob/main/BasicDgcTestsList.md
We can add there more columns. Is it that what you have in mind?

No, not really. As a provider of test data I would like to get feedback from those that have tested against the files. As it is now BG and SE has commented on eachother's pull requests and given the status. SE can parse and validate BG etc.

I have a test program that will iterate through all json files and run out implementation of the certificate validation code.
There are several issues still to be ironed out, however it might be useful for others to take a look at: https://github.com/jeroentrappers/dart_dgc_validate

@jeroentrappers Yes, that may be useful, but it is not a replacement for a matrix where each and every country apply their  validation code on the different test files. As a provider of a test file I typically want to get confirmation that other countries can parse it.

@martin-lindstrom agreed. We are not sure we will finally be able to use the dart implementation, because currently there are some issues with dependencies we use upstream. Otherwise we will fall back to the iOS and Android implementations available. However, these too currently have their issues with the test set in this respository.


add: null-handling to stacktrace callback

When using with vscode-unity-debug request came null due to server returns empty stacktrace for other threads and this leads to nvim-dap-ui to pollute the UI with bunch of error messages. this PR fixes the repeated error messages which makes UI nearly unusable by adding null check in the stacktrace callback

Thnaks for the PR! However to me this seems like an nvim-dap issue. I'm not sure of what the exact cause for the request being nil is...

due to server returns empty stacktrace for other threads

Why would the request be nil here? It shouldn't be affected by the response data as far as I know

except main thread all other thread's stack trace is returning empty/null from vscode-unity-debug, same when using VSCode, no stack trace except main, when request cames to nvim-dap-ui in self._frames[request.threadId] = response.stackFrames we got errors there due to null request, I think this problem is an adapter issue rather nvim-dap's issue since we have same case on VSCode

I understand that the stackFrames response is empty but the request shouldn't be. nvim-dap is providing the request in the listener, not the adapter, and if it is nil then that's an nvim-dap issue.

Oh got it now, I'm gonna create new issue nvim-dap since this issue doesn't belongs here we can safely close this PR

It turned out to be a bug with vscode-unity-debug. since there is no alternative adapter for unity engine I propose simply adding nil check and make nvim-dap-ui usable in unity development

I'll follow up in the nvim-dap issue, but I'm going to close this PR. Whether the solution is put into nvim-dap or nvim-dap-ui, there'll need to be a more generic approach to handling possibly null requests.

With latest nvim-dap this should be solved :smile:


Rendering issues on Android GLES2 with MSAA (depth buffer issue?)

Godot version:
3.2 master (8570b9b0), including with manual cherrypick of #33181
OS/device including version:
Android 9, Xiaomi Pocophone F1, Adreno 630
http://opengles.gpuinfo.org/displayreport.php?id=3448
Issue description:
Follow up to #33172.
When enabling MSAA 2x or more on a simple 3D scene, I get rendering issues seemingly related to the depth buffer.
With the test project included below, I get:

PanoramaSky, no MSAA (as expected):

PanoramaSky, MSAA 2x (broken):

Clear Color, MSAA 2x (kind of works, weird transparency effect):

In all scenarios (so also without MSAA) I get this error spammed in the logs:
ERROR: GL ERROR: Source: OpenGL	Type: Error	ID:<PHONE_NUMBER>	Severity: High	Message: location of the uniform is -1
At: drivers/gles2/rasterizer_gles2.cpp:133:_gl_debug_print() - GL ERROR: Source: OpenGL	Type: Error	ID:<PHONE_NUMBER>	Severity: High	Message: location of the uniform is -1

When MSAA is enabled, I get this additional error:
**ERROR**: GL ERROR: Source: OpenGL	Type: Error	ID:<PHONE_NUMBER>	Severity: High	Message: internalformat 6402 is not a renderbuffer format
   At: drivers/gles2/rasterizer_gles2.cpp:133:_gl_debug_print() - GL ERROR: Source: OpenGL	Type: Error	ID:<PHONE_NUMBER>	Severity: High	Message: internalformat 6402 is not a renderbuffer format

Steps to reproduce:

Set up a simple 3D scene
Enable MSAA 2x or more
Export to a device with Adreno 630 or similar

Minimal reproduction project:
Test 3D.zip

As mentioned in https://github.com/godotengine/godot/issues/33188
the issue might be that in line
https://github.com/godotengine/godot/blob/23a381d8827dbcb2d295fa393a5400f7a7e0a1c1/drivers/gles2/rasterizer_storage_gles2.cpp#L4717
no 24bit depth buffer is created; forcing a 24bit buffer via the line
glRenderbufferStorageMultisample(GL_RENDERBUFFER, msaa, _DEPTH_COMPONENT24_OES, rt->width, rt->height);

seemed to fix the issue on the Oculus Quest with GLES2 msaa


Provide flow and views for managing password request questions

Is there any interest in allowing users to manage security questions in CAS itself? We have a home-grown interface to do so bolted on to CAS 4 right now. As part of our migration to CAS 5 I need to either make the old one work or modify cas-server-support-pm and related modules to allow for editing security questions.
I'd much rather go the second route and get it built in to base CAS (fewer migration issues in the future). But, if I go this route and I can't get it in base CAS it'll end up far worse than the first option since I'd be editing/overlaying actual CAS files.
Let me know...

May I see some screenshots?

and also, I'd like to see how you have managed to do this with CAS 4. Seeing a concrete impl would help to evaluate; even if it's just notes at a high level.

"Bolted on" was probably the wrong term to use. The CAS 4 implementation we have is not really how it would/should/could be implemented for CAS 5 (whether into base CAS or not). To start off, the existing add-on replaces both login-webflow.xml and web.xml with heavily modified versions. The new CAS 5 stuff we're doing doesn't use xml at all. We're trying to avoid modifying/overlaying CAS code altogether. I'm not really sure how much of the existing code is even usable while maintaining a small modification footprint.
I'll see if I can get approval to share screenshots.
If question management isn't a feature you guys are interested in including in cas-server-support-pm I'll probably take the guts of cas-server-support-consent or cas-server-support-aup and create a cas-server-support-pm-editquestions (or a better named version). Maybe that's the right approach to take regardless?

These are the two portions that handle the question stuff. There's an 'Update my settings' link on the login page that will trigger the flow interruption.
The second shot is where the actual changes are made.

Thanks for the screenshot. It goes without saying that we (I) are interested. I'd recommend that you start with a WIP pull request and continue. Remember the cutoff for RC4 and note that it's rare for changes to go in post RC4. Time yourself accordingly, or worse we shall push this into the 5.3 line. Also remember that this behavior you are adding must not be tied to a specific store implementation for pm. Regardless of store, all should be supported (which might increase your initial scope).
If you need help with anything, please post back.

Maybe that's the right approach to take regardless?

Yes, that what you will need to be doing anyways. I'd welcome a better-named module for sure.

Going to close this for now. I look forward to reviewing the PR and working with you to finalize this.


How do I use ShouldHaveChanges() method?

I've tried to find an example in the docs as how to use the ShouldHaveChanges() method where I am checking that the snapshot has 2 or more new pieces of text in the markup.
Please could you provide an example?

In FetchDataTest.cs#L51 there is an example that uses ShouldHaveChanges on a list of diffs, which GetChangesSinceSnapshot() also returns. In the example, it is retrieved from GetChangesSinceFirstRender() though.
That is the only example I have right now. I do recommend using the debugger to inspect the list of diffs if you want to learn more. Otherwise feel free to post a follow up here with some example code so I can better help out.
If you get it working and can share your test code or an example, please do, so others might benefit.

Unfortunately I am not able to successfully get this to work. I have the following test where I am looking for the addition of "Test Title" to the markup:
      cut.SaveSnapshot();

         Renderer.Dispatcher.InvokeAsync(() => cut.Instance.ShowMessageBox("Test Title", "Test message"));

         cut.GetChangesSinceSnapshot().ShouldHaveChanges(
             diff => diff.ShouldBeAddition("Test Title")
         );

But the test fails, presumably because the Test Title is surrounded by DOM elements etc?
There is some whitespace and line breaks surrounding the text so I also tried:
           cut.GetChangesSinceSnapshot().ShouldHaveChanges(
                diff => diff.ShouldBeAddition("/n        Test Title    /n")
            );

But unfortunately this had the same result.

Try ShouldBeTextChange instead (that only looks at changes to text). Btw. Insignificant whitespace is ignored by default.
If you only expect one change, the ShouldHaveSingleTextChange can be used instead of ShouldHaveChanges.
More here: https://github.com/egil/bunit/blob/master/src/Asserting/ShouldBeTextChangeAssertExtensions.cs

OK I have made some progress (thank you) with the current snapshot check as follows:

            cut.SaveSnapshot();

            Renderer.Dispatcher.InvokeAsync(() => cut.Instance.ShowMessageBox("Test Title", "Test message"));

            cut.GetChangesSinceSnapshot().ShouldHaveChanges(
                diff => diff.ShouldBeAttributeChange("class", "modal fade show"),
                diff => diff.ShouldBeAttributeChange("style", "display: block; padding-right: 17px;;"),
                diff => diff.ShouldBeAttributeChange("class", "modal-backdrop fade show"),

                diff => diff.ShouldBeAddition("Test Title"),
                diff => diff.ShouldBeAddition("Test message")
            );

But in order to do this I had to hookup the bunit project and debug the tests to find out all the exceptions. The first error I hit was that the ShouldHaveChanges requires the same number of changes as there are actual changes (I was hitting an Assert.Collection() Failure exception). So, I had to add the ShouldBeAttributeChange checks as well into the expected change list. The second issue I had was that in order to find out what these attribute expected changes were I had to hit the exceptions to get the actual change and add this in.
Is there any way to run through many ShouldHaveChanges but not have to provide ALL changes that have occurred? i.e. the expected and actual collection counts are not equal.

Admittedly, the idiff assert API is not fully baked at this point. So you are probably running up against some limits here.
A few options:

What I normally do is to create my own assertion helpers, e.g. create your own should have changes.

Place the result of snapshot comparison into var, e.g. var diffs = cut.GetChangesSinceSnapshot(), and them index into the array and only assert against the individual diffs you are interested in, e.g. diffs[2]. ShouldBeTextChange....

(Probably preferred in this case) Use cut.Find("css selector to find msg box"). MarkupMatches("expected html here"). This allows you to drop the snapshot stuff, and simply provide all the expected  html. Since you have so many changes, it is likely this will provide you with the cleanest test.

Here is some docs on using MarkupMatches and the many options you have to customize the comparison: https://bunit.egilhansen.com/docs/semantic-html-markup-comparison.html

I went for option 2:
        var diffs = cut.GetChangesSinceSnapshot();

        diffs[3].ShouldBeAddition("Test Title");
        diffs[4].ShouldBeAddition("Test message");

But what would be good is to filter on diffs, for example:
            IEnumerable<IDiff> diffs = cut.GetChangesSinceSnapshot().Where(x => x.Result == AngleSharp.Diffing.Core.DiffResult.Unexpected);

            diffs.ShouldHaveChanges(
                          diff => diff.ShouldBeAddition("Test Title"),
                          diff => diff.ShouldBeAddition("Test message")
                      );

But I cannot do this at present as the extension method is for IReadOnlyList, not IEnumerable

Happy you find a solution that works for you.
IReadOnlyList implements IEnumerable, so you can definitely use Linq. You are probably missing a using statement.

hmm, not sure that I am missing a using statement, I can see that IReadOnlyList does implement IEnumerable, but I don't think that extension methods can work with other types.... (?)
'IEnumerable' does not contain a definition for 'ShouldHaveChanges' and the best extension method overload 'DiffAssertExtensions.ShouldHaveChanges(IReadOnlyList, params Action[])' requires a receiver of type 'IReadOnlyList'

C# inheritance definitely allow you to use Linq on derived types of IEnumerable<T>. See this example if you are unsure: https://dotnetfiddle.net/40PuCc

The issue is that your extension method ShouldHaveChanges() works only with IReadOnlyList and not IEnumerable. I don't think Inheritance works on extension methods like it does with regular methods.
See here:
https://dotnetfiddle.net/qJlHSd
I've created a helper method to iterate and recast to IReadOnlyList so I can use bUnit after filtering. Thanks again for your help with this.

Ahh, I get what you are saying now. The workaround for now is to add a e.g. .AsArray(): https://dotnetfiddle.net/cJxTvq
I will make the extension methods work on IEnumerable in the future as well.


device-list.component.html does not compile

npm run build fails and can be easily fixed by updating the src/app/device-list/device-list.component.html file
Replace

{{device.icon}}

with

{{device.icon}}

Why is it not compiling? That syntax was working when I ran a build.

ArthursMacBookPro:ngapp arthurvernon$ npm run build

<EMAIL_ADDRESS>build /Users/arthurvernon/https/smart-home-dashboard/src/main/ngapp
ng build --prod && cp -r ../appengine/* ../webapp/

Date: 2019-09-18T04:31:04.057Z
Hash: f79b39e48d050f471bdd
Time: 4474ms
chunk {0} styles.8a126321729b8c0aa334.bundle.css (styles) 46.9 kB [initial] [rendered]
chunk {1} polyfills.997d8cc03812de50ae67.bundle.js (polyfills) 84 bytes [initial] [rendered]
chunk {2} main.ee32620ecd1edff94184.bundle.js (main) 84 bytes [initial] [rendered]
chunk {3} inline.318b50c57b4eba3d437b.bundle.js (inline) 796 bytes [entry] [rendered]
ERROR in : Can't bind to 'ng-show' since it isn't a known property of 'img'. ("gStyle]="{'background-color':device.updated ? 'green' : '#424242' }">
    ]ng-show="{{device.giphyUrl}}" src="{{device.giphyUrl}}" alt="{{device.name}}">

]ng-show="{{!device.giphyUrl}}">{{device.icon}}
{{device.")
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: ng build --prod && cp -r ../appengine/* ../webapp/
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      /Users/arthurvernon/.npm/_logs/2019-09-18T04_31_04_091Z-debug.log

If my understanding is correct, ng-show is angular 1.

Okay I'll get a bug fix started.

Seems like it works without the curly brackets, ie. ng-show="device.giphyUrl". Is there an advantage to using hidden?

{{device.icon}}
{{device.name}} ({{device.nicknames.join(',')}}) ({{device.id}})

hidden is standard html
https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/hidden


Support PCL 4.5 - Profile111

I'm having problems when trying to install the package in a Xamarin.Forms project.
Here is the error:
Checking compatibility for Scandit.BarcodePicker.Unified 5.0.1 with .NETPortable,Version=v4.5,Profile=Profile111.
Package Scandit.BarcodePicker.Unified 5.0.1 is not compatible with portable45-net45+win8+wpa81 (.NETPortable,Version=v4.5,Profile=Profile111). Package Scandit.BarcodePicker.Unified 5.0.1 supports:

monoandroid10 (MonoAndroid,Version=v1.0)
net (.NETFramework,Version=v0.0)
uap10 (UAP,Version=v1.0)
win81 (Windows,Version=v8.1)
wpa81 (WindowsPhoneApp,Version=v8.1)
xamarinios (Xamarin.iOS,Version=v0.0)

Thanks,
Homero

Hi,
I'm not sure I understand what's going on here. It's possible that the Windows 8 part is the problem. Could you try if removing support for Windows 8 (not 8.1) solves the issue for you?
Best,
Marco

Have you been able to resolve the issue on your side? I'm closing the ticket, since there hasn't been any update on this in a while. If the problem persists, feel free to reopen it.

I apologize for the delay, but I could not solve the problem.
I'm redoing the application and using the Scandit.BarcodePicker.Unified library. However I have the following error when trying to deploy to the device:
/Users/HomeroKzam/Projects/ParkCode/iOS/MTOUCH: Error MT5211: Native linking failed, undefined Objective-C class: SBSBarcodePicker. The symbol 'OBJC_CLASS$_SBSBarcodePicker' could not be found in any of the libraries or frameworks linked with your application. (MT5211) (ParkCode.iOS)
When I deploy to the simulator I have no problem compiling. I use Visual Studio for Mac.
Thanks,
Homero

Would it be possible to send us a project, so we can reproduce the issue on our side?

Follow the link for the project.
https://github.com/homerokzam/PC
Thanks,
Homero

Great, thanks. We will take a look and get back to you.

Thanks for the project. We were able to reproduce the issue on our side. It appears to be specific issue with the project that you sent. Have you tried to create the project with Xamarin Studio instead of Visual Studio for Mac (VS for Mac is still in preview).

Also, make sure to update to the latest version of Xcode and Xamarin.iOS.

I have not yet created the project using Xamarin Studio.
I'll do it now.
Thanks,
Homero

Hello!
I have a problem with using your component in Xamarin.Forms. Problems appear only for iOS project : during building I got 13 errors about native linking failed. These errors also appear when I try to build SimpleSample project for your component.
http://joxi.ru/Dr8OWG5UwLb7A6
Here is attached screenshot with errors.
Thanks, Alex

We have seen this happening when not using the latest version of Xcode. Did you update to the latest version (8.2.1)? Also make sure that Xamarin Studio and the iOS framework are completely up-to-date.

But I have the latest version of Xcode and iOS framework
Apple Developer Tools
Xcode 8.2.1 (11766.1)
Build 8C1002
Xamarin.iOS
Version: <IP_ADDRESS> (Xamarin Enterprise)
Hash: ba11e48
Branch: cycle9
Build date: 2017-03-10 08:48:04-0500
I think errors can be related to mono version. I have this version
Mono 4.8.0 (mono-4.8.0-branch/e4a3cf3) (64-bit)

Thanks for the update. We will take a look and get back to you.

These errors are caused by the incremental build feature of Xamarin Studio. Please disable it for now.
More details: https://scandit.zendesk.com/hc/en-us/articles/115003233085-I-am-getting-error-Native-linking-failed-undefined-Objective-C-class-SBSBarcodePicker
Feel free to reopen the ticket if your issue still remains.


Set rabbitmq monitoring password

Depends on support added to kolla-ansible:
https://review.openstack.org/#/c/517386/

Please could someone merge this - I don't seem to have rights yet.


Repeating results when go back to search

Hi,
The search results are repeating.
Steps:

Make a search
Navigate to anywhere on the site
Go back

Thanks @seromenho for your comment, patch have been published  here #31 :)
Should not happen anymore.


Font swapping on a per-message basis

This could be implemented as a TbyOption, e.g. TbyOption.SetFont, with a scribble declared font name.
https://github.com/glitchroy/textboxy/blob/9ab8e43be61a47224db60c88d053c1097f9b11e7/scripts/tby_init_scribble/tby_init_scribble.gml#L1-L9
The default font is already defined as a scribble declared font name, in tby_default_font, but currently, the default is used for every message box.
https://github.com/glitchroy/textboxy/blob/9ab8e43be61a47224db60c88d053c1097f9b11e7/scripts/tby_st_tb_init/tby_st_tb_init.gml#L21

Implemented in scribble v3.1.1 (a4bfc483da338c6a8951e679652f27a1a8cfa432).


refactor: assign immutable default value without dataclass field() function

Just a minor refactoring which simplifies setting an immutable default value for a dataclass attribute without calling the field() function.

Rebased to get install-nix-action v22 which should let CI / flake-check (macos-latest) pass now.


WRP-565: Refine custom event dispatching

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
Some custom events are based on native user inputs like keyboard, mouse, or touch, and it would be helpful for apps to deal with scenarios that we provide stopPropagation and preventDefault in custom events' payloads.
Resolution

Added stopPropagation and preventDefault to some custom events
Refined code forwarding events

Additional Considerations
Links
WRP-565
Comments
Enact-DCO-1.0-Signed-off-by: Seungcheon Baek<EMAIL_ADDRESS>

Code coverage dropping is due to removing of actively tested lines.

Thank you @MikyungKim! I will check listed ones. 👍


Tile caching - debug and control

I'm using the 5.5 release, with Mapnik tile source and default cache settings. I'm examining the same area (about 500m across) and zooming through the three highest magnifications.
When online, all tiles are retrieved and the map view works flawlessly. If I return to the app 12 hours later, offline, only a handful of tiles are cached at the lowest zoom, and zooming in further only uses scaled tiles.
The app appears to be retaining far fewer tiles than I expected and I'm not sure the best way to debug what is being cached, or to control it. Ultimately, I'd like a Google Maps style 'offline map' facility (and may be in a position to implement this depending on my business), but short term, I'd like to ensure that tiles at a certain zoom level are cached for a predictable period.
As I'm working within a small area, I assumed that the cache would easily cover the tiles required?

do you mean 500 miles or 500 meters?
there's some info here for debugging tile loading
https://github.com/osmdroid/osmdroid/wiki/FAQ#debugging-tile-loading-issues

500 meters. Essentially, just a small number of tiles at the three highest zoom levels and they don't appear to be cached. I'll enable the debug for the tile source to see what it's doing.

What map source are you using? The expiry code comes from a header in the mapnik response. Its possible the response is short.

interesting, can you get me a bounding box and zoom levels for the area affected?

Around 52.2742, 0.21130 three highest zoom levels for Mapnik, the default Tile source

I have seen OpenStreetMap keep low expiration times. If you browse to one of the URLs for a tile in question in chrome and open the developer console, you can see the headers. You will see one for expiration. It should be a date or long since epoch time. I bet the date is short, in the end of my company created our own mapnik servers and I configured the expiration based on the zoom levels to keep tiles almost a year out for the lower levels. Osm runs code to keep the database size down to delete rows older than the expiration date.

What's the time delta from the initial tile load and cache to when you go into offline mode? I just made a sample and test case for this. honestly, we should have had the test ages ago, but it appears to work and pass. I'll wait for travis before merging.

@cfsbhawkins i noticed that today too, it was a very fast expiration time. We may need to relook at the algorithms for purging the database of expired tiles. If a tile is expired and there's no network connection (or a newer one doesn't exist), we shouldn't purge the tile

merged with test cases. it probably needs more work to handle the "i was online but am now offline, don't purge the cache" scenario

@atoone we the map starts up and let's just say that the database is bigger than cache is supposed to be, what's the preference for trimming the cache?

delete the oldest tiles first? (tile source independent)
delete expired tiles first?

i'm not sure what's right here

@atoone was this problem present with previous versions? I'm thinking it was and it's isolated to the mapnik/osm tile source.

@spyhunter99 Yes, I suspect that it was present with previous versions, but the change to the SQLite database was a distraction that muddied the water a little. It might be handy to have some sort of log/debug/metric to indicate when tiles come in with unexpectedly short expiration times?
As I've been working within a small area for testing purposes, I don't expect my database is getting large enough to require trimming - again this is a metric it would be helpful to expose. It might help to have a 'last seen' timestamp for tiles so you could trim the least-recently used tiles rather than those that have been in regular use for a long time.
It makes a lot of sense to not expire tiles when there is no network connection. If I've recently seen a tile, I'd only expect it to go away when a newer one can immediately replace it. Most of my use cases revolve around field work, so suddenly having maps disappear when you have no network connection is distressing. Of course there is then the challenge of 'partial connection' where you may be connected to a hotspot, but still cannot get a link to the server.
Perhaps the answer here is not to use expiry time at all for purging tiles from the database, but use it purely as a flag to indicate when an attempt should be made to get a fresher tile? If one can be retrieved then it replaces the old one, but otherwise the cache is legitimately used?

found something. MapTileDownloader has a public static boolean for debug logging, that may hold the clue

i just added some more logging, looks like mapnik is returning a 60 hr window give or take for tile expiration

To address your other concerns.
The way the sqlwriter is right now, it only trims the database if needed (the database is bigger than the user/developer selected max size or the amount of free disk space).
If you're going to be offline for an extended period, consider using the cache manager to produce an archival database (similar to how mobac works). This will make sure that you get all the tiles for the given area and store them in a database which will never purge or expire.

Perhaps the answer here is not to use expiry time at all for purging tiles from the database, but use it purely as a flag to indicate when an attempt should be made to get a fresher tile? If one can be retrieved then it replaces the old one, but otherwise the cache is legitimately used?

This is the current behavior. The MapTileProviderBase attempts to load from all local sources. if the tile is expired, it will be a new one will attempt to be loaded, but it should use the expired in the event of a load failure


Some records being assigned entries in CR_SUBJECT

Expected Behavior
When ocw_oer_export.csv generates, all entries should have a value in the CR_SUBJECT field.
Current Behavior
On the most recent export, 52 rows do not have a value in CR_SUBJECT.
Steps to Reproduce

Most recent csv attached for review (ocw_oer_export (7).csv), along with screenshot of csv in OpenRefine:

Possible Solution
I had a casual look through the records to see if there was some common thread (e.g. publish time or obvious subject that isn't mapping) but nothing jumped out at me. If someone can give me an export either as a csv or even a comment attached to this issue containing the course name, URL, and OCW subject headings for the entries that haven't been mapped, I can give mappings for them.

@ibrahimjaved12 can you take a look at this?
I looked at a couple of courses (e.g. 11.255  and 15.760a) and couldn't find any commonalities.

@pdpinch It looks like there are two topics fields in the MIT Learn API. And we may not be using the right one.

First topics field (represents MIT Learn topics for OCW courses):

"topics": [
            {
                "id": 720,
                "name": "Policy and Administration",
                "icon": "RiUserSearchLine",
                "parent": 712,
                "channel_url": "https://learn.mit.edu/c/topic/policy-and-administration/"
            },
            {
                "id": 698,
                "name": "Data Science, Analytics & Computer Technology",
                "icon": "RiLineChartLine",
                "parent": null,
                "channel_url": "https://learn.mit.edu/c/topic/data-science-analytics-computer-technology/"
            },
            {
                "id": 715,
                "name": "Economics",
                "icon": "RiUserSearchLine",
                "parent": 712,
                "channel_url": "https://learn.mit.edu/c/topic/economics/"
            },
        ],

Second ocw_topics field (specific to OCW topics):

"ocw_topics": [
            "Developmental Economics",
            "Economics",
            "Energy",
            "International Development",
            "Public Administration",
            "Public Policy",
            "Science and Technology Policy",
            "Security Studies",
            "Social Science",
            "Technology"
        ],

The Issue
Currently, we are relying on the first topics field (MIT Learn topics), but I believe we should be using the second ocw_topics field (OCW-specific topics) ? I'm not sure how that got to be the case. But this is what is happening right now.
So it's not the courses that are problematic, it's that their ocw topics are being ignored. Can you please confirm this, so I can go ahead to creating a PR for this that uses ocw_topics instead.
Some additional analysis
@pmusser I also performed an analysis of the OCW topics that might be useful.
In flattened structure, we have 335 OCW topics. And we have 328 OCW topics to OER Subject in the mapping file.
These are the OCW Topics that are missing OER Subject:
African American Studies (15 published courses)
Biomaterials (19 published courses)
Discrete Mathematics (39 published courses)
Italian (1 published course)
Nonfiction Prose (27 published courses)
Periodic Literature (8 published courses)
Project Management (29 published courses)
Quechua (0 published course)
Russian (1 published course)
However, these are 2nd-level subtopics or 3rd-level specialties in the unflattened hierarchy. Which means, their OER subject would also be populated by their parent category. So in essence, they should be populated, but regardless I wanted to share this in case we would like to have more specific subjects for them.
OCW Topics Hierarchy for Missing Topics:
- Humanities:
  - Language:
    - Italian .........3rd level missing speciality
    - Quechua .........3rd level missing speciality
    - Russian .........3rd level missing speciality
  - Literature:
    - Periodic Literature .........3rd level missing speciality
    - Nonfiction Prose .........3rd level missing speciality
- Business:
  - Project Management .........2nd level missing subtopic
- Engineering:
  - Biological Engineering:
    - Biomaterials .........3rd level missing speciality
- Mathematics:
  - Discrete Mathematics .........2nd level missing subtopic
- Society:
  - African American Studies .........2nd level missing subtopic

Analysis File
For a complete list of OCW courses and their mappings, please see this analysis file:
https://docs.google.com/spreadsheets/d/102Wdw1he5qVYh458XNTHwP5p9IVd0RBTRZx3piIgpTE/edit?gid=0#gid=0
All OCW Topics Hierarchy file
ocw_topics_hierarchy.txt
In any case, this is the list of the 52 courses that do not have a CR_SUBJECT currently:
https://docs.google.com/spreadsheets/d/1UoMRJUVtLHIWechYEXWa6GZNTpRcOIQWS23wK_5_GFk/edit?gid=632809784#gid=632809784

@ibrahimjaved12 If I recall correctly, the initial mapping for this app was written by a member of the OCW team, using ocw_topics_hierarchy.txt and the "Subject Area" values from OER Commons Search.
Sometime later, the MIT Learn site established a new (simpler) topic hierarchy, which replaced the values in topics in the MIT Learn API.
The original OCW topics should be available in the MIT Learn API via "ocw_topics" above. Does that seem correct?
I'm not sure which topic hierarchy to use for OER mapping. The current plan is to keep both. Given that we already have a mapping from OCW Topics to OER Commons Subject Areas, the simpler solution is to complete that mapping. Can you identify which values have been overlooked, and I can refer them to the OCW team?

@pdpinch
I created a new test CSV using values from ocw_topics for both CR_SUBJECT and for fallback of CR_KEYWORDS. In this case, there are no missing mappings for these fields in the mapping files. The existing mappings are sufficient. Using ocw_topics would resolve both of these issues.
However, there are missing values in the CSV for these two courses:

Social Theory and Analysis (https://ocw.mit.edu/courses/21a-750j-social-theory-and-analysis-fall-2011) -- This should be removed from search index as we discussed earlier.
Teaching La Princesse de Clèves (https://ocw.mit.edu/courses/res-21g-3001-teaching-la-princesse-de-cleves-fall-2023) -- This seems a course in progress that is yet to be published with actual data: https://ocw-studio.odl.mit.edu/sites/teaching-la-princesse-de-cleves

Since both CR_SUBJECT and CR_KEYWORDS are required fields for OER Commons, I propose:

Using ocw_topics instead of topics.
Use the fallback value "MIT Open Learning" so required fields aren't empty in any case. As proposed here: .https://github.com/mitodl/ocw_oer_export/issues/122#issuecomment-2516166683

Please note the current logic:

For CR_KEYWORDS, the function:

Checks for the course's keywords in fm_ocw_keywords_mapping based on Course_URL.
If none are found, as fallback it currently uses topics (instead of ocw_topics).

For CR_SUBJECT, the function:

Relies on the mapping file for conversion (but course topics are used, instead of ocw_topics).

Please let me know how you'd like to proceed.


[Ecosystem] Block Explorers, TestNet Faucet & Documentation

Objective
Unblock Public Shannon TestNet discoverability, usage & maintainability.
Origin Document

This is a follow-up to #27
@okdas was speaking to other TestNet maintainers and share some learnings w.r.t faucets

Goals

Make Shannon TestNet usable by introducing a POKT faucet
Make Shannon TestNet Block Explorers & Faucets accessible & maintainable

Deliverables

[ ] Introduce a Shannon TestNet faucet

[ ] Evaluate Cosmos SDK tools (e.g. https://github.com/ping-pub/faucet)

[ ] Add a TestNet section to the top of dev.poktroll.com that has the following subsections:

[ ] Maintenance (internal)
[ ] Usage (external)
For both Maintenance & Usage, make sure to cover

[ ] Block Explorer
[ ] Faucet

Creator: @Olshansk
Co-Owners: @okdas @moatus

@moatus Please review this ticket and lmk if you have any questions.
@okdas Feel free to edit / add any additional details.

Faucet Repo: https://github.com/Eastman-Enterprises/Faucet

@moatus Thoughts on moving this to https://github.com/pokt-network/?

@breezytm Do you mind posting the relevant resources (repos, results, etc) here when you have a chance for posterity?


SyntaxError: Cannot use import statement outside a module

Jest not working because of the above error
I'm using typescript template

What's the error? There is nothing above. Also, please don't post a screenshot. Post the text of the error you are seeing. Screenshots aren't helpful.

I mean the error in the title of this issue
SyntaxError: Cannot use import statement outside a module
I've spent hours on this error, please help

Unless the create-mf-app created a project that immediately resulted in this error, this is not a bug on this generator.
I'm sorry you're running into issues. If you like you can join my Discord server (https://discord.gg/nGxhQ3SY) then read the #rules carefully and post a question that can be answered in an appropriate channel.

Thanks
FYI I'm using create-mf-app
was wondering if there was a setup already I could just apply to my situation.

Honestly, create-mf-app was built to produce prototype code for Module Federation setups. It wasn't meant to bootstrap production apps. Each one of these frameworks have starter kits that are designed for that purpose. I wouldn't be trying to use Jest with create-mf-app I would instead use something like Create React App (CRA) that has unit testing built into it.

ahh I see, thank you again


Added 't Saam Diksmuide

Official website: www.tsaam.be
Long-term IT related course: https://www.tsaam.be/Informaticabeheer

@JLauwyck First stage of verification passed, official domain is tsaam.be as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@JLauwyck Pull request merged. Thank you.


Messages with type 'html' are ignored in chat thread

Trying to display messages sended from java client with 'html' type (valid values for java SDK of azure communication are 'text' and 'html' https://github.com/Azure/azure-sdk-for-java/blob/af17a1edd24e440b8ea83535a5ebcd3acce9f9b8/sdk/communication/azure-communication-chat/src/main/java/com/azure/communication/chat/models/ChatMessageType.java#L17; ), and chatThread component doesn't display it (beacause of filtering of messages with types 'text' and 'richtext/html').
But from Java side we can't pass 'richtext/html' value for type field, it fails validate message:
Request processing failed; nested exception is com.azure.communication.chat.models.ChatErrorResponseException: Status code 400, "{"communicationError":{"code":"BadRequest","message":"One or more validation errors occurred.","details":[{"message":"The input was not valid.","target":"type"}]}}"
Please add support 'html' type at this point:
https://github.com/Azure/communication-ui-library/blob/67d957723e67f2a3fd5ab444f085cb89113966de/packages/chat-component-bindings/src/chatThreadSelector.ts#L48

Thanks for filing this issue. I will file a bug and work on the fix.


Hi im having a problem here and im very new to react native :( badly need help asap

Hi @hazzacookie. Check out the Support and Contact section of the README. You'll find your way there.


Incorrect indentation using Atom Beautify with PHP_CodeSniffer and WordPress-Coding-Standards

Bug Description
I’m developing a WordPress theme, as a editor I use Atom with Atom Beautify. I have also installed as default beautifier “PHP_CodeSniffer” with the “WordPress-Coding-Standards “as ruleset (WordPress-VIP,WordPress,WordPress-Extra,WordPress-Docs,WordPress-Core).
Both, the “PHP_CodeSniffer” beautifier and the “WordPress-Coding-Standards” configuration file, work correctly, but they create files that are a mess, and in which some spaces become tabs instead others remain such spaces. Furthermore, the structure of the file itself is also incorrect (at least for what I would like to get).
I don't know if the code shows the problem so I'm going to integrate the issue also with images.
Input Before Beautification
This is what the code looked like before.
As you can see in the file, spaces are used and there is an initial indent that should not exist.

Expected Output
The beautified code should have looked like this.

Actual Output (image)
The beautified code actually looked like this.
This is what happens after starting Atom Beautify. The initial indent has not been corrected, the general structure is incorrect, and there are both tabs and spaces.

Actual Output (code)
<?php
/**
 * The template for displaying 404 pages (not found)
 *
 * @link https://codex.wordpress.org/Creating_an_Error_404_Page
 *
 * @package help
 */

get_header(); ?>

Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

<?php get_footer(); ?>

Error Code

Environment

Question
Answer

PHP version
5.6.30

PHP_CodeSniffer version
3.2.3

WPCS version
0.14.1 (I took this information from the file "CHANGELOG.md")

WPCS install type
Standalone

IDE (if relevant)
Atom 1.38.2

Additional Context (optional)
Here instead you find my personal configuration file phpcs.xml.dist:
<?xml version="1.0" encoding="UTF-8"?>

<ruleset name="WordPress Custom">
  <description>Custom PHP_CodeSniffer ruleset</description>

  <arg name="tab-width" value="2"/>

  <rule ref="WordPress">
    <exclude name="Generic.WhiteSpace.DisallowSpaceIndent"/>
  </rule>

  <rule ref="Generic.WhiteSpace.ScopeIndent">
    <properties>
      <property name="indent" value="2"/>
      <property name="exact" value="true"/>
      <property name="tabIndent" value="false"/>
    </properties>
  </rule>

  <rule ref="Generic.WhiteSpace.DisallowTabIndent"/>

  <rule ref="PEAR.Functions.FunctionCallSignature">
    <properties>
      <property name="indent" value="2"/>
    </properties>
  </rule>
</ruleset>

Tested Against develop branch?

[ ] I have verified the issue still exists in the develop branch of WPCS.

Hi @vizzale Before I start looking into this I have two questions:

Is there any particular reason why you are using an old version - 0.14.1 - of WPCS ? The last release of WPCS is 2.1.1.
A lot of improvements have been made to WPCS between then and now, so your problem may already have been addressed.
Please test with the current version of WPCS to see if the problem still exists.
To understand your use-case: why do you want to use spaces, and then 2 instead of 4, for your indentation ?

Hi @jrfnl I answer you questions:

There is no particular reason, I simply installed both PHPCS and WPCS in May last year and I no longer updated WPCS.
Also every time for this type of application I always have the fear that something could go wrong, in fact I updated WPCS a few moments ago and now it doesn't work anymore neither Atom Beautify nor linter-phpcs.
I've always used an indent of two spaces for HTML, and CSS and many style guides suggest this type of indentation so it simply became part of my workflow. But regardless of whether to use tabs or spaces for indentation, I would like to be able to do it in both ways as long as it is correctly set by my configuration file.

I hope that we will be able to solve both problems.
Thank you very much.

However I add this information, it could be useful.
Yesterday I immediately proceeded to redo the standalone procedure to set the new version:

PHP_CodeSniffer is already installed;
git clone -b master https://github.com/WordPress/WordPress-Coding-Standards.git wpcs;
phpcs --config-set installed_paths / path / to / wpcs.

Unfortunately however, as I told you, Atom Beautify and the linter-phpcs stopped working. Returning instead to the 0.14.1 version, it started working again.
Looking forward to receiving your valuable advice, I thank you.

@vizzale Well, getting your setup working properly for Atom should probably be the first priority.
So, for Atom, could you please verify that things are set up correctly based this article in the wiki ? https://github.com/WordPress/WordPress-Coding-Standards/wiki/Setting-up-WPCS-to-work-in-Atom
Also: do you only use WPCS with Atom or also on the command-line ? If so, could you test on the command-line if things work correctly with the latest version of WPCS when you use it there ?
A good indication of whether or not you are using WPCS 2+ or older is to check phpcs -i.
If WordPress-VIP is no longer listed, you are using version 2.0 or higher.

I'm trying to do things right so that we can easily find the problem.
These are all the steps done so far:

First of all, through Homebrew brew upgrade php-code-sniffer, I updated PHPCS and checked that it was the last version with the command: phpcs --version
I have completely updated Homebrew brew update
I cloned WPCS via the command git clone -b master https://github.com/WordPress/WordPress-Coding-Standards.git wpcs
I set the folder as default phpcs --config-set installed_paths / path / to / wpcs
I checked if everything went well phpcs -i (WordPress-VIP is missing)
I followed the steps of the link you posted

So now, in summary, the situation is this:

Fully updated homebrew
PHP_CodeSniffer updated to version 3.4.2
WordPress-Coding-Standards updated to version 2.1.1
linter-phpcs works well
atom-phpcs still does not perform a correct indentation
run the phpcs command line:
phpcs --standard=WordPress 404.php
and then
phpcbf --standard=WordPress 404.php
and the result is the same, an incorrect indentation.

@vizzale If I understand you correctly, the first problem of using the up-to-date version in Atom is now solved. Is that correct ?
Now for me to try and reproduce the issue with the tabs vs spaces and to figure out what's going on, I need the original code before your ran the beautifier, not the output. I seem to remember that we do actually ask for that in the issue template... ?
Anyway, I'm not going to type it over from the screenshot, so would you please either post a link to the code or post the original code in a reply ?

Oh and also, you say you ran PHPCBF with this command: phpcbf --standard=WordPress 404.php.
That would use the WordPress standard as is, not your custom ruleset.
So if that's the case, it's no surprise that your space vs tabs settings aren't being respected as you're not using your own ruleset.
Have you tried running it with just phpcbf 404.php (from the project root presuming your phpcs.xml.dist file is also in the project root) or phpcbf --standard=/path/to/phpcs.xml.dist 404.php if your custom ruleset would be located elsewhere ?

First of all, thanks for the patience @jrfnl.
As recommended by you I used the command phpcbf 404.php (my file is in the project folder). If I use this command finally the tabs are converted, as per custom ruleset, into spaces. So, first question: how can I make this work via Atom Beautify without having to use the command line each time?
Second question: the indentation still does not meet my expectations, I try to explain to you what I expect, to understand if it is possible to get it. In the image attached here there is no space or tab:
! [Image] (https://user-images.githubusercontent.com/37701762/60717363-dd79e080-9f21-11e9-8da4-5243ac6c063f.png)
Whether you launch Atom Beautify, or run the phpcbf 404.php command, the code is not correctly spelled, that is:

So, first question: how can I make this work via Atom Beautify without having to use the command line each time?

I honestly have no idea. I don't use Atom myself and that's an Atom support question, not a WPCS question.
I don't know if any of the other WPCS maintainers/watchers use Atom, if so: please jump in and share your wisdom :smile:

Second question: the indentation still does not meet my expectations, I try to explain to you what I expect, to understand if it is possible to get it.

Well, as I asked before, it would be helpful for me to have the original code to run some tests with.

I apologize, as a fool I only read your last message: D
Here's to you:
<?php
/**
 * The template for displaying 404 pages (not found)
 *
 * @link https://codex.wordpress.org/Creating_an_Error_404_Page
 *
 * @package help
 */

get_header(); ?>

Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

<?php get_footer(); ?>

@vizzale Thanks for that, but that doesn't look like the original input from the screenshot at the top ?

I've done various tests these days. This version is structurally similar to the initial one, only the indentation that is totally missing here changes.

@vizzale Ok, well I've ran some tests and I'll try and explain what is happening, but am a bit short of time, so if anything is unclear, please ask about it and I'll try and explain better when I have a little more time.

The WordPress-Core ruleset, which is included in the WordPress ruleset sets a number of properties for the Generic.WhiteSpace.ScopeIndent sniff. You overrule most in your custom ruleset, but not all.
The ignoreIndentationTokens property set by WPCS - which you don't overrule - actually excludes T_INLINE_HTML from being handled by the scope indent sniff which is why WPCS won't touch the indentation of the HTML.
If you want to change this, you need to overrule that property too in your custom ruleset, like so:<rule ref="Generic.WhiteSpace.ScopeIndent">
  <properties>
    <property name="indent" value="2"/>
    <property name="exact" value="true"/>
    <property name="tabIndent" value="false"/>
    <property name="ignoreIndentationTokens" type="array">
      <element value="T_HEREDOC"/>
      <element value="T_NOWDOC"/>
    </property>
  </properties>
</rule>

To make the ruleset completely 2-space-based, you will also need to set another property:<rule ref="PSR2.ControlStructures.SwitchDeclaration">
  <properties>
    <property name="indent" value="2"/>
  </properties>
</rule>

That sniff has a default of 4 spaces which WPCS doesn't overrule as 4 spaces suits WPCS just fine, but for your 2-space default, you would want to change this.

Now the ruleset is set up properly for 2-space indents, you'll find that the HTML indentation is still not being fixed (correctly).
The short of that, is that PHPCS only deals with PHP, JS and CSS. PHPCS does not parse HTML, nor does it reformat it.
You may now say, "Hang on, but it did make changes initially...". Yes, it did, but it only interchanged tabs vs spaces, it didn't make any changes to the actual indentation as it has no concept of, nor keeps track of HTML open/close tags.
So what I suspect is happening here, is that Atom runs PHPCS and then runs some Atom-native HTML indentation formatting over the code.
To test this:

Adjust your custom ruleset with the suggestions I made above.
Using the sample code below (yes, I know horribly formatted, but that's the point), run phpcs --standard=phpcs.xml.dist ./testfile.php over the test file (pointing to your custom ruleset as adjusted). You will see errors/warnings about precision alignment and end of line whitespace, but no errors reported about (scope) indentation.
Next run phpcbf --standard=phpcs.xml.dist ./testfile.php --suffix=.fixed over the same test file (pointing to your custom ruleset as adjusted). This will output the result to a separate file with a .fixed suffix. Take note that the HTML indentation in the .fixed file is unchanged when compared to the original .php file.
Now run the Atom Beautifier over the same code in testfile.php (which should still be unchanged up to now).
Compare the testfile.php.fixed file (the PHPCBF output) with the file beautified by Atom to see the difference. Any differences you see are Atom doing something extra outside of the scope/rules set by WPCS.

I hope that explains it.
You may then say, but shouldn't WPCS deal with HTML indentation too ?
And the answer would be "no". Your template is quite "clean" in that it has the open/close tags for each HTML element in the same block of inline HTML.
For most PHP code, this is not the case.
The open tags may be in one method and the close tags in another. There can be blocks of PHP in between blocks of inline HTML, open and close tags may be in different files (think header.php/footer.php) etc etc.
In short, it would be impossible to reliably keep track of the HTML indentation needed, so there is no intention to add any checks of that kind to WPCS. (which is part of the reason why WPCS excludes inline HTML from the scope indent sniff in the first place)
Probably not the answer you were hoping for, but I hope it helps to understand how these things work anyhow.

Horribly formatted HTML sample code
<?php
/**
 * The template for displaying 404 pages (not found)
 *
 * @link https://codex.wordpress.org/Creating_an_Error_404_Page
 *
 * @package help
 */

get_header(); ?>

Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.

<?php get_footer(); ?>

Thanks @jrfnl you have been more than exhaustive and I appreciate your availability.
I followed all your suggestions and I corrected my custom ruleset with your instructions.
I have only one last question: based on your experience, are there any "more suitable" beautifiers that I could use, perhaps continuing to use PHP_CodeSniffer only for linter-phpcs, to process this type of code?
I have also tried to solve the problem with Atom Beautify by doing several attempts, but this doesn't take anything at all considering my custom ruleset. If anyone has any ideas, whistle.

Hi @jrfnl  I'm going to bother you again because this morning I tried to run the command phpcbf --standard = phpcs.xml.dist 404.php on a file and now PHP_CodeSniffer converts the tabs into 4 spaces instead of 2.
My custom ruleset is identical to yesterday's. By the way, I also removed the last rule you suggested but nothing, it converts the tabs into 4 spaces. I can't understand what could have happened.

Hi @jrfnl,
no idea what may depend on or what can be done to understand what is creating this incongruity of the 4 spaces?

Hi @jrfnl I'm trying in every way to solve the problem but I'm not able to do it.
Although my custom ruleset is the one we set together this converts the tab into 4 spaces instead of 2 spaces.
It seems therefore that my custom ruleset is not complete. Could you help me understand what's still missing?
Thanks a lot.

@vizzale Sorry for my slow response. I only just saw that I hadn't replied to your last questions.
Try setting   <arg name="tab-width" value="2"/> after the inclusion of the WordPress ruleset.
The WordPress ruleset sets the tab-width to 4, so your custom setting  may be overruled.
The order of directives in a ruleset is important as the rules are read top to bottom.
Please let me know if that helps.

Hi Juliette, I too am writing to you very late.
I do it because since then, despite your suggestion, I have not been able to solve the problem.
Can we carry out some type of investigation to understand why the configuration file cannot convert a tab into two spaces? Tell me everything you need to help you understand what the wrong operation depends on.
I hope we can finally understand the problem.
Thanks in advance for your patience.


pipelinerunid - provide a link

pipelinerunid is referenced in all examples for powershell as a required param - might be nice to have a complete solution that references how it's gotten.

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

ID: bf124c4c-3ba1-57ed-3679-8539f2a311fc
Version Independent ID: 4b32b092-c4ad-74aa-58d3-75449d87d306
Content: Programmatically monitor an Azure data factory
Content Source: articles/data-factory/monitor-programmatically.md
Service: data-factory
GitHub Login: @douglaslMS
Microsoft Alias: douglasl

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

@iamthehousedog at the top of each language section there is a link to creating and monitoring a pipeline, take a look at the Powershell example:
https://docs.microsoft.com/en-us/azure/data-factory/monitor-programmatically#powershell
When you follow the link  you will see examples of how to create a pipeline activity and save its RunID:
https://docs.microsoft.com/en-us/azure/data-factory/quickstart-create-data-factory-powershell#create-a-pipeline-run
The top of the document refers to the ''Get-AzureRmDataFactoryV2PipelineRun" Powershell command:
https://docs.microsoft.com/en-us/azure/data-factory/monitor-programmatically#data-range
You can use this to list the RunIDs of a particular pipeline:
https://docs.microsoft.com/en-us/powershell/module/azurerm.datafactories/get-azurermdatafactoryv2pipelinerun?view=azurermps-4.4.1#description
We are closing this issue for now but feel free to add comments as needed, and once again thank you for the feedback!


Content Suggestion: Einundzwanzig Podcast

Link to Content: https://anchor.fm/einundzwanzig
Briefly Describe Content: Bitcoin-only podcast in German
Suggested Section: Podcasts

Translated website and it all appears bitcoin only. Also, it is Gigi's pod.

ACK


Opacity

It feels like I'm being a total rookie here, but I can seem to find the 'opacity' controle in my menu. I've read FAQ and issues posted, but ti feels like 'opacity' should be somewhere in the menu easily accessible and I can't see it (version 4.0.221).
Thanks! The app is GREAT!

It feels like I'm being a total rookie here, but I can seem to find the 'opacity' controle in my menu. I've read FAQ and issues posted, but ti feels like 'opacity' should be somewhere in the menu easily accessible and I can't see it (version 4.0.221).
Thanks! The app is GREAT!

Hi! It's not in the menu .. it's a slider in the top left corner of the window

Much appreciated! found it! Truly sorry to take your time... i really enjoy the OotD experience! Thanks again.


Improve performance significantly across the board

This implements what I described in option 2 in issue #53 .
Here's a small video of scrolling (holding down the j key) on a 800,000 line file (with line numbers enabled) before this change:
https://user-images.githubusercontent.com/61218022/141846528-395d0b17-124a-44e3-a7ac-e55e86bf21a5.mov
And here's after:
https://user-images.githubusercontent.com/61218022/141846562-93b9922f-5aa7-4922-9c5a-28f39e1b78bc.mov
All tests still pass, and, as far as I can tell, this PR implements no new bugs or regressions to be aware of. I have not yet tested with dynamic paging (just static paging), so I would appreciate at least some basic tests with this branch on something dynamically paged.
This PR, outside of the logic rewrite, also allows for significantly less vector clones and overall allocations; e.g., instead of cloning the lines that needed to be printed to format them and then take the section to be printed, this can simply borrow a slice from the formatted_lines vector and print that directly; no copies needed.
I recognize that there is a significant difference in the level of commenting on the lines I wrote; if you'd like, I can comment them and push some new commits before you merge. Most of this, however, is just logic being moved around, so I expect you would also be able to just copy over the comments from current main.

Thanks aga

Sorry I wrongly closed your PR before merging it.


Update version of react etc.

Please do work for this task in a branch named issue-003.

Followed the instructions for updating React Router to v6, https://reactrouter.com/docs/en/v6/upgrading/v5.
Made the changes to App.jsx. Now using React 18.1.0, React Router 6.3.0.


The Production deployment is not working, no changes from dev end, but the dpeloyment stopped working

The Production deployment is not working, no changes from dev end, but the dpeloyment stopped working.  can you please help here.
We not able to find any issue in log
Trying to pass milestone 80
[2021-01-06T11:52:26.836Z] --- Begin library step of: dockerExecuteOnKubernetes ---
[2021-01-06T11:52:27.211Z] [dockerExecuteOnKubernetes] Sending telemetry data is disabled.
[2021-01-06T11:52:41.176Z] ----------------------------------------------------------
[2021-01-06T11:52:41.176Z] --- An error occurred in the library step: dockerExecuteOnKubernetes
[2021-01-06T11:52:41.176Z] ----------------------------------------------------------
[2021-01-06T11:52:41.176Z]
[2021-01-06T11:52:41.176Z] The following parameters were available to the step:
[2021-01-06T11:52:41.176Z] ***
[2021-01-06T11:52:41.176Z] *** to show step parameters, set verbose:true in general pipeline configuration
[2021-01-06T11:52:41.176Z] *** WARNING: this may reveal sensitive information. ***
[2021-01-06T11:52:41.176Z] ***
[2021-01-06T11:52:41.176Z]
[2021-01-06T11:52:41.176Z] The error was:
[2021-01-06T11:52:41.176Z] ***
[2021-01-06T11:52:41.176Z] org.jenkinsci.plugins.workflow.steps.FlowInterruptedException
[2021-01-06T11:52:41.176Z] ***
[2021-01-06T11:52:41.176Z]
[2021-01-06T11:52:41.176Z] Further information:
[2021-01-06T11:52:41.176Z] * Documentation of library step dockerExecuteOnKubernetes: https://sap.github.io/jenkins-library/steps/dockerExecuteOnKubernetes/
[2021-01-06T11:52:41.176Z] * Source code of library step dockerExecuteOnKubernetes: https://github.com/SAP/jenkins-library/blob/master/vars/dockerExecuteOnKubernetes.groovy
[2021-01-06T11:52:41.176Z] * Library documentation: https://sap.github.io/jenkins-library/
[2021-01-06T11:52:41.176Z] * Library repository: https://github.com/SAP/jenkins-library/
[2021-01-06T11:52:41.176Z]
[2021-01-06T11:52:41.176Z] ----------------------------------------------------------
[2021-01-06T11:52:41.176Z] --- End library step of: dockerExecuteOnKubernetes ---
[2021-01-06T11:52:41.905Z] [debugReportArchive] Sending telemetry data is disabled.
[2021-01-06T11:52:42.087Z] Archiving artifacts
[2021-01-06T11:52:42.123Z] Successfully archived debug report as 'confidential_debug_log_2021-01-06-11-52_com.sap.qm.sip.txt'
[2021-01-06T11:52:42.292Z] --- BEGIN LIBRARY STEP: postActionSendNotification.groovy ---
[2021-01-06T11:52:42.323Z] --- END LIBRARY STEP: postActionSendNotification.groovy ---
[2021-01-06T11:52:42.389Z] --- BEGIN LIBRARY STEP: postActionCleanupStashesLocks.groovy ---
[2021-01-06T11:52:42.529Z] --- END LIBRARY STEP: postActionCleanupStashesLocks.groovy ---
[2021-01-06T11:52:42.594Z] Sending telemetry data is disabled.
https://qmpspa.jaas-gcp.cloud.sap.corp/job/qm-pspa/job/master/

Hi
FlowInterruptedException hints that the build you point out was aborted. What did you expect?

Hi
FlowInterruptedException hints that the build you point out was aborted. What did you expect?


Fix #171: improper formatting of unit-y wavelength quantities

I did a pass through looking for places where wavelength was being suspiciously multiplied by numerical constants and caught another instance that needs fixing.

looks good, thanks.


[WIP][SPARK-28737][CORE] Update Jersey to 2.29

What changes were proposed in this pull request?
Update Jersey to 2.27+, ideally 2.29, for possible JDK 11 fixes.
How was this patch tested?
Existing tests.

Test build #109124 has started for PR 25455 at commit ad2317c.

Test build #109124 has finished for PR 25455 at commit ad2317c.

This patch fails build dependency tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109124/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14192/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14193/
Test PASSed.

Test build #109126 has started for PR 25455 at commit f6c4ed1.

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14196/
Test PASSed.

Test build #109128 has started for PR 25455 at commit f6c4ed1.

Interesting, in this PR, I also saw the same 500 error failure.

https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109128/testReport

[info] UISeleniumSuite:
...
[info] - effects of unpersist() / persist() should be reflected *** FAILED *** (1 second, 120 milliseconds)
[info]   java.lang.RuntimeException: got code: 500 when getting http://amp-jenkins-worker-05.amp:38637/api/v1/applications/local-1565819875887/storage/rdd w/ error: Some(<html>
[info] <head>
[info] <meta http-equiv="Content-Type" content="text/html;charset=utf-8"/>

cc @wangyum

Yep this one doesn't seem straightforward. I'll work on figuring out what's the new error on update. I feel like I tried this a while ago and ran into similar problems. It seems to be mostly that Jersey reorganized its modules and some new ones must be included.

Test build #109126 has finished for PR 25455 at commit f6c4ed1.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109126/
Test FAILed.

Test build #109128 has finished for PR 25455 at commit f6c4ed1.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109128/
Test FAILed.

Test build #109132 has started for PR 25455 at commit 3e0b534.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14200/
Test PASSed.

Merged build finished. Test PASSed.

Test build #109132 has finished for PR 25455 at commit 3e0b534.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109132/
Test PASSed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14205/
Test PASSed.

Test build #109137 has started for PR 25455 at commit 3e0b534.

Test build #109137 has finished for PR 25455 at commit 3e0b534.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109137/
Test FAILed.

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14229/
Test PASSed.

Test build #109160 has started for PR 25455 at commit 57add37.

Test build #109160 has finished for PR 25455 at commit 57add37.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109160/
Test FAILed.

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14275/
Test PASSed.

Test build #109205 has started for PR 25455 at commit 57add37.

The test appeared to pass actually, but let me re-run it.

Test build #4833 has started for PR 25455 at commit 57add37.

Yes. It fails at GitHub posting. It seems to be GitHub outage.
Tests passed.
Attempting to post to Github...
Failed to post message to Github.
 > urllib_status: Temporary failure in name resolution
 > data: {"body": "**[Test build #109205 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109205/testReport)** for PR 25455 at commit [`57add37`](https://github.com/apache/spark/commit/57add373ba6cc95730f3f57f9e5b2deb13ab121a).\n * This patch passes all tests.\n * This patch merges cleanly.\n * This patch adds no public classes."}

Oops. Sorry. Although #25474 touches the same files, I thought it's okay because it touches a different part. Could you rebase once more?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14305/
Test PASSed.

Test build #109236 has started for PR 25455 at commit 431f3d5.

Test build #4833 has finished for PR 25455 at commit 57add37.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #109236 has finished for PR 25455 at commit 431f3d5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109236/
Test PASSed.


toOption() should be consistent with the result of Option.of()

https://github.com/vavr-io/vavr/blob/e43e44b683467d94df90b38b937db2eaaa6a2603/src/main/java/io/vavr/Value.java#L1187
Suggest to change to：
return (get() == null) ? Option.none() : Option.some(get());

Although the state of Try only depends on a success or a failure of a computation, reflected by the types Success and Failure, null has nothing to do with that state , but should be consistent with the result of Option.of().

We can't do that because Option wouldn't be monadic (read: composable) anymore. We align to the semantics of Scala's Option, not to that of Optional.
Please read also our blog post on that topic.
Further reading:

Why java.util.Optional is broken by Atlassian (2015)
How Optional Breaks the Monad Laws and Why It Matters by Marcello La Rocca (2016)

Vavr is here to make a difference in the space of functional programming with Java. Sometimes this means to fix the broken Java standard library.
Thank you for your question, this is important! However, I will close this issue because we can't change the behavior without braking core functionality (like the For-comprehension).


"Cannot read property 'id' of undefined"

yarn run v1.22.10
$ node build/index.js
[INFO] Using configuration file: ../config.json
[INFO] Using Discord WebHooks to send messages
C:\Users\Xport\Downloads\shulker-3.0.0\shulker-3.0.0\node_modules\discord.js\src\client\ClientDataManager.js:81
        guild.channels.set(channel.id, channel);
                                   ^

TypeError: Cannot read property 'id' of undefined
    at ClientDataManager.newChannel (C:\Users\Xport\Downloads\shulker-3.0.0\shulker-3.0.0\node_modules\discord.js\src\client\ClientDataManager.js:81:36)
    at Guild.setup (C:\Users\Xport\Downloads\shulker-3.0.0\shulker-3.0.0\node_modules\discord.js\src\structures\Guild.js:206:68)
    at GuildCreateHandler.handle (C:\Users\Xport\Downloads\shulker-3.0.0\shulker-3.0.0\node_modules\discord.js\src\client\websocket\packets\handlers\GuildCreate.js:12:15)
    at WebSocketPacketManager.handle (C:\Users\Xport\Downloads\shulker-3.0.0\shulker-3.0.0\node_modules\discord.js\src\client\websocket\packets\WebSocketPacketManager.js:105:65)
    at WebSocketConnection.onPacket (C:\Users\Xport\Downloads\shulker-3.0.0\shulker-3.0.0\node_modules\discord.js\src\client\websocket\WebSocketConnection.js:333:35)
    at WebSocketConnection.onMessage (C:\Users\Xport\Downloads\shulker-3.0.0\shulker-3.0.0\node_modules\discord.js\src\client\websocket\WebSocketConnection.js:296:17)
    at WebSocket.onMessage (C:\Users\Xport\Downloads\shulker-3.0.0\shulker-3.0.0\node_modules\ws\lib\event-target.js:120:16)
    at WebSocket.emit (events.js:314:20)
    at Receiver.receiverOnMessage (C:\Users\Xport\Downloads\shulker-3.0.0\shulker-3.0.0\node_modules\ws\lib\websocket.js:789:20)
    at Receiver.emit (events.js:314:20)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Not sure exactly what the problem is here. What does your config look like (removing any sensitive data)? Is your bot setup correclty?

I just had the same issue. It happened after I tried to add a server image at "SERVER_IMAGE": "smithhappy.png", at least I think. I'll update if I get it working again
config.json:
`{
"PORT": 8000,
"USE_WEBHOOKS": true,
"WEBHOOK_URL": "xxx",
"IGNORE_WEBHOOKS": true,
"DISCORD_TOKEN": "xxx",
"DISCORD_CHANNEL_ID": "xxx",
"DISCORD_CHANNEL_NAME": "#mc-chat",
"DISCORD_MESSAGE_TEMPLATE": "`%username%`: %message%",

"MINECRAFT_SERVER_RCON_IP": "<IP_ADDRESS>",
"MINECRAFT_SERVER_RCON_PORT": 25575,
"MINECRAFT_SERVER_RCON_PASSWORD": "xxx",
"MINECRAFT_TELLRAW_TEMPLATE": "[{\"color\": \"#55ffff\", \"text\": \"<%username%> %message%\"}]",
"MINECRAFT_TELLRAW_DOESNT_EXIST": false,
"MINECRAFT_TELLRAW_DOESNT_EXIST_SAY_TEMPLATE": "<%username%> %message%",

"IS_LOCAL_FILE": true,
"LOCAL_FILE_PATH": "E:/Games/Minecraft/2022 server/logs/latest.log",

"SHOW_INIT_MESSAGE": true,

"ALLOW_USER_MENTIONS": true,
"ALLOW_HERE_EVERYONE_MENTIONS": false,
"ALLOW_SLASH_COMMANDS": true,
"SLASH_COMMAND_ROLES": ["op"],

"WEBHOOK": "/minecraft/hook",
"REGEX_SERVER_PREFIX": "\\[Server thread/INFO\\]:",
"REGEX_MATCH_CHAT_MC": "^<([^>]*)> (.*)",
"REGEX_IGNORED_CHAT": "packets too frequently",
"DEBUG": false,

"SERVER_NAME": "Agent_Smith",
"SERVER_IMAGE": "smithhappy.png",
"SHOW_SERVER_STATUS": true,
"SHOW_PLAYER_CONN_STAT": true,
"SHOW_PLAYER_ADVANCEMENT": true,
"SHOW_PLAYER_DEATH": true,
"SHOW_PLAYER_ME": true,
"DEATH_KEY_WORDS": ["shot", "fell", "death", "died", "doomed", "pummeled", "removed", "didn't want", "withered", "squashed", "flames", "burnt", "walked into", "bang", "roasted", "squished", "drowned", "killed", "slain", "blown", "blew", "suffocated", "struck", "lava", "impaled", "speared", "fireballed", "finished", "kinetic"]

}
`
cmd log:

<EMAIL_ADDRESS>start E:\Games\Minecraft\2022 server\shulker-master
node build/index.js

[INFO] Using configuration file: ../config.json
[INFO] Using Discord WebHooks to send messages
E:\Games\Minecraft\2022 server\shulker-master\node_modules\discord.js\src\client\ClientDataManager.js:81
guild.channels.set(channel.id, channel);
^
TypeError: Cannot read property 'id' of undefined
at ClientDataManager.newChannel (E:\Games\Minecraft\2022 server\shulker-master\node_modules\discord.js\src\client\ClientDataManager.js:81:36)
at Guild.setup (E:\Games\Minecraft\2022 server\shulker-master\node_modules\discord.js\src\structures\Guild.js:307:68)
at GuildCreateHandler.handle (E:\Games\Minecraft\2022 server\shulker-master\node_modules\discord.js\src\client\websocket\packets\handlers\GuildCreate.js:12:15)
at WebSocketPacketManager.handle (E:\Games\Minecraft\2022 server\shulker-master\node_modules\discord.js\src\client\websocket\packets\WebSocketPacketManager.js:108:65)
at WebSocketConnection.onPacket (E:\Games\Minecraft\2022 server\shulker-master\node_modules\discord.js\src\client\websocket\WebSocketConnection.js:336:35)
at WebSocketConnection.onMessage (E:\Games\Minecraft\2022 server\shulker-master\node_modules\discord.js\src\client\websocket\WebSocketConnection.js:299:17)
at WebSocket.onMessage (E:\Games\Minecraft\2022 server\shulker-master\node_modules\ws\lib\event-target.js:120:16)
at WebSocket.emit (events.js:315:20)
at Receiver.receiverOnMessage (E:\Games\Minecraft\2022 server\shulker-master\node_modules\ws\lib\websocket.js:789:20)
at Receiver.emit (events.js:315:20)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: node build/index.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\josep\AppData\Roaming\npm-cache_logs\2021-11-30T20_47_58_731Z-debug.log

sidenote, I'm not familiar at all with java but I'm running the most recent version of Adoptium. I attached the debug log
2021-11-30T20_59_20_057Z-debug.log
.

tried removing all of the related files and going through all of the setup steps again, still getting the same issue :/

What channel type is it? I feel like Discord may of updated this https://github.com/discordjs/discord.js/blob/v11/src/client/ClientDataManager.js#L63-L79

It's a text channel. What's strange is I had it set up exactly the same on my feature test discord and it was working great, but I just realized this actually started when I invited the bot to the server on which I intend to use it.
Since then even removing all the files related to the bot and running those npm commands again causes the same error. I'll double check the settings this afternoon and run everything again just to be sure!

@speakwithanimals I just pushed a couple of updates (https://github.com/destruc7i0n/shulker/commit/527ba4df2211f8787a4ccee707861dd0959eae37) including a check for the channel on launch, can you try again with these changes?

same error unfortunately :/
2021-12-08T17_43_39_748Z-debug.log
my config.json (with bot info redacted)
`{
"PORT": 8000,
"USE_WEBHOOKS": true,
"WEBHOOK_URL": "xxx",
"IGNORE_WEBHOOKS": true,
"DISCORD_TOKEN": "xxx",
"DISCORD_CHANNEL_ID": "xxx",
"DISCORD_CHANNEL_NAME": "#server-chat",
"DISCORD_MESSAGE_TEMPLATE": "%username%: %message%",
"MINECRAFT_SERVER_RCON_IP": "<IP_ADDRESS>",
"MINECRAFT_SERVER_RCON_PORT": 25575,
"MINECRAFT_SERVER_RCON_PASSWORD": "xxx",
"MINECRAFT_TELLRAW_TEMPLATE": "[{"color": "white", "text": "<%username%> %message%"}]",
"MINECRAFT_TELLRAW_DOESNT_EXIST": false,
"MINECRAFT_TELLRAW_DOESNT_EXIST_SAY_TEMPLATE": "<%username%> %message%",
"IS_LOCAL_FILE": true,
"LOCAL_FILE_PATH": "E:/Games/Minecraft/2022 server/logs/latest.log",
"SHOW_INIT_MESSAGE": true,
"ALLOW_USER_MENTIONS": true,
"ALLOW_HERE_EVERYONE_MENTIONS": false,
"ALLOW_SLASH_COMMANDS": true,
"SLASH_COMMAND_ROLES": ["op"],
"WEBHOOK": "/minecraft/hook",
"REGEX_SERVER_PREFIX": "\[Server thread/INFO\]:",
"REGEX_MATCH_CHAT_MC": "^<([^>])> (.)",
"REGEX_IGNORED_CHAT": "packets too frequently",
"DEBUG": false,
"SERVER_NAME": "Agent Smith",
"SERVER_IMAGE": "",
"HEAD_IMAGE_URL": "https://mc-heads.net/avatar/%uuid%/256",
"DEFAULT_PLAYER_HEAD": "c06f89064c8a49119c29ea1dbd1aab82",
"SHOW_SERVER_STATUS": false,
"SHOW_PLAYER_CONN_STAT": false,
"SHOW_PLAYER_ADVANCEMENT": false,
"SHOW_PLAYER_DEATH": false,
"SHOW_PLAYER_ME": false,
"DEATH_KEY_WORDS": [
"shot",
"fell",
"death",
"died",
"doomed",
"pummeled",
"removed",
"didn't want",
"withered",
"squashed",
"flames",
"burnt",
"walked into",
"bang",
"roasted",
"squished",
"drowned",
"killed",
"slain",
"blown",
"blew",
"suffocated",
"struck",
"lava",
"impaled",
"speared",
"fireballed",
"finished",
"kinetic"
]
}`

@speakwithanimals Thanks for testing it and letting me know!
I did some digging and it seems to be because of a stage channel and the version of discord.js I was using not supporting it. I've updated discord.js to the latest version which should hopefully fix this. Can you try the bot from this branch with the update to discord.js? https://github.com/destruc7i0n/shulker/tree/discord-js-13

I must be cursed or something, haha. I tried two separate channels and I'm getting what looks like the same issue in both.
2021-12-20T03_29_51_456Z-debug.log
2021-12-20T03_32_22_577Z-debug.log
2021-12-20T03_33_23_029Z-debug.log
config redacted.txt
.

@speakwithanimals would you mind sending the output of the script itself rather than the debug log? The debug log is just about the same in all three files there

I don't know why, but I just assumed I wouldn't be able to understand the output myself, so after taking a look at it I realized I just needed to update Node! Sorry about the confusion 🙃

@speakwithanimals That is great to hear, thanks for helping me debug this nonetheless. Happy holidays!

Fixed in #77


Previous @tanstack/react-query versions unpublished?

Describe the bug
Unable to install versions before 4.0.5
npm view @tanstack/react-query versions  results in only 4.0.5 being listed
Your minimal, reproducible example
https://www.npmjs.com/package/@tanstack/react-query
Steps to reproduce
Try installing with a fixed version like 4.0.4
Expected behavior
Expected to install 4.0.4 successfully
How often does this bug happen?
No response
Screenshots or Videos
npm only shows a single version as well

No response
Platform

OS: MacOS

react-query version
4.0.3
TypeScript version
No response
Additional context
No response

same here!

They have been unpublished by accident. please start with 4.0.5.


[fastgltf] New port

Describe the pull request

Which triplets are supported/not supported?

Yes

By default, vcpkg does not build test and example, so could you use macros to control the enabling of these two parts, and then pass the macro value of off in vcpkg_cmake_configure(). Thank you!

Considering that building tests and examples for each port will impose a great burden on the ci system, we do not build them by default, but they can be used as features of ports. We could add the test and the example as a feature of fastgltf.
About disable parallel configure, you could add DISABLE_PARALLEL_CONFIGURE option into the vcpkg_cmake_configure(), such as:
vcpkg_cmake_configure(
    SOURCE_PATH "${SOURCE_PATH}"
    DISABLE_PARALLEL_CONFIGURE
    OPTIONS -DFASTGLTF_DOWNLOAD_SIMDJSON=OFF
)

Please add the test and the example as a feature of fastgltf, and use macros to control whether they are enabled or not.
Thank you!

No, I don't see any gain in adding the tests and examples as port features, especially when the tests require a 3GB download of sample files. I don't know of any other port that does that and I seriously don't see why one would want these tests or examples. If anybody wants to, and I doubt anyone will want to if they're not already working on contributing to it, they can clone the repo and compile them. Perhaps you could add this afterwards if anyone opens an issue and asks for it. As no doubt you are aware vcpkg is used for installing library dependencies, not for running tests.

I think the PR is okay as is.
IIUC examples and tests have the EXCLUDE_FROM_ALL property, so no patching for feature control is required. But I didn't verify in the build log.

Yes, the dependencies required for tests and examples are only downloaded through a Python script. As that script is never invoked the tests and examples would result in build failures because of missing headers and libraries.

Test and example were not built.
[1/4] C:\PROGRA~1\MICROS~2\2022\ENTERP~1\VC\Tools\MSVC\1433~1.316\bin\Hostx64\x64\cl.exe   /TP -DSIMDJSON_THREADS_ENABLED=1 -IF:\fastgltf\buildtrees\fastgltf\src\d1f0969b11-2105bfd15b\src -external:IF:\fastgltf\installed\x64-windows-static\include -external:W0 /nologo /DWIN32 /D_WINDOWS /W3 /utf-8 /GR /EHsc /MP  /D_DEBUG /MTd /Z7 /Ob0 /Od /RTC1  /EHsc -std:c++17 /showIncludes /Fosrc\CMakeFiles\fastgltf.dir\base64_decode.cpp.obj /Fdsrc\CMakeFiles\fastgltf.dir\fastgltf.pdb /FS -c F:\fastgltf\buildtrees\fastgltf\src\d1f0969b11-2105bfd15b\src\base64_decode.cpp
[2/4] C:\PROGRA~1\MICROS~2\2022\ENTERP~1\VC\Tools\MSVC\1433~1.316\bin\Hostx64\x64\cl.exe   /TP -DSIMDJSON_THREADS_ENABLED=1 -IF:\fastgltf\buildtrees\fastgltf\src\d1f0969b11-2105bfd15b\src -external:IF:\fastgltf\installed\x64-windows-static\include -external:W0 /nologo /DWIN32 /D_WINDOWS /W3 /utf-8 /GR /EHsc /MP  /D_DEBUG /MTd /Z7 /Ob0 /Od /RTC1  /EHsc -std:c++17 /showIncludes /Fosrc\CMakeFiles\fastgltf.dir\fastgltf.cpp.obj /Fdsrc\CMakeFiles\fastgltf.dir\fastgltf.pdb /FS -c F:\fastgltf\buildtrees\fastgltf\src\d1f0969b11-2105bfd15b\src\fastgltf.cpp
[3/4] cmd.exe /C "cd . && C:\PROGRA~1\MICROS~2\2022\ENTERP~1\VC\Tools\MSVC\1433~1.316\bin\Hostx64\x64\lib.exe  /machine:x64 /nologo /out:src\fastgltf.lib src\CMakeFiles\fastgltf.dir\base64_decode.cpp.obj src\CMakeFiles\fastgltf.dir\fastgltf.cpp.obj  && cd ."
[3/4] cmd.exe /C "cd /D F:\fastgltf\buildtrees\fastgltf\x64-windows-static-dbg && F:\fastgltf\downloads\tools\cmake-3.24.0-windows\cmake-3.24.0-windows-i386\bin\cmake.exe -P cmake_install.cmake"
-- Install configuration: "Debug"
-- Installing: F:/fastgltf/packages/fastgltf_x64-windows-static/debug/include/base64_decode.hpp
-- Installing: F:/fastgltf/packages/fastgltf_x64-windows-static/debug/include/fastgltf_parser.hpp
-- Installing: F:/fastgltf/packages/fastgltf_x64-windows-static/debug/include/fastgltf_types.hpp
-- Installing: F:/fastgltf/packages/fastgltf_x64-windows-static/debug/include/fastgltf_util.hpp
-- Installing: F:/fastgltf/packages/fastgltf_x64-windows-static/debug/lib/fastgltf.lib
-- Installing: F:/fastgltf/packages/fastgltf_x64-windows-static/debug/lib/cmake/fastgltf/fastgltfConfig.cmake
-- Installing: F:/fastgltf/packages/fastgltf_x64-windows-static/debug/lib/cmake/fastgltf/fastgltfConfig-debug.cmake


Don't print gpu-func compile messages to standard-out, use something quieter

I'm not sure what the correct approach is for this. I'll ask on IRC later

error-output is a sensible destination for error messages.

True, but this is about notes from the compiler. Sbcl prints some ancillary info for certain actions (like when compiling a function) I think this is outside the spec


RC9 creates files called "C" with  tags inside

Expected Behavior
Running ./gradlew --rerun-tasks detekt inside a Git repository with a mutli-project Gradle project should not create any (untracked) files.
Current Behavior
After running ./gradlew --rerun-tasks detekt and git status I see
Untracked files:
  (use "git add <file>..." to include in what will be committed)

        analyzer/C
        cli/C
        downloader/C
        model/C
        reporter/C
        scanner/C
        utils-test/C
        utils/C

Steps to Reproduce (for bugs)

git clone git clone --recurse-submodules https://github.com/heremaps/oss-review-toolkit.git
./gradlew --rerun-tasks detekt
git status

Context
I was trying to run detekt on our Kotlin project to identify code smells.
Your Environment

Version used: 1.0.0.RC9
Operating System and version: Windows 10, 64-bit
Link to your project: https://github.com/heremaps/oss-review-toolkit

Thanks for the report. This seems to be an issue related to our report path parsing on Windows.
Closing this as a duplicate of #1123
See #1145 for the fix


fix: transform pipe

:tada: This PR is included in version 5.0.0-rc.10 :tada:
The release is available on:

npm package (@rc dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


allow repeating eldest keys in the sigchain, and respect 'eldest' link resets

Before, we used to short-circuit a lot of our sigchain replay code until
we got to the current subchain. Now, we actually do a complete replay of
all subchains, and implement a _reset_subchain() method to clear the
relevant state when we detect a reset.
We need this logic because now there might be more than one subchain
that corresponds to our current eldest key. The only other option
would've been to detect the current subchain on a first pass, and only
then do a full replay. But that makes it difficult to keep track of the
KeyState that tells us what PGP full hash to use at any given time.
Fully replaying everything is actually simpler, and our signature cache
hopefully absolves us of any major perf worries.

Depends on https://github.com/keybase/keybase-test-vectors/pull/1


[OHIF-251] - Double click to maximise viewport/ revert to previous layout.

Includes cornerstoneTools 4.18.0 which allows us to cleanly cancel and broadcast removed toolState if we destroy the element whilst making a cornerstoneTools annotation.
Otherwise the viewport logic is common to all viewport types.
Needs to prevent click through on ViewportActionbar as right now jumping between measurements quickly triggers a double click on the cornerstone element.

LGTM 👍

I don't see this feature has been implemented in the main branch as I have downloaded a copy last week.  Please correct me if I'm mistaken.
If this has not be merged to the main branch how do I go by making this change happen.  I see 18 files have been modified in this thread and some are outdated.  Any advice would be much appreciated!
Thanks,
Ric


Fix links

On "Contributing with GitHub via its interface" (Introduction and Github sections) the links for the Github repository redirect back to the tutorial instead of going to Github.
I Changed this:
[{{ site.github_repository }}]({{ site.github.repository_url }})
For this:
[{{ site.github_repository }}]({{ site.github_repository }})
To correct the links.

Seems to be fixed since then.


scrolls, staves, spellbooks

Scrolls are 1-off casts that use up the scroll but take no mana.
Staffs can hold a number of spell charges and take little mana.
Spellbooks hold 'special' spells and do take mana.
effectively read, recite, and brandish commands.

moved to 0.3.5

any game item can now have a skill object (or array of objects) attached to a 'spell' property. a new command, brandish, was added to cover all cases. any item can be slotted with spells. check out the brandish function directly to see advanced options.