unauthorised error with http-01

I have read and try to figure this out for weeks. Those I read said to turn of IPv6 but this is already off for me. I have also tested accessing a page to the website from the internet without issue.
We have another server that we can install the cert without issue in the same environment.
How else can I troubleshoot this?
This is the error I am getting:
[DBUG] Scanning IIS site bindings for hosts
[VERB] 48 named bindings found in IIS
[DBUG] Filtering by site(s) [39]
[VERB] 1 bindings remaining after site filter
[VERB] No host filter applied
[VERB] 1 matching binding found
[DBUG] Scanning IIS sites
[INFO] Target generated using plugin IIS: a.cps4.me
[DBUG] Scanning IIS site bindings for hosts
[VERB] 48 named bindings found in IIS
[DBUG] Filtering by site(s) [39]
[VERB] 1 bindings remaining after site filter
[VERB] No host filter applied
[VERB] 1 matching binding found
[DBUG] Scanning IIS sites
[VERB] Targeted convert into 1 order(s)
[VERB] Checking [IIS] a.cps4.me, (any host)
[VERB] Handle order 1/1: Main
[DBUG] Refreshing order...
[VERB] Loading ACME account signer...
[DBUG] Loading signer from C:\ProgramData\win-acme\acme-staging-v02.api.letsencrypt.org\Signer_v2
[VERB] Constructing ACME protocol client...
[DBUG] Send GET request to https://acme-staging-v02.api.letsencrypt.org/directory
[VERB] Request completed with status OK
[DBUG] Send HEAD request to https://acme-staging-v02.api.letsencrypt.org/acme/new-nonce
[VERB] Request completed with status OK
[DBUG] Loading account information from C:\ProgramData\win-acme\acme-staging-v02.api.letsencrypt.org\Registration_v2
[DBUG] Send POST request to https://acme-staging-v02.api.letsencrypt.org/acme/order/13362916/96775092
[VERB] Request completed with status OK
[DBUG] Cached order has status invalid, discarding
[VERB] Creating order for hosts: ["a.cps4.me"]
[DBUG] Send POST request to https://acme-staging-v02.api.letsencrypt.org/acme/new-order
[VERB] Request completed with status Created
[VERB] Order https://acme-staging-v02.api.letsencrypt.org/acme/order/13362916/96775432 created
[VERB] Handle authorization 1/1
[DBUG] Send POST request to https://acme-staging-v02.api.letsencrypt.org/acme/authz-v3/60346301
[VERB] Request completed with status OK
[INFO] Authorize identifier a.cps4.me
[VERB] Initial authorization status: pending
[VERB] Challenge types available: ["http-01", "dns-01", "tls-alpn-01"]
[VERB] Initial challenge status: pending
[INFO] Authorizing a.cps4.me using http-01 validation (SelfHosting)
[DBUG] Submitting challenge answer
[DBUG] Send POST request to https://acme-staging-v02.api.letsencrypt.org/acme/chall-v3/60346301/DQ1_WA
[VERB] Request completed with status OK
[VERB] SelfHosting plugin serving file /.well-known/acme-challenge/xeuu_00nH2LvuuwokR1wEP7lSmNxlHPEg9dE0RSNHtk
[VERB] SelfHosting plugin serving file /.well-known/acme-challenge/xeuu_00nH2LvuuwokR1wEP7lSmNxlHPEg9dE0RSNHtk
[DBUG] Refreshing authorization (1/15)
[DBUG] Send POST request to https://acme-staging-v02.api.letsencrypt.org/acme/chall-v3/60346301/DQ1_WA
[VERB] Request completed with status OK
[EROR] {
"type": "urn:ietf:params:acme:error:unauthorized",
"detail": "During secondary validation: Invalid response from http://a.cps4.me/.well-known/acme-challenge/xeuu_00nH2LvuuwokR1wEP7lSmNxlHPEg9dE0RSNHtk [<IP_ADDRESS>]: "\r\nNot Found\r"",
"status": 403
}
[EROR] Authorization result: invalid
[VERB] Starting post-validation cleanup
[VERB] Post-validation cleanup was succesful

This looks like some requests are hitting the correct server but others are either blocked or routed elsewhere. This can be because of a load balancer, geofencing/limited whitelisting, multihomed ip?
For http validation to reliably work your web server has to publicaly accessible from anywhere. For self-hosting all requests need to hit the server wacs.exe is running on.

The server is behind a fw, one to one nat, i cant think of any reason that the webserver is not accessible from anywhere. We have live traffic to other domain on the same server, not seeing any issue accessing that from internet.
Any possibility of permission or server setting that may cause this issue?

It looks more like a network level issue to be honest because I can tell that one of the requests is handled just fine, it's the secondary validation (e.g. from another location) that fails. Any chance of multiple A records in your DNS?
You should try to figure out where the 403 error is coming from. Perhaps run wacs with --test and try it in your own browser or use https://letsdebug.net/

I think the firewall is giving a different server, thanks! I will check out the firewall settings instead


where fetched documents saved?

I set in configuration xml file the option of fetching documents by that

200
404
fetched_doc_header

but I didn't get any files downloads?? what is wrong here ?

See my answer to https://github.com/Norconex/collector-http/issues/257

Closing since duplicate of #257.


Two-way syncing?

I’m experimenting with the logitech syncing app. Is it possible to use it for syncing from Logitech to WLED, too? At the moment I only got the sync from WLED to Logitech working.

Thanks for checking out my app :)
No, syncing from Logitech to WLED is not possible right now, I'm afraid. The official Logitech RGB SDK that I'm using only allows taking full control. You can't intercept lighting effects from the G Hub and then synchronize WLED to that.
A solution to your use case might be OpenRGB. It supports the E1.31 protocol and various Logitech devices. You'd still be replacing G Hub but OpenRGB has similar effects.


Unable to add discoverable resource to My Resources

This may be a design decision, but it warrants reconsideration.  I am unable to add a discoverable resource to My Resources.  There is no green add to My Resources on the landing page interface.  The case in point is this resource https://www.hydroshare.org/resource/285a5b5714c549ac8bba269b79863c0a/
@horsburgh @mauriel133 @alvacouch could you please comment on whether we think a user should be able to put a discoverable resource in their My Resources.  If not, we can close this issue.  If yes, then this is an interface (and perhaps backend) issue to fix.

@dtarb - That resource is already shared with you and is already part of your my-resources page... Do you really want to see the green icon for a resources that is already part or your My Resources?
A user that has not had that resource shared with them will have the icon / option available. (image attached)

OK.  My mistake.  I'll close this.


Add playwright script to take screenshots

Takes screenshots of the login page, demo RouteList, and RouteActivity.
To run:
node src/ci/screenshots.cjs http://localhost:3000

Also made a follow up to add them to the PR comments: https://github.com/commaai/new-connect/pull/38

For https://github.com/commaai/new-connect/issues/36

Love how simple this is, though it does now introduce managing a separate Python environment now. Can we keep it this simple in JS?

is just links to the screenshot better? The PR looks cluttered

isn't just links to the screenshot better? The PR looks cluttered

This change doesn't post them to the PR comments, I just added them here to show the screenshots that the script produces. But yeah we could post links, or just make them smaller in the comment. Working on that in https://github.com/commaai/new-connect/pull/38

Got this even after bun install. Is it expected?
batman:new-connect$ node src/ci/screenshots.cjs http://localhost:3000
node:internal/process/promises:289
            triggerUncaughtException(err, true /* fromPromise */);
            ^

browserType.launch: Executable doesn't exist at /home/batman/.cache/ms-playwright/webkit-2003/pw_run.sh
╔═════════════════════════════════════════════════════════════════════════╗
║ Looks like Playwright Test or Playwright was just installed or updated. ║
║ Please run the following command to download new browsers:              ║
║                                                                         ║
║     npx playwright install                                              ║
║                                                                         ║
║ <3 Playwright Team                                                      ║
╚═════════════════════════════════════════════════════════════════════════╝
    at /home/batman/src/new-connect/src/ci/screenshots.cjs:24:39
    at Object.<anonymous> (/home/batman/src/new-connect/src/ci/screenshots.cjs:34:3) {
  name: 'Error'
}

Node.js v21.1.0

Got this even after bun install. Is it expected?
batman:new-connect$ node src/ci/screenshots.cjs http://localhost:3000
node:internal/process/promises:289
            triggerUncaughtException(err, true /* fromPromise */);
            ^

browserType.launch: Executable doesn't exist at /home/batman/.cache/ms-playwright/webkit-2003/pw_run.sh
╔═════════════════════════════════════════════════════════════════════════╗
║ Looks like Playwright Test or Playwright was just installed or updated. ║
║ Please run the following command to download new browsers:              ║
║                                                                         ║
║     npx playwright install                                              ║
║                                                                         ║
║ <3 Playwright Team                                                      ║
╚═════════════════════════════════════════════════════════════════════════╝
    at /home/batman/src/new-connect/src/ci/screenshots.cjs:24:39
    at Object.<anonymous> (/home/batman/src/new-connect/src/ci/screenshots.cjs:34:3) {
  name: 'Error'
}

Node.js v21.1.0

Ah yeah need to install the browser bun playwright install webkit


Wine Gecko reports a very old user-agent, which blocks the Adobe login page

This affects all products.
The latest wine gecko for wine 8.4 reports a really old user agent, which makes the Adobe login url show this message:

So although now we can run adobe applications in windows very easily, we can't login to acquire license.
This is the user-agent wine-gecko returns:

It may be possible to login by faking wine-gecko user-agent to something newer, so adobe will send the correct login page. Although if the login page uses features not supported by wine-gecko, this won't work.
Another solution is replace wine-gecko by something more up to date. Maybe try to install MS Edge on wine? Anyone?

I tried using the following registry to fix the problem as per instructions in this tutorial, but no luck.


Quest Lore doesnt keep color in GUI

2.9.0

Whenever i hover over the item i made as the GUI part of a quest, the Lore (ask message) does not keep the color all the way through.

Here is an example screenshot of what I am trying to describe (notice the lore color changes to green):
https://preview.ibb.co/ehaNnv/2017_07_16_14_22_14.png
If you would like to test and see a live version of the issue, feel free to on my server:
AomReloaded.mcpro.co

Iirc this is intentional to keep things consistent, but I hope to take a look soon.

Alright, thanks! Hope you can get to it soon.

