Fix WACy pandemic

WAC is out of control and spreading into all authorization matters eg:
eg. https://gitter.im/solid/authorization-panel?at=6064881756debe325c5323ac ( an archival snapshot for future generations: https://web.archive.org/web/20210401065723/https://gitter.im/solid/authorization-panel?at=6064881756debe325c5323ac )

control is WACy , ACP doesn't have control access mode

Hence, this PR is an attempt to universally disassociate authorization-panel's work from WAC overarching control by using the term "change" in place of "control".
TODOs:

Update all local and other repositories..
Burn all printouts..
Smash SD cards containing any photos..
Update Wikipedia..
Hack into Internet Archive and replace all terms..

Friends, there is no "control" in a decentralised system. Down with authority! Who is with me?
All access models are equal, but some access models are more equal than others.
Four access modes good, two access modes bad.

https://gitter.im/solid/authorization-panel?at=60648961f8add214d80920ff

Sarven Capadisli @csarven Mar 31 08:38
ACP: Access "Control" Policy I don't know. not so wacy
"control" is a fairly common term out there. again, not so wacy
elf Pavlik @elf-pavlik Mar 31 08:39
right point taken

I understand that we should consider this PR as done tongue in cheek. I still wonder what we need to clarify before closing it.
UCR document initially was written as WAC UCR, later chanted to AuthZ UCR. WAC has very specific definition of acl:Append and acl:Control modes. While I agree that we can consider control as a common and neutral term, at the same time I think that we can't avoid that people familiar with WAC potentially see it through WAC goggles.
Possibly we should make another iteration on https://solid.github.io/authorization-panel/authorization-ucr/#definitions
and after that make sure we use them in consistent manner.

😆 😆 😆

@csarven , you think this will keep us from talking about WAC? You can control^h^h^hchange us!


adding blockexplorer-blockscout and other changes to the dtl and l2geth services to actually allow the replicas to run fine

Codecov Report

Merging #457 (3cee9c1) into develop (7814bca) will increase coverage by 0.14%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #457      +/-   ##
===========================================
+ Coverage    76.09%   76.23%   +0.14%
===========================================
  Files           81       79       -2
  Lines         3037     2929     -108
  Branches       469      456      -13
===========================================
- Hits          2311     2233      -78
+ Misses         726      696      -30

Flag
Coverage Δ

batch-submitter
59.51% <ø> (ø)

contracts
86.05% <ø> (ø)

core-utils
68.16% <ø> (ø)

data-transport-layer
37.68% <ø> (ø)

message-relayer
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/message-relayer/src/relay-tx.ts

packages/message-relayer/hardhat.config.ts

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 7814bca...3cee9c1. Read the comment docs.


Network stats are not sampled correct

Hey there,
the sampled network stats are not valid anymore - I suppose.
When sampling the stats for a given container, the network stats are now exposed per interface.
$ curl -s --unix-socket /var/run/docker.sock http:/containers/carbon/stats|jq .
*snip*
  "networks": {
    "eth0": {
      "rx_bytes":<PHONE_NUMBER>51,
      "rx_packets": 9150523,
      "rx_errors": 0,
      "rx_dropped": 0,
      "tx_bytes": 295780357,
      "tx_packets": 4407470,
      "tx_errors": 0,
      "tx_dropped": 0
    }
  }
}

The current code assumes to find a key network, which sums up the usage over all interfaces.
With API v1.22 this seems not to be valid anymore.
But that might be something for the dockerclient library...

Hey @ChristianKniep , indeed this seems to not be the case, and a fix was introduced to the docker client library with https://github.com/fsouza/go-dockerclient/commit/adc42c2863dd78dcabe6b50b649a7ac1cee8b5c2
which was then followed up by another patch for backwards compatibility: https://github.com/fsouza/go-dockerclient/commit/02a8beb401b20e112cff3ea740545960b667eab1
However we have pinned a certain commit in our Gomfile for this library: https://github.com/Yelp/fullerite/blob/master/Gomfile#L7
Which is a commit dated the 10th of December 2015, that precedes either of the aforementioned commits. If you want to go ahead and update the Gomfile with and pin commit https://github.com/fsouza/go-dockerclient/commit/02a8beb401b20e112cff3ea740545960b667eab1 or after that would be awesome!

Dealt with in #199

Please keep it open, since I see gaps in between the samplings and I want to understand this.
I would have expected, that fullerite samples the same amount of metrics each time it runs - but that's somehow not the case. :/
bash-4.3# tail -f /var/log/supervisor/fullerite.log
time="09 Mar 16 15:53 UCT" level=info msg="Starting to emit 9 metrics" app=fullerite handler=Graphite pkg=handler
time="09 Mar 16 15:53 UCT" level=info msg="POST of 9 metrics to Graphite took 0.002851 seconds" app=fullerite handler=Graphite pkg=handler
time="09 Mar 16 15:54 UCT" level=info msg="Starting to emit 9 metrics" app=fullerite handler=Graphite pkg=handler
time="09 Mar 16 15:54 UCT" level=info msg="POST of 9 metrics to Graphite took 0.015013 seconds" app=fullerite handler=Graphite pkg=handler
time="09 Mar 16 15:54 UCT" level=info msg="Starting to emit 7 metrics" app=fullerite handler=Graphite pkg=handler
time="09 Mar 16 15:54 UCT" level=info msg="POST of 7 metrics to Graphite took 0.002433 seconds" app=fullerite handler=Graphite pkg=handler
time="09 Mar 16 15:54 UCT" level=info msg="Starting to emit 45 metrics" app=fullerite handler=Graphite pkg=handler
time="09 Mar 16 15:54 UCT" level=info msg="POST of 45 metrics to Graphite took 0.010613 seconds" app=fullerite handler=Graphite pkg=handler
time="09 Mar 16 15:55 UCT" level=info msg="Starting to emit 32 metrics" app=fullerite handler=Graphite pkg=handler
time="09 Mar 16 15:55 UCT" level=info msg="POST of 32 metrics to Graphite took 0.004707 seconds" app=fullerite handler=Graphite pkg=handler

The config...
$ cat /etc/fullerite/fullerite.conf
{
    "prefix": "fullerite.",
    "interval": 10,
    "fulleritePort": 19191,
    "internalServer": {"port":"29090","path":"/metrics"},
    "collectorsConfigPath": "/etc/fullerite/conf.d",

    "collectors": ["DockerStats"],

    "handlers": {
        "Graphite": {
            "server": "carbon.service.consul",
            "port": "2003",
            "interval": "10",
            "max_buffer_size": 300,
            "timeout": 2
        }
    }
}

hey @ChristianKniep , fullerite does get the same number of collected metrics each time it runs, and what you see in the logs is not saying anything different. The reason is, each collector has a dedicated channel to the handler, this means as soon as a collection happens, the handler gets the metrics and assigns them to a bucket for that collector. Each time the number of metrics in the bucket exceeds $max_buffer_size of interval of emission,. this is flushed to the backend, in your case Graphite.
So basically what you see in your logs is:
time="09 Mar 16 15:53 UCT" level=info msg="POST of 9 metrics to Graphite took 0.002851 seconds" app=fullerite handler=Graphite pkg=handler     ====> this represent the emission of the metrics from one collector
time="09 Mar 16 15:54 UCT" level=info msg="Starting to emit 45 metrics" app=fullerite handler=Graphite pkg=handler ===> this represents another collector and so on and so forth

Hey @tsheasha,
ok, that make sense, but since time-series data is timestamped there should not be any loss involved, right? I'll have another look tomorrow; still fighting my jet-lack. :)


Left/Right buttons to hit context menu

Hello,
Would it be possible to add a better control to the keyboard of PowerToys Run?
Indeed, when typing the name of the program and selecting it from the keyboard arrows, it is, right now, impossible to "Launch as administrator" or "Open folder" using the keyboard arrows.
So I propose, like Windows Search (see gif below), the possibility to select these two buttons with the right and left direction arrows.

We just released 0.20 and this should be resolved there.  Please try it out.


Corrected location to developmnet requirement and updated app_dirs to false in settings Production

Corrected location to developmnet requirement in README.md
and updated APP_DIRS to false in settings/production.py

Thank you! Let me know if you have any test cases as well :smile:

I am new to django.so just looking how can I contribute in writting test cases.I will send a PR soon.


Run the tests

This is the PR addressing Issue #708
fyi @MihaiSurdeanu
Most of the tests were original, meaning that they had been sneakily not running ever since we first started developing rules, so many were about aspects of reading we didn't end up handling (i.e., different dimensions of entity modification, etc).
I modified some tests, others I tweaked rules to capture the info. Some I removed bc they weren't within the scope of what Eidos does
I guess we're not allowed to merge until after the freeze is lifted?  Or can we?  There are new rules, so I'd imagine we can't.
Also, idk why it can't automatically merge, i'll check in a sec, once I open it.
Closes #708

Thanks @BeckySharp !


Cell magics cannot handle --help when the cell is empty

It is not possible to get help using a command like %%gremlin --help unless there is something else in the cell on another line. It would be nice if that was not required. Perhaps for argument processing we should move from  directly using argparse to using @magic_arguments() decorators
Also an exception is displayed beneath the help text.

Reference doc:
https://ipython.readthedocs.io/en/stable/api/generated/IPython.core.magic_arguments.html?highlight=magic_arguments#module-IPython.core.magic_arguments

Impact: Improve getting started experience.
Currently, users have to include a Gremlin query underneath for the help to show up, which is not ideal for beginners.
%%gremlin --help

g.V().limit(10)


Pluggable Storage

I'm trying to migrate from v1 to the latest version. I have a custom storage backend for secure MacOS keychain storage written as an implementation of the old yup_oauth2::TokenStorage trait, but as far as I can tell, there's no way to do this in the new version, and none of the existing storage options suit my needs.
Would you be open to replacing the current Storage enum with a trait instead?

That refactoring work was done by @ggriffiniii, a bit more than a year ago.
I definitely agree that more useful token storage is required and wonder why we moved away from the trait based model (which you mention and I remember).
So I have a few proposals that seem viable:

