support defaultDarkMode

Feature request
Right now, the default-them use the prefers-color-scheme via const darkStorage = useStorage('vuepress-color-scheme', 'auto'), so builder does not have explicit control on the default darkMode.
Description
support defaultMarkMode which should be auto, light or dark. Where auto is defined by perfers-color-scheme.
Proposed Solution
There should be a configuration defaultDarkMode so that builder can specify in the config.ts.

1, 'auto' do fit most situations (you are talking about edge case)
2. You should be able to achieve this by overide templateBuild option. (you have workaround)

How do we do that with templateBuild? I did a google and find nothing. Link?
On Tue, Apr 5, 2022 at 9:17 AM Mr.Hope @.***> wrote:

Closed #796 https://github.com/vuepress/vuepress-next/issues/796.
—
Reply to this email directly, view it on GitHub
https://github.com/vuepress/vuepress-next/issues/796#event-6372714452,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJYKUMLPKKHCPZ5L27DL3DVDRRR3ANCNFSM5STI4UHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

https://v2.vuepress.vuejs.org/reference/theme-api.html#templatebuild

Sorry, I meant how-to-use-templateBuild-to-specify-default-darkMode. And
the doc on template build does not have information on how to use it even
for other things. How do we know how to use it?
On Tue, Apr 5, 2022 at 9:22 AM Mr.Hope @.***> wrote:

https://v2.vuepress.vuejs.org/reference/theme-api.html#templatebuild
—
Reply to this email directly, view it on GitHub
https://github.com/vuepress/vuepress-next/issues/796#issuecomment-1088968653,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJYKUI4DTYGBWBVADNKHU3VDRSDRANCNFSM5STI4UHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Place the above template somewhere in your project, and add templateBuild option in your user config file, and setting its value to be the path of the file above

the doc on template build does not have information on how to use it even
for other things. How do we know how to use it?

Unless the docs is totally missing description or there are missing apis, this kind of problems should be regarded on your side.
We would like to give some simple hints on how you can achieve something, but we do not have reposonability impletement that for you just because you can not understand how to achieve that.
Just as a reminder, this is an open source project, which means we can ( and I will) ignore some personal cutomize help.

Also, if the above way is hacky for you, https://v2.vuepress.vuejs.org/reference/default-theme/extending.html can be another way.

First of all, I really appreciate your effort in helping around, and make
vuepress great option for creating content site (almost, I did find there
are things missing, like blog2). You are really kind on actions, and I
totally understand you have to maintain a clear boundary.
for  your next suggestion: you mean do something like:

but I do think both options requires significant increase in term of level
of understanding of how vuepress works (templateBuild is not mentioned here
https://v2.vuepress.vuejs.org/advanced/architecture.html#overview) in the
needed to control default dark mode. But then again, I think it is your
decision in terms of whether defaultdarkmode should be exposed in
accessible fashion like darkmode (should really be darkModeToggle if you
ask me) or not.
Again, thanks for your effort, really prompt, and understood covid is
hurting all of us. Hang in there.
Sean.
On Tue, Apr 5, 2022 at 9:39 AM Mr.Hope @.***> wrote:

the doc on template build does not have information on how to use it even
for other things. How do we know how to use it?
Unless the docs is totally missing description or there are missing apis,
this kind of problems should be regarded on your side.
We would like to give some simple hints on how you can achieve something,
but we do not have reposonability impletement that for you just because you
can not understand how to achieve that.
Just as a reminder, this is an open source project, which means we can (
and I will) ignore some personal cutomize help.
—
Reply to this email directly, view it on GitHub
https://github.com/vuepress/vuepress-next/issues/796#issuecomment-1089008415,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJYKUP3NMLLJSJGETJKJJTVDRUFDANCNFSM5STI4UHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

You may overide the darkmode toggle button using alias or theme extending, and setting a new default value, the understanding above is incorrrect.
For details you may check the hint in my theme here:

https://vuepress-theme-hope.github.io/v2/cookbook/advanced/replace.html#replace-components
https://vuepress-theme-hope.github.io/v2/cookbook/advanced/extend.html#theme-extending

many thanks, will come back to this tmr.
On Tue, Apr 5, 2022 at 8:51 PM Mr.Hope @.***> wrote:

You may overide the darkmode toggle button using alias or theme extending,
and setting a new default value, the understanding above is incorrrect.
For details you may check the hint in my theme here:

https://vuepress-theme-hope.github.io/v2/cookbook/advanced/replace.html#replace-components
https://vuepress-theme-hope.github.io/v2/cookbook/advanced/extend.html#theme-extending
—
Reply to this email directly, view it on GitHub
https://github.com/vuepress/vuepress-next/issues/796#issuecomment-1089766096,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJYKUMNI2KGGJVRLOUQCFTVDUC3DANCNFSM5STI4UHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

I agree with the OP @xiaoyunwu here. I am new to vuepress and find it is odd that there is no config option to default the theme to dark. The templateBuild approach shown here is way too complicated and cumbersome.


[release v1.26] Cherry pick: Add mount-cgroup init container to node ds

Description
Cherry-pick PR https://github.com/tigera/operator/pull/1957 to release v.1.26

For PR author

[ ] Tests for change.
[ ] If changing pkg/apis/, run make gen-files
[ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

I put the hold-merge label on this and will merge it after the upcoming EE related patch release, which has a calico-node that does not utilize this new mount yet.


text generation webui not a model?

Comment on mastodon: https://amok.recoil.org/@nick/110813522014804587

That’s a great page for comparing degrees of openness. I just wonder why https://github.com/oobabooga/text-generation-webui is included, as it isn’t a model in and of itself, it’s a set of runtime tools with a web ui?

Correct, shouldn't be there, might be mentioned as a relevant resource somewhere


If a key is pressed when UTM pauses, when it unpauses, the key would have been pressed repeatedly.

BEFORE SUBMITTING YOUR ISSUE, PLEASE LOOK AT THE PINNED ISSUES AND USE THE SEARCH FUNCTION TO MAKE SURE IT IS NOT ALREADY REPORTED. ALWAYS COMMENT ON AN EXISTING ISSUE INSTEAD OF MAKING A NEW ONE.
Describe the issue
Over the runtime for UTM, there are sometimes pauses. They usually only last for a few seconds. However, if a key was pressed when the pause happens, when it unpauses, the guest will show that that key was pressed repeatedly, proportional to the length of the pause.
Configuration

UTM Version: 4.2.5
macOS Version: 11
Mac Chip (Intel, M1, ...): M1

Crash log
Debug log
Upload VM

With the latest beta (4.3.2), I don't think this is a problem anymore. I'll reopen this issue if it reoccurs.


Possible copy/paste bug (or something) in _path.js?

Take a look at this line: https://github.com/svg/svgo/blob/4495a34f55560b2bc05cf837ba9dd01ca20cd479/plugins/_path.js#L755
and this line: https://github.com/svg/svgo/blob/4495a34f55560b2bc05cf837ba9dd01ca20cd479/plugins/_path.js#L769
I don't understand this logic... will either of these two if statements ever be true? Should it be using || instead of && or something?

Should be || of course.

I created a pull request here: #850


Use of new t = inlineTranslate() in useVisibleTask$

Hi! New version seems awesome
I got a warning when trying to use t in useVisibleTask$
When referencing "t" inside a different scope (useVisibleTask$), Qwik needs to serialize the value, however it is a function, which is not serializable.

What's the right way of translating in such context?

useVisibleTask$ is a different scope wrapped in a QRL: now inlineTranslate is a pure function, so you have to declare it in this new scope:
useVisibleTask$(() => {
  const t$ = inlineTranslate();
  console.log(t$('app.title'));
})

However, I find weird that you have to translate into useVisibleTask and not in the component: what is the use case?

I'm setting the zod error map like this:
export default component$(() => {
  const t = inlineTranslate();

  useVisibleTask$(() => {
    z.setErrorMap((issue, ctx) => {
      if (issue.code === ZodIssueCode.invalid_type) {
        return {
          message: t("forms.errors.required@@Requerido"),
        };
      }

      if (issue.code === ZodIssueCode.too_small && issue.type === "number") {
        return {
          message: t(
            "forms.errors.numberTooSmall@@Debe ser al menos {{minimum}}",

            {
              minimum: issue.minimum,
            },
          ),
        };
      }

      return { message: ctx.defaultError };
    });
  });

  return <Slot />;
});

I have already expressed my thoughts on translation with zod here: https://github.com/robisim74/qwik-speak/issues/61 (solution A)
I think the best and clearest solution is for the schema to do the schema, and translate into the jsx (runtimeAssets in this library  exist for this reason: to be able to handle dynamic keys such as server response errors or validation errors).
Then I can't figure out a component that only runs a useVisibleTask$ (which you should use with caution) and returns a slot, but that's just my problem.
That said, if you move const t = inlineTranslate(); into useVisibleTask$ it should work: certainly in production, maybe in dev it might not return the current translation, because useVisibleTask$ is executed eagerly in the browser, before the new inlining feature in dev mode can know the current locale (let me know if you experience this behavior).

Oh, I get it. Thanks!


[imap] trouble connecting to fastmail

#3

Can anyone reach out to me hoa at dejalu dot me and provides me account information that I can test with? Thanks!

fastmail has trial (limited) accounts with hard limits on sent emails. you can create one for free here https://www.fastmail.com/signup/

Maybe this one will fix it: https://github.com/MailCore/mailcore2/pull/1701
I'm double-checking.

I've tried adding the correct IMAP/SMTP during account setup, unfortunately that didn't solve the connection issue
--
Sent from iPhone
On 13 February 2018 at 06:40:25, Hoà V. DINH (notifications@github.com(mailto:notifications@github.com)) wrote:

Maybe this one will fix it: MailCore/mailcore2#1701(https://github.com/MailCore/mailcore2/pull/1701)
I'm double-checking.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub(https://github.com/dinhviethoa/dejalu/issues/11#issuecomment-365157070), or mute the thread(https://github.com/notifications/unsubscribe-auth/AACFpQ-wsY2QSuqmBtJltTahDuab5xgCks5tUSBIgaJpZM4RYO01).

I just checked build 192 and after removing and re-creating my fastmail account (without clicking on custom parameters button) it worked out of the box.

Tried again on build 192 with little success.
Worth noting is that I'm using a custom domain so email != username, which means that I have to use custom settings.
The custom settings screen verifies that my details are correct, and lets me create and account, but afterwards all I get is:

@oskarkrawczyk I'm also using a custom domain and it works fine.

Unfortunately no joy here.
If there's anything I can provide to pinpoint the issue, please let me know.

@oskarkrawczyk same for me

I think that even with a custom domain, DejaLu will detect it as being Fastmail.
Could you try just adding it and not going through the custom domain?

DejaLu can detect the provider, sure, but it cannot detect my actual username for SMTP/IMAP, which in FastMail is not the same as the email, so it's not possible to skip the Custom workflow.
Here's a screencast of the process (including the error about the wrong username as aforementioned): https://v.usetapes.com/TXQH6PwSrp

@oskarkrawczyk Can you try using your @fastmail.com address/username to create your account then use alias to set the email address you want to work with?
You can make an email alias the default email to use with an account.

@dinhviethoa That works as a workaround, thanks.
That said, I don't think this is the optimal way to set up a Fastmail account :-)


How to accept ssl certificates in Chrome and Firefox?

Where to add capabilities to accept trusted SSL certificates in Chrome and Firefox browsers?

Is this when using http_get and http_post?
if so, quick work around is to use requests.get(url, verify=False).
@lucianopuccio working on this one at the moment, will have a pull request for you today hopefully.

pull regrest for http_get and http_post submitted #58
Still not sure if this is the same issue as @zeeshanyshaikh is having.

Sorry for the late reply..but am using selenium to invoke the browser with custom ssl certificate.


python312Packages.pytouchlinesl: 0.1.8 -> 0.2.0

Diff: https://github.com/jnsgruk/pytouchlinesl/compare/refs/tags/0.1.8...0.2.0
Changelog: https://github.com/jnsgruk/pytouchlinesl/releases/tag/0.2.0

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 357785

x86_64-linux

:white_check_mark: 5 packages built:

home-assistant-component-tests.touchline_sl
python311Packages.pytouchlinesl
python311Packages.pytouchlinesl.dist
python312Packages.pytouchlinesl
python312Packages.pytouchlinesl.dist


fix: remove duplicated decodeURIComponent

As it is mentioned by @claudiocastro I proposed the change made at #3268 adding some e2e tests, in order to get the fix merged. This change is a fix for the #2725 issue.
Before this change the URL such as:

/query/%25
/query;q=%25

Produced an initial error:

URIError: URI malformed at decodeURIComponent ()

And recursive errors:

TypeError: Cannot read property 'path' of undefined

Fix change fix the issue and other test cases still pass.
Best regards
Close #2725
Close #3268
Close #3138

I've added the restrictions you have proposed and solved the differences. The point was that when initially executes the application the route path was decoded (obtains the URL from the browser); whilst when changed the route from inside the application it was encoded (via clicking router-link).
This fact is what initially produces the decoding error on initially load (issue #2725), because it tried to decode an already decoded URL.
I have assumed that the URL used as 'to' param in router-link must be decoded and it worked. As it can be checked in the test, when changing the route via clicking a router-link which has defined a decoded URL the application URL is properly encoded, so I assume it is right.
Thank you for your comments.

I have assumed that the URL used as 'to' param in router-link must be decoded and it worked. As it can be checked in the test, when changing the route via clicking a router-link which has defined a decoded URL the application URL is properly encoded, so I assume it is right.

That's the issue though, to as a string or path should be encoded 🤔 I will have to check this on older browsers first to make sure it still works

Ok @posva, thank you very much for you attention. Can I help in any way? or it is up to you checking the changes in old browsers?

After doing some research, coding and testing I found out the following:
The URL got from window.location was decoded, as it can be seen in html5.js and hash.js. But, I cannot see where the URL got from the "to" attribute of router-link is decoded, neither on push, replace, ... methods. As posva said before, the mentioned attribute should be encoded, so I presume that the internal URL was different (encoded or not) depending on the way to access the route. If the application has initially load, the internal URL was decoded, but when change the route without reloading the page, the URL was encoded.
This point caused the duplication decodeURIComponent, which is the core of the issue.
In order to fix this I have made the following changes:

Remove the decoding of the initial URL in html5.js and hash.js, to ensure the internal URL is the same independently the way the route is accessed.
Apply de decode before executing route matching, instead of decoding each parameter independently, to ensure the application found the route.

With these changes all tests works. If I can help you in any way I would be glad to do it.

Thanks a lot, I think this is starting to look good. Sorry, it's taking some time, I must make sure it doesn't break existing code. So far, it's working well on all my tests. One thing I need to add is the application not to crash when the URL is malformed (happens on old browsers like IE9 on /oops%not-valid). This is done by doing a try catch and using the original value if it fails.

I'm glad he have finally managed to fix the issue successfully! One more thing, what is the protocol or schedule to publish a new release? I am looking forward to finally installing it on my project and close the issues.
Many thanks

I still need to fix a few introduced problems and will likely release it next week

I just installed dev in our project to see the results of this patch. But it doesn't seem to solve the issue for us. We do have a URL like: /records/$%25$%25$%25 I do see the window.location contains this exact value.
The error:
[Vue warn]: Error in beforeCreate hook: "URIError: malformed URI sequence"

(found in <Root>)

[Vue warn]: Error in render: "TypeError: route is undefined"

(found in <Root>)

This happens when a user refreshes the page.

@jaapio you need to build vue-router to update the dist folder in order to use the patch

Will check that

This change breaks existing behavior.
I have a route like /join/:server where server is a url encoded value like https:%2F%2Fsubdomain.domain.com.
Before v3.4.6, it decoded the server param to: https//subdomain.domain.com.
Now, in v.3.4.6, it does not: https:%2F%2Fsubdomain.domain.com.

Still duplicating decodeURI in case of regex path. I found this code in function matchRoute:
  try {
    m = decodeURI(path).match(regex)
  } catch (err) {
    if (process.env.NODE_ENV !== 'production') {
      warn(false, `Error decoding "${path}". Leaving it intact.`)
    }
  }

  if (!m) {
    return false
  } else if (!params) {
    return true
  }

When second decoding fails, function return false and route can not be matched. I think code must be edited like this:
  try {
    m = decodeURI(path)
  } catch (err) {
    if (process.env.NODE_ENV !== 'production') {
      warn(false, `Error decoding "${path}". Leaving it intact.`)
    }
  }

  m = path.match(regex)

  if (!m) {
    return false
  } else if (!params) {
    return true
  }


Add data models tests

[x] Add unit tests for data application models

Coverage increased (+4.07%) to 95.131% when pulling 8dd634d249acef5a84a38b3313d406e9431423da on add-data-models-tests into e6e1826dc57ed2fc9ba036a7453a5238300bdb15 on master.


Error saving message

When I save and edit the steps containing the code, they transform
What I want to keep:( http://sabir.greenred.ru/chrome_oE6VAo9htz.png)
Save - Update
What is really saved: http://sabir.greenred.ru/chrome_63o89dCGk6.png

@PolinaDominion can you send me that code?
I was able to save html or simple xml code - https://prnt.sc/xqJI9POQhSDi

Sure

<yml_catalog date="2021-07-16 08:32">

Name
Appa
https://tlt.kuva.ru
<local_delivery_cost>300</local_delivery_cost>

false new!!!

<yml_catalog date="2021-07-16 08:32">

Ка
Э
https://ta.ru
<local_delivery_cost>300</local_delivery_cost>

...

@PolinaDominion I saved it as simple text in step description. Maybe you copied it and inserted with some css styles, that's why the editor couldn't handle it.

https://prnt.sc/RsKGbV48X0uF


Unable to join slack channel

Hey!
I wish to join this community's slack channel. But it seems that I can only log-in (if I have credentials) but can't register for same. Does anybody have any idea why?
Thanks!

I sent an invite to the email registered with GH.

Thanks! Joined :-)

@afrind I have the same problem.
Would you please send me an invitation?

@afrind I have the same problem.
Would you please send me an invitation?

Here is the updated link: https://join.slack.com/t/mvfst/shared_invite/zt-7ug084te-iMvzkeEdZzN1FFNbobx9Ow
(note, this is a temporary invite -- it rotates every 30 days or so)

Here is the updated link: https://join.slack.com/t/mvfst/shared_invite/zt-7ug084te-iMvzkeEdZzN1FFNbobx9Ow
(note, this is a temporary invite -- it rotates every 30 days or so)

Thanks a lot.

Thanks a lot.


Add a MediawikiOAuthenticator

Hello! I've been using https://github.com/yuvipanda/mwoauthenticator at Wikimedia's test installation, and would like to get it merged 'upstream' (here). Is this repo accepting new such authenticators, or should it continue to live in a separate repo? If this is acceptable here I'll do the work in integrating it and send a PR. Wanted to ask before I expend effort :)
Thanks!

It's unclear to me what exactly the scope of this repo is in terms of new authenticators. I didn't originally intend for it to be anything more than GitHub, but we've added a couple already. I'd say it's fine to add this, if you would prefer it to be bundled as part of this repo, but you don't need to if you are happy maintaining your own package. Since CI for OAuth is hard, we'll have to rely on you for testing/maintaining it.

I'm happy to be maintaining it :) I'll work on a PR to move that into this repo! Thanks :)


Ng2 smart table pagination

How do you handle pagination in ng2-smart-table in angular 7?
I have a too many records and it would take too much time to load all of them. How do i set the table to request 10 records at a time ?
For example: when the table loads for the first time is requests the 10 first records. When I press one of the pagination buttons, it requests the "n'th" records ?
I've read the previous articles but I'm still unclear on how to handle this issue.
Thank you

Hi @mPishtra, did you get the solution? If so, could you please share the example.

Use ServerDataSource

No，you can't do it.

There was something that was clashing with the pagination. The guides provided by ng2-smart-table work.
Thank you

Check out my solution here:
https://stackoverflow.com/a/65583770/4306436

Check out my solution here:
https://stackoverflow.com/a/65583770/4306436

How did u solve this issue @mPishtra ?


[BUG] Backups don't work after upgrading 1.1.2 to 1.2.0

Describe the bug
I'm using the Longhorn Helm chart and updated from version 1.1.2 to 1.2.0. The new engine image got deployed and all volumes successfully updated to the new one.
I cannot create or list backups anymore, all buttons for creating backups are disabled and show this error:

To Reproduce
Steps to reproduce the behavior:

Upgrade from 1.1.2 to 1.2.0
Try to create or list backups

Expected behavior
I should be able to create/list backups.
Log
If applicable, add the Longhorn managers' log when the issue happens.
You can also attach a Support Bundle here. You can generate a Support Bundle using the link at the footer of the Longhorn UI.
Environment:

Longhorn version: 1.2.0
Installation method: Helm
Kubernetes distro and version: K3s 1.21.4+k3s1

Number of management node in the cluster: 1
Number of worker node in the cluster: 3

Node config

OS type and version: Debian 10
CPU per node: 4
Memory per node: 8GB
Disk type: NVMe
Network bandwidth between the nodes: 10GBit/s

Underlying Infrastructure: KVM
Number of Longhorn volumes in the cluster: 33

@timoschwarzer  Can you provide us a support bundle to<EMAIL_ADDRESS>?
The error is complaining that engine image v1.2.0 is not deployed on the node that is managing the volume.

@PhanLe1010 thanks, sent! :)

Thanks @timoschwarzer! Checking

@timoschwarzer I couldn't open the file you sent. Can you send it in unencrypted form?

@PhanLe1010 The message is not encrypted, only signed. I've sent an unsigned message.

@PhanLe1010 Hmm, RWX volumes seem to be the culprit. After the engine upgrade, RWX volumes didn't contain any data anymore. Luckily, after shutting down the workloads and detaching the RWX volumes manually, the data was back. After I reattached all RWX volumes by hand, the backup create buttons came back and I could list backups again

It seems like a temporary problem with Longhorn UI not being refreshed. From the support bundle I see that engine image v1.2.0 is deployed on all nodes but the above picture is still complaining the engine image doesn't  exist on the node.

@PhanLe1010 yep. The UI showed version 1.2.0 in the bottom left at the time backups didn't work though.

After the engine upgrade, RWX volumes didn't contain any data anymore (they were just empty for the workloads they were attached to)

This should be a different problem. I see in the support bundle that all volumes has the field remountrequestedat set which indicate somehow they are crash unexpectedly. For example,
2021-08-31T23:33:04.607966627+02:00 time="2021-08-31T21:33:04Z" level=info msg="Event(v1.ObjectReference{Kind:\"Volume\", Namespace:\"longhorn-system\", Name:\"pvc-8fa72453-4d17-4524-9ea7-7d35587996e2\", UID:\"56971b2b-32d7-411d-bb22-1e87b29ff069\", APIVersion:\"longhorn.io/v1beta1\", ResourceVersion:\"79432469\", FieldPath:\"\"}): type: 'Warning' reason: 'DetachedUnexpectly' Engine of volume pvc-8fa72453-4d17-4524-9ea7-7d35587996e2 dead unexpectedly, reattach the volume"

Were there any kind of node reboot or something?
By design, Longhorn should restart the workloads that use those volumes. However, if the volume is RWX and the workload that use the volume has only 1 pod, you may hit this known issue https://github.com/longhorn/longhorn/issues/2946. e.g, the workload pod cannot comeback. Can you confirm that one of such volume is pushgate-pv ?

The UI showed version 1.2.0 in the bottom left at the time backups didn't work though.

Still possible that Longhorn UI is temporary lagging behide after the upgrade.

@PhanLe1010 I've checked and there was no node reboot or crash. Also, the volume pushgate-pv is expected to have no workload as it's scaled down currently.

Two out of the three longhorn-manager pods restarted 2 and 5 times respectively at the beginning. Could that be related?

No, Longhorn manager pods restarting shouldn't cause volume to crash.

@PhanLe1010 yep. The UI showed version 1.2.0 in the bottom left at the time backups didn't work though.

Have you refreshed the page at that time? If the backup target is available,  the button change the status only when you refresh the page.

@jenting Yep, I refreshed the page several times. I even restarted the ui pod, without success.

@jenting Yep, I refreshed the page several times. I even restarted the ui pod, without success.

Another possibility is that even the default engine image is ready, however, the backup_target_controller still waits for the poll interval to reconcile the backup target. Ref to:

https://github.com/longhorn/longhorn-manager/blob/v1.2.0/controller/backup_target_controller.go#L288-L296
https://github.com/longhorn/longhorn-manager/blob/v1.2.0/controller/backup_target_controller.go#L244-L253
https://github.com/longhorn/longhorn-manager/blob/v1.2.0/controller/backup_target_controller.go#L235-L238

@jenting I disabled polling by setting the interval to 0

@jenting I disabled polling by setting the interval to 0 as it says right below the setting

Okay, thank you. I think we should only check the backup target available or not based on the backup target configuration only.

The current workaround for you is to re-configure one of the backup target settings, either backup target URL, backup target credential secret, or poll interval to re-trigger the backup_target_controller run the reconcile loop, to make the BackupTarget CR status.available changes from false to true.

The current workaround for you is to re-configure one of the backup target settings, either backup target URL, backup target credential secret, or poll interval to re-trigger the backup_target_controller run the reconcile loop, to make the BackupTarget CR status.available changes from false to true.

I had the same issue.
Changing the backup poll interval worked for me to workaround it.

Validation: PASSED

Prepare Longhorn 1.2.x, S3, NFS
When engine image scale down, backup become inactive
When engine image scale up, backup become active
Test Steps:
1

Scale down engine image
Configure backup target to S3, poll interval  0
Scale up engine image, backup become active

2

Scale down engine image
Configure backup target to S3, poll interval  300
Scale up engine image, backup become active

3

Scale down engine image
Configure backup target to NFS, poll interval  0
Scale up engine image, backup become active

4

Scale down engine image
Configure backup target to NFS, poll interval  3000
Scale up engine image, backup become active

Tested on master as well, and the validation result was passed


[WIP] Rewrite Android lightbox

Fixes #1485.
This is a comprehensive rewrite of the Android lightbox aligning it closer with behavior you see in stock Android apps. We're using Reanimated + RNGH since these provide more control and will let us compose with scroll views better.
Not ready for a full review yet.

[x] Pinch and pan gestures

[x] Basic interaction
[x] Panning and pinching doesn't go offscreen (new!)
[x] Pinching is centered around focal point (new!)
[x] Panning is disabled while black bars are visible (new!)
[x] Enforce max zoom level

[x] Feature parity

[x] Swipe away gesture

[x] Basic implementation
[x] Make it more tolerant X-wise

[x] Double tap gesture
[x] Show/hide controls based on zoom level
[x] Loading indicator

[ ] Check why sometimes it appears when you don't expect (multiple images?)

[ ] Integration

[x] Re-enable the "gallery" scroll view above (temporarily disabled)

[x] Fix conflict with pinching

[ ] Delete dead code from old implementation
[ ] Fix types / lint
[ ] Move utility functions into own file

Still unhappy with how pan and pinch gestures compose but maybe it's not blocking. Left a comment in https://github.com/software-mansion/react-native-gesture-handler/issues/2616#issuecomment-1745657371 for the RNGH maintainers.

As for other notes, I'll do them in a follow-up. I want this PR to just get us to the baseline with the custom implementation. I'll polish up the code and make some videos to show the difference with what we had.

I'll keep this for historical reasons but will resubmit cause a rebase was too hard and I needed to restructure things anyway.

Resubmit: https://github.com/bluesky-social/social-app/pull/1624


Lifespan seems not needed for the tensor_pool

Tensor_pool is saving lifespan but  it seems there is no need save the lifespan directly.

there is no public method to query lifespan so there must be internal use.
Internally, it is only used to check if it is longterm tensor.
While lifespan does not convey any meaning by having FORWARD_FUNC_LIFE_SPAN, BACKWARD_FUNC_LIFESPAN,  ITERATION_LIFESPAN.
Also, lifespan easily loose it's actual meaning in case of extension or viewing.

What the tensor_pool really need is that querying if a certian tensor is persistent or managed or unmanaged inside tensor pool.
Option 1. We can have a dedicated enum value for this (UNMANAGED, PERSISTENT, MANAGED)
Option 2. We can just use exec_order to query the state.
1. exec_order is empty == UNMANAGED
2. exec_order has 0 ~ uint_max == PERSISTENT
3. none of above == MANAGED

Seeing the current code, PERSISTENT and MANAGED is actually the same state, so we don't strictly need to distinguish those two though for option 2.
I prefer to option 2. If there is no caveat I haven't though of.
Please leave some opinions or correct me if I am missing something :)