Please give this build a try and let me know the results (you'll need to recreate the GUI item)

Closing due to lack of response from OP.

This is still an issue in 3.3.7 build #483.


completely random crashes when playing minecraft

this issue has been going on for such a long period of time, and finally after 2-3 years, an actual crash result appeared.
[11:00:28] [Render thread/INFO]: OpenGL debug message: id=0, source=API, type=ERROR, severity=HIGH, message='Unknown internal debug message. The NVIDIA OpenGL driver has encountered
an out of memory error. This application might
behave inconsistently and fail.
(pid=7784 javaw.exe 64bit)'
[11:00:28] [Render thread/INFO]: OpenGL debug message: id=1285, source=API, type=ERROR, severity=HIGH, message='GL_OUT_OF_MEMORY error generated. Failed to allocate memory for buffer object.'
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  EXCEPTION_ACCESS_VIOLATION (0xc0000005) at pc=0x00007ff88eda1991, pid=7784, tid=8920
#
# JRE version: OpenJDK Runtime Environment Temurin-17.0.2+8 (17.0.2+8) (build 17.0.2+8)
# Java VM: OpenJDK 64-Bit Server VM Temurin-17.0.2+8 (17.0.2+8, mixed mode, sharing, tiered, compressed oops, compressed class ptrs, shenandoah gc, windows-amd64)
# Problematic frame:
# C  [nvoglv64.dll+0x861991]
#
# No core dump will be written. Minidumps are not enabled by default on client versions of Windows
#
# An error report file with more information is saved as:
# D:\Geek Squad Data Backup 3-4-14\Desktop2018\NPC\MultiMC\instances\Minecraft Beta 1.18 [Fabric] [aristois]\.minecraft\hs_err_pid7784.log
#
# If you would like to submit a bug report, please visit:
#   https://github.com/adoptium/adoptium-support/issues
# The crash happened outside the Java Virtual Machine in native code.
# See problematic frame for where to report the bug.
#
AL lib: (EE) alc_cleanup: 2 devices not closed

log here:
hs_err_pid7784.log

Hi there,
You'll need to report this to Mojang.  I'll not you're also using a Beta 1.18 server.

Hi there,
You'll need to report this to Mojang. I'll not you're also using a Beta 1.18 server.

I'm sort of confused about the 1.18 beta server since this is on a client, but for some reason I can't report this to Mojang for whatever reason. There's no way for me to create an account. Here: https://bugs.mojang.com/Signup!default.jspa

Hi there,
You'll need to report this to Mojang. I'll not you're also using a Beta 1.18 server.

I'm sort of confused about the 1.18 beta server since this is on a client, but for some reason I can't report this to Mojang for whatever reason. There's no way for me to create an account. Here: https://bugs.mojang.com/Signup!default.jspa

I think you want https://bugs.mojang.com/register/

Hi there,
You'll need to report this to Mojang. I'll not you're also using a Beta 1.18 server.

I'm sort of confused about the 1.18 beta server since this is on a client, but for some reason I can't report this to Mojang for whatever reason. There's no way for me to create an account. Here: https://bugs.mojang.com/Signup!default.jspa

I think you want https://bugs.mojang.com/register/

sorry for the late reply, but thank u so much. ive been having this issue for so long. ill go ahead n attempt to do it in vanilla and report the issue.


NodaTime plugin

With the new NodaTime plugin coming to Npgsql 3.3, we need to create the corresponding plugin for the EF Core provider (depends on #300).. It would provide a db context options extension, UseNodaTime(), which would do the following:

Add the Npgsql NodaTime plugin to the global type mapper
Add the appropriate RelationalTypeMapping to the NpgsqlTypeMappingSource
Add the appropriate translators for NodaTime operations

I've just tried using this alongside an experimental plugin for some custom types. It seems they can't coexist. If I do:
.UseNpgsql("Host=...", options => options.UseNodaTime().UseMyPlugin())

Then I get an error:
The property 'MyEntity.MyType' could not be mapped, because it is of type 'MyType' which is not a supported primitive type or a valid entity type.

If I remove NodaTime's types I can successful use my plugin, or if I remove my types I can successfully use the NodaTime plugin. But I can't use both at the same time.
I should be able to provide some more information if needed, but I just wanted to see if this rung any bells first.

Thanks for pointing this out and especially for testing this! I can see it happening here and will investigate.


Deliver keys towards the APP verifier - implementation

Hi everyone,
first of all, thank you for your valuable work which certainly makes development easier for the Member States.
Looking at the implementation of the APIs in the verifier-service invoked by the verifier-apps, we can see:

/signercertificateStatus

/signercertificateUpdate

I noticed that the first API (/signercertificateStatus) is invoked by the client to understand which are the valid KIDs and possibly discard/delete the certificates whose KIDs are no longer in that specific list. I would say everything very clear.
when instead the second API (/signercertificateUpdate) is called, the client invokes the service for the first time without any X_RESUME_TOKEN_HEADER  in the header, while the following times the client uses the X_RESUME_TOKEN_HEADER that is returned by the same API (similar to the batch_tag logic used for the efgs).
Here two doubts arise:
1- Why does the app have to invoke the  /signercertificateUpdate with a X_RESUME_TOKEN_HEADER instead of using the same KID returned by the /signercertificateStatus API?
2- Wouldn't it be reasonable to invoke the API in this sequence? :

the app first invokes the /signercertificateStatus service and gets the list of all valid certificates (as already implemented);
the app checks from this list which certificates are no longer available (revoked) and how many of these certificates in the list are new (new KIDs).
At this point the app calls the /signercertificateUpdate API passing only the KIDs of the new certificates without using the RESUME_TOKEN.

Couldn't this logic be simpler to implement? Why use the resume token when we already have the KID?
Why use the request header rather than the request param? the second would be better for countries that will use a CDN.
thanks in advance for your reply!
regards
Alfonso

@alfo1995
The logic behind is, that the verifier app is downloading all new certificates. Means everything newer than Resume Token X (just the deltas of the previous). This works for new and updated certificates, but not for deleted certificates. So the signercertificateStatus crosschecks the deleted one.
Regarding your proposed sequence, the second point is the problem. The app can not check for revocation. This should the backend do here, because we have signer certificates which have maybe no public revocation list (revocation is still a open discussion point in the IOP group).
Regarding the CDN point:
Would it be possible for you, to just use the status call and create then your CDN links like /cdn/{kid} ?
The sequence would be then:

do signerCertificateUpdate (get kid list back), remove all which are not on this list internally.
Option 1: Call static routes in a loop /signercertificateUpdate/{kid} (CDN Way)
Option 2: Do the normal Update over /signercertificateUpdate

If yes, then we can insert in a later Version after 1.0 a /signercertificateUpdate/{kid} route, which you can use together with the status call (or you create directly a pull request for the app and the verifier).
Let me know that you think about it:)

Thanks for your reply @SchulzeStTSI, maybe you misunderstood my second point:

the app checks from this list which certificates are no longer available (revoked) and how many of these certificates in the list are new (new KIDs).

I meant, that the verifier app after invoking the /signercertificateStatus and getting the list of valid KIDs is able to discard the KIDs that are no longer valid simply by comparing the list it already has in the storage with the new one downloaded (nothing new is the mechanism that already exists).
Anyway, we followed faithfully your implementation and we used a purging cached mechanism each time we download the new certificates from your server. In this way, the data served by both the API /signercertificateStatus and /signercertificateUpdate are always refreshed once a day.
Best,
Alfonso

Hi @alfo1995,
once again thanks for the input. We may consider your ideas in a later version as SchulzeStTSI mentioned, but for now as there is no activity in this issue or urgent need to change, I will close this issue. Feel free to reopen it or create a new one, if it is needed.
Best regards
Simon


Fix LoadBorrowImmutabilityAnalysis in cases it raises false errors on destroy_value

We should only flag an error if there is a destroy_value on the first owned root of the access path.

@swift-ci test

@swift-ci smoke test and merge

@swift-ci smoke test macOS platform


Return full errors in response bodies in torii

Currently probably due to migration to thiserror and eyre, our torii error return bodies stopped giving back full error chain.
Only specifying #[source] there does not seem to be enough as is demonstrated by this playground. It seems we need to add where appropriate {0} or other formatting specifiers to error descriptions, to have full chain in bodies.

Closing until an MWE is found to reproduce the issue.


Review Slack WebHooks support

I've just noticed that Gogs supports and suggest using two different kind of WebHooks. One is Gogs, which is basically a POST request. The other one is Slack, that's an integration with "slack.com" service.
It looks like "slack.com" is a non-free team communication tool. Wikipedia lists it as Freeware. I've never heard of that website before, so I might be wrong. However, I urge you to check this issue and confirm whether Slack is indeed free or not. If it's not, please remove it from Gogs as soon as possible.
Thanks

Slack has free plan.

If it's not, please remove it from Gogs as soon as possible

Why should Gogs?

@aPeer You are correct that slack is non-free software, however the WebHook is just integration with their API so there is no reason for that not to be in Gogs. If you don't want to setup that WebHook you don't have to.
Gogs should have as many WebHook options as possible to integrate with as many services as possible (both free and non-free) so it can integrate into the user's workflow no matter what systems they choose to use.

@matt-h I'm really sorry, but that's not a valid argument. I don't want to opt-out of private software. It just doesn't have to be there. If you or anybody else want to use non-free software, then just fork Gogs and add your stuff there. But please don't pollute an otherwise free project. Gogs can support as many services as it wants to, as long as they are free. Currently it's suggesting to use a non-free service.. Again, if you want to use a non-free option in your workflow, that's your own business. So, please take care of maintaining a separte fork for yourself, and don't put the burden on other people.
@Unknwon I'm not talking about their "free plan". Free plan just means gratis; that is you don't have to pay in order to use it. I'm talking about freedom, free software. I'm asking you to take a strong position here: if Gogs is free software, then please remove Slack integration, otherwise you should write clearly on your README that Gogs is integrated with non-free software. Thank you.

@aPeer Gogs doesn't force you use Slack integration, your freedom to choose not use it, or fork Gogs to delete it. Thank you.

As @Unknwon said, you're free to do whatever you want with free software, including using it with non-free software. Do you consider Adium or Pidgin to not be free software? Once compiled, gogs also interfaces (Statically!) with the Win32 API, which is not free, and by analogy it would be a shame to drop support for such operating system, on the sole basis that it is not free.
You said that "Gogs is integrated with non-free software", IMO it's quite the opposite. Gogs integrates / will integrate implementations of connectors to various third party software. It doesn't make it in any way a subsidiary of those third party services.
I agree on the fact that all the connectors / third party providers supported by Gogs should be in the Readme, because it will make people aware of their options.

For now we have removed support for slack in the fork we run at notabug.org. Would you be open to a patch that added configuration option to gogs.ini along the lines of 'disable proprietary third party extensions' or something? I'd like to keep the nab fork of gogs as close to main upstream as I can but we seem to have slightly different goals. :)

I think it is fine if free projects support non-free webhooks, as long as you have to enable it or sign up. Providing that the code you're using to integrate is free, ie: just sending POST calls after being enabled rather than including code from them.

@hpvb

Would you be open to a patch that added configuration option to gogs.ini along the lines of 'disable proprietary third party extensions' or something?

Mostly like not.

@Unknwon OK. Sorry to bother you with this then :) it'll make it a little harder for us to directly send you pull requests from our fork but I'm sure we can figure it out.