Keep the enum and extend it with a third variant External(Box<ExternalStorage>) mimicking the old model, but keeping the enum (where I'm not aware of all the up- and downsides of doing it that way). It would also keep API compatibility.
Replace the enum with the old TokenStorage trait. Breaks API, but seems most generally useful (and I remember that originally the idea was that mostly every serious consumer of this library would write their own storage).
Or, keeping more or less with the current approach: You write a somewhat generic Mac OS keychain storage backend, pack it as a crate, and we add "Mac OS keychain support" as an optional feature. The enum will gain feature-guarded variants, and so will the impl. Not super elegant, but nice for a "curated ecosystem" of storage backends, not entirely unreasonable.

Please, you and @ggriffiniii, let me know what you think.

It's been a while, but I believe the reason for going with the enum rather than the trait was because returning futures via a trait requires a heap allocation and we could avoid that with the enum. I did a quick (non-exhaustive) look at many of the public consumers of yup-oauth2 and none were using a custom trait impl so I proposed removing it until a use case arose.
Sounds like we have such a use case, and my vote is for Option 1 creating another variant of the enum for custom instances of a storage trait. Seems like the best of both worlds by providing the flexibility of a custom implementation while the built-in implementation don't pay the cost of a heap allocation on every call.

Thanks for the quick responses!
Option 3 will be difficult to adapt for my use case (the custom storage provider actually combines the MacOS keychain storage with some encryption using a key unlocked by 2-factor authentication, it's pretty specific to our use case and not suitable as a generic storage backend).
I've been playing around with a prototype of option 2, but I'll stash that and have a go at option 1 later today so that we can see how it looks.

I've got a working version of option 1 working here: https://github.com/dermesser/yup-oauth2/pull/146
Using the new interface to implement our custom storage type was pretty painless.
Let me know what you think about the approach!

I too need this!

Any movement on this?


pytorch1.3 is not support

in ./fastreid/solver/optim/adam.py

TypeError: zeros_like() received an invalid combination of arguments - got (Parameter, memory_format=torch.memory_format), but expected one of:

(Tensor input, torch.dtype dtype, torch.layout layout, torch.device device, bool pin_memory, bool requires_grad)
(Tensor input, bool requires_grad)
didn't match because some of the keywords were incorrect: memory_format

We only officially support pytorch 1.6 now.

We only officially support pytorch 1.6 now.

so install.md should modify

fixed.


[Explorador] Wrap the project with docker to ensure redundancy

[Explorador] Wrap the project with docker to ensure redundancy

Not needed yet!


change config by accepting base64-encoding with UP_CONFIG_BASE64

This is meant to address #593 and since it was trivial to implement I decided to open a PR where discussion could also take place.
In short, if up detects UP_CONFIG_BASE64 in the env, it will base64-decode the string before attempting to json-decode it. This is not especially clean, but I'm largely maintaining backwards-compatibility through this implementation.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

It'd maybe be a bit ugly implementation-wise, but maybe we should add a hacky check for { / } with util.IsJSON() in internal, and then we can just document it as base64-only while maintaining the backwards compat

@tj now this PR just looks at $UP_CONFIG using util.IsJSON() and attempts to base64-decode when it does not see a JSON value.

awesome! thanks man


feat(sbb-status): allow custom icons

Closes #2398

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@b94d92d). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2403   +/-   ##
=======================================
  Coverage        ?   93.69%
=======================================
  Files           ?      228
  Lines           ?    23484
  Branches        ?     2059
=======================================
  Hits            ?    22003
  Misses          ?     1442
  Partials        ?       39

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Issue1248 sorted tx inputs and outputs (refund bool in Output)

This is an alternative implementation of identifying a refund output within a freeze transaction.
The output struct has an additional bool field named refund which is set to true when the leftover is used to generate a refund output.
However, if a freeze transaction has only a single output the field will always be set to false as it makes no sense to have a refund only output in a freeze transaction. It would be the same as a payment transaction.

IMO it makes no sense to have a bool here. We might as well move the TxType field to the output.

IMO it makes no sense to have a bool here. We might as well move the TxType field to the output.

Yes that would be my preferred solution also. I was thinking it may be too many changes but in fact it probably is not much more than adding the bool and is a cleaner solution.

IMO it makes no sense to have a bool here. We might as well move the TxType field to the output.

Yes that would be my preferred solution also. I was thinking it may be too many changes but in fact it probably is not much more than adding the bool and is a cleaner solution.

we will have to remember to change the Output.sizeInBytes too

Decided not to use this design


test(storage): use new emulator for storage integration test

Part of #4751

This change is

@coryan Could you please kick off a CI run so I could see what happend ?

Almost tests passed so there is no error in the Dockerfile. The problem is all the integration tests passed with cached. So I could not know is there any error with the new testbench. Do you have any solution for this problem @coryan ?

To disable the test cache we need to change one of the inputs that Bazel considers.  One approach would be to change this value:
https://github.com/googleapis/google-cloud-cpp/blob/b0020c1b380dda0c94f2adc608dceac8e8f80a1a/google/cloud/storage/ci/run_integration_tests_emulator_bazel.sh#L76
The value of the environment variable does not matter, but (I think) any different value will force Bazel to re-run the tests.  Another approach would be to pick a different port for the testbench:
https://github.com/googleapis/google-cloud-cpp/blob/b0020c1b380dda0c94f2adc608dceac8e8f80a1a/google/cloud/storage/ci/run_integration_tests_emulator_bazel.sh#L56

@coryan
I think it should be good now

Everything should be good now.

FWIW, I am running some of the CI builds locally with caching disabled.

We should also retake the discussion around running the Python tests, even if only for the Bazel builds.

I have just done some experiments between pytest and Bazel. Here are the pros and cons:
Pros:

pytest: It is super easy. pip + python tests_entry.py
bazel: It could cache the tests result so we don't need to rerun the tests in every build.

Cons:

pytest: Each run takes around 7s
bazel: We will need to add a dependecy to our WORKSPACE https://github.com/bazelbuild/rules_python#getting-started

In addition, I think we could add the tests to kokoro/docker/integration.

would it be possible to pin this to a specific SHA or tag instead of a branch? Maybe @crwilcox can create a tag for us if we need something symbolic instead of a SHA?

I prefer SHA, because we will need to update the proto files regularly until the upstream proto are GA or at least stable ( they are missing some REST features now ).
What do you think ?

Up to you, we should discuss this in a separate PR / issue, so you can make progress here.

I will make a PR for the tests after finishing grpc

@vnvo2409 I could make a tag if helpful. The other thing to consider is if we could start including this surface in the client, but keep it separated from user surface. @frankyn to weigh in on that though, as it could be a while until we begin using that code in our python client library.
Also, I don't think we considered before, that it could even make sense to move the generation of that surface to alongside the emulator? Likely depends on the timeline of gRPC support in the python client.

I could make a tag if helpful

IMO, a SHA or a tag does not matter much so I think we can use SHA now.

Also, I don't think we considered before, that it could even make sense to move the generation of that surface to alongside the emulator.

I'm sorry. I don't understand what you mean here. Could you make it clearer please ? Thank you.

Sure. We currently are relying on generated code in the client library repo, on a non-maintained branch, that isn't surface used by the client library. We expect that this surface will eventually be used, but we currently haven't started work on a gRPC surface for that client. We could move the generation process alongside the emulator if this is a longer term thing.

We could move the generation process alongside the emulator if this is a longer term thing.

Thank you. It would be really useful, especially when grpc are missing some metadata properties compared with rest.


attachPaymentMethod for CustomerAdapter / CustomerSheet never fired

Describe the bug
When using CustomerSheet with customer adapter, it doesn't look like the custom attachPaymentMethod is being fired.
detachPaymentMethod fires as expected (minus a secondary issue where I'm unable to reject the detachment on UI as we have logic where we want a minimum of one card on file.
My functions are as below defined inside StripeCustomerAdapter.ts and fetchPaymentMethods and detachPaymentMethod work via the customer adapter but looks like attachPaymentMethod is not.
async attachPaymentMethod(paymentMethodId: string) {
    console.log("--> CustomerAdapter: attachPaymentMethod");
    let paymentMethod = null;

    const result = api.postAttachPaymentMethod(
      this.token,
      this.customerId,
      paymentMethodId
    );
    result.then((data) => {
      if (data.kind !== "ok") {
        Alert.alert("Error. Payment method could not be attached.");
      } else {
        paymentMethod = data.result.paymentMethod;
      }
    });
    return paymentMethod;
  }

  async detachPaymentMethod(paymentMethodId: string) {
    console.log("--> CustomerAdapter: detachPaymentMethod");
    let paymentMethod = null;
    const result = api.postDetachPaymentMethod(
      this.token,
      this.customerId,
      paymentMethodId
    );

    result.then((data) => {
      if (data.kind !== "ok") {
        Alert.alert(
          "Error. A minimum of one payment method is required to be attached to your account."
        );
      } else {
        paymentMethod = data.result.paymentMethod;
      }
    });

    return paymentMethod;
  }

To Reproduce
Steps to reproduce the behavior:

Using CustomerSheet, use CustomerAdapter as per https://github.com/stripe/stripe-react-native/blob/master/example/src/screens/ExampleCustomerAdapter.ts
Attach card and notice that your custom logic does not get fired and there is no console log of --> CustomerAdapter: attachPaymentMethod

Expected behavior
attachPaymentMethod is expected to be used.
Additional context
"@stripe/stripe-react-native": "0.33.0",
"expo": "~47.0.14",

Hi! Expo SDK 47 requires you use v0.19.0 - https://github.com/stripe/stripe-react-native/blob/master/CHANGELOG.md#0190---2022-09-16--expo-sdk-47

ah understood. thanks for the reply @charliecruzan-stripe


Undefined variable: use_pwnedpasswords

Prodblem:
Notice: Undefined variable: use_pwnedpasswords in /www/ssp/index.php on line 35
which env need to set ?

i meet this too

see PR #24

the current version does no longer have this issue - pls upgrade and eventually close this issue :-)


Improve the security of our oauth implementation

Our OAuth is very useful (we use it for Brassau, for Spotify, and potentially more projects in the future), but not quite following security best-practices.
This PR strengthens it.

Pull Request Test Coverage Report for Build 337

55 of 214 (25.7%)  changed or added relevant lines in 8 files are covered.
5 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.6%) to 53.592%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

util/user.js
7
11
63.64%

routes/user.js
12
17
70.59%

util/secret_key.js
0
6
0.0%

model/oauth2.js
1
8
12.5%

util/passport.js
4
12
33.33%

routes/my_api.js
22
68
32.35%

routes/oauth2.js
7
90
7.78%

Files with Coverage Reduction
New Missed Lines
%

routes/my_api.js
1
34.23%

routes/gassistant.js
1
15.63%

util/passport.js
1
37.11%

util/user.js
2
52.14%

Totals

Change from base Build 334:
-0.6%

Covered Lines:
4274

Relevant Lines:
7399

💛  - Coveralls


Logo is darkened on hover in dark mode

The logo in the top bar has a darkened hover state, which is hard to see:

Other top bar items have a light hover state, which is much more visible:

Yikes, that's caused by us inverting the logo color for browsers, that don't support svg color changes. Gnome Web/Safari.


fix: Ledger extraction