:octocat: cibot: Thank you for posting issue #1696. The person in charge will reply soon.

AFAIK, we will decide when ths tensors are off-loaded with tensor life span and execution order.

AFAIK, we will decide when ths tensors are off-loaded with tensor life span and execution order.

I think using lifespan outside of tensor is not possible because we simply can't believe the value inside it.
for example, we have a tensor of lifetime = lifespan::FORWARD in layer A, later we extend the lifespan to lifespan::BACKWARD in layer B. the value of tensor lifespan will be lifespan::FORWARD | lifespan::BACKWARD but it no longer conveys any meaningful meaning.

Consider two tensors, which have the same execution order usage.

Tensor 1 : exec_order - x1, x2 (x2 > x1)
Tensor 2 : exec_order - x1, x2

Now, let's assign Tensor 1  with lifespan ITERATION_LIFESPAN and Tensor 2 with EPOCH_LIFESPAN.
With these lifespans, Tensor 1 memory can be overwritten after x2, but tensor 2 memory cannot be overwritten after x2.
The information of lifespan and execution order is combined to create validity information which is passed to the memory pool.
So, exec_order itself is not sufficient to determine the validity of the tensor, and lifespan is needed to evaluate the exact validity.

Consider two tensors, which have the same execution order usage.

Tensor 1 : exec_order - x1, x2 (x2 > x1)
Tensor 2 : exec_order - x1, x2

Now, let's assign Tensor 1 with lifespan ITERATION_LIFESPAN and Tensor 2 with EPOCH_LIFESPAN. With these lifespans, Tensor 1 memory can be overwritten after x2, but tensor 2 memory cannot be overwritten after x2. The information of lifespan and execution order is combined to create validity information which is passed to the memory pool.
So, exec_order itself is not sufficient to determine the validity of the tensor, and lifespan is needed to evaluate the exact validity.

Yes, for this example.. we need three state. 1. UNMANAGED, 2. PERSISTENT, 3. MANAGED. and those can be inferred from exec order if we mean it. but others (like forward, backward lifespan) conveys no meaning.
We cannot define a validity of forward lifespan because we cannot simple trust it's lifespan.


Docs change: More detailed description of StreamPeerTCP.is_connected_to_host()

Related issues: #37036 #18323
The StreamPeerTCP.is_connected_to_host() method description is very misleading which may cause some bugs. The method returns TRUE when a socket is connected or the socket is in connecting state which was not mentioned in docs. Lets consider the code similar to this:
var tcp = StreamPeerTCP.new()
var tcp_packet = PacketPeerStream.new()
tcp_packet.set_stream_peer(tcp)

tcp.connect_to_host('valid_ip', valid_port)
if tcp.is_connected_to_host():
    tcp_packet.put_packet(some_data)

In this case when we pass if statement we should be ensured that connection is established and ready to transfer data but in fact it may be in STATUS_CONNECTING state under the hood and the packet will be lost.

Thanks!

Cherry-picked for 3.2.4.


(v4 inline) the spacing around \implies is different from v3

I'm loading MathJax v3 via
<html>
<script<EMAIL_ADDRESS>\(P\implies Q\)
</html>

Screenshot:

I'm loading MathJax v4 via
<html>
<script<EMAIL_ADDRESS>\(P\implies Q\)
</html>

Screenshot:

The CHTML output is fine.

This is an interaction of the in-line lin-breaking code with the spacing in SVG output, similar to #3165 and #3135, but apparently not quite the same one.  I will need to look into it further.

PS, the current version for v3 is 3.2.2, not 3.0.1, so best to test against that in the future.

I have made a PR to resolve this issue, but for now, you can incorporate
MathJax = {
  startup: {
    ready() {
      const {SvgWrappers} = MathJax._.output.svg.Wrappers_ts;
      SvgWrappers.mspace = class mySvgMspace extends SvgWrappers.mspace {
        computeLineBBox(i) {
          const bbox = super.computeLineBBox(i);
          if (i === 1) {
            bbox.w = this.getBBox().w;
            bbox.isFirst = (bbox.w === 0);
          }
          return bbox;
        }
      }
      MathJax.startup.defaultReady();
    }
  }
};