@hpvb no problem! But don't send new feature pull request without discussing wits us on https://gitter.im/gogits/gogs :)

@Unknwon It's why I'm in there! :D

Thanks for all participants!
Few words before closing this issue:
Gogs core team's standpoint is very clear, we will not remove Slack webhook options(or any other future hook integrations) as HTTP POST actions. When HTTP POST is no longer free, that would be a good time to remove it.
:eyes::eyes::eyes:

I wanted to add that Mattermost exist and is an open source slack alternative written in Go and it rocks. It supports Slack webhooks so supporting Slack webhooks is supporting Mattermost web hooks!
I think it's fine!


Clickhouse models that ingest data from external sources using clickhouse SQL should materialize to true by default

We can only mark models materialized by default for cases when input and output connector are different.
But for clickhouse since we rely on clickhouse connectors to ingest all data it is not possible for us to set that default.
This leads to very hard to know issues where clickhouse keeps pulling data from external source for every query.
The same issue will also be present in duckDB when we move to its internal connectors.

I wonder if there's a way we can analyze a query for external dependencies? For example in DuckDB, maybe creating a temporary view with SET enable_external_access=false?

The same issue will also be present in duckDB when we move to its internal connectors.

For backwards compatibility at least, this can be solved by having type: source emit a model where materialize: true.

For example in DuckDB, maybe creating a temporary view with SET enable_external_access=false

Yes this can be evaluated for duckdb. Though enable_external_access once set to false can't be set to true and is a global configuration. We may need to do throw away the write handle once we do this check.


Better tqdm wrapper

This PR aligns the tqdm logic with huggingface_hub (without introducing breaking changes), as the current one is error-prone.
Additionally, it improves the doc page about the datasets' utilities, and the handling of fsspec paths in cached_path, is_local_path, is_remote_path, is_relative_path and relative_to_aboluste_path.
Fix #6409

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Network as provider while mocking using an application

Hello ,
We are facing issue while getting geolocation using network as provider if mock on android device is enabled.
following screen shots while doing that
mocking is disabled and same location is used.

mocking is enabled here.

why this is happening even we are getting information from network not from gps device embedded on the phone.
thanks,
Omar

Hi Omar, I'm not sure I understand the issue you are having? You can always reject non-mock locations by verifying where they come from: https://developer.android.com/reference/android/location/Location#isFromMockProvider().
Android devices will send network and gps location information unless those providers have been disabled by the user, operating system, or thru code.

Closing.


Profile permissions

What's this PR do?
Fixes issues with profile permissions.
How should this be reviewed?
Check that changes work as expected!
Checklist

[x] Documentation added for changed endpoints.
[x] Tests added for new features/bug fixes.

For review: @angaither @weerd

👍

cool!


send lbry_id for user me call

@seanyesmunt I added some information to the user/me api call so we can link the node_id and the user_id for rewards. Can you double check that I did this correctly? I think there may be something wrong because the app is not authenticating properly.

actually wrong area. disregard.


[SPARK-22408][SQL] RelationalGroupedDataset's distinct pivot value calculation launches unnecessary stages

What changes were proposed in this pull request?
Adding a global limit on top of the distinct values before sorting and collecting will reduce the overall work in the case where we have more distinct values. We will also eagerly perform a collect rather than a take because we know we only have at most (maxValues + 1) rows.
How was this patch tested?
Existing tests cover sorted order

@rxin @aray since you were the last few to touch around here.

Can one of the admins verify this patch?

diff LGTM

Jenkins, test this please.

Test build #3971 has started for PR 19629 at commit aa809e3.

Test build #3971 has finished for PR 19629 at commit aa809e3.

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

Merging in master.


Update documentation using model.h of bazel

I would like to run a trained random forest using c++ and I was following the documentation: https://ydf.readthedocs.io/en/stable/tutorial/cpp/#generate-the-c-code
The “simple” example just mentions that one would have to add some dependencies to bazel, but no mansion on where these dependencies come from. I was not able to get this example up and running because I couldn’t get the dependencies to line up so I and probably others could benefit from an updated documentation, with maybe a fully contained bazel file.

I think you can patch the YDF standalone example at https://github.com/google/yggdrasil-decision-forests/tree/main/examples/standalone and replace the Bazel file with
package(
    default_visibility = ["//visibility:public"],
    licenses = ["notice"],
)

cc_library(
    name = "ydf_tutorial_model",
    hdrs = ["ydf_tutorial_model.h"],
    deps = [
        "@com_google_absl//absl/strings",
        "@com_google_absl//absl/status:statusor",
        "@ydf//yggdrasil_decision_forests/api:serving",
    ],
)

In ydf_tutorial_model.h, you'll have to replace #include "external/ydf_cc/yggdrasil_decision_forests/api/serving.h" with #include "yggdrasil_decision_forests/api/serving.h"
This will compile the library to run the model (I tested it quickly with Bazel 5.3.0 and the compile options in our .bazelrc, so roughly
bazel build --cxxopt=-std=c++17 --host_cxxopt=-std=c++17  --define=use_fast_cpp_protos=true  --define=allow_oversize_protos=true  --noincompatible_strict_action_env  --define=use_ydf_tensorflow_proto=1 //:ydf_tutorial_model

Of course, this will only compile a library, you'll still need to implement a main() to actually call this library

O.K. thanks I was now able to build it and my main.cpp looks like this:
#include <bits/stdc++.h>
#include "tf_model.h"

namespace ydf = yggdrasil_decision_forests;

int main() {
  auto model =
     ydf::exported_model_tf_model::Load("/home/heimchen/Documents/Programming/CPPModel/tf_model");

  if(model.ok()){
    const std::vector<float> result = model->Predict();
    for(auto v : result){
      std::cout << v << " ";
    }
    std::cout << std::endl;
  }

  return 0;
}

The problem now is, that running the ./main results in this error:
2024-11-26 10:42:05.647879: I tensorflow/cc/saved_model/reader.cc:83] Reading SavedModel from: ./tf_model
2024-11-26 10:42:05.647931: I tensorflow/cc/saved_model/loader.cc:466] SavedModel load for tags { serve }; Status: fail: NOT_FOUND: Could not find SavedModel .pb or .pbtxt at supplied export directory path: ./tf_model. Check that the directory exists and that you have the right permissions for accessing it.. Took 61 microseconds.
Error loading model: Could not find SavedModel .pb or .pbtxt at supplied export directory path: ./tf_model. Check that the directory exists and that you have the right permissions for accessing it.
Error: Failed to load model

Which changes to
2024-11-26 10:44:30.262314: I tensorflow/cc/saved_model/reader.cc:83] Reading SavedModel from: ./tf_model
2024-11-26 10:44:30.262413: I tensorflow/cc/saved_model/reader.cc:52] Reading meta graph with tags { serve }
2024-11-26 10:44:30.262424: I tensorflow/cc/saved_model/loader.cc:466] SavedModel load for tags { serve }; Status: fail: NOT_FOUND: Could not find meta graph def matching supplied tags: { serve }. To inspect available tag-sets in the SavedModel, please use the SavedModel CLI: `saved_model_cli`. Took 121 microseconds.
Error loading model: Could not find meta graph def matching supplied tags: { serve }. To inspect available tag-sets in the SavedModel, please use the SavedModel CLI: `saved_model_cli`
Error: Failed to load model

When renaming the header.pb -> saved_model.pb file in the tf_model directory so I guess the code is searching for a file that is not included in the exported model?
I am saving the model from python with:
model.save("tf_model")
with open("tf_model.h", "w") as f:
  f.write(model.to_cpp(key="tf_model"))

I think in this case the problem is that the given code just tries to run a tensorflow model, as the build in save function, does not work, but the one that converts to tensorflow.
# Does not work
model.save("tf_model")
# Does work
model.to_tensorflow_saved_model("tf_model", mode="tf")

As this now tries to run as tensorflow, it also has problems with the custom functions yggdrasil uses.
Is this intended behavior? It is never stated directly in the documentation but I got the impression that could run as standalone.


Dragging scrollbar also moves grid item in Firefox

Hi there,
Problem in Firefox:
Some of my grid-items are actually larger than their containers, so I've set overflow:auto on them. The scrollbars don't seem to work well in this CodeSandbox:

So far so good. If I move the mouse into the first one and scroll with the mouse wheel, it scrolls as expected. But if I use the mouse and try to click on the scroll handle and drag it down a centimeter or two, it looks like this:

So the grid item did scroll, but it moved as well, which I didn't expect. Can I get vue-grid-layout to "play nice" with scrollbars?
"Play nice" is a little ill-defined, but at least I'm expecting that dragging the (blue) scroll handle in the scrollbar just scrolls.
Software versions

Browser: Firefox 67.0.4
Vue Version: 2.6.10
vue-grid-layout Version: 2.3.4

@pmorch Hello! Did you find a solution?
I have a similar problem. I have an input tag in the draggable area, and I can't select text using the mouse because of the cell is dragging while I try to select text inside the input tag

Hi,
What we ended up doing was specific to our problem. We're making a dashboard with some widgets. Each widget has a title area, and for application-specific purposes it made sense to only allow dragging the title area. It is a work in progress (the CSS/styling is likely to improve) but this is what it looks like today:

One can move the widget only by dragging the top bar. Then this problem is not an issue.
This was done by using
 dragAllowFrom=".widget-header-enclosure"

on the <grid-item/> - see the doc


Move invocation of initialize above setting of attributes

If initialize is used to initialize objects (provide defaults, etc) it should happen before the provided attributes are set on the object and validated, not after.
This attempts to resolve part of the second blocker in #73.

Current coverage is 85.22%

Merging #81 into master will increase coverage by +0.03% as of f2a68f9

@@            master     #81   diff @@
======================================
  Files           37      37
  Stmts         2796    2796
  Branches         0       0
  Methods          0       0
======================================
+ Hit           2382    2383     +1
  Partial          0       0
+ Missed         414     413     -1

Review entire Coverage Diff as of f2a68f9

Powered by Codecov. Updated on successful CI builds.


Problem when trying script routing rules example in Chapter 6.33 of Dubbo user manual

When I tried the  script routing rules example ,  the return value of the example script was always the value of "invokers".
The example script is as follow：
function route(invokers) {
    var result = new java.util.ArrayList(invokers.size());
    for (i = 0; i < invokers.size(); i ++) {
        if ("<IP_ADDRESS>".equals(invokers.get(i).getUrl().getHost())) {
            result.add(invokers.get(i));
        }
    }
    return result;
} (invokers); // Indicates that the method is executed immediately

According to the grammar of javascript, the meaning of the example script is :
function route(invokers) { ... } ;

(invokers)  // This is a single expression .The return value is always invokers.

