Remove After= lines that don't do what people want / expect.

see Systemd Rules
When we have restarting services, Before= and After= don't really mean anything.
That these don't match expectations can be seen by the needing to add ping checks for the various services / DNS resolutions because the services came up before what they were After

:ship:  in next train


Migrating all cert-manager side-projects to "Makefile modules"

We want to reduce the time spent on "dumb" tasks (upgrading dependencies, fixing small bugs, ...).

A first improvement we made was enabling dependabot grouped upgrades, which allows us to more easily upgrade Go dependencies.
The second improvement we envision is to deduplicate most of our Makefile logic across projects and centralise the logic in a single location. This will prevent Makefile improvements & fixes to be applied to only a single project and thus should help us use the best quality and most feature-complete logic in every project. Additionally, this simplifies setting up the necessary tools to easily release new versions of every project using CD (instead of building locally on our laptops).

This project is the "source of truth" for the shared Makefile modules used in the cert-manager side-projects.
In this issue, we will track the progress of migrating all our side-projects to use "Makefile modules":

[ ] approver-policy [ready to be merged]
https://github.com/cert-manager/approver-policy/pull/306
[ ] istio-csr [need to rename tests]
https://github.com/cert-manager/istio-csr/pull/221
[ ] csi-driver [need to update test definition]
https://github.com/cert-manager/csi-driver/pull/161
[ ] trust-manager [unresolved challenge: trust bundles]
https://github.com/cert-manager/trust-manager/pull/195
[ ] csi-driver-spiffe [not started yet]
[ ] issuer-lib [tests not finished]
https://github.com/cert-manager/issuer-lib/pull/75
[ ] csi-lib [not started yet]

moved to https://github.com/cert-manager/makefile-modules/issues/3


fix site product links

a few product links are linked to the wrong products

@michdolan @scoopxyz Any advice to @maxnbk for the CI build: macOS 10.15 <AppleClang 11.0 config=Release, shared=ON, sse=ON, cxx=11, python=3.7, docs=ON>:
ERROR: Exception:
Traceback (most recent call last):
  File "/Users/runner/hostedtoolcache/Python/3.7.8/x64/lib/python3.7/site-packages/pip/_internal/cli/base_command.py", line 216, in _main
    status = self.run(options, args)
  File "/Users/runner/hostedtoolcache/Python/3.7.8/x64/lib/python3.7/site-packages/pip/_internal/cli/req_command.py", line 182, in wrapper
    return func(self, options, args)
  File "/Users/runner/hostedtoolcache/Python/3.7.8/x64/lib/python3.7/site-packages/pip/_internal/commands/install.py", line 421, in run
    pycompile=options.compile,
  File "/Users/runner/hostedtoolcache/Python/3.7.8/x64/lib/python3.7/site-packages/pip/_internal/req/__init__.py", line 90, in install_given_reqs
    pycompile=pycompile,
  File "/Users/runner/hostedtoolcache/Python/3.7.8/x64/lib/python3.7/site-packages/pip/_internal/req/req_install.py", line 821, in install
    requested=self.user_supplied,
  File "/Users/runner/hostedtoolcache/Python/3.7.8/x64/lib/python3.7/site-packages/pip/_internal/operations/install/wheel.py", line 860, in install_wheel
    requested=requested,
  File "/Users/runner/hostedtoolcache/Python/3.7.8/x64/lib/python3.7/site-packages/pip/_internal/operations/install/wheel.py", line 734, in _install_wheel
    assert os.path.exists(pyc_path)
AssertionError
make[2]: *** [sphinx-tabs] Error 2
make[1]: *** [CMakeFiles/sphinx-tabs.dir/all] Error 2

Related to CI failure: https://github.com/pypa/pip/issues/8653

I'll close and reopen to see if this is still an issue. #1112 passed the same job.

Must have been a PyPi or package hiccup.


YARN-9429. Fixing status code error in ResourceManager REST api doc

Description of PR
Fixing status code error in ResourceManager REST api doc

JIRA: YARN-9429

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  0s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
markdownlint
0m  0s

markdownlint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
42m 21s

trunk passed

+1 :green_heart:
mvnsite
0m 38s

trunk passed

+1 :green_heart:
shadedclient
65m 30s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 17s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 23s

the patch passed

+1 :green_heart:
shadedclient
22m 43s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
asflicense
0m 45s

The patch does not generate ASF License warnings.

92m  0s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4225/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4225

Optional Tests
dupname asflicense mvnsite codespell markdownlint

uname
Linux e921baf51105 4.15.0-175-generic #184-Ubuntu SMP Thu Mar 24 17:48:36 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / e481b6d68856fc3c00aa7dd06d246ca84826a1aa

Max. process+thread count
521 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4225/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Thanks @ashutoshcipher for your contribution. I will close this PR because it is duplicated with #4221.

Thanks @ashutoshcipher for your contribution. I will close this PR because it is duplicated with #4221.

Thanks


Formatting long extension function declarations results in lines that are too long, and should be split across multiple lines instead.

Formatting a line like
suspend fun AbstractFactoryModuleFactoryBuilderFactoryModuleFactoryBuilder.constructBuilderModuleFactory()

Results in the same line, which exceeds 100 characters in width.
The kotlin android style guide for line wrapping indicates that the line should be split at the nearest "operator like" symbol, in this case the line should be formated to.
suspend fun AbstractFactoryModuleFactoryBuilderFactoryModuleFactoryBuilder
   .constructBuilderModuleFactory()

Sorry for the delay, this will be fixed (for functions at least) with the next release.


Fix a bug that breaks release builds

Release build failed with
❯ cargo build --release
   Compiling concurrent_arena v0.1.5 (/Users/bax/projects/concurrent_arena)
error[E0599]: no method named `is_same_arc` found for struct `Slice<'_, T>` in the current scope
   --> src/arcs.rs:100:29
    |
100 |         debug_assert!(slice.is_same_arc(_old.as_ref()));
    |                             ^^^^^^^^^^^ method not found in `Slice<'_, T>`
...
105 | pub(crate) struct Slice<'a, T>(Guard<Option<ThinArc<(), T>>>, PhantomData<&'a Arcs<T>>);
    | ---------------------------------------------------------------------------------------- method `is_same_arc` not found for this

For more information about this error, try `rustc --explain E0599`.
error: could not compile `concurrent_arena` due to previous error

To fix this, I flag code that only works in debug mode
There a slice.is_same_arc call in a debug assert. The is_same_arc method is behind the debug_assertions flag, which compiles in debug mode, but not in release mode. The debug_assert! in Arcs::do_grow contained a is_same_arc call, but debug_assert was still being compiled in release, so it failed.
Note: I didn't bump the version number or anything like that.

@BaxterEaves Thank you very much for your PR!
I will add another workflow which run tests on release mode once this PR is merged.

@BaxterEaves Thanks again!
concurrent_arena-0.1.6 is now released, containing the fix in this PR.


Activate python-deduckt command console

Enable the console script from setup.py & corrected the function name as expected from console script
Minor adjustment over main.py as it's seen that deduckt module have main def which compile the source and generates ast - so introduce absolute import and use the main function as base of the main.py
Currently   if __name__ == '__main__': doesn't have used , please suggest if any more concern on it

The original structure was copied from another command-line debugger for Python. My impression was that the goal of this stub module is to ensure that  __name__ == '__main__' doesn't hold true in deduckt.main, but this is just an assumption.
Have you ran into any problems prior to the change?

@zah Yes that python-deduckt command was not running due to unavailability of cli function , actually no def cli() is available in the library itself. compile is used in def main() of deduckt.py

OK, merging. I assume you've tested it a bit by now and it doesn't seem to break anything.

py2nim won't work with master after this change.


Modifing property in causes significant performance degradation.

In the didInsertElement hook, one of the properties is explicitly updated,
this.set('bsDateTimePicker', bsDateTimePickerFn);

Which makes the date selecting significantly slower then the native js plugin, and also the ember-cli reports a deprecation warning
DEPRECATION: A property of <perx-dashboard@view:-outlet::ember476> was modified inside the didInsertElement hook.
You should never change properties on components, services or models during didInsertElement because it causes significant performance degradation.
[deprecation id: ember-views.dispatching-modify-property]

Please kindly refactor this part a bit, the performance could be much boosted.
Cheers.

I've just experienced the same issue and I've solved it by wrapping the setting of my own date property into Ember.run.next. Not nice but seems to work:
{{bs-datetimepicker date=fromDate updateDate=(action 'updateStartDate')}}
  actions: {
    updateStartDate(startDate) {
      // this.set('fromDate', startDate);
      Ember.run.next(this, 'set', 'fromDate', startDate);
    }
  }

