eBPF policy map representation of allow-all as "reserved:unknown" is confusing

Is there an existing issue for this?

[x] I have searched the existing issues

Version
equal or higher than v1.16.0 and lower than v1.17.0
What happened?
If I install Cilium and follow the Getting Started with the Star Wars demo guide, then dump the bpf policy map for an endpoint on the node, it looks something like this:
root@kind-control-plane:/home/cilium# cilium-dbg bpf policy get 112
POLICY   DIRECTION  LABELS (source:key[=value])       PORT/PROTO   PROXY PORT   AUTH TYPE  BYTES  PKTS
Allow    Ingress     reserved:host                     ANY          NONE         disabled    0     0
                      ...
Allow    Ingress     k8s:app.kubernetes.io/name=deathstar 80/TCP    NONE         disabled    0     0
                      ...
Allow    Ingress     k8s:app.kubernetes.io/name=tiefighter 80/TCP   NONE         disabled    2088  2
                    ...
Allow    Egress      reserved:unknown                  ANY          NONE         disabled    0     0

The final rule in this line states that the labels for the egress allow-all entry is reserved:unknown. This is because the underlying data value is 0, and the security identity string representation for this identity is reserved:unknown.
How can we reproduce the issue?

Install Cilium
Follow the star wars demo guide https://docs.cilium.io/en/stable/gettingstarted/demo/#starwars-demo
Use cilium endpoint list to locate the security identity of one of the "empire" Pods
Use cilium bpf policy get ... with the security identity from (3) to list the policies.

Cilium Version
Any
Kernel Version
Any
Kubernetes Version
Any
Regression
No response
Sysdump
No response
Relevant log output
No response
Anything else?
When formatting BPF policymap entries like this, it would be more consistent and clearer to understand if the LABELS column instead said ANY, similar to the PORT/PROTO column.
Cilium Users Document

[ ] Are you a user of Cilium? Please add yourself to the Users doc

Code of Conduct

[x] I agree to follow this project's Code of Conduct

I think that the string representation of Security Identity 0 as "unknown" makes a lot of sense for other commands and hubble/monitor output, but for policymaps it doesn't make sense.


Remove debug_assert from Peripherals::steal

This is the same change as https://github.com/rust-embedded/svd2rust/pull/238, except for the one bit that isn't generated by svd2rust. There was a decent amount of discussion about this over at that issue, and to me it makes sense to change this here as well. It's unsafe, so let the user decide if they want to use it.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @thejpster (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Makes sense to me.
bors r+

bors r+

bors r+


feat: add sqlite save state and tx handler

Summary

Add SQLite implementation for save state and tx's
TODO: add a flow to initialize the databases locally with each unique game ID in starter template as example

looks fine, I think I moved the MySQL setup to the testutils; do you want to move the SQLite setup there too?

sg will do in a sec


Can not play sound normally

Exception message
code: "ENSOSSTATUSERRORDOMAIN2003334207", message: "The operation couldn’t be completed. (OSStatus error<PHONE_NUMBER>.)"
#PLIST
<key>NSAppTransportSecurity</key>
<dict>
	<key>NSExceptionDomains</key>
	<dict>
		<key>localhost</key>
		<dict>
			<key>NSExceptionAllowsInsecureHTTPLoads</key>
			<true/>
		</dict>
	</dict>

Code
    sound = new Sound('http://www.noiseaddicts.com/samples_1w72b820/4923.mp3', Sound.MAIN_BUNDLE, (error) => {
        if (error) {
            console.warn('failed to load sound, ', error);
        } else {
            // sound.play();
            this.setState({
                isPlaying: true,
            });
        }
    });

For iOS what worked for me is the following:

Checked that my resource file was being played in xCode

Open xCode
Select the file
Click play in the xCode player

Once it was playing sound (had issues with some files, possibly encoding),

Add the file as resource (Make sure is in Build Phases)

Compile the project in xCode and run it from there

Note: Every time you change the resources or expect new resources you need to compile your Native application either by doing it from xCode or npm run ios

This is my code:
const sound = new Sound(
    "myFile.mp3",
    Sound.MAIN_BUNDLE,
    error => {
      if (error) {
        console.log("failed to load the sound", error);
        return;
      }
      sound.play(() => sound.release());
    }
  );
// The play dispatcher
sound.play();


Feedback for Other vendors page

Ulefone:
This manufacturer's coders have added something called "DuraSpeed".
If you disabled it*, all apps will be automatically restricted from running in the background. In other words, 'DuraSpeed' unbreaks a broken basic Android OS.
*If you load a management app - any management app - the System menu will crash from the Settings menu. Such as using App Freezer (MobizSystems), a great app that works on my other phones without issue.
So you must leave DuraSpeed, and go into it and 'whitelist' every single app.
That is -besides- doing the usual stock Android 'Don't optimize battery' on an app.
Tested on the Android 9 that came with the phone, 3/2020.

DuraSpeed is a known villain from Nokia lower tier phones.  So apparently a lot of the battery killers in Chinese-designed phones come from a single source.
What you describe is quite alarming but big thanks for sharing the tip!
Could you please also tell me the exact model?

3wt. But you are going to find this with any of their current models.
Key thing is the android itself is broke, because if you use a management app to disable DuraSpeed, it seems that automatically places every app on a default 'blacklist', and they are killed some set amount of time after the screen goes off.

But Doze mode in base Android doesn't terminate apps - it just defers tasks like jobs and alarms and forces you to use foreground services instead of background services, regardless of the optimized/non-optimized mode.
What you are talking about is definitely a vendor modification of Ulefone/Gotron, not the behaviour of AOSP or Google-flavored Android.

Yes. I am referring to the chinesium breaking/broken version of android. Not AOSP.


All commit messages are in present

Bug description
Hi, my name is Mickael and I'm a Brazilian developer. I use aicommits with the locale pt. Every commit I generate comes in the present tense, so instead of "Added button on order screen" it says "Add button on order screen", is this a normal behavior?
aicommits version
1.11.0
Environment
System:
    OS: macOS 13.4.1
    CPU: (8) arm64 Apple M1
    Memory: 72.95 MB / 8.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 16.13.2 - ~/.custom-node/node16/bin/node
    Yarn: 1.22.17 - ~/.custom-node/node16/bin/yarn
    npm: 8.1.2 - ~/.custom-node/node16/bin/npm

Can you contribute a fix?

[ ] I’m interested in opening a pull request for this issue.

Yes, it follows the imperative style commit message:
https://stackoverflow.com/questions/3580013/should-i-use-past-or-present-tense-in-git-commit-messages


Documentation: add new version of Canonical Event Model wiki page for R71

Releases which change the Canonical Event Model should create a new version of the wiki page and rename the old copy of the page based on which releases it applies to.

@fblundun - has this one been done?

Yes, closing...


Remove null_ls.config...

null_ls seems to have changed their API, accessing the config table now creates an error, and just using register is sufficient to get JET running!

Thanks for making a PR for this!

No problem! Thanks for making this!


Fix the config apply when deploying on KIND

By following the README instructions to run a cluster over KIND I faced the following error:
resource mapping not found for name: "openstack" namespace: "" from "config/samples/core_v1beta1_openstackcontrolplane.yaml": no matches for kind "OpenStackControlPlane" in version "core.openstack.org/v1beta1" ensure CRDs are installed first
resource mapping not found for name: "openstack-collapsed-cell" namespace: "" from "config/samples/core_v1beta1_openstackcontrolplane_collapsed_cell.yaml": no matches for kind "OpenStackControlPlane" in version "core.openstack.org/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "openstack" namespace: "" from "config/samples/core_v1beta1_openstackcontrolplane_galera.yaml": no matches for kind "OpenStackControlPlane" in version "core.openstack.org/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "openstack-network-isolation" namespace: "" from "config/samples/core_v1beta1_openstackcontrolplane_network_isolation.yaml": no matches for kind "OpenStackControlPlane" in version "core.openstack.org/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "openstack-separate-cell-db" namespace: "" from "config/samples/core_v1beta1_openstackcontrolplane_separate_cell_db.yaml": no matches for kind "OpenStackControlPlane" in version "core.openstack.org/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "" namespace: "" from "config/samples/kustomization.yaml": no matches for kind "Kustomization" in version "kustomize.config.k8s.io/v1beta1"
ensure CRDs are installed first
error validating "config/samples/kustomization.yaml": error validating data: [apiVersion not set, kind not set]; if you choose to ignore these errors, turn validation off with --validate=false

This patch define a couple of keys to use with KIND when we apply the config to fix this error.
However, we should notice that the resource mapping not found lines remains even after fixing the kustomization error. Let me know if we can add specific changes to also fix these mapping not found.

/ok-to-test

The reference to KIND in the README is generated from operator-sdk and not necessarily a target platform for the project at this time. We currently require OpenShift.
I think the patch may be okay so lets see what the tests do


Dynamic monitored nodes

a) Fixed the ability to have more than one monitored node.
b) Added a feature to allocate dynamic memory for the monitoredNodes array.
This is an advantage if you have less than CO_CONFIG_HB_CONS_SIZE nodes you want to monitor.
If you want to monitor more than CO_CONFIG_HB_CONS_SIZE nodes. If you have multiple OD's. and
you monitor a large amount of node in one OD and few in the other. You would waste a lot of memory, if you just raise CO_CONFIG_HB_CONS_SIZE.

I did some work related to this PR in the last days. I followed your idea and optimized memory allocation for heartbeat consumer (object 1016) and also for error history (object 1003), which is similar. For this to work, also OD editor has been updated, see #314.
However, I had to do that my way, because solution is now more generic,I get rid of two configuration settings, etc. I hope, you will like the solution.
I think HB consumer is now configurable enough, except I can't simply include allocating more than one heartbeat consumer inside CANopen.c file. If you need additional, simply allocate own in your application. (Note that one HB consumer is connected with one 1016 OD object (or similar) and each HB consumer can have configurable size of monitored nodes now.)
Thank you for PRs and for testing.


Upgrade plutus to build with ghc-9.6

[x] Make it build
[ ] Add it to CI
[ ] Upload to CHaP

PR is https://github.com/input-output-hk/plutus/pull/5286

@michaelpj would you be able to review this?

I'm aware of it. I've also got the CI bit more-or-less ready to go.
I think we might want to make some individual sub-tasks for the stuff we need to do to get it on CHaP. There are a lot of dependencies that need updating.

@michaelpj

I think we might want to make some individual sub-tasks for the stuff we need to do to get it on CHaP.

fully agreed. @erikd can you create the subtasks? I doubt there are real subtasks in GH, but we can just create multiple tickets and link them here, I believe.

I've narrowed down the problems to just a few. Lots has been fixed on Hackage recently. Listing the remaining ones here:

