Feature: view findings per engagement

Based on the CICD example in https://github.com/aaronweaver/defectdojo_api and https://github.com/aaronweaver/AppSecPipeline/blob/master/tools/defectdojo/dojo_ci_cd.py I have setup our DD and Jenkins pipelines to use CICD engagements and upload a new scan every day.
Every branch has their own engagement. This works fine, except that DD by default shows findings per product. So you get findings for different engagements (branches) mixed through each other.
We have discovered we "need" to be able to view findings per engagement.
This PR is a start / work in progress to see if it can be achieved and if we can fit it into the UI in some more or less intuitive way. Curently the UI is based on products and the product tab provides menu entries to view all/open/critical/... findings per product. I would like to have something similar for engagements.
For now I have added a menu item to view all findings for an engagement to the view product page and the view engagment page.
Let me know what you think!

@valentijnscholten Looks good and very useful, just a few flake8 items:
Found violations: dojo/engagement/urls.py
	W291 @ 42:73 - trailing whitespace
Found violations: dojo/engagement/views.py
	F821 @ 1017:27 - undefined name 'Engagment'
	F821 @ 1024:14 - undefined name 'ProductFindingFilter'
Found violations: dojo/finding/views.py
	E265 @ 69:5 - block comment should start with '# '
	E265 @ 579:13 - block comment should start with '# '
	E265 @ 1513:5 - block comment should start with '# '
Found violations: dojo/models.py
	E265 @ 808:5 - block comment should start with '# '

Done. I also removed a view and url pattern that I previously added as I don't think it is used/needed.
I will remove the TODOs once I have done some more testing on monday.
I will probably also add a "View Open Findings" menu item below the "View All Findings" I already added.

This is now ready for merging I believe.

@valentijnscholten some TODO items seem to still be around , do you still need them or can they be removed ? :) Otherwise this looks great !

I left them as help for anyone who might want to build more "per engagement" stuff. But maybe better if in remove them?

@valentijnscholten, agree with @adracea  I'd probably remove the todo comments. Looks good!


Python Server 1.0.0 raises serialisation exception when return grpc error

I modified example hello world to return grpc error with context.set_details and context.set_code
diff --git a/examples/python/helloworld/greeter_server.py b/examples/python/helloworld/greeter_server.py
index 37d8bd4..dd32ea1 100644
--- a/examples/python/helloworld/greeter_server.py
+++ b/examples/python/helloworld/greeter_server.py
@@ -42,7 +42,9 @@ _ONE_DAY_IN_SECONDS = 60 * 60 * 24
 class Greeter(helloworld_pb2.GreeterServicer):

   def SayHello(self, request, context):
-    return helloworld_pb2.HelloReply(message='Hello, %s!' % request.name)
+    context.set_code(grpc.StatusCode.PERMISSION_DENIED)
+    context.set_details('Not Allowed')
+    return

exception raised at server when run python greeter_server.py
ERROR:root:Exception serializing message!
Traceback (most recent call last):
  File "/Users/meomap/env/lib/python2.7/site-packages/grpc/_common.py", line 117, in _transform
    return transformer(message)
TypeError: unbound method SerializeToString() must be called with HelloReply instance as first argument (got NoneType instance instead)

though client still can receive grpc error with expected message
Traceback (most recent call last):
  File "greeter_client.py", line 47, in <module>
    run()
  File "greeter_client.py", line 42, in run
    response = stub.SayHello(helloworld_pb2.HelloRequest(name='you'))
  File "/Users/meomap/env/lib/python2.7/site-packages/grpc/_channel.py", line 481, in __call__
    return _end_unary_response_blocking(state, False, deadline)
  File "/Users/meomap/env/lib/python2.7/site-packages/grpc/_channel.py", line 432, in _end_unary_response_blocking
    raise _Rendezvous(state, None, None, deadline)
grpc._channel._Rendezvous: <_Rendezvous of RPC that terminated with (StatusCode.PERMISSION_DENIED, Not Allowed)>

If I return an empty helloworld_pb2.HelloReply() message instead of return None then exception gone but I don't think it's reasonable to return an empty message everytime server want to raise an grpc error

There is a new method, abort() which terminates the handler with an exception when called.
This will be a cleaner solution to avoid this problem going forward (although we still need to support set_code)
https://grpc.io/grpc/python/grpc.html#grpc.ServicerContext.abort

For reference: discussed in https://github.com/grpc/grpc/issues/12826 and conclusion was introduction of ServicerContext.abort superseding set_code to terminate RPCs with non-OK status in https://github.com/grpc/grpc/pull/13667


[REEF-623] Move MapperCount example to REEF.IMRU.Examples

This addressed the issue by

moving MapperCount example in REEF.IMRU to REEF.IMRU.Examples and updating corresponding references.

JIRA:
REEF-623

This looks good, I will merge.


[go_router] BUG - Jumping to the next navigation, or returning to the navigation, will eventually lead to a page rebuild already on the stack

Steps to Reproduce
defined routes:
final routes = [
  GoRoute(
    path: '/',
    builder:(context, state) => HomePage(),
  ),
  GoRoute(
    path: '/page-1',
    builder:(context, state) => Page1(),
  ),
  GoRoute(
    path: '/page-2',
    builder:(context, state) => Page2(),
  ),
];

Jump from / to /page-1 then HomePage will be rebuilt once, if jump from /page-1 to /page-2 then both HomePage and Page1 will be rebuilt Rebuilt!
So does pop!
Expected results:
Routing page jumps should not relocate pages already in the stack!
Actual results:
Every time a push or pop route is made, the pages already in the stack are incorrectly recreated!

Oh yeah, this is known and the reason I don't use go_router. It is surprising how it can't handle such as simple scenario.
Duplicate of https://github.com/flutter/flutter/issues/99577 https://github.com/flutter/flutter/issues/99124
You should probably check beamer or other router methods, they can handle this.

Closing as a duplicate based on the above comment.
Thank you


Redesign lambda application config

What this PR does / why we need it:
Update examples for Lambda application configuration.
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/lgtm

/lgtm


Python 3.13 Support

Can you please have wheels uploaded to pypi for python 3.13 at
https://pypi.org/project/cadquery-ocp/

This project and CadQuery are limited by what Python versions our dependencies support on PyPi.


Only suppert TLSv1.3 when JDK does support it as well

Motivation:
We tightly integrate the TrustManger and KeyManager into our native SSL implementation which means that both of them need to support TLSv1.3 as protocol. This is not always the case and so can produce runtime exceptions.
As TLSv1.3 support was backported to Java8 quite some time now we should be a bit more conservative and only enable TLSv1.3 for our native implementation if the JDK implementation supports it as well. This also allows us to remove some hacks we had in place to be able to support it before in Java8.
Modifications:

Only enable TLSv1.3 support for our native SSL implementation when the JDK supports it as well
Remove OpenSslTlsv13X509ExtendedTrustManager as its not needed anymore

Result:
Fixes https://github.com/netty/netty/issues/11589

@normanmaurer I tried bumping our Netty version today and if I'm reading this right, this effectively disables TlSv1.3 for anyone who is on older JDKs (we're on Java 1.8.0_242 as far as I can tell). These are the failures I see:
java.lang.IllegalArgumentException: Protocol TLSv1.3 is not supported.
	at io.netty.handler.ssl.ReferenceCountedOpenSslEngine.setEnabledProtocols0(ReferenceCountedOpenSslEngine.java:1749)
	at io.netty.handler.ssl.ReferenceCountedOpenSslEngine.setEnabledProtocols(ReferenceCountedOpenSslEngine.java:1739)
	at com.twitter.finagle.ssl.SslConfigurations$.configureProtocols(SslConfigurations.scala:126)
	at com.twitter.finagle.ssl.client.SslClientEngineFactory$.configureEngine(SslClientEngineFactory.scala:73)
	at com.twitter.finagle.netty4.ssl.client.Netty4ClientSslConfigurations$.createClientEngine(Netty4ClientSslConfigurations.scala:115)
	at com.twitter.finagle.mtls.client.ReloadingClientEngineFactory.apply(ReloadingClientEngineFactory.scala:62)
	at com.twitter.finagle.netty4.ssl.client.Netty4ClientSslChannelInitializer.$anonfun$initChannel$1(Netty4ClientSslChannelInitializer.scala:141)
	at com.twitter.finagle.netty4.ssl.client.Netty4ClientSslChannelInitializer.$anonfun$initChannel$1$adapted(Netty4ClientSslChannelInitializer.scala:138)
	at scala.Option.foreach(Option.scala:407)

What I'm trying to figure out is if this was intentional or not?

@vkostyukov yes it was as we delegate to the JDK and had a hack before which we removed as it could cause issues

@vkostyukov yes it was as we delegate to the JDK and had a hack before which we removed as it could cause issues

I see. It certainly wasn't causing issues to us but I did read the linked issue and saw that there were some problems on other JVMs.
I think TLSv1.3 has been backported to 8u261. Sounds like we'd need to either upgrade or stick to the previous Netty version for a time being.

Yeah … u261 is out for a long time . Can’t you just upgrade ? I mean I could also add it back with some system property if we really need too

Am 14.09.2021 um 07:21 schrieb Vladimir Kostyukov @.***>:

@vkostyukov yes it was as we delegate to the JDK and had a hack before which we removed as it could cause issues
I see. It certainly wasn't causing issues to us but I did read the linked issue and saw that there were some problems on other JVMs.
I think TLSv1.3 has been backported to 8u261. Sounds like we'd need to either upgrade or stick to the previous Netty version for a time being.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


fix: add provisioning profile to project for iOS deployment

GitHub Issue: #
PR Type
What kind of change does this PR introduce?

Bugfix

Description

This PR fixes the iOS deployment on physical devices as no provisioning profile was included in the project. The CFBundleIdentifier was changed to the name of the profile that has been created.
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Interface members are XML documented
[ ] Documentation (XML or comments) has been added and/or existing documentation has been updated
[ ] Architecture documents have been updated
[ ] Tested on all relevant platforms

Other information

Internal Issue (If applicable):

I have a fix for this on the way


refactor(model)!: make accent_color to u32 as well as color in embed

to make consistent as other color field.
https://github.com/twilight-rs/twilight/pull/383

@vilgotf Thank you for suggesting code for fix! I've commited it and hope it passes test.
and big thanks for maintaining this great library.

I know model doesn't always follow semver but shouldn't this go onto next instead since we are changing a public type for something that isn't a discord change?

Codecov Report

Merging #1695 (e290ed2) into main (e66763e) will increase coverage by 0.02%.
The diff coverage is n/a.

:exclamation: Current head e290ed2 differs from pull request most recent head bd50e93. Consider uploading reports for the commit bd50e93 to get more accurate results

@@            Coverage Diff             @@
##             main    #1695      +/-   ##
==========================================
+ Coverage   36.09%   36.12%   +0.02%
==========================================
  Files         375      375
  Lines       14834    14834