Describe your changes
Reverts #1510 and fixes the add_all_blocks test to use checked subtraction.
Link issue(s) fixed

#1499

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] I have verified new and existing tests pass locally with my changes.
[x] I verified whether it was necessary to increment the database version.

FYI, it would be totally acceptable to me to have submitted a74f4e4525609ce20a47c80adee77e7df60643d0 (revert) in a separate PR and stack other PRs on top of it.


Question:

Is there a android build of this engine?
If not, can you pls try to make one

there isn't and that's currently not a priority of the engine so yoshi probably won't make one

no


Switch from top level to private hive

The functionName attribute represents private inside-baseball information.  As a result, storing it at the top level of the event seems inappropraite.  This commit and its changes propose that such settings should be stored within plugin specific hives inside of which a plugin can do whatever it likes for its own consumption.  Data stored in such a location makes no guarantees to anyone about its continued existence and should not be depended upon by any other code.

Finding the characterization of this change to be confusing. What are you getting at here? Can you describe what kind of failure scenarios or conflicts you are trying to avoid?
Not really sure what you mean by "hive" here. Can you explain? Also, can you help me with your use of the term "plugin"? I'm not sure I entirely understand.
Sorry of the dealy at looking at this.

Hi @gwsii - the refactoring that was done over the last while begins preparing for a serverless-star world.  In that world, the "func" and "task" modules (i.e. plugins) comprise separate and (importantly) "non-interacting" plugins.  That is to say, they need to have their interfaces formalized and avoid "inside baseball" behaviors.  As this project has matured, we have developed a habit of slapping things in at the top level which represents future risk for contention between an increasingly complex and arbitrary codebase.  An additional risk is that plugins inappropriately start to use content from other plugins which they should not assume are being used.  To help avoid these risks, a rough namespace based separation of private configuration (i.e. hives - by which I mean a map/JSON object) for non-generic, plugin specific needs seems fit for purpose.  This is as opposed to configuration that we would expect all function or task plugins to be sensitive to (such as the parameters for how tasks are broken across function executions)
I shared the following a little back in one of our presentations but the eventual desire is that the serverless-artillery codebase will entirely comprise a package.json pulling dependencies and a code file containing:
#!/usr/bin/env node
const star = require('serverless-star')({
  func: {
    plugin: 'serverless-star-func-aws',
    config: lambdaConfig,
  },
  task: {
    plugin: 'serverless-star-task-artillery',
    config: artilleryConfig,
  },
  ...: ...,
})
star.commandLine()

proactively avoiding practices that can tie our hands in the future seems like a good idea.

Awesome! Thanks for taking the time to explain. The serverless-star context helps so much.

Thank you both!


TaskLog expects a string to be set as an output, but Events::trigger() returns bool

I have write the events with schedule to run every hour But it throw error
TypeError: Cannot assign true to property CodeIgniter\Tasks\TaskLog::$output of type ?string
public static function sent_unqueue_mail()
{
$repo     = new SQLEmail_listRepository();
$data     = $repo->getUnQueueJobs();
$totalJob = 0;
if (is_array($data) && count($data)) {
$eLib = new \Mail\Libraries\Email();
foreach ($data as $k => $v) {
try {
$eLib->send($v);
$totalJob++;
} catch (\Exception $e) {
log_message('error', "Failed to send email for job $k: " . $e->getMessage());
}
}
}
log_message('info', "Sent Unqueue Completed $totalJob");
return ["Sent Unqueue Completed"];
}
I assigned events
Events::on('sent_unqueue_mail', '\Mail\Events\EventsMail::sent_unqueue_mail');
please help me to resolve this issues

Return of event is boolean, but TaskLog only capture string|null in output. I confirm this bug
https://github.com/codeigniter4/tasks/blob/develop/src/Task.php#L250


Confirm implementation of MathUtil.transformMatrix_ is correct

Your source code looks like this:
function transformMatrix_(s, t) {
  return [
    s[0] * t[0] + s[1] * t[2],
    s[0] * t[1] + s[1] * t[3],
    t[0] * s[2] + t[2] * s[3],
    t[1] * s[2] + s[3] * t[3],
    t[0] * s[4] + t[4] + t[2] * s[5],
    t[1] * s[4] + t[3] * s[5] + t[5],
  ];
}

However, when I compute the dot product in wolfram alpha, the first matrix slot is s[0] * t[0] + s[2] * t[1] instead of s[0] * t[0] + s[1] * t[2].
Is it possible you mixed up the matrix array ordering?

Whoa. Nevermind, you were right. Somehow things magically worked :-/

The typo wouldnt have affected scaling I think... So maybe you didn't test rotations or whatever when you made the change? Who knows :p

Nah what happened was I had the order of operations reversed in the getMatrixForLayer routine, and pushed instead of unshifted child layers' transforms to the transform stack. Basically it all still worked great but the code was wrong :-) thanks for catching it.


DM-41051: Redo how disabled fileserver is handled

To make testing easier and allow enabling of the fileserver after the FastAPI application has been started, unconditionally register the fileserver routes but raise exceptions in the fileserver state manager if it's not enabled. Change DisabledError to NotConfiguredError to match the exception for the same purpose in Gafaelfawr and configure it to translate into 404 errors for users.

We never would in the running service, but an upcoming change to the test suite does exactly that to make it easier to manage how configurations and fixtures work (basically to work around the problem that you can't easily pass arguments into fixtures).


Check if react/react-dom version is incompatible with current release of next

After doing an urgent upgrade to<EMAIL_ADDRESS>I ran into a fatal error that stopped one of my apps working in production mode (but it was fine in development mode).
The error was obtuse and preventing react from running on the client, so could I not use the react debug tools, and the error was about PropTypes (and not even code I was invoking) made it hard to track down the problem.
It turns out the problem was an incompatibility between<EMAIL_ADDRESS>/<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Upgrading to<EMAIL_ADDRESS>and<EMAIL_ADDRESS>resolved the fatal error I was seeing in production.
This was a very simple fix but not an easy issue to track down due to the nature of the error. While I know what to look now I probably won't have a problem if I run into this again - but as both next.js and react are updated frequently I can imagine it will bite other people in future.
(It's not actually the first time I've run into a problem like this in the last 6 months or so due to a next+react versioning issue, but this one caught me off guard.)
I'm not sure of the best method to implement this but I'd like to suggest that next.js should check the version of react specified in package.json (or that is being included with webpack?) and raise an error and/or warning if it is earlier than a known compatible version for the the specified release of next.js.

Good point.
We can do this updating peerDeps here: https://github.com/zeit/next.js/blob/v3-beta/package.json#L130
Feel free to send a PR.

Thanks @arunoda, I've submitted and referenced a PR with react and react-dom bumped in both dependencies and peerDependencies.
(Not sure it the pragma the team is using is to create a branch for chances, couldn't find any docs on it so just did it on master as it's a simple package.json change).
The test suite is 💯 BTW, I am super impressed.

Closed by https://github.com/zeit/next.js/pull/2212


build!: update library to use Node 12

Migrate library to Node 12

@sofisl the test(10) is not running, any idea why?


relic hunter and prize mice not being recorded

Due to RH and prize mice not taking up a hunt, they are not being recorded.

RH is now being recorded only for tracker. Might want to add him to personal records too. Prize mice are still not being recorded.

rh is broken with uuid

A subset of the prize mouse journal entry (Treasurer catch):
"render_data": {
  "css_class": "entry short prizemouse catchsuccessloot catchsuccessprize",
  "mouse_type": "treasurer",
  "text": "Lucky me, a prize mouse wandered by and fell for my trap!

I caught a 3 oz. Treasurer Mouse before it could even touch my cheese!

The mouse dropped the following prize:
3 King's Credits",
  "social_link_data": {
    "slst1": "journal_ActivePrizeMouseWinWithLoot"
  }
}


Improve types and type checks

Description of changes:

AuthenticatorParams type is now exported, making it easier to typecheck params before passing them to the handle function
The return type of the handle function has been narrowed to exclude undefined and null.
Pino dependency has been updated to a typed release
Type casts have been made explicit
Strict type checking has been enabled in the source files and explicitly disabled in the test files

Nice :). This would make code I'm working on cleaner.

Hi Peter,
Thanks for your contribution. We're looking to release a new version of the package this week, so I will try to review this ASAP.
Cheers,
Jean


feat: add private keys to wallet config

closes: https://github.com/FuelLabs/fuels-rs/issues/513
The approach:

The user can now add private keys to the WalletsConfig using the with_private_keys function.
The idea is to chain the config generation with the new function if private keys are necessary.
This will not introduce breaking changes and the workflow looks like this: WalletsConfig::new(Some(num_wallets), None, None).with_private_keys(private_keys);

I have added tests and examples and updated the docs.

Will review later but I'm usually not a fan at all of using private keys "in plain text" as I suppose it's being done here. Seems like the best way to allow a user to footgun.

My 2 cents on this is that I think test wallet addresses (So anything created with a WalletConfig) should be deterministic by default.
I think we should discourage users from bringing their own private keys, especially when working on a test net. I can't think of a case where having a specific address is necessary, so much as having a deterministic one. There's only really a use case for users' own private keys when interacting with a public network, but in that case WalletConfig can't be used at all.
Happy to go with the consensus on this, but since I filed the original issue I thought I should weigh in 🙂 .

Just to be on the same page here. Until now we would create test wallets with LocalWallet::new_random() which underneath would call new_from_private_key() where the private key would be randomly generated. The user does not have to bring his private keys (I do not even what his private keys here would mean). This is just a way to get deterministic wallets if you need them. And I do not thing we need them to be deterministic for all tests.

I just mean that with this set up, when setting private_keys in   .with_private_keys(private_keys)
most of the time I would probably end up putting 0x0000000000000000000000000000000000000000000000000000000000000001 through 0x000000000000000000000000000000000000000000000000000000000000000n where n is however many wallets I need., in which case we could just do this under the hood, and then wallets would always be deterministic.
In my mind it's clear the advantages that deterministic wallet addresses bring. Are there any downsides you can see that I'm overlooking to having them by default?

I implemented the deterministic wallet generation. I have also refactored the WalletsConfig and added a missing example in the docs for custom assets.

Looks really clean, well done! ship

Thanks!


support decorator in cells

What do you think of supporting a decorator in the cell decorator?
We do this a lot in gdsfactory
from functools import partial
import gdsfactory as gf
c = partial(gf.c.mzi, decorator=gf.add_padding)
c = gf.routing.add_fiber_array(c)
c.show()

Where the cell decorator pops the decorator function and applies to the function call
but pyright does not like it :)

Shall we deprecate this behavior towards kfactory/gdsfactory compatibility?

