Role Based Access Control (RBAC) Version 1

We need a way to identify who all are inductees, members and officers in HKN and also a way to restrict access of features on the portal based on their role in the organization.

Write a script that can modify roles of existing users. Probably take in a list of users (emails) and a role that needs to be assigned to these users. Roles are "Officer", "Member", "Inductee". This script will come handy when we want to officially induct people into HKN (leading to a change of role form "Inductee" to "Member"), or when promoting/demoting a "Member" to an "Officer" role.
Have a way of restricting/providing access to content on the portal based on a user's role.

Write Python cloud function that changes user role using setCustomClaim

Check auth of incoming request (needs officer)
Invoked via HTTP
Invoke function below for changing user's doc
Create a separate function for changing user's doc
Write script that changes batch for inductees to members
Reads in text file
For each email: Invokes the Python cloud function

Done


Lint against explicit use of default npm bundler preset in liferay portal

Just like we did in https://github.com/liferay/eslint-config-liferay/issues/130 to stop redundant instances of @babel/preset-react/@babel/preset-env in liferay-portal, we should do the same for .npmbundlerrc and the default preset ("liferay-npm-bundler-preset-liferay-dev"), which applies — unless configured otherwise — wherever we use the bundler via this config file in liferay-npm-scripts.
As was the case with .babelrc and .eslintrc, we need to force the usage of .npmbundlerrc.js instead of just .npmbundlerc (JSON) so that ESLint can actually process the config. If we do this, we get this and any other linting/formatting "for free". I don't know whether the bundler is currently equipped to read ".js" files though, so changes may be needed there in order for this to happen.

@wincent .npmbundlerrc.js support is not yet available. I wanted to add it long ago but it was not so easy, so I decided to leave it for bundler v3.

Just echoing what I said here:

It may be quite easy if you use cosmiconfig. It's already in the liferay-js-toolkit dependency graph, so you could make that explicit and rely on that:
=> Found<EMAIL_ADDRESS>info Reasons this module exists
   - "_project_#lerna#@lerna#list#@lerna#command#@lerna#project" depends on it
   - Hoisted from "_project_#lerna#@lerna#list#@lerna#command#@lerna#project#cosmiconfig"

We're using it in liferay-npm-tools also.

Not quite sure, because we don't want a JS function that returns a static object, but something more complex. Some of the configuration in .npmbundlerrc is pure strings, but some other is loaders, plugins, and the like, that should be returned as instantiated functions by the .npmbundlerrc.js instead of as strings and the code managing the configuration in the bundler is not prepared for that right now.
I'm not saying that it is too difficult, but definitely not direct. Also, I think we should take advantage of that implementation to enhance the format a bit to ease configuration merging, for example.
Nevertheless, .npmbundlerrc is pure JSON. Can't we format it with eslint? :thinking:

Not quite sure, because we don't want a JS function that returns a static object, but something more complex.

I'm not convinced about that. Do we have an actual use case that would necessitate that complexity?
Pretty much all of the tools that support JS configuration files expect (or at least allow) them to export an object that corresponds to what you would write in JSON if you were doing it statically.
As counter examples to Babel, both ESLint and Prettier just expect dumb objects. I've yet to see a case where the Babel function-export approach was actually necessary.

Nevertheless, .npmbundlerrc is pure JSON. Can't we format it with eslint? 🤔

We can likely configure Prettier to read this file and tell it that it actually contains JSON, but it is much easier to do so when we can rely on a file extension. (This is probably why tools like Babel encourage people to use config files with explicit file extensions, even though it can still read .babelrc for legacy compatibility reasons.) Prettier won't help us with sorting, however, only formatting.
Not sure about ESLint though; AFAIK it only wants to deal with JavaScript source, and while JSON is a JavaScript, it's probably using the wrong tool for the job.

I'm not convinced about that. Do we have an actual use case that would necessitate that complexity?

I was thinking in creating loaders on the fly inside the very .npmbundlerrc.js file, for example. If you cannot return functions, you would need to put them on their own .js file and reference them by absolute path to make sure they get loaded. Also, you wouldn't be able to make use of closures and would need to pass all explicit configuration as a configuration object.
It's not so dramatic, but it would be great to be able to declare loaders on the fly, as you can do in webpack, for example. It leads to a much simpler configuration and lets the developer make little tweaks with ease.
We suffered a similar limitation when we switched from packages to modules to specify loaders and plugins in the .npmbundlerrc file and the change was quite positive. In this case it would even be better.

In any case, I'm not saying that we should go one way or the other. Simply, that it deserves a bit of analysis to decide what to do.

If this is needed we can go with the first approach in bundler 2 and rethink it in bundler 3. In any case we won't lose anything when supporting .js files returning plain objects.

I was thinking in creating loaders on the fly inside the very .npmbundlerrc.js file, for example. If you cannot return functions, you would need to put them on their own .js file and reference them by absolute path to make sure they get loaded.
It's not so dramatic, but it would be great to be able to declare loaders on the fly, as you can do in webpack, for example. It leads to a much simpler configuration and lets the developer make little tweaks with ease.

I think you might be conflating two orthogonal things here:

Whether the config file should export an object (as opposed to a function).
Whether you can use functions as properties within that object.

Even webpack, which is incredibly dynamic, expects its config files to export objects. I don't see any reason why the bundler couldn't allow people to define loaders inline as functions if we decide we want them to (still not clear to me that it would be necessary though, but I think that's a separate topic).
In other words, there is a ladder of complexity here that runs like this:

Config if a dumb JSON file.

This is the level that the bundler currently uses.

Config is a JS module that returns an object:

The code in that JS module is arbitrary JS and so can compute anything that JS can compute.

This is the level that we currently use in other config files in liferay-portal, and is also the level that corresponds to your example of "generating an absolute path" for later reference.

Config is a JS module that returns an object, and we allow properties in that object to be dynamic (ie. functions instead of static values):

This is the level that webpack uses.

Config may be a JS module that returns a function:

Function receives an API object as a parameter, effectively allowing inversion of control.
This is the level that Babel (optionally) uses, and which I haven't seen a convincing case for yet.

I don't think we need to race all the way to level 4 without clear cause.


I would like to add a feature to monitor replication lag (Seconds_Behind_Master) and return a 500 error if it exceeds the threshold

Hello,
First, thank you for creating such great software.
I want to propose a new feature to enhance the MySQL replication monitoring functionality of this software. Specifically, I suggest adding the ability to monitor the replication lag (Seconds_Behind_Master) and return a 500 error if the lag exceeds a specified threshold.
Feature Details:
-max-seconds-behind-master: This flag will allow users to set the maximum acceptable replication lag (in seconds). If the Seconds_Behind_Master exceeds this value, the application will return a 500 error.
This feature would be highly beneficial for users who must ensure their replication lag remains within acceptable limits.
If this approach sounds good, I would be happy to create a pull request for it.
Thank you for considering this proposal. I look forward to your feedback.

I apologize for the confusion. We need to monitor the value of Seconds_Behind_Master, not SQL_Delay.

@rnakamine Thank you for your suggestion! I haven't used this project for many years, so I don't have the motivation to make changes. Please feel free to fork the repository and modify it as needed.

@fujiwara Thank you for your response and for confirming. I understand. I will fork the repository and make the necessary modifications.


Notify organization owners about data deletion date when organization is deactivated

This issue is a follow-up to:

[ ] #24677
[ ] #24685

Once both have been implemented, we should add the following information to the notification email for organization deactivation. This should be the second line.
If data has already been automatically deleted:

All data associated with this organization has been permanently deleted.

If data deletion date is in the future:

All data associated with this organization will be permanently deleted on <date>.

@zulipbot claim

@zulipbot claim
@alya I will work on this issue


[url_launcher] Added a note to the README

The action won't work on the simulator works on physical iOS device which was not mentioned here so it was added
Description
Replace this paragraph with a description of what this PR is doing. If you're modifying existing behavior, describe the existing behavior, how this PR is changing it, and what motivated the change.
Related Issues
Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR. Note that you'll have to prefix the issue numbers with flutter/flutter#.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[ ] No, this is not a breaking change.

Thanks again for the contribution @kalkotekedar. It looks like this PR has been sitting for a little while since the last round of feedback, do you still have time to work on it? If not, let us know and one of the Flutter maintainers will be happy to shepherd it forward from here.

I've added the requested changes in the latest commit. Thanks again for the contribution @kalkotekedar!

Had this README been part of the documentation, I'd have saved myself a lot of time yesterday - please address the branch conflicts so that it can be merged and to save others the same pain - thanks.
@mklim @amirh

Landing with the red macOS run since that was a known issue with master for a day, and clearly this change can't break macOS so there's no risk of hidden test failures.


Feature: Add edit with VS Code button

📄 Description

Add edit with VS Code button (and various other GitCloneFlyout changes
💭 Motivation and context

(Partially) fixes #188 and #495

I think we should reduce the size of the button as well since there's lots of options

Need to add Code Insiders support as well.

Should navigate to VS(Code) download page when those events fail too

@0x5bfa I think we should merge. Could you review?

Thank you.


Redis minimum version requirement

Hi,
Why is the minimum version requirement v2.2? Which required feature or bug fix was included in this version?
Cheers,

Is anything below Redis 2.2 even available anymore?

That said, my guess would be the ability to write to keys with expirations set.  There are other enhancements that make everything a hell of a lot faster, but otherwise, that's the only major change I see that would make an impact.

I'm using Redis server 1.3.15 on Ubuntu 10.10. php-resque seems to work fine.
How do I test if php-resque can write to keys with expirations set?

If it isn't exploding in your face, then it should be fine.  But I'd upgrade that server ASAP.  10.10 support has been gone for a long while, and 10.04 expires in a few months.

Thanks, will do :)


Block certain substrings of a forwarded domain.

I currently use the following forward to expose my bitwarden addon to the web.
- hostname: bw.example.com
  service: https://addon_a0d7b954_bitwarden:7277

Would it be possible to for example block the following substring from being accessed by the web?
https://bw.example.com/Admin

Hi @jaddel, this is indeed possible with Cloudflare but not part of this add-on.

Open Cloudflare Zero Trust Dashboard
Navigate to Access > Applications
Add a new application (type Self-hosted)
Configure the add-on and (if you want to be able to access the admin page) add/choose the correct identity provider(s).

Take a look here for further information.

This is awesome! Many thanks for the great explanation. :)