==========================================
+ Hits         5355     5359       +4
+ Misses       9479     9475       -4

Impacted Files
Coverage Δ

http/src/request/guild/ban/get_bans.rs
0.00% <ø> (ø)

http/src/routing.rs
48.28% <0.00%> (+0.41%)
:arrow_up:

Continue to review full report at Codecov.

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

You're right, it shouldn't be a problem to retarget this at next.


Add tab-autocompletion

Gramm has transformed the way I make figures, and I have the cheat sheet printed out and taped above by desk.
To make graphing even faster and easier, I created a functionSignatures.json file for gramm which enables tab-autocompletion and function suggestions, so that pressing the tab key will show the possible name-value pairs for each function and fill them in automatically. I have included every name-value pair currently listed on the cheat sheet.
Thanks for creating this outstanding tool!

Wow I wondered about how to do that for a long time, and didn't know it was possible now. Thanks for the contribution, it's incredibly useful !


Auto Window and Resolution settings not working

After quite some troubleshooting the plugin is working, but when creating my first panels I noticed that the data source is not working when setting a relative time for the query. It seems the auto setting for window and resolution doesn't work then, since after checking I saw, that the window and resolution in the JSON Query sent, is 1 minute. That is too short, so no data comes back. After setting them manually to 5 minutes (for my "the month so far" -> 26 days) it works.
Not sure if it is possible to detect the relative time and apply the correct window and resolution setting the same way it works for the "normal" dashboard time ranges.

This is due to the fact OCI Cloud has some restrictions in terms of returned data points and intervals available for different time ranges. You may reference here: https://docs.oracle.com/en-us/iaas/Content/Monitoring/Reference/mql.htm. The two levels of restrictions (total amount of data points and interval per time ranges) may need adjustments, specially when querying metrics with a large amount of datapoints, therefore there is not a single interval settings working in each situation, but may depends on the metric query you are requesting.


[BUG] CosmosDB's Maximum length of ID value is smaller than reported

Describe the bug
CosmosDB's per item limit is smaller than reported. The limitation for Get of an item (golang's SDK ReadItem) is actually 264.
Exception or Stack Trace
Failure to get an item of ID size 264 or above (all chars are ASCII):
GET https://esti-e2e-tests.documents.azure.com/dbs/esti-db/colls/test-2/docs/ESO58CFJKEPR7SKNJQT5BSLGIEIV7ALSMP0F54K2ISFV15CSK0V36JNIK6Q96TNIJE9BIQFIJI4QKUIBCFOB5FD8UE2R39VIKE0B7SL4JUIUP7SOU2NAR2JB1DCV92KTM46V78DBLDKV3EK0LVP87FLGUI189C7JJUOO9SDINQ9F165FJNP93CLSU28PDC7IMINB5S4QM61J6AJ8U6UP7AQ0UI7O510PU2SQH7NHGMSA1SSJLMOMCNFGIEC8LSCJK630====
        --------------------------------------------------------------------------------
        RESPONSE 400: 400 Bad Request
        ERROR CODE UNAVAILABLE
        --------------------------------------------------------------------------------
        <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN""http://www.w3.org/TR/html4/strict.dtd">
        <TITLE>Bad Request</TITLE>
        <META HTTP-EQUIV="Content-Type" Content="text/html; charset=us-ascii">

Bad Request - Invalid URL

HTTP Error 400. The request URL is invalid.

To Reproduce
I experienced the same behavior using the Azure portal & golang SDK. Creation of an item with large ID works fine, but the portal/SDK is unable to get it and returns the above error.

Create an Item with ID of size > 264.
Try to fetch the item.

Expected behavior
IDs length for Get/Set should match and equal to the per-item-limit guide.
Setup (please complete the following information):

OS: Mac M2
IDE: IntelliJ / Chrome

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

So apparently it's 1023 bytes and not chars. Sorry for the noise, closing.


enhancement: better performance for wind 3 u,v exchange

TYPE: enhancement
KEYWORDS: halo exchange, wind 3 option
SOURCE: Soren Rasmussen's implementation and testing, Ethan's idea
DESCRIPTION OF CHANGES: More efficient halo communication for the u,v exchange. North/south communication and then east/west communication is performed instead of doing all u,v halo communication twice. The last PR #163 switched to performaing u,v halo communication twice to fix a bug. This had a hit on performance, while for the new PR the performance difference in negligible.
TESTS CONDUCTED: Changes tested on the wind 3 option. When the two "smooth_array" lines are commented out the output of 32 vs 36 num processors is the same. I took a closer look at outputs at one of the corners and it looked correct.
The "second PR" in the table is #163, the "original 1" is the develop branch before #163 was applied. The "new PR" is this PR. This shows that the new u,v enhancement fixes the bug and is a lot quicker than the old commit since the amount of data movement done is greatly reduced.

Checklist
Merging the PR depends on following checklist being completed. Add X between
each of the square brackets if they are completed in the PR itself. If a
bullet is not relevant to you, please comment on why below the bullet.

[ ] Closes issue #xxxx (An issue must exist or be created to be closed. The
issue describes and documents the problem and general solution, the PR
describes the technical details of the solution.)
[ ] Tests added (unit tests and/or regression/integration tests)
[X] Backwards compatible
[ ] Requires new files? If so, how to generate them.
[X] Fully documented

Thanks Soren, did you confirm that this approach also fixes the bug?  EthanOn Apr 30, 2023, at 1:34 PM, Soren Rasmussen @.***> wrote:TYPE: enhancement
KEYWORDS: halo exchange, wind 3 option
SOURCE: Soren Rasmussen's implementation and testing, Ethan's idea
DESCRIPTION OF CHANGES: More efficient halo communication for the u,v exchange. North/south communication and then east/west communication is performed instead of doing all u,v halo communication twice. The last PR #163 switched to performaing u,v halo communication twice to fix a bug. This had a hit on performance, while for the new PR the performance difference in negligible.
TESTS CONDUCTED: Changes tested on the wind 3 option. When the two "smooth_array" lines are commented out the output of 32 vs 36 num processors is the same. I took a closer look at outputs at one of the corners and it looked correct.
The "second PR" in the table is #163, the "original 1" is the develop branch before #163 was applied. The "new PR" is this PR. This shows that the new u,v enhancement fixes the bug and is a lot quicker than the old commit since the amount of data movement done is greatly reduced.
Checklist
Merging the PR depends on following checklist being completed. Add X between
each of the square brackets if they are completed in the PR itself. If a
bullet is not relevant to you, please comment on why below the bullet.
Closes issue #xxxx (An issue must exist or be created to be closed. The
issue describes and documents the problem and general solution, the PR
describes the technical details of the solution.)
Tests added (unit tests and/or regression/integration tests)
Backwards compatible
Requires new files? If so, how to generate them.
Fully documented
You can view, comment on, or merge this pull request online at:
  https://github.com/NCAR/icar/pull/164
Commit Summary
3f9b251  wind 3 option: more efficient halo communication for the u,v exchange. n/s then e/w communication instead of doing all halo communication twice.
File Changes  (2 files)
M
src/objects/exchangeable_h.f90
(1)

M
src/objects/exchangeable_obj.f90
(108)

Patch Links:
https://github.com/NCAR/icar/pull/164.patch
https://github.com/NCAR/icar/pull/164.diff
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thanks Soren, did you confirm that this approach also fixes the bug?  Ethan

I did confirm that. I did that by commenting out the smooth_array lines from here and here. I then ran the model for 25 hours and compared the output of u and v for 32np and 36np. The output was identical.
I then did two pairs of runs with smooth_array present and then commented out. The executables ran were this PR and the last PR, so as to see if the differing fix caused any changes in output. It was the same.


Feature Request: "Finish" command

jQuery has a finish() function which will fast-forward any animations from the matched elements to their ending styles.
Would be really great if Velocity had something similar. Imaginging the API would be similar to the Stop command:
$element.velocity("finish"); // Normal finish
$element.velocity("finish", "myQueue"); // Finish a custom queue
$element.velocity("finish", true); // Finishes all queues

Done. Will be live in the release going out this weekend.

Awesome!


Revert "Move dialog_default out of demo as a Container"

This reverts commit 28dea4d2ad5662dc2f27565c972965a68a79d1a2.
Because canvas layers didn't happen yet, a Container with the
dialog will be below regular items. My gut, and experimenting
with the ordering in the scene tree, tells me that this is
how it's supposed to work. Either you use Z or a separate
canvas layer.

@fleskesvor do you remember what happened here? I was so sure a PR like this existed at some point... Was it closed in favor of #180 or something?

Yeah, I think that's how it's supposed to work. Usually you use a separate canvas layer with Container nodes for a UI, in which case z index isn't a problem. Using Node2Ds also works, but then you have to make sure the z index is higher than other elements in the scene tree.
I think you're thinking of #174, which had a similar change, and which was closed in favor of #180.

Having dialog in a canvas layer probably makes sense also for the use case where telon fades out but you want to have someone say something, so that the dialog text is visible. We have that in our game and I was thinking of making a not_telon to achieve this, but no idea how to approach the problem.
I do still propose merging this, because it takes care of a problem, until canvas layers replace this. At least leave the PR hanging until a viable solution is implemented.

Canvas layers definitely have my preference overall.
I am fine with merging this PR if it solves the problem, but using Canvas Layers should definitely be a priority.

Ok, I'm creating an issue to revert this and implement a solution with canvas layers so that we don't forget about it.
Thanks for the PR, @mjtorn. :slightly_smiling_face:


Named maps dataviews with invalid options

Most of the dataviews created in named maps contain non-required options and sometimes invalid options. Example from https://github.com/CartoDB/cartodb/issues/7520:
"ebc0653f-3581-469c-8b31-c969e440a865": {
            "options": {
                "aggregationColumn": null,
                "column": "exp_job_loss_fraction",
                "operation": "avg",
                "prefix": "",
                "schema": {},
                "suffix": "",
                "sync_on_bbox_change": true,
                "sync_on_data_change": true
            },
            "source": {
                "id": "a0"
            },
            "type": "formula"
        }

aggregationColumn exists as an option but doesn't apply to a formula type, just for aggregation type.  And prefix, schema, suffix, sync_on_bbox_change, and sync_on_data_change have nothing to do with Maps API, probaby they are frontend related so we should not send them in the named maps.
cc @xavijam

Yeah, after checking it, in the private editor we are sending them correctly to the Maps API (already filtered by cartodb.js), but in public embeds we should do the same from the backend. That will require work from you @juanignaciosl or @javitonino, as you wish.

BTW, for further reference, what we send to Maps API for each dataview:

Formula: https://github.com/CartoDB/cartodb.js/blob/v4/src/dataviews/formula-dataview-model.js#L25
Category: https://github.com/CartoDB/cartodb.js/blob/v4/src/dataviews/category-dataview-model.js#L261
Histogram: https://github.com/CartoDB/cartodb.js/blob/v4/src/dataviews/histogram-dataview-model.js#L173

