Feature support group concat with distinct and order by

Proposed changes
support group concat with distinct and order by

run buildall


JAMES-4003 [FIX] Avoid forwarding bounces as it might create infinite loops

Why Destination service might consider Header from do not match sender envelop and reject with a bounce.
This creates an infinite loop.
=> This is DIRTY FIXING
Real fix: do like outlook...

For local domains: like today.
For remote domains: Wrap the forwarded message ina nested mime part, set subject to the value of the mail.

This test failed on the last two builds: https://ci-builds.apache.org/job/james/job/ApacheJames/job/PR-2022/5/testReport/org.apache.james.transport.mailets/RemoteDeliveryErrorHandlingTest/remoteDeliveryShouldStoreTemporaryFailureAsPermanentWhenExceedsMaximumRetries_SMTPMessageSender__DockerMockSmtp_/
Issue? Flaky? Might need to double check

I suspect the recent build change to not retrigger the test and report it as failed...

Any thoughts on this work?


About time embedding algorithm algorithm

Hi,
When I read your function:
https://github.com/jasonkyuyim/se3_diffusion/blob/2c6405fb5a1c4a23354cc697c18ff478dff8546b/model/score_network.py#L35
i find your code is slightly different with the original DDPM in:timesteps = timesteps * max_positions. Is there any reason to add this line of code ?

Yes, DDPM uses discrete time so timesteps is a element of positive integers, $\mathbb{Z}^n$. This fits in with the original sinusoidal embedding which was originally developed for embedding integer positions. However, our time steps are continuous over [0, 1] so we multiple it by the max_positions to emulate integers. This is important because if we don't then the time step embeddings will be nearly identical (assuming max_positions = 10000). Maybe max_positions=1 works equally but I haven't thought about it too much.

Great! I'm curious about the choice of continuous time instead of discrete time. Does it correspond to the Stochastic Differential Equations (SDE) mentioned in your paper?

Yes exactly

Great! Thanks for your reply.


Add smoothing and minor modification in eh-FDTD

This PR adds smoothing and minor modification in eh-FDTD.
test.zip

For both examples, dielectric functions are correctly reproduced. So I merge this PR.

Thank you for checking and merging, @mn2007 san!


Crash on exit

After synching with lastest code, I am getting the following errors while char exits:
[2022-02-22 17:23:39,403] INFO       Run Shenk
Exception in thread Thread-7:
Traceback (most recent call last):
File "C:\Users<usr>\Miniconda3\envs\botty\lib\threading.py", line 973, in _bootstrap_inner
self.run()
File "C:\Users<usr>\Miniconda3\envs\botty\lib\threading.py", line 910, in run
self._target(*self._args, **self._kwargs)
File "C:\Users<usr>\d2r\botty\src\bot.py", line 153, in start
self.trigger_or_stop('init')
File "C:\Users<usr>\d2r\botty\src\bot.py", line 173, in trigger_or_stop
self.trigger(name, **kwargs)
File "C:\Users<usr>\Miniconda3\envs\botty\lib\site-packages\transitions\core.py", line 899, in _get_trigger
return event.trigger(model, *args, **kwargs)
File "C:\Users<usr>\Miniconda3\envs\botty\lib\site-packages\transitions\core.py", line 401, in trigger
return self.machine._process(func)
File "C:\Users<usr>\Miniconda3\envs\botty\lib\site-packages\transitions\core.py", line 1201, in _process
self._transition_queue0
File "C:\Users<usr>\Miniconda3\envs\botty\lib\site-packages\transitions\core.py", line 426, in _trigger
return self._process(event_data)
File "C:\Users<usr>\Miniconda3\envs\botty\lib\site-packages\transitions\core.py", line 435, in _process
if trans.execute(event_data):
File "C:\Users<usr>\Miniconda3\envs\botty\lib\site-packages\transitions\core.py", line 272, in execute
event_data.machine.callbacks(itertools.chain(event_data.machine.before_state_change, self.before), event_data)
File "C:\Users<usr>\Miniconda3\envs\botty\lib\site-packages\transitions\core.py", line 1123, in callbacks
self.callback(func, event_data)
File "C:\Users<usr>\Miniconda3\envs\botty\lib\site-packages\transitions\core.py", line 1144, in callback
func(*event_data.args, **event_data.kwargs)
File "C:\Users<usr>\d2r\botty\src\bot.py", line 366, in on_end_game
view.save_and_exit()
File "C:\Users<usr>\d2r\botty\src\ui_components\view.py", line 63, in save_and_exit
if click_save_exit(exit_button):
File "C:\Users<usr>\d2r\botty\src\ui_components\view.py", line 47, in click_save_exit
select_screen_object_match(match, delay_factor=(0.1, 0.3))
File "C:\Users<usr>\d2r\botty\src\ui\ui_manager.py", line 202, in select_screen_object_match
mouse.move(*convert_screen_to_monitor(match.center), delay_factor=delay_factor)
File "C:\Users<usr>\d3r\botty\src\screen.py", line 25, in convert_screen_to_monitor
x = screen_coord[0] + monitor_roi["left"]
TypeError: 'NoneType' object is not subscriptable

I've tried on a different pc it's working well. Guess it's related to my low-end configuration.
Can we do something like disable the fancy pattern recognition for low-end pc?

I've tried on a different pc it's working well. Guess it's related to my low-end configuration.
Can we do something like disable the fancy pattern recognition for low-end pc?

This should be fixed on latest master, try again on the PC that wasn't working

I've tried on a different pc it's working well. Guess it's related to my low-end configuration.
Can we do something like disable the fancy pattern recognition for low-end pc?

This should be fixed on latest master, try again on the PC that wasn't working

After syncing with latest code, the trace is gone. But 'exit' still doesn't work for me:
The 'menu' appears for  a very short time then disappears, leaving char stay in the game.
While the 'debug' shows:
Starting game #2
Waiting for templates: , etc.

I've tried on a different pc it's working well. Guess it's related to my low-end configuration.
Can we do something like disable the fancy pattern recognition for low-end pc?

This should be fixed on latest master, try again on the PC that wasn't working

After syncing with latest code, the trace is gone. But 'exit' still doesn't work for me:
The 'menu' appears for a very short time then disappears, leaving char stay in the game. While the 'debug' shows:
Starting game #2 Waiting for templates: , etc.

Could you give more detail? Perhaps video?

Video at the end of a game
Related debug:
[0.7.1-dev 2022-03-10 21:24:38,637] INFO       Run Eldritch
[0.7.1-dev 2022-03-10 21:24:38,637] DEBUG      Traverse from a5_town_start to a5_wp
[0.7.1-dev 2022-03-10 21:24:41,472] DEBUG      Select A5_WP (90.9% confidence)
[0.7.1-dev 2022-03-10 21:24:46,136] DEBUG      Waiting for templates: ['ELDRITCH_0', 'ELDRITCH_START']
[0.7.1-dev 2022-03-10 21:24:46,401] DEBUG      Found Match: ELDRITCH_START (96.4% confidence)
[0.7.1-dev 2022-03-10 21:24:46,486] INFO       Health Manager is now active
[0.7.1-dev 2022-03-10 21:25:02,222] INFO       End game. Elapsed time: 112.36s
[0.7.1-dev 2022-03-10 21:25:02,684] INFO       Starting game #2
[0.7.1-dev 2022-03-10 21:25:02,685] DEBUG      Waiting for templates: ['MAIN_MENU_TOP_LEFT', 'MAIN_MENU_TOP_LEFT_DARK', 'A5_TOWN_0', 'A5_TOWN_1', 'A4_TOWN_4', 'A4_TOWN_5', 'A3_TOWN_0', 'A3_TOWN_1', 'A2_TOWN_0', 'A2_TOWN_1', 'A2_TOWN_10', 'A1_TOWN_1', 'A1_TOWN_3']
[0.7.1-dev 2022-03-10 21:25:41,894] INFO       Force Exit
[0.7.1-dev 2022-03-10 21:25:42,747] DEBUG      Detect window thread stopped
Restored D2R window visibility

Are you running shenk as well? Looks like your trying to run eld only which it might be glitching it.

Yes I do run both. And I only captured the Eldritch part for video and debug.

Will look into this @offerduo. I can't identify yet what's causing the issue but it just might require reworking the save/exit logic.

@offerduo try out latest master, see if most recent PR helped

@mgleed After picking up the latest code, the login/logout works perfectly for me on my laptop.
Thanks for working on it.


.cs code files in Shared Projects are not formatted

If you use Shared Projects .shproj, the contents are not formatted here. Although the code actually belongs to the consuming .csproj projects.

Do you have an example of your setup? I am assuming the projects are using CSharpier.MSBuild. The dotnet tool just works on directory contents and has been working with the CSharpier VS plugin, which uses a shared project.

https://github.com/dailydevops/healthchecks/tree/feature%2Fsqlserver

Can you provide more than just a link? How do I reproduce the problem?

I have created a sample repository with the error image. - https://github.com/samtrion/bug-csharpier-945
By way of explanation, I use a Directory.Packages.props file in parallel with the .sln file to manage packages. Through this file I inject CSharpier into all projects.
The code inside the .csproj projects is formatted correctly, but the code inside the .shproj or .projitems is not formatted. There is a standard import between the projects.
The same error pattern can be observed via a link reference to a file.
<ItemGroup>
  <Compile Include="..\Test.Shared\ExtraLongMethodBody.cs" Link="ExtraLongMethodBody.cs" />
</ItemGroup>

Also in this case the file ExtraLongMethodBody.cs is not formatted.

Thanks, I can see the problem now, but there isn't an easy fix.
The way CSharpier.MSBuild currently works is by just running dotnet csharpier against the project directory. It doesn't currently understand anything about the project. It also will not format a file if it was included from outside of the project directory and will format files that were excluded from the project.
dotnet format does this to determine formattable files - https://github.com/dotnet/format/blob/452f6263277533da0af2b557003256d98fbecc27/src/CodeFormatter.cs#L171
There may be some other way to do this in the context of msbuild, but my googling hasn't turned up anything yet.


Gh 1939 optional core handlers

Codecov Report

Merging #1951 into develop will increase coverage by 0.12%.
The diff coverage is 83.33%.

@@              Coverage Diff              @@
##             develop    #1951      +/-   ##
=============================================
+ Coverage      64.75%   64.87%   +0.12%
- Complexity      2099     2185      +86
=============================================
  Files            874      840      -34
  Lines          28542    27464    -1078
  Branches        3583     3488      -95
=============================================
- Hits           18481    17816     -665
+ Misses          8630     8256     -374
+ Partials        1431     1392      -39

Impacted Files
Coverage Δ
Complexity Δ

.../src/main/java/uk/gov/gchq/gaffer/store/Store.java
70.42% <83.33%> (+0.05%)
0 <0> (ø)
:arrow_down:

...onutil/iterable/LimitedInMemorySortedIterable.java
47.22% <0%> (-2.78%)
15% <0%> (-1%)

...affer/rest/service/v1/example/ExamplesService.java

...chq/gaffer/rest/example/ExampleFilterFunction.java

...ava/uk/gov/gchq/gaffer/rest/JaxRsErrorFactory.java

.../gchq/gaffer/rest/service/v1/OperationService.java

...v/gchq/gaffer/rest/factory/UnknownUserFactory.java

...fer/rest/service/v1/GraphConfigurationService.java

...fer/rest/service/v2/example/ExamplesServiceV2.java

...est/service/v2/example/DefaultExamplesFactory.java

... and 43 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a669e77...18a8937. Read the comment docs.


Dispose service instances only once

This is a fix for aspnet/AspNetCore#2932. Unfortunately, just changing the collection to a hash set would not work as #505 requires this to be an ordered collection. So instead, I’m remembering what services I have disposed at the time they get disposed.
An alternative solution would be to have the hash set around as an instance member to avoid adding duplicate services to the collection. This would have the benefit that services are disposed in the reversed order of their first use. With this fix, they are disposed in the order of their last use. Not sure which of those is better; this version at least avoids having a second collection around all the time.

@poke thanks for the PR. I'm going to close this because we're merging this repo with https://github.com/aspnet/Extensions. After https://github.com/aspnet/DependencyInjection/pull/662 has been merged, please feel free to re-open this PR over there.
cc @davidfowl


Add an action that opens the plugin's tab

Note that we had com.virtuslab.gitmachete.frontend.actions.ideprobe.OpenTabAction until it's been moved to test code on #311; most likely this could be much less reverted + assigned some keyboard shortcut.

From https://github.com/VirtusLab/git-machete-intellij-plugin/issues/274#issuecomment-629621134:

Adding (Open) Machete Tab as an action to the main menu is an option. Probably under VCS > Git.


ci: setup ossf scorecard and codql workflows

CodeQL and OSSF scorecard are both complementary tools useful to detect vulnerabilities.
They also provide help on how to fix them.
There is also a score to publish on the readme.md
This PR includes the pin of GitHub Action and Docker images .
It also provides dependabot config for them as it is not easy to keep track of those hash manually.
Also using an updater is preconised by ossf.
I didn't configured it for go mod as it is the object of another PR
Signed-off-by: Matthieu MOREL<EMAIL_ADDRESS>

can we revert this. I don't agree with the pinning philosophy. And I also think there is limited benefit to dependabot on this repo. In my experience it is very noisy
I understand the perceived benefits, but I think that's for people who like busy work.
We do not want to pin, we want to always move with the latest when possible, and fix things as they break. It's just a different way of looking at the problem, but pinning is a major security risk so we don't want to do that.


Heavily document Sensitive and Deferred requirements

Old documentation did not go into necessary depths to ensure that the
user was provided with all the necessary information about how the
Deferred and Sensitive chaining work and why they are critical in
ensuring secure use of credentials. This PR ensures that we do that and
fixes some outdated examples.
What ticket does this PR close?
Connected to #198
Checklists
Change log

[ ] The CHANGELOG has been updated, or
[x] This PR does not include user-facing changes and doesn't require a CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code changes, or
[x] The changes in this PR do not require tests

Documentation

[x] Docs (e.g. READMEs) were updated in this PR, and/or there is a follow-on issue to update docs, or
[ ] This PR does not require updating any documentation

@sgnn7: I'm not sure that I understand why the double unwrap is needed.  Could you explain a bit more?

@alexkalish If you only have one Sensitive wrapping which is the default one for the secret returned by Conjur, if you print out the deferred method value, it will dump any parameters raw onto the console (including the API key and hostname when you use the long form of the signature). If you only have the single wrap but in the flipped places (have it on the method invocation itself), you will be able to print out the raw fetched secret to the log. Since we want neither, we have to wrap both the invocation and the retrieved value in the Sensitive type.

@alexkalish : @diverdane and I talked about this and since we require the API key to be Sensitive type already (and other params are not privileged), the double wrap seems unnecessary in the guidance. I am updating the documentation to reflect the new usage recommendation that does not require the .unwrap.unwrap.


EventHubTarget - Actual support for userproperty to match ServiceBus

ServiceBus has the concept UserProperty + SystemProperty. Align EventHub-Target with ServiceBus-Target.

symbols are in nuget
https://www.nuget.org/packages/NLog.Extensions.AzureEventHub/2.7.0

Thank you for the merge. Getting closer with deterministic builds #100


Add isFirst / isLast flags to the BreadcrumbData object

The flags can be useful to style the first / last breadcrumb differently. Things I've seen where this can come in handy:

Home icon before the first breadcrumb
Make the last breadcrumb just plain text / non-interactive
Add aria-current="page" to the last breadcrumb for a11y and SEO

