Clarify to which nodes genqlient directives apply

If you do
# @genqlient(...)
query Q(arg: T!) { field { subfield } }

then the directive applies -- and must be legal for -- not only query Q but also arg: T!, field, and subfield.  This is documented but in a somewhat obscure place and has caused quite a bit of confusion (see e.g. #148/#149).  It's also not clear how useful it is either.
Instead we could:

Warn more clearly if a directive applies multiple places, or at least add text to the error in the case where one of those places is invalid.
Only apply @genqlient directives to the first following node (so the above directive would apply to query Q only).
Don't allow @genqlient directives which would apply to multiple nodes.
Figure out how to make @genqlient directives real directives instead of comments -- this may be more possible now that we're rewriting queries although it has costs if it means other tools no longer consider the query in source valid -- and then ordinary GraphQL syntax applies.
Do (1), (2), or (3), but only for when one of the nodes is the entire query (the others are typically arguments), which has empirically been the most common source of confusion and seems the least useful.  (Namely, one can imagine wanting the same directive to apply to several sibling fields, but applying it to both query and a particular argument seems nonsensical.)

I lean towards option 5.3, i.e. forbid this with a clear error if one of the nodes is the entire query; it seems to me that's unlikely to break anyone who wasn't already seeing confusing behavior and it's simple enough to avoid.  But I'm open to other options depending on what others think or what's easy to implement!

@csilvers and @benjaminjkraft thanks for both of your help with debugging #148, I've managed to debug the the root cause of the strange errors I was receiving. This discussion likely belongs on this issue since it relates to how comment directives are applied.
While these two statements appear identical, the comment will be applied differently:
This statement works fine.
# @genqlient(for: message_insert_input.created_by_id", omitempty: true)
mutation insertComment(
    $comment: message_insert_input!
) {
  insert_task_comment_one( object: $comment ) { id }
}

But this one fails with either: for is only applicable to operations and arguments or omitempty may only be used on optional arguments
# @genqlient(for: message_insert_input.created_by_id", omitempty: true)
mutation insertComment($comment: message_insert_input!) {
  insert_task_comment_one( object: $comment ) { id }
}

Note that the above statement lacks the newline before the variable declaration.  The comment is parsed for both the *ast.OperationDefinition. and *ast.VariableDefinition: blocks in https://github.com/Khan/genqlient/blob/main/generate/genqlient_directive.go#L241
I'm not sure if this is really a bug, maybe the newline behavior just needs to be documented?

Yeah, that's the maybe-bug I mean!  It is documented but the documentation is very hidden and also clearly not explained well enough.  I think what you say makes it clear that what we need to do is change the behavior, not just try to document it.  I'm liking option 3 above, personally, I think that leaves the least room for confusion.  If I have time this week I can take a stab at doing that.  (Note to self: it may be best to do it as a separate pass so that we can also look for directives that would apply to no nodes, which are also probably an error.)

yes, option 3 would be great from my perspective.
if it had a clear error that the directive was applying to multiple nodes that would make it clear what would need to change in the graphql statement.


simple instructions on editing content

Might be good to add that the Docs WG intends to make the documentation contributing process easy, but I decided to wait on that til we know what it is, but of course am open to other opinions. #28

Corrected the README, now says we created a gem from a jekyll-based custom theme, with links to both.


Fix client certificate validation

Description of changes:
Changes how we validate the client certificate chain provided when clients connect to CDA and get verified. Now we are just checking that the leaf certificate of their chain was issued but the leaf CA certificate we have registered. For example:

If we have a RootCA and an Intermediate CA (signed but the root ca)
When a component connects we will issue a certificate signed with the Intermediate CA
When a component connects to GG (e.g. Bridge or Moquette) we will just validate that the are gg components if their leaf certificate was signed by the Intermediate CA.

Why is this change necessary:
Allowing component verification when providing custom CA.
How was this change tested:
Wrote a few tests.

We identified new issues on unchanged lines of code. Navigate to the Amazon CodeGuru Reviewer console to view the recommendations to fix them.


Route word mapping include space

hi,
this is very helpful project, to start with any plain javascript project.
trying to set up route as eg:   /app/{{pagename}}/create  (having space in route param)
trying to map the route to /app/admin page/create.  its not mapping pagename with "admin page". its going to 404 page not found route.
Any pointers?
thanks
Vimal

Hi Vimal,
You can use regex for special rules.
router.add(/^hello\/(\w+)/i, function(name){ });
The default parameters don't include the space, because it is bad practice to use whitespace in URL.
Best regards,
Grigore
PS. Let me know if this is solution worked for you and I can close the ticket.

That works thanks, i dint know regular expression work in library, now i can reduce route mapping
thanks


Headers

It would be a good thing to be able to set headers.

Hello, do you mean the metadata? It can be done using -m and -M flags as documented:
-m  Request metadata as stringified JSON.
-M  Path for call metadata JSON file. For example, /home/user/metadata.json or ./metadata.json.

Is this not working for your scenario?

Indeed, I completely missed this one.
My apologies... 🤦♂
Thank you for providing grpcannon. Amazing tool 👍


Docker engine crash quickly after start orb

Describe the bug
Docker engine crash quickly after start orb app.
To Reproduce
Don't know how to reproduce it exactly. Docker engine crash quickly after start orb every time.
Expected behavior
Docker engine works normally.
Screenshots
If applicable, add screenshots to help explain your problem.

Logs
vmgr.log
Info report

OrbStack info:
  Version: 0.10.1 (100100)
  Commit: 8c7d0a2519b43c7145f9bf632d1ef1ea5953ea2b (v0.10.1)

System info:
  macOS: 13.4 (22F63)
  CPU: arm64, 8 cores
  CPU model: Apple M1

Additional context
Add any other context about the problem here.

It looks like Apple's virtualization service is crashing. Please attach the latest logs and/or diagnostic reports for com.apple.Virtualization.VirtualMachine that you can find in /Library/Logs and in Console.app.

It looks like Apple's virtualization service is crashing. Please attach the latest logs and/or crash or diagnostic reports for com.apple.Virtualization.VirtualMachine that you can find in /Library/Logs and in Console.app.
find /Library/Logs -name '*VirtualMachine*' | xargs tar cvf ~/Downloads/vmmlogs.gz

nothing found..

Check all the tabs in Console.app as well.

@Zhaojing-Wang Are you still experiencing this issue?

@Zhaojing-Wang Are you still experiencing this issue?

Yes, there is still this problem, no related logs are found in /Library/Logs and console app. However, this problem may indeed be related to virtualization. After I closed the "Use Virtualization framework" configuration of docker, docker desktop can start the docker engine normally, but orbStack still fails.

Yes, it's very strange that Apple's Virtualization service keeps crashing on your machine. Disabling Virtualization framework in Docker Desktop works around the issue by not using it.
It's really odd that there are no logs or diagnostic reports though. Can you check all subfolders in ~/Library/Logs one more time? I ran into one instance of a crash recently and the report was at ~/Library/Logs/DiagnosticReports/com.apple.Virtualization.VirtualMachine-2023-05-30-135015.ips. I've never seen consistent crashes like this though and I use a similar environment for development (macOS 13.4, M1 generation).
Unfortunately I don't think we can do anything about this, or even report it to Apple, if there's no debugging information.
Also, can you update OrbStack and share new logs and the contents of ~/.orbstack/vmconfig.json (if it exists) just in case? Did you do anything unusual, like moving data to another disk?

Yes, it's very strange that Apple's Virtualization service keeps crashing on your machine. Disabling Virtualization framework in Docker Desktop works around the issue by not using it.
It's really odd that there are no logs or diagnostic reports though. Can you check all subfolders in ~/Library/Logs one more time? I ran into one instance of a crash recently and the report was at ~/Library/Logs/DiagnosticReports/com.apple.Virtualization.VirtualMachine-2023-05-30-135015.ips. I've never seen consistent crashes like this though and I use a similar environment for development (macOS 13.4, M1 generation).
Unfortunately I don't think we can do anything about this, or even report it to Apple, if there's no debugging information.
Also, can you update OrbStack and share new logs and the contents of ~/.orbstack/vmconfig.json (if it exists) just in case? Did you do anything unusual, like moving data to another disk? Does deleting ~/.orbstack help?

I did not find any more useful logs. I have tried to completely uninstall OrbStack and manually clean all residual files, but after reinstalling, it still crashes. (￣▽￣)

Similar to #568

Released in v1.6.0 Canary 1.
To update to Canary: Settings > Update channel

Released in v1.6.0 Canary 1.
To update to Canary: Settings > Update channel

It works on my M1 Macbook Pro. I appreciate your help and responsiveness.

Released in v1.6.0.
New: Truly fast container filesystems on macOS: 2–10x faster, within 75-95% of native


1.14 Support

Plugin doesn't load on a 1.14 spigot server.

Yeah, I'm waiting for a dependent library to update, hopefully that happens soon

What library?

Probably ProtocolLib

Oh, it's just waiting for it to come out of "development", then?

@RoboTricker The last dev build of ProtocolLib happily runs on 1.14.4, anything we can help you with?

I'm working on an initial 1.14 update, more news soon

this plugin looks great and would love to see it on 1.14 before 1.15 comes out.
I don't mean to rush (at all), but would like to know if we are looking to see this before christmas?


retention: Remove outdated compat block for delete_message events.

As noted in the comment in that block, this is no longer useful.

Merged, thanks @mateuszmandera!


update brewu method

Homebrew decided to not change the behaviour of brew upgrade so
brew upgrade --all is equivalent to brew upgrade without any other
arguments (so the --all is a no-op and can be removed).

Hi @mrgrauel , your PR got merged in the community maintained fork of this repo.
zsh-users is an organization dedicated to host Zsh community projects and therefore became the rational home for a community maintained version of prezto.
Your contribution is greatly appreciated!
https://github.com/zsh-users/prezto

Thank-you for your contribution!


fix line break problem in doc

Signed-off-by: ChengZi<EMAIL_ADDRESS>

Codecov Report

Merging #1060 (10bbdfc) into main (01ed093) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1060      +/-   ##
==========================================
- Coverage   70.17%   70.11%   -0.06%
==========================================
  Files         213      213
  Lines        9390     9390
  Branches     1549     1549
==========================================
- Hits         6589     6584       -5
- Misses       2347     2352       +5
  Partials      454      454

Impacted Files
Coverage Δ

towhee/dataframe/dataframe.py
73.10% <0.00%> (-1.05%)
:arrow_down:

towhee/hub/repo_manager.py
80.18% <0.00%> (-0.91%)
:arrow_down:

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 01ed093...10bbdfc. Read the comment docs.


Set up CI/CD to deploy to static hosting service

In order to get something online without needing to spend time deploying it, it might be interesting to use Github Actions to deploy the application to a Static Hosting Site (Mostly because this is cheaper). A good contender is Netlify

So... This was suspiciously easy. Vercel Integration is set up and normally, every push to master will update the production website, while any push to another branch will create preview versions.


In POST request data type is now mandatory when it was not before.

Is there an existing issue for this?

[X] I have searched the existing issues

Current behavior
That change made the data attribute of post request mandatory which is a breaking change https://github.com/nestjs/axios/pull/1029/files
Minimum reproduction code
..
Steps to reproduce
Install 3.1.0 and all post request that didn't specify a data param are not passing typecheck anymore.
Expected behavior
No breaking changes in minor releases
Package version
3.1.1
NestJS version
10.4.6
Node.js version
20.15.0
In which operating systems have you tested?

[X] macOS
[ ] Windows
[ ] Linux

Other
No response

Hello, I've fixed this issue!
This is what I've done:
I have identified the issue in the HttpService class where the post method's signature was changed, making the data parameter mandatory. This was indeed a breaking change. To fix this, I've updated the post method to maintain backwards compatibility by making the data parameter optional again. I've also applied the same fix to the put and patch methods for consistency.
I also created pull request: https://github.com/nestjs/axios/pull/1243

[!CAUTION]
Disclaimer: This fix was created by Latta AI and you should never merge before you check the correctness of generated code!

This bug was fixed for free by Latta AI - https://latta.ai/ourmission

Fixed in 3.1.2


Implement Brush Slope

Brush Slope does nothing and was hidden in 283767e1162e558f4f32bb1037014a08507406e8, merged in #91.
Implement it.

It's looking like I'll need this tool for my project. Do you have a particular idea in mind for how it would work? If our needs align, I could consider implementing & contributing it.
Something like the flatten tool, where you pick a starting height, then click and drag, with the difference being the further you drag from your starting point, the lower the height gets, according to an angle setting?

I think there are two implementations of slope. I'll take PRs for either or both.

Slope when applying textures and color, painting only on flat surfaces or only on slopes.

Slope height brush: Select starting point, Select ending point, and it automatically creates a grade between them in the general shape and size of the current brush.

Starting/ending elevations can be predefined with the other brushes.
I've found when brushing heights that I need a slope to connect between a higher area and a lower area. I don't think I've ever needed an arbitrary length brush. Have you found differently? Where you want to start making a grade without a destination in mind?

Have you found differently? Where you want to start making a grade without a destination in mind?

We're only just in the process of initially blocking out terrain. In our case, the destination doesn't matter too much because almost all space is unused. And it can be useful to guarantee a particular angle to the slope to ensure navigability. But I can see that if you've already got a lot of stuff on your terrain that you want to keep, it makes sense to need a particular end point.
Do you see slope height brush as being:

Drag from starting point to ending point. Then when you release the mouse button, it modifies the terrain with a slope between them in a straight line from start to end.
Or click starting point, click ending point, then click and drag out a potentially wavy line between the two, modifying terrain as the mouse moves?

What I envision is a simple: Click start, click end, straight slope is created. No drag.
However, if you want to do # 2, click start, click end, then it becomes drawable with height auto adjusting depending on proximity to start or ending points, also using opacity. That's an interesting option.
Why don't you do both? Have a drawable checkbox. If it's not checked we get the instant straight slope. If checked it is drawable.

Try the height brush with low opacity and you'll see it fills in more gradually than when it is 100%.

1. Slope when applying textures and color, painting only on flat surfaces or only on slopes.

Did a quick walk over some of the header files -- I think this case would be the spot to add logic which restricts painting based on slope? Coupled with expanding the brush data definition here to include some slope-restricted-painting parameters? (and producing some editor UI stuff for end users)
(Confirming my understanding before attempting a PR)
Looking at the current issues in the 9.x and 1.0 roadmap, only GPU Painting pops out to me as a coupled feature. Would GPU painting introduce breaking changes in _operate_map() potentially?

I have separated the two issues. Let's keep the discussion on sculpting a slope here. Painting based on slope has been moved to #282. @niceandgoodonline if you move your comment over there, I'll answer it.

Fixed by #297


Fitting large ODE model

Hi,
I am trying to replicate the following paper:
https://www.nature.com/articles/s41598-01
Here are the codes I have for parameter fitting. The last version of PEtab worked ok but I cannot reproduce what I was doing in the current one. I also used the original published model to generate some artificial data but I was not able to capture the behavior
nfat_model.zip

This proved to be an interesting problem. You get the error because the event handling in SciMLSensitivity has changed, which is bug (I actually do not test PEtab.jl for the setting adjoint with events which I should).
Regardless, after doing some benchmarks, I do not think gradient_method = :Adjoint is the configuration you should use. You have a relatively large stiff models, for these adjoint methods in Julia are not super reliable (you often get gradient computation failures, which can be detrimental). Moreover, your problem is not that large so that adjoint must be used, as it takes around (only) 2 seconds to compute the gradient with forward methods. Therefore, I would use a setup like:
petab_problem = PEtabODEProblem(petab_model,
                                gradient_method = :ForwardEquations,
                                sensealg = :ForwardDiff,
                                hessian_method = :GaussNewton,
                                reuse_sensitivities = true,
                                odesolver=ODESolver(QNDF() , abstol=1e-3, reltol=1e-6),
                                sparse_jacobian = false, verbose = true)

The sparse_jacobian = false is needed here to have forward gradients working (I missed fixing this when setting default options, so when you build the problem you get sparse_jacobian = true by default, hence strange results with ForwardDiff).  In particular, I recomend using this setting together with Fides https://sebapersson.github.io/PEtab.jl/stable/pest_algs/#Fides (if you cannot install Fides consider Ipopt with BFGS Hessian). This is because the Gauss-Newton Hessian is often the best approximation you can use if you have a trust-region method, and in my experience, Fides is a good trust-region implementation.
Hope this helps!

I am closing this issue as I think you problem was solved, and because I created two new issues that should help prevent the problems you ran into (#246 and #247)


Support subsets of numeric literals

Allow configuration of ForbiddenNumerics to be passes with one of Float, Integer, or All, forbidding only the literal types provided
Defaults to All

I think this is the main config that this gem might want; I think ideally the tests would be as complete as the main tests (so testing assignments through locals, through instances, through writers, etc), but for speed I've implemented testing using multiassigment since that already had parameterized testing similar to what I wanted to do here.

Should the collection of valid representations of floats and integers live in test helper, so they're not being redefined in each of the tests that need them?

Rubocop::Cop::MagicNumbers.illegal_magic_types # => [:float, :int]


Typography migration guide

Description

This pull request does the following:

Add migration guide for new Typography API
Switch from markdown (deprecated) to markdown-it
Implement Spectrum classes for markdown'd links, tables, and code samples
Use new Typography components on site template for header/body text
Use new Typography Code component for markup examples
Fix color styles that leaked into site/index.css (should be in site/skin.css)
Fix header for Code on Typography landing page

How and where has this been tested?

How this was tested: Load up https://git.corp.adobe.com/pages/lawdavis/spectrum-css-typography/docs/typography.html#migratingfromdeprecatedtypography, use eyes
Browser(s) and OS(s) this was tested with: Chrome on macOS

Screenshots

Typography migration guide

Pretty new links and code

To-do list

[x] If my change impacts other components, I have tested to make sure they don't break.
[x] If my change impacts documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

[ ] This pull request is ready to merge.

https://spectrum-visual-regression.ci.corp.adobe.com/job/spectrum-css-vr-test-asset/221/

there is a hover effect and a cursor change to pointer on those example rows?
I clicked on it but didn't notice an action - are you planning on adding an action this?
Screencast:

Not sure why the action-bar case failed.

@bernhard-adobe so unfortunately, Table be like that. We have an issue filed to support a non-interactive mode to remove the hover state and cursor: https://github.com/adobe/spectrum-css/issues/450


feat: introduce Method Override Middleware

This PR introduces new middleware, "Method Override Middleware."
This middleware executes the handler of the specified method, which is different from the actual method of the request, depending on the value of the form, header, or query, and returns its response.
For example, since HTML forms cannot send a DELETE method, you can put the value DELETE in the property named _method and send it. And the handler for app.delete() will be executed.
The HTML form:
<form action='/posts' method='POST'>
  <input type='hidden' name='_method' value='DELETE' />
  <input type='text' name='id' />
</form>

The application:
import { methodOverride } from 'hono/method-override'

const app = new Hono()
app.use('/posts', methodOverride({ app }))

app.delete('/posts', () => {
  // ...
})

You can change the default values or use the header value and query value:
app.use('/posts', methodOverride({ app, form: '_custom_name' }))
app.use('/posts', methodOverride({ app, header: 'X-METHOD-OVERRIDE' }))
app.use('/posts', methodOverride({ app, query: '_method' }))

TODO

[x] Ignore a GET request
[x] Write tests for Header
[x] Write tests for Query

Author should do the followings, if applicable

[x] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

Hi @usualoma !
I think this is good. But what do you think? Maybe there's a point I'm missing.

Wow, I think it is great!
I think it is working fine.
I thought there was room for refactoring with type definitions and minor conditional branching, so I made the following PR.
#2429

Let's go!


feat(style) allow custom hidden list class and active item class

Description
Allow user to specify their own class names for active suggestion items and the visibility of the suggestion list.
Checklist

[x] All tests passing
[x] Extended the README / documentation, if necessary
[x] Commits and PR follow conventions

Closes #223

any update on getting this merged?

Thanks for bringing this up and creating the PR!


Issue with non-DASH buffering (video "de-buffers" when skipping)

Hi, I've been trying out this extension mainly because I want to get rid of DASH on youtube. It's looking promising but I'm having this issue.
I've disabled DASH in the extension and switched to Flash mode (as apparently non-DASH doesn't work with HTML5). The videos buffer all the way to the end, and I can watch them from start to finish without problem. So far, so good...
However, if I try to skip (in the progress bar) to a point later or earlier in the video, sometimes it will do it correctly and just play the buffered video from there, but very often it will "de-buffer" the video and start loading everything again from that point on as though it had never loaded the video to start with.
Is there a way to avoid this by fiddling with the options?
Because this sort of defeats the purpose of the addon for me, as it ends up re-buffering the video all the time anyway.
Note: If the addon is turned off and DASH is back on, this doesn't happen. As long as the chunks of the video are buffered, I can skip forward and back anywhere throughout the already-buffered parts and everything that was already loaded remains loaded.
Note2: I tried enabling Custom Buffer with different buffer sizes too. Don't know exactly what this does but in any case it didn't help either.
Thank you.

Please be aware that DASH is now a central feature and streaming method used by YouTube. By intentionally disabling DASH, you are limiting what playback options are available to you.
Using DASH, YouTube buffers videos in chunks. Without DASH, whenever you change the playback position, the player must recalculate where to begin streaming the video from the source file. This can (and does, more often than not) cause you to lose whatever buffer you had built up before.

This has been an issue for me since the first day I downloaded YTC. Either it's something the devs can't fix, since it's been like this for so long, or perhaps something is broken?

This is an issue with YouTube ever since they introduced DASH playback, it is not something YTC is causing nor can it change.

Than I will close this issue. ;)