deprecated in gdsfactory
this will make the transition to kfactory backend easier

deprecated in gdsfactory
this will make the transition to kfactory backend easier


Two off-canvas menu on same side

is that possible to create two off-canvas sidr menu on same side ?
Example : http://codyhouse.co/demo/side-team-member-bio/
Thanks

Something like this??
http://codepen.io/gorkhali/pen/bNZXwG

Thanks @alokpant


Integrate OpenAI API

Several applications and engines use the openai api to interact with LLMs.
Change from inhouse simple protocol to openai.

Taking into account OpenAI API for chat completion with stream.

Management of no-system LLMs like Mistral.


Bangor CAE

There are 2 entries in the Gazetteer for Bangor CAE. Please make Bangor City an alternative to Bangor. Please move the search records with the entry.
Geoff

Done


docs: updating pull request template to be sparse

As discussed at some point we can't remember, pulling the verbose instructions from our pull request template into a verbose doc.  Ideally new contributors can still easily figure out how to fill in our PR template, and this streamlines the PR creation process for folks who contribute on a regular basis.
Risk Level: n/a
Testing: none. Is there a way to test this or do we just land it?
Docs Changes: changes to our new PR template
Release Notes: n/a

I'd initially had the PR template attempt to link to sections in the long form doc but links don't actually show up as links in the PR creation box so I think it's more clear to just leave it as plaintext.

Oh ugh, I'd been tricked by the "preview this file" md view but of course github descriptions are plaintext so we really don't want the .
I'll send a follow-up :-/


swift_build_support: make build duration more readable

build-script invocations print a total duration summary at the end of the build, like this:
Total Duration: 4558.030000000001

With this change build duration summary is printed as
Total Duration: 487.93 seconds (8m 7s)

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS


fix recipient address mobile view

PR Checklist

Please check all that apply to this PR using "x":

[X] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[X] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

What is the current behavior?

What is the new behavior?

Other information

Thank you!


DeriveFromObject readiness doesn't work for a Job object

What happened?
When I was trying to create an Object (with DeriveFromObject readiness parameter ) that manages a Kubernetes Job it hangs in Ready: False state. Job is successfully finished and Object doesn't respect that
CC: @phisco I think you should know more since I sow that you implemented that new parameter :-)
How can we reproduce it?
Create an object like here:
apiVersion: kubernetes.crossplane.io/v1alpha1
kind: Object
metadata:
  name: test-job
spec:
  deletionPolicy: Delete
  forProvider:
    manifest:
      apiVersion: batch/v1
      kind: Job
      metadata:
        name: test-job
        namespace: apps
      spec:
        template:
          spec:
            containers:
            - name: pi
              image: perl:5.34.0
              command: ["perl",  "-Mbignum=bpi", "-wle", "print bpi(2000)"]
            restartPolicy: Never
        backoffLimit: 4
  managementPolicy: Default
  providerConfigRef:
    name: default
  readiness:
    policy: DeriveFromObject

Wait till the job will finish and notice that Object status didn't change:
...
      Status:
        Completion Time:  2023-07-12T11:17:26Z
        Conditions:
          Last Probe Time:       2023-07-12T11:17:26Z
          Last Transition Time:  2023-07-12T11:17:26Z
          Status:                True
          Type:                  Complete
        Ready:                   0
        Start Time:              2023-07-12T11:16:06Z
        Succeeded:               1
        Uncounted Terminated Pods:
  Conditions:
    Last Transition Time:  2023-07-12T11:16:33Z
    Reason:                Unavailable
    Status:                False
    Type:                  Ready
    Last Transition Time:  2023-07-12T11:16:06Z
    Reason:                ReconcileSuccess
    Status:                True
    Type:                  Synced
Events:
  Type    Reason                   Age   From                                     Message
  ----    ------                   ----  ----                                     -------
  Normal  CreatedExternalResource  10m   managed/object.kubernetes.crossplane.io  Successfully requested creation of external resource

What environment did it happen in?
Crossplane version: upbound/crossplane:v1.12.1-up.2
Provider kubernetes: xpkg.upbound.io/crossplane-contrib/provider-kubernetes:v0.9.0
Kubernetes distribution: EKS

DeriveFromObject is going to derive the status from the status of the object, but only if the underlying resource has an element in status.conditions with type: Ready. Which is not the case for jobs as you can see, they only have type: Completed. With the current implementation there is no way to pick the condition to match, however I don't think in this case it would make sense to mark the resource as ready once it's Completed only, so I'd suggest you to switch to the default SuccessfulCreate readiness policy

DeriveFromObject is going to derive the status from the status of the object, but only if the underlying resource has an element in status.conditions with type: Ready. Which is not the case for jobs as you can see, they only have type: Completed. With the current implementation there is no way to pick the condition to match, however I don't think in this case it would make sense to mark the resource as ready once it's Completed only, so I'd suggest you to switch to the default SuccessfulCreate readiness policy

Hm, why do you think that I don't think in this case it would make sense to mark the resource as ready once it's Completed only?

I see your point - we would have to stream somehow type: Completed into Object

I have the same problem for deployments - would it make sense with another AllStatusTrue policy that simply checks that all conditions have status: True?


Starting package to handle SpringBoot integration.

Description
Starts SpringBoot integration work + migrates Actor example to use SpringBoot.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #30 #31
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

I'm confused about the term "integration": this is for the SDK using SpringBoot internally to get callbacks from Dapr, no?

Or is this about calling back into the user's app using SpringBoot?

Or is this about calling back into the user's app using SpringBoot?

This is about using SpringBoot to get calls from Dapr, using a SpringBoot aplication provided by dapr sdk.

It's not a biggie, but I wouldn't necciseraly use the term integration as it generally denotes some sort of feature wise extensions between two systems, where here it's Dapr using Spring Boot for it's server purpose.
Wdyt?

It's not a biggie, but I wouldn't necciseraly use the term integration as it generally denotes some sort of feature wise extensions between two systems, where here it's Dapr using Spring Boot for it's server purpose.
Wdyt?

OK, the term "integration" can create ambiguity.


Microsoft To-Do

Hi,
Does Microsoft To-Do fully support conditional access or Intune app policies? I've my policy set to allow any app but To-Do is always blocked with "Application used is not an approved application for conditional access. User needs to use one of the apps from the list of approved applications to use in order to get access."

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

ID: 0073f5a8-daf6-7f14-127f-e0c9d80af78d
Version Independent ID: d6d0afdf-3619-ca03-718b-a40bfd07bab9
Content: Azure Active Directory conditional access settings reference
Content Source: articles/active-directory/conditional-access/technical-reference.md
Service: active-directory
GitHub Login: @MarkusVi
Microsoft Alias: markvi

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

@DaithiG Yes, as per the list of client applications mentioned in Approved client app requirement, Microsoft To-Do is mentioned. In case you are facing issue, please open a Azure Support Case in this regard so that Identity Support Engineer can further check on it and help with resolution.

Thanks. Had raised a call with Azure support but they keep saying it's an issue with the app itself which I don't get. It's fine. Thanks for your help.


Add Laravel Command to rebuild an index

I'll add to my project an artisan command which will drop and rebuild my index on my embeddings table. Which will be triggered when new embeddings will be added to the table.
If interested I can make a generic version to be added to this package.
At which point I would start wondering if it wouldn't make sense to have two separate packages, one for native PHP and one for Laravel specifically.

Thanks for another suggestion. Indexes shouldn't need to be rebuilt when rows are added/updated/deleted in most cases, so I'd prefer to hold off on adding this for now (and if lists needs to be adjusted, it should happen through a migration).
Re separate packages: I'd prefer to keep it as one for now, unless it somehow limits what can be added.

