Failed to resolve entry for package...

I get the following error when I try to use this in my app:
Failed to resolve entry for package "@coddicat/vue-pinch-scroll-zoom". The package may have incorrect main/module/exports specified in its package.json: Failed to resolve entry for package "@coddicat/vue-pinch-scroll-zoom". The package may have incorrect main/module/exports specified in its package.json.
Am I missing something obvious?  It's a vue 3 / Quasar 2 app and the component I'm importing into used the options API if that's relevant?
This occurs even in this simple component:
<template>

  <PinchScrollZoom>

  </PinchScrollZoom>

</template>

Any help greatly appreciated as this looks like just what I've been looking for to replace PinchZoom.js! :)

@skmbr thanks, i am looking right now

Thank you!

@skmbr fixed, I have just updated package. plz install the newest 4.2.2, and let me know if it works?

@skmbr and plz note that you need to set
:width="…"
:height="…"
These required properties

Ok... that sorted the original error, now I get this in the console:

@skmbr Please provide your code

As in my original post, but with height and width added:
<template>

  <PinchScrollZoom width="200" height="200">

  </PinchScrollZoom>

</template>

If I replace my main App.vue with the above code I get slightly less errors:

I mean - how do you use your component?
where did you put it? If you can provide your entire project it will be easier to figure out

Can't share the entire project, but I'll try and spin up a new Quasar project tomorrow and see if it still does it with that!

So I can't say I exactly understand this, but with the help of this SO post...
https://stackoverflow.com/questions/71063992/when-importing-self-made-vue-3-library-into-vue-3-project-uncaught-typeerror
... and setting this in my Vite config:
viteConf.resolve.dedupe = ['vue']
I do now have it working! 🥳


How that work ?  what is it ?

You provide a really awsome gif that trigger my interess and seem provide a really good functionality
But your readme O_o ...
Well a pip command and some raw code !!!
What i have to do with this ?
this is only python module without housing programme ont it ?
it just a poc ?
how that realy work ?
think about the all new community without any knowlege,..

Sorry not sure I quite understand the issue. This is an experimental module (so yes you can call it a proof-of-concept). That's why pip installs directly from git. I hope that helps.

Sorry not sure I quite understand the issue. This is an experimental module (so yes you can call it a proof-of-concept). That's why pip installs directly from git. I hope that helps.

i have to make some code for using it ?

i have to make some code for using it ?

If you plan to use the core RMLines module then yes. If you would like to try and reproduce what I did in the demonstration gif you can try the pdf_converter script. Admittedly it's not straight forward and will need some leg work. Best of luck!

If you plan to use the core RMLines module then yes. If you would like to try and reproduce what I did in the demonstration gif you can try the pdf_converter script. Admittedly it's not straight forward and will need some leg work. Best of luck!


Fix minor typo's

Coverage remained the same at 100.0% when pulling 41478ef3ec8b554c429f648b17c28b6b511e58ac on anthonyringoet:patch-1 into 95f68f32ce702cc2405f0d5693bb97f8dce56827 on node-apn:master.


Issues when building a Devbox Project with --engine=podman

The Dockerfile that we are generating appears to be incompatible with Podman, because we are using features that are only supported by Docker. Some issues we've encountered:

[ ] Our COPY steps use the --link flag, which is not supported by Podman:
[1/3] STEP 4/5: COPY --link ./.devbox/gen/runtime.nix ./.devbox/gen/
Error: error building at STEP "COPY --link ./.devbox/gen/runtime.nix ./.devbox/gen/": COPY only supports the --chmod=<permissions> --chown=<uid:gid> and the --from=<image|stage> flags

[ ] The step SHELL [ "/busybox/sh", "-eu", "-o", "pipefail", "-c"] is ignored, because SHELL is not suported in the OCI image format.

[ ] (Maybe related to the prior step?) The step RUN cp -rs /busybox/. /bin/ && cp -rs /busybox/. /usr/bin/ fails with the following error:

error running container: error from /usr/bin/crun creating container for [/bin/sh -c cp -rs /busybox/. /bin/ && cp -rs /busybox/. /usr/bin/]: executable file `/bin/sh` not found in $PATH: No such file or directory

These issues do not happen when running with --engine=docker, so we need to either generate a fully OCI compliant Containerfile for all engines, or have a separate Dockerfile that we use for non-Docker engines.

Related issues #55 #18 #14

This issue is no longer relevant, since we deprecated devbox build


I got this error at autocomplete

[Enter steps to reproduce below:]

...
...

Atom Version: 0.190.0
System: Microsoft Windows 7 Professional
Thrown From: autocomplete-plus package, v2.9.0
Stack Trace
Failed to activate the autocomplete-plus package
At Cannot find module 'underscore-plus'

Error: Cannot find module 'underscore-plus'
    at Module._resolveFilename (module.js:328:15)
    at Function.Module._resolveFilename (C:\Users\vntalco\AppData\Local\atom\app-0.190.0\resources\app\src\module-cache.js:383:52)
    at Function.Module._load (module.js:270:25)
    at Module.require (module.js:357:17)
    at require (module.js:376:17)
    at Object.<anonymous> (/C:/Users/vntalco/.atom/packages/autocomplete-plus/lib/autocomplete-manager.coffee:8:7)
    at Object.<anonymous> (/C:/Users/vntalco/.atom/packages/autocomplete-plus/lib/autocomplete-manager.coffee:616:4)
    at Module._compile (module.js:452:26)
    at Object.requireCoffeeScript (C:\Users\vntalco\AppData\Local\atom\app-0.190.0\resources\app\node_modules\coffee-cash\lib\coffee-cash.js:85:19)
    at Module.load (module.js:347:32)

Commands

Config
{
  "core": {
    "themes": [
      "seti-ui",
      "seti-syntax"
    ]
  }
}

Installed Packages
# User
autocomplete-plus, v2.9.0
css-color-underline, v1.0.1
seti-syntax, v0.3.3
seti-ui, v0.6.3

# Dev
No dev packages

Does this happen when you restart atom? Or was this a one time fluke?

Closing due to no response.


⚠ Spryr has degraded performance

In 54013fa, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 8051 ms

Resolved: Spryr performance has improved in 66d42ca after 10 minutes.


Fix accessing name from ffi schema

Which issue does this PR close?
Closes #6251.
Rationale for this change
Reading from FFI_ArrowSchema should not panic on spec-valid input.
What changes are included in this PR?

Convert return value of name() to Option<&str>
Default to empty string when importing FFI_ArrowSchema to Field

Are there any user-facing changes?
Yes

Thanks again @kylebarron


Delete twap history

This PR Adds removing old twap history that would be saved in the state machine.
The following are set as Params of the gamm module for twap history.
Spec for Gamm module to be added.

The number of twap pool it tends go over and delete per iteration
Twap history Deletion Interval(Per how many blocks would it be activating deletion iteration)
How long would twap history be kept

Codecov Report

:exclamation: No coverage uploaded for pull request base (twap@1926162). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             twap     #461   +/-   ##
=======================================
  Coverage        ?   18.88%
=======================================
  Files           ?      146
  Lines           ?    23658
  Branches        ?        0
=======================================
  Hits            ?     4467
  Misses          ?    18426
  Partials        ?      765

Continue to review full report at Codecov.

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

We will re-plan the twap architecture.

Should address https://github.com/osmosis-labs/osmosis/tree/twap as well and discuss how we can improve things


controllers: implement atlas migration

Implementation of Atlas Migration Controller
Context
Following PR https://github.com/ariga/atlas-operator/pull/22, This PR contains logic that implements the controller only.
Changes
Support migrating to the latest version using the migration directory defined in the config map and the migration directory in the Atlas Cloud
What's next

Support migration with specific version
Add watcher to detect changes in config maps and secrets
Add unit tests

Need tests, unit tests, integration tests for new controller behavior

@giautm, of course, we will have tests in the next PR. Let's review the logic to ensure it is heading in the right direction, I don't want to keep PR for too long, we already gate this controller


Prefer per-element assertions

Using per-element assertions results in better failure messages because AssertJ will pull out the unexpected and missing elements rather than just indicating the two values are not equal. This is especially useful when the objects are large and the output is less readable.
Before:
expected: ["foo", "bar"]
 but was: ["foo", "baz"]

After:
Expecting actual:
  ["foo", "baz"]
to contain exactly (and in same order):
  ["foo", "bar"]
but some elements were not found:
  ["bar"]
and others were not expected:
  ["baz"]

Released 0.4.0

So normally I'm in favour of pretty much any assertj automation that improves failure reporting, but after receiving a big PR on apollo #10374, I'm actually not sure about this one.
My concern is that it actually reduces the readability of tests, because the number of words in containsExactlyInAnyOrderEntriesOf is pretty high (7) and you have to read it quite carefully to double check whether this is the method you expect, or whether it's one of the other funny assertj methods that has subtly different behaviour.
I think I would personally prefer to leave isEqualTo on many of my tests

There’s a great deal of subtlety here. Do we want to test the equality method of the given collections (lhs or rhs?), or the collection contents? Upon failure, how do the messages differ?
In the singleton map case I think this can be further simplified for readability, perhaps there’s an upstream contribution we can make for map assertions with up to N key-value pair overloads similar to ImmutableMap.of.


HIVE-27277: GH actions to build linux/amd64 and arm64 images and push docker hub

Set up github actions workflow to build and push docker image to docker hub
Hi Everyone,
I have got the docker hub repository setup for Apache hive from Infra.
https://issues.apache.org/jira/browse/INFRA-24505
DockerHub: https://hub.docker.com/r/apache/hive
In this PR I have set up GitHub actions workflow to to automatically publish the docker image to Docker hub on every release.
Opening a new PR as older PR got auto closed  : https://github.com/apache/hive/pull/4298
Updating the description after latest changes:
In the latest patch the Github action is divide to two parts:
Build from exiting binaries for old releases. (BuildFromArchive - manually triggered)
Build from source on tag creation for new release. (BuildFromSource - auto-triggered on new tag creation )
This github action publishes docker images for both:

linux/amd64
linux/arm64