basic registration/login

https://github.com/dawnwages/wagtail-indymeet/issues/30

@dawnwages  this is just a basic registration and login ...not very pretty but I wanted to get it functional first so you can do what you'd like with it.

I did some of this too! my fault. I'm going to merge your branch into mine and default to your changes.

Can't merge! I'm sure there's some of the same stuff between the two. I'll stop going rogue now. If you want to check it out and try to resolve the differences, its https://github.com/dawnwages/wagtail-indymeet/pull/32
I'm going to be away from my keyboard all tomorrow so if you want to merge my two commits into a working branch -- we should probably now have a develop branch. They're straightforward and complete.

@dawnwages
This is great, thanks for all your work on #32! :tada:  I was able to merge it with only a few places that I had to choose one way or the other or make changes:

followed documentation example for urls :  https://docs.djangoproject.com/en/4.1/topics/http/urls/#including-other-urlconfs
I renamed the profile function to lowercase. https://peps.python.org/pep-0008/#function-and-variable-names

I noticed django-bootstrap has been removed, I don't know if this was intentional, but wanted to draw attention.
I saw you added Black and Flake. YAY! I did not run any formatter on this because it's a bit complicated. Let's set up an automated (?) process and add documentation to for running these. And maybe we do a run on a clean branch.

yes let's set up github CI/CD? re; I saw you added Black and Flake. YAY! I did not run any formatter on this because it's a bit complicated. Let's set up an automated (?) process and add documentation to for running these. And maybe we do a run on a clean branch.
I've also wanted to use pre-commit on a personal project but haven't gotten around to it. I'd have to fiddle my way through both so I'm happy to go through the least complicated path :) If you have a preference, I'm not particular.

