Version increment command for selected targets.

Having a command that allows for incrementing the version of a specific plugin would help with how we define and perform semver increments.
Example Command Usage
Patch
build-tools version patch
# 1.2.2 → 1.2.3

Minor
build-tools version minor
# 1.2.3 → 1.3.0

Major
build-tools version major
# 1.3.0 → 2.0.0

Beta
build-tools version major:beta
# 2.0.0 → 3.0.0-beta.1

build-tools version beta
# 3.0.0-beta.1 → 3.0.0-beta.2

Increment Multiple
build-tools version minor my-plugin,my-theme
# my-plugin: 2.0.0 → 2.1.0
# my-theme: 3.4.0 → 3.5.0

@ampersarnie Should this have the same functionality as NPM-VERSION i.e | major | minor | patch | premajor | preminor | prepatch | prerelease | from-git] ?
I noticed npm version is only compatible with package.json files so couldn't even wrap this in it as it wouldn't work with PHP files.

Yeah they should, however, some of the naming between what was listed in the description and npm doesn't match word for word but are the same. i.e. premajor = major:beta
You can ignore the from-git version. To help with that comparison, here's what you can use:

NPM/semver
Build Tools
Example Result (x|y)

major
major
x.0.0

minor
minor
1.x.0

patch
patch
1.0.x

premajor
major:beta
x.0.0-beta.y

preminor
minor:beta
1.x.0-beta.y

prepatch
patch:beta
1.0.x-beta.y

If you're unsure on what these versions mean, take a look at our internal RFC on Semantic Versioning.

@ampersarnie Another question on this sorry! If the version of a file is for example 1.0.0-alpha.1 if I then run build-tools version minor:beta is it expected that this then changes to 1.1.0-beta.0 ?
In your example when would the Y value change?


Adds Phoenix.Param implementation

If not implemented, you can stumble upon this kind when using TypeID:
** (ArgumentError) structs expect an :id key when converting to_param or a custom implementation of the Phoenix.Param protocol (read Phoenix.Param docs for more information), got: #TypeID<"booking_01h8xpqbz6f1p97w7rgqe5bvxh">

Ah! I just saw that the protocol was already implemented, however I still get this error above. Will dig more into it, sorry!


CB-2970. Telemetry publisher install fails with: account already exists error message

if cm cluster install flow step re-runs for some reason, it will try to re-create altus external account with the same name as it was created, so that api call can fail
can be solved easily just create an account with an additional uuid string

seems it's failed on an unrelated test, will re-trigger UT

retest this please


ApplicationProtocolNegotiationHandler doesn't work with SniHandler

Motivation:
ApplicationProtocolNegotiationHandler attempts to get a reference to an SslHandler in handlerAdded, but when SNI is in use the actual SslHandler will be added to the pipeline dynamically at some later time. When the handshake completes ApplicationProtocolNegotiationHandler throws an IllegalStateException because its reference to SslHandler is null.
Modifications:

Instead of saving a reference to SslHandler in handlerAdded just search the pipeline when the SslHandler is needed

Result:
ApplicationProtocolNegotiationHandler support SniHandler.
Fixes https://github.com/netty/netty/issues/5066

@trustin @normanmaurer @rkapsi - PTAL

Cherry-picked into 4.0 (2263d9b46b658641cbecdad97a1edb930ca71914) and 4.1
(fcbeebf6dff759a1e0cfe2e20fe5103e4c40075c)
@Scottmitch thanks!


[BUG] "Must supply freq for datetime value" in forecasting pipeline

posted by @anthonygiorgio97 in #2452
@fkiraly ok I have tried to reverse pipelines. Now if I use a make_reduction model like SVR() as follow, the pipeline work.
from sktime.transformations.series.boxcox import LogTransformer
from sktime.forecasting.compose import ForecastingPipeline, TransformedTargetForecaster
from sktime.transformations.series.compose import OptionalPassthrough
from sktime.forecasting.fbprophet import Prophet
from sktime.transformations.series.difference import Differencer
from sktime.forecasting.model_selection import ExpandingWindowSplitter, temporal_train_test_split, ForecastingGridSearchCV
from sktime.forecasting.compose import make_reduction
from sklearn.svm import SVR
from sktime.datasets import load_longley
y, X = load_longley()

#split train/test both y and X
fh = [1,2]
train_model, test_model = temporal_train_test_split(y, fh = fh)
X_train = X[X.index.isin(train_model.index)]
X_test = X[X.index.isin(test_model.index)]

#pipeline
pipe = TransformedTargetForecaster(steps = [
            ("log", OptionalPassthrough(LogTransformer())),
            ("differencer", Differencer(drop_na = True)),
            ("myforecasterpipe", ForecastingPipeline(steps = [
                ("logX", OptionalPassthrough(LogTransformer())),
                ("differencerX", Differencer(drop_na = True)),
                ("myforecaster", make_reduction(SVR(), window_length = 3))
        ]))
])

#cv setup
N_cv_fold = 1
step_cv = 1
cv = ExpandingWindowSplitter(initial_window = len(train_model) - (N_cv_fold - 1) * step_cv - len(fh),
                            start_with_window = True,
                            step_length = step_cv,
                            fh = fh)

param_grid = [{"log__passthrough": [False],
               "myforecasterpipe__logX__passthrough": [False]
              },
              {"log__passthrough": [True],
               "myforecasterpipe__logX__passthrough": [True]
              }
             ]

#grid search
gscv = ForecastingGridSearchCV(
            forecaster = pipe,
            cv=cv,
            param_grid=param_grid,
            verbose = 1
        )

#fit
gscv.fit(train_model, X = X_train)

However if I want to use Prophet now I face up with another error. This is the sample code I'm using:
from sktime.transformations.series.boxcox import LogTransformer
from sktime.forecasting.compose import ForecastingPipeline, TransformedTargetForecaster
from sktime.transformations.series.compose import OptionalPassthrough
from sktime.forecasting.fbprophet import Prophet
from sktime.transformations.series.difference import Differencer
from sktime.forecasting.model_selection import ExpandingWindowSplitter, temporal_train_test_split, ForecastingGridSearchCV
from sktime.datasets import load_longley
y, X = load_longley()

#split train/test both y and X
fh = [1,2]
train_model, test_model = temporal_train_test_split(y, fh = fh)
X_train = X[X.index.isin(train_model.index)]
X_test = X[X.index.isin(test_model.index)]

train_model.index = train_model.index.to_timestamp(freq='A')
X_train.index = X_train.index.to_timestamp(freq='A')
X_test.index = X_test.index.to_timestamp(freq='A')

#pipeline
pipe = TransformedTargetForecaster(steps = [
            ("log", OptionalPassthrough(LogTransformer())),
            ("differencer", Differencer(drop_na = True)),
            ("myforecasterpipe", ForecastingPipeline(steps = [
                ("logX", OptionalPassthrough(LogTransformer())),
                ("differencerX", Differencer(drop_na = True)),
                ("myforecaster", Prophet())
        ]))
])

#cv setup
N_cv_fold = 1
step_cv = 1
cv = ExpandingWindowSplitter(initial_window = len(train_model) - (N_cv_fold - 1) * step_cv - len(fh),
                            start_with_window = True,
                            step_length = step_cv,
                            fh = fh)

param_grid = [{"log__passthrough": [False],
               "myforecasterpipe__logX__passthrough": [False]
              },
              {"log__passthrough": [True],
               "myforecasterpipe__logX__passthrough": [True]
              }
             ]

#grid search
gscv = ForecastingGridSearchCV(
            forecaster = pipe,
            cv=cv,
            param_grid=param_grid,
            verbose = 1
        )

#fit
gscv.fit(train_model, X = X_train)

The error is this:
ValueError: Must supply freq for datetime value
It seems strange because freq is set as 'A' before fitting for train_model and X_train
Originally posted by @anthonygiorgio97 in https://github.com/alan-turing-institute/sktime/issues/2452#issuecomment-1100713332

FYI @anthonygiorgio97, moved one of the unresolved bugs to a new issue.

changing the drop_na=True to na_handling="fill_zero" does not help and indicates that the problem is probably not that too short series are being created

So, @ohbtorres, @anthonygiorgio97, I've understood what the problem was, here.#
Long story short, pandas has been slowly changing how they handle freq in indices, deprecating it in various places, including pd.Timestamp. That was breaking functionality around ForecastingHorizon where we assumed we have access to freq.
The latter was a larger issue, which, for ForecastingHorizon, has been fixed and future-proofed by a series of PR by @khrapovs.
With that now in, we can fix the issue here easily (I hope), here:
https://github.com/alan-turing-institute/sktime/pull/3007

As far as I understand, ForecastingHorizon was used a utility to convert integer offsets into time stamps by @RNKuhns, in a transformer. Since we are not really forecasting, it might be worth thinking about whether it makes sense to factor this out into a utility, and use it from both here and the ForecastingHorizon?
Thoughts about that, @RNKuhns, @khrapovs? Might also relate to our recent discussions on the splitters, @khrapovs, another location where this problem appears.

FYI @anthonygiorgio97, fixed now. Was coming from the Differencer, see #3007.


The link in adsorbml/scripts /README.md/How to run AdsorbML/Step 1 is invalid

Hello,
I was trying to run this project and wanted to check the instructions, however, I found that the link in adsorbml/scripts /README.md/How to run AdsorbML/Step 1: Run ML relaxations on the LMDB to generate predictions of relaxed energy and relaxed structures. For example, see these instructions. The corresponding link is invalid now. Thanks for your help!
Best,
Zebin

I think the link used to point here:
https://github.com/FAIR-Chem/fairchem/blob/v0.1.0/TRAIN.md


Support XGBoost distributed training and batch predication on Kubernetes

What is proposed in this PR ?
The readme tutorial to train XGBoost job on Kubernetes cluster is added.
How to test this PR ?
Just readme, and related tutorial is linked.

@CodingCat and @hcho3 can you review this ?

@yinlou  thanks for your comments, it is fixed.

@merlintang I made minor edits for greater clarity. Can you review?

Merging this for now. Feel free to open a new pull request to fix typos etc.

thanks for Cho, I would update the typo later.
On Sun, Jul 14, 2019 at 11:28 PM Philip Hyunsu Cho<EMAIL_ADDRESS>wrote:

Merged #4621 https://github.com/dmlc/xgboost/pull/4621 into master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/xgboost/pull/4621?email_source=notifications&email_token=AAK5R6JRYFR7FHBEVNE3HZ3P7QKIPA5CNFSM4H4I4S7KYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOSPXIALY#event-2481881135,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAK5R6IPWYWZ3BZ4TNH4YJDP7QKIPANCNFSM4H4I4S7A
.

@merlintang Here's is the official link: https://xgboost.readthedocs.io/en/latest/tutorials/kubernetes.html. Thank you for your contribution.


