VCL rules Question

I have this working, for sub vcl_receive, do I need equivalents in vcl miss or anywhere else to be effective against DDoS?
sub vcl_recv {
# Check requests per second using vsthrottle
if (vsthrottle.is_denied(client.identity, 15, 10s)) {
    # Client has exceeded 15 reqs per 10s, reset connection using shield
	shield.conn_reset();
	return (synth(429, "Too Many Requests"));
	}
}

This is an issue tracker, not a help forum. Please come by on IRC (irc.linpro.no / #varnish ) if you need help.


Align configuration of kokoro and manual foundry runs

Currently configuration for manual foundry runs and foundry runs on kokoro lives in two different places and it's hard to keep the two in sync (the manual config is currently out of date)
it's very important that the config for the manual runs is as similar to the kokoro one (in terms of execution environment) so the flakes on kokoro are easy to reproduce locally - without local reproducibility, it's hard to fix anything
The manual config currently lives in a google doc and that makes it very hard to make sure that everyone on the team is using the right and up-to-date configuration.

This PR moves common configuration for kokoro and manual run to a bazelrc file that is imported by both kokoro-specific config and manual-specific config.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

@billfeng327 @nicolasnoble there are a few more TODOs to figure out in the code, but otherwise this is basically ready for review.

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,997,294      Total (=)      1,997,294

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
11,021,760      Total (<)     11,021,766

 No significant differences in binary sizes

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,997,294      Total (=)      1,997,294

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
11,021,762      Total (>)     11,021,761

 No significant differences in binary sizes

Corrupt JSON data (indicates timeout or crash):
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.new: 10
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.old: 10

[microbenchmarks] No significant performance differences

Corrupt JSON data (indicates timeout or crash):
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.new: 10
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.old: 10

[microbenchmarks] No significant performance differences

I addressed the remaining problems and this is now ready for review & merge.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Corrupt JSON data (indicates timeout or crash):
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.new: 10
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.old: 10

[microbenchmarks] No significant performance differences


A migrator for upgrading v1 to v2

Mainly for Docker users

Change the data dir's owner to node from root

docker run --rm -v uptime-kuma:/app/data --pull always louislam/uptime-kuma:migrator

Alternatively, if users know their path, they could use:
chown -R 1000:1000 <volume or binding path>

But not everyone knows the path, so the migrator will be easier to use.
Also renamed dockerfile to Dockerfile along with this pr.

@louislam
Also a migrator between SQLite and MySQL would be awesome

Close as the plan was changed #4052


Add Initial Snowfakery recipe from Jung to membership repo

Critical Changes
Changes
Added changes to the Cumulus.YML file and two Snowfakey recipes
Issues Closed

ready for merge @selfuntitled thanks for catching the typo


KAFKA-5063: Flaky ResetIntegrationTest

Call for review @enothereska @dguy @guozhangwang

I noticed one of the wait functions does not have the 60000 timeout.

Also there is a test that failed, but it seems the results are not there anymore.

retest this please

About the timeout -- I guess we can got back to default anyway -- increasing the timeouts to 60000 was a try to stabilize the test -- it was not a proper fix though. WDTY about this?

Is this still needed?

retest this please

I think yes -- the failure we observed is unrelated to the consumer reset strategy (cf https://github.com/apache/kafka/pull/2931/files#r114402394)

Updated this.

Retest this please.

Retest this please

retest this please

Created https://issues.apache.org/jira/browse/KAFKA-5209 for the newly observed transient failures.


ppx_deriving_yojson generates a bad signature (val _ : ....)

I'm on Ocaml 4.07.1 and ppx_deriving_yojson 3.5.1 and eliom 6.7.0
It seems like ppx_deriving_yojson is generating an extra function with the name _, for the last of the derived types.
Am I doing something wrong?
trading_model.eliom:
[%%shared
type sellOrderCapture = {
  sellOrderId : int;
}
[@@deriving yojson]

type buyOrderCapture = {
  buyOrderId : int;
  quantity : int;
}
[@@deriving yojson]

type t = {
  sellOrder : sellOrderCapture;
  buyOrders : buyOrderCapture list;
}
[@@deriving yojson]

Error:
File "_server/trading_model.type_mli", line 17, characters 8-9:
Error: Syntax error

trading_model.type_mli:
module MatchCapture :
sig
  type sellOrderCapture = { sellOrderId : int; }
  val sellOrderCapture_to_yojson : sellOrderCapture -> Yojson.Safe.t
  val sellOrderCapture_of_yojson :
    Yojson.Safe.t -> sellOrderCapture Ppx_deriving_yojson_runtime.error_or
  type buyOrderCapture = { buyOrderId : int; quantity : int; }
  val buyOrderCapture_to_yojson : buyOrderCapture -> Yojson.Safe.t
  val buyOrderCapture_of_yojson :
    Yojson.Safe.t -> buyOrderCapture Ppx_deriving_yojson_runtime.error_or
  type t = {
    sellOrder : sellOrderCapture;
    buyOrders : buyOrderCapture list;
  }
  val to_yojson : t -> Yojson.Safe.t
  val of_yojson : Yojson.Safe.t -> t Ppx_deriving_yojson_runtime.error_or
  val _ : Yojson.Safe.t -> t Ppx_deriving_yojson_runtime.error_or
end

I can circumvent it by removing the offending line, but this is not a stable solution.

An early attempt at reproducing this locally (see below) fails to reproduce on my machine (ppx_deriving_yojson 3.5.1, OCaml 4.08). (Out of curiosity, what is your ppx_deriving version? Is your build really using ppx_deriving, or a different implementation specific to Eliom/Ocsigen? Can you reproduce the failure in a non-eliom project?)
module Test_regression_104 : sig
  (* regression test for https://github.com/ocaml-ppx/ppx_deriving_yojson/issues/104 *)
  type sellOrderCapture = {
    sellOrderId : int;
  }
  [@@deriving yojson]

  type buyOrderCapture = {
    buyOrderId : int;
    quantity : int;
  }
  [@@deriving yojson]

  type t = {
    sellOrder : sellOrderCapture;
    buyOrders : buyOrderCapture list;
  }
  [@@deriving yojson]
end = struct
  type sellOrderCapture = {
    sellOrderId : int;
  }
  [@@deriving yojson]

  type buyOrderCapture = {
    buyOrderId : int;
    quantity : int;
  }
  [@@deriving yojson]

  type t = {
    sellOrder : sellOrderCapture;
    buyOrders : buyOrderCapture list;
  }
  [@@deriving yojson]
end

My build was using the actual ppx_deriving, and trying it out with normal OCaml/no Eliom, I couldn't replicate this either.
But I've made an issue in Ocsigen/tuto instead, which is where I found the instructions that lead to this in the first place.


fixed address_info response schema

Changes
Changed address_info/ response schema

How to test these changes?
manually

Related issues
Resolves #194

Example requests:
localhost:7481/address_info/ethereum_blockchain?address=0x2e98a6804e4b6c832ed0ca876a943abd3400b224
Expected responcse:


Hash method for ACSets

To avoid situations where a==b but hash(a)!=hash(b), a hash method for C-Sets is added.
Addresses https://github.com/AlgebraicJulia/Catlab.jl/issues/656

Thanks for the fix, Kris! Unfortunately, the CompTime refactor just got merged so you'll have to update this PR to fix the conflict.


US Core V3.1.1 Smoking Status Observation Tests 14.08 error on effectiveDateTime (FI-1634)

Expecting since this is V3.1.1 that the effectivePeriod would be valid.
https://www.hl7.org/fhir/us/core/STU3.1.1/StructureDefinition-us-core-smokingstatus.html

@healthland
Thank you for reach to us. We also noticed this problem after introducing US Core 4.0.0 Test Kit. The issue is that US Core 3.1.1 Test Kit validates using US Core 4.0.0 profile. We are working on the fix of that.

Thank you for confirming what we thought we were seeing as an issue.  Thanks for working on a fix.  Thanks!

Reopen for release process

Hey all,
With regards to the update to the us-core-test-kit, I thought the root cause of this issue was that the FHIR validator couldn't have multiple versions of the same IG loaded at once. Has that issue been resolved in the hapi fhir validator? Or I suppose the better question is: have you confirmed that adding the profile version to the request to the FHIR Validator actually uses the correct profile version?

have you confirmed that adding the profile version to the request to the FHIR Validator actually uses the correct profile version?

Yes, we have confirmed this. There are caveats (I can provide more details if you want), but none that should affect US Core. This functionality is in main in the US Core test kit (not yet in an official release), so you can test it out there. As a simple spot check, our reference server is loaded with 3.1.1 data, and the Smoking Status observations fail validation against the 4.0.0 profile. You can verify that they pass in 3.1.1 but fail in 4.0.0.


question about .fcb file

@mhalber
In the project website, all information introducing  .fcb file is only the following sentence:
"Feature files (.fcb) are sets of preprocessed RGB-D frames put into a single file. We provide them for the ease of experimentation with our system. Our code also provides conf2fet program that is able to produce a fet file from a configuration file."
However, it is not enough for us to understand the fcb file. Could you provide more introduction about .fcn file? Thanks.
In addition, is there a way to display the extracted structure features, such as plane and Silhouette?

Hi, thank you for the issue. The fcb files are representation for FETReconstruction class found in gaps/pkgs/FET. I guess I can create a better write-up on fcb files in near future, but it would be useful to know what information you are exactly looking for.
As far as visualization go, the view tab has a plenty of visualization options available. For what you're asking there is a feature display mode options - there is rgb, feature_id, normals and phong shading. Feature_id color scheme is explained in vs_pointcloud source code in rendering.h. See screenshot for the display mode slider.

@mhalber
Thanks for your reply.  I have two questions.
The first question is which paper do you refer to extract geometric features sucn as plane,  silhouette, ridge and valley. The codes in gaps/pkgs/FET do not indicate the reference papers, although the cvpr paper gives a reference list:
"
We have experimented with a number of feature types, including SIFT and Harris corners in both
color and depth images. However, we have ultimately found planar patches [3, 10, 11, 12, 25, 29, 31, 35,
39, 41, 43] and linear edges along creases and contours in depth images [52] to be most robust.
"
The second question is that the code dose not provide us with visualization application. After compilation, we only get two executable files: conf2fet and fetregistration. How could I get the visualization application?

Hi again,
As far as the extracting geometric features, in the paper we list the publications that used such type of a feature, as we do not want to claim a contribution in this space. That being said we do not exactly implement any existing solution, rather we came up with an algorithm that worked for us here. So I can't point you to a specific paper here. conf2fet provides implementation that we created and used.
Secondly the fetregister, if compiled using command
make USE_WINDOW=1

Then running
fetregister fet/5_0.05.fcb hv_c11_2_initial.conf -v

in the respective folder(provided fetregister is in search path) should create a interface like the one in the screenshot above. Is it not the case for you? I

@mhalber
Thanks very much. There are some errors when compiling under "USE_WINDOW=1" condition. Anyway , Thank you again.

I've pushed an update to bulid the fetregister under linux with "USE_WINDOW=1". Please check if these help you. If not, could you open new issue?

Hi,
I have compile the code under linux with "USE_WINDOW=1". Thanks very much.
In addition, is there any documents about gaps package? The code is hard for me to understand, and the documents will make me better to understand the paper.
Thanks again!
2017-05-13
why_scholar
发件人：Maciej Halber<EMAIL_ADDRESS>发送时间：2017-05-10 22:23
主题：Re: [mhalber/Fine-To-Coarse-Registration] question about .fcb file (#3)
<EMAIL_ADDRESS>抄送："why-scholar"why_scholar<EMAIL_ADDRESS>I've pushed an update to bulid the fetregister under linux with "USE_WINDOW=1". Please check if these help you. If not, could you open new issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Gaps is a package written by prof. Thomas Funkhouser, hosted here: https://github.com/tomfunkhouser/gaps
As far as I know there is no documentation, other than just reading the code and comments within.

Ok！ Thanks very much!
2017-05-17
why_scholar
发件人：Maciej Halber<EMAIL_ADDRESS>发送时间：2017-05-17 03:11
主题：Re: [mhalber/Fine-To-Coarse-Registration] question about .fcb file (#3)
<EMAIL_ADDRESS>抄送："why-scholar"why_scholar<EMAIL_ADDRESS>Gaps is a package written by prof. Thomas Funkhouser, hosted here: https://github.com/tomfunkhouser/gaps
As far as I know there is no documentation, other than just reading the code and comments within.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Can't see my URL bar at all when a request is selected

Expected Behavior
I'd expect there to be a URL bar to use in each request, above the request/response panels. There is not!
Actual Behavior

Reproduction Steps

Download a fresh copy of Insomnia (M1 Macbook)
Log in
Open a collection (or create one)
Create a request
There is no url bar.

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
9.3.2
What operating system are you using?
macOS
Operating System Version
macOS Sonoma Version 14.5
Installation method
.dmg from website
Last Known Working Insomnia version
No response

Hi @ryanoillataguerre it looks like the request panel is minimized. If you try to grab the resize handle from the top/left based on if your layout is horizontal/vertical you can expand it back.
I've also made an update to not close those panels completely when resizing so that it doesn't create confusion like this.
Let me know if this worked for you

@gatzjames Whoa haha that's wild, would never have guessed. Thanks a bunch though! Good stuff


Zoomed page looks jaggy on some devices.

System information

MangaView version: 1.0.0
Android version: Android 9.0(API Level 28)
Device: Lenovo TB-7305F(Lenovo Tab M7)

Describe the current behavior

Launch the sample app.
Zoom page.
The zoomed pages(images) are jagged.

Lenovo Tab M7

Pixel 4a

Describe the expected behavior
The page is processed anti-alias.

Already fixed 44523925af317b8eb5406396b0fc23bc5012aa12


⚠ Secret Website 3 has degraded performance

In 5cba235, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2088 ms

Resolved: Secret Website 3 performance has improved in 2386285.


typecheker+codegen: Add basic support for ranges in match cases

function main() {
    let x = 42

    let y = match x {
        0..30 => "foo"
        30..100 => "bar"
        else => "baz"
    }

    println("{}", y)
}

I'd prefer to hold out on this (+ generating specialised code for it) until we can implement it via traits...but if it helps simplify something in the meantime, we could have the specialised implementation

I'd prefer to hold out on this (+ generating specialised code for it) until we can implement it via traits...but if it helps simplify something in the meantime, we could have the specialised implementation

I have no problem waiting for a trait based implementation, if it is agreed that is best to wait.
I am implementing a Gameboy Emulator (Inspired by JT's idea of implementing a NES emu). And there is lots and lots of places where I use ranges + match cases. This is the main reason why I created this PR (make my life easier).
The alternative is that I myself can do a trait based implementation of this same feature once traits are available.

We can definitely have the feature before traits are in, no problem.


meaning of collect_episodes_with_rollouts

May I know the meaning of the following code? Thx.

Hi,

we sample further actions from the probability distribution of the policy network (line 196-200)
then get rewards for these actions from the environment (line 205-208)
and store the additionally collected data in the replay buffer (209-211)

If we only sample one action, the update would be very sparse (since we can have up to 1000 possible actions to choose from). Therefore, we "try out" 20 actions and get feedback for them for better exploration.
Best Regards,
Magdalena

Thanks.


Rebuild for R3.5

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy
[x] Ensured the license file is being packaged.

Addresses https://github.com/conda-forge/r-deriv-feedstock/issues/2 and https://github.com/conda-forge/r-deriv-feedstock/pull/3

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


[Windows] Issue, while running

What I do: 1) download develop build
2) download and install Git-<IP_ADDRESS>-64-bit.exe as this guide say
2) download and install python-3.5.1.exe (also use this guide)
3) configure option.txt
4) run as admin runbot.bat
First time I get some error and in bat and get solution with this command python -m pip install --upgrade pip and I use it.
5) run again and also see errors
Log: http://hnng.moe/f/7uj
Attempting to install dependencies...
Collecting discord.py>=0.10.0 from<EMAIL_ADDRESS>>= 0.10.0 (from -r requirements.txt (line 1))
  Cloning https://github.com/Rapptz/discord.py (to async) to c:\users\x4\appdata\local\temp\pip-build-85om_69g\discord.py
  Requested discord.py>=0.10.0 from<EMAIL_ADDRESS>>= 0.10.0 (from -r requirements.txt (line 1)), but installing version None
Collecting youtube-dl>=2016.01.01 (from -r requirements.txt (line 2))
  Using cached youtube_dl-2016.02.13-py2.py3-none-any.whl
Collecting aiohttp==0.19.0 (from discord.py>=0.10.0->-r requirements.txt (line 1))
  Using cached aiohttp-0.19.0.tar.gz
Collecting websockets==2.7 (from discord.py>=0.10.0->-r requirements.txt (line 1))
  Using cached websockets-2.7-py33.py34-none-any.whl
Requirement already satisfied (use --upgrade to upgrade): chardet in c:\users\x4\appdata\local\programs\python\python35-32\lib\site-packages (from aiohttp==0.19.0->discord.py>=0.10.0->-r requirements.txt (line 1))
Installing collected packages: aiohttp, websockets, discord.py, youtube-dl
  Running setup.py install for aiohttp ... error
Exception:
Traceback (most recent call last):
  File "C:\Users\X4\AppData\Local\Programs\Python\Python35-32\lib\site-packages\pip\compat\__init__.py", line 67, in console_to_str
    return s.decode(sys.__stdout__.encoding)
  File "C:\Users\X4\AppData\Local\Programs\Python\Python35-32\lib\encodings\cp65001.py", line 17, in decode
    return codecs.code_page_decode(65001, input, errors, True)
UnicodeDecodeError: 'CP_UTF8' codec can't decode bytes in position 0--1: No mapping for the Unicode character exists in the target code page.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\X4\AppData\Local\Programs\Python\Python35-32\lib\site-packages\pip\basecommand.py", line 209, in main
    status = self.run(options, args)
  File "C:\Users\X4\AppData\Local\Programs\Python\Python35-32\lib\site-packages\pip\commands\install.py", line 317, in run
    prefix=options.prefix_path,
  File "C:\Users\X4\AppData\Local\Programs\Python\Python35-32\lib\site-packages\pip\req\req_set.py", line 731, in install
    **kwargs
  File "C:\Users\X4\AppData\Local\Programs\Python\Python35-32\lib\site-packages\pip\req\req_install.py", line 890, in install
    spinner=spinner,
  File "C:\Users\X4\AppData\Local\Programs\Python\Python35-32\lib\site-packages\pip\utils\__init__.py", line 660, in call_subprocess
    line = console_to_str(proc.stdout.readline())
  File "C:\Users\X4\AppData\Local\Programs\Python\Python35-32\lib\site-packages\pip\compat\__init__.py", line 69, in console_to_str
    return s.decode('utf_8')
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xd1 in position 3: invalid continuation byte

You may need to run as admin to install dependencies.

E:\MusicBot-develop>

Usually when I see this its because someone set their username to something with a character like é, but I don't see one here.  This is the fix I give for this.  Try this and see if it works.
python -c "import pip, os; os.environ['PYTHONIOENCODING'] = 'utf_8'; pip.main('install --upgrade pip'.split()); pip.main('install --upgrade -r requirements.txt'.split())"

Hm, I found solution when I go on page https://github.com/Rapptz/discord.py#installing and use command
pip install git+https://github.com/Rapptz/discord.py@async
in cmd.exe.
After that when I starting runbot.bat bot starting normally:
Attempting to install dependencies...
Collecting discord.py>=0.10.0 from<EMAIL_ADDRESS>>= 0.10.0 (from -r requirements.txt (line 1))
  Cloning https://github.com/Rapptz/discord.py (to async) to c:\users\x4\appdata\local\temp\pip-build-00jgtuxg\discord.py
  Requested discord.py>=0.10.0 from<EMAIL_ADDRESS>>= 0.10.0 (from -r requirements.txt (line 1)), but installing version 0.10.0a0
Collecting youtube-dl>=2016.01.01 (from -r requirements.txt (line 2))
  Using cached youtube_dl-2016.02.13-py2.py3-none-any.whl
Requirement already satisfied (use --upgrade to upgrade): aiohttp==0.19.0 in c:\users\x4\appdata\local\programs\python\python35-32\lib\site-packages (from discord.py>=0.10.0->-r requirements.txt (line 1))
Requirement already satisfied (use --upgrade to upgrade): websockets==2.7 in c:\users\x4\appdata\local\programs\python\python35-32\lib\site-packages (from discord.py>=0.10.0->-r requirements.txt (line 1))
Requirement already satisfied (use --upgrade to upgrade): chardet in c:\users\x4\appdata\local\programs\python\python35-32\lib\site-packages (from aiohttp==0.19.0->discord.py>=0.10.0->-r requirements.txt (line 1))
Installing collected packages: discord.py, youtube-dl
  Found existing installation: discord.py 0.10.0a0
    Uninstalling discord.py-0.10.0a0:
      Successfully uninstalled discord.py-0.10.0a0
  Running setup.py install for discord.py ... done
Successfully installed discord.py-0.10.0a0 youtube-dl-2016.2.13

Ok lets hope it worked

Connected!

Username: MusicBot-X4
Bot ID:<PHONE_NUMBER>97748992
Owner ID:<PHONE_NUMBER>03130627

Command prefix is /
Whitelist check is disabled
Skip threshold at 2 votes or 50%
Now Playing message @mentions are disabled
Autosummon is enabled
Auto-playlist is enabled
Downloaded songs will be saved after playback

--Server List--
JumpJet

:ok_hand:


Isolate id_minter tests

What is this PR trying to achieve?
Part of https://github.com/wellcometrust/platform/issues/1643
Who is this change for?
🍶

Looks good, tests much improved.


Add field progress to campaign

this field displays the total progress of the number of x things y'ed
if left blank, use the total number of approved reportback quanity goes there
otherwise, use the number in the new total progress field
once a new reportback is approved, add to the progress value set in the campaign node (if it's overridden we are adding to that overridden value)

@angaither FYI that we're using dosomething_helper_variables to store the current Reportback count by status of each Campaign and Cause's. This is also really similar to #4051 which is an open issue to store current count of Signups, which suggests also using the helper variables or @mshmsh5000 mentions storing it in Redis in that thread.
Fields could work here, it would just be a different implementation from the current Reportback Status counts. I used the helper variables specifically for the Reportback stuff to avoid node_save upon each Reportback File create/update, and to avoid adding 4 fields to the Campaign node / Cause vocab.

This is also perhaps a duplicate of #3899 ?

Ah, yes. it does look like a dup of #3899, however this need to be able to potentially be override, like the totals in a campaign run.

My $0.02 is that we should keep the build simpler by simply not allowing overrides.
If that's not an option, we could always build an edit form for the Helper variables, to avoid storing all this stuff in fields.  Keeping in fields has its pros and cons but I think the biggest con would be a frequent node_save.

The case for overrides comes from campaigns with external partners, where we hear count numbers higher than than what we have in the rb_quanitity, like H&M reports we got 30K more pounds of clothes, but only 10k was reported back to us.

Ah, I see. Thanks for clarifying / documenting!  Those are the sort of scenarios where it can be easy to forget why we built something a certain way when we look back at it in the future :watch:

Closed with #4320


C++ : Say "Hello, World!" With C++

Write your solution to this C++ problem of hackerrank:-
Problem link : Say "Hello, World!" With C++

please assign also to me under hacktober tag

@kataranandini This issue has already been assigned to someone else. You may claim any other open issue or create your own issue.


BUG: Dataframe.fillna with np.nan for dtype=category(GH 14021)

[x] closes #14021
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

Current coverage is 85.25% (diff: 100%)

Merging #14051 into master will increase coverage by <.01%

@@             master     #14051   diff @@
==========================================
  Files           139        139
  Lines         50379      50381     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          42952      42954     +2
  Misses         7427       7427
  Partials          0          0

Powered by Codecov. Last update 53447f4...80aa5aa

thanks!


Removed Windows support is experimental

Removed "Windows Support is currently Experimental as of v1.21.3+rke2r1"

@cwayne18, if you have some time, can you or someone on your team review this PR?


Testing, please ignore. (mwzitq)

This is a bug bounty test. Please ignore.. Please DO NOT APPROVE THIS! (mwzitq)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jukrakuv seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


New funcionality about positions of sideshow-subject-arrow

That will be great if the sideshow-subject-arrow could be positioned per step on the top, left, right or bottom (because in some pages with elements on the top, the arrow doesnt appaears)

You can now position the arrow by using the arrowPosition property in a step.
See release v0.4.3


mbrola: fix cross compilation

Motivation for this change
Things done

Built on platform(s)

[x] x86_64-linux
[x] riscv64-linux (cross)
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

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

[x] Fits CONTRIBUTING.md.

I see it fails without this change:
...
gcc -ansi -pedantic -DLITTLE_ENDIAN -O6 -IParser -IStandalone -IMisc -ILibOneChannel -ILibMultiChannel -IEngine -IDatabase -Wall -DROMDATABASE_STORE -DROMDATABASE_INIT -DSIGNAL  -o Bin/Standalone/Standalone/synth.o -c Standalone/synth.c
/nix/store/v1d8l3zqnia3hccqd0701szhlx22g54z-bash-5.1-p8/bin/bash: line 1: gcc: command not found
make: *** [Makefile:150: Bin/Standalone/Standalone/synth.o] Error 127
builder for '/nix/store/5vlkm9r3hdyzgfkivymb3vxq5dbxa0i2-mbrola-riscv64-unknown-linux-gnu-3.3.drv' failed with exit code 2
error: build of '/nix/store/5vlkm9r3hdyzgfkivymb3vxq5dbxa0i2-mbrola-riscv64-unknown-linux-gnu-3.3.drv' failed

and works with this change:
[davidak@gaming:~/code/nixpkgs]$ nix-build . -A pkgsCross.riscv64.mbrola
...
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/bv00bp3k71q8p8gp712n83d914kq31iq-mbrola-riscv64-unknown-linux-gnu-3.3
shrinking /nix/store/bv00bp3k71q8p8gp712n83d914kq31iq-mbrola-riscv64-unknown-linux-gnu-3.3/bin/mbrola
riscv64-unknown-linux-gnu-strip is /nix/store/n4pyn69crs68zmidxhacmmsqajs982jn-riscv64-unknown-linux-gnu-stage-final-gcc-debug-wrapper-10.3.0/bin/riscv64-unknown-linux-gnu-strip
stripping (with command riscv64-unknown-linux-gnu-strip and flags -S) in /nix/store/bv00bp3k71q8p8gp712n83d914kq31iq-mbrola-riscv64-unknown-linux-gnu-3.3/bin
patching script interpreter paths in /nix/store/bv00bp3k71q8p8gp712n83d914kq31iq-mbrola-riscv64-unknown-linux-gnu-3.3
checking for references to /build/ in /nix/store/bv00bp3k71q8p8gp712n83d914kq31iq-mbrola-riscv64-unknown-linux-gnu-3.3...
/nix/store/bv00bp3k71q8p8gp712n83d914kq31iq-mbrola-riscv64-unknown-linux-gnu-3.3


FIX: copyfile()/symlinks on Linux, Matlab < R2020a

I tried running the SPM container on ds000001 which I pulled with datalad. It turns out that datalad is using symbolic links for the images in anat/ and func/ etc pointing to a .git-annex folder two levels above.
Unfortunately, copyfile() on Linux with Matlab <= 2019b does not dereference the file and when SPM is copying a relative symlink to the tmp/output folder, the link breaks.

Also, as I mention in the comment, it would probably be much nicer to keep the symbolic links for all platforms, but I'm having a hard time coming up with a portable solution for that. And I'm also not entirely sure this is compatible with SPM's behavior (would it modify source files in that case?)

There is now a function spm_copy that could be used in spm_BIDS_App.m. The issue  with symbolic links could then be dealt with there (spm_check_version might come handy) or we could just switch to the MATLAB Runtime R2020a as you mention.
It seems safer to make hard copies as sources could be modified.

Okay, cool. So should I just revert the changes and change copyfile() to spm_copy() and leave this here until spm_copy is updated?

Yes, spm_copy is just a wrapper around copyfile but it will also uncompress and move sidecar .json files as well. I will look building a new SPM12 standalone with R2020a that we could use here.


Make shebang portable; rescope script

A fast update to do two things:

Make the script portable (allow it to execute when the bash binary isn't at /bin/bash)
Break down the docker system prune -af command to only affect containers running a New Relic Synthetics image, remove only images related to Synthetics, and remove unused networks and volumes.

Script is functionally the same, but will prevent containers not running a Synthetics image from being wiped out.

@ASchneider-GitHub Thanks for this! I made some updates based on your feedback.


Issue with generating thumbnails in fillValues

I'm having an issue because I have ImageViews in my layout to display file thumbnails but when an item is deleted every single item in the list is refreshed, several times, which makes all thumbnails be created again and be
I discovered these issues when creating the thumbnails in both the generateView and fillValues. In generateView it ran fine but if an item was deleted the thumbnail would stay in place, meaning the item below(above in the list) would use the thumbnail of the deleted item, understandable seeing has I should use fillValues to fill the views with the values but here I encountered the other issue. Fill values seems to run over and over again, passing through the same positions around 4 times. I managed to avoid this when creating the thumbnails but I still have an issue.
When I create the thumbnails I need the file which either needs to be downloaded or returned if it's in cache. So I have a Map<Item, ImageView> that I use so that I can update the corresponding view when the callback returns the thumbnail but, fillvalues seems to completely destroy all instances of the previous imageviews which means all thumbnails need to be created again. This results in an obvious issue where you can see the thumbnails being generated again. This specific issue doesn't happen in generateView but generateView isn't refreshing the view so the thumbnail stays in place and the item below will keep it making the thumbnails placed incorrectly.
My Adapter(extended from BaseSwipeAdapter):
private ItemDetailContract.View itemsView;
private ItemDetailFragment.FileActionListener actionsListener;

private List<ItemFile> listFiles;
private HashMap<ItemFile, ImageView> attachedImageViews;
private boolean listingDeleted;

ItemFileAdapter(List<ItemFile> listFiles, ItemDetailFragment.FileActionListener actionsListener,
                boolean listingDeleted, ItemDetailContract.View itemsView) {
    setList(listFiles);
    this.itemsView = itemsView;
    this.actionsListener = actionsListener;
    this.listingDeleted = listingDeleted;

    this.attachedImageViews = new HashMap<>();
}

private void setList(List<ItemFile> files) {
    listFiles = files;
}

void replaceData(List<ItemFile> files) {
    setList(files);
    notifyDataSetChanged();
}

void onFileAdded(ItemFile file) {
    int pos = listFiles.indexOf(file);
    if (pos < 0) {
        listFiles.add(file);
    } else {
        listFiles.remove(pos);
        listFiles.add(pos, file);
    }
    notifyDataSetChanged();
}

void onFileRemoved(ItemFile deletedFile) {
    listFiles.remove(deletedFile);
    // all views seem to be different when fillvalues re-runs
    // so deleting only one key,value pair makes no difference
    attachedImageViews.clear();
    notifyDataSetChanged();
}

@Override
public int getSwipeLayoutResourceId(int position) {
    return R.id.bottom_layout_actions;
}

@Override
public View generateView(final int position, ViewGroup parent) {
    Context context = parent.getContext();
    View itemView = LayoutInflater.from(context).inflate(R.layout.list_file_item, null);

    SwipeLayout swipeLayout = (SwipeLayout) itemView.findViewById(R.id.bottom_layout_actions);
    swipeLayout.setShowMode(SwipeLayout.ShowMode.LayDown);
    swipeLayout.setClickToClose(true);
    ...
    // set icons and listeners
    ...

    /*ImageView thumbView = (ImageView) itemView.findViewById(R.id.file_thumbnail);
    thumbView.setImageDrawable(ContextCompat.getDrawable(itemView.getContext(), R.drawable.file_placeholder));
    attachImageViewToFile((ItemFile) getItem(position), thumbView);*/

    return itemView;
}

@Override
public void fillValues(int position, View convertView) {
    ItemFile file = (ItemFile) getItem(position);

    TextView filename = (TextView) convertView.findViewById(R.id.filename);
    filename.setText(file.getFilename());

    ImageView thumbView = (ImageView) convertView.findViewById(R.id.file_thumbnail);
    // avoid being caught in fillvalues loop
    if (!attachedImageViews.containsValue(thumbView)) {
        thumbView.setImageDrawable(ContextCompat.getDrawable(convertView.getContext(), R.drawable.file_placeholder));
        attachImageViewToFile(file, thumbView);
    }
}

@Override
public Object getItem(int position) {
    return listFiles.get(position);
}

@Override
public int getCount() {
    return listFiles.size();
}

@Override
public long getItemId(int position) {
    return position;
}

void setThumbnail(ItemFile file, File thumbnail) {
    ImageView imageView = attachedImageViews.get(file);
    Log.d("THUMB", "Creating thumbnail... " + thumbnail.getAbsolutePath());
    Picasso.with(imageView.getContext())
            .load(thumbnail)
            .resize(100, 100)
            .noFade()
            .placeholder(R.drawable.file_placeholder)
            .error(R.drawable.file_placeholder)
            .memoryPolicy(MemoryPolicy.NO_CACHE, MemoryPolicy.NO_STORE)
            .into(imageView);
}

private void attachImageViewToFile(ItemFile file, ImageView thumbView) {
    attachedImageViews.put(file, thumbView);
    itemsView.requestThumbnail(file);
}

Also, another weird issue. I might think that in the fillValues I should use
if (!attachedImageViews.containsKey(file))

instead of
if (!attachedImageViews.containsValue(thumbView))

and you'd be right but what happens in that case is all thumbnails are only using the first ImageView so not only are all the ImageViews incorrect but only the first oitem will actually have one.

I've fixed the issue but for future reference the way I fixed was with this code in fillValues:
public void fillValues(int position, View convertView) {
    ItemFile file = (ItemFile) getItem(position);

    TextView filename = (TextView) convertView.findViewById(R.id.filename);
    filename.setText(file.getFilename());

    ImageView thumbView = (ImageView) convertView.findViewById(R.id.file_thumbnail);
    if (!attachedImageViews.containsValue(thumbView)) {
        System.out.println(new ArrayList<>(attachedImageViews.keySet()));
        ImageView currentFilePreviousThumb = attachedImageViews.get(file);
        if (currentFilePreviousThumb != null && thumbView != currentFilePreviousThumb) {
            thumbView.setImageDrawable(currentFilePreviousThumb.getDrawable());
            attachedImageViews.put(file, thumbView);
        } else {
            //should only reach here on the first run
            attachImageViewToFile(file, thumbView);
        }
    }
}

This code though, still didn't fix the issue completely because it seems that Picasso was not correctly setting certain images as Drawables, so in those cases the placeholder would be displayed instead. So I just removed Picasso(didn't really needed it in this situation because I don't need to cache or transformations) and just used:
imageView.setImageDrawable(Drawable.createFromPath(thumbnail.getPath()));


fix(v2): isInternal targetLink validation

Motivation
The isInternal const received a function instead of a boolean value. So, all external links were incorrect. Ex: Github link in the navigation bar and page footer;
Have you read the Contributing Guidelines on pull requests?
Yes, I have read the Contributing Guidelines.
Test Plan
Check the redirection of external links in the navigation bar and page footer..

@yangshun, you're welcome! It is a pleasure for me to be able to contribute to such an important project. (:


In calculate_registration_image_based, should we check that zarr_url != reference_zarr_url?

It's mostly the job of the init function to not provide the reference_zarr_url as one of the parallelization items. If it does, nothing would break though (it's just a trivial registration that gets calculated).
We can add a check. In the past, we had something there to just skip the task if that was provided. Not sure whether this is still the most relevant


feature: add option hotarea_pos and hotarea_monitor

Add two new config options: hotarea_pos and hotarea_monitor.
Currently hycov only support triggering hot area at the bottom left corner of all monitors. But users may want to change this to top right etc., and users with multiple monitors may want to use hot area on only one monitor.
This pr implements these two options.
hotarea_pos = 1 # 1: top left, 2: top right, 3: bottom left, 4: bottom right
hotarea_monitor = HDMI-A-1 # default is all

TODOs:

[ ] update  readme
[ ] be able to set multiple hotarea_monitor

thank you for your pr!
But there are a few minor issues that need to be optimized,
1. This variable is defined, but not used
   ![image](https://private-user-images.githubusercontent.com/30348075/297706598-b6fa65c6-aeba-41f1-8af5-d6c3f18fbfcc.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MDU1ODA2NTIsIm5iZiI6MTcwNTU4MDM1MiwicGF0aCI6Ii8zMDM0ODA3NS8yOTc3MDY1OTgtYjZmYTY1YzYtYWViYS00MWYxLThhZjUtZDZjM2YxOGZiZmNjLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDAxMTglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwMTE4VDEyMTkxMlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTE3MGRhMjc4YmU5ODIzNTc5ODg5NGQ0MmUwOWYzMTQyNTA3OGY3YWRhNDJjNzIzMGI2Nzk5NmMzM2VkOGJhMGImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.2RnFiAqKFKrMtq9KyUCjHpp0ytkDMDyfx_lyvCJjF00)

2. In order to be compatible with past user habits, the default hotarea  should be set  to 3(bottomLeft).

These have been optimized by the latest commit.
I changed the numbering of hotarea_pos to 1: bottom left, 2: bottom right, 3: top left, 4: top right since it makes more sense to set the default value to 1..


[C++] Remove serial dependency on jemalloc

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@github-actions crossbow submit -g cpp -g wheel

@kou @assignUser This works for me locally to remove the blocking build step on jemalloc.
Without this PR, jemalloc first waits configuring and building, then Arrow C++ sources start building.
With this PR, Arrow C++ sources start building immediately.

Ok, it doesn't work for other configurations. I don't see any simple way out of this, and our CMake setup is really messy here.

I think it comes down to this: setting either ARROW_STATIC_LINK_LIBS, ARROW_SHARED_LINK_LIBS or ARROW_SHARED_PRIVATE_LINK_LIBS introduces a dependency for all Arrow C++ source files on these libraries.
A global solution might be to define separate OBJECT libraries for every private dependency (such jemalloc but also AWS, etc.).

I tried the following:

diff --git a/cpp/CMakeLists.txt b/cpp/CMakeLists.txt
index d26e06a146..f15eb64f85 100644
--- a/cpp/CMakeLists.txt
+++ b/cpp/CMakeLists.txt
@@ -942,15 +942,8 @@ if(ARROW_BUILD_BENCHMARKS)
   endif()
 endif()

-if(ARROW_JEMALLOC)
-  list(APPEND ARROW_SHARED_LINK_LIBS jemalloc::jemalloc)
-  list(APPEND ARROW_STATIC_LINK_LIBS jemalloc::jemalloc)
-endif()
-
 if(ARROW_MIMALLOC)
   add_definitions(-DARROW_MIMALLOC)
-  list(APPEND ARROW_SHARED_LINK_LIBS mimalloc::mimalloc)
-  list(APPEND ARROW_STATIC_LINK_LIBS mimalloc::mimalloc)
 endif()

 # ----------------------------------------------------------------------
diff --git a/cpp/cmake_modules/ThirdpartyToolchain.cmake b/cpp/cmake_modules/ThirdpartyToolchain.cmake
index a2627c190f..4cbcdaa505 100644
--- a/cpp/cmake_modules/ThirdpartyToolchain.cmake
+++ b/cpp/cmake_modules/ThirdpartyToolchain.cmake
@@ -2143,7 +2143,6 @@ if(ARROW_MIMALLOC)
     target_link_libraries(mimalloc::mimalloc INTERFACE "bcrypt.lib" "psapi.lib")
   endif()
   add_dependencies(mimalloc::mimalloc mimalloc_ep)
-  add_dependencies(toolchain mimalloc_ep)

   list(APPEND ARROW_BUNDLED_STATIC_LIBS mimalloc::mimalloc)

diff --git a/cpp/src/arrow/CMakeLists.txt b/cpp/src/arrow/CMakeLists.txt
index c1fafeebc0..2263dc1a31 100644
--- a/cpp/src/arrow/CMakeLists.txt
+++ b/cpp/src/arrow/CMakeLists.txt
@@ -145,27 +145,6 @@ macro(append_runtime_avx512_src SRC)
 endmacro()

 set(ARROW_SRCS
-    array/array_base.cc
-    array/array_binary.cc
-    array/array_decimal.cc
-    array/array_dict.cc
-    array/array_nested.cc
-    array/array_primitive.cc
-    array/array_run_end.cc
-    array/builder_adaptive.cc
-    array/builder_base.cc
-    array/builder_binary.cc
-    array/builder_decimal.cc
-    array/builder_dict.cc
-    array/builder_run_end.cc
-    array/builder_nested.cc
-    array/builder_primitive.cc
-    array/builder_union.cc
-    array/concatenate.cc
-    array/data.cc
-    array/diff.cc
-    array/util.cc
-    array/validate.cc
     builder.cc
     buffer.cc
     chunked_array.cc
@@ -175,7 +154,6 @@ set(ARROW_SRCS
     datum.cc
     device.cc
     extension_type.cc
-    memory_pool.cc
     pretty_print.cc
     record_batch.cc
     result.cc
@@ -261,11 +239,42 @@ set(ARROW_SRCS
     vendored/double-conversion/string-to-double.cc
     vendored/double-conversion/strtod.cc)

+add_library(arrow_array OBJECT
+    array/array_base.cc
+    array/array_binary.cc
+    array/array_decimal.cc
+    array/array_dict.cc
+    array/array_nested.cc
+    array/array_primitive.cc
+    array/array_run_end.cc
+    array/builder_adaptive.cc
+    array/builder_base.cc
+    array/builder_binary.cc
+    array/builder_decimal.cc
+    array/builder_dict.cc
+    array/builder_run_end.cc
+    array/builder_nested.cc
+    array/builder_primitive.cc
+    array/builder_union.cc
+    array/concatenate.cc
+    array/data.cc
+    array/diff.cc
+    array/util.cc
+    array/validate.cc)
+
+set(ARROW_MEMORY_POOL_SRCS memory_pool.cc)
 if(ARROW_JEMALLOC)
-  list(APPEND ARROW_SRCS memory_pool_jemalloc.cc)
+  list(APPEND ARROW_MEMORY_POOL_SRCS memory_pool_jemalloc.cc)
   set_source_files_properties(memory_pool_jemalloc.cc
                               PROPERTIES SKIP_UNITY_BUILD_INCLUSION ON)
 endif()
+add_library(arrow_memory_pool OBJECT ${ARROW_MEMORY_POOL_SRCS})
+if(ARROW_JEMALLOC)
+  target_link_libraries(arrow_memory_pool jemalloc::jemalloc)
+endif()
+if(ARROW_MIMALLOC)
+  target_link_libraries(arrow_memory_pool mimalloc::mimalloc)
+endif()

 append_runtime_avx2_src(util/bpacking_avx2.cc)
 append_runtime_avx512_src(util/bpacking_avx512.cc)
@@ -350,32 +359,6 @@ set(ARROW_TESTING_SRCS
     testing/generator.cc
     testing/util.cc)

-# Add dependencies for third-party allocators.
-# If possible we only want memory_pool.cc to wait for allocators to finish building,
-# but that only works with Ninja
-# (see https://gitlab.kitware.com/cmake/cmake/issues/19677)
-
-set(_allocator_dependencies "") # Empty list
-if(jemalloc_VENDORED)
-  list(APPEND _allocator_dependencies jemalloc_ep)
-endif()
-if(mimalloc_VENDORED)
-  list(APPEND _allocator_dependencies mimalloc_ep)
-endif()
-
-if(_allocator_dependencies)
-  if("${CMAKE_GENERATOR}" STREQUAL "Ninja")
-    set_source_files_properties(memory_pool.cc PROPERTIES OBJECT_DEPENDS
-                                                          "${_allocator_dependencies}")
-  else()
-    add_dependencies(arrow_dependencies ${_allocator_dependencies})
-  endif()
-  set_source_files_properties(memory_pool.cc PROPERTIES SKIP_PRECOMPILE_HEADERS ON
-                                                        SKIP_UNITY_BUILD_INCLUSION ON)
-endif()
-
-unset(_allocator_dependencies)
-
 if(CMAKE_CXX_COMPILER_ID STREQUAL "AppleClang" OR CMAKE_CXX_COMPILER_ID STREQUAL "Clang")
   set_property(SOURCE util/io_util.cc
                APPEND_STRING
@@ -666,8 +649,12 @@ add_arrow_lib(arrow
               ${ARROW_SHARED_LINK_LIBS}
               SHARED_PRIVATE_LINK_LIBS
               ${ARROW_SHARED_PRIVATE_LINK_LIBS}
+              arrow_array
+              arrow_memory_pool
               STATIC_LINK_LIBS
               ${ARROW_STATIC_LINK_LIBS}
+              arrow_array
+              arrow_memory_pool
               STATIC_INSTALL_INTERFACE_LIBS
               ${ARROW_STATIC_INSTALL_INTERFACE_LIBS}
               SHARED_INSTALL_INTERFACE_LIBS

We can build cpp/src/arrow/array/*.cc before jemalloc with it. If we split more files to separated OBJECT libraries, we will be able to build them before jemalloc.

It's funny because I tried something similar (a arrow_memory_pool object library) but I bailed out after I failed making it work with the conda-cpp Docker build. Apparently reusing an objlib into a static library is not easy to achieve.

It's funny because I tried something similar (a arrow_memory_pool object library) but I bailed out after I failed making it work with the conda-cpp Docker build. Apparently reusing an objlib into a static library is not easy to achieve.

Just out of curiosity, what went wrong with the objlib? I want to help on this.

The linker couldn't find some jemalloc symbols. I was not able to understand the reason and I don't remember the details, sorry.
Le 26 janvier 2024 11:37:43 GMT+01:00, Rossi Sun @.***> a écrit :

It's funny because I tried something similar (a arrow_memory_pool object library) but I bailed out after I failed making it work with the conda-cpp Docker build. Apparently reusing an objlib into a static library is not easy to achieve.

Just out of curiosity, what went wrong with the objlib? I want to help on this.
--
Reply to this email directly or view it on GitHub:
https://github.com/apache/arrow/pull/39559#issuecomment-1911832930
You are receiving this because you authored the thread.
Message ID: @.***>

The linker couldn't find some jemalloc symbols. I was not able to understand the reason and I don't remember the details, sorry. Le 26 janvier 2024 11:37:43 GMT+01:00, Rossi Sun @.***> a écrit :
…

It's funny because I tried something similar (a arrow_memory_pool object library) but I bailed out after I failed making it work with the conda-cpp Docker build. Apparently reusing an objlib into a static library is not easy to achieve. Just out of curiosity, what went wrong with the objlib? I want to help on this. -- Reply to this email directly or view it on GitHub: #39559 (comment) You are receiving this because you authored the thread. Message ID: @.***>

No problem.

#39824

Closing this in favor of Kou's PR #39824.


go-ethereum: 1.10.23 -> 1.10.25

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/go-ethereum/versions.
meta.description for go-ethereum is: "Official golang implementation of the Ethereum protocol"
meta.homepage for go-ethereum is: "https://geth.ethereum.org/"
meta.changelog for go-ethereum is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 1.10.24 with grep in /nix/store/i3xighaii71ls9gf9ybl4y91gsj187b9-go-ethereum-1.10.24
found 1.10.24 in filename of file in /nix/store/i3xighaii71ls9gf9ybl4y91gsj187b9-go-ethereum-1.10.24

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
bee-clef
go-ethereum

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/i3xighaii71ls9gf9ybl4y91gsj187b9-go-ethereum-1.10.24 \
  --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 go-ethereum https://github.com/r-ryantm/nixpkgs/archive/2ce17b5b8f34cec35439c4496330913b34ddcf23.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/i3xighaii71ls9gf9ybl4y91gsj187b9-go-ethereum-1.10.24
ls -la /nix/store/i3xighaii71ls9gf9ybl4y91gsj187b9-go-ethereum-1.10.24/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

2 packages built:

bee-clef
go-ethereum

Maintainer pings
cc @adisbladis @RaghavSood for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/go-ethereum/versions.
meta.description for go-ethereum is: "Official golang implementation of the Ethereum protocol"
meta.homepage for go-ethereum is: "https://geth.ethereum.org/"
meta.changelog for go-ethereum is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 1.10.25 with grep in /nix/store/vsfxga9mfzgh7w351hvm4cy61jfqz49l-go-ethereum-1.10.25
found 1.10.25 in filename of file in /nix/store/vsfxga9mfzgh7w351hvm4cy61jfqz49l-go-ethereum-1.10.25

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
bee-clef
go-ethereum

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vsfxga9mfzgh7w351hvm4cy61jfqz49l-go-ethereum-1.10.25 \
  --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 go-ethereum https://github.com/r-ryantm/nixpkgs/archive/46744932370efdd87648c44bbfbcf72049c8d96d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vsfxga9mfzgh7w351hvm4cy61jfqz49l-go-ethereum-1.10.25
ls -la /nix/store/vsfxga9mfzgh7w351hvm4cy61jfqz49l-go-ethereum-1.10.25/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

2 packages built:

bee-clef
go-ethereum

Maintainer pings
cc @adisbladis @RaghavSood for testing.


Tab Reloading

It's me again! Another feature request if it doesn't already exist haha.
Is there currently a way to reload a view in a tab? What Im after is either being able to tell a tab to reload its data or when the tab is loaded each time fetch the data it needs. Does something like this exist that I can use? If not what would be the best route to implement something like this?

The way the tabs work is it creates a document per tab. If the document exists then we don't load it again.
https://github.com/toshi0383/TVMLKitchen/blob/swift2.2/Sources/kitchen.js#L50-L58
We could possibly add a new method like: Kitchen.reload() which would use navigationDocument.replaceDocument method to reload that current view.

That would work. Currently if data changes on an API there is no way or refreshing anything.

I would love to have this feature, I'm trying to figure out a work around right now to fix this. Because if a user updates something within their Defaults there is no way of viewing that back. I can take a look at this and submit a PR if you would like.

Just submitted a prototype in #90.
What do you guys think ?

Merged.
As you can see in the sample code here, reloading the tab before selected solves "hitching animation" issue.
It still seems to be a workaround, but it works fine this way I guess.
Let us know any further feature requests or bug reports !

Awesome!

Released ! 🚢


feat(TriangulatedSurface): add smesh input

:tada: This PR is included in version 5.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Example: Using polygon with agents

Fixes https://github.com/gizatechxyz/actions-sdk/issues/33
I have added an ipynb notebook in example/agents/ folder. This follows the same method as that of Arbitrum.

Hi @silent-cipher
I've read the tutorial and it looks pretty good!
Once I get some time I will run it with my own RPC node and check that it works as expected.

I have went through all the steps and got the results. But, it will be great if you go through this once.

@Gonmeso, If you get time please review this?

Hi @silent-cipher I will review it by EOD

Working as expected! Great job and thanks to the contribution. LGTM


Updates to add special keys to mouse/filedrop events

Adds data about: ctrlKey, altKey, shiftKey, guiKey to mouse events.
Updates Keyboard data to have guiKey
Fixes Scroll view to have horizontal scroll
Update ScrollView to Handle horizontal scrool with Shift + MouseWheel.
Fixed state when was no scroll bars but bounce was working

@bryphe Updated with your comments!

One other note - the controls we use in Onivim are actually special (not based on this scrollable component, unfortunately)
I know that, but I needed somehow to test and showcase that special keys actually pass through, ScrollView was the easiest one :smiley:


Editing projects prefix mappings errors

when you are on nexus web on the page
https://example.com/{org} you see a list of projects for a selected organization.
_Edit -> Add API mapping (you fill a prefix and a uri) -> Save _
Then you click again in Edit, and the prefix mapping you already added is gone.
What's more, if you try to add a new or the same prefix mapping and click save again, that will output an error.

Looks like we don't refresh after the update, so we have the previous state after adding the prefix mapping. That's why we get an error if we try to add it again without refreshing the page.


Use the extension system

Summary
All the providers are already built-in inside the app which makes the app really easy to setup but it increases the chances of a DMCA strike like the one on cloudstream, so i recommend that you use the extension system or a similiar way to add functionality to the app.
Please confirm the following

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.

Man ripped sorastream from Cloudstream

now it just needs to release on PC and we're doomed

GitHub is a DMCA takedown goldmine. An extension system seems to be the only workaround. Personally, if I were building apps that could get taken down I'd self-host instead.

@KillerDogeEmpire, @IndusAryan

Man ripped sorastream from Cloudstream

I know you 2 are CloudStream devs/contributors and it looks like you're not happy about this. Stantanasi is only fulfilling user requests like any good dev would.
Using SoraStream as a provider is an issue for you guys?

now it just needs to release on PC and we're doomed

Nah, Android emulator on PC should suffice. I haven't tested that yet but I will now.

@KillerDogeEmpire, @IndusAryan

Man ripped sorastream from Cloudstream

I know you 2 are CloudStream devs/contributors and it looks like you're not happy about this. Stantanasi is only fulfilling user requests like any good dev would.
Using SoraStream as a provider is an issue for you guys?

not at all(idk about others but atleast not me), the main devs planned to make the cloudstream api multi-platform themselves for some time so nothing to be angry about and its not like the devs make money off of cloudstream.

now it just needs to release on PC and we're doomed

Nah, Android emulator on PC should suffice. I haven't tested that yet but I will now.

an emulator is the problem, native support is what i am talking about. Emulators are slow, clunky and full of trackers. WSA is dead which we initially recommended to use.

not at all(idk about others but atleast not me), the main devs planned to make the cloudstream api multi-platform themselves for some time so nothing to be angry about and its not like the devs make money off of cloudstream.

Ok, that's what I thought. I also thought SoraStream was open source so that's why I was curious. I would of been confused if this caused a conflict. There's more apps that would benefit using this provider but I wasn't sure if I should say anything.

an emulator is the problem, native support is what i am talking about. Emulators are slow, clunky and full of trackers. WSA is dead which we initially recommended to use.

This is very true. I'm pissed off WSA is falling off the map, I really like it a lot.

not at all(idk about others but atleast not me), the main devs planned to make the cloudstream api multi-platform themselves for some time so nothing to be angry about and its not like the devs make money off of cloudstream.

Ok, that's what I thought. I also thought SoraStream was open source so that's why I was curious. I would of been confused if this caused a conflict. There's more apps that would benefit using this provider but I wasn't sure if I should say anything.

@DarkCrypt
I'm not really upset, just think that if many apps use it, the api's that it uses will notice and prevent it from working. Just think if people do use it they should hide the api's but they cam still be found with decompiler.

@KillerDogeEmpire
I was hoping you'd reply, thank you for doing so. That clears it up for me and also makes sense. I don't want bad things to happen either. I'll keep this provider to myself for said reasons.
Just my opinion, between CloudStream and Streamflix, there's really no need to use other apps anyway. I really respect and appreciate all participating devs with both projects. God bless you all friends!

@DarkCrypt apps are bound to come and people will use them, just take music player or note taking apps for examples.
StreamFlix or cloudstream neither of them is perfect, an app offering even a single of those lacking feature would be used by someone for their needs, some examples would be PC support, chromecast, torrenting, built-in vpn(i bet you it will be a thing), better UI,etc etc
Most people will just use what they come across first and wouldn't make the effort to switch unless they are facing some issue.

@DarkCrypt not problem, neither we are owners of sorastream, but i would suggest to use a different names for extensions if copied from cs3, like make sorastream -> turbostream, because this app comes with built in extensions.

Closing this issue because it's not possible to use the extension system in my app


[FEATURE] Add USA zip code as part of default vector map in region maps

Is your feature request related to a problem?
I would like to visualize aggregated data based on USA Zip codes on Region maps.
What solution would you like?
I should be able to choose USA Zip codes from Default Vector map to join zip code from my index pattern.
What alternatives have you considered?
I used custom geojson to upload USA Zip code GeoJSON, but upload API can only support up to 10 k Features.
Do you have any additional context?
https://forum.opensearch.org/t/opensearch-dashboards-2-3-0-region-maps-not-working/11967/32

+1 on the issue.
I have a use case with USA zip codes for region maps and stuck on how to proceed further.
Could you please give an ETA when this issue would be resolved


Support for complex tenacy rules

It would be nice to have support to more complex multi-tenancy rules. Most of my models work nicely with tenant_id, but one table has additional is_public flag. So record can be accessed by user.id==record.tenant_id OR record.is_public==TRUE.

Could you not just chain a eloquent query scope for this?

+1


Maximum call stack size exceeded in template engine

Steps to reproduce:

Enable static files introspection
Open a directory with large number of files (in our case, 1418) in browser

Fixed in metasync: https://github.com/metarhia/MetaSync/issues/35


WIP: Implement the test result attribute

What to do with this test?
/wrong-result:
    summary: Bad matadata
    test: 'true'
    result: not-valid-result

IIRC (but I can't find any issue) it was proposed that original result should be stored as well.

Yes, I think the proposal was to mention the original value in the note field.

IIRC (but I can't find any issue) it was proposed that original result should be stored as well.

Yes, I think the proposal was to mention the original value in the note field.

Agreed. I'll will finish it in the text commit.

I squashed commits to one. Ready to final review and merge. Thanks!

@ZelenyMartin Don't you want to reset author to yourself? (and probably date as well).
Current author is psss on Thu Dec 16 21:36:21 2021
Hint: git commit --amend --reset-author --no-edit (yes, I had to google it last week)


Track statistics for user access

We can track statistics on user access both at the website level and at Globus level.
I'll investigate the 2 and write more details here.

FYI: At the website level just be cautious about privacy concerns. Since the site is statically hosted on GitHub servers the obvious solution is to use a JavaScript tracking or analytics package. As part of another project we ensured that a similar mechanism was configured to capture only a minimal amount of information and we clearly disclosed what was being captured and why. That helps to address many privacy concerns.


Update default cache directory to XDG_CACHE_HOME

Updates default behavior to use XDG cache spec
Updated annotations in default config
TODO:

[x] Update TTS caching to use /tmp by default https://github.com/OpenVoiceOS/OVOS-plugin-manager/blob/dev/ovos_plugin_manager/utils/tts_cache.py#L225

[ ]  Update TTS caching to use /tmp by default https://github.com/OpenVoiceOS/OVOS-plugin-manager/blob/dev/ovos_plugin_manager/utils/tts_cache.py#L225

It appears the method referenced here is from ovos_utils and already references /tmp

companion PR https://github.com/OpenVoiceOS/OVOS-plugin-manager/pull/66


更改 nginx端口号 然后克隆大文件克隆不下来？

git lfs clone<EMAIL_ADDRESS>WARNING: git lfs clone is deprecated and will not be updated
with new flags from git clone
git clone has been updated in upstream Git to have comparable
speeds to git lfs clone.
Cloning into 'image_restore'...
remote: Enumerating objects: 37, done.
remote: Counting objects: 100% (37/37), done.
remote: Compressing objects: 100% (35/35), done.
remote: Total 37 (delta 14), reused 0 (delta 0), pack-reused 0
Receiving objects: 100% (37/37), 6.51 KiB | 3.26 MiB/s, done.
Resolving deltas: 100% (14/14), done.
LFS: Get "http://<IP_ADDRESS>/gitserver/models_w8156481/image_restore.git/info/lfs/objects/a319fc995fed963a93c43dba86f93f29b7a2d0a949f7b9dfd2e268ccae77384d": dial tcp <IP_ADDRESS>:80: connectex: No connection could be made because the target machine actively refused it.
LFS: Get "http://<IP_ADDRESS>/gitserver/models_w8156481/image_restore.git/info/lfs/objects/1001fb2538c3eab5602b7c8fae28959cfe257c1eff295fa1296f26b1b2d3e87e": dial tcp <IP_ADDRESS>:80: connectex: No connection could be made because the target machine actively refused it.
LFS: Get "http://<IP_ADDRESS>/gitserver/models_w8156481/image_restore.git/info/lfs/objects/cb8e5d038151192f423d894d08f722921706d4343036e7a6331097c9132b96ca": dial tcp <IP_ADDRESS>:80: connectex: No connection could be made because the target machine actively refused it.
LFS: Get "http://<IP_ADDRESS>/gitserver/models_w8156481/image_restore.git/info/lfs/objects/e12afa8060ee60e6f4be68b63cd337b6307073683e7ee6e5bed646c8aa85d0b6": dial tcp <IP_ADDRESS>:80: connectex: No connection could be made because the target machine actively refused it.
LFS: Get "http://<IP_ADDRESS>/gitserver/models_w8156481/image_restore.git/info/lfs/objects/63efe999992832f809ec667e677f483d89cbc8796ba50d8a1a7dd30c442fbc5c": dial tcp <IP_ADDRESS>:80: connectex: No connection could be made because the target machine actively refused it.
Failed to fetch some objects from<EMAIL_ADDRESS>PS C:\Users\SunCh\Desktop>

请上传一下，您的安装后的nginx的配置内容和每个服务所使用的端口号，以便于我们判断。


Q: Does Parcelvoy merge user attributes across multiple calls to the /identify API?

What happens if I call /identify twice, using the same external_id but different data attributes for my user? Will they add up in the same user entry in the database? (At least that is what I would expect).

Correct, however it does a soft merge replacing existing values with new ones. Take that into account when passing up objects for example or arrays. Two arrays wont be merged together, the first one will be replaced by the subsequent one


Add CODEOWNERS file

As per our conversation today, this PR adds a CODEOWNERS file to our github
configuration. This provides a mechanism whereby anyone can volunteer to curate
some subset of the sourcecode. Curating some code just means you'll be tagged
for reviews of that code automatically when PRs are opened introducing changes
to it.
Anyone from the @informalsystems/apalache team who wants to should feel
free to add themselves to the parts of the code they want to care for on
this branch. Of course, then can also add themselves in followup changes
as needed too.
I also added myself as a curator for the stuff that seemed most obvious to me
given the focus of my work so far, but this is mostly meant as an example
of the syntax.

Codecov Report

Merging #1618 (bb5fed9) into unstable (12e8a09) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           unstable    #1618   +/-   ##
=========================================
  Coverage     75.12%   75.12%
=========================================
  Files           356      356
  Lines         11555    11555
  Branches        541      541
=========================================
  Hits           8681     8681
  Misses         2874     2874

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 12e8a09...bb5fed9. Read the comment docs.

I’m gonna set this for merge now. Anyone else can add themselves as they wish in follow up prs.


Automatic instrumention for custom modules/classes

Is your feature request related to a problem? Please describe.
Currently using APM with Django apps and @capture_span has been incredibly helpful in finding bottlenecks and optimizing performance. However when new functionality is added, its tedious for the developer to remember to apply the decorator everywhere.
While explicit is better than implicit, explicit also increases the likelihood that it's missed or forgotten, and at times (of which this may be one) implicit can be preferred when something useful comes for free in future development.
Describe the solution you'd like
An option to auto-instrument custom written modules and classes with @capture_span in a Django application would be a killer feature. Ideally this could just be a settings flag or included module. Additional useful configuration would be ignoring certain files, modules, or directories; for example for ignoring test code or unimportant-to-trace framework files like manage.py in Django.
Describe alternatives you've considered
We've looked into a couple different ways of solving this with code, such as programatically adding decorator to all class functions or module functions.
We've also considered using CI validation to validate on pull requests that the decorator was applied to new functionality, which would follow "explicit is better than implicit" more closely. This also might spawn a python module to check this, as experiments validating with a multi-line grep have been...hairy.
Additional context
As a final note, an argument could be made (although I'm sure someone will correct me) that monitoring is more a deployment concern than an application concern, and so more comprehensive automatic instrumentation without explicit references in source code could be appropriate.

Thanks for the request! This has been an idea we've been throwing around for awhile (the Java agent supports something like this), but it's not on our immediate roadmap. But now we have an issue for tracking it, so thank you! :)

Note to self: setuptools entry_points could be useful here.


Allow configuraiton of the full filepath to the SSH key

In the orchestration scenario the ssh key is mounted on the filesystem
from a [kubernetes] secret. This location needs to reside outside of
the mounted data volume, because a mount can't contain a mount.
This change allows the current practice to continue working when
GIT_SSH_KEY_FILEPATH is undefined.
A problem would arrise if both GIT_SSH_KEY and
GIT_SSH_KEY_FILEPATH are supplied. But if the
permissions are set correctly to read-only on the file at
GIT_SSH_KEY_FILEPATH, the operation should fail with an OSError.

The one failing integration test is bogus. We could rerun the tests, but for a one line change, I'm just going to ask that you ignore the failure.


User authentication in Jitsi Meet using JWTs

The lack of user authentication functionality in Jitsi Meet makes it virtually impossible to use the internal Jitsi Meet server. No reasonable person in a company would run a Jitsi Meet video meeting server without an authorization feature. It would be nice to have such a feature.

@zulipbot claim

@PratikGoud can i work on it?

up

up


Add retry logic on aws eks wait cluster-active to /etc/eks/bootstrap.sh

What would you like to be added:
I get error Waiter ClusterActive failed: Too Many Requests when launch cluster with 400 Node, only 256 Node registered with the master and no more
Why is this needed:
Adding logic to retry on aws eks wait cluster-active the same way we did with aws eks describe-cluster

As a possible workaround, you could pass the API Server endpoint and the Cluster certificate data to the bootstrap script and avoid the EKS API calls

Any updates on this? Would be pretty useful.

The best solution here is to avoid the eks:DescribeCluster call entirely by passing the cluster CA and endpoint to the bootstrap script.


question about edge feature encoding

I notice that in graphormer/modules/graphormer_layers.py you modify the spatial_pos_ by setting 1 to 1, x > 1 to x - 1, could you please tell me the reason?
    if self.edge_type == "multi_hop":
        spatial_pos_ = spatial_pos.clone()
        spatial_pos_[spatial_pos_ == 0] = 1  # set pad to 1
        # set 1 to 1, x > 1 to x - 1
        spatial_pos_ = torch.where(spatial_pos_ > 1, spatial_pos_ - 1, spatial_pos_)

Hi, sorry for the late response. (Other authors have been too busy to answer)

About setting x > 1 to x - 1: in function pad_spatial_pos_unsqueeze() in collator.py because we increment each element in spatial_pos by 1, and add zero padding. So later we set x > 1 to x - 1 to restore spatial_pos to real shortest distance between nodes.
About setting 1 to 1: we calculate an average value of edge embedding along the shortest path just a bit after your code snippet. If we set 1 to 0, there'll be division by zero, so we set 1 to 1.

edge_input = (
                edge_input.sum(-2) / (spatial_pos_.float().unsqueeze(-1))
            ).permute(0, 3, 1, 2)

I notice that in graphormer/modules/graphormer_layers.py you modify the spatial_pos_ by setting 1 to 1, x > 1 to x - 1, could you please tell me the reason?
    if self.edge_type == "multi_hop":
        spatial_pos_ = spatial_pos.clone()
        spatial_pos_[spatial_pos_ == 0] = 1  # set pad to 1
        # set 1 to 1, x > 1 to x - 1
        spatial_pos_ = torch.where(spatial_pos_ > 1, spatial_pos_ - 1, spatial_pos_)

Hi, sorry for the late response. (Other team members have been just too busy to answer!)
About setting x > 1 to x - 1: in function pad_spatial_pos_unsqueeze() in collator.py because we increment each element in spatial_pos by 1, and add zero padding. So later we set x > 1 to x - 1 to restore spatial_pos to real shortest distance between nodes.
About setting 1 to 1: we calculate an average value of edge embedding along the shortest path just a bit after your code snippet. If we set 1 to 0, there'll be division by zero, so we set 1 to 1.
edge_input = (
edge_input.sum(-2) / (spatial_pos_.float().unsqueeze(-1))
).permute(0, 3, 1, 2)

Thanks for you answering : )


Push notifications with Firebase

❓ Context

Impacted projects: live-mobile

Linked resource(s): N/A

Add push notifications capabilities to the mobile app through the magic of Firebase. The notification permission pop-up on iOS is feature flagged so this can reach production without impact.
✅ Checklist

[ ] Test coverage: Did you write any tests to cover the changes introduced by this pull request?
[ ] Atomic delivery: Is this pull request standalone? In order words, does it depend on nothing else?
[ ] Breaking changes: Does this pull request contain breaking changes of any kind? If so, please explain why.

📸 Demo

N/A
🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

Todos

[ ] Regen pnpm lockfile
[ ] Regen pods lockfile
[ ] Test on iOS
[ ] Fix wrong usage of feature flag


Release 0.12.0

[CHANGE] Unified CGO crossbuild image #219

Signed-off-by: Ben Kochie<EMAIL_ADDRESS>

This will fix darwin/arm64 CGO builds: https://github.com/prometheus/node_exporter/pull/2020


test set proportions or

Hi, I would like to ask when you compare each method, did you ensure that their test set proportions are the same when calculating their mse or Pearson or the value in the test set are all the same? Thank you very much for your reply, thank you!

test set proportions or the value in the test set


Cannot run wasm_bindgen_test on headless browsers

Describe the Bug
When trying to run a wasm_bindgen_test on a headless browser, i get the following error:
Running headless tests in Chrome on `http://<IP_ADDRESS>:41061/`
Try find `webdriver.json` for configure browser's capabilities:
Not found
driver status: signal: 9
driver stdout:
    Starting ChromeDriver 83.0.4103.39 (ccbf011cb2d2b19b506d844400483861342c20cd-refs/branch-heads/4103@{#416}) on port 41061
    Only local connections are allowed.
    Please see https://chromedriver.chromium.org/security-considerations for suggestions on keeping ChromeDriver safe.
    ChromeDriver was started successfully.

Error: non-200 response code: 404
{"value":{"error":"invalid session id","message":"invalid session id","stacktrace":"#0 0x55e28740e579 \u003Cunknown>\n"}}

Steps to Reproduce
I followed the following guide https://rustwasm.github.io/docs/book/game-of-life
Expected Behavior
I'd expect the test to be able to run, or to have more indications on how to resolve it.

wasm_game_of_life.zip

Are you sure your chromedriver version is up to date? Or it matches your version of chrome? Otherwise I'm not sure how to debug this myself.

I'm working on WSL2, sorry i forgot to mention that

This is probably a problem with the connection between WSL2 and windows so i'm closing this

I'm running into a very similar issue on Pop!_OS, trying to run wasm_bindgen_test with a headless chrome browser. @TomPradat I know this is like an ancient issue by now, but were you ever able to resolve it?
Here's the error I'm seeing after running wasm-pack test --chrome --headless
Running headless tests in Chrome on `http://<IP_ADDRESS>:45141/`
Try find `webdriver.json` for configure browser's capabilities:
Ok
driver status: signal: 9
driver stdout:
    Starting ChromeDriver 104.0.5112.79 (3cf3e8c8a07d104b9e1260c910efb8f383285dc5-refs/branch-heads/5112@{#1307}) on port 45141
    Only local connections are allowed.
    Please see https://chromedriver.chromium.org/security-considerations for suggestions on keeping ChromeDriver safe.
    ChromeDriver was started successfully.

Error: non-200 response code: 404
{"value":{"error":"invalid session id","message":"invalid session id","stacktrace":"#0 0x55bbe8885403 \u003Cunknown>\n#1 0x55bbe868b64b \u003Cunknown>\n#2 0x55bbe86b5a0b \u003Cunknown>\n#3 0x55bbe86e097c \u003Cunknown>\n#4 0x55bbe86de620 \u003Cunknown>\n#5 0x55bbe86dde47 \u003Cunknown>\n#6 0x55bbe8663584 \u003Cunknown>\n#7 0x55bbe86643e3 \u003Cunknown>\n#8 0x55bbe88cce3d \u003Cunknown>\n#9 0x55bbe88cfdb6 \u003Cunknown>\n#10 0x55bbe88b613e \u003Cunknown>\n#11 0x55bbe88d09b5 \u003Cunknown>\n#12 0x55bbe88aa970 \u003Cunknown>\n#13 0x55bbe86630a5 \u003Cunknown>\n#14 0x7fcf4be71b80 \u003Cunknown>\n#15 0x55bbe863136a \u003Cunknown>\n"}}
error: test failed, to rerun pass '--test app'
Error: Running Wasm tests with wasm-bindgen-test failed
Caused by: failed to execute `cargo test`: exited with exit status: 1
  full command: "cargo" "test" "--target" "wasm32-unknown-unknown"

I'm running into a very similar issue on Pop!_OS, trying to run wasm_bindgen_test with a headless chrome browser. @TomPradat I know this is like an ancient issue by now, but were you ever able to resolve it?

It's been some time y !
If I recall right I didn't solve the problem, I suspected this was because of WSL2
Good luck with your problem !

I just run into  this today, not even work with pure generated wasm-pack-template
wasm-pack test --headless --chrome

got
Set timeout to 20 seconds...
Running headless tests in Chrome on `http://<IP_ADDRESS>:54155/`
Try find `webdriver.json` for configure browser's capabilities:
Not found
driver status: signal: 9 (SIGKILL)
driver stdout:
    Starting ChromeDriver 106.0.5249.61<PHONE_NUMBER>44955cd3e264779baf0dd38212a4d0-refs/branch-heads/5249@{#569}) on port 54155
    Only local connections are allowed.
    Please see https://chromedriver.chromium.org/security-considerations for suggestions on keeping ChromeDriver safe.
    ChromeDriver was started successfully.

Error: non-200 response code: 404
{"value":{"error":"invalid session id","message":"invalid session id","stacktrace":"0   chromedriver                        0x000000010ed97598 chromedriver + 4404632\n1   chromedriver                        0x000000010ed1efa3 chromedriver + 3911587\n2   chromedriver                        0x000000010e9cbd20 chromedriver + 425248\n3   chromedriver                        0x000000010e9f65e1 chromedriver + 599521\n4   chromedriver                        0x000000010ea21cac chromedriver + 777388\n5   chromedriver                        0x000000010ea1ec0d chromedriver + 764941\n6   chromedriver                        0x000000010ea1e35b chromedriver + 762715\n7   chromedriver                        0x000000010e9a214a chromedriver + 254282\n8   chromedriver                        0x000000010ed670ce chromedriver + 4206798\n9   chromedriver                        0x000000010ed6b797 chromedriver + 4224919\n10  chromedriver                        0x000000010ed72a4f chromedriver + 4254287\n11  chromedriver                        0x000000010ed6c4da chromedriver + 4228314\n12  chromedriver                        0x000000010ed4523f chromedriver + 4067903\n13  chromedriver                        0x000000010e9a1182 chromedriver + 250242\n14  libdyld.dylib                       0x00007fff67fc8cc9 start + 1\n"}}
error: test failed, to rerun pass '--test web'
Error: Running Wasm tests with wasm-bindgen-test failed

Workaround is
wasm-pack test --headless --firefox

or w/ head
wasm-pack test --chrome

Good luck for someone passing by.

Also ran into this today - on a single file very small project.
attaching stacktrace for any future adventurers
[INFO]: Checking for the Wasm target...
    Finished dev [unoptimized + debuginfo] target(s) in 0.03s
[INFO]: Installing wasm-bindgen...
    Finished test [unoptimized + debuginfo] target(s) in 0.03s
     Running unittests src/lib.rs (target/wasm32-unknown-unknown/debug/deps/mipsy_web-6e787a29ea4d3aec.wasm)
Set timeout to 20 seconds...
Executing bindgen...

Running headless tests in Chrome on `http://<IP_ADDRESS>:44501/`
Try find `webdriver.json` for configure browser's capabilities:
Ok
Starting new webdriver session...
Visiting http://<IP_ADDRESS>:39469...

driver status: signal: 9 (SIGKILL)
driver stdout:
    Starting ChromeDriver 106.0.5249.61<PHONE_NUMBER>44955cd3e264779baf0dd38212a4d0-refs/branch-heads/5249@{#569}) on port 44501
    Only local connections are allowed.
    Please see https://chromedriver.chromium.org/security-considerations for suggestions on keeping ChromeDriver safe.
    ChromeDriver was started successfully.

Error: non-200 response code: 404
{"value":{"error":"invalid session id","message":"invalid session id","stacktrace":"#0 0x560a2e5a62c3 \u003Cunknown>\n#1 0x560a2e3af700 \u003Cunknown>\n#2 0x560a2e3db067 \u003Cunknown>\n#3 0x560a2e406e3c \u003Cunknown>\n#4 0x560a2e403c90 \u003Cunknown>\n#5 0x560a2e40335b \u003Cunknown>\n#6 0x560a2e383ab4 \u003Cunknown>\n#7 0x560a2e3848a3 \u003Cunknown>\n#8 0x560a2e5f418e \u003Cunknown>\n#9 0x560a2e5f7622 \u003Cunknown>\n#10 0x560a2e5daaae \u003Cunknown>\n#11 0x560a2e5f82a3 \u003Cunknown>\n#12 0x560a2e5ceecf \u003Cunknown>\n#13 0x560a2e38355c \u003Cunknown>\n#14 0x7f2bc92e1d90 \u003Cunknown>\n"}}
error: test failed, to rerun pass '--lib'
Error: Running Wasm tests with wasm-bindgen-test failed
Caused by: failed to execute `cargo test`: exited with exit status: 1
  full command: "cargo" "test" "--target" "wasm32-unknown-unknown"

Further research indicates this issue occurs when chromedriver and google-chrome are of different versions:
https://github.com/elixir-wallaby/wallaby/issues/468#issuecomment-770048540
I upgraded my google chrome to the latest version and it worked perfectly fine :)
Wallaby project has a script here: https://github.com/elixir-wallaby/wallaby/issues/468 that may be useful for future


feat: cosmjs #ntrn-269

cosmosclient to cosmjs migration
removes old generated proto files in favor of cosmjs types (including newly generated repo for neutron)
removes sequence retries for execTx (just use different wallets for each parallel test to avoid collisions)
move over cosmos local state class to this repo
increment node version to >= 20
lots of small stuff

PR's:

neutron-integration-tests

cosmjs migration and more

neutronjsplus

cosmjs migration and more; remove old proto types

neutron-tests

increment node version to >= 20

neutron

because dao_assert tests break of not matching contracts

https://github.com/neutron-org/neutron-tests/actions/runs/8993085461

https://github.com/neutron-org/neutron-tests/actions/runs/9008635184

https://github.com/neutron-org/neutron-tests/actions/runs/9114703647

https://github.com/neutron-org/neutron-tests/actions/runs/9616294522

https://github.com/neutron-org/neutron-tests/actions/runs/10201636953


Functions for testing are not exported

Hi. I'm trying to remove all deep imports from my project, and that's currently impossible because none of the classes/functions for testing are exported from the @itwin/oidc-signin-tool package.
In the civil repo, we're using a subset of the functions/classes used in itwinjs-core integration tests:
getAccessTokenFromBackend
TestBrowserAuthorizationClientConfiguration
TestFrontendAuthorizationClient
TestUserCredentials
TestUsers

Would it be possible to export those, or provide an alternative to the above for testing?

Hello.
While these aren't exported from the barrel and are internal, they are exported from their respective modules. For now, can you try to access them like:
import { TestFrontendAuthorizationClient } from "@itwin/oidc-signin-tool/lib/cjs/frontend";

And we'll take another look as time permits?

Hey Ben.
We're already using the internal barrel file in our tests and that's fine for now.
I just wanted to file something to make sure it eventually gets fixed.
No rush!

Sounds good - thanks.


Don't automatically upgrade engines while migrating

Which issue(s) this PR fixes:
longhorn/longhorn#7833
What this PR does / why we need it:
The key change here is that we prevent automatic live engine upgrade while migrating. (We already checked for migration for user-directed live engine upgrades.)
Special notes for your reviewer:
In the process of checking the user-directed flow, I noticed the check for migration there was different than the one for the automatic flow (which uses a nice new utility function). I moved the utility function so it could be used in both locations.

To test:
Follow steps 1-7 in the reproduce (https://github.com/longhorn/longhorn/issues/7833#issue-2115652385). Then:

Notice that the volume is NOT automatically upgraded.

Detach the volume from NodeB. The kubectl command completes successfully and the VolumeAttachment goes away.

eweber@laptop:~/issue-files/7833> k delete -f va-2.yaml
volumeattachment.storage.k8s.io "test-va-2" deleted
eweber@laptop:~/issue-files/7833> k get volumeattachment
NAME        ATTACHER             PV                                         NODE                                ATTACHED   AGE
test-va-1   driver.longhorn.io   pvc-991a9849-aebe-4a3a-ae6a-81207a073a15   eweber-v126-worker-9c1451b4-kgxdq   true       15m

Wait a few seconds and notice that the volume is successfully automatically upgraded. (Since it was detached from NodeB, the migration was cancelled, and the automatic upgrade proceeded easily.)

Detach the volume from NodeA. The kubectl command completes successfully and the VolumeAttachment goes away.

eweber@laptop:~/issue-files/7833> kl delete -f va-1.yaml
Warning: deleting cluster-scoped resources, not scoped to the provided namespace
volumeattachment.storage.k8s.io "test-va-1" deleted
eweber@laptop:~/issue-files/7833> k get volumeattachment
No resources found

@mergify backport v1.6.x v1.5.x v1.4.x


Add quick_save() and quick_load() methods to Client

Add quick_save() and quick_load() methods to the Client class. These methods correspond to the quick_save and quick_load requests to the game.

These functions already exist in the same file here:
https://github.com/Dentosal/python-sc2/blob/67619da692690f9b42b84e43a87f4119b2529c9d/sc2/client.py#L599-L611

Oops! I guess I forgot to check the develop branch and didn't notice it was already in there. Thanks for the clarification!


Constrain the genre type

There are four possible genres. Instead of storing the genre as a String where it would have to be checked for validity every time it's used, use an enum and only validate once.
And various other minor tweaks.

A bugger, I forgot to turn off my auto-formatter before doing this. Some of the changes are just reformatting. 😞


CPP-674 - Fix memaccess in rapidjson

This addresses the build issues with GCC 8

Does it make sense to just pull in the latest version? There's a fix on master: https://github.com/Tencent/rapidjson/commit/fa5963a2f5b231ee2babff771f169ccca22870ed
I'm pretty sure it also fixes our two other patches too. Maybe at the least we can create a JIRA to pull in the latest.

I would prefer we create a ticket and bring in the latest during another release cycle. Created CPP-686

We should pull in that full patch then: https://github.com/Tencent/rapidjson/commit/fa5963a2f5b231ee2babff771f169ccca22870ed


Add new function for checking if waived.

:nut_and_bolt: Description: What code changed, and why?
The filters for compliance scans were incorrectly checking for waived values.  With guidance from Superteam, this branch corrects the check for waived values, so that the number matches the amount of reports displayed in all column.
:chains: Related Resources
fixes https://github.com/chef/automate/issues/3277
:+1: Definition of Done
Counts on the top of each of the compliance reports columns matches the number or reports shown.
:athletic_shoe: How to Build and Test the Change
build component/automate-ui-devproxy && start_all_services
make serve
run load_compliance_reports in hab studio
navigate to https://a2-dev.test/compliance/reports/nodes
click on one of the nodes that has a lot of runs to see the detail view
filter by waived controls
the number on the top of waived controls column should match how many are in the filtered display
:white_check_mark: Checklist

[x] I have read the CONTRIBUTING document.
[ ] Tests added/updated?
[ ] Docs added/updated?
[x] All commits have been signed-off for the Developer Certification of Origin.

:camera: Screenshots, if applicable

Thanks @vjeffrey @rickmarry  - will merge as soon as buildkite is back


TypeError: '>' not supported between instances of 'dict' and 'int'

I am facing an error in /search
builtins.TypeError
TypeError: '>' not supported between instances of 'dict' and 'int'
TypeError: '>' not supported between instances of 'dict' and 'int'

The debugger caught an exception in your WSGI application. You can now look at the traceback which led to the error.
To switch between the interactive traceback and the plaintext one, you can click on the "Traceback" headline. From the text traceback you can also create a paste of it. For code execution mouse-over the frame you want to debug and click on the console icon on the right side.
You can execute arbitrary Python code in the stack frames and there are some extra helpers available for introspection:
dump() shows all variables in the frame
dump(obj) dumps all that's known about the object

This issue is resolved in the recent merge.


x/build/maintner: nil (*GerritCL).Meta when it shouldn't be

While iterating through the Corpus, I've encountered two GerritCLs where the their Meta field is empty:
{
  Project:0xc42f6d0f50
  Number:44072
  Created:0001-01-01 00:00:00 +0000 UTC
  Version:1
  Commit:{GitCommit 993bdcf6e6a7cf7c4b742349b6d28dd4de3359b8}
  Meta:<nil *GitCommit>
  MetaCommits:[]
  Status:
  GitHubIssueRefs:[]
  Messages:[]
}

{
  Project:0xc42f6d0f50
  Number:44073
  Created:0001-01-01 00:00:00 +0000 UTC
  Version:1
  Commit:{GitCommit 73004ad04bc0d0786977c27e205d6e3801daafe3}
  Meta:<nil *GitCommit>
  MetaCommits:[]
  Status:
  GitHubIssueRefs:[]
  Messages:[]
}

Moreover, I cannot find these two CLs either: 44072 and 44073.
/cc @kevinburke @bradfitz

I would think these two CLs have just been abandoned and purged from the Gerrit DB for some reason.
The timestamps are both 0.
Neither of the two commit SHAs are in the go project tree.
And CL 44071 and 44074 are both from around 4 months ago.
Are you seeing failures of some kind? I can't think of any negative consequences of this. @judepereira

Related to https://github.com/golang/go/issues/21984

I came across this while fixing cmd/cl. Hence thought I'd put it up here to see if somebody knew anything about it.
If it's a no-op, then we can close this.

We should do something here. At least understand what happened in the mutation log and add some docs if nothing else before closing this.

@bradfitz, any pointers on diving into the mutation logs?

@judepereira, I don't understand the question.
The logs are at http://maintner.golang.org/logs
I imagine there was a bug in the past that caused such log entries to be created. To close this bug, I'd at least document that it may be nil, but ideally we'd also understand when this bug was introduced and (hopefully) fixed.

@bradfitz, I meant pointers on how to find the cause of this, as there's no date info that's available (can't really pin point when it happened).

I can document that this field can be nil though.

Sarah pointed to abandoned CL’s above, which seems possible.... another
option is maybe if Gerrit shows different CL’s to different people. Another
option would be to add information that will crash the server again
before adding CL’s in this state, along with a ton of debugging
information that will help us pinpoint the problem.
On Sat, Nov 4, 2017 at 08:38 Jude Pereira<EMAIL_ADDRESS>wrote:

I can document that this field can be nil though.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/22060#issuecomment-341906394, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAOSIwr-I1yG0FOBy_IF8-C2PzqOQAqFks5szITvgaJpZM4PlKw8
.

--
--
Kevin Burke
925.271.7005 | kev.inburke.com


Do not use clone to do deep copy for array copy

Push same code with another hash to trigger the CI again

Linux build succeeded in CI but Mac hanged.
Merge this directly.