Switch from virtus to dry-types

I don't really know what you do with virtus (I think it's coercion) but I think it could be a good idea to replace virtus by dry-types (http://dry-rb.org/gems/dry-types/) for many reasons:

This reddit post from Solnic, the virtus creator: https://www.reddit.com/r/ruby/comments/3sjb24/virtus_to_be_abandoned_by_its_creator/
Performance (from the dry-types doc): "roughly 10-12x faster than Virtus"

Thanks for sharing this @guizmaii! I read the post with great interest. I would be open to a PR that replaces Virtus with something else, we can look at the implications.

I added the PR request to my TODO list but I really don't know the Grape code for now and I don't have a lot of time but it could be a good start for me to contribute to Grape.

Happy to see this proposal. I was also wondering if maybe you could leverage dry-validation too.

Very good, if possible, I want to get involved @dblock

Looking forward to pull requests @totothink.

Looks like this enhancement requires a lot of refactoring. Grape tied on Axiom::Types and Virtus coercion logic. It instantiates Virtus::Attribute objects, which I don't sure have Dry::Types equivalent. So to achieve success for this issue we need to completely refactor Grape::Validations module and it's ancestors.

require 'dry/schema'

module Validation
  class DSL
    attr_reader :schema

    OPTS_MAPPING = {
      regexp: -> v { { format?: v } },
      type: -> v { v.name.downcase.to_sym }
    }

    def initialize
      @schema = Dry::Schema::DSL.new(processor_type: Dry::Schema::Params)
    end

    def call
      schema.call
    end

    def requires(name, opts)
      schema.required(name).value(*convert_opts(opts))
      self
    end

    def optional(name, opts)
      schema.optional(name).value(*convert_opts(opts))
      self
    end

    private

    def convert_opts(opts)
      opts.each_with_object([]) do |(k, v), arr|
        arg = OPTS_MAPPING.fetch(k).(v)
        arr << arg
      end
    end
  end

  def params(&block)
    @schema =
      begin
        dsl = DSL.new
        dsl.instance_eval(&block)
        dsl.call
      end
  end

  def schema
    @schema
  end
end

class Grape
  extend Validation

  params do
    requires :id, type: Integer
    optional :text, type: String, regexp: /\A[a-z]+\z/
  end
end

puts Grape.schema.(id: '1', text: '123').inspect
#<Dry::Schema::Result{:id=>1, :text=>"123"} errors={:text=>["is in invalid format"]}>

puts Grape.schema.(id: '1', text: 'foo').inspect
#<Dry::Schema::Result{:id=>1, :text=>"foo"} errors={}>

👋 so I just wrote this, as a quick PoC. This uses dry-types under the hood for coercion.

as already mentioned above, dry-schema(that extracted from dry-validation) looks like something that is more suitable for grape needs(coercion/validation of input params). though, it should be checked if the interface of the library can be easily used without its DSL.
if something will be refactored, it would be nice to see a solution with an ability to disable/replace validation component at all. I see params block as "meta description" of a user input for a specific endpoint. this description can be used by different consumers(validation component/documentation component/tests generator component/etc). as the result, we should have single "cross-cutting" DSL(as we have now) which provides an endpoint schema to consumers(we do not have now). so I don't really like the snippet above, even if it is a POC :)

You may be interested then in the fact dry-schema exposes the definition via AST which can be reused for any purpose you mentioned
schema = Dry::Schema.Params do
  required(:user).type(:hash).schema do
    required(:email).type(:string)
    required(:age).type(:integer)

    required(:address).type(:hash).schema do
      required(:street).type(:string)
      required(:city).type(:string)
      required(:zipcode).type(:string)

      required(:location).type(:hash).schema do
        required(:lat).type(:float)
        required(:lng).type(:float)
      end
    end
  end
end
schema.to_ast
=> [:set,
 [[:and,
   [[:predicate, [:key?, [[:name, :user], [:input, Undefined]]]],
    [:key,
     [:user,
      [:and,
       [[:predicate, [:hash?, [[:input, Undefined]]]],
        [:set,
         [[:predicate, [:key?, [[:name, :email], [:input, Undefined]]]],
          [:predicate, [:key?, [[:name, :age], [:input, Undefined]]]],
          [:and,
           [[:predicate, [:key?, [[:name, :address], [:input, Undefined]]]],
            [:key,
             [:address,
              [:and,
               [[:predicate, [:hash?, [[:input, Undefined]]]],
                [:set,
                 [[:predicate, [:key?, [[:name, :street], [:input, Undefined]]]],
                  [:predicate, [:key?, [[:name, :city], [:input, Undefined]]]],
                  [:predicate, [:key?, [[:name, :zipcode], [:input, Undefined]]]],
                  [:and,
                   [[:predicate, [:key?, [[:name, :location], [:input, Undefined]]]],
                    [:key,
                     [:location,
                      [:and,
                       [[:predicate, [:hash?, [[:input, Undefined]]]],
                        [:set, [[:predicate, [:key?, [[:name, :lat], [:input, Undefined]]]], [:predicate, [:key?, [[:name, :lng], [:input, Undefined]]]]]]]]]]]]]]]]]]]]]]]]]]]]]]

Hello community, you are welcome to review my PR replacing virtus with dry-types. :slightly_smiling_face: Later we might bring dry-schema as well, but it will require more work and it will bring lots of breaking changes. So, let's start with dry-types :wink:


Clean up internal.pyx (get_contiguous_strides)

Removed get_contiguous_strides because it's never used
Renamed set_contiguous_strides -> get_contiguous_strides_inplace

Hmm, instead of exposing this function back to Python, is it ok to test it indirectly via ndarray.__init__()?

Sounds good!

Jenkins CI test (for commit a4c842c70fc4ced20ebfa94db9dd8808d8ea7d57, target branch master) failed with status FAILURE.

Sorry for delay. PTAL
(Btw, itemsize=3 can't be tested so I removed it. I think it's ok if itemsize=2 is tested.)

Jenkins, test this please.

Jenkins CI test (for commit 4d76fcbe57c50beae45ef245df8ee5077bb9fd25, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 4d76fcbe57c50beae45ef245df8ee5077bb9fd25, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 4d76fcbe57c50beae45ef245df8ee5077bb9fd25, target branch master) succeeded!


Generate R2 entries for all resource types

Fixes #871

We tried this, and it fails for "mipmap" resource types, as it generates a @MipmapRes annotation that does not exist

Merged #969.


[PIP-22] Introduce repair metastore procedure

Purpose

Introduce repair metastore procedure

Tests

HiveCatalogITCaseBase#testRepairTable
API and Format

Documentation

Thanks @zhuangchong , left some comments.

Thanks. @JingsongLi
I have modified the comment, please help with code review.

@JingsongLi
I added a new test HiveCatalogITCaseBase#testRepairTableWithCustomLocation,

If the database in hive already exists and a custom location is used, use the custom location lookup table in the file system;

If the table already exists and a custom location is used, updating the Hive table will not change the location;

+1


Message commands reply not shown

Everything is up to date.
The message commands works (except corp, char and fleetupops), I saw the replies in the DB file, but not shown in discord. The onTick plugins replies are shown, the bot responds the ping command.
Dramiel-error.log:
PHP Notice:  Undefined index: fleetUpOps in /var/www/Dramiel/src/plugins/onMessage/fleetUpOps.php on line 72
PHP Notice:  Undefined index: fleetUpOps in /var/www/Dramiel/src/plugins/onMessage/fleetUpOps.php on line 73
PHP Notice:  Undefined index: fleetUpOps in /var/www/Dramiel/src/plugins/onMessage/fleetUpOps.php on line 74
PHP Warning:  Missing argument 3 for queueReplyMessage(), called in /var/www/Dramiel/src/plugins/onMessage/charInfo.php on line 117 and defined in /var/www/Dramiel/src/lib/queueManagement.php on line 53
PHP Notice:  Undefined variable: message in /var/www/Dramiel/src/lib/queueManagement.php on line 55
PHP Warning:  Invalid argument supplied for foreach() in /var/www/Dramiel/src/plugins/onMessage/fleetUpOps.php on line 118
DramielLog.log (when exec !ops command):
[2016-11-05 14:51:17] Dramiel.ERROR: Error: SQLSTATE[23000]: Integrity constraint violation: 19 NOT NULL constraint failed: replysQueue.message [] []
Op. system: Ubuntu 16.04
PHP: 7.0

should be fixed, reverted a change that was working but ended up fucking the bot

The bot now answers for commands except for fleetupops:
dramielError.log:
PHP Notice:  Undefined index: fleetUpOps in /var/www/Dramiel/src/plugins/onMessage/fleetUpOps.php on line 72
PHP Notice:  Undefined index: fleetUpOps in /var/www/Dramiel/src/plugins/onMessage/fleetUpOps.php on line 73
PHP Notice:  Undefined index: fleetUpOps in /var/www/Dramiel/src/plugins/onMessage/fleetUpOps.php on line 74
PHP Warning:  Invalid argument supplied for foreach() in /var/www/Dramiel/src/plugins/onMessage/fleetUpOps.php on line 116
The char and corp commands never find data, allways get this:
Charinfo: Error: no data available
Corpinfo: Error: No data available for that group.

not a bot error, eve-kill is down which is where it pulls alot of data from for those 2. Added a more descriptive message in the latest push


Bug fix 3.7/issue 12507

Scope & Purpose
Bugfix for Github issue #12507.
Backport of https://github.com/arangodb/arangodb/pull/12510

[x] :hankey: Bugfix
[ ] :pizza: New feature
[ ] :hammer: Refactoring
[x] :book: CHANGELOG entry made
[x] :muscle: The behavior in this PR was manually tested
[x] :computer: The behavior change can be verified via automatic tests

Backports:

[ ] No backports required
[x] Backports required for: none - version 3.6 is not affected

Related Information

[x] GitHub issue / Jira ticket number: https://github.com/arangodb/arangodb/issues/12507

Testing & Verification

[x] This PR adds tests that were used to verify all changes:

[x] Added new integration tests (i.e. in shell_server / shell_server_aql)

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/11621/

Tests blue

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/11626/

Tests blue


Jason

Hello

Hello

Hello, This is Shohag. I am a professional Ethical Hacker,
Security Specialist as well as Penetration Tester. I can test the following vulnerability and I will provide you with a perfect assessment of your website security
which makes your website safe and sound.
My Experience:
💗. Recovering Hacked WordPress Website
💗. Full Vulnerability Scan and WordPress website malware removal
💗. Vulnerability Assessment
💗. Antispam protection
💗. Analysis of database servers
💗. Restore a hacked website
💗. Ethical Hacking(White Hat Hacker)
💗. Hacked account recovery
💗. Password recovery
💗. Decode encrypted files
💗. Vulnerability Testing
💗....my  WhatsApp number +8801314956025

Hello

My Experience:
💗. Recovering Hacked WordPress Website
💗. Full Vulnerability Scan and WordPress website malware removal
💗. Vulnerability Assessment
💗. Antispam protection
💗. Analysis of database servers
💗. Restore a hacked website
💗. Ethical Hacking(White Hat Hacker)
💗. Hacked account recovery
💗. Password recovery
💗. Decode encrypted files
💗. Vulnerability Testing
💗....my WhatsApp number +8801314956025


e2e: add e2e test for fallback annotations

Ⅰ. Describe what this PR did
add an e2e test for nginx.ingress.kubernetes.io/default-backend && nginx.ingress.kubernetes.io/custom-http-errors
Ⅱ. Does this pull request fix one issue?

fixes #193
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #259 (7eda2a2) into main (09e563c) will decrease coverage by 0.38%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #259      +/-   ##
==========================================
- Coverage   44.58%   44.20%   -0.38%
==========================================
  Files          32       32
  Lines        5275     5275
==========================================
- Hits         2352     2332      -20
- Misses       2754     2774      +20
  Partials      169      169

see 1 file with indirect coverage changes

already done in #441


New feature: Dark mode for toolbar, page tree, structure tree, modals, login and wizard

Description
Since Django 3.2 the admin interface respects "dark mode", i.e. if selected by the user, the admin shows on dark background instead if light background. Dark mode is often used by IT people. Currently, django CMS does not respect a selected dark mode leading to a admin pages partly on dark background, partly on bright background giving the impression of a broken UI design.
This PR introduces changes to the css so than django CMS can respect dark mode. This affects the toolbar, page tree, structure tree and modals. It does not affect the admin site nor the frontend editing forms.
Changes:

Replaced the $gray sass variables with css custom properties --dca-gray etc. (dca for django CMS association)
Added different color codes for dark mode for those variables
Replaced brighten and darken sass functions with css alternatives (mostly transform: brightness() since functions do not work on custom properties.
Removed a few !important statements to allow other apps to properly adjust their admin forms to dark mode.

Reflected @marksweb’s suggestion to have the django CMS color adjusted in dark mode to be better for the eye

Related resources

There is a similar PR for djangocms-admin-style. With both accepted django CMS will give a nice dark mode design: PR 443
This is the second incarnation of #7238 without changes to the static/css/3.9.0 files.

Checklist
No tests are affected.

[x] I have opened this pull request against develop
[ ] I have added or modified the tests when changing logic
[x] I have followed the conventional commits guidelines to add meaningful information into the changelog
[x] I have read the contribution guidelines  and I have joined #workgroup-pr-review on Slack to find a “pr review buddy” who is going to review my pull request.

Sass building now 👍
I'll merge this if you're happy with things @fsbraun

@fsbraun What's the status of this one?

Should be merged! I just resolved the conflict in the CHANGELOG.rst. You can merge it.


Add Git Integration to README

Since v1.0.0 doesn't support the git integration anymore, I extracted the parts out of v0.9.1 and created a package for that.
EDIT:
Package available here: https://github.com/Jubeki/orbit-git

@Jubeki I'm actually going to be adding the Git integration back in 2.x which will be released soon with some performance optimisations and better DX.

@ryangjchandler okay. Do you mind creating a pinned issue which links to the repo and says that version 2 is in progress?
I will archive my repository when version 2 is out. Thanks for your work ❤


register TextLayoutInfo and TextFlags type.

derive Reflect to GlyphAtlasInfo,PositionedGlyph and TextLayoutInfo.
Objective

I need reflection gets all components of the TextBundle and clone_value it

Solution

registry it

I also need this, I'd like if this got merged. Could you please address the comments?


Nested Insert and update

I wanna do nested CRUD operation with the following schema type...
input MemberInput {
    _id: String!
    username: String
    name: String!
    email: String
    password: String
    fatherName: String!
    address: String!
    dob: String!
    age: Int!
    mobile: Int!
    monthlyIncome: Float!
    qualification: String!
    occupation: String!
    muhalla: String!
    familyMembers: [String]
  }

  type FamilyMember {
    name: String!
    relationship: String!
    age: Int
    qualification: String
    occupation: String
    remarks: String
  }

  type Member {
    _id: ID!
    username: String!
    name: String!
    email: String!
    password: String!
    fatherName: String!
    address: String
    dob: String
    age: Int
    mobile: Int
    monthlyIncome: Float
    qualification: String
    occupation: String
    muhalla: MUHALLAH
    familyMembers: [FamilyMember]
  }

Have no idea on how to do this. Help needed. :)

Hi! Your question doesn't seem to be related to vue-apollo. Please ask it again directly on Apollo or GraphQL help sources.


a simple line...

we forgot to compute the mRange in the json reloading... so now it is fixed :)

