Support alternate outputPath

When using certain Webpack configurations, particularly the one used by Create-React-App, compiled JS files and their associated mappings are not stored in the base build folder. Instead, they're stored in a nested build/static/js folder. As the readdirSync function is not recursive, this means the JS files will not be found using the default folder. This PR adds the ability to set the outputPath in the plugin configuration.
I did not touch the Rollup plugin, as I do not use that system and am not familiar enough with it to make changes or even know if it needs the same change.

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

@zackman0010 thank you for the addition! I had this in an earlier iteration but it must have gotten lost. I will take care of getting this into rollup. much appreciated.


docs(configuration): fix heading level

infrastructureLogging should be heading 2 instead of heading 3.
Before

After

Would be good to strip the first part of the sub sub heading if the name is too long, see for example in your second screenshot that you can't really read the real option name.

I can't ignore this CLA bot :D, it seems its resources are too low to process its state correctly or maybe it is using something old to check for statutes.

@montogeek updated.

In this case its fine, but i assume there can be cases where we wont do this for SEO reasons

Thanks!


change package name

What is the purpose of the change
XXXXX
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #3667 into 2.7.1-release will increase coverage by 0.71%.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##             2.7.1-release   #3667      +/-   ##
==================================================
+ Coverage            63.19%   63.9%   +0.71%
  Complexity              71      71
==================================================
  Files                  720     699      -21
  Lines                32600   30757    -1843
  Branches              5230    4989     -241
==================================================
- Hits                 20600   19656     -944
+ Misses                9625    8831     -794
+ Partials              2375    2270     -105

Impacted Files
Coverage Δ
Complexity Δ

.../apache/dubbo/remoting/transport/AbstractPeer.java
63.04% <0%> (-4.35%)
0% <0%> (ø)

...bo/rpc/cluster/support/FailbackClusterInvoker.java
67.21% <0%> (-3.28%)
0% <0%> (ø)

...dubbo/remoting/exchange/support/DefaultFuture.java
72.29% <0%> (-2.71%)
0% <0%> (ø)

...a/org/apache/dubbo/common/json/ParseException.java

...o/rpc/protocol/thrift/DubboClassNameGenerator.java

.../java/org/apache/dubbo/common/json/JSONObject.java

.../java/org/apache/dubbo/common/json/JSONWriter.java

.../apache/dubbo/rpc/protocol/thrift/ThriftUtils.java

.../java/org/apache/dubbo/common/json/JSONReader.java

...bbo/rpc/protocol/thrift/io/InputStreamWrapper.java

... and 20 more

Continue to review full report at Codecov.

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


Replace gunicorn with waitress and fix logging issues

Description
This PR replaces the embedded gunicorn instance with waitress https://docs.pylonsproject.org/projects/waitress/en/latest/#
Gunicorn was not particularly set up to be run in this way. It's largely meant to be used from the command line rather than embedded in application. Its forking model just didn't well in qcfractal and it conflicted with other multiprocessing stuff. Also, logging was a pain.
So let's try waitress instead. It is much simpler, and much more amenable to embedding in a python application.
I hope this also fixes the spurious 502 Bad Gateway errors, although I wouldn't be surprised if it doesn't.
Waitress only works with one process, so the num_workers parameter in the config is no longer used.
Also as part of this PR, logging will work through a queue. This will help in the multi-process environment, and allow for using TimedRotatingFileHandler and handlers from python.
Lastly, this also quiets some logging when doing admin stuff from the CLI (like modifying users). It enables using -v and -vv
to increase verbosity
Changelog description
Replace gunicorn with waitress and fix logging issues
Allow increasing verbosity on the CLI with  -v and -vv on the cli
Status

[x] Code base linted
[x] Ready to go

Tested on the production server and everything seems reasonable


Checker build script: improve dependency handling, support out of source builds.

Also fixes #40

Download:
Environment: MINGW=C:\Qt\Tools\mingw530_32, QTDIR=C:\Qt\5.11\mingw53_32, QT=5.11
trik-studio-master_qt5.11.zip
Environment: MINGW=C:\Qt\Tools\mingw492_32, QTDIR=C:\Qt\5.6\mingw49_32, QT=5.6
trik-studio-master_qt5.6.zip


Update Go version to 1.22

By some reason, linter crashes. Putting this on hold until they fix the issues.


[Relay][Frontend] Span filling common API

Expose and add span attribute of Expr-derived types from C++ to Python
Add common API of span filling
Add test cases of span filling
Add function to control whether to fill span via environment variable

Hi community,
This is the very first PR of the RFC, TVM Explorer Infrastructure. We spend a while to extract the code blocks, and verify them after rebasing. Feel free to give us and comment. Thank you! :D
For your reference, here is the related links.
PreRFC in forum
RFC in git
Tracking issue
@haowhsu-quic, @zack-ch

@FrozenGene @areusch @mbs-octoml
Tag reviewers in the forum thread and previously reverted PR.
It would be a big help to have your advices. Thank you! :D

thanks @chunit-quic and sorry for the delay--i've been out on PTO for a while. i've left some comments here, overall this is a great addition!

Hi @areusch,
Thank you for reviewing!
We have modified several parts based on your previous comments. :D
Take your time (It's thanks giving recently.) We will wait for your reply.


Change set support

Unified some session Management stuff, Added support for changeSet creation and execution, cleaned up some code
Before submitting a pull request, please make sure the following is done:

Fork the repository and create your branch from master.
Run make in the repository root.
If you've fixed a bug or added code that should be tested, add tests!
Ensure the test suite passes (make test).
Format your code accordingly to the .editorconfig rules.
Make sure your code lints (make code-analysis) and builds properly on Travis CI.

Example usage:

Please add some unit tests.


Width of vue-notification on mobile

Hey,
it seems like plugin doesn't take mobile devices into account. When width is set to 500px it is still set to 500px on 360px width mobile.
Is it just my misunderstanding of how should I properly work with plugin?

Same issue.


Argocd v2.3.1

updated argocd dependencies to v2.3.1
upgraded to golang v1.17
removed packr dependency
moved to use applicationset argocd addon as dependency

Will solve #246


Consider adding to firebaseopensource.com

This is a great project and it surely would be helpful for Firebase Developers who use Test Lab. Consider adding it to firebaseopensource.com :)

https://github.com/firebase/firebaseopensource.com/issues/184


DTO refresh enhancements

enhancing the base DTO refresh brought in https://github.com/ManageIQ/manageiq/pull/11579
and the AWS part https://github.com/ManageIQ/manageiq-providers-amazon/pull/52
Comments from @Fryguy
Things I like:
Replacing the Hash interface to EMS Refresh with actual objects. In particular, we are prevented from keeping metadata on collections since we use Arrays inside of the Hashes, and you can't really decorate an Array (unlike a Hash where you can fake it with :_object like keys). In addition, the more we fake those decorations, the more complex the code is to understand, so it's a great time to get away from that.
Things I don't like
The name DTO is too general. Perhaps RefreshDTO or EmsRefreshDTO or EmsRefresh::DTO or something. Not sure. I like the string "DTO", but I feel it needs a qualifier.
I don't think I like the graph definition being spread across the models. I've run into problems with this before with respect to C&U parent definitions and it gets extremely complicated to describe the graph to people because they have to hunt down these definitions. On the other hand, subclassing and pluggability give a provider a nice way to override these values. I think I'd prefer the definition in one place, and perhaps, if necessary, we can provide some sort of interface to it to providers to manipulate it.
Other things
Replacing the custom unique references with [ems_id, ems_ref] or perhaps a new column so it will be [ems_id, ems_refresh_unique_thing]. As discussed, the ems_ref's purpose is to hold the native id of the provider. I don't really like overloading ems_ref with multiple responsibilities, hence the "new column" idea. We may also want to just use uid_ems, whose purpose sort of started out as this "alternate unique reference". You may want to do this first before we complete this PR in order to make this PR simpler.
This PR only implements the DTO strategy for Amazon NetworkManager, which is fine. However, before we move on to alternate ways to leverage the DTO strategy, I think we should spread it out to more managers (e.g. try it for Azure NetworkManager (same manager type), and then try RHEV InfraManager (different manager type))
comments from @kbrock how to change the parser itself
class Parser
    node :availability_zones, :strategy => :local_db_cache_all
    node :cloud_subnet do |n|
      n.manager_ref :address, :cloud_subnet, :network_port
      n.attributes :address, :cloud_subnet, :network_port

      # class CloudSubnet (optional - assumes node name)
      n.parse :get_cloud_subnet
      n.parse_dependencies :availability_zone
      n.parse_dependencies :cloud_networks, :lazy => false
    end

Not sure if always defining parse or making it optional is better.
For the singular nodes, you may want to allow attributes to be used in the node declaration.
While just yielding the actual object from node will be simpler. Also remember you can yield a parser object. this give you more flexibility, more code to test, and allows you to keep dto separate from the dsl.

While the format you explained above would work, this alternate one would as well:
class Parser
    node :cloud_subnet, :parse => :get_cloud_subnet, :klass => 'CloudSubnet' do |n|
      n.manager_ref :address, :cloud_subnet, :network_port
      n.attributes :address, :cloud_subnet, :network_port

      n.parse_dependencies :availability_zone
      n.parse_dependencies :cloud_networks, :lazy => false
    end

