Authorization: Unable to use method where token header is a mandatory field

I have configured swagger to use a named header 'x-auth-token' for authorization and it works where the authentication header is not marked as part of the required fields of the method (for example, I have some open API methods that do not need the x-auth-token).
However, where the 'x-auth-token' is a mandatory field, I cannot use it in swagger without manually setting it in the UI.

In this screenshot, it appears that swagger-ui does not respect the fact that I have configured authorization to use this field. The UI should check if there is a mandatory header and it is configured with Authorization, then the input validation should be skipped and allow the call to proceed as Authorization will provide this required header.
Swagger is configured thus..
	private ApiKey authTokenSecurityKey() {
		return new ApiKey(SECURITY_KEY, "x-auth-token", "header");
	}

	@Bean
	protected SecurityConfiguration security() {
		return new SecurityConfiguration(null, null, null, null,
                                        SECURITY_KEY, ApiKeyVehicle.HEADER, "x-auth-token", null);
	}

	private SecurityContext securityContext() {
	    return SecurityContext.builder()
	        .securityReferences(defaultAuth())
	        .forPaths(PathSelectors.regex("/.*"))
	        .build();
	  }

	List<SecurityReference> defaultAuth() {
	    AuthorizationScope authorizationScope
	        = new AuthorizationScope("global", "accessEverything");
	    AuthorizationScope[] authorizationScopes = new AuthorizationScope[1];
	    authorizationScopes[0] = authorizationScope;
	    return Lists.newArrayList(
	        new SecurityReference(SECURITY_KEY, authorizationScopes));
	}

I think you need to mark it as hidden. @ApiParam(hidden=true) where its being used as a required parameter

@dilipkrish
I manually add these parameters via an OperationBuilderPlugin (so that it is relevant to certain conditions within my system).
The hidden parameter setting has no affect, I suspect it is down to the ParameterMerger code
  private Parameter merged(Parameter destination, Parameter source) {
    return new ParameterBuilder()
        .from(destination)
        .name(source.getName())
        .allowableValues(source.getAllowableValues())
        .allowMultiple(source.isAllowMultiple())
        .defaultValue(source.getDefaultValue())
        .description(source.getDescription())
        .modelRef(source.getModelRef())
        .parameterAccess(source.getParamAccess())
        .parameterType(source.getParamType())
        .required(source.isRequired())
        .type(source.getType().orNull())
        .build();
  }

It does not copy over the 'hidden' setting

@knocknarea you may have found a bug. Thank you for pointing that out. Perhaps a PR as well, so you get credit for it?


Update minimum Python version to 3.11

This will allow for the use of nice Asyncio additions

Could you also add 3.12 to the CI matrix?

3.12 didn't work because of softioc dependencies.


Removed tips from pay model

Description
Removed tips from pay model.
SASS-1126
Checklist PR Reviewer
Before Reviewing

[x]  Have you pulled the branch down?
[x]  Have you assigned yourself to the PR?
[x]  Have you moved the task to “in review” on JIRA?
[x]  Have you checked to ensure all dependencies are up to date?
[x]  Have you checked to ensure its been rebased against the current version of master?

Whilst Reviewing

[ ]  Have you run the tests?
[ ]  Have you run the journey tests?
[ ]  Have you looked at the JIRA story to make sure all Acceptance Criteria has been met?

After Reviewing

[ ]  Have you checked for merge conflicts or any changes in the current master that may affect the current pull request? i.e. does it need another rebase?
[ ]  Have you checked to make sure there are no builds in the pipeline before you merge?
[ ]  Have you moved the task to “in pipeline” on Jira?

[X]  Have you rebased against the current version of master?
[X]  Have you checked code coverage isn’t lower than previously?

After PRs been raised

[ ]  Have you checked the PR Builder passes?

Dependencies could be updated from 5.7.05 to 12.0, sbt plugin is now 3.4.0


Add support of *.routes file

Plugin supports only routes files.
It will be great to have support all *.routes files

Good point, that's an easy fix, I'll add it.

I've released a new version which supports also *.routes. The update should be available in the plugin repository in a few days, depends on how quickly Jetbrains guys review the update.

Thanks! Looking forward!

Available now.


feat: cache for remote module

Description
为 loadRemote 新增缓存机制，确保多次加载会复用同一个结果，同时在执行 removeRemote 时清除缓存
Related Issue

N/A
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Checklist

[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] I have updated the documentation.

Casue this change will make hooks no execute if hit cache , it belongs a break change , what do you think @zhoushaw

Module federation 内部已存在缓存，无需添加该缓存逻辑


Is the final goal of the project to reflash the controller's firmware or just redirect input?

Sorry for cluttering up the issue tracker with this, I didn't know of any other place to ask this question.
A couple months ago there was talk from some Steam Controller devs that had mentioned that Steam reflashes the firmware every time the bindings are changed to not have any of the inherent latency that comes with software based input redirection.
Is the plan to eventually be able to do the same?