and obviously, this opens a can of worm. @weefuzzy until we find a way (if ever) to update the @attributes in the client from the json, then I reckon no-change-of-range-post-fitting is the only way forward (as it was for the SC .kr code)
if you are not offended by this, I'll merge this for now to be consistent.


Increase install speed

by using package's ability to use list (instead of calling package multiple times which with_items does) and not updating cache twice, we can speed up our code

Fix for #224


Failed to login when using a cell address without the ending "/"

Even though a correct cell address is entered and verified, login still fails if the ending "/" is missing.
Cannot just populate the URL with "/", need to find out where lg.rootUrl must be populated properly.
Found the following bug that will assign an empty string to lg.rootUrl.
var i = cellUrl.indexOf("/", 8); // first slash
i = cellUrl.indexOf("/", i + 1);  // second slash's index become -1
lg.rootUrl = cellUrl.substring(0, i + 1); // substring is a empty string

Fixed by 288f5ff31b12a11d109fa298866971a594029569


Update commons-codec to 1.17.0

About this PR
📦 Updates commons-codec:commons-codec from 1.15 to 1.17.0
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "commons-codec", artifactId = "commons-codec" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "commons-codec", artifactId = "commons-codec" }
}]

labels: library-update, commit-count:1

Superseded by #409.


Is this project still active?

Hi there. It looks like the last commit to this project was back in January 2023. Is this project still maintained or has it reached end of life?

I cannot answer for your question. However, you can also check similar project, https://github.com/dbhi/qus.


add: 'Learn with Shakti' Tutorial Resource

appends this resource to the intermediate section of README.md

added link to ASM Programmer Manaul Version 1
added lnik to Learn With Shakti

Hi, I have made the changes to date (format: dd-mm-yyyy) and added anchor links.


Add basic CI pipeline

@mgdelacroix Any idea what is happening here? The error pattern<EMAIL_ADDRESS>missing go.sum entry doesn't make sense. go mod tidy doesn't update go.sum.

@mgdelacroix I'm going to merge this one. Please let me know if there is anything you would like to see in CI.


One-line output for print_level=1

Display some basic information:
Satus - Objective - Constraints - iterations ?

What do you mean exactly ?

Hi @joseph-gergaud !
I would like ipopt to display only one line of output, typically the success / failure and maybe the objective.
Currently we have this
julia> solve(ocp,print_level=0);

julia> solve(ocp,print_level=1);

julia> solve(ocp,print_level=2);

julia> solve(ocp,print_level=3);
Total number of variables............................:      404
                     variables with only lower bounds:      202
                variables with lower and upper bounds:        0
                     variables with only upper bounds:        0
Total number of equality constraints.................:      203
Total number of inequality constraints...............:        0
        inequality constraints with only lower bounds:        0
   inequality constraints with lower and upper bounds:        0
        inequality constraints with only upper bounds:        0

Number of Iterations....: 9

                                   (scaled)                 (unscaled)
Objective...............:   3.1303695485806754e-01    3.1303695485806754e-01
Dual infeasibility......:   4.4408920985006262e-16    4.4408920985006262e-16
Constraint violation....:   2.8779752019036930e-12    2.8779752019036930e-12
Variable bound violation:   9.7032585134461507e-09    9.7032585134461507e-09
Complementarity.........:   1.5503759072192753e-11    1.5503759072192753e-11
Overall NLP error.......:   1.5503759072192753e-11    1.5503759072192753e-11

Number of objective function evaluations             = 10
Number of objective gradient evaluations             = 10
Number of equality constraint evaluations            = 10
Number of inequality constraint evaluations          = 0
Number of equality constraint Jacobian evaluations   = 10
Number of inequality constraint Jacobian evaluations = 0
Number of Lagrangian Hessian evaluations             = 9
Total seconds in IPOPT                               = 0.015

EXIT: Optimal Solution Found.

I'll try to print the internal_msg recovered by NLPModelsIpopt for print_level=1.
For print_level=2 we could add the objective, contraints violation and iterations.


Change ipfs image to ipfs/kubo:latest

fixes Issue #12
Signed-off-by: Brian at Podping<EMAIL_ADDRESS>

IPFS was failing:
Error: no supported interface
Changing user to ipfs
ipfs version 0.12.2
Found IPFS fs-repo at /data/ipfs
Initializing daemon...
go-ipfs version: 0.12.2-0e8b121
Repo version: 12
System version: amd64/linux
Golang version: go1.16.15
2022/08/26 11:53:28 failed to sufficiently increase receive buffer size (was: 512 kiB, wanted: 2048 kiB, got: 1024 kiB). See https://github.com/lucas-clemente/quic-go/wiki/UDP-Receive-Buffer-Size for details.
2022/08/26 11:53:28 [zeroconf] no suitable IPv4 interface: udp4: failed to join any of these interfaces: []
2022/08/26 11:53:28 [zeroconf] no suitable IPv6 interface: udp6: failed to join any of these interfaces: []
2022-08-26T11:53:28.353Z    ERROR    cmd/ipfs    ipfs/daemon.go:430    error from node construction: no supported interface
Error: no supported interface

Switching to a new image fixes it.

Code is outside of consensus: Out of the app and in the IPFS package.


Using IPv6

Were you able to get IPv6 to work for this and if so what are you ip route/rule and ip6tables settings?

Yes.
Have a look at https://github.com/seud0nym/tproxy-go/blob/master/examples/OpenWRT/etc/init.d/tproxy


Adding accelerated networking udev rules for RHEL 7 and RHEL 8

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

Looks good to me, Thanks Anh!
#sign-off

@Court72 can you please merge this PR - since I am not the author, I am unable to sign-off


Implement UPNP port forwarding in teku

This is the besu upnp implementation using jupnp, but using our service model, and reduced to just port forwarding upnp.
fixes #2301
Signed-off-by: Paul Harris<EMAIL_ADDRESS>Documentation

[X] I thought about documentation and added the documentation label to this PR if updates are required.

Changelog

[X] I thought about adding a changelog entry, and added one if I deemed necessary.

I did note that port forwarding worked when I enabled upnp but failed if I only had upnp2 enabled (using an EdgeRouter).

To follow up on this - upnp2 is working, it was just hitting a port conflict because the forward wasn't being cancelled correctly.
It does make me wonder if we should use a shorter port forwarding TTL (instead of 0) but have Teku repeatedly refresh the port forwards.  That would avoid leaving ports open if there was a shutdown with no warning and would also better handle router reboots etc as the port forwards would be lost but reopened again when the refresh next triggers.  Not sure how complex that is though so probably a follow up ticket.

I did note that port forwarding worked when I enabled upnp but failed if I only had upnp2 enabled (using an EdgeRouter).

To follow up on this - upnp2 is working, it was just hitting a port conflict because the forward wasn't being cancelled correctly.
It does make me wonder if we should use a shorter port forwarding TTL (instead of 0) but have Teku repeatedly refresh the port forwards.  That would avoid leaving ports open if there was a shutdown with no warning and would also better handle router reboots etc as the port forwards would be lost but reopened again when the refresh next triggers.  Not sure how complex that is though so probably a follow up ticket.