Hope that helps.
Fred

This is still a problem now, but probably due to another problem.
didReceiveAttrs calls _updateDateTimePicker, which updates all the attributes, not only the ones that changed. This seems to trigger a big update on the base datepicker that makes the browser lag. Maybe _updateDateTimePicker should get the attributes that changed so it can check which ones have changed? And then for the date parameter maybe have an extra check to see if the current value in the control is already the same? Something like this?
      let oldDate = dateTimePicker.date();
      let newDate = this.getAttr('date');
      let dateChanged = (!oldDate && newDate || !oldDate.isSame(newDate));

With something like this I got the control much more responsive, though not quite the same as the 'native js' version.


[feature request] Output warning to log if game uses fixed function

so we could know why game doesn't work.

Technically it already does that, you just have to know what to look for. There's also the issue that some games set fixed func state and never use that.

@K0bin Would you mind explaining which log statements may refer to fixed function usage?

For example:

SetTransform
MultiplyTransform
SetMaterial
SetLight
LightEnable

warn:  D3D9DeviceEx::SetTransform: Stub
So Stalker SC uses fixed function. Sad.


Replace primo.af with primo.so

Thought this could be useful b/c primo.af doesn't seem to be up anymore

Thanks dude!


Write automated tests: Android Basics with Compose

This line -> fun calculateTip(amount: Double, tipPercent: Double: roundUp: Boolean)
Suppose to be
fun calculateTip(amount: Double, tipPercent: Double, roundUp: Boolean)

Fixed and to be published.


Update dependency sbt/sbt to v1.9.8

This PR contains the following updates:

Package
Update
Change

sbt/sbt
patch
1.9.6 -> 1.9.8

Release Notes

sbt/sbt (sbt/sbt)
v1.9.8: 1.9.8
Compare Source
updates

Fixes IO.getModifiedOrZero on Alpine etc, by using clib stat() instead of non-standard __xstat64 abi by @bratkartoffel in https://github.com/sbt/io/pull/362
As a temporary fix for JLine issue, this disables vi-style effects inside emacs by @hvesalai in https://github.com/sbt/sbt/pull/7420
Backports fix for updateSbtClassifiers not downloading sources https://github.com/sbt/sbt/pull/7437 by @azdrojowa123
Backports missing logger methods that take Java Supplier https://github.com/sbt/sbt/pull/7447 by @mkurz

Full Changelog: https://github.com/sbt/sbt/compare/v1.9.7...v1.9.8
v1.9.7: 1.9.7
Compare Source
Highlights

sbt 1.9.7 updates its IO module to 1.9.7, which fixes parent path traversal vulnerability in IO.unzip. This was discovered and reported by Kenji Yoshida (@xuwei-k), and fixed by @eed3si9n in io#360.

Zip Slip (arbitrary file write) vulnerability
See https://github.com/sbt/sbt/security/advisories/GHSA-h9mw-grgx-2fhf for the most up to date information. This affects all sbt versions prior to 1.9.7.
Path traversal vulnerabilty was discovered in IO.unzip code. This is a very common vulnerability known as Zip Slip, and was found and fixed in plexus-archiver, Ant, etc.
Given a specially crafted zip or JAR file, IO.unzip allows writing of arbitrary file. The follow is an example of a malicious entry:
+2018-04-15 22:04:42 ..... 20 20 ../../../../../../root/.ssh/authorized_keys

When executed on some path with six levels, IO.unzip could then overwrite a file under /root/. sbt main uses IO.unzip only in pullRemoteCache and Resolvers.remote, however, many projects use IO.unzip(...) directly to implement custom tasks and tests.
Non-determinism from AutoPlugins loading
We've known that occasionally some builds non-deterministically flip-flops its behavior when a task or a setting is set by two independent AutoPlugins, i.e. two plugins that neither depends on the other.
sbt 1.9.7 attempts to fix non-determinism of plugin loading order.
This was contributed by @eed3si9n in #7404.
Other updates and fixes

Updates Coursier to 2.1.7 by @regiskuckaertz in #7392

Updates Swoval to 2.1.12 by @eatkins in io#353.

Fixes .sbtopts support for sbt runner script on Windows by @ptrdom in #7393

Adds documentation on scriptedSbt key by @mdedetrich in #7383

Includes the URL in dependencyBrowseTree log by @mkurz in #7396

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update (1.9.8). You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


add interceptor-service test cases

add interceptor-service test cases

Codecov Report
Patch coverage has no change and project coverage change: -2.82 :warning:

Comparison is base (417b800) 34.14% compared to head (92226f0) 31.33%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1102      +/-   ##
==========================================
- Coverage   34.14%   31.33%   -2.82%
==========================================
  Files         352       78     -274
  Lines       38954     7410   -31544
  Branches    12182     4209    -7973
==========================================
- Hits        13302     2322   -10980
+ Misses      25412     5088   -20324
+ Partials      240        0     -240

Flag
Coverage Δ

adapter
?

admin-domain-service
?

backoffice-domain-service
?

devportal-domain-service
?

idp-domain-service
?

runtime-domain-service
31.33% <ø> (-0.17%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 276 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fix broken CI

Currently the CI is broken, and from a quick skim, it looks like there's a plonky2 version mismatch somewhere?

Hmm I also ran into it locally as well, seems like the patch section isn't working properly somehow. Not sure why though

A temporary not so pretty fix, at least until plonky2_evm is on crates.io could be to have a dedicated branch for this repo (and possibly the cluster), where there the Cargo.toml of the evm crate points to the local plonky2 (i.e. with path = "../plonky2"). This should alleviate the problem.


Chapter 4 Logged user

Logged user
So we implemented the following logic: API returns the authentication token to the client if credentials are correct.
We will now implement the following logic: we’ll find the corresponding user of an authentication token given into the HTTP header. We’ll need to do so each time this client requests a protected page.

My question is: Do you mean "protected page"? Protected pages make sense in a standard Rails application but I'm confused about how an API app has pages. I was wondering if it might be better described as:
We’ll need to do so each time a client request requires permission.
Otherwise authorization?

Thank you for report. You totally right. Maybe the perfect
- We’ll need to do so each time this client requests a protected page.
+ We’ll need to do so each time this client requests an entry point who requires permission.

What do you think about?

+ We’ll need to do so each time this client requests an entry point who requires permission.

Swap who to which and we're good 👍
We’ll need to do so each time this client requests an entry point which requires permission.
who vs which

Thank you for report this. I corrected on 74efcea8627112d179e47b676e0c4727739db835


Replace Redis

Lets implement qdrant for embeddings. Use Redis for what it's good at - caching chats. Qdrant is fast and stable and excellent at search and filtering see benchmark.
Redis single thread execution is bad for vertical scaling. Down the road we should allow BYOD (bring your own db)
Please implement as separate docker for  independent scaling docker run -p 6333:6333 qdrant/qdrant

Since many of my projects already rely on redis, replacing the database is a very difficult task. Modify that code and request a pull request.
What's more, REDIS is not a CPU-bound operation, so it can handle 100,000+ requests per second quickly, even if it's single-threaded.

I am looking into qdrant and will make a pr once I make progress. This stuff is not for Redis - we need capabilities to insert, delete, filter, advance operations with indices etc. We want to offload this task and reduce memory requirements

I will close this for now until need is critical.


Start using CSS modules

Is your feature request related to a problem? Please describe.
Some of the consumers are still targeting internal AppUI class names. Utilizing CSS modules would prevent that.
Describe the Solution you'd like
Start using CSS modules in AppUI components. This can be an incremental effort since some of the components target elements outside of the component scope.
Describe alternatives you've considered
No response
Additional context
Our consumers are already using build tools that support building SCSS files.
Most bundlers should support CSS modules out of the box, however, it would be safer to introduce CSS modules with a new major release.

We will not work on this for AppUI 5.0 since this adds additional non-standard and bundler-specific requirements for our consumers. Ideally, CSS should be delivered separately or loaded via standard JS methods and CSS modules can still be used as an internal build step, but this is outside the scope of this release.


Ajax Supported?

is Ajax Supported?
I want to fetch result from AJAX how to do it with this Plugin?
This is My JSON data
[{"user_name":"admin","full_name":"Admin"},{"user_name":"admin1","full_name":"admin1"},{"user_name":"admin2","full_name":"admin2"}]
My JS Code
$('textarea').suggest('@', { data: $.post('/ajax/suggest',function(e){ return e; }), , map: function(user) {     return {       value: user.user_name,       text: ''+user.user_name+' '     }   } });

The plugin does not support real time when you search but I'm open for a PR :)
Anyway, you can put the initialization inside the ajax callback like this:
$.post('/ajax/suggest', function(jsonData) {
	$('textarea').suggest('@', {
		data: jsonData,
		map: function(user) {
			return { value: user.user_name, text: ''+user.user_name+' ' }
		}
	});
});

Thanks for Code.

nope not yet.. I'm still working on that. but I'm open for PR

@lodev09  PR?

pull request

okay thanks. I'm also trying to make it.

You can check how Hawkers/triggeredAutocomplete done it
https://github.com/Hawkers/triggeredAutocomplete/blob/master/jquery-ui.triggeredAutocomplete.js

I'm not so sure if I will be able to do this feature in the plugin anytime soon since this will entirely change some core codes. Basically, I'm pre-rendering the dropdown layout from the initial data and using the hidden class to toggle visible items when searching.
I'll leave this issue open incase I may have time to work on this :)

Thanks. Take your Time :)