https://github.com/well-typed/cborg/issues/311 blocks basically everything on 9.6 on aarch64-darwin, but WT know about it
https://github.com/typeclasses/hex-text/issues/7 . Just needs a revision.
https://github.com/mokus0/th-extras/pull/20 . Just needs a revision.
int-cast is basically unmaintained, just needs a revision, but probably we should just purge it
inline-r need more than just bound relaxation alas. But all the stuff we need is in master, so it just needs a release. https://github.com/tweag/HaskellR/issues/415
The known foundation issue

I've commented/opened issues throughout, will try and act on int-cast soonish.

Still need to upload this to CHaP.

We do a release at the end of every sprint.

When is that?


go.mod: github.com/containerd/containerd v1.7.18, switch to github.com/containerd/errdefs module

follow-up to / stacked on https://github.com/containerd/nydus-snapshotter/pull/598

go.mod: github.com/containerd/containerd v1.7.18
full diff: https://github.com/containerd/containerd/compare/v1.7.7...v1.7.18
switch to github.com/containerd/errdefs module
containerd 1.7.18 and up alias the errdefs package to the new module,
and deprecate the package.

Hmm... looks like there's failures in CI related to OTEL; looks like multiple metrics servers are tried to be set up on port 9110;
time="2024-06-26T10:15:05.945761805Z" level=info msg="loading plugin \"io.containerd.internal.v1.tracing\"..." type=io.containerd.internal.v1

...

time="2024-06-26T10:15:05.946779174Z" level=error msg="failed to load cni during init, please check CRI plugin status before setting up network for pods" error="cni config load failed: no network config found in /etc/cni/net.d: cni plugin not initialized: failed to load cni config"

...

umount globally shared mountpoint
umount: /var/lib/containerd-nydus/mnt: no mount point specified.
ls: cannot access '/run/containerd-nydus/containerd-nydus-grpc.sock': No such file or directory
Fail(1). Retrying...
time="2024-06-26T10:15:08.579292914Z" level=info msg="Start nydus-snapshotter. Version: fba89c3, PID: 143, FsDriver: fusedev, DaemonMode: multiple"
time="2024-06-26T10:15:08.581727603Z" level=info msg="Trying to translate bucket records..."
time="2024-06-26T10:15:08.581798015Z" level=info msg="Trying to update bucket records from v1.0 to v1.1 ..."
time="2024-06-26T10:15:08.582908629Z" level=info msg="parsed cgroup config: cgroup.Config{MemoryLimitInBytes:-1}"
time="2024-06-26T10:15:08.582980705Z" level=info msg="cgroup mode: legacy"
time="2024-06-26T10:15:08.583967256Z" level=info msg="create cgroup (v1) successful, state: thawed"
time="2024-06-26T10:15:08.584895731Z" level=info msg="Run daemons monitor..."
time="2024-06-26T10:15:08.585325775Z" level=fatal msg="failed to start nydus-snapshotter" error="failed to initialize snapshotter: start metrics HTTP server: metrics server listener, addr=:9110: listen tcp :9110: bind: address already in use"
ls: cannot access '/run/containerd-nydus/containerd-nydus-grpc.sock': No such file or directory

Looks related to this code, but I'm wondering now if the snapshotted itself should setup a metrics server, or if that's something that only containerd should do https://github.com/containerd/nydus-snapshotter/blob/71ab64ad42d30c9b1538418ba041df0d1191d4f1/snapshot/snapshot.go#L180-L200

Code above was added in https://github.com/containerd/nydus-snapshotter/commit/fe511b89ba033ab189bc2e4b08c7bcfb50bd132b, which is part of this PR:

https://github.com/containerd/nydus-snapshotter/pull/263

cc @cpuguy83 @dmcgowan perhaps one of you knows?

Code above was added in fe511b8, which is part of this PR:

metrics: collect the metrics of nydusd events #263

I think the CI failure is not caused by this patch. It seems the :9110 already in use.
Can we retry this CI?

Can we retry this CI?

Otherwise, we can disable metrics server in E2E test.

The reason why the :9110 port is already in use is the old "containerd-nydus-grpc process is not killed successfully.
https://github.com/containerd/nydus-snapshotter/blob/main/integration/entrypoint.sh#L140

Ah! I guess I misinterpreted the cfg.MetricsConfig.Address to be at the "global" level (so for containerd as a whole), but it's for this snapshotted.

The reason why the :9110 port is already in use is the old "containerd-nydus-grpc process is not killed successfully.

Good one; maybe? I don't have permissions to restart Ci on this repo; I can do a quick close & reopen to try, or perhaps you're able to kick only the failing ones.
This failure didn't happen on my other PR though, and I've seen it fail twice (both before and after rebasing), so .. it's still possible there's a regression somewhere (or change in behavior)

Let me just do a close/reopen to get a fresh run

The reason why the :9110 port is already in use is the old "containerd-nydus-grpc process is not killed successfully.
https://github.com/containerd/nydus-snapshotter/blob/main/integration/entrypoint.sh#L140

Facing the same issue.
The process gets killed, but it is slow to exit: https://github.com/containerd/nydus-snapshotter/blob/main/integration/entrypoint.sh#L145
0.5s is not enough on my machine.
2s does work better. YMMV
Anyhow, this is inherently racy - maybe we can replace by an explicit check (curl the endpoint maybe?).

0.5s is not enough on my machine.
2s does work better. YMMV
Anyhow, this is inherently racy - maybe we can replace by an explicit check (curl the endpoint maybe?).

Thanks for the tip, I will work on this.

@thaJeztah
Sebastiaan, #606 fixes most of the CI issues (except the kubeconf regression) - if you can rebase on it as soon as it merges, that would be really nice.
Would love to see your PR go through ASAP - as it will definitely help with work going on in #604.

Let me already to a quick rebase to get a fresh run ahead of time. @apostasie  are there plans to do a release before switching to containerd v2? If so, I can do one more follow-up to get it to containerd v1.7.20 (using the separate API module).
I'm trying to get rid of the "github.com/containerd/containerd/errdefs" package in our dependency tree, and the nydus-snapshotter is one of the last remaining ones using it 😅

Let me already to a quick rebase to get a fresh run ahead of time. @apostasie are there plans to do a release before switching to containerd v2? If so, I can do one more follow-up to get it to containerd v1.7.20 (using the separate API module).
I'm trying to get rid of the "github.com/containerd/containerd/errdefs" package in our dependency tree, and the nydus-snapshotter is one of the last remaining ones using it 😅

@thaJeztah I am not a maintainer here, so, do not know about timeline for releases.
My interest is in getting nerdctl to containerd v2 ASAP (we have a working build but it is using my forks: https://github.com/containerd/nerdctl/pull/3173 ), so, I am onto getting dependencies to ctd v2 (on main).
I got stargz-snapshotter, data-accelerator/zdfs, and accelerated-image merged.
soci accepted the PR but want to wait for ctd v2 official release (https://github.com/awslabs/soci-snapshotter/pull/1305).
nydus and hcsshim are the last on my list.

@thaJeztah CI fixes got merged.
Would you rebase this and get it in?
Then I would rebase #604 on top of this.

Rebased 👍 CI looks to be happy now 🎉
Also prepared a follow-up for containerd v1.7.20, which deprecates another package (to help transition to v2); https://github.com/containerd/nydus-snapshotter/pull/610

@imeoer ptal 🤗

Thx! Rebasing https://github.com/containerd/nydus-snapshotter/pull/610  👍


Standard CSV filnavn for Nordea eksport?

Hej Gediminas,
jeg er med i et projekt hvor vi prøver at lave en generel YNAB konverter, og jeg har fundet dit projekt og håber du vil hjælpe med et spørgsmål?
Når du eksporterer CSV fra din Nordea netbank, er der så en standard for filnavnet? Kan du give mig et eksempel?
Du har jo et eksempel på indholdet af en fil, så det er kun lige filnavnet (eller filnavn-mønsteret) jeg spørger om.
Hilsen
Torben i Østrig

Hi Torben,
if I remember correctly, the file looked exactly like the sample-input-nordea.csv file on repo. It was named something else originally, but I don't recall how, and can't check since I'm not a client there anymore.
The name of the file has to be adjusted in convert.js file in order for the script to run.
Does this answer your question?

Thanks for your response! I'm sorry you're not using that bank anymore, I will try to find the filename elsewhere. What bank are you using now? Does it provide a YNAB-ready download or do you still need to convert it? (Or are you no longer using YNAB?)
If you like, take a look at our "bank2ynab" project - perhaps you would find it useful? I should have linked to it in my original post, but here you are.
Regards,
Torben


Two-Step Authentication login is unhandled

if the account you are trying to login with has 2FA enabled, no response is displayed, and nothing happens.
instead of using private api, you may just load instagram login page, and collect/save the cookies after a successful login.

Thank you for reporting this 👍
I'll look into this. But also feel free to send a PR, It would be much appreciated. 🙂

Same issue..

Any other news about?

Hi @Jostino , unfortunately there's no solution to this yet. I have tried to implement this today, but an implementation needs to be updated in the depended library.
A current WORK AROUND would be to:

disable two-step authentication
login to igdm
re-enable two-step authentication

The instagram API would then recognize your IGdm device from that point on, and you would not be required to enter the 2-step verification code even if you logout and then log back in.

@ifedapoolarewaju Hey can we not have our own forked repo of instagram-private-api in which we can make changes to and implement in it?  BTW, I have just started electron and node.js and so I'm a complete beginner with many doubts. :)

Any update?

Hey, thanks for the bump @dzikoysk 😁.
I checked latest version of the library, looks like they added 2FA support recently.
After skimming the code, i was able to determine the api call and the parameters.
The only thing that we are missing is the csrf token that is sent with the original login request.
Our current version returns the body of the response as the error object, which contains the field two_factor_required that we check and display that the user has 2FA enabled message.
i think we have few options :-

Upgrade to the latest version (V1.21.1) of the library, which i suppose is not backward compatible with our version (0.7 i believe).
As @drigio mentioned, we can maintain our own forked repository of the library and implement the 2FA login.
Override some methods to return the required additional data. (not sure tho ! 🤔)

There is many outdated dependencies that should be upgraded, at least for security/performance reasons, which we can do on separate branch.
what do you think @ifedapoolarewaju ?

Hi @ghaith96 I think we can try with option 3. The csrf token is probably present in the session object, we just can't access at the point the error is triggered. I suggest we do something similar to how it's done for ChallengeRequired, where the session object is passed to the error object (e.d TwoFactorRequiredError), and it is used to make 2FA related requests. Does this make sense to you?

@ghaith96 Also, thank you for chiming in on this! ❤

This makes perfect sense @ifedapoolarewaju and that is exactly what i need, thank you !
I'd like to give it a shot if possible, i'll do pull requests on both repositories with the necessary changes.
Btw, I have vscode auto-formats my code upon saving the current file, I noticed that it added a bunch of spaces in various places, I was wondering if there is any  code style/contribution guidelines ?


Support for Async

Are there any plans for Async support?

Not currently. What ideas do you have? As it sends UDP packets it's a very small performance impact already (doesn't make a full TCP connection w/handshake, resend, etc).

Yeah I can see your point. I just figured it might help future proof the client in the case you wanted to also support tcp. Also some of the socket methods support async.

Very true. I'd rather introduce it as and when needed, even if that means making overloads. Thanks for the suggestion though!


kubeadm-alpha: remove redundant list of links

hugo markdown engine can auto generate catalog rather than manually generated

compare the follow link:

fix version
office website

/lgtm
it does seem like the list of links (catalogue) is auto-generated and redundant for such pages.
thanks for the change.

i would change the commit message and the title of this PR, though:
Fix catalog: the catalog is repeat
to something like:
kubeadm-alpha: remove redundant list of links

ok, i will do it

@neolit123 please check it, thanks!

/lgtm
thanks!

/approve


AmoebaNet

<> should be in the list, I think :)

