Translate RBI attached_class to RBS instance

The attached_class in RBI:
sig { returns(T.attached_class) }
def foo; end

should be translated to instance in RBS:
def foo: -> instance

Are you sure that that is the right translation? The example in the RBS docs doesn't make it sound like it is so:
class Foo
  attr_reader parent: class

  def foo: () -> instance

  @@foos: Array[instances]

  include Enumerable[class]
end

Note that the foo method is an instance method, so the concept AttachedClass does not even make sense.
Does instance mean a more general "attached object" then?

I'm not sure about the foo method in this case, the class attribute would make sense (it seems there is a typo in the example though):
class Foo
  attr_reader parent: class
  @@foos: Array[instances]
end

The document says:

instance denotes the type of instance of the class. class is the singleton of the class.

Looking at how it is used in the RBS files for core it seems to mean T.attached_class?
The usage in foo is indeed surprising. I though that depending on the context it could mean T.self_type but there is already self in RBS. I'll inquire.

I though that depending on the context it could mean T.self_type but there is already self in RBS. I'll inquire.

So it's indeed dependent on the context.

On a class method it does return the type of the instance, so the same as T.attached_class.
On an instance method it behaves as self, so it should be translated as T.self_type.

I think this is useful for RBS methods declared with the syntax def self?.foo: instance where the return type will be instance in the case of the singleton and self in the case of the instance.
Since we come from RBI, if we find a T.attached_class we can presume that it was used in the context of a singleton method (or Sorbet would have errored) so I think it's safe to translate it to instance.


Fix minor typos in Configurations/README

Checklist

[x] documentation is added or updated

PR good to go ?

Merged to master.  Thanks!


[multistage] Initial planner changes to support the v1 aggregation functions in multistage

This work is a collaboration between @vvivekiyer and myself
Important Note: This PR must be rebased on top of @vvivekiyer 's runtime changes (PR: https://github.com/apache/pinot/pull/10845) and integrated before it can be merged
This PR should also be reviewed with @vvivekiyer 's https://github.com/apache/pinot/pull/10845
Motivation:

Today only a subset of aggregation functions (sum, min, max, count, avg, count(distinct), bool_or, bool_and, kurtosis, and skewness) work in the multistage engine.
To support each new aggregation function changes are required on both the planner and runtime side. Pinot contributors should not need to understand the nitty gritty details of the planner and execution engine to add new aggregation functions.
The multistage AggregateOperator essentially reimplements the aggregation logic rather than reusing the aggregation logic present in the v1 engine functions. This results in code duplication and a high maintenance overhead (e.g. bug fixes must be made in two places which may even have different logic or different mechanisms to support things like null handling).
The v1 execution engine supports a large number of aggregation functions and it is not practical to manually add support for each new aggregation function to multistage. This also results in a situation in which each new aggregation function needs to be added to both the v1 engine and the multistage engine, or if support is only added for the v1 engine, the multistage engine must constantly play catch up.

To get around the above limitations we are refactoring the aggregation function framework to work for both the v1 and multistage engine. This PR tackles the changes required on the planner side to make the planner more generic. This is phase 1 of the planned set of changed to support the aggregation functions and tackles only the existing set of aggregation functions already supported in the multistage engine.

OSS Issue: https://github.com/apache/pinot/issues/10745
Design document: https://docs.google.com/document/d/1Us6aBvTpNLMEy0ODo34OgTk73h_LVFFAH6q17689h1M/edit?usp=sharing
@vvivekiyer 's runtime changes to support this: https://github.com/apache/pinot/pull/10845

Types of functions this PR does not support (and for which support will be added later on):

Add support for taking function arguments as literals (e.g. PERCENTILE)
Add support for aggregation functions which take more than one column as arguments (e.g . COVARIANCE)
Arg min/max type aggregation functions using the Parent-Child relationship

cc @siddharthteotia @vvivekiyer @walterddr @kishoreg

Codecov Report

Merging #10846 (ba818df) into master (da69448) will decrease coverage by 54.82%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #10846       +/-   ##
=============================================
- Coverage     68.42%   13.61%   -54.82%
+ Complexity     6573      439     -6134
=============================================
  Files          2170     2116       -54
  Lines        116688   114263     -2425
  Branches      17661    17364      -297
=============================================
- Hits          79846    15559    -64287
- Misses        31196    97431    +66235
+ Partials       5646     1273     -4373

Flag
Coverage Δ

integration1
?

unittests1
?

unittests2
13.61% <0.00%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

...gregation/function/AggregationFunctionFactory.java
0.00% <0.00%> (-81.36%)
:arrow_down:

...ation/function/BaseBooleanAggregationFunction.java
0.00% <ø> (-60.68%)
:arrow_down:

...n/function/BaseSingleInputAggregationFunction.java
0.00% <0.00%> (-75.00%)
:arrow_down:

...gation/function/BooleanAndAggregationFunction.java
0.00% <0.00%> (-60.00%)
:arrow_down:

...egation/function/BooleanOrAggregationFunction.java
0.00% <0.00%> (-60.00%)
:arrow_down:

...aggregation/function/CountAggregationFunction.java
0.00% <0.00%> (-76.55%)
:arrow_down:

...ion/function/DistinctCountAggregationFunction.java
0.00% <0.00%> (-80.00%)
:arrow_down:

...tion/function/FourthMomentAggregationFunction.java
0.00% <0.00%> (-66.08%)
:arrow_down:

...y/aggregation/function/MaxAggregationFunction.java
0.00% <0.00%> (-72.42%)
:arrow_down:

...y/aggregation/function/MinAggregationFunction.java
0.00% <0.00%> (-72.42%)
:arrow_down:

... and 3 more

... and 1664 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Just a note: the test failures are currently expected as the changes need to be integrated with the runtime code

Have addressed the suggestions from @walterddr and @Jackie-Jiang
This PR is ready to be reviewed @walterddr @Jackie-Jiang @kishoreg @xiangfu0 but must only be merged after rebasing and incorporating changes from https://github.com/apache/pinot/pull/10845


fix support for ie10

doScroll in ie10 needs to be called as a function for hack to work properly.
Fix for issue #53

This fix seems to scroll the page down as soon as it loads due to call to doc.documentElement.doScroll(). I'm not sure how this fixes anything. I've logged the bug on gatsby as well: https://github.com/gatsbyjs/gatsby/issues/20582

@andrejpavlovic this fix is because ie10 doesn't support window.onload. we need to create this hack to poll for document to be ready. When it does not return as an error document is ready.
What we could do is change it from doScroll() to doScroll('left') to prevent scrolling down.

I've seen doScroll('left') being used, so that should work in most cases. I'm not clear where the polling happens, since I don't see a loop in the code.


feat: md support prerequisite

Fixes #3427
Describe the changes you have made in this PR -
prerequisite of the PR
Screenshots of the changes (If any) -
CURRENT VIEW..