Perhaps we can create a job or similar which can be called whenever necessary. I also had some issues with the index (https://github.com/pgvector/pgvector/issues/109) and @ankane advice is clear and worked: Use the right values for lists and probes.
My current solution is a job which runs after every batch of inserting new data. It drops the index and recreates it based on the suggested variable values (https://github.com/pgvector/pgvector#indexing).
I would open a pull request with my suggested solution if you agree.

Hi @kainiklas, recreating the index after inserts isn't needed in most cases, so it's not something I'd like to include in the library, but someone could always create a separate package for it.


Failure to deserialize externally tagged enums from dotted keys

Trying to deserialize a dotted key as an enum variant returns an error. For example, deserialization of pet.Kitty = {} fails, and pet = { Kitty = {} }(or a different enum form) has to be used instead. This is rather inconvenient when dealing with nested enums, e.g.:
[axes]
move_vertical.Emulated = { pos.Key = "W", neg.Key = "S" }
move_horizontal.Emulated = { pos.Key = "D", neg.Key = "A" }

has to be replaced with:
[axes]
move_vertical = { Emulated = { pos = { Key = "W" }, neg = { Key = "S" } } }
move_horizontal = { Emulated = { pos = { Key = "D" }, neg = { Key = "A" } } }

A minimal test case can look like this:
#[derive(Debug, serde::Deserialize)]
enum Cute {
    Kitty,
    Hamster,
}
#[derive(Debug, serde::Deserialize)]
struct Room {
    pet: Cute,
}

fn main() {
    let parsed_1 = toml::from_str::<Room>("pet = \"Kitty\"");
    let parsed_2 = toml::from_str::<Room>("pet = { Kitty = {} }");
    let parsed_3 = toml::from_str::<Room>("[pet.Kitty]");
    let parsed_4 = toml::from_str::<Room>("pet.Kitty = {}");
    let _ = dbg!(parsed_1);
    let _ = dbg!(parsed_2);
    let _ = dbg!(parsed_3); // does work on master
    let _ = dbg!(parsed_4); // does not work
}

Maintenance of this crate has moved to the https://github.com/toml-rs/toml repo.  As a heads up, we plan to move toml to be on top of toml_edit, see https://github.com/toml-rs/toml/issues/340.
Closing this out.  If this is still a problem, feel free to recreate this issue in the new repo.


Add Producer Scheduler Version to Block Header & Validate It #1671

You are right, I referenced the wrong field...


Draco WASM does not load when used in WebViews like Electron apps

Create an electron app as follows here: https://www.electronforge.io/

On command line, run yarn create electron-app my-app
cd my-app
In file explorer, copy the contents of this zip into the src folder of my-app (overwrite index.html)
draco-example.zip

eg

yarn start

App will fail to load Draco with the following error:

Fixed


Crash: Fatal Exception: java.lang.RuntimeException Illegal callback invocation from native module. This callback type only permits a single invocation from native code.

http://crashes.to/s/83fd4bcf5e6
We're seeing this relatively frequent crash with beta-3.
Is there anything we could do about it?
I'm not sure if we could resolve this on the JS side.

It’s the promise for load that is rejected twice, but should be easy to fix.
Problematic part: https://github.com/sbugert/react-native-admob/blob/master/android/src/main/java/com/sbugert/rnadmob/RNAdMobInterstitialAdModule.java#L84

@koenpunt : do you have any instructions how we could fix this? I'm not a native Android developer, but we have someone on the team who could get on this and submit a PR soon.


Update Prometheus to stable 2.2.1

• bump prometheus to 2.2.1
• prometheus chart set to 0.0.21
• kube-prometheus chart set to 0.0.31

For helm, you will need to update prometheus Chart version too and its entry on the requirements.yaml file in kube-prometheus

Should we wait to merge this before this issue be enlightened ?
https://github.com/prometheus/prometheus/issues/3943

@gianrubio agreed we should hold off until we get more details on this.
While I have not been able to reproduce it, it looks like it's affecting a few people.

I think we should just move forward, it's unclear whether the problem was already present for the rc.1 which we currently have. At this point I prefer having lots of people report to report and describe their problem than keeping the release candidate. Sadly the 2.1 release was also broken in some ways, we're actively trying to improve the situation upstream. Sorry for the trouble.

Agreed @brancz, thanks for clarifying. The CI is failing but the helm changes are good for me!!

can you just run make generate to regenerate the auto generated contents? then this lgtm

@brancz sure thing

two things:

2.2.1 was just released (builds still running) but we should probably just bump that once built
there is a problem with the helm build (I think the version needs to be bumped, but @gianrubio knows best)

rebasing this branch + following the guide should be enough :)

Lgtm, thanks!

Tests passing locally with the new charts and prometheus 2.2.1.

thanks!


FR: Multiple windows

I would like to edit 2 or more files side by side at the same time in tile wm. It seems snaked does not support multiple windows in one process. Although someone could open another window in a separate process, but the session is not shared so it is a bit pain to get everything in sync.
Also it would be nice to reload the file content when a editing file has been modified externally.

Implemented. v0.5dev has the ability to open multiple windows with the same buffers. Root menu -> Tab -> Detach.


Keyframes doesn't use config with TimingAnimation and Konva

Hello,
I want to have an animated dashed line with Konva and React-Spring but the config is not used.
Here is an example of my problem : https://codesandbox.io/s/3y9p9pr83p
The first part of the animation should be slower and linear but it isn't.
Thanks,
Julien Usson

Very, very strange indeed. It's like as if it doesn't even know duration. But for some reason here it works: https://codesandbox.io/s/x7xrp39zp
I'll look into it

should be fixed now


Add range end to /remove

Rationale
Sometimes you want to remove an entire range of songs from the queue.
Description
Add an optional positive integer range end parameter. When set, instead of removing a single track from the queue, remove all tracks in the given range.

How should this be displayed to the user?
Right now we have this:

The index is currently the only parameter and is required (obviously), should we simply add an optional until (other name suggestions accepted) parameter to ask for the end of the range?

An optional parameter sounds good! May I suggest renaming index to start and introducing end?

@aquelemiguel this is the same problem as with the skip optional. I sort of disagree with renaming index to start, because if you are simply removing one of the tracks it is not the start of anything. The range removal is a feature that only exists if the until optional is sent.

I don't see "start" as necessarily needing an "end" to make sense, but I see your concern. "until" is fine as well. 🙂

I like index and until, mainly for the reasons @afonsojramos has described. Personally, I don't even think that the index tooltip needs to be changed, unless you want to do some "Position of the (starting) track (...)" shenanigans. I'd rather have the until tooltip be more descriptive:

If set, remove a range of tracks from index until the one set here instead.

Add a subsequent track position to remove a range of tracks.
If set, remove a range of tracks from index until the one set here instead.

I also considered a variation of 2 at the time, but from these two, I chose 1 because if until is an optional parameter it is already implicit that it only applies if set.


Unclear Instruction Initial Configuration/ Reverse Zone - Leading to no IP - ReverseZone mappings

Versions
NetBox Version: NetBox Community v4.1.7
NetBox DNS Version: 1.1.6
Python Version: 3.12.5
Unclear Documentation on how to configure reverse zones.
After following the documentation, it is unclear how to correctly configure a reverse zone within the plugin. FQDN/DNS Names from IPAM Records do not get converted to reverse zone entries when running rebuild_dnssync.
To Reproduce

Setup Netbox
Add a prefix (ie: <IP_ADDRESS>/22) set as reserved
Add a child-prefix (ie: <IP_ADDRESS>/24) set as active
Add a child-child-prefix (ie: <IP_ADDRESS>/29) set as active.
Add an IPv4 Address <IP_ADDRESS> within the /29.
Add a DNS/Hostname in the IP-Address edit section. (ie: testrouter.example.com)
Install the netbox dns plugin
edit the prefix <IP_ADDRESS>/24 add it to the default view.
create a nameservers (ns1.example.com / ns2.example.com)
create a reverse zone for the the child-prefix ie: 0.0.10.in-addr.arpa as part of the default view.
run rebuild_dnssync
check reverse zone: no entry
debug step 1: edit ip-address manually change hostname to  testrouter99.example.com
check reverse zone: no entry
run rebuild_dnssync
check reverse zone: no entry

Expected result
Reading the instructions suggests that rebuild_dnssync fetches the fqdn/hostname from the ip-address entry in netbox. then creates an entry in the reverse zone. This does not appear to be the case.
Actual result
Nothing happens.
Updating DNS records for IP Address <IP_ADDRESS>/24, VRF None
  Zones: []

additional notes
Adding the "optional fields" as suggested in the docs does not improve this.

@ObjectiveWriter thanks for raising this.
The IPAM DNSsync feature does not directly create PTR records at all, it creates address (A/AAAA) records in forward zones. If a matching reverse zone is found, the PTR records are generated accordingly.
I will update the documentation to clarify that.

In your example you'll just need to add the zone example.com in the default view.

This would be "unexpected" behavior. Because this means it cannot be used to automate reverse zone creation, if we do not also at the same time carry a forward zone.
Meaning:
A tenant gets a reverse dns/fqdn entry for an ip-address associated with their service. The corresponding "forward" entry is in their own external DNS zone. (not mapped in netbox) as the netbox operator has no control over the zone. (Or any small to medium sized DC, would have 100's of "client zones" they have no control over.)
In this case (wich i suspect will be quite common) it becomes impossible to use the dns plugin to manage PTR records.
Solution:

Crate A record in Forward zone if FQDN Exists
Create PTR in Reverse zone if FQDN Exists

Instead of going only crate a PTR if there is a matching A.

It is not unexpected as it is documented exactly that way.
I see your use case, and I think it would be a possible addition to enable direct PTR creation from IPAM DNSsync, but it's just not the way it's implemented today. I fully agree that it's possible and that it makes sense, but it's definitely not a bug but an FR.

By the way, doing this is by no means trivial ... there are several situations that need to be considered:

A new forward zone gets created that matches the DNS name of an IPAM IP address
The reverse zone gets renamed and no longer matches the IPAM IP address
The reverse zone gets renamed and still matches the IPAM IP address, but with a longer/shorter prefix
A new reverse zone with a longer prefix is created and part of the PTR records need to migrate to it (and vice verse when it's deleted)
etc.pp.

All these cases need to be handled correctly (they are with forward zones in the play, but all of this functionality would need to be recreated for "pure" reverse zones, and tests need to be written for all these cases. So my estimate for the effort of implementing this is between two and four weeks. Of course, PRs are welcome :-)
The documentation update will close this issue, but as it's an interesting point I will convert it to a discussion so we can collect some other users' opinions and ideas as well.


trying to download windows fusion helper (1.1.3) says its a virus

When i go to download the latest one it says a virus idk what to do i wanna play fusion but i dont trust it because when i also checked my windows security it says its a trojan.

It's a false positive


https request in Android 4.4

I am calling https api calls and works on Android 5 series but fails under Android 4.4. I tried acceptAllCerts method and even this fails. Have anybody gone thru the same issue?

Now I tried with signed request, SSL pinning even now its not working.

This issue is related to #58 ,


Draw overlay of human mesh and original picture

Hi, is there code to draw human mesh on top of the original rgb picture as displayed in the paper?

Hi, it’s in the demo code

Hi,
did you also comment this line?
img = np.zeros((img_res, img_res, 3))  # dummy

yes

Hi,
you should change this line, toimg_shape = 255 * renderer(pred_vertices, camera_translation, img / 255.)
I have plan to update some codes, and I will update the demo code also.
Thank you for noting me!

Thanks, that works.


Diagram view - Make it nice looking for demos

When we do talks at conferences then the projector is usually 1024 or 1280 screen resolution.
Then the diagram overview get squashed a bit on the left hand side and you cannot see the arrows properly. This can be done by just resizing the window to be smaller.
Maybe that services box can be smaller by default. And maybe the port number can be moved to the top left corner, and the service name below. Also why do they all have 'default-as prefix in the services? If you open the services tab, then the id column do not include that. I guess its the namespace prefix, but then there is already a drop down with the selected namespace, and if so then we can drop thedefault-` prefix so the name is shorter.

+1, agreed...

Oh also in general one thing you can get away with sometimes when using a projector is to back off the zoom to like 80% or so.  That way folks get a decent view of the page but it's still generally legible also.

Ah yeah for the diagram you can maybe zoom out. But I tend to zoom in even with low resolution, as eg some of the table pages are harder for the audience to read.
Maybe we need a "presentation mode" styling ;) I love that IDEA for instance has that.


Duplicate field error on Interaction when the entitlements field is present

Using the latest git version.
This seems to trigger if the field is present in the payload, the cause of the error is this line: https://github.com/twilight-rs/twilight/blob/main/twilight-model/src/application/interaction/mod.rs#L272

I made a patch that should resolve this, if you have some time it would be nice if you could test it.

Seems to work, was gonna open a pr myself but forgot to actually open the pr after i made the fix heh...


resolved conflict with RNNCell property name

Thanksa lot! This should also lift the tensorflow requirements I presume. I have some questions before merging.
With what Tensorflow have you tested this? And have you been able verify that training works?
I will run some experiments after merging on my end to ensure that everything is still reproducible.

No problem! I've tested this on Tensorflow version 1.10.0 and the training worked fine :)


 If i disconnect right after connecting, i'm getting two floating window, which i want to avoid. How to hide the first one? I think it's in the "public func show", but all my changes doesn't change anything... I'm new to swift... does the "timer.invalidate()" supposed to do this thing? I have also replaced the "return" in "if self.isVisible { return }" to "self.hide()", with no luck.
public func show(on screen: NSScreen = NSScreen.screens[0], for time: Double = 0) {
        if self.isVisible { return } //  self.isVisible = false or self.hide() doesn't change anything...
        timer?.invalidate()  // <-- does it supposed to do this thing?

        self.initializeWindow(screen: screen)

        DispatchQueue.main.async {
            withAnimation(self.animation) {
                self.isVisible = true
            }
        }

        if time != 0 {
            DispatchQueue.main.asyncAfter(deadline: .now() + time) {
                self.hide()
            }
        }
    }

Help please!
Thanks.

Actually, you don't need to change anything in DynamicNotch, and should already work.
Note that you should declare DynamicNotch as an instance variable, instead of a local one, which is likely what is causing your problem.
If you need more help, please send your code (or fragments of it) :)

As i said, i'm new to swift.
Here's how i'm calling it:
func checkOutputDevice() {
		if let outputDevice = simplyCA.defaultOutputDevice {
			if outputDevice.name.contains("AirPods") {
				VolumeWindow.isAirPodsConnected = true
			} else {
				VolumeWindow.isAirPodsConnected = false
			}
			let notch = DynamicNotchInfo(
				icon: ap,
				title: "Airpods",
				description: VolumeWindow.isAirPodsConnected ? "Connectés" : "Déconnectés",
				style: .floating
			)
			notch.show(for: 5)
			if VolumeWindow.externalDisplay {
				let notch = DynamicNotchInfo(
					icon: ap,
					title: "Airpods",
					description: VolumeWindow.isAirPodsConnected ? "Connectés" : "Déconnectés",
					style: .floating
				)
				notch.show(on: NSScreen.screens[1], for: 5)
			}
		}
	}

What i'm doing wrong?

so to make DynamicNotchInfo an instance variable, you would need to do something like:
...class declaration

var notch: DynamicNotchInfo?

func checkOutputDevice() {
	if let outputDevice = simplyCA.defaultOutputDevice {
		if outputDevice.name.contains("AirPods") {
			VolumeWindow.isAirPodsConnected = true
		} else {
			VolumeWindow.isAirPodsConnected = false
		}

		notch = DynamicNotchInfo(
			icon: app,
			title: "Airpods",
			description: VolumeWindow.isAirPodsConnected ? "Connectés" : "Déconnectés",
			style: .floating
		)

		notch?.show(for: 5)

		if VolumeWindow.externalDisplay {
			notch = DynamicNotchInfo(
				icon: ap,
				title: "Airpods",
				description: VolumeWindow.isAirPodsConnected ? "Connectés" : "Déconnectés",
				style: .floating
			)

			notch?.show(on: NSScreen.screens[1], for: 5)
		}
	}
	}
}

