New Bluesky user

Title
Shannon Lindsay
Bluesky profile url
https://bsky.app/profile/shan-gsd.bsky.social
Twitter profile (url or username)
https://x.com/shan_gsd
Category
data
Type
microsoft

Thanks so much for creating these badges - super appreciate your contribution the the community!!


Fix #1814: check if series is not empty before deleting first element

Hi all.
This is not the best way to fix this bug probably, but it works.

Does the latest change break something?
If it is not, might these changes be merged?

ok, for me. @DanCech What do you think?

I don't think this is a good idea as it stands, because of the change it introduces in how the final step is calculated.


quicksand: init at 2.0-unstable-2021-01-15

Description of changes

Wanted to add this font, as it is part of my Fursona's refsheet graphical design.
First time making a font package, but I don't know how to test it. Please let me know if I need to change the directories.
The Google fonts version is known to break on inkscape and blender, so I used the Github version from the author.
Since I have multiple PRs waiting without hubble added as maintainer, I'll make it a separate PR. added!
Homepage: https://github.com/andrew-paglinawan/QuicksandFamily
Google fonts preview: https://fonts.google.com/specimen/Quicksand
Things done

Built on platform(s)

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

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

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

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

[ ] Fits CONTRIBUTING.md.


Add S3 bucket connectivity or integration into SageMaker Studio Lab

Is your feature request related to a problem? Please describe.
I'd like to be able to temporarily connect different S3 buckets to SageMaker Studio Lab (SMSL) for either extra library storage or dataset storage.  I'm currently having trouble installing the latest RAPIDS version on SMSL due to running out of space, and have resorted to deleting datasets that take a long time to download.  I even have to delete the zip files.  The downloads, conda environment recreation, and failed installs take up precious GPU time that is already hard to come by.
Describe the solution you'd like
I'd like to be able to connect an s3 bucket to SMSL with the conda envs and data ready to go.
Describe alternatives you've considered

Deleting and redownloading the data needed.
Removing other conda environments not currently being used and reinstalling them when they are needed again
using Pip install packages

Additional context
Some additional libraries for GPU deep learning and machine learning take up a bit of room when downloading and expanding using conda.

@taureandyernv , thank you for the feedback! I understood to evacuate existing conda environment and dataset and back these after the operation is painful.
I felt "mount S3 to Studio Lab" experience will be what you need. Does goofys match your need? (For now, we can not install it because apt install is required now.)
https://github.com/kahing/goofys

@taureandyernv, as your ML experiments get too big for Studio Lab you may want to consider launching your notebooks in Sagemaker Studio.  We tried to make this easy to do with a new feature called Notebook jobs (just released last December 22). With Notebook jobs, you can work on your notebook in Studio Lab but then schedule it to run in SageMaker (you will need an AWS account). The job will kick off and shut down when complete. Yes, there will be some cost, but depending on the instance type you select it may be negligible.
Check out this blog here and let me know what you think: https://aws.amazon.com/blogs/machine-learning/run-notebooks-as-batch-jobs-in-amazon-sagemaker-studio-lab/

Are there any plans to allow mounting of external drives such as S3 in Studio Lab?
I am a student and cannot use Sagemaker.

@fkunn1326 thank you for the comment, we have not had the specific plan to mount the S3. You aren't allowed to use SageMaker in your school AWS account?

I think installing Mountpoint S3 to Studio Lab will work (it requires sudo so users can not install for now).
https://github.com/awslabs/mountpoint-s3


Ipss 71

Added the policy page and put links to it within the footer and sign-up modal.
I think the page itself looks kind of ugly so any feedback would be appreciated

Looks Great! But it is displaying LoggedInNavbar when I wasn't logged in...need to fix that.

Should be all good now!


[WRK-1496] raven/sentry replace

Hi, I noticed what was probably a mistake target PR to the source Scimitar branch and incorporated the idea into v1.3 for Rails 6 and v2.1 for Rails 7. This has other fixes that I think you would find useful, so it could be worth seeing if you can drop your custom fork in favour of the v1.3 release via RubyGems to avoid some maintenance overhead.
If you've any future changes that you think would be worth contributing to the mainline development, please do submit PRs - just target the v1 branch rather than main, for as long as you're still using Rails 6 and Scimiar v1.x.

...Oh yeah, almost forgot - the "generic" feature makes the exception reporter configurable, so in your engine configuration in config/initializers/scimitar.rb you would add something like:
Scimitar.engine_configuration = Scimitar::EngineConfiguration.new({

  # ...existing configuration, then add:
  #
  exception_reporter: Proc.new do | exception |
    Sentry.capture_exception(exception)
  end

})


Get rid of old QuartzCore-bound ShapeType protocols

The following protocols are gross:

Buildable
PathMaking
FrameMaking
ShapeType
CompositeShapeType
GraphicsContainerType

For the following reasons:

Too coupled to the Quartz framework
Compose in a fragile way
Awkwardly hooked into the CALayer hierarchy.

Given that we have StyledPath, we can replace these with the Renderable, ConfiguredRenderable, CompositeRenderable interfaces.


Documentation from Documentation/technical/ not hosted

The Git man pages make occasional references to information in technical/; however, this documentation is not hosted anywhere on the site, so the link 302's to the home page for the docs.
kernel.org does host them FWIW.
@peff thoughts?

In ye olden days when git-scm.com was first written, the pages in technical/ did not even build to HTML. And since there's so rarely referenced by the main documentation, nobody ever noticed or added it. I suspect it would just need some tweaks to the rake task that pulls in the pages. Patches definitely welcome.

@sxlijin Can you point to where the 302 happens?

@jnavila see #512
$ curl -is http://git-scm.com/docs/technical/api-credentials.html | head -n 1
HTTP/1.1 301 Moved Permanently

The 301 is to redirect you to https, and then you get the 302 (curl doesn't follow the redirects by default, but if you add in -L you can see it switching to HTTP/2. Neat!).
I'm not sure about the 307. It's obviously accomplishing the same thng, but it's interesting that the server generates a different code. This is all Cloudflare, so they may tailor the response based on user-agent sniffing.
Anyway, that's all orthogonal to the actual problem that we don't import the technical/ docs.


Setting showHint on uibTypeAhead makes it unable to tab to it

Setting the typeahead-show-hint option on an input with uib-typeahead makes it unable to use the TAB key to activate the typeahead input.
Plunkr

true, the tabindex of the hintInputElement should be set to -1.
At the moment you need to tab twice to get to the right element.

Feel free to open a PR for this - should be an easy fix.

Thanks @deeg!

This still seems to be broken. If I add the show-hint attribute it will always focus the outer element first and on the next tab select the inner input. Everything works as expected if I remove the attribute.


dump : fix service file and install as part of repository build

Made changes to install the<EMAIL_ADDRESS>as part of the repository build and also create soft links.

Now modified to install the service file and create softlink as part of repository build
Modified the service file with templated approach as the service starts before start host@i target.

Upstream commit link
https://gerrit.openbmc.org/c/openbmc/openpower-debug-collector/+/42328
Tested
root@p10bmc:~# obmcutil state
CurrentBMCState     : xyz.openbmc_project.State.BMC.BMCState.Ready
CurrentPowerState   : xyz.openbmc_project.State.Chassis.PowerState.On
CurrentHostState    : xyz.openbmc_project.State.Host.HostState.Off
BootProgress        : xyz.openbmc_project.State.Boot.Progress.ProgressStages.Unspecified
OperatingSystemState: xyz.openbmc_project.State.OperatingSystem.Status.OSStatus.Inactive
root@p10bmc:~# systemctl status<EMAIL_ADDRESS>
<EMAIL_ADDRESS>- OpenPOWER Dump Manager Host 0
Loaded: loaded<EMAIL_ADDRESS>static)
Active: inactive (dead)

root@p10bmc:~#obmcutil poweron
root@p10bmc:~# systemctl status<EMAIL_ADDRESS>
<EMAIL_ADDRESS>- OpenPOWER Dump Manager Host 0
Loaded: loaded<EMAIL_ADDRESS>static)
Active: active (running) since Wed 2023-02-22 08:17:26 UTC; 1min 29s ago
Main PID: 917 (openpower-dump-)
CGroup<EMAIL_ADDRESS>`-917 /usr/bin/openpower-dump-manager

Feb 22 08:17:25 p10bmc systemd[1]: Starting OpenPOWER Dump Manager Host 0...
Feb 22 08:17:26 p10bmc systemd[1]: Started OpenPOWER Dump Manager Host 0.
root@p10bmc:~# busctl tree org.open_power.Dump.Manager
-/org   -/org/openpower
`-/org/openpower/dump
Change-Id: Ice236af70cec8433835d24d3ae4bdaf55c7e82cf

Can one of the admins verify this patch?

Include gerrit link.

Include gerrit link.

done


F/serverless hls

This is a rough pull request for serverless HLS.
It has been tested with ffmpeg 4.2.3, but it should work with 4.1.x.
The demo workflow is fast-HLS.xml.
I changed paella to ignore mp4s if there is a master playlist, which may not be the correct behavior if you want to allow users to play mp4s as well as HLS.

Hi Phoebe!
I'm reviewing your code. great work you have done!! :+1:  It has also unit tests!! :smile:
I have some questions, from what you presented on Gent last winter, HLS serverless capability only needs to be added like an encoding profile.  What is the reason to create 3 new WOHs? Can you add in the PR description a little abstract of each?
Also, I saw that are some changes to the editor to be compatible with HLS, I though that HLS is only for publishing because it is transport stream and not is suitable to work with for editing tasks. What let you make that modification?

Hi Max,
Thanks for reviewing my code. I hope that there are enough comments. Please don't hesitate to ask me questions.
We run highly specialized and optimized workflows here, so it may look a bit different from the default.
First of all, there is only one new woh - SanitizeAdaptiveWorkflowOperationHandler - It is useful only when an HLS file is uploaded (from archive) and republished without re-encoding. It is something that we do to republish old lectures.
The other 2 (MultiEncode and ProcessSmil) have been in the repo for a few releases and are modified to use the new EncodingProfile type.
We use them to encode directly from source to delivery and they share the same underlying code. The difference is that ProcessSmil uses the smil file to do a trim as well. They both call the same code to create a huge ffmpeg command to do all the encoding.
The default workflow would first run the editor UI and encode the "/source" into "/trimmed", and then after some processing, segmentation, slides, etc, there is another encode into "/delivery".
Our wf takes the output from the editor - the smil file with the trim points - and save it in the mediapackage. Then ProcessSmil uses the smil file and the encoding profiles in its configuration to encode the source into the delivery formats. The delivery files are tagged with profile names and we use the highest quality one to do segmentation and generate slides.
We do this to save one encode ("/source" -> "/trimmed") which can take ~90m for a 2 hr lecture.
To use it to generate HLS just for delivery, you can always use the same wf, but use MultiEncode to encode from "/trimmed" to "*/delivery" at the end.

