Added missing webpack peer dependency

The html-webpack-tags-plugin hasn't specified webpack as a peer dependency. This lead to problems, because strict package managers can't resolve and install the correct dependency. This PR adds this missing dependency and fixes this problem.

pnpm workaround
If you are using pnpm, you can temporarily fix this problem using the following pnpmfile.js:
module.exports = {
  hooks: { readPackage },
};

function readPackage($package, context) {
  if ($package.name === 'html-webpack-tags-plugin') {
    $package.peerDependencies.webpack = '^5.30.0';
  }
  return $package;
}


Add support for IP wildcard mask type for address objects

Is your feature request related to a problem?
I'd like to be able to add or modify address objects with the type IP wildcard mask, which was introduced in PanOS version 9.0
Describe the solution you'd like
Support added for the AddressObject class to handle the IP wildcard mask type
Describe alternatives you've considered
Using python requests for API calls to create/modify these types of objects

Sounds good! We'll look into adding this.


include-ing a subchart's _helpers.tpl template yields top-level chart values

Hard to write a description for this one.
I have a top-level chart named foobar.  It uses the stable minio chart as a subchart.  No other charts in play.
minio defines a minio.fullname template.  Templates defined in this way are global, I've read.
My expectation is that therefore I can do something like this:
MINIO_HOST: {{ .Values.foobar.minio.host | default (printf "%s-svc" (include "minio.fullname" .)) | trunc 63 | quote }}

The intent here is: "Use the top level value whose path is foobar.minio.host (not a value of the subchart), but if that's not there, use the 'return value' of the minio.fullname template executed against the current context, appended with -svc."
I'd expect, for example, flabby-hummingbird-minio-svc in the default case.
Instead, what I see is: flabby-hummingbird-foobar-svc.

I'm struggling with the same thing from another stable chart. I just want to call e.g. the "minio.fullname" helper template directly in my parent. How are you meant to do that? I can't see it documented anywhere.

I just had a change merged that allows this - it's not perfect, but allows reuse - https://github.com/kubernetes/charts/pull/3972

Thanks for the workaround idea @kiall!
Global template are not much use if we can't call them globally, but just passing e.g. .Values.subchart context doesn't work because there is no .Values prefix. Probably we should have written the helpers to expect the chart context and not the root context.
e.g. {{ template foo.fullname .Values }}
Then subschart templates could be used directly, e.g.
e.g. {{ template bar.fullname .Values.bar }}

/remove-lifecycle stale

Then subschart templates could be used directly, e.g.
e.g. {{ template bar.fullname .Values.bar }}

That's not enough to make it always work - you'll be missing the other properties that are needed, e.g. .Chart and .Release are needed by the default helpers, and the data for .Chart exists nowhere that I can find inside of the template engine.
Helm could generate something like a .Contexts.SubchartName which we could then pass in:
{{ template bar.fullname .Contexts.bar }}