...continue code

There may be some mistakes here, but here's the rough structure your code should have 😉

Sorry, it was late (5am) yesterday...
class MyDynamicNotchInfo {
	let icon: Image
	let title: String
	let titleColor: Color
	let description: String
	let style: DynamicNotch.Style

	init(icon: Image, title: String, titleColor: Color, description: String, style: DynamicNotch.Style) {
		self.icon = icon
		self.title = title
		self.titleColor = titleColor
		self.description = description
		self.style = style
	}
}

struct VolumeView: View {
	var notch = MyDynamicNotchInfo?.self    //    if not .self, "Expected member name or constructor call after type name"

and i'm getting:

No problem!
It should look like this:
struct VolumeView: View {
	@State var notch: MyDynamicNotchInfo?

The @State is needed in SwiftUI views, and you should use a colon when assigning a type rather than an actual value :)

My bad!!! Thanks for the colon trick!! Sorry for that!
Still getting Value of type 'MyDynamicNotchInfo' has no member 'show' in my checkOutputDevice() method. How to call show() from my class?
import SwiftUI
import DynamicNotchKit

class MyDynamicNotchInfo {
	let icon: Image
	let title: String
	let titleColor: Color
	let description: String
	let style: DynamicNotch.Style

	init(icon: Image, title: String, titleColor: Color = .white, description: String, style: DynamicNotch.Style) {
		self.icon = icon
		self.title = title
		self.titleColor = titleColor
		self.description = description
		self.style = style
	}
}

struct VolumeView: View {
	@State var notch: MyDynamicNotchInfo?

        var body: some View {...}

        func checkOutputDevice() {
		notch = MyDynamicNotchInfo(
			icon: ap,
			title: "Airpods",
			description: VolumeWindow.isAirPodsConnected ? "Connectés" : "Déconnectés",
			style: .floating
		)
		notch?.show(for: 5)
		if VolumeWindow.externalDisplay {
			notch = MyDynamicNotchInfo(
				icon: ap,
				title: "Airpods",
				description: VolumeWindow.isAirPodsConnected ? "Connectés" : "Déconnectés",
				style: .floating
			)
			notch?.show(on: NSScreen.screens[1], for: 5)
		}
	}
}

Can you confirm i'm not doing something wrong here?
Many thanks for all your work and your help!!

This might not be perfect as for your use case (so please edit it to your needs), but something like this will work better :)
import SwiftUI
import DynamicNotchKit

struct VolumeView: View {
	@State var notch: DynamicNotchInfo? // I am using DynamicNotchInfo instead of MyDynamicNotchInfo here

    var body: some View {...}

    func checkOutputDevice() {
		notch = DynamicNotchInfo(
			icon: ap,
			title: "Airpods",
			description: VolumeWindow.isAirPodsConnected ? "Connectés" : "Déconnectés",
			style: .floating
		)
		notch?.show(for: 5)

		if VolumeWindow.externalDisplay {
			notch = DynamicNotchInfo(
				icon: ap,
				title: "Airpods",
				description: VolumeWindow.isAirPodsConnected ? "Connectés" : "Déconnectés",
				style: .floating
			)
			notch?.show(on: NSScreen.screens[1], for: 5)
		}
	}
}

Since your MyDynamicNotchInfo doesn't have a show() method, it wasn't working. I assume you meant to use DynamicNotchInfo instead, so I used that in my example above. If you did intend to use MyDynamicNotchInfo, you would need to implement a show() method there.

It was a mistake yes... sorry.
The code you gave me is the code i tried first, but it doesn't work, it still display 2 floating windows, one above the other... don't know why but... No matter, i don't want to take you all your time for my little problem...
Thanks for all!!

It's alright!
I also just realized that I forgot to add the hiding part, try this:
import SwiftUI
import DynamicNotchKit

struct VolumeView: View {
	@State var notch: DynamicNotchInfo? // I am using DynamicNotchInfo instead of MyDynamicNotchInfo here

    var body: some View {...}

    func checkOutputDevice() {
		notch?.hide() // hide :)
		notch = DynamicNotchInfo(
			icon: ap,
			title: "Airpods",
			description: VolumeWindow.isAirPodsConnected ? "Connectés" : "Déconnectés",
			style: .floating
		)
		notch?.show(for: 5)

		if VolumeWindow.externalDisplay {
 			notch?.hide() // hide :)
			notch = DynamicNotchInfo(
				icon: ap,
				title: "Airpods",
				description: VolumeWindow.isAirPodsConnected ? "Connectés" : "Déconnectés",
				style: .floating
			)
			notch?.show(on: NSScreen.screens[1], for: 5)
		}
	}
}

Another option would be to use the setContent() method instead of initializing a new DynamicNotchInfo, which could potentially be seen as more efficient, and might be the way to go.

Wow!! Tricky!! But no luck with that too...
Tell me more about the setContent() method?

It is documented here.
This way, instead of redoing notch = DynamicNotchInfo(...) every time it's triggered, you could simply intitialize it once, and every time you need to show the popup, you can simply do: notch.setContent(...), then show it.

for some reason, i couldn't get it to work... it doesn't even display!!
func checkOutputDevice() {
	notch?.setContent(icon: ap, title: "Airpods", description: VolumeWindow.isAirPodsConnected ? "Connectés" : "Déconnectés")
	if VolumeWindow.externalDisplay {
		notch?.setContent(icon: ap, title: "Airpods", description: VolumeWindow.isAirPodsConnected ? "Connectés" : "Déconnectés")
	}
}

Is this good?

First, you will need to initiate the DynamicNotch, since currently, you are trying to call a function on a nil value.
Also, you will still need to have the show() method :)

I'm still using @State var notch: DynamicNotchInfo?...

Sorry, i couldn't get it to work. i'm thinking to leave it like this, it's gonna be less beautiful, but no matter. You're not gonna create my app for me... So, many thanks for what you've done for now and for this package!!
Today it look like this and it's good enough:

Again, many thanks for all!

I got it to work!!! The problem come from the fact that i display it on external screen...
If i delete the if VolumeWindow.externalDisplay { statement, it work on main screen! if i keep the if statement, it work on the external display but not on the main... Normal behaviour?

Sorry for the late answer!

Today it look like this and it's good enough:

This looks absolutely amazing! If you ever end up publishing the app - let me know, I would be a proud user :)

If i delete the if VolumeWindow.externalDisplay { statement, it work on main screen!

That's interesting... I'm guessing that this is MacOS doing something weird, but I can't be completely sure without looking at the full code.

This looks absolutely amazing! If you ever end up publishing the app - let me know, I would be a proud user :)

I'm sure you can do better than me!! I've just modified part of your work... It's an app than show battery level around the (real) notch on notched MacBooks, and change the volume HUD which is ugly in my opinion... just my opinion! Different volume HUD styles, and positions can be set, inspired by MediaMate app, but without media player.

That's interesting... I'm guessing that this is MacOS doing something weird, but I can't be completely sure without looking at the full code.

It can comes from my code, i'm not a dev, i just started xcode about 6 month ago just to create this notched battery level app, and added some features from time to time, getting this entire app at last, with the help of AI and maaany internet researches... Still doesn't know swiftui, i'm just "swimming"... this is why i'm not planning to release it, as it's not well written...  But i'm pretty sure anyone can do better than this!!

 Take care!!
PS: sorry for my poor english, french here...

No problem!

Anyways, good luck on your app!

Je connais un peu le français aussi :)

Quebec? Tu a appris où le français, à l'école?

Je connais un peu le français aussi :)

Quebec? Tu a appris où le français, à l'école?

Oui! Je vais à une école d'immersion française (pas en Quebec) 😁

Super!! Si tu passe un jour en France, fais moi signe!!
Bon courage et bonne continuation!!
Salut l'ami!!


📃: FlexFLow (Fitness Website ) in Frontend project

:red_circle: Title : FLEXFLOW (FITNESS WEBSITE)
:red_circle: Aim : revolves around providing resources, information, and tools to help individuals improve their health and fitness levels.
:red_circle: Tech Stack Used : HTML,CSS
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name : Khushi Pushkar
What is your participant role? (Mention the Open Source Program name. Eg. GSSOC, SSOC, JWOC, etc.)
GSSOC'24

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@Khushi-Pushkar go ahead.