Fix grammar issue in navbar link title

The hover title of the browser save icon is Browser Save(Experimental), and it should be Browser Save[ ](Experimental).
According to a Stack Exchange English Language post, adding a space before parentheses can increase readability.

Merged, thanks for the fix @fifn2.


extended readme on styled-components usage

README for React: Included a simpler example on how to use styled-components.

Hi, I sent a change on README for React and Travis is failing. I think this repository has another issues and all PR are failing, no matter its content.


Secret plugin configuration options

Refs #538

For some reason the way I have set these tests up keeps throwing variations on this error:
self = <click.testing.CliRunner object at 0x7fccc0d940f0>
    @contextlib.contextmanager
    def isolated_filesystem(self):
        """A context manager that creates a temporary folder and changes
        the current working directory to it for isolated filesystem tests.
        """
>       cwd = os.getcwd()
E       FileNotFoundError: [Errno 2] No such file or directory
/home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/click/testing.py:364: FileNotFoundError

After much trial and error the cause appears to be the way I was trying to create custom metadata like so:
 def test_plugin_config_env():
    os.environ["FOO_ENV"] = "FROM_ENVIRONMENT"
    for client in make_app_client(
        metadata={"plugins": {"env-plugin": {"foo": {"$env": "FOO_ENV"}}}}
    ):
        assert {"foo": "FROM_ENVIRONMENT"} == client.ds.plugin_config("env-plugin")
    del os.environ["FOO_ENV"]

It wasnt't the os.environ manipulation OR the tempfile stuff - it appeared to be the usage of make_app_client(). When I stopped trying to do that (and moved my test configuration into METADATA in the fixtures.py module) the tests started passing again.

This seems to have fixed the weird errors: https://github.com/simonw/datasette/pull/539/commits/fd6164b03ebe450a9a00df2e5be2dc7bbfbd9a3f


[9.x] Set SES MessageId to the sent message ID

This PR sets the given SES message ID as the SentMessage's message ID.
As mentioned in #40696, the addition of Message ID's to the headers of the just-sent message was removed because Symfony is generating one. But the actual SES message ID may be needed, in my case, to manage the status of the sent e-mail, that may be afterwards sent by the SES service, referencing the previously given ID.

Would this be a breaking change for existing applications? Why or why not?

After you asked me, I dug deeper and realized that this behavior is actually wrong, because the generated ID has already sent in the message headers. I'll try to find another approach to attach the returned ID. Sorry.


[WIP] #490 - Part 2 - Automated stability test

This is part 2 of the fixes for #490 - need a way to reproduce the crash to validate if it is fixed or not.
This test runs 50 times and opens the built bundle, and on my machine, tends to fail 1/50 times.
Open issues:

[ ] Reduce duplication between ci and stability tests (refactoring to common folder)

The stability test can be run via npm run test:stability. This will be helpful in verify that #490 is actually fixed...


fix example code syntax error on 5. Write Your Own Auth UI

I've try this example code.
https://github.com/aws/aws-amplify/blob/master/media/authentication_guide.md#5-write-your-own-auth-ui
But the code does not works.
./src/App.js
Syntax error: Unterminated string constant (4:72)

  2 | import logo from './logo.svg';
  3 | import './App.css';