The forum page is not formatted as we would have to formatted_contenthelper method here to make all the threads formatted as it is missing by default,Screenshot ref. previous comment.
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

You don't need to gem to support markdown. you can simply override the helper method from the app.

You don't need to gem to support markdown. you can simply override the helper method from the app.

You mean that I dont need to gem in simple_discussion, and just override the method in helper, and install the gem here?

Yes
On Wed, 28 Dec, 2022, 7:38 pm Vaibhav Upreti, @.***>
wrote:

You don't need to gem to support markdown. you can simply override the
helper method from the app.
You mean that I dont need to gem in simple_discussion, and just override
the method in helper, and install the gem here?
—
Reply to this email directly, view it on GitHub
https://github.com/CircuitVerse/CircuitVerse/pull/3440#issuecomment-1366679054,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXYAEHBCZ2J56APR5PEQ5DWPRCVXANCNFSM6AAAAAATKKWHEI
.
You are receiving this because you commented.Message ID:
@.***>

Then can you pls reopen PR https://github.com/CircuitVerse/simple_discussion/pull/2So that I can remove the gem … otherwise the code looks good Sent from my iPhoneOn 28-Dec-2022, at 8:55 PM, Aboobacker MK @.***> wrote:
Yes
On Wed, 28 Dec, 2022, 7:38 pm Vaibhav Upreti, @.***>
wrote:

You don't need to gem to support markdown. you can simply override the
helper method from the app.
You mean that I dont need to gem in simple_discussion, and just override
the method in helper, and install the gem here?
—
Reply to this email directly, view it on GitHub
https://github.com/CircuitVerse/CircuitVerse/pull/3440#issuecomment-1366679054,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXYAEHBCZ2J56APR5PEQ5DWPRCVXANCNFSM6AAAAAATKKWHEI
.
You are receiving this because you commented.Message ID:
@.***>

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

We don't have to make any changes on the gem, adding redcarpet gem and
overriding helper should be in the circuitverse, not in the simple
discussion
On Wed, 28 Dec, 2022, 9:03 pm Vaibhav Upreti, @.***>
wrote:

Then can you pls reopen PR
https://github.com/CircuitVerse/simple_discussion/pull/2So that I can
remove the gem … otherwise the code looks good Sent from my iPhoneOn
28-Dec-2022, at 8:55 PM, Aboobacker MK @.***> wrote:
Yes
On Wed, 28 Dec, 2022, 7:38 pm Vaibhav Upreti, @.***>
wrote:

You don't need to gem to support markdown. you can simply override the
helper method from the app.
You mean that I dont need to gem in simple_discussion, and just override
the method in helper, and install the gem here?
—
Reply to this email directly, view it on GitHub
<
https://github.com/CircuitVerse/CircuitVerse/pull/3440#issuecomment-1366679054
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAXYAEHBCZ2J56APR5PEQ5DWPRCVXANCNFSM6AAAAAATKKWHEI
.
You are receiving this because you commented.Message ID:
@.***>

—Reply to this email directly, view it on GitHub, or unsubscribe.You are
receiving this because you authored the thread.Message ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/CircuitVerse/CircuitVerse/pull/3440#issuecomment-1366737820,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXYAEEX3UC5QWNDPLEPWRLWPRMVLANCNFSM6AAAAAATKKWHEI
.
You are receiving this because you commented.Message ID:
@.***>

@tachyons CURRENT VIEW..

The forum page is not formatted as we would have to helper method here to make that formatted.

@tachyons I tried this approach earlier, tested these changes too
but it not overriding the helper in simple_discussion. Unless we place the helper in app/helpers/simple_discussion/forum_posts_helper.rb and rename it as forum_posts_helper.rb
something like this...(Like it was in previous commit )..
module SimpleDiscussion::ForumPostsHelper
Also we would have to add formatted_content helper  here to make all the threads formatted as it is missing by default,Screenshot ref. previous comment.

Had to add rest of the helper methods too, as the helper seems to override the whole module instead of certain methods and was throwing up errors.
Also @tachyons  can you pls reopen PR would have to add formatted_content helper  here to make all the threads formatted as it is missing by default,Screenshot ref. previous comment.

You can override the method by creating an forum_helper.rb in helpers folder and override the method by using alias.
module ForumHelper
  include SimpleDiscussion::ForumPostsHelper

  def new_formatted_content(text)
    options = %I[hard_wrap filter_html autolink tables]
    md = Redcarpet.new(text, *options).to_html
    sanitize(md)
  end

  alias formatted_content new_formatted_content
end

I think @tachyons was telling to do this .
@VaibhavUpreti try this changes.

Yea, you are right. This probably seems the best way to do this.

Works done on this one, just need to add formatted_content helper method here to make all the threads formatted as it is missing by default.

Works almost done on this one, just need to add formatted_content helper method here to make all the threads formatted as it is missing by default.

In the thread , whole post not shown , an truncated part only shown there. So I think we need to check , if we truncate the post body [with markdown enabled] how much that can affect the formatting.

Works almost done on this one, just need to add formatted_content helper method here to make all the threads formatted as it is missing by default.

In the thread list, whole post not shown , an truncated part only shown there. So I think we need to check , if we truncate the post body [with markdown enabled] how much that can affect the formatting.

It formats the content fairly well, but the h1's do tend to be bigger than the post name itself

Works almost done on this one, just need to add formatted_content helper method here to make all the threads formatted as it is missing by default.

In the thread list, whole post not shown , an truncated part only shown there. So I think we need to check , if we truncate the post body [with markdown enabled] how much that can affect the formatting.

 It formats the content fairly well, but the h1's do tend to be bigger than the post name itself

yes that seems may be an problem. Because as the space taken by the post grows, users need to scroll more to find an thread.
@tachyons @vedant-jain03 can suggest the correct outcome.

We can skip that for now. This looks fine. We will have better markdown support with a builtin editor and preview later. something like Github and gitlab does

Lint job is failing because you are using old and vulnerable version of redcarpet

@tachyons this one's good to go 👍 , though need to add formatted_content method here, and then revise the commit in Gemfile.lock for simple_discussion. (to make all the threads formatted which is missing by default)


Allow users to connect to existing contracts - closes #46

See summary.

Please update the readme too.


[v8][test] bdd_tests localhost server connection refused for unknown reason when enter interactive mode

node delegate is up
node bob is up
node mail is up
Error: node alice is down
Error: node alice is down
[delegate] request: wallet_open default
ok
[delegate] request: wallet_unlock 9999999 password
ok
[delegate] request: wallet_delegate_set_block_production ALL true
ok
[alice] request: wallet_open default
ok
[alice] request: wallet_unlock 9999999 password
ok
[bob] request: wallet_open default
ok
[bob] request: wallet_unlock 9999999 password
ok
[alice] request: execute_command_line enable_raw
ok
[bob] request: execute_command_line enable_raw
ok
[mail] request: execute_command_line enable_raw
ok
@pause: use the following urls to access the nodes via browser:
- delegate node: http://localhost:5690
- alice node: http://localhost:5691
- bob node: http://localhost:5692
or press [d],[a], [b], or [m] to have console access
or press any other key to shutdown testnet and continue..

entering node 'delegate' interactive mode, enter 'exit' or 'quit' to exit
/usr/local/Cellar/ruby/2.1.3_1/lib/ruby/2.1.0/net/http.rb:879:in `initialize': Connection refused - connect(2) for "localhost" port 5690 (Errno::ECONNREFUSED)
	from /usr/local/Cellar/ruby/2.1.3_1/lib/ruby/2.1.0/net/http.rb:879:in `open'
	from /usr/local/Cellar/ruby/2.1.3_1/lib/ruby/2.1.0/net/http.rb:879:in `block in connect'
	from /usr/local/Cellar/ruby/2.1.3_1/lib/ruby/2.1.0/timeout.rb:76:in `timeout'
	from /usr/local/Cellar/ruby/2.1.3_1/lib/ruby/2.1.0/net/http.rb:878:in `connect'
	from /usr/local/Cellar/ruby/2.1.3_1/lib/ruby/2.1.0/net/http.rb:863:in `do_start'
	from /usr/local/Cellar/ruby/2.1.3_1/lib/ruby/2.1.0/net/http.rb:852:in `start'
	from /usr/local/Cellar/ruby/2.1.3_1/lib/ruby/2.1.0/net/http.rb:583:in `start'
	from /Users/fisher/superlab/bitshares_play/tests/bdd_tests/bitshares_api.rb:76:in `request'
	from /Users/fisher/superlab/bitshares_play/tests/bdd_tests/client.rb:140:in `interactive_mode'
	from ./testnet.rb:211:in `pause'
	from ./testnet.rb:235:in `<main>'

But it is normal when setup the network manually instead of run
./testnet.rb

no further reproducible now


Fast Navigation on SSG Initial Page Load Causes Nuxt to Rerun Asyncdata and Not Load payload.json

Environment

Operating System: Windows_NT but has happened on multiple systems
Node Version:     v18.19.0
Nuxt Version:     3.9.0
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      build, nitro, runtimeConfig, modules, plugins, gtm, lazyLoad, $meta, dayjs, css, vite
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
Will provide asap - I'm wrapping up a big project and will need to jury rig a solution around this first. Expect it by 1/6/2024.
Describe the bug
This is occurring on statically rendered sites in Nuxt 3. The site is prerendering using Nitro. Some routes are SSR, more are prerendered, a few are CSR. What's happening is, on first page load, if someone attempts to quickly (in less than a second) hit one of navigation links, then the payload.json that should be returned is not returned and instead Nuxt attempts to rerun asyncdata, but fails to do that as well.
Below is an extremely simplified version of the asyncData on the statically rendered page. What's interesting is the console log will log when this bug occurs, but will not log on normal navigation. It seems as though asyncData is rerunning when it shouldn't be for these static pages if the navigation link is clicked too quickly on the first page load. After that, I'm not having issues with further navigation.
async asyncData() {
   try {
      console.log('i am here')
      const route = useRoute()
      const { category } = await $fetch(`/category/${route.params.category}`)
      return { category, error: null }
    } catch (e) {
      return {
        error: e
    }
 }

I do wonder if it relates to this closed issue: #13350. When I upgraded some of the random, unclear error messages disappeared. However, with this new asyncData bug, what seems to happen is it attempts to rerun asyncData but does not wait to try to render the page and considers the "category" 'undefined.' As a string, which is making it difficult for me to even keep the user out of a complete error state. How it got to 'undefined' I have no idea. Feel free to tell me if any of this sounds like an operator error.
Additional context
The navigation links I'm referring to are all prefetched nuxt links.
Logs
No response

Still tagging it with "needs reproduction" - looking forward to it 👍🏻

So I just tried to create a reproduction - even calling for large amounts of data to try to mimic how large the repo I'm working on is. It doesn't work. But I did find this ticket #9448 - I know it was in Nuxt v2 but it is exactly the issue I'm experiencing. It seems to have something to do with a file nuxt is loading onto the browser on the first page hit since sometimes it happens, sometimes it doesn't, and it happens more often for our team members (like me) who are furthest away from the server. It's only a fast navigation on the first page hit.
I don't know if you can help me since I can't reproduce it...

I can see similar behavior even in SSR/ISR and dev server where I can force suspend to break when navigating before the page is fully hydrated. I think it can be made more evident when using a page transition.

Yeah, it still seems to have something to do with the Suspense component and I don't think these minimal reproduction links are going to cut it. It may still be upstream with Vue.

I diagnosed the issue - it's in our code, not Nuxt!

I diagnosed the issue - it's in our code, not Nuxt!

i am having similar issues, and gone through all my code and cant find anything.
Do you mind sharing what your issue was so I can see if it would be similar in my code?


Pyarrow Parquet Pagesize question

I'm trying to figure out how to specify page size in the parquet writer but I'm only seeing the option for chunk_size / row_group_size which I believe is block size.  How do you set page size?  I saw a reference to it being set as a default 1MB in a previous issue, but can't find any other references to change it.

see https://issues.apache.org/jira/browse/ARROW-2057. Let's discuss there


geoserver: 2.25.3 -> 2.26.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for geoserver is: Open source server for sharing geospatial data
meta.homepage for geoserver is: https://geoserver.org/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS

[!CAUTION]
A test defined in passthru.tests did not pass.

found 2.26.0 with grep in /nix/store/80f7viv7jxykhrzl07myndgvb7psrp61-geoserver-2.26.0
found 2.26.0 in filename of file in /nix/store/80f7viv7jxykhrzl07myndgvb7psrp61-geoserver-2.26.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

geoserver

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/80f7viv7jxykhrzl07myndgvb7psrp61-geoserver-2.26.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A geoserver https://github.com/r-ryantm/nixpkgs/archive/bf82b0c4d6e1b6e80b7cc6f05634590a3b4e60b9.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/bf82b0c4d6e1b6e80b7cc6f05634590a3b4e60b9#geoserver

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/80f7viv7jxykhrzl07myndgvb7psrp61-geoserver-2.26.0
ls -la /nix/store/80f7viv7jxykhrzl07myndgvb7psrp61-geoserver-2.26.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

geoserver

Maintainer pings
cc @autra @imincik @l0b0 @nh2 @nialov @sikmir @willcohen for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

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

1 package built:

geoserver

The failing test (with a warning that might need investigating):
Native library load failed.
java.lang.UnsatisfiedLinkError: /nix/store/hlb9wys01pr51x004mh8pk95z9y4q6m6-gdal-3.9.2/lib/jni/libgdalalljni.so: /nix/store/hfb55givv2afsz93b705hkjd9y98wqam-libspatialite-5.1.0/lib/libspatialite.so.8: undefined symbol: xmlNanoHTTPCleanup, version LIBXML2_2.4.30
Oct 03, 2024 8:56:33 PM it.geosolutions.imageio.gdalframework.GDALUtilities loadGDAL
WARNING: Failed to load the GDAL native libs. This is not a problem unless you need to use the GDAL plugins: they won't be enabled.
java.lang.UnsatisfiedLinkError: 'void org.gdal.gdal.gdalJNI.AllRegister()'
Oct 03, 2024 8:56:33 PM it.geosolutions.imageio.plugins.turbojpeg.TurboJpegUtilities load
INFO: TurboJPEG library loaded (turbojpeg)
20:56:33 WARN   [geoserver.logging] - Could not find GeoServer logging.xml (or old services.xml) settings for logging
2

We again have that libxml2 problem. Can we define an overlay for one package, or do I need to patch every single dependency that depends on libxml2?

libxml2

It is really annoying. I'm not sure what others are doing.
Quick search shows only few packages with enableHttp override, most of them are our ones. But I have a feeling that it was more before.
git grep "libxml2" | grep "enableHttp"
pkgs/applications/gis/grass/default.nix:    (libxml2.override { enableHttp = true; })
pkgs/applications/misc/mysql-workbench/default.nix:    (libxml2.override { enableHttp = true; })
pkgs/by-name/ma/mapserver/package.nix:    (libxml2.override { enableHttp = true; })
pkgs/development/libraries/gdal/default.nix:      (libxml2.override { enableHttp = true; })
pkgs/development/libraries/librasterlite2/default.nix:    (libxml2.override { enableHttp = true; })
pkgs/development/libraries/libspatialite/default.nix:    (libxml2.override { enableHttp = true; })
pkgs/development/libraries/mapnik/default.nix:    (libxml2.override { enableHttp = true; })
pkgs/development/libraries/pdal/default.nix:    (libxml2.override { enableHttp = true; })
pkgs/games/armagetronad/default.nix:      buildInputs = lib.singleton (libxml2.override { enableHttp = true; })
pkgs/servers/sql/postgresql/generic.nix:      (libxml2.override {enableHttp = true;})

Let's wait for #350694 (and possibly #318188)

This PR is ready for merging now.


path templating parameter name include  '-' would caused validateRequest failed

I create a swagger  with path templating , and  one parameter name is  'key-name'
e.g.
"paths": {
    "/keys/{key-name}/create": {
      "post": {

        "parameters": [
          {
             "name":"key-name"

the request validate would always failed  with  result below:
{ code: 'MISSING_REQUIRED_PARAMETER',
            message: 'Value is required but was not provided',
            schemaPath: '#/paths/~1keys~1{key+name}~1create/post/parameters/0' }

Since I refer to  the swagger specification document , but I did not find any information about the parameter name could not include special character like '-'

I'll take a peek.  I'm not sure why it would fail.  Can you give more details on how you tested this?  (The weird schemaPath value isn't from sway but z-schema and it shouldn't mislead you.)  Can you give the full parameter definition and the requested path made?  Or maybe even a simple reproduction snippet of code?

it

I'll take a peek. I'm not sure why it would fail. Can you give more details on how you tested this? (The weird schemaPath value isn't from sway but z-schema and it shouldn't mislead you.) Can you give the full parameter definition and the requested path made? Or maybe even a simple repr

it

I'll take a peek. I'm not sure why it would fail. Can you give more details on how you tested this? (The weird schemaPath value isn't from sway but z-schema and it shouldn't mislead you.) Can you give the full parameter definition and the requested path made? Or maybe ev

I'll take a peek. I'm not sure why it would fail. Can you give more details on how you tested this? (The weird schemaPath value isn't from sway but z-schema and it shouldn't mislead you.) Can you give the full parameter definition and the requested path made? Or maybe even a simple reproduction snippet of code?

Since I have debug it, below code goes wrong , that cause the validate consider the parameter missed
https://github.com/apigee-127/sway/blob/e9ad6e81af41f526567e9d5a0f150e59e429e513/lib/types/path.js#L95
I think  it caused by the path-to-regexp ,
per https://github.com/pillarjs/path-to-regexp,  parameter names must use "word characters" ([A-Za-z0-9_]).


The default SignatureMaxLengthBase is too short, should be increased

I was trying to sign a PDF with a SHA256-RSA with a 2048 bit public key and a sizeable certificate. It kept failing, it was calling sign.Sign() once but context.SignPDF() twice. Upon opening the PDF in notepad there was only one trailing 0 in /Contents. It kept running out of space to write the PKCS#7 object. The issue was resolved after increasing the size of SignatureMaxLengthBase from 512 to 1024.
I think for the sake of users the default value should be increased to 2048.

The default size should be a reasonable, we don't want to make it too big as this would unnecessarily increase the default size of the resulting PDF file.
Did you try to include a timestamp and revocation information as well?
Some information that influences the "signature" size are taken into account dynamically, others such as the timestamp are not known in advance and this needs to be guessed.
Can you share more information about the certificate or the certificate itself, ideally an example that replicates the issue?

Here's is an example PDF that's been written twice to.
output.pdf
Here's the same PDF as above but with SignatureMaxLengthBase set to 1024.
output_with_sufficient_space.pdf
To replicate the problem follow the steps in this stack overflow answer https://stackoverflow.com/questions/77583031/sign-a-pdf-file-in-go-golang-using-digitorus-pdfsign but change privateKey, err := ecdsa.GenerateKey(elliptic.P384(), rand.Reader) into 	privateKey, err := rsa.GenerateKey(rand.Reader, 2048)
I understand your concern regarding not wanting to unnecessarily increase the default size but I think it's very inconvenient for users to have to download the source code themselves to increase the size. I think it'd be a good idea if a user can decide to set the size when calling the function.

Thanks for reporting!
The issue was caused by the relatively long subject DN of the self-signed certificate. The pkcs7 AddSignerChain adds the raw issuer to the pkcs7 envelop again, this was not taken into account when calculating the size.


Console App Can't Read Template Image on Windows

When generating the image on Windows, the template image is not found. It looks like the file path needs to be set as a hard link or the image needs to be copied to the output directory.
Expected Result: The text is displayed on the image.
Actual Result: It throws an exception.

Closed with #3


Add a test case to increase the nodecount of a running instance

Signed-off-by: Kevin Earls<EMAIL_ADDRESS>

@objectiser @jpkrohling  @pavolloffay Please review

Replaced by #824 because of problems I had with dependency checksums after the update to operator sdk 0.12 and golang 1.13


Fix memory leak in BitColorPicker

Related to #640
Address the memory leak issue in the BitColorPicker component.

Add a throttling mechanism to the HandlePointerMove method in src/BlazorUI/Bit.BlazorUI/Components/Inputs/_Pickers/ColorPicker/BitColorPicker.razor.cs to reduce the frequency of updates.
Ensure the Dispose method in src/BlazorUI/Bit.BlazorUI/Components/Inputs/_Pickers/ColorPicker/BitColorPicker.razor.cs properly disposes of registered events and the DotNetObjectReference.
Add a BitThrottler class instance to the BitColorPicker component in src/BlazorUI/Bit.BlazorUI/Components/Inputs/_Pickers/ColorPicker/BitColorPicker.razor.cs.
Ensure registered events are properly disposed of in src/BlazorUI/Bit.BlazorUI/Components/Inputs/_Pickers/ColorPicker/BitColorPicker.ts.
Add a unit test in src/BlazorUI/Bit.BlazorUI.Tests/Components/Inputs/ColorPicker/BitColorPickerTests.cs to verify that the memory leak issue is resolved.

@hootanht Thanks for submitting this PR.
First of all, this PR is not resolving the memory leak issues of the ColorPicker. it is trying to fix another issue of preventing the Immediate value binding. also resolving this new issue is not that trivial, you need to inherit this component from the input base component since we've already implemented the concept.
I don't think this issue can be resolved by someone out of our team who is not familiar with some important design concepts of our components, so I'm going to close this PR for now.
The only thing in this PR that is accepted is the disposing of the DotNetObjectReference that is required to be resolved in its own issue/PR.


Front end implementation of "Project" CRUD

Front implementation of creating , Reading, Updating and Deleting of a Project.
Add Project
Update Project
View project
List projects

Goog morning, the mockups for these interfaces already been done?

no, you can do the minimal design for now, and implement the front end, styling can be added later

who must set up the application layout?
I think we can use a free administration template and use it as a layout.
we could use this for example.

hello @ndounalex the design is already available ,let's work with that, check in the docs under mockups of @cliffordten  PR #17

Any updates on this task?


Get feedback form data on Mail

We can send the data of feedback form on mail with clean and clear ui

as in send the data back to user ?

@ANSHIKA-26 we can send this data to admin mail so that admin or if you want we can send this to user too
for user ...
"thanks for your feedback and details of feedback"
for admin ....
"person send a feedback and feedback details...."

sending data to admin makes sense , sending feedback back to user seems weird , just implement it for admin


Update rct to 1.2.2

Please update my adapter ioBroker.rct to version 1.2.2.
This pull request was created by https://www.iobroker.dev c0726ff.

replaces https://github.com/ioBroker/ioBroker.repositories/pull/2616

issues from old PR to be checkend:
The following issues must be fixed and require a new release:

[ ] "iobroker.js-controller": "^5.0.11" must not be part of package.jsosn dependencies.
has already been fixed in source but this fix is not part of release 1.1.7
[ ]  README.md must be written in pure english, please write changelog using english language
[ ]  merge PR https://github.com/aruttkamp/ioBroker.rct/pull/36 to add engines check
[ ]  consider updating adapter-core to current version
[ ]  consider using dependabot (this is optional and not blocking)

1.2.2 created 13.09.2023
18 User
OK


Multiple image posting support

Posting photosets isn't currently supported. Adding this issue in for reference if someone is ever looking for information on this in the issue history.
I've made a pull for it here: https://github.com/tumblr/tumblr.js/pull/23.

anyone that has control over this repo paying attention? This would be really nice to have in the official api, since it is supported . . .

Your brunch just solve my life and saved tons of time!!!
Thanks a lot!! If you need beer, vodka, anything else, i can send it :)
Super great
@Tumblr, why this repo is su outdated ((((

we haven't forgotten about this. 😁
there's some inconsistent behavior around photosets using source and data64 on the api side that i want to iron out that makes me hesitant to have this implemented in the client right this second,

+1
been awhile, any update on this?


Feature Request: Custom [default].yml

Hi, I have been working on my local installed version of tmuxinator and setting up a way for me to pass a template for it to use rather than default.yml if it exists.
Use Case: I have multiple projects that match a similar structure and will need to execute commands on launch to run servers ect. Using default.yml as that project setup is nice, but I also have other projects that match setup that aren't like the default.yml.  What I would like is a syntax to pass the template to use for its default.
This is a feature I will be adding to my downloaded version and was wondering if it was something the community was interested in and if so what would the preferred syntax for passing in the default look like?
I don't think the change would be too major as the File system already has a method exists which could be passed the filename (which it uses to check if it should open the file or create a new yml) and it already has a CMD line parser using THOR.  So it would just be passing the choosen syntax into the exists function then using that as [default] rather than default.yml.

I have it working on my local with this syntax
tmuxinator open [projectName] --default [defaultName]
It then checks if the file exists if it does not then it falls back to default if this doesn't exist then it uses the sample provided.
Open to suggestions.


Cannot refresh configs in @FeignClient with @RefreshScope

I'm running a Spring Config Server and another Spring boot app that uses config server. I have a Feign-Client in my second app like this:
@FeignClient(url = "${myconfig.url}", name = "ConfigServerFeign",
        configuration = {FeignConfig.class}, fallback = ConfigServerFeignFallback.class)
public interface ConfigServerFeign
{
    @RequestMapping(value = "/test", method = RequestMethod.POST,
            headers = {"Content-Type=application/json; charset=UTF-8"}, consumes = {MediaType.APPLICATION_JSON_UTF8_VALUE})
    RequestModel test(@RequestBody RequestModel request);
}

and i have a config class for feign like this:
@Configuration
public class FeignConfig
{
    @Value("${myconfig.header}")
    private String authorization;
    @Bean
    public RequestInterceptor basicAuthRequestInterceptor()
    {
        return requestTemplate -> requestTemplate.header("Authorization", "Basic " + authorization);
    }
}

i need to change ${myconfig.url} or ${myconfig.header} in runtime. i think i have two option, first one is using @RefreshScope on top of ConfigServerConfig that does not work, the second option is to use @RefreshScope on FeignConfig class that gives me this error:
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'com.example.configclient.ConfigServerFeign': FactoryBean threw exception on object creation; nested exception is java.lang.IllegalStateException: No Scope registered for scope name 'refresh'
    at org.springframework.beans.factory.support.FactoryBeanRegistrySupport.doGetObjectFromFactoryBean(FactoryBeanRegistrySupport.java:175) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.FactoryBeanRegistrySupport.getObjectFromFactoryBean(FactoryBeanRegistrySupport.java:103) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.AbstractBeanFactory.getObjectForBeanInstance(AbstractBeanFactory.java:1634) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:254) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:202) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.config.DependencyDescriptor.resolveCandidate(DependencyDescriptor.java:208) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.DefaultListableBeanFactory.addCandidateEntry(DefaultListableBeanFactory.java:1316) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.DefaultListableBeanFactory.findAutowireCandidates(DefaultListableBeanFactory.java:1282) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.DefaultListableBeanFactory.doResolveDependency(DefaultListableBeanFactory.java:1101) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.DefaultListableBeanFactory.resolveDependency(DefaultListableBeanFactory.java:1066) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.annotation.AutowiredAnnotationBeanPostProcessor$AutowiredFieldElement.inject(AutowiredAnnotationBeanPostProcessor.java:585) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    ... 19 common frames omitted
Caused by: java.lang.IllegalStateException: No Scope registered for scope name 'refresh'
    at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:337) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:197) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.ConstructorResolver.instantiateUsingFactoryMethod(ConstructorResolver.java:372) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.instantiateUsingFactoryMethod(AbstractAutowireCapableBeanFactory.java:1173) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBeanInstance(AbstractAutowireCapableBeanFactory.java:1067) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:513) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:483) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.AbstractBeanFactory$1.getObject(AbstractBeanFactory.java:306) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:230) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:302) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:197) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.beans.factory.support.DefaultListableBeanFactory.preInstantiateSingletons(DefaultListableBeanFactory.java:761) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.context.support.AbstractApplicationContext.finishBeanFactoryInitialization(AbstractApplicationContext.java:867) ~[spring-context-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:543) ~[spring-context-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    at org.springframework.cloud.context.named.NamedContextFactory.createContext(NamedContextFactory.java:116) ~[spring-cloud-context-1.2.4.RELEASE.jar:1.2.4.RELEASE]
    at org.springframework.cloud.context.named.NamedContextFactory.getContext(NamedContextFactory.java:85) ~[spring-cloud-context-1.2.4.RELEASE.jar:1.2.4.RELEASE]
    at org.springframework.cloud.context.named.NamedContextFactory.getInstance(NamedContextFactory.java:121) ~[spring-cloud-context-1.2.4.RELEASE.jar:1.2.4.RELEASE]
    at org.springframework.cloud.netflix.feign.FeignClientFactoryBean.get(FeignClientFactoryBean.java:127) ~[spring-cloud-netflix-core-1.3.5.RELEASE.jar:1.3.5.RELEASE]
    at org.springframework.cloud.netflix.feign.FeignClientFactoryBean.feign(FeignClientFactoryBean.java:84) ~[spring-cloud-netflix-core-1.3.5.RELEASE.jar:1.3.5.RELEASE]
    at org.springframework.cloud.netflix.feign.FeignClientFactoryBean.getObject(FeignClientFactoryBean.java:155) ~[spring-cloud-netflix-core-1.3.5.RELEASE.jar:1.3.5.RELEASE]
    at org.springframework.beans.factory.support.FactoryBeanRegistrySupport.doGetObjectFromFactoryBean(FactoryBeanRegistrySupport.java:168) ~[spring-beans-4.3.12.RELEASE.jar:4.3.12.RELEASE]
    ... 29 common frames omitted

how can i fix this ??? :-(

I would use @CongurationProperties instead of @Value.

i face with this problem too,
@FeignClient(name="balabala" , url = "${myconfig.url}") ;   i want to refresh the "url"     but @RefreshScope doesn't work
version Edgware.SR2

sorry be absent  , it was Tomb-Sweeping Day in china,  we got 3 days off
here is my demo show this case :
https://github.com/minorpoet/feignclient-url-not-refresh

any response ?

Can you please simplify your sample?  All those components should not be needed based on the description of the problem.

I think I stumbled into the same problem and prepared a quite simple example: https://github.com/BastiOfBerlin/feignclient-refreshscope-problem
I change the URL after the EmbeddedServletContainer has been initialized. In my real app, I need to access the (randomly selected) port number to change the url property.
I then refresh the scope. Unfortunately, the FeignClient seems to be already initialized and does not get refreshed.

@BastiOfBerlin wouldn't it be simpler to use ribbon in that case. The url field was never meant to be updated.

In this demo, definitely.
But I have to implement sort of a loopback to the exact same microservice. Using client.api-url=${server.port} in the config doesn't work either. So I came up with the idea of waiting until the servlet container came up and set the config value to '"http://localhost:" + localServerPort'.
If ribbon could do that, I'd be happy, but have no idea on how to implement that.
I guess there's no way to re-initialize all or specific FeignClients?

but,  lacking of these components   spring cloud config with bus won't work

@minorpoet

but, lacking of these components spring cloud config with bus won't work

what does this mean?

i mean,  i  need git repo and mq to support notifying config changing event  to  spring cloud config consumer;   if not using mq and bus,  git repo is needed at least  and refresh config consumer directly through  {consumer-host}:{port}/refresh

@minorpoet not sure what those components have to do with the problem.  As @spencergibb said the url field is not meant to be refreshed.  You can instead use Ribbon and set client.ribbon.listOfServers.  When that property is updated and a refresh is performed than the Feign client should use the new property.

but my target url  is php server,  not registered in eureka; while the config consumer is resigtered

I am facing a similar issue with the "name" attribute. I'd like to be able to replace the eureka service ID that I am using at runtime without restarting the application. Instead of calling target-ms-v1, I want to call target-ms-v2. However, since "name" is in an annotation attribute it cannot be refreshed at runtime.
I there a way to use code to define the feign client instead of doing it with @FeignClient? That way I might be able to use @RefreshScope myself.

There is a section in the docs about creating feign clients manually

Thanks Spencer, I'll check it out!
So just to clarify - you do not see a way to refresh the name attribute via @RefreshScope (or other means), correct?

No there is not.

@minorpoet 我也有相同的疑问 兄弟你解决了吗

Sorry to hijack the issue. I have a question that's related to this problem but has more to do with using refresh scope on the override beans we can provide in the configuration, e.g Logging.Level, Request.Options. You cannot put @RefreshScope on these beans either as it gives the same error. Does the provided solution of using the property "feign.client.config.feignName.readTimeout" apply dynamic values if changed in config server?

@sj853  没呢，github通知没注意~~    现在才看到

Closing due to age of the question. If you would like us to look at this issue, please comment and we will look at re-opening the issue.

I have same question. Two years passed, did this question solved?

I am also looking forward for a way to refresh feign when url has been changed and it's in RefreshScope

its of current interests


mmdetection version

Hi! Could you please provide the specific version of the mmdetection used in your paper? I cloned the v2.18 from mmdetection repository and it seems like your code is incompatible with this version of mmdetection.
Thank You!

Hi, @yqjiang6
Have you solved this version problem?
Maybe mmdet==2.3.0 will be ok.

I found the code is compatible with mmdet-2.3, and it works well for me.

@ruiningTang
Ok, thanks bro!

Thank for your reply. MMdet-2.3 works well!


cookies

i ave a php page which response a json data  but to access this page i need a user login cookies  so is it possible to access it from this AsyncHttpClient

Issues in this project are meant to track bugs and feature requests, not general programming questions.


Scheduler memory just keep increasing in idle mode

Hello Dask community !
I am running Dask on a linux server and I am very limited by my memory. My main problem is that the the dask scheduler process just keep eating more and more memory even though I don't submit any work to the workers yet ! I would like to understand what's going on and if there is any mitigation to this problem ?
To reproduce it:
from dask.distributed import Client
client = Client(memory_limit='100MB', processes=False, n_workers=4, threads_per_worker=1)
Then check memory with top | grep python
Ps : I succeded in limiting the worker process memory by adding --memory limit to the client call but this doesn't seem to be possible for the scheduler process.

Hi @ridhachahed are you perhaps using the dask-labextension? If so, might you be running into https://github.com/dask/dask-labextension/issues/185 ?

Hello @ian-r-rose , unfortunately I am not using dask-labextension everything is run form the command line without  the dashboard. You will find attached the numbers I have where we can the see scheduler memory increasing while the worker memory reaches a plateau at one point (when we don't put a limit on their memory).


platform adapter

What happened:

What you expected to happen:
expected create a k8s cluster with multi master
sudo sealer run kubernetes:v1.19.8 --masters <IP_ADDRESS>,<IP_ADDRESS> --nodes <IP_ADDRESS> --user ubuntu

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:
Environment:

sealer version (use sealer version):
sealer was build from latest code( main branch)
Cloud provider or hardware configuration:

OS (e.g: cat /etc/os-release):

ubuntu@vm01:/var/lib/sealer/data/my-cluster/rootfs/scripts$ cat /etc/os-release
NAME="Ubuntu"
VERSION="18.04.6 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.6 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

Kernel (e.g. uname -a):
Install tools:
Others:

@shaobo322 ，pls try to use root user to execute sealer run with version 0.8.5 . Clusterimage kubernetes:v1.19.8 support arm64 architecture.

@shaobo322 ，pls try to use root user to execute sealer run with version 0.8.5 . Clusterimage kubernetes:v1.19.8 support arm64 architecture.

@kakaZhou719
No use. I ran with root user, and met the same problem. It seems to be the problem of docker installation and Ubuntu Distribution issues.

It has been fixed in https://github.com/sealerio/sealer/pull/1508


Fix a bug in how UOM ignores punctuations

Punctuations should be treated as empty words by the UOM, but a bug in
the code missed the check for the frontmost (called the "anterior")
unigram. For example, after typing "！！自會" and choosing "字彙", the
next time the UOM sees "？？自會", it should suggest "字彙", but because
of the bug it didn't happen.
Also addresses the issues found by clang-tidy.

I tested it and it looks good. :)


Calling option not showing up in Skype client

Documentation in app.js says:
"If you're adding calling to an existing bot can take a few minutes for the calling option to show up."
I waited 10 minutes, stopped & re-started Skype several times. Still no option to call. Went for a walk. An hour later, still no option to call.
Resolved by removing the Skype contact completely and re-adding it (via the bot page on dev.botframework.com)

Ill make sure the documentation is updated to reflect this.

Thanks for the report. We will update the documentation to indicate that it can take up to 24 hrs.

Hi @gkc, you mention the documentation in app.js, where is it exactly?

This has been fixed in the sample comment
https://github.com/Microsoft/BotBuilder/blob/master/Node/examples/demo-skype-calling/app.js
Thanks a lot for reporting this issue, and let us know if you have any further questions.


Create 0.1.0 release for rai_test_utils

Description
Create 0.1.0 release for rai_test_utils
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #1952 (0bfdc45) into main (d7dd849) will decrease coverage by 0.56%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1952      +/-   ##
==========================================
- Coverage   94.16%   93.60%   -0.56%
==========================================
  Files          95       31      -64
  Lines        4712      579    -4133
==========================================
- Hits         4437      542    -3895
+ Misses        275       37     -238

Flag
Coverage Δ

unittests
93.60% <100.00%> (-0.56%)
:arrow_down:

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

Impacted Files
Coverage Δ

rai_test_utils/rai_test_utils/version.py
100.00% <100.00%> (ø)

...is/erroranalysis/_internal/process_categoricals.py

...bleai/responsibleai/_tools/causal/causal_result.py

...is/erroranalysis/_internal/surrogate_error_tree.py

...sibleai/responsibleai/_tools/shared/base_result.py

responsibleai/responsibleai/feature_metadata.py

...ibleai/responsibleai/managers/explainer_manager.py

...ibleai/_tools/shared/state_directory_management.py

...nsibleai/databalanceanalysis/data_balance_utils.py

responsibleai/responsibleai/exceptions.py

... and 55 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


@conda-forge-admin, please add user @cboulay

Additional comment:
No response

https://conda-forge.org/docs/maintainer/updating_pkgs.html#updating-the-maintainer-list

@conda-forge-admin, please add user @cboulay


Add a first try at circleci configuration

${COMMENTBODY}

View build details on CDash}

View build details on CDash

View build and test results for change 5d3ce4d on CDash

View build and test results for change add-fir on CDash

View build and test results for change 373e000 on CDash

View build and test results for change 83f2285 on CDash


Create dedicated LD4P2 branch capturing state of code at end of LD4P2 grant

In case we continue to use the code originally forked from blacklight-cornell, it would be useful to capture the code in a dedicated branch (and not just dev)

Branch now exists: https://github.com/LD4P/blacklight-cornell/tree/ld4p2


in ubuntu environment - error while loading shared libraries: libfontconfig.so.1

/node_modules/screenshot-stream/node_modules/phantom-bridge/node_modules/phantomjs/lib/phantom/bin/phantomjs: error while loading shared libraries: libfontconfig.so.1: cannot open shared object file: No such file or directory"}
https://github.com/ariya/phantomjs/issues/10904
They need us to install on certain linux environment.
sudo apt-get install libfontconfig

https://github.com/Medium/phantomjs

Thanks dude... i found this... https://github.com/ariya/phantomjs/issues/12948

hope screenshot-stream will use phantomjs 2?


vscode not sync python.venvFolders

VS Code Version: 1.95.3
OS Version: win11 23H2 22631.4460

Could you give some steps to help us reproduce the issue?

add this line "python.venvFolders":["~\\AppData\\Local\\pypoetry\\Cache\\virtualenvs"] to settings.json


Outputs do not take into account when working with an existing subnet

Is there an existing issue for this?

[X] I have searched the existing issues

Greenfield/Brownfield provisioning
greenfield
Terraform Version
1.4.7
Module Version
0.1.4
AzureRM Provider Version
3.79.0
Affected Resource(s)/Data Source(s)
outputs
Terraform Configuration Files
When you give it an existing subnet the resource azurerm_subnet "vgw" will not run so the output fails.

tfvars variables values
{
  "resource_group_location": "italynorth",
  "resource_group_name": "some_name",
  "subnet_id": "/subscriptions/<some_subscription>/resourceGroups/rg/providers/Microsoft.Network/virtualNetworks/vnet/subnets/subnet_name",
  "tags": {
    "Creator": "tf",
    "Environment": "prod",
    "Inventory": "bla",
    "Role": "vpn"
  },
  "virtual_network_name": "bla-prod-tf-vpc"
}

Debug Output/Panic Output
Error: Error in function call
│
│   on .terraform/modules/vgw/outputs.tf line 35, in output "subnet":
│   35:     name = try(azurerm_subnet.vgw[0].name)
│     ├────────────────
│     │ while calling try(expressions...)
│     │ azurerm_subnet.vgw is empty tuple

Expected Behaviour
The module should work even when you give it a "subnet_id".
Actual Behaviour
The module crashes when you give it a "subnet_id".
Steps to Reproduce
Run the module with "subnet_id" set.
Important Factoids
None
References
None

Thanks for raising, will get this change implemented for the next release.


Update Composer dependencies (2022-01-29-00-04)

Visual regression test passed!
View the visual regression test report


Revert "Don't deploy or install container-jersey2 [run-systemtest]"

Reverts vespa-engine/vespa#18570
Seems like a jar is installed but not packaged:
19:18:33 error: Installed (but unpackaged) file(s) found:
19:18:33    /opt/vespa/lib/jars/container-jersey2-jar-with-dependencies.jar

Thanks, I didn't know that it also must be removed from install.


Add tool tip for EOLStatus

Noticed one of 'summary descriptions' had a tooltip missing, figured it's worth adding; if additional code / tests needed for this, can close this PR or complete without me as I have no knowledge on IDEs.

@OverLordGoldDragon, thanks a lot for your contribution! We'd like to include it in our next release in the Spyder 4 series. For that, please make your branch to derive from our 4.x branch (instead of master), with the following commands:
git checkout master
git reset --soft HEAD~1
git checkout 4.x
git checkout -b add-eol-tooltip (or whatever branch name is fine for you)
git commit -a -m "Add tool tip to EOLStatus"
git push -f origin master
git push origin add-eol-tooltip

If I'm not wrong, the last command will lead you to open a new pull request for this. But that's not a problem, you can simply close this one and do that.
If you decline to do that, your work will be part of Spyder 5, to be released in 2021.

How unusual seeing "assigned OverLordGoldDragon" (but in a good way), with all the Spyder PR texts floating -- Would simply making a new fork derived from 4.x instead work?

Used the git commands - good to go with new PR.


The article link for Trace Flag 1148 returns a 404 error

The link for trace flag 1448:
https://support.microsoft.com/kb/937041
Returned a 404 error.

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

ID: d0e35dc1-14be-0678-aeae-88de1c7601ac
Version Independent ID: b3700e3e-acbf-9248-c333-ac9013827d84
Content: Trace Flags (Transact-SQL) - SQL Server
Content Source: docs/t-sql/database-console-commands/dbcc-traceon-trace-flags-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @pmasl
Microsoft Alias: pelopes

Correction, that is tf 1448.

Thank you, I've requested for the link to be reactivated.

#please-close

It still returns http 404. Please re-open! Thanks!


Development: Improve and simplify local playwright setup

Checklist
General

[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Motivation and Context

Setting up playwright initially is quite cumbersome, as several steps need to be followed before it can be used.
Description

This PR simplifies the setup by providing three simple scripts, the user can execute to run playwright fast.
Steps for Testing

Locally, checkout this branch.  (checkout feature/playwright/simplify-setup)
See the README.md in supporting_scripts/playwright
Follow the steps, and verify that the steps are easy to follow, and you can run playwright tests in the UI.

Review Progress

Code Review

[x] Code Review 1
[ ] Code Review 2

Manual Tests

[x] Test 1 on linux (ubuntu/debian, other distros might not work)
[x] Test 2 on macOS
[x] Test 3 on macOS

Summary by CodeRabbit
Release Notes

New Features

Enhanced documentation for Playwright setup, including a streamlined three-step process.
New scripts for automating user setup and running Artemis in Docker on both macOS and Linux.
Introduced a new script for installing and launching Playwright.

Improvements

Simplified username and password template configuration.
Added timeout configuration options for better test execution control.
Updated installation command for Playwright to reflect changes in the setup process.

Documentation

Added a README file for clearer instructions on setting up Playwright with Artemis.

docker/playwright.yml compose file needs to be changed to correctly execute the tests on CI. Command of artemis-playwright should have npm run playwright:install now instead of npm run playwright:setup. Also we probably need to execute setupUsers.sh script between playwright:install and playwright:test steps in compose file, now that setting up users is a separate script.

That's a good point. I reverted the changes, which the CLI tests need to run on bamboo. For the user, using the setup locally nothing has changed.
To have a clear separation between local/bamboo (especially because we want to move to github for the e2e tests soon) I decided to not modify the current playwright-docker setup

Left a small comment. Also I added this branch manually to bamboo, as it wasn't created for some reason 😅


fix: beerusup uses beerus_cli full path on install

Pull Request type

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Testing
[ ] Other (please describe):

What is the current behavior?

Issue Number: Fix #116
What is the new behavior?

beerusup uses beerus_cli full path on install
No error and instead version is shown

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

Thank you, just saw the same error. Can you include the full path on line 146 as well(CURRENT_VER=$(beerus_cli --version | awk '{print $2}'))?

Codecov Report
Base: 83.64% // Head: 83.64% // No change to project coverage :thumbsup:

Coverage data is based on head (ba299ab) compared to base (1141943).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #117   +/-   ##
=======================================
  Coverage   83.64%   83.64%
=======================================
  Files          21       21
  Lines         911      911
=======================================
  Hits          762      762
  Misses        149      149

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Thank you, just saw the same error. Can you include the full path on line 146 as well.

Thanks @drspacemn ,will add this on line 146 as well 👍


[BUG] Default path not finding default Julia 1.4.1 Binaries

I just did a fresh install of Julia, Atom, uber-juno in Windows 10.

the new default installation directory for Julia 1.4.1 is
C:\Users\azevelev\AppData\Local\Programs\Julia\Julia-1.4.1
After installing uber-juno, Atom was not able to locate Julia.
I had to go to: Packages > Juno > Settings
and change Julia Path to: C:\Users\azevelev\AppData\Local\Programs\Julia\Julia-1.4.1\bin\julia.exe

Also, there is a typo in the juno installation instructions:
It says:
"Go to Packages > Julia > Settings and change "
It should say:
"Go to Packages > Juno> Settings and change "

Didn't your Julia installer automatically set up a env path to the Julia executable ?
Our Julia path detection logic will use that unless an user specified manually.

I used the default directory. I used all default options