this pointer is destroyed but other and out are still accessible in backward

This most likely persists for add and sub as well.
Tensor* Tensor::mul(Tensor* other) {
  if (this->shape() != other->shape()) {
    throw std::logic_error("Shapes do not match");
  }
  size_t length = shape_to_length(this->shape());
  std::vector<double> res_data(length, 0.0);
  for (int i = 0; i < length; i++) {
    res_data[i] = this->data()[i] * other->data()[i];
  }
  Tensor* out = new Tensor(res_data, this->shape());
  // set has_grad of output
  if (this->has_grad || other->has_grad) { out->has_grad = true; }
  else { out->has_grad = false; }

  out->prev = std::vector<Tensor*> {};
  if (this->has_grad) { out->prev.push_back(this); }
  if (other->has_grad) { out->prev.push_back(other); }

  out->backward = [this, other, out] {
    std::vector<size_t> newshape = concat_indices(out->shape(), this->shape());
    std::cout << "=====";
    for (auto s : out->shape()) { std::cout << s << " "; }
    std::cout << "\n";
    std::cout << "=====";
    std:: cout << std::string(*this);
    std::cout << "\n";
    std::cout << "=====";
    for (auto s : other->shape()) { std::cout << s << " "; }
    std::cout << "\n";
    std::cout << "=====";

    if (this->has_grad) {
      this->grad = new GradTensor(newshape, this->shape().size());
      std::vector<size_t> pairshape = duplicate_indices(this->shape_);

      for (std::vector<size_t> l_idx : generate_all_indices(this->shape_)) {
        for (std::vector<size_t> r_idx : generate_all_indices(other->shape_)) {
          std::vector<size_t> idx = concat_indices(l_idx, r_idx);
          if (l_idx == r_idx) {
            (this->grad)->at(idx) = other->at(r_idx);
          }
          else {
            (this->grad)->at(idx) = 0.0;
          }
        }
      }
    }
    if (other->has_grad) {
      other->grad = new GradTensor(newshape, other->shape().size());
      std::vector<size_t> pairshape = duplicate_indices(this->shape_);

      for (std::vector<size_t> l_idx : generate_all_indices(this->shape_)) {
        for (std::vector<size_t> r_idx : generate_all_indices(other->shape_)) {
          std::vector<size_t> idx = concat_indices(l_idx, r_idx);
          if (l_idx == r_idx) {
            (other->grad)->at(idx) = this->at(l_idx);
          }
          else {
            (other->grad)->at(idx) = 0.0;
          }
        }
      }
    }
  };
  return out;
}

Segfault occurs from this line std:: cout << std::string(*this);

From looking at lldb it turns out that the this capture variable references the lambda function class. This behavior is due to how C++ lambda captures work. When you define the lambda function with [this, other, out], you're actually creating an unnamed class (closure type) that captures these variables. The this pointer inside the lambda is not the same as the original Tensor* - it's a member of the closure type. Therefore you must explicitly name the pointer to the original Tensor as a separate variable, e.g. this_ptr.


python3Packages.fastparquet: 0.7.1 -> 0.8.1

Description of changes
With the latest version of nixpkgs, I've seeing fastparquet 0.7.1 failing. 0.7.0 is fine, and 0.8.1 as well.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python39Packages.fastparquet

Landed in 35a6f46d6d85a0138808422afda2f0a84aaeb1d9


Just a Question. Do you have this for Androids

When filing an issue, please include the following information if
possible as well as a description of the problem.  Make sure you are
using the latest version of rclone.

What is your rclone version (eg output from rclone -V)
Which OS you are using and how many bits (eg Windows 7, 64 bit)
Which cloud storage system are you using? (eg Google Drive)
The command you were trying to run (eg rclone copy /tmp remote:tmp)
A log from the command with the -v flag (eg output from rclone -v copy /tmp remote:tmp)

Not sure. Found this neat article on how to allow usage of Go package in
Android and iOS:
https://godoc.org/golang.org/x/mobile/cmd/gobind
Regards,
Zeshan
On Thu, Sep 1, 2016, 9:41 PM MrsChubsCollins<EMAIL_ADDRESS>wrote:

When filing an issue, please include the following information if
possible as well as a description of the problem. Make sure you are
using the latest version of rclone http://rclone.org/downloads/.
What is your rclone version (eg output from rclone -V)
Which OS you are using and how many bits (eg Windows 7, 64 bit)
Which cloud storage system are you using? (eg Google Drive)
The command you were trying to run (eg rclone copy /tmp remote:tmp)
A log from the command with the -v flag (eg output from rclone -v copy
/tmp remote:tmp)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ncw/rclone/issues/674, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGXRYxGW10H9_f-26J-t4DTiUcp_NfIUks5ql37UgaJpZM4JzSra
.

I have used rclone on android using the following app.
https://play.google.com/store/apps/details?id=com.termux&hl=en

You can compile rclone on Android using Termux and use it from the command
line there.
I'm not currently building releases for Android though so you'll have to
build it yourself.

Just thought i would through in a quick update. I just used the compiled one for linux arm and it worked for a few hours but then started throwing errors. See this issue.
https://github.com/ncw/rclone/issues/600
Long story short its hit or miss right now.

The ARM linux one doesn't have the right android bits in in.  android != linux especially when it comes to system services like DNS.
You can try this one: http://pub.rclone.org/rclone-android-arm-v1.32.gz
Or build one from source with Termux.

Just thought i would throw out some more information... Like i said i had this working, then it stopped then it started again. I found the secret sauce(short term).
If i have my mobile hotspot activated it works. (DNS issue that your talking about? that hotspot fixes?!?)
Reboot phone throws the same error as always. "read: connection refused" activate hotspot, works fine.

Usually preferred to isolate the OS related and hardware issues first. By
getting a generic Android phone with standard Android OS without the
manufacturer "icing".
When reporting these types of issues one needs to mention the phone
hardware maker, model of phone, and Android OS version name and number.
On Tue, Sep 6, 2016, 8:53 AM justusiv<EMAIL_ADDRESS>wrote:

Just thought i would throw out some more information... Like i said i had
this working, then it stopped then it started again. I found the secret
sauce(short term).
If i have my mobile hotspot activated it works. (DNS issue that your
talking about? that hotspot fixes?!?)
Reboot phone throws the same error as always. "read: connection refused"
activate hotspot, works fine.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ncw/rclone/issues/674#issuecomment-244940466, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGXRY1Qu2M71t5Mrp0cqHlvkx7Qx3lo_ks5qnWI0gaJpZM4JzSra
.

This project already on this issue:
https://github.com/CloudRail
Unfortunately rClone doesn't seem to be intended for use with mobile
devices.
Regards,
Zeshan
On Tue, Sep 6, 2016, 8:53 AM justusiv<EMAIL_ADDRESS>wrote:

Just thought i would throw out some more information... Like i said i had
this working, then it stopped then it started again. I found the secret
sauce(short term).
If i have my mobile hotspot activated it works. (DNS issue that your
talking about? that hotspot fixes?!?)
Reboot phone throws the same error as always. "read: connection refused"
activate hotspot, works fine.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ncw/rclone/issues/674#issuecomment-244940466, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGXRY1Qu2M71t5Mrp0cqHlvkx7Qx3lo_ks5qnWI0gaJpZM4JzSra
.

I am just reporting my findings. I have it working so....
My phone is a Nexus 6P running MTC20F (latest Marshmellow)
ncw analysis of dns being broken seems legit. For some reason toggling on Hotspot resolves the issue for me.

I built the current code with termux and uploaded the result - try this.  It should work better than the linux-arm version.
http://pub.rclone.org/rclone-android-arm-v1.33.gz
I should probably write some instructions on how to do this - it isn't too hard.

Just tried the rclone-android-arm-v1.33.gz i had no difference in results.
Works fine with hotspot on. gives the "read: connection refused" when off.
Not sure if it matters but the way i am doing it is i have to type su into termux then i copy rclone to /data/data/com.termux/file/usr/bin
then chmod 777 rclone
then i can use it as long as i run su on ther terminal first.
Not sure if anything i am doing would create the error i am seeing.

Just to keep adding info.
I setup rclone without su and had the same issue.

For anyone with this running, were they able to use the fuse mount via rclone?  My phone isn't rooted so I couldn't test it.  The actual syncing and such seems to work ok on android so far.

I have not. Will list error in a few.

Just thought i would add. I added a second storage location on google drive. Same issue. blah blah connection refused.
Anything i saw related to this was only on Amazon Cloud Drive so i just wanted to make a note that i am seeing it on other storage.

Just another thing i noticed. I setup acd_cli for grins just to test on my android device using termux. It works just fine without my hotspot being on.

If anyone have this issue with DNS not being accessed for rclone's arm build on Android, create the /etc/resolv.conf file and populate it with a DNS server on your phone.
Thanks for pointing that rclone was working with hotspot on.

We now have how to build rclone for termux in the wiki so I'm going to close this question.


Resize Root Disk | compute offering with Disk offering strictness > true

ISSUE TYPE

Bug Report

COMPONENT NAME

UI
Compute Offerings

CLOUDSTACK VERSION

4.18.1

CONFIGURATION

ACS 4.18.1 Hypervisor XCP-ng
OS / ENVIRONMENT

ACS 4.18.1 on Ubuntu Hypervisor XCP-ng
SUMMARY

Resizing Root Disk after VM deployment is possible after VM deployed with compute offering with Disk offering strictness as true
STEPS TO REPRODUCE

Create Compute Offering
set root disk size
enable Disk offering strictness

EXPECTED RESULTS

Once the VM is deployed We should not be able to change ROOT volume size

ACTUAL RESULTS

Select Root Disk,
Select resize volume and update to higher value
Root Disk is getting upgraded to new size

@midhunpjos  this property on compute offering is expected to restrict the resize volume? it is expected only to restrict changing the disk offering right? I haven't tried it myself though.

diskofferingstrictness
True/False to indicate the strictness of the disk offering association with the compute offering. When set to true, override of disk offering is not allowed when VM is deployed and change disk offering is not allowed for the ROOT disk after the VM is deployed

@midhunpjos this property on compute offering is expected to restrict the resize volume? it is expected only to restrict changing the disk offering right? I haven't tried it myself though.

diskofferingstrictness
True/False to indicate the strictness of the disk offering association with the compute offering. When set to true, override of disk offering is not allowed when VM is deployed and change disk offering is not allowed for the ROOT disk after the VM is deployed

@rajujith But resizing a disk almost approximates to change offering.
Can this be considered as an enhancement?

@midhunpjos  You are looking for way to restrict volume resizing?

@midhunpjos You are looking for way to restrict volume resizing?

Yes. Exactly. Its better to restrict the resizing too. right ?