New package: JuliaGPsDocs v0.1.0

Registering package: JuliaGPsDocs
Repository: https://github.com/JuliaGaussianProcesses/JuliaGPsDocs.jl
Created by: @theogf
Version: v0.1.0
Commit: 3898af4d7a1b4e5ef8ec0cd772c27b62c5227e6f
Reviewed by: @theogf
Reference: https://github.com/JuliaGaussianProcesses/JuliaGPsDocs.jl/commit/3898af4d7a1b4e5ef8ec0cd772c27b62c5227e6f#commitcomment-70287567
Description: General setup to generate examples for the JuliaGP packages

[noblock]
We plan to keep Julia in the title since it refers to our organization JuliaGaussianProcesses.

Why does this need to be in the registry though?

Hi @fredrikekre,
The package would be used across the whole JuliaGPs ecosystem (4 packages using it at the moment), and we would like to be able to use versioning on it. Would renaming it to JuliaGaussianProcessesDocs.jl be more acceptable ?


Do the necessary changes in coding to meet the new specification.

Published with pre-release 3.14.02


[FLINK-4847] Let RpcEndpoint.start/shutDown throw exceptions

Allowing the RpcEndpoint.start/shutDown to throw exceptions will help to let rpc endpoints
to quickly fail without having to use a callback like the FatalErrorHandler.

LGTM

Merged


clients not reporting properly

in the database I see last checked, but the clients say no updates needed. today almost every single machine wanted updates (debian and ubuntu) CentOS didn't need updates.
if I run the /opt/patch_manager/patch_checker.sh manually everything is fine and reports on how much updates are needed.
could you advice on that? or is eventually an apt-get update needed before apt-get upgrade --just-print ?
Could you eventually make the bug-report/CVE link open in a new tab / window?
thanks in advance

I will review the scripts.  I do apologize. I will create a gist for each file you need modified. Right now, we are focusing on re-working the code to enterprise-class, while keeping it FOSS.  Because this breaks the system, I think this should take priority, and will personally resolve this one for you.

Gist: https://gist.github.com/jonsjava/220d183a8c846f7e617b
Just add a new line above "#apt-get --just-print upgrade" that has "apt-get update" on each server, and in client/patch_checker.sh
I'm sorry for that simple oversight.  With as many balls as we've had in the air, we apparently only fully tested CentOS

thanks a lot nice to get updates and help that fast :-)

Everything fine after one day testing all debian / ubuntu clients are reporting the right amount of patches needed.


Add a Run Scheduled Workflow System Task (or add the ability to schedule to the Run Workflow task)

The code for this is inside io.boomerang.service.refactor.TaskServiceImpl.java

Hi @amhudson this is the new task for the loader
{
    "_id" : ObjectId("61dcb509c570b75ec2c432f8"),
    "currentVersion" : NumberLong(1),
    "description" : "This task runs the selected workflow one time in the future. It only schedules the workflow, it does not wait for it to execute. Additionally you can pass in workflow parameters if they have been defined.",
    "lastModified" : ISODate("2020-04-30T22:53:23.337Z"),
    "name" : "Run Scheduled Workflow",
    "category" : "Workflow",
    "nodetype" : "runscheduledworkflow",
    "revisions" : [
        {
            "version" : 1,
            "image" : "",
            "command" : [],
            "arguments" : [
                "runscheduledworkflow"
            ],
            "config" : [],
            "changelog" : {
                "userId" : "5e8a6c6cd0827100011c2a35",
                "reason" : "",
                "date" : ISODate("2021-01-10T22:53:23.337Z")
            }
        }
    ],
    "status" : "active",
    "createdDate" : ISODate("2021-01-10T00:01:00.000Z"),
    "icon" : "Automated task",
    "_class" : "net.boomerangplatform.model.FlowTaskTemplate",
    "verified" : true
}

https://github.com/boomerang-io/flow.loader/pull/61


Account confirmation not working

I've been in the process of implementing this library, at first with not so much success, but have been gaining traction, and have had some success. I went from (all within my own custom-hand made project with the use of membershipreboot's nuget packages, and taking a little bit of the SingleTenant's example code for a little bit of models, views, and the ChangeEmailController) it not working at all, to having di issues. I fixed all that (with a little help). Now, the login and register pages work, and registration works as well as sending out verification emails, they all work. This is where I'm now stuck at. After receiving the verification email, and clicking the link in the email - I go to the localhost:####/ChangeEmail/(verification key) page - and this is where my new problem is. I'm using some of the code taken from the (SingleTenant) sample code.
When it goes to that page, in the default setup, it told me it had "System.ArgumentException: account" and it pointed to the code "if (account.HasPassword())" in the Confirm ActionResult of the ChangeEmailController. I added a line of code above that if statement in that ActionResult - "Redirect("http://" + account.Email);" - just to see what it would do with that. With that addition, it gave me another (related) error - "Object reference not set to an instance of an object." and it pointed to that exact code I added.
So, it's obvious that the system works in loading the library, it works in being able to actually register a user, it works in being able to send an email with the proper verification key (I compared it to the database entry, it was the same code). But, then when the link to the verification method is clicked on - it doesn't seem to want to retrieve that same account data from the database.
I also made a post on stacoverflow about the issue - http://stackoverflow.com/questions/34459825/brockallen-membershipreboot-account-confirmation-not-working

So I'd suggest setting up the trace config to see the log file. This might help you diagnose how far in the code it gets and then where/why it starts to go south.

Thanks for the quick response. But, how do I set up the trace config?

Well, I tried one method of tracing, from what I found in https://msdn.microsoft.com/en-us/library/wwh16c6c.aspx - and putting this node in the system.web section of web.config -  - and got information through the trace.axd file - this information to me, kind of seems useless as to helping me "diagnose how far in the code it gets and then where/why it starts to go south". But, I don't know, clearly you have more knowledge in programming than I do, maybe you might find some of it of use, but I kind of doubt it from what I see, but who knows. This is the most interesting information I got from trace.axd:
ALL_HTTP	HTTP_CACHE_CONTROL:max-age=0 HTTP_CONNECTION:keep-alive HTTP_ACCEPT:text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,/;q=0.8 HTTP_ACCEPT_ENCODING:gzip, deflate, sdch HTTP_ACCEPT_LANGUAGE:en-US,en;q=0.8 HTTP_COOKIE:__RequestVerificationToken=IlPAmmS0DEVgvKB-RIOVk2XXJddG3ty3CY1pMvfTVuMCu7TLyumsyk2_vE-TzXDw7MnoFpfKNFYB3Mvsn7rm_AFQEumTYOBW6c_pbJwKxGA1 HTTP_HOST:localhost:5613 HTTP_USER_AGENT:Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36 HTTP_UPGRADE_INSECURE_REQUESTS:1
ALL_RAW	Cache-Control: max-age=0 Connection: keep-alive Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,/;q=0.8 Accept-Encoding: gzip, deflate, sdch Accept-Language: en-US,en;q=0.8 Cookie: __RequestVerificationToken=IlPAmmS0DEVgvKB-RIOVk2XXJddG3ty3CY1pMvfTVuMCu7TLyumsyk2_vE-TzXDw7MnoFpfKNFYB3Mvsn7rm_AFQEumTYOBW6c_pbJwKxGA1 Host: localhost:5613 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36 Upgrade-Insecure-Requests: 1
APPL_MD_PATH	/LM/W3SVC/2/ROOT
But, I must add, I looked online for tracing methods for MVC, and there seems like there're a few options, but I don't know which one in particular would provide the best data. I also know about Profiling (like RedGate tools). But, I don't know which would provide the best details in these regards, so again, I ask, what do you suggest when it comes to tracing? If you want, I can upload my project to a download source, and you can download it and see it for yourself, I'll do whatever it is you ask in order to help me fix this issue. I really do like MembershipReboot, and I know, with your help, this matter will be resolved.