ahh dope! Reviewed :)
LGTM!


PowerFlex/ScaleIO volume live migration - use usable bytes from source disk to format the destination disk

Description
This PR improves the ScaleIO volume live migration, uses usable bytes from source disk to format the destination disk.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?
Tested PowerFlex/ScaleIO volume live migration across storage clusters, with volumes of varied sizes, normal & encrypted volumes (with 32MB & 128MB reserve), with/without data.

How did you try to break this feature and the system with this change?

@blueorangutan package

@sureshanaparti a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✖ el7 ✖ el8 ✖ el9 ✖ debian ✖ suse15. SL-JID 9792

@blueorangutan package

@sureshanaparti a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 9798

@blueorangutan test

@sureshanaparti a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-10364)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 42413 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9174-t10364-kvm-centos7.zip
Smoke tests completed. 131 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

Changes LGTM, thanks for testing @kiranchavala
Merging based on manual QA and smoketests and reviews.


next 13.4.2 - Hydration failure with Theme

When I use <Theme> inside _app.tsx (from  import { Theme } from '@radix-ui/themes')
I got this error:

Hydration failed because the initial UI does not match what was rendered on the server.

Removing Theme gets rid of Hydration failure completely.

what could be the culprit here? (It was working fine earlier)

https://www.radix-ui.com/themes/docs/theme/dark-mode#automatic-background-color


Error: ld: warning: Could not find or use auto-linked library 'swiftCoreMIDI'

if i follow the instructions to run i get:
ld: warning: Could not find or use auto-linked library 'swiftCoreMIDI'
ld: warning: Could not find or use auto-linked library 'swiftUniformTypeIdentifiers'
Undefined symbol: _swift_FORCE_LOAD$_swiftUniformTypeIdentifiers
Undefined symbol: _swift_FORCE_LOAD$_swiftCoreMIDI
Even if i run the example project i get the same error
This is happening in IOS

this is happening in IOS

I'm getting the same error over and over again. I have no idea why these errors showing up in Xcode 12.4. It works fine with Xcode 11.4 but whenever I try to implement the same pugin for Xcode 12.4 it throws the same below error all the time.
If you get any solution over this please let me know @javierdac  Thank you.

Managed to fix this issue by following the steps shared by fritzfr: "Fixed it by placing a File.swift in our root and then let Xcode create a Objective-C bridging header.". Worked for me in Xcode 12.4.
https://github.com/AgoraIO-Community/react-native-agora/issues/266

@lsmacedo Great Thank you.


smartcontract: add NEP-11 standard check

Close #1713 .

name method is missing  because it is provided in manifest.
Return type of properties method is Map, not string in JSON.

Well, I've consulted wrong proposal https://github.com/neo-project/proposals/blob/61e53a07a319c537c57c988cbcb216ea75de0f89/nep-11.mediawiki , seems like correct one is in the PRs to repo.

NEP management is probably not the strongest side of Neo, true. This is not top priority for now (preview5 functionality is), though it'd be very nice to have because our NEP-17 checking feature is still kinda unique and adding NEP-11 seems like an obvious and welcome extension of that.


Density3

Description
Resolve discrepancy between density3 and density https://github.com/terrapower/armi/issues/865 for open source ARMI materials.
This includes:

creating a subclass SimpleSolid material to help define very simple materials.
adding to the base class of the materials unit test to verify the density3 method produces a non-zero result
updating the framework materials and the unit tests so they all pass

Checklist

[X] This PR has only one purpose or idea.
[X] Tests have been added/updated to verify that the new/changed code works.

[X] The code style follows good practices.
[X] The commit message(s) follow good practices.

[X] The release notes (location doc/release/0.X.rst) are up-to-date with any bug fixes or new features.
[X] The documentation is still up-to-date in the doc folder.
[X] The dependencies are still up-to-date in setup.py.

Hi @sombrereau ! @john-science requested that I give this a review as well.
I'm still making my way through your changes but have a thought. What if, instead of a _density() method on each material, we add a constructor to each material and directly set self.p.refDens and self.refTempK? I think it might be clearer as some materials have the _density() method and some don't. It also might be confusing relative to armi.material.density. Just a thought.
One more: What do you think about making self.refTempK a param? I could see it being useful to know on a DB load what the reference temp refDens correlates to.

A few examples of how the reference density is being set for different materials. I think it would be great to have consistency - maybe through the constructor of each material? I'm certainly open to other ideas; I just advocate for consistency and clarity.
https://github.com/terrapower/armi/blob/4d925b522141be7352276de4b5c80855136970ec/armi/materials/b4c.py#L22
https://github.com/terrapower/armi/blob/4d925b522141be7352276de4b5c80855136970ec/armi/materials/concrete.py#L42-L43
https://github.com/terrapower/armi/blob/4d925b522141be7352276de4b5c80855136970ec/armi/materials/ht9.py#L60
https://github.com/terrapower/armi/blob/afd2ead0db0328b16af9319d3aab76255e167a7a/armi/materials/californium.py#L33-L37

A few examples of how the reference density is being set for different materials. I think it would be great to have consistency - maybe through the constructor of each material? I'm certainly open to other ideas; I just advocate for consistency and clarity.

Tony, obviously, I 100% agree.
But I think that would be a bigger change to the system than this PR aims for.  Tommy identified a real bug, where a lot of materials are reporting a density3 of 0.  Which... makes me unhappy.
I like the idea though.  Can you propose that in our umbrella "materials make us sad" discussion?    https://github.com/terrapower/armi/discussions/767

Can you propose that in our umbrella "materials make us sad" discussion?

@john-science definitely. I think I missed the fact that Tommy's PR was to simply fix materials with a density3 of 0.0 - I thought he was aiming to address #865 in full.
I realize that many of my comments may result in scope creep if they were actually addressed... Those that do should 100% be punted to future work. We can iterate on which ones fit that bill.

@sombrereau Okay, so the idea here is that we have a bunch of 3D densities incorrectly reported as 3D?  BUT... instead of just changing the name from density() to density3() for all these materials, you create the _density() method, because we can use that for your new linear expansion method?
Sorry, I'm not trying to be obtuse.  Just trying to figure it out.
Why didn't we just change all the incorrect density to density3?


Fixes race condition bug with Pod not being scheduled before Ready()

This fix does several things:

Breaks out ip and port setting of the Status into its own state, to make
implementation and testing easier.
Listen for Pod updates, and if a Pod is a GameServer pod that has had
its NodeName set, then queue the backing GameServer
Finally - if Ready() gets called before any of this, populate the Status ip
and port values at this time as well.

Fixes #354

I think that was my fault, I was futzing with the cluster.


Update select on model change

Update select on model change, is necessary on load multiselect option on ajax

this solution works for me, thanks


Sonatype snapshots badge failing

Are you experiencing an issue with...

[x] shields.io

:beetle: Description
Take a dependency like this:
https://oss.sonatype.org/content/repositories/snapshots/io/kotest/kotest-extensions-jvm/
Badge fails:
https://img.shields.io/nexus/s/io.kotest/kotest-extensions-jvm?server=https%3A%2F%2Foss.sonatype.org
However other dependencies work fine:
https://img.shields.io/nexus/s/io.kotest/kotest-assertions-js?server=https%3A%2F%2Foss.sonatype.org
This only just started failing. Is there a max limit to the number of snapshot deps it will parse?

Oddly it seems to have started working again.


#3236 add filtering to search results

Signed-off-by: David Radley<EMAIL_ADDRESS>

I also added more standard users to the presentation server in this pr.


Add source in JSON output

Q
A

🐛 Bug fix?
no

🚀 New feature?
yes

⚠ Deprecations?
no

❌ BC Break
no

🔗 Related issues
#874

❓ Documentation
https://github.com/cloudskiff/driftctl-docs/pull/90

Wonder where we still use FakeResource and FakeResourceStringer and whether they still make sense or not.

They definitively not make sense anymore, we should remove them when we'll do the Resource interface removal refactoring.
I just wanted to avoid any big impact for this PR


Info message sends blank DevId

When sending an /info message, the DevId property should contain the UUID of the device, but the message is a "".
cc @eriknordmark @rvs

Looking at the code I see the info (device and app) as well as metrics messages all consistently using the zcdevUUID, which is set first from /config/uuid, and later potentially updated from the UUID in the reply to the GET of /config.
@deitch can you check that you are setting a uuid in the config payload, and what /config/uuid contains when using your controller?

It is possible that the config message was misconfigured. If so, that would do it. I will check it out early next week (traveling now).

Question: assuming this is the problem, should the device in some way validate the config to say, "I don't have a valid UUID" or "the UUID in this GET /config is different from the last one", and some way to tell the Controller that it has that issue?

I checked an actually zedcloud-generated config, it has the same structure, specifically:
{
  "id": {
    "uuid": "abcd-546-abcdf"
  }
}

The auto-generated one is:
{
  "id": {
    "uuid": "bd2c05c0-f7ec-4db3-9a54-acaea973ab43",
    "version": "4"
  }
}

So they do align. Looks like need to dig deeper.

@nordmark already fixed it in the EVE I believe

FIXED by https://github.com/lf-edge/eve/commit/472237267212f22575c7a26687a439dbc5cb1ab5#diff-83f59c66c1453f094f27390ffa085704


How to use this source?

Is there any document explain how to compile and build this source by use VS2019?


Why is Graphviz being installed?

Why is graphviz defined in the template? Seems like this will be unnecessary in most pips.
https://github.com/DiamondLightSource/python-copier-template/blob/2d761101ae3c273475ffd21f74a337a3304af637/template/Dockerfile.jinja#L8
It's probably being installed by many projects for no reason.

It's there because the default api docs used to include a class hierarchy which required graphviz, however although the directive is included:
https://github.com/DiamondLightSource/python-copier-template/blob/2d761101ae3c273475ffd21f74a337a3304af637/template/{%25 if sphinx %25}docs{%25 endif %25}/conf.py.jinja#L45
It doesn't appear to be used in the docs at all. Maybe we should remove both the directive and the apt install and make a how-to telling you how to put it back in?

Yeah, sounds sensible (can always put it back again if it causes a problem).
I guess we'll still need to include the apt-get update though, as some builds may assume it's already run? (Or perhaps no‑one retrospectively copies the template...)


model: tensorlfow: dnnc: Make classification property predict

To maintain consistancy with the other models, we should rename the classification property to predict in dnnc.py
Reporter: @aghinsa

changes in https://github.com/aghinsa/dffml/commit/fd6c91b06aa41620cdd07eff7ce7f267d086eaf4,in pr https://github.com/intel/dffml/pull/274


histology should be a default value for the Image type

I thought that this was already done? but it was not when we were running it today in the meeting...

It will default for a New document but it does not load the value from the Upload of an XML document.


Error while installing tauri cli via cargo

Describe the bug
I faced with the following error(s) when installing tauri/cli via Cargo.

unresolved import clap::crate_version
no crate_version in the root
expected struct App, found &yaml_rust::yaml::Yaml

To Reproduce
Steps to reproduce the behaviour:
cargo install tauri-cli --version ^1.0.0-beta

Expected behavior
It should install successfully...
Screenshots

Platform and Versions (required):
Operating System: Ubuntu 20.04

Node.js environment
Node.js: v14.18.2
@tauri-apps/cli: ___________
@tauri-apps/api: ___________

Global packages
npm: 6.14.15
yarn: 1.22.17

Rust environment
rustc: 1.59.0-nightly (e6b883c74 2021-12-08)
cargo: 1.58.0-nightly (40dc28175 2021-12-06)

Additional context
I'm trying to build Graviton App
Stack Trace
bmatovu@home-pc:graviton$ cargo install tauri-cli --version ^1.0.0-beta
    Updating crates.io index
  Installing tauri-cli v1.0.0-beta.7
  .
  .
  .
   Compiling tauri-cli v1.0.0-beta.7
error[E0432]: unresolved import `clap::crate_version`
 --> /home/bmatovu/.cargo/registry/src/github.com-1ecc6299db9ec823/tauri-cli-1.0.0-beta.7/src/main.rs:6:12
  |
6 | use clap::{crate_version, load_yaml, App, AppSettings, ArgMatches};
  |            ^^^^^^^^^^^^^ no `crate_version` in the root

error: cannot determine resolution for the macro `crate_version`
   --> /home/bmatovu/.cargo/registry/src/github.com-1ecc6299db9ec823/tauri-cli-1.0.0-beta.7/src/main.rs:265:14
    |
265 |     .version(crate_version!())
    |              ^^^^^^^^^^^^^
    |
    = note: import resolution is stuck, try simplifying macro imports

error[E0308]: mismatched types
   --> /home/bmatovu/.cargo/registry/src/github.com-1ecc6299db9ec823/tauri-cli-1.0.0-beta.7/src/main.rs:264:23
    |
264 |   let app = App::from(yaml)
    |                       ^^^^ expected struct `App`, found `&yaml_rust::yaml::Yaml`

Some errors have detailed explanations: E0308, E0432.
For more information about an error, try `rustc --explain E0308`.
error: failed to compile `tauri-cli v1.0.0-beta.7`, intermediate artifacts can be found at `/tmp/cargo-installsXqrl8`

Caused by:
  could not compile `tauri-cli` due to 3 previous errors

wtf cargooooo, don't do this to us.
For an explanation, clap released a new version yesterday (1.0.0-rc.0), which removed the yaml stuff and it looks like cargo selects that version instead of the specified 1.0.0-beta.2 (or beta.5 on next branch). Whyyy tho 😢
It doesn't to that for the beta versions btw (so beta.2 actually resolved to beta.2 instead of beta.5).

For now you can install the cli from a temporary branch like this: cargo install tauri-cli --git https://github.com/tauri-apps/tauri --branch tmp/pin-clap

@FabianLars the tmp/pin-clap has worked. Thanks a lot

You can use next branch instead of tmp/pin-clap once we merge #2928.

Just a heads up for future readers, tmp/pin-clap is now gone, use next.


Debug Server is running but VSCode says it is not

Trying to debug a COBOL program, starting the debugger reports Debug Server is not running but is indeed running.

Context
Version

Code for IBM i version
2.10.1

Visual Studio Code version
1.89.1

Operating System
linux_x64

Active extensions
COBOL (cobol): 10.5.29
Code Coverage for IBM i (code-coverage-ibmi): 0.1.7
Code for IBM i Walkthroughs (vscode-ibmi-walkthroughs): 0.5.0
CodeLLDB (vscode-lldb): 1.10.0
Db2 for IBM i (vscode-db2i): 1.0.0
Dev Containers (remote-containers): 0.362.0
ESLint (vscode-eslint): 2.4.4
EditorConfig for VS Code (EditorConfig): 0.16.4
Emmet (emmet): 1.0.0
Error Lens (errorlens): 3.18.0
Git (git): 1.0.0
Git Base (git-base): 1.0.0
GitHub (github): 0.0.1
GitHub Authentication (github-authentication): 0.0.2
GitHub Pull Requests (vscode-pull-request-github): 0.88.1
IBM i Debug (ibmidebug): 1.0.0
IBM i Notebooks (vscode-ibmi-notebooks): 0.0.6
JSON Language Features (json-language-features): 1.0.0
Live Share (vsliveshare): 1.0.5918
Merge Conflict (merge-conflict): 1.0.0
MongoDB for VS Code (mongodb-vscode): 1.6.0
Node Debug Auto-attach (debug-auto-launch): 1.0.0
Path Intellisense (path-intellisense): 2.8.5
Prettier - Code formatter (prettier-vscode): 10.4.0
Server Ready Action (debug-server-ready): 1.0.0
TODO Highlight (vscode-todo-highlight): 1.0.5
TypeScript and JavaScript Language Features (typescript-language-features): 1.0.0
Vite (vite): 0.2.5
Vue - Official (volar): 2.0.10

Remote system

Setting
Value

IBM i OS
V7R5M0

Tech Refresh
3

CCSID Origin
user

Runtime CCSID
37

Default CCSID
37

SQL
Enabled

Source dates
Disabled

Enabled features

/QOpenSys/pkgs/bin
/usr/bin

/QSYS.LIB
/QIBM/ProdData/IBMiDebugService/bin

bash
attr
ILEDITOR.GETMBRINFO
QZDFMDB2.PGM
startDebugService.sh

chsh
iconv
ILEDITOR.GETNEWLIBL

git
setccsid

grep

ls

md5sum

sort

stat

tar

tn5250

Shell env
ANT_HOME=/opt/apache-ant-1.10.1
BUILDLIB=JWOEHR
CC=gcc-10 -pthread
CURLIB=JWOEHR
CXX=g++-10 -pthread
HISTFILESIZE=50000
HISTSIZE=50000
HOME=/home/JWOEHR
HOST=**************
JAVA_HOME=/QOpenSys/pkgs/lib/jvm/openjdk-11
LC_ALL=EN_US.UTF-8
LIBLS=QTEMP QGPL
LOGIN=jwoehr
LOGNAME=jwoehr
MAIL=/var/spool/mail/jwoehr
NVM_CD_FLAGS=
NVM_DIR=/home/JWOEHR/.nvm
OBJECT_MODE=64
OLDPWD=/home/JWOEHR
PASE_USRGRP_LIMITED=N
PATH=/home/JWOEHR/bin:/home/JWOEHR/.local/bin:/QOpenSys/pkgs/lib/jvm/openjdk-11/bin:/QOpenSys/pkgs/bin:/QOpenSys/pkgs/sbin:/QOpenSys/usr/bin:/usr/local/bin:/QOpenSys/pkgs/lib/nodejs12/bin:/QOpenSys/usr/bin:/usr/ccs/bin:/QOpenSys/usr/bin/X11:/usr/sbin:.:/usr/bin:/opt/apache-ant-1.10.1/bin
PS1=\u@\h:\w\$
PWD=/home/JWOEHR
SHELL=/QOpenSys/pkgs/bin/bash
SHLVL=1
SSH_CLIENT=******************
SSH_CONNECTION=****************
TERM=xterm
TZ=<EST>5<EDT>,M3.2.0,M11.1.0
UBLU_HOME=/opt/ublu
USER=jwoehr
USERNAME=jwoehr
WORKDIR=/home/JWOEHR
_=/QOpenSys/pkgs/bin/env

Variants
{
  "american": "#@$",
  "local": "#@$"
}

Errors
[
  {
    "command": "/QOpenSys/usr/bin/qsh",
    "code": 1,
    "stderr": "CPF2111:  Library ILEDITOR already exists.",
    "cwd": "/home/JWOEHR"
  },
  {
    "command": "cd '/home/JWOEHR/work' && /QOpenSys/pkgs/bin/stat --dereference --printf=\"%A\t%h\t%U\t%G\t%s\t%Y\t%n\n\" * .*  ",
    "code": 1,
    "stderr": "/QOpenSys/pkgs/bin/stat: cannot stat '.*': A file or directory in the path name does not exist.",
    "cwd": "/home/JWOEHR"
  },
  {
    "command": "cd '/home/JWOEHR/work/cobol' && /QOpenSys/pkgs/bin/stat --dereference --printf=\"%A\t%h\t%U\t%G\t%s\t%Y\t%n\n\" * .*  ",
    "code": 1,
    "stderr": "/QOpenSys/pkgs/bin/stat: cannot stat '.*': A file or directory in the path name does not exist.",
    "cwd": "/home/JWOEHR"
  }
]

Are both the Debug Server and Service shown as running in the IBM i Debugger view?

Yes, it shows in that view as running! How odd ...

We have two sets of debug service PTFs before. The older ones are:
v7r3 PTF SI80858, v7r4 PTF SI81031 & v7r5 PTF SI81035
And the newer ones are:
V7r3 PTF SI83249, v7r4 PTF SI83261 & V7r5 PTF SI83262
The even newer v2.0 PTFs will be out soon. Please let us know which PTF you have.

IBM i Debug v2.0.1 was just released. It contains a number of fixes in debugger startup. Please upgrade to the v2 client (a host PTF update is also required) and try the same scenario again.

Where is the PTF announced, please?