Tests Made/planned

[x]  local machine
[x] With no privileged user
[ ] Multi-node installation

I will test in our local develop multi-node scheme, after that I think is ok to merge :)

Marking this as on-hold pending results.

Hi, I want to notice that I'm now able to make the last test with a cluster, I have a problems with the search endpoint in the presenter node but I think are unrelated to this commit. If I have a correct result I will merge it.


VTT Subtitel Support

hi guys this is an interesting project i want to ask if it supports Subtitles if yes can you tell me how it works?

Hello. Supporting subtitles is the prerogative of the player, not the media server.


Add docs and comments on how Rancher handles Helm chart values

This PR contains no code changes. It addresses https://github.com/rancher/dashboard/issues/5904 by adding developer docs and comments about Helm chart values in Rancher.
Background
I created this documentation for two reasons:

As part of the work that needs to be done for project monitoring, Sean needs to create a component that allows users to manipulate values in a ProjectHelmChart custom resource. We didn't include that component in Rancher v2.6.5 because we weren't sure how much logic from the regular Helm chart install/upgrade process also applied to what the user experience should be for editing the ProjectHelmChart. These docs should help us figure out how to either tailor this component to the needs of the ProjectHelmChart or to helm create a much simplified version of the component just for that purpose.
Arvind requested docs on specifically what values Rancher injects into Helm chart values so that in the future, we can move toward a Helm chart upgrade process in which Rancher injects values into a ConfigMap and the Helm chart app itself can use the values from there. That process would place more control into the hands of chart developers and could help prevent problems when upgrading Helm chart apps.

Summary
The original issue mentioned a four-way merge for upgrading Helm chart apps, but I didn't find evidence of a four-way merge. The logic seems to be simpler than that. If you are upgrading a Helm chart app, the Rancher UI fetches the standard values.yaml for the source chart and compares them with the actual values. It takes the difference and assumes all differences are "user values" or user-customized values. Then when you select a chart version to upgrade to, it applies the customizations of the old chart onto the new chart to set up the default values of the values.yaml form. That's the most complicated thing that happens.
New developer docs
The upshot of what I found is added to the new developer docs here: https://github.com/rancher/dashboard/blob/c9c09b34af9a7613795ef2ba17cccb21fbf03326/docs/developer/helm-chart-apps.md
Tech debt
While documenting this logic I found some opportunities to clean up the code and opened a tech debt issue about it. https://github.com/rancher/dashboard/issues/6069

It looks like this build is failing because of a couple added console.logs causing lint warnings, otherwise LGTM: your explanation aligns with my understanding of this functionality.

It looks like this build is failing because of a couple added console.logs causing lint warnings, otherwise LGTM: your explanation aligns with my understanding of this functionality.

Removed the console.logs


Update deps

This updates all deps other than SQLalch, which I am planning to do in another PR.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

40 of 40 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 81.03%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
692

Relevant Lines:
854

💛  - Coveralls


New Tool - Remove Trailing Whitespace Characters

Type of Change

[x] New feature

Description
Will provide a new tool for contributors to use when developing WinUtil, this tool in particular will remove trailing whitespace characters.
Testing
it works as expected, As long as one condition is met, which's running it inside WinUtil Root Folder (couldn't make it work properly, as there'll be too many edge cases)
Impact
It won't impact WinUtil Users, but it'll provide a better experience for devs/contributors.
Checklist

[x] My code adheres to the coding and style guidelines of the project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generate no errors/warnings/merge conflicts.

@ChrisTitusTech you can change the script if you want, I've left Allow edits and access to secrets by maintainers enabled.

Would it be a good idea to run the script automatically on compile so that all files will be kept "clean" on every compile?


Upgrade PW_HASH_ALGORITHM

The use of less robust hashes would prevent a system deploying Accumulo
from meeting FIPS requirements so by updating it we avoid forcing
downstream users seeking FIPS certification from getting an exception.