Update 2:
We don't have to worry about the github limits anymore. Now the images are built and published in a single job. Any artefact created and stored with in a job does not count towards github storage limit.

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

Is the change a dependency upgrade?

How was this patch tested?

Whenever we have a new release we create a new tag under rel e.g., https://github.com/apache/hive/tree/rel/release-4.0.0-beta-1.
Can't we simply launch the workflow on rel/tag creation, build the project, and publish from there?

Whenever we have a new release we create a new tag under rel e.g., https://github.com/apache/hive/tree/rel/release-4.0.0-beta-1.
Can't we simply launch the workflow on rel/tag creation, build the project, and publish from there?

We can.  But there is one constraint with Github actions. One hive tar.gz is about ~450 MB and for linux/amd64 and arm64 we will need a total of almost 1 GB.
But github free storage is only 1 GB as far as I am aware. Once we exceed the threshold it will affect other github actions as well and actions well fail.
https://docs.github.com/en/billing/managing-billing-for-github-actions/about-billing-for-github-actions#included-storage-and-minutes
I am not sure if Apache hive GitHub repo has special limits because of this, we decided to download the tars from the apache archive.

Maybe we should ask INFRA ticket if there are really limitations about artifacts sizes in GitHub actions.
Apart from GitHub actions we have our Jenkins precommit infra and there we have full control of what we build and run.
The current PR still requires manual intervention from someone to actually do something. Wouldn't be roughly the same to add a one line mvn command in the release instructions and let the RM build the image and publish it?
It would be great if we could find a way to fully automate this process to minimize the room for error.

Tested the build from source workflow on my personal account here : https://github.com/simhadri-g/hive/actions/runs/5944353617/job/16121443889
Images pushed to Docker hub: https://hub.docker.com/layers/simhadri064/hive/4.0.0-beta-1-snapshot/images/sha256-4a883c3b2e616e101cc6d8ce313078d89f608cff8a5e7862f4f3abab30c38fda?context=explore

Yes, i think it should be possible .
I think currently only @ayushtkn has access to hive dockerhub account to delete images.


Is it possible to run a callback after each resubscribe in pubsub?

I have a key (say cnf) containing some config that will be changed time by time, and a channel (say cnf:changed) to notify other clients that cnf has been changed. The producer side looks like
SET cnf newconfig_blablabla
PUBLISH cnf:changed ""

And the consumer side looks like:
SUBSCRIBE cnf:changed
// wait subscribed
GET cnf                 // get the newest config

The problem is that now the PubSub type seems do the reconnection/resubscribe automatically, if there is an update during the gap, the notification will be lost and the consumer will not be able to get the newest config. If a callback can be run after each subscribe/resubscribe, then i can GET cnf again to make sure we can get the newest one.

You are welcome to contribute something like that. Method like PubSub.OnReconnect(fn func()) looks like a good fit...

@vmihailenco I've created a PR https://github.com/go-redis/redis/pull/1074 to add PubSub.OnSubscription.

Should be addressed with PubSub.ChannelWIthSubscriptions


make API instance keep session object for HTTP keep-alive

To enable HTTP keep-alive over multiple requests, I would like to propose this pull request (including some minor changes).

Sorry, I opend new pull request #7 that satisfied with backward compability. Please see it. So I'll close this pull request.


with custom xsl, entries do not show formatting in viewer, but do in editor

Describe the bug
Our dictionary uses a custom xsl and the formatting displays correctly in the editor, but only shows unformatted text in the viewer. This was not an issue until a couple months ago. The links are to a cloned version of the dictionary in which this issue was first encountered.
To Reproduce
Steps to reproduce the behavior:

Go to edit in this dictionary. 2.
View any entry.
Go to view in this dictionary.
See that no entry displays the formatting seen in the edit section.

Expected behavior
We expect the entries to display in the viewer the same way they do in the editor.
Screenshots
"Edit" Viewer:

"View" Viewer:

Environment (please complete the following information):

Encountered on Mac and iOS
Using Chrome Version 100.0.4896.75 and Safari 15.5

Additional context
This displayed correctly until a couple months ago, but broke during an update to Lexonomy. Unfourtunately I do not know exactly when this happened.

I would perhaps recommend moving your dictionary to lexonomy.elex.is (where an older stable version is running), as we are currently working on significantly updates.

Thank you for reporting, display is now fixed


s local start 不支持 nodejs18版本

TypeError: Cannot read properties of undefined (reading 'split')
at Object.iDe (C:\Users\EDY\.s\components\devsapp.cn\devsapp\fc-core\dist\index.js:226:319)

at Object.<anonymous> (C:\Users\EDY\.s\components\devsapp.cn\devsapp\fc-local-invoke\dist\index.js:244513:29)

at Generator.next (<anonymous>)