> 4 | import { Authenticator, SignIn, SignUp, ConfirmSignUp, Greetings } from 'aws-amplify-react’;
    |                                                                         ^
  5 | import Amplify from 'aws-amplify';
  6 |
  7 | Amplify.configure({

So I've made PR to fix the example code to work well.
Thanks

Codecov Report

Merging #67 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #67   +/-   ##
=======================================
  Coverage   71.03%   71.03%
=======================================
  Files          49       49
  Lines        2213     2213
  Branches      464      464
=======================================
  Hits         1572     1572
  Misses        565      565
  Partials       76       76

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 e1ea070...eeb49dd. Read the comment docs.

thanks, not seeing any changes here seems like perhaps a copy/paste issue.


Decentralized one not working during the hosted-service downtime

Describe the bug
Getting Invalid auth issue from decentralized subgraphs.
To Reproduce
Steps to reproduce the behavior:

Go to datadog

We didn't have Invalid auth issue when we generate the graphclient without hosted-service endpoints.
But as soon as the hosted-service downtime starts, it shows Invalid auth.

Additional context
Add any other context about the problem here.

This was an issue on Graph's end.


How to approach multilingual collection tags

Say I've got a collection where a user can configure tags with the Chips field, like
export default defineCollection({
  name: 'events',
  mode: 'multi',
  translatable: true,
  fields: {
    tags: {
      type: 'chips',
      options: {
        choices: {
          marketing: 'Event marketing',
          promo: 'Promotion',
          conference: 'Conference',
          workshop: 'Workshop',
          social: 'Networking',
          festival: 'Festival',
        },
        required: true,
        placeholder: 'Select at least one category',
        visibleSuggestions: 2,
        sortable: true,
      },
    },
  },
})

How would I go about translating the tag values? They're used in the UI for filtering.
If not currently possible, then as a workaround, should I just use the value as a translation key on the client side?

The choice labels are already prepared as translatable strings under the pruvious-dashboard domain. See here:
https://github.com/pruvious/pruvious/blob/0035b6e74cde6a8967dc4f619d152ca350f28317/src/runtime/components/fields/ChipsField.vue#L12
The pruvious-dashboard is a standard translation, but you can easily override it:

Disable the standard import in nuxt.config.ts.

https://github.com/pruvious/pruvious/blob/0035b6e74cde6a8967dc4f619d152ca350f28317/src/runtime/module-types.ts#L936-L955

Extend it in a custom translatable-strings/pruvious-dashboard.en.ts file:

import { defineTranslatableStrings } from '#pruvious'
import { standardTranslatableStringsDefinition } from '#pruvious/standard'

export default defineTranslatableStrings({
  ...standardTranslatableStringsDefinition.dashboard.en,
  strings: {
    ...standardTranslatableStringsDefinition.dashboard.en.strings,
    // ...
  },
})

For the frontend (your website), it's best to use the choice values and map them to a custom translatable strings domain.


Chapter 10

Reflow and add some bits about 3 Cs, code layout effects, and perf impact of never taken branches.

Thanks Amir, I will accept your additions but revert reflow commit. The rest of the book is not formatted as 80 characters per line, so I will not introduce any inconsistency. Maybe later.
Also, I will make some more cosmetic changes as I planned for this section.

No problem, I was just editing the text offline from cli, hence the reflow.


Add CAN rgb led

Add CAN rgb led

Maybe @dagar has an opinion on this as well?

Yes, but if we're going to do anything here I'd probably want to start by making it a bit more dynamic (at startup). We're paying for a lot of additional state for every possible use case.

There's a built failure that needs to be fixed, I'll take a look.

Thank you

I've tried this PR my branch based on 1.10.1 running on the V5 Nano and I've notice that the board's led won't light up.
Stoping rgbled_pwm and starting it again will fix the issue but will to crash the UavcanNode.
Also should led be added to the DEPENDS in the CMakeLists?


Change minimum instalment value to 5 BRL

Codecov Report

Merging #99 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #99   +/-   ##
=======================================
  Coverage   99.87%   99.87%
=======================================
  Files          52       52
  Lines         818      818
=======================================
  Hits          817      817
  Misses          1        1

Impacted Files
Coverage Δ

src/Models/Configs/CreditCardConfig.php
100% <100%> (ø)
:arrow_up:

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 f7762af...f726dc0. Read the comment docs.


Tag move not being detected

As discussed in #692, when packages have changes force pushed into their repositories, the default branch rebuilds without issues. 👍 This is good!
However, if the force pushed changes include re-targeting of a tagged version, we do not invalidate those builds.
Even though it's going to be unusual to have a force push re-target a tagged version, we should probably invalidate builds when this happens.

We are definitely discarding the builds when we delete a version. The issue in this case seems to be that we still have the old association 1.0.0 → commit 3545d3d901f36f1e1a8c54af457ef3a26a704891 in the database.
main is pointing to 5477ac1e0fb8440f1e34d235c17fb922da98644c.
So is the 1.0.0 tag on Github. For some reason the re-tag hasn't been picked up.

Running this locally works just fine. But this executed a fresh clone because I hadn't run this before. I suspect there's an issue with the fetch. I'll extract the actual repo from our cache.

Our cached repo isn't correctly updating with the new tag from upstream for some reason:

The problem is that we're running git fetch and not git fetch --tags - which still wouldn't work but shows the error:
❯ git fetch --tags
From https://github.com/quickbirdstudios/XServiceLocator
 ! [rejected]        1.0.0      -> 1.0.0  (would clobber existing tag)

So the issue is that we don't see tags moving upstream because our fetch doesn't refresh them.
I'll add a pass to delete all local tags before fetching. I hope that's the last issue we'll have to deal with in this regard... I wish there was some canonical way to tell git to just accept the upstream as canonical.
For future reference, that got me to look up git repo mirroring and there is indeed git clone --mirror and git remote update.
The problem with that is that --mirror implies --bare and that means the clone isn't a checkout. I.e. we'd need to make another clone from that mirror. So while we'd save bandwidth and avoid the hassle of dealing with cruft we'd have to juggle with mirror and checkout locations. At this point it's probably just easier to deal with this issue specifically.

would clobber existing tag

I have to think this error message was written by a British person 😂

@finestructure and @daveverwer Thank you for the quick support and fix.  The patch is still giving us some trouble but I think your tag got updated before I had to roll it back 🙂

Re-opened because we're rolling back #695 via #699

Since tag deletion didn't work, I think the best way to deal with it is as follows:

run git fetch --tag
if this errors, ditch the directory and clone

It's just not worth fiddling with a repo at that point. It's sort of a cache invalidation signal and git fetch is really just a cache sync. If there's trouble we should ditch it instead of trying to patch it up.

I think this sounds great. It should also be very rare that this happens.


many warnings in our test

When I set tests on github actions saw many warnings.
We should fix then before any problems occurs.

See this https://numpy.org/devdocs/release/1.20.0-notes.html#using-the-aliases-of-builtin-types-like-np-int-is-deprecated
We have several choice to replace np.int:

int
np.int_
np.int32 or np.int64 depending the context and wanted precision.

Choice 1. et 2. don't change the behavior : the precission depend on the platform.
I would prefer choice 3. and fix the precision in the code for reader (IO implementation)
In many situtaion np.int64 is  needed and it is better to explicit.
In our test I will left np.int_ and np.float_ depending the context.
@apdavison @JuliaSprenger @mdenker ?

Hi @samuelgarcia.
Thanks for looking into the warning, the output is indeed very verbose currently.
Regarding your preferred choice option 3, we should be careful not not use np.int64 in places where it unnecessarily drastically increases memory usage. So a reader specific choice is also the best option in my mind.
Regarding options 1 and 2: Why would you go for the more verbose option of np.int_ in the tests if int would do the same job? In terms of readability int should be better, no?

Hi Julia,
I propose you have a look to the PR and make comments case by case because it depends.
you will see #991

close by #991 and #1010


Rethink device handling

model.pd_to('cuda')

How should the model be sent to device? And how should postprocess handle device?

Consider linking with the outcome of #215


Additional methods for the API

Not sure if you are looking to add this functionality but I found these useful for extracting a users post history from discourse for analysis.
Added methods to

get users replies with optional offset value
get contents of single post by post_id
get members of a group
create a group

That's helpful, do you plan to add the "add users to to a group" methods like in the php wraper?
https://github.com/discoursehosting/discourse-api-php/blob/master/lib/DiscourseAPI.php

+1 for getting an individual post by id

Thank you for contributing

No Problem, glad to contribute back to the project it's been super helpful!


Memory leak in .Net 8 using Entity Framework and Sqllite (.Net Maui Android)

Android framework version
net8.0-android, net9.0-android
Affected platform version
.NET 8 & .NET 9
Description
Hi,
We have been running our .Net Maui Android app on .Net 7 for some time now and upgraded to .Net 8.
After the upgrade, we realized that the app crashes after a few hours of use.
I have narrowed down the culprit to EF call on Sqllite. I am able to induce this problem by just reading from a table. See below:
private async Task TesterLoop()
    {
        while (true)
        {
            var testDataMessages = await GetTestDataMessages(new CancellationToken());
        }
    }

private async Task<List<TestData>> GetTestDataMessages(CancellationToken cancellationToken)
    {
	await using var dbContext = await _dbContextFactory.CreateDbContextAsync(cancellationToken);

        List<TestData> returnResult = await dbContext.TestData.AsNoTracking()
            .ToListAsync(cancellationToken);

        await dbContext.DisposeAsync().ConfigureAwait(false);

        return returnResult;
 }

Please see attached zip of test application where you can clearly see the memory rise in .Net 8 and not using .Net 7.
See below memory usage of .Net 7 app after 10 min.

See below memory usage of .Net 8 app after 10 min.

See below test application.
TestMaui.zip
Steps to Reproduce

Run Test.Maui.7 on Android device
Notice memory stays constant
Run Test.Maui.8 on Android device
Notice native memory climb fast

Did you find any workaround?
No
Relevant log output
No response

Just an update.
I was able to narrow down even more by completely removing the Maui UI from the test applications.
With the Maui UI removed, the memory still climbs rapidly for .Net 8 Android projects.
See updated test app:
TestMaui.zip
See results below:
Running Test.Android.7 for 3 min

Running Test.Android.8 for 3 min


Fix crash when uploading a package through scoped API key

fixes #642
Problem

the uploader_id gets set to the user_id of the uploader when uploading package files

in case a scoped API key is used, that user_id points to an "anonymous user"

when listing packages, that anonymous user does not have a name (null) and provokes a crash

Solution

when uploading through an API key, always retrieve the owner_id and use that as uploader_id

@gabm, can you please rebase with the latest commit on main?

done

@gabm, can you fix the lint, please?

actually the linter was very helpful.. I fixed the problems.. pls have a look

I am actually not sure whether we also want to use the owner in other places... the theory goes like this

the user might be a "virtual user" corresponding to "downgraded" privileges
the owner is the "physical user" that issued the API key and therefore the "parent" of the "virtual user" above

In some places we want to use one and in other places the other. For example when uploading a package we want to set the owner as the owner of the pkgs, but in other places we need the reduced privileges of the virtual user...
I am not familiar enough to review all the other uses or get_user and assert_user, but some might be replaced the owner method in some places. But then again its "dangerous" to touch, as it also escalates the privileges again..

I am actually not sure whether we also want to use the owner in other places... the theory goes like this

the user might be a "virtual user" corresponding to "downgraded" privileges
the owner is the "physical user" that issued the API key and therefore the "parent" of the "virtual user" above

In some places we want to use one and in other places the other. For example when uploading a package we want to set the owner as the owner of the pkgs, but in other places we need the reduced privileges of the virtual user...
I am not familiar enough to review all the other uses or get_user and assert_user, but some might be replaced the owner method in some places. But then again its "dangerous" to touch, as it also escalates the privileges again..

My intuition here would be that the current implementation before your fix, where an anonymous user is used, is the most robust in the sense that it never leaks permissions. If anything, it is problematic because you sometimes get too few permissions. Therefore, unless we are fixing a specific problem (as you did in this PR), I would not change the existing system for fear of accidentally leaking permissions for no gain.

thanks for those hints @AndreasAlbertQC . I added the tests as well.. pls have a look.
P.S.: I had a hard time working through the pytest fixtures, they are all over the place :/

Hi! Thanks for working on this. I have the same issue :-)
I did a quick test and this branch indeed works for new uploaded packages.
What about existing packages? Should this PR include a migration script to fix the database?
Also just wondering if an alternative solution could not have been to show "anonymous/none" for the uploader.
There must be a reason for the user_id to be set to this user?

The CI is fixed on main (pydantic dependency pin). @gabm rebasing your branch on main should fix the CI here.

done

What about existing packages? Should this PR include a migration script to fix the database?

For us its not a huge problem, we fixed it manually so far. Currently there is no straight forward way to fix it in the DB because the uploader information is lost iirc.

Also just wondering if an alternative solution could not have been to show "anonymous/none" for the uploader. There must be a reason for the user_id to be set to this user?

The full details are outlined here #642. In short: the current user model mixes "identity" and "authorization" to some extent. When you create an API key with a scope, then a new user without username is created and added as a member/maintainer to the packages/channels etc... When uploading a package with that API Key, it correctly resolves to that new, anonymous user that has the restricted authorization - thus preventing privilege escalation. Unfortunately, as this anonymous user has no identity (no name etc), the uploader field remains unset and that causes the error. The uploader field should be set with the identity of the "owner of the API Key". This identity should imho not be "anonymous"...

Also just wondering if an alternative solution could not have been to show "anonymous/none" for the uploader. There must be a reason for the user_id to be set to this user?

The full details are outlined here #642. In short: the current user model mixes "identity" and "authorization" to some extent. When you create an API key with a scope, then a new user without username is created and added as a member/maintainer to the packages/channels etc... When uploading a package with that API Key, it correctly resolves to that new, anonymous user that has the restricted authorization - thus preventing privilege escalation. Unfortunately, as this anonymous user has no identity (no name etc), the uploader field remains unset and that causes the error. The uploader field should be set with the identity of the "owner of the API Key". This identity should imho not be "anonymous"...

Thanks for the additional information about restricted authorization. I agree using the owner_id makes sense.

What about existing packages? Should this PR include a migration script to fix the database?

For us its not a huge problem, we fixed it manually so far. Currently there is no straight forward way to fix it in the DB because the uploader information is lost iirc.

I think it's possible to have a migration script:

we can get null users from users table
in the api_keys table, we have the link user_id / owner_id
In package_members table, we want to replace the user_id of null users with owner_id from the api_keys

Thanks @gabm!
@beenje Do you want to break out the migration followup into a separate issue or PR? In the interest of somewhat timely turnaround, I'd like to merge this PR now so that we at least have the fix in the main.

Thanks @gabm!
@beenje Do you want to break out the migration followup into a separate issue or PR? In the interest of somewhat timely turnaround, I'd like to merge this PR now so that we at least have the fix in the main.

Yes, go ahead! I will try to create another PR for the migration.


Chapter 6: DisplayText type void not assignable to Button element

/Users/daz/Code/_learn/fullstack-react-typescript-node/06/ejected-app/src/DisplayText.tsx
TypeScript error in /Users/daz/Code/_learn/fullstack-react-typescript-node/06/ejected-app/src/DisplayText.tsx(46,44):
Type '(e: React.MouseEvent<HTMLButtonElement, MouseEvent>) => void' is not assignable to type 'MouseEventHandler<HTMLButtonElement>'.
  Types of parameters 'e' and 'event' are incompatible.
    Type 'MouseEvent<HTMLButtonElement, MouseEvent>' is not assignable to type 'MouseEvent<HTMLButtonElement, MouseEvent<Element, MouseEvent>>'.
      Type 'MouseEvent' is missing the following properties from type 'MouseEvent<Element, MouseEvent>': nativeEvent, isDefaultPrevented, isPropagationStopped, persist  TS2322

    44 |
    45 |       <div>
  > 46 |         <button data-testid="input-submit" onClick={onClickShowMsg}>
       |                                            ^
    47 |           Show Message
    48 |         </button>
    49 |       </div>

Hi, I encountered the above error, while following the page 184 example verbatim.
Managed to resolve it, thanks to this answer, but don't understand why {onClickShowMsg as any} is required.  Could I get further guidance or a pointer to documentation for this?
Many thanks in advance.

Hi @darrenbarklie, I apologize for the delayed response. I'm not encountering this error on my end. Did you upgrade any of the dependencies?

Please note you should be using the same versions for your dependencies, and not just have the same code, because as versions change they can sometimes change behavior or type signatures. This is particularly true for TypeScript.
As for the issue, the typescript config file was set to force explicit use of any--as opposed to implicit. What I suspect is going on is that you've upgraded to either a newer typescript version or react and the event handler signatures have changed somewhat. By giving it an any type you're basically making it open to any type. The book had a small section, discussing the usage of the any type and how it should be avoided but can be helpful in a pinch.
If you hover over the onClick handler of your button element (NOT the onClickShowMsg function) you should see the exact function signature needed for your local version of typescript and react. If you switch to that signature you will be able to get rid of the any and it should work. Hope this helps.

By way it's pretty cool you're into Solana. That blockchain has a ton of potential.

Hi @mikerourke, thanks for the guidance on this.
I should have been more considered with the dependencies used, so appreciate how this might have generated the discrepancy. I'm now more familiar with hovering on elements to interpret function signatures, appreciate this tip.
I've successfully progressed through the Redux and React Router chapter.  Took a little time to map to the new RRv6 API, but I'm quite used to the continued rewrites of this package!
@jsoneaday I'm very interested to learn of your DeChat project on GitCoin. Is it under active development?
My ambition for this year is to level up toward a React, TypeScript, Rust stack to be able to build on Solana.  I've still a lot to learn, but find Distributed IDs and Zero Knowledge Proofs interesting to equip with the utility and speed of Solana.

Hi @mikerourke, thanks for the guidance on this.
I should have been more considered with the dependencies used, so appreciate how this might have generated the discrepancy. I'm now more familiar with hovering on elements to interpret function signatures, appreciate this tip.
I've successfully progressed through the Redux and React Router chapter.  Took a little time to map to the new RRv6 API, but I'm quite used to the continued rewrites of this package!
@jsoneaday I'm very interested to learn of your DeChat project on GitCoin. Is it under active development?
My ambition for this year is to level up toward a React, TypeScript, Rust stack to be able to build on Solana.  I've still a lot to learn, but find Distributed IDs and Zero Knowledge Proofs interesting to equip with the utility and speed of Solana.

Hey Darren its great to hear your using my book to achieve your goals.
Yes DeChat is still in active development and will use a mix of technologies including React, SwiftUI, and Rust. If you’re interested you can follow the project”s Twitter @dechataap.
Also since you’re learning Solana you’ll need to learn Rust Smart Contracts. Try my YouTube for that https://youtube.com/playlist?list=PL41Cw3fN3CfdbmhgxADwyDyIoDrxc22v2

Hey Darren its great to hear your using my book to achieve your goals.
Yes DeChat is still in active development and will use a mix of technologies including React, SwiftUI, and Rust. If you’re interested you can follow the project”s Twitter @dechataap.
Also since you’re learning Solana you’ll need to learn Rust Smart Contracts. Try my YouTube for that https://youtube.com/playlist?list=PL41Cw3fN3CfdbmhgxADwyDyIoDrxc22v2


Error running git attestor when following the quick start guide

Hello, I am currently attempting to try out witness and followed the quick start guide, but I must be missing something because I am seeing this error
$ witness run -s build -k buildkey.pem -o build-attestation.json -- bash -c "echo 'hello' > hello.txt"
INFO    Using config file: .witness.yaml
INFO    Starting environment attestor...
INFO    Starting git attestor...
ERROR   Error running git attestor: repository does not exist
ERROR   failed to run attestors: repository does not exist

when trying to follow the witness example.
My system is a VM Ubuntu 22.04.2 LTS (GNU/Linux 5.15.0-25-generic x86_64)
The witness version I am trying to run is latest, witness_0.1.13-linux_amd64.tar.gz.
 Thanks for this issue.
This is due to the fact that by default witness will attempt to run the git and environment attestors. If the git attestor runs outside of a git repo this will cause the error you're seeing.
A fix for this is to either disable the git attestor by passing in -a environment, disabling the git attestor, or running git init to create a git repo in your example directory.
Coincidentally this issue with our example and UX is something we are actively tracking.

Hello @MFry and welcome! Thanks for all the wonderful information, it's very helpful.
 If not, you can try running git init which may address this.
Alternatively, you may be able to update your witness config if you don't require a git attestor.
Let us know if this helps or if you have any more questions! Happy securing 😄

Thank you for the fast response @mikhailswift (1st place 😆 ) and @kriscoleman , I was hoping there would be a open/closed issue in github, but my search came up with nothing.
Regarding the configuration of witness config to disable the git attestor, what would that look like? I wasn't able to find anything about it in git attestor. Thanks again!

[update] It also seems like initializing an empty repo isn't enough

Ah yeah, it looks like our git attestor expects at least one commit to use as a baseline.
We may need to make some adjustments to the git attestor to handle empty repos.

@MFry thanks so much for creating this issue, we have also updated our git attestor so that it will gracefully handle an empty git repo, your feedback is much appreciated 😄


PERFSTATS is not listed in the placeholder values list

could you at least describe what kind of data are exepcted: integer (0,1), boolean?
thanks

This makes a purple, clickable box appear if you set it to 1. true does not work.


Add .checked(:compiled)

Adds a new checked option, which specifies that it will only be checked when the code is compiled.
Motivation

The compiler always checks sigs at runtime, so this provides a way to specify that sigs should not be checked in both tests and production, while accurately reflecting that the sigs will be checked when compiled (which might in fact include at test time, if the Sorbet Compiler is being used to run the tests).
Test plan

See included automated tests.

https://github.com/sorbet/sorbet/blob/4ce2239697a5c6ce9985fa4af33f4fe9cdf4672f/rewriter/AttrReader.cc#L164
^ do you mind adding another comment here like
 // Also treats `.checked(:compiled)` as checked, therefore not unchecked.

?
Also, would you mind writing a compiler test for this case?
There's some further notes about this in the Complications section of the Stripe-internal design doc.

https://github.com/sorbet/sorbet/blob/4ce2239697a5c6ce9985fa4af33f4fe9cdf4672f/gems/sorbet-runtime/lib/types/private/methods/signature_validation.rb#L173-L179
^ one thing i was curious to see here was whether we could write for whether overrides are validated with .checked(:compiled). I think that we should be able to have enough of the Signature data structures populated that we should actually be able to do the signature override checking here even when the code is running in the interpreter (despite the .checked(:compiled) behaving like .checked(:never)) as well as when the code is running in the compiler.
I think you should be able to set up a compiler test that calls like
# compiled: true
class Parent
  # ...
end
class Child < Parent
  # ...
end

T::Utils.run_all_sig_blocks
T::Private::Abstract::Validate.validate_subclass(Parent)
T::Private::Abstract::Validate.validate_subclass(Child)

or something and makes sure that if there's an override error on a child or parent sig that was marked .checked(:compiled) that even the compiler raises the error. (You could write a sorbet-runtime test that does the same thing if you wanted, but it would be redundant with the compiler test.)

If you'd like (up to you) feel free to add a note to the Sorbet docs about this:
https://sorbet.org/docs/runtime#checked-whether-to-check-in-the-first-place
or alternatively, document it somewhere in the compiler's README or docs/ folder.


Build using Mesa/Software emulation enabled for GPU-less devices

Is your feature request related to a problem? Please describe.
Nope.
Describe the solution you'd like
More so a question than a feature but is it possible to build Filament using Mesa and it's software emulation abilities to run on machines without GPUs?

It should be possible to use a SwiftShader driver since it supports ES 3.0.  Out of curiosity, what are you interested in using Filament for?

I don't know about Mesa specifically but any implementation that's OpenGL 4.1 should be compatible. We use SwiftShader (another software implementation) internally at Google to run tests on machines without GPUs.

It should be possible to use a SwiftShader driver since it supports ES 3.0. Out of curiosity, what are you interested in using Filament for?

Just to clarify, when you're building with SwiftShader/ES 3.0, are you making use of the experimental USE_EXTERNAL_GLES3 flag that is defined in the CMake files? I'm curious about passing in the SwiftShader libs into Filament's build scripts.

Yep, we use that flag internally at Google to build with SwiftShader.  One gotcha is that you might need to provide a stub implementation for glTexStorage2DMultisample because SwiftShader does not support it even though it claims to support 3.1.

My apologizes for necro'ing this closed issue but last question: When you're linking in the SwiftShader drivers along with the USE_EXTERNAL_GLES3 flag, are you modifying the rpath as well to the CMake scripts to locate the drivers? Just trying to connect the last piece of the puzzle by connecting the two libraries.
For context, I'm looking to do a similar thing as running filament on a cloud-based VM that doesn't have a GPU for testing and as a stop-gap for some light rendering until we can provision some devices with GPUs.

@AdrianAtGoogle might know the answer to your question, which might be that we staticly link it or something crazy like that :)