Get and remove Hunting Rules broken

Environment
Windows build number: Version 10.0.19042.487
PowerShell version (if applicable): 5.1.19041.1
AzSentinel version: 0.6.5 (latest)

Steps to reproduce
Call Get-AzSentinelHuntingRule or Remove-AzSentinelHuntingRule  with -RuleName set to the display name of an existing hunting rule.
Expected behavior
Get should return filtered hunting rules, and remove should remove the filtered hunting rules.
Actual behavior
After debugging the underlying API requests, it seems like Azure is returning a different response than AzSentinel is expecting. It looks like the API version hardcoded in AzSentinel has no effect, and Azure is returning the newer/latest response structure regardless (assuming the hunting rules functions worked in the past).
Specifically, I think AzSentinel should be using the DisplayName property in the response rather than name for hunting rules. name contains a guid now, while DisplayName is the actual name that should be filtered on.

hi @darvid thanks for your great detailed feedback! I have added the -Filter option that you can use to only get "Custom Queries" for example. Also fixed the response from the API and updated the remove function. proposed fixes will be released in next version! Please let me now if you are still missing feature. Kr Pouyan


partly solve recv data interrupt on http_parse

        AsWeKnow header_line is \r\n(end flag) | ..*\r\n..*(header_content)

        assert queue & next_queue is header_line
        // TODO: bound condition would cause error
        discuss:
        have known queue, don't know next_queue.
        if queue.empty == false:
            assert queue is \r\n | ..*[!\r\n] | ..*\r\n.*
            exec line = queue.pop(delimiter=\r\n)
            if queue.empty:
                assert queue is \r\n | ..*[!\r\n]
                assert line is EMPTY | ..*
                if line.empty:
                    assert line is end_flag.
                else:
                    push ..*
            else: // queue not empty
                assert queue is ..*\r\n.*
                assert line is ..*
                assert line is header_content.

CI没通过，可以看一下这个改一下https://github.com/0x404/cgi-server/actions/runs/3164537000/jobs/5153553905


Lab 6 - Perhaps add top or htop to differentiate that the criteria is mainly about if the student can use stress-ng

https://github.com/HOGENT-MLOps/mlops-labs/blob/76cb66e62ad0be4c37a79277d23bb71a47e43643/assignment/06-monitoring.md?plain=1#L80
Testing that Grafana picks it up comes in a later criteria, but at the moment the difference between these two isn't clear.

I'll add something


migrate to jest projects

jest has a feature to support monorepos called projects. i've setup the configuration in a way that we can run jest from the root to run all tests, run jest from the root for a single project or run the test script in an individual package to just run the package's tests. examples:
from the project root we can do
jest ->  runs all the tests
jest --project package/avatar -> runs just the avatar tests
jest --project package/avatar package/badge -> runs avatar and badge tests
from the avatar package
npm test  -> runs just the avatar tests
Perceived Benefits

one global configuration to maintain
each package can still update their own config
polyfills and enzyme setup are in one place
can add custom jest matchers/plugins in one place. i'm planning to add jest-axe at some point
i plan to add the enzyme snapshot serializer to get nice jest snapshots of react elements
faster installs - jest(and related) are installed in the root
faster runs(local and in ci)
project wide coverage reports
if everything is configured in a similar way there is no context switching between packages. ie...i know i'll have this matcher or that serializer installed in every test

Possible Downsides

everything uses the same versions and configs. this could make upgrading more painful.

Outstanding tasks:

[x] update the remaining packages to use the shared config
[x] update the component:create script with new config
[x] add jest typeahead for easy searching in the cli gui
[x] upgrade root npm test to use single jest instance instead of lerna run

Questions that come to mind:

I'd like to know how these two benefits co-exist: "one global configuration to maintain", "each package can still update their own config"
Can a dev come in and dev a single component without knowing/caring about the other unrelated packages?
Can we have a totally different one-off test runner in the future for a single package?

this shaved ~2min off the build. not as much as I hope. bootstrapping is still the longest part

oops, I merged.  old habits... hopefully we were ready for that.  :) anyway, pr approved!


WARNING: Could not get response from query

Steps to reproduce

Expected behavior
Hello, while publishing Publish-AWSPowerShellLambda, I get the following issue,
Please help how to solve it,
Thank you
Actual behavior
Copying PowerShell script to staging directory
WARNING: Could not get response from query 'https://www.powershellgallery.com/api/v2/FindPackagesById()?id='AWSPowerShell.NetCore'&$skip=0&$top=40'.
PackageManagement\Save-Package : No match was found for the specified search criteria and module name 'AWSPowerShell.NetCore'. Try Get-PSRepository to see all available registered module repositories.
At C:\program files\powershell\6\Modules\PowerShellGet\PSModule.psm1:11493 char:21
+$null = PackageManagement\Save-Package @PSBoundParameters

CategoryInfo          : ObjectNotFound: (Microsoft.PowerShel\u2026Cmdlets.SavePackage:SavePackage) [Save-Package], Exception
FullyQualifiedErrorId : NoMatchFoundForCriteria,Microsoft.PowerShell.PackageManagement.Cmdlets.SavePackage

Environment data

Steps to reproduce
Expected behavior
Hello, while publishing Publish-AWSPowerShellLambda, I get the following issue,
Please help how to solve it,
Thank you
Actual behavior
Copying PowerShell script to staging directory
WARNING: Could not get response from query 'https://www.powershellgallery.com/api/v2/FindPackagesById()?id='AWSPowerShell.NetCore'&$skip=0&$top=40'.
PackageManagement\Save-Package : No match was found for the specified search criteria and module name 'AWSPowerShell.NetCore'. Try Get-PSRepository to see all available registered module repositories.
At C:\program files\powershell\6\Modules\PowerShellGet\PSModule.psm1:11493 char:21
+$null = PackageManagement\Save-Package @PSBoundParameters

CategoryInfo          : ObjectNotFound: (Microsoft.PowerShel\u2026Cmdlets.SavePackage:SavePackage) [Save-Package], Exception
FullyQualifiedErrorId : NoMatchFoundForCriteria,Microsoft.PowerShell.PackageManagement.Cmdlets.SavePackage

Environment data


Replays have commit SHAs that do not reflect anything on the public server

https://github.com/Zarel/Pokemon-Showdown/blob/6c1ab309774e3ae1111abea114dc63a1cd5bc49f/server/room-battle.js#L1080-L1087
Is this what is being written into replay logs on the first line?

Yeah, there was an unpushed commit on the server. I've pushed it now: https://github.com/Zarel/Pokemon-Showdown/commit/12f49b20e16c0fab0cd1a1ca9ac1a5a1cc8d7e82

I think it might be more than that. Here's a few examples:

https://replay.pokemonshowdown.com/gen7randombattle-887606429.json references 664768c238d88325eea8f848b9726dcf8abd958e which doesn't exist in the public repo
https://replay.pokemonshowdown.com/gen7randombattle-879384978.json references dab6083d84cf10c82638ffd4b09ad28aa8458e44 which also doesn't exist in the public repo

Is the master branch being overwritten locally on the deploy machine(s)?

664768c238d8 is actually ac5c0a5760cff588b9c25 in the public repo
dab6083d84cf10c is actually 9315b8d4d83df3 in the public repo
They're named differently because 12f49b2 was being constantly rebased on top of them.

Oof, okay. I'm going to go back to using guess and check, then.

Out of curiosity, what are you using it for?

Testing two ideas:

Imitation learning
Computing a measure of how "human-like" my current agent is

On Fri, Apr 5, 2019 at 10:25 AM Kirk Scheibelhut<EMAIL_ADDRESS>wrote:

Out of curiosity, what are you using it for?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Zarel/Pokemon-Showdown/issues/5412#issuecomment-480356354,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVbL9yc_4Usay7rIhRyLj5PC_QmfDHxks5vd4b9gaJpZM4cfOPr
.

Looks like it'd be better to run git merge-base origin/master HEAD ?

@Slayer95 agreed. I'll submit a PR.

