jquery.jstree patched to improve drag-and-drop handling

This proposes a fix of #4881. The two small patches are backported from https://github.com/vakata/jstree/issues/174#issuecomment-8060768 and https://github.com/vakata/jstree/issues/174#issuecomment-8135128.
There are some other useability issues remaining but it is the fastest way to fix this especially annoying issue. Updating to a new jstree version seems to be more work...
P.S.: I am not sure about how to handle changes to a thirdparty library. I have only added a notice at the beginning of the js file.

This is a super annoying bug so it would be great to have it fixed. At some point I hope we'll upgrade to the latest version of jsTree or something different, but that's pretty sure no to happen in the 3.x line so I'm definitely in favour if patching this file in the mean time. I don't see any harm in adding a patch because the 1.x line of jsTree will never be updated again so we don't have to worry about re-patching future releases.
Behat tests passed, just an unrelated date test failing - some kind of New Year issue (expected 2015 got 2016). @tractorcow maybe you know what happened there?

Thanks for the feedback, I have updated my pull request. Regarding the not passing tests, see #3749.
Updating jstree seems to be difficult, I have tried to update to the final 1.0 version (not a release candidate), but there have been some api changes between these versions.

Thanks @sb-relaxt-at, closed and re-opened to force the tests to re-run

Tests seem ok now. :)
I'll ask the UX team here to have a go at testing your fix.

Great, looking forward for this :-). We have done some limited testing in Firefox/Chrome/IE11 and deployed it to some of our SilverStripe instances. Seems to be fine so far.

Can't this be fixed without editing the js plugin or fixed in the upstream?

The related upstream issue has been closed by @vakata (owner) in 2013 (most likely because it has been fixed in 1.0, 2.0 or 3.0 ;-)). We can be quite sure, that there will be no compatible upstream updates for the used version 1.0-rc3. Even updating to 1.0 seems to be difficult due to some changes. Therefore I believe direct patching (which I usually avoid as hell) is acceptable.
It might be possible to fix it without touching the original library, by adding some event handlers and delegations. Nevertheless this seems cumbersome, would require a lot more effort.

I thought I'd chime in with some info on jstree. As far as I can remember all tickets regarding v.1 were closed when v.3 proved to be at least as stable as v.1.
The reason for closing all the tickets was that unfortunately I did not have enough resources to support both versions, so I decided to drop support for v.1.
As for the current situation - v.1 has not been supported for years. v.3 is the only supported version, with v.4 planned for the end of 2016. If you are still using v.1 code I strongly suggest to upgrade to v.3.

@dhensby as @vakata points out we really shouldn't be using the v1 line, but there is apparently a lot of custom code that relies on it in the CMS so updating to a new version looks pretty hard. I don't think it's worth the effort for the SS 3.x line especially if it doesn't carry over to 4.x, which may be the case if we're redeveloping a lot of components. Patching the file seems like the best short-term option.

By the way thanks for the input @vakata and for your plugin!

Time to update to v3 I guess!

I'm happy with this fix - nice work!

@scott1702 Thanks, I am looking forward to see it in the release :-)


Bump version to 1.2.0

See CHANGELOG.md

Released https://github.com/influitive/apartment-sidekiq/releases/tag/v1.2.0


Cannot use pickle module in ironpython3

Hi, Everyone
I can not use pickle load model python file please add

Yes,

It's hard to tell from your example, but I assume you're getting an ImportError? Something like:

ImportError: No module named 'something'

This is not an issue with pickle.load but a missing 3rd party module. If it's not an ImportError then it would be helpful to have a sample model.sav so the issue can be reproduced.

Have your email, please tell me I will send my model


Angular: parameter for custom pagination element ignored (pagination.el)

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 6.4.6

Platform/Target and Browser Versions: IOS, ANDROID, CHROME

Live Link or JSFiddle/Codepen or website with isssue: Reproduced in StackBlitz

What you did
For details see stackblitz
component.ts:
swiperOptions: SwiperOptions = {
  pagination: {
    type: "progressbar",
    el: ".swiper-pagination.my-pagination-container"
  }
};

component.html:
<swiper [pagination]="swiperOptions.pagination">
    <ng-template swiperSlide>

Hello world

    </ng-template>
    <ng-template swiperSlide>

How are you?

    </ng-template>

</swiper>

Expected Behavior
Pagination should be inserted in specified html node.
Actual Behavior
Default pagination is inserted.
Possibly related issue
I saw that there was a similar issue for react, which was never fully solved unfortunately: #3766

fixed in<EMAIL_ADDRESS>

fixed in<EMAIL_ADDRESS>

bonjour j'aimerai utiliser les swiper avec une pagination personnalise peut tu maider


export context setter for recorder

I've exported the context setter for recorder.
Every runner in this library, that exposes metrics, accesses the metrics' context key. The function setting the recorder to the context, however, is unexported and can only be used by the metrics's package middleware.
By exporting it, anybody can write their own metrics middleware, while remaining all other runners metrics reporting to be compatible.

Thanks for the contribution @moepi
Yes, this adds flexibility, merging!


Update pytest fixtures

Currently used fixtures are deprecated, and will be removed with AiiDA version 3, so we should look into using fixtures from aiida.tools.pytest_fixtures.
Most fixtures have a direct correspondence to aiida.manage.tests.pytest_fixtures, but currently doing this seems to result in the default profile not being set when callling verdi run.
This can be fixed along the lines of
def test_example_md(example_path, aiida_profile, janus_code):
    command = ["verdi", "profile", "set-default", f"{aiida_profile.name}"]
    result = subprocess.run(command, capture_output=True, text=True, check=False)

but it ought to be fixable via the fixtures directly.

just be careful how you split the command, passing it like that can be dangerous if not all things are correctly escaped


LawinServer.EXE

It wont open I runned it as admin i restarted my pc nothing will work 😢

Hey. Please take a look a the closed issues and if it doesn't help, hit me up on Discord: Psycho#1234


test.concurrent() code is executed when inside a describe.skip() block

🐛 Bug Report

test.concurrent() appears to run any synchronous code in the test function when nested inside a describe.skip() block.
This code is not run for it() nested inside a describe.skip() block.
This code is not run for test.concurrent.skip().
This can be particularly problematic in integration tests where some functions can have side effects.

To Reproduce
A small example to reproduce:
describe.skip("When in a describe.skip() block", () => {
  it.concurrent("it.concurrent() code should not run", async () => {
    console.log("This (concurrent) line should never be run");
    await new Promise((resolve) => setTimeout(resolve, 0));
    console.log("This async (concurrent) line should never be run");
    fail();
  });

  it("it() code should not run", async () => {
    console.log("This (non-concurrent) line should never be run");
    fail();
  });
});

describe("When in a describe() block", () => {
  it.concurrent("it.concurrent() tests should be run", async () => {
    expect(true).toBe(true);
  });

  it("it() tests should be run", async () => {
    expect(true).toBe(true);
  });
});

The output of jest --verbose is:

Expected behavior
I would expect the behaviour of the following examples to be identical:
describe.skip("", () => {
  it("", async () => {});

  it.skip("", async () => {});

  it.concurrent("", async () => {});

  it.concurrent.skip("", async () => {});
});

it.skip("", async () => {});

it.concurrent.skip("", async () => {});

Link to repl or repo (highly encouraged)
The example above should be sufficient.
envinfo
  System:
    OS: macOS 10.15.7
    CPU: (8) x64 Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
  Binaries:
    Node: 14.15.0 - ~/.nvm/versions/node/v14.15.0/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.8 - ~/.nvm/versions/node/v14.15.0/bin/npm
  npmPackages:
    jest: ^26.6.3 => 26.6.3

Thanks @SimenB .

Thank @dmitri-gb! 😀


pjsip python library

Hi, I was browsing a bit and I read that the PJSUA python module has been superseded by a new PJSUA2 module.
https://www.pjsip.org/docs/book-latest/html/intro_pjsua2.html
Any idea how much work it would be to upgrade this in this project?

Hi @emielvanberlo, I have no experience with PJSUA2 so I cannot say. But you can look through the OpenPhone source code to see in which places we are using PJSUA to get an idea of how much would need to be touched. Have fun!


Add Expr supprt to DAGCircuit simple constructors

Summary
This adds support to the DAGCircuit builder methods apply_operation_front and apply_operation_back, and consequently to the converter circuit_to_dag.  This commit does not directly enhance DAGCircuit.compose, which will come later.
The principal change is that _bits_in_condition is modified to the more general _bits_in_operation, which now accounts for both a condition (if set) and any additional fields from a target if the operation is a suitable ControlFlowOp.  This has the effect of upgrading the converters into the SabreDAG format as well, but nothing is done to change the Sabre algorithms themselves.
Details and comments
Close #10226.  Depends on #10358.  Changelog to follow as part of #10331.
The change of _bits_in_condition to _bits_in_operation likely does most of the remaining work for #10232 on top of its prerequisites #9419 and #9421 (although a PR for #10232 will still need to add tests).

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

478 of 508 (94.09%)  changed or added relevant lines in 20 files are covered.
5 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.06%) to 86.007%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/circuit/classical/types/ordering.py
36
37
97.3%

qiskit/circuit/controlflow/switch_case.py
8
9
88.89%

qiskit/dagcircuit/dagcircuit.py
20
22
90.91%

qiskit/circuit/classical/types/types.py
44
48
91.67%

qiskit/circuit/classical/expr/expr.py
102
111
91.89%

qiskit/circuit/classical/expr/constructors.py
142
155
91.61%

Files with Coverage Reduction
New Missed Lines
%

qiskit/circuit/controlflow/switch_case.py
1
94.08%

qiskit/transpiler/passes/synthesis/unitary_synthesis.py
1
90.39%

crates/qasm2/src/lex.rs
3
90.89%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
71866

Relevant Lines:
83558

💛  - Coveralls

Rebased on top of main.


Adding navbar to blog section

It would  be better if we add navbar to the blog section too, kindly assign this to me.

We are currently creating the blog page using hashnode, so we won't be ableto give access to everyone!