IBM i Debug v2 client is available here:
https://marketplace.visualstudio.com/items?itemName=IBM.ibmidebug&ssr=false#overview
The Overview page also describes the required host PTF update.


Development with libcurl list not completely displayed

See screenshot, third item is partially hidden.

Safari on macOS

Right, thanks. Someone else also pointed this out. Clearly the CSS is not good enough.
I welcome help / replacements to address this problem and maybe make the entire thing look better.

I know, saw your toot on Mastodon, that's the reason I came here to report it.


Jinja templating and google tts not working?

Home Assistant release (hass --version):
0.45.0
Python release (python3 --version):
Python 3.4.2
Component/platform:
google tts/raspberry pi
Description of problem:
Hello :)
For the Google TTS (or possible other services as well) Jinja templating doesn't seem to work. The service simply gets the object descriptor and then synthesises it, result in the voice line:

It is states.sensor.time_utc.state.

The exact automation is shown below.
Notice how I also configured an iOS notification which yields the expected result (the given time embedded in the string).
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:
  - alias: Full minute
    trigger:
        platform: time
        seconds: 00
    action:
      - service: tts.google_say
        data:
          message: "It is {{ states.sensor.time_utc.state }}."
      - service: notify.ios_maurice_iphone
        data:
          title: 'Full minute'
          message: "It is {{ states.sensor.time_utc.state }}."

Use data_template instead of data, like so:
    action:
      - service: tts.google_say
        data_template:
          message: "It is {{ states.sensor.time_utc.state }}."

 Thanks!


Prevent false map click after fast dragging of marker

Fix #4457 (and duplicates).
Fix final position after dragging
For unclear reason _startPos was set wrong (increasing it for offset of first move event), so we fix that,
like it was in initial implementation (no idea why it was ever changed).
(https://github.com/Leaflet/Leaflet/commit/2d2fc74110f2fe7fbc06d7f6e0a2e90528ce46c6#diff-d8f00e9d1726fd6483110dc44c93d9f8R84)
Prevent false map click after fast dragging of marker
To solve this we have to ensure that draggable final position is already set on the time
of mouseup/touchend. Thus we have only option - reject using of requestAnimationFrame,
which was originally introduced to fix performance issues of map panning, see
https://github.com/Leaflet/Leaflet/commit/c93d91bfad73483990fd5340ed0db8908ba5a4b6#diff-46eca63ac2c5237cbd2d45c8a60f6ee28ce34231aef53669247ead03f012dff2
Perhaps browsers evolved since then.
Adapt test to expect exact position values
(as we do not depend on timings anymore)
Todo:

[ ] Check panning performance.
In case if it is not sufficient we could use requestAnimationFrame but for map only.

Demo pages:

Original bug: https://gist.githack.com/johnd0e/279b2c5bd985027fd3f959d8b1cec9ea/raw/bug.html
Fixed with this PR: https://gist.githack.com/johnd0e/279b2c5bd985027fd3f959d8b1cec9ea/raw/fixed.html

Looks good to me, I can confirm it solves both the offset and the fast dragging issue. I have only tested DivIcon Marker, not other elements, nor performance.
The Marker.DragSpec should have detected the offset bug, so I think it should be fixed as well.
Either by also removing the moveBy(5, 0, 20), whose omission by the wrong offset fell within the tested tolerance. Or by testing exact values instead of a within range. I don't know if the timing issue mentioned in the comment still exists or if it was just assumed because the offset bug produced an unexpected result. Repeated tests with exact values do succeed, yet I do see some sporadic failures when calling a reduced spec/index.html (only Marker.Drag and Map.Drag spec) in Chromium and Firefox, but those also occur on current master and the failing results are far outside the tested tolerance.
Testing for pixel values as used for moving instead of coordinates would also make the test better understandable, so my proposal would be to replace the current expect statements with this:
var markerPos = marker._icon._leaflet_pos;
expect(markerPos.x).to.be(300 + 5 + 256);
expect(markerPos.y).to.be(300 + 32);

nor performance.

I hope that you'll find some time to check that too

Either by also removing the moveBy(5, 0, 20),

I'm thinking about returning that move (but replace following moveBy with moveTo).

Testing for pixel values as used for moving instead of coordinates would also make the test better understandable

Sounds good, but it can be not so obvious for scaled container test.

The Marker.DragSpec should have detected the offset bug, so I think it should be fixed as well.

How to reproduce that?
There is no problems on travis: https://travis-ci.org/github/Leaflet/Leaflet/builds/757065282
Do you mean "in CSS scaled container" tests? Make sure that you have browser window is large enough (like 1280*1024).

There is no problems on travis: https://travis-ci.org/github/Leaflet/Leaflet/builds/757065282

What I mean is it should fail on master, because there is a bug.
But I see you already implemented what I meant.

What I mean is it should fail on master, because there is a bug.

PhantomJS 2.1.1 (Windows 8) Map.Drag mouse events change the center of the map FAILED
        Error: expected { x: 251, y: 32 } to sort of equal { x: 256, y: 32 } (node_modules/expect.js/index.js:102)
PhantomJS 2.1.1 (Windows 8) Map.Drag mouse events in CSS scaled container change the center of the map, compensating for CSS scale FAILED
        Error: expected { x: 253.5, y: 32 } to sort of equal { x: 256, y: 32 } (node_modules/expect.js/index.js:102)
PhantomJS 2.1.1 (Windows 8) Map.Drag touch events change the center of the map FAILED
        Error: expected { x: 251, y: 32 } to sort of equal { x: 256, y: 32 } (node_modules/expect.js/index.js:102)
PhantomJS 2.1.1 (Windows 8) Marker.Drag drag drags a marker with mouse FAILED
        Error: expected { x: 251, y: 32 } to sort of equal { x: 256, y: 32 } (node_modules/expect.js/index.js:102)
PhantomJS 2.1.1 (Windows 8) Marker.Drag drag pans map when autoPan is enabled FAILED
        Error: expected { x: 285, y: 32 } to sort of equal { x: 290, y: 32 } (node_modules/expect.js/index.js:102)
PhantomJS 2.1.1 (Windows 8) Marker.Drag drag in CSS scaled container drags a marker with mouse, compensating for CSS scale FAILED
        Error: expected { x: 253.5, y: 32 } to sort of equal { x: 256, y: 32 } (node_modules/expect.js/index.js:102)
PhantomJS 2.1.1 (Windows 8): Executed 16 of 704 (6 FAILED) (1.307 secs / 1.254 secs)

@Falke-Design
Could you confirm that there is no performance loss (see first message)?

I dragged the map, the marker, the DivIcon, clicked and has done some monkey tests but there was never a performance loss.

...but there was never a performance loss.

Perhaps to see it we need hard conditions, like large map with many objects.

@mourner I don't know how fast we get response from @johnd0e, he was not available since 2022. If you want you can give me write access to update the PR. The other option is that you change it or we remove it from the 1.8 Release. But the PR would be ready after the changes

If you want you can give me write access

Just did so!  Just make sure you don't merge potentially breaking or performance-impacting PRs without other maintainer's review.

Thank you, I will be careful!

There is a open todo:

Todo:

[ ] Check panning performance.
In case if it is not sufficient we could use requestAnimationFrame but for map only.


Quick Action Menu: Text Element

Context
Part of new quick action menu feature in editor.
Summary
Adds a new menu type for when a single text element is selected on the canvas. Has ability to: 1) navigate to change text color 2) navigate to change font 3) add animation 4) add link 5) if animation is present clear animation
Relevant Technical Choices

Renamed the highlight states.TEXT to be states.TEXT_SET since adding in states for the text style panel makes that bit a little confusing.
Gave CHANGE_TEXT_COLOR it's own copy constant because if copy changes I want to make sure it's an easy update despite right now it being the same as changing shape color. Also makes karma fixture easier to read since we're setting up each button there.
2 new highlights - TEXT_COLOR and FONT - both go to the text style panel.
Had to forward some refs in the textStyle component 😅   - only added forwardRef to DropDown - otherwise just passing it through the levels as a prop since we don't need it until getting to the actual input. Color already had a forwardRef I could tap into.
Purposefully didn't update the isPersistable prop in the StylePanel because we always expand that panel when a text element is either added or selected in the canvas. This lets the highlight work as expected while maintaining existing functionality (shout out Karma tests for being really useful to avoid this regression!)

To-do

You'll notice that the font picker doesn't highlight, it does focus though. Once the keyboard update #7623 is merged this will show the focus outline when the keyboard is getting used.
Something potentially odd is happening in the textStyle tests. It's passing, but if you go to the javascript tests below and just scroll through them a little you will see two warnings like so: Check the render method of FontPicker. It's got to do something with how this whole panel is getting mocked to test the sub sections of it? I spent some time debugging but the render methods seem fine. If anyone reading this knows what's up please let me know!

User-facing changes
Menu is behind a feature flag.
Testing Instructions
Make sure the quick actions experiment is active.
Add a text element to a page, while it is selected you should see the new quick action menu for them. Click the buttons and see that they take you to panels as specified in the summary (or the video below).
https://user-images.githubusercontent.com/10720454/119906659-7db8c400-bf03-11eb-9a27-bc364c9b814d.mp4
QA

[x] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

UAT

[x] UAT should use the same steps as above.

This PR can be tested by following these steps:

Reviews
Does this PR have a security-related impact?

Does this PR change what data or activity we track or use?

Does this PR have a legal-related impact?

Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #6146

Text quick actions work great!
A note: Just saw this in the figma designs:

I don't believe that the other panels close when the quick actions are clicked. Should we add a ticket for that?

I don't believe that the other panels close when the quick actions are clicked. Should we add a ticket for that?

They don't! We discussed this while you were out, design agreed to not have panels collapse since it breaks with the panel structure.  (#7750 )

Looks like clicking the none animation makes the animation quick action come back.

weird!! good catch! I think I can just fix that here real quick.


Update Overview.md

Fixed spelling --> “remoatly” to “remotely” in line 15.
Fixed spelling --> "challanging" to "challenging" in line 12.
Fixed spelling --> "can not" to "cannot" in line 12
Fixed spelling --> "ip" to "IP" in line 15
Fixed spelling --> "powershell" --> "PowerShell" in line 34
Fixed spelling --> "detailed" --> "details" in line 37
Fixed grammar as needed (lots of missing commas)

Docs Build status updates of commit d9fd031:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/manage-your-device/Overview.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Fix typo

ruby -> Ruby

@changeworld : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off

@jboback: I'm sorry - only the author of this article, @v-jaswel, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Not sure how to implement the "Tree-shakeable" feature to reduce loading time

Hi bherbruck,
I wanted to ask you if you can help me out how to implement the tree shakeable feature that you have implemented.
E.g. in my sveltekit component https://github.com/stocknear/frontend/blob/main/src/lib/components/DarkPool.svelte I plot a graph, where I want to import only files that are absolutely necessary to reduce loading time for the end user.

It looks like you're already doing it. Are you having an issue with your linked component?

