Passing an empty Array as the ignore_files value generates a RegexpError

Hi @danmayer! 👋
Thanks for maintaining churn!
It seems that I've found an edge case while trying to use churn from metric_fu. I recently noticed that the Appveyor build was failing: https://ci.appveyor.com/project/bf4/metric-fu/builds/32531867/job/ja1pbtisllw801sv -- Failure is happening within the ChurnCalculator#reject_ignored_files method.
Here is the backtrace:
******* STARTING METRIC churn
Traceback (most recent call last):
	22: from bin/metric_fu:9:in `<main>'
	21: from /Users/etagwerker/Projects/fastruby/metric_fu/lib/metric_fu/cli/client.rb:19:in `run'
	20: from /Users/etagwerker/Projects/fastruby/metric_fu/lib/metric_fu/cli/helper.rb:19:in `run'
	19: from /Users/etagwerker/Projects/fastruby/metric_fu/lib/metric_fu/run.rb:9:in `run'
	18: from /Users/etagwerker/Projects/fastruby/metric_fu/lib/metric_fu/run.rb:19:in `measure'
	17: from /Users/etagwerker/Projects/fastruby/metric_fu/lib/metric_fu/run.rb:19:in `each'
	16: from /Users/etagwerker/Projects/fastruby/metric_fu/lib/metric_fu/run.rb:21:in `block in measure'
	15: from /Users/etagwerker/Projects/fastruby/metric_fu/lib/metric_fu/reporting/result.rb:48:in `add'
	14: from /Users/etagwerker/Projects/fastruby/metric_fu/lib/metric_fu/generator.rb:104:in `generate_result'
	13: from /Users/etagwerker/Projects/fastruby/metric_fu/lib/metric_fu/metrics/churn/generator.rb:12:in `emit'
	12: from /Users/etagwerker/Projects/fastruby/metric_fu/lib/metric_fu/metrics/churn/generator.rb:35:in `run'
	11: from /Users/etagwerker/Projects/fastruby/metric_fu/vendor/bundle/ruby/2.5.0/gems/churn-1.0.5/lib/churn/calculator.rb:56:in `report'
	10: from /Users/etagwerker/Projects/fastruby/metric_fu/vendor/bundle/ruby/2.5.0/gems/churn-1.0.5/lib/churn/calculator.rb:95:in `analyze'
	 9: from /Users/etagwerker/Projects/fastruby/metric_fu/vendor/bundle/ruby/2.5.0/gems/churn-1.0.5/lib/churn/calculator.rb:191:in `calculate_revision_changes'
	 8: from /Users/etagwerker/Projects/fastruby/metric_fu/vendor/bundle/ruby/2.5.0/gems/churn-1.0.5/lib/churn/calculator.rb:191:in `each'
	 7: from /Users/etagwerker/Projects/fastruby/metric_fu/vendor/bundle/ruby/2.5.0/gems/churn-1.0.5/lib/churn/calculator.rb:196:in `block in calculate_revision_changes'
	 6: from /Users/etagwerker/Projects/fastruby/metric_fu/vendor/bundle/ruby/2.5.0/gems/churn-1.0.5/lib/churn/calculator.rb:208:in `calculate_revision_data'
	 5: from /Users/etagwerker/Projects/fastruby/metric_fu/vendor/bundle/ruby/2.5.0/gems/churn-1.0.5/lib/churn/calculator.rb:279:in `parse_logs_for_updated_files'
	 4: from /Users/etagwerker/Projects/fastruby/metric_fu/vendor/bundle/ruby/2.5.0/gems/churn-1.0.5/lib/churn/calculator.rb:287:in `reject_ignored_files'
	 3: from /Users/etagwerker/Projects/fastruby/metric_fu/vendor/bundle/ruby/2.5.0/gems/churn-1.0.5/lib/churn/calculator.rb:287:in `reject'
	 2: from /Users/etagwerker/Projects/fastruby/metric_fu/vendor/bundle/ruby/2.5.0/gems/churn-1.0.5/lib/churn/calculator.rb:287:in `block in reject_ignored_files'
	 1: from /Users/etagwerker/Projects/fastruby/metric_fu/vendor/bundle/ruby/2.5.0/gems/churn-1.0.5/lib/churn/calculator.rb:287:in `any?'
/Users/etagwerker/Projects/fastruby/metric_fu/vendor/bundle/ruby/2.5.0/gems/churn-1.0.5/lib/churn/calculator.rb:287:in `block (2 levels) in reject_ignored_files': empty char-class: /[]/ (RegexpError)

Here are the arguments that we are using to create an instance:
ARGS: {:start_date=>"\"1 year ago\"", :minimum_churn_count=>10, :ignore_files=>[], :data_directory=>"tmp/metric_fu/scratch/churn"}

Here is a potential solution that is working locally for me:
    def reject_ignored_files(files)
      return files if @ignores.compact.any?

      files.reject{ |file, _| @ignores.any?{ |ignore| /#{ignore}/ =~ file } }
    end

If you're interested in a solution like that, I can submit a PR.
Please let me know.
Thanks!

So churn seems to expect a string for ignores...
@ignores             = (options.fetch(:ignore_files){ options[:ignores] || @ignores }).to_s.split(',').map(&:strip)
This will fix your issue, https://github.com/danmayer/churn/pull/80
Although I wanted to see if you could dig a bit closer, it seems like this should be passing :ignore_files as a comma-delimited string...
ARGS: {:start_date=>"\"1 year ago\"", :minimum_churn_count=>10, :ignore_files=>[], :data_directory=>"tmp/metric_fu/scratch/churn"}

see the code above that splits and the CLI expects a string arg, https://github.com/danmayer/churn/blob/master/bin/churn#L26

@danmayer Good point. I don't understand why the default values we are setting in metric_fu don't match the documentation: https://github.com/metricfu/metric_fu#example-configuration -- I'm new to the project and trying to help with maintenance.
I've submitted a PR to fix metric_fu's churn integration: https://github.com/metricfu/metric_fu/pull/313/files#diff-80da03c2cd45ea7450503d07f9c505ffL11
So I'm going to try setting that value to an empty string by default, which should work... :)

ok release 1.0.6 should resolve this issue

@danmayer Awesome, thanks for the quick response! I'll check this later today and close the issue. 👍

I can confirm this is fixed in v1.0.6 🎉


GH-26: Accommodate the new storybook implementation

@bashlk @GGKapanadze I've now published an image out of this branch, you can use this one on the design system twig-migration branch to have the test passing.
docker pull ghcr.io/international-labour-organization/ilo_base_theme:preview-gh-26-twig-migration-2


Custom pool with paragraph, sentence, word

Example:
chance.paragraph({pool: 'A'}) //  AAAAAAA AA AAAAAAA AAAAA AAA AAAAAA AAAAAAA AAAAAA AAAAA AAA AAAAAAAA AAAAAA AAAAA. AAAAA AAA AAAAAAAAA AAAAAAA AAAAA AAAA AAAAAA AAA AAAAAAAA AAAAAAAAA AAAAAAAAA AAAAAAAAA AAAAAAA AAA AAAA AAA AA. AAAAAAAA AAA AAAA AAAAAAAA AAAA AAAAA AA AAAAAAAAA AAAAAA AAAAA AAAAAAAA AAAAA AA. AAAAA AA AA AAAAAAAA AAAAA AAAAAAAA AAAAAAA AAAAAA AAAAAA AAAAAA AAAAAA AAAAAAAA AA.

Is there any ways to do this? I think this feature is useful.

Basic implementation of this premise is listed above.
Currently in Review.

Many thanks 👍


Use SignMessageLib for signing

What it solves
Resolves #2677
How this PR fixes it

Adds a handler for signMessage RPC call from Safe Apps
Introduces a new component SignMessageModal which is displayed after the interface receives such a message
If the user approves signing, a delegate call transaction is sent to SignMessageLib

How to test it
Launch https://github.com/gnosis/safe-test-app, on the main screen at the bottom there should be an input for a message, type your message and sign it. Then click the check button to see if the message has been signed.
Some cases to consider:

Regular text
UTF-8 text converted to hex -> should see UTF-8 text in the modal
Transaction hash, address -> should see the hash, it cannot be converted to utf-8

Screenshots

Tried the Hex conversion messages. Hex text has to be decoded to UTF-8 in the modal. So that works fine. But then when I check if a message is signed is just the HEX text the one that is a validated as signed, but not the UTF-8 equivalent. Is this expected?

Yeah, this is expected

Mikhail answer the question in the daily. The behavior is expected.
The ticket is fine then


fzf doesn't work with overloaded history command

Category

[x] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ ] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish

I'm using overloaded history command that strips the history entry numbering like so:
history() {
  if [ -t 1 ]; then
    # to terminal: print numbers and timestamps
    builtin history "$@"
  else
    # grepping: only command
    HISTTIMEFORMAT='' builtin history "$@" | awk '{$1="";print substr($0,2)}'
  fi
}

Without the entry numbering the ctrl+r feature of fzf doesn't work properly - when I choose the command it's not filled into the prompt
Note that with the command above stuff like !12 still works properly.

Does this fix your problem?
diff --git a/shell/key-bindings.bash b/shell/key-bindings.bash
index be24dec..2ecf685 100644
--- a/shell/key-bindings.bash
+++ b/shell/key-bindings.bash
@@ -55,7 +55,7 @@ __fzf_history__() (
   local line
   shopt -u nocaseglob nocasematch
   line=$(
-    HISTTIMEFORMAT= history |
+    HISTTIMEFORMAT= builtin history |
     FZF_DEFAULT_OPTS="--height ${FZF_TMUX_HEIGHT:-40%} $FZF_DEFAULT_OPTS --tac --sync -n2..,.. --tiebreak=index --bind=ctrl-r:toggle-sort $FZF_CTRL_R_OPTS +m" $(__fzfcmd) |
     command grep '^ *[0-9]') &&
     if [[ $- =~ H ]]; then

Yes, it does

Hi @shalak , can you explain step by step how to only display command and remove line number from history ? I trying modify the key-bindings.bash but dont accomplish

IMO when you overload a command, you have to keep its interface (input arguments, stdout output format).

Hi @shalak , can you explain step by step how to only display command and remove line number from history ? I trying modify the key-bindings.bash but dont accomplish

Hi @kiemrong08! Sorry for late reply, I must've missed it.
The code I pasted does exactly that. If I use the history command in terminal (it's checked by [ -t ] it's forced to bypass the recurrent call to said history by using builtin history, so the behavior is not changed whatsoever. But if you use history in some script (i.e. the output is not terminal), is forces consistent format by setting HISTTIMEFORMAT temporarily. Then, since the format is consistent, I can pipe the output to awk which first removes the first field from the whole line (i.e. the number). Then it prints substring of the the line starting from the second character till the end - that way the whitespace between number and command is also not printed.
@bAndie91
Dude, it's bash, it's scripting, not a strictly typed language with contracts :) The possibility to do this is the very reason for existence of the builtin command.


Image without suffix

Hi,
Current filenames in thumbd are bound to have an underscore(_suffix). Suggest a change to allow create files without suffix (Worker.js)
Worker.prototype._thumbnailKey = function (prefix, suffix, format) {   return suffix ? prefix + '_' + suffix + '.' + (format || 'jpg') : prefix + '.' + (format || 'jpg'); }
If you can add this change, it will help my team.

@akashbudhia I would happily accept a pull request for this feature, would you be willing to add it with a test?


Documentation sample for multiple replications per replication_config

Is your feature request related to a problem? Please describe.
We have a JFrog Enterprise+ license which means we're about to create multiple replications for a repository. Quite possibly it's my Terraform experience but I can't see how you do achieve this in HCL.
Describe the solution you'd like
An example showing how you configure multiple replications for the same repository would be really helpful alongside the per resource docs.
Describe alternatives you've considered
I've tried digging through other resources that use TypeList to see if that can give some hints but I can't work this one out.
Additional context
N/A

Based on reading the documentation for virtual repositories, it looks like it should be:
resource "artifactory_replication_config" "foo-rep" {
	repo_key = "${artifactory_local_repository.provider_test_source.key}"
	cron_exp = "0 0 * * * ?"
	enable_event_replication = true

	replications = [
	{
		url = "$var.artifactory_url"
		username = "$var.artifactory_username"
		password = "$var.artifactory_password"
                 path_prefix = "/xyz"
	},
	{
		url = "$var.artifactory_url"
		username = "$var.artifactory_username"
		password = "$var.artifactory_password"
                path_prefix = "/abc"
	},
        ]
}

But this gives:
│ Error: Unsupported argument
│
│   on replications.tf line 25, in resource "artifactory_replication_config" "XXX":
│   25:   replications = [
│
│ An argument named "replications" is not expected here. Did you mean to
│ define a block of type "replications"?

@mikeycmccarthy Terraform doesn't provide explicit documentation for resource schema with type TypeSet or TypeList in the general HCL documentation section. However, in the provider documentation section for TypeSet, they left a clue in "Configuration example".
For you example, the HCL should be like:

Thanks @alexhung.
I had actually tried this one, should have put it into the original issue. When I planned as above, I saw that each subsequent replication would force a change and I'd be left with one replication.
I've done some more digging by hitting the UI first and it appears that this is actually my issue. Multiple replications likely work exactly as you say, but these cannot have the same url. Terraform seems to work basically by taking the last replication value and using that, the UI complains about the URL already existing for the replication target.
I guess this is something that isn't entirely obvious unless you try on the UI first.
Let me know if you'd welcome some documentation for this corner case.

@mikeycmccarthy

Let me know if you'd welcome some documentation for this corner case.

Documentation will be welcomed. Depending on the length of the documentation, there are a few places to put it. If it is relatively short, we can add to the existing artifactory_push_replication.md. Otherwise, I can enable the wiki on this repo and add the link to artifactory_push_replication.md.

@mikeycmccarthy I'm closing this issue. Feel free to re-open this once you have a PR ready to be reviewed.


Reduce clutter when calling geometry test helpers

Update the helper functions that are used to construct geometries in tests so they
accept any kind of number rather than only Double. We almost always call them
with whole-number arguments anyway.
The point() helper method defaults to using the X coordinate as the Y
coordinate, but we were passing it the same value twice in a lot of places. Remove
the redundant second argument in those cases.
There was an alternate helper function to create Point objects, but it was only
used in one test; remove it and extend point() to support its functionality.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #1666

PR #1667   👈

PR #1668

This stack of pull requests is managed by Graphite.

Merge activity

Feb 12, 12:13 PM EST: @sgrimm started a stack merge that includes this pull request via Graphite.


Slack hyperlink to badge

Should work but might need some github md changes for direct hyperlink to work.

Codecov Report

Merging #295 (fa68eb2) into main (992d7e0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #295   +/-   ##
=======================================
  Coverage   67.41%   67.41%
=======================================
  Files         193      193
  Lines        6881     6881
  Branches      731      731
=======================================
  Hits         4639     4639
  Misses       2000     2000
  Partials      242      242

Continue to review full report at Codecov.

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

/approve
/lgtm


about typescript.check.npmIsInstalled or other bug ?

VSCode Version: 1.12.2
OS Version: Ubuntu 16.04

As the picture shows，It's about a typescript.check.npmIsInstalled or other bug ?

This is a duplicate of #24961
See this comment for the workaround we have planned for VSCode 1.13. In the next insiders build, you should be able to set "typescript.npm" to point to where you have npm installed to fix this issue

@mjbvz  thank you


Add generate container CI URL for Web App on Linux

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[ ] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[ ] Each command and parameter has a meaningful description.
[ ] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #4043 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4043      +/-   ##
==========================================
+ Coverage   70.56%   70.58%   +0.01%
==========================================
  Files         428      428
  Lines       27835    27849      +14
  Branches     4260     4263       +3
==========================================
+ Hits        19642    19656      +14
  Misses       6917     6917
  Partials     1276     1276

Impacted Files
Coverage Δ

...vice/azure/cli/command_modules/appservice/_help.py
100% <100%> (ø)
:arrow_up:

...e/azure/cli/command_modules/appservice/commands.py
86.71% <100%> (+0.1%)
:arrow_up:

...ice/azure/cli/command_modules/appservice/custom.py
73.82% <100%> (+0.37%)
:arrow_up:

Continue to review full report at Codecov.

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


Update Makefile.windows

#558 was still not perfect, because in some case as the $(TARGET_PATH)/openvr_api.dll rule create the $(WEBOTS_HOME_PATH)/include/openvr directory, the $(WEBOTS_HOME_PATH)/include/openvr rule was not called.

👍

:eyes:


Error: error thrown. Last cursor available in context.cursor_stack

julia> c = init(output_file="gen_new.jl",headers=["x1.h"])
WrapContext(Index(Ptr{Nothing} @0x0000000002a5b9f0, 0, 1), ["x1.h"], "gen_new.jl", "gen_new.jl", String[], String[], getfield(Clang, Symbol("##49#60"))(), getfield(Clang, Symbol("##42#53"))(), getfield(Clang, Symbol("##44#55")){String,String}("", "gen_new.jl"), getfield(Clang, Symbol("##50#61"))(), OrderedCollections.OrderedDict{Symbol,ExprUnit}(), DataStructures.DefaultOrderedDict{String,Array{Any,N} where N,getfield(Clang, Symbol("##45#56"))}(), Clang.InternalOptions(true, false), getfield(Clang, Symbol("##51#62"))())

julia> run(c)
[ Info: wrapping header: x1.h ...
┌ Error: error thrown. Last cursor available in context.cursor_stack.
└ @ Clang ~/.julia/packages/Clang/CiPzM/src/compat.jl:125
ERROR: AssertionError: ptr_ref[] != C_NULL
Stacktrace:
 [1] TokenList(::Ptr{Nothing}, ::Clang.LibClang.CXSourceRange) at /home/lobi/.julia/packages/Clang/CiPzM/src/token.jl:13
 [2] tokenize(::CLMacroDefinition) at /home/lobi/.julia/packages/Clang/CiPzM/src/token.jl:100
 [3] wrap!(::DefaultContext, ::CLMacroDefinition) at /home/lobi/.julia/packages/Clang/CiPzM/src/wrap_c.jl:389
 [4] run(::WrapContext, ::Bool) at /home/lobi/.julia/packages/Clang/CiPzM/src/compat.jl:122
 [5] run(::WrapContext) at /home/lobi/.julia/packages/Clang/CiPzM/src/compat.jl:89
 [6] top-level scope at REPL[4]:1

julia> c.cursor_stack
ERROR: type WrapContext has no field cursor_stack
Stacktrace:
 [1] getproperty(::Any, ::Symbol) at ./Base.jl:20

Looks like something happened in the process of tokenization. Could you share the content of "x1.h"?

https://gist.github.com/lobingera/ec5ee777f459eb923972 (note the date ...)
Actually i wanted to point out, that the error message has a problem, as c.cursor_stack seems to be existing (anymore), so it's not visible to the user where to find the stop in the input

was #129

sorry, that’s a old copy-paste bug when I was refactoring the code.

and where would i find the cursor 'state' ?

there is no way to retrieve that info when using init-run API. the context will be gc-ed when run exits. could you rewrite it based on this template: https://github.com/JuliaInterop/Clang.jl/blob/master/gen/generator.jl ?

Seriously? Clang cannot report (by default) where it stopped?

what I meant is: since there is a bug in the implementation of current init-run API, there is no way to extract that info(generated by Clang) in julia. there is no bug in the above-mentioned template code, the cursor state can be retrieved.

the init-run API is actually implemented via DefaultContext, we keep these two APIs for backward compatibility reasons.

@lobingera sorry for the late reply. I tried to wrap https://github.com/stapelberg/libxcb using the following script:
using Clang

XCB_INCLUDE = joinpath(@__DIR__, "..", "xcb") |> normpath
XCB_HEADERS = [joinpath(XCB_INCLUDE, header) for header in readdir(XCB_INCLUDE) if endswith(header, ".h")]

# create a work context
ctx = DefaultContext()

# parse headers
parse_headers!(ctx, XCB_HEADERS,
               args=["-I", joinpath(XCB_INCLUDE, ".."), map(x->"-I"*x, find_std_headers())...],
               includes=vcat(XCB_HEADERS, CLANG_INCLUDE),
               )

# settings
ctx.libname = "libxcb"
ctx.options["is_function_strictly_typed"] = false
ctx.options["is_struct_mutable"] = false

# write output
api_file = joinpath(@__DIR__, "libxcb_api.jl")
api_stream = open(api_file, "w")

for trans_unit in ctx.trans_units
    root_cursor = getcursor(trans_unit)
    push!(ctx.cursor_stack, root_cursor)
    header = spelling(root_cursor)
    @info "wrapping header: $header ..."
    # loop over all of the child cursors and wrap them, if appropriate.
    ctx.children = children(root_cursor)
    for (i, child) in enumerate(ctx.children)
        child_name = name(child)
        child_header = filename(child)
        ctx.children_index = i
        # choose which cursor to wrap
        startswith(child_name, "__") && continue  # skip compiler definitions
        child_name in keys(ctx.common_buffer) && continue  # already wrapped
        child_header != header && continue  # skip if cursor filename is not in the headers to be wrapped

        wrap!(ctx, child)
    end
    @info "writing $(api_file)"
    println(api_stream, "# Julia wrapper for header: $(basename(header))")
    println(api_stream, "# Automatically generated using Clang.jl\n")
    print_buffer(api_stream, ctx.api_buffer)
    empty!(ctx.api_buffer)  # clean up api_buffer for the next header
end
close(api_stream)

# write "common" definitions: types, typealiases, etc.
common_file = joinpath(@__DIR__, "libclang_common.jl")
open(common_file, "w") do f
    println(f, "# Automatically generated using Clang.jl\n")
    print_buffer(f, dump_to_buffer(ctx.common_buffer))
end

# uncomment the following code to generate dependency and template files
# copydeps(dirname(api_file))
# print_template(joinpath(dirname(api_file), "LibTemplate.jl"))

there are some expansion-to-defined warnings and an error:
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/pthread.h:331:6: warning: macro expansion producing 'defined' has undefined behavior [-Wexpansion-to-defined]
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/pthread.h:331:6: warning: macro expansion producing 'defined' has undefined behavior [-Wexpansion-to-defined]
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/pthread.h:540:6: warning: macro expansion producing 'defined' has undefined behavior [-Wexpansion-to-defined]
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/pthread.h:540:6: warning: macro expansion producing 'defined' has undefined behavior [-Wexpansion-to-defined]
/Users/gnimuc/Codes/Hatchery/xcb/xcb.h:168:10: fatal error: 'xproto.h' file not found
/Users/gnimuc/Codes/Hatchery/xcb/xcb_windefs.h:34:10: fatal error: 'winsock2.h' file not found
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/pthread.h:331:6: warning: macro expansion producing 'defined' has undefined behavior [-Wexpansion-to-defined]
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/pthread.h:331:6: warning: macro expansion producing 'defined' has undefined behavior [-Wexpansion-to-defined]
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/pthread.h:540:6: warning: macro expansion producing 'defined' has undefined behavior [-Wexpansion-to-defined]
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/pthread.h:540:6: warning: macro expansion producing 'defined' has undefined behavior [-Wexpansion-to-defined]
/Users/gnimuc/Codes/Hatchery/xcb/xcb.h:168:10: fatal error: 'xproto.h' file not found

/Users/gnimuc/Codes/Hatchery/xcb/xcb.h:168:10: fatal error: 'xproto.h' file not found

I didn't build libxcb, so the generated header file xproto.h was missing.

Thank you for your help.


Implement base real-value genome support

Implement support for genomes comprised entirely of real-valued vectors.

This will be the foundation for basic ES support, which includes initializers and Gaussian-based mutation operators.

Need to implement the following:

[ ] deterministic selection that shuffles populations between cycles
[ ] Ken's EV example, whereby a single static std is applied to all mutations for the run
[ ] ES that uses a single std that's annealed over a run
[ ] ES that uses "shadow stds" that're annealed over a run

We only have need to implement the shadow stds, which we'll do later.


fix(otelgorm): Make metric reporting work with prepared statements

Previously we only initialized metrics reporting by calling otelsql.ReportDBStatsMetrics if the gorm.DB.ConnPool interface pointer we received happened to be a *sql.DB. This was often the case, but not if prepared statements were enabled (and possibly in other circumstances too). This caused metrics reporting to be silently disabled. We fix this by calling gorm.DB.DB whose sole purpose is to extract the underlying *sql.DB.

Thanks


Tour and travel website

Describe the project
A clear and concise description of the project you are going to add to this repository.
Tour and travel website in html,css and js.It includes 5 slides along with other sections like login,home,etc.
Issue Type:

[x] New Project

#472 already done.


Actually This is not any issue i am just asking my requirment....

Here you added somany toolbar.. i just want just 4 needed.. first one is text adding.. second orderedList,3rd bullets, images... how can i add this..

I'm guessing you are just using the default RichEditorToolbar that comes included with the editor.
The RichEditorToolbar has an options property that takes an array of items of type RichEditorOption. Now, RichEditorOption is a protocol, so you can pass in whatever you need, and each button can perform whatever you want it to (including the RichEditorOptionItem, a struct that takes a title, optional image, and a tap handler)
If you just need some of the default options (you said ordered lists and bullets), then you might be able to pass in some of the values of RichEditorOptions, an enum that has some default options available to you. In particular, you might want do do something like:
toolbar.options = [RichEditorOptions.OrderedList, RichEditorOptions.UnorderedList]

thankx for the reply.. Is their any way to change number ordered list to checkbox list... we can edit the number list to checklist.. ?

Yeah no problem!
As far as I know, there's no way to change <ul> or <ol> to a list of checkboxes without putting a checkbox element inside each <li> tag.. If you wanted to add that kind of functionality, you'd have to write some JS in order to handle it the way you wanted. 😥

ok sir...  i just want that format...

My Requiremrnt is this....

Sorry, but that's beyond the scope of what this library provides. I would suggest finding out how to do it in HTML and JS, and loading the JS into the RichEditorView, and triggering the actions that you need to happen in there. This library just wraps around a UIWebView, so anything you can do in there, you can do in RichEditorView. You will just have to figure out how to do it.

thanks for the reply... :) :+1:

@cjwirth Need to know something, on return key tap the focus moves to the next line. where have written that code ???

I haven't written anything to make the return key go to the next line? I think that's just how the web view handles return presses. Unless something weird is going on?

@cjwirth no actually i am just asking my doubt. is this possible in uiscrollview?.

Do you mean you want something like this https://github.com/cjwirth/RichEditorView/pull/34 ?
@jesiegel1 Is working on getting it to work. It's really close to being done, but has one more bug we gotta fix.

@cjwirth  ya as same as this issue.


Incorrectly Ordered Sharing events

Setup a battle with a number of creature with initiatives
Share the battle
Rapidly advance through the initiative order
Notice it's quite easy for the player view to end up out of sync with the DM view, being a turn or two behind the actual initiative
Have definitely seen updates in the player view being applied out of order, i.e. the latest update is applied and followed by an older update, leaving the player view out of date
Not sure whether the problem is the order the graphql updates are being sent from the DM or applied in the player view

Tested this recently and was not able to reproduce the problem.


Fix pip install ".[metaworld]"

Hi! If you install fancy_gym with pip install ".[metaworld]",
it fetches MuJoCo in version 3.0.0 which seems incompatible,
causing import fancy_gym to throw an exception.
You can use this docker file to reproduce:
from fedora:38

run dnf install -y clang cmake python3-pip git glfw && dnf clean -y all

run mkdir /workdir
workdir /workdir

run git clone https://github.com/ALRhub/fancy_gym.git

run cd fancy_gym && git checkout master && pip install ".[metaworld]"

cmd cd / && python3 -c "import fancy_gym"

Will already get fixed by PR#84.
(Also addresses some other issues with optional dependencies missing causing weird Exceptions, like trying to make MP-versions of Metaworld envs if Metaworld is not installed...)


Customize Kubernetes Manifest YAML files

FEATURE REQUEST
#3840 added functionality to customize Addon YAML files. We have a requirement to customize the pod security yaml which is part of kubernetesManifestSettingsInit https://github.com/Azure/acs-engine/blob/master/pkg/acsengine/artifacts.go#L239-L272
We would like this functionality extended to kubernetesManifestSettingsInit
@tariq1890

Hi @tariq1890, sorry to chase, have you had cycles to look at this ? thanks o/

@pidah I haven't had the time to devote cycles for this unfortunately :(. Feel free to send in a PR (with tests if possible) our way and we'd be more than happy to review for you :)

Thanks @tariq1890 ! Will try and do that next week.


Please provide installation guide

Could you please provide a guide how to get it running on Windows Server? I will then try to work with your Solution.

You can download the preview releases: https://github.com/jantari/rewinged/releases
There's an executable for Windows and for Linux attached

Please let me know if you got it running or have further issues

I have also added a more complete getting started section to the readme with 563f7e7, so I'll close this as completed


how to use same repo from multiple computers

Ideal to discuss at the end: how to make a repo/code usable on multiple computers.  for example, it might involve copying a lot of data that's not in git.  Or lots of different file paths, and these might be hard-coded.  How to easily develop locally and then run remotely.  what happens if you need to edit remotely, too? - how to think about this.

Great point. Thanks.

What would your answers be?


Enhancement request for a regex mapper function to pick out fields from strings

I'm submitting a ...  (check one with "x")
[] bug report => search github for a similar issue or PR before submitting
[X] feature request
[] support request
[] general question

Current behavior (how does the issue manifest):
Limited set of functions available to manipulate the messages passed between activities in a flow.
Expected behavior:
Providing a regex function would be a powerful way to manipulate the information passed between activities in any manner desired.
Minimal steps to reproduce the problem (not required if feature enhancement):
Requested function does not exist, no need to reproduce.
What is the motivation / use case for changing the behavior?
Powerful way to make the tool more useful.
Please tell us about your environment (Operating system, docker version, browser & web ui version, etc):
MacOS X version 10.13.5
Docker version 18.03.1-ce, build 9ee9f40
Browser: Firefox Quantum 61.0
Flogo version (CLI & contrib/lib. If unknown, leave empty or state unknown): 0.X.X
Flogo version 0.5.3
Additional information you deem important (e.g. issue happens only occasionally):

Hi ... what regex function are you looking for ? Looking at https://gobyexample.com/regular-expressions which functions did you have in mind ? I was thinking about adding a MatchString function but i guess that's not what you were thinking of .....

Hi Andy,
Apologies for the delay, I missed this email somehow.
Ideally all the regular expressions matches would be supported, for instance, if I could do something like:
(a, b, c) = regex ($input_string, /^\d*(\w+)[a-f0-9]+(\w+)\s+(\S+)\w+$/)
The idea is to be able to get any kind of output format from an input string. For example, in a MQTT message, I could have a topic that says:
“Sensors/door/12”
And a message that says
“Status=open”,
I would then be able to subscribe for “Sensors/#” and create a message that says “Door 12 is open” by using the regex on both topic and message. (/^\w+/(\w+)/(\d+$)/) and (/^Status=(\w+)$/)
Thanks,
Shekhar
Shékhar Patkar | Product Marketing, Open Source | O +1<PHONE_NUMBER> <> | M +1<PHONE_NUMBER> <> | E<EMAIL_ADDRESS><EMAIL_ADDRESS>
On Jul 3, 2018, at 4:32 AM, Andy Hampshire<EMAIL_ADDRESS>wrote:
Hi ... what regex function are you looking for ? Looking at https://gobyexample.com/regular-expressions https://gobyexample.com/regular-expressions which functions did you have in mind ? I was thinking about adding a MatchString function but i guess that's not what you were thinking of .....
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/TIBCOSoftware/flogo/issues/305#issuecomment-402121880, or mute the thread https://github.com/notifications/unsubscribe-auth/Ak_yNqsA0o3HdEyLaJlaR_6trfZX-Zl3ks5uC1ZngaJpZM4VAKF6.


Enhancement request. "getProducts()" function call

This request is a really too small for a PR.
For proper (early) initiatiation of the websocket functions we need one single parameter passed as an option to the
getProducts function on line 22 of the lib/engine.js file, The parameter is s.product_id
    s.product_id = selector_parts[1]
    s.asset = s.product_id.split('-')[0]
    s.currency = s.product_id.split('-')[1]
    var products = s.exchange.getProducts(s.product_id)
    products.forEach(function (product) {
      if (product.asset === s.asset && product.currency === s.currency) {
        s.product = product
      }
    })

In the future it may prove useful to change the name to exchangeInit(params) and make it a callback for exchange spesific information such as API capabilities. Just a thought.
This change is desparately needed for further work on websockets implementation.
(Even the whole s object could have been passed. In that case it probably wold have been the smallest change ever. LOL)

@eigilb I don't understand why this is "too small for a PR", nothing is too small for a PR. Could you send in a PR showcasing what you exactly need? I think that code is easier to reason about than words.

Never mind. The change is not necessary. Instead the needed information (trading pair) is now generated from the command line params in "argv".


Add v37 Salesforce connector code to v46

Description:

Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Fixed in https://github.com/wso2-ballerina/module-salesforce/pull/60


all_gather should raise an error if it didn't gather anything.

🚀 Feature
Currently, all_gather passes silently when non-compatible type is passed into it.
Motivation
This can lead to bugs, when the wrong variable is passed in and nothing is gathered.
Pitch
Keep track if anything was actually gathered in all_gather and raise an error if not.
Alternatives
Check type during all_gather and if non-gatherable stuff is passed in, raise an Error.
Additional context

Thanks for the issue! Want to try sending a PR with the fix?

@edenlightning will take a look on weekend.


fix(mongo): update go.mod module prefix.

Isn't this running in the CI process?

I feel a little weird, too.


Default model options

This deals with issue 181. Generally I removed defaults from parser.add_argument and instead added those defaults to where the model architectures were registered. The main exception is for the fconv_self_att_wp model where I made it's defaults consistent with what is in the stories readme so that'd it match closer with the paper. I didn't change the readme, although if wanted I can simplify the train line a bit now as the model options used there now consider to the ones it'll have without specifying them.

@huihuifan, One discrepancy I've noticed and am not sure what's the source of it is the number of parameters for the wp model is about 100 million while trying to match hyperparameter values with the paper. The paper has about another 34 million parameters.


Automated Package Upgrade

:tada: This PR is included in version 25.3.56 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


STATUS_HEAP_CORRUPTION after modifying rows/columns for the breakout example

In the breakout example at SHA 3abfcad7ab5e194e1b79daedc0234e2d3afbfb24
Modifying this:
let brick_rows = 4;
let brick_columns = 5;

To this:
let brick_rows = 9;
let brick_columns = 9;

Causes this:
$ cargo run --example breakout
   Compiling bevy v0.2.1 (C:\Users\jg\github\bevy)
    Finished dev [unoptimized + debuginfo] target(s) in 8.25s
     Running `target\debug\examples\breakout.exe`
error: process didn't exit successfully: `target\debug\examples\breakout.exe` (exit code: 0xc0000374, STATUS_HEAP_CORRUPTION)

Here is my compiler version:
active toolchain
----------------

stable-x86_64-pc-windows-msvc (default)
rustc 1.46.0 (04488afe3 2020-08-24)

Can you update to latest master? I think this is fixed by #582

@cart Yep, that fixed it. Thanks!


Assumptions removal

Proved most of the assumptions in the assumptions file
Rearranged and organized code
Proved a bunch of useful lemmas and theorems
CHANGED AXIOMS so they use freshness constraints instead of disjointness (also renamed a few that were mislabeled)
NB: Tests are broken atm because the assumptions related to kleene needed freshness and positivity constraints. Those need to be added to the generation code!

Looks good besides the naming issue. I'm assuming the only change to the positivity/negativity is moving them down, and positive_disjoint => _freshness


Add preflight to Momentum section

This PR just adds a link to your new package.

Thank you Vince!


FEATURE: Turn on/off duplicate images

As spoilers get released and higher quality scans of cards are available, 'duplicate' spoilers are sent out.
There should be a UserOption to turn off duplicate spoilers, alongside a warning to users that turning off duplicate spoilers will result in only ever seeing the first, lowest-quality version of a card.
Other issues that will need to be resolved, on first examination:

If a user gets an update prompt but they have duplicates turned off, how many unseen spoilers do they have? Is it okay for the number of unseen spoilers to be different than what's sent? What if an update consists entirely of duplicates?
What about the recent command? Do we need to create an entirely new mechanism for sending cards to users based on these preferences? Is it bad UX for RECENT to follow this preference or for it to not follow this preference?

UGH FINE ITS ANNOYING LETS DO IT

Implemented via #37 and #38


Progress bar not working as intended

As you can see, I am testing deew a lot.
I have found that with very complex files. the progress bar doesn't work as intended, as it jumps from zero to 100% and doesn "scroll" at all.
My environment is pure Windows 11 (no WSL), with Command prompt under Terminal Preview.
deew python only version
Test file

If the input file's length can't be parsed, the progress bar will show a spinning animation for the intermediate phase (ffmpeg)


Cross forest authentication support.

When attempting to authenticate across forest trusts, I'm getting the following output:
INFO: Found AD domain: acme.com
INFO: Connecting to LDAP server: acme.com
Traceback (most recent call last):
  File "./bloodhound.py", line 5, in <module>
    bloodhound.main()
  File "/root/james/BloodHound.py/bloodhound/__init__.py", line 286, in main
    disable_pooling=args.disable_pooling)
  File "/root/james/BloodHound.py/bloodhound/__init__.py", line 72, in run
    self.pdc.prefetch_info('objectprops' in collect, 'acl' in collect)
  File "/root/james/BloodHound.py/bloodhound/ad/domain.py", line 387, in prefetch_info
    self.get_objecttype()
  File "/root/james/BloodHound.py/bloodhound/ad/domain.py", line 216, in get_objecttype
    self.ldap_connect()
  File "/root/james/BloodHound.py/bloodhound/ad/domain.py", line 69, in ldap_connect
    baseDN=self.ad.baseDN, protocol=protocol)
  File "/root/james/BloodHound.py/bloodhound/ad/authentication.py", line 78, in getLDAPConnection
    if not conn.bind():
  File "/usr/lib/python2.7/dist-packages/ldap3/core/connection.py", line 570, in bind
    raise LDAPUnknownAuthenticationMethodError(self.last_error)
ldap3.core.exceptions.LDAPUnknownAuthenticationMethodError: NTLM needs domain\username and a password

I did note a previous issue where it's stated that this is not fully implemented:
https://github.com/fox-it/BloodHound.py/issues/8
Is this still the case, or is there a way to get this working?

How are you providing the autoentication? For cross forest users you should specify -u<EMAIL_ADDRESS>

Yup format was indeed the problem. New problem, loads of SID resolution errors, and eventually hangs and just sits there with only users.json created. Any ideas?

Can you run it with -w 1 --disable-pooling -v and let me know where it's stuck (or a stacktrace if there is one)? And also which collection options you are using?

Also I just pushed a test to master which uses a timeout for receiving data, could you try with that one?

Did and I'm not getting the above error anymore, getting this one now:
WARNING: Failed to resolve SAM name REMOTE_FOREST_HOST$ in current forest
WARNING: LDAP Server reported that the search in  for (sAMAccountName=REMOTE_FOREST_USER) does not exist.

Keep coming up with new ones, so thanks a bunch.

So far my tests have been incomplete, let's try this again, ignoring my previous posts:
./bloodhound.py --collectionmethod All,LoggedOn --username SOURCE_USER@SOURCE_FOREST --domain DEST_DOMAIN.DEST_FOREST --workers 16 --global-catalog DC1.DEST_DOMAIN.DEST_FOREST -w 1 --disable-pooling -v
Password:
DEBUG: Resolved collection methods: localadmin, rdp, psremote, dcom, acl, objectprops, session, loggedon, group, trusts
DEBUG: Using DNS to retrieve domain information
DEBUG: Querying domain controller information from DNS
DEBUG: Using domain hint: DEST_DOMAIN.DEST_FOREST
INFO: Found AD domain: DEST_DOMAIN.DEST_FOREST
DEBUG: Found primary DC: DC1.DEST_DOMAIN.DEST_FOREST
DEBUG: Using LDAP server: DC1.DEST_DOMAIN.DEST_FOREST
DEBUG: Using base DN: DC=DEST_DOMAIN,DC=DEST_FOREST
INFO: Connecting to LDAP server: DC1.DEST_DOMAIN.DEST_FOREST
DEBUG: Authenticating to LDAP server
DEBUG: No LAPS attributes found in schema
INFO: Found 1 domains
INFO: Found 2 domains in the forest
INFO: Found 6347 computers
DEBUG: Writing users to file: users.json
DEBUG: Don't care about acetype 6
INFO: Connecting to LDAP server: DC1.DEST_DOMAIN.DEST_FOREST
DEBUG: Authenticating to LDAP server
DEBUG: Querying resolver LDAP for SID SID_DEST_DOMAIN-512
DEBUG: Querying resolver LDAP for SID SID_DEST_DOMAIN-20053
DEBUG: Querying resolver LDAP for SID SID_DEST_DOMAIN-56673
DEBUG: Querying resolver LDAP for SID SID_DEST_DOMAIN-120436
INFO: Connecting to GC LDAP server: DC1.DEST_DOMAIN.DEST_FOREST
DEBUG: Authenticating to LDAP server
DEBUG: Querying GC for SID SID_SOURCE_DOMAIN-13122
DEBUG: Querying resolver LDAP for SID SID_DEST_DOMAIN-1118
DEBUG: Querying resolver LDAP for SID SID_DEST_DOMAIN-120140
DEBUG: Querying GC for SID SID_SOURCE_DOMAIN-519
DEBUG: Don't care about acetype 6
DEBUG: Querying GC for SID SID_UNKNOWN-1515
WARNING: Could not resolve SID: SID_UNKNOWN-1515
DEBUG: Querying resolver LDAP for SID SID_DEST_DOMAIN-5937
DEBUG: Don't care about acetype 6
DEBUG: Querying resolver LDAP for SID SID_DEST_DOMAIN-39030
WARNING: Could not resolve SID: SID_DEST_DOMAIN-39030
DEBUG: Querying resolver LDAP for SID SID_DEST_DOMAIN-24810
DEBUG: Don't care about acetype 6
DEBUG: Don't care about acetype 6
DEBUG: Don't care about acetype 6
DEBUG: Querying GC for SID SID_UNKNOWN-1380
WARNING: Could not resolve SID: SID_UNKNOWN-1380
DEBUG: Don't care about acetype 6
DEBUG: Don't care about acetype 6
DEBUG: Don't care about acetype 6
DEBUG: Querying resolver LDAP for SID SID_DEST_DOMAIN-29186
DEBUG: Don't care about acetype 6
DEBUG: Don't care about acetype 6
DEBUG: Don't care about acetype 6
DEBUG: Don't care about acetype 6
DEBUG: Don't care about acetype 6
DEBUG: Don't care about acetype 6
...

When trying to look up SID_UNKNOWN, wmic hangs, and Get-ADUser gives me a SID not found error.

More bugfixes where just pushed to master, maybe those solve your timeout issue as well. Could you check?

Closing due to lack of updates, if there is any new info on this feel free to reopen.


doc. explain the existence of 'Offset'

Add comments to explain the intended use of the Offset struct member.
I've  taken Eleanor's explanation for the existence of the 'Offset' struct member and applied it as a comment in a style similar to that used in the golang libraries, for example:
https://github.com/golang/go/blob/2be7788f8383c2330cd96db53273e2995d4468f8/src/net/http/http.go#L129

This happens in a lot of places, it would be better to document in a central standards document than repeat this comment throughout the codebase

These comments moved into API_STANDARDS.md file


Training with Hidet compiler

Hello!
I was wondering if there was anything extra that needed to be done to get training with Hidet compiler working.
Out of the box I seem to be running into errors
import torch
from palm_rlhf_pytorch import PaLM

palm = PaLM(
    num_tokens = 20000,
    dim = 512,
    depth = 12,
    flash_attn = True, # https://arxiv.org/abs/2205.14135
    cross_entropy_ignore_index = 0
).to(torch.bfloat16).cuda()

palm_opt = torch.compile(palm, backend='hidet')

seq = torch.randint(0, 20000, (1, 1024)).cuda()

loss = palm_opt(seq, return_loss = True)
loss.backward()

Some of the errors I faced were around the usage of rearrange here and here.
It also seems like einsum isn't supported. Even after replacing those OPs with equivalent alternatives. I'm still running into some reshape errors from hidet
AssertionError: , occurred when interpreting reshape with
  tensor_reshape(tensor(...), [1023])

I can post additional info as needed, but wondering if you ran into those same errors or if I'm doing something incorrectly.
Thanks!

Hi @RameshArvind ,
To use torch.compile with einops for training you need to set:
from einops._torch_specific import allow_ops_in_compiled_graph  # requires einops>=0.6.1
allow_ops_in_compiled_graph()

I will have to do further investigation into the Hidet backend for training as I have only tested it out for inference.
I am also refactoring the code to provide a second training script and remove Huggingface and work directly with PyTorch FSDP/DDP.
I will work on integrating torch.compile there.
Best,
Enrico


bugfix: Avoid compositor issue of CalendarDatePicker

What does the pull request do?
We met the same issue in #8864, but I found one way to avoid the compositor issue of CalendarDatePicker
What is the current behavior?
As described in #8864.
What is the updated/expected behavior with this PR?
Calendar month names shown as normal.
Tested on both Fluent and Simple Theme.

How was the solution implemented (if it's not obvious)?
Replace ContentControl to ContentPresenter
Checklist
- [ ] Added unit tests (if possible)?
- [ ] Added XML documentation to any related classes?
- [ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation
Fixed issues
Fixes #8864

You can test this PR using the following package version. 11.0.999-cibuild0026794-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-nightly/nuget/v3/index.json) [PRBUILDID]

I believe the issue should be fixed by this PR - https://github.com/AvaloniaUI/Avalonia/pull/9487

I believe the issue should be fixed by this PR - https://github.com/AvaloniaUI/Avalonia/pull/9487

Regardless, I think ContentPresenter makes more sense and is what should have always been used here. So this PR should still get merged.

Yes. ContentPresenter is a better choice for this control.

You can test this PR using the following package version. 11.0.999-cibuild0026859-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-nightly/nuget/v3/index.json) [PRBUILDID]


[ ] concept

[ ] DO Droplet for future prod

[ ] Check ghcr (limits, howto, security, public)

[ ] Pipeline (node cache, test, build docker, push, deploy trigger...)

Orchestration

[ ] Traefik/SSL/Community Hosting
[ ] make trigger for deployment
[ ] make target rollback (nice 2 have)

Prerequisites

[ ] droplet with dockerd
[ ] unattended upgrades
[ ] firewall
[ ] able to send emails
[ ] (reachable with DNS entry)
[ ] Subdomain for droplet (requires ip first)

nice2have

[ ] zero downtime deployment

must (later)

[ ] infrastructure as code
[ ] monitoring

When this is done it potentially closes #365 and #366
Please correct me if i'm wrong


[TEST] Dummy PR for benchmarking

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/benchmark ref=master module=tabular preset=good benchmark=test time_limit=1h


Add new topic - cloud

And the first check - public SharePoint sites:
https://zolder.io/public-sharepoint-sites-the-new-open-shares/
afterwards other Azure enumeration modules.

See
https://github.com/S3cur3Th1sSh1t/WinPwn/issues/32


GPX Roadmap

GPX
References
Schema Documentation
GPX on OpenStreetMap's Wiki
GPX on Wikipedia
Good overview of the format
Somewhat related javascript Implementations
https://github.com/mpetazzoni/leaflet-gpx
http://openlayers.org/en/master/examples/gpx.html
http://dev.openlayers.org/releases/OpenLayers-2.7/doc/apidocs/files/OpenLayers/Format/GPX-js.html
https://code.google.com/p/gpxviewer/
https://www.npmjs.com/package/gpx-parse
Roadmap

[x] GpxDataSource

[x] load/parse .gpx
[x] load Metadata from gpx document

[x] Waypoint
[x] Track

[x] trk
[x] trkseg (Track Segment)
[x] trkpt (Track Point)

[x] Route
[x] Other ComplexTypes

[x] copyrightType
[x] linkType
[x] emailType
[x] personType
[x] bounds (Two lat/lon pairs defining the extent of an element.)

[x] Achieve good test coverage
[ ] Write/improve documentation
[ ] Write Tutorial/Blog post about usage in Cesium

PR with initial support here: https://github.com/AnalyticalGraphicsInc/cesium/pull/2939

GPX supported requested again on the forum: https://community.cesium.com/t/problem-rendering-kml-file-with-3d-terrain/9307.
The latest on this is that there is a branch gpx that's almost complete. Contributions welcome!


controllers: send the client profile mapping required for DR

This mapping is sent only if rns is enabled for mirroring

remote poolID is fetched from GetBlockPoolInfo rpc, which is then written to a blockpool as an annotation to be read by provider server
local poolID is fetched from cephBlockPool status

/lgtm


Active directory config

Adding authConfig for activedirectory
PR for rancher/rancher: https://github.com/rancher/rancher/pull/11274
https://github.com/rancher/rancher/issues/11162

Pulled in with another PR


chore(main): release 0.10.0

:robot: I have created a release beep boop
0.10.0 (2024-10-01)
Features

add specs for CustomRoute with POST method (53de711)

Bug Fixes

CSRF token (e891f79)
CSRF token => encrypt session (43927b0)
method for assignroutes (b4f07e5)
update package path (3da86bb)
use hook properly (04aea63)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/solisoft/luaonbeans/releases/tag/v0.10.0 :sunflower:


2 new presets and add living street to Road preset category

Add presets to:

noexit=yes -> https://wiki.openstreetmap.org/wiki/Key:noexit
shop=watches -> https://wiki.openstreetmap.org/wiki/Tag:shop%3Dwatches

and include living street prest to the road category.

Kind of iffy on noexit.  It's really only appropriate in certain situations, and it's purpose is mostly just to silence Q/A tools, it doesn't actually add any information about the endpoint of a way.  A user would need to read the wiki just to learn when to use it.
Everything else seems ok.


rerun() should not direct users to use the terminal

I hope 82f5d27 is an adequate compromise. I am hesitant to include a feature that is difficult to make CRAN compliant.
@brandmaier Please close if you approve.


Dataset Query

@zhangxy0517
I appreciate your good work. I have a query regarding the Python demo data.
Can you please let me know from which dataset are the scans used for the Python demo? Is it a3dloc dataset?
Thanks

Yes, the scans are randomly chosen from the 3DLoMatch dataset.

Yes, the scans are randomly chosen from the 3DLoMatch dataset.

Thanks


Update adaptive dialog samples to 4.9.1

Add composer dialog with LUIS sample

Can you retarget these to 4.9.2 which was just released?


ModuleTracker : Implement Archive features

Fixes #470

Codecov Report

Merging #471 into master will decrease coverage by 0.15%.
The diff coverage is 13.19%.

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


typo: authorization/resource-manager/Microsoft.Authorization

roleassignment -> role assignment
delgation -> delegation

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

Automation for azure-sdk-for-java

Can one of the admins verify this patch?

@sarangan12 could you take a look at the ruby failure?


Stop all spawned tasks on Node drop

At least the ones that are spawned in the constructor. Otherwise they could keep running indefinitely unexpectedly in situations where we remake a Node e.g. in tests.

bors r+

bors retry


Add a top-level stack element

Currently, the only top-level element is page. We could add a stack element that contains one or more page elements. It should probably also have an info element, and perhaps a title, though we need to figure out to what extent stack info is inherited by page info. This has a few advantages:

It can be easier to author and track a related set of pages.
It can allow people to put actual pages into subfolders, and just XInclude them into a stack not in the subfolder, to make management of large documents better.
Pages could be put into stacks for installation, reducing the number of on-disk files. This can be nice for plugin pages, so you just have one file for each plugin package.
It actually allows an entire document to be distributed as a single file.

Additionally, the stack element can have a type attribute that can affect its pages. Extensions can add stack types. We'd define two in the core: set and series. A set is the default and adds no semantics.  Next links are cumbersome, and this is a much easier way of making a linear series.
Question: can stacks contain stacks? What's the semantics when a series stack contains stacks?

GNOME people want this. Maybe punt to 1.2 if implementation is too difficult. Series stacks are especially useful. Stacks can NOT contain stacks, because there lies insanity.

Bumping to Mallard 1.2

http://projectmallard.org/mep/mep0013.html


Queue Shift on .end() causes null dereference

There's a small glitch when closing a connection shortly after it opens where Connection._recieve will get shift().resolve() called on a null.
The error stack looks like:
stream.js:74
throw er; // Unhandled stream error in pipe.
^
TypeError: Cannot read property 'resolve' of undefined
at Connection._receive (d:\Github\kafka-poc\node_modules\no-kafka\lib\connection.js:167:11)
at emitOne (events.js:77:13)
at Socket.emit (events.js:169:7)
at readableAddChunk (_stream_readable.js:146:16)
at Socket.Readable.push (_stream_readable.js:110:10)
at TCP.onread (net.js:523:20)

Thanks, missed that!


LaTeX not rendering in Plotly figures

Environment data

VS Code version: 1.61.2
Jupyter Extension version (available under the Extensions sidebar): v2021.9.1101343141
Python Extension version (available under the Extensions sidebar): v2021.10.1365161279
OS (Windows | Mac | Linux distro) and version: Mac Big Sur Version 11.6
Python and/or Anaconda version: Python 3.9.7
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): XXX
Jupyter server running: Tried both Local and Remote
Plotly version: 4.14.3

Expected behaviour
Successfully have LaTeX render in Plotly figures when running on either Jupyter Notebook or JupyterLab from a Chrome browser (Version 95.0.4638.54). Tested this with the examples directly from Plotly: https://plotly.com/python/LaTeX/
import plotly.express as px

fig = px.line(x=[1, 2, 3, 4], y=[1, 4, 9, 16], title=r'$\alpha_{1c} = 352 \pm 11 \text{ km s}^{-1}$')
fig.update_layout(
    xaxis_title=r'$\sqrt{(n_\text{c}(t|{T_\text{early}}))}$',
    yaxis_title=r'$d, r \text{ (solar radius)}$'
)
fig.show()

import plotly.graph_objs as go

fig = go.Figure()
fig.add_trace(go.Scatter(
    x=[1, 2, 3, 4],
    y=[1, 4, 9, 16],
    name=r'$\alpha_{1c} = 352 \pm 11 \text{ km s}^{-1}$'
))
fig.add_trace(go.Scatter(
    x=[1, 2, 3, 4],
    y=[0.5, 2, 4.5, 8],
    name=r'$\beta_{1c} = 25 \pm 11 \text{ km s}^{-1}$'
))
fig.update_layout(
    xaxis_title=r'$\sqrt{(n_\text{c}(t|{T_\text{early}}))}$',
    yaxis_title=r'$d, r \text{ (solar radius)}$'
)
fig.show()

Note: Using Chrome because there is a bug in Firefox that prevents proper rendering of Plotly figures with Mathjax (https://github.com/plotly/plotly.js/pull/5993)
It looks like this may be related to this issue: https://github.com/microsoft/vscode-jupyter/issues/6802 since you can wrap the above figure in Plotly FigureWidget:
f = go.FigureWidget(fig)
f.show()

Actual behaviour
No LaTeX rendering when using the same Jupyter notebook in VS Code.

Steps to reproduce:
Simply run any of the above code in VS Code with the same environment as listed above.

Thanks for the bug. Related issue #7801

Duplicate of https://github.com/microsoft/vscode-jupyter/issues/7801

@tomas-mazak Thanks for the workaround!
Would be great if this issue could be revisited so plotly LaTeX labels in VS Code work out of the box. Thanks!

This  fix the title but removes the rest of the labels for me.


[NSP] Hat.sh

Hat.sh

Hat.sh - A Free, Fast, Secure and Serverless File Encryption.
License: MIT License
Why do you think this helps users privacy?:  Hat.sh is a opensource Javascript app that provides secure file encryption using the AES-256-GCM algorithm from the WebCryptoAPI included in your browser. it can be very useful for people who want to encrypt a file on the go, and for privacy beginners.
Under what section should this service be listed?: File Encryption
Additional comments / info:  files are not uploaded to a server, everything is done offline in your browser. the app is available for offline use mac/linux/windows.

Added!


get_all analyses

Returns all analyses given a group/project/session/acquisition
It will return all analyses within the hierarchy

example, /sessions/{SessionId}/analyses will return any analyses that have that session as a parent or any of the acquisitions in that session as a parent

Must have permission to the top level container and (only applies when making calls at the group level) will only return analyses for which the user has permission to its parent

Review Checklist

Tests were added to cover all code changes
Documentation was added / updated
Code and tests follow standards in CONTRIBUTING.md

Codecov Report

Merging #1017 into master will increase coverage by 0.07%.
The diff coverage is 95.23%.

@@            Coverage Diff             @@
##           master    #1017      +/-   ##
==========================================
+ Coverage    90.6%   90.68%   +0.07%
==========================================
  Files          50       50
  Lines        6762     6783      +21
==========================================
+ Hits         6127     6151      +24
+ Misses        635      632       -3

Awaiting final review by @nagem

LGTM, this came together really nicely - thanks @hkethi002 !


CNS to DNC communication in Managed DNC

What this PR does / why we need it:

CNS talks to DNC to get node goalstate
CNS checks with NMA if it has goalstate for NC

Special notes for your reviewer:
do not merge yet

Can you add the UTs to test register, sync and NMA api? You will need to inject the dependency / mock those components


License clarification

In the root of the repository is attached the MIT license but in all the projects folders, the LGPL is present and also, the file are marked at the top with the LGPL license header.
Can you clarify under which license the project is?
Thanks.

Sorry, it should be MIT. I'll delete old license.txt files.


Fix build badge

The build badge at the top of the README became unavailable/broken due to https://github.com/badges/shields/issues/8671.
I also changed the link target to a more direct, workflow file-based URL to show the CI workflows for the master branch.

lgtm 🚀 thanks


Change to command logic for compatibility with LuckPerms

When running OTG on SpongeForge alongside LuckPerms, neither commands nor the O menu work as intended, because LuckPerms "disables" the OP system. The game sees everyone as OP, and LuckPerms instead makes everything rely on permissions. As OTG isn't designed to fit in with Sponge permissions, this fails, and the GUI opens for any player on the server.
Sponge handles permissions by taking the command name string in a canCommandSenderUseCommand(int permlevel, String commandName) and making it a permission. So, if we sent "openterraingenerator.openui" as a commandname in the code that opens the ui, then vanilla Forge still only looks at the permLevel int, but Sponge now checks to see the user has the permission given as CommandName. Right now the commandName string is just the name of the command (like "map" or "biomes"), whereas with this change it'd be "openterraingenerator.map" or something along those lines. Again, as far as I'm aware this should not change Forge behaviour at all, and should make it compatible with LuckPerms.
More info here, from Sponge
Normally I would change this myself and submit a pull request, but I'm without a PC for a few days so I'm submitting the idea as a ticket instead for now.
TL;DR: Change how the names of commands are passed to fit the Sponge standard

This is unlikely to be implemented anywhere in the near future, because:

OTG does not officially support Sponge
This seems like a Luckperms problem

I know OTG doesn't officially support Sponge, but that doesn't mean we can't fix a simple compatibility error for a popular plugin. And I thought it was a LuckPerms problem too at first, but they follow Sponge standard just fine. If this is to be fixed, it has to be done from OTG's side. And like I said, I'm very much willing to implement this myself and submit a pull request. It's not a big change.

because LuckPerms "disables" the OP system

I thought it was a LuckPerms problem

I mean it still sounds like a LuckPerms problem.

The incompatibility comes from OTG not following Sponge permission standards. Considering how easy it is to comply with that standard, I'd say this is an OTG problem, not a LuckPerms one.

This is an issue with OTG not following Forge permission standards not Sponge or LuckPerms.
LuckPerms is simply forcing a low op permission level.
Checking for OP status doesn't necessarily mean that a user has permission as there is an OP permission level specified in the server.properties file hence the presence of permLevel parameter on canCommandSenderUseCommand

#441 should fix this


Add view to action blocks

Sometimes you need a reference to the actual UIAlertView, for example to get the values out of UITextFields. This change adds the UIAlertView or UIActionSheet to the action block.
It breaks backward compatibility, but I didn't want to introduce a second action property, which I thought could be buggy. Updating calling code is straightforward--you just add an id view parameter to the action block.

I see, using addButton you can just capture the alert view in the block. OK, closing this PR.


Convert preferences view to WPF

Also tabs should be used in preferences view see issue #1374
Add advanced Reminder preferences similar to osx

Now merged into master.


fix cifar-10 download random issue

Signed-off-by: chensuyue<EMAIL_ADDRESS>Type of Change
UT
Description
Workaround for dataset download issue

cifar-10 download random issue has been fixed, the failed one is another pt distribute case random issue.


BaseModel.model_validate does not behave as former BaseModel.parse_obj

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
I'm currently migrating our code base over to Pydantic V2 and noticed a behaviour of model_validate(), which I can't quite understand. We use parse_obj() quite heavily to convert data between different Pydantic objects (even inherited ones).
In V1 parse_obj() would convert an given pydantic object to the model of my choice.
from pydantic.v1 import BaseModel

class MyModel(BaseModel):
    a: int

class MyExtendedModel(MyModel):
    b: float

extended_model = MyExtendedModel(a=1, b="foo", c=3.14)
model = MyModel.parse_obj(extended_model)

print("model is of type: " + type(model).__name__)

# Expectations
assert not hasattr(model, "b"), f"model is of type {model.__class__.__name__} instead of MyModel"

With the V2 equivalent (model_validate()) these expectations are no longer met (see Example Code).
I can still achieve the original behavior by dumping the model and then validating again:
model = MyModel.model_validate(extended_model.model_dump())

I know, that the old parse_obj method is still available until V3, but I wonder if we are misusing the parse_obj() method, or whether I overlooked something in the Migration guide?
I'm happy to hear from you.
Please keep up the good work on Pydantic! It's a major backbone for the solution we all love to work with.
Example Code
from pydantic import BaseModel

class MyModel(BaseModel):
    a: int

class MyExtendedModel(MyModel):
    b: float

extended_model = MyExtendedModel(a=1, b=3.14)
model = MyModel.model_validate(extended_model)

print("model is of type: " + type(model).__name__)

assert not hasattr(model, "b"), f"model is of type {model.__class__.__name__} instead of MyModel"

Python, Pydantic & OS Version
pydantic version: 2.1.1
        pydantic-core version: 2.4.0
          pydantic-core build: profile=release pgo=false mimalloc=true
                 install path: /Users/.../lib/py_qmulus_core/.venv/lib/python3.11/site-packages/pydantic
               python version: 3.11.3 (main, Apr  7 2023, 20:13:31) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]
                     platform: macOS-13.4.1-arm64-arm-64bit
     optional deps. installed: ['typing-extensions']

Selected Assignee: @dmontagu

Another thing I just realized, is that the objects returned by the model_validate is the exact same object:
# This is True
assert model is extended_model

Whereas parse_obj returned a new instance of the object.
The fix
model = MyModel.model_validate(extended_model.model_dump())

Also works, here, but then I would suggest updating the migration guide/docs to hint at that behavior.

I agree, I expected MyModel.model_validate(my_dict) to work similarly to MyModel(**my_dict), as described in the documentation:
https://docs.pydantic.dev/dev-v2/usage/models/#extra-fields

I think the reasoning for this change is roughly as follows:

When you have a model whose fields are themselves models, it is much faster to just do an isinstance check than to revalidate the nested data. Therefore, there are cases where it is significantly beneficial to not revalidate model data — generally, once a model's data has been validated, it is safe to assume that it will always be valid, and there can be a significant performance benefit to doing so, e.g. if passing a nested attribute value around between different parent model types.
For consistency with that behavior for fields, we use that same logic even when doing a model_validate at the top level. This does have significant performance benefits if the behavior is what you want, but I can understand how you might be more likely to take advantage of the old behavior when calling methods at the top level.

However, I'll note that there is the model_config = {'revalidate_instances': 'always'} option, which at least makes your code above behave the way I think you expected it to:
from pydantic import BaseModel

class MyModel(BaseModel):
    model_config = {'revalidate_instances': 'always'}
    a: int

class MyExtendedModel(MyModel):
    b: float

extended_model = MyExtendedModel(a=1, b=3.14)
model = MyModel.model_validate(extended_model)

print("model is of type: " + type(model).__name__)
#> model is of type: MyModel

assert not hasattr(model, "b"), f"model is of type {model.__class__.__name__} instead of MyModel"

If you feel the documentation is not sufficient on this point, I think we can improve it, and would be interested in any suggestions.

Thanks for the explanation. It's a very reasonable and desirable approach.
I did not know about the config value. Thank your for the hint!
Regarding the docs: I'll think about how this can be made more explicit.

Just hit that behaviour, IMHO it is a bug and not very consistent.
I use "ParentModel.model_validate(child_model_instance)" to convert a model to a base class model. As explained, this doesn't work and returns the same child_model_instance.
Can you recommend what's the proper way to achieve that, I think it's a rather common use case.
What bothers me in that scenario is that even if I set extra="forbid" on the ParentModel, this validation passes.
I can understand that the validation logically passes, as the child_model_instance is already of type ParentModel, so there's nothing to validate. But I guess if that path is taken, then the docs should discourage inheriting models in the first place.

Just hit this same issue today.  I use various inherited Pydantic models to define return schemas from a fastAPI app.
I expected:
UserRead.model_validate(AdminUserRead, from_attributes=True)

to return a type UserRead, but it doesn't, it returns AdminUserRead since it's a sublcass of UserRead.
Adding model_config = {"revalidate_instances": "always"} to the UserRead model does solve the problem, but that seems very obscure to me and isn't clear from the code what it's changing.
UserRead.model_validate(AdminUserRead.model_dump())

works just fine, and is what I'm using for now.  But this could definitely be improved in the docs, or maybe there needs to be a new method like UserRead.from_model(AdminUserRead) or something that conforms one already-validated model to another.
I think this is a pretty common use case.

That is weird. The solution https://github.com/pydantic/pydantic/issues/6944#issuecomment-1677999411 provided works as expected with FastAPI.
The documentation of model_validate has also been updated. From my point of view the issue is solved.
Example
pydantic >= 2.5.3
fastapi >= 0.108.0

from fastapi import FastAPI
from pydantic import BaseModel

class MyModel(BaseModel):
    model_config = {"revalidate_instances": "always"}
    a: int

class MyExtendedModel(MyModel):
    b: float

extended_model = MyExtendedModel(a=1, b=3.14)
model = MyModel.model_validate(extended_model)

print("model is of type: " + type(model).__name__)

assert not hasattr(
    model, "b"
), f"model is of type {model.__class__.__name__} instead of MyModel"

app = FastAPI(
    docs_url="/docs",
)

@app.get("/", response_model=MyModel)
def root() -> MyModel:
    return extended_model

@app.get("/extended", response_model=MyExtendedModel)
def root() -> MyExtendedModel:
    return extended_model

if __name__ == "__main__":
    import uvicorn

    uvicorn.run(app)

As you can see from the output below both endpoints return the expected content.

@slingshotvfx Maybe your project is making use of the responses keyword argument? There is a seems to be a bug in FastAPI. I did not have time to search for issues/report an issue with FastAPI. It for example fails to serialize the @computed_field decorator.
Example:
app = FastAPI(
    docs_url="/docs",
)

- @app.get("/", response_model=MyModel)
- def root() -> MyModel:
+ @app.get("/",  responses={200: {"model": MyModel}})
+ def root():
    return extended_model

As you can see below the response indeed contains the field b which is not desired.

After playing around a bit more my issue seems to stem from multi-level sub-classes:
from pydantic import BaseModel

class Model(BaseModel):
    a: bool = True

class SubclassedModel(Model):
    b: bool | None = None

class SubSubclassedModel(SubclassedModel):
    c: bool | None = None

sub = SubclassedModel()
base = Model.model_validate(SubclassedModel, from_attributes=True)

assert type(base) == Model  # passes
assert not hasattr(base, "b")  # passes

subsub = SubSubclassedModel()
sub = SubclassedModel.model_validate(subsub, from_attributes=True)

assert type(sub) == SubclassedModel, f"model is {type(sub)}" # fails
assert not hasattr(sub, "c") # fails

assert type(sub) == SubclassedModel, f"model is {type(sub)}"  # fails
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: model is <class '__main__.SubSubclassedModel'>

That Model.model_validate() can return a model that is not Model (in some instances) is unexpected to me and not mentioned in the docs.
Setting revalidate_instances='always' on SubclassedModel does pass the final two assertions. But again the documentation about revalidate_instances='always' isn't clear that it changes the resulting model type in those situations.


Integrate uW Catalog

The catalog at https://api.unfoldingword.org/uw/txt/2/catalog.json gives a complete list of the Scripture (and other resources) that we have published.  We need this app to be able to read from the catalog and present the biblical resources available in it.
UPDATE
The publish process for the api needs to be updated so that html is generated for use in uw-web. USFM-tools is able to convert the usfm files to html. The code for the api update can be found in tools.
The generated html should be placed in a new location in the api that is accessible to and structured for uw-web.

Fixed this issue.


Updated mongochef version to 3.0.7

Thank you for the contribution. It was merged directly as abf33ab560aa25fb85d998ef155d7bb3bf790190 to keep commit history cleaner. Your contribution is still credited to you.


User Notification Settings

Hi where can I upload my file?

[](url
setting.zip
)

Working with #56 Pull Request and Merge


Incomplete answer

The answer to the question "Is Azure Front Door Service a dedicated deployment for my application or is it shared across customers?" is incomplete.  The sentence at "However, by creating a Front Door you define the specific configuration required for your application and" is missing content after the "and".

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

ID: f7be424c-f540-a753-6ecc-6ff3913368d5
Version Independent ID: dc80b25f-374d-8775-16e7-ae7d57b4b94f
Content: Azure Front Door Service - Frequently Asked Questions for Front Door
Content Source: articles/frontdoor/front-door-faq.md
Service: frontdoor
GitHub Login: @sharad4u
Microsoft Alias: sharadag

@mcollier Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

#reassign: amitsriva

@mcollier: Thanks for providing feedback that helps improve our documentation. This issue has been resolved and we are closing the issue. #please-close


New OS Support

Currently MIVisionX is supported only for CentOS 7.8 and below. Latest CentOS 8.0 release support required.

Currently MIVisionX is supported only for Ubuntu 18.04 and below. Latest Ubuntu 20.04 LTS release support required.

Dockers Created

Ubuntu 20.04 - https://hub.docker.com/r/kiritigowda/ubuntu-20.04

CentOS 8.1 - https://hub.docker.com/r/kiritigowda/centos-8.1

Ubuntu 20.04 - Supported with warnings with TOT MIVisionX

MIVisionX is now supported on Ubuntu 20.04 & CentOS 8 with a few compilation warnings. See issue #292

@paveltc & @LakshmiKumar23 can you verify this issue with above-provided dockers.

New OS supported with a few warnings, Bugs are open for the warnings.


Create/Update TRGs to enforce usage of Dash related workflows

Everyone besides sigi thinks its soooooo easy to run dash on any project and don't want to have a trg...


Electrum auto wallet "import" and TrustedCoin

Hello,
@fyookball asked me to post this here.
We were discussing an Electron Cash issue on this reddit post.
I'll repost it here. Here are the details:
TrustedCoin integration and wallet "auto import" feature should be removed from Electron Cash because people are losing their funds because of a non-working integration that should've been removed before Electron Cash release. Take a look at the following example:
Bob had his Electrum wallet with TrustedCoin 2FA enabled.
He's been using it for a while but lost his seed.
Since he still has access to the 2FA code, he can still use this Bitcoin wallet.
Then Bitcoin Cash comes out. Bob installs Electron Cash.
The installation pretty much "imports" the wallet Bob had in Electrum without notice.
Bob start receiving BCH coins that adds up to his existing balance. He might even sweep a wallet or two.
Now Bob tries to send his BCH coins. He clicks send but oh surprise: he can't spend the funds because TrustedCoin is not compatible. Even though he would've lost access to the existing balance from the start, he also lost access to the extra funds received or swept. The latter could've been avoided if Electron Cash developers would had removed TrustedCoin from the start or if Electron Cash wouldn't had imported an existing Electrum wallet by default.
Sadly, even if Bob still has access to the 2FA code from the previous Electrum wallet, he can't spend the coins because of the signature error. I suggest you to remove TrustedCoin to avoid future issues like this.
While I know that TrustedCoins have a lot of the fault here, sadly there's also some fault from Electron Cash devs because: they should've removed the "wallet import feature" to avoid Electron Cash using an old 2FA enabled wallet from Electrum.
Please consider this for your next release.
tl;dr   Remove the "wallet import feature" to avoid Electron Cash from using old Electrum wallets and disable TrustedCoin wallet integration.

I agree we need to avoid this situation.
Actually I even think we might disable copying of ~/.electrum/wallet/* altogether, as you suggest. Having to copy wallets or restore from seed gives more transparency to the user as to what is happening. He has more control.

2fa will be disabled next release


"OCIS_TRACING_TYPE":"otlp" - value "otlp is not described in envvar descriptions

Reading that we can now use otlp as value for OCIS_TRACING_TYPE, I checked if that value is described in the envvar descriptions in master - which is not.
This value needs to be added for each service where OCIS_TRACING_TYPE is used.
For a list of services using that envvar see: Global Environment Variables.
@butonic

dont close


Fix exceptions that occured because of reading the EEPROM data at inappropriate time

The error showd when running the program with unflashed OAK-D-LITE because I was trying to read the EEPROM when none were flashed yet.
To fix the issue I added orDefault to readCalibration and readFactoryCalibration calls. This made the error disapear but logged an empty EEPROM. So I moved the EEPROM reading code further down the code path where in all cases the device is flashed.
Also I removed the duplicated function flash_eeprom with prior discussion with @alex-luxonis (see 11acce4d30ecad83958cc8d899c50a2f7b5e34fc), because at first I thought it could be a good place to store the EEPROM data for logging. But later I decided to move the logging even further down the code path. If the removal poses to much risk, we can revet the commit.

Tested with:

OAK-D-LITE
OAK-D-PRO-W

both had their EEPROM erased before the test.


Support LDAP-based CRLs

Handles https://github.com/Jasig/cas/issues/926
This pull adds support for fetching CRLs from an LDAP directory server.
The general contract is defined by CRLFetcher with the below options. The resource to fetch is specified as an object parameter to allow a flexible configuration for implementations, specially for LDAP.
ResourceCRLFetcher
Defines how a CRL should be retrieved from a resource. Resources may include URLs, URIs, Strings and Spring Resources.
LdapResourceCRLFetcher
Fetches a CRL from LDAP. Includes configuration to define security settings, credentials, and objects to look up.
All checkers are configured to use a provided fetcher and integration tests are provided based on the LDAP module.

Adding @serac to the pull to review.

@leleuj you may want to review this

A few comments here and there...

I have updated the pull request to reuse the same const for certificate revocation attribute.

+1

I had a chance to work on this pull again, and was able to add the ldaptive configuration. The configuration allows one to inject the search filter, request and the connection config to the fetcher. Also note that try-with-resource syntax is still not available in this pull for ldap connections because we are on v1.0.6.

@serac please review this again when you get a chance. I was able to add ldaptive to the config.
Would be great if we could resolve this PR by the weekend.

Sorry, dropped the ball on this. Reviewing now. Looks good generally -- appreciate your patience and persistence.

I'll say, this was under CAS 4 and ldaptive added way too much noise to be tenable

Can you elaborate on that? I assume you mean that your deployers did not want to be burdened with the necessary configuration.

@mmoayyed
Here is some sample code to illustrate how this could be done in ldaptive 1.0.x
    ConnectionConfig cc = new ConnectionConfig("%s");
    DefaultConnectionFactory cf = new DefaultConnectionFactory(cc);
    cf.getProvider().getProviderConfig().setConnectionStrategy(
      new ConnectionStrategy() {
      @Override
      public String[] getLdapUrls(ConnectionFactoryMetadata metadata)
      {
        // provide the LDAP URL from the cert here
        return new String[] {String.format(metadata.getLdapUrl(), "ldap://url.from.the.cert")};
      }
    });

    ConnectionPool cp = new BlockingConnectionPool(cf);
    PooledConnectionFactory pcf = new PooledConnectionFactory(cp);
    cp.initialize();

    AggregatePooledSearchExecutor ex = new AggregatePooledSearchExecutor();
    ex.setBaseDn("ou=People,dc=vt,dc=edu");
    Collection<Response<SearchResult>> results = ex.search(
      new PooledConnectionFactory[] { pcf, },
      new SearchFilter[] { new SearchFilter("(uupid=dfisher)"), },
      new String[] {"displayName"});

    cp.close();
    ex.shutdown();

The ConnectionStrategy interface was intended to facilitate URL selection for availability in non-load balanced environments.
I believe it can be used for this problem, just provide an implementation that knows about the URLs found in the cert.
Also, by using the AggregatePooledSearchExecutor you can easily execute the same search over all the connection pools.

@dfish3r Thank you. I'll get to work.

I have modified this PR to take note of @dfish3r's suggestions. The fetcher now is plugged in with the connection factory and a search executor. The pool is initialized as such:
final AbstractConnectionPool pool = getConnectionPool();
        final ProviderConfig providerConfig = pool.getConnectionFactory().getProvider().getProviderConfig();
        try {
            providerConfig.checkImmutable();
            providerConfig.setConnectionStrategy(
                new ConnectionStrategy() {
                    @Override
                    public String[] getLdapUrls(final ConnectionFactoryMetadata metadata) {
                        return new String[]{String.format(metadata.getLdapUrl(), ldapURL)};
                    }
                }
            );
            logger.debug("Connection strategy to [{}] configured. Initializing pool...", ldapURL);
            pool.initialize();
            logger.debug("Initialized connection pool for [{}]", ldapURL);
        } catch (final IllegalStateException e){
            logger.trace("Connection pool failed to initialize, or is already initialized", e);
        }

and search
final Collection<Response<SearchResult>> results =
            this.searchExecutor.search(
                            new PooledConnectionFactory[] {this.connectionFactory},
                            new SearchFilter[] {this.searchExecutor.getSearchFilter()},
                            this.searchExecutor.getBinaryAttributes());

            for (final Response<SearchResult> result : results) {
                if (result.getResultCode() == ResultCode.SUCCESS) {
                    ...
                } else {
                    logger.debug("Failed to execute the search [{}]", result);
                }
            }

Test cases attempt exhaust LDAP fetching with and without caching.
Pool and search are shut down upon @PreDestroy

@serac @dfish3r let me know what you think of the most recent changes.
Coincidentally, this pull is holding up #497 so it'd be most helpful if we could figure out how to proceed.

I don't think you're latest commit is going to work.
You can't initialize a pool more than once.
I spoke with @serac about the impetus for this change and he said it had more to do with removing old JNDI code than anything else.
With that in mind I created a patch that provides two implementations using ldaptive, one that provides pooling and one that does not:
https://github.com/dfish3r/cas-x509-crl-ldaptive/commit/52aef4dc0fd7b781f730d3144df43743ac568a40
IMO you should remove the implementation that uses JNDI and renaming the Ldaptive classes.

@dfish3r thanks! I had a look at the patch and I think it's perfectly fine. Would you prefer if I updated this PR with the patch you have?

I went ahead and applied your patch to this pull. Verified all test cases.
Thanks again.

+1 to merge

I have updated the x509 docs with details on how to configure ldap fetching.


fix: path normalization should not case-normalize entire path

This follows Rust's approach: https://doc.rust-lang.org/std/path/index.html#case-sensitivity

But now I do worry that the structure References which has some HashMaps on these paths that it looks up using references.findAt module
I'm not sure if this function will still do the right thing, and the comment is no longer valid:
@[export lean_module_name_of_file]
def moduleNameOfFileName (fname : FilePath) (rootDir : Option FilePath) : IO Name := do
  let fname ← IO.FS.realPath fname
  let rootDir ← match rootDir with
    | some rootDir => pure rootDir
    | none         => IO.currentDir
  let mut rootDir ← realPathNormalized rootDir
  if !rootDir.toString.endsWith System.FilePath.pathSeparator.toString then
    rootDir := ⟨rootDir.toString ++ System.FilePath.pathSeparator.toString⟩
  if !rootDir.toString.isPrefixOf fname.normalize.toString then
    throw $ IO.userError s!"input file '{fname}' must be contained in root directory ({rootDir})"
  -- NOTE: use `fname` instead of `fname.normalize` to preserve casing on all platforms
  let fnameSuffix := fname.toString.drop rootDir.toString.length
  let modNameStr := FilePath.mk fnameSuffix |>.withExtension ""
  let modName    := modNameStr.components.foldl Name.mkStr Name.anonymous
  pure modName

And I believe we will need some "case insensitive" file name comparisons in LSP as a result of this change, for example the if !rootDir.toString.isPrefixOf fname.normalize.toString then comparison here will need to be case insensitive comparison on Windows.  We should encapsulate all that logic in FilePath and only use FilePath methods to compare file names, and subsets of the path thereof.

That problem already existed before this PR, and in fact should improve from trying harder to use only one case. I dearly hope that with this, the language server does not really have to worry about casing in practice.

I am testing it and so far it looks fine, I renamed a subfolder to lower case, but referenced it in the lakefile.lean using upper case, and goto-definition still works across these files.  The VS code file tab shows upper case "D:\Temp\lean_examples\Foo\Foo\Hello.lean" when the real file on disk is "D:\Temp\lean_examples\Foo\foo\hello.lean".  So somehow this still works through the LSP and VS code doesn't care about the case differences.  So I think part is good for now.

@lovettchris What shall we do about the broken Uri test? Should pathToUri do the reverse transformation?

I guess I'm curious why drive letter case folding is even needed?  it was only needed in rollup because they had nodejs regular expressions looking for uppercase char colon regex patterns which we don't have.
So I'm now testing to see what happens when I comment out lines 45,46 of normalize. What bug was that fixing?

So I'm now testing to see what happens when I comment out lines 45,46 of normalize. What bug was that fixing?

I couldn't find the reason, it was added very early: https://github.com/rust-lang/rust/commit/4a9dd3f8403f84ad518b0119fa2312c379df5dd7#diff-2fddd6220cb2006f2e025a3a682366ec85198a4c192e4e8c479bf72be6f2aa5aR240. I'm okay with just reverting it for now and seeing what, if anything, breaks.

I'm okay with just reverting it for now and seeing what, if anything, breaks.

Please don't revert. Let's go with the Rust approach.
This PR is high priority. Could you please disable the broken test for now and merge?

Since a conversion roundtrip failure presumably is the source of the bug, I decided to be explicit and do the reverse conversions in the URI functions. With that the drive letter normalization in FilePath.normalize is not strictly necessary anymore to fix the bug, but I'm leaving it in there to follow Rust's behavior.


Update http.rb

Signed-off-by: infa-btan<EMAIL_ADDRESS>
Description
Fetch refers to chef::http.streaming_request on line 182. This function sets the tempfile to
stream_to_tempfile(url, http_response, tempfile, &progress_block) # lib/chef/http.rb: line 226
On line 546 of lib/chef/http.rb stream_to_tempfile closes the file. This means that the fetch function attempts to close the file a second time, causing an IOError
Related Issue

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

Hi, could you provide some example code that demonstrates the failure?

Yeah that's in the common path of just about every remote_file http request and has been stable for years, so its more than a bit weird that nobody else has reported this.  A repro case would be useful to validate that its a real bug.

This is the case where I saw the bug
remote_file("apache 8.5.58 tarball") do
action [:create_if_missing]
default_guard_interpreter :default
source ["http://archive.apache.org/dist/tomcat/tomcat-8/v8.5.58/bin/apache-tomcat-8.5.58.tar.gz"]
declared_type :remote_file
cookbook_name "test"
path "/root/.chef/local-mode-cache/cache/apache-tomcat-8.5.58.tar.gz"
remote_domain nil
emote_user nil
owner nil
group nil
mode nil
headers {}
verifications []
end
 System Info:
   ------------
   chef_version=15.4.45
  platform=centos
 platform_version=7.8.2003
 ruby=ruby 2.6.5p114 (2019-10-01 revision 67812) [x86_64-linux]
  program_name=/usr/bin/chef-client
  executable=/opt/chef/bin/chef-client

Error appears as
================================================================================
Error executing action create_if_missing on resource 'remote_file[apache 8.5.58 tarball]'
================================================================================
IOError
closed stream
What's odd is that this issue only occurs intermittently when i run this, even though this kind of thing should be consistent

What is the stacktrace.out on that?

Also closing a tempfile twice isn't an error:
[1] pry(main)> file = Tempfile.new('foo')
=> #<File:/var/folders/vl/7__gnv9x5sg745ntrsh0yglm0000gn/T/foo20201105-28404-1fkx1cs>
[2] pry(main)> file.close
=> nil
[3] pry(main)> file.close
=> nil

This looks more like the http connection was aborted early.

My stacktrace doesn't go further than that and I can't repro the error. Must have been a transitive network issue then


Add yamllint in checks

Coverage remained the same at 85.35% when pulling 77c4db57745ce212e2fe1df8392bb3faff61b1db on yamllint into 40814f71f49d1f4e12348781a7d27b75a50a03b9 on master.


matplotlib not included in nri-cat conda env

Obviously this isn't needed for intake to work, but is needed to work through the provided example in example_usage.ipynb

Closing as we're now recommending users use the xp65 environment, which includes matplotlib. Those wanting to manage their own envs will need to make sure matplotlib is installed to run the tutorials.


Readme file update

#111 Summary of Key Changes:

Images Added: Visual content showing MetaMask login, vehicle search and booking, and real-time updates.

More Detailed Web3 Integration: A comprehensive description of blockchain transactions and interactions with smart contracts.

Refinement in Features: Expanded descriptions on decentralized login, smart contracts, and blockchain-powered features for better user understanding.

Contributing Section Enrichment: Step-by-step instructions added with practical examples to make contributing easier.

Markdown Formatting: Enhanced readability and aesthetic appeal with appropriate use of formatting and icons.

nice


convert to futures 0.2 (alpha)

I'm experimenting with introducing futures 0.2 into the 0.12.x branch of hyper, where we're using futures-timer. In order to do that, I needed it to work with futures 0.2 :)
You probably don't want to merge this just yet, but maybe you'll find the time to review the changes so that this is ready to merge once futures 0.2 proper is released!

Thanks! I'm fine holding off on publishing until futures 0.2 is ready


Fix Service Connect TLS IAM role dependency chain and orphaned services

what

AWS does not provide documentation about which principal should be used for the Service Connect TLS role.  I've updated the trust relationship with the correct principal.
Because IAM is eventually consistent, Terraform creates the ECS service too quickly after creating the Service Connect TLS role (verified by AWS Support).
A hard dependency between the ECS Service and the Service Connect TLS IAM policy attachment, otherwise the ECS service will fail to delete (verified by AWS Support).

why

Fixes bugs that cause ECS service creation failures and orphaned Service Connect services.

references

Internal AWS support tickets.

Thanks @awkaplan we'll take a look.
Would you be open to adding more values to the example fixture to test the Service Connect TLS feature?


Fixes numberOnly behaviour on mobile devices

Fixes
Mobile device behaviour when numberOnly prop is true.
Now:

The numpad will show instead of a full keyboard.
When the user presses a number, the cursor moves to the next OtpItem input just like it does on desktop.

can this be merged ?


[FEAT] Recommended config

Fixes #144
Requires #259, #260.

added a util to make it standardised and easier to add default config for a rule
configured recommended based on #144

purposely switched recommended prop to be "error" | "warning" | false

inside the eslint repo, it should be true. otherwise it's just a property that isn't used officially by eslint. It's truthy so eslint-docs still work.

changed recommended generator to accept "error"/"warning" for more configurability.

adjusted default config of certain rules that didn't match our recommendations.

@Aladdin-ADD - I have updated the readme

@bradzacher looks like there are errors in tests for no-type-alias

@armano2 yup - it needed the config changes from #259

It does not work for me. ESLint complains that "warning" should be "warn" instead. Bonus question: how to make it work with prettier?

it was fixed in master, but not released yet.
@bradzacher plan to release a patch version?

I'm back from holidays so I'll look to cleaning things up and releasing rc.1


Building with clang 9 changes ABI

Version 0.7.26 build number 0 has
__ZNSt3__16vectorIN5GiNaC2exENS_9allocatorIS2_EEE10deallocateEv

whereas build number 1 has
__ZNSt3__16vectorIN5GiNaC2exENS_9allocatorIS2_EEE13__vdeallocateEv

cc @saraedum
xref: https://groups.google.com/d/msgid/sage-support/4bc2fcb7-7398-402e-be7f-e948f6efb31f%40googlegroups.com

I've removed the build number 1, py27, py36, py37 builds for now.


qTip2 tooltips position incorrectly when combined with imagesloaded.pkgd.js

When I include the imagesloaded script (I don't even use it), the tooltips for some reason get their positioning thrown off.  I am using the tooltips on image maps.
Any ideas on what could be the problem?

Only when using an effect and JQuery's slideUp/Down animations?


Badges in Collapsibles Aligns Left

Expected Behavior
Labels in Badges in Collapsibles should align right.
Current Behavior
Labels in Badges in Collapsibles aligns left.
Possible Solution
Apply same css on docs site.
 It all works fine!