instead of
function route(invokers) { ... } ;
route(invokers)

Actually, brackets surrounding the codeblock is simple solution.
(function route(invokers) { ... } (invokers) )

Hi, would you mind send a pull request?

@ralf0131 I have send a pull request #20 in incubator-dubbo-docs.


Move generated files to the generic models_index defined in graphql-server

Resolvers are using the old not generic index. Move all resolvers to the new global model_index.js defined in graphql-server. Also use it in utils/login.js and utils/join_models.js of the graphql-server.

Done


squid:S2142 -'InterruptedException' should not be ignored

This pull request is focused on resolving occurrences of Sonar rule
squid:S2142 -  "InterruptedException" should not be ignored.
You can find more information about the issue here:
InterruptedExceptions should never be ignored in the code, and simply logging the exception counts in this case as "ignoring". Instead, InterruptedExceptions should either be rethrown - immediately or after cleaning up the method's state - or the method should be reinterrupted. Any other course of action risks delaying thread shutdown and loses the information that the thread was interrupted - probably without finishing its task.
Noncompliant Code Example
public void run () {
  try {
   while (true) {
      // do stuff
    }
  }catch (InterruptedException e) { // Noncompliant; logging is not enough
    LOGGER.log(Level.WARN, "Interrupted!", e);
  }
}
Compliant Solution
public void run () throws InterruptedException{
  try {
   while (true) {
      // do stuff
    }
  }catch (InterruptedException e) {
    LOGGER.log(Level.WARN, "Interrupted!", e);
    // clean up state...
    throw e;
  }
}
or
public void run () {
  try {
   while (true) {
      // do stuff
    }
  }catch (InterruptedException e) {
    LOGGER.log(Level.WARN, "Interrupted!", e);
    // clean up state...
    Thread.currentThread().interrupt();
  }
}
Please let me know if you have any questions.
George Kankava

Will throw a new exception:
http://www.yegor256.com/2015/10/20/interrupted-exception.html


Feature/blacklist

Added blacklist feature:
A array that will be checked to see if the mutation type matches an item in the blacklist. If it matches it won't save that mutation. Defaults to []

Hi @AhadCove, thanks for submitting the PR. I understand your use case, but this can be easily added by implementing a custom filter.


Time Weight > 0 result in tank levels greater than maximum tank levels

Sorry, this was meant  for Epanet 3 issue.

Thank you!


 The problem is that some browsers will buffer the response because of the "text/plain" mime type

Mac chrome and safari both take several minutes to load. Shall I try windows IE?

Closing since this is really a browser issue. If you use curl, it works with no delays.

For others who might run into this issue in the future, try the Log Stream UI in Azure portal. It’s so much faster!!! Thanks @davidebbo! 👍

@davidebbo It's an issue in ALL browsers... doesn't it make it an issue for kudu indeed?? I've NEVER been able to acually see this console. I used to go to Azure UI as @ritazh suggests, but now I don't have access to it and I really need the kudu console, and it doesn't work. It's been like that for years now. Could you please kindly take a look at it and make it work?
Best
PS: my console hasn't loaded yet... still waiting..

@luismesagravedeperalta Please just use curl instead as explained above. We should probably just remove that option from the Kudu menu.


pages.updated_at should only be updated when a revision is made, or the page "Updated by" date should use the latest revision

Describe the bug
When a user changes the sort order of a page within a book or chapter, the pages.updated_at field is updated (the updated_by field is not updated).
When viewing the page in the browser, the pages.updated_at field is used to display when the page was last updates. After a sort, this will give the wrong impression of who and when the page was last updated.
In the below image, no page content was modified, and the displayed user "Stephen Hill" did not perform the sort.
Steps To Reproduce

User "Abby" creates a new book with two pages.
User "Bobby" sort/re-orders the pages within the book.
The page displays that it was updated by Abby, on the datetime Bobby sorted the book.

Expected behavior

The entity-meta.blade.php and entity-export-meta.blade.php templates should be updated to use the datetime and user of the latest page revision for rendering the "entities.meta_updated_name" string. This would give a clear indication of who actually performed a meaningful update to the page.
Performing a sort should not update the pages.updated_at field.

Screenshots
entity-meta.blade.php output after the page was sorted. The last revision was over a year ago.

Your Configuration (please complete the following information):

BookStack v0.26.2
PHP 7.2.4
Hosting Method: FreeBSD Jail, Nginx, PHP-FPM

Many Thanks
Stephen

Thanks for the request @stephen-hill-safe-uk.
I'd prefer not to base any dates off the revisions system, since there are options to prune and disable those. I can see why the current change to updated_at can cause confusion. I think it would be best to still have some indicator on the page-level of a order/location change, Maybe log activities against each item that's moved in a sort operation but leave the updated_at fields untouched?

I'm having this same issue. An employee who has since retired is showing as having last updated pages a couple weeks ago when his last revision to the page was months ago. As it turns out, all the pages in question are part of books that were recently sorted, which changed the updated_at timestamp and is now misleading.
At a minimum, I think the updated_at column shouldn't be updated when a book is sorted (or any non-user driven action to the page happens). If you still want an additional audit outside of the activities table to indicate when the page record was last modified, perhaps there should be a system_updated_at column or similar that can track any system-driven changes like sorting?

This is still reproducable with v23.10.4.
To reproduce:

Login as user A
Edit a page as user A
Login as user B
Sort the book, move the page

See that last updated got refreshed, showing last edited by user A and set to the time user B sorted the book.
This caused confusion, as collegues assumed they work on the same page at the same time.

We've continued to have this issue cause confusion with users. We store our scheduled job runbooks in BookStack, and someone thought that the instructions were recently modified because it had a new updated date in the sidebar. But, when looking at the page revisions (and we store all revisions with no purging), the page hadn't been modified in two years.
I think it makes sense to log the sort as an activity in the audit log and update the updated at date on the book itself when sorted, but all of the pages and chapters updated at should be left alone.

I also noticed and was confused by this. As I understand, the timestamp from a sort operation and the user from an edit operation are being munged together, resulting in an output like "Updated 2 hours ago by So and So" that isn't accurate.
Personally, I'd prefer the "Updated" label ignore sort updates and show the last revision. An easy fix might be simply dropping the user, like "Updated 2 hours ago," when updated_at doesn't match the last revision timestamp.


new reader

I'm new to the series and would like to start from beginning but it only goes back to volume 6 is there a way to see the rest

read manga version, or anime.

We have not and will not translate the previous volumes. See the official translations for that.


Login: IGResponseException: Please wait a few minutes before you try again.

Hey,
I've been getting an error during login for over a hour now
Login: IGResponseException: Please wait a few minutes before you try again.
I understand the error is due to limits being reached as I was always performing a new login every time I executed the application.
After I started getting the error, I implemented a logic to serialize a successful login and always try to deserialize the login when the application starts and only login if it's not possible to deserialize so this should prevent this from happening again, but do you have any ideia what's the limit? I would expect it wouldn't take more than 1 hour to refresh.

I am also getting same issue. In my local system it is working fine but when I deployed my application to azure with same credentials this issue pops up.

It may be due to IP address, I've heart that sometimes these limits are applied to IP addresses. However, I setup a VPN and my problem persists so who knows... anyway, my limit has refreshed, it took a little more than 24 hours, I believe, probably close to 26.


feat: last doc updates with comment addressed

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 5 committers have signed the CLA.:white_check_mark: snork-alt:white_check_mark: chloeminkyung:white_check_mark: abcpro1:white_check_mark: MrunmayS:x: mediuminvaderYou have signed the CLA already but the status is still pending? Let us recheck it.


TR-2295: "Windows Kerberos Local Successful Logon" Detection SPL

Splunk research says: "The target user security identified will be set to the built-in local Administrator account, along with the remote address as localhost - <IP_ADDRESS>."
Local admin account SID is something like "SID S-1-5-domain-500". Can we please add the following two key-value in the SPL to filter out FP:
Security_ID=S-1-5-*-500 AND Account_Name=Administrator

Hi @sahoo8920 ,
Apologies for the delay. Are you able to share some of the false positives?
From what we understand is that the way the UF works is it will perform a reverse lookup of the security_ID to username and it is one or the other.


Update deps

update tokio to 1.2
replace tokio-tls with tokio-native-tls
update async-tungstenite to 0.12
update reqwest to 0.11

All tests pass with the updated code.
Signed-off-by: Chris Lee<EMAIL_ADDRESS>

Thanks :)


[ZEPPELIN-6141] Update the link to the page where all issues are sorted

What is this PR for?
I recommend that clicking on Issues Tracking in the Zeppelin Web UI redirects to a page where all issues are sorted.
What type of PR is it?
Feature
Todos

[x] - README.md
[x] - zeppelin-web-angular/src/app/pages/workspace/home/home.component.html
[x] - zeppelin-web/src/app/home/home.html

What is the Jira issue?

https://issues.apache.org/jira/projects/ZEPPELIN/issues/ZEPPELIN-6141

How should this be tested?

Click Issues Tracking after build

Screenshots (if appropriate)
Zeppelin Web UI

Before

After

Questions:

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

Please review. @jongyoul

Merged into master/branch-0.12.


Removed igw and vpc_gateway_attachment from template

because it's only used in base_network.
I kept the other ones because they are frequently used by other stacks.


manage OneToNBroadcastConnector's operator's open and close status

output operators owned by a OneToNBroadcastConnector now have three status: pre-open, opened, and closed.
OneToNBroadcastConnector won't close the input operator until all its output operators have closed.

Current coverage is 60.28% (diff: 100%)

Merging #216 into master will not change coverage

@@             master       #216   diff @@
==========================================
  Files           114        114
  Lines          3004       3004
  Methods           0          0
  Messages          0          0
  Branches        492        492
==========================================
  Hits           1811       1811
  Misses         1075       1075
  Partials        118        118

Powered by Codecov. Last update d271b1b...4ed642d

LGTM


Add non runnable initialize and cleanup test methods

Fixes #1497
Adds AssemblyInitialize, AssemblyCleanup, ClassInitialize, ClassCleanup tests methods.
Run all tests

Run only one test => Class and Assembly test methods are also executed

Run non runnable tests only => Ignored

I'd like to review this more thoroughly, but I am not sure when I will be able to do that. Please don't close this too fast.


Mocking API with @searchable in schema fails

Describe the bug
My GraphQL schema uses the @searchable directive. This can't currently be mocked locally (by running amplify mock api) and the CLI tells me this, however it also reports that it will set a NONE data source as a work around for that but it still fails to run.
This prevents any mocking while @searchable is in the schema.
To around around this I've been commenting out @searchable from my schema and then runnning amplify mock api. I'm happy with not being able to test the elasticsearch part of my schema functionality local but it is stopping me testing any of it easily currently.
Amplify CLI Version
You can use amplify -v to check the amplify cli version on your system
v4.32.0
To Reproduce
Create a schema:
type Todo   @model   @searchable {   id: ID!   name: String! }
in the CLI run amplify mock api and watch it complain with something similar to the following:

Expected behavior
Couple of options I think:

Have the existing fix actually work, provide a dummy data source or whatever its doing and run that.
Just ignore the @searchable directive when run locally, warn us that it won't work and then skip over it, effectively "commenting it out" when you run by just ignoring it when parsing it locally in a mock.

Desktop (please complete the following information):

OS: Windows
Node Version. v12.18.4

@PeteDuncanson Yes, unfortunately @searchable is not currently supported with mock. But as you recommended, we can potentially ignore @searchable and do the same mocking with local DDB as we do without @searchable - but then the queries (with searching and sorting from the client) won't potentially work causing a discrepancy between what you see locally and what you see in the cloud. Marking this as an enhancement.

Is it possible to just get a fix that ignores @searchable when you run mock?  It makes no sense that I would need to remove it from the schema.graphql to run 'amplify mock', and then accidentally deploy ( which I did, and deleted the elasticsearch domain ).  These types of things have killed so much time developing with Amplify.

I'm getting to the point of writing a script that automatically comments/uncomments the @searchable fields from my schema at this point. Any update on if this might get looked at 6 months on?

Would be interested in such a script too @PeteDuncanson . Better would be an official solution from Amplify of course.

Another solution could be to allow creating separate GraphQL schemas per environment. Like that one could not only remove the @searchable directive in the dev schema to allow mocking, but also save those $30/month for ElasticSearch in the pushed dev environment.

@flogy can't you do that anyway? The schema is commited into source control, if you branch for live/dev then you would get that benefit?

@PeteDuncanson That is what I tried to do. I have a custom npm/yarn function that first comments out @searchable, then runs amplify mock, and then puts back searchable when the process exits.  But then the GQL API becomes out of whack, and breaks typescript in my app. So not really a good workaround.

Aren't these two duplicates? #4333

@AlessandroVol23 yes, I'm going to close this as a duplicate of #4333.
Also, we recently posted an RFC for @searchable improvements at https://github.com/aws-amplify/amplify-cli/issues/7546. Local mocking is one of the enhancements we are targeting.


you need another layer of recursion

this is a joke, sorry

:joy: I am having this issue when transforming pling to a python package, trying to understand if it is a snakemake issue or not


Feature/make columns globally configurable

As mentioned in discord, there is currently no way to configure table columns globally. With this pull request, you can make for example all table columns sortable and/or toggleable without copy+pasting it on all field definitions.
If this pull request is ok for you, I could do the same for form fields as well. Usecases:

All fields should be required by default.
All DatePicker fields should have a the central european format format('d.m.Y')
All fields should have a specific custom attribute
All Toggle fields should be inline.

Thanks, I love this. I've extracted it into a new "support" package which we share between packages. Looking forward to adding it to other objects, like form components!

#2229

Thank you for enhancing/merging and your great work.


Certain levels dont work with this level viewer

I have tried a few levels. two seem to work but most don't. I keep getting an Internal sever error or Level ID could not be downloaded. I tried both JiXiaomai's program and your website both result in errors.
Only two levels that seem to work
C81-8H4-RGG
SQG-9NT-9GF
Levels that don't work
5BJ-G6X-LSG
S0Q-MRG-5GG
FXY-QKH-41G
NJ0-0TF-4CG
XFP-RLF-MGF
XGM-H9L-0CG
LB7-LDG-KJG
BRG-477-VYG
0BH-621-HJF
MKV-L45-TTG

There was an outage yesterday that I believe I've fixed


Adds 'purview retentionlabel remove' command. Closes #4149

Adds purview retentionlabel remove command. Closes #4149

Hi @nanddeepn, could you fix the merge conflict? It's probably only the eslintrc file that's the problem due to my PR with the same change.

@martinlingstuyl
Resolved the conflicts. Thank you.

Thank you. Will review it shortly

Thank you. Manually merged


[WIP] Add service workers to content publisher

Experimental work to improve the performance of content publisher. Also potentially add offline capabilities.
https://trello.com/c/cN83jHLw/61-implement-service-workers-on-content-publisher-app

@DilwoarH this PR has been here for quite a while now. Are you planning to do any more work on it?

@DilwoarH this PR has been here for quite a while now. Are you planning to do any more work on it?

Hi @benthorner - I do plan to work on it at some point. I'm happy to close and can reopen when it's ready?

@DilwoarH that would be helpful. This means it's easier for us to see which PRs we need to review.


Change "add" button foreground color

Partially fixes #997

The 'f' should be black in dark mode (see Issue #997)
Description of the changes:

Don't modify EquationButton.Foreground when the EquationBox is a "add an equation" placeholder.

How changes were validated:

manually in Light, Dark and High contrast mode

Abandoned


beutify readme, fuggly uptown style

@Mottie
I played a little with negative spaces there, maybe Ill do that for a revamped Stylish logo too
preview https://github.com/un1versal/GitHub-Dark-Script/blob/Born-to-Be-Blue/README.md

Nice! Umm, but wouldn't the brackets not look all stretched out on the bottom half?

BTW, can you export your work images SVG and link to those instead? Makes it much easier to edit and works better on Retina or Retina-like displays :)
You can submit them to https://github.com/StylishThemes/logo and then link to the raw files from that repo ideally :)

BTW, can you export your images to SVG and link to those instead? Makes it much easier to edit and works better on Retina or Retina-like displays :)

I might, not atm though its a pita GitHub dont let you do relative links with svg's

Nice!

Oh btw dont merge it, just wanted to show it. atm,

unless theres any comments this is OK to go

Thanks!


fix(compiler): bring back compiled output with inline sourcemap

Summary

Fixes #3482
Test plan

Check if breakpoints are hit in JetBrains IDE
Check if VS Code doesn't print errors.

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
N.A.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 16 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 94.916%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
1127

Relevant Lines:
1153

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

16 of 16 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 94.916%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
1127

Relevant Lines:
1153

💛  - Coveralls


Create basic-calculator-js

calculator_viswanath akash(viswakash)

@viswakash put your code in a folder with the name _ to get this PR merged. Thank you.


ImportError: cannot import name 'quote' from 'pytube.compat'

Whenever I enter pytube into command prompt, I get the following error:
Traceback (most recent call last):
  File "c:\program files\python\installations\python38\lib\runpy.py", line 193, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "c:\program files\python\installations\python38\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "C:\Users\boob\AppData\Roaming\Python\Python38\Scripts\pytube.exe\__main__.py", line 4, in <module>
  File "C:\Users\boob\AppData\Roaming\Python\Python38\site-packages\pytube\__init__.py", line 16, in <module>
    from pytube.streams import Stream
  File "C:\Users\boob\AppData\Roaming\Python\Python38\site-packages\pytube\streams.py", line 17, in <module>
    from pytube import extract
  File "C:\Users\boob\AppData\Roaming\Python\Python38\site-packages\pytube\extract.py", line 7, in <module>
    from pytube.compat import quote
ImportError: cannot import name 'quote' from 'pytube.compat' (C:\Users\boob\AppData\Roaming\Python\Python38\site-packages\pytube\compat.py)

@Perzan
Try "pip install pytube3"

@Perzan
Try "pip install pytube3"

I had the same issue. This corrected it.

its worked for me too

It's working for me as well. But run
pip uninstall pytube
before running pip install pytube3

Don't install pytube3, it's not build from this repository. To install from the head of this repository:
pip install git+https://github.com/nficano/pytube.git

not close yet?which is correct?

@Bouke  @jiangno111 @SaranyaKesavan @aquilesics @ngen-brett @JakobLewis  @Perzan
Can you try again and let me know if you're still encountering the issue?
Also pytube3 is not related to  Pytube -- they are different libraries.
Thanks

@Bouke @jiangno111 @SaranyaKesavan @aquilesics @ngen-brett @JakobLewis @Perzan
Can you try again and let me know if you're still encountering the issue?
Also pytube3 is not related to Pytube -- they are different libraries.
Thanks

The pip/git install worked

There is a more recent issue regarding python 2 support here, so I'm going to close this issue


closure-stylesheets.jar missing?

@seanmfoy There was a closure-stylesheets.jar for 1.5.0 but it seems to be missing now. Was there a problem with it?

No problem, it was merely delayed. The jars are posted now.


Improve docs on the ingress module

Closes #61.

Clarification:
I had written feedback about reverting buffer -> buf changes.
I don't particularly care which variable name is used, so long as it's consistent with the rest of the examples in questdb-rs/examples as well.

I had a specific reason to use buf instead of buffer --  making the construct sender.flush(&mut buffer) a bit shorter, as I was going to use it in sentences whenever referring to flush. But in the end, I use it just once, so I'll revert to buffer.

cargo test --no-default-features doesn't compile on either main or this branch.

Sorry. Correct. Need to enable at least webpki-roots.

test_tls_ca_webpki_and_os_roots is failing with
LineProtocolException: [-1] table: 77836915f4494765acbd56ec18f6dd7e, column: a; cast error from protocol type: TAG to column type: STRING

My guess is that there's something wrong with the isolation between tests. I suppose this test should create its own table, but it seems to already exist and there's a type clash on column a.

Working on the docs, I noticed that line_sender.hpp doesn't declare flush_and_keep_with_flags, so it seems you can't ensure a transactional flush from C++.


Changes to enable debugging for all microservices using "Local process"

@daniv-msft  Can you please take a look? Thanks!


Czml path sandcastle example change

The CZML docs and sandcastle were a little confusing here.  I removed the polylineGlow property from the path material to reduce confusion in the example, since it wasn't being applied.

Thanks @denverpierce!


Question: How to set and use readyPromise ?

Disclaimer: I am not a JS, React developer, meaning I ignore many things of this world.
I have seen this sandbox : https://codesandbox.io/s/excalidraw-ehlz3?file=/src/App.js
I have seen the documentation of the ref property here :

https://github.com/excalidraw/excalidraw/tree/master/src/packages/excalidraw#props
https://github.com/excalidraw/excalidraw/tree/master/src/packages/excalidraw#ref

Which links to this code snippet : https://github.com/excalidraw/excalidraw/blob/master/src/utils.ts#L343-L357
And there's also the readyPromise property which I don't quite get
https://github.com/excalidraw/excalidraw/tree/master/src/packages/excalidraw#readypromise
But when declaring the promise this way didn't work :
import React from "react";
import Excalidraw, {
  ...
} from "@excalidraw/excalidraw";

export default function App() {
    const excalidrawRef = React.useRef(null);

    ...

    return (
        <div className="excalidraw-wrapper">
            <Excalidraw
                ref={excalidrawRef}
                onChange={(elements, state) =>
                    console.log("Elements :", elements, "State : ", state)
                }
                onCollabButtonClick={() =>
                    window.alert("Not supported")
                }
                readyPromise={() => {
                    console.log("excalidraw ready")
                }}
                initialData={{ appState: {
                        exportEmbedScene: true
                    }
                }}
                UIOptions={{canvasActions: {loadScene: false}}}
            />
        </div>
    );
}

I expected the lambda function to be called when Excalidraw is loaded and ready. But it didn't happen. There's something that's eluding me.
Thank you in advance for shedding the light on this!

Closed as question is moved over discussion https://github.com/excalidraw/excalidraw/discussions/3756


Memory requirements

Can I know the memory requirements for running this, I keep running into an unexpected bus error even when I use (8GB RAM + 24GB Swap file), also the swapfile does not even get fully occupied when the error occurs. I am using a cpu version of torch to run this.
=> loading checkpoint 'model_e220_v-4.700.pth.tar'
=> loaded checkpoint 'model_e220_v-4.700.pth.tar' (epoch 220)
/home/akhil/anaconda2/envs/im2recipe/lib/python2.7/site-packages/torchvision-0.2.1-py2.7.egg/torchvision/transforms/transforms.py:188: UserWarning: The use of the transforms.Scale transform is deprecated, please use transforms.Resize instead.
Test loader prepared.
321
i
0
321
test.py:110: UserWarning: volatile was removed and now has no effect. Use with torch.no_grad(): instead.
v = torch.autograd.Variable(input[j], volatile=True)
test.py:116: UserWarning: volatile was removed and now has no effect. Use with torch.no_grad(): instead.
v = torch.autograd.Variable(target[j], volatile=True)
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
Exception NameError: "global name 'FileNotFoundError' is not defined" in <bound method _DataLoaderIter.del of <torch.utils.data.dataloader._DataLoaderIter object at 0x7f97abe148d0>> ignored
Traceback (most recent call last):
File "test.py", line 199, in
main()
File "test.py", line 90, in main
test(test_loader, model, criterion)
File "test.py", line 127, in test
output = model(input_var[0],input_var[1], input_var[2], input_var[3], input_var[4])
File "/home/akhil/anaconda2/envs/im2recipe/lib/python2.7/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "/home/akhil/Documents/im2recipe/im2recipe-Pytorch/trijoint.py", line 134, in forward
visual_emb = self.visionMLP(x)
File "/home/akhil/anaconda2/envs/im2recipe/lib/python2.7/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "/home/akhil/anaconda2/envs/im2recipe/lib/python2.7/site-packages/torch/nn/parallel/data_parallel.py", line 109, in forward
return self.module(*inputs, **kwargs)
File "/home/akhil/anaconda2/envs/im2recipe/lib/python2.7/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "/home/akhil/anaconda2/envs/im2recipe/lib/python2.7/site-packages/torch/nn/modules/container.py", line 91, in forward
input = module(input)
File "/home/akhil/anaconda2/envs/im2recipe/lib/python2.7/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "/home/akhil/anaconda2/envs/im2recipe/lib/python2.7/site-packages/torch/nn/modules/container.py", line 91, in forward
input = module(input)
File "/home/akhil/anaconda2/envs/im2recipe/lib/python2.7/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "/home/akhil/anaconda2/envs/im2recipe/lib/python2.7/site-packages/torchvision-0.2.1-py2.7.egg/torchvision/models/resnet.py", line 76, in forward
File "/home/akhil/anaconda2/envs/im2recipe/lib/python2.7/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "/home/akhil/anaconda2/envs/im2recipe/lib/python2.7/site-packages/torch/nn/modules/conv.py", line 301, in forward
self.padding, self.dilation, self.groups)
File "/home/akhil/anaconda2/envs/im2recipe/lib/python2.7/site-packages/torch/utils/data/dataloader.py", line 178, in handler
_error_if_any_worker_fails()
RuntimeError: DataLoader worker (pid 12487) is killed by signal: Bus error.

Hi @makhilbabu, thanks for bringing this up. I can't say for sure what the exact memory requirements are but 32 GB should be around what you need. You can probably get away with reducing the number of workers, though. Swap won't be very useful when you're trying to share memory between processes.
And, of course you should take the advice of PyTorch and use with torch.no_grad (or use PyTorch v0.3.*)

Thank you, one more thing, do you know how many cores the command python test.py --model_path=snapshots/model_e220_v-4.700.pth.tar uses by default when run on cuda, does it depend on number of workers set?

does it depend on number of workers set?

Yes, this is the only variable that affects number of CPUs used when running on CUDA.


Scatter: Avoid evaluation within a symbolic scope

When performing scatters within symbolic loops, there is already an implied precondition that writes are non-conflicting across iterations.
So if a user want to call scatter multiple times in the loop body the same precondition should be assumed.

Hi Rami,
thinking more about this, I think it is good to try to evaluate when possible. For example,
dr.scatter(x, ...)
while ....:
    dr.scatter(x, ..)

Here, we can do the first scatter outside of the symbolic region when getting to the second one inside. Can you modify the approach so that the scatters do not raise an error message if evaluating fails, and if we're in a symbolic scope?

Thanks Wenzel, that's a good point. I hadn't considered that. I've updated the PR based on your suggestion. I was planning on adding some Dr.Jit python tests related to this but I've also added a (crude) test here to check the expected behaviour of the kernel splitting for scatters to the same target outside and within the loop body.


Ci sage update

Summary of changes

Update the Sage portability CI: Fix gentoo, drop centos-8, add centos-stream-8, centos-stream-9
Pull Request Checklist

[ ] Changes have tests
[ ] News fragment added in changelog.d/.
(See documentation for details)

Thanks Matthias!


Update yt-dlp to version 2024.09.27

This is an automated Pull Request to update the yt-dlp versions that is used in the docker image.

Merged to dev branch due to a requirement change on the python version of yt-dlp.


Run modkit once + general changes to methylation subworkflow

Closes #385.
Changes:

Added phased input to METHYLATION subworkflow
Only call modkit once, with parameters depending on whether phasing was run
Moved subworkflow to own its own directory
Added tests

PR checklist

[x] This comment contains a description of changes (with reason).
[x] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[x] Make sure your code lints (nf-core pipelines lint).
[x] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[x] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Let's add it to the list of tests to run in the CI as well: https://github.com/genomic-medicine-sweden/nallo/blob/dev/.github/workflows/ci.yml.

I don't believe methylation tags is being copied over during alignment, could you check the output files you get when running the tests and see if they look reasonable?

I added the flag you suggested and at least now the BAM files have MM and ML tags and modkit's output looks ok, I guess.


Fix invocation of go vet in Makefile

go tool vet is not supported since go 1.12, we should use go vet instead. This
fixes the flags used, and all call it for the base package.

Hmm, travis tests fail with vet: flag "-structtag" not defined - could you please investigate?

LGTM, merging. Thanks!


Atlas Kubernetes Operator migration baseline error

Hi there, and firstly, thanks for this awesome project!
Im currently testing atlas as k8s operator and trying to apply AtlasMigration
As I already have the data inside my database, I firstly created the baseline file
atlas migrate diff my_baseline \
  --dir "file://migrations" \
  --dev-url "docker://postgres/13" \
  --to "postgres://MY_URL/MY_DB"

then i created the migration file itself and atlas.sum. Then I applied them as configmap
apiVersion: v1
kind: ConfigMap
metadata:
  name: migrations
  namespace: default
data:
  20240430_cars.sql: |-
    CREATE TABLE cars (
      brand VARCHAR(255),
      model VARCHAR(255),
      year INT
    );
<PHONE_NUMBER>0416_my_baseline.sql: |
  ...
  atlas.sum: |
    h1:XvxavNQEVsV5YOhis38sBUhE7EcQbJtD4WjZ2a/r48o=
<PHONE_NUMBER>0416_my_baseline.sql h1:opoj6rJBR24O4XrnEZrck6nFGhJ7iQyvRYzDELx0bJU=
    20240430_cars.sql h1:2D4fsS8tMnRswHQU8CS42arptXNv/lbgAggfkJ3f948=

After that I declared AtlasMigration
apiVersion: db.atlasgo.io/v1alpha1
kind: AtlasMigration
metadata:
  name: atlasmigration-test
  namespace: default
spec:
  baseline: "20240430130416"
  dir:
    configMapRef:
      name: migrations
  execOrder: linear
  urlFrom:
    secretKeyRef:
      key: url
      name: psql-creds

But after applying, I receive an error
Error: sql/migrate: connected database is not clean: found schema "NAME". baseline version or allow-dirty is required

Could you explain what am I doing wrong ?

Hello, I think it somehow related to the version of the first migration file 20240430_cars.sql can you please create new migration with atlas migrate new --edit then put its content here?

Hi @giautm
So I did atlas migrate new --edit, then applied new configmap and now its like this:
apiVersion: v1
kind: ConfigMap
metadata:
  name: migrations
  namespace: default
data:
<PHONE_NUMBER>2340.sql: |-
    CREATE TABLE cars (
      brand VARCHAR(255),
      model VARCHAR(255),
      year INT
    );
<PHONE_NUMBER>0416_my_baseline.sql: |
  ...
  atlas.sum: |
    h1:d0/QK4uX9nll+fLt+qjJMEAz7huRFGHzLSzTown2TKc=
<PHONE_NUMBER>0416_my_baseline.sql h1:opoj6rJBR24O4XrnEZrck6nFGhJ7iQyvRYzDELx0bJU=
<PHONE_NUMBER>2340.sql h1:VSLoTZmXMjQNKHYM7FxTLbtcOKBBBGAjPTD83BDtn70=

but unfortunately its the same (i recreated AtlasMigration object for clean testing)
Error: sql/migrate: connected database is not clean: found schema "NAME". baseline version or allow-dirty is required

@oleg-darrow, the baseline also need to re-generate, it should happen after the first version.

hi again @giautm
i didnt understand to the end this part it should happen after the first version, could you explain ?
but i tried to recreate all from scratch, so i deleted everything from migrations folder and i did next in order:

(I have migrations folder in the root of git repo)
new baseline file with atlas migrate diff my_baseline ...
new migration file with atlas migrate new --edit
(atlas.sum was created and calculated automatically)
I applied CM with kubectl create configmap migrations --from-file=migrations
I applied AtlasMigration object with new baseline string

and now result the same. Error: sql/migrate: connected database is not clean: found schema "NAME". baseline version or allow-dirty is required

@giautm strange, but its the same 🤔
so i deleted old baseline and created new one, so atlas.sum now is
h1:8SKfpgDIjzF1Ifl1RzgN1N7SNHqxLw4XJl2bRGrGka0=
20240502083707.sql h1:ELDU10vDlKvS2nbmHf/rs2dIB91RCIWxHiUonVm7SFE=
20240502090742_my_baseline.sql h1:G3QmCnPUi83VgdZxAfRS31nq1I+yjZbNvoA30tZ6eYk=

i changed baseline version to<PHONE_NUMBER>0742 in AtlasMigration but the error is still there

@oleg-darrow sorry, can you please share with version of atlas-operator you have here?

btw, please ensure you config the correct schema name on the url with search_path=NAME if you're working on schema scope

@giautm
arigaio/atlas-operator:0.4.4
i have 30 schemas in this database, so i use db bound url
postgres://USER:PASS@RDS_ADDRESS:5432/DB

In this case you have to config exclude option to excude the schema NAME

~In this case you have to config exclude option to exclude the schema NAME~

i have 30 schemas in this database, so i use db bound URL

How many schemas do you want to control via the AtlasMigration resource? It should be one schema per AtlasMigration (schema connection) or you have to put all migrations of all schemas into single AtlasMigration and use the realm connection.

I want to have ability to manage all schemas that i have, could you explain whats realm connection ?
for example this i create in public one
    CREATE TABLE cars (
      brand VARCHAR(255),
      model VARCHAR(255),
      year INT
    );

but this in schema1
    CREATE TABLE schema1.cars (
      brand VARCHAR(255),
      model VARCHAR(255),
      year INT
    );

https://atlasgo.io/integrations/kubernetes/operator#schema-and-database-bound-urls

well, as i said previously, im using DB bound connection (which is realm connection as i understood)
postgres://USER:PASS@RDS_ADDRESS:5432/DB
sorry, if im missing something

Hi guys @giautm @rotemtam and sorry again for disturbing.
It still doesnt work on my side, maybe you have more advices that I could try ?

Hi - any updates on this? I have the same issue. Thanks @oleg-darrow for raising this issue

After reading through the atlas operator code and making some iterative guesses over a few hours I figured out I could change the revisionSchema to a different one than the default with a baseline value and it worked for me. Hopefully, this helps someone else :D
  baseline: '20241007080718'
  dir:
    configMapRef:
      name: postgres-migrations
  revisionsSchema: "not_default_schema_name"


error in 'make test'

Hi Clifford,
I am trying to install yosys (latest git clone), and am getting the following error while running 'make test'. Similar errors seen for test/share & test/fsm.
I'm on Centos 6.4 with python version 2.6.6. Are there any python version dependencies? Couldn't find any mention on your website/README, hence the question.
--- begin error message ---
cd tests/realmath && bash run-test.sh
generating tests..
  File "generate.py", line 43
    with file('temp/uut_%05d.v' % idx, 'w') as f, redirect_stdout(f):
                                                ^
SyntaxError: invalid syntax
make: *** [test] Error 1
--- end error message ---

Thanks & regards,
Bhaskar

I have no python 2.6.6 for testing but I think I have fixed that now int commit 604c097f9818a5b29fa06223b88930630fafca5e. Please give it a try and let me know if that solved the issue for you.

Hi Clifford,
Thanks for the prompt reply. I will try a git pull in a bit and confirm.
Regards,
Bhaskar Karmakar
S3Craft Technologies
Bangalore, India
-original message-
Subject: Re: [yosys] error in 'make test' (#54)
From: Clifford Wolf<EMAIL_ADDRESS>Date: 20/03/2015 1:44 pm
I have no python 2.6.6 for testing but I think I have fixed that now int commit 604c097f9818a5b29fa06223b88930630fafca5e. Please give it a try and let me know if that solved the issue for you.

Reply to this email directly or view it on GitHub:
https://github.com/cliffordwolf/yosys/issues/54#issuecomment-83946735

Clifford,
The 'make test' errors have been fixed. Thanks.
I see 2 warnings as below, maybe you know of them already.
cd tests/various && bash run-test.sh
Running constmsk_test.ys..
Warning: Range select out of bounds on signal `\tmp' at constmsk_testmap.v:45: Setting 1 MSB bits to undef.
cd tests/sat && bash run-test.sh
Running initval.ys..
Warning: ignoring initial value on non-register: \bar [1:0]
Regards,
Bhaskar
-----Original Message-----
From: Bhaskar Karmakar<EMAIL_ADDRESS>Sent: 20 March 2015 14:06
To: cliffordwolf/yosys
Cc<EMAIL_ADDRESS>Subject: RE: [yosys] error in 'make test' (#54)
Hi Clifford,
Thanks for the prompt reply. I will try a git pull in a bit and confirm.
Regards,
Bhaskar Karmakar
S3Craft Technologies
Bangalore, India
-original message-
Subject: Re: [yosys] error in 'make test' (#54)
From: Clifford Wolf<EMAIL_ADDRESS>Date: 20/03/2015 1:44 pm
I have no python 2.6.6 for testing but I think I have fixed that now int commit 604c097f9818a5b29fa06223b88930630fafca5e. Please give it a try and let me know if that solved the issue for you.

Reply to this email directly or view it on GitHub:
https://github.com/cliffordwolf/yosys/issues/54#issuecomment-83946735

Thanks for verifying the fix. Those two warning are ok. This test cases deliberately test if yosys does the right thing in the situations that trigger those warnings..


Adding a UserCommand to convert etl files to speedscope.json or updating NetperfToSpeedScope

I've noticed that there is a UserCommand called NetperfToSpeedScope however giving that method a provided etl file, as it is stated in its comments results in an error due to the etlx file not being found.  I am unsure if there is a way to give the UserCommand a .netperf file since as far as I can tell from documentation the .netperf format is deprecated but this would likely cause the program to also look for a .etlx file which may not exist since I cant find where these are generated.
My main question is, is it worth the time to put in a PR to add a UserCommand to convert an etl file to SpeedScope, or does the current method work but need some more specific files that I am not looking for?
If NetPerf is not deprecated, how would I obtain this data using PerfView so I can quickly convert traces to the SpeedScope format?

@EmosewaMC, I recommend that you look at https://docs.microsoft.com/en-us/dotnet/core/diagnostics/dotnet-trace.  It is possible to convert from nettrace to speedscope via dotnet-trace convert.  You can also specify the output format directly if using dotnet-trace to collect.

The main question which I kind of buried in here is how can I convert an etl file to a speedscope format?  This would work for nettrace but my main goal is to take a trace with PerfView, and convert that resulting etl file to a speedscope without needing to do so via the CPUStacks gui.  Is this possible in the current version of PerfView?

Oh, I see.  Sorry, I mis-read your question.  I am not aware of a way to do this via the command line.  It is available through the Save View As menu option in the Stack Viewer.  Totally reasonable to create a UserCommand for this if you like.


DRILL-8408: Allow Implicit Casts on Join

DRILL-8408: Allow Implicit Casts on Join
Description
With the revision of Drill's implicit casting rules as a part of DRILL-8136, Drill now supports much improved implicit casting logic.  However, that does not carry over to joins.  This PR allows the implicit casting to carry through to joins as well.
Documentation
N/A
Testing
Updated unit tests

I agree that it would be nice to be able to switch this on or off using a session option. And I wonder if we should begin with that option defaulted to false so that we can

include this in 1.21.x and
collect some experience from opt-ins (like ourselves) about whether such joins turn out to be badly behaved, before exposing out-of-the-box users to it.

I added a new exec option defaulted to false.


Feature Request: Expose a way to modify the `MeasuredTemplate when created from an Item

When casting a spell, like fireball, dnd5e runs through a series of steps to link the item (e.g., the fireball spell) to the template. But it is very difficult for modules to modify the template that is created at that point.
In item.mjs, this line of code is particularly problematic:
templates = await (dnd5e.canvas.AbilityTemplate.fromItem(item))?.drawPreview();

There is not, as far as I can tell, a straightforward way to modify the created template prior to its preview being drawn. I could not find anywhere that AbilityTemplate is exposed, so I don't think I can use libWrapper. There are hooks before and after this step, which are nice, but they are all either too early or too late. I need to take information from the item and use it to modify the template parameters. (My current workaround is to hook the MeasuredTemplate drawing, but that is not great because it is overinclusive and because I don't really care about the template drawing per se, I care about getting information from the item upon template creation.)
So, my request is to add something to let modules modify the template upon creation. Something like:
const template = dnd5e.canvas.AbilityTemplate.fromItem(item);
Hooks.call("dnd5e.itemTemplateCreation," template, item, options);
await template.drawPreview();

That is a good idea.
I will say, you can patch AbilityTemplate using exactly the path you used there dnd5e.canvas.AbilityTemplate.

Good to know. I was trying to figure out if that class was exposed.


Fix clusterSingleton Restriction

towards https://github.com/giantswarm/giantswarm/issues/24568
Ref: https://github.com/giantswarm/app/pull/266

Please 👀  @giantswarm/team-honeybadger

@mcbenjemaa let's roll


Add connstant for Chef Server and improve help messaging

We talk about the server in messages and we need to brand that appropriately.
In the help:

Periods after each sentence
Use the branding better
Fix some weird capitalization

The final bit is the removal of a link to the RFC docs since those are in the boneyard now.

There's some specs to update here.

LGTM with the exception of the two suggestions I found.


Dimensional Dungeon/ Create interactions

Issues that do not follow this template will be closed
Describe the Bug: Use create wrench to pick up create items and you can infinitely duplicate any item the wrench picks up
How Can We Reproduce the Bug:
1.use create wrench to pick up create item
2.
3.
4.
More steps...
Expected behavior: wrench picks up item or does nothing in that dimension.
If Applicable, Add Screenshots or Video to Explain the Bug:
Crash Report or Game Log
Paste your crash report or log file to https://gist.github.com/ and paste the link here.
Please complete the following information:

Have you checked if your bug has already been reported, including closed issues? : yes
Have you added or removed any mods, and if so, which?:no
Modpack Version Number :1.01.0
Is OptiFine installed? : No
Does this issue affect Server, Client, or Both? : Both

Additional context:

Can you actually fill out how to reproduce this? Your instructions are unclear.
What create item?

I figured it out. Took longer than it had to, but I found some Create blocks to test on.
Fixed in dev

This actually wasn't fixed in dev.


NO-ISSUE: Synchronize From Upstream Repositories

The downstream repository has been updated through the following upstream commit:

Date
Commit
Author
Message

2024-11-08 22:33:25
operator-framework/catalogd@2f96c57
Per Goncalves da Silva
:warning: Bump ClusterCatalog API to v1 (#380)

upstream commit list

The vendor/ directory has been updated and the following commits were carried:

Date
Commit
Author
Message

2024-11-08 00:04:51
openshift/operator-framework-catalogd@9d65f48
dtfranz
UPSTREAM: <carry>: Add OpenShift specific files

2024-11-08 00:04:52
openshift/operator-framework-catalogd@5472f55
Alexander Greene
UPSTREAM: <carry>: Drop commitchecker

2024-11-08 00:04:52
openshift/operator-framework-catalogd@26adfce
AOS Automation Release Team
UPSTREAM: <carry>: Updating ose-olm-catalogd-container image to be consistent with ART Reconciling with https://github.com/openshift/ocp-build-data/tree/e5df10e41b14c724468c8f78e2690f219279bfc4/images/ose-olm-catalogd.yml

2024-11-08 00:04:53
openshift/operator-framework-catalogd@ccfc5ac
AOS Automation Release Team
UPSTREAM: <carry>: Updating ose-olm-catalogd-container image to be consistent with ART Reconciling with https://github.com/openshift/ocp-build-data/tree/4022cd290f00a44d667dda03f2d78d84a488c7ed/images/ose-olm-catalogd.yml

2024-11-08 00:04:53
openshift/operator-framework-catalogd@8f98969
Andy Goldstein
UPSTREAM: <carry>: Remove ncdc from OWNERS

2024-11-08 00:04:54
openshift/operator-framework-catalogd@6f84c1d
Mikalai Radchuk
UPSTREAM: <carry>: update owners

2024-11-08 00:04:54
openshift/operator-framework-catalogd@20b0069
AOS Automation Release Team
UPSTREAM: <carry>: Updating ose-olm-catalogd-container image to be consistent with ART for 4.16 Reconciling with https://github.com/openshift/ocp-build-data/tree/cd35345d4d061d63bddd2911a9c77149106dee55/images/ose-olm-catalogd.yml

2024-11-08 00:04:55
openshift/operator-framework-catalogd@71c4ce2
Ilias Rinis
UPSTREAM: <carry>: manifests: set required-scc for openshift workloads

2024-11-08 00:04:55
openshift/operator-framework-catalogd@fe8db7f
Todd Short
UPSTREAM: <carry>: Update CI base to 1.22/4.17

2024-11-08 00:04:56
openshift/operator-framework-catalogd@b49d0ee
Todd Short
UPSTREAM: <carry>: Update manifest generation for catalogd

2024-11-08 00:04:56
openshift/operator-framework-catalogd@c199b54
everettraven
UPSTREAM: <carry>: add everettraven to approvers+reviewers

2024-11-08 00:04:57
openshift/operator-framework-catalogd@df65f24
Tayler Geiger
UPSTREAM: <carry>: add openshift kustomize overlay

2024-11-08 00:04:57
openshift/operator-framework-catalogd@1609603
AOS Automation Release Team
UPSTREAM: <carry>: Updating ose-olm-catalogd-container image to be consistent with ART for 4.18 Reconciling with https://github.com/openshift/ocp-build-data/tree/827ab4ccce9cbbcf82c9dbaf6398b61d6cff8d7a/images/ose-olm-catalogd.yml

2024-11-08 00:04:58
openshift/operator-framework-catalogd@bce89da
Todd Short
UPSTREAM: <carry>: Fix webhook manifest generation

2024-11-08 00:04:59
openshift/operator-framework-catalogd@94c9074
Todd Short
UPSTREAM: <carry>: Add tmshort as an approver

2024-11-08 00:04:59
openshift/operator-framework-catalogd@0595e68
Joe Lanford
UPSTREAM: <carry>: manifests: add hostPath mount for /etc/containers

2024-11-08 00:05:00
openshift/operator-framework-catalogd@ebd218b
dtfranz
UPSTREAM: <carry>: Add test-e2e target for downstream Makefile to be run by openshift/release.

2024-11-08 00:05:01
openshift/operator-framework-catalogd@fb176fc
Anik Bhattacharjee
UPSTREAM: <carry>: Add hostPath mount for /var/lib/kubelet

2024-11-08 00:05:01
openshift/operator-framework-catalogd@53085de
Joe Lanford
UPSTREAM: <carry>: Revert "UPSTREAM: <carry>: Add hostPath mount for /var/lib/kubelet"

2024-11-08 00:05:02
openshift/operator-framework-catalogd@cc7ab12
dtfranz
UPSTREAM: <carry>: Add downstream verify makefile target

2024-11-08 00:05:03
openshift/operator-framework-catalogd@06fb1c8
Mikalai Radchuk
UPSTREAM: <carry>: Add global-pull-secret flag

2024-11-08 00:05:03
openshift/operator-framework-catalogd@8eede90
Joe Lanford
UPSTREAM: <carry>: openshift: template log verbosity for control by cluster-olm-operator

2024-11-08 00:05:04
openshift/operator-framework-catalogd@3299411
Brett Tofel
UPSTREAM: <carry>: Changes to add default catalogs

2024-11-08 00:05:05
openshift/operator-framework-catalogd@f0ed398
Brett Tofel
UPSTREAM: <carry>: catalogd- namePrefix spec dirs

2024-11-08 00:05:06
openshift/operator-framework-catalogd@f223d9f
Todd Short
UPSTREAM: <carry>: Add openshift CAs to catalogd

2024-11-08 00:05:06
openshift/operator-framework-catalogd@8ea6e34
Joe Lanford
UPSTREAM: <carry>: set priorities on default catalogs

2024-11-08 09:32:53
openshift/operator-framework-catalogd@3f08439
Todd Short
UPSTREAM: <carry>: Update default catalogs

This pull request is expected to merge without any human intervention. If tests are failing here, changes must land upstream to fix any issues so that future downstreaming efforts succeed.
/cc @openshift/openshift-team-operator-framework
/label tide/merge-method-merge

Any chance we could update this PR with this change as well?
https://github.com/operator-framework/operator-controller/pull/1457

/override ci/prow/openshift-e2e-aws-techpreview
/lgtm


fix(curriculum): refactor decimal to binary converter to check for positive numbers only

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #54651

Step 1 text updated to show project will only covert positive numbers.
https://github.com/freeCodeCamp/freeCodeCamp/issues/54651#issuecomment-2096829428 @jdwilkin4

But IMO, I think we should just add a note in the beginning for step 1 that this project will build out a binary converter for positive numbers.

https://github.com/freeCodeCamp/freeCodeCamp/issues/54651#issuecomment-2097497157 @scissorsneedfoodtoo

Like Jessica mentioned, we could adjust the text in step 1 to emphasize that this project will only convert positive numbers into binary.

Updated instructions for step 12

Hint message for step 12 second condition

Hint message for step 12 third condition

Step 12 success message

Updated instructions for step 13

Hint message for step 13 - alert without text

Step 13 success message

Step 78 instructions

Hint message for step 78 - second condition

Hint message for step 78 - third condition

This PR was just merged in which addresses the same issue
https://github.com/freeCodeCamp/freeCodeCamp/pull/54708
As result, we are going to close your PR


Wrap AmbientLightSensor

See https://developer.mozilla.org/en-US/docs/Web/API/AmbientLightSensor
I can probably do this one myself, but I'm not sure if we want to support it seeing as it's an experimental API only supported by Chrome right now.

Just FYU- it also has worked in Firefox for macOS for some time now.

I'm running Firefox 57 on macOS and it doesn't seem to be available


Decimal data type?

Can I define a decimal data type?
Or is decimal not even supported in the OpenApi standard? And if so what can I do instead?
I would like to be able to do something like this.
["amounOffered"] = new OpenApiSchema 	{ 							Type = "number", 							Format = "decimal" 						}

The "decimal" format is not natively recognized by the standard. This doesn't mean that you can't use it, but it means that most tools out there probably would not have support for it since it's not required by the spec.
If you have your own tool that can interpret "decimal" format, then there should be no issues for you to use it. Our OpenAPI.NET library does not enforce any specific values in Schema.Format field. In other words, what you wrote above is valid - the question would be whether the consumer of that object knows what to do with the "decimal" format.
Does this help?

I'll close this. Please feel free to reopen if you have more questions or concerns.


Conditional Access & Identity Protection is not available as Global Reader (AAD)

No access as Global Reader in the Azure Active Directory admin center to Conditional Access & Identity Protection
Please add the following to "Global reader role has a few limitations right now"

Azure Active Directory admin center - No access to Conditional Access & Identity Protection in the Web

Conditional Access | Policies
= No access
Conditional Access | Named locations
= Unable to complete due to service connection error. Please try again later.
[Microsoft_AAD_IAM]  3:54:54 PM ajax_calls 403 ajax_calls: Client side call failed with 403 status code ListNamedLocations
Conditional Access | Terms of use
= In order to access this page, at least one of the following roles must be assigned to you - global administrator, global reader, security administrator, or conditional access administrator.
Identity Protection | Overview
= No access

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

ID: 0767aac4-fcd9-eaac-2c30-8f17d8eadd92
Version Independent ID: a380a828-03f9-e037-3a62-e4165fda7cb6
Content: Azure AD built-in roles - Azure Active Directory
Content Source: articles/active-directory/roles/permissions-reference.md
Service: active-directory
Sub-service: roles
GitHub Login: @rolyon
Microsoft Alias: rolyon

@weyCC81
Thanks for your feedback! We will investigate and update as appropriate.

@weyCC81
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@weyCC81 I am able to access Conditional Access and Identity Protection with my account that only has Global Reader assigned to it. Are you using an Administrative-unit scoped role or a custom role?

Hi @weyCC81
We are going to close this for now, but let us know if you have any additional feedback.
thanks

#please-close


Add instructions on how to use facil.io with meson

I hope this PR will help this project.
Thank you for this great framework.

This looks great! Thanks!
👏🏻👏🏻👏🏻👍🏻


Switch to Go releaser with GitHub actions for releases

As it is a super nice and quick way of getting releases out in automated fashion. Examples here: https://github.com/kinvolk/terraform-provider-tinkerbell.

Specifically opened for https://github.com/kinvolk/container-linux-config-transpiler, but really this issue can apply to a number of our repos.

IMO we should have an issue for this kind of things independently for all affected repos. We can still link here to track global progress on that.

Did you know the config-transpiler bin files are not build anymore? I have got to go to CoreOS releases to download ct v0.6.1 ... https://github.com/coreos/container-linux-config-transpiler/releases


Use https url to download node

Bump!

For context, our 18F team (which includes @adelevie, @afeld, and me) has a fork of this repository at https://github.com/18F/cf-meteor-buildpack with only this very small change, and we'd love to deprecate our fork. :)

I just got the power to merge - hope that's ok! Checked that the new URL still works.


#Include errors

Type: Bug
There are multiple errors in the output log of the #include files saying that they cannot be opened. I already watched multiple youtube videos on how to fix the problem but i cannot find the source to put in the includePath in the .json tab. Thanks!
Here is an image of the output log errors.

VS Code version: Code 1.79.2 (695af097c7bd098fbf017ce3ac85e09bbc5dda06, 2023-06-14T08:57:04.379Z)
OS version: Windows_NT x64 10.0.22621
Modes:

/extC++
Please seek advice at the home of the extension that is producing the diagnostic messages.

It looks like this is caused by the C++ extension. Please file the issue to the C++ extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!