Sorry I resolved this issue myself. Thank you for the awesome library


Show Search Results With PDS Styles

💪 Motivation
Present search results with PDS styling.
📖 Additional Details
Update state with search results. Show these results with the PDS styling and format. The PDS styles will have to be created.
⚖ Acceptance Criteria
Entering some text into the search box and pressing enter will show any results. They will be styled according to the mock ups and PDS WDS.

⚙ Engineering Details
Use saga to store the results in the state and show when updated.
Add the styles to the theme.

@eddiesarevalo status: working these styles out with material UI

still working styles

For now, @eddiesarevalo worked from a static result page and it works. He is now blocked because the API is not accessbile from a cross origin domain (CORS configuration).

CORS issue solved. Now connected to API

Added Card and connected to API at https://pds.nasa.gov/api/search-en-gamma/1.1/

On hover (animated zoom and hand pointer):

@eddiesarevalo will move the code to wds-react repo. The component does not need to be re-useable yet, that can be a prototype thing.


Offset simplified boundary polygons to ensure full coverage

Doing this with the same distance value guarantees coverage, but it does result on average in too large polygons.
A better solution would be some form of "simplified hull" algorithm, but that's not something easily available off the shelf it seems.
Add coverage polygons produced that way to IVB for demonstration.

Taking silence as lack of objections, I'd merge this and apply the same to other files with missing coverage areas then.


fix: max_output_tokens param ignored in config

Hi there, I was using AiChat with local KoboldCPP instance and noticed that max_output_tokens param is ignored and not inserted into the request body, causing koboldcpp api to only generate 100 tokens of output. Looks like the issue is related to the variable ownership due to passing option to another object, repacking it into a new option fixed the issue for me.
My config:
model: koboldcpp:kobold
function_calling: true
wrap_code: false
temperature: 0.5

clients:
- type: openai-compatible
  name: koboldcpp
  api_base: http://DESKTOP-IC823FN.local:5001/v1
  api_key: 'kobold'
  models:
  - name: kobold
    require_max_tokens: true
    max_output_tokens: 1024

Without fix:
Input: {"model": "kobold", "messages": [{"role": "user", "content": "write sample python class"}], "temperature": 0.5, "stream": true}

Processing Prompt (9 / 9 tokens)
Generating (100 / 100 tokens)
CtxLimit: 117/2048, Process:1.00s (111.7ms/T = 8.96T/s), Generate:15.87s (158.7ms/T = 6.30T/s), Total:16.88s (5.92T/s)
Output: Here's a simple example of a Python class that represents a book.

class Book:
    def __init__(self, title, author, pages):
        self.title = title
        self.author = author
        self.pages = pages

    def display_info(self):
        print(f"Title: {self.title}")
        print(f"Author: {self.author}")
        print(f

With fix:
Input: {"model": "kobold", "messages": [{"role": "user", "content": "write sample python class"}], "max_tokens": 1024, "temperature": 0.5, "stream": true}

Processing Prompt (1 / 1 tokens)
Generating (321 / 1024 tokens)
(EOS token triggered!)
CtxLimit: 338/2048, Process:0.39s (390.0ms/T = 2.56T/s), Generate:49.98s (155.7ms/T = 6.42T/s), Total:50.37s (6.37T/s)
Output: Sure, here's a simple example of a Python class that represents a basic "Car" object:

class Car:
    def __init__(self, make, model, year):
        self.make = make
        self.model = model
        self.year = year
        self.odometer_reading = 0

    def get_descriptive_name(self):
        long_name = f"{self.year} {self.make} {self.model}"
        return long_name.title()

    def read_odometer(self):
        print(f"This car has {self.odometer_reading} miles on it.")

    def update_odometer(self, mileage):
        if mileage >= self.odometer_reading:
            self.odometer_reading = mileage
        else:
            print("You can't roll back an odometer!")

    def increment_odometer(self, miles):
        self.odometer_reading += miles

Thanks for the great tool, keep it up!

The field should be pass_max_tokens in aichat v0.18.0. see #562
https://github.com/sigoden/aichat/blob/38797e3264ba9aa903b82b5879f5802e2597333b/src/client/model.rs#L202


New refdata "regional" for global field in package and additional field for country

Momentarily the package field "global" has three values: "global", "consortial" and "unknown/other". There should be an additional value "regional" that represents a package that is only specific to a certain country, region etc. Some Springer packages are an example.
Additionaly, there should be a field where you can select the specific region the package belongs to. This should be a refdata field with different countries.
This field should be integrated in the different APIs, so that third party systems can filter for their region.

Problems

A country list doesn't necessarily include all possible regions like "European Union" or "German-Austria-Swiss Region"
A multiple selection is hard to implement


Move test unit generator test to generators dir

Summary
This is a tiny change.
Though almost generator test files are in generators directory, only ViewComponent::TestUnitGeneratorTest was in view_component directory. So I moved them to generators directory,  and unified the code of them with other generators test files.

@Spone Thank you for your review :pray: Next time, I remember to add `` to the CHANGELOG code.


Questions about PTv3 and the efficacy of depthwise SubMConv3d

Hello, this is truly remarkable research! In the paper, when you mentioned using xCPE similar to the octree-based depthwise convolution (CPE), I assumed that depthwise submconv would be utilized. However, upon examining the code, I noticed that dense submconv was employed instead. Is there a specific reason for not using depthwise convolution? Additionally, in most point cloud processing backbone models, depthwise submanifold convolution is not typically used. Is this due to a significant performance drop, or is it because using dense convolution does not significantly increase computational overhead?

Thank you for the kind response! If I modify spconv.py to enable depthwise submconv, would there be any issues when using it? I'm considering using depthwise in my experiments for the following reasons, and I'd greatly appreciate your opinion on this as well:
3D data tends to be on a smaller scale compared to 2D datasets, making it more susceptible to overfitting issues, and I believe depthwise could help mitigate this.
Most 3D models use 3x3x3 kernels, but I think increasing them to 5x5x5 or 7x7x7 wouldn’t incur significant overhead.
Simply replacing regular conv with depthwise might reduce channel interaction capacity and degrade performance, but I think using it alongside MLPs, similar to PT models, might prevent performance loss.
Additionally, the depthwise submconv I'm referring to is available at https://github.com/LHDuan/ConDaFormer/blob/main/libs/spconv/conv.py (Neurips2023 ConDaFormer).

From what I've examined so far, it appears that the proposed modification simply multiplies a mask by the weights to mimic depthwise operation, which doesn't seem to offer any computational benefits.

From what I've examined so far, it appears that the proposed modification simply multiplies a mask by the weights to mimic depthwise operation, which doesn't seem to offer any computational benefits.

A well-optimized depthwise convolution is not trivial to implementation, I think it is the reason why from new SpConv still does not support it.

3D data tends to be on a smaller scale compared to 2D datasets, making it more susceptible to overfitting issues, and I believe depthwise could help mitigate this.

I think the Scaling Law can tell that over-parameters won't positively and negatively influence the final performance, so there is no need to care too much about parameters. The major issue is the scale of data, that's why my previous two works focus on pre-training technology on point cloud data

Most 3D models use 3x3x3 kernels, but I think increasing them to 5x5x5 or 7x7x7 wouldn’t incur significant overhead.

In PTv3, SpConv only serves as positional encoding, with no need for a large kernel. Attention is already doing well at large kernels.

Thanks for answering!!!


how could I run rancher without docker images

Rancher versions:
rancher/server or rancher/rancher: v2.0 +
rancher/agent or rancher/rancher-agent:v2.0+
I want to change some code,and debug. I don't know how to do,but I think if I can run code directly（Ubuntu/CentOS），things will be easier。
Any suggestions or methods will be grateful.

This page shows to run it into our preferred IDE Goland: https://github.com/rancher/rancher/wiki/Running-and-debugging-Rancher-2.0-from-an-IDE
Building and packaging is done in Docker but can easily be done by yourself if wanted: https://github.com/rancher/rancher/blob/master/scripts/build-server


port conflict for webhook on Fargate

What happened:
Using the default helm chart to run external dns on AWS Fargate there is a port conflict 10250 this port number is the default port for kubelet
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Deploy EKS with Fargate, deploy helm chart to run pods on Fargate pods
The helm chart values.yaml default value webhook.port should be changed to another value like 9443

If its ok I can submit PR to update https://github.com/external-secrets/external-secrets/blob/main/deploy/charts/external-secrets/values.yaml#L205
To webhook.port: 9443

What happened:
Using the default helm chart to run external dns on AWS Fargate there is a port conflict 10250 this port number is the default port for kubelet
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Deploy EKS with Fargate, deploy helm chart to run pods on Fargate pods
The helm chart values.yaml default value webhook.port should be changed to another value like 9443

any reason to not modify this directly in your values and do it in the chart itself?


model support with LMSKerasCallback on yolo

I've got a problem with toposort in large-model-support
Using TensorFlow backend. INFO:tensorflow:[LMS][0] Editing model for LMS INFO:tensorflow:[LMS][0] n_tensors: all tensors INFO:tensorflow:[LMS][0] lb: 1 Create YOLOv3 model with 9 anchors and 3 classes. Train on 6103 samples, val on 678 samples, with batch size 1. Traceback (most recent call last):   File "/net/scratch/people/plgmkowalski94/master/new/keras-yolo3/lms_train.py", line 216, in <module>     _main()   File "/net/scratch/people/plgmkowalski94/master/new/keras-yolo3/lms_train.py", line 88, in _main     callbacks=[logging, checkpoint, lms])   File "/net/people/plgmkowalski94/.local/lib/python3.6/site-packages/keras/legacy/interfaces.py", line 91, in wrapper     return func(*args, **kwargs)   File "/net/people/plgmkowalski94/.local/lib/python3.6/site-packages/keras/engine/training.py", line 1418, in fit_generator     initial_epoch=initial_epoch)   File "/net/people/plgmkowalski94/.local/lib/python3.6/site-packages/keras/engine/training_generator.py", line 94, in fit_generator     callbacks.set_model(callback_model)   File "/net/people/plgmkowalski94/.local/lib/python3.6/site-packages/keras/callbacks.py", line 54, in set_model     callback.set_model(model)   File "/net/scratch/people/plgmkowalski94/master/new/keras-yolo3/tensorflow_large_model_support/lms.py", line 957, in set_model     lmsMod.run()   File "/net/scratch/people/plgmkowalski94/master/new/keras-yolo3/tensorflow_large_model_support/lms.py", line 336, in run     self._topo_sort.build()   File "/net/scratch/people/plgmkowalski94/master/new/keras-yolo3/tensorflow_large_model_support/topos.py", line 45, in build     topo_sort = list(toposort.toposort(self._build_dependency_dict()))   File "/net/people/plgmkowalski94/.local/lib/python3.6/site-packages/toposort.py", line 81, in toposort     raise CircularDependencyError(data)   File "/net/people/plgmkowalski94/.local/lib/python3.6/site-packages/toposort.py", line 45, in __init__     s = 'Circular dependencies exist among these items: {{{}}}'.format(', '.join('{!r}:{!r}'.format(key, value) for key, value in sorted(data.items()))) TypeError: '<' not supported between instances of 'Operation' and 'Operation'
I'm using this code
https://pastebin.com/Nut6Ss3n

