Wrong path in cell of active notebook causes failure to reformat

I was testing the command
nbqa black notebook.ipynb --nbqa-mutate

from here and while figuring out dependencies (black was missing) ended up having that command in a code cell in the active notebook I wanted to test the mutation on, with the path to the notebook (stated as correctpath below) originally mis-specified (stated as wrongpath below).
When calling the command (from a terminal) on that notebook, with the path specified correctly, I got the error message
error: cannot format correctpath/notebook.ipynb: Cannot parse: 38:5: nbqa black wrongpath/notebook.ipynb --nbqa-mutate
Oh no! 💥 💔 💥
1 file failed to reformat.

I would expect nbQA to warn about the missing file, but not fail because of it.

Hey @Daniel-Mietchen
Thanks for the report! This was fixed yesterday (#393), we just hadn't made a release yet. I'll fix something else up and then I'll push out a new version

I've made a new release (0.3.6), if you could try it out and let us know if the issue isn't fixed we'd really appreciate it :pray:

@MarcoGorelli I just retested it with the new version 0.3.6, and the error message remained the same.

Hi @Daniel-Mietchen
Thank you so much for your report! I can't reproduce this though: I just downloaded the notebook, ran it, and this is what I see:

I'm thinking about what the issue could be... @nbQA-dev/nbqa-core any ideas?

I think I see what's happening - you actually do have a notebook called <> saved, and that one has the line
nbqa black 100days/100days-Day-038.ipynb --nbqa-mutate

which is not valid Python syntax.
So, I think it's correct that this error message is shown

Yes, changing the line numbers such that they refer to the notebook cell rather than the script would make sense.
I wasn't aware that nbQA is meant to fail reformatting a notebook if there is wrong syntax anywhere in any cell, but I agree that this makes sense in a lot of contexts.
The set of notebooks that I am currently working on is meant to test the documentation of learning processes though, which means I want to encourage students (or anyone else who happens to come upon such resources) to see the mistakes as well as the processes they trigger towards any solutions. With that in mind, I essentially avoid re-running a cell to fix a problem, and I want to encourage students to do the same.
Ideally, a tool like Provbook would keep track of the different versions of a given cell, so the errors and associated learning process could still be documented, and nbQA (especially with a --nbqa-diff flag, as per #352 ) could assist in finding a solution by pointing out some obvious things to check.

I wasn't aware that nbQA is meant to fail reformatting a notebook if there is wrong syntax anywhere in any cell, but I agree that this makes sense in a lot of contexts.

I wasn't aware that nbQA is meant to fail reformatting a notebook if there is wrong syntax anywhere in any cell, but I agree that this makes sense in a lot of contexts.

Currently, if the code quality tool you're running (in this case, black) throws an error when it finds invalid syntax, then so will nbqa black. If the code quality tool you're running doesn't throw errors on invalid syntax (e.g. isort doesn't), the neither will nqba (e.g. nbqa isort Downloads/100days-Day-038.ipynb doesn't error)
We could do some hacking to avoid that, but I'm inclined to say we shouldn't

Yes, changing the line numbers such that they refer to the notebook cell rather than the script would make sense.


nezha-agent: 0.16.9 -> 0.16.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/nezhahq/agent/releases.
meta.description for nezha-agent is: Agent of Nezha Monitoring
meta.homepage for nezha-agent is: https://github.com/nezhahq/agent
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.16.10 with grep in /nix/store/p6ry75y9yipn66xqwx7s6r8j1dgqhiy8-nezha-agent-0.16.10
found 0.16.10 in filename of file in /nix/store/p6ry75y9yipn66xqwx7s6r8j1dgqhiy8-nezha-agent-0.16.10

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

nezha-agent

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/dd89f445a918c408c471507ef50157155e7e1be8#nezha-agent

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/p6ry75y9yipn66xqwx7s6r8j1dgqhiy8-nezha-agent-0.16.10
ls -la /nix/store/p6ry75y9yipn66xqwx7s6r8j1dgqhiy8-nezha-agent-0.16.10/bin

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

1 package built:

nezha-agent

Maintainer pings
cc @Moraxyc for testing.

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

1 package built:

nezha-agent


Revert "Show relevant error in UI"

This PR proposes a revert of OpenDevin/OpenDevin#2516.
Please check out the history of this issue:

sending str(e) in the ErrorObservation has been shown to leak private info: screenshot
we have tried a while ago to remove the raw exceptions from the UI chat box. Ref https://github.com/OpenDevin/OpenDevin/pull/1296: this PR has removed all exceptions / errors from being displayed in the chat box. Instead, it replaces them with some custom exceptions that have user-friendly messages, expressing the problem in plain language or directing the user to the logs
it can be unexpected to users and to the agent, to get in chat (and in history) some backend errors that the agent tried to fix even though it cannot do that (example)

I re-opened https://github.com/OpenDevin/OpenDevin/issues/1451 to look for a better way.
Maybe we could split the send_error function into the UI part and the backend stuff we do there? The exception there was intended only to be saved in state for use in evals, and not for user's chatbox, but that might be the problem here, we do different things in that spot, and it's not always clear how to keep them separate. 😅 I don't know, but I do think it's better to revert this and choose an alternative.

The user will mask if so like in the screenshot itself.
the errors in the example in the 3rd point are refactored into friendly messages. So, no problem?
they shouldn't go to the LLM as mentioned here


Add REST API

Hello,
Would it be possible to add a REST API in order to be able to generate the certificate from any other development language? It would:

increase it's usability using curl/python/cobol...
reduce the bandwidth required (save the planet)
All coders will be grateful

Thanks for your commitment.

Yes indeed it would be a very useful feature

I don't really understand why it's necessary (or even useful) as you can generate a qr code (offline) with so much apis.

The certificate is a service, and every service should have a REST API to my opinion, to make it sleek and scalable

I think this project is only about giving an alternative to a paper travel certificate which does not require a REST API. I don't know what is displayed on a police officer's phone but I suppose that it is only the info given by the QR code. Therefore, a REST API may be useless and may generate an unnecessary traffic.
Nota Bene: I agree with you on the fact that a REST API is generally better but in this case I think it may create concerns about data collections. I thank the team working o. This project for their whish to bring transparency.

The idea of the REST API was only to provide the fields as json to generate the QR code and the pdf. In this case there is no collection of data.
I did not wanted to create side apps for the service but extend it.

Hi @eramox @vartur,
Please take into account that this project has a limited lifetime. The goal of Johan Padarnaud and after that, the Interior Ministry was to quickly publish a simple, reliable tool to generate the form.
There is no point in asking to develop such feature, as well as asking for optimisation or useless features.
It don't see any reason that the Ministry would like to invest more time in a service that is actually functional and will be forgotten in a few weeks.
Keep in mind that the website and the project will not be used more than one or two months in the future.
Best regards,
Adam

Also noting that an API will resolve these issues;
https://github.com/LAB-MI/deplacement-covid-19/issues/98
https://github.com/LAB-MI/deplacement-covid-19/issues/96
https://github.com/LAB-MI/deplacement-covid-19/issues/92

It looks like the objective is to generate the certificates on the client side, so that no data hits the server. API will create traces on servers.

and even if it was hitting a server,  a form-url-encoded POST  would also qualify as "REST" ,  REST != Json API ,  but as it's not the case, having an API would actually require more infrastructure and be less environment-friendly.
At the opposite yes having the format of the json that is then qrcode encode would be nice so that one can implement it in whichever language he wants without doing any network call :)

I’m working on a proof-of-concept hack to use the url instead of an additional POST request.
All customization entirely done in the browser
Nginx server on Ubuntu 18.04 with config something like:
server {
    listen       80;
    server_name  *.example.org;
    ...
}

The catch-all may or may not be a security issue, I’m not a sys admin...
I’d be glad to post a link if anyone is interested

I've done that in the same nature : https://oeil.github.io/deplacement-covid-19/

Project source & hosted on Github-pages : https://github.com/oeil/deplacement-covid-19

I'm using Siri to generate my certificate each time I need.

I've done that in the same nature : https://oeil.github.io/deplacement-covid-19/

Project source & hosted on Github-pages : https://github.com/oeil/deplacement-covid-19

I'm using Siri to generate my certificate each time I need.

Perfect thank you 🍻


Bft block period to be configurable using transitions

Make the block period configurable using the transitions mechanism.
Acceptance Criteria

Block period can be changed at a specific block for IBFT2
Block period can be changed at a specific block for QBFT

Hello @jframe when you have something please ping me for i can test

Hey @helderjnpinto will let you know shouldn't be too far away. Currently working on some common code between IBFT2 and QBFt to allow so that we be able to create this transition first. Once that is done, will be able to add this transition.

@jframe Hi, after a long time off i will test this feature finally, sorry i suddenly disappeared, i was father recently :)

@helderjnpinto Congratulations on becoming a father. We closed this as we tested the feature working. Let us know if you find any problems though. It's included in the Besu 20.10.1 release https://github.com/hyperledger/besu/blob/main/CHANGELOG.md#21101.


Viewport based block-/section-visibility

Authoring classes for blocks and sections to toggle their visibility based on viewport.

hide-on-mobile: Only hide on mobile
hide-on-tablet: Only hide on tablet
hide-on-desktop: Only hide on desktop
show-on-mobile: Only show on desktop
show-on-tablet: Only show on tablet
show-on-desktop: Only show on mobile

@kronnox looks good from my side. Can we have some of documentation on how to implement on blocks and page sections?


Moving hardcoded values to controller environment variables, and config maps ready (Consumer Scheduling and Scaling)

Signed-off-by: aavarghese<EMAIL_ADDRESS>Part of #1537

/cc @pierDipi

@pierDipi sorry!! There was a newline lint err. Can you lgtm again please?

@pierDipi I missed to specify the seconds unit for refresh period. Oops. When you get a chance, one more time please? :)


Segmentation fault - abrupt crash when training