at fulfilled (C:\Users\EDY\.s\components\devsapp.cn\devsapp\fc-local-invoke\dist\index.js:244095:58

https://github.com/devsapp/fc-core/pull/27


Adding a company starting with I

Alright, will do send in the PR now

No description provided.

Can be found in PR #111

Alright


list/firmware: fix filtering by component type

fixes #106

The problem is due to --limit: it should be applied after the component filtering, not before
$ ./mctl list firmware --component bios --limit 30
[
  {
    "uuid": "64bcac34-affc-41f5-bb62-33f0b3f73a4a",
    "vendor": "supermicro",
...

@ofaurax  yes, the default limit is set to 10 and so the result set is what you see without a higher limit. We require a FleetDB API change to accept a component filter - as of now this is done in mctl.


Excuse me，why Alamofire 4.0.0+ requirement iOS9.0+ ?

This is a must, and the future will always be like this？

This is a must, and the future will always be like this？

You can see issue #1367 for a full explanation as to why we dropped iOS 8 support.

@jshier  got it, thank you so much


DBC22-344: Added camera list page and component

Added React hot reloading volumes to Docker-Compose as a side addition. Right now, the camera entries load twice on refresh due to the Strict Mode wrapper around our React app, which means there may be a better way to structure our loading function or a clean-up function we may implement, I am open to suggestions.

Thanks for pushing this WIP @tyrel-oxd! Just want to verify if that this should be for DBC22-344, not 334.

 Just fixed it.


add support for vue-tsc 2.0.1

resolves #306

it's not easy to solve the issue by this way, vue-tsc 2 changed a lot. We need to figure out a new way to integrate with it.

I may be wrong, but based on the changelog there aren't really any breaking changes. If this plugin mostly just runs vue-tsc during build, then wouldn't it be enough to just import from the right place?

build part should be working fine even without this change, issue is with what is done during "patching" of lib


File Path

is it possible for us to set the download file path as default?

or can we get the path it downloaded

köyüne dön


Add Polish translation

Since it is not app for use with only GrapheneOS and it is available on Google Play, I thought it's time for adding translations. My friends use it successfully e.g. on Xiaomi/Samsung phones/tablets with LineageOS.

Needs to wait until after all strings are moved to resources and we set up an official translation team for the language. Please help with finishing up moving strings to resources. There's an open PR but it had some issues that haven't been addressed.


Feature request: an alternative mode to workaround jumpy header during scroll

We have some rather large tables (based on Handsontable) and the window scroll event does not trigger fast enough compared to the framerate (or something), leading to a jumpy header during scroll.
It takes lots of effort to make these tables lighter, and we want to use floatThead even without making them leaner.
This is the proposed behavior in this alternative mode:

Upon scrolling start, the floating thead is hidden.
Upon scrolling end, the floating thead is repositioned and shown.

Ok,
I don't have time right now to work on this, but you can test out how it looks locally by making changes here:
https://github.com/mkoryak/floatThead/blob/master/jquery.floatThead.js#L730
change that conditional to statement to:

            if(eventType == 'windowScrollDone'){
              $floatContainer.css('visibility', 'hidden');
            } else {
              $floatContainer.css('visibility', 'visible');
            }

that should probably accomplish what you proposed.

Hi, I tried the code suggestions but didn't have much luck.
For me, the issue is most visible on firefox mobile: the thead jumps around when you quickly scroll up/down, but only prior to the thead being affixed to the top of the viewport.  Once you scroll down far enough, and the thead reaches the top of the viewport and becomes affixed, then there's no more jankiness, works great.
gif screen captures:
from http://mkoryak.github.io/floatThead/examples/window-scrolling/

from my site https://tld-list.com using floatThead v1.4.4:

This issue is probably due to using "scroll-linked effects" (ie. binding to scroll events on browsers that use async scrolling).  Firefox gives a console warning when using scroll-linked effects and points to this:
https://developer.mozilla.org/en-US/docs/Mozilla/Performance/Scroll-linked_effects
I made rudimentary attempts at using both of their suggested implementations( position: 'sticky' , "scroll snapping") but neither seemed to work.
BTW thanks for the plugin, definitely the best "sticky table header" plugin out there.

I looked at that async scrolling article on mdn. Its pretty useless. If position:sticky was the real solution, then this plugin would not be needed. They also give some other suggestion which relies on non-standard or non-implemented techniques.
You should try playing with the value of the position option this plugin takes. Changing it might yield better scrolling performance (or might not)
Good luck.
Ill look into implementing this suggestion when I have time for enhancements again.

this page has 4 of 6 different scrolling/positioning modes on a single page:
http://mkoryak.github.io/floatThead/examples/caption-tag/
see what one looks least horrible to you. (it missing the responsive container modes, but those dont apply to you anyway)

Thanks. Tested using firefox mobile 49 on a Moto G4.  Overall, fixed vs. absolute positioning was about the same.
fixed:

absolute:

With fixed, the thead jumps around only before being affixed.  Once affixed, then its display is solid.
With absolute, the thead jumps a bit after being affixed.
When I get some more time I'll take a closer look and see if I can whip up a PR.

also tell me if adding this helps:
$window.on(eventName('wheel'), windowScrollEvent);
right under the line that does this:
$window.on(eventName('scroll'), windowScrollEvent);

Tried it, didnt seem to have any effect:
      } else { //window scrolling
        $responsiveContainer.on(eventName('scroll'), containerScrollEvent);
        $window.on(eventName('scroll'), windowScrollEvent);
        $window.on(eventName('wheel'), windowScrollEvent);
      }

any updates here? I am going to close this for now

Finally got around to looking at this. Seems the issue is that shitty browsers and cheap phones don't like having the thead's position set on scroll events.
So I changed the code a bit to only float and position the thead when necessary: when the top of the viewport is within the vertical bounds of the table.  If the viewport hasn't reached the top of the table yet, or has passed the bottom of the table, it unfloats the thead.
It's not perfect, but i think it's an improvement, screen cap: https://vid.me/VtRJ
The code is currently running on my site: https://tld-list.com
I've only manually tested this with window scrolling and in 'fixed' mode across a number of browsers/devices, so there's a very good chance the changes break something else.  Pull request on the way.


Double Scrollbar on long pages

With the CSS:
html {overflow: y-scroll}
the modal cause browsers with long pages to have a double scrollbars.

removing
html {
overflow-y: scroll;
}
or changing it to
html {
overflow-y: visible;
}
actually slove the issue

Just add to your css
body.modal-open{overflow:hidden!important;}
/optional/
.modal-open, .modal{padding-right:0!important}

just add this script if your bootstrap model have double scroll
this is the case if your css have html tag styling
$(function(){
$(document.body).on('show.bs.modal', function () {
$(window.document).find('html').addClass('modal-open');
});
$(document.body).on('hide.bs.modal', function () {
$(window.document).find('html').removeClass('modal-open');
});
});
this is the case if your css doesn't have html tag styling
$(function(){
$(document.body).on('show.bs.modal', function () {
$(document.body).addClass('modal-open');
});
$(document.body).on('hide.bs.modal', function () {
$(document.body).removeClass('modal-open');
});
});

same error here

I got same when dealing with bootstrap just add style with component directly
<InfiniteScroll style={{    overflow: "hidden"}}
dataLength={this.state.articles.length}
next={this.fetchMoreData}
hasMore={this.state.articles.length !== this.state.totalResults}
loader={}
>
in my case InfiniteScroll making this isuue so  I change his style its woked

i"m new to coding, I'm having a double vertical scroll bar issue, I don't even know where to begin, I don't know where to remove overflow, nor do i know how to locate the css thingy, please I need help it's really frustrating

i"m new to coding, I'm having a double vertical scroll bar issue, I don't even know where to begin, I don't know where to remove overflow, nor do i know how to locate the css thingy, please I need help it's really frustrating

just inspect and in css select div which you think having a scroll bar write property in  css overflow:"hidden" in inspect you will get autocomplete if scrollbar is hidden or not showing and ui working properly then this div is culprit then in code overflow :hidden him ok if you want any help you can text


Fix non working examples

There are a couple of examples that won't work because the necessary data is not available:

file_read
parquet_column_read

We should either provide a small sample with the tests or change the tests with some fake data so those examples can be executed.

I will add the sample data today for both txt and parquet formats

Hi, I didn't find the data you added. I wonder where it is. Thank you.


Add a Code of Conduct

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #948

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Documentation has been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated (with summary of main changes)

[ ] Link to issue (:issue:number) and pull request (:pull:number) has been added.

[ ] bumpversion patch has been called on this branch
[ ] The relevant author information has been added to .zenodo.json

What kind of change does this PR introduce?

Added the Contributor Covenant Code of Conduct

Does this PR introduce a breaking change?
No.
Other information:
Discussion and links in #948

Coverage remained the same at 91.458% when pulling 83e87ae5b69729206f5f70b951142dee9b535353 on add_code_of_conduct into ae3ac4d4d2da60b20e7cc5ee7fd4e404295250b8 on master.


[5.2] Fix an issue with fluent routes with uses()

As reported in https://github.com/laravel/framework/issues/13074.
This PR fixes the issue with routing using ->uses('Controller@method').

Should we add a test case for this?

I've already added two tests to check for the controller action after using uses(), do you mean a full test for the fluent routes thing? middleware() & name()?

Got it! Thanks


Add Initial Support for Cecotec Readywarm 8400

Added Support for Cecotec Readywarm 8400 Bladeless Heater.
https://www.amazon.co.uk/Consumption-Bladeless-Connected-Oscillation-Security/dp/B08LNQJTT4

@make-all  Just seen your commit.  Let me test with that as removing the "Off Preset" doesnt actually work as well as I would like.


Added a simple stats command

Why the two new packages???? EZ to format yourself???

Read CONTRIBUTING.md please

It's not rly needed


Dumping index fails because of security policy (CSP)

Hello,
i want to export one index to a json dump, but doing so get a long error message finishing with
\"kbnWelcomeTitle\">Please upgrade your browser

This Elastic installation has strict security requirements enabled that your current browser does not meet.

"
I have little not no knowledge of elastic. Is this an issue with how I use elasticsearch-dump? Or how our elastic server is configured?
Can someone point in min a direction?
thx,
Manuel

https://stackoverflow.com/questions/63897826/kibana-has-strict-security-requirements-enabled-that-your-current-browser-does-n
That's for Kibana, not Elasticsearch

For people having similar issues.
Yes this is an Kibana issue, and I ran into it, because I was specifying the kibana web ui address, instead of the elastic server address.


SONARCLOUD bug fixes

fixed potential bugs as suggested by sonar cloud.

Please follow commit message guidelines that can be found at https://github.com/Health-Education-England/dev-handbook/blob/master/styleguides.md#commit-messages

Please follow commit message guidelines that can be found at https://github.com/Health-Education-England/dev-handbook/blob/master/styleguides.md#commit-messages

@Judge40 sweet, i will also bear this in mind


Add Docker Compose file for local testing

Summary
Adds a local Docker Compose file to start an instance of Prefect that we can reach with terraform commands.
Closes https://github.com/PrefectHQ/terraform-provider-prefect/issues/186
Testing

Click to expand
$ terraform plan
╷
│ Warning: Provider development overrides are in effect
│
│ The following provider development overrides are set in the CLI configuration:
│  - prefecthq/prefect in /Users/mitch/go/bin
│
│ The behavior may therefore not match any released version of the provider and applying changes may cause the state to become incompatible with
│ published releases.
╵

Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # prefect_work_pool.example will be created
  + resource "prefect_work_pool" "example" {
      + base_job_template = jsonencode({})
      + created           = (known after apply)
      + default_queue_id  = (known after apply)
      + id                = (known after apply)
      + name              = "my-work-pool"
      + paused            = true
      + type              = "kubernetes"
      + updated           = (known after apply)
    }

Plan: 1 to add, 0 to change, 0 to destroy.

───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

Note: You didn't use the -out option to save this plan, so Terraform can't guarantee to take exactly these actions if you run "terraform apply"
now.

~/code/github.com/prefecthq/terraform-provider-prefect/add-testing-in-docker/examples/provider-install-verification on  add-testing-in-docker *?!
$ terraform apply
╷
│ Warning: Provider development overrides are in effect
│
│ The following provider development overrides are set in the CLI configuration:
│  - prefecthq/prefect in /Users/mitch/go/bin
│
│ The behavior may therefore not match any released version of the provider and applying changes may cause the state to become incompatible with
│ published releases.
╵

Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # prefect_work_pool.example will be created
  + resource "prefect_work_pool" "example" {
      + base_job_template = jsonencode({})
      + created           = (known after apply)
      + default_queue_id  = (known after apply)
      + id                = (known after apply)
      + name              = "my-work-pool"
      + paused            = true
      + type              = "kubernetes"
      + updated           = (known after apply)
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

prefect_work_pool.example: Creating...
prefect_work_pool.example: Creation complete after 0s [id=c088c80f-9612-4912-b03b-d5bea4facac7]

Apply complete! Resources: 1 added, 0 changed, 0 destroyed.

@parkedwards - looking at the PREFECT_API_URL in the acceptance tests CI file, I suspect we could replace the value (which I think points to dev or staging) and instead spin up a Prefect server instance using this new Docker Compose setup. That'd give us fresh environments to test against in CI and might help with transient errors related to environment pollution.
Does that sound right/fair? If so I'll open an issue and we can look into it more in a separate PR.

just a caveat - one wrinkle is that our OSS server isn't an exact match/port of our SaaS (which the current acceptance tests are mostly oriented towards supporting) - the OSS server has a subset of features and endpoints. so there may be some extra potholes we hit as we try to marry the tests<>this local prefect server instance. but i like where this is going

Great point. I added this note in https://github.com/PrefectHQ/terraform-provider-prefect/issues/192.


Inaccurate case in inference for new character

When I tried to change the source image to a new character, I got this result.

Is there a way to improve the quality with inference only? Or is it necessary to finetune the model?

Our model doesn't always generalize well to unseen subjects. Currently, the only way to encode the identity of the input subject is from CLIP embeddings. It works OK for general objects but isn't so reliable for human faces. We are still exploring more efficient ways to maintain the resemblance. A more important reason is that our training data only covers Asian subjects wearing red swimming caps, which means the hairstyles are not learned by the model. We are interested in trying our method on the RenderMe360 dataset, but I'm not sure when they will release the whole dataset with all subjects.

I think it's still an open problem. ID consistency with these models is a tricky thing. Even models like diffusion rig, headnerf fail when the test image is out of distribution though they are trained on diverse datasets.


A bunch of changes

Format the code, and add missing newlines between blocks
Collect arguments in Vec instead of HashMap to preserve call order
Let text_id be a normal variable or literal
Fix cut() in parser
Don't feature gate most on the implementation to make the
implementation simpler to read and understand
Add more error messages how to use the feature
Build the hashmap in one place

Wow, thank you for the changes!! Really means a lot, I don't how long I would have needed to write the parser like you did. Anyway one day I will know how to :)
Gonna merge it tomorrow morning.

I don't how long I would have needed to write the parser like you did. Anyway one day I will know how to :)

My bachelor thesis was about programming languages / compiler construction, so I might have had an unfair advantage. ;) Keep at it! IMO learning how a language actually works can be a lot of fun.


Roll Flutter Engine from bd44b58e3204 to 5bc1bc6d95c9 (6 revisions)

https://github.com/flutter/engine/compare/bd44b58e3204...5bc1bc6d95c9
2024-10-03<EMAIL_ADDRESS>Roll Skia from f074e2bd0aba to 68fea8aa589b (1 revision) (flutter/engine#55609)
2024-10-03<EMAIL_ADDRESS>Roll Skia from df2e478e9f8d to f074e2bd0aba (1 revision) (flutter/engine#55608)
2024-10-03<EMAIL_ADDRESS>Roll Skia from ba95ec201dfd to df2e478e9f8d (2 revisions) (flutter/engine#55607)
2024-10-03<EMAIL_ADDRESS>Roll Dart SDK from 14b931c40076 to 8b45ba3fdbfa (1 revision) (flutter/engine#55606)
2024-10-03<EMAIL_ADDRESS>Roll Skia from 03431ca9d337 to ba95ec201dfd (3 revisions) (flutter/engine#55599)
2024-10-03<EMAIL_ADDRESS>Vectorize rrect_blur (flutter/engine#55576)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC matanl<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Refactor clsx handling in Button component for improved readability and flexibility

This commit refines the clsx handling in the Button component by simplifying the logic for conditional class assignment. Previously, the className prop was being assigned using the nullish coalescing operator (??), which could be simplified by directly utilizing clsx for combining default classes with the custom className provided.
Changes made:

Replaced the ?? operator with direct usage of clsx to concatenate default classes with the custom className in a more readable and concise manner.
Ensured that clsx effectively handles undefined or null values for className, reducing the need for explicit checks.

The refactor improves readability, reduces unnecessary checks, and ensures that the Button component continues to function as expected with the correct class names being applied.
This change does not affect the functionality or appearance of the component but enhances code clarity, making it easier to manage and extend the class names in the future.

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


How to handle Double Click Event in Navigator and Check which TileSource is Double Clicked

Hi,
For viewer we can use scope.osd.addHandler('canvas-double-click', function(e)  to get the event source.
But how could we handle such event in navigator? Thanks

scope.osd.navigator.addHandler('canvas-double-click', function(e)  should work.
To find out which TiledImage is hit, you'll have to get them all from the scope.osd.navigator.world and loop through them comparing their getBounds() to the click location; we don't have a hit test function yet. Anyway, a navigator is basically just another Viewer, so the same APIs apply.


Refactoring has_n.rb so that it isn't one giant string

lib/neo4j/active_node/has_n.rb:91:7: C: Metrics/AbcSize: Assignment Branch Condition size for has_many is too high. [57.36/18]
      def has_many(direction, name, options = {})
      ^^^
lib/neo4j/active_node/has_n.rb:147:7: C: Metrics/AbcSize: Assignment Branch Condition size for has_one is too high. [48.76/18]
      def has_one(direction, name, options = {})
      ^^^

Oh my...  Working on that next...

Coverage decreased (-0.08%) to 98.5% when pulling 746095619b2b3c12e2ad3594a787ddec5127a470 on has_n_refactor into 31db269858015e6e47f529373aec806b01d8ce69 on master.

Coverage decreased (-0.03%) to 98.55% when pulling 84b83e5ac4b3d26c4a0b59fe37a179a669f1c734 on has_n_refactor into 31db269858015e6e47f529373aec806b01d8ce69 on master.

Coverage decreased (-0.02%) to 98.56% when pulling d58ed88ced479ba48afee7c36b4bf1085b43cdb1 on has_n_refactor into 31db269858015e6e47f529373aec806b01d8ce69 on master.

Coverage decreased (-0.03%) to 98.55% when pulling 95cdc70a5c6b8bf63f6217dd42ba662165605e3b on has_n_refactor into 31db269858015e6e47f529373aec806b01d8ce69 on master.

Coverage decreased (-0.07%) to 98.5% when pulling eff9604a399e69ce4d1d29f47728ecdaa9db04a2 on has_n_refactor into 31db269858015e6e47f529373aec806b01d8ce69 on master.

Coverage decreased (-0.01%) to 98.5% when pulling eb8f3c61372380c72b23613b5c0ae4e955569c3c on has_n_refactor into 3b8298a895b786317dba2a3fd2096254abda447d on master.

Coverage decreased (-0.01%) to 98.5% when pulling eb8f3c61372380c72b23613b5c0ae4e955569c3c on has_n_refactor into 3b8298a895b786317dba2a3fd2096254abda447d on master.

Coverage decreased (-0.01%) to 98.5% when pulling eb8f3c61372380c72b23613b5c0ae4e955569c3c on has_n_refactor into 3b8298a895b786317dba2a3fd2096254abda447d on master.

Coverage decreased (-0.06%) to 98.45% when pulling 400ef27d209f8de670ecd25d8113854f82c15287 on has_n_refactor into 3b8298a895b786317dba2a3fd2096254abda447d on master.

Coverage decreased (-0.01%) to 98.5% when pulling 6a83331b718ccad4406cc52a4e5f4de657b17d7b on has_n_refactor into 3b8298a895b786317dba2a3fd2096254abda447d on master.

Coverage decreased (-0.01%) to 98.5% when pulling c1629c03eceda5d7cf1e1a0bf36e22edcacaa741 on has_n_refactor into 3b8298a895b786317dba2a3fd2096254abda447d on master.


Cancel TernRename with escape

Hey,
Currently when you start a rename with :TernRename and decide not to by pressing escape, the variable is remove entirely.

Stumbled upon this again after trying Tern again. :P

@moll Ctrl+C works fine

Hey! Does Escape work, too, as the origin bug report was about? ^_^


fix: UnrolledList calls constructors only once on clear

Fixes #168

Fixes #168

https://travis-ci.org/github/dlang-community/containers/builds/769485926

Tests work, that's a problem in stdx.allocator.

Can you please rebase so that we can trigger the CI?


T8020 Wi-Fi Doorbell Chime & T8220 Doorbell 1080p Camera support?

Hi bropat, great work done with the integrations. :-)
I think I encountered a (new) Eufy product that is not added yet?
The T8020 Wi-Fi doorbell chime & T8220 doorbell combination.
It also does not appear in your list of devices.
I encountered this in my installation process with fuatakgun's Home Assistant add-on.
I did the setup exactly as described, but in the logs it says:
2021-11-28 11:04:43.297 INFO Eufy Security server listening on port 3000
2021-11-28 11:04:44.995 INFO No stations found.
2021-11-28 11:04:45.125 INFO No devices found.
2021-11-28 11:04:45.894 INFO Push notification connection successfully established
They are on the same network as my HA device.
Looks like the account is also connected properly.
Type number WiFi chime: T8020P10212301D6, Wi-Fi Doorbell Chime
Type number doorbell: T8220P6021210810, Battery doorbell 1080P
Manufacturer Part Number (MPN) for this set: E8220311
It looks the same as the T8222 model, only the number is different: T8220.
If you would like to add the product and need some data, let me know how I can be of assistance.
Greetings, Richard

Hi Richard,
thank you.
 Regardless of whether the device is supported or not, it should still appear as a station or device.
Therefore, I suggest you to share the devices again with admin rights and then try again.

Hi Bropat,
Thanks for the reply, really appreciate it.
After adding again it still did not work, but after that I switched from my second to my first account and that worked like a charm. Switching back to the second account, device found again. No idea why it was not working the first time, but nice! :-)
Thanks again!
Greeting, Richard


Wrong version in Cargo

Can't add it to cargo with just "oath = 0.1.3" - some old code with "feature" on the first line.
Have to write
oath = { git = "https://github.com/vlad003/rust-oath.git", rev = "f3f6b37" }

to get new code (and then it successfully compiles).
Maybe you need to update version in Cargo.

I published a new version (0.1.4), does that help?
The stuff that's on crates.io (even for 0.1.3) is exactly what's in this repo.


I recommend multi select input than just several checkboxs.

Is your feature request related to a problem?
Please select any NFT collection. You can see several tabs under the description section.
If you select the 'Holders' and 'Flippers' tabs, you will find several checkboxes arranged in a straight line above the table.
I hate this style because it doesn't looks like professional and especially it's displayed by several lines when mobile view.
Describe the solution you would like
I recommend Multi Select Input feature.
Describe alternatives you have considered

I like this style.
Screenshots

Are there opened related issues?
No response

Might be good to use custom TagInput component

Is your feature request related to a problem?
Please select any NFT collection. You can see several tabs under the description section.
If you select the 'Holders' and 'Flippers' tabs, you will find several checkboxes arranged in a straight line above the table. I hate this style because it doesn't looks like professional and especially it's displayed by several lines when mobile view.
Describe the solution you would like
I recommend Multi Select Input feature.
Describe alternatives you have considered

I like this style.
Screenshots

Are there opened related issues?
No response

Hi! thanks for reporting, collection view is under redesign and activity tab as well.
I will maybe post some screens once i have something - any feedback is welcomed

update @yeshealer

#5000
activity tab tbd :D


Added memory hard limit to minio

Has been stable for quite a while now. Will let it run a few more hours

@m88614 Still stable?

@m88614 Still stable?

Yeah, it was. I accidentally stopped it at like two/three in the afternoon, but I'll have it running during the night, and I'll update tomorrow morning. :)

Nah.. lets merge it


Font Question

Hi,
How can I use custom font ?
How can I create my atlas.inl ?
Thank you.

The demo program and its renderer are meant only as a small example of how the library should be used. In real projects you are expected to bring your own code for rendering rectangles, icons and text. Alternatively you could use a library like nanovg for rendering.
Once you're set up you can switch fonts by changing the ctx->style->font value. This value will be passed to the resultant MU_COMMAND_TEXT commands for all subsequently rendered text.


When will you release the code?

@gallenszl  hi, nice work! So when will you release the code?

I am not sure. Maybe at the end of this month.

Thank you for your concerns about our work. I am really sorry that we may delay one or two weeks to publish our code. After we making our code available, we will inform you through this issue.

Thank you for your concerns about our work. Code has been released.


Use etc/conf from apt/dpkg?

It seems the current shell-etc contains tons of non-hashbang configs, most of them are from upstream Debian packages. I think we can remove them add those files to .gitignore and let dpkg manage them.
And also, we should cleanup obsolete config files such as .java, lighttpd, apache2/conf-available.

I second this.
@KellerFuchs @daurnimator thoughts?
implementation wise we would  just .gitignore * and only add hashbang specific changes.
Then this is just an overlay repo and much easier for people to follow.

I'm okay with this.

@mayli Do you already have suggested commands to identify hashbang specific changes and mask everything else?

Let's move the discussion to the pull request.
I have no issue with conffiles for obsolete packages being dropped, obviously.
However, I don't think not-versionning default conffiles is a good idea; I described the main reasons in the PR.


Fix delegate object array thunk for cases when a pointer is passed by reference

@davidwrighton PTAL. This is a port of the Project N fix you have in flight.

You should use the boxable type on line 626 as well with the stobj. the use of pointer types for that instruction may also be confusing to backends, and there is no reason not to be conservative.


General improvements and modernizations

Here are some suggestions I have for your wonderful module, the most important of which is adding a luarocks rockspec which I would like to give you the honor to upload yourself to luarocks and by that make your module more accessible.
Also, You need to choose a license and put in the rockspec file. I wasn't sure what is it from the comment in the beginning of mpd.lua.
EDIT:

Fixed comment
Here they explain how to upload a rock to luarocks.org: https://github.com/luarocks/luarocks/wiki/Creating-a-rock#submitting-a-rockspec-for-inclusion-in-the-rocks-server

Hi @doronbehar,
Many thanks, I'm super happy if this small module is useful to others than me. I'll definitely take a close look at this PR this week-end!
Regards,
Alex.

Looking for your reply!

Hi @doronbehar!
So I finally managed to get some time to test and merge your PR, thanks for your patience :)
I took the liberty to adjust the documentation, follow semver, and adapt the rockspec file to use github directly.
I've uploaded the rockspec and I've been able to install and use the module. I would be glad if you could test it too!
Also I'm in the process of switching the license to what was a BSD-2-Clause to MIT, so that it should be more familiar for Lua users.
Regards,
Alex.

Tested and working. Cheers!


Interactivity.BehaviourCollection previous page values when navigaion to new page

Hi,
I am facing a memory leak issue with interactivity behaviour collection. It is not able to clear previous page records(objects) from the collection if user is navigating to new page. This will result in increase in app memory every time user navigates to new page.
Is there any way of clear those objects?
Regards,
Abhishek

Can you please provide a sample app for the problem you are seeing?

Here is the image of behaviour collection of the profiler. Is this helpful?
Will share a sample app soon.

@PedroLamas PFA sample for this issue.
App4.zip

I ran your sample app through the profiler with "Memory Usage" enabled, took a snaphot when it started, navigated to a couple of pages, took another snapshot, and finally clicked the GC button and took a final snapshot.
In the end, I got a single instance of the BehaviorsCollection which is what I expected, so I'm not sure if I'm missing somethin here, but seems the class is getting correctly GC'ed when it is not needed anymore.

@PedroLamas  BehaviorsCollection has a single instance, but it contains all the behaviors of the current page and previous pages. You can see the behaviors collection counter increasing by 3 when navigating to new page in the snapshots. Previous page objects are not clearing from the collection when navigating to a new page.
Are we have to dispose these objects manually? If yes, then how?

Pretty much the same result... I see the number rising, but it goes down as expected when I force GC.

Open the app, take snapshot:

Click a couple of times on navigation buttons, take snapshot:

Force GC, take snapshot:

Are we have to dispose these objects manually? If yes, then how?

No, it should be automatically GC'ed when they are not in use!

@PedroLamas  , I am facing same issue  but unable to figure out why these interactivity instances are not getting dispose, Can you please  check once , why these instances are not getting dispose?

@pbdev972 This is quite hard to help without knowing your specific scenario. Ideally, I would ask that you provide a sample app that shows the problem you are having so we can debug and try to understand what is the problem.

@PedroLamas , you can use same sample app  attached above and can check scenario -navigating to next page and can verify why these interactivity instances are still there or getting increase for multiple navigation , please let me know what else I can provide to debug this ? basically  my previous viewmodel instances are not getting dispose on navigation as they are having dependency on interactivity instance :(

First of all, the sample app above is using Microsoft.Xaml.Behaviors.Uwp.Managed version 2.0.0, please make sure to use the latest 2.0.1 as this contains some bug fixes.
I took the app and added some debug code to it so I could monitor the classes being created and destroyed, something like this:
public sealed partial class MainPage : Page
{
    private static int InstanceCount = 0;
    private int currentInstance = 0;

    public MainPage()
    {
        currentInstance = System.Threading.Interlocked.Increment(ref InstanceCount);

        Debug.WriteLine($" ++ created MainPage {currentInstance}");

        this.InitializeComponent();
    }

    ~MainPage()
    {
        Debug.WriteLine($" -- destroyed MainPage {currentInstance}");
    }
}

I used this same approach for MainPage, SecondPage, MainPageViewModel and SecondPageViewModel
Then I added a new button on both pages that when clicked, runs the following code:
private void Button_Click(object sender, RoutedEventArgs e)
{
    Debug.WriteLine(" ** GC.Collect");

    GC.Collect();
}

Run the app, output:
 ++ created MainPage 1
 ++ created MainPageViewModel 1

Click "Next Page", output:
 ++ created SecondPage 1
 ++ created SecondPageViewModel 1

Click "Next Page", output:
 ++ created MainPage 2
 ++ created MainPageViewModel 2

Click "GC", output:
 ** GC.Collect
 -- destroyed SecondPage 1
 -- destroyed MainPage 1

Click "GC" again, output:
 ** GC.Collect
 -- destroyed MainPageViewModel 1
 -- destroyed SecondPageViewModel 1

So what I see here is that the first GC collected the pages, and the second collected the view models.
It's normal that things don't get all GC'ed on first run, but in the end, everything was correctly collected and there was no memory leak!

From my tests with the demo app, it doesn't seem to be any memory leak.
If no further information is provided soon, I will consider this issue settled and close it.


Animation by changing aspect ratio of AspectRatio widget.

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
I want to animate the height or width when the property aspectRatio in AspectRatio widget is changed or size changed, like AnimatedContainer, AnimatedAlign
Proposal
I have tried with different approaches like AnimatedContainer or AnimatedSize widget. Animated size works perfectly when container expands it's size but does not animate work when it's shrink. So what's approach i must use with aspect ratio widget

Hi @fahad-whizpool. You can try using AnimatedBuilder for this:
https://api.flutter.dev/flutter/widgets/AnimatedBuilder-class.html
Please see https://flutter.dev/community for resources and ask questions like this, you may also get some help if you post it on Stack Overflow
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


Site Structure changed

DSpace Amrita changed their site structure. There is a need for updating webscraping logic to accommodate for the new changes in order to view the latest question papers.

I am interested in solving this issue.

I am interested in solving this issue.

I am interested in solving this issue.

Hi @beppvis, I have assigned you the issue, You have time till Jan 2nd to come up with a partially completed pull request, after which the issue will be reassigned to other participants, if there is no progress.

Can you provide me more details on the issue, about what is the expected .Thank you

Can you provide me more details on the issue, about what is the expected .Thank you

http://dspace.amritanet.edu:8080/xmlui/handle/123456789/16 is the site from where we are scrapping all the data in the application.
To explain with an example:
http://dspace.amritanet.edu:8080/xmlui/handle/123456789/220
In this webpage

The existing logic only scrapes the Collections in this community part, not Sub-communities within this community. Which misses out the latest papers. So I would like to include sub communities in the scraping logic.


Collab Editing Broken after Frontend Refactoring

Frontend crashed when creating sessions

This only happens after the frontend refactoring commits

The collab editing is still working until #1377, and broke after #1219. Tagging @blackening into this issue as well.

No Session id created when "invite" is pressed (Tested on chrome)

@JinHao-L Where are you testing this? It has only been deployed to GitHub Pages; the CS1101S deployment doesn't have it yet.

Hi @angelsl, it appears that the problem was on my end. It works once I cleared the cookies. The feature is working well on both GitHub Pages and the CS1101S deployment! 👍
Thanks for the prompt reply and sorry for any inconvenience caused!

@JinHao-L Yup, I just deployed the latter. No worries!


Is there a way to authenticate with an image resource's registry?

Expected Behavior
I am attempting to use an image as an input resource, and it actually seems as though it might work, however I am running into problems due to the fact that the registry requires authentication.
My use case is that I have another process outside of the Tekton pipeline purview which creates an image and pushes it to a registry. My Task then needs to access this image from the registry and copy some data from the running container to a volume mount.
Here is a simple example showing just the input.
apiVersion: tekton.dev/v1alpha1
kind: Task
metadata:
  name: deploy-webapp
spec:
  inputs:
    resources:
      - name: source-image
        type: image
    params:
      - name: SOURCE_PATH
        description: The location of the build output
        default: build
  steps:
    - name: copy-source
      image: ${inputs.resources.source-image.url}
      workingdir: /opt/app-root/source
      command: ['cp', '-Rp', '${inputs.params.SOURCE_PATH}', '/gen-source/']
      volumeMounts:
        - name: gen-source
          mountPath: /gen-source
      securityContext:
        privileged: true

I would like a way to specify authentication credentials for the pipeline when accessing the source-image in this example. Is this possible?
Actual Behavior
When running a Task for this I get an error message on the deploy-webapp-task-run container.

Failed to pull image "example-webapp": rpc error: code = Unknown desc = Error reading manifest latest in docker.io/library/example-webapp: errors: denied: requested access to the resource is denied unauthorized: authentication required

Steps to Reproduce the Problem

Apply the above Task to a cluster
Create a TaskRun that specifies a source-image on a registry requiring authentication
Apply the TaskRun to a cluster

Additional Info
While looking for a way to achieve this, I stumbled on this issue https://github.com/tektoncd/pipeline/issues/260 which is related. Given that I am getting authentication errors when the TaskRun executes, I think maybe that issue is resolved, but it is not clear since I have never made it past the authentication requirement.

hey @lance
Did you try with what's described in auth.md](https://github.com/tektoncd/pipeline/blob/master/docs/auth.md) : here and here (preferred imho).

@vdemeester so I have been trying to get this to work for a while now, but am still not successful. Let me provide a little more detail and perhaps you can tell me if I'm trying to do something that's just not possible yet.
A process external to pipelines (nodeshift) generates an ImageStream using s2i, resulting in an image in the internal repository at image-registry.openshift-image-registry.svc:5000/example-webapp. I would like to use a TaskRun which uses this image as input. My TaskRun looks like this.
apiVersion: tekton.dev/v1alpha1
kind: TaskRun
metadata:
  name: deploy-webapp-taskrun
spec:
  # Use service account with git and image repo credentials
  serviceAccount: pipeline
  taskRef:
    name: deploy-webapp
  inputs:
    resources:
      - name: source-image
        resourceSpec:
          type: image
          params:
            - name: url
              value: https://image-registry.openshift-image-registry.svc:5000/example-webapp
  outputs:
    resources:
      - name: runtime-image
        resourceSpec:
          type: image
          params:
            - name: url
              value: quay.io/lanceball/example-webapp-runtime

The pipelines service account has been grated the proper roles for accessing the registry per the documentation, and I have added the login secret per the instructions here. Do you think that this is possible and I have just done something wrong? Or am I pushing the boundaries of what pipelines can do at the moment?

Just a little more info - here is the log file from the deploy-webapp-taskrun-pod-nnnnnn.
  Warning  Failed     7m12s                 kubelet, crc-cvgnz-master-0  Failed to pull image "image-registry.openshift-image-registry.svc:5000/example-webapp": rpc error: code = Unknown desc = unable to retrieve auth token: invalid username/password
  Warning  Failed     7m12s                 kubelet, crc-cvgnz-master-0  Error: ErrImagePull
  Normal   Pulling    7m12s                 kubelet, crc-cvgnz-master-0  Pulling image "image-registry.openshift-image-registry.svc:5000/example-webapp"
  Normal   Created    7m12s                 kubelet, crc-cvgnz-master-0  Created container build-step-image-digest-exporter-copy-source-ms2nq
  Normal   Started    7m12s                 kubelet, crc-cvgnz-master-0  Started container build-step-image-digest-exporter-copy-source-ms2nq

And this is a little snipped of output showing that the authentication has been set up.
Init Containers:
  build-step-credential-initializer-x65rz:
    Container ID:  cri-o://1889ca244af0ae2db6298e7bd40b821a30e87f6e50bff65587125beeb4346d45
    Image:         quay.io/openshift-pipeline/tektoncd-pipeline-creds-init:v0.4.0
    Image ID:      quay.io/openshift-pipeline/tektoncd-pipeline-creds-init@sha256:9f60f7e1ea7c6f293c5297b46aa716264ab7dc02be13eb23f964c6b7f2212872
    Port:          <none>
    Host Port:     <none>
    Command:
      /ko-app/creds-init
    Args:
      -docker-cfg=pipeline-dockercfg-76v8c
      -basic-docker=basic-user-pass=image-registry.openshift-image-registry.svc:5000
    State:          Terminated
      Reason:       Completed
      Exit Code:    0
      Started:      Mon, 22 Jul 2019 22:49:52 -0400
      Finished:     Mon, 22 Jul 2019 22:49:52 -0400
    Ready:          True

The error message is unable to retrieve auth token: invalid username/password which makes me think my secret is wrong. Hmmm.

It just struck me that you are using the image resource url in the image, so our creds-init magic doesn't work in there. This is something we need to tackle somehow.
/kind feature

@vdemeester so I managed to get this to work. The resources, tasks and pipelines are here, if you are interested. It's actually not as complicated as I thought - mostly it was misconfiguration on my part. But the long and short is that I am able to specify an image from the internal repository as an input resource on a task and it works as I expect. I'm closing this. Thanks for your help.


Fixing installation steps in the README

And I added some useful instructions.

:+1:

:smile_cat:


Is there any way to change the style of pivot table by the mehod like SetColStyle?

I'm trying to make a pivot table with custom style (not default style from excel).  It seems SetColStyle("Sheet1", "H", style) can't style pivot table. Is there any way to fill color or change font color in any place of pivot table? Thanks for help !
ps. I tried to make custom style template from excel pivot table, but that function can only style some fixed place like 'title', 'sum row or col' etc. It's not enough for me. I prefer to adjust every cells by myself like  SetColStyle("Sheet1", "H", style) feature.

Thanks for your issue. The library doesn't support set pivot table cell style currently. We need to implement pivot formats for this. Contributions are welcome. I'll certainly accept this feature patch if anybody did that.

Hi
I am not sure if my issue is linked to this one.
I am trying to set conditional formatting on a column which has a pivot table in it.
The excel sheet creates the conditional formatting rule, but specifically excludes the cells in the pivot table. Is this normal behaviour?
I noticed that xlsxConditionalFormatting has a Pivot bool:
type xlsxConditionalFormatting struct {
XMLName xml.Name xml:"conditionalFormatting"
Pivot   bool          xml:"pivot,attr,omitempty"
SQRef   string        xml:"sqref,attr,omitempty"
CfRule  []*xlsxCfRule xml:"cfRule"
}
If we cant use conditional formatting is there any other way to format these columns/cells?
TIA

That is exactly related to the same reason as this issue. Currently, the library doesn't support changing the cell style in the pivot table. We need to implement pivot formats for this.

Thanks for clarification.
I did a quick test and if the Pivot field is set to true then the conditional formatting does apply to the pivot table.
	ws.ConditionalFormatting = append(ws.ConditionalFormatting, &xlsxConditionalFormatting{
		Pivot:  true,
		SQRef:  area,
		CfRule: cfRule,
	})

Would it be possible to set that as standard as a quick fix?

Okay. Would you like to create a pull request to fix that?


deb package

Hi,
I'm looking for a Debian package maintainer. It should be pretty simple but due to my small amount of spare time, I would like to have someone that would do that instead of me.
There is cargo helper cargo-deb here: https://crates.io/crates/cargo-deb that should help with packaging netscanner and a tutorial on how to make a package for a Debian is here: https://wiki.debian.org/HowToPackageForDebian
It would be much appreciated if someone would take this task under his/her wing.
Thanks a lot.

Hi @Chleba,
I will work on getting a Debian package for netscanner since I have a bit of free time. I have done a deb package before for a Rust program but nothing as complex as netscanner. I'll report back in the next few days with updates a progress.

Hi @DominicOrsi,
That is wonderful news. Thank You so much that You're putting Your valuable time into this. Let me know if I could help You with anything.

@DominicOrsi btw. recently Kali Linux adopt netscanner and because it's based on Debian You may use something from it ... maybe? Here is some commits: https://gitlab.com/kalilinux/packages/netscanner/-/commits/kali/master

I'll take a look at the Kali release and see what I can use from there. Thanks for showing me this!

Hey @Chleba
How are you wanting the Debian package to be released. As a PPA or just a .deb package that someone can download then install?
Been looking at both options and curious what route you would like to do?

I'm an old Debian guy but I haven't used it for more than a decade now so I am inclined to *.deb but I will completely leave that to You as a maintainer. You have my complete trust in this matter. Thank You.

Sounds good, I'll work on getting one up and running. Hopefully it can just be a GitHub workflow.

Hopefully will have a PR for .deb package by tonight. GitHub actions have me going :thinking:

Amazing. Thank You so much for that. Will you share a link for a PR and github action code? I would really love to see that and learn from it something new.

How do you update the release tag at the moment? Is it manually changing it in Cargo.toml i.e. 0.5.1 to 0.5.2?

Amazing. Thank You so much for that. Will you share a link for a PR and github action code? I would really love to see that and learn from it something new.

Yes, I will. I have a fork of the repo that I am testing everything with. You can see the GitHub action. It is almost done, I had to create a new GitHub action to create the deb package with GitHub actions. Was a fun side quest.

How do you update the release tag at the moment? Is it manually changing it in Cargo.toml i.e. 0.5.1 to 0.5.2?

Yes

Yes, I will. I have a fork of the repo that I am testing everything with. You can see the GitHub action. It is almost done, I had to create a new GitHub action to create the deb package with GitHub actions. Was a fun side quest.

That's great :). Thanks again. Learning something new every day is fun, right ? :)

It is, was great dive into GitHub actions and docker containers in GitHub actions.

Yea - docker is just great. I love it. I also have to take a look at you debian packager: https://github.com/DominicOrsi/cargo-deb-ubuntu.

Closing this Issue with PR #33 merged.
Awesome work @DominicOrsi :1st_place_medal:


fix docblock of shell extension point

Codecov Report

Merging #81 into master will increase coverage by 3.68%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
+ Coverage   74.58%   78.27%   +3.68%
==========================================
  Files          48       50       +2
  Lines        2621     2766     +145
  Branches      447      462      +15
==========================================
+ Hits         1955     2165     +210
+ Misses        630      579      -51
+ Partials       36       22      -14

Impacted Files
Coverage Δ

colcon_core/shell/__init__.py
100% <ø> (ø)
:arrow_up:

colcon_core/command.py
100% <0%> (ø)

colcon_core/__init__.py
100% <0%> (ø)

colcon_core/package_discovery/path.py
100% <0%> (+2.04%)
:arrow_up:

colcon_core/subprocess.py
48.78% <0%> (+2.43%)
:arrow_up:

colcon_core/environment/path.py
100% <0%> (+3.44%)
:arrow_up:

colcon_core/shell/bat.py
100% <0%> (+8%)
:arrow_up:

colcon_core/event/command.py
100% <0%> (+9.43%)
:arrow_up:

colcon_core/task/__init__.py
91.75% <0%> (+11.34%)
:arrow_up:

colcon_core/location.py
97.29% <0%> (+18.91%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


ENDOC-643 Upgrade Guide E7.1.2 ⇒ E7.1.3 - OLM & DIRECT/MANUAL (non-OpenShift)

still need to test

@nshaw @jyunmitch this isn't going on the website right now, and def not on the dummy page i created for it, but the steps/language are finalized, so i marked it as draft and the content is ready to go where it needs to

@Lyd1aCla1r3 you can close this one now since it's live on the entando.com in the CP Knowledge Base.


Updates the markup of some titles

Summary of Changes
Adds some heading tags on titles around the application
Tests
Updated tests
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes, including screenshots of any UI changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public python functions and the classes in the PR contain docstrings that explain what it does

[x] PR passes all tests documented in the developer guide

Codecov Report

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

@@           Coverage Diff           @@
##           master     #490   +/-   ##
=======================================
  Coverage   75.33%   75.33%
=======================================
  Files          42       42
  Lines        1784     1784
  Branches      167      167
=======================================
  Hits         1344     1344
  Misses        392      392
  Partials       48       48

Continue to review full report at Codecov.

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


Export-PnPTermGroupToXml empty for groups in "site collection" term store

Discussed in https://github.com/pnp/powershell/discussions/2988
Export-PnPTermGroupToXml  produces an empty string when used on term groups in the "site local" term store.
This behavior came up on a customer's live tenant and could be reproduced  on a current test tenant.
Term groups in the "global" term store are exported to XML as expected.
Using pnp/pwershell 1.12.0.
Please refer to the discussion cited below.

Originally posted by mfxits April  7, 2023
Hi,
I'm using PnP.Powershell 1.12.0 and want to export a term group from the "site collection term store" of the root site collection.
Let's say the group is named "ABC". It's the only group in the "site local" terms store.
Get-PnpTermGroup lists all groups from the global term store including the local "ABC".
For any global group piped to Export-PnPTermGroupToXml the result is as expected:
Get-PnpTermGroup -Identity "XYZ" | Export-PnPTermGroupToXml
<pnp:TermGroups xmlns:pnp="http://schemas.dev.office.com/PnP/2022/09/ProvisioningSchema">
  <pnp:TermGroup Name="XYZ" ID="8574c038-e187-435d-b2e5-e0520ffafea7" Description="" UpdateBehavior="Overwrite">
    <pnp:TermSets> ... </pnp:TermSets>
  </pnp:TermGroup>
</pnp:TermGroups>

But when I do the same with "ABC", the result is empty. It doesn't matter if I use the ID or name as parameter for "Identity".
Get-PnpTermGroup -Identity "ABC" | Export-PnPTermGroupToXml

Both groups returned from Get-PnpTermGroup have the type "Microsoft.SharePoint.Client.Taxonomy.TermGroup" and I can't see any difference between them.
I also played with Get-PnPSiteCollectionTermStore, but that command just returns the same term group object.
I searched for the topic but found nothing. Can anyone help please?

Markus

@KoenZomers seems like the issue is that local term groups are named {sitecollectiontermgroupname} and ID is set to 00000000-0000-0000-0000-000000000000 when Site Template is extracted, so the code fails to match on ID and name for that reason.
Get-PnPSiteTemplate -Handlers TermGroups -IncludeAllTermGroups -IncludeSiteCollectionTermGroup
gives you
<pnp:TermGroup Name="{sitecollectiontermgroupname}" ID="00000000-0000-0000-0000-000000000000" Description="" SiteCollectionTermGroup="true" UpdateBehavior="Overwrite">
Hence this works:
export-PnPTermGroupToXml -Identity "{sitecollectiontermgroupname}"
or
export-PnPTermGroupToXml -Identity 00000000-0000-0000-0000-000000000000

Have fixed this issue, will be available in the nightly builds as well as the next release !
Thank you @mfxits for raising this issue.
Thank you @AndersRask for investigating it.


some tags don't work.

Whenever I have a tag that has a dash in it - tumblethree won't download anything from the person's blog.  Example would be my-art or Joes-images-are-here.
No idea if tags can have spaces and if those download ok or not.  All my non dash - tagged tumblr sites download just fine. Using version 1.08.48 and this has been going on for a few versions now from what I recall.

If it is I have no idea what could be causing it. I just went to a new nsfw tumbler of an artist i'd seen years ago and when I put the tag my-art in his tumblr wouldn't download but when i removed the tag entirely it downloaded fine. That isn't the first tumblr that I've had this issue with. This is also a new install of windows 7, few months old at tops as my old computer died.
Tumblr  tag my-art
|
|
|  |
Tumblr
Tumblr is a place to express yourself, discover yourself, and bond over the stuff you love. It's where your inte...
|
|
|
On Wednesday, June 13, 2018, 10:25:57 AM PDT, Johannes Meyer zum Alten Borgloh<EMAIL_ADDRESS>wrote:

I've just created a test post in my test blog, tried to download a tag with a dash, no issue:
blog: https://privtumbl.tumblr.com/
post: https://privtumbl.tumblr.com/post/174555488822/wallpaperfx-lofoten-islands-sunset-wallpaper
tags: my-art, test, test test, myblog test
Maybe it's something else?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

This one also doesn't download anything with the tag in. It's also nsfw.
tag the-potato's-art-crapTumblr
|
|
|  |
Tumblr
Tumblr is a place to express yourself, discover yourself, and bond over the stuff you love. It's where your inte...
|
|
|
On Wednesday, June 13, 2018, 10:25:57 AM PDT, Johannes Meyer zum Alten Borgloh<EMAIL_ADDRESS>wrote:

I've just created a test post in my test blog, tried to download a tag with a dash, no issue:
blog: https://privtumbl.tumblr.com/
post: https://privtumbl.tumblr.com/post/174555488822/wallpaperfx-lofoten-islands-sunset-wallpaper
tags: my-art, test, test test, myblog test
Maybe it's something else?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I get the same issue with your test blog :( I hope we can figure this out. Is there a file or something I can send you for more info? Here's my setup screen for the blog. Maybe i'm doing something wrong?
On Wednesday, June 13, 2018, 10:25:57 AM PDT, Johannes Meyer zum Alten Borgloh<EMAIL_ADDRESS>wrote:
I've just created a test post in my test blog, tried to download a tag with a dash, no issue:
blog: https://privtumbl.tumblr.com/
post: https://privtumbl.tumblr.com/post/174555488822/wallpaperfx-lofoten-islands-sunset-wallpaper
tags: my-art, test, test test, myblog test
Maybe it's something else?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I can't see any of the images you've attached to your mail, but make sure that you use the latest version (v<IP_ADDRESS>) and not v<IP_ADDRESS> anymore as you've stated in your initial post. Tumblr made some ToS changes that resulted in necessary code changes within TumblThree to properly work on Tumblr blogs.
Additionally, are you sure you've check marked the "download reblogged posts" or that the posts you are missing are not reblogs in case you've not selected that box?

I'll update to 51 and see if there is a difference. I don't have reblogged posts checked because I only want the author's works which aren't reblogged :). Though I guess I do need to check that to test it out on your test site :/. I only attached a single image to the last post I sent you. No idea why you wouldn't get that. I'm sending these mails from Yahoo using Chrome browser, so they should be working ok.
On the artists blog it wasn't downloading any of his art at all unless I had no tag at all. Yet on other artists who have tags (none of which have a dash or space in the tag name) they download just fine. I'll update and see if that fixes the issue.
On Wednesday, June 13, 2018, 10:45:39 PM PDT, Johannes Meyer zum Alten Borgloh<EMAIL_ADDRESS>wrote:
I can't see any of the images you've attached to your mail, but make sure that you use the latest version (v<IP_ADDRESS>) and not v<IP_ADDRESS> anymore as you've stated in your initial post. Tumblr made some ToS changes that resulted in necessary code changes within TumblThree to properly work on Tumblr blogs.
Additionally, are you sure you've check marked the "download reblogged posts" or that the posts you are missing are not reblogs in case you've not selected that box?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Fixed audio loading

The audioplayer would continue to play when you visit another page or go back to the surahs list.

Yeah


test-branch/execute-job/quoted-authorization/9451eb8f-b592-42ec-ad91-4eb169a98e4f

PR for testing 3b5974d713f53767ce0dcacf7d139049d5cb3f53

/canonical/self-hosted-runners/run-workflows 3b5974d713f53767ce0dcacf7d139049d5cb3f53


Add dependencies for Melodic that rospilot needs

Description: PostGIS adds support for geographic objects to the PostgreSQL object-relational
database.
Links:

https://admin.fedoraproject.org/pkgdb/package/rpms/postgis/
https://packages.debian.org/stretch/postgresql-9.6-postgis-2.3
https://packages.ubuntu.com/bionic/postgresql-10-postgis-2.4

updated. Let me know if there's anything else!

updated. Thanks for the comments!

@mikaelarguedas any other changes you'd like to see?


Positive signed integers fail

The following test fails when added to /tests/test.rs:
#[test]
fn positive_signed() {
    #[bitfield(u32)]
    struct MyBitfield {
        negative: i32,
    }

    let v1 = MyBitfield::new().with_negative(-3);
    assert_eq!(v1.negative(), -3);

    let v2 = MyBitfield::new().with_negative(0); // <-- panics here
    assert_eq!(v2.negative(), 0);

    let v3 = MyBitfield::new().with_negative(3); // <-- here as well
    assert_eq!(v3.negative(), 3);
}

with:
---- positive_signed stdout ----
thread 'positive_signed' panicked at 'assertion failed: value <= 0xffffffff', tests/test.rs:125:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

The problem is that the condition of the debug_assert here
https://github.com/wrenger/bitfield-struct-rs/blob/ff0107b1c237348ca266d9a1fe2d041dde093f75/src/lib.rs#L469
does not work for signed integers. The mask will be a hexadecimal number which rust interprets "literally" (ie the condition is value< = 0xfffffff which Rust sees as value <= -1)

Yes, the mask overflows. The check generally does not cover signed integers very well, as it does not detect if an integer is too small.

I've improved the assert in 0.4.2 (https://github.com/wrenger/bitfield-struct-rs/commit/ceefaaa49b60b45ed7ab33ed3fb513b48fd17ca3)


Dont fail build when CodeCov fails

Codecov Report

Merging #3923 (b0a5d6c) into master (8bb0feb) will increase coverage by 0.00%.
The diff coverage is 94.87%.

@@           Coverage Diff           @@
##           master    #3923   +/-   ##
=======================================
  Coverage   94.85%   94.86%
=======================================
  Files         454      454
  Lines       49275    49312   +37
=======================================
+ Hits        46741    46778   +37
  Misses       2534     2534

Flag
Coverage Δ

servertests
55.83% <69.23%> (+<0.01%)
:arrow_up:

unittests
94.73% <94.87%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

moto/managedblockchain/exceptions.py
95.74% <92.59%> (-4.26%)
:arrow_down:

moto/managedblockchain/responses.py
97.64% <100.00%> (+0.09%)
:arrow_up:

moto/iam/models.py
95.27% <0.00%> (+0.14%)
:arrow_up:

Continue to review full report at Codecov.

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


Add initial common microgrid definitions & boilerplate

The inverter "ev charger" type should go, that was a mistake. The enum types for sensor and ev charger from the microgrid API should be added as separate enums.


TypeScript: Deceleration or Migrate

What pain point are you perceiving?.
Even when including @types/marked, including marked-xhtml doesn't support TypeScript.
Describe the solution you'd like
As Marked migrates to TypeScript, so should these new Marked support libraries. As they are small projects and the main project will remove the @types/marked dependency, I would request the migration route to reduce the need to add two packages per package.

@UziTech this can be closed i presume?

Yes, thank you!


fix: add null pointer judgment in client daemon network_topology.go

Add null pointer judgment in client daemon network_topology.go

Related Issue

Motivation and Context

Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.

For panic log:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0xf43915]

goroutine 18669 [running]:
d7y.io/dragonfly/v2/client/daemon/networktopology.(*networkTopology).pingHosts.func1(0xc000fd2f80)
	/go/src/d7y.io/dragonfly/v2/client/daemon/networktopology/network_topology.go:185 +0x535
created by d7y.io/dragonfly/v2/client/daemon/networktopology.(*networkTopology).pingHosts in goroutine 184
	/go/src/d7y.io/dragonfly/v2/client/daemon/networktopology/network_topology.go:164 +0x7d
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0xf43915]

goroutine 18670 [running]:
d7y.io/dragonfly/v2/client/daemon/networktopology.(*networkTopology).pingHosts.func1(0xc000fd3000)
	/go/src/d7y.io/dragonfly/v2/client/daemon/networktopology/network_topology.go:185 +0x535
created by d7y.io/dragonfly/v2/client/daemon/networktopology.(*networkTopology).pingHosts in goroutine 184
	/go/src/d7y.io/dragonfly/v2/client/daemon/networktopology/network_topology.go:164 +0x7d
stderr redirect at 2024-05-15 08:39:32.473712366 +0000 UTC m=+0.040284877

This PR may not resolve the panic.


Registering an empty repo causes an errors for rkpg

Expected behavior
An empty repo should be the same as an empty main branch.
Actual behavior
Porch and kpt alpha rkpg commands get confused if there is an empty repo:
jbelamaric@jbelamaric:~/proj/nephio-packages$ kpt alpha rpkg get
Error: branch "main" doesn't exist: reference not found

Information
kpt 1.0.0-beta.18
Porch 0.0.7
Using CSR with workload identity (though likely it doesn't matter).
Steps to reproduce the behavior
Create and register an empty Git repository. Run kpt alpha rpkg get. The Porch UI shows errors as well.

I created a main branch by pushing a README.md file to it, and now it's working.

This is at least partly intended behavior as described in https://github.com/GoogleContainerTools/kpt/pull/3024. But while we don't want to end up creating the first branch through porch (which makes it the default branch in github), it doesn't seem necessary to error out when we are listing packages.

This is fixed with https://github.com/GoogleContainerTools/kpt/pull/3472 and https://github.com/GoogleContainerTools/kpt/pull/3473


build: enable dependabot

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

Project coverage is 22.78%. Comparing base (4186308) to head (d07b980).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #102   +/-   ##
=======================================
  Coverage   22.78%   22.78%
=======================================
  Files          77       77
  Lines        6644     6644
=======================================
  Hits         1514     1514
  Misses       5130     5130

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


Can't save changes

Options window is so big that there is no way to click Okay button so I'm unable to save changes.

Thanks for raising this, the window not fitting on the screen is a known issue. I've included the "Improved Advanced Options" mod to attempt to correct this by rescaling the window. In this case using the current UI scale the window is still too big.
I've previously submitted a feature request for ShaguTweaks to be able to fit a larger amount of mods here so hopefully this might be added in the future.
One solution other than changing the UI scale would be to change the Improved Advanced Options window scaling value by opening the Lua file located in "\Interface\AddOns\ShaguTweaks\mods\improved-advanced-options.lua" and changing line 11 "AdvancedSettingsGUI:SetScale(0.88)" to a smaller value such as "AdvancedSettingsGUI:SetScale(0.6)". This should rescale the window so the bottom bit of the window is accessible.
Hope this helps!

An update to the Improved Advanced Options mod now allows moving and scaling which should finally resolve until additional columns can be implemented.

The Advanced Options do now scroll with the latest version. (https://github.com/shagu/ShaguTweaks/pull/33)


Renal and Brain bundle update according to monai 1.0

Fixes:
Dimensions in Convolution according to MONAI 1.0 upgrade.
Whole brain segmentation readme update.

/build


load_folder() panics when given an absolute path but load() does not

Bevy version
0.11
The issue is currently present in the main branch as well:
https://github.com/bevyengine/bevy/blob/b6a9d8eba7807b63be525e423911c0217706f13a/crates/bevy_asset/src/io/file_asset_io.rs#L112C4-L123C6
What you did
server.load_folder("C:\\Absolute\\Path\\To\\Folder\\")

What went wrong
thread 'Compute Task Pool (3)' panicked at 'called `Result::unwrap()` on an `Err` value: StripPrefixError(())', C:\<user>\.cargo\registry\src\index.crates.io-6f17d22bba15001f\bevy_asset-0.11.0\src\io\file_asset_io.rs:120:47

What were you expecting?

The folder contents are loaded into untyped handles or throw an error for invalid types

What actually happened?

Bevy panics when given an absolute path to a folder

Additional information
I'm using toml files as project files (like Unreal/Unity/Godot) and want to load and watch the files in the project folder for changes; this folder can be anywhere on the filesystem. The 'read_directory' function assumes that the provided path is relative to self and appends the provided path to the self root path which turns into "C:\Path\To\Executable\C:\Absolute\Path\To\Folder\" and panics.
 fn read_directory(
        &self,
        path: &Path,
    ) -> Result<Box<dyn Iterator<Item = PathBuf>>, AssetIoError> {
        let root_path = self.root_path.to_owned();

        // This panics if path is absolute
        Ok(Box::new(fs::read_dir(root_path.join(path))?.map(
            move |entry| {
                let path = entry.unwrap().path();
                path.strip_prefix(&root_path).unwrap().to_owned()
            },
        )))
    }

The load function works as expected but the load_folder variation panics which is inconsistent behavior. I think the best fix would be to auto-detect absolute paths and add the root path if it's not. If that's not in the cards then an optional boolean parameter for is_absolute could work.

What OS are you on? File system bugs are sometimes OS specific, so it's good to record it.

I'm on windows but it will have the same behavior on linux. The issue is that load_folder adds the root path regardless of if the provided path is absolute or not while load does not.
The workaround that I found is to set the root directory to the project path; if that's the intended use then maybe it would be better to add a line or two in the docs rather than a code fix.

he 'read_directory' function assumes that the provided path is relative to self and appends the provided path to the self root path which turns into "C:\Path\To\Executable\C:\Absolute\Path\To\Folder" and panics.

I don't believe this is true. According to the Path::join docs, a.join(b) will yield b if b is already an absolute path. The real error is that Path::strip_prefix will return an error if you attempt to strip a prefix that does not match the current path. You can't strip a from b if b does not already start with a.
To follow-up on your example:
let a: Path; // "C:\Path\To\Executable\"
let b: Path; // "C:\Absolute\Path\To\Folder\"

let c = a.join(b); // "C:\Absolute\Path\To\Folder\"
let d = c.strip_prefix(&a); // Error: `c` does not start with `a`

The minimal change that would resolve this bug would be to construct a relative path from a to b when b.is_absolute() is true. It looks like the most appropriate solution for that problem is using pathdiff::diff_paths.
fn read_directory(
    &self,
    path: &Path,
) -> Result<Box<dyn Iterator<Item = PathBuf>>, AssetIoError> {
    let root_path = self.root_path.to_owned();

    // This panics if path is absolute
    Ok(Box::new(fs::read_dir(root_path.join(path))?.map(
        move |entry| {
            let path = entry.unwrap().path();
            pathdiff::diff_paths(path, root_path).unwrap()
        },
    )))
}

I'm new to rust so I'm not used to a sane(er) standard library so this is nice to learn.


Images not loading from css

Perhaps based on the new Copy plugin? or webpack 2?

It's working for me I've modified your script quite a bit but essentially I'm doing this for it to work.
I reference my images as:
scss
.logo {
  background-image: url('/assets/images/logo-iberdrola.png');
}

My make script does this:
webpack.make.js
      // Copy assets from the public folder
      // Reference: https://github.com/kevlened/copy-webpack-plugin

      new CopyWebpackPlugin( [ {
        from: config.output.assets_src,
        to: config.output.assets_dst
      } ] )

      // Assets directories are defined as
      assets_src: `${ __dirname }/src/assets/images`,
      assets_dst: `${ __dirname }/assets/images`,

I figured this out but never updated the issue.
the css-loader (not sure about scss to be honest) does a require() with the images, so the path needs to be relative to the css file.
I am going to make some changes and remove the need of CopyWebpackPlugin (hopefully) with the html-loader so css and html images will have the same behavior.

OK! cool!


Demonstrate jwt auth support

Following from this comment on #40, here are the docs/test. I see you're working on something new with #33 but this should be a help to people in the short term.

Ugh, the error is to do with the underlying pyjwt library blowing up in the python3.2 environment. It appears to not be supported by them. Oh well, I don't suppose it is a big deal, feel free to close since this was only a 'nice to have'.

I've integrated rest-framework-jwt with djoser but need to copy the code to Github (we use Stash). Hopefully when I get time I can go through the steps I took and hopefully that'll be of some use.

Closing this one off. I am sure the maintainer will take care of this in due time.