Thanks for the contributions.  Just wanted to point out that the accumulo dev mailing list or the Accumulo slack channel (https://the-asf.slack.com/messages/CERNB8NDC) can be used if you have questions on impacts / secondary considerations.

@ctubbsii is your request specifically that the PR be changed to allow for upgrades that maintain user credentials?

e.g. would an approach where the algorithm is configurable but defaults to the current SHA-256 be acceptable? that would allow existing users to continue to work as-is while allowing those with more stringent requirements to make a trade-off decision on starting from a new set of user credentials vs trying to get an exception in a certification process?

@ctubbsii is your request specifically that the PR be changed to allow for upgrades that maintain user credentials?

I didn't necessarily have a request. Just an observation that merging this as-is would break users on upgrade, and that it overlapped with an old JIRA issue.

e.g. would an approach where the algorithm is configurable but defaults to the current SHA-256 be acceptable? that would allow existing users to continue to work as-is while allowing those with more stringent requirements to make a trade-off decision on starting from a new set of user credentials vs trying to get an exception in a certification process?

That approach would allow new users to use stronger hashing, while not affecting current users. However, if we make it configurable, that would provide a path for eager security-conscious users to break themselves by switching the algorithm. I think a better approach would be to implement an upgrade path to SHA-512 by detecting the old format when a user authenticates, and the system automatically replacing it with a new ZK node using the crypt(3) format with commons-codec. On startup, we can warn if there are any users in ZK that have passwords hashed with the old format, with instructions to authenticate to upgrade.
This would take care of the upgrade issue, ensure stronger security by default, would not allow users to break themselves, and would use a standard serialization that is robust in the face of new algorithms being added in future.
If somebody wants to do this, I know I would appreciate it, and I think it would add great value to the project. @BukrosSzabolcs is welcome to tackle it, if they are interested.

Given unlimited volunteer effort I agree that such an implementation would be nice.
Can you think of a technical justification for a veto for the simpler incremental change where the hash is configurable, presuming it is paired with a suitable warning on compatibility?

Adding a change to the upgrade code is not that big of deal anymore, since it is now done in one spot.  There is already a method for upgrading data in ZK going from 2.0 to 2.1.  See https://github.com/apache/accumulo/blob/cccac86255cd2d0ecd49e791162ca5cdf07512b6/server/manager/src/main/java/org/apache/accumulo/master/upgrade/Upgrader9to10.java#L96

would an approach where the algorithm is configurable but defaults to the current SHA-256 be acceptable?

This seems super reasonable to me.

Given unlimited volunteer effort I agree that such an implementation would be nice.

I was operating on the assumption that there was already an interested volunteer willing to pursue this a little bit. Is that not the case? I don't think the solution I explained above is that complex, and I'm happy to help as needed. I think this would be a good opportunity to encourage a contributor, if they are interested!

Can you think of a technical justification for a veto for the simpler incremental change where the hash is configurable, presuming it is paired with a suitable warning on compatibility?

A possible technical justification for a veto would be that the Authenticator component is pluggable already, and there is no reason to implement half measures when a use case for specific Authenticator needs can already be satisfied with what's there now: copy/rename/replace hash/deploy... no change to Accumulo needed if all one wants to do is use SHA-512.
My hope in discussing the design of the preferred solution was to encourage a possible contribution from an interested party. If such a party does not exist, I would prefer to do it myself rather than go with an intermediate solution. However, my highest preference was to encourage a contributor effort. Vetoing was the last thing on my mind.

Adding a change to the upgrade code is not that big of deal anymore, since it is now done in one spot. There is already a method for upgrading data in ZK going from 2.0 to 2.1. See
https://github.com/apache/accumulo/blob/cccac86255cd2d0ecd49e791162ca5cdf07512b6/server/manager/src/main/java/org/apache/accumulo/master/upgrade/Upgrader9to10.java#L96

The problem with this upgrade code is that we can't upgrade our hashes without knowing the original passwords, and we don't know that until we can successfully authenticate using a provided password. We might be able to transform our existing passwords into something that is readable using commons-codec, but I doubt we use the salt exactly the same way.

My hope in discussing the design of the preferred solution was to encourage a possible contribution from an interested party. If such a party does not exist, I would prefer to do it myself rather than go with an intermediate solution

My only worry in this line of thinking is that it hinges on your seniority and can act as a form of exclusion. I would hope that we could accept the contribution that Szabolcs wants to do as long as it is not harmful. If you would want to later come along and change this, it would, of course, be in your purview to do so.
When you say "I'd rather it be done this way and I'll do it that way if you don't want to", it removes all incentive for the contributor to stay engaged. My $0.02.

I was researching the PR and assessing the impact that Christopher pointed out - he just types faster.  I am also concerned with breaking existing installations and would need to see either an upgrade path or have it implemented as an option.
I tried to point out that we have a couple of communication channels available where we could work this out while encouraging additional contributions.
I did not want to see Szabolcs become frustrated if the PR was not immediately accepted - the code and the contribution are welcome, but there are additional considerations that may not be obvious at the start.  If there had been an issue / question raised first, or if there was a dialog, via email or slack, then maybe those could have worked out.  Taking the time and effort to create a code change is welcome, but engaging with the community first might alter the approach, make the change easier to integrate and would help ensure that everyone's interest and concerns are also addressed.
The intent of the PR is valid and I'm assuming being driven by a requirement, but there also is an obligation to the existing user base. I would not expect the community to accept allowing a change that broke existing installations on an upgrade without providing some path forward.

@EdColeman to clarify, the problem with making the change verbatim as it is now (s/256/512/) is understood and not being contested.
The question is: if using sha512 instead of sha256 was something which was configurable (opt-in via the user), and there was a contribution for that, would it be accepted? Christopher had stated that he would prefer that not happen, which is how we got into the current discussion.
My point was that there is always a "more elegant" solution which can be had. If there is a simple solution which is not harmful, I would err on accepting such a solution (configurable PW Hash algorithm) and letting those with appetite build their elegant solutions on their own time.

If there is a simple solution which is not harmful, I would err on accepting such a solution (configurable PW Hash algorithm) and letting those with appetite build their elegant solutions on their own time.

I think you missed part of what @ctubbsii said, pointing out how that solution would be harmful:

However, if we make it configurable, that would provide a path for eager security-conscious users to break themselves by switching the algorithm.

Making it configurable is better but still potentially harmful and incomplete.

pointing out how that solution would be harmful

I do not agree that "lack of upgrade support" is a harmful path when behind configuration. The only pain felt is when the user chooses to change this, and that would be pain they choose to self-inflict. Harm, to me, is something that a user could not avoid.
Please correct me if I've missed some subtlety as to how a configurable password algorithm would be considered harmful.

I do not agree that "lack of upgrade support" is a harmful path when behind configuration.

I was not saying "lack of upgrade support" is harmful.  I was saying providing the user with a way to easily harm themselves would be when they believe they are improving the security of their system.

The only pain felt is when the user chooses to change this, and that would be pain they choose to self-inflict. Harm, to me, is something that a user could not avoid.

Harmful or not, this is dangerous.  Best case scenario for an existing cluster would be that Accumulo just won't start.  Worst case would be locking users out of a system and having to re-initialize a completely new cluster.  I don't know what would happen without testing.

I was not saying "lack of upgrade support" is harmful. I was saying providing the user with a way to easily harm themselves would be when they believe they are improving the security of their system.

That is why we have documentation. If you want to see documentation with such a configuration option, please say that. If you want something else to come along with a configuration option, please say exactly what you'd want to see.
I'm trying to take what is very lofty "harmful" assertions and get these translated into specific, tangible issues that aren't just "this could be bad".

That is why we have documentation. If you want to see documentation with such a configuration option, please say that. If you want something else to come along with a configuration option, please say exactly what you'd want to see.

I created an GitHub issue: https://github.com/apache/accumulo/issues/1788

@BukrosSzabolcs Since you opened this issue, I would like to ask if you are interested in contributing to the design I proposed, or if you are interested in contributing to the configurable solution proposed by others (with the understanding that it may be replaced by a different implementation in future), or if you have any suggestions for a different design that addresses some of the concerns raised here. Or, would implementing your own alternate implementation of the pluggable Authenticator interface be adequate to meet your needs until Accumulo has updated its implementation?

I would like to thank everyone for the feedback, now I can see this change was a "bit" rushed. Sorry for not communicating my intentions on other channels I was under the impression the PR would suffice. I'll look up the slack channel to avoid issues like this in the future.
@ctubbsii I like the design proposed in ACCUMULO-4044. I would be happy to contribute.

@ctubbsii I like the design proposed in ACCUMULO-4044. I would be happy to contribute.

Awesome! Let me know if I can be of any help.
And, thank you for your interest in Accumulo, and your patience in the process. Sometimes the process is turbulent, but I'm confident we can all work together to make Accumulo even better in the end!
I'll close this current PR, so we can start fresh with a new conversation when things have progressed to a new PR, but feel free to continue to comment here or on the tracking issue Mike created and linked above, if there's more to discuss (or the mailing list or Slack).


Readme

Worth mentionning: the keyring issue has been fixed upstream very recently. The fix is already in a new version of poetry-core, and the next poetry version should be right around the corner.
Delaying the merge for now.

Poetry 1.7.0 has been released 5 days ago - TODO: check if the above fix/workaround is still necessary.

Closing. This is no longer necessary.


ci: ensure compatibility with 19.09

There are a few steps I've taken to make this as robust as possible:

Add a nixpkgs arg to default.nix and shell.nix, allowing us to more
easily specify which nixpkgs to use -- rather than having to use
nix-build --arg pkgs 'import ./nix/nixpkgs-1909.nix', one only needs
to run nix-build --arg nixpkgs ./nix/nixpkgs-1909.nix.

Create two separate nixpkgs expressions, nixpkgs-stable and
nixpkgs-1909. This allows us to build against NixOS 19.09 by running the
aforementioned nix-build --arg nixpkgs ./nix/nixpkgs-1909.nix
commandline. (Also add nixos-19.09 to the update-dependencies script.)

Add an overlay build step that overlays the lorri available from
the current stable nixpkgs, to ensure that it is still possible to do
so. ("in order to not have to wait on a new lorri release in nixpkgs" --
Profpatsch)

Fixes https://github.com/target/lorri/issues/406.
Also, I updated to the latest action because it lets us drop the use of NIX_PATH/channels since we use our own versions stored in the .jsons (which ever-so-slightly speeds up CI).
Let me know if there's anything you would like to see improved.
cc @curiousleo @Profpatsch

Hm, before we just built lorri with the pinned nixpkgs via default.nix
iirc, so why not just have it build the default.nix twice, once with
19.09 and once with 20.03?
On Mon, Jun 1, 2020 at 12:52 AM Cole Helbling<EMAIL_ADDRESS>wrote:

@cole-h commented on this pull request.
In nix/overlay.nix
https://github.com/target/lorri/pull/411#discussion_r432992628:

@@ -0,0 +1,14 @@
+let

srcDef = builtins.fromJSON (builtins.readFile ./nixpkgs-stable.json);

Oops. You are indeed correct.
Do you want two overlays-tests, or just the one (for 19.09)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/target/lorri/pull/411#discussion_r432992628, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYB5ZWMKTFUP4XABWL5FMDRULNTBANCNFSM4NOK2FBA
.

Perfect, thank you.


[fontconfig] update to 2.14.1 (microsoft#27372)

Describe the pull request

What does your PR fix?
Fixes #27372

Which triplets are supported/not supported? Have you updated the CI baseline?
triplets unchanged, ci baseline not updated

Does your PR follow the maintainer guide?

Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Thanks for the update!


Assignment 06 -- 26 Mar 2024

Read the slide.
Do the examples and challenges in a notebook.
Share the notebook as a gist.
Put below the link to the gist.

Here is my work for assignment 6: https://gist.github.com/halserach/e980a24aa83f6409532479c452a2729d

Here my gist for the assigment: https://gist.github.com/denikaryono/0a5a7b1ec8b608ff0f8664da9853f797

Here is my gist for Discrete Fourier Transform (DFT) assignment: https://gist.github.com/gitaameliiaa/e211a39e03fa87005021b8233a6698fa


Window switcher in lbonn/rofi is broken

Hyprland Version
 System/Version info
Hyprland, built from branch makepkg at commit 17339e0ae9ad01dcfdede76e8dfaf516a8cbb924 dirty (input: track exclusive LSes).
Date: Fri Jan 19 23:45:34 2024
Tag: v0.34.0-54-g17339e0a

flags: (if any)

System Information:
System name: Linux
Node name: arch-g14
Release: 6.6.13-1-lts
Version: #1 SMP PREEMPT_DYNAMIC Sat, 20 Jan 2024 14:48:01 +0000

GPU information:
04:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Renoir [Radeon RX Vega 6 (Ryzen 4000/5000 Mobile Series)] [1002:1636] (rev c6) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://bugs.archlinux.org/"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Bug or Regression?
Regression
Description
Using https://github.com/lbonn/rofi , rofi -show window can no longer switch windows with 17339e0ae9ad01dcfdede76e8dfaf516a8cbb924.
How to reproduce

Install rofi-lbonn-wayland-git
Run rofi -show window
Select a window and press enter.
Observe that no window switching happens.

Crash reports, logs, images, videos
hyprland-rofi-logs.log

I believe that's a rofi bug then.
I don't know how it works, but if rofi calls focuswindow while the LS is still open (not unmapped) then that will now be ignored (because rofi has exclusive keyboard focus) which is adhering to the spec.

exclusive keyboard focus

Hyprland could very well switch to the window while still forwarding keyboard events to rofi.

I'm inclined to think this as a Hyprland bug, as the window-switching works in sway... but we may be better off to fix it in rofi.

(This bemenu equivalent still works for anyone adversely affected, CC-0)
#!/bin/sh
hyprctl dispatch focuswindow address:"$(hyprctl -j clients | jq -r '.[]|(.class+" "+.title+"\t"+.address)' | bemenu | cut -f2)"

Hyprland could very well switch to the window while still forwarding keyboard events to rofi.

no, not really. The definition of window focus is keyboard focus

(This bemenu equivalent still works for anyone adversely affected, CC-0)
#!/bin/sh
hyprctl dispatch focuswindow address:"$(hyprctl -j clients | jq -r '.[]|(.class+" "+.title+"\t"+.address)' | bemenu | cut -f2)"

I also added so that the windows who don't have a title to not appear in the list and used rofi -dmenu instead
hyprctl dispatch focuswindow address:"$(hyprctl -j clients | jq -r '.[]|select(.address != ""  and .title != "")|(.class + "\t" + .address + "\t" + .title)' | rofi -dmenu | cut -f2)"


sql error

hi,when i add a new item to the menu.i got this error,please help,thank you!
"SQLSTATE[42S22]: Column not found: 1054 Unknown column 'internal_url' in 'field list' (SQL: insert into benfreke_menumanager_menus (title, description, enabled, is_external, internal_url, external_url, link_target, parameters, query_string, url, nest_left, nest_right, updated_at, created_at) values (asd, , 1, 0, , , _self, , , , 1, 2, 2015-02-21 06:46:15, 2015-02-21 06:46:15))" on line 614 of D:\xampp\htdocs\octobercms\vendor\laravel\framework\src\Illuminate\Database\Connection.php

Yeah, I've fixed that but haven't had a chance to push the 1.4.0 branch as
I'm having issue with the translation part of it.
Temporary fix is to add internal_url and external_url to the purgeable
attribute on the class, and use the Purgeable trait.
Should have a fix by this time tomorrow, assuming I can figure out the
translation bug ....
On 20 March 2015 at 15:50, shaohualee<EMAIL_ADDRESS>wrote:

hi,when i add a new item to the menu.i got this error,please help,thank
you!
"SQLSTATE[42S22]: Column not found: 1054 Unknown column 'internal_url' in
'field list' (SQL: insert into benfreke_menumanager_menus (title,
description, enabled, is_external, internal_url, external_url, link_target,
parameters, query_string, url, nest_left, nest_right, updated_at,
created_at) values (asd, , 1, 0, , , _self, , , , 1, 2, 2015-02-21
06:46:15, 2015-02-21 06:46:15))" on line 614 of
D:\xampp\htdocs\octobercms\vendor\laravel\framework\src\Illuminate\Database\Connection.php
—
Reply to this email directly or view it on GitHub
https://github.com/benfreke/oc-menumanager-plugin/issues/43.

thank you


Extend the CI to make sure the code is sound

sifis-generate template has a snippet to use miri or asan or valgrind for the purpose.
see here

Done in #5


Create a management command to count API requests, create/update ApiBlock records, and send notifications

Overview
Create a management command to:

Count API requests
Create/update ApiBlock records
Send notification emails about changes to ApiBlock records

Describe the solution you'd like
Use unit tests to verify the multiple logic branches
The OAR moderation team will create the text for the email notifications.
NOTE: Make sure to use transactions where appropriate so that running this command multiple times concurrently will not result in an invalid state.

Add API_FREE_REQUEST_LIMIT to settings.py with a value of 50.
Add NOTIFICATION_EMAIL_TO to settings.py. Read the value from the NOTIFICATION_EMAIL_TO environment variable.
Add a placeholder value of<EMAIL_ADDRESS>for the NOTIFICATION_EMAIL_TO environment variable in docker-compose.yml
Create a check_api_limits management command with no arguments
Put the logic of the command in a testable check_api_limits(at_datetime) function

Management command calls this function passing UTC now as at_datetime

Add a ContributorNotifications model

Non-null OneToOneField to Contributor
api_limit_warning_sent_on - nullable datetime
api_limit_exceeded_sent_on - nullable datetime
api_grace_limit_exceeded_sent_on - nullable datetime

check_api_limits logic:

Count the RequestLog records with a status code between 200 and 299 created during the month at_datetime is within grouped by Contributor
For each Contributor that made requests`

Get or create a ContributorNotifications instance.
Get the request limit by checking if there is an ApiLimit record for the Contributor

If there is no limit, use the API_FREE_REQUEST_LIMIT from settings

The the limit is zero, do nothing. Zero represents unlimited usage.
If the count is over 90% of the limit but less than the limit, and ContributorNotifications.api_limit_warning_sent_on is null or in a previous month.

Send a user-facing notification email to the Contributor.admin

If the email is sent successfully, update  ContributorNotifications.api_limit_warning_sent_on to the current time

Send an admin-facing notification email to the NOTIFICATION_EMAIL_TO address

If the count is over the limit

Check if there is an existing, ApiBlock for the Contributor where at_datetime is less than or equal to ApiBlock.until. There may be multiple blocks so sort by until descending and look at the first block.
If there is an existing block

If the block is active, do nothing.
If the block is not active and grace_limit is non null, compare the count

If the count is less than grace_limit do nothing
If the count is greater than grace_limit

Set active to true
Send a user-facing notification email to the Contributor.admin

If the email is sent successfully, update  ContributorNotifications.api_grace_limit_exceeded_sent_on to the current time

Send an admin-facing notification email to the NOTIFICATION_EMAIL_TO address

If there is not an existing block

Create an active block with until set to the first day of the next month minus 1 second UTC
ContributorNotifications.api_limit_exceeded_sent_on is null or in a previous month.

Send a user-facing notification email to the Contributor.admin

If the email is sent successfully, update  ContributorNotifications.api_limit_exceeded_sent_on to the current time

Send an admin-facing notification email to the NOTIFICATION_EMAIL_TO address

@TaiWilkin I updated the description with a feature that I realized was omitted during our initial design meeting. We intend to run the check_api_limits management command on a schedule. If we send an email notification, we need to record that it was sent so that we don't keep sending repeat notifications every time we run check_api_limits.
My solution was to add a ContributorNotifications table with nullable date fields for each type of notification. Please let me know if you can think of another way of implementing this.


Cursor Sync

Description
I have two sets of candlestick data, one with a 1-minute interval and another with a 4-minute interval. However, when synchronizing the crosshair, I found that the last minute of the 1-minute interval's candlestick is not synchronized with the corresponding candlestick of the 4-minute interval. Instead, it is synchronized with the next candlestick.
The synchronization code I am using is as follows:
 cursorUpdate(cursor) {
    console.log(this.CHART.cursor)
    let time = this.CHART.cursor.time
    if (this.PRE == time) return
    this.PRE = time
    let value = this.CHART.layout.main.y2value(cursor.y)

    for (let mychart of this.ALL) {
      if (mychart.TIMEFRAME == this.TIMEFRAME) continue;
      if (!mychart.CHART.layout.main) return;
      let b = JSON.parse(JSON.stringify(cursor))
      let x = mychart.CHART.layout.main.time2x(time)
      let y = mychart.CHART.layout.main.value2y(value)
      b.x = x
      b.y = y
      mychart.CHART.cursor = b
    }
  }

Suggested solution

In the second chart, the 1-minute candlestick should be the first candlestick of the 4-minute interval. However, in the first chart, the 1-minute candlestick is also being synchronized with the 4-minute candlestick, which is clearly incorrect.
Alternative
No response
Additional context
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

What about 5min chart, did you try?

What about 5min chart, did you try?

I'm glad to receive your response, and thank you for your great work. Night Vision is indeed a fantastic framework that satisfies most of my requirements. In my design, the data download module only fetches 1-minute data, and data for other time intervals is calculated through downsampling. As a result, there is no 5-minute time interval available. Since my synchronization is based on timestamp synchronization, the x-axis of each chart with a different time interval is not the same. Although Night Vision provides an example of cursor synchronization, it doesn't exactly match my situation. Additionally, during subsequent testing, I discovered that synchronizing smaller time intervals to larger ones presents issues, but synchronizing larger time intervals to smaller ones works fine.

Thanks. Yes, it seems that sync is not working (right now it gives you only x, y coordinates). For now you can try to play around t and time props:
// Cursor sync
// Subscribing to the events of the first instance
chart1.events.on("app:$cursor-update", (cursor) => {
  chart2.cursor = chart1.cursor;
  chart2.cursor.time = chart1.cursor.time + 1000000
  chart2.cursor.ti = chart2.cursor.ti + 1000000
});

(the code is not working, but It ca be a path to solve the issue)

Thanks. Yes, it seems that sync is not working (right now it gives you only x, y coordinates). For now you can try to play around t and time props :
// Cursor sync
// Subscribing to the events of the first instance
// Copy cursor from another chart instance instead of `cursor` event
chart1.events.on("app:$cursor-update", (cursor) => {
  chart2.cursor = chart1.cursor;
  chart2.cursor.time = chart1.cursor.time + 1000000
  chart2.update()
});

(now it is working)

Okay, I'll give the new approach a try. By the way, what is the difference between 'time' and 'ti'?

ti is for indexBased mode (stocks, renko etc)

I have tried the new method, but testing suggests that it still doesn't seem to work. So for now, I am still using my own method. Currently, there is only a small issue when synchronizing from a smaller timeframe to a larger timeframe, but there are no issues when synchronizing from a larger timeframe to a smaller one. Overall, the problem is not significant at the moment. If there are any fixes for this issue in the future, please mention them in the version update notes. Thank you for your response.


kafka: add shared consumer manager

Commit Message: kafka: add shared consumer manager
Additional Description: Stub for shared consumer manager. In future, SCM is going to be interacted-with by incoming Fetch requests, which are going to register themselves as callbacks for inbound fetch records. Part of https://github.com/envoyproxy/envoy/issues/24372.
Risk Level: Low
Testing: unit tests, integration tests with full stack
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

/retest

/retest

/retest

/assign @mattklein123

Thank you!


[WFLY-20218] Upgrade WildFly Core to 27.0.0.Final

Jira issue: https://issues.redhat.com/browse/WFLY-20218

Tag: https://github.com/wildfly/wildfly-core/releases/tag/27.0.0.Final
Diff: https://github.com/wildfly/wildfly-core/compare/27.0.0.Beta7...27.0.0.Final

        Bug

[WFCORE-7117] -         LayersTest does not takes into account JBoss Modules layers

        Task

[WFCORE-7095] -         Deprecate ModuleDependency.getIdentifier() for removal

[WFCORE-7111] -         Prepare org.jboss.as.server.deployment.Attachments for removal of ModuleIdentifier

[WFCORE-7112] -         Deprecate PermissionsParser.parse that takes ModuleIdentifier

[WFCORE-7113] -         Deprecate AdditionalModuleSpecification.getModuleIdentifier()

[WFCORE-7114] -         Deprecate use of ModuleIdentifier in ModuleDefinition

        Component Upgrade

[WFCORE-7109] -         Upgrade Logback to 1.5.13

[WFCORE-7119] -         Upgrade WildFly Maven Plugin to 5.1.1.Final

        Enhancement

[WFCORE-7108] -         Add a ModuleIdentifierUtil method to parse a module identifier into name and slot and process the tuple

Team City CI status: https://ci.wildfly.org/project.html?projectId=WF_PullRequest&branch_WF_PullRequest=pull%2F18598


How to update labels using your own label?

I tried to train with my own database. The annotations, train, val are all converted to coco type. I also updated the mmdetection/configs/dcn/faster_rcnn_dconv_c3-c5_r50_fpn_1x.py, and mmdetection/mmdet/datasets/coco.py based on my own labels and training set. But the strange thing is the original labels from coco (such as truck, person, car) remains in the testing image when I run the demo code from official https://github.com/open-mmlab/mmdetection#test-images I am pretty sure that I use the trained model. I am pretty sure that there is no error in my dataset (with coco style). Does anyone know the reason?

Found the reason, because I also need to update the mmdetection/mmdet/core/evaluation/class_names.py

You can refer to the testing script.

I had changed my own database to the coco type ,but maybe the way I changed have some problem, when I use run the demo code with the checkpoint from my train,
it warns"UserWarning: Class names are not saved in the checkpoint's meta data, use COCO classes by default.
warnings.warn('Class names are not saved in the checkpoint's ' "
but it worked out, it showed the classes of my data in the result.
when I run the test command with"python tools/test.py configs/faster_rcnn_r50_fpn_1x.py work_dirs/epoch_12.pt
h --out ./result/result_12.pkl --eval bbox --show"
It reported error that AttributeError: 'FasterRCNN' object has no attribute 'CLASSES'. How can I solve the  issue?


SignalR Get Started - v7 prep

Internal Review - versions 7, 6 and 3-5
Prep for #28460 SignalR with JavaScript: Update to 7.0 VS - VSC - VS Mac

Move SignalR Getting Started tutorial to new versioning by file system using includes, set all code and art paths accordingly.
Some clean up where numbered steps jumped from numbers to bullet points towards the end on version 6.

@tdykstra, just a prep to version by file before doing a 7.0 update, but I cleaned it up a little too, so hence the review request.

LGTM aside from a couple of minor comments.
What's the motivation for removing numbers from steps?

Rick preferred I don't do numbered steps.  So I have been avoiding them.
I have come to appreciate why they don't always work well:

For tabbed tutorials like this one, it is problematic to used numbered steps.  For example: Let's say there are 10 steps total in a section, and twice in those set of steps there are tabs for 3 different dev environment/IDE, but the number of steps for the tabbed sections differ.  One has 2 steps for VS and another 3 steps for VSC or 4 steps for VS for Mac.  The numbered steps end up being off, and then you see work arounds, like seemed to be done in this doc originally, where it jumps back and forth between numbers and bullets to make it all work out after the tabs.
We also use aspnetcore/includes a lot in this repo.  So when those includes have steps, the the formatting of numbers/vs bullets also don't match.
I personally like numbered steps in tutorials since it is easier to see steps one needs to take sorted out of other background information that is being provided, plus a quick reference to which step you left off on or had trouble on, and for consistency since the style guide recommends them, except in certain cases.  However, per points above, they don't always fit well with tab variations on the procedural content and/or includes.

Update this topic and app samples to .NET 7.0

Why does this need a .NET 7 version, what's changed? Why not update to .NET 8 which is LTS?
If there are code changes, then the prep PR should include a copy of the old code so the next PR has only the differences.

Update this topic and app samples to .NET 7.0

Why does this need a .NET 7 version, what's changed? Why not update to .NET 8 which is LTS?
If there are code changes, then the prep PR should include a copy of the old code so the next PR has only the differences.

The target version and sample would change at least to 7.  I need to look over what changed and test, which is part of the work/time reserved for this issue.  Then it is off to 8.0.

Verified in doc build that the code is all showing properly.

The target version and sample would change at least to 7. I need to look over what changed and test, which is part of the work/time reserved for this issue. Then it is off to 8.0. This is just the prep to organize before I do version 7.

Skip 7 and go straight to 8


Skip SELinux tests in CI by default

Currently, no CI job has SELinux configured, so skip all tests that require it by default.. When a job supports SELinux, it can set CSI_PROW_E2E_SKIP='Disruptive' in its prow yaml.
Fixes: https://github.com/kubernetes-csi/csi-driver-host-path/issues/376
Needs https://github.com/kubernetes/kubernetes/pull/113809 merged first.
NONE

/priority critical-urgent


Resnet18 decoder version

As title, do you have the resnet18 encoder and decoder version? (with BasicBlock)
Thanks!

Hi, sorry we currently don't have the ResNet18 version.


Update http4s-prometheus-metrics to 0.24.5

About this PR
📦 Updates org.http4s:http4s-prometheus-metrics from 0.21.34 to 0.24.5
📜 GitHub Release Notes
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

💡 Applied Scalafix Migrations

org.http4s:http4s-.*:0.22.0

github:http4s/http4s/v0_22?sha=series/0.22

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s", artifactId = "http4s-prometheus-metrics" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.http4s", artifactId = "http4s-prometheus-metrics" }
}]

labels: library-update, early-semver-major, semver-spec-minor, version-scheme:early-semver, scalafix-migrations, commit-count:n:2

Superseded by #1578.


Removed deletion of billing address on purchase

This fixes a problem where the billing address was deleted and not automatically sent through the authorize call options.

@schwartzdev please review and merge.

Looks good, but we actually want to merge this into 2-1-stable-dotandbo branch

nm my previous comment. merged.


chore(dist): replace profile id & unify the dependencies/plugins

@javeme this PR will merge until the release done, then use rebase & merge option

@javeme this PR will merge until the release done, then use rebase & merge option

get it


Support for installing splunk apps from local file

Cookbook version
1.7.3
Chef-client version
14.5.33
Platform Details
CentOS 7.5 (VirtualBox via Vagrant)
Scenario:
Feature Request: Ability to install Splunk Apps via local file instead of remote file
Steps to Reproduce:
Assume that we download the app package (.tar.gz) onto the local system through a method aside from remote_file (ex. download from an S3 bucket or copied onto a system through an out of band method), I'd like to point to a local path (ex. /tmp/splunk_special_app.tar.gz) and install directly.
Expected Result:
Using the remote_file method fails, even if I specify file://tmp/splunk_special_app.tar.gz.  I would expect the splunk_app resource to look something like this:
splunk_app 'sanitycheck' do
local_file '/tmp/sanitycheck.tar.gz'
splunk_auth 'admin:notarealpassword'
action :install
end

You can use remote_file 'file:///tmp/sanitycheck.tar.gz'

I think this can be done rather easily; however, with so many ways to "unpack" a compressed bundle file (e.g., tar.gz, zip, bz2), this feature should not attempt to support any/all of the possibilities. In contrast, this feature should support installing an app from any local source on the chef node and into the /opt/splunk/etc/apps directory, unless otherwise specified by the app_dir property.


Not a drop in replacement

Okay, to be fair, I am pretty new at Go, so pardon my naiveté.
I am trying to use http_dialer#New as a drop in replacement for a constructor that can accept a parameter of type *net.Dialer. However, when I use it, I get:
cannot use http_dialer.New(forwarder.Proxy, nil) (type *http_dialer.HttpTunnel) as type *net.Dialer in field value

I tried casting it (net.Dialer(*http_dialer.New(forwarder.Proxy, nil))), but I get:
cannot convert *http_dialer.New(forwarder.Proxy, nil) (type http_dialer.HttpTunnel) to type net.Dialer

Okay, I think I am getting closer. I tried setting Dial of the client:
client.Dialer.Dial = http_dialer.New(
					forwarder.Proxy,
					http_dialer.WithDialer(client.Dialer),
				).Dial

where client is an instance of github.com/miekg/dns.Client
However, now I get:
cannot assign to client.Dialer.Dial


osdn: Fix out of range panic

Panic found in http://www.fpaste.org/292728/44802319/

LGTM, thanks

LGTM


Upgrade to Ruby 2.7.7

Motivation

This fixes some problems that I have building Ruby 2.7.2 on my Ubuntu
machine (with rbenv installed via "Homebrew for Linux").
Test plan

See included automated tests.

In the background I'll take care of pushing a new sorbet-build-image docker image that pre-installs 2.7.7. Our buildkite config files should all take care of installing the version that they need.


ArgumentNullException in Inventory.cs

Apparently sometimes the Items array is null?

[Shaun The Sheep 2015-02-28 11:32:11] WARN: Unknown error occurred: System.ArgumentNullException: Argument cannot be null.
Parameter name: source
in System.Linq.Enumerable.Where{TSource}(IEnumerable1 source, Func2 predicate)
in SteamTrade.Inventory.GetItemsByDefindex(Int32 defindex) in X:\SteamBot-master\SteamTrade\Inventory.cs:line 79

Ideas?

You are using outdated SteamBot. This has been fixed.

Happens randomly with my own inventory which is public though (close/reopen 'cause wrong button)

Well, hard to tell where the error is since the line 79 from your error output does not match with the current codebase.

Running a version of inventory.cs w/o the new privatechecks in getItem and getItembydefindex, line 79 is this:
            return Items.Where(item => item.Defindex == defindex).ToList();

Well, Items is null so the call fails.
In this case the inventory response must have been empty (?), although I don't remember this happening to me ever.
Would be probably best to log the actual response to see what's going on.

I know it's null, that's why I opened an issue bro ;d
idk why this happens, as said it does randomly, with the bot's inventory - not some other user's. Pretty frickin weird, re-trading after the trade dies fixes it. I don't even ....

That's why I'm a complete noob in networking

Well, there is not much that can be done. I suspect the API sometimes returns "200 OK" with response body { }. We could either try to re-fetch the inventory (how many tries before we give up?) or just end the trade and try again which is IMO better option.

Yea, what I hate the most about it is that users tend to quickly lose patiency, especially if they're in a hurry, so having something like that happen is pretty much -rep for the bot, like this one on my bot's profile:

Shadow 27 Feb @ 8:44pm
-rep
Will later have a look at the inventory fetching thingy and re-try fetching a couple times.
If I find a good and reliable fix, I'll make a PR maybe, unless it's gonna be fixed by then. :P


scaleByDistance: NearFarScalar not working for Point

The scaleByDistance for a point is only scaling correctly in 1 direction, if you switch the values to scale larger instead of smaller while zooming in it only scales to half of what it is supposed to and it stops earlier than it should
Try this Sandcastle example and zoom in and watch both points size, 1 stops changing early and at wrong size unlike the other point which correctly changes size across the whole zoom distance

Ah I think it is working ok, optical illusion makes it appear they are different but they end up the same size at the end, would be nice to be able to define the scaling function instead of just a linear scale, I will play with this some more


unref slice before cancel

This could fix (maybe partially) #3578
The test involved was maxmessagesize test without proxy or tls.


why add-in giving error After deployment On Azure

After deployment on Azure add-in always giving me error this is my live manifest

It is always taking 5 to 10 minutes then it is working

Hi @junaid042,
Given the error message, it seems like you're trying to access office.js before it's initialized. Please make sure you're following the guidance listed in these articles:

Load the DOM and runtime environment
Initialize your Office Add-in

@AlexJerabek  This is react add-in when I run it locally or host it on github then its working profect but when I host it to Azure Storage Account then this is giving me this error

@junaid042,
Sounds like you might have a race condition when Initializing your Add-in. There is somewhere that uses Office.js before the onReady event is triggered. It's hard for me to say more without understanding your add-in. If you can find the code generating that error and are still stuck, I'd recommend posting that on Stack Overflow.
Since this isn't a problem with the documentation, I'm going to close this issue. If after further troubleshooting, you believe you've found a product bug, please raise the issue of the office-js repo. Thank you.

@AlexJerabek thanks for your help I don't know why this was giving error when I host it on Azure Office.onReady(() => {}) is added in index.tsx file

I have fixed this by adding Office.onReady(() => {}) in taskpane.html file now its working fine


Update repository url of tfjs-core

deeplearnjs is now moved to tfjs-core under TensorFlow organization.

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Thank you!


refactor: move hnsw from lance crate to lance-index crate

In the process of move all index code to lance-index.

Codecov Report
Attention: Patch coverage is 81.25000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 79.46%. Comparing base (7d43030) to head (f32f3cd).

Files
Patch %
Lines

rust/lance-index/src/vector/quantizer.rs
60.00%
0 Missing and 2 partials :warning:

rust/lance/src/index/vector/ivf/io.rs
50.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2446      +/-   ##
==========================================
- Coverage   79.46%   79.46%   -0.01%
==========================================
  Files         205      204       -1
  Lines       56975    56972       -3
  Branches    56975    56972       -3
==========================================
- Hits        45273    45270       -3
  Misses       9014     9014
  Partials     2688     2688

Flag
Coverage Δ

unittests
79.46% <81.25%> (-0.01%)
:arrow_down:

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

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


Version detection for GoogleDrive & Bitwarden

When using Installomator 10.3 to update apps it re-installs GoogleDrive even though the version is the same as that being installed
For Bitwarden, it re-installs 2022.5.1 even though the latest version is already installed.
The -Force flag is not being used

It looks like the Bitwarden label points to a github repository that's been archived Jul 22 2022, hence the 2022.5.1 version: https://github.com/bitwarden/desktop
Now all releases are done through https://github.com/bitwarden/clients/release (with the latest now being 2023.4.0) and the label will need to change to reflect that. I'll try working on a fix but I'm not sure if the multiple releases for different platforms is going to affect how Installomator can grab these.

Just to be clear, the default Installomator behavior with labels missing appNewVersion is to install regardless of the force flag. More info here: https://github.com/Installomator/Installomator/wiki/Configuration-and-Variables#install-behavior-force-installation
For the Google Drive part, the only source of a version number on the web from an official source is https://support.google.com/a/answer/7577057?hl=en but that only gives 75.0, when currently the latest installed version detected by installomator for me now is <IP_ADDRESS>. I can find that version number on 3rd party sites, like from here: https://community.chocolatey.org/packages/googledrive but I'm not sure if we'd want to check 3rd party sites like that since I haven't seen that done much elsewhere. Any opinions on that @scriptingosx ?


Arrow triangles for directional LineStrings should be proportionate to scale

The triangles drawn to make LineStrings into arrowed routes are too large when at a high zoom level. They need to be proportionate to something, but it's not clear to me what that should be. It really depends on the pixel-length of the lines rather than the actual zoom level.

For the moment, I've taken a different approach to this: triangles are only added to LineString segments longer than 25px on the screen. Implemented in commit #59e5e70. May reopen if this doesn't prove ideal, or if it's deemed essential to add arrows to the first and last segments irrespective of their length.


navbar_added

Addressing issue #4
Branch-name = navbar
Added Navbar for homepage

Hey @MukulTan10, thank you for creating this pull request! Have a wonderful day...

hey , issue is arrissing cause of changing header in route with new navbar and replacing it with new, you can review the code , if you would like to replace content of routes.php to old one , i can make another push for same

hye the conflict in routes file is because of replacing the old header with new navbar, you can review it and if you want i can replace it to old file and warning is because of image that is not used anywhere as much i seen .you can review if you feel any changes need to be made please write back

Hi @MukulTan10,
It seems like some issue with your code which throws a warning by our Deepcode bot. I would like to advise you to use Functional components instead of Class components. Can you change the index.js file to a functional based component? Thanks in advance. Have a nice day.
BTW, I resolved the merge conflict and when you fixed the issue I will merge it into the master branch.

hey , i have resolved the issue , warning issue we are getting is because of search bar ( i have take the pull of merge branch) , can you ask the developer who made the search to resolve this,attacing you warning description in this too . if i can do anything please let me know

hey , i have resolved the issue , warning issue left is because of search bar ( i have take the pull of merge branch) , can you ask the developer who made the search to resolve this,attaching warning description in this too . if i can do anything please let me know

You haven't resolved the issue that I have asked for. You have to convert the class component to a functional component.
Please update this file index.js and commit again.
 Use the following lines.
$ git pull upstream master

hey , i have converted the class component to functional


feature: add owasp zap as dast tool

A working implementation of DAST - Owasp ZAP. It scans the staging environment with a full scan by default.
The report will be uploaded to the development S3 bucket inside /report/security/dast. We will keep all the reports in a hierarchy inside /report.
Added optional ZAP config files for the rules and work-in-progress tickets.

Also added basic auth token variables depending on the environment, as they are different.

[AUTOMATED] Latest version has been pushed to preview.
Preview deployed at https://preview-ea3924d.dev.nextjs-boilerplate.jam3.cloud.jam3.net

[AUTOMATED] Latest version has been pushed to preview.
Preview deployed at https://preview-2c66c78.dev.nextjs-boilerplate.jam3.cloud.jam3.net

Closing this since the PR has been open for more than 1 year
Let's re-open if this gains traction at some point
Thx


Animation Library saves animations as binary .res instead of text .tres by default?

Godot version
v4.0.2.stable.official [7a0977ce2]
System information
Windows 11, Core i5-12400F, RX 6600
Issue description
When click the disk icon and select "Save As" in Animation Library:

dialog has .res set as the extension and saves as binary
if change ext to .tres saves as text like 'normal'
i have never seen anything use that binary format by default in godot
so i thought it was an oversight? maybe not

Steps to reproduce
n/a
Minimal reproduction project
n/a

You do not want to have animations stored by default in a slow and bloated text format. Having binary as the default for animations protects beginner users from performance issues caused by the choice of file format.
Most new users do not know or understand the differences between text (e.g .tscn and .tres) and binary (.scn and .res) until it is already too late and they have a lot of performance issues with their assets stored in inefficient formats.
Especially in the case of skeletons with bones you are talking about thousands of transforms and small float numbers stored in an inefficient text file format that is very slow to load at runtime and consumes a lot of project disk space.
In my opinion even more resources should be changed to binary by default to protect beginners from making bad default choices.

all .tres are slow and bloated not just anim? idk
my animations are small and simple 2d sprite animations
never too late to convert
idc

If you have a small 2D animation it usually does not matter but the text format scales poorly with more complex and heavy files.

I am storing over a hundred animations for my main player as text, organized into several animation libraries based on type (base walk/idle, armed versions, attacks, crouching, torch holding, etc) for our 3D adventure game. "text scales poorly... facing performance issues" has not been my experience.
We use tres so that if I change even one animation parameter, like a name or loop mode, I don't have to upload the whole binary file to git. Instead just the minor difference is uploaded. I have long recommended every resource in Godot be text for teams that use git, except textures, sounds, and anything Godot does not fully support in text (e.g. Meshes.)
Meshes work fine as text, but the core devs keep changing the format and breaking it without an upgrade path, requiring manual reimporting and extraction of all assets every time. I've been through it 3 times on hundreds of assets. So now I include glbs stripped of textures, and for npcs that will share animations, they are also stripped out of the glb and stored separately. Animations for say an animal I leave embedded in the glb.

I've been through it 3 times on hundreds of assets.

Since 4.0 on the stable branches?

No, stable was just released. I've been using Godot a long time. They started this process of breaking mesh formats w/o upgrade paths back in 3.x stable. And because they haven't yet committed to including an upgrade path, they can and will break it again on some future release.

Unfortunately the very nature of binary formats is that they are susceptible to these kinds of compatibility breaking changes, and the main focus or guarantee is to prevent such (to the greatest possible degree, one cannot always account for bugs) in stable sequences of releases, providing upgrade paths is an important thing though
Are we talking breaking changes between 3.x versions? Or between 2.x, 3.x, and 4.x?

I was talking about changes to the format of meshes stored in Godot's native text resource files. Nothing to do with binary formats. Anyway this has gotten off topic. The thread is about storing animations as res or tres by default, and I shared my experience that we have hundreds stored as tres just fine, with the added benefit of being git friendly.

My bad we were talking about binary formats and the issues with them so I assumed you were talking about them


SSH version problem

Windows Terminal version
1.11.2921.0
Windows build number
10.0.19043.1288
Other Software
No response
Steps to reproduce
If you use SSH protocol to connect to Alibaba cloud fortress, you will be prompted with the wrong version of the client
Expected Behavior
No response
Actual Behavior
(SSH-2.0-OpenSSH_for_Windows_8.1) SSH key-exchange error: Invalid major version number: SSH-2.0-OpenSSH_for_Windows_8.1

This doesn't necessarily sound related to the Windows Terminal - I'd suggest filing this over at https://github.com/PowerShell/Win32-OpenSSH/issues


⚠ Jenkins Plugins Artifactory has degraded performance

In 654095f, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1398 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 87dbafd after 17 minutes.


gatk-launch fails after ./gradlew clean shadowJar

This is because the name for the shadow jar differs between public vs. protected, and gatk-launch looks for the name used in public. We should just have protected use the suffix local.jar like in public.

What's the target for building the spark jar, sparkJar like for gatk?

Yes, sparkJar. Just need to add a localJar target, and adjust the name of the created jar, to fix this issue.

OK I should have just tried before asking; it works :)
And gatk-launch uses it correctly too, for the record.


Ability to send custom headers

I want to send send custom headers as I request a url.

ops forget about the request. thanks


Sequence diagram view not visible on the docs page

Describe the bug
Whereas on Github diagrams like the one for the Offchain transaction signatures for the tx-service are displayed as expected:

On the Docs page this and other diagrams following the same structure are not:

Will be resolved with switch to Nextra.


Read / import mc2 from url in order to execute it by the widget

Read / import mc2 from url in order to execute it by the widget (geoquantumviz for example).
In this case, you case edit the mc2 file locally (with any editor) and execute it inside the widget without copy/paste (with all error that <ctrl+c><ctrl+v> implies...)

It's done in the last release of Quantum


Canvas integration not returning citation as the link name when a CAL portion is selected

Describe the bug
When a portion is added to Moodle, the link name becomes the citation.  (See screenshot)
When added to Canvas, the filename is being used.
To Reproduce

TODO: find out which methods of adding to Canvas exhibit this issue.

Expected behavior
CAL portions should display as the citation text in Canvas
Screenshots
Moodle:

Canvas:

Possible point to look at is to use link.getDescription() around CanvasIntegration.java:392
            // e.g.
            retUrl.append("&title=");
            retUrl.append(URLEncoder.encode(link.getName(), "UTF-8"));

            // e.g. ${text}
            retUrl.append("&text=");
            retUrl.append(URLEncoder.encode(link.getName(), "UTF-8"));


nvcc failed with exit status 1

I have Ubuntu 18.04, python 3.6.9, and I've tried installing this in a virtual environment with each combination of pytorch 1.3.1 and 1.4.0 and cuda 10.1 and 10.2.
I put symlinks to gcc and g++ 8 in my cuda/bin directory.
When I run pip install . in the root directory I get:
"6 errors detected in the compilation of "/tmp/tmpxft_0000137f_00000000-6_searchsorted_cuda_kernel.cpp1.ii".
error: command '/usr/bin/nvcc' failed with exit status 1"
(the actual number of errors here was different between cuda 10.1 and cuda 10.2).
Some of the errors during compilation:
"    /home/andrew/.virtualenvs/nerf/lib/python3.6/site-packages/torch/include/torch/csrc/api/include/torch/enum.h(164): error: no instance of function template "c10::visit" matches the argument list
argument types are: (torch::enumtype::_compute_enum_name, torch::nn::KLDivLossOptions::reduction_t)
detected during:
instantiation of "std::string torch::enumtype::get_enum_name(V) [with V=torch::nn::KLDivLossOptions::reduction_t]"
(176): here
instantiation of "at::Reduction::Reduction torch::enumtype::reduction_get_enum(V) [with V=torch::nn::KLDivLossOptions::reduction_t]"
/home/andrew/.virtualenvs/nerf/lib/python3.6/site-packages/torch/include/torch/csrc/api/include/torch/nn/functional/loss.h(49): here
/home/andrew/.virtualenvs/nerf/lib/python3.6/site-packages/torch/include/torch/csrc/api/include/torch/enum.h(164): error: no instance of function template "c10::visit" matches the argument list
            argument types are: (torch::enumtype::_compute_enum_name, torch::nn::MultiLabelSoftMarginLossOptions::reduction_t)
          detected during instantiation of "std::string torch::enumtype::get_enum_name(V) [with V=torch::nn::MultiLabelSoftMarginLossOptions::reduction_t]"
/home/andrew/.virtualenvs/nerf/lib/python3.6/site-packages/torch/include/torch/csrc/api/include/torch/nn/functional/loss.h(336): here

/home/andrew/.virtualenvs/nerf/lib/python3.6/site-packages/torch/include/torch/csrc/api/include/torch/enum.h(164): error: no instance of function template "c10::visit" matches the argument list
            argument types are: (torch::enumtype::_compute_enum_name, torch::nn::functional::PadFuncOptions::mode_t)
          detected during instantiation of "std::string torch::enumtype::get_enum_name(V) [with V=torch::nn::functional::PadFuncOptions::mode_t]"
/home/andrew/.virtualenvs/nerf/lib/python3.6/site-packages/torch/include/torch/csrc/api/include/torch/nn/functional/padding.h(40): here

/home/andrew/.virtualenvs/nerf/lib/python3.6/site-packages/torch/include/torch/csrc/api/include/torch/enum.h(164): error: no instance of function template "c10::visit" matches the argument list
            argument types are: (torch::enumtype::_compute_enum_name, torch::nn::functional::InterpolateFuncOptions::mode_t)
          detected during instantiation of "std::string torch::enumtype::get_enum_name(V) [with V=torch::nn::functional::InterpolateFuncOptions::mode_t]"
/home/andrew/.virtualenvs/nerf/lib/python3.6/site-packages/torch/include/torch/csrc/api/include/torch/nn/functional/upsampling.h(103): here"

Any ideas?

Were you able to solve this? I'm getting similar errors.

could you try with latest commit ?

any news on this ? did you try again with latest commit ?

Hello,
I am getting similar errors while trying to install. Below the error text.
/usr/include/c++/8/bits/basic_string.tcc: In instantiation of ‘static std::basic_string<_CharT, _Traits, _Alloc>::_Rep* std::basic_string<_CharT, _Traits, _Alloc>::_Rep::_S_create(std::basic_string<_CharT, _Traits, _Alloc>::size_type, std::basic_string<_CharT, _Traits, _Alloc>::size_type, const _Alloc&) [with _CharT = char16_t; _Traits = std::char_traits<char16_t>; _Alloc = std::allocator<char16_t>; std::basic_string<_CharT, _Traits, _Alloc>::size_type = long unsigned int]’:
  /usr/include/c++/8/bits/basic_string.tcc:578:28:   required from ‘static _CharT* std::basic_string<_CharT, _Traits, _Alloc>::_S_construct(_InIterator, _InIterator, const _Alloc&, std::forward_iterator_tag) [with _FwdIterator = const char16_t*; _CharT = char16_t; _Traits = std::char_traits<char16_t>; _Alloc = std::allocator<char16_t>]’
  /usr/include/c++/8/bits/basic_string.h:5052:20:   required from ‘static _CharT* std::basic_string<_CharT, _Traits, _Alloc>::_S_construct_aux(_InIterator, _InIterator, const _Alloc&, std::__false_type) [with _InIterator = const char16_t*; _CharT = char16_t; _Traits = std::char_traits<char16_t>; _Alloc = std::allocator<char16_t>]’
  /usr/include/c++/8/bits/basic_string.h:5073:24:   required from ‘static _CharT* std::basic_string<_CharT, _Traits, _Alloc>::_S_construct(_InIterator, _InIterator, const _Alloc&) [with _InIterator = const char16_t*; _CharT = char16_t; _Traits = std::char_traits<char16_t>; _Alloc = std::allocator<char16_t>]’
  /usr/include/c++/8/bits/basic_string.tcc:656:134:   required from ‘std::basic_string<_CharT, _Traits, _Alloc>::basic_string(const _CharT*, std::basic_string<_CharT, _Traits, _Alloc>::size_type, const _Alloc&) [with _CharT = char16_t; _Traits = std::char_traits<char16_t>; _Alloc = std::allocator<char16_t>; std::basic_string<_CharT, _Traits, _Alloc>::size_type = long unsigned int]’
  /usr/include/c++/8/bits/basic_string.h:6725:95:   required from here
  /usr/include/c++/8/bits/basic_string.tcc:1067:1: error: cannot call member function ‘void std::basic_string<_CharT, _Traits, _Alloc>::_Rep::_M_set_sharable() [with _CharT = char16_t; _Traits = std::char_traits<char16_t>; _Alloc = std::allocator<char16_t>]’ without object
         __p->_M_set_sharable();
   ^     ~~~~~~~~~
  /usr/include/c++/8/bits/basic_string.tcc: In instantiation of ‘static std::basic_string<_CharT, _Traits, _Alloc>::_Rep* std::basic_string<_CharT, _Traits, _Alloc>::_Rep::_S_create(std::basic_string<_CharT, _Traits, _Alloc>::size_type, std::basic_string<_CharT, _Traits, _Alloc>::size_type, const _Alloc&) [with _CharT = char32_t; _Traits = std::char_traits<char32_t>; _Alloc = std::allocator<char32_t>; std::basic_string<_CharT, _Traits, _Alloc>::size_type = long unsigned int]’:
  /usr/include/c++/8/bits/basic_string.tcc:578:28:   required from ‘static _CharT* std::basic_string<_CharT, _Traits, _Alloc>::_S_construct(_InIterator, _InIterator, const _Alloc&, std::forward_iterator_tag) [with _FwdIterator = const char32_t*; _CharT = char32_t; _Traits = std::char_traits<char32_t>; _Alloc = std::allocator<char32_t>]’
  /usr/include/c++/8/bits/basic_string.h:5052:20:   required from ‘static _CharT* std::basic_string<_CharT, _Traits, _Alloc>::_S_construct_aux(_InIterator, _InIterator, const _Alloc&, std::__false_type) [with _InIterator = const char32_t*; _CharT = char32_t; _Traits = std::char_traits<char32_t>; _Alloc = std::allocator<char32_t>]’
  /usr/include/c++/8/bits/basic_string.h:5073:24:   required from ‘static _CharT* std::basic_string<_CharT, _Traits, _Alloc>::_S_construct(_InIterator, _InIterator, const _Alloc&) [with _InIterator = const char32_t*; _CharT = char32_t; _Traits = std::char_traits<char32_t>; _Alloc = std::allocator<char32_t>]’
  /usr/include/c++/8/bits/basic_string.tcc:656:134:   required from ‘std::basic_string<_CharT, _Traits, _Alloc>::basic_string(const _CharT*, std::basic_string<_CharT, _Traits, _Alloc>::size_type, const _Alloc&) [with _CharT = char32_t; _Traits = std::char_traits<char32_t>; _Alloc = std::allocator<char32_t>; std::basic_string<_CharT, _Traits, _Alloc>::size_type = long unsigned int]’
  /usr/include/c++/8/bits/basic_string.h:6730:95:   required from here
  /usr/include/c++/8/bits/basic_string.tcc:1067:1: error: cannot call member function ‘void std::basic_string<_CharT, _Traits, _Alloc>::_Rep::_M_set_sharable() [with _CharT = char32_t; _Traits = std::char_traits<char32_t>; _Alloc = std::allocator<char32_t>]’ without object
  error: command '/usr/local/cuda-10.1/bin/nvcc' failed with exit status 1

what is your gcc/g++ version?

gcc 8, I think  it's  because  of cuda version.

I reinstalled cuda (nvcc) by making gcc-8/g++-8 as default gcc/g++ compilers, then the problem gone.

great!


Generate usage graphs for all methods within all classes

Builds on #37, for review check commit "Add list of class names to java projects" and newer.
__
This PR adds a method to generate to generate library usage graphs for a whole user project (i.e. all methods of all classes).

Codecov Report

Merging #38 into master will increase coverage by 0.87%.
The diff coverage is 88%.

@@             Coverage Diff              @@
##             master      #38      +/-   ##
============================================
+ Coverage     80.35%   81.22%   +0.87%
- Complexity      148      159      +11
============================================
  Files            15       16       +1
  Lines           280      309      +29
  Branches         79       86       +7
============================================
+ Hits            225      251      +26
  Misses           24       24
- Partials         31       34       +3

Impacted Files
Coverage Δ
Complexity Δ

.../kotlin/org/cafejojo/schaapi/common/JavaProject.kt
0% <ø> (ø)
0 <0> (?)

...schaapi/usagegraphgenerator/filters/ValueFilter.kt
60.97% <100%> (ø)
30 <2> (ø)
:arrow_down:

...schaapi/usagegraphgenerator/UsageGraphGenerator.kt
88.23% <100%> (+2.52%)
0 <0> (ø)
:arrow_down:

...api/usagegraphgenerator/filters/StatementFilter.kt
92.3% <100%> (+10.48%)
12 <4> (+4)
:arrow_up:

...i/usagegraphgenerator/filters/IfStatementFilter.kt
80% <80%> (ø)
6 <6> (?)

...fejojo/schaapi/projectcompiler/JavaMavenProject.kt
62.79% <83.33%> (-3.12%)
14 <1> (ø)

...api/usagegraphgenerator/ControlFlowGraphCreator.kt
66.66% <0%> (+16.66%)
6% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


Access KeyVault in a web application (Gatsby React)

When I try to setup the access to key vault I get the error
Unhandled Rejection (Error): EnvironmentCredential is not supported in the browser.

For reference I have my code that is encapsulated in a class like:
import { DefaultAzureCredential } from '@azure/identity';
import{ SecretClient } from '@azure/keyvault-secrets';

export default class AzureKeyVault {
    private keyVaultUri: string;
    private keyVaultClient: SecretClient;
    constructor(keyVaultName: string) {
        this.keyVaultUri = `https://${keyVaultName}.vault.azure.net`;
        // https://docs.microsoft.com/en-us/azure/key-vault/secrets/quick-create-node
        // The DefaultAzureCredential method in our application relies on three environmental variables: AZURE_CLIENT_ID, AZURE_CLIENT_SECRET, and AZURE_TENANT_ID
        const credential = new DefaultAzureCredential();
        this.keyVaultClient = new SecretClient(this.keyVaultUri, credential);
    }
    get Auth0Secret() {
        return (async () => await this.keyVaultClient.getSecret('Auth0Secret'));
    }
    get Auth0ClientId() {
        return (async () => await this.keyVaultClient.getSecret('Auth0ClientId'));
    }
    get Auth0Domain() {
        return (async () => await this.keyVaultClient.getSecret('Auth0Domain'));
    }
    get WodiftyPassword() {
        return (async () => await this.keyVaultClient.getSecret('WodiftyPassword'));
    }
    get WodiftyUser() {
        return (async () => await this.keyVaultClient.getSecret('WodiftyUser'));
    }
    get GoogleMapApiKey() {
        return (async () => await this.keyVaultClient.getSecret('GoogleMapApiKey'));
    }
}