in logs when TCP 9000 is already used:
2021-01-29 08:13:59.181+10:00 | jupnp-10 | INFO  | NatManager | External IP address XXX.XXX.XXX.XXX detected for internal address <IP_ADDRESS>
2021-01-29 08:13:59.202+10:00 | jupnp-16 | INFO  | TekuPortMappingAdd | Port forward request for UDP 9000 -> 9000 succeeded.
2021-01-29 08:13:59.202+10:00 | jupnp-12 | WARN  | TekuPortMappingAdd | Port forward request for TCP 9000 -> 9000 failed: Error: ConflictInMappingEntry (HTTP response was: 500 Internal Server Error)
2021-01-29 08:14:13.781+10:00 | jupnp-17 | INFO  | TekuPortMappingDelete | Port forward UDP 9000 -> 9000 removed successfully.

when both ports are available to allocated
2021-01-29 08:14:47.976+10:00 | jupnp-6 | INFO  | NatManager | External IP address XXX.XXX.XXX.XXX detected for internal address <IP_ADDRESS>
2021-01-29 08:14:47.997+10:00 | jupnp-14 | INFO  | TekuPortMappingAdd | Port forward request for UDP 9001 -> 9001 succeeded.
2021-01-29 08:14:48.001+10:00 | jupnp-10 | INFO  | TekuPortMappingAdd | Port forward request for TCP 9001 -> 9001 succeeded.
2021-01-29 08:15:10.331+10:00 | jupnp-19 | INFO  | TekuPortMappingDelete | Port forward TCP 9001 -> 9001 removed successfully.
2021-01-29 08:15:10.336+10:00 | jupnp-18 | INFO  | TekuPortMappingDelete | Port forward UDP 9001 -> 9001 removed successfully.

in logs when TCP 9000 is already used:
2021-01-29 08:13:59.181+10:00 | jupnp-10 | INFO  | NatManager | External IP address XXX.XXX.XXX.XXX detected for internal address <IP_ADDRESS>
2021-01-29 08:13:59.202+10:00 | jupnp-16 | INFO  | TekuPortMappingAdd | Port forward request for UDP 9000 -> 9000 succeeded.
2021-01-29 08:13:59.202+10:00 | jupnp-12 | WARN  | TekuPortMappingAdd | Port forward request for TCP 9000 -> 9000 failed: Error: ConflictInMappingEntry (HTTP response was: 500 Internal Server Error)
2021-01-29 08:14:13.781+10:00 | jupnp-17 | INFO  | TekuPortMappingDelete | Port forward UDP 9000 -> 9000 removed successfully.

when both ports are available to allocated
2021-01-29 08:14:47.976+10:00 | jupnp-6 | INFO  | NatManager | External IP address XXX.XXX.XXX.XXX detected for internal address <IP_ADDRESS>
2021-01-29 08:14:47.997+10:00 | jupnp-14 | INFO  | TekuPortMappingAdd | Port forward request for UDP 9001 -> 9001 succeeded.
2021-01-29 08:14:48.001+10:00 | jupnp-10 | INFO  | TekuPortMappingAdd | Port forward request for TCP 9001 -> 9001 succeeded.
2021-01-29 08:15:10.331+10:00 | jupnp-19 | INFO  | TekuPortMappingDelete | Port forward TCP 9001 -> 9001 removed successfully.
2021-01-29 08:15:10.336+10:00 | jupnp-18 | INFO  | TekuPortMappingDelete | Port forward UDP 9001 -> 9001 removed successfully.


raise NotImplementedError for unsupported steering vectors in BeamformerTime classes

Currently only one steering vector formulation is implemented in the time beamformers. This is misleading because different formulations can be set using the steering vector class. However, these are not used by BeamformerTime classes.
It would therefore be better to allow only one Steering vector formulation and to throw a NotImplementedError for all others.

see also #58
By Gert Herold on 2020-04-09T15:06:49 (imported from GitLab)

since #318 , time beamformers can use different steering vector formulations (no need for not implemented errors).


ColorPicker: transparent swatch

Originally posted by @claudiaareneee in https://github.com/iTwin/itwinjs-core/issues/3197
Feature
Currently, if a transparent color is passed to our ColorSwatch, then it just shows the color of the background:

Our UX standards have a diagonal line to indicate transparency, so we should handle styling for it.

Example
I imagine we could add a modifier class (e.g. iui-transparent) on iui-color-swatch, and then in iTwinUI-react we can use our ColorValue util to check if a color is transparent and automatically apply this class.

Decided to go with the checkered background pattern used on the opacity slider for consistency sake.  I'll ask @VirginieGaudreault next week to update the documentation site.


Add job link in SMT

Add the link to the job in the Saagie Monitoring Tool, dashboard "Jobs Failed Timeline"