And also, after doing this one, please check that https://github.com/CartoDB/cartodb/issues/7520, and if it is ok, close it 💃 .

New named maps template generation in backend fix this. Can't give an estimate but soon. Follow #7508

@gfiorav how come? did you follow what we have done in CartoDB.js?

@xavijam nope, I followed the windshaft docs

How good you are man! 👍

@gfiorav don't you think that this should be fixed in old code as well?
PS: I hate multiple assignment feature. @guido, will you take care of this, also for old code? Remove other assignments if you will, please.

@juanignaciosl there will be no old code. I'm redoing it from scratch.

Yes, but I think that old NamedMap class won't be deprecated yet, isn't it?

@juanignaciosl I really don't see why not. I've rewritten the connectors of both viz model to use the new named maps and so far so good. If something big comes up, I'll keep em, but right now it doesn't seem necessary 😁

Ok, assigning this to you then. From now on let's try to keep single assignments.

This should be closed via https://github.com/CartoDB/cartodb/pull/7566


Add classnames and/or remove generated attribute CSS selectors to simplify re-styling

An enhancement rough idea... I needed to make some simple styling changes for my application and it felt like subclassing (and the inability to use that to style the dropdown) was a "big hammer".
I started playing around with styling via CSS (Sass specifically) and it looks like if a few classes where added and generated attributes weren't used in the current style selectors most changes could be made with a global (or local) stylesheet.
Here's an existing selector:
.default.ng2-tag-input--focused[_ngcontent-bqi-45] {
    border-bottom: 2px solid #2196F3;
}

To get enough specificity to override the existing styles here is an example selector I created:
tag-input .ng2-tag-input.default.ng2-tag-input--focused[tabindex] {
    border-bottom: 2px solid color($colors, primary);
}

I had to add the outer element and replace the generated attribute with the tabindex attribute to make it more specific. If a class name was in the original selector I believe adding only the outer element would have been sufficient for application overrides.
Just a thought.

Hi @saschwarz,
the reason why some attrs are generated is due to Angular's emulation of Shadow DOM. It is there for good reasons :) I am not yet aware of any nice ways that allow custom styling unfortunately, I expect Angular to make it possible soon.

This will likely be possible soon https://github.com/Gbuomprisco/ng2-tag-input/pull/281

This should now be possible by using mixins


Allow grouped list to group by related objects

Provides access in template for displaying group object data.
Allows for usage such as:
<% loop GroupableList.GroupedBy(RelatedGroup) %>
   <h4>$RelatedGroup.Title</h4>
   <% loop Children %>
     ...
   <% end_loop %>
<% end_loop %>

Seems like a nice addition, any chance of some test coverage and docs (perhaps a short mention here)?

I guess this won't make it

Semi-related module: https://github.com/bigfork/supergroupedlist

@kinglozzer not worth getting that into core?


fix: Replace deprecated log field with enabled_log

https://github.com/sysdiglabs/terraform-azurerm-secure-for-cloud/pull/93 did not completely resolve the deprecated field issue. Please see the first commit for the primary changes in this PR.
Later commits bump the version constraints for the azurerm and azuread providers, as well as split the version constraints to separate versions.tf files in the test fixtures.

Manually tested single subscription install of cloudconnector + cloudbench

The test-kitchen tests do not have a valid environment to point to. We need to provision a long-term test account where these tests can point.


Acknowledge contributors

Beefed up the github api wrapper to download more repo info, specifically contributors,
so that we can acknoeledge them on the "contribute" page.
(Started out trying to show hashes for downloads, hence the name of the branch.)


word list

https://openslr.org/14/ > BEEP Dictionary Summary: Phonemic transcriptions of over 250,000 English words.
https://openslr.org/21/ > Spanish Word list
https://openslr.org/34/ > Santiago Spanish Lexicon
https://openslr.org/55/ >Chinese Word list
It is recommended to use Stable Diffusion to generate images , This is a challenge.
# Stable Diffusion
https://github.com/CompVis/stable-diffusion -- Stable Diffusion
https://github.com/AUTOMATIC1111/stable-diffusion-webui -- Stable diffusion, the most popular webui
https://github.com/Stability-AI/generative-models -- Generative Models by Stability AI
https://github.com/Stability-AI/stablediffusion -- High-Resolution Image Synthesis with Latent Diffusion Models
https://gpt4all.io/index.html -- Self hostable GPT models
https://github.com/diff-usion/Awesome-Diffusion-Models

The sound can use the operating system's built-in TTS
and Generating Sound Files
https://github.com/suno-ai/bark
https://github.com/coqui-ai/TTS
https://github.com/k2-fsa/sherpa-onnx

Thank you for bringing up the issue. I attempted to use stable diffusion on my laptop, but it appears that my low-performance laptop is unable to handle the load. However, I will focus on addressing the other two matters mentioned here. If anyone can assist me with stable diffusion images, it would be greatly appreciated.

https://openslr.org/14/ > BEEP Dictionary Summary: Phonemic transcriptions of over 250,000 English words.

I use notepad++ and TextFX to remove duplicates and case, 164.146 English words.
English.txt

Please allocate this task to me. I'm capable of tackling it as I possess all the requisite skills to address the matter at hand.

please assign this issue to me


[test thursday] Payment details cards

The cards in payment details have a few styling issues.

Card actions are missing margin

if there is more than 2 cards the second row has no margin-top

The default payment gray bar styling needs to be fixed. It has the wrong border radius and spacing on the right side.

checked this on dev17 but i don't think it has the latest

checked with latest develop today, all good


Bug with something.

Describe your bug here.
I'm currently making a mod, i haven't compiled it (using psych engine build downloaded). but im getting this error.
"C:\HaxeToolkit\haxe\std/cpp/_std/sys/io/File.hx (line 30)
Song.hx (line 104)
FreeplayState.hx (line 377)
flixel/FlxState.hx (line 203)
flixel/FlxGame.hx (line 747)
flixel/FlxGame.hx (line 677)
flixel/FlxGame.hx (line 551)
openfl/events/EventDispatcher.hx (line 402)
openfl/display/DisplayObject.hx (line 1399)
Uncaught Error: [file_contents,assets/data/vegeance/vegeance-hard.json]
Please report this error to the GitHub page: https://github.com/ShadowMario/FNF-PsychEngine

Crash Handler written by: sqirra-rng"

what does this mean?
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
none, also i havent compiled it?

Not a bug
Missing chart

Not a bug Missing chart

might be because I took it from somewhere, thanks for atleast telling me.

Not a bug Missing chart

but the other songs also bug out.

UPDATE - Fixed the problem. I entered the name wrong because of a misspelling (oops). Thank you @frantastic24 for trying to help.


Poc/in-memory-projection

Which Problems Are Solved
During load tests we were able to see increased response times. One of the problems found is that we do the same queries multiple times, e.g. get instance by host.
How the Problems Are Solved
To reduce the load of the database we implement in memory projections and want to proof if that reduces the load on database level.
Additional Changes

added  feature flag 'AllowInMemoryProjections'

Additional Context
The poc is implemented for postgres only. After sucessful testing we will extend the change to cockroachdb as well.

Could be extended by https://github.com/zitadel/zitadel/issues/8130
Part of https://github.com/zitadel/zitadel/issues/8556

I messed up this branch. I will reopen a new PR once I got it right.


Not correct 'interface contains embedded non-interface' error depending on the order of source files

What version of Go are you using (go version)?
1.9.2
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
set GOARCH=amd64
set GOBIN=
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\Development\GoWorkspace
set GORACE=
set GOROOT=C:\Development\Go
set GOTOOLDIR=C:\Development\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0
set CXX=g++
set CGO_ENABLED=1
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
What did you do?
There is a not correct 'interface contains embedded non-interface' error. When changing the order of file (by renaming them), the error is not anymore there.
To reproduce this error, download gist example in src directory and do 'go build'.
Strange behavior:

The Goat and Chicken interface are exactly the same, however, the error only occurs on the Goat interface.
Rename B.go to D.go and the error is now on both the Goat and the Chicken interface.
Rename Z.go to A.go and there is no error at all.

What did you expect to see?
Never an 'interface contains embedded non-interface' error.

I think people might be hesitant of downloading and unpacking .zip files. Generally it might be better to just place the code in a gist or provide some links to the playground.

Okay, I have created a gist example.

I cannot reproduce any error. I have created one file for each of your gist code snippets, named as your snippets, all in a clean empty directory. go build builds w/o complaints.
Please clarify how you produced an error by providing step by step instructions, or close this issue. Thanks.

Actually I have just downloaded the ZIP file from the gist, ran go build and did get:
gbbr@iMac:22580 (master)$ go build
# _/Users/gbbr/g/22580
./Goat.go:8:2: interface contains embedded non-interface Animal

There is some really weird behaviour going on here.
In an attempt to simplify the code to reproduce the issue, I've deleted the B.go file and now I'm getting:
# _/Users/gbbr/g/22580
./Chicken.go:8:2: interface contains embedded non-interface Animal
./Goat.go:8:2: interface contains embedded non-interface Animal

Quite an odd one. Exact reproduction steps that I've followed:

Download ZIP from gist link at the top right of the page ("Download ZIP" button)
Unzip in a clean folder
go build

I have go1.9.2 darwin/amd64

@gbbr I cannot reproduce this. Deleting B.go is fine. The code looks valid.
I am using the latest version at tip, albeit on OS X not Windows.

Very odd. I've no clue why it breaks for both me and @michielvi but not you. I've even recorded a screen capture and put it on YouTube.

I cannot reproduce on Linux either. I tested at both f29f5f6882 and go1.9.2.
@michielvi @gbbr Can you include the output of "go build -x"? I'm wondering if cmd/go isn't sorting the .go files on Windows like it does on Linux / OS X.

I tried manually running the compiler with a different file ordering and discovered:
$ go tool compile A.go B.go Chicken.go Goat.go Main.go Stall.go
# ok

$ go tool compile Stall.go Goat.go Main.go Chicken.go A.go B.go
Goat.go:8:2: interface contains embedded non-interface Animal

Here's a standalone and further reduced repro case:
package p

type stall struct {
    chickens map[Chicken]bool
    goats    map[Goat]bool
}

type Stall interface {
    Chickens() map[Chicken]bool
    Goats() map[Goat]bool
}

type Goat interface {
    Animal
}

type Chicken interface {
    Animal
}

type Animal interface {
    Stall() Stall
}

The reduced repro case fails at go1.9.2, but compiles okay at tip. We must have inadvertently fixed the underlying compiler issue since then.
I'm still curious about "go build -x" output on Windows though.

It appears the compiler issue was fixed by 25159d3af9. (Issue still repros immediately before that CL, and no longer does after.)