WIP: Add a schema customizer to add support for jackson polymorphism

Currently not running, because the schema resolver ends in a endless loop (schema resolver calls the schema customizer again)

Closed in favor of #22


ci: Fix focus handling in ginkgo-ext

There was a bug in ginkgo-ext, which caused focused test to run, but not
run AfterAll blocks for them. This change causes our focus matching
logic to take whole scope (including all parent context/describe blocks)
and properly match test cases.

test-me-please

Coverage increased (+0.007%) to 37.023% when pulling 65af7891307ec5724bcb8a5894e03085b80e8613 on pr/fix-ginkgo-ext into 969600f450abb678b4cfb95075743cc8af86e3ed on master.

test-me-please

retest-runtime

retest-4.9

retest-net-next

retest-gke

retest-runtime

retest-gke

I'm going to cherry-pick this on #11553

test-me-please

test-me-please

retest-runtime

retest-net-next

retest-4.9

retest-4.9


Fixing #644

Multispan.CopyTo should take a T[], not byte[]

@jamesqo, thanks for catching this.

@KrzysztofCwalina Sure, no problem. :smile:


Feature: Use the Color Scheme customisation option

Hello Germán,
first let me express my gratitude, that you picked up the old Color Highlighter plugin and extended it. Love it very much, especially the possibility to see colours in other filet types (e. g. Markdown).
I'm currently in the progress of rewriting my colour scheme and while I love to see the color values, it's kind of distracting to have the ColorHighlight additions written in my .sublime-color-scheme file, while I'm writing it … 😉
Sublime Text has a feature to "customize a color scheme by overriding variables or globals, and adding rules" by creating "a new file with the same filename as the color scheme, but" saving "it in the Packages/User/ directory". That way any original color scheme files stay the same as they are and your needed scopes & rules would be easy to use.  (for more information see Sublime Texts Color Scheme Documentation).
Thanks for considering and take care
Mick

This is interesting... instead of modifying the original, one can modify one file  in Packages/User/? I'll try that to see how it works.
Do files in User Directory need to be always JSON or in the same type as the original?
If you can, please add a pull request... it may be some time before I can dig into this, I’ve been way too busy lately.

Do files in User Directory need to be always JSON or in the same type as the original?

You can override/extend tmTheme color schemes using the new JSON format.


[nat64] update counter types from uint64_t to int64_t

We may want to avoid the use of uint64_t types as int64_t won't overflow for most cases.
This PR update the type of counters for nat64 to int64_t.
Some considerations: almost every existing API is using uint32_t / uint64_t for counters, should we still follow the existing pattern?

Close the PR based on discussion. Further discussion required for the types for similar fields in the future.


Not connecting to RPC Daemon

Hi,
my monero-wallet-rcp and monerod daemon is running. i change the .env many times. I always get monero addresses with XXXXXXXXXXX.....
is there a way to generate addresses from monero-wallet-rpc?
{"id":"tJpdPTub4mGRFnX6","address":"4AXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX"}
Kind Regards

I think you've accidentally set CRYPTO_MOCK to true

I think you've accidentally set CRYPTO_MOCK to true

nice opsec


Migrating AceEditor to Monaco

Monaco is a better code editor and more popular in the VSCode development community

Sorry, we are not interested to update this component at this moment :( maybe in the future we can discuss about this topic again.
Thanks for understanding


More complete table for CSP 2.

I finished testing the remaining browsers. Please let me know if there's anything else needed to toggle "shown:true". The tests are run against http://csp.metaltoad.com/. Thanks!

Nice! I'll look through it and let you know.

@dylantack Okay, I've gone ahead and published it on the site. For future reference, numbered notes should be combined when possible (I did this for the Chrome & Opera ones) and including the browser names & version numbers is not strictly necessary since the UI already highlights them. Sometimes I do include them for clarity though, so I'm fine leaving them.
Oh, one question...it seems Firefox for Android (38) should match Firefox 38 on desktop, when I run the tests it seems it does. Can you confirm?
Thanks again!

Whoops, you're right about FF38. Thanks for combining the notes. Some of those intermediate versions appear to have implemented CSP2 piecemeal, so things got a little messy.


python3 SyntaxError

λ caterpillar -h
Traceback (most recent call last):
  File "d:\python3\lib\runpy.py", line 184, in _run_module_as_main
    "__main__", mod_spec)
  File "d:\python3\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "D:\python3\Scripts\caterpillar.exe\__main__.py", line 5, in <module>
  File "d:\python3\lib\site-packages\caterpillar\caterpillar.py", line 46
    """
      ^
SyntaxError: invalid syntax

Line 46 of caterpillar.py is the end of an f-string [1], available starting from Python 3.6. It appears you're using Python 3.5 or earlier. README does say Python 3.6 or later required. You can check your Python version with python3 --version, or, using the pip with which you installed caterpillar-hls: pip --version.
I should probably add a version check to setup.py to fail outright on Python 3.5 or earlier.
[1] https://docs.python.org/3/whatsnew/3.6.html#whatsnew36-pep498

Of course, feel free to ask questions although the issue has been closed.

setup.py improved in 07ad4afce29aaab8e053c30d46d6f28692c5d009, and released in v1.0.1. With a not-too-outdated version of pip, it should now refuse to install caterpillar-hls outright on Python <3.6.


use both package.json and .babelrc

Describe the bug
It doesn't appear you can use the babel key in the package.json, otherwise you get errors.  It seems you are only allowed to use a .babelrc.
System:

OS: [e.g. iOS, Windows10, MacOS] unrelated
Device: [e.g. iPhoneX, Macbook Pro 2018] unrelated
Browser: [e.g. chrome, safari] unrelated
Framework: [e.g. react, vue, angular] react
Addons: [if relevant] unrelated
Version: [e.g. 4.0.0] 3.4.11

Additional context
Add any other context about the problem here.

With the v4 you can


no result for SpecHLA_single

I use the SpecHLA_single (Pacbio CCS reads) to do HLA type,  but no result ouput . If the coverage of data was two slow ?

Hello,
Up to now, SpecHLA does not support only use third-generation data. The NGS data is necessary for typing. Then other data can be used to improve the NGS-based results.
Thanks for your usage.
Shuai

Hi, shuai:
I got this error:
processing reads mapped to chrom "HLA_A"
run SpecHap ...
input_rephase_para:200  1
Traceback (most recent call last):
File "single_tgs.py", line 1182, in
os.system(str(reph))
File "single_tgs.py", line 433, in newphase
phased_locus=seq[he]
IndexError: index 0 is out of bounds for axis 0 with size 0
empty for $HLA_A_break_points_spechap.txt HLA_A_break_points_phased.txt
$cat HLA_A_break_points_spechap.txt
#gene   locus   00      01      10      11      points_num      next_locus
next, I will use high coverage PacBio CCS data and NGS data for test.

Or  do you have any tools for third-generation data only HLA typing.  Thanks.

Hi,
As far as I am concerned, there is no tool just use PacBio CCS data only. I know a paper used PacBio HiFi data, but they have not provided the code.
Please let me know if you face any problems running SpecHLA based on NGS and TGS data.

As I know HLA-LA support PacBio CCS data only.
My problems is single_tgs.py reported erro when running SpecHLA, as you mentioned it can not use CCS data only.
And I will add NGS data to do hla typing, and what`s the title of the paper which have had do typing use PacBio HiFi data ?
Thank you !

Hello,
The paper named "Chin, Chen-Shan, et al. "A diploid assembly-based benchmark for variants in the major histocompatibility complex." Nature communications 11.1 (2020): 1-9." uses PacBio HiFi data to assemble the whole MHC region.
Thank you for your information, we will solve the bug using TGS data only, and compare the performance to HLA-LA.
Best,
Shuai

Thank you, I am looking forward to your new work.
Good Luck!
------------------ 原始邮件 ------------------
发件人:                                                                                "deepomicslab/SpecHLA"                                                                                @.>;
发送时间: 2022年3月15日(星期二) 下午2:30
@.>;
@.***>;
主题: Re: [deepomicslab/SpecHLA] no result for SpecHLA_single (Issue #1)
Hello,
The paper named "Chin, Chen-Shan, et al. "A diploid assembly-based benchmark for variants in the major histocompatibility complex." Nature communications 11.1 (2020): 1-9." uses PacBio HiFi data to assemble the whole MHC region.
Thank you for your information, we will solve the bug using TGS data only, and compare the performance to HLA-LA.
Best,
Shuai
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

Hello,
I have revised the TGS-only mode of SpecHLA, and now it works. To use this mode, Please remember to install Longshot 0.4.1 and add it to the system path.
Please let me know if there are other problems.
Sincerely,
WANG Shuai

Thank You ！

Hello：
Do you finished your TGS mode only HLA typing ?
I have had noticed that you updated some script for HLA assemble.
Thank you!

There is no pg_run.py in your git.

Hello,
Apologies for the delayed response. I'm glad to inform you that we have made significant improvements to the software. The latest version now offers a more streamlined installation process using Conda. To perform HLA typing exclusively with long reads, we recommend utilizing the script/long_read_typing.py. This script is specifically designed to handle long reads and provide accurate HLA typing results.
Thank you for your patience, and please don't hesitate to reach out if you have any further questions or need assistance.
Best regards.


Fixed #24788 -- Allowed Forms to specify a prefix at the class level.

Can you revert the file mode changes?

ok what should I keep file mode ?

Look at the "Files changed 5,109 " tab of this PR> I see lots of 100644 → 100755 changes that shouldn't be there.

There are too many files, is there a easy way to do that.

chmod?

I am unable to do that you can reject my pull request.

Oh, this is from the contributing tutorial. I guess you didn't see the request not to submit a pull request added in<PHONE_NUMBER>3eb470851b35174c10573bbf717117.

From now on I will be much more careful, and I will try to contribute for django.


Failed to activate the timekeeper package

[Enter steps to reproduce below:]

Tried to install timekeeper package
Got bug report.

Atom Version: 0.187.0
System: Microsoft Windows 8.1 Pro
Thrown From: Atom Core
Stack Trace
Failed to activate the timekeeper package
At undefined

- Timekeeper is active

@akours I'm having trouble reproducing this. Can you reproduce this reliably?

Will try again in a couple of minutes and let you know.
On Mar 18, 2015 8:05 PM, "Ivan Žužak"<EMAIL_ADDRESS>wrote:

@akours https://github.com/akours I'm having trouble reproducing this.
Can you reproduce this reliably?
—
Reply to this email directly or view it on GitHub
https://github.com/atom/atom/issues/6004#issuecomment-83103302.

@akours Wondering if you've managed to reproduce this? :panda_face:

I have ;)
[Enter steps to reproduce below:]

...
...

Atom Version: 0.188.0
System: linux 3.13.0-46-generic
Thrown From: Atom Core
Stack Trace
Failed to activate the timekeeper package
At undefined

- Timekeeper is active & functional only with a valid project open

Commands
     -1:04.6 settings-view:open (a.btn-copy-report.icon.icon-clippy)
     -0:32.0 core:confirm (atom-text-editor.editor.mini)

Config
{
  "core": {
    "themes": [
      "one-dark-ui",
      "one-dark-syntax"
    ],
    "followSymlinks": true
  },
  "editor": {
    "invisibles": {}
  }
}

Installed Packages
# User
color-picker, v1.5.0
linter, v0.12.0
linter-php, v0.0.11
minimap, v4.6.0
timekeeper, v0.4.0
wakatime, v4.0.1

# Dev
No dev packages

Duplicate of https://github.com/queenp/punchclock/issues/1 and https://github.com/atom/atom/issues/6090.
Problem is caused by an uncaught exception in the package's initialization. Easily reproduced if Atom is opened from e.g. the dock or in any other form that does not load a project.

Sorry for not responding guys. Had some issues with my pc. Glad it was sorted.


Update different-id.md

🚀 Deployed on https://660f458ee3655a186106186d--cozy-fox-0defec.netlify.app


gltfpack: Added -km (keep unused materials) command line option

Thanks! Just a minor question - for the use case in question, are the materials named? There was a request at some point to keep named materials in tact without merging them; it would be nice to change the behavior here later to only keep named materials and prevent merging - this doesn't have to happen as part of this change, I just want to know if it's safe to do it later.

In my use case the materials are named, yes. But the name isn't relevant, as there is only one material per glTF and I access simply it via index 0.
I guess keeping name materials only would work for me as well.

Ok - I'm going to merge this as is but I'd like to rework this later to be name-based.
The motivation is as follows: as a general rule of thumb, right now gltfpack treats names as not-semantically-meaningful, but there are obvious use cases where you'd want to manipulate the objects by name externally. E.g. gltfpack by default merges equivalent materials (which is really important for some use cases), but an application that wants to go beyond just displaying the content and have a way to change materials should have a way to do so - that ideally doesn't involve maintaining indices because indices are fragile.
So it would be good to have a general behavior where:

gltfpack by default treats various objects (nodes, materials, meshes) as having structural equivalence and is free to merge / remove / create them as applicable
For each object type, there's a command line switch that allows to say "things that have a name are meaningful to me".

Yes, that makes sense.
Thanks for merging!


Augment RiskyPermissions.json to cover additional service principal API permissions

💡 Summary
Go through the permissions listed in the Graph reference and determine if we need to augment \PowerShell\ScubaGear\Modules\Permissions\RiskyPermissions.json with any additional permissions to cover more risks. We can also see if there are any other permissions for EXO, Teams, Sharepoint that could be covered.
Motivation and context
Continuous improvement by increasing the risk surface area covered by ScubaGear
Implementation notes

[ ] Examine the MS Graph permissions list linked above and compare it to the RiskyPermissions.json file. Modify the json file if necessary.
[ ] For EXO, Teams, Sharepoint, use the Entra Id > App Registrations > app > Manage > API Permissions page and click to add a permission in order to reference the available permissions for those services. Modify the json if necessary.

In the future we might want to look at permissions prefixed with DeviceManagement*. There could be some risks that we want to flag there related to exposing user devices.


Update ListContainersParam filters

There are lots of new filters available when listing containers. From the latest swagger.yaml:

Filters to process on the container list, encoded as JSON (a map[string][]string). For example, {"status": ["paused"]} will only return paused containers.

ancestor=(<image-name>[:<tag>], <image id>, or <image@digest>)
before=(<container id> or <container name>)
expose=(<port>[/<proto>]|<startport-endport>/[<proto>])
exited=<int> containers with exit code of <int>
health=(starting|healthy|unhealthy|none)
id=<ID> a container's ID
isolation=(default|process|hyperv) (Windows daemon only)
is-task=(true|false)
label=key or label="key=value" of a container label
name=<name> a container's name
network=(<network id> or <network name>)
publish=(<port>[/<proto>]|<startport-endport>/[<proto>])
since=(<container id> or <container name>)
status=(created|restarting|running|removing|paused|exited|dead)
volume=(<volume name> or <mount point destination>)

We should add these new filters. Might also be nice to dig through the older docker API documentation, figure out when each filter was added, annotate the methods with javadoc @since (docker api version).
I think this should wait on #707, which refactors all the filter params.

I also noticed some missing filters and was thinking about adding them.
I'd like to take over that task.

@johnflavin We can close this as #707 was merged, right?

No, this is a different issue. #707 refactored all the existing filters to be a consistent style; its only additions were filters for networks. This issue is about adding new filters for containers.

Ah I see. PRs are welcome then :)

Hi @albers ,
Since this project went on mature status, please re-open this issue (if it still stands) to https://github.com/dmandalidis/docker-client. Thanks


Custom Built JSON

Earlier suggestions were wrong, instead of true-false we need to keep or delete the animations.
Documentation Improvement!

It'll get fixed on #695.


Create a /discover page.

This is an idea borrowed from #1518--we may want a page in our system that helps folks discover stuff like streams they should subscribe to or users who they should pin on their buddy lists, etc.  This would be different than search--it would be for finding stuff that you don't even know exists.  This could possibly have some tie-ins to analytics data.

Currently the "All streams" tab of the  "/#subscriptions" page is intended to serve that purpose; but I think it needs some additional data+filtering/sorting elements to really complete that vision.  This issue can be for implementing such elements.

We now have a /#subscriptions/all (soon to be /#streams/all) link that will take you to the discover part of this page.

So are we going to remove "All streams" tab under /streams page, or can both the pages exist simultaneously? Since the /discover is going to have content of "All streams" tab (Along with analytics and user pinning thing), will we still need that tab?

I think we should probably start with imagining this as an enhancement to the "All streams" tab rather than a separate page and see if we can make that experience great.  If not, we'd make a separate one, but obviously life will be a lot easier if we don't need to make a second thing :)

@timabbott imho, first step for improvment "all streams" rather than create new page is displaying the most popular tags and streams. Next thing is tab "getting started" which asks you questions, like that: "need help with code review or problem or are you wanna to contribute? " and then shows best suited streams.

I may be misinterpreting what @misteraverin is saying in the previous comment, but I like the idea of a "Getting Started" page somewhere that the org can customize with their own text.  For example, it might have a basic code of conduct, links to the org's website, and some text about good streams to subscribe to.  This would be similar in concept to the user-profile page we're trying to develop now, but at the org level.

@showell yeah, you got it right. @timabbott what do you think about it?

Yeah, I think that idea is basically #4379 (including the suggestion of showing it when users sign up the first time).

The most useful thing for me (given my experience of using the Recurse Center Zulip for four years) would be a way to, for example, see new streams in the last three months that have non-negligible activity.
So creation time is the key filter/sort dimension for me. It's easy to miss the news about creation of a new stream as an only occasional reader, and if the stream already exists I likely saw it when I read through the full list and decided not to join.
(Also, apologies if this is not the best issue for discussing this item. I was linked here so feel free to move my comment if needed.)

@qqrs we have most of the data pipeline done to support displaying an activity level (in messages/week), which would then easily support "sort by stream activity" .  See #3989 and #4369; helping with reviewing #4369 would be a great way to contribute to this :)

Right now we have a /#streams/all page which very effectively displays the average traffic and number of users subscribed. Is there anything else which was on your mind with this /discover page @showell @timabbott ? If not, then this can be closed now.

@showell I think the main thing that we need to add to that page is a way to sort by traffic or number of subscribers.  Probably makes sense to open a separate issue for that that's more directed.  Closing this and will open such an issue.

See https://github.com/zulip/zulip/issues/8659 for the follow-up work.


feat: solve year 2022, day 13

https://adventofcode.com/2022/day/13

:tada: This issue has been resolved in version 1.13.0 :tada:
The release is available on:

v1.13.0
GitHub release

Your semantic-release bot :package::rocket:


WSL Integration with SSH and GPG

Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct
Read.
Please search existing issues to avoid creating duplicates.
None, after a quick glance.
Describe the feature you'd like.
Integrated terminal to popup when using WSL and git to allow passphrase entry for SSH and GPG.
Currently GPG gives and error and SSH just loops.
This would allow for people using SSH and git in WSL to use the VSCode native git integration without having to use the terminal.

@eskse do you use the Remote-WSL extension? Can you provide more information about your setup?

/needsMoreInfo

Hi @kieferrm, yes I'm using the Remote-WSL extension with Ubuntu as the Linux distro.
Both my GPG and SSH keys are configured on WSL only (potentially this is where I am going wrong) as VS Code doesn't display a passphrase from when being requested to sign commits or push changes.

I believe I am running into the same issue. I am using the Remote-WSL extension version 0.41.7 in VS Code 1.41.1 on Windows 10 1903 18362.535. When working with a remote repo configured with an SSH key that requires a passphrase I can perform all git actions from the terminal but if I attempt to push/pull from within VS Code I am never prompted for my passphrase and the program just spins.
Further, If I close and re-open VS Code its git plugin no longer functions properly; it seems that VS Code will no longer track changes made to files in the repo. If I save changes to an open file the source control icon in the activity bar does not show any status. Even in this state, I can still interact with git and GitHub from the terminal but VS Code refuses to tract changes to local files.

https://github.com/microsoft/vscode-remote-release/issues/1022


Some issues running code-server within code-server

It should spin up a new code-server as long as something in the format code-server [file] is not passed (in that case it opens in the existing instance) but it seems to spawn some other CLI instead.
Installing extensions also seems to hang at the end.

I think this is an edge case that can be moved to 4.0.1. @code-asher let me know if you disagree.

This happens consistently and all you have to do is run code-server so
it does not feel like an edge case to me.
Anyone trying to develop code-server from code-server will run into
issues (for example from the product).
The extension installation hang is also consistent and pretty serious
IMO.

(for example from the product)

Didn't consider that scenario. Fair point! Moved back to 4.0.0

Heads-up: we've started working on this.


feat(curriculum): create tables and forms quiz

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #56176

The failing test is unrelated, but you should apply the suggested changes, resolving only is not enough

I have made all the suggested changes. After the other half is also reviewed, I'll push the changes. I just mark them resolved to easily track of all the changes.

You have feedback that you can apply to some of the other questions, like code snippets.
Also it would be great if you would update the PR with your changes, reviewers can't see what you have not pushed to the PR

Hey @Neema-Joju !
I think this is off to a good start.
I went ahead and merged in some of the requested changes that Tom left you.
But there are still some questions he left, that I suggest you take a look at.
Also, at quick glance, there are a couple of areas for revisions where I think it can better align with the lecture videos we have. So I will leave a review soon 👍

Hi @moT01, @jdwilkin4
I have added the rest of the edits.


Fix the GitHub link.

@ehrlinger,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Add web scraping to fetch latest published articles from followed blogs

Add web scraping to real python blog

created^ :)


Make namespace hal::soft

This way its clear that these are from the soft library and will prevent name clashes.

this one has been completed

Resolved by #42


[BUGFIX] Make BeanValidationAnnotationHandler public

That way, extending BeanValidationRandomizerRegistry and appending to annotationHandlers is possible.
This is a follow up for change mentioned here - making the map protected was unfortunately, not enought. The test was passing because BeanValidationAnnotationHandler is in the same package as the test.

Thank you for the PR, LGTM. Even though the bean validation module will be discontinued (see #499), I am going to merge this change so that it is included in the module before removing it (for someone else to maintain or copy).


fix for docker overlay network

so, this is quite a large change so no ill feeling if you don't accept the pull.
the change was needed into our environment to fix the default dns clib in pgbouncer not resolving dns names as provided by docker swarm mode/docker overlay network, as discovered here https://github.com/pgbouncer/pgbouncer/issues/122

you're welcome, sorry about the layering thing I'm quite new to docker and didn't thought of that


So when you right click a wall the vehicle is spawned in that wall.

This wil be fixed soon.


fix(chunks): using numchunks didn't cover image

I believe this is straight forward -- found in 0.3.x branch, and also exists in mainline development.
I hope the expansion/tweaks to the chunking test make sense to cover the cases.  The previous test_GeoResource.test_chunking didn't actually assert about the final chunk.
:+1: to @justinfisk for the help in sorting this out on the 0.3.x branch.

I just broke the single commit into two, so that the test demonstrates (hopefully it's been a while) the failure with one CI run, and then the fix with the second CI run.  I'll push the second commit once the checks have completed on the first commit.

Well, the 2.7 fail looks more CI config, or underlying base image for the docker build.

Ok, @matthewhanson, i pushed a copy of develop to test if I broke something, and it is definitely not anything I did. Given that python2 is deprecated, and this works for 3, are you ok with this merging?

this is the develop branch that I added a newline to today.

@matthewhanson -- the issue in the test_27 ci pipeline looks to be related to a change in the stack such that in the pip install . does not install the gippy.py file.  As a work around, doing a pip install -e . shows that this passes the test when the module is loadable.  As this only hits py2, I'm not concerned with it, but wonder what your thoughts are?


#Issue854 : Prefix-Postfix-Infix interconversion algorithm in C & Java

🛠 Issue 854
#854
👨💻 Prefix-Postfix-Infix interconversion algorithm to be implemented in C & Java
I was given the reference of the following GithHb Repository.
https://github.com/Kumar-laxmi/Algorithms/blob/main/C%2B%2B/Maths/Infix_Postfix_Prefix_Interconversion.cpp
https://github.com/Kumar-laxmi/Algorithms/blob/main/Python/Maths/prefix_infix_postfix_interconversion.py
Based on the above code:

I have implemented a Menu Driven Program for the interconversion of Prefix-Postfix-Infix Expressions.

Kindly Review my Code.

@AnubhaSharma2709 Kindly make the required changes ASAP

Sorry for the delay
I have changed the file name
and added the screenshots


quick fix for #1107

fixes #1107

@jsugarman

I took another look at the issue about the number of claims fled stretching and wondered if this PR would inadvertently fix it turns out it breaks the field so users cant click ok the input field

good news is it fixes #1108  being all one long column but this is on the claim detail page not the start a new claim page


Start customer session step 2 - provide customer email

As an asagent I want to impersonate a customer for which I know the userId (email)
The behaviour from step 1 will evolve to this:

Instead of providing the customerId, the asagent will provide the userId (email) of the customer to impersonate.
When the asagent clicks the "start session" button, we will use the customer search to lookup the user from the customer search in assistedservicewebservices.
add logic in js to find a uid match in the results.

PR #4110


⚠ Bofrost.fr has degraded performance

In e6b1843, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1260 ms

Resolved: Bofrost.fr performance has improved in d45466b after 6 minutes.


update(deps) ocular to v1.0.0-alpha.7

Lint changes are coming from updated rules in ocular.
Use Node 14 now since the website build seems to be compatible now with this change.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 26.442%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
256

Relevant Lines:
843

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 26.442%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
256

Relevant Lines:
843

💛  - Coveralls


.gitignore file and Testing in Visual Studio Code

Hello,
I've observed that the httpyac plugin in Visual Studio Code displays an 'httpyac Tests' section upon selecting 'Testing'. This section unexpectedly lists all http files within my codebase, even those within directories marked as hidden in the .gitignore file. IMHO it seems more intuitive for the httpyac plugin to exclude files and folders specified in the .gitignore from its listings.
Thanks in advance.

I use globby internally and the exclude would be activated See https://github.com/AnWeber/httpyac/blob/main/src/cli/send/send.ts#L200. I think you need to be at the .gitignore file level for it to work correctly.


Add test files for OptionalCoordinateFrame type

Nothing fancy, just adding some test files for OptionalCoordinateFrame so we can use them in rbx-dom. Let me know if you think the folder should be named something different. I wasn't sure about it.

Upon further inspection it looks like OptionalCoordinateFrame has no serialization in the binary format (the PROP chunk ends after the property name), so I'm not sure if we should bother including a binary version of the file in this one. Up to you. I think we should at least add a disclaimer though.

Let's remove the binary file and add a note to the README for now.

On second thought, let's leave it. If it's broken for now, it's broken for now.


feat: filter non bug panic section

Motivation
Closes #2912
provides a way to inspect panic! messages and conditionally set the panic section with link to new issue

Solution

add panichook wrapper that inspects the payload and set the panic section conditionally whether the message is considered a bug or not

Any reason we can't just throw an error here? If an error is thrown then the bug section is not displayed

Any reason we can't just throw an error here? If an error is thrown then the bug section is not displayed

throwing where?
loading the Config with panic makes sense imo, since no way to recover anyways

The reason I defaulted to adding the "This is a bug" thing on panic is to push us to use errors instead of panicking since a panic would lead users to report the messages (even if it wasn't really a bug - in which case, the panic itself is the bug).

but with the message for all panics we get issues like #2912 even if no bug

@mattsse Throwing an error when loading the config: imo it doesn't make sense to panic even though the error is not recoverable in the CLI. Panics should be reserved for when something is really wrong. Another way I could phrase it is: error if it's because of the user input, panic if it's because of developer error

but with the message for all panics we get issues like https://github.com/foundry-rs/foundry/issues/2912 even if no bug

yea given above philosophy it makes sense since the panic itself is the bug imo. distinction is errors -> no stack trace, panics -> stack trace.

closing as overengineered, not very useful after all


Out of sequence chunks no longer considered a CorruptionErr and never deleted from disk

What did you do?
Note: This came from investigate a incident we had in production in https://gitlab.com/gitlab-com/gl-infra/production/-/issues/5998, we still don't know the root cause and are investigating in https://gitlab.com/gitlab-com/gl-infra/infrastructure/-/issues/14821 and when doing so we found the following:

Start seeing the following errors inside of Prometheus logs: msg: Deletion of mmap chunk files failed, discarding chunk files completely err: cannot handle error: iterate on on-disk chunks: out of sequence m-mapped chunk for series ref<PHONE_NUMBER>.
We start looking at the stack trace where error is being return and saw the following:

Head.Init is called
loadMmappedChunks is called
Finds out of sequence m-mapped chunk and returns error
Check if error is of type CorruptionErr

Corruption counter never incremented.
Nothing else from the loadMmappedChunks return CorruptionErr so this check is either reduntant or CorruptionErr should be returned.
This was changed in https://github.com/prometheus/prometheus/pull/7229/files#diff-d9f058bd14a07ebec78d70aaa8d6e8e705bc09086e338b74b12666e7ac121605L542-R573

Start recovery process by calling removeCorruptedMmappedChunks
Deleting corrupted chunks with DeleteCorrupted
Check error type

In the past this used to be true for out of sequence, only after https://github.com/prometheus/prometheus/pull/7229 is merged.
This is where we get cannot handle error from the log message

What did you expect to see?
When out of sequence m-mapped chunk

Increment the prometheus_tsdb_mmap_chunk_corruptions_total counter increments.
DeleteCorrupted err assertion passes
Delete chunk file from disk

What did you see instead? Under which circumstances?
We started seeing large WAL files and large chunks_head files which exhausted memory and disk usage. We aren't 100% sure that this specific "bug" resulted in resource exhaustion but, it still seems like it should delete the chunk from the disk, and might not helped recovery from a corrupted WAL/chunk
Environment

System information:

$ uname -srm
Linux 5.4.120+ x86_64

Using the quay.io/prometheus/prometheus image in a GKE cluster.

Prometheus version:

$ prometheus --version
prometheus, version 2.30.0 (branch: HEAD, revision: 37468d88dce85ac507f3fb7864c7d1c078e3e27d)
  build user:       root@e913c65aa170
  build date:       20210914-09:49:24
  go version:       go1.17.1
  platform:         linux/amd64

However this problem exists in main branch as well.

Logs:

{"caller":"head.go:649","component":"tsdb","err":"cannot handle error: iterate on on-disk chunks: out of sequence m-mapped chunk for series ref<PHONE_NUMBER>","level":"info","msg":"Deletion of mmap chunk files failed, discarding chunk files completely"}

Kudos to @mwasilew2 for helping me out

I think the potential fix for this is https://github.com/steveazz/prometheus/commit/a6c3a836114d1289de285959793c82f579ee35b2 however, first I want to discuss if in https://github.com/prometheus/prometheus/pull/7229/files#diff-d9f058bd14a07ebec78d70aaa8d6e8e705bc09086e338b74b12666e7ac121605L542-R573 it was the intention to stop considering it a CorruptionErr and it's me misunderstanding the code.

@codesome @bwplotka apologies for pinging you directly I appreciate that you folks are busy, but since this change happened in https://github.com/prometheus/prometheus/pull/7229 I'm curious about your opinion here and see if this is a bug or not.
Thank you so much for that hard work you put in Prometheus and Thanos 💪

Legit bug, we should be cleaning up the head chunk files. These out of sequence chunks are something mysterious (i.e. I haven't investigated the cause for it, though there were some other reason that could cause it which no longer errors and is fixed, but still something can cause it).
We cannot (or should't) delete it via the corruption error mechanism because we that is not technically a corruption but some other potential race. Since we are totally discarding the m-map chunks in this case, I suggest a "fix" of deleting all the m-map chunk files when we face this particular out of sequence error.

Okay, I take back my words about We cannot (or should't) delete it via the corruption error mechanism. Yes you are right, it was a bug introduced in #7229 and your suggested fix makes sense; that is the easiest way to delete all the m-map chunk files :)

We will need unit tests to avoid this bug from slipping in again.

Thanks @codesome I agree that a test needs to be added to prevent this front happening again.
I’d like to take this on if possible, but might be a bit slow because of holiday season!

I'm now actively working on this again, I'm working on a test case where it can generate an out-of-sequence chunk so we can validate the chunk is deleted on disk.
I'm struggling a little bit writing the test, but will reach out for help if I face a brick wall!

@steveazz you will find existing tests here where it's creating files and adding chunks manually.

One other idea I had was to increase verbosity around this error a little bit. So in addition to fixing the logic that deletes the broken chunk (gets Prometheus out of the crashloop), the log message could contain more debug information. It currently contains a series ref which is a number. Example error message:
    cannot handle error: iterate on on-disk chunks: out of sequence m-mapped chunk for series ref<PHONE_NUMBER>

and here's the error in the code: https://github.com/prometheus/prometheus/blob/main/tsdb/head.go#L630
I wonder if it would be possible to surface in the error message some more information about the series, for example the __name__ label.
@steveazz feel free to include it in your PR. If on the other hand you think it would fit better in a separate PR let me know and I'm happy to open one.

One other idea I had was to increase verbosity around this error a little bit. So in addition to fixing the logic that deletes the broken chunk (gets Prometheus out of the crashloop), the log message could contain more debug information. It currently contains a series ref which is a number. Example error message:
    cannot handle error: iterate on on-disk chunks: out of sequence m-mapped chunk for series ref<PHONE_NUMBER>

and here's the error in the code: https://github.com/prometheus/prometheus/blob/main/tsdb/head.go#L630
I wonder if it would be possible to surface in the error message some more information about the series, for example the __name__ label.
@steveazz feel free to include it in your PR. If on the other hand you think it would fit better in a separate PR let me know and I'm happy to open one.

This actually got improved in https://github.com/prometheus/prometheus/commit/24827782cbc546adba99c7645dc8a01750023597  where it now shows the last chunk and the new chunk

Hey folks
Is there an update on this? or any help we could provide? We just had this happening to one of our prometheus instances, filling up 80GB of disk-size with the retention set to max 64Gi

The issue title is a bit misleading. The files are actually deleted for out of sequence chunks. I am closing this issue because of that.

@xvzf if you have any specific details (logs, sizes of various TSDB artifacts, etc) and if it does not heal eventually (with a bit of increased disk size), please file another issue with details or comment the details on any relevant open issue.


Add content to the feature/landingPageRedesign branch

Please provide a summary of the feature
Landing page is missing content in the bottom boxes in the feature/landingPageRedesign branch.
Some things that couple be included.

Info about 100Devs
Description of the Together being open-source and a link to the github
Meet the team
Maybe a link to https://communitytaught.org/resources/

Are there any potential issues you foresee with this feature
Is this feature related to any current or past issues? Please provide the
Additional context
Add any other context or screenshots about the feature request here.

Which Branch should this fix be Pulled into?
PR into feature/landingPageRedesign
If you're interested in working on this feature, please comment so you can be assigned

I imagine the text itself will go through some wordsmithing / have several drafts.  Updating text in a react project is a lot harder than updating text in, say, a Google Doc - so I think it would be best if we hammered out the text to add before someone picks this up.

Hello! I would like to help with this issue and help write out the content for the landing page.

Hey ill also be over seeing this issue as well 😃

Hello! I would like to help with this issue and help write out the content for the landing page.

Perfect! In the meeting we discussed.
Box 1: Talk about 100Devs and how to join.
Box 2: Talks about how we have videos on how to use and contribute to Together. Should probably mention the videos are a work in progress.
Box 3: Talk about community taught and link to the resources
Bottom Box: Talk about the Together Team.
Can you respond back with the content when it's created? I'd like to have some time to provide some feedback prior to @HondaChan14 or you spending time developing it.

Hey Yes will do!

Thank you, @Caleb-Cohen.
Box 1: 100Devs and how to join.
100Devs (https://leonnoel.com/100devs/) is a diverse and inclusive community of participants in a software engineering training program run by Leon Noel. It is 100% free and requires absolutely no previous technical experience.
Get started by joining us on Discord (discord link)
(Include catchup crew info?)
Box 2: Talks about how we have videos on how to use and contribute to Together. Should probably mention the videos are a work in progress.
Would you like to contribute to Together?
Follow (along? on) our team* (our repo? on something? On Github?) for (coming soon) our video series on how to use and contribute to Together, together
Box 3: Talk about community taught and link to the resources
Keep track of your 100devs course progress using CommunityTaught.org created by 100devs alumni (alumnae) Laura Abro.
Additional 100devs resources (link: https://communitytaught.org/resources/)
Last box: about the team
Together is a community-driven open source project backed by 100devs students led by (include names of maintainers). Together is not only designed to be a valuable tool to the 100devs community but also provides an additional opportunity for students to learn in a group environment. Together is open to all varying skill sets and encourages peer programming. Interested in collaborating with us? (link to onboarding guide) (discord channel link?)

What are your thoughts?
Box 1: I like it - what about this slight change?
100Devs (https://leonnoel.com/100devs/) is a diverse and inclusive community of participants in a software engineering agency and training program run by Leon Noel. It is 100% free and requires absolutely no previous technical experience.
Box 2:
We are currently working on video resources for individuals that would like to learn more about how to contribute to Together and how Together works. Follow along on our Youtube when it is ready!
Box 3:
Looks good! It would be Alumna in this case.
ast Box - Looks great! I think it should link to the github contributing documents or Onboarding on the wiki not discord.

That's perfect, thank you @Caleb-Cohen. @HondaChan14, final text:
Box 1:
100Devs (https://leonnoel.com/100devs/) is a diverse and inclusive community of participants in a software engineering agency and training program run by Leon Noel. It is 100% free and requires absolutely no previous technical experience.
Box 2:
We are currently working on video resources for individuals that would like to learn more about how to contribute to Together and how Together works. Follow along on our Youtube when it is ready!
Box 3:
Keep track of your 100devs course progress using CommunityTaught.org created by 100devs alumna, Laura Abro.
Additional 100devs resources (link: https://communitytaught.org/resources/)
Last box: Together is a community-driven open source project backed by 100devs students led by (include names of maintainers). Together is not only designed to be a valuable tool to the 100devs community but also provides an additional opportunity for students to learn in a group environment. Together is open to all varying skill sets and encourages peer programming. Interested in collaborating with us? (link to github contributing documents/onboarding guide)


[Event Request] Codeunit 90 "Purch.-Post" – Existing Event “OnBeforePostInvPostBuffer”

Could we please have the existing publisher “OnBeforePostInvPostBuffer” in Function “PostInvoicePostBufferLine”, Codeunit 90 "Purch.-Post”, be expanded by an additional parameter?
local procedure PostInvoicePostBufferLine(var PurchHeader: Record "Purchase Header"; InvoicePostBuffer: Record "Invoice Post. Buffer") GLEntryNo: Integer
    var
        GenJnlLine: Record "Gen. Journal Line";
    begin
        OnBeforePostInvoicePostBufferLine(PurchHeader, InvoicePostBuffer);
        with GenJnlLine do begin
                InitNewLine(
                  PurchHeader."Posting Date", PurchHeader."Document Date", InvoicePostBuffer."Entry Description",
…
                CopyFromInvoicePostBufferFA(InvoicePostBuffer);
            end;

*** start old code ***
            //OnBeforePostInvPostBuffer(GenJnlLine, InvoicePostBuffer, PurchHeader, GenJnlPostLine, PreviewMode,
            //SuppressCommit);
*** stop old code ***
*** start new code ***
            OnBeforePostInvPostBuffer(GenJnlLine, InvoicePostBuffer, PurchHeader, GenJnlPostLine, PreviewMode, SuppressCommit, GenJnlLineDocNo);
*** stop new code ***
            GLEntryNo := RunGenJnlPostLine(GenJnlLine);
            OnAfterPostInvPostBuffer(GenJnlLine, InvoicePostBuffer, PurchHeader, GLEntryNo, SuppressCommit, GenJnlPostLine);
…
    [IntegrationEvent(false, false)]
*** start old code ***
    //local procedure OnBeforePostInvPostBuffer(var GenJnlLine: Record "Gen. Journal Line"; var InvoicePostBuffer: Record
    //"Invoice Post. Buffer"; var PurchHeader: Record "Purchase Header"; var GenJnlPostLine: Codeunit "Gen. Jnl.-Post Line";
    //PreviewMode: Boolean; CommitIsSupressed: Boolean)
*** stop old code ***
*** start new code ***
    local procedure OnBeforePostInvPostBuffer(var GenJnlLine: Record "Gen. Journal Line"; var InvoicePostBuffer: Record "Invoice Post. Buffer"; var PurchHeader: Record "Purchase Header"; var GenJnlPostLine: Codeunit "Gen. Jnl.-Post Line"; PreviewMode: Boolean; CommitIsSupressed: Boolean; var GenJnlLineDocNo: code[20])
*** stop new code ***
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Invalid date return after date format

Describe the bug
The date formats that start with DD are not working correctly. For example, the dates 31.12.2000 and 13.12.2000 are invalid when they are passed to the DD.MM.YYYY, DD/MM/YYYY, or DD-MM-YYYY formats.  The error occurs when the day is between 13 and 31. @Jasenkoo
To Reproduce
Steps to reproduce the behavior:

Date format change to hard form or static form

Expected behavior
Must be changed but not changed in input field as given attachment
Screenshots

Desktop & mobile (please complete the following information):

Browser [chrome]
Library version [5.4.0]

@mzee19 I don't get what the issue is here, in your time label, it is properly formatted

@mzee19 I don't get what the issue is here, in your time label, it is properly formatted

But not displayed inside the date-picker.

DD is not a valid token for format, please refer to valid tokens here: https://github.com/date-fns/date-fns/blob/main/docs/unicodeTokens.md


upgrade express-validator to 5.0.1

to include the new features like check, sanitize, checkSchema ...
check https://github.com/ctavan/express-validator

Closing the PR since a more up-to-date version of express-validator has been integrated as part of a more comprehensive dependency update commit at this point.


Get a simulation working

Once the leader election is done, I'll need to move to simulations because of the fleet-managing component of the project. This will of course be a must for the project itself, but it would also be useful early on to actually see how the fleet behaves.
The goal is to have at least 2-3 copters in a simulation and getting to move them manually and together, with a single input.

A simple pre-arm checks + arm operation is working both with a single agent and in the overall example simulation at commit 098a912

Multi-agent synchronized operations are now achievable.

Rendezvous is complete, even though it might require some fine-tuning in the future.
For now, it is found at commit a0d386a


latest cargo update broke cargo install command in my ci

Hi, it looks like the latest cargo submodule bump in the rustc repo
https://github.com/rust-lang/rust/commit/f23db015ad5d306bbb0de104817c9a8cc4ee1db9
broke my cargo-cache ci.
I was running the command cargo install --force --git https://github.com/matthiaskrgr/cargo-cache --no-default-features --features ci-autoclean cargo-cache
which worked prior to the submodule update and now fails with
    Updating git repository `https://github.com/matthiaskrgr/cargo-cache`
  Installing cargo-cache v0.4.2 (https://github.com/matthiaskrgr/cargo-cache#379c40c2)
    Updating crates.io index
error: failed to compile `cargo-cache v0.4.2 (https://github.com/matthiaskrgr/cargo-cache#379c40c2)`, intermediate artifacts can be found at `/tmp/cargo-installDT5PxT`

Caused by:
  failed to determine package fingerprint for build script for cargo-cache v0.4.2 (/home/runner/.cargo/git/checkouts/cargo-cache-16826c8e13331adc/379c40c)

Caused by:
  failed to determine the most recently modified file in /home/runner/.cargo/git/checkouts/cargo-cache-16826c8e13331adc/379c40c

Caused by:
  failed to determine list of files in /home/runner/.cargo/git/checkouts/cargo-cache-16826c8e13331adc/379c40c

Caused by:
  expected git repo /usr/share/rust/.cargo/git/checkouts/cargo-cache-16826c8e13331adc/379c40c/.git/ to be parent of package /home/runner/.cargo/git/checkouts/cargo-cache-16826c8e13331adc/379c40c

Caused by:
  prefix not found
##[error]Process completed with exit code 101.

I have never seen this error before and was not able to reproduce locally yet with rustc 1.45.0-nightly (fa51f810e 2020-04-29 / cargo 1.45.0-nightly (90931d9b3 2020-04-28)
Do you have any ideas what could be happening here?

Thanks for the report. It looks like if CARGO_HOME is a symlink causes the problem. Part of the fallout from #8095.  I'll take a look.


Detect tensor element equality through different indexing

Consider the terms
I1[i⁺j⁺a⁻k⁻](/\/\) = - H[i⁺j⁺a⁻b⁻](....) T1[b⁺k⁻](..)

and
I2[i⁺j⁺a⁻k⁻](/\\/) = H[i⁺j⁺b⁻a⁻](....) T1[b⁺k⁻](..)

From this, it can be seen that the following relations between the intermediates:
I2[i⁺j⁺a⁻k](/\\/) = - I1[j⁺i⁺a⁻k⁻](/\/\)

Are these two intermediates occurring in two different contexts?
For a single context (i.e. if the two terms in the example arise from one and the same term generated by GeCCo), we should avoid this by keeping the canonical initial indexing (i.e. the interchange of a and b in the "H" tensor should not happen).
I think I need more context to give a more helpful comment without confusing everyone.

In this specific example (CC2 singles residual) the terms are coming from the same diagram:
>>>> R1[a⁺i⁻](||) -= 0.5 * H[j⁺k⁺b⁻c⁻](||||) T1[b⁺i⁻](||) T2[a⁺c⁺j⁻k⁻](||||) <<<<
# of Terms: 4
- 1: {
  H_T1[i⁺j⁺a⁻k⁻](/\/\) -= H[i⁺j⁺a⁻b⁻](....) T1[b⁺k⁻](..)
}
- 2: {
  H_T1[i⁺j⁺a⁻k⁻](/\\/) += H[i⁺j⁺b⁻a⁻](....) T1[b⁺k⁻](..)
}
- 3: {
  H_T1[i⁺j⁺a⁻k⁻](////) -= H[i⁺j⁺a⁻b⁻](....) T1[b⁺k⁻](..)
  H_T1[i⁺j⁺a⁻k⁻](////) += H[i⁺j⁺b⁻a⁻](....) T1[b⁺k⁻](..)
}
- 4: {
  R1[a⁺i⁻](..) -= 0.5 * H_T1[j⁺k⁺b⁻i⁻](////) T2[a⁺b⁺j⁻k⁻](....)
  R1[a⁺i⁻](..) += 0.5 * H_T1[j⁺k⁺b⁻i⁻](////) T2[a⁺b⁺k⁻j⁻](....)
  R1[a⁺i⁻](..) -= H_T1[j⁺k⁺b⁻i⁻](/\\/) T2[a⁺b⁺j⁻k⁻](....)
}

As can be seen, in this case the abab spin case of H_T1 isn't even needed in the first place.
This issue isn't about optimizing a special case though. Rather it shall server as an example to test the general optimizer capabilities on, with the hope that when dealing with arbitrary inputs, this kind of optimization may end up being useful in some places.

we should avoid this by keeping the canonical initial indexing (i.e. the interchange of a and b in the "H" tensor should not happen).

see #16

The two intermediates are equivalent, indeed. This can be better understood by replacing the actual indices by their orbital space identifier (I think your considerations concerning introduction of the TensorBlock object were in this direction). If we use the ITF nomenclature: "e" external=virtual and "c" closed=occupied, the Intermediate is:
H_T1[ccec]
From this we see that the two creation indices (i and j) run over the same index space such that the assignment of either /\ or \/ does not make a difference (in the sense that these cases can be mapped to one unique element).
Two small comments:

For the isolated term, the better factorization would be to contract with T2 first. (But H_T1 can be added to a H element with similar shape, which would be the typical intermediate in a "hand-made" code.)
The ideal final result would be:

The program recognizes that H_T1[jkbi](////) = H_T1[jkbi](/\/\) - H_T1[kjbi](/\/\) viz. H_T1[jkbi](....) - H_T1[kjbi](....)
and gets this term: R1[ai](..) -= 0.5*H_T1[jkbi] ( 2*T2[abkj] - T2[bakj] )

The second issue may be a bit challenging in the general case, but we should keep an eye on it.


Add an option to use decimal comma instead of decimal point

I’d be nice if one could define invoice locale and thus output decimal numbers (e.g. in currencies) using this locale.
From implementation point of you, we could use toLocaleString(). The implementation is fairly easy.
Update: Or maybe it won’t be that easy: ariya/phantomjs#12581.

Or we don’t need to implement anything (not even #38 ), because we can use new Intl.NumberFormat('sk, {style: 'currency', currency: 123456.78}).format(item.priceUnit.main) right way.
Closing this as a non-issue and because it seems like this project is unmaintained and also because I moved my code to puppeteer while reimplementing some stuff from nodeice.


Modify ignored evt.coordinate, which did not play well with Snap.

The Modify interaction used map.getCoordinateFromPixel in handleDownEvent and handlePointerAtPixel instead of using evt.coordinate. evt.coordinate is set by the Snap interaction. Depending on the resolution, map.getCoordinateFromPixel will not return the same result as evt.coordinate leading to not modifying the intented feature.

Can you elaborate a bit on cases where evt.coordinate and map.getCoordinateFromPixel yield different results? When does that happen in the context of the Modify interaction? Ideally you could also come up with a test that fails before and passes after your change.

Closing due to lack of response.

Sorry, I didn't get any notification on this comment. It's merely different due to the mathematics. Pixel granularity is not decimal coordinate granularity. Translating from pixel to coordinate gives a value but maybe only the center of the radius of coordinate points, which may not exactly match the coordinate of the vertex in the Modify interaction. Therefore, the Snap interaction will not match the intended vertex.

Thanks for your response. Can you add a test that fails before and passes after your change?

Sorry, I'm just getting back to this. I will create a test to show the problem.

I have a couple of JS Fiddles that illustrate the problem:
First is the problem,
https://jsfiddle.net/polar/kh20ytmr/
The above illustrates that when you grab the dots(point), which are snapped, most of the time, the underlying line vertexes are modified and not the point that was supposed to be snapped. This problem
has to do with line segments that are close to the snapped feature.
The following fiddle is the fix based on the pull request
https://jsfiddle.net/polar/env5nkdt/
This fiddle is using an ol-debug.js with the patch and is residing on my S3 bucket. The Snap grabs the vertex that is snapped and puts it in the event. However, it calculates the pixel (and rounds it to an integer). This causes conversion problems when modify is using map.getCoordinateFromPixel. It just doesn't match the vertex. The pull request merely carries the event.coordinate through the handle calls.
I've developed a test that will fail in the current version v4.6.4, and will succeed with the patch.
modify.snap.test.js.txt


NixOS flake config won't build because of missing attribute

Describe the bug
nixos-rebuild won't build my dotfiles I ripped from @fufexan because boot.kernelPackages.extend does not exists.

Screenshots

Additional context
It doesn't change whether I used an out-of-nixpkgs or in-nixpkgs kernel.
Notify maintainers
@TredwellGit
@Ma27
@NeQuissimus

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
- system: `"x86_64-linux"`
 - host os: `Linux 6.1.38, NixOS, 23.05 (Stoat), 23.05.2084.08700de174b`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(root): `"nixos-23.05"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Please provide some code that triggers the problem. A screenshot and a config that was "ripped from $repository" isn't very helpful for investigating the problem.

boot.kernelPackages = inputs.chaotic.packages.x86_64-linux.linux_cachyos
That's taken out of my config

You have to use linuxPackages_cachyos rather than linux_cachyos. The latter only refers to the kernel, not the package-set for a linux kernel (including kernel modules etc).
Anyways, overriding kernelPackages in nixpkgs works fine, so if something else breaks with that, it's most likely an issue with nyx.


supports rejectUnauthorized option

I want to be able to connect to my own "bad" SSL

Actually, passing it a http.Agent with the same option works:
https://github.com/bitinn/node-fetch/issues/15#issuecomment-97366559

ah, I missed that. Disregard this then, I will do it that way :)
Thanks


bug 🐝: docker escape code doesn't work properly

What happened?
the default code to escape a docker container is ctrl p q, but something about the HLDS just eats that input instead of letting docker take care of it. currently i can just press ctrl c three times and that works, but it feels wrong.
similarly (but this might just be an HLDS thing, unrelated to the docker image), i cant figure out how to gracefully exit the container. i've tried every variation of stop, kill, quit, end i can think of, but there's nothing. ideally ctrl c would take care of that, which is the behavior when i run the HLDS executable directly. so i just docker kill it.
if any of this is intended/known won't fix, it should probably be put in the docs somewhere (or maybe i just missed that)
Approach
I am using a pre-built image on Dockerhub (jives/hlds)
What games are you trying to run?
Half-Life Deathmatch (valve)
Do you have custom server configs or mods installed?

[ ] I have custom config files located in the config directory
[ ] I have a custom mod located in the mods directory

Docker Logs
No response
Docker Command
No response

I can't seem to reproduce this- 🤔
Gracefully stopping... (press Ctrl+C again to force)
Aborting on container exit...
[+] Running 1/1
 ✔ Container hlds-docker-hlds-1  Stopped                                                                    10.5s
canceled

After cancelling the container shuts down, as does the server.

oh sorry, i forgot to mention i was doing compose up with the -d flag, to start it detached. from what i can find online, once i attach back into the container, ctrl p q should spit me back out of the container
if i don't start detached, then ctrl c just kills it as you said

Ah gotcha. I don't think this is a problem with the container directly, just how the commands are being used.  The following worked for me using the docker run command in the README (I updated this to add the -ti flag), and then hitting Control + P, Control + Q.
docker run -d -ti \
  --name hlds \
  -v "$(pwd)/config:/temp/config" \
  -v "$(pwd)/mods:/temp/mods" \
  -p 27015:27015/udp \
  -p 27015:27015 \
  -p 26900:26900/udp \
  -e GAME=${GAME} \
  jives/hlds:valve  \
  "+log on +rcon_password changeme +maxplayers 12 +map crossfire"

Results:
root@docker2503onubuntu2204-s-1vcpu-1gb-nyc1-01:~/hlds-docker# docker attach 9eccb6926216
read escape sequence
root@docker2503onubuntu2204-s-1vcpu-1gb-nyc1-01:~/hlds-docker#

I'm also able to execute any combination of stop, kill, etc via docker stop <container-id>, it does take a few minutes though for the server shutdown command to finish beforehand though.
root@docker2503onubuntu2204-s-1vcpu-1gb-nyc1-01:~/hlds-docker# docker stop 9eccb6926216
9eccb6926216

For the inputs to work correctly with docker compose, start the server with docker compose -d, and then attach to the container with docker attach --sig-proxy=false <container-id>. You can then use Control + C to exit the container instance without killing it. You can then use docker kill|stop|etc <container-id> to terminate the process.


MemberConverter() with a different target type

I would like to be able to do something like this:
VerifierSettings.MemberConverter<Order, Customer, string>(order => order.Customer, customer => customer.Name);

Such that when snapshotting an Order its Customer will be converted to a string.
I am currently achieving this by registering a custom converter that projects Order for serialization.
However, I instinctively dislike this approach as it feels brittle, and will be harder to maintain:
class CustomConverter<T>(Func<T?, object?> convert)
    : WriteOnlyJsonConverter<T?>
{
    public override void Write(VerifyJsonWriter writer, T? value)
    {
        writer.Serializer.Serialize(writer, convert(value));
    }
}

VerifierSettings.AddExtraSettings(settings =>
{
   settings.Converters.Add(new CustomConverter<Order>(
        o => new { ... }));
});

I am currently achieving this by registering a custom converter that projects Order for serialization.
However, I instinctively dislike this approach as it feels brittle, and will be harder to maintain:

why do you think this is fragile?

why do you think this is fragile?

I would prefer to snapshot Order directly, so that if it is refactored in future I can be sure this will lead to changed snapshots, forcing a review of the tests. The projection gets in the way of this. (For example, the projection won't change if a new member is added to Order.)
But I suppose that's just my preference for my particular situation. What do you think of the idea in principle? I think it might be nice simplification to be able to remap a member to a different type for serialization, without needing the ceremony of a custom converter + projection.


Create React form.

Use Formik to create the React form and manage its state. For starters the form should have the following inputs:

Walk: number.
Type: select.
Amenity: select radio.
For now I will hardcode a few types and amenities but eventually I would like a call to get these on page load. @ZaneBartlett1 we should discuss this on the call.

I feel like this would be discussed on this issue - https://github.com/DartExplore/dallas-by-metro-backend/issues/128. This is what I want to confirm if there's any other API calls that are needed.
Again, I think we should start out with that /all endpoint so there is a way to get that for right now. Then as we have time, we can develop more specific calls.


fix(tiller): fix helm status failure on missing resource

This fixes a bug in which 'helm status' fails if any of the expected
resources are missing in Kubernetes. Now it prints a list of missing
resources at the end of the status report.
Closes #2118

To test:

Install a Helm chart with one or more resources
Using kubectl, delete one of the resources (like a pod or service)
Run helm status on the release
Verify that the bottom has a section like this:

==> MISSING
KIND    NAME
pods    foolhardy-alligator-i2118


Alternative "Origin" header to bypass forbidden requirement

I'm attempting to use client-side code to fetch remote resources (e.g. https://ron-swanson-quotes.herokuapp.com/v2/quotes). As an example...
axios({
    method: 'get',
    url: "http://localhost:8080/https://github.com/Rob--W.atom",
  })
  .then((response) => {
    console.log(`Fetching ${url}: ${JSON.stringify(response)}`);
  })
  .catch((err) => {
    console.error(err.status)
    console.error(`Fetching ${url} : \n${err["message"]}`);
    console.error(err);
  });

This is an interesting scenario. Because the remote server treats the request as a CORS request, so it (usually) denies the request (unless the server is set to wide-open CORS!)
That's where cors-anywhere is supposed to help. However, since the request is made in the client the Origin header is not set and we get the following error:
Missing required request header. Must specify one of: origin,x-requested-with

So, set the Origin header right? Unfortunately, most browsers disallow this. This might work for server-side code, but this doesn't work for client-side.
I have a working solution (which I'll link to) where we use a "fake" header (like x-cors-origin that is then substituted in for Origin. I tested it in postman:
GET http://localhost:8080/https://github.com/Rob--W.atom
HEADERS:
  "x-cors-origin": "https://github.com/Rob--W.atom"

RESPONSE:
  <?xml version="1.0" encoding="UTF-8"?>
    <feed xmlns="http://www.w3.org/2005/Atom" xmlns:media="http://search.yahoo.com/mrss/" xml:lang="en-US">
    <id>tag:github.com,2008:/Rob--W</id>
    <!-- .... -->

Here's my given solution: https://github.com/src-r-r/cors-anywhere
Would you like me to submit a pull request?

The Origin header is intentionally not spoofable for requests from web browsers. That's why it is used to authenticate the client. When you add a custom header, then anyone can spoof this value, which makes it rather unfitting for verifying the origin.
The only remaining use of the custom header is to check that it is present (to distinguish it from direct requests, i.e. visiting the page in a web browser). For that, I have already added the X-Requested-With header, as you can see from the error message.

Missing required request header. Must specify one of: origin,x-requested-with

Just spoofing the origin header may be insufficient as some endpoints deny additional headers (triggering preflight requests that cannot be spoofed). What might work would be some means for the cors proxy to proxy specific endpoints according to some predefined swop list, without the use of custom headers.


Distributed workers stop reporting request stats

Hi. I am using locust4j 1.0.12 with locust master 1.4.3.  My master is on a remote host from the workers.  Frequently, I've noticed that when starting and stopping tests via the UI the workers stop reporting metrics to the master.  This does not happen every time, I have seen it restart successfully too.  But, I can get into this state pretty easily by just:

Bring up distributed worker
Start test in UI, let it run a few seconds.
Stop test in UI
Start new test in UI

Does not seem to be load related as it will happen even with very light load.  Restarting the worker restores the stats again but the issue can be reproduced.  Heartbeats appear not to be affected as the worker is reporting in its state and cpu stats without issue.
On another note, when I restart the worker, the worker count increases in the UI despite the Workers tab reflecting that the worker is missing (it also continues to report cpu stats for the missing worker).  Presumably this is all because I am still using the same host when I restart the worker.
Let me know if I can provide additional details.  I have been using tcpdump to inspect the traffic on port 5557 and its pretty clear that the heartbeat is consistently sent but the stat packets stop being sent.  I can also see that the workers are actually executing the tasks just not reporting the stats for them.

I'll also note that when this happens, my worker quickly runs out of memory:

fwiw, I have not yet been able to reproduce this running with a local master and worker on the same host.

@lolwhitaker Thanks for reporting, when you reproduce this issue, can you get me a jstack sample? And if you used tcpdump to confirm no stat packets, I think the state of runner may be wrong, see aslo: https://github.com/myzhan/locust4j/blob/master/src/main/java/com/github/myzhan/locust4j/runtime/Runner.java#L385

Yea, I think you are right and the state is probably the issue.
I think I understand at least part of my issue now.  Locust is only setup to handle the SIGTERM as far as I can tell: https://github.com/locustio/locust/blob/b000d5f229e6a0bf7b6f5472667b29e925371538/locust/main.py#L417-L422
If I kill -9 the master process and then restart it, the locust workers will never report back in to the new master.  Using kill -15 locally I do see that they do properly deregister:
18:02:55.269 [locust4j-stats#0#stats-timer] stats.Stats [DEBUG] - Sleeping
18:02:58.228 [Thread-9receive-from-client] runtime.Runner [INFO] - Received message: stop
18:02:58.230 [Thread-9receive-from-client] runtime.Runner [DEBUG] - Recv stop message from master, all the workers are stopped
18:02:58.231 [Thread-9receive-from-client] runtime.Runner [INFO] - Received message: quit
18:02:58.231 [Thread-9receive-from-client] runtime.Runner [DEBUG] - Got quit message from master, shutting down...

I am using AWS to bring up and down my master and worker processes. As I understand it, when I terminate an instance in AWS it should be letting systemd gracefully shutdown the running processes (which sends a SIGTERM).  When the new master comes up it is on a different host, but behind the same elb via the same dns in route53.  They appear to register successfully with the new master to send heartbeats, but maybe some other state flag is preventing the stats?  I haven't yet confirmed exactly what happens in the AWS space from the worker perspective.  I will need to enable some debug flags on the worker and dig into this more to understand exactly what they get.  It probably is specific to how I am running things.

You can add some logs to trace the state of locust4j runner and the message sent by the master.

I never got to the bottom of this and don't think I ever will at this point.   I think there still might be some edge case with the states lurking, but its probably a pretty hairy scenario to get there and better tracked in a simpler use case if someone is able to repro.


SD1.5 and SD2.1 not suitable for DDIM inverse

I tried the code in playground_real.ipynb on SD1.4, SD1.5, and SD2.1, and found that the inverse process using DDIM can only generate images that are consistent with the original image for SD1.4. For SD1.5, the generated images have significant deviations, while for SD2.1, the reconstructed images through the inverse process completely collapse.

Hi @shouwangzhe200, for the failure of DDIM inversion, you can increase the success rate by:

Increase the inversion steps. In our script, the default steps for inversion is only 50, thus you can use larger inversion steps to obtain satisfying results (\eg, 500). This may alleviate the poor reconstruction quality problem with the SD1.5 model.
As for the collapse of SD2.1, you should be cautious about the prediction_type of the U-Net. That is to say, the checkpoint in the hugging face repo here is trained with velocity prediction (prediction_type="v_prediction" rather than the epsilon). While the implementation in our repo relies on the epsilon prediction, thus please ensure that you use their released model here trained with prediction_type="epsilon".

In addition, with the above suggestions, DDIM inversion still reconstructs the image with poor quality in some cases. Some following works like Null-text Inversion can further alleviate this. You can refer to https://github.com/TencentARC/MasaCtrl/issues/30#issuecomment-1685575445 for more details.
Hope this can help you.

Thank you for your very detailed explanation, which has been very helpful to me. Another question is, why does SD1.4 only need 50 steps for inversion, while SD1.5 needs 500 steps?

Hi @shouwangzhe200, for the failure of DDIM inversion, you can increase the success rate by:

Increase the inversion steps. In our script, the default steps for inversion is only 50, thus you can use larger inversion steps to obtain satisfying results (\eg, 500). This may alleviate the poor reconstruction quality problem with the SD1.5 model.
As for the collapse of SD2.1, you should be cautious about the prediction_type of the U-Net. That is to say, the checkpoint in the hugging face repo here is trained with velocity prediction (prediction_type="v_prediction" rather than the epsilon). While the implementation in our repo relies on the epsilon prediction, thus please ensure that you use their released model here trained with prediction_type="epsilon".

In addition, with the above suggestions, DDIM inversion still reconstructs the image with poor quality in some cases. Some following works like Null-text Inversion can further alleviate this. You can refer to #30 (comment) for more details.
Hope this can help you.

Thank you for your very detailed explanation, which has been very helpful to me. Another question is, why does SD1.4 only need 50 steps for inversion, while SD1.5 needs 500 steps?

Hi @shouwangzhe200, in my view, this is not true, and this phenomenon can be case-specific and uneven. Some images can be reconstructed well when only 50 steps are employed for inversion on SD1.5 too, and many images still cannot be inverted successfully with 500 steps on SD1.4. In general, more denoising steps help obtain higher reconstruction quality.

@ljzycmd Thank you for your great job! May I ask if for Inversion, it is necessary to use the training method on the epsilon prediction, or can I still use inversion on the training method on the v_prediction?

Hi @LiuShiyu95, the prediction type during the inversion process should comply with the prediction type during model training. You can use the DDIMInverseScheduler for the inversion: https://huggingface.co/docs/diffusers/api/schedulers/ddim_inverse.
Hope the above can help you.

I understand, thanks very much!


Potential issue with FactorVAE discriminator loss

Hello,
I am writing about the discrimnator loss for FactorVAE. Should the two values in discr_loss be negative? It seems like the probs_z[:, 0] is the probability that the sample comes from the joint distribution, but the line that I linked would encourage the values in probs_z[:, 0] to go towards 0 instead of 1.

Sorry, I just realized that -discr_loss is minimized instead of discr_loss

Thank for the great implement, I have a related question.
In DCI metric, dose the dimention D of code C (in 2 THEORETICAL FRAMEWORK of the paper) affect the output number? If so, how it affect disentanglement, completeness and informativeness? Can I use this metric to compare two latent codes  with different length?
Thank you very much for your help.
Best Wishes,
Alex


CSL JSON and CSL YAML exporters: ignore Fields to omit from export setting

[x] I have verified in the Zotero addons screen that I have the latest release of BBT (https://github.com/retorquere/zotero-better-bibtex/releases/latest) and Zotero installed
[x] I am posting a single question, bug or feature request
[x] I have included a descriptive subject of the problem
[x] I am available for follow-up questions and testing
[x] I have included the error-report ID here after selecting the problematic reference(s), right-clicking, and submitting an BBT error report from that popup menu

Report ID:
NNWWEVRY-apse
Exporters used:
Better CSL JSON and Better CSL YAML
Configuration option:
Export option Fields to omit from export (comma-seperated) has an export field added e.g., abstract.
Expected behaviour:
In this example, the abstract field is not included in the exported file,  as occurs for Better BibLaTeX and Better BibTeX
Actual behaviour:
In this example, the abstract field is included in the exported file.

Please also include a debug log (see the last checkbox).

:robot: this is your friendly neighborhood build bot announcing test build <IP_ADDRESS>30 ("awaiting test case from debug ID")
Install in Zotero by downloading test build <IP_ADDRESS>30, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

Sorry about that, wasn't sure it was a bug or a feature request. Report ID addd to first post. Will now test your test build and get back.

Thank you. I confirm that test build <IP_ADDRESS>30 resolves the issue.

Both the bug report and the feature request templates ask for a debug report - in fact I still prefer to have one because it gives me a test case to add to my test suite.

I've added a test case to prevent regressions; I'm closing this because it's folded into master, when #1178 passes I'll cut a new release.


Export RenderTask type

While using TypeScript, I noticed that the RenderTask type is being returned out of pageProxy.render(renderContext), but the package "pdfjs-dist/types/display/api" isn't exporting this type.
This PR adds RenderTask to the export group.
/**
 * Sample pseudo-code; I'm not bothering with proper async management in this snippet.
 * Proper async management is left as an exercise to the reader.
 */
import {
  PDFDocumentProxy,
  PDFPageProxy,
  RenderParameters,
  // RenderTask <-- Missing ~ being added in this PR
} from "pdfjs-dist/types/display/api";
import { PageViewport } from "pdfjs-dist/types/display/display_utils";

const pdfProxy: PDFDocumentProxy = getProxySomehow();
const pageProxy: PDFPageProxy = pdfProxy.getPage(1)
const viewport: PageViewport = pageProxy.getViewport({ scale: 1 });

const renderContext: RenderParameters = {
    canvasContext: getContextSomehow(),
    viewport: viewport
  };

// Type RenderTask is missing
const task: RenderTask = pageProxy.render(renderContext);

Thank you.

Thanks for the PR, however we unfortunately cannot accept extending the official API-surface only for the TypeScript definitions; please refer to e.g. https://github.com/mozilla/pdf.js/pull/13930#issuecomment-906201600 and the links provided therein for additional information about an acceptable work-around.

Thanks @Snuffleupagus for taking a look at the PR.
I looked at the proposed workaround in the supplied link, but I think it requires me to build the code myself instead of consuming the types from pdfjs-dist.
Anyhow, if anyone else is dealing with this issue in TypeScript, I just provided a stub interface and moved on.
Example:
  // Pseudo-code again
  interface RenderTask {
    promise: Promise<void> // getter only
  }
  const task: RenderTask = pageProxy.render(renderContext);
  task.then(doSomething);

I looked at the proposed workaround in the supplied link, but I think it requires me to build the code myself instead of consuming the types from pdfjs-dist.

I realize that it maybe wasn't clear, but the point is that we'd accept a patch that implements the idea from e.g. https://github.com/mozilla/pdf.js/issues/12384#issuecomment-695143287 since that should (hopefully) fix the pdfjs-dist types without you needing to build them yourself.

Hi @Snuffleupagus
I tried adding
/**
 * @typedef {import("./display/api").RenderTask} RenderTask
 */

into src/pdf.js as shown in the linked comment, then run gulp types, and checked the content in build/types/src/pdf.d.ts and found the following output:
export type RenderTask = import("./display/api").RenderTask;
import { addLinkAttributes } from "./display/display_utils.js";
import { getFilenameFromUrl } from "./display/display_utils.js";
... // more stuff

So far so good!
The problem now is that type RenderTask is mapped to any, which doesn't really help:

And in a way it makes sense to me, given that RenderTask is not being exported from src/display/api.js.
Maybe I'm missing something? Any further guidance is much appreciated!

The problem now is that type RenderTask is mapped to any, which doesn't really help:

That's indeed unfortunate!
I suppose that as long as the RenderTask isn't exported directly through src/pdf.js it might be OK if it's exported in src/display/api.js, however we really don't want to have "unused" exports anywhere in the code-base (it could e.g. be at constant risk of being cleaned-up at any point).
Possibly we could reconcile the points above by adding unit-tests, please let me look into this a little bit more.

Please check if PR #14013 helps.

Please check if PR #14013 helps.

I checked this other PR, but it's the same story: For the solution to work, the symbols need to be exported.


driver Update - Oracle 18.5

I was getting the infamed error:
C:\Go\pkg\tool\windows_amd64\link.exe: running gcc failed: exit status 1
C:/TDM-GCC-64/bin/../lib/gcc/x86_64-w64-mingw32/5.1.0/../../../../x86_64-w64-mingw32/bin/ld.exe: cannot find -loci
collect2.exe: error: ld returned 1 exit status

And since I couldn't find the problem with my oci8.pc, I tried updating the driver, but then I got the following error:
C:/Oracle/18.0.0/dbhomeXE/oci/lib/msvc/oci.lib: error adding symbols: File in wrong format
collect2.exe: error: ld returned 1 exit status

I then downloaded the latest instantclient (18.5) but got the same error.
Is the driver incompatible with the Oracle 18+?

I have tested it with 12.2 and 11.2 and know those version will work. I gather from other people's posts that it works with the newer versions of instantclient. One of the post was for 18.3.

For anyone having this problem, I solved both problems by downloading instantclient 18.5 and SDK and going back to x86_64-w64-mingw32... The file in wrong format was an issue of TDM-GCC-64.
@MichaelS11, this can be closed. Thank you for your response!

@thigaz Glad you got it fixed, thanks for letting us know! Please close this issue.


⚠ Bofrost.es has degraded performance

In 82fba7f, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1503 ms

Resolved: Bofrost.es performance has improved in d40cb04 after 10 minutes.


[Event Request] Codeunit 1620 "PEPPOL Validation": Run Event OnBeforeRun

Hi.
I need an event in Codeunit 1620 "PEPPOL Validation" at the begininig of the OnRun trigger since the localization I am using (Norwegian) always runs this codeunit when E-Invoice is enabled for a Sales Header. I therefore have no way of using the standard E-Invoicing functionality for a different format.
The event needs to have either an IsHandled parameter or return value.
The OnRun trigger could look like this:
    trigger OnRun()
    var
        IsHandled: Boolean;
    begin
        OnBeforeRun(Rec, IsHandled);
        if IsHandled then
             exit;

        CheckSalesDocument(Rec);
        CheckSalesDocumentLines(Rec);
    end;

and the event like this:
    [IntegrationEvent(false, false)]
    local procedure OnBeforeRun(SalesHeader: Record "Sales Header"; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[Evaluation] Simplify eval & and multi-processing related fixes

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
Follow up of https://github.com/OpenDevin/OpenDevin/pull/2775 and https://github.com/OpenDevin/OpenDevin/pull/2805.
Give a brief summary of what the PR does, explaining any non-trivial design decisions

Pass the entire config to metadata, instead of llm config
Set max_iterations to global config (I've found a bug on previous main branch code due to this)
Standardize that the agent should be created inside process_instance function, so that we can avoid the ERROR:root:<class 'AttributeError'>: Can't pickle local object 'LLM.__init__.<locals>.wrapper' occurred in https://github.com/OpenDevin/OpenDevin/pull/2805 when trying to run eval in multi-processing.

Other references

Hey @neubig, i modify this PR a bit and try to remove as much dependencies on config as possible. However, i think we might still need to use config.llm (read from config.toml) to get the llm_config.