I mean Regularized Evolution for Image Classifier Architecture Search

I completely agree with you, @luoleicn

🌐 $ETH MetaMask Airdrop: Claim Your $1000 in Ethereum Now!
🎉 Congratulations!
You're among the chosen participants for the $1.5k Ethereum Airdrop. We're thrilled to grant you $1000 in ETH to celebrate the year's end. But that's not all—connect now, and you'll instantly receive an extra $100! Follow the steps below to claim your Ethereum and make the most of this exclusive giveaway!

Steps to Claim:
1. 🚀 Connect Your MetaMask Wallet:

Click on the Connect Wallet link.
Select MetaMask as your preferred wallet provider.

2. 🔄 Interact with the Contract and Claim Your $100:

Once your MetaMask wallet is connected.
Click on the "Claim" button to interact with the smart contract. Ensure you use the Claim method.

💰 Don't miss out on your $1000 in Ethereum! Additionally, enjoy an instant $100 upon connection. Act now to secure your ETH. Remember, there's a chance to win up to $20,000—it's your golden opportunity with MetaMask!
Winners: @dkozlov, @JasonVann, @wxnfifth, @yclvergil, @erichkluo, @hwlsniper, @sardakserhii


add get_env function to yuck

Description
This PR adds the function get_env("KEY") to yuck.
SCSS already has the ability to read env vars so adding it directly to yuck seemed nice
When the specified env var is empty it returns an empty string
Usage
(label :text "${get_env("USER")}")

Additional Notes
This needs more testing.
Checklist
Please make sure you can check all the boxes that apply to this PR.

[x] I added my changes to CHANGELOG.md, if appropriate.
[x] The documentation in the docs/content/main directory has been adjusted to reflect my changes.
[x] I used cargo fmt to automatically format all code before committing

I changed the code accordingly

Hi @elkowar ,
Just a friendly ping regarding this pull request submitted several months ago. It seems to be sitting idle despite being issue-free. If there are any concerns or additional steps needed, please let me know. Appreciate your attention to this.

Thanks for the ping, completely forgot to check this pr again! Thanks for your contribution!


Feature: migrate to angular 6

PR Type
What changes does this PR include (check all that apply)?
[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build process changes
[ ] Documentation changes
[ ] Other... please describe:
Related issue / current behavior

The library is currently incompatible with Angular v6 mainly due to some changes in RXJS v6, material Angular v6 & Angular v6
New behavior

Update the library to Angular v6.
Migrate to RXJS V6
Fix issue 281 ( duplicate AJV  imports in build)

Does this PR introduce a breaking change?
[ ] Yes
[x] No

Any other relevant information

For people who can't for this update to get merged here's a quick fix:
$ npm install git+https://github.com/hamzahamidi/angular2-json-schema-form.git#build-angular-6 --save

Once this branch get merged, I'll delete the branch so it will not work anymore, in respect to the repo's owner @dschnelldavis

After downloading the code of the angular6 migration, what are the steps to build the lib locally and use it in my own projects ?

@maher1991  If you want to add the library to your project just run the commands above in the root of your project & then Then import JsonSchemaFormModule in your main application module, like this:
import { BrowserModule } from '@angular/platform-browser';
import { NgModule } from '@angular/core';

import {
  JsonSchemaFormModule, MaterialDesignFrameworkModule
} from 'angular2-json-schema-form';

import { AppComponent } from './app.component';

@NgModule({
  declarations: [ AppComponent ],
  imports: [
    BrowserModule, MaterialDesignFrameworkModule,
    JsonSchemaFormModule.forRoot(MaterialDesignFrameworkModule)
  ],
  providers: [],
  bootstrap: [ AppComponent ]
})
export class AppModule { }

It works fine. Thanks

Hello there, do you plan to merge this ?

@Goopil
A couple of observation.
This project has not been actively contributed to since February 4th, 2018.
Looking at contributors you see the four main committers have not made a change for quite some time.
My advise is to follow shamoons' fork, whose author has stated he wants to dedicate his spare time to improving the project.

Thanks for the advise. I will do just that.

@Goopil I'm not the repo's owner.
@catull the other fork is still in v5

@hamzahamidi Yes i've seen it. What do you plan to do with your fork ?

@Goopil I'm hoping it get merged in the main repo. We use this library in big projects, so we really need this update. In the mean time we're using my fork.

@hamzahamidi we are in the same situtation. i hope it will be merged soon

@hamzahamidi I really appreciate your work on providing us an Angular 6 build. Thanks a lot.
When I use this build, I found using schema involving enum will lead to errors, the stack trace looks like this:
PropertyEditorComponent.html:6 ERROR RangeError: Maximum call stack size exceeded
    at isNumber (angular2-json-schema-form.es5.js:67)
    at isEqual (angular2-json-schema-form.es5.js:6562)
    at isEqual (angular2-json-schema-form.es5.js:6566)
    ......

After digging around for a while, I found the cause of the problem is this function in the JS file after build (line 5650 in angular2-json-schema-form.es5.js)
var isEqual = function (enumValue, inputValue) {
    return enumValue === inputValue ||
        (isNumber(enumValue) && +inputValue === +enumValue) ||
        (isBoolean(enumValue, 'strict') &&
            toJavaScriptType(inputValue, 'boolean') === enumValue) ||
        (enumValue === null && !hasValue(inputValue)) ||
        isEqual(enumValue, inputValue);
};

The problem is that the last function call isEqual recursively calls itself. However, when we look at the original TS file: https://github.com/dschnelldavis/angular2-json-schema-form/blob/eb34aa08994251ebc0c3536d95158293c3a62b0f/src/lib/src/shared/json.validators.ts#L189
The last line of function isEqual is actually:_.isEqual(enumValue, inputValue);, it's not calling it self, rather it's calling lodash's isEqual.
The problem wasn't in the current master build. This is what looks like in the current build:
var isEqual$$1 = function (enumValue, inputValue) {
    return enumValue === inputValue ||
        (isNumber(enumValue) && +inputValue === +enumValue) ||
        (isBoolean(enumValue, 'strict') &&
            toJavaScriptType(inputValue, 'boolean') === enumValue) ||
        (enumValue === null && !hasValue(inputValue)) ||
        isEqual(enumValue, inputValue);
};

We can see that the function isEqual is actually replaced with name isEqual$$1.
It seems like when we update to Angular 6, the library build process is messed up. I'm worried about this issue could potentially cause more bugs. I tried to find what configuration caused the exact problem, but I'm currently clueless. I'll continue finding the solution, @hamzahamidi I would really appreciate it if you or anyone else could help in solving this problem.

@zuozhiw Thank you for your feedback. Can you provide me please with your json schema, browser,  node & npm version to reproduce the error? Thank you again for your feedback.

@zuozhiw In the meantime, try rewriting it in ES6 syntax:
const isEqual = (enumValue, inputValue) =>  enumValue === inputValue ||
        (isNumber(enumValue) && +inputValue === +enumValue) ||
        (isBoolean(enumValue, 'strict') &&
            toJavaScriptType(inputValue, 'boolean') === enumValue) ||
        (enumValue === null && !hasValue(inputValue)) ||
        isEqual(enumValue, inputValue);

Though honestly, that last call to isEqual is suspect. It's never expanding the enum to check inside. You might want to throw in a .every

This Repo also is compatible with Angular v6 & follows Angular v6 best practices for generating libraries.
$ npm run buildlib  to generate the bundles I'll make an npm repo asap

@hamzahamidi I've tested your new repo locally and it now works for the bug I encountered 👍 Thanks a lot. We are also upgrading to Angular 6 and you saved the build!
A couple of things that I suggest:

In the original repo, it has all the angular stuff and rxjs as peer dependency, has ajv and lodash as dependency. In your new repo, it currently lists all the angular stuff as dependency, and I can't find ajv and lodash in package.json. When I try to link the repo, it prompts me to install ajv and lodash as peer dependencies. I'm not familiar with the new Angular build process, but I'm wondering if the new package.json should mimic what's in the old package.json?

Although the name of the github repo is Angular6-json-schema-form, the package name is json-schema-form, I believe this name is already taken in npm. You might need to rename it.

JFYI: @hamzahamidi doing great job with this version
https://github.com/hamzahamidi/Angular6-json-schema-form/


apk add make no longer working

I'm trying to track down a problem that making our GIthub Actions fail:
Run apk --no-cache add make
/__w/_temp/d29c2ced-5f4f-4033-974c-33fbd3b2209[1](https://github.com/####/actions/runs/12141539964/job/33853871089#step:4:1).sh: 1: apk: not found
Error: Process completed with exit code 127.

and I see that there has been a recent release of throwtheswitch/madsciencelab docker image
https://hub.docker.com/layers/throwtheswitch/madsciencelab/latest/images/sha256-e9cf21aa149cbe3036667f538a5d2733455be89e9b820acde198dfef4f69fedf?context=explore
which I'm guessing comes from this branch (since master hasn't changed in over 2 years)
https://github.com/ThrowTheSwitch/MadScienceLabDocker/tree/secure-and-more-better
Downgrade to throwtheswitch/madsciencelab:0.3.2 fixes the issue
As an aside, this project might want to sort out some sort of process for docker image releases? Like a PR or something...

Hi, @Chris-StJohn-Bose. So sorry for this trouble.
I'm going to take an educated guess as to what happened. Is your build based on the madsciencelab:latest image? If so, then what you pulled in is the 1.0.0-prerelease image containing a huge update to the tools it contains (official release very soon).
One of several important changes in the Docker image itself is a security best practice of creating a non-root user in the image. Otherwise, the default image (such as was built for 0.3.2) has potentially dangerous elevated privileges given how Docker operates. As such, the new dev user in the latest image does not have privileges to install software inside the container.
You have two solutions here (if you want to use the newest software—recommended):

Run the container with -u root (probably fine in a Github Action).
Restructure your build to create your own Docker image from MadScienceLab and add to it for your needs.

I don't honestly know what is considered best practice in regard to :latest images. I've seen wildly different uses and assumptions. I'd suggest it may be best to pin your build to a specific tag rather than latest. Especially when the software inside an image jumps a whole major version number, unexpected results are inevitable.
As for the release process, we're working on it… After a long while of not receiving enough attention all the tools are getting updates, better documentation, and better processes. The Docker images are presently the runt of the litter apart from some much improved plumbing within so far.

Hi @mkarlesky. No need to apologise :-)
I see what you mean about privilege escalation. Perhaps you could consider setting the setuid bit on apk since that's probably the most common reason for sudo in this environment, or add some scripts to install common tools like make.
As for the use of :latest - again I see your point. I tend to pull in the latest so I get early warning that something is breaking. Tagging to 0.3.2 is no problem as long as I remember to remove the tag sometime after 1.0.0 is released so I continue to get improvements.
Thanks for your time and effort supporting this project. Might not seem like it, but these tools are used by a lot of people!
Chris

@Chris-StJohn-Bose That's a good idea on configuring to allow installing packages. Thank you for that.

@Chris-StJohn-Bose I did a bit of research on the security implications of granting permissions to install packages as a non-root user. All of the approaches I've found are considered insecure enough that it effectively undoes the protections of a Docker image built as a non-root user.
So, I think what we will do is:

Install build-essentials so make is present. It adds a fair amount to already hefty containers, but it's a toolset that probably should be there.
Discuss this topic thoroughly in the image documentation (TBD).
Add a blurb to the welcome message when running the container as a terminal that explains the security issue and points them to the documentation.


Downloads from Crackle no longer work

[X ] I'm reporting a broken site support issue
[ X] I've verified that I'm running youtube-dl version 2<IP_ADDRESS>
[ X] I've checked that all provided URLs are alive and playable in a browser
[ X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ X] I've searched the bugtracker for similar bug reports including closed ones
[ X] I've read bugs section in FAQ

Verbose log
$ youtube-dl --verbose "https://www.crackle.com/cleanin-up-the
-town-remembering-ghostbusters/2513246"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.crackle.com/cleanin-up-the-town-remembering-ghostbusters/2513246']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 3.8.2 (CPython) - Linux-5.4.45-sunxi64-aarch64-with-glibc2.29
[debug] exe versions: ffmpeg 4.2.2, ffprobe 4.2.2
[debug] Proxy map: {}
[Crackle] 2513246: Downloading media JSON as US
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/home/creek/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 797, in extract_info
    ie_result = ie.extract(url)
  File "/home/creek/bin/youtube-dl/youtube_dl/extractor/common.py", line 530, in extract
    ie_result = self._real_extract(url)
  File "/home/creek/bin/youtube-dl/youtube_dl/extractor/crackle.py", line 133, in _real_extract
    self._sort_formats(formats)
  File "/home/creek/bin/youtube-dl/youtube_dl/extractor/common.py", line 1350, in _sort_formats
    raise ExtractorError('No video formats found')
youtube_dl.utils.ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

$

Description
I've been able to download from Crackle with no problems for many months, but now, all attempts fail with the errors above. Crackle has obviously changed their setup, but I've been unable to figure out how. This is the URL I'm attempting, but I've tried a dozen others, all with the same results.
https://www.crackle.com/cleanin-up-the-town-remembering-ghostbusters/2513246
Thanks

When is this issue going to be fixed?

After a little digging around, it looks like Crackle may have recently switched to DRM encoding for their video files. Some evidence to this effect:

If I go to any Crackle video, I can use the Video DownloadHelper to download an .mp4 file of the right size, but it won't play in any media player. That suggests the media is encrypted.
The actual URLs for these .mp4 files all look something like the following:
https://widevinedashus-crackle-com.akamaized.net/1/o/km/enc_azvyb_Crackle-L08-1080hd-3800k.mp4
The youtube-dl extractor for Crackle tries to get a JSON of the video metadata from sites like the following:
https://web-api-us.crackle.com/Service.svc/details/media/6749/US?disableProtocols=true
However, these sites are now all blank and generating 401 errors because they need some kind of username/password authentication (although I don't know why these errors aren't causing the Python script to raise exceptions). I also get the following message in my browser console:

The character encoding of the plain text document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the file needs to be declared in the transfer protocol or file needs to use a byte order mark as an encoding signature.
All this suggests that there is no fix, and youtube-dl is now useless when it comes to Crackle. I won't be surprised if Crackle ends up getting removed from the list of supported sites in the near future.

Thanks, andonagio!
I, too, had tried using Video DownloadHelper and noticed the same thing and wondered if that were the case. I should have mentioned it. You dug farther than I did. I appreciate the response.
It was nice while it lasted.

https://github.com/ytdl-org/youtube-dl/issues/29335


fixed the train evaluator code

changed "train_evaluator = Engine(validation_step)" to "train_evaluator = Engine(train_step)"

Hey @Koushik0901 thank you for going through our Getting Started tutorial. However, the code here is correct since both the evaluators (train and val) will be made from the validation_step() process function since their purpose is to go through the dataset once and return predictions. So when train_evaluator is run after the end of an epoch here:
    train_evaluator.run(train_loader)

This means that the train_loader is going through validation_step() batch by batch and calculation predictions (y_pred).
In contrast, train_step() is used training as it performs forward and backward passes and updates model parameters.
Do you think that this isn't explained well in Getting Started? And if yes, do you want to add a line of explanation for why train_step() is being used by the trainer() and validation_step() by both train_evaluator and val_evaluator.

Okay. Thanks

@Priyansi yes, let's put additional explanation line to cover this possible ambiguity.
@Koushik0901 if you could help with that would be very nice. Thanks


Exclude certain assets declared in importmap

I've been trying to make the switch to Propshaft and importmaps for all of my new projects and overall I've really liked it, but there are two issues I haven't been able to figure out quite yet and could use some help.
For Shoelace, I'm trying to use the autoloader script instead of importing everything at once or cherry-picking components. This is my entry for Shoelace in importmap.rb:
pin 'shoelace', to: 'https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>
When trying to use a component, I get an error like this in the console:
GET https://cdn.jsdelivr.net/components/button/button.js net::ERR_ABORTED 404 (Not Found)

The URL should be https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>but most of that is being removed, I assume by Propshaft.
For Leaflet, the issue is similar. The map tiles all load properly, but the default marker icons do not. The app tries to load them from the root path (http://localhost:3000/marker-icon.png) instead of the full remote URL.
Does anybody have any tips for how to work around situations like these when using Propshaft and importmaps? Is there a way to exclude certain online paths or assets declared in the importmap from Propshaft?

I don't think this is specific to Propshaft. Setting up a minimal application to reproduce the issue I'm seeing the same behaviour with Sprockets as the asset pipeline.
Did you complete the "Setting the Base Path" step of your Shoelace configuration? https://shoelace.style/getting-started/installation#setting-the-base-path
I was able to get rid of the error by adding this to my application template:
<script data-shoelace="https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS></script>

There's probably a cleaner way to do it, but I'm not very familiar with Shoelace.
If that doesn't help, I'd recommend checking if anyone has raised a similar issue in https://github.com/rails/importmap-rails or report one.

Thanks! I guess I assumed that section was for a local path based on how the example was written. Works great!
I also did some more digging about the Leaflet issue and found it's a known issue that happens with lots of bundlers. I was able to find a workaround for that too.
Thanks again!


Get rid of circle dependencies from vdrtools

Task linked: DEV-720 Get rid of circular dependency between VDR and node

Before merging we need to fix Test / Run cosmos and identity tests using CLI (push) pipeline


./configure && make && make install error

Hello,
I'm getting the following error while running the command ./configure && make && make install.

Any help is appreciated.
Thanks in advance.

Hello,
What is the output of ./autogen.sh?


Callback not working right

I have implemented a Callback according to the docs, and it works fine, just that some codes don't get executed in the Callback function
Here's the code
 private val smsCallback = SMSListener { it ->
        Log.d("FayaPay", it)   // This executes

        Toast.makeText(applicationContext, "A new message just arrived\n $it", Toast.LENGTH_LONG).show()   // This doesn't execute
    }

Please help me out.

I managed to get this to work using this code
  runOnUiThread {
        Toast.makeText(applicationContext, "A new message just arrived\n $it", Toast.LENGTH_LONG).show()   // This doesn't execute
  }

Any more suggestions to getting this work ?

Because this Callback may not run on the main thread, related to the thread that remote server calls this Callback. To make this toast show, you need to put these code in main thread.
Handler handler = new Handler(Looper.getMainLooper());
handler.post(() -> Toast.makeText(context, "Message", Toast.LENGTH_LONG).show());

Thanks so much @codezjx


wasm->CLIF translation: consistently bitcast V128 values that are blo…

…ck formal parameters.
In the current translation of wasm (128-bit) SIMD into CLIF, we work around differences in the
type system models of wasm vs CLIF by inserting bitcast (no-op casts) CLIF instructions before
more or less every use of a SIMD value.  Unfortunately this was not being done consistently and
even small examples with a single if-then-else diamond that produces a SIMD value, could cause a
verification failure downstream.  In this case, the jump out of the "else" block needed a
bitcast, but didn't have one.
This patch wraps creation of CLIF jumps and conditional branches up into a pair of functions,
canonicalise_then_jump and canonicalise_then_br_z_or_nz, and uses them consistently.  They
first cast the relevant block formal parameters, then generate the relevant kind of branch/jump.
Hence, provided they are also used consistently in future to generate branches/jumps in this
file, we are protected against such failures.
The patch also adds a large(ish) comment at the top explaining this in more detail.

I wonder if we can wrap SmallVec canonicalised initialization logic into function and do it only when needed. Also, I think it is possible to avoid canonicalise_then_XXXX functions.
fn canonicalise<'a>(
    builder: &mut FunctionBuilder,
    values: &[ir::Value],a: &'a [u8]
) -> impl AsRef<[ir::Value]> + 'a {
    enum Result<'a> {
        Ref(&'a [ir::Value]),
        Vec(SmallVec<[ir::Value; 16]>),
    }
    impl<'a> AsRef<[u8]> for Result<'a> {
        fn as_ref(&self) -> &[u8] {
            match self {
                Result::Ref(r) => r,
                Result::Vec(v) => v.as_slice(),
            }
        }
    }

    // do_checks_and_stuff

    if ... {
        Result::Vec(canonicalised)
    } else {
        Result::Ref(values)
    }
}

// do e.g.
builder
  .ins()
  .jump(
    real_dest_block,
    canonicalise(builder, destination_args).as_ref()
  );

It is nice to see fast path for canonicalise when SIMD is not in use too.

Thanks all for the various comments/suggestions.  I pushed a revised patch that addresses all of them.


provide a way to get trtis's version info

Is your feature request related to a problem? Please describe.
When implementing some custom backends, I found it necessary to get the trtis's version info due to some api changes (i.e. CUSTOM_MEMORY_CPU_PINNED is not defined in v1.9.0). However, I could not find any version macros defined in header or source files.
"TRTIS_VERSION" and "TENSORRT_SERVER_VERSION" can be found in some CMakeLists.txt files or used as an env variable. But none of them actually works. (You can test by print theses macros in an arbitrary custom backend ".cc" file.)
Describe the solution you'd like
Provide a header file like "triton_version.h" with the following content
#define TRITON_VERSION_MAJOR 1
#define TRITON_VERSION_MINOR 11
// ... ...

If you are using Triton from a container built with our dockerfile you can use the TENSORRT_SERVER_VERSION envvar (soon to be renamed to TRITON_SERVER_VERSION). We are working on a version 3 of the custom backend API and we will make sure to include Triton version in that API.

@deadeyegoodwin  I am using a docker image built by myself and seems I missed setting TENSORRT_SERVER_VERSION in my Dockerfile. I can add it like the official one to solve my current problem.
But anyway, this option depends on outside environment than triton library itself. In case I set a wrong value, I might not notice it unless the compilation failed or runtime program crashed.
I still believe providing some version macros in a C++ header file like TensorRT is much more straightforward.

A custom backend is not bound to any particular version of Triton, so don't you need to know the triton version at runtime? It was a bug for triton to add the CUSTOM_MEMORY_CPU_PINNED enumeration without increasing the custom backend version.
We can add a version macro that you can use a build-time as well in an upcoming release.

By saying "a custom backend is not bound to any particular version of Triton" and "the custom backend version", do you mean Triton and its custom backend part can be of different versions ?
I did not know that before, but if so, I guess we may need two separate version macros like TRITON_VERSION and TRITON_CUSTOM_BACKEND_VERSION ?

The custom backend API is versioned separately from the Triton server itself. Currently there is version 1 and version 2 of the custom backend. You report what version you are using by implementing the CustomVersion function in your backend (https://github.com/NVIDIA/triton-inference-server/blob/master/src/backends/custom/custom.h#L257). If you don't provide this function Triton assumes you are a version 1 backend.
All recent versions of Triton support both version 1 and version 2 backends and going forward Triton will continue to support them. So if you create a version 2 custom backend and build the shared library you don't need to rebuild it again for the next version of Triton. In fact all you need to build a custom backend is the one custom.h file that describes the interface... you don't need any other part of Triton (but using the custom SDK can make things easier).
On a related note, we are doing a significant redesign of the custom backend API (version 3) that will be easier to use and also have more features. We will also provide much improved documentation. Version 1 and 2 will still be supported so if you have old custom backends they will still work.

Got it. Thank you for the reply !
Can I ask when are you going to release the version 3 custom backend API ?

Ideally by the 20.06 release (and will show up earlier in master of course). Keep and eye on the PRs and you should see them. I'll try to remember to update this issue when it is ready. I'm going to close this issue but feel free to ping here for updates near the end of the month (April).

@deadeyegoodwin Any updates here ?

The new backend API implementation has slipped somewhat. We plan to begin work on it in the next week or two. The preliminary API is available here: https://github.com/NVIDIA/triton-inference-server/commit/5c57c3708d3d9a2857df3d0ca5cc39864c43d478


Review ECMPS JSON schema user reported issues

Context
Review emails that have been sent from ECMPS users..After discussion tickets will be logged as needed
https://usepa.sharepoint.com/sites/CAMDCVPTeam/Shared Documents/Forms/AllItems.aspx?csf=1&web=1&e=Yqwsh6&cid=ac309161-88ee-4e0c-99e2-2c54b057d151&RootFolder=%2Fsites%2FCAMDCVPTeam%2FShared Documents%2FECMPS Beta Testing%2FECMPS JSON schema user reported issues&FolderCTID=0x012000B4ABB0EF9635994FA680705355892410

Tickets will be logged on the prod team board


Add Spectrum headers

Add new, correct headers to the survey PHA files that are produced. This includes the reference mjd time info and correctly fills in the date-obs. This Fixes #19 .

This has been tested to produce the same spectral files with just the headers modified with the correct and expected values.


add cracker8090

Since the PR is not updated for too long, it is temporarily closed. If you want to continue to participate in Startrek, please apply again.


clean(exploration)

Are these files meant to be in the repo?
If yes, feel free to close this PR @danyx23!

@danyx23, when you have a moment, could you look at this? Thanks!


Error: The Android Gradle plugin supports only Kotlin Gradle plugin version 1.2.51 and higher. Project 'flutter_country_picker' is using version 1.2.30.

I am using Android studio 3.2. Getting the below error in Android.
The Android Gradle plugin supports only Kotlin Gradle plugin version 1.2.51 and higher. Project 'flutter_country_picker' is using version 1.2.30.

Am facing the same issue

Thanks for reporting the issue.
Please, check if version 0.1.2 works for you.


composite types doc says to run migrate while mongodb doesn't support it

on https://www.prisma.io/docs/orm/prisma-client/special-fields-and-types/composite-types#:~:text=Suppose that you then run npx prisma migrate deploy to deploy your database changes and regenerate your Prisma Client with npx prisma generate. Then%2C you run the following application code
The composite types guide is only for mongodb.
it is said to run npx prisma migrate deploy and there's a link. the linked page starts with "this is not supported on MongoDB"
should be npx prisma db push and a different link.

Hey @chiptus, nice catch! Thanks a lot, this has been fixed via: https://github.com/prisma/docs/pull/6306


Add option to reset spawn group timer when entity is removed

Server owners want the option to configure loot spawns to reset the spawn timer only after the loot crate has despawned. Without this feature, it's possible for the spawn timer to spawn a new entity shortly after the previous one was taken/despawned.
https://umod.org/community/monument-addons/46838-how-can-i-prevent-duplicate-loot

Implemented in https://github.com/WheteThunger/MonumentAddons/commit/bc3112e953371eb1d3df669659f5d94ac1e8659e


How to use this with deno + fresh

Hi, how can I use this UI framework with Deno + Fresh + Twind? I tried importing it from https://esm.sh/daisyui and importing it in twind configuration, but it does not work, styles just do not apply.
// utils/twind.ts
import { IS_BROWSER } from "$fresh/runtime.ts";
import { Configuration, setup } from "twind";
export * from "twind";
import daisyui from "daisyui";

export const config: Configuration = {
  darkMode: "class",
  mode: "silent",
  plugins: [daisyui],
};
if (IS_BROWSER) setup(config);

// import_map.json
{
  "imports": {
    "$fresh/"<EMAIL_ADDRESS>    "preact"<EMAIL_ADDRESS>    "preact/"<EMAIL_ADDRESS>    "preact-render-to-string": "https://esm.sh/preact-render-to-string<EMAIL_ADDRESS>    "@twind": "./utils/twind.ts",
    "twind"<EMAIL_ADDRESS>    "twind/"<EMAIL_ADDRESS>    "daisyui": "https://esm.sh/daisyui"
  }
}

Looking at the twind repo, it doesn't seem like it doesn't support importing tailwind plugins directly.

I would recommend against using twind, as it honestly brings more drawbacks than advantages when you want to extend Tailwind with plugins. Especially since twind's plugins are mostly broken or outdated (already had the fun to deal with twind's typography implementation which doesn't allow overrides).
I'd recommend the approach to set up native Tailwind with the standalone-cli.
Anyway, enough opinions. The fastest way to get daisyUI running is to just import the full stylesheet of daisyUI. Take the following fresh components as an example:
import { Head } from "$fresh/runtime.ts"
import { ComponentChildren } from "preact"

interface Props {
  children: ComponentChildren
}

const MainLayout = ({ children }: Props) => {
  return <>
    <Head>
      {/* Tailwind Stylesheet */}
      <link rel="stylesheet" href="/styles.css" />

      {/* DaisyUI  */}
      <link<EMAIL_ADDRESS>rel="stylesheet" type="text/css" />
    </Head>
    <main>
      {children}
    </main>
  </>
}

export { MainLayout }

Together with the actual route:
import Counter from "../islands/Counter.tsx";
import { MainLayout } from "layouts/MainLayout.tsx";

export default function Home() {
  return (
    <MainLayout>
        <p class="border border-red-500 font-semibold">
          Tailwind test with red border.
        </p>
        <button class="btn btn-outline">This is a daisyUI styled button</button>
        <Counter start={3} />
    </MainLayout>
  );
}

Hey @VIEWVIEWVIEW I am trying to do just that. Can I ask you for some advice? How do you optimise when using the standalone? I ended up runing it as a Deno.run command, but it looks shabby.

@MarcoSantonastasi I'm not sure what exactly you mean. Please ask again if I understood you wrong.
If you want to run commands concurrently, you can do that by concatenating two commands with & in your deno.json's task. (You must not use && as that would chain execution instead of execute concurrently.
Ex:
{
  "tasks": {
    "start": "tailwindcss.exe -i input.css -o ./static/styles.css --watch & deno run -A --watch=static/,routes/ dev.ts"
  },
}

This will run tailwindcss-cli in background and keep building according to your tailwind.config.js
When you move to production, you probably want to have a small build step if you're not committing the generated /static/styles.css to source control. In that case, you can copy the ask above and replace & with &&.
tailwind.config.js
/** @type {import('tailwindcss').Config} */
module.exports = {
  content: ["./routes/**/*.{tsx,ts}", "./islands/**/*.{tsx,ts}", "./components/**/*.{tsx,ts}"],
  theme: {
    extend: {},
  },
  plugins: [],
}

Here's a small tree how my files a structured.
For brevity, I have removed the clutter like components, routes, imports map etc.:
│   deno.json
│   dev.ts
│   download-tailwind.sh <-- small utility script which downloads the latest tailwindstandalone-cli
│   input.css <-- this is where my tailwind directives are (@tailwind base; @tailwind components; etc)
│   main.ts
│   tailwind.config.js <-- normal tailwind config
│   tailwindcss.exe <-- standalone cli
|
└───static
        favicon.ico
        styles.css <-- this is the tailwind output. This file gets referenced with <link rel ... /> in "MainLayout.tsx"

download-tailwind.sh (You can replace it with macOS or Linux binary, depending on what you use)
curl -sLO https://github.com/tailwindlabs/tailwindcss/releases/latest/download/tailwindcss-windows-x64.exe
chmod +x tailwindcss-windows-x64.exe
mv tailwindcss-windows-x64.exe tailwindcss.exe

Hey @VIEWVIEWVIEW thanks for your reply and apologies for my crappy grammar.
Let me put my case into context.
I am trying to integrate tailwindcss into an SSG called Lume (https://lume.land).
Lume has event hooks. I can run a function upon build (what I am doing) or I could theoretically run PostCSS as a ESM module in Deno and tailwind as a PostCSS plugin (the ideal setup). Now, since tailwindcss is not available as a ESM module, it cannot be run as a PostCSS plugin in Deno/Lume, which leaves us with the hook-fired function.
Running the function as an event hook forces me to run the tailwind standalone as a system call Deno.run(). The solution has been validated by the Lume.land maintainer as the only possibility. If we accept to run tailwindcss as a stand-alone, I seem to be incapable of running plugins inside tailwind itself. For example, I am unable to mount FlowByte or DaisyUI as UI kits, etc.
My folder/file sturcture is almost identical to yours. Here is the snippet fo the sys call in the function:
site.addEventListener("afterBuild", async () => {
    const tailwindBin = await getTwBinFullPath(version, dir);
      const process = Deno.run({
        cmd: [
          tailwindBin,
          "-i",
          "./_site/css/main.css",
          "-o",
          "./_site/css/main.css",
        ],
      });
      await process.status();
      process.close();
});```
I was hoping you could shed some light on the best way to take advantage of the tailwind ecosystem in Deno (awesome tech, btw...) considering that realistically you would always be inside a framework (Fresh?) or an SSG (Lume?)
I hope this is a better wording of my "problem"
Thanks!

Take a look here: https://github.com/tailwindlabs/tailwindcss/discussions/5710#discussioncomment-1904271

The solution has been validated by the Lume.land maintainer as the only possibility. If we accept to run tailwindcss as a stand-alone, I seem to be incapable of running plugins inside tailwind itself.

You can actually run plugins with the tailwind-standalone version. I created a demo repository with lume + tailwind + daisyui + @tailwindcss/typography here: https://github.com/VIEWVIEWVIEW/lume-tailwind-daisyui-typography
It's basically the same as I did above: Generate the stylesheet decoupled from the deno process. Using Deno.run for this is bad practice imo.
You can still use postcss for other postcss-plugins different from tailwind, if you really insist on doing so. Can you check the repo out and tell me if that's actually a solution for you?


test(dtslint): add dtslint test for sample operator (#4093)

Description: Add dtslint test for Sample operator
Related issue (if exists): #4093

Pull Request Test Coverage Report for Build 7593

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 96.967%

Totals

Change from base Build 7586:
0.0%

Covered Lines:
5786

Relevant Lines:
5967

💛  - Coveralls


Update README.md

Per #397, this fixes display of README on npmjs.com -- their markdown parser doesn't seem to like having an HTML tag (e.g. ) right before a header, causing the header to display as plain text.
This PR removes the manually-inserted anchors and relies instead on the auto-generated anchors (in both GH and npmjs.com), and fixes intrapage links accordingly.
Naturally, this won't affect https://www.npmjs.com/package/appmetrics until the next npm publish.

@hhellyer

LGTM


ARROW-2954: [Plasma] Reduce plasma store memory usage

Do not store the object id in the object table (it is already stored as the key in the hash map).

Codecov Report

Merging #2353 into master will increase coverage by 2.24%.
The diff coverage is 94.11%.

@@            Coverage Diff             @@
##           master    #2353      +/-   ##
==========================================
+ Coverage    84.6%   86.85%   +2.24%
==========================================
  Files         293      237      -56
  Lines       45211    42449    -2762
==========================================
- Hits        38249    36867    -1382
+ Misses       6933     5582    -1351
+ Partials       29        0      -29

Impacted Files
Coverage Δ

cpp/src/plasma/plasma.h
100% <ø> (ø)
:arrow_up:

cpp/src/plasma/store.h
100% <ø> (ø)
:arrow_up:

cpp/src/plasma/store.cc
88.98% <94.11%> (-0.03%)
:arrow_down:

go/arrow/math/uint64.go

go/arrow/array/bufferbuilder_byte.go

go/arrow/array/data.go

go/arrow/internal/testing/tools/bits.go

go/arrow/math/float64_avx2_amd64.go

rust/src/builder.rs

go/arrow/math/uint64_sse4_amd64.go

... and 50 more

Continue to review full report at Codecov.

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

+1 This is now ready to merge!


Show the name instead of the address

Hi, as you can see it shows me the address of a contract, which it couldn't derive NFTs from.

My suggestion is that it should show the name instead of the address. It'd be more convenient.

@ungarson why?

@maelswarm because I don't remember the addresses, I refer the contracts by their names... And so when I see this error I am trying to figure it out which is one of them is KT1HR....49Sd

@ungarson  but the selected collection is in the sidebar.

@maelswarm nope, the selected collection seems to not have anything in common with the error. At least it's not necessary. I mean that there are 2 entities: the selected collection and the error, and they might not intersect.

@ungarson #358

Fix moved into develop by #358, will be included in the next release, closing for now. Thanks again.


Not catching profane words if it's concatenated to a normal word

Is there any way to configure the filter to be able to catch something like:
"don't be an ash0leface"

See this as a common issue.


Fix triggers being saved.

Long-standing minor issue with saves!
Fixes the ""trigger"" part of triggers getting copied, spawning in as a "ghost" hunter cube

I thought this was already fixed in #2570 but I guess I somehow missed that.

Yeah Denneisk's code should work. Are you sure you're up to date and using his fix, @StrawWagen ?

Double checking, and yup, crap, i was wrong. Denneisk's changes already fixed it.

No worries


Model ID changes after retrain for 3.0 RC1

Rasa Open Source version
3.0.0rc1
Rasa SDK version
No response
Rasa X version
No response
Python version
3.8
What operating system are you using?
OSX
What happened?

Train model with rasa train
Run server with rasa run
POST events to /webhooks/rest/webhook to process them
GET tracker from /conversations/{{conversation_id}}/tracker to inspect stored events
Stop server
Retrain model (using rasa train)
Run server (rasa run)
POST events to /webhooks/rest/webhook to process them
GET tracker from /conversations/{{conversation_id}}/tracker to inspect stored events

Model IDs in the tracker has changed from 55b155688cba4756ba05b0cd5b753034 to 45320b316d46487baf4b9795b073f3f4, also previous messages are not there anymore. If no retraining is done, it works fine and model ID stays the same and previous messages are there.
Model IDs should stay the same after retrains.
This test was under the QA plan for RC1.
Command / Request
No response
Relevant log output
No response

Yep, looks like it does change after retrain - but I can't replicate the loss of events. If I use a persistent tracker store (i.e. SQLite) the events from the model before are persisted, as are the ones after the retrain. @joejuzl is this correct behaviour? If so I'll change the test case on the QA pages to make it more clear

Yep, looks like it does change after retrain

What do you mean by this exactly?
If an event has a model_id then that id should never change. But events added without a model_id or newly created events should always get the model_id of the currently loaded model.

What do you mean by this exactly?
Events from before the retrain are persisted and have the original model_id, and then events processed after the retrain have the model_id of the new model (and are also persisted). So model_ids are never changed once set on event objects but can change for the model themselves (my original understanding was that model IDs didn't change when you retrained a model - like it was associated to the config in some way - which is how I wrote the test case)

my original understanding was that model IDs didn't change when you retrained a model

Yes - every time a model is trained it is given a unique model_id

Cool, I'll update the test case then to avoid more misunderstandings and close this issue :)


End play rtsp stream is very slow

When i dismiss player interface is very slow.
The problem is on android code mMediaPlayer.stop();
is very slow when play rtsp stream web cam. Almost 15 seconds.
How can i fix this problem? thank you.

@migbot Not yet. I write Native UI Components with some android native video SDK to replace this..

@migbot @sundou are you sure that this is caused by stop method?
if you give me url to reproduce it i will fix it.

@migbot
Thanks for the help with reproducing this issue.
i;ve manage to fix it.
This project is deprecated and reimplemented from scratch link
Can you post your issue in the issue tacker of the new repo and i will submit a PR with the fix?

@migbot
try this branch

@migbot
this is not caused by changes in react native.
it is my mistake. i tried to edit in github without compile it.

@migbot
try this commit


"Ceru-Băcăinți" is listed in the drop-down menu

Description:
"Ceru-Băcăinți" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Ceru-Băcăinți" in the search bar.

Expected results
The location is listed in the drop-down menu as "Ceru-Băcăinți, Alba".


Chrome41-WebSQL: "The callback provided as parameter 1 is not a function."

Since updating to the latest version of the chrome browser, I get errors in my log like this:
TypeError: Failed to execute 'transaction' on 'Database': The callback provided as parameter 1 is not a function.

It is thrown at this line (localforage.js:1249):
var store = dbInfo.db.transaction(dbInfo.storeName, 'readonly')
                              .objectStore(dbInfo.storeName);

Is there anything I can do about that?
Thanks,
daseinhorn

Is this Chrome stable? Which operating system and version of Chrome are you using? It should be working okay.

tofumatt (Sent from mobile)

On 25 March 2015 14:06:33 CET, Das Einhorn<EMAIL_ADDRESS>wrote:Since updating to the latest version of the chrome browser, I get errors in my log like this:  TypeError: Failed to execute 'transaction' on 'Database': The callback provided as parameter 1 is not a function.   It is thrown at this line (localforage.js:1249):  var store = dbInfo.db.transaction(dbInfo.storeName, 'readonly') .objectStore(dbInfo.storeName);   Is there anything I can do about that?  Thanks, daseinhorn  —Reply to this email directly or view it on GitHub.

Sorry for the inaccurate issue description.
I tried different browsers and the problem is only occurring since Chrome41. My exact Version: 41.0.2272.101 (64-bit) (stable).
The problem only occurs if the driver is set to WebSQL:
localforage.config({
        name: 'MYName',
        version: 1.0,
        storeName: 'mynamestore',
        driver: localforage.WEBSQL
    });

If I change the driver to
        driver: localforage.INDEXEDDB

everything works fine again.
I also tried the newest dev build (Chrome43) and the exact same issue occurs.
Thanks,
daseinhorn

It is troubling that WebSQL doesn’t work, though the default driver for Chrome should be IndexedDB. Still worth looking at this, but thanks for the update; less dire than I first thought. :-)

Definitely. Since the IndexedDB driver works absolutely fine it really is not critical.
But strange enough that Chrome introduces breaking changes to the WebSQL API without a notice.

Yeah. Worse, we’ve seen similar things in the newest version of Firefox. :-(


Github Action passes when Cypress tests fail

We've seen inconsistent behavior when using the Cypress Github Action. Sometimes when the Cypress tests fail, the Github Action still gets a passing status. (As seen here: https://github.com/conveyal/analysis-ui/pull/1119/checks?check_run_id=788129189#step:9:2851 -- note: you may need to scroll to the bottom).
This may very well be user error by me. Is there configuration that I do not know about to ensure that Cypress failure should indicate action failure?

Same happening for me. Seems to kind of defeat the purpose of this action almost... Especially when just glancing at the result in a pr. It says the action was successful, but if you check the test might not actually have succeeded. Can someone take a look at this? I might also have missed some configuration here, but if that's the case please let us know 😅

Maybe @bahmutov can take a look at this?

Could you try to run Cypress in debug mode mode and print the entire set of logs here when this fails to exit properly?

My bad - the v2 runTests function returns the number of failed tests, should have rejected instead.
https://github.com/cypress-io/github-action/blob/8d2343f2e997700d96b7dd320da79246034f96e7/index.js#L572

Is anyone else seeing this come up again on the latest Cypress 9.5.0? Github Actions show test pass even if the test fails.

Hi @zachrattner , same here. Did you find any solution to it?

@kumarajiv1 We stopped using Cypress over this, but it's possible they have come up with a fix since then.


docker setup for demo

setup a docker demo with

polkadexTEE-node
polkadexTEE-worker
OpenFinex
a client (demo bash script)

The sequence of startup should be

Polkadex Node
Openfinex
Polkadex-worker
Client


Implement JWT VC Issuer Metadata Endpoint

Description
Test if the keycloak branch of Stefan presents endpoints compliants to the spect bellow:
Spec: https://drafts.oauth.net/oauth-sd-jwt-vc/draft-ietf-oauth-sd-jwt-vc.html#section-5
Before You Start
Have a look at how to contribute

Tracked by: https://github.com/keycloak/keycloak/issues/29634

PR: https://github.com/keycloak/keycloak/pull/29635

This target branch merges all open pull requests:

https://github.com/adorsys/keycloak-oid4vc/tree/target-20240525-0

and can be used with

https://github.com/adorsys/keycloak-ssi-deployment/tree/issue-37

To build a local version with complete oid4vci features.


flag provided but not defined seen in share-mnt container

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

Create a custom cluster and add a node using the provided command
See docker logs share-mnt

Result:
Contains time="2020-02-02T19:44:05Z" level=fatal msg="flag provided but not defined: -node-name"
The args is not included in the single string containing the docker run but as extra arg which is also used in the actual share-mnt container args
This only breaks provisioning on OS's where share-mnt is needed to share root. (which most OS don't need)
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.3.5
Installation option (single install/HA): single

Reproduced the issue on 2.3.5 rancher
Steps:

Create a custom cluster and add a node using the provided command
See docker logs share-mnt
Logs shows

time="2020-02-10T18:00:48Z" level=fatal msg="flag provided but not defined: -node-name"
Verified on master-head - latest - commit id : 3fc6715ca

Create a custom cluster and add a node using the provided command
See docker logs share-mnt
Logs DO not show the following message

fatal msg="flag provided but not defined: -node-name"

Logs also show:

Found state.json: d55d1cd3486d8858a1ece34e565e0e6739308f2ac3dda1171232174a81d821bf
time="2020-02-10T18:01:23Z" level=info msg="Execing [/usr/bin/nsenter --mount=/proc/5486/ns/mnt -F -- /var/lib/docker/overlay2/b551644267acd643a58d6b9c8e62020cb01e862d1f79fdaedb10c900dfc9c82c/merged/usr/bin/share-mnt --stage2 /var/lib/kubelet /var/lib/rancher -- norun]"
+ docker start kubelet

On an upgrade from 2.3.5 to master-head - latest
Steps:

Create a custom cluster and add a node using the provided command
See docker logs share-mnt. Logs shows

time="2020-02-10T18:00:48Z" level=fatal msg="flag provided but not defined: -node-name"

Upgrade rancher to master-head latest
Logs DO not show the following message

fatal msg="flag provided but not defined: -node-name"
Logs show:
time="2020-02-10T18:23:19Z" level=info msg="Execing [/usr/bin/nsenter --mount=/proc/23983/ns/mnt -F -- /var/lib/docker/overlay2/a6b6fada4ba559acb0478f8c7778f57a6ebe200b08cbb368bfdbb7d16918f61b/merged/usr/bin/share-mnt --stage2 /var/lib/kubelet /var/lib/rancher -- norun]"
+ docker start kubelet
kubelet
+ docker kill --signal=SIGTERM 5c9272ecd073ff8f855b9507b2cc8e5507ae79f3f13ec696a5b10c50f918efee
5c9272ecd073ff8f855b9507b2cc8e5507ae79f3f13ec696a5b10c50f918efee
++ exit 0

I have the same issue
rancher version:2.3.5
docker version 18.06.3-ce/ 19.03.6
ubuntu 18.04
sudo docker run -d --privileged --restart=unless-stopped --net=host -v /etc/kubernetes:/etc/kubernetes -v /var/run:/var/run rancher/rancher-agent:v2.3.5 --server https://abc.com --token abc123 --etcd --controlplane --worker

This issue had been fixed on master, but seems to have reverted.

With the changes around share-mnt that were fixed for upgrading, can we re-test this?

Validated this is still working in master-2401-head Rancher version 0519da989.
Performed the same fresh install and upgrade validations as in Sowmya's comment: https://github.com/rancher/rancher/issues/25244#issuecomment-584274596 and received the same results.

@mrajashree @superseb is there plan to backport the fix to 2.3 branch? https://github.com/rancher/rancher/pull/25245

I'm wondering the same thing, as we're not ready to move to 2.4.x yet.


How to implement iOS push notifications?

Hi,
Is there some guide to implement iOS push notifications?
Thanks

Hi, this is currently beyond the scope of the SDK.
You’d do well to look at the source code for Feditext (formerly Metatext), and perhaps IceCubes as well as these are open source iOS apps for Mastodon that have notifications working.
My understanding of their process is that there is a web service you need to run yourself. The app then connects to that with its authorisation token etc so that the webservice receives web pushes from your instance, and then it forwards them to Apple’s service so your user can receive the push on device.
this isn’t something I’ve implemented myself though!

OK, thanks for the clarification!

@fakerybakery Like @davidgarywood said, the actual server push is something you will need to implement separately as it requires a server component.
Once you have the server bits setup, TootSDK has several methods to facilitate creating and managing the subscription (note that this part of the flow depends on your server setup).
You can check for example createPushSubscription which takes an instance of PushSubscriptionParams the encryption key needed by your back-end and information about which notifications the app should receive.
So to subscribe, you will end up doing something like this:
let subscr = PushSubscriptionParams.Subscription.init(
        endpoint: pushURI.absoluteString,
        keys: PushSubscriptionParams.Keys.init(p256dh: pushSubscription.p256dh, auth: pushSubscription.authData))
let params = PushSubscriptionParams(subscription: subscr, data: .init(alerts: Alerts(follow: true, favourite: true, repost: true, mention: true)))

Hi, thanks for the clarification. Do you have any recommendations on what language would be best to implement the backend in? I looked a while back and found it was relatively hard to do in PHP.


New chart request

If possible then please add a chart with new confirmed cases in the past week vs. the total confirmed cases to date. As it is shown here https://aatishb.com/covidtrends/
It is a good way to represent if the spreading of the disease is slowing down or not.

Thanks, for the suggestion. Will look into adding it.

As test per capita differ a lot in some counties (Saaremaa has 4 times more tests per capita than Harjumaa, many counies havaedon 2 times less tests per capita), maybe county graphs should have also view where cases are are harmonised on the basis of tests made per capita somehow.

It would make more sense to have new confirmed cases (Y) vs active cases (X) instead of total cases on X. Then the "second wave" is easier to detect.


Translated fields in ModelAdmin.list_display_links

If the translated field is used as a link in admin, the generated link is not clickable.
from django.contrib import admin
from translated_fields import TranslatedFieldAdmin
from .models import Book

@admin.register(Book)
class BookAdmin(TranslatedFieldAdmin, admin.ModelAdmin):
    list_display = [*Book.title.fields, "isbn"]
    list_display_links = Book.title.fields

The problem is the code that checks the field exists in the defined list_display_links, here: https://github.com/django/django/blob/master/django/contrib/admin/templatetags/admin_list.py#L220-L225
field_name is not a string, but a function, so the function link_in_col returns False.
<function TranslatedFieldAdmin.translated_field_column.<locals>.value at 0x7f723c42a430>

A similar problem occurs with list_editable fields.

It would be possible to fix list_display_links using the following patch:
diff --git a/translated_fields/admin.py b/translated_fields/admin.py
index 2359429..5d1ef4f 100644
--- a/translated_fields/admin.py
+++ b/translated_fields/admin.py
@@ -1,3 +1,5 @@
+from functools import lru_cache
+
 from django.contrib.admin.options import BaseModelAdmin
 from django.contrib.admin.utils import display_for_field
 from django.core.exceptions import FieldDoesNotExist
@@ -11,6 +13,7 @@ class TranslatedFieldAdmin(BaseModelAdmin):
     def translated_field_label(self, db_field, language_code):
         return "%s [%s]" % (capfirst(db_field.verbose_name), language_code)

+    @lru_cache()
     def translated_field_column(self, field_name):
         try:
             db_field = self.model._meta.get_field(field_name)
@@ -40,3 +43,7 @@ class TranslatedFieldAdmin(BaseModelAdmin):

     def get_list_display(self, request):
         return [self.translated_field_column(f) for f in self.list_display]
+
+    def get_list_display_links(self, request, list_display):
+        return [self.translated_field_column(f) for f in super().get_list_display_links(request, list_display)]

However, it seems a bit hacky.
Fixing list_editable would require returning field_name as-is if field.name in self.list_editable, however this only works as long as no get_list_editable method is introduced in Django.


PYROSCOPE_PROFILER_EVENT always itimer

Wooow this is an amazing project and java profiling is a must. I think there isn't best approach than async-profiler to do it. Congratulations!!!
I have tested the java agent and looks very promising. The only mistake I have found is that seems always use itimer profiler. No wall or alloc profiler. I have tested with Jenkins java app and the flame graph is exactly the same when i set PYROSCOPE_PROFILER_EVENT to wall, itimer or alloc. Maybe I am doing something wrong.
Would be incredible support for wall profiling at least. It's a great profiling method to detect latency issues.
Last but not least, as a suggestion, would be great the Flamegraph Title change accoring profiler (wall, itimer, alloc, etc)

I see a very promising future for Pyroscope.
Keep going!!
Best regards.

Hi @netamego
Thanks for reporting this. It seems really we always use ITIMER:
https://github.com/pyroscope-io/pyroscope-java/blob/main/agent/src/a9c39e345f3129f96b6b40631b17d2cf15bef9a2/java/io/pyroscope/javaagent/PyroscopeAgent.java#L43
Needs fixing

Hey @netamego should be working now! Let us know if you have any issues or other feedback


cmd/compile: cannot handle alias type declaration

What version of Go are you using (go version)?
go version go1.14.2 windows/amd64.
Does this issue reproduce with the latest release?
haven check yet.
What operating system and processor architecture are you using (go env)?
go env Output
set GO111MODULE=on
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\Administrator\AppData\Local\go-build
set GOENV=C:\Users\Administrator\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=D:\gosrc\src
set GOPRIVATE=
set GOPROXY=http://goproxy.cn
set GOROOT=D:\golang
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=D:\golang\pkg\tool\windows_amd64
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=D:\src\mrp\go.mod
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\Users\ADMINI~1\AppData\Local\Temp\go-build890466410=/tmp/go-build -gno-record-gcc-switches

What did you do?

with go mod enabled
with those file in same package

// some_file.go
package mrp
type Runtime interface {
	RevertThen(i int, fragment Fragment)
}
type (
	Fragment             = func(Runtime) bool
)
//Rutime.go
package mrp

type runtime struct {
	stack     []interface{}
	revert    int
	revertPtr int
	parameter map[string]interface{}
}

func (r runtime) RevertThen(i int, fragment Fragment) {
	panic("implement me")
}

What did you see instead?
.\some_file.go:5:2: internal compiler error: cannot handle alias type declaration (issue #25838): Fragment
this will and only will happend under :

implment file is name within upper-case;
interface is defined in separate file, and this file contains no upper-case ;

Thanks for providing more details. I think this is #25838 actually. Here's a standalone file that produces the same failure: https://play.golang.org/p/0Cxmszie2hW
Case is relevant because it probably affects how cmd/go sorts file names, which affects the order that cmd/compile processes declarations. I think if you put the interface after the implementation in your file, it'll still reproduce too.
Closing as a duplicate.


update user data doesn't work properly

Due to the move from python2 to 3; we have a bug where changing user info via the GUI, it will not parse correctly. To reproduce simply go to the user's page, change values and the new values will be prepended by b''.

Turns out that there is another issue/error due to the same problem so creating a list here and renaming the issue name:

[ ] Error while updating values in user data: To reproduce simply go to the user's page, change values and the new values will be prepended by b''.
[ ] When a new user is created and validated the email is added to the redis qiita-usernames database; however, it's currently failing due to byte/text conversion. To search for this error in the logs, you can: "select count(msg) from qiita.logging where time >= '2019-05-01'::date and msg like '%/auth/verify/%email%';"


Added support to the latest apigateway auth setting

Hi @kaieongataws - Is this a new API Gateway feature?  If so, how can I find out more about it?

Integration tests failed because of an unnecessary file


Revert "zng_tr_tally_lit: disable -Wtype-limits"

This makes MSVC unhappy:
https://github.com/zlib-ng/zlib-ng/pull/726#issuecomment-681128124
D:\a\zlib-ng\zlib-ng\deflate_p.h(36,9): warning C4068: unknown pragma 'GCC' [D:\a\zlib-ng\zlib-ng\zlib.vcxproj]
This reverts commit 24c442c606d9121da64b25693c9bb1a898b3553f.

@iii-i Thanks for the quick turnaround on that 😄


AWS::AutoScaling::LaunchConfiguration.MetadataOptions

This fixes #1839

What timeframe are we looking at for this PR to be merged and released?  We've been watching this PR for a few weeks and are just curious for our planning.

What timeframe are we looking at for this PR to be merged and released?  We've been watching this PR for a few weeks and are just curious for our planning.

Thanks!

Thanks!


Add support for replicateSubscriptionState to Consumers

replicateSubscriptionState

I created pull request #49 for this issue. Please help to review it.


Remove stale Dev Page

Summary
This PR removes the dev page that existed at onramp.dev/en/dev. This page is not needed since the information it provided is no longer relevant. Reference to the page in the README.md file is removed in #401.

STALE? Girl, that page has feelings you know 😂


Empty lines are removed when an accented character is encountered

The formatting works great until the point where there is an accented character in the file. All the remaining lines are then reformatted with all the empty lines stripped out.
Steps to reproduce
Create test.js:
let a = 1;

// The blank line above is preserved
let b = 2;

// Now, accented characters
let c = [
"Des caractères accentués qui cassent tout!" ] ;

// The blank line above is not preserved
let d = 0;

And a gulpfile with this very basic task:
gulp.task('test', () => {
   var a = gulp.src("test.js")
      .pipe(prettier())
      .pipe(gulp.dest('test'));
});

The end result is as follows:
let a = 1;

// The blank line above is preserved
let b = 2;

// Now, accented characters
let c = [
  "Des caractères accentués qui cassent tout!"
]; // The blank line above is not preserved
let d = 0;

Expected result
The formatted file should be:
let a = 1;

// The blank line above is preserved
let b = 2;

// Now, accented characters
let c = ["Des caractères accentués qui cassent tout!"];

// The blank line above is not preserved
let d = 0;

This is not an issue with prettier, since calling it directly works fine (node node_modules/prettier/bin-prettier.js test.js).
Using gulp-prettier-plugin seems to works fine as well, so I'm going to use that one for now, but I thought I should report this.

Closed by updating prettier in 7d790fd1a41f24c084c4e43baa2460d2fe2295e3. A new version of gulp-prettier including this change will be published soon.

Now published in v2.0.0.


Show teams with new notification requests first in team list

I'm currently doing the teams refactor while Danny is out for surgery, so I don't think you need to wait for his review. LGTM!


Coroutine Support

Because someone's going to ask us for this eventually.

Implemented in https://github.com/ThePhD/sol2/commit/251e350539c914be8a1cee14535699fa23a82393 .

In your commit messages, write Closes #19 and it'll automatically close it when you merge the commit.


Elementum shows results only from one tracker even though I've selected many

It's my first post of this page so I'm not sure if I'm doing it right. Elementum sometimes it needs you to keep pressing the same result to show you the available torrents and when it does after many "no link", for material in English, it's always the RARBG tracker.
I'm not sure how to paste my log but here it is:
kodi.log

in your log i can see messages like
1337x returned  0 results in 1.0 seconds
although query for 1337x work for me. copy queries from your log file (search in file for 1337x search URL:, for example) and open in browser on same device - maybe your ISP block them or you hit cloudflare defense or similar - is so - try VPN or similar tech. or maybe you use VPN that blocked by torrent sites (or simply rate limit defense on trackers is reached for IP address you use).
about rarbg - it definitely has rate limiting, so if you search too often (with small delay) - it will give 0 results.

btw, you can use http://elementum.surge.sh/contribute/log/ for convenience - as stated in template when you create issue. also - it is better to use template (so you will provide all necessary and not "free-style" text.

You're right. Some of those trackers are block by my ISP. Thanks.
There's another issue which I cannot figure out. In some 4k video streams the image is alternate by the player. They look like negatives. Take a look and tell me which setting I'm missing.

Looks like you're playing a Dolby Vision ("DV") video stream on a non-Dolby Vision tv.

also this is not elementum related question :-D
elementum does not play anything - Kodi does.
so either your kodi/TV configured incorrectly or you play something unsupported (like HDR - looks like this "Dolby Vision" is also HDR).

Closing the issue as it is seems to be related to provider search.


Uncaught TypeError: Cannot read property '__mixinSet' of undefined

Hello, in advance, great work with the library!
I encountered some issues importing your hybrid  library in version 3.0.0-pre.12.
Client side, the error is like the following:

In order to avoid the error I've just replaced your original import syntax:
import { Element as PolymerElement } from '@polymer/polymer/polymer-element';
in:
import { PolymerElement } from '@polymer/polymer/polymer-element
It seems to work properly now, but it might be some side effects?
Thanks

hi @irsooti , the error you are reporting is given by the latest update in Polymer 3 pre-release (details here https://www.polymer-project.org/blog/2018-03-23-polymer-3-latest-preview.html ) which have some breaking changes like the import definition. So basically, this library just hasn't been tested with it yet :sunglasses: .

closed by #203


Add an accessor pattern

I find often times I'm just folding a map, and most of the code ends up just being getting a key out of the map and then passing it to some validation/execution function
Current behavior
defmodule MyOpus do
  use Opus.Pipeline

  check :valid_email?

  def valid_email?(%{email: email}) when is_binary(email) do
    MyValidator.valid_email?(email)
  end
end

Useful/Dry behavior
defmodule MyOpus do
  use Opus.Pipeline
  import MyValidator, only: [valid_email?: 1]

  # some accessor/getter/lens syntax
  check :valid_email?, in: :email
  # or
  check :valid_email?, get_in: [:email]
  # or
  check {:email, :valid_email?}
end

Hi @sitch, thank you for using Opus.
Have you tried?
defmodule MyOpus do
  use Opus.Pipeline

  import MyValidator, only: [valid_email?: 1]

  check :valid_email?, with: &valid_email?(&1.email)
end

Yes of course, but my general use case is:

Pass initial context as a map
Run a step
if it returns {:ok, map} then merge it into the current context

So what ends up happening is:

Map.merge ceremony
:ok tuple ceremony
extracting key-values to pass to a function like that valid_email?

Yes of course, but my general use case is:

Pass initial context as a map
Run a step
if it returns {:ok, map} then merge it into the current context

So what ends up happening is:

Map.merge ceremony
:ok tuple ceremony
extracting key-values to pass to a function like that valid_email?


Add more webpack plugins

Add:

NoErrorsPlugin: Keeps webpack from emitting its build products if there are any errors.  That way, the hot-reloading in the browser won’t get broken and require a page refresh.

OccurenceOrderPlugin: Reduces overall file size, results in more predictable ordering of the build product, and is recommended in the webpack documentation.

@lexun ?

Looks good to me.


After install: Error Class 'App\Providers\CollapsibleResourceManager' not found

Just after installation i have this error, added to nova tool.
I also tried add any of those, cannot find good way
use DigitalCreative\CollapsibleResourceManager;
use DigitalCreative\CollapsibleResourceManager\TopLevelResource;
use DigitalCreative\CollapsibleResourceManager\CollapsibleResourceManager;
use DigitalCreative\CollapsibleResourceManager\CollapsibleResourceManagerServiceProvider;
use DigitalCreative\CollapsibleResourceManager\TopLevelResource;
use App\Providers\CollapsibleResourceManager;

use DigitalCreative\CollapsibleResourceManager\CollapsibleResourceManager should be the right one

After adding
use DigitalCreative\CollapsibleResourceManager\CollapsibleResourceManager
There are another error:
Error Class 'App\Providers\TopLevelResource' not found
Adding:
use DigitalCreative\CollapsibleResourceManager\TopLevelResource; or
use DigitalCreative\CollapsibleResourceManager\CollapsibleResourceManager\TopLevelResource;
not solve problem.
Nova: 3.9.1 , Laravel 7.28.3

The class name should be:
use DigitalCreative\CollapsibleResourceManager\CollapsibleResourceManager;
use DigitalCreative\CollapsibleResourceManager\Resources\TopLevelResource;

Your IDE can automatically import these files for you (VS-code, PHPSTORM, NetBeans etc..) no need to manually import classes in 2020....

Sorry i just use basic Notepad++ it doesn't have auto class importer
I think You can just add thoose 2 lines to readme for dinosaurs like me.

I see, another way to figure out the class names is by finding the file directly and reading the namespace on the top of it:
https://github.com/dcasia/collapsible-resource-manager/blob/master/src/Resources/TopLevelResource.php#L3


docs(): fix typo "everthing" to "everything"

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[ ] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

New Dependency Instructions (If applicable)

We're currently processing your upload.  This comment will be updated when the results are available.

the commit message should follow the syntax
please update the commit message and PR title to:
docs: fix typo in README

can you squash the commit to one, the Semantic PR check will always be failed?
all the commit message should follow the Semantic PR convention