Error while converting to keras model

I have trained yolo.weights (with my custom dataset having 3 classes) and the detections are coming fine while testing with that model. But when i am converting it to keras model, I am getting below error
File "convert.py", line 172, in <module>
    yolo4_model = Yolo4(score, iou, anchors_path, classes_path, model_path, weights_path)
  File "convert.py", line 157, in __init__
    self.load_yolo()
  File "convert.py", line 98, in load_yolo
    buffer=weights_file.read(weights_size * 4))
TypeError: buffer is too small for requested array

Could someone tell me why it is happening and what changes I should make

same problem

same problem with custom dataset

@elesun2018 @farhantandia any updates? I am also on this issue with the original yolov4.weights downloaded/

the model is successfully converted, I forgot to specify the classes.txt if you are using custom dataset.

@farhantandia I am using the stock yolov4.weight files?
When I run the convert.py, I am getting the buffer issue size. How did you resolve?

It also works for standard yolov4 weights in my case.

@farhantandia can you share your code?
the convert.py - specifically? Still not working for me haha

same problem, any solution?

convert.zip
@Amokstakov

This convert file is also not working please provide working codes

@vihari1729 @xellDart which version of keras (and TF) do you use? I faced the same error due to another version of keras that has slightly different indexing mechanism of new created layers, i.e. in my case the first Conv2D layer had name "conv2d" (in keras 2.4.0) instead of expected "conv2d_1" (in keras 2.2.4). So, some layers become missed and weights are assigned by wrong names with shift to another layers. This leads to this error when it faces with another shape of some layer.
Try to use these  versions

In line 71(convert.py) change "conv2d_" to conv2D
if layer_name.startswith('conv2D'):
There will be warnings but the tensorflow model "yolo4_weight.h5" will be generated.


FR: Stream.merge()

dart:async should have an easy way to combine multiple streams into one.
Example implementation which supports pause/resume.
Stream merge(List<Stream> streams) {
  int _open = streams.length;
  var c = new StreamController();
  for (var stream in streams) {
    stream.listen(c.add)
         ..onError(c.addError)
         ..onDone(() {
      if (--_open == 0) {
        c.close();
      }
    });
  }
  return c.stream;
}

Does StreamZip from dart:async do what you want? http://stackoverflow.com/questions/36571924/how-can-i-merge-multiple-streams-into-a-higher-level-stream

Sounds like StreamGroup.merge from package:async.


Revert to working image + add linux benchmarks.

Waiting for #1183 to land before rebasing and rerunning tests..


API token support

Hi guys
Will support for the new API tokens be added soon?
Thanks!

Hi,
i create PR #101

Please merge this and cut a release

Any idea on when it would be available ?

This has been closed as pat of #101 , I can't close it though as I'm not a maintainer.


Custom namespace when creating avro schema

I would like to customize the namespace when generating an avro schema. Currently this defaults to the name of the pydantic model but this can be problematic in some avro reader implementation in for example java where the name of the namespace cannot be the same as the name of the schema.

https://github.com/godatadriven/pydantic-avro/pull/49


Report in blank (desktop versión installed locally on Mac)

Hi Mike,
I have the desktop version installed locally on my mac and for a few days when I want to see the model report, it shows me the blank summary, I can't find a way to see what's wrong.
When loading the project I get an error "Could not call remote function 'setFeedURL', check that the function signature is correct. Underlying error: could not get code signature for running application"
I can add threats, flows and anything but when entering by the reports option I get the following error message "[Error] Cannot read property" cells "of undefined." for each diagram included in the model.
Do you know what may be happening?
I can share the model in case you have any idea what may be happening.

Hello @alexpeju
It works OK for me when running on MacOS 10.15 Catalina, so this may be a permissions or installation issue. Could you advise what Macbook and what MacOS you are running on? Agreed that it would be good if you would like to share your threat model json file.
Can you check that you are running the latest version? Either clone this repo and then invoke npm run install and then npm run start, or you can use the beta version dmg file from https://github.com/jgadsden/owasp-threat-dragon-desktop/releases/download/v0.1.27-beta/threatdragon_0.6.3_osx_x64.dmg

Will retest this when the new version of TD desktop is released

Hello @alexpeju - can you now use the latest version of TD desktop? It has been released here:
https://github.com/mike-goodwin/owasp-threat-dragon-desktop/releases/tag/v1.0
and as far as I can tell it behaves correctly now


New: Added slides link and comments to the user profile talks page

https://joindin.jira.com/browse/JOINDIN-615

Can one of the admins verify this patch?

ok to test

That's much nicer. Thanks @mheap!


SUMO Co-Simulation vehicle dynamics are not correct

Hi,
CARLA version: any
Platform/OS: (SUMO/OS)-versions that were tested: 1.12/ubuntu 18.04, 1.4.0/ubuntu 20.04
We are trying to use SUMO and Traci for traffic management and CARLA for the sensor aquisition, but the angular position of the vehicle is somewhat strange. As you can see from this picture

the yaw of the vehicle does not have a smooth evolution (see zoomed picture:)

The pitch, which should be constant and always equal to 0, also presents some discontinuities that are definitely noticeable in CARLA:

I have tried different car-following models (IDM, EIDM, smartSK), step lengths, integration methods (default Euler and ballistic), and action step lengths. The issue is always present.
I attach the files to reproduce the problem (and the graphs, remove the corresponding lines if not needed)

E-N_Crossing_CropField.rou.xml contains the "ego" vehicle route, which is the only vehicle present in the simulation
sumo_design.sumocfg is the SUMO configuration file
Town07_Opt.net.xml is the SUMO network for the map. It is automatically generated from a XODR file available from the CARLA Simulator (included in the zip file as Town07_Opt.xodr) using the script netconvert_carla.py, also included.
test_sumo_ego_only.py is the Python script using Traci for the simulation.

At this point, I can't tell if I'm looking at the wrong settings, or if there is an issue with the net file. I'm fairly new to SUMO so it may be either:)
Thank you for your help!
mwe.zip

The carla transfor sumo pos using below code
def get_carla_transform(in_sumo_transform, extent):
        """
        Returns carla transform based on sumo transform.
        """
        offset = BridgeHelper.offset
        in_location = in_sumo_transform.location
        in_rotation = in_sumo_transform.rotation

        # From front-center-bumper to center (sumo reference system).
        # (http://sumo.sourceforge.net/userdoc/Purgatory/Vehicle_Values.html#angle)
        yaw = -1 * in_rotation.yaw + 90
        pitch = in_rotation.pitch
        out_location = (in_location.x - math.cos(math.radians(yaw)) * extent.x,
                        in_location.y - math.sin(math.radians(yaw)) * extent.x,
                        in_location.z - math.sin(math.radians(pitch)) * extent.x)
        out_rotation = (in_rotation.pitch, in_rotation.yaw, in_rotation.roll)

        # Applying offset sumo-carla net.
        out_location = (out_location[0] - offset[0], out_location[1] - offset[1], out_location[2])

        # Transform to carla reference system (left-handed system).
        out_transform = carla.Transform(
            carla.Location(out_location[0], -out_location[1], out_location[2]),
            carla.Rotation(out_rotation[0], out_rotation[1] - 90, out_rotation[2]))

        return out_transform

The yaw is mostly a function of the network geometries which can be made smoother by setting netconvert option --junctions.corner-detail (i.e. 10 instead of the default 5).  Depending on the simulation option, lane changing operations can influence the yaw at might be another cause of discontinuities (https://sumo.dlr.de/docs/Simulation/SublaneModel.html)
The pitch comes directly out of the network geometry at the vehicle front and is currently not interpolated between front and back of the vehicle (a known issue in SUMO)


Measure tooltip disappearing (Google Chrome update)

Describe the bug
When using the measure interaction in the most recent Google Chrome browser update (version 72.0.3626.121), the tooltip intermittently disappears when moving the cursor around the map.
To Reproduce
Steps to reproduce the behavior:

Open the Measure OpenLayers example in Google Chrome v72.0.3626.121.

Expected behavior
The tooltip should persist as in previous versions of Google Chrome and other web browsers. Clearly this is a result of changes to the Google Chrome browser and could affect other functionality which uses tooltips or labels.

I'm unable to reproduce this. Can you verify on a different machine that it happens there too? Issues like this are sometimes caused by graphics card driver problems.

I can confirm the same issue happens on two other machines I have tested with the stated Chrome version (screen record attached of the issue).

I'm able to reproduce too. The following solves the issue (PS: dumb fix, didn't dive why it behaves this way...)
.ol-overlay-container {
  z-index: 1;
}

Does https://openlayers.org/en/master/examples/measure.html expose the same issue? We have changed z-index handling since the last release, so I think this issue should be fixed.

Does https://openlayers.org/en/master/examples/measure.html expose the same issue? We have changed z-index handling since the last release, so I think this issue should be fixed.

On initial load it appears to display correctly, but on changing measurement type or measuring more than one feature it has the same intermittent display issue.

Can this be worked around by changing the .tooltip css in the example? If so, a pull request would be appreciated.

Can this be worked around by changing the .tooltip css in the example?

