Use Transport Request

Description
Remove usage of deprecated BaseNodeRequest
Signed-off-by: Vijayan Balasubramanian<EMAIL_ADDRESS>Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #163 (fdcae01) into main (1eebc3f) will increase coverage by 0.28%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #163      +/-   ##
============================================
+ Coverage     84.85%   85.13%   +0.28%
- Complexity      319      320       +1
============================================
  Files            44       44
  Lines          1063     1063
  Branches         86       86
============================================
+ Hits            902      905       +3
+ Misses          126      124       -2
+ Partials         35       34       -1

Impacted Files
Coverage Δ

...eospatial/stats/upload/UploadStatsNodeRequest.java
0.00% <ø> (ø)

...earch/geospatial/index/mapper/xypoint/XYPoint.java
74.13% <0.00%> (+5.17%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

As part of this OpenSearch issue https://github.com/opensearch-project/OpenSearch/issues/2747 BaseNodeRequest class is removed in 3.0 and deprecated in 2.x . Use TransportRequest which was super class for BaseNodeRequest as an alternative.


Camera tutorial updates

new tutorial for time-lapse tutorial on thursday july 15

see failing checks @catherine-m-breen you need to clear the notebooks of output before submitting (timelapse-camera-tutorial.ipynb cell 21: execution count
Also since you changed the name of the notebook you'll have to update the table of contents: /home/jovyan/book/tutorials/day_4.md:7: WARNING: toctree contains reference to nonexisting document 'tutorials/camera-traps-tutorial/camera-traps-notebook'

see failing checks @catherine-m-breen you need to clear the notebooks of output before submitting (timelapse-camera-tutorial.ipynb cell 21: execution count
Also since you changed the name of the notebook you'll have to update the table of contents: /home/jovyan/book/tutorials/day_4.md:7: WARNING: toctree contains reference to nonexisting document 'tutorials/camera-traps-tutorial/camera-traps-notebook'

Katie note: Thank you for the tip on TOC. I should have fixed that now. I also cleared the kernel.

Looks like you have cloned the  github_setup_check inside the webfolder and committed that.
Could you remove this again?
A simple
rm -rf  github_setup_check

of the folder followed by a commit should do it.

For the folium map code block, I learned about tags for the first time. It looks like this needs to be added as cell metadata, and not as code within the block? It took me a long time to figure out that cell metadata is accessed on the little gears button, upper right side of Jupyter Lab. But I still didn't succeed in getting this to work!

Otherwise this looks great and I think we can merge it.

@catherine-m-breen couple requests:

could you please remove the bold formatting for your title?  # **Time-lapse Cameras and Snow Applications**
you have a repeat header ## Time-lapse Camera Applications (can remove the second)

@catherine-m-breen couple requests:

could you please remove the bold formatting for your title?  # **Time-lapse Cameras and Snow Applications**
you have a repeat header ## Time-lapse Camera Applications (can remove the second)
aws s3 sync --no-progress s3://snowex-data/tutorial-data/camera-trap/ /tmp/camera-trap currently pulls 4.3GB of data. can we keep this small (using aws s3 cp to get only images you display and work with later in the notebook?

@catherine-m-breen couple requests:

could you please remove the bold formatting for your title?  # **Time-lapse Cameras and Snow Applications**
you have a repeat header ## Time-lapse Camera Applications (can remove the second)
aws s3 sync --no-progress s3://snowex-data/tutorial-data/camera-trap/ /tmp/camera-trap currently pulls 4.3GB of data. can we keep this small (using aws s3 cp to get only images you display and work with later in the notebook?

thank you @catherine-m-breen and @cassielumbrazo looks great!

i accidentally merged this in w/o checking that the notebook was cleaned of output :( will try to remedy after tmrw's session. it's not a big issue (renders fine on the website, but does add bloat to the total repo size). Need to fix the CI a bit...

Let me know if you need help on this.
One note regards the repo size - Once committed you will have a hard time reducing that again without changing the original commit, which complicates the effort.
Maybe we can just leaves this as is for the hackweek to reduce potential major merge conflicts with participants and have a look at the end of the week regards the repository size.


Fix sorting admin user list by last login (#22081)

Backport of https://github.com/go-gitea/gitea/pull/22081

🎺 🤖


How to move object from Anchor to Anchor

My use case is:
tap on the screen and save the "point" as starting anchor
tap on the screen second time and save the "point" as end anchor
push the button that will move the object from starting to end anchor
I've built my own node that is using ObjectAnimator similar like in the solar system example. My only problem is that I do not know how to determine start and end point for the the evaluator. My first thought was to take the x,y,z from Pose of start and end anchor
Vector3 start = new Vector3(startAnchor.getPose().tx(), startAnchor.getPose().ty(), startAnchor.getPose().tz());
Vector3 end = new Vector3(endAnchor.getPose().tx(), endAnchor.getPose().ty(), endAnchor.getPose().tz());

…
movingAnimation.setObjectValues(startingPoint, endPoint);
movingAnimation.setPropertyName("localPosition");
movingAnimation.setEvaluator(new Vector3Evaluator());

but when I do that animation is done from completely different places.
I havent found any reference to inbuild tools for such operation.
Im using sceneform
So the question is. How to make a fluent animation (a simple slide is enough) from anchor A to anchor B

I haven't tested this, but I would approach this problem as follows:
// Set the animated node as a child of the destination anchor.
    animatedNode.setParent(anchorB);

    // Set the world position of the animated node to the world position of the initial anchor.
    Vector3 anchorAPosition = anchorA.getWorldPosition();
    animatedNode.setWorldPosition(anchorAPosition);

    // Animate the local position of the node towards zero to move it to the position of it's parent (anchorB).
    Vector3 start = animatedNode.getLocalPosition();
    Vector3 end = Vector3.zero();
    movingAnimation.setObjectValues(start, end);
    movingAnimation.setPropertyName("localPosition");
    movingAnimation.setEvaluator(new Vector3Evaluator());

What's important about this is that the animation is done in the local coordinate space of anchorB, this allows the animation to occur while still respecting the tracking of the anchor.

Closing due to inactivity.


Enchantment Applicator applies everything

Industrial Foregoing Version: 1.5.6-93
TeslaCoreLib Version: 1.0.11
Applied a enchanted book with a lot of stuff on it to a pair of boots, and it ended up apply every enchant, despite only 1 of them actually being a valid enchant.

The Mending/Silk Touch is a 2nd book I applied after to test it out.
So dunno if this should count as a bug, or a suggestion, but I'd think it should only apply valid enchants.

I think I will leave it like that, it might be useful in some situations.

Yeah like duping enchants :^) I can write another report on that if you want

How?

FWIW, I'm on the side of leaving it in.  In the pack I'm currently playing, it's the only way to apply a Fortune V enchantment from a book without it reverting down to Fortune III.

To make Fortune V you need 4 Fortune III and you need to work a bit to get them so it's not that bad

I already have a Fortune 5 book made from an enchant I pulled off of another tool.  If I attempt to apply that to a new tool with any means other than the Enchantment Applicator, it applies as Fortune 3.  I assume if you changed the current behavior, it would behave the same as other enchantment methods, like the anvil, and do the same.

But thats not duping 🤔

I know.  I was arguing for keeping the current functionality in.  I don't know how you'd use it for duping. @Lukas45 would need to explain that one.

I'm not against having it able to apply Fortune V without downgrading, it's more that it applies Fortune V to a pair of boots.
As for the dupe, the Enchant Applicator accepts any item, and will apply to any item. So if you put an enchanted book for application with Fortune V, and a stack of sticks (actually only tested with normal books, and not in a standalone client), it will create a stack of sticks enchanted with Fortune V, and each of those sticks can be extracted again.
I did get half a stack of books enchanted (not enchanted books, books with enchants) with what I have on those boots in the picture (excl. the mending/silk touch).
This can also lead to server danger, as I put 4 books (stacked) with enchants into the extractor, and it removed the enchant from all of them (only gave 1 enchanted book, and one book as output), so there was now an unenchanted item in the input slot (still had nbt tag for "no enchants"), which caused the server to crash on ticking entity. It had to be MCEdited out. Can link server crash log tomorrow.

I will limit the slot amount to 1 on this machines to fix the dupe bug.


Error Connecting to Power BI Premium Workspace

Hello All,
I am working on a Project for my client in which we are using Power BI Actions tool from Azure DevOps Market place to create a release pipeline in which we have to Use Tabular Editor.
I followed the link https://docs.tabulareditor.com/te2/Command-line-Options.html and tried to connect to Power BI Workspace.
But seems there is hard time connecting. Every time I want to deploy to Power Bi workspace I am getting error that Tabular Editor "Cannot find the workspace"
Here is the syntax of command that I am using.
Start-Process -filePath TabularEditor.exe -Wait -NoNewWindow -PassThru -ArgumentList ""Provider=MSOLAP;DataSource=powerbi://api.powerbi.com/v1.0/myorg/T&ESCentralDataModeling Recruiting Data Model;User ID=app:AppID@TenantID;Password=XXXX;Persist Security Info=True;Impersonation Level=Impersonate" RecruitingDataModel -Deploy C:\Users\n1566969\source\repos\Model.bim"
Please help me resolving this issue and let me know how can I deploy using the Tabular Editor to Power BI workspace.

Try to replace myorg in the connection string, with the full name of your org. Also, make sure that the Service Principal has admin access to the workspace, and that Service Principal XMLA access has been enabled for your tenant, and that XMLA read/write access in general is enabled.

I am running into the same issue as #835, and have done the steps you mentioned here. Can you give an example of the full powershell command?
What I have is the following:
$p = Start-Process -filePath "$workDir/TE/TabularEditor.exe" -Wait -NoNewWindow -PassThru -ArgumentList "`"./Model.bim`" -D `"Provider=MSOLAP;DataSource=powerbi://api.powerbi.com/v1.0/myorg/PBIPremiumPerUserWorkspace;User ID=appName:app-UUid@tenant-uuid;Password=AADRegisteredAppSecret;`" `"DataSetOnPBI`" -O -C -E -W -P"

Where $workDir is the path of the directory with all our necessary files.
This gives the following error message
Deployment failed! Unable to obtain authentication token using the credentials provided. If your Active Directory tenant administrator has configured Multi-Factor Authentication or if your account is a Microsoft Account, please remove the user name and password from the connection string, and then retry. You should then be prompted to enter your credentials.
This will be part of a pipeline so I can't do the authentication, and need it to work headlessly.
Any help is appreciated, thanks!

@nader2929 I see you have User ID=appName:app-UUid@tenant-uuid in the connection string. Please change this to:
User ID=app:<appId>@<tenantId>
The app: part should be left as is. The <appId> and <tenantId> parts are the respective GUIDs. More info here.

Ah perfect! That worked a charm, thanks for the quick response and help.


create soap and wsdl

Laravel Version: 5.4.*
PHP Version: 7
Database Driver & Version:

Description:
I want to know if it is possible to create a soap server with wsdl
Steps To Reproduce:

Yes.

alugna sugeerencia..?
Any suggestion..?

@blueblazerchrist this repo is for bug tracking. Use the forums or the slack channel to solve your issues

Use the forums for such issues.

Sorry @blueblazerchrist did you find a solution for your issue? I'm trying to create a soap web service with laravel but with no success.
Thanks in advance.


Fix orpo trainer loss device

What does this PR do ?
Same PR as https://github.com/huggingface/trl/pull/1439 but for ORPOTrainer.
TLDR: It's a very strong assumption that tr_loss and tr_loss_step are on the same device. arg.device may not be as same as the current device.
Fixes: https://github.com/huggingface/transformers/pull/32480
cc @kashif

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

I'm unable to merge as I don't have write access @kashif. Can you merge it please ?


Add new class Set for simple Set-theoretical operations

Features

Works only with primitive types int, float, string
Implements set theortic operations union, intersection, complement
Modifies set by adding, removing elements
Implements \IteratorAggregate for use in loops

Implementation details
Based on array functions:

array_diff,
array_merge,
array_intersection,
array_unique,
array_values,
sort.

Drawbacks

Do not work with objects.
Power set and Cartesian product returning array of Set

Thanks. You can add link to docs/math/set.md in readme and docs files.


os/mac/diagnostic: output non-default Xcode prefix.

This should help people find it when this is a fatal error (and they don't look at brew config).

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

2018-11-28T16:06:26.3139510Z   1) Cask::Quarantine by default quarantines Cask audits
2018-11-28T16:06:26.3141020Z      Failure/Error: Cask::Cmd::Audit.run("local-transmission", "--download")
2018-11-28T16:06:26.3141640Z
2018-11-28T16:06:26.3142550Z      Cask::CaskError:
2018-11-28T16:06:26.3143740Z        audit failed for casks: local-transmission
2018-11-28T16:06:26.3144880Z      # ./cask/cmd/audit.rb:17:in `run'
2018-11-28T16:06:26.3146010Z      # ./cask/cmd/abstract_command.rb:34:in `run'
2018-11-28T16:06:26.3147190Z      # ./test/cask/quarantine_spec.rb:32:in `block (3 levels) in <top (required)>'
2018-11-28T16:06:26.3148320Z      # ./test/support/helper/spec/shared_context/homebrew_cask.rb:38:in `block (2 levels) in <top (required)>'
2018-11-28T16:06:26.3149380Z      # ./test/spec_helper.rb:137:in `block (2 levels) in <top (required)>'
2018-11-28T16:06:26.3150430Z      # ./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.6.1/lib/rspec/retry.rb:123:in `block in run'
2018-11-28T16:06:26.3151490Z      # ./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.6.1/lib/rspec/retry.rb:110:in `loop'
2018-11-28T16:06:26.3152510Z      # ./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.6.1/lib/rspec/retry.rb:110:in `run'
2018-11-28T16:06:26.3153560Z      # ./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.6.1/lib/rspec_ext/rspec_ext.rb:12:in `run_with_retry'
2018-11-28T16:06:26.3154620Z      # ./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.6.1/lib/rspec/retry.rb:37:in `block (2 levels) in setup'
2018-11-28T16:06:26.3155690Z      # ./vendor/bundle/ruby/2.3.0/gems/rspec-wait-0.0.9/lib/rspec/wait.rb:46:in `block (2 levels) in <top (required)>'
2018-11-28T16:06:26.3156410Z
2018-11-28T16:06:26.3156880Z   2) Cask::Quarantine when disabled does not quarantine Cask audits
2018-11-28T16:06:26.3157900Z      Failure/Error: Cask::Cmd::Audit.run("local-transmission", "--download", "--no-quarantine")
2018-11-28T16:06:26.3158380Z
2018-11-28T16:06:26.3158820Z      Cask::CaskError:
2018-11-28T16:06:26.3159740Z        audit failed for casks: local-transmission
2018-11-28T16:06:26.3160700Z      # ./cask/cmd/audit.rb:17:in `run'
2018-11-28T16:06:26.3161720Z      # ./cask/cmd/abstract_command.rb:34:in `run'
2018-11-28T16:06:26.3162720Z      # ./test/cask/quarantine_spec.rb:145:in `block (3 levels) in <top (required)>'
2018-11-28T16:06:26.3163770Z      # ./test/support/helper/spec/shared_context/homebrew_cask.rb:38:in `block (2 levels) in <top (required)>'
2018-11-28T16:06:26.3164800Z      # ./test/spec_helper.rb:137:in `block (2 levels) in <top (required)>'
2018-11-28T16:06:26.3165840Z      # ./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.6.1/lib/rspec/retry.rb:123:in `block in run'
2018-11-28T16:06:26.3166890Z      # ./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.6.1/lib/rspec/retry.rb:110:in `loop'
2018-11-28T16:06:26.3167910Z      # ./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.6.1/lib/rspec/retry.rb:110:in `run'
2018-11-28T16:06:26.3168950Z      # ./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.6.1/lib/rspec_ext/rspec_ext.rb:12:in `run_with_retry'
2018-11-28T16:06:26.3170010Z      # ./vendor/bundle/ruby/2.3.0/gems/rspec-retry-0.6.1/lib/rspec/retry.rb:37:in `block (2 levels) in setup'
2018-11-28T16:06:26.3171070Z      # ./vendor/bundle/ruby/2.3.0/gems/rspec-wait-0.0.9/lib/rspec/wait.rb:46:in `block (2 levels) in <top (required)>'

@Homebrew/cask could one of you take a look at this? It seems to be failing pretty consistently.

It seems that the TLS certificate served by https://example.com/ has expired. 😱😛

@claui Ouch! Any chance of a PR to update the relevant casks?

Ouch! Any chance of a PR to update the relevant casks?

@MikeMcQuaid https://github.com/Homebrew/brew/pull/5359


fixed GPG key error in Dockerfile

Motivation
I resolved https://github.com/open-mmlab/mmdetection3d/issues/1498.
Modification
I fixed GPG key. I applied https://github.com/open-mmlab/mmsegmentation/pull/1534.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

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

resolved in https://github.com/open-mmlab/mmdetection3d/pull/1508


authMiddleware() apiRoutes shows incorrect warning

Brief Summary of the Issue
Clerk Next.js middleware authMiddleware() is showing an incorrect warning in server logs when using apiRoutes to protect routes (which works as expected)

See: https://clerk.com/docs/references/nextjs/auth-middleware?utm_source=www.google.com&utm_medium=referral&utm_campaign=none#options

Name
Type
Description

apiRoutes?
string[]
A list of routes that should return 401. You can use glob patterns to match multiple routes or a function to match against the request object. For example: ['/foo', '/bar(.)'] or [/^/foo/.$/]

Minimal Reproduction or Replay
Zipped project: clerk-next-app-main.zip

Add a .env.local with your test keys
Run npm install
Run server with npm run dev
Go to http://localhost:3000/api/test (change port if needed)
Look for warnings in the server log

It works well, I get a 401 as expected, but I also get this warning on every request (which is wrong, and fills the logs):
INFO: Clerk: The request to /api/test is being redirected because there is no signed-in user, and the path is not included in `ignoredRoutes` or `publicRoutes`. To prevent this behavior, choose one of:

1. To make the route accessible to both signed in and signed out users, add "/api/test" to the `publicRoutes` array passed to authMiddleware
2. To prevent Clerk authentication from running at all, add "/api/test" to the `ignoredRoutes` array passed to authMiddleware
3. Pass a custom `afterAuth` to authMiddleware, and replace Clerk's default behavior of redirecting unless a route is included in publicRoutes

For additional information about middleware, please visit https://clerk.com/docs/nextjs/middleware
(This log only appears in development mode, or if `debug: true` is passed to authMiddleware)

Browser/OS
NextJS (browser / node)
Package + Version

[X] @clerk/nextjs 4.23.3

Hello @theo-lubert 👋
I cannot reproduce what you described, also based on the error message in the issue description :

(This log only appears in development mode, or if debug: true is passed to authMiddleware)

Looking at the code here and here it seems that the info log is only visible when url is for a publicRoute (not a apiRoute) and is only generated when debug: true option is enabled.
I have also added screenshots using your codebase as is and with the debug: true option.

I don't think this should be closed
Reading the code, I understands this is meant to appear (when the route is protected).
Allright, but then  it message should be specific to apiRoutes in our case, because:

The warning indicates that it is being redirected, which it a false statement, it returns a 401
The developper is actively configuring the middleware to protect the route with apiRoute, and yet it is not mentioned from the message. A step should be added to help disable/update the apiRoutes config

I'm adding a PR to correct that, hoping it can help avoid some confusion

@theo-lubert i accidentally closed the issue (sorry about that).
Thank you for opening a PR to improve the info message. We will review this and come

@theo-lubert you will find the related changes to the latest release.
Thank you for the contribution 🎉

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Application not displaying anything when ran due to sourcemap loading failure and server error

Not sure what went wrong, I had only just git cloned the project and attempted to run the firebase serve command to run but encountered nothing(a blank page) on the localhost and these errors. Seems to have to do with some of these links. I'm a complete novice I was just trying to learn how to use firebase, so any help on how to fix this would be great to continue forward on the codelab.

Same thing is happening to me


Add demo

Codecov Report

Merging #32 (92ec11e) into master (9c906eb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   97.43%   97.43%
=======================================
  Files           7        7
  Lines         234      234
=======================================
  Hits          228      228
  Misses          6        6

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 9c906eb...92ec11e. Read the comment docs.


Missing data fro NY

Hy guys, yesterday data for NY from March 12 till March 22 disappeared.

Duplicate of https://github.com/lazd/coronadatascraper/issues/323

Yes it does seem like this is a dupe. Closing, @ekaterinakuzmina please follow along in the other issue.


Remove code related to s2i

In #4906 we're removing Service Catalog related code from odo. We also need to remove everything else that's tied to OCP 3.11. Few places to look at:

s2i code beyond what's helping with odo create --s2i
DeploymentConfigs, maybe?

/kind cleanup

So far I have found at least two objects that we can consider removing i.e deployment configs and build configs.
Deployment Configs have a usage in a bunch of places across the odo code base.
BuildConfigs seems to mostly be used for building from git and is linked to localconfig (the old odo config)
Both are heavily tied to s2i which we are currently supporting conversion for.
I would assume we want to keep localconfig structs around for a bit longer to support conversion of existing local config components.
So the first step would be to remove build configs and deployment configs from occlient (while removing usages across odo code). The only other openshift only resource is route but that is not heavily tied to s2i and is in fact used in devfile as well
wdyt @dharmit @mik-dass ?

BuildConfigs seems to mostly be used for building from git and is linked to localconfig (the old odo config)

@kadel how long should we support odo create --git and odo create --binary? How much do the Adapters folks rely on it? cc @mohitsuman

The only other openshift only resource is route but that is not heavily tied to s2i and is in fact used in devfile as well

Right. Route has to be kept as-is.

title got me confused. I'm not sure how 3.11 support is related to s2i
/title Remove code related to s2i
Lets be a little nice to our users:
First lets add a  deprecation notice to s2i related features

add deprecation warning when someone executes odo create --s2i
add deprecation warning when someone executes odo create --binary
add deprecation warning when someone executes odo create --git or odo create --ref

After the warning is there for at least one release we can start removing it.

removing all flags related to s2i from odo create
remove everything that touches DeploymentConfigs, BuildConfigs, ImageStreams
remove everything that touches LocalConfig .odo/config.yaml

I don't think we need to remove s2i support as is as we have devfile conversation for s2i.
But we should remove server side implementation that is openshift specific  as it's not likely to be used anymore due to conversion.
@kadel @dharmit wdyt

But for git I think it makes sense to deprecate git first

Based on the information I have gathered:

odo create --s2i converts to devfile automatically
odo push does not convert existing s2i component
While we have conversion code, we dont seem to have a command to convert s2i to devfile

I agree with @kadel that this will have to happen in 2 phases
1. Deprecation

odo push with s2i component should show deprecation and recommend the user to recreate component with --s2i flag to allow automatic conversion to devfile
usage of --git should throw deprecation warning that this will no longer be supported (unless we can support git for devfile)

2. Removal

Drop support and strip dcand buildconfigs. Image streams should continued to be supported for as long as we support conversion of s2i to devfile

Drop support and strip dcand buildconfigs. Image streams should continued to be supported for as long as we support conversion of s2i to devfile

The goal is to drop s2i to devfile conversion as well.

Drop support and strip dcand buildconfigs. Image streams should continued to be supported for as long as we support conversion of s2i to devfile

The goal is to drop s2i to devfile conversion as well.

I guess it still kinda holds true :D, just more quickly

/kind user-story

/remove-triage needs-information
/triage ready

Task 1: #4951
Task 2: 4952

/unassign

@kadel how long should we support odo create --git and odo create --binary? How much do the Adapters folks rely on it? cc @mohitsuman

@dharmit Regarding the workflow from adapters, it should be fine if we gradually deprecate the git and binary create component  support.

Replying to: https://github.com/openshift/odo/issues/4932#issuecomment-888071007

While we have conversion code, we dont seem to have a command to convert s2i to devfile

@mohammedzee1000 I think we do. There is subcommand convert-to-devfile under odo utils which seems to be doing just this.


Rename /slave to /agent

Listen to server error.

Reviewer: @aldipower

Need to get people to test this stuff because their UI isn't working because of backend changes. So I'm going to merge as there's no one around to review.


BroadCast Screen-Admin-App - Sprint 2

Description:

Build the Screen to match the Figma design.
Integrate Screen with the Backend

I'm also building this


Migrate to bevy 0.11

Hello!
Two problems with the migration:

I can't find the correct bevy feature combination to make it work with minimal features. As you can see in the Cargo.toml I tried and actually tried a lot of combinations, but without all of bevy I couldn't get the StandardMaterial to work and it was pink instead of black

I can't get the uv_mapping example to work (no git lfs problem, I have the texture), it simple doesn't appear.

Still wanted to open this, because maybe it's useful for someone and maybe someone will jump in to fix the problems mentioned

@nerdachse I think the pink texture issue is a result of missing the proper set of features to support compressed textures. I believe you need the features I added here:
https://github.com/bonsairobo/smooth-bevy-cameras/commit/171521759169ada15d83c3ebec02771af20f5552

Just a headsup: I am currently on vacation and will try this out once back in a few weeks :)
For me the example doesn't show at all though, not a pink texture

Took me a while, but it's now targeting 0.13 - I still can't get the textures to work though, even if I have a valid file on my PC (I can open it just fine) and everything seems to load in bevy.

Going to merge this into a local bevy-0.13 branch and see if I can fix the texturing issue.


How to generate MolecularSequence resource?

Greetings,
How to generate MolecularSequence resource for cancer patients?
https://www.hl7.org/fhir/molecularsequence.html
Thanks!

Sorry, Synthea cannot yet generate molecular or genomic data.

Thank you for the quick reply. I will find a workaround.


[BUG] Regression: "SAML assertion AudienceRestriction has no Audience value"

After upgrading to beta 3 I get this error: "SAML assertion AudienceRestriction has no Audience value". I don't have assertion in my logs, unfortunately I don't have assertion available to debug
To Reproduce
The only information I have for now is that the client uses OneLogin. The code is:
const { profile } = await saml.validatePostResponseAsync(req.body);
Expected behavior
It should work as before
Environment

Node.js version: 16
node-saml version: 4.0.0-beta.3

What was the fix?

It was me not reading changelog. I did not have audience parameter and defauld behaviour was changed


Default flowchart

The default new flowchart should include one node, connected to start and end. The end node should be selected.

Also, auto zoom to the flowchart so that it maximised.


Docs: Improve headings

Changed Getting started > Getting started to Getting started > Setup since it's bad practice to have duplicated headings. Also updated the page title to Set up Scenes to align with the new sidebar naming.
Changed Learn Scenes to Scene objects to make it more explicit about what kind of information will be in that section. All of the docs are aimed at helping you learn Scenes and it doesn't give you a clear idea of what content will be in that section.

Thanks @imatwawana!


use g++-12 compile with "-std=c++20"  ,but pre defined macro __cplusplus value 201402L

compile_commands.json
[
{
  "directory": "/home/guijiyang/Code/cxx/My/gcommon",
  "arguments": ["g++-12", "-c", "-fPIC", "-g", "-Wall", "-Werror", "-O0", "-std=c++20", "-I/home/guijiyang/Code/cxx/My/gcommon", "-fexceptions", "-o", "build/.objs/utils/linux/x86_64/debug/gcommon/ScopeGuard.cc.o", "gcommon/ScopeGuard.cc"],
  "file": "gcommon/ScopeGuard.cc"
}]

with -std=c++20 still mcaro __cplusplus wrong defined value  201402L.
when I try use some traits newer than c++14 like std::is_const_v or std::is_same_v， but clangd tips dont work.

System information
Output of clangd --version: clangd version 15.0.7
Editor/LSP plugin: vscode
Operating system: linux

Please share clangd logs

Please share [clangd logs]
Failed to find compilation database for /usr/local/gcc-12.2.0/include/c++/12.2.0/type_traits I[14:23:06.684] ASTWorker building file /usr/local/gcc-12.2.0/include/c++/12.2.0/type_traits version 1 with command clangd fallback [/usr/local/gcc-12.2.0/include/c++/12.2.0] /usr/local/llvm-15.0.7/bin/clang -xobjective-c++-header -resource-dir=/usr/local/llvm-15.0.7/lib/clang/15.0.7 -- /usr/local/gcc-12.2.0/include/c++/12.2.0/type_traits

Please see https://clangd.llvm.org/faq#how-do-i-fix-errors-i-get-when-opening-headers-outside-of-my-project-directory

Please see https://clangd.llvm.org/faq#how-do-i-fix-errors-i-get-when-opening-headers-outside-of-my-project-directory

thank you , and below text to clangd arguments, it worked normally now.
 --compile-commands-dir=/home/guijiyang/Code/cxx/My/gcommon


ClassCleanup should trigger after all test methods have been triggered

When running multiple [TestClass]es at the same time, why does the [ClassCleanup()] methods only get called at the very end?
File: UnitTest1.cs
[TestClass]
public class UnitTest1
{
[ClassInitialize()]
public static void ClassInit(TestContext context) { }
[ClassCleanup()]
public static void ClassCleanup() { }
[TestMethod]
public void TestMethod() { }
}
File: UnitTest2.cs
[TestClass]
public class UnitTest2
{
[ClassInitialize()]
public static void ClassInit(TestContext context) { }
[ClassCleanup()]
public static void ClassCleanup() { }
[TestMethod]
public void TestMethod() { }
}
The execution order is the following:

UnitTest2.ClassInit()
UnitTest2.TestMethod()
UnitTest1.ClassInit()
UnitTest1.TestMethod()
UnitTest2.ClassCleanup()
UnitTest1.ClassCleanup()

Notice that both [ClassCleanup] methods are executed at the end of the set; not after each TestClass is "done".
But I expected a different behavior:

UnitTest2.ClassInit()
UnitTest2.TestMethod()
UnitTest2.ClassCleanup() - Expected
UnitTest1.ClassInit()
UnitTest1.TestMethod()
UnitTest1.ClassCleanup() - Expected

Microsoft's Documentation - ClassCleanupAttribute Class says, "Identifies a method that contains code to be used after all the tests in the test class have run and to free resources obtained by the test class."
But it appears to be run/executed late, after ALL tests have run. This seems wrong, and it is blocking my development.

Hi, I think it is same issue as is #580

Hey folks. This is a super confusing behavior of MSTest! Basically ClassCleanup default mode is equivalent to AssemblyCleanup and is run only after the last test of the run. There is a property you can use to change this [ClassCleanup(ClassCleanupBehavior.EndOfClass)].
Note that we are planning to change the default in the next major release of MSTest (after v3), see https://github.com/microsoft/testfx/issues/1316


Not working well with Interactive Window

Currently, vscode-R and languageserver should work with Jupyter notebook along with IRkernel, yet they do not seem to work well with the Interactive Window.
The uri looks like vscode-notebook-cell:/Interactive-1.interactive#W1sdnNjb2RlLWludGVyYWN0aXZl which should be properly handled in vscode-R and languageserver.

Working with the Interactive Window (instead of Terminals) would be really great.


build-webrtc.py error

Expected behavior
for webtrc to build
Observerd behavior
Syncing projects:  74% (97/130) src/third_party/boringssl/src
src/third_party/android_ndk (ERROR)
[0:00:07] Started.
Error: 79>
79> ____ src/third_party/android_ndk at 4e2cea441bfd43f0863d14f57b1e1844260b9884
79> 	You have unstaged changes.
79> 	Please commit, stash, or reset.
Steps to reproduce the problem
git clone repo
mkdir ~/src
cd react-native-webrtc/tools
python build-webrtc.py --setup --android ~/src/
Platform information

React-native-cli: 2.0.1
React-native: 0.57.7
Plugin version: 1.67.1
OS:
OS version: osx 10.14.1

WebRTC for Android can only be built on Linux at the moment. https://github.com/oney/react-native-webrtc/blob/master/Documentation/BuildingWebRTC.md#android-1


[Suggestion] "Follow the Leader" (add Segment and Tail checkboxes to token config)

Hi oOve, in one of our previous conversations you had asked about how to move away from using token names to identify segments and the tail. I had suggested checkboxes at the time but the image path system you went with works fine and makes setup much easier.
However I saw a comment on Reddit about using this mod to have other tokens "follow" a main token and this got me thinking about checkboxes again.
I definitely wouldn't want to take away the image path method you have now. Would it be possible to keep that as the default, but if no image path is assigned there then the mod looks for any tokens on the canvas with a segment or tail checkbox enabled?
This would allow things like assigning a party leader and toggling on and off different party members, hirelings, other NPCs, animal companions, etc. to "follow the leader".

Not too hard to implement I suppose.. And then players could even select another players token and just click follow. If four tokens follow that one, then we have  a train going.. (hmm.  what if I follow you, and you follow me?)

what if I follow you, and you follow me?

Easy way I'd say they just cancel each other out with a console error.
Probably a better way would be to popup a dialog giving the option to either cancel the old "follow" and accept the new, or vice versa.


“Failed to find a contiguous region!” Fugu14 after patch (iPad Air 4th Gen // iOS 14.4)

Seems like I'm not the only one having this issue. People on Reddit is having this exact error with this exact device on this exact iPadOS version.
The iPad Air 4th Gen has the same processor as the iPhone 12 Pro Max (A14 Bionic), and if it works on iPhone 12/Pro/Max, it technically should work on this iPad too.
Well, no. Not exactly. Attaching a picture below so you can see what's really happening:

any update on this issue?

any update on this issue?

I jailbroke using the old method (compiling it) and it worked!

any update on this issue?

I jailbroke using the old method (compiling it) and it worked!

can you help me out by giving me a guide link for the method you used?is it the xcode one?

any update on this issue?

I jailbroke using the old method (compiling it) and it worked!

can you help me out by giving me a guide link for the method you used?is it the xcode one?

Yep, it's the Xcode method. There are plenty of YouTube tutorials out there, but tbh I just followed ProfeJuAntonio's tutorials (it's in Spanish), but if you want to get it in English, you can browse it on YouTube and you'll find it first.

any update on this issue?

I jailbroke using the old method (compiling it) and it worked!

can you help me out by giving me a guide link for the method you used?is it the xcode one?

Yep, it's the Xcode method. There are plenty of YouTube tutorials out there, but tbh I just followed ProfeJuAntonio's tutorials (it's in Spanish), but if you want to get it in English, you can browse it on YouTube and you'll find it first.

Give me Plesse the spanish Link of this YouTube video!

I'm also having this issue on Ipad Air 4 14.4 will a new version, please advise.

Latest build fixes this issue


feat: add back button and anonymous login warning (#720)

Fixes Issue
**My PR closes #720 **
👨💻 Changes proposed(What did you do ?)

I have implemented the back to home functionality in the Login.jsx
I have also implemented the anonymously login we will get a dialog box asking should we login anonymously or not

✔ Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[X] This PR does not contain plagiarized content.
[X] The title and description of the PR is clear and explains the approach.

Note to reviewers

📷 Screenshots

Added the default dialog please check @Dun-sin

Hello @Dun-sin i have made requested changes and raised the PR

yes sure @Dun-sin will be happy to contribute to this project in future

@sant0143 on testing, the dialog doesn't work, you can check the preview with the vercel comment

@sant0143 and can we have a back button where the arrow is pointing to instead

i have asked for approval please give approval will check it

i have asked for approval please give approval will check it

the vercel link is the updated code

i have asked for approval please give approval will check it

the vercel link is the updated code

can u provide the link here?


Ackerman Function #1361 solution in python

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
I read each and every detail mentioned in the contributing.md
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
This PR will add solution of issue #1361 on Ackermann Function in python in which I was assigned.

Please review again the changes I made, @iamrajiv.

Please review again the changes I made, @atarax665 .


Upgrade recast

Summary:
There were a couple of files that the codemod fails on.  Upgrading recast fixed the issue.  This PR also updates the patch file so that retain that change after the upgrade.
Issue: None
Test plan:

yarn typescriptify convert --path ../webapp/services/static/javascript/tutorial-scratchpad-package --dropImportExtensions --write --delete --disableFlow --skipNoFlow
see that the codemod no longer errors on javascript/tutorial-scratchpad-package/scratchpad-page.jsx with Error: did not recognize object of type "IndexedAccessType"

Unfortunately, this doesn't fix An error occurred Unsupported AST node: "InterfaceTypeAnnotation" that occurs when processing javascript/video-package/video-player-model.js.


feat: pkg.pr.new

Before running the ci, we should install https://github.com/apps/pkg-pr-new

Thanks for the PR, dear @Aslemammad. Sorry for simple question since following up on idea in #215, why a Github action is required for integration?


Update inspec to 1.43.8-1

Update inspec to 1.43.8-1
After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask's name and version.

Thanks for the PR!

Huh, the checkpoint hash was accurate earlier today... ah well. Glad to help! :+1:


Change iholder-redhat username to iholder101 in owners file

What this PR does / why we need it:
My username is changed from iholder-redhat to iholder101, updating owners file with the new username.

Before we proceed with this change, @iholder101, you will have to add your new account to the kubevirt org. Otherwise we get an invalid owners file.

Done :+1:
Thanks for bringing it to my attention!

/verify-owners

/ok-to-test
/approve

/approve


Prediction runs slowly

Hi! Thank you so much for this project.
I'm trying to speed up the  running time of the predictions, but even when running on GPU, it still takes a long time.
Is there any way to speed up the prediction? (I have tried to increase the data loader workers to 10 and batch size to 1000, but it didn't help).
Thank you!

How many files do you have and how long does it take? With a GPU the inference time should be really quick but the best settings for number of workers and batch size depend on your system. Maybe you could try just 4 workers and a batch size of 150. Also, how long are the durations of your files?

Thanks for your help!
I have about 100K files, each of length ~10 seconds, and from what I've seen so far it takes about 10 minutes for 100 files.
I didn't see a different between using GPU and CPU, the speed remain approximately the same.

That is too long, even on CPU 100 files should just take around 10-20 seconds. Not sure what is wrong here. Which OS are you running it on? Did you use the env.yml for installing a new conda environment? Which model weights are you using?

Your problem might actually be the large batch size. You may want to try something smaller and then see how high you can go. Maybe start with 8.


Is there a way to send Data as bytes array?

My C# application needs to send large chunks of raw bytes data to Electron App.
Is there anyway for C# side to send data as bytes or int array? Thanks very regards!

Hi @orange-beans
You can base64 encode your data and send it that way, here's an example that sends a file from .Net to Node (assumed to be an jpg image in this example):
DotNet:
using System;
using System.IO;
using ElectronCgi.DotNet;

namespace Base64DotNet
{
    class Program
    {
        static void Main(string[] args)
        {
            var connection = new ConnectionBuilder().Build();
            connection.OnAsync<string, string>("read-file", async filePath =>
            {
                var fileBytes = await File.ReadAllBytesAsync(filePath);
                return Convert.ToBase64String(fileBytes);
            });
            connection.Listen();
        }
    }
}

Node/Electron:
const { ConnectionBuilder } = require('electron-cgi');
const fs = require('fs');

const connection = new ConnectionBuilder()
    .connectTo('dotnet', 'run', '--project', '../Base64DotNet')
    .build();

connection.send('read-file', '/home/rui/Pictures/cat.jpg', (err, base64Image) => {
    if (err) {
        console.log(err);
        return;
    }
    const bytes = Buffer.from(base64Image, 'base64');

    fs.writeFileSync('test.jpg', bytes);
    connection.close();
});

I'm assuming this works for you so I'm closing this issue


How to hide cursor

I am using my custom input box to enter keywords, and I don't want to show the cursor comes after tags. and when I do editable= false, I am not able to delete tags so is there any other way to hide the cursor and still able to delete tags.

No, sorry, this isn't currently possible. This component is set up to have the TextInput flow inline after the list of selected tags. I'd suggest forking or copying some of the code in this project in order to spin your own solution, keeping in mind the license requirements.


Content-Length or Transfer-Encoding MUST NOT be required for 1xx and 204 requests

According to https://tools.ietf.org/html/rfc7230

A server MUST NOT send a Transfer-Encoding header field in any
response with a status code of 1xx (Informational) or 204 (No
Content).  A server MUST NOT send a Transfer-Encoding header field in
any 2xx (Successful) response to a CONNECT request (Section 4.3.6 of
[RFC7231]).

A server MUST NOT send a Content-Length header field in any response
with a status code of 1xx (Informational) or 204 (No Content).  A
server MUST NOT send a Content-Length header field in any 2xx
(Successful) response to a CONNECT request (Section 4.3.6 of
[RFC7231]).

For this response:
HTTP/1.1 204 No Content.
Date: Thu, 27 Sep 2018 08:32:28 GMT.
Connection: keep-alive.
.

I have this error:
Error: We need Content-Length or 'Transfer-Encoding: chunked'
    at parseResponse (<project root>/node_modules/parse-raw-http/parse-response.js:65:13)

Also
   A user agent SHOULD NOT send a
   Content-Length header field when the request message does not contain
   a payload body and the method semantics do not anticipate such a
   body.


How to config client sdk?

I need to config certificates for java clients which depends on spring-cloud-config-client,it has a little boring and complex.If i configed  spring-cloud-config-client and packaged certificates into jar,i wouldn't need to config any java clients.
So,the problem is how can i config the sdk( it's better to config it with a file,such as .yaml).
If not,i could only config java client with
-Djavax.net.ssl.keyStoreType=pkcs12
-Djavax.net.ssl.keyStore=client.p12
-Djavax.net.ssl.keyStorePassword=123456.
Any good ideas?
For great appreciate!

Finally, i worked it out.


data-selected not working for 'new' values

At first - I have to say "great work!". I can use it very well in a first little project.
But then it came to the special case ...
I read tags with your input and I allow the user to add own tags (not provided from the server before)
I get the tags form the request, I can save them in the DB. No problem so far.
When the user wants to edit the entry, i send the saved tags back to the form in the "data-selected" attribute.
So far also good. The former saved tags are shown in the input and I get back the changes.
But not the 'new' or 'own' tags. They are not shown - and I lose them in the described process.
The example shows it with "server4" in the second "select".
Bug or feature?
How should I deal with it?
Thx!
<!DOCTYPE html>

	<meta charset="utf-8" />
	<meta name="viewport" content="width=device-width, initial-scale=1" />
	<link<EMAIL_ADDRESS>rel="stylesheet" />

		<form method="get" action="https://vercel-dumper.vercel.app/">

					like the demo + allow clear
					<select class="form-select" id="serverSideTags" name="server_side_tags[]" multiple data-allow-new="true"  data-allow-clear="true" data-server="demo.json"
						data-search-fields="value,label" data-selected="server1,server2">
						<option disabled hidden value="">Choose a tag...</option>
					</select>

					like the other but with a 'new' entry in data-selected
					<select class="form-select" id="serverSideTags2" name="server_side_tags2[]" multiple data-allow-new="true"  data-allow-clear="true" data-server="demo.json"
						data-search-fields="value,label" data-selected="server1,server4,server2">
						<option disabled hidden value="">Choose a tag...</option>
					</select>

			<button class="btn btn-primary" type="submit">Submit form</button>
		</form>

so if i get this correctly, you have another select with data-selected where you pass the updated selection?
but is that value available in the other select?
in your provided code, i fail to see how this could work. data-selected is meant for existing values. maybe i could add a special case where if you have data-allow-new, then data-selected could also allow arbitrary values.. but then, it's not really a "new" value either since it was saved before...
if you provide server side data, you can also simply mark them selected directly in the returned data
eg:
{
"value": "server3",
"label": "Server 3",
"selected": true
},

Thx for your reply. I think I got your point of view.
I see "data-selected" more generalized as "things they should appear in the tags input field on start".
I think about your control not much as a "specialized html-select" more as a "tags input".
Now look at it from the use case side:
We have a default set of suggested tags and allow the user to create new ones for his date entry - but we don't like to add them to the "default set". If the user want's to update his entry, the tags input should show all the things saved before ... but the tags list should not contain all additional created tags by him or other users.
I could work around the limits anyhow, but I want to make clear first, you don't see it as a needed functionality.

data-selected is only meant to select initial values when populating the data
as soon as you add a value, it becomes selectable. so that means you can simply add options with the previous values if needed. i'm not sure about the distinction you make about the tags input and the tags list. if a value is added as an option or through setData, you will have it in the suggested list. if you set it as selected, it will appear initially in the input.
i don't see why he would see values from other users, that's up to you to filter things no?

Okay I'm down
I found a workaround ...

;-) great happy that you have a solution


Closing an issue results in double automation of moving the issue to Triaged

When an issue is closed, we're seeing artifacts that indicate the issue is moved to the Triaged column twice. While the issue page is open, this shows up on the issue page like this:

The project board activity stream shows what actually happened:

Both the fabricbot configuration and the project board preset automation are moving the issue to the Triaged column. This is a race condition resulting in duplicated updates, as fabricbot is initiating the move before the board automation makes the move.

We should consider removing the project board automation presets and make fabricbot the single source of board automation. This would address other race condition or "tug of war" scenarios/risks. Issues #5 and #6 are both also caused by this conflict, and #3 is related too.

Since we already have logic in place to move to the Triaged column when an issue is closed (as part of isTriaged), we can safely remove the board automation for the Triaged column without any code change.

All of the issue triage boards have been updated to remove the Triaged column automation.


 Or all namespaces if --all option is given

A general comment, just in case it's forgotten. Can you please comment all public functions and methods.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@12d1694). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #198   +/-   ##
=======================================
  Coverage        ?   24.69%
=======================================
  Files           ?       45
  Lines           ?     2430
  Branches        ?        0
=======================================
  Hits            ?      600
  Misses          ?     1764
  Partials        ?       66

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[doc] Support multilingual doc by google-translator plugin

Purpose
Support for Chinese Documentation for PyIceberg better spreading in Chinese developer community.
What I've Done

Add google-translator-plugin in mkdocs.yml and requirements.txt

Preview

Thanks for raising this @Waterkin. I noticed that Google Translator also translates package names and some terminology into Chinese:

I think these should remain in English. Is there any way to manually correct them after the translation is applied?

@HonahX Totally agree! A way to remain the terminology in English is to build a multilingual website. However, this may consume some time and the website will be not easy to maintain.


Lighting on bear is too "flashy"

Though this has been improved since the initial release on GitHub, the lighting on the bear still seems a little wonky. Not sure if it is in the specular or the shadows (or both), but the lighting flashes as the bear animates. The issue goes away if you disable simulation in the UI.
We are looking into it.

The issue seems to be related to the shape matching constraint. The stiffness seems to increase visually with the number of shape matching iterations. Having ~20 iterations seems to stop the hair movement almost completely. It seems that a physics constraint should converge to a stable value earlier, especially since the hairs are rather short.

The choice of x-axis is arbitrary but it should be consistent during asset loading and simulation. x-axis is not for computing the angle between two segment but it is to compute basis vectors. Since hair has bending and twisting effects as well, simple angle between two segments are not good enough to represent both effects.
You may find more details from the paper linked below.
https://amd.box.com/s/w904brn22n3e10bkh9j89f7a2trevlyi

But you could represent the twist and swing as quaternion between the two segments afaik. I assume using the x-axis will create a gimbal lock in certain situations.

Is the compute tangent code correct ? I switched from tangent computation from 2.2 code to the one used in 3.0/3.1 and it looks slightly different (darker). Although it shouldn't change anything

nvm this was a bug on my end


Appear To Be Missing Extends

This feature doesn't appear to be present.
https://www.prisma.io/docs/orm/prisma-client/client-extensions#about-prisma-client-extensions
Feature from 4.16 onwards.

I need to update client oops my bad.

@nestharus
I don't have much plan to implement it in Dart 3.x. Once $extends is implemented in Prisma Dart, it will become a counterexample that API consistency cannot be guaranteed.
The premise that $extends appears in Prisma Client is that TS/JS do not have a good language-level mix. Prisma does not want users to write complex code to achieve type-safe extension functions.
Prisma uses TS type generates to achieve this, but it will never be possible in Dart. Using it in Dart is a counterexample.
Maybe in the future, when there is a better solution after the macro is officially launched, there is no breakthrough at the language level, or the design is incompatible, Prisma Dart will shelve some Prisma features indefinitely.


reduce queue size to camInstance.buffer_frame_count_-1 , not just by 1 item

In this section where we handle full queues...
https://github.com/danionella/PyVCAM/blob/06f7eee109d5209227b7ff5736c187c4295063d5/src/pyvcam/pvcmodule.cpp#L291-L302
... I believe we should reduce the queue size down to (camInstance.buffer_frame_count_-1) , not just by -1. Imagine that the system is temporarily overloaded and the queue size grows multiple items beyond camInstance.buffer_frame_count_. It has no chance of getting reduced again because a few lines further down we always push one frame back in.
So I believe we need a while loop
if ((int)camInstance.frameQueue_.size() >= camInstance.buffer_frame_count_) {
   int n_dropped = 0
    while ((int)camInstance.frameQueue_.size() >= camInstance.buffer_frame_count_) {
        camInstance.frameQueue_.pop();
        n_dropped++
    }
    printf("Dropped %zd frame(s)\n", n_dropped);
}

Yes, that's safer!


fix remove_agent in GeoSpace

Fix https://github.com/Corvince/mesa-geo/issues/28

I haven't tested, but it looks good to me, assuming you have tested it locally.
Also, this is tangent to this PR, but remove_agent should probably be renamed to remove to make it consistent with the schedulers defined in Mesa's time.py.

Thanks for your feedback!
Since GeoSpace is a form of Space (e.g., Grid, NetworkGrid), I guess the remove_agent method should be consistent with their remove_agent methods?

Ah, right, not sure why I was looking at time.py instead of space.py.

Shall we go ahead and merge this, or is there any workflow that we should follow...?

Yeah, merging sounds good. We can add the GH Actions CI right after this PR.

@jackiekazil Can you give us merge permissions to mesa-geo please

It seems to be a merge commit: https://github.com/projectmesa/mesa-geo/commit/40db9f5d0679c8076bc91e8ad0d711acfb77b211 instead of a rebase and merge

The set up needs to be setup on this repo is settings. (On my phone, will
fix when I get back to my computer, unless someone else beats me to it.)
On Fri, Apr 1, 2022 at 7:15 PM rht @.***> wrote:

It seems to be a merge commit: 40db9f5
https://github.com/projectmesa/mesa-geo/commit/40db9f5d0679c8076bc91e8ad0d711acfb77b211
instead of a rebase and merge
—
Reply to this email directly, view it on GitHub
https://github.com/projectmesa/mesa-geo/pull/34#issuecomment-1086406134,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABIWTVVBGDHZ5QBQWB7LCTVC57SLANCNFSM5LGPKKYQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Jacqueline Kazil | @jackiekazil

Repo is now set to rebase.


Missing Lines in an email get rendered as None

If an email is converted that does not have all the lines that a received email has (say a template email that will be sent and does not have message-id), they will be included, but rendered as None.  These elements should simply not be included.

Thanks, @jmgnc . This should be fixed in the latest version.


RuntimeError: CUDA failed with error CUDA driver version is insufficient for CUDA runtime version

Hi, all! I'm trying to deploy an app that uses faster-whisper on google kubernetes engine and I encountered this runtime error when I tried to use "cuda" as the device type.
This is my current docker image file:
# Alternatively, use a base image with CUDA and cuDNN support
FROM nvidia/cuda:11.7.1-cudnn8-runtime-ubuntu20.04

# Install necessary dependencies
RUN apt-get update && apt-get install -y python3-pip

# Set the working directory
WORKDIR /app

# Copy the app code and requirements filed
COPY . /app

# Install dependencies
RUN pip3 install --no-cache-dir -r requirements.txt

# Install whisper-ctranslate2
RUN pip install -U whisper-ctranslate2

# Preload model so that startup time isn't too slow
RUN python3 -c 'from faster_whisper import WhisperModel;WhisperModel("large-v2", device="cpu")'

and this is the yaml file for deployment:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: fasterwhisper-deployment
spec:
  replicas: 1
  selector:
    matchLabels:
      app: fasterwhisper
  template:
    metadata:
      labels:
        app: fasterwhisper
    spec:
      containers:
      - name:
        image:
        ports:
        - containerPort: 8080
        resources:
          limits:
            cpu: "4"
            ephemeral-storage: 1Gi
            memory: 24Gi
            nvidia.com/gpu: "1"
          requests:
            cpu: "4"
            ephemeral-storage: 1Gi
            memory: 24Gi
            nvidia.com/gpu: "1"
      nodeSelector:
        # ask for a GPU node with a Tesla T4 on it
        cloud.google.com/gke-accelerator: nvidia-tesla-t4
        cloud.google.com/gke-accelerator-count: "1"

I'm wondering what changes should I do to prevent this error from occuring

Try
FROM nvidia/cuda:12.0.0-devel-ubuntu20.04

WORKDIR /root

RUN apt-get update -y && apt-get install -y python3 python3-pip libcudnn8 libcudnn8-dev

Thanks for this @Purfview . It worked on my end. Closing this now.


PulsarClientOauthAuthenticationService Private key file field doesn't support Base64 encoded value.

Instead of creating a file for nifi, you should be able to add a base64 encoded value
This works with the Pulsar Client
String oauth2Creds = "{" +
                "\"type\":\"client_credentials\"," +
                "\"client_id\": \"" + clientId + "\"," +
                "\"client_secret\": \"" + clientSecret + "\"," +
                "\"issuer_url\": \"" + issuerUrl + "\"" +
                "}";
        String base64data = Base64.getEncoder().encodeToString(oauth2Creds.getBytes());
        String json = "{" +
                "\"type\":\"client_credentials\"," +
                "\"privateKey\":\"data:application/json;base64," + base64data + "\"," +
                "\"issuerUrl\":\"" + issuerUrl + "\"," +
                "\"audience\":\""+audience+"\"" +
                "}";
        var auth = AuthenticationFactory .create(AuthenticationOAuth2.class.getName(), json);

        client = PulsarClient.builder()
                        .serviceUrl(myPulsarUrl)
                        .authentication(auth)
                        .build();

+1


Is there a public testnet yet?

I'd be very happy to run a node.

Hello I'm working on getting a testnet running for everyone right now! Should be up and running very soon I'll let you know when you can start testing.

Awesome. Has anyone taken a look at a DNS seed for Hivemind (like bitcoin-seeder)?

Yes I've added our seed node to the testnet I'm working on. I'm in the process of getting everything ready for a pull request but there will be seed nodes once it's ready :) I think we are going to have to setup up some more in the near future as well though, so I'll let you know once we are ready to get some more running.

Hey I have to finish a few things before the testnet can be created, and I've created https://github.com/bitcoin-hivemind/testnet-canonical which is where all testnet stuff is going.

Renaming this to fit the hivemind roadmap, and make it clear that it is a high priority issue

I think we can probably close this issue now @bitjson and @psztorc everything on https://github.com/bitcoin-hivemind/testnet-canonical is ready to go.

Agreed. Great work!


SAK-32136 Change web links text in Resources actions menu and Add Web…

… Link page.
I have left the ResourceToolAction enum constant as NEW_URLS (and added a comment) as from the linked Jira (SAK-31809) it looks like we may go back to wanting the ability to add multiple links.

I've resolved the conflicts, it would be great if you could look at this again, Matthew.

Tested locally, it does what it needs to do.

Yeah, I do wonder to Shawn's comment that it might have been better to just remove the "Add details for this item" rather than removing the ability to add multiples. I don't know which is used more.

This is the text for web links, not for the add more folders thing. Similar, but different issues.

Yeah, I get that, but I think Shawn's point could be applicable to both things that were changed, Web Links and Folders. What's more important in both cases being able to add more details there or more links/resources. I feel like this is appropriate for now but all of this might just end up being reverted.

The way I understood the tickets, the actual problem between the issues was similar but not exactly the same.
With web links, the problem was that the 'add another web link' link wasn't working; so it was hidden. This is the appropriate decision short of fixing the feature (which no one seems to have time to do), making these message changes necessary in the short term.
With folders, the problem was that the 'add details' didn't work for any but the first item, but the chosen solution was to hide the 'add more folders' link rather than hide the 'add details' link. Shawn's point on this one was that that maybe we should have hidden the 'add details' link rather than the 'add more folders' link, which wasn't really the problem. I don't think that argument applies here (web links).

No, I think they both didn't work because the add details on the additional links was broken. There was code that copied the entire details HTML element client side and it broke with the new time pickers and I couldn't figure out a fix. This could have been redesigned to either remove the dates too, or just make the dates apply to all of the elements (like it does in drag and drop). Those might have been good ideas too.

I see what you mean now. But regardless, until some sort of other fix is developed and adopted, these messaging changes are necessary in it's current state.


History function only returns "inf" values

Dear Zipline Maintainers,
Before I tell you about my issue, let me describe my environment:
Environment

Operating System: macOS
Python Version: 3.5
How did you install Zipline: Conda

Now that you know a little about me, let me tell you about the issue I am
having:
I'm using the handle_data() function to access the price data of the last three days. All the data is stored inside a CSV file. However, only the last value is returned while all other values are returned as inf.
def initialize(context):
    context.i = zipline.api.symbol("AAPL")

def handle_data(context, data):
    x = data.history(context.i, "open", 3, "1d")
    print(x)

def run_backtest():
    start = pd.to_datetime("2017-01-10").tz_localize('US/Eastern')  # Format: YYYY-MM-DD
    end = pd.to_datetime("2017-03-29").tz_localize('US/Eastern')

    zipline.run_algorithm(start=start, end=end, initialize=initialize, handle_data=handle_data,
                          bundle="apple_data", capital_base=1000)

run_backtest()

Output:
2017-01-06 00:00:00+00:00           inf
2017-01-09 00:00:00+00:00           inf
2017-01-10 00:00:00+00:00    118.769000
Freq: C, Name: Equity(0 [AAPL]), dtype: float64
2017-01-09 00:00:00+00:00           inf
2017-01-10 00:00:00+00:00           inf
2017-01-11 00:00:00+00:00    118.739000
Freq: C, Name: Equity(0 [AAPL]), dtype: float64

CSV file:
date,open,high,low,close,volume,dividend,split
2017-01-03,115.800003,116.330002,114.760002,116.150002,28781900,0,0
2017-01-04,115.849998,116.510002,115.750000,116.019997,21118100,0,0
2017-01-05,115.919998,116.860001,115.809998,116.610001,22193600,0,0
2017-01-06,116.779999,118.160004,116.470001,117.910004,31751900,0,0
2017-01-09,117.949997,119.430000,117.940002,118.989998,33561900,0,0
2017-01-10,118.769997,119.379997,118.300003,119.110001,24462100,0,0
2017-01-11,118.739998,119.930000,118.599998,119.750000,27588600,0,0

Sincerely,
Niklas

I tried several solutions like converting the dtype to a bigger "size", but none of them worked @freddiev4

@niklasamslgruber
I have exactly same issue. Did you solve it?

Yes, I did @Kiminaka. My CSV in the format date,open,high,low,close,volume,dividend,split had 0 values for the column split. I changed the values of split to 1 and it worked (I only had price data without dividends and splits)

@niklasamslgruber Thanks so much for the quick reply! Ah, I had 0 values for split.. Great, this worked for me as well.


feature(makefile): Makefiles cross-compile arm

Describe what this PR does / why we need it
Makefiles cross-compile arm
Does this pull request fix one issue?

Fixes #2050 use local go env to build sealer binary
Describe how to verify it
❯ ls ./_output/bin/sealer/linux_arm64/sealer -al
-rwxr-xr-x 1 root root 52035584 Mar 28 14:05 ./_output/bin/sealer/linux_arm64/sealer

Special notes for reviews
no

Codecov Report
Patch coverage has no change and project coverage change: -17.03 :warning:

Comparison is base (8f38cb7) 18.82% compared to head (2ee404c) 1.80%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #2149       +/-   ##
==========================================
- Coverage   18.82%   1.80%   -17.03%
==========================================
  Files          96     163       +67
  Lines        8950   13310     +4360
==========================================
- Hits         1685     240     -1445
- Misses       7035   13051     +6016
+ Partials      230      19      -211

Flag
Coverage Δ

e2e-tests
1.80% <ø> (?)

unit-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 259 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add support for Chrome Beta browser

Description
Add support for Chrome Beta browsers.
Address #7604.
Checklist

[x]  I read the extension guidelines
[x]  I read the documentation about publishing
[x]  I ran npm run build and tested this distribution build in Raycast
[x]  I checked that files in the assets folder are used by the extension itself
[x]  I checked that assets used by the README are placed outside of the metadata folder

Thanks for the PR! Can you please look at the CI errors? It seems you need to format the code 😊


chore(main): release 1.15.1

:robot: I have created a release beep boop
1.15.1 (2024-11-05)
Miscellaneous

improve resource exhausted error message (#247) (c29417e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/momentohq/client-sdk-php/releases/tag/v1.15.1 :sunflower:


Rewrite with Nuxt UI Pro

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

Too much work right now and not much value as the current UI is decent.


Fix Build() input handling

Hi @jsoref,
Thanks for the pull request. I will review it in the next couple of days.
Thanks,

Hi @jsoref,
Would you mind reopening the pull request? I think the check that you have added in useful. I was not sure if the test case was meant to check the condition though. In case you are not able to reopen the PR (I cannot as the branch is deleted), I can add this check in the code.
Thanks for finding the issue.
Thanks.

The test case was meant to match the code change. After thinking about it longer, I don't think the if I'm adding is technically necessary, the loop will just compare some character after the , or the null byte thereafter with some character from the string. Thus, the current code is safe-enough.


Performance improvements for manual updates

Been working on auto-update mode & performance.
Remaining perf issue with manual updates - each manual update currently runs updateMatrixWorld() on the parent, and computes the inverse of the parent matrixWorld.
It would improve performance if we could calculate these once for multiple updates.  One approach would be to batch mesh member position updates together, and do them in bulk on the tick, as we do with auto updates.
We already have a queuing mechanism for these updates, so not too much work.
Didn't do this as part of the auto-updates / perf work on the basis that we now have auto updates, and can always switch to that update mode if there's so many mesh member transform updates as to cause a performance issue in manual mode.
In other words:

if a use case has few updates, manual update perf shouldn't be a big deal
and if it has lots, then auto updates is probably a better solution anyway...

Not clear at this point that there's any use cases in between that would provide motivation for this fix.

Fixed by #25


Update README.md

this should be fixed in the new version - phantomjs now installs when npm install is run. Give it a shot.


Adding custom Vue to change the vm service offering

Fixes #381

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/413 (JID-1990)

@rhtyd Could you check how to get the input in the form to be the proper length? Otherwise all good to go!

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/413 (JID-2031)

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/413 (JID-2034)

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/413 (JID-2036)

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/413 (JID-2043)

@rhtyd good to go!

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/413 (JID-2057)

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/413 (JID-2078)

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

@blueorangutan package

@davidjumani pl fix conflict

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/413 (JID-2091)

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/413 (JID-2104)

@shwstppr can you review and help test this, compare against legacy UI? I found some issues but I'm not sure if that's because of simulator. Perhaps when VM is stopped we should call changeServiceOffering API (otherwise, at least for Simulator it fails, possibly others hypervisors?)

Investigate if we can safely use the changeSer.... instead of scale... API for a stopped VM (for all hypervisors?); then fix the UI component to change the API depending on state

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/413 (JID-2120)

@shwstppr can you review and test please?


Add feeabs module

This PR adding fee abstraction v7 module for Nolus built with cosmos-sdk v0.47.x.

Hi @desislavva, could you take a look at this PR?

Hey @trungnt1811, thanks for your interest in contributing to us!
We have already implemented a governance based mechanism for paying fees with denoms that the validators/network have agreed to accept. We took advantage of the option to use a custom TxFeeChecker and utilize our own oracles so we don't depend on more external modules which will complicate future upgrades.
You can take a look at the code here for more details:


Design property override in the inheritance context

TypeSpec allows property change of optionality and type.
Case 1: optionality widen
(Seems this case is not allowed in the future https://github.com/microsoft/typespec/pull/3659.)
model Base {
  a: string;
}
model Derived extends Base {
  a?: string;
}

Case 2: optionality narrow down
model Base {
  a?: string;
}
model Derived extends Base {
  a: string;
}

Case 3: type narrow down
model Base {
  a: string | int32;
}
model Derived extends Base {
  a: string;
}

Problem description:
C# doesn't allow same property name with different type in base model and derived model. A compromise is we add new key to the property in the derived class.
We could simply reject such case or design this in MGC. If we are going to design, we should consider:

How to do the serialization for this property in both models.
How to do casting between these two models (if we go with "add new solution").

widen: how to do this Base b = new Derived(){A = 1}; Console.WriteLine(b.A); to write out 1
narrow down: base has set a wider value, how to cast to narrower type in derived

This was discussed in .NET team meeting.
The decision was that when you narrow in the derived class, we will override. If you are just changing the description and not the signature, then we will new. But this may already be working like this. This can likely be closed once we find the fix PR.

Implemented in https://github.com/microsoft/typespec/pull/4325


Add DiscreteTracker

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (ec69632) 100.00% compared to head (2470685) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #64   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           28        29    +1
  Lines          785       836   +51
  Branches        86        92    +6
=========================================
+ Hits           785       836   +51

Impacted Files
Coverage Δ

src/arctix/stats/__init__.py
100.00% <100.00%> (ø)

src/arctix/stats/discrete.py
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Auto registration of Querydsl DataFetcher's

LGTM!
I think we can think about simplifying a bit the GraphQlAutoConfiguration after this change.
Also, I'm wondering if we really need to make RuntimeWiring a first-class citizen (it's a bean and something that developers can override in their context). GraphQlSource seems to be the central point here and we should focus our configuration efforts around that.
Do you think that we should stay with a setter for RuntimeWiring on GraphQlSource, or should we add Consumer<RuntimeWiring> on the builder to customize a RuntimeWiring.newRuntimeWiring() provided as a default instance?

Simplifying GraphQlAutoConfiguration after this PR and shifting the focus away from RuntimeWiring and to GraphQlSource.Builder as the entry point for configuration makes sense.

Do you think that we should stay with a setter for RuntimeWiring on GraphQlSource, or should we add Consumer on the builder to customize a RuntimeWiring.newRuntimeWiring() provided as a default instance?

I was also thinking about that. We could maybe make it RuntimeWiring.Builder, expecting a bean of that type (optionally) or otherwise exposing a Consume<RuntimeWiring.Builder> with a default instance created internally.

This is now merged in main, see https://github.com/spring-projects/spring-graphql/issues/93#issuecomment-883472288.


is:issue is:open hello can i know that how you sample  from the log-uniform distribution? thank u so much!

is:issue is:open hello can i know that how you sample  from the log-uniform distribution? thank u so much!

Hello!
Sampling is implemented in this class.
https://github.com/statsu1990/yoto_class_balanced_loss/blob/d05c97c6cea08efa431d458897199bf940bce4a7/src/model/resnet_yoto.py#L185
After uniform sampling from the log-converted space, log inverse conversion is performed.
Thank you.


Add react-stampit link to README

Nice!

This is good to mentions react-stampit. But the placement of the text is rather too high. React is somewhat irrelevant to our main objective - the stamps. In my opinion, this text should go somewhere below "What is a Stamp" section.

@troutowicz You were too fast to merge. :)
Nevertheless, should we move the React mention lower?

I merged simply because everyone has pretty much agreed the README needs a rewrite. Figured just having the mention for now wherever was fine.

I believe that making README worse just before making a large blog post about stampit is not wise. Can we at least minimize the bloating impact?
Just use bold instead of h3. See below.

React Users. Stampit loves React. Check out react-stampit for composable components.

I don't consider the placement bad to be honest. The placement should be close enough to the top and noticeable enough for anyone to see without having to scroll through API related docs.
But your change would probably be noticeable enough.

@troutowicz see #124


Disable IPv6 query server option result in syntax error

According to the server option described in https://www.nlnetlabs.nl/documentation/unbound/unbound.conf/ I am trying to disable IPv6 queries.
unbound.conf contains at the bottom of these two lines
#custom
do-ip6: no
As soon as I start unbound, this immediately results in an error.
/opt/unbound/etc/unbound/unbound.conf:318: error: syntax error
,read /opt/unbound/etc/unbound/unbound.conf failed: 1 errors in configuration file
I am using the latest version Unbound 1.13.2

Likely you have the option in the wrong section. The option has to be in the 'server:' section. And often, at the end of the file there are other sections, like stub, forward, remote-control sections. You can start the server section again for server section options by putting server: in a line above the do-ip6: no option. Like this:
server:
    do-ip6: no

Wow, so simple... thanks that did the trick!

Good to hear that it worked! This ticket is closed, then.


[ASCII-976] Status component implementation

What does this PR do?

This PR creates the status component skeleton.
The main idea with the status component work is to avoid big bang PR changing many parts of the agent.
So this PR is one of several PR that would shift the responsibility  of rendering the status output from the status package (pkg/status) to the status component (comp/core/status)
The new status components leverages fx values groups  to access the different components status providers.
The PR provides test to validate that the status output is as we expect 😄
Some key concepts of the new code:

The status component manages the title section of the status output. newCommonHeaderProvider
We only aggregate the errors for the text format. This replicated the same behaviour that we have today. (Nothing is written in stone, so that could change in the future)

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

/merge


Only 21 of 75 Tests succeed when running them on main.

When trying to work on some of the issues I filed earlier today, I discovered that when running the tests on main (simply opening the Package.swift in XCode an pressing Command + U) about 75% of the Tests fail.
The failures seem to indicate that @Codable is not expanded:
An example failure:
CodableTests.swift:75: failed - Actual output (+) differed from expected output (-):
 struct SomeCodable: Encodable {
     let value: String

     func encode(to encoder: Encoder) throws {
     }
 }
–
–extension SomeCodable: Decodable {
–    init(from decoder: Decoder) throws {
–        let container = try decoder.container(keyedBy: CodingKeys.self)
–        self.value = try container.decode(String.self, forKey: CodingKeys.value)
–    }
–}
–
–extension SomeCodable {
–    enum CodingKeys: String, CodingKey {
–        case value = "value"
–    }
–}

Actual expanded source:

struct SomeCodable: Encodable {
    let value: String

    func encode(to encoder: Encoder) throws {
    }
}

Is this an error on my end?

This is because testing of extension macros are not supported by swift-syntax.
The workaround I use currently to test is to comment these lines and run tests to check:
https://github.com/SwiftyLab/MetaCodable/blob/main/Sources/CodableMacroPlugin/Registration/Registrar.swift#L121-L127
https://github.com/SwiftyLab/MetaCodable/blob/main/Sources/CodableMacroPlugin/Registration/Registrar.swift#L129-L131
https://github.com/SwiftyLab/MetaCodable/blob/main/Sources/CodableMacroPlugin/Registration/Registrar.swift#L133-L139
https://github.com/SwiftyLab/MetaCodable/blob/main/Sources/CodableMacroPlugin/Registration/Registrar.swift#L141-L143
With these lines commented most of the test cases will pass but some cases asserting extension macro behaviour will fail.
I am waiting for a fix from swift-syntax so that I can and unit tests part of CI. Until then this work around is all I have.

@Midbin I have added fix in my forked version of swift-syntax which is now used by this library, hopefully the fix is accepted by upstream.
The fix works well with VSCode without any additional steps.
For XCode, you might need to comment out line 49 and 63 in Package.swift:
https://github.com/SwiftyLab/MetaCodable/blob/5344f133a6458fa20458427e8f3fed252907fda4/Package.swift#L49
https://github.com/SwiftyLab/MetaCodable/blob/5344f133a6458fa20458427e8f3fed252907fda4/Package.swift#L63


Guide for adding instruments

Updated the docs to address #148. Added description and examples of platform, name, tag, sat_id to aid new users in adding instruments.

Putting this here for discussion next week.

Coverage remained the same at 77.658% when pulling 5b57849c7d6500a3c155a0febd2c391d3c2c354b on doc_instrum_format into d765c178559a69ef838801e7e725f5099cfe1e2f on develop.

rewrite top level demo code as generalized instrument

Updated top-level demo code.  Ready for comments.


Feature request: add json formating via opanai entrypoint

Add option to
https://github.com/vllm-project/vllm/blob/de60a3fb93957dce6b242299b5d163f02ef7f383/vllm/entrypoints/openai/protocol.py#L82
 json_format: Union[Boolean, string, None] = None
which False or None says do not force json format, True says format to some json, and string should describe specific json (usind pydantic.schema())
using lm-format-enforcer

And maybe support regex in the same way

Structured output is now supported by #2819


MockComponent ViewChild Service

Is there possibility to mock ViewChild component injected services? Currently my parent component has ViewChild component and it tries to call exposed public service method of this child component. Now it fails because the service of ViewChild is undefined.
https://stackblitz.com/edit/ng-mocks-examples-tybub8?file=app%2Fapp.component.spec.ts

Hi,
thanks for the request.
I've checked the test and I'll try to figure out how a mocked component might be extended next days.

Hi again,
might you check that MockInstance solves the issue for you?
Simply replace node_modules/ng-mocks with the content from ng-mocks.zip.
Docs you can find here:
https://github.com/satanTime/ng-mocks/tree/issues/170#mockinstance
A test example: https://github.com/satanTime/ng-mocks/blob/issues/170/tests/issue-170/test.spec.ts
Thank you in advance!

Hi,
Yes it works, thanks.

Thanks! Feel free to report again if you're facing any issues.


add Temporal API key to scanner

I added patterns for Temporal API keys.
Every Temporal API key has a prefix "tmprl_" follow by two alphanumeric ids.
Pending tests passing.

I did take a look at this, and a check over GitHub Code Search with the regex revealed the potential for false positives, even without deeper testing.
Could you tighten up on length constraints on the two alphanumeric parts of the pattern please? At the moment the lack of a lower limit means it can easily match on variable names in code.
Knowing the exact lengths of those alphanumeric parts could really make the FP potential plummet.

@mattkim if you've got time to take a look at the requested changes I asked for then I can look at retesting and merging the pattern


Should I have access to version in my custom changelog-template?

Hi
I'm using python semantic version to create a changelog for an established repo with a somewhat messy history.
I've got it to work but was hoping to remove the Unknown sections
At the moment, for the latest release I get

CHANGELOG
v1.1.2 (2024-03-20)
Feature

feat: create changelog with python semantic versioning

Automate changelog generation with python semantic release
The first CHANGELOG.md file will be created by python semantic release after landing (4f94356)
Unknown

Merge branch 'changelog' into 'dev'

feat: create changelog with python semantic versioning

I looked at the default template and noticed that it appears to carry out the same action whether the commmit type is unknown or not
# {{ version.as_tag() }} ({{ release.tagged_date.strftime("%Y-%m-%d") }})
{% for type_, commits in release["elements"] | dictsort %}
## {{ type_ | capitalize }}
{% for commit in commits %}{% if type_ != "unknown" %}
* {{ commit.message.rstrip() }} ([`{{ commit.short_hash }}`]({{ commit.hexsha | commit_hash_url }}))
{% else %}
* {{ commit.message.rstrip() }} ([`{{ commit.short_hash }}`]({{ commit.hexsha | commit_hash_url }}))
{% endif %}{% endfor %}{% endfor %}

So I used that to create my own release_notes.md.j2 file but leaving off the else statement
# {{ version.as_tag() }} ({{ release.tagged_date.strftime("%Y-%m-%d") }})
{% for type_, commits in release["elements"] | dictsort %}
## {{ type_ | capitalize }}
{% for commit in commits %}{% if type_ != "unknown" %}
* {{ commit.message.rstrip() }} ([`{{ commit.short_hash }}`]({{ commit.hexsha | commit_hash_url }}))
{% endif %}{% endfor %}{% endfor %}

This is giving me the the error 'jinja2.exceptions.UndefinedError: 'version' is undefined'
  File "/usr/local/lib/python3.9/site-packages/jinja2/environment.py", line 936, in handle_exception
    raise rewrite_traceback_stack(source=source)
  File "pipelines/release_notes.md.j2", line 1, in top-level template code
    # {{ version.as_tag() }} ({{ release.tagged_date.strftime("%Y-%m-%d") }})
  File "/usr/local/lib/python3.9/site-packages/jinja2/sandbox.py", line 326, in getattr
    value = getattr(obj, attribute)
jinja2.exceptions.UndefinedError: 'version' is undefined

But my reading of the docs is that version should be available
"All of the changelog’s template context is exposed to the Jinja template when rendering the release notes. Additionally, the following two globals are available to the template: - release, -version"
I feel like I haven't done enough digging to call this a bug yet but can I ask if my expectations are correct? Is there something obvious that I'm missing?

@rob-dendy, you mention that you are generating a changelog for a project which means there is more than one version provided to the generator. You need to start from our changelog template and extract each version from the larger release history object.
The release notes template is separately used for the HVCS (GitHub, GitLab, etc.) posting of a specific versions details. Because of this difference, the context variables are different. Conceptually you can consider the release notes template is a sub component of the changelog template and PSR only hands the single version to the file.
The other thing you are likely missing is that if you are trying to override both the file generated CHANGELOG and the release notes posted to your HVCS. You will need to have both templates in your template directory. The minor difference is that the release notes file must be "hidden" or start with a period (.).
Separately, to accomplish the ignoring of ill-formatted commits there are a few options. One is to list a few regular expressions in the tool.semantic_release.changelog.exclude_commit_patterns configuration. This will prevent PSR from providing them to the template.  Secondly, you can use an if statement on the for-loop to filter out when the type_ is "unknown".
Another option is to use a macro to define only the headings you want to display and iterate through that list, and only creating that heading (and body) if a commit is of the corresponding type.  This is actually something I plan to implement in PSR in upcoming versions. This approach also allows you to set the specific order of headings rather than alphabetical order.

Thanks these seem to have given us what we need (just incase anyone else can use them).
.release_notes.md.j2
# {{ version.as_tag() }} ({{ release.tagged_date.strftime("%Y-%m-%d") }})
{% set found = false %}{% for type_, commits in release["elements"] | dictsort %}{% for commit in commits %}{% if type_ != "unknown" and not found %}
* {{ commit.message.rstrip() }} ([`{{ commit.short_hash }}`]({{ commit.hexsha | commit_hash_url }})){% set found = true %}
{% endif %}{% endfor %}{% endfor %}

CHANGELOG.md.j2
# CHANGELOG
{% for version, release in context.history.released.items() %}
## {{ version.as_semver_tag() }} ({{ release.tagged_date.strftime("%Y-%m-%d") }})
{% set found = false %}{% for type_, commits in release["elements"] | dictsort %}{% if type_ != "unknown" and not found %}
* {{ commits[0].message.rstrip() }} ([`{{ commits[0].short_hash }}`]({{ commits[0].hexsha | commit_hash_url }})){% set found = true %}
{% endif %}{% endfor %}{% endfor %}


Adds VAction Unit tests

Cleaned up and amended my history a bit to not include all that extra git fluff

@AustinGil  any additional feedback here? Should I roll this back?

@AustinGil  any additional feedback here? Should I roll this back?

Hey, sorry. Don't roll it back. It's just a lot and Im running into issues getting my local dev up and running.

Hey, sorry. Don't roll it back. It's just a lot and Im running into issues getting my local dev up and running.

Merged! Thanks :)


How to change color's theme?

Hi, Guys
I'm using this amazing tool, but I'm facing a problem with current theme, I don't like the color, How can I change the colors?

did you solve it?