@Slayer95 I've thought about this a bit more. If rebasing on the deploy-local master branch is going to be a common occurrence, I'd rather the system generate a confusing SHA than provide a SHA that is potentially wrong (and won't reconstruct properly).
@Zarel thoughts?

Local edits to the battle engine shouldn't really happen -most should be to the server infrastructure AFAICT.
Even if they do happen, I think it would be better to provide the version in a best-effort basis. In fact, in the specific circumstance above, the hash returned would have been totally trustworthy. Note that you should have logic in place to handle failures in the reconstruction process from the input log (never trust remote server data).
If the proposed change is done, you could iteratively check nearby commits until the battle reconstructs properly (ensuring the winner matches should be enough), and fix the commit hash in your own database.

Yeah that's a good point. Even with a hash origin/master that doesn't reconstruct properly, the incorrect hash is a good starting point for a search through the commit history.
Okay, this change sounds good to me.

In general, the main server should never have unpushed commits. This latest one was just something I accidentally forgot to push.


[Bug] N12 view - הכל text is in white rather than black

Describe the bug
N12 view - text of "הכל" is in white, should be black

Hi
that happen also in "חיפוש" after selecting road segment
Kindly check the text  is in white and need to be Black

Additonal case: kindly check the text is in white and need to be Black

additional issue of text not visible

@shaked-hayek I reopened since @MichalOren added additional changes

@shaked-hayek
one more "white on white"
Text in the bottom


myjenkins: Update plugins as of 2017-07-03

ez-templates:1.3.0

Signed-off-by: Gianpaolo Macario<EMAIL_ADDRESS>

Tested locally on a VirtualBox VM on itm-gpaolo-w10


Updating 5star.mp4

Less laggy meteor animation. I got it by splicing 5star-single.mp4 and 5star.mp4 together.
The result is seamless, UID is not shown, and there's no audio/video warping.

Bumped the bitrate down to 5000. New file size is 3.75 MB

Excuse me, I haven't merged it yet. 😅
I was a bit busy earlier, so I just approved and ran code checker.


[RFE] Ensure charts are update to use alternate registry

NEO: Webhook, ~CSP adapter~, etc.
SURE-5047

csp-adapater always has to come from ECR

I think this is a non-issue, with how we are doing the registry. Closing for now.


Proguard config

If you can, add in description that following lines are needed in proguard:
-keep class in.uncod.android.** { *; }

Source:
https://github.com/Uncodin/bypass/issues/195

Thanks for the feedback! This has been done in version 1.0.3


FE: only allow transferring to "main" if on correct MetaMask network

added https://github.com/webaverse/webaverse.com/compare/607ba8e36ad0...21cea5bca3da

added https://github.com/webaverse/webaverse.com/compare/607ba8e36ad0...21cea5bca3da


Spurious output on t/021-accessors.t causes failure

I'm trying to install Math::Matrix using Perl 6 2018.09 and zef is reporting a test failure.
The failing test is t/021-accessors.t and the output is:
1..5
    1..4
    ok 1 - first cell
    ok 2 - first cell
    ok 3 - Out of range row
    ok 4 - Out of range column
ok 1 - Cell
    1..2
    ok 1 - got second row
    ok 2 - tried none existing row
ok 2 - Row
    1..2
    ok 1 - got second column
    ok 2 - tried none existing column
ok 3 - Column
    1..3
    ok 1 - custom diagonal
    ok 2 - identity diagonal
    ok 3 - tried get diagonal of none square matrix
ok 4 - Diagonal
    1..15
-1
    ok 1 - demanded colums are out of range
    ok 2 - demanded submatrix goes out of scope due second cell
    ok 3 - reject none int indices
    ok 4 - rows and colums are out of range
-1
    ok 5 - submatrix built by removing first cell
-1
    ok 6 - submatrix built by removing last cell
    ok 7 - submatrix with range syntax
    ok 8 - submatrix with range syntax using * aka Inf
    ok 9 - simple submatrix created with list syntax
    ok 10 - rehashed submatrix using list syntax
    ok 11 - submatrix with no arguments is matrix itself
    ok 12 - submatrix with range syntax omiting column parameter
    ok 13 - submatrix with range syntax omiting row parameter
    ok 14 - submatrix with list syntax omiting column parameter
    ok 15 - submatrix with list syntax omiting row parameter
ok 5 - Submatrix

Apparently the test is perfectly OK, but its output can't be parsed correctly because for those "-1".
Probably this is a problem in the Test module that comes with Perl 6 2018.09, but since I don't know your module's inner workings I'm asking if there's any reason why a .new method should output a -1 before failing.

Just to let you know: I'm writing an interface to the libfftw3 C library to do Fast Fourier Transforms and I wanted to see if it could take a Math::Matrix object as an input.
It works as long as a user feeds a $matrix.list-rows.flat to it, that is as long as it's not a Math::Matrix object anymore :-)
I know, it's not a proper interface to your module, but I guess it's the best I can do without declaring an explicit dependency.
Thank for your work!

Thanks,
just tell me if you need something.

Now Math::FFT::Libfftw3 accepts a Math::Matrix object; no need to add .list-rows.flat anymore.


@@unique currentely not supported ?

schema.prisma:
datasource db {
    provider = "sqlite"
    url      = "file:dev.db"
}

generator db {
    provider = "go run github.com/prisma/prisma-client-go"
    output = "./db/db_gen.go"
    package = "db"
}

model Customer {
  id              String   @id @default(cuid())
  createdAt       DateTime @default(now())
  updatedAt       DateTime @updatedAt

  name            String
  phone           String
  email           String

  @@unique([name, phone, email])
}

Generating the client with go run github.com/prisma/prisma-client-go generate does not give any output and the client is not generated.
When I omit the @@unique constraint, the client is sucessfully generated.

Can you please specify what version you're on? Can you try adding go get github.com/prisma/prisma-client-go@master to see if it works with the latest dev version?

Thank you very much!
The addition of @master solved this issue.

Great, thanks! I'll soon release these fixes in a new tag.

Duplicate of #179. Released in v4.0.0.

wow, from v0.0.3 to v4.0.0! That must be a huge step forward! ;-) :-P

Oops, thanks for the hint! At least I named the tag correctly :P


feat(typescript): Updated typescript + removed typings

@aitboudad whats the intention behind removing typings?

see https://blogs.msdn.microsoft.com/typescript/2016/06/15/the-future-of-declaration-files/


Re-installation ERROR after python 3.12.3 UPGRADE

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
United Kingdom
Provide a description that is worded well enough to be understood
I've UPGRADED my Arch system, python 3.12.3 has been installed... hmm, this is the second application that has stopped working(ProtonVPN). Look forward to the rest...
FAILED LINE:
test/test_InfoExtractor.py::TestInfoExtractor::test_search_nextjs_data FAILED                                    [  1%]
======================================================= FAILURES =======================================================
______________________________________ TestInfoExtractor.test_search_nextjs_data _______________________________________
self = <test.test_InfoExtractor.TestInfoExtractor testMethod=test_search_nextjs_data>
def test_search_nextjs_data(self):
    data = '<script id="__NEXT_DATA__" type="application/json">{"props":{}}</script>'
    self.assertEqual(self.ie._search_nextjs_data(data, None), {'props': {}})
    self.assertEqual(self.ie._search_nextjs_data('', None, fatal=False), {})
    self.assertEqual(self.ie._search_nextjs_data('', None, default=None), None)
    self.assertEqual(self.ie._search_nextjs_data('', None, default={}), {})

  with self.assertRaises(DeprecationWarning):

E       AssertionError: DeprecationWarning not raised
test/test_InfoExtractor.py:1915: AssertionError
------------------------------------------------- Captured stderr call -------------------------------------------------
WARNING: [Dummy] unable to extract next.js data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
=================================================== warnings summary ===================================================
test/test_InfoExtractor.py::TestInfoExtractor::test_search_nextjs_data
/home/traibr/aur-package/yt-dlp-git/src/yt-dlp/test/test_InfoExtractor.py:1916: DeprecationWarning: using default='{}' is deprecated, use default={} instead
self.assertEqual(self.ie._search_nextjs_data('', None, default='{}'), {})
-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [ff0779267]
[debug] Python 3.12.3 (CPython x86_64 64bit) - Linux-6.8.7-arch1-2-x86_64-with-glibc2.39 (OpenSSL 3.2.1 30 Jan 2024, glibc 2.39)
[debug] exe versions: ffmpeg 6.1.1 (setts), ffprobe 6.1.1, rtmpdump 2.4
[debug] Optional libraries: certifi-2024.02.02, requests-2.31.0, secretstorage-3.3.3, sqlite3-3.45.3, urllib3-1.26.18
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests
[debug] Loaded 1810 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)

Are you trying to install yt-dlp-git from the AUR? Is that why you are running pytest?
This failing test has nothing to do with Python 3.12; it was an oversight on our part. See #9763
You can make this change to the AUR PKGBUILD so that the tests pass:
diff --git a/PKGBUILD b/PKGBUILD
index face841..5cc2eb3 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -47,7 +47,7 @@ build() {

 check() {
     cd "${pkgname%-git}"
-    pytest -v -m "not download"
+    pytest -v -Werror -m "not download"
 }

 package() {

Thanks for the reply @bashonly,
I'm pulling down the AUR from git & running "makepkg" within the directory as I usually do, it's a simple as that. Due to a prior FULL system update that day, python was upgraded, & a number of other AUR app's stopped working as a result. So assumed the usual, which I know I shouldn't do.
I'm not manually running any pytest, unless that's being done via the PKGBUILD file itself (which it is, as I write this); I've made the change as directed within the file, & the compile has worked successfully:
=============================================================== 708 passed, 196 skipped, 5736 deselected in 242.75s (0:04:02) ================================================================ ==> Entering fakeroot environment... ==> Starting package()... ==> Tidying install...   -> Removing libtool files...   -> Purging unwanted files...   -> Removing static library files...   -> Stripping unneeded symbols from binaries and libraries...   -> Compressing man and info pages... ==> Checking for packaging issues... ==> Creating package "yt-dlp-git"...   -> Generating .PKGINFO file...   -> Generating .BUILDINFO file...   -> Generating .MTREE file...   -> Compressing package... ==> Leaving fakeroot environment. ==> Finished making: yt-dlp-git 2024.04.09.r13.gac817bc-1 (Fri 03 May 2024 17:24:53 BST)
Apologies, my coding skills are very much baseline, however I really do appreciate your response :)
Thanks for all the help!

Good to know, thanks again!


feat(conf) configurable request body handling

Summary
Add directives for client_max_body_size and client_body_buffer_size, allowing users to work around request body read limitations without the need to use a custom Nginx config.
Note that we do not adjust or make configurable these values in the Admin API section, as there is currently no need.
Full changelog

Add client_max_body_size and client_body_buffer_size directives and config notes.

These tests passed, but yesterday's shenanigans broke the webhook delivery.
https://travis-ci.org/Mashape/kong/builds/240117836
https://travis-ci.org/Mashape/kong/builds/240117829

Docs PR in https://github.com/Mashape/getkong.org/pull/437

Last minute second thought: I feel like we should just add a couple unit tests for those 2 new properties, to make sure the conf loader and the Nginx template both parse/set the values for those.


RESTful API handler should avoid using mode: 'insensitive' on SQL Server

I am using ZenStack's Express middleware to serve a REST API with an SQL Server as database. When I try to use a contains filter, as in
http://<IP_ADDRESS>:3000/api/gPW_formulations?filter[name$icontains]=sometext

the Prisma client returns the following error:

Indeed, the Prisma documentation states that you can't use mode: 'insensitive' on SQL Server:

Microsoft SQL Server uses case-insensitive collation by default. Therefore, filtering with Prisma Client and Microsoft SQL Server is case-insensitive by default.
mode: 'insensitive' property is not required and therefore not available in the generated Prisma Client API.

https://www.prisma.io/docs/concepts/components/prisma-client/case-sensitivity#microsoft-sql-server-provider
What I'm using:

ZenStack version: 1.3.0
Prisma version: 5.6.0
Database type: SQL Server

Thanks for catching this @elsantoalcielo . For case-senstive filter you can use the "$contains" operator. Currently, the API handlers don't know what database is used so don't have a good way to reject queries that are actually not supported ...

Closing for now due to no follow up.


Upgrade github package to octokit/rest

Upgrade github package to octokit/rest
This couldn't happen automatically because of the way we were constructing the
proxy object for the github API.
The old version of the API returned an object that took a config and could be
extended, the new version of the api dynamically constructs the rest client by
going through it's own list of plugins and attaching methods to itself. This
didn't play well with the es6 extends syntax and after the version bump the
authenticate method we overrode was still pointing to the library version.
I've changed to shadowing it in the constructor instead of overriding it. This
seems to work.
resolves #178
https://trello.com/c/2jHQX3Gu/894-upgrade-github-dependency-to-octokit-rest

go to the apps/new endpoint
try to search for a github repo in the input box
the dropdown should have github repos

Just needed to add the repo prefix hardcoded into the template, since it's no longer coming from config. 😄

LGTM 👍

BTW @r4vi , branch names with a slash in them always fail the docker_build_and_push task. It's ok to merge. 👍