Unable to solve it from my attempts (PS: I'm weak in CSS). Only solved the issue by playing at parent level of .tooltip hence the previous .ol-overlay-container use.


Update power-automate-vs-logic-apps.md

Fixed two grammar glitches.
Category

[x] Content fix
[ ] New article

Contents of the Pull Request
Fixing two grammar glitches. The first fixes a sentence that is just wrongly structured. The second fixes a sentence that is semantically wrong (claiming "same as" and "similar to" about the same thing).

Docs Build status updates of commit 5b5c3b0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Community/power-automate-vs-logic-apps.md
:bulb:Suggestion
View
Details

Community/power-automate-vs-logic-apps.md

Line 67, Column 1: [Suggestion: duplicate-alt-text - See documentation] Alt text 'Mobile view of Power Automate.' is duplicated. Within an article, alt text must be unique.
Line 82, Column 1: [Suggestion: duplicate-alt-text - See documentation] Alt text 'Mobile view of Power Automate.' is duplicated. Within an article, alt text must be unique.

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


Broken MQTT examples

Summary

From feedback widget:
In the publish example the 'auth' is missing from auth={'username':"app1... leading to a SyntaxError: positional argument follows keyword argument error while compiling. Also the 'context' module seems to be discontinued ?
https://www.thethingsindustries.com/docs/integrations/mqtt/mqtt-clients/eclipse-paho/

hmm, I fixed the auth part, but I can't find any info about the context module being discontinued. I double-checked examples to make sure they work, so I think this can be closed for now


[WIP]HIVE-21456:  Support Thrift over HTTP for HMS

What changes were proposed in this pull request?

This patch adds an initial version of supporting thrift over http in Hive Metastore.  It supports SSL with HTTP. The following would be addressed in follow up patches:

JWT authentication mechanism. In the current patch, a user has to pass x-actor-username in the HTTP header request.
Cookie based authentication
XSRF-filters support
Data Compression like GZIP in HTTP mode

Why are the changes needed?

This is a new feature
Does this PR introduce any user-facing change?

How was this patch tested?

Added unit and integration tests.

Closing this one as I need to address some issues. Will create a new PR

New PR: https://github.com/apache/hive/pull/3105


Not a best practice of the MPDS usage

Please avoid caching MPDS data into entries.csv and reading using ast.literal_eval. Moreover, the API key can be stored relatively securely in a user shell environment. Although CI usage is inapplicable in this particular case, a user should be able to test MPDS retrieval on his own machine with his own API key.

All the work on structures is now done in the mpds-aiida workflow; MPDS_KEY is now read from the environment variables.


Error in create_shader_module

When running cargo run --release I get the following output:
2023-10-09T08:56:21.939482Z  INFO bevy_winit::system: Creating new window "shadplay" (0v0)
2023-10-09T08:56:21.939613Z  INFO winit::platform_impl::platform::x11::window: Guessed window scale factor: 1.6666666666666667
2023-10-09T08:56:22.187716Z  INFO bevy_render::renderer: AdapterInfo { name: "Intel(R) Graphics (ADL GT2)", vendor: 32902, device: 18086, device_type: IntegratedGpu, driver: "Intel open-source Mesa driver", driver_info: "Mesa 23.1.6-3", backend: Vulkan }
2023-10-09T08:56:22.236138Z ERROR bevy_render::render_resource::pipeline_cache: failed to create shader module: Validation Error

Caused by:
    In Device::create_shader_module

Shader validation error:
  ┌─ :1:1
  │
1 │
  │
  │
  │ naga::Expression [36]
  │ invalid function call

    Entry point fragment at Fragment is invalid
    Call to [3] is invalid
    Argument 0 value [36] doesn't match the type [2]

Rust version/toolchain: (rustup show)
active toolchain
----------------
stable-x86_64-unknown-linux-gnu (default)
rustc 1.72.0 (5680fa18f 2023-08-23)

Operating system: Manjaro Linux.

By default the app opens with the my_shader2d.wgsl -- swap the contents of that file with this:
@fragment
fn fragment(in: MeshVertexOutput) -> @location(0) vec4<f32> {
    var col = vec4f(1.0);

    return col;
}

You should see a pure white canvas.

Are you able to run other bevy projects on 0.11.3.

Yes that works, thanks! Yeah I can compile bevy 0.11.3 projects.

Must've accidentally committed a bad shader at some stage on the branch you pulled -- sorry.


AbstractFoldingStrategy not found

AbstractFoldingStrategy isnt found,
and this is all thats in the file
namespace E2Edit.Editor
{
class E2FoldingStrategy : AbstractFoldingStrategy
{
public override IEnumerable CreateNewFoldings(TextDocument document, out int firstErrorOffset)
{
throw new NotImplementedException();
}
}
}

the file isn't needed just remove it


Destination "unrecognized argument"

When I try to create an image, I get an error saying the dest argument is unrecognized. Even directly using the example command:
"dynmap-timemachine.py http://map.majncraft.cz/ world surface \ [3300,65,-2630] [3,2] 0 majncraft.test.png"
gives me:
"dynmap-timemachine.py: error: unrecognized arguments: majncraft.test.png"

The readme is wrong, do this instead:
"dynmap-timemachine.py http://map.majncraft.cz/ world surface [3300,65,-2630] [3,2] 0 majncraft.test.png"

Thanks, it works now!


Feature Request: Get List of Shared-Links

First - Thank you for this fantastic tool. Being a Box Administrator would be extremely difficult without the CLI.
I'm using @box/cli/2.0.0 win32-x64 node-v10.10.0 . It has the feature to
box shared-links:get <url>

that allows me to get the details of an existing shared-link.
REQUEST:
Can you add a feature to list all existing shared-links in my enterprise? This is a needed feature for security audits. As an administrator, I need to be sure that sensitive info isn't accidentally shared by my users and the first step is to see what has been shared.

@grugknuckle I'm glad the CLI is useful for you!  Getting all shared links in an enterprise is actually something that's supported directly in the Box Admin Console.  If you go to the Reports section of the Admin Console, there should be a Shared Links report that you can run.  It will create a spreadsheet for you of all the links in your enterprise with information about the settings for each of them.  Does that do what you're looking for?

@mattwiller Using the Admin Console does allow me to produce the data - yes - but it requires a human being to log in and make the request. For my particular use case, I have bot which periodically performs administrative maintenance tasks like moving files and folders around. This bot uses the CLI binaries and the box-node-skd.
One of the tasks I'd like to perform is an audit of the shared links. Because I don't trust that my users understand the security implications of sharing folders and files to "People with the Link". See this and this for detail about what I mean. Having the ability to programatically audit all of the enterprises' shared links will permit me to do things like

Determine which folders / files are public facing
Determine if sensitive information is unintentionally exposed
Notify the person or people responsible for those links so that they can modify the link to have an expiration date or password, or even just delete the link.

Of course, all of this can still done by first exporting a .csv file from the admin console, but it's an extra step that a human has to remember to perform.
I recognize that for my use case, this feature request might be more appropriate for one of the Box SDKs - since the SDK doesn't have this feature either. But I thought that if I asked for the feature here, you might eventually implement it there.

@grugknuckle Unfortunately, the ability to get all shared links in an enterprise does not exist in the Box API — there's not really a good way for us to add it to either the SDKs or the CLI at this time.  The best way to get this information is through the Admin Console reports.  If you're using the SDK, you could always crawl through the entire set of users in the enterprise and check every file or folder they own for a shared link — not ideal, but that's the only way I can think of to do this programmatically today.


Generate graphs with blank node graph name

In R2RML a graph map always generates an IRI. However in RDF there is no such requirement. See also the draft https://www.w3.org/TR/rdf12-concepts/#dfn-graph-name.
Should RML allow graphs with blank node graph names?

If you look at Section 4 of https://www.w3.org/TR/rdf11-concepts/ : "Zero or more named graphs. Each named graph is a pair consisting of an IRI or a blank node (the graph name), and an RDF graph. Graph names are unique within an RDF dataset."
Then the answer would be yes. It makes, to me, a lot more sense for non-constant graph maps. The mapper should know that referring to a constant blank node that may have metadata in the mapping: e.g.,
_:a123 dc:title "blank!".
and its "counterpart in the mappings will not refer to the same node.

@chrdebru I'm not sure I follow.

and its "counterpart in the mappings will not refer to the same node.

Or did you mean, its counterpart in the output dataset?

Indeed, in the output dataset, first, if I use _:a123 as a BN identifier in my mapping, then it does not guarantee that the same BN identifier is used in the output dataset. In (R2)RML, this is even more general, as values yielded by templates are not to be used for BN identifiers in the output dataset; it's just a mapping from such values to internal identifiers.
If I have in my mapping:
<#sm> rml:graph _:a123 . and elsewhere in that file declare that ```_:a123 dc:title "blank!".```` then the resulting RDF will only refer to a blank node (potentially with the same BN identifier, but not guaranteed) and thus the two BNs will refer to two different resources.
See the following note https://www.w3.org/TR/rdf11-concepts/#dfn-blank-node: "Blank node identifiers are local identifiers that are used in some concrete RDF syntaxes or RDF store implementations. They are always locally scoped to the file or RDF store, and are not persistent or portable identifiers for blank nodes. Blank node identifiers are not part of the RDF abstract syntax, but are entirely dependent on the concrete syntax or implementation. The syntactic restrictions on blank node identifiers, if any, therefore also depend on the concrete RDF syntax or implementation. Implementations that handle blank node identifiers in concrete syntaxes need to be careful not to create the same blank node from multiple occurrences of the same blank node identifier except in situations where this is supported by the syntax."
Base IRIs (available in Turtle and not in n-triples) are used to resolve relative IRIs. Using base IRIs to "scope" blank nodes is a misuse of base IRIs.

Yes, so we maintain the rule of no constant blank nodes, as with the other term maps, however allow for blank nodes to be generated by non-constant graph maps.


Protocol 22 Changes

Protocol 22
XDR Changes

Added new XDR support for Soroban constructors which allows a contract to run custom initialization logic atomically when it's first created. You can read more about this change in CAP-58. The following changes are introduced:

A new special contract function __constructor that may only be called by the Soroban host environment. The environment will call __constructor function if and only if the contract is being created from a Wasm exporting that function.
Introduce a new host function create_contract_with_constructor in Soroban environment that allows constracts to instantiate other contracts with constructors.
Introduce a new HostFunction XDR variant HOST_FUNCTION_TYPE_CREATE_CONTRACT_V2 that acts in the same way as HOST_FUNCTION_TYPE_CREATE_CONTRACT, but also allows users to specify the constructor arguments.
Introduce a new SorobanAuthorizedFunction XDR variant SOROBAN_AUTHORIZED_FUNCTION_TYPE_CREATE_CONTRACT_V2_HOST_FN that allows users to sign the authorization payload corresponding to HOST_FUNCTION_TYPE_CREATE_CONTRACT_V2 host function calls.

Horizon API
Breaking Changes

Changed ErrorResultXDR field naming from camelcase to snakecase - error_result_xdr - for transactions_async endpoint. (#5445)
The following deprecated fields have been removed (#5478):

/asset: num_accounts and amount fields
/transactions/:id: valid_before and valid_after fields

Soroban-RPC
Breaking Changes

createdAt field in getTransaction response is now encoded as string instead of int64. (#251)
The fields in getVersionInfo response were changed to Camel-case naming (#164):

commitHash
buildTimestamp
captiveCoreVersion
protocolVersion

The legacy cost field has been removed from simulateTransaction response. (#295):

The correct resource costs can now be retrieved from the transactionData XDR in the response.

Remove pagingToken from getEvents response and replace it with cursor. (#297)

Instead of being present in all event objects, the cursor will be at the top level of the response similar to getTransactions. This makes it easier to use and brings it in sync with how Cursor is used in other RPC endpoints

Deprecated getLedgerEntry endpoint is now removed. (#276)

Users can use the more powerful getLedgerEntries endpoint to get the same result.

Non-Breaking Changes

Add transaction hash field - txHash - to getTransactions and getTransaction response. (#299, #314)

Each transaction object in the response now also includes a hex-encoded transaction hash string.

SDF Reference Implementations

JavaScript
Java

Hello @aditya1702, thank you for adding this issue. It has been resolved in release 3.0.1-beta.1


add university

University :
Catholic University Of Korea
Link:
https://www.catholic.ac.kr/ko/index.do

@kokomai Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


Update dependency version and change zooKeeper variable to zookeeper

What this PR does / why we need it:

Update patroni dependency version:

zookeeper (1.0.0)
etcd (0.3.9)

Change zooKeeper variable to zookeeper to save dependency convention compatibility.

Special notes for your reviewer:

/assign @sameersbn

/assign

/ok-to-test

/lgtm


Simulate modifier key press autogen?

I would like to add few autogens to Mouse Keys Mode v2 if I can.
I want to make it so that when I hold 'd' key together with 'b' key, it would simulate a mouse click with 'control' modifier. This would be very useful for me especially in program like Xcode where I have to drag things while holding the control modifier. There was no place in the documentation where I could find how to do such a thing.
I would really appreciate any help on this. Thank you a lot once again for such a wonderful piece of software.

open file "/Applications/Karabiner.app/Contents/Resources/include/checkbox/mouse_keys.xml"
add below to line around 103 seems like a good spot ( any where between line 97 to 216 will do. )
change all the  name with prifix "private." and save it to where your private.xml file folder. Then add  in private.xml file to point to the new mouse xml file. This way when ever the software update you won't lose your setting. key "B" is taken for "middle mouse click", I use "A" here for your "control" key. You can edit to fit your need maybe swap the "A" and "B" key.
`
KeyToKey
KeyCode::A,
    KeyCode::CONTROL_R, ModifierFlag::MOUSE_KEYS_MODE_2
  </autogen>`

You can use MOUSE_KEYS_MODE_2_EXTRA to add key binds.
Example:
https://github.com/tekezo/Files/blob/master/Karabiner/private.mouse_keys_mode_v2_ijkl/private.xml


Add type-agnostic CABundleInjectorConfig

Required by https://github.com/openshift/service-ca-operator/pull/78

/hold

Rebased

Rebased

/lgtm

/hold cancel

@sttts Can this merge?

Create a new value for the new type and keep the old in place.  We don't document how to adjust this anywhere.
I will ask a separate question: do we actually need to expose these options anywhere?  Why would a user change it?

Create a new value for the new type and keep the old in place. We don't document how to adjust this anywhere.

Understood.  So the existing types will just linger indefinitely?

I will ask a separate question: do we actually need to expose these options anywhere? Why would a user change it?

That was my thought in a previous comment. But I started to question my assumption that the controller didn't need configuration - was GenericControllerConfig a  mandatory requirement for operator-managed controllers? If not, I'm happy to configure the injection controllers without a configuration type and one can be added in the future if users demand it.

Updated to add a new type and leave the old ones in place
Removed CABundleFile field - this will be hard-coded or supplied by an argument to the command rathe than provided via config since it is entirely dependent on pod configuration.

@deads2k @marun since this is alpha, will be we at least able to remove the API in later versions, say, 4.4/4.5? If so, perhaps we should add that to the deprecation note and proceed with the removal in the later version mentioned.

I've revised this PR to add deprecation comments to the controller-specific configuration types and fields, and will be updating the operator to reuse the operator controller configuration for the controllers that it manages.

@sttts @stlaz I propose that the types and fields marked by deprecation warnings instead be removed immediately. You can confirm the assertions below by reviewing the proposed removal of their use.

The types are alpha and internal-only - they do not provide a way for a customer to influence controller behavior:

For each of the 3 controllers it manages, the operator creates a configmap that includes a serialized instance of the controller-specific configuration type (e.g. ServiceServingCertSignerConfig)
The controller-specific config is sourced only from bindata
Each controller mounts the bindata-sourced configmap and deserializes the contained conttroller-specific configuration type

The fields are unused. There is no support for reading controller-specific configuration types from the operator configuration.

/lgtm
/approve

/retest

/lgtm


Update junit to 4.13.1

Signed-off-by: Ashish Agrawal<EMAIL_ADDRESS>Description
Update junit from 4.12 to 4.13.2
Issues Resolved
N/A
Check List

[x] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

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

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

In test report 1757:
* What went wrong:
Execution failed for task ':dependencyLicenses'.
> Missing SHA for junit-4.13.2.jar. Run "gradle updateSHAs" to create them

Hi @lezzago , please run ./gradlew updateSHAs to update the checksum for the jar file. 😁

In test report 1757:
* What went wrong:
Execution failed for task ':dependencyLicenses'.
> Missing SHA for junit-4.13.2.jar. Run "gradle updateSHAs" to create them

Hi @lezzago , please run ./gradlew updateSHAs to update the checksum for the jar file. 😁

Oh thanks, I will do that. Also is this in the developing guide? If not, might be worthwhile to add this in there for others in the future.

Oh thanks, I will do that. Also is this in the developing guide? If not, might be worthwhile to add this in there for others in the future.

It's not explicitly told in the developing guide, but it's shown in the test report. 😄 Thanks for your opinion, maybe we'd better find a place to mention it.

The above test failure is a known issue https://github.com/opensearch-project/OpenSearch/issues/1806.
I will re-triggert the check: start gradle check.


Question about BCEWithLogitsLoss

Hi, thanks for your nice library. I like the way the tasks are laid out cleanly and easy to read.
So I have a 5-task network that I'd like to build off of a shared backbone ... pretty simple. The simplest, vanilla multi-task PyTorch (no backbone) code might look something like
model = nn.Linear(20, 5) # predict logits for 5 classes
x = torch.randn(1, 20)
y = torch.tensor([[1., 0., 1., 0., 0.]]) # get classA and classC as active

criterion = nn.BCEWithLogitsLoss()
optimizer = optim.SGD(model.parameters(), lr=1e-1)

for epoch in range(20):
    optimizer.zero_grad()
    output = model(x)
    loss = criterion(output, y)
    loss.backward()
    optimizer.step()
    print('Loss: {:.3f}'.format(loss.item()))

now using torchMTL ... I have my tasks setup like below with a number of tasks, built on task-specific "stacks", sitting on a shared backbone. Each task is a binary 0/1 decision, so a Linear layer with dimension=1 for the output and BCEWithLogitsLoss. My question is the the correct setup, i.e. output-dimension==1?
...
    {
        'name': "Task2",
        'layers': Sequential(*[Linear(64, 1)]),
        'loss': BCEWithLogitsLoss(),
        'loss_weight': 1.0,
        'anchor_layer': 'StackB'
    },
    {
        'name': "Task3",
        'layers': Sequential(*[Linear(64, 1)]),
        'loss': BCEWithLogitsLoss(),
        'loss_weight': 1.0,
        'anchor_layer': 'StackC'
    },

The reason I ask the question, is that in my trivial example above. I use multi-hot encoded targets, nn.BCE(WithLogits)Loss and an output layer returning [batch_size, nb_classes] so BCEWithLogitsLoss processes all the output tasks at once, treating each output independently and so is suitable for multi-label classification use cases. So 1 criterion, 5 classes
In my torchMTL code, I'm not sure how to aggregate the outputs of each of the tasks in order to stack them together or even if I should, because it looks like I have 5 criterions and 5 classes ...which might work for regression, but my code is broken wrt to  BCEWithLogitsLoss. Do you have any guidance?  Thank you!

I resolved the issue/question that I had. I can indeed use/track individual criterions separately. So by setting up my training example labels y to have an extra dimension via unsqueeze(2), then I sliced y along the appropriate dimension when computing the per-task loss, i.e.
ith_loss = l_funcs[i](y_hat[i], y[:, i])

The choice to of how many "tasks per loss" to track/compute with BCEWithLogitsLoss (which can process multiple tasks at once) is still a design issue, but multiple Linear layer stacks with single output dimensions (as above are not limited by this framework. Closing the issue.


Setting pages icon get cropped

Microsoft PowerToys version
0.49.1
Running as admin

[X] Yes

Area(s) with issue?
Settings
Steps to reproduce

Collapse the side menu

✔ Expected Behavior
Icon not getting cropped.
❌ Actual Behavior
Icon gets cropped.

Other Software

cc: @niels9001

This may not be a thing after WinUI 3 upgrade

I don't think this happens anymore with settings after the WinUI shift.
/needinfo


For documentation repositories with only 1 file (README.md), have documentation file structure auto-hide

We're moving toward a "single page" documentation style, which makes the right-hand panel for the documentation file structure irrelevant.

In this case, have the page (by default) switch to the "hidden menu" view:

(Maybe we'll come up with a better way to utilize a right-hand panel in the future with other things besides the documentation file structure)

This sounds fine, we could also have it try to automatically determine a table of contents that would create anchor links throughout the page if you like.

Sounds worth trying - if single page, build TOC based on ## sections.

Ok, so single page documentation now automatically creates a table of contents with anchor links that are clickable.


Test commit

Current dependencies on/for this PR:

develop

PR #130   👈

This stack of pull requests is managed by Graphite.


Enhancements & fix a few bugs

Hey there!
This PR adds some enhancements and fixes a few bugs I found while using this tree sitter in another project! I've updated grammar.js, added tests, and have made sure that make test_wild runs with no issues. The enhancements/bugs are:

Wrap raw JS in it's own node for JS tree sitter injection
Parsing mutually recursive type declarations (https://github.com/nkrkv/tree-sitter-rescript/issues/56)
Parsing a list of statements in a case branch
Having multiple statements in a %%private(...)

Sorry theses aren't broken up into multiple PRs. These fixes were just based on use in my projects and I figured others may benefit from them too.
Thanks for the great project!

That’s great, thank you a lot! I’m going to test & merge tonight.

Thank you!
I wonder, you say “Wrap raw JS in its own node for JS tree sitter injection”, do you have JS inside %raw highlighted properly? If so, how? Are there some extra scm’s defining what to do with (raw_js) to make it happen?
P.S> In this PR I see quite a lot of $._newline and $._statement_delimiter references that are very low-level and should not leak around grammar.js. Now I see, that scanner.c needs improvement to properly address the issue, but for now, I merge as-is. Will try to refactor/simplify soon, keeping the resulting behavior intact.

I'm working on getting JS properly highlighted! I know it's possible, because HTML tree sitter it can highlight JS and CSS right in the HTML file.
Here's the scm file they use to denote it:
((script_element
  (raw_text) @injection.content)
 (#set! injection.language "javascript"))

((style_element
  (raw_text) @injection.content)
 (#set! injection.language "css"))

I believe with rescript, we could so something like:
((raw_js) @injection.content
 (#set! injection.language "javascript"))

But I haven't had time to test it yet.
If I can get it to work I'll probably PR something similar for GraphQL as I use the graphql_ppx often and I know there's a GraphQL tree sitter.

@nkrkv, quick screenshot of JS tree sitter injection in my editor!


docker run flyway fails

Which version and edition of Flyway are you using?
latest docker image
If this is not the latest version, can you reproduce the issue with the latest one as well?
(Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Which database are you using (type & version)?
Which operating system are you using?
windows 10 pro 64x
What did you do?
(Please include the content causing the issue, any relevant configuration settings, the SQL statement that failed (if relevant) and the command you ran.)
commandline: docker run --rm boxfuse/flyway
What did you expect to see?
download of latest flyway docker image
What did you see instead?
docker: Error response from daemon: Get https://registry-1.docker.io/v2/: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers).
See 'docker run --help'

Not a Flyway issue. Most likely an issue with your environment.


Add Channel Creation Benchmark

Run from my desktop:
[hi on] DESK ~/Desktop/grpc [channel-benchmark] (0) $ bins/opt/bm_channel
Run on (12 X 3800 MHz CPU s)
2018-06-13 15:59:51
***WARNING*** CPU scaling is enabled, the benchmark real time measurements may be noisy and will incur extra overhead.
---------------------------------------------------------------------------------------------------
Benchmark                                                            Time           CPU Iterations
---------------------------------------------------------------------------------------------------
BM_InsecureChannelCreateDestroy<InsecureChannelFixture>/0        11489 ns      11442 ns      50065
BM_InsecureChannelCreateDestroy<InsecureChannelFixture>/1         4719 ns       4719 ns     137706
BM_InsecureChannelCreateDestroy<InsecureChannelFixture>/8         4780 ns       4778 ns     147264
BM_InsecureChannelCreateDestroy<InsecureChannelFixture>/64        4778 ns       4778 ns     147318
BM_InsecureChannelCreateDestroy<InsecureChannelFixture>/512       4769 ns       4769 ns     148001
BM_InsecureChannelCreateDestroy<LameChannelFixture>/0              806 ns        806 ns     888703
BM_InsecureChannelCreateDestroy<LameChannelFixture>/1              784 ns        784 ns     894061
BM_InsecureChannelCreateDestroy<LameChannelFixture>/8              809 ns        809 ns     846039
BM_InsecureChannelCreateDestroy<LameChannelFixture>/64             797 ns        796 ns     873181
BM_InsecureChannelCreateDestroy<LameChannelFixture>/512            766 ns        766 ns     913368

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

I though I checked this in with my channelz change. This will be useful to have moving forward. We don't have much coverage on channel creation time.
@tdbhacks, this can be a starting point for your benchmarking of the changes to security_connector

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

@ncteisen thank you!

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences


GitGitGadget PR Handler pipeline is broken with a Git error

Something is broken about the GitGitGadget PR Handler pipeline.
See this pipeline error (I needed to visit it anonymously to not get a strange 404 screen in the log output) for this error:
Starting: Pull GitGitGadget, npm install & build
==============================================================================
Task         : Bash
Description  : Run a Bash script on macOS, Linux, or Windows
Version      : 3.246.1
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/bash
==============================================================================
Generating script.
========================== Starting Command Output ===========================
/bin/bash /data/gitgitgadget-agent/agent/_work/_temp/91495ef7-bbc2-42cf-a9e1-e0b03054b1d7.sh
From https://github.com/gitgitgadget/gitgitgadget
 * branch            main       -> FETCH_HEAD
error: update_ref failed for ref 'HEAD': cannot lock ref 'HEAD': Unable to create '/data/gitgitgadget-agent/agent/_work/1/s/.git/HEAD.lock': File exists.

Another git process seems to be running in this repository, e.g.
an editor opened by 'git commit'. Please make sure all processes
are terminated then try again. If it still fails, a git process
may have crashed in this repository earlier:
remove the file manually to continue.

This is blocking gitgitgadget/git#1818 from getting sent to the mailing list.

Sorry for the breakage! This patch series iteration is submitted now: https://github.com/gitgitgadget/git/pull/1818#issuecomment-2449086855
The build agent had gotten into a bad state, which I have fixed (there were a stale HEAD.lock and a stale logs/HEAD.lock file, both created Oct 28 19:41 UTC, and then there were corresponding lock files for the main branch that were created Oct 29 01:54 UTC, I deleted them all).
I guess this ticket provides more motivation to address #609 (which would imply a move from a self-hosted agent to hosted agents).


Fix CI / Standalone CMake so it doesn't always try to pull JUCE

Had this problem on the plane, test without wifi...

Has to be built out of source or use https://forum.juce.com/t/infinite-loop-calling-juce-add-module-from-a-cmakelists-txt-in-the-modules-own-directory/58783/15


Enhancement: Support parsing plain HTML

I see that some people are using external files with vue templates only. We have an open issue regarding this topic here: https://github.com/vuejs/eslint-plugin-vue/issues/490
Would it be possible to support parsing plain HTML in addition to SFC?

Thank you for this issue.
Hmm, I'm thinking. This parser doesn't support whole HTML spec. For example, this doesn't parse <script> tags in the template properly as different from eslint-plugin-html. It's hard to distinguish between a normal HTML file and a Vue.js template HTML file. eslint-plugin-vue needs a preprocessor, so I have a concern that HTML supports will break if someone is checking normal HTML files with another plugin because ESLint cannot use multiple preprocessors for each extension.

We could make it an optional rule, and let's say require templates to have .vue.html suffix in order to be parsed and checked against. Also we already mention that our plugin doesn't work with eslint-plugin-html, so it shouldn't be a problem in my opinion.
We have plenty of useful rules, and there is also a great eslint-plugin-vue-a11y, that people could also take advantage of if this parser would support parsing html files :)
It doesn't have to support whole HTML spec. What it supports at this moment should be perfect. I wouldn't bother about <script> tags in plain HTML. JS is to be used outside of the .html templates anyway :)

Adding a <!-- @vue/component --> in the template.html isn't on option here? Similar to how the components handled in separate js files.

We could make it an optional rule

We cannot make plugin preprocessors optional.

let's say require templates to have .vue.html suffix in order to be parsed and checked against.

I guess that this solution cannot avoid the confliction between eslint-plugin-vue and eslint-plugin-html. Because .vue.html ends with .html, so eslint-plugin-html should unwrap script tags in .vue.html files.

Also we already mention that our plugin doesn't work with eslint-plugin-html, so it shouldn't be a problem in my opinion.

We can use those together. I can check .vue files with eslint-plugin-vue and the script tags of .html files with eslint-plugin-html. I'm not glad if this use case is broken.

Adding a  in the template.html isn't on option here? Similar to how the components handled in separate js files.

We cannot check the HTML comment because this is a parsing matter. We have to distinguish it before parsing files.

eslint-plugin-html should unwrap script tags in .vue.html files.

There shouldn't be a reason to add <script> tags to the .vue.html files. It would be like adding <script> tags inside <template>.
So eslint-plugin-html would parse the .vue.html, but it wouldn't lint anything.

@tfoxy It's a confliction matter. Both vue and html plugins handle the same file, then both plugins will get unexpected results.

Is there really no way to get this to work? I'm not using eslint-plugin-html, which doesn't seem necessary for a Vue project. For a moment I thought src imports were just a hack, but they appear to be part of the SFC Spec. It would be really nice if everything behaved the same way if someone moved from a regular SFC to using imports.

Any idea how I could lint .html files if they are just sourced by .vue files ?

Any idea how I could lint .html files if they are just sourced by .vue files ?

Well, looks like I have kinda hacky idea. Here is my-component.vue file:
<script lang="js" src="./my-component.js" />
<template lang="html" src="./my-component.html.vue" />

As you can see, template extension is .vue now. So lint works fine. This doesn't affect compiling process since loader doesn't care about files extensions.
Can confirm that most of rules work as expected, e.g. vue/attributes-order, vue/multiline-html-element-content-newline, etc.
Important note: your template should be wrapped into <template> tag.
<template>
  <div>
     ...
  </div>
</template>

Still not sure if it can be recommended to use safely, hope @mysticatea can leave a comment like an expert.

I got something to somewhat work:
{
  "processor": "vue/.vue",
  "rules": {
    "vue/no-multiple-template-root": "off"
  },
  "overrides": [{
    "files": "*.html",
    "processor": "vue/.vue"
  }]
}


Deploy programs

As a developer, I would like to be able to deploy my programs to "devnet", "testnet" or "mainnet-beta" from the UI.

@anoushk1234 I'm working on this one already. I forgot to assign it to myself.


net-snmp: split outputs

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

I tested all packages directly depending on net-snmp.

cc @thoughtpolice for h2o, maybe the choice of building against libressl was deliberate?

dnsdist is the only package in nixpkgs that uses it and it already caused problems to mix ssl libraries.

Keepalived works fine. Added a NixOS test.

cc @thoughtpolice for h2o, maybe the choice of building against libressl was deliberate?

No, this was probably an error on my part, and it should be fine to change it.


investigate possibility to add open source test case management into the infra

need research in adding any open source test case management system like testlink, rth etc as part of default infra

closing as won't fix


Use subject id as user id

Hey there,
just a tiny fix that made it work for me at least
for me there wasn't a userid in the headers.
Digging around I think, that Oidc_claim_sub, the subject ID is the one to use

The headers are set by Apache, the claim is configured automatically to use "sub" by the exchange with the OIDC provider (although there are more options to configure it).
This particular line in the config sets the name Apache gives for the sub field obtained in the profile response when it adds the headers before forwarding the request to servicehub/your app:
https://github.com/HumanBrainProject/servicehub/blob/master/config/servicehub.conf#L23

okay, thank you for updating the config then!


objects should return binary/octet-stream when no content type is specified during put

currently s3proxy returns application/unknown vs aws s3 returns binary/octet-stream

@kahing The various Java SDKs and HTTP clients interfere with me testing this.  Can you change S3ProxyHandler.addContentMetdataFromHttpRequest to add the default Content-Type and test your change locally?


OpenStack Nova Helm Chart Initial Commit

This is an initial commit of a functional nova chart.  It has been
tested with the neutron chart using a flat network configuration.

This change is

The bulk of the effort was accomplished by @DTadrzak's existing Nova WIP closed in https://github.com/att-comdev/openstack-helm/pull/45

@intlabs We can also apply that fix directly in image. AFAIK image on quay.io is already fixed.

@DTadrzak we can, and that's what I do with my old ones as well - but to ensure we are image agnostic I think the solution Alan has made is better.


GPSTime broken

Firmware download from site today along with latest code from BangleApps.
Similar issue with apps/firmware last Wednesday too. But I was also getting GPS time errors on main settings screen last week. And only satellite image showing on screen in app last week, not satellite count.

So everything works fine just after I do "install default apps", as the watch has received an accurate time from the PC. But if I reboot, it obviously reverts to Jan 1 1970, and then all of the errors described above appear. As it's a Firmware issue I should probably close this and open it on Espruino repo?

No, that's fine - I think it's an app issue. Newer firmwares set fix.time===undefined if the GPS doesn't have a time yet, and I imagine that's causing the issue.

Also, new firmware automatically grabs GPS time on reboot (even without the app) if the GPS time is set

Thanks Gordon. Something is also logging "GPS receiver's time not set" to the screen too after a reboot. Trashes the watchface.

Thanks - yes, that'd be the code above. I guess that logging could just be removed. Also related to https://github.com/espruino/BangleApps/issues/102 so I'm interested to see what you think the best fix might be

Option 1 in #102 for sure. Should definitely be different behaviour between "I'm using this as a watch day to day" vs  "I'm developing on the watch". For the former, it should be as close to a consumer experience as possible.


feat(fromEvent): improve type inference on event name

This change updates the type of the event name to be covariant with string. This permits using listeners that are defined as:
addEventListener(event: 'foo'|'bar', listener: ...)

Without this change, the following error is produced because the typings are too tight around string:
Argument of type 'BrowserWindow' is not assignable to parameter of type 'FromEventTarget<unknown>'.
  Type 'BrowserWindow' is not assignable to type 'JQueryStyleEventEmitter'.
    Types of property 'on' are incompatible.
      Type '{ (event: "always-on-top-changed", listener: (event: Event, isAlwaysOnTop: boolean) => void): BrowserWindow; (event: "app-command", listener: (event: Event, command: string) => void): BrowserWindow; (event: "blur", listener: Function): BrowserWindow; (event: "close", listener: (event: Event) => void): BrowserWindow;...' is not assignable to type '(eventName: string, handler: Function) => void'.
        Types of parameters 'event' and 'eventName' are incompatible.
          Type 'string' is not assignable to type '"always-on-top-changed"'.ts(2345)

Hrm okay, nevermind. It seems that making this change will break a lot of existing interfaces and backwards-compatibility means that the type parameters will be ordered awkwardly...


First Exchange Online Test

Added Connect-ExchangeOnline support to Connect-Maester
Added MS.EXO.1.1v1
Updated template files

Builds on #184 and #206

Handling permissions may need some additional review. The O365 Exchange Online app permissions and Entra roles aren't directly aligned with the EXO management roles. All the ExchangeOnlineManagement cmdlets have a minimal read only management role though.


Line endings on Windows.

Black outputs CRLF line endings on Windows even if the source file has Unix-style lines. It's possible to work around using for example Git configuration, but it would be nice if the behavior was configurable. Some of us like to keep Unix line endings even on Windows.

Thanks, that's exactly the bug I'm noticing! default_line_ending is a Sublime setting no? So not exactly the right solution but fixing this bug should also fix that one.

Hm. I think what Black should do here is to retain existing line endings. That doesn't require any configuration.

@zsol, I will assign this to you as you have opportunity to test this on Windows.

@ambv there are projects that have a mix of windows and unix line endings even in a single file so if you picked such a project you won't be able to make it consistent with only black run, you will need to run all files through some tool that will convert line endings across the project, it's not hard to do but inconvenient.

black brands itself as uncompromising and opinionated. Perhaps it would be simpler to always choose Unix-style line endings for a consistent output. I recently started working on a project with Windows line endings and noticed black had no opinion on them.


Fix intermittently failing tests

Description
Integration test: shouldFetchUnknownAncestorsOfPropagatedBlock() and the acceptance test: shouldCreateTheSameGenesisState() fails intermittently, making it hard to understand if the problem is related to the current PR or something we've had previously.

May be related to https://github.com/libp2p/jvm-libp2p/issues/94

This issue is distinct from https://github.com/libp2p/jvm-libp2p/issues/94


Restrict CFE_EVS_SendEventWithAppID to core use only

Is your feature request related to a problem? Please describe.
Similar to #1388 which would be a bigger change, but at minimum should remove this from the public API so libs don't start using it.
Describe the solution you'd like
Move to internal/private.  Also need to confirm requirements match up.
Describe alternatives you've considered
#1388 - probably should be removed all together, but that's a bigger issue
Additional context
Code review
Requester Info
Jacob Hageman - NASA/GSFC

CCB: 2021-04-21 - decided against restricting.  There is a valid use case, and filtering/reporting could be updated to make this more obvious (There's EID context and calling context that is helpful).


fix: Restrict e as input for table header index

Description
Fixes # (issue)
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Manually in local
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
No changes to code coverage between the base branch and the head branch

Merging this as the failing test are not related to the changes.


Add git hook to shell to check formatting before commit

@MangoIV I think this does everything you want?

I just thought that one cleaned up proven to be working, nixified thing that works for both formatting and shell hooks both locally and in the ci was nice. it also had a really neat ui and integrated well with direnv.
Other tools were easy to add. (e.g. hlint and shellcheck will need more bashscripting, why do this if there's a done thing?)
I don't get the sentiment of reducing "bloat" for the sake of it.

It's extremely complex and offers no real benefit. I don't think ignoring files that haven't changed is a great feature, because just checking that can often be more expensive than just doing it. In our case, one of our checking tools is horribly slow, can't remember which, but just optimising that would be more fruitful.
What do you mean it doesn't work? This sets the pre-commit hook if it hasn't been set already.

Done in central flake


no closing in Firefox with surrounding label-tag

First: thx a lot fo this great picker :+1:  :)
In Firefox the picker doesn't close if there is a surrounding label tag. Neither on click on "close" nore onSelect or onClear.
In Chrome only the closeOnSelect and the closeOnClear don't work.
Closing with "esc" key does work.
I think it has something to do with the focus, but I didn't figure out what could be changed.
Regards

I didn't notice this on desktop Firefox, but when closing the date picker in FF Mobile I need to tap twice.
Do you think it's the same trouble?

Hi,
hm, as it happens I just used the latest version on a new installation and everything works fine. I didn't figure, what is the cause.
Maybe double selectors (classes or ids) on the page? Just a shot in the dark :)

A label performs another focus event..in turn opening it again. There's a workaround mentioned in the docs: http://amsul.ca/pickadate.js/date/#container


Add Age Validation in Profile Section

Description
In the profile section's form, there is a label for "age", which allows users to update their age. However, currently, there is no validation in place, and users can enter negative values. To ensure data accuracy and consistency, we need to add age validation to ensure that only positive values are accepted.
Screenshots

Age shouldn't be negative.
Additional information

We should add client-side validation to prevent users from entering negative values for age. This can be achieved using HTML5 input attributes or JavaScript validation.

🥦 Browser
Google Chrome
👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read and understood the rules in the Contributing Guidelines

Are you willing to work on this issue ?
Yes I am willing to submit a PR!

Hi @varundhand i would like to work on this issue?could u please assign to me

Hi @varundhand  I have worked on this tech stack and i can fix this issue within a day so please assign me this issue

Good spot @varundhand assigned it to you, make sure you follow the rules 👉🏽here👈🏽, else your PR will not be accepted and will be closed. Good Luck

Hi @Dun-sin ,
I hope this message finds you well. I'm currently experiencing an issue with the local setup of the project and I could use some guidance.
I've followed the installation instructions outlined in the README file, and everything seems to be in order. However, when I attempt to login locally, I encounter an "Invalid callback URL" error. The specific error message is:

I believe there might be an issue with my Kinde setup, causing this problem.
And these are my Kinde app credentials:

I would greatly appreciate your assistance in troubleshooting this issue.
Thank you in advance for your help!
Best regards,
varundhand

Hi @Dun-sin ,
I hope this message finds you well. I'm currently experiencing an issue with the local setup of the project and I could use some guidance.
I've followed the installation instructions outlined in the README file, and everything seems to be in order. However, when I attempt to login locally, I encounter an "Invalid callback URL" error. The specific error message is:

I believe there might be an issue with my Kinde setup, causing this problem.
And these are my Kinde app credentials:

I would greatly appreciate your assistance in troubleshooting this issue.
Thank you in advance for your help!
Best regards,
varundhand

You can text normally and not talk like it's a formal letter, drop the chatGPT it's hard for me to read everything when you are just asking a simple question.
It should be the localhost/login.
Did you read the docs for how to implement it, just changing the name won't do, read their docs and use the necessary keys.
I can't help you past that I'm off the grid.


2.13.0 is the latest version on cocoapods

Let me know if you need help fixing anything with the cocoapods release project. I can try to help.

Yes, please.  We are spread thin and unable to keep on top of cocoa pods support

I'm happy to pick this up... I ran pod spec lint and it flagged up warnings about unguarded availability checks and after adding CLANG_WARN_UNGUARDED_AVAILABILITY to TwitterImagePipeline.xcconfig I got the 9 compile errors to show up.
All these errors relate to use of new APIs introduced in iOS 10 so @NSProgrammer would you prefer that I dig into these and look to try and keep iOS 8 and 9 support or can we just bump the minimum iOS deployment target to iOS 10?
Given that Xcode 12 drops iOS 8 support I think it'll probably be better (and easier) to just support iOS 10+ as we'd have to consider upping it to iOS 9 anyway.. I'm not sure what the official stance on this is though

We have an upcoming release for TIP that will also drop iOS 8 & 9 support.  Let’s wait for that to land.  Hopefully that will make it easier.

Awesome, that sounds like a good plan.
Please let me know if there is anything else that you need help with in terms of getting a CocoaPod's release out when the time comes, thanks!

v2.24.0 has been released.  If you are able to update the CocoaPod's for TIP, that would be very welcomed.  Thanks!

So I listed the spec on master and all looks good:
$ pod spec lint

 -> TwitterImagePipeline (2.24.0)
    - NOTE  | [TwitterImagePipeline/Default] xcodebuild:  note: Using new build system
    - NOTE  | [TwitterImagePipeline/Default] xcodebuild:  note: Building targets in parallel
    - NOTE  | [TwitterImagePipeline/Default] xcodebuild:  note: Planning build
    - NOTE  | [TwitterImagePipeline/Default] xcodebuild:  note: Constructing build description
    - NOTE  | [iOS] xcodebuild:  note: Build description constructed in 0.223138173s
    - NOTE  | [iOS] xcodebuild:  note: Using build description '9985b1ca1524a24f7429e027bb521fa3'
    - NOTE  | [TwitterImagePipeline/Default] xcodebuild:  note: Using eager compilation
    - NOTE  | [TwitterImagePipeline/Default] xcodebuild:  warning: Skipping code signing because the target does not have an Info.plist file and one is not being generated automatically. (in target 'App' from project 'App')
    - NOTE  | [iOS] [TwitterImagePipeline/Default] xcodebuild:  note: Build description constructed in 0.203846421s
    - NOTE  | [iOS] [TwitterImagePipeline/Default] xcodebuild:  note: Using build description 'b801f9deee45ce8f33e6164b19105d12'

Analyzed 1 podspec.

TwitterImagePipeline.podspec passed validation.

Only you (nsprogrammer [at] nsprogrammer [dot] com) or Twitter (cocoapods-owner [at] twitter [dot] com) are listed as owners of this library so you'll have to be the one who runs pod trunk publish... It should be as simple as that I think 🙏

OK, should be done.


Not working after updating Cerbo Firmware

Version
0.6.3
VenusOS version
v3.4.2
Does this issue occur during installation or at runtime ?
Runtime
Is there a Discussion thread for this issue? Please paste a link.
No response
What is the issue you are actually experiencing?
I updated my Cerbo to the version v3.4.2. Afterwards my small additional inverter integrated via mqtt did not show up in the Venus console. I run the setup.sh again as noted in the readme file. Then the inverter appeared in the console but without any values. After troubleshooting I decided to uninstall dbus-mqtt-devices 0.6.3 and installed 0.8.0. Now everything is working again without issues.
Just for your information. System is working again.
Thanks for your work!!!
If this is not installation related, what does the status payload look like that you are publishing to the driver to register your device?
No response
Relevant log output
I could not identify any related message in the log before I updated to 0.8.0.

Glad you got the latest version working.


eza v0.19.1 missing

It seems that there is v.0.19.1 of eza available: https://github.com/eza-community/eza/releases/tag/v0.19.1
could you please add it to the releases of cargo-quickinstall please?

Thanks, in cargo-quickinstall we uses a telemetry to decide what crate to built.
Our telemetry is down (reported in #268), so that's probably why it's not building it.
I'd recommend you to try cargo-binstall, which should be able to install from official github release artifacts @lwiechec

@NobodyXu thanks for the info!


Terminal unresponsive in Production

Describe the bug
Terminal unresponsive in Production. Commands can be typed in but when pressed enter, they are disappeared with no output. The cursor is as below.
This is deployed on AWS ECS.
Screenshots

Can confirm that the DISABLE_TERMINAL variable is not set

@dumim How did you deploy Mage? Do you use Nginx?

We have deployed Mage in production using an ECS cluster setup as recommended for AWS (using Terraform).

We have deployed Mage in production using an ECS cluster setup as recommended for AWS (using Terraform). We do not use Nginx

can you check whether your websocket connection has any errors?

We have deployed Mage in production using an ECS cluster setup as recommended for AWS (using Terraform). We do not use Nginx

can you check whether your websocket connection has any errors?

They are returned as finished wit no response

We have deployed Mage in production using an ECS cluster setup as recommended for AWS (using Terraform). We do not use Nginx

can you check whether your websocket connection has any errors?

They are returned as finished wit no response

Do you want to join our slack channel? I can help you debug via a slack huddle call. https://www.mage.ai/chat

The issue was with the configuration of the Pomerium server that sat in front of Mage for authentication, which was blocking the websocket connections
Had to add allow_websockets: true to the policy as per https://github.com/pomerium/pomerium/issues/412
Not a mage issue.


The server_facts_hash method should allow overrides

There are various times when you need to be able to test against different server versions, particularly with the changes in puppet 5.4.1+.
Users should be able to use let(:server_facts) to set their own hash attributes similarly to let(:facts)

It's not just to test against different versions that one might want to override entries in the $server_facts hash.  It came up in the comments thread of #643 that one might want to override the servername, and although one vocal commenter seemed not to appreciate the importance, such an override is indeed necessary for testing some important scenarios.
One should first recognize that in the real world, the $servername (== $facts[servername]) can indeed plausibly differ from $server_facts[servername], because the client provides the former and the server provides the latter.  Perhaps, then, the client is configured to use one of the server's aliases to contact it, whereas the server_facts gives the server's idea of its own name.  Alternatively, the client might try to inject falsified data.  One would like to be able to test such scenarios to ensure that classes sensitive to such issues handle them in the desired way, but that does not seem presently to be possible.

I, for one, configure Puppet with Puppet because, you know, automation and things: https://github.com/simp/pupmod-simp-pupmod/blob/master/spec/classes/master_spec.rb#L19


Document clearly what is the purpose and use case of {to|from}_{serialized|raw|dict} methods added recently

Classes like SerializableContainer solve genuine issues, but in an unusual way that's new for tmt developers. It may be unclear and puzzling why so many new methods were added, and what's their purpose and use cases.

@psss I added this issue to 1.18 milestone right away - while working on links, I noticed how these methods start to spread, and I think you'd appreciate nice documentation :)
Any tips for the best place? I'd like to add a paragraph or two, something along the lines "if you see in code: * from_raw() ..., * from_serialized() ..., etc.". A docstring of tmt.base or tmt.utils? Or https://tmt.readthedocs.io/en/latest/contribute.html#introduction which now contains the recent guidance WRT error messages?

Probably does not make sense to repeat the same info in each and every docstring. What about adding a new section around here?

https://tmt.readthedocs.io/en/latest/classes.html#attributes

This page already contains some kind of hints for developers.


"value out of range" error when writing PNG

I'm new to Nim and trying out nimPNG.    (Values up to 10, in fact)
#grey[i] = 0.char; i += 1
#grey[i] = 0.char; i += 1
#grey[i] = 0.char; i += 1
#grey[i] = 0.char; i += 1
#grey[i] = 0.char; i += 1

if not savePNG("/tmp/out.png", grey, LCT_GREY, 8, width, height):
    echo "Could not write to /tmp/out.png"

This code gives a "value out of range: -25" message printed out, and savePNG() returns false.  The strange thing is that any values greater than 10, I think, for the bottom row will give some form of this error (the actual number complained about varies).  I also tried LCT_RGB with the string 3 times as large, writing the value 3 times each time, and got exactly the same error.  I also couldn't find the error string anywhere in the nimPNG code, so I'm not sure if this is a bug in nimPNG or Nim 1.0 or my code.  I'd assume my code, but I can't figure out what would be wrong.  Especially since the following code works:
for i in 0 ..< width * height:
    grey[i] = 255.char
if not savePNG("/tmp/out.png", grey, LCT_GREY, 8, width, height):
    echo "Could not write to /tmp/out.png"

fixed


Explicit cast to HANDLE for Read/WriteConsoleW calls

We have a custom build of Python 2.7 which is built against MSVC 14. We found that Click's windows console functionality was degraded after making the switch the MSVC 14, hitting that explicit raise OSError  in _WindowsConsoleWriter.write() with errno 6 (invalid file handle).
I run into the error using click.echo(), just for some cases, but it seems like you hit one of the bad cases and it ruins the rest of the cases. For example, here are the results I get for individual calls:
click.echo('')  # Fails
click.echo('f') # Succeeds

but run the two in sequence and they both fail:
import click
for s in ['', 'f']:
    try:
        click.echo(s)
        print "Pass", repr(s)
    except OSError:
        print "Fail", repr(s)

My output:
Fail ''
f
Fail 'f'

I may be a very remote corner case, since the MSVC 9 build of Python 2.7.16 and the MSVC 14 build of Python 3.6.8 both work fine.
Stack for reference:
Traceback (most recent call last):
  File "script.py", line 10, in <module>
    click.echo(s)
  File "utils.py", line 260, in echo
    file.write(message)
  File "_compat.py", line 628, in _safe_write
    return _write(s)
  File "ansitowin32.py", line 40, in write
    self.__convertor.write(text)
  File "ansitowin32.py", line 141, in write
    self.write_and_convert(text)
  File "ansitowin32.py", line 169, in write_and_convert
    self.write_plain_text(text, cursor, len(text))
  File "ansitowin32.py", line 175, in write_plain_text
    self.wrapped.flush()
  File "_winconsole.py", line 164, in write
    raise OSError(self._get_error_message(GetLastError()))
exceptions.OSError, Windows error 6

What is HANDLE and why does it fix this?

It's a Windows API type which is just a void pointer. The docs for WriteConsoleW/ReadConsoleW say the first argument should be a HANDLE.
If you step through the code self.handle is an int, which ctypes converts to a signed C int if I read the code right, and converting int to void* didn't seem like a safe cast to me. Most of the other args which actually get used are already correctly wrapped in a ctypes type, and error number 6 is just "invalid handle," so it seemed like a good thing to try. And it happened to fix my problem.
As for why or if it causes the specific behavior I ran into, I'm less sure about. I could wave my hands and say "undefined behavior" but I was hoping someone in the community could say with more conviction.

Does this still work for the older builds that didn't demonstrate the issue before? Sorry for the questions, I am not a Windows dev :weary:

These are the builds that I tried, and they all seem to work with this patch:

Unofficial MSVC 14 Python 2.7 build (the one that exhibits the problem)
Unofficial MSVC 10 Python 2.7 build
Unofficial MSVC 14 Python 3.6 build
Official Python 2.7 build
Official Python 3.7 build


Global configuration could manage global number of threads

XGBoost has a global configuration state:
https://xgboost.readthedocs.io/en/stable/parameter.html#global-configuration
Some places in the code try to retrieve a global number of threads, such as here:
https://github.com/dmlc/xgboost/blob/4dfbe2a8935ca45321e906ab8ee62b5d592d1fe8/src/gbm/gbtree_model.cc#L112
The default number of threads is determined from OpenMP, which is a reasonable thing to do:
https://github.com/dmlc/xgboost/blob/4dfbe2a8935ca45321e906ab8ee62b5d592d1fe8/src/common/threading_utils.cc#L100
But being a global configuration, would be more logical to allow users to modify it through the functions that are named as such.
It would make things better for the R interface, since currently there are workarounds like these which result in degraded performance for users:
https://github.com/dmlc/xgboost/blob/4dfbe2a8935ca45321e906ab8ee62b5d592d1fe8/src/gbm/gbtree_model.cc#L120
If it were a global option, should be possible to adopt a mechanism like LightGBM's which changes the number of configured threads before tests and examples, but doesn't have the lines that change the threads in the user-visible part of the examples, so that it executes with reduced threads on CRAN but doesn't suggest doing so to the user.
Example:
https://github.com/microsoft/LightGBM/blob/252828fd86627d7405021c3377534d6a8239dd69/R-package/R/lgb.Dataset.R#L783

After working on distributed systems, I'm a bit afraid of this global variable, it's global and it's thread local. This results in the fact that it needs special handling for distributed systems, and it fails without possible fix when user launches a new thread without setting the value again.

After working on distributed systems, I'm a bit afraid of this global variable, it's global and it's thread local. This results in the fact that it needs special handling for distributed systems, and it fails without possible fix when user launches a new thread without setting the value again.

But a failure here would be no issue, since then it could just default back to OMP's config.
The idea would be to use this as a means to override the OMP config, so that it would check for this first, and then if it's not set, would look at the OMP config.

since then it could just default back to OMP's config.

Depending on your perspective, this change of behavior can be considered a bug. In general, global variable are not very desirable.

The idea would be to use this as a means to override the OMP config

I think booster local variable is fine. It's a bit inconvenient when writing tests and examples, but it's really a CRAN issue instead of a real world problem. We can workaround it by setting global OMP environment variable before running any test, XGBoost respects those variables. https://github.com/dmlc/xgboost/blob/4dfbe2a8935ca45321e906ab8ee62b5d592d1fe8/tests/python/test_openmp.py#L83

I think we can close this now that https://github.com/dmlc/xgboost/pull/10028 is merged.


feat: use v3 emote and event api

Pull request checklist:

[x] CHANGELOG.md was updated, if applicable

Description
I'm still waiting on changes in upstream chatterino. I will update this branch once the upstream PR is merged/ready.
TODO

[x] Unsubscribe from emotesets and users
[x] Wait for chatterino/Chatterino2#4002

Time to update this

Can you get upstream updated in this PR?

I'll try to make this PR be on the same state as upstream. I don't know if I can just merge the changes.

Figured as much, things have diverged a fair bit. Thanks for the continued work, I'm sure chatterino users are very grateful for your contributions

There's on change that might be bad for users:
The setting showUnlistedEmotes was renamed to showUnlistedSevenTVEmotes. So anyone that had it checked before must check it again.

Would it be possible to add a case to handle both properties? For purpose of backwards compatibility and also making sure configs work with either fork or upstream

Looks good to me thank you

The problem is this branch rebases master.


Linux Support?

Linker error occurrs on Ubuntu 20.04, Swift 5.3 in resolving packages.
.build/checkouts/swift-atomics/Sources/_AtomicsShims/include/_AtomicsShims.h:325: error: undefined reference to '__atomic_load'
.build/checkouts/swift-atomics/Sources/_AtomicsShims/include/_AtomicsShims.h:325: error: undefined reference to '__atomic_load'
.build/checkouts/swift-atomics/Sources/_AtomicsShims/include/_AtomicsShims.h:325: error: undefined reference to '__atomic_compare_exchange'
.build/checkouts/swift-atomics/Sources/_AtomicsShims/include/_AtomicsShims.h:325: error: undefined reference to '__atomic_load'
.build/checkouts/swift-atomics/Sources/_AtomicsShims/include/_AtomicsShims.h:325: error: undefined reference to '__atomic_load'
.build/checkouts/swift-atomics/Sources/_AtomicsShims/include/_AtomicsShims.h:325: error: undefined reference to '__atomic_is_lock_free'
.build/checkouts/swift-atomics/Sources/_AtomicsShims/include/_AtomicsShims.h:325: error: undefined reference to '__atomic_is_lock_free'
.build/checkouts/swift-atomics/Sources/_AtomicsShims/include/_AtomicsShims.h:325: error: undefined reference to '__atomic_is_lock_free'
.build/checkouts/swift-atomics/Sources/_AtomicsShims/include/_AtomicsShims.h:325: error: undefined reference to '__atomic_compare_exchange'
.build/checkouts/swift-atomics/Sources/_AtomicsShims/include/_AtomicsShims.h:325: error: undefined reference to '__atomic_compare_exchange'
.build/checkouts/swift-atomics/Sources/_AtomicsShims/include/_AtomicsShims.h:325: error: undefined reference to '__atomic_compare_exchange'
.build/checkouts/swift-atomics/Sources/_AtomicsShims/include/_AtomicsShims.h:325: error: undefined reference to '__atomic_is_lock_free'
../.build/checkouts/swift-atomics/Sources/Atomics/autogenerated/IntegerConformances.swift:0: error: undefined reference to '__atomic_store'
../.build/checkouts/swift-atomics/Sources/Atomics/autogenerated/IntegerConformances.swift:0: error: undefined reference to '__atomic_exchange'
clang-10: error: linker command failed with exit code 1 (use -v to see invocation)
<unknown>:0: error: link command failed with exit code 1 (use -v to see invocation)

There are multiple problems, most of which have a relatively straightforward resolution:

The package definition is lacking a conditional link dependency on libatomics.so on Linux. (Triggered by support for double-wide atomics -- see point 4 below.)
One of the test cases has grown to >1600 tests, which triggers a compiler timeout while type checking the test registry. (Oops)
PR #9 reintroduced a compiler issue (https://bugs.swift.org/browse/SR-13708).
Double-wide atomics need to be disabled on Linux/x86_64 unless cmpxchg16b support is manually enabled via -Xcc -mcx16. Declaring this to the package manager is tricky, but there are some potential workarounds. This does mean that for most Linux users, double-wide atomics won't be available.

Re: item 4
I thought that https://github.com/apple/swift/pull/31053 meant that Swift on Linux now requires cx16. If that is the case, shouldn't swift-pm be fixed to always emit the flag?

Oh, excellent catch! I agree completely -- SPM packages ought to build using the same target architecture as the stdlib itself.
I wonder if the stdlib really needs core2, or it could work with the narrower cx16 feature. (IIUC, core2 also enables SSE3/SSSE3 and SAHF. On the other hand, core2 conveniently matches the default arch on Darwin, and I expect it might be a universal baseline at this point, including for AMD processors.) (cc @drexin)
In the short term, we should probably update the instructions in the README to ask people to manually add -mcore2 rather than -mcx16 -- if only for consistency's sake.
If/when a version of SPM or swiftc ships that configures the target correctly, Package.swift will need to somehow figure out whether it is being compiled with the right SPM version, and if so, then automatically define the ENABLE_DOUBLEWIDE_ATOMICS conditional flag. (I think the only way to do this right now is by basing the decision on an #if swift() or if compiler() conditional.)


Do we still need the stringcase dependency?

you introduced it in https://github.com/Hochfrequenz/BO4E-python/pull/27/commits/123b8e738b8ad074af009e143db0f4710eb032cc
https://github.com/Hochfrequenz/BO4E-python/blob/21cceaff2133a9c2ea59d3cc1d41f8c8b5ea1e93/requirements.in#L5

nope not needed anymore. Fixed in #393


feature request: Add darvin_arm64 as architecture to output artifacts of this github project (executable binary)

Current situation:

Currently only darwin_x86_64 is built (for mac)
M1/M2 Mac* use darwin_arm64 as architecture
presumeably it is not so difficult to just add this to the go builder. But I am not a Go dude and did not find a solution in the last 4 minutes ;).

Let me know whether this is a 2 mins thing, whether it is super complicated and whether maybe I should look for myself :).

It shouldn't be hard to do. I will try to address this today

New release https://github.com/mpostument/grafana-sync/releases/tag/1.4.10 contain arm build


Want to add a contact us page

Hi, I want to add a contact us page in the website which will be more user friendly.So please assign this issue to me under GSSOC'23

Already assigned issue, please check the issue section before raising any new issues.


⚠ Bofrost.de has degraded performance

In a33e38d, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2340 ms

Resolved: Bofrost.de performance has improved in 9a4d62e after 1 hour, 20 minutes.


Dtportal 19264 static analysis php 74

Zend Framework 1 reaches its End of Life (EOL) and is no longer maintained.
No Pull Requests will be accepted.
https://framework.zend.com/blog/2016-06-28-zf1-eol.html
This PR is composed of the following changes:

Pulling shardj/zf1-future changes, purported to be PHP 7.4 compatible.
Adding alerts for remaining PHP 7.4 compatibility issues in order to detect whether we use any of that framework code in our applications.
Adding PHP 7.4 to the travis build.

Beyond pulling the most recent changes from shardj, the diff is composed only of https://github.com/cloudvox/zf1-future/compare/9c6672cd500a028088df4a9127662c9dbc497890..a3942189468508c0cc1d25130c77a7c2d4bc0cc3

@abhishek-algoworks please try using this branch in your attempts at getting the sandbox to use PHP 7.4, and let me know how it goes, in case i can help get that working.

@gfaza This is great. I just ran an updated unit test on php7.4 and no deprecation warnings were thrown. Earlier it would just give
[PHP Deprecated:  array_key_exists(): Using array_key_exists() on objects is deprecated. Use isset() or property_exists() instead in /docker/Organic/vendor/shardj/zf1-future/library/Zend/Registry.php on line 206]

Draft PR https://github.com/cloudvox/Organic/pull/2199

@yadnad, if you please!

@mattcar do you remember what the process is for this library?  merge into develop then bump all composer.json files that reference it?  if so, who bumps them?

@gfaza that is indeed the process. I can, you can, doesn't matter to me.

@mattcar could you point me at the steps you'd use to bump the composer.json files, after this PR is merged?  I'm guessing a sequence of

git checkout develop ; git pull ; git checkout -b DTPORTAL-19264-bumping_composer_json
some variation on /usr/bin/php ./bin/composer.phar update shardj/zf1-future ... please correct me as necessary ... I'm not confident that old cmd will work with the dockerized sandbox ...
git commit -am 'DTPORTAL-19264 bumping composer.json' ; git push origin HEAD

then

Merge change to secure-dev (or equivalent) and deploy to DEV.
Assuming all's well, then open a PR for the particular app

i'll gladly take you up on the offer @mattcar, to bump those consuming apps, sir! 🙏

I've found this dependency referenced in the following repositories:

integrations-adobe-media-optimizer
integrations-marin
Ifbyphone
CRBN
Organic

@gfaza @yadnad can you confirm if this update needs to be applied to all or a subset of the above?

@gfaza this is what has worked for me:
./bin/composer.phar --no-progress --no-interaction --no-ansi --verbose update shardj/zf1-future

I've found this dependency referenced in the following repositories:

integrations-adobe-media-optimizer
integrations-marin
Ifbyphone
CRBN
Organic

@gfaza @yadnad can you confirm if this update needs to be applied to all or a subset of the above?

that's all I'm seeing in Github as well

I've found this dependency referenced in the following repositories:

integrations-adobe-media-optimizer
integrations-marin
Ifbyphone
CRBN
Organic

@gfaza @yadnad can you confirm if this update needs to be applied to all or a subset of the above?

should be to all, though i'm only hearing of those integrations-* apps for the first time.  perhaps to Ifbyphone prior than to CRBN and Organic?  unsure - but the order may matter there.

@gfaza @yadnad @mikeheidenreich i think we'll need to cut individual cards for each repo update and ensure tester availability before proceeding


⚠ ClamAV has degraded performance

In bf03ad7, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 228 ms

Resolved: ClamAV performance has improved in 9aa3db8 after 7 minutes.


In Server-Sent Events responses, there is no token usage.

In Server-Sent Events responses, there is no token usage.

Hello @lishiyu , thanks for raising the issue.
This will be a low priority item. The purpose of this repo is to help you quickly explore the Bedrock models capability, not all OpenAI API features will be supported.
The recommendation for you is to use Bedrock SDK instead if you need to track the token usage. Or you can do full customization on your own based on this.

Hello @lishiyu The usage info is now added to the streaming response. Please follow the upgrade steps to try the new features.
Make sure you have added "stream_options": {"include_usage": true} parameter to the request as per OpenAI doc.

Close for no updates


What is the acceptable format for domains?

Trying to find out how the syntax works - does an entry of acme.com also block spam.acme.com?  I checked this page and another
Neither explain the coverage - unless I missed it.
If it does not block subdomains, we need this feature!

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

ID: bbb3daf4-fa28-88d9-4543-89d8ff5ead5d
Version Independent ID: bbb3daf4-fa28-88d9-4543-89d8ff5ead5d
Content: Manage allows and blocks in the Tenant Allow/Block List
Content Source: microsoft-365/security/office-365-security/tenant-allow-block-list-about.md
Service: microsoft-365-security
Sub-service: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

@mntorq, I don't think subdomains are blocked.
Checking

@mntorq subdomains are not blocked. We are working to add top-level(TLD) domain blocking in Tenant allow block list under domains & email addresses.

Thanks for confirming.  I know in other areas (such as transport rules) subdomains are automatically included - so while it wasn't explicitly stated it wasn't - it could have been assumed that it was (but clearly wasn't working).   I find these details to be helpful in the documentation.
I appreciate @chrisda and @dhairyya for your responses!

@mntorq, I added the following text to the big note at the top of  Create block entries for domains and email addresses:

Currently, subdomains of the specified domain aren't blocked. For example, if you create a block entry for email from contoso.com, mail from marketing.contoso.com isn't also blocked. You need to create a separate block entry for marketing.contoso.com.

Perfect - thank you for clarifying.


Remove deprecated fields, like "difficulty", from responses

We have some responses to some requests, such as the "difficulty":1 value we include in a "store" request (and possibly others).  We should remove them for the next mandatory upgrade as I think enough time has passed since Session stopped using them.

Fixed in #469


Property torchState of MobileScannerController class disappeared in v5.0.0

Hello! I did not found any explanation why this property disappeared. Last presence in v4.0.1
But TorchState enum still exists. What can be used instead now?
Android version.

From version 5.0.0 onwards, the MobileScannerController now implements ValueNotifier.
The torch state is present in the value of the controller.
final MobileScannerController controller = MobileScannerController();

final TorchState torchState = controller.value.torchState;

This was mentioned in the changelog, although we did not write a migration guide to move from v4 to v5.

I added an issue for writing a proper migration guide in #1168 so that issues like this can be addressed with docs instead. Thank you for reporting this issue!

Ok. Thank you for clarifications. Now I see that this was mentioned under the sentence "The various ValueNotifiers for the camera state have been removed. Use the value of the MobileScannerController instead". I just did not realize that tourchState goes there. Finally did the migration. Thanks!


filters step 1

Write the interpreter side of filters.
Make a python side command called "make-filter" that exposes a named variable bound to a filter constraint.
Write some unit tested code that uses filters to shape output transformed data

Done.  look at the 0.7.3 release with commands of only-outliers,  search_col, and search


Do updateRecommended in Supabase

This seems like a very big source of Firestore reads and it also is slow as shit (the loading process takes longer than half an hour) so let's use Supabase to make that not the case.

@jahooma FYI -- I easily made the matrix factorization stuff 5-10x faster, it could probably be faster but it would get uglier, so I think this is fine for now.


On Option change, gauge label not updating

When I change options parameters, my needle value is getting changed properly. However, my rangeLabel is not updating.
I have a list of users(say) having balance and total amount which I am showing in gauge representation. When I select a particular user, that users balances should be shown in gauge. Needle change is working, however not the rangeLabel. In console.log my option object is updated but not the Gauge value.

Just published an updated version of the library. You can get it via npm i<EMAIL_ADDRESS>Closing the issue for now. Let us know if it appears again.


'Status' positioning error. Wrong line.

Hi,
My problem is simple i hope. I guess status icon in a wrong place. I would like to see status icon parallel to the wrong command. How i can do this?
Screenshot
My .zshrc
POWERLEVEL9K_MODE='nerdfont-complete'

POWERLEVEL9K_LEFT_PROMPT_ELEMENTS=(dir dir_writable vcs)
POWERLEVEL9K_RIGHT_PROMPT_ELEMENTS=(status)
POWERLEVEL9K_RPROMPT_ON_NEWLINE=false
POWERLEVEL9K_SHORTEN_DIR_LENGTH=2
POWERLEVEL9K_STATUS_VERBOSE=false
POWERLEVEL9K_VCS_GIT_ICON='\uE1AA'
POWERLEVEL9K_VCS_GIT_GITHUB_ICON='\uE1AA'

Positions are relative to the prompt, which will typically start on a new line following any command unless that command fails to end in a newline. That means that in order to get the prompt on the same line as the command, it would need to be moved backwards, or up a line. Technically this is possible on some terminals, according to this StackOverflow answer: https://stackoverflow.com/a/4873071
You would need to print the \e[A character before the right prompt, but that seems to break P9K in a few ways. I threw together this quick custom segment to test it out:
zsh_back_line(){
  printf '\e[A'
}
POWERLEVEL9K_CUSTOM_BACK_LINE='zsh_back_line'
POWERLEVEL9K_LEFT_PROMPT_ELEMENTS=(os_icon dir vcs)
POWERLEVEL9K_RIGHT_PROMPT_ELEMENTS=(custom_back_line status go_version time)

And the results, well... :grimacing:

Technically, this would you require to move up and down, but the problem is that there is no way to know how big the output is between command lines, so it could be 2 lines or 200...


Funny annotation

@mira-mayhem commented on Fri Feb 14 2020
Hi!
You might want to expand on issue on the strange annotation for the setting version below: "[...)" is not very common. And I, at least, would be interested in an explanation.
{
"version": "2.0",
"extensionBundle": {
"id": "Microsoft.Azure.Functions.ExtensionBundle",
"version": "[1.*, 2.0.0)"
}
}
regards,
Ola L Martins

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

ID: b2646ad4-34b7-fc5e-dcc9-d53a061df394
Version Independent ID: bbff4fda-24e4-2b62-24b9-94cda5f53bed
Content: Work with Azure Functions Core Tools
Content Source: articles/azure-functions/functions-run-local.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@srvbpigh commented on Fri Feb 14 2020
Hello, @mira-mayhem
Thank you for your feedback.
We are actively reviewing your comments and will get back to you soon.
Kind regards,
Microsoft DOCS International Team

Hi Team, could you please check issue from user?
Kind regards,
Microsoft DOCS International Team

@mira-mayhem Thank you for your feedback! We will review and update as appropriate.
cc: @ticianaciminari

@mira-mayhem Notation derives from NuGet package versioning, defines a range called "Mixed inclusive minimum and exclusive maximum version". Please see: Version ranges and wildcards
Since this channel is more for driving improvements towards MS Docs and we haven't determined any changes after reviewing this feedback, we will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Enable processor reloading in shells

Account for reloading in shell menus and reload (i.e., create new processors) when requested.
Nothing here yet accounts for locations of files.  It is likely that the exact same rules will be used upon reload.

Will do.

Here in in the just pushed commits is a second draft.  I'm not sure how the directory to support reading (reloading) rules from files should be specified.  In NumericEntityRecognizerShell, I just made it a command line argument.  Perhaps there should be a standard place like the current directory specified in a config file somewhere.
For the VariableShell and VariableProcessor in habitus, the same question applies.  There I also wonder which parts of the VariableProcessor should be reloaded.  There is a lexiconNer that might be changed and also an ExtractorEngine which relies on local files.  In VariableProcessor.apply there is a resource_dir which might serve as the hard-coded directory for reading those rules.  That directory only exists during development and would disappear upon deployment.  I'm not sure when the app will be used.

It looks like I still need to get the LexiconNER to load from files.  Those seem to only load from resources so far.  habitus is wanting that capability.


POLARGAP survey -- not all radargrams have polarised_chirp_PPVV_data

While trying to track down #13, I discovered that the viewer doesn't work with some of the radar data from BAS's POLARGAP survey.
It appears that the file format changed partway through, and the field that I had selected for plotting isn't available.
In my spot checking, P05 - P09 are OK, while P29 and P35 raise the KeyError.

From the netcdf metadata: "from flight line P24- onwards, there is no polarised data for this survey"
And, in place of the polarised variables "polarised_chirp_{PPVV, SSHH}_data", they provide "chirp_data"


New Extension - Graph Slider Data

I would love the ability to chart slider data pulled from references to a page in order to better understand patterns in my data.
Use Case: I have a Smartblock that I use as the base of my daily page. I have 3 sliders I use as mood trackers. Each slider is nested under a link like [[How are you feeling?]]. I would like to place a chart on the [[How are you feeling?]] page that graphs the results of the referenced sliders, by date. When you mouse over a day's data point, I'd like the contents of the sub-block to be displayed. I use the sub-block to explain why I rated myself as I did.
Example Data Structure:

I'd like to attach a $50 bounty to this.

Instead of as a new extension, how do you feel about this being an enhancement on charts?
https://roamjs.com/docs/extensions/charts

If that’s where you think it’s best, that’s fine by me.