I'm experiencing the same issue using a seq2seq model using the corresponding tensorflow contrib module. There are cyclic dependencies within the BasicDecoder according to toposort.

Models with cycling dependencies are now supported with TensorFlow Large Model Support for TensorFlow 2 (currently in the master branch level of this repository.)


Debug Terminal: Tx/Rx Indicators

Add Tx and Rx activity indicators for the Debug Terminal.
The Tx (transmit) and Rx (receive) indicators should be somewhere near the terminal itself.  They each individually flash once for every block of data transmitted or received.
More information may be read in the Settings section of the Debug Terminal document.

@PropGit I wanted to reach out to you regarding how the RX/TX indicator elements are to be implemented.  There seems to be some questions about how this element is to appear.  Could you provide us with a note about it's role in the user experience and your thoughts about it might look or where it should go?  Screenshots, mocks or wireframes would be excellent.
If there are any further questions I'll let you know.

@urbantumbleweed I'm somewhat open to suggestions here, but we definitely need to open up some area on screen where this and a few other terminal-related controls can reside.  Somewhere that is visible only when the Debug Terminal is visible.
Here's a crude mock-up of one possibility.  The circles next to TX and RX are like LED lights.  They each should light up when there's activity.

The point of these is to provide a quick visual indication that some data was received or transmitted as there are many cases when data can move but there's no visual change to the textual display in the Debug Terminal.  For any data movement event, the indicators should light up for a minimum of about 50 ms so that the user can see that something happened (even if it was just one character received at 3 Megabaud that cause it to light).
On our development boards, we use blue LEDs for Rx and red LEDs or Tx.  I think using this color scheme for this case would be good for consistency.

Completed by #147.  Will need to add some functionality to this when we implement DEBUGIN functionality, but that should be covered in that issue.


remove unnecessary status.internalregistryhostname field

After further consideration i think setting the internal registry hostname in the registry resource status was unnecessary, and not doing it simplifies the code and makes the generator more like the other generators.
As part of this PR I also fixed up the node-ca daemon to actually use the internalregistryhostname value, which it previously was not doing, and switched it to fetch the value from the image config resource instead of the registry resource.

/retest


container_create: Set a minimum memory limit

We set a minimum limit of 4MB. Lower values could result
in container failing to start.
Signed-off-by: Mrunal Patel<EMAIL_ADDRESS>

/test all

/test kata-containers

/test e2e

/test e2e

/test e2e

/test e2e_rhel

This is ready to be merged.


minor bug in chat history (NameError)

Minor bug fix needed in nano_llm/chat/history.py
Fails when chat_template is a dict
  File "/opt/NanoLLM/nano_llm/chat/history.py", line 74, in __init__
    self.template = AttributeDict(template)
NameError: name 'template' is not defined

Suggested Fix
self.template = AttributeDict(chat_template)

@jayant-mil fixed in https://github.com/dusty-nv/NanoLLM/pull/29. Now you can pass chat_template as dict :)


Errors trying to connect to puppet master

I keep getting this error spamming in the logs:
{:timestamp=>"2015-06-05T20:20:17.328000+0000", :message=>"A plugin had an unrecoverable error. Will restart this plugin.\n  Plugin: <LogStash::Inputs::PuppetFacter host=>\"pupmaster.example.net\", private_key=>\"/etc/logstash/logstash.crt\", public_key=>\"/etc/logstash/logstash.key\", tags=>[\"elasticsearch\"], environment=>\"production\">\n  Error: No message available", :level=>:error}

This config:
input {
  puppet_facter {
    host => "pupmaster.example.net"
    interval => 3600
    private_key => "/etc/logstash/logstash.crt"
    public_key => "/etc/logstash/logstash.key"
    tags => ["elasticsearch"]
  }
}

I have read the plugin and tried connecting to our puppet master with valid certs using curl:
curl --cert /etc/logstash/logstash.crt --key /etc/logstash/logstash.key -k https://pupmaster.example.net:8140/production/certificate_statuses/no_key
Forbidden request: logstash() access to /certificate_status/no_key [search] authenticated  at :120[

I am thinking there is some puppetmaster configuration that needs to be done. Something in auth.conf maybe.
https://docs.puppetlabs.com/puppet/latest/reference/config_file_auth.html

I'm having the same issue, and am interested in getting this logstash plugin working. Any progress on this? I am not at all clear how to obtain the required certs, do we use the existing certs in /var/lib/puppet/ssl?

I never did get this working instead I opted for a puppetdb api in a python script to get the data I needed. https://github.com/puppet-community/pypuppetdb

I'll have a look at it this week, see if i can find out what's wrong with it and report back.

Thanks for the replies! It is nice to know other options, though I'd also like to get puppet_facter working.

Did a bit of digging.
You can re-use the existing puppet certs in /var/lib/puppet/ssl yeah.
The only thing you will need to do is give the logstash instance access to the api's via the auth.conf file since most api calls are blocked.
For example https://github.com/logstash-plugins/logstash-input-puppet_facter/blob/master/lib/logstash/inputs/puppet_facter.rb#L62 is blocked.
and https://github.com/logstash-plugins/logstash-input-puppet_facter/blob/master/lib/logstash/inputs/puppet_facter.rb#L81 is locked by default that only the requesting node can see its own stuff.

Hi,
Has anyone been able to get this plugin to work? I'm having issues configuring it.
I have a similar set up to the original poster's configuration. And I've gone through @electrical 's steps above.
I've tried the following curl commands and they both work:
`
curl --cert  --key  -k https://puppetserver:8140/production/certificate_statuses/no_key
curl --cert  --key  -k https://puppetserver:8140/production/facts/machinename
`
I also get a validated configuration file when I run the logstash -f config.conf -t
Both keys have read permissions set.
However when I run the service I'm getting this:

[2017-08-09T08:49:55,455][ERROR][logstash.inputs.puppetfacter] Invalid setting for puppet_facter input plugin:
input {
puppet_facter {
# This setting must be a path
# File does not exist or cannot be opened /var/lib/puppet/ssl/certs/certhere.pem
public_key => "/var/lib/puppet/ssl/certs/certhere.pem"
...
}
}
[2017-08-09T08:49:55,455][ERROR][logstash.inputs.puppetfacter] Invalid setting for puppet_facter input plugin:
input {
puppet_facter {
# This setting must be a path
# File does not exist or cannot be opened /var/lib/puppet/ssl/private_keys/private.pem
private_key => "/var/lib/puppet/ssl/private_keys/private.pem"
...
}
}


fix: updated dependencies

:tada: This PR is included in version 1.10.88 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Feature] Why is the LSS-based 3D detection method not provided in the new version of codebase?

What is the feature?
Thanks for your excellent job. However, I think is so convenient for users to build their own model on the base of the LSS transformer codes in previsous mmdet3d, in the new version , I can not find any codes relatived.
Any other context?
No response

You can view the similar code of BEVFusion in projects/BEVFusion


New problem function: Team Pursuit Track Cycling

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context / Related issue

Addition of a real-world problem which sets up future benchmarking studies. Adding functionality and broadening the Nevergrad platform. I would also like to participate in the paper track of the Open Optimisation Competition. A paper titled "Extending Nevergrad, an Optimisation Platform" is included in the pull request (found in the /nevergrad/functions/cycling directory). This paper serves as the documentation for this pull request.

Not an issue, but it is a problem that represents a physical model - more interesting than an artificial test function.

How Has This Been Tested (if it applies)

An experiment called TeamCycling has been created in experiments.py. The experiment relates to the paper "Extending Nevergrad, an Optimisation Platform".
Checklist

[x] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CLA).
[x] All tests passed, and additional code has been covered with new tests.

Hi Jeremy Rapin,
Apologies for the very late reply.
I have finally made all the changes that you have suggested and have
committed (and pushed) these to my branch teamCycling.
I am still a bit of a rookie with git - is there anything else I need to
do? Do I need to open another pull request for these changes to be official?
Kind regards,
Ryan
On Sat, Jun 19, 2021 at 1:42 AM Jérémy Rapin @.***>
wrote:

@.**** requested changes on this pull request.
Hi,
Thanks for the PR
A few initial comments
For the paper part, we'll need @teytaud https://github.com/teytaud 's
inputs
In nevergrad/functions/cycling/Cyclist.py
https://github.com/facebookresearch/nevergrad/pull/1153#discussion_r654537694
:

@@ -0,0 +1,119 @@

+#!/usr/bin/env python3
you'll need to copy paste the standard header from other files to all of
your new files
In nevergrad/functions/cycling/Cyclist.py
https://github.com/facebookresearch/nevergrad/pull/1153#discussion_r654539209
:

@@ -0,0 +1,119 @@

+#!/usr/bin/env python3

+from .TeamPursuit import TeamPursuit
I try to keep pep8 standards throughout the code so that it feels uniform
and simpler to read.
For your PR, this means quite a few name changes

module names should be lowercase and attached
(TeamPursuit->teampursuit, etc)
methods/functions/variables should be lowercase with underscores
(meanMaximumPower -> mean_maximum_power, etc....)
I've seen tabs, please only use 4-space indents

In nevergrad/functions/cycling/Cyclist.py
https://github.com/facebookresearch/nevergrad/pull/1153#discussion_r654539933
:

   if self.gender == "male":

       self.totalEnergy = self.meanMaximumPower * self.weight * 240

   else:

       self.totalEnergy = self.meanMaximumPower * self.weight * 210

less duplication is always beneficial on the long run (fewer bugs if
changes are not propagated everywhere, easier to change, faster to read):
⬇ Suggested change

   if self.gender == "male":

       self.totalEnergy = self.meanMaximumPower * self.weight * 240

   else:

       self.totalEnergy = self.meanMaximumPower * self.weight * 210

   coeff = 240 if self.gender == "male" else 210

   self.totalEnergy = self.meanMaximumPower * self.weight *  coeff

In nevergrad/functions/cycling/MensTeamPursuit.py
https://github.com/facebookresearch/nevergrad/pull/1153#discussion_r654541641
:

+from .TeamPursuit import TeamPursuit

+from .Cyclist import Cyclist
+from .SimulationResult import SimulationResult
+import sys
+import math

+class MensTeamPursuit(TeamPursuit):

TEAM_SIZE = 4

RACE_DISTANCE = 4000

LAP_DISTANCE = 250

RACE_SEGMENTS = int((2 * (RACE_DISTANCE / LAP_DISTANCE)) - 1)

MAXIMUM_TRANSITIONS = RACE_SEGMENTS - 1

def init(self):

super().updateAirDensity()

A couple of things here:

you should always call the init method of the parent class
no need to call super().updateAirDensity(), you can directly use
self, unless you have overriden it but wants to explicitely call the
parent's one

