compiler: remove intermediary copy of compiled functions code

This PR modifies the compilation of modules to remove the intermediary buffers where the code of each function was copied before being written to the executable memory mapping.
The optimization reduces memory usage since we now only keep a single copy of the module code in memory.
Tested on the compilation of Python:
Type: alloc_space
Time: May 14, 2023 at 3:51pm (PDT)
Showing nodes accounting for -51.12MB, 15.15% of 337.43MB total
Dropped 11 nodes (cum <= 1.69MB)
      flat  flat%   sum%        cum   cum%
  -55.28MB 16.38% 16.38%   -51.15MB 15.16%  github.com/tetratelabs/wazero.(*runtime).CompileModule
    3.10MB  0.92% 15.46%     3.10MB  0.92%  github.com/tetratelabs/wazero/internal/wazeroir.(*Compiler).emit
    2.60MB  0.77% 14.69%     2.48MB  0.73%  github.com/tetratelabs/wazero/internal/engine/compiler.compileWasmFunction
    2.01MB   0.6% 14.10%     2.01MB   0.6%  github.com/tetratelabs/wazero/internal/engine/compiler.(*runtimeValueLocationStack).push (inline)
   -1.51MB  0.45% 14.55%    -1.51MB  0.45%  debug/dwarf.(*Data).parseAbbrev
   -1.49MB  0.44% 14.99%    -1.49MB  0.44%  github.com/tetratelabs/wazero/internal/engine/compiler.(*runtimeValueLocationStack).cloneFrom (inline)
   -0.64MB  0.19% 15.18%    -0.64MB  0.19%  github.com/tetratelabs/wazero/internal/wasm/binary.decodeCustomSection (inline)
    0.64MB  0.19% 14.99%     6.71MB  1.99%  github.com/tetratelabs/wazero/internal/engine/compiler.(*engine).CompileModule
   -0.54MB  0.16% 15.15%    -0.54MB  0.16%  github.com/tetratelabs/wazero/internal/asm/arm64.(*AssemblerImpl).encodeRelativeBranch
   -0.50MB  0.15% 15.30%    -0.50MB  0.15%  github.com/tetratelabs/wazero/internal/wasm.(*Module).validateFunctionWithMaxStackValues
   -0.50MB  0.15% 15.45%    -2.01MB  0.59%  debug/dwarf.New
    0.50MB  0.15% 15.30%     0.50MB  0.15%  github.com/tetratelabs/wazero/internal/wasm.(*Store).GetFunctionTypeID
    0.50MB  0.15% 15.15%     3.60MB  1.07%  github.com/tetratelabs/wazero/internal/wazeroir.(*Compiler).handleInstruction

hah this is looking fantastic

I think I addressed all the comments, thank you all for taking the time to review and submit feedback!
@codefromthecrypt it turns out syscall.SYS_MREMAP is not defined for FreeBSD; since remapping is an optimization and is emulated via mmap+copy+munmap, I'd suggest we leave as-is until someone cares enough to implement this optimization on FreeBSD.

@codefromthecrypt it turns out syscall.SYS_MREMAP is not defined for FreeBSD; since remapping is an optimization and is emulated via mmap+copy+munmap, I'd suggest we leave as-is until someone cares enough to implement this optimization on FreeBSD.

Works for me. These notes are here for the next person if it comes up!

I completed a fuzz run on linux/amd64, no issues were detected:
$ make fuzz fuzz_timeout_seconds=180
...
Done 685736 runs in 181 second(s)


Add environ to context

cc @dhirschfeld @mariusvniekerk
There you go :)

There you go :)

Ha ha, thanks @wolfv! It doesn't look too tricky after all! 😅

Thanks @wolfv! :rocket:


Link zzzone game

Hello Ali,
thank you so much for your excellent planck.js library!
I found no other way to contact you to ask if it would be okay to link to my game from your repository, so I created this pull request. It is just a one-liner in the README.md with the link.
I am also referring to your library in the game ending credits: https://github.com/smurf667/zzzone/blob/master/src/level/gameOver.json#L37-L38
Kind regards,
Jan Engehausen

Hi Jan, your game is awesome! Thanks for making the PR!


TSC Meeting #31 2020-10-07 20:00 UTC

Agenda

[@nainar] Pixi: https://github.com/ampproject/meta-tsc/issues/50
Review TSC issue backlog

https://github.com/ampproject/meta-tsc/issues/51

Review TSC-tagged issues (toggle to "open" after checking for recently "closed" items)

https://github.com/ampproject/meta/issues/65

@ampproject/tsc
@tobie Can you specify AC attendees?

I would like to discuss Pixi: https://github.com/ampproject/meta-tsc/issues/50

Notes:

[@nainar] Pixi: #50

Looks good
Q: We could capture every test run with the tool. We can have a record of every time there were no recommendations given. Could perform statistical analysis of traits in common between the examples. Possibly privacy concerns here?

Not logging individual URLs at this stage.

Q: This is one-off process where developer can go in and look at and correct issues. No way to run this on an ongoing basis to capture regressions. What’s thinking for regression prevention use case?

Other tools like Search Console step in here

Possible confusion about all the tools available

Conscious of this. For non-AMP pages, PSI has a good experience. For AMP pages, PSI provides distracting advice.
Working with PSI team to make further AMP-specific improvements

Review TSC issue backlog

https://github.com/ampproject/meta-tsc/issues/51

What if election coverage goes beyond?

We could extend beyond and make a call about doing so a few days before Nov 8 (when the window ends). How would TSC assess whether an extension is merited?
Let’s keep Nov 4 TSC meeting

TSC approves election freeze proposal
Should be conscious of LTS schedule

Would be good to understand within major news publisher space what adoption of LTS is.
Guidance from TSC to also ensure LTS is frozen in this period

Review TSC-tagged issues (toggle to "open" after checking for recently "closed" items)

https://github.com/ampproject/meta/issues/65

Q: Bento WG timeline seems ephemeral. What is timeline for keeping that

Want to see public beta by end of year and public release by end of Q2.
And then at that point, we would have a follow-up convo. Would also want to have a standalone WG for Accessibility. A lot of what’s learned in the Bento effort would translate well toward long-term leadership of an Accessibility-focused group.

Q: Why merge performance and runtime together?

These are in the same space and people are working on both at the same time.

Q: Is there a possibility that people who need to follow-up on runtime won’t know where to go?
Q: Facilitators - is it figured out who they will be?

Kris will update issue to nominate the facilitators

Approved by TSC

Panel prep for TSC

8:45 am start for 9am AMA
Use Zoom link listed in calendar invite


Power levels

Added functions for reading and modifying user power levels and power level requirements

ok to test

@anewusername Your lines are very slightly over the limit of characters, could you fix that :)

Sorry this has taken so long. Slipped right under my radar.

I'm not really sure how to approach this. On one hand, set_power_levels(room_id, **kwargs) matches the API nicely and is symmetric with get_power_levels(room_id).
On the other hand, the API seems to be optimized for convenient storage rather than easy/clear usage:

set_power_levels will trash all your previously set restrictions and permissions, so you'll almost always want to call get_power_levels first and modify its output (hence modify_*_power_levels)
set_power_levels will control both the power level requirements for the room, as well as the power levels assigned to individual users, so it's unclear whether any given call is intended to affect the entire room or just a specific set of users in the room.

Based on the project readme, maybe the modify_*_power_levels should live in the higher level, e.g. room.py?
Your call, I can definitely make do with just having set_power_levels and moving the other functions out to my own projects.

Based on the project readme, maybe the modify_*_power_levels should live in the higher level, e.g. room.py?

That would be ideal yes. The API is ideally used for basic wrapping round requests rather than being smart.
My suggestion would be to:
Have set_power_levels(room_id, **kwargs) in api.py where it will dumbly set the levels you have given it without trying to keep old power levels.
Have modify_*_power_levels(**kwargs) in room.py where it will pull the power levels from get_power_levels and fill in where a kwarg was not specified.

Made the changes, with a few minor differences:
set_power_levels(room_id, content) in api.py takes a dict rather than **kwargs, since you'll usually want all the fields filled in anyways.
modify_required_power_levels(self, events=None, **kwargs) in room.py takes events as a separate optional arg since it requires a long explanation in the docs and special handling in the code.

Actually made use of this today. @Half-Shot, what needs to happen for this to be merged?

This has been merged now (with added tests). Thanks!


Fix codecov ignore coverage for certain files

Also update the codecov bash script

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9260696). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1054   +/-   ##
=========================================
  Coverage          ?   66.84%
=========================================
  Files             ?      741
  Lines             ?    35318
  Branches          ?     4486
=========================================
  Hits              ?    23608
  Misses            ?    10155
  Partials          ?     1555

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 9260696...cfd654a. Read the comment docs.


grid Creator - Web design

https://images01.nicepagecdn.com/page/58/92/html-template-5892811.jpg Design a grid like the above image
Create a file "grid_NUM_by_(YourName).html" in the modules/grid folder.
Use simple HTML and CSS and JAVASCRIPT to create the grid animation. You can use the interface development https://kwickerhub.com/ (This project) or you can write the lines of code yourself.
Step by Step Guide.
Fork this project(Use the 'fork' button in the top right corner) and Clone your Fork.
git clone https://github.com/YOUR_USERNAME/frontend
Open your code Editor and Create a file "grid_NUM_YOUR_NAME_.html" in the "modules/grid" folder of this project you just cloned. No need to create head, title and body tags, Just Add a div tag with some embedded style(i.e use the style tag) and the script tag where necessary.
If you want to add an image resource, please add it in the folder ••
"modules/grid/images_and_icons"
We recommend you use an svg for your image/icon.
Push your Code: You need to push your recent changes back to the cloud. Use the command below in the main directory of this Repository
git push origin dev
or use a GUI tool to avoid mistakes or complexity. LOL.
Then make you Pull Request...
Below is a example of the code structure we need:
'''

'''
For more examples:
modules->buttons
modules->cards
for examples of contributions or Pull request accepted in to the project.
And please do not wait for this issue to be assigned to you as we have limited hands but have a lot to do/cover. Please send in your PR.
Good-luck.

I've created a PR, please check and merge ASAP. #309 @NtemKenyor @salilbc


Update Jenkins text logo navigation

What feature do you want to see added?
At this point in time, when someone clicks on the Jenkins text logo in the upper left hand corner, they will be brought to the contributors.jenkins.io home page. This is the same functionality as the plugins.jenkins.io page & stories.jenkins.io page, where if a user navigates to one of the plugin pages or a success story, and clicks on the Jenkins text logo, they are brought to the home page for that specific url/domain.
https://github.com/jenkins-infra/contributor-spotlight/assets/99040580/cf2c3871-687d-41b9-8375-8ed9bdaddb26
This could be confusing, as users may expect the Jenkins text logo to bring them directly to jenkins.io instead of the contributor home page or plugin home page. However, changing this may cause a chain reaction where the functionality is questioned on other pages.
Should this be updated or changed?
Upstream changes
No response

Currently being discussed at https://github.com/jenkins-infra/jenkins-io-components/pull/118.


RuntimeError: random_ expects 'from' to be less than 'to', but got from=0 >= to=0

Dear Dr. Rosen,
Many thanks for providing the wonderful integration tool. I have used SATURN for my cell type evolution project. While other cell types were successfully integrated, I met an error when I integrated fibroblast across 10 species (mouse and other 9 species).

Pretraining...
^M  0%|          | 0/200 [00:00<?, ?it/s]^MEpoch 1: L1 Loss 0.0 Rank Loss 97.66471862792969, Avg Loss CPLA: 1212, Avg Loss EBUR: 940,
Saving Pretrain AnnData
^M  0%|          | 0/32 [00:00<?, ?it/s]^M  3%|▎         | 1/32 [00:00<00:06,  4.50it/s]^M  6%|▋         | 2/32 [00:00<00:06,  4.83it
STARTING METRIC LEARNING
STARTING METRIC TRAINING
Epoch 1 Iteration 0: Loss = 0.14469335973262787, Number of mined triplets = 128462
Traceback (most recent call last):
File "/share/home/bin/miniconda3/envs/SATURN_CUDA121/SATURN/train-saturn.ncores.aggl.py", line 1104, in <modu
trainer(args)
File "/share/home/bin/miniconda3/envs/SATURN_CUDA121/SATURN/train-saturn.ncores.aggl.py", line 829, in traine
train(metric_model, loss_func, mining_func, device,
File "/share/home/bin/miniconda3/envs/SATURN_CUDA121/SATURN/train-saturn.ncores.aggl.py", line 103, in train
indices_tuple = mining_func(embeddings, labels, species, mnn=mnn)
File "/share/home/bin/miniconda3/envs/SATURN_CUDA121/lib/python3.9/site-packages/torch/nn/modules/module.py",
return self.call_impl(*args, **kwargs)
File "/share/home/bin/miniconda3/envs/SATURN_CUDA121/lib/python3.9/site-packages/torch/nn/modules/module.py",
return forward_call(*args, **kwargs)
File "/share/home/bin/miniconda3/envs/SATURN_CUDA121/SATURN/miners/base_miner.py", line 31, in forward
mining_output = self.mine(embeddings, labels, ref_emb, ref_labels, species, mnn)
File "/share/home/bin/miniconda3/envs/SATURN_CUDA121/SATURN/miners/triplet_margin_miner.py", line 35, in mine
anchor_idx, positive_idx, negative_idx = lmu.get_species_triplet_indices(
File "/share/home/bin/miniconda3/envs/SATURN_CUDA121/SATURN/utils/loss_and_miner_utils.py", line 423, in get
n_rand = torch.randint(0, len(poss_n_ind), (choice_size,))
RuntimeError: random_ expects 'from' to be less than 'to', but got from=0 >= to=0

I wonder whether it because the number of fibroblast subcluster varied greatly (see example below). But pairwise integration (like mouse vs another one species) is okay.

species 1, cell_type_final
c0                2790
c1                  18
Name: count, dtype: int64
species 2, cell_type_final
c0                1753
c1                  94
Name: count, dtype: int64
species 3, cell_type_final
c0                1067
c1                1004
Name: count, dtype: int64
...
species 10, cell_type_final
cel_type_final
c0                1989
c1                  34
Name: count, dtype: int64

I'd appreciate it if you could get back to me. Thank you.
Best wishes,
Sheng

What batch size are you using? How many cell types are there that could possibly be negative examples (eg, from the same species but with a different cell type to the anchor cell?

Many thanks for your reply. The batch size and pretrain batch size are both 1024. The detailed parameters are below. There are only two sub cell types of fibroblast (as shown above, c0 and c1) within each species.

python3 -u /share/home/qiansheng/bin/miniconda3/envs/SATURN_CUDA121/SATURN/train-saturn.ncores.aggl.py
--in_data=./${input_data}.csv
--device_num=$device
--pretrain_batch_size 1024
--batch_size 1024
--in_label_col=cel_type_final
--ref_label_col=cel_type_final
--work_dir=./multiple_seeds_results/
--num_macrogenes=${nmac}
--pretrain
--model_dim=256
--polling_freq=201
--epochs=50
--hv_genes=0
--hv_span=1
--pretrain_epochs=200
--pe_sim_penalty=1.0
--l1_penalty=0
--centroid_score_func=default
--seed=0
--org=${input_data}l1_0_pe_1.0_ESM2_macrogenes${nmac}_hv_genes_0_centroid_score_func_default_batch_label_split
--embedding_model=ESM2 >s_mac${nmac}.log 2>&1

How many cells of each cell type are there?

The number of cells in each sub-cell type (c0, c1, c2, ...) is below.
species1
cel_type_final
c0                3006
c1                 281
c2                  39
Name: count, dtype: int64
species2
cel_type_final
c0                19918
c1                 5631
c2                  670
c3                  648
c4                  409
c5                  345
Name: count, dtype: int64
species3
cel_type_final
c0                8410
c1                6952
c2                1324
Name: count, dtype: int64
species4
cel_type_final
c0                5091
c1                 459
c2                  55
Name: count, dtype: int64
species5
cel_type_final
c0                899
c1                668
c2                591
c3                 16
Name: count, dtype: int64
species6
cel_type_final
c0                7016
c1                 108
Name: count, dtype: int64
species7
cel_type_final
c0                1247
c1                1030
c2                 585
Name: count, dtype: int64
species8
cel_type_final
c0                23188
c1                  286
Name: count, dtype: int64

I think the most likely reason this is happening is that within a batch, there are not enough examples of the negative cell type for the triplet mining function.
Without making major modifications to the codebase, you could consider the following steps to try and avoid this:

increase the batch size for metric learning (so that there are more likely to be possible negative examples)
reduce the number of species being integrated
balance the cell type frequencies more, perhaps by re-annotating or by re-sampling the dataset

Alternatively, you could also check out our method UCE https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>8918v1 https://github.com/snap-stanford/UCE which won't have any of these issues since it doesn't require training a model from scratch.


New URL for Yahoo

Hello,
this is the new URL for yahoo: https://postmaster.yahooinc.com/mail-servers
Change it on scrape_yahoo script.
Andrea

Works! Thanks

This url is now blocked by a cookie-banner. Clicking through brings an 500 "Internal Server Error".

Hello,
please try https://senders.yahooinc.com/outbound-mail-servers
Andrea

@agabellini Thx a lot! This works!

  Does that matter?
ipv6:::
ipv6:ea::
ipv6:::
ipv6:::
ipv6:::
ipv6:::

I am filtering these out, see: https://github.com/jfqd/mi-qutic-mail/blob/43792fe8f5a7160d8a49f1dcffe900f91524f3fd/customize#L206


[Proposal]: New WhatTheHack covering Cosmos DB

Topic
This WTH will try to cover best practices in using Cosmos DB as both an operation backend and extending to Analytics. It will have challenges on Data Modeling and leveraging Cosmos DB features to solve interesting scenarios.
Abstract/Learning Objectives

Cosmos DB capacity options
Cosmos DB data modeling and throughput, including troubleshooting a bad data model
Cosmos DB monitoring
Usage of features such as Change Feed, Synapse Link, Hierarchical Partition Keys, Partition merging and more

Delivery Date
September 2022
Authors
No response
Other
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks @istavrinides for volunteering to create a new hack on Cosmos DB!  We look forward to adding it to the WTH collection soon!

This Issue is closed by PR #507


Todo List

This list details what remains to be done to reach MVP:

[x] Create the needed models and migrations
[x] Create a middleware
[x] Create a Controller that utilizes a trait such that the trait methods can be overridden by the end-user.
[x] Create routes. Don't force the routes presence. That is, Have a way to disable them.
[x] Create config files
[x] Create a default notification
[x] Create views (index and code entry)

Done


Sentiment Analysis and EDA and increasing the accuracy through Gradient Boost Algorithms

Prerequisites

[X] I am an Open Source Contributor
[X] I checked to make sure that this issue has not already been filed

Description
I want to perform Sentiment Analysis and EDA like pre-processing, feature engineering and implementing N-grams and stuff like that. I think classification algorithms like XGBoost , CatBoost will increase the accuracy.
I have good knowledge in machine learning algorithms.
Kindly assign me with this issue under GSSOC '23 :)
Screenshots
No response
Code of Conduct

[X] I follow Contributing Guidelines of this project.

Assigning, proceed @Soumya1219

Hi, I am working on it.


SharePoint PNP (Custom Learning for Office 365) app

Issue Category: SharePoint PNP (Custom Learning for Office 365) app.

Question: how to get Custom Learning for Office 365 App in French language.

Expected or Desired Behavior: When I add Playlist on SharePoint modern Communication site for French site collection, then playlist appears in English only, not in French. Although the language has been chosen as French while creating the SharePoint Modern Communication site.

Current Version we are using: VERSION | <IP_ADDRESS> | RELEASE DATE | May 2019

Hello,
We are currently working on multilingual support for the following languages and are looking to have this released for CY19 Q3 (French being among them):
Chinese (Simplified)
Dutch (Netherlands)
French
German
Italian (Italy)
Japanese (Japan)
Portuguese (Brazilian)
Russian (Russian)
Spanish
Please let us know if you have any other questions or concerns. Apologies for any inconvenience this may cause  in the meantime. Thank you!
Regards,
Justin

Hello Justin,
Thank you so much for your quick response. I am really glad to see that you did feedback this to Microsoft team back in.
I have few more questions, will open new issue/thread.
Thanks again!
Regards,
Aditya


How do I enable Spinner mode for TimePicker?

How do I enable Spinner mode for TimePicker? I don't want to expose a circular, clock-like UI.

The Spinner mode is just for iOS, but android, desktop and web will have the material clock ui.


Game Crashed

I forked the sample project and changed the design little bit (didnt touched code)
Issue:
When i try to start Optifine 1.16.5 (local custom) it loads and gives Game Crashed log
Crash logs:
crash-2023-01-31_20.35.41-client.txt
crash-2023-01-31_20.33.15-client.txt
I'm new to these and since i dont really know Java, i dont know what is this issue

Update: looks like it can run vanilla 1.16.5 after i download via launcher but it cant load my optifine 1.16.5 which i installed before with TLauncher

Can you send the code pls

#16
TLauncher has quite different version json structure.

Oh so if i install Optifine gl seperately it will work?

Need to install optifine seperately because of other launchers' json format


Nice work

@spidergears Nice work bro. keep it up.  I'll like you to start an android project that we can collaborate together

Thank you. :)


Tweak to RENCI SPARQL endpoint URL?

RENCI Ubergraph Endpoint mentioned in https://oboacademy.github.io/obook/tutorial/sparql/ triggers a download of an rdf file, but maybe the website ubergraph app was intended? If so then https://stars-app.renci.org/ubergraph/sparql should be #sparql anchored https://stars-app.renci.org/ubergraph#sparql  ?

Turns out this will be deprecated soon.  See https://github.com/INCATools/ubergraph/issues/37

Thanks @ddooley for the report, changed the link to yasgui instead.


add --order-attributes option

If a tag has many attributes it can be hard to visually find the attribute you are looking for. Especially since many similar tags might have a very different order of the same attributes.
Therefore this patch adds a new option --order-attributes which orders the attribute alphabetically. It still keeps the id attributes in the front (as it was already before).

Thank for merging this quickly. Do you happen to know what it would take to make this option available when exporting "optimized svg" from Inkscape?

nope, no clue. we use scour from CI / command line on normal inkscape svgs. maybe @Ede123 knows ..

I'm currently maintaining the Inkscape Scour extension and can add the
necessary code.
It's to late to include it in 0.92 though, since we're in string freeze. I
can give instructions though how to manually upgrade it when I'm back from
holiday if required.
However what about making this a default (maybe not an option at all)? I
don't see a reason not to always sort attributes in a reasonable way. In
that case I'd favor a little more sophisticated sort order though (e.g. ID
first, then default attributes of the object like x/y/width/height, then
color/stroke style, Css style, and only then the other attributes in
alphabetical order.
Am 1. August 2016 8:24:08 vorm. schrieb Tobias Oberstein
<EMAIL_ADDRESS>
nope, no clue. we use scour from CI / command line on normal inkscape svgs.
maybe @Ede123 knows ..

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/scour-project/scour/pull/59#issuecomment-236613510

<aking it a default - maybe with a more sophisticated order - sounds good to me too. If we agree on what the order should be I would be happy to provide another PR.

@dirk-thomas I opened #60 to further discuss making this a default as well as possible ordering mechanisms. Feedback is welcome.


Fixes Bug in GetQualifier

Hi. GetQualifier has a typo in it and does not work. This little patch should fix it. Hope you can merge it. Thanks

Hey thanks, that's a glaring bug!


feat(tikv/*): add job to sync prow OWNERS files

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


New message when starting program.

New message on startup.

We are asking everybody if anyone wants this update, Thanks.


chore: Updating pre-commit hook for staged files to not run spotless check if its a merge commit

Description
Updating pre-commit hook for staged files to not run spotless check if its a merge commit
PR fixes following issue(s)
Fixes # (issue number)
Type of change

Chore (housekeeping or task changes that don't impact user perception)

Testing
How Has This Been Tested?

[ ] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

@ankitakinger My bad! Sorry for adding the comment early


could be use in VS2019?

Expected Behaviour
Actual Behaviour
Reproduce Scenario (including but not limited to)
Steps to Reproduce
Platform and Version
Sample Code that illustrates the problem
Logs taken while reproducing problem

Latest release of v2023.12 XMP-Toolkit-SDK can be build with VS2022.


[Inadyn] Backup size

Hello,
just checked the backup of Inadyn and it's very big, around 750mo.

When going inside this is what I see:

. is 760mo

image.tar is 760 mo

addon.json
apparmor.txt
data is 204bytes

--- Edit ---
Also in the backup log, only for this addon I get this:

I actually noticed the same behaviour in the other addons (webtrees), backup was giant and again the same image.tar
I think it is because the images of the addons are not build and pushed into a registries so hassio is backuping the whole image...
Do you think it would be possible to add a github action to automatically build and push the docker images and so avoid all this giant backup size ?

Hi, indeed it's the same for all addons autobuilding : the image is stored when doing a backup.
The only way to avoid that is to enable autobuilds. However, I can't do that due to this bug in supervisor : https://github.com/home-assistant/supervisor/issues/3414. As long as this is not solved, i can't enable autobuilds or all my users won't be able to,update my addons.
So, if you want to help please like the supervisor issue so it is taken care off by the devs, allowing me to enable autobuilds and allowing all the nice things coming with it : less sd wear, faster updates, decrease backup size...

hoooo I see :/

No, I can't duplicate everything, and the testing was already done : I'm just waiting for the supervisor bug being solved for implementation. So only new addons have the images enabled as it avoids the update bug. Here is the list of those addons : https://github.com/alexbelgium/hassio-addons/blob/master/.github/paths-filter.yml

Perfect so we can close the issues, waiting for the fixes I will just fork the addons :)
Thanks for your quick answers


[Feature Request] ip spoof

Details
https://github.com/net4people/bbs/issues/159
For tcp it has ACK to cope, but udp don't need.
Value
As title
Available alternatives
No response
Additional information
No response

Related resource repository: https://github.com/dimalinux/spoofsourceip

Technically, I don't think this is senseful.
Most of VPS providers and ISP verify the source address of IP packet (especially for the IPv4 network) to neutralize the DDoS attack on the Internet. Just take a look at RFC 3704 and rp_filter sysctl on the Linux.

Interesting read, but imo it's beyond the scope of this project. Not planning on adding it to Hysteria for now.

😌 OK
Feb 28, 2023 13:29:52 Toby @.***>:

Interesting read, but imo it's beyond the scope of this project. Not planning on adding it to Hysteria for now.
—
Reply to this email directly, view it on GitHub[https://github.com/apernet/hysteria/issues/577#issuecomment-1447604588], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AKGBAYCSICDBBX4UEW5PLBTWZWEM7ANCNFSM6AAAAAAVBI66B4].
You are receiving this because you authored the thread.[Tracking image][https://github.com/notifications/beacon/AKGBAYDVCS4LCV7MD4US2KLWZWEM7A5CNFSM6AAAAAAVBI66B6WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSWJCYWY.gif]


[Bug Fix] change the snap point to be attached the largest island id

Motivation and Context
 This PR ensures that the snap point of the object is always in the largest island. Previously it was checked by computing the largest island radius, which is not stable.
This is the joint work with Alex!
How Has This Been Tested
 Run the script, and it generates the episode.
Checklist

[x] My code follows the code style of this project.
[] I have updated the documentation if required.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes if required.

@aclegg3 Do you mind if you take a one more look? Thank you! :D


NO-JIRA: Fixes Dependabot PR #3713

What this PR does / why we need it:
Fixes Dependabot PR #3713 for the most part. This PR doesn't include the github.com/IBM/networking-go-sdk update as that is causing issues with PowerVS infra create documented in HOSTEDCP-1479.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/area ci-tooling

/test e2e-aws

/close


fix maxlen filter in gptq sampling

What does this PR do?
Fixes https://github.com/hiyouga/LLaMA-Factory/issues/5650
Before submitting

[x] Did you read the contributor guideline?
[x] Did you write any new necessary tests?

先不要合并


expose line-number metadata to codeblock viewer

I was exploring a bit clerk/editor integrations and found myself wanting line number information in the codeviewer, which isn't currently available. This PR threads that metadata through to the viewer, allowing for silly notebooks like the following, that allow codeblocks to be clicked to jump your editor's cursor to the specific line. While this may be a weird/cludgy idea, having this line info would also be useful if one wanted to show linenumbers in the codeblock viewer for instance
;; # 🦘Jump-to-code in editor
(ns lines
  (:require [nextjournal.clerk :as clerk]
            [nextjournal.clerk.viewer :as v]
            [clojure.java.shell :as shell]))

;; start nvim with:
;; `nvim --listen /tmp/nvimsocket`

(defn nvim-nav-to-line [address line-number]
  (shell/sh "nvim" "--server" address "--remote-send" (str "'<C-\\><C-N>:call cursor(" line-number ",1)<CR>'")))

#_(nvim-nav-to-line "/tmp/nvimsocket" 9)

5

(clerk/add-viewers!
  [(-> v/code-viewer
       (update :transform-fn
               (fn [old-transform-fn] (fn [value]
                                        ((v/update-val (fn [v] [v (-> value meta :row)]))
                                         (old-transform-fn value)))))
       (update :render-fn
               (fn [_old-viewer]
                 `(fn [value]
                    (v/html
                      [:div.viewer-code {:style {:background "var(--gray-panel-color)"}}
                       [:button.mx-auto.flex.items-center.rounded-sm.cursor-pointer.bg-indigo-200.hover:bg-indigo-300.leading-none
                        {:style {:font-size "11px" :padding "1px 3px"}
                         :on-click (fn [e] (v/clerk-eval `(nvim-nav-to-line "/tmp/nvimsocket" ~(second value))))}
                        "Goto"]
                       [v/inspect (v/code (first value))]])))))])

^:bar
(defn hi ^{:foo 1} [] "there")
(meta #'hi)

https://user-images.githubusercontent.com/94817/189311504-c3e277e9-6fa8-46d7-a45c-960f7fde5076.mp4
as a bonus:

include the line number in exceptions (:line 34 in the snapshot below, which wasn't there before)
set the :file to be what is being shown (the notebook file) and not the internal clerk.clj file

Nice, using clojure.main/ex-triage and setting the location metadata on the read form we get nicer errors still:

With 453423c the custom viewer code can be simplified:
(clerk/add-viewers!
 [(assoc v/code-viewer
         :render-fn
         `(fn [value opts]
            (v/html
             [:div.viewer-code {:style {:background "var(--gray-panel-color)"}}
              [:button.mx-auto.flex.items-center.rounded-sm.cursor-pointer.bg-indigo-200.hover:bg-indigo-300.leading-none
               {:style {:font-size "11px" :padding "1px 3px"}
                :on-click (fn [e] (v/clerk-eval `(nvim-nav-to-line "/tmp/nvimsocket" ~(-> opts :loc :line))))}
               "Goto Line " (-> opts :loc :line)]
              [v/inspect (v/code value)]])))])


Guest users + chat

So plug.dj recently added Guest Users, which are users that aren't logged in and don't have any user data. their behaviour is a bit funny so we have to deal with it in a rather silly way...
When a user joins as a guest, and then logs in, plug.dj doesn't always (but sometimes!) send a new USER_JOIN event. So you might end up with users chatting without joining. This patch first fetches user data from plug.dj before emitting chat events for non-logged-in users.
There's a few non-intuitive things happening in this PR because of this:

this might emit chat events out of order, since the next chat event might come in and be fired instantly whilst the current chat event is still waiting for user data to be loaded
guest users are not added to the user list, because USER_LEAVE events for guests always have user ID 0, even though guests do have different user IDs. this could be done differently in a future patch: guest users could be added to their own separate array and a random/the first one could be removed when the USER_LEAVE event comes in (guest user data is kinda useless, so it doesn't really matter which is removed)

the _pushUser method now sets the room population to the size of the users list instead of incrementing it, because the USER_JOIN/USER_LEAVE events aren't necessarily 1-to-1 anymore, so things might get out of sync with the real world based purely on those events. in general you end up with more USER_LEAVEs than USER_JOINs in the current situation.
gotta love plug <3

What else would we ever except, am I right? haha
However, the problem of inconsistency of data is something that really annoys the heavens out of me, sadly we can hardly do anything about it other than deal with the behaviour of the server. Thus said, yes it's sadly the only feasible option to actually support guest users, although they are so stripped down in their possibilities. Funny enough when you consider that this new feature makes us more trouble than gains on the dev side.
The question is how much does point 1 interrupt the flow of the bot and if it's the case, if it would be more applicable to solve the problem per implementation or directly in plugged.
The second is, have you been able to identify any inconsistent behaviour considering the USER_LEAVE when it comes to guest users?
Actually it's kind of a shame that guest users don't have an ID, since the USER_LEAVE could perfectly compensate for the inconsistency of the USER_JOIN event.
indeed we gotta love plug :heart:

When a user joins as a guest user, logs in, and later leaves, you'll get something like:

user joins: USER_JOIN for a unique user ID, with guest:true
user logs in: USER_LEAVE for ID 0, and sometimes a USER_JOIN for the new user ID with guest:false
user leaves: USER_LEAVE for their actual user ID

There's really just two things you can do for chat messages by such users:

emit CHAT without user data;
fetch user data and then emit CHAT, whilst also emitting CHAT for new messages, which is what plug.dj does;
fetch user data and then emit CHAT, blocking new messages until this one is emitted

The last one would be nice for bots, but fetching the user data can take a long time (10+ seconds if plug is slow) so it'll essentially make the bot unresponsive for that amount of time.

I see. Yea it's really inconvenient with the USER_JOIN in that case.
latter is exactly the problem I see about controlling the flow again and plug tends to have hiccups every now and then.


Use lombok

@Nick-Triller - would you care if we included lombok as a dependency?  This would make beans less verbose and error-prone.  I'd understand completely if you would rather not include another dependency, but I've found it very useful and it's pretty popular in general.
I can add it if you're cool with it.  If not, feel free to close this issue.  Note: I'd add it with provided scope.

See https://github.com/Nick-Triller/accounting/pull/4


UserPoolGroup has different LogicalID when upgrading from amplify-cli 9 to 12

Amplify CLI Version
12
Question
Hello!
I am encountering an error when pushing a userPoolGroups stack

MYGROUP already exists in stack arn:aws:cloudformation:us-west-2:...

I previously deployed an amplify userPoolGroup stack using amplify CLI 9 which allowed direct changes to the cloud formation template. We added several userPoolGroups to the template using logicalIDs that didn't append "Group".
This appears to an issue as when deploying in amplify cli 12 it generates CF templates with "Group" appended to all user pool group logicalIDs (so [MyGroupName]Group)
For the groups we created in amplify CLI 9 that have "Group" appended it correctly registers as an update to an existing user pool group, however for the few that have the different logicalIDs it fails since it views them as new user pool groups, and encounters an error when attempting to create them.
Is it possible to modify the logicalID using override.ts or some other method?
previous template we deployed via cli 9
{
   ...
   Resources: {
     ...
    "MYGROUP": {
      "Type": "AWS::Cognito::UserPoolGroup",
      "Properties": {
        "UserPoolId": {
          "Ref": "authwwwUserPoolId"
        },
        "Description": "override success",
        "GroupName": "MYGROUP",
        "Precedence": 116,
        "RoleArn": {
          "Fn::GetAtt": ["MYGROUPGroupRole", "Arn"]
        }
      }
    },
   ...
   }
}

generated by amplify-cli 12 during amplify push
{
   ...
   Resources: {
     ...
    "MYGROUPGroup": {
      "Type": "AWS::Cognito::UserPoolGroup",
      "Properties": {
        "UserPoolId": {
          "Ref": "authwwwUserPoolId"
        },
        "Description": "override success",
        "GroupName": "MYGROUP",
        "Precedence": 116,
        "RoleArn": {
          "Fn::GetAtt": ["MYGROUPGroupRole", "Arn"]
        }
      }
    },
   ...
   }
}

I have tried using override.ts but I haven't found a way to change the logicalID, based on the typescript types it doesn't seem to be an available field.
If the logicalID cannot be changed, is there another method I can use to deploy that doesn't remove the existing groups which are in use?

I found the function overrideLogicalId which worked
function removeGroupSuffixFromUserPoolLogicalID(
  resources: AmplifyUserPoolGroupStackTemplate,
  group: string,
) {
  if (resources.userPoolGroup && resources.userPoolGroup[group]) {
    console.log(`removeGroupSuffixFromUserPoolLogicalID for ${group}`);
    resources.userPoolGroup[group].overrideLogicalId(group);
  } else {
    console.log(`cannot removeGroupSuffixFromUserPoolLogicalID, no userPoolGroup for ${group}`);
  }
}

export function override(
  resources: AmplifyUserPoolGroupStackTemplate,
  amplifyProjectInfo: AmplifyProjectInfo,
) {
  if (resources.userPoolGroup) {
    //  these groups were created in Amplify CLI 9 using a name different than the generated group name.
    removeGroupSuffixFromUserPoolLogicalID(resources, 'MYGROUP');
}


Python version 2.x print in error_fixer.py

When I ran fixjsstyle, in Python 3.5.2, I got this traceback (closure-linter-2.3.13):
Traceback (most recent call last):
  File "/home/cnobile/.virtualenvs/testsite/bin/fixjsstyle", line 7, in <module>
    from closure_linter.fixjsstyle import main
  File "/home/cnobile/.virtualenvs/testsite/lib/python3.5/site-packages/closure_linter/fixjsstyle.py", line 25, in <module>
    from closure_linter import error_fixer
  File "/home/cnobile/.virtualenvs/testsite/lib/python3.5/site-packages/closure_linter/error_fixer.py", line 489
    print 'Fixed %d errors in %s' % (self._file_fix_count, self._file_name)
                                ^
SyntaxError: invalid syntax

Yes, unfortunately python-3 is not supported by this project and since it’s deprecated there won’t be any work on it either.


Are you going to be Supporting Preview versions of UE? or just latest releases?

Sorry, preview versions are not supported, due to not stable status.
Normally, we'll add support to the latest releases in one or two weeks once the new one is published.

By the way, Unreal 5.3 support will be ready in one or two days.

okay thanks for letting me know :)

My pleasure, and here is the new release to support Unreal 5.3: https://github.com/GameTechDev/XeSSUnrealPlugin/releases/tag/v1.3.3
Welcome any feedback.


add hands publish

We should publish the hands tracked in space from unity, and well as publish events from hand controller buttons/actions.

Add<EMAIL_ADDRESS>

Add https://docs.unity3d.com/Manual/com.unity.xr.arfoundation.html


Fix Data.Lens.Zoom.zoom

Without the type annotation Data.Lens.Zoom.zoom yields the following error:
[1/1 NoInstanceFound] src/Data/Lens/Zoom.purs:22:21

  22  zoom p = StateT <<< underF Focusing (under Star p) <<< runStateT
                          ^^^^^^^^^^^^^^^

  No type class instance was found for

    Prim.Coerce.Coercible t4
                          (m7 (Tuple r8 s6))

  The instance head contains unknown type variables. Consider adding a type annotation.

  while solving type class constraint

    Prim.Coerce.Coercible (s6 -> t4)
                          (s6 -> m7 (Tuple r8 s6))

  while applying a function underF
    of type Coercible @Type (t0 t1) (t0 t2) => Coercible @Type (t3 t4) (t3 t5) => Newtype t1 t2 => Newtype t4 t5 => (t2 -> t1) -> (t0 t1 -> t3 t4) -> t0 t2 -> t3 t5
    to argument Focusing
  while inferring the type of underF Focusing
  in value declaration zoom

  where s6 is a rigid type variable
          bound at (line 0, column 0 - line 0, column 0)
        r8 is a rigid type variable
          bound at (line 0, column 0 - line 0, column 0)
        m7 is a rigid type variable
          bound at (line 0, column 0 - line 0, column 0)
        t0 is an unknown type
        t3 is an unknown type
        t1 is an unknown type
        t2 is an unknown type
        t4 is an unknown type
        t5 is an unknown type

I suspect this happens because the functional dependency computing s -> Focusing m r s from Star (Focusing m r) s s in under Star p doesn’t propagate its result, so the compiler cannot solve the Coercible constraints implied by the Newtype constraints in underF Focusing (under Star p).

The build fails because newtype doesn’t depend on safe-coerce in the package set but it compiles with the rc5 branch of my fork set:
let upstream =
      https://raw.githubusercontent.com/kl0tl/package-sets/rc5/src/packages.dhall

in  upstream


 Solus Budgie with last tmodloader.

Was this tested without using the installed mono?
Either way, I'm not sure what we could do about it, it might be a problem with the runtime itself.

Try mono --profile (see the mono(1) manpage for more info) to find out what the bottleneck is. (I bet lots of allocations due to memory mismanagement.)

@PoroCYon How does one execute Terraria with custom mono flags?
Also, the latest version of tmodloader gives some additional info during loading.  On Linux, most of the extra time seems to be spent on loading PNG images.  As I noted on another thread, PNG is ridiculously slow compared to other compressed image formats, so its possible that the mono bindings for libPNG have a ton of overhead.
One possible way to mitigate this problem would be to use threads to load the files, though that could create other issues if the mod files are doing anything extra with the images while loading.

@zauberparacelsus Run Terraria.exe instead of Terraria, and do something like this.

Was that commit rolled into the latest version?

kennethdhau: it should be in v<IP_ADDRESS>

Closing due to lack of discussion


How to load pre-trained checkpoint?

Hi, could you tell me how to reload the pre-trained checkpoint you released?
When I try to use this pre-trained checkpoint to generate, there is some errors:
Traceback (most recent call last):
File "generate.py", line 172, in
main(args)
File "generate.py", line 44, in main
models, _ = utils.load_ensemble_for_inference(args.path.split(':'), task, model_arg_overrides=eval(args.model_overrides))
File "/users6/ychuang/program/python/control-length-master/encdec/fairseq/utils.py", line 160, in load_ensemble_for_inference
model.load_state_dict(state['model'], strict=True)
File "/users6/ychuang/program/python/control-length-master/encdec/fairseq/models/fairseq_model.py", line 64, in load_state_dict
super().load_state_dict(state_dict, strict)
File "/users6/ychuang/anaconda3/envs/py3.6_torch1.0/lib/python3.6/site-packages/torch/nn/modules/module.py", line 839, in load_state_dict
self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for TransformerModel:
size mismatch for encoder.embed_tokens.weight: copying a param with shape torch.Size([16152, 512]) from checkpoint, the shape in current model is torch.Size([124408, 512]).
size mismatch for decoder.embed_tokens.weight: copying a param with shape torch.Size([16152, 512]) from checkpoint, the shape in current model is torch.Size([124408, 512]).
Thank you very much!

I think that this error is caused by the difference among vocabulary sizes.
Did you apply my BPE code (which is attached to the gz file, as I remember) to your text file?

Should I fist apply BPE code to my text, then preprocess the text, right?

Yes! Please try such procedure.

Thank you!
Should I invoke these two instruction:
subword-nmt learn-bpe -s {num_operations} < {train_file} > {codes_file}
subword-nmt apply-bpe -c {codes_file} < {test_file} > {out_file}

or just the second instruction, and replace the codes_file with train.article.BPE16k.code you released?

I add dictionary file which is used to construct binary file in my experiments.
If you need it, please download the trained model file from my google drive again.
https://drive.google.com/file/d/15Sy8rv6Snw6Nso7T5MxYHSAZDdieXpE7/view?usp=sharing

So nice are you!
I have used the dictionary to construct binary file of test set.
And I get a best_15_nbest.txt after running generate.py.
But the headlines generated seem terrible, there two examples:

S-67    the agenda might be global , but the men@@ u will be malaysian when world leaders meet next week for the asia-pacific economic cooperation forum .

T-362   <> <> condem@@ ns slaying of 4 hostages in <> he@@ chn@@ ya@@ ; were telephone engine@@ er@@ s.
H-362   -0.11744093894958496    e u @@@@ c o n d e m n s @@@@ c h e
P-362   -0.3688 -0.0332 -0.0409 -0.2489 -0.0318 -0.0254 -0.0284 -0.0278 -0.0296 -0.0259 -0.0304 -0.0305 -0.5614 -0.0417 -0.0373 -0.3170

Before use the dictionary file you released to construct binary file, I preprocessed train and valid set of Gigaword and had got a dictionary whose size is 15380. However the size of dictionary you released is 16148.
It seems not avaliable to use your dictionary to preprocess test set on my Gigaword, which may be the reason why I failed to use your checkpoint to generate.

In our experimental configuration, we use a character-based decoder and use @@@@ as a space (a segmentation marker of words).
(I'm sorry the lack of my explanation. We applied the BPE code to a source side only.)
Thus, you pick up decoder outputs and replace the segmentation marker with the space as follows:
cat decoder_output.txt | sed 's/ //g' | sed 's/\@\@\@\@/ /g'
(decoder_output.txt contains the output hypotheses only)
In addition, you can modify the output length with --desired-length option.

You means just apply BPE to train.article, valid.article and test.article, and then preprocess them with *.title together to get a joint-dictionary?
after doing that, I get a dictionary whose size is 74932......(which is much larger than you released dictionary)

Could I confirm your purpose?
I thought that you planned to apply pre-trained model to a text for generation.
But you want to train a model on a new corpus?

My purpose is to apply pre-trained model to a text for generation, rather than train a model on a new corpus.
However, I get terrible generated headlines even if applying the checkpoint and dictionary you released.
Therefor I'm wondering the reason why I failed.

I see.
Could you extract the generated sentences from best_75_nbest.txt with the following commands?
 cat best_75_nbest.txt | grep '^H' | sed 's/^H\-//g' | sort -t ' ' -k1,1 -n | cut -f 3- > output.txt
Then, replace segmentation markers into spaces:
cat output.txt | sed 's/ //g' | sed 's/\@\@\@\@/ /g'
I expect that you can obtain sentences whose lengths are 15.
(Or, since our pre-trained model is fragile in the same as other neural encoder-decoders, the model might be broken by unknown inputs)

Hi, I have succeed using checkpoint and dictionary you released to generate headlines with following instructions:

CUDA_VISIBLE_DEVICES=0,1,2,3  python generate.py /users6/ychuang/program/python/control-length-master/databin2/test --source-lang article --target-lang title --path /users6/ychuang/program/python/control-length-master/pretrained_model/trained_model_lrpe_pe_averaged.pt --desired-length 75 --batch-size 32 --beam 5 > best_75_nbest.txt
cat best_75_nbest.txt | grep '^H' | sed 's/^H\-//g' | sort -t ' ' -k1,1 -n | cut -f 3- > output.txt
cat output.txt | sed 's/ //g' | sed 's/\@\@\@\@/ /g' > res_75.txtcat output.txt | sed 's/ //g' | sed 's/\@\@\@\@/ /g' > res_75.txt

Then I used the script in directory 'eval' to evaluate ROUGE of res_75.txt, follow is the instruction and result:

./eval.sh /users6/ychuang/program/python/control-length-master/eval/testdata /users6/ychuang/program/python/control-length-master/eval/testdata/input.txt 75 /users6/ychuang/program/python/control-length-master/eval /users6/ychuang/software/ROUGE/

ROUGE result：
a ROUGE-1 Average_R: 0.31056 (95%-conf.int. 0.29791 - 0.32237)
a ROUGE-1 Average_P: 0.28526 (95%-conf.int. 0.27364 - 0.29666)
a ROUGE-1 Average_F: 0.29650 (95%-conf.int. 0.28433 - 0.30811)

a ROUGE-2 Average_R: 0.11021 (95%-conf.int. 0.10188 - 0.11849)
a ROUGE-2 Average_P: 0.10084 (95%-conf.int. 0.09295 - 0.10858)
a ROUGE-2 Average_F: 0.10499 (95%-conf.int. 0.09703 - 0.11296)
a ROUGE-L Average_R: 0.27212 (95%-conf.int. 0.26072 - 0.28356)
a ROUGE-L Average_P: 0.25024 (95%-conf.int. 0.23987 - 0.26069)
a ROUGE-L Average_F: 0.25995 (95%-conf.int. 0.24929 - 0.27060)
The ROUGE of DUC2004 in your paper is:

My question is that:
The ROUGE you present in your paper is Recall or F1?

That table represents recall-based ROUGE scores i.e., ROUGE-N Average_R in results of the evaluation script.

Thank you again!


Add editor

You can try out the editor file, it should allow you to change all parameters of the model. It'll spit out a url that you can then use for the "player.html" which you input into OBS. I'll check for bugs and update the docs later 👍 Works great for me in OBS.
Let me know what you think!

In #6 he changes the netowrk to segmentMultiPerson, I reverted it back in this PR but I'm curious what the usecase is for that instead of segmentPerson. Segmentperson should still detect multiple people and create a single mask, and it will be faster.

Double-checked the official document.

Given an image with one or more people, person segmentation predicts segmentation for all people together. It returns a
PersonSegmentation object corresponding to segmentation for people in the image. It does not disambiguate between different
individuals. If you need to segment individuals separately use segmentMultiPerson (the caveat is this method is slower).
 I guess edurenye was confused by the name segmentMultiPerson. Multi here is not about "people" but it's is multi segments data. In GreenCam, we don't actually need segmentMultiPerson.

@nhtua I updated with the changes, let me know your thoughts. In the future I can improve the UI and make some other tweaks, but it's functional for now.

Tested, It works wells. I'm going to merge now. I will rewrite some document later.


Prepare sheet for monitoring metrics to implement and how

As we discussed in our Pelican kickoff meeting, we want to have a nice suite of monitoring metrics implemented. As the very first step, we need to identify the exact metrics and sort out where to find them, how to implement (if not existed yet), or if we need help from XRootD team. The sheet will live in the shared Google drive for Pelican

@bbockelm Here is a draft of metrics to add based on that photo capture of the white board: https://docs.google.com/spreadsheets/d/1iMnKOd8TMfIVB8172bGz5ZoqjlGXi1dG2F9ZL9wKQT8/edit#gid=0

Closed this issue as we have started migrating content from the sheet to GitHub project as well as individual issues


Keyword "Wait Until Network Is Idle" is not using the default timeout

Describe the bug
using the keyword without a certain timeout should use the default. But the test result is an error.
Invalid time string ''
By using a certain timeout like 500ms the keyword is working fine.
To Reproduce
use this core to reproduce
*** Settings ***
Library   Browser
*** Test Cases ***
Example Test
New Page    https://playwright.dev
Get Text    h1    ==    🎭 Playwright
Wait Until Network Is Idle
Expected behavior
The Test should not exit without a certain timeout

Thanks for the report :).
We didn't correctly get the library-wide default if timeout was not set in this keyword. Fixed in 8feed8e9b98d0b1f9a161ff14428af1f9934cbcf

@all-contributors please add gdroes for data

@all-contributors please add gdroes for test


Satellite remote sensing tuto

Initiating the PR for the tutorial for conducting a biodiversity study on Sentinel 2 data.

Hi @Marie59! I've added some final tweaks. It looks good to me! I see there are still some TODOs written in there, so just let me know when you are completely ready and I will do final review and merge it :)

@Marie59 it seems there is some TODO on remote sensing tuto notably. Can you look at it and update before asking @shiltemann to review and merge ? Moreover, It would be nice to propose an example datafile as input for the tuto. Don't remember if we were finding a good example data not too big?

@Marie59 it seems there is some TODO on remote sensing tuto notably. Can you look at it and update before asking @shiltemann to review and merge ? Moreover, It would be nice to propose an example datafile as input for the tuto. Don't remember if we were finding a good example data not too big?

I have one or 2 datafiles not that big but still it would take 2 days I think to do the full analysis

Should I open a new PR and close this one ? I don't know how to resolve the conflicts here

I can try to help with that

@Marie59 why does the last commit delete the champs-blocs directory? I believe this is the main cause of the conflict. Is that tutorial no longer needed or something?

Ahh no worries, that makes it easier to resolve. :)

Just the same workflow issue (the test input missing/creator details.)

I have one or 2 datafiles not that big but still it would take 2 days I think to do the full analysis

I think even that is good, better to have an example dataset than none.

Ooooh could it be finally ready ?!
Thank you for all your help @hexylena @shiltemann and @yvanlebras  !!

Thanks, it looks good!
Just a question about the time, you said it takes 2 days to complete the tutorial or did I misunderstand? Can we maybe add a warning box to say the analysis will take 2 days so people don't wait around for it? And maybe update it in the time estimate metadata? (it currently says 2 hours).
Which step is it that takes this long? is it possible/would it make sense to provide the outputs from that 2-day step as well (e.g. on Zenodo), so that learners or instructors can continue with the next steps without waiting 2 days?

Yes, of course I will add a warning and change the time. It's one of the last step that takes a lot of time. I can add one of the output to show but the output is not needed for the following steps. So, the users can easily just skip it or continue with the rest.

I think it would still be nice to show the output so people can view it without waiting the 2 days, if it's  not too much trouble to do that.

I put to photos for it. It's the beta part is it okay or do we need more ?

I have one last photo too add (I am waiting for the tool to run to put it)

I put two photos for it. It's the map biodiversity part is it okay or do we need more ?

Just whatever you think is needed so that they can evaluate the results of the tool without having to wait the 2 days :) :+1:

perfect, thanks!

Youhou!! Thank you for your help !!


Update External Incentives Gauges list

-removes finished gauges
-adds missing gauges

Hell yes! This is great


Remove dead code

Remove some unused classes, imports, mocks and methods

Have to be careful with this. PrivateKeyStoreFactory is used in ida-compliance-tool, for example.

Not any more? https://github.com/alphagov/ida-compliance-tool/pull/50


customEntityTransform argument missing in getSectionMarkdown call

Without passing this, section entities will not be run through any custom transformation.

Thanks a lot for the PR @mmatuson, I will soon release it 👍


Block multipart/form-data like other content-types

Hi,
Before explaining my issue, please take into account I'm fairly new to Ruby and Grape.
So, I'm trying to modify an ' legacy'  api that is using Grape, and our security team mentioned that accepting 'multipart/form-data' is a potential security risk due to CSRF. I've been reading about this and I've seen thar Ruby supposedly already prevents those kinds of attacks, but nevertheless it seems that there is still some potential way to overcome the protection: http://weblog.rubyonrails.org/2011/2/8/csrf-protection-bypass-in-ruby-on-rails/. Problem is I don't think I can upgrade or patch, and even then I don't know if it addresses the issue.
Now I've also seen that there was a fix for Grape 0.9.2 that addressed an issue that is similar to this, #726  regarding content-types. I've tested and it works with some, like for example 'application/xml', if I set 'format : json' in the API, but it doesn't for 'multipart/form-data'.
I've traced to the formatter.rb, method read_body_input that returns if the request is 'form_data'.
My question is if it's possible to block multipart/form-data content In some way, and only accept the format I specified in the API (:json in this case) ?
Thanks in advance.

Hi @dblock ,
i'm really new to ruby/grape. I can try, but can't promise you any timeframe.
The PR you mention has some conflicts, so  the idea would be to basically copy that PR and add some more specs?

Yes, but we also need to understand what's going on in the underlying infrastructure in terms of parsing.

Right, that's going to be a bit harder. Anyway, I'll see what i can do.


Photo frame

Blocked because of http://jira.kaaproject.org/browse/APP-148  - no integration with Sandbox Builder
Code is working. This task is completed.
Please, review code.

Please, reopen against develop.


Latest Plex Server update breaks Music section

Plex pushed an update to their server software this week (<IP_ADDRESS>) which was mainly focussed on overhauling the music side of things.  This seems to have broken the Plex Music tab in HTPC - clicking on tab results in "Error connecting to Plex" message.  If I already have music playing it still shows up in the Now Playing section but the controls don't work.

Thanks! i dont use plex for music so i would never had noticed

Should be fixed. Can you confirm?

Checked it out last night and music now loads correctly. Thanks for fixing it so fast! :)


add dot-env-file

_ No description provided. _

Have you tested the code with your API keys? This won’t work, we need to write the server-side code.

_ No description provided. _

Have you tested the code with your API keys? This won’t work, we need to write the server-side code.

Yeah, this will not work. Server side code is required to run env variables. Also after replacing with my own API key, it's showing Error generating text.

@akssshh As this is hosted on vercel, might be a good idea to take a look at vercel serverless functions to proxy requests to OpenAI.
This way API key won't get exposed to client devices.

@akssshh As this is hosted on vercel, might be a good idea to take a look at vercel serverless functions to proxy requests to OpenAI. This way API key won't get exposed to client devices.

Yeah. This will work but configuration has to be done on the deployment page.

@akssshh As this is hosted on vercel, might be a good idea to take a look at vercel serverless functions to proxy requests to OpenAI. This way API key won't get exposed to client devices.

This is a good suggestion. Thanks @prithweedas .
@akssshh do you want me to write the solution for it? Or do you want to write the server-side code?


Add option null to interface UpdateCardForm

This PR was created to add the null option to creditLimit, amount, and interval to the UpdateCardForm interface so that when the user updates the unlimited card interval this data can be passed as null.

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


Added tests for main

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+10.0%) to 80.397%

Files with Coverage Reduction
New Missed Lines
%

autoagora/main.py
1
71.93%

Totals

Change from base Build<PHONE_NUMBER>:
10.0%

Covered Lines:
324

Relevant Lines:
403

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+10.1%) to 80.597%

Totals

Change from base Build<PHONE_NUMBER>:
10.1%

Covered Lines:
324

Relevant Lines:
402

💛  - Coveralls


Q: How do I use color settings?

Maybe it's because it's Monday, but I do not really understand for what I should use the color stuff in ha-bridge, if Alexa does not understand it (which is, what I understand from all documentation, that Alexa understand from direct hue detection only ON/OFF/DIM)?
So before I start fiddeling around with all that color settings: how do I use that then?

Have you filled in any color command in the device editor?

Have you filled in any color command in the device editor?

No, although I could, but what could I then do with it? I use ha-bridge exclusively for voice control via Alexa...

I believe that has been fixed in Alexa. You can always test it by adding the color command and saying Alexa, set light to blue

great! it just works! I was using the wiki here for understanding the concept and it is stating different, I will change it. Thanks for your great work!

Not sure on what Alexa completely supports, but the ha-bridge only offers the commands for lights with on/off/dim/color/temperature


feat(Client) - added option to get last response body from client object

Added option to store response body in class variable and get it by getLastResponseBody() function.
This is needed for example to log response data.
Same functionality exists in:

PHP Soap: https://www.php.net/manual/en/soapclient.getlastresponse.php
Zend Framework: https://framework.zend.com/apidoc/1.12/classes/Zend_Http_Client.html#method_getLastResponse

Usage
$client = new \Iris\Client($login, $password, ['url' => 'https://dashboard.bandwidth.com/api/']);
$account = new \Iris\Account($your_account_id, $client);
$account->availableNumbers([ "areaCode" => "818" ]);
$lastResponse = $client->getLastResponseBody();

$lastResponse =
 <?xml version="1.0" encoding="UTF-8" standalone="yes"?><SearchResult><ResultCount>2</ResultCount><TelephoneNumberList><TelephoneNumber>8182132977</TelephoneNumber><TelephoneNumber>8182728306</TelephoneNumber></TelephoneNumberList></SearchResult>

Thanks @re8260!

@jhindman-bw Thanks !


Update README.md

Updated the example with the latest version of SpectaQL
Issue:
https://github.com/stepzen-dev/examples/issues/49
Changes:
Updated the guide with the latest version of SpectaQL i.e., 1.5.4
Reviewer:
@royderks

Hello @debxrshi thanks a lot for creating a PR.
To use the latest version of SpectaQL you don’t have to define this in the npm install script, npm will automatically use the latest stable version.
What I would expect for issue #49 is to regenerate the documentation for a GraphQL API, which would lead to an updated “build” directory.
The steps to generate docs with SpectaQL can be found in the readme for.
Let me know if this makes sense


Review Foundry series: 01-test-an-event-using-foundry

https://github.com/hashgraph/hedera-docs/pull/555

This is the gitbook preview link: https://hedera-dev.gitbook.io/preview-hedera-docs/~/revisions/J34eRQU4xi8U4qIboxQe/tutorials/smart-contracts/foundry/setup-foundry-and-write-basic-unit-test

This item has been reviewed and merged
Ref: #555


[Feat. Req.] Disable trigger from table detais, trigger list

I found no way to disable a trigger in table details (both selecting a trigger an looking at options downunder, or by right-click) or in trigger detail

Hello @wiwwo
What DBeaver version do you use?
What database version do you use?
Are you talking about this kind of operations? https://docs.microsoft.com/en-us/sql/t-sql/statements/disable-trigger-transact-sql?view=sql-server-ver15

Wow,
sorry :-P
DB: PG 13
DBEaver: Version <IP_ADDRESS>111281534
No way to disable trigger in red sport

or, after double click on trigger name in pic above:

Yeah, this is a little bit tricky:


Typos in the German version of the landing page

Music tab in the carousel: "blitz-schnell" should be "blitzschnell"
Midori tab in the carousel: "läd" should be "lädt"
Column about advertisement at the very bottom of the page: "Werbe-deals" should be "Werbedeals", "[...]wir sammeln keine sensible und persönliche Daten ein." should be "[..]wir sammeln keine sensiblen und persönlichen Daten ein". In fact, the whole column would sound much better in my opinion by rephrasing it to "Wir machen keine Werbedeals und sammeln keine sensiblen und persönlichen Daten. Unser Einkommen stammt direkt von unseren Nutzern."
Column about safety at the very bottom of the page: A comma is missing after "Software" and it should be "das" U.S. Department of Defense, but again the whole sentence sounds strange in my opinion. Much better would be: "Wir setzen auf Linux: Dieselbe Software, mit der unter anderem das U.S. Department of Defense und die Bank of China betrieben werden."

I just noticed that the translation is available on Transifex and most of the issues mentioned above have already been fixed there. The only thing left was part of the second bullet point which I just corrected there.

Closing this report, since as you said translations are done through Transifex :)


Update README

Changes:

Updated release shield to detect pre-release version
Removed build status
Added Getting started with IncludeOS development
Replaced Installing libraries with Dependencies
Removed install.sh notes
Moved/Updated conan/README.md notes to be in main README.

Code coverage: http://vaskemaskin.includeos.org:8080/coverage_includeos/hioa-cs-org-test/IncludeOS/PR-2071

Should we move the Getting started developing packages out of the README to readthedocs ?
please review:
@alfred-bratterud
@KristianJerpetjon
@mnordsletten

please review  commit 937bb1a on How to build and start a service.

Thanks @staiyeba - some minor comments from me. Do you want this merged or hanging until you've resolved the <!-- TODO's?

I can update some of the sections today.
I would like and some notes on the section:

How building the kernel with Conan works?
i think that's very relevant if we are releasing with conan.

I think now it is ready to be merged, please review again.
Just two #TODOs we want to update soon:

Building macos (line: 135)
How building with conan works? (line: 251)


Add ability to disable Windows 10 music controls

Hi!
While I understand why the app now supports the native music controls in Windows 10 (the ones that pup up next to the volume slider), I really dislike this popup. When I'm typing or something and pause my music using a media button on my keyboard, the stupid popup shows up and covers what I'm doing for a few seconds and I can't make it go away faster.
Being able to disable this would be awesome.
If is is already possible to disable this popup entirely, I'd love to hear it.

#2100 Fixed, just waiting for release.


[Windows] Search for Dlls in Bundles Properly

On Windows, dlls don't begin with "lib" so the tests should be searching
for "MyDLL.dll", not "libMyDLL.dll"

@swift-ci please test and merge


Request: move finished downloads

Hi and thanks for the great work!
It would be awesome to be able to define a folder for complete downloads.
This would be especially handy for the use of "whatcha folder" apps so they would not move incomplete files.

I think it's a good idea

No, aria2 doesn't provide a file operation interface, which is really a bad idea.
However, move after complete maybe not so bad, consider to raise issues at aria2/aria2


DOMException: The user aborted a request error thrown when reading cloned response

Describe the bug
I'll start by saying I'm not 100% sure that this is a bug, but rather that it is behaviour we are seeing which may not be intentional on urql's part.
We are using urql for our GQL client, and in one production instance of our code a 3rd party script is overwriting the window.fetch function, causing it to re-read a cloned version of the response so that it can scrape the content and do some stuff.  We are not in control of what this script does, but conceptually it is this:
var originalFetch=window.fetch;
window.fetch = function() {
  var fetch = originalFetch.apply(this, arguments);
  fetch.then(function(Response) {
    Response = Response.clone();
    return Response.text(); // perform another read of the response, via a cloned response object
  }).then(function(responseBody) {
    someCallbackWhichParsesTheResponseBody(responseBody);
  });
  return fetch;
}

(I'm not advocating the above logic; it is just what presents this issue)
Bearing in mind that urql will likely be doing a response.json() (or similar) read of the response stream, it seems that the act of "re-reading" the response (via the clone) triggers the above error.  My suspicion is that this relates to the behaviour of aborting in-flight requests which have a newer request triggered, although in this case, only one request is actually issued.... it is just being "read" twice.
Is the fact that the The User aborted a request error is thrown in this scenario valid, or is it something that should not happen?
Reproduction
https://codesandbox.io/s/loving-carlos-js83g6
Urql version
v4.0.0
Validations

[X] I can confirm that this is a bug report, and not a feature request, RFC, question, or discussion, for which GitHub Discussions should be used
[X] Read the docs.
[X] Follow our Code of Conduct

This pretty simply looks like your script there isn't respecting that fetch can throw and hence any error will show up as uncaught, which is correct behaviour by the browser. Specifically, when a request is aborted this may lead to an AbortError being thrown. At least, I'm pretty sure that's what's going on.
However, either way, since your error lies in your third-party script, it's not a bug in urql, and not something we can fix or take responsibility for 😅
Also, please upgrade before submitting issues. Your sandbox is using an outdated version and CodeSandbox is likely not resolving the latest version of @urql/core. Specifically, you'll likely want to have this fix so the error that's being caused isn't rethrown :v: https://github.com/urql-graphql/urql/blob/main/packages/core/CHANGELOG.md#patch-changes-4

Cheers.  Seems like Codesandbox only goes up to 4.0.0 🤔 but I'll try your suggestion in a local toy project.
Thanks for the quick reply.

Closing due to inactivity :v:


transformer-variant-group broken in 0.62

UnoCSS version
0.62
Describe the bug
Looks like transformer-variant-group is broken in 0.62.2 (with webpack/react at least)
Styles like b-(2 solid red) are not properly extracted when the project is built with webpack build
Reverting to 0.61.0 fixes this
Reproduction
Minimal reproduction repo: https://github.com/urzhumskov/unocss-trans-v-g
$ pnpm install
$ NODE_ENV=production webpack build
$ grep b-solid build/*.css

App.tsx contains a single div with className="b-t-blue b-(2 solid red)" -- while the b-t-blue is properly extracted during build, the expected b-2 b-solid b-red classes are missing from the resulting CSS
Simply reverting everything uno-related to 0.61.0 seems to fix this:
$ pnpm add<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>$ NODE_ENV=production webpack build
$ grep b-solid build/*.css

System Info
No response
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This broke in 0.61.7, not sure why but my guess is https://github.com/unocss/unocss/pull/3814.


Unclear sentence under unsupported scenarios

This sentence was really hard to understand. Couldn't find a better way to phrase it myself but it should be a little more clearer:
"Azure AD Domain Services needs Password Hash Synchronization to be enabled on the tenant. Therefore tenants that use Pass-through Authentication only don't work for scenarios that need Azure AD Domain Services."

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

ID: 12cbaf93-735d-e257-9203-0437bdf9d1ea
Version Independent ID: f12c0f8c-86e3-72f6-1a3c-c20ad6654017
Content: Azure AD Connect: Pass-through Authentication - Current limitations
Content Source: articles/active-directory/hybrid/how-to-connect-pta-current-limitations.md
Service: active-directory
GitHub Login: @billmath
Microsoft Alias: billmath

@roberthstrand Thank you for the valuable feedback,we are investigating the issue.

The statement means is that Azure AD Domain Services only work with Password Hash Synchronization not with PTA. Hence, tenants that have been configured with PTA would not work for Azure AD Domain Services.

We will be closing this thread. Let us know if you need further assistance.


Track latest nightly regression async_trait

Describe the bug
fixed by patching nightly version in CI #642

investigate switch to stable, likely blocked by GATs?
switch to nightly channel once resolved

Steps to reproduce

Node logs
-

Platform(s)
No response
Code of Conduct

[X] I agree to follow the Code of Conduct

I think we actually don't need nightly at all? The GATs we use are the stabilized one, and I've been building reth locally on stable.

ref #640

@gakonst we need nightly for fmt & maybe clippy

Yep but that's fine - my point being is we're not blocked on GATs or any nightly feature to build.


Deprecated pyproj argument

At this line, the argument skip_equivalent of pyproj.transformer.from_crs is deprecated since version 3.1 and removed since 3.4.0 (see here) .
It's a bit unclear to me if skip_equivalent is the default now... The change in pyproj is here: https://github.com/pyproj4/pyproj/pull/824

Yes, it is now the default! So nothing to do except remove, will close this in the next PR.


502 error while trying to create new Node Template

I'm attempting to get this set up and am running into 502 errors while creating my node template. I've tried this with the IP of my proxmox node and also the DNS name on my network for it with no luck. I'd be happy to contribute by writing a README about the node template, but I'm not sure how it works.

seems it is no longer compatible/works with current versions of rancher. been noticing this since Rancher 2.9 early releases and seems the dev has dropped the project and is no longer maintaining it


Scroll problem with onTimeChange - Limiting visible range view

I have a problem that I want to limite visible range view using the follow code:
`
const onTimeChange = (
visibleTimeStart,
visibleTimeEnd,
updateScrollCanvas
) => {
const minDate = filter.startDate.clone().startOf("day").valueOf();
const maxDate = filter.endDate.clone().endOf("day").valueOf();
if (visibleTimeStart < minDate && visibleTimeEnd > maxDate) {
  updateScrollCanvas(minDate, maxDate);
} else if (visibleTimeStart < minDate) {
  updateScrollCanvas(
    minDate,
    minDate + (visibleTimeEnd - visibleTimeStart)
  );
} else if (visibleTimeEnd > maxDate) {
  updateScrollCanvas(
    maxDate - (visibleTimeEnd - visibleTimeStart),
    maxDate
  );
} else {
  updateScrollCanvas(visibleTimeStart, visibleTimeEnd);
}

};
`
The results is bellow:

Left side - The Header keep fixed when the body is scrolling

Right side - The body finish before the header

Anyone to help in this bug.
I suggest pass the start and end datetime to limite the range view.

Any workaround?, I have the same issue

Came up with a workaround based on looking up the internal calculations:
`onTimeChange={(visibleTimeStart, visibleTimeEnd, updateScrollCanvas) => {
let zoom = visibleTimeEnd - visibleTimeStart; // buffer - 1 (1 is visible area) divided by 2 (2 is the buffer split on the right and left of the timeline)
          const minTimeAdjusted = minTime + zoom * (buffer - 1) / 2; // Hack to prevent scroll past the minTime
          const maxTimeAdjusted = maxTime - zoom * (buffer - 1) / 2; // Hack to prevent scroll past the maxTime

          if (visibleTimeStart < minTimeAdjusted && visibleTimeEnd > maxTimeAdjusted) {
            updateScrollCanvas(minTimeAdjusted, maxTimeAdjusted, true)
          } else if (visibleTimeStart < minTimeAdjusted) {
            updateScrollCanvas(minTimeAdjusted, minTimeAdjusted + zoom, true)
          } else if (visibleTimeEnd > maxTimeAdjusted) {
            updateScrollCanvas(maxTimeAdjusted - zoom, maxTimeAdjusted, true)
          } else {
            updateScrollCanvas(visibleTimeStart, visibleTimeEnd)
          }
        }}`

Hey, @rezgar, what is the buffer in your workaround?


WIP - Fixes #1896 - Campaign Access Lister

Hoping this addresses Campaign Access Lister #1896.
I added a method to the CampaignController to access the authorized users -Admin/Campaign/[campaignId]/Users.
Being relatively new to this problem domain, I made some assumptions.  Hopefully correctly 😄.

assumed that a campaign may have more than on manager
assumed that an event may have more than on manager

@stevejgordon Am I on the right track? If so, I can apply the same pattern to #1900

@c0g1t8 Looks like you're on the right path here and your assumptions are correct.

@c0g1t8 Is this one ready for a final review?

@stevejgordon I've been looking into #1900 as well and can see that it has a lot in common with this issue. This makes me want to re-think what I've done in PR #1965. I'm now thinking there are two other options.

Create a new User controller and provide a partial view that can be called from the Campaign or Event controller.

Create a partial view associated with the Organization controller that can be called by the Campaign or Event Controller.

I'm favoring the second option. I am assuming that a user/volunteer may be assigned to multiple events in multiple campaigns.

Thoughts?

I made this comment originally in #1896 a couple of weeks ago.  😏  I'm still not comfortable enough with the domain model to decide.


Add a SetRegistryValue MSBuild task to DeploymentFramework.BuildTasks.dll

Add a SetRegistryValue MSBuild task to DeploymentFramework.BuildTasks.dll. This can be used in any .btdfproj file (as can the SDC Tasks).
This work item was migrated from CodePlex
CodePlex work item ID: '6303'
Assigned to: 'tfabraham'
Vote count: '0'

[UnknownUser@4/12/2010]
Resolved with changeset 39648.

[UnknownUser@2/21/2013]

[UnknownUser@5/16/2013]


How to prevent from modifying players with certain group.

I'm trying to set Moderator group, so it can't change group of player with Admin group, but it's not working.
To do that I'm using Argument based command permissions and Moderator permission looks like that.
luckperms.user.parent.set
luckperms.user.parent.set.modify.others
luckperms.user.parent.set.builder
And now - What should I add to make it work as I want it to ?

How is it happens that nobody ever aswers on my questions. ?

Using verbose mode is really the only way to do this properly

https://github.com/lucko/LuckPerms/wiki/Argument-based-command-permissions

did you enabled arguement-based in luckperms config.yml
use verbose to see the perms https://github.com/lucko/LuckPerms/wiki/Verbose

Yes I did
I do

please show your verbose output when doing the relevant commands.

Pay particularly close attention to this part here: https://github.com/lucko/LuckPerms/wiki/Argument-based-command-permissions#checks-when-a-player-makes-changes-in-a-specific-context

I've read that

and all permission from verbose are there exept for
luckperms.user.parent.set.usecontext.global
and that's all

do send a screenshot of the verbose output.

I dont see any issue, if you don't want to be able to add someone to admin, then set luckperms.user.parent.set.admin to false.

but just prevent form giving this group

and so u know how I've don it ^

I can't believe there is no way to do that...

I mean that's literally the purpose of this feature. Your verbose logs show that everything is true so maybe you've set a wildcard permission somewhere or maybe you're even opped (always deop yourself when doing any permission debugging).

@Turbotailz I think what he wants like if i am a mod, I caant touch the permissions for users in admin group. For me id suggest using tracks instead of parent  set command.

Okey so I'll use trakcs then
@benwoo1110  and that's exactly what I meat..
Thanks guys for help


Switch login window 3.1

Once Google OIDC stuff is removed from teh profile XCreds defaults to it's own login window which is fine.  I'd like to have the option of both though with a profile key to specify Google/Azure login, XCreds AD login, or mac login window by default.  Also when the Google info is removed the switch login window button never shows the Apple login window.

Unnecessary.  Didn't see that macos login window will not be switched to now as of 3.1.


Gekkobot - backtest with nogui mode doesn't work

I have the best config:

But if I try backtest in cmd (node gekko.js --config config.js --backtest), I got this:

My config.js:
config.watch = {
  exchange: 'poloniex',
  currency: 'USDT',
  asset: 'BTC',
}

config.tradingAdvisor = {
  enabled: true,
  method: 'RSI',
  candleSize: 20,
  historySize: 10,
  adapter: 'sqlite',
  talib: {
    enabled: true,
    version: '1.0.2'
  }
}

config.RSI = {
  interval: 12,
  thresholds: {
    low: 30,
    high: 70,
    persistence: 1
  }
};

Can you post your full config? The screenshot you posted is talking about the telegram plugin, which tells me this is not your full config.

@askmike http://wklejto.pl/284562

You need to have the pluginPaper trader enabled.

Still doesn't work.

What does not work? Could you turn on debug and post full output?

stdout.txt

Gekko is not a fully automatic bot that you can turn on and in it will trade for you. You need to create  your own strategies (or properly configure the ones that come with Gekko). The logs show that everything is running but Gekko is waiting for advices from the (by you configured RSI) strategy. I suggest tweaking the paramteres or running over a different market.

But on the web simulation with the same settings sells/buys.

Could you let me know exactly what market and what timeframe this happens
at?
On Wed, Jun 7, 2017 at 11:56 AM, Michał Rosiak<EMAIL_ADDRESS>wrote:

But on the web simulation with the same settings sells/buys.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/771#issuecomment-306760900, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA7MDxJW8QXt771ksX9_o1oyjCJ7UqSBks5sBoHFgaJpZM4NxQFf
.

--
PGP key at keybase.io/mikevanrossum
https://keybase.io/mikevanrossum/key.asc


Ensure check argument is passed through correctly

Resolves #26

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.92%. Comparing base (3750bc2) to head (55fa742).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #27   +/-   ##
=======================================
  Coverage   97.92%   97.92%
=======================================
  Files           7        7
  Lines         965      965
=======================================
  Hits          945      945
  Misses         20       20

Files
Coverage Δ

interpax/_ppoly.py
96.93% <100.00%> (ø)


Update sheet.json for fix Dark mode issue

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] I have authorization from the game's publisher to make this an official sheet on Roll20 with their name attached.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

changed the legacy option to false in the json files as it fix the same dark mode issue i had on my sheet who base on this one

Character Sheet Info Roll20 Internal Use only.


[WIP][SPARK-35275][CORE] Add checksum for shuffle blocks and diagnose corruption

What changes were proposed in this pull request?

This PR proposes to add checksum support for shuffle blocks. The basic idea is:
On the mapper side, we'll wrap a CheckedOutputStream upon the FileOutputStream to calculate the checksum (use the same checksum calculator Adler32 with broadcast) for each shuffle block (a.k.a partition) at the same time when we writing map output files.  And similar to the index file, we'll have a checksum file to save these checksums.
On the reducer side, we'll also wrap a CheckedInputStream upon the FileInputStream to read the block.  When block corruption is detected, we'll try to diagnose corruption for the cause:
First, we'll use the CheckedInputStream to consume the remaining data of the corrupted block to calculate the checksum (c1);
Second, the reducer send an RPC request called DiagnoseCorruption (which contains c1) to the server (where the reducer executed)
Third, the server will read (using a very small memory) the corresponding block back from the disk and calculate the checksum (c2) again for it. And also read back the checksum(c3) of the block saved in the checksum file. Then, if c2 != c3, we'll suspect the corruption is caused by the disk issue. Otherwise, if c1 != c3, we'll suspect the corruption is caused by the network issue. Otherwise, the cause remains unknown. The server then will reply to the reducer with CorruptionCause containing the cause.
Fourth, the reducer needs to take action after it receives the cause. If it's a disk issue or unknown, it will throw fetch failure directly. If it's a network issue, it will re-fetch the block later. Also note that, if the corruption happens inside BufferReleasingInputStream, the reducer will throw the fetch failure immediately no matter what the cause is since the data has been partially consumed by downstream RDDs.  If corruption happens again after retry, the reducer will throw the fetch failure directly this time without the diagnosis.
Overall, I think we don't introduce severe overhead with this proposal. In a normal case, the checksum is calculated in the streaming way as well as other streams, e.g., encryption, compression. And the major overhead here is that we need an extra data file traverse in the error case in order to calculate the checksum (c2).
And the proposal in this PR is much simpler compared to the previous one https://github.com/apache/spark/pull/15894/ (abandoned due to complexity ), which introduce more overhead as it need to traverse the data file twice for every block. In that proposal, the checksum is appended to each block data, so it's also invasive to the existing code.
Why are the changes needed?

Shuffle data corruption is a long-standing issue in Spark. For example, in SPARK-18105, people continually reports corruption issue. However, data corruption is difficult to reproduce in most cases and even harder to tell the root cause. We don't know if it's a Spark issue or not.  With the checksum support for the shuffle, Spark itself can at least distinguish the cause between disk and network, which is very important for users.
Does this PR introduce any user-facing change?

Yes.

Added a conf spark.shuffle.checksum to let user enables/disables the checksum (enabled by default)
With checksum enabled, users can know the possible cause of corruption rather than "Stream is corrupted" only.

How was this patch tested?

Added an end-to-end unit test in ShuffleSuite.
I'll add more tests if the community accepts the proposal.

Test build #138049 has started for PR 32385 at commit c09ccd0.

I marked PR as WIP because I want to hear the community's feedback before working further, e.g., adding more unit tests.
And there will be two following PRs (if the community accept this proposal):

Corruption diagnosis support for the batch fetched block

Corruption diagnosis implementation for the external shuffle service

cc @mridulm @otterc  @attilapiros @tgravescs @cloud-fan Please take a look, thanks!

Test build #138049 has finished for PR 32385 at commit c09ccd0.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138049/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42568/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42568/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/42568/

+CC @otterc This should be of interest given recent discussions.

Thanks for copying me. I will take a look at it in few days.

Hi all, to ease the review for everyone, I have planned to split these PRs into 2 smaller PR first:

calculate checksums and save them into a checksum file

(I have created the PR (https://github.com/apache/spark/pull/32401) for this part so you can start reviewing from there.)

use the checksum file to diagnose the corruption.

(I'll create the PR later)

I haven't had time to look in detail, if this is a working prototype, have you done any performance measurements to see example impact this has?  I know you said it should be minimal but some numbers would be nice.
Also at the high level how does this affect other shuffle work going on - like merging and pluggable?  Is it independent of that or would need to be reimplemented?

@tgravescs Thanks for the good points!
I did find some perf regression by benchmarking with the change. I'll double-check it for sure and try to get rid of it if possible.

Also at the high level how does this affect other shuffle work going on - like merging and pluggable? Is it independent of that or would need to be implemented?

For merging, it needs extension to send checksum values along with the block data while merging. The extension is also needed for the decommission feature.
For pluggable, my current implementation is added at LocalDiskShuffleMapOutputWriter, which is supposed to be the default shuffle writer plugin for Spark. It means, in this way, other custom plugins needs its own implementation for checksum support. I adopted that way becase I realized it's easier  and more clear to implement at that time.
An alternative way to support checksum for all plugins or say to make it a built-in feature maybe is to implement it in DiskBlockObjectWriter/ShufflePartitionPairsWriter, which is the upstream to the shuffle I/O plugin. I need more investigation on this.

Hi @tgravescs @mridulm @otterc , I have resolved the regression issue and made the checksum as a built-in feature of Spark.
And I have updated PR #32401 (which adds checksum support at shuffle writer side only) and I think it's ready for review.

thanks @Ngone51, I'm very busy this week, so will take a look early next week.

Sure, take your time :)

oh..@mridulm Sorry if I confused you here. I have planed to split this PR into two separate PRs to ease the review:

write checksum file (Ready to review https://github.com/apache/spark/pull/32401)
diagnose corruption (Not done yet)

So please help review the smaller PR there.
And I'll try to resolve your comments in the separate PRs. Thanks!

lol, thanks for the links @Ngone51  :-)
Glad I went through this once more anyway - will help me with better understanding of the sub-pr's !

Hi @Ngone51, thanks for providing the checksum feature for shuffle, have a thought about the following case.

if the corruption happens inside BufferReleasingInputStream, the reducer will throw the fetch failure immediately no matter what the cause is since the data has been partially consumed by downstream RDDs.

I think the checksum could handle this case. Pick the idea from #28525, we can consume the input stream at the beginning and verify the checksum immediately, then

If it's a disk issue or unknown, it will throw fetch failure directly. If it's a network issue, it will re-fetch the block later.

To achieve this, we may need to update the shuffle network protocol to support passing checksums when fetching shuffle blocks.
Compare to the existing Utils.copyStreamUpTo(input, maxBytesInFlight / 3), this approach use less memory and can verify any size of blocks, but it introduce another overhead because it need to read the data 2 times.
We encounter this issues in our production everyday, for some jobs, the performance overhead is acceptable comparing to stability.
@Ngone51 WDYT?


KAFKA-15776: Support added to update remote.fetch.max.wait.ms dynamically

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

This patch also depends on #16199

A few test failures that are unrelated to this change, will merge it to trunk and 3.8 branches.

@kamalcph Can you raise a PR against 3.8 branch as I would like those changes to be reviewed in PR as it has conflicts with that branch?

@kamalcph Can you raise a PR against 3.8 branch as I would like those changes to be reviewed in PR as it has conflicts with that branch?

Opened #16275 and #16276 to cherry-pick the KAFKA-15776 feature to the 3.8 branch. PTAL.