Or, in the mean time, in waiting for your response, maybe the stack trace information can come handy, I don't know. Well here it is:
[NullReferenceException: Object reference not set to an instance of an object.]
BrockAllen.MembershipReboot.Mvc.Areas.UserAccount.Controllers.ChangeEmailController.Confirm(String id) in C:\Users\Big Pill_-\Documents\Visual Studio 2015\Projects\brockallen_MembershipReboot_WithoutExtraSamplesCode\brockallen_MembershipReboot\Controllers\ChangeEmailController.cs:63
lambda_method(Closure , ControllerBase , Object[] ) +104
System.Web.Mvc.ActionMethodDispatcher.Execute(ControllerBase controller, Object[] parameters) +14
System.Web.Mvc.ReflectedActionDescriptor.Execute(ControllerContext controllerContext, IDictionary2 parameters) +157    System.Web.Mvc.ControllerActionInvoker.InvokeActionMethod(ControllerContext controllerContext, ActionDescriptor actionDescriptor, IDictionary2 parameters) +27
System.Web.Mvc.Async.AsyncControllerActionInvoker.b__39(IAsyncResult asyncResult, ActionInvocation innerInvokeState) +22
System.Web.Mvc.Async.WrappedAsyncResult2.CallEndDelegate(IAsyncResult asyncResult) +29    System.Web.Mvc.Async.WrappedAsyncResultBase1.End() +49
System.Web.Mvc.Async.AsyncControllerActionInvoker.EndInvokeActionMethod(IAsyncResult asyncResult) +32
System.Web.Mvc.Async.AsyncInvocationWithFilters.b__3d() +50
System.Web.Mvc.Async.<>c__DisplayClass46.b__3f() +225
System.Web.Mvc.Async.<>c__DisplayClass33.b__32(IAsyncResult asyncResult) +10
System.Web.Mvc.Async.WrappedAsyncResult1.CallEndDelegate(IAsyncResult asyncResult) +10    System.Web.Mvc.Async.WrappedAsyncResultBase1.End() +49
System.Web.Mvc.Async.AsyncControllerActionInvoker.EndInvokeActionMethodWithFilters(IAsyncResult asyncResult) +34
System.Web.Mvc.Async.<>c__DisplayClass2b.b__1c() +26
System.Web.Mvc.Async.<>c__DisplayClass21.b__1e(IAsyncResult asyncResult) +100
System.Web.Mvc.Async.WrappedAsyncResult1.CallEndDelegate(IAsyncResult asyncResult) +10    System.Web.Mvc.Async.WrappedAsyncResultBase1.End() +49
System.Web.Mvc.Async.AsyncControllerActionInvoker.EndInvokeAction(IAsyncResult asyncResult) +27
System.Web.Mvc.Controller.b__1d(IAsyncResult asyncResult, ExecuteCoreState innerState) +13
System.Web.Mvc.Async.WrappedAsyncVoid1.CallEndDelegate(IAsyncResult asyncResult) +29    System.Web.Mvc.Async.WrappedAsyncResultBase1.End() +49
System.Web.Mvc.Controller.EndExecuteCore(IAsyncResult asyncResult) +36
System.Web.Mvc.Controller.b__15(IAsyncResult asyncResult, Controller controller) +12
System.Web.Mvc.Async.WrappedAsyncVoid1.CallEndDelegate(IAsyncResult asyncResult) +22    System.Web.Mvc.Async.WrappedAsyncResultBase1.End() +49
System.Web.Mvc.Controller.EndExecute(IAsyncResult asyncResult) +26
System.Web.Mvc.Controller.System.Web.Mvc.Async.IAsyncController.EndExecute(IAsyncResult asyncResult) +10
System.Web.Mvc.MvcHandler.b__5(IAsyncResult asyncResult, ProcessRequestState innerState) +21
System.Web.Mvc.Async.WrappedAsyncVoid1.CallEndDelegate(IAsyncResult asyncResult) +29    System.Web.Mvc.Async.WrappedAsyncResultBase1.End() +49
System.Web.Mvc.MvcHandler.EndProcessRequest(IAsyncResult asyncResult) +28
System.Web.Mvc.MvcHandler.System.Web.IHttpAsyncHandler.EndProcessRequest(IAsyncResult result) +9
System.Web.CallHandlerExecutionStep.System.Web.HttpApplication.IExecutionStep.Execute() +576
System.Web.HttpApplication.ExecuteStep(IExecutionStep step, Boolean& completedSynchronously) +157