I've been digging more into this issue and I'm 92% confident I'm linking in EGLESv3 correctly from SwiftShader when compiling with USE_EXTERNAL_GLES3 enabled but keep getting a segfault when executing sample apps. It looks like this line is the culprit:
https://github.com/google/filament/blob/master/filament/backend/src/Platform.cpp#L112
I would imagine it should be returning PlatformEGL if we are using Open GL ES externally? Unless I'm completely missing something :/

Be sure to pass an instance of your custom Platform class into the second optional argument of Engine::create.

e.g. you could define a type like class PlatformSwiftshader : public filament::backend::OpenGLPlatform, which @AdrianAtGoogle has done internally.

Since OpenGLPlatform is an interface class, is it possible to reuse an existing Open GL ES platform implementation for PlatformSwiftShader or would I need to write custom implementations for the virtual methods?

I've attached a small example implementation (cpp + h), it might need some fixups but should give you the basic idea.
PlatformSwiftshader.zip


Add exercise sum of multiples

Just did this on the scala track and wondered why it wasn't here

Thanks for the contribution, sorry for not getting to this earlier.


Pin modes

Option of detecting and using the bits of the operating mode flags.

I think the error log parsing is a great addition, but I'd like to use it differently.
I would refactor the fuzzer to not generate invalid combinations, and throw an exception if anything fishy shows up in the error log.
This way you don't waste resources fuzzing invalid combinations, and you make the fuzzer more robust by throwing an error on unexpected warnings.
This means hardcoding that an input doesn't have a slew rate, but that just seems logical.

We don't have much automated testing, mostly because it requires hardware in the loop, but I think having an example that demonstrates these features would be helpful.

It seems like the IOB mode does not affect the flags, like 'IBUF&PULL_MODE=NONE': {(3, 24)}, 'OBUF&PULL_MODE=NONE': {(3, 24)}, 'IOBUF&PULL_MODE=NONE': {(3, 24)} so I would suggest omitting the mode from the flags.  This would probably also simplify the code and reduce runs.
You can basically fuzz the input flags on IBUF and the output flags on OBUF and then just support both on IOBUF.

Yes, it will reduce the time of base generation significantly. Although I must note that so far I do not see how this will reduce the code: the bit string for IOBUF mode will have to get at least once, which means a special case in the loop.
And the second point: now gowin_pack automatically ignores invalid flags for the mode because such information is in the database, if the mode is removed from the flags, then the correctness will have to check programmatically.
But in principle it is not difficult to do.