Trello card: [#159] Add job link in SMT

Closed with PR https://github.com/saagie/technologies-community/pull/160


Workaround for the 64 characters problem

Hello,
(Copied from PR #196)
this PR addresses the issue #129. When the VM name is equal to 64 characters, the deployment is failing due to ssh problem (agent could not copy files to the VSI).
When the name with 64 characters was identified, a padding "-1" is appended to it.
I have removed the previous workaround for > 63 characters, since longer names are accepted.
@mattcui @maximilien Could you please have a look at this PR?

Hi, @vvraskin,
Thanks for resubmitting. I appreciate. Made a few comments, please address.
Also, I am requesting that @jianqiu reviews this PR as well. And also @gu-bin and @zhanggbj to make sure it does not break pipeline and our acceptance tests.
Finally, note that the Beijing team is on vacation for Chinese New Year, so not sure when they will have a chance to address these.
Best,
Max

Hello guys, as discussed with @mattcui, a friendly reminder :)

@vvraskin I just had a quick review and found a problem there, I think there doesn't need the variable LengthOfHostName any more, so better to remove this variable and the corresponding UT in fs_agent_env_service_test.go, and there doesn't need this line also. What do you think? Feel free to discuss with me either here or in Slack. Thanks

@mattcui Actually the unit test fs_agent_env_service_test.go is already removed in my fork and in this PR. LengthOfHostName is required to verify the length in this UT. Do you still think that we need to remove the variable LengthOfHostName?

@vvraskin Ok, I would like to leave the decision to @maximilien.
@maximilien Actually, the public variable LengthOfHostName  is not needed, but it would be useful for this UT, do you think if we could remove this variable and the UT as well? Thanks.

I agree with @mattcui as by having this var public could create a security risk plus allow the value to change by any code (so hard to track).
@vvraskin any way to test and making this public or make it a const (perhaps)?

@mattcui @maximilien Ok, agreed. Please see the latest commit. I've got rid of the usage of LengthOfHostName variable coming from the helper, added a new public method GetVMCloudProperties where I get the length from and updated the unit tests accordingly. Please review.
regards, Vadim

@mattcui and @jianqiu I am adding you both as reviewers. Please take a look at let's discuss in scrum tonight and get back with @vvraskin. My main concern is what impact on our tests envs and Bluemix team.

@maximilien The only potential problem I can see is there will be some trouble when the length of director hostname is 64, but I think we just need to document it to ask the user to not set 64 characters hostname when creating director, @gu-bin will address it. So in my opinion, you can merge this PR now. Thanks.

Thanks @mattcui. Merging.
/cc @gu-bin please ensure you have docs updated before next release. Thx

@vvraskin, please note this PR is reverted as we talked in Slack.
An error occurs when calling create_vm action:
STDOUT: '{"result":null,"error":{"type":"Bosh::Clouds::CloudError","message":"Run method not found","ok_to_retry":false},"log":""}'

Please fix it and resubmit your PR. Thanks!

@zhanggbj When I submitted the PR all the tests were green, do you think that it would be better if you create a test which will fail under the same PR? And then I will fix it.

@vvraskin as we just talked on slack, I'll take a look to improve the dispatcher UT to cover this case, but for this PR, maybe changing another way to implement create_vm UT is a better and faster option. Thanks.

@zhanggbj  I think you could show @vvraskin on how to run create_vm integration test to recreate the problem.

Sure @mattcui @vvraskin , here are steps to reproduce it. Thanks.

Build softlayer_cpi binary under src/bosh-softlayer-cpi/

go build -o out/softlayer_cpi_debug main/main.go

Prepare cpi.json

{
    "cloud": {
        "plugin": "softlayer",
        "properties": {
            "softlayer": {
                "username": "fake-username",
                "apiKey": "fake-api-key",
                "featureOptions": {}
            },
            "agent": {
                "ntp": [
                    "0.pool.ntp.org",
                    "1.pool.ntp.org"
                ],
                "blobstore": {
                    "provider": "dav",
                    "options": {
                        "endpoint": "http://<IP_ADDRESS>:25250",
                        "user": "agent",
                        "password": "fake-password"
                    }
                },
                "mbus": "nats://nats:{fake-password}@director-v2.softlayer.com:4222"
            },
            "baremetal": {},
            "pool": {}
        }
    }
}

Run create_vm integration test as below

out/softlayer_cpi -configPath cpi.json < dev/create_vm.json

All, FYI, for building CPI, the src/bosh-softlayer-cpi/bin/ should also be a usable option. And perhaps preferred since that's what the pipeline should use to create binaries.


[OPIK-444] [SDK] Update OpenAI integration to support logging model and provider in a new format

Details
Added model and provider handling for OpenAI integration.
If OpenAI client was initialized with specific base URL - it will be put in provider span attribute instead of openapi.
Issues
Resolves #
Testing
e2e test updated
Documentation

Lets not merge until Monday discussion.


peerDependencies will be deprecated in<EMAIL_ADDRESS>

https://docs.npmjs.com/files/package.json#peerdependencies - kthxbye!

I don't think it says they are deprecated. It says that they will no longer be automatically installed. There's nothing RR as a library should do here: with NPM 3, it's up to you to install React on your own (which you probably already have in your app anyway).


Unable to see the push notification for F&F request.

ios: Push notifications are not coming on iOS device.

Android:Push notification is visible on Android but clicking on it takes it to home page but message is not showing.

Android:Also check the App icon.

Notification is also not showing in the notification drawer. Sometimes they show up on iOS but are showing as of late.

Tested on dev app v1.8.0(320)

Verified this issue on dev app v1.8.5(324)


Create Cancellation Context

In preparing for more robust Deployment Plans (supporting rollback, cancellation, etc) and other Actions, we should add an opt-in Thread Local Storage like entity for Cancellation.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-2665. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Android support

Hi there,
Tried to get the example up and running, but it just displays a black screen and nothing als. Tried to run it on Pixel 2 real device and Pixel 2 from AVD.
Launching lib/main.dart on Android SDK built for x86 in debug mode...
Initializing gradle...
Resolving dependencies...
Running Gradle task 'assembleDebug'...
Built build/app/outputs/apk/debug/app-debug.apk.
I/flutter (16757): {publisherSettings: {"name":"Mr. John Doe","audioTrack":true,"videoTrack":true,"audioBitrate":null,"cameraResolution":null,"cameraFrameRate":null}, loggingEnabled: true}
Syncing files to device Android SDK built for x86...
D/eglCodecCommon(16757): setVertexArrayObject: set vao to 0 (0) 1 0
D/EGL_emulation(16757): eglMakeCurrent: 0xe3d1a3c0: ver 2 0 (tinfo 0xe08bab10)
D/eglCodecCommon(16757): setVertexArrayObject: set vao to 0 (0) 7 0

Should the example work already?
Mark

Hi @mark-veenstra
Currently there is no Android support, only iOS.
We have plans to implement Android in the future as well but current focus is on iOS as we are rolling iOS app currently.

Has the android support for the app resolved ?

Little update: we did release our app that uses this OpenTok library in iOS' App Store as so far everything works fine.
But to comment on Android part, no, we haven't gotten to that point, yet.
It might take some months before we roll Android support. I cannot give you realistic estimate right now as we do not know about that part yet.

Hey @genert ,
Which app is that?
Thanks

Hi there,
Tried to get the example up and running, but it just displays a black screen and nothing als. Tried to run it on Pixel 2 real device and Pixel 2 from AVD.
Launching lib/main.dart on Android SDK built for x86 in debug mode...
Initializing gradle...
Resolving dependencies...
Running Gradle task 'assembleDebug'...
Built build/app/outputs/apk/debug/app-debug.apk.
I/flutter (16757): {publisherSettings: {"name":"Mr. John Doe","audioTrack":true,"videoTrack":true,"audioBitrate":null,"cameraResolution":null,"cameraFrameRate":null}, loggingEnabled: true}
Syncing files to device Android SDK built for x86...
D/eglCodecCommon(16757): setVertexArrayObject: set vao to 0 (0) 1 0
D/EGL_emulation(16757): eglMakeCurrent: 0xe3d1a3c0: ver 2 0 (tinfo 0xe08bab10)
D/eglCodecCommon(16757): setVertexArrayObject: set vao to 0 (0) 7 0

Should the example work already?
Mark

Is it working now on android?


'connectionStringSetting' is really confusing

it took quite a while to figure out that 'connectionStringSetting' really should have been named 'connectionString_Attribute_KeyName_That_Appears_In_Local.Settings.JSON'

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

ID: 28aac762-b154-1d0e-9afc-67889fdf9762
Version Independent ID: 9a406f60-6c72-d3d9-2bc9-8d8e8c1922ae
Content: Azure Cosmos DB trigger for Functions 2.x and higher
Content Source: articles/azure-functions/functions-bindings-cosmosdb-v2-trigger.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@aronchick, Thank you for your feedback! We will review and update as appropriate.

@aronchick, thank you for reaching out, I am sorry for the delay in my response.
The ‘connectionStringSetting’ is listed in the App setting when function app is running on the portal. The ‘connectionStringSetting’ is listed in the local.settings.json file when function app is running locally. The document referenced does mention it as shown in the screenshot below which is just above the usage paragraph.

Please let me know if this does not resolve your concern, I will be happy to continue with our discussion.

The problem is that none of this is visible in VSCode – it’s not even obvious why it failed when run. Worse, I didn’t think, based on the way this was phrased, that I was missing anything, so I kept copying strings over and none of it worked.
<EMAIL_ADDRESS>I think it’s a combination of doc and error messages. At a minimum, I’d rewrite the above to say SOMETHING like:
“The name of the app setting that contains the name of the connection string used to connect to the Azure Cosmos DB account being monitored. You will need to a have a value with this name in local.settings.json (if developing locally), to the value you set here and the connection string provided by the service. If running in the cloud, you should set it in ‘appSettings (or whatever this is, with a link to that page’.”
I know it’s not possible, but the name of the connectionStringSetting variable really doesn’t make sense. It’s connectionStringFieldName.
From: ChaitanyaNaykodi-MSFT<EMAIL_ADDRESS>Sent: Wednesday, July 29, 2020 2:41 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: David Aronchick<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] 'connectionStringSetting' is really confusing (#59801)
@aronchickhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Faronchick&data=02|01|david.aronchick%40microsoft.com|a49972a9304d4015a07208d834080b44|72f988bf86f141af91ab2d7cd011db47|1|0|637316556435950753&sdata=Jd2bdUyGEFRx3wTwTtcrlqSTrp%2BaHq6f9HCupWAwxRs%3D&reserved=0, thank you for reaching out, I am sorry for the delay in my response.
The ‘connectionStringSetting’ is listed in the App setting when function app is running on the portal. The ‘connectionStringSetting’ is listed in the local.settings.json file when function app is running locally. The document referenced does mention it as shown in the screenshot below which is just above the usage paragraphhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fazure%2Fazure-functions%2Ffunctions-bindings-cosmosdb-v2-trigger%3Ftabs%3Dcsharp%23usage&data=02|01|david.aronchick%40microsoft.com|a49972a9304d4015a07208d834080b44|72f988bf86f141af91ab2d7cd011db47|1|0|637316556435950753&sdata=o%2FTrPD3fsp8O9Jefjvc9%2BSkhbBVqaIEYGcIKD44vvbs%3D&reserved=0.
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F67453298%2F88856271-0b733800-d1a9-11ea-8676-c6db326bac17.png&data=02|01|david.aronchick%40microsoft.com|a49972a9304d4015a07208d834080b44|72f988bf86f141af91ab2d7cd011db47|1|0|637316556435960750&sdata=0ut7hxJ3RQVTbE218Pe%2FcLR42vDOhZ8lgZtvLqxy5Bo%3D&reserved=0
Please let me know if this does not resolve your concern, I will be happy to continue with our discussion.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F59801%23issuecomment-665943527&data=02|01|david.aronchick%40microsoft.com|a49972a9304d4015a07208d834080b44|72f988bf86f141af91ab2d7cd011db47|1|0|637316556435960750&sdata=e%2B3yPI9ypp8mq4xH6C5u4ve%2F5ngfIIwdMnexnh4ZIj0%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAAMQ5IQKOHSZEGBKIY22GLR6CJNVANCNFSM4PJCLNWQ&data=02|01|david.aronchick%40microsoft.com|a49972a9304d4015a07208d834080b44|72f988bf86f141af91ab2d7cd011db47|1|0|637316556435970749&sdata=%2B7JjP1xUneUE9Kz2SmHXLJqBaXZfOIr8U%2FxqvlOS5Ks%3D&reserved=0.

@aronchick, Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.


Default backbuffer clearing to wrong color

The purple color visible in this screenshot is the default color we provide to BGFX, but Babylon.js should be overriding this color with the Playground's blue. This is indicative that the clear state may not be being set properly for the default view.

This is the color the backbuffer used to clear to.

There was a setviewclear in setviewport method that was using the default BabylonNative color:
https://github.com/BabylonJS/BabylonNative/pull/82/files#diff-dbfa6ca10fa035a631932160cd003264L1349
Because there is a setviewport (and a viewId == 1), the viewId 0 was set to clear view with the right color but because of no drawcall in view 0, it wasn't cleared. so, I added a touch here:
https://github.com/BabylonJS/BabylonNative/pull/82/files#diff-21d665b1d21081b0ae44c5cce05bdf0dR72

Will do the fix in this PR https://github.com/BabylonJS/BabylonNative/pull/86 as @bghgary suggested

PR is merged. Should be fixed now. Closing.


rpc_id() executes remotely, but not locally, even with the remotesync keyword

Godot version:
3.2.1.stable.official
OS/device including version:
Ubuntu 18.04 64-bit
Issue description:
rpc_id() executes remotely, but not locally, even with the remotesync keyword.
Details:
I created a simple match making server (remote, aws) that waits for two clients to connect (my pc and cellphone), pairs them and then provides them with each others' network peer ids.
That all works great.
The game starts and each client session remains connected to the server but performs rpc_id() calls on some game functions with the `remotesync' keyword, since I would like the functions to execute on the matched clients, but not the server or any other potentially connected clients.
The function is correctly called on the peer with id specified in my 'rpc_id(id, "func")' call, but it is not executed locally as it should be.
Replacing rpc_id(id,"func") with rpc("func") calls and executes correctly both locally and on the peer (but obviously also on the server and every other peer... so this is not really a solution)
Is this intended functionality? It seems a bit counterintuitive, since the docs claim that remotesync and rpc_id() are designed for this purpose.
Thank you.

Update:
rpc_id(id, "func") + remotesync seems to work okay when "func" is a function in the calling node.
Calling, child_node.rpc_id(id, "func") in the parent node, where "func" is in the child, causes it to be called remotely but not locally, still.

Do you have a minimal reproduction project you can provide as an example here?

Closing as there is no signal from the OP.
Feel free to comment if you still experience the issue, so we can reopen it in case.

Currently experiencing this issue, or at least a similar one.
In the parent class, I have the following code:
@rpc("any_peer", "call_local") func Sync_addImpulse(swingImpulseString : String): 	if multiplayer.get_remote_sender_id() == 0: 		rpc("Sync_addImpulse", swingImpulseString)                 print("forwarding rpc") 	else:                 # some code 		print("some code")
When I call Sync_addImpulse in the child class, the function is called (e.g., "forwarding rpc" is printed). However, the RPC isn't received.
To fix, this, I simply echoed the call in the child as follows:
`
@rpc("call_local", "any_peer")
func Sync_addImpulse(impulse): super.Sync_addImpulse(impulse)
`
Let me know if you have any questions or if I can help at all.

@Faless, @Calinou sorry for the ping - not sure who to ask but can someone reopen this?


Tutorial on how to remove hud?

Hello! Is it possible to fully remove hud from the game, and let the game use the entire screen for a map? When I try to do it, it ends up in garbage textures or sometimes, game freezing when I try to go to a different room. I apologize if I sound stupid, but I'm a complete newbie to C and this guide in general.
Thanks in advance!

Hello!
So this is technically possible, but it's going to be pretty tough, and it's not something I'd recommend to someone new to the tool or retro development.
The problem is how levels are laid out (relevant chapter). The map generator breaks levels into rooms that are 16 tiles wide by 12 tiles high, which covers a 256px by 192px area. In addition to the tiles, each room contains 32 bytes of sprite information and 32 bytes of padding. If you extend the data you are pulling from the map in your game without changing the map data, you end up trying to draw the sprite data as map tiles, resulting in drawing garbage textures. (Or potential crashes)
It's definitely possible to do, but a tutorial on this would be the most complicated and difficult chapter in the guide, so I am not inclined to write it at this time.

That said, depending on what you want to do I do see a workaround! If you had the current map area centered on the screen without the hud, would that be good enough?
If so, I would update the hud to have almost no data in it, then look at how we use the scroll() method in map.c - you can play with these values to get it more centered on the screen.
A couple disclaimers about this:

I would suggest against changing HUD_PIXEL_HEIGHT for now, as this controls where our sprite0 split happens, and could cause crashes when going between screens
Likewise, don't remove the entire border around the hud - there has to be at least one pixel that lines up with sprite0 in order for split scrolling to work (split() and split_y())

Due to the complexity of the topic I do not plan to write a tutorial on this, but if I can help you further for your case, let's continue the conversation!

Thank you so much for the response, I appreciate it a lot! I was able to make some progress using your instructions, and now the hud looks like this:

Is there any way to remove it completely and how do I increase the level size? I'm trying to recreate this game  and already got the palette working in my homebrew rom.

So unfortunately, making bigger maps is going to be somewhat difficult. I outlined the basic steps above, here's a bit more detail:

Move the hud up by updating HUD_PIXEL_HEIGHT, then update the sprite zero position to match that
Update the hud tiles to get rid of almost all of the border. Keep a few pixels in the lower right area - you'll have to experiment a bit to see when you can scroll without the game crashing. (The sprite zero/sprite 0 thing mentioned above)
Change the map generation tool to increase the size of each screen, paying close attention to how big your levels are. You want to keep them under 256 bytes per level, unless you want to rework things a lot more. this is that tool: https://github.com/igwgames/nes-starter-kit-tools/blob/main/tmx2c/src/index.js - you will need to update the javascript, then update the .create-nes-game.json to run your new javascript in place of the existing tool -- If you reduce the number of sprites to 8, and use the 32 unused bytes at the end, it should just fit.
Update the map code to pull in those extra tiles when loading the screen.

I know that's a bit loose, and definitely not to the level I write chapters in this guide, but I hope that's a good starting point.


[BUG] Neorg does not work due to missing dependency

Field
Description

Plugin
neorg

Nixpkgs
unstable

Home Manager
unstable

[x] I have read the FAQ and my bug is not listed there.

Description
It seems the version of the neorg plugin got bumpped and now no longer works due to a missing dependency. on lua-utils. It seems this dependency was added in v8.0.0 but the change must have only recently hit nixpkgs unstable.
Minimal, Reproducible Example (MRE)

programs.nixvim = {
    plugins.neorg.enable = true;
}

I think this is the related discussion: https://github.com/nvim-neorg/neorg/issues/1342
I just use the neorg from nixos-23.11 branch
For standalone configuration: https://github.com/nix-community/nixvim/discussions/1397#discussioncomment-9054044

In the mean time yes I know I can use the Neorg package from stable, but at the same time it would be nice for the latest version be fixed as well.

I think a fix for that needs to be landed in nixpkgs for a proper fix, not in nixvim. I'd love for a simple way to fix it without resorting to the stable fix, cause it kinda doesn't work on my end.

Thank you for reporting. I am aware about this.
However, we cannot do much on the nixvim side.
The solution has to be implemented on the nixpkgs side: we should package neorg as a lua package (luaPackages.neorg) there and build vimPlugins.neorg on top of it.
This is worked on here.

Another solution is found is to declare their official overlay to override the bugged package
https://github.com/nvim-neorg/nixpkgs-neorg-overlay

they updated there pkg

now it should work ha ?

I'll close this then. We can re-open if the issue is still present.

The update has nothing to do with the general issue which is not neorg-specific.
Indeed, the core problem here is that the lua dependencies of "neovim plugins" in nixpkgs (i.e. built from a luaPackage.*) are not propagated.
This is the same reason why luasnip complains with missing the jsregexp library.

Is there something still needed for this to work?

Is there something still needed for this to work?

This PR is where it's being worked on: https://github.com/NixOS/nixpkgs/pull/302442
As mentioned above, you can use the overlay as a temporary workaround.

sorry, came out rude, thank you for the response

For a temporary solution on 24.05 you can just add the dependencies as extraPlugins to nixvim.
programs.nixvim.extraPlugins = [
    (pkgs.vimUtils.buildVimPlugin {
      inherit (pkgs.luaPackages.lua-utils-nvim) pname version src;
    })

    (pkgs.vimUtils.buildVimPlugin {
      inherit (pkgs.luaPackages.pathlib-nvim) pname version src;
    })

    (pkgs.vimUtils.buildVimPlugin {
      inherit (pkgs.luaPackages.nvim-nio) pname version src;
    })
];

This alone will get it to work on nixos-24.05 (as of writing rev cf05eeada35e122770c5c14add958790fcfcbef5 ).
And if you want to bring in the more recent neorg from the current unstable(rev c3392ad349a5227f4a3464dce87bcc5046692fce), you can just swap out the neovim package of through nixvim to meet the 0.10 minimum requirement like so:
programs.nixvim.package = nixpkgs-unstable.legacyPackages.${pkgs.system}.neovim-unwrapped;
programs.nixvim.plugins.neorg.package = nixpkgs.unstable.legacyPackages.${pkgs.system}.vimPlugins.neorg;

https://github.com/NixOS/nixpkgs/pull/302442 has been merged so we should have a neorg plugin working out of the box very soon !
Sorry for the massive delay...

The patch has landed in nixos-unstable and this issue is now solved.
You don't need to use the overlay anymore.
Warning
If you have performance.combinePlugins enabled, make sure to add "neorg" to performance.combinePlugins.standalonePlugins.
Otherwise, it will keep complaining about missing dependencies.


Share custom components in nested MDX files

Closes #195
What:
This pull request adds a Readme guide to make sure custom components are accessible in nested MDX files.
Why:
With the current behaviour, passing custom components to nested MDX files is leading to unexpected rendering issues. This solution and Readme guide allows to share custom components across the MDX structure.
How:

Adding a new section to the README.md file providing a step-by-step solution
The solution uses MDXProvider from @mdx-js/react

Checklist:

[x] Documentation
[ ] Tests N/A
[x] Ready to be merged

Hi @kentcdodds, can you check out the PR when you have a moment. Your feedback would be great and maybe the PR will be merged. Thanks!

This is not a very common usecase though

Just ran across this - do you think it's not common? I thought that would be the most common use case.
Currently when you have the following simple setup, it breaks:
1.mdx
import Partial from './2.mdx'

<Partial />

2.mdx
# Partial content

Some content that is shared across multiple pages

<SomeComponentFromProvider /> {/* 💥 */}

🤷♂ I'd still like the docs for this moved down.

@kentcdodds , thanks for the review! I changed the position and moved it before the "Known Issues" section


Create an NGINX based proof of concept (for CSS)

Update: After working on LXML, I don't think there's a hope in hell of getting this to work for HTML, but it could be useful for CSS. It would also prove the same techniques as trying (and failing) to get it to work for HTML
NOTE: If the solution is totally hopeless, make a note and STOP!
This should:

Be time-boxed to no more than a half day
At a minimum pass the HTML through and out again (acting as a transparent proxy)
It should inject the required javascript to load the client
As an added extra, try and rewrite the links in the page
It would be a very nice feature if the different rewriters could be swapped out with a flag of some kind

In addition:

Record the time spent on this approach (for comparison to others)
The subjective experience of working this way (docs, frustration, ease etc.)

The purpose of this is:

Assess how easy or hard it is to work with
Assess the likely maintenance burden of this approach
Find out the speed characteristics
To provide a basis for comparison with other methods
Not to create the basis we will continue on (this code is disposable)

I think I'm going to put a pin in this before it really gets going. I don't think I'm honestly going to learn much I can't guess:

NGINX is generally a pain to work with
The solution would be incredibly fast, reliable (it'll never go down) and fragile (parsing docs with regexes is bad)
There's basically no chance of encoding the URL as a parameter correctly for proxying so that's out
It would be very hard to craft regexes to catch all of the various ways a URL can be expressed

With regards to various solutions:

The built in ngx_http_sub_module (https://nginx.org/en/docs/http/ngx_http_sub_module.html#sub_filter) doesn't support regular expressions, only direct string replacements
There are others that do:

https://github.com/yaoweibin/ngx_http_substitutions_filter_module
https://github.com/openresty/replace-filter-nginx-module

I can't see alpine packages for those, so it would be a case of compiling NGINX from scratch
If I go down that route, that's all I'll be doing all day

In general I'm more pesimistic about the likelyhood of this working as an actual rewriting solution after trying it in Python for a while. For example, for CSS files we can get:

URLs for image - which can be served directly
URLs for fonts - which must be proxied to work
URLs expressed as absolute, relative, no-schema, with or without params etc.

I can't see alpine packages for those, so it would be a case of compiling NGINX from scratch

FWIW there are Ubuntu packages for these so you can get them without compiling anything:
# Dockerfile
FROM ubuntu:bionic-20191029
RUN apt-get update
RUN apt-get install -y nginx=1.14.0-0ubuntu1.6 libnginx-mod-http-subs-filter=1.14.0-0ubuntu1.6 libnginx-mod-http-lua=1.14.0-0ubuntu1.6
COPY nginx.conf /etc/nginx/nginx.conf
CMD ["nginx", "-g", "daemon off;"]


(Feature) Add Coveralls

Closes #1034
Description

Adds coveralls and updates readme
I have also disabled the failing tests to make the CI work, will work on fixing the failing tests on other PR, meanwhile we can have the coveralls working :)

EDIT: Also adds typing for web3 methods

Travis automatic deployment:
https://pr1057--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1057--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1057--safereact.review.gnosisdev.com/app

Does it already show what % of percentage we have?

Travis automatic deployment:
https://pr1057--safereact.review.gnosisdev.com/app

Yes, once merged we will be able to see the badge on the readme, currently we have 38%. You can see the results here.

I am really questioning whether this number is correct 😄

Travis automatic deployment:
https://pr1057--safereact.review.gnosisdev.com/app

I checked the value because it was quite weird, I run coverage again and now it's 8% which it's more accurate 😅

I'm not sure whether we want to show such a low percentage

until we start adding more test, is what we have. What we can do is just remove the badge from the readme and merge this pr. Once we add more tests we add the badge to the readme again
CC @lukasschor

Travis automatic deployment:
https://pr1057--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1057--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1057--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1057--safereact.review.gnosisdev.com/app

Seems like this has been pending for a few days. Can this be merged?
CC @mikheevm

Travis automatic deployment:
https://pr1057--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1057--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1057--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1057--safereact.review.gnosisdev.com/app


iOS(swift) - Rotation Gesture about Compass

TO REPRODUCE THE ISSUE, FOLLOW THESE STEPS:
I'm trying to make compass Btn.
I set gesturedelegate.
self.gestureDelegate = self

and about transform code is Here
func mapView(_ view: TGMapViewController, recognizer: UIGestureRecognizer, shouldRecognizeRotationGesture location: CGPoint) -> Bool{
        DispatchQueue.main.async {
            self.compassBtn.transform = CGAffineTransform(rotationAngle: CGFloat(self.rotation))
            self.layoutAboutCompassBtn() // This code is set compassBtn's frame.
        }
        return true
    }

and compassButton rotate like 3D.
How I can fix at 2D?
and if I delete about self.layoutAboutCompassBtn() which is about setting compassBtn's frame.
rotation is okay. but when I move compass button, then compass button is weird.
like this.

RESULT:
This picture is compass button that not rotate.

and this picture is rotated picture.

I just want rotate compass button 2D, but It rotate like 3D.
EXPECTED RESULT:
not 3D just rotate compass button 2D.
ENVIRONMENT:
operating system :
OS : Sierra version : 10.12.5
device : iPhone6s.
I use tangram-es version 0.6.2 in cocoa pods
OTHER:
I really thank you for your library lol :).
Could you solve my problem?

Hey @seubseub!
This kind of feature is out of the scope of Tangram, but we have a higher level SDK where you might want to open an issue for a feature request for support of a compass similar to you what you need here. (cc @msmollin)
Concerning your specific problem, I am quite unsure of what happens in layoutAboutCompassBtn that could interfere with your transform? I don't see any wrong usage of CGAffineTransform here after looking at https://developer.apple.com/reference/coregraphics/cgaffinetransform/1455666-init. Maybe you can also explain the need of DispatchQueue.main.async in the callback, I am not sure that this is necessary.

@karimnaaji yes I thought that this may be out of the scope of Tangram too. I apologize for my issue. sorry.
About DispatchQueue.main.async,  I thought that maybe your callback is not ui thread like my previous issue about mapData.clear() #1445, which don't do in main thread.  but I tried without DispatchQueue.main.async, It works well. I'll delete them :).
About layoutAboutCompassBtn : It just set compassBtn's frame again. like  self.compassBtn.frame = CGRect(x: x, y:  y, width: width, height:height)
and transform works well. but when I just move button's y value, button's appearance is weird like my picture that I uploaded.
Maybe My mistake... I'll try again. Thank you for kind reply.  👍

@karimnaaji I solve problems. thank's for your help. I rotate my compass. so I change transform to layer transform. like
self.compassBtn.transform = CGAffineTransform(rotationAngle: CGFloat(self.rotation))

to
self.compassBtn.layer.transform = CATransform3DMakeRotation(CGFloat(self.rotation), 0, 0, 1.0)

then It works like a charm! Thank you!

Glad it worked out :)
Concerning DispatchQueue.main.async, this is indeed not required on our callbacks since all of them are already called from the main thread.


add super simple hello world

I see a lot of guys who want to compare (web) frameworks in terms of the number of characters used for a Hello World application. This PR is for these guys.
Pippo.send("Hello World");

Now Pippo is the best :)

Hehehe. Good one. :)


TV is not using the available screen space

Problem
As per titile:

EndeavourOS, kitty - Hack 16pt (I'm old :smile: )
Feature
Decrease the borders

Read the docs more carefully: https://github.com/alexpasmantier/television?tab=readme-ov-file#customization
You can edit config.toml and add:
[ui]
ui_scale = 100

I think as evidenced by OP, and my own confusion/frustration, perhaps it would make sense to change the default UI scale to a normal 100? I've never seen a CLI/TUI app constrict itself to a default of 80% of available space before, and on my terminal with my viewing size even the help text about the keybinds is partially cut off. It feels a little janky, and like the user has something wrong about their setup, when it is really just a poor default.

Indeed, that makes sense. Will issue a PR for that asap 👍
Thanks for the feedback.

Done in #47

Thank you!


[Rework] Add checks on workspace params

@amberhassaan I have to revert #2498 for now but the work is still valuable even critical so we would like to update that feature branch further before we can merge another PR on it
CC: @JehandadKhan @atamazov
[Observations}:
CI: http://micimaster.amd.com/blue/organizations/jenkins/MLLibs%2FMIOpen/detail/develop/1265/pipeline
Also observed on gfx1100:
LastTest.log
It appears that the value of workSpaceSize is not updated from the latest evaluated solver:

In this case, after "[FindSolutionImpl] ConvDirectNaiveConvWrw" with "[ValidateWorkspace] workSpace = 0x7f6ea4607000, workSpaceSize = 8192", eventually the chosen one "[FindConvFwdAlgorithm] FW Chosen Algorithm: ConvDirectNaiveConvFwd" which should have "[ValidateWorkspace] workSpace = 0, workSpaceSize = 0"
However, workSpace seems to be updated while workSpaceSize is not updated from the last value, so it stayed at 8192 in "[ValidateWorkspace] workSpace = 0, workSpaceSize = 8192"
This will cause the workSpace check to fail

@junliume : I'll take a look. It seems the check did its job :). I have a fix in progress here and I'll make sure it handles the case above too: https://github.com/ROCmSoftwarePlatform/MIOpen/pull/2524

[Notice] @junliume

the work is still valuable even critical

This is definitely valuable! but I am not sure if this is critical or not. Maybe there are (or were) some P0 tickets related to incorrect workspace ptr/size passed by the client to the library?

I have to revert https://github.com/ROCmSoftwarePlatform/MIOpen/pull/2498 for now

That's exactly the case I meant in https://github.com/ROCmSoftwarePlatform/MIOpen/pull/2498#issuecomment-1802910907.


draw.line does not work with array of points

In https://github.com/wout/svg.js/blob/master/src/line.js#L19 it checks for 4 arguments but if called from https://github.com/wout/svg.js/blob/master/src/line.js#L42 it will always get 4 arguments, even if the last three are undefined and the first one contains an array of points.

True, It should be checked for single arguments instead of the lengths of the arguments

Probably just checking whether y1 is undefined or not would do.


Fix application submission journey

Changes in this PR
The PR addresses several issues around application submission

The "confirm" check box and submit button no longer appear until the user has completed all tasks in the application
If the "confirm" check box is not checked, and the user clicks submit, an error is show, and the task list no longer disappears
If the user changes their answers after completing "Check Your Answers", they must go through CYA again

Screenshots of UI changes
Before

After

Release checklist
Release process documentation.
Pre-merge checklist

[ ] Are any changes required to dependent services for this change to work?
(eg. CAS API)
[ ] Have they been released to production already?

Post-merge checklist

[ ] Manually approve
release to test
[ ] Manually approve
release to preprod
[ ] Manually approve
release to prod

 Fix application submission journey

Looks good, thanks

@Harriethw Thanks! Yep, you probably need something like this - https://github.com/ministryofjustice/hmpps-approved-premises-api/pull/742

@Harriethw Looking at this again I think I misunderstood your question. Applications in CAS3 aren't wired up to the dashboard yet, you need to manually navigate to localhost:3000/applications


Removing unused Data placeholders in the Announcement Page

Subject of the issue
Some Rows in the Announcements tab is empty
This maybe due to some discrepancy in the data in the db and should be removed as more priority announcements will be added in the near future.

If these are still left out more the build times might increase

This has been done.

Cool.. Closing this


ReactNative.I18nManager.localeIdentifier is undefined in iOS

Description
Android exposes localeIdentifier constant I18nManagerModule.java#L48
The iOS versions doesn't seem to expose it resulting in undefined RCTI18nManager.m#L30
Is this for any specific reason?

React Native version: 0.36.0
Platform: iOS

Thanks

I doubt it. Do we even document this constant? Doesn't seem like we do. You might want to try using something like https://www.npmjs.com/package/react-native-locale . Anyway I am going to close this issue because we are trying to focus issues on bug reports, but thanks for reporting this!


Request to relocate repo to NW Utils

Hi @zcbenz,
I'm a member of a GitHub organisation called NW Utils (https://github.com/nwutils/). I'm trying to consolidate NW tooling so it's easier to find useful projects and give much needed life to stagnant ones. I also think this would be a great opportunity to exchange ideas and learn new things.
Let me know what you think and if you’d be willing to.
Best Regards
Ayushman

I can transfer this project to your organization if you are interested in maintaining it.

If you're not going to maintain it anymore I can take over. There's a lot of good examples of how to use NW.js in this repo!

@zcbenz thanks!


Fixes #100 : Stopwatch

Description
This is a Stopwatch Extension, with a clean and attractive UI, that allows users to concentrate on their tasks, while recording the time taken to finish that task.
Fixes:  #100

Type of change

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

Checklist:

[x] I have made this from my own
[ ] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@Sulagna-Dutta-Roy Please review the code and if possible, kindly assign Level 2 to this issue.
Thank you!

@Sulagna-Dutta-Roy Please review the code and if possible, kindly assign Level 2 to this issue.
Thank you!

I've checked the code. Also levels are defined in terms of more complexity and ideas. So assigned to level1.

Okay mam, no issues!


🔩 Ticket #12345 - Create Azure Virtual Network from Backstage.

Creating Azure Virtual Network in stack bobitfhat1
This is an initial pull request to create an Azure Virtual Network with suffix 001 in stack bobitfhat1 and was created based on the Backstage template.
If you need to add more parameters, check the official documentation - https://github.com/claranet/terraform-azurerm-vnet
created by: Backstage Software Template 👷♂⚙👷♀

Terraform Format and Style 🖌success
Terraform Initialization ⚙success
Terraform Validation 🤖Success! The configuration is valid.
Terraform Plan 📖success
Show Plan

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # module.azure_virtual_network_12345.azurerm_virtual_network.main_vnet will be created
  + resource "azurerm_virtual_network" "main_vnet" {
      + address_space       = [
          + "<IP_ADDRESS>/16",
        ]
      + dns_servers         = []
      + guid                = (known after apply)
      + id                  = (known after apply)
      + location            = "eastus"
      + name                = "vnet-bobitfhat1-client-use-prod-001"
      + resource_group_name = "rg-bobitfhat1-client-prod"
      + subnet              = (known after apply)
      + tags                = {
          + "CostCenter"  = "00000"
          + "Criticality" = "High"
          + "Entity"      = "UK"
          + "Owner"       = "Bob Tayara"
          + "env"         = "prod"
          + "stack"       = "bobitfhat1"
        }
    }

  # module.rg.azurerm_resource_group.main_rg will be created
  + resource "azurerm_resource_group" "main_rg" {
      + id       = (known after apply)
      + location = "eastus"
      + name     = "rg-bobitfhat1-client-prod"
      + tags     = {
          + "env"   = "prod"
          + "stack" = "bobitfhat1"
        }
    }

Plan: 2 to add, 0 to change, 0 to destroy.

Pusher: @ibrt2016, Action: pull_request, Workflow: Terraform Plan


Add EvalPushDown rule in LogicalPlanOptimizer

Description
Add a rule EvalPushDown.PUSH_DOWN_LIMIT, this rule will push down limit under eval. Thus, limit has chance to be pushed down into TableScanBuilder later.
e.g.
POST _plugins/_ppl/_explain
{
  "query": """
    source=opensearch_dashboards_sample_data_flights |  eval FlightMin = FlightTimeMin | head 5 | fields  FlightMin
  """
}

# Before optimization
{
  "root": {
    "name": "ProjectOperator",
    "description": {
      "fields": "[FlightMin]"
    },
    "children": [
      {
        "name": "LimitOperator",
        "description": {
          "limit": 5,
          "offset": 0
        },
        "children": [
          {
            "name": "EvalOperator",
            "description": {
              "expressions": {
                "FlightMin": "FlightTimeMin"
              }
            },
            "children": [
              {
                "name": "OpenSearchIndexScan",
                "description": {
                  "request": """OpenSearchQueryRequest(indexName=opensearch_dashboards_sample_data_flights,

sourceBuilder={"from":0,"size":200,"timeout":"1m"}, searchDone=false)"""
                },
                "children": []
              }
            ]
          }
        ]
      }
    ]
  }
}

# After optimization
{
  "root": {
    "name": "ProjectOperator",
    "description": {
      "fields": "[FlightMin]"
    },
    "children": [
      {
        "name": "EvalOperator",
        "description": {
          "expressions": {
            "FlightMin": "FlightTimeMin"
          }
        },
        "children": [
          {
            "name": "OpenSearchIndexScan",
            "description": {
              "request": """OpenSearchQueryRequest(indexName=opensearch_dashboards_sample_data_flights,

sourceBuilder={"from":0,"size":5,"timeout":"1m"}, searchDone=false)"""
            },
            "children": []
          }
        ]
      }
    ]
  }
}

Note: Also added TODOs in this PR to implement rules PUSH_DOWN_SORT and PUSH_DOWN_PROJECT as follow-ups, which are more complex due to expression replacement.
Related Issues
Resolves #2903
Check List

[x] New functionality includes testing.
[x] New functionality has been documented.
[x] New functionality has javadoc added.
[ ] New functionality has a user manual doc added.
[ ] API changes companion pull request created.
[x] Commits are signed per the DCO using --signoff.
[ ] Public documentation issue/PR created.

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

@qianheng-aws could you update the PR title more precise? PR title will be recorded in release notes.


[WIP] Model multi connection

This PR addresses an issue encountered by a customer with several of his tables in MySQL, and a few distributed tables in postgresql.
The problem was this: https://github.com/rails/rails/blob/master/activerecord/lib/arel/nodes/node.rb#L36
was picking the ActiveRecord::Base engine, which in their case is MySQL

@louiseGrandjonc I can confirm that the fix works please merge and release a new version

@louiseGrandjonc I can confirm that the fix works please merge and release a new version


Forward+ acts like compatability

Tested versions
v4.2.stable.official [46dc27791] tested only
System information
windows 7 v4.2.stable Forward+ rendering GeForce 540 GT embedded without vulkan support
Issue description
probably just error absence issue

as seen here Volumetric fog is set ot be enabled. settings to default.

no visible fog in the scene with simple MeshInstance3D in it.

(dont mind warning in russian language, its about .blend files)
popup ans console warning of Volumetric fog working only in Forward+(happens with all Forward+ only features)
Steps to reproduce
Dowload MRP and play it, i guess it works only with non-vulkan graphics card.
Minimal reproduction project (MRP)
NewProj2.zip

You're not on Forward+, you're running compatibility, you're falling back to it because of a lack of vulkan support

You're not on Forward+, you're running compatibility, you're falling back to it because of a lack of vulkan support
Can you please test on a supported version? 4.2 is outdated, try 4.2.1 to see if it displays this clearer

the error is clearly there on load. i dont why, i remember being on 4.2.2rc1. anyway error solved, sorry for time losed on this.

No problem at all :)


Unexpected message during docker startup

Describe the bug
scargo version: 1.6.0
I'm getting following message when entering the docker:
/usr/lib/python3/dist-packages/paramiko/transport.py:219: CryptographyDeprecationWarning: Blowfish has been deprecated
"class": algorithms.Blowfish,
To Reproduce

scargo new presp32 --target esp32
scargo docker run

Expected behavior
This message should not pop up in the console during docker startup

What pops up in 1.6.5 is:
/opt/esp-idf/tools/check_python_dependencies.py:12: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
  import pkg_resources

Fixed in https://github.com/Spyro-Soft/scargo/pull/347


Add phalcon extension

pecl install phalcon

Supported versions: 7.4, 8.0 and 8.1

Why not. BTW PR is welcome!

Thank you @joseluisq
This will definitely help our development!

@niden @Jeckerson
when I try to build using the following
ARG PHALCON_VERSION="5.1.2"

RUN set -eux \
    && git clone --depth=1 --branch=v${PHALCON_VERSION} https://github.com/phalcon/cphalcon.git \
    && cd cphalcon/ext \
    && phpize && ./configure \
    && make -j$(nproc) \
    && make install \
    && docker-php-ext-enable phalcon \
    && true

I got this inside the container.
$ php -m

Warning: PHP Startup: Unable to load dynamic library 'phalcon' (tried: /usr/local/lib/php/extensions/no-debug-non-zts-20200930/phalcon (Error loading shared library /usr/local/lib/php/extensions/no-debug-non-zts-20200930/phalcon: No such file or directory), /usr/local/lib/php/extensions/no-debug-non-zts-20200930/phalcon.so (Error relocating /usr/local/lib/php/extensions/no-debug-non-zts-20200930/phalcon.so: backtrace: symbol not found)) in Unknown on line 0
[PHP Modules]
Core
ctype
curl
date
dom
fileinfo
filter
ftp
hash
iconv
json
libxml
mbstring
mysqlnd
openssl
pcre
PDO
pdo_sqlite
Phar
posix
readline
Reflection
session
SimpleXML
sodium
SPL
sqlite3
standard
tokenizer
xml
xmlreader
xmlwriter
zlib
[Zend Modules]

The PHP.ini is created at /usr/local/etc/php/conf.d/docker-php-ext-phalcon.ini

The reason I would like to go with installing it in that way rather than the pecl install phalcon is that when using pecl I faced a "Virtual memory exhausted: Cannot allocate memory" error on CI.
pecl install phalcon is working in the native Docker build (x86_64) but when I use qemu for example it fails like this:
#8 4373.2  2018 | static zend_always_inline bool zend_parse_arg_array(zval *arg, zval **dest, bool check_null, bool or_object)
#8 4373.2       |                                                                ~~~~~~~^~~~
#8 4373.2 virtual memory exhausted: Out of memory
#8 4373.2 make: *** [Makefile:209: phalcon.lo] Error 1
#8 4373.2 ERROR: `make' failed

https://github.com/joseluisq/alpine-php-fpm/actions/runs/3625455830/jobs/6113522781#step:7:16006

@Jeckerson great! I will give it a try with PHP 8.1

BTW I will also include phalcon in the next 8.0.27 release.

Update: on PHP 8.0, phalcon is still failing with a "virtual memory exhausted: Out of memory" error via qemu.
https://github.com/joseluisq/alpine-php-fpm/actions/runs/3763057946/jobs/6396274244#step:6:35992
Any idea how to make it build?

@Jeckerson might be able to shed some more light here  but from what I recall (and looking at my old Stack overflow comment)
https://stackoverflow.com/questions/22819934/compiling-phalcon-virtual-memory-exhausted-cannot-allocate-memory

Add these flags to the make command

$CFLAGS --param ggc-min-expand=0 --param ggc-min-heapsize=8192

so that will make the script:
cd ext
phpize
./configure --enable-phalcon CFLAGS="-O2 -Wall -fvisibility=hidden -flto -DZEPHIR_RELEASE=1 --param ggc-min-expand=0 --param ggc-min-heapsize=8192"
make -s
sudo make -s install

Second option is to install from the ext/ folder and not the build one. From what Andres wrote in our forum years ago, the build folder has one file that has the whole build recipe for all components. That is convenient but apparently uses more memory.

cd ext/
./install

Third is to increase the swap of the container at least for the build process. The article is for Ubuntu but I am sure there is something for Alpine also.

https://www.digitalocean.com/community/tutorials/how-to-add-swap-on-ubuntu-14-04
Notable is that I have the same issue with a bug report, where a member of the community sent me a repo with test code to reproduce a bug and the docker containers are in Alpine. I have several failures with it and was diagnosing it also :/

@niden Yes, it might help.

cd ext
phpize
./configure --enable-phalcon CFLAGS="-O2 -Wall -fvisibility=hidden -flto -DZEPHIR_RELEASE=1 --param ggc-min-expand=0 --param ggc-min-heapsize=8192"
make -s
sudo make -s install

This first option is slower and consumes all the allowed time per running job (6h). So discarded.

Second option is to install from the ext/ folder and not the build one. From what Andres wrote in our forum years ago, the build folder has one file that has the whole build recipe for all components. That is convenient but apparently uses more memory.

cd ext/
./install

I did that https://github.com/joseluisq/alpine-php-fpm/issues/16#issuecomment-1339005919 for 8.0 as well but it failed some time ago. But curious that It works for 8.1.

Third is to increase the swap of the container at least for the build process. The article is for Ubuntu but I am sure there is something for Alpine also.

I will investigate if I can address the problem in a similar way increasing swap space.

Minor update about Phalcon extension. Looks like you guys made possible to build the latest Phalcon 5.x for PHP 8.2.
It already passed the building process for the latest PHP 8.2.4 using Phalcon 5.2.1, so I will release it today as well as PHP 8.1.17 where it already is working.

But for PHP 8.0, it is still running out of memory on KVM ones with the latest Phalcon 5.2.1, which is weird because it is not the case for 8.1 or 8.2.


Add your plugins to awesome-yazi

Hey, I noticed your plugins are up on yazi-rs GitHub, you can also add all ur plugins added there to awesome-yazi list! thanks.

Could you add them for me?

Yes I can, although would feel nice to have other people add in. For any more plugins later on, do add them in the repo. thanks!


It is not clear what regex syntax Mull accept

Description
There are at least two Mull options where it accepts regex: --include-path regex and --exclude-path regex  (see documentation). And it is not clear what syntax should be used.
There are PCRE, POSIX and other syntax. It is worth to specify which one user should use.
I suppose Mull uses std::regex.
I have a test project with following files:
sergeyb@pony:~/sources/mull_testing$ ls -1
ddd
example
example.c
func1.c
func2.c
main
main.c
Makefile

with grep I can flter files with prefix func, extension .c and number after a prefix:
sergeyb@pony:~/sources/mull_testing$ ls -1 | grep -E "func[0-9]+\.c"
func1.c
func2.c

But Mull ignores such regex:
$ mull-cxx --exclude-path="func[0-9]+\.c" ./main
...
[info] Survived mutants (6/6):
/home/sergeyb/sources/mull_testing/func1.c:6:11: warning: Survived: Replaced + with - [cxx_add_to_sub]
    a = b + c;
          ^
/home/sergeyb/sources/mull_testing/func2.c:5:11: warning: Survived: Replaced + with - [cxx_add_to_sub]
    c = a + b;
          ^
/home/sergeyb/sources/mull_testing/func2.c:7:11: warning: Survived: Replaced - with + [cxx_sub_to_add]
    a = b - c;
          ^
/home/sergeyb/sources/mull_testing/func2.c:8:11: warning: Survived: Replaced - with + [cxx_sub_to_add]
    b = a - b;
          ^
/home/sergeyb/sources/mull_testing/main.c:4:11: warning: Survived: Replaced + with - [cxx_add_to_sub]
    a = a + 2;
          ^
/home/sergeyb/sources/mull_testing/main.c:5:11: warning: Survived: Replaced + with - [cxx_add_to_sub]
    a = 2 + a;
          ^
...

Full output:
[debug] Diagnostics: Debug Mode enabled. Debug-level messages will be printed.
[info] Extracting bitcode from executable (threads: 1)

       [################################] 1/1. Finished in 1ms
[info] Loading bitcode files (threads: 3)

       [################################] 3/3. Finished in 11ms
[info] Sanity check run (threads: 1)

       [################################] 1/1. Finished in 4ms
[info] Gathering functions under test (threads: 1)

       [################################] 1/1. Finished in 0ms
[info] Applying function filter: no debug info (threads: 8)

       [################################] 9/9. Finished in 2ms
[info] Applying function filter: file path (threads: 4)

       [################################] 4/4. Finished in 0ms
[info] Instruction selection (threads: 4)

       [################################] 4/4. Finished in 1ms
[info] Searching mutants across functions (threads: 4)

       [################################] 4/4. Finished in 0ms
[info] Applying filter: no debug info (threads: 8)

       [################################] 16/16. Finished in 1ms
[info] Applying filter: file path (threads: 8)

       [################################] 16/16. Finished in 1ms
[info] Applying filter: junk (threads: 8)
[debug] ASTMutationStorage: recording mutation "Add to Sub": /home/sergeyb/sources/mull_testing/main.c:4:11
[debug] ASTMutationStorage: recording mutation "Add to Sub": /home/sergeyb/sources/mull_testing/main.c:5:11

       [##################--------------] 9/16
[debug] ASTMutationStorage: recording mutation "Add to Sub": /home/sergeyb/sources/mull_testing/func2.c:5:11
[debug] ASTMutationStorage: recording mutation "Add to Sub": /home/sergeyb/sources/mull_testing/func1.c:6:11
[debug] ASTMutationStorage: recording mutation "Sub to Add": /home/sergeyb/sources/mull_testing/func2.c:7:11
[debug] ASTMutationStorage: recording mutation "Sub to Add": /home/sergeyb/sources/mull_testing/func2.c:8:11

       [################################] 16/16. Finished in 32ms
[info] Deduplicate mutants (threads: 1)

       [################################] 1/1. Finished in 0ms
[info] Prepare mutations (threads: 1)

       [################################] 1/1. Finished in 0ms
[info] Cloning functions for mutation (threads: 3)

       [################################] 3/3. Finished in 10ms
[info] Removing original functions (threads: 3)

       [################################] 3/3. Finished in 11ms
[info] Redirect mutated functions (threads: 3)

       [################################] 3/3. Finished in 10ms
[info] Applying mutations (threads: 1)

       [################################] 6/6. Finished in 1ms
[info] Compiling original code (threads: 3)
[debug] Emitted object file: /tmp/mull-18a2ca.o for bitcode file: /tmp/mull-e1084d.bc
[debug] Emitted object file: /tmp/mull-53c353.o for bitcode file: /tmp/mull-755eae.bc
[debug] Emitted object file: /tmp/mull-0e3966.o for bitcode file: /tmp/mull-ef1a27.bc

       [################################] 3/3. Finished in 21ms
[info] Link mutated program (threads: 1)
[debug] Link command: clang /tmp/mull-0e3966.o /tmp/mull-18a2ca.o /tmp/mull-53c353.o -o /tmp/mull-d250bb.exe
[info] Compiled executable: /tmp/mull-d250bb.exe

       [################################] 1/1. Finished in 80ms
[info] Warm up run (threads: 1)

       [################################] 1/1. Finished in 2ms
[info] Baseline run (threads: 1)

       [################################] 1/1. Finished in 2ms
[info] Running mutants (threads: 6)

       [################################] 6/6. Finished in 11ms
[info] Survived mutants (6/6):
/home/sergeyb/sources/mull_testing/func1.c:6:11: warning: Survived: Replaced + with - [cxx_add_to_sub]
    a = b + c;
          ^
/home/sergeyb/sources/mull_testing/func2.c:5:11: warning: Survived: Replaced + with - [cxx_add_to_sub]
    c = a + b;
          ^
/home/sergeyb/sources/mull_testing/func2.c:7:11: warning: Survived: Replaced - with + [cxx_sub_to_add]
    a = b - c;
          ^
/home/sergeyb/sources/mull_testing/func2.c:8:11: warning: Survived: Replaced - with + [cxx_sub_to_add]
    b = a - b;
          ^
/home/sergeyb/sources/mull_testing/main.c:4:11: warning: Survived: Replaced + with - [cxx_add_to_sub]
    a = a + 2;
          ^
/home/sergeyb/sources/mull_testing/main.c:5:11: warning: Survived: Replaced + with - [cxx_add_to_sub]
    a = 2 + a;
          ^
[info] Mutation score: 0%
[info] Total execution time: 212ms

Version
Mull: LLVM-based mutation testing
https://github.com/mull-project/mull
Version: 0.10.0-trunk1
Commit: eed5960
Date: 22 Mar 2021
LLVM: 11.0.0

Mull uses std::regex with grep Syntax Option.
We should certainly document it and check whether the grep is the best default.


fix #1257 and unicode error in Contrast importer

fix #1257
**Note: DefectDojo is now on Python3 and Django 2.2. Please submit your pull requests to the 'python3_dev' branch as the 'dev' branch is only for bug fixes. Any PR's submitted to the 'dev' branch should also be converted to Python3 and submited to the 'python3_dev' branch.
When submitting a pull request, please make sure you have completed the following checklist:

[x] Your code is flake8 compliant
[ ] If this is a new feature and not a bug fix, you've included the proper documentation in the ReadTheDocs documentation folder. https://github.com/DefectDojo/Documentation/tree/master/docs or provide feature documentation in the PR.
[ ] Model changes must include the necessary migrations in the dojo/dd_migrations folder.
[ ] Add applicable tests to the unit tests.

I'm not quite sure if this actually would still replicate on python3 ? Otherwise it looks good

hello, @adracea I have tested it on python3 with a contrast scan sample file from the sample repo. and it's working fine.


Error during loading

Lua: Error during loading:
wswamp-dis... :89: bad argument #1 to 'new' (field_nw: a field with this name must exist)
Version 2.6.1 (Git v2.6.1 packaged as 2.6.1-1)  with Lua 5.2.4

I noticed that on my side - Version 2.6.2 (v2.6.2-0-g1b3cedbc) with Lua 5.2.4 running on Mac OS X 10.13.6, build 17G65 (Darwin 17.7.0) - the same error happens when renaming as you did (wswamp -> ws-wamp), whereas it does not happen with another rename (ws_wamp) I tried.
Can you try on your side under Linux?
In case it works for you too, I would suggest another pull accordingly.
Thanks and regards


Add api types to toc

Fixes #1450.
I'll add a PR for the docs folder too.

Signed-off-by: Carlisia<EMAIL_ADDRESS>

We're probably going to end up having to redo this once the new site PR goes in, so we may just want to hold off until then.

Maybe they'll end up rebasing? Either way is fine.

so...unfortunately you'll have to redirect this to master, to the site/docs dir

Np!

Closing in lieu of https://github.com/heptio/velero/pull/1496.


errors: invalid tcpdump header

Hi, I've been trying to get your program to work, but I get a message -packages\dpkt\pcap.py", line 122, in init raise ValueError('invalid tcpdump header')  ValueError: invalid tcpdump header
My xml file is from the CME (followed your link) and so is the data which is e-mini futures posted at http://www.cmegroup.com/market-data/datamine-historical-data.html
I was running 2.7 python (anaconda) and ran setup.py before running mdp_decoder.py
I ran your tests and they were fine, but it seems that you don't use real data in your test.
Any suggestions?
Thanks in advance.

@rumcode Thanks for trying the SBE decoder. I took a look at those sample files at http://www.cmegroup.com/market-data/datamine-historical-data.html.  While the files there are MDP 3.0 they aren't SBE encoded -- they are plan ASCII FIX files:
$ gunzip XCME_MD_ES_FUT_20160315.gz
$ cat XCME_MD_ES_FUT_20160315 | tr '\001' '\|' | head
1128=9|9=131|35=f|49=CME|34=5915937|52=20160314214500009131503|60=20160314214500001946065|75=20160315|5799=10000000|1151=ES|326=21|327=0|1174=4|10=075|
1128=9|9=181|35=X|49=CME|75=20160315|34=5915938|52=20160314214500437290133|60=20160314214500437207105|5799=10000100|268=1|279=0|269=1|48=49705|55=ESH6|83=2434488|270=201875.0|271=1|346=1|1023=1|10=153|
1128=9|9=182|35=X|49=CME|75=20160315|34=5915939|52=20160314214500964855160|60=20160314214500964728850|5799=10000100|268=1|279=0|269=1|48=49705|55=ESH6|83=2434489|270=202425.0|271=1|346=1|1023=10|10=228|
1128=9|9=181|35=X|49=CME|75=20160315|34=5915940|52=20160314214501236422156|60=20160314214501236279685|5799=10000100|268=1|279=1|269=1|48=6505|55=ESM6|83=4428027|270=201000.0|271=14|346=5|1023=2|10=146|
1128=9|9=181|35=X|49=CME|75=20160315|34=5915941|52=20160314214501260912493|60=20160314214501260788281|5799=10000100|268=1|279=1|269=1|48=49705|55=ESH6|83=2434490|270=201875.0|271=4|346=2|1023=1|10=164|
1128=9|9=183|35=X|49=CME|75=20160315|34=5915942|52=20160314214501469025445|60=20160314214501468880637|5799=10000100|268=1|279=1|269=0|48=49705|55=ESH6|83=2434491|270=201575.0|271=12|346=4|1023=10|10=016|
1128=9|9=181|35=X|49=CME|75=20160315|34=5915943|52=20160314214502284489462|60=20160314214502284402077|5799=10000100|268=1|279=0|269=1|48=49705|55=ESH6|83=2434492|270=201900.0|271=3|346=1|1023=2|10=160|
1128=9|9=181|35=X|49=CME|75=20160315|34=5915944|52=20160314214502900288541|60=20160314214502900214739|5799=10000100|268=1|279=0|269=1|48=49705|55=ESH6|83=2434493|270=201925.0|271=3|346=1|1023=3|10=161|
1128=9|9=181|35=X|49=CME|75=20160315|34=5915945|52=20160314214503380287759|60=20160314214503380201283|5799=10000100|268=1|279=0|269=1|48=49705|55=ESH6|83=2434494|270=201950.0|271=3|346=1|1023=4|10=168|
1128=9|9=181|35=X|49=CME|75=20160315|34=5915946|52=20160314214503973789379|60=20160314214503973709166|5799=10000100|268=1|279=0|269=1|48=49705|55=ESH6|83=2434495|270=202000.0|271=3|346=1|1023=6|10=193|

The upside for you is that you don't need to decode those files to use them.  The downside is that you can't take advantage of sbe decoder's ability to print the tag=value pairs in a more human readable format or the sbe decoder's ability to assemble market data books.  :-(
Since it is just FIX, it is pretty easy to work with though.  A quick and dirty approach to get something more readable is to just use "split" to parse the lines into a list of key value pairs and then use a dictionary lookup to translate the keys (and values as necessary) to a textual form.
If you need book building, you probably could adapt our 'orderbook' pretty easily too:
https://github.com/tfgm/sbedecoder/tree/master/orderbook


[nis] feat: jenkins docker build, publish integration

An alternative(probably better) approach to https://github.com/NemProject/nem/pull/191.
What was the issue?

in https://github.com/NemProject/nem/pull/191 dockerfile was pulling from remote repo, we weren't successful to pass the current branch to the dockerfile.
it was adding a new jenkins job to the multi-branch jenkins job which was running in any case therefore causing a performance hit.
In our discussions with @Wayonb we agreed that this can cause issues, better to build from workspace as it's been done for symbol/rest.

What's the fix?

Created an alternative Dockerfile in the nis module, which is copying the code from the jenkins workspace using the current branch which will allow us to push images from feature branches for testing. (the same way as symbol/rest was configured`)
Docker publish parameters are placed inside nis/Jenkinsfile thus we'll utilize the current job and we won't need to create any other Job like we did in https://github.com/NemProject/nem/pull/191, which puts an overhead to the Jenkins job even we don't publish any docker images.

@Wayonb comments should be fixed and also updated the nis/README.md for the docker instructions.


Add library to NPM registry.

Is there a way to use this library from the NPM registry?

The library is available via http://bower.io/
This is usually the better approach for client libraries.
npm install -g bower
bower install Leaflet.StyleEditor