@midhunpjos @rajujith Volume resize restriction check is there when disk size strictness is enabled, but not for disk offering strictness. I think, you have to enable the disk size strictness for the disk offering associated to the compute/service offering.
https://github.com/apache/cloudstack/blob/ab808995ffe76a2a308d91bbc761693538cfa0dd/server/src/main/java/com/cloud/storage/VolumeApiServiceImpl.java#L1116-L1119

thanks @sureshanaparti for the comments. This will solve my concern..

closing according to submitter comment.


Update the Concepts section with webpack v4 configuration

In the Concepts section we still have old config snippets and we also are not reflecting the fact that webopack is now zero config out of the box.
Some content might also need to be change so it's compatible with the new webpack default behavior.

Also the home page of the web site still reflects that you need a config file. Should be changed too.

Yeap. From above:

we also are not reflecting the fact that webopack is now zero config out of the box.

@jeremenichelli u on it or i can pick up both concepts configuration and homepage?

Go for it @EugeneHlushko! Assign the issues to yourself so we don't overlap (this one and the homepage and configuration).
Here's a list of things to look at on the config files or warnings:

update node.js to 8
install webpack-cli
upgrade loaders and pluginsConfig
add mode: "development"/"production" to config files
remove DefinePlugin process.env.NODE_ENV
remove NoEmitOnErrorsPlugin/NoErrorsPlugin
remove NewWatchingPlugin
remove ModuleConcatenationPlugin
remove NamedModulesPlugin
remove CommonsChunkPlugin
may add optimization.runtimeChunk
may add optimization.splitChunks.chunks: "all"
remove uglifyjs-webpack-plugin
may add optimization.minimizer for custom objects
module.loaders -> module.rulesCode
when import(cjs) access the default property

They are not mandatory, just a check list for you to look at when updating the config snippets. Have fun and thanks for collaborating!