Cow: Blacklist<PHONE_NUMBER>(?#NO NorAm)

Cow requests the blacklist of the number<PHONE_NUMBER>(?#NO NorAm). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Jeff Schaller in Charcoal HQ


Metadata Forms Enhancements and Development

[ ] MF Enforcement. Description

[ ] MF attachement:
- [ ] Organizations
- [ ] Organization Teams
- [ ] Environments
- [ ] Environment Teams
- [ ] Datasets
- [ ] Worksheets
- [ ] Dashboards
- [ ] Consumption roles
- [ ] Notebooks
- [ ] ML Studio entities
- [ ] Pipelines

[ ] Search by MF fields

[ ] Target Entities. E.g. MF A can be attached only to Datasets.

[ ] Rollback all MF changes when fields are not created

[ ] Draft mode?

[ ] UI
- [ ] Display value of glossary node
- [ ] view attached entities

[ ] Integration tests

Some additional enhancements / fixes requested for Metadata Forms (cc: @rbernotas):
Enhancement/ Fix Requests:

[Nice to have] "Add Field" button also at bottom in the MF Create form, for MF's with many fields to enter
[Nice to have] "Attach" button at bottom of the MF attach Form, for MF’s with many values to enter
Need to be able to audit:  For a given MF, which Datasets have attached it, and which Datasets have not?
We should be able to export/download metadata form data, either from a single attached metadata form, or in bulk (like from all datasets, for a given metadata form)
Catalog Search (mentinoed above but in more detail)

Filter to see only which datasets have a particular metadata form attached
Broadly search for some text that might appear in any metadata form that is attached to any dataset
Filter on a specific metadata form field, presence of that field in a dataset, or value of it.

Input Validation and UI View

(BUG) value entry in the MF fields isn’t checked for string length.  Tried entering 37k chars in a field, got a network error.  String input should also be sanitized in some way to check for injection attacks, etc.
(BUG) a long string in a string entry field, once attached and rendered, is truncated.  I saw this starting to happen around 68 characters.
(BUG) when attaching a MF, the name of the field can be truncated if the Name is long.  Seems like the name should be outside of the value entry box, and line wrapped.  Also seems like we should be checking and/or limiting number of characters for the Name of a field.
(BUG) after a MF is attached, if the name of the field is very long, in the metadata table the value of the field will be truncated.

Required boolean field isn't useful, have to always pick True (bug?) or double-click UX is confusing
Currently there is no enforcement, and the MF form tab for Enforcement reads "Under Construction"
If a MF is already attached to a dataset - Need to be able to edit existing values of already attached MF:

I can edit the MF fields as the owner of the MF
Adding a field does not require the dataset to enter a value for the new field

It's not shown, there is also no way to edit the data in the attached form on the Dataset
Seems like we should be able to add a field to the MF, and then there should be a process to update the attached MF on Datasets, with the new field and value.

Deleting a field DOES remove it from any attached metadata forms on the Datasets.

This seems fine, but also feels inconsistent if I can't also add fields to already attached MF's.

@SofiaSazonova - I think we already have some parts of the above around:

enforcement strategy
View attached entities / audit enhancement
catalog search
attach metadata form editability
... all as part of future release but please confirm if that is the case.

Otherwise, I see a couple of other requests in the above, mainly:

Input Validation / UI Fixes for String Length + Boolean Field
Export/Download MF data
Additional Buttons

Maybe we can figure out if/how we plan to prioritize the other pieces?

Our top priorities would be in this order:

Enforcement - ability to enforce a given metadata form across all datasets in data.all
Edit attached form - ability for user to edit values in a metadata form that has been attached to a dataset already, without having to detach and reattach the form.

UI view enhancements for attached metadata forms (fix things like truncation of values and input fields, etc.).

Discoverability using metadata form information in Catalog search
Ability to audit what has been attached to a metadata form

The rest are also valuable but these would be our top 5, with particular emphasis on 1 & 2.  Thanks!


Switch the returned coordinates in station linestrings

Its currently -122 then 45, it should be 45 then -122

Mind changed. Per discussion with @pgicking this appears to be an issue in PostGis. (It should be returning points in EPSG:4326's lat/lng order)
The only good solution would be to take the points apart in PHP and put them into individual "lat:" and "lng:" fields in the JSON, and that's a parsing overhead we just don't want to introduce. We'll leave them as is.


Comp1

Before issuing a pull request, please see the contributing page.

It looks good.

Sorry about this. This pull request was created by accident.


 In application/controllers/Checkout.php find method sendNotifications() and there you can find the logic of notifications. The query is "SELECT email FROM users WHERE notify = 1". You admin account must have notify value 1 and you must have configured email server in  - application/libraries/SendMail.php

thank you sir but now an eroor occuered sir in admin/orders new orderid not
confirmed msg shown what is that sir
On Tue, Feb 19, 2019 at 9:04 PM Kiril Kirkov<EMAIL_ADDRESS>wrote:

Closed #143
https://github.com/kirilkirkov/Ecommerce-CodeIgniter-Bootstrap/issues/143
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kirilkirkov/Ecommerce-CodeIgniter-Bootstrap/issues/143#event-2149231180,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdGuQSbgBuNpjk7nnmaGeUNF3lU-OOEbks5vPBl4gaJpZM4bCKMv
.

thank you sir but now an eroor occuered sir in admin/orders new orderid not
confirmed msg shown what is that sir  and when i click processed button
error with status change check log msg appear as popup
On Tue, Feb 19, 2019 at 9:13 PM Aswin S<EMAIL_ADDRESS>wrote:

thank you sir but now an eroor occuered sir in admin/orders new orderid
not confirmed msg shown what is that sir
On Tue, Feb 19, 2019 at 9:04 PM Kiril Kirkov<EMAIL_ADDRESS>wrote:

Closed #143
https://github.com/kirilkirkov/Ecommerce-CodeIgniter-Bootstrap/issues/143
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kirilkirkov/Ecommerce-CodeIgniter-Bootstrap/issues/143#event-2149231180,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdGuQSbgBuNpjk7nnmaGeUNF3lU-OOEbks5vPBl4gaJpZM4bCKMv
.

A PHP Error was encountered
Severity: Warning
Message: A non-numeric value encountered
Filename: ecommerce/orders.php
Line Number: 146
Backtrace:
when i click admin/orders moreifo under product details this will appear
On Tue, Feb 19, 2019 at 9:15 PM Aswin S<EMAIL_ADDRESS>wrote:

thank you sir but now an eroor occuered sir in admin/orders new orderid
not confirmed msg shown what is that sir  and when i click processed button
error with status change check log msg appear as popup
On Tue, Feb 19, 2019 at 9:13 PM Aswin S<EMAIL_ADDRESS>wrote:

thank you sir but now an eroor occuered sir in admin/orders new orderid
not confirmed msg shown what is that sir
On Tue, Feb 19, 2019 at 9:04 PM Kiril Kirkov<EMAIL_ADDRESS>wrote:

Closed #143
https://github.com/kirilkirkov/Ecommerce-CodeIgniter-Bootstrap/issues/143
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kirilkirkov/Ecommerce-CodeIgniter-Bootstrap/issues/143#event-2149231180,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdGuQSbgBuNpjk7nnmaGeUNF3lU-OOEbks5vPBl4gaJpZM4bCKMv
.

@aswinsasi I cant reproduce. Give me step by step

Sir i cant get email notifiaction after i put my email address and password
on phpmailer ..
There is no notification ...what i need to change....the username and
password in sendmail() is changed witg my gmail email and password...what
to do to get notificaion
On Wed, Feb 20, 2019, 00:37 Kiril Kirkov<EMAIL_ADDRESS>wrote:

@aswinsasi https://github.com/aswinsasi I cant reproduce. Give me step
by step
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kirilkirkov/Ecommerce-CodeIgniter-Bootstrap/issues/143#issuecomment-465268364,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdGuQcxxgF3PDH6eUy-WQCi7jcA5siueks5vPEuAgaJpZM4bCKMv
.

Thank you.i got it
On Wed, Feb 20, 2019, 08:02 Aswin S<EMAIL_ADDRESS>wrote:

Sir i cant get email notifiaction after i put my email address and
password on phpmailer ..
There is no notification ...what i need to change....the username and
password in sendmail() is changed witg my gmail email and password...what
to do to get notificaion
On Wed, Feb 20, 2019, 00:37 Kiril Kirkov<EMAIL_ADDRESS>wrote:

@aswinsasi https://github.com/aswinsasi I cant reproduce. Give me step
by step
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kirilkirkov/Ecommerce-CodeIgniter-Bootstrap/issues/143#issuecomment-465268364,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdGuQcxxgF3PDH6eUy-WQCi7jcA5siueks5vPEuAgaJpZM4bCKMv
.


build osx_arm64

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

refs #36
@conda-forge-admin  please rerender

@conda-forge-admin please rerender

Another green one for you @ocefpaf


Inertia not updated when calling Body.setMass()

I started using Matter.js in my game editor where you can create bodies and edit them later on. I noticed that these two code snippets give different results:
var body = Matter.Bodies.rectangle(x, y, w, h, { density: 0.01 });

var body = Matter.Bodies.rectangle(x, y, w, h, { density: 0.001 });
Matter.Body.setDensity(body, 0.01);

Both of the snippets will set density to 0.01, but since inertial is only calculated on creation of the body, these snippets will result in different body.inertial value. Therefore bodies act really weird.
When density changes, mass is recalculated. Shouldn't inertial be also recalculated when mass changes? If you think so, I would be happy to contribute.

Not recalculating inertia is probably good idea if plenty of bodies change their mass a little. But atleast a mention in comment or taking third parameter Body.setMass = function(body, mass, recalculateInertia)

Good spot. Recalculating inertia in this case is easy, we can just do:
var moment = body.inertia / (body.mass / 6);
body.inertia = moment * (mass / 6);

I'll tag this as a bug, thanks.

// allow options to override the automatically calculated properties
Body.set(body, {
	axes: options.axes || body.axes,
	area: options.area || body.area,
	mass: options.mass || body.mass,
	inertia: options.inertia || body.inertia
});

This code might break if inertia is set in setMass function. Might have to separately call setInertia afterwards.
I really like how you have structured the library. Well done!

Ran into this bug too, had me real confused for a hot minute.
For me, I was using Matter.Body.setDensity to set the density of a body, but when I tested it, it seemed that the body's rotational velocity was being computed correctly, but the inertia didn't account for its linear velocity.
I'm still a bit confused as to why the inertia not being updated only seemed to be effecting the linear velocity and not the rotational velocity.
var moment = body.inertia / (body.mass / 6); body.inertia = moment * (mass / 6);
I assume is supposed to be:
var moment = body.inertia / (body.mass / 6); body.inertia = moment * (body.mass / 6);


Identify observability frameworks/tools and add links to starter guides

Issue:
Team invested time to spike and understand if a observability framework (Open Telemetry) will fit their needs. When a framework is not ready for prime time, multiple teams go down the same investigation path.
Notes:
Identify observability frameworks/tools and add links to starter guides.
For example - Open Telemetry - starter guide (I thnk we already have this one on the PB)

@narmathabala - this issue has been stale for a while, is it still relevant with an active plan or should it be closed

Content added


Generating multiple schemas with same name

Describe the issue
There's multiple DataLakeStorageError classes when generating azure-storage-blob.
** REQUIRED **
ModelerFour version: 4.10.246
AutoRest-core version: 3.0.6258
CommandLine : autorest ../azure-sdk-for-python/sdk/storage/azure-storage-blob/swagger/README.md --use=. --output-artifact=code-model-v4-no-tags
OpenAPI/Swagger file: https://raw.githubusercontent.com/Azure/azure-rest-api-specs/storage-dataplane-preview/specification/storage/data-plane/Microsoft.BlobStorage/preview/2019-07-07/blob.json

You'll have to fix the swagger.  The name guesser can't make a good name for the unnamed inner type.
Add a x-ms-client-name to the inner definition
"DataLakeStorageError": {
      "type": "object",
      "properties": {
         # it's guessing the name for this type should be DataLakeStorageErrorError,
         # but deduplicates Error so it collides.
        "error": {
          "x-ms-client-name" : "ErrorDetails",  # add this line to give it a real name.
          "description": "The service error response object.",
          "properties": {
        "Code": {
              "description": "The service error code.",
          "type": "string"
        },
        "Message": {
              "description": "The service error message.",
          "type": "string"
            }
          }
        }
      }
    },

I've also turned on the checker to produce errors when there are duplicate names.

https://github.com/Azure/azure-rest-api-specs/pull/8739 fixed this for 2019-07-07, it should be propagated to the other API versions.

Hmm actually I'm not so sure that "fix the swagger" is the right solution here.  This is also present in blob storage management plane.
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/storage/resource-manager/Microsoft.Storage/stable/2019-06-01/blob.json#L1123
Presumably the pattern exists elsewhere and is perfectly legal.  In track 1 there was some "disambiguation phase" that presumably resolved this (I haven't looked at the code).  For Go, we ended up with a type name BlobServicePropertiesProperties; while this isn't the best name, the generated code compiles.  IMO if the authors want a better name than this default they should specify x-ms-client-name to fix the name.

We're better off making the author be explicit, rather than just getting lousy inferred model names.
Auto-generated names are at best, a courtesy, and we can try to do a good job, but at the end of the day, this is an authoring failure (because well, you know, swagger sucks)

I think before we commit to this course of action we should identify how many swaggers are impacted by this design change.
Assuming we do decide to not fix it, I will also point out that fixing up the names in already released swaggers will introduce breaking changes in track 1 unless we apply the fix such that we preserve the old names.

given that the logic in generating names is completely re-engineered from what was there before in v2, we should do it for that reason alone :D

@lmazuel and @iscai-msft : I believe Python has tried almost all swaggers. How many places have we encountered this?
@lmazuel : is this a place where we want a linter rule to catch these things in review?

@mayurid I submitted a linter rule indeed
@iscai-msft  could confirm, but I think we didn't see that much

@lmazuel: for the error we're talking about i've only seen DataLakeStorage

I think this is related?  https://github.com/Azure/autorest.modelerfour/issues/241

It looks like #241 is listing all of the swaggers @lmazuel has run into that have caused a duplicate object schema error.
Whether these errors caused by not specifying an x-ms-client-name (the error we see in DataLakeStorage), or having duplicate schemas defined in separate files, or another reason, is unknown, but it's great to have them all in one spot. Thanks @lmazuel for compiling!

confirmed, closing now

Awesome, thank you!


change sendgrid bounce webhook from http simple auth to checking a query...

Removes http basic auth from the sendgrid webhook endpoint, and replaces it with a simple check of a "token" param.
In addition to merging, a SENDGRID_EVENT_TOKEN env var needs set, and we need to tell SendGrid about it.

+1


Add dependency check to Makefile

A new Bash script dependancy_check.sh has been added to ensure that the minimum required Bash version is used and that required commands used by generate are available. This script is now called in the Makefile as part of the check_dependencies task. This aims to prevent errors due to incompatible Bash versions or missing commands.
After encountering some minor inconvenience debugging why then generate.sh script was failing, I added this script to ensure others don't waste any time.
Also, minor clean up of the Development readme.

 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.

Thanks of this, the DEVELOPMENT.md is totally out-of-date and requires a proper rewrite, perhaps we can do that in a separate PR.

I've made updates to the Makefile in #129 and rewritten DEVELOPMENT.md in #133, please have a look at these.


Update project setup

Here is the configuration to build the library and sample with Gradle.
This will also allow you to import the project into Android Studio while keeping the folder structure as created with Eclipse.

@bradipao Ping.

Sorry for lack of answer so far, but since september I was very busy with my actual job (also my activity graph on github is flat-zero since then). I planned to merge the pull request and restart experimenting, but at present I don't know when... maybe next summer holidays. Sorry again. :(


Board Review: Azure Communication Services (SPOOL)  Video effects, background blur [JS]

Service name:  ACS Calling
Key contact for this review: @Prathmesh Kestikar
Link to previous API version:  ...
Description of the material to be reviewed/changed: ...
Champion scenarios that briefly explain how developers will use your API: ...
Languages for this review. JavaScript (TypeScript )
Artifacts per language.
https://apiview.dev/Assemblies/Review/9a67b94f720c4138845ecf5e8f0efac8?revisionId=3a33e39d7d544c5fa29d9c095e47de3a&doc=False

#Scheduled

Scheduled for 9/20 2p-4p PDT

Update:
Service name: ACS Calling
Key contact for this review: @Prathmesh Kestikar
Link to previous API version: ...
Description of the material to be reviewed/changed:
A new feature video effects feature in the Web SDK, will enable devs to add Background Blur/Background Replacement feature in their calling apps.
Champion scenarios that briefly explain how developers will use your API:
// Import packages import { LocalVideoStream } from '@azure/communication-calling'; import {      BackgroundBlurEffect,     BackgroundReplacementEffect  } from '@azure/communication-calling-effects';
To start effects:
`
const localVideoStream = new LocalVideoStream(); // Or get it from the call instance
const effectsFeatureApi = localVideoStream.feature(Features.EffectsFeature);
const bgBlurEffect = new BackgroundBlurEffect({});
// Start effects
await effectsFeatureApi.startEffects([bgBlurEffect]);
`
To change effect type (example for background replacement with image):
`
let imageURL = 'https://backgroundimage.com/background1.jpg';
// NOTE: URL access would be the dev's responsibility (CORS issues will result in failure to apply effects).
// To change effect:
const bgReplacementEffect = new BackgroundReplacementEffect({ backgroundImageURL: imageURL });
// Start the new effects
await effectsFeatureApi.startEffects([bgReplacementEffect]);
// To change the background image:
imageURL = 'https://backgroundimage.com/background2.jpg';
await bgReplacementEffect.configure({ backgroundImageURL: imageURL });
`
Stopping effects:
await effectsFeatureApi.stopEffects();
If you want to create/use your own custom effects, do it the following way:
`
// Create your frame processing function, following the 'FrameCallback' type,
const frameCallback: FrameCallback = async (inputFrameCanvas, outputFrameCanvas) => {
// Take frames from the inputFrameCanvas,
// do your processing on it, tensorflow models to detect stuff, draw stuff, whatever
// draw that image (frame) back on the outputFrameCanvas
};
// Create a Custom Effect Processor using the method 'createCustomVideoEffect' and passing in your frame callback:
const customEffect = createCustomVideoEffect(frameCallback);
// And then apply the effects in the SDK
await effectsFeatureApi.startEffects([customEffect]);
// To stop
await effectsFeatureApi.stopEffects();
`
Languages for this review. JavaScript (TypeScript )
Artifacts per language.
https://apiview.dev/Assemblies/Review/9a67b94f720c4138845ecf5e8f0efac8/efa81a9c6ac546af9ea8f94615008cfc?diffRevisionId=db85d0b440e94933be368663bfa9221d&doc=False&diffOnly=True

Recording (MS INTERNAL ONLY)


how to hidden object in uiSchema

Prerequisites

[x] I have read the documentation;
[x] In the case of a bug report, I understand that providing a SSCCE example is tremendously useful to the maintainers.

Description
Object type still display in the UI even setting "ui:widget": "hidden" to properties
Steps to Reproduce

module.exports = {
schema: {
title: "A registration form",
description: "A Organization form example.",
type: "object",
required: ["OrgaizationID", "Site","OrganizationName","Legal_Name","Lifecycle_Status","Account_Type"],
properties: {  "id":  {
"type": "object",
"properties": {
"$oid": {
"type": "string"
}
},
"additionalProperties": false,
"required": [
"$oid"
]
},
"etag": {
"type": "object",
"properties": {
"$oid": {
"type": "string"
}
},
"additionalProperties": false,
"required": [
"$oid"
]
},
},
}

[Second Step]
uiSchema: {
_id : {
"ui:widget": "hidden",
},
_etag: {
"ui:widget": "hidden",
}
},

[and so on...]
_id,  _etag still showing in UI

Expected behavior
_id,  _etag which is object type can be hidden in UI
Actual behavior
_id,  _etag still display in UI
Version
0.43.0

please take a look at the issues. Thanks.

please make your issue readable by formatting the code, see https://guides.github.com/features/mastering-markdown
also, in your schema you're using properties id and etag while you use _id and _etag in the uiSchema.

It's explained in the documentation https://github.com/mozilla-services/react-jsonschema-form/#hidden-widgets


2D Projection visualisation - color color diagram

Hi @MaxRonce could you put some plots in here ?

The 2D projection visualisation was done in the PCA branch
And we'll try do add some C/C diagram and physc/cluster corelation diagram

The 2D projection visualisation was done in the PCA branch
And we'll try do add some C/C diagram and physc/cluster corelation diagram


[Doc Improvement][Update SBS Teams AI library]

updated skprompt.txt

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/sbs-botbuilder-conversation-AI.yml
88
72
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


Unable to load images in the site if deployed in a sub-folder.

Describe the bug
Failing to load images in the website as well as the preview section for sites in a sub-folder.
For instance, www.example.com/netlify-cms-demo/.
To Reproduce

Deploy a site at a sub-folder level.
Try uploading an image from the Netlify-CMS. The image in the preview pane is broken as well as when I publish the site.

Expected behavior
The images should load irrespective of the sub-folder deployment. The request for the images go to www.example.com/img/sample.png.
But ideally it should go to www.example.com/netlify-cms-demo/img/sample.png.
I am using Gatsby along with netlify-cms.
Applicable Versions:

Netlify CMS version: ^2.9.1
Git provider: Gitlab
OS: MacOS Mojave, version 10.14.5
Browser version Chrome, 74.0.3729.131
Node.JS version: 10.15.3
Gatsby version: 2.4.3
Gatsby-plugin-netlify-cms version: ^4.0.1

CMS configuration
media_folder: static/img
public_folder: /img
Additional context
I resolved the issue by prepending the sub-folder name to the public_folder property like
media_folder: static/img
public_folder: netlify-cms-demo/img

As I am using Gatsby, they handle the sub-folder issue using the property pathPrefix in the gatsby-config.js file. Similarly Hugo has baseUrl.
If possible, can config.yml read the pathPrefix property in gatsby-config.jsor similar properties from other site generators and append it to the public_folder?

Using public_folder isn't a workaround, it's the feature that provides support for your use case. This issue would be better presented as a feature request for automatically detecting site root from your SSG config. It's non-trivial as site generators vary, and a new extension interface would need to be defined and implemented to do it right, but we should at least start the conversation: #2347


In Label Studio Enterprise version I couldn't find Enable Scripts option

I took Label Studio enterprise version for adding custom scripts but even after granting accesss from owner I couldn't see Custom Scripts option.It is said that Custom scripts are defined in the project settings under the Labeling Interface section
But I couldn't see any option Please help me with this

How did you solve it, i have the same problem

Hi , I encountered a similar issue with Label Studio, and I was able to resolve it by contacting the Label Studio support team directly at<EMAIL_ADDRESS>or raise an issue in label studio support tab


Resize window to not include sidebar

as per design, calculate appropriate SVG size in window. right now the sidebar overlaps the nodes.

the latest PR with the background fixes this. however, the nodes are not constrained by d3 to stay within the window.

seems like this could be solved by using Math.min/max to create a bounding box http://stackoverflow.com/questions/9573178/d3-force-directed-layout-with-bounding-box


Fix flaky elasticsearch integration test

Resolves #1108

I will rerun in the evening to see if it fixes the problem.
please run go-fmt on the changed file.

I think this is default. At least this value was used here before.
On Thu, Nov 1, 2018 at 7:48 PM Yuri Shkuro<EMAIL_ADDRESS>wrote:

@yurishkuro commented on this pull request.
In plugin/storage/integration/elasticsearch_test.go
https://github.com/jaegertracing/jaeger/pull/1160#discussion_r230153031:

@@ -101,6 +101,7 @@ func (s *ESStorageIntegration) initSpanstore(allTagsAsFields bool) {
Logger:            s.logger,
MetricsFactory:    metrics.NullFactory,
IndexPrefix:       indexPrefix,

MaxSpanAge: 72*time.Hour,

what's the default?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/jaegertracing/jaeger/pull/1160#pullrequestreview-170825884,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFW6-CTLSpA0YxbQ03TmRt2aKi2ZaILQks5uq0HwgaJpZM4YIFZ6
.

I will merge this. The failures are annoying.


CLN: Cleaner Series col name assignment in SampledPlot

area plot still works and tests all pass locally, @HyukjinKwon thanks for pointing it out!!

ooops, hahah LGTM


Debian 11 (bulleyes) - crowdsec-nginx-bouncer (0.0.7) - missing envsubst

sudo apt install crowdsec-nginx-bouncer
...
Setting up crowdsec-nginx-bouncer (0.0.7) ...
cscli is /usr/bin/cscli
cscli/crowdsec is present, generating API key
API Key : f8f59946a13b3a6f0f69f41e9a5289ef
/var/lib/dpkg/info/crowdsec-nginx-bouncer.postinst: 26: envsubst: not found
Restart nginx to enable the crowdsec bouncer : systemctl restart nginx
...

Some verifications:
admin@myREVERSE:~$ cat /etc/crowdsec/bouncers/crowdsec-nginx-bouncer.conf

envsubst is missing from Debian !
Some possible workarounds and/or resolutions:

gettext-base dependencie

admin@myREVERSE:~$ sudo apt install gettext-base

admin@myREVERSE:~$ envsubst --version
envsubst (GNU gettext-runtime) 0.21
Copyright (C) 2003-2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Written by Bruno Haible.

envsubst in go
https://github.com/a8m/envsubst
It is the planned solution for OpenWrt...
PR porposed https://github.com/openwrt/packages/pull/16664

replace envsubst with sed patterns
It is the actual workaround used in Crowdsec PR for OpenWrt

## Gen&ConfigApiKey
SUFFIX=`tr -dc A-Za-z0-9 </dev/urandom | head -c 8`
API_KEY=`/usr/bin/cscli bouncers add cs-firewall-bouncer-${SUFFIX} -o raw`
sed -i "s,^\(\s*api_key\s*:\s*\).*\$,\1$API_KEY," $CONFIG

and
 	sed -i "s,^\(\s*mode\s*:\s*\).*\$,\1$BACKEND," $VARCONFIG

from : https://github.com/openwrt/packages/pull/16244/files

For (more) information:
sudo dpkg-reconfigure crowdsec-nginx-bouncer
do not help to fix
sudo apt install crowdsec-nginx-bouncer --reinstall
do not help to fix
sudo apt purge crowdsec-nginx-bouncer
sudo apt install crowdsec-nginx-bouncer

fixed the installation (with envsubst added manually by sudo apt install gettext-base)
admin@myREVERSE:~$ cat /etc/crowdsec/bouncers/crowdsec-nginx-bouncer.conf
API_URL=http://<IP_ADDRESS>:8080
API_KEY= 2aa9ca638b5eeec97cff173c42a9cb8b
LOG_FILE=/var/log/crowdsec_lua_bouncer.log
LOG_LEVEL=INFO
CACHE_EXPIRATION=1
CACHE_SIZE=1000

Hello @erdoukki !
Seen the issue, nice spotting.
Adding it to the deps right now :+1:


Sentinel-3 has invalid checksums

The values for file:checksum are not valid multihash values, e.g. in https://planetarycomputer.microsoft.com/api/stac/v1/collections/sentinel-3-olci-wfr-l2-netcdf/items/S3A_OL_2_WFR_20240310T021506_20240310T021806_0179_110_046_1980

xref https://github.com/stactools-packages/sentinel3/issues/32
For reference, that item currently has:
    "assets": {
        "iwv": {
            "href": "https://sentinel3euwest.blob.core.windows.net/sentinel-3/OLCI/OL_2_WFR___/2024/03/10/S3A_OL_2_WFR____20240310T021506_20240310T021806_20240311T083844_0179_110_046_1980_MAR_O_NT_003.SEN3/iwv.nc",
            "s3:shape": [
                4091,
                4865
            ],
            "file:size": 2866583,
            "file:checksum": "43cffe252dc0eacac1f462a27e833b87"
...

Once that's fixed upstream we can update our production version and reprocess the existing items.


Added Back Button to AppBar in the sub-pages of the Guidelines tab

New animation:

Slightly slower to make it look smooth
AppBar has a back button
Spans the entire top AppBar portion of the screen

Entire top AppBar portion is modestPink
Leaves no white space

Old
New
New (slo-mo)

Accidentally put the PR to go from forked dev into original main. Corrected it to go from forked dev into original dev.

@daemon29 @leilanihagen Please let me know if I need to make any changes. Thanks!

It looks great.


create image unsupression script

We have a simple way to suppress a miro image.
But we don't have a symmetrical way to reverse the suppression.
(as requested here)
This needs to reverse what happens here
We do have a function, make_image_available, which doesn't appear to be called from anywhere.  I also don't think it does everything that it needs to do to make an image reappear on the website.

When I was searching for the procedure for registering on DLCS, one of the things I found was https://github.com/wellcomecollection/catalogue-pipeline/issues/2144 ("We should reingest it, and document the procedure if it isn't already."). I could not find a documented procedure.
The unsuppression script created for this ticket forms the new documented procedure.

While working on this, I noticed that L0071354 and L0071355
are invisible but show as Image records on the website.  I wonder if they also both need to be re-registered on DLCS


DO NOT MERGE, uploading 0.6.1 binaries

to include the 1.5.1 release of pytorch
cc @fmassa
Signed-off-by: Eli Uriegas<EMAIL_ADDRESS>

The binaries have been uploaded, closing this


Add deborahchua as new user

I've fixed a lint error a re-pushed.
https://ci.dev.publishing.service.gov.uk/job/puppet-branches/18473//warnings8Result/
Will merge on green.


GDScript 2.0: Typed assigning an inner class leads to a runtime error

Godot version
v4.0.beta3.official [01ae26d31]
System information
Windows 10
Issue description
I have a function auto_free(<instance>) : Object to register instances to be get auto freed after test execution on my plugin GdUnit3.
The function signature is
func auto_free(obj) -> Object:
and returns the same obj
When going back to the original class, an error occurs at runtime
var instance :AdvancedTestClass.AtmosphereData = auto_free(AdvancedTestClass.AtmosphereData.new())
Trying to assign value of type '' to a variable of type 'AdvancedTestClass.gd'.
Steps to reproduce
Copy this code and run it.
class_name TestScene
extends Node

class Custom:
	pass

func _ready():
	test_wrapper()

func test_wrapper():
	# with this inner class it works
	var original: = TestScene.Custom.new()
	var as_object :TestScene.Custom = auto_free(original)
	prints(as_object)

	# with this inner class defined in ExampleClass it breaks
	var original_: = ExampleClass.Data.new()
	# this line fails at runtime
	var as_object_ :ExampleClass.Data = auto_free(original_)

# just return same value
func auto_free(value :Object) -> Object:
	return value

The external script/class
class_name ExampleClass
extends Resource

class Data:
	pass

Minimal reproduction project
No response

just tested that this was an issue on beta 3 and that it's fixed as of https://github.com/godotengine/godot/pull/68374 @akien-mga


would be nice to hook into codegen

whether codegen in specific files e.g. builder, action,  base ents etc
or

run this codegen script before codegen

or even run this script, get JSON back which adheres to an API and use that JSON to indicate what new queries/connections are added, args for that and even imports, contents of that code


[Bug Report] Weird loading and play behavior with allot of explainable errors

Describe the bug
No music is playing, when I checked on the console it was doing really weird stuff:
https://gyazo.com/e3752b02d9f1764c6a6bb9c03bf7805a
I have no clue what it can be, but after a minute of being weird like that it start playing 1 song.
In discord it keeps cycling through the music numbers in queue.
When I try and load my playlist I get the following error (it contains Youtube links):
https://gyazo.com/be04dc30d12223679000863a073e2cc4
I had this bot already in my Discord for over a month and the new update worked like a charm.
To Reproduce
Steps to reproduce the behavior:

Just play music and it happens

Expected behavior
My expected behavior is playing music like it always did.
Screenshots
See up top
Debug Info
Bot 1:
System Properties:
java.version = 1.8.0_211
java.vm.name = Java HotSpot(TM) 64-Bit Server VM
java.vm.specification.version = 1.8
java.runtime.name = Java(TM) SE Runtime Environment
java.runtime.version = 1.8.0_211-b12
java.specification.version = 1.8
os.arch = amd64
os.name = Windows 10
JMusicBot Information:
Version = 0.2.6
Owner =<PHONE_NUMBER>50108416
Prefix = ?
AltPrefix = null
MaxSeconds = 0
NPImages = false
SongInStatus = true
StayInChannel = true
UseEval = false
UpdateAlerts = true
Dependency Information:
JDA Version = 3.8.3_464
JDA-Utilities Version = 2.1.5
Lavaplayer Version = 1.3.25
Runtime Information:
Total Memory = 56
Used Memory = 22
Discord Information:
ID =<PHONE_NUMBER>14583563
Guilds = 2
Users = 333
Bot 2:
System Properties:
java.version = 1.8.0_211
java.vm.name = Java HotSpot(TM) 64-Bit Server VM
java.vm.specification.version = 1.8
java.runtime.name = Java(TM) SE Runtime Environment
java.runtime.version = 1.8.0_211-b12
java.specification.version = 1.8
os.arch = amd64
os.name = Windows 10
JMusicBot Information:
Version = 0.2.6
Owner =<PHONE_NUMBER>50108416
Prefix = !
AltPrefix = null
MaxSeconds = 0
NPImages = false
SongInStatus = true
StayInChannel = true
UseEval = false
UpdateAlerts = true
Dependency Information:
JDA Version = 3.8.3_464
JDA-Utilities Version = 2.1.5
Lavaplayer Version = 1.3.25
Runtime Information:
Total Memory = 49
Used Memory = 13
Discord Information:
ID =<PHONE_NUMBER>70730241
Guilds = 1
Users = 281
Additional context
Not really :)
Thanks in advance
Kind regards,
Dennis

This appears to be a variant of #305. If you can, please copy the contents of one of the errors from the console (from your mp4) onto a text host like hastebin.com and add that as a comment to #305.


Issue from website page Working with cached images

From URL: https://flutter.io/cookbook/images/cached-images/
This example program gives an error in the IDE. The part of the code that fails;
import 'package:cached_network_image/cached_network_image.dart';

What's the error?
Please add the output of flutter doctor

Hi,
See attached screen dumps for reference. Maybe there is a step that I
have missed which cause the android studio IDE to complain.
Rgds
On 5/23/18, Günter Zöchbauer<EMAIL_ADDRESS>wrote:

What's the error?
Please add the output of flutter doctor
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/flutter/flutter/issues/17844#issuecomment-391417009

That didn't work. Can you please try to add them using the web page?

Ok, went to web page and uploaded. Did that work?
On 5/23/18, Günter Zöchbauer<EMAIL_ADDRESS>wrote:

That didn't work. Can you please try to add them using the web page?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/flutter/flutter/issues/17844#issuecomment-391455457

No, but you should be able to verify yourself on the web page..

OK, not sure if it worked. Looks like some files are there now
On 5/23/18, Günter Zöchbauer<EMAIL_ADDRESS>wrote:

No, but you should be able to verify yourself on the web page..
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/flutter/flutter/issues/17844#issuecomment-391457936

How is transparent_image related to https://flutter.io/cookbook/images/cached-images/ ?

It's only related in the fact that its listed as an example in the same  "Cookbook" section on the website. I have attached one more screen dump for ref.

Did you run flutter packages get
Also similar problems disappeared after restarting the IDE.

Yes, I ran the "flutter packages get" command and restarted the IDE.
No success, same issues.
On 5/24/18, Günter Zöchbauer<EMAIL_ADDRESS>wrote:

Did you run flutter packages get
Also similar problems disappeared after restarting the IDE.
--
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
https://github.com/flutter/flutter/issues/17844#issuecomment-391578277

Please post the content of the pubspec.yaml file.

The content of the pubspec.yaml file;
name: flutter_app2
description: A new Flutter application.
dependencies:
flutter:
sdk: flutter
The following adds the Cupertino Icons font to your application.
Use with the CupertinoIcons class for iOS style icons.
cupertino_icons: ^0.1.0
dev_dependencies:
flutter_test:
sdk: flutter
For information on the generic Dart part of this file, see the
following page: https://www.dartlang.org/tools/pub/pubspec
The following section is specific to Flutter.
flutter:
fonts:
- family: Raleway
fonts:
- asset: fonts/Raleway-Regular.ttf
- asset: fonts/Raleway-Italic.ttf
style: italic
- family: RobotoMono
fonts:
- asset: fonts/RobotoMono-Regular.ttf
- asset: fonts/RobotoMono-Bold.ttf
weight: 700
The following line ensures that the Material Icons font is
included with your application, so that you can use the icons in
the material Icons class.
uses-material-design: true
To add assets to your application, add an assets section, like this:
assets:
- images/a_dot_burr.jpeg
- images/a_dot_ham.jpeg
An image asset can refer to one or more resolution-specific "variants", see
https://flutter.io/assets-and-images/#resolution-aware.
For details regarding adding assets from package dependencies, see
https://flutter.io/assets-and-images/#from-packages
To add custom fonts to your application, add a fonts section here,
in this "flutter" section. Each entry in this list should have a
"family" key with the font family name, and a "fonts" key with a
list giving the asset and other descriptors for the font. For
example:

For details regarding fonts from package dependencies,
see https://flutter.io/custom-fonts/#from-packages

Anyway, I have moved onto other example code in the Cookbook section. No other issues found. pubspec.yaml file not changed. I have just been "cutting and pasting" the example code into main.dart


Publish H1 Report on ENS forum

Report on achievements from the first semester to the ENS DAO.

https://www.notion.so/blockful/H1-Report-4c8ee204e98346fb8b8c080c77788b63

https://discuss.ens.domains/t/blockful-service-provider-reports/19553


Client to Client (not resource) token validation

Hello,
Not really a problem here, just checking on preferred approach/best practices.
My scenario:
A suite of microservice apis that perform various duties (store data, send email, etc) which are secured with oauth/oidc via identityserver4.
Client apps sit on top of these apis and authenticate users through identityserver, and pass the resulting access token along when interfacing with the microservice apis mentioned above.
If a client appA needs to call an api on client appB, we cannot use the introspection endpoint to validate the access token because client appB is a client, NOT a resource, and thus doesn't have a resource scope to authenticate with the introspection endpoint.
Instead, our approach is to use the access token to hit the userinfo endpoint.  if we get claims back we're all good, if not, 401.
Is this the expected approach or are we headed down the wrong path?
Thanks much!

Instead, our approach is to use the access token to hit the userinfo endpoint. if we get claims back we're all good, if not, 401.

The userinfo endpoint is def not suitable for token validation.
You basically lost me at this sentence:

If a client appA needs to call an api on client appB

There are clients and there are APIs - they are distinct entities, so the above sentence does not make sense to me. Can you elaborate? Also - why introspection?

The concept you are describing requires some central "anchor point" to assure trust among the providers. Enabling these "clients" as openid constructs requires that each of them get user consent to share information. Since you are AFAICT inside an enterprise that coerces employees and contractors to sign away their rights, the IdP could infer consent, but now you're outside the scope of OpenID.
The idea that could help is that of a "trust framework". I have written this about trust frameworks, but it is not yet at the point of being a finished spec, just a set of requirements.  https://wiki.idesg.org/wiki/index.php?title=Trust_Framework_Membership_Validation

I guess I never will understand why people try to use an OpenID inside a closed system.

I'm a coworker of @engenb .  We built a Platform which provides an IdSrv and other services.  We are building SPA applications on the Platform, but trying to treat the Platform as external and generic so we can create unrelated applications that use the same Platform.  The SPA applications want to use Implicit Flow with the Platform's IdSrv.  Each SPA has it's own Client, so a client ID and secret.  The SPA is not a resource so it does not have it's own scope or scope secret, and therefore cannot use the introspection endpoint.
Right now, the SPA application uses cookie authentication for the browser and does not have the ability to verify the reference token.  However, we still need the reference token to use Platform services like sending email, which uses the subject claim to identify the user.  We're trying to avoid managing multiple instances of IdSrv by only using the Platform IdSrv for every SPA application.  We also want to add some micro-services that sit between the SPA and the Platform, which would additionally need to verify the reference token.
Our first shot is using the User Info endpoint to verify that the reference token is active and cross-check some claims.  It sounds like that is not the way to go.  Is there another feasible solution?  Our next path might be adding the SPAs as resources in the Platform IdSrv.

I really think you need a bit of consulting to get your base architecture right. The issue tracker is not the right place for that.
https://identityserver4.readthedocs.io/en/release/intro/support.html

Each SPA has it's own Client, so a client ID and secret.
SPAs use implicit flow - this implies no client secret

Right now, the SPA application uses cookie authentication
that's the next problem

we still need the reference token
why reference tokens?

Our first shot is using the User Info endpoint to verify that the reference token
that's wrong

Our next path might be adding the SPAs
that also doesn't make sense to me.


Allow to define conditional groups with query parameter values as null

Requirement: In a particular end point for different queries parameter we want to have different throttling policies. Example: /resource/1?view=selective should be 5 TPS and /resource/1?CountOnly =true should be 10 TPS and /resource/1?view=selective&CountOnly=true should be 15 TPS.
This is the answer I got from Harsha on how to do this.
his requirement can be achieved using advanced policies as well as custom policies. Custom policy approach may comple for him. When defining advance throttle policy, we currently don't have a way to check parameter is empty or not. Hence he will need some modification to siddhi query generated by API Manager until we provide that it. Basically he needs to create three condition groups as below
Conditional Group 1 : Add query parameter condition with view=selective and another query parameter with CountOnly = null which later need to edited in siddhi  query to check is Null with limit 5
Conditional Group 2 : Add query parameter condition with CountOnly=trye and another query parameter with view = null which later need to edited in siddhi  query to check is Null with limit 10
Add query parameter condition with view=selective and another query parameter with CountOnly = true which later need to edited in siddhi  query to check is Null with limit 15
But, at the moment we cannot define a conditional group to check if query params are null. We need this support in api cloud.

By @amilam
From: https://wso2.org/jira/projects/APIMANAGER/issues/APIMANAGER-5401

Closing Since the issue has been stale for a long time.


Handle null site resolvers.

see issue: https://github.com/chapter-three/next-drupal/issues/517

Fixed by #481


Implement requirement rule CPMR0023 - Tags Missing

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

GitHub Release
Chocolatey Package

Your GitReleaseManager bot :package: :rocket:


Remove long-renamed deprecated classes and interfaces

We should remove several old classes and interfaces that were renamed years ago, e.g. RepositoryBase (which is now AbstractRepository).

"Perfection is achieved not when there is nothing more to add, but when there is nothing left to take away"  (Antoine de Saint-Exupéry)


Cursor pointer and setLayoutProperty on spider markers

What I need to do is to have 3 different marker states - default, active (when user clicks on marker) and visited (when user clicked on marker at least once, but it's not current clicked marker). I did manage to make it work for regular markers, which aren't grouped with spiderfy.
So this is my setup:
Unclustered markers; inside layout prop mapImg is my custom property that holds image name, and cluster is default (fallback) image for case mapImg is invalid.
map.value.addLayer({
    id: ids.marker,
    type: "symbol",
    source: sourceName,
    filter: ["!", ["has", "point_count"]],
    layout: {
      "icon-image": [
        "case",
        ["!=", ["get", "mapImg"], null],
        ["get", "mapImg"],
        "cluster",
      ],
      "icon-ignore-placement": true,
    },
    paint: {
      "text-color": "#000",
    },
  });

Clusters
map.value.addLayer({
    id: ids.cluster,
    type: "symbol",
    source: sourceName,
    filter: ["has", "point_count"],
    layout: {
      "icon-image": "cluster",
      "icon-allow-overlap": true,
    },
  });

  map.value.addLayer({
    id: "counters",
    type: "symbol",
    source: sourceName,
    filter: ["has", "point_count"],
    layout: {
      "text-field": ["get", "point_count_abbreviated"],
      "text-size": 14,
      "text-allow-overlap": true,
    },
    paint: {
      "text-color": "#000",
    },
  });

Spiderfy
spiderfy.value = new Spiderfy(map.value, {
    onLeafClick: (f) => handleMarkerClick(f),
    minZoomLevel: 10,
    zoomIncrement: 2,
    closeOnLeafClick: false,
    spiderLegsAreHidden: false,
    spiderLegsWidth: 2,
    spiderLegsColor: "#000",
    spiderLeavesLayout: {
      "icon-image": [
        "case",
        ["!=", ["get", "mapImg"], null],
        ["get", "mapImg"],
        "cluster",
      ],
      "icon-ignore-placement": true,
    },
    spiderLeavesPaint: {
      "text-color": "#000",
    },
    maxLeaves: 6,
    renderMethod: "flat",
  });

  spiderfy.value.applyTo(ids.cluster);

I use handleMarkerClick function for both leaf click and marker click and inside I have this command that changes my icon on the fly. So when I click on marker it will switch from default marker to the active marker which has different visual:
map.value.setLayoutProperty(ids.marker, "icon-image", [
    "match",
    ["id"],
    id, // unique id from marker properties
    'active-marker', // this is dynamically generated in my code, but hardcoded for example
    ["get", "mapImg"], // fallback to my custom mapImg icon to keep original icon for other markers
  ]);

And hover is handled here:
map.value.on("mouseenter", [ids.cluster, ids.marker], (e) => {
  map.value.getCanvas().style.cursor = "pointer";
});

map.value.on("mouseleave", [ids.cluster, ids.marker], (e) => {
  map.value.getCanvas().style.cursor = "";
});

I can't find in docs how to do the same with spiderfy markers, since it doesn't have unique ID as markers - so clicking on marker to update it's icon image and to set cursor pointer when mouse is over spiderfy markers.
I am using mapbox. Props with .value is because I use vue, that's not important for the functionality.

Thanks for the feedback, I had some buzzy weeks so my apologies for the late reply.
The major shortcoming from this plugin right now is that every single spiderfied marker is rendered in a separate layer (relevant code). At the time it seemed like the easiest implementation but it does have some shortcomings. This means it's not very easy to solve your problems.
Since you use maxLeaves: 6 in your example you could ugly solve the cursor problem with this:
map.on("mouseenter",
[
  ids.cluster,
  `${ids.cluster}-spiderfy-leaf0`,
  `${ids.cluster}-spiderfy-leaf1`,
  `${ids.cluster}-spiderfy-leaf2`,
  `${ids.cluster}-spiderfy-leaf3`,
  `${ids.cluster}-spiderfy-leaf4`,
  `${ids.cluster}-spiderfy-leaf5`,
], (e) => {
  map.getCanvas().style.cursor = "pointer";
});

map.on("mouseleave",
[
  ids.cluster,
  `${ids.cluster}-spiderfy-leaf0`,
  `${ids.cluster}-spiderfy-leaf1`,
  `${ids.cluster}-spiderfy-leaf2`,
  `${ids.cluster}-spiderfy-leaf3`,
  `${ids.cluster}-spiderfy-leaf4`,
  `${ids.cluster}-spiderfy-leaf5`,
], (e) => {
  map.getCanvas().style.cursor = "";
});

For the setLayoutProperty issue you might be able to do something similar. Alternatively you could update a source data value remembering which one was clicked using source.setData. It may cause a flicker so not ideal either.
The real solution to your problems is fixing that shortcoming of this plugin if I find the time at some point. Pull requests are always welcome!

Thank you for such detailed answer. We made the solution making parent pin active, while spiderfy pins remain as is. Since the client will try to put as little same coordinates as they can, they are satisfied with the current implementation. We end up using more than 6, now the minimum we set is 50.
But still, if I find some free time, I will try to create PR that would solve this issue.


Runtime package is not imported

Runtime package used isn't imported in runtime.go file
Please fix it as fast as you can :)

Runtime package used isn't imported in reflect2.go

Also running into this issue

https://github.com/modern-go/reflect2/pull/15
Thanks for the PR fixing this @aosting
Hope it gets merged soon

Also having this issue, hoping it gets fixed asap 🙏

Also waiting for the merge.

sorry about that


Build error after removal of the plugin (duplicate entry: com/google/android/gms/internal/zzanq.class)

cordova-plugin-firebase 0.1.22
Just removing this plugin and trying to rebuild consistently throws this error when trying to build afterwards.
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':transformClassesWithJarMergingForArmv7Debug'.
> com.android.build.api.transform.TransformException: java.util.zip.ZipException: duplicate entry: com/google/android/gms/internal/zzanq.class

try a clean build (remove platforms folder)


Plug chinese segmenter

SEGMENT_BANK is missing for chinese. Needs to plug the old TermSuite 1.5 chinese segmenter.
Exception in thread "main" eu.project.ttc.engines.desc.TermSuitePipelineException: java.lang.NullPointerException: Failed to load resource SEGMENT_BANK. Url is null.
    at eu.project.ttc.tools.TermSuitePipeline.aeWordTokenizer(TermSuitePipeline.java:676)
    at eu.project.ttc.api.TerminoExtractor.execute(TerminoExtractor.java:307)
    at org.ttc.project.builder.TerminoExtractorLauncher.fromTxtCorpus(TerminoExtractorLauncher.java:58)
    at org.ttc.project.builder.TerminoExtractorLauncher.main(TerminoExtractorLauncher.java:36)
Caused by: java.lang.NullPointerException: Failed to load resource SEGMENT_BANK. Url is null.
    at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:251)
    at eu.project.ttc.tools.TermSuiteResource.checkUrl(TermSuiteResource.java:114)
    at eu.project.ttc.tools.TermSuiteResource.fromClasspath(TermSuiteResource.java:135)
    at eu.project.ttc.tools.TermSuitePipeline.getResUrl(TermSuitePipeline.java:771)
    at eu.project.ttc.tools.TermSuitePipeline.aeWordTokenizer(TermSuitePipeline.java:664)
    ... 3 more

Fixed, but linguistic resources in chinese are still poor and the output of terminology extraction in chinese is polluted.
For example, current top-10 terms extracted from wind energy corpus are:
"n: ないので",
"nn: _ vicinity",
"nn: _ ansi",
"nn: slope _",
"n: 動作",
"n: economic",
"nn: gpsprom watchdog",
"nn: _ うものです",
"nn: _ ジョイントボルト",
"n: srppq"


"If you installed via Marketplace, the color scheme can be changed directly in the theme options." - Location?

Hi, could you please explain where to change the theme settings, when the theme is installed through spicetify marketplace?
I Can't find it in the spicetify/spotify client.

you need to click the user avatar on top with will open a menu with better bloom settings option just select it and change settings
https://discord.com/channels/1242343326175330396/1245652053867630703/1249014853935431731
i will add them in the readme after v2 release

here it will show better bloom settings for you

there is no built in theme settings if you are looking to change the colors you can do it by editing the config file config-xpui.ini
run this to open the folderspicetify config-dir

Alright thank you very much for your quick reply. I was a bit confused, because this is in your description:

Alright thank you very much for your quick reply. I was a bit confused, because this is in your description:

ohh soory it was ai gen

you can choose in this menu


remove external streaming job from sql project context menu

This removes the "Add External Streaming Job" command from the sql project context menu to help declutter the long list of commands. External streaming jobs can still be added by using "Add Item..."
before:

after:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/dialog/configurePython/configurePathPage.ts
1
57.02%

extensions/machine-learning/src/common/processService.ts
2
81.48%

extensions/notebook/src/common/ports.ts
4
74.14%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
28641

Relevant Lines:
63290

💛  - Coveralls

is this requested by PM?

is this requested by PM?

I didn't check with Drew since he's on vacation, but we made a similar decision to remove the edge project template because of the lack of usage. "Add External Streaming Job" has only been used 4 times in the last month and I'm pretty sure those are from our internal testing.
A customer recently shared a video of them using the sql projects extension and it took him a few seconds to find the command he was looking for in the project context menu, which is what motivated this change.


ru language

which pre-trained model can work for russian language? i want get only vectors

of course i can use bert aka multi-language but they work very bad  in my mind for ru

I think XLM models is better than mBERT. There are XLM models for 17 and 100 languages including ru.

cool
can u help me where i can download pre-trained xlm from ru because here i can't find models for RU?

i think here i can get pre-trained

@vtrokhymenko Yes, it's here


test: add tests for graphql-apollo-subscriptions example

Thanks! Hopefully we can add a cancellation API at some point. Although stopping a subscription might be a bit of an edge case, it requires too much ceremony right now.

Thanks! Hopefully we can add a cancellation API at some point. Although stopping a subscription might be a bit of an edge case, it requires too much ceremony right now.

What do you mean by cancellation? Calling cancel on the abortcontroller will cancel the subscription. What's missing?

Thanks! Hopefully we can add a cancellation API at some point. Although stopping a subscription might be a bit of an edge case, it requires too much ceremony right now.

What do you mean by cancellation? Calling cancel on the abortcontroller will cancel the subscription. What's missing?

Not having to use the AbortController explicitly would be nicer, IMHO.

Thanks! Hopefully we can add a cancellation API at some point. Although stopping a subscription might be a bit of an edge case, it requires too much ceremony right now.

What do you mean by cancellation? Calling cancel on the abortcontroller will cancel the subscription. What's missing?

Not having to use the AbortController explicitly would be nicer, IMHO.

extend the subscription with a stop() func and make it implicit
but not sure how much value is in it
most clients, e.g. SWR and React Query hide this completely from the user

@jensneuse Something I forgot to mention: this test only runs on Node 18 because we do not support subscriptions with earlier Node versions, but it can be added. Thing is: is it worth it or should we just tell users that subscriptions in Node require version 18+?

@jensneuse Something I forgot to mention: this test only runs on Node 18 because we do not support subscriptions with earlier Node versions, but it can be added. Thing is: is it worth it or should we just tell users that subscriptions in Node require version 18+?

I'd say go with the easier option.

@jensneuse Something I forgot to mention: this test only runs on Node 18 because we do not support subscriptions with earlier Node versions, but it can be added. Thing is: is it worth it or should we just tell users that subscriptions in Node require version 18+?

I'd say go with the easier option.

Node 18+ only is definitely the easier, no code required.


Which module to use for Google Analytics

I have a new NUXT application that needs to have Google Analytics integrated within the application. It appears that there is a very stable analytics-module (https://www.npmjs.com/package/@nuxtjs/google-analytics), but it is utilizing the analytics.js script that Google no longer recommends. Google currently recommends using gtag.js script for integrating with Google Analytics. The only package in the NUXTJS community is the google-gtag (https://www.npmjs.com/package/@nuxtjs/google-gtag) module. Is this the currently recommended module to use when integrating with Google Analytics? It appears that this package is not downloaded that frequently or released that often compared to analytics-module. Any help in the right direction would be appreciated.

same !

same !

2021 updated question
There are 3 libraries
https://github.com/nuxt-community/google-gtag-module/
https://github.com/nuxt-community/gtm-module
https://github.com/nuxt-community/google-analytics-module
I see google-analytics-modules is using vue-analytics which is deprecated, that leaves with 2 more
which one to use if

I want to add universal analytics
I want to add GA4
I want to support both


Issue #76

issue #76

Update expiration dates

The copyright dates on file headers get updated as they are edited.
If the file hasn't been touched we leave the file copyright as is.
We learned this practice from the Bitcoin-Core project.

The copyright dates on file headers get updated as they are edited.
If the file hasn't been touched we leave the file copyright as is.
We learned this practice from the Bitcoin-Core project.


test: fix PostgreSQL with non-UTC timezone, fixes #203

When we use kallax against a PostgreSQL with a timezone other than UTC, times will be returned using that timezone. So, on tests, we just convert returned times to UTC before comparison.

This fixes #203.
Initially I thought it would be a problem in lib/pq or Go, since a lot of operations with locations are system-dependent, including parsing and formatting. But the only way I manated to reproduce this is changing PostgreSQL's default timezone:
docker run --name postgres -e POSTGRES_DB=testing -e POSTGRES_USER=testing -e POSTGRES_PASSWORD=testing  -p 5432:5432 -d postgres -c 'timezone=GMT'

Here's an example build on macOS with the fix. https://travis-ci.org/smola/go-kallax/jobs/392798180

When we use kallax against a PostgreSQL with a timezone other than UTC, times will be returned using that timezone

It seems consistent to expect database timestamps being returned in database's timezone, so fixing the tests seems ok,


[Key Vault - Keys] Merge subscription configurations failing in nightly runs

Key Vault - Keys nightly test runs are failing with:

Error message:
Get-Content: /mnt/vss/_work/_temp/bac3d6dc-e250-4e7f-82f2-21d5fdbe2190.ps1:37
Line |
37 |      $subConfig = Get-Content $file | ConvertFrom-Json -AsHashtable
|                               ~~~~~
| Cannot bind argument to parameter 'Path' because it is an empty string.
##[error]PowerShell exited with code '1'.

For more details check here:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3917224&view=results

Haven't seen this particular failure for a while.


39001 PS2 and Brunswick 1 - Red Screen

I'm trying to get into the exploit screen on v1.4
I'm on a SCPH-39001 with Playstation driver 1.10 listed in the version info screen.
I downloaded v1.4 from the github release page, and read the readme. I loaded the BASLUS-00571 file from the entrypoints folder onto my FAT32 USB drive, and used Launchelf to copy the file over to my PS1 memory card. Loading the save from the Brunswick game lands me on a solid red screen. I understand this is a fatal error?
I've re-copied the file over a couple more times, and tried the "psuPaste" function in LaunchElf with no success. Please let me know if any more info is needed. Thanks!

Thanks for the report.
At the moment seems broken and it indeed randomly fails. On no$psx with a 4.4 BIOS from a SCPH-102 seems to have a failure rate of almost 50%. It's something I've observed also myself while working on #89.
Will update you as soon as I figure out something.

Glad to help anyway needed.
Romans 8:28
Zachary Drummond

Tested on a second PS2 model SCPH-30001 R according to the version screen. It's missing the label on the back, so I can't check that.
Same game. Same save. Same result.

tonyhax-v1.4.1b.zip
Can you please try with this version? Only the game's save has been updated, you don't really need to update the tonyhax save.
You can enjoy my explanation/rant on why this bug occured here: https://github.com/socram8888/tonyhax/blob/master/entrypoints/entry.S#L45-L57

Tried the BASLUS-00571 file in the entrypoints folder of the v1.4.1b folder. Same red screen.

I am going to close this since I cannot reproduce it (it's working fine on no$psx, regardless of how many times I reboot) and I've got reports from another user (see #89) that it's also working for him.

What would you suggest I try next? I still can't get it to work.

Can you try reading back the save files and the tonyhax file and ensure they haven't gotten corrupted? Certain versions of uLaunchELF (I am not sure if it was 4.42 or 4.43) can cause data corruption when copying from a USB drive (and has in the past to at least one GBAtemp user).

I copied the save from the memory card back to my USB drive. Checked the results.

I'm using LaunchELF v4.43a from https://github.com/ps2homebrew/wLaunchELF/releases/tag/190107

Wait a second. I think I may have misunderstood the instructions. Do I need to place both the save I just tested for corruption and the BESLEM-99999TONYHAX file on the memory card?

Yup. I'm dumb. haha Thanks for the hack, man! Got it working!


Optimisation for delegate message forwarding

Looks like forwardInvocation: is an overhead for calling delegate methods in this case. Only target should be changed, arguments and selector are unmodified.

Hey


Fix the Python CMD

:star: PR built and available in a preview environment itamaro-python-http-pr-1 here


Add an endpoint to calculate Inflation rate

Shall I take this up?

Go ahead and make a PR :))


Off-the-rack night theme

We’re very likely to ship with a “standard” theme for night mode. There are a couple of issues we must deal with, especially as it is a11y-related. In other words, we can probably make a few compromises but authors will have to make the most.
In the spirit of issue #5, I’d like to kickstart the conversation way ahead of time.
The current situation
The sad truth is that nobody knows how to deal with that.

Invert/High contrast modes at the Operating System Level use a global filter (Windows | Mac).
MacOS/iOS are not a global setting but a list of independent settings e.g. you can invert colors and use monochrome, and then add high contrast.
Google has had a hard time implementing high contrast mode in Chrome.
Reader Modes usually get rid of authors’ CSS so they don’t even have to deal with the related issues.
Reading Systems sometimes use a global filter as well, sometimes try to do it via CSS.
Kindle is normalizing colors and enforcing sufficient contrast ratio when converting EPUB files to KFX (+ font-family and font-size by the way)

Major issues

Night mode ≠ white on black: it can be white on dark grey, it can be #FCFCFC on #121212, etc.
Images: when there are a lot and they have a white background, night modes becomes useless. And we don’t know which images to invert, we can’t even rely on the format. There is some stuff I can manage with, say, 95% confidence (epub:type title page + image) though.
SVG: even more complex than images.
Backgrounds: figures, asides, etc. are complex to manage, especially as there are CSS hacks out there (using a linear-gradient + -webkit-text-fill-color for text). There’s also text on background images which we’d better not override if we don’t override background-image (JS territory).
Borders: what do do of black borders, borders for which the color doesn’t provide sufficient contrast, etc? And we can’t check the color for the border’s element in pure CSS, that’s JS territory.
Colors: contrast is an issue, especially when night mode is used as an a11y tool. Once again, the -webkit-text-fill-color hack might be used to enforce links’ or headings’ color.
Gaiji i.e. a character that is valid in a given writing system, but that is not included in the present font and/or the present character encoding, and which therefore is displayed using an image, which we must invert at any cost necessary (and we can’t do p > img). iBooks had to create a metadata for that.
Performance: if we have to check every element in the DOM, it will kill performance. Once again, this is something for which I wish we had Houdini (at 97% support).

Design
At first sight:

Will override background-color and border-color but not background-image;
Will override color but not -webkit-text-fill-color;
Won’t invert images, and this is why I’m pushing for a mechanism so that authors can signal some images must be inverted. It could be ARIA territory (cc @rdeltour) since it also impacts high-contrast mode in nasty ways (cf. google issues in Chrome implem);
SVG is a tough one, especially if the background is transparent and the author is not using currentColor for fill (can check some SVG attributes and the colors but it will result in insane CSS selectors);
We must find something for gaiji; the fact iBooks and Kindle bothered implementing specific mechanisms to deal with it kind of show this is critical;
Performance is @danielweck’s territory. I can find clever ways to deal with some overrides but for the complex ones, we can’t do without computed styles.

Possible issues this will create

RSs stand in the first line so they will take the heat if users complain, especially if other RSs manage night mode the way the user wishes;
Since there is no real “standard” in managing night mode, I can’t even tell what the users’ expectations are;
Some authors will probably complain about color and stuff (but it’s an accessibility issue);
Some others will point out alternate stylesheets exist (but it doesn’t solve the issues for the millions of files out there).

Please note this is related to personalization:

Personalization Semantics
Draft of user settings, a section which has not beed addressed yet
PWG Personalization Task Force Minutes (July 27)

For gaiji, we’ll take the Japanese EPUB profile as a reference: http://ebpaj.jp/cmspage/wp/wp-content/uploads/2015/04/ebpaj_epub3guide_ver1.1.3-en-1.zip

An interesting idea from Wikipedia that we might implement as well, especially as it is one line of CSS: letting the user darken all images in night mode (sorry, screenshot in french)

In that case, having another switch for inverting all images could be offered as well, it may be unperfected but for some books, it could solve a lot of issues (headings or table as images, B&W illustrations, etc.).

Won’t invert images

I think it's a safe approach. In the vast majority of cases, images should not be inverted.

and this is why I’m pushing for a mechanism so that authors can signal some images must be inverted. It could be ARIA territory (cc @rdeltour) since it also impacts high-contrast mode in nasty ways (cf. google issues in Chrome implem);

An author-defined mechanism might be interesting. The right venue to ask for feedback/interest could be WICG.
Another approach, as you state in you last comment, is to have a user preference. But "coloring" and "inverting" should be kept separated IMO. People could want the image to be darkened, or be applied a sepia filter, but still don't want to be inverted in a full night mode.

But "coloring" and "inverting" should be kept separated IMO. People could want the image to be darkened, or be applied a sepia filter, but still don't want to be inverted in a full night mode.

Sorry if it was unclear, darken and invert would be 2 different switches, obviously (this is how I implemented them yesterday BTW).
For the record:

darken uses filter: brightness()
invert uses filter: invert()

As for sepia, we currently apply a mix-blend-mode by default so that white backgrounds are sepia—that may change in the future. Unfortunately filter: sepia() has way too much impact on colors when it comes to photos.
Thanks for the feedback!

As an update, I did a quick round of testing today and night mode is quite alright. Darken and invert options definitely help, they can improve comfort pretty dramatically in some illustrated-heavy publications.
Haven’t had the time to check everything yet but that’s a good start, we don’t have any major issue to deal with.

After further testing, we’d probably better create some kind of CSS inverted variable implementers can use.
The assumption is:

we have styles which sole purpose is applying the night mode (their reusability is null at the moment, and that doesn’t philosophically fit that well in our design approach);
users are currently able to override it (applying night mode then setting their own background and text color in advanced settings);
the night theme stylesheet could be used to manage all inverted modes (custom colors set by the user);
invert state can be interpolated from the custom background-color (it’s a range in rgba?);
theme variables could become user variables with a fallback;
themes would therefore become part of user settings, and would be appended on load (no extra stylesheet necessary).

Benefits for implementers:

night modes (plural) from the same stylesheet;
inverting images could use a dynamic factor (variable), based on the colors picked by the user (which means it can also be 0 to not invert images if needed);
sepia could be adopting this approach as well (for pastel backgrounds, which would be useful in terms of a11y);
extra flexibility.

On the other hand, we should probably start discussing a fallback mechanism for browsers which don’t support CSS variables. Although we’re not impacted with the reference implementations (iOS + Readium Desktop), some implementers will have to deal with IE11 and Edge 14 (web app).
I can open a dedicated issue now but feedback from implementers will come later, obviously (it needs practical implementations to see which options are the best ones).


⚠ GitLab Server has degraded performance

In 7dd687a, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6574 ms

Resolved: GitLab Server performance has improved in 8a305c1.


Simplifying Manim's color handling

Overview: What does this pull request change?
This PR improves the color handling by removing uses of the Colors class and replace usage of the Enums with color constants as specified in #2451 . At this point, a deprecation warning is raised when trying to use the Colors class using manim.utils.deprecation.
Some of the functions previously defined in color.py that has existing functionality in the colour module has also been replaced.
For instance:
rgb_to_hex() has been replaced by rgb2hex().

Motivation and Explanation: Why and how do your changes improve the library?

The overall goal is to simplify the use of color by standardizing the approach as stated in #2451 . By deprecating the use of the Enums and providing a way of using the constants in a similar way, we're creating a standard for how colors should be used.
Links to added or changed documentation pages

Further Information and Comments

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

Looks like these changes broke some part of BarChart, as can be seen from the docbuild. Can you fix it?

I've converted this into a draft for now. Feel free to unconvert once the docbuild passes, and let us know if you need any help.

Hello, are you planning to continue working on this?

I think i will go about my way doing this in a few days


How does dashdash work in 2.x?

i want to have a command line as:
exe -g -- filename1 filename2

So I tried
 exe -g -- filename1

class Opts a
  [Option('g')]
  bool Generate {get; set;}
  [Value(0)]
  string Filename {get; set;}
}

var p = new Parser(settings => settings.EnableDashDash = true);
p.ParseArguments<Opts>(args);

I get the error:

ERROR(S):
Error setting value to option 'unknownarguments': Property set method not
found.

So what's the correct way of doing this in 2.x?

To use double dash, you define options with longName, like
       [Option('g',"generate")]
         bool Generate {get; set;}

Then, to use
         exe    --generate  filename

No space between -- and the option
Filename is a free option without singlee /or double dash.
For more details, read the Wiki

Is DashDash not an option to enable the end of options delimiter? Or does command line always do that?
[Option('g')]
bool someFlag

[Value(0)]
string filename

Usage
exe -g -- -g

then -g is on and filename will also contain -g preventing it from being recognized as the option -g.
See https://www.in-ulm.de/~mascheck/bourne/set-.html

Is DashDash not an option to enable the end of options delimiter?

DashDash enable the end of options delimiter in the custom Parser as you did.
I wasn't thinking that you mean using DashDash  as an end of option :D
See this answer by @nemec  with example
I test your example with minor modification, and it's working with DashDash support
These are working
            -g -- -g
	-g -- filename1
	-g -- -filename1

Try it online

I thought it was broke due to https://github.com/commandlineparser/commandline/issues/156
I tried
exe -g -- expected unexpected

and got no error.
Since dashdash is working I can close this.


nvim-cmp stops working when having vim-dadbod-completion installed

I am using LunarVim (which again uses nvim-cmp), with Neovim 0.9.1. I am installing the plugin using Lazy.
Just by having this plugin installed I lose all my code autocompletion. Things looks right when I do LspInfo (all the language servers are connected).
There is nothing different in the LspLog when having the plugin installed or not.
My config is basically:
lvim.plugins = {
  {
    "kristijanhusak/vim-dadbod-ui",
    dependencies = { "tpope/vim-dadbod", "kristijanhusak/vim-dadbod-completion"}
  }
}

vim.api.nvim_create_autocmd(
  "FileType",
  { pattern = { "dbui" }, command = ":nmap <buffer> l <Plug>(DBUI_SelectLineVsplit)" }
)

vim.api.nvim_create_autocmd("FileType", {
  pattern = { "sql", "mysql", "plsql" },
  command = ":lua require('cmp').setup.buffer({ sources = {{ name = 'vim-dadbod-completion' }} })",
})

I have also had this issue.
return {
  {
    "tpope/vim-dadbod",
    dependencies = {
      "kristijanhusak/vim-dadbod-ui",
    },
  },
  {
    "kristijanhusak/vim-dadbod-ui",
  },
  -- {
  --   "kristijanhusak/vim-dadbod-completion",
  -- },
}

Commenting and uncommenting this line toggles the issue.
I am using LazyVim and my nvim-cmp config is completely default apart from a slight style choice.
return {
  {
    "hrsh7th/nvim-cmp",
    lazy = true,
    opts = function(_, opts)
      local cmp = require("cmp")
      opts.completion = {
        completeopt = "menu,menuone,noinsert",
      }
      opts.window = {
        completion = cmp.config.window.bordered(),
        documentation = cmp.config.window.bordered(),
      }
      return opts
    end,
  },
}

I would love this behavior inside of SQL editors but this just disables it globally for me.
I notice a strange behavior where it seems that nvim-cmp loading gets placed "behind" dadbod-completion when it is active.
-- `:Lazy profile` with dadbod-completion
      ➜  after 5.14ms
        ★   after/plugin/vim_dadbod_completion.lua 5.06ms
          ‒   nvim-cmp 4.53ms
            ●   crates.nvim 2.32ms
            ●   copilot-cmp 0.53ms
              ➜   copilot.lua 0.46ms
                ★   copilot.lua/plugin/copilot.lua 0.03ms
            ●   cmp-nvim-lsp 0.04ms
            ●   cmp-buffer 0.08ms
            ●   cmp-path 0.06ms
            ●   cmp_luasnip 0.07ms
            ●   nvim-cmp/plugin/cmp.lua 0.26ms
        ★   after/plugin/vim_dadbod_completion.vim 0.02ms

and removed from "InsertEnter" where I normally see it.
I'm not sure if the extension is taking ownership and overwriting the "event" somewhere. But something along those lines is what the behavior seems to suggest.
-- `:Lazy profile` without dadbod-completion
InsertEnter 21.42ms
      ➜   nvim-cmp 21.31ms
        ★   crates.nvim 10.62ms
          ‒   after/plugin/cmp_crates.lua 0.69ms
        ★   copilot-cmp 3ms
          ‒   copilot.lua 2.67ms
            ●   copilot.lua/plugin/copilot.lua 0.18ms
        ★   cmp-buffer 5.08ms
          ‒   after/plugin/cmp_buffer.lua 4.95ms
        ★   cmp-path 0.57ms
          ‒   after/plugin/cmp_path.lua 0.47ms
        ★   cmp_luasnip 0.66ms
          ‒   after/plugin/cmp_luasnip.lua 0.6ms
        ★   nvim-cmp/plugin/cmp.lua 0.27ms

I noticed that my dependencies might have caused some confusion. dadbod requiring ui instead of the other way around.
I will invert these tomorrow and see if that fixes the issue on my end.
I tried reviewing all of the places where this extension interacts with cmp and nothing stood out as being a potential cause.
I will let you know if this corrects the behavior or if I find anything else.

I'm also using Lazy. You can try my setup. Here's how I set up dadbod related stuff: https://github.com/kristijanhusak/neovim-config/blob/master/nvim/lua/partials/plugins/db.lua
And this is where I set up cmp for it: https://github.com/kristijanhusak/neovim-config/blob/master/nvim/lua/partials/plugins/completion.lua#L93-L106

Thanks! It was not having the "cmd" line in my case. Apart from changing vsnip to luasnip copy pasting the second worked like a charm as well.
Reference solution for anyone else using LazyVim that finds this issue.
Thanks for the help and the great applications! @kristijanhusak
Keep up the great work!
-- plugins/cmp.lua
return {
  {
    "hrsh7th/nvim-cmp",
    lazy = true,
    opts = function(_, opts)
      local cmp = require("cmp")
      opts.completion = {
        completeopt = "menu,menuone,noinsert",
      }
      opts.window = {
        completion = cmp.config.window.bordered(),
        documentation = cmp.config.window.bordered(),
      }
      return opts
    end,
  },
}

-- plugins/dadbod.lua
return {
  {
    "kristijanhusak/vim-dadbod-ui",
    dependencies = {
      { "tpope/vim-dadbod", lazy = true },
      { "kristijanhusak/vim-dadbod-completion", lazy = true, ft = { "sql", "mysql", "plsql" } },
    },
    cmd = { "DBUI", "DBUIFindBuffer" },
  },
}

@kristijanhusak Thank you very much for the help. For me the solution was a little weird. I kept my code just the same, but the only thing I added was local cmp = require("cmp") (could be before or after the autocommand). That is, my code is this at the moment:
vim.api.nvim_create_autocmd("FileType", {
	pattern = { "sql", "mysql", "plsql" },
	command = ":lua require('cmp').setup.buffer({ sources = {{ name = 'vim-dadbod-completion' }} })",
})

local cmp = require("cmp")

I don't understand why requiring cmp as a local config variable solves anything. Doing this alone for example won't work:
vim.api.nvim_create_autocmd("FileType", {
  pattern = { "sql", "mysql", "plsql" },
  callback = function()
    require("cmp").setup.buffer({
      sources = {
        { name = "vim-dadbod-completion" },
        { name = "buffer" },
        { name = "luasnip" },
      },
    })
  end,
})

To make it work I MUST require cmp at some point in the code, for example:
vim.api.nvim_create_autocmd("FileType", {
  pattern = { "sql", "mysql", "plsql" },
  callback = function()
    require("cmp").setup.buffer({
      sources = {
        { name = "vim-dadbod-completion" },
        { name = "buffer" },
        { name = "luasnip" },
      },
    })
  end,
})

require("cmp") -- must have this

Maybe it is because LunarVim is doing something funny with cmp? I don't feel it is because of that, but at least this fixed my issue.
Either way, thanks for the great plugins @kristijanhusak. They work really well.

I'm closing this since it seems that everyone managed to solve the configuration issue.

Does anyone here know if it's possible to extend the setup of the current buffer rather than hardcode it with cmp.setup.buffer()?
For example, with global cmp sources, you can use something like this:
local cmp =require("cmp")
local current_config = cmp.get_config()
local dadbod_completion_source = { name = "vim-dadbod-completion" }

table.insert(current_cmp_config.sources, dadbod_completion_source)

But that will affect all buffers, not only the current (sql) buffer.

@fredrikaverpil this is how I set up dadbod completion only for sql buffers:
https://github.com/kristijanhusak/neovim-config/blob/master/nvim/lua/partials/plugins/completion.lua#L93C1-L106
I'm not sure how could you extend. Maybe extract the defaults you want to extend into separate variable and just use them both for SQL buffers.

@kristijanhusak hey 👋  yeah that might be a good solution.
Looks like this works: https://github.com/fredrikaverpil/dotfiles/blob/eef23ab2c1101242eba24c3a4cb02de35074520f/nvim-lazyvim/lua/plugins/sql.lua#L41-L64

I just came across it, when naively adding completions with the config explained in the README. I'm trying @fredrikaverpil example config above
How do you feel about an update to the README to include one of the above solutions (ideally the simplest one), otherwise future users of nvim-cmp will see the same results...