into your MathJax configuration in order to work around the problem.


Attrs Support

Is your feature request related to a problem? Please describe.
As of right now, this library does not support attrs dataclasses. Default values or types are not recognised.
Describe the solution you'd like
Attrs to be detected and their attributes documented correctly
Describe alternatives you've considered
Writing the documentation in the class docstring manually.
Additional context
Attrs: https://www.attrs.org/en/stable/

Boost priority

Boost priority in our backlog through Polar.sh. Higher pledge, higher priority.
Minimum pledge by user/organization is $5, minimum amount for boost is $30.
View all issues with pledges.
We receive the funds once the issue is completed and confirmed by you.
Features with the insiders label are released to sponsors first, and tied to a funding goal.

As per https://github.com/mkdocstrings/mkdocstrings/issues/317#issuecomment-922075687, the new extractor will make it easy to write extensions to support third-party libs such as attrs.

Any chance to bump this? :)

I attempted to implement an extension for Griffe. However it really needs to be a post processing extension.
To my knowledge this isn't possible yet

Thank you for giving it a try @LordOfPolls. Post-processing extensions are on the Griffe to do list :slightly_smiling_face:
I'll also try to think a bit more about attrs support in particular.

Oh neat, congratulations on the move. If attrs support became a insiders feature, i'd definitely jump on your sponsors list

Hi, is this extension still planned? Would be very much interested 🙃

Still planned!

Hi @pawamoy, great to hear. In this case, I'd also like to point you to here

Thanks @AdrianSosic, that's useful context!


Add duplicate icon

This PR adds a duplicate icon. There's not much more to say. Below is a screenshot of the addition.

Coverage remained the same at 100.0% when pulling 32ed2fd219c73e85a4ebba412e6c869607752c99 on add-duplicate-icon into 955bcf1796424727a9a96670a5eb1c8f3a8430b5 on master.


Don't allow Python 3 to fail

Now pycorpora's Python 3 compatible (#5), let's keep it that way: don't allow Python 3 failures on Travis CI.

good catch, thanks!


X[:, i].copy() is extremely slow in nopython mode

When running this
import numpy as np
from numba import njit
X = np.random.randn(2, 1000000)
@njit
def f(index):
    Y = X[:, index].copy()
    return Y
f(0)

It takes about 60 seconds to compile the function on my machine
whereas by running
def g(index):
    Y = X[:, index].copy()
    return Y
g(0)

It is instantaneous.
Running
%timeit f(0)
%timeit g(0)

Does show that after compilation f is faster (510 ns per loop) than g (809 ns per loop). But in terms of compilation time it is really slow! Does this make sense?

I agree that 60 seconds to compile that function seems every long.  What Numba version, OS and hardware platform are you using?

It's probably a duplicate of https://github.com/numba/numba/issues/875
This is due to the global array being materialized as a LLVM array constant...

Closing as duplicate.

ah, missed that the array was not being passed in.  The general workaround here is to pass large arrays as parameters for now.

This also might be a good use case for @jitclass if X does not need to be truly global.

@seibert I'm in OSX 10.10.5 with numba 0.23.1. You said that this might be a use case for @jitclass and this depends on X being or not truly global. I'm failing to make sense of why using classes might help in this situation.

A global variable + a function behaves a lot like a class where the global variable becomes an instance attribute, and the function becomes a method.  This would allow you to carry around some state (X) and the code to use it (f()) in a way that avoids Numba trying to inline the data.


feat: Rework import generation for stubs.

Is your feature request related to a problem?
Currently import information are directly taken from the api_analyzer to the stubs generator, but for the stubs we need to check all type hints in the Python code whether the types are new types imported from other files or not. For those new types imports have to be generated and other imports from the api_analyzer do not have to be taken over to the stubs.
Desired solution
Mypy should be able to specify source paths for type hints, therefore for the first step, those information have to be found and added to the api_analyzer. The second step would be to generate imports for with the stubs generator.

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

v0.2.0
GitHub release

Your semantic-release bot :package::rocket:


Documentation [Error]

Hi, just to let you know, there's an error in the Docs.
Note: In django 1.10 and above: make sure to replace MIDDLEWARE with MIDDLEWARE_CLASSES
Should be:
Note: In django 1.10 and above: make sure to replace MIDDLEWARE_CLASSES with MIDDLEWARE

Thanks!


Fix build issues

This PR is a followup to https://github.com/Shopify/vitess/pull/149 in order to fix build-specific issues.
Essentially, vttestserver builds were failing with the error:
#10 93.94 The following packages have unmet dependencies:
#10 94.01  percona-xtrabackup-80 : Depends: libprocps7 (>= 2:3.3.15-1) but it is not installable
#10 94.02 E: Unable to correct problems, you have held broken packages.

This is because they updated vttestserver Dockerfile to use bullseye image in https://github.com/vitessio/vitess/pull/13921 but try to install packages for buster inside install_dependencies.sh.
This PR adds another "flavor" to the install_dependencies.sh script to use bullseye repo and install the correct packages that would work with bullseye.

Is this impacting upstream as well?

This is fixed in upstream version 18 onwards as that is when they changed the repositories to bullseye for everything except mysql 5.7, but does impact versions 16 and 17 as far as we can tell.

Would it make sense to update the package references rather than introduce a new "flavor" which generally refers to the mysql version/distribution and not the OS?

We discussed this with Austen earlier today and decided to backport the upstream changes that fix the issue in later releases into our releases for 16 and 17. Shanth can expand more. This is the main PR to bring in I think: https://github.com/vitessio/vitess/pull/13664


Please consider current Moq privacy issues and replace with something else

Privacy issues with SponsorLink, starting from version 4.20:
https://github.com/moq/moq/issues/1372
Video "Remove Moq From Your .NET Projects Right NOW!" with more details:
https://www.youtube.com/watch?v=A06nNjBKV7I
I won't expect Microsoft will continue to advice using of Moq considering this info.

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

ID: 1a04a5e4-3060-2331-f78d-8b45064b8983
Version Independent ID: 673d9406-3fa1-a9eb-f5cc-a431155e0260
Content: Testing with a mocking framework - EF6
Content Source: entity-framework/ef6/fundamentals/testing/mocking.md
Product: entity-framework
Technology: entity-framework-6
GitHub Login: @ajcvickers
Microsoft Alias: avickers

Still this is in Microsoft page and nobody reads REDME :)

This PR has been closed because EF6 is no longer being actively developed. We are instead focusing on stability of the codebase, which means we will only make changes to address security issues. See the repo README for more information.

This only changes suggestions in the documentation, and doesn't affect the codebase?

@srijken Yes. Delete this topic and add the reference to "Moq is dangerours" topic. No code affected.
See also: https://github.com/Azure/azure-sdk-for-net/issues/38111#issuecomment-1671961470

@ajcvickers I think your autoclose script tripped up here.  Can you reopen this?


WIP: Queue shift for reject on timeout

[x] tests and linter show no problems (npm t)
[x] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)
[x] description of changes is added in CHANGELOG.md

Landed in https://github.com/metarhia/metautil/commit/b36d2c0108b75fdca0a0e382c7f7563e131e96bb


Add clusterctl provider label to BYOH infrastructure components

Clusterctl integration requires provider label(clusterctl.cluster.x-k8s.io/provider) to be available with all BYOH infrastructure components. Details here.
This label cluster.x-k8s.io/provider: "infrastructure-byoh" has to be added here in kustomization.yaml and validate the same via make manifests .

Hi, i would like to work on this issue

@himanshu007-creator Thank you! I have assigned the issue to you. Let me know if you need any help 🙂

@himanshu007-creator Thank you! I have assigned the issue to you. Let me know if you need any help 🙂

Hey!
I am done with the addition in kustomization.yaml. But when i run make manifests command i got error make ✖ ERR  not found: make-##
I ran make manifests --debug  and got the following logs

Hey @himanshu007-creator , So the make target is trying to download controller-gen locally. Are you running this on a windows env? I'm not sure of the behavior. @dharmjit, Do you know?
You can also open a draft PR and we can look at your changes and try to debug from there.
Thanks!

Hi @himanshu007-creator, As @anusha94 mentioned, please raise a draft PR with only the change in kustomization.yaml fille. Thanks.


Add placeholder to guarantee that www directory always exists

Sometimes git would omit the www directory if the project was checked in
causing the next user to get an error because it was not present.
Add an update notifier
Fix the appveyor badge

Current coverage is 34.55% (diff: 100%)

Merging #52 into master will increase coverage by 0.35%

@@             master        #52   diff @@
==========================================
  Files            25         25
  Lines          1275       1282     +7
  Methods         152        154     +2
  Messages          0          0
  Branches        161        161
==========================================
+ Hits            436        443     +7
  Misses          839        839
  Partials          0          0

Powered by Codecov. Last update a4dbf2b...c79e0b2


chore: add issue template

Add issue template for notation-go, which is copied from notation repo.
Signed-off-by: Yi Zha<EMAIL_ADDRESS>

Codecov Report

Merging #293 (fbfcf42) into main (fb79e6d) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #293   +/-   ##
=======================================
  Coverage   73.63%   73.63%
=======================================
  Files          23       23
  Lines        1999     1999
=======================================
  Hits         1472     1472
  Misses        427      427
  Partials      100      100

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Unable to query on time, datetime2, or datetimeoffset with precision of 0

I receive the error message "Invalid scale for TIME/DATETIME2/DATETIMEOFFSET type" when attempting a query that contains a column of type datetime2(0).
I believe this is due to the readVarLen function in types.go not supporting a scale of 0 for these types.
See the precision, scale description in the datetime2 documentation

fixed

Thanks! :+1:


extending encap decap test to use tescale library for updated scale n…

…umbers

Pull Request Functional Test Report for #2970 / 3735ab24baed99ca989f821bd12f4afeaf9ed343
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

TE-14.2: encap and decap scale
d20f7547
Log

Cisco 8000E

TE-14.2: encap and decap scale
e1c3e5e0
Log

Cisco XRd

TE-14.2: encap and decap scale
d686f683
Log

Juniper ncPTX

TE-14.2: encap and decap scale
98dcc69f
Log

Nokia SR Linux

TE-14.2: encap and decap scale
9db65ff5
Log

Openconfig Lemming

TE-14.2: encap and decap scale
806cb42f
Log

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

TE-14.2: encap and decap scale

Cisco 8808

TE-14.2: encap and decap scale

Juniper PTX10008

TE-14.2: encap and decap scale

Nokia 7250 IXR-10e

TE-14.2: encap and decap scale

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1983

Relevant Lines:
3550

💛  - Coveralls


System Freeze when playing media with mpv

Hi
Today I noticed system freeze for a couple of seconds (between 10 to 40) when playing a media with mpv for first time.
I don't know how to gather required information and traces so I would be grateful if you help me with this.

Am Freitag, dem 27.10.2023 um 03:56 -0700 schrieb HJavid:

Hi
Today I noticed system freeze for a couple of seconds (between 10 to
40) when playing a media with mpv for first time.
I don't know how to gather required information and traces so I would
be grateful if you help me with this.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this
thread.Message ID: @.***>

Why you think this is a problem of the extension ?
I never observed freezes tbh.
Is this 100% reproducible with the mpv file ?

Please also add more info: https://github.com/sakithb/media-controls#reporting-issues

Hi again
i'm on fedora 38, gnome 44.5
bug is easily reproducible and occurs even when i disable all other extensions
when i play a video file with mpv(a video player) the system completely freezes, i cant even go to tty in that moment
after 20 seconds system restores and system doesn't hang till next login.

When I play a video file with mpv video player the extension is not even shown.
Seems mpv has no MPRIS interface. Any plugins for mpv used ?

Found the mpv-mpris package. I can play videos with this player. I also got some freeze 1 time when I dragged a file inside the player but I could not reproduce it a second time.
Probably the mpris implementation of mpv to blame. It seemds to be the reason for the freeze as all other players I use do not have this problem.


How to call the oidc provider without the login button

I am running the latest version of context provider example. I am wondering how I can redirect to the oidc provider directly without click on the login button. any example code will be appreciated.

You just have to display a secured component in order to trigger a login.

Let me Known if you really need a sample

I think that I got the idea, Thank you for the help.

I think we can close this issue.


May I re-use some workflow scripts?

I would like to re-use the scripts that automate the translation workflow for another GNOME Extension: https://github.com/stuarthayhurst/alphabetical-grid-extension.
Is this ok? I would of course leave a comment in the file with links to this repo and its license.

Of course, it's all MIT :smile:

Long live FOSS Licenses! Thanks ;)

https://github.com/stuarthayhurst/alphabetical-grid-extension/pull/3, in case you are interested!


chore(deps): update dependency css-loader to ^5.2.7

This PR contains the following updates:

Package
Type
Update
Change

css-loader
devDependencies
minor
^5.0.0 -> ^5.2.7

Release Notes

webpack-contrib/css-loader
v5.2.7
Compare Source
v5.2.6
Compare Source
v5.2.5
Compare Source
v5.2.4
Compare Source
v5.2.3
Compare Source
v5.2.2
Compare Source
v5.2.1
Compare Source
v5.2.0
Compare Source
Features

