fix: remove steps to loosen orgpolicies

After the dependency on App Engine Flex is removed in the merge of #72, we can remove the setup steps that check and loosen org policy to accommodate App Engine Flex.

Accidentally opened against wrong branch


[AutoPR mysql/resource-manager/postgresql/resource-manager] ORCASQL VNET with API Version 2017-12-01 for Public Preview.

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3138

(message created by the CI based on PR content)

Codecov Report

Merging #2626 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2626   +/-   ##
=======================================
  Coverage   56.09%   56.09%
=======================================
  Files        6979     6979
  Lines      147807   147807
=======================================
  Hits        82906    82906
  Misses      64901    64901

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 3110473...07c8b57. Read the comment docs.


Add walkthrough verifications to categorical market creation

CH issue

Pull Request Test Coverage Report for Build 962

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

Totals

Change from base Build 959:
0.0%

Covered Lines:
3594

Relevant Lines:
4936

💛  - Coveralls


does Tire work with Mongomapper?

I'd like to use MongoMapper instead of Mongoid for a project. Does Tire support MongoMapper?

Yes, I'm using Tire with MongoMapper and so far I haven't had any issues. I'm using the ActiveModel integration as described in the README.

Thanks! It's good to hear that there are no known gotchas.


Setup merge automation for this repo

Use an OWNERS file, /approve, /lgtm to allow people to merge PR's into this repo:

If there isn't an OWNERS file, create one and populate it with people who already have write access to this repo (as approvers)
Configure prow to turn on the "approval" plugin for this repo
Configure prow/tide to include this repo in the tide query

In addition, I'd like to enable the majority of prow plugins that we use in the kubernetes org, excluding:

golint (no sense turning this on until we have golang code in here)
slackevents (until we have automated merges setup, no sense alerting on manual merges)
trigger (not going to kick off any jobs for this repo yet)
milestonestatus (I don't think we have any interest in using the status/foo labels)
sigmention (we don't have all the sig teams setup in this org)

Hello @spiffxp
Regarding your list of things you wanted to enbable:

golint (no sense turning this on until we have golang code in here)

We have some golang now. Can we just enable that? We are happy to create a PR into https://github.com/kubernetes/test-infra/blob/master/prow/plugins.yaml if needed. Or is there something more involved?

slackevents (until we have automated merges setup, no sense alerting on manual merges)

We have automatic merges set up now. Which channel should alerts go to?

re: golint, PR's to plugins.yaml are the way to go, don't let me hold you up there
re: channel for manual merges, do you have a channel dedicated to the testing-commons subproject or this repo? we could probably create one if you'd like
do you all lurk in sig-testing or testing-ops? we could spam one of those instead

/shrug

/unshrug

/close
This has been done for a while


making client works with CentOS

@tmoreau did you want this reviewed and merged? or no?

you can discard, has been replaced by PR #33
sorry for the noise...

np, https://github.com/shortdudey123/chef-gluster/commit/972713ccb83873b569e71045e34ebbc02207a464 would be nice to have too :)

ah see that in 34, nvm


TRG 7.07 Legal notice for documentation

Could you please recheck and add the notice section in your documentation?
https://eclipse-tractusx.github.io/docs/release/trg-7/trg-7-07#documentation
## NOTICE

This work is licensed under the [Apache-2.0](https://www.apache.org/licenses/LICENSE-2.0).

- SPDX-License-Identifier: Apache-2.0
- SPDX-FileCopyrightText: 202x,202y <MyCompany>
- SPDX-FileCopyrightText: 202x,202y Contributors to the Eclipse Foundation
- Source URL: https://github.com/eclipse-tractusx/<myrepo>

examples:

https://github.com/eclipse-tractusx/bpdm/blob/bpdm-3.1.2/docs/decision-records/004-openapi_descriptions.md
https://github.com/eclipse-tractusx/bpdm/blob/bpdm-3.1.2/docs/edc/README.md
(https://github.com/eclipse-tractusx/bpdm/blob/bpdm-3.1.2/docs/decision-records/004-openapi_descriptions.md

Add a NOTICE section (post or preamble) into your dedicated documentation file

input from the TRG

Add a NOTICE section (post or preamble) into your dedicated documentation file

simple writing each documentation file under your /docs folder

@FaGru3n done


Add children option to state configs

Would be an array of states (strings/objects). This would allow us to attach a child to multiple parents, thereby allowing amazing code reuse (and opening up the opportunity for truly pluggable modules).
If you create a path-relative (agnostic) implementation of an 'edit' module, you could attach it to both a person and project and task module and get some excellent code reuse across the board.

Can this be closed or is it still needed?


Entrez proteomes_type_strains.tsv.gz moved

The file proteomes_type_strains.tsv.gz that is requested by type_strains.sh is no longer available on the Entrez-database-server. When running proteomes_type_strains.tsv.gz this results in a HTTP/2.0 framing error from Curl. Trying to access the URL on Chrome leads to a HTTP 404 error.
Note: the full URL that's no longer available is: https://eutils.ncbi.nlm.nih.gov/entrez/eutils/proteomes_type_strains.tsv.gz
Original issue by @pverscha on Mon Aug 05 2019 at 15:14.

This URL seems to be back up now, so I'm closing this issue.


Please add python 3 support

Hi,
We are using DIGITS in our organization along with internally developed applications. The lack of support for Python3 for DIGITS is preventing us too from using it. Do you have any plan to port it to python 3?

We don't have plans to do this anytime soon, no. However, you're the first to ask for it - we might consider it if enough people ask or upvote this request.
If you'd like to contribute small, incremental updates towards Python3 support like e7b3466dbf82dffad0ffb7d771ba589d7c255968 (from https://github.com/NVIDIA/DIGITS/pull/501), I'd be happy to review them.

Thank you. I will send PRs with incremental updates.

Well, we're also using python 3 and all libraries we use (numpy, scipy, scikit-learn, theano and many others) have python 3 support for some time now, we'd love it if Digits/Caffe went the same way.
I don't think that upvotes for this request actually represents the need.

I would like to have python3 support too.

+1 more for Python 3 support

+1

+1
Thanks for considering supporting Python 3.  I'm sure you're aware that the original EOL for python 2 was set to 2015. It has been extended to 2020 but basically python 2 is dead. All major AI frameworks (including Caffe, TF, CNTK) support python 3 so it would be great to see DIGITS move in that direction as well.  Limiting support to python 2 only muddles the python environments where we have to keep a special environment just for DIGITS while everything else runs on python 3.

+1

+1.  I'm avoiding starting any projects in Python2 for the reasons listed above.

I went ahead and made a fork and hacked my way through most of the most glaring Python2/3 errors that were thrown up.
https://github.com/jed-frey/DIGITS/commits/master

I spent the weekend hacking through more 2-->3 migration stuff. I've tested it with the Dolphin & Seahorses Have Fun with Machine Learning classification example. Next free time sprint I will try to work through all the other examples.
I spent the weekend hacking through more 2->3 migration stuff to the point where I'm happy tagging it as working.
https://github.com/jed-frey/DIGITS/tree/v5.1-dev-py3
I've tested it with the Dolphin & Seahorses Have Fun with Machine Learning classification example. Next free time sprint I will try to work through all the other examples.
If anyone is interested in using DIGITS on Python 3 it would be a big help if you could try to use your use cases to figure out where the pain points are and report any additional bugs.
Tested Environment:

Ubuntu 16.04 x86_64
CUDA 8.0 w/cudnn 5.1
Python 3.5.1 (apt-get install python3)
Numpy 1.11.0 (apt-get install python3-numpy)

I built OpenCV and Caffe for Python3, CUDA8, etc. For simplicity here are the binaries. Install opencv with dpkg and exatract caffe-distribute.

https://jed-frey.github.io/files/opencv3_3.2.0-1_amd64.deb build script
https://jed-frey.github.io/files/caffe-distribute.tar.bz2 build script

If you run into any problems or have any questions open an issue on my forked repo.
https://pythonclock.org/

Dear Jef-rey
can i tried to use in windows 8.1 ? and i already to install all dependencies for this. using python 3.5.3, and i need to set to cpu only
here the result
C:\DIGITS>python -m digits
Traceback (most recent call last):
File "C:\Users\MyPC\AppData\Local\Programs\Python\Python35\lib\runpy.py", line 193, in _run_module_as_main "main", mod_spec)
File "C:\Users\MyPC\AppData\Local\Programs\Python\Python35\lib\runpy.py", line 85, in run_code                                                                   exec(code, run_globals)
File "C:\DIGITS\digits_main.py", line 72, in main()                                                                        File "C:\DIGITS\digits_main_.py", line 57, in main import digits.webapp                                                          File "C:\DIGITS\digits\webapp.py", line 48, in import digits.views  # noqa                                                   File "C:\DIGITS\digits\views.py", line 648, in  app.register_error_handler(code, handle_error)
File "C:\Users\MyPC\AppData\Local\Programs\Python\Python35\lib\site-packages\flask\app.py", line 1170, in register_error_handler self._register_error_handler(None, code_or_exception, f)
File "C:\Users\MyPC\AppData\Local\Programs\Python\Python35\lib\site-packages\flask\app.py", line 64, in wrapper_func return f(self, *args, **kwargs)
File "C:\Users\MyPC\AppData\Local\Programs\Python\Python35\lib\site-packages\flask\app.py", line 1185, in _register_error_handler exc_class, code = self._get_exc_class_and_code(code_or_exception)             File "C:\Users\MyPC\AppData\Local\Programs\Python\Python35\lib\site-packages\flask\app.py", line 1104, in _get_exc_class_and_code exc_class = default_exceptions[exc_class_or_code]                           KeyError: 402                                                                                C:\DIGITS>python -m digits-devserver
C:\Users\MyPC\AppData\Local\Programs\Python\Python35\python.exe: No module named digits-devserver
#using digits-devserver
C:\DIGITS>python digits-devserver
File "digits-devserver", line 6 python -m digits $@  ^ SyntaxError: invalid syntax
Thx

^ That's not related to either Python3 or Windows. You're trying to run a bash script (see the first line) with Python. Don't do that. Just start the server with python -m digits like the script is trying to do.

https://pythonclock.org/

does digits 6.0 support python3?

No. You can't even run setup.py because of the Python2-isms.
python setup.py install
Traceback (most recent call last):
  File "setup.py", line 11, in <module>
    execfile(os.path.join(LOCAL_DIR, 'digits', 'version.py'), {}, version_locals)
NameError: name 'execfile' is not defined

And requirements.txt has a Python2 only version of pydot selected.
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-cglb3f20/pydot/setup.py", line 6, in <module>
        import pydot
      File "/tmp/pip-install-cglb3f20/pydot/pydot.py", line 31
        except Exception, e:
                        ^
    SyntaxError: invalid syntax

And if you skip setup.py, fix the versions in requirements, it fails to start because of the print functions.
SyntaxError: Missing parentheses in call to 'print'
My DIGITS5 Python3 branch above still works. I haven't played with machine vision in a while and don't currently have the motivation ($) to work on a Python3 DIGITS6 release.
I just tossed up a Bounty Source if it's something DIGITS users really want: https://www.bountysource.com/issues/57707267-digits6-update.
[And the reason I haven't submitted any pull requests of my Python 3 changes is because of the CLA]

Please add python3 support this has been open since 2016....
It's 2018 guys...

+1 - Tensoflow is only available on Python 3 for Windows.  It took me ages just to get your out-dated DIGITS to work on Windows because your digits has such old requirements it's hard to cobble together and source the old libraries.  After all of that I find that it won't run Tensorflow - so I feel that I wasted my time on your training (which was digits based), I've wasted my time installing DIGITS and for what?  Like the last guy said, it's 2018 and the request has been there since 2016 - which seems like the last date that you seriously updated DIGITS anyway.

From the errors people are reporting, it seems the only real change needed to upgrade Digits for Python 3 is the syntax for print. Since in Python3 print is used as print('String') instead of print 'String'
tl;dr add parentheses to inside print statements?
Did anyone find any errors that's not related to syntax? I'm thinking of just running a script through Digits' source code to update the print statements.

+1 (like)

@xemose:

From the errors people are reporting

Look at my fork and Python3 version of DIGITS 5. There's a lot more "Python 2 isms" in there than just print. (Which is trivial to support both with __future__ module). The only reason 'print' is what is reported is that is what fails first.
I did the conversion in about a weekend. There are probably edge use cases that will fail. But I don't have the motivation to provide Nvidia free work on DIGITS 6. If they really see value in this project it should be trivial to get an Intern to work on it. But if NVidia doesn't care enough to do that at the minimum it's not worth my time to do it for them.
Alternatively, https://www.bountysource.com/issues/57707267-digits6-update

Also need python3 support please.

Also need python3 support please.

Jetson Nano doesn't have TensorFlow support for Python2.7. so it's impossible to run DIGITS with TensorFlow support on the Nano

Officially? Created a fork for DIGITS on Python3.

506 open issues.
26 Pull Requests.
You will need to send a signed copy of the Contributor License Agreement to<EMAIL_ADDRESS>before your change can be accepted.

Developers absolutely welcome because NVIDIA did a great of obfuscation with the lack of documentation everywhere.

Sorry to bother you, but I recently face the same problem to make DIGITS work with python3, however when I go to your link, I find the readme says that "it is no longer developed since there are other better options in 2019", may I ask what are the better options, you mean DIGITS provide support for python 3 or other applications?
Thanks in advance.

@richardcoder, Sorry I didn't get back to you sooner. For some reason GitHub didn't show me the notification in this thread.
A bit of a backstory on the existence of DIGITS3. I started playing with machine learning back in 2017. I found DIGITS and immediately loved it. I got tinkering under the hood and just wanted to see how much it would take to convert to Python 3. A long the way I learned how to use Flask, blueprints, etc.
However all of my changes were a big tear up of the code, it wasn't mergeable. NVIDIA seemed to have no interest in doing any work internally to convert to Python3, so I lost interest. This seems to be a perpetual problem with large companies and open source. It's not an immediate profit line in a budget so it's not "worth" it.
Better options for a 1:1 copy of digits with a web gui, user accounts, etc. Nothing I know of (but that's not my use case.)
PyTorch, Tensorflow and other higher level libraries have gotten a lot better since 2017 so I've moved to them for my ML stuff.
I'd be happy to have a hackathon with Nvidia updating it to Python3 in a crash week for some free pizza or a TITAN card, but it doesn't seem like NVIDIA really cares much about this product anymore. For example it supports "Ubuntu 14.04, 16.04". Ubuntu's next LTS 2020 is coming. I've been on 18.04 for 16 months.
DIGITS seems to be on life support and NVIDIA pulled the rug out from under those that built their work around it.
Also, I don't want sign away my contributor rights to NVIDIA.
2¢

+1 more for Python 3 support

#2210  I am having similar issue making Nvidia Digits work with python3 after I have installed caffe and tensorflow with python 3. Is anyone been able to successfully build, install and use Nvidia Digits with Caffe, tensorflow on python 3?


fix(app): Deprecated Sass api

See the changelog of sass here https://github.com/webpack/webpack.js.org/pull/4402:

Deprecate passing non-% numbers as lightness and saturation to hsl(), hsla(), color.adjust(), and color.change(). This matches the CSS specification, which also requires % for all lightness and saturation parameters. See [the Sass website][color-units] for more details.

Thanks!

Thanks!


powershell: 7.2.4 -> 7.3.0

Description of changes
This PR is similar to #183738, with the exception that I actually updated the hashes for all platforms.

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[x] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 202733 run on x86_64-linux 1

1 package failed to build:

powershell

Oops, thanks, I'll fix that

Result of nixpkgs-review pr 202733 run on x86_64-linux 1

1 package failed to build:

powershell

Can the version be bumped to 7.3.1?


Multiproxy infrastructure [do not merge]

Description
Multiproxy infrastructure
Design
A new variable in .env files called VALIDATOR_PROXY_COUNTS has replaced PROXIED_VALIDATORS. This variable allows specifying varying numbers of proxies per validator with the format <# of validators>:<proxy count>,<# of validators>:<proxy count>;…. For example, to have 2 validator with 5 proxies and 3 validators with 4 proxies, it would be: VALIDATOR_PROXY_COUNTS=2:5,3:4
Celotool expands VALIDATOR_PROXY_COUNTS into an array where each index corresponds to a validator, and each element is the number of proxies for the corresponding validator. Continuing the previous example, the array would be: [5, 5, 4, 4, 4]
K8s Setup
If n validators are to be proxied, the first n validators of the validator statefulset are still proxied. Each proxied validator has a corresponding statefulset for its proxies (rather than the previous setup where there was 1 statefulset and each replica was associated with the corresponding validator).
Terraform VM Setup
Terraform allows you to create multiple replicas of a resource in two ways: using count, which will create the specified number of replicas and allow specific resources to be indexed by their count.index (in the range of [0, count - 1]). The other way is using for_each, which allows a developer to specify a list or map and will create a resource for each corresponding list element or map key. I changed the full node Terraform module to use for_each to allow us to avoid having to map index ranges to validators, which would also make scaling up proxies harder.
Specifying proxies for the vm-exec and ssh-vm-node celotool commands is done by providing <validator index>:<proxy index> for the index parameter
Tested
Ran public, private k8s testnets. Ran terraform testnets
Other changes

Removed .env.integrationtesting, which is no longer used
Removed unused helm variables
Removed some unused env vars
Created a new env var for specifying the disk size in GB for nodes
Made max_peers configurable for VM tx nodes / proxies
Passed some arrays to helm as actual arrays rather than joined strings or variable names with numbers in them

Related issues

Fixes #2597

Blocked by the multiproxy PR: https://github.com/celo-org/celo-blockchain/pull/821
Backwards compatibility
Not backwards compatible. One flag to geth has been changed, and proxy keys are generated differently

Closing in favor of the latest a greatest version #4150


Extractor configurability

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe
Association extractor is useful to modify to e.g. add a circuit-breaker, implement caching and for a few other things. Right now it's somewhat complicated to override/modify.
Describe the feature you'd like to see implemented
Right now, there are two types of extractors: AssociationExtractor and AutoExtractor (along with some sub-extractors).
Their roles are somewhat intertwined (AssociationExtractor calls the AutoExtractor, for example), and only one of them can be modified globally.
I'd propose a stricter separation, splitting it up into (probably) three extractors:

FieldExtractor
AssociationExtractor
ValueExtractor (or DataExtractor)

The default ValueExtractor would encapsulate the logic currently existing in AutoExtractor, and I'd move the logic of deferring to block to field/association extractors. ValueExtractor could also be named something like DataExtractor instead.
Example
This is a rough sketch, in reality it would probably be slightly different.
class ValueExtractor < Extractor
  def extract(field_name, object, local_options, options = {})
    if object.is_a?(Hash)
      value = extract_hash(field_name, object, local_options, options)
    else
      value = extract_send(field_name, object, local_options, options)
    end
  end

  private

  def extract_hash(field_name, object, _local_options, _options = {})
    object[field_name]
  end

  def extract_send(field_name, object, _local_options, _options = {})
    object.public_send(field_name)
  end
end

class FieldExtractor < Extractor
  include EmptyTypes

  def initialize
    @extractor = Blueprinter.configuration.field_extractor_default.new # `ValueExtractor.new` by default
  end

  def extract(field_name, object, local_options, options = {})
    if options[:block]
      value = options[:block].call(object, local_options)
    else
      value = @value_extractor.extract(field_name, object, local_options, options)
    end

    value = @datetime_formatter.format(value, options)
    use_default_value?(value, options[:default_if]) ? default_value(options) : value
  end

  private

  def default_value(field_options)
    field_options.key?(:default) ? field_options.fetch(:default) : Blueprinter.configuration.field_default
  end
end

class AssociationExtractor < Extractor
  include EmptyTypes

  def initialize
    @extractor = Blueprinter.configuration.field_extractor_default.new # `ValueExtractor.new` by default
  end

  def extract(association_name, object, local_options, options = {})
    options_without_default = options.except(:default, :default_if)

    # Merge in assocation options hash
    local_options = local_options.merge(options[:options]) if options[:options].is_a?(Hash)

    if options[:block]
      value = options[:block].call(object, local_options)
    else
      value = @value_extractor.extract(association_name, object, local_options, options)
    end

    return default_value(options) if use_default_value?(value, options[:default_if])

    view = options[:view] || :default
    blueprint = association_blueprint(options[:blueprint], value)
    blueprint.prepare(value, view_name: view, local_options: local_options)
  end

  private

  def default_value(association_options)
    return association_options.fetch(:default) if association_options.key?(:default)

    Blueprinter.configuration.association_default
  end

  def association_blueprint(blueprint, value)
    blueprint.is_a?(Proc) ? blueprint.call(value) : blueprint
  end
end

Describe alternatives you've considered
No response
Additional context
No response

Happy to discuss this in more detail, and elaborate more.

Friendly ping @lessthanjacob @njbbaer @ritikesh
If you think this is a reasonable suggestion, let me know! If so, I'd propose these steps:

I'll flesh out the proposed change in some more detail (written, in this issue)
You get another chance to provide more feedback, and if you're happy we'll open a PR

For reference, there are also a few other issues that I've opened, where I'd also be happy to get your input.
Full list

https://github.com/procore-oss/blueprinter/issues/403
https://github.com/procore-oss/blueprinter/issues/404
https://github.com/procore-oss/blueprinter/issues/405
https://github.com/procore-oss/blueprinter/issues/406
https://github.com/procore-oss/blueprinter/issues/407

I think this would be a reasonable refactor.
@njbbaer @ritikesh any thoughts/concerns here?

Agreed, we are open to contributions @sandstrom.

@lessthanjacob @ritikesh Thanks!
I'll have a chat with a colleague of mine, and we'll try to get back with a proposal.

Still relevant!

Not stale

I've been experimenting with this V2 extractor in #479. I think it covers the concerns in this issue.
Note that V2 (currently - nothing is set in stone) makes a distinction between a has_one/belongs_to association (aka object) and a has_many (aka collection). Those terms may change, but the distinction is proving useful.
module Blueprinter
  module V2
    # The default extractor and base class for custom extractors
    class Extractor
      #
      # Extract a field value.
      #
      # @param ctx [Blueprinter::V2::Context]
      #
      def field(ctx)
        if ctx.field.value_proc
          ctx.blueprint.instance_exec(ctx.object, ctx.options, &ctx.field.value_proc)
        elsif ctx.object.is_a? Hash
          ctx.object[ctx.field.from]
        else
          ctx.object.public_send(ctx.field.from)
        end
      end

      #
      # Extract an object field (singular association) value.
      #
      # @param ctx [Blueprinter::V2::Context]
      #
      def object(ctx)
        field ctx
      end

      #
      # Extract a collection field (array of objects) value.
      #
      # @param ctx [Blueprinter::V2::Context]
      #
      def collection(ctx)
        field ctx
      end
    end
  end
end

I think it looks good!
I would consider a name other than object for the singular association. Maybe has_one/has_many, single_item/multiple_items, item/collection, element/collection.
The things we've done in our extractors are:

Circuit breaker (to avoid infinite recursion)
Caching layer, to avoid doing some costly lookups multiple times
Formatting

I think those would be covered here.
Any code example of what context would be?

@sandstrom Again, this is very much a WIP, but context is currently defined as:
#
# The Context struct is used for most extension hooks and all extractor methods.
# Some fields are always present, others are context-dependant. Each hook and extractor
# method will separately document which fields to expect and any special meanings.
#
# blueprint = Instance of the current Blueprint class (always)
# field = Field | ObjectField | Collection (optional)
# value = The current value of `field` or the Blueprint output (optional)
# object = The object currently being evaluated (e.g. passed to `render` or from an association) (optional)
# options = Arbitrary options passed to `render` (always)
# instances = Allows this entire render to share instances of Blueprints and Extractors (always)
# data = A Hash to for extensions, etc to cache render data in
#
Context = Struct.new(:blueprint, :field, :value, :object, :options, :instances, :data)


Spike: Parse a project with Gradle build file

Try to parse a project that uses Gradle instead of Maven to learn about problems and answer questions like:

Can Gradle and Maven be abstracted behind the same API? BuildFile
How must the parser change, are two separate parsers required, how does a common API look like

It makes sense to look into Gradle support before implementing multi module projects only in Maven to learn about implications early on.

With OpenRewrite 7.26.0 the support for Gradle is not as complete as for Maven.
Gradle files are parsed as Groovy G.CompilationUnits and the operations for Gradle, like adding plugins or getting dependencies, must be implemented as Recipes/Visitors for these Groovy files.
To support the same API for Gradle as for Maven many Recipes/Visitors are missing and obviously the API would need to diverge or work on the lowest common denominator where possible, e.g. with scopes for dependencies.
Not now...


compilation error: ‘runtime_error’ is not a member of ‘std’

With the following versions
ClickHouse server version <IP_ADDRESS> (official build).
gcc (GCC) 10.1.0
psql (PostgreSQL) 12.2
The cmake runs just fine:
-- The C compiler identification is GNU 10.1.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc - works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Setting clickhouse_fdw build type -
-- The CXX compiler identification is GNU 10.1.0
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ - works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found PkgConfig: /usr/bin/pkg-config (found version "1.6.3")
-- Checking for module 'libcurl'
--   Found libcurl, version 7.70.0
-- Checking for module 'uuid'
--   Found uuid, version 2.35.1
-- Configuring done
-- Generating done
-- Build files have been written to: /home/ctlaltdefeat/clickhouse_fdw/build

But I get the following error when running make:
Scanning dependencies of target clickhouse_fdw_sql
[  1%] Generating ../clickhouse_fdw--1.2.sql
[  1%] Built target clickhouse_fdw_sql
Scanning dependencies of target lz4-lib
[  3%] Building C object src/clickhouse-cpp/contrib/lz4/CMakeFiles/lz4-lib.dir/lz4.c.o
[  4%] Building C object src/clickhouse-cpp/contrib/lz4/CMakeFiles/lz4-lib.dir/lz4hc.c.o
[  6%] Linking C static library liblz4-lib.a
[  6%] Built target lz4-lib
Scanning dependencies of target cityhash-lib
[  7%] Building CXX object src/clickhouse-cpp/contrib/cityhash/CMakeFiles/cityhash-lib.dir/city.cc.o
[  9%] Linking CXX static library libcityhash-lib.a
[  9%] Built target cityhash-lib
Scanning dependencies of target clickhouse-cpp-lib-static
[ 10%] Building CXX object src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/base/coded.cpp.o
[ 12%] Building CXX object src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/base/compressed.cpp.o
[ 14%] Building CXX object src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/base/input.cpp.o
[ 15%] Building CXX object src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/base/output.cpp.o
[ 17%] Building CXX object src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/base/platform.cpp.o
[ 18%] Building CXX object src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/base/socket.cpp.o
[ 20%] Building CXX object src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/columns/array.cpp.o
[ 21%] Building CXX object src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/columns/date.cpp.o
[ 23%] Building CXX object src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/columns/decimal.cpp.o
[ 25%] Building CXX object src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/columns/enum.cpp.o
[ 26%] Building CXX object src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/columns/factory.cpp.o
In file included from /home/ctlaltdefeat/clickhouse_fdw/src/clickhouse-cpp/clickhouse/columns/factory.cpp:12:
/home/ctlaltdefeat/clickhouse_fdw/src/clickhouse-cpp/clickhouse/columns/nothing.h: In member function ‘virtual void clickhouse::ColumnNothing::Save(clickhouse::CodedOutputStream*)’:
/home/ctlaltdefeat/clickhouse_fdw/src/clickhouse-cpp/clickhouse/columns/nothing.h:46:20: error: ‘runtime_error’ is not a member of ‘std’
   46 |         throw std::runtime_error("method Save is not supported for Nothing column");
      |                    ^~~~~~~~~~~~~
make[2]: *** [src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/build.make:213: src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/columns/factory.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:284: src/clickhouse-cpp/clickhouse/CMakeFiles/clickhouse-cpp-lib-static.dir/all] Error 2
make: *** [Makefile:150: all] Error 2

Should be fixed, thanks


PR: Bump minimal required version of python-lsp-server (Dependencies)

Description of Changes

This is necessary to fix building our conda-based installers both on 5.x and master.
The new constraint is >=1.8.0,<1.9.0

Issue(s) Resolved
Follow up of PR #21330.
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @ccordoba12.

@ccordoba12, I think line 254 in setup.py also needs updating, probably to >=1.8.0,<1.10.0. Ping @CAM-Gerlach .

 Please open a new PR to fix that.


pysqlite3-binary compatible with Python 3.11

Could you provide a version of the pysqlite3-binary that is compatible with Python 3.11? I would appreciate it very much :)

We already do, https://pypi.org/project/pysqlite3-binary/#files


Franz support for Amazon Chime

Amazon Chime web feature as service in Franz

Hello Bryan,
From the "New issue" template that you obviously didn't read : This repository is only for the Franz client. Please use this form ( https://bitly.com/franz-service-request ) for service requests or check out the guide ( https://github.com/meetfranz/plugins ) to create your own service integration.
You can close your issue :slightly_smiling_face:


use torch.nn.functional.one_hot rewrite convert_to_one_hot

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Torch has the implementation of one-hot, here we can replace the one-hot implemented by MMCLS itself
Modification
as described above
BC-breaking (Optional)
Does the modification introduce changes that break the backward compatibility of the downstream repositories?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here and update the documentation.
Checklist
Before PR:

[ ] Pre-commit or other linting tools are used to fix the potential lint issues.
[ ] Bug fixes are fully covered by unit tests, the case that causes the bug should be added in the unit tests.
[ ] The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[ ] The documentation has been modified accordingly, like docstring or example tutorials.

After PR:

[ ] If the modification has potential influence on downstream or other related projects, this PR should be tested with those projects, like MMDet or MMSeg.
[ ] CLA has been signed and all committers have signed the CLA in this PR.

 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.

Thank you for your contribution, please fix the lint refer to the guide before we review it.

Thank you for your contribution, please fix the lint refer to this guide before we review it.

@Ezra-Yu
I have followed the lint hint to make changes to the codes format, but it looks like the tests didn't restart automatically, what else do I need to do?

After installing pre-commit, the pre-commit does not work until you create a commit,
you could run
pre-commit run --all-files

in '$MMClassification' to check if there is any style error. then fix it and commit it.

cannot understand this error:
yapf.....................................................................Failed
- hook id: yapf
- files were modified by this hook

@Ezra-Yu

you can let the L117 in one line:
    one_hot_targets = F.one_hot(targets.long().squeeze(-1), num_classes=classes)

and run pre-commit run --all-files twice, the pre-commit would modify this line automatically.
it would be:
    one_hot_targets = F.one_hot(
        targets.long().squeeze(-1), num_classes=classes)

That means the pre-commit would modify the code automatically.
you can let the L117 in one line:
    one_hot_targets = F.one_hot(targets.long().squeeze(-1), num_classes=classes)

and run pre-commit run --all-files twice in command, the pre-commit would modify this line automatically. it would be:
    one_hot_targets = F.one_hot(
        targets.long().squeeze(-1), num_classes=classes)

doesn't seem like a necessary change, currently I cannot install&use pre-commit successfully for some permission problem, so is there a work-around?

me permission problem, s

OK, get it. We would modify it later when reviewing.

me permission problem, s

OK, get it. We would modify it later when reviewing.

that's great help


Was Historic Rates for GDAX ever implemented?

I found this issue from July, but it seems like the feature wasn't implemented? #1580
Any plans for it?

Closing as GDAX became coinbasepro


activeworkspace IPC/hyprctl command

Yes I know this is quite easy to replicate, but it would be more concise and probably more performant if it was built in to the compositor!

hyprctl -j monitors | jq .[0].activeWorkspace.id
If you are using only 1 monitor. It would be nice to have this be a hyprctl command because having to call jq multiple times when you have multiple monitors is very heavy. For example this is what it looks like with 2 monitors.
if [ $(hyprctl -j monitors | jq .[0].focused) = "true" ]; then
	hyprctl -j monitors | jq .[0].activeWorkspace.id)
else
	hyprctl -j monitors | jq .[1].activeWorkspace.id)
fi

The more monitors you have the more you have to pipe into jq and that is painfully resource intensive and slow.

Yeah thats how I have it implemented with Hyprland-rs

After messing more with jq I found a way faster command
hyprctl -j monitors | jq '.[] | select(.focused == true) | .activeWorkspace.id'


fix: unset row_limit when it's not a number

🐛 Bug Fix
Recently Superset was changed to not allow row_limit to be null, it can only be unset or a number. This fixes the library to match it
to: @williaster @ktmud @villebro @kristw

My apologies, apparently one of my changes caused this problem. I'll try not to make breaking changes to QueryContext/QueryObject going forward without including all central stakeholders in the review process. Who would be the right person(s) to tag when making changes to these classes?

thanks @villebro. I think tagging @kristw or myself or @ktmud would be good 👍 (apologies for not keeping a close pulse on the refactoring, I think you may already be tagging Krist + Jesse)

I think ideally there would be some integration tests between the main repo + these packages, but I think that's likely easiest once the API is refactored and the app itself is using these client calls.


Handle zip file

If you send in a zip file instead of nupkg file you will get error.
2022-11-15T13:43:44.4071794Z
2022-11-15T13:43:44.4072147Z
2022-11-15T13:43:44.4111598Z Loaded cms package:
2022-11-15T13:43:44.4112132Z
2022-11-15T13:43:44.4112363Z
2022-11-15T13:43:44.7136163Z ##[error]Publish-Package : Cannot bind argument to parameter 'PackageFile' because it is null.
In EpinovaDxpDeploymentUtil.ps1 function Publish-Package it is only looking for nupkg files. It would be nice to also check for zip files and handle that. Rename or something simular...

@ovelartelius any updates on this? :)

Get-Item [filepath] | Rename-Item -newname { [io.path]::ChangeExtension($_.name, "nupkg") }

The error when try to upload zip with old task:
##[error]Exception: API call failed! The error was: Response status code does not indicate success:
400 (Bad Request). StatusCode: 400, ReasonPhrase: 'Bad Request', Version: 1.1,
Content: System.Net.Http.HttpConnectionResponseContent, Headers: {   Date: Mon,
11 Sep 2023 11:50:28 GMT   Connection: keep-alive   Cache-Control: no-cache
Pragma: no-cache   Set-Cookie:
ARRAffinity=34dc192faaa9d0b965caeb984296252404f91dd2795ce26d868945f67daac397;Pat
h=/;HttpOnly;Secure;Domain=paasportal.episerver.net   Set-Cookie:
ARRAffinitySameSite=34dc192faaa9d0b965caeb984296252404f91dd2795ce26d868945f67daa
c397;Path=/;HttpOnly;SameSite=None;Secure;Domain=paasportal.episerver.net
Strict-Transport-Security: max-age=31536000; includeSubDomains;
X-AspNet-Version: 4.0.30319   X-Powered-By: ASP.NET   X-XSS-Protection: 1
X-Content-Type-Options: nosniff   X-Frame-Options: SAMEORIGIN
Referrer-Policy: no-referrer   X-Permitted-Cross-Domain-Policies: none
CF-Cache-Status: DYNAMIC   Server: cloudflare   CF-RAY: 804fb17b1a16d512-CDG
Content-Type: application/json; charset=utf-8   Content-Length: 123   Expires:
-1 }

Will now find *.zip file and rename it to *.nupkg.
No *.cms.*.nupkg found. Will try to find *.cms.*.zip
Found /home/vsts/work/r1/a/_Build/DeployableArtifacts/Site.cms.app.2023.0911.2-develop.zip . Will rename to *.nuget. EPICloud does not support zip.
Renamed to *.nuget.
Loaded cms package:    /home/vsts/work/r1/a/_Build/DeployableArtifacts/Site.cms.app.2023.0911.2-develop.nupkg
cms package 'Site.cms.app.2023.0911.2-develop.nupkg' start upload...
cms package 'Site.cms.app.2023.0911.2-develop.nupkg' is uploaded.
Package 'Site.cms.app.2023.0911.2-develop.nupkg' file size => 146889794.

Tested both with zip and nupkg. Works as expected. Close task.


fix: State descriptions can have HTML in them, mark them safe

Whiie this may be better in the short run, it moves us further away from the right place to be.
State descriptions (and DocEvent descriptions) should NOT have html in them. If we want this stuff to be marked up, we should use a safer markup language (limited markdown perhaps). Or we should stop trying to mark it up.

To be clear, since state descriptions can only be provided by code or by someone with admin access, we can take this PR, but we really need to squeeze these pockets of html fragments out of the database.


<EMAIL_ADDRESS>breaks client auth with edge functions

Bug report

[ ] I confirm this is a bug with Supabase, not with my own application.
[ ] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
When I get the authorization header from the client in an edge function I'm unable to make a supabase client with the newest version of supabase-js (using deno edge functions). The behavior doesn't happen on 2.42.4 and below. I have rolled back but I wanted to bring this to your attention.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Go to '…'
Click on '…'
Scroll down to '…'
See error

Expected behavior
Able to get user auth details. See code snippet at bottom of how I typically do that.
Screenshots
If applicable, add screenshots to help explain your problem.
System information

OS: [e.g. macOS, Windows]
Browser (if applies) [e.g. chrome, safari]
Version of supabase-js: [e.g. 6.0.2]
Version of Node.js: [e.g. 10.10.0]

Additional context
`import { corsHeaders } from '../_shared/cors.ts'
import { createClient } from "supabase-js"
import { createSupabaseClient } from '../_shared/supabaseClient.ts'
Deno.serve(async (req) => {
if (req.method === 'OPTIONS') {
return new Response('ok', { headers: corsHeaders })
}
const authHeader = req.headers.get('Authorization')!
const supabaseClient = createSupabaseClient(authHeader)
// Get the session or user object
const { data } = await supabaseClient.auth.getUser()
const user = data.user
if (!user) {
throw new Error('User is null');
}
}`
and here is the definition for my client function:
`import { createClient, SupabaseClient } from "supabase-js"
export function createSupabaseClient(authHeader: string): SupabaseClient {
return createClient(
Deno.env.get('SUPABASE_URL') ?? '',
Deno.env.get('SUPABASE_ANON_KEY') ?? '',
{ global: { headers: { Authorization: authHeader } } }
);
}`

@kangmingtay Does the supabase-js team know about this fix? Users are still running into the issue on Edge functions and your fix here has not been merged with supabase-js yet, or released.

You can use any version below 2.42.4

Sure and that's what I'm telling user that come to discord, or github and I see them.  Or just use getUser(jwt) with the jwt from the header.
BUT users are hitting this and not knowing what the issue is, so wasting alot of time on their part till they find the solution.

Just to reinforce what GaryAustin1 is saying...
When I wrote in to support with this issue I got back:
Thank you for contacting Supabase support. This works and I have it in a few of my edge functions.

Have you tried logging the headers to debug it?

In reference to calling auth.getUser() with no JWT.    It seems like the organization doesn't understand the existence or magnitude of the problem.
It's a big breaking issue for anyone who copied code form the JavaScript edge function auth docs.  It's insidious in that it only shows up when you deploy a new version of your edge function and only if you are using the import line from the docs which just references @supabase/supabase-js@2.
Then you spend a lot of time trying to figure out how you completely broke your edge function with whatever change you made.


Support for PodTolerationRestriction

Description of the feature you would like to add / User story

As a cluster manager
I would like to dedicate nodes for some namespaces
in order to reduce blast ratio and allow fine grained resource management
Solution details

Use PodNodeSelector to automatically add nodeSelector to pods (Supported directly in KubeOne API)
Use PodTolerationRestriction to automatically add tolerations to pods (Not supported directly or indirectly, this can be done indirectly if https://github.com/kubermatic/kubeone/issues/2275 is supported)

Alternative approaches

Write a custom admission controller

Use cases

Allow easy and effortless resource allocation for simple use cases

This can now be done with the custom API server flags feature (#2275)
/close


Remove NCurses Dependency from EventLoop Example

This will make the event loop example more platform independent. @svaarala

Just to be clear, are you concerned about Windows portability specifically?
If so, follow-up pulls should then add Windows socket and poll compatibility (WSAWaitForMultipleEvents(), or whatever is most appropriate today)?

I agree with this change if curses is considered problematic. But just to be clear, what portability targets are you trying to address?
It would make sense for the example to be portable to Linux, macOS, and Windows at least. At present the example is Linux only.
Adding macOS support should be simple but requires some testing. Adding Windows support would require adding Windows socket bindings and using WSAWaitForMultipleEvents() (or whatever is most appropriate today)?
These can (and should) be done in separate pulls - but it'd be good to first define a goal what this and the follow-up pulls are trying to achieve.

Currently this demo is quite saturated, removing, some of the stuff that has no relation the the eventloop, should make the example easier to understand.

We can finish this curses change as I agree it's not necessarily a very good dependency.
But are you proposing some other changes?

No

Ok, please squash the commits for merge (see https://github.blog/2011-09-06-shiny-new-commit-styles/) and this should be done.

Squahed

Done


[ALLUXIO-1942] Enable partial read block caching by default

@calvinjia

add this property and its default value to docs/_data/table/user-configuration.csv

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9996/
Test FAILed.

@peisun1115 Could you leave a TODO to handle this property correctly in integration tests?

the csv entry is needed to correctly display the property on our documentation page for "Configuration properties"

@jsimsa @apc999 Fixed

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10004/
Test FAILed.

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10000/
Test PASSed.

LGTM

LGTM

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10005/
Test PASSed.


WIP: Scroll to top of page on link click

Done:
Add scrollTo to gatsby browser
To Do:
Make it work... Add scrollTo to handleNavigate? Or to close function?

Hmmm... this doesn't actually seem to be working at all. Could you remove and add to the clickhandler on the GlobalNavLink just before the timeout on line 35. Let's see what that does!
@johnnycrisp

Hmmm... this doesn't actually seem to be working at all. Could you remove and add to the clickhandler on the GlobalNavLink just before the timeout on line 35. Let's see what that does! @johnnycrisp

Done, but still doesn't seem to to anything :/


HomePage scrollbar

Make the scrollbar visible constantly in the HomePage class and after searching for titles in the SearchPage class.

#16


Support Typings Definitions for latest versions

We are using Application Insights with TypeScript and are using Typings as well. But I found there is only 0.15.12 version's definition in the registry of Typings. Would you add Typescript definitions of each version including latest to the registry?

Registry: https://api.typings.org/entries/dt/applicationinsights/versions
Application Insights 0.15.12 Definition: https://raw.githubusercontent.com/DefinitelyTyped/DefinitelyTyped/253e456e3c0bf4bd34afaceb7dcbae282da14066/applicationinsights/index.d.ts

FYI, we are using 0.18.0 and the interfaces of 0.15.12 are not the same as it, so we are using it with workaround.

I believe the latest source correctly generates ts types embedded in the npm package, making this just work (at least for TS 2.0+).
I'll verify with a generated package, but copying latest build output directly to my node_modules folder seems to show correct types in vscode.


Allow passing Sprite destroy options down from PIXI.Application

Overview
This PR adds the ability to completely destroy an application including it's textures from a single call to PIXI.Application.destroy at the top level.
Usage
const app = new PIXI.Application();
app.destroy(false, true);

More details

Looks great would you mind adding a test for this? Otherwise 👍

Ok, now I'll add

Nice work. Could you revert the changes to package-lock.json?

@bigtimebuddy I've finished. added tests and deleted package-lock.json


Never Receiving CALL_HISTORY_METHOD

Version
"react-router-dom": "^4.1.2"
"react-router-redux": "^5.0.0-alpha.6"
Steps to reproduce

Connect react-router and react-router-redux to a simple application
call a function that does `dispatch(push('/abc'))

Expected Behavior

That my system will receive CALL_HISTORY_METHOD emitted into it.

Actual Behavior

Only receiving LOCATION_CHANGED

From these lines, it sounds like if we need to hook into URL changes BEFORE it affects our state, we should use CALL_HISTORY_METHOD. However, it seems that my system is never given the option to listen for this action. Am I missing something here? How would I listen for changes on the URL BEFORE they affect my state?

The middleware swallows these actions: https://github.com/ReactTraining/react-router/blob/master/packages/react-router-redux/modules/middleware.js
You should set up a listener on the history instance that you create. As long as you do so before rendering your React app, it will get them first and you can choose to take action before the listener for RRR is fired.


feat: implement ways to interact with collect via transform API

This provides a more consistent API to interact with collect plugin methods via regular transform API. This would allow to streamline passing of the token stream to collects of plugins when compiling MD files via CLI's md2md track.

Tests?


An attribute that has "reportable=true" in the XML is not made reportable automatically

Attributes that have the "reportable" tag set operate like this:
1.) When you ENABLE the attribute on the "attribute" tab, they should immediatelly be made reportable, so if you go to "reporting" tab after enabling them, you should immediatelly see them enabled.
2.) If they are disabled later in the "reporting" tab, there should be a yellow triangle with a message "this attribute should be reportable" or something like that.

Note: this may or may not be a UI bug. It's possible that the bug happens in the back-end when the endpoint is created. We have to make sure that the database state itself is correct after creation of endpoint.
If the UI is simply reflecting the DB, and DB itself is wrong, then this is NOT a UI bug, but a backend bug.

In terms of the DB, if I examine the DB right after enabling the attribute on the endpoint the ATTRIBUTE table has IS_REPORTABLE true, but the ENDPOINT_TYPE_ATTRIBUTE has INCLUDED_REPORTABLE false for that attribute.  So the UI and the DB are in sync; what I don't know is which is supposed to be the source of truth here.  Obviously if I change things in the UI the DB gets updated.

Note that this bug is constantly corrupting the Matter .zap files: people add attributes, and forget to set them reportable, even though the spec requires them to be reportable.  What's the chance for progress on this?

This looks like it has been resolved in the consultant branch and can open a PR


Added the instructions to set the proper permissions for the key file…

…s and directory

@mostafaahamidmanon thanks. please resolve conflicts.


Cannot use "Format Modified Lines" command or modifications formatOnSaveMode

I would like to use the registered formatter to auto-fix only the lines of code that have changed according to version control and am having trouble figuring out how.
VS Code supports this via at least two mechanisms:

The editor.formatOnSaveMode setting (with value of modifications or modificationsIfAvailable):

The Format Modified Lines action in the Command Palette

When I try to use the Format on Save approach with the following config:
{
    "eslint.format.enable": true,
    "editor.formatOnSave": true,
    "editor.formatOnSaveMode": "modifications"
}

...saving a file does not result in auto-formatting at all.
If I add:
"editor.codeActionsOnSave": {
    "source.fixAll.eslint": true
}

...it does auto-format the file, but it does not limit to changed lines, so it seems that setting bypasses the editor.formatOnSaveMode setting.

When I try to use the Format Modified Lines action in the Command Palette, I'm shown an error message:

Message says: Configure Default Formatter -- Extension 'ESLint' is configured as formatter but it cannot format 'JavaScript'-files

In neither case is there a message in any Output pane that seems related to this behavior.
Is there a setting I'm missing or another way to accomplish this that I've missed?
I'm using Extension v2.2.6 on VSCode 1.69.1 on Fedora 36 installed via Microsoft's Repo, but have confirmed the behavior with a Mac user.
Thanks for any guidance!

This is currently not supported. Major reason is that the eslint --fix option (which the extension depends on) doesn't allow formatting regions of code. It can only format the whole file.
I do agree that the error message should be improved here.


Use the atom-workspace tag instead of the workspace class.

In keymaps/api-blueprint-preview.cson: Use the atom-workspace tag instead of the workspace class.

This should be fixed in the latest release.


stacktrace for _signal_callback error

Godot version:
3.0, Beta 2, Windows 7, GTX 950
Sometimes I get this error because my management of signals and yields are poor (still honing them down):
0:00:17:0256 - Resumed after yield, but class instance is gone
----------
Type:Error
Description: Resumed after yield, but class instance is gone
Time: 0:00:17:0256
C Error: Method/Function Failed, returning: Variant()
C Source: modules\gdscript\gdscript_function.cpp:1490
C Function: GDScriptFunctionState::_signal_callback

Would help a lot if there was a stacktrace  on where the signal failed from. Except, it's not possible because the class instance is gone, right? Sorry if it sounds like a silly suggestion :P. Not sure how it really works internally but the stack trace really helps for regular GDScript debugging too

Also experiencing this.


Build error websocket

When I run npm run build and then I load my unpacked extension I get the error
WebSocket connection to 'ws://localhost:9090/6' failed: Error in connection establishment: net::ERR_CONNECTION_REFUSED

I have moved the wcer plugin from webpack.base.js to webpack.dev.js and I still get the same error
after building, it works correctly when I run npm run dev.

I have the same problem = =.
@boriscy  Have you solved this problem?

@boriscy @huxiaoqiang
use node 8.x.x  to solve this problem

@llqi thanks, it works!

which version did you use solved this problem?

@HaoweiCh v8.15.0

8.0.4
HWei<EMAIL_ADDRESS>周二21:49写道：

which version did you use solved this problem?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/YuraDev/vue-chrome-extension-template/issues/22#issuecomment-483667234,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AgCSOvyq1a4kUOC7xKWW210K2Gq2A5S9ks5vhdTvgaJpZM4ZTvkm
.


zone defaults to public, instead of default_zone

Most tasks where a zone is required default to 'public'.
I suggest they should default to 'default_zone' and then to public, which can be achieved by doing this:
zone: "{{ item.value.zone|default(default_zone|default('public')) }}"

@goniomdq Thank you for this enhancement hint, I'll try it.


Feature |  Import Existing Token (Create Workflow)

Description
This PR adds the ability to import an existing ERC20 token and wrap it for use as a Governance token in a DAO. It covers the first part of deploying a new DAO with this new feature and updates the UI during the token configuration step.
Main changes

Modified AzoriusTxBuilder.ts to handle the import and wrapping of existing ERC20 tokens.
Updated AzoriusTokenDetails.tsx to include new UI elements for the token configuration step.

Additional Notes

Fixed an issue with Toasts not properly closing and a missing translation.
This PR also covers checking for tokens that are already presumed to be ERC20 Votes tokens.

Testing

Create a new Token DAO using the following methods:

As normal
With existing VotesToken
With existing ERC20 token

Ensure that the DAO deploys correctly and shows up as a Token Voting DAO. Note that there is no other way to ensure proper deployment until the other end of this feature is worked on.

Screenshots (if applicable)

When selecting a token, when I input an existing token that does have the ERC20Votes functionality, it still seems that a new wrapped token is being deployed for the DAO. In this case, we wouldn't want to re-deploy a token, and just let the existing VotesToken be used, right?

When selecting a token, when I input an existing token that does have the ERC20Votes functionality, it still seems that a new wrapped token is being deployed for the DAO. In this case, we wouldn't want to re-deploy a token, and just let the existing VotesToken be used, right?

Oh true good point,
It's not actually being deployed as wrapped, but the logic should be:
No blue text until the address if legit
Then blue text appears based on whether it's being wrapped or not


Add a new F&F and Keeper

Add a new F&F
Add a new Keeper
Mak an existing F&F a Keeper

Add new F&F would most probably remain the same if state and reducers continue to have the same data.
Need to consider user same trusted channel mechanism for sharing RK to all types of keepers
Review the journeys related to Keeper and making changes based on new communication mechanism using trusted channel

Changes for contact and device create channel
Encrypt sm share with ch2 key
Accept Request by qr scan and link
Fetch last seen and update status to accessible after process complete

Changes for contact and device create channel [Done]
Encrypt sm share with ch2 key (Issue #3436) [Done]
Accept Request by qr scan and link [Done]
Fetch last seen and update status to accessible after process complete [Done]

-Verified this issue on dev app 1.7.5(300)
1.Can add new F&F.
2.Can add New Keeper.
-Mak an existing F&F a Keeper-this will be covered in another issue.So closing this issue .


binutils override withAllTargets does not work

Describe the bug
I am trying to get binutils to support all targets by overriding the withAllTargets argument. But I get the following error message:
error: anonymous function at /nix/store/5w02kc6nw8xybgfy4n787ssks3a88c5i-nixpkgs-20.03pre211462.0c960262d15/nixpkgs/pkgs/build-support/fetchurl/boot.nix:5:1 called with unexpected argument 'meta', at /nix/store/5w02kc6nw8xybgfy4n787ssks3a88c5i-nixpkgs-20.03pre211462.0c960262d15/nixpkgs/pkgs/build-support/fetchpatch/default.nix:14:1
(use '--show-trace' to show detailed location information)

To Reproduce
Steps to reproduce the behavior:

Create shell.nix:

{
  pkgs ? import <nixpkgs> {},
}:

pkgs.mkShell {
  nativeBuildInputs = [ (pkgs.binutils-unwrapped.override { withAllTargets = true; }) ];
}

Execute nix-shell

Expected behavior
A shell environment containing binutils with all targets enabled.
Additional context
The higher-level goal is to obtain a version of objcopy that supports pe-i386 and pe-x86-64 targets. I need to copy a 32-bit PE object file to a 64-bit one. If there is a better way to accomplish this goal, I'd appreciate guidance.
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 4.15.0-76-generic, Ubuntu, 18.04.4 LTS (Bionic Beaver)`
 - multi-user?: `no`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.2`
 - channels(parthy): `"nixpkgs-20.03pre211462.0c960262d15"`
 - nixpkgs: `/home/parthy/.nix-defexpr/channels/nixpkgs`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

Setting enableShared = false as well seems to be working fine. Is this expected? Then maybe the error message should somehow reflect that, if possible.

/cc @ericson2314

Maybe i am completely wrong about this, but I think since this works with enableShared = false the issue most likely comes from here.

I'm experiencing the same issue with 7e9b0dff974c89e070da1ad85713ff3c20b0ca97.

Still a problem. I think the trigger is the use of fetchpatch in libbfd, but I don't understand why it matter for override.
CLI reproducer:
$ nix build  --impure --expr 'with import ./. {}; binutils-unwrapped.override { withAllTargets = true; }'
error: anonymous function at /home/slyfox/dev/git/nixpkgs-master/pkgs/build-support/fetchurl/boot.nix:5:1 called with unexpected argument 'postFetch'

       at /home/slyfox/dev/git/nixpkgs-master/pkgs/build-support/fetchpatch/default.nix:31:1:

           30| in
           31| lib.throwIfNot (excludes == [] || includes == [])
             | ^
           32|   "fetchpatch: cannot use excludes and includes simultaneously"
(use '--show-trace' to show detailed location information)

Proposed the fix (and a new explicit attribute) as https://github.com/NixOS/nixpkgs/pull/168106


Large Repository Clone

We have a fairly large repository (~8GB) and we cant seem to clone the repo using WebGitNet.
We are running Windows 2008 R2 Server x64 with IIS 7.5, using standard windows authentication.
We have smaller repos that we have no issues cloning.
Please advice what kind of tracing is available or I can turn on to figure out whats the problem.

@otac0n Is there any chance of building a new release with this fix included?


sql: all dataset selection to be disabled

in postgres-based datasources we don't use datasets. this pr allows datasources to decide if they want to use datasets.
if datasets are disabled, it will use datasource.dataset as the database.

@grafana/enterprise-datasources / @grafana/partner-datasources
can i please get a review on this from your side also as databricks and astradb use this component.


Project polymorphysm base on its type

Describe the problem
We have 3 project types: document classification, sequence labeling and seq2seq. All of this types are represented as a single Project class with lots of if-elses on project type. It will be more better to use inheritance and proxy model to represent these polymorphic relations. It will be easier to extend, write and reason about.
This issue was extracted from the comment in this PR #55

@serzh Are you still working on this? The model is changed a little and I am worried this will make trouble for your work.  And other issues might also need to change the model part too. Let me know your status.

@BrambleXu Sorry for the delay, have some hard days on work. I haven't started any refactoring now, I expect to start working on it within 2 weeks.

@serzh It's totally okay. I am sorry to hear that you have bad days on work. That does sound frustrating. In the other side, it is good to hear that you haven't started the refactoring work. @Hironsan is working on this task recently and it should finish soon. After his work finished, it would be great to hear your opinion.

That's great! I would be happy to participate in a review.

Should we change Assignee on this task to @Hironsan ?

OK.

Done.


Superpath methods missing from the AAS Repository?

According to IDTA API Spec, only profile for the AAS Repository is the one specified with the Submodel Repository as the superpath, however this seems to be missing from the AAS Repository off-the-shelf component.
Is this ommited on purpose, and if yes, can you shortly explain why?

If my understanding is correct, these methods cannot be ommited (according to spec I mean :) ), because the /description interface should return a list of profiles which the API supports - and there is no profile with AAS-Repository methods only without a superpath.

Hi, I see from the linked issue that you already found the related issues that I've raised in the aas-api-sec group. I think it would be better to continue discussion there. If you agree, I will close this issue here.


Add note on google_storage_bucket_iam_binding

Actually, allUsers is a special identifier and does not need any prefix like group:.
const binding = new gcp.storage.BucketIAMBinding("member", {
 bucket: config.bucketName,
 members: ["allUsers"],
 role: "roles/storage.objectViewer",
});

Maybe you could consider adding it to the documentation, just here.
I'm closing this issue myself
Originally posted by @sebght in https://github.com/pulumi/pulumi-gcp/issues/177#issuecomment-510928342

Details on the members property of BucketIAMBindingArgs is missing from https://www.pulumi.com/docs/reference/pkg/nodejs/pulumi/gcp/storage/#BucketIAMBindingArgs
See https://www.terraform.io/docs/providers/google/r/storage_bucket_iam.html
The docs should include the following:

Fixed in https://github.com/pulumi/terraform-provider-google-beta/commit/54c1f8e3a8f517ccf08b35a931ae68d0b8d6593f.


Plant should die naturally after some time passes based partly on its genes

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Nb. Plant doesn't automatically die when it's old - it just becomes harder and harder to maintain and branches don't grow well and instead more weaken until reaching the dead threshold <1 branch.


Triple click should include newline

VSCode uses non-standard behavior for triple-clicking from every other text editor on the market (including Visual Studio):
When triple-clicking on a line of code, the entire line should be selected, including the newline. VSCode skips the newline in its selection.

The new line is correctly picked up on triple click on Ubuntu 15.10/vscode 0.10.10-insider.

Fixed in stable channel.

How to triple click without a newline?


Live preview extension doesn't display window at full height in codespaces

The live preview window won't display full window height in codespaces. My webpage looks as expected using the external browser (firefox/chrome). Here is my public repo if anyone wants to open this in codespaces to try and reproduce the issue.
This is what live preview looks like for me in Codespaces

This is what my webpage looks like previewed in Chrome

Same issue in gitpod


Document "week" / "w" as an allowed unit

Hi,
The current documentation - http://graphite.readthedocs.io/en/latest/config-carbon.html#storage-schemas-conf lists second/minute/hour/day/year as the possible time units; but week is also in use (e.g. the O'Reilly Graphite book uses it), and the code supports it. Could this be added to the documentation, please?
Thanks :)

Hello @mcv21,
Sure! Even better - you can add it too if you want.
Sources for documentation from https://graphite.readthedocs.io/ is in graphite-web repo - https://github.com/graphite-project/graphite-web/tree/master/docs - any user can fork repo, edirt docs and send PR.
Not saying that you should do that, of course, just a reminder.
Thanks for reporting! 👍

I think that PR should do the trick...

Merged! Thanks


[BUG] Arguments still get strings when "Fill all fields" instead of null

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I'm upgrading Annotations for GraphQL inside Symfony project with https://github.com/overblog/GraphQLBundle/blob/master/docs/attributes/attributes-reference.md.
Before it was working well with .graphql file like this :

And Altair's documentation was :

Finally when using "Auto fill" :

Now moving to Annotations in a different file for each Type, here it comes :

New documentations related to arguments:

When using "Fill all fields", arguments always get "_____" instead of null.

Am I missing something or is it impossible to change it like it was before ?
Thanks
Expected Behavior
Have argument 'id' filled with null when auto "Fill all fields" with no string.

Steps To Reproduce
Use GraphQL Bundle Annotations (https://github.com/overblog/GraphQLBundle/blob/master/docs/attributes/attributes-reference.md)
Environment
- OS:Win11
- Browser:Firefox 133.0
- Platform:Firefox extension
- Version:App Version: 8.0.1

Additional context
No response

Find a workaround with ArgsBuilder

If you have a GraphQL schema with the unintended behavior, I can investigate

Thanks for the reply.
After i manage to find a workaround everything is fine.
Just for documentations if someone wants the same behavior...
In the end i make it work like this with an agsBuilder :

And Class :


[mixcloud:playlist] Downloading 0 videos

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2<IP_ADDRESS>
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

$ youtube-dl --verbose "https://www.mixcloud.com/soulection/playlists/soulection-radio-special-guests/"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.mixcloud.com/soulection/playlists/soulection-radio-special-guests/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 3.7.4 (CPython) - Linux-5.2.13-arch1-1-ARCH-x86_64-with-arch-Arch-Linux
[debug] exe versions: ffmpeg 4.2.1, ffprobe 4.2.1, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[mixcloud:playlist] soulection: Downloading playlist page
[download] Downloading playlist: Soulection Radio Special Guests by Soulection
[mixcloud:playlist] soulection_soulection-radio-special-guests: Download tracklist (page 1)
[mixcloud:playlist] playlist Soulection Radio Special Guests by Soulection: Downloading 0 videos
[download] Finished downloading playlist: Soulection Radio Special Guests by Soulection

Description

Seems to be parsing an empty playlist from Mixcloud, this one should have >100 items in it.  Latest build from git.

This issue is known and open for more than one year, I don't think any body is working on it, unfortunately I have not the capability to solve this, otherwise I would have as I'm really in need for this for a long time to make my automation media center to work well with Mixcloud.


problems with using --headers

I am having trouble adding extra headers (added by #8). For example. using http-server (as described here) - when I start it like this: http-server --dir "." --port 9998 --headers '{X-SomeHeader" "value"}', I do not see the header anywhere:
[luke:~] 7 $ curl -v http://localhost:9999
* Host localhost:9999 was resolved.
* IPv6: ::1
* IPv4: <IP_ADDRESS>
*   Trying [::1]:9999...
* Connected to localhost (::1) port 9999
> GET / HTTP/1.1
> Host: localhost:9999
> User-Agent: curl/8.5.0
> Accept: */*
>
< HTTP/1.1 200 OK
< content-length: 1488
< Server: http-kit
< Date: Thu, 8 Feb 2024 15:29:03 GMT
<
<html><head>...

I am using babashka v1.3.186.
Maybe I am not using it correctly ?

Got it: the headers are currently only returned when a file is retrieved:
$ curl -I http://localhost:8090/README.md
HTTP/1.1 200 OK
X-Foo-Bar: bar
content-length: 2558
Server: http-kit
Date: Thu, 8 Feb 2024 15:40:24 GMT

thank you @borkdude for a quick check! I was too confused to try with getting a file...

I changed the logic to always return the headers on every response.


can not access parentElement within custom directive's bind function

I have following simple code:
define a test-comp component; define a custom directive log-something
within that component's template, i use a custom directive v-log-something;
within the diretive's bind function**, i want to log out the belonging element's parentElement;
Unfortunately, the null result is returned.**
Note: It works if I put the v-log-something diretive in a normal html tag( Not within the component's template).
I am not sure whether or not this is a bug. I guess it is caused by some lifecycle issue: When diretive's bind function is called, the belonging component has not been inserted into dom.
When can we make sure everything is in dom and we can access parentElement node within the directive's bind function safely??
here is the fiddle:
http://jsbin.com/wajarad/edit?html,js,console,output
<body id="body">
  <test-comp></test-comp>
</body>
Vue.component('test-comp',{
  template: '<div v-log-something>this comes from test-comp</div>'
});
Vue.directive('log-something',function(){
  console.log(this.el.parentElement);
});
new Vue({
  el: '#body'
});

You can do it in or after the ready hook.

You can use $nextTick at the moment to fix the issue. Let me come back at you when I have a better answer:
Vue.directive('log', function () {
  this.vm.$nextTick(() => ...)
})

Components are compiled in a DocumentFragment before inserted, so you'll have to use one of the workarounds mentioned above.
BTW it also seems like a bad idea to access outside DOM from within a component.

@posva , thanks, got your workaround, it works now.
@Twiknight , Your workaround i can not catch up:  How component ready function can access function defined in the directive bind function?
@yyx990803 , you are right, it is not a good idea to access outside dom from within a component, but in some situation, it is more reasonable to access parent within component itself. thanks for your great effort!

@cnweibo I mean, you can move your function from directive to a read() function. Or you can use a read mixin.
Another way is to define a event listener with your component, and emit the event after it's mounted (ready)

@Twiknight , thanks! got your 2nd point..  But for the first proposal of "move your function from directive to a read() function", I can not catch yet. The most annoying problem is: we can not make sure v-log-something directive attached element is in the dom.
What I raise this question is: I want the v-log-something directive can be used anywhere to access some information for its parent element.
thanks~!

@cnweibo sorry to have made a typo , I mean ready() hook.
you can declare a mixin:
const logMixin = {
  ready: () => /*log something here*/
}

And then use it this way
const myOptions = {
  /*your options*/,
  mixins: [logMixin]
}

const myComp = Vue.component(myOptions)

see it here: http://vuejs.org/guide/mixins.html#Option-Merging

@Twiknight ,
Thanks, i got it.

Can someone elaborate on the best method on resolving this issue in Vue 2?  I am running into a similar issue, however, I can't get access to the vue instance within the directive.  "This'" scope is on the Window object instead of the Vue instance.
Is someone able to clarify on the best way to achieve waiting for the the target instance or component's ready state for the directive to use the appropriate "this" context to gain access to the Vue instance?
https://jsfiddle.net/6p6de29p/4/

@nat1597 in Vue 2 directives changed a lot. Hook signatures are now different and no more this.
You can access the vm as vnode.context, with vnode being the third argument to the hook. And the old workaround with nextTick is still needed if you want to access vm from the directive's bind.

Thank you for your prompt reply this helps, I thought I tested a 3rd argument in bind() but I obviously missed it =).
So this is a sub issue of the ultimate goal I am aiming for and I've searched pretty extensively on how to do this but I'm pretty new to Vue so I don't have many of the directives/components concepts down.
Ultimately, I am trying to add a custom directive to a component I am building.  The custom directive is a plugin I would like incorporated into the component's behavior and I am able to get it to work against basic html structure with a simple vue instance.  However, my confusion arises with integrating this custom directive with the component's html template definition.  I am having trouble with the directive accessing the vue instance I am attempting to bind to the directive.
I'm assuming I need to define this directive within 'props' in the component and define it within the '<custom-component' as an option as well as within the instance's data object but I can't figure out how to get the directive to access the instance appropriately with a handle to the plugin's object I need.
Thanks for the help.

Thank you for your prompt reply this helps, I thought I tested a 3rd argument in bind() but I obviously missed it =).
So this is a sub issue of the ultimate goal I am aiming for and I've searched pretty extensively on how to do this but I'm pretty new to Vue so I don't have many of the directives/components concepts down.
Ultimately, I am trying to add a custom directive to a component I am building. The custom directive is a plugin I would like incorporated into the component's behavior and I am able to get it to work against basic html structure with a simple vue instance. However, my confusion arises with integrating this custom directive with the component's html template definition. I am having trouble with the directive accessing the vue instance I am attempting to bind to the directive.
I'm assuming I need to define this directive within 'props' in the component and define it within the '<custom-component' as an option as well as within the instance's data object but I can't figure out how to get the directive to access the instance appropriately with a handle to the plugin's object I need.
I am able to instantiate it now with using vnode but I'm still not able to get it to work, I need a little more understanding on how a custom directive communicates with a component through props if that is indeed how the directive behavior is appropriately translated to components.
Thanks for the help.

I faced this issue in Vue 2.5.13.
Here is workaround works for me:
bind(el, binding, vnode) {
    vnode.context.$nextTick(() => {
        // vnode.elm.parentElement
    });
}


Cache Storage Account and Private Endpoints

Section under Important header indicates:
The cache storage account must be accessible on its public endpoint. Azure Migrate uses the cache storage account's public endpoint to move data from the storage account to replica-managed disks.
I do not believe this is true any longer, as Migrate supports Private Endpoints and thus, public endpoints should no longer be required on the cache storage account.  Per this documentation, https://docs.microsoft.com/en-us/azure/migrate/migrate-servers-to-azure-using-private-link?pivots=agentlessvmware:  "To orchestrate replications, Azure Migrate will grant the trusted Microsoft services and the Recovery Services vault managed identity access to the selected storage account."  This means Public Endpoints should be possible to disable on the cache storage account.
Either the documents are wrong or they are confusingly written.

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

ID: 46fd5665-034e-fdb4-824b-a089d14e1886
Version Independent ID: 9f50c8ab-9029-d951-17c6-6ac1c48b26d0
Content: Replicate data over ExpressRoute with Azure Migrate Server Migration - Azure Migrate
Content Source: articles/migrate/replicate-using-expressroute.md
Service: azure-migrate
GitHub Login: @DeSeelam
Microsoft Alias: deseelam

thanks for bringing this to our attention. I have assigned the issue to the content author to evaluate and update this doc as appropriate

Thank you for surfacing this. We understand that this can be confusing. This document is to be referenced if you want to use ExpressRoute for your replications when using an Azure Migrate project with public endpoint connectivity while the control plane communication between the Azure Migrate appliance and the Azure Migrate service continues to happen over the internet on the Azure Migrate service's public endpoint. To use private endpoint support end-to-end, see https://docs.microsoft.com/en-us/azure/migrate/how-to-use-azure-migrate-with-private-endpoints. We’ll make the necessary documentation changes.

This change has been updated as part of another PR and the update is available at https://docs.microsoft.com/en-us/azure/migrate/replicate-using-expressroute. Hence, closing this issue.
#please-close


Add flow for testing a nested game instance inside a TestScene

Turns out that the way we had this setup previously made it very dangerous if a nested game instance was to call Game.Exit() (exiting full host execution and potentially skipping test cases with a "Success" status.
Breaking changes
Game instances can no longer be Added to a test scene directly
To test a Game instance, use the new AddGame method. Following is an example of how to update an existing usage.
Before:
AddStep("Add game", () =>
{
    var game = new TestGame();
    game.SetHost(host);
    Add(game)
});

After:
// SetHost and Add are automatically called internally.
AddStep("Add game", () => AddGame(game = new TestGame()));

Needs a fix for visual executions, looking into it now.

In general this looks good to me, but I'm not super certain as I haven't seen this code too much. It is a bit of a bummer that these changes don't look to have any effect on the test browser - any Exit() from a child game will still exit the entire browser. But that's probably out of scope here..

Fixing the test browser is definitely something on my radar, but as you can probably guess it's a bit more involved (may require a different fix altogether) as we can't change the game host class.

Looks like something regressed in https://github.com/ppy/osu-framework/pull/4771/commits/2ff3e3fce0e72188f3dfb28d7c932afb442bbdf2.


HCHunger=false not working

If HCHunger=false, there is still notification and debuff ingame, as if HCHunger=true
MC 1.11.2
Forge 2310
BWM 1.2.16

HCInjury=false doesn't work either

Only notifications should be showing up. Any debuffs that exist are due to your player existing with HCHunger enabled at some point and having leftover data, to stop the HCHunger you will have to kill your player.


fix(core): merge of kept / mock declarations in CommonModule #5465

closes #5465

v14.10.0 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


How can I print out request bodies

Hi, I'm trying to create a small development proxy to print out to the console headers, query params, and body.
But I can't get the body and prioxy to work together, and all my research had not given me any obvious solution.
In the code below, when I enable bodyParsers, I get the body printed to the console,  but any PUT or POST request that contains a body ends up hanging. With no bodyParsers, req.body is undefined (as pre express docs) and the proxied request completes successfully.
Any advise on how I might approach this?
Thanks in advance.
var express = require('express');
var app = express();
var bodyParser = require('body-parser');
var proxy = require('http-proxy-middleware');

var apiProxy = proxy({
    target: 'http://localhost:7040',
    changeOrigin: true,
    onError: function(err, req, res) {
        console.error(err.message);
        res.writeHead(500, { 'Content-Type': 'text/plain' });
        res.end(err.message);
    }
});

// When enabled I get body output, but server/proxy hangs and request never completes.
app.use(bodyParser.raw());
app.use(bodyParser.text());
app.use(bodyParser.urlencoded({ extended: false }));
app.use(bodyParser.json());

app.use(function(req, res, next) {

    console.log('\n\nHeaders');
    console.log(req.headers);

    console.log('\nQuery');
    console.log(req.query);

    console.log('\nBody');
    console.log(req.body);

    next();
});

app.use(apiProxy);
app.listen(8888);

mount your apiProxy before mount the bodyParser should fix the issue.

Thanks for the suggestion. I did try this. I get a working proxy, but get 'undefined' for req.body (probably because then bodyParsers are now after my app.use() that dumps the data.)
If I put apiProxy before my app.use(),  my app.use() never gets called.

Update - I had success using the npm 'body' package rather 'body-parser'.

@garyns can you tell the sample how it worked for you. When I am using npm body still my socket is hanging.

can you please show an example of how to use body ?
thanks

Hi @chimurai
Would you mind adding the "mount your apiProxy before mounting the bodyParser middlewares" nugget to the README file?
I just spent three days trying to figure out why my proxy was hanging, until I stumbled upon this issue.
Might save someone else the trouble...
Thanks,

const anyBody = require('body/any')
onProxyReq(proxyReq, req, res) {
    anyBody(req, res, function (err, body) {
        if (err) console.error(err)
        console.log(body)
    })
})

Found that it is possible to do it with the bodyParser middleware by following this example which shows the need to re-stream the body in the onProxyReq method.

Found that it is possible to do it with the bodyParser middleware by following this example which shows the need to re-stream the body in the onProxyReq method.

Didn't handle to upload file.


Please make the 'read in english' button stay fixed at the top even while scrolling.

Whenever I try to see it again in English while reading a long document, I have to scroll, press the toggle button and go back to my original position.
It's really uncomfortable and inefficient.
Please make the 'read in english' button stay fixed at the top even while scrolling.

#3270 is requesting similar functionality.


Making SceneTree dock floating removes ability to use certain shortcuts in the editor viewport (2D/3D)

Godot version
4.0 92bcd3c
System information
Win 10
Issue description
Because the canvas/3d viewport uses the shortcuts of the SceneTreeDock for some actions, like add child node, or delete node, when you make this window floating those actions stop working because shortcuts do not propagate to the sub window - which I think is correct behaviour (?).
This might be a good reason to duplicate those shortcuts on the canvas item editor or 3d node editor so they do not depend on the SceneTreeDock.
This may also help resolve issues like https://github.com/godotengine/godot/issues/66511 and https://github.com/godotengine/godot/issues/63804, because then Add Node/Delete Node would not need to be global shortcuts, so would not be triggered by usages in unexpected places.
Steps to reproduce

Create scene, add node.
Make SceneTree panel floating
Try use Delete to delete node, or Ctrl+A to add child node.
Doesn't work

Minimal reproduction project
Editor only

I've just tested on v4.0.beta.custom_build [767f8fb48] on Linux Ubuntu 20.04 (Unity desktop) and Delete / Add child node both worked in the floating panel.
That said, the related issue of having Delete work globally in unexpected contexts is still here, so even if the floating window issue happened to be fixed in the meantime, we'd still need to care about the general problem of Delete context. Maybe the latter would deserve a meta-issue that gathers links to Delete not working as expected in various contexts?

I guess it should be better for the stable version, as there have been some improvements to input handling between different windows. But I'm not going to close this just yet, as we probably need to assess and test this better.

I am still experiencing this issue in 4.1 (linux).

@mieldepoche Which shortcuts no longer work for you? Could you give a bit more detail?

The behavior I'm seeing is the one described in the issue, that is SceneTreeDock related shortcuts don't work: add node, duplicate, etc.


Remove sequence component

This PR closes #902 .
Core implementation of the wizard and sequence are being paused while design makes revisions.

Deploy preview for pf-next ready!
Built with commit e41502f4b4f3e2a65d26812a2909dd686de9f780
https://deploy-preview-903--pf-next.netlify.com


make logo look clickable when we hover on it

Description
make logo look clickable when we hover on it
Changes
make cursor pointer
Related Issues
https://github.com/threefoldtech/tfgrid-sdk-ts/issues/1753
Screencast from 01-10-2024 02:55:48 PM.webm
Checklist

[ ] Tests included
[ ] Build pass
[ ] Documentation
[ ] Code format and docstrings
[ ] Screenshots/Video attached (needed for UI changes)

there is no hover effect and when i click on it it does not direct me to home :/ https://www.loom.com/share/4e634d04d1c244558a71acadeeb83cf2?sid=7b7d3efe-8c08-46d8-8b23-c411fb7a83fc

should be resolved after merging ur pr.


ImportError: cannot import name 'BertConfig' from 'pytorch_pretrained_bert.modeling'

My code was running was successfully, but suddenly got this config error

It might be caused by missing cd /path/to/code/src ; pip install --user --editable ., which installs the code as a package.


Feature: Add categories to Library

I was just reading through https://github.com/Rotonde/rotonde-portal/issues/7 and came to think of sorting entries into categories as being a useful feature for the Library, especially as more p2p apps arise.

This is an interesting UX issue, yeah. @taravancil you should take a look at the linked rotonde issue.


Set acces token in req.session while logged in

Is your feature request related to a problem? Please describe.
At the moment the acces token is appended the hash in the url. Then in the client it's taken out and used to do requests. That's nice, we also need a way to have easy acces to the token within the server.
Describe the solution you'd like
I would like the acces token to be saved within the req.session object once authorisation is established. And of course to be destroyed whenever the user isn't logged in.
Describe alternatives you've considered
Alternative could be to fetch everything on the client. That could make it more app alike, but for a request to retrieve like data for the first view, isn't it better anyway to do that straight away from the server instead of waiting on the client-side script to inject it?
Additional context
Something else: this issue hold information that is valuable for the our process. It's about a strategy to conquer a challenge which relates heavily to the WAFS and PWA course. It's documentation worthy, together as a team and individually.

This is fixed, the logout issue and destroying the session should be a separate issue.


Introduces PipelineValidator that checks the well-formedness of a pipeline

Motivation: being more confident about various code that manipulates the pipeline, e.g. fusion code, SDF expansion, etc.
Didn't add explicit tests for the validation code - it will be implicitly tested by every time every portable pipeline runs; the incremental benefit of adding explicit tests doesn't seem to justify the cost. Any false positives will be caught by a respective pipeline, false negatives will be caught at runtime.
(piggybacked here: deleted unused and empty ReferenceRunner class - the proper name for that concept is PortableRunner. At some point we should rename the module to beam-runners-portable-java too)
R: @lukecwik

Hm Jenkins seems to not be picking up this change. Guess I'll have to wait a bit more.

retest this please

retest this please


Getting a sprockets error

Getting a sprockets error after install, can't find the required css files

Can you please provide backtrace error?


Less

builds off of the incomplete babel6 pr.
@insin this is a question as much as a PR. the css loading is specified in a way I've never seen. why the combine loaders, then the + '!'. I couldn't seem to get less loading to work in the same way. Also, are the require.resolve statements necessary?

Everything which is configured with require.resolve is to save us having to worry about which working directory heatpack is running in, as the config points at absolute paths to heatpack's own dependencies.
The manual tagging on of the ! for autoprefixer is because autoprefixer can't deal with an empty query object - see https://github.com/jsdf/webpack-combine-loaders/pull/1. I ended up rolling my own version of this for nwb, where it's done this way to allow the user to tweak the configuration of every loader, so that might have been some cross-pollination :)

@insin this should be ready now...any idea how many types of files you'd like to support?

@pwmckenna It'd also be nice to include sass/scss support, since those are fairly commonly used too. I don't mind making another PR based on this one, but it seems like @insin might've forgotten about this?


Updating build instructions to adhere to the terraform provider standards

Fixes #620
I updated the README to adhere to the "standards" set by the official Terraform provider READMEs. Theses changes will add more clarity when a user wants to build a specific version. Additionally, these build instructions are how I will be completing the Docker examples.
A couple of questions for @MalloZup, is the cgo requirement still a thing? I was able to build the plugin without setting export CGO_ENABLED="1". If so, I think we should remove it from the Building from source section.

@MalloZup I actually ran into issues with cgo on alpine. I needed to install libc-dev on alpine which in turned installed musl-dev. But I still did not need to set the environment var of export CGO_ENABLED="1".
Let me fix up some of the other issues you addressed up above.

@james-crowley  I will let this PR a little pending to let @dmacvicar  the time to review and respect his own time and privacy and if he is in vacation or has other things todo.
I will let this pr like 1/2 weeks open waiting for review, if he will not answer I will assume everything is ok. So just for your info.

@MalloZup It is all good. No rush. I'll continue to work on the Docker examples and submit a PR for that soon.