Not actual code, but tried to show the notes above in real code
def node name, options = {}
```ruby
  parser_node = create_parser_node(name, options[:strategy])
  # comment 2: allow any node values as attribute:
  options.each do |n, v|
    next if n == :strategy
    parser_node.send(n, v)
  end
  # comment 1:
  parser_node.parse "get_#{name}"
  parser_node.klass name.camelcase
  yield parser_node
end

@kbrock @durandom @Fryguy @agrare ok, I am thinking we will need to separate Node (Dto collection, a prescription for saving) from Remote (remote proxy fetcher of data, which produces data by parsing, or produces other collections on side)
  remote :cloud_subnets do |r|
    r.parse :parse_cloud_subnet, :into => :cloud_subnets
  end

  remote :network_ports do |r|
    r.parse :parse_network_ports, :into => :network_ports, :produces => :cloud_subnet_network_ports
  end

  remote :security_groups do |r|
    r.parse :parse_security_groups, :into => :security_groups
  end

  remote :firewall_rules_egress do |r|
    r.depends_on :security_groups
    r.parse :parse_firewall_rule, :with => ["egress"], :into => :firewall_rules
  end

  remote :firewall_rules_ingress do |r|
    r.depends_on :security_groups
    r.parse :parse_firewall_rule, :with => ["ingress"], :into => :firewall_rules
  end

  remote :ec2_instances do |r|
    r.parse :parse_network_port_inferred_from_instance, :into => :network_ports
    r.parse :parse_floating_ip_inferred_from_instance, :into => :floating_ips
  end

  node :availability_zones, :strategy => :local_db_cache_all

  node :cloud_subnets do |n|
    n.depends_on :cloud_networks
    n.except_attributes :parent_cloud_subnet
  end

  node :cloud_subnets_relationships, :klass => ManageIQ::Providers::Amazon::CloudManager::CloudSubnet do |n|
    n.only_attributes :parent_cloud_subnet
  end

  node :cloud_networks

  node :cloud_subnet_network_ports, :klass => CloudSubnetNetworkPort do |n|
    n.depends_on :cloud_subnets, :network_ports
    n.manager_ref :address, :cloud_subnet, :network_port
  end

  node :security_groups do |n|
    n.depends_on :cloud_networks, :orchestration_stacks
  end

  node :firewall_rules do |n|
    n.depends_on :security_groups,
    n.manager_ref :resource, :source_security_group, :direction, :host_protocol, :port, :end_port, :source_ip_range
  end

I'm not sure if adding a DSL style parser description at this point makes sense, sorry.
My main concern is making the parser part as easy as possible for provider authors.
Hence, I think we should just define one API from MIQ to the Provider and this API is the DtoCollection.
Miq requests DtoCollection -> Provider returns DtoCollection.
How the collection is filled? That is up to the Provider. They could implement a streaming parser, like StAX, or events like SAX.
So in pseudo code:
class Parser
  def parse(dto_collection)
     dto_collection.dependencies.each do |dependency|
        send("parse_#{dependency.name}", dependency)
     end
     dto_collection
  end

  def parse_cloud_subnets(dto_collection)
  end
end

@durandom the main reason of the DSL is not being pretty. :-) We need to build a graph, so we can know e.g. what API requests can be done in threads/parallel. So it's enhancing what ovirt does https://github.com/Ladas/manageiq/blob/e1009e0ee38e2bf9eac0ed85e26736ddbba4047b/gems/pending/ovirt_provider/inventory/ovirt_inventory.rb#L117-L117
They have 2 levels, that they collect in parallel. So we should to extract it to graph in general and connect it to DtoCollections.

@Ladas Can this be closed or is this still planned?

@Fryguy yeah we can close this, @durandom implemented something similar. We are just blocked by dynamic targeted refresh in some providers, so we need to solve that and put the new API to all providers.

Darn. This is going away?

@kbrock yeah, Marcel did a bit different API, but it's a bit more chatty than we would like. So we'll tweak that a bit more.

@kbrock yeah, and what is in ansible now, but we want to change that with @agrare and @slemrmartin , because it is too verbose
https://github.com/Ladas/manageiq-providers-ansible_tower/blob/2be6834ecdbdc5607268f8c3c033ee048f562909/app/models/manageiq/providers/ansible_tower/shared/inventory/persister/automation_manager.rb#L5
https://github.com/Ladas/manageiq/blob/b1e1ba89997872e924da8f16cb8af39c9b9c29ae/app/models/manager_refresh/inventory/automation_manager.rb#L6

@kbrock so ^ is the perister api, https://github.com/ManageIQ/manageiq/issues/13113 is more as parser api, which got implemented partially

yea - the need for modules here and then having the options override stuff suggests that we are not defaulting enough variables or we are passing data around at the wrong granularity


Fix broken links

@danroth27  can we get a fix so local Sphinx builds detect bad links? This is going to be a major problem.

@Rick-Anderson Yes, I made an update to the build script to turn on nick-picky mode, which will catch these errors.

:shipit:


Update resource-files.md

Clarified that the resource limits are based on filename/URL length.

@Padmalathas
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Devin/feature/add burnwire component

It might actually be easier to kill this PR and create a new one that only has what you want.

It might actually be easier to kill this PR and create a new one that only has what you want.

How do I go about this? I have removed the build files and added a git ignore in the most recent commit, but those 5000+ changes are part of the commit history...

I'm guessing you created 5-10 new files, and changed at most 1 or 2 existing files? The easiest thing is to do a new git clone in a different directory and copy over the new files. Make sure you make a new branch before committing, and specifically commit on the files you added/changed. The crazy history is on your current branch, and won't be on the new one.


Fix (#1008) by removing comments from JSON linter.

Fix false linting errors for comments inside of JSON body. This is needed since Bruno already removes comments from the body of the body of JSON requests.
Description
This is to fix #1008 by replacing all comments with spaces for the JSON linter. Had to override the normal JSON linter with one that can replace comments and keep placement of linting errors. There are other JSON linting things that could be done but I will leave them to other commits after this gets merged.
https://github.com/usebruno/bruno/assets/115114979/0c6b3c55-3dd9-461e-9129-96fdfc4a1f37

Libraries added

strip-json-comments (pretty compact and few dependencies)

Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

Thanks for taking care of this @n00o !


DVConstraints test currently fails due to no numpy-stl installation

Description
Numpy stl not found during Azure tests
Steps to reproduce issue
Make a PR
The tests fail on DVConstraints because stl cannot be found
Current behavior
…
Expected behavior
pip install -e .[testing] installs the optional numpy-stl dependency
@nwu63 or someone, do you know why the Azure transition changed the way pygeo is installed on the machine during reg tests?

Yeah, this file is incorrect -- we should change it to pip install .[testing].


commit not consistent with image

Possible bug,
This showed up in my git commit but did not show up in the rowan diff:
 { #category : 'examples' }
 RowanProjectService >> removeProjectNamed: projectName [
 	(Rowan loadedProjectNamed: projectName ifAbsent: [  ])

Image method is this. Note the difference.
removeProjectNamed: projectName
        (Rowan loadedProjectNamed: projectName ifAbsent: [  ])
                ifNotNil: [ :project | Rowan image _removeLoadedProject: project ].

It's possible that I compiled that method change outside of Rowan before we went to Rowan-in-Rowan. Recording just in case.
After I found the discrepancy, I tried compiling the method again with a small change. Rowan found that.
diff code:
(Rowan projectTools diff diffForProjectName: 'Rowan') asString

Okay ... I'm not sure if there is anything to do here ... it sounds like you figured out that you you probably forgot to hit the "compile in Rowan" button, is that correct?

Yes, I believe so. I'll close the issue.


Solidity/Toy Verifier

Are they any plans to implement a solidity verifier?
I think this would significantly boost adoption of risc0 as a plug and play layer 2.
If there is not currently a solidity verifier in the roadmap, our team would be happy to work on/contribute a Solidity implementation if there was more documentation of the zkSTARK protocol used or a toy reference implementation to work off of.
See also #222

Hi, I am currently working on a solidity verifier. I would be happy to collaborate

We are not currently working on a Solidity verifier ourselves due to resource constraints, but would love for there to be one.  In terms of documentation, we have a some of high level stuff on our website, as well as a python implementation of a ZK-Stark for a toy problem and also a minimal stand alone rust implementation of a verifier, which are currently not in the public repo (and also may be slightly out of date).  I'm attaching them here, caveat emptor.
didactic_stark.zip
risc0-verify.tar.gz

We are not currently working on a Solidity verifier ourselves due to resource constraints, but would love for there to be one. In terms of documentation, we have a some of high level stuff on our website, as well as a python implementation of a ZK-Stark for a toy problem and also a minimal stand alone rust implementation of a verifier, which are currently not in the public repo (and also may be slightly out of date). I'm attaching them here, caveat emptor. didactic_stark.zip risc0-verify.tar.gz

Thanks for the reply, Jeremy. It seems that the structure of the seal u32 array has changed since the stand alone verifier was made (Verifying the seal fails at https://github.com/cf/risc0-export-verify-seal/blob/main/risc0-verify/src/verify.rs#L112, see https://github.com/cf/risc0-export-verify-seal/blob/main/starter/src/main.rs#L31). Do you by any chance know roughly when this was made or which git commit produces valid seals, I would be happy to update the code to work with 0.11.0, but it would be very helpful if you could see the git diff to better understand how the structure of the seal data has changed.

I put together some diagrams that show a little bit more about what would be nice to accomplish with a standalone verifier, and we got the go ahead to invest some of our team's time into building this out for the risc0 community, and since this is more of a feature request than an issue, we'll publish a separate library over at https://github.com/OpenAssetStandards/dr-grouper if anyone wants to check out our progress or make a PR.
Phase 1 Architecture

Phase 1 Key Milestones

[ ] Dependency-free STARK verifier library for risc0 execution receipts (written in rust)

[ ] Serialize/Deserialize Helpers for Receipts generated by risc0
[ ] Basic demo that generates a risc0 proof from a guest and serializing/saves the receipt to a receipt.bin file
[ ] A separate dependency-free demo which can open, parse and verify a receipt.bin file using the dependency free library
[ ] Create an example risc0 guest demo which shows in-guest recursive proof verification.

[ ] Create a demo AntelopeIO smart contract using the eosio-rust sdk which can verify a risc0 proof using the dependency-free rust verifier library

Phase 2 Architecture

Phase 2 Key Milestones

[ ] Create a zkSNARK circuit capable of verifying a risc0 seal STARK proof (this could also potentially be broken up into phases of verification so that the proof can be generated in paralell and to minimize per proof gate count, since we only need to verify a single STARK proof. It is ok to need to verify 4-5 proofs on chain as verification cost for groth16 snarks is very low)
[ ] Create an ethereum smart contract capable of verifying the zkSNARK circuit and updating/storing the 3 merkle tree root hashes (tree of elf hashes, tree of journal hashes, tree of seal hashes)
[ ] Create an example 3rd party contract which demonstrates how to use a merkle proof to prove that the result of some execution with input arguments (x,y,z) is (a,b,c) using the tree of journal hashes and a Poseidon merkle proof
[ ] Create a proof of concept peer to peer networking example and incentive model for decentralized proving of arbitrary risc0 executions
[ ] Create a circom gadget which makes it easy to show how users can easily create proofs of complex relationships based on the verified inputs/outputs of multiple risc0 executions in the 3 trees and verify this fact on chain with 300k or less gas

So I think this is a very interesting project.  The phase one part largely looks like it is about getting recursion working with RISC Zero (i.e. running the verifier from within the prover).  Currently that is our most important near term goal, and almost half of our engineering staff is working on it in some way.  You are certainly welcome to contribute, but we should coordinate.  Regarding doing the STARK -> SNARK transform (i.e. verifying STARKS via SNARKS), we would love to have support for that, but currently don't have anyone working on it, so that may be a more useful place to contribute in the near term.

We will be announcing https://dev.risczero.com/bonsai soon which provides the ability to generate a groth16 proof which can be verified in solidity. See this example: https://github.com/risc0/risc0/tree/main/bonsai/examples/governance


Flexible widget

Done Implementing Flexible Widget in the Widget Catalog ! 👍

Hey I've done the required changes  ! Kindly mention the hacktoberfest-accepted label while merging


Node Locations With Nodal Solution

Hi Alex,
Sorry if this is the wrong platform to ask this, is it possible to get the nodal solutions to export with the node locations from the pyansys.ResultReader object? Or get the node numbers with node locations from the result file?

Yes, it's possible and it seems it's not too clear in the docs.  I made a quick example in a wiki for you:
https://github.com/akaszynski/pyansys/wiki/Print-Node-Numbers-and-Locations

Thanks so much Alex, that is perfect


[DatePicker] How do you get the name/event of the control?  Should we handle "event"?

Problem description
I looked through the docs and it looks like we're missing the event property that is null, which is used to properly set the values of the controlled component(such as TextField).
For instance to do the following:
property = event.target.name; is not possible with a DatePicker because the event is null.
The date property is properly hydrated but it makes things a bit wonky not being able to access the event itself, specifically the name of the component.
Maybe we just need to add a name property?
What do you guys think?
EDIT: Looks like the biggest issue is when you want two DatePicker controls within the same root component.
Link to minimal working code that reproduces the issue
     <DatePicker
          fullWidth
          style={{ margin: '-1rem auto' }}
          textFieldStyle={{ fontSize: fonts.md, marginTop: '0.25rem' }}
          name={propertyName}
          value={propertyValue}
          onChange={this.onDateChange}
        />

  onDateChange(event, date) {
    console.log(event); //null
    console.log('date', date); //whatever the date the user picked
  }

Versions

Material-UI: 0.17.1
React: 15.4.2
Browser:  Latest Chrome

I ended up wrapping the Material UI component and I'm going to move forward.  Works kind of nice based on what I need.  Here's the code snippet.

import React, { PropTypes } from 'react';
import BaseDatePicker from 'material-ui/DatePicker';

class DatePicker extends React.Component {

  constructor() {
    super();
    this.state = {
      passedDownProps: {},
      name: ''
    };

    this.onChange = this.onChange.bind(this);
    this.convertToMillis = this.convertToMillis.bind(this);
  }

  convertToMillis(dateString) {
    return dateString.valueOf();
  }

  // Note that the default value for a MaterialUI DatePicker is {}
  // NOT an empty string or any other value type
  convertToDateString(dateMillis) {
    if (dateMillis !== null && typeof yourVariable === 'object') {
      return dateMillis;
    }
    return new Date(dateMillis);
  }

  // Note that event is always null, but it is the first arg.
  // The reason it is null and it exists is because MUI wanted to maintain consistency
  // with the Textfield control.
  // http://www.material-ui.com/#/components/date-picker
  onChange(event, date) { //eslint-disable-line no-unused-vars
    const { onDataChange, name } = this.props;
    const value = this.convertToMillis(date);
    onDataChange(name, value);
  }

  render() {
    const {
      name,
      valueMillis,
      fullWidth,
      style,
      textFieldStyle
    } = this.props;

    const value = this.convertToDateString(valueMillis);

    return (
      <BaseDatePicker
        onChange={this.onChange}
        name={name} // This is required as Material-UI needs a key index on TextField
        value={value}
        fullWidth={fullWidth}
        style={style}
        textFieldStyle={textFieldStyle}/>
    );
  }
}

DatePicker.propTypes = {
  name: PropTypes.string.isRequired,
  onDataChange: PropTypes.func.isRequired,
  valueMillis: PropTypes.any.isRequired,
  fullWidth: PropTypes.bool,
  style: PropTypes.object,
  textFieldStyle: PropTypes.object
};

export default DatePicker;

But I would be interested to see what you guys are thinking long term for DatePicker because I'd love to help more.

@oshalygin DatePicker needs a rewrite as part of the migration to the new styling system on the next branch. Standardising callbacks is part of that effort, but no-one has tacked the pickers yet.
How much help were you thinking? :stuck_out_tongue_winking_eye:

Yeah I'd love to help as much as I can.
I don't think it would be too bad based on what I did to wrap the current DatePicker but I totally get that it will need to leverage the new styling effort.
I'll start tracking the next branch and digging deeper over the next week :)

@oshalygin Fantastic! There's a lot going on inside those components, so it's more than just updating to jss (although that's a big part of it). We're working towards:

Separating presentation and state
Composable components (where appropriate)
Standardising the API (where possible)
Squashing bugs
Tests! 🎉
Accessibility (ARIA labels, full keyboard support)
Improving the documentation

@oliviertassinari will be a good person to discuss possible approaches with. We have this placeholder issue (https://github.com/callemall/material-ui/issues/4787).

@oshalygin Yeah, the DatePicker, and TimePicker definitely need to be reworked! You can find all the relevant issues using the tag filter. We need someone to take the ownership of the migration 😄 .
I want to add some other stuff that are disturbing me with the two components:

UX

Not desktop optimized, for instance:

users might want to use their keyboard
the mouse travels too much distance

Not mobile optimized, for instance:

phone's OS have already have pretty good date picker and time picker, why reimplementing the wheel with a custom component?

The API

We miss quite some customization power, we could break the component into multiple indepentant one.

Awesome thank you gents, @oliviertassinari @mbrookes .  Going to get crackin!
Closing this issue out and will track with the filter.

@oshalygin I am still facing the same issue. Using v0.20.0.

just a heads-up, still facing this issue.


room assistant companion does not start transmitting after restart

I have successfully changed from Bluetooth Classic to BLE and startet using the Companion App on my iPhone.
The App works good and the room detection is good. But I have experience now a couple of times that after restart of my iPhone room assistant shows not_home
I have to open the companion app again in order to make it work.

I'm seeing the same issue, haven't thought of a good way around it yet though. The app already implements the APIs that Apple documents for restoring Bluetooth state after crashes/reboots, but it doesn't seem to work properly for reboots (anymore?). Unfortunately, Apple also does not allow the advertising state to be changed in the background as far as I read. As a quick workaround it may be nice to send a notifications when this occurs, so that it's quicker to notice & act. For a more long term solution I had a new idea for tracking iOS devices that wouldn't suffer from this problem, but that one is still very unproven and would also require a lot of changes.
Anyway, thanks for reporting!

I see. Let know if I can be of any assistance (e.g., testing etc)

I don't believe is happening only on an iPhone app restart. Leaving it on in the background, after a while, it also stops advertising - rendering it invisible.

I just added an automation from the iOS Shortcut app in order to launch Room Assistant iOS app upon geofence Arrival trigger (location  = home). This shall secure Room Assistant app is running when I get back home. Not ideal / little bit naggy but shall make the job. Additional trigger are also available (such as when iPhone is charging mode is on/off).


First commit of animation tools. Contains very experimental code.

Adds a new tool to test animation of entities. Right now only contains option to speed up the animation.
To get the tool in game open console (by pressing '~') and type "give animTest".
The tool is very experimental right now and does not have a proper UI. The speed can be changed using the UISlider and then the entity needs to spawned again.

Can one of the admins please verify this patch?

Cool :-) Looking forward to reviewing when I get a chance
@GooeyHub: add to whitelist

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Latest changes leave it pretty good and functional :-)
Some more hardening against nulls would be good in a next PR, such as validating that a given entity actually has a needed component or logging a warning that it doesn't then returning early.
Next up another dropdown for individual animations would be ideal (probably clearing out any other animations so only the selected one plays)


remove silence from curl for debug message

Jira link

See DTSPO-18508
Change description
remove silence from curl for debug message

Testing done

Checklist

[x] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)
[ ] Does this PR introduce a breaking change

Think you only need to remove it from the curl trying to send to slack channel at the end of the file

its already been removed already


Define variable for empty iterations

💡 Summary
In reference to https://github.com/cisagov/pe-reports/pull/6#discussion_r672830853, all report objects need to be defined as a variable in the event an iteration renders empty. This primarily applies in the pptx object 'shape' from 'shapes' and will need to be adjusted accordingly when applying the matplotlib libraries. Issue -https://github.com/cisagov/pe-reports/issues/14
Motivation and context
Considered a quality improvement to code.
Implementation notes
Please provide details for implementation, such as:

From pptx-python - the scope of 'shape':

- from pptx import Presentation

SLD_LAYOUT_TITLE_AND_CONTENT = 1

prs = Presentation()
slide_layout = prs.slide_layouts[SLD_LAYOUT_TITLE_AND_CONTENT]
slide = prs.slides.add_slide(slide_layout)
shapes = slide.shapes

Example stylesheets.py: Declare a shape variable in the event slide.shapes is empty.

   @staticmethod
    def shapes(slide):
        """Create a text frame."""
        for shape in slide.shapes:
            if not shape.has_text_frame:
                continue
        return shape

Acceptance criteria
How do we know when this work is done?

[x] Return value shape on empty iteration

This has been merged into develop branch per PR#31


plugin idea

am getting the hang of hugo.
we have lots of binary format stuff. 3D models, images etc
i often setup so that these non web file types are converted into web friendly type.
for example sketchup model i convert to webgl and threejs
inkscape i convert to png often
i was wondering if anyone had any thoughts about a general way to allow hugo to be told when it finds a file extension type, to call a shell extension. It could all be done with configs, and other people just write shell extensions they need for the conversions have they see fit.
what you you think ??

How about Pingo? https://github.com/dullgiulio/pingo

I've seen something similar called Pie written by @natefinch. Regardless which package could be used it seems that an RPC-based API could be an alternative to an restful API.

+1 for pingo or pie. I want to do some automatic image resizing and spriting. It would be great if I could do something through hugo.

Another plugin system with a proven track record:
https://github.com/hashicorp/go-plugin
This plugin system is used in a number of the Hashicorp tools: Packer, Vault, Terraform.
Credits to @mitchellh.

Note/Update: This issue is marked as stale, and I may have said something earlier about "opening a thread on the discussion forum". Please don't.
If this is a bug and you can still reproduce this error on the latest release or the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still relevant and valuable, please tell us why.


macOS sound improvement

I wanted to suggest a change the macOS sound notification to use the applescript notification instead of the terminal bell. It could also be upgraded to support different sounds.
            else if type -q osascript # AppleScript
                if test "$__done_notify_sound" -eq 1
                    osascript -e "display notification \"$message\" with title \"$title\" sound name \"Glass\""
                else
                    osascript -e "display notification \"$message\" with title \"$title\""
                end

I'm happy to send a PR over, but before I do, is this something you'd be interested in including?

Unfortunately it looks like @franciscolourenco isn't maintaining this anymore

I've not been spending much time with this project lately, but I will probably have more time moving forward.
@jjanyan what you suggest makes sense, feel free to open a PR. Supporting multiple sounds however might be overkill, no?

PR created https://github.com/franciscolourenco/done/pull/136

Merged, thanks!. It will be released in v1.17.0


MongoError: Topology was destroyed

All REST APIs are running very well. One day all APIs generate this error:
MongoError: Topology was destroyed
    at nextFunction (/srv/api/babylover_api/node_modules/mongodb-core/lib/cursor.js:603:27)
    at Cursor.next (/srv/api/babylover_api/node_modules/mongodb-core/lib/cursor.js:824:3)
    at Cursor._next (/srv/api/babylover_api/node_modules/mongodb/lib/cursor.js:211:36)
    at nextObject (/srv/api/babylover_api/node_modules/mongodb/lib/operations/cursor_ops.js:186:10)
    at next (/srv/api/babylover_api/node_modules/mongodb/lib/operations/cursor_ops.js:165:3)
    at executeOperation (/srv/api/babylover_api/node_modules/mongodb/lib/utils.js:420:24)
    at Cursor.next (/srv/api/babylover_api/node_modules/mongodb/lib/cursor.js:253:10)
    at findOne (/srv/api/babylover_api/node_modules/mongodb/lib/operations/collection_ops.js:542:10)
    at executeOperation (/srv/api/babylover_api/node_modules/mongodb/lib/utils.js:420:24)
    at Collection.findOne (/srv/api/babylover_api/node_modules/mongodb/lib/collection.js:1021:10)
    at NativeCollection.(anonymous function) [as findOne] (/srv/api/babylover_api/node_modules/mongoose/lib/drivers/node-mongodb-native/collection.js:142:28)
    at NodeCollection.findOne (/srv/api/babylover_api/node_modules/mquery/lib/collection/node.js:42:19)
    at model.Query.Query.findOne (/srv/api/babylover_api/node_modules/mquery/lib/mquery.js:1816:20)
    at model.Query.Query._findOne (/srv/api/babylover_api/node_modules/mongoose/lib/query.js:1792:22)
    at process.nextTick (/srv/api/babylover_api/node_modules/kareem/index.js:333:33)
    at _combinedTickCallback (internal/process/next_tick.js:131:7)
    at process._tickCallback (internal/process/next_tick.js:180:9)

That's a mongo or mongoose error. Not part of this repo.
Have you tried searching for the error in google or stack overflow ?


[FRG] Update right-aligned element position after margin changes

I had some right-aligned elements, and when I adjusted my margins, they were beyond the edge of the page margins.
It’s almost as if when you right-align elements, it’s setting an absoluteX position, instead of just figuring out where the right edge of the printable area is.

Hey just double checking, but from what I've been able to reproduce, this doesn't affect when creating the actual report, just the editor preview.
If align is set to something other than none than absoluteX isn't used on the actual report. It is only used to show where the element kinda would be in the report preview. Which at the moment is broken, because it aligns right of the report preview instead of it should be right of the supposed page size.
So double checking and confirming, this is what this issue is about? Currently applying a fix for this, but ensuring I've fixed the actual issue you are reporting.


[Feature Request] fate tracker - Amount of runs

Possibility to add a option to fate tracker.
Main option: Do "x" amounts of fates and then stop.
Additional check box option: Exit game after completion of runs.

I'm not sure fate completion is something that can be easily tracked directly since all that's really happening under the hood is "am I in a fate? if yes, kill fate mobs, if no, go to nearest doable fate"
but if you're looking to do something like "run until gemstones are full" or "run until shared fate complete for zone" then that's probably doable since those are accessible in game values


[Hub Generated] Review request for Microsoft.Insights to add version stable/2019-03-01

If you are a MSFT employee you can view your work branch via this link.
Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

Can one of the admins verify this patch?


chore: Update package version to keep dependencies up to date

Fixed : Issue #14 (Updated dependencies)


all doxidize commands print out hello world

As per description. I'm following the quickstart guide to cargo install it, and can't proceed past doxidize init, as that doesn't do anything else but print hello world.
I'm using the nightly toolchain.

The release on crates.io is just a hello world, IIRC, so you’d have to install from git.
Doxidize isn’t intended for use yet, I’m not even sure if it’s currently working.

On Sep 15, 2018, at 8:53 AM, Dickson Tan<EMAIL_ADDRESS>wrote:
As per description. I'm following the quickstart guide to cargo install it, and can't proceed past doxidize init, as that doesn't do anything else but print hello world.
I'm using the nightly toolchain.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Font is tiny and the boxes are slim

Hello,

This is my problem, i dont know how to fix it.
I updated my graphics driver and everything.
I also used the command for PowerShell, its show me nothing goes wrong.

Help me please :(

I'm guessing this is a font related issue. Try navigating to the display panel and adjusting the font size and style. It is the 2nd option down on the left hand side. Here is what the menu should look like for reference:

Closed due to lack of activity. Try the solution above.


Make Rewarding Snipes Relic Carts not die at sea

Giving relic class keeps things from dying at sea

Makes it not convertible and attackable. Needs more testing


gtm.enhancedEcommerce errors

I get javascript error to all sites.
[Error] Failed to load resource: the server responded with a status of 500 ()
xxxxxx.....syliusgtmenhancedecommerceplugin/gtm.enhancedEcommerce.checkout.js
xxxxxx.....syliusgtmenhancedecommerceplugin/gtm.enhancedEcommerce.productClicks.js
xxxxxx.....syliusgtmenhancedecommerceplugin/gtm.enhancedEcommerce.cart.js
xxxxxx...../gtmplugin/prototype.events.js
Any idea how to solve this?

was reinstalling the assets - now it works.


confd requires http:// prefix for etcd

Since when is it needed?

I don't know. All I know is, that without the "http://" prefix it doesn't work for me...

I haven't used this recently, @Ulexus any idea? Did we bump the confd version?

@MichaelSp do you mind rebasing your patch and do your modifications in ceph-releases/jewel/ubuntu/14.04? Thanks!

If the files "don't exist" why don't you remove them? This is version control. You can have them back any time you want. This just confuses others and me.
I'll try to find to to rebase...

@MichaelSp please read the recent changes on the README.

why this hasn't been merged?

@einyx because it needs a rebase, @MichaelSp doesn't have time, @einyx you can take over if you want. Please update ceph-releases/jewel/ubuntu/14.04/daemon/config.kv.sh.

saw this to late and fixed the same thing. also fixed httpS for etcd https://github.com/ceph/ceph-docker/pull/327


Move legacy code to legacy directory

Oh ok, so I guess when we remove the shared code we can split up by branch?

Yes I think so. As you already see, there are some types and so on used from the legacy packages which we need to remove.


Can't connect when running clojure code or updating the clojure plug-in

I am receiving an error whenever I try to run clojure code or update the plugin. My issue seems to be related to issue #851 and #1090, but it seems to be different from those and it is not fixed by the suggestions in these issues. I have included relevant information below, but let me know if you need more.
I had run into this problem earlier in the day when trying to evaluate some code, but only really realized that something was wrong when I tried to make a repl for clojure using this tutorial and got the same error when trying to execute the code.
Here is a link to project.clj. The first file is the original and the second is after applying the changes suggested in issue #851.
Interventions

Intervention
Result

Changing the project.clj as instructed in issue #851
No change

Restarting LightTable
No change

Restarting the PC
No change

Reinstalling LightTable
No change

Turning off AntiVirus and Windows Firewall
No change

Environment

Info
Details

Light Table version
0.8.1

Operating system
Windows 7

Clojure version
1.8.0

Relevant Plugins

Plugin
Version

Clojure
0.3.3

### Error Messages

We couldn't connect.

Looks like there was an issue trying to connect to the project.

I just included a link into the main post, at the description.

Hmm, the first project.clj looks like what I have and it works fine for me. Based on conversation in the Clojurians Slack, I think something else is amiss and not the project configuration.
When you open up the plugin manager, are there any errors showing up in the console?

No errors show when I open the plugin manager. I will try reinstalling the program just to see what happens.

No change after reinstalling or after trying to evaluate code with antivirus turned-off.

I decided to delete the interact folder to see what would happen, and the error message changed (more details in the original post).

I'm not too sure what else could be the problem. I have had no luck in reproducing any of the error messages. :(
Maybe try deleting the contents of C:\Users\<your_user_name>\AppData\Roaming\LightTable, where <your_user_name> is the appropriate path?

No luck =(

Perhaps you have some conflicting deps/plugins in ~/.lein/profiles.clj ? (Assuming you have installed leiningen at some point, and ~ being your home folder)

I have lein but it is a fresh install. I checked the file and there are no plugins there.


Inline SVG is marked as an invalid tag

Hi, when I try to validate this document, I get some errors:
<!DOCTYPE html>
<html><head><title>hi</title></head>
	<body>
		<svg width="400" height="200">
			<circle cx="150" cy="100" r="50" />
		</svg>
	</body>
</html>

Linter Messages
validation http://example.org/: Tag svg invalid
validation http://example.org/: Tag circle invalid

As far as I can tell it is valid.  Am I missing something?

The linter uses Nokogiri to do HTML parsing and validation, and there seems to be an open issue on this particular subject: https://github.com/sparklemotion/nokogiri/issues/1243. I'll investigate further to see if there's a way to silence such issues, as the main purpose is to look for illegal use of HTML elements and attributes, and bad wellformedness.
In the mean time, if it can't be easily resolved otherwise, a comment that polyglot markup may be marked in Linter Messages may be all we can do.

Oh oops I did find that issue by googling and assumed it was unrelated!  Seems like noone is working on that one, so I'll ignore it in our tests if nothing else.  I'm currently working on calling linter directly locally and I may create a PR for you if I have something useful to contribute back.

HTML validation is done in the rdf-rdfa gem, and it may require some configuration there (https://github.com/ruby-rdf/rdf-rdfa/blob/develop/lib/rdf/rdfa/reader/nokogiri.rb#L283).
I think it may be possible to configure when parsing, which is done here: https://github.com/ruby-rdf/rdf-rdfa/blob/develop/lib/rdf/rdfa/reader/nokogiri.rb#L180.

Fixed in rdf/rdfa/#3.


[22.03]luci-mod-status,mod-network: Added fqdn-name to DHCPv4 lease table

Added code to display fcdn-name in DHCPv4 lease table.
(based on code in DHCPv6 lease table)
Signed-off-by: Max S Kash<EMAIL_ADDRESS>Indentation adjused and wrapped commit message
Signed-off-by: Florian Eckert<EMAIL_ADDRESS>cherry pick from master https://github.com/openwrt/luci/commit/8a46648e5bfb4b8f4d2452c51b5181631d260b6f

Thanks

Good


4.40.0 broke our release process

Bug report

What is the current behavior?
There was introduced a breaking change on 4.40.0 that was working before (didnt follow semver). I have my building using version 4.39.3 without issues.
If the current behavior is a bug, please provide the steps to reproduce.
Just run webpack

What is the expected behavior?

Other relevant information:
webpack version:  4.40.0 (worked 4.39.3 without issues)
Node.js version: 10.16.2
Operating System:  macOS Sierra
Additional tools:

Just run webpack

I was unable to reproduce this.


Servicing Stack updates

[Enter feedback here]
Can you add this blog: https://techcommunity.microsoft.com/t5/windows-it-pro-blog/simplifying-on-premises-deployment-of-servicing-stack-updates/ba-p/1646039

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

ID: b706f0af-d504-a8ca-7c38-b8bb9283bfad
Version Independent ID: 14f31acf-9f47-59a6-269f-62ab249fe560
Content: Servicing stack updates (Windows 10) - Windows Deployment
Content Source: windows/deployment/update/servicing-stack-updates.md
Product: w10
Technology: windows
GitHub Login: @jaimeo
Microsoft Alias: jaimeo

@felizondojr . Ok I add the link that you shared

Closing via merged PR, #8484


Issues in Safari

Please give details of the issue:
Support info:
Wed Apr 26 2023 13:42:01 GMT+0100 (British Summer Time)
Safari
16.2 [1961,475]
https://v2.virtualflybrain.org/org.geppetto.frontend/geppetto?id=VFB_00101567&i=VFB_00101567
[Log] THREE.WebGLRenderer – "87" (main.bundle.js, line 1)
[Log] Loading IDS to add to the scene from url (main.bundle.js, line 303)
[Log] Host for MessageSocket to connect: wss://v2.virtualflybrain.org/org.geppetto.frontend/GeppettoServlet (main.bundle.js, line 303)
[Log]  WebSocket Status - Opened  (main.bundle.js, line 303)
[Error] SyntaxError: JSON Parse error: Unterminated string
parse (main.bundle.js:290:3263)
l (main.bundle.js:290:3263)
(anonymous function) (main.bundle.js:290:5126)
(anonymous function) (main.bundle.js:290:5278)
[Error] WebSocket connection to 'wss://v2.virtualflybrain.org/org.geppetto.frontend/GeppettoServlet' failed: The operation couldn’t be completed. (kNWErrorDomainPOSIX error 100 - Protocol error)
[Log]  WebSocket Status - Open connection error  (main.bundle.js, line 303)
[Log] WebSocket Status - retry in 5000ms – CloseEvent (main.bundle.js, line 303)
CloseEvent
[Log]  WebSocket Status - reconnecting...  (main.bundle.js, line 303)
[Log]  WebSocket Status - Opened  (main.bundle.js, line 303)

Selected Element
< …
[Warning] [blocked] The page at https://v2.virtualflybrain.org/org.geppetto.frontend/geppetto?id=VFB_00101567&i=VFB_00101567 was not allowed to run insecure content from http://gd.geobytes.com/GetCityDetails?callback=jQuery36109875237128015402_1682512888182&_=1682512888183. (common.bundle.js, line 25)

This issue was resolved in latest releases


chore(dev branch): merge master into dev

chore(dev branch): merge master into dev
Previous PR lost link from master to dev. This should fix that.

Coverage remained the same at 98.942% when pulling 97eec7a3415cb2ebe36d4c4fad89b8afd18d82f3 on gihrig:dev into 57ac874de78e58c091875180c6abdc57aeaa2c1c on mxstbr:dev.

@mxstbr I've messed something up here, and my git fu is not up to figuring out what it is 😢
With dev selected:

This branch is 6 commits ahead, 4 commits behind master.

dev is 6 commits ahead of master but all of masters changes are visible in dev. So, dev should not be be behind master.
Further, when I attempt to merge upstream/dev into my local/dev a fast forward merge is not possible, a full merge is required.
This says to me that something is wrong with git's linkage between branches...
Thoughts, advice?
PS. feel free to revert either of these commits if that's appropriate.


Google wifi intergration error

Hi I was wondering if anyone could help me I keep having the error with Google wifi intergration, I have uninstalled a couple of times, along with restarting and using different token. I'm probably doing something wrong. below is the error log.
2022-03-30 17:34:56 ERROR (MainThread) [homeassistant.components.binary_sensor] Error while setting up googlewifi platform for binary_sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 249, in _async_setup_platform
await asyncio.shield(task)
File "/config/custom_components/googlewifi/binary_sensor.py", line 29, in async_setup_entry
for system_id, system in coordinator.data.items():
AttributeError: 'NoneType' object has no attribute 'items'
2022-03-30 17:34:56 ERROR (MainThread) [homeassistant.components.device_tracker] Error while setting up googlewifi platform for device_tracker
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 249, in _async_setup_platform
await asyncio.shield(task)
File "/config/custom_components/googlewifi/device_tracker.py", line 31, in async_setup_entry
for system_id, system in coordinator.data.items():
AttributeError: 'NoneType' object has no attribute 'items'
2022-03-30 17:34:56 ERROR (MainThread) [homeassistant.components.switch] Error while setting up googlewifi platform for switch
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 249, in _async_setup_platform
await asyncio.shield(task)
File "/config/custom_components/googlewifi/switch.py", line 45, in async_setup_entry
for system_id, system in coordinator.data.items():
AttributeError: 'NoneType' object has no attribute 'items'
2022-03-30 17:34:56 ERROR (MainThread) [homeassistant.components.light] Error while setting up googlewifi platform for light
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 249, in _async_setup_platform
await asyncio.shield(task)
File "/config/custom_components/googlewifi/light.py", line 30, in async_setup_entry
for system_id, system in coordinator.data.items():
AttributeError: 'NoneType' object has no attribute 'items'
2022-03-30 17:34:56 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up googlewifi platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 249, in _async_setup_platform
await asyncio.shield(task)
File "/config/custom_components/googlewifi/sensor.py", line 32, in async_setup_entry
for system_id, system in coordinator.data.items():
AttributeError: 'NoneType' object has no attribute 'items'

Try again with the latest release 0.1.31


fix links in contibutor docs.

I have some doubts, even on the other PR, I'm not sure the website will build in this way.

Yeah I was curious about it. The website is working fine as things are. Wondering why these changes are needed?

i am not sure if this have an impact on how the site is built. i just followed github links and it was giving error as

Indeed, now your PR makes much more sense to us! As it happens, these adoc files are not primarily intended for github browsing. They serve as an input for Apache Camel site build whose resuts can be visible at https://camel.apache.org/camel-quarkus/latest/index.html Hence links work as designed. Thanks for taking care anyway!


fix: drawer click outside

Description
Add control for event on Drawer click outside
Test

[x] I have added tests that prove my fix is effective or that my feature works

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Two tests called "attaches currentScope on $emit" in scope_spec.js

In scope_spec.js, there are two tests named "attaches currentScope on $emit". I believe the second one should be "attaches currentScope on $broadcast" as this is what the test is targeting.

Fixing, thanks


Handling Content-Type in 304 responses

This is sort of a bug report, sort of a feature request ...
A certain popular CDN is occasionally returning Content-Type in it's 304 responses (I'm pretty sure it's them and not the upstream actual server).  Even worse it's the wrong Content-Type - text/plain instead of application/json
I dropped a conditional breakpoint in CacheInterceptor.intercept() in it's handling of a 304 that would only trigger when networkResponse.header("Content-Type")!=null.  After playing with my app heavily and continuously for about an hour this breakpoint eventually triggers.  So I'm confident this is happening.
So where does okhttp (i'm using 3.6.0 on android) come in to this?  Why the report/request?  Because this Content-Type is returned to my app instead of the correct value from the cache.  I feel okhttp should ignore the headers that should not be in a 304 response - call it a bug, call it a feature (defend against bad servers).

Section 13.5.3 and 13.5.1 is what we implemented.
https://www.ietf.org/rfc/rfc2616.txt
Have you reported the problem to the server's administrator? Presumably a link to that RFC should be compelling.
In the interim you can strip unwanted headers with a network interceptor.

I have reported the issue to the service provider (waiting for reply).
I have also changed my app to sniff the content when I receive plain/text to see if it's really a json, this works just fine.
since this took me over a week to catch it in a debugger to prove the issue, I feel it would be nice if you added defence into okhttp for this.  it's bound to trip up many other devs ...

What would that defense look like?

I would have thought simply removing all headers from the network response that related to the body when a 304 is received. ie, Content-Type, Content-Encoding, Content-Language, etc.  basically Content-*
the simplest would to be do this in the 304 handler in CacheInterceptor, but really it could be performed a lot earlier (probably should be?)
this probably should be for a few other http codes as well (100-199, 204, see http://stackoverflow.com/questions/8628725/comprehensive-list-of-http-status-codes-that-dont-include-a-response-body)

Seems like it’d be convenient for you, but it contradicts the HTTP spec.

In other words, the set of end-to-end headers received in the incoming response overrides all corresponding end-to-end headers stored with the cache entry (except for stored Warning headers with warn-code 1xx, which are deleted even if not overridden).

Another question – what would have helped you to diagnose this earlier? Were you using a tool like Charles or Stetho?

no, I wasn't using any proxies when debugging.  the slowness in debugging was the rareness of the problem and that nothing spat a warning anywhere (eg, nothing in logcat, no exceptions ...).  it took a while before I thought of logging mime as I didn't know what the problem was.  then when I saw it I didn't really believe it so then tried to work out was there a bug in okhttp's cache or was the server really doing the wrong thing.  like if it was repeatable or at least frequent then I would have nailed it in no time.
/me shakes fist in the air at the bl**dy CDN provider (still no response from them)

hmm ... another thought: maybe if a 304 is received with header values that it shouldn't have then maybe just reject/fail the response instead of processing it?

That would break caching completely.

Consider stripping the content-type from the response in a network interceptor. It’ll workaround the bad behavior of your server until they fix their stuff.


Uber and Instagram

We have Uber and Instagram as examples over Swagger Editor example files. Instagram is not complete yet, I'm working on it. If you want to have those feel free to steal them from there

@mohsen1 Thank you.
I don't have enough resources to manually  check and maintain validity of Swagger documents.
And my experience show that Google for example update it's API documents almost every day.
So for the time being, I'm focused only on official sources, this allow me to automatically update and react on all changes. I specify official link inside Swagger extension, like here or here and use them inside update script.
Do you official URLs for these Swagger documents?
Or URLs to documents in any other format?
Can you please describe how you acquired information for this documents?

Nope. It's all done manually

@mohsen1 I founder complete version of Instagram API here:
https://github.com/darklynx/swagger-api-collection
As for Uber, I need that someone maintain this spec and update it periodically.
But I think it's a very good example for swagger-editor :+1:


add Typo3

fixes #1485

@hebbet why close this?

Not Sure 🤔
Didn't have the time this week to look more into it why it fails

opps it looks like we couldnt  restoreback the branch
i think that time releasePolicyLink: was the problem

@captn3m0
i think for automation we need something like we have for mariadb
https://github.com/TYPO3/typo3/tags

also
not sure how we want to add the paid ELTS Versions

Let's put a mention in the text for now. Maybe a table like:

Version
Commercial ELTS Support Ends

7
2022-11-30

8
2023-03-31

9
2024-09-30

Can you create an issue with the list of pages where we need an extra column?

We should add notes about PHP Compatibility as well, similar to Drupal

i think for automation we need something like we have for mariadb

With mariadb, we are forced to keep that regex to ensure we drop the RC releases and have an accurate date for the start of the release cycle.
I'm unsure what a "Sprint release" means? That's how the project seems to describe releases that haven't graduated to LTS (11.5 > R>=11.0) (https://get.typo3.org/release-notes/11.4.0 for eg).
Are Sprint releases considered stable? If yes, we should be okay listing them on the table. If not, we can avoid adding a new release cycle for 12 till a LTS release is announced.

[ ] Add notes about PHP Compatibility as well, similar to Drupal
[ ] Add ELTS note in text for now
[ ] Add older releases cycles (7,8)

add PHP Info and ELTS versions as table.

Are Sprint releases considered stable? If yes, we should be okay listing them on the table. If not, we can avoid adding a new release cycle for 12 till a LTS release is announced.

Sprint releases aren't stable at typo3. When they announce a new LTS release, it is stable. For Typo3 11 eg. the first stable Version was 11.5.0, the current stable version is 11.5.15, for v10 the stable branche is 10.4.x, v9 was 9.5.x, v8 was 8.7.x and v7's stable version is 7.6.x

added older version from here.
https://typo3.org/cms/roadmap
latestversion is from github tags

@captn3m0 ping :-)


Cannot read property 'file' of undefined

When loading the package, I get the following error.
Failed to activate the linter-pep8 package

Cannot read property 'file' of undefined

TypeError: Cannot read property 'file' of undefined
    at Object.install (C:/Users/Kevin Rose/.atom/packages/linter-pep8/node_modules/atom-package-deps/lib/main.js:14:57)
    at Object.module.exports.activate (file:///C:/Users/Kevin Rose/.atom/packages/linter-pep8/lib/main.coffee:18:34)
    at Package.module.exports.Package.activateNow (C:\Users\Kevin Rose\AppData\Local\atom\app-1.3.3\resources\app.asar\src\package.js:170:19)
    at C:\Users\Kevin Rose\AppData\Local\atom\app-1.3.3\resources\app.asar\src\package.js:150:32
    at Package.module.exports.Package.measure (C:\Users\Kevin Rose\AppData\Local\atom\app-1.3.3\resources\app.asar\src\package.js:92:15)
    at C:\Users\Kevin Rose\AppData\Local\atom\app-1.3.3\resources\app.asar\src\package.js:143:26
    at Package.module.exports.Package.activate (C:\Users\Kevin Rose\AppData\Local\atom\app-1.3.3\resources\app.asar\src\package.js:140:34)
    at PackageManager.module.exports.PackageManager.activatePackage (C:\Users\Kevin Rose\AppData\Local\atom\app-1.3.3\resources\app.asar\src\package-manager.js:517:21)
    at C:\Users\Kevin Rose\AppData\Local\atom\app-1.3.3\resources\app.asar\src\package-manager.js:499:29
    at Config.module.exports.Config.transactAsync (C:\Users\Kevin Rose\AppData\Local\atom\app-1.3.3\resources\app.asar\src\config.js:337:18)
    at PackageManager.module.exports.PackageManager.activatePackages (C:\Users\Kevin Rose\AppData\Local\atom\app-1.3.3\resources\app.asar\src\package-manager.js:494:19)
    at PackageManager.module.exports.PackageManager.activate (C:\Users\Kevin Rose\AppData\Local\atom\app-1.3.3\resources\app.asar\src\package-manager.js:478:46)
    at AtomEnvironment.module.exports.AtomEnvironment.startEditorWindow (C:\Users\Kevin Rose\AppData\Local\atom\app-1.3.3\resources\app.asar\src\atom-environment.js:686:21)
    at module.exports (C:\Users\Kevin Rose\AppData\Local\atom\app-1.3.3\resources\app.asar\src\initialize-application-window.js:28:10)
    at setupWindow (file:///C:/Users/Kevin%20Rose/AppData/Local/atom/app-1.3.3/resources/app.asar/static/index.js:86:5)
    at window.onload (file:///C:/Users/Kevin%20Rose/AppData/Local/atom/app-1.3.3/resources/app.asar/static/index.js:41:9)

Windows 10
linter v1.11.3
linter-pep8 v1.2.1
Atom 1.3.3

Looking into this, I think I've experienced a similar issue as well so I might know how to reproduce this.

This is fixed by https://github.com/steelbrain/callsite/pull/3, I'll be releasing a new version of this incorporating it soon. You can also just go into the folder and run apm rebuild.

Marking as closed since this wasn't actually an issue with linter-pep8.

Can you expand a little more on what you mean by "just go into the folder and run apm rebuild"? I have gone into both ~/.atom/packages/linter-pep8/ and ~/.atom/packages/linter/ and run apm rebuild and it hasn't had any effect.
I've not done any JS/Coffescript programming, so it's hard for me to see where the callsite is actually being used or I could just go and change the source code directly while waiting for the official update.

That's odd, apm rebuild is supposed to update all the dependencies to the latest allowed versions, which should have pulled in the updates. I guess Atom isn't quite working right there...
In any case, glad you got it figured out!


update to april release

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Any specific reason to choose the April release rather than the June one?

Any specific reason to choose the April release rather than the June one?

The feature we needed was in the April release

Please have a look at https://github.com/conda-forge/lammps-feedstock/pull/201/commits/333d57daf4eb2db26fa2e62bd1136e4a1c845754
I gobbled this together from resources on the linter warnings but I can't claim I really understand all of it

hm... getting this on osx
 nothing provides requested c_osx-64

@jan-janssen , so if the checks except for the linter were to pass, would you merge this?
Or is there something that still does not look right to you?

E.g. do I still need to add
os_version:
  # cos7 to get c_stdlib_version 2.17
  # which isn't working from conda_forge_config.yaml
  linux_64: cos7

to the  conda-forge.yaml?

@jan-janssen , so if the checks except for the linter were to pass, would you merge this?

I can merge it once the tests pass - no need to do anything more from your side.


Is there a reason to use getFields instead of getDeclaredField in BTreeParser

Issue details
On my way to try the framework I struggled to get the @TaskAttribute annotation to work. I always got the following exception : throw stackedTaskException(getCurrentTask(), "unknown attribute '" + name + "'");
The issue was that I set my field as private and in the BehaviorTreeParser the findMetadata method() use the method ClassReflection.getField which only return the public field.
Is there a reason for that ? Wouldn't it be more intuitive to use ClassReflection.getDeclaredField method instead ? If not I think we should update the documentation to notify this behaviour.
Reproduction steps/code
Create a simple LeaftTask with a private field annotated with @TaskAttribute, set the TaskAttribute in the .btree file and try to parse it. You will get a unknow attribute error.
Version of gdx-ai and/or relevant dependencies
1.8.1

This also becomes a problem when using Kotlin instead of Java. It would be awesome, if the Annotation could be extended to be allowed on Setter Methods so it's also possible to do some post-processing.

I created a PR for this issue.

@tommyettinger Should we close this issue since 117 has been merged?

@jhoukem Since using getDeclaredFields() won't find inherited members but getFields() will, 117 was just reverted (117 breaks existing code that inherited fields, and existing code couldn't have used private fields before and so still can't). I'd like to be able to have this handle the Kotlin case mentioned above and the current inheritance use case, but it doesn't seem straightforward. I'm not sure why you actually would need private fields in your behavior tree Tasks, except for Kotlin's quirks which 117 wouldn't be able to handle (since it would need to handle setters).
So there's your reason, getFields() allows inheriting fields as long as they are public, and getDeclaredFields() does not allow inheriting fields.

As of Mar 2022, the unexpected behavior described in this issue still appears for LeafTasks in Kotlin when annotated as documented for Java.
What seems to have worked for me was adding @JvmField annotation before the @TaskAttribute annotation in the LeafTask class as so:
@JvmField
@TaskAttribute
var doing : String? = null

Also important appears to be how one creates the tree. I was unable to get attribute data into the LeafTask class from the btree using the following tree parsing approach (from GdxDemo3d):
// Create behavior tree through the library
BehaviorTreeLibraryManager btlm = BehaviorTreeLibraryManager.getInstance();
this.tree = btlm.createBehaviorTree("btrees/dog.btree", this);

However, the following tree parsing approach (inspired by Benoit Dumas's Alchemist project) populates the LeafTask attribute as I would expect:
var tree : BehaviorTree<KoboldCharacter>

init {
    val reader = FileReader("android/assets/btree/kobold.btree")
    val parser = BehaviorTreeParser<KoboldCharacter>(BehaviorTreeParser.DEBUG_HIGH)
    tree = parser.parse(reader, this)
}

Hope this helps.
Thank you,
Ian


canceling BRPOP gracefully

Hi,
I use redis as a lightweight message queue with BRPOP. I would like to disconnect redis on SIGTERM, so I call client.quit() on the client that is listening for the messages with BRPOP. The problem is that the promise returned by client.quit() is never resolved. client.disconnect() works but I would not like to lose a payload that's on it's way to the client.
Is this behaviour intentional? My guess is that QUIT waits for BRPOP to complete but is there a way to end a BRPOP safely?
I would not like to use timeout and periodically resending BRPOP just to handle process shutdowns gracefully.
Thanks for the help!

I'm afraid there's no way to cancel BRPOP since Redis doesn't provide an interface for it. The timeout argument of BRPOP is designed for this use case as for me 😆

Thank you for your response! Is the underlying connection reused for multiple subsequent timeouted BRPOP-s?

A client only has a single connection to Redis. If a BRPOP timeouts, the connection is able to process subsequent commands (e.g. another BRPOP).

Ok, thanks!


fix(core): :bug: the docomentation is not present has been updated

Description
-the docomentation was not present has been updated
-it was showing not working - by using the sample code the user can understand how it works
Image

Ref: #189

Merging since stale over 24hrs.

please check if there is any changes needed or not
@techoneel

Merging since stale over 24hrs.


Add support for input file

This Pull Request closes this issue #160
Description

Add support for file input

Code sample

Codepen.

@marcusasplund please, review this PR 🙏

Looking good. Two minor things; in the example you have the class names  button input-file,
on the label the input-file is obsolete there and could be removed.
Also, i am not sure that it is totally ARIA-compliant to have duplicate labels with same name. But i am unsure of this, and it probably not a problem at all.
Thx for a nice lib, i use it extensively!

 Release v1.5.0

@marcusasplund Thank you for your thoughts. 🙏
By default, I think important shows the file input, and if you want to hide just add input-file class.
For now, I will accept this PR and the future I will improve the behavior of this feature.


Getting 503 intermittently on ingress gateway

Title:  Getting 503 intermittently on ingress gateway
Description:
We are using ISTIO as service mesh.
We are seeing "response_code":503 intermittently on our ingress gateway.
Note: We have applied WASM filter- customfilter on ingress gateway.
When we enabled trace logs we can see below for all the flows which are giving 503:
2024-04-01T14:38:10.133142Z trace envoy http external/envoy/source/common/http/filter_manager.cc:539 [C458447][S4288012434896823960] decode headers called: filter=istio.metadata_exchange status=0 thread=63
2024-04-01T14:38:10.133153Z debug envoy http external/envoy/source/common/http/filter_manager.cc:917 [C458447][S4288012434896823960] Preparing local reply with details wasm_fail_stream thread=63
2024-04-01T14:38:10.133166Z debug envoy http external/envoy/source/common/http/filter_manager.cc:959 [C458447][S4288012434896823960] Executing sending local reply. thread=63
2024-04-01T14:38:10.133173Z trace envoy http external/envoy/source/common/http/filter_manager.cc:1129 [C458447][S4288012434896823960] encode headers called: filter=istio.stats status=0 thread=63
2024-04-01T14:38:10.133176Z trace envoy http external/envoy/source/common/http/filter_manager.cc:1129 [C458447][S4288012434896823960] encode headers called: filter=envoy.filters.http.cors status=0 thread=63
2024-04-01T14:38:10.133183Z trace envoy http external/envoy/source/common/http/filter_manager.cc:1129 [C458447][S4288012434896823960] encode headers called: filter=envoy.filters.http.fault status=0 thread=63
2024-04-01T14:38:10.133184Z trace envoy http external/envoy/source/common/http/filter_manager.cc:1129 [C458447][S4288012434896823960] encode headers called: filter=istio-system.customfilter status=4 thread=63
2024-04-01T14:38:10.133187Z trace envoy http external/envoy/source/common/http/filter_manager.cc:530 [C458447][S4288012434896823960] decodeHeaders filter iteration aborted due to local reply: filter=istio-system.customfilter thread=63
2024-04-01T14:38:10.133191Z trace envoy http external/envoy/source/common/http/filter_manager.cc:539 [C458447][S4288012434896823960] decode headers called: filter=istio-system.customfilter status=1 thread=63
2024-04-01T14:38:10.133194Z trace envoy http2 external/envoy/source/common/http/http2/codec_impl.cc:1002 [C458447] dispatched 1446 bytes thread=63

Envoy version - v1.25

cc: @zhaohuabing

"wasm_fail_stream" seems to be called when the wasm VM fails, fwiw.
CC: @mpwarres @lizan


[event-protocol] Use $ref to reduce duplication in schemas

Summary
In JSON schemas, you can use refs to other files to re-use parts of your schema. This PR demonstrates how it can be done, though it doesn't do it everywhere yet.
Details
I extracted the definitions for the timestamp and series properties into a separate defs.json file and referenced that.
I also cleaned up the validate.js code a bit, based on some examples I was reading about using ajv.
Motivation and Context
As the schema grows, we need to be able to easily maintain it. Keeping duplication down will be important in this.
How Has This Been Tested?
I ran make and everything is still green.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[x] Refactoring

Can I ask you to prefix issues and. Onmita as sub-project: rather than [sub-project]. It's a convention already used for commits.

Sure, sounds good! Shall we document that in the contribution guidelines? (Or is it already and I missed it?!)

We should document it in CONTRIBUTING.md which needs some cleanup.

I put something in the github PR / Issue templates for now. I'm a bit outfaced by the contributing file.


Modify action for json files

My proposition is to add a modify action for JSON files, where you will provide callback instead of pattern and template. In callback you would get JSON.parsed object and you should also return POJsO which will be JSON.stringifyd and written to file.

I went ahead and made a custom action for this, but I definitely think this could get rolled in as a flag on modify action types or something. Here's the code for alterJsonAction.js: https://gist.github.com/wbobeirne/63e1b8c3a16ffce6dac5c7d81afd5cec?ts=2
Example usage from my use case is:
actions: [alterJsonAction({
	path: "../webpack/entries.json",
	alter: (json, data) => {
		const name = snakeCase(data.name);
		json[data.platform][name] = `pages/${name}`;
		return json;
	},
})],

I also threw in an indentation arg there that takes an int or "\t", if 2 spaces ain't your game.

sorry for the late response, I agree that this would be pretty useful in some cases. I'll have to give some more thought to the best implementation for this feature. Thanks!

hey, with plop 1.8.0 you can now create custom action types that you can share with other projects :-)

I created a custom action type to add data to my config.json file.
Tried to follow the plop style to keep it customizable.
https://gist.github.com/mritzco/88306b56c0184e66e33374d6b6ed2751

nice! You could publish this out as an NPM module that others can use via plop.load(). If you want to do that and need help, let me know.

I'll happily do! Just publish to npm or any specific requirements?

Sorry for the delay on this response. You can look at this repo as a guide to how this is done. I'm going to try to get more instructional content together soon.

@amwmedia
Ok, that took a bit longer than expected:
Could you please take a look at this repo and let me know if that would do.
Original method gets the config on the method call, while the load method passes the config on the require function. It's working with the testing file but I'm not sure it will work properly in all cases.


Clicking on the filename under a debug message isn't sending me to the file in Sublime

Thanks for the great app guys. Just reporting this bug, hope it's helpful feedback.
Describe the bug
In my ray preferences, I have specified that my IDE is Sublime.
When I am clicking the filename under a debug message within my trial version of Ray, I am expecting to be taken back to Sublime and be taken to the line of code where I added the debug message.
Versions
Ray version (you can see this in "About Ray"): 1.2.2 (Trial)
To Reproduce
Steps to reproduce the behavior:

Open ray
Set my IDE in ray to 'Sublime'
Open sublime text
Add a ray() debug message to my code
See the error within Ray
Click on the filename

Expected behavior
I'm expecting to sent back to sublime, at the right file and line number.
Screenshots

Desktop (please complete the following information):

OS: [e.g. macOS] macOS
Version [e.g. 22] 10.15.7

Additional context
Sublime Text Version 3.2.2 Build 3211

@AdrianMrn do users need to register that Sublime URL protocol?

@AdrianMrn do users need to register that Sublime URL protocol?

I believe so. @simderoemer Sublime doesn't allow opening the editor by default, could you try installing this and report back? https://github.com/saetia/sublime-url-protocol-mac
If that works, we'll find a place to add it to the docs.

I believe so. @simderoemer Sublime doesn't allow opening the editor by default, could you try installing this and report back? https://github.com/saetia/sublime-url-protocol-mac
If that works, we'll find a place to add it to the docs.

That's fixed it, thank you.
Thanks for your help and making these great tools,
S

That's fixed it, thank you.
Thanks for your help and making these great tools,
S


Serbian Dinar symbol fix

the proper way to write 100 serbian dinars is "RSD 100"

A rebased version of this has been merged to master, thanks!


Switch MySQL from nfs+bindfs to an external persistent .vdi

We've known for awhile that MySQL over NFS is terrible, but it was the best perf we could get to accomplish the following two goals:

Ensure MySQL Table Files survive a destroy / reprovision
Ensure that MySQL Table Files don't max out the size of the base box

. . . but NFS is really terrible for large file reads, and MySQL is all about large file reads.
Turns out there's a vagrant plugin for that. So in this PR we switch from MySQL being mounted over nfs+bindfs to MySQL being stored in mysql.vdi, managed by vagrant-persistent-storage.
To Test:

Export any databases from your current install you may want to keep
Pull this branch
vagrant destroy -f && vagrant up
Watch it go
Import any new DBs you need

How it actually works is seamless, I'm just not building a migration path from directory to image because honestly it's a good idea to flush out and rebuild your MySQL storage from time to time, due to how MySQL allocates and manages its storage.
Couple Notes:

Vagrant should install the plugin "vagrant-persistent-storage" on reload, if it doesn't and it freaks out run vagrant plugin install vagrant-persistent-storage. Also tell me so I can work on fixing that.
This does not delete your old /mysql directory. You'll need to clear that out whenever you think you're done with it. Try not to forget it, because it could be a lot of space.
With this new system we trade better perf for unlimited scaling storage, so the MySQL storage is capped at 32Gb. If it turns out we need more than 32Gb we can tweak it, but for now I'm considering needing 32 gigs of sql storage out of scope. (We may consider changing it to something gigantic like a 256Gb image, because it does auto-scale up, but only up.)

As for some perf stats, here's a 100-count @ 10 concurrency run of Apache Bench vs a completely uncached Drupal page with a webform embedded, with MySQL running over nfs+bindfs:
Concurrency Level:      10
Time taken for tests:   38.072 seconds
Complete requests:      100
Failed requests:        0
Total transferred:      13503200 bytes
HTML transferred:       13444600 bytes
Requests per second:    2.63 [#/sec] (mean)
Time per request:       3807.173 [ms] (mean)
Time per request:       380.717 [ms] (mean, across all concurrent requests)
Transfer rate:          346.37 [Kbytes/sec] received

Connection Times (ms)
              min  mean[+/-sd] median   max
Connect:        0    0   0.1      0       1
Processing:  2097 3751 786.7   3603    5660
Waiting:     2073 3622 764.8   3450    5500
Total:       2098 3751 786.7   3603    5660

And here's the same test with MySQL in a VDI:
Concurrency Level:      10
Time taken for tests:   33.173 seconds
Complete requests:      100
Failed requests:        0
Total transferred:      13503200 bytes
HTML transferred:       13444600 bytes
Requests per second:    3.01 [#/sec] (mean)
Time per request:       3317.280 [ms] (mean)
Time per request:       331.728 [ms] (mean, across all concurrent requests)
Transfer rate:          397.52 [Kbytes/sec] received

Connection Times (ms)
              min  mean[+/-sd] median   max
Connect:        0    0   0.0      0       0
Processing:  2174 3279 286.4   3341    3743
Waiting:     2055 3169 274.1   3227    3621
Total:       2174 3279 286.4   3341    3743

It's not amazing perf difference, but it's definitely better and the overhead is lower. I was seeing significantly less memory usage inside the box during these tests with a mounted VDI instead of nfs+bindfs.


[.Net] Fix document comment from the most recent AutoGen.Net engineer sync

Why are these changes needed?
This PR fixes the following comment/feedback from the most recent AutoGen.Net engineer sync

Adding package installing command in getting-start page && fix missing using statement

In installation page, moving nightly build to sub-section to make it less obvious

reminder to restart VSCode when creating partial class

adding missing using statement to all the rest of examples

renaming registerPrintFormatHook to RegisterPrintMessage

Include Azure.AI.OpenAI to openai examples

Removing RegisterStreaming from documentation

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 36.01%.  Consider uploading reports for the commit b58f81a to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           dotnet    #2231      +/-   ##
==========================================
- Coverage   37.36%   36.01%   -1.35%
==========================================
  Files          74       77       +3
  Lines        7481     7769     +288
  Branches     1617     1664      +47
==========================================
+ Hits         2795     2798       +3
- Misses       4445     4733     +288
+ Partials      241      238       -3

Flag
Coverage Δ

unittests
36.01% <ø> (-1.34%)
:arrow_down:

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

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


Document global.elasticsearch.useSSL in values.yaml

global.elasticsearch.useSSL isn't documented in values.yaml.
Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable)

If making change in values.yaml then Chart.yaml also needs to update version.

@anshbansal done

Thanks for the contribution.


RenderWindow::create() with Qt's QWidget::winId() crashes program on Windows

Prerequisite Checklist

[X] I searched for existing issues to prevent duplicates
[X] I searched for existing discussions on the forum to prevent duplicates
[X] I am here to report an issue and not to just ask a question or look for help (use the forum or Discord instead)

Describe your issue here
I am creating a program with Qt 6.8 and SFML, I started off with SFML 2.6 and I decided to upgrade to SFML 3.0.
Migrating gave me no issues other than a new bug that involves sf::Window::create(sf::WindowHandle Handle).
The program becomes unresponsive only when I use this specific overload.
This appears to only happen on the Windows build, as Linux doesn't experience this issue.
I found a forum post from 2016 that talks about the same issue that appears to have returned:
https://en.sfml-dev.org/forums/index.php?topic=21049.msg150558#msg150558
Sadly there are no error messages that I know of, the program just becomes unresponsive.
Your Environment

OS / distro / window manager: Windows 10
SFML version: 3.0.0
Compiler / toolchain: MingGW
Special compiler / CMake flags:

Steps to reproduce

Start Program

#include <QApplication>
#include <QWidget>
#include <QObject>
#include <QScopedPointer>

#include <SFML/Graphics.hpp>

class GameDisplay : public QWidget
{
    Q_OBJECT
public:
    explicit GameDisplay(QWidget *parent = nullptr, QString rom_url = QString()) : QWidget{parent},
                                                                                render_window(new sf::RenderWindow(sf::VideoMode({800, 600}), "OpenGL", sf::Style::Default)),
                                                                                texture({800, 600}),
                                                                                sprite(texture),
                                                                                m_initialized(false)
    {
        // Window attributes
        setAttribute(Qt::WA_PaintOnScreen);
        setAttribute(Qt::WA_OpaquePaintEvent);
        setAttribute(Qt::WA_NoSystemBackground);
        setAttribute(Qt::WA_DeleteOnClose);

        setWindowFlags(Qt::Window);
        resize(800, 600);
    };
    ~GameDisplay(){};

private:
    QScopedPointer<sf::RenderWindow> render_window;
    sf::Texture texture;
    sf::Sprite sprite;
    bool m_initialized;

protected:
    void showEvent(QShowEvent *event) override
    {
        // Initial initialization of the SFML widget
        if (!m_initialized)
        {

            render_window->create(sf::WindowHandle(winId())); // QWidget::winId() returns a native handle
            render_window->clear(sf::Color::Blue);
            render_window->draw(sprite);
            render_window->display();

        }
    };

    QPaintEngine *paintEngine() const override
    {
        return nullptr;
    };
};

#include "Main.moc" // Enable MOC for GameDisplay

int main(int argc, char **argv)
{
    QApplication a(argc, argv);

    GameDisplay w;
    w.show();

    return a.exec();
}

Expected behavior
The program should respond and SFML's output should display within the confines of Qt's QWidget, which in this case is the GameDisplay class from the example
Actual behavior
The program becomes unresponsive

This wasn't very pleasant to dig through but I think I know what is happening, but not the why. I abandoned the Qt notion here to see if I could reproduce the unresponsive window via a Win32 window being made then passing the handle to SFML. Used some source from here to make a Win32 window then passed the handle to SFML and noticed on 3.0.0 the window is totally unresponsive. When debugging I see we get trapped infinitely processing GL errors. Whereas on 2.6.2 we process a single error then report that.
I presume a default Qt app redirects the stdout or is just a Win32 app so you can't see the stdout easily, that means you would never see the stream of GL errors being shouted by SFML. I also tried this by using glfw to create a window then passing the handle to SFML and get the same result. I have no clue why this is happening but this is my leading theory. If someone wants to install Qt and verify this be my guest, but the fact the below example works fine with 2.6.2 but not 3.0.0 leads me to believe this might be the issue you're experiencing.
#include <GLFW/glfw3.h>
#define GLFW_EXPOSE_NATIVE_WIN32
#include <GLFW/glfw3native.h>

int main(void)
{
    GLFWwindow* window;

    /* Initialize the library */
    if (!glfwInit())
        return -1;

    /* Create a windowed mode window and its OpenGL context */
    window = glfwCreateWindow(640, 480, "Hello World", NULL, NULL);
    if (!window)
    {
        glfwTerminate();
        return -1;
    }

    sf::RenderWindow wind (glfwGetWin32Window(window));

    /* Make the window's context current */
    glfwMakeContextCurrent(window);

    /* Loop until the user closes the window */
    while (!glfwWindowShouldClose(window))
    {
        /* Render here */
        glClear(GL_COLOR_BUFFER_BIT);

        /* Swap front and back buffers */
        glfwSwapBuffers(window);

        /* Poll for and process events */
        glfwPollEvents();
    }

    glfwTerminate();
    return 0;
}

I assume the stuck loop is related to #3247

I assume the stuck loop is related to #3247

Yep it is

@Wolfmyths if you attach a debugger and pause it during the unresponsiveness, you can confirm my findings by telling us if the debugger halts within SFML's code for processing GL errors (glCheckError function in SFML).

Yup, prints the same error on SFML 3.0
An internal OpenGL call failed in "RenderWindow.cpp"(113).
Expression:
   glGetIntegerv(GLEXT_GL_FRAMEBUFFER_BINDING, reinterpret_cast<GLint*>(&m_defaultFrameBuffer))
Error description:
   GL_INVALID_OPERATION
   The specified operation is not allowed in the current state.

I presume a default Qt app redirects the stdout or is just a Win32 app so you can't see the stdout easily, that means you would never see the stream of GL errors being shouted by SFML.

Yes, you have to set an environmental variable for it to capture stdout: QT_WIN_DEBUG_CONSOLE=1
I use VSCode so I had to write it into launch.json

@Wolfmyths Thanks, that at least confirms the cause of the problem. Not sure what the best course is here without understanding why.

Did some digging and I have the answers we need for now. This is because we're trying to poll errors without a valid GL context. According to this resource:

Note: No OpenGL function call is valid when an OpenGL Context is not made current. Depending on the platform, a call to glGetError without having a context made current may crash or return any value, including indefinitely returning a valid OpenGL error code. So if you see an infinite loop in such a function, this is likely the reason why

Having taken a look through the code it seems the flow is :

https://github.com/SFML/SFML/blob/3.0.0/src/SFML/Window/Window.cpp#L126 override of create on the sf::Window
We then call https://github.com/SFML/SFML/blob/3.0.0/src/SFML/Window/WindowBase.cpp#L114
Then we call the initialize on sf::WindowBase, which calls onCreate()
onCreate is an inherited method which sf::RenderWindow has a implemented which tries to do some GL stuff which checks for an error https://github.com/SFML/SFML/blob/7f1162dfea4969bc17417563ac55d93b72e84c1e/src/SFML/Graphics/RenderWindow.cpp#L109

All of that happens before the call to create the context here https://github.com/SFML/SFML/blob/3.0.0/src/SFML/Window/Window.cpp#L129

Just as an extra note, seems like the return value of winId() is kind of "unreliable" and might change, if it's a QWidget:

This value may change at run-time. An event with type QEvent::WinIdChange will be sent to the widget following a change in window system identifier.

This doesn't seem to apply to QWindow, though.

Just as an extra note, seems like the return value of winId() is kind of "unreliable" and might change, if it's a QWidget:

QWidgets are typically given their own QWindow if they don't have a parent, so in this example it shouldn't change since the widget isn't moving windows
There are a couple ways for a QWidget to be given a window handle, they aren't given by default apparently.
My example uses option 5 already setAttribute(Qt::WA_PaintOnScreen);