support async functions for url and import options (#1277) (c5062db)

5.1.4 (2021-03-24)
Bug Fixes

crash with thread-loader (#1281) (7095a7c)

5.1.3 (2021-03-15)
Bug Fixes

the auto option works using inline module syntax (#1274) (1db2f4d)
ident generation for CSS modules using inline module syntax (#1274) (1db2f4d)

5.1.2 (2021-03-10)
Bug Fixes

handling @import with spaces before and after and any extensions (#1272) (0c47cf7)
inline loader syntax in @import and modules (3f49ed0)

5.1.1 (2021-03-01)
Bug Fixes

crash on modified AST from postcss-loader (#1268) (d2a1a84)

v5.1.4
Compare Source
v5.1.3
Compare Source
v5.1.2
Compare Source
v5.1.1
Compare Source
v5.1.0
Compare Source
Features

added support webpackIgnore comment (#1264) (53d40a9)

5.0.2 (2021-02-08)
Bug Fixes

pass query with hash to other loaders (#1261) (729a314)

5.0.1 (2020-11-04)
Bug Fixes

sources in source maps have relative paths (#1219) (3229b3c)

v5.0.2
Compare Source
v5.0.1
Compare Source

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

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will now ignore this update (^5.2.7). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


希望V2RAYN可以添加trojan go websocket功能。谢谢大神

希望V2RAYN可以添加trojan go websocket功能。谢谢大神

You went to the wrong place for that.


Add Ritual bag

Alternative Bag design

And colored (color is picked corresponding to bound ritual)

Alternative Bag design

And colored (color is picked corresponding to bound ritual)

This doesn't look like vomit at least 😄


Request: Method to identify duplicate connections to the same peer

If the answer is "It's a privacy thing, so - No.  There is no unique identifier" , just let me know.  ;-)
Here is what I would like to do:
When a user connects to a wallet, I would like the ability to check if there is now a duplicate relay connection to the same peer (wallet app on a single device.)  This way I have an easier time supporting and tracing requests to the end wallet.  (I might disconnect the prior connection and associated sessions for example.)
I understand a user may connect to the same wallet app on multiple devices, so using the peer metadata seems insufficient.
It's a lower priority for my DAPP, because I could limit my app connecting to only a single wallet at a time.  (i.e. disconnect all sessions so I know there are no duplicates.)

Cross posted to the WalletConnect Discussion https://github.com/orgs/WalletConnect/discussions/2410

I'm not sure I understand precisely what you're asking for.
Let's give an example to see if I have understood.

Dapp on Computer 1, connects to wallet on mobile device A
Dapp on mobile device A connects to wallet on mobile device A

There are now two connections in the wallet to the same Dapp.
Are you wanting to disconnect one of the sessions? Or am I wrong?

I am more looking to detect a misbehaving / misconfigured app.   Or simply an unexpected state.

Dapp on mobile device A connects to wallet on mobile device A
Dapp on mobile device A connects to wallet on mobile device A

Two pairings existing between the same two peers.

Can you programmatically detect two pairings from the same peer?   It is possible the user re-installs the Dapp and loses the initial pairing config.  Could the wallet detect the older pairing and consider remediation?  I know the old pairing will expire eventually.   I just wanted to add a warning in my code if I happened to detect it.
Interop testing with all these wallets - I am getting all sorts of unexpected combinations of state.  :-/

It is not possible to detect duplicate pairings unless you're working from the same dApp and the same wallet. Only that dApp and wallet will have the pairing stored, and only they will be aware of and able to use any sessions on top of it.
This is purposeful for privacy reasons: don't want dApps/Wallets to be able to query pairings that were established for others.
A question:
Is the Dapp and Wallet the same Dapp and Wallet each time?
Let's give them some names to be explicit:

Dapp X on mobile device A connects to wallet Y on mobile device A
Dapp Z on mobile device A connects to wallet Y on mobile device A

Two pairings, but there are technically 3 different peers here, as each Dapp is treated as its own peer, the wallet as a third.
A "peer" is not a device.
A "peer" is a WalletConnect enabled Dapp or wallet.

I'm going to close this, as the request is not possible by design.
Hopefully my descriptions above make that clear.


[Bug] Add HasOneThrough on relationship pro column

Bug report
What I did
Add HasOneThrough in the relationship column pro field.
What I expected to happen
Show the value of my HasOneThrough relationship in the relationship column.
TODO
Show the value of my HasOneThrough relationship in the relationship column.
File : vendor/backpack/pro/resources/views/columns/relationship.blade.php
Code :
{{-- relationships (switchboard; supports both single and multiple: 1-1, 1-n, n-n) --}}
@php
    $allowsMultiple = $crud->guessIfFieldHasMultipleFromRelationType($column['relation_type']);
    switch($column['relation_type']) {
        case 'HasOne':
        case 'MorphOne':
            $column['type'] =  isset($column['subfields']) ? 'repeatable' : 'text';
        break;
        case 'HasMany':
        case 'MorphMany':
        case 'BelongsToMany':
        case 'MorphToMany':
            $column['type'] = isset($column['subfields']) ? 'repeatable' : ($allowsMultiple ? 'select_multiple' : 'select');
        break;
        case 'BelongsTo':
        case 'HasOneThrough':
        case 'MorphTo':
            $column['type'] = 'select';
        break;
        default:
            $column['type'] = 'text';
    }
@endphp

@includeFirst(\Backpack\CRUD\ViewNamespaces::getViewPathsFor('columns', $column['type']))

Instead of
{{-- relationships (switchboard; supports both single and multiple: 1-1, 1-n, n-n) --}}
@php
    $allowsMultiple = $crud->guessIfFieldHasMultipleFromRelationType($column['relation_type']);
    switch($column['relation_type']) {
        case 'HasOne':
        case 'MorphOne':
            $column['type'] =  isset($column['subfields']) ? 'repeatable' : 'text';
        break;
        case 'HasMany':
        case 'MorphMany':
        case 'BelongsToMany':
        case 'MorphToMany':
            $column['type'] = isset($column['subfields']) ? 'repeatable' : ($allowsMultiple ? 'select_multiple' : 'select');
        break;
        case 'BelongsTo':
        case 'MorphTo':
            $column['type'] = 'select';
        break;
        default:
            $column['type'] = 'text';
    }
@endphp

@includeFirst(\Backpack\CRUD\ViewNamespaces::getViewPathsFor('columns', $column['type']))

Hey @dimer47 hope you are doing well man 🙏
I haven't tested it, but if it's that simple we can surely give it a go.
Want to submit a PR for it or should I pick it from here ?
Cheers

Hello @pxpm,
I'm fine thank you, I hope you are too? I wanted to make a PR but it is a file present in the backpack PRO repo which is not a public repo or a repo to which I have access. This is why I created an issue instead of directly making a PR.
Am I wrong, do I have access to the backpack/pro repo? If so, could you tell me how?
Cheers

Hey @dimer47 sorry you are right.
I will add it myself and get back here with news. 🙏
Thanks

Sorry @dimer47 totally slipped.
This should be an easy one, I will reserv time today to do it 👍
Thanks for the heads up. 🙏

Hey @dimer47
It's tagged in backpack/pro 2.2.6, a composer update should get you the updated version.
// in model
 public function petAvatar()
    {
        return $this->hasOneThrough(\App\Models\PetShop\Avatar::class, \App\Models\PetShop\Pet::class, 'id', 'avatarable_id', 'pet_id', 'id');
    }

// in controller
CRUD::column('petAvatar');

Thanks again for the suggestion and providing the solution 🙏
Let me know if something does not work as you expected 👍
Cheers

Hey @pxpm,
Thank you! I updated the dependencies of my project, and now I can use this feature. Awesome, thanks a lot!
Cheers


Screen Reader is not announcing all items of the context menu list nor does it read items in collapsed dropdown

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: N/A
OS Version: Windows 10

Context:
Issues were originally reported here:
https://github.com/microsoft/vscode-mssql/issues/17200
https://github.com/microsoft/vscode-mssql/issues/17204
Steps to Reproduce:

Run the screen reader NVDA or Jaws
Hover over an item in explorer or similar dropdowns in vscode (2nd issue)
Right click on the item and hover to the left of the first item, as shown here:
 (1st issue)

Actual results:
1st issue: Screen reader only announces the first item in the context menu and not the rest of the items.
2nd issue: Screen reader only announces the name of the item, and no indication that it contains items inside.
Expected results:
1st issue: Screen reader should announce all items in context menu with positions such as 1 of 5, 2 of 5, etc when hovered over.
2nd issue: Screen reader should announce if an item in a dropdown has additional items inside it when hovered over.

Screen readers are not announcing things based on hover but on focus. So can you please clarify the issue from above, and you can please file two issues if they are unrelated please?
Screen reader users do not use the mouse, so this sounds very invalid to me....

@isidorn they are two similar issues with screen reader, also it's not necessary to click on an item on explorer (pressing shift + f10 will do the same), navigate it via arrow keys to the first item, the screen reader will then ONLY announce the first item and not the other ones below after a pause. (For more details see here: https://github.com/microsoft/vscode-mssql/issues/17200)
The second issue involves navigating to a folder item in explorer, and the screen reader not announcing anything about additional items inside, or that you can press shift-f10 to access the items inside (Again for more details see here: https://github.com/microsoft/vscode-mssql/issues/17204).
I think these issues are similar enough that they should be combined, unless they are separate, in which case please let me know.
These are issues I found with the vscode-mssql extension (which uses a similar folder and file browser) that also reproduce on the file explorer and other similar menus.

@smartguest ok so the first feature request is to announce all menus items on menu focus. This is a fair request but something which I think will bring too much noise to the user experience. So no plans to add this until more users request it. Based on this renaming this issue.
Second item sounds like screen reader not announcing if an item is expanded or not. We set the aria-expanded state properly on the explorer item and aria-posinset on the children.
If these things are not being read I suggest you file an issue against your screen reader. Thanks

@isidorn These were not my issues originally, they were brought up by @Yashi1201 during the review of our product. Please let him know because I'm merely relaying ones that also affect vscode.


[py23] disambiguate bytes vs unicode in-memory streams

This fixes issue #328, whereby an exception is raised when running traceback.print_exc function.
The error occurs because the file argument is expecting a stream of str objects: on Python 2, this must be a stream of bytes, whereas on Python 3 it must be a stream of unicode strings.
Currently we are using the name StringIO to mean an in-memory stream of bytes on both python versions.
This patch redefines the import statements inside py23 so that, on both 2 and 3:

the name BytesIO will unambiguously mean a stream of bytes;
the name UnicodeIO means a stream of unicodes.

Finally, the name StringIO is declined in a version-specific sense: that is, as a stream of bytes on Python 2 and a stream of unicodes on Python 3.
So we give streams a similar treatment as the one reserved for strings themselves (cf. tobytes, tostr and tounicode functions).
Of course, I replaced all usages of StringIO in the codebase with BytesIO -- except where it means "a stream of 'str', whatever that means in the Python version I'm running".
I hope this makes sense.

lgtm.  Please merge.  Thanks!
On Aug 7, 2015 6:48 PM, "Cosimo Lupo"<EMAIL_ADDRESS>wrote:

This fixes issue #328 https://github.com/behdad/fonttools/issues/328,
whereby an exception is raised when running traceback.print_exc function.
The error occurs because the file argument is expecting a stream of str
objects: on Python 2, this must be a stream of bytes, whereas on Python
3 it must be a stream of unicode strings.
Currently we are using the name StringIO to mean an in-memory stream of
bytes on both python versions.
This patch redefines the import statements inside py23 so that, on both 2
and 3:

the name BytesIO will unambiguously mean a stream of bytes;
the name UnicodeIO means a stream of unicodes.

Finally, the name StringIO is declined in a version-specific sense: that
is, as a stream of bytes on Python 2 and a stream of unicodes on
Python 3.
So we give streams a similar treatment as the one reserved for strings
themselves (cf. tobytes, tostr and tounicode functions).
Of course, I replaced all usages of StringIO in the codebase with BytesIO
-- except where it means "a stream of 'str', whatever that means in the
Python version I'm running".
I hope this makes sense.
You can view, comment on, or merge this pull request online at:
https://github.com/behdad/fonttools/pull/330
Commit Summary

[py23] define BytesIO, StringIO and 'UnicodeIO' to disambiguate
bytes vs unicode in-memory streams
replace all 'StringIO' with 'BytesIO'

File Changes

M Lib/fontTools/misc/psLib.py
https://github.com/behdad/fonttools/pull/330/files#diff-0 (2)
M Lib/fontTools/misc/py23.py
https://github.com/behdad/fonttools/pull/330/files#diff-1 (10)
M Lib/fontTools/misc/xmlWriter_test.py
https://github.com/behdad/fonttools/pull/330/files#diff-2 (30)
M Lib/fontTools/pens/pointInsidePen_test.py
https://github.com/behdad/fonttools/pull/330/files#diff-3 (2)
M Lib/fontTools/ttLib/init.py
https://github.com/behdad/fonttools/pull/330/files#diff-4 (2)
M Lib/fontTools/ttLib/macUtils.py
https://github.com/behdad/fonttools/pull/330/files#diff-5 (2)
M Lib/fontTools/ttLib/tables/C_F_F_.py
https://github.com/behdad/fonttools/pull/330/files#diff-6 (4)
M Lib/fontTools/ttLib/tables/S_V_G_.py
https://github.com/behdad/fonttools/pull/330/files#diff-7 (14)
M Lib/fontTools/ttLib/tables/_a_v_a_r_test.py
https://github.com/behdad/fonttools/pull/330/files#diff-8 (2)
M Lib/fontTools/ttLib/tables/_f_v_a_r_test.py
https://github.com/behdad/fonttools/pull/330/files#diff-9 (6)
M Lib/fontTools/ttLib/tables/_g_v_a_r_test.py
https://github.com/behdad/fonttools/pull/330/files#diff-10 (4)
M Lib/fontTools/ttLib/tables/_l_t_a_g_test.py
https://github.com/behdad/fonttools/pull/330/files#diff-11 (2)
M Lib/fontTools/ttLib/tables/_m_e_t_a_test.py
https://github.com/behdad/fonttools/pull/330/files#diff-12 (2)
M Lib/fontTools/ttLib/tables/_n_a_m_e_test.py
https://github.com/behdad/fonttools/pull/330/files#diff-13 (2)

Patch Links:

https://github.com/behdad/fonttools/pull/330.patch
https://github.com/behdad/fonttools/pull/330.diff

—
Reply to this email directly or view it on GitHub
https://github.com/behdad/fonttools/pull/330.


TypeError: this.eWidget is null

Hi @jojoee @YoKoGFX I'm getting the following error:

(Tried using Node v 12.18.2 and 11.15.0)
Code(React JS):
import * as Add2Calendar from "add2calendar"
import './App.css';

function App() {

  const AddToCalendar = () => {
    var singleEventArgs = {
      title       : 'Add2Calendar plugin event',
      start       : 'December 19, 2020 10:30',
      end         : 'December 19, 2020 10:50',
      location    : 'Bangkok, Thailand',
      description : 'Event description'
    };
    var singleEvent = new Add2Calendar(singleEventArgs);
    singleEvent.createWidget('#single-normal');
  }

  return (
    <div className="App">
    {/* <button onClick={AddToCalendar}>Click me</button> */}
    {AddToCalendar()}
    </div>
  );
}

export default App;

ok, let me check by today.

@mutahhar-ahmad you're right, the plugin did not support it. So I made an update and release it in v1.1.5.
Please try

@mutahhar-ahmad you're right, the plugin did not support it. So I made an update and release it in v1.1.5.
Please try


Unable to import jotai-devtools within an expo + react native app

Discussed in https://github.com/jotaijs/jotai-devtools/discussions/41
See discussions/41#discussioncomment-5393898

Originally posted by madsbuch March 12, 2023
We just had to stop using jotai-devtools as it includes a dependency on the https://mantine.dev/ framework.
Generally, it would be nice if packages like this one were headless and do not expect a document or to be run in a browser.
Anyways, nice work and thanks for the time we got to have together!

Root cause: Metro — expo's bundler is unable to resolve the multi-entry node module paths using exports field correctly (most probably due to jotai-devtools's ./dist?).
Ideally this should be fixed by turning on the unstable_enablePackageExports option but I couldn't work using config.resolver.unstable_enablePackageExports = true so  I'd suggest filing an issue with Metro and getting their recommendation as well.
Here are a few fixes I could think of.
Fix 1 (recommended)
Expo users would resolve the path (jotai-devtools/utils) manually by configuring metro.config.js. Retains type safety for jotai-devtools/utils.
// metro.config.js
const path = require('path');
const { getDefaultConfig } = require('expo/metro-config');

const config = getDefaultConfig(__dirname);
config.resolver.resolveRequest = (context, moduleName, platform) => {
    if (moduleName === 'jotai-devtools/utils') {
        return {
            filePath: path.resolve(__dirname, 'node_modules/jotai-devtools/dist/utils.cjs.js'),
            type: 'sourceFile',
        };
    }

    return context.resolveRequest(context, moduleName, platform);
}
module.exports = config

Fix 2 (requires further investigation)
We add react-native field to jotai-devtools's package.json file to help metro resolve modules correctly.
The caveat is that it only supports a string value.
Outstanding questions

How do we export the correct types i.e. ./dist/utils.d.ts for react-native property?
How do we add support for ESM? Does Metro support it?

I'm not a React-native expert by any means 😅 so it'd be great if someone could help me test if the hooks under jotai-devtools/utils work as expected 🙏

Closing this as we have a good workaround for now. Happy to re-open if the issue persist.

Hello @arjunvegda, is the good workaround you're talking about, the one you answered above or something else ? I can't find anything in the doc

hey @johannbuscail  -  I was referring to "Fix 1" and "Fix 2"mentioned in the comment above


URL Detection and Formatting for Table Chart

Is your feature request related to a problem? Please describe.
Adding clickable links into table charts is a bit cumbersome, as it requires you to create a calculated columns with an HTML-formatted link like so: CONCAT('', content, '')
This is a bit intimidating and technical for many BI users who simply want to create a quick chart.
Describe the solution you'd like
It would be great to add a checkbox option to the Table Chart that allows for automatic url detection of a column. If the column's value is detected to be a url, it should be formatted as such by automatically turning it into an HTML link.

good idea! would love to implement this feature in the future.
community, please upvote by replying "+1" in the thread if you would like to see this feature


added R port details

Thank you.

Thank you.

Thank you for checking this!

Thank you for checking this!


Add GET /v1/safes overview endpoint

This adds a new GET /v1/safes endpoint for "overviews" of specified Safes, returning the following:
type AddressInfo = {
  value: string,
  name: string | null,
  logoUri: string | null
}

type Response = Array<{
  chainId: string;
  address: AddressInfo;
  owners: Array<AddressInfo>;
  threshold: number;
  fiatTotal: string; // e.g. "69.42"
  queued: number;
  awaitingConfirmation: number | null; // `null` if no `walletAddress` specified
}>

It requires the following search parameters: currency and safes. It also accepts walletAddress, exclude_spam and trusted search parameters:
currency
The desired fiat to return the balance of the Safe in, e.g. USD
safes
A CAIP-10 comma separated list of Safe addresses, without the EIP155 namespace, e.g. 1:0x123...,1:0x456...,137:0x789.
⚠ Note: this is limited to a maximum of 7 but can be modified by setting the MAX_SAFE_OVERVIEWS env. var.
walletAddress
The owner which is checked for awaiting confirmations as a "standard" address, e.g. 0x123...
trusted (default: false)
Whether to return "trusted" transactions or not.
exclude_spam (default: true)
Whether to exclude "spam" tokens or not.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 49 (100.0%)  changed or added relevant lines in 4 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.03%) to 93.036%

Files with Coverage Reduction
New Missed Lines
%

src/routes/transactions/entities/tests/human-description.builder.ts
3
60.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
5847

Relevant Lines:
6089

💛  - Coveralls

Marking as a draft whilst I fix the tests.


[feature request] Allow free text data

Related to #14 #15
This issue is here to be an open discussion to talk about this feature request.
Feature Request:
Desired Behavior:
Allow the user to not be limited by the select fields, and allow him to type free text instead.
Technical implementation:
This options would be handled by a new parameter: strict: bool = true. By default it will be true, and if set to false, it will activate the 'allow free text' behavior.
Change the behavior who are doing the 'autoselection of the first item element' to put is as an parameter option: autoselectfirst: bool = true. This will allow, with the free text option, to choose if you want to enable 'free text' only when no other items are available. Or if you want to use it even if items are available eg:
data = ['toto', 'titi', 'tata']
userType: 't' => autocomplete will select the first element starting with t into the list, here 'toto'. But the user may just want to input 't' and validate with Enter, without selection an option.

It would be nice to also have another parameters who will be a function: freeTextEntered: (cellEditor: AutocompleteSelectCellEditor, item: T)
Who will be triggered when a non-select option is entered. Allowing to for instance, add this new option to the list of the already available ones. Or to perform a 'post' request to create a new item on the server side, ...
Problematic to consider:

Since a select field is not only used to do 'input text' selection, and that I don't want to have many different return types. I think the better is to format the 'new typed text' as an option like another who will look like: {value: inputText, label: inputText}. I think it is the better option, do you see any downside on it ?

Please, be free to comment and say what do you think about the proposed changes.
I will try to get some free time this weekend to implement theses new options.

I had a busy weekend and only now had the possibility to read this. But I like the API with strict and autoSelectFirst. As soon as you publish this on NPM I will try it out. Thank you again for this great library. With this free text feature, it saves me a lot of work 🙏🏻

New version has been uploaded on npm (v.1.0.11).


Movie Stops When I Resume playback

Hi ,
Every time I resume playback the movie stops by it self .
And one more thing , also I always receive a message " Not Found"  why?
Thanks.

You should provide log file and give more information about your issue.
http://elementum.surge.sh/contribute/log/

@Tarek-Depp I will close the issue if there is no response.

Closing without no response.


Migrate internal-only guidance to our public guidance docs

This link is mentioned in our Coding Guidelines in OneNote
Originally posted by @AlexanderSher in https://github.com/Azure/azure-sdk-for-net/pull/10065

@heaths can this be closed?

There are already plans in place to do much of this anyway, and given I'm not even sure exactly which documentation I'm talking about (I think this was supposed to be more timely, but I should've been more specific), sure.


The combining acute accent (´) by itself is much bolder than the accent drawn with "é"

Comparing the behavior to TextEdit, it seems that the bolder accent is the correct one – the lone accent is the same boldness as we're getting, but the accent on the é is bolder than ours.


User can set custom URIs

Ability for the user to provide their own URIs (overwriting defaults should they so wish).
This is focused on purely existing dimensions, opposed to establishing new namespaces.

When a user wants to describe a dimension in their dataset, they are able to provide their own custom URI and the model will accept that.
e.g. user can provide http://gss-data.org.uk/example/dimension/geography and the model will populate the correct relevant information using that URI.


Opt-out of sensitive keys (remove when found)

Signed-off-by: Joshua Packer<EMAIL_ADDRESS>
Opt-out of platform install-config data parameters (currently we opt-in by choosing specific keys)
This is a much shorter code path, but risks introduction of sensitive data if a new provider has values are introduced at some point.
The alternate is also true, if new keys are introduced, that are NOT sensitive, in the opt-in approach those would be missed till we made a change.

/test pr-image-mirror

/cc @leena-jawale @chrisahl

Looking for your thoughts on whether to just remove the opt-in code or maybe keep the opt-in code for VMware, and still use the opt-out code as well (Your votes)?

Issue Ref: https://github.com/open-cluster-management/backlog/issues/13205


Failed to import - @solidjs/router

Failing module

GitHub: https://github.com/solidjs/solid-router/
npm: https://www.npmjs.com/package/@solidjs/router

import { Router } from 'https://esm.sh/@solidjs/router'

Error message
After running deno run I got this:
Uncaught SyntaxError: The requested module<EMAIL_ADDRESS>does not provide an export named 'template'.

Additional info

esm.sh version:
Deno version: 1.29.1

This is occurring because the project uses Conditional exports  to export server code versus client-code.
https://github.com/solidjs/solid-router/blob/main/package.json#L21-L26
This could be fixed by giving us an option to provide conditionals.
https://esbuild.github.io/api/#conditions

currently we don't use conditionals of esbuild, since solid-js/web server.js doesn't export template, you can set the target to es2020 to force using web.js instead of `server.js of solid-js/web if you don't use ssr

i think this is a solid-router bug

Isn't that running the client version of solid-js/web? So it's looking for browser stuff?

for deno target, the server will build the module with the ./web/dist/server.js to support SSR, check https://github.com/solidjs/solid/blob/695d99b6fbc1f55c91aed0600b0c1f13e4ee426c/packages/solid/package.json#LL180C16-L180C16


Dynamic Selection Colors Based on Object Color

Currently, the selection color is static and defaults to blue. While this works, it may not always provide the best visual clarity, especially when working with a variety of object colors. The static blue selection color can sometimes clash with the colors of certain objects, making it difficult to distinguish selections and reducing overall visual appeal.
We suggest making the selection color dynamic, where the selection highlight color is based on the object's original color.
A slight alteration in brightness or saturation could be used to highlight the selected object without drastically changing its appearance.

Thanks for starting an issue for this! Indeed we should improve the visual indication about selection.
I kind of like the threejs editor approach of displaying the bounding box of the selected mesh (see video). Also the fact that position controls are showing up upon selection helps a lot (see the work started in https://github.com/jupytercad/JupyterCAD/pull/151). If we take this approach, I'm not sure dynamic coloring is really needed.
https://github.com/user-attachments/assets/7978fe76-693e-4afb-bbd6-b7f314122115

Thanks @martinRenou, this also makes much sense and I'll try to put together a draft for this bounding box thing and update.


Installation

Hi,
I'm very interested in this repo and would like to incorporate this plugin into an existing au-cli/systemjs progject using javascript.
I've tried to convert the ts files your readme but get systemjs errrors on startup. Is there a repo with examples as shown in your demo.
Any tips appreciated and TIA
John
SyntaxError: Unexpected token '<'
    at eval (<anonymous>)
    at Qe (system.js:4)
    at system.js:4
    at T (system.js:4)
    at T.h (system.js:4)
    at T (system.js:4)
    at T.h (system.js:4)
    at T (system.js:4)
    at T.h (system.js:4)
    at T (system.js:4)

hi John, I believe I need to create additional output files in order for this to work with au cli's bundler, currently I'm only outputting bundle as ES6. I'll try to tackle that within the next few days.

On Thu, Nov 26, 2020 at 10:32 AM jbockle<EMAIL_ADDRESS>wrote:

hi John, I believe I need to create additional output files in order for
this to work with au cli's bundler, currently I'm only outputting bundle as
ES6. I'll try to tackle that within the next few days.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jbockle/au-jsonschema-form/issues/15#issuecomment-734362446,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOKLAEK23G6XTIVCL2KF4LSRZYJLANCNFSM4T5Z6R6A
.

Thanks for your consideration


feat(message-input): DLT-1947 add slot for sms count badge

feat(message-input): DLT-1947 add slot for sms count badge
Obligatory GIF (super important!)

:hammer_and_wrench: Type Of Change

These types will increment the version number on release:

[x] Feature

:book: Jira Ticket
https://dialpad.atlassian.net/browse/DLT-1947

:book: Description
Add a slot that will be used on the product side to display the amount of SMS/segments that the user is sending.

:bulb: Context
As part of the SMS Count project, we need to add a badge to the message input that displays the amount of SMS/segments the user is sending. The logic for this will be calculated on firespotter side, so I think the best way to handle this is with a slot.

Do you need to put a wrapper around this slot with default styling so it is aligned to the other elements? Might be more difficult to do this within the slot itself.

good call. I added a wrapper with flex and center alignment and looks good now


Enable Tilt 2 support. #1920

Relaxed requirements for Tilt to ['>= 1.4.1', '< 3.0']. Check Tilt version and use Tilt.default_mapping.lazy_map instead of Tilt.mappings with Tilt 2. Drop RedcarpetTemplate::Redcarpet2 which was removed in Tilt 2.
Some tests checking for markdown configuration options still fail.

Is there anything to keep us from moving fully to v2 rather than backwards compat?

I was following what other Ruby libraries are doing:

sinatra: tilt < 3, >= 1.3
padrino-helpers: `tilt < 3, >= 1.4.1``

But not sure whether we really need support for tilt 1.x.

Tilt 2 came out 2.5 years ago, I think we're good to upgrade. In fact, we may be one of the ones holding Padrino back from fully upgrading :)

OK, changed requirements to ~> 2.0.

Cool. Any issues getting those tests to pass (or removing them in the Redcarpet 1 case)?

Ping @mfenner any movement on the remaining failing tests?

Hey guys,
We noticed a major issue with our project going from mm 4.1.10 to 4.1.11.
Doing so updates us to Tilt 2 and Tilt 2 try's to handle our .jsx files... I don't want MM touching or even looking at those files as I have webpack on it! ?? Going back to 4.1.10 fixes our below issue.
How can i tell MM to ignore .jsx files on Tilt 2?
See our stack trace
bundler: failed to load command: middleman (/Users/jd/.gem/bin/middleman)
LoadError: cannot load such file -- babel/transpiler
  /Users/jd/.gem/gems/tilt-2.0.5/lib/tilt/babel.rb:2:in `require'
  /Users/jd/.gem/gems/tilt-2.0.5/lib/tilt/babel.rb:2:in `<top (required)>'
  /Users/jd/.gem/gems/tilt-2.0.5/lib/tilt/mapping.rb:243:in `require'
  /Users/jd/.gem/gems/tilt-2.0.5/lib/tilt/mapping.rb:243:in `block in lazy_load'
  /Users/jd/.gem/gems/tilt-2.0.5/lib/tilt/mapping.rb:241:in `each'
  /Users/jd/.gem/gems/tilt-2.0.5/lib/tilt/mapping.rb:241:in `lazy_load'
  /Users/jd/.gem/gems/tilt-2.0.5/lib/tilt/mapping.rb:216:in `lookup'
  /Users/jd/.gem/gems/tilt-2.0.5/lib/tilt/mapping.rb:154:in `[]'
  /Users/jd/.gem/gems/tilt-2.0.5/lib/tilt.rb:48:in `[]'

Does ignore "*.jsx" work in config.rb?
Might make sense for us to add that by default, we already remove some of Tilt's other over aggressive filters... I think they do .html for some reason

@theozaurus you can install gem 'babel-transpiler'


Remove launching of weave plugin and unnecessary weave detach call

fixes #1319 and #1320

I suppose the "unnecessary" means that "by the time we get to detach_network, the container has already been removed and the weave veth no longer exists"?

Yes, updated the PR description with some more details. :)


Add rating section for Pages

Describe the feature you'd like
allow end users to rate pages and articles if they are usefull:

this would be helpfull to discover helpfull articels. those ratings should be filtered per Shelf / Book...

Describe the benefits this would bring to existing BookStack users
Improves content of articels. Helps to maintain the content better. Gives more statistics
Can the goal of this request already be achieved via other means?
Maybe with Tags and allow to add them by End Users
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundamental request

How long have you been using BookStack?
3 months to 1 year
Additional context
No response

Thanks for the request @nie3mals but I'd consider this already covered by previous issues #4722, #1524 and #505 so I'm going to therefore close this off as a duplicate.

I see, sorry I was only checking the open feature request! thanks for the prompt response!


Check file/folder permissions and raise problems if wrong

For whoever finds this next. The owner:group is set by the envs PUID/PGID defaulting to 1000:1000.
I was using a docker volume for my libraries instead of a folder.
nas ➜  manyfold cat docker-compose.yml
services:
  app:
    image: ghcr.io/manyfold3d/manyfold:latest
    ports:
      - 3214:3214
    volumes:
      - libraries:/libraries
    environment:
      PUID: 1000
      PGID: 1000
...
volumes:
  db_data:
  libraries:
...

nas ➜  manyfold docker volume ls | grep manyfold
local     manyfold_db_data
local     manyfold_libraries

Default service name is app
nas ➜  manyfold docker compose ps
NAME                         IMAGE                                COMMAND                  SERVICE           CREATED          STATUS          PORTS
manyfold-app-1               ghcr.io/manyfold3d/manyfold:latest   "/init"                  app               42 minutes ago   Up 42 minutes   <IP_ADDRESS>:3214->3214/tcp, :::3214->3214/tcp
manyfold-postgres-server-1   postgres:15                          "docker-entrypoint.s…"   postgres-server   42 minutes ago   Up 42 minutes   5432/tcp
manyfold-redis-server-1      redis:7                              "docker-entrypoint.s…"   redis-server      42 minutes ago   Up 42 minutes   6379/tcp

My library folder is called default
nas ➜  manyfold docker compose exec -it app mkdir /libraries/default
nas ➜  manyfold docker compose exec -it app ls -lah /libraries/
total 16K
drwxr-xr-x    4 root     root        4.0K Sep  1 15:18 .
drwxr-xr-x    1 root     root        4.0K Sep  1 14:40 ..
drwxr-xr-x    2 root     root        4.0K Sep  1 15:18 default
nas ➜  manyfold docker compose exec -it app chown 1000:1000 /libraries/default
nas ➜  manyfold docker compose exec -it app ls -lah /libraries/
total 16K
drwxr-xr-x    4 root     root        4.0K Sep  1 15:18 .
drwxr-xr-x    1 root     root        4.0K Sep  1 14:40 ..
drwxr-xr-x    2 1000     1000        4.0K Sep  1 15:18 default
nas ➜  manyfold


remove mtg default status/image

posting on behalf of @Zughy
caption at the very least should be updated to reflect mtg's non default status

I'd put something like https://content.minetest.net/uploads/0563926b9b.jpg instead (Backrooms test). The other MTG screenshot I guess isn't harmful and it also makes nostalgic players happy

or has this been fixed?

@sfan5 nope

Made a PR for it @sfan5


Default URL 'http://...' doesn't highlight retrieval of favicon

Problem Description: ShortcutHelper sets the shortcut's icon to the URLs favicon, but since URLConnection doesn't follow cross-protocol (http -> https) redirects by default, the default URL 'http://www.android.com' doesn't show off this neat feature.
Solution: Change the default url from 'http' to 'https'.
Items:

Change default URL in Main.java
Update screenshot to show example with favicon.

Created PR #27.


Create/update nodes and relationships

Are you planning to add functionality for creating/updating nodes and relationships and push that back to neo4j?

It would be great to include that functionality but that would require a lot of work and I do not have much free time at this moment.


Ship Thruster Movement

Thrusters moving the ship in space
Lever controls for Yaw, Pitch, Roll

Levers are working, ship movement is kind of wonky, might need to smooth it out (or not)


refactor: switch from react-router peer dep to react-router-dom

#59

Coverage remained the same at 100.0% when pulling fb4d69209f8571b7b645087dfa56060bbbdfeda4 on migrate-to-react-router-dom into 0f213681257f67a75f09408015ceda84240fc394 on master.


Deprecate?

It seems this is no longer maintained.
To prevent confusion, could you officially deprecate the library?
FWIW, I've now switched over to luaossl.

I found this https://github.com/wahern/luaossl but it does not seems to provide the crypto dependency for lapis

Correct, it is a different library. However it has the same functionality/abilities

I wouldn't depreciate this until luaossl can actually build out-of-the-box on windows.

@greatwolf have a look over at https://github.com/wahern/luaossl/issues/77

If there is a luacrypto fork that's maintaned, or someone wants me to transfer ownership, let me know. As you may have noticed, I no longer have the time/ability to contribute. Otherwise I will just explicitly mark this project deprecated.

@mkottman thanks for getting back. If there are no other volunteers I wouldn't mind taking up the mantle. I have a lua project that does crypto trading/lending on various exchanges so I depend on a lot of the hmac* functions for authentication so I have an interest in keeping luacrypto current.
How does the ownership transfer work? Do I need to remove my existing fork first?

@mkottman I just tested a dummy project out to see how ownership transfer works. It looks like I need to delete the fork under my account first before github will allow the transfer. I've already done that so you can do the ownership transfer when you get some time.

@greatwolf are you willing to maintain luacrypto though? There is quite a bit of code that needs updating.
I'd think it would be easier to migrate over to luaossl?

Yes, I don't mind putting some time into maintaining this. I get wanting to migrate over to luaossl but it just feels like it's making too many *six assumptions in the code. Keeping another ssl binding alternative in the ecosystem would be nice in case users have trouble getting one of them to work.
Actually, I already have the code changes on my local luacrypto repo to make it buildable on openssl 1.1.0. It would be nice to be able to commit it to upstream which is part of why I'm pushing to get this project maintained again.
It doesn't have to be me though, if there are other interested parties, luacrypto can be transfer into a organization group account with multiple collaborators. I'm not aware of any other takers atm though.

@greatwolf I also need the compatible version with openssl 1.1.0 and made buildable changes locally. Could you supply a PR ?

luaossl now works on windows.
As there is still no one maintaining luacrypto, could we add the deprecation notice?


Check for pass through only if it is not a debug query

Fix JANUS-13598 by adding a check for pass-through only if the query is not a debug query.

See https://github.com/infobloxopen/themis/commit/137545bdc7e5287049e2d4230a932ade28400ce2


Additional db indexes on lookup tables

Coverage increased (+0.0009%) to 95.152% when pulling 6c1c5d3d7297c5445ffdfa11ab46b1493c9a022b on cody-lookup-table-indexes into 1de6a0dcaa10bc20c71cf8954c31efa58fa270fb on main.


Explorer context-menu is missing when Windows Terminal is installed side-by-side with Windows Terminal Preview

Environment
Windows build number: 19042
Windows Terminal version (if applicable): 1.6.10571.0

Any other software?
Windows Terminal Preview 1.7.572.0

Steps to reproduce

Install Windows Terminal from the Microsoft Store.
Install Windows Terminal Preview from the Microsoft Store.
Open a folder in Windows Explorer, for example C:\.
Right click.

Expected behavior
Two distinct options: Open in Windows Terminal and Open in Windows Terminal Preview.
Windows Terminal and Windows Terminal Preview are distinct packages. They have distinct package names and AUMIDs. They can be installed independently. They can run independently. They have distinct configurations in separate directories.
I expect to be able to run both of them (or at least the one of my choosing!) from Windows Explorer.
Actual behavior
Note the icon is for Windows Terminal Preview and indeed it opens the Preview.

Possible implementation
I suspect using distinct CLSIDs for the two packages would suffice.
If you can have different strings, icons, etc. having different CLSIDs shouldn't be that hard.

Also https://github.com/microsoft/terminal/issues/6416. I expect that distinct CLSIDs would indeed be required.

Yea, we're tracking this in #6416. Thanks!
/dup #6416


Error on trying to close the floating window with only one entry

First of all thank you for this great plugin idea. I love that it enhances builtin concepts.
I am just trying it out, but I run into an error everytime I want to close the floating window with q, which is the default mapping as I believe from the README.md, when only one line is left.
Screenshot

Reproduction

Run :Marks
Delete all entries with d until one line remains
Press q

Config
---@type LazyPluginSpec
return {
    "gcmt/vessel.nvim",
    event = "BufRead",
    opts = {
        create_commands = true,
        commands = {
            view_marks = "Marks",
            view_jumps = "Jumps",
        },
    },
    keys = {
        {
            "gm",
            "<CMD>Marks<CR>",
            desc = "View Marks",
        },
        {
            "gj",
            "<CMD>Jumps<CR>",
            desc = "View Jumps",
        },
    },
}

Thanks for taking the time to report the issue. That's the first time I see that error and sadly I cannot reproduce the issue. Which neovim version are you using? I pushed an attempted fix by changing culprit line. Try updating the plugin and report back!

Hey thank you for getting back so quickly.
At the moment it seems your fix has helped! So thank you for that. 🙏
I am using this:
:version
NVIM v0.10.1
Build type: Release
LuaJIT 2.1.1725453128

That's great to hear! Closing for now


feature: Refactor to separate frontend and backend plugins

Issue #, if available:
Description of changes:
This work refactors the single plugin in to separate frontend and backend plugins that mirrors the AWS Proton plugins.
It also migrates the plugin to the new AWS credential provider mechanism available in Backstage 1.9.0.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

thanks @niallthomson for the PR. I saw there is a lot that happens in this PR which makes it really hard to review. If you are confident about the changes, I am happy to let this through. But it would be really nice to split it into smaller chunks.

change to the new auth mechanism
separation of frontend and backend
docs changes

otherwise I cant possibly make sense of all the changes. let me know how you want to proceed.

@nimakaviani thanks for taking a look. The reason I packaged the auth mechanism and separation as a single change is the separation fundamentally changes the authentication mechanism anyway. The old AWS auth plugin is no longer used.