Add new EKS K8S versions

Description
Safety checklist

[x] This change is backwards compatible and safe to apply by existing users
[x] This change will NOT lead to data loss
[x] This change will NOT lead to downtime who already has an env/service setup

How has this change been tested, beside unit tests?
YOUR_ANSWER

Several deprecated API versions are completely dropped in 1.25 - batch/v1beta1/Cronjob, policy/v1beta1/PodDisruptionBudget. As such, in order to add support, the linkerd helm chart in aws_eks_base needs to be upgraded to 2.12.x, which is a pretty involved update: https://linkerd.io/2.12/tasks/upgrade/#upgrade-notice-stable-2-12-0.
Leaving these changes out of this PR, but we could consider adding support for K8S versions >=1.25 in a subsequent one.


Update hello-world.md

Why:
Closes [issue link]

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

@codergolu21 Thanks for opening a PR! ⚡ However, we are not going to accept this contribution. Please check out our style guide for more information on how we use headers. ✍
Please take a look at our help wanted section to find open issues you can work on.
Thank you for your interest in improving GitHub Docs! :sparkling_heart:


Wrong reason for GoDaddy/secureserver.net diagnostic code; 5.1.1 too generic?

This is an issue and also a request for help contributing to your project.
Given the following diagnostic code...
<EMAIL_ADDRESS>host smtp.secureserver.net[<IP_ADDRESS>] said: 550     5.1.1<EMAIL_ADDRESS>- <IP_ADDRESS> has sent to too many     recipients this hour. IB607  <http://x.co/rlbounce> (in reply to RCPT TO     command)
... Sisimai categorizes the bounce as userunknown when it should be toomanyconn.
I've forked Sisimai and created a local branch with the proper pattern in Sisimai::Reason::TooManyConn to match as toomanyconn. Unfortunately, as long as Sisimai groks the status code as 5.1.1, the bounce is still designated userunknown. Once you edit the bounce and change the status code to anything other than 5.1.1, it finally matches as toomanyconn.
In general, I would argue that simply designating all 5.1.1 bounces to userunknown is too aggressive and leads to false positives.
Specifically, I'd like to solve this issue by contributing to the project. I would appreciate any hints you have on how I can make sure my pattern is applied without defaulting to userunknown.
I process about 100,000 bounces a day, and I have a number of patterns from onHold and other false positives that I'd like to add to the project once I better understand how it works.

Thanks for the report and your contributing !
Status: 5.1.1
RFC3464 says that "X.1.1" means Bad destination mailbox address at the following link:
https://tools.ietf.org/html/rfc3463
however, some MTAs or email services returns a error message which is not userunknown with status code "5.1.1". For example, the value of reason of email-postfix-01.eml is 5.1.1 but the value of deliverystatus is 5.1.1.
email-mailfoundry-02.eml is also the same.
Modules in lib/Sisimai/Bite/Email directory exist to select a correct bounce reason with parsing these bounce messages (and status codes) above.
To resolve the issue
I think, it may be a bit difficult to resolve the issue by improvements in Sisimai::Reason::UserUnknown. Perhaps, Sisimai::Bite::Email::GoDaddy (or other suitable name) might be implemented.
Sample email
Would you post the entire email message (including email headers) bounced from GoDaddy at this issue after substituting secret information (a local part, a hostname, and IP addresses) with some safe string.
If you have many bounce emails  which reason is onhold or undefined, fork sisimai/set-of-emails repository and add these emails into to-be-debugged-becase/* directory.
Best regards.

Here you go, hope this helps:
`
From MAILER-DAEMON  Tue Sep 19 12:00:02 2017
Return-Path: <>
X-Original-To<EMAIL_ADDRESS>Delivered-To<EMAIL_ADDRESS>Received: from mx.sender.com (mx.sender.com [<IP_ADDRESS>])
by mail.sender.com (Postfix) with ESMTP id 084521680B5A
for<EMAIL_ADDRESS>Tue, 19 Sep 2017 12:00:02 -0700 (PDT)
Received: by mx.sender.com (Postfix)
id F1F29402F2229; Tue, 19 Sep 2017 12:00:01 -0700 (PDT)
Date: Tue, 19 Sep 2017 12:00:01 -0700 (PDT)
From<EMAIL_ADDRESS>(Mail Delivery System)
Subject: Undelivered Mail Returned to Sender
To<EMAIL_ADDRESS>Auto-Submitted: auto-replied
MIME-Version: 1.0
Content-Type: multipart/report; report-type=delivery-status;
boundary="DA209402F1B99.1505847601/mx.sender.com"
Content-Transfer-Encoding: 8bit
Message-Id<EMAIL_ADDRESS>This is a MIME-encapsulated message.
--DA209402F1B99.1505847601/mx.sender.com
Content-Description: Notification
Content-Type: text/plain; charset=us-ascii
This is the mail system at host mx.sender.com.
I'm sorry to have to inform you that your message could not
be delivered to one or more recipients. It's attached below.
For further assistance, please send mail to postmaster.
If you do so, please include this problem report. You can
delete your own text from the attached returned message.
               The mail system

<EMAIL_ADDRESS>host smtp.secureserver.net[<IP_ADDRESS>] said: 550
5.1.1<EMAIL_ADDRESS>- <IP_ADDRESS> has sent to too many
recipients this hour. IB607  http://x.co/rlbounce (in reply to RCPT TO
command)
--DA209402F1B99.1505847601/mx.sender.com
Content-Description: Delivery report
Content-Type: message/delivery-status
Reporting-MTA: dns; mx.sender.com
X-Postfix-Queue-ID: DA209402F1B99
X-Postfix-Sender: rfc822<EMAIL_ADDRESS>Arrival-Date: Tue, 19 Sep 2017 11:59:55 -0700 (PDT)
Final-Recipient: rfc822<EMAIL_ADDRESS>Original-Recipient<EMAIL_ADDRESS>Action: failed
Status: 5.1.1
Remote-MTA: dns; smtp.secureserver.net
Diagnostic-Code: smtp; 550 5.1.1<EMAIL_ADDRESS>- <IP_ADDRESS> has
sent to too many recipients this hour. IB607  http://x.co/rlbounce
--DA209402F1B99.1505847601/mx.sender.com
Content-Description: Undelivered Message
Content-Type: message/rfc822
Content-Transfer-Encoding: 8bit
Return-Path<EMAIL_ADDRESS>Received: from localhost.localdomain (mx1.sender.com [<IP_ADDRESS>])
by mx.sender.com (Postfix) with ESMTP id DA209402F1B99
for<EMAIL_ADDRESS>Tue, 19 Sep 2017 11:59:55 -0700 (PDT)
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
Content-Type: text/html
MIME-Version: 1.0
X-Mailer: MIME::Lite 3.030 (F2.84; T2.13; A2.12; B3.13; Q3.13)
Date: Tue, 19 Sep 2017 11:59:55 -0700
X-Report-Abuse-To<EMAIL_ADDRESS>From: "Original Sender"<EMAIL_ADDRESS>To: "Original Recipient"<EMAIL_ADDRESS>Subject: Test
Test
--DA209402F1B99.1505847601/mx.sender.com--
`
Thanks!
V

Thanks for the sample email, I'll try to implement code for fixing this problem.

@ViktorNacht I've implemented Sisimai::Rhost::GoDaddy module and added 2 sample emails in issue-236 branch by Pull-Request #237. Would you try to run issue-236 branch and try to parse bounce mails from GoDaddy ?

@azumakuniyuki,
Thank you so much for working on this. This was much better, but it looks like GoDaddy domains can also receive emails at mailstore1.secureserver.net. For us, roughly 25% of these messages are from mailstore1, while the rest are from smtp.secureserver.net. Our logs don't show any additional hosts, but maybe wildcarding the host might be a good idea.
<EMAIL_ADDRESS>host mailstore1.secureserver.net[<IP_ADDRESS>] said:     550 5.1.1<EMAIL_ADDRESS>- <IP_ADDRESS> has sent to too many     recipients this hour. IB607  <http://x.co/rlbounce> (in reply to RCPT TO     command)
I can send you a complete bounce if you need it.
Also, there is a new variation of this error message:
host smtp.secureserver.net[<IP_ADDRESS>] said: 550 5.1.0 <<IP_ADDRESS>> This IP has sent too many messages this hour. IB504 <http://x.co/rlbounce> (in reply to MAIL FROM command)
Again, if you need a full bounce, let me know.
Finally, do you know of any existing script that can anonymize bounces in bulk?
Thanks!

@ViktorNacht Thank you for telling me about the another mail host name. I've added mailstore1 into Sisimai::Rhost class as a regular expression like (?:smtp|mailstore)[.]secureserver[.]net. When another host of sercureserver.net appeared, I will replace the first part of keys in $RhostClass with wildcard.
And other error codes beginning with IB are also defined in Sisimai::Rhost::GoDaddy https://github.com/sisimai/p5-Sisimai/blob/cec1da6a9a5795b4768cf0a34f88d3cc50eae91f/lib/Sisimai/Rhost/GoDaddy.pm#L6-L30 . Error codes and messages are picked from https://www.godaddy.com/help/what-does-my-email-bounceback-mean-3568 . If you have any bounce message which error code is not found at the URL above, please tell me the code and messages.
And then, I have a plan to implement an anonymizing tool but it has not been completed yet...
Thanks for testing again :-)

This worked great! What a wonderfully comprehensive solution. Thank you.

Thanks for testing :-) I'll merge the branch to master.


How to specify the --compile-commands-dir option?

Sorry for a newbie question, since just converted from vim-lsp, and not really familiar with json at all.
Although the README.md specify the option clangd.compilationDatabasePath for the json config file, but the following config is just causing more problem than it solved:
"languageserver":{
        "clangd":{
                "compilationDatabasePath":"~/compile_commands.d/kernel/"
        }
}

I guess that's not the way how to pass the options for coc-clangd?

:CocInstall coc-clangd
set "clangd.compilationDatabasePath":"" in your coc-settings.json

2. set `"clangd.compilationDatabasePath":""` in your coc-settings.json

No need to put it under any section?


[feature/backup] EtcdBackup: take etcd endpoint

Currently EtcdBackup takes in "ClusterName" which is the name of EtcdCluster. This is coupled with EtcdCluster API.
It'd better take an etcd endpoint instead. This would also allow to take backup on. external, non-k8s etcd cluster.

fixed


Strip quesitonmark from query method in assignment

This is a feature suggestion.
Suppose the following code:
obj.foo? = bar

This should actually call obj.foo=(bar) and not obj.foo?= (bar). Why? Because then you can write the following:
class A
  property! foo
end

a = A.new
a.foo? ||= bar

Whether this is accepted or not, I think we should disallow methods ending in ?=.

This is interesting. Right now when you do:
hash[key] ||= value

it gets rewritten to:
hash[key]? || (hash[key] = value)

While when you do:
obj.property ||= value

it gets rewritten to:
obj.property || (obj.property = value)

There is a small inconsistency here, but the object implementing [] is most probably a Hash or a Hash-like object where you could get a nilable value (and Hash already has the []? method implemented), while for any object it's hard to know whether it's going to have property be nilable, raise, or there would be an alternative property? method.
So, we can leave things as they are and make:
obj.property? ||= value

be rewritten to:
obj.property? || (obj.property = value)

That way we cover all cases.
I'm not sure this is what you are suggesting. I would forbid writing obj.property? = value just by itself, but allow it when there's the ||= (or &&=).
To make it even more consistent, one would have to write hash[key]? ||= value, but I'm not really sure about this last thing.

It's sort of what I'm suggesting yes, I think it could be more intuitive by letting the compiler check whether there's obj.property? defined. When it is, rewrite to obj.property? || (object.property = value), if it's not defined rewrite to obj.property? || (object.property = value). Then the same behavior could be used for Hash like calls too.

@jhass I think we can do that. But right now the syntax sugar is expanded by the parser, so we'd have to change that to expand it in the semantic phase. I don't see why we shouldn't do that: the smarter the compiler and the less the user has to choose what's the obvious correct way to do things, the better. Then we wouldn't even need that obj.property? ||= value syntax.
The only "issue" is if obj is a union type and there's a mixed behaviour. We'll have to think about that as well.

I don't think this should be driven by semantics. That would change the behaviour when a matching method is added in otherwise unrelated code.
I experienced this issue in combination with property! and I think the best solution would simply to expand obj.property? ||= value to obj.property? || (obj.property = value). There can't be a foo?= method anyway, so this would make perfectly sense.


Fixing typo in keyboard teleop launch file and syntax errors in keyboard teleop node

There are a few syntax errors in the keyboard teleop node and launch file. Now, I can properly run the node with ros2 run uuv_teleop vehicle_keyboard_teleop.py.
One thing to note is that the launch file fails to run for me. I run ros2 launch uuv_teleop uuv_keyboard_teleop.launch uuv_name:=rexrov and get the error termios.error: (25, 'Inappropriate ioctl for device'). I suspect this is something due to the ROS2 Launch system not correctly piping stdin from the shell to the subprocess. I tried to fix this using the emulate_tty Node option in a Python launch file, but it didn't work either. This probably should be opened as a different issue though.

Hi @AVSurfer123,
Thanks for reporting these issues! I couldn't test the keyboard teleop node so far. I will take a look at your changes.

Hi @AVSurfer123,
Thanks for reporting these issues! I couldn't test the keyboard teleop node so far. I will take a look at your changes.

I was able to test your changes, it looks good to me.
Currently, I have no clue how to fix the issues that occur when running ros2 launch [...], but I will spend some time to see if a workaround can be found. Thank you for your suggestions.

I was able to test your changes, it looks good to me.
Currently, I have no clue how to fix the issues that occur when running ros2 launch [...], but I will spend some time to see if a workaround can be found. Thank you for your suggestions.


Show Mattermost message/thread IDs for file attachments

This allows replying to or adding reactions.
Builds on and depends on #393

Can you rebase?


Tslint enable unused no unused variable warning

The rebase introduced a new issue :/

@fb55 Rebased and fixed the new issue. PTAL.

Nice, thanks!


Platforms does not effect the ball

I'd like to have moving platforms that move the ball with it, at the moment the ball doesn't move with the platform stays stationary.
Also allow platforms to better push the ball, I have hammers that swing around and ideally should push the ball off the map but they just nudge them a bit at the moment.

The ball will stick to ground entities now, having other entities push the ball needs a bit more thinking though.

@handsomematt  How can one build and run this?


Add SHAP explanations

This PR adds local and global explanations for the mesh-cnn model. It exposes those through grants_tagger expain. Local explanations are also incorporated in the streamlit app.
Local

Global

Note that it is generated only on 2 abstracts for demonstration so it is not as informative as if we run it through a sample of our data which would take a lot of time though

This is open for review @aCampello :)

Sounds good, thanks for the PR. I didn't notice it was ready before (and just read a previous version of it).
It seems that the global explanations didn't make the cut to the app?
(Btw, I confess I skim read the test, but trust that it is passing...)

Global explanations can really only make it if you precompute them because you need to pass a lot of documents. Not sure if this is valuable for the streamlit app which is supposed to be interactive i.e. you add text and it gives you predictions for that text and explanations for that text.


Include a Vagrantfile with the package

With this, people could start using it graphically without having to specifically activate on their Vagrantfiles.

:tada: This issue has been resolved in version 3.0.0 :tada:
The release is available on:

v3.0.0
GitHub release

Your semantic-release bot :package::rocket:


Fix tabindex and still make key events Just Work

This makes Model provide a method hook in JS that gets called when a handler is connected to an event that was previously unconnected. In Widget we use that hook to check if we need to support key events and set the tab_index if necessary. Over time, we might want to use this hook for other purposes as well (e.g. mouse events prevent-default stuff?)

Note that not all HTML elements actually support tabindex the same way. They sometimes even implement it differently.

Yes. This actually make Flexx be much more careful about setting tabindex. Previously the default value was -1, now the default value is None which means "do not set".


ss

Checklist

[X] I have read and accepted the contributing guidelines.

Is your feature request related to a problem? Please describe

논리적//

논리적 표기법//

정규화 과정//

Describe the solution you'd like
..
Describe alternatives you've considered
No response
Additional context
No response


Skip tests for docs folders

Our mandatory checks will fail if there are docs only changes. The helm-chart-ci.yaml skips these files, so the workflows never run, and the mandatory checks are stuck waiting for results.
This adds a parallel workflow that runs fake tests for docs changes. So that docs only changes can pass the required status checks. It uses paths-ignore instead of paths. This is documented here.

Hmm.. Interesting problem...
Would it make sense to just always run the tests in this case too? Remove: https://github.com/spiffe/helm-charts/blob/main/.github/workflows/helm-chart-ci.yaml#L7-L15
It would mean the test suite would run on doc only changes too, but there has not been very many of those pr's so maybe the simplification would be worth it?

Hmm.. Interesting problem...
Would it make sense to just always run the tests in this case too? Remove: https://github.com/spiffe/helm-charts/blob/main/.github/workflows/helm-chart-ci.yaml#L7-L15
It would mean the test suite would run on doc only changes too, but there has not been very many of those pr's so maybe the simplification would be worth it?

Im fine either way though slightly prefer not running tests for docs only changes.

Ok. There's stuff in here that is kind of duplication from other things and needs to be synchronized between the two or else the pipeline breaks. so maybe some changes could be added to make that stuff common so we can change it just in one place?
Since things are broken, I'm ok with that kind of thing being done in a followup pr though.

Ok. There's stuff in here that is kind of duplication from other things and needs to be synchronized between the two or else the pipeline breaks. so maybe some changes could be added to make that stuff common so we can change it just in one place?
Since things are broken, I'm ok with that kind of thing being done in a followup pr though.

I agree. Is there a good way to do that? I always end up with a lot of duplication with actions.

Might be able to use an environment variable from a file along with maybe the fromJson thing refereed to here: https://stackoverflow.com/questions/61919141/read-json-file-in-github-actions
for building the path list.
There may be simpler ways though. Maybe @marcofranssen knows better way?


Add dry mode and allow selection for tweets.

Dry mode can help to know which tweets you will delete.
I add a tweet selection mode to only select advertising tweets posted by apps and delete them.
That might be usefull to the project.

Thank you for the additions
It seems I don't have the access rights to update your PR, and it needed a few fixes + an update to the current master branch.  Please look at allemas/TweetDeletion#2 and merge it if you are OK with my proposals. When done so, tell me here so I cand merge your PR


[HUDI-289] Implement a test suite to support long running test for Hudi writing and querying end-end

Flexible schema payload generation
Different types of workload generation such as inserts, upserts etc
Post process actions to perform validations
Interoperability of test suite to use HoodieWriteClient and HoodieDeltaStreamer so both code paths can be tested
Custom workload sequence generator
Ability to perform parallel operations, such as upsert and compaction

I have rebased the old hudi_test_suite_refactor branch based on master (Fixed conflicts and check style errors). From now, let track the work in this PR. FYI @n3nash @vinothchandar

@yanghua great, thanks. Can you please take a pass at the current PR and leave any comments that we should address ? Also, lets start running the test suite (I'm doing that too) to see if it works as expected in a longer duration.

Will create an issue in Jira and open a PR to address these changes.

@yanghua Can you check why the build is failing now ?

@yanghua Can you check why the build is failing now ?

I saw the HoodieActiveTimeline#createNewCommitTime has been renamed to HoodieActiveTimeline#createNewInstantTime after merging HUDI-308.

@yanghua thanks for fixing the build. Were you able to give the test-suite a try ? That is :

Create your own yaml
Run the TestHudiTestSuiteJob
Run it in docker vs run it in your own spark cluster if you have one

@yanghua thanks for fixing the build. Were you able to give the test-suite a try ? That is :

Create your own yaml
Run the TestHudiTestSuiteJob
Run it in docker vs run it in your own spark cluster if you have one

OK, will try to run the suite later.
I am trying to integrate it with azure pipelines. Here is a demo: https://dev.azure.com/vinoyang/Hudi/_build

@yanghua Left a comment in the other PR

@n3nash This PR has conflicts, I have rebased.

@yanghua merged the pom PR, can you rebase this so we can merge this PR ? I think we have gone through it multiple times, let's have the first version committed so we can come up with more action items around this

@n3nash @yanghua once ready, please open a PR against master.. Love to do one final review before merging.

Or is it this PR?

@yanghua merged the pom PR, can you rebase this so we can merge this PR ? I think we have gone through it multiple times, let's have the first version committed so we can come up with more action items around this

OK, will rebase with the master branch.

Or is it this PR?

Yes

sg. In that case, I would like to re-ask my intiial questions again.. How have we validated the framework? are there real tests now? How/when are they run? :)

sg. In that case, I would like to re-ask my intiial questions again.. How have we validated the framework? are there real tests now? How/when are they run? :)

In my opinion, we will trigger it with azure-pipelines after it merges into master branch.

How have we validated the framework? are there real tests now?

Can you also please answer these?

How have we validated the framework? are there real tests now?

Can you also please answer these?

We validate the test suite via running the unit tests and e2e tests(TestHoodieTestSuiteJob) by local, Travis and docker. There is one mock test here1. About real tests, do you mean how to validate all the functions of hudi via this test suite?

About real tests, do you mean how to validate all the functions of hudi via this test suite?

yes.. not all.. but a good amount .. The real value is in the actual tests.. right

@n3nash report the current state, the whole project can be compiled successfully, I can run the whole test cases in hudi-test-suite module. But the Travis always reports The job exceeded the maximum log length, and has been terminated..

@n3nash Can you verify all the test cases in your local. I have verified them. Everything is OK. The Travis can not finish the full verification.

Will back to this PR and rebase it with master branch later.

Codecov Report

Merging #1100 into master will decrease coverage by 0.66%.
The diff coverage is 62.74%.

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

@yanghua @bvaradar @vinothchandar I've fixed this PR, cleaned up a bunch of the code. This test suite now has test cases that one can use to run end to end tests in junit.
At the moment, the test suite does not run in docker due to Spark 2.4 bringing in Hive 1.x dependencies and our code using Hive 2.x to spin up local hive server nodes. There is a hacky approach that we are using at Uber which will be upstreamed in the next couple of weeks (https://issues.apache.org/jira/browse/HUDI-830) and as part of that we can discuss how to solve it (the right way to solve without any hacks is to move to spark 3.x since they upgraded the Hive libs there but that might take a while etc). Once that is done, we can even run this as an integration test.
The following is my suggestion for a plan for this :

Land this PR which provides an initial test suite to test basic end to end functionality. This allows folks to atleast start using this framework to test large PRs but spending minimal amount of time to enhance the test suite (especially since lots of refactoring is happening). Last I checked, @yanghua I think you went over the PR and were okay to merge. @vinothchandar @bvaradar unless we have major concerns on the PR, we can merge it and then take incremental pr's
There are many tickets under HUDI-289, all the enhancements to the test suite from Uber will follow in subsequent PR's in the next 2-3 weeks. I'm working with the necessary folks along with some of the enhancements that I want to do in the coming weeks.
@yanghua need you to lead the Azure pipelines for the test suite and other tickets assigned to you under the umbrella ticket.

Additionally, since the test suite actually tests all end to end functionality, the main class tests take a while to run. Also, without the test suite we are nearing the max time allowed to run unit tests on travis (50 mins), so I've separated out the test suite unit tests into a different job, just like the integration tests.

@n3nash well done !

@yanghua need you to lead the Azure pipelines for the test suite and other tickets assigned to you under the umbrella ticket.

@n3nash Thanks for your hard work. I will active this work after merging this huge PR.

@n3nash Still conflicting files... I have tried to fix it yesterday. You may need to pull --rebase before force push?

@yanghua no worries, I've pushed the changes

@yanghua can you go through this PR and approve it ?

@yanghua can you go through this PR and approve it ?

Thanks for the great work. Will review it tomorrow.

hi @n3nash , changes from hudi-397 seems lost

Just ended a five-day holiday and will come back soon to post more comments or fixes.

@n3nash conflicting files.

@yanghua addressed comments, rebased.

Codecov Report

Merging #1100 into master will increase coverage by 0.14%.
The diff coverage is 19.44%.

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

@n3nash Thanks for the updates. I have no more concerns. I suggest we can merge it then iterate it in the future. WDYT? @vinothchandar cc @hddong

I will review this over the weekend and merge with some changes on top.. this will speed things up

@vinothchandar Addressed all your comments. I feel hudi-test-suite could be fine, personally find it hard to differentiate between hudi and hudi-system, not sure if the difference would be obvious to other users. The plan is to setup the nightly performance build by @yanghua and a bunch of follow up items that probably captures the essence of what you have in mind for the test-suite instead of system-test-suite. If you feel strongly, let me know, I'll change the name.

@n3nash  let’s please call this integration test to be consistent with the direction @xushiyan has been pushing towards. Unit, integration, functional are standard test terms industry wide. We have to classify this into one of these. hudi-test-suite is very generic IMO

The plan is to setup the nightly performance build by @yanghua and a bunch of follow up items that probably captures the essence of what you have in mind for th

As it stands now, is this going to add new tests to our ci?

I can take a final pass, make any small edits myself and merge once you do the renaming

@n3nash @yanghua do you mind me pushing some changes to this and land this?

@n3nash @yanghua do you mind me pushing some changes to this and land this?

Of course No, please feel free to improve it.

@vinothchandar I refactored the test suite under the hudi-integ-test module. Been running into some issues with the other integration tests passing, looking into this.


kernel: Remove unused absolute symbols

Removes unused absolute symbols (generated via the GEN_ABSOLUTE_SYM() macro) from both common kernel code and architecture specific code.

Uhm, not a big fan of this. The GEN_ABSOLUTE_SYM() macro is creating some defines that could (or could not) be used by downstream code that is relaying on those to be present.
I understand the cleanup frenzy but I think that leaving these in place makes no difference in code size or footprint so why deleting those and maybe creating havoc to downstream or out-of-tree projects?

@carlocaione - To be honest, that potential downstream impact is one that I had not previously considered. That being said, I don't think that the removal of these specific symbols is that large a concern (perhaps I am too close to this to see this objectively). However, should a downstream project require those symbols at assembly level, I would expect that they would most likely employ something similar to the following to get what they need ...
In a 'C' file ...
const size_t _structure_name_SIZE = sizeof(structure_name);
In the asembly file ...
GDATA(_structure_name_SIZE)
In the longer term, my aim is to eliminate the GEN_OFFSET_SYM and GEN_ABSOLUTE_SYM macros. If this can be done cleanly, then it sets the stage for simplifying the build process (as their use requires a separate compilation and custom python script just to create the defines used in assembly).

Fixed the build errors resulting from __z_arch_esf_t_SIZEOF being used in riscv. (For some reason, during the first pass it showed up as not used--I must have searched the wrong directory with grep.) The current failure is, I think, spurious.
One objection thus far is that these symbols may be used in a downstream project. These symbols seem to fall into a bit grey zone. That is, to my knowledge they are not part of the official API, but we do have confirmation that from @carlocaione that some are used there.
I previously stated that one of my longer term goals is to get rid of the GEN_ABSOLUTE_SYM() and GEN_OFFSET_SYM() to allow the build to be simplified a bit. (This could be done using by replacing them with a mixture of global variables and pre-calculated offsets, and it would also preserve the ability to allow that information to be passed downstream--albeit with different names.) I don't know if this additional information would ameliorate any concerns that people may have.

The GEN_ABSOLUTE_SYM() macro is creating some defines that could (or could not) be used by downstream code that is relaying on those to be present.

Here's one✋ We were relying on ___thread_stack_info_t_size_OFFSET to present stack usage during debugging in Nordic's VS Code extensions.
I'll acknowledge that these aren't technically part of the API, so we were on shaky ground here, but these generated symbols have no alternative solution that is covered by the API.
I humbly ask you to please not delete any more of these without providing alternative ways of accomplishing the same use cases. They don't affect production binary sizes and the addition in build time and build system complexity is trivial.
There was a similar discussion in #53951, where some deprecated command line options were removed, then restored. There's definitely value in cleaning up stuff that is actively a burden on the maintainers, but I think this project is big enough that Hyrum's Law should be taken into consideration at this point.

lol, I called it.
@trond-snekvik do you have a way to re-introduce GEN_ABSOLUTE_SYM() without having to revert this?

@carlocaione we actually use nm to read this out of the elf file before launching the debug session. We can't insert our own code into the binary, so as far as I can tell, there are only solutions: either inspect the type through some C parser or GDB, or hardcode the offset and wait for the day when someone inevitably changes the layout of that struct. The latter is the only realistic option for us now, I'm afraid.

@carlocaione that wouldn't do it, no. The symbol we were relying on was actually generated by GEN_OFFSET_SYM(), and technically it was #54446 that broke our workflow. I commented on this PR since this is where you started the conversation that applies to all these changes.
If anything, I think the offsets should be added to the thread info object here:
https://github.com/zephyrproject-rtos/zephyr/blob/9f37ae902bf30580b32b567c8f95d4b54bfac186/subsys/debug/thread_info.c#L42
The downside is that this can't be read at compile time in the same way. Also, this object isn't part of the API either, and it's not used anywhere else in Zephyr, so I suppose there's nothing stopping anyone from merging a PR that deletes that without deprecation as well.
I don't expect that everything that ever existed in zephyr is kept around forever, I'm just hoping maintainers can keep in mind that any change to the observed behavior (and not API) will disrupt someone's workflow. That doesn't mean that the change shouldn't be made, it just means that it will always come at a cost.

I never noticed the file zephyr/subsys/debug/thread_info.c before--I learned something new today. :) Adding the value to the _kernel_thread_info_offsets[] array seems reasonable.

@trond-snekvik - Following up a little on this, I see that there is a pre-processor check that claims that this feature does not work properly with multiple CPUs. I am not sure if this is a make-or-break issue for you, but I thought that I would point it out.
If I am following things correctly, the reason for this limitation is the entry THREAD_INFO_OFFSET_K_CURR_THREAD--offset of the current thread. In an MP system, this entry corresponds only to the offset of the current thread on cpu[0]. Perhaps the list of offsets could be extended to indicate the size of the struct _cpu? This is new code to me, so I assume that there are subtleties about which I am unaware.
If you would like to file an enhancement request, it will ensure that this issue and potential solution is tracked.


Crash when trying to overclock/set a non-standard monitor frequency

Hyprland Version
d63a42e93fc49f57f69270ad70adea008a3bafec (ae69b9a2fa559d869f4e9c61ddf24e152d97df2f)
Bug or Regression?
Regression
Description
Last working commit: 116b9a80566e7956233b39697ac78c112d514c3c
Regression commit: d63a42e93fc49f57f69270ad70adea008a3bafec (ae69b9a2fa559d869f4e9c61ddf24e152d97df2f)
(when wlroots updated from c74f89 to 717ded)
I typically use hyprctl keyword monitor "eDP-1,1920x1080@$FREQ,1280x0,1"  to overclocking/downclocking the monitor.
This may not be the correct way, but it worked before
However, since the regression commit, Hyprland crashes when trying to set any non-standard frequency (for example: 30, 75, 81 for a 60Hz laptop screen; this problem affects all connected monitors)."
How to reproduce
Compile d63a42e93fc49f57f69270ad70adea008a3bafec or later
hyprctl keyword monitor "$,$@$ANY_NOT_STANDART_FREQ,$,$"
Crash reports, logs, images, videos
d63a42e_hyprland.log

update to -git. Besides, this is a dupe, but I won't try and find it now, since it's fixed anyways.


CIF user name and logout button

Describe the task
Show the user <lastname>, <firstname> and email address (one above the other) and a logout button in the header

something like that?

yes!

Or the one illustrated in the prototypes uploaded, which looks like:

We don't have a profile/preferences page yet, so I was more in favor of a simple button for now
Users logging in with a different email has caused issues in the past, so I wanted to see emails up there
We could probably figure out how to get the ministry info (ENV), but we haven't set that up yet

Ok, good to know. Thanks. Best to flag this info on the wireframes pages in Miro so I can account for them before prototyping. Perhaps the ministry title is unnecessary. I thought we'd be pulling that info from the GAL.

Tested in dev environment - works as expected. I was able to log in, see my user name and email, and log out of the system.


Allow fewer dashboards for small deployments

This saves some memory (about 120MB) on single-node deployments.

https://ocd.appscale.com:8080/job/Daily Build/3383/


typo CHANGELOG.md is 2023 year

https://github.com/microsoft/vscode-test/blob/main/CHANGELOG.md#223--2022-01-30
v2.2.3 ~ v2.3.3 is 2023?

it was fixed with this unrelated commit https://github.com/microsoft/vscode-test/commit/6d096d4ff63489fb51d877d2d65151d5a1918119


[In Progress] Feat : alerts module

Added alerts module.
Added SNS implementation.
Updated tests.

coverage: 56.435% (+0.1%) from 56.328%
when pulling 01c553062e11caa9e10192cc017ddec7fb87c985 on feat/alerts-module
into fa59cdf79aa13de3934503767b77634ba9c267f1 on main.


Build fails on MacOS

Building on MacOS, intel fails.
It worked about 12 months ago but the latest does not.
Latest MacOS, latest brew components, clean build
output showing the error is below.
Fixing headers into /Users/ian/dev/ps2dev/build/ps2toolchain/build/ps2toolchain-ee/build/gcc/build-mips64r5900el-ps2-elf-stage1/gcc/include-fixed for mips64r5900el-ps2-elf target
Forbidden identifiers: LANGUAGE_C MIPSEL R4000 _mips mips
cc1: note: self-tests are not enabled in this build
Checking multilib configuration for libgcc...
mkdir mips64r5900el-ps2-elf
mkdir mips64r5900el-ps2-elf/libgcc
Configuring in mips64r5900el-ps2-elf/libgcc
configure: WARNING: decimal float is not supported for this target, ignored
Mode = sf
Suffix = si\|2\|3
Mode = sf
Suffix = si\|2\|3
Mode = sf
Suffix = si\|2\|3
../../../libgcc/config/hardfp.c:61:2: error: #error Unknown operation
   61 | #error Unknown operation
      |  ^~~~~
Mode = sf
make[2]: *** [addsf3.o] Error 1
make[2]: *** Waiting for unfinished jobs....
Suffix = si\|2\|3
../../../libgcc/config/hardfp.c:61:2: error: #error Unknown operation
   61 | #error Unknown operation
      |  ^~~~~
make[2]: *** [subsf3.o] Error 1
../../../libgcc/config/hardfp.c:61:2: error: #error Unknown operation
   61 | #error Unknown operation
      |  ^~~~~
make[2]: *** [negsf2.o] Error 1
../../../libgcc/config/hardfp.c:61:2: error: #error Unknown operation
   61 | #error Unknown operation
      |  ^~~~~
make[2]: *** [mulsf3.o] Error 1
make[1]: *** [all-target-libgcc] Error 2
make: *** [all] Error 2
../scripts/002-gcc-stage1.sh: Failed.
../scripts/003-ee.sh: Failed.
../scripts/001-toolchain.sh: Failed.

The error occurs when running the https://github.com/ps2dev/ps2toolchain-ee/blob/main/scripts/002-gcc-stage1.sh script.

Please try compiling a native unpatched GCC 11.2 first.
If the issue still occurs, then this would not appear to be our problem and should be reported to GCC.

Please, wipe old installation before installing new.

I wiped the previous installation. I've successfully installed on ubuntu, just failing on my imac.
Happy to try an unpatched gcc 11.2 - how would I go about that ... I mean any particular git repo to clone with a given set of options to configure?

You can find a list of mirrors to download GCC from here: https://gcc.gnu.org/mirrors.html

I thought about this some more and the iop version compiles fine so it can't be upstream gcc. Just the EE version fails:
Ians-iMac:bin ian$ ./mipsel-ps2-irx-gcc --version
mipsel-ps2-irx-gcc (GCC) 11.2.0
Copyright (C) 2021 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Ians-iMac:bin ian$ uname -a
Darwin Ians-iMac 21.2.0 Darwin Kernel Version 21.2.0: Sun Nov 28 20:28:54 PST 2021; root:xnu-8019.61.5~1/RELEASE_X86_64 x86_64
Ians-iMac:bin ian$ ```

I didn't try using the latest macOS version, however, this shouldn't have any problem.
Can you just try to type?
gcc --version

Maybe the problem is with the latest Apple clang version in that case, what could work for you is to install another Xcode version and use that one for compile ps2dev.
Currently, in the CI/CD that we have in the project, we are compiling in MacOS without any problem but I think it is still using OSX 10.15

Ians-iMac:bin ian$ gcc --version
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/4.2.1
Apple clang version 13.0.0 (clang-13<IP_ADDRESS>)
Target: x86_64-apple-darwin21.2.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

It is known stuff, look how it is done in the CI
https://github.com/ps2dev/ps2dev/blob/master/.github/workflows/compilation.yml#L56

anyone with M1 (arm64) macOS has been able to build the ps2dev sdk?
I've tried and I'm getting errors on the configure when trying to build the toolchain. I did install GNU sed and did the path change but the error seems to come from the ./configure step before sed.

If I can get my hands on / have access to an M1 Mac, I can debug the issue that is causing it to not work on it


inla.mesh.2d error "Error in qr.coef(a, b)"

On a Windows PC with R 4.1.0 this gives me the error message "Error in qr.coef(a, b) : 'qr' and 'y' must have the same number of rows':
Mesh3 <- inla.mesh.2d(grid2, boundary = boundary, max.edge = c(25000, 250000), cutoff = 3000)
matern <- inla.spde2.pcmatern(Mesh3,
                                prior.sigma = c(3000, 0.01),
                                prior.range = c(10, 0.01))

(shapefiles here)
shapefiles.zip
....I got the same error when running in R4.0.3. So I tried the same code with the same shapefiles on my laptop running R4.0.3 with no error message...and upon updating the laptop to R 4.1.0, it still worked(!).
I'm on the same rgdal version (1.2-23) on both computers, spdep is 1.1-7 and rgeos 0.5-5. I get the same error on the desktop using both RStudio 1.3.1093 and 1.4.1106.
Best wiehs,
Michael

Please post the actual code needed to load the data as well, i.e. complete working code.

Also, please clarify (e.g. by supplying the sessionInfo() outputs for each case) where it works and where it doesn't. There are laptops with Windows, Linux, and Mac...  But it looks like you're saying that it fails on Windows only, and regardless of the R version?

The code is:
setwd("C:/Users/MSmith/Documents/R/data/INLA")
grid2 <- readOGR(paste(getwd(), "/PNG_grid_excl_nullrows_singlepart.shp", sep = ""))
boundary <- readOGR(paste(getwd(), "/PNG_boundary_10km_buffer_UTM_ex_BGV.shp", sep = ""))
Mesh3 <- inla.mesh.2d(grid2, boundary = boundary, max.edge = c(25000, 250000), cutoff = 3000)
matern <- inla.spde2.pcmatern(Mesh3,
                                prior.sigma = c(3000, 0.01),
                                prior.range = c(10, 0.01))

Both PCs are Windows so I'm not sure about other platforms. Session info for where it works:
R version 4.1.0 (2021-05-18)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 19042)
Matrix products: default
locale:
[1] LC_COLLATE=English_United Kingdom.1252  LC_CTYPE=English_United Kingdom.1252    LC_MONETARY=English_United Kingdom.1252
[4] LC_NUMERIC=C                            LC_TIME=English_United Kingdom.1252
attached base packages:
[1] parallel  stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] rgeos_0.5-5   rgdal_1.5-23  INLA_21.02.23 foreach_1.5.1 Matrix_1.3-3  RANN_2.6.1    spdep_1.1-7   sf_0.9-8      spData_0.3.8
[10] sp_1.4-5
loaded via a namespace (and not attached):
[1] tinytex_0.31       gtools_3.8.2       tidyselect_1.1.1   xfun_0.23          purrr_0.3.4        splines_4.1.0      lattice_0.20-44
[8] colorspace_2.0-1   vctrs_0.3.8        generics_0.1.0     expm_0.999-6       utf8_1.2.1         rlang_0.4.11       e1071_1.7-6
[15] pillar_1.6.1       glue_1.4.2         DBI_1.1.1          lifecycle_1.0.0    inlabru_2.3.1      munsell_0.5.0      gtable_0.3.0
[22] raster_3.4-10      codetools_0.2-18   coda_0.19-4        knitr_1.33         class_7.3-19       fansi_0.4.2        Rcpp_1.0.6
[29] KernSmooth_2.23-20 scales_1.1.1       classInt_0.4-3     gdata_2.18.0       deldir_0.2-10      ggplot2_3.3.3      gmodels_2.18.1
[36] dplyr_1.0.6        grid_4.1.0         cli_2.5.0          tools_4.1.0        LearnBayes_2.15.1  magrittr_2.0.1     proxy_0.4-25
[43] tibble_3.1.2       crayon_1.4.1       pkgconfig_2.0.3    MASS_7.3-54        ellipsis_0.3.2     iterators_1.0.13   R6_2.5.0
[50] boot_1.3-28        units_0.7-1        nlme_3.1-152       compiler_4.1.0
And session info for where it doesn't work:
R version 4.1.0 (2021-05-18)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 19042)
Matrix products: default
locale:
[1] LC_COLLATE=English_United Kingdom.1252  LC_CTYPE=English_United Kingdom.1252
[3] LC_MONETARY=English_United Kingdom.1252 LC_NUMERIC=C
[5] LC_TIME=English_United Kingdom.1252
attached base packages:
[1] parallel  stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] inlabru_2.3.1 ggplot2_3.3.3 rgeos_0.5-5   rgdal_1.5-23  INLA_21.05.02 foreach_1.5.1 Matrix_1.3-3  RANN_2.6.1
[9] spdep_1.1-7   sf_0.9-8      spData_0.3.8  sp_1.4-5
loaded via a namespace (and not attached):
[1] tinytex_0.31       gtools_3.8.2       tidyselect_1.1.1   xfun_0.23          purrr_0.3.4        splines_4.1.0
[7] lattice_0.20-44    colorspace_2.0-1   vctrs_0.3.8        generics_0.1.0     expm_0.999-6       utf8_1.2.1
[13] rlang_0.4.11       e1071_1.7-6        pillar_1.6.1       withr_2.4.2        glue_1.4.2         DBI_1.1.1
[19] lifecycle_1.0.0    MatrixModels_0.5-0 munsell_0.5.0      gtable_0.3.0       raster_3.4-10      codetools_0.2-18
[25] coda_0.19-4        class_7.3-19       fansi_0.4.2        Rcpp_1.0.6         KernSmooth_2.23-20 scales_1.1.1
[31] classInt_0.4-3     gdata_2.18.0       deldir_0.2-10      gmodels_2.18.1     dplyr_1.0.6        grid_4.1.0
[37] tools_4.1.0        LearnBayes_2.15.1  magrittr_2.0.1     proxy_0.4-25       tibble_3.1.2       crayon_1.4.1
[43] pkgconfig_2.0.3    ellipsis_0.3.2     MASS_7.3-54        iterators_1.0.13   assertthat_0.2.1   R6_2.5.0
[49] boot_1.3-28        units_0.7-1        nlme_3.1-152       compiler_4.1.0
...hope that helps.

So it fails on your Windows "PC", but not on your windows "laptop"?

Also, range 10 is several orders of magnitude smaller than the inner max.edge setting, 25000 (as well as the cutoff value 3000). This may also be a cause of the numerical problem.

Thanks for your advice....I'm quite new to INLA and was going to try and use inlabru lgcp to model environmental variables at the points of the grid, which are species record locations. Or at least to get an idea of the spatial effect with something like:
comp <- coordinates ~ mySmooth(coordinates, model = matern) +  Intercept(1)
fit <- lgcp(comp, grid2, samplers = boundary, domain = list(coordinates = Mesh3))

I will try changing the units and range. I  I suppose this is more of a discussion for the INLA google group so I should probably close the issue and follow up there.

Secondly, since the mySmooth model operates on the log-scale, the value 3000 as a "unlikely large sd" is also much larger than reasonable. Even a value as large as 3 might be too large, so perhaps prior.sigma=c(1,0.01) may be more reasonable.

This is really useful advice, thank you - I'm afraid I fall over a bit between online tutorials (with not very much background) and the documentation pages (for those with more advanced coding skills than mine).
I'll carry on experimenting, so far tried:
mycrs <- fm_sp_get_crs(grid2)
mycrs2 <- fm_crs_set_lengthunit(mycrs, "km")
grid3 <- inla.spTransform(grid2, crs0 = mycrs, crs2 = mycrs2)

...but get an error "Invalid target CRS for SpatialPointsDataFrame". Looking at mycrs2 it's clearly incorrect. I thought this might work:
grid3 <- inla.spTransform(grid2, crs0 = "+proj=utm +zone=54 +south +ellps=WGS72 +units=m +no_defs", crs1 = "+proj=utm +zone=54 +south +ellps=WGS72 +units=km +no_defs")
...but that returned the same "Invalid target CRS for SpatialPointsDataFrame". Maybe I should try and convert it to WGS84 and see if I can work in degrees...

the inlabru fm_* functions may have bug fixes that aren't in the inla-versions, so fm_spTransform may work better.
And "clearly incorrect" is likely not true; proj4 strings can't always be generated for the new proj6 wkt style specifications; you can go from proj4 to proj6, but not always back, and setting the units is one of those cases where rgdal has trouble figuring out what the proj4 would be. as long as the wkt part is ok, it should work. So the problem is likely with inla.spTransform. Let me know if fm_spTransform() works better.

Oh that's odd, I get 'could not find function "fm_spTransform" and if I try inlabru::fm_spTransform I'm told it's 'not an exported object form namespace:inlabru'.
I don't envy you the task of writing about projections! It's an admirable idea though. For jobbing ecologists PROJ4 strings are the stuff of nightmares.

That's odd! It should be in your inlabru version. Perhaps reinstall inlabru?

Please confirm that you have at least the latest cran version of inlabru, or the latest github dev version? fm_spTransform is an S3 method that isn't exported separately, which may explain the lack of direct inlabru::fm_spTransform access to the function itself, but the documentation should be available, and the function should be useable, ?inlabru::fm_spTransform. The function itself should exist as inlabru:::fm_spTransform. Make sure you've done library(sp), in case that affects the S3 method dispatch. Many other S3methods are also explicitly exported; not sure why this one isn't.

Try the latest inlabru devel version from github; I just added an explicit @export for fm_spTransform, which should solve the problem.

Thanks! That gets fm_spTransform recognized, although I haven't quite got things going (see below for outputs).
I thought I'd try fitting a model on my old laptop (still in metres admittedly) as follows, with the boundary only as you suggested:
Mesh4 <-inla.mesh.2d(boundary = boundary, max.edge=c(25000, 250000), cutoff = 3000)
matern <- inla.spde2.pcmatern(Mesh4,
                              prior.sigma = c(1, 0.01),
                              prior.range = c(3000, 0.01))
comp <- coordinates ~ mySmooth(coordinates, model = matern) +  Intercept(1)
fit <- lgcp(comp, data = boundary, domain = list(coordinates = Mesh4))

...but I suppose that's the wrong syntax for the lgcp? At that point it returns "Error in dimnames(x@bbox) <- 'vtmp' : length of 'dimnames' [1] not equal to array extent" - sorry probably some beginner's error there...
Anyway: for fm_spTransform:

mycrs
CRS arguments:
+proj=utm +zone=54 +south +ellps=WGS72 +units=m +no_defs

mycrs2
CRS arguments:
+proj=tmerc +lat_0=0 +lon_0=141 +k=0.9996 +x_0=500000 +y_0=10000000 +ellps=WGS72 +units=km +no_defs

class(grid2)
[1] "SpatialPointsDataFrame"
attr(,"package")
[1] "sp"

grid3 <- fm_spTransform(grid2, crs0 = mycrs, crs1 = mycrs2)
Error in fm_spTransform.SpatialPointsDataFrame(grid2, crs0 = mycrs, crs1 = mycrs2) :
Invalid target CRS for sp::SpatialPointsDataFrame

class(boundary)
[1] "SpatialPolygonsDataFrame"
attr(,"package")
[1] "sp"

boundary2 <- fm_spTransform(boundary, crs0 = mycrs, crs1 = mycrs2)
Error in fm_crs_bounds_check(x, bounds) :
inherits(x, "matrix") is not TRUE
In addition: Warning messages:
1: In showSRID(SRS_string, format = "PROJ", multiline = "NO", prefer_proj = prefer_proj) :
Discarded ellps unknown in Proj4 definition: +proj=longlat +R=1 +no_defs +type=crs
2: In showSRID(SRS_string, format = "PROJ", multiline = "NO", prefer_proj = prefer_proj) :
Discarded datum unknown in Proj4 definition
3: In showSRID(SRS_string, format = "PROJ", multiline = "NO", prefer_proj = prefer_proj) :
Discarded ellps unknown in Proj4 definition: +proj=longlat +R=6378135 +no_defs +type=crs
4: In showSRID(SRS_string, format = "PROJ", multiline = "NO", prefer_proj = prefer_proj) :
Discarded datum unknown in Proj4 definition
5: In showSRID(SRS_string, format = "PROJ", multiline = "NO", prefer_proj = prefer_proj) :
Discarded ellps unknown in Proj4 definition: +proj=longlat +R=6378135 +no_defs +type=crs
6: In showSRID(SRS_string, format = "PROJ", multiline = "NO", prefer_proj = prefer_proj) :
Discarded datum unknown in Proj4 definition
7: In showSRID(SRS_string, format = "PROJ", multiline = "NO", prefer_proj = prefer_proj) :
Discarded ellps unknown in Proj4 definition: +proj=geocent +R=1 +units=m +no_defs +type=crs
8: In showSRID(SRS_string, format = "PROJ", multiline = "NO", prefer_proj = prefer_proj) :
Discarded datum unknown in Proj4 definition

The lgcp syntax looks fine; it seems to be confused about names of the coordinate variables. Make sure they have names (via coordnames() I think).
For the rest, I to run it here, but can't do that today. Hopefully tomorrow.

Thank you and no hurry of course - on the contrary, I hugely appreciate your patience so far.

Perhaps I didn't need to have recourse to fm_spTransform as this worked:
boundary= spTransform(boundary, "+proj=utm +zone=54 +south +ellps=WGS72 +units=km +no_defs")
Looking at my mesh, I saw that CRS was NULL - it hadn't acquired the CRS of the SpatialPolygonsDataFrame boundary.
....so having transformed the dataframes to km units, the fit seems to work after I add the following argument to inla.mesh.2d: crs="+proj=utm +zone=54 +south +ellps=WGS72 +units=km +no_defs"


[cPP Correction] Section Numbering and Appendices

Appendix D and E are now Sections 8 and 9, missing [appendix]
Appendix D Entropy Documentation and Assessment
Appendix E Rationales

Fixes found here, https://docs.asciidoctor.org/asciidoc/latest/sections/numbers/

Issue resolved with PR#94


Add xiyaowong/virtcolumn.nvim

Checklist:

[x] The plugin is specifically built for Neovim.
[x] The lines end with a .. This is to conform to awesome-list linting and requirements.
[x] It's not already on the list.
[x] If it's a colorscheme, it supports treesitter syntax.
[x] The title of the pull request is Add `username/repo`  when adding a new plugin.
[x] Neovim is spelled as Neovim (not nvim, NeoVim or neovim), Lua is spelled as Lua (capitalized).

Thanks for the PR! I don't think colorcolumn deserves to be it's own category though...

I'm not sure which category this plugin fits into either

I think "Utility" is a good category for this.

@tversteeg

I second @yhu266 , Utility seems to be the proper one.

@xiyaowong I've revised it.


Problems with training deep RNN and with multiple examples

Hi,
I'm currently trying to train the RNN with multiple test inputs, in what format will the 'train_targets' and 'train_inputs' variables have to be to successfully train the network? I thought of concatenating all the inputs but as the train_seq_len and number of targets per input varies I cannot concatenate the entire database.
Also if I increase the number of layers to more than 1 the training cost does not reduce sufficiently (still using only one example), is this because there is not enough data? If that is the case then shouldn't the RNN resort to over-fitting but still predict the correct output?
Regards,
Deepak

Hi, @razor1179. Sorry for the delay on answering your question.
If I understood this right, you wan't to train this model with more than 1 data, am I right?
To do this, you need to generate a tensor for input of size [n_batch, max_timesteps, n_features] where max_timesteps will be the maximum length of input along the batch. train_seq_len will be an one dimensional vector (n_batch,) where you'll put the length of each input, and this will help the network at the calculation of the cost function and on unrolling. Finally, your targets must be an sparse tensor of shape (n_batch, max_utterance_size).
For you second question: did you initialize the weights in the proper way?

Hi,
I've was watching this code to try to make my own speech recognition for latin spanish. I'm trying to use audio book files to train my net but I can't get a good result when i switch to another file. I can save and restore the model pretty well but when I use a female voice to train and a male voice to recognize only got some pieces of words.
What could be the problem? Should I apply some audio filters? how can I add more features to the inputs? (ex. mffc, fft, amplitud, etc)
Regards,
Sergio

Hi @igormq,
I followed the instructions to generate a tensor for input of size [n_batch, max_timesteps, n_features], but as multiple training samples do not have the same timestep I padded them with zeros to match the training sample which has the the max_timestep. The train_seq_len however has the actual timestep of each input sample in a one dimensional vector (n_batch). Is this correct? Also in the example you have chosen the MomentumOptimizer, is there any reason why you chose this out of the many Optimizers provided by Ternsorflow?

Hi @igormq,
I've been trying to write a version of my own so as to use CTC loss to process languages. I came across one thing that I cannot figure out even after going through your code and tensorflow official document. It would be great if you could help me:
When we feed the labels (targets) into ctc_loss, how are we organizing the tuple for sparse tensor? My understanding is that each row of indices corresponds to the [batch, time], where, for batch size as 2, it can be [0, 0], [0, 1], [1, 0], [1, 1] and represents the labels of two sentences with two words (or letters) in each one?
Hi @razor1179 @zergioguitar , I noticed that you both had some experience playing with this, can you kindly help me on this one?
Biyi

@jackyfff ,
From my understanding the targets are usually a 1D list for each utterance, converting it to the appropriate sparse tensor can be done using the code suggested here: http://stackoverflow.com/questions/42127505/tensorflow-dense-to-sparse. Hope this helps.


anyone can alter the result

https://8values.github.io/results.html?e=59.0&d=61.1&g=54.0&s=53.3
url should be more sophisticated to show the result.

It may be worth noting that the page does not check whether the URL Values are within range:
https://8values.github.io/results.html?e=590&d=611&g=540&s=533

Personally I would disagree that this needs to be restricted. @ricealexander is correct though.

I also don't see need for this. As long as you can take the test as often as you want you could also alter just your choices.
However I agree if you really want to manipulate your result at least you have to "learn" what choices you have to make to get the result you want. The question is what is the goal of this quiz and in what way the change brings us closer to the goal.

Personally i don't see the point in changing the logic, other that than the bug fix mentioned.
This is simply a result page, it doesn't need to be anything else.


Remove requirement for AWS_BUCKET_URL or BUCKETEER_BUCKET_URL environment variable

Bucketeer does not create or populate this environment variable, and its value can be constructed using the BUCKETEER_BUCKET_NAME or AWS_BUCKET_NAME environment variable.
So config.js can probably be changed like this:
AWS_BUCKET_URL: `https://${process.env.BUCKETEER_BUCKET_NAME || process.env.AWS_BUCKET_NAME}.s3.amazonaws.com`

@crcastle I would Iike to work on this. Can you please assign it to me


Image scale to same size before comparison

When comparing two images from the same source but were saved with different dimensions such as a thumbnail, Resemble.js reports that they are not similar as the pixel data comparison only occurs in the overlap of the photos starting from the top left corner.
Example 1: Thumbnail (50x50) vs Source (282x281) Note the thumbnail overlaid on top left of the larger image, failing comparison

Desired behavior is to scale the images to be the same size before comparing pixel data (still reporting that the input images have different resolutions). For demonstration purposes, I resized the source image in Photoshop to match 50x50 of the thumbnail and re-ran the comparison, which reported much more accurate results.
Example 2: Thumbnail (50x50) vs Scaled Source (50x50)** Much more accurate comparison results.

I am experimenting with resizing the images with Canvas elements before loading into Resemble.js, but I believe this is an issue worth considering in the library core.

It's a good feature suggestion, I'll accept a PR.

@timkarnold In your example you downscaled the larger image. Did you also experiment with upscaling the smaller image? If so, what have been the results?

Looking forward to this as I have images that are produced on a retina display at 2x resolution and being compared to images taken on non-retina at 1x. Built in downscaling the 2x one would be awesome!

@jamescryer please review PR with feature https://github.com/Huddle/Resemble.js/pull/60

I believe that PR #60 resolved this ticket.. Closing.


Addon Info: Omit empty inline info header

Issue: Creating stories using addWithInfo, configured not to show headers, and to render info inline displays a blank header container on each story.
What I did
Instead of unconditionally rendering the header container and filling it with the (possibly null) result of _getInfoHeader(), I check the result first, and render the container conditionally.
How to test
Check the 'simple usage (no header)' example. Before this change is applied, it appears with a large empty white area above the example.

After this change is applied, the empty white area is absent.

Looks like this is a duplicate of https://github.com/storybooks/storybook/pull/1297/files
Feel free to take whichever approach seems appropriate.

@duncanbeevers this one is a little cleaner, so I'm going to merge it. thanks for the PR!

@duncanbeevers NOTE: we are planning to move the info addon into a panel soon: #1147


Robert Longson: Watch skypanacea.com

Robert Longson requests the watch of the watch_keyword skypanacea\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

skypanacea\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


[entropy, sw] restore entropy_smoketest in CI

Now that #6105 is in, we have a reliable way to disable and re-enable entropy_src.
This should mean now the entropy_smoketest can create reliable outputs when given a specific seed.
Update the test and re-enable in CI.

@mwbranstad
@senelson7
FYI

done with #6135


Pong Game in Python

Script Description - Pong Game in Python
Instructions

Create a new folder for your script and file/folder name should be appropriate.
Create a README.md in your folder for program Instructions
add requirements.txt if needed

Please add/delete options that are not relevant.

[x] Adding New Code
[x] Improving Documentation

Programming Language

[x] Python

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server  - https://discord.gg/FXyh2S3
Happy Coding,

PLease assign this issue to me.

#594 Please review this PR


[rtabmap] build failure

Operating system
Windows
Compiler
MSVC
Steps to reproduce the behavior
https://dev.azure.com/vcpkg/public/_build/results?buildId=101669&view=results
@Neumann-A Seems like there is something wrong with BLAS selection
Failure logs

```
CMake Warning at D:/installed/x64-windows-static-md/share/curl/vcpkg-cmake-wrapper.cmake:49 (message):
  CURL_LIBRARIES list at least one target.  This will not work for use cases
  where targets are not resolved.
Call Stack (most recent call first):
  D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:813 (include)
  D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
  D:/installed/x64-windows-static-md/share/tesseract/TesseractConfig.cmake:24 (find_dependency)
  D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
  D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
  D:/installed/x64-windows-static-md/share/opencv/OpenCVModules.cmake:40 (find_dependency)
  D:/installed/x64-windows-static-md/share/opencv/OpenCVConfig.cmake:133 (include)
  D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
  CMakeLists.txt:230 (FIND_PACKAGE)
CMake Warning (dev) at D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/FindJasper.cmake:76 (set_property):
Policy CMP0160 is not set: More read-only target properties now error when
trying to set them.  Run "cmake --help-policy CMP0160" for policy details.
Use the cmake_policy command to set the policy and suppress this warning.
IMPORTED property is read-only for target("Jasper::Jasper")
Call Stack (most recent call first):
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/installed/x64-windows-static-md/share/Qt6/FindWrapJasper.cmake:12 (find_package)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
D:/installed/x64-windows-static-md/share/Qt6/QtPublicDependencyHelpers.cmake:36 (find_dependency)
D:/installed/x64-windows-static-md/share/Qt6Gui/Qt6QJp2PluginDependencies.cmake:5 (_qt_internal_find_third_party_dependencies)
D:/installed/x64-windows-static-md/share/Qt6Gui/Qt6QJp2PluginConfig.cmake:48 (include)
D:/installed/x64-windows-static-md/share/Qt6/QtPublicPluginHelpers.cmake:518 (include)
D:/installed/x64-windows-static-md/share/Qt6Gui/Qt6GuiPlugins.cmake:5 (__qt_internal_include_plugin_packages)
D:/installed/x64-windows-static-md/share/Qt6Gui/Qt6GuiConfig.cmake:125 (include)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
D:/installed/x64-windows-static-md/share/Qt6/QtPublicDependencyHelpers.cmake:111 (find_dependency)
D:/installed/x64-windows-static-md/share/Qt6Widgets/Qt6WidgetsDependencies.cmake:39 (_qt_internal_find_qt_dependencies)
D:/installed/x64-windows-static-md/share/Qt6Widgets/Qt6WidgetsConfig.cmake:40 (include)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/installed/x64-windows-static-md/share/Qt6/Qt6Config.cmake:156 (find_package)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/installed/x64-windows-static-md/share/vtk/VTK-vtk-module-find-packages.cmake:115 (find_package)
D:/installed/x64-windows-static-md/share/vtk/vtk-config.cmake:148 (include)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
D:/installed/x64-windows-static-md/share/opencv/OpenCVModules.cmake:42 (find_dependency)
D:/installed/x64-windows-static-md/share/opencv/OpenCVConfig.cmake:133 (include)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
CMakeLists.txt:230 (FIND_PACKAGE)
This warning is for project developers.  Use -Wno-dev to suppress it.
CMake Warning (dev) at D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/FindJasper.cmake:83 (set_property):
Policy CMP0160 is not set: More read-only target properties now error when
trying to set them.  Run "cmake --help-policy CMP0160" for policy details.
Use the cmake_policy command to set the policy and suppress this warning.
IMPORTED property is read-only for target("Jasper::Jasper")
Call Stack (most recent call first):
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/installed/x64-windows-static-md/share/Qt6/FindWrapJasper.cmake:12 (find_package)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
D:/installed/x64-windows-static-md/share/Qt6/QtPublicDependencyHelpers.cmake:36 (find_dependency)
D:/installed/x64-windows-static-md/share/Qt6Gui/Qt6QJp2PluginDependencies.cmake:5 (_qt_internal_find_third_party_dependencies)
D:/installed/x64-windows-static-md/share/Qt6Gui/Qt6QJp2PluginConfig.cmake:48 (include)
D:/installed/x64-windows-static-md/share/Qt6/QtPublicPluginHelpers.cmake:518 (include)
D:/installed/x64-windows-static-md/share/Qt6Gui/Qt6GuiPlugins.cmake:5 (__qt_internal_include_plugin_packages)
D:/installed/x64-windows-static-md/share/Qt6Gui/Qt6GuiConfig.cmake:125 (include)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
D:/installed/x64-windows-static-md/share/Qt6/QtPublicDependencyHelpers.cmake:111 (find_dependency)
D:/installed/x64-windows-static-md/share/Qt6Widgets/Qt6WidgetsDependencies.cmake:39 (_qt_internal_find_qt_dependencies)
D:/installed/x64-windows-static-md/share/Qt6Widgets/Qt6WidgetsConfig.cmake:40 (include)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/installed/x64-windows-static-md/share/Qt6/Qt6Config.cmake:156 (find_package)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/installed/x64-windows-static-md/share/vtk/VTK-vtk-module-find-packages.cmake:115 (find_package)
D:/installed/x64-windows-static-md/share/vtk/vtk-config.cmake:148 (include)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
D:/installed/x64-windows-static-md/share/opencv/OpenCVModules.cmake:42 (find_dependency)
D:/installed/x64-windows-static-md/share/opencv/OpenCVConfig.cmake:133 (include)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
CMakeLists.txt:230 (FIND_PACKAGE)
This warning is for project developers.  Use -Wno-dev to suppress it.
CMake Warning at D:/installed/x64-windows-static-md/share/curl/vcpkg-cmake-wrapper.cmake:49 (message):
CURL_LIBRARIES list at least one target.  This will not work for use cases
where targets are not resolved.
Call Stack (most recent call first):
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:813 (include)
D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
D:/installed/x64-windows-static-md/share/netcdf/netCDFConfig.cmake:34 (find_dependency)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/installed/x64-windows-static-md/share/vtk/VTK-vtk-module-find-packages.cmake:585 (find_package)
D:/installed/x64-windows-static-md/share/vtk/vtk-config.cmake:148 (include)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
D:/installed/x64-windows-static-md/share/opencv/OpenCVModules.cmake:42 (find_dependency)
D:/installed/x64-windows-static-md/share/opencv/OpenCVConfig.cmake:133 (include)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
CMakeLists.txt:230 (FIND_PACKAGE)
CMake Deprecation Warning at D:/installed/x64-windows-static-md/share/jsoncpp/jsoncppConfig.cmake:2 (cmake_policy):
Compatibility with CMake < 3.5 will be removed from a future version of
CMake.
Update the VERSION argument  value or use a ... suffix to tell
CMake that the project does not need compatibility with older versions.
Call Stack (most recent call first):
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/installed/x64-windows-static-md/share/vtk/VTK-vtk-module-find-packages.cmake:632 (find_package)
D:/installed/x64-windows-static-md/share/vtk/vtk-config.cmake:148 (include)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
D:/installed/x64-windows-static-md/share/opencv/OpenCVModules.cmake:42 (find_dependency)
D:/installed/x64-windows-static-md/share/opencv/OpenCVConfig.cmake:133 (include)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
CMakeLists.txt:230 (FIND_PACKAGE)
CMake Error at D:/installed/x64-windows-static-md/share/lapack-3.11.0/lapack-targets.cmake:81 (message):
The imported target "blas" references the file
 "D:/installed/x64-windows-static-md/debug/lib/libblas.lib"

but this file does not exist.  Possible reasons include:

The file was deleted, renamed, or moved to another location.

An install or uninstall procedure did not complete successfully.

The installation package was faulty and contained
"D:/installed/x64-windows-static-md/share/lapack-3.11.0/lapack-targets.cmake"

but not all the files it references.
Call Stack (most recent call first):
D:/installed/x64-windows-static-md/share/lapack-3.11.0/lapack-config.cmake:7 (include)
D:/installed/x64-windows-static-md/share/lapack/vcpkg-cmake-wrapper.cmake:19 (_find_package)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:813 (include)
D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
D:/installed/x64-windows-static-md/share/suitesparse/suitesparse-config.cmake:8 (find_dependency)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
D:/installed/x64-windows-static-md/share/ceres/CeresConfig.cmake:183 (find_dependency)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
D:/downloads/tools/cmake-3.29.0-windows/cmake-3.29.0-windows-i386/share/cmake-3.29/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
D:/installed/x64-windows-static-md/share/opencv/OpenCVModules.cmake:44 (find_dependency)
D:/installed/x64-windows-static-md/share/opencv/OpenCVConfig.cmake:133 (include)
D:/a/_work/1/s/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
CMakeLists.txt:230 (FIND_PACKAGE)
</details>

"D:/installed/x64-windows-static-md/debug/lib/libblas.lib" is from LAPACK config...
https://github.com/microsoft/vcpkg/issues/38009

I feel like something is setting CMAKE_FIND_PACKAGE_PREFER_CONFIG_MODE

I feel like something is setting CMAKE_FIND_PACKAGE_PREFER_CONFIG_MODE

Like PRs from @Thomas1664 ? https://github.com/microsoft/vcpkg/pull/38049/files#r1557920439

I feel like something is setting CMAKE_FIND_PACKAGE_PREFER_CONFIG_MODE

Like PRs from @Thomas1664 ? https://github.com/microsoft/vcpkg/pull/38049/files#r1557920439

No, that also happened before I did this.

I feel like something is setting CMAKE_FIND_PACKAGE_PREFER_CONFIG_MODE

Like PRs from @Thomas1664 ? https://github.com/microsoft/vcpkg/pull/38049/files#r1557920439

No, that also happened before I did this.

Please provide a reference.
The PR CI run in this bug report has CMAKE_FIND_PACKAGE_PREFER_CONFIG=ON added by you in that PR.

The PR CI run in this bug report has CMAKE_FIND_PACKAGE_PREFER_CONFIG=ON added by you in that PR.

Admittedly for port colmap.
We have another regression in #38071.


monaco-editor/esm/vs/editor/editor.api is transformed into cjs instead of esm any more

With regards to the common js usage warnings, it seems that MonacoEditorWebpackPlugin is augmenting the source of Monaco Editor and adding CJS modules.
https://github.com/microsoft/monaco-editor/blob/a5298e13bba5ab5d97ccef97c7b56df9cec489ce/webpack-plugin/src/loaders/include.ts#L29-L36

I tested a fix on a fork which seems to work : https://github.com/titouanmathis/monaco-editor/commit/424abe722e5b2c3d72ce60a3b2cb7b67ddc475b6
Tests are ok when I run them, maybe someone else could confirm?


When Knowledge expands, wisdom shrinks

Hello, I wanted to share my opinion.
The memary is based on knowledge graph expansion, though there is a compression in the pipleline, but in the knowledge graph, compression by data storage technique will not lead anywhere nor stay ahead of others.
Instead, by introducing a balance in the graph based on knowledge and wisdom, an efficient storage can be developed.
An example between knowledge is, knowing more is knowledge. Where as, wisdom is knowing less, yet be convincing. In terms of LLMs, for knowledge based communication, there will be huge number of words involved, where as, in wisdom based communication system, the number of words will be less, but stronger words.

Hi @imvetri
Thank you for your insights. Regardless of the scale of the knowledge graph (the larger, the better), retrieving information always results in the formation of a subgraph centered around the main query entity, facilitating low-cost computation. This is highly beneficial for multi-hop reasoning. However, the knowledge graph itself does not undergo memory compression. The more comprehensive the graph, the greater the volume of accessible information for responding to queries.
The memory stream tracks users' exposure to various concepts - their 'wisdom' or breadth of knowledge - while the entity knowledge store assesses the depth of their understanding.
Currently, the only form of memory compression applies to the entity knowledge store, where we compress the top N entities. These are then provided to the LLM finite context window with specific instructions to avoid detailed explanations, enhancing personalization for users already familiar with these concepts.
We hope to see improvements in the memory compression implemented as its fairly basic. One future contribution listed in the README is that instead of compressing the most frequent entities (according to their counts), we could see improved results by compressing the entities that are included in the current query. This improvement will enable the LLM to more effectively determine which concepts require detailed explanations and which can be addressed more succinctly in response. We invite you to contribute to this development!
We also have compression for the chat history using an eviction policy but that's a bit outside the scope of this discussion. Hope this helps.


How to use expireAfterSeconds?

I'm not sure if there is an error in the source code but when i use expireAfterSeconds in nodeJS, i doesn't give me an error nor does it work. There are no answers online so i figured this should be a last resort. Please help.

@louischatriot I'm awaiting a response and I'm very sorry because I know you are busy but my code isn't working and it's driving me nuts.


Add idsrv3test.cer into your Trusted People

When I try to follow the instructions for this, I get the error:
the file type is not recognisable
Please advise.

would need much more info. But I think this step isn't even necessary anymore.

Feel free to re-open if still an issue


Runelite Black Screen after newest update

Please check if your issue is not a duplicate by searching existing issues
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: [e.g. Windows, Ubuntu, macOS]
RuneLite version: [e.g 1.4.6]
Launcher version: [e.g 1.5.2]

Additional context
Add any other context about the problem here, e.g. logs. Your client logs can usually be found in
C:\Users\<your_user_name>\.runelite\logs on Windows and ~/.runelite/logs on Linux and macOS.

GPU option was OFF, changing it didnt help with the issue

OSRS is having some technical difficulties, not much that can be done until Jagex fixes things

https://twitter.com/OldSchoolRS/status/1243156082631086081


Make use of Proxy.Labels instead of Meatdata.Labels

Please provide a description of this PR:
This is to fix misuse of labels because of rebase issue

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
ec959eca468aea1434e573e6101b21ad83892c2d
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
ec959eca468aea1434e573e6101b21ad83892c2d
link
true
/test release-notes

integ-ipv6_istio
ec959eca468aea1434e573e6101b21ad83892c2d
link
true
/test integ-ipv6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ipv6_istio
ec959eca468aea1434e573e6101b21ad83892c2d
link
true
/test integ-ipv6

release-notes_istio
f940618ea506d7f72af61a28e9bc0bed7a139f3c
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ipv6_istio
ec959eca468aea1434e573e6101b21ad83892c2d
link
true
/test integ-ipv6

release-notes_istio
522bf1fd2229447e135f504b631cccd15c33f296
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-notes


test(engine): enable leak test in engine codebase

What problem does this PR solve?

Issue Number: ref #6433
What is changed and how it works?

Add main_test.go that contains leak test in each package of dataflow engine. Note leak test in part of packages are not enabled.

Check List
Tests

Unit test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

Codecov Report

Merging #6434 (93297da) into master (f14088c) will increase coverage by 0.1121%.
The diff coverage is 81.3008%.

Flag
Coverage Δ

cdc
65.9542% <79.5698%> (+0.0608%)
:arrow_up:

dm
52.0081% <ø> (-0.0051%)
:arrow_down:

engine
63.9614% <83.0601%> (+0.4982%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #6434        +/-   ##
================================================
+ Coverage   59.2403%   59.3525%   +0.1121%
================================================
  Files           754        757         +3
  Lines         86755      87105       +350
================================================
+ Hits          51394      51699       +305
- Misses        30799      30823        +24
- Partials       4562       4583        +21

/merge

/merge


Add release note for 2.5.0

Signed-off-by: Ryan Liang<EMAIL_ADDRESS>Description
Add release note for 2.5.0

Category (Maintenance)

Issues Resolved

Relate https://github.com/opensearch-project/security/issues/2373

Is this a backport? If so, please add backport PR # and/or commits #
2.x, 2.5
Check List

[ ] ~New functionality includes testing~
[ ] ~New functionality has been documented~
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2394 (e4b4908) into main (3ea4702) will decrease coverage by 0.07%.
The diff coverage is n/a.

:exclamation: Current head e4b4908 differs from pull request most recent head da98248. Consider uploading reports for the commit da98248 to get more accurate results

@@             Coverage Diff              @@
##               main    #2394      +/-   ##
============================================
- Coverage     61.07%   61.00%   -0.08%
+ Complexity     3269     3262       -7
============================================
  Files           260      260
  Lines         18362    18361       -1
  Branches       3249     3249
============================================
- Hits          11215    11201      -14
- Misses         5560     5566       +6
- Partials       1587     1594       +7

Impacted Files
Coverage Δ

...urity/ssl/transport/SecuritySSLNettyTransport.java
62.36% <0.00%> (-4.31%)
:arrow_down:

...earch/security/ssl/util/SSLConnectionTestUtil.java
93.18% <0.00%> (-2.28%)
:arrow_down:

...earch/security/auditlog/impl/AbstractAuditLog.java
72.09% <0.00%> (-1.66%)
:arrow_down:

...ensearch/security/compliance/ComplianceConfig.java
82.63% <0.00%> (-0.70%)
:arrow_down:

...ecurity/ssl/rest/SecuritySSLReloadCertsAction.java
84.78% <0.00%> (-0.33%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


docs: add close old issues workflow

Closes #272

to close issues that have been inactive for more than 30 days.

Working

The workflow is triggered on a schedule using a cron expression. In this case, it is set to run every day at midnight (0 0 * * *).

The workflow job named "close-issues" runs on the latest version of the Ubuntu environment.

The steps within the job are as follows:
a. "Checkout Repository" step: This step checks out the repository's code, allowing subsequent actions to access its contents.
b. "Close Old Issues" step: This step contains the main logic to close old issues. Here's what it does:

It makes a GET request to the GitHub API to retrieve a list of open issues in the repository.

For each open issue, it retrieves the last updated timestamp using another API request.

It calculates the number of days that have passed since the last update of the issue.

If the number of days since the last update is greater than 30, it makes a PATCH request to the GitHub API to close the issue by updating its state to "closed".

if [ $days_since_update -gt 30 ]; then

The workflow utilizes the secrets.GITHUB_TOKEN secret, which is a token provided by GitHub to authenticate and authorize API requests made by the workflow.
Overall, this workflow helps automate the process of closing old issues in a GitHub repository by checking their last update timestamp and closing them if they have been inactive for more than 30 days.

@Vikash-8090-Yadav
Could you please review this pull request :)

If possible make this for 90 days, I mean open for more than 90 days. Thanks!

Done!


Adds support for swift1.2

I appreciate the PR, but there is already a swift-1.2 branch.
https://github.com/daltoniam/Starscream/tree/swift-1.2
It is in a branch until version 1.2 comes out of beta. Any questions, please let me know.

K thanks :)
On Mon, Mar 2, 2015 at 10:39 AM, Dalton<EMAIL_ADDRESS>wrote:

I appreciate the PR, but there is already a swift-1.2 branch.
https://github.com/daltoniam/Starscream/tree/swift-1.2
It is in a branch until version 1.2 comes out of beta. Any questions, please let me know.
Reply to this email directly or view it on GitHub:
https://github.com/daltoniam/Starscream/pull/61#issuecomment-76733912


Moc labs review patch 12

Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

Submitted by ParamRamki on behalf of MOC Labs Review


[Bug] topic subscription merges foo/# with foobar/#

What did I do
Created a topic subscription in the format tray/# in order to subscribe to messages in the format tray/abcd1234/state tray/12cbac123/ping etc.
Note we have a seperate subscriptions setup for traybot/#
What happened
Messages of the format traybot/# are also being subscribed to
Expected
Expected result would be that traybot/# messages are ignored as they do not conform to the pattern tray/# ~it appears that the trailing / character is being ignored and that the subscription is instead tray# which is unexpected behaviour.~ (edit, see comment below)
Environment

OS: OS version, e.g. macOS Big Sur 11.1
MQTTX version, e.g. v1.5.0

More detail
Screenshot showing subscription and also one correct and one incorrect message.  (Please note that the payloads, redacted as I don't have approval to share, do not contain the words "tray" anywhere)

I deleted the subscription to traybot/# and the offending messages were not shown in the tray/# subscription.  I recreated the traybot/# subscription and messages sent to the topic traybot/# were again shown in the tray/# subscription.
So it looks like that they're being "merged" somehow.

Yep confirmed.
can easily recreate the bug.

create topic foo/#

send message to foo/1/test and it subscribes correctly

send message to foobar/1/test and it does not show (correct behaviour)

create seperate topic foobar/#

click on foo/# topic window

send message to foobar/1/test and message now shows in foo/# window

Hi, sorry for my late reply. Thanks for your response, but I cannot reproduce your issue:

And I noticed your MQTTX version is outdated. Please upgrade to the latest version (https://mqttx.app/downloads) to use.

(Just want to say first that we love the product!)
Version updated. Was able to recreate on a new subscription to a new local mosquitto server instance (running on my computer).  I've screenshotted each step.
![0  version](https://github.com/user-attachments/assets/bb457cf2-f2b1-4d7b-bb96-27b

2bcab0d75)

macOS 14.6.1
mosquito version 2.0.18
using mqtt version 3.1

Hi, sorry for my late reply. Thanks for your response, but I cannot reproduce your issue:

And I noticed your MQTTX version is **outdated**. Please upgrade to the latest version (https://mqttx.app/downloads) to use.

Looks like you did recreate it... you've got the "tray/#" subscription selected in the subscription list, so you should only be seeing "tray/#" messages in the message stream.  However the "traybot/3/state" message appeared erroneously.

(Just want to say first that we love the product!)

Thanks so much! 💛
I'm sorry, I misunderstood this issue. I reproduced it, and it's caused by the filter function on the search method for the database. It's a BUG, and we will fix it in the next version. Thanks.

Happy days!

Updated on https://github.com/emqx/MQTTX/releases/tag/v1.11.0. Thanks for your feedback.


[BUG] echarts not defined

Describe the bug
When using the echarts shortcode I get
theme.js:724 ReferenceError: echarts is not defined
at theme.js:528
at a.forEach (theme.js:4)
at a._echartsOnSwitchTheme (theme.js:527)
at a.initEcharts (theme.js:534)
at a.init (theme.js:720)
at HTMLDocument.themeInit (theme.js:740)
init @ theme.js:724
themeInit @ theme.js:740
Expected behavior
The chart should appear.
Build Environment

OS: Windows 10
Theme version/commit 2.0.1
Hugo version  v0.78.2/extended

Preview Environment

OS: Windows 10
Browser Chrome

Yes, unfortunately something broke there. So far there was no time to see what.

Hi, thanks for the answer. It works when I add echarts as a custom js.

Strange, but I don't see any problems with echarts, everything is still available on the demo site.
There was a problem with css mermaid and I fixed it, but there seems to be no problem with echarts

Strange indeed. I will look into it, maybe I found something. I noticed that the demo page works without problems. Thanks for your response. I will close it and notice you if I found something helpfull. Thanks for your work on the project!

The only problem I have been able to find is as follows. If you run the hugo server locally, it may not be rendered at the first start, so that the graphs appear, you need to make some change to the md file. I don’t know yet what it’s connected with, but this problem is not observed during the final generation of the public folder.

Try with the --disableFastRender argument when serving. This solved it for me:
hugo serve -D --disableFastRender

When building and deploying the static site, the problem does not occur by itself.


Switch the default imagery from Maxar

Not Bing, since my token is being used for scraping all the time. Probably Mapbox.

Done in 3.2.


Modification Request?

So not exactly an issue, but I since there isn't a Discussions area I figured this would be the next best option.
I was working on building a similar project, but then I ran across yours and it is much further developed than mine. I first started taking some inspiration from your project but then realized that I was narrowing in on similar methods for solving the problems as what you had taken so I figured I and hopefully others would be better served from just helping this project out.
Would you be interested/open to me issuing some fairly heavy updates to your code that don't change a huge amount but do make a few things more idiomatic to go and knock out some of the TODOs that were listed throughout the code?
If not, that is cool. I just didn't want to fork and issue a bunch of PRs that you are not interested in.

Yeah, that would be great.  There are a number of things that I want to refactor and just haven't done.  I've purposefully kept the version in beta because I knew the api would probably change, so I'm totally open to any improvements you've got.

Probably will do more, but at least this was my first "go" at making some updates to the code. As I mentioned in there, please feel free to accept/reject any of the edits, and if there are some other things that you would like to see done in the code base (specifically around the client), let me know and I will see if I can take care of some of that as well.
PR: #21

I closed this, but if you'd like to make any other changes, feel free to open up more pull requests.


Mac OS: Image compiles and runs, but no display.

Hello!
I've using version 1.31 of the plugin with JavaFx 11.
When I compile and run a native image, the executable runs, but no window is displayed.
In the command line I get the following exceptions:
java.lang.RuntimeException: Unable to construct Application instance: [application class name]
Caused by: java.lang.NoSuchMethodException: [application class name].()
One possible reason might have to do with the fact that it's written in Kotlin with TornadoFx.
I've tried modifying the launch method to use more java-like conventions, but to no avail.
Any idea what might be causing this?

See related issue: https://github.com/gluonhq/substrate/issues/572
You can see that you have to add the classes from the main application class to the reflection list.

Thank you!
How does this configuration look in gradle?
At this point, I'm trying to add it to the gluonPlugin as a list of strings, but now it won't compile
gluonClient {
       reflectionList = [
            "org.test.AdminApp",
            "org.test.MainView",
            "org.test.AdminStyle"
           ...
    ]
}

That should work. What error do you get?

After a little more checking I just had some references wrong, so the reflection list is working.
I followed a few different error paths and ended up resolving a few more "unsupported feature" errors by rearranging code.
I can get a basic app to display nicely, but there are a few remaining hangups:
--it doesn't seem to pick up the TornadoFX stylesheet class, but no big deal since inline style is working just fine.
--it can't seem to find the JavaFx resources necessary to instantiate a media player
(java.lang.NoClassDefFoundError: javafx/scene/media/Media)
I'm including the media module with the openjfx gradle plugin like this:
javafx {
    version = "11"
    modules = [ 'javafx.controls', 'javafx.media', 'javafx.fxml']
}

Should I be including some kind of additional arguments for native image regarding javafx.media?

What extension is your style class, css?
Media is not supported yet.

The css is a tornadofx construct--no extension on the file, but the css is stored in a class like this:
class Style : Stylesheet() {
    companion object {
        //css rules declared here and delegated, then defined in the init block
    }
}

Then it's compiled with type safety. Though, it doesn't seem to be working with inline style either now that I look.
For some reason I thought I could get away with javaFX media player if the target was desktop--not sure where I got that idea actually. I suppose I'll look into the charm-down plugin, or libVLC or something.
If it works with the css migrated to actual css files, I might just go that route. Reformatting a few files is a small price to pay for a nice binary.
Thank you for all your amazing help!

If the css class is used reflectively, try adding it to the reflectionList as well. That might work. Migrated to .css should work too, of course.


Release azure-cli-component 2.0.5

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Started releasing azure-cli-component 2.0.5...

Release of 'azure-cli-component' with version '2.0.5' successful. View at https://pypi.python.org/pypi/azure-cli-component/2.0.5.

GitHub release created. https://github.com/Azure/azure-cli/releases.