Fair. Do what you think is best.

Looks awesome! Doing some testing and then it's good to merge I think.
Actually I should make CI run on PRs.


feat(routing): Add routing producer strategy

Add a strategy which is capable of routing messages to multiple producers and topics. It delegates determining the route on which to send a message to a router which can inspect messages and provide a producer and topic on which the message needs to be sent. This would be useful in scenarios like slicing where a single metrics indexer needs the flexibility to send messages to different slices of snuba consumers.

Could you please beef up a bit the docstrings (they are the primary api docs) and add this module to the docs  ?
https://github.com/getsentry/arroyo/blob/main/docs/source/strategies.rst

Could you please beef up a bit the docstrings (they are the primary api docs) and add this module to the docs ? https://github.com/getsentry/arroyo/blob/main/docs/source/strategies.rst

I added more documentation. If you have ideas on what additional things to document let me know. I am open to doing that.


Territory links are case-sensitive

Description
Territories links are case-sensitive. This means that https://stacker.news/~Psychedelics works but https://stacker.news/~psychedelics does not.
Steps to Reproduce
Click on https://stacker.news/~psychedelics -> 404
Click on https://stacker.news/~Psychedelics -> 200
Expected behavior
Both links should work.
Additional context
Since we use citext, we already made sure that all queries hitting Sub.name are implicitly case-insensitive. This means we already made sure that territories with different cases cannot be created. But for some reason, the links are still case-sensitive.
https://stacker.news/items/361271

For some reason, the sub resolver seems to not be case-insensitive. With the following patch, https://stacker.news/~psychedelics finds the sub but other queries seem to fail (since the dropdown does not reflect the sub we're in)
diff --git a/api/resolvers/sub.js b/api/resolvers/sub.js
index 5d8923d..65063bd 100644
--- a/api/resolvers/sub.js
+++ b/api/resolvers/sub.js
@@ -76,9 +76,12 @@ export default {
         }).catch(console.log)
       }