⬇ Suggested change

super().updateAirDensity()

super().__init__()

self.updateAirDensity()

In nevergrad/functions/cycling/MensTeamPursuit.py
https://github.com/facebookresearch/nevergrad/pull/1153#discussion_r654542735
:

TEAM_SIZE = 4

RACE_DISTANCE = 4000

LAP_DISTANCE = 250

RACE_SEGMENTS = int((2 * (RACE_DISTANCE / LAP_DISTANCE)) - 1)

MAXIMUM_TRANSITIONS = RACE_SEGMENTS - 1

def init(self):

super().updateAirDensity()

self.team = []

for i in range(0, self.TEAM_SIZE):

	self.team.append(Cyclist(1.75, 75.0, 6.0, self, i+1, "male"))

def simulate(self, transitionStrategy, pacingStrategy):

if (len(transitionStrategy) != self.MAXIMUM_TRANSITIONS):

	sys.exit("Transition strategy for the mens team pursuit must have exactly " + str(self.MAXIMUM_TRANSITIONS) +" elements")

Please raise errors instead of exiting, and you can also use f-string for
formatting string, they are very coder-friendly:
⬇ Suggested change

	sys.exit("Transition strategy for the mens team pursuit must have exactly " + str(self.MAXIMUM_TRANSITIONS) +" elements")

	raise ValueError(f"Transition strategy for the mens team pursuit must have exactly {self.MAXIMUM_TRANSITIONS} elements")

In nevergrad/functions/cycling/WomensTeamPursuit.py
https://github.com/facebookresearch/nevergrad/pull/1153#discussion_r654544707
:

@@ -0,0 +1,86 @@

+#!/usr/bin/env python3

+from .TeamPursuit import TeamPursuit
+from .Cyclist import Cyclist
+from .SimulationResult import SimulationResult
+import sys
+import math

+class WomensTeamPursuit(TeamPursuit):
how much of this is shared with MensTeamPursuit? There seems to be a lot
of similar code. Factorizing it will be helpful on the long term, at the
very least for maintenance (and some remarks left on the other class are
similar for this class, so instead of duplicating work, it's actually
simpler to find a way to merge them ;) )
In nevergrad/functions/cycling/cycling.py
https://github.com/facebookresearch/nevergrad/pull/1153#discussion_r654545266
:

	init = 550 * np.ones(Strategy)

	parameter = p.Array(init=init)

	parameter.set_bounds(200, 1200)

New simplified API:
⬇ Suggested change

	init = 550 * np.ones(Strategy)

	parameter = p.Array(init=init)

	parameter.set_bounds(200, 1200)

	init = 550 * np.ones(Strategy)

	parameter = p.Array(init=init, lower=200, upper=1200)

In nevergrad/functions/cycling/cycling.py
https://github.com/facebookresearch/nevergrad/pull/1153#discussion_r654545725
:

	super().__init__(WomensTeamPursuitSimulation, parameter)

else raise an exception
In nevergrad/functions/cycling/cycling.py
https://github.com/facebookresearch/nevergrad/pull/1153#discussion_r654546130
:

MENS_PACING_STRATEGY = [550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550]

⬇ Suggested change

MENS_PACING_STRATEGY = [550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550, 550]

MENS_PACING_STRATEGY = [550] * 30

In nevergrad/functions/cycling/cycling.py
https://github.com/facebookresearch/nevergrad/pull/1153#discussion_r654546362
:

Simulate event with the default strategies

result = mensTeamPursuit.simulate(MENS_TRANSITION_STRATEGY, MENS_PACING_STRATEGY)

#print(result.getFinishTime())

if result.getFinishTime() > 10000: # in case of inf

return 10000

else:

return float(result.getFinishTime())

+def WomensTeamPursuitSimulation(x: np.ndarray) -> float:

if len(x) == 22:

WOMENS_TRANSITION_STRATEGY = x

WOMENS_PACING_STRATEGY = [400, 400, 400, 400, 400, 400, 400, 400, 400, 400, 400, 400, 400, 400, 400, 400, 400, 400, 400, 400, 400, 400, 400]

simplify as well ;)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/facebookresearch/nevergrad/pull/1153#pullrequestreview-687551225,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMADP7NTZQIOVCGY4I6CXSDTTNV6FANCNFSM465CPXMQ
.

Deprecated by https://github.com/facebookresearch/nevergrad/pull/1387


readloop speedup

The old rw will error out on read() op so we don't need to recheck rw on each iteration

Note: not final PR yet, adding pooling of read buffer.

This last speed up was great, please review


CVRPPAD problem with delivery, pickup have demand delivery do not match vehicle capactity

What version of OR-tools and what language are you using?
Version: v7.2
Language: Python
I am having some problems in the CVRPPAD problem. Vehicles departing with all the goods to be delivered (delivery), on the delivery points, can always take action (pickup) and return to the starting point.
I created the data by doubling the C position, forming C 'with C (positive demand +20, delivery), C' (negative demand -20, pickup).
The model was still working, but the result was not what I expected, the total delivery of the entire route exceeded the vehicle's capacity.
For example A (depot) -> B (delivery) -> C (delivery) -> C ' (pickup) -> D (delivery) -> A (depot).
Capacity of vehicle is 50, at B need delivery 20, at C need delivery 20, at C ' need pickup 15 and at D need delivery 20.  (20 + 20 -15 +20 = 45 meet the conditions 50)
Something's wrong, the starting vehicle capacity must be 60 !!!
Do you have any suggestions? Thank very much.

You should use the mailing list instead.
You problem here is the pickup quantity is used to make the next delivery...
IMHO you should create 2 dimensions, one for deliveries and one for pickup, then add constraint that the sum of both CumulVar at any location can't exceed the vehicle capacity...

You should use the mailing list instead.
You problem here is the pickup quantity is used to make the next delivery...
IMHO you should create 2 dimensions, one for deliveries and one for pickup, then add constraint that the sum of both CumulVar at any location can't exceed the vehicle capacity...

For this example:
Vehicle starting with 60 demand

at B, delivery 20, vehicle capacity still 40
at C, delivery 20, vehicle capacity still 20
at C', pickup 15, vehicle capacity add 15 => 35
at D, delivery 20, vehicle capacity still 15
at depot A, vehicle capacity have 15

The purpose of delivery is to maximize the capacity of the vehicle, with the shortest distance, optimize the least cars.I further enhance the delivery and pick up, delivery from the depot goes to the point of need, and the goods from the depot must also go to the depot.


yiisoft/yii-bootstrapX maintenance

Opening a discussion about yii-bootstrap3 & yii-bootstrap4 maintenance.
Since at this exact moment, both these packages are "perfect" copies (with Yii3 adaptations) of yii2-bootstrap & yii2-bootstrap4, I feel that we may want to adopt the following rules to avoid divergences between the two versions:

Only accept new features & enhancements in Yii 2 packages (most activity for now)
Only update Yii 3 packages to backport new changes in Yii 2 packages, or improve the internal Yii-3 stuff (di, config, etc)

Someone could easily watch over Yii 2 changes an be responsible of copying.

Sounds OK to me. Do you want to take care of these?


Cannot see build definition selection in Configuration

The only Tile, Width, and Height are shown in Configuraiton

@leonids2005, can you please attach a screenshot of what you are seeing?

Actually, there is a second widget from you in the markeplace - and I can
select build definition there.
On Mon, May 11, 2020 at 5:27 PM Shane Davis<EMAIL_ADDRESS>wrote:

@leonids2005 https://github.com/leonids2005, can you please attach a
screenshot of what you are seeing?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sdavis3/CodeCoverageDashboardWidgets/issues/35#issuecomment-626809158,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSJXZ4B3AKMX7R5CUAPLU3RRAROBANCNFSM4M6AWP4A
.

strange, but now everything works ok.  But believe me when I first
installed it I could select build definition and other controls.
On Mon, May 11, 2020 at 8:44 PM Leonid Sokolovsky<EMAIL_ADDRESS>wrote:

Actually, there is a second widget from you in the markeplace - and I can
select build definition there.
On Mon, May 11, 2020 at 5:27 PM Shane Davis<EMAIL_ADDRESS>wrote:

@leonids2005 https://github.com/leonids2005, can you please attach a
screenshot of what you are seeing?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sdavis3/CodeCoverageDashboardWidgets/issues/35#issuecomment-626809158,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSJXZ4B3AKMX7R5CUAPLU3RRAROBANCNFSM4M6AWP4A
.

@leonids2005 a second widget in the marketplace? Can you share a link to it? There shouldn't be a second widget in there.

I am having the same problem, there is no way for me to configure the build I want to use.

And as for 2 in the marketplace, here they are

I'm having the same problem.. there is no option to choose build definition, only Width & Height

@rofehaim & @jvanzella Did this just start or has it been this way since you installed the widget? I'm attempting to reproduce.

This was my first install.  I can't identify them, but one of the Widgets works in the store works, and one of them doesn't.  See my first image.

Yes, I suspect it’s the other one or they are conflicting. I’m trying to get the clone removed from the marketplace. Uninstall both widgets and then go back and reinstall mine. It’s at the following url: https://marketplace.visualstudio.com/items?itemName=shanebdavis.code-coverage-dashboard-widgets

I uninstalled the extension and reinstall it from your link but still the same behavior, I just have Hight and Width to choose..

I'm unable to reproduce this issue with my extension. I do plan to release a new version soon with some additional features and I want to be sure I'm capturing any bug fixes related to this issue if I can.
With the rouge copy cat extension now removed, please uninstall the version you have currently installed and reinstall with the Code Coverage Widgets by Shane Davis.

Ok, I reinstalled it and look like the issue was fixed.. Thanks!

Excellent! I will close this item. Thank you for responding!


HIVE-24031: Infinite planning time on syntactically big queries

What changes were proposed in this pull request?

Drop the defensive copy of children inside ASTNode#getChildren.
Protect clients by accidentally modifying the list via an
unmodifiable collection.

Why are the changes needed?
Profiling shows the vast majority of time spend on creating defensive
copies of the node expression list inside ASTNode#getChildren.
The method is called extensively from various places in the code
especially those walking over the expression tree so it needs to be
efficient.
Most of the time creating defensive copies is not necessary. For those
cases (if any) that the list needs to be modified clients should perform
a copy themselves.
Does this PR introduce any user-facing change?
No
How was this patch tested?
The test was added in a separate branch since it is not meant to be committed upstream for the following reasons:

the query for reproducing the problem takes up a few MBs
requires some changes in the default configurations.

If you want to run the test run the following commands:
git checkout -b HIVE-24031-TEST master
git pull git@github.com:zabetak/hive.git HIVE-24031-PLUS-TEST
mvn clean install -DskipTests
cd itests
mvn clean install -DskipTests
cd qtest
mvn test -Dtest=TestMiniLlapLocalCliDriver -Dqfile=big_query_with_array_constructor.q -Dtest.output.overwrite

Closing pull request to trigger pre-commits