Actually, the thing is, I have it set up and working with ninject, and I prefer lightinject, so I have set up and working with both di's, and the app has the same error whether it's ninject or lightinject, but, when the system is set up with one or the other di, the stack trace is actually different (it's a noticeably longer list with ninject, from what I can see), the stack trace I posted earlier was from the version of the app with lightinject. Also, from what I've seen, the stack trace seemed to have been a longer list of information when I implemented another method for tracing shown to me through Microsoft. I was intending to include all of this in this message, but I looked at the time, and it's getting late, and I have work tomorrow so I can get some extra overtime with holiday pay at my day job, Happy Holidays, by the way. Anyway, I assume you'll message back before I'll be back from work tomorrow, I will do what you suggest, and respond accordingly. Again, thank you.

The .NET trace mechanism is used for logging. One of the samples have the trace listener configured in web.config -- you can look for that for how to set it up.
Ah, here it is:
https://github.com/brockallen/BrockAllen.MembershipReboot/blob/master/samples/SingleTenant/SingleTenantWebApp/Web.config#L167

Ok, so, I got some trace data to show you. But first let me say, to me it kind of makes no sense, but since this is your project, I'm sure it's more than likely it'd make more sense to you. Well, the thing is, when it does a call to userAccountService.GetByVerificationKey - it does what it's supposed to, and then the trace reports "[userAccountService.GetByVerificationKey] failed to locate account: (a long id number)" - and digging deeper, going into your sample code for the single tenant project, I see that GetByVerificationKey in UserAcccountService has this line of code - "key = this.Configuration.Crypto.Hash(key);" - so, this might explain my confusion. You see, what confuses me is that when it says "failed to locate account: (a long id number)" - that "long id number" - is different from the id number I see in the database. Also, in the method GetByVerificationKey in UserAcccountService  in the single tenant project, this error is displayed after another check on "var account = userRepository.GetByVerificationKey(key);" and if "(account == null)" - which is not the case, the account exists, and that key provided is the exact key in the account. So, obviously this has me scratching my head. Hopefully, you can provide some kind of tip that'd fix this for me based on all this information. As I said, I can give you a link to the project, after I upload it to some download place online, and you can see for yourself. I have Visual Studio 2015 Enterprise, if that helps.
Anyway, these are the two trace nodes specifically associated with that request (there aren't any other nodes associated with that request):

0
3
0
8

BIGPILL_--PC

[UserAccountService.GetByVerificationKey] called for key: 91F3F7554616559526146F9CBEC3AA1BD18CE7AB513E9D55E459280A6536C7B0

0
3
0
4

BIGPILL_--PC

[UserAccountService.GetByVerificationKey] failed to locate account: 7A5F053864FC699211B4CF159A4BA900E4273BB191247862FAEA736D7065597D

And by the way, obviously you are a very well experienced programmer, to make such an ambitious goal of making a security project that is more advanced than what Microsoft is offering, you clearly are an expert. I know there must be something here that I must've done wrong, though it is not at all close to obvious to me what that may be, and I'm sure after some more back and forth, you'll be able to help me. I, and surely all the other developers out there that are using membershipreboot are clearly thankful for your work, and your willingness to help us when we have such problems.

The verification key is hashed so that if the DB is compromised that the attacker can't use the confirmation codes.
As for it not matching -- maybe case sensitivity in the DB query?

Ok, well, I didn't respond till now, because it was hard for me to think of how do I put this. I don't know what to make of what you just said, I didn't alter the db query, so I don't know how that makes sense. I decided I can always make my own hack-fix - by creating my own method doing the exact same thing. But, it's just quite a bit troubling to see that this is even occurring. It's even more troubling to see that upon doing various searches through google on this issue, and not one, from what I can see, other person has even reported this happening. I don't know why I seem to be the only person on the internet that has encountered this problem. But, I assure you, I did not change your code, and am strictly relying on your library (in it's most up-to-date Nuget packages) as it's supposed to be implemented. As I said, I can upload my project to the internet, and you can download it, and see for yourself. I don't have a clue why this is happening, and it just doesn't make any sense that it's happening. Yes, I can hack it and create a custom method, but first, I hope I can get a more accurate response from you. Clearly something strange is going on here, and I just hope you can help me in these regards.

Ok, since you didn't respond, I guess I'll have no choice but to make verification happen with the system with my own code I write (what I previously called a "hack-fix") that does what membershipreboot is supposed to be doing, but for some reason doesn't. You're explanation is odd though, I mean you are the author of this library, and you don't have any interest as to why such weird behaviour is occurring for me? All you offer is - maybe - "case sensitivity in the DB query"?  I didn't write the query, you did. But I digress, I'm not even sure if membershipreboot is such a great idea anymore, seeing as how there are over 600 issues mentioned by those that have or have tried - to implement it. But, I'm a risk taker by nature, so I say, ok, I'll give it a shot, overlooking this problem I originally was talking about, as well as the concerns I just raised (among others). I'm not trying to sound rude towards you, I just thought your behaviour has been just a little odd here. But, I have a lot of respect for you, and it's unfortunate you're not as willing to help me here as I wish you would.
But since you chose not to respond to my previous most recent comment before this one, and so I obviously guess what you expect me to do is figure it out, and leave you alone on this matter.
I just have one new question on this matter before I do that, and please let me know as good of an answer as you can on the matter:
The question: So, if I do create my own custom code (EntityFramework based mainly) that does the same thing membershipreboot should be doing when it comes to verification, will it in any way effect the functionality of membershipreboot, are there any negatives to this approach? Please let me know, thank you.

I'm sorry you had problems. I do have a day job and have to attend to those meddlesome burdens.  I wish there were more time in the day, as I'd love to be able to help. I really would. If you have more patience then I can get to it eventually, but if you need immediate support then pick a library where you can hire someone to help. Or build one yourself.

The verification key is hashed so that if the DB is compromised that the attacker can't use the confirmation codes. As for it not matching -- maybe case sensitivity in the DB query?
Ok, well, I didn't respond till now, because it was hard for me to think of how do I put this. I don't know what to make of what you just said, I didn't alter the db query, so I don't know how that makes sense.

So this verification key is what's sent to a user. It's the key that proves the owner of the email account received the email. You don't want this stored in the DB in plaintext, as an attacker with read access to the DB could then lift the value and resubmit it to compromise the system. This is more of a threat around password reset, rather than email verification, but it's the same column in the DB so for both scenarios the value is hashed.
So, if the key being sent is not matching, then either the DB query is case sensitive (I don't know what DB you're using or how it's configured and that was my first guess), or there's simply no match. There could be no match because you generated a new key and overwrote the prior value in the DB (for example if you tried to re-send the verification key, or tried to register a new email). This would be triggered by your code so you'd need to watch your DB to see why it changed from the time it was generated and sent to the time you tried to use it to verify the account.
Do the samples work for you as-is? Usually that's what people do -- get those working as expected, then figure out the deltas from the samples to your code.
From your prior issues above, it sounds like the verification key not working was not your original issue. I guess you solved the null reference exception?
As for the rest of it all, I'll basically don't know what the issue is and I can't tell from the info you've posted. I've tried to give the hints that I'd use to try to track down the problem. As for the last question -- I can't recommend altering or overriding aspects of MR unless you fully understand what MR is doing.

Ok, since you didn't respond, I guess I'll have no choice but to

This and the other comments that exhibit a sort of blackmail-like passive-aggressive tone is the main reason for my frustration expressed in the above post. Perhaps I misread them.
I'll admit there is so much ponderous text that it makes it difficult to wade thru the information. Maybe others respond better to that verbosity level, but it's simply impedance for me trying to understand the issue and provide help. I'd recommend concise questions without the conversational aspects (such as Happy New Year).
So have a Happy New Year.

The verification key is hashed so that if the DB is compromised that the attacker can't use the confirmation codes. As for it not matching -- maybe case sensitivity in the DB query?
Ok, well, I didn't respond till now, because it was hard for me to think of how do I put this. I don't know what to make of what you just said, I didn't alter the db query, so I don't know how that makes sense.

So this verification key is what's sent to a user. It's the key that proves the owner of the email account received the email. You don't want this stored in the DB in plaintext, as an attacker with read access to the DB could then lift the value and resubmit it to compromise the system. This is more of a threat around password reset, rather than email verification, but it's the same column in the DB so for both scenarios the value is hashed.
So, if the key being sent is not matching, then either the DB query is case sensitive (I don't know what DB you're using or how it's configured and that was my first guess), or there's simply no match. There could be no match because you generated a new key and overwrote the prior value in the DB (for example if you tried to re-send the verification key, or tried to register a new email). This would be triggered by your code so you'd need to watch your DB to see why it changed from the time it was generated and sent to the time you tried to use it to verify the account.
Do the samples work for you as-is? Usually that's what people do -- get those working as expected, then figure out the deltas from the samples to your code.
From your prior issues above, it sounds like the verification key not working was not your original issue. I guess you solved the null reference exception?
As for the rest of it all, I'll basically don't know what the issue is and I can't tell from the info you've posted. I've tried to give the hints that I'd use to try to track down the problem. As for the last question -- I can't recommend altering or overriding aspects of MR unless you fully understand what MR is doing.

Ok, since you didn't respond, I guess I'll have no choice but to

This and the other comments that exhibit a sort of blackmail-like passive-aggressive tone is the main reason for my frustration expressed in the above post. Perhaps I misread them.
I'll admit there is so much ponderous text that it makes it difficult to wade thru the information. Maybe others respond better to that verbosity level, but it's simply impedance for me trying to understand the issue and provide help. I'd recommend concise questions without the conversational aspects (such as Happy New Year).
So have a Happy New Year.

I said Happy New Year because I'm a human being, and I'm talking to you, another human being, and it turns out at the time of the writing, it was New Years, and since I say Happy New Years to everybody, even strangers, I said it to you too. You said I wrote a lot of words, but so did you in your response, but that's fine, you gave some good insights into your thinking on the matter.
Anyways, so, you wrote "Do the samples work for you as-is? Usually that's what people do -- get those working as expected, then figure out the deltas from the samples to your code." But, that was after I already wrote in a previous comment " I just remembered, at first I wasn't even able to make membershipreboot run through the nuget package in my own project (which I figured out how to do a few days later), and I was at that time checking out the sample projects, and upon registering a user with the singletenant project, and receiving the email - and by the way, that's another thing, you're default email mechanism doesn't seem to work either, I had to add my own code for it, but that's fine... Anyway, once the email was received in that situation, I went to the verification (for the singletentant project) link in the email - and it too had a problem - with a different error message, but there was problem there too. After that I was hoping it wouldn't happen with my custom project, but as I said - it does - just with a different error message." - so - I already addressed that question.
You also wrote "So, if the key being sent is not matching, then either the DB query is case sensitive (I don't know what DB you're using or how it's configured and that was my first guess), or there's simply no match." Good point, I never gave you that option, and yeah, I see from your samples, there's more than one option in these regards. Although, on your documentation wiki, you did specifically suggest using the SQL DB default, and actually, that is what I'm using, but I never did clarify that till now. I have SQL Server 2014 Express (with Advanced Services) installed on my computer. The DB itself was auto-configured upon first build of the Project - which I think had something to do with the code "Database.SetInitializer(new MigrateDatabaseToLatestVersion<DefaultMembershipRebootDatabase, BrockAllen.MembershipReboot.Ef.Migrations.Configuration>());" - but I'm not sure of that, all I know is it was auto-configured, and I'm sure it's the standard DB from your Project, it has these Tables in it "_MigrationHistory, GroupChilds, Groups, LinkedAccountClaims, LinkedAccounts, PasswordResetSecrets, TwoFactorAuthTokens, UserAccounts, UserCertificates, and - UserClaims".
In that same paragraph I just referenced, you continued to say "...or there's simply no match. There could be no match because you generated a new key and overwrote the prior value in the DB (for example if you tried to re-send the verification key, or tried to register a new email). This would be triggered by your code so you'd need to watch your DB to see why it changed from the time it was generated and sent to the time you tried to use it to verify the account." I'm pretty sure I'm not overwriting the original key, because, upon account creation, when it comes to the code that sends the verification email, the way it retrieves the verification key is like "var vc = account.VerificationKey;" (and that's included in the body of the email message), the "account" variable I use in that statement is the same one that creates the account "var account = _userAccountService.CreateAccount(model.Username, model.Password, model.Email);". You also said "So this verification key is what's sent to a user. It's the key that proves the owner of the email account received the email. You don't want this stored in the DB in plaintext, as an attacker with read access to the DB could then lift the value and resubmit it to compromise the system. This is more of a threat around password reset, rather than email verification, but it's the same column in the DB so for both scenarios the value is hashed." And I already said in a previous comment "proper verification key (I compared it to the database entry, it was the same code)." - so I don't know how this hashing works, but it is the exact same key in the VerificationKey column of the UserAccounts Table - the exact same key that is sent in the email upon registration, same case, same characters, everything.
I write a lot of words so that you can have as much information about this as possible. I tried to address as much of what you brought up in your previous comment as possible. I still hope you can help me. It never was my intent to have a "blackmail-like passive-aggressive tone", I just was hoping you can help me, I do not by any means want to sound rude, I'm sorry if I came off that way.

I write a lot of words so that you can have as much information about this as possible.

This is why I'm slow to respond. It takes a lot of time to parse and attempt to make sense of it all given the verbosity level.
Back to your question/issue -- From all the other stuff you just said, it sounds like perhaps what's being sent in the email is the wrong value. Are you sending the hashed value? If so, that's the wrong value. You need to email the user the verification key from the account created event. This is the property to send: https://github.com/brockallen/BrockAllen.MembershipReboot/blob/master/src/BrockAllen.MembershipReboot/AccountService/UserAccountEvents.cs#L24
That's what the default email implementation does: https://github.com/brockallen/BrockAllen.MembershipReboot/blob/master/src/BrockAllen.MembershipReboot/Notification/Email/EmailNotificationEventHandlers.cs#L102
The sample doesn't send emails at all -- if you notice, emails are just written to disk. See this line of config: https://github.com/brockallen/BrockAllen.MembershipReboot/blob/master/samples/SingleTenant/SingleTenantWebApp/Web.config#L155
I suspect all of this originated because you couldn't get the sample to work. So your changes to send the email is where the original error arose (by sending the wrong value). Just the latest guess.

Ok, that is some interesting information. Thank you. But, I'm trying to figure out how do I actually implement it, I mean, as I said, I create the account with "var account = _userAccountService.CreateAccount(model.Username, model.Password, model.Email);", and then, I had attempted to get the proper verification key with "var vk = account.VerificationKey;" - but you're now telling me to get it through the "AccountCreatedEvent" - which contains the proper verification key I should be sending in the email. Ok, that's really promising, but, my question now is - how do I get the verification key from the AccountCreatedEvent - in a way that's similar to my previous attempt at retrieving the verification key ("var vk = account.VerificationKey;") (maybe something like "var vk = AccountCreatedEvent.VerificationKey;" - which obviously is not it (I tried something like it...) - but you know what I mean - how do I get the key and put it in a variable object like that so I can easily put it in the text of an email body)?

MR uses an event bus. In retrospect, it was not the right design (or something easier should have been used), but it is currently the way it is. The wiki might be slightly out of date but IIRC it covers some of these basics, no?
YOu need to register an event handler for those events to send the key. That's the short of it.

Honestly, I am learning advanced computer programming, especially MVC development, I`m sure I will get to the point where I fully understand advanced concepts like Event Handling - and how to implement in such a way that would be able to do exactly what you're referring to - with just what you said as an explanation, and not need to come and ask such a follow-up question, and just be able to do it, no problem. But, unfortunately, I'm not at that level currently. I'm honestly sorry I'm asking you for help in such a way, seeing as how you're expecting me to just know. I'm actually embarrassed that I'm asking this.
Please overlook my non-understanding on this, and offer me information on how do I actually set up an event handler that detects the AccountCreatedEvent and reveals the VerificationKey that is associated with an actual account upon account creation. I really tried to look into your wiki, and the SingleTenant Project, and tried my best to understand how to do this on my own, but I just couldn't figure out a way to do it.
I came to this problem because initially (as I previously mentioned) - I had problems sending the verification email itself, originally, I gave the email information in the system.net section of the web.config, and just assumed your system auto-sends the verification email, and found it wasn't just auto-sent, but didn't mind, because a concern of mine was having the control of creating custom verification email messages - so I can phrase it however I wanted, and include custom html and such. But now I see it's harder than I thought to include the proper VerificationKey, so I need this further help I asked from you in the previous paragraph.
Another idea that came to me while writing this was - MembershipReboot is clearly a powerful library offering a much better level of security than the Microsoft defaults, but, and this wasn't obvious to me at the start, it is clearly made for the more advanced developers, and those that're like me don't have the knowledge to easily implement it. I hope you can help me in the way I asked in the first paragraph. But, on the other hand, I do remember reading you also offer another library with similar functionality and level of security - as a layer placed on top of the Microsoft defaults - IdentityReboot. I kind of like MembershipReboot even more than IdentityReboot. But a quick question I have is - is IdentityReboot more easy to implement? I mean the Microsoft defaults are pretty simple to work with and there documentation makes it pretty simple to understand. I do understand that you are just one man, and you humbly made this free-to-use, and I truly do appreciate that, and I'm sure everyone else that uses it feels the same. I thank you for creating these libraries, being willing to help people when they have questions, and helping me. But, on the matter of IdentityReboot, I'd rather use MembershipReboot, and I hope you can help me as I asked in the first paragraph.

Something like this:
public class CustomEmailEventHandler : IEventHandler<AccountCreatedEvent<UserAccount>>
    {
        public void Handle(AccountCreatedEvent<UserAccount> evt)
        {
            // send evt.VerificationKey to evt.Account.Email
        }
    }

and then register it with the MR config:
var config = new MembershipRebootConfiguration();
config.AddEventHandler(new CustomEmailEventHandler());

That's roughly it.
As for support and which library to use -- you will certainly get more support by using Microsoft's ASP.NET Identity -- many more people use that and thus there are more people to support it (community & Microsoft).

Thank you so much, it works! Now that you provided me the details on how to detect the Events, I am almost positive I will have no issues from this point with the system. I might not be as advanced in my understanding of development techniques as I wish, but, I am not exactly an amateur neither, though it may have sounded a bit like it at points in our conversation. Event Handling is something I haven't really done much - especially with custom libraries. But, I do have a pretty good understanding with many c#, ASP.NET, and MVC functionalities. Thanks so much again, issue closed :)

I just remembered I wanted to say to you - actually, you're wrong, your event bus system is a good way to do this. I know the way I thought would've been more optimal - which, typically, is pretty standard (like "account.VerificationKey") - but - your event bus way with event handling is good too. And, I actually I'm thankful you introduced me to this concept - now I know even more about programming than before - I know event handling, and will obviously use some of these concepts in my own code.
Also, I mentioned in my original question - that I asked the same question on stackoverflow. Obviously it was you who answered my question here, and I thank you. But, since that question is there on stackoverflow, it's best to have the question answered there too. So, I hope you read this, and I'm unsure of how GitHub works, maybe you'll be notified about this comment I just made, and if so, I assume you'll read it. But, without some kind of response from you, I'll be unsure of that. Since you answered my question here, I would hope you can go over to that stackoverflow page with my question and answer it there (with the answer that worked here), and I would mark that as the answer. I'll give that some time, and if you don't answer that question there for me to mark it as the answer, or respond to this comment in the next 5 days - then I'll just answer my question with the answer that worked here on stackoverflow on that page I linked to. Thanks a lot again.

Go ahead and answer it on SO if it's the right thing. No worries, and thanks for following up here.


Sync with master

Codecov Report

Merging #15576 into http+io will increase coverage by 0.05%.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             http+io   #15576      +/-   ##
=============================================
+ Coverage       5.91%    5.96%   +0.05%
+ Complexity      1574     1475      -99
=============================================
  Files           1560     1395     -165
  Lines          77762    71152    -6610
  Branches       10210     9187    -1023
=============================================
- Hits            4596     4241     -355
+ Misses         72611    66423    -6188
+ Partials         555      488      -67

Impacted Files
Coverage Δ
Complexity Δ

...llerinalang/compiler/semantics/analyzer/Types.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../langserver/common/constants/CommandConstants.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../ballerinalang/langserver/command/CommandUtil.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...inalang/stdlib/file/service/DirectoryCallback.java
0% <0%> (-66.67%)
0% <0%> (-3%)

.../ballerinalang/stdlib/file/service/FSListener.java
19.44% <0%> (-25%)
2% <0%> (-3%)

...inalang/stdlib/file/service/endpoint/Register.java
33.33% <0%> (-5.13%)
5% <0%> (-2%)

...ang/net/http/WebSocketServerConnectorListener.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ng/messaging/artemis/externimpl/consumer/Stop.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...g/messaging/artemis/ArtemisTransactionContext.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ging/artemis/externimpl/session/CreateSession.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

... and 203 more

Continue to review full report at Codecov.

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


MudTimePicker - error message "Conversion from String to Nullable`1 failed: String '' was not recognized as a valid DateTime."

Bug type
Component
Component name
MudTimePicker
What happened?
We get this errormessage if we use the timepicker with the editable flag.
   <MudItem xs="12" sm="6" md="4">         <MudTimePicker Label="24 hours (editable)" Editable="true" />     </MudItem>

Error message onely at the timepicker, datepicker is working fine.

Expected behavior
Same behavior like the datetimepicker.
Reproduction link
https://mudblazor.com/components/timepicker#basic-usage
Reproduction steps

Fill a time in the (24h editable) time picker
Remove the time with keyboard (backspace)
Leave the time picker focus

Relevant log output
No response
Version (bug)
5.1.5
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

I do have the same problem.

I'm seeing this exact same issue in Version 6.0.5.


3D to 2D bounding box projection

Hi,
I wanted to project a 3D bounding box to 2D. However when I visualize the bounding boxes in the image, they do not fit correctly. Here are the steps I took:

Draw and visualize the bounding box in the nerfstudio viewer using this API in the viser: https://viser.studio/server/#viser.ViserServer.add_box
Save the center, dimensions, and the Quaternion rotation of the bounding box
Load the frame, frame transform matrix (camera extrinsics) and camera intrinsics using the default dataloader.
To my understanding, transform matrix is of shape (3x4) and is in camera to world notation, so I append [0,0,0,1] to the last row and take the inverse of it to get world to camera projections.
compute the rotation matrix R as:

qw, qx, qy, qz = quaternion
R = [[1 - 2*qy**2 - 2*qz**2, 2*qx*qy - 2*qz*qw, 2*qx*qz + 2*qy*qw],
        [2*qx*qy + 2*qz*qw, 1 - 2*qx**2 - 2*qz**2, 2*qy*qz - 2*qx*qw],
        [2*qx*qz - 2*qy*qw, 2*qy*qz + 2*qx*qw, 1 - 2*qx**2 - 2*qy**2]]

Compute the 8 corners of the bounding box:

w, h, d = dimensions / 2
vertices = torch.tensor([
                [-w, -h, -d],
                [+w, -h, -d],
                [-w, +h, -d],
                [+w, +h, -d],
                [-w, -h, +d],
                [+w, -h, +d],
                [-w, +h, +d],
                [+w, +h, +d]])

Compute corners in world coordinates:

v_world = torch.matmul(vertices, R.T) + position
v_world_homogeneous = torch.cat([vertices_world, torch.ones(8,1)], dim=1)
extrinsics = camera.camera_to_worlds
extrinsics = torch.cat((extrinsics, torch.tensor([[[0.,0.,0.,1.]]])),dim=1) # a 4x4 matrix
extrinsics = torch.linalg.inv(extrinsics)
K = camera.get_intrinsics_matrices()
K =  torch.cat((K, torch.tensor([[[0.,0.,1.]]])),dim=-1) # a 3x4 matrix
camera_parameters = K @ extrinsics
v_image = v_world_homogeneous @ camera_parameters.T
v_image = v_image[:, :2] / v_image[:, 2:3]
xmin, ymin = v_image[:,0].min(), v_image[:,1].min()
xmax, ymax = v_image[:,0].max(), v_image[:,1].max()

and xmin, ymin, xmax, ymax will be the top left and bottom right of the projected bounding box. This is the result I get:

The projections are incorrect for all of images in the dataset. Any idea what could be wrong?

Potentially there's an issue with camera downscaling? the train dataset downscales cameras automatically and maybe the resolution of the image you're visualizing doesn't match these downscaled intrinsics?

I didn't specify the downscaling option during training. To my understanding, the trainer uses the original resolution of images by default.


[BUG]: Operator chart "WATCH_NAMESPACE" cannot be set correctly

Setting the WATCH_NAMESPACE env variable does not work via configmap as it is loaded after the operator starts and chooses namespaces to watch.  Also there is not other way to set it currently.
In reference to the issue at https://github.com/cloudnative-pg/cloudnative-pg/issues/3649
The operator helm chart supports setting ENV via --set config.data and passing a list.  You can use this as a file override.  See here that mentions it in the values.yaml https://github.com/cloudnative-pg/charts/blob/main/charts/cloudnative-pg/values.yaml#L64 and here that passes in the ENV via a configmap https://github.com/cloudnative-pg/charts/blob/f7e6ddc2b3a1211588d07d723e9e3dfcb4f66485/charts/cloudnative-pg/templates/config.yaml#L29
I made an override.yaml to pass in with the contents
config:
  data:
    WATCH_NAMESPACE: namespace-a,namespace-b

then I made two namespaces to match:  namespace-a and namespace-b and one to ignore "ignoreme"

Then I deployed the operator using the helm chart https://github.com/cloudnative-pg/charts/tree/main/charts/cloudnative-pg to the default namespace
helm repo add cnpg https://cloudnative-pg.github.io/charts
helm upgrade -i cnpg -n default cnpg/cloudnative-pg -f override.yaml

You can see in the logs that it watches ALL namespaces but the configuration was passed in to the operator.

The configmap has the correct values

I was unable to exec into the pod to verify the ENV values but you can at least see that the values listed in decribe pod do not include WATCH_NAMESPACE

If you check the logs above you can see that it prints "wathcing all namespaces" BEFORE it loads the configurations!!!
It might (for the helm charts process describe) at least be an order of operations thing.  The operator should load configurations before deciding which namespace to watch.
I also tried various ways of passing this in as an additonalArg and it does not start or print errors to the log, but it does not appear that "watch-namespace" is any sort of valid commandline arg for the operator.  Below is one of MANY ways I tried to pass this list in.

EDIT:
I did fork and update the helm chart to pass the ENV "WATCH_NAMESPACE" and that fixed the log output

It also detected the namespace in the logs but did not deploy the resources. Which is good.

Though the cluster still deployed in the namespace without pods and that is a bit confusing.  I guess that will just ahve to be though since that is on the helm user and not the operator since the operator prevented pods from coming up and did its part.

Deploying to "namespace-a" worked as expected once the ENV in the chart was fixed.

I have forked and created a local way to set this for testing by updating deployment.yaml container section to have
- name: WATCH_NAMESPACE
   value: "{{ .Values.watchNamespace }}"

and adding the following to values.yaml

-- Namespaces to watch as comma separated list i.e namespace-a,namespace-b. Empty will default to all namespaces.
watchNamespace: ""

this is very ugly though as it requires a --set flag like this where the comma needs to be escaped, but is fine via a yaml file.

helm upgrade -i cnpg -n default charts/cnpg/ --set watchNamespace="namespace-a,namespace-b"

PR created https://github.com/cloudnative-pg/charts/pull/301

I do have to note that the Operator doesn't seem to list WATCH_NAMESPACE under the list of Available Options for this config map.

If I am not mistaken, WATCH_NAMESPACE comes from the operator framework and requires that it is set via an Environment Variable.
Since it's not an officially accepted configuration option by the CloudNativePG Operator, I believe we'll accept a PR that adds an additionalEnv option, so that you can set it manually at your own risk.

Ok I can make a new PR that uses "additionalEnv" instead.  This is fine for now.  It appears that it was intended since the chart does list it under config.data https://github.com/cloudnative-pg/charts/blob/main/charts/cloudnative-pg/values.yaml#L64
Perhaps it was a regression of the Operator because it is actually broken because they load the config after they load the namespaces to watch as laid out here https://github.com/cloudnative-pg/cloudnative-pg/issues/3649

@itay-grudev I have created a new PR that uses additional env vars  https://github.com/cloudnative-pg/charts/pull/303

Closed as completed in #303.
You can now supply your own env to the operator at your own risk. Again - I'm stressing the fact that WATCH_NAMESPACE is not officially supported by CloudNativePG.

It is supported though. https://cloudnative-pg.io/documentation/1.23/kubectl-plugin/#generation-of-installation-manifests allows you to set it during cnpg generate, which creates a yaml with the WATCH_NAMESPACE env set.  It is also mentioned here https://github.com/cloudnative-pg/cloudnative-pg/issues/3649#issuecomment-1873695889 as being supported.


Maybe need Usage

I want to more details to know how to use this tool, I'm a learner,so i want to test poc by this tools, i want to kown which file shoud i use first,maybe ..?filtering.py or fuzzer_container.py....

If you want to know the usage, a good idea would be to start with the repository README


Dependabot fails to parse a more complex poetry dependency

Package ecosystem
pip
Package manager version
poetry
Language version
Python 3.7-3.10
Manifest location and content before the Dependabot update
https://github.com/XRPLF/xrpl-py/blob/master/pyproject.toml
dependabot.yml content
version: 2
updates:
- package-ecosystem: pip
  directory: "/"
  schedule:
    interval: monthly
    time: "15:00"
  open-pull-requests-limit: 10

What you expected to see, versus what you actually saw
Dependabot hasn't been running at all on this repo, but it has on a repo with an identical dependabot.yml file. I went to Insights -> Dependency graph -> Dependabot and found this Illformed requirement error:

I'm pretty sure it's coming from this part of the pyproject.toml file:
websockets = [
    {version = ">=9.0.1 <11.0", python = ">= 3.7, < 3.10"},
    {version = "^10.0", python = "^3.10"}
]

This is completely valid in poetry, but Dependabot is struggling.
🕹 Bonus points: Smallest manifest that reproduces the issue
See above.

Hei @mvadari, thanks for the report!
This is a weird one because while Poetry supports this format, standard Python does not. I should've probably got back to you earlier when I first looked at this, to at least mention the easy workaround to you: change the requirements to be separated by commas instead of spaces.
Anyways, if Poetry supports this, we probably should too, so I opened a PR to fix it! See #5735.

Thanks!
I'm unfamiliar with how new features are added, will dependabot just automatically be resolved at some future point or will I need to change something in the config to update?

No problem!
I don't think you need to change anything, it should work next time Dependabot tries to update your repo after the fix is deployed!

Confirming that it is indeed working, thanks! 💯

Great! Thanks for circling back to let us know.


Broken images url in commit

Gitea version (or commit ref): 1.3.0+rc1
Git version: 2.15.0
Operating system: Arch Linux
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
Images in commit have the following URL:
/$USER/$REPO/raw/$COMMIT/$PATH/$FILENAME
has to be:
/$USER/$REPO/raw/commit/$COMMIT/$PATH/$FILENAME
I believe it was fine before 1.3.0-rc1.
Screenshots

I have submitted PR to fix this


Still need the "Advanced GitHub Setup" section

Relates to ongoing work described in #99

We can close this after #223 is merged

Closing now with the merger of #223


New IDs

New IDs that actually fit in the ID slot

Independent to the visuals of the new cards: should we be re-spriting things to fit in inventory slots? Most item icons in the game aren't designed around fitting neatly in a less-than-32x32 box, so doing this might mean a lot of work. I also don't inherently see the benefit.

@PJB3005 Is there any reason for the 22x22 UI slots?

The SS13 sprites are like this because they are HUD icons so they are smaller to take less room.

Looks ready for merging to me unless anyone else has an issue with the sprites.
I wish we could have some smaller sprites make for IDs on the ground since they look massive but that might hinder readability.


Simulator showing a static image without animation

When I run the simulation and go to http://localhost:8000/simulator_home, the page just displays the static image below. The command run 20 seems to run without obvious error but no characters and animation show up.

so do i. do you know how to fix it?

I have the same issue. I also checked whole city map using arrow keys but I wasn't see any animations. I also found out the list of characters didn't show information up.

同样的问题，而且不能保存

same issue, how to solve it

同样 的问题，prompt上面有输出，但是画面没有显示

Same issue. I thought reloading it in browser might update the image, but it just shows "Please start the backend first." instead of any images.

If you are developers from China mainland (like me), this problem may come from openai api failure. When excuting step 3 in README.md, frontend server keeps printing ""POST /update_environment/ HTTP/1.1" 200 14" and nothing new appears in backend server. At least it works this way on my machine. To solve it, I add these code in reverie/backend_server/utils.py:
PROXY = 'socks5h://<IP_ADDRESS>:7890'   os.environ['HTTP_PROXY'] = os.environ['http_proxy'] = PROXY   os.environ['HTTPS_PROXY'] = os.environ['https_proxy'] = PROXY   os.environ['NO_PROXY'] = os.environ['no_proxy'] = '<IP_ADDRESS>,localhost,.local'  ,
then it seems to be ok now.

I solved the issue by creating movement folder. You can refer how to solve this to https://github.com/joonspk-research/generative_agents/pull/55
The problem is reverie.py didn't check and create target directory.

Thanks, but where exactly should I put a "movement" folder in the directory structure? Should I put anything else in the folder or just make a folder. Apologies if this was covered somewhere else but I could not find it.

Thanks. I think it is discussed here: https://github.com/joonspk-research/generative_agents/issues/44#issuecomment-1676376850

Yes, you're right. You can fix that by adding codes to reveire.py to check or create the folder before accessing files under the folder. Please refer to #55.

same issue , and i have tried to create movement folder following  https://github.com/joonspk-research/generative_agents/pull/55 , it doesn't work .

Just found out this simulator screen can be moved by the ←↑↓→ key, but still don't know how to move by cursor or zoom in and out.

same issue, how to solve it

P.S: I also tried to create movement folder manually in case that's the problem (because that code that I inserted still doesn't creating that folder for some reason) and the new movement chunks (like 0,1,2,3,4,etc) in .json format in that folder are still not created automatically for my simulation so we can't see our agents and time moving we can only see it moving by playing saved by default replays because those replays like: July1_the_ville_isabella_maria_klaus-step-3-1 and etc have movement folder and those movement chunks in json format inside movement folder present.

they are sleeping in the first 2000 steps or more, be patient. After 6 am, IR will start to move.

they are sleeping in the first 2000 steps or more, be patient. After 6 am, IR will start to move.

As you can see from my image it says that agent is working at the counter thought. But the agent didn't move at all since I started the simulation.

agents processing information while sleeping or having REM phase?

They plan their scheduler and do everything they supposed to do but we can't see it actually happening we can only see their actions in the console window though.

agents processing information while sleeping or having REM phase?

They plan their scheduler and do everything they supposed to do but we can't see it actually happening we can only see their actions in the console window though.

Just wait for them. In the first step they will plan for a long time. When they are planning, they do not move. After they finish planning(it costs a long time), they will start to sleep, and in your cmd window you will see lines run fast. Then when it comes to the 2000th step or more, 6 am in the simulated town, IR will start to move. So just wait.

agents processing information while sleeping or having REM phase?

They plan their scheduler and do everything they supposed to do but we can't see it actually happening we can only see their actions in the console window though.

You can run a demo first to see what will happen if everything is ok

agents processing information while sleeping or having REM phase?

I believe they do nothing while sleeping.

Just wait for them. In the first step they will plan for a long time. When they are planning, they do not move. After they finish planning(it costs a long time), they will start to sleep, and in your cmd window you will see lines run fast. Then when it comes to the 2000th step or more, 6 am in the simulated town, IR will start to move. So just wait.

They don't move at all I waited a whole hour till the simulation ended! Do you understand.
Here is the proof that agents do planned actions but they don't gets reflected visually except the console window:

You can run a demo first to see what will happen if everything is ok

What demo? You mean replays? Replays are working great because agents movement information being saved properly to the movement folder during the simulation. We are not talking about replay here we are talking about simulation not working as it should with local LLMs though. It may work with GPT4 but many countries have no access to it and it pointless to make it online service only and it should support local LLMs in the first place.

I am having the same issue where the browser is not updating the gamestate to show anyone do any movements. It seems to be working as it is showing actions by the agents, but the map and characters and current time are not updating in any way.

have you resolved?

I am able to run the replays fine. But the simluation is not running when I am trying to see it live via: http://localhost:8000/simulator_home I have to wait till the simulation finishes to be able to see replays. I'd like to be able to see the simulation as it is running.

When I run the simulation and go to http://localhost:8000/simulator_home, the page just displays the static image below. The command run 20 seems to run without obvious error but no characters and animation show up.

have you resolved?

同样的问题依旧存在，画面不动

“environment\frontend_server\templates\home\main_script.html“  change like this, which will show the whole image.

   // Phaser 3.0 global settings.
// Configuration meant to be passed to the main Phaser game instance.
// const config = {
//   type: Phaser.AUTO,
//   width: 1500,
//   height: 800,
//   parent: "game-container",
//   pixelArt: true,
//   physics: {
//     default: "arcade",
//     arcade: {
//       gravity: { y: 0 } } },
//   scene: {
//     preload: preload,
//     create: create,
//     update: update } };

const config = {
  type: Phaser.AUTO,
  width: 4480,
  height: 3200,
  parent: "game-container",
  pixelArt: true,
  physics: {
    default: "arcade",
    arcade: {
      gravity: { y: 0 } } },
  scene: {
    preload: preload,
    create: create,
    update: update },
  scale: {zoom: 0.5} };


feat: pedersen_hash added to stark curve

Added the pedersen hash to the stark curve ecc family

Thank you @anshulforyou for you contribution! Actually, for other curves, the Pedersen hash we would be interested in would be on the companion twisted Edwards curves (twistededwards package). So either we move your Pedersen hash to the root stark-curve/ in which case the other Pedersen hashes would be in twistededwards/ (as in eddsa) or we move it to fp/ in which case the others would be in fr/ (which is the fp/ of these companion twisted Edwards).
Anyway, let me review your PR and get it merged so that you can guys use it and then we'll figure out for the other curves.

continues in #327