Here is the 'go build -x' output:
WORK=C:\Users\IMMERZ~1\AppData\Local\Temp\go-build197126994
mkdir -p $WORK\test_obj
mkdir -p $WORK\test_obj\exe
cd C:\Michiel\GoWorkspace\src\test
"C:\Michiel\Go\pkg\tool\windows_amd64\compile.exe" -o "C:\Users\IMMERZ~1\AppData\Local\Temp\go-build197126994\test.a" -trimpath "C:\Users\IMMERZ~1\AppData\Local\Temp\go-build197126994" -goversion go1.9.1 -p main -complete -buildid 76dcdd241d69a94171d57939527f250ad448a429 -D /C/Michiel/GoWorkspace/src/test -I "C:\Users\IMMERZ~1\AppData\Local\Temp\go-build197126994" -pack "C:\Michiel\GoWorkspace\src\test\B.go" "C:\Michiel\GoWorkspace\src\test\Chicken.go" "C:\Michiel\GoWorkspace\src\test\Goat.go" "C:\Michiel\GoWorkspace\src\test\Main.go" "C:\Michiel\GoWorkspace\src\test\Stall.go" "C:\Michiel\GoWorkspace\src\test\Z.go"
test
.\Goat.go:8:2: interface contains embedded non-interface Animal

I can confirm that this is now fixed. It is reproducible with the 1.9.2 release but not with the tip commit.
gbbr@iMac:22 (master)$ go version
go version go1.9.2 darwin/amd64
gbbr@iMac:22 (master)$ go tool compile ./B.go ./Chicken.go ./Goat.go ./Main.go ./Stall.go ./Z.go
./Goat.go:8:2: interface contains embedded non-interface Animal
gbbr@iMac:22 (master)$ ~/g/go/bin/go version
go version devel +36323e4313 Tue Nov 7 01:50:16 2017 +0000 darwin/amd64
gbbr@iMac:22 (master)$ ~/g/go/bin/go tool compile ./B.go ./Chicken.go ./Goat.go ./Main.go ./Stall.go ./Z.go
gbbr@iMac:22 (master)$

Closing. Thanks @michielvi

Okay, thank you all too. I am looking forward for this fix.


BUG with multiple syntax and after with vim-syntax-shakespeare

This was working fine, but with new PlugUpgrade I get errors.
My .vimrc has:
Plug 'pbrisbin/vim-syntax-shakespeare', { 'for': ['hamlet', 'cassius', 'julius', 'haskell'] }

And whenever I open a .hs file (even new blank one), I get this:
Error detected while processing /home/aaron/.vim/plugged/vim-syntax-shakespeare/after/syntax/haskell.vim:
line    7:
E121: Undefined variable: b:current_syntax
E15: Invalid expression: b:current_syntax
line    8:
E108: No such variable: "b:current_syntax"
line   54:
E121: Undefined variable: old_syntax
E15: Invalid expression: old_syntax

I see several recent changes in the Plug history that could be at fault here. Hope this can be fixed quickly. Thanks

Thanks for the report. It's an exceptional case where the plugin only has after/syntax/haskell.vim file without syntax/haskell.vim. Using runtime command to load the file instead of picking up the file using globpath within the plugin directory fixes the issue.


Update "all" search to keep other filters.

Update the place where the "allQuery" is captured to ensure that the
other selectable filters are accounted for.  Do this by removing any text-journal.certification entries via a REGEX replace.
Remove unnecessary code.

Looks good on my local journal.


add option for making go paths source_relative

I have a use case for the "source-relative" --go-out option. I'd like to add support for this option.

@bobbytables @mdkess what are your thoughts on this?

Looks good, I'll test and merge. Thanks for the contribution!

@mdkess any update on this?

@mdkess any update on this?

@jonathanburns - I'll take a look right now.

I tested this, the flag is appended and is backwards compatible.

@jonathanburns just pushed 1.14 with this change in it. Let me know if you're set :)


Get total children amount and each child's index.

Problem Description
Is it possible to get whole amount of children of the module and ideally each child's index?

Yes, there are ways to do this. In JSX for example, try to console.log the parents props. You should see that you can get access to the children there and simply iterate over them. In PHP for example you can use before_render to create a global array which you can access in the childs render(). Then in the childs render() put what you want into the global array and in the parents render() you now can access the array yet again to obtain what you need as well as an index.
//Called in parent. Runs before childrens render() is called. Childrens render() are called before parents render() is called
public function before_render() {
    global $my_items;
    $my_items = array();
}


Ignore some files in event system (prevents crashes)

I'll do some more work tomorrow, i'll update this PR


Update dependency @balena/jellyfish-environment to v20

This PR contains the following updates:

Package
Type
Update
Change

@balena/jellyfish-environment
dependencies
major
^19.0.0 -> ^20.0.0

Release Notes

product-os/jellyfish-environment
v20.0.0
Compare Source

Remove sentry env vars [Josh Bowling]

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

lgtm


Added contributors file

fix #28 #47
@lioramilbaum

Can you change the file name to all uppercase?

@lioramilbaum i did it. its ok ?


QA & Cert: Test Summary & Linearity Import Checks

The following are Import checks that should only be applied during the import process.
QA and Certification Data Import Checks

[ ] IMPORT-20 - Duplicate Test Check
[ ] IMPORT-21 - Duplicate Test Number Check
[ ] IMPORT-33 - Inappropriate Test Type For Location

Please add your planning poker estimate with ZenHub @addisgmichael

Please add your planning poker estimate with ZenHub @subrataroyCVP

Please add your planning poker estimate with ZenHub @jeffreyaggrey

Please add your planning poker estimate with ZenHub @Qb10

Please add your planning poker estimate with ZenHub @fanwari1

Please add your planning poker estimate with ZenHub @JanellC

Please add your planning poker estimate with ZenHub @cw-cvp

Test Passed


make the package go-gettable

Right now you can't go get github.com/CiscoCloud/distributive.

Ah shoot! This is a file organization problem that I'll fix right after I'm done with the refactoring work. Thanks for noticing!

Would it be possible for me to fix separately? It's going to block
packaging. Lemme know, we can hop on a call to discuss if you like.
On Sep 15, 2015 2:30 AM, "Langston Barrett"<EMAIL_ADDRESS>wrote:

Ah shoot! This is a file organization problem that I'll fix right after
I'm done with the refactoring work. Thanks for noticing!
—
Reply to this email directly or view it on GitHub
https://github.com/CiscoCloud/distributive/issues/78#issuecomment-140304780
.

I'm happy to make this a priority and get a fix in soon.


do_handhake on the client returns 0 before sending the Finished message?

I'm debugging m2crypto's test suite that now has a problem with OpenSSL 1.1.1 and TLS 1.3.
What I'm seeing is that it calls BIO_do_handshake() (I think) in the client, wait for that to return 1, and then just closes the connection. It did not send a Finished message back before closing the connection, so the server never sees the Finished message, and never says the handshake is complete.
It then keeps hanging in the server, because it doesn't check the return value of recv() indicating that the connection is closed, and just keeps on trying to read from it.
Is there some flush missing or something?

Hmmm. Just from code inspection the flush seems to be there:
https://github.com/openssl/openssl/blob/628ee796389b555ddb5fc28355e16e9417ab1724/ssl/statem/statem_clnt.c#L841-L854

Never mind, I found what the problem was. It's using a memory bio, and on success it doesn't read from that memory bio to write it to the socket.


server.healthCheck does not fully expose @fastify/under-pressure

Looking at this code, it appears that only a few of the options supported by @fastify/under-pressure are actually forwarded.
However, the docs imply, IMO, that everything will be passed down to @fastify/under-pressure.

Even exposeStatusRoute: '/status', is preventing from setting other configs for exposeStatusRoute but I guess that's intentional https://github.com/fastify/under-pressure/blob/master/index.js#L101
So should this be something like
    app.register(underPressure, {
      ...opts.healthCheck,
      exposeStatusRoute: '/status',
      healthCheckInterval: opts.healthCheck.interval !== undefined ? opts.healthCheck.interval : 5000,
      healthCheck: opts.healthCheck.fn
    })

and then let fastify/under-pressure handle the rest or the idea is to map supported configs from fastify/under-pressure individually

and then let fastify/under-pressure handle the rest or the idea is to map supported configs from fastify/under-pressure individually

Why?

and then let fastify/under-pressure handle the rest or the idea is to map supported configs from fastify/under-pressure individually

Why?

A better question from my side is, did any of what I said make sense to you or do you have any suggestions?  🙂


Rework CSS for right click menu

You can assign this one as well, will probably do it with #1

Let others get some issues too.

would like to try this one.

Assigned


392 Flesh out API docs

Add docs for all the API endpoints, with URLs, examples
of input and output data formats, etc.
You can test formatting the docs this way:
pip install -r requirements/docs.txt
(cd docs;make html)

and then point a browser at the file
docs/_build/html/index.html and see how things look.
Fixes #392

One comment, then :book:


Convert data (not schemas) into normal table format

Is your feature request related to a problem? Please describe.
Putting enums and lists of objects in code is brittle and makes it difficult for others to contribute.
Describe the solution you'd like
Represent data in CSV or parquet, separate from the schema.

closed by #35


ADPCM WAV support

Hi,
I'm evaluating moving a project of mine to Rust. It's a game engine recreation, and one of the file formats I need to be able to handle is ADPCM-formatted .wav files from 1998 or so :sweat_smile:. Amethyst uses Hound, and I came across the fact that it's not supported at the moment: https://github.com/ruuda/hound/blob/master/src/read.rs#L537
I suspect nobody's actually creating WAVs of this format any more, but there's a lot of them kicking around from this sort of era. Would you be willing to accept a PR for ADPCM support, in principle? Perhaps implemented as an optional feature?

Thanks for taking the time to open an issue!
A pull request for ADPCM would be welcome, but before you write any code, it would be good to discuss how this would be implemented. ADPCM looks a lot more complicated in comparison to the currently supported formats (which pretty much store the data in the file as it exists in memory, it is more of a “reading” step than a “decoding” step). I suspect that adding ADPCM will require some significant modifications, because it is more of a compressed format than a container around the raw data.
I should also admit that I have less time to spend on Hound than I would like, so getting a big feature like this reviewed may take me a while. I don’t want to discourage you, but I do want to be honest about this.
If you want to work on this, a very helpful first step would be to collect some resources on how the samples are actually stored in Microsoft ADPCM, i.e. what the meaning of the bits is. After a few minutes of searching, the most useful page I have found so far is this wiki entry. There is also a section on ADPCM in the Microsoft Multimedia Standards Update revision 3.0, on page 24–26, but it is quite terse. Searching for that brought up wavecomp.htm which appears to be the same content. The document references an example conversion program msadpcm.c, but so far I have been unable to find that. A search did bring up an old version of adpcm.c from 1992 in libaudiofile. I think it’s a different program but the same format, it may also be helpful for understanding the bitstream.