Steam Controller no longer reflash the firmware, it uses uinput redirection like my driver (on Linux don't know the equivalent mechanism on Windows). I think they changed to permit very simple remapping at live, even while playing. It's very usefull when you fine tune you settings.
With firmware flashing there is a lot of potential issue, and also a delay to reconnect the gamepad that should have required to restart the game.
The latency might be increased but I think that this time would be be very very small compared to human reaction time.
If I have some time I'll perform some measurement (maybe with an oscilloscope) of the time between the usb message reception and the generated event received by a user application.


GRPC Online Query

Adds the onlineQuery method to GRPCClient, with supporting changes:

[x] Add createGrpc() to ChalkClient and withGrpc() to the client builder
[x] Add requiredResolverTags, now, queryNameVersion to set of online query params.
[x] Change GRPC interceptors to actually call super.start()
[x] Add a "engine-grpc" deployment type header

The grpc engine also returns a trace ID in the trailing metadata, the key is x-chalk-trace-id
Can you get that and store it as a field on the response?


bugfix: return fake getter if poolcoordinator is not enabled

What type of PR is this?
/kind bug
What this PR does / why we need it:
Fix getter function nil pointer, use fake getter instead of nil if poolcoordinator is not enabled.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

e2e-tests is expected to pass

I'll fix golangci-lint in another pr. PTAL @rambohe-ch


Superset Database is locked

[yes] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[no] I have reproduced the issue with at least the latest released version of superset.
[ yes] I have checked the issue tracker for the same issue and I haven't found one similar.

Superset version: 0.25.5
Hello,
I use Superset since 6 months with success.
My problem already happened several times until this morning, when a new time, Superset was locked.
I stopped superset and I tried to restart it with following command:
"nohup superset runserver -d -p 8088 &"
the product did not start because the Database is locked, here is the error message:
OperationalError:
(sqlite3.OperationalError) database is locked [SQL: u'SELECT ab_user.id AS ab_user_id, ab_user.first_name AS ab_user_first_name, ab_user.last_name AS ab_user_last_name, ab_user.username AS ab_user_username, ab_user.password AS ab_user_password, ab_user.active AS ab_user_active, ab_user.email AS ab_user_email, ab_user.last_login AS ab_user_last_login, ab_user.login_count AS ab_user_login_count, ab_user.fail_login_count AS ab_user_fail_login_count, ab_user.created_on AS ab_user_created_on, ab_user.changed_on AS ab_user_changed_on, ab_user.created_by_fk AS ab_user_created_by_fk, ab_user.changed_by_fk AS ab_user_changed_by_fk \nFROM ab_user \nWHERE ab_user.id = ?'] [parameters: (1,)]
After several kill and attempts (even rebooting the server), it has finally restarted.
As said, this problem already occurred several times.
could you help me to understand better this error, and how to process correctly ?
Expected results
Avoid to lock the Database.
Have a command / a solution to unlock the Database.
Actual results
Superset has restarted, but I am afraid the problem occur again.
Steps to reproduce
I don't know
Many thanks for your ideas / help

I haven't seen that before, but use a local mysql in my dev environment. Others use SQLLite for dev and it's ok. Maybe you have zombie processes you should clean up/kill?

On my Mac SQLLite always looks. now I use MariaDB for local dev and so far smooth

Superset is installed on Linux

@cycy9999  Did you get any solution?

unfortunately, no :-(

I am getting "database is locked" error on superset version 0.36.0. I could not solve it. any ideas or help would be appreciated

I am getting the same "database is locked" error on superset version 1.0. My system is Red Hat Enterprise Linux 8.2.
Does anyone have a good solution? Thanks!


Compiler error

I am attempting to compile to following program, and I receive a compiler error:
app "foreverFizzBuzz"
    packages {
        pf: "https://github.com/roc-lang/basic-cli/releases/download/0.7.1/Icc3xJoIixF3hCcfXrDwLCu4wQHtNdPyoJkEbkgIElA.tar.br",
    }
    imports [pf.Stdout, pf.Sleep, pf.Task.{ Task }]
    provides [main] to pf

runFizzBuzz : U64 -> Task.Task {} d
runFizzBuzz = \n ->
    {} <- Stdout.line (Num.toStr n) |> Task.await
    {} <- runFizzBuzz (n + 1) |> Task.await
    Stdout.line "done"

main =
    runFizzBuzz 1

The output looks as follows:
$ RUST_BACKTRACE=1 roc run foreverFizzBuzz.roc
An internal compiler expectation was broken.
This is definitely a compiler bug.
Please file an issue here: https://github.com/roc-lang/roc/issues/new/choose
thread '<unnamed>' panicked at 'I thought a non-nullable-unwrapped variant for a lambda set was impossible: how could such a lambda set be created without a base case?', crates/compiler/mono/src/layout.rs:1713:61
stack backtrace:
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

The compiler then hangs forever.
I should say I have attempted a number of variants of this code. If I remove the recursion it runs as expected. If I remove the trailing Stdout.line "done" (and the backpassing and await) so that the function is tail-recursive, I get a compiler panic with no message as above:
thread 'main' panicked at 'Error in alias analysis: error in module ModName("UserApp"), function definition FuncName("\x14\x00\x00\x00\x02\x00\x00\x00\xa1\xfb\xe2\xb3\x18i\xd5#"), definition of value binding ValueId(3): could not find func in module ModName("UserApp") with name FuncName("(\x00\x00\x00\x04\x00\x00\x00\xa1\xfb\xe2\xb3\x18i\xd5#")', crates/compiler/gen_llvm/src/llvm/build.rs:5761:19
stack backtrace:
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Probably a duplicate of #6484 but the example is simpler?

Uhu, the shortest reproductions of these errors are in #6252 and #5701, so I'll close this one.


Replace entire entry deletion logic with a simple wipe

Well this was a surprising realization. 1/3 of the code, entire logic for choosing what to delete is useless and I was able to delete it.
The only thing that changed by this is that all uefi entries within our managed range are regenerated every time uefi-mkconfig is ran.
This even fixes the issue from #5

This even fixes the issue from https://github.com/Biosias/uefi-mkconfig/issues/5

Do I understand correctly then that the issue in #5 relates to the order in which entries are created and that the sorting does sort the .old entries in the correct order (i.e. after the new entry)?

Not sure yet how I feel about re-generating the full list every time. I suppose it is simpler then setting the BootOrder to our sorting order, and it also has the advantage of propagating configuration changes to all old entries. So perhaps this is the best approach, I'm just a bit worried that we are now removing "known good" entries when the kernel is upgraded, which might increase the chance of ending up with a system where there are no working entries. For example, if I change the cmdline in a wrong way, now all my entries are wrong and none of them work anymore.

The scenario you describe is not relevant for this pull request.
If you change cmdline arguments, all entries will be regenerated even in the current released version.

That is the funny part, nothing has changed. Everything this new version does, was already being done by the released version, just in a more convoluted way.

Done


Upgrade weave to 0.17.0

Closes #1104
The upgrade to 0.17.0 is an intermediate step. The next release of IOV-Core will support 0.18.0 or even 0.19.0, depending on how the backend team will deal with the issues found.
In review, please have a special look at the manual pending(" calls in the tests

Sorry I didn't get a chance to review, but great work.
My only missing point was mutlisig support, but I made another issue for that


when visualizing, throw Error: Maximum call stack size exceeded

viz:  https://stately.ai/viz/f4d0b7e0-7f6f-445b-9c36-9ab48d565b9c

Hi @WangLarry, this happened because you misused the eventless transitions here "always" so it creates infinite loops, if you remove the always it should work.
as the docs say

Eventless transitions should be defined either with target, cond + target, cond + actions, or cond + target + actions


The inline code lexer now copes with internal backticks.

Coverage increased (+0.2%) to 90.244% when pulling 50ba72c7606ed6832a9ace0d6c3262dc99a9085d on Witiko:master into 0f5b3dd05f835e8ab92cac2dc2df8ad53a6e6441 on jhermann:master.

Coverage increased (+0.2%) to 90.244% when pulling 50ba72c7606ed6832a9ace0d6c3262dc99a9085d on Witiko:master into 0f5b3dd05f835e8ab92cac2dc2df8ad53a6e6441 on jhermann:master.


[discussion] Get it working with a non-Laravel SPA?

Hi, great package by the way!
Question
Was just wondering if it's possible to get pan working with a single-page application. Meaning, the backed is Laravel but the frontend is a totally different domain and not a Laravel application.
Suggestion?
My guess is that a <script> containing the Pan JS code and maybe a PAN_ENDPOINT environment variable pointing to the Laravel route might be a good developer experience for SPAs?
Again, thanks for the lovely little package.

Also looking forward PAN being available as an API for SPA applications


Include analysis functions for NEQ and inst switching experiments

Description
Providing analysis functions that automatically return QC plots for switching functions and facilitate to identify problematic systems
Todos

[x] adding routine to load the relevant data
[x]  provide plots that visualize dW and dE
[x] plot Crooks, Jarzynski and FEP results
[x] plot cummulative stddev vs sample size for NEQ systems

The analysis plot includes now the following:

we might want to improve naming in a further commit or PR
Status

[ ] Ready to go

Codecov Report

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


Contribute in this-week-in-neovim

We should contribute to This Week In Neovim with the changes I made.
If you're not familiar with the website, it's a weekly newsletter about the Neovim editor. Unfortunately, the dev is currently going through a rough time maintaining it, as described in this Reddit post. By contributing, we could help support the developer and also promote your plugin.

Originally posted by @lnc3l0t in https://github.com/Rawnly/gist.nvim/issues/7#issuecomment-1547464513

Yeah I'm super busy by my job in this period, any help is appreciated. :)


Feature Request: Add Facebook Messenger to the notification area on Linux

When opening Facebook Messenger, I would want that it would create an icon in the notification area:

Note that I am using LXDE (lxpanel).

This would be much appreciated. I'm using Linuxmint Cinnamon DE.

I am using Gnome 3 and I would love to have an icon in the notification area so the application can run in background and be opened only by clicking on the icon.

This is fixed in the latest beta,
Please see https://github.com/Aluxian/Facebook-Messenger-Desktop/issues/458


Branch refactor person to stall

This will refactor all persons into stalls, including method names and tests

lgtm


Add more ops and todos for missing ops

Hi @sbrunk - Apologies for initially contacting you to contribute to storch and then disappearing in a long hiatus; I am now finally free to contribute.
I can help implement some of the items in your TODO list, I will start implementing some of them on a branch forking this one.

Awesome!

Some functions, such as softmax are implemented in different modules and exported in torch. I noticed PyTorch documentation states that some of the torch.* functions are aliases from other modules, are you thinking Storch should keep a similar structure?

Yes I'm trying to keep the package structure similar to upstream PyTorch. I hope this makes it easier to find and keep up to the original implementations. Feel free to improve on that if you have suggestions. I might not have been entirely consistent.

Even though this module contains thin wrapper for torch functions, I assume tests will be useful on storch end to make sure things are wrapped properly. I would happily add tests for each, and could help me understand each of them, but wouldn't do it if you think its not needed.

Yes please do! Having more tests would be super helpful.
A good source for simple test cases are the examples in the Python docs.


Should migrations lock the database to allow concurrency?

I've noticed that when executing migrations concurrently e.g. php artisan migrate &; php artisan migrate one of them will fail if there is an operation like creating a table or inserting a unique ID.
This is causing some trouble when running our applications on a multi-instance setup were all  instances run the same deployment script for creating caches, etc... and finally executing a migrate on the same database, sometimes these migrations seem to overlap and some instances will report the deployment as failed.
I know there are three ways to avoid this issue, one is dealing with it on the migrations, always checking if a unique value is present, or a table exists before inserting.
The other is dealing with it on the cloud, and just running the migrations from one machine.
The third one would be allowing the framework to execute concurrent migrations safely.
I've checked how other frameworks similar to Laravel deal with this problem to determine which of these three options is the better solution, and I've found this issue on rails discussing the same thing, and they ended up merging this code for using advisory locks to allow concurrent migrations.
Do you think this behavior is desirable and candidate to be implemented?

Hi there,
Welcome to Laravel and we are glad to have you as part of the community.
Unfortunately this GitHub area is not for ideas, suggestions etc. This is only for issues/bugs with the framework code itself.
I will be closing your ticket here. You are able to open a ticket at https://github.com/laravel/ideas
Alternatively you are able to open a PR using the Contributions guidelines: https://laravel.com/docs/5.7/contributions#which-branch
If you feel I've closed this issue in error, please provide more information about how this is a framework issue, and I'll reopen the ticket.
Thanks in advance.

I've opened the issue there. Sorry for the inconvenience!


how to compile so files suitable for Android

Thank you very much, this is a great project, can you provide some compiled so files suitable for Android, which require arm64-v8a, arm64-v7a, x86, x86-64, or you can use ndk- Is there a specific command for build compilation?

https://github.com/heiher/hev-socks5-tunnel#android


OS 2.1: motion text display bug

OS  makes a mistake in projection motion texts: The gray box with the motion status is drawn too deep so that the motion text lines are broken. This also breaks the great new line numbering feature:

I have to correct myself: The gray box is not drawn to deep. The box will later grown when the election results are displayed:

The bug is probably that OS allows more than 70 characters per line but the the gray box limits the characters per line to 70 (71 if I counted correctly). It the space between the line number and the text is slightly reduced one could use 72 chars ber line. This is a sensible maximum limit. All the typographic advices I know say that one should not use more than 70 characters per line to keep the readability.

I think about this issue and I think this is not a bug because you can solve it:

by changing the line length in motion settings (default 80 chars)
by changing the projector resolution to get more space for full motion lines (default 1024x768).

So "wan't fix" for changing the default line length.

@emanuelschuetze you misunderstood me. The proper bugfix would be to set the default character number per line to 70. Currently the default is 80. I also explained why 80 chars is no good default value in terms of readability.

I think we can move this to 2.2.


Make concurrency configurable

If I see correctly, all the workers for the feeds in the configuration are started at once. As a result I'm getting a bunch of

Error while processing feed ...: Network error, status code TooManyRequests

It's then tried again after 10 minutes.
Suggested improvements:

make the delay slightly random, for example in an interval between 5 and 15 minutes to start with;
make the concurrency configurable and perhaps reduce the default concurrency to something like 4.

Thank you a lot, I'm curious to see where this project goes!

Thank you. I'll habe a look at this.
How many feeds have you configured?

Thank you so much! I have about 50 feeds at the moment but with pleroma-bot I used to have over 1200 (yes, it's a bit crazy).
If you tell me your preferred approach I can also try to send a patch (no promises; I coded a bit in Java and C before, C# not really).

@nemobis
I never thought that this tool would have to handle so many feeds at once :D
I've added a start delay to each feed worker. The delay is random between 0 and (currently) 30 seconds. Feel free to test if this will solve your errors.
#9

Thank you! This is better but I still see all workers start together, so they all finish within 30 seconds or so. My use case is a bit extreme but I'd want to spread them our across multiple hours, as I have a lot of infrequently updated feeds.

I have a task on my todo list to make the workers run-intervals configurable per feed. Currently they will run every 60 seconds.

I have added the ability to configure the run intervals, the global config is done via appsettings.json, per feed you can set the delay with worker_loop_delay, e.g. worker_loop_delay = 01:30:00 for 90 minutes (see the updated readme file)

Thank you! I'm now trying a 8h worker_loop_delay but I saw that all 50 feeds started within a hour of so, and after failing they're retried every 10 minutes or so (leading to more failures). Is some other default delay taking precedence?

Hey. 50 http calls within an hour shold not trigger any raten limiting.  Do you make more requests to the Server from the same ip address?
what would be your expected behavior?
The start delay is random between 0 and half of configured the worker delay time.

Hello! Yes this is a rather extreme case (heavily throttled Nitter instance). I figured it out, the retries every 10 minutes stop after the first successful request which prints a "New feed detected" log. So after the first time this is not an issue any more. My feature request is solved as far as I'm concerned.
Thanks a lot again!


fix(sourcemap): preserve original sourcesContent

Description

fix missing source files

fixes #12916 (#13657 happens though)
refs #11860 (not fixed, similar to #13657)
fixes #11238
fixes #10068
close #13092

refs https://github.com/vitejs/vite/pull/13092#issuecomment-1541492359
discuss https://discord.com/channels/804011606160703521/1106421226689265684

Additional context

excludeContent needs to be false unconditionally to pass sourcesContent all along the way.

Personally, I think there is no need to include the source content when source file exists. But we could exclude it for now in 4.4 beta to see how it works.

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[ ] Read the Contributing Guidelines.
[ ] Read the Pull Request Guidelines and follow the PR Title Convention.
[ ] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

@sapphi-red cc

Personally, I think there is no need to include the source content when source file exists.

We need to include it for now. If we exclude it, the browser will fetch the content. But because Vite cannot differentiate that request and a normal JS import request, Vite returns a transformed request. This results in showing a transformed file as a original source (related: #13391).

Let's merge it and do one more beta patch before releasing 4.4. cc @bmeurer


feat/imx8mp verdin

Add support for iMX8MP Verdin Dahlia
Depends on #179

Also @DavidMCerdeira can we add this to the list of supported platforms on the README file?

@josecm addressed all you comments

Thanks  @DavidMCerdeira. Just need to finish #179 before merging this.

@DavidMCerdeira please run the code formatter.
And don't forget to add #define PSCI_POWERDOWN_NOT_SUPPORTED 1 to the platform?s header.


(fix) runtime: tweak _wait_until_alive tenacity and exception handling

Short description of the problem this fixes or functionality that this introduces. This may be used for the CHANGELOG
EventStreamRuntime: tweak _wait_until_alive tenacity and exception handling

Give a summary of what the PR does, explaining any non-trivial design decisions

extra docker log fetching in run_action (PR #3870) commented out and added TODO
added exception handling to leave method on KeyboardInterrupt

Link of any specific issues this addresses
Fixes #3876

It's still doing this:

(llama): open the UI, write: 42
Llama starts to create an app.py as described by our example in the prompt... 😅
I let it run for ~18-20 steps (it got BrowsingAgent to work but it didn't finish)
do Ctrl+C

Then two things happen:

first, it seems to run more steps. I think this shouldn't happen, and didn't happen in the past
second, it tries to get container logs, over and over.

The last looks like this: (the lines with DEBUG were when I pressed Ctrl+C again)
23:13:28 - openhands:DEBUG: runtime.py:297 - Getting container logs...
23:13:32 - openhands:DEBUG: runtime.py:297 - Getting container logs...

openhands-py3.12(base) ➜  odie git:(wait-alive-exception) ✗ 23:13:40 - openhands:DEBUG: runtime.py:297 - Getting container logs...
23:13:56 - openhands:DEBUG: runtime.py:297 - Getting container logs...

openhands-py3.12(base) ➜  odie git:(wait-alive-exception) ✗ 23:14:16 - openhands:DEBUG: runtime.py:297 - Getting container logs...
23:14:36 - openhands:DEBUG: runtime.py:297 - Getting container logs...
23:14:56 - openhands:DEBUG: runtime.py:297 - Getting container logs...
23:15:16 - openhands:DEBUG: runtime.py:297 - Getting container logs...
23:15:36 - openhands:DEBUG: runtime.py:297 - Getting container logs...
ERROR:asyncio:Task exception was never retrieved
future: <Task finished name='Task-232' coro=<Runtime.on_event() done, defined at /Users/enyst/repos/odie/openhands/runtime/runtime.py:109> exception=ConnectionError(MaxRetryError("HTTPConnectionPool(host='localhost', port=32363): Max retries exceeded with url: /alive (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x16d0fc350>: Failed to establish a new connection: [Errno 61] Connection refused'))"))>

It's still doing this:
* (llama): open the UI, write: 42

* Llama starts to create an app.py as described by our example in the prompt... 😅

* I let it run for ~18-20 steps (it got BrowsingAgent to work but it didn't finish)

* do Ctrl+C

Then two things happen:
* first, it seems to run more steps. I think this shouldn't happen, and didn't happen in the past

* second, it tries to get container logs, over and over.

The last looks like this: (the lines with DEBUG were when I pressed Ctrl+C again)
23:13:28 - openhands:DEBUG: runtime.py:297 - Getting container logs...
23:13:32 - openhands:DEBUG: runtime.py:297 - Getting container logs...

openhands-py3.12(base) ➜  odie git:(wait-alive-exception) ✗ 23:13:40 - openhands:DEBUG: runtime.py:297 - Getting container logs...
23:13:56 - openhands:DEBUG: runtime.py:297 - Getting container logs...

openhands-py3.12(base) ➜  odie git:(wait-alive-exception) ✗ 23:14:16 - openhands:DEBUG: runtime.py:297 - Getting container logs...
23:14:36 - openhands:DEBUG: runtime.py:297 - Getting container logs...
23:14:56 - openhands:DEBUG: runtime.py:297 - Getting container logs...
23:15:16 - openhands:DEBUG: runtime.py:297 - Getting container logs...
23:15:36 - openhands:DEBUG: runtime.py:297 - Getting container logs...
ERROR:asyncio:Task exception was never retrieved
future: <Task finished name='Task-232' coro=<Runtime.on_event() done, defined at /Users/enyst/repos/odie/openhands/runtime/runtime.py:109> exception=ConnectionError(MaxRetryError("HTTPConnectionPool(host='localhost', port=32363): Max retries exceeded with url: /alive (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x16d0fc350>: Failed to establish a new connection: [Errno 61] Connection refused'))"))>

As far I can tell, that's all "normal" behavior as the runtime in on_event fetches logs e.g. before an action is about to be executed.
The problem here is potentially, that the app inside the container (or the websocket?) just died. The ConnectionRefusedError is, I think, an artificial error raised somewhere else in the codebase, but don't have that in front of me right now.
Seeing the intermittent errors in integration tests, I'd still go through with the revert in here at least.
Maybe @xingyaoww has additional ideas on how to tweak/change either the retry/tenacity and/or the LogBuffer?

We can use retry_when_not_exception_type to avoid the retry: https://tenacity.readthedocs.io/en/latest/api.html?highlight=except#tenacity.retry.retry_if_not_exception_type

Thanks for looking into this. I think we can merge the removal of the container logs line, it does appear to work - not always, but better...- on the PR branch.


Tox usage with pyenv local

So I've set pyenv global 3.5.0 3.4.3 2.7.8 and this works well if I'm outside any project that has python version locally defined using .python-version. This causes issue when running tox for project that has .python-version defined.
For example:
Outside project with no pyenv local set, I can do:
$ python --version
Python 3.5.0

$ python3.4 --version
Python 3.4.3

$ python2.7 --version
Python 2.7.8

Now, if I'm inside a project with pyenv locally set (having .python-version) like so,
$ pyenv virtualenv 2.7.8 my-project
$ pyenv local my-project
(my-project) $ python3.5 --version
pyenv: python3.5: command not found

The `python3.5' command exists in these Python versions:
  3.5.0

Is this by design or am I doing something wrong? Due to this, I can't run tox with project that has .python-version defined.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27891238-tox-usage-with-pyenv-local?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github).

Basically it is by design. If you'd like to usemultiple versions at once, you need to specify versions in some source (e.g. global, local or shell).
If your project has .python-version (local) in its repository, you can override the configuration with using 'pyenv shell' (or PYENV_VERSION envairoment variable) sice shell is the most favable in pyenv. Writing some wrapper script to run tox would work for your project.


Drop filter options from root_path, Home

Description of change
Drop filter options from root_path, Home

Will this work? Will the filter options all be automatically selected? I feel like I'm missing something

The routes.rb has this definition:
https://github.com/unboxed/bops/blob/6af61c9a42b62adf5f386b24ff5b4e447006ddbf/config/routes.rb#L8-L12
This means the params has two copies of the filter options - one at the top level and one inside :planning_application_search_filter. Seems like this one is what's affecting whether the checkboxes are selected.

Ah yes me being silly, got it


Cmake

is there interest from Slamtec to merge this? I think is really useful.
@warrenbocphet  I believe I can improve so it works for any OS


Pipe painter

About the PR

taken from https://github.com/SerbiaStrong-220/space-station-14/pull/73
Media

https://user-images.githubusercontent.com/16546901/245219430-0735db52-9f77-4086-91e1-6f3745543ca4.mp4

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: Added pipe painter

This is so rad

ok but what if it could paint everything 😳

What’s the point of just taking someone’s work from another fork, not ensuring it works, not running it on the local server, not trying to fix any issues?

I think it was working two weeks ago and it just broke after BoundUserInterface refactoring.

works right

Shouldnt it be the same item with the airlock painter?

Shouldnt it be the same item with the airlock painter?

merge them into the omnipainter

bruh

What’s the point of just taking someone’s work from another fork, not ensuring it works, not running it on the local server, not trying to fix any issues?

I think it was working two weeks ago and it just broke after BoundUserInterface refactoring.

anyways, it'd never happen if they at least ensured it worked and copied the latest version from that codebase, had to test it locally first

@lzk228 are you coming back to this?

nah bruh


Update api-management-key-concepts.md

correcting typo, (façade to facade) due to issue with searching the right word in the provided context.

@ghtali : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

LGTM, thanks for submitting

Thank you for approving.

LGTM, thanks for submitting

Thanks for approving.


Develop better algorithm for extracting context

Currently, I'm doing this:
extracted_info = {}
extracted_info["subject"] = sent_nps[0]["s"] # this isn't a good way of doing it, but works
extracted_info["verb"] = sent_vps[0]["s"] # bad way of doing it
extracted_info["object"] = sent_nps[1]["s"] # arrghh, buggy
extracted_info["action_context"] = sent_pps[0]["s"] if len(sent_pps) > 0 else "none" # ...
return extracted_info

This just sets the object properties to the first noun, first verb, and first preposition it finds. This works alright for very basic sentences, but it would be cool to develop a better algorithm that does this better using the position of the individual nouns and verbs.

Thought of a better method here.


Add unit tests for v2 runtime handler

Codecov Report

Merging #439 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #439   +/-   ##
========================================
  Coverage    48.35%   48.35%
========================================
  Files           69       69
  Lines         2072     2072
  Branches       230      230
========================================
  Hits          1002     1002
  Misses        1002     1002
  Partials        68       68

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 8b7f331...c980a03. Read the comment docs.


[Snyk] Security upgrade cryptography from 42.0.0 to 43.0.1

Snyk has created this PR to fix 5 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):

requirements.txt

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.

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
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Use of a Broken or Risky Cryptographic Algorithm
🦉 Resource Exhaustion
🦉 NULL Pointer Dereference
🦉 More lessons are available in Snyk Learn

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: security/snyk check is complete. No issues have been found. (View Details)


Show animation when extension is doing work

Is your feature request related to a problem? Please describe.
I'm working with a very large notebook, and a very slow network connection. It takes about 30 seconds for the extension to finish doing it's thing. During that time, it's not clear whether it has already finished, or if it's still working, or if it failed somehow.
Describe the solution you'd like
When you click the  icon, it starts rotating. Once the formatting has finished, it stops rotating.
Describe alternatives you've considered

A popup/dialogue is probably too annoying.
Changing icon colour might work, but you'd want to be sure it works with all themes

Additional context

Python package version: 3.0.2

Extension version: 3.0.2

Operating System and its version: Ubuntu 24

This sounds like a good idea, I would be happy to review a PR if someone is interested in implementing this!


Ansible Job doesn't start and timeout

@Fryguy @agrare Could you please let me know how to debug the ansible issue?  It doesn't start at all and timeout after 1 hour.
There are no additional logs generated until its timeout after 1 hour. I need info on how to debug this issue.
[----] W, [2022-09-09T17:39:41.515159 #682:8cdc]  WARN -- evm: MIQ(ManageIQ::Providers::AnsiblePlaybookWorkflow#timeout!) Job: guid: [d074a083-1cfe-4860-8361-3a05c9882461], job timed out after 4004.5297576 seconds of inactivity.  Inactivity threshold [900 seconds], aborting{"@timestamp":"2022-09-09T17:39:41.516999 ","hostname":"12c44e331d53","pid":682,"tid":"8cdc","service":"evm","level":"warning","message":"MIQ(ManageIQ::Providers::AnsiblePlaybookWorkflow#timeout!) Job: guid: [ab3befac-3883-4324-a080-9b5bf9ea6a60], job timed out after 4010.0471375 seconds of inactivity.  Inactivity threshold [900 seconds], aborting"}
[----] W, [2022-09-09T17:39:41.517099 #682:8cdc]  WARN -- evm: MIQ(ManageIQ::Providers::AnsiblePlaybookWorkflow#timeout!) Job: guid: [ab3befac-3883-4324-a080-9b5bf9ea6a60], job timed out after 4010.0471375 seconds of inactivity.  Inactivity threshold [900 seconds], aborting{"@timestamp":"2022-09-09T17:39:42.029054 ","hostname":"12c44e331d53","pid":709,"tid":"8cdc","service":"automation","level":"info","message":"Q-task_id([r1193_miq_provision_2977]) Updated namespace [Cloud/VM/Provisioning/StateMachines/Methods/ansible_azure_sg_provisioning  ManageIQ_Custom/Cloud/VM/Provisioning/StateMachines]"}

@Fryguy I get the following error after the job timeout.
{"@timestamp":"2022-09-10T11:18:57.566193 ","hostname":"e7dd69fe83d8","pid":31811,"tid":"2b17960db960","service":"evm","level":"warning","message":"MIQ(ManageIQ::Providers::EmbeddedAnsible::AutomationManager#authentication_check_no_validation) type: [nil] for [1] [Embedded Ansible] Validation failed: incomplete, Missing credentials"}
[----] W, [2022-09-10T11:18:57.566283 #31811:2b17960db960]  WARN -- evm: MIQ(ManageIQ::Providers::EmbeddedAnsible::AutomationManager#authentication_check_no_validation) type: [nil] for [1] [Embedded Ansible] Validation failed: incomplete, Missing credentials

Closing this. Increasing the Generic Worker count fixed the issue.


Remove obsolete APIs for 3.0

There are several obsolete APIs that have been accumulated over the versions, and still exist in order to maintain backwards compatibility. I'm hoping it's planned to remove these for version 3.0, to keep the codebase clean and lean.
This could be a matter of going over everything tagged with [Obsolete], but it may be worth listing and tracking in an issue.

[ ] Old type mapping API (ITypeMapper and derived, FallbackTypeMappingSource).
[ ] Overload of MigrationsSqlGenerator.ColumnDefinition that doesn't accept FixedLength
[ ] DefaultQuerySqlGenerator.GenerateBinaryOperator(ExpressionType)

(there are obviously many more).
Some of these could be a bit tricky, as they'd require potentially considerable effort to port to the newer APIs (I'm thinking about the old type mapping for example). It may make sense to carefully evaluate, API by API, whether it's worth it to remove or not.

@roji We are planning to remove anything that is marked as Obsolete in 3.0. There should already be a way to replace any of these calls with other, non-obsolete code that achieves the same objective. For example, code that is using the old type mapping API should move to using the new API.
We already did this for 2.0, so there should not be any obsolete APIs that remain from pre-2.0 days, unless we missed some. Also, we tend to create issues like some of these for specific cases where we need to do cleanup and just searching for Obsolete is not sufficient to find the case.
Keeping this open as a work item to track doing this work.

Great!

When we remove FallbackTypeMappingSource, we should consider updating CoreTypeMappingSource to map value types, string, and byte[].

In addition to obsolete methods, we should collapse overloads that were added to prevent breaks around parameter defaults. (They couldn't be marked obsolete)


Question: Is there a way to identify a route and create href link to it?

Hi there,
I'm using Express for what I imagine to be a large project, and I have many href links inside my views. I do not want to hardcode them at all! Any time I change something I must go and search in the code in order to make it work.
Short story: how can I track back a route, so I can use this in views:
<a href="{{ url('route_id') }} /> <!-- // as you don't mind what the template engine is -->

Long one: I've searched for a day about routing in Express and I found a few "reverse" routing packages, ALL of them quite broken - they either don't work with Router() or they create new Router every time, etc.
I got to know all the Layers and stacks that the Routing part creates. I understand that you guys want order in routes, as well as multiple handlers for the same path. I agree with all that.
But isn't there some easy way to add ids for those routes? I found that if I simply use
app.use('/path', function namedFunction(req, res, next) {});

I get namedFunction as a name of the Layer in the stack, which is great. But if I use router (like router.route('/path').get(...).post(...), then the Layer is named router and inner stack contains anonymous functions.
My idea here was that after I add all routes, I can browse app._router and it's stacks, getting names of routes, then using a function with name parameter so I can construct back a route.
Am I missing something, or there is no way to make this easy working? I can't imagine someone having 100+ routes in a project, that hardcodes their links inside it - I just refuse to think of it :)

Come on guys, what's up, not even a single comment? No one reading those issues?

Hi! Yes, currently it's not possible, especially when you start to think about how we actually allow your middleware to alter req.url/req.method and we continute to route based on these new values! Basically, the way our router is written, you cannot actually figure out the destination without executing the stack with a real request.
I believe, though, there are third party modules (I think part of kraken?) than provide this functionality, though.

Thanks for the answer, @dougwilson ! I've just seen their enrouten package. It does some pretty magical stuff that I'll have to deeply investigate tomorrow and see if I can use. Thanks for that!
Otherwise - is it really that bad idea to check if one of the params passed to the router is String (as they are all now checked to be Function), and if so - set it as a name of the Layer? Because now the name is set to the function name (which is sometimes undefined) and moreover the nested layers are all anonymous I guess because of this bind, right? I guess if it's because of it, there should be some pretty simple solution to at least set proper name to the router, and let people like me browse them however they like :)

+1 to this, will allow nested routes a breeze too.
Currently I'm doing the following, which is not the most pretty thing out there but it works :) (I tested several reverse packages too and didn't liked the results):
global.erp.app.set('route:organizations', global.erp.app
    .route(global.erp.config.get('api:prefix') + '/organizations')
    .get(require('./routes/index'))
    .post(require('./routes/create')));

I created a PR for this in the Router project if anyone from this issue wants to comment - https://github.com/pillarjs/router/pull/36

Is there still no way to generate urls to routes instead of hardcoding paths all over the place?

@wesleytodd I didn't mean to sound dismissive. Since I am new to the Node ecosystem and this was an open ticket with more than 2 years without activity, I really thought that there was something more and it had simply been forgotten.
Regarding https://github.com/pillarjs/router/pull/36, it also seems stalled, depending on https://github.com/pillarjs/router/pull/29 which is also stalled. Is this the router that express uses underneath?

Hey, I understand it looking forgotten.  It takes someone championing the topic to push things forward, sometimes pinging on a topic for a while to get people's attention.  Yes pillarjs router is the express router, but for the next major release we are working on.  This would be a great feature to get landed before then, so maybe you can start some conversations over there on how you think we can move it forward.


git push --follow-tags

Have you consider using git v1.8.3 git push --follow-tags instead of git push && git push --tags ?
As the tag is annotated, this will prevent for uploading developer unannotated custom tags

--follow-tags
Push all the refs that would be pushed without this option, and also push
annotated tags in refs/tags that are missing from the remote but are pointing
at commit-ish that are reachable from the refs being pushed. This can also
be specified with configuration variable push.followTags. For more information,
see push.followTags in git-config[1].

That's very interesting, @jmendiara, didn't know about this option. Will definitely try this out soon. Thanks!

First, thanks for making a great release tool.
Could you implement this --follow-tags option by default instead of git push --tags
git push --tags sends all of your local tags even if they are tags which are only pointing to your local repo that you have not committed to the server. It creates tag pollution from multiple users in the shared central repo, and some of those tags may not point to anything in the destination repo since the tag was only local.
Follow tags is much better since it only pushes tags that match your commits to the shared destination repo.
git push --follow-tags
That won't push all the local tags though, only the one referenced by commits which are pushed with the git push.
In case you don't like the --follow-tags option for some reason, git push with the specific tag name is also quite valid and is perhaps what is intended when you're running the release-it command. This also avoids tag pollution on the server from multiple users since only one tag is pushed rather than all of the tags from the local repo.

Thanks for the report and explanation. I've released this change as a bugfix release in v2.4.1.


error when running Nuxt I18n module with latest Nuxt version (π version) in StackBlitz

Reporting a bug?
Looks like Vite warmup calling ssrTransform directly without calling config hooks, directive Vite plugin here requires compiler from vite:vue plugin and it is missing or still not configured (maybe Nuxt bug).
Here a repo using pnpm to patch @intlify/unplugin-vue-i18n/vite (link to SB in the readme file):
https://github.com/userquin/nuxt-i18n-stackblitz-starter
Expected behavior
The app should start but  sometimes doesn't work.
Reproduction
https://stackblitz.com/fork/github/BobbieGoede/nuxt-i18n-starter/tree/v9
Issue Package
unplugin-vue-i18n
System Info
NA: works on local, the problem is StackBlitz

Anyway check the reproduction and the fix.

Screenshot

Here the log patching vite manually:

Additional context
I'm preparing a PR to fix this.
Validations

[X] Read the Contributing Guidelines.
[X] Read the README
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.

closing this, it is a bug in @vitejs/plugin-vue, compiler not being initialized correctly: https://github.com/vitejs/vite-plugin-vue/pull/475


execute multiple lanes in loop

I have white labeled android app code base.
and like to execute multiple lanes as per different product flavors using one command.
Is there any way for that ?
I do not want to use
fastlane lane1 && fastlane lane2 etc..
Is there any way like..
lane :Release_all do
// here I want to loop through different lanes
end
so if I use
fastlane Release_all
then it will build and deploy each of my product flavors to playstore..
My sample app file
for_lane "Product1" do
  json_key_file "./fastlane/keys/product1.json"
  package_name "com.example.procuuct1"
end

for_lane "Product2" do
  json_key_file "./fastlane/keys/product2.json"
  package_name "com.example.procuuct2"
end

My sample fast file
update_fastlane
default_platform(:android)

desc "Deploy Product1 on Google Play Beta"
lane :Product1 do
      gradle(task: "clean")
      gradle(task: "assemble",
      build_type: "Release",
      flavor:"Product1")
      upload_to_play_store(track:"beta",
      skip_upload_screenshots: "true",
      skip_upload_metadata: "true",
      skip_upload_images:"true")
end

desc "Deploy Product2 on Google Play Beta"
lane :Product2 do
      gradle(task: "clean")
      gradle(task: "assemble",
      build_type: "Release",
      flavor:"Product2")
      upload_to_play_store(track:"beta",
      skip_upload_screenshots: "true",
      skip_upload_metadata: "true",
      skip_upload_images:"true")
end
..

Also need help to optimize fastfile code..

Yes, of course - Fastfile is just ruby code.
If you need to keep the for_lane construct to set json_key_file and package_name you can just create a Release_all lane that first calls Product1 and then Product2 as this will switch to those lanes.
lane :Release_all do
	Product1
	Product2
end

might already be enough.

Did this work for your @bhavyahmehta?

Hello..  Apologize for late reply..
Got following error..
Fastfile:153 :in `block in parsing_binding': [!] uninitialized constant Fastlane::FastFile::Product1 (NameError)

Did you put that into the same Fastfile? This was trying to use the "lane switching" feature: https://docs.fastlane.tools/advanced/lanes/#switching-lanes
(In general lanes are usually written lowercase, but that shouldn't really influence the result here.)

Here is my new sample fastfile
update_fastlane
default_platform(:android)

desc "Deploy Product1 on Google Play Beta"
lane :product1 do
      gradle(task: "clean")
      gradle(task: "assemble",
      build_type: "Release",
      flavor:"Product1")
      upload_to_play_store(track:"beta",
      skip_upload_screenshots: "true",
      skip_upload_metadata: "true",
      skip_upload_images:"true")
end

desc "Deploy Product2 on Google Play Beta"
lane :product2 do
      gradle(task: "clean")
      gradle(task: "assemble",
      build_type: "Release",
      flavor:"Product2")
      upload_to_play_store(track:"beta",
      skip_upload_screenshots: "true",
      skip_upload_metadata: "true",
      skip_upload_images:"true")
end

lane :release_all do
	product1
        product2
end

I want after product1 lane completes it task , product2's task get start automatically.
I want like fastlane release_all  command execute all mention lanes (like product 1 and product 2).. one by one..

I want after product1 lane completes it task , product2's lane task starts automatically.
Like fastlane release_all command execute all mention lanes (like product 1 and product 2).. one by one..

Isn't this the same thing and exactly what is happening?

Got same error.. as previous..
'block in parsing_binding': [!] uninitialized constant Fastlane::FastFile::product1 (NameError)

Following code works..
lane :release_all do
      sh "fastlane product1"
      sh "fastlane product2"
end


Fix broken doc versioning

Corrects usage of sphinx-multiversion to build all versions of the documentation
Adds option on documentation page to switch between doc versions
Adds banner to warn users about old doc versions

Closes #94

Documentation build has been tested locally and in Github action, merging into dev.


Replace Travis with GitHub actions

Also set AppVeyor to test Ruby 2.6 only

This change is

:lgtm:

:lgtm:


Cythonizing a module containing numba functions

Hello,
I am trying to cythonize a module which contains numba functions and I am getting an Exception when trying to use such functions. Is there a way to cythonize a module containing njitted functions?
Thank you!

After checking Numba documentation, I have come up with a solution. I can use Ahead-Of-Time compilation so as to generate a .pyd extension of my .py module containing Numba functions. However, the documentation says nothing about the 'fastmath' flag when carrying out AOT compilation. Is there a way to enable fast match capabilities?
Thank you so much.

@cmartinezdem thanks for asking about this on the Numba issue tracker. Since this is a fairly exotic request, could you perhaps submit a small example of how you want to combine Numba and Cython?

I have several .py modules with some njitted functions. For instance, my .py modules could be as the following one:
import numpy as np
import numba as nb

@nb.njit
def foo(a, b):
    return a**b

Everything works fine but I need to cythonize the module, generating a .pyd file in Windows (.so in Linux) and generate a wheel so as to distribute it, and Numba seems not to work when the .py module is cythonized.

After trying AOT compilation using a setup.py file, I am getting the following error in the linker:

modulemixin.obj : error LNK2001: unresolved external symbol nrt_atomic_sub
modulemixin.obj : error LNK2001: unresolved external symbol nrt_atomic_add

This is my setup.py file:
from setuptools import setup
from utils import cc as utils_cc

setup(
    ext_modules=[utils_cc.distutils_extension()],
)

However, if I manually compile the code using iPython, it works:
from utils import cc as utils_cc
utils_cc.compile()

@cmartinezdem thanks for updating this issue! I not sure how to help in this case so I have a suggestion: perhaps it would make sense to setup something like a demo repository to showcase Numba and Cython interoperability? This would include a setup.py file and a few demo files to quickly get the use case you are describing going. Does that  sound like something you  might be able to set up?

Hi @esc. Thank you for your support.
The best way to fix this is to use AOT compilation, so all Numba related functions get compiled into a .pyd library (exactly the same that Cython does).
I have managed to make it work with the following setup.py:
from setuptools import setup  # To setup package

from source_module import cc

setup(
    ext_modules=[cc.distutils_extension()]
)

where 'source_module' is the python module containing my Numba functions. All Numba functions need to be decorated with<EMAIL_ADDRESS>The only drawback I have found (for now) is that @njit flags, such as 'fastmath' or 'parallel' cannot be used.

After trying AOT compilation using a setup.py file, I am getting the following error in the linker:

modulemixin.obj : error LNK2001: unresolved external symbol nrt_atomic_sub
modulemixin.obj : error LNK2001: unresolved external symbol nrt_atomic_add

This is my setup.py file:
from setuptools import setup
from utils import cc as utils_cc

setup(
    ext_modules=[utils_cc.distutils_extension()],
)

However, if I manually compile the code using iPython, it works:
from utils import cc as utils_cc
utils_cc.compile()

I am running Windows 10 with Python 3.8.2 (64 bits).

This was happening because I was using a complex setup.py which was preventing Numba from applying monkey patching to distutils.build_ext

Adding the possibility of using @njit flags in @cc.export would be great! Is it possible?

I have managed to add the 'fastmath' flag to the ExportEntry object:
class ExportEntry(object):
    """
    A simple record for exporting symbols.
    """

    def __init__(self, symbol, signature, function, fastmath=False):
        self.symbol = symbol
        self.signature = signature
        self.function = function
        self.fastmath = fastmath

Then, the _cull_exports methodof the _ModuleCompiler object shall me modified with the following code:
for entry in self.export_entries:
            flags_aux = flags.copy()
            if entry.fastmath:
                flags_aux.set('fastmath', value=cpu.FastMathOptions(True))

            cres = compile_extra(self.typing_context, self.context,
                                 entry.function,
                                 entry.signature.args,
                                 entry.signature.return_type, flags_aux,
                                 locals={}, library=library)

And it is working for me right now. I guess this code may be improved, but this simple patch works, so I would expect that 'fastmath' and 'parallel' flags could be added to @cc.export in an "official" way.

@cmartinezdem how would you like to proceed with this issue. Do you want to open (or convert this into) a feature request?

New feature request #7179 created. Closing this issue as the original issue appears to be resolved.


Add data-i18n="[title]translation_key" throughout site

I just realized that it's really easy to translate attributes like title, placeholder, etc. using i18next:
a(data-i18n="[title]common.submit_patch")
input(data-i18n="[placeholder]editor.enter_level_name", placeholder="Type in a level name")
button(data-i18n="[title]editor.randomize_terrain;editor.randomize", title="Randomize Terrain") Randomize

We should go through the site and add i18n tags for all the title and placeholder attributes found throughout the templates so they can be translated.

@Imperadeiro98 are there any remaining places in the site you know about that don't already have their own specific issues here?

Not that I know. As I find new places without i18n, I add them myself or open an issue, so I think this issue can be closed.


Can't click inside of a slide

Describe the bug
If you have a button to click on into a slide, the mobile will treat the click as a drag and it does nothing when you click on it.
To Reproduce
Steps to reproduce the behavior:

Add a  tag inside of a carousel slide
Click on it on mobile (Chrome's inspect element with toggle device toolbar on works also)
Nothing happens

Expected behavior
The click event is treated differently than the drag event and the button does what was meant to do.

up for this. the plugin is good. If this will be fixed soon it will be more awesome

<slide v-for="(announcement, index) in announcements" :key="index">               <img                 :src="announcement.image_path"                 :alt="announcement.title"                 @click="directToPath(announcement.redirect_url)"                 @touchend="directToPath(announcement.redirect_url)"                 class="img-fluid"               />             </slide>
What i did is just put a carousel for my announcements that contains images however i tried to click the image and the @click is not firing, so i tried @touchend and it works. This is only my temporary fix. i hope that you can fix this issue. Thank you


Auth failure after enabling 2FA

I performed the following sequence:

Logged in with heroku login in shell
Enable 2FA on dashboard.heroku.com
Go back to CLI and type heroku apps

At this point I get an auth failure as expected and I am prompted for username, password and 2FA code. Then the CLI prints Authentication successful. followed by Authentication failure.. I tried a few times again with no success. Then I stopped it CTRL-C and ran heroku login which succeeded and now it all works. Here's some sample terminal output:
hdev@f51524cc34c6:~$ heroku apps
Authentication failure
Enter your Heroku credentials.
Email<EMAIL_ADDRESS>Password (typing will be hidden):
Two-factor code: 459579
Authentication successful.
Authentication failure
Enter your Heroku credentials.
Email<EMAIL_ADDRESS>Password (typing will be hidden):
Authentication successful.
Authentication failure
Enter your Heroku credentials.
Email: Password (typing will be hidden):  !    Command cancelled.
hdev@f51524cc34c6:~$
hdev@f51524cc34c6:~$ cat .netrc
hdev@f51524cc34c6:~$ heroku login
Enter your Heroku credentials.
Email<EMAIL_ADDRESS>Password (typing will be hidden):
Two-factor code: 684420
Authentication successful.
hdev@f51524cc34c6:~$

This is using the deb package of toolbelt and runs in an Ubuntu 14.04 docker container.

I had this same issue as well w/ 3.24.4 after enabling 2FA. A fresh install fixed it.
I noticed that if I got my credentials wrong then it wouldn't even get to the 2FA step, and if I typed the token wrong then it would give an error specific to the 2FA token. So it seems like I was successfully authenticating, but still ending up denied.


[Storage] Cannot build .NET Storage SDK with default installation of Visual Studio 2019

Describe the bug
I cannot build the .NET Azure Storage SDK with the default installation of Visual Studio 2019 (Community and Enterprise)
Exception or Stack Trace
None.
To Reproduce
Steps to reproduce the behavior:

Create a new Azure VM

https://ms.portal.azure.com/
Virtual Machines tab on the left
"+ Add"
On the "Create a virtual machine" page

Image: Windows 10 Pro, Version 1809
Size: Standard D3 v2
Wait for your VM to deploy

Connect to the new VM

Click the "Virtual machines" tab on the left
Search for your new VM by name
Select the VM, click the "Connect" button at the top

Download and install Git

https://git-scm.com/downloads
Install Git with default settings

Download Visual Studio Community 2019

https://visualstudio.microsoft.com/downloads/
Install Visual Studio Community 2019 without any workloads.  Note that there are no workloads listed in https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/storage/CONTRIBUTING.md or https://github.com/Azure/azure-sdk-for-net/blob/master/CONTRIBUTING.md

Clone the repository

C:\>git clone https://github.com/Azure/azure-sdk-for-net.git

Attempt to open the repository in Visual Studio

Open the solution file in Visual Studio

C:\azure-sdk-for-net\sdk\storage\Azure.Storage.sln

Code Snippet
None.
Expected behavior
I expect to be able to build the solution in Visual Studio 2019.
Screenshots

Setup (please complete the following information):

OS: Windows 10 version 1809
IDE : Visual Studio 2019 Community
Version of the Library used: latest commit in master

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

Someone on the Engineering Systems side should run through this, capture all of our implicit requirements, and update CONTRIBUTING.md to enable new folks getting started with our project.

/cc @weshaggard @pakrym - after fixing the initial problem with a missing .NET Core, it will repro the Source Link  error MSB4018: The "Microsoft.Build.Tasks.Git.LocateRepository" task failed unexpectedly. bug we've been seeing.

I have updated contributing guidelines for this. I still cannot repro the source link error. It might be best to file a separate issue to track that, and include more details about the system and environment where it occurs.


Isssue 849 - Add the cloudformation:ContinueUpdateRollback permission to the pivotRole, for administration of linked environment accounts.

Feature or Bugfix

Feature

Detail
Adds the cloudformation:ContinueUpdateRollback permission to the pivotRole.  This makes it possible for an admin of the central infra account, who assumes the pivotRole of a linked environment, to help get a user's environment out of a bad state where an Update Rollback might have failed.  An administrator can iterate through all linked environments and trigger the Continue Update Rollback if many environment accounts have somehow gotten into the Update Rollback Failed state.
Relates
https://github.com/awslabs/aws-dataall/issues/849
Security
Please answer the questions below briefly where applicable, or write N/A. Based on
OWASP 10.

Does this PR introduce or modify any input fields or queries - this includes
fetching data from storage outside the application (e.g. a database, an S3 bucket)?  No

Is the input sanitized?  N/A
What precautions are you taking before deserializing the data you consume?  N/A
Is injection prevented by parametrizing queries?  N/A
Have you ensured no eval or similar functions are used?  N/A

Does this PR introduce any functionality or component that requires authorization? No

How have you ensured it respects the existing AuthN/AuthZ mechanisms?  N/A
Are you logging failed auth attempts?  N/A

Are you using or adding any cryptographic features? No

Do you use a standard proven implementations?  N/A
Are the used keys controlled by the customer? Where are they stored?  N/A

Are you introducing any new policies/roles/users?  The PR adds the cloudformation:ContinueUpdateRollback permission to the pivotRole.

Have you used the least-privilege principle? How? N/A

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hi @rbernotas, I have read the attached issue and the PR looks good


Support of lower versions of php

PHP 8.2 is not as widespread in the world as at least 8.0, the library looks extremely useful, but the version limitation will greatly reduce its prevalence
So please lower the requirements to at least 8.0,

This extension uses enums, which are available starting in PHP 8.1. Additionally, there are some Zend API features this extension uses that were not introduced until PHP 8.2. As a result, I'm not able to lower the requirements to 8.0.


POST certificate endpoint

This adds an endpoint to POST a certificate to the store. By default the certificate will be decrypted using the password in the store, but there is an option in the request to not decrypt the password.

Looks good @pdeziel - I had a suggestion on the status code to use.
The only other comment I had was a question about using the gzip writer and reader versus the regular zip reader and writer that was used previously. It's ok that a file name is not provided?

gzip is for single-file compression so no file name is required. A file name is required for zip because a zip file can compress multiple files into the archive. I figured that gzip made more sense for storing the passwords because it's a single file. Compressing the password is a little silly but I was thinking it at least prevents it from being read accidentally, although we could also base64 encode it accomplish something similar.


The hash value is different when running on ARM Linux

I am using the same online torrent file on both Mac OS and Arm Linux through electron.
On X86, the hash value is b0f4244cb5b78fa1ae96177d8d25d7be9dcf8b55, 5981b91b7cec2dd9ec16a276cff6297919ef3509  on ARM Linux.
So they can't see each other for seeding.
On X86 (Mac osx)

On Arm Linux

This is not BTorrent's issue, and I don't think it's a WebTorrent's issue either.
I googled around and found nothing. But maybe this is applicable to the simple-sha1 package.

The hash methods are platform dependent. Different CPU architectures, for example 32-bit vs 64-bit, Intel vs ARM, might produce different results for a given input.

@DiegoRBaquero Wow, crazy. I think we should fix this in simple-sha1.

I just run the test script of simple-sha1 on ARM, both browser and node, they all passed.
Anything I can do to confirm the root cause please let me know.

The result.infoBuffer (after benched.encode(torrent.info)) is not the same:
On Mac:

On Arm Linux:

This is the root cause:
https://github.com/themasch/node-bencode/blob/master/lib/encode.js#L48
When the piece length is 2097152, this code block will be triggered:
  if( val !== data && !encode._floatConversionDetected ) {     encode._floatConversionDetected = true     console.warn(       'WARNING: Possible data corruption detected with value "'+data+'":',       'Bencoding only defines support for integers, value was converted to "'+val+'"'     )     console.trace()   }
For verification, I just modified
buffers.push( new Buffer( 'i' + val + 'e' ))
to
buffers.push( new Buffer( 'i' + data + 'e' ))
then the hash value will be correct one 'b0f4244cb5b78fa1ae96177d8d25d7be9dcf8b55' on Arm.
I don't know how to fix this without side effect...

@solderzzc Nice debugging! bencode's lack of bignum support is also causing this issue for bittorrent-tracker: https://github.com/feross/bittorrent-dht/issues/115
Maybe it's time to finally fix it.

Why do they compute val in the first place?

Maybe the code is detecting if working on a float. But the format stored is different on ARM?
https://github.com/themasch/node-bencode/blob/master/test/encode.test.js#L42

I'm gonna go ahead and close this as it isn't a BTorrent's issue.


Crash from unsynchronized access to MediaCalendar.movies

https://ruddarr.sentry.io/issues/5856640924/?environment=appstore&project=4506759167803392&query=is%3Aunresolved issue.priority%3A[high%2C medium]&referrer=issue-stream&statsPeriod=14d&stream_index=2

Reading from this dictionary in CalendarView.body prior to the crash is legit (main thread)
But there can also be writes on background threads originating in MediaCalendar.fetchMovies
Solution: mark the dictionary as @MainActor, mark any method that uses it (e.g. maybeInsertMovie) as @MainActor. Consider marking all view models as @MainActor by default, use nonisolated as an exception where needed.

Closed via #341.


ES6 modules default exported function/class/generator name is not declared

// function
export default function foo() {
}
export {foo}; // hello.ts(3,9): error TS2304: Cannot find name 'foo'.

// generator
export default function *foo() {
}
export {foo}; // hello.ts(3,9): error TS2304: Cannot find name 'foo'.

// class
export default class Foo {
}
export {Foo}; // hello.ts(3,9): error TS2304: Cannot find name 'Foo'.

// hoistable
export {foo}; // hello.ts(1,9): error TS2304: Cannot find name 'foo'.
export default function foo() {
}

http://www.ecma-international.org/ecma-262/6.0/#sec-source-text-module-records
http://www.ecma-international.org/ecma-262/6.0/#table-42

is this fixed in 1.8?

Not yet. I haven't had a chance to look at it.

It looks like an earlier fix prevents exported identifiers from being available to namespaces that have merged with the module in question. It doesn't take into account that default actually does introduce a local name.

@WangJi and @falsandtru can you give tonight's typescript@next a try.

Tried on Version 1.8.0-dev.20151112, still got the same warning.
And
export default class Foo{
}
``
is complied into

exports.Foo = Foo; //instead of default
``

This bug fix indeed only fixes the name resolution, not the emit. Although the emit is wrong for classes but not functions. I'll open a separate bug for the incorrect emit.

And it only happens when the target is ES6, not ES5.

Turns out the bug is already known: #5594


pestcontrol: add widget definitions

Adds some more widget definitions for the Pest Control minigame.
I'd like to make #8096 into a plugin for the plugin hub, but I think these definitions should be pushed to the core client.

We don't add things to WidgetInfo for hub plugins since they bitrot easily. You can use the 2 arg variant of getWidget in your plugin.


Increase size of dummy return data for expectRevert to 512 bytes

Since the expectRevert cheatcode causes execution to continue after a function reverts, it needs to provide dummy return data, otherwise it will not pass the checks that the Solidity compiler performs on the return value. This issue was partially addressed in PR #1564 by returning 32 bytes of 0, but this is not enough if the compiler expects the return data to be greater than 32 bytes. Therefore, this PR makes the following changes:

Increase the dummy return data from 32 to 512 bytes of 0, matching the behavior implemented in Forge (see here)
Modifies the test_expectRevert_returnValue to test a return type that requires more than 32 bytes. This test did not pass before the change above, but passes after.

Looks good to me. You still need to update the expected files to pass CI:
* tests/foundry/foundry-list.check.expected

* tests/foundry/golden/*.expected

@anvacaru Ah, sorry, I didn't realize. Is there an automated way to do that or do I need to just do it manually?

@lucasmt I updated the files for now, but there are a few ways:
For the golden test files you can do make test-foundry-kcfg-diff -j2 FOUNDRY_PAR=12 CHECK=cp - takes around 25 mins
For the foundry-list you can do

make test-foundry-list  -j2 FOUNDRY_PAR=12 CHECK=cp - this runs all the foundry proofs, so that it might take a while.
You can instead grab the diff from the failed stage and apply it locally. Save the diff in a file (i.e., tests/foundry/diff) and then run git apply diff. Make sure to format the beginning of the diff file to modify the same file.

diff --git b/tests/foundry/foundry-list.check.expected a/tests/foundry/foundry-list.check.expected
index e6a59b0..581cb2c 100644
--- b/tests/foundry/foundry-list.check.expected
+++ a/tests/foundry/foundry-list.check.expected

Is there/can we craft an example in which more than 512 bytes is returned? What would happen in Forge then?

@PetarMax I'll merge this for now and then try to work on an example with more than 512 bytes. My guess is that Forge should also fail on that, but it's unlikely to have a function that would require the return data to be 512 bytes. This would correspond to 16 uint256s.

@PetarMax For the record, if I try to create a function with a return type that needs 512 bytes, the compiler gives me a "stack too deep" error.


[Idea/BrainStorming] NewSdk: Add PaketReference MSBuild item

Instead of having the references in a second file paket.references, I would like to include them into the project file.
This could look like this:
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net461</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <PaketReference Include="FSharp.Core" />
    <PaketReference Include="xunit" Group="Test" />
  </ItemGroup>

  <ItemGroup>
    <Compile Include="Library.fs" />
  </ItemGroup>

  <Import Project="..\.paket\Paket.Restore.targets" />

</Project>

Advantages:

Similar to the Nuget PackageReference, so maybe easier to explain to someone coming from nuget (just replace all PackageReferences with PaketReferences, and remove the explicit versions, because the versions are handled by paket)
It can be dynamically programmed. For example let's say I want to include some Analyzer in all my C# projects.
** Currently I need to explicitly add it to all paket.reference files. The most I could do is to then write a unit test that verifies that yes, all projects really import it. If I create a new project, and forget it, the UT fails.
** With this feature, I could solve this via MSBuild programming and e.g. a Directory.props
The new project format is nice and pretty (well, relative for xml) readable. So why split it up into two files?
You could do a conditional reference (this would solve https://github.com/fsprojects/Paket/issues/2596) via <PaketReference Include="FSharp.Core" Group="netcore" Condition=" '$(TargetFramework)' != 'net35' " />

Edit:
I just saw that this was already proposed at https://github.com/fsprojects/Paket/issues/2019.
I guess it was unlikely that I would be the only one to think of this.

Yes we are interested in in this. Like making it work in addition to the
existing references files.
Am 11.08.2017 16:22 schrieb "Lukas Rieger"<EMAIL_ADDRESS>
Instead of having the references in a second file paket.references, I
would like to include them into the project file.
This could look like this:

net461

Advantages:

Similar to the Nuget PackageReference, so maybe easier to explain to
someone coming from nuget (just replace all PackageReferences with
PaketReferences, and remove the explicit versions, because the versions are
handled by paket)
It can be dynamically programmed. For example let's say I want to
include some Analyzer in all my C# projects.
** Currently I need to explicitly add it to all paket.reference files.
The most I could do is to then write a unit test that verifies that yes,
all projects really import it. If I create a new project, and forget it,
the UT fails.
** With this feature, I could solve this via MSBuild programming and
e.g. a Directory.props
The new project format is nice and pretty (well, relative for xml)
readable. So why split it up into two files?
You could do a conditional reference (this would solve #2596
https://github.com/fsprojects/Paket/issues/2596) via

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/2606, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNJY9RX-3lG_06X59Ge9A8dwjC5ciks5sXGO0gaJpZM4O0uK_
.

We still need to keep references files around since it provides support for
other features that are not yet in csproj.
PR with PoC code very welcome
Am 11.08.2017 16:25 schrieb "Steffen Forkmann"<EMAIL_ADDRESS>
Yes we are interested in in this. Like making it work in addition to the
existing references files.
Am 11.08.2017 16:22 schrieb "Lukas Rieger"<EMAIL_ADDRESS>
Instead of having the references in a second file paket.references, I
would like to include them into the project file.
This could look like this:

net461

Advantages:

Similar to the Nuget PackageReference, so maybe easier to explain
to someone coming from nuget (just replace all PackageReferences with
PaketReferences, and remove the explicit versions, because the versions are
handled by paket)
It can be dynamically programmed. For example let's say I want to
include some Analyzer in all my C# projects.
** Currently I need to explicitly add it to all paket.reference
files. The most I could do is to then write a unit test that verifies that
yes, all projects really import it. If I create a new project, and forget
it, the UT fails.
** With this feature, I could solve this via MSBuild programming and
e.g. a Directory.props
The new project format is nice and pretty (well, relative for xml)
readable. So why split it up into two files?
You could do a conditional reference (this would solve #2596
https://github.com/fsprojects/Paket/issues/2596) via

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/2606, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNJY9RX-3lG_06X59Ge9A8dwjC5ciks5sXGO0gaJpZM4O0uK_
.

Yes we are interested in in this. Like making it work in addition to the
existing references files.

I assume that in the first stage, both would be additive? That is, if I have PaketReferences and a paket.references file, the sum of both should be used?

PR with PoC code very welcome

Sure, unfortunately the day has only 24 hours and I need to prioritize this against catching up with GoT ;-)
I want to finish the other stuff I already started first, and next week I am on vacation. If until after then no-one else has started with anything, I will try, but don't necessarily wait for me.

Yes it should merge both sources.
Am 11.08.2017 16:42 schrieb "Lukas Rieger"<EMAIL_ADDRESS>
Yes we are interested in in this. Like making it work in addition to the
existing references files.
I assume that in the first stage, both would be additive? That is, if I
have PaketReferences and a paket.references file, the sum of both should
be used?
PR with PoC code very welcome
Sure, unfortunately the day has only 24 hours and I need to prioritize
this against catching up with GoT ;-)
I want to finish the other stuff I already started first, and next week I
am on vacation. If until after then no-one else has started with anything,
I will try, but don't necessarily wait for me.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/2606#issuecomment-321832474,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNM9yHHxQqzOvqF0VeqF7QaY48IcYks5sXGhggaJpZM4O0uK_
.

I like the idea. I'm not sure about additive bahavior, because this can lead to unexpected results if one parent directory contains a paket.references file...

We need both.
Parent directory paket.references have always been a weird thing. We don't
advertise them that much, so I think it's fine.
Am 11.08.2017 4:49 nachm. schrieb "Matthias Dittrich" <
<EMAIL_ADDRESS>
I like the idea. I'm not sure about additive bahavior, because this can
lead to unexpected results if one parent directory contains a
paket.references file...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/2606#issuecomment-321834291,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNAuGd2CExsFj0owBZW3w4kJv6pDLks5sXGn2gaJpZM4O0uK_
.

@forki Can we deprecate parent paket.references? Not remove them (yet), but print a warning?

Note that I already added this to the (wish-)list of breaking changes for the next major (I really think we should just go ahead and do it. Therefore, I'm fully supporting the warning.

We could deprecate them for modern type csproj.
Am 21.08.2017 12:28 vorm. schrieb "Matthias Dittrich" <
<EMAIL_ADDRESS>
Note that I already added this to the (wish-)list of breaking changes for
the next major (I really think we should just go ahead and do it.
Therefore, I'm fully supporting the warning.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/2606#issuecomment-323616124,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNP0lw0NiwCsT2es3to4cj69Bluy4ks5saLL-gaJpZM4O0uK_
.

Why not for all? I never saw anyone actually using them? (I don't count "by accident" which is most of the time not what you want)
I can show you some examples of where it hurts or was added a workaround (empty files)

Question is imho more what should be the replacement, and to answer this we need to know some real life examples of people using it

Regarding the parent paket.references file: When doing https://github.com/fsprojects/Paket/pull/2675 I noticed they are not even properly supported when doing msbuild restore for old and new-style projects.
So they aren't even working as expected TODAY.

Ok, so IMHO we should remove support for them, but maybe print a warning if we detect that an old version would have used them.


Mutation Deprecated

Hello, when using xComponent.create(), I am getting the following warning in Firefox Quantum:
Use of Mutation Events is deprecated. Use MutationObserver instead
Please let me know if you need an example to help triage.
Thanks!

This should be fixed, please let me know if you still see it.

The warning is still there when using the latest minor of version 6 of the library. I tracked it down to the event DOMNodeInserted. Would you like me to submit a PR?

@christianpv I'd create a new issue or PR and link it to this one. Most maintainers don't check closed issues.


storcon: storcon deployments cause cplane operation failures (connection refused\nrequest must not be retried)

Context: https://neondb.slack.com/archives/C06K38EB05D/p1718209960490099?thread_ts=1718184799.253779&cid=C06K38EB05D
Problem
In prodlike cloudbench, we have observed that a storcon deployment can, 44s (!) after the storcon logs that it's up again, cause cplane to get connection refused errors when it tries to talk to storcon.
Analysis
@ololobus :

Networking in k8s may take some time to rollout and storcon has only one pod
Same for LB / ingress to discover targets

Impact
When a Cplane client does a POST request, it doesn't retry them when it gets connection refused because it doesn't assume idempotency.
Example cplane log message
{"level":"ERR","ts":"2024-06-11T21:06:56.135Z","logger":"publicapiv2","message":"incoming request finished with internal error","http_meth":"POST","http_path":"/api/v2/projects/broad-boat-65064583/branches","route":"CreateProjectBranch","request_id":"70326701-9c98-4a9a-8fcd-aea2770ec8ed","trace_id":"T9WXJzpEJmibet3VyxrRjh","project_id":"broad-boat-65064583","account_id":"3eeaaef0-50fa-4074-8ed7-0a20f097d9fb","ingress_duration_ms":7277,"status":500,"account_id":"3eeaaef0-50fa-4074-8ed7-0a20f097d9fb","error":"could not create project-branch: Get \"http://neon-storage-controller.neon-storage-controller.svc.cluster.local:50051/v1/tenant/50f4379c3e4849ff7025fa4c14dced53/timeline/b7fd1f7c95d710c166c93d1ed0871324\": dial tcp <IP_ADDRESS>:50051: connect: connection refused\nrequest must not be retried"}

Related

neondatabase/cloud#13222

cplane is tweaking retries there. possibly this specific branch creation API is idempotent and hence safe to retry
but @ololobus argument:

but retrying for >=1m in the http request handler doesn't look like a feasible approach. So having a zero-downtime rollout for storcon is likely a must-have for GA

Is this distinct from https://github.com/neondatabase/neon/issues/7797 ?

#7797 mentions 503, so, storcon was running.
What we observed here was connection refused, i.e., not even able to establish TCP connection.
A (very) narrow-minded solution to #7797 may not address connection refused issue.
But yeah, in spirit this is a dupe of #7797

POST is idempotent as long as it includes a timeline ID -- @Bodobolero, until we make the controller more seamlessly available during restarts (in Q3), can you make your client retry past this class of error?

Sorry overlooked this notification. yes as we discussed on slack I have added a workaround, see
https://github.com/neondatabase/cloud/blob/64be60c3f47228af2801f1e2e3261b4327d38dbb/bench/internal/controllers/productionlikecontroller/bench/productionlikebenchinit.go#L164

Closing this as dupe of https://github.com/neondatabase/neon/issues/7797


Headers on Android ignored when using shouldOverrideUrlLoading()

Environment

Technology
Version

Flutter version
2.2.3

Plugin version
5.3.2

Android version
9.0 (API Version 28)

Device information: Google Pixel 2 (Emulator)
Description
Expected behavior:
Using shouldOverrideUrlLoading() does not have any impact on the headers being set on initialUrlRequest.
Current behavior:
Once I use shouldOverrideUrlLoading() Android does not send the headers anymore. This behaviour is only observed on Android, not on iOS.
Steps to reproduce

Create a working example with headers being set:

InAppWebView(
  initialUrlRequest: URLRequest(
    url: Uri.parse('https://some.url'),
    headers: {'Some-Header': 'NO_HEADER'},
  ),
)

=> This works as expected. Header is sent.

Intercept the request, put allow it directly:

InAppWebView(
  initialUrlRequest: URLRequest(
    url: Uri.parse('https://some.url'),
    headers: {'Some-Header': 'NO_HEADER'},
  ),
  crossPlatform: InAppWebViewOptions(
    useShouldOverrideUrlLoading: true,
  ),
  shouldOverrideUrlLoading: (controller, event) async {
    return NavigationActionPolicy.ALLOW;
  },
)

=> Android does not send the Header anymore (iOS does).

In addition, header set in   initialUrlRequest: URLRequest() disappear in the following scenario :
A page required authentification from header and send an error if missing. I set the header in initialUrlRequest: URLRequest() and so I can access the page. The page contain a form as you can see bellow :
<form action="removed_for_github_issue" id="inscription" class="fields" enctype="multipart/form-data" method="post" accept-charset="utf-8">

When submitting the form, the error page about missing header appear. I think header aren't sent or aren't maintained on refresh...

While intercepting, you need to add the headers again
  shouldOverrideUrlLoading: (controller, event) async {
        event.request.headers = {'Some-Header': 'NO_HEADER'};
        controller.loadUrl(urlRequest: event.request);
        return NavigationActionPolicy.CANCEL;
  }


Testing ENS Merch Shop: I think I should be able only to see brichis.eth

Describe the feature you'd like to request

I think that the customize options should only show the ones I have in my wallet and should not show the option you use as an example.
Describe the solution you'd like
If I only have brichis.eth I think I should only be able to see brichis.eth
Describe alternatives you've considered
I think I'll feel it more clean If I'm only able to see brichis.eth because I only have one ENS

@brichis This is behavior we want to have because rilxxlir.eth is the first ENS name ever made, and we want users to be able to order it (it's also an easter egg)


What about M365 A3/A5 SKU for Education ?

The article only mentions Enterprise SKUs of M365. Lots of schools would love to use this policy with their academic licensing.

@amitpawar74 - May we know which Microsoft document link you used in reference to this issue?
Please share it with us so we can investigate further. Thank you.

This is in reference to https://docs.microsoft.com/en-us/microsoft-365/compliance/communication-compliance-case-study?view=o365-worldwide
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Joyce Y<EMAIL_ADDRESS>Sent: Thursday, April 2, 2020 5:31:29 AM
To: MicrosoftDocs/microsoft-365-docs<EMAIL_ADDRESS>Cc: amitpawar74<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/microsoft-365-docs] What about M365 A3/A5 SKU for Education ? (#1536)
@amitpawar74https://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Famitpawar74&data=02|01||b4f21891cfc9456097a108d7d6840ac2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637213734914514075&sdata=ZoXbVT4g36eagq2mG2OeMYnANz6WtY87JFxvK4IfRwM%3D&reserved=0 - May we know which Microsoft document link you used in reference to this issue?
Please share it with us so we can investigate further. Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fmicrosoft-365-docs%2Fissues%2F1536%23issuecomment-607500164&data=02|01||b4f21891cfc9456097a108d7d6840ac2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637213734914514075&sdata=XLrcZMtqnxrifg3bg7WoBteRCDyDcED7WZlfIj%2BqeVM%3D&reserved=0, or unsubscribehttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMMIAMNCQLJW4VPLUCVYYZTRKOXDDANCNFSM4LZBZHDQ&data=02|01||b4f21891cfc9456097a108d7d6840ac2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637213734914524068&sdata=Dtl7M2%2FtcFfjNxO7%2FrMs2BX1uDLNVoUhWQ5O5Z69Cow%3D&reserved=0.

@amitpawar74 - Thank you for this inquiry.
I will get this issue over to the Microsoft 365 writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@robmazz - Can you please share your insights on this issue?
Thank you.
cc: @kenwith

Adding my work email.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Joyce<EMAIL_ADDRESS>Sent: Monday, April 6, 2020 12:22 AM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/microsoft-365-docs] What about M365 A3/A5 SKU for Education ? (#1536)
@robmazzhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frobmazz&data=02|01||7b7cdae5c37d4e80dea908d7d97d7a0a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637217005242382783&sdata=FNS%2FzWvlPnE8bqCzRW0RQYMhG7vHlsLq85As6ezyB5E%3D&reserved=0 - Can you please share your insights on this issue?
Thank you.
cc: @kenwithhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkenwith&data=02|01||7b7cdae5c37d4e80dea908d7d97d7a0a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637217005242382783&sdata=PnG%2Ff6bqc7b6Oi9aOPYE5QyikW15CMUCuaEpbzauTuM%3D&reserved=0
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fmicrosoft-365-docs%2Fissues%2F1536%23issuecomment-609442438&data=02|01||7b7cdae5c37d4e80dea908d7d97d7a0a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637217005242392791&sdata=HwH%2BQ%2FyC4zHnl0JO%2FgyJZ0cltg65I3csk%2FsvzWZIwhY%3D&reserved=0, or unsubscribehttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMMIAMPFHNI5K7XTIGQP6HTRLCV2VANCNFSM4LZBZHDQ&data=02|01||7b7cdae5c37d4e80dea908d7d97d7a0a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637217005242392791&sdata=B9MFrvVp2rhkc%2BSLwlMKkfc8tDBf58%2F1Li7KYFIvLws%3D&reserved=0.

@amitpawar74 - thanks for the feedback, we've recently updated this topic and the "Get Started with communication compliance" topics with new licensing guidance. Here's a link (https://docs.microsoft.com/en-us/microsoft-365/compliance/communication-compliance-case-study?view=o365-worldwide#licensing) to the section you've referenced in this issue.


ArcRotateCamera fails after setting the upVector

Repro
const camera = new BABYLON.ArcRotateCamera(
   "orbitalCamera",  0, 0, 0, new BABYLON.Vector3(0, 0, 0),
   scene, true);
camera.upVector = new BABYLON.Vector3(0, 0, 1);
camera.setPosition(0, 200, 0);

Set a breakpoint and inspect the camera. The alpha, beta and radius are all NaN and position is undefined.

Expected result:
alpha = -pi/2
beta = pi/2
radius = 200

Current result:
alpha = NaN
beta = NaN
radius = NaN

See https://www.babylonjs-playground.com/#1A3M5C#158 for an example

Not a bug, see answer in forum https://forum.babylonjs.com/t/arc-rotate-camera-relationship-to-axes/15393/11

So, the problem is that setPosition takes a Vector3 as input, so this was my error.


[Fix #1127] Allow RSpec/EmptyExampleGroup to have example group types that should be allowed specified in its configuration

Allows certain example group methods to be excluded from RSpec/EmptyExampleGroup. Fixes #1127.

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Updated documentation.
[x] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

If you have created a new cop:

[ ] Added the new cop to config/default.yml.
[ ] The cop is configured as Enabled: pending in config/default.yml.
[ ] The cop documents examples of good and bad code.
[ ] The tests assert both that bad code is reported and that good code is not reported.
[ ] Set VersionAdded in default/config.yml to the next minor version.

If you have modified an existing cop's configuration options:

[x] Set VersionChanged in config/default.yml to the next major version.

@pirj suggested a better fix in the issue, so I'm closing.