@jeremenichelli there are not enough privileges i suppose to assign it on myself :(

Whenever you want to grab some issue just be verbose in the comment thread and I will proceed to assign it to you 😊

@montogeek @jeremenichelli this seems done now, can close i guess

According to the list, I guess NamedModulesPlugin is no longer required.
But it is still mentioned in the documentation:

https://webpack.js.org/guides/hot-module-replacement/
https://webpack.js.org/plugins/named-modules-plugin/


Use context to store logger

introduce ctx as first arg, same parent ctx everywhere
config now in separate package
NewManagerContext and NewReconcilerContext
zap logger in context, removed from func args and structs
use context.WithTimeout to get warnings for missing cancel() calls
reorder imports (go, github, k8s, cfo)
removed context type alias
context no longer part of any struct, except Reconciler

#164933036
Another PR to add events via recorder will have to follow later.

@manno a quick question on how to use the context with value. I was of the impression that is not the best practice to store pointers as the values, not sure what is the exact use case here, but I think it´s valid to think about the type of data to store and what to not.

@qu1queee hm, I was following https://github.com/grpc-ecosystem/go-grpc-middleware/blob/master/logging/zap/ctxzap/context.go
I didn't' see any docs discouraging storing the pointer?

Yes, I think the ctx value should be of the type request scoped data, which means all data that originates in a request and its deleted when the request is over.(https://golang.org/pkg/context/, https://dave.cheney.net/2017/08/20/context-isnt-for-cancellation) , therefore the logger could be not ideal mainly because you lose:

type safety
refactoring could be hard later

But I also read this one(https://medium.com/@cep21/how-to-correctly-use-context-context-in-go-1-7-8f2c0fafdf39) , I like the explanation about informing/controlling. As long as the value in the ctx its only used to inform, meaning it will not affect the request somehow, I will say it is fine to keep the logger.


Portable apps and in-app updates tweaks

Is your feature request related to a problem? Please describe.
For the next release I will create Ferdi in PortableApps format (Already done for Franz and I will do it for Ferdi too) and It will be very great If a switch could be added so the app will not rewriting by himself and maybe asking for admin write.
Describe the solution you'd like
Like adding a button/swith for desactivate update under accepting beta version so it is in the same area.

I just published 098c46e which should implement this feature. Unfortunately, this is something I can't really test so we'll have to test it over the next updates.

@Makazzz from what I understand, the portable apps should not update at all, rather than have an option not-to-update, is that right?

@kytwb normally when a portable apps is created we use a pre-created config file from the app (like settings/config.json) and change some value update: false so like that in the first run the user is not having an application that break himself by trying to update and/or asking admin right. OR we can block the update by renaming/changing some file so the app dosn't know where to find information about doing update. There is a few app that can be auto-updating without setup or admin right: JDownloader, Discord, etc... I will have to test if ferdi can be updated without a prompt for update or if Ferdi is pre-downloading the packaging in the Appdata folder (like Bitwarden do). And it is usual to desactivate the update option when bulding portableapps (with config file or a trick).

I see electron-builder support the portable target for Windows; we should give it a try and see how the auto-update behave then. Maybe something is already handled in the portable build regarding in-app updates?

@Makazzz you can find the portable version built by electron-builder in here (the executable without Setup in the filename).

@kytwb thanks for this I will use this .exe for extract the app-32.7z and portabilize without using temps files. And it is possible to create an environment variable like Bitwarden did (an electron app too writen in TypeScript) for helping the portableapps format to work?  Because the laucher of portableapps can use environement variable for the user and use relative path for beign sure the data go to a folder choosen Like FERDI_APPDATA_DIR.

Possible, on it 👍

Thank you

@vantezzen I guess if needed we can use PORTABLE_EXECUTABLE_DIR to make the define if it's a portable app and disable updates by default?

Thanks a lot I will finish my repo of FerdiPortable and the build tomorrow. Thanks for your work.

@Makazzz Welcome!
@vantezzen In the doc they say we also have access to:

PORTABLE_EXECUTABLE_APP_FILENAME - sanitized app name to use in file paths.

Following the links on paths:

I want to toggle shortcut and startup, which will need dir and app name

Do we need to use PORTABLE_EXECUTABLE_APP_FILENAME somewhere for that purpose/to make things compatible with portable apps?

@kytwb I tested my Ferdi portable apps and with the FERDI_USERDATA_DIR variable and the config file for settings.json and proxy.json inside the config folder still stay in the appdata dir but all the rest of the data go to where I say so (so in a portable folder).

@Makazzz understood; I think I need to set app.setPath('appData', ...) instead of app.setPath('userData', ...). Will come up with a fix soon.

@Makazzz last commit should do the trick; please use FERDI_APPDATA_DIR instead of FERDI_USERDATA_DIR.

Thanks, I will wait to AppVeyor to increase your maximum storage and test the next builded Ferdi.

Alternatively, you could build the project locally on your Windows. You should get the .exe in the out folder as explained in the README. If you don't manage to, I will follow-up once AppVeyor has.

I will wait because at the moment I am not home but thanks for the info.

Finally I will wait because I have to many error (like cb() never called) with Node.js 10.16.3 and nothing work for building

@kytwb I have a problem... the FERDI_APPDATA_DIR don't do anything now and I don't know why.

@kytwb I have a problem... the FERDI_APPDATA_DIR don't do anything now and I don't know why.

@vantezzen Can you check about this please?

I think we need to roll back to using FERDI_USERDATA_DIR instead of FERDI_APPDATA_DIR . It looks like appData only gets used to generate the userData folder during dev builds.
I think we need to fix settings.json and proxy.json in another way. These are the only files that get updated through calls to the local API, so the problem needs to be somewhere in there.
I'll look into this the next days or @kytwb may find the bug.

Ok thanks for the answer

It looks like appData only gets used to generate the userData folder during dev builds.

In our code yes, but I think it's also used by Electron's internals afterward.
What got me confused is:

the config file for settings.json and proxy.json inside the config folder still stay in the appdata dir but all the rest of the data go to where I say so

But the SETTINGS_PATH is set to:
export const SETTINGS_PATH = path.join(app.getPath('userData'), 'config');

So not sure how the settings.json inside the config folder was in the appdata dir instead of the set userdata dir.
Giving another try to FERDI_APPDATA_DIR with the next commit I'll push as I noticed I userData wasn't explicitly set on that iteration.

Still not working :/ . And now Ferdi is forcing me to have english has locale language.

What's happening now if you don't set FERDI_APPDATA_DIR? Does it fallback properly to PORTABLE_EXECUTABLE_DIR? If it's not working, I guess I'll revert the changes to FERDI_USERDATA_DIR and investigate why the two files you previously mentioned are not where they're expected to based on @vantezzen suggestion.

@kytwb If I remove the variable Ferdi just act like a normal install (use %AppData%)

@Makazzz roger that, I'll spend some more time checking over the weekend 🙏

I found this when doing research. https://github.com/electron/electron/issues/1404#issuecomment-194391247 I don't know if it help.

@Makazzz Did you ever managed to build the project locally on your Windows? I'd be happy to pair with you to make it work, I think it would really help if we have a contributor that can test things that are Windows-specific to avoid back and forth as we experienced on this issue. WDYT?
I will push the snippet as it:
if (process.platform == "win32") {
  app.setPath("appData", process.env.LOCALAPPDATA);
  app.setPath("userData", path.join(process.env.LOCALAPPDATA, app.getName()));
}

But I don't understand why that would work and not the previous commits.

@kytwb I did not managed to make a working build but I want to do it and it will be really helpfull if we pair to make it work. And I was thinking to make an section in the wiki for windows build with steps etc.

@Makazzz awesome, let's do this! You can use this link to pick a date/time at your convenience!

@kytwb Sure but I am not so confident about doing a call/video but more feeling into text/messaging. And my english speaking is not the best (sorry in advance).

No worries, we can do what you're more comfortable with. Received your calendly invite, talk to you in the morning, signing off here!
PS: I speak French as well 😉

I think the last bit to be taken care of here is to disable in-app updates for Portable apps? Or is that not relevant anymore?

Normally vantezzen did it with this commit 098c46e.


chore(deps): Update textwrap to 0.15.0

To selfishly remove the duplicate I get when using miette with the latest clap@v3 💜

thank you!

This has been released as part of<EMAIL_ADDRESS>


ots-xml project needs to be adapted to new Stripe classes

The ots-xml project has several errors as a result of the changes in the Stripe definition. Several of the generated classes, as well as the DefinitionsParser, DemandParser, and NetworkParser classes have references to non-existent classes in ots-road.

Solved.


CRC Image Registry docker login issue on Windows - <IP_ADDRESS>:80 connection refused

OS: Windows 10
Hyper-V
Laptop
When trying to docker login to CRC internal image registry default-route-openshift-image-registry.apps-crc.testing throws error.
Its only happening on windows. Works fine when running on Linux.
Version
Client Version: 4.7.5
Kubernetes Version: v1.20.0+bafe72f
Steps To Reproduce
crc daemon
crc start (Windows)
docker login -u kubeadmin -p $(oc whoami -t) default-route-openshift-image-registry.apps-crc.testing
Current Result
Get http://default-route-openshift-image-registry.apps-crc.testing/v2/: dial tcp <IP_ADDRESS>:80: connect: connection refused
Expected Result
Login Succeeded!
Additional Information
I have tried the following -
oc patch configs.imageregistry.operator.openshift.io/cluster --type merge -p '{"spec":{"defaultRoute":true}}'
oc policy add-role-to-user registry-editor kubeadmin
oc policy add-role-to-user registry-viewer kubeadmin
oc get clusteroperator image-registry
NAME VERSION AVAILABLE PROGRESSING DEGRADED SINCE
image-registry 4.7.5 True False False 15d

oc registry info --public
default-route-openshift-image-registry.apps-crc.testing

oc get pods -n openshift-image-registry NAME READY STATUS RESTARTS AGE
cluster-image-registry-operator-66b48f9bd-ln99j 1/1 Running 0 33d
image-registry-77979c4dc7-nszrj 1/1 Running 0 15d
node-ca-q6gc8 1/1 Running 0 34d

oc edit configs.imageregistry.operator.openshift.io
apiVersion: imageregistry.operator.openshift.io/v1
kind: Config
metadata:
creationTimestamp: "2021-04-08T06:11:05Z"
finalizers:

imageregistry.operator.openshift.io/finalizer
generation: 5
name: cluster
resourceVersion: "28869"
selfLink: /apis/imageregistry.operator.openshift.io/v1/configs/cluster
uid: 4644048c-fdd2-4c9d-a86a-d398326ee3e3
spec:
defaultRoute: true
httpSecret: a2361d60ce4c3abf3d39b2810532c3ad6252772f0c1894f68013bbb82904475f8ecd5425ab8df66fad0a6aee616a4c1fc95bb463440b07fced01908543e804d6
logLevel: Normal
managementState: Managed
observedConfig: null
operatorLogLevel: Normal
proxy: {}
replicas: 1
requests:
read:
maxWaitInQueue: 0s
write:
maxWaitInQueue: 0s
rolloutStrategy: RollingUpdate
storage:
managementState: Managed
pvc:
claim: crc-image-registry-storage
unsupportedConfigOverrides: null
status:
conditions:
lastTransitionTime: "2021-04-09T06:37:34Z"
reason: PVC Exists
status: "True"
type: StorageExists
lastTransitionTime: "2021-04-26T15:55:33Z"
message: The registry is ready
reason: Ready
status: "True"
type: Available
lastTransitionTime: "2021-04-26T15:55:56Z"
message: The registry is ready
reason: Ready
status: "False"
type: Progressing
lastTransitionTime: "2021-04-26T15:55:33Z"
status: "False"
type: Degraded
lastTransitionTime: "2021-04-08T06:11:06Z"
status: "False"
type: Removed
lastTransitionTime: "2021-04-08T06:11:06Z"
reason: AsExpected
status: "False"
type: ImageRegistryCertificatesControllerDegraded
lastTransitionTime: "2021-04-08T06:11:07Z"
reason: AsExpected
status: "False"
type: ImageConfigControllerDegraded
lastTransitionTime: "2021-04-08T06:11:07Z"
reason: AsExpected
status: "False"
type: NodeCADaemonControllerDegraded
generations:
group: apps
hash: ""
lastGeneration: 5
name: image-registry
namespace: openshift-image-registry
resource: deployments
group: apps
hash: ""
lastGeneration: 0
name: node-ca
namespace: openshift-image-registry
resource: daemonsets
observedGeneration: 5
readyReplicas: 0
storage:
managementState: Managed
pvc:
claim: crc-image-registry-storage
storageManaged: true

@ShrayRastogi Are you using the vsock as part of config? Can you provide output of crc config view ?

In crc 1.25.0, the registry on the port 80 is not exposed. It is fixed in 1.26.0. Can you give it a try?
Thanks.

In crc 1.25.0, the registry on the port 80 is not exposed. It is fixed in 1.26.0. Can you give it a try?
Thanks.

I upgraded the CRC to v4.7.8 and 1.26.0+31f06c09, still same error

@ShrayRastogi Are you using the vsock as part of config? Can you provide output of crc config view ?

crc config view

consent-telemetry                     : yes
network-mode                          : vsock

Here is the gist of CRC Start logs - https://gist.github.com/ShrayRastogi/77b08a9738d59f510530981d027fcf8b

In crc 1.25.0, the registry on the port 80 is not exposed. It is fixed in 1.26.0. Can you give it a try?
Thanks.

I upgraded the CRC to v4.7.8 and 1.26.0+31f06c09, still same error

Have you deleted the old crc instance and started fresh with new version. I just tested vsock on linux and it does work as expected.
$ dig +short default-route-openshift-image-registry.apps-crc.testing
<IP_ADDRESS>
$ podman login -u kubeadmin -p $(oc whoami -t) default-route-openshift-image-registry.apps-crc.testing --tls-verify=false
Login Succeeded!

In crc 1.25.0, the registry on the port 80 is not exposed. It is fixed in 1.26.0. Can you give it a try?
Thanks.

I upgraded the CRC to v4.7.8 and 1.26.0+31f06c09, still same error

Have you deleted the old crc instance and started fresh with new version. I just tested vsock on linux and it does work as expected.
$ dig +short default-route-openshift-image-registry.apps-crc.testing
<IP_ADDRESS>
$ podman login -u kubeadmin -p $(oc whoami -t) default-route-openshift-image-registry.apps-crc.testing --tls-verify=false
Login Succeeded!

@praveenkumar - Yes, I deleted the old instance and started fresh. It is working on linux for me as well, however, not working on Windows Enterprise. Please help! Let me know if you need any other logs or something.
I am also able to login to default image registry via VSCode Docker Extension. However, via command line - docker login or docker push, I get same error as above.

I am also able to login to default image registry via VSCode Docker Extension (using URL like https://default-route-openshift-image-registry.apps-crc.testing). However, via command line - docker login or docker push, I get same error as above. Via command-line docker by default is going to http://default-route-openshift-image-registry.apps-crc.testing instead of https.

This is because you are using Docker Desktop or running docker/podman in WSL I guess.
If it is the case, please look at this issue https://github.com/code-ready/crc/issues/2092#issuecomment-797713335.
Let me know if it helps or not.

I am also able to login to default image registry via VSCode Docker Extension (using URL like https://default-route-openshift-image-registry.apps-crc.testing). However, via command line - docker login or docker push, I get same error as above. Via command-line docker by default is going to http://default-route-openshift-image-registry.apps-crc.testing instead of https.

This is because you are using Docker Desktop or running docker/podman in WSL I guess.
If it is the case, please look at this issue #2092 (comment).
Let me know if it helps or not.

@guillaumerose @praveenkumar - Yes, I am using Docker Desktop. However, I am getting this error now -
docker login -u kubeadmin -p $(oc whoami -t) host.docker.internal
Error response from daemon: Get https://host.docker.internal/v2/: x509: certificate is valid for *.apps-crc.testing, not host.docker.internal

You need to configure certificates then. Follow this: https://docs.docker.com/docker-for-windows/#adding-tls-certificates

You need to configure certificates then. Follow this: https://docs.docker.com/docker-for-windows/#adding-tls-certificates

@praveenkumar @guillaumerose - So there is no any other alternative than to use host.docker.internal during docker login and docker push commands?

I can't find an alternative. But for sure, we can simplify this and crc could configure that for you.

I can't find an alternative. But for sure, we can simplify this and crc could configure that for you.

@praveenkumar @guillaumerose - That would be super helpful. So, by using host.docker.internal, do I need to select external image registry and creating image pull secret while creating an app or would the image be pushed to default internal openshift registry?

@praveenkumar @guillaumerose - I have tried the following based on your suggestion with docker -

docker login -u kubeadmin -p $(oc whoami -t) host.docker.internal
WARNING! Using --password via the CLI is insecure. Use --password-stdin.
Error response from daemon: Get https://host.docker.internal/v2/: x509: certificate is valid for *.apps-crc.testing, not host.docker.internal

If I create a registry container as suggested here - https://docs.docker.com/registry/deploying/#get-a-certificate
docker run -d --restart=always --name registry -v C:/Projects/certs:/certs -e REGISTRY_HTTP_ADDR=<IP_ADDRESS>:443 -e REGISTRY_HTTP_TLS_CERTIFICATE=/certs/crc.crt -e REGISTRY_HTTP_TLS_KEY=/certs/crc.key -p 4430:443 registry:2
I created the certificate for host.docker.internal, but I have to run the container on different host port other than 443 as it complains that something is already running on 443 (probably default openshift registry).
I am able to login and push the images like -
docker login -u kubeadmin -p $(oc whoami -t) host.docker.internal:4430
WARNING! Using --password via the CLI is insecure. Use --password-stdin.
Login Succeeded

docker tag myimage:v1 host.docker.internal:4430/myproject/myimage:v1
docker push host.docker.internal:4430/myproject/myimage:v1
But I don't see the images in Openshift internal image registry
Could you please help with this, if possible? How could I solve this docker login using host.docker.internal on 443 port?

@guillaumerose @praveenkumar - could you help with this?

@ShrayRastogi what is the output of oc get is after you push the image to openshift internal registry?

@ShrayRastogi what is the output of oc get is after you push the image to openshift internal registry?

@praveenkumar - I am not able to login to openshift internal registry itself using docker. I tried multiple things but to no avail.
For method 2 using docker container as registry, I don't think that would route internal to openshift so.... thats why I dont see images there.
If you could, then help me with docker login issue on Windows Enterprise... else you can close the bug...

docker login -u kubeadmin -p $(oc whoami -t) host.docker.internal
WARNING! Using --password via the CLI is insecure. Use --password-stdin.
Error response from daemon: Get https://host.docker.internal/v2/: x509: certificate is valid for *.apps-crc.testing, not host.docker.internal

the certificate does not work for this name; SNI will deny this... please only try to use something using .apps-crc.testing

@ShrayRastogi I found a solution for you:

Add default-route-openshift-image-registry.apps-crc.testing in the ìnsecure-registries` list in Docker Desktop configuration
Run docker run -it --rm --privileged --pid=host dockerpinata/nsenter-dockerd
Inside this shell, run echo "<IP_ADDRESS> default-route-openshift-image-registry.apps-crc.testing" >> /etc/hosts. <IP_ADDRESS> is the result of nslookup host.docker.internal.

You will be able to login with docker login -u kubeadmin -p $(oc whoami -t) default-route-openshift-image-registry.apps-crc.testing
Each time you restart Docker you will have to run step 2 and 3 again.
I also tried on macOS and it works fine by default. Directly docker login.

docker login -u kubeadmin -p $(oc whoami -t) host.docker.internal
WARNING! Using --password via the CLI is insecure. Use --password-stdin.
Error response from daemon: Get https://host.docker.internal/v2/: x509: certificate is valid for *.apps-crc.testing, not host.docker.internal

the certificate does not work for this name; SNI will deny this... please only try to use something using .apps-crc.testing. We might be able to add ANOTHER hostname to the cluster, but instead we might allow crc.testing.

@gbraad - Thank you for input, however, the default route for internal registry goes to http instead of https when running crc with vsock. That's why need to use host.docker.internal which goes to https but that causes an issue with certificate. I tried creating multiple certificates that has subject with alt names for both host.docker.internal and default route but nothing worked.

@ShrayRastogi I found a solution for you:

Add default-route-openshift-image-registry.apps-crc.testing in the ìnsecure-registries` list in Docker Desktop configuration
Run docker run -it --rm --privileged --pid=host dockerpinata/nsenter-dockerd
Inside this shell, run echo "<IP_ADDRESS> default-route-openshift-image-registry.apps-crc.testing" >> /etc/hosts. <IP_ADDRESS> is the result of nslookup host.docker.internal.

You will be able to login with docker login -u kubeadmin -p $(oc whoami -t) default-route-openshift-image-registry.apps-crc.testing
Each time you restart Docker you will have to run step 2 and 3 again.
I also tried on macOS and it works fine by default. Directly docker login.

@guillaumerose - Thanks for providing the way. I will try this out and let you know if it's works.

docker login -u kubeadmin -p $(oc whoami -t) default-route-openshift-image-registry.apps-crc.testing

@guillaumerose - IT WORKED!!!!!!!!!!!!!!!!!!!!!!!!!!!! THANK YOU SO MUCH.........................!!!!!!!!!!!!!!!!!!!!!!!!!!! Able to now docker login and push images to internal registry.
Could you please explain a bit more, if that's ok, as to what this docker container is doing and what made it work?

@ShrayRastogi
The nsenter binary allows you to enter the namespace of another process, which i how containers work. It runs as privileged and allows to write to the host's host-lookup file and adds an entry to 192.... This is why you need to re-run this, as these settings aren't permanent... however, we could make that in the SNC image generation; @guillaumerose, right?

@ShrayRastogi
The nsenter binary allows you to enter the namespace of another process, which i how containers work. It runs as privileged and allows to write to the ~host's~ Docker container-runtime image host-lookup file and adds an entry to 192.... This is why you need to re-run this, as these settings aren't permanent... however, we could make that in the SNC image generation; @guillaumerose, right?
Oh wait, that is on the docker's runtime image of course... So, this is something specific to Docker for Windows. BUT, if we add this entry to the host itself, would this get taken?

@gbraad @guillaumerose - Thanks for the explanation. If CRC would be able to do it by default, it would help a lot or at least a documentation of this process to help Windows Enterprise users.
I think we can close the bug now.
Again, thank you for all the help, really appreciate it.

On macOS, dockerd is configured to use the built-in http proxy of VPNKit. That means that all HTTP requests made by dockerd are in reality, made on the host: DNS resolution, connection etc.
On Windows, there is no http proxy configured for dockerd, so: the DNS resolution happens and VPNKit answers <IP_ADDRESS> for default-route-openshift-image-registry.apps-crc.testing.
Then, dockerd tries to connect to this. It is localhost, it can't.

On macOS, dockerd is configured to use the built-in http proxy of VPNKit. That means that all HTTP requests made by dockerd are in reality, made on the host: DNS resolution, connection etc.
On Windows, there is no http proxy configured for dockerd, so: the DNS resolution happens and VPNKit answers <IP_ADDRESS> for default-route-openshift-image-registry.apps-crc.testing.
Then, dockerd tries to connect to this. It is localhost, it can't.

On Windows, there is no http proxy configured for dockerd

Which means this issue is due to missing functionality from DfW; there is no feature parity. Perhaps an oversight (forgotten use-case as it is not often needed)

I'm having the same error with PODMAN on MacOS with latest version of CRC, login worked built image with podman but push to internal registry with CRC failed.

@rprakashg can you open a new issue, and provide the details listed in the issue template? Thanks!


Validate ownership of the dist bucket before uploading deployment packages

Update script statements like this:
aws s3 sync $global_dist_dir s3://$global_bucket/aws-media-insights-engine/$version/
aws s3 sync $regional_dist_dir s3://${regional_bucket}-${region}/aws-media-insights-engine/$version/
To include checks like this before running s3 sync or cp:
aws s3api head-bucket --bucket $global_bucket --expected-bucket-owner $bucket_account
aws s3api head-bucket --bucket $regional_bucket --expected-bucket-owner $bucket_account
The head-bucket command will return a non-zero result (API returns a 403) if the bucket ownership doesn’t match. If you have error handling set to short-circuit the script, the above statements would stop the script before uploading.
Directly uploading to regional buckets means checking each bucket before uploading.

This is included in the PR to merge from CAS (#139 )

Fixed here:
https://github.com/aws-samples/aws-media-insights-content-localization/blob/dc945e895c0f4d4a6159e383368c9ce9ce909dc8/deployment/build-s3-dist.sh#L351
This issue will automatically close when #139 is merged.


feat(support): generate library

This change is

Hmmm.
/workspace/cmake-out/google/cloud/support/support_quickstart
google::cloud::Status thrown: UNIMPLEMENTED: Permanent error in ListCases: Method not found.

Hmmm.
/workspace/cmake-out/google/cloud/support/support_quickstart
google::cloud::Status thrown: UNIMPLEMENTED: Permanent error in ListCases: Method not found.

Being added to the allow-list seems to have addressed this.


System.TypeInitializationException: The type initializer for 'Aetherment.Native' threw an exception.

Hi,
after a fresh install of both my OS and FFXIV, I get this error when enabling Aetherment.
13:04:11.126 | FTL | [Aetherment] System.TypeInitializationException: The type initializer for 'Aetherment.Native' threw an exception.
		 ---> System.ArgumentNullException: Value cannot be null. (Parameter 'handle')
		   at System.ArgumentNullException.Throw(String paramName)
		   at System.Runtime.InteropServices.NativeLibrary.GetExport(IntPtr handle, String name)
		   at Aetherment.Native..cctor()
		   --- End of inner exception stack trace ---
		   at Aetherment.Aetherment..ctor()

I'm running
Fedora Silverblue 41
Aetherment <IP_ADDRESS>
Release branch of Dalamud (2<IP_ADDRESS>0.0000)
I tried removing/reinstalling the plugin, deleting my .xlcore folder and reinstalling everything, same result.
Am I missing a dependency or something?

duplicate of #4, Aetherment currently doesn't work on linux out of the box. You can try the steps posted there or copying the file from another location, it worked for me during testing but didn't for others so your milage may vary.


Adding Scripts

I would like to know how can I use the services for my own DAG scripts. Thanks.

I tried forking the repo and added a new script. But it does not show up on the Airflow webserver.


Weird peak showing in HA

I am getting the odd peak over 200,000 w which distrots the graphs. What would be causing such high peaks. Usually happens during the night time when I am asleep but sometimes during the day

There could be several reasons why this could go wrong. Suspect that 1 pulse was measured as 2 separate pulses, possible solution could be by setting an internal_filter.

ok will have to check and see

Thanks, that sorted it for me


Update README to render 'Contribute Code' and 'More Information' sections as headings.

Description
Missing a whitespace in both 'Contribute Code' and 'More Information' sections. As a result, Markdown is not transformed to a heading.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Can one of the admins verify this patch?

@azuresdkci test this please


Chocolist.txt referenced in user path not this repo

This line references a file that looks like it should be in this repo but isn't
https://github.com/microsoft/AttackSurfaceAnalyzer/blob/0e3c7e1fa5ff50a328166b537547fab671bb1724/Tools/Docker/AsaDetonatorWindows/runall.ps1#L1
Is this additional work that's needed to complete #464

Good catch. The problem is actually that runall.ps1 shouldn't be in the repository - Choco.txt was just a list of package names.

Runall.ps1 has been removed.


Error in Next.js 13.4.4 using the app router

I've used the following config with Next.js 13.4.4 and the app router:
const stailwc = require('stailwc/install')

/** @type {import('next').NextConfig} */
const nextConfig = {
  swcMinify: true,
  compiler: {
    styledComponents: true
  },
  experimental: {
    swcPlugins: [stailwc()]
  }
}

module.exports = nextConfig

I get the following error from Next trying to load a page:
- error ./node_modules/next/dist/pages/_app.js
Error: Filesystem cache is not enabled, cannot read plugin from phsyical path

This particular error seems to be related to https://github.com/vercel/next.js/pull/50651 which seems to have changed in 13.4.*.
I tried rolling Next.js back to 13.0.1 since that's listed as the supported version, but I'm now seeing this:
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: LayoutError', thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: LayoutError', /home/arlyon/.cargo/registry/src/github.com-1ecc6299db9ec823/rkyv-0.7.42/src/impls/core/mod.rs:/home/arlyon/.cargo/registry/src/github.com-1ecc6299db9ec823/rkyv-0.7.42/src/impls/core/mod.rs266:67
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
:266:67
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread 'thread '<unnamed>' panicked at 'failed to invoke plugin: failed to invoke plugin on 'Some("<removed>/node_modules/next/dist/client/app-next-dev.js")'

Caused by:
    0: failed to invoke `<removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm` as js transform plugin at <removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm
    1: RuntimeError: out of bounds memory access
    2: heap_get_oob', /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/swc-0.232.53/src/plugin.rs:228:14
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
<unnamed>' panicked at 'failed to invoke plugin: failed to invoke plugin on 'Some("<removed>/node_modules/next/dist/client/next-dev.js")'

Caused by:
    0: failed to invoke `<removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm` as js transform plugin at <removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm
    1: RuntimeError: out of bounds memory access
    2: heap_get_oob', /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/swc-0.232.53/src/plugin.rs:228:14
thread '<unnamed>' panicked at 'failed to invoke plugin: failed to invoke plugin on 'Some("<removed>/node_modules/next/dist/client/dev/amp-dev.js")'

Caused by:
    0: failed to invoke `<removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm` as js transform plugin at <removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm
    1: RuntimeError: call stack exhausted
    2: stk_ovf', /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/swc-0.232.53/src/plugin.rs:228:14
thread '<unnamed>' panicked at 'failed to invoke plugin: failed to invoke plugin on 'Some("<removed>/node_modules/next/dist/client/router.js")'

Caused by:
    0: failed to invoke `<removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm` as js transform plugin at <removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm
    1: RuntimeError: call stack exhausted
    2: stk_ovf', /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/swc-0.232.53/src/plugin.rs:228:14
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: LayoutError', /home/arlyon/.cargo/registry/src/github.com-1ecc6299db9ec823/rkyv-0.7.42/src/impls/core/mod.rs:266:67
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread '<unnamed>' panicked at 'failed to invoke plugin: failed to invoke plugin on 'Some("<removed>/node_modules/next/dist/pages/_error.js")'

Caused by:
    0: failed to invoke `<removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm` as js transform plugin at <removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm
    1: RuntimeError: call stack exhausted
    2: stk_ovf', /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/swc-0.232.53/src/plugin.rs:228:14
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: LayoutError', /home/arlyon/.cargo/registry/src/github.com-1ecc6299db9ec823/rkyv-0.7.42/src/impls/core/mod.rs:266:67
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread '<unnamed>' panicked at 'failed to invoke plugin: failed to invoke plugin on 'Some("<removed>/node_modules/next/dist/pages/_app.js")'

Caused by:
    0: failed to invoke `<removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm` as js transform plugin at <removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm
    1: RuntimeError: call stack exhausted
    2: stk_ovf', /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/swc-0.232.53/src/plugin.rs:228:14
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: LayoutError', /home/arlyon/.cargo/registry/src/github.com-1ecc6299db9ec823/rkyv-0.7.42/src/impls/core/mod.rs:266:67
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread '<unnamed>' panicked at 'failed to invoke plugin: failed to invoke plugin on 'Some("<removed>/node_modules/next/dist/pages/_app.js")'

Caused by:
    0: failed to invoke `<removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm` as js transform plugin at <removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm
    1: RuntimeError: call stack exhausted
    2: stk_ovf', /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/swc-0.232.53/src/plugin.rs:228:14
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: LayoutError', /home/arlyon/.cargo/registry/src/github.com-1ecc6299db9ec823/rkyv-0.7.42/src/impls/core/mod.rs:266:67
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread '<unnamed>' panicked at 'failed to invoke plugin: failed to invoke plugin on 'Some("<removed>/node_modules/next/dist/pages/_error.js")'

Caused by:
    0: failed to invoke `<removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm` as js transform plugin at <removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm
    1: RuntimeError: call stack exhausted
    2: stk_ovf', /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/swc-0.232.53/src/plugin.rs:228:14
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: LayoutError', /home/arlyon/.cargo/registry/src/github.com-1ecc6299db9ec823/rkyv-0.7.42/src/impls/core/mod.rs:266:67
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread '<unnamed>' panicked at 'failed to invoke plugin: failed to invoke plugin on 'Some("<removed>/node_modules/next/dist/pages/_document.js")'

Caused by:
    0: failed to invoke `<removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm` as js transform plugin at <removed>/node_modules/stailwc/target/wasm32-wasi/release/stailwc.wasm
    1: RuntimeError: call stack exhausted
    2: stk_ovf', /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/swc-0.232.53/src/plugin.rs:228:14

I'm running macOS Monterey 12.6.3 on an M1 Mac.

Hi! I am assuming you are using the default emotion engine, which currently is not supported with the app router. Please see this for more info. https://github.com/emotion-js/emotion/issues/2928
If you decide to use pages, I am releasing a new version today and have verified that all the examples work so have a look on master to see if they work for you.

Hi @arlyon! I'm using styled components on this project and as far as I can tell emotion shouldn't have been enabled.
I've tried installing Next 13.4.3 following the latest readme and it all works now. I'm not sure what the issue could have been though, since I'm certain I tried that version of Next before opening this issue. I'll close since it's fixed now.
Thanks for following up and thanks for your work on this project!


Steps direct one to configure logs but not sufficiently to avoid a warning later when running iotedge check

Why not prevent that later warning by including the configuration to enable circular logging. After running iotedge check and investigating that warning, I was recommended to use the following:
{
    "log-driver": "local",
    "log-opts": {
        "max-size": "10m",
        "max-file": "3"
    }
}

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

ID: a3b8296d-c923-e990-dcad-35ef2b908a2a
Version Independent ID: 5f85fd7b-f2b6-b3b1-6be2-b6054bb33507
Content: Create and provision an IoT Edge device on Linux using symmetric keys - Azure IoT Edge
Content Source: articles/iot-edge/how-to-provision-single-device-linux-symmetric.md
Service: iot-edge
GitHub Login: @PatAltimore
Microsoft Alias: patricka

@drwill-ms
Thanks for your feedback! We will investigate and update as appropriate.

@PatAltimore Please look into this feedback and update the doc as appropriate.

Hi @drwill-ms,
iotedge check will be changed in the referenced bug https://github.com/Azure/iotedge/issues/6754. I'm going to request the doc bug be closed.
Thanks,
Pat
#please-close


changefeed(ticdc): fix barrier ts set incorrectly cause monitor become year

What problem does this PR solve?

Issue Number: close #11553
What is changed and how it works?

use MinTableBarrierTs as the checkpoint barrier.

Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`.

/cherry-pick release-8.1

/cherry-pick release-7.5


segment content scroll position

the first segment content scrolled to the bottom
the second one also scrolled bottom
but I dont want the second one scroll
how to solve

Hello! Thank you for opening an issue with us! As this seems like more of a support question I will urge that you ask this question on our forum or on our slack channel.
If this is an Ionic Framework issue (as in, you are reporting a bug in the Ionic Framework code), please open a new issue that has the issue template filled out, please.
Thanks for using Ionic!


Load pages from an api

Hi, is it possible to load the pages object from an api, so instead of storing the pages object inside the code, we save them in the db, to easily change them without having to deploy the app again.

thanks for checking this, do u have an example on how to do that exactly, will help me alot

okay thanks alot


XNXX doesn't seem to be blocked.

xnxx.com, along with MANY more similar ones such as www.xnxx.com are in the hosts file. They are still allowing me to load the site. It's one of the only sites that doesn't cooperate with the hosts blocking. I'm not sure if anyone else has any problems?

Only
<IP_ADDRESS> multi.xnxx.com
<IP_ADDRESS> img.xnxx.com

are blocked.
You can add
<IP_ADDRESS> www.xnxx.com

to your custom black list.

I think we're fully covered and properly here.
data/someonewhocares.org/hosts
7928:<IP_ADDRESS> img.xnxx.com
8496:<IP_ADDRESS> multi.xnxx.com

extensions/porn/clefspeare13/hosts
2092:<IP_ADDRESS> forum.xnxx.com
5637:<IP_ADDRESS> xnxx.com

extensions/porn/sinfonietta/hosts
2662:<IP_ADDRESS> forum.xnxx.com
6650:<IP_ADDRESS> xnxx.com
6651:<IP_ADDRESS> www.xnxx.com
6652:<IP_ADDRESS> www.xnxx.com

Closing.

@StevenBlack I have all of those in my hosts file. I still can access the website. Sending a video now.. (uploading)
Can you test it and see if it works for you?

Could the multi occurrence be to blame ?

2 options.
a) you have to download the video because some reason Google Drive wants to convert it to 360p
https://drive.google.com/file/d/0BzpMdr0-hwrOQy1CdmFHc2tNejg/view?usp=sharing
MD5: 99E6A51E794E98205962B8D4F6CDE586
b) download this rar and unzip it.
https://drive.google.com/open?id=0BzpMdr0-hwrONmNVemZxTnZ2eVk
MD5: 28725E888252FDE2F5179011AEBEDA97

@rautamiekka You don't have to. I'm on Windows 10 and everything is working just fine with <IP_ADDRESS> except xnxx.com
But I did try for you. Still didn't work.

@Jezza-x3 you applied a proper DNS cache flush?

@StevenBlack If you mean ipconfig /flushdns then yes

And I know it worked because I deleted the hosts file and made sure the porn sites would load. I then put the hosts file in. (completely fresh from your page) and did the flushing.
The porn sites stop loading and the block works... Except xnxx.
Did you try it yourself?

@Jezza-x3 I don't use Windows.

@StevenBlack I know. I just wanted you to see if the problem affected your system. What do you use btw?

Found the problem while writing this:
The problem still persists while using Arch Linux with Chrome dev build in the Cinnamon desktop.
Extensions are uBlock Origin, LastPass, Hangouts, Https Everywhere (disabled and enabled still same), Grammarly, WebRTC Network Limiter, Google Data Saver.
Google Data Saver (basically a proxy) screws it up. https://chrome.google.com/webstore/detail/data-saver/pfmgfdlgomnbgkofeojodiodmgpgmkac
After disabling it, it's blocked. It's only non-https sites the Data Saver works on, and xnxx is http only. With https everywhere off, it allowed xvideos to become http and was also unblocked.
So as far as I know right now, the Data Saver is the problem.

Thanks @Jezza-x3 !

No problem @StevenBlack
I already submitted feedback to Google Data Saver and HTTPS Everywhere.


Linked List Cycle in python

Rules:-
1)Follow me and add star in this repository
2)No c++/c file will be accepted
3)Comment for assign this task to you
Follow all rules you will get hacktoberfest-accepted tag

Hey,
Can i write the code for this issue?

Hey,
I added the issue for this code #103 Please tell me if theres a mistake, ill correct it,
Thanks


Faulty link

I believe the link to Azure Resource Graph in this paragraph is faulty: https://learn.microsoft.com/en-us/azure/governance/resource-graph/overview#throttling
The link sends the user to Azure Policy, not Resource Graph.
Best,
Karl

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

ID: 824e201e-3023-45c1-d14d-488e2584b9ef
Version Independent ID: fa14046b-ceb4-1368-2766-39e70c2088c2
Content: Overview of Azure Resource Graph - Azure Resource Graph
Content Source: articles/governance/resource-graph/overview.md
Service: resource-graph
GitHub Login: @davidsmatlak
Microsoft Alias: davidsmatlak

@hammar Good catch! We have verified and updated the document as required. The update will go live in less than 24 hours. Thank you.


Fatal Exception: java.lang.NoSuchMethodError

Fatal Exception: java.lang.NoSuchMethodError
No virtual method put(Ljava/lang/String;Ljava/util/Collection;)Lorg/json/JSONObject; in class Lorg/json/JSONObject; or its super classes (declaration of 'org.json.JSONObject' appears in /system/framework/core-libart.jar)
cordova 8.1.2
ionic 4.5.0
Please help me to resolve this issue.

Hi, I have not seen that issue before.  I do have a test application - https://github.com/darryncampbell/plugin-intent-api-exerciser, are you able to build that?

Hello!
Your example app compiled and works well.
I've compared config.xml and found what your example use android platform 7.0.0,
but my project latest - 7.1.4.
I downgraded to 7.0.0 and plugin in my project works with no errors.
But I need to use latest android engine (it supports fresh firebase and have some urgent for me bugs fixed)
Can you please update your plugin?
P.S. I found what

Thanks for the extra information, I am just trying to reproduce this but am unable to do so.
I am using the https://github.com/darryncampbell/plugin-intent-api-exerciser app and updating the android engine by following the instructions at https://cordova.apache.org/docs/en/latest/guide/platforms/android/upgrade.html to update to 7.1.4.  Everything seems to still work, could you please let me know what I might be doing wrong to reproduce?  Tested on an Oreo 8.1 device.

I've changed android engine to 7.1.4 and recompiled my project again and for now it works fine.
Sorry, looks like it was some build issue in my project. May be some files was not downloaded correctly...
Thank you for your time and great plugin you develop!

No worries, glad the issue is now resolved


BehaviorSystem- changed Map to ConcurrentHashMap

This is done for the new changes merged in https://github.com/Terasology/WildAnimals/ to work.
The additions to the WildAnimals module make changes to the entity's behavior component to switch behavior.
The current system uses events to trigger changes in the Behavior Tree. This means that if the deer is hit, the deer's BehaviorComponent.tree is extracted and it's value is changed from "Pathfinding:Stray" to "WildAnimals:Flee". The exact reverse happens when the deer reaches a safe distance from the player, i.e. BehaviorComponent.tree is changed from "WildAnimals:Flee" to "Pathfinding:Stray".
Whenever a change is made, the entity's (deer's) new entry is added to the entityInterpreter defined in the BehaviorSystem. (Using the onBehaviorActivated and onBehaviorDeactivated methods)
Now the BehaviorSystem has a update running, which gets called from time to time. The update fetches values from the entityInterpreter.
entityInterpreter is a hashMap, mapping entityRefs to their behaviorComponent.
So the problem that surfaces on every second or third run is that I get a java.util.ConcurrentModificationException, which I reckon means that the onBehaviorDeactivated makes a change to entityInterpreter just while the update method tries to read it.
This PR changes the Map to ConcurrentHashMap in the BehaviorSystem, which solves the above issue.

Hooray Jenkins reported success with all tests good!

Have you checked via debugging why this is necessary? Won't it only be called from the same thread? The one that runs the main loop? e.g. have you figured out which thread and method is causing the concurrent modificaiton besides the main thread?

Hooray Jenkins reported success with all tests good!

Leaving slack discussion here-
This has nothing to do with different threads. The entityInterpreter being iterated over in the BehaviorSystem loop is removed at some point when the UpdateBehaviorEvent is called (Everything happens in the main thread).

To summarize: BehaviorSystem.update is called which somehow ends in a call to onBehaviorActivated due to changes in the behavior strategy of an entity, causing an ConcurrentModificationException.
Can't we just decouple the update and remove operations and store the entities and behavior components until the next update?
At the beginning of the update, we could then drain the Maps? with the entities and components to add and to remove and proceed with a default update cycle.
This does pretty much the same because ConcurrentHashMap does not reflect add operations in the iterator, meaning that added entities are first visible in the next update call.
We have a little overhead for two extra collections but only if behaviors are activated and deactivated, compared to the likely more expensive calls to the ConcurrentHashMap (evaluation would be nice, I found only this one: http://stackoverflow.com/a/32141829).
In case we stick to the ConcurrentHashMap, it could make sense to use the advanced constructor and pass a concurrencyLevel less than 16 as the system is not designed for multithreading anyway :)

Hooray Jenkins reported success with all tests good!

@oniatus Thanks for the performance tip.
After a discussion with @flo on Slack, we decided to come up with a new way to deal with this. Since the loop inside update was changing the Interpreters while iterating over it, the ConcurrentModificationException was raised. The new method deals with this by having a cachedInterpreters list which is set to empty every time a Component event handler (OnChangedComponent/ OnActivatedComponent/ BeforeDeactivateComponent) makes a change to any interpreter. Further, in the update() method we now fill the cachedInterpreters list with entityInterpreters.values() (values of the hash map). This list- cachedInterpreters is used for iteration in the subsequent for loop (where changes are made to the entityInterpreters).
I've tested this with around 10 deer, and it seems to work without any issues :)

Hooray Jenkins reported success with all tests good!


Remove code from JsonableError.init.

Since this class was built, folks have always chosen
to subclass JsonableError for situations where
the default of ErrorCode.BAD_REQUEST is insufficient.
So now we simplify the use cases, which also gets
us 100% coverage on this core module.

I think there's probably at a 50% chance we'd end up re-adding this later, but I'm OK with merging.


Fix headers

I messed up the headers for my PPP. Would like to fix that.

Pull Request Test Coverage Report for Build 896

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

Totals

Change from base Build 873:
0.0%

Covered Lines:
2498

Relevant Lines:
2654

💛  - Coveralls


bumpalo and tracing are yanked in Cargo.lock

$ cargo install --locked cargo-about
...
warning: package `bumpalo v3.12.1` in Cargo.lock is yanked in registry `crates-io`, consider running without --locked
warning: package `tracing v0.1.38` in Cargo.lock is yanked in registry `crates-io`, consider running without --locked
...

Furthermore, two unsoundness issues has been reported by cargo audit:
Crate:     atty
Version:   0.2.14
Warning:   unsound
Title:     Potential unaligned read
Date:      2021-07-04
ID:        RUSTSEC-2021-0145
URL:       https://rustsec.org/advisories/RUSTSEC-2021-0145
Dependency tree:
atty 0.2.14
└── cargo-about 0.5.6

Crate:     rmp-serde
Version:   0.14.4
Warning:   unsound
Title:     `rmp-serde` `Raw` and `RawRef` unsound
Date:      2022-04-13
ID:        RUSTSEC-2022-0092
URL:       https://rustsec.org/advisories/RUSTSEC-2022-0092
Dependency tree:
rmp-serde 0.14.4
└── askalono 0.4.6
    └── cargo-about 0.5.6

The --locked is so that latest compatible versions don't get pulled in, potentially breaking compilation, you can try just not adding it and it will use latest versions of all crates and probably be fine.
The audit reports are meaningless for this crate's usage.


Recognition of merged cells

Hi.
Is there any possibility to recognize merged cells?
When cell A1 is merged with A2, the value is stored in A1 only. Cell A2 has System.DBNull value, but how detect that is merged with A1?
Thanks in advance.

This is not possible to do at the moment.
It's a valid feature request, and shares some plumbing work with #215.

I would also like this.

Hi @waclaw66 and @TimSirmovics (and @appel1), do you have any expectations/ideas what this could look like API-wise?
There is progress on reading merged cells in #297 thanks to @Nodrog-82
Looks there are three options, ordered by level of abstraction:

API to get list of merged cell ranges (maps 1:1 to underlying file format)
API GetColumnSpan/GetRowSpan per cell (HTML-like abstraction)
Transparently return same value for all merged cells (practical abstraction)

My personal preference is based entirely on how another Excel API I use works, and that is to return the same value for all merged cells.

How would you determine that it is a merged cell? And not two adjacent cells with the same value?

Updated original comment.

For the use of data loads the fact it is merged is irrelevant. To allow display of data in the same layout e.g. web viewer for excel, then it is relevant.
Personally I would prefer to either leave cells as is and expose the merged cell ranges (This has the the least overhead) or an indicator on the cell that it is merged e.g. ref to the merge cell range object.
If I have time tonight I will profile those two options .

I don't think it is irrelevant for data loads. Depending on the format of the data, a value that has been merged across multiple cells can indicate the intent from the author is to show that each of those cells should contain the same value. Consider a table where a value in one column has been merged across multiple rows.
The fact that the actual value is not stored in each cell seems to be an Excel file format implementation detail of not duplicating the value unnecesarily.

Ok, think I prefer to leave the cells as is, and expose only the merged cell ranges. It's the simplest and most flexible from the ExcelDataReader point of view, and that is what counts :)
Later, it's possible to f.ex implement an option how to handle merged cells with AsDataSet(), and have good documentation/code samples for the reader interface.

For my limited purposes of importing to a database, having the value in the cell in code whether it was stored there or whether part of a merged cell make life easier, but I know it may not be what other need.
Looking again at the code, I think I would have to put a new method on "abstract class ExcelDataReader" for "GetMergedDetails" that returns the range then MergedCells operate on. Any better ideas?
Would adding an option in the ExcelReaderConfiguration on whether to auto populate the cells be useful?
Would you be happier to just expose the MergedCell ranges and let the end developer cope? One word of warning is some cells in the merged range are empty string, others are not loaded at all.

Looks like you beat me to it, I will update the code and resubmit

Hey! I encountered this trouble today.My form is like this.
Obviously, "name" and "Gender" are combined cells, and "Hobby" is not.

I hope to support such a excel.
I saw the code you submitted, Nodrog-82. I hope to update it as soon as possible
Thank you for your efforts

Here is an example MergeCellsReader wrapper class for IExcelDataReader using the new MergeCells property. It returns the top left value of merged cells transparently, and can be used with AsDataSet etc:
https://gist.github.com/andersnm/7205389e5e5a71773a9716e0853e3ee7
Usage:
using (var reader = new MergeCellsReader(ExcelReaderFactory.CreateReader(stream))) {
 ...
}

How about a new "ExcelDataReader.Extensions" library for stuff like this?

For my current need I am going via the reader direct due to certain
limitations (e.g. column headers have to be in row 0, whereas I deal with
files with actual headers in row 5 due to users having titles on the first
few rows) I. Not sure how best to add this helper to the Reader without
putting it actually in the reader class which is was decided against.
Having the extensions library is a great Idea and will make it easier for
simple usages of scraping data, not layout/intent.
On Sun, Dec 3, 2017 at 11:07 AM, andersnm<EMAIL_ADDRESS>wrote:

Here is an example MergeCellsReader wrapper class for IExcelDataReader
using the new MergeCells property. It returns the top left value of merged
cells transparently, and can be used with AsDataSet etc:
https://gist.github.com/andersnm/7205389e5e5a71773a9716e0853e3ee7
Usage:
using (var reader = new MergeCellsReader(ExcelReaderFactory.CreateReader(stream))) {
...
}
How about a new "ExcelDataReader.Extensions" library for stuff like this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ExcelDataReader/ExcelDataReader/issues/269#issuecomment-348756467,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYoICDwlvfdyV7vM4T2Bi_MguX0OAdG8ks5s8oD0gaJpZM4PCh1-
.


Observed an unusual scrolling issue with a screening effect on Mac Safari aswell as Chrome.

What version of Remix are you using?

Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce

https://user-images.githubusercontent.com/75626747/233609148-497150d8-6323-4ca6-8078-22dafd5b732a.mov
Expected Behavior
.
Actual Behavior

Logged this on the private repo for the remix website, thanks!


fuzzywuzzy is now thefuzz

The fuzzywuzzy Python library has been renamed, it is now thefuzz.

Hello Edward,
Thank you for letting me know and preparing the pull request. I appreciate it; and support many of the organizations you are involved in. Debian and the Internet Archive and Wikidata are all tools I use regularly.
Sincerely,
Caleb


make headers in documentation clickable to generate sharing links

All documentation headers on https://lit-html.polymer-project.org/guide/* should be clickable to enable people to share direct links to the documentation sections.
For example clicking this header

should update the URL to:
https://lit-html.polymer-project.org/guide/writing-templates#bind-to-attributes
or potentially copy the link to my clipboard.

Good news: we can generate a trailing symbol with a permalink just by setting permalink: true here:
https://github.com/Polymer/lit-html/blob/master/docs/.eleventy.js#L29
Bad news: the e11y TOC plugin we're using doesn't know any better than to include the permalink symbol in the TOC, which looks terrible.
Ideal might be to make the whole heading a link, but I don't see an option for that in markdown-it-anchor.
Reassigning to @justinfagnani to see if he has any ideas.

Fixed in 2.0 docs.


pre-commit: PR114499

Link: https://github.com/llvm/llvm-project/pull/114499
Requested by: @dtcxzyw

/add-label reviewed
/close


fix: migrate from "debug" to "@open-draft/logger"

Migrate to https://github.com/open-draft/logger
Related https://github.com/mswjs/msw/pull/1436 (to improve the ESM support)

Released: v0.22.11 🎉
This has been released in v0.22.11!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/interceptors@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


Incompatibility warning card on main screen (COMMUNITY)

Continuation of #2481 by @fynngodau
I rebased the PR and made some minor adjustments.

[ ] Adjust strings
[ ] Add onclick event
[ ] Finish unit tests

Also see comments in #2481

Internal Tracking ID: EXPOSUREAPP-5464

Thank you so much for picking this one up!

@fynngodau
I'm very happy that this will go into 2.2. At that time, Event Registration (=presence tracing) will already be the second main feature. I wonder, if the currently proposed texts for the cards of this PR might get confusing for the less advanced users, and if they might be irritated that the app as a whole is not working, eventhough the incompatibility warning refers to RPI receiving/submitting bluetooth functionality only.
Would it make sense to adapt the texts?
For (a very quick and dirty) example: Unfortunately, your device is only partially compatible with Exposure Notifications that work with bluetooth. You can be notified for risk contacts that were in range of your bluetooth receiver but won't be able to warn others in the same way. Exposure notifications from registered events (Event Registration) and warning others in that way will work as expected.
And: Unfortunately, your device is not compatible with Exposure Notifications that work with bluetooth. Exposure notifications from registered events (Event Registration) and warning others in that way will work as expected.
(the challenge is, to be as precise and as short as possible...)
This might be even more important, when support for Android 5 is about to come, as more people with compatibility limitations will adopt CWA, imho.
What do you think?

A good change would be to add an "open externally" icon to the card to indicate that it can be clicked. You can use this commit for that: https://codeberg.org/corona-contact-tracing-germany/cwa-android/commit/a22275e6e75af1b75b8141bed07626c14153d940


Clarify doc page on Databricks execution

Codecov Report

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

@@          Coverage Diff           @@
##           master     #35   +/-   ##
======================================
  Coverage    69.6%   69.6%
======================================
  Files          44      44
  Lines        2527    2527
======================================
  Hits         1759    1759
  Misses        768     768

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 5f75253...9ef72e2. Read the comment docs.


feat(scheduled-tasks): expose the strategy client as getter

This is an easy way to get at the original client of the strategy.
I have opted to not expose it on the ScheduledTaskHandler level
because that would mean the handler would need the types
for both SQS and Bull which is what we want to avoid.

@sapphiredev pack

@sapphiredev pack

@sapphiredev pack

The problem this was intended to solve is not solved by this PR, therefore we're closing and rejecting it.


Success Resposnce issue

Entray Success Page url is dynamic how to setup success responce

Use RegEx
https://efiller-doc.netlify.app/documentation/site.html#form-url-match-types


DataSet: finalize and tidy up

...so that it can be considered "implemented" and we can move over to normal update/fix routine.

will also pick up the fix for #574.

Resolved by #575.


PHP Tag Problem

When I write
<?php
echo "Hi;
?>

Nothing shows.
Without the <?php ?> it works.
Please help with this bug.

You need to escape < as <.

Thank you Golmote. Solved the issue by using  tags.

I have this same issue and escaping it does solve the problem. But how come it works without having to escape the code when I test it on the test-drive page: http://prismjs.com/test.html . ?


Rspec tests are failing.

Make Travis green again.

Le'ts put a travis badge in the README as well.

We will need to implement rspec-puppet-facts to help with strict_variables, and then review other failures afterward.

See https://gist.github.com/rnelson0/337815862e07af1849605b2736cfcc0d for a VERY rough way to get the tests working (I don't recommend implementing as is, I just ran out of time to tune it right now).

They are not failing for v.1.3.0, so closing this.


New version: AWSCore v0.6.4

Registering package: AWSCore
Repository: https://github.com/JuliaCloud/AWSCore.jl
Created by: @mattBrzezinski
Version: v0.6.4
Commit: adc540db8fad1fc5fc8e9446de23276b038f899b
Reviewed by: @mattBrzezinski
Reference: https://github.com/JuliaCloud/AWSCore.jl/commit/adc540db8fad1fc5fc8e9446de23276b038f899b#commitcomment-35711050

Your new version pull request met all of the guidelines for automatic merging.
I will automatically merge this pull request during the next cron job.
If you want to prevent this pull request from being auto-merged, simply leave a comment.
(If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.)

[noblock]


Fwdport/7.4 8.0

PR has been updated. Reviewers, please be cautious.

Sounds like there's some commits that are already on development/8.0 (my commits)
Also the fwdport/* is not supported in the main.yml file so you won't have any build on that branch. If that's ok for you LGTM

Oh right. Let me just cherry pick instead.


list of words remove the entire line if found in other list

to understand what i mean, please go to:
https://alphabetizer.flap.tv/remove-one-list-from-another.php
add on top:
www
xbet

and on the bottom add:
www text
one more
xbet

and when you hit the button, it only removes the word "xbet", i need it to remove also "www text"
lets call this first function "Remove list of words from another list"

no, only if the line matches


Reference-counting AdvisoryLockGuard

Is your feature request related to a problem? Please describe.
I have an issue trying to use AdvisoryLockGuard. The lifetime prevents me from writing code that acquires the lock and then passes the guard into a 'static closure. This is relevant because I want to be able to write an axum request handler that upgrades to a websocket, but I want any errors that occur while acquiring the lock to be sent in a Response instead of a websocket close message.
A rought sketch of how I want the code to look:
async fn my_handler(
    ws: WebSocketUpgrade,
    State(state): State<MyState>,
) -> Result<Response, ApiError> {
    let db = state.get_db();
    let lock = PgAdvisoryLock::new(...);
    let Either::Left(mut guard) = model_lock.try_acquire(&mut db).await? else {
        return Err(ApiError::Foo(...));
    };

    let response = ws.on_upgrade(move |socket| async move {
        // Have the guard in here while doing stuff with the socket...
    });

    Ok(response)
}

Describe the solution you'd like
I think having an API similar to this would work, though I'm not sure how feasible it would be to implement that for sqlx.
Describe alternatives you've considered
AFAICT my only (safe) option is to move the PgAdvisoryLock into the task scope that handles the socket. This means I must return any errors within that scope along the socket instead of in an HTTP response.

If the key string you pass to PgAdvisoryLock is constant, you could put it in a smol::lock::OnceCell or a std::sync::LazyLock inside a 'static which would give you &'static references to it.
We could probably promote with_key() to a const fn so that it can be constructed directly in a static.
Otherwise, I'm not against .acquire_owned()/.try_acquire_owned()  methods taking Arc<Self>. There's no real reason it doesn't exist already besides "I didn't think of it".
Couple bikeshedding things:

I'd prefer the _owned suffix to match Tokio: https://docs.rs/tokio/latest/tokio/sync/struct.RwLock.html#method.write_owned
It's a judgement call whether to take Arc<Self> or &Arc<Self>. The latter would increment the reference count lazily for less thrashing on highly contended locks, but the former would allow even the acquire() call itself to be moved into a task if desired.

I'm leaning toward Arc<Self> since it's strictly more flexible at the minor cost of requiring an explicit .clone(). Thrashing the refcount doesn't really matter since it's going to be dwarfed by the network overhead anyway. It also matches Tokio's API.

Unfortunately my keys are not constant. But I appreciate the idea.
I agree with your design thoughts.
I might have time to try implementing this feature if you're open to that.

Yeah, I'd appreciate it!

@abonander I'm starting to work on this, and I've had a few realizations.
First, we probably don't need reference counting for this. I think I incorrectly pattern matched when proposing the RwLockReadGuardArc. Postgres advisory locks are more like mutexes, so we don't really need the ability to clone read guards.
But maybe there is some desire to use Arc<AdvisoryLock> instead of just AdvisoryLock; at least this is what tests are doing. I have not personally needed to do this. Is this an optimization to avoid deeply cloning the AdvisoryLock?
If we really did want to have a cloneable guard, I think it would also need a mutex around the connection, since most users would need a AsMut<PgConnection> to keep making queries with the lock guard. This seems different enough from the existing API that I don't think it's worth going down that route.
So for now I've opted to support Arc<AdvisoryLock>, but there is no reference counting of the guard itself.
pub struct PgAdvisoryLockGuardOwned<C: AsMut<PgConnection>> {
    lock: Arc<PgAdvisoryLock>,
    conn: Option<C>,
}

I'm also wondering how we want to handle documentation. It seems like any new documentation would mostly duplicate existing documentation. Same with the tests.
Here's some progress in a draft PR: https://github.com/launchbadge/sqlx/pull/3442
Any thoughts?


Testing color and component themes in themeTest

Flutter is moving towards removing/deprecating all color properties from ThemeData and having components depend only on ThemeData.colorScheme, ThemeData.textTheme, and component-specific themes such as ThemeData.floatingActionButtonTheme (see here). The colors in ThemeData.colorScheme and most of the component-specific themes are currently not tested by themeTest. Are you planning on supporting all possible text/background pairs in these themes as well?

I would definitely want to support newest colour combos, thanks for bringing that up. Adding support should be easy because, as I understand, it's just grouping the properties differently. I can't give any timeline though, so PRs will be welcome 👍

Thanks! I can give it a try once I get round to it (will be a couple of weeks though).


GPC_4T - Upload a PPT with the Core Data Path

[ ] add PPT in the doc/ folder.
[ ] For Each instruction in the RV32I ISA Please add a Slide with the DataPath+Ctrl Bits highlights.
[ ] List the Ctrl Bit in a Slide and explain why they are needed.
[ ] Add a Slide with a Table of all Instructions -> for each Instruction Set the Ctrl Bits.
Use the riscv-Spec sheet (see doc/)  to determine the data path for each instruction.

I believe the ppt is ready for 0.1 what do you think?

I agree - i have some notes regarding the diagram.
So its probably better to discuss the PPT before you finalize the drawing.
You may close this issue


Documentation for error messages

Hello,
Was stoked to find this pam module, but I'm getting an error "Authentication was successful but authorization failed" that is not explained within the code comments or README.
Would you kindly consider adding an explanation of what this error means? ...and also tell me so I might be able to leverage this amazing module?

Glad you liked the module. I will update the documentation soon.
The error message is shown up because the designated Keycloak role was not found on the current user. To trouble shoot, please check:

the role exists and assigned to the user
name of the role is configured in the config file
the role is included in the OIDC token scope

For 1), an easy way to check is Keycloak -> Users ->  -> Role Mappings. Depending on how you configured Keycloak: if the pam role is at Realm level, check Real Roles -> Effective Roles; or if the pam role is at Client level, check Client Roles ->  -> Effective Roles. Either way you should see the designated role, let's call it demo-pam-authentication. In case you don't see it, please assign the role to the user directly, or assign the role to a group then assign the user to the group.
For 2), check your pam-keycloak-oidc.tml, the line vpn-user-role="demo-pam-authentication" should refer to the same role above. This is a misleading name, I initially created the module for a Radius authentication for VPN, but when I open sourced it, everywhere else is generalized to pam but not this property.
For 3), please refer to the README and double check the settings of the Client Scope (including its Mappers), and make sure the Client Scope is in the "Assigned Default Client Scopes" of the corresponding Client.

Thanks for the support and commitment to adding docs!
I thought that I triple checked all of the configs, but I forgot to set the name of the scope in the token. With that being fixed it works as expected.
Now to figure out why I get error code 13 when actually trying to use it with PAM. 🙃


.expect(statusCode) is not being asserted

For a long time i have being trying to correct this but I am unable to find what the issue is.
it('Should be able to DELETE a Valid asset', function(done) {
var jsonPayload = JSON.parse(JSON.stringify(deFaultJsonPayload));
request.post('/ngp/asset').send(jsonPayload)
.set('User', 'admin')
.set('Password', 'YWRtaW4=')
.set('Accept', 'application/json')
.set('Content-Type', 'application/json')
.set('Authorization', 'Basic YWRtaW46YWRtaW4=').expect(200)
request.delete('/ngp/asset' + jsonPayload.location)
.set('User', 'admin')
.set('Password', 'YWRtaW4=')
.set('Accept', 'application/json')
.set('Content-Type', 'application/json')
.set('Authorization', 'Basic YWRtaW46YWRtaW4=')
.expect(404)
.end(function(err, res) {
if (err) {
done.fail(err);
} else {
done();
}
});
});
Here .expect is not being evaluated and returns true for any code.
please help.

+1 for this - is driving me mad

I :+1: for this issue, there really is a bug here.

:+1:  This issue makes supertest unusable as a valid testframework!
I have no clue about internals, but please fix this issue ASAP as it's a real show-stopper...

I'm seeing this too.  I'm using Jasmine in a node app.  Here's a simple example that fails:
var request = require('supertest')('http://google.com');

describe('Just open Google front page', function(){
  it('works', function(done) {
    request.get('')
      .expect(400)
      .expect(function(res) {
        if (true) return 'should not this fail?!?';
      })
      .end(function(err, res){
        done();
      });
  });
});

Then I run it thusly...
$ jasmine path/to/this/file.js

(jasmine version 2.3.4)
and get...
Started
.

1 spec, 0 failures
Finished in 0.101 seconds

@heck, I think your sample is incorrectly set up - the expect is evaluated, and the error is being passed to the end callback, but you are not passing it as an argument when calling done. Try the following test... it works fine for me.
var request = require('supertest')('http://google.com');

describe('Just open Google front page', function(){
  it('works', function(done) {
    request.get('')
      .expect(400)
      .expect(function(res) {
        if (true) return 'should not this fail?!?';
      })
      .end(function(err, res){
        done(err);
      });
  });
});

results in:
  1 failing

  1) Just open Google front page works:
     Error: expected 400 "Bad Request", got 301 "Moved Permanently"
      at Test._assertStatus (node_modules/supertest/lib/test.js:232:12)
      at Test._assertFunction (node_modules/supertest/lib/test.js:247:11)
      at Test.assert (node_modules/supertest/lib/test.js:148:18)
      at assert (node_modules/supertest/lib/test.js:127:12)
      at node_modules/supertest/lib/test.js:124:5
      at Test.Request.callback (node_modules/supertest/node_modules/superagent/lib/node/index.js:797:3)
      at IncomingMessage.<anonymous> (node_modules/supertest/node_modules/superagent/lib/node/index.js:990:12)
      at endReadableNT (_stream_readable.js:893:12)

@mona2052, it actually makes sense that the expect method is not raising any errors - your server is probably returning 404. With the way you've set the test up, the two requests (POST and then DELETE) are performed asynchronously - the delete request might be processed before the post request, at which point it does not find the asset, and you actually do get 404 response. You should instead nest the second request inside the end callback of the first one - this guarantees that the asset is added to the database before trying to delete it. Maybe if you could give the following test a try?
it('Should be able to DELETE a Valid asset', function(done) {
  var jsonPayload = JSON.parse(JSON.stringify(deFaultJsonPayload));

  // Create asset
  request.post('/ngp/asset').send(jsonPayload)
    .set('User', 'admin')
    .set('Password', 'YWRtaW4=')
    .set('Accept', 'application/json')
    .set('Content-Type', 'application/json')
    .set('Authorization', 'Basic YWRtaW46YWRtaW4=')
    .expect(200)
    .end(function(err) {
      if (err) return done(err);

      // Delete asset
      request.delete('/ngp/asset' + jsonPayload.location)
        .set('User', 'admin')
        .set('Password', 'YWRtaW4=')
        .set('Accept', 'application/json')
        .set('Content-Type', 'application/json')
        .set('Authorization', 'Basic YWRtaW46YWRtaW4=')
        .expect(404) // This should fail now - the asset should be found and deleted
        .end(function(err, res) {
          if (err) {
            done.fail(err);
          } else {
            done();
          }
        });
    });
});

@dylanrhysscott @Codii @oschwengers, it would be nice if you could provide a sample test where the .expect is failing for you.

@AhmedAKZM I'm not sure if this is 100% relevant to this thread, but I've been experiencing similar issues. Here is an example of a test that does not work. The output suggests that the test was looking for status code '200' instead of the '404' specified.
    it('should handle requests to non-existent routes', function(done) {
      request(app)
        .get('/doesnotexist')
        .expect('Content-Type', /json/)
        .expect(404, {
          errors: [
            {
              status: "404",
              title: "Resource not found",
              detail: "Could not locate the requested resource."
            }
          ]
        }
)
        .end(function(err, res) {
          if(err)
            done.fail(err);
          else
            done();
        });
    });

Thanks for your reply @AhmedAKZM!
Sadly, if I try to run your code exactly I get...
$ jasmine spec/server/supertestSpec.js
Started
.

1 spec, 0 failures
Finished in 0.121 seconds

But, if I take @peterhadlaw 's suggestion...
var request = require('supertest')('http://google.com');

describe('Just open Google front page', function(){
  it('works', function(done) {
    request.get('')
      .expect(400)
      .expect(function(res) {
        if (true) return 'should not this fail?!?';
      })
      .end(function(err, res){
        if (err) done.fail(err);
      });
  });
});

Then it does fail:
$ jasmine spec/server/supertestSpec.js
Started
F

Failures:
1) Just open Google front page works
  Message:
    Failed: expected 400 "Bad Request", got 301 "Moved Permanently"
  Stack:
    Error: expected 400 "Bad Request", got 301 "Moved Permanently"
        at Test._assertStatus
<CALL STACK DELETED BY ME>

1 spec, 1 failure
Finished in 0.104 seconds

Which is livable for me.
@AhmedAKZM: If you clue me in on how to run this through a debugger (Node debugger?) I'll give it a roll.
Thanks,
heck

@peterhadlaw, I tried your test against my own (Hapi.js) server, and it seems to be working fine (*disclaimer: I had to change the expected response to match Hapi's default 404 response). The test passes for any non-existent endpoints. I am using jasmine v2.3.2 and supertest v1.1.0.
var request = require('supertest');

it('should handle requests to non-existent routes', function(done) {
  request('http://localhost:8000')
    .get('/doesnotexist')
    .expect('Content-Type', /json/)
    .expect(404, {
      statusCode: 404,
      error: "Not Found"
    })
    .end(function(err, res) {
      if (err)
        done.fail(err);
      else
        done();
    });
});

Rather odd - I'm unable to reproduce your error. What versions of jasmine and supertest are you using? Could you post the output here?
@heck, you're right. My code does pass (when it shouldn't) when using jasmine. I was previously using the mocha suite, which seems to handle the done callback differently. I'm sorry; I should have read your comment more carefully (i.e, should have used jasmine to try to reproduce the error instead of mocha). Your test works fine for me with jasmine.

I have the same problem using v1.1.0

+1
supertest.agent(myMainJS).send({}).expect(422, function(err, res) {

I think in these examples supertest is working as expected.
Please pay attention to this section in the READMe about handling errors in expect if you also are using a .end function.

If you are using the .end() method .expect() assertions that fail will not throw - they will return the assertion as an error to the .end() callback. In order to fail the test case, you will need to rethrow or pass err to done(), as follows:

@AhmedAKZM  you sir are a golden god. I was doing done() instead of done(err) in all my tests and pulling my hair out trying to figure out why expect(200) was still passing on a 404 error.

@hamoderr in your second code snippet it uses .end twice. I'm trying to do the same thing and am being yelled at by the console, so I switched it to .then and it still isn't working. Does .end and then .end again in the same block work for you?

@heck     it('should handle requests to non-existent routes', function(done) {
request(app)
.get('/doesnotexist')
.expect('Content-Type', /json/)
.expect(404, {
errors: [
{
status: "404",
title: "Resource not found",
detail: "Could not locate the requested resource."
}
]
}
)
.end(function(err, res) {
if(err)
done.fail(err);
else
done();
});
});


corpus-level bleu & sentence-level bleu

I noticed that two different bleu metrics in your eval.py.
It's thoughtful of you to include both of them.
But, which one is the metrics used in current published papers, such as in <Show, attend and tell- Neural image caption generation with visual attention (2015), K. Xu et al>?
Thanks for your answers.

Show, Attend and Tell uses this and so does Show and Tell. Not sure if their implementation computes the micro-average precision(corpus-level) or the macro(sentence-level).

@metalaman thanks, I will try https://github.com/tylin/coco-caption.