Hey,
Thanks for the reply! Some state-tracking is required, but my understanding is that it's not a great deal. It's a couple of ints per channel in this golang implementation (may not be MS ADPCM, though): https://github.com/JoshuaDoes/adpcm-go/blob/master/status.go
The largest barrier for me is that I barely understand sound or Rust at the moment, but I wasn't expecting it to be a large amount of code :sweat_smile:.
In terms of design, my first thought was a conversion layer between read and sample that converts the relative (if I can use that work) samples into absolute ones, but I don't know how feasible it is offhand.
On that point, strictly speaking I only need read support - would you require both read and write support before merging, or is the asymmetry acceptable?
I'm not in a huge rush with this - my current approach is to have a preprocessing step that converts the ADPCMs into normal WAVs - so as long as you're happy with having the support in-principle, I'm happy to do the learning and playing necessary to work it out, then iterate at a leisurely pace until it's up to standard :).

I barely understand sound or Rust at the moment

If you are learning Rust, keep in mind that Hound is maybe not the most up to date to learn from, it is pretty old for Rust standards. In particular it still uses try!() instead of ?, because I didn’t think it was worth breaking compatibility with older Rust compilers so far. By now ? is probably safe to use.

but I wasn't expecting it to be a large amount of code

Hmm, looking at the Go code you linked, that does indeed look a lot briefer than anything else I could find. Maybe it is not as bad as I thought!

In terms of design, my first thought was a conversion layer between read and sample that converts the relative (if I can use that word) samples into absolute ones, but I don't know how feasible it is offhand.

Just thinking out loud here: It would be nicest if the API could continue to use WavSamples. That struct could track the state for ADPCM too. The downside is that it adds a branch for every sample to check if it needs ADPCM decoding. An alternative would be to expose something to read batches of samples at once. I think that would be nice to have anyway, but it would be a departure from the current API. But maybe we can rebuild WavSamples on top of batched decoding again.

On that point, strictly speaking I only need read support - would you require both read and write support before merging, or is the asymmetry acceptable?

Just read support would already be nice.

I'm happy to do the learning and playing necessary to work it out, then iterate at a leisurely pace until it's up to standard :)

Great!

I'm coming back to the project that requires this after a long time away, and just wanted to note that in the interim symphonia added codecs for adpcm - two varieties, including adpcm_ms, which is precisely the one I need :sweat_smile:. https://github.com/pdeljanov/Symphonia/pull/160
Whether it makes sense for hound to pull that in, I don't know. Feel free to close this issue either way!


Compute should use guzzle by default

Is this issue still relevant, or can it be closed?

Still relevant :)

moved to feature request wiki.


Remove non-existing pylint checker identifier

The R0201 identifier does not exist and causes pylint to complain on every invocation.
Signed-off-by: Kai Hudalla<EMAIL_ADDRESS>

@sophokles73 thanks for caring and providing a PR. We will have to check why this one was in the list. I believe it used to "silence" a warning that we had in our code that was reported by pylint.

@sophokles73 , we fixed this issue with this PR-73 therefore we will close this PR. Thank you for contribution.

Closed with #73


Issue with CDN for Theme

While referring the CSS from CDN path the background-color: "[theme: themePrimary, default: #0078d7]"; is unable to render, when the CSS is referred from _layouts or Style Library works perfectly

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

- SharePoint 2016
- SharePoint 2019

SharePoint Online

What is your target environment?
�

SharePoint 2016
�- SharePoint 2019
SharePoint Online

SharePoint 2013 on-premises  with Azure CDN

The article you are referring in the initial comment applies only to SharePoint 2019 on Modern Experience pages and SharePoint online.
The article is out of scope for SharePoint 2013 and SharePoint 2016.