Then I use the class like:
    const getMapApiKey = async() => {
        const keyVault = new AzureKeyVault('GreatStateKeyVault');
        const mapsKey = await keyVault.GoogleMapApiKey;
        setGoogleMapsApiKey(mapsKey.name);
    }

I this what was intended?

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

ID: 4342d30f-dc5d-d723-de88-909ab22e1a85
Version Independent ID: c9877a7d-27ec-cbdc-2abb-ef04711cd56b
Content: Quickstart -  Azure Key Vault client library for Node.js (v4)
Content Source: articles/key-vault/secrets/quick-create-node.md
Service: key-vault
Sub-service: secrets
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@KevinBurton
Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

@KevinBurton, Thanks a lot for reaching out. It would be great if you can post this request in Microsoft Q&A platform, as questions of this nature, is best when posted there as there we have a lot more visibility.
We recommend using this forum to mostly go ahead with doc enhancements and questions related to the product. But questions where you need help with code etc, its better to post it on Microsoft Q&A platform.

Closing this thread for now


rename walletPath -> daedalusBin and remove walletArgs

Let it be known that there are last-minute changes that I'm not responsible for and I would be surprised if everything worked correctly the first time.
From the list of issues that I stumbled when trying to fix the tests:

added the "real" configuration files that are going to be used
env variable APPDATA is missing
renaming statePath to stateDir