I'm running into this also, using stable/mysql as a subchart I'm like to be able to {{ template "mysql.fullname" .Contexts.mysql }} to configure the service name without having to redefine name and mysql.nameOverride etc
Right now I have a workaorund in the parent charts' _helpers.tpl
{{- define "mysql.mysql.fullname" -}}
{{- $name := default "mysql" .Values.mysql.nameOverride -}}
{{- printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" -}}
{{- end -}}

/remove-lifecycle stale
This IMO is still something Helm needs to solve to make chart dependencies more usable.

closing due to inactivity. Please have a look at the new library charts feature.


Error from outputs .tf when destroying

Hi,
It seems running destroy on this module reliably gives this error:
Error: Invalid index

  on .terraform/modules/eks_node_group/outputs.tf line 23, in output "eks_node_group_resources":
  23:   value       = var.enabled ? aws_eks_node_group.default.0.resources : []
    |----------------
    | aws_eks_node_group.default is empty tuple

The given key does not identify an element in this collection value.

Would you be open to also try this (simple apply & delete) of latest version (0.1.0) of this module?

Closed by #9


Localization (Phase 1 of 3)

Phase 1
Support for Translation of UI Interface.
NO Support for Translated Instance Names / Class Names / Spec Names.
NO Support for Translated LFG Messages.
NO Support for Matching of Translated Instance Names / Localized Abbreviations of Instances.
Also Relevant

https://github.com/Gogo1951/Groupie/issues/157
https://github.com/Gogo1951/Groupie/issues/158

Closing as stale.


Namespace not specified

A problem occurred configuring project ':pusher_channels_flutter'.

Could not create an instance of type com.android.build.api.variant.impl.LibraryVariantBuilderImpl.
Namespace not specified. Specify a namespace in the module's build file. See https://d.android.com/r/tools/upgrade-assistant/set-namespace for information about setting the namespace.

gradle-8.3

Same issue, I am facing from the past 2 days.

I am also facing the same issue when I upgraded the gradle and SDKs. Has anyone solved the issue yet?

try
pusher_channels_flutter:
    git:
      url: https://github.com/kishormainali/pusher-channels-flutter.git
      ref: master

@we-prajapati-c001 where do I put that config?

instead of fetching the package from pub.dev, use below in you project pubspec.yaml
pusher_channels_flutter:
    git:
      url: https://github.com/kishormainali/pusher-channels-flutter.git
      ref: master

Thank, I found out the problem that latest version of gradle don't support older version

try
pusher_channels_flutter:
    git:
      url: https://github.com/kishormainali/pusher-channels-flutter.git
      ref: master

This works so far


Fix rsync username when using local_mirror

We should use the configured username for rsync to connect
to the remote host.
Signed-off-by: Wanlong Gao<EMAIL_ADDRESS>

Can one of the admins verify this patch?

OK to test


Updated Chinese (Traditional) Translation

Updated Chinese (Traditional) Translation
CJK characters are shorter, so align the translation menu to the right to make it easier to be clicked.

CJK characters are shorter, so align the translation menu to the right to make it easier to be clicked.

But here you are aligning to the right in English as well, right? I am not sure if it makes sense.


HttpLoggingInterceptor crashes when dealing with websockets

Hello,
The current implementation (3.4.1) of HttpLoggingInterceptor always expects a response body to be present. That is not always the case. For example, when using okhttp-ws, the body of the response is null.
A NPE is thrown here, though there are plenty of other places where one will trigger as well:
https://github.com/square/okhttp/blob/413323f10d68e7581431b289971ec780cd0485a6/okhttp-logging-interceptor/src/main/java/okhttp3/logging/HttpLoggingInterceptor.java#L220

Dupe of #2811 and already fixed on master.


Improve CSET UI

The current CSET user interface (UI) has a number of issues. This ticket tracks efforts to resolve them.
UI Issues

[ ] https://github.com/MetOffice/CSET/issues/795
[ ] Information on welcome screen:
Models and configurations run, case dates, recipes selected.
[ ] Start on plot page
[ ] View 4 plots at once.
[ ] Postage stamp view for previewing many available plots.
[ ] Disable description by default
[ ] Additional information in description:
More information for derived diagnostic. Input model / file that plot came from. Maybe even path to output files.
[ ] https://github.com/MetOffice/CSET/issues/551
[ ] https://github.com/MetOffice/CSET/issues/535
[ ] https://github.com/MetOffice/CSET/issues/302
[ ] Interactive plots, e.g. toggling lines or extra info on click.
[ ] Consistent Y axis length across line plots.
[ ] Toggling with observations.
[ ] https://github.com/MetOffice/CSET/issues/552
[ ] https://github.com/MetOffice/CSET/issues/815
[ ] https://github.com/MetOffice/CSET/issues/811
[x] #932
[ ] https://github.com/MetOffice/CSET/issues/368
[ ] Order in menu isn't consistent.
[ ] Select dropdown series might be easier to find corresponding plots. Dropdown can then be filled in any order.
[ ] #940
[ ] Drop-down menu for dates to select cases

Examples of visualisation UIs

RES
OVELAYS
IMPROVER pages
Dust monitoring pages
ESMValTool output

Links in menu should be converted into larger click targets, so there isn't the unclickable space between them.


Not Following Symlink CWD

Hi,
I don't know if this is intended or not, but it's causing my issues because on Mac OS /tmp is a symbolic link to /private/tmp. On most Unix-y OSs and in AWS Lambdas it is just a regular directory.
The problem comes from mkdir.js:72, or thereabouts where the check for a directory does an fs.lstat (which treats a symlink as a file) instead of an fs.stat (which treats it as a directory).
Just wondering if there's a good reason for preferring lstat to stat? Just curious really, since it's mildly inconvenient (yes I know I should probably use os.tmpdir()).

I've run into the same problem when using npm link with a package that downloads a binary straight into its own directory (which is a symlink because of npm link). Have created #214 to fix it.


Allow Setting CRUD View Template By Creating File in App

I should be able to use a custom template for any CRUD action by adding a template file alone.
Example: a file edit.blade.php placed in my local resources/views/{$myBackpackPrefix}/{$myEntityName}/ should be autoloaded without any further configuration of the controller.
May be accomplished with something like this:
CrudController.php
public function edit($id)
{
    ...
    /**
     * Check For User-Extended View
     *
     * A blade file in `resources/views/{base_prefix}/{entity_name}/ will override
     * the default view.
     */
    $extendedViewPath = $this->crud->getRoute() . '/edit';
    if (\View::exists($extendedViewPath)) {
        return view($extendedViewPath, $this->data);
    }

    // render default view
    return view('crud::edit', $this->data);

Any issues with this implementation?

Hi @zschuessler ,
I did have this in mind when creating the packages :-) return view('crud::edit', $this->data); should already be doing that. That's what the laravel docs say and I've just tested it and it works. Just place your view into your app's /resources/views/backpack/crud/.
Or do you mean you'd like different "edit.blade.php" files, for different entities?
Cheers!

You can create this functionality by over-writing the show method on the crud controller:
public function show($id)
{
    $this->crud->hasAccessOrFail('show');

    // get the info for that entry
    $this->data['entry'] = $this->crud->getEntry($id);
    $this->data['crud'] = $this->crud;
    $this->data['title'] = trans('backpack::crud.preview').' '.$this->crud->entity_name;

    if(\View::exists('crud::' . $this->crud->entity_name . '.show'){
      return view('crud::' . $this->crud->entity_name . '.show', $this->data);
    }

    // load the view from /resources/views/vendor/backpack/crud/ if it exists, otherwise load the one in the package
    return view('crud::show', $this->data);
}

I use this pattern in my application as well and think this would be great to build into all of the CrudController methods!

@tabacitu Correct on the per-entity files! I now see I wasn't abundantly clear on that, sorry.
Something like @jsiebach did, but perhaps using the getRoute() method for the path instead. That allows you to get the controller name, too.
So with implementing the getRRoute() in my specific case, the path is views/account/product/show instead of views/product/show. Could be useful for when you have multiple controllers sharing an entity, I suppose.

maybe something like
return view($this->getTemplate('show'), $this->data);

this way you can add your own getTemplate function which returns customised views


foldl-1.3.2

foldl-1.3.2 (Gabriel Gonzalez<EMAIL_ADDRESS>@Gabriel439) is out of bounds for:

[ ] vector-builder-<IP_ADDRESS> (==1.2.*). No maintainer. @nikita-volkov. Used by: benchmark

Is this a duplicate of #2896 ?

Yes, thanks!
Duplicate of #2896


apply-system-agent-upgrader job on the rke2 clusters won't come up active when the PSP is set to restricted or restricted-noroot during cluster creation

Information about the Cluster
Rancher Server Setup

Rancher version: v2.6.4-rc3
Installation option (Docker install/Helm Chart): Helm Chart

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE1, v1.22.6, 1.3.7

Proxy/Cert Details: Selfsigned

Information about the Cluster

Kubernetes version: v1.21.9+rke2r1
Cluster Type (Local/Downstream): Downstream node driver 1 worker + etcd + cp RKE2

Describe the bug
When the Pod security policy is set to restricted, the job apply-system-agent-upgrader-on-anuketest3-pool1-c82d6bdc-8f147e for each node stays in In progress state with a forbidden error seen in the logs. Updated the full log under Result
To Reproduce

Create an RKE2 cluster with 1 node all roles - I tried with DO
Under Security >> select the restricted Default Pod Security Policy [Happens when psp is set to restricted-noroot as well]
Cluster comes up active

Result
From the UI explorer >> Workloads >> jobs >> under cattle-system namespace we can see the job apply-system-agent-upgrader-on-anuketest3-pool1-c82d6bdc-8f147e stays in In progress and won't reconcile.
Error:
Error creating: pods "apply-system-agent-upgrader-on-anuketest3-pool1-c82d6bdc-8f147e-" is forbidden: PodSecurityPolicy: unable to admit pod: [spec.securityContext.hostNetwork: Invalid value: true: Host network is not allowed to be used spec.securityContext.hostPID: Invalid value: true: Host PID is not allowed to be used spec.securityContext.hostIPC: Invalid value: true: Host IPC is not allowed to be used spec.volumes[0]: Invalid value: "hostPath": hostPath volumes are not allowed to be used spec.containers[0].securityContext.privileged: Invalid value: true: Privileged containers are not allowed spec.containers[0].securityContext.capabilities.add: Invalid value: "CAP_SYS_BOOT": capability may not be added]

Expected Result
Expected the jobs to come up active
Screenshots

Additional context
If we change the pod security policy to unrestricted policy the jobs come up active and the respective pods apply-system-agent-upgrader-on-anuketest3-pool1-c82d6bdc-8bjgpv are created for these jobs.

can be tested using 2.6.4-rc5

Verified on 2.6.4-rc6

Created two rke2 node driver clusters with 1 worker, 1 etcd + cp by setting the psp to restricted and restricted-noroot
Jobs come up active


python311Packages.georss-qld-bushfire-alert-client: 0.6 -> 0.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.georss-qld-bushfire-alert-client is: Python library for accessing Queensland Bushfire Alert feed
meta.homepage for python311Packages.georss-qld-bushfire-alert-client is: https://github.com/exxamalte/python-georss-qld-bushfire-alert-client
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.7 with grep in /nix/store/wgdvqy0s1syjnbmg21j0hy0210k1is1n-python3.11-georss-qld-bushfire-alert-client-0.7
found 0.7 in filename of file in /nix/store/wgdvqy0s1syjnbmg21j0hy0210k1is1n-python3.11-georss-qld-bushfire-alert-client-0.7

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
home-assistant-component-tests.qld_bushfire
python311Packages.georss-qld-bushfire-alert-client

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/wgdvqy0s1syjnbmg21j0hy0210k1is1n-python3.11-georss-qld-bushfire-alert-client-0.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.georss-qld-bushfire-alert-client https://github.com/r-ryantm/nixpkgs/archive/3ddb4329916ffdfc11c924a968ffb3748e28eb85.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/3ddb4329916ffdfc11c924a968ffb3748e28eb85#python311Packages.georss-qld-bushfire-alert-client

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/wgdvqy0s1syjnbmg21j0hy0210k1is1n-python3.11-georss-qld-bushfire-alert-client-0.7
ls -la /nix/store/wgdvqy0s1syjnbmg21j0hy0210k1is1n-python3.11-georss-qld-bushfire-alert-client-0.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

python311Packages.georss-qld-bushfire-alert-client

Maintainer pings
cc @fabaff for testing.

Result of nixpkgs-review pr 291311 run on x86_64-linux 1

1 package failed to build:

home-assistant-component-tests.qld_bushfire

2 packages built:

python311Packages.georss-qld-bushfire-alert-client
python311Packages.georss-qld-bushfire-alert-client.dist


Compilation failure on Android in latest 0.31 release

Seems that wasmtime-runtime tries to use map flags that don't exist in Android:
error[E0599]: no associated item named `FIXED` found for struct `MapFlags` in the current scope
  --> /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/wasmtime-runtime-0.31.0/src/instance/allocator/pooling/unix.rs:21:63
   |
21 |             rsix::io::MapFlags::PRIVATE | rsix::io::MapFlags::FIXED,
   |                                                               ^^^^^ associated item not found in `MapFlags`

Could we use another way / method to decommit memory on Android? cc @sunfishcode

It looks like Android doesn't support MAP_FIXED. It looks like we'll need to add a way to disable the pooling instance allocator at compile time, and have it disable on Android.

I can work on putting the pooling allocator behind a feature gate that you can opt out of for Android.

@peterhuene yes, that fixed it, thanks a bunch! Would it make sense to do a patch release to get that fix in the stable branch? cc @tschneidereit @alexcrichton

Would it make sense to do a patch release to get that fix in the stable branch?

I'll leave it to Alex to comment on that, because I can't judge all that well how much effort that'd be.
I wonder if we should have (an) Android CI configuration(s) though? This seems pretty likely to regress again otherwise. @bnjbvr, would you perhaps be up for adding this to our CI setup to ensure that the builds keep working for you?

We don't really have a clear/crisp definition of supported targets for Wasmtime, so to me it's somewhat unclear on whether we should do a patch/etc. I don't think we can release main as is since it technically contains semver-breaking changes to both Cranelift and Wasmtime, but we could do a patch-release where this is backported.
Ideally we would have a list of targets along the same lines as rust-lang/rust where tier 1 is tested on CI, tier 2 is "guaranteed to build", and tier 3 is everything else. Changes to tiers 1/2 would necessitate a patch release of some kind, so whether or not doing a release here makes sense depends on how we want to think about Android.

Ok, no worries, we can point at a specific commit in a fork of Wasmtime for our embedding, it is slightly less ideal but doable (we'll likely require other fixes, like https://github.com/bytecodealliance/wasmtime/commit/12bfbdfaca3ecd3da95b30670df1b71fdba01be0, which otherwise seem to break debugging entirely).
Definitely it would be nice to have a list of officially supported targets, for documentation and for providing stability guarantees.

I am running into this same issue for freebsd.
error[E0599]: no associated item named `FIXED` found for struct `MapFlags` in the current scope
  --> /root/.cargo/registry/src/github.com-1ecc6299db9ec823/wasmtime-runtime-0.31.0/src/instance/allocator/pooling/unix.rs:21:63
   |
21 |             rsix::io::MapFlags::PRIVATE | rsix::io::MapFlags::FIXED,
   |                                                               ^^^^^ associated item not found in `MapFlags`

@densone Switching to the latest version of wasmtime, which is 0.33 at this time, should fix it.


Fairy entity registration

Hi :)
First of all thank you for porting this wonderful mod to minecraft 1.7.10 :)
Second, I noticed that the mod conflicts with Orespawn: they both add a mob simply called "fairy" and this causes Forge to crash.
Is there a chance you can change the name to something not conflicting like "fairymod.fairy" ?

Huh? I... was fairly certain I was doing precisely that because I knew other mods added fairies... checking this out in further detail.

Works for me... make sure you're using a current build of the mod? Old revs did not register the entity safely.


Activate Not Working in Menu

Hi all
I have a problem with menu item activation. I have a below code

{{item.name}}

When I click, this is the function I have:
        function pageItemSelected(item){             vm.itemsPerPage = parseInt(item.name);             $('divItem_' + item.id).addClass('active');         }
But it never activates! How coud I fix it?
Thanks.

This does not seem to be a problem with Semantic-UI, but with AngularJS. I have no experience with Angular so I do not know exactly how its rewrite rules work, but what I suspect is that the function scope is not set up right; Maybe it might help to change the ng-click part in a function like this: ng-click="function(){vm.pageItemSelected(item);}"
Again, I might be completely wrong.

In case you haven't fixed it already, please repost your question to either ProjectTalk or stackoverflow. The Github Issues Tracker should only be used for bug reports and feature requests.


dispatcher: Make debug log clear that dispatcher updated the task

Changes the debug message to make it clearer that swarmkit actively updated the task status itself, rather than passively detecting the status change. I'm still familiarizing myself to the swarmkit architecture and lexicon so please let me know if this can be improved.
Closes #2208
Signed-off-by: Riyaz Faizullabhoy<EMAIL_ADDRESS>

What's actually happening here is that the agent reports a status change, and the dispatcher commits that change to the data store.
Perhaps this could be worded "dispatcher committed status update to store".

@aaronlehmann: ok got it, thanks for the explanation.  Pushed with your suggestion

LGTM


feat(alerting): Add overrides for Ntfy provider

Summary

Add group overrides to Ntfy alert provider (context: https://github.com/TwiN/gatus/issues/96)
Checklist

[x] Tested and/or added tests to validate that the changes work as intended, if applicable.
[x] Updated documentation in README.md, if applicable.

Thank you for the contribution!


(feature) Generate Bork file from current system status

Idea based on this tweet: https://twitter.com/russshanahan/status/1365340483418017803 (thanks @roostr!)
The idea: a new Bork subcommand (bork inspect maybe?) that runs a new action on a type, outputting to stdout a list of Bork-compatible assertions to make the system look like that. So for the brew type it would output a list of ok brew [app-name] assertions for every package currently installed with Homebrew; for the mas type it would output a list of ok mas assertions for everything mas reports is installed from the App Store.
You could then output this to a shell file, remove the ones you don't care about, and use it as the basis to generate a new Bork script.
Not appropriate for every type so would not be required on every type; ~should just exit 0 when not implemented.~ I changed my mind: I think it makes the most sense to leave this as-is — exiting 1 when a type doesn't respond to inspect. The inspect call will have to check for some preconditions to be able to work (e.g. the brew inspector needs to check Homebrew is installed) so we can't just run this on every type Bork responds to; a user might not be interested in inspecting a given type on their system.
inspect implementation

[ ] apk
[ ] apm
[ ] apt
[x] brew-tap
[x] brew
[x] cask
[x] gem
[ ] go-get
[ ] group
[ ] iptables
[x] mas
[x] npm
[ ] pip
[x] pip3
~pipsi~
[x] shells
[ ] user
[ ] yum
[ ] zypper

I have added the inspect command and started off implementing it on some types. In this comment I'll keep a list of types where inspect makes sense, to track progress implementing it.

[ ] apk
[ ] apm
[ ] apt
[ ] brew-tap
[x] brew
[x] cask
[x] gem
[ ] go-get
[ ] group
[ ] iptables
[x] mas
[ ] npm
[ ] pip
[x] pip3
[ ] pipsi
[ ] shells
[ ] user
[ ] yum
[ ] zypper

I have decided against doing this with pipsi myself for now, as pipsi is unmaintained upstream. I would still welcome a PR to add this functionality to the pipsi type, though chances are the pipsi type will be deprecated come 1.0.


File manager, 8.2RC2: back button on first page of file list has link - error

Wondering why the list of files in a folder in file manager provides "previous" and "next" buttons instead of the default pagination, i noticed that the "previous" button on the first page (showing the first 10 files), which should not link to anything, points to "www.my_domain/index.php/dashboard/files/search#" which results in an error when clicked.

#5628 should fix this issue

Wondering why the list of files in a folder in file manager provides "previous" and "next" buttons instead of the default pagination

I guess for performance reasons: the default pagination requires that we know in advance the total number of results. In the case of the file manager, that means that we need to check every file for permissions, and that's really a time consuming operation if you have thousands of files. This new pagination system implies that we have to check only the files in the current page.

Ok, but strangely, if i run an empty search inside the file manager, the pagination is here again. And this was running very fast.

@mlocati is correct – it's the permissioned pagination that was killing our performance in the past. The next/previous fixes that.
However you have found a weird quirk – if you're logged in as the super user admin, we serve the old style pagination when searching files – since we know you'll be able to view them – but we didn't when serving up the file manager folder list view, leading to the weird inconsistency with the pagination. I have fixed that in https://github.com/concrete5/concrete5/commit/1d73f41cc1f3ac16c0a47c3b9dea102e01015e10


Allow more sections to be hidden on dashboard

Thanks for reporting an issue for Streama, please review the instructions below before submitting the
issue.
YOUR ISSUE WILL BE CLOSED IF IT DOES NOT USE THIS TEMPLATE OR IT IS INCOMPLETE!
Feature requests can be done without this template.
If you have multiple things to report please create one issue per item.
Please use english when reporting issues.
If you are unsure about something and the issue is more of a question a better
place to ask questions is on gitter.
DO NOT use the issue tracker to ask questions.
Please fill out the sections below the header and review the task list below.
Task List:

[X] Issue description provided
[X ] Steps to reproduce provided
[ ] Stacktrace (if present) provided
[ ] Screenshots (if UI related) provided
[X] Full description of the issue provided

PLEASE REMOVE THIS HEADER SECTION BEFORE SUBMISSION

Issue description
I have a use case where I'd like to be able to hide the "discover movies" and "discover shows" sections.
Steps to Reproduce

Open up streama dashboard

Expected Behaviour
Ability to hide movies and shows. I have no movies or shows added so there are empty sections filling up page space. Custom videos are at the bottom.
Actual Behaviour
Empty movie and shows category comes up before my custom media.
Environment Information

Operating System: Latest Streama war

Hiding the other sections has been added to master, will be in the next release.
Section names:
recommends
discover-shows
discover-movies
discover-generic

Looks like this is not working properly.
Streama 1.8.0 on Firefox latest (71) on Mac.
Several attempts to hide one or more section(s) following the given syntax (with several attempts), they still appear (even after logging out / back in)


StackOverflowError with SpringBoot 3.1.5 and jasypt-spring-boot-starter 3.0.5 with ${cors.origins.allowed:#{http://localhost:4200}}

Hi
Upgraded from SpringBoot 2.6.6 with
     <dependency>
            <groupId>com.github.ulisesbocchio</groupId>
            <artifactId>jasypt-maven-plugin</artifactId>
            <version>3.0.4</version>
        </dependency>

To SpringBoot 3.1.5 with
       <dependency>
            <groupId>com.github.ulisesbocchio</groupId>
            <artifactId>jasypt-spring-boot-starter</artifactId>
            <version>3.0.5</version>
        </dependency>

It used to work with application.properties
cors.origins.allowed=${cors.origins.allowed:#{http://localhost:4200}}
But after the upgrade i get a  java.lang.StackOverflowError
similar to https://github.com/ulisesbocchio/jasypt-spring-boot/issues/209

did you resolve this problem, because i'm having the same one for days now

@boufenzi I changed the way i use it. Now i do
...allowed-origins: '*'
and i Overwrite the value on the environment with the correct value.
so this Bug is still open.


BLOCKED: feat: intermediate state aggregation through a persistent AxEffects object

Description:
Stacked on

[ ] #200
[ ] #207
[ ] #211

This PR changes sym_n to keep a single AxEffects object which is updated (rather than replaced) each step.
Currently, this significantly slows down symbolic simulation, causing timeouts when simulation more than ~60 steps of SHA512, and thus breaking a lot of proofs.
Testing:
What tests have been run? Did make all succeed for your changes? Was
conformance testing successful on an Aarch64 machine?
License:
By submitting this pull request, I confirm that my contribution is
made under the terms of the Apache 2.0 license.

This is currently blocked, waiting on a diagnosis of https://github.com/leanprover/lean4/issues/5610


Attention Heatmap?

How do I create the attention heat map based from Attention model like the image below?
PS: I can access the graph through tfdebug but I don't know what I am searching for

@oahziur I run into two issues with the template code for Attention summary generation:
When beam search is enabled, the above throws an error. I'm assuming beam search and alignment history are not compatible right now.
When I disable beam search, I get the following error:
Traceback (most recent call last):
  File "/home/nave01314/IdeaProjects/tf-nmt/main.py", line 19, in <module>
    infer_model = model_builder.create_infer_model(hparams)
  File "/home/nave01314/IdeaProjects/tf-nmt/model_builder.py", line 125, in create_infer_model
    tgt_vocab_table=tgt_vocab_table)
  File "/home/nave01314/IdeaProjects/tf-nmt/models.py", line 41, in __init__
    self.attn_summary = self.make_attention_summary(final_context_state)
  File "/home/nave01314/IdeaProjects/tf-nmt/models.py", line 192, in make_attention_summary
    attention_images = (final_context_state.alignment_history.stack())
AttributeError: 'tuple' object has no attribute 'stack'

Hi, @ngcabang !
I faced the same problem now. Have you solved your issue?
Thanks

Same here, I would like to get each alignment map for each of the top-k translations when beam search is activated, is this planned any time soon? If not, what should be changed for this to work @oahziur ?

@ttrouill @nave01314 Actually, I am not sure if the attention image for BeamSearch will be correct out of the box since we traversed the decoding tree in BeamSearchDecoder to get the final results, so I guess we will need to do the similar gather_tree operation in the end. We can revisit this after beam_search and alignment_history are compatible.
@nave01314 Can you print the final_context_state.alignment_history see what's inside the tuple?

@oahziur Ok thanks for the precisions, is there an already existing branch working on beam_search and alignment_history compatibility that I can follow/contribute?

@ttrouill I don't know if there is any existing branch or pull request on this, but I think you can follow this issue to get update if there is any.

Looks like beam_search and alignment_history compatibility issue will soon be [resolved and merged]https://github.com/tensorflow/tensorflow/pull/13312#pullrequestreview-87614206)

@nave01314 the same problem, how do you solve it?
I do not use beam search, but I still can't get the alignment.

@oahziur Dear, I can see the attention image by tensorboard now , But there this is only one image for one step,  does that mean the image is for all training data?
How can I get  image for one given sentences?


feat(release-runbook): added some UX improvements while running through the runbook

Just a few more changes while working through the patch release process

👋 Hi hi @dorbin, comments addressed!


Updating func to 1.7 release

Changes

:broom: updating to 1.7 release

/kind cleanup

Just updating to latest version

Release Note

Docs

@lance @maximilien added Linux arm64 and checked the links.. please confirm that the version is ok.

/approve


Remove undefined env

Without determine_extension_dir.sh, $extension_dir is not define yet.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for the contribution


CLI should not use non-public Cilium APIs

The following code uses a non-public API from https://github.com/cilium/cilium in order to extract configurations out from an individual agent:
https://github.com/cilium/cilium-cli/blob/e97bddd0e98139d70416371c50e3740625e9e4d7/connectivity/check/features.go#L274
There are a few issues with this:

This is the internal structure used by Cilium and is subject to change without warning
Over time, Cilium will migrate away from including all daemon configurations in this structure towards per-Cell configuration, so it will not contain all configurations.
This ties the Cilium CLI to a specific version of the internal agent code, which may cause compatibility issues between multiple versions.

https://github.com/cilium/cilium/pull/27352 points out an example where a new flag was introduced during cilium development, then cilium-cli was synced to cilium, then the flag was changed. This caused connectivity failures because the type was made invalid:
connectivity test failed: unmarshaling cilium runtime config json: json: cannot unmarshal bool into Go struct field DaemonConfig.HubbleRedact of type []string

This code should probably unmarshal the configuration into a more liberal type to account for differences between Cilium versions.

Would it be reasonable to do this autodetection from the helm configuration level or using cilium-config ConfigMap rather than executing into a Cilium pod to assess desired state from the agent?

yeah i was thinking we should just look at cilium-config config map 💡

FWIW, the PR adding the runtime feature detection (https://github.com/cilium/cilium-cli/pull/1022) has some context on why this is needed and why the information cannot easily be gathered from the ConfigMap:

Previously, we have been relying on the Cilium ConfigMap to detect the
presence of features. Relying on the ConfigMap has however some
downsides, as one has to know the default value of a certain feature if
the option is absent from the ConfigMap. To make things worse, this
default value is also version dependent in some cases.

/cc @gandro

that's a good point. i guess helm values will have the same issue? 💭

struggle

There was also an issue with the default used for the tunneling feature, which tried to use the config-map value if available (https://github.com/cilium/cilium-cli/pull/1870/commits/da5c0c9eb9cff9e8baebabb0d9f16acd371c38db).
Therefore I tried to document the current guidelines in the code. I mostly copied the descriptions from the commits, bit I thought it makes sense to also have them in code (https://github.com/cilium/cilium-cli/pull/1870/commits/7c54c5fc7b20b26e4bc434a7b58635161d929b30).

I agree that relying on the agent-runtime-config.json is a hack. But as already mentioned, we want to know the runtime configuration of the agent, not what has been configured via Helm or ConfigMap, since Cilium does a lot of auto-detection or runtime-dependent configuration of certain features. I opted to use the JSON file because it was already there and worked with existing versions of Cilium (something that the CLI also needs to consider).
I think one way  answer here is to start exposing per-cell configurations in the Cilium API. We already have an API endpoint that exposes a subset the global runtime daemon configuration https://github.com/cilium/cilium/blob/95e25bfb132750e97304dfbe0f5770f4b6debd96/api/v1/openapi.yaml#L74-L85
The /config endpoint is lacking per-cell configurations, but that can be solved by having cells exposing their configuration via API too. It could even be semi-automated, i.e. have a registry where cells register themselves as "having some configuration" which then automatically exposes that state via an API endpoint.

I wonder if we could put some infrastructure in place so that those configuration API endpoints are somehow tied to feature stability in the code. For instance, in the case from the issue description, the feature is brand new, so could be considered alpha. Maybe for alpha options we don't expose it in the API because it's subject to change. Then we wouldn't trigger this overall issue in the first place because Cilium-CLI wouldn't be relying on an alpha setting having a specific type or format. For other settings, we could graduate the features themselves to beta/GA and somewhere later in that process we start exposing the flags as a public API from Cilium.


2.0 beta-3 TextView flickers on size animation in MotionLayout

I'm currently trying to create a toolbar animation with the MotionLayout. Everything works fine except the fact that the TextView with the height and width "wrap_content" is partly flickering when the size of the TextView is increased.

Solved!!
Width to wrap_content - wrap_content to Width
wrap_content to wrap_content  not working, fix bug pls.

Still need wrap_content to wrap_content. just add +1 to witdth animation pls.


Fixing travis build

clean makes it build, and the scheme is not configured for building, so it should just be test
tested locally after a git clone and it worked

I signed it.

I signed it.

Thank you for this fix!


Memory.Pin doc needs to be filled out

We should have a summary, remarks and examples on how to use this API. What am I supposed to do with the MemoryHandle? What are the best practices?  etc.

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

ID: 1c0c5532-ec7e-21f2-9e33-8b1c89073abf
Version Independent ID: 851ecd39-f742-6f2a-a2d9-c1a7f1dd0f82
Content: Memory<T>.Pin Method (System)
Content Source: [xml/System/Memory1.xml](https://github.com/dotnet/dotnet-api-docs/blob/master/xml/System/Memory1.xml)
Product: dotnet-api
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

@eerhardt  Hi! Thank you for opening this issue. We just shipped the docs for Memory<T> and IMemoryOwner<T> in PR dotnet/dotnet-api-docs#548 which should be published today.
I'm going to close this issue. Thanks again.


fix deleted workflows triggering

Description
Fixes #402  .
Fixes deleted workflows triggering
Status
Ready
Types of changes

[x] Non-breaking change (fix or new feature that would not break existing functionality).
[ ] Breaking change (fix or new feature that would cause existing functionality to change).
[ ] New tests added to cover the changes.
[x] All tests passed locally.
[ ] Documentation comments included/updated.

Needs int tests before merging @jackschofield23


copr: add vectorized truncate int

UCP #5751
Signed-off-by: ZiheLiu<EMAIL_ADDRESS>
What have you changed?

Add vectorized truncate_uint.
Add test cases for vectorized truncate_uint.

What is the type of the changes?

New feature (a change which adds functionality)

How is the PR tested?

Unit test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?

No.
Does this PR affect tidb-ansible?

No.
Refer to a related PR or issue link (optional)
#5751

@ZiheLiu Hi, please fix the conflicts so our robot can merge this PR

/label status/waiting

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


Merging static methods needs rewrites for static dispatches

@MergeMethod annotations for static methods does not support static dispatches to the preserved base class method. Currently it acts as a @DefineMethod (except it preserves the original method with the renamed method prefix), which is not correct. Base class static method dispatches to the method to merge should be treated as recursive calls and parent class static method dispatches to the method should be treated as dispatches to the preserved method.

Fixed in c1032b3f855b67d2192242ac6ca9a257ce5283c0.


Fix PublishWebm call in example

The call of PublishWebm changed and now needs two extra parameters - for saving audio & video tracks. Add them for simple example to build and work.

Thank you! I noticed we needed this fix as well recently, we really appreciate your sending a PR!


Add generic issue without template

requirements
The extension removes the default New Issue button, but in some instances it's useful to work without a template (or to use the github's ISSUE_TEMPLATE feature, e..g in buildo/core).
I propose we add an other issue type that simply mimics the new issue button we removed.

@gabro what do you mean with 'add another button that mimics the new issue button we removed'?
Isn't much easier and sensed to unhide it? Like in the screenshot below

As discussed offline, this would change the visual priority more than indented.
We still want to discourage an issue to be opened without a template.


Uncaught ReferenceError: YUI is not defined

Unable to load demo site
http://esprima.org/demo/parse.html
Cross-Origin Read Blocking (CORB) blocked cross-origin response http://yui.yahooapis.com/3.9.0/build/yui/yui-min.js with MIME type text/html. See https://www.chromestatus.com/feature/5629709824032768 for more details.

http://yui.yahooapis.com/3.9.0/build/yui/yui-min.js was getting blocked by internal firewall. Closing this.


update pipeline docs

For https://github.com/rancher/docs/issues/671

@cloudnautique Above comments are addressed.

Closing in favor of: https://github.com/rancher/docs/pull/828


 AND I CAN"T FIND…

…  ARRRGHHHHH!
  ARRRGHHHHH!
Before submitting a pull request, please make sure the following is done:

Fork the repository and create your branch from master.
Run yarn in the repository root.
If you've fixed a bug or added code that should be tested, add tests!
Ensure the test suite passes (yarn test). Tip: yarn test:dev is helpful in development.
Format your code with prettier (yarn prettier).
Run the Flow typechecks (yarn flow).

                                                ARRRGHHHHH!UT OF MY DASHBOARD! ARRRGHHHHH!


ask some questions

Hello, I am learning the principle of reverse ssh in your code and jumping to the target server through the rssh service.
Because I am not familiar with the go language, I do not understand the principle after viewing the source code. My current understanding is that the rssh client will actively initiate a tcp connection to the rssh service port 3232, and the rssh server listens to port 3232 after receiving the connection. Generate a random id to record this reverse ssh tunnel. The attacker specifies the id to connect to the rssh server through the ssh -J parameter. The rssh server will query the corresponding ssh tunnel and jump to the target host according to the id. I don’t know if my understanding is correct.
I currently have a need for python programming. I want to use the paramiko library to obtain a session that jumps to the target host ssh as a programming interface. How can I connect to the rssh server and then continue to connect to the target machine? How to restore the parameter(-J) id to host and port?

Hi there!
This is a really great beginner question so I'll try and clarify what I think your questions are.
First and foremost everything in the RSSH project complies with the SSH standard, so all the standard tooling and most of the RFCs for SSH apply to RSSH. I.e it might be a good idea to go and have a read of how ssh works to kind of wrap your head around what the RSSH client and server are actually doing (I also have a blog post coming out soon that I'll link here to help perhaps).

the rssh client will actively initiate a tcp connection to the rssh service port 3232

Yep this is totally correct! The server will generate a random ID, and record the hostname, connecting IP address and public key hash all as IDs.

The attacker specifies the id to connect to the rssh server through the ssh -J parameter

Almost correct. The -J or "jump" flag actually specifies what server you want to connect to before you connect to your target machine.
In this instance the -J parameter is actually your rssh server not the client.
E.g
ssh -J your.rssh.server:3232 <target_id>

I want to use the paramiko library to obtain a session...
Unfortunately I dont do a lot of python programming. But this gist (https://gist.github.com/tintoy/443c42ea3865680cd624039c4bb46219) may help you as it showcases using the SSH protocol jump host directive (called direct-tcpip).

This is a little less low level: https://github.com/paramiko/paramiko/issues/1018#issuecomment-744151656
Hope that helps!

Thank you very much for your enthusiastic answer and look forward to your blog. In the link you shared with me, I seem to have found a higher-level library fabric than paramiko, which can more conveniently fulfill my needs and jump through gatway.
I still have a little doubt that I can use my rssh address and port (for example: 3232) to identify the connection for the hostname of the rssh server. For the target host, can I directly use the corresponding target id to mark it? Just like using native commands line.ssh -J your.rssh.server:3232 <target_id>

Yes, so any of the identifiers given from the output of ls in rssh can be used to connect to a target.
Unique ID is best if you'll be running multiple agents on the same host with the same public keypair.

Thank you so much, I successfully obtained the session interface of the target host through python paramiko. In the future, I will publish a B/S architecture program based on your rssh project on my github. In the web page, you can execute the shell of the target host and manage files. Welcome to use and make suggestions when I release！
Now I have another question, the id can uniquely identify a direct-tcpip session, if I use the host name + port, is it also unique? I think so.

The hostname may not be unique. Obviously hosts may have the same hostname, or you may run multiple rssh clients on the same host.
When jumping through the rssh server you should not specify a port on the remote target as technically no port is actually opened by the rssh client.
Just use one of the identifiers.
If you try and jump to with an identifier that stands for multiple hosts you'll get an error message.


Precompile parser

This leverages jison's ability to precompile a parser to greatly reduce the size of the final bundle, from 331kb to 30kb.
A npm command and grammar.json file replaces the original parser.js, and the stateful lexer is added to the runner after the parser is required.
All tests are passing
Also, moves the build system to esbuild, to reduce complexity and accessibility of the build.
Let me know what you think!
Edit: I'm seeing that the Travis CI tests fail with the esbuild, I'm kinda assuming because its testing a lower node version.  I can certainly modify this to use the old build system, but keep the precompile thing.  Is there a reason we are testing against a lower node version (<=5)?

Not really sure what to do about the Travis checks, but have everything actually working in the browser and node with a new rollup config, as well as with changing the jison dependency to this more maintained fork, which mostly just creates a saner compiled parser module that can be consumed by a bundler and loaded by the node cli/tests.


chore: refactor ops package

What this PR does / why we need it:
the ops package has been refactored with the following changes:

all the mock files have been moved under controller/konnect/ops/mock
the generated mock files have been prefixed with zz_generated.
all the SKD interface files have been moved under controller/konnect/ops/sdk

Which issue this PR fixes
Fixes #
Special notes for your reviewer:
PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[ ] the CHANGELOG.md release notes have been updated to reflect significant changes

Thanks for doing this. 👍
One thing that I would maybe change is the location of mocks. As they're all SDK mocks, IMO it makes ops/sdk/mock package more suitable for them.

Good point, see 63f1a70c1ae18607f3863ab6a9cf1b4dafa53f48

Linking #766 as related.


Add branching by notify_type to the wwwpoll transport

Need to branch by notify_type so we can store the correctly formatted data as expected by condenser in the wwwpoll table.
@relativityboy if you can, it would be of great benefit to have a simple list of templates - 1 per notify_type
Basically just need a template for each type of notification that I can then import into wwwpoll's code quickly.

Taking this one myself now, will put list of templates into docs/ for reference on future issues

This appears to be obsolete in current design, closing to keep issues clean


Errors when building for iOS

Hi, and thanks for this package! 👍
Just wanted to let you know that I tested the latest version (0.0.1+1) just now and got the following error when trying to build my project:
Launching lib/main.dart on iPhone 7 in debug mode...
Running Xcode clean...
Starting Xcode build...
Xcode build done
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **

Xcode's output:
↳
    /.pub-cache/hosted/pub.dartlang.org/flutter_statusbarcolor-0.0.1+1/ios/Classes/FlutterStatusbarcolorPlugin.m:18:33: error: invalid operands to binary expression ('NSNumber *' and 'int')
        statusBar.backgroundColor = ANDROID_COLOR(color);
                                    ^~~~~~~~~~~~~~~~~~~~
    /.pub-cache/hosted/pub.dartlang.org/flutter_statusbarcolor-0.0.1+1/ios/Classes/FlutterStatusbarcolorPlugin.m:3:51: note: expanded from macro 'ANDROID_COLOR'
    #define ANDROID_COLOR(c) [UIColor colorWithRed:((c>>16)&0xFF)/255.0 green:((c>>8)&0xFF)/255.0 blue:((c)&0xFF)/255.0  alpha:((c>>24)&0xFF)/255.0]
                                                     ~^ ~~
    1 error generated.

Oh, i am sorry you have this problem. I have released new version and hope this can help :)

Cheers, but still getting the same error, I think:
Launching lib/main.dart on iPhone 7 in debug mode...
Running pod install...
Running Xcode clean...
Starting Xcode build...
Xcode build done
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **

Xcode's output:
↳
    /.pub-cache/hosted/pub.dartlang.org/flutter_statusbarcolor-0.0.2/ios/Classes/FlutterStatusbarcolorPlugin.m:18:33: error: invalid operands to binary expression ('NSNumber *' and 'int')
        statusBar.backgroundColor = ANDROID_COLOR(@([color intValue]));
                                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    /.pub-cache/hosted/pub.dartlang.org/flutter_statusbarcolor-0.0.2/ios/Classes/FlutterStatusbarcolorPlugin.m:3:51: note: expanded from macro 'ANDROID_COLOR'
    #define ANDROID_COLOR(c) [UIColor colorWithRed:((c>>16)&0xFF)/255.0 green:((c>>8)&0xFF)/255.0 blue:((c)&0xFF)/255.0  alpha:((c>>24)&0xFF)/255.0]
                                                     ~^ ~~
    1 error generated.

No rush on my part on this one, just wanted to report it :) Turns out I won't be needing this package in any case, as what I want to do is change the text colour (not the background colour) :-/

OK, thank you.

But let me know if you want me to test for you, it's no problem for me to pull the latest release and see if it works 👍


u3: restore old-style, efficient reap to u3m_love()

This PR effectively reverts urbit/urbit#1753, which simplified cache promotion when falling back to an outer road, at the expense of allocation efficiency.
Early testing shows significant positive effects mitigating snapshot patch size (see urbit/urbit#6805)

For posterity, f747d86 is extending the fix from urbit/urbit#1059 to hank cache promotion.


Support export for the BeyondTheRepository BagIt profile

Partially resolves (imo): https://jira.lyrasis.org/browse/FCREPO-3176
This adds support for some of the newer additions in both the  bagit-profiles-spec and the beyond the repository bagit profile:

BagIt-Profile-Identifier is now written to the bag-info.txt

BagIt-Profile-Version is not, but it looks it will be required for validation of v1.2.0 (though currently this passes validation using the bagit-profiles-validator)

Bag-Info now supports the repeatable and description options

recommended was also added from the beyond the repository spec; a warning is logged on any missing recommended fields
currently it is not possible to have repeated values in the bagit configs, so it is there only for completion

Tag-Files-Allowed is checked against when validating a BagConfig

I also opted to read in the full bag profile json in BagProfile in anticipation of doing more validation against BagIt Bags, as I think there's some additional opportunity for both the BagIt Profiles and import of BagIt Bags.
For the BagIt Profiles, you'll find requirements mentioned throughout the spec such as:

Every file in Tag-Files-Required must also be present in Tag-Files-Allowed.

This I haven't added yet but could be done either when reading the BagIt Profile json or with the BagConfig validation.
For the Import, it looks like work had been started for validating BagIt Bags before running the Import process itself but had been put on hold. I don't think would be onerous to add support for ensuring a Bag conforms to a profile, but I wanted to keep this PR scoped to export.

@mikejritter : Thanks for this work!
@bbranan : I will be offline the rest of the week. If you resolve/address my minor comments before next week and you want to get this merged, feel free. Otherwise, I will approve/merge next week.

The updates based on comments look good to me. @awoods feel free to merge when you've had a chance to take a look.


Bump SHA & vers to current Bose SoundTouch <IP_ADDRESS>39

Checklist

[X] The commit message includes the cask’s name and version.

The formula was failing silently as the it was serving the current version with the old filename, so had sha mismatch.

You’re not adding any new cask, it makes no sense to fill the bottom part.

edited

Not that bottom part, now you’ve deleted too much. I had already edited your post to fix the issue.

one of these days I'll get it right, thanks for the edit and merge


Remove GOOS from scripts/start.sh

This PR removes GOOS, some of the extra coloring and also catches up the start script to the latest code base.

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


[ZEPPELIN-3910] IPython delete temp file and close stream

What is this PR for?
delete temporary file  and  close inputstream in IPythonInterpreter.checkIPythonPrerequisite()
What type of PR is it?
[Bug Fix]
Todos
*no
What is the Jira issue?

https://jira.apache.org/jira/browse/ZEPPELIN-3910, [ZEPPELIN-3910]

How should this be tested?

manual.

Screenshots (if appropriate)
Questions:

Does the licenses files need update?
no
Is there breaking changes for older versions?
no
Does this needs documentation?
no

can you check failed tests https://travis-ci.org/yx91490/zeppelin/builds/468316435

can you check failed tests https://travis-ci.org/yx91490/zeppelin/builds/468316435

I rebuild it and all test passed.

LGTM


[BUG] Can't find any Resumes that were created earlier

Describe the bug
I've logged in Reactive-Resume after some months, and now using the same Google Account I used to log in with, shows me no resumes, I've had 2-3 resumes created earlier, not sure where they went.
Product Flavor

[x] Managed (https://rxresu.me)

To Reproduce

Go to https://rxresu.me/
Click on 'Login'
Use Google Account to login
See no Resumes!!

Expected behavior
There should've been 2-3 resumes that I had created earlier.
Screenshots

Desktop (please complete the following information):

OS: Pop!_OS
Browser Microsoft Edge
Version 100.0.1185.36

Please use https://v2.rxresu.me to access your older resumes :)


Password change broken after password reset initiated

When posting issues, please include the following information to speed up the troubleshooting process:

Version:master branch
Request and response headers: 500 status, N/A
Rails Stacktrace:

ActiveModel::UnknownAttributeError (unknown attribute 'current_password' for User.):

activemodel (<IP_ADDRESS>) lib/active_model/attribute_assignment.rb:53:in `_assign_attribute'
activemodel (<IP_ADDRESS>) lib/active_model/attribute_assignment.rb:44:in `block in _assign_attributes'
activemodel (<IP_ADDRESS>) lib/active_model/attribute_assignment.rb:43:in `each'
activemodel (<IP_ADDRESS>) lib/active_model/attribute_assignment.rb:43:in `_assign_attributes'
activerecord (<IP_ADDRESS>) lib/active_record/attribute_assignment.rb:22:in `_assign_attributes'
activemodel (<IP_ADDRESS>) lib/active_model/attribute_assignment.rb:35:in `assign_attributes'
activerecord (<IP_ADDRESS>) lib/active_record/persistence.rb:620:in `block in update'
activerecord (<IP_ADDRESS>) lib/active_record/transactions.rb:375:in `block in with_transaction_returning_status'
activerecord (<IP_ADDRESS>) lib/active_record/connection_adapters/abstract/database_statements.rb:281:in `block in transaction'
activerecord (<IP_ADDRESS>) lib/active_record/connection_adapters/abstract/transaction.rb:280:in `block in within_new_transaction'
activerecord (<IP_ADDRESS>) lib/active_record/connection_adapters/abstract/transaction.rb:278:in `synchronize'
activerecord (<IP_ADDRESS>) lib/active_record/connection_adapters/abstract/transaction.rb:278:in `within_new_transaction'
activerecord (<IP_ADDRESS>) lib/active_record/connection_adapters/abstract/database_statements.rb:281:in `transaction'
activerecord (<IP_ADDRESS>) lib/active_record/transactions.rb:212:in `transaction'
activerecord (<IP_ADDRESS>) lib/active_record/transactions.rb:366:in `with_transaction_returning_status'
activerecord (<IP_ADDRESS>) lib/active_record/persistence.rb:619:in `update'
devise_token_auth (275da3c1960b) app/controllers/devise_token_auth/passwords_controller.rb:89:in `update'

Environmental Info:

Routes: Just a plain, simple API versioning

Rails.application.routes.draw do
  namespace :api, defaults: { format: 'json' } do
    scope module: :v1, constraints: ApiConstraints.new(version: 1, default: true) do
      mount_devise_token_auth_for 'User', at: 'auth'
    end
  end
end

Gems: Standard Rails API (--api-only) app
Custom Overrides: None
Custom Frontend: No front end yet!

Steps to reproduce

Initiate a password reset (POST /password) and click on the email link (GET /password/edit)
Do not complete the password reset (e.g. you suddenly remember your password!)
Try to update your password calling PUT /password DeviseTokenAuth.check_current_password_before_update set to true.

This will cause the exception ActiveModel::UnknownAttributeError (unknown attribute 'current_password' for User.).
### Explanation and initial investigation
When clicking on the email link to reset your password, the GET /password/edit is called, causing @resource.allow_password_change being set to true (https://github.com/lynndylanhurley/devise_token_auth/blob/3bf9b9be40661f9024083962ee9660cc45ac5dbc/app/controllers/devise_token_auth/passwords_controller.rb#L45).
This is necessary to allow the password reset without knowing the current password.
However, assuming you then remember your password and try to change it with DeviseTokenAuth.check_current_password_before_update setting set to true, you'll theoretically need to send your current_password as well. This is however overridden by the allow_password_change flag (https://github.com/lynndylanhurley/devise_token_auth/blob/3bf9b9be40661f9024083962ee9660cc45ac5dbc/app/controllers/devise_token_auth/passwords_controller.rb#L103), so @resource.update is called instead of @resource.update_with_password.
This results in the error mentioned above.
The error can be avoided by NOT sending the current_password, this works because the allow_password_change flag overrides the DeviseTokenAuth.check_current_password_before_update setting, but this is not expected behaviour.
### Expected behaviour
IMHO, PUT /password should ALWAYS require the current_password if DeviseTokenAuth.check_current_password_before_update is set to true, regardless of the @resource. allow_password_change  flag.
I believe this can easily be reproduced with a test and fixed in the PasswordsController, but before opening a PR I wanted to double-check with you if you agree with this behaviour or if you believe the current one is better (no test at the moment test this scenario, so I believe it's an unexpected behaviour).
It's worth saying that a FE application page allowing to change the password will unlikely check the @resource. allow_password_change  flag, so this could cause the situation where users are unable to change their password after initiating the password reset flow.
This is very probably related to #1068

@iMacTia I was having the same, but in a slightly different way. I was using the passwords#update entry to change user password using the current_password attribute too, but I guess that is not correct. It broken after I set the require_client_password_reset_token. So after reading this issue I started to investigate the problem. And what I concluded is that we are not suppose to use passwords#update action to change users password, but registrations#update. It worked for me! Give it a try!

@rodrigovcortezi I can see what you're referring to. Looking again at the code it seems, as you said, that the passwords#update action should only be used for password reset, hence for cases where we don't have a valid access token, although for some reason there's an else loading the user based on the actual acces-token, which doesn't really make sense under that assumption (https://github.com/lynndylanhurley/devise_token_auth/blob/387306a180e955da077a67a6b77b1feda9497202/app/controllers/devise_token_auth/passwords_controller.rb#L73).
I guess there's a bit of confusion around the correct password reset flow, but as you correctly pointed out I can successfully change the password using the registrations#update endpoint instead.
Still, it would be good to hear from a maintainer about why is passwords#update allowing to set the user by token, if that should not be possible during a password reset?

only auth endpoint fulfill the purpose with current and changed password as params, no need of registrations#update


Accept special characters and length limit for social media

Fixes #104

should we add some tests to check the validation checks? Not too sure why codecov isn't running as well...


0.1.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4898

Relevant Lines:
4933

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4898

Relevant Lines:
4933

💛  - Coveralls


In the realsense viewer, My RGB feed keeps locking up and it says no frames received...now my depth feed is doing so as well

librealsense
2.35.2 RELEASE

OS
Windows

Name
Intel RealSense L515

Serial Number
00000000f0210047

Firmware Version
<IP_ADDRESS>

Camera Locked
YES

Usb Type Descriptor
3.2

Product Line
L500

Asic Serial Number
00000003ac1e88b3

Firmware Update Id
00000003ac1e88b3

I just got this thin (L515) a couple of days ago and can't really figure out how to even use it.  The viewer (Realsense Viewer) comes with examples that are just code, no explanation of what I'm supposed to do with said code...I assume something in Visual Studio?  I used Turbo Pascal 20+years ago, that is the extent of my coding abilities,  I am just trying to capture point cloud data on various objects, to then make parts for via CNC routing or 3D printing.  I thought this was going to be a simple plug and play device, but cant seem to figure out how to do anything other than see what it's seeing, like an overpriced webcam at this point.  I have noticed that after a few minutes of being plugged in to my pc, a seemingly random stream will just stop being received - at first, it was always the video feed, but as I started posting this, the depth feed stopped working.  The only way I have found to reconnect it is to close the program and reopen it.    I have been told that it is possible with the realsense viewer...I must be missing something because I've tried everything a non-programmer would think to do, click every button on the screen.

Hi @KEdwards79
For full control/functionality of RealSense devices the SDK is the preferred method. You can however use RealSense Viewer to export a PointCloud file. Enable the RGB and Depth stream together then switch to the 3D view in the upper right corner. You should be able to see a textured point cloud of your scene. The export button above can save that to a PLY file which can then be read by supported software.
Thanks

Hi,
Will you be needing further help with this? If we don’t hear from you in 7 days, this issue will be closed.
Thanks

no, it turns out that the unit was overheating on the tripod.  I have since
made a fan-cooled case that works great!
On Wed, Aug 5, 2020 at 12:39 PM RealSenseSupport<EMAIL_ADDRESS>wrote:

Hi,
Will you be needing further help with this? If we don’t hear from you in 7
days, this issue will be closed.
Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/6924#issuecomment-669330783,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQMZ775FCU6AICIKRXQVPX3R7GKLTANCNFSM4PHYU54Q
.


[SPARK-45785][CORE] Support spark.deploy.appNumberModulo to rotate app number

What changes were proposed in this pull request?
This PR aims to support to rotate app number by introducing a new configuration, spark.deploy.appNumberModulo.
Why are the changes needed?
Historically, Apache Spark's App ID has a style, app-yyyyMMddHHmmss-1234. Since the 3rd part, 1234, is a simple sequentially incremented number without any rotation, the generated IDs are like the following.
app-yyyyMMddHHmmss-0000
app-yyyyMMddHHmmss-0001
...
app-yyyyMMddHHmmss-9999
app-yyyyMMddHHmmss-10000

If we support rotation by modulo 10000, it will keep 4 digits.
app-yyyyMMddHHmmss-0000
app-yyyyMMddHHmmss-0001
...
app-yyyyMMddHHmmss-9999
app-yyyyMMddHHmmss-0000

Please note that the second part changes every seconds. In general, modulo by 10000 is enough to generate unique AppIDs.
The following is an example to use modulo 1000. You can tune further by using spark.deploy.appIdPattern configuration.
$ SPARK_MASTER_OPTS="-Dspark.deploy.appNumberModulo=1000 -Dspark.master.rest.enabled=true" sbin/start-master.sh

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the CIs with newly added test case.
Was this patch authored or co-authored using generative AI tooling?
No.

Could you review this PR when you have some time, @LuciferYang ?

Could you review this when you have a chance, @yaooqinn ?

Thank you so much for review and approval, @yaooqinn !

Merged to master for Apache Spark 4.0.0.

late LGTM


Import not working

Hi guys,
I am using the latest version and trying to import it using ES6 syntax:
import Swiper from "swiper";`
I have the slider working, but it sounds like the whole is not working: pagination is not working, responsiveness neither.
Have you some recommendation for that? I can't find any doc on that.
Thanks.

Platform/Target and Browser Versions: Chrome on macOS

V3 doesn’t support ES modules. You can use v4 beta for this

Thanks for your answer - any way to install it using npm?

@bdebever v4.0.0-beta2 is already the latest published version on npm: https://www.npmjs.com/package/swiper


[BUG] Cannot set a numeric quota to zero. Set an amount of zero (0) to remove an existing plan from an assigned directory or subaccount.

Is there an existing issue for this?

[X] I have searched the existing issues

What version of the Terraform provider are you using?
1.5.0
What version of the Terraform CLI are you using?
1.9.0
What type of issue are you facing
bug report
Describe the bug
When trying to remove a numeric quota I'm getting the below error message.
╷
│ Error: Invalid Attribute Value
│
│   with btp_subaccount_entitlement.free-trial-kymaruntime-quota,
│   on bootstrap-kymaruntime.tf line 24, in resource "btp_subaccount_entitlement" "free-trial-kymaruntime-quota":
│   24:   amount        = 0
│
│ Attribute amount value must be between 1 and<PHONE_NUMBER>, got: 0

Expected Behavior
Set an amount of zero (0) to remove an existing plan from an assigned directory or subaccount.
Steps To Reproduce
/*
.PHONY: free-trial-kymaruntime-quota
free-trial-kymaruntime-quota:
btp assign accounts/entitlement --to-subaccount $$(btp list accounts/subaccount | jq -r '.value[] | select(.displayName == "trial") | .guid ') --for-service kymaruntime --plan trial --amount 0
*/
data "btp_subaccounts" "all" {}
resource "btp_subaccount_entitlement" "free-trial-kymaruntime-quota" {
for_each      = { for sub in data.btp_subaccounts.all.values : sub.id => sub if sub.name == "trial" && var.BTP_KYMA_PLAN == "trial"}
subaccount_id = each.key
service_name  = "kymaruntime"
plan_name     = var.BTP_KYMA_PLAN
amount        = 0
}
User's Role Collections
Subaccount Administrator
Add screenshots to help explain your problem
No response
Additional context
No response

@ptesny what could be the usecase to set entitlement to 0 instead of deleting the entitlement via terraform configuration. Setting entitlement value 0 via BTP CLI is same as removing the entitlement.

@CHERIANS
this is exactly what I am trying to do: to set the entitlement to zero:
data "btp_subaccounts" "all" {}

resource "btp_subaccount_entitlement" "free-trial-kymaruntime-quota" {
for_each = { for sub in data.btp_subaccounts.all.values : sub.id => sub if sub.name == "trial" && var.BTP_KYMA_PLAN == "trial"}

subaccount_id = each.key
service_name = "kymaruntime"
plan_name = var.BTP_KYMA_PLAN
amount = 0
}

However, btp provider restricts the amount value to Attribute amount value must be between 1 and<PHONE_NUMBER>, got: 0

Also from the BTP cockpit one can never set the entitlement value to 0

Not correct. It is well documented and nicely working with BTP cli.
The BTP cockpit uses this BTP cli feature (amount to zero) to allow to remove/delete an entitlement from the cockpit.
Please read the doc:
https://help.sap.com/docs/btp/btp-cli-command-reference/btp-assign-accounts-entitlement#:~:text=Set an amount of zero (0) to remove an existing plan from an assigned directory or subaccount.
For plans that have a numeric quota, use the '--amount' parameter to set the maximum allowed consumption (quota) of a plan for the directory or subaccount. Do not use the '--enable' parameter for these plans. Set an amount of zero (0) to remove an existing plan from an assigned directory or subaccount.

@ptesny The approach you would like to achieve is not how Terraform works. Terraform is relying on a resource-based approach (unfortunately BTP is not) with CRUD and Import operations. Consequently there are some scenarios that might work with BTP tooling but a different approach (from BTP perspective, but standard when it comes to Terraform) is needed when managing the resources via Terraform.
Hence, in case you want to remove a resource, you basically have remove the resource from the Terraform configuration, execute terraform plan and terraform apply.
Ther are variants deppending on the next steps you want to achieve like executing a terraform destroy -target <Address Of The Resource> to explicitly destroy one specific resource or a terraform state rm for removing the resource from the state, but if I understand your sceanrio correctly, then the first approach is the one that does the trick for you.

@ptesny  in addition to what @lechnerc77  mentioned another workaround to achieve  what you want would be to use the count  meta-argument  which is by default 1 (if not specified).  Set count = 0 as shown in snippet below and if you do terraform apply the entitlement should be removed. When you want to add the entitlement again just set the count = 1 do terraform apply
data "btp_subaccounts" "all" {}

resource "btp_subaccount_entitlement" "free-trial-kymaruntime-quota" {
for_each = { for sub in data.btp_subaccounts.all.values : sub.id => sub if sub.name == "trial" && var.BTP_KYMA_PLAN == "trial"}

subaccount_id = each.key
service_name = "kymaruntime"
plan_name = var.BTP_KYMA_PLAN
amount = 1
count = 0
}

@lechnerc77 thanks for the heads up.
IMO what is missing with the BTP TF provider is the ability to somehow manage the numeric quotas of the quota-based service plans separately from the resource entitlements.
We have the btp_globalaccount_entitlements (Data Source) but there is no btp_globalaccount_entitlements or btp_globalaccount_entitlement_quotas resource.
Anyway, there is a functional gap between BTPCLI and BTP TF provider with regard to the quota-based service plans and the quota distribution across a given BTP landscape.

@ptesny how about the below code snippet
data "btp_subaccounts" "all" {}
amt = 0 /* entitlement amount */
resource "btp_subaccount_entitlement" "free-trial-kymaruntime-quota" {
for_each = { for sub in data.btp_subaccounts.all.values : sub.id => sub if sub.name == "trial" && var.BTP_KYMA_PLAN == "trial"}

subaccount_id = each.key
service_name = "kymaruntime"
plan_name = var.BTP_KYMA_PLAN
amount = var.amt >1 ?var.amt : 1
count = var.amt > = 1 ? 1 : 0
}

╷
│ Error: Invalid combination of "count" and "for_each"
│
│   on bootstrap-kymaruntime.tf line 20, in resource "btp_subaccount_entitlement" "free-trial-kymaruntime-quota":
│   20:   for_each      = { for sub in data.btp_subaccounts.all.values : sub.id => sub if sub.name == "trial" }
│
│ The "count" and "for_each" meta-arguments are mutually-exclusive, only one should be used to be explicit about the
│ number of resources to be created.
╵

fyi: I have implemented this missing BTP TF provider functionality using the TF external provider's external data source as a shell script with BTPCLI.

free-trial-kymaruntime-quota.tf

# https://registry.terraform.io/providers/hashicorp/external/latest/docs/data-sources/external
#
data "external" "free-trial-kymaruntime-quota" {
  program = ["bash", "${path.module}/free-trial-kymaruntime-quota.sh"]

  query = {
    # arbitrary map from strings to strings, passed to the external program as the data query.
    username = "${var.username}"
    password = "${var.password}"
    globalaccount = "${var.globalaccount}"
    url = "https://cli.btp.cloud.sap"
  }
}

free-trial-kymaruntime-quota.sh:

#!/bin/bash

set -e -o pipefail
ISSUER=$(jq -r '{url: "\(.url)", username: "\(.username)", password: "\(.password)", globalaccount: "\(.globalaccount)" }' )
LOGIN=$(btp login \
--url $(jq  -r '. | .url' <<< $ISSUER ) \
--subdomain $(jq  -r '. | .globalaccount' <<< $ISSUER ) \
--user $(jq  -r '. | .username' <<< $ISSUER ) \
--password $(jq  -r '. | .password | tostring' <<< $ISSUER ))
QUOTA=$(btp assign accounts/entitlement --to-subaccount $(btp list accounts/subaccount | jq -r '.value[] | select(.displayName == "trial") | .guid ') --for-service kymaruntime --plan trial --amount 0)
jq -n --arg quota "$QUOTA" --arg issuer "$ISSUER"  '{"quota": $quota, "issuer": $issuer}'

after apply from terraform.tfstate

    {
      "mode": "data",
      "type": "external",
      "name": "free-trial-kymaruntime-quota",
      "provider": "provider[\"registry.terraform.io/hashicorp/external\"]",
      "instances": [
        {
          "schema_version": 0,
          "attributes": {
            "id": "-",
            "program": [
              "bash",
              "./free-trial-kymaruntime-quota.sh"
            ],
            "query": {
              "globalaccount": "****-ga",
              "password": "****",
              "url": "https://cli.btp.cloud.sap",
              "username": "****"
            },
            "result": {
              "issuer": "{\n  \"url\": \"https://cli.btp.cloud.sap\",\n  \"username\": \"****\",\n  \"password\": \"******\",\n  \"globalaccount\": \"******l-ga\"\n}",
              "quota": "{\n  \"jobId\": \"48749261\"\n}"
            },
            "working_dir": null
          },
          "sensitive_attributes": []
        }
      ]
    },

@CHERIANS @ptesny Since for_each and count meta-arguments can't be used together while creating a resource, I tried a workaround where we can have ternary operators in the for_each loop. We can have a variable named amount of type number and check whether the number is 0 or not based on the user input. If the condition is true will get the subaccount details and the amount attribute will take the value from the variable and accordingly resource can be created and entitlement can be assigned to directory or subaccount.
But I observed one more thing with this approach that if the condition fails we can return an empty set {} which will put all the attributes of the resources to null which automatically triggers 'terraform destoy' command and delete the resource as well as unassign the entitlement.
Attaching code for reference
variable "amount" {
  type = number
  description = "Numeric Quota Input"
  default = 0
}

data "btp_subaccounts" "all" {}

resource "btp_subaccount_entitlement" "assign" {
  for_each = var.amount!=0 ? { for sub in data.btp_subaccounts.all.values : sub.id => sub if sub.name == "trial" && var.BTP_KYMA_PLAN == "trial"} : {}
  subaccount_id = each.key
  service_name = "kymaruntime"
  plan_name = var.BTP_KYMA_PLAN
  amount = var.amount
}

Attaching screenshots for reference:

Closing this issue. Reason:
The resource for entitlements works in a Terraform compliant way offering the CRUD + LIST operations. We consistently mapped the BTP entitlement logic to this. No changes to the Terraform provider will be made.
In general, the flow for entitlements would also be questionable from a Terraform design perspective. Changes to the Infrastructure as Code should be transparent, consistent and follow an "as-Code" paradigm, namely changes in the infrastructure configuration should be done via pull requests that need to be reviewed. Sticking to Terraform a reviewer would expect a removal of a configuration (or a variable value) when a resource should be moved from am existing configuration. Achieving this via "magic" parameters would not be the preferred approach. Hence, removing the resource (in this case the entitlement) is the preferred approach in the declarative setup Terraform provides.


Refactor issue templates

Create issue page is misleading:

What is this "Custom template"? I am not sure that this is a good DX.
And I propose to rename it.

I agree it's better to remove it.


summit.creativecommons.org/kr

Hello,
I'm Hong, developer, CCKorea team.
This is my first time that I write issue at creativecommons. :)
repository:
http://github.com/creativecommonskorea/2015-summit
temporary domain:
http://summit.cckorea.org
I've just finished up korean landing page.
I'm afraid whether I shoud pull request for those codes to www/summit/kr,
or maybe would it be enough that I just send _site.zip to matt?
Please let me know what you think.

Please send a pull request :)

I tested on local but there were no special issues.
Please review my PR #24

So I've accepted this PR. We need to fix a few things, mostly around how bootstrap and jquery are loaded. We should load these from CDN instead of serving them locally.
<!-- Latest compiled and minified CSS -->
<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.4/css/bootstrap.min.css">

<!-- Optional theme -->
<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.4/css/bootstrap-theme.min.css">

<!-- Latest compiled and minified JavaScript -->
<script src="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.4/js/bootstrap.min.js"></script>

 <script src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.3/jquery.min.js"></script>

Thanks @mozodev

@mattl You're welcome. :smile:


Snippets for AXI-Stream signals and generics

This makes assumptions about the signal port names that there would only ever be 1 master and 1 slave in a port map which isn’t always the case. Personally, I don’t like the idea of providing this because 1. The full AXI-S signal list is provided, 2. It forces user to have all caps for each port signal, 3. The names are not changeable by the user, 4. The generic width name is not changeable by the user.
IMO this doesn’t belong in the tool itself and should instead be a personal snippet.

While I see your points and i admit that this particular version of the snippet may be a bit too personal, i think there is room for improvement here towards a more general version of these snippets. As long as having many snippets does not interfere with the correct functioning of the software, why not adding more, as long as they are significant

Also, per you commit about making tab spacing consistent, I would prefer not to align anything for the declaration. Some users use tabs, some use spaces and VSC doesn't convert \t in snippets to spaces when it detects tabs using spaces. Not only this, but not everyone aligns ports on colons, direction, etc. I would prefer to leave it up to the user to align their ports/signals whether that be manually or another command/extension that does this.

I think that having AXI snippets (as well as snippets for other industry standard busses) would be a very good addition. Some comments though:
I think that aligning could be done as long as it is done using spaces, e.g:
s_axi_awaddr  : in  std_ulogic_vector(11 downto 0);
s_axi_awready : out std_ulogic;

With regards to coding style, there is also the matter of the usage of upper/lower case. While I personally avoid using UPPER_CASE for names except for generics/constants I know that there are shops where e.g. ports are always written in upper case (and to be fair, the AXI signals are defined in UPPER_CASE in the specification). It should also be noted that while the std and ieee libraries do name types using UPPER_CASE many still refer to them using lower case.
Another problem, albeit probably smaller, is the type used for the signals. 'std_logic(_vector)' is in my experience by far the most commonly used, however at my current work, we use 'std_ulogic(_vector)' to allow for (theoretically) faster simulation and the ability for the simulator to detect multiple drivers at compile time.
Using generics to set the widths should also be optional. While it provides a single point of definition for the address and data widths, it does at the same time indicate to the user of the entity that they may be set freely, i.e. the design supports different widths which it may not do. Another use case for not using generics could be that the widths are defined using custom constants in a package
I also think that the name/prefix of the interface, e.g. the M_AXI_ should also be a variable, although it should have a default.
For the full AXI interface, it should be enough with a snippet with all the AXI signals and the user will then have to remove the unused ones.
With AXI5 being released, it might also make sense to add a "4" somewhere in the snippet name.
The above requires the snippets to support a couple of customizations.

UPPER_CASE vs. lower_case identifiers
UPPER_CASE vs. lower_case subtype_indication
Using generics for widths or not
Customizable prefix
Sideband signals for AXI Streaming interfaces.

Some proposals for further discussion (not sure how VS Code treats identical names and upper/lower case in the prefix). One way to have upper/lower case variation in the identifiers would be to make two variants of each template, one with the prefix in lower case and one with the prefix in uppercase.
Snippets

Name
Prefix
Description

AXI4-Stream Master Interface
axi4sm / AXI4SM
Basic AXI4 Stream Master interface with tdata, tvalid, tready

AXI4-Stream Slave Interface
axi4ss / AXI4SS
Basic AXI4 Stream Slave interface with tdata, tvalid, tready

AXI4-Stream Master Interface
axi4sm / AXI4SM
Basic AXI4 Stream Master interface with tdata, tvalid, tready

AXI4-Stream Slave Interface
axi4ss / AXI4SS
Basic AXI4 Stream Slave interface with tdata, tvalid, tready

AXI4-Stream Master Interface /w Sideband
?
Full AXI4 Stream Master interface

AXI4-Stream Slave Interface /w Sideband
?
Full AXI4 Stream Slave interface

AXI4-Lite Master Interface
axi4lm
AXI4 Lite Master interface

AXI4-Lite Slave Interface
axi4ls
AXI4 Lite Slave interface

AXI4 Master Interface
axi4m
AXI4 Lite Master interface

AXI4 Slave Interface
axi4s
AXI4 Lite Slave interface

Thanks @Bochlin for your precious comment.
How about we close this PR and open an issue to further discuss the most sensible implementation of the snippets?


Update needed

sir can you update the code to support a higher scale as no scale above 2x seems to be implemented can you do it like this app
support needed scale > x4 x8 x16 etc.
support for model
Real-ESRGAN
Real-CUGAN
https://play.google.com/store/apps/details?id=pro.archiemeng.waifu2x

It's just a demo project which means it will only support 2x by default. You can change it if you want.

yes i tried 3x and 4x but the app crashes would be better if you could just update some of the code to fix the issue kindly


fetchbranches: add support for authenticated requests

after running fetchbranches tool several times I got next error
Building and running the fetch branches tool
/usr/local/var/lib/localCI/gopath216490046/src/github.com/clearcontainers/tests/cmd/fetchbranches /usr/local/var/lib/localCI/gopath216490046/src/github.com/clearcontainers/tests
PULL_REQUEST_NUMBER : 6, OWNER : devimc, REPOSITORY: tests
ERROR: GET https://api.github.com/repos/devimc/tests/pulls/6: 403 API rate limit exceeded for <IP_ADDRESS>. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.) [rate reset in 551m53s]

@MarioCarrilloA who has just been seeing this whilst working on metrics localCI stuff - let's share experience and data.

@grahamwhaley  @GabyCT  I can tackle this issue

@devimc it seems it may be related to our tokens maybe not getting the higher access rate enabled - not sure - might need some digging.

@grahamwhaley fetchbranches tool should also support tokens


Laravel 8 support

Hi,
to support Laravel 8 guzzle version has to be bumped up to ^7.0.1 and illuminate/contracts to ^7.20.0.
Should i create a pull request for that or could you do it?
BTW I did a quick search in src/ and i couldn't find any usage of Guzzle.

It uses nategood/httpful but I'm considering on replacing that http wrapper package with laravel's new build in http client. Will see if I can start on it over the weekend.

Hi,
Any update on Laravel 8 support?


Error: yt-dlp v2024.7.9 Returns "The following content is not available on this app" for All Videos

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

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

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[x] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
After initially using yt-dlp version 2024.4.9, I encountered an issue where only some videos from YouTube displayed the error message: "The following content is not available on this app. Watch on the latest version of YouTube." However, after updating to yt-dlp version 2024.7.9, the issue has worsened, and now all video download attempts result in the same error. Even after downgrading back to version 2024.4.9, the issue persists, indicating that the problem is not limited to a specific version but might be due to recent changes in YouTube’s site or API that affect how yt-dlp handles video extraction.
Provide verbose output that clearly demonstrates the problem

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

Complete Verbose Output
[debug] Command-line config: ['-v', 'https://www.youtube.com/shorts/ABwOK9YrMJw']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [ff0779267] (pip)
[debug] Python 3.10.14 (CPython x86_64 64bit) - Linux-<IP_ADDRESS>-1.cm2-x86_64-with-glibc2.31 (OpenSSL 1.1.1w  11 Sep 2023, glibc 2.31)
[debug] exe versions: ffmpeg 4.3.6-0, ffprobe 4.3.6-0
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.02.02, mutagen-1.47.0, requests-2.32.3, sqlite3-3.34.1, urllib3-1.26.18, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1810 extractors
[youtube] Extracting URL: https://www.youtube.com/shorts/ABwOK9YrMJw
[youtube] ABwOK9YrMJw: Downloading webpage
[youtube] ABwOK9YrMJw: Downloading ios player API JSON
[youtube] ABwOK9YrMJw: Downloading android player API JSON
ERROR: [youtube] ABwOK9YrMJw: The following content is not available on this app. Watch on the latest version of YouTube.
  File "/tmp/8dca58883ae455f/antenv/lib/python3.10/site-packages/yt_dlp/extractor/common.py", line 734, in extract
    ie_result = self._real_extract(url)
  File "/tmp/8dca58883ae455f/antenv/lib/python3.10/site-packages/yt_dlp/extractor/youtube.py", line 4182, in _real_extract
    self.raise_no_formats(reason, expected=True)
  File "/tmp/8dca58883ae455f/antenv/lib/python3.10/site-packages/yt_dlp/extractor/common.py", line 1251, in raise_no_formats
    raise ExtractorError(msg, expected=expected, video_id=video_id)

Duplicate of #10108
Your IP has been blocked by Youtube

is there any way to prevent this like how much should I wait before downloading next video

It's not known what exactly triggers an IP ban like this.
You could try using some options like these:
--sleep-requests 1.25 --min-sleep-interval 60 ---max-sleep-interval 90

Youtube does seem to be targeting data center IP addresses with these bans, so if you're using one, the above options may not even help

[youtube] ABwOK9YrMJw: Downloading webpage
[youtube] ABwOK9YrMJw: Downloading ios player API JSON
[youtube] ABwOK9YrMJw: Downloading android player API JSON

actually, something is off here: why is android being requested? this was removed from the defaults a while ago. did you edit any source code files? did you edit this log?

[debug] Loaded 1810 extractors

yeah this is wrong. you are actually on an old version of yt-dlp. completely uninstall and re-install

No I haven't changed any source, and I am currently on the latest version of yt-dlp 2024.7.9
@bashonly

completely uninstall and re-install

``[youtube] Extracting URL: https://www.youtube.com/shorts/mgBbAwviTNY
[youtube] mgBbAwviTNY: Downloading webpage
[youtube] mgBbAwviTNY: Downloading ios player API JSON
ERROR: [youtube] mgBbAwviTNY: The following content is not available on this app.. Watch on the latest version of YouTube.
(antenv) root@fb-automat_250cd29e5c:/tmp/8dca58883ae455f# yt-dlp -v https://www.youtube.com/shorts/mgBbAwviTNY
[debug] Command-line config: ['-v', 'https://www.youtube.com/shorts/mgBbAwviTNY']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [89a161e8c] (pip)
[debug] Python 3.10.14 (CPython x86_64 64bit) - Linux-<IP_ADDRESS>-1.cm2-x86_64-with-glibc2.31 (OpenSSL 1.1.1w  11 Sep 2023, glibc 2.31)
[debug] exe versions: ffmpeg 4.3.6-0, ffprobe 4.3.6-0
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.02.02, mutagen-1.47.0, requests-2.32.3, sqlite3-3.34.1, urllib3-1.26.18, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1829 extractors
[youtube] Extracting URL: https://www.youtube.com/shorts/mgBbAwviTNY
[youtube] mgBbAwviTNY: Downloading webpage
WARNING: [youtube] Webpage contains broken formats (poToken experiment detected). Ignoring initial player response
[youtube] mgBbAwviTNY: Downloading ios player API JSON
[youtube] mgBbAwviTNY: Downloading player 65e3708e
[youtube] mgBbAwviTNY: Downloading web player API JSON
ERROR: [youtube] mgBbAwviTNY: The following content is not available on this app.. Watch on the latest version of YouTube.
File "/tmp/8dca58883ae455f/antenv/lib/python3.10/site-packages/yt_dlp/extractor/common.py", line 740, in extract
ie_result = self._real_extract(url)
File "/tmp/8dca58883ae455f/antenv/lib/python3.10/site-packages/yt_dlp/extractor/youtube.py", line 4267, in _real_extract
self.raise_no_formats(reason, expected=True)
File "/tmp/8dca58883ae455f/antenv/lib/python3.10/site-packages/yt_dlp/extractor/common.py", line 1263, in raise_no_formats
raise ExtractorError(msg, expected=expected, video_id=video_id)
still the error is same but the logs has changed a bit

@bashonly

i think its the issue with ip only because if i try downloading the same video by passing proxy the videos starts downloading
@bashonly


Move AI model configs to presets and add a dropdown to swap between configs

Adds the ability to specify multiple AI configs using the presets.json file and adds a new UI dropdown to switch between model configs. Will move to a separate ai-presets.json file later. To use the existing config from settings.json, use the "Global Default" preset
Updates the Menu component to use the floating-ui library, which provides some really nice primitives for floating overlays.
Also fixes useDimensions to better handle null refs.
https://github.com/user-attachments/assets/ca37b291-1c11-4618-b77b-a79695f29b74
closes #945

TODO: reintroduce the settings compatibility and use UpdateMetaObject to apply the presets

Hello
When opening multiple AI windows, the labels in the title (e.g., "global Default") accumulate additional values (e.g., "gpt-4o-mini"). This seems to be related to the logic of recalculating the presetMap, which is executed each time the map is recalculated.
The function that modifies the display:name of presets is called every time the presetMap is recalculated, leading to additional labels in the title. The current conditions for determining the display:name do not appear to be strict enough, allowing values to accumulate.
cosmo
Manjaro user

Haha whoops, will fix