Feature Request:  Automate assigning variable labels

Feature Idea
In current template code, the newly created variables do not have labels assigned to them. Bearing in mind the esub-readiness principle, this should be added to the template code. Moreover, instead of manually assigning labels, there should be a way to automate this process by directly reading spec.
Relevant Input
No response
Relevant Output
No response
Reproducible Example/Pseudo Code
No response

Labels are brought up quite a bit and are covered in the e2e examples.  However, a user might not be aware of the e2e examples and so maybe we should start applying them in our templates and vignettes.
We could leverage metacore/metatools or the xportr package to take in spec and apply the labels.  I don't think a spec file would take up too much space in our package.  Or maybe we could store them somewhere other than the package and read them in with the script and then apply them.
@pharmaverse/admiral any thoughts?

We could add a section to the "Get Started" page, which refers to the e2e example page.
Creating specs and adding the metacore, metatool, xportr calls to all the vignettes and templates looks like a lot of initial and maintenance work to me.

So far we refrained from adding this to our vignettes because that would make {admiral} depend on the other packages mentioned. Thus, I'd much rather point users to the e2e example (ideally also directly towards the end of the templates in a comment.

I think pointing to e2e examples is the best way forward to minimize work and dependencies like you all suggested.   @w316liu do you mind making this change to our vignettes?


process.pid on phantom.create()

Is there a way to get the process ID of the shim after phantom.create? Not on a mac...

You could before and now it is phantom.process.pid.


Relaxed device serial validation

Device serial number validation pattern is too strict for older devices. My Yubikey 4 supports TOTP however its SN has a 7 only digits. I suggest completely relaxing the rule as a simple fixup.
I can imagine device SN validation could be removed altogether at the cost of a BC break (ErrDeviceSerial removal).

That's fair point! No reason to block other Yubikey models "just because validation rule".
I'll upgrade the vegas-credential dependency later today as well.

@RadekDvorak sorry, I forgot to do the upgrade previously but now this is available in v0.14.0 https://github.com/aripalo/vegas-credentials/issues/34


[Bug] Incorrect result otlp collect when send delay message

Before Creating the Bug Report

[X] I found a bug, not just asking a question, which should be created in GitHub Discussions.

[X] I have searched the GitHub Issues and GitHub Discussions  of this repository and believe that this is not a duplicate.

[X] I have confirmed that this bug belongs to the current repository, not other repositories of RocketMQ.

Runtime platform environment
contos 7
RocketMQ version
5.1.5.SNAPSHOT
JDK Version
jdk8
Describe the Bug

send 1000 messages to the topic "test", and message typs incorrect
Steps to Reproduce

send delay message
msg.setDelayTimeMs(3000);
search by prometheus

What Did You Expect to See?
message type is "delay"
What Did You See Instead?
message type is "normal"
Additional Context
No response

when get message type in brokermetrics, there losts PROPERTY_TIMER_DELAY_MS


Multi-backends for UDPRoute and TCPRoute

We started work on TCPRoute and UDPRoute in parallel with https://github.com/Kong/kubernetes-ingress-controller/issues/2166, now that that's complete we just need to tidy up and add the support to these APIs as well.

waiting on #2086


add tab for any query from editor

add a button or tab through which i can generate issue from directly your editor. 👍

I'll implement it in next update for sure


Exception when using manual merge (Variations screen)

When doing a manual merge from branch-a to branch-b, I get a warning:
Cannot read property 'branchName' of undefined

Steps:

Run local vrt on
Create new project REPRO, select master as main branch
Create any test, which has at leat 1 image
Run test with config

    {
      apiUrl: "http://localhost:4200",
      apiKey: "....",
      project: "REPRO",
      branchName:  "dev",
      ciBuildId: `LocalRun_0`,
      enableSoftAssert: true,
    }

Approve image using web-UI
Go to project variations
Merge dev->master

Actual:
"TypeError: Cannot read property 'branchName' of undefined
at TestVariationsService.merge (/dist/test-variations/test-variations.service.js:140:51)
at runMicrotasks (<anonymous>)
at processTicksAndRejections (internal/process/task_queues.js:97:5)"

Expected:
new testrun on master with approved baselines


Login menu looks really big

I'm using a 2560x1080 screen and all the menus look really big but I couldn't find anything about resizing them.
Any tips?

This is why your having that issue: https://github.com/artur9010/lightdm-webkit-material/blob/master/assets/css/style.css#L84-L126
I targeted 4K screens when I wrote that. I think I'll add a gui scale option in the settings, but idk when that will actually come to fruition. Changing min-width:2000px to min-width:3000px on line 84 is a quick hack solution.


Subject is not a constructor

I can't seem to get rid of this error when I use the component. It doesn't seem to like that the service uses the 'Subject' from rxjs as a constructor, i.e. 'new Subject()';
ERROR TypeError: Subject_1.Subject is not a constructor
    at new ContextMenuService (contextMenu.service.js:23)
    at FakeModuleInjector.get (module.ngfactory.js:144)
    at FakeModuleInjector.getInternal (module.ngfactory.js:256)
    at FakeModuleInjector.NgModuleInjector.get (ng_module_factory.ts:141)
    at resolveDep (provider.ts:504)
    at createClass (provider.ts:371)
    at createDirectiveInstance (provider.ts:192)
    at createViewNodes (view.ts:290)
    at callViewAction (view.ts:699)
    at execComponentViewsAction (view.ts:643)
    at createViewNodes (view.ts:355)
    at Object.createEmbeddedView (view.ts:208)
    at TemplateRef_.createEmbeddedView (refs.ts:519)
    at ViewContainerRef_.createEmbeddedView (refs.ts:274)
    at NgIf._updateView (ng_if.ts:159)

I can't figure out how to reproduce this.
What version of rxjs are you using?  And what version of Angular?  It would be really helpful if you could make a small github repository that reproduced this.

Rxjs 5.3.0 (but it does the same with 5.2.0) and Angular 4.0.1

Hm.  I just created a sample app that works without this error.  https://github.com/isaacplmann/ngx-contextmenu-test
Can you modify this one or make your own that demonstrates the problem?

I finally figured out where the problem lies, however, I am not sure what the resolution is. I am using a Node.js server and I am using systemjs to load my modules. There is some type of breaking change that happened after<EMAIL_ADDRESS>that is currently incompatible with ngx-contextmenu.

Ah, so you're using systemjs to build.  I'm not using that myself, so I'm afraid I'm not supporting it as well as I do webpack builds.  Issue #5 should help resolve this somewhat, once I get around to it.
I found an issue filed against the angular cli that might be related to what you're experiencing.  Let me know if you find a solution or if you have any guidance for how I can better package the library.  (PRs are always welcome.)

Closing this in favor of #5

"rxjs": "^5.2.0"
Problem with version of rxjs just change and do npm install, error will fix


[BUG] new JDate wrong example

according:
https://alihoseiny.github.io/ngx-persian/classes/JDate.html#source
line: 99
@example new JDate(1397, 12, 25)
input month must be in 1 base (not zero base),
but this produce next GDate,
month must be zero base: @example new JDate(1397, 12-1, 25)
in isValidJDate you mention this point (month number starting from zero),
please check and fix example, also write some Where this point.

Thank you for reporting. Fixed.


Hoisting logic needs to consider dependency on prior unhoistable variable

Fix #9581, #9490.

@DrTodd13, i just realize you have a PR that sounds similar. Will https://github.com/numba/numba/pull/9397 fix this as well?

@DrTodd13, i just realize you have a PR that sounds similar. Will #9397 fix this as well?

My instinct is to say that if the dependency is two lines right after each other then that other PR wouldn't help but it is worth trying it.  I'd also suggest maybe rebasing this PR against mine and doing them together rather than separately.

I confirmed that #9397 do not fix the issues https://github.com/numba/numba/issues/9581, https://github.com/numba/numba/issues/9490.


Create hacktoberCTF.html

I don't really see how this contributes to the website or the project? :/


About X-Total-Count

May we provide information in X-Total-Count like JSON parameter?
In my case: I use ExtJS for build my application. Currently I have problem with getting information from response header.
In my opinion: We can make json-server response like
{ count: .... , data: .... }

There's many way to return a slice of an array, and you're right that's a valid way to do.
There was also a discussion about pagination here: https://github.com/typicode/json-server/issues/57
For the moment, I prefer to return an array and meta in headers. But maybe a mock server implementing JSON API would work better with ExtJS (never used this lib)... can't remember right now other mock server names, but there is fortunejs.

Hi @typicode
we can enable this (a count property in the body response) via a flag?
Several (request) solution doesn't provide a way to access to header data.

Hi,
You can use middleware mechanism to take count from header and put it in Jason returned

Is there any solution to do that?

I found a solution by creating server.js :
Const jsonServer = require('json-server')
.....
....
....
router.render = function(req, res) {
Var newData = {
TotalCount: res.get('X-Total-Count'),
Result: res.locals.data
}
res.json(newData)
}

With your solution, you'll not have the "total count", but the number of returned items.
I found a solution using lowdb. It's not perfect because we need to call the db twice (one for the initial request, one for the total count).
For example, I want to count the number of users.
const db = await low(myStorage);

const countItems = resource =>
  db
    .get(resource)
    .map('id')
    .value().length;

router.render = (req, res) => {
  res.jsonp({
    items: res.locals.data.items,
    totalItems: countItems('users') // parse url to get the desired resource
  });
};

I found a solution by creating server.js :
Const jsonServer = require('json-server')
.....
....
....
router.render = function(req, res) {
Var newData = {
TotalCount: res.get('X-Total-Count'),
Result: res.locals.data
}
res.json(newData)
}

Thank you,  This should be the accepted answer.
I was searching for how to get X-Total-Count from render, you answer inspired me, put some slightly formatted code here.
router.render = (req, res) => {
  // if it's a GET request and result is an Array
  if (req.method === 'GET' && Array.isArray(res.locals.data)) {
    res.json({
      total: res.get('X-Total-Count'),
      list: res.locals.data,
    })
  }
}


Loading libMoltenVK.dylib instead of libvulkan.dylib on iOS