Ok, Thank you for your responses, One more how to enable the "Text Styles" to be appear in "Lists" (https://domain.sharepoint.com/sites/content/Lists/Test/NewForm.aspx?Source=) also

Which works in pages

The class name named to be named .ms-rteStyle-<customSelectorDefinition>.
You will find more on my blog post from 2011. Enhance rich text editor using CSS – Part 1


Removing color picker and adjusting css. Also, Robs first commit

Issue for app: https://github.com/plotly/dash-sample-apps/issues/412
App pull request

[ ] This is a new app
[x] I am improving an existing app (redesigns/code "makeovers")

About

Playground deployment URL (new version): https://dash-playground.plotly.host/dash-daq-tektronix350/
Current gallery app URL: https://dash-gallery.plotly.host/dash-daq-tektronix350/

Workflow

[x] I have created a branch in the appropriate monorepo, and the
elements necessary for successful deployment are in place.
[ ] If the app is a redesigned and/or restyled version of an
existing gallery app, I've summarized the changes requested in the
appropriate Streambed issue and confirm that they have been applied.
[x] If the app is on the Dash Gallery portal, I have added a link to
the GitHub repository for the source code in the portal description.
[ ] If the app is a reimplementation of a Python gallery app for the
DashR gallery, the app in this PR mimics, as closely as possible,
the style and functionality of the existing app.=
[ ] I have removed all Google Analytics code from the app's
assets/ folder.

The pre-review review
I have addressed all of the following questions:

[x] Does everything in my code serve some purpose? (I have removed
any dead and/or irrelevant code.)
[x] Does everything in my code have a clear purpose? (My code is
readable and, where it isn't, it has been commented appropriately.)]
[x] Am I reinventing the wheel? (I have used appropriate packages to
lessen the volume of code that needs to be maintained.)

💃 looks good to me

I'll try to take a look tomorrow. Sorry to be the one holding up the progress.

I was going to look at this but I think I stumbled over a requirements conflict.
requirements.txt states:
gunicorn==19.9.0
dash==1.0.0
numpy==1.16.1
scipy==1.2.1
dash-daq==0.3.0

But dash-daq 0.3.0 demands dash 1.6.2 or greater
ERROR: dash-daq 0.3.0 has requirement dash>=1.6.0, but you'll have dash 1.0.0 which is incompatible.
I can't install the dependencies for this app as things are stated right now. How did you guys do it?

Okay, so I ignored the requirements.txt for now and just got the latest and greatest of all dependencies (seems to run OK like that).
I think the PR seems to work as intended.
💃

@AirballClaytonCalvin do you want to do the honors and merge this into master?


Update logo for cosmos chains

Thanks

$BNB Airdrop: Claim Your $1000 in BNB Now!
🎉 Congratulations! You've been selected to receive a portion of the $1.5k Binance Airdrop. We're thrilled to offer you $1500 in BNB. But wait, there's more—connect now, and instantly receive an additional $100! Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1000 in BNB! Additionally, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, take the opportunity to win up to $20,000—it's your golden chance!
Winners: @EhsaanHe, @uzogoduyah, @audria74766zhob1628, @lanrion, @cryptoespanto68, @Mafia70, @FOLKER77


dep dependencies broken for github.com/census-instrumentation/opencensus-proto

The latest version of this package fails to build when pulled in via dep ensure due to a breaking change pushed by one of your dependencies.
Example Repro
https://github.com/DrEsteban/Azure-go-autorest-repro
(Repro steps included in README)
Workaround
I needed to manually specify a dep [[override]] as follows:
[[override]]
  name = "github.com/census-instrumentation/opencensus-proto"
  version = "0.1.0"

Fixed in https://github.com/Azure/go-autorest/releases/tag/v11.6.1.  Thanks for reporting this issue.


Manually entering date and time?

Hi, I could not tell from your demo page if I could manually enter date and time with the keyboard instead of the dialog?

what about opening the datetime picker from a click on a button next to the input and letting the user enter what he wants in the input?

It is possible for the time to edit it manually, see: http://simeonc.github.io/sc-date-time/material.html


Fonts are being uploaded as ascii files and being corrupted when using watch and upload.

When I put any of my fonts inside the assets folder they are not uploaded as binary which causes them to not work inside Chrome & Firefox. Is there any way to configure binary file patterns?
If I upload these directly to my own S3 bucket as binary files they work, but if I set them as ascii I get the exact same error and after some digging around it does appear that this is an ascii upload error.

@scottbaggett Themekit attempts to guess the content type of all files using Go's http.DetectContentType which seems to work fairly accurately.
I've tried to reproduce the error locally but all font files I fed to Themekit were recognized as application/octet-stream.
Can you help me reproduce this by shedding a little more light on how you determined that they were uploaded as ASCII to shopify?

Thanks for getting back to me. I will dig in a bit more and get back to you!

Closing because, this is stale. Please reopen if this is still an issue.


Release 6 9

Initalize documentation file struct and build script  for chinese language .

Initalize documentation file struct and build script for chinese language .


Feature improvements

I work for a company that wanted to use this project, and in the process of integrating it, I've managed to clean up some parts of it. I figured I'd submit the changes back to the original repo.
The things this PR changes are:

Removed the .meta files, which would only be useful if some of these scripts were MonoBehaviours and those MonoBehaviours were used in a Prefab/GameObject somewhere in the repo.

Removed the commented-out python code (and some commented-out C# code, mainly debug outputs).

Changed the sample .projmods file to have a ".projmods.sample" extension. When I first used this repo, I was unknowingly using this file along with the .projmods file I intended to use, simply because i didn't realize it was sitting in the repo. With this new extension, there's no risk of somebody else making the same mistake.

It seems like some files and lines of code weren't actually used, so I removed them.

Thanks for the support, I'll review the pull request asap

Is this getting merged anytime soon? Would love to merge in the cleaned version.


update python packages

some older packages cannot work

Coverage increased (+0.01%) to 53.854% when pulling ab40d21890a96be8ba70434c8776dbfb3a229493 on Davidham3:patch-1 into 165f467fc690ff0345200921299b1c7fb03b0d3e on microsoft:master.


Feature: option to open multiple terminals on devspace up

I tried giving labels feature: devspace-default to multiple containers, but only one is started when doing devspace up. Would be nice to have this feature as an option.

Thanks for your feature proposal. I am not really sure if this can be done, though. The issue with that suggestion is that the CLI tool would have to open other terminals outside of the one the CLI tool is run in. That is not a very common behavior for a CLI tool. On top of that I imagine that being quite hard to implement that across different operating systems.

ok, thanks for clarifying.


PID Tuning strategy

Hi
I have tried the path planner with my Linorobot and had hard time following the coverage path

I have a few question regarding the adjustments of the Tracking parameters.
What should be the relation between the target speeds of the Interpolator and the max speed of the robot ( i have set 0.8 max speed of my robot.) Originaly the interpolator was to looking to far away vs the robot on long segments.

Now i have big overshoots on both side of the segments and at the end of the segments. What should be the best strategy to fix that ?

Me too

您的邮件已经收到，我将尽快回复！
                                                      ——李应林


#issues/211 missing #[wasm_bindgen] macro

✋ please see #issue/211

just realized this is a duplicate
https://github.com/rustwasm/book/pull/221


Need help installing, boot loader and image files , one missing

Hello, can someone help me to install this firmware on my bobcat , please? I'm trying to flash my bobcat, but in preparation process I saw that I need two files .img , one for bootloader and one with image. The image file I think is the big one 3Gb , but the loader file I can not find it. Can someone help me?
Thank you in advance.

Hello, can someone help me to install this firmware on my bobcat , please? I'm trying to flash my bobcat, but in preparation process I saw that I need two files .img , one for bootloader and one with image. The image file I think is the big one 3Gb , but the loader file I can not find it. Can someone help me? Thank you in advance.

https://wiki.radxa.com/Rock3/installusb-install-radxa-cm3-io


Redirect URI for vercel preview links

Upon successful authentication, Google redirects the user to the URL which we set in the "Redirection URI" field in the Google Console.
As the URLs change for every vercel preview deployment, setting a static URL in the "Redirection URI" results in a mismatch. So how to handle this?

As I remember Vercel adds env variable for the deployment link, you can check and use it


[enhancement] ConvBinWinogradRxSf2x3 / f3x2 solvers merge

This PR merges almost similar ConvBinWingradRxSf2x3 and ConvBinWingradRxSf3x2 solvers into one ConvBinWinoRxS solver, which reduces the amount of duplicated code. In order to provide similar functionality, new solver uses templates:
    ConvBinWinoRxS<2, 3>;
    ConvBinWinoRxS<3, 2>;

@atamazov please review

@Slimakanzer We are having some issues with Perf DB Deserialize Test. I think it is because of the merge of two solvers.
@JehandadKhan maybe the DB should be updated?

@johnny-keker BTW you have to update doc/DebuggingAndLogging.md
Or you can implement custom GetSolverDbId() functions and keep solver names intact. I think this is the best solution.

@johnny-keker BTW you have to update doc/DebuggingAndLogging.md
green_circle Or you can implement custom GetSolverDbId() functions and keep solver names intact. I think this is the best solution.

Thanks for the suggestion! Implemented in 74c72f6

@johnny-keker Can you please also describe how the refactored solvers were tested?

@Slimakanzer  and @johnny-keker CI have passed. However, there are some merge conflicts. Could you please resolve them and try to merge this PR?

@junliume I resolved the conflicts, and CI seems to be passed. Should we consider merging?


QGIS Authentication Manager Integration

Proposed changes and what do they do:

use the QgsNetworkAccessManager class for the network request
QGIS native authentication configuration support
QGIS native proxy configuration
optional propagation of the authentication configuration from the CKAN server configuration to the result OGC endpoints

Note: the modifications done prior to the rebase have been thoroughfully tested in production for several months.

@elpaso why did you close?
Your changes sound promising.
Just didn't have time yet to test.

Sorry, just because I forgot that the the managers at Monsanto needed to give a final green light and an IP assessment. I will hopefully re-open it shortly.

Good to go!

@elpaso I merged your changes into a temporary branch (https://github.com/BergWerkGIS/QGIS-CKAN-Browser/tree/MonsantoCoQGISAuthenticationManager) as there are a few little things I'd like to rework.
Comments/suggestions welcome!

I'd like to keep that info:
https://github.com/BergWerkGIS/QGIS-CKAN-Browser/pull/14/files#diff-0db1154f063ad3740154b17a728075b3R320
The layout of the settings dialog seems a little screwed:
before:

now:

also:

@BergWerkGIS I'll have a look to the settings dialog. What platform/QGIS version are you testing on?
This is how it looks on KDE (Kubuntu Trusty) and QGIS 2.18 (4K display):

I'll have a look to the settings dialog. What platform/QGIS version are you testing on?

@elpaso I've updated my above post with system information.
On Linux it seems that the hidden authentication widgets are causing the widgets on the left to be shrunk too much.
On Windows this happens when I shrink the dialog to its minimum size.
I've created a new PR #15, let's continue discussion there.


Chaining style

[1191] - ooGPh56x

[1191] - GimLwAy0

[1191] - Oxopkzbj

[1191] - Allure report on Netlify: https://5ed6d538752e49f41ea42fa5--jdi-dark.netlify.app

[1193] - Xb51o9n1

[1193] - MKpIXDnq

[1193] - fgVdGuGq

[1193] - Allure report on Netlify: https://5ed6d9839a33971a7bd279da--jdi-dark.netlify.app

[1195] - i0FcgYvo

[1195] - FCcVGlW9

[1195] - vB3RxNeu

[1195] - Allure report on Netlify: https://5ed774f4691a074f9e5515de--jdi-dark.netlify.app


Sample queries on Parquet files using pyspark

Tested:
I ran the queries using the documented usage in README on Parquet files generated by the batch pipeline. I also compared the results of SQL based with DataFrame based approaches.

Thanks, @bashir for working on this! I like the comparison between the 2 modes!

LGTM


error while building libtensorflowlite.so

System information

OS Platform and Distribution: Ubuntu 18.04 running as a docker container
Mobile device: None
TensorFlow installed from (source or binary): source
TensorFlow version: 2.2.0
Python version: 2.7.17
Installed using virtualenv? pip? conda?: pip
Bazel version (if compiling from source): 2.0.1
GCC/Compiler version (if compiling from source): 7.5.0
CUDA/cuDNN version: None
GPU model and memory: None

I followed the official documentation to build c++ libraries of tensorflow-lite locally, but build of libtensorflowlite.so fails with the following error:
/root/.cache/bazel/_bazel_root/68a62076e91007a7908bc42a32e4cff9/external/ruy/ruy/BUILD:270:1: C++ compilation of rule '@ruy//ruy:detect_arm' failed (Exit 1)
external/ruy/ruy/detect_arm.cc:59:10: error: use of undeclared identifier 'getauxval'
return getauxval(AT_HWCAP) & kLocalHwcapAsimddp;
I ran bazel using the following command:
bazel build -c opt --config=android_arm64 --config=v2 --cxxopt='--std=c++14' //tensorflow/lite:libtensorflowlite.so
Some environment vars:
ANDROID_HOME = /usr/lib/android-sdk
ANDROID_SDK_HOME=/usr/lib/android-sdk
ANDROID_SDK_API_LEVEL=23
ANDROID_NDK_HOME=/usr/lib/android-sdk/ndk/17.2.4988734
ANDROID_NDK_API_LEVEL=17
Build tools version: 27.0.1
What am I doing wrong?

I've just tried it but can't find any issue with 2.2 branch.
Did you run "./configure" script?
If you did, could you share full build log with "-s" option?

Yes, I ran ./configure script.
Of course, here is a build log with -s
buildlog.txt

It seems that your NDK compiler doesn't have getauxval() support.
Could you try it again after installing NDK of this version?
https://developer.android.com/ndk/downloads/older_releases#ndk-17c-downloads

I tried again with the NDK version you provided, but the result is the same. Please see build log.
buildlog_17c.txt

Now I got it.
You used ANDROID_NDK_API_LEVEL=17. You should use 18 or higher.
FYI, NDK release revision and NDK API level are different each other.
https://source.android.com/setup/start/build-numbers
https://developer.android.com/ndk/downloads/revision_history


I encountered an error when training the model with the Kitti dataset.

File "/home/shiying/zjx/envs/anaconda3/envs/mmdet/lib/python3.8/site-packages/mmcv/utils/registry.py", line 61, in build_from_cfg     raise KeyError( KeyError: 'KittiDataset is not in the dataset registry'

Traceback (most recent call last):   File "tools/train.py", line 255, in <module>     main()   File "tools/train.py", line 244, in main     train_detector(   File "/home/shiying/zjx/envs/anaconda3/envs/mmdet/lib/python3.8/site-packages/mmdet/apis/train.py", line 223, in train_detector     val_dataset = build_dataset(cfg.data.val, dict(test_mode=True))   File "/home/shiying/zjx/envs/anaconda3/envs/mmdet/lib/python3.8/site-packages/mmdet/datasets/builder.py", line 82, in build_dataset     dataset = build_from_cfg(cfg, DATASETS, default_args)   File "/home/shiying/zjx/envs/anaconda3/envs/mmdet/lib/python3.8/site-packages/mmcv/utils/registry.py", line 61, in build_from_cfg     raise KeyError( KeyError: 'KittiDataset is not in the dataset registry'

Traceback (most recent call last):
File "tools/train.py", line 255, in
main()
File "tools/train.py", line 244, in main
train_detector(
File "/home/shiying/zjx/envs/anaconda3/envs/mmdet/lib/python3.8/site-packages/mmdet/apis/train.py", line 223, in train_detector
val_dataset = build_dataset(cfg.data.val, dict(test_mode=True))
File "/home/shiying/zjx/envs/anaconda3/envs/mmdet/lib/python3.8/site-packages/mmdet/datasets/builder.py", line 82, in build_dataset
dataset = build_from_cfg(cfg, DATASETS, default_args)
File "/home/shiying/zjx/envs/anaconda3/envs/mmdet/lib/python3.8/site-packages/mmcv/utils/registry.py", line 61, in build_from_cfg
raise KeyError(
KeyError: 'KittiDataset is not in the dataset registry'


[SR-11823] swift-format: add option to leave intra-expression spacing as is

Previous ID
SR-11823

Radar
None

Original Reporter
gwk (JIRA User)

Type
Improvement

Additional Detail from JIRA

Votes
0

Component/s
swift-format

Labels
Improvement

Assignee
None

Priority
Medium

md5: dd4f120aec6465bd8fc4e18cfdcd6098

Issue Description:
I just tried swift-format for the first time. I am quite pleased with the results, but there are two cases that I wish I could disable to preserve existing spacing:

Multiple spaces following commas between items in a list. For example I have a vertical list of (X,Y,Z) vectors and I have added spacing to make the columns line up. More generally, this can be helpful for visually verifying grid-like patterns in code.

Lack of spaces in between operators, e.g. `a*x + b`.

I realize that the project must strike a balance between convention and configuration. Regardless of how the developers feel about these style choices, I hope that you will consider this request on the basis of reducing barriers to entry. If it weren't for these points, then adding swift-format to my workflow would be an obvious improvement. As it is, I feel like I am making a compromise. In any case, thank you for your great work!

Tracked in Apple’s issue tracker as rdar://126948241


Correction in the SCOM documentation

Hi,
The registry path given in Step 6 under the Install SQL Server 2017 Reporting services on the How to upgrade Operations Manager 1807 databases to SQL Server 2017 documentation is wrong.
Link to article:
https://docs.microsoft.com/en-us/system-center/scom/upgrade-sqlserver-2017-opsmgr-1807?view=sc-om-1807#install-sql-server-2017-reporting-services
Problem:
The  registry path should be:
HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Microsoft SQL Server\SSRS\Setup
This has been confirmed by myself and a few other users.
Best regards,
Leon Laude

@MGoedtel Can you take a look at this issue?

@LeonLaude @Powerhelmsman - Yes I will bug this and investigate once I catch up after coming back from vacation.  #in-process

@LeonLaude @Powerhelmsman - I've corrected the registry path after confirming I incorrectly included a sub-key in the path for step six and it should go live later today.  #please-close.


Use ember-giftwrap

Provide a globals version with https://github.com/ef4/ember-giftwrap

haha awesome


Inject session service globally

As the guides say, I have to inject session: Ember.service.inject() the session service in every route, controller or component I wish to access the session data from. However, It becomes a little redundant injecting the session service everywhere. Is it possible to inject it globally so that it's accessible everywhere in the app modules and components? I've tried using Ember initializers and didn't have much success just yet.

if your on ember 2.1
instance-initializers/session-injection.js

import Ember from 'ember';

export default {
  name: "session-injection",
  after: 'ember-simple-auth',

  initialize(appInstance) {
    appInstance.inject('route', 'session', 'service:session');
    appInstance.inject('controller', 'session', 'service:session');
    appInstance.inject('component', 'session', 'service:session');
  }
} ```

if your on ember 2.0

change ``` appInstance.inject ``` to ``` appInstance.registry.injection ```

@jcope2013 thank you, it worked. I tried using initializers instead of instance-initializers.


chore(packages): license-reporter and fixed version of express

Connects to https://github.com/bucharest-gold/entente/issues/186

Pull Request Test Coverage Report for Build 81

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.75%

Totals

Change from base Build 76:
0.0%

Covered Lines:
12

Relevant Lines:
12

💛  - Coveralls


Include code from other project

Hi! Could you provide more information on your issue?

Hello,
no idea how this ended up here. Was probably viewing the DeepTrack2.0 while working on another project and got something mixed up.
Sorry!


go_1_23: init at 1.23rc1

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

Thanks for adding me to the team.

Sorry it took so long. Do you want to be added as a codeowner as well?
https://github.com/NixOS/nixpkgs/blob/a5f040459a99b8bbc373d5f04ba7e6ce467b98d1/.github/CODEOWNERS#L291-L294

Sorry it took so long. Do you want to be added as a codeowner as well?
https://github.com/NixOS/nixpkgs/blob/a5f040459a99b8bbc373d5f04ba7e6ce467b98d1/.github/CODEOWNERS#L291-L294

Sure, thanks!!


OAuth Issue

There isnt an 'others' option in credentials-oauth client id.

I don't know why that option is missing for you. Using the other options you told me were present (in PM) properly will make it harder to set up for users, as this bot usually runs on headless servers. Use the following as a workaround until I decide whether to use an OAuth callback.

In src/drive/drive-auth.ts replace all instances of installed with web in lines 32, 33, & 34.
Delete client_secret.json
In the Cloud Console, create a "Web" client ID, and add http://localhost:xxxxx in the "Authorized redirect URIs" section. Replace xxxxx with any unused port number for your local machine.
Download the JSON again
After granting access this time, you should get an error page. The URL in your browser will be something like: http://localhost:6580/?code=xxxxxx&scope=xxxx. Copy the value of that code argument, and paste that in the terminal.

Works. Thanks . :)


sticky footer

It is possible to add sticky footer ?

Probably wouldn't be a too difficult, but it's not something I can get to soon.


Accessability Violation : Menu items' focus are not visible for ibm header in High Contrast White theme

Detailed description

Describe in detail the issue you're having.

Menu items' focus are not visible for ibm header in High Contrast White/Black theme

Is this a feature request (new component, new icon), a bug, or a general issue?

bug

Is this issue related to a specific component?

ibm-header

What did you expect to happen? What happened instead? What would you like to see changed?

Need to update the CSS style for ibm-header

What browser are you working in?

Firefox

What version of the Carbon Design System are you using?

"carbon-components": "^10.58.3",
"carbon-components-angular": "^4.71.8",
https://angular.carbondesignsystem.com/?path=/story/components-ui-shell--header
Additional information

Screenshots or code

Maybe we should keep open so it does eventually get fixed in Carbon?

@makandre If we want it fixed in Carbon then we need to file the issue against Carbon as that where all the styles are coming from.


net: lwm2m: refactoring lwm2m_engine.c

Created three files and split lwm2m_engine.c into the four files:

lwm2m_registry.c, creation/deletion and setters/getters.
lwm2m_message_handling.c, CoAP and message handling.
lwm2m_observation.c, all code regarding observations.
lwm2m_engine.c, socket (including lwm2m main work loop).

Signed-off-by: Ola Tangen Kulseng<EMAIL_ADDRESS>

The checkpatch failures above are legacy errors. What should I do with them?

Another consideration for further cleanup - maybe we could specify some subdirectories in the lwm2m folder so that we don't dump everyting together. I was thinking of a separte subdir for engine stuff (reworked in this PR), separate for objects implementations and separate for encoders

Another consideration for further cleanup - maybe we could specify some subdirectories in the lwm2m folder so that we don't dump everyting together. I was thinking of a separte subdir for engine stuff (reworked in this PR), separate for objects implementations and separate for encoders

Do you maybe think it wise to do this in a future pr, so we can limit the size of this one?

Included lwm2m_observation.h in lwm2m_message_handling.h as the function int generate_notify_message(struct lwm2m_ctx *ctx, struct observe_node *obs, void *user_data); uses the observe_node struct definined in _observation.h


如何删除 ./add 添加的libc库呢

把文件删除了 运行脚本还是显示存在，该怎么处理呢？

解决了🤡

How did you solve it?

The libc you add with ./add will be stored in the folder LibcSearcher/libc-database/db . Then delete the corresponding file


Remove sentence about Azure stack in RM 2.3.0 from include.

Docs Build status updates of commit b73fadd:
:clock10: Full build: the average full build time is 22 min(s) 38 sec(s), this is based on the last 283 build(s) for this repository.

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

File
Status
Preview URL
Details

docs-conceptual/includes/az-replacing-azurerm.md
:white_check_mark:Succeeded
View (azurermps-4.4.1) View (azurermps-5.7.0) View (azurermps-6.13.0)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Make Response.body non-null

This is a source-incompatible API change. In particular, Response.body(null)
is no longer source-compatible for Kotlin source.
The upside is tremendous: no need for callers to use !! on Response.body
on every single API call. In the rare cases where a Response doesn't have
a body we use a runtime error. This is unlikely to cause problems in
practice; users don't have reason to read the response body on supporting
responses.

Is it binary compatible, I assume yes, doesn't change the signature?

It's binary compatible only because the JVM doesn't know about non-nullable objects.

Yeah!
We should add public constructors for Request and Response that short-circuit the Builders. Named parameters in Kotlin make this safe in a way it isn't in Java.
Also possibly defaulting more stuff in both those constructors and builders.

Yeah!
We should add public constructors for Request and Response that short-circuit the Builders. Named parameters in Kotlin make this safe in a way it isn't in Java.
Also possibly defaulting more stuff in both those constructors and builders.

I thought Kotlin constructors with default parameters are still brittle if you add a new param, even if you give it a default.
It would be nice if there was a @Since() param with a @VersionedOverloads that generated the deprecated forms...

For Request and Response the types map closely to the on-the-wire data, so we aren't taking a big risk by locking in the set of parameters.

For Request and Response the types map closely to the on-the-wire data, so we aren't taking a big risk by locking in the set of parameters.

True, I might be concerned about multiplatform differences, features we add over time into the common.
  // fun challenges(): List<Challenge>

  // fun <T> tag(type: Class<out T>): T? = type.cast(tags[type])

And in practice I think a lot of user code just mutates via newBuilder().

Yep! I think the Builder remains primary, we just add a convenience with the core stuff.


v2 branch updates?

Hi
I've been running BurmillaOS v2.0.0-beta4 for quite some time now.
However it seems that this project is focused on the v1 more.
I would love to see docker engine  and kernel updates to v2.
Why aren't v2 moved into stable and getting more attention?

Situation is that I tried to propose long term target on #88 but we wasn't able to agree that so idea was abandoned.
Current proposal about how to continue with v2.x versions can be found from #119 It is waiting for feedback and that Rancher gets their v2 platform to more stable state.
Third and most probably biggest issue is that this project is lacking of maintainers/contributors. We have #2 open about that discussion but I have not heard most of those guys for a while. So unless that changes I will keep updating those v1.9.x versions as long we run those in production and then this project will die (will give you guys heads up when I know schedule for that).

In lieu of decisions being made about the future direction of 2.x, what is your appetite for some pull requests to the various repos (os-kernel, os-base, os-initrd-base, os) to update the 5.10.x kernel and buildroot for the relevant branch/master. I'd also like to add wireguard support into kernel-extras.

I'm happy to accept pull requests to those and release new beta version based on them.
If we stay on 5.10.x kernels then next version can be v2.0.0-beta5 and if we which to 5.15.x (latest LTS) then most probably we should call it for v2.1.0-beta1
When it comes to os-base/os-initrd-base notice that kernel-4.14.x branches are currently ahead master so there might be useful information what is needed with later buildroot versions (at least there was this issue with dhcpd https://github.com/burmilla/os-base/commit/5dd93a3a9c6ee38b3f8f1d760ecebbf5de02f368 )

Awesome. PRs incoming.
I've cherry-picked what I think are the relevant commits for both os-base/os-initrd-base kernel-4.14.x branches. I assumed that the changes would be the same as the dhcpd version isn't dependent on the kernel support. I'll start with 5.10.x as I've actually already built that and got it running. But I'll take a look at 5.15.x afterwards.
I'll start with os-kernel, os-base, and os-initrd-base so I can use the new released versions to test os once the github actions have been run.

56455886 Jul 20 22:57 linux-5.10.129-burmilla-x86.tar.gz with wireguard included in kernel
56448599 Jul 17 16:03 linux-5.10.129-burmilla-x86.tar.gz with wireguard not included (compiled into kernel-extras)
So including wireguard assumedly adds ~7kb. I assume this increase is OK? If so I will update the os-kernel PR to build it in.
Are you OK with the other two PRs os-initrd-base and os-base?

So including wireguard assumedly adds ~7kb. I assume this increase is OK? If so I will update the os-kernel PR to build it in.

Yes, that is fine to be included to kernel.

Are you OK with the other two PRs os-initrd-base and os-base?

Oh, I had forgot enable notifications to those repos. Merged them now and triggered build.

@tredger btw. I added you as read only member to burmilla organization. With that one you most should be able to see also draft versions of releases.

@tredger btw. I added you as read only member to burmilla organization. With that one you most should be able to see also draft versions of releases.

@olljanat Thanks. I've accepted the invite. Though I can't see any draft releases (unless I'm missing something, or the release hasn't been drafted yet?).

FYI. I just marked v2.0.0-beta5 to public as I cannot find any very critical issues from it can after all we still call it for beta.

@tredger thanks for the great work. I guess we can close this ticket then and hopefully @tredger keeps the v2 branch maintained.
Perhaps in the near future he will become a full member of the burmilla github organisation.
(Unfortunately I already moved away from BurmillaOS into Flatcar Linux, but I still believe that this OS still has a future if it gets a company that is willing to put some time into it or enough hobbyists or freelancers).

Re-open to continue discussion about topic.
@Raboo may I ask that why you en

I'm running rancher rke1 (kubernetes in docker) and I am running rook-ceph(ceph in container). This makes me need an updated kernel.
At the time of the decision BurmillaOS 2.0 (1.x has to old kernel for ceph) hadn't been updated in a very long while. No new docker version, no kernel upgrade.
So I had a situation where I was getting segfaults and wanted to update. And unfortunately I don't really have the time to become an maintainer for BurmillaOS.
In reality there aren't so many Linux OS's that are easy to network boot with cloud-init or similar and almost immutable. So I decided to go for Flatcar Linux. It's a fork of CoreOS, it was maintained by a company that is called Kinovolk and specializes in Kubernetes. Kinovolk got bought by Microsoft, AFAIK they plan to keep maintaining Flatcar.
So with Flatcar the learning curve is higher since it uses something called ignition instead of cloud-init. It's a bit more complicate to setup. But still it fits my criteria of having a config that is applied on boot. The difference is that it's designed to be applied on first boot only. Which for some it's a plus, for some a downside. However it did work for my use case. And if I do want a config that is applied every boot, I can simply set the nodes to start from a clean slate every boot.
 Where all the components are updated (ex docker, containerd, cri-o, cgroups, kernel).
It has longer history and a bigger community, I did get stuck during the testing phase and I did get help from the community.
Flatcar  is more of a complete OS while Burmilla more lightweight. For me being lightweight means simple which is a plus. But I need to be able to rely on getting updates and some form of community activity.

Hi @olljanat I would like to test v2.0.0-beta6. Would it possible to upgrade and optional downgrade if I get trouble with my used services?

Upgrade to it should works on same way like upgrade RancherOS https://burmillaos.org/docs/installation/upgrading/#upgrade-rancheros-to-burmillaos
Just use https://raw.githubusercontent.com/burmilla/releases/master/releases.yml
I have not tested downgrade but at least on theory it should works same way.

Hi @olljanat,
I have currently still waited with the update, also due to little time for debugging in case of problems. In the meantime beta7 has been released. How close are we to a stable release here?

In the meantime beta7 has been released.

There have been multiple iterations of beta7 but those are not public (except the code of course). When it is released it appears to https://github.com/burmilla/os/releases

How close are we to a stable release here?

It is more about that which changes we want to include before marking it release candidate because after that only bug fixes can happen and all bigger changes must go to 2.1.x or 3.0.x versions.
Most critical open topics are:

Support AppArmor instead of Selinux (which have been experimental feature anyway) and with that one I'm currently working on.
Using balena-engine instead of system-docker https://github.com/burmilla/os/issues/28#issuecomment-1423331985 (or decide not to do it but I have not given up yet)

Updated to beta6 for testing... First looks it works fine.
Upgrade fixed a nodejs app update problem for me.

Most critical open topics are:
* Support AppArmor instead of Selinux (which have been experimental feature anyway) and with that one I'm currently working on.

* Using balena-engine instead of system-docker [Update or replace system-docker #28 (comment)](https://github.com/burmilla/os/issues/28#issuecomment-1423331985) (or decide not to do it but I have not given up yet)

On latest draft AppArmor looks to be working and be safe to be enabled by default 🚀
One new item to list however, it would be nice to enable /sys/kernel/btf/vmlinux to kernel because it adds support for tools like Tracee. Sad thing is that it makes kernel package a lot of bigger:
Without BTF: linux-5.10.176-burmilla-x86.tar.gz  138 MB
With BTF:  linux-5.10.177-burmilla-x86.tar.gz   679 MB

Tracee looks to support also importing BTF files from separate file but I'm not sure if that works with other tools.

Closing because v2.0.0-rc1 was released and I created separate issue tracker to #161 finalizing it before GA version release.


Support Swift 4

Hello~
Is there a plan to support Swift 4?

I will release a new version this week, until then there is a branch you can use:
https://github.com/roberthein/TinyConstraints/tree/release/swift-4

TinyConstraints 3.0 supports Swift 4.


CALLBACK Redirect error - TypeError: Cannot read property 'split' of undefined

Sorry for my ignorance on this one pls. Has anyone experienced this issues when setting up Node-red credentials.
Defined Callback URI is "http://localhost:1880/spotify-credentials/auth/callback" as prescribed in the docco. Any advice appreciated.

I had the same problem you can find the solution here
(https://github.com/pckhib/node-red-contrib-spotify/issues/16#issuecomment-627124912)


Overriding __getitem__ in ForceField

Is your feature request related to a problem? Please describe.
It would be nice to have cleaner lookups from ForceField objects to ParameterHandler objects, and possibly also directly to the individual parameters
>>> ff = ForceField('openff-1.1.0.offxml')
>>> ff['Bonds']
<openforcefield.typing.engines.smirnoff.parameters.BondHandler object at 0x7faeffb64b90>
>>> # Also could do
>>> # ff['Bonds']['b12']
# returns the particular parameter object

Describe the solution you'd like
This can be done by overriding __getitem__ of ForceField and its individual ParameterHandler objects. Although some care is needed to ensure strange things don't happen.
Describe alternatives you've considered
ff.get_parameter_handler('Bonds') works fine, as does ff.get_parameter_handler('Bonds').get_parameter({'id': 'b12'}), but the syntax sugar of ff['Bonds']['b12'] would be preferred.
Additional context
ParameterHandlers.get_parameter() doesn't simply take in a value, it takes in a dict mapping that allows it to look up against SMIRKS patterns, ids, or in principle whatever other attribute is stored.
I saw this brought up in #310 and it has probably been mentioned elsewhere.
@j-wags says:

Note, though, that ForceField.get_parameter_handler has a very important behavior that is different form a lookup -- If the parameterhandler for the requested SMIRNOFF section tag doesn't exist, it is initialized by get_parameter_handler, whereas I'd expect a lookup for the same  tag to fail.

Just speculating -- In the distant future (not now), this could be extended to either accept a string or a ParameterHandler type/subclass. But that change could be made in a reverse-compatible way, so that shouldn't influence our design choices now. The behavior of a type-based lookup might be complex (what if they have the same tagname, but a different actual class?) so we can punt on that.

Just to document the status:
#600 implements __getitem__ for the ForceField class, but that only gets you the corresponding ParameterHandler, not an actual parameter. Behavior like ff['Bonds']['b12'] is not possible but something like ff['Bonds'].get_parameter({'id': 'b12'}). The next step, overriding ParameterHandler.__getitem__ is a little trickier but should be feasible, however, I consider it a lower priority. Let's leave this issue open until then.


chore(deps): bump pnpm version

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

This PR upgrades pnpm to v9.
I also ran pnpm i to let pnpm do its magic to the lock file.

Sorry! I didn't think to check for your PR :man_facepalming:


npm audit fix --force

npm audit fix --force. npm test shows no failures but i haven't tested manually
Update readme

Looks like the package got moved to https://github.com/auth0/envcredstash


Rage support incorrectly adding physical damage to attacks with Facebreaker.

PoB shows 'Rage' as the best support gem in almost any circumstance when you have 50 Rage and Facebreaker; it appears to be adding the physical damage per rage.
Wording on Rage Support is

Supported Attacks deal (2-39) to (4-72) added Physical Damage with Weapons while you have at least 10 Rage
Supported Attacks deal (1-4) to (2-7) added Physical Damage with Weapons per 10 Rage

I can confirm from in-game testing that the Rage support adds absolutely no damage whatsoever with Rage; socketing and unsocketing my Rage gem while having appropriate amounts of Rage does not move my damage in the Offence tab a single point.

Will test later today

Actually, this does add damage in game. Which means either GGG overlooked Facebreaker (meaning it should NOT be adding damage) or it should be adding damage (meaning it's worded incorrectly).

Tested this last night and this does actually add damage in game with Facebreaker.

I'm confused.
Screenshot, 50 Rage, Rage Support linked. 37,634 DPS.

Screenshot, 50 Rage. Rage Support removed. 37,634 DPS. (It is removed; the screenshot was taken immediately after removal to get it before Rage dropped.)


Featuring on createjs.com

Hi guys
While I was going to shamelessly ask if you could feature our project in the carousel on createjs.com I have noticed that it does not work as intended, there seems to be too many dots and while Ion Drift is the last content, the rest is empty. See https://www.dropbox.com/s/lxuqj4jgdj3bh2r/Screen Shot 2017-03-23 at 15.55.52.png?dl=0
Our project is zoomable budget visualizator
Example: https://uk.wikibudgets.org/w/united-kingdom-budget-2015
Drag & Drop Editor: https://www.wikibudgets.org/sankey/
Also I would like to thank you, you have answered my questions on several occasions and we could not have build our project without the precise documentation and familiar architecture of EaselJS.
Have a great day!

Thanks for the report, we will check it out.
We would be happy to include it. We came across it last week on the FWA submissions, awesome work!
Cheers,

I am not sure if we were submitted to FWA, but thanks anyway. I have been following Grant Skinner since I learned programming 14 years ago, being featured on your website would be a great honour for me.

We added your site this morning. Cheers!
For anyone else with cool projects you want to share, tweet us @createjs!

Awesome, thanks! I can see it on http://www.createjs.com/easeljs but not on http://www.createjs.com/. Also your OG tag image seems to be blank.


added highest expressed genes QC

This PR adds the option to make an image like the following:
sc.pl.highest_expr_genes(adata, n_top=40)

This plot is similar to the one produced by scater function plotQC and is useful to identify highly expressed genes in a sample.
To keep the code tidy I added the new plot on  scanpy/plotting/qc.py I imagine that other QC plots can be added in the future. Possible future improvements can plot multiple panels by splitting the data using batch for example.
Additionally, this PR:

Changes the grey dot color of _scatter_obs to ligh_grey. This results in a better contrast of colors. E.g.:

sc.pl.umap(bdata, color='batch', groups=['PBMC'])

Added option to select the number of panels for rank_genes_groups. Without this option, if there are too many louvain groups, then the image is too wide. With the new parameter, it is easy to select how many panels per row should be plotted.

Awesome! :smile:

Another possible extension: Allow to select different distribution plots (for all of our distribution plots):

box plot
violin plot
swarm plot
joyplot


fix: :bug: Add missing operator-() for Quantity

Closing since https://github.com/LemLib/units/pull/33 implements this already.


osiris crashes the game with the new update.

I used 2 different injectors so I'm sure that it is Osiris. This happened after the new cs:go update. damiel pls fix

Please close
https://github.com/danielkrupinski/Osiris/issues/3577