Potential API:
{{#each (breadcrumbs) as |breadcrumb|}}

      {{#if breadcrumb.isFirst}}
        <Icon @name="home" />
      {{/if}}
      {{breadcrumb.title}}

{{/each}}

Not sure if this needs to be implemented in the addon itself since this can already be accomplished through app code. Maybe it's enough to document how to do it.
Example using ember-truth-helpers and has-next from ember-composable-helpers:
{{#each (breadcrumbs) as |breadcrumb index|}}
  {{#let (eq index 0) (not (has-next breadcrumb breadcrumbs) as |isFirst isLast|}}

        {{#if isFirst}}
          <Icon @name="home" />
        {{/if}}
        {{breadcrumb.title}}

  {{/let}}
{{/each}}

It's also possible to add custom is-first / is-last helpers which makes this cleaner:
{{#each (breadcrumbs) as |breadcrumb|}}
  {{#let (is-first breadcrumb breadcrumbs) (is-last breadcrumb breadcrumbs) as |isFirst isLast|}}

        {{#if isFirst}}
          <Icon @name="home" />
        {{/if}}
        {{breadcrumb.title}}

  {{/let}}
{{/each}}

Closing since this is a general issue and I think this is something that should be provided by Ember itself.


Wrong command doesn't get processed

add n/John Doe p/98765432<EMAIL_ADDRESS>a/John street, block 123, #01-01 d/13-05-2021 o/2 x Strawberry Cake : Adds a 2 x Strawberry Cake
this command doesn't get processed, like there's no change in the command output or anything indicating an error. Note the colon in the input.
weeeirddd rare bug I think.

I think this might be happening because of the regex that I set for order items so probably it isn't able to convert the order description to an order item. Will try allow the regex to accept special characters as well and see if it works.


add check to support docker alpine

My fork got out of sync.  To make easier, I will close this and resubmit a new PR


Set docker host for windows

@simonjhy could you try the fix https://github.com/liferay/liferay-cli/pull/73

@simonjhy  reports that this is no longer an issue with version. 0.1.2 cli.  Closing this accordingly.


ddev describe should show more information about 3rd party services

Describe the bug
As of v1.14+ ddev describe shows some information about third-party services like solr that have an exposed URL. However, it doesn't say anything about services that don't have an http url, and it should at least list them. For example, if rabbitmq or memcached is enabled, it should at least show in the output, and perhaps inform more output.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Version and configuration information (please complete the following information):

Host computer OS and Version: [e.g. Windows 10, macOS High Sierra]
Docker version information (use docker version) and insert it here.
ddev version information (use ddev version)
config.yaml contents for the misbehaving project
Do you have any custom configuration (nginx, php, mysql) in the .ddev folder? If so, have you tried without them?

Additional context
Add any other context about the problem here. Thanks!

Perhaps an environment variable like DESCRIBE_TEXT could be added. In the following example myserviceA is exposed and myserviceB is not exposed, both have that new environment variable.


fix(consensus): recover and insert tx to mempool to avoid inactivation

What this PR does / why we need it:

Add an insert_without_check interface for mempool.
Fix a bug that may cause inactivation. Recover signed transactions from wal when reboot. Then insert them to the mempool.
Add a check of the ordered transaction hash when commit.
Upgrade overlord version.

Which issue(s) this PR fixes:

Fixes #
Which docs this PR relation:
Ref #
Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:

/hold

/test unit

/hold cancel

/test chaos

/test chaos

/lgtm


[BACKPORT to v0.8] Replacing Bintray repository

PR related to task with backport: #2165
Backported also task: #2123

/azp run


duplicate error warning

closes #284
📑 Description
✅ Checks

[ ✅ ] I have updated the documentation as required
[ ✅ ] I have performed a self-review of my code

ℹ Additional context

Hey @AKHIL-DyC. This error should still pop up, but those should handle different errors. If there is an error when fetching the current list of templates, you should show the error with this pop up , but if there was an error validating new template, you should set the error here.
You can add the template validation error to state and render it the same way we do with the first pop up

Hey @AKHIL-DyC, do you have any updates on this?

sorry for late reply, I broke my Linux ,now I can't contribute. Also I have exams, feel free to unassign me


微博分享网页效果不理想，有没有办法加上文字或者图片呢？

推荐图文分享，分享内容后面跟上url

Specs satisfying the fake_weibo (from .symlinks/plugins/fake_weibo/ios) dependency were found, but they required a higher minimum deployment target.
iOS端所需最低版本是多少呢？

看podspec

非常感谢


improve JsonError handling

It seems the error field shouldn't be optinal after all.

You can watch the development stream on youtube.


The :disconnection log option is not used anywhere.

In the 0.9.1 tag of the repo in the lib/redix/connection.ex file, there are no instances where the :disconnection log type is used.
This feels like a bug, since messages with the word "Disconnected" and those are logged as :failed_connection.

Great catch! That was a mistake. Pushed a commit that should fix this.


preserve results of @log_to_file-decorated function

store value, returned by original_func, and return it after restoring stdout and stderr

Thanks.


Ansible Playbook

Hi ,
I want to create an Ansible Playbook, which will

create database
create VM instance and deploy service(Can be any application) on it
configure CDN with VM instance

I am stuck at step 3. Can anyone suggest any solution.
Thanks in advance.
Regards,
Anup

First elaborate on all the manual linux/unix commands

Ansible.txt
This my Playbook
user_data.txt
The user data file to install Redmine during Bootstrap
I am mostly stuck on getting the CDN DNS name as ouput and linking the load balancer to CDN

I dont want ansible playbook, please write the manual commands


What about suggester flag on properties inside complex propertie (collection)? (SDK)

I can set this flag on Azure UI but I don't found info about how to set it from SDK.
I tried "ParentProp.Childone" but no luck (ParentProp is a collection)

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

ID: 0c80cb97-e34a-1c2d-7e51-8104bf5079c8
Version Independent ID: 56a3fd9d-dd1b-3271-c208-5967b95f6514
Content: Add typeahead queries to an index - Azure Cognitive Search
Content Source: articles/search/index-add-suggesters.md
Service: cognitive-search
GitHub Login: @brjohnstmsft
Microsoft Alias: brjohnst

Seems it works with "ParentProp/Childone" https://stackoverflow.com/a/57548959/2336493

@gsubiran Is your issue resolved? Please let us know.

@gsubiran Is there an update?

We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.

@gsubiran Is there an update?

Yes thanks! Finally worked with the following syntax "ParentProp/Childone".

The document explicitly says that this does not work: "Fields must be of type Edm.String and Collection(Edm.String). " And should probably be updated with the parameters of how it does work now. Is it just parent prop/child prod, are their further functionalities that aren't explained? Further, 3 nodes down the side navigation is an article entitled "Model complex data types" which suggests that users should model their own indexes to utilize complex data types - while it's awesome that this is the direction things are moving, it really does need to be reflected in the associated documents (a la, this one).


Update 'gRPC' to v1.17.2-p0

I've followed this guide
step by step carefully. Yes

I've tested this package remotely and have excluded all broken builds.
Here is the links to the Travis/AppVeyor with status "All passed":

https://ci.appveyor.com/project/rbsheth/hunter/builds/21142619
https://travis-ci.com/rbsheth/hunter/jobs/166388489 (in progress)

Testing:

https://ci.appveyor.com/project/ingenue/hunter/builds/21150193
https://travis-ci.org/ingenue/hunter/builds/470543786?utm_source=github_status&utm_medium=notification

Uploading:

https://ci.appveyor.com/project/ingenue/hunter/builds/21150854
https://travis-ci.org/ingenue/hunter/builds/470556162?utm_source=github_status&utm_medium=notification

Released: https://github.com/ruslo/hunter/releases/tag/v0.23.81


[Help Wanted] Filtering comments out of hostfiles in Linux

Hi,
I'm using a hostfile from this project on my local dns server to block ads. However I have some problems in my script with inline comments:
# copy dns to all_domains (tempfile)
cat StevenBlack-hosts | grep '^<IP_ADDRESS>' | egrep -v '^$|#|<IP_ADDRESS>|<IP_ADDRESS>|::1' | cut -d " " -f 2 >> $all_domains
# include local blacklist and hostfiles from other sources
cat $manuell_blacklist >> $all_domains

# remove comments and empty lines
cat $all_domains | egrep -v '^$|#' | sort | uniq  > $all_domains_uniq
# dns entries should be resolved with zone containing <IP_ADDRESS>
cat $all_domains_uniq | sed -r 's/(.*)/zone "\1" {type master; file "\/etc\/bind\/db.blocked";};/' > $zonefile

This will remove entries like
foo.bar # This may interefere with example.com

@mueller-ma nice!   Thanks for this.
Probably should make a flag for this, and make what you do the default.

I can provide the full script, maybe to the wiki page. But there are some lines very quick and dirty

https://github.com/StevenBlack/hosts/wiki/Bind-DNS-Server
https://github.com/mueller-ma/block-ads-via-dns

Hey @mueller-ma this is great!

@StevenBlack It still has the limitation I mentioned in the first post

I'll try to change the python script today.

Done!  Closing!

Reopening this because I want to document and link to @mueller-ma's repo.


Ensure Dispatcher.__eq__ always returns a bool

Previously typeof(some_jit_func) == "nonsense" would return None instead of False

Test added


Database migration error

Hi team,
Thank for your great app but it is unable to create a new database schema using the docker image
Please check

Missing version for ActiveRecord::Migration
Unable to create the database from migrations

Workaround by

bundle exec rake db:schema:load instead of bundle exec rake db:migrate for the first time

Hi Phạm Tiến Thành,
Can you produce a list of the exact steps you did to reproduce the problem?
Thank you!

Hi @remi
Sorry for the late response, this issue only affects first-time installation.

When data migrate is empty, killswitch docker unable to create tables
After the first time deployed, the database contains the tables, and the docker container works normally

Error log
docker-compose up killswitch
[+] Running 2/2
 ⠿ Container killswitch-postgresql-1  Running                                                                                0.0s
 ⠿ Container killswitch-1  Created                                                                                0.7s
Attaching to killswitch-1
killswitch-1  | Executing migrations…
killswitch-1  | rake aborted!
killswitch-1  | StandardError: An error has occurred, this and all later migrations canceled:
killswitch-1  |
killswitch-1  | Directly inheriting from ActiveRecord::Migration is not supported. Please specify the Rails release the migration was written for:
killswitch-1  |
killswitch-1  |   class CreateApplications < ActiveRecord::Migration[4.2]
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:559:in `inherited'
killswitch-1  | /opt/killswitch/db/migrate/20131108183954_create_applications.rb:1:in `<main>'
killswitch-1  | /usr/local/bundle/gems/bootsnap-1.7.5/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:23:in `require'
killswitch-1  | /usr/local/bundle/gems/bootsnap-1.7.5/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:23:in `block in require_with_bootsnap_lfi'
killswitch-1  | /usr/local/bundle/gems/bootsnap-1.7.5/lib/bootsnap/load_path_cache/loaded_features_index.rb:92:in `register'
killswitch-1  | /usr/local/bundle/gems/bootsnap-1.7.5/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:22:in `require_with_bootsnap_lfi'
killswitch-1  | /usr/local/bundle/gems/bootsnap-1.7.5/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:31:in `require'
killswitch-1  | /usr/local/bundle/gems/zeitwerk-2.5.4/lib/zeitwerk/kernel.rb:35:in `require'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1045:in `load_migration'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1041:in `migration'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1037:in `disable_ddl_transaction'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1387:in `use_transaction?'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1379:in `ddl_transaction'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1328:in `execute_migration_in_transaction'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1302:in `each'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1302:in `migrate_without_lock'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1251:in `block in migrate'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1401:in `block in with_advisory_lock'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1416:in `block in with_advisory_lock_connection'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/connection_adapters/abstract/connection_pool.rb:462:in `with_connection'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1416:in `with_advisory_lock_connection'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1397:in `with_advisory_lock'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1251:in `migrate'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1086:in `up'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1061:in `migrate'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/tasks/database_tasks.rb:237:in `migrate'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/railties/databases.rake:92:in `block (3 levels) in <main>'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/railties/databases.rake:90:in `each'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/railties/databases.rake:90:in `block (2 levels) in <main>'
killswitch-1  | /usr/local/bundle/gems/rake-13.0.6/exe/rake:27:in `<top (required)>'
killswitch-1  | /usr/local/bin/bundle:23:in `load'
killswitch-1  | /usr/local/bin/bundle:23:in `<main>'
killswitch-1  |
killswitch-1  | Caused by:
killswitch-1  | StandardError: Directly inheriting from ActiveRecord::Migration is not supported. Please specify the Rails release the migration was written for:
killswitch-1  |
killswitch-1  |   class CreateApplications < ActiveRecord::Migration[4.2]
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:559:in `inherited'
killswitch-1  | /opt/killswitch/db/migrate/20131108183954_create_applications.rb:1:in `<main>'
killswitch-1  | /usr/local/bundle/gems/bootsnap-1.7.5/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:23:in `require'
killswitch-1  | /usr/local/bundle/gems/bootsnap-1.7.5/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:23:in `block in require_with_bootsnap_lfi'
killswitch-1  | /usr/local/bundle/gems/bootsnap-1.7.5/lib/bootsnap/load_path_cache/loaded_features_index.rb:92:in `register'
killswitch-1  | /usr/local/bundle/gems/bootsnap-1.7.5/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:22:in `require_with_bootsnap_lfi'
killswitch-1  | /usr/local/bundle/gems/bootsnap-1.7.5/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:31:in `require'
killswitch-1  | /usr/local/bundle/gems/zeitwerk-2.5.4/lib/zeitwerk/kernel.rb:35:in `require'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1045:in `load_migration'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1041:in `migration'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1037:in `disable_ddl_transaction'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1387:in `use_transaction?'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1379:in `ddl_transaction'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1328:in `execute_migration_in_transaction'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1302:in `each'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1302:in `migrate_without_lock'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1251:in `block in migrate'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1401:in `block in with_advisory_lock'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1416:in `block in with_advisory_lock_connection'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/connection_adapters/abstract/connection_pool.rb:462:in `with_connection'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1416:in `with_advisory_lock_connection'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1397:in `with_advisory_lock'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1251:in `migrate'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1086:in `up'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/migration.rb:1061:in `migrate'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/tasks/database_tasks.rb:237:in `migrate'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/railties/databases.rake:92:in `block (3 levels) in <main>'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/railties/databases.rake:90:in `each'
killswitch-1  | /usr/local/bundle/gems/activerecord-<IP_ADDRESS>/lib/active_record/railties/databases.rake:90:in `block (2 levels) in <main>'
killswitch-1  | /usr/local/bundle/gems/rake-13.0.6/exe/rake:27:in `<top (required)>'
killswitch-1  | /usr/local/bin/bundle:23:in `load'
killswitch-1  | /usr/local/bin/bundle:23:in `<main>'
killswitch-1  | Tasks: TOP => db:migrate
killswitch-1  | (See full trace by running task with --trace)
killswitch-1 exited with code 1

Thank you

@tienthanh2509 Version 1.0.5 should fix this! 🎉


IL2CPP does not support marshaling delegates that point to instance methods to native code.

Hello, I keep getting these Errors while trying to open the file browser in a  standalone build.
using Unity 2021.2.17f1 Personal with IL2CPP.
I imported Mono.Posix and Mono.WebBrowser successfully, and it worked in the previous unity version.
After updating to the newer Unity version, there were some weird problems with plastic that got fixed after restarting.
It still works well in the unity editor, but on build accessing the file browser will always throw this exception
NotSupportedException: IL2CPP does not support marshaling delegates that point to instance methods to native code. The method we're attempting to marshal is: System.Windows.Forms.CommonDialog::OwnerWndProc
at System.Windows.Forms.CommonDialog.ShowDialog (System.Windows.Forms.IWin32Window owner) [0x00000] in <00000000000000000000000000000000>:0
at AnotherFileBrowser.Windows.FileBrowser.OpenFileBrowser (AnotherFileBrowser.Windows.BrowserProperties browserProperties, System.Action`1[T] filepath) [0x00000] in <00000000000000000000000000000000>:0
Does anybody know a fix?

Do you fix this problem? I have the same problem!

same issue ,seems Ookii.Dialogs.dll not support il2cpp


Revert long post as ScrollView instead of View

Summary
Reverting an unwanted change

@migbot the View was cause I was trying some things to avoid panning. But it was not needed at all


Change this stat on the repo

Hello @sanjay-kv, I am interested in this issue, can you assign it to me?

Hello @sanjay-kv, I just create pull request related to this issue. Please check that request and let me know if there are any updates needed.


Best 60 & 120 seconds HRR (Heart Recovery Rate) for each ride

http://www.enhancedmedicalcare.com/2013/01/20/improve-heart-health-by-knowing-your-recovery-heart-rate/
http://www.livestrong.com/article/260805-the-recovery-heart-rate-time-after-cardio-exercise/

Done with commits on best drop/rise in best splits


various issues after update ( helm-install-traefik, dns errors)

Environmental Info:
K3s Version:
k3s version v1.24.4+k3s1 (c3f830e9)
go version go1.18.1
Node(s) CPU architecture, OS, and Version:
Linux kubernetes 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Cluster Configuration:
Only one Server
Describe the bug:
I upgraded the cluster like described here: https://rancher.com/docs/k3s/latest/en/upgrades/basic/
Then stuff runs fine (i do not know, if the traefik job works but DNS works)
After restarting the host, i got some issues on the DSN.
When investigating further i also figured out, that the helm-install-traefik job stuck.
I put this together in one report, cause i think that could be connected somehow.
If not, i will split it up later on.
Steps To Reproduce:
The Installation of k3s was pretty straightforward only the simple curl -sfL https://get.k3s.io | sh - on one machine.
Additional context / logs:
CORE-DNS Logs:
[ERROR] plugin/errors: 2 auth.mydomain.org. A: read udp <IP_ADDRESS>:44628->192.168.XXX.YYY:53: i/o timeout
[ERROR] plugin/errors: 2 auth.mydomain.org. AAAA: read udp <IP_ADDRESS>:33794->192.168.XXX.ZZZ:53: i/o timeout
[ERROR] plugin/errors: 2 auth.mydomain.org. A: read udp <IP_ADDRESS>:48597->192.168.XXX.ZZZ:53: i/o timeout
[ERROR] plugin/errors: 2 auth.mydomain.org. AAAA: read udp <IP_ADDRESS>:36233->192.168.XXX.ZZZ:53: i/o timeout
[ERROR] plugin/errors: 2 auth.mydomain.org. A: read udp <IP_ADDRESS>:47208->192.168.XXX.ZZZ:53: i/o timeout
[ERROR] plugin/errors: 2 auth.mydomain.org. AAAA: read udp <IP_ADDRESS>:55953->192.168.XXX.ZZZ:53: i/o timeout
[ERROR] plugin/errors: 2 auth.mydomain.org. A: read udp <IP_ADDRESS>:58944->192.168.XXX.YYY:53: i/o timeout
[ERROR] plugin/errors: 2 auth.mydomain.org. AAAA: read udp <IP_ADDRESS>:57563->192.168.XXX.YYY:53: i/o timeout
[ERROR] plugin/errors: 2 auth.mydomain.org. A: read udp <IP_ADDRESS>:45613->192.168.XXX.YYY:53: i/o timeout
[ERROR] plugin/errors: 2 auth.mydomain.org. AAAA: read udp <IP_ADDRESS>:46321->192.168.XXX.ZZZ:53: i/o timeout

While 192.168.XXX.YYY and 192.168.XXX.ZZZ are the two internal domain controllers which are doing DNS.
The domain is also reachable from the NODE itself. so its not an connection issue.
and here are the helm-install-traefik LOGS:
if [[ ${KUBERNETES_SERVICE_HOST} =~ .*:.* ]]; then
 echo "KUBERNETES_SERVICE_HOST is using IPv6"
 CHART="${CHART//%\{KUBERNETES_API\}%/[${KUBERNETES_SERVICE_HOST}]:${KUBERNETES_SERVICE_PORT}}"
else
 CHART="${CHART//%\{KUBERNETES_API\}%/${KUBERNETES_SERVICE_HOST}:${KUBERNETES_SERVICE_PORT}}"
fi
set +v -x
+ [[ '' != \t\r\u\e ]]
+ export HELM_HOST=<IP_ADDRESS>:44134
+ HELM_HOST=<IP_ADDRESS>:44134
+ helm_v2 init --skip-refresh --client-only --stable-repo-url https://charts.helm.sh/stable/
+ tiller --listen=<IP_ADDRESS>:44134 --storage=secret
Creating /home/klipper-helm/.helm
Creating /home/klipper-helm/.helm/repository
Creating /home/klipper-helm/.helm/repository/cache
Creating /home/klipper-helm/.helm/repository/local
Creating /home/klipper-helm/.helm/plugins
Creating /home/klipper-helm/.helm/starters
Creating /home/klipper-helm/.helm/cache/archive
Creating /home/klipper-helm/.helm/repository/repositories.yaml
Adding stable repo with URL: https://charts.helm.sh/stable/
Adding local repo with URL: http://<IP_ADDRESS>:8879/charts
[main] 2022/08/28 09:16:53 Starting Tiller v2.17.0 (tls=false)
[main] 2022/08/28 09:16:53 GRPC listening on <IP_ADDRESS>:44134
[main] 2022/08/28 09:16:53 Probes listening on :44135
[main] 2022/08/28 09:16:53 Storage driver is Secret
[main] 2022/08/28 09:16:53 Max history per release is 0
$HELM_HOME has been configured at /home/klipper-helm/.helm.
Not installing Tiller due to 'client-only' flag having been set
++ jq -r '.Releases | length'
++ timeout -s KILL 30 helm_v2 ls --all '^traefik$' --output json
[storage] 2022/08/28 09:16:53 listing all releases with filter
+ V2_CHART_EXISTS=1
+ [[ 1 == \1 ]]
+ [[ '' == \t\r\u\e ]]
+ HELM=helm_v2
+ NAME_ARG=--name
+ JQ_CMD='"\(.Releases[0].AppVersion),\(.Releases[0].Status)"'
+ [[ -f /config/ca-file.pem ]]
+ [[ -n '' ]]
+ shopt -s nullglob
+ helm_content_decode
+ set -e
+ ENC_CHART_PATH=/chart/traefik.tgz.base64
+ CHART_PATH=/tmp/traefik.tgz
+ [[ ! -f /chart/traefik.tgz.base64 ]]
+ return
+ [[ install != \d\e\l\e\t\e ]]
+ helm_repo_init
+ grep -q -e 'https\?://'
+ echo 'chart path is a url, skipping repo update'
chart path is a url, skipping repo update
+ helm_v2 repo remove stable
"stable" has been removed from your repositories
+ return
+ helm_update install
+ [[ helm_v2 == \h\e\l\m\_\v\3 ]]
++ tr '[:upper:]' '[:lower:]'
++ jq -r '"\(.Releases[0].AppVersion),\(.Releases[0].Status)"'
++ helm_v2 ls --all '^traefik$' --output json
[storage] 2022/08/28 09:16:53 listing all releases with filter
+ LINE=1.7.19,deployed
+ IFS=,
+ read -r INSTALLED_VERSION STATUS _
+ VALUES=
+ for VALUES_FILE in /config/*.yaml
+ VALUES=' --values /config/values-01_HelmChart.yaml'
+ [[ install = \d\e\l\e\t\e ]]
+ [[ 1.7.19 =~ ^(|null)$ ]]
+ [[ deployed =~ ^(pending-install|pending-upgrade|pending-rollback)$ ]]
+ [[ deployed == \d\e\p\l\o\y\e\d ]]
+ echo 'Already installed traefik'
+ [[ helm_v2 == \h\e\l\m\_\v\3 ]]
Already installed traefik
+ helm_v2 mapkubeapis traefik --v2
Error: unknown command "mapkubeapis" for "helm"
Run 'helm --help' for usage.

Which installed k3s version did you try to upgrade?

i don't know, i think the last update was in april this year ..

It's hard to say exactly what's going on, but based on the fact that it appears to be upgrading from a very old Traefik release, it seems like you were quite a few versions back. You might try running the k3s-killall.sh script to terminate the running pods, then reboot the node. See if coredns and other pods come up into a better state after that.

Did not help, neighter did another update to the current version. Its still failing.

You might try starting k3s with --disable=traefik to remove the legacy traefik v1 helm chart. Once you see that the uninstallation job has completed, you can remove the flag and restart, and it should install the newer chart.


GET BY LOCATION AND FILTER BY TAG FOR SPECIFIC DATE AND TIME

Hello Steven and everyone else!
Is there a way I can do this? Like for example, get photos in Central Park NY City, #kiss, June 21, 2015, 3:00  to 5:00 pm.
Thanks, Alberto Zayas

The Instagram API doesn't all you to do this :( What you'd have to do is pull all posts from the specific area and then filter them on caption and create_time. Not doable with instafeed.js

OK, you are the only one that has responded so far, thanks Gregory. You can
take a look at my website here: hashtagdevents.com
ALBERTO
2015-07-05 0:01 GMT-04:00 Gregory Clarke<EMAIL_ADDRESS>
The Instagram API doesn't all you to do this :( What you'd have to do is
pull all posts from the specific area and then filter them on caption and
create_time. Not doable with instafeed.js
—
Reply to this email directly or view it on GitHub
https://github.com/stevenschobert/instafeed.js/issues/271#issuecomment-118577390
.


[BUG]  CC="ccache gcc" stopped working for C++ builds since setuptools-60.0.0

setuptools version
setuptools>=60.0.0
Python version
CPython 3.9
OS
Ubuntu 20.04 LTS
Additional environment information

Description
Running setup.py build_ext with CC set to "ccache gcc-11" causes this to happen when building a c++ extension module:
g++-11 gcc-11 -shared -Wl,-rpath=/opt/cpython-3.9-debug/lib -Wl,--disable-new-dtags -Wl,-rpath,/opt/sundials-5.7.0-noklu-nolapack-dbg/lib -L/opt/sundials-5.7.0-noklu-nolapack-dbg/lib -isystem /opt/sundials-5.7.0-noklu-nolapack-dbg/include build/temp.linux-x86_64-3.9-pydebug/pyodeint/_odeint.o -L/opt/cpython-3.9-debug/lib -o build/lib.linux-x86_64-3.9-pydebug/pyodeint/_odeint.cpython-39d-x86_64-linux-gnu.so
/usr/bin/ld: cannot find gcc-11: No such file or directory

Setting SETUPTOOLS_USE_DISTUTILS=stdlib yields the expected linkline:
g++-11 -pthread -shared -Wl,-rpath=/opt/cpython-3.9-debug/lib -Wl,--disable-new-dtags -Wl,-rpath,/opt/sundials-5.7.0-noklu-nolapack-dbg/lib -L/opt/sundials-5.7.0-noklu-nolapack-dbg/lib -isystem /opt/sundials-5.7
.0-noklu-nolapack-dbg/include build/temp.linux-x86_64-3.9-pydebug/pyodeint/_odeint.o -L/opt/cpython-3.9-debug/lib -o build/lib.linux-x86_64-3.9-pydebug/pyodeint/_odeint.cpython-39d-x86_64-linux-gnu.so
copying build/lib.linux-x86_64-3.9-pydebug/pyodeint/_odeint.cpython-39d-x86_64-linux-gnu.so -> pyodeint

The compile comands look alright though:
ccache gcc-11 -Wno-unused-result -Wsign-compare -DDYNAMIC_ANNOTATIONS_ENABLED=1 -g -Og -Wall -Og -g3 -Og -g3 -isystem /opt/sundials-5.7.0-noklu-nolapack-dbg/include -fPIC -DANYODE_NO_LAPACK=1 -DNDEBUG -DBOOST_UB LAS_TYPE_CHECK=0 -Ipyodeint/include -I/opt/cpython-3.9-debug/lib/python3.9/site-packages/numpy/core/include -Iexternal/anyode/include -I/opt/cpython-3.9-debug/include/python3.9d -c pyodeint/_odeint.cpp -o build/ temp.linux-x86_64-3.9-pydebug/pyodeint/_odeint.o -std=c++20
Perhaps the space between ccache and g++ is unaccounted for in setuptools (own?) distutils?
Expected behavior

How to Reproduce

Output

I can reproduce this with CC="ccache gcc" pip install --no-binary spead2 spead2==3.7.0, although it has a sizable compile time so one can probably find simpler packages to exhibit the same behaviour.

It's quite possible that something evolved in the distutils implementation after it was adopted from the stdlib. Setuptools' own copy of distutils has been around since v50 (IIRC), but was opt-in with SETUPTOOLS_USE_DISTUTILS=local. Maybe it would make sense to bisect those versions to see which version of setuptools introduced the change. But before we could do that, we'd need a good minimal reproducer.
I also notice that 'numpy' is implicated here. It's a known issue that numpy patches distutils (stdlib only) (#2372), so it could be the expected behavior only works with those patches.
Can you determine if the issue is unique to numpy or if simple extensions are affected as well?

@jaraco , thanks, I created a reproducer:
https://github.com/bjodah/reproducer-setuptools-ccache-gh3130
My findings are as follows:
When setting SETUPTOOLS_USE_DISTUTILS=local, then:

Using (latest) NumPy (1.22.2):

58.0.4 is OK
58.1.0 is bad

Using no numpy:

58.0.4 is OK
58.1.0 is bad

Hope this is of any help.

That's a huge help. You've narrowed the problem to these changes. From there, I was quickly able to find f182b5e2f6e492cd13eed98d7f248d4146456d8c, which is almost certainly implicated in the regression. I'll transfer this issue to distutils.

That commit was added in pypa/distutils#54.

@isuruf Would you be willing to work out a solution that captures this regression and restores the prior expectation?

Sure. I'll look into it

See https://github.com/pypa/distutils/pull/117

It's not pypa/distutils#54, it's pypa/distutils#53.

Thanks for the PR @isuruf. I looked at the PR and these two previous PRs, and it's not at all obvious to me why #53 was the cause of the reported issue. Can you describe how the changes in #53 led to the failure?


update ibmcloud cis webhook link

Old ibmcloud cis webhook is out of support, update to new one.
Signed-off-by: hzhihui<EMAIL_ADDRESS>

Thanks @hughhuangzh
I looked at https://github.com/IBM/cert-manager-webhook-ibmcis  repo and noticed a few issues:

It seems to be a copy of https://github.com/jb-dk/cert-manager-webhook-ibmcis/ without maintaining any of the original commit history. Wouldn't it be better to fork the original repo and maintain the commits and to make it clear that the new repo is derived from another.

I don't see any copyright headers in the main.go file giving credit to @jb-dk for the original work.

The new IBM repo README file still directs users to install https://raw.githubusercontent.com/jb-dk/cert-manager-webhook-ibmcis/master/cert-manager-webhook-ibmcis.yaml

If (as you say) the old repo is "out of support" I think you should add a note to the new repo explaining how it is supported and which versions of cert-manager and IBM Kubernetes it has been tested with.

Please answer or address those points and then I'll be happy to merge this.

xref

https://github.com/IBM/cert-manager-webhook-ibmcis/issues/1

@wallrj  Appreciate your feedback.  your comments were addressed, please review if new feedback.

It seems to be a copy of https://github.com/jb-dk/cert-manager-webhook-ibmcis/ without maintaining any of the original commit history. Wouldn't it be better to fork the original repo and maintain the commits and to make it clear that the new repo is derived from another.
A: the background is that I'm not the owner of repo https://github.com/jb-dk/cert-manager-webhook-ibmcis/, so copy the code with new pr,  I have addressed your comments which add commits history, please check.

I don't see any copyright headers in the main.go file giving credit to @jb-dk for the original work.
A:  addressed

The new IBM repo README file still directs users to install https://raw.githubusercontent.com/jb-dk/cert-manager-webhook-ibmcis/master/cert-manager-webhook-ibmcis.yaml
A:  addressed

If (as you say) the old repo is "out of support" I think you should add a note to the new repo explaining how it is supported and which versions of cert-manager and IBM Kubernetes it has been tested with.
A: addressed

/assign @wallrj

Sorry I forgot to respond to your updates.
/reopen

@hughhuangzh Looks like something went wrong with the rebase. There are 0 changes now. Please try again and pester me for a review. I'd like to get this merged.

@wallrj NW, it looks the doc link is outage. No need to update any more.


Update azure-stack-provision-vm.md

Fixed minor formatting issue.

Hi @alexandair, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

@alexandair Thanks for the update. @tysonn Please merge.

@byronr @tysonn After 25 days this is still not merged. :(


Doc: parent config has more features than balancer plugin

The balancer plugin has consistent hashing, and round robin
(not weighted). As a remap plugin, it is limited to url attribnutes.
It neither has passive nor active health checks.
Really, Balancer plugin should be deprecated.

[approve ci]

[approve ci docs]


Add logging utility functions

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

Project coverage is 100.00%. Comparing base (803f119) to head (c14cbbf).

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #7   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         4    +1
  Lines           28        42   +14
  Branches         1         3    +2
=========================================
+ Hits            28        42   +14

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Cannot execute the test when it is redirected to another application with auth token in the URL

What is your Scenario?
Log in to the application with valid credentials.
Click on the Quik Update icon on the left and the menu (side bar)
What is the Current behavior?
A white screen is loaded, and the user is not directed to the application.
Following is shown in the URL
http://<IP_ADDRESS>:65486/IcH97oB0G*czEtTUt9m/about:error
What is the Expected behavior?
Test should be directed to the quick update page
What is your public website URL? (or attach your complete example)
"base_url" : "https://test.cammsrisk.com.au/CammsRisk"
"OrganizationName": "Auto_Risk_Incident"
"UserName": "jsmith"
"Password": "3l@st!c5ity6"
What is your TestCafe test code?
import { Selector } from "testcafe";
fixture('TestCafe SOF Quation -1 ').page("https://test.cammsrisk.com.au/CammsRisk")
test("Quick demo of the issue",async (t) => {
console.log("Starting the test")
await t
    .typeText(Selector('#LoginControlSAAS_txtOrganisation'), "Auto_Risk_Incident")
    .typeText(Selector('#LoginControlSAAS_txtUsername'), "jsmith")
    .typeText(Selector('#LoginControlSAAS_txtPassword'), "3l@st!c5ity6")
    .click(Selector('#LoginControlSAAS_btnLogin'))
    .click(Selector("#ctl00_WelcomeControlSAAS_aMyQuickUpdate"))
   .expect(Selector("#mainContainer>span").textContent).contains("My Quick Update")
    .debug()

})
Your complete configuration file
module.exports=
{
"browsers": [
"chrome:headless"
],
"concurrency": 1,
"developmentMode": false,
"screenshots": {
"takeOnFails": true,
"path":"artifacts/screenshots",
"fullPage": true
},
"selectorTimeout": 120001,
"skipJsErrors": true,
"baseUrl": "https://test.cammsrisk.com.au/CammsRisk",
"userVariables": {
"base_url" : "https://test.cammsrisk.com.au/CammsRisk",
"OrganizationName": "Auto_Risk_Incident",
"UserName": "jsmith",
"Password": "3l@st!c5ity6",
"serverId": "",
"serverDomain": "",
"mailosaurApiKey": ""
},
"assertionTimeout": 120000
}
Your complete test report
he "browsers" option from the configuration file will be ignored.
Running tests in:

Chrome <IP_ADDRESS> / Monterey 12

TestCafe SOF Quation -1
Starting the test
✖ Quick demo of the issue

Failed to load the page at "https://test.cammsrisk.com.au/CammsRisk/Authorised/Home/MyUpdateHome.aspx".
Increase the value of the "pageRequestTimeout" variable, enable the "retryTestPages" option, or use quarantine mode to
perform additional attempts to execute this test.
You can find troubleshooting information for this issue at "https://go.devexpress.com/TestCafe_FAQ_ARequestHasFailed.aspx".
Error details:
Failed to complete a request to "https://test.cammsrisk.com.au/CammsRisk/Authorised/Home/MyUpdateHome.aspx" within the
timeout period. The problem may be related to local machine's network or firewall settings, server outage, or network
problems that make the server inaccessible.
Browser: Chrome <IP_ADDRESS> / macOS 10.15.7

1/1 failed (1m 15s)
Warnings (1):
The "browsers" option from the configuration file will be ignored.
Screenshots
No response
Steps to Reproduce

TestCafe version
2.3.0
Node.js version
v18.13.0
Command-line arguments
testcafe chrome testSuite/tests/stackoverflow-q.ts
Browser name(s) and version(s)
No response
Platform(s) and version(s)
macOS 12.6
Other
No response

Apparently the issue is due to local configuration

@Aleksey28 Please let me know if you need further information

Hi,
I've reproduced the issue with the latest TestCafe version. However, I also tried to run this test with proxyless mode enabled and it passed without errors. Could you try enabling proxyless mode on your side and check if the test passes correctly?

The issue was resolved with proxyless mode . Thanks @Artem-Babich


vault-addr annotation being ignored.

Describe the bug:
I'm trying to create a deployment with the annotations below, but the VAULT_ADDR variable is not being set, so vault-env is running with the default address and fails to obtain the token with the error below.
It's happening with many deployments, not just one.
 vault.security.banzaicloud.io/vault-addr: "https://vault.default:8200"
 vault.security.banzaicloud.io/vault-role: omaticcloudrecordroutingservice
 vault.security.banzaicloud.io/vault-tls-secret: vault-tls
 vault.security.banzaicloud.io/vault-env-passthrough: "VAULT_ADDR, VAULT_ROLE"

omaticcloudrecordroutingservice time="2023-02-11T02:40:25Z" level=error msg="failed to request new Vault token" app=vault-env err="Put \"https://vault:8200/v1/auth/kubernetes/login\": dial tcp: lookup vault on <IP_ADDRESS>:53: no such host"
omaticcloudrecordroutingservice time="2023-02-11T02:40:29Z" level=error msg="failed to request new Vault token" app=vault-env err="Put \"https://vault:8200/v1/auth/kubernetes/login\": dial tcp: lookup vault on <IP_ADDRESS>:53: no such host"
omaticcloudrecordroutingservice time="2023-02-11T02:40:31Z" level=fatal msg="failed to create vault clienttimeout [10s] during waiting for Vault token" app=vault-env
Stream closed EOF for omatic/omaticcloudrecordroutingservice-5567cb9d79-8pnv2 (copy-vault-env)
Stream closed EOF for omatic/omaticcloudrecordroutingservice-5567cb9d79-8pnv2 (omaticcloudrecordroutingservice)

Expected behaviour:
It was expected that the value set by the annotation created an environment variable VAULT_ADDR withm the value defined previously so that vault-env is able to request a new token. The variable VAULT_ADDR exists, but with the default value, not the one I set.
Steps to reproduce the bug:
Steps to reproduce the bug should be clear and easily reproducible to help people
gain an understanding of the problem.
Environment details:

Kubernetes version (e.g. v1.23.12):
Cloud-provider/provisioner: AKS
bank-vaults version: 1.18.0
Install method: helm
Logs from the misbehaving component (and any other relevant logs):
-- Secrets webhook show nothing about the deployment
Resource definition: https://gist.github.com/guirgouveia/00f07041e70b00de86ad8e7c1c6387aa
/kind bug

Hi @guirgouveia! In the annotations of your example you used the vault-addr with the default value, that is why it was used by vault-env. Basically the annotations here are for configuring envs for vault-env, and the vault-env-passthrough annotation is used when you want an env var to be passed to the other containers in the pod by vault-env.
Also looking at the example on the link, these annotations should go under spec/template/metadata, like in this example.
I hope this helps you figuring out the correct configuration, please drop a feedback whether it worked or not!


Fixes #288. And added some additional Javadoc to other methods.

If this gets merged I would cherry-pick the commit into the asciidoctorj-1.6.0 branch.

Hi Robert there is an issue about converting javadoc to asciidoc and rendered with asciidoclet. If you want I accept the PR and we will change in future. WDYT?

Oh, I was not aware of that.
I'd like to fix that beforehand.
How can I reproduce that?

well the idea is add the asciidoclet plugin on pom.xml and write javadoc with asciidoc format.

the idea is add the asciidoclet plugin on pom.xml and write javadoc with asciidoc format.

And we should depend on a stable version of Asciidoclet. It's okay of Asciidoclet is behind the version of AsciidoctorJ. We want to treat it like a third-party library just like any other project would do. The fact that Asciidoclet uses AsciidoctorJ under the covers should be inconsequential.
:+1: to moving to Asciidoclet. Eat your own dog food :)

Of course, we can proceed with this update and switch to Asciidoclet in a seperate issue. See #35.

@lordofthejars I probably did not understand you correctly, I thought that there already was a pom.xml that uses asciidoclet.
But I think it is planned in the future to use asciidoclet, isn't it?
Because I could not find a pom.xml.
If this was your intention I would prefer doing this with a separate issue and first committing this, because probably other classes might also need changes when transitioning to asciidoclet. (For example GlobDirectoryWalker where I added some HTML javadoc just recently ;-) )

Because I could not find a pom.xml.

I think he meant build.gradle :)

BTW when adding missing javadoc I stumbled over this method:
public void setSectNumLevels(int sectNumLevels)
I think I can guess what it means (The number of sections that are numbered like 2 to give numbers to level 0 and 1 sections) but I could not find anything that confirms my guesses in the user manual.

Sorry yes I mean gradle file. Yes I think the best way is to accept the PR
and change in future. Sorry :)
El dt., 5 de maig, 2015 a les 22.29 Robert Panzer<EMAIL_ADDRESS>va escriure:

BTW when adding missing javadoc I stumbled over this method:
public void setSectNumLevels(int sectNumLevels)
I think I can guess what it means (The number of sections that are
numbered like 2 to give numbers to level 0 and 1 sections) but I could not
find anything that confirms my guesses in the user manual.
—
Reply to this email directly or view it on GitHub
https://github.com/asciidoctor/asciidoctorj/pull/295#issuecomment-99210912
.

I think I can guess what it means (The number of sections that are numbered

Yes, the depth to which the numbering goes. I've opened an issue to document this in the user manual. See https://github.com/asciidoctor/asciidoctor.org/issues/383. Thanks for the heads up!

Claimed (can't assign to me).


feat: arbitrum eips diffs

Related to #37.
Requires #59 to be merged (and to rebase this PR).
Add Arbitrum EIPs diffs. Just like Optimism diffs (#53), it adds diffs for EIP-1599 and EIP-4399 (PREVRANDAO).
@mds1 Not sure about EIP-4895 (withdrawals lists)?
Also fix a url formatting issue.

Hey @gzeoneth, could you help us with some Arbitrum EIP information? Context is listing which mainnet EIPs are implemented, and what parameter differences there may be from mainnet. We're only listing execution related EIPs, no consensus, networking, etc. EIPs.
For example, here's what it looks like for Optimism‚the same EIPs are implemented, but with a few differences explained in the "Notes" section of each row: https://www.evmdiff.com/diff?base=1&target=10#eips
Hoping to get something similar for Arbitrum. I assume all EIPs are implemented, with the following exceptions:

No EIP-1559: A similar approach is used. Are there parameters we should use reference here? Any references you might have showing how it's equivalent would also be great.
EIP-4399: Just returns the constant 1
EIP-4895: Does arbitrum have an empty withdrawal list similar to Optimism?

Thanks!

Hey @gzeoneth, could you help us with some Arbitrum EIP information? Context is listing which mainnet EIPs are implemented, and what parameter differences there may be from mainnet. We're only listing execution related EIPs, no consensus, networking, etc. EIPs.
For example, here's what it looks like for Optimism‚the same EIPs are implemented, but with a few differences explained in the "Notes" section of each row: https://www.evmdiff.com/diff?base=1&target=10#eips
Hoping to get something similar for Arbitrum. I assume all EIPs are implemented, with the following exceptions:

No EIP-1559: A similar approach is used. Are there parameters we should use reference here? Any references you might have showing how it's equivalent would also be great.
EIP-4399: Just returns the constant 1
EIP-4895: Does arbitrum have an empty withdrawal list similar to Optimism?

Thanks!

EIP-1559: partially implemented with the following exception

target a gas per second ("speed limit") instead of block gas limit to account for the variable block time in Arbitrum
priority fee is not collected by the network even when nonzero value is specified

EIP-4399: Always returns 1
EIP-4895: Arbitrum currently do not generate block with withdrawalsRoot

Let me also circulate to my team to see if anything is missing

Let me also circulate to my team to see if anything is missing

Hey @gzeoneth just following up here to see if there was anything missing here?

Let me also circulate to my team to see if anything is missing

Hey @gzeoneth just following up here to see if there was anything missing here?

LGTM tho fyi ArbOS 20 would be shipped around 10 days from now


feat: Implement DAP protocol in Nargo

Description
Implements the DAP protocol to allow a tool such as VS.Code to drive the debugger in a Noir binary project. A separate PR for the VS.Code Noir extension will be submitted later.
Problem
Part of #3015
The beginning of this implementation was heavily inspired by @dmvict implementation of an alternative debugger for Noir programs.
Summary
This PR implements a new nargo subcommand dap. This starts a DAP server in single session mode through stdin/stdout and waits for a launch request. Using the arguments in the launch request (projectFolder, and optionally package and proverName) it compiles the Noir binary package and starts it in debug mode. Through DAP requests, a tool can then step through the program, set breakpoints and list the generated opcodes using a disassemble request.
Additional Context
Documentation
Check one:

[ ] No documentation needed.
[ ] Documentation included in this PR.
[X] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist*

[X] I have tested the changes locally.
[X] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Argh, sorry looks like there are some merge conflicts that have arisen.

Argh, sorry looks like there are some merge conflicts that have arisen.

No worries, I'll update to latest master.


Allow signing to be disabled for test runs of release pipeline

Sometimes it's necessary to run the release pipeline to test a specific change as part of development. By default, the pipeline will execute the signing logic which takes an obscene amount of time. This wastes both time and machine resources just to test a change. We should provide an option for signing to be disabled. This should only apply to the release pipeline intended for testing.

PR - https://dev.azure.com/dnceng/internal/_git/dotnet-release/pullrequest/38739


Refactor Sync onUnknownBlockRoot

The current approach uses a while and maybe prone to infinite loops as reported by @wemeetagain.
https://github.com/ChainSafe/lodestar/blob/096ea27a9dd12b1db5fded62b0cc9688780e86c4/packages/lodestar/src/sync/sync.ts#L171-L174

oh yeah, because someone changed peerBalalncer to loop around. I changed that in regular sync v2.
So we have two options, either reset RoundRobinArray to return null if all peers are discarded or reduce maxAttempts here to this.getPeers().length x 2 (so we try each peer twice).
if you have 10 peers and block doesn't exists this will surely make you banned from those 10 peers
Originally posted by @mpetrunic in https://github.com/ChainSafe/lodestar/pull/1378

unstale
I think it's resolves with #1655 cc @tuyennhv

can this issue be closed?

Yeah


LightningCSS does not work in dev mode (HMR)

Stackblitz (https://stackblitz.com/edit/remix-run-remix-3gnvyb?file=app%2Froutes%2F_index.tsx)
Post-processing with LightningCSS does not work in dev mode on HMR updates. The production build works as expected.
const styles = stylex.create({
  x: {
     color: 'lch(from pink calc(l + 10%) c h)',
  },
});

css output:
color: lch(from pink calc(l + 10%) c h);

should be:
color: lch(from pink calc(l + 10%) c h);
color: lab(93.7873% 24.4367 3.75943);

Notice the defined CSS in the regular imported stylesheet (in our case the main import /styles/index.css), the post-processing is working properly.

I might think is a bug for lightningCSS. According lch I think expression calc(1 + 10%) not correct syntax. But lightningCSS can make it work well. These are all digressions.
It's a bug for the plugin. I have been fixed this issue in my local. But i need to ensure all of scene can be passed.

All related issues will be work at fix


Add an option to hide/filter out full games

Implementing this as "sort full games to the end of the game list".


results.csv have wrong Storage type and environment

I ran my tests against EFS storage on a ROSA cluster but the results.csv show and Env  = IBM Cloud and Storage type = ocs.
result.csv

The params.yml contain these labels here:
cluster_infrastructure: ibmcloud # optional labels eg ibmcloud, aws, azure, vmware
cluster_name: storage-performance-cluster # optional labels
storage_type: ocs # optional label eg portworx, ocs, <storage vendor>

You need to edit these if this is important to you.


Unnamed block statements not supported

Description
Unnamed block statements are supported in Yul, e.g.
let zero := 0
let v := calldataload(zero)
{
     let y := add(sload(v), 1)
     v := y
}
sstore(v, zero)

But in Yul+, they do not work.
Steps to reproduce
object "SimpleStore" {
  code {
    datacopy(0, dataoffset("Runtime"), datasize("Runtime"))
    return(0, datasize("Runtime"))
  }
  object "Runtime" {
    code {
      calldatacopy(0, 0, 36) // write calldata to memory

      mstruct Calldata(
        sig: 4,
        val: 32
      )

      switch Calldata.sig(0) // select signature from memory

      case sig"function store(uint256 val)" { // new signature method
        sstore(0, Calldata.val(0)) // sstore calldata value

        {
            let i := 1
        }
      }

      case sig"function get() returns (uint256)" {
        mstore(100, sload(0))
        return (100, 32)
      }
    }
  }
}

Versions
yulp 0.0.6

Yeah, I didn’t support them due to parsing setup.
Perhaps we can find a way! I suppose it’s just a block, so possibly we can just open the block up.
I’ll think on it, let me know if you can find a fix!
Sent from my iPhone

On Apr 15, 2020, at 11:35 AM, Loredana Cirstea<EMAIL_ADDRESS>wrote:
Description
Unnamed block statements are supported in Yul, e.g.
let zero := 0
let v := calldataload(zero)
{
let y := add(sload(v), 1)
v := y
}
sstore(v, zero)
But in Yul+, they do not work.
Steps to reproduce
object "SimpleStore" {
code {
datacopy(0, dataoffset("Runtime"), datasize("Runtime"))
return(0, datasize("Runtime"))
}
object "Runtime" {
code {
calldatacopy(0, 0, 36) // write calldata to memory
  mstruct Calldata(
    sig: 4,
    val: 32
  )

  switch Calldata.sig(0) // select signature from memory

  case sig"function store(uint256 val)" { // new signature method
    sstore(0, Calldata.val(0)) // sstore calldata value

    {
        let i := 1
    }
  }

  case sig"function get() returns (uint256)" {
    mstore(100, sload(0))
    return (100, 32)
  }
}

}
}
Versions
yulp 0.0.6
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Switch to C++ 17

I don’t know whether C++ 14 was chosen for a particular reason. If not, can we switch to C++ 17?
It has a couple of nice additions that I have refrained from using because we are still at C++ 14:

Structured binding: https://en.cppreference.com/w/cpp/language/structured_binding
string_view: https://en.cppreference.com/w/cpp/string/basic_string_view
std::filesystem: https://en.cppreference.com/w/cpp/filesystem
std::clamp https://en.cppreference.com/w/cpp/algorithm/clamp

Summary of C++ 17 changes: https://en.cppreference.com/w/cpp/17 That page also lists compiler support.

I am okay with switching to C++17. The only reason keeping to C++14 was, from what I remember, that I had an easier time getting strobealign to compile on my mac (given the versions I had of gcc and clang at the time on my mac). With the new Cmake installation it is not an issue anymore.

Great, thanks! I think that will make a couple of things a bit nicer.


Full path of the downloaded file is not $env:DOWNLOADSECUREFILE_SECUREFILEPATH

Based on my experience, the full path of the downloaded file is not $env:DOWNLOADSECUREFILE_SECUREFILEPATH as documented. If it's in a task group, then i can't even control it as an output variable. It generates some kind of the ID and concatenated with "securefilepath". If it is used in a release pipeline directly, you can specify an output variable so it can be referenced as a more user friendly name such as myCertFilePath.SecureFilePath

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

ID: 2a5f8376-3654-0ed4-d9f8-d15f28350783
Version Independent ID: f393daba-6515-6c26-71ac-b557e8c8dad1
Content: Download Secure File task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/download-secure-file.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

This path worked for me in the task inside Task Group. I also placed Download Secure File task in the same task group.
$(Agent.TempDirectory)\file-name-used-when-adding-secure-file-to-the-library

Here's a nice little hack to get the filepath from yaml:
steps:

script: echo $(Agent.TempDirectory)


CLOUDSTACK-8783: specify ciphersuite SSL_DH_anon_WITH_3DES_EDE_CBC_SHA in MockServerTest.java

The ciphersuite could be different on os.
Sometimes the MockServerTest fails due to the ciphersuite does not work (for instance misconfiguration).
SSL_DH_anon_WITH_3DES_EDE_CBC_SHA has 168-bit encryption and anonymous auth, which is suitable for SSL testing.

LGTM

:+1: merging


[Snyk] Security upgrade @angular/cli from 1.0.0 to 1.5.3

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

infrastructure/oss/testdata/package.json
infrastructure/oss/testdata/package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

704/1000  Why? Has a fix available, CVSS 9.8
Prototype Pollution SNYK-JS-HANDLEBARS-534988
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Prototype Pollution

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


feat: Uploading/Attaching files to channel messages

About
This pull request is about adding the ability to upload & attach files to channel messages.
This include a new File class and related modifications to send/create_message functions.
Checklist

[x] I've ran pre-commit to format and lint the change(s) made.
[x] I've checked to make sure the change(s) work on 3.8.6 and higher.
[x] This fixes/solves an Issue.

(If existent): https://github.com/interactions-py/library/issues/636

[x] I've made this pull request for/as: (check all that apply)

[ ] Documentation
[ ] Breaking change
[x] New feature/enhancement
[ ] Bugfix

I'll be moving forward with merging this. Thank you for contributing to interactions.py, and welcome to the group! 😄


less: apply patch for security issue when opening files with \n in paths

Description of changes
https://www.openwall.com/lists/oss-security/2024/04/12/5

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

Less than 500 rebuilds? I thought it was bigger than that, moving back to master.


Improve Query dashboards

This PR fixes the query dashboards by using correct metrics.
As @bwplotka pointed it out in another PR's discussion, it now uses http_requests_total and http_request_duration_seconds_bucket.

Awesome that we have these metrics. :ok_hand:


[question] How is address of node sufficient for communication

Hi,
OrbitDB looks really great, my team is currently evaluating its use in a medical environment. After reading the docs + issues, I still don't know how a node's address (orbit-db-protocol/hash/db-name) is sufficient to be used as address for a remote db, even more as communication isn't required to be in the same local network (which is insanely great, btw). If a kind of middleware/broker is used, it would be really great to know more about it - it's the only detail I couldn't find in the docs.
Thanks in advance, and keep up the great work!
Tom

It depends on your IPFS setup — The IPFS network is used to store the data and thus content-routing and communication relies on how you setup and use IPFS. For instance, DHT content-routing, a big part of connectivity magic, is not fully operational in the JS implementation but working in the Go implementation. Thus, in the JS implementation its helpful to configure a shared signalling server so that your nodes can find each other.
Brief overview of OrbitDBAddress as it relates to IPFS:
The magic and crux of an OrbitDBAddress is the hash part, which is an IPFS Multihash. The hash is used to retrieve a database manifest, and subsequently an access controller. Both of which are objects stored on IPFS which is a content-addressable storage system. The hash in the OrbitDBAddress points to the database manifest and inside the database manifest is another hash that points to the Access Controller.
To replicate the entries in the database, the OrbitDBAddress is used as a topic on IPFS PubSub to exchange heads which can be used to retrieve the rest of the entries as each entry links to the previous one, see IPFS-Log.

Thanks, that's one hell of an answer 😁, exactly what I was looking for. I seriously recommend copy/pasting your explanation as a paragraph into the docs.
Again, thx for the great support!

@CaptainQuark Thanks for the kind words!
@mistakia Want to PR the answer into the docs? If not, I (or @CaptainQuark, maybe) would be happy to.

@RichardLitt go for it! as I'm unsure of the best course of action but I'm more than happy to help out along the way.
This could fit into the GUIDE but I think perhaps we should start a separate FAQs doc to aggregate many of the general & helpful questions answered in the issues.


feat: Publish to edge/branch-name

Description
Adds a job that, on push, will publish to the 1.3/edge/branch-name track.
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
N/A I have made corresponding changes to the documentation
[x] I have added tests that validate the behaviour of the software
[x] I validated that new and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
N/A I have bumped the version of the library

Need to wait for integration tests before this becomes valid


[dotnet] Fix regex to ignore RC releases

https://dotnet.microsoft.com/en-us/platform/support/policy/dotnet-core
added a new table with EOL dates for some RC releases, which changed the EOL date of 9 to EOL(9RC1) < releaseDate (9),
which causes some validation issues.

Tested this locally, auto-merging since this has caused build to break.


external integration

I'm trying to make a profile for Savant that sends the power on and off command to the plugin. But I can't get an answer. Could you give me some light?

What you want to know?


Compatibility with Jmix 1.3

Please build a version on Jmix 1.3.0 and make a PR to include it in Jmix BOM.

Hi Konstantin
I see in Jmix documentation that
Java 11 Is Required
Projects on Jmix 1.3 require JDK 11 or 17.
Java 8 is not supported anymore.
So if I compile the addon with Java 17, is ok?

Hi Panos,
I'm afraid if you compile the add-on with Java 17 then people with Java 11 will not be able to use it.
So add-ons should be built with the minimal Java version supported by the framework, which is now Java 11.

I have publish version 1.3.1 and made the PR


Potential bottleneck

We use util.concat to join fed pieces, but that actually might be redundant action. We can save the sequence and read straight from it. Basically that is BufferList but for audio-buffers.

Use audio-buffer-list to store and feed items

Done


Gitub task to format code

Motivation
Formatting a PR made a huge diff with main because main is not formatted.
Changes

Formatted with: npm run format and shfmt.
Added a github action to format the code, so that main hopefully stays formatted.

Tests
None.  Formatting SHOULD not change the behaviour of the code.

I think we should probably exclude the candid directory since those files are autogenerated by didc.

Actually, ignoring file matching "*.idl.js" is probably better.

Haha.  I just checked compiling idl.js from did and it fails because the did contains an invalid comment.

@hpeebles  Ok, preparing the fix to that comment in another PR.  However, back to this question, should not all files be formatted in the same way, regardless of how they were generated?  I am not too keen on exceptions unless they are really needed.  Exceptions yield surprises.
I don't mind much as formatting SHOULD not change the meaning of the code so SHOULD NOT cause any nasty surprises, however just to make sure that we really want this I will recite an argument used elsewhere:

There is the mechanical view, where any consistent algorithm is needed to factor out changes that do not affect the meaning of the code, and there is the human perspective where humans want to be able to read any file in a consistent way.

Personally as a human reader I really don't care if formatting is slightly different in different parts of the code but there are people whose eyes start to bleed, or who have their editor set up to automatically format any file they open, so if they look at the autogenerated files they might end up formatting them anyway.
If you really want the exception I'll add it.

@hpeebles  It checked and we do list *.did.js in .prettierignore here and elsewhere.  I am not sure why we use idl.js instead of did.js.  But it looks as if the intention is that we don't format this code so I will run with that.


added & tweaked various services

This PR does the following:

Makes !github actually search
Adds !gh alias for GitHub
Adds !ddg alias for DuckDuckGo
Adds !tmdbfor TheMovieDB
Adds !nebula for Nebula
Adds !thalia for Thalia (biggest german book store)
Adds !sakutafor Sakuta (one of my own projects; I can remove it again if the site isn't of enough relevance)
Adds !animenewsnetwork & !ann for AnimeNewsNetwork
Adds !anidb for AniDB
Adds !gmail for Gmail

Would it be alright to remove the changes for !github as that's one I use quite a lot! E.g. !github alii/website :)


Add [Coderabbit] PR Stats service and tests

Implemented a new service CoderabbitStats to fetch and display pull request statistics from the CodeRabbit API.
Created a corresponding tester file to validate the service's functionality, including tests for valid repositories, repository not found, and server errors.
The service returns a badge with the number of PRs and appropriate error messages based on the API response.

This addition enhances the analysis capabilities of the application by integrating CodeRabbit statistics.
GitHub Issue - https://github.com/badges/shields/issues/10748

@jNullj I have made relevant changes as per your request, requesting you to see if it fits the guidelines.

@chris48s I'm contributing a badge for the first time to shields.io.
Sorry for the trouble with obvious mistakes. I've read the contributor.md and tutorial.md, but I still missed a lot. Please re-review and let me know if things are out of place.


[Question] How to catch Response event for iframe?

I know we can catch response event with page.onResponse(). Is there any way to catch response events for iframes?

Which Playwright version is this? Can you also run with DEBUG=pw:protocol and upload the log?

It's unclear why you don't see the initial requests to the server but you may be missing image response because you use Thread.sleep(). Replace it with page.waitForTimeout(100000); to see the responses that come after the click. This is a known limitation of the synchronous API in Java version of Playwright: messages from the driver are only dispatched when there is an active call of one of the Playwright API methods.

@yury-s You are genius. Thread.sleep was the problem.
Thank you for your help.


[Views] render cornerAnnotation + annotated cube button

This Pull Request is based on https://github.com/medInria/medInria-public/pull/539
I'm continuing to solve the min/max intensities slowness for the 1st layer of multiples ones. Doing it, i updated a bit the code.
Changes:

Add Annotated Cube button in order to deactivate the cube in view, like the other parameters in view the view navigator.
Add 2 layers, set the 1st layer as current one: the patient name, study and series are not updated. Fixes https://github.com/medInria/medInria-public/issues/545
Remove useless code to continue to improve speed when the user opens an image (as done in PR 539)

:m:

If needed, i will do a PR on 3.1.1 after the approval of this one

Pull request rebased

< this pull request is dead, merge problems...
I'm closing it. All my changes are in this PR: https://github.com/medInria/medInria-public/pull/549


chore: make the controller response be easy

What's changed?

Below is the test result

Checklist

[x]  I have read the Contributing Guide
[x]  I have written the necessary doc or comment.
[x]  I have added the necessary unit tests and all cases have passed.

Add or update API

[ ] I have added the necessary e2e tests and all cases have passed.

ptal @tomsun28 @zqr10159

How about use ResponseBodyAdvice to handle response entity globally? @tomsun28 @LinuxSuRen

I'm not sure if all controllers could have the same or similar way to handle it.

+1 the response stru may not always same


Allow installation without API keys

Please tell us your new feature idea
Allow plugin installation without API keys, currently if no API is provided the installation fails with an error.
Why do you need it?
Not all installation is manual. For example, when adding cordova platform this plugin fails to install and prevents all other plugins in the project from installing as well.
How helps other people?
Allows plugin installation to be automated by cordova and prevents hard to track errors when automated installation fails.

nop.

Why? The current behavior is poor and does not work with cordova.

Cordova provides only one way to define a node in AndroidManifest.xml is to define plugin.xml.
There is no other solution.

https://github.com/mapsplugin/cordova-plugin-googlemaps/blob/v2.4.6/plugin.xml#L236-L240
If you find alternative way (but not using cordova hook script, because cloud builds do not support it), please send a pull request.

It is possible. The same edit-config and config-file nodes available in plugin.xml also work in config.xml

👍


Observe changes with query parameter

For this moment TableObservable#withQuery and ContentTableObservable reads all the table when it changes. It would be more flexible to add Where as a parameter for observing changes in tables.
It would be also great to observe changes for the single object.

Observing changes for the single object is rather large feature and it will be only in 0.2.0 version (or may be 0.1.3)

Fixed in #8 for query parameters. Observing changes for certain row in tables requires research and it'll be separate issue and out of the scope of 0.1.2 release.


Provide key completions for TypedDict dictionaries

E.g., for the following code:
import typing

class Spam(typing.TypedDict):
    attr: str
    other_attr: int

def func(arg: Spam) -> None:
    print(arg[])

I would expect to be able to get a completion from within arg[] as the possible set of keys are known.

I think this may be a dupe of #654.

I think this may be a dupe of #654.

Not entirely, as that's for building the dict; this is for getting values from it. I'm surprised we don't do this, but I guess this is a string literal completion in [] which isn't plain . member access.

Not entirely, as that's for building the dict; this is for getting values from it. I'm surprised we don't do this, but I guess this is a string literal completion in [] which isn't plain . member access.

Yep, it's completing to a string in an index context and subtly different from #654 for the reasons you listed.

Yep, it's completing to a string in an index context and subtly different from #654 for the reasons you listed.

This feature is already implemented, but the usability is not great currently. The problem is that the suggestion is provided only when you type the first quote character, but the client doesn't automatically trigger a suggestion request at that point. If you place the cursor between the [ and the ] and type a quote, then press control-space to force the completion suggestions to be displayed, you'll actually see the available TypedDict attributes appear in the completion list.
Any suggestions for how to improve the usability here?

This feature is already implemented, but the usability is not great currently. The problem is that the suggestion is provided only when you type the first quote character, but the client doesn't automatically trigger a suggestion request at that point. If you place the cursor between the [ and the ] and type a quote, then press control-space to force the completion suggestions to be displayed, you'll actually see the available TypedDict attributes appear in the completion list.
Any suggestions for how to improve the usability here?

We do declare that [ is a character that can trigger completion: https://github.com/microsoft/pyright/blob/86221745ca1e9122e4953ed2c63515b4de374bbf/packages/pyright-internal/src/languageServerBase.ts#L960
I think what would be an improvement would be at that point to quite literally offer up the string literals as full completions, rather requiring that a user types a quote. For example, TS does something like this:

We do declare that [ is a character that can trigger completion: https://github.com/microsoft/pyright/blob/86221745ca1e9122e4953ed2c63515b4de374bbf/packages/pyright-internal/src/languageServerBase.ts#L960
I think what would be an improvement would be at that point to quite literally offer up the string literals as full completions, rather requiring that a user types a quote. For example, TS does something like this:

(The above is an example of making the object, so that's really #654, but I just mean to point at the completion itself.)

(The above is an example of making the object, so that's really #654, but I just mean to point at the completion itself.)

(The above is an example of making the object, so that's really #654, but I just mean to point at the completion itself.)

(The above is an example of making the object, so that's really #654, but I just mean to point at the completion itself.)

Yeah, that's a good suggestion. I just checked in a change that implements this.

Yeah, that's a good suggestion. I just checked in a change that implements this.

A gotcha is which quote character to choose; I think TS probably scans the file to figure out the predominant one to offer.
I don't see the commit yet on pyright, but I'll take a look once it's there.

A gotcha is which quote character to choose; I think TS probably scans the file to figure out the predominant one to offer.
I don't see the commit yet on pyright, but I'll take a look once it's there.

Pyright also chooses the predominant quote in the document.

Pyright also chooses the predominant quote in the document.

Appears to work as expected:

Not sure how I feel about ' being the default choice when there aren't any quotes, though. A nit.

Appears to work as expected:

Not sure how I feel about ' being the default choice when there aren't any quotes, though. A nit.

Slightly curious which completion item kind is being used; TS has that horizontal line icon while we have the abc (which in my mind means "VS Code broke and is offering me word completions"). I think abc is CompletionItemKind.Text while the other is Enum. Can leave it to another time to compare the icons and fix that up, since we use quoted completions for other reasons as well.

Slightly curious which completion item kind is being used; TS has that horizontal line icon while we have the abc (which in my mind means "VS Code broke and is offering me word completions"). I think abc is CompletionItemKind.Text while the other is Enum. Can leave it to another time to compare the icons and fix that up, since we use quoted completions for other reasons as well.

Yeah, the use of abc as the completion type also threw me initially and made me think it was just picking up on terms in the file.
As for which quotes to use by default, best I can tell you is Black defaults to "/double quotes.

Yeah, the use of abc as the completion type also threw me initially and made me think it was just picking up on terms in the file.
As for which quotes to use by default, best I can tell you is Black defaults to "/double quotes.

My preference would be to swap this to Constant as Text is VSC's default "IDK" completion icon and that might be off-putting for some people. I'll see if I can change that for today (depending on if the power outage gets me back my machine...).
Reading the quote character, the parser seems to count and then have a boolean of predominantSingleQuoteCharacter which defaults to true if they're equal. That might be a further improvement later to change if we believe we want to be as close to black as possible in quote choicing.

My preference would be to swap this to Constant as Text is VSC's default "IDK" completion icon and that might be off-putting for some people. I'll see if I can change that for today (depending on if the power outage gets me back my machine...).
Reading the quote character, the parser seems to count and then have a boolean of predominantSingleQuoteCharacter which defaults to true if they're equal. That might be a further improvement later to change if we believe we want to be as close to black as possible in quote choicing.

This issue has been fixed in version 2021.1.1, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/main/CHANGELOG.md#202111-13-january-2021

This issue has been fixed in version 2021.1.1, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/main/CHANGELOG.md#202111-13-january-2021


Ordering projects

Being able to define a run order in morula.yml (or dependencies) would be useful.
My use case is as follows:
I have sub-projects A, B, and C.  Building subproject A requires subproject B to have already been build because the output of building B is used to build A.

There is a "before" and "after" key that you could use. Put project A into "before" and project "C" into after. Also keep in mind Morula runs projects in alphabetic order.

We could add a key to the configuration called dependencies:
dependencies:
  B: A

Then Morula makes sure that the command is run in B before it is run in A.
How would this work for morula changed? What if B has changes, but A doesn't? Now Morula needs to build B. But in order to do that it also needs to build A. But it wouldn't build A, because A doesn't have changes. Does that mean it now also needs to build A, even though A doesn't have changes?

Yeah, ideally if we declare dependencies, if A changes then we need to build B as well. Although we don't need to run tests or deploy for B. Maybe we can declare commands in morula.yml? Although we should be careful to not turn it into another make replacement

What about this:

building with dependencies: morula changed --include-dependencies build
testing without dependencies: morula changed test

Yeah I like that


fix: updated lib file source

:tada: This PR is included in version 0.0.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: