Configurabe scrape interval / scrape timeout via relabeling still marked as experimental

Proposal
Hello,
the possibility of configuring scrape_interval/scrape_timeout via relabeling has been implemented in #8911 which was merged in Aug, 2021, roughly 2.5 years ago. Still the feature is marked as "experimental". This seems to be the case only in the documentation, I did not find anything in the code that marks the feature as experimental, it isn't hidden behind a feature flag either.
I haven't been able to find any issues caused by this feature (and I'm currently using it myself successfully), so I'd propose to remove the hint in the docs telling users that it is experimental. Or are there any objections to that which I'm not aware of?
Thanks in advance,
Sven

Hello Sven,
Yes, this feature can be marked as stable.


Deadlock under load caused by Wait() in GoogleAdsGrpcInterceptor

Under some load in asynchronous context, our call to service like GoogleAdsService through GoogleAdsServiceClient lock our app with threads starvation.
Steps to Reproduce:
Put some load and watch it fails.
We do not have the stack trace anymore, but all of our threads were blocked at this method on a Wait() operation: Google.Ads.GoogleAds.Interceptors.GoogleAdsGrpcInterceptor.AsyncUnaryCall
Expected behavior:
Asynchronous operations should not lock
Client library version and API version:
Client library version: v.11.0.0
Google Ads API version: N/A
.NET version: .Net 6
Operating system (Linux, Windows, ...) and version (if the bug is platform-specific): Windows App service on Azure
Workaround:
We built our own version with the logging code removed and everything is now fine
public override AsyncUnaryCall<TResponse> AsyncUnaryCall<TRequest, TResponse>(
            TRequest request, ClientInterceptorContext<TRequest, TResponse> context,
            AsyncUnaryCallContinuation<TRequest, TResponse> continuationCallback)
        {
            AsyncUnaryCall<TResponse> call = continuationCallback(request, context);

            //Task t = call.ResponseAsync.ContinueWith(
            //    delegate (Task<TResponse> oldTask)
            //    {
            //        loggingHandler.HandleAsyncUnaryLogging(request, context, oldTask, call);
            //    });
            //t.Wait();  <------------- We should never Wait in an async context

            return UnaryRpcInterceptor.Intercept(call);
        }

Hi @AnashOommen
We rescued some stack trace.  Our code is calling googleAdsServiceClient.SearchAsync(...) and while we iterate the results PagedAsyncEnumerable<SearchGoogleAdsResponse, GoogleAdsRow> it locks with this stack trace
HelperMethodFrame_1OBJ
System.Threading.Monitor.Wait
System.Threading.ManualResetEventSlim.Wait
System.Threading.Tasks.Task.SpinThenBlockingWait
System.Threading.Tasks.Task.InternalWaitCore
System.Threading.Tasks.Task.Wait
System.Threading.Tasks.Task.Wait
Google.Ads.GoogleAds.Interceptors.GoogleAdsGrpcInterceptor.AsyncUnaryCall[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]]
Grpc.Core.Interceptors.InterceptingCallInvoker.AsyncUnaryCall[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]]
Grpc.Core.Interceptors.InterceptingCallInvoker.<AsyncUnaryCall>b__4_0[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]]
Grpc.Core.ClientBase+ClientBaseConfiguration+ClientBaseConfigurationInterceptor.AsyncUnaryCall[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]]
Grpc.Core.Interceptors.InterceptingCallInvoker.AsyncUnaryCall[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]]
Google.Ads.GoogleAds.V10.Services.GoogleAdsService+GoogleAdsServiceClient.SearchAsync
Google.Api.Gax.Grpc.ApiCall+GrpcCallAdapter`2[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]].CallAsync
Google.Api.Gax.Grpc.ApiCallRetryExtensions+<>c__DisplayClass0_0`2+<<WithRetry>b__0>d[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]].MoveNext
System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[[Google.Api.Gax.Grpc.ApiCallRetryExtensions+<>c__DisplayClass0_0`2+<<WithRetry>b__0>d[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]], Google.Api.Gax.Grpc]]
System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1[[System.__Canon, System.Private.CoreLib]].Start[[Google.Api.Gax.Grpc.ApiCallRetryExtensions+<>c__DisplayClass0_0`2+<<WithRetry>b__0>d[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]], Google.Api.Gax.Grpc]]
Google.Api.Gax.Grpc.ApiCallRetryExtensions+<>c__DisplayClass0_0`2[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]].<WithRetry>b__0
Google.Api.Gax.Grpc.ApiCall`2+<>c__DisplayClass10_0[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]].<WithCallSettingsOverlay>b__0
Google.Api.Gax.Grpc.ResponseAsyncEnumerable`3+ResponseAsyncEnumerator+<MoveNextAsync>d__10[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]].MoveNext
System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[[Google.Api.Gax.Grpc.ResponseAsyncEnumerable`3+ResponseAsyncEnumerator+<MoveNextAsync>d__10[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]], Google.Api.Gax.Grpc]]
System.Runtime.CompilerServices.AsyncValueTaskMethodBuilder`1[[System.Boolean, System.Private.CoreLib]].Start[[Google.Api.Gax.Grpc.ResponseAsyncEnumerable`3+ResponseAsyncEnumerator+<MoveNextAsync>d__10[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]], Google.Api.Gax.Grpc]]
Google.Api.Gax.Grpc.ResponseAsyncEnumerable`3+ResponseAsyncEnumerator[[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib],[System.__Canon, System.Private.CoreLib]].MoveNextAsync

We'll eventually try to deactivate the logging with an official release.
Thanks
Phil

Hi Phil,
Could you check if the issue is replicable with v12.0.0? I made some changes with the lifecycle of GoogleAdsInterceptor, so hopefully this isn't an issue.

@AnashOommen We did not try the v12 release yet.  Should come in a few weeks

I was facing an issue that has the same origin I believe:

GoogleAdsClient always uses GoogleAdsServiceClientFactory to create services.
GoogleAdsServiceClientFactory always adds GoogleAdsGrpcInterceptor.
GoogleAdsGrpcInterceptor always blocks the thread by executing t.Wait(), waiting for the response before returning. It's seen from the code that no config options can change that, it doesn't matter whether logging is on or off.

That basically means that (possibly) all async calls are blocking.

I removed all the Wait() calls in v14.0.0.

@AnashOommen and now we have #488 and #480 😟


Seeking in custom controls broken on Android

Seems to be specific to Android Chrome, but seeking is broken.  This is a UI issue only, since setting currentTime directly via the JS console still works.  I have tested all the way back to v1.3.0 (the introduction of the custom controls on the demo page) and it has always been broken.
Originally reported here: https://code.google.com/p/chromium/issues/detail?id=523533

This bug is twofold.
One is a race in controls.js file between input events and timeupdate events.  This race is not an issue for some reason on desktop, but manifests on mobile.
The other bug is that we are only using mousedown/mouseup and need to add support for touchstart/touchend as well.


changes to support sf-mongo binding rotation

Pull Request Test Coverage Report for Build 4554

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 89.246%

Files with Coverage Reduction
New Missed Lines
%

core/utils/src/commonFunctions.js
1
85.16%

data-access-layer/cf/src/TokenIssuer.js
1
98.55%

Totals

Change from base Build 4547:
-0.02%

Covered Lines:
9347

Relevant Lines:
10111

💛  - Coveralls


Suppress error to one line

Suppress java.lang.NumberFormatException: For input string: "enterprises" errors done to a single "invalid license" line.

Fixed in 1.2-DPoE Branch


Add invitation only button to course detail view

Description
OSPR-1782
This PR disables the enroll button in a course if it is invite only and changes the text.
Notes
For this PR to work it depends on this other PR
How to test this PR

Have a course invite only.
Find the course in the app.
Click on it, should see button disabled.

Reviewers

[ ] Code review: @miankhalid
[ ] Code review: @farhan

cc @marcotuts @gsong

Thanks for the pull request, @jagonzalr! I've created OSPR-1782 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.
Feel free to add as much of the following information to the ticket:

supporting documentation
edx-code email threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.
If you like, you can add yourself to the AUTHORS      file for this repo, though that isn't required.     Please see the CONTRIBUTING     file for more information.

Codecov Report

Merging #956 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #956      +/-   ##
============================================
- Coverage     16.88%   16.88%   -0.01%
  Complexity     1012     1012
============================================
  Files           466      466
  Lines         19952    19958       +6
  Branches       2302     2303       +1
============================================
  Hits           3369     3369
- Misses        16202    16208       +6
  Partials        381      381

Impacted Files
Coverage Δ
Complexity Δ

.../main/java/org/edx/mobile/course/CourseDetail.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...java/org/edx/mobile/view/CourseDetailFragment.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

It would be great if you can attach the screenshot of this change.

@miankhalid done son.

Visual Nit
The button's edges are not rounded in the screenshot you attached as opposed to the button in the screenshot below:

@miankhalid like this?

This PR will be mergeable once:

you squash the commits into 1 commit.
the server PR https://github.com/edx/edx-platform/pull/15223 gets merged.

@miankhalid done with the squash, maybe @marcotuts or @gsong can accelerate the PR edx/edx-platform#15223 getting merged.

@miankhalid the PR edx/edx-platform#15223 has been merged.


nmap options

Add the follow options to nmap:
-sV (Version detection): Implemented some kind of version detection against target
-O (Enable OS detection): Implement some kind of OS detection against target
-sC: Implement script execution against target; folder with default scripts and a folder with user scripts
Enhancements and corrections:

Use -sT as default for port scans

-sT Issue corrected with ISSUES RESOLVED of 4/7/2019


hotreload broken for renderer after adding WS dependency.

client.ts:78 WebSocket connection to 'ws://localhost:5173/' failed: Error during WebSocket handshake: net::ERR_CONNECTION_RESET
setupWebSocket @ client.ts:78
client.ts:78 Uncaught (in promise) DOMException: Failed to construct 'WebSocket': The URL 'ws://<IP_ADDRESS>:undefined/' is invalid.
at setupWebSocket (http://localhost:5173/@vite/client:465:20)
at fallback (http://localhost:5173/@vite/client:444:22)
at WebSocket. (http://localhost:5173/@vite/client:480:13)

Can you provide a minimal production repo?


RecordNotFound error when Find returns an empty list

I just upgraded the latest master branch. It seems that the behaviour of Find has changed when it returns an empty list with mysql.
Example:
var obj_list []Obj
err := gorm.Where("completed=true").Find(&obj_list).Error
If the result is an empty list (len(obj_list) == 0), then err is set to gorm.RecordNotFound. Previously it was nil.
This seems like the wrong behaviour: an empty list is generally not an error; it's often an expected result. Is this new behaviour intentional? I would suggest reverting back to the old behaviour if possible.

I dont know, but seems lastest gorm throw error when query result is empty. And message is record not found.
The code:
import (
	"fmt"
	_ "github.com/go-sql-driver/mysql"
	"github.com/jinzhu/gorm"
)
func main() {
    // connecting db and more...
    panicOnErr(db.Find(&user, "id = ?", id).Error)    // OR panicOnErr(db.First(&user, id).Error)
}
func panicOnErr(err error) {
	fmt.Println(err)
	if err != nil {
		panic(err)
	}
}

And  debug output:
[2017-02-22 12:05:37]  [1.74ms]  SELECT * FROM `users`  WHERE `users`.`deleted_at` IS NULL AND ((id = '8'))
record not found

@maple-leaf Find a slice won't return error, bug find a struct will.

@jinzhu I will temporary use slice instead. But hope this can be fixed soon. 😄  Btw, this is a amazing project. 👍


rsconf_service_order[@]: unbound variable

curl -L -s -S -n http://v3.radia.run:2916/host/v4.radia.run/000.sh
Source: /var/tmp/radia-run-7599-137/20220802214631-000.sh
/var/tmp/radia-run-7599-137/20220802214630-rsconf.sh: line 428: rsconf_service_order[@]: unbound variable

Not a bug. Was confused about which machine I was ssh'ed into.


WS2801 possible?

Hi,
is it possible to use this project with an WS2801 LED stripe on Raspberry Pi?
Thank You
Henri

@mischa67 I do not have such an LED strip but this question was asked a number of times before. Check out #123. A user there found a way of adapting this code for these lights. you might give it a shot.

Also a duplicate of #47, it's since been solved.


Current project status

Hi, what is the current status of this project? Is it still actively maintained?

Hi Daniel,
Yes, we actively maintain our hosted version of this project.
https://freeboard.io
This free and open source version is stable and maintained.

It doesn't look like it from the pulse of the repo :slightly_frowning_face:
https://github.com/Freeboard/freeboard/graphs/code-frequency

This looked promising until I realized no commits for almost 2 years.  I find it hard to believe that this in any way mirrors what is on the website and is actively maintained.


[feature] A Simple Data Mirroring Needed

I think we need to a simple data mirroring for developers who want to test the SQL statement on their PC.
As @TechQuery  said, that can also help us with simple automated tests.
I hope to use a script and a simple configuration file to connect the local project and the mirroring.
Thanks for your time.

actually we already discussed the problem in #157

Oh, I'm sorry. Should I close this issue, and discuss in #157 ？

No problem, we can discuss here instead. Because #157 has a doc label which not right.


add optional prepend to preInstructions method builder

With the recent need to calculate compute units by simulating the Transaction it's become desirable to prepend instructions to the preInstructions. Specifically instructions like ComputeBudgetProgram.setComputeUnitPrice and ComputeBudgetProgram.setComputeUnitLimit should be prepended to the preInstructions array after simulating the transaction.

if you're using the MethodBuilder rpc method to send the TX, then mutating the tx from the transaction() method wouldn't work, right?

Sort of — it would require a bit more setup compared to what we can do with this PR.
Before we merge this, could you also note this change in the CHANGELOG?

@acheroncrypto added


Investigate GuruX DLMS library

As OSGP we want to implement smart metering
Smart Investigate DLMS library.
Part of BI #46.

Currently HLS 5 security mechanism is not implemented.
We expect not to use this library for OSGP


Strong constraint 4DVar for 1D Burgers equation

Adds a script forward_burgers.py for a simple trapezium rule timestepping for reference.
Adds two scripts for strong constraint 4DVar for the 1D Burgers eq:

sc4dvar.py: Strong constraint 4DVar using a standard ReducedFunctional.
sc4dvar_aaorf.py: Strong constraint 4Dvar using the new AllAtOnceReducedFunctional.

Both scripts start from a target initial condition (two superimposed sin waves of different magnitude/frequency/phase) and run the forward model to generate the "ground truth" observations.
A new initial condition is created (different magnitudes/phase for the sin waves plus some noise). This initial condition (the background) is stepped forward, accumulating the functional for the observation error through the timeseries. This run of the forward model is taped. The second script builds the functional by registering the observations with AllAtOnceReducedFunctional.
The 4DVar system is solved with L-BFGS-B and the optimised initial condition is then stepped forward to collect some errors vs the target solution. The three solutions (target, background, optimised) are written to vtk to compare.
Observations are taken either evenly spaced or randomly in space, but always evenly spaced in time. The functional can include scalar error covariances for the background and observation errors.

Tape visualisation from sc4dvar_aaorf.py with three observations (including at initial time) and 3 timesteps between each observation:


[3] Can't move thread to another category

The dropdown is empty.
All issues i could find so far, thank you for your time for this package :)

Fixed in laravel-forum-frontend.
Thanks for reporting :)


abstract FieldValue

The first PR to address Issue#125. It includes the basic abstraction and updates to all InputVector related logic.

wrong branch


history visualization + a notification fix

I've included a date on question notifications, and I've added an [x]/[y] part to the notifications, too, to let participants know how many questions they've completed.
just a warning: I've been having issues testing this code because of a lack of data on my development device, but generating random data has given good visuals.

@eschloss please review!


[Fix] packages/react issue with useQuery not supporting dynamic query parameters

Original issue reported at https://github.com/powersync-ja/powersync-js/issues/323.
Dynamic query dependencies would warn with the following if the query parameter array changed in size:
Warning: The final argument passed to useMemo changed size between renders. The order and size of this array must remain constant. Previous: [] Incoming: [xxxxxxxxxxxxx]
Tested with the following to verify that it works now:
const [params, setParams] = React.useState<string[]>([]);
const { data } = useQuery(`select * from lists where id=?`, params);

onclick:: setParams(["some-uuid"])

The change from React.useMemo(() => {}, [...array]); to React.useMemo(() => {},[JSON.stringify(array)]) should not introduce any issues, as valid query parameters are expected to be serialisable.

We may want to look at something like this in the future https://github.com/sandiiarov/use-deep-compare#readme.

Very cool Christian!

I just got hit with this today in our process of porting over our codebase from Electric. Spent quite some time trying to figure out which useMemo it was, until I found this. Thanks for fixing it @Chriztiaan!
Any idea when this could be merged and released? Is there any canary / nightly one could test after it is merged?

@fooware, I'll release this soonest and ping you.


Fix  [BUG] UI issue in activity_course_detail.xml

This is a fix for #113  to fix activity_course_detail.xml
Before

After

Thanks @kulloveth


Set physics.velocity_threshold to 100.0

Described in detail here - https://github.com/defold/template-desktop/pull/5#issue-2434059720

Thanks!


Update HOWTO.md

I just spent some time because I read the HOWTO instead of the readme to work with some project to find out that in only supports BSV. So I think updating this reference to the one from the readme makes sense

Thanks


Update influxd-systemd-start.sh

Adding a timeout to the curl test command on systemd startup.  There is a race condition where influx is not up, but curl has started. curl can block on the initial failure long enough for systemd to restart the process.
Closes #
Describe your proposed changes here.

[ ] CHANGELOG.md updated with a link to the PR (not the Issue)
[ ] Well-formatted commit messages
[ ] Rebased/mergeable
[ ] Tests pass
[ ] http/swagger.yml updated (if modified Go structs or API)
[ ] Feature flagged (if modified API)
[ ] Documentation updated or issue created (provide link to issue/pr)
[ ] Signed CLA (if not already signed)

Superseded by #22076, closing this one


src/greenpak/i2c.py: Bug-fix SMBus adapter write.

.. Somehow did not see this on the first "fix"..

@zapta  : couple of fixes for our smbus2 adapter.
(the gp_xxx  methods transition will be per other pr work)

Please take a look at my comment in the pull request regarding the
write_read method. The interface deals only with the abstract interface and
not implementation. Please move the write_read method to the implementation
of the SMBUS class.
On Thu, Mar 7, 2024 at 4:46 PM dry-75 @.***> wrote:

@zapta https://github.com/zapta : couple of fixes for our smbus2
adapter.
(the gp_xxx methods transition will be per other pr work)
—
Reply to this email directly, view it on GitHub
https://github.com/zapta/greenpak/pull/9#issuecomment-1984851353, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQVMQPFRKXMLS45N47M5XTYXEC45AVCNFSM6AAAAABEKHYSQGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBUHA2TCMZVGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Can't find "Cory Doctorow's Futuristic Tales of the Here and Now"

When searching for "Cory Doctorow's Futuristic Tales of the Here and Now" nothing is returned, searching for "Cory Doctorow" does return the correct result.
SQLite has a full text search option which is probably useful, I checked python 3.8-3.12 and pypy3 and they all include the FTS5 extension. You could also check if it is supported first https://stackoverflow.com/a/36663390
The literal search also doesn't work because it removes the ' from the search.

I did have a look at FTS and decided against it and went with adding %'s instead for the search. Both problems come down to the ' being removed. Removing utils.sanitize_title(series_name, literal) should fix both issues.
The other option is to replace ' and " (maybe some more) with % the same as any spaces.

https://github.com/mizaki/gcd_talker/commit/97daa5f98ec5a3fe0242a25702476844285b9227 should have fixed both issues.
I had another look at FTS and it wouldn't help with the ' issue and I don't see it helping unless wanting to say search over the descriptions too. Which I don't think is all that helpful?

Interestingly removing the ' makes it fail on GCD itself: https://www.comics.org/series/name/Cory Doctorows Futuristic Tales of the Here and Now/sort/alpha/

I've added FTS in https://github.com/mizaki/gcd_talker/commit/b6252756b384147e0cd9aa59b82b7f9a0f33e4ea
Not sure on checking the fts and fts_docsize tables to see if a rebuild is needed. I can't really see where it would be required.

It shouldn't be needed unless someone is doing some custom modifications to the db, you can also setup hooks to keep it up to date in case someone does

Thanks for fixing this!


chore: 🤖 update ecr role for short lived creds

Codecov Report
All modified lines are covered by tests :white_check_mark:
:loudspeaker: Thoughts on this report? Let us know!.


add option to find circumcenter of triangle using equation of triangle

Proposal
add option to find circumcenter of triangle using equation of sides
(A clear and concise description of what the proposal is.)

@sairish2001 i would like to work on it


WASI support

WASI support
Hello :wave: , quick question :)
According to this article - https://hacks.mozilla.org/2019/03/standardizing-wasi-a-webassembly-system-interface/ one is able to just run WASM binary that was compiled with WASI support (it's possible that I mix terms, sorry)
I've found somewhere on Twitter, that PHP is not yet supported by https://github.com/CraneStation/wasmtime.
But what if I'm gonna use this extension, which uses Wasmer, which has support for WASI, it's gonna work right :pray:?
Thanks for your time.

Yes, we can add support for WASI quite easily. Give me few days to work on it :-)!

Oh my... You are so awesome! Thanks :pray:

(more than a few weeks actually… working on it soon, see https://github.com/wasmerio/wasmer/pull/1030 to follow latest improvements)

What's the status on this, given the above linked PR was merged?


Explore Manticore support

Problem
It's unclear to what degree https://github.com/trailofbits/manticore is supported on the network
Solution
Explore https://github.com/trailofbits/manticore in a POC
Capture any integration issues encountered
Alternatives
No response

@se7enarianelabs I think the Ariane labs team did this right.
Please note the merged PR that resolved this and close it.
If not let's comment here what's left.
Thanks

@se7enarianelabs I think the Ariane labs team did this right. Please note the merged PR that resolved this and close it. If not let's comment here what's left. Thanks

In this PR (https://github.com/hashgraph/hedera-smart-contracts/pull/767), we briefly analyzed the tools MAIAN, Manticore, and Slither. It's important to note that Manticore is no longer developed or maintained.


Cleaning all beta features

In the 1.19.0-2.0.0 Google provider (stable) all beta features is deprecated :
https://github.com/terraform-providers/terraform-provider-google/blob/2.0.0/CHANGELOG.md
Warning: module.gke-cluster.google_container_cluster.new_container_cluster: "master_ipv4_cidr_block": [DEPRECATED] This field is in beta and will be removed from this provider. Use it in the the google-beta provider instead. See https://terraform.io/docs/providers/google/provider_versions.html for more details.

Warning: module.gke-cluster.google_container_cluster.new_container_cluster: "node_config.0.taint": [DEPRECATED] This field is in beta and will be removed from this provider. Use it in the the google-beta provider instead. See https://terraform.io/docs/providers/google/provider_versions.html for more details.

Warning: module.gke-cluster.google_container_cluster.new_container_cluster: "node_config.0.workload_metadata_config": [DEPRECATED] This field is in beta and will be removed from this provider. Use it in the the google-beta provider instead. See https://terraform.io/docs/providers/google/provider_versions.html for more details.

Warning: module.gke-cluster.google_container_cluster.new_container_cluster: "pod_security_policy_config": [DEPRECATED] This field is in beta and will be removed from this provider. Use it in the the google-beta provider instead. See https://terraform.io/docs/providers/google/provider_versions.html for more details.

Warning: module.gke-cluster.google_container_cluster.new_container_cluster: "private_cluster": [DEPRECATED] This field is in beta and will be removed from this provider. Use it in the the google-beta provider instead. See https://terraform.io/docs/providers/google/provider_versions.html for more details.

Thanks @rekcah78 👍

I have create a new branch for the beta provider : beta
All PR are welcome :)

A new repository is created for the new beta provider : https://github.com/google-terraform-modules/terraform-google-beta-kubernetes-engine


Renewal date incorrect

I setup the client for SAN Exchange 2013 / IIS on 2008R2. Got cert, added it, setup scheduled task just fine. ( Great job on that)
Problem is, the cert in cert manager shows renewal date of 7/26/2016. renew task runs daily and noticed I didn't get an updated cert. Ran the command manually from command line and shows renewal date of 8/26/2016.  Below is shell session with username and domain names changed.
Just got an email from LetsEncrypt today telling me I have 10 days until certificate expiration.
C:\LE>letsencrypt --renew --baseuri "https://acme-v01.api.letsencrypt.org/"
Let's Encrypt (Simple Windows ACME Client)
Renewal Period: 60
Certificate Store: WebHosting
ACME Server: https://acme-v01.api.letsencrypt.org/
Config Folder: C:\Users<my username>\AppData\Roaming\letsencrypt-win-simple\httpsacme-v01.api.letsencrypt.org
Certificate Folder: C:\Users<my username>\AppData\Roaming\letsencrypt-win-simple\httpsacme-v01.api.letsencrypt.org
Loading Signer from C:\Users<my username>\AppData\Roaming\letsencrypt-win-simple\httpsacme-v01.api.letsencrypt.org\Signer
Getting AcmeServerDirectory
Loading Registration from C:\Users<my username>\AppData\Roaming\letsencrypt-win-simple\httpsacme-v01.api.letsencrypt.org\Registration
Checking Renewals
Checking Manual hostname.domainname.net (%SystemDrive%\inetpub\wwwroot) Renew After 8/26/2016

Duplicate
#256
#254
#248
#195


Fix linodes groups

When part of a group, linode label alignment was off, because it is defined in a td as part of the linodes table.
This separates group names from lists (table) of linodes, allowing us to style group headers independently.
Example from @na3d of group labels effecting cell widths:

Example Screenshot with change:

Coverage decreased (-0.01%) to 86.597% when pulling 66330d360a804f586fdd68255b099aa05afbc8ea on mparke:fix-linodes-groups into 269b5d6fbf342a1ee34dd57161a9f179088c6271 on linode:master.

Group labels became table rows recently because consistent column widths was / is a priority. (You will notice that not all columns are the same width all the time -- for instance when displaying backup last taken.) The simplest solution to that being to just use one table.
If you'd like to get rid of the table row group header, please make sure you also deal with the inconsistent column widths.

Inconsistent column widths is not part of this issue.
Not sure what you mean just use one table?
The group name is not being used to represent a column header here, so there is no reason for it to be in a table.

Inconsistent column widths across tables was a solved issue. We dealt with it by joining all groups into one table. But it looks like your changes will reintroduce the issue unless you find a reasonable way to explicitly prevent it.

Link to issue or more detailed description/screencap? I don't follow what problem your referencing.

https://github.com/linode/manager/pull/787

Coverage decreased (-0.01%) to 86.597% when pulling dc2aef280e387917ce8a8cfa0b96e43a2dda7eb1 on mparke:fix-linodes-groups into 269b5d6fbf342a1ee34dd57161a9f179088c6271 on linode:master.

Setting colspan=5 on the group label td seems to fix things fine for now and doesn't reintroduce the potential mixed-width columns across tables:

closing in favor of #1069


Merging some pythonutils parts up to RDFlib

Hi @tgbugs,
In the README for ttlset you state "ttlser cannot produce deterministic results without the changes added in https://github.com/RDFLib/rdflib/pull/649". Well 649's been merged for some time now.
Are you interested in getting this family of serializers into RDFlib core? If they are stable and have value over the default Turtle serializer, surely it would be good to have them as serializer options in core?
Cheers, Nick

@nicholascar Yes, I would love to get some of these into core.
The DeterministicTurtleSerializer has a lot to offer over the default serializer. It is the one that would be by far of most use to the community and is currently implemented in CustomTurtleSerializer https://github.com/tgbugs/pyontutils/blob/f57b7698e9ad83871401ac2c5a8dffbc583220a2/ttlser/ttlser/serializers.py#L155. The behavior is documented here https://github.com/tgbugs/pyontutils/blob/master/ttlser/docs/ttlser.md.
There is at least one known error that must be fixed before we merge. Some malformed nt graphs can induce an infinite loop here https://github.com/tgbugs/pyontutils/blob/f57b7698e9ad83871401ac2c5a8dffbc583220a2/ttlser/ttlser/serializers.py#L105. I haven't had a chance to get that fixed, when I do I will also do a bit of shuffling so that it is easier to move the implementation upstream. I'm guessing they could go in rdflib/plugins/serializers/dturtle.py if we don't want to put them directly in rdflib/plugins/serializers/turtle.py or something like that? The only reason I hesitate is because of the helper functions and classes that are used to implement the deterministic sort.
There are some other considerations that will need to be taken, such as the default value for predicateOrder.
The other one that would probably be of use is the HttpTurtleSerializer https://github.com/tgbugs/pyontutils/blob/f57b7698e9ad83871401ac2c5a8dffbc583220a2/ttlser/ttlser/serializers.py#L808. It makes it clear why
newline and space are abstracted.
CompactTurtleSerializer probably should not go in. It is a completely non-standard implementation of a bad compression algorithm using the serializer that I created in order to come up with a realistic worst possible case when I was testing the performance of the (then new) trie based namespace system. Using a sane serialized with gzip is the right thing to do.
The SubClassOfTurtleSerializer is broken and not implemented correctly so is not ready to go in even if it might be of use to people who would like to have a bit more semantic ordering than purely syntactical ordering as is currently implemented by the DeterministicTurtleSerializer.

@tgbugs Great, well we - my company - has an interest here beyond just "make RDFlib better" in that we too use lots of version controlled turtle files as feedstock for RDF graphs and would love to see this deterministic serialiser implemented to help with that. In about 2013, I created a reasonably deterministic turtle serialiser on top of RDFlib that did some out-of-band (i.e. outside the graph) counting to order BNs etc. but it wasn't heavily tested and I've lost it! I'm happier to help with your implementation which looks miles ahead of where I got to!
Q1: can we shift from the older @prefix style to PREFIX or perhaps provide newer style as the default (PREFIX) with a param to apply older style instead?
Q2: any idea how far off an N3 serialiser this is? I just wonder two things:
1. in RDFlib we might need to replace both the Turtle & N3 serialiser at once, since they are dependent
2. I would like to get into using N3 but will have to learn more about what N3 can do that Turtle can't
Q3: as above but for Trig - how close are these serializers to being able to handle Trig?

The more people that can make use of it the better! The only way I was able to get it to work at all was to try to come up with the most pathological tests cases I could imagine, and I'm sure there are still some lurking out there.
Q1: I'm fairly certain that @prefix has to be lower case in turtle https://www.w3.org/TeamSubmission/turtle/#sec-grammar-grammar. It is certainly possible to add that as a toggle, but I don't think that would go in a pure ttl serializer. Can you point me to a document on this?
Q2: No idea. https://www.w3.org/TeamSubmission/n3/#subsets for reference.  No idea what it would take to get there since I haven't interact with n3 much at all. However where I have interacted with it is in https://github.com/RDFLib/rdflib/blob/master/test/n3/example-lots_of_graphs.n3. That definitely won't serialize. Minimally it would require an additional rule for ranking whole serialized graphs as well as expressions.
Q2.1: The deterministic serializer isn't something that can replace the current default serializer, it comes with a significant performance penalty as well as a significant increase in memory usage. In theory it is possible to write a ttl file to disk that is larger than memory using the standard serializer, as implemented, the current deterministic serializer cannot do that.
Q2.2: Likewise. Named graphs mainly I think, but I try to stay away from anonymous named graphs, so not much perspective there.
Q3: Off the top of my head no idea here as well. The named graphs will definitely give it trouble. https://www.w3.org/TR/trig/#grammar-ebnf

OK, so we add a new serialization option, format="dturtle" or similar for deterministic turtle? Certainly easy to just add another option.
See the ticket Issue #1207 that my engineer @jamiefeiss is working on. I think this would go nicely with the deterministic serializer. You can think of RDF (dturtle) files in Git and then the a Graph being made from that for use. Haven't worked out all the little bits yet but, at the very least, the new file Store would, at start up, look for any changes in Git to see whether it needed to re-deserialize the RDF files or just rely on a cached pickled version.

The options would seem to be dttl dettl or dturtle. I don't really have a preference since I will continue to use nifttl which is configured with predicates that are specific to the needs of our ontology. Maybe both dttl and dturtle? There will need to be a bit of additional guidance for getting good deterministic results or we will need to come up with a set of reasonable defaults so the folks don't have to figure out how to subclass and register a new serializer (which can be a big hurdle).

Yes, I think both dttl & dturtle since there is already ttl == turtle.
We might bypass the infinite loop issue you know about by an input requirement check? Since this will be a secondary and optional Turtle serialiser, it can have some conditions of use (i.e. not handle absolutely all, edge case, RDF)?

I'm fairly certain that @prefix has to be lower case ... Can you point me to a document on this?

See https://www.w3.org/TR/turtle/#h3_sec-iri
Currently RDFlib 5.0.0 supports both in parsing but serializes to @prefix only.

Trig...

I think serialising to Trig will be much simpler than catering for N3. I think the serializer just have to be context aware, i.e. if being called on a ConjunctiveGraph or Dataset as opposed to a Graph then it needs to Turtle each graph and wrap them in <GRAPH-URI> { ... } and just aggregate all PREFIX statements at the top.

@tgbugs how does the determanistic serializer relate/not relate to RDF normalization algorithms like https://json-ld.github.io/normalization/spec/? I always assumed that if a graph could be converted into normalized form then determanistic serialization, in Turtle or JSON-LD or RDF/XML etc would be "easy", i.e. the serialiser could just "start at the start and go to the end"

@tgbugs how does the determanistic serializer relate/not relate to RDF normalization algorithms like https://json-ld.github.io/normalization/spec/? I always assumed that if a graph could be converted into normalized form then determanistic serialization, in Turtle or JSON-LD or RDF/XML etc would be "easy", i.e. the serialiser could just "start at the start and go to the end"

@nicholascar The deterministic ttl serializer was written before I was thinking about normalization or graph identity at all. I also didn't manage to become aware of the normalization algorithm until fairly recently, at which point in time I had already implemented https://github.com/tgbugs/pyontutils/blob/master/pyontutils/identity_bnode.py which provides mostly stable identities for any bnode or graph. When I was first implementing ibnode I tried to reuse the deterministic serialization code, however it was very difficult to adapt, so I wrote the identity from scratch. I haven't had an opportunity to review and compare ibnode and the deterministic serializer the rdf normalization spec as I only became aware of it when I started to dig into json-ld in the last year, but it is likely that the ibnode implementation is quite similar in some respects, but I can't say more than that without comparing the algorithms more closely.
What I can say is that just having some normalized form is no guarantee that you can serialize the graph in the way you want. The deterministic serializer works with any deterministic total ordering on the triples in the graph, which is part of why I was working on the subClassOf serializer.
The issue with trying to go from the normalized form of a graph is that it provides only one total ordering that may or may not be the one that you want. The normalization approach only deals with the normalized form of the graph and as far as I can tell doesn't provide any guidance about what to do about the compacted form. The deterministic serializer was intentionally designed to handle the compacted form since that is what humans end up seeing in version control systems and having the compact identifiers subject to expanded ordering is extremely confusing.
In terms of start at the start and go to the end, yes, if you already have everything in order already. But if it is not in the order you want you will have to reorder it. In my original use case I was accounting for the fact that any change to the prefixes could (and often does) result in a reordering. If you don't care what total order your triples are in, you could store them sorted by the normalization and then serialization would incidentally be deterministic. The issue is what happens if someone changes something in the implementation that has a side effect of changing the serialization ordering.

@nicholascar The deterministic ttl serializer was written before I was thinking about normalization or graph identity at all. I also didn't manage to become aware of the normalization algorithm until fairly recently, at which point in time I had already implemented https://github.com/tgbugs/pyontutils/blob/master/pyontutils/identity_bnode.py which provides mostly stable identities for any bnode or graph. When I was first implementing ibnode I tried to reuse the deterministic serialization code, however it was very difficult to adapt, so I wrote the identity from scratch. I haven't had an opportunity to review and compare ibnode and the deterministic serializer the rdf normalization spec as I only became aware of it when I started to dig into json-ld in the last year, but it is likely that the ibnode implementation is quite similar in some respects, but I can't say more than that without comparing the algorithms more closely.
What I can say is that just having some normalized form is no guarantee that you can serialize the graph in the way you want. The deterministic serializer works with any deterministic total ordering on the triples in the graph, which is part of why I was working on the subClassOf serializer.
The issue with trying to go from the normalized form of a graph is that it provides only one total ordering that may or may not be the one that you want. The normalization approach only deals with the normalized form of the graph and as far as I can tell doesn't provide any guidance about what to do about the compacted form. The deterministic serializer was intentionally designed to handle the compacted form since that is what humans end up seeing in version control systems and having the compact identifiers subject to expanded ordering is extremely confusing.
In terms of start at the start and go to the end, yes, if you already have everything in order already. But if it is not in the order you want you will have to reorder it. In my original use case I was accounting for the fact that any change to the prefixes could (and often does) result in a reordering. If you don't care what total order your triples are in, you could store them sorted by the normalization and then serialization would incidentally be deterministic. The issue is what happens if someone changes something in the implementation that has a side effect of changing the serialization ordering.

@nicholascar I finally got around to fixing the issues with the deterministic serializer and cut a release https://github.com/tgbugs/pyontutils/releases/tag/ttlser-1.1.4.
I think we can revisit the integration question now.

@tgbugs: @aucampia is dead keen to improve, perhaps even replace, serializers, so please communicate with him on this on. Also, I have a student working on Turtle-star processing right now (work in https://github.com/RDFLib/rdflib-rdfstar/) and there he's using a Lark-based parser for Turtle-star which then converts Turtle-star to Turtle. It's probably sensible to move to a Lark-based parser for all Turtle as well as Turtle-star.


(persisted-fetch) - The Node Crypto API is not available.

Using nextjs 12, when trying to use Automatic Persisted Queries using Node lts/fermium, I got this error:
[@urql/exchange-persisted-fetch]: The Node Crypto API is not available.
In order to make it work, I copied the hash.js file locally and replaced the nodeCrypto = new Function('require', 'return require("crypto")')(require);by nodeCrypto = require('crypto')
export const client = createClient({
  url: `${process.env.BACKEND_URL}/graphql`,
  preferGetMethod: true,
  exchanges: [
    dedupExchange,
    cacheExchange({
      updates: {...},
      optimistic: {...},
    }),
    persistedFetchExchange({
      preferGetForPersistedQueries: true,
    }),
    fetchExchange,
  ],
  fetchOptions: () => {
    const token =
      typeof window !== 'undefined' ? localStorage.getItem('jwt') : undefined
    return {
      headers: { authorization: token ? `Bearer ${token}` : '' },
    }
  },
})

urql version & exchanges:
urql: ^2.0.5
urql/exchange-persisted-fetch: ^1.3.2

Expected behavior
No error message
Actual behavior
Error message

We may need a further reproduction here or more information on this; for instance, while it's clear that this may be a problem with Next.js 12, and we could just look into this, in order to reduce work for whomever will pick this up it'd be great to know whether this is happening on the frontend or in Node.js; If it's happening on the frontend it could very well be a Next.js issue with bundling in general, if it's on the backend, it could be a problem with Next.js transpiling the Node.js code incorrectly.
Basically, if you're replacing new Function('require', 'return require("crypto")')(require); this could still be a frontend issue as you're then instructing Webpack to add a rather huge polyfill for the crypto Node API on the client-side. The intention of that code is to explicitly avoid this and never require any polyfill or external code to be added. Hence, this obscures this built-in Node module so that client-side builds never use the "crypto" module.
If this is happening on the backend however, we'll have to see what's happening to require and why we're unable to get the builtin module there.

I understand ! Thank you !
Here is the repo

Hi, I've been checking the provided sample and I think could be related to the new rust compiler Next 12 is using.

I do not think it is related to swc. I tested importing crypto in Next 12.0.4 the same way persisted-fetch does with babel and swc:
if (typeof window === 'undefined') {
    try {
        // Indirect eval/require to guarantee no side-effects in module scope
        // (optimization for minifiers)
        nodeCrypto= new Function('require', 'return require("crypto")')(require);
    } catch (e) {}
}

Results in the same error in both cases: Critical dependency: require function is used in a way in which dependencies cannot be statically extracted and nodeCrypto being undefined. Which is a webpack error.
Using import or require works:
import nodeCrypto from 'crypto';

Using import nodeCrypto from 'crypto' will include it in the bundle unless you explicitly provide an empty stub for it. Maybe Webpack 5/ Next 12 have a new way of dealing with this.

From what I can see using
let nodeCrypto; if (typeof window === 'undefined') {     try {         // Indirect eval/require to guarantee no side-effects in module scope         // (optimization for minifiers)         nodeCrypto = require('crypto');     } catch (e) {} }
Does seem to work. Without theif (typeof window === 'undefined') an additional ~0.5mb of javascript is added to the client bundle.

Sadly that does only automatically work for Next, other applications aren't doing an auto-replace of typeof window 😅

I think what's worth investigating here is how crypto is not loading:

[@urql/exchange-persisted-fetch]: The Node Crypto API is not available.
Because this already implies that typeof window === 'undefined' is returning true, but the require('crypto') call that is eventually done does not succeed.
Critical dependency: require function is used in a way in which dependencies cannot be statically extracted
Regarding this output; It's not a straight-up error. I've seen it before and it shouldn't break the build unless something really odd is going on. It simply should bail out on the require-eval call in question, which is intended

Ah, should have added the actual exception as well:
 ReferenceError: require is not defined
    at file://...//node_modules/@urql/exchange-persisted-fetch/dist/urql-exchange-persisted-fetch.mjs:31:61
    at ModuleJob.run (internal/modules/esm/module_job.js:170:25)
    at async Loader.import (internal/modules/esm/loader.js:178:24)

Ah, not sure if I'm helping or not but the error thrown is:
 ReferenceError: require is not defined
    at file://..//node_modules/@urql/exchange-persisted-fetch/dist/urql-exchange-persisted-fetch.mjs:31:61
    at ModuleJob.run (internal/modules/esm/module_job.js:170:25)
    at async Loader.import (internal/modules/esm/loader.js:178:24)

@stefanaxelsson That's in fact super helpful 👍 so that's very telling; we're apparently in ESM mode in Next.js with Node here and hence can't use require. So that puts us into a spot where we'll either have to choose to split this into browser and non-browser bundles or do a similar approach to this with an async import() in the catch

Hi! So, I haven't built any of the below code at all, but just in case it's useful, here's something I've been hacking and testing out, design-wise.
In sha256.ts, switch from top-level import to async getter function (aka what @kitten suggested):
let _nodeCrypto;
const getNodeCrypto = async () => {
  // Memoize.
  if (typeof _nodeCrypto !== 'undefined') { return _nodeCrypto; }

  // Default. Couldn't resolve (and will stay memoized).
  _nodeCrypto = Promise.resolve(undefined);

  if (typeof window === 'undefined') {
    try {
      _nodeCrypto = await new Function('return import("crypto")')();
    } catch (e) {}
  }

  return _nodeCrypto;
};

Then, consume like:
export const hash = async (query: string): Promise<string> => {
  const nodeCrypto = await getNodeCrypto();
  // ... all the rest of the other code.
};

or in transpiled CJS form:
module.exports.hash = async (query) => {
  const nodeCrypto = await getNodeCrypto();
  // ... all the rest of the other code.
};

This should work with any of the following (assuming transpiled output names):
// CJS
const { hash } = require("./sha256.cjs");
const { hash } = await import("./sha256.cjs");

// ESM
import { hash } from "./sha256.mjs";
const { hash } = await import("./sha256.mjs");

Some caveats:

I don't know if new Function('return import("crypto")')() will correctly keep the crypto lib out of all frontend bundlers. I haven't even tried any. But it's a string, so hopefully?

And it avoids having to use the named require variable which will cause some issues in situations discussed above.

The import() only works in Node.js v12.17.0 (at least without esm flags). So that's going to mean more Node.js runtimes have to use the non-node crypto code.

At any rate, hope that helps if this is a direction we want to experiment with!

<EMAIL_ADDRESS>should be resolving this issue, however, we're hoping that there's no bundlers or setups out there that will run into more issues with the improved handling of ESM Node. So, do let us know if it's working or still broken for anyone, or if there's more environments that we've missed.
We do still want to avoid costly polyfills or JS-only implementations and the native APIs here are very important to us, so if they need further adjustments we'll come back to this once more :v:


re-order html attributes for easy reading issue #292

Description
Re ordered html attributes to current NHS spec following  issue #292
Note: I'm keen to contribute. Let me know if further changes are required to this PR
Checklist

[ ] SCSS
[ ] Nunjucks macro
[ ] A standalone example
[ ] README/Documentation with HTML snippet
[ ] Tested against our testing policy (Resolution, Browser & Accessibility)
[ ] Print stylesheet considered
[ ] CHANGELOG entry

Hey @theodore-q thanks for taking the time to have a look at this! If you’re wanting some more html attributes to reorder, our individual components each have a template.njk file where the HTML for that component lives. Some of the attributes are in the wrong order there.
You can find them in the packages/components/ folder. These get outputted by the nunjucks macro snippet in the docs/components pages.
For example the action link component, the example page lives at docs/components/action-link and the nunjucks macro is called by:
{{ actionLink({
  "text": "Find a minor injuries unit",
   "href": "https://www.nhs.uk/service-search/minor-injuries-unit/locationsearch/551"
}) }}

which outputs the contents of packages/components/action-link/template.njk with the {{ params.text }} and {{ params.href } being the above. The attributes in some of the HTML are in the wrong order.
Thanks again for contributing, we appreciate it!


Adding support for .NET Standard 2.0

This PR adds support for .NET Standard 2.0, aiming at supporting .NET Framework OOP applications.
Adding @mattchenderson for awareness. This will require work on templates and tooling, but I'm not currently looking at that as blockers.
Pull request checklist

[ ] My changes do not require documentation changes

[x] Otherwise: Documentation issue linked to PR (coming. will sync with @mattchenderson )

[ ] My changes should not be added to the release notes for the next release

[x] Otherwise: I've added my notes to release_notes.md

[x] My changes do not need to be backported to a previous version

[ ] Otherwise: Backport tracked by issue/PR #issue_or_pr

[x] I have added all required tests (Unit tests, E2E tests)

/check-enforcer evaluate

Merging. Will follow up with samples and work with @mattchenderson on next steps


Prevent dropdown from going off screen

When I have a dropdown near the bottom or top of the screen and it contains many options, the dropdown options list goes off the screen. How do I prevent it from going off the screen? dropdownPosition would need to calculate the position because right now adding that prop will apply the same position to all dropdowns. I believe this dropdown has this functionality:  https://github.com/sohobloo/react-native-modal-dropdown

Any update here please?
Here is the issue I am facing -

I've fixed the issue by adding dropdownPosition={0}..
Thank you.

thanks @dongawli for the answer, it work for me


Command api:routes only shows Clouse under the Action column.

Maybe this line should use uses instead of controller
https://github.com/dingo/api/blob/master/src/Routing/Route.php#L525

@adrian-fjellberg Are you saying the api:routes command is not showing routes that are defined using controllers? Or that even if its defined with a controller, it still shows Closure?

This might only be a problem when registering resource routes (ie. $api->resource(..)). The reason this is a problem is that the ResourceRegistar class in this package extends the Illuminate\Routing\ResourceRegistrar, and as you can se from this line the Illuminate class specifies the uses keyword only and not the controller keyword. So when registering an API resource the api:routes command does not have a controller key for the routes registed by the $api->resource(..) and therefore only returns Closure. I think this can be fixed if the getActionName method does a search for the uses key if it cannot find the controller key.

@adrian-fjellberg Will aim to get this in before version 1.0.0

Closed via 402680c


nyanmisaka/jellyfin:latest-legacy

IMAGE SYNC

I'm looking for nyanmisaka/jellyfin:10.8.9-legacy. Is there a synchronized image available?


Installer changes

Motivation for this change
This introduces some additional changes on top of the bug fix in #27341. I've put these in a separate PR as they not as obviously correct and probably warrant further discussion. Feedback is very welcome!
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

cc @copumpkin

Bump — any further issues?

Nope, this is probably fine, but I can't test reasonably right now. I guess we can merge and see how it goes?

Sounds like a plan to me, will it happen? :D


Can CogVLM accept interleaved images and texts (multiple images) as input?

Technically supported. But we didn't train with multi-image cases. Maybe fine-tuning is needed.

Technically supported. But we didn't train with multi-image cases. Maybe fine-tuning is needed.

Thank you. How to input multiple images into to model? Is there any specific input format?

First, you need to insert <EOI> in your str to indicate an image to be inserted in text:
https://github.com/THUDM/CogVLM/blob/3c254565408e180207de8a0b69f20473ecd5c157/utils/language.py#L48-L49
Then, you need to modify the word_embedding_forward to correctly insert image embeddings into text embeddings:
https://github.com/THUDM/CogVLM/blob/3c254565408e180207de8a0b69f20473ecd5c157/models/cogvlm_model.py#L84-L91

@McCarrtney My trick is to let CogVLM to summarize the first image, and put the first image's summary together with the second image to chat about both images. It is a bit odd tho, but can work well in my use cases. A downside of this approach is that the current language model of CogVLM cannot understand a large context very well (due to its model size) so you might want to keep the size of summary of the first image small.

Yeah, that's a workaround.


ERROR:rasa_core.training.interactive:An exception occurred while recording messages.

Rasa Core version:
rasa-nlu ==0.14.1
rasa-coe == 0.13.0
rasa-core-sdk==0.12.1
Python version: 3.6.8(64bit)
Operating system (windows, osx, ...):windows
Issue:
I try to train the model online but everytime i get the following error
Chat History

 #    Bot                                                                       You
───────────────────────────────────────────────────────────────────────────────────────────
 1    action_listen
───────────────────────────────────────────────────────────────────────────────────────────
 2                                                                                hey
                                                                        intent: greet 0.72
───────────────────────────────────────────────────────────────────────────────────────────
 3    utter_greet 1.00
      Hello! How can I help?
      utter_ask_category
      can you choose the category you are interested in?
      Buttons:
      1: Tag
      (/inform{"category":"Tag"})
      2: Market (inform{"category":"Market"})
      action_listen
───────────────────────────────────────────────────────────────────────────────────────────
 4                                                            /inform{"category":"Market"}
                                                                       intent: inform 1.00
───────────────────────────────────────────────────────────────────────────────────────────
 5    slot{"category": "Market"}

Current slots:
        category: Market, name: None

------
? What is the next action of the bot?  0.08 action_listen
Thanks! The bot will now run action_listen.

ERROR:rasa_core.training.interactive:An exception occurred while recording messages.
Traceback (most recent call last):
  File "C:\Users\swarnalathaa\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\training\interactive.py", line 1216, in record_messages
    finetune, sender_ids, plot_file)
  File "C:\Users\swarnalathaa\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\training\interactive.py", line 824, in _predict_till_next_listen
    _plot_trackers(sender_ids, plot_file, endpoint)
  File "C:\Users\swarnalathaa\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\training\interactive.py", line 1151, in _plot_trackers
    max_history=2)
  File "C:\Users\swarnalathaa\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\training\visualization.py", line 448, in visualize_neighborhood
    _merge_equivalent_nodes(graph, max_history)
  File "C:\Users\swarnalathaa\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\training\visualization.py", line 200, in _merge_equivalent_nodes
    _nodes_are_equivalent(graph, i, j, max_history)):
  File "C:\Users\swarnalathaa\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\training\visualization.py", line 145, in _nodes_are_equivalent
    _fingerprint_node(graph, node_b, max_history)))
  File "C:\Users\swarnalathaa\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\training\visualization.py", line 111, in _fingerprint_node
    for continuation in continuations}
  File "C:\Users\swarnalathaa\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\training\visualization.py", line 111, in <setcomp>
    for continuation in continuations}
TypeError: sequence item 1: expected str instance, dict found

Content of domain file (if used & relevant):
slots:
  category:
    type: text
  name:
    type: text

intents:
 - greet
 - goodbye
 - inform

entities:
  - category
  - name

templates:
  utter_ask_category:
    -
      buttons:
        -
          payload: "/inform{\"category\":\"Tag\"}"
          title: Tag
        -
          payload: "inform{\"category\":\"Market\"}"
          title: Market
      text: "can you choose the category you are interested in?"

  utter_dont_understand:
    - "Sorry, I dont understand what you are asking for :("
  utter_goodbye:
    - "Talk to you later."
    - "Bye bye :("
  utter_greet:
    - "Hello! How can I help?"

actions:
 - utter_greet
 - utter_goodbye
 - utter_ask_category
 - utter_dont_understand
 - action_news

Thanks for raising this issue, @MetcalfeTom  will get back to you about it soon.

Hi @Swarnalathaa,
I believe this is because the intent in your second button is not formatted properly - please ensure the payload is mapped to (/inform{"category":"Market"}).  At the moment it seems you are missing the / character and returning the intent in the button text.
If this fixes the issue, please come back here and leave a comment because we can also prevent this error in future

Hi @MetcalfeTom ,
Thanks for your reply.
No, it didn't work.
I still get this error even if i remove the button option completely from the bot.
Following screenshot is from running bot after removing button option, but still i get the same kind of error

Ahh OK, it is my understanding that is a bug with our visualisation script that arises when setting the same slot twice in one turn.  We can work a fix for this, however in the meantime I would recommend trying:

Running the script with the flag --skip_visualization
Creating a separate action 'reset_slots' which is predicted in your story data after -action_news.  For this you just need the action to return the AllSlotsReset() event from rasa_core_sdk.events

Thanks for bringing this to our attention!

Hi @MetcalfeTom ,
I followed the above suggestion and I am not getting the error. Thanks :)

I was getting this same error. Before I could look into it it disappeared again and I can't seem to reproduce it, having not changed the code or any settings.

hi @MetcalfeTom,
can you please explain the steps
Ahh OK, it is my understanding that is a bug with our visualisation script that arises when setting the same slot twice in one turn. We can work a fix for this, however in the meantime I would recommend trying:
Running the script with the flag --skip_visualization
Creating a separate action 'reset_slots' which is predicted in your story data after -action_news. For this you just need the action to return the AllSlotsReset() event from rasa_core_sdk.events
Thanks for bringing this to our attention!

Hi @shanushawan ,

If you are getting error something like the one mentioned above during interactive training then try to run your interactive training by disabling visualization. check this link https://rasa.com/docs/core/interactive_learning/

Since I wanted to reset all slots after the action_news has been performed, I made a separate action in actions.py which will reset all the slots.

class Actionreset(Action):
def name(self):
    return 'action_reset'

def run(self, dispatcher, tracker, domain):
    return [AllSlotsReset()]

and in my stories I need to call this action after action_news to reset all slots.
Hope this helps :)

@Swarnalathaa i followed same steps, but visualization seems to be still running for me.can you tell what command exactly you used ?

@Swarnalathaa i got it working. And the error is gone for now @MetcalfeTom hope to see fix soon.

@Swarnalathaa thank you! it's working for me

hello.
How can I simulate this error and test solution for my tcc?

I tried to run --skip_visualization but it still doesn't stop the visualization.

@Nexemics you are doing it wrongly.If you are using script then while calling run_interactive_learning pass skip_visualization=True
like this : interactive.run_interactive_learning(agent,training_data_file,skip_visualization=True)
or through terminal you can do like this:
python -m rasa_core.train interactive -o models/dialogue -d domain.yml -c policy_config.yml -s data/stories.md --skip_visualization --nlu models/current/nlu --endpoints endpoints.yml

Yeah I followed the first method and got it working. Thanks mate!

@MetcalfeTom does this require any fixes from our side or can this be closed?

@Griffin98 This solution worked for me


Meaning of shuffle when using tfdata in Keras

Hey, so basically when using tf data (where you can specify a shuffle operation in the dataset). What is the meaning of shuffle in the fit operation? Will it also shuffle the data, should you specify it? Which shuffle will count the one from tf data or the fit function?

Bye the way from my experiments i get much better generalization if I do the shuffeling with tf data. The Keras shuffle is propabely meaningless when using tf data. Maybe it should throw an error?


fix Container#disconnect calling each_key instead of each_value on ga…

…teways hash

Lemme know if you need help with coming up with a test case for that fix.


node: fix certificate.committee-size on testnet

Motivation
Fixes #6578.
Description
Only use half of pre-upgrade hare committee size if certificate.committee-size is set to 0.
Otherwise, use certificate.committee-size which is 200 for testnet, with CertifyThreshold being 101.
Test Plan
Verify that node syncs against testnet.

bors merge


Limits to number of "copy compiler" you can have

I think this screenshot can tell everything you need :)

What i did was just spamming the "copy compiler" button until i got that mess.
The feature im refering to is some kind of limit to how many "copy compiler"'s you can have.

Nothing we implement client side could effectively mitigate potential denial of service vectors. A malicious user could simply open multiple tabs to circumvent any limit, or more likely capture and replay valid web requests from a script to cut the web browser out of the equation entirely.
I don't believe we have ever experienced a malicious DoS, and in general I would say we are pretty low risk on that regard - there just isn't really much to be gained by bringing our site down.
If we were concerned we could implement server side rate limiting using Amazon's WAF offering, but until it's proven to be needed we would just be flushing money down the toilet.

Hi! Thanks for your report, we really appreciate that you took the time to let us know about it, but as @apmorton said, it's very unlikely that we'd add a client-side solution for this, as it's a really low priority issue. - Also note that simply spamming the clone button does not do much by itself, as every subsequent compilation should hit the cache.
That said, we'd gladly accept any PR addressing this, for anyone interested. :)


Enable getting segmentations at any threshold

Until now, the RAG would output the currently implied segmentation
when using the get_segmentation method. However, since we are storing
the entire merge tree, it is efficient to get segmentations at any
threshold. This commit enables that by passing a threshold to
self.tree.get_map.
See question (5) in this thread on the mailing list for motivation.

Coverage increased (+0.01%) to 40.1% when pulling 53dee1fa8e5eae58882a665c3c83715fa4b453c3 on jni:get-any-segmentation into 403543a4b1ba94cd3cd2c42a26be20d7dcce8b17 on janelia-flyem:master.


⚠ Bofrost.at has degraded performance

In a6e03ed, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1954 ms

Resolved: Bofrost.at performance has improved in 0e3bc62 after 15 minutes.


[Resources] Add deployment scripts resource type to the Resources SDK

The new preview version of Resources SDK will support deployment scripts resource type. This new resource type is added to the Resources namespace.
Question for the reviewer
There was a name clash while generating the Page.cs - I renamed our Page.cs file temporarily. I see that there are already two separate page classes in the SDK (see Page.cs and Page1.cs) Should we rename our page class to Page2? If so, is there a automated way of doing it through Autorest(AFAIK there is none) or should we do it manually? I would think doing it manually will create extra work in the future if/when we re-generate the SDK since we need to rename the class again. It'd be good to know if this can be done through Autorest.
Another option would be using the class that is already in the project. Though this also might not be ideal since we'll be tying our code to a class that is being generated by some other swagger.
What would be a good way to solve this issue? Your help in this matter is appreciated.

@AlexGhiondea @kurtzeborn Hi, resources team met something like an autorest bug. Could you help investigate? Thanks

Hi @filizt , you'll need the following things:

Add a link to the swagger spec review PR in the description
Include the .txt file generated when you run generate.ps1(details)
Update versions, tags, if necessary
Tests
Thanks

@erich-wang could you help with this?

Where is the spec that contains the schema that is called page?

Where is the spec that contains the schema that is called page?

Here's the swagger that we use to generate our SDK. The page.cs gets created by the autorest since we have a pagable list call.
"x-ms-pageable": {           "nextLinkName": "nextLink"         }

Ok, I'm thoroughly confused here.
Are you saying the generator is creating a third Page.cs class in addition to the Page and Page1 classes in Models?
Where does the extra Page.cs  that you have to delete come from?

Another option would be using the class that is already in the project. Though this also might not be ideal since we'll be tying our code to a class that is being generated by some other swagger.

The Page.cs class that generated by the c# generator will be the same no matter which swagger file created it.
And given that no new work is going into that generator, it's not going to change.

Hi @filizt , you'll need the following things:

Add a link to the swagger spec review PR in the description
Include the .txt file generated when you run generate.ps1(details)
Update versions, tags, if necessary
Fix existing tests and write new ones to cover new code
Thanks

@isra-fel I have included the generated .txt file, updated versions, and tags. Could you see if I'm missing anything?
I'm recording tests and will submit them soon.

Ok, I'm thoroughly confused here.
Are you saying the generator is creating a third Page.cs class in addition to the Page and Page1 classes in Models?
Where does the extra Page.cs that you have to delete come from?

Another option would be using the class that is already in the project. Though this also might not be ideal since we'll be tying our code to a class that is being generated by some other swagger.

The Page.cs class that generated by the c# generator will be the same no matter which swagger file created it.
And given that no new work is going into that generator, it's not going to change.

@fearthecowboy Sorry for the confusion. Let me try to elaborate.
I think the root of my question is coming from the fact that there are two Page files in Models in the Microsoft.Resources SDK: Page.cs and Page1.cs.  I'm curious to find out why that is and if autorest tool created these two separate Page files.
Here're the Page files I am referring to: https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/resources/Microsoft.Azure.Management.Resource/src/Generated/Models

The C# generator did create both of those.
I'm not entirely sure why it does that. It's not going to affect anything, there's not any reason to do anything with them.

Hi @filizt , you'll need the following things:

Add a link to the swagger spec review PR in the description
Include the .txt file generated when you run generate.ps1(details)
Update versions, tags, if necessary
Fix existing tests and write new ones to cover new code
Thanks

@isra-fel I have included the generated .txt file, updated versions, and tags. Could you see if I'm missing anything?
I'm recording tests and will submit them soon.

Submited the test recordings. Also updated the version number as per out discussion. Is there anything else missing from the PR?

Submited the test recordings. Also updated the version number as per out discussion. Is there anything else missing from the PR?

Looks good to me


~[ZWATCH_S.E.X√NOW!] Sofia Ansari Viral Leaked Video 2024 LINK On Social Media X Telegram [Trending√FULL70K]

01 sec ago Sofia Ansari Leaked Video Viral On Social Media. ~[ZWATCH_S.E.X√NOW!] Sofia Ansari Viral Leaked Video 2024 LINK On Social Media X Telegram [Trending√FULL70K]

Sofia Ansari's latest video has taken the internet by storm, becoming a viral hit that has captivated social media users and ignited widespread discussions. This rising digital creator, celebrated for her captivating content, especially her recent Spiderman-themed video, has experienced a notable increase in interest from fans eager to discover where they can view the complete version. Many are flocking to platforms like Instagram, X (formerly Twitter), and Discord to access her work.
As the buzz grows, fans are enthusiastically sharing their thoughts online, demonstrating the immense power of social media in propelling creators like Sofia Ansari into the limelight. With her distinctive style and imaginative approach, she has rapidly made a name for herself in the digital realm, leaving many eager to learn about her future projects and content.
[Trending√_S.E.X√NOW!] Sofia Ansari Viral Leaked Video 2024[FULL70K]

thank you

FG

vf

FG

vf

FG

vf

dsfdsf


[webui] show "no results" on search page

Story description

go to https://mrvivacious.github.io/My_Bhagavad_Gita/
click on Search
Search for "mrvivacious"
note that Search has no results

Acceptance criteria:
Pull request is created.
In that request, a search with no results should show the text "no results found for query"
Work description:
This is where the output could go
You might want to do a check for the resulting output before this line

Ah sorry I merged before seeing your hacktoberfest comment. I'll add the label to this issue later today and then I'll reference your commit here as a comment 👍

i added the label

resolving thanks to https://github.com/mrvivacious/My_Bhagavad_Gita/commit/1e555350646e50ad4e037938e089b0613576029e

Assigned u


MM-T702 Edit to invalid URL

Summary
Added a end to end test to verify that editing a custom slash with invalid url results in an error
Ticket Link
MM-T702

reopened the PR here https://github.com/mattermost/mattermost-webapp/pull/6160


Types not exported through package.json

The types property of package.json is no longer set in the latest @next version.
The property was removed in this commit: https://github.com/jasonkuhrt/graphql-request/commit/c3a309fcb2fcd0bcdcd4f530bb4035490f68c682#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L18
Was this deliberate?

@lswinblad Yes, types is an obsolete field. TS with the "right" (modern) setup will consider types as specified in the exports field.


Will insert a gutter at the beginning of the current editing table cell from Obsidian v.1.5.*

The plugins lapel and obsidian42-strange-new-worlds both insert a gutter to a cell, so the current workaround is to make sure both are turned off if you want to remove the gutter. Waiting for a fix. Thank you for developing and maintaining this plugin! ❤

A quick fix is to add this line to your CSS snippets:
.table-cell-wrapper .cm-gutters { display: none !important; }

Thanks! Fixed in 0.1.5


[5.6] Fix call to "setConnection" method that is not present in the "PresenceVerifierInterface"

Before this PR, when I tried to define a custom implementation for the PresenceVerifierInterface in the Validator, I received a "call to undefined method CustomPresenceVerifier::setConnection()" error message, because the interface does not have that method. So I just added an if statement checking if the verifier object has the setConnection method, so that it won't break the existing default implementation (DatabasePresenceVerifier).
Specifically, this method was called when the "unique" and "exists" rules were evaluated. I added a test as well...

Would rather just re-think the interface than check for method_exists.


Option to exclude file extensions from tab completion/expansion results

Summary of the new feature/enhancement
As a user I want to have an option to hide the file extensions for executables and executable scripts known to the system, when using the tab expansion feature of Powershell.
This would allow me to use them like I'm used to, because I do not care about the file extension.
Proposed technical implementation details (optional)
Hide the file extension when using tab expansion to expand commands located in $env:PATH that have extensions listed in the $env:PATHEXT.
This would make the behavior similar to how the search for executable scripts work.
Example 1:

I have a script called My-Function.ps1 in a folder in my $env:PATH
I can run it by typing My-Function as the search inside the path includes .ps1 files
When I type My<tab>, it expands to My-Function.ps1
Instead My<tab> should expand to My-Function as the extension is redundant / not needed

Example 2:

I type powersh<tab> and it expands to powershell and not powershell.exe

How the extension can be ignore, what if there a folder with the same name ?

@kvprasoon powershell already works like this. You don't have to write the extension and it still works. Hence tab completion should work the same way.
Just try it with cmd. You can run it like this, but if you press <tab>, you get cmd.exe, which is the same.

That applies to executables located via $env:Path, but I don't know that it applies to all commands or scripts in arbitrary locations?

@vexx32 Actually, that's what I meant. I guess I was not precise enough, going to update the original post.

I type powersh and it expands to powershell and not powershell.exe

PowerShell do tab complete it to PowerShell.exe . What is you PowerShell version ?

@kvprasoon that's the request at hand; he wants it to expand to only powershell, not completing the extension, since it will run just fine without it.

So the ask is to change tab completion. @vexx32 do you see any ROI for this ?

I suppose it might make some sense in terms of increasing UX consistency between what tab completes and what actually can be executed?
I don't think it's overly complicated to implement, but I'm not sure exactly how this argumentcompleter is handled, so it might wind up being more complicated depending on the existing implementation... Hard to say without digging up the code, unfortunately. 😄

What's the actual real benefit here? Because TabCompletion is for completing to Full Command names which in this case includes the .exe or .ps1 whilst the engine will look to the command values you can see with the below you could technically brake all scripts that don't reference the full powershell.exe with the following
function powershell { "Whoops you broke using PowerShell for PowerShell.exe in the terminal due to command preference - for more info run 'Get-Help about_Command_Precedence'"}

Then tab complete and you would have a PowerShell Function as well as the PowerShell Application powershell.exe
for more info on command preference have a look at  Get-Help about_Command_Precedence
Also - This would make the behavior similar to how the search for executable scripts work. - technically this is incorrect as per the below tabcompletion for slmgr.vbs which is an executable script and also shows the extension as per mentioned above (and IMO always should)
Cd C:\
slmg

The reason at the prompt I think that you should always be shown the file extension is that you may have scripts in different languages but are still executable scripts and share the same file name - build scripts are like this on xPlat projects
Take the following (in an admin prompt if you have v6 installed then this will work as it's a location on $env:Path)
New-Item "C:\Program Files\PowerShell\6\slmgr.ps1" -Value 'Write-Host "This is not the Slmgr vb executable script"'

now in the console type slm and tab complete
That's when you need to have the file extension and why I can't see any real benefit behind removing the extension from the returned completed commands and IMO actually hampers the UX - yes the extension may be surplus to most requirements but as it doesn't impact performance by doing what it's intended to do, whereas your suggested change would add lots of overhead with potential perf losses and be of marginal gain to not only the everyday Windows Scripter but also be a possible downturn on the xPlat ones as well.
Just my thoughts on this :-)

@kvprasoon powershell already works like this. You don't have to write the extension and it still works. Hence tab completion should work the same way.
Just try it with cmd. You can run it like this, but if you press <tab>, you get cmd.exe, which is the same.

For some commands, the behavior when tab-completed is not the same.
I have Debian installed in wsl.
In powershell, when I enter Debian or debian, I get the Debian running inside the same window as powershell.
But if I type Debian or debian, then press tab, in either one gets completed to Debian.bat. If I then hit enter, a separate console window is spawned and Debian runs inside that instead.


Shorts appear on notifications

I am sorry if this is not fixable or I have done something wrong, but when I opened notifications I got a youtube short there.
It happens kinda frequently and I hope this is fixable.
This is an example of that:

I don't think this is possible, but I will try to look into it to be 100% sure. I think I could hide the notification if it is a short, but I think the main problem would be the notification dot that appears when there are open notifications. I can't hide that based on the video types in the notifications.


Materialized views take up a lot of disk

Steps to reproduce the behavior (Required)
Expected behavior (Required)
Real behavior (Required)

StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()

This can not reproduce so close it first.

the reason is that dropped partitions of insert overwrite go into trash and not deleted directly. trash occupies too much disk space.
I will fix it by delete dropped partitions data directly


Build Rviz for ROS2 fails, lots of ament_add_gmock errors

I build rviz of version 8.2.8 for arm ros2 in a docker with image ros:foxy --platform=arm64 (from https://discourse.ros.org/t/announcing-ros-foxy-docker-images-availability/14511), and after I fix some errors due to lack of third-party libraries, I am stucked in the following errors:
`
root@9e1882f604d9:/home/michael/work/src/rviz2# colcon build --merge-install
Starting >>> rviz_assimp_vendor
Starting >>> rviz_ogre_vendor
Finished <<< rviz_assimp_vendor [2.63s]
Finished <<< rviz_ogre_vendor [2.69s]
Starting >>> rviz_rendering
--- stderr: rviz_rendering
CMake Error at /opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:69 (add_executable):
Target "point_cloud_test_target" links to target
"rviz_ogre_vendor::OgreOverlay" but the target was not found.  Perhaps a
find_package() call is missing for an IMPORTED target, or an ALIAS target
is missing?
Call Stack (most recent call first):
/opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:52 (_ament_add_gmock)
CMakeLists.txt:186 (ament_add_gmock)
CMake Error at /opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:69 (add_executable):
Target "wrench_visual_test_target" links to target
"rviz_ogre_vendor::OgreOverlay" but the target was not found.  Perhaps a
find_package() call is missing for an IMPORTED target, or an ALIAS target
is missing?
Call Stack (most recent call first):
/opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:52 (_ament_add_gmock)
CMakeLists.txt:271 (ament_add_gmock)
CMake Error at /opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:69 (add_executable):
Target "string_helper_test" links to target "rviz_ogre_vendor::OgreOverlay"
but the target was not found.  Perhaps a find_package() call is missing for
an IMPORTED target, or an ALIAS target is missing?
Call Stack (most recent call first):
/opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:52 (_ament_add_gmock)
CMakeLists.txt:181 (ament_add_gmock)
CMake Error at /opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:69 (add_executable):
Target "point_cloud_renderable_test_target" links to target
"rviz_ogre_vendor::OgreOverlay" but the target was not found.  Perhaps a
find_package() call is missing for an IMPORTED target, or an ALIAS target
is missing?
Call Stack (most recent call first):
/opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:52 (_ament_add_gmock)
CMakeLists.txt:198 (ament_add_gmock)
CMake Error at CMakeLists.txt:171 (add_library):
Target "rviz_rendering_test_utils" links to target
"rviz_ogre_vendor::OgreOverlay" but the target was not found.  Perhaps a
find_package() call is missing for an IMPORTED target, or an ALIAS target
is missing?
CMake Error at /opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:69 (add_executable):
Target "billboard_line_test_target" links to target
"rviz_ogre_vendor::OgreOverlay" but the target was not found.  Perhaps a
find_package() call is missing for an IMPORTED target, or an ALIAS target
is missing?
Call Stack (most recent call first):
/opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:52 (_ament_add_gmock)
CMakeLists.txt:210 (ament_add_gmock)
CMake Error at CMakeLists.txt:67 (add_library):
Target "rviz_rendering" links to target "rviz_ogre_vendor::OgreOverlay" but
the target was not found.  Perhaps a find_package() call is missing for an
IMPORTED target, or an ALIAS target is missing?
CMake Error at /opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:69 (add_executable):
Target "covariance_visual_test_target" links to target
"rviz_ogre_vendor::OgreOverlay" but the target was not found.  Perhaps a
find_package() call is missing for an IMPORTED target, or an ALIAS target
is missing?
Call Stack (most recent call first):
/opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:52 (_ament_add_gmock)
CMakeLists.txt:222 (ament_add_gmock)
CMake Error at /opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:69 (add_executable):
Target "grid_test_target" links to target "rviz_ogre_vendor::OgreOverlay"
but the target was not found.  Perhaps a find_package() call is missing for
an IMPORTED target, or an ALIAS target is missing?
Call Stack (most recent call first):
/opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:52 (_ament_add_gmock)
CMakeLists.txt:234 (ament_add_gmock)
CMake Error at /opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:69 (add_executable):
Target "movable_text_test_target" links to target
"rviz_ogre_vendor::OgreOverlay" but the target was not found.  Perhaps a
find_package() call is missing for an IMPORTED target, or an ALIAS target
is missing?
Call Stack (most recent call first):
/opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:52 (_ament_add_gmock)
CMakeLists.txt:246 (ament_add_gmock)
CMake Error at /opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:69 (add_executable):
Target "movable_text_test_target" links to target
"rviz_ogre_vendor::OgreOverlay" but the target was not found.  Perhaps a
find_package() call is missing for an IMPORTED target, or an ALIAS target
is missing?
Call Stack (most recent call first):
/opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:52 (_ament_add_gmock)
CMakeLists.txt:246 (ament_add_gmock)
CMake Error at /opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:69 (add_executable):
Target "line_test_target" links to target "rviz_ogre_vendor::OgreOverlay"
but the target was not found.  Perhaps a find_package() call is missing for
an IMPORTED target, or an ALIAS target is missing?
Call Stack (most recent call first):
/opt/ros/foxy/share/ament_cmake_gmock/cmake/ament_add_gmock.cmake:52 (_ament_add_gmock)
CMakeLists.txt:259 (ament_add_gmock)
CMake Generate step failed.  Build files cannot be regenerated correctly.
Failed   <<< rviz_rendering [3.88s, exited with code 1]
Summary: 2 packages finished [7.82s]
1 package failed: rviz_rendering
1 package had stderr output: rviz_rendering
5 packages not processed
`
I installed may packages of ros-ament-*, but can't fix them.
root@9e1882f604d9:/home/michael/work/src/rviz2# apt list ros-*-ament* | grep installed ros-foxy-ament-cmake-auto/focal,now 0.9.12-1focal.20230527.034115 arm64 [installed,automatic] ros-foxy-ament-cmake-copyright/focal,now 0.9.8-1focal.20230527.040734 arm64 [installed,automatic] ros-foxy-ament-cmake-core/focal,now 0.9.12-1focal.20230527.032629 arm64 [installed,automatic] ros-foxy-ament-cmake-cppcheck/focal,now 0.9.8-1focal.20230527.041005 arm64 [installed,automatic] ros-foxy-ament-cmake-cpplint/focal,now 0.9.8-1focal.20230527.044538 arm64 [installed,automatic] ros-foxy-ament-cmake-export-definitions/focal,now 0.9.12-1focal.20230527.033821 arm64 [installed,automatic] ros-foxy-ament-cmake-export-dependencies/focal,now 0.9.12-1focal.20230527.033904 arm64 [installed,automatic] ros-foxy-ament-cmake-export-include-directories/focal,now 0.9.12-1focal.20230527.033816 arm64 [installed,automatic] ros-foxy-ament-cmake-export-interfaces/focal,now 0.9.12-1focal.20230527.033600 arm64 [installed,automatic] ros-foxy-ament-cmake-export-libraries/focal,now 0.9.12-1focal.20230527.033536 arm64 [installed,automatic] ros-foxy-ament-cmake-export-link-flags/focal,now 0.9.12-1focal.20230527.033819 arm64 [installed,automatic] ros-foxy-ament-cmake-export-targets/focal,now 0.9.12-1focal.20230527.033602 arm64 [installed,automatic] ros-foxy-ament-cmake-flake8/focal,now 0.9.8-1focal.20230527.041059 arm64 [installed,automatic] ros-foxy-ament-cmake-gmock/focal,now 0.9.12-1focal.20230527.040917 arm64 [installed,automatic] ros-foxy-ament-cmake-gtest/focal,now 0.9.12-1focal.20230527.040644 arm64 [installed,automatic] ros-foxy-ament-cmake-include-directories/focal,now 0.9.12-1focal.20230527.033849 arm64 [installed,automatic] ros-foxy-ament-cmake-libraries/focal,now 0.9.12-1focal.20230527.033841 arm64 [installed,automatic] ros-foxy-ament-cmake-lint-cmake/focal,now 0.9.8-1focal.20230527.040626 arm64 [installed,automatic] ros-foxy-ament-cmake-pep257/focal,now 0.9.8-1focal.20230527.041105 arm64 [installed,automatic] ros-foxy-ament-cmake-pytest/focal,now 0.9.12-1focal.20230527.043854 arm64 [installed,automatic] ros-foxy-ament-cmake-python/focal,now 0.9.12-1focal.20230527.033817 arm64 [installed,automatic] ros-foxy-ament-cmake-ros/focal,now 0.9.2-1focal.20230527.045739 arm64 [installed,automatic] ros-foxy-ament-cmake-target-dependencies/focal,now 0.9.12-1focal.20230527.033914 arm64 [installed,automatic] ros-foxy-ament-cmake-test/focal,now 0.9.12-1focal.20230527.033843 arm64 [installed,automatic] ros-foxy-ament-cmake-uncrustify/focal,now 0.9.8-1focal.20230527.040848 arm64 [installed,automatic] ros-foxy-ament-cmake-version/focal,now 0.9.12-1focal.20230527.033844 arm64 [installed,automatic] ros-foxy-ament-cmake-xmllint/focal,now 0.9.8-1focal.20230527.041123 arm64 [installed,automatic] ros-foxy-ament-cmake/focal,now 0.9.12-1focal.20230527.034003 arm64 [installed,automatic] ros-foxy-ament-copyright/focal,now 0.9.8-1focal.20230527.040308 arm64 [installed,automatic] ros-foxy-ament-cppcheck/focal,now 0.9.8-1focal.20230527.033907 arm64 [installed,automatic] ros-foxy-ament-cpplint/focal,now 0.9.8-1focal.20230527.044409 arm64 [installed,automatic] ros-foxy-ament-flake8/focal,now 0.9.8-1focal.20230527.034018 arm64 [installed,automatic] ros-foxy-ament-index-cpp/focal,now 1.1.0-1focal.20230527.044824 arm64 [installed,automatic] ros-foxy-ament-index-python/focal,now 1.1.0-1focal.20230527.040511 arm64 [installed,automatic] ros-foxy-ament-lint-auto/focal,now 0.9.8-1focal.20230527.034012 arm64 [installed,automatic] ros-foxy-ament-lint-cmake/focal,now 0.9.8-1focal.20230527.040514 arm64 [installed,automatic] ros-foxy-ament-lint-common/focal,now 0.9.8-1focal.20230527.044618 arm64 [installed,automatic] ros-foxy-ament-lint/focal,now 0.9.8-1focal.20230527.033909 arm64 [installed,automatic] ros-foxy-ament-package/focal,now 0.9.5-1focal.20210422.231141 arm64 [installed,automatic] ros-foxy-ament-pep257/focal,now 0.9.8-1focal.20230527.040159 arm64 [installed,automatic] ros-foxy-ament-uncrustify/focal,now 0.9.8-1focal.20230527.034242 arm64 [installed,automatic] ros-foxy-ament-xmllint/focal,now 0.9.8-1focal.20230527.040536 arm64 [installed,automatic] ros-foxy-launch-testing-ament-cmake/focal,now 0.10.10-1focal.20230527.045532 arm64 [installed,automatic]
How to fix them?
Thanks.

Hi @michaelbcd,
foxy is End of life, I would not expect that this should work. You should use an updated version of ROS 2.
In the future, please use one of the supported channels for help requests like this, such as the Robotics Stack Exchange. GitHub tickets should be used for bugs and enhancement tracking, not technical assistance.

Hi @ahcorde,
OK, thanks. I searched this issue, and found some similar ones on this web, so I expected to get some answers or recommendations here.
I'll ask it on Robotics Stack Exchange https://robotics.stackexchange.com.


Corrected version of #126, with checkstyle amendments

In order to properly support this does the following :

Adds Clock-keeping to relevant commands, as well as Model, and by extension Logic. Semantically, this still makes sense, as these classes are in charge of the overall system, which would reasonably need to keep time in order to timestamp entries.
Expose API methods to set & get the Clock used.
Initialises previously-added field in Creates LastModifiedDateTime field in Person #124 to be time-stamped whenever add or edit is executed.

Does NOT :

Expose user interface to change clock used. Consider as a future possible addition.
Allow for serialisation of clock used. Currently, just uses system default.

Test cases as amended/added in this PR were all passed on my machine.
Closes #111.

From #126

Missing test coverage includes the following :

Accidental double-tagging in SampleDataUtil
AddressBookParser#parse, which is the exact same function signature as AddressBookParser#parseCommand, but added due to AddressBookParser now implementing a new interface introduced in this commit, As suggested, it seems logical to refactor all calls to parseCommand to instead call parse, and move parsing logic under parse, so we make use of more shared interfaces.

Missing test coverage is hence (mostly) trivial.


implementation guideline

Resolves #
Depends on: #471
Description
mostly worked on the python class tuff

[ ] applied state
[ ] relation-specific drop and rename
[ ] modify materializations according to above
[x] (not all the way done yet) Python class configs
[ ] new MV tests

Checklist

[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change to the "dbt-databricks next" section.

Please see tests failing re: hologram removal.

@dataders is this ready for me to take over or do you still plan on another commit?

@dataders is this ready for me to take over or do you still plan on another commit?

not even close. 🥲  i'll be working on this more today and tomorrow. stay tuned.
also dbt-spark 1.6.0b2 (which includes the hologram work https://github.com/dbt-labs/dbt-spark/pull/880) is being cut as we speak. this'll obviate my most recent commit

@dataders status update?  At any point feel free to hand off and give me instructions on what is remaining :).

@dataders status update? At any point feel free to hand off and give me instructions on what is remaining :).

you @benc-db ask and you shall receive :) #473

@dataders status update? At any point feel free to hand off and give me instructions on what is remaining :).

you @benc-db ask and you shall receive :) #473

Ok, taking it now.

@benc-db, after discussing internally, we're dropping this MV refactor work as a requirement for 1.7.0rc1 or even 1.7.0, but since you already have your hands on it, I wanted to show the next step. I was afraid to push directly to the branch in this PR, so I made https://github.com/dataders/dbt-databricks/pull/1 that you can pull in if you like. #477 gives more context