Hi there,
I saw there is an old commit made here: 594a476fdb7a9fe211a7257f353dbf9807e87786, which renamed the dynamic library load path on iOS to be libvulkan.dylib instead of libMoltenVK.dylib. However, it seems to cause crashes on iOS if the standard MoltenVK build instructions are followed.
In specific, it seems that building MoltenVK somehow internally assumes that the file is named in this specific way. When any code where ash instance is created using the load() function, it searches for libvulkan.dylib in the iOS Frameworks folder. The unexpected behaviour is that if this file does exists, an error is produced which says that libMoltenVK.dylib does not exist. If copies of the two files are added inside the Frameworks folder, everything works.
I can put sample applications online in a few days and submit a PR if this seems something you want to fix. Alternatively, I could try to find a way to fix the naming scheme and make instructions on how to do so. If you already know how to fix this, maybe reference that in the README?

I was not aware that there could be some other loader program that could be used instead of MoltenVK on iOS. Because of this I was thinking that maybe the assumption could be made, but I see the reasoning why this is not the case now.
Thanks for the prompt reply, I will close this ticket for now.

I also wasn't aware that libMoltenVK could be loaded through the standard Vulkan loader (does MacOS ship with one at all? I've never physically touched this platform). There is however ash-molten which statically links against libMoltenVK which might do what you need @jhvst?

In my initial description both libvulkan.dylib and libMoltenVK.dylib are the same file that is produced by the MoltenVK build process for iOS. I now recognize how you might have misread this, but libvulkan.dylib is not a Vulkan loader, it's the MoltenVK library renamed. Since there is no other way currently than MoltenVK to use Vulkan bindings on Mac and iOS, I thought that changing the default on iOS makes sense -- I use MacOS as my development machine and this I control with the ICD files as environment paths. My problem with iOS is that I do not know how to set up similar environment paths (I will look into this eventually, but it's not my priority right now), nor did it even occur to me at the time of opening this ticket that I had setup my Mac a long time ago with the ICD paths. With the iOS in particular being more constrained environment, I was thinking if it would make sense to revert the referenced change for iOS.
ash-molten is also something I completely missed, the process seems generalized version of mine. I can see why it does not run into the same problem, because it's not using dynamic linking. I think I might as well move into that crate later on. Thanks for the heads-up and sorry about the fuss!

I don't have personal experience with MacOS either, but I do know it doesn't ship with the loader; applications would have to bundle it themselves.

In my initial description both libvulkan.dylib and libMoltenVK.dylib are the same file that is produced by the MoltenVK build process for iOS. I now recognize how you might have misread this, but libvulkan.dylib is not a Vulkan loader, it's the MoltenVK library renamed.

I have not misread this, I have simply subsumed that - if there exists a MacOS loader - it's probably called something like libvulkan.dylib (with .1 actually) which would be a plausible explanation for 594a476fdb7a9fe211a7257f353dbf9807e87786 to exist. I clearly understood that you renamed it in hopes of tricking ash into loading libMoltenVK through the "ICD loader" library name (this is possible with some drivers, as long as they expose the vkGetInstanceProcAddr symbol).
Do Mac dylib files have something similar to SONAME that might trip up a load function somewhere, resulting in a search for libMoltenVK.dylib at all? Are you sure your replaced libvulkan.dylib in the right place, that ash really isn't accidentally poking the ICD loader library which subsequently tries to find libMoltenVK.dylib through a JSON ICD file?

@jhvst All I can think about here is the Referenced from: explicitly printing your project name and not the "equally-built" libvulkan.dylib. Could this error originate from a dlopen call or is it implying dynamic linking in the executable? Does Mac have tools like readelf to inspect the dynamic linker section? If listed in there, something else could be forcing your application to link dynamically against libMoltenVK.dylib. I don't see how ash/build.rs could do that though.


[10.4 stable] Backport updated publish.yml

Once https://github.com/lf-edge/eve/pull/3539 is merged and tested in workflow I'll backport both patches, but if it's urgent I can do it right now @eriknordmark


Refactoring new_branch_name function in branch_namer

Added a separate method name branch_version_suffix which basically gets the version related suffix for the branch name. We need to use this function in our own service hence created a separate method for this. @feelepxyz @jurre Does adding this refactoring in dependabot-core make sense?

Yeah this makes sense to me, but currently that method is private, is that intended?

Yeah this makes sense to me, but currently that method is private, is that intended?

Yes i wanted to keep that as private

Yes i wanted to keep that as private

You mentioned you wanted to use it in your service, are you planning to use send to access it?

Yes i wanted to keep that as private

You mentioned you wanted to use it in your service, are you planning to use send to access it?

We actually have a custom implementation of this branch_namer which inherits from this class i.e Dependabot::PullRequestCreator::BranchNamer so i would be using super() call in our custom implementation

Trying to figure out what's up with the CI failures, they're not related to the changes here

Trying to figure out what's up with the CI failures, they're not related to the changes here

yeah cool. We are also getting these failures while downloading rubygems in our CI pipeline

Ah, seems like they're having some issues: https://status.rubygems.org/ caused by https://dnsimple.statuspage.io/

@jurre Can we merge this PR?


Backport wamr release 23.11

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.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

[x] Fits CONTRIBUTING.md.

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

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

1 package built:

wamr

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

1 package built:

wamr

Result of nixpkgs-review pr 274072 run on aarch64-darwin 1

1 package built:

wamr

FYI, adding the label backport release-23.11 to #273129 would've caused a bot to create this PR for you.

FYI, adding the label backport release-23.11 to https://github.com/NixOS/nixpkgs/pull/273129 would've caused a bot to create this PR for you. This label can be added after merge

Thanks for the suggestion. Unfortunately in this case, the backport is not clean due to a missing intermediate backport.

Ah, while diving through the git blame i didn't see that the final PR i arrived at didn't have the darwin fix.

Is there something I am missing? :)

No really. I think you are right in general about the breaking changes notice. However, we are only providing the iwasm executable in our derivation and the only change relative to it guest::host when mapping files and directories, which is the expected behavior on other WebAssembly CLI runtimes (e.g. wasmtime or wasmedge).
However I understand that we should not backport if acting by the book. I have no strong reasons to backport either at this time other than "keep the latest release on release-23.11", so I'm good closing this PR.

We should perhaps revisit this backport given CVE-2023-52284.
If there are compatibility problems with backporting the bump, the patch, https://github.com/bytecodealliance/wasm-micro-runtime/commit/6382162711a9cfaedc63de9065b0bc6eda8123f3 does apply (after some hacking due to sourceRoot-relativeness) and build. Though it would be nice to have some tests on this package if we were going to go down that route.

Thank you for the heads up @risicle. Reopened.

Hi, sorry, I won't be able to find time for this any time soon.
Hope you can find someone else who looks into this.

All things considered, I'd go ahead and merge this backport. We are not installing headers on this derivation, so breaking changes related to the API are a non-issue for us.


[Bug] small issue in the vertical navbar link css

Current Behavior
in the vertical navbar the 4th doc link [RECENT DISCUSSIONS ON NIGHTHAWK] goes out of the navbar

Expected Behavior

Environment

Host OS: Mac Linux Windows
Platform: Docker or Kubernetes
Meshery Server Version: stable-v
Meshery Client Version: stable-v

Contributor Resources

See the Contributing to getnighthawk.dev Website
See the Project Site section of GetNighthawk project doc
🎨 Wireframes and designs for Nighthawk Site in Figma
Meshery UI Nighthawk (Performance) in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

@Savio629 @leecalcote  can u assign this issue to me 👀

Thank you, gents. 👏

@omesh-omg I checked in edge and chrome browser, but won't able to reproduce the issue!

It happens when reduce the size of window like for responsive @abhijeetgauravm

Width of the window
When reduced like for ipad maybe

It happens when reduce the size of window like for responsive @abhijeetgauravm

Yeah I checked at different screen width then i reproduced the issue. Thanks for raising it @omesh-omg !

Thanks for mentioning I thought it's useless 😅


Improve Complexity of Board Class Methods

There is room for improvement in the Board class, since it was originally developed with the mantra of "get it working". Some ideas/weaknesses:

[x]  pegs_left method: currently iterates through every hole in the board. It shouldn't have to do this. All operations related to the board should be contained within methods of the Board class, so we should be able to maintain an attribute for the number of pegs left on the board. We would decrement it with each move. First, need to move the make_jump method from the Game class into the Board class. Also, make sure that the save_state method in Game accurately saves this new attribute.
[x] locate_peg method: could I use a dictionary to map each peg letter to its current location? Should be able to, yes. The only changes are to 2 pegs on each move (jumper and jumpee), and when the first peg is removed to start the game.
[x] remove method: would be more efficient if we have a dictionary mapping pegs to locations (O(1))
[x] get_moves method: currently iterates through each spot in the board looking for holes. Could we store as an attribute a list of all locations with holes? May not be any better...how would we remove a hole from the list? Maybe using a set...that would provide O(1) add and remove...OK, that actually works.

Completed review and improvements of board class


Don't compile with native flag

Fixes #22

@kxepal @janl I'm really sorry but it looks like all my access to the various Boundary repos I maintain have been revoked unexpectedly. I don't work there any more but previously had access to merge PRs and etc. My only suggestion is to fork this repo and make your changes. Sorry :crying_cat_face:

@joewilliams Do you know anyone to ping who has a write bit? Certainly, we can fix that in our fork, but we're not the only users of bear (indirectly via folsom) so it's good to have problem fixed in upstream for everyone (:

@dominicphillips is the only public Boundary contributor, maybe he can help.

Folsom has moved, please resubmit your PR at https://github.com/folsom-project Thanks!

@joewilliams Thanks!


Backup / Restore

Hello
I start using Mistserver on my vod platform, and I have some question about usage ( at this moment I use open source version ), My first question is: how I can get backup from my data ( stream data ) and restore it

Hello Voltan,
It depends on what you mean exactly. If you mean a backup of the server itself all configurations are saved in the MistServer configuration file (either the json.conf in the same folder as MistServer or available at /etc/mistserver.conf ).
If you mean a backup of your actual streams, MistServer doesn't do library management, it just uses what's given. So if you want to backup the streams you will have to do that manually or with another application.
I hope that answers your question, if not could you please word it differently?

Hello, Rokamun
thank you very much for your information, then the best way is take a full backup from VM, just can you please tell me, does mistserver usa any database to save stream information? I can get a backup from that database.
I will get backup from config files too

MistServer does not do any database handling unless you've included it through our Trigger system, which requires a MistServer Pro. So if you're looking to backup an Open Source MistServer from a VM the only thing you really need is the configuration file, it contains everything you've set up in MistServer as the Open Source version of MistServer just saves the location of the VoD files used in the configuration file.
MistServer does generate a small .dtsh file for every VoD file that's been included in MistServer (it will be placed next to the VoD file), but these are generated pretty quickly the first time a file gets used by MistServer so there's no need to pay them any mind. Though we have seen that it can take some time if your VoD files get larger than ~ 5GB.
The .dtsh file includes header information to allow consistent use of more advanced media playback features over all protocols.

Thank you very much for your information. this information helps me to manage the service very well.
I published my video system as open-source and I support stream by mistserver to my system. this information helps me to improve my codes, I will write some api calls next week


Singleton link

I like the introduction of singletons - it definitely fills a hole. It would be nice to have a field for referencing singletons like you can with a collection link, though I understand why this might be more complicated. Is it planned?

I have multiple Laravel projects that would benefit from being able to link singletons to each other or to a collection item.
A singletonLink field would really open up the possibilities of cockpit as a cms.

That would be interesting for sure, for now, and since I have the same need I created a simple singleton select link that can permit to link to a singleton and have the singleton data in the field on api requests - https://github.com/pauloamgomes/CockpitCMS-Helpers#singleton-select-field


The terraform-provider-aiven plugin returns error after calling terraform validate

Hello,
We encountered the following error after calling the terraform validate command. We are using the master branch, we tried also the latest released version(1.0.16), terraform version(0.12.8), go version(1.12.8):
Failed to instantiate provider "aiven" to obtain schema: Unrecognized remote plugin message: [ENVY] unable to load env file(s) on init: open .env: no such file or directory

This usually means that the plugin is either invalid or simply
needs to be recompiled to support the latest protocol.

We are doing the steps from here: https://help.aiven.io/en/articles/2370350-aiven-terraform-integration.
Also, I would like to mention that during debug, I discovered that the git.apache.org is down(https://github.com/hashicorp/terraform/issues/22664) and tried also with you branch: mte-replace-git-apache-org, but the same error still appears when I run only the plugin binary:
/go/src/github.com/aiven/terraform-provider-aiven # ./terraform-provider-aiven
[ENVY] unable to load env file(s) on init: open .env: no such file or directory
This binary is a plugin. These are not meant to be executed directly.
Please execute the program that consumes these plugins, which will
load any plugins automatically

Thank you!

I installed go 1.13 and after that the terraform command worked again, but still will not work with the latest terraform docker image because the apk installs the go 1.12.8 version.

I've not run into this issue before could you try the prebuilt binaries in release 1.0.16? Aiven code does not use the envy library directly but some of the libraries we do utilize seem to.
The branch you tried was an experiment and actually a better way to circumvent the issues caused by git.apache.org being down is to use public goproxy (export GOPROXY=https://proxy.golang.org).

Ok, Thank you!


RabbitMQ monitoring automation (#1487)

Enable RabbitMQ's plugin for Prometheus metrics exposure

Add target for Prometheus to be able to scrape metrics from rabbitmq nodes

Download Grafana dashboard for displaying scraped metrics from RabbitMQ

@sbbroot network security groups have yet to be updated:
https://github.com/epiphany-platform/epiphany/blob/develop/schema/aws/defaults/infrastructure/virtual-machine.yml
https://github.com/epiphany-platform/epiphany/blob/develop/schema/azure/defaults/infrastructure/virtual-machine.yml

/azp run

/azp run

/azp run


Cached Table meta data

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.31.2(94)
Regression From
No response
Architecture
x64
Storage Explorer Build Number
10231003.6
Platform
All
OS Version
Windows 10
Bug Description
Azure table meta data is cached somewhere.
Steps to Reproduce

Create an Azure table.
Load that table with data such that it will add columns as needed.
Delete the table.
Wait a minute or so for the table to REALLY get deleted.
Re-create the table using the same name.

Actual Experience
I see a table with the same metadata columns as the one I just deleted, instead of a clean, empty table with no columns
Expected Experience
I expect to get a clean, empty table with only the key columns, no custom columns that I added in the old one. What I SEE is a table with columns the same as the one I just deleted.
Additional Context
A co-worker with access to the same table sees a table with NO columns. That list of columns is ONLY seen by me, indicating that its metadata is cached somewhere, either in the tool or on my machine.

If you right-click any table column header and select Reset Columns, does that help?

@ToddChitt yes, the columns are saved between sessions on a per table basis. That way, as you rearrange, hide, and resize columns, even after restarting the Storage Explorer, all those customizations are restored.
In your re-created table though, once you've imported some data/added some entities, you should be able to right click on any of the column headers and select "Reset Columns" to clear out the columns that existed for entities in the old version of the table but not in entities in the new version of the table.

If the table is empty, as it is when I first create it, I cannot do a
'Reset Column'. There must first be data in it. Yes, I can manually insert
a row, Reset the Columns, then Delete that row. But that's 'defect support
system'.
Question:
WHY was it decided that when creating a table of the same name as one just
deleted it should pick up cached columns? For me, the REASON I'm dropping
that table is because the columns are all messed up and I can't simply
delete a column that contains data. I want a clean, fresh table. The fact
that my co-worker see a clean, fresh table and I do not is both confusing
and frustrating.
-Todd
On Thu, Oct 19, 2023 at 5:57 PM Craig Alvord @.***>
wrote:

If you right-click any table column header and select Reset Columns, does
that help?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/AzureStorageExplorer/issues/7427#issuecomment-1771761969,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVZOQHBNYLIXCDVUG36QW3YAGO5BAVCNFSM6AAAAAA6GDW4FKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZRG43DCOJWHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

@ToddChitt We understand, and we agree that it can be confusing.
Storage Explorer stores data about a table's columns using the table's URL as a key. When you create a table with the same name as a table that was just deleted, they will have the same URL. Unfortunately, there really is no way to distinguish either table from each other. It was not expected that creating a table with the same name as a deleted one would be a common occurrence.
A fix for this should be fairly straightforward. We just need to delete the column data with the table.

@ToddChitt
First, just to make sure it is clear, the columns you see in Storage Explorer in the new version of the table are not "real". They are simply there as a display convenience. Ultimately, the columns in a table are defined by the properties that exist on the entities in the table. So if there's any worry about Storage Explorer creating columns that don't exist when you open the old table, understand that there is no need to worry that.

WHY was it decided that when creating a table of the same name as one just deleted it should pick up cached columns?

If the table is deleted outside of Storage Explorer, there isn't a way for Storage Explorer to know that. As far as it knows, it is the same table but now with 0 entities. And if that is the case, then it is very much convenient if Storage Explorer maintains the column customizations, so once new entities are added, those customizations are still active.
If the table was deleted in Storage Explorer, then sure, that is a fair ask, and I've open a feature request for that here. However, given this is the first time we've heard of this confusion popping up I cannot promise that we will address it. I can see other situations in which someone deletes a table and then recreate it, but would want to still have their customized columns restored. We'll need to talk more about it as a team. Feel free to follow the feature request for updates.

There must first be data in it.

Yes, perhaps there should be a way to reset columns in the absence of entities. I've created a feature request for that here. Again, we'll discuss this more as a team, and you can follow it for updates.

Going to close this now given the lack of further response. Please feel free to share further feedback/thoughts on either of the two linked issues (if relevant to them) or in a new issue. Thanks.


ndimage uniform_filter expands NaNs all the way to the edge of the array

The ndimage routines uniform_filter and uniform_filter1d both expand NaNs out to the far edge of the original data, regardless of the size of the filter being used.  I expected uniform_filter to behave similarly to convolve with a uniform kernel of the same size - namely that anywhere the kernel touches a NaN the result is also a NaN. However, the NaNs continue to corrupt the data from the moment a NaN is hit.
Reproducing code example:
import numpy as np
from scipy.ndimage import uniform_filter1d, convolve1d

x, y = np.mgrid[-2*np.pi:2*np.pi:51j, -2*np.pi:2*np.pi:51j]
z = np.sin(x)*np.sin(y) + np.random.randn(51, 51)/5

klen = 5
kernel = np.ones(klen)

z[20, 30] = np.nan

z1 = convolve1d(z, kernel)
z2 = uniform_filter1d(z, klen)

import matplotlib.pyplot as plt

fig, axs = plt.subplots(ncols=3, figsize=(9,3))
axs[0].imshow(z)
axs[0].set_title('Data')
axs[1].imshow(z1)
axs[1].set_title('Convolve')
axs[2].imshow(z2)
axs[2].set_title('Uniform Filter')

plt.show()

There is no error, but as you can see the right-most plot has a swath of NaNs out where the data were perfectly good.  Using uniform_filter in both dimensions wipes out the entire lower right corner.

Scipy/Numpy/Python version information:
0.19.1 1.12.1 sys.version_info(major=3, minor=5, micro=3, releaselevel='final', serial=0)

I have been hitting this for awhile, but for some reason thought there was a bug report and fix in the works ... however, if it exists I can't find it so here it is.

Hi all, is there any plans for movement on this, or has it been back-burnered? The bug still exists as of today in Scipy 1.1.0. (Numpy 1.14.2, Python 3.6)

The uniform filter in ndimage is computed with a running sum in a "sliding window".  When the window moves by one index, the newly visible data value is added to the sum, and the value that left the window is subtracted from the sum.  The net effect is (almost) the same as a typical convolution calculation, but it is more efficient.   The problem is that once a nan enters the running sum, the sum remains nan even after the nan in the input array has left the window.
Technically (weaselly?), this is not a bug--the filters in ndimage make no promises about how nan is to be handled (see also https://github.com/scipy/scipy/issues/9077).  For now, if you want the nan handling described by @Jeitan above, you'll have to explicitly convolve with a uniform kernel instead of using uniform_filter or uniform_filter1d.

@Jeitan I guess the implied sense of guilt in my parenthetical use of weaselly didn't come across the way I intended 😳.   I think most folks will agree that, regardless of what the documentation does or does not promise, it is surprising that the uniform filters don't handle nan they way the convolution functions do.  I wouldn't object to making the behavior consistent with convolution.  I hope it can be done without too much impact on the performance of the filters.
Be aware that there is a lot of code in scipy that was written without nan in mind.  For example, over in scipy.stats, the addition of the nan_policy argument in many functions--which makes explicit how nan is to be handled--is a relatively recent enhancement.   I don't think anyone has worked on or even considered a similar enhancement for the filters in scipy.ndimage.  If you are using a function that doesn't explicitly document how nan is handled, you should test that it does what you expect.

Hi all,
I just ran into this issue myself, and I tracked down what I think is the issue.
https://github.com/scipy/scipy/blob/8dba340293fe20e62e173bdf2c10ae208286692f/scipy/ndimage/src/ni_filters.c#L346-L356
tmp behaves like an accumulation shift register. When a new value is added, an old value a fix window behind the new value is subtracted. If a NaN is added to the accumulation value, it sets the value of the register to NaN. Once this occurs, adding or subtracting will not change the register value resulting in NaN for the rest of the input.
To fix this issue, we would need to do the accumulation over the entire window at every step.
Alternatively, we could monitor for NaNs and reset the accumulation register by summing over the window when the NaN leaves the window.
Either way, this will be a performance hit, so what are your opinions?
I have never made a pull request, so I might need some help.
Thanks!
-Tom

I just did a workaround, not yet a pull request, and I'm posting this message in case it's of any interest for scipy.
Instead of writing a new C function, or costly convolution, I did the following:

take a given array, let's call it image
create a mask to replace all non finite values with 0, this way, no more propagation elsewhere with the running sum (but of course, the mean is not correct where we replaced nan with 0

tmp_image = image.copy()
selection = ~np.isfinite(image)
tmp_image[selection] = 0

create a fake array, with 1 everywhere, and 0 where there were non finite values in the original array

# We will get the total sum of valid pixel in a given window with that
nan_mask = np.ones_like(image)
nan_mask[selection] = 0

compute the uniform filter on it, then multiply by b_elem=win_x*win_y, this gives me the number of valid pixels in every slide window

n_elem = np.prod(win_size)
nb_valid_pixels = _move_mean(nan_mask, win_size) * n_elem

do a selection once more to change the mean where it's needed:

win_selection = nb_valid_pixels != n_elem

Change the mean:

result[win_selection] *= n_elem / nb_valid_pixels[win_selection]

Of course, I assume what you will not like with my solution is the copies and increase in memory in general. Maybe the numerical stability is not ideal given that you compute the mean, to remultiply by the number of values to get the sum.
A better solution would probably be to use a uniform_sum filter (I don't think it exist and using generic_filter would be much slower)
But anyway, I wanted to share if anyone is interested.
Note: In my case, I force dtype=float internally, which is not what uniform_filter does. uniform_filter will give inaccurate means with integers.


chore: pnpm and sn-gh-workflows

If you're doing this you should take a look at #1124, that also fixes some build errors/warnings and removes redundent packages

I'd even suggest we get #1124 in first. Then you deal with the workflow change?

PTAL #1124


Some questions on running Hello Docker Compose.

Still trying to understand this:
Q) The Node app exposes port 3000, but here we map 50002 to 50002, why and what if multiple ports are exposed by service?
nodeapp:
build: ./node
ports:
- "50002:50002"
Q)  a single -app-port, what if the container exposes multiple ports, 80, 443?
nodeapp-dapr:
image: "daprio/daprd:edge"
command: ["./daprd",
"-app-id", "nodeapp",
"-app-port", "3000",
Q)
Do I need to stop the default containers that Dapr Init creates ?
Q)
Is the placement service only necessary when using actors ?
Q)
Will these services display with dashboard and zipking ?
If so how do I access them ? the default urls are not displaying the services.
thx... :-)

Same as https://github.com/dapr/samples/issues/53. Looks like we could do with having improved Compose examples.


Corrected command to list running containers

Proposed changes
I corrected the command so that now it will list the container IDs, which is what is needed to stop a container. Previously, the command would only list image IDs, which is useless in this context and confusing for new users.

Misread the document, please ignore

Deploy preview for docsdocker ready!
Built with commit 083db392637d6bfc621e8a58338be7c4989d748e
https://deploy-preview-8974--docsdocker.netlify.com


Convert project to ES2015

Gulpfile is using ES2015
Move current codebase to generators/ folder, which is then compiled to the root via Babel

f78bc60261b14f752519176596aeb0cc66d8c50a Moved codebase to lib/ and compiled to generators/ via Babel.


Better support for writing tests that rely upon the injection of promises into for the return of $q.all()

So we don't have to do:
            var deferred = $q.defer();
            deferred.resolve([VALUE1, [VALUE2]]);
            loadPromise = deferred.promise;
            loadPromise.spread = function (resolve, reject) {
                function spread(data) {
                    resolve.apply(void 0, data);
                }
                return loadPromise.then(spread, reject);
            };

resolved in #11


New histogram metric for snapshot duration

Signed-off-by: arthursens<EMAIL_ADDRESS>
What this PR does / why we need it:
This PR adds a new histogram kubevirt_snapshot_duration_seconds that provides detailed information about time spent on VMI snapshots.
This new metric can be used in the following use cases:

Detect if snapshots are failing:

delta(kubevirt_snapshot_duration_seconds_count{status=”failed”}[5m]) > 1

Detect average time spent on snapshots per status:

kubevirt_snapshot_duration_seconds_sum / kubevirt_snapshot_duration_seconds_count
rate(kubevirt_snapshot_duration_seconds_sum[5m]) / rate(kubevirt_snapshot_duration_seconds_count[5m]) #average time spent in last 5 minutes

Detect average time spent on snapshot, ignoring status:

sum without (status) (kubevirt_snapshot_duration_seconds_sum) /
sum without (status) (kubevirt_snapshot_duration_seconds_count)

Calculate the 90th percentile in the last 5 minutes taking source node, target node, and status in consideration:

histogram_quantile(0.9,
  rate(kubevirt_snapshot_duration_seconds_bucket[5m])
)

Calculate the 90th percentile in the last 5 minutes ignoring status:

histogram_quantile(0.9,
  sum without (status)(
    rate(kubevirt_snapshot_duration_seconds_bucket[5m])
  )
)

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
References used to create this PR:

How does a prometheus histogram work
Prometheus client_golang examples

Release note:

Added new histogram "kubevirt_snapshot_duration_seconds" to the virt-controller

Similar to #3887, still needs e2e tests and we need to come to a consensus regarding bucket's values

/hold
/lgtm cancel
/approve cancel
This PR was created at a time when #3887 did not introduce any API changes for configuring histogram metrics.
I will address what ashley commented but I'd like to wait until #3887 is reviewed and merged since this PR will extend the API in a similar way

@sradco @yuvalturg @rmohr what should we do about this PR - drop or keep and rebase?

@sradco @yuvalturg @rmohr what should we do about this PR - drop or keep and rebase?

/unhold

/unapprove
?


[pcf dev] Error: failed to open file: open C:\Users\xxx.pcfdev\token: Access is denied.. URGENT

Team,
I have installed brand new cf version 6.35.0+5c7de3e32.2018-03-08 (over an uninstalled previous version) of PCF DEV.
On cf dev start i am getting the attached cf dev start.jpg and in subject.
May you please assist?
It is urgent for me as I need the environment for the preparation of LFS232 exam.

ps i am executing as an administrator on windows 10.
Thank you.
Slobodanka

Additionally, if a change the locatioon of the cf cli installation ex: C:/Cloud Foundry, the same error now occurs on pcf dev start - access denied as the location under Program Files does not exist any more and the .exe would not look or ask for another location nevertheless the cli install gives you an option to modify the default installation folder?

#313 indicates that you figured this out. Is that the case?

Yes, true.
It was rather OS problem after an update - I will close the issue.
Thanks,
Slobodanka


Integration provider ID lost when updating Episodic

When editing an Episodic, the integration ID is not kept after the submission is completed.

This was fixed in 680d9fed4713b9bd99cbc60e64ca6f3a27f1129f


Add a Codacy badge to README.md

Coverage remained the same at 47.125% when pulling ac72fd6c94ebd06e73ea565592034bcd205137e3 on codacy-badger:codacy-badge into 2aa720a4414d4d47d2fbe1363a4823e834877834 on maxibanki:master.


Flush failing to call DeliveryHandler for each messsage

Description
The code below calls BeginProduce 50000 times.  If producer.Flush is not called then the DeliveryHandler is called 50000 times.
However there appears to be an issue if producer.Flush is called, which results in the DeliveryHandler being called around 49857 times.
How to reproduce
class Program
{
private static readonly List<Message<string, string>> responses = new List<Message<string, string>>();
    static void Main(string[] args)
    {
        int batchSize = 50000;

        var config = new[]
        {
            new KeyValuePair<string, string>("bootstrap.servers", "SERVERNAME:9092"),
        };

        var producer = new Producer<string, string>(config);

        var messages = Enumerable.Range(0, batchSize)
                                 .Select(ToMessage);

        foreach (var batchMessage in messages)
        {
            producer.BeginProduce("MY.TEST", batchMessage, DeliveryHandler);
        }

        producer.Flush(TimeSpan.FromSeconds(100));

        while (responses.Count != batchSize)
        {
        }
    }

    public static void DeliveryHandler(DeliveryReportResult<string, string> report)
    {
        responses.Add(report.Message);
    }

    private static Message<string, string> ToMessage(int i)
    {
        var message = new Message<string, string>();
        message.Value = "Message " + i;

        return message;
    }
}

Checklist
Please provide the following information:

[x] Confluent.Kafka nuget version: 1.0.0-beta2
[ ] Apache Kafka version: 2.12-2.1.0
[ ] Client configuration:
[ ] Operating system: Windows Server 2012 R2
[ ] Provide logs (with "debug" : "..." as necessary in configuration)
[ ] Provide broker log excerpts
[ ] Critical issue

If you call flush, your delivery handler can potentially be called on two separate threads - your application thread and the producer background poll thread. Given this, you'll need to synchronize access to the responses list and you're not. Try putting a lock around responses - pretty sure this will resolve the problem.
I've marked this as enhancement since I think it would be better if we pause the producer background thread whilst a Flush call is blocking.


Indicate modified cookies in an event

We'll need it for indicating cookies modified by $cookie rules or stealth mode:

https://github.com/AdguardTeam/AdguardForAndroid/issues/2365
https://github.com/AdguardTeam/AdguardForMac/issues/386
https://github.com/AdguardTeam/AdguardForWindows/issues/2387

core/pull-requests/935


Can i use your code in a commercial project ?

Hi,
We want to use your module for one project. Can you tell us what the license is?
Can you add a open source license
Thanks,
Xuan

Yes it can be used for commercial projects. I'll add license info in a bit. Optionally you can PR license details and I'll accept.

@tvxuan4 the MIT license record is added. Please feel free to use the library. Thank you!


Add developer guide documentation for login and logout

Note: The class diagrams should include the visibility of the attributes, as in PR comment (https://github.com/nus-cs2103-AY1819S1/addressbook-level4/pull/68)

Add sequence diagrams for login and logout


Fix clusters with single candidates.

Closes #2152.

@wetneb Can we get this merged in?  I'd like Bharath (an intern I coached over the summer, now at USC) to work on showing the busy pointer when moving Cluster sliders and the UI is not available to interact with until cluster ops finish.


How to bind json file in ng-grid in button click event

I guess you'll want to open this issue here: https://github.com/angular-ui/ng-grid

Hello Narretz i need one help  add summary row in  footer for all columns

Please direct support questions like this one to one of the support channels


Ignore watching dot files in dist/ directory

What are you trying to accomplish with this PR?
Fixes https://github.com/Shopify/slate/issues/227
Ignores watching .DS_store and any other dot files in the dist/ directory.
For maintainers:

[ ] I have :tophat:'d these changes.

I've 🎩'd these changes.  Changes in dot-files
Something to note: dot-files are being ignore in the dist/ directory, however, their presence in the src/assets directory will still trigger the build:asset task.  This only seems to effect the assets/ directory, other directories will ignore dot-files.
In the screenshot below, I experiment with adding *.txt files and .DS_Store files to the src/assets directory.  Note: The .DS_Store files are not being generated by my macOS as I have 10.12.6 installed; I'm am generating them manually with touch .DS_Store
You can see that in case 1 and 3, a new asset triggers a build and then deploy.  In case 2 and the last one, a .DS_Store file will trigger build:assets.

Note about hanging spinner
A note about the hanging spinner mentioned in the original issue:
[development]: 0 / 0 [|]  0%

This was previously raised as a bug in Themekit 0.6.12.  You can find the original issue here: https://github.com/Shopify/themekit/issues/436
It appears to be addressed in an upcoming release of ThemeKit.  Until then, this is a decent patch.

Bumping to v0.7.0 shouldn't be taken lightly though.  It introduces theme.lock for remote file checking.  Super useful, but we'll want to make sure Slate handles and displays errors correctly for this new tool.

Something to note: dot-files are being ignore in the dist/ directory, however, their presence in the src/assets directory will still trigger the build:asset task. This only seems to effect the assets/ directory, other directories will ignore dot-files.

This makes sense since the build/watch process for src/assets/ is to copy any file to dist/assets. Should we ignore all dot files in the src/assets/ folder? I think yes.


Unexpected end of JSON input with empty response

Description
When a response has an empty body, the client fails with a JSON.parse error:
SyntaxError: Unexpected end of JSON input
      at JSON.parse (<anonymous>)
      at parseJSONFromBytes (node:internal/deps/undici/undici:4747:19)
      at successSteps (node:internal/deps/undici/undici:4718:27)
      at specConsumeBody (node:internal/deps/undici/undici:4724:9)
      at HttpResponse.json (node:internal/deps/undici/undici:4614:18)

This error happens here. response[parseAs]() fails to convert the empty response to json and crashes, never reaching the fallback.
Expected result
The client should be able to handle empty responses without crashing with JSON.parse errors.
Checklist

[x] I’m willing to open a PR (see CONTRIBUTING.md)

We've had this issue as well. Generally you want your API to return 204 status if there is no body, which would make open-api skip the json parse and avoid crash.
In our case, we had PUT requests that returned empty body by design, and returned 201 status. For those requests you can add parseAs: 'stream' option and it should skip the json parse as well.
For cases where you sometimes get response, I have no idea how to resolve.


✨[Feature - Polish] Geo hide the Card tab when outside the US

Problem
A majority of our users are outside the US and the Litecoin Card is of no use to them at the moment.  This is burdening our Customer portal.
Approach

Check though FB/Geolocation and or Default locale for the user
No GPS Frameworks! (this would drain the battery)
Hide the tab if they are outside the US
Add a switch in settings to allow user to unhide

Screenshots

Current (ex. Japan)
New (ex. Japan)

Closing due to due to: https://github.com/litecoin-foundation/litewallet-ios/issues/40


Where to get the API key?

Not exactly an issue with the addon, but this requirement is very vague:
Type your API Key, API Base and API Version used following the example below and hit submit:
API Key:<PHONE_NUMBER>abcdef1234567890abcdef
API Base: https://iotlabopenai.openai.azure.com/
API Version: 2023-03-15-preview
After visiting the azure site, there seems to be so many choices on where exactly you need to go to get the key for this app. Can ya at least give a few steps or a guide somewhere to get this?

Not exactly an issue with the addon, but this requirement is very vague:
Type your API Key, API Base and API Version used following the example below and hit submit: API Key:<PHONE_NUMBER>abcdef1234567890abcdef API Base: https://iotlabopenai.openai.azure.com/ API Version: 2023-03-15-preview
After visiting the azure site, there seems to be so many choices on where exactly you need to go to get the key for this app. Can ya at least give a few steps or a guide somewhere to get this?

I wouldn't bother. I created a free plan with Azure cloud services and search for OpenAI and found "Azure OpenAI", created a new resource and it specifically tells you that it's not open to private individuals. You have to request access and personal email will not work. Basically, this is currently locked behind a paywall for business use right now.


Authentication: How would I getClient with Clerk?

I'm only interested in the next js app folder approach at this point.
With Clerk for next.js app, I can get a token from a page, but I can't see how to get it from the environment like you do with the Next Auth approach https://github.com/apollographql/apollo-client-nextjs/issues/44
So I'd think I'd need to pass the token in with the getClient/registerApolloClient call in the examples,
export const { getClient } = registerApolloClient(() => {
but I don't see how to do that because the registerApolloClient doesn't take args.
Could you share how to get the client for an Clerk token?
I'm also curious -- where you've said in other posts that registerApolloClient is a singleton, is that also the case on the server only app folder approach?  And if so, how does that work in a multi-user situation where each user has different creds?

ah -- just saw this: https://github.com/apollographql/apollo-client-nextjs/issues/103

registerApolloClient is a singleton

Yes, but per request. Your registerApolloClient makeClient function will be executed once per request, so you can also just call something like cookies in there - you just never ever ever should access the cookies outside of makeClient and pass them in via scope or parameters, since then they'll be shared across all requests.

Great, thanks for explaining. I'd appreciate some help here then.
To be totally clear: If you're saying that I shouldn't pass in the cookie outside of makeClient, am I correct in reading that I should also not be setting the cookie with the new:   client.defaultContext.token = newToken; in the calling page because there's a possibility that between the setting and the makeClient, it could be set or over-written by another unrelated request?
If that's the case, then I'm also not clear how to get the jwt from inside of makeClient because I don't see how I can have access to any page, context or cookies there.

wonderful.  Thanks for clarifying how it should be organized. I'm in good shape now.

@phryneas
Wondering if this looks fine to you (seems like it works for me), specifically for RSC. Thanks in advance!
import { ApolloClient, HttpLink, InMemoryCache } from '@apollo/client';
import { setContext } from '@apollo/client/link/context';
import { registerApolloClient } from '@apollo/experimental-nextjs-app-support/rsc';
import { fetchAuthSession } from 'aws-amplify/auth/server';
import { cookies } from 'next/headers';

import { env } from '~/_env';
import { runWithAmplifyServerContext } from '~/authentication/utils/serverSideAuth';

const authTokenLink = setContext(async () => {
  // Could be any async way of fetching the token, but this is with Amplify
  const session = await runWithAmplifyServerContext({
    nextServerContext: { cookies },
    operation: (contextSpec) => fetchAuthSession(contextSpec),
  });

  if (session.tokens?.idToken) {
    return {
      headers: {
        Authorization: session.tokens?.idToken?.toString(),
      },
    };
  }

  return {
    headers: {
      'X-Api-Key': env.NEXT_PUBLIC_AWS_APPSYNC_API_KEY,
    },
  };
});

export const { getClient } = registerApolloClient(() => {
  return new ApolloClient({
    cache: new InMemoryCache(),
    link: authTokenLink.concat(
      new HttpLink({
        uri: env.NEXT_PUBLIC_AWS_APPSYNC_GRAPHQL_ENDPOINT,
      }),
    ),
  });
});

@shawngustaw that looks good to me, but I have to admit that I am not familiar with your specific way of fetching the token. Glad to hear it works 😊


tests: kernel: xip: set CONFIG_LOG_MODE_IMMEDIATE

Workaround for https://github.com/zephyrproject-rtos/zephyr/issues/62975

qemu_riscv32_xip seems to have some data alignment issue and for some reason CONFIG_LOG_MODE_IMMEDIATE is working around the problem and make the test go again. Set that while the actual problem is investigated.

Funny, now this makes west build -t run -p -b qemu_arc_em -T tests/kernel/xip/arch.common.xip.minimallibc report a fail even if the test passes

Too random, nvm.


Weighted error minimizer

Hi François
This extends the error minimizer to use weighted least squares minimization. It requires an appropriate OutlierFilter that outputs weights in [0, 1]. With {0, 1} weights, the behavior is unchanged.

runtime performance similar as before
unit test passing
based on: http://igl.ethz.ch/projects/ARAP/svd_rot.pdf (not umeyama 91, this is for the scaling)

Great contribution. Thanks for looking into that and cleaning the old code.


warn: sma-em.0 (9509) State "sma-em.0.3009872298.pregard" has no existing object

Describe the bug
When the adapter is started, numerous warnings like:
"warn: sma-em.0 (9509) State "sma-em.0.3009872298.pregard" has no existing object, this might lead to an error in future versions"
appear in the iobroker log (see attached logfile).
To Reproduce
The warnings appear at each startup of the adapter. The js-controller version needs to be 3.2.x; the warnings do not appear with js-controller 3.1.x
Expected behavior
No warnings should appear, regardless of js-controller version.
It seems that a setState is done for non-existant states. A create state or extendObjectNotExists seems to be missing.
Screenshots & Logfiles
sma-em-063-errlog.txt
Versions:
Adapter version: <0.6.3>
JS-Controller version: <3.2.16>
Node version: <v12.21.0>
Operating system:
Additional context
None

I've pushed a new version for testing in my repository.

@TGuybrush
I have reinstalled your new version in my environment and it starts just fine. The warnings are gone.

Thank you for fixing this. I think it is not necessary to have others test it before the next official release. So please include your fixes in your next PR for version 0.6.4
Will you be able to implement #142 soon so that we can include it in 0.6.4? Or should we better release 0.6.4 with the fixes you implemented up to now and make a new release for #142 ? What do you think?

I'm working on #142 but it will take same time. I propose to release the new version now since it's better to do incremental  improvements to detect potential problems.

@TGuybrush
I agree. I think your proposal makes much sense. Will you then provide a PR for a version 0.6.4 with your latest fixes, please. I will take care that it will be released and communicate it on the beta test thread in the forum so that we hopefully get some additional test feedback.

v0.6.4 has been released to stable. The problem is fixed with this version.


ChoiceFactory.ToChoices not prompting with choices

When I use the above sample as a guide and use this snippet:
await dialogContext.PromptAsync(
                                SomeDialogId,
                                new PromptOptions
                                {
                                    Prompt = MessageFactory.Text("Was this what you were looking for?"),
                                    RetryPrompt = MessageFactory.Text("Sorry, please choose from the options."),
                                    Choices = ChoiceFactory.ToChoices(new List<string> { "Yes", "No" }),
                                },
                                cancellationToken);

I don't see any prompt options showing up in either the emulator or the webchat. I couldn't find any samples on the Samples repo either. Curious if I am doing something wrong? Or if it is not supported and the document needs to be updated.

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

ID: 72538432-fde5-22ab-7997-8a34b11aa24e
Version Independent ID: c469fa5b-2590-ee16-1491-51b90c069204
Content: Gather user input using a dialog prompt - Bot Service
Content Source: articles/v4sdk/bot-builder-prompts.md
Service: bot-service
Product: bot-framework
GitHub Login: @JonathanFingold
Microsoft Alias: v-jofing

@ovishesh The prompt classes are designed to be used within a dialog step. While they can be used directly form the bot's turn handler, that's more complicated and not a supported scenario.
Note that we have a ConfirmationPrompt class that specifically designed for confirmation prompts. It handles variations of the terms "yes" and "no". The ConfirmationPrompt would ignore the Choices parameter.
Assuming your code snippet is from within a dialog step, you appear to be missing the return before the await. It should look something like:
private async Task<DialogTurnResult> PromptForLocationAsync(WaterfallStepContext stepContext, CancellationToken cancellationToken)
{
    // Handle previous input here...

    return await stepContext.PromptAsync(
        "locationPrompt",
        new PromptOptions
        {
            Prompt = MessageFactory.Text("Was this what you were looking for?"),
            RetryPrompt = MessageFactory.Text("Sorry, please choose from the options."),
            Choices = ChoiceFactory.ToChoices(new List<string> { "Yes", "No" }),
        },
        cancellationToken);
}

Closing due to inactivity. @ovishesh , please reopen if your are still having this issue.


rttr::type::get_by_name() is not thread safe

Consider an example when a thread asks for a type using templated rttr::type::get  call which can auto-register a type, at a same time when another thread uses rttr::type::get_by_name call without any mutex lock in the implementation which leads to the corrupt memory access.

Since I already have a working fix for this in my own repository, I will paste it there if it can help.
Add this to public section of type_register_private class (rttr/src/rttr/detail/type/type_register_p.h):
`type get_by_custom_name(string_view custom_name);
Implement it in the rttr/src/rttr/detail/type/type_register.cpp:
type type_register_private::get_by_custom_name(string_view custom_name)
{
	std::lock_guard<std::mutex> lock(m_mutex);

	auto ret = m_custom_name_to_id.find(custom_name);
	if (ret != m_custom_name_to_id.end())
	{
		return (*ret);
	}

	return get_invalid_type();
}

Modify the type::get_by_name in rttr/src/rttr/type.cpp to be like this:
type type::get_by_name(string_view name) RTTR_NOEXCEPT
{
	return detail::type_register_private::get_instance().get_by_custom_name(name);
}

That's it.

How are you using the registration process? Are you using it with a plugin system?
Because ideally every type should be registered when you enter the main() function

Types can be requested by name at any time, from any thread. For example, when deserialization of object from a binary stream is performed in background thread during load of the game scene. There is no guarantee that every single possible type is registered at the moment of type request. To achieve that, you have to write a plenty of manual registration code in main(), including types of containers, built-in types, const and non-const and so on.


Cleanup Identity

copy the whole shared project over verbatim from FSH.WebApi
rename authentication namespace to auth
update changes from api
add musthavepermission and use it everywhere

This one goes together with https://github.com/fullstackhero/dotnet-webapi-boilerplate/pull/443

is this with a view to making it easier to share between the 2 solutions?

is this with a view to making it easier to share between the 2 solutions?

I'm not sure I understand your question.
It's mainly to make things more consistent overall. And the permissions got an overhaul that should make them easier to manage.


Harbor should lock out user for at least 15 minutes after 5 invalid password attempts

According to Security testing criteria,
Accounts should lock out for at least 15 minutes after 5 invalid password attempts
But Harbor don't lock out user for any failed password attempts

We have a lock mechanism, if a login failure happens the user can't be logged in 5 seconds, so brutal force won't work.  This is also suggested in the guide line.

Also for VIC 1.2， given we're moving to LW for user/credentials management, I'm leaving this issue as open just for tracking.

this is covered here https://github.com/goharbor/harbor/issues/1572


Expanding the clickable area

The TabBar Buttons can only be clicked by the center images.
Sometimes it takes many invalid touches when trying to click the button.
How to expand the clickable area？

I had the same problem where it sometimes took several clicks for the bar buttons to register. I fixed this by going into YALFoldingTabBar.m and under the barItemDidTipped: method and commenting out the section that returns if self.isAnimated is true.
This made the buttons much more responsive for me since I did not have to wait for the animations to finish before they were interactive.
Hope this helps

This works perfectly. Thanks for the tip.

Is there a reason that these lines are included? (i.e. not just deleted in the master branch?) And if they are important, why can't it only return until the animation is mostly not done, but once its mostly done, i.e. 70% done, that it will no longer return?

Flowers to @cminicus ! :bouquet:

Please fix this issue as @cminicus  pointed out, I don't find anything useful in keeping check on completion of animation of buttons.


workspace::NewTerminal opens two terminal instead of one

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it
In keymap and command palette, workspace::NewTerminal opens two terminal instead of one. The default key binding ctrl-` is fine though.
Environment
Zed: v0.96.4 (stable)
OS: macOS 13.4.1
Memory: 16 GiB
Architecture: aarch64
If applicable, add mockups / screenshots to help explain present your vision of the feature
No response
If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
No response

Ah yeah, its happening because opening the terminal dock, when no terminals exist, automatically adds one, and then the action is adding another. Nice catch @networkhermit.


Glendale Arizona Address Data

I found this data awhile back. Didn't have a github account to submit it. Now i do. I have no clue what the license is but i'm pretty sure it's open.
Here's the link http://gis.glendaleaz.com/gis/

Thanks for tracking this down! We've got this source in our list, we just need to add a conform object to it so it'll show up in our data outputs.

I apologise for that, had no clue. Uploaded this issue at work from my phone so i wasn't able to check the prior sources beforehand. Thanks for taking care of the issue right away.

No need to apologize! Thank you for pointing out the data. Keep looking for other data sets :smile:.


Documented Getting Started Guide for Shark Release

This issue is in preparation to kick off QA to complete by 11/15/2024. The three workflows that need to be documented are the following (assumes user has already done basic installation which has already been documented):

sharktank: User wants to use custom weights to run an optimized punet based model. User downloads weights and uses sharktank to export MLIR. Then user compiles that MLIR with iree-compile and runs it with the iree run module. Existing documentation is here for Punet and IREE compile/run here
SDXL with Shortfin: User is interested in leveraging our high performance inference engine to run SDXL. User installs additional dependencies for SDXL, ensures it's installed correctly, starts a server, and runs the client via an example json file. Point to API to show how the json can be changed to use SDXL. Existing documentation is here and API for the server is still in code here
Llama with SGLang: User is interested in leveraging our high performance inference engine to run Llama (but interacts with it through SGLang since the user interface in Shortfin is still fairly small compared to SGLang).

Focus for today

[ ] Test out non-SGLang workflows
[ ] Send out initial PR

Wasn't able to get SDXL workflow to work e2e.
What did work:

Following the nightly release installation guide I was able to install sharktank as well as shortfin from the last nightly release (from 11/7/2024) in a Python3.11 environment created with pyenv. This included basic tests loading up shortfin and sharktank modules.
Installing deps from the shortfin apps page worked.

First issue:

python -m shortfin_apps.sd.server --help

Traceback (most recent call last):
  File "<frozen runpy>", line 189, in _run_module_as_main
  File "<frozen runpy>", line 159, in _get_module_details
  File "<frozen importlib._bootstrap_external>", line 1074, in get_code
  File "<frozen importlib._bootstrap_external>", line 1004, in source_to_code
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/sosa/SHARK-platform-v1-workspace/3.11.venv/lib/python3.11/site-packages/shortfin_apps/sd/server.py", line 154
    f"--iree-compile-extra-args={" ".join(ireec_args)}",
                                                       ^
SyntaxError: f-string: expecting '}'

Didn't work. Issue was a bug that @stellaraccident addressed here
Second issue pytest instructions didn't work. In hindsight this makes sense as pytest is path dependent and with a release you are running without a checkout
Third issue trying to work around the first issue without a new nightly release. Tried to run pip install . from shortfin instructions. Got error
  /tmp/pip-build-env-svnqb1lj/overlay/lib/python3.11/site-packages/setuptools/dist.py:337: InformationOnly: Normalizing '2.9.0.dev' to '2.9.0.dev0'
        self.metadata.version = self._normalize_version(self.metadata.version)
      -- The C compiler identification is unknown
      -- The CXX compiler identification is unknown
      CMake Error at CMakeLists.txt:21 (project):
        No CMAKE_C_COMPILER could be found.

        Tell CMake where to find the compiler by setting either the environment
        variable "CC" or the CMake cache entry CMAKE_C_COMPILER to the full path to
        the compiler, or to the compiler name if it is in the PATH.

      CMake Error at CMakeLists.txt:21 (project):
        No CMAKE_CXX_COMPILER could be found.

        Tell CMake where to find the compiler by setting either the environment
        variable "CXX" or the CMake cache entry CMAKE_CXX_COMPILER to the full path
        to the compiler, or to the compiler name if it is in the PATH.

      -- Configuring incomplete, errors occurred!
      Traceback (most recent call last):
        File "<string>", line 261, in run
        File "<string>", line 277, in build_default_configuration
        File "<string>", line 233, in build_cmake_configuration
        File "/usr/lib/python3.11/subprocess.py", line 413, in check_call
          raise CalledProcessError(retcode, cmd)
      subprocess.CalledProcessError: Command '['cmake', PosixPath('/home/sosa/SHARK-Platform/shortfin'), '-GNinja', '-Wno-dev', '--log-level=VERBOSE', '-DSHORTFIN_BUNDLE_DEPS=ON', '-DCMAKE_BUILD_TYPE=Release', '-DSHORTFIN_BUILD_PYTHON_BINDINGS=ON', '-DPython3_EXECUTABLE=/home/sosa/SHARK-platform-v1-workspace/3.11.venv/bin/python', '-DSHORTFIN_ENABLE_LTO=ON']' returned non-zero exit status 1.
      *** Tracy instrumentation not enabled (enable with SHORTFIN_ENABLE_TRACING=ON)
      setup.py running in cmake build mode:
        SOURCE_DIR = /home/sosa/SHARK-Platform/shortfin
        BINARY_DIR = /home/sosa/SHARK-Platform/shortfin/build
      version_info_rc.json not found. Default to dev build
      Using PACKAGE_VERSION: '2.9.0.dev'
      Found shortfin packages: ['shortfin_apps', '_shortfin', 'shortfin', 'shortfin_apps.llm', 'shortfin_apps.sd', 'shortfin_apps.llm.components', 'shortfin_apps.sd.components', 'shortfin_apps.sd.examples', 'shortfin.array', 'shortfin.interop', 'shortfin.support', 'shortfin.interop.fastapi', 'shortfin.interop.support']
        *********************************
        * Building base shortfin        *
        *********************************
      Removing CMakeCache.txt because Python version changed
      CMake build dir: /home/sosa/SHARK-Platform/shortfin/build/cmake/default
      Configuring with: ['-GNinja', '-Wno-dev', '--log-level=VERBOSE', '-DSHORTFIN_BUNDLE_DEPS=ON', '-DCMAKE_BUILD_TYPE=Release', '-DSHORTFIN_BUILD_PYTHON_BINDINGS=ON', '-DPython3_EXECUTABLE=/home/sosa/SHARK-platform-v1-workspace/3.11.venv/bin/python', '-DSHORTFIN_ENABLE_LTO=ON']
      Native build failed:
      [end of output]

Fourth issue assumed issue was with not having the right setup so followed dev instructions to run dev_me.py. Ran into an issue running CMake
First time configure...
Building
no such file or directory
CMake Error: Generator: execution of make failed. Make command was: /tmp/pip-build-env-mii6rgj2/overlay/bin/ninja
Traceback (most recent call last):
  File "/home/sosa/SHARK-Platform/shortfin/./dev_me.py", line 262, in <module>
    main(sys.argv[1:])
  File "/home/sosa/SHARK-Platform/shortfin/./dev_me.py", line 212, in main
    build_mode(env_info)
  File "/home/sosa/SHARK-Platform/shortfin/./dev_me.py", line 258, in build_mode
    subprocess.check_call([env_info.cmake_exe, "--build", str(build_dir)])
  File "/usr/lib/python3.11/subprocess.py", line 413, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/home/sosa/SHARK-platform-v1-workspace/3.11.venv/bin/cmake', '--build', '/home/sosa/SHARK-Platform/shortfin/build/cmake/default']' returned non-zero exit status 1.

At this point, I think my environment is hosed and need to re-create to get dev setup env working. Will try again over the weekend.

Third issue trying to work around the first issue without a new nightly release. Tried to run pip install . from shortfin instructions. Got error

Looks like you are missing a C compiler. That's not going to work :)

subprocess.check_call([env_info.cmake_exe, "--build", str(build_dir)])

Something wrong with cmake. Needs to log its actual command lines more robustly.
This is a hint: CMake Error: Generator: execution of make failed. Make command was: /tmp/pip-build-env-mii6rgj2/overlay/bin/ninja
What that is saying is that it is trying to use the cmake/ninja from within an isolated venv that pip creates. This is most likely left-over from your prior attempt. Fix would have been to rm -Rf build/. Once it gets wedged like that, it cannot be un-wedged.

F strings are a real work in progress across those python versions. We should just fix them and then run tests. No better way. If you leave them out, more poor souls will try anyway and have a bad first experience. You don't even need to run tests for this stuff... Just need to load the code.
But we should have a strong recommendation on highest performing setup. Every of the last several versions of python has enabled significant performance improvements. Anyone who cares will want the bleeding edge. But we want oobe and casual use to be ok.
While people can certainly self source on how to get their python env setup, it'd be nice to script the CI better for env setup then recommend people do that for the best outcome. We should just use pyenv for that, which let's us precisely control the python setup. That's what anyone who cares is going to do (either because of perf or because of supply chain management).


Use Cloudinary for Image Processing and Storage

Use the free tier for this service https://cloudinary.com/pricing
Create a common account for our team.

done


Arabic Number Model and Percent Model Pull Request

Dev Changes

NumberModel (Covered 98 Cases / In progress 13 Cases)
Integer
Decimal
Fraction
Power
-Percent Model (Covered 100%)
Test Specs
-NumberModel Test Specs: 111
-PercentModel Specs: 100%

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

Approved with comments.
Please make sure the usage of NotSupported/NotSupportedByDesign is consistent. A few cases are not.
Do you have a tentative ETA by when the previous spec cases can be validated?
Lastly, when can we expect the PRs for the other Arabic number types?
Thanks!

We will be raising PRs for other Arabic number types Today  by EOD.
We will send separate mail on ETA for previous spec validation
Thank-you!


Test Following feature, now with w3c

[x] Loading animation in follow button in modal is unmodified
[x] When clicking Follow while other buttons are in the process
[x] Following tab loading animation
[x] Switching account (mm to fortmatic) did not reset myprofile values - not with "Change wallet" but logging in again with "Choose"
[x] "No previous account saved" when it recognizes that Im on my public profile page
[x] "Your profile" margin remains when redirecting to non-public profile page
[x] Don't let following in profile wait for activity
[x] Following from pub profile on logged out state not working with fortmatic
[x] Following working with MM
[x] Following working with Fortmatic
[x] Following working with Portis
[x] Test with multiple tabs open

[ ] Following working with WalletConnect
[ ] Following join thread bug prevents MM working on first time, and always for Walletconnect
[ ] following is public is showing up multiple time
[ ] unfollow when youve already unfollowed from another tab, what would happen? dont do the inverse
[ ] remove word "Foloowing" from mutual
[ ] make border the background grey but keep white on modal

closing this issue and opening a new one for the upcoming sprint


Custom parser

I want to make custom parser based on markdown syntax.
But I want to use only asterisk (*, **, ***) syntax and
Link syntax to []() -> []<>
There is so many options in parser so I can't test it all.
Is there method that turn off all of parse options?
URL Link parsing, code parsing is still doing when I set option to
  static final DataHolder OPTIONS = new MutableDataSet()
      .set(Parser.ASTERISK_DELIMITER_PROCESSOR, false)
      .set(Parser.BLANK_LINES_IN_AST, true)
      .set(Parser.BLOCK_QUOTE_PARSER, false)
      .set(Parser.FENCED_CODE_BLOCK_PARSER, false)
      .set(Parser.HEADING_PARSER, false)
      .set(Parser.HTML_BLOCK_PARSER, false)
      .set(Parser.INDENTED_CODE_BLOCK_PARSER, false)
      .set(Parser.LIST_BLOCK_PARSER, false)
      .set(Parser.THEMATIC_BREAK_PARSER, false)
      .set(Parser.REFERENCE_PARAGRAPH_PRE_PROCESSOR, false)
      .set(Parser.UNDERSCORE_DELIMITER_PROCESSOR, false);

@shimika, the options you have listed are for block elements. Inline parsing is done by the InlineParser and does not have individual element options because it is all one big pile of code.
You can take a look at the InlineParserImpl and write your own and to the parser builder customInlineParserFactory
There are many nuances to what the code expects and if you are writing your own inline parser you will need to make it work with what the code expects. It might be easier for you just to write your own complete parser and not try to re-use pieces of the library.


Fix #291

Close #291

This fix fixed a problem I was having, and the change is so simple. +1

It is just overlooked function call in 5888211c7ccea0d1d62b621450d5862633cf6651

Thanks for tracking this down!


Page interfaces changes in hugo master break te theme

This (https://github.com/gohugoio/hugo/commit/597e418cb02883418f2cebb41400e8e61413f651) recent big landing on hugo master seems to break this theme. This means anyone using go get to install hugo will not be able to use this theme. I'm not familiar enough with hugo to fully debug this, but this looks related to the taxonomy changes. Opening this up for more knowledgeable eyes than mine to take a look.
Sample errors include:
ERROR 2019/03/24 14:46:41 render of "taxonomyTerm" failed: execute of template failed: template: _default/terms.html:24:19: executing "_default/terms.html" at <partial "nav.html" .>: error calling partial: "/home/travis/gopath/src/github.com/davidski/tidyrisk/themes/introduction/layouts/partials/nav.html:5:27": execute of template failed: template: partials/nav.html:5:27: executing "partials/nav.html" at <.Page.IsHome>: can't evaluate field Page in type *hugolib.pageState
My source deployment is at https://github.com/davidski/tidyrisk.
Thanks for all the work on this nifty theme!

Hey @davidski,
Thanks for the report and heads up. I will look into this issue after the next release of Hugo has been published. Who knows what kind of other breaking changes will come up?
Site note: Is there a particular reason you install Hugo via go get instead of fetching the binary? You can also use go get<EMAIL_ADDRESS>if you are using Go Modules.

See also https://github.com/gohugoio/hugo/issues/5784#thread-subscription-status

@hanzei
The problem is in /layouts/partials/nav.html
Basically you need to change .Page.IsHome to .IsHome and every other instance of .Page e.g. .Page.Section since now Page has become an interface.
However also keep track of https://github.com/gohugoio/hugo/issues/5784 in case something changes.

Thanks for the quick investigation!
Pinning a particular version of hugo would definitely be a much better idea. I'm running this particular build on Travis CI, where I've been spectacularly unsuccessful in getting a defined version of hugo up and running. The linux x64 builds surprisingly seem to require some additional C++ compilers to run, which was a bit too exhausting to track down this weekend.
I'll give the go get syntax you suggest another try. I'm pretty sure I tried that with some issue, but will hack at that some more.

I'm running this particular build on Travis CI, where I've been spectacularly unsuccessful in getting a defined version of hugo up and running.

Take a look at https://github.com/victoriadotdev/hugo-theme-introduction/blob/master/.travis.yml#L13-L16

Appreciate the tip! Unfortunately, that still errors out for me due to the versions of glibc available. This may be because I'm running hugo extended for full resource regeneration...
https://travis-ci.org/davidski/tidyrisk/builds/510726068 shows the errors I get, if you're curious. :smile:

We are getting off-topic quite fast, but still:
I strongly think you should check in your resources directory and get rid of the extended version for CI. There is just no need for it.

@hanzei Feel free to close this issue. You don't really need to worry about this as it was fixed.
Note that several variables became deprected and these will now throw warnings that you will need to address once Hugo 0.55 is out.

Thanks for the heads up @onedrawingperday.
And as always: Thank you very much for the work you do for Hugo and the community! It's awesome! :tada:


modify bug #2768

Fixes #2768.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Pull Request Test Coverage Report for Build 9000

7 of 9 (77.78%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.002%) to 63.994%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/sharding-core-optimize/src/main/java/org/apache/shardingsphere/core/optimize/encrypt/engine/dml/EncryptInsertOptimizeEngine.java
3
4
75.0%

sharding-core/sharding-core-optimize/src/main/java/org/apache/shardingsphere/core/optimize/sharding/engnie/dml/ShardingInsertOptimizeEngine.java
4
5
80.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-orchestration/sharding-orchestration-core/src/main/java/org/apache/shardingsphere/orchestration/internal/util/IpUtils.java
3
76.0%

Totals

Change from base Build 182:
-0.002%

Covered Lines:
8906

Relevant Lines:
13917

💛  - Coveralls


enable shared config state

Thanks for making greate tool.
I want to use ecschedule with ecspresso in my production environment. But, ecschedule doesn't support shared config though ecspresso do.
This is a little bit comfusing in local development.
If you merge this PR,  ecspresso will load a config from ~/.aws/config.

Sorry for my slow response. Thanks for your contribution!

Thanks for the merge!


PDF rendering is quite slow with some PDFs, compared to other viewers

Attach (recommended) or Link to PDF file here:
Floor2.pdf
Configuration:

Web browser and its version: Chrome 99.0.4844.51
Operating system and its version: macOS monterey 12.2.1 (21D62)
PDF.js version: newest
Is a browser extension: no

Steps to reproduce the problem:

Open the attached PDF with the official viewer: https://mozilla.github.io/pdf.js/web/viewer.html
Try zooming in/out, move the PDF around, etc.

What is the expected behavior? (add screenshot)
It should be quicker. I tried opening the same PDF with "preview" on mac, and with chrome, and they both render the PDF much-much faster, without any noticable loading/hanging.
What went wrong? (add screenshot)
It takes quite a while to load up/render the PDF, and then when I start zooming in and out, it often just turns white, and it hangs for 5-10 seconds before I can see something again.
By the way, I noticed this issue in a complex project, where we use the pdf.js javascript API to render certain parts of the PDF (with a defined viewbox), but it's much easier to reproduce the issue with the official PDF viewer. I sometimes had performance issues before, but in those cases other viewers weren't really any better for those PDFs. With this particular PDF, other viewers/renderers can outperform PDF.js by a lot.
So maybe someone here can find the reason behind it, and even fix it.

That PDF document contains a huge amount of path rendering operators, which likely explains why this is a bit slow since that's something that's not entirely easy to optimize (and it thus looks a tiny bit similar to e.g. issue #10565).

@Snuffleupagus how do you check the number of paths in a PDF? Also, any idea how to reduce that number, without visually affecting anything? I mean offline solutions, of course, not realtime ones. Although those would be the best, but I believe those are just too complicated, as you mentioned.

@Snuffleupagus where is /Contents-streams ? Is that somehow accessible from the javascript API? I can't find it :(

@soadzoor This refers to the built-in structure of the PDF file. You can open the file with a PDF browser, for example https://brendandahl.github.io/pdf.js.utils/browser/, to see the inner structure of the PDF file. In the tree there is also a /Contents field.

@timvandermeij Hm, but on the linked page, I can only see things like
Filter = /FlateDecode
Length = 3618135

How do I check how many path rendering operators are there? In https://github.com/mozilla/pdf.js/issues/10565, someone even left a comment with the exact number of paths (7.8 million). If I could retrieve that information, I could warn the users at least, that they're trying to render a complex PDF, so they should expect performance problems.
The question is: how can I retrieve that information?

How do I check how many path rendering operators are there?

You'd essentially have to manually instrument the code, if you want a fully accurate number (w.r.t. the PDF document itself), since no such functionality is provided (there's no general use-case that warrants adding such a feature).
Although, I suppose that it's also possible to use the getOperatorList-method and then loop through the operatorList to check for OPS.constructPath and using its args length to calculate the total number of path operators. Please see https://github.com/mozilla/pdf.js/blob/ee39499a5ab6063fdc5cf58df81ad5b36bd8efed/src/display/api.js#L1537-L1546 and https://github.com/mozilla/pdf.js/blob/ee39499a5ab6063fdc5cf58df81ad5b36bd8efed/src/display/api.js#L1212-L1219

Thanks! I played around with this, and it seems the PDF file above has 54 127 path rendering operators (I simply filter the operatorlist.fnArray for OPS.constructPath to get this number).
That doesn't seem like a lot. For example the following PDF has more than 3 times as many path rendering operators (167 160), yet it's still faster to render:
167160.pdf
Or this one has more than 1.2 million path rendering operators, yet it still seems to be faster to render than the original one in the first post:
murietta.pdf
So I don't think there's a linear correlation between the number of path rendering operators and performance. I think there's more to this 🤔

I played around with this, and it seems the PDF file above has 54 127 path rendering operators (I simply filter the operatorlist.fnArray for OPS.constructPath to get this number).

Note that you've underestimated the actual number by at least one order of magnitude, since only counting OPS.constructPath isn't accurate given that each one can contain lots of actual path operators.
As mentioned in https://github.com/mozilla/pdf.js/issues/14652#issuecomment-1064328358 you need to count the args (i.e. the contents of the argsArray) instead; although I don't think putting time/effort into that really helps fixing this issue :-)

you can try pdf slice loading, very fast: Introduces the address

It renders in about 3 seconds on Windows, and one can see that something is happening. My PC is from 2017 but with lots of memory. I don't see this as a performance problem at all.

If I could retrieve that information, I could warn the users at least, that they're trying to render a complex PDF, so they should expect performance problems.

this would take longer than rendering this PDF itself.


Allowed memory size of 134217728 bytes exhausted

When I tried to explore a image(about 70M), I got the following error,
Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 97135717 bytes) in /var/www/index.php(10) : runtime-created function(1) : eval()'d code on line 42

btw, I run b374k on my Android Phone with the help of gnuroot.

can you try with plain version (without encode -b or compression -z) ?


Add external key derivation feature

Description
~Add functionality for user to derive keys, using tls-prf,
sby defining a callback function.~
Extend the MBEDTLS_SSL_EXPORT_KEYS feature, to export the handshake randbytes and the use tls_prf callback.
Status
READY
Requires Backporting
NO
Comments
Since there isn't an infrastructure for comparing output buffers in ssl-opts.sh, I manually checked the ssl_client2 example, printing the EAP-TLS generated key material with the output of openssl s_server -keymatexport "client EAP encryption" -keymatexportlen 16
Todos

[ ] Tests
[x] Documentation
[x] Changelog updated
[ ] Backported

I have an open question regarding the callback function:
Should it accept mbedtls_ssl_session as a parameter or should the key for the derivation be part of the user defined context given as parameter, and in case the key is the master key, it should be exported using MBEDTLS_SSL_EXPORT_KEYS feature?
Pros for adding the session as parameter:

Remove the coupling for MBEDTLS_SSL_EXPORT_KEYS in case the key is the master key \ keyblk
The need for implementing both this callback and the export keys callback can be confusing and prone to errors.

Cons for adding the session:

In case the key is a key other than the master key, the session is probably unused parameter.
Using a structure that consumes some RAM, only for the key is in efficient.
Requires accessing internal members of the session, which could break in an ABI break.
Using a separate function for exporting the master key is much more flexible for changes., and "cleaner"

@manuelgavidia Yes a user should know what key they wish to derive, using the tls-prf.
The intention for this feature is for users to be able to use the tls-prf function for deriving keys not used for TLS, such as EAP-TLS (and perhaps DTLS-SRTP)
This feature is similar to the openssl options keymatexport , except that it allows the user flexibility to choose what is the key used for derivation.
Thanks for your confirmation.

@RonEld If we are striving for maximal flexibility, why don't we simply pass the ssl context to the callback?
@manuelgavidia You don't need to know it beforehand, but you need to take a different approach with the callback. Instead of generating your keys in the callbacks, you just save the client random and server random values along with the master secret and a pointer to the prf in your custom context. After the handshake has finished you can use them to generate only the keys you need.
@RonEld Can we use this approach in the example instead of calculating the keys on the fly? I think it is more flexible and we would make things easier for users if we showed this in the sample application.

@yanesca , in this case, I would suggest enhancing the f_export_keys callback ( using the MBEDTLS_SSL_EXPORT_KEYS) , by adding an additional parameter for the random buffer

@yanesca Regarding you first comment, would this expose too much data to user? I tough ssl context should be used only by mbedtls functions and not user callback/functions. Agree/understood your two other comments.

@RonEld Sounds good, I like the idea!
@manuelgavidia Yes it would, but the session is internal struct and we are exposing it already and I thought it wouldn't make much of a difference. Fortunately now with Ron's suggestion we don't need to expose either of them.

@yanesca I ammended the commit to reflect the change in design we have discussed.
Please review

@k-stachowiak I amended the commit with your comments.
Please review.
Note that this still needs design approval.

@yanesca I have addressed your comments, in addition separating to several commits, per your request.
Please update whether to resolve conflicts

@RonEld Next time when you split up or modify commits, could you please not change bases, it makes review much harder.

@yanesca I believe I have addressed all your comments

I have rebased to resolve the conflicts
@yanesca @k-stachowiak please review

@RonEld Can you please fix the CI failures? Do you have a link to the old branch, before the rebase?

@yanesca This failure is not related to the rebase.
It's because MBEDTLS_SSL_PROTO_TLS1_2 is not configured in the failued test.
I'll fix

@yanesca I believe I have fixed the CI failures.
Let's see

For reference, the branch prior to the rebase is in https://github.com/RonEld/mbedtls/tree/mbedtls_eap_tls_key_material_before_rebase

@yanesca @k-stachowiak I noticed the tests fail in case MBEDTLS_USE_PSA_CRYPTO is used. I believe I have fixed this

@k-stachowiak thank you for your comments!
I have addressed your comments and fixed yet another CI failure in mingw build

The PR has been fully reviewed and approved.
The CI failures are in IAR8 and win32 being stuck
cc @Patater