Also, this assumes that Daedalus will magically know where to find the Jormungandar executable and the correct arguments.
@disassembler @cleverca22


Using Sundials CVODE solver in MATLAB via QSPcc

Hi @ciropom,
I am Deepa, I had a question on Sundials forum yesterday, and I would like to get some help here.
Thank you for the amazing library!
I would like to use QSPcc for two tasks: (1) to establish the Sudials interface for solving odes in MATLAB (2) to solve an optimization problem (currently using SBML model + COPASI's evolutionary algorithm). I am also reading through your paper and I understand QSPcc offers great support for both my tasks. I would like to start with the first task.
For calling CVODE in MATLAB, I want try the second method you suggested on the Sundials forum.
i.e. generate the MEX code and call the CVODE solver from MATLAB.
Please let me know if I should first use the cvode C code and generate the MEX file via QSPcc.
Could you please direct me to an example on how to proceed?
I've downloaded the docker from here https://www.cosbi.eu/prototypes/qspcc. I'm on Windows, could you please let me know if I need WSL?

Hello Deepa,
thank you for your message.
The first problem is the operating system: I understand that your MATLAB installation is on windows, and QSPcc works on windows only through docker (better with WSL), but anyway your MATLAB can't access qspcc inside the docker.
The simplest thing you can do is to do a pure-C simulation and optimization in windows with the docker.
But, if you really need MATLAB to run the optimization, you should follow the technical guide to install qspcc directly on your computer without docker
https://github.com/cosbi-research/QSPcc#method-2-custom-installation-technical
And use linux or macosx that are officially supported by QSPcc (while windows is not).
Another problem, as I was mentioning in the Sundials mailing list where this discussion originated,
is that the qspcc module that allows the auto-generation of the MEX interface is not open-source.
You can still translate your model to pure-C code with the open-source QSPcc and write manually only the MEX interface following the Mathworks MEX guide
Note also that if the model you have is in SBML format,
I advise you to first translate SBML in MATLAB using SBFC
and then use QSPcc on the SBFC-generated MATLAB code to get the QSPcc-generated C code,
and at this point you are only left with the task of writing the MEX interface manually
or go with the pure-C translation, that is fully automatic.

Hello @DeepaMahm
I wanted to inform you that the latest release of QSPcc includes the ablity to translate out-of-the-box also
optimization problems through lsqnonlin MATLAB function.
This would allow you to write a MATLAB program to do the simulation and optimzation, and translate everything in C with QSPcc without the need to use any other software.
For a minimal example see this optimization + simulation test


Ain't able to train text to 3d in low memory!

hi,
I'm currently training your work on RTX3090 with 24G memory.
when I tried "python main.py --text "a hamburger" --workspace trial -O --vram_O --w 300 --h 300" , my CUDA was out of memory.
but according to your comment, you tested it on 8G GPU.
I'm using default SD version(2.1).
what may cause this issue? did I miss anything?

This can happen if this is your only GPU and something else is using up some of the memory. I ran into the same issue and using the second GPU that wasn't running my Ubuntu desktop fixed it. I suppose somehow there is a way to cap it so it doesn't try to use 100% of the memory.

thanks for reply! I do have second GPU, also RTX3090, which memory was only used 10M before I run this training.
still have same issue.
currently, I 'm only able to run it with 128x128 resolution, which costs 19G memory. so basically, resolution on longer can be increased.
and if I run it without "--vram_O", it will still cost 19G for 128x128 resolution.
so seems like "--vram_O" does not work in my case.

@csawtelle Hi, actually I have changed some network structure after vram_O (revert to finite difference for normal evaluation, which increases memory cost largely but makes the current as_latent geometry initialization work), so the information is not correct in readme. I'll correct and improve it later.

In case anyone else is interested, I also ran into the same issue:
python main.py --text "a hamburger" --workspace trial -O --vram_O --w 64 --h 64

Condition   Mem usage reported by nvidia-smi
---------   --------------------------------
idle         3.2GB
Varying h/w:
  8x8        6.4GB
 64x64      10.6GB
128x128     18.4GB
256x256     OOM on RTX4090 24GB VRAM


Update ingress-controller-annotations.md to cover all current annotations

Proposing documentation updates to AGIC Annotations to a current complete list based on https://github.com/Azure/application-gateway-kubernetes-ingress/blob/master/pkg/annotations/ingress_annotations.go
Currently, the page is very dated and missing many annotations.

@microsoft-github-policy-service agree company="Microsoft"

Closing to correct markdown errors.


Support for Custom Spans

I'm excited to read about PHP support for more granular tracing at https://www.instana.com/blog/illuminate-critical-php-code-with-custom-spans/ and tried to find out how to set it up for the nodejs sensor.
Is this kind of use possible with an Instana NodeJS sensor? I did try to look up from the code and docs but failed to find a reference.

Hi ahtik,
currently this is not supported in Node.js. We have support for OpenTracing (https://github.com/instana/nodejs-sensor#opentracing) but OpenTracing spans can never be combined with spans created by out automatic tracing capabilities, so our OpenTracing support will not give you the same functionality as described in the PHP blog post.
If you would like to see that functionality added to the Node.js sensor, would please create a feature  request at https://support.instana.com/hc/en-us/requests/new for it? Alternatively, if you are using npm packages that we currently do not instrument, you could also add feature requests for auto-instrumentation for them.
Kind regards
Bastian

Hi, I see this issue is very old, are there any updates on combining automatic tracing and custom spans in nodejs?
I see docs still suggest that with open tracking automatic tracing should be disabled https://docs.instana.io/ecosystem/node-js/api/#opentracing-integration
Also one suggestion for docs, would be good to explain why to disable automatic tracing and what is result in one or the other case, currently it just says "should" but no explanation what is the outcome if not disabled.
Thanks!

@amirilovic-adidas looks like that's possible! I have created some example and it's even possible to link custom spans to automatic spans
Information about usage can be found here: https://www.instana.com/blog/instana-announces-support-opentracing/
I think documentation should reflect this use case as it can be really useful!

This issue is indeed very old. In the meantime, we have added an SDK to create custom spans: https://docs.instana.io/ecosystem/node-js/api/#creating-spans-manually-with-the-sdk
Custom spans created via the SDK integrate seamlessly with auto-tracing spans. On the other hand, combining auto-tracing spans with OpenTracing is not officially supported.
Thus, if you want to enhance Instana's Node.js auto tracing with custom spans, the SDK is the way to go.
This is also reflected in the documentation you linked to (see the section "Limits"):

OpenTracing is not integrated with Instana’s automatic tracing. In particular, spans created by using the OpenTracing API will not be part of the same trace as the spans created by our automatic tracing instrumentation. If you want to add additional spans to the automatically created spans, you should prefer the SDK over OpenTracing. In fact, we recommend to either use automatic tracing (optionally augmented by SDK spans) or OpenTracing, but not both in one application.

@RecuencoJones @basti1302  Thanks for the info, will try to add to my app. Cheers!