-      return await models.sub.findUnique({
+      return await models.sub.findFirst({
         where: {
-          name
+          name: {
+            equals: name,
+            mode: 'insensitive'
+          }
         }
       })
     },

video
https://github.com/stackernews/stacker.news/assets/27162016/38b0d855-f58f-4c61-be8c-d4ba8e4d7d99

But Prisma seems to be aware of citext ...


Write a subhed and chatter for all three subsections

I just tossed one of these in for the charts.


blank-safe directive acts like a one-time binding

Steps:

launch the portal
open a page that uses the blank-safe directive
make sure that the parent object for the expression is available but the field is undefined, so the UI starts rendering, but then asynchronously update the field to have a value

Expected:
The directive initially sees a blank value so puts the placeholder in but then when the new value is set, it updates are uses the value over the placeholder.
Actual:
It seems to act like a one time binding where if the value isn't available at the start, that's what it goes with. Then it never updates.

Done and pushed to dev branch.


Windows version glyphs seem off for size

SegoeUI Emoji

Noto - The new Windows compatible build

An old version of Noto

The up arrow in the new version as an example of the uneven look. As if it was supposed to be very large and thus looks badly aliased when small.

Compare with the proper look:

SegoeUI Emoji

Noto - The new Windows compatible build

An old version of Noto

The up arrow in the new version as an example of the uneven look. As if it was supposed to be very large and thus looks badly aliased when small.

Compare with the proper look:

How did you manage to install them?

@joshua-612

How did you manage to install them as a system font?

Which? If you mean the current Noto, here.


model shows all 100% but not completing

Not sure exactly what's going on, but it's been like this for many minutes.  I'm running your vwplatform branch locally with my latest master merged in.  I tried refreshing, but it stayed the same.

Did it happen for only one instance or happening for every instance?

Ah, sorry dude, just that one.  A new one finished fine.


exclude_from_weight_decay in RectifiedAdam()

Describe the feature and the current behavior/state.
Can RectifiedAdam have same parameter exclude_from_weight_decay as AdamW?  We must exclude some parameters of the model from weight decay.
Relevant information

Are you willing to contribute it (yes/no): yes
If you wish to contribute, then read the requirements for new contributions in CONTRIBUTING.md
Are you willing to maintain it going forward? (yes/no): yes
Is there a relevant academic paper? (if so, where): no
Does the relavent academic paper exceed 50 citations? (yes/no): no
Is there already an implementation in another framework? (if so, where): yes
Was it part of tf.contrib? (if so, where): No

Which API type would this fall under (layer, metric, optimizer, etc.)
optimizer
Who will benefit with this feature?
Everyone
Any other info.
Thanks a lot.

Probably this component it is orphan.
Gently ping @szutenberg
@markub3327 Can you check with a ticket on https://github.com/keras-team/keras if they are interested if you port this in Keras with your exclude_from_weight_decay extension?

@bhack  The ticket is done.

Thank you as AdamW was just duplicated there 25 days ago probably you could add a comment there also for exclude_from_weight_decay for AdamW

@bhack I have done it.
Please look here.

TensorFlow Addons is transitioning to a minimal maintenance and release mode. New features will not be added to this repository. For more information, please see our public messaging on this decision:
TensorFlow Addons Wind Down
Please consider sending feature requests / contributions to other repositories in the TF community with a similar charters to TFA:
Keras
Keras-CV
Keras-NLP


Add language server for openscad files

https://github.com/dzhu/openscad-language-server looks like a good candidate.

https://github.com/Leathong/openscad-LSP looks like a better option


Implement "render function timeouts" to prevent a specific render function e.g. doInitial* from running for more than a configurable 500ms

Emit an error "<trace ... />" took more than 500ms to execute doInitialPcbComponentRender
/bounty $30

/attempt #348 I'll give it a shot

@ShiboSoftwareDev I've been looking into this one for a bit, I think it's basically impossible w/o webworkers, I probably should have investigated more before bountying. Lmk if you come to a different conclusion!!!

@ShiboSoftwareDev I've been looking into this one for a bit, I think it's basically impossible w/o webworkers, I probably should have investigated more before bountying. Lmk if you come to a different conclusion!!!

@anas-sarkez

At this point I think this is mostly not possible (technically there is probably a solution but it's a dramatic architecture change), there is a version of this that is possible by considering the maximum execution time of effects, but imo we should basically do most of the freeze detection via webworkers


Uncaught RangeError: Maximum call stack size exceeded

Getting the same thing with my map once I reached a certain number of timepoints.

@L0per can you set up an example of this happening? Would help in debugging it.

@skeate I'm currently using it on a covid-19 tracking site I made as a project: http://covid2019tracker.appspot.com/
The map tracks the total number of cases. The github is https://github.com/L0per/wuhan-coronavirus.
Using chrome, you can see the map stops at 3/9, which is when I first started getting the error, but it works fine in firefox.
The javascript for the map is in static/js/mas.js.
Sorry if that's what you meant by setting up an example, I'm pretty new to coding and github. Thanks!

That's very helpful, thank you!

This is fixed in 1.4.1. Not only is it now using iterative versions of the algorithms (so call stack size is no longer an issue), it's also using a self-balancing tree so you should see significant performance improvement.
@L0per do you mind if I link to that site in the readme, as an example of the plugin "in the wild"?

Maybe not yet, it appears to have broken with the more recent database
updates, so I'll have to fix it first within the next few weeks.
On Sat, Mar 21, 2020 at 12:37 PM Jonathan Skeate<EMAIL_ADDRESS>wrote:

This is fixed in 1.4.1. Not only is it now using iterative versions of the
algorithms (so call stack size is no longer an issue), it's also using a
self-balancing tree so you should see significant performance improvement.
@L0per https://github.com/L0per do you mind if I link to that site in
the readme, as an example of the plugin "in the wild"?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/skeate/Leaflet.timeline/issues/133#issuecomment-602093069,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMK4T46H2GNZOYD5HIZDBPDRIUJRJANCNFSM4JCVQS2A
.


Clicking on the breadcrumbs in the Files tab is broken

Hi,
Clicking on the breadcrumbs in the Files tab is broken. It redirects to a /tree/todo page. Is this implemented yet?
Thanks.

Snap you are right, this has to land in the next version!


Unexpected token - punctuation in tsv file

I have a tab-separated file that looks something like this (notice the apostrophe):
title	group
It's an apostrophe	ABC

Which I read using:
df = pd.read_csv('test.tsv', sep='\t')
Pandas can read this fine - if you print(df) it will show the data correctly. It also shows up in the variable inspector list but if you click the dataframe to view the contents I get the following error because of the apostrophe:
default.js:132 SyntaxError: Unexpected token ' in JSON at position 208
    at JSON.parse (<anonymous>)
    at handler.js:150
    at kernelconnector.js:88
    at KernelFutureHandler.<anonymous> (future.js:212)
    at Generator.next (<anonymous>)
    at fulfilled (future.js:6)

Specifically, the problem is with this line in handler.js :
var modelOptions = JSON.parse(content.replace(/^'|'$/g, ""));
quotes in the middle of a word/sentence aren't replaced with the regex. even if they are, I'm not sure thats the best approach because its changing the data (I can imagine some cases where ' is important to keep). Is there no way to parse JSON and still retain the quotes?


Fixed a regex on getprop

Add a \s* at the end of PROP_RE to catch some corner cases.
On my device (Sony Xperia Z1 (C6903)), lines returned by adb getprop end with \r\n. So the regex would not match anything.

Very thanks for your PR  :) Such kind of  feebacks/improvements are very helpfull for users.


breaking change: adjust interfaces

Breaking changes

Add RelyingParty.GetRevokeEndpoint
Add GetRefreshTokenInfo to op.Storage
Rename GetKeyByIDAndUserID -> GetKeyByIDAndClientID

@muhlemmer : if you would like this to be split into separate PRs, I can do that.

This can be reviewed commit-by-commit


Cannot clear field after selecting a date

After clicking the pikaday input box and selecting a date, you cannot clear the field with backspace.

@dylanfoster can you explain your issue in more detail? When I select a date, mark the text within the input and press backspace everything is gone for me?

@stravid I'm on 0.6.1. It looks like the onClose option was added in 0.7.0. Closing.


⚠ Secret Website 3 has degraded performance

In 2bca311, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2179 ms

Resolved: Secret Website 3 performance has improved in 97f74cf.


split registration permissions

Signed-off-by: haoqing0110<EMAIL_ADDRESS>

/assign @qiujian16 @elgnay
/cc @zhujian7

/assign @zhiweiyin318

I think we still need a secret management permission in namespace of the registration when addon-management is not needed. right?

yes, it's defined in https://github.com/open-cluster-management-io/registration-operator/pull/250/files#diff-b807821ba4bfaebd0ca64c3b0d36d3fd9430173f2b7feecd2c088fea7530f9e2R13

/approve
/lgtm


Bounding box grab handles flicker when nearing 15cm far pointer threshold and disappear when the threshold is crossed

Describe the bug
The following are two issues noticed with the bounding box component when the activation behavior is set to Activate By Proximity And Pointer.

Bounding box flickers when the hand is approximately 15cm away from it.
There's a dead zone between 0cm - 15cm range of the bounding box. The far pointer ray disappears because the hand is in a 15cm radius of a collider. The bounding box grab handles are not visible because the hand is not yet colliding with it.

[Posting both issues in a single ticket as they're closely related and might be dependent]
To reproduce
Steps to reproduce the behavior:

Go to the "BoundingBoxExamples" scene in MRTK examples.
Play the scene.
Point the hand ray to "CoffeeCup4" gameobject (far interaction).
Scroll the mouse wheel to move the hand ray towards the cup.
[Issue 1] The bounding box grab handles will start flickering at the 15cm threshold.
[Issue 2] Scroll the mouse wheel further towards the cup by approx 1-2 cm. The grab handles will disappear and only appear when the hand's index tip collides with the bounding box.

Expected behavior
For the first issue (Flickering): Flickering should not happen when the transition from far to near pointer transition is happening.
For the second issue: The grab handles should not disappear as long as the far/near ray is hitting the bounding box.
Screenshots

My Setup

Unity Version 2018.4.9f1
MRTK Version v2.0

Target Platform

HoloLens 2 (Can be simulated in the editor as well)

@thalbern
I can repo in stabilization branch. Appears to happen on transition but also when focus is on the BB handles

only

I've done some investigations into the cause of this bug. The reason this happens is because of a combination of behaviours in the focus system, near pointers and the bounding box. Here's the problem:

The hand is quite close to the BoudingBox but just not close enough for near interaction.
The ShellHandRayPointer gives focus to the BoundingBox causing the handles to become active.
The handles are closer to the hand, close enough for near interaction, so the SpherePointer returns true for IsNearObject.
This causes the ShellHandRayPointer to become inactive in DefaultPointerMediator.
With the ShellHandRayPointer inactive, and the SpherePointer not close enough to give focus, the BoundingBox loses focus and the handles become inactive again.
With the handles deactivated, the SpherePointer now returns false again and so the ShellHandRayPointer becomes active again.
We are now in the same state we started with, so return to the first step and loop.

So potential fixes for this that myself and @thalbern thought about:

making it so that the handles are not detected by the SpherePointer, so that focus remains on the root BoundingBox object.
Making it so that the handles are never deactivated, simply made invisible, so that they cannot suddenly disable far interaction by becoming active.

Method 1 has the problem that it disables near interaction grab on the handles, which is definitely undesirable.
Method 2 has the problem that it potentially introduces 20 new active colliders per BoundingBox object in the scene, which may have performance implications.
We could do a hacky fix in the BoundingBox that checks if this flickering is occurring and stops it there, but it would be treating the symptoms, rather than fixing the problem and we'd still have a few frames of flicker before it was detected and stopped.
With no obvious solution at the moment we may not fix this for 2.3, as the issue is not that severe, and solutions may require large fundamental or dangerous changes.

I moved this issue to 2.4.0 for now


donators, cytostatic (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


“Error on dataset load, continuing” when set MFT as modeltype

First of all, congratulations on the successful organization of the STIR challenge!!
When reproducing the MFT tracking method, I met an error related to loading dataset: https://github.com/athaddius/STIRMetrics/blob/5e79b195838f7e7aa3d1ef7c81d32d5280fce324/src/datatest/flow2d.py#L257             print(f"error on dataset load, continuing")
The only modification I made was to comment the 'with importlib ...' line as I am using Python 3.7 while the files function is introduced in Python 3.9
def load_config(path):     """ https://stackoverflow.com/a/67692 """     import importlib.util     import importlib     # with importlib.resources.files(__package__) as package_path: # https://stackoverflow.com/questions/6028000/how-to-read-a-static-file-from-inside-a-python-package?noredirect=1&lq=1     path = Path(path)     assert Path(path).exists(), f"config {path} does not exist!"     spec = importlib.util.spec_from_file_location("tracker_config", path)     foo = importlib.util.module_from_spec(spec)     spec.loader.exec_module(foo)     return foo.get_config(package_path)
Meanwhile, I remember that there was a MFT_STIR repo previously, which I git cloned and used here. I think this repo is missing becasue the link in the ReadMe currently directs to STIRLoader.
However, I have already reproduced the CSRT method and got almost the same results you provided (70.6 mean EPE and its accuracy).
I'd appreciate it if you could give me any advice!!

Thank you!
I would guess the reason for this error is that MFT is not installed. You should be able to install it here at MFT_STIR. We need to update the code to provide more informative error messages in future iterations.
We will fix the link on the wiki now.


self.trainer.training/testing/validating/predicting all are False

During the checking phase before the first training step, the all self.trainer.training/testing/validating/predicting status in Datamodule are Fasle.

@rohitgr7 on_after_batch_transfer

must be happening during sanity_check. for that it's self.trainer.sanity_checking. you can disable it using Trainer(num_sanity_val_steps=0) if you want.


Error when using base branch

Description
I'm getting an error when running this against my base branch.  I'm expecting :latest but getting :main
Pipeline
name: Build and Publish

on:
  push:
     branches: [ main ]

jobs:
  publish:
    runs-on: ubuntu-latest
    steps:
    - id: docker-publish
      uses:  bitovi/github-actions-docker-publish@v1
      with:
        docker_username: ${{ secrets.DOCKERHUB_USERNAME }}
        docker_password: ${{ secrets.DOCKERHUB_PASSWORD }}
        org_name: bitovi
        repo_name: fast-react-static-renderer-build
    - run: |
        echo "Image Created:  ${{ env.image }}"
        echo "Tag Created: ${{ env.tag }}"

main is the default branch

Error when I merge to main
/home/runner/work/_temp/0cad1224-9050-477e-b38d-a0bd48742d42.sh: line 10: [: missing `]'
Image_Tag: main

Expected behavior
Image_Tag: latest

Additional details
I suspect this line: https://github.com/bitovi/github-actions-docker-publish/blob/main/action.yaml#L65
should be:
elif [ "$BRANCH_OR_TAG_NAME" == "$DEFAULT_BRANCH" ] && [ ${{ inputs.use_latest }} == 'true'  ]; then

It currently ends with ]] ; then

Removed the extra bracket, and published a new release


Unable to retrieve document from URI (Windows)

When clicking "Edit in Grafana" I'm getting the error as shown in the screenshot below (Windows).

How large is the file? This suggests there's a 5Mb limit :-(

No, it's really tiny, ~ 15kb

Hmmm. Unfortunately, that error doesn't help me a great deal. Is there any more to it? It seems to be an error from vscode, not from the extension itself. Is the file in one line, perchance?

after i click "Edit in Grafana" the whole content of the folder is missing. in addition the breadcrumb navigation of the editor looks strange...

can you try something? Rename your file to *.grafana instead of *.json. Then you don't need to right click, just select the file and it should open. Does that work for you?
Still need to fix the .json version, but would be good to know if this works for you (it does for me).

@swiffer I think #57 fixes this. Can you try it? If it works, I can roll out a new release to the marketplace.

Yes, "run from repository", or I can share a vsix file

We've seen a few confirmations of #57, so I've merged and released it. Can you update your extension from the marketplace (v0.0.11) and let me know if this fixes it?

👍 it's working now. thanks - real timesaver!


Can't access WSL2 after moving

I originally installed my instance through the app store. Today I tried to move it using
lxrunoffline m -n Ubuntu -d C:\\Ubuntu

The directory appeared where expected. But it became inaccessible. Also instead of showing up as a directory with rootfs/[home|bin|etc.],  C:\\Ubuntu just has a file called ext4.vhdx.
I tried to register it with
C:\Users\ariel>lxrunoffline rg -n Ubuntu -d C:\\Ubuntu                                                                  [ERROR] Couldn't open the file "\\?\C:\Ubuntu\ext4.vhdx".                                                               Reason: The process cannot access the file because it is being used by another process.

I also tried installing with
C:\Users\ariel>lxrunoffline i -n Ubuntu -d C:\\wsl -f C:\\Ubuntu\ext4.vhdx                                              [ERROR] Couldn't open the file "C:\\Ubuntu\ext4.vhdx".                                                                  Reason: The process cannot access the file because it is being used by another process.

I tried restarting the computer a couple of times, but I still get the "used by another process" message.
How can I get my distro back given that I only have a vhdx file now?

What do you mean by saying "But it became inaccessible"? How did you access it and what's the error message?
If it is a WSL2 distro, it is expected that the directory only contains ext4.vhdx. This is because WSL2 internally uses Hyper-V for virtualization and thus uses a vhdx virtual disk to store Linux files.

What do you mean by saying "But it became inaccessible"? How did you access it and what's the error message?

When I tried bash or wsl at a CMD prompt it timed out. When I opened MobaXTerm my usual wsl tabs opened, but then they timed out and closed and the whole app closed.

If it is a WSL2 distro, it is expected that the directory only contains ext4.vhdx. This is because WSL2 internally uses Hyper-V for virtualization and thus uses a vhdx virtual disk to store Linux files.

In that case, how do I reinstall or re-register it? My attempts gave the error shown about "used by another process" even after restarting the device.
Also, without a rootfs folder, how would I access the files form Windows? A "vanilla" installed WSL has those.
Incidentally, I installed another wsl2 instance through the app store, and was able to successfully move that one using lxrunoffline. So my main issue right now is to be able to get to the files in the .vhdx file created by the first move attempt.

In that case, how do I reinstall or re-register it? My attempts gave the error shown about "used by another process" even after restarting the device.

It is incorrect to use register or install commands in the way you described. The register command is for registering a directory that has NOT been registered (e.g. when a distro is installed in a USB stick and inserted to a new computer). The install command is for installing a NEW distro from an archive file containing the required files.
I'm not very sure about why the vhdx file is in use. Probably it's because you've tried launching the distro so WSL itself has it opened. If that is the case, running wsl --shutdown will shutdown WSL and release the file.

So my main issue right now is to be able to get to the files in the .vhdx file created by the first move attempt.

I would suggest try running "unregister" and then "register" on the old distro. If this doesn't work, it is likely that either there's something wrong with WSL(2) or the files required to launch the distro in ext4.vhdx in corrupt. If WSL is refusing to load the vhdx file, the easiest way I can come up with to recover the files inside is to configure a Linux virtual machine in Hyper-V and attach the vhdx file to it. Inside the vhdx file is normal ext4 filesystem so you should be able to mount it normally and read the files.

@DDoSolitary the update solved my problem, thanks.

I fixed same problem by "netsh winsock reset"


Mock - Running existing mock crashes

When running the app with mock data for the second (or more) time crashes the app with a
thread 'main' panicked at 'assertion failed: state.friends().initialized', ui/src/main.rs:374:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I guess this is due to a recent change of #398 in friend.rs causing the initialized to not serialize properly

I believe this is fixed now .


Remote Sensing Example

Hi, Thanks for providing the code to your paper. I was wondering if there is an ETA for the remote sensing example you provide in your paper. We are trying to add your method to our UQ library. Our background is Earth Observation data and we are planning a new release soonish, so we would like include your method and create a tutorial notebook showcasing the method for a remote sensing task. Cheers and thanks again!

Hey @nilsleh, thanks for the interest in our work!
Sure, I think it would be great adding our approach to the library. How do you think the tutorial would look like?
We were thinking about sharing pretrained model with some input images showcasing confident / uncertain cases (similar to the teaser image in the paper).
I'm not sure about time, but let's say we'll try to figure it out within a couple of weeks?
Let me know if it sounds good!
Best,
Nikita

Thanks for your reply, the question was really out of curiosity and didn't mean to pressure time-wise! So personally, there would be two aspects that we like to consider for a notebook:

one, training a module with our integrated Lightning setup, to make sure we can reproduce your results, such that we have "some" confidence about having it implemented correctly. So for that case I suppose it would be checking that we can get similar quantitative results and generate a figure like the one you have in your paper
showing how a user can plug in their own dataset to apply the method for their task of interest

We tried doing that with the ZigZag MNIST Example (adding some explanation and mathematical details to the notebook is still on our ToDO List). So for the remote sensing example with the Iterative Method, we would:

train on the dataset with our implementation
evaluate and check that we can reproduce the results
have the training code in the notebook, so people know what happened, but for the purpose only load the pretrained checkpoint
with the pretrained checkpoint give some visual intuitions about the UQ with some nice figures that demonstrate the concepts

Generally we try to structure the notebooks as follows, for example in this SWAG example:

give a quick mathematical introduction to a method
show how you can use that method with code on a dataset
show some qualitative and quantitative results

From our perspective, we would only need to have details about the dataset and training scheme you used (some script similar to the ones you have provided for ZigZag MNIST or the regression cases here, would be helpful), so we can try to reproduce it, and then we can add the notebook. Of course, if you like and have any time availability, we'd also be happy if you want to add some details to a notebook or provide feedback/suggestions etc. Thanks in advance :)

Hey @nilsleh,
Sounds good! The issue with the code release for remote sensing experiments is that my colleague responsible for them recently left the lab. However, I've obtained the code from him. We have two options now: I can share the model architecture, training pipeline, data, etc., with you so you can re-implement it. Meanwhile, I'll prepare the code to add to this repository. I believe that's the best option; please email me at my university address :)
Additionally, I could release the MNIST experiments for the ICML paper (similar to what you have for Zigzag). It's a good idea too.
Best,
Nikita


Normalize geography tables in database

As a non-developer pathways user
I want the GeographyMap table to be normalized
so that I can use whatever geography layers are applicable to my geography, rather than having "us", "census division", etc. hard-coded in the column names
AC

GeographyMap and associated tables are updated so that a geographical slice can be formed out of arbitrary geographical layers, not just the ones currently named in the columns.

Reference
A (non-public) Google sheet where we are hashing out how to approach this.

@ryandrewjones, I believe this is solved in the above referenced commit (normalize_geo_14 branch) and you seem like the natural person to review it since I know Ben is extra busy right now. Note that this is based off of my postgres branch so it won't really make sense to merge it into master until the existing dispatch/postgres work is merged in.

I can't test yet until I get the new DB set up, but I went through the code and it looks good. Since you are seeing the behavior you expect, I'd say this is resolved.

Ok, we'll call it ready to merge then. (And once it's merged, then it'll be resolved :wink:)

For the record, this was merged along with #12 in 8936ffa28ef50bac261c0f67b9c74450d174fa86 (though a couple of subsequent commits were needed for cleanup).


Problem in registering for new user

I am trying to create my account for first time but it is returning same error every time. Actually, when I click on register after filling form then it is returning error.
Someone please guide me with it!

Could you elaborate on the error that you are seeing


[Question] Many-to-Many Relationship

First of all thanks for this great library. It made a lot of stuff I have to handle much easier. But now I'm confronted with a hard-to-solve problem. I have a many-to-many relationship and want to subscribe to all the children. Just like the documentation (https://github.com/hishnash/djangochannelsrestframework#subscribing-to-a-filtered-list-of-models) shows. But I can't think of a way to implement the groups_for_signal without database calls.
Let's say my models look like this:
from django.db import models

class Publication(models.Model):
    title = models.CharField(max_length=30)

    class Meta:
        ordering = ['title']

    def __str__(self):
        return self.title

class Article(models.Model):
    headline = models.CharField(max_length=100)
    publications = models.ManyToManyField(Publication)

    class Meta:
        ordering = ['headline']

    def __str__(self):
        return self.headline

(https://docs.djangoproject.com/en/3.2/topics/db/examples/many_to_many/)
To get the groups_for_signal I would need to do something like this:
for publication in instance.publications.all():
    yield f"-publication__{publication.pk}"

But as stated in the documentation this doesn't seem to be a good idea. Is there a better way that I forgot?

yer this is not a trival situation.
maybe what you could do is:
When subsceribing subscribe to all Articles (assume this is a small enough list... not in the 1000s)

Then for changes just emmit an event based on the article ID.

create a ManyToMany table and subscribe to changes on this (based on the publication id)

In the consume when you detect a change to a ManyToMany table (eg a new article is created) subscribe to that article. Or if the many to many table is updated and the article is removed from the publication un-subscribe from the articles changes.
does this make any sense? happy to mock up how this migh look in code if you need?
(if you expect there to be many articles then we need to come up with a differnt solution)
Go through your code and ensure the many to many table is also included as a prefetch related (this way you could emmmit an event for every single publication without doing extra db quries).

Hi, thanks for the ideas.
Sadly I cannot predict how many Articles will be in the system, so there I think subscribing to all of them is not a really good idea either. But the point you made with the prefetch related seems really promising in my opinion. When I prefetch all publications then should be fine using the code snippet I provided without any additional db query (please correct me if I'm wrong).


Update header to 1.3.231

Add Array2DToString and PointerDecoderArrayPointerToString to handle ppUsageCounts(array pointer). But I can't write sample to see how ppUsageCounts looks like on ascii.

CI gfxreconstruct build queued with queue ID 14108.

CI gfxreconstruct build # 2083 running.

CI gfxreconstruct build # 2083 passed.

CI gfxreconstruct build queued with queue ID 14214.

CI gfxreconstruct build # 2085 running.

CI gfxreconstruct build # 2085 passed.


Tenant Database and Main Database

I have App/User.php which use Hyn\Tenancy\Traits\UsesTenantConnection, so when my hostname is a tenanat then it authenticates from tenant's database users table.
Now what I want when the host is not a tenant I want to use App/User.php master database's users table not tenant's database users table. How to use one App/User model for both tenant and system database by identify if host tanant or not.
How to achieve this.
Thanks for your help in advance.

Short answer is "don't".  The best option is to use 2 different models (they're representing different things, right?), "Admin" and "User" (or whatever you'd like to call them).  Give them each the appropriate "Uses___Connection" trait, then update the auth guards to use the right model class at the right time.
Also, we have issue templates for a reason.  They help ensure the correct information is provided as early as possible so we can properly debug issues.  Please use them in the future.


Updated some formatting issues and a link

Nothing major, just some cleanup

You should also change the link at the top (right beside the description) to https://apps.rebble.io/en_US/application/57ef907a05e4b17e1c000186 instead of https://apps.getpebble.com/applications/57ef907a05e4b17e1c000186


Create an empty zip file in ZipToolBuilder if the entry list is empty

This can happen if you build an FLX in release mode for an app with no assets
(such as the hello_world example)

lgtm


[dart2js] Inconsistent .call results with dart2js-production-linux-d8 configuration

Mixing static interop with .call gives some results on dart2js-production-linux-d8 that are different from all the other dart2js configurations.
I only noticed because I added tests with expectations for the the current behavior and this is the one configuration that differs.
tests/lib/js/static_interop_test/call_getter_test.dart
tests/lib/js/static_interop_test/call_method_test.dart

This seems to be as expected. A test failing only on production dart2js, especially on a line expecting a TypeError, usually means the test is affected by --omit-implicit-checks. Local testing shows that flag is affecting these two tests.
I'd have to take a look at the static interop kernel transformations to be sure, but presumably it's using implicit rather than explicit checks to ensure type soundness. I don't know if it was intentionally designed like this or not, but this has the advantage that we don't have to pay for those checks in production mode, though with the usual tradeoff of needing to first test in DDC/non-production mode.
/cc @srujzs

The transformations on the JS backends lower to the js_util calls which do indeed use an implicit cast from dynamic e.g. https://github.com/dart-lang/sdk/blob/ac06b9e00685976c5e41bc9d2162abdf8fb432b3/sdk/lib/_internal/js_shared/lib/js_util_patch.dart#L108.
I presume we can make this work by just adding /// Requirements=checked-implicit-downcasts to the tests.


Error on "delete batch 1000"

OrientDB Version, operating system, or hardware.

v2.2.13 SNAPSHOT[* ]

Operating System

[ *] Linux
[ ] MacOSX
[ ] Windows
[ ] Other Unix
[ ] Other, name?

Expected behavior and actual behavior
Run "delete edge tb_edge1 where tag like '%thread-6%' batch 1000",  while responding as below:
2016-12-09 11:35:42:300 SEVER Internal server error:
Error parsing query:
select from tb_edge1 WHERE   tag LIKE '%thread-6%' BATCH 1000
^
Encountered "  "BATCH "" at line 1, column 52.
Was expecting one of:

 ...
 ...
 ...
 ...
 ...
Steps to reproduce the problem

BATCH isn't part of ODB's DELETE command.
http://orientdb.com/docs/2.2.x/SQL-Delete.html
Scott

http://orientdb.com/docs/2.2.x/SQL-Delete-Edge.html

Run "delete edge tb_edge1 where tag like '%thread-6%' batch 1000"
Quote:
http://orientdb.com/docs/2.2.x/SQL-Delete-Edge.html

Upps. I stand corrected.
One would think the BATCH keyword would be available for all mutating queries. Hmm......
Scott

Hi @wq1101
The SELECT statement does not support BATCH, you can use SKIP/LIMIT if you need pagination
Thanks
Luigi

@luigidellaquila - the select seems to be part of the error message??? The command he is running is a "delete edge".
delete edge tb_edge1 where tag like '%thread-6%' batch 1000
I am now sort of confused too.
Scott

Ouch, sorry, I misunderstood it...
I think I got the problem, I'll work on it asap
Thanks
Luigi

OK, fixed in branch 2.2.x, the fix will be released with 2.2.15
Thanks
Luigi


Add learn pages about custom types and error types

What
Add learn pages about custom types and error types.
Why
Close https://github.com/stellar/soroban-docs/issues/130

Preview is available here:http://soroban-docs-pr210.previews.kube001.services.stellar-ops.com/


Toolbar Text and Buttons color

Is there any color resource value that can be use to change the color of the text on the tool bar and the tick/cross button?
If there is no such value already could you please add one and let me know how I could achieve it?

Toolbar Icon & Text Color:
<color name="ucrop_color_toolbar_widget">@color/red_500</color>

Toolbar Background Color:
<color name="ucrop_color_toolbar">@color/blue_500</color>

Statusbar Color:
<color name="ucrop_color_statusbar">@color/blue_500</color>


Docs: Alphabetize CLI options

Partitioning this list into "has shortcut" vs. "no shortcut"
is not particularly helpful, I think. (Blaming myself, I'm the
last person to sort this list :D)
I'd argue this alphabetization is more useful than what
we had previously. Imagine browsing this list, looking for
something, but you don't know what it's called. In that
scenario, you are looking for words, not for single-character
shortcuts.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Thanks!


Bumping decimal, db_connection, and friends

What's new?

Bumps Decimal to 2.0
Removes use of Decimal.decimal? (deprecated in a 1.x version)
Bumps nimble_options to 0.5.2
Updates docker compose System.cmd calls

Why?
Great question. We intended to use this library and ran into a roadblock with some stale deps. This solves that issue for us.

Thanks @vinniefranco!
We definitely should update a bunch of this stuff.

I pushed a commit that updates a few dev dependencies.
I opened https://github.com/lexhide/xandra/pull/276 to add support for Decimal 2.0+ as well
I will look into updating Scylla separately since I have a bunch of testing to do there

Because of these reasons, I'll close the PR, but I used a bunch of the changes so thanks for opening this!! 💟

@whatyouhide thank you! And I agree. It's a better approach to break the changes up into discrete steps. 🙌🏽

To close the loop:

Bumped Scylla version in https://github.com/lexhide/xandra/pull/277
Did a bunch of digging for the ~T[] test that was failing, and it turns out it was an Elixir bug. Fixed it in https://github.com/lexhide/xandra/commit/cc2321a13c12893202f6abd34c8872c3998b9901.

Nice! Thank you! This unblocks our team 🙌🙌🙌🙌🙌


Adding WMIAntivirusProduct to wmi.yaml

Returns all information about the registered AV product.

This change is

Please fix format validation error: https://travis-ci.org/ForensicArtifacts/artifacts/jobs/286232691#L586
I've also pushed https://github.com/ForensicArtifacts/artifacts/commit/0c23f306b890a03325359d18f1903c1dbeaa407a to fix issue an on AppVeyor.

thx, LGTM


refactor: removing immer to fix max call stack error with production …

Summary:
Removed immer dependency after seeing similar issues related to production only Max Call Stack errors within the immer issue tracker.
Verification:
Tested the change against the example application along with my application. This should resolve #15.

thanks @egadstar for submitting this PR :)


Deleting relations if disconnect operation is not present

You can force relation deletion upon update by filtering out from your backend gql schema disconnect but keeping delete in the related UpdateManyWithout or UpdateOneWithout inputs.

hi @hajnalben can you add a test for that if possible? And can you describe the issue a bit better? I currently do not understand it (second vaccine shot made me dizzy ;-) )

@macrozone, I have added tests however to make this possible I had to add a new utility (nexus plugin) to your test suite.
I have learned for days the nexus-plugin-prisma code and figured that they are registering the whole DMMF in one big object tree at the end of the nexus schemas registering phase. So if you create your customized input object types then there is a chance that the schema nodes under the customized one won't be registered.
Here comes my plugin: it listens to nexus's missing type errors and from the DMMF it registers the missing ones.
With the help of this plugin I am able to fully customize my scheme anywhere. So I can filter out the disconnect operation from any UpdateOne/UpdateMany inputs and indicate the dataprovider that I want my relation to be deleted instead of simple disconnection.
Feel free to ask any questions.

@hajnalben ok i got it! I think we can merge this.
would also be nice to show an example how to force deletion instead of disconnecting.
also by the way in the tests the cat case would more be something for "disconnect"  (As a cat does not die if you no longer own it), while the UserSocialMedia is a case to delete.
There are some conflicts in the generated files. I think if you rerun the thests, it should resolve these.

@hajnalben friendly ping

@macrozone sorry for the delay. I have changed Cat to Site to be more realistic as UserSocialMedia is used for the disconnection test and I cannot find any fitting relation from your test schema.
My real life use-case of disconnection:
I have a project where I handle bookings for sport classes. The Classes has related Schedules (e.g.: every week at monday 20:00 from 2021.07.20. to infinite). On the react-admin side the Schedules can be configured through the related Class resource's edit view by an ArrayInput. When I remove an existing Schedule from the ArrayInput I want it to be deleted as a Schedule without Class is not possible due to the constraints. So disconnect here couldn't work thats why I have implemented delete.

:tada: This PR is included in version @ra-data-prisma/dataprovider-v6.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fatal Error setsockopt: connection reset by peer

I've been trying to get fabio working in production. I've tested it with curl and It works great. But when i redirected http trafic to fabio it crashes with fatal error.
2019/01/25 10:17:41 [FATAL] set tcp <IP_ADDRESS>:9999-><IP_ADDRESS>:43273: setsockopt: connection reset by peer
2019/01/25 10:17:41 [INFO] consul: Deregistering "fabio"
2019/01/25 10:17:41 [INFO] consul: Deregistering fabio
2019/01/25 10:17:41 [FATAL] ui: http: Server closed

I also tried to run it with daemon
daemon -r fabio

so it starts again after crash. It works but i don't think this is right.
Output is:
2019/01/25 10:25:00 [ERR] Failed to read proxy prefix: read tcp <IP_ADDRESS>:9999-><IP_ADDRESS>:57638: read: connection reset by peer
2019/01/25 10:25:02 [ERR] Failed to read proxy prefix: read tcp <IP_ADDRESS>:9999-><IP_ADDRESS>:57663: read: connection reset by peer
2019/01/25 10:25:04 [ERR] Failed to read proxy prefix: read tcp <IP_ADDRESS>:9999-><IP_ADDRESS>:57673: read: connection reset by peer
2019/01/25 10:25:06 [FATAL] set tcp <IP_ADDRESS>:9999-><IP_ADDRESS>:57674: setsockopt: connection reset by peer

root@gateway:~ # fabio --version
1.5.9

I have the same problem. We had changed and added a number of service when we got this:
Sep 27 14:07:34 proxy fabio[79651]: 2022/09/27 14:07:34 [FATAL] set tcp <IP_ADDRESS>:9999-><IP_ADDRESS>:45630: setsockopt: connection reset by peer Sep 27 14:07:34 proxy fabio[79651]: 2022/09/27 14:07:34 [INFO] consul: Deregistering "fabio" Sep 27 14:07:34 proxy consul[484]: 2022-09-27T14:07:34.907+0200 [INFO]  agent: Deregistered service: service=fabio-proxy.hq.ismobile.com-9998

I have the same problem.
2023/05/26 01:34:01 [ERROR] read tcp <IP_ADDRESS>:32283-><IP_ADDRESS>:8020: read: connection reset by peer


0.3.11

fixed bugs:

error when trying save model with embeds
incorrect property path in error message
required flag not respected for nested properties

dependency update
bump version

Coverage decreased (-0.2%) to 89.236% when pulling ed2e95902abd9687fa9a4ffcbf046995b929cc36 on ponury-kostek:master into 300bcc0c6c4f2620a9160a71e36fdcbeef12344b on alexandrajs:master.

Coverage decreased (-0.2%) to 89.236% when pulling ed2e95902abd9687fa9a4ffcbf046995b929cc36 on ponury-kostek:master into 300bcc0c6c4f2620a9160a71e36fdcbeef12344b on alexandrajs:master.

Coverage decreased (-0.2%) to 89.236% when pulling ed2e95902abd9687fa9a4ffcbf046995b929cc36 on ponury-kostek:master into 300bcc0c6c4f2620a9160a71e36fdcbeef12344b on alexandrajs:master.


fix: revert fallback on btree indexes when hash is unavailable

Reverts supabase/auth#1856. We plan to roll this out in the next version (v2.166.0) as per internal discussion

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 56.998%

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
9546

Relevant Lines:
16748

💛  - Coveralls


Allow Users to Register Through the Website

Currently, users need to directly contact an administrator in order to register for the website.
Most websites have a two-step registration process in which the user must follow an activation link before they can use their account. This is likely the method we will follow.

Available on production as of https://github.com/UPDDI/mps-database-server/commit/1223eab6d71075188e6c13bdc80b56ac252d700d.


Unbound: blocklists: allow scripts to extend blocklist functionality

@swhite2 some small remarks, but looks good!