Hello guys,
I ran the 'train_end2end.py' module (without modification) found within the rcnn example that was provided with the mxnet package. Each time, I ran the training, the training would abruptly stop before the completion of even the first node, stating 'segmentation fault (core dumped)' and with the exact same error. I used gdb to traceback the error (provided below), but I am not familiar with c++. Could anyone kindly assist me on this error?
Environment info
Operating System: Ubuntu 16.04 LTS
Compiler: cmake
Package used (Python/R/Scala/Julia): Python
MXNet version: 0.93
If you are using python package, please provide
Python version and distribution: 2.7.13
Error Message:
Thread 20 "python" received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7fff8affd700 (LWP 15392)]
0x00007fffe5a97028 in std::_Function_handler<void (mxnet::RunContext, mxnet::engine::CallbackOnComplete), mxnet::Engine::PushSync(std::function<void (mxnet::RunContext)>, mxnet::Context, std::vector<mxnet::engine::Var*, std::allocatormxnet::engine::Var* > const&, std::vector<mxnet::engine::Var*, std::allocatormxnet::engine::Var* > const&, mxnet::FnProperty, int, char const*)::{lambda(mxnet::RunContext, mxnet::engine::CallbackOnComplete)#1}>::_M_invoke(std::_Any_data const&, mxnet::RunContext&&, mxnet::engine::CallbackOnComplete&&) (__functor=...,
__args#0=..., __args#1=...) at /usr/include/c++/5/functional:1869
1869	      _M_invoke(const _Any_data& __functor, _ArgTypes&&... __args)

@precedenceguo

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!


Disable the free kakera snipe

The bot is running the rolls perfectly, with few pauses, but there's still no way to disable the free kakera snipe.
To be perfect would need the "soulemoji_list" working.
Would it be possible for the bot to use $tu after claiming a waifu, so that it gets the new claim time?

You can disable it via the code yourself its pretty simple to see the code that does the free kakera check
in regards to soulemoji a new way has to be suggested to determine "soulmates" and the previous way was pretty inefficient. possible that im gonna remove the soulmate_emoji variable tho as it only used by the end game users and Users havent brought it up
the bot already get the new claim timer based on the $settings

basically $tu has no difference in terms of whose $tu so if let say I type $tu the bot will now determine that my $tu is now ur new timer and will cause "False" positives hence making you never able to snipe because as long as one person rolls $tu then the bot will be set
of course i could probably make it to where it could determine who rolled $tu but then again that is more computational power and you already had issues with "Slow" roll times
I repeat though the current implementation is that if the bot finds a $settings it will get another claim timer

You can disable it via the code yourself its pretty simple to see the code that does the free kakera check

The only way I saw it would be, for an absurd amount here, but it would still click in time

in regards to soulemoji a new way has to be suggested to determine "soulmates" and the previous way was pretty inefficient. possible that im gonna remove the soulmate_emoji variable tho as it only used by the end game users and Users havent brought it up

Soulmate is not something endgame as you say, as it is possible for those starting today, to have 2 waifu with lvl 10 key.
Running the bot on a new server myself, I managed in just over 1 month 4 waifu key level 10+ starting from level 0.

the bot already get the new claim timer based on the $settings

basically $tu has no difference in terms of whose $tu so if let say I type $tu the bot will now determine that my $tu is now ur new timer and will cause "False" positives hence making you never able to snipe because as long as one person rolls $tu then the bot will be set

What I've already noticed is that, if he uses a claim at the last minute, it will reset and won't use it in the next 2 hours, I'm forced to close the bot every time it takes a waifu, so that it takes the reset time

The changes to the code you made doesnt change anything for the 1st one i recommend if you dont enjoy it just remove the whole function
I do not have a soulmate and I have no ability to personally test any soulmate sniping implementation so If your "Developer" can not test a function how does the Developer make a system to share. I was literally shooting in the dark with soul mate sniping.
the claim window is set automaticlly based on the $settings if you $settings are invalid or are not being read from a file then it will not be appropriate set the new claim window

The changes to the code you made doesnt change anything for the 1st one i recommend if you dont enjoy it just remove the whole function

I thought if I turned it off it would stop working perfectly, thanks for the reply

I do not have a soulmate and I have no ability to personally test any soulmate sniping implementation so If your "Developer" can not test a function how does the Developer make a system to share. I was literally shooting in the dark with soul mate sniping.

It is not necessary to have soulmater to test, just have key lvl 1, because kakera's claim would be based on embedcolor

the claim window is set automaticlly based on the $settings if you $settings are invalid or are not being read from a file then it will not be appropriate set the new claim window

It happened now, claim missing 1h to reset.
A character appeared after the reset and he didn't catch. I had to close and reopen the bot.
I will use $settings again and delete the existing file

the last minute claim is working fine if you dont want it either edit your Last_claim_window or turn true to false as testing it for 1 day it has been working fine. The last minute claim window was added to use a claim there is a risk to it so there is a understanding that it might claim something that is not what you want for the sake of not wasting a claim.
show me when the bot claimed, a list of the hours that resets happen ex:
3:56
6:56
9:56
12:56
and the claim that it ignored , $settings and channeldata pertaining to that channel
you can implement it back in based on a previous version but manually having to set embedcolor for each key'd character means that its no longer just start and forget about it so until a better way is known I will not be pushing a commit to add this feature back as this will just add more "checks" which will slow things down

the last minute claim is working fine if you dont want it either edit your Last_claim_window or turn true to false as testing it for 1 day it has been working fine. The last minute claim window was added to use a claim there is a risk to it so there is a understanding that it might claim something that is not what you want for the sake of not wasting a claim.
show me when the bot claimed, a list of the hours that resets happen ex:
3:56
6:56
9:56
12:56
and the claim that it ignored , $settings and channeldata pertaining to that channel

I will test for 24h

you can implement it back in based on a previous version but manually having to set embedcolor for each key'd character means that its no longer just start and forget about it so until a better way is known I will not be pushing a commit to add this feature back as this will just add more "checks" which will slow things down

Unfortunately the only quick way to identify your 10+ key would be to use these items:
1 (chaoskey) linked to, 2 (Owner)
3 embedcolor

in terms of kakera theoretically it could be added here:

but it would have to run react_m which is a get_message call on every roll twice which will slow thing down even more. and you already had issues with rolling taking to long. I can't see a way to add without double checking 1 message for both conditions

alright so the new commit should add it back but no in the setting json. you will have to make all changes in code and basically uncomment out all the code because I dont want to implement it without testing if it makes anything else slower

basically remove the # on every line of the changes

thank you very much, i will test


Neptune.ai logger slow, lags behind training

🐛 Bug

When running a script which trains multiple models after another, I came across the problem that the neptune.ai logger lags behind my training quite severely ( when the model has finished training, the logger is only about halfway there). This would not be such a big problem for me if the next model would start training while the logger still processes the logs of the previous model, but training continues only after the logger has finished.
I tried logging fewer things, only logging metrics per epoch and disabling hardware monitoring by uninstalling psutil. I don't know if this is really a bug, but even if it is not, is there maybe a way to bypass/mitigate this? Is this maybe related to the windows timing error from #1186 @jakubczakon?
To Reproduce
Steps to reproduce the behavior:

Use the neptune.ai logger (with Windows)
Log a metric

Expected behavior

The logger should keep up with training or continue logging previous models while another model is already being trained.

Hi @Dunrar are you closing loggers per model explicitly:
neptune_logger_1 = NeptuneLogger(close_after_fit=False)

# your logic

neptune_logger_1.experiment.stop()

neptune_logger_2 = NeptuneLogger(close_after_fit=False)

# your logic

neptune_logger_2.experiment.stop()

Could you elaborate on what do you mean by model here?

Yes, I tried that as well. By model I mean model = MemoryTest(hparams). My code looks like this:
agg_losses = []
    for celltype in ['RNN', 'LSTM', 'GRU']:
        setattr(hparams, 'celltype', celltype)
        for layer in range(1, 4):
            setattr(hparams, 'n_layers', layer)
            for cell in range(1, 6):
                setattr(hparams, 'n_cells', cell)
                agg_losses_pred_col = []
                for predict_col in range(1, 6):
                    setattr(hparams, 'predict_col', predict_col)
                    print(' Updated hyperparameters: ', hparams)
                    agg_losses_runs = []
                    for run in range(10):
                        exp_name = 'Type ' + celltype + ', Layers ' + str(layer) + ', Cells ' + str(cell) + ', PredCol ' + str(predict_col) + ', Run ' + str(run)

                        neptune_logger = NeptuneLogger(
                            project_name="dunrar/bachelor-thesis",
                            experiment_name=exp_name,
                            params=vars(hparams),
                            close_after_fit=False
                            # tags=['truncated bptt', 'cyclical learning rates', 'epoch data reload', ...]
                        )
                        if hparams.early_stopping == 'yes':
                            early_stopping = EarlyStopping(
                                monitor='batch/mean_absolute_loss',
                                min_delta=hparams.min_delta,
                                patience=hparams.patience,
                                mode='min'
                            )
                        else:
                            early_stopping = False

                        if hparams.reload_data == 'yes':
                            reload_data = True
                        else:
                            reload_data = False

                        model = MemoryTest(hparams)
                        trainer = pl.Trainer(
                            benchmark=True,
                            checkpoint_callback=False,
                            logger=neptune_logger,
                            reload_dataloaders_every_epoch=reload_data,
                            gpus=hparams.cuda,
                            val_percent_check=0,
                            early_stop_callback=early_stopping,
                            default_save_path=src.settings.LOG_DIR,
                            max_epochs=hparams.epochs
                        )
                        trainer.fit(model)
                        neptune_logger.experiment.stop()
                        agg_losses_runs.append(model.final_loss)
                    agg_losses_pred_col.append(np.mean(agg_losses_runs))
                agg_losses.insert(0, agg_losses_pred_col)

But even when I only train one model and the model stops/finishes training (CPU load goes down as well) it takes a lot of time until the logger catches up and the console becomes responsive again.

Got it, let me talk to the devs cause I think this should work nicely.

Hi @Dunrar , I'm one of the developers of Neptune.ai.
The stop() method waits for some threads to join. One of those threads is responsible for sending channel values and stderr/out.
My current suspicion is that this problem is related to another problem with stderr/out we have on Windows.
Can we check what happens when you do the following (upload_std*=False)?
neptune_logger = NeptuneLogger(
                            project_name="dunrar/bachelor-thesis",
                            experiment_name=exp_name,
                            params=vars(hparams),
                            close_after_fit=False,
                            # tags=['truncated bptt', 'cyclical learning rates', 'epoch data reload', ...],
                            upload_stderr=False,
                            upload_stdout=False
                        )

Does the problem persist then?

Hey @pitercl , that's it, way faster now. It takes care of the other problem as well, of course.

Hi @Dunrar, we're working on a fix. It should be release this week.

Hi @Dunrar, I've released neptune-client in version 0.4.108, which should fix the problem with colliding x-s in stdout/stderr.
However, the problem with slow closing of experiments seems to lie elsewhere. I'll look into that as well and see what can be done.
Just wanted to give you a head's up on the progress.

Thank you @pitercl, I will check it out tomorrow! upload_stderr=False and upload_stdout=False already took care of the neptune logger lagging behind the training and it closes with just a short delay. Seems fine to me, I can't compare it to anything though.

@pitercl @jakubczakon will you send changes to Pl via PR or is it only Neptune task/issue?

Hi @Borda, this is a Neptune-only task. I think this can be closed as it's not related to PL directly.


Repository

My Introduction
Name
Karin Cheng
Class
Tech 4
About me
I like animals, especially dogs.
Goals
Getting better at coding.
Skill level
Beginner
GitHub username
https://github.com/KarinCheng
GitHub repository
https://github.com/KarinCheng/Tech

👋 Hi Karin, the link to your repo has been added to the admin sheet. Thanks! 🎉


about train from scratch

If I want to train from scratch, which code should I run.

@guijuzhejiang which model did you ask?


WinUI 2.6 TabView places controls under window controls

Describe the bug
WinUI 2.6 TabView places controls under window controls
Steps to reproduce the bug

Open latest version of XAML Controls Gallery
Open TabView page
Scroll to "Complete TabView windowing sample"
Open Window with TabView
See bug

Expected behavior
"Add" button must be placed before Window controls

Screenshots

Version Info
WinUI 2.6
NuGet package version:
[Microsoft.UI.Xaml 2.6.0]

Windows app type:

UWP
Win32

Yes

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

October 2020 Update (19042)
Yes

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

Additional context

@Alkanores Thanks for reporting this! do you know if it is new to 2.6 or it it was present in 2.5 and before as well?

@Austin-Lamb and @pratikone does winui2 have the tools needed to avoid the Titlebar controls?

@Alkanores Thanks for reporting this! do you know if it is new to 2.6 or it it was present in 2.5 and before as well?

Yep, WinUI 2.5 also contains this bug.

Could you use a TabViewStripFooter that allocates space in that area to avoid the overlap as a workaround ?

Yes that should work. Question is if this is actually an bug with TabView or just the fact that the control was not used properly. I suspect that a NavigationView with DisplayMode set to top would also exhibit this behavior.

Fixed in winui3? @StephenLPeters


Move away from Jobs to Pods

The test-operator is using Jobs to spawn test pods even though it does
not use any features of this k8s object. Plus usage of the Jobs requires
creation of ServiceAccount in the target namespaces. In order to be able
to create a new, SA the test-oprator has to have a rights to create new
roles and rolebindings which in our case makes the attack surface
larger.
This patch drops the usage of Jobs and moves to Pods.
Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/2604

recheck

recheck

/hold
This PRs need to be merged at the same time. The idea is to first get /approved and /lgtm for both of them and then remove /hold at the same time:

https://github.com/openstack-k8s-operators/test-operator/pull/266
https://github.com/openstack-k8s-operators/ci-framework/pull/2604

/test all


Enable linking to a hidden page within another hidden page

Background: I really like the hidden metadata in the front matter as a replacement for draft, because that enables me to send my boss a link to a draft page without having him to pull the git project and start the hugo dev server. I often have multiple hidden pages and, as far as I know, my only option to make them visible for navigation is the children shortcode: {{% children showhidden="true" %}}
Therefore, it would be nice if the navigation buttons of a hidden page link to the next hidden page, only if the next page in the order is a hidden page. This should not  change the existing behavior that the navigation buttons of a not-hidden page should not link to a hidden page - even if it is the next page in the order.
(Moreover, it would be cool if the  hidden metadata would also work on topmost pages, so that they are not shown in the navigation pane.)

This sounds like possibly a limitation of Hugo rather than this theme. Instead of trying to have Hugo handle your draft pages could I suggest an alternative?
I maintain a private hugo website using this theme at my employer. We have a deployment pipeline configured so that whenever a branch is pushed to the git server a new copy of the website is published at a temporary URL using the git hash. For example:
https://staging.mywebsite.com/2b79e4b/index.html
Once the website is published at a temporary URL we open a pull request to review the code, and the reviewers can see a complete copy of the website to validate the markdown is rendering correctly.
This theme and others who forked it use Netlify to provide this functionality. I rolled something custom, actually there's a blog post about this going out soon. I'll provide the link when it's out.


Unable to use goose with Python 3

Using Google Collab
!pip install goose3
Version - Python 3.7.11
/content/goose/utils/__init__.py in <module>()
     27 import goose
     28 import codecs
---> 29 import urlparse

ModuleNotFoundError: No module named 'urlparse'

So this package is left unmaintained. Does anyone have any working alternative?


Add additional HA requirements from https://github.com/Azure/DASH/pull/56/ into high-availability-and-scale.md

https://github.com/Azure/DASH/pull/56/ contains the following content in the proposal document. If this is indeed the case, these requirements should go into https://github.com/Azure/DASH/blob/main/documentation/high-avail/design/high-availability-and-scale.md
Microsoft has provided some additional requirements:

HA Interoperability is required between vendors

Pairing cards from different vendors is not the typical deployment, but must work

The HA packet format and protocol must be public

This allows sniffed/mirrored HA messages to be analyzed
No vendor-private protocol is allowed

The HA protocol for syncing active flows could have a base mode and optional modes

Additional modes could be defined, for example to the reduce PPS/bps needed for the active sync messages
A vendor only needs to support the base mode
Any optional modes must also be public

The HA protocol does not need to reliably sync 100% of the flows between cards

Ideally all flows are synced. But is ok if a small number of flows (hundreds out of 10s of millions) are missed.

Hi @budgrise we triaged this and the consensus is: could you take the additional "MSFT requirements" captured above, which must have arisen from some conversation and do a PR to add it to the defining HA doc https://github.com/Azure/DASH/blob/main/documentation/high-avail/design/high-availability-and-scale.md? This way MSFT can review and hopefully just get it into the existing spec while we debate the new content you've proposed. Thanks.

Please, see also Xsight HA proposal.

@r12f will you be able to review this to see if those are still the correct requirements so we can close this issue?
in https://github.com/sonic-net/DASH/blob/main/documentation/high-avail/high-availability-and-scale.md i see
anything pending for this item ?

thanks @mgheorghe for bringing it up! will check on this one.


Not accessing all titles

When manually searching Libgen's Fiction for Inkheart by Cornelia Funke with filters for English & EPUB, I get numerous results.  For reference, here is the link.
However, when I search (by both author and title) for it using the api, I'm only able to get one pdf result and no epub results.  For reference, the pdf file's id number is 2439096.
This is happening for numerous titles, not just this one.

#32 I just saw this


Add Charmo University to list of University

Location: Peshawa Street & Chamchamal/Sulaimani Kurdistan region, F.R. Iraq.
Website: http://charmouniversity.org

@DiyarBakir Your website can not be open because it is possibly infected with CryptoMiner malware. Please tell your web server administrator that an anti-malware check can be necessary. I can not add this domain to the repository without seeing the official university website. Please comment here as soon as the website is back to normal work. Thank you.

@philipto Thanks for your support, the website works fine from our side, the link that I gave before not opened directly if you clicked it from Github.
you can check the website from: www.charmouniversity.org
your right I check the website it look like effected with Malware , I will tell the Administrator to check their server as soon as possible.
Thank you.

@DiyarBakir Pull request merged. Thank you!

@philipto  Thanks for your support.


endpoint: remove revision check around L4 policy calculation

Endpoint policy regeneration logic would check if the policy repository revision
changed from the endpoint's prior policy calculation to determine if the
endpoint's L4 policy should be calculated. However, endpoint L4 policy should be
calculated for a variety of other reasons, including endpoint identity change.
Thus, short-circuiting endpoint L4 policy regeneration only based of the
difference of the endpoint's policy revision and the policy repository's
revision is incorrect and could lead to cases where L4 policy for an endpoint is
not calculated.
Signed-off by: Ian Vernon<EMAIL_ADDRESS>Always recalculate endpoint L4 policy when regeneratePolicy is called

This change is

test-me-please

Will work on getting a test written for this as well...

Forgot a : which caused compilation problems... trying tests again.

test-me-please

We need to be careful backporting this to 1.0. I don't remember if we support at all identity changes for example in 1.0. I'll feel more comfortable if this, and all backport tests run k8s test as well, just to see if we get any flake or so.

@aanm I will definitely do that.

I'm not sure whether 1.0 is affected at all.

I'm working on a manual testing workflow (since it's hard to unit test this) and will test against each branch.

Validated that the issue is reproducible with latest master with the following manual test:

Launch the K8s Vagrant VM:

NFS=1 K8S=1 ./contrib/vagrant/start.sh

SSH into the VM:

K8S=1 vagrant ssh

Create deployments used in CI tests:

kubectl apply -f test/k8sT/manifests/demo.yaml

Apply policy which selects endpoints from demo.yaml:

kubectl apply -f test/k8sT/manifests/l3-l4-policy.yaml

Check cilium endpoint get for any endpoint with label app1:

$ cilium endpoint list
ENDPOINT   POLICY (ingress)   POLICY (egress)   IDENTITY   LABELS (source:key[=value])                            IPv6                 IPv4           STATUS
           ENFORCEMENT        ENFORCEMENT
19306      Disabled           Disabled          42100      k8s:id=app3                                            f00d::a00:0:0:4b6a   <IP_ADDRESS>   ready
                                                           k8s:io.cilium.k8s.policy.serviceaccount=default
                                                           k8s:io.kubernetes.pod.namespace=default
                                                           k8s:zgroup=testapp
50879      Enabled            Disabled          8697       k8s:id=app1                                            f00d::a00:0:0:c6bf   <IP_ADDRESS>     ready
                                                           k8s:io.cilium.k8s.policy.serviceaccount=app1-account
                                                           k8s:io.kubernetes.pod.namespace=default
                                                           k8s:zgroup=testapp
56075      Enabled            Disabled          8697       k8s:id=app1                                            f00d::a00:0:0:db0b   <IP_ADDRESS>    ready
                                                           k8s:io.cilium.k8s.policy.serviceaccount=app1-account
                                                           k8s:io.kubernetes.pod.namespace=default
                                                           k8s:zgroup=testapp
57111      Disabled           Disabled          55379      k8s:io.cilium.k8s.policy.serviceaccount=kube-dns       f00d::a00:0:0:df17   <IP_ADDRESS>    ready
                                                           k8s:io.kubernetes.pod.namespace=kube-system
                                                           k8s:k8s-app=kube-dns
57260      Disabled           Disabled          59404      k8s:appSecond=true                                     f00d::a00:0:0:dfac   <IP_ADDRESS>    ready
                                                           k8s:id=app2
                                                           k8s:io.cilium.k8s.policy.serviceaccount=app2-account
                                                           k8s:io.kubernetes.pod.namespace=default
                                                           k8s:zgroup=testapp
58487      Disabled           Disabled          4          reserved:health                                        f00d::a00:0:0:e477   <IP_ADDRESS>   ready

e.g., 56075 and 50879. Check their status.policy.realized.l4 fields:
$ cilium endpoint get 50879
[
  {
    "id": 50879,
...
    "status": {
...
      "policy": {
...
        "realized": {
...
          "l4": {
            "egress": [],
            "ingress": [
              {
                "derived-from-rules": [
                  [
                    "k8s:io.cilium.k8s.policy.name=l3-l4-policy",
                    "k8s:io.cilium.k8s.policy.namespace=default"
                  ]
                ],
                "rule": {
		  "port": 80,
		  "protocol": "TCP"
		}
              }
            ]
          },
...

Note that the ingress policy that applies at L4 is correctly plumbed.

Scale the app1 deployment from 2 to 3 replicas:

$ kubectl scale deployment app1 --replicas=3

Find the ID of the new endpoint with label app1 via cilium endpoint list. In my test, it was 37386.

Check the status.policy.realized.l4 field for this endpoint. It will be empty.
$ cilium endpoint get 37386
[
  {
    "id": 50879,
...
    "status": {
...
      "policy": {
...
        "realized": {
...
          "l4": {
            "egress": [],
            "ingress": []
          },
...

This is incorrect; the l4.ingress field should be populated.
I validated that my change correctly has the L4 policy getting plumbed (i.e., for the pod which was created from scaling the deployment, the policy was correctly generated).

test-me-please

can you add a CI test with your steps from #5515 (comment)

@aanm I added a CI test, but instead of checking the endpoint model, I perform a datapath check. I ran the test against latest master  as well (i.e., without the actual fix) and it failed as expected, indicating that the test validates my fix.

Confirmed locally that the issue does not affect v1.0. Removing according backport label.


upsun validate fails against config files

upsun validate fails after running upsun project:init with upsun v5.0.2

have a project
run upsun project:init
walk through the steps and answer as appropriate
after completion, run upsun validate
cli reports errors (see screenshot in slack channel)

Expected behavior: no validation errors
Actual behavior: reported validation errors
Links: N/A
Browser: N/A
OS: mac
User info:
❯ upsun auth:info
+-----------------------+--------------------------------------+
| Property              | Value                                |
+-----------------------+--------------------------------------+
| id                    | 9340d67e-a9d9-4d3e-a665-5e458c0ca1e0 |
| first_name            | Paul                                 |
| last_name             | Gilzow                               |
| username              | paul-gilzowatplatform-sh             |
| email                 |<EMAIL_ADDRESS>            |
| phone_number_verified | false                                |
+-----------------------+--------------------------------------+

@gilzow can you please confirm this CLI 4.10.1?

doesnt appear to be 😞
❯ upsunstg validate
Upsun (Staging) config validation failed: yaml: unmarshal errors:
  line 1: cannot unmarshal !!str `7d6agwz...` into map[string]interface {}
  line 3: cannot unmarshal !!str `proxy.u...` into map[string]interface {}
❯ upsunstg --version
Upsun CLI (Staging) 4.10.1-2023-09-19-11ad3d41-next (Wrapped legacy CLI 4.10.0)

Unless there's a newer 4.10.1 version?

Isn't this issue connected to https://github.com/platformsh/platformify/issues/150 ?

I thought it was a different issue, cool it's going to be resolved in that PR then. Sorry for the wrong ping.


[Issue]: I have to sometimes try 2-3 times to rip subtitles

Confirmations

[X] I have checked the issues tab, and couldn't find an existing open issue for the issue I want to report.

OS Type
Windows
Python Version
Python 3.12.4
Package Version
2.5.6
Description
Now and then some rips get various error. Trying to rip right away a second or third time works.
I think it started with version 2.5.4.
Looking back at output logs from February and June I can't find any similar errors, neither do I remember it being an issue.
isubrip_2024-07-16_00-17-47-ERROR1.log
isubrip_2024-07-16_20-45-33-ERROR2.log
Output Log
<ERROR 1>
Error while scraping 'https://tv.apple.com/se/movie/insidious-the-last-key/umc.cmc.4gosbjb9erx6np1zvn684pfv5?playableId=tvs.sbd.9001%3A1319448763': [WinError 10051] A socket operation was attempted to an unreachable network

<ERROR 2 - Second rip attempt error output log is correct. It has "empty" error info.>
Error while scraping 'https://tv.apple.com/se/movie/knox-goes-away/umc.cmc.18b7pkpx0uhxdnqn3ey64y09i': [Errno 11004] getaddrinfo failed
Error while scraping 'https://tv.apple.com/se/movie/knox-goes-away/umc.cmc.18b7pkpx0uhxdnqn3ey64y09i':

Config
[downloads]
# Folder to downloads files to.
# The default "." value means it will download to the same folder the script ran from.
# Use double backslashes in path to avoid escaping characters. Example: "C:\\Users\\<username>\\Downloads\\"
folder = "."

# A list of iTunes language codes to download.
# An empty array (like the one currently being used) will result in downloading all of the available subtitles.
# Example: ["en-US", "fr-FR", "he"]
languages = ["en", "en-US", "en-UK", "sv", "sv-SE", "da", "da-DK", "no", "no-NO", "fi", "fi-FI"]
#languages = []

# Save files into a zip archive if there is more than one matching subtitles.
# Value can be either 'true' or 'false'.
zip = true

# Whether to convert subtitles to SRT format.
# NOTE: This can cause loss of subtitles metadata that is not supported by SRT format.
convert-to-srt = false

Hey @insurw, looking at both of the attached log files, it seems that in both cases iSubRip worked as expected without any errors?
Did you perhaps attach the wrong log files?
Also, while I'll try my best to help, do note that network issues such as this, that are reported by single users and aren't wide-spread, usually indicate that the issue is specifically with your network / computer (firewall, network randomly disconnects, very slow connection, DNS resolving issues, etc.), like https://github.com/MichaelYochpaz/iSubRip/issues/71 for example.

Closing this issue as there has been no response for over a month.
Feel free to reopen / add a comment if it's still relevant. @insurw


Otel integration into flow

Description
Transform the OTEL integration into a flow based use case

Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

moved to catalog repo


Локализация мелких страниц из раздела "Структура каталога" (15 штук)

https://nuxt-ru.vercel.app/docs/guide/directory-structure/nuxt - docs/guide/directory-structure/nuxt.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/output - docs/guide/directory-structure/output.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/assets - docs/guide/directory-structure/assets.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/content - docs/guide/directory-structure/content.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/modules - docs/guide/directory-structure/modules.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/node_modules - docs/guide/directory-structure/node_modules.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/public - docs/guide/directory-structure/public.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/utils - docs/guide/directory-structure/utils.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/gitignore - docs/guide/directory-structure/gitignore.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/nuxtignore - docs/guide/directory-structure/nuxtignore.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/app - docs/guide/directory-structure/app.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/error - docs/guide/directory-structure/error.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/nuxt-config - docs/guide/directory-structure/nuxt-config.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/package - docs/guide/directory-structure/package.md
https://nuxt-ru.vercel.app/docs/guide/directory-structure/tsconfig - docs/guide/directory-structure/tsconfig.md

[ ] docs/guide/directory-structure/nuxt.md
[ ] docs/guide/directory-structure/output.md
[ ] docs/guide/directory-structure/assets.md
[ ] docs/guide/directory-structure/content.md
[ ] docs/guide/directory-structure/modules.md
[ ] docs/guide/directory-structure/node_modules.md
[ ] docs/guide/directory-structure/public.md
[ ] docs/guide/directory-structure/utils.md
[ ] docs/guide/directory-structure/gitignore.md
[ ] docs/guide/directory-structure/nuxtignore.md
[ ] docs/guide/directory-structure/app.md
[ ] docs/guide/directory-structure/error.md
[ ] docs/guide/directory-structure/nuxt-config.md
[ ] docs/guide/directory-structure/package.md
[ ] docs/guide/directory-structure/tsconfig.md


fix compilation after breaking API changes

the APIs have been kept very stable, but in these years there are two changes that prevent compilation:

using leptonica's internal Box struct
(the struct was removed from public API in https://github.com/DanBloomberg/leptonica/commit/e8b670d9357c78a6a93fa23e27bb25675c695fa3)

Tesseract's GenericVector and STRING were replaced by C++ stdlib equivalents
https://github.com/tesseract-ocr/tesseract/pull/3198

these have trivial fixes

Hi @mildsunrise - which specific version of Tesseract do you envision your changes should work with? There will need to be an update to the build process to make sure it's used instead of the current 4.1.0

@StephenHogg this PR was tested with the latest version (5.3.4), but it is a very reasonable ask to keep it building for older versions. we could add preprocessor directives for that, I'll try that when I have some time

That would be great! Thanks so much for all you're doing, we're really keen to incorporate all these changes


[BUG] - Sometimes methods with annotations aren't processed correctly

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I haven't been able to determine a cause yet but on Linux systems, occasionally a method with annotations will complete parsing incorrectly. The __annotatedMethod placeholder isn't always replaced with the method name.
Expected Behavior
No response
Steps To Reproduce
No response
Anything else?
No response

This is interesting, do you have an example source file I can try?

It's all good! Yeah, if you can piece together something where you can reproduce that would be incredibly helpful, thank you

The relevant code is the last method in this class.
@Controller("/api/v1/releases")
class Releases {
    private dataStore;

    init(dataStore) {
        this.dataStore = dataStore;
    }

    @Post
    new(request) {
        if (request.query.exists("id")) {
            const key = request.query.get("id");
            return {
                key: this.dataStore.get(key)
            };
        }

        return [this.dataStore];
    }

    @Get
    @Cache(10)
    retrieve(request) {
        if (request.query.exists("id")) {
            const key = request.query.get("id");
            return {
                key: this.dataStore.get(key)
            };
        }

        return [this.dataStore];
    }

    @Get
    @Path("/last")
    last(request) {
        const keys = this.dataStore.keys();
        const idx = this.dataStore.len()-1;
        const lastKey = keys[idx];

        return JsonResponse({lastKey: this.dataStore.get(lastKey)});
    }
}

print(Releases.methodAnnotations);

Results:
{"new": {"Post": nil}, "retrieve": {"Cache": 10, "Get": nil}, "__annotatedMethodName": {"Path": "/last", "Get": nil}}

I think it may actually be to do with dictDelete. Removing that call seems to have all the methods along with the placeholder

Okay yeah, it's the dictSet. The robinhood hashing is moving entries about so entry->key now points at something else. Should be resolvable


Error: /Login/token/staff can obtain token, but invalid for other API call.

Symptom Description:
I following the instruction installed lab environment on top of windows 10.
The key was generated from putty, the public key was directly saved from putty. The private key was export to OPENSSH key format as looks like Authentication service doesn't accept pkcs#12 format.
Issue 1
First the docker-compose handing on Attaching to sdk
PS C:\dev\sdk> docker-compose up
Building sdk
Step 1/5 : FROM williamyeh/ansible:alpine3
alpine3: Pulling from williamyeh/ansible
1160f4abea84: Pull complete
f39516e0568c: Pull complete
Digest: sha256:7b8d95ddca0736bfc6e81955aee04c5b8ff9597e8c68151b0893a84c6ec70b17
Status: Downloaded newer image for williamyeh/ansible:alpine3
 ---> 995ede431897
Step 2/5 : RUN mkdir -p /root/.ssh
 ---> Running in d57d28408250
 ---> 56eafb8050a6
Removing intermediate container d57d28408250
Step 3/5 : COPY sdk/ssh /root/.ssh
 ---> cd7a04d25bdf
Step 4/5 : RUN chmod -R 600 /root/.ssh
 ---> Running in 9999bf75f2e1
 ---> 25d5216a493a
Removing intermediate container 9999bf75f2e1
Step 5/5 : WORKDIR /etc/ansible/env/lab
 ---> 913f7c449017
Removing intermediate container 3cbd4889178a
Successfully built 913f7c449017
Successfully tagged sdk_sdk:latest
WARNING: Image for service sdk was built because it did not already exist. To rebuild this image you must use `docker-compose build` or `docker-compose up --build`.
Creating sdk ...
Creating sdk ... done
Attaching to sdk

I login to sdk container, checked all volumes been correctly mounted. So I leave it and continue deploy the lab environment.
Tried restart docker daemon and windows, no helps.
Issue 2
The deployment successfully done. Then I try to access those API by following the instruction.
First API call is Login/tokens/staff. I can get the token and I checked in the postman the token variable was updated to returned token by Login/tokens/staff api call.
Second API call I try to access User/users, then I got error 401.
{
    "code": 401,
    "message": "Invalid JWT Token"
}

Quesion:
Any suggestion how to fix issue 2? Possible issue 2 caused by issue 1?

For issue 2:
Seems like the authentication microservice is unable to validate the token. Perhaps the issue is related to the keys.
I have posted a sample private/public key pair here (the passphrase is 1111) for you to test to rule out if the issue is related to putty keys.
Either copy/paste the key files manually in each applicable microservices (C:\dev\sdk\app*\var\jwt) or copy the the key files in  C:\dev\sdk\resource\jwt\lab\ and re-run manually the ansible playbook
cd /etc/ansible/env/lab
ansible-playbookt jwt/download.yml

While on windows, I actually use the sdk docker container to generate keys (instead of putty) by running the following commands:
cd /srv/authentication/var/jwt
openssl genrsa -out key -aes256 4096
openssl rsa -pubout -in key -out key.pub

In the example above, I cd to a folder that's already shared with my host computer. You see the key appearing in windows explorer, from their I simply cut/paste them to the C:\dev\sdk\resource\jwt\lab folder.

Thanks @mario-digitalstate . The problem was fixed by your suggestion.
I verified that gained token validated by other service.
Issue closed.

Hey @marioprudhomme I am having the same issue, 401 Unauthorised and Invalid JWT key.
Can you please share the keys (kinda urgently)
Or let me know anything else to change besides jwt_key_pass_phrase. encryption.secret needs to be changed too (someone mentioned that)?


[5] Add functionality to RequestManager for new LPRequest class

Copied code from the old LeanplumRequest but for RequestManager

Can one of the admins verify this patch?


Replace .Wait(), .Result, .GetAwaiter().GetResult() with await .Confi…

…gureAwait(false) in Test projects
As a starting point of resolving the issue: https://github.com/stratisproject/StratisBitcoinFullNode/issues/565

@Neurosploit can you help review this as well?

@nopara73 we normally try to work with smaller commits so its easier to review.

Can you first clarify why configureawait(false) is needed at all? See http://blog.stephencleary.com/2017/03/aspnetcore-synchronization-context.html

Ha they dropped the AspNetSynchronizationContext wowoo so the aspnet core context is now multi threaded by default.
We can remove that in tests but in libraries (the fullnode) we probably keep it.

@Neurosploit What do you propose?
ConfigureAwait(false) in Stratis.Bitcoin, NStratis and Features projects
and not ConfigureAwait(false) in Tests, Api, Cli and Daemon projects?
@dangershony  What do you think?

Sounds good @nopara73.


License Scanning and Policy : manage unknown license with local file

Dependabot sometimes fails to get the license information as it is not well documented in a repository  for example:

https://github.com/pugjs/pug
https://github.com/jrburke/amdefine

The idea would be to:

each time we do a test and the licence is unknown:

log an issue/contribution in the source repository to allow Dependabot to recognize the license
add an entry in a this Action project that will be the list of project/url without license
use the information in the policy management with clear information about the fact that it is coming from local scann

@tgrall thanks for the feedback. Right now if a license is unknown by the Dependency Graph we log an error to the Action workflow.
https://github.com/GeekMasher/advanced-security-compliance/blob/main/ghascompliance/defaults/policy.yml#L18-L22
You can create and manage an import file containing the known licensing information.
https://github.com/GeekMasher/advanced-security-compliance/blob/main/examples/policies/conditions.yml#L33-L40

Thanks for this interesting project and your feedback on this issue.
It would be more helpful to me to enhance with a list of known-good instead of known-bad dependencies.
E.g. I'm using the following example:
licensing:
  conditions:
    ids:
      - GPL-*
      - LGPL-*
      - AGPL-*

  warnings:
    # Warning if the dependency isn't known
    ids:
      - Other
      - NA
  ignores:
    # known good packages
    names:
      - pip://packaging

However, this still produces warnings for pip://packaging. Is there another syntax available? Something with negation maybe?
  warnings:
    # Warning if the dependency isn't known
    ids:
      - Other
      - NA
    names:
      - ~ "pip://packaging"

@timdittler I am closing this issue now, if you want to talk more on this issue please raise a new one in https://github.com/advanced-security/policy-as-code


Properly display list items

A newline is required to tell asciidoctor to start a new list, and not append to the current paragraph.

Can you rebase to the 2.x branch please?

Sure. Is a new PR needed?

I'll re-open with a new PR.


Unable to bind (TCP)

Orange Pi PC with DietPi: http://www.orangepi.org/orangepibbsen/forum.php?mod=viewthread&tid=662&extra=page%3D1
Linux dns1 3.4.39-01-lobo #1 SMP PREEMPT Sun Oct 25 14:46:41 CET 2015 armv7l GNU/Linux

PRETTY_NAME="Debian GNU/Linux 8 (jessie)"
NAME="Debian GNU/Linux"
VERSION_ID="8"
VERSION="8 (jessie)"
ID=debian
HOME_URL="http://www.debian.org/"
SUPPORT_URL="http://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"

Version 1.6.1 of DNScrypt fails to launch with the following error:
root@dns1:~# dnscrypt-proxy -R dnscrypt.eu-nl
[INFO] + DNS Security Extensions are supported
[INFO] + Provider supposedly doesn't keep logs
[NOTICE] Starting dnscrypt-proxy 1.6.1
[INFO] Generating a new session key pair
[INFO] Done
[ERROR] Unable to bind (TCP)

It happens no matter which interface or port is used.
Version 1.6.0 launches without any problems.

It means that something else is already listening to port 53.
Maybe a DNS resolver, such as dnsmasq.

There's definitely nothing running that's using the same port; I've also tried binding it to different ports than 53.
Proto Recv-Q Send-Q Local Address           Foreign Address         State
tcp        0      0 *:ssh                   *:*                     LISTEN
tcp        0     36 <IP_ADDRESS>:ssh            <IP_ADDRESS>:51665        ESTABLISHED
tcp        0      0 <IP_ADDRESS>:ssh            <IP_ADDRESS>:51667        ESTABLISHED
tcp6       0      0 [::]:ssh                [::]:*                  LISTEN
udp        0      0 *:bootpc                *:*
udp        0      0 *:56161                 *:*
udp6       0      0 [::]:45909              [::]:*

Seems the Orange Pi is at fault - 1.6.1 works without any problems on a Raspberry Pi 2

Happens for me too on Banana Pi M1+, but only with certain distros. Debian Bananian has this issue, Ubuntu Armbian doesn't

Please look into this. The other commenters are not in error. I had been running an older version of dnscrypt-proxy and today decided to update my libsodium and dnscrypt-proxy to the lastest, so from 1.4.0 to 1.6.1, and I ran into this error. I compiled from source and installed the prescribed way on my Ubuntu 14.04.4 virtual machine.
I think it probably could be regarded as proof that in this situation, compiling and installing 1.6.0 right over the top of 1.6.1 -- and having it WORK -- means that 1.6.1 has a problem.
[NOTICE] Starting dnscrypt-proxy 1.6.1
[INFO] Ephemeral keys enabled - generating a new seed
[INFO] Done
[ERROR] Unable to bind (TCP)
[NOTICE] Starting dnscrypt-proxy 1.6.0
[INFO] Ephemeral keys enabled - generating a new seed
[INFO] Done
[INFO] Server certificate #808464433 received
[INFO] This certificate looks valid
[INFO] Chosen certificate #808464433 is valid from [2015-09-11] to [2016-09-10]
[INFO] Server key fingerprint is E7AC:5C21:A4E6:6A90:B254:DD73:5229:3BA1:5BE9:8EB3:4E8F:E538:52DE:A2FB:DDB6:1357
[NOTICE] Proxying from <IP_ADDRESS>:40 to <IP_ADDRESS>:443

I changed nothing in between running 1.6.1 in a failure state and running 1.6.0 successfully. To help others reduce frustration I post this information, and look forward to 1.6.2! Thank you for creating this very useful software!

I'm also having issue with dnscrypt, you can see at #561 with Orange Pi Zero. The formula that worked with Raspberry Pi Zero / Raspbian didn't want to work with ARMBian / OrangePi Zero. (Google brought me here by the way when I was searching why it didn't work.) Actually, @chauffer 's comment made me try other distros, such as DietPi and Armbian Ubuntu. DietPi had the same issue but ARMBian Ubuntu worked as it should. So the source of the issue may be the kernel + Debian base itself. I'd suggest you guys try the same steps with Ubuntu.

Catch-up , please see the discussion #561 , the newly-released 1.9.3 version (and hopefully possible future versions) fixed the problem for me on OPi Zero ARMBian.


W7001 Mapping used in sar location not found

cfn-lint version: 0.24.6
Given this CloudFormation template:
AWSTemplateFormatVersion: 2010-09-09
Transform: AWS::Serverless-2016-10-31

Mappings:
  Alarms:
    SemanticVersion:
      Value: 1.0.0

Resources:
  MyApplication:
    Type: AWS::Serverless::Application
    Properties:
      Location:
        ApplicationId: arn:aws:serverlessrepo:us-west-1:000000000000:applications/myapp
        SemanticVersion: !FindInMap [Alarms, SemanticVersion, Value]

cfn-lint gives W7001 Mapping 'Alarms' is defined but not used
This is valid CloudFormation - FindInMap and Ref are (the only) allowed intrinsic functions for the Location of a Serverless::Application

I'll take a look into this.  We use the sam translator package to lint the resulting template.  It could be that the translator is removing that Mapping.  We have had to make some changes to handle this for some other things as well.

For tracking purposes this is the translated template.  We will have to check if mappings are used against the pre-transformed template.  I'll get that in.  Thanks for reporting this.
{
  "AWSTemplateFormatVersion": null,
  "Mappings": {
    "Alarms": {
      "SemanticVersion": {
        "Value": "1.0.0"
      }
    }
  },
  "Resources": {
    "MyApplication": {
      "Properties": {
        "Tags": [
          {
            "Key": "lambda:createdBy",
            "Value": "SAM"
          }
        ],
        "TemplateURL": "s3://bucket/value"
      },
      "Type": "AWS::CloudFormation::Stack"
    }
  }
}


Issue 482 Support executable copy instructions for image objects

Although copying of media objects is "not our problem" we should help the users of the stylesheets.
A precondition is that we know the filepath of the generated HTML file(s). This is the case when we do chunking ($cunk-output-base-uri).
I suggest to introduce a new attribute ghost:sourcefile for html elements for mediaobjects. I did it only for images, because i don't know if you like this approach.
With this new attribute, we can

generate executable copy instructions and execute them in the stylesheets. This is maximum support, but needs a function file:copy. Needs Saxon PE or EE Edition or a alternative implementation of expath:file. Works for users of Oxygen (Saxon EE included).
generate an external file with copy instructions (maybe a shell script). Not included in this PR, because result-document conflicted with XSpec.

Greetings, Frank

Withdrawn: an improved solution will be provided soon.
Frank Steimke


Unable to install Kresus

See log here:
https://gist.github.com/ArchangeGabriel/2ea4bb60c609fee316d3

Hi! You need to install Kresus dependencies, as explained in the README file: https://github.com/bnjbvr/kresus#on-any-other-unix-based-operating-system

Well, I followed the first part there: “On CozyCloud”. Maybe that only applies to CozyCloud instance, but given the proposed ways (opening a shell on the machine) I supposed this was generic and also apply to a self-hosted Cozy instance.
Also, the section your link point me at doesn’t apply, this would make Kresus as a standalone app, which is not what I want. Unless you were only talking about the dependencies part.
So, I suppose that the dependencies part also apply to Cozy. Maybe you should state it more clearly, and also have a message about this at https://github.com/bnjbvr/kresus/blob/master/client/components/LoadScreen.js ?
Thanks.

That is right, dependencies are needed on all installations (CozyCloud or standalone, Debian or not). That should be made clearer in the README and in the LoadScreen as well. That's a good suggestion, thanks.

OK. Question: is it python2 or python3 dependencies that are needed? Guess python2? Are there specific version needed for other packages? (I’m running ArchLinux, and the dependencies list is quite different, since for instance they are no -dev package on this distribution).

And are python-bindings for any of the required libraries needed?

@babolivier, can you help here please?

If you installed Cozy via one of the Archlinux packages, you should have the right dependencies installed.
Kresus requires Python2. I didn't tried running the app with Python3, but there were several modules for which the installation failed at Kresus's install when using Python 3.
Concerning python-bindings, it's not required. Or if it is, it was installed by another required Python package.

Indeed, except for libyaml I’m having all the required libraries. However, it still fails. New logs here:
https://gist.github.com/ArchangeGabriel/0708fb4e175f7c7ab55e
By the way, thanks @babolivier for ArchLinux packages.

My pleasure @ArchangeGabriel :)
libyaml is indeed missing in the dependencies. Thanks for noticing it, I'll update the package later (I created the issue https://github.com/babolivier/cozy-archlinux/issues/3 to remind me).
The errors seems like a package can't be installed properly because of a missing header file. Please check if libffi is installed on your machine.
The python : commande introuvable errors can be ignored, and will be fixed as soon as I finish working on #272.

$ yaourt -Ql libffi
libffi /usr/
libffi /usr/lib/
libffi /usr/lib/libffi-3.2.1/
libffi /usr/lib/libffi-3.2.1/include/
libffi /usr/lib/libffi-3.2.1/include/ffi.h
libffi /usr/lib/libffi-3.2.1/include/ffitarget.h
libffi /usr/lib/libffi.so
libffi /usr/lib/libffi.so.6
libffi /usr/lib/libffi.so.6.0.4
libffi /usr/lib/pkgconfig/
libffi /usr/lib/pkgconfig/libffi.pc
libffi /usr/share/
libffi /usr/share/info/
libffi /usr/share/info/libffi.info.gz
libffi /usr/share/licenses/
libffi /usr/share/licenses/libffi/
libffi /usr/share/licenses/libffi/LICENSE
libffi /usr/share/man/
libffi /usr/share/man/man3/
libffi /usr/share/man/man3/ffi.3.gz
libffi /usr/share/man/man3/ffi_call.3.gz
libffi /usr/share/man/man3/ffi_prep_cif.3.gz
libffi /usr/share/man/man3/ffi_prep_cif_var.3.gz

I think you need libffi-dev (to build kresus)

@ZeHiro On Archlinux, you don't have -dev packages, the base packages are the -dev (look at the paste from @ArchangeGabriel, libffi alone includes the dev headers).
Thanks for the paste, @ArchangeGabriel, I'll try to look into it tomorrow.

yeah, I always forget they can (are?) compiled on the flow

On the official repos, librairies aren't compiled on the flow. That only happens on AUR. What I'm trying to tell is that, on Ubuntu you usually have a "base" package with only the libraries, and a -dev package containing the .h files you can include to your program if you want to dev it with this library; but on Archlinux, you only have the dev one, containing both the library and the .h files (and that's why you usually don't find packages named "libxxx-dev" on the repos).

@babolivier Have you had the time to look at this? Thanks.

@ArchangeGabriel, there's an open pull request about this, iirc. In the meanwhile, I've enhanced the README on incoming and linked it from the load screen.

@bnjbvr Sorry, I wasn’t talking of the initial issue but my install one (https://github.com/bnjbvr/kresus/issues/289#issuecomment-158927369 and followings). Maybe should I open a different issue?

No problem. Feel free to comment directly on the PR that is going to solve this: https://github.com/bnjbvr/kresus/pull/272 (or if the PR isn't solving this, feel free to open a new issue). Thanks!

This PR is not solving the issue AFAIU (only removes an harmless warning here). I’m opening a new one for this, and close this one since fixed by https://github.com/bnjbvr/kresus/commit/d038d046168244499ff103937644ce4e8cb9728b.


Add upgrade wrapper for clusterctl and skeleton for clusterapi upgrader

This is part of #314
Description of changes:
In order to have smaller PRs, splitting this issue into multiple parts
This one adds the upgrade apply command wrapper to clusterctl and the skeleton for a clusterapi upgrader
The idea is that the clusterapi.Upgrader is a higher level entity that takes care of comparing versions and determine what has changed and needs to be upgraded. And clusterctl just generates the proper command to update those components to those specific versions
The clusterapi.Upgrader will probably be called from the upgrade workflow or from ClusterManager directly
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: g-gaston
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [g-gaston]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: danbudris, g-gaston
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [danbudris,g-gaston]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@g-gaston: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

eks-anywhere-attribution-files-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-attribution-files-presubmit

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@g-gaston: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

eks-anywhere-attribution-files-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-attribution-files-presubmit

eks-anywhere-e2e-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-e2e-presubmit

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@g-gaston: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

eks-anywhere-attribution-files-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-attribution-files-presubmit

eks-anywhere-e2e-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-e2e-presubmit

eks-anywhere-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-presubmit

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@g-gaston: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

eks-anywhere-attribution-files-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-attribution-files-presubmit

eks-anywhere-e2e-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-e2e-presubmit

eks-anywhere-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-presubmit

eks-anywhere-mocks-presubmits
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-mocks-presubmits

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@g-gaston: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

eks-anywhere-attribution-files-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-attribution-files-presubmit

eks-anywhere-e2e-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-e2e-presubmit

eks-anywhere-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-presubmit

eks-anywhere-mocks-presubmits
a58190d86420ecfc34a2de15e8682e7ea295685f
link
/test eks-anywhere-mocks-presubmits

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@g-gaston: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

eks-anywhere-attribution-files-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-attribution-files-presubmit

eks-anywhere-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-presubmit

eks-anywhere-mocks-presubmits
a58190d86420ecfc34a2de15e8682e7ea295685f
link
/test eks-anywhere-mocks-presubmits

eks-anywhere-e2e-presubmit
a58190d86420ecfc34a2de15e8682e7ea295685f
link
/test eks-anywhere-e2e-presubmit

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@g-gaston: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

eks-anywhere-presubmit
c9f464153c75b4bf74f1dd65de97f5c2da8b1e3d
link
/test eks-anywhere-presubmit

eks-anywhere-mocks-presubmits
a58190d86420ecfc34a2de15e8682e7ea295685f
link
/test eks-anywhere-mocks-presubmits

eks-anywhere-e2e-presubmit
a58190d86420ecfc34a2de15e8682e7ea295685f
link
/test eks-anywhere-e2e-presubmit

eks-anywhere-attribution-files-presubmit
a58190d86420ecfc34a2de15e8682e7ea295685f
link
/test eks-anywhere-attribution-files-presubmit

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Inference Service

I am following these instruction to get the inference service running.
But the realtime_inference_service.py is failing with following error --
Traceback (most recent call last):
  File "/Users/avnish5.kumar/Desktop/projects/speech/inference_service/realtime_inference_service.py", line 11, in <module>
    from model_service import ModelService
  File "/Users/avnish5.kumar/Desktop/projects/speech/inference_service/model_service.py", line 3, in <module>
    from inverse_text_normalization.run_predict import inverse_normalize_text
ModuleNotFoundError: No module named 'inverse_text_normalization'

I can see "inverse_text_normalization" module was not found but this is also not present in the project directory. How are we suppose to get this module during execution?
-- I ran the setup script before running the realtime_inference_service.py script.

There is a typo in setup_new_env.sh
In line3 creating a conda environment "fairseq3"
but in line4 activating conda "fairseq" [apparently I had "fairseq" env present that's why it didn't return any error.. kind of a silent fail]

There is a typo in setup_new_env.sh
In line3 creating a conda environment "fairseq3"
but in line4 activating conda "fairseq" [apparently I had "fairseq" env present that's why it didn't return any error.. kind of a silent fail]


Update zh translation in secret.md

Update zh translation in secret.md

This change doesn't look necessary.

/close


conflict in main.js unsolved

Hi @coastcdl, I have no idea how it happened but it should be fixed now!


Jenkins Swagger specifications update 20.14.0-SNAPSHOT

This is an auto swagger specifications update from Jenkins

Closed as typo discovered


Different materials

Let us make some interesting objects in the simulation.
there should be walls, stones on the ground, steps, trees, etc. Those could be represented by some fixed voxels. Let's call them material 1.
There should be stem voxels, so they will attach to what ever touches them. Let's call them material 2.
Then there should be materials 3-6 that only attaching the voxels with the same material. And material 7 doesn't sticky at all.
Any other creative ideas?

Check out fixed regions in voxcad/voxelyze. This locks the voxels into the environment and treats them differently and more efficiently than normal material.

Need more use cases and scenarios. Don't worry about implementation. ^_^


challenges fail when using 301 redirects

I've been using dehydrated for about a year now, without problems, but now challenges for domains which use a 301/302 redirect fail.
Example fail:
ERROR: Challenge is invalid! (returned: invalid) (result: {
"type": "http-01",
"status": "invalid",
"error": {
"type": "urn:ietf:params:acme:error:unauthorized",
"detail": "Invalid response from https://twitter.com/XXXXXXXXXXXXXXXX [<IP_ADDRESS>]:     \"\u003c!DOCTYPE html\u003e\\n\u003chtml lang=\\\"en\\\" data-scribe-reduced-action-queue=\\\"true\\\"\u003e\\n  \u003chead\u003e\\n    \\n    \\n    \\n    \\n    \\n    \\n    \\n    \u003cmeta c\"",
"status": 403},

This occurs when my domain uses a 301 redirect to another one (e.g. twitter.com in this example), but hasn't been a problem ever before.
OS: Debian stable (10.0)
dehydrated version: 0.6.5-1~bpo10+1

I'm not sure how this would ever have worked before. How would the CA validate your tokens if you don't serve them but instead redirect to Twitter?


New method for setting multiple on selects.

Chaining multiple() method:
$builder->select('people')->multiple();
Generates:
<select name="people[]" multiple="multiple"></select>;
Great for tags, etc.

Coverage increased (+0.06%) to 94.67% when pulling df7236291e4a0177a49c6320b7c7c4abafed64f7 on JerseyMilker:select-multiple into cff7aff5a7f7e213091053d2134d3db0d605f2fc on adamwathan:master.

I dunno if I like automatically appending the brackets, it seems easy enough to do $builder->select('people[]')->multiple() and I'd be worried it would be a pain if there was ever a situation where someone didn't want to have the brackets added, or wanted to control it manually.
Right now the package already automatically treats missing method calls as attribute setters, so multiple('multiple') does what you want to do here, which makes me think it might be a good idea to always pass the attribute name as the value for the attribute if a missing method is ever called with no parameters, you know?
That would give you this same API:
$builder->select('people[]')->multiple()
...without ever having to declare the multiple method on the select class.
Basically just means doing this in the Element class instead of what we have currently:
    public function __call($method, $params)
    {
        $params = count($params) ? $params : array($method);
        $params = array_merge(array($method), $params);
        call_user_func_array(array($this, 'attribute'), $params);
        return $this;
    }

I think I like that solution better personally because it can be applied to things like "required" and stuff too.
If you like that approach feel free to update the PR so you get the credit :)

Hmmm.
$builder->select('people[]')->multiple()

Reason I don't like this:
$builder->select('people[]')->multiple()
Is because I'm thinking in context of BootForms.  If I do this:
{!! BootForm::select('Names', 'names[]')->options(['Jesse Leite', 'Adam Wathan'])->multiple('multiple') !!}
Then 'names[]' with square brackets get rendered into more than just the name attribute:
	Names
	<select name="names[]" id="names[]" class="form-control" multiple="multiple">
		<option value="0">Jesse Leite</option>
		<option value="1">Adam Wathan</option>
	</select>

```
See `id` and `for` attributes. I assume this would make it into bootstrap help blocks for validation handling as well.  Allowing the `multiple()` method to handle the `[]`, means that the other attributes aren't affected when using extensions like BootForm.  Your thoughts?

Reason I don't like this:
$builder->select('people[]')->multiple()

Is because I'm thinking in context of BootForms.  If I do this:
{!! BootForm::select('Names', 'names[]')->options(['Jesse Leite', 'Adam Wathan'])->multiple('multiple') !!}

Then 'names[]' with square brackets get rendered into more than just the name attribute:

	Names
	<select name="names[]" id="names[]" class="form-control" multiple="multiple">
		<option value="0">Jesse Leite</option>
		<option value="1">Adam Wathan</option>
	</select>

See id and for attributes. I assume this would make it into bootstrap help blocks for validation handling as well.  Allowing the multiple() method to handle the [], means that the other attributes aren't affected when using extensions like BootForm.  Your thoughts?

...all that said, I agree with making the param optional:
->required()

Same as:
->required('required')

:)

I agree with making the magic method param optional, so that an empty required() method is same as required('required'), etc.
Reason I don't like this:
$builder->select('people[]')->multiple()

Is because I'm thinking in context of BootForms.  If I do as you suggest:
{!! BootForm::select('Names', 'names[]')->options(['Jesse Leite', 'Adam Wathan'])->multiple('multiple') !!}

Then 'names[]' with square brackets get rendered into more than just the name attribute:

	Names
	<select name="names[]" id="names[]" class="form-control" multiple="multiple">
		<option value="0">Jesse Leite</option>
		<option value="1">Adam Wathan</option>
	</select>

See id and for attributes. I assume this would affect bootstrap help blocks for validation handling as well.  Allowing the multiple() method to handle the [], means that the other attributes aren't affected when using extensions like BootForm.  And more food for thought: When would you ever use [] without multiple="multiple", or vice versa?
That said, if you want the option to manually $builder->select('people[]'), the Select class' multiple() method could regex for [] and only add to name if it doesn't already exist in name.  This could be a best of both worlds solution?  Your thoughts?

Coverage increased (+0.13%) to 94.74% when pulling f1e662454a8cf14daf49d69676721d226f982858 on JerseyMilker:select-multiple into cff7aff5a7f7e213091053d2134d3db0d605f2fc on adamwathan:master.

I committed my suggestion anyway.
This satisfies your requests:

Add [] manually.
Select's multiple() will add [] to name, but only if it doesn't exist.
Attributes added via magic method have optional param as you suggested in your code example.

This satisfies my requests:

BootForm::select('people')->multiple() knows name should have [], but doesn't add [] to other attributes, such as id, for, and attributes in Bootstrap help block.

Your thoughts? Let me know when you Form them ;)

Looks good after addressing my comments above :+1:

Good call, committed change regarding existing name attribute.
Also, I already have an assertion which tests for existing [] in multi-select name.  See SelectTest' > testSelectCanBeMultiple()`.

Coverage increased (+0.11%) to 94.72% when pulling 8e5968da46cd8a2f01ca461a4069ee9408cd50af on JerseyMilker:select-multiple into cff7aff5a7f7e213091053d2134d3db0d605f2fc on adamwathan:master.

<3


Unit tests for editor.go

Add unit tests for each of the exported functions in editor.go.


remove dep check

Coverage: 90.164% (+1.0%) from 89.177% when pulling 93f1761d7e4bb31d72f53cf059986790e409afc3 on remove_dep_check into 92ce42b7abd3162530bc74557534584c1f7a2b07 on main.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/37668

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/38637

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/102653


YAML error in rendering example.qmd

quarto_titlepages relies on features of Quarto that are new. To help us out,

quarto --version : 0.9.566
Tell us what operating system you are on: Windows 10 Enterprise, OS build 19044.2130

I run into this error when rendering example.qmd from my RStudio (using the "Render" button)

tex --version returns MixTex-TeX 4.4 (MikTeX 22.3)

Description
When I try to render example.qmd, I get the following YAML error in the Render tab:

I would troubleshoot more but... don't really know anything about yaml and am slightly scared of it. So this is as much as I've got! I think this may be related to a TeX package I need to get that scrbook format or something.
Steps to replicate

clone quarto_titlepages repo
open example.qmd in RStudio
click "Render" at the top of the RStudio console

Your Quarto version is very old. Current version is 1.2. I think you need to update Quarto. Note if you update RStudio, you should get Quarto version 1.1 which is enough.
Quarto comes with scrbook so you don't need to install that. Quarto should also install tinytex.

Also by the way, MixTex-TeX 4.4 (MikTeX 22.3) will likely cause you frustration when rendering PDFs in Quarto.  Try installing tinytex.  You can install from the R command line.  Instructions here: https://yihui.org/tinytex/

OK -- I installed both but I may need to update RStudio as you mentioned to get the quarto update; when I install the newest version it shows up as version 1.2 in my Packages tab, but when I put quarto --version in the console it still gives me 0.9.566. I guess I will update after getting some help from IT.

The 1.2 that you see in the Packages tab is the quarto R package; same name, different thing (). For some reason when you installed Quarto, it didn't get rid of the old one. You'll probably have to figure out how to delete/uninstall 0.9.566.


IndexError: list index out of range

(C:\Program Files\Anaconda3\envs\pdftomd) C:\Users\Administrator\PycharmProjects\pdftomd>python main.py test.pdf
Parsing test.pdf
Traceback (most recent call last):
  File "main.py", line 31, in <module>
    main(sys.argv)
  File "main.py", line 26, in main
    writer.write(piles)
  File "C:\Users\Administrator\PycharmProjects\pdftomd\pdf2md\writer.py", line 27, in write
    self._write_simple(piles)
  File "C:\Users\Administrator\PycharmProjects\pdftomd\pdf2md\writer.py", line 50, in _write_simple
    markdown = pile.gen_markdown(self._syntax)
  File "C:\Users\Administrator\PycharmProjects\pdftomd\pdf2md\pile.py", line 76, in gen_markdown
    return self._gen_table_markdown(syntax)
  File "C:\Users\Administrator\PycharmProjects\pdftomd\pdf2md\pile.py", line 290, in _gen_table_markdown
    intermediate = self._gen_table_intermediate()
  File "C:\Users\Administrator\PycharmProjects\pdftomd\pdf2md\pile.py", line 319, in _gen_table_intermediate
    bottom, rowspan = self._find_exist_coor(left, right, row_idx, horizontal_coor, 'horizontal')
  File "C:\Users\Administrator\PycharmProjects\pdftomd\pdf2md\pile.py", line 357, in _find_exist_coor
    coor = line_coor[start_idx + span]
IndexError: list index out of range

I works for some pdf files and fails sometimes. I try to figure out what is going on here but nothing comes out from my mind.

I forget to make cmap before using it. I will give it a try. Close it for now.

Same error occurs even after making cmap.

I finally figure out whats going on here.  The error was caused by multi-cells in row and column so that it caused the index out of range.


commit sql:first version

Anonymized sql

we have a few things todo

make the database name anonymous (e.g., q0_db)
remove the set and SET and DEPLOY statements, keep the SQL only
add a readme file @zhouxh19
upload the compressed data (if the data contains table or column names, also make it anonymous) to the given server
for internal datasets, make the column name anonymous (c0, c1, ...)
upload the benchmark code after full test


Upgrade package dependencies

Description
This PR removes the deprecated io/ioutil packages and updates all direct dependencies, notably the Open Telemetry dependencies.
GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

https://github.com/dell/csm/issues/916

Checklist:

[x] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[x] I have verified that new and existing unit tests pass locally with my changes
[x] I have not allowed coverage numbers to degenerate
[ ] I have maintained at least 90% code coverage
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] Backward compatibility is not broken

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

run e2e test

run e2e test

run e2e test

run e2e test

run e2e test

run e2e test

run e2e test

run e2e test

run e2e test

run e2e test

run e2e test

run e2e test

run e2e test

Manually ran e2e for helm. Results in description

e2e pass

e2e pass


add Set::to_json

Pull Request Test Coverage Report for Build 3491
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 82.347%

Totals

Change from base Build 3490:
0.003%

Covered Lines:
4189

Relevant Lines:
5087

💛  - Coveralls


QUESTION: How to use commands

Hey how to use commands through the discord?

Hi, you can check all commands through !help:

Below is an example of command execution:

I will close this issue for now. If you still have any questions, feel free to reopen this issue or submit another issue.


Clearer | Folders & Errors | Windows 10 Icon Support

My attempt to make the project look nicer, no idea if it helps.
Added HTML5 Boilerplate (https://html5boilerplate.com/)
Folder for the images
Removed some HTML errors

thanks


activerecord: Fix type of #*_joins, #includes, #eager_load and #preload

The following methods can take strings, symbols and hashes at the same time.  So the current definitions should be merged into one instead of overloaded.

ActiveRecord::Base#joins
ActiveRecord::Base#left_joins
ActiveRecord::Base#left_outer_joins
ActiveRecord::Base#includes
ActiveRecord::Base#eager_load
ActiveRecord::Base#preload
ActiveRecord::Relation#joins
ActiveRecord::Relation#left_joins
ActiveRecord::Relation#left_outer_joins
ActiveRecord::Relation#includes
ActiveRecord::Relation#eager_load
ActiveRecord::Relation#preload

refs: https://github.com/rails/rails/blob/e88857bbb9d4e1dd64555c34541301870de4a45b/activerecord/lib/active_record/relation/query_methods.rb#L586

Could you please fix the conflict?

Updated :-)


Improve MCF file resolution flow

Currently, all the MCF nodes are loaded into memory and resolved.  Instead, like TMCF/CSV flow we should do it in a streamed fashion.

Tracked in internal bug.


NativeBinaryHasFailedError when launching browsers from WSL2 or Linux without X11

What is your Test Scenario?
Use Windows Subsystem for Linux v2 to run TestCafe.
What is the Current behavior?
TestCafe throws a NativeBinaryHasFailedError when launching non-headless browsers with any test.
What is the Expected behavior?
Tests should start successfully.
What is your web application and your TestCafe test code?
It can be reproduced with any app and test.
WORKAROUND: use headless browsers or Windows shells (cmd/powershell).
Extracted from https://github.com/DevExpress/testcafe/issues/4652#issuecomment-578135072.
Your Environment details:

testcafe version:                   1.8.1
node.js version:                    any
command-line arguments:  testcafe chrome test.js
browser name and version:
platform and version:
other:

Connecting a browser as a remote from to a headless Linux container (no X11, no Xvfb) can cause these issue too.

Closed by #5293.


[REQUEST] Download Only...

Good afternoon!
I am writing in with a request to optimize the Videos section tab. Under the options tab located at the bottom of the videos tab:

I would like to see the following options added:

Check only "Day" Videos
Check only "Night" Videos
Check only specific location videos > And have a subsequent menu or option to select which specific location.

I realized this option would be a huge help after deciding that I wanted to check and download only nighttime videos but that there was no easy way to select just a given subset of them.
Thanks so much!

That's a good idea. I planned to rework the whole video selection/enabling process as part of adding a cache size limit but that's a quick stop gap in the meantime. I'll keep you posted !

That's a good idea. I planned to rework the whole video selection/enabling process as part of adding a cache size limit but that's a quick stop gap in the meantime. I'll keep you posted !

I saw that idea mentioned and I totally support that change as well, I think a cache limit will be a huge help as well! Thanks for your time and consideration!

So I added Check only day and night. I'll also add the submenu but didn't get it to work today, so it will be for a next beta ;)
https://github.com/JohnCoates/Aerial/releases/download/v.1.8.3beta3/Aerial.saver.zip

Just tested it out and it's great! Thanks so much, that was super fast! Using it right now!!


SOLR-16975: Remove loading of solr.xml from zookeeper (main branch only)

https://issues.apache.org/jira/browse/SOLR-16975
Solr will now fail and exit if it finds /solr.xml in zookeeper, with a message on how to remove it.
Have also cleaned up docs and some scripts that used /solr.xml as example of how to upload stuff to zookeeper.
The MiniSolrCloudCluster relied on uploading a provided solr.xml to ZK, now it instead copies the provided file to each Jetty.

So I fixed the last tests and back some ZK connection code to NodeConfig. It really does not belong there, but ZkFailoverTest uses it plus a sysProp to simulate cluster restart failure. It should be fixed but I don't want to tackle it here, so I added a TODO.
Final review welcome. @dsmiley ?


Crashed!

Randomly, while playing Taylor Swift. Probably her fault

I've also had this crash but thought is because my crappy internet.  I get it my starting some tracks quickly after eachother with enter (i.e. browsing the library)

Sorry for the delayed response, I've been busy with other things, I will take a look when I'm free

I also get this crash on occasion. Also, to add, if for whatever reason the app gets closed while syncing the database, it refuses to ever launch again and gives this error:

@CyanJDroid Do you think #21 is related to the first case? If so, I can't currently do anything about it, I will investigate, hopefully. As for the latter case the app doesn't run because the database is corrupt, you need to remove it (~/.local/share/jamdb in your case) and run the app again

Just to update, I haven't had any jam crashes ever since I opened this report. :knock on wood: