Doesn't work

I've verified my mail address, but it still doesn't work:
export MapboxAccessToken=pk.eyJ1Ijoic3Ryb21uZX[...]aHcyd3BobjEydTZ4Y3QifQ
geojson-thumbnail test.geojson test.png

/usr/lib/node_modules/@mapbox/geojson-thumbnail/bin/geojson-thumbnail.js:38
    if (err) throw err;
             ^

Error: Server returned HTTP 401
    at Request.<anonymous> (/usr/lib/node_modules/@mapbox/geojson-thumbnail/node_modules/@mapbox/tilejson/lib/tilejson.js:223:19)
    at Request._callback (/usr/lib/node_modules/@mapbox/geojson-thumbnail/node_modules/lodash/lodash.js:10050:25)
    at Request.requestRetryReply [as reply] (/usr/lib/node_modules/@mapbox/geojson-thumbnail/node_modules/requestretry/index.js:105:19)
    at Request.<anonymous> (/usr/lib/node_modules/@mapbox/geojson-thumbnail/node_modules/requestretry/index.js:138:10)
    at Request.self.callback (/usr/lib/node_modules/@mapbox/geojson-thumbnail/node_modules/request/request.js:186:22)
    at emitTwo (events.js:126:13)
    at Request.emit (events.js:214:7)
    at Request.<anonymous> (/usr/lib/node_modules/@mapbox/geojson-thumbnail/node_modules/request/request.js:1163:10)
    at emitOne (events.js:116:13)
    at Request.emit (events.js:211:7)

Did you use
export
MapboxAccessToken

or
export MapboxAccessToken

Of course in a single line. I'm using 100 % Linux since > 5 years.

What do you mean with scope?
PS: You can already remove the "bug" label. I assume that this issue is a layer 8 problem.
Thanks for developing this tool!

@eigenwillig If you go to the mapbox.com web interface and go to the token section of your account it should let you list "allowed actions" (scopes) of a particular token.

The final .65iEY9SnvDxNPeCKo8Gkag was missing!

@eigenwillig Good stuff! Going to close, feel free to open a new ticket and cc @ingalls if you run into any more problems.


mudselect bind-value and bind-text has a bug take a look at it

Bug type
Component
Component name
mudselect
What happened?
https://try.mudblazor.com/snippet/GacQayQzEdGqlZNW
not changed bind value
https://user-images.githubusercontent.com/1549611/165004508-697fb6b4-b7a6-43f9-89ea-1fe7a4d9e729.mp4
Expected behavior
bing value and bing text changed at same time
Reproduction link
https://try.mudblazor.com/snippet/GacQayQzEdGqlZNW
Reproduction steps

...
Relevant log output
No response
Version (bug)
6.0.9
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

https://user-images.githubusercontent.com/1549611/165004609-02200753-93c2-4f28-af30-c38e529a86d3.mp4

https://user-images.githubusercontent.com/1549611/165004745-e1fbcb26-480a-4fbd-8202-34be574ae2bd.mp4

https://user-images.githubusercontent.com/1549611/165004775-80b5d645-b779-4f43-8d79-0813d8e6c0fa.mp4

some time select text display 'One' or 'Two', some time display '1','2'. I don't know why

Now MudSelectExtended is released. It should fix your issue.
Ready on Nuget: https://www.nuget.org/packages/CodeBeam.MudExtensions/6.2.1
Try online: https://codebeam-mudextensions.pages.dev/mudselectextended


Add k8s metrics for jobs and cronjobs

Part of https://github.com/open-telemetry/semantic-conventions/issues/1032
Changes
This PR adds metrics for k8s jobs and cronjobs that are already in use by the Opentelemetry Collector (k8scluster receiver).

k8s.job.active_pods (collector)
k8s.job.failed_pods (collector)
k8s.job.successful_pods (collector)
k8s.job.desired_successful_pods (collector)
k8s.job.max_parallel_pods (collector)
k8s.cronjob.active_jobs (collector)

Merge requirement checklist

[x] CONTRIBUTING.md guidelines followed.
[x] Change log entry added, according to the guidelines in When to add a changelog entry.

If your PR does not need a change log, start the PR title with [chore]

[ ] schema-next.yaml updated with changes to existing conventions.

Good catch @dashpole! My only concern is about the k8s.job.desired_successful_pods and
k8s.job.max_parallel_pods. Should those be also UpDownCounters since they are set by users and are mostly desired-state values?

The main question is how they should be aggregated.  UpDownCounters aggregate by summing, gauges aggregate by averaging.  If I was aggregating k8s.job.desired_successful_pods k8s.job.max_parallel_pods` across a cluster, I would probably want to see the sum of all desired_successful_pods or the sum of max_parallel_pods, rather than the average of either.  So IMO those should both be UpDownCounters.

This is a very interesting point. These metrics being additive makes sense to me. But the data point type from the receiver is gauge. I think most instrumentation libraries export UpDownCounter as non-monotonic sums but I am not sure if this a standard. Would this cause some confusion for users? Should there be a way to clarify export and instrument type in cases as this?

I think most instrumentation libraries export UpDownCounter as non-monotonic sums but I am not sure if this a standard.

Yes, that is correct.  It is a bit odd to specify an instrument type for metrics that are not recorded using an instrument, but that seems mostly editorial here.
IMO we should update the data point type in the receiver to be a non-monotonic sum.

I think most instrumentation libraries export UpDownCounter as non-monotonic sums but I am not sure if this a standard.

I think this is the closest we have:
https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/sdk_exporters/otlp.md#additional-environment-variable-configuration
which specifies exporting UpDownCounter with Cumulative aggregation in all cases

Do @semconv-system-approvers want to take a look?


[Event Hubs Client] Track Two: First Preview (Samples)

Summary
The intent of these changes is to build the initial set of documentation for the preview, focusing on the samples.
Goals

Create a set of samples for common target scenarios to help developers onboard to the Event Hubs client library.

Update the README with content for the samples.

Ensure that the samples are included in nightly builds and test runs.

Last Upstream Rebase
Monday, June 24, 2019  10:47pm (EDT)
Resources

.NET Event Hubs Client: Track Two Proposal (First Preview)
.NET Event Hubs Client: Deferred Features and Functionality

Related and Follow-Up Issues

[Epic] Release Event Hubs Client Library Track Two Preview 1 (#6030)

/cc: @chrisjbarr.   Would appreciate your thoughts on content.


Fixed issue where callback_metrics was replaced instead of updated

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
[x] Did you read the contributor guideline?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Fixes #490 .
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@ryanwongsa  thanks!


add applicants to query

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

29 of 34 (85.29%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 75.337%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sis_provisioner/utils.py
4
5
80.0%

sis_provisioner/dao/student.py
0
4
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
614

Relevant Lines:
815

💛  - Coveralls


[BC API 2.0 | Create salesCreditMemo] Update Request and Response Body

Added an example for response body and updated the request body. Removed the read-only id and manual nos. number
{
  "error": {
    "code": "Application_DialogException",
    "message": "You may not enter numbers manually. If you want to enter numbers manually, please activate Manual Nos. in No. Series S-CR.  CorrelationId:  7d7d7b4f-dffb-480c-a392-22760da74cde."
  }
}

Learn Build status updates of commit ae93eaa:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dev-itpro/api-reference/v2.0/api/dynamics_salesCreditMemo_create.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 7d3923d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dev-itpro/api-reference/v2.0/api/dynamics_salesCreditMemo_create.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 284583f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dev-itpro/api-reference/v2.0/api/dynamics_salesCreditMemo_create.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Thanks for your contribution, much appreciated!


⚠ PrivateHD has degraded performance

In 8bc2368, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 3543 ms

Resolved: PrivateHD performance has improved in 9b6fbfa.


feature request, openai API proxy.

Clear and concise description of the problem
any idea to provide OPNAI API proxy within this server?
Suggested solution
may merge some api http proxy service within server.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

We already have OPENAI_API_BASE_URL environment variable, you can set a proxy-server in self-deployed site.


Feature request: compare with clipboard

As in IntelliJ IDEA I'd like to compare a section of a file with what's in the clipboard.

VSCode Version: Code 1.10.2 (8076a19fdcab7e1fc1707952d652f0bb6c6db331, 2017-03-08T14:00:46.854Z)
OS Version: Linux x64 3.16.0-4-amd64
Extensions:

Extension
Author
Version

indenticator
SirTori
0.2.1

html-css-class-completion
Zignd
1.0.3

vscode-eslint
dbaeumer
1.2.7

jquerysnippets
donjayamanne
0.0.1

vscode-html-css
ecmel
0.0.33

backbone-js-snippets
ioanaapetrei
0.0.1

debugger-for-chrome
msjsdiag
2.6.0

better-merge
pprice
0.5.1

diff
rafaelmaiolla
0.0.1

stylelint
shinnn
0.23.0

JavaScriptSnippets
xabikos
1.4.0

Steps to Reproduce:

This issue has already been reported. Please use the search function next time before reporting an issue and/or this time if you are curious about the progress of the issue.

I saw that other issue it was about comparing a file. I was thinking about a section


test: improve coverage across the board

Codecov Report

Merging #744 (71b1d0a) into release/v5.26.0 (0f9c02d) will increase coverage by 0.77%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@                 Coverage Diff                 @@
##           release/v5.26.0     #744      +/-   ##
===================================================
+ Coverage            64.00%   64.77%   +0.77%
===================================================
  Files                   44       41       -3
  Lines                 2375     2322      -53
  Branches                 0      451     +451
===================================================
- Hits                  1520     1504      -16
+ Misses                 855      774      -81
- Partials                 0       44      +44

Impacted Files
Coverage Δ

...ework/commands/openapi_generator/json_to_object.py
100.00% <100.00%> (+26.92%)
:arrow_up:

...amework/commands/rest_builder/validator_builder.py
100.00% <100.00%> (+5.06%)
:arrow_up:

...k_add_on_ucc_framework/install_python_libraries.py
100.00% <100.00%> (+19.27%)
:arrow_up:

... and 21 files with indirect coverage changes


Implement blocking transport

This implements a http.RoundTripper wrapper to block multiple simultaneous upstream requests. Currently it's optional to use via NewBlockingTransport so as to not change the existing API.
Tries to solve Issue #60

You should probably take a look on this lib!
https://godoc.org/golang.org/x/sync/singleflight

I don't think it makes sense to add extra responsibilities to this library; roundtrippers can be composed, and this seems like something that could be implemented elsewhere.


enforce ios or android tab navigator style regardless of platform

I want to have ios tab navigator as the root navigator. In each ios tab, I would have android tab navigator. However it seems that the platform will dictate how tab navigator looks like. Can I enforce a tab navigator to be ios or android style?

You can use the Tab Bar options to enforce specific setups for tabs. https://reactnavigation.org/docs/navigators/tab#TabNavigatorConfig
You would need to set at minimum the following settings (I believe):

tabBarComponent
tabBarPosition
showLabel (on tabBarOptions)
showIcon (on tabBarOptions)

Use TabNavigatorConfig someHow like this `   {
//tabBarComponent: 'TabBarBottom',
tabBarPosition: 'bottom',
animationEnabled: false,
swipeEnabled: false,
    tabBarOptions: {
        // showLabel:true,
        showIcon: true,
        activeTintColor: 'blue',
        inactiveTintColor:'#999999',
        style: {
            backgroundColor: '#ffffff',
        },
        indicatorStyle: {
            backgroundColor: 'white'
        }
    }
}`

I do the following:
const tabs = TabNavigator({
    // Defining tabs as usual
}, {
    ...TabNavigator.Presets.AndroidTopTabs, // or iOSBottomTabs
    tabBarPosition: "bottom",
    // Defining options as usual
});

I specify all properties from the default presets, this overrides the platform specific tabbar behavior on all platforms. Perhaps this is an easy way to get what you need?


Save Trip Modal Messages

What is a quick description of the change?
Add string literals for SaveTripModal default value and placeholders to messages.js.
Is this fixing an issue?
none
Are there more details that are relevant?
none
Check lists (check x in [ ] of list items)

[  ] Test written/updating
[  ] Tests passing
[x] Coding style (indentation, etc)

Any additional comments?

Received @Hiram-Silvey 's verbal approval to merge since this is the last day of the internship and he was not able to review the PR.


It is normality to get DUP packet in ping by using vpn over dns mode ?

Prerequisites

[ √]  Can you reproduce?

[ X]  Are you running the latest version of SoftEtherVPN?

SoftEther version: 5.01.9672
Component: Server
Operating system: Windows 2008
Architecture: 64bit
Description
I always get DUP packet in ping by using linux client, it is normal ?

Should be the case only in ICMP mode.


IndexedDB is not working with iOS

The reason: Alasql uses Chrome's specific function indexedDB.webkitGetDatabaseNames().
This function is not critical, and can be modeled with openDatabase().
UPDATE
The same problem with Firefox

This bug was related to this issue:
https://github.com/agershun/alasql/issues/126
But it still was not solved.

Has  indexedDB.webkitGetDatabaseNames() been changed to openDatabase() ?

Has indexedDB.webkitGetDatabaseNames() been changed to openDatabase() ?

Yes it has. I'm not sure if this is a problem anymore.
if it's doing some conditional check or something.
but at least my PR: #877 should at least cover this issue if it's not already solved.
so i think this can be closed


⚠ Bofrost.at has degraded performance

In 6c17302, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2034 ms

Resolved: Bofrost.at performance has improved in ffcdb84 after 6 minutes.


Update es_ES translation

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[ ] a bug fix (Fix #...)
[ ] a new Ripper
[ ] a refactoring
[x] a style change/fix
[ ] a new feature

Description
Hi team -
I took the en_us file and checked for entries that were not defined in the es_es file. I added those and added their spanish translations. I've also updated some of the old ones.
Testing
Required verification:

[ ] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[ ] I've verified that this change works as intended.

[ ] Downloads all relevant content.
[ ] Downloads content from multiple pages (as necessary or appropriate).
[ ] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[ ] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.

tx for the pull request, you mind doing it here as well? https://github.com/ripmeapp2/ripme
nobody with time has rights here at the moment i uess.

@soloturn Done!


Initial labels

I am not quite sure how the initial labels for memories are initialized for odc. To make it online I guess it should be initialized to zeros or random numbers but doesn't seem like it.

They are initialized via k-means. Please check deepcluster_hook.py.

Thanks for reply. Just to make sure, is the model using   all samples for  initial k-mean?

Yes, you're right.


BUG: Fix for .extractall (single group with quantifier) #13382

[X] closes #13382
[X] tests added / passed
[X] passes git diff upstream/master | flake8 --diff
[X] whatsnew entry

Note that I had to fix it based on this thread, rather than directly with [x for x in ['ab']] as this broke previous tests.

minor changes, looks good. ping when green.

[Current coverage][cc-pull] is 84.23%

Merging [#13397][cc-pull] into [master][cc-base-branch] will increase coverage by <.01%

@@             master     #13397   diff @@
==========================================
  Files           138        138
  Lines         50803      50805     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          42793      42795     +2
  Misses         8010       8010
  Partials          0          0

Powered by Codecov. Last updated by [d5bea25...2afae62][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pydata/pandas/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pydata/pandas/compare/d5bea25274b5faad5d2014b279a1da6451ebe1c7...2afae6272783f0fd3de76ae520b6bdf1e593eeca
[cc-pull]: https://codecov.io/gh/pydata/pandas/pull/13397?src=pr

@jreback Done.

thanks!


Use webpack-single-page-boilerplate

Done on https://github.com/nihey/blog/commit/94a6bf2ca30040a99b432b8969ee9a48fd5fcbf5


The editor completely softlocks in weird ways when editing a Curve2D, which forces a reset.

Tested versions
Only present in 4.3 dev 5
System information
Godot v4.3.dev5.mono - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3070 Ti (NVIDIA; <IP_ADDRESS>92) - AMD Ryzen 7 5800X 8-Core Processor (16 Threads)
Issue description
When editing a Path2D and various other nodes (Happens reliably with Path2D, other nodes are not known, but still happens occasionally), the ability to change values, add nodes / remove nodes, etc is completely disabled, regardless of the scene, or if the path2D node is selected. It is incredibly frustrating as i frequently use Curve2D's in my project, and each time i have to reload the editor completely once i've finished editing the curve.
Steps to reproduce

Create a Path2D node
Select it, to have the points editor tab show up on the top of the window.
Observe how certain actions just cannot be performed, such as pressing the "reset" button next to the curve2D, Setting of values is completely disabled as well as adding and deleting nodes.

Minimal reproduction project (MRP)
N/A

Tested some more. Found out the issue is only persistent if the scene with the Curve2D is present, closing and reopening the scene fixes it, still frustrating.

did some more testing, i think its something to do with adding or editing a point on the curve, as simply selecting the curve2D does not cause this issue?

I've noticed this, saving the scene also unlocked things for me.

Issue #89781 fixes this, thanks guys! ❤

this appears to be happening again in 4.3.beta3 :(

@uzkbwza Can you give more details on how exactly you're reproducing the issue?
I can't reproduce the issue following the steps in the OP in 4.3.beta3 on Linux.

same here, cannot seem to replicate the issue based on what the bug originally was. Interested to see if its a new thing.

@uzkbwza Can you give more details on how exactly you're reproducing the issue?
I can't reproduce the issue following the steps in the OP in 4.3.beta3 on Linux.

creating a path2D node and adding or editing points breaks the undo functionality for the whole editor, requiring a restart. I'll try to see if I can replicate it with a minimal example when I'm home

here is a video of what happens with just a base Path2D node.
https://github.com/user-attachments/assets/e7415033-12a6-4ec5-a63d-1028bca25477


Returning a tuple of closures that utilizes a parameter pack to compiles, but is impossible to use.

Description
This is highly related to #75558. Make sure to read that first.
Reproduction
Both to get around that bug, and to use the "instance method" syntax we want, I tried this. It compiles.
func map<each Input, each Output>(
  _ input: (repeat each Input)
) -> ((repeat (each Input) -> each Output)) -> (repeat each Output) {
  { transform in (repeat (each transform)(each input)) }
}

However, it is similarly impossible to use:
typealias Closure = (Int) -> Int
map((0, 1)) as ((Closure, Closure)) -> (Int, Int)

Cannot convert value of type
'(((Int) -> Int, (Int) -> Int)) -> (Int, Int)' to type '((Closure, Closure)) -> (Int, Int)' (aka '
'(((Int) -> Int, (Int) -> Int)) -> (Int, Int)') in coercion

(Those are the same type. 🙃)

Typical usage generates more confusing errors.
func ƒ(_: Int) { }
map((0, 1))((ƒ, ƒ))

Expected behavior
As with the earlier bug, it should be possible to match up a tuple of closures to a parameter pack of inputs and outputs.
Environment
swift-driver version: 1.112.3 Apple Swift version 6.0 (swiftlang-<IP_ADDRESS>.8 clang-16<IP_ADDRESS>)
Target: arm64-apple-macosx14.0
Additional Information
This is not a problem with only one parameter pack.
func forEach<each Input>(
  _ input: (repeat each Input)
) -> ((repeat (each Input) -> Void)) -> Void {
  { body in repeat (each body)(each input) }
}

var value = 0
func add(_ summand: Int) { value += summand }
forEach((1, 2))((add, add))
#expect(value == 3) // ✅

Very similar issues indeed.
Reduction:
func test<each I, each O>(
  _: (repeat each I)
) -> (repeat (each I) -> each O) {}

let _: ((Int) -> Int, (Int) -> Int) = test((0, 1))

295 | ) -> (repeat (each I) -> each O) {}
296 |
297 | let _: ((Int) -> Int, (Int) -> Int) = test((0, 1))
    |                                       `- error: '(repeat (Int, Int) -> _, _)' is not convertible to '((Int) -> Int, (Int) -> Int)', tuples have a different number of elements [tuple_types_not_convertible_nelts]
298 |
299 |


Add issue templates

lots of bug reports which don't have enough information to be diagnosed.

Codecov Report

Merging #361 (9493c5b) into master (f78bb5d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #361   +/-   ##
=======================================
  Coverage   77.80%   77.80%
=======================================
  Files          26       26
  Lines        3685     3685
=======================================
  Hits         2867     2867
  Misses        818      818

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 f78bb5d...9493c5b. Read the comment docs.


Why do we have to use an OAuth2 service if we consider to host it in the company network ?

Hi,
I'm trying to integrate your product in my company to aggregate all of our zabbix servers.
My concern here is it seems to be mandatory to have an OAuth with google or github to display the alerts.
But I don't want to expose it outside my company....
Is there a work arround for that ?
By the way I tried to setup even google or twitter auth without success, I always have a json popup and never get logged in.
It seems that I need a callback url (saw it on the docker alerta page), but I don't know what to put.
I tried to put the same as the homepage url but it fail.
Have you got any idea ?
And last to be really clear, I've also setup the alertad oauth part and alerts are being pushed successfully by zabbix in my mongodb db.
Thanks for your help

There is comprehensive documentation on setting up authentication on the docs website ... http://docs.alerta.io/en/latest/authentication.html If you still can't get OAuth2 working I'd like to know as I've spent a lot of time trying to make a potentially complex aspect of the config as straight-forward as possible.
Note though, that it is not mandatory to use authentication at all. The default out-of-the-box configuration does not require any authentication ... https://github.com/guardian/alerta/blob/master/alerta/settings.py#L23 However, you won't be able to "watch" alerts in the console because alerta won't know who's logged.
BTW, thanks for letting me know about the Docker documentation -- it seems to be out-of-date. I'll fix it.

Fixed in #17


Use Bundler::SharedHelpers.filesystem_access in CompactIndexClient#update

Related to bundler/bundler#4429

This repo is not up to date and cannot use Bundler functionality

Closing due to pending repo update/sync with vendored code in Bundler.


[tests-only] Bump core commit id for API tests

to get TUS tests for upload to moved file and upload to overwrite file, see owncloud/core#38177
to get TUS tests for the creation extension, see owncloud/core#38178

@swoichha needs some updating of expected-failures files.

@swoichha needs some updating of expected-failures files.
runsh: Total unexpected failed scenarios throughout the test run:
apiWebdavUploadTUS/lowLevelCreationExtension.feature:19
apiWebdavUploadTUS/lowLevelCreationExtension.feature:20
apiWebdavUploadTUS/uploadToShare.feature:66
apiWebdavUploadTUS/uploadToShare.feature:67

Also, maybe there is some settings change that can be made so that lowLevelCreationExtension.feature will pass? Those test scenarios are passing in owncloud/ocis

I have updated the expected-failure file and https://github.com/owncloud/core/pull/38180 must fix the problem with lowLevelCreationExtension.feature


Workaround for incompatibilities between Prism 0.24.0 and 0.25.0

This is a workaround for incompatibilities between Prism 0.24.0 and 0.25.0. To upgrade to Prism 0.25+, it is necessary to investigate the following build error and provide feedback to Prism:
https://github.com/rubocop/rubocop/actions/runs/8578707777/job/23512878899
I will proceed with these separately.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

I did some quick digging here and this is probably from prism https://github.com/ruby/prism/pull/2665. The parser gem doesn't seem to support this type of check: https://github.com/whitequark/parser/pull/975


Fix data race in app.Run with reused commands

Enables commands to be reused across multiple instances of cli.App and executed in parallel tests.
Turned out to be a pretty easy fix after all 😄
It's likely I didn't catch all race conditions around app setup.
For example, I'm fairly certain parallel calls to app.Run(...) on the same app would fail, since they'd both call Setup() and mutate their commands.
I deferred fixing that since I think it's both easily avoidable in tests and unrealistic outside of tests.
(If someone does need to fix that, I'd bet a setupOnce sync.Once field in *cli.App would do the trick. Though it would make shallow copies of App fail go vet 🤷 )
What type of PR is this?

bug

What this PR does / why we need it:

Fixes data race described in #1242, adds new (previously failing) test
Enables race detection flag in GitHub Actions

Which issue(s) this PR fixes:
Fixes #1242
Testing
Red -> Green: Added a failing test, then made it pass 🟢
Release Notes
This might pose an issue for folks relying on the slice of command pointers being identical before and after calling Run() (and Setup() indirectly). I imagine that could be an issue if app.Run is called more than once and an old command ptr is mutated, but I don't think that would happen outside of tests anyway.
Fixes data race for reused commands in parallel tests

@lynncyrin @saschagrunert Hi all! This is all set for review, let me know what you think 🙂

Thanks for the contribution here!
Based on the architecture of the this application, as well as the use case for a CLI tool, there's little doubt in my mind that this is not designed to be safe for concurrent use. With that in mind, a shallow copy is going to protect us on some fronts, but not others. In other words, we're adding a small piece of additional complexity to address this particular use case, but it doesn't resolve the underlying issues with the code patterns being unsafe for concurrency.
If we're interested in solving the problem completely, here a few more places where mutations occur on a Command during its lifecycle:

https://github.com/urfave/cli/blob/13ded1e7c4174565d9e8cde054c5cead43b75018/command.go#L276
https://github.com/urfave/cli/blob/13ded1e7c4174565d9e8cde054c5cead43b75018/app.go#L349
https://github.com/urfave/cli/blob/13ded1e7c4174565d9e8cde054c5cead43b75018/command.go#L99

Those are not comprehensive—just what I found from some initial digging—but this leads me to believe that there still would exist multiple sections in the code base that would likely flag races if our test suite were more comprehensive and parallel.
All this is my very long-winded way of asking the question: Is thread-safety a goal for this as a CLI library? If so, I'd like to see the underlying problems addressed, and we should be actively working toward complete thread safety both from a code and a test perspective. If not—and I think "not" is a reasonable enough direction for this type of tool—I would rather avoid the additional complexity of this kind of check.
CC @urfave/cli (I have no idea if tagging the team like this works, but here goes)

@rliebz That's a completely fair point, and it's part of why I opened the original issue:

I found it surprising that a shared slice of commands causes data races when starting a new app.
I realize this scenario is essentially non-existent in the real world, but this does crop up when running a suite of tests in parallel.
I'm curious if this [data race] is an expected result?

If the data race is an expected result, then we can close. Unfortunately, that'll mean I'm forced to restructure my program and tests, or I will need to choose a different framework that supports this to move forward.
Fortunately, by creating a shallow copy of the commands during boot we leave the original command slice untouched. If every test uses a distinct App instance, then we can prevent data races in concurrent calls to App.Run.
It seems useful to me to say "an App is not thread-safe, but 2 Apps with the same input commands can be run concurrently."

@rliebz Bump! 🙂 Where do we go from here?

Apologies for the delays in getting back here. I really appreciate the time you've taken for this contribution and your patience. I commented on the original issue here, but I'd like to address this change directly as well.
My challenge with merging this in is that it makes the code more complex and through the addition of the -race flag, implies that it is safe for concurrency. It is still not safe for concurrent use—we just don't have any tests that would catch the other races. The three places I flagged in my previous comment are still not addressed by this change.
While there might be a way to make *cli.Context safe for use in multiple  instances of *cli.App, it's such a niche use case with a straightforward enough way to handle outside the library that I'm not sure the extra complexity is warranted, and I'm not convinced that this PR actually brings us closer to that solution. A shallow copy does not give unique references to any fields that happen to be reference types, including slices, pointers, and interfaces, all of which are used as fields of *cli.Command.
My personal recommendation would be to improve the documentation to make clear that a single *cli.Command should not be passed to multiple instances of *cli.App. If you're interested in pushing for something that is safe for concurrent use, there might be a path forward, but it's not something I currently have the time to really help figure out.
Again, thank you for your time on this. We're all just trying to get this library to a better place 🙂

@urfave/cli I'd like to close this issue. Any thoughts ?


Add sunodo-cli at 0.10.3

Description of changes
This adds a new package sunodo-cli.
https://github.com/sunodo/sunodo
The monorepo contains many node packages, the one built and packaged for this PR is the one in apps/cli (that uses some of the other packages in the repo).
Probably it's the only one worth to be packaged as standalone application.
I also tried to package it with mkYarnPackage but it ended up being quite difficult because the monorepo has only one lockfile for several package.json and contains node modules that require different versions of the same package. Probably it would need separate derivations for each node module in the monorepo.
Also, when building with hardhat, one of the needed packages to build some modules, it tries to fetch stuff from the internet. This behavior is patchable adding a subtask to the hardhat.config.ts files but obviously it requires a patch (that has to maintained).
The final solution was simply adding it to the nodeModules packages set and creating an alias.
For my very limited use it seems to work.

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

2 packages marked as broken and skipped:

emanote
emanote.data

10 packages failed to build:

discourse
discourseAllPlugins
geph.gui
joplin
lv_img_conv
openvscode-server
pot
sharedown
teck-programmer
vscode-extensions.rust-lang.rust-analyzer

53 packages built:

akkoma-frontends.akkoma-fe
balena-cli
bat-extras.prettybat
commitlint
concurrently
create-cycle-app
documenso
elmPackages.elm-doc-preview
elmPackages.elm-pages
elmPackages.elm-spa
emacsPackages.lsp-bridge
emojione
fast-cli
graphite-cli
homepage-dashboard
image_optim
lemmy-ui
lessc
lighthouse
matrix-appservice-irc
mx-puppet-discord
nodehun
pgrok
pgrok.server
pm2
postcss-cli
pscid
pulp
pyright
styx
styx.lib
styx.themes
sunodo-cli
theLoungePlugins.plugins.closepms
theLoungePlugins.plugins.giphy
theLoungePlugins.themes.flat-blue
theLoungePlugins.themes.flat-dark
uppy-companion
vesktop
vimPlugins.coc-clangd
vimPlugins.coc-go
vimPlugins.coc-markdownlint
vimPlugins.coc-metals
vimPlugins.coc-pyright
vimPlugins.coc-sh
vimPlugins.coc-stylelint
vimPlugins.coc-tslint
vimPlugins.coc-tsserver
vimPlugins.coc-vetur
vscode-extensions.ms-python.vscode-pylance
wasm-text-gen
wrangler
youtube-music

It breaks too many packages, I will try to manually edit node-packages.nix leaving only additions and not removals/edits.

Please use buildNpmPackage

@SuperSandro2000 I've tried using mkYarnPackage but I had problems since some dependencies are subdirectories (I had to set the packageResolutions argument) and cannot be built in the derivation returned by mkYarnPackage because building them requires internet. They could be patched but I fear it could be quite time consuming.
Also IIRC I was getting a resolution error by yarn because some of these dependencies in subdirectories shared the different versions of the same dependencies, and this was somehow not contemplated by mkYarnPackage (to my understanding). A solution could be to package these dependencies as separate mkYarnPackage/buildNpmPackage derivations probably.
To my understanding nodeEnv.buildNodePackage (automatically added to node-packages.nix) simply works because it doesn't build anything and simply fetches the already built npm package from npm. What if manually use buildNodePackage instead?
I really don't have time patch (and maintain patches) of everything.

I mean buildNpmPackage which is different from buildNodePackage and has nothing to do with nodePackages or mkYarnPackage.

buildNpmPackage would try to execute commands that require internet access to build the local dependencies. I cannot see how it would solve my problems. Am I missing something?

You probably need to patch that but adding anything new to nodePackages is also not a good solution.


Getting Started and Configuring Parse Server

Adds an opinionated Getting Started guide that encourages running Parse Server locally as the first step.
The different sections on Configuring Parse Server have been consolidated as well.
There is still quite a bit of overlap between the README and the Parse Server Guide in the wiki. I plan to address this in a future PR.

Awesome! Glad to see our getting started flow getting back to excellent.


sentinels equipped with splash potions throw potions with no effect

ALSO - tipped arrows placed in a sentinels inventory using /npc inventory are ignored when a sentinel is equipped with a bow. I will be troubleshooting this a bit today but I'm not surprised about the splash potions not working since spigot has problems with the new data values that potions use- actually I dont know where the problem is

Can't replicate this - my Sentinel NPCs are successfully able to take out a cow target with some Instant Damage splash potions.

Also, an arrow tipped with Instant Damage II played the correct effect and rapidly slaughtered cows.


Inactive tabs are hard to distinguish since "framed" update

Since the 0.6.3 "framed" update, inactive tabs no longer have a bounding box. I'm finding it very difficult to distinguish between tabs with many files open (reality). With a hard divider between tabs the padding could be decreased significantly. It's just not usable as-is.
Along the same lines, a minimum tab width would be welcome with an overflow mechanism.

The larger gap is because now the close icon is next to the text label and not on top of it. That would need to be changed back again to allow less padding, even with a divider.

Along the same lines, a minimum tab width would be welcome with an overflow mechanism.

I started to experiment on the sm-tabs branch. How do you like having the text labels never be truncated unless it exceeds the max width.
Before:

After:

So there are less tabs shown.. but

you can actually read them
there is no expanding when a tab is active. Resulting in no changing in positions which feels much calmer.
The close button is always next to the text label. Making it easier to guess when hovering.

ps. There is also an issue #13 about adding a "compact mode", which could be the same as pre 0.6.0.

1.1 — If the close icon padding is permanent, then I might always show the icon (not just for active tabs) so I can close tabs without switching to them. The spacing around the active tab is awkward in the before/after examples above. With the icon permanent it'd be tight feeling, but I think that wouldn't be an issue if inactive tabs had a dark 1px border delineating them.
1.2 — I think I prefer the close icon overlapping the content in the case when there are tons of tabs and names are being truncated. This leaves more room in the tab to read the title. Chrome does this well.
2.1 — It's definitely nicer not having the tabs change width on selection/deselection. A max-width for truncation would help.
2.2 — I wouldn't want all tabs over 20 chars being truncated though — only if there are too many tabs open. Otherwise that makes it harder to distinguish, ex: comparing the same files from two themes.
2.3 — Caveat: the issue with variable width tabs is closing them in quick succession jostles the close button's position. Again, I like how Chrome does this. When hovering the tab bar tabs don't resize, so you can click-click-click without moving the pointer to close multiple tabs. That only works if they're all the same width.
3.0 — Unrelated: highlight the active tab on the left side is brain-twitchy for me. It takes me a second to associate it to the right tab. I think I'd prefer that highlight be border-top, so it's a stronger visual signal and it anchors the size of the tab not the height of the tab bar. Especially if the tabs are variable width this would help register where the close icon will be quicker.

1.1 and 1.2: I totally agree with this, would be amazing @simurai if you could do that :) Would save a lot of space :)
2.3 https://github.com/atom/tabs/issues/129 I started experimenting with this, but my problem was that the active tab currently has another size than the other tabs :/ Would be great when they could have the same size, then we could implement the "delaying tab size" feature :)

Ok, I followed these points:
1.2 Close icon covers the text if it gets narrow.
2.1 Active tab stays the same width, so no shifting around.
2.2 Increased the max width a bit. Since the width has to stay the same, it can't just be wider only for tabs with long text labels.
2.3 All tabs have the same width. Should allow the "delaying tab size" feature for quick succession closing.
2.4 Also removed the ellipsis and instead the text just fades away. This lets you see an extra 2-3 characters.
2.5 Close icon is hidden also on the active tab and only shown on hover. Otherwise it would cover too much text. Making the active tab wider would conflict with some points above.
2.6 (half of 1.1) Added divider lines between inactive tabs. Making it easier to guess where the close icon is.

3.0 Tried that out in the past. When there was no padding at the top, it clashed a bit with the bright title bar. With padding would work better. But it would make it a bit stronger since it stretches based on the tab width. Haven't changed it on the sm-tabs branch, but would look like this:

imho I've enjoyed the left highlight on the active tab.  personally, i don't like it as much on the top.

Great work @simurai seems a lot better than the current version :) I really like how the text of the tab is fading away.. Just amazing :boom: The border on the left is definitely better!

I just tried the branch out and it looks extremely good! Maybe the maximum tab size is a little bit to big? With a lot of tabs open its extremly clear now. And finally i see what the other tabs (Its more important to see the other tabs than the active tab, since i know where i currently am)

Maybe the maximum tab size is a little bit to big?

Would be cool if the max width is larger but only if there is at least one large text label. That's probably what @brendanfalkowski meant in 2.2. Don't think it's possible with CSS alone and would need some JS to measure the length and change the max width.

Ok i understand! Lets leave it like this, its just amazing to work with this tabs now :)

Do you like that there is no close icon at all? I don't really miss it since I know it's there on hover, but could be controversial.
Also, personally I would increase the padding on the tabs a bit more. :innocent: Just to make it look less squeezed. Maybe there is no way around adding #40. People just have different priorities.

I don't see the close button as a problem.. Maybe we could show the close button on big tabs and hide it when they are smaller (again stolen from Chrome). But... Is that possible just with CSS?

Hmm.. the only way I can think of is having the close button relative as inline-block so that it wrapps (and is hidden) at some point.. then on hover position it absolute.

Looking really good, the fixed tab widths and separators reduce the need for permanent close icons (for me). Agree that I'd much rather see more filename than the icon. Fade vs ellipsis was a good change to help that.
One man's opinion, but I really like the top highlight for active tabs (vs left side). Just more obvious that way.
For me, the case where max-width matters is opening two files with the same name, but from a different path. Because the path string starts from the tree root (and is appended not prepended) I always need the tooltip to distinguish them anyway (because they're deep paths) so it's not much of an issue.

I'd be more interested in seeing a min-width for tabs though, and have the tab bar use overflow-x:scroll like Firefox. I can see that being disliked, but I think readable tabs is better than crunching to 3 chars.

Made a PR #59 with the above.

I'd be more interested in seeing a min-width for tabs though, and have the tab bar use overflow-x:scroll like Firefox.

There is overflow-x scrolling.

Actually should already work now. The min-width could be larger, but maybe that's something for styles.less.
There are also packages like https://atom.io/packages/tidy-tabs or https://atom.io/packages/zentabs that start closing old tabs so it never gets too crowded. Or https://atom.io/packages/sublime-tabs that only keeps a tab if you edit it.

@simurai  this is already working in atom 198 ? How can i enable it? just overflow-x : scroll ?

Its already enabled out-of-the-box. But you need a mouse which is able to scroll horizontally (mine is). Maybe we could translate the y mouse scroll to an x mouse scroll.

i can scroll tabs with my mouse (Logitech : G602) in chrome , or do you mean something that have two scroll wheels?

Oh i see , something that allow to scroll after click a scroll button? not mouse wheel?

No i mean something like this:

Currently its not possible. Maybe open an issue to request that feature. Would be interesting of course. I don't know if it should be theme-related or atom-tabs related.

Thanks , now I can do it using Shift + Mouse wheel as advised by @izuzak
https://github.com/atom/tabs/issues/137#issuecomment-100877401
I also advised to add "tab scroll" buttons at both ends of tab bar as advised here , may be you will interest in it. https://github.com/atom/tabs/issues/136
that way , possible to work with devices without mousewheels too.


Annotate zerver.views.webhooks.freshdesk

Also change the comments into docstrings

Automated message from Dropbox CLA bot
@dawran6, it looks like you've already signed the Dropbox CLA. Thanks!

Posted a couple comments, otherwise looks great!

Merged, thanks @dawran6!


unknown time zone

Summary
v3.0.0-rc2 regresssion from this https://github.com/argoproj/argo-workflows/pull/5057 causing LoadLocation to not find timezone by name.
This issue is not present in v3.0.0-rc1
Diagnostics
EKS 1.17
v3.0.0-rc2
time="2021-02-23T18:17:17.550Z" level=error msg="could not run outstanding Workflow" cronWorkflow=prod/foobar error="invalid timezone 'America/Los_Angeles': unknown time zone America/Los_Angeles"

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

that's odd:
COPY --from=workflow-controller-build /usr/share/zoneinfo /usr/share/

I wonder what happenid.

yeah, seems like that should work...  It's possible that is not the cause, but it is quite suspect since it's behaving like the zoneinfo is missing.

https://stackoverflow.com/questions/59094236/error-unknown-time-zone-america-los-angeles-in-time-loadlocation


x/pkgsite: reports vulnerabilities incorrectly for default branch

What is the URL of the page with the issue?
https://pkg.go.dev/net/http@master
Screenshot

What did you do?
Go to the link above
What did you expect to see?
Not so many vulnerabilities reported. I believe that most(all?) of these are actually fixed on HEAD. Maybe vulns should only be reported on released versions?
What did you see instead?
A lot of vulnerabilities listed.

Some of these are maybe valid and just have not had a release yet, I am not sure. But there are for sure some really old ones listed too, example: https://pkg.go.dev/vuln/GO-2022-0761

CC @julieqiu @jamalc

I think the solution here is to not show vuln information on stdlib pages at master. These vulns show up because the tag master translates to a psuedoversion (e.g., v0.0.0-20230104211531-bae7d772e800) which is within the vulnerable range according to golang.org/x/vuln/osv.Affects.AffectsSemver.
cc @golang/security


Revise mapping structure

Following discussion with @larrybabb, we will create the MappableConcept, which will reuse the ConceptMapping, containing:

label
mappings
id

This will help us address issues identified in #78.

We will also add an Element class that this inherits from with the id attribute.


Any plan to NodeJS support?

Hi Daniel, supporting NodeJS is exactly the next step in our plan, our drivers will cover most of popular language in market and will be released soon. Stay tuned!


Add scroll for filters in results page

Fixes
Fixes https://github.com/creativecommons/cccatalog-frontend/issues/689 by @ManishBasavalinga and https://github.com/creativecommons/cccatalog-frontend/issues/904 by @abhisheknaiidu
Description
Add scrollbar to the filters in results page.
Technical details
Previously two pr's where open to tackle this problem https://github.com/creativecommons/cccatalog-frontend/pull/903 and https://github.com/creativecommons/cccatalog-frontend/pull/763
however the implementation wasn't complete so this PR is to address solving the problems described.

Works great on mobile device by adding a media query to only allow scrolling on desktops
Doesn't use a package to add scrollable bar to filters list

Tests
Currently tested manually :

Desktop Macbook air 13 inch
For mobile device to check responsiveness.

Screenshots

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@Fahme can you fix the conflicts here?

@brenoferreira done

Thanks for the feedback, I didn't think much on the corner cases as i just
went with what was missing in previous PR's,  if you have a better idea on
fixing it i think it's good to close this pr 😊
On Tue, May 12, 2020, 21:08 Breno Ferreira<EMAIL_ADDRESS>wrote:

@brenoferreira requested changes on this pull request.
This solution isn't working well
[image: Screen Shot 2020-05-12 at 15 06 21]
https://user-images.githubusercontent.com/707019/81729371-3aa7c200-9462-11ea-8850-25ac3929700e.png
I think you need to add independent scrolling for both columns here.
Otherwise we see the behaviour above, where some of the content gets
clipped with still enough space below, when the user scrolls down on the
image grid.
In src/components/Filters/SearchGridFilter.vue
https://github.com/creativecommons/cccatalog-frontend/pull/917#discussion_r423931869
:

@@ -132,6 +132,10 @@ export default {
color: #333333;
}
&__visible {

@include desktop {

You're missing @import "node_modules/bulma/sass/utilities/_all"; in the
beginning of the styles
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/creativecommons/cccatalog-frontend/pull/917#pullrequestreview-410295153,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACSBNOQOWXQLUIO65ATTXM3RRGGDNANCNFSM4M25MM4Q
.


Wow this is amazing!

This is what I was looking for! I do need some help with using it though. Is there a method to simulate just one attack? For example, a Gorgon uses its 'gore' attack against an elephant on one turn. Which method can I use for that?
Also, do you support spells? Like if a Night Hag uses an 'enfeebling ray' against a commoner, which method would I use for that?
Great work BTW!

Thanks!
Okay. To answer your questions, let's declare the creatures.
donald=DnD.Creature("Donald",base="elephant",alignment='Murica')
xi=DnD.Creature("Jinping",base="gorgon", alignment='中国')
theresa=DnD.Creature("Theresa",base="night hag", alignment='Engaland')

"Just one attack" by that do you mean change to secondary attack (I don't have my MM on me)?
The simulations runs n encounters via Encounter(<...>).go_to_war(n)_, which runs .battle() n times. The latter method iterates across the creatures running their Creature().act() method, which makes them decide whether to heal, dodge, attack, free themselves, buff, throw net etc. The attack is called .multiattack()
If you want to override the behaviour of say Donald the elephant to attack regardlessly and at random you can change the class's method act()
import random
from DnD import Encounter

def faux(self,*args,**kawrgs):
    print(self)
    pass

def trumpconomy(self,verbose=0, assess=0): # a partial copypaste of multiattack... see TrumpMods
    if not self.arena.find('alive enemy') and len(self.arena.find('alive ally')) == 1: #TrumpMod: Win when all bar one.
        raise Encounter.Victory()
    for i in range(len(self.attacks)):
        try:
            opponent = random.choice([other for other in self.arena.combattants if other is not self]) #TrumpMod kill all bar self.
        except IndexError:
            raise self.arena.Victory()
        if verbose:
            verbose.append(self.name + ' attacks ' + opponent.name + ' with ' + str(self.attacks[i]['name']))
        # This was the hit method. put here for now.
        self.attacks[i]['attack'].advantage = self.check_advantage(opponent)
        if self.attacks[i]['attack'].roll(verbose) >= opponent.ac:
            # self.attacks[i]['damage'].crit = self.attacks[i]['attack'].crit  #Pass the crit if present.
            h = self.attacks[i]['damage'].roll(verbose)
            opponent.take_damage(h, verbose)
            self.tally['damage'] += h
            self.tally['hits'] += 1
        else:
            self.tally['misses'] += 1

import types
# adding the unbound method as a bound method...
donald.act=types.MethodType(trumpconomy, donald)

#lets add an "ally"
rex=DnD.Creature("Rex",base="owlbear",alignment='Murica')

print(DnD.Encounter(donald,rex,xi).battle(verbose=1).masterlog)

In round one Donald attacks his ally Rex! Thus proving the behavior is altered!
Similarly spells. There is a possibility of using the a backdoor override of conc_fx with argument 'custom' during initialisation but that is less flexible.

ROFL!:
donald...alignment="Maurica"

I've been tweeking your code, you don't actually need to modify your act() method at all, I managed to do the above all with your  multiattack() method. You did mention net() method, I've tried using it but doesn't work. Could you give an example of how to use it?

I've been tweeking your code, you don't actually need to modify your act() method at all, I managed to do the above all with your multiattack() method. I am trying to create a custom creature, a marilith with only a "tail" attack and a net alternate attack, but I can't seem to do it. Could you tell me how?

I have patched a lot of your code, updated some rules and added some monsters. I also added an option to change the number of rounds that monsters battle. In fact, you don’t need to change multiattack() to do the above.
Since your code is difficult to read, all I need to know is if you have implemented condition damage? For instance if a poisonous snake bites an enemy there should be ongoing poison damage. What is the method? If you haven’t I’d be happy to implement it.

Great, I'll have a look at your branch when I get the chance. Thanks.
Yes, true that multiattack() is the main attack, but I spoke of change to act() was because it controls the behaviour (e.g. dodging if weakest etc), which is most likely what one would like altered.
I do not believe I ever did that case. Advantage and disadvantage are implemented though.


build.cache:true does not work together with build.extractCSS:true in static mode

Versions

nuxt: v2.14.0
node: v12.18.3

Steps to reproduce

Set cache:true in build configuration
Set extractCSS:true in build configuration
Run yarn nuxt generate
First time it will work and look as it should
Run yarn nuxt generate again; it will work fine, still.
Go back and update your nuxt.config.js file (e.g. add a new line), so full rebuild will be made
Run yarn nuxt generate
Visit your site. No stylesheets have been injected, and Times New Roman font everywhere.
Set cache:false in build configuration
Run yarn nuxt generate
Visit your site - it works fine

What is Expected?
Stylesheets should be loaded when extractCSS:true even though cache:true is also set
What is actually happening?
It works the first time, but when you rebuild (e.g. made a change to nuxt.config.js) it generates your site without any stylesheets at all.

This is same issue as #7446, along with #8048.

Still can reproduce

Hi, this issue is from https://github.com/nuxt/nuxt.js/pull/5004, the reason is that extract-css-chunks-webpack-plugin doesn't work well with thread-loader because thread-loader will create separate context which can't accessed by extract-css-chunks-webpack-plugin.
The issue has been a while, I'll look into in again to see if thread-loader has changed anything.

I just checked, looks thread-loader is still not working with extract-css-chunks-webpack-plugin and there has been an issue opened there https://github.com/webpack-contrib/thread-loader/issues/66, let's keep tracking that issue.

Yes, parallel and extractCSS does not work together, and it has already been addressed with linked PR. But is it really related with cache option?
If so, combination of parallel and extractCSS options can be disabled, just like #5004.

We are approaching the Nuxt 2 EOL date (June 30, 2024) - see this article for more information. This is advance warning that I'm going to close this issue then, as it's currently marked as a Nuxt 2 related bug.
If it's a critical or security issue, please do comment and let me know, in case it is possible to address it before the EOL date.
If it's a an issue you think is relevant for Nuxt 3, please feel free to open a fresh issue (or just comment here so I can update labels, etc.). 🙏
Thank you for your understanding and for your contribution to Nuxt. ❤

It's the day, at last. Nuxt 2 is now marked end-of-life.
My apologies we never got round to resolving this bug in the 2.x branch, but we do have to draw a line somewhere. Again, if you think this is still relevant for Nuxt going forward, please feel free to open a fresh issue (or just comment here so I can update labels, etc.). 🙏


Closes: #1

Fixed issue in validations.py that allowed usernames to start with forbidden characters.
Usernames can now only start with an alphabet.
Fixes #1

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

@googlebot I signed it!

@googlebot I signed it!


Local storage is blocked in iframes using cross domains sources.

When the "Block third party cookies" (default for incognito mode) is enabled in chrome, the local storage is not accessible from iframes that have a different domain source.
I think before trying to setItem or getItem, should be made a check if the localStorage is accessible at all and to have a fallback if not.
Version
@nuxtjs/color-mode: "^2.1.1",
nuxt: "^2.2.0",
Steps to reproduce

put the site that uses @nuxtjs/color-mode as an iframe source in a different site, the domains should be different.
open the site in an incognito window (In incognito chrome blocks third party cookies by default, in a normal chrome window you have to go to settings and check the option)

What is Expected?
I'm expecting it to not throw errors and to not block the entire app
What is actually happening?

Can be closed - resolved via https://github.com/nuxt-modules/color-mode/commit/e4e84fe11e1eec8946e157a4a659ed7c911fe3f1

Is this actually resolved? I was just trying my blog's RSS feed in Feedbro and the pages all immediately crash with "it.getColorScheme is not a function". The logs show:

Interesting! Needs a better fix then indeed


Failing unit tests

With current master I have the following unit test failure:
    1. /home/jvermillar/sandbox/wakaama/tests/tlv_json_lwm2m_data_test.c:113  - CU_ASSERT_EQUAL(original_length,length)
    2. /home/jvermillar/sandbox/wakaama/tests/tlv_json_lwm2m_data_test.c:122  - CU_FALSE

JSON unit tests need improvements. The returned buffer is compared to a hard-coded one. It should be parsed instead.

I disagree. Hardcoded is suitable here, as there is no 3rd party json library pulled in and everything is in a controlled condition. But the unit tests need to be adapted if the core is changed, of course.

See #324
JSON tests 9 and 10 are still failing.
9 because it contains Object Links which are not handled by the JSON encoder.
10 because a float value exceeds the encoder limits.

Fixed by #324


How to initialize multiple AmazonSQSAsync with different credentials?

Describe the bug
I have a service which has to receive messages from one queue and write some messages to another queue.
After configuring sqs-related stuff and running the service I get the following error:
Field awsCredentialsProvider in org.springframework.cloud.aws.messaging.config.annotation.SqsClientConfiguration required a single bean, but 2 were found:
- managementAwsCredentialsProvider: defined by method 'managementAwsCredentialsProvider' in class path resource [...managementAwsConfiguration.class]
- changeAwsCredentialsProvider: defined by method 'changeAwsCredentialsProvider' in class path resource [.../changeAwsConfiguration.class]

Spring cloud version:
Spring-cloud 2.1.3.RELEASE

Sample
First config which receives messages:
@Configuration
@EnableSqs
public class ManagementAwsConfiguration {

    @Bean
    public SQSConnectionFactory sqsConnectionFactory(@Qualifier("management") AmazonSQSAsync amazonSQS) {
        return new SQSConnectionFactory(new ProviderConfiguration(), amazonSQS);
    }

    @Bean
    public QueueMessagingTemplate queueMessagingTemplate(@Qualifier("management") AmazonSQSAsync amazonSQSAsync) {
        return new QueueMessagingTemplate(amazonSQSAsync);
    }

    @Bean
    @Qualifier("management")
    public AWSCredentialsProvider managementAwsCredentialsProvider(AWSCredentialsStore awsCredentialsStore) {
        return awsCredentialsStore.getAWSCredentialsProvider("management");
    }

    @Bean
    @Qualifier("management")
    public AmazonSQSAsync managementAmazonSQS(@Qualifier("management") AWSCredentialsProvider awsCredentialsProvider) {
        return AmazonSQSAsyncClientBuilder.standard()
                .withCredentials(awsCredentialsProvider)
                .withRegion(Regions.EU_CENTRAL_1)
                .build();
    }

}

second config which sends messages:
@Configuration
@EnableSqs
public class ChangeAwsConfiguration {

    @Bean
    @Qualifier("change")
    public AWSCredentialsProvider changeAwsCredentialsProvider(AWSCredentialsStore awsCredentialsStore) {
        return awsCredentialsStore.getAWSCredentialsProvider("change");
    }

    @Bean
    @Qualifier("change")
    public AmazonSQS futureStockChangeAmazonSQS(@Qualifier("change") AWSCredentialsProvider awsCredentialsProvider) {
        return AmazonSQSClient.builder()
                .withCredentials(awsCredentialsProvider)
                .withRegion(Regions.EU_CENTRAL_1)
                .build();
    }

}

By looking at SqsClientConfiguration in spring sources it doesn't look like there is a straightforward way to configure two SQS instances, is there a way to do that?

I'm on the lookout for the same thing.
I don't think there is a straightforward way, no: https://stackoverflow.com/questions/58064573/multiple-aws-sqs-queues-with-different-credentials-in-spring-boot

There is currently no out-of-the-box support for multiple SQS listener each with separate credentials. Lets track this issue here: https://github.com/awspring/spring-cloud-aws/issues/112


⚠ Bofrost.be has degraded performance

In 278d0d2, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1306 ms

Resolved: Bofrost.be performance has improved in f760cf0 after 6 minutes.


UseProxyToSpaDevelopmentServer doesn't auto-refresh browser using an Aurelia ClientApp with HMR

I created a project using the latest dotnet new angular template and replaced the ClientApp folder with an Aurelia app created using the Aurelia CLI.
The command
au new
was ran inside the project folder to create a new ClientApp folder, the following options were used:
Name: ClientApp
Platform: Web
Bundler: Webpack
Startup.cs was modified to use spa.UseProxyToSpaDevelopmentServer:
//spa.UseAngularCliServer(npmScript: "start");
spa.UseProxyToSpaDevelopmentServer(baseUri: "http://localhost:8080");
When i run both
au run --watch
and
dotnet run
it works fine, i can edit a file in the Aurelia app and the browser refreshes automatically using a full page refresh.
However if i run
au run --watch --hmr
instead, the auto refresh no longer works, but still works if i visit the Aurelia app directly via http://localhost:8080 (still refreshes the whole page as there is only 1 module in the au new template) it just no longer works from the dotnet host on http://localhost:5000 or https://localhost:5001
I then checked to see if "Hot Module Replacement" worked in the Angular app which was created by the template by enabling it using the following guide:
https://github.com/angular/angular-cli/wiki/1-x-stories-configure-hmr
and edited startup.cs to use the angular url:
spa.UseProxyToSpaDevelopmentServer(baseUri: "http://localhost:4200");
It works fine, i can edit a file in the angular project and see that the page updates without a full page refresh.
I have not read enough about HMR to understand how it does what it does, but as both ClientApps are using Webpack hmr, i would have expected it to also work in Aurelia through the spa proxy.

Just an update on this, checking the console log, this error can be seen when using hmr with my Aurelia ClientApp as described above:

GET https://localhost:8080/sockjs-node/info?t=1540995938707 net::ERR_CONNECTION_CLOSED

The above request should be going via the dotnet host rather than directly to the webpack server.
I have now compared this to what happens in the Angular Spa template, it seems when using Angular as the ClientApp, the request is correctly sent to:

https://localhost:5001/sockjs-node/info?t=1540996146714

For some reason, in the Aurelia Template i created the request is being sent directly to the webpack dev server rather than to the dotnet host to be proxied through to the dev server. This in theory should still work, but fails because the dev server is not running on https, only the dotnet host.
So it works in the Angular Spa template, I am not sure what magic is happening behind the scenes in the Angular CLI when i run npm run hmr which in turn runs the CLI with ng serve --hmr -e=hmr
There must be some magic because if i run ng eject which "ejects" us from the friendly and magic world of the Angular CLI and generates the required webpack config, it breaks.
Once ejected from the CLI, running npm run hmr which in now runs webpack directly with webpack-dev-server --port=4200 --hot  (after editing the package.json) we see the following error:

GET https://localhost:4200/sockjs-node/info?t=1540997851706 net::ERR_CONNECTION_CLOSED

That is the same issue as what i am seeing with the Aurelia ClientApp. Ejecting from the CLI is a likely scenario for many Angular projects, at which point HMR will no longer work it seems. To complicate things further, when accessing Angular directly via "http://localhost:4200/" after ejecting, although there are no errors when requesting "/sockjs-node/", HMR is now not working properly as the whole page now refreshes when doing the same edit that previously caused a partial page refresh.
The Aurelia ClientApp always works with HMR when accessing it directly.
I haven't tried different versions of webpack-dev-server as yet, the versions are different between the Angular and Aurelia ClientApp's but the behaviour is the same once ejected from the Angular CLI.
Does anyone happen to know what magic is happening in the Angular CLI that makes this work?

Ok, i tried to figure what what was causing the websocket redirect to port 8080 from 5001 and spotted this:
Inside "ClientApp/aurelia_project/tasks/run.ts" there is a line which updates a copy of the webpack config used when running the app:
config.entry.app.unshift(`webpack-dev-server/client?http://${opts.host}:${opts.port}/`, 'webpack/hot/dev-server');

changing this to point to https://${opts.host}:5001/ solves the problem, that is not ideal because it will break when accessing the dev server directly.
Luckily the address is not required, it also works fine using this:
config.entry.app.unshift(`webpack-dev-server/client`, 'webpack/hot/dev-server');

Solved! my Aurelia ClientApp now works correctly with HMR either through https://localhost:5001 or http://localhost:8080 or any other address it is made to run on.

Update: If you try to run Aurelia from a subpath for example by setting const baseUrl = '/app/'; in the webpack config, which sets output.publicPath you need to use this format in run.ts to set the client path to '/'
config.entry.app.unshift(`webpack-dev-server/client?/`, 'webpack/hot/dev-server');

It seems that the path used by the dev server client to access the websocket server is inherited from the publicPath setting or derived some other variable generated from that . Currently the dev server websocket is not able to listen on a subpath until this is done:
https://github.com/webpack/webpack-dev-server/pull/1553
so the client must be forced back to using '/' as the path


Postgres 9.6 is not buildable from master

When I try to build from master in directory 9.6 I stops with
E: Version '9.6.4-1.pgdg80+1' for 'postgresql-9.6' was not found
E: Version '9.6.4-1.pgdg80+1' for 'postgresql-contrib-9.6' was not found
The command '/bin/sh -c apt-get update  && apt-get install -y postgresql-common         && sed -ri 's/#(create_main_cluster) .*$/\1 = false/' /etc/postgresql-common/createcluster.conf         && apt-get install -y           postgresql-$PG_MAJOR=$PG_VERSION                postgresql-contrib-$PG_MAJOR=$PG_VERSION        && rm -rf /var/lib/apt/lists/*' returned a non-zero code: 100

It seems like version 9.6.4 is not yet available for debian jessie: https://packages.debian.org/jessie-backports/postgresql-9.6, only 9.6.3
Maybe related mail discussion: https://lists.debian.org/debian-user/2017/08/msg01324.html

Having the same issue.

It is probably related to http://apt.postgresql.org/pub/repos/apt/dists/jessie-pgdg/9.6/binary-amd64/Packages being empty. Seems like someone needs to contact postgres.

I have notified<EMAIL_ADDRESS>about the problem.

The new package names:
[pgsql-pkg-debian] postgresql-9.6 updated to version 9.6.5-1.pgdg+1
apt.postgresql.org repository<EMAIL_ADDRESS>2017. aug 31. 12:56
to: PostgreSQL in Debian<EMAIL_ADDRESS>The package postgresql-9.6 was updated on apt.postgresql.org:

New version 9.6.5-1.pgdg+1:

postgresql-9.6             |  9.6.5-1.pgdg+1           |  sid-pgdg      |  amd64,  i386,  ppc64el,  source
postgresql-9.6             |  9.6.5-1.pgdg90+1         |  stretch-pgdg  |  amd64,  i386,  ppc64el,  source
postgresql-9.6             |  9.6.5-1.pgdg80+1         |  jessie-pgdg   |  amd64,  i386,  ppc64el,  source
postgresql-9.6             |  9.6.5-1.pgdg70+1         |  wheezy-pgdg   |  amd64,  i386,  source
postgresql-9.6             |  9.6.5-1.pgdg17.04+1      |  zesty-pgdg    |  amd64,  i386,  ppc64el,  source
postgresql-9.6             |  9.6.5-1.pgdg16.04+1      |  xenial-pgdg   |  amd64,  i386,  ppc64el,  source
postgresql-9.6             |  9.6~rc1-1.pgdg15.10+1    |  wily-pgdg     |  amd64,  i386,  source
postgresql-9.6             |  9.6.5-1.pgdg14.04+1      |  trusty-pgdg   |  amd64,  i386,  ppc64el,  source
postgresql-9.6             |  9.6.3-1.pgdg12.4+1       |  precise-pgdg  |  amd64,  i386,  source
....

Thank you, working on my end.

:+1:
d1725ff
43f393c
b1fc9b6
b2649f9
ee1bb1f
7b518a1
c6d7684
9e862fa
8904bff
b3cb990
b8bc661
96498cc
I'm going to close, since the issue is fixed.  I would be curious though why we've got so many users building from source -- anyone willing to share? :smile:

@jarl-dk: Some Packages files being empty is entirely normal: http://apt.postgresql.org/pub/repos/apt/pool/README.components.txt

This solves the problem.


[camera] [ios] Preview freezes when using torch with ultrawide or telephoto camera

Hi.  My app requires access to the torch and a preview from the telephoto lens if using a 3-lens iPhone (e.g. iPhone 12).   However, for 2 of the four camera on this kind of phone the preview freezes if the torch is active.
I have repeated the behaviour with a few different torch plugins so I think this is a camera issue rather than a torch issue.
Steps to Reproduce

if I select cameras[0] or cameras[1] (front-facing) I can generate a live preview with torch active
If I select cameras [2] (telephoto) or [3] (ultrawide), the preview freezes while the torch is active and unfreezes when I turn the torch off. There are no error messages reported, so there is nothing obvious to 'fix' from my end.

Execute flutter run on the code sample
Select a camera by changing the index value in onNewCameraSelected in initState method.
Toggle the torch using the text button

Expected results: Preview should not freeze for any cameras
Actual results: Preview freezes for cameras 2 and 3 but does not freeze for camera 0 and 1
code snippet
import 'package:camera/camera.dart';
import 'package:flutter/material.dart';
import 'package:torch_controller/torch_controller.dart';

List<CameraDescription> cameras = [];

void main() async {
  TorchController().initialize();
  WidgetsFlutterBinding.ensureInitialized();
  cameras = await availableCameras();

  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      debugShowCheckedModeBanner: false,
      title: 'Flutter Demo',
      theme: ThemeData(
        // enable Material 3
        useMaterial3: true,
        colorScheme: const ColorScheme.light(
          primary: Colors.indigo,
          outline: Colors.indigo,
        ),
      ),
      home: const CameraTest(),
    );
  }
}

class CameraTest extends StatefulWidget {
  const CameraTest({Key? key}) : super(key: key);

  @override
  _CameraTestState createState() => _CameraTestState();
}

class _CameraTestState extends State<CameraTest> with WidgetsBindingObserver {
  double? previewHt;
  CameraController? controller;
  bool _isCameraInitialized = false;
  TorchController? tc;

  void onNewCameraSelected(CameraDescription cameraDescription) async {
    final previousCameraController = controller;
    // Instantiating the camera controller
    final CameraController cameraController = CameraController(
      cameraDescription,
      ResolutionPreset.high,
      imageFormatGroup: ImageFormatGroup.bgra8888,
    );

    // Dispose the previous controller
    await previousCameraController?.dispose();

    // Replace with the new controller
    if (mounted) {
      setState(() {
        controller = cameraController;
      });
    }

    // Update UI if controller updated
    cameraController.addListener(() {
      if (mounted) setState(() {});
    });

    // Initialize controller
    try {
      await cameraController.initialize();
    } on CameraException catch (e) {
      print('Error initializing camera: $e');
    }
    // Update the Boolean
    if (mounted) {
      setState(() {
        _isCameraInitialized = controller!.value.isInitialized;
      });
    }
  }

  @override
  void initState() {
    onNewCameraSelected(cameras[2]); // < change to 0 or 1 and preview is fine
    tc = TorchController();
    super.initState();
  }

  @override
  void dispose() {
    controller?.dispose();
    super.dispose();
  }

  @override
  void didChangeAppLifecycleState(AppLifecycleState state) {
    final CameraController? cameraController = controller;

    // App state changed before we got the chance to initialize.
    if (cameraController == null || !cameraController.value.isInitialized) {
      return;
    }

    if (state == AppLifecycleState.inactive) {
      // Free up memory when camera not active
      cameraController.dispose();
    } else if (state == AppLifecycleState.resumed) {
      // Reinitialize the camera with same properties
      onNewCameraSelected(cameraController.description);
    }
  }

  @override
  Widget build(BuildContext context) {
    previewHt = MediaQuery.of(context).size.height*.7;

    return Scaffold(
        body: Column(
          children: [
            SizedBox(
              height: previewHt,
              child: _isCameraInitialized
                  ? AspectRatio(
                aspectRatio: 1 / controller!.value.aspectRatio,
                child: controller!.buildPreview(),
              )
                  : Container(),
            ),
            Center(
              child: TextButton(
                child: const Text('Toggle Torch',
                  style: TextStyle(fontSize: 20),),
              onPressed: ()=>tc!.toggle(),),
            )
          ],
        )
    );
  }
}

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 3.1.0-0.0.pre.1507, on macOS 12.2.1 21D62 darwin-arm, locale en-GB)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] VS Code (version 1.71.2)
[✓] Connected device (3 available)
[✓] HTTP Host Availability

• No issues found!

Hi @Strain-solutions, I do not have an iOS device with an ultra-wide camera as such I cannot verify this locally. Can you share a recording of the phenomena so it's easier for us to visualise the issue?
Thank you

HI sorry for the delay.  I was away from my desk over the weekend.  Ive attached a screen recording using the above code and choosing camera [3].  You can see from the recording that the preview initially works fine.  At around 5 sec in the 'toggle torch" button is pressed an the preview freezes as the torch comes on.  A few seconds later "toggle torch" is pressed again, the torch goes off and the preview resumes.
If you need more information or a longer video please let me know.
https://user-images.githubusercontent.com/58543579/198937983-fdc5bbcc-3a80-457e-ac80-fb1c6c7d2d23.mp4

HI, sorry for the delay.  I was away from my desk over the weekend.  I've attached a screen recording using the above code and choosing camera [3].
You can see from the recording that the preview initially works fine.
At around 5 sec the 'toggle torch" button is pressed, and the preview freezes as the torch comes on (to be clear, I am still moving the phone from side to side at this point).
A few seconds later "toggle torch" is pressed again, the torch goes off, and the preview resumes.
If you need more information or a longer video, please let me know.
https://user-images.githubusercontent.com/58543579/198938916-311e4c15-d504-49bc-a557-3c1e2f83800c.mp4

Thanks for the video.
Labeling this for further investigation as I do not have the device required to reproduce this issue locally.

This could be a limitation on iOS - when I launch the Apple's camera app, the torch option becomes unavailable. So you may want to do the same and disable the torch when previewing the video.

Thanks, @hellohuanlin. I don't think that's it (also, I specifically need both torch and preview together).  Anyway, I think this is some limitation with how the camera package interacts with other packages (or perhaps more likely how they interact with the camera plugin); I'll try to explain for other poor souls who may find this thread, but the long and short of it is you can probably close this issue.
This is an old project I've come back to, and at the time I initially wrote the code, the camera controller in the camera package did not have a controller.setFlashMode(Torch) option (which it does now), and so I used a different torch package.  If you look at the code above, I use the torch controller package.  That package and a couple of others I have tried give the behaviour outlined in this thread (the torch turns off when the preview is activated).  However, if I use the controller.setFlashMode(Torch), everything works as required
TLDR: activate the torch using the camera package controller and not another package.
I've not closed it from my end in case you need to look any further into how the different packages interact, but Im happy for this to be closed.
This is actually my first bug report to the flutter team so thanks for taking it seriously!


Is flow.js (the compiled-to-js version of Flow) published anywhere?

I was really happy to discover that Flow is runnable in the browser with the flow.js bin a la https://flow.org/try/.
I haven't been able to find it published anywhere (unless you count https://flow.org/static/v0.47.0/flow.js) for easy external use though. Am I not looking hard enough or is it really not on npm?
Followup question if the answer is "no": are there any plans to publish flow.js? I think it would be a nice additional offering, for very low extra effort!

yep, https://www.npmjs.com/package/flow-parser!

@mroch I get that flow-parser comes with the parser, but I was under the impression it did not include the type checker. Is there an option to turn on type checking?


Website common names need to be updated

Malin has a new file, spptaxonomy_2015-04-09_plusManual.csv, with a spiny dogfish and yellowtail flounder names corrected (were incorrect before). These need to be sent to Lucas Marxen and incorporated.

@mpinsky It'll be good to get information back from Rachel after her manual grooming of the entire set of species names.
Ultimately it would be good to cease working from manually generated csv files, and do something with a better-documented history.

This has now been fixed.


Implement "Entries on same page" page

There is a link from the detailed record view to the page which displays all the entries for the same page. The target page (path same_page_entries) has not yet been implemented.

Since raising this issue on Friday and today, this link has started to work. So closing issue immediately.


SQL dump should NEVER include the migrations table

Currently I got the issue, that I have multiple setups, where I often
update the data with a dump file. Currently I just take the whole dump file, with
the migrations table, pop it in and hope it works. Now I got some misalignment and
found the following post from you:
@Mojo90 The failure is on your side. Look through your sql dump. This should NEVER include the migrations table. This is ok as long as you don't restore your dump via db-migrate, but in this case you try to manipulate the migrations table from a migration itself. This will of course fail. Just remove this portion and it should work.
Originally posted by @wzrdtales in https://github.com/db-migrate/node-db-migrate/issues/589#issuecomment-456756411
What is the alternative? I would rather run the migrations and have them do nothing, if the table/column already exists. But I don't find any informations in the documentation
how to do that.
When not having (or removing) the migration table I just get the following error at my first migration:
[ERROR] AssertionError [ERR_ASSERTION]: ifError got unwanted exception: Relation »material« already exists

Thanks for your help :)

Yeah, if db-migrate is executing the sql for you, then it shouldn't contain its own tables. If you're dumping straightforward a table, the first question for me is, why don't you play it in with the appropriate command instead? db-migrate is a migration tool and doesn't have any backup functionality yet. That is planned however but not implemented and thus currently not intended.

so the comment there ment. you shouldn't just dump your db and use the output as a migration sql file without removing the db-migrate system tables. however if you restore the dump over normal procedures and not by abusing the migrations somehow for it you shouldn't have any issues.

So when I remove the migrations table manually and run
db-migrate up

It should restore the structure (or ignore it if it already exists) and not give me any errors?
I thought maybe I have to set a 'IF NOT EXIST' somewhere in the migration files or there is something like:
db-migrate restore

Then I must check my code again.
Thank you for your quick reply!

db-migrate will create the table, but it can't restore the knowledge in it, so that might not be sufficient. so if your purpose is dump and restore, please dump and restore outside of db-migrate since as said a migration is a not a backup, it is a state transition. and db-migrates real backup functionality I planned for is not there yet.

It's perfectly clear to me, that the migration just provides the structure, but has nothing to do with the data inside.
But I would need the migration to run on a restored set of data and ignore the migration file if it is already applied (table exist, column was added,...). But this doesn't seem possible at the moment.
It worked for me to just restore the data with the migrations table in place. But when there is one divergance it fails,
thats why, as you mentioned, it's a bad idea to do so.
Then I have to finde some other solution :)
Thank you for your hard work and the thorough explanation


New Uni.replaceWith shortcuts

See #290

/cc @gavinking, do you like that? It's really your code suggestion with a different name

LGTM.
I wonder if we need it on any other interface, e.g. UniOnItem?

I think we should stick to a shortcut in Uni, and then see if you or others have use cases for repeating in UniOnItem.
WYT @cescoffier?

Yes, I would stick with the shortcut for now. Let's see how it's used.


[engine] cannot add new files in flutter/third_party

To reproduce:
Add a new file under any "excluded" path of https://github.com/flutter/engine/blob/c0fcf94d54aba6db9ec7b722d92cf2acbee5ac5f/third_party/.gitignore
See the new file is not showing up in git status

Hmm, thanks for filing.
I think we need to do !{folder}/**, oopsie!

It seems like you're encountering an issue with Git. To troubleshoot this, make sure you've properly added the file to the repository and it's not covered by any global or local .gitignore rules. You can use the following steps:

Ensure you've added the file using git add <file>.

Check the global and local .gitignore files for any rules that might be excluding the file.

Verify the file is not listed in a .git/info/exclude file in the repository.

Confirm you're in the correct branch and haven't accidentally switched to a different one.

Use git status to see if Git recognizes the new file.

If you've done all these steps and the file is still not showing up in git status, please provide more specific information about the problem so I can assist you further.


Move SubNetwork and abstractTensor to ITensorAutoHOOT

Looks reasonable, feel free to merge if you think it's ready.


[BUG] How is the COCO17 dataset 144GB?

System information

Windows 10.0 Enterprise
Python 3.9.13
FiftyOne 0.19.1
FiftyOne installed from pip

Commands to reproduce
import fiftyone as fo
import fiftyone.zoo as foz
from fiftyone import ViewField as F

dataset = foz.load_zoo_dataset(
    "coco-2017",
    splits=("train", "validation"),
    label_types=["detections"],
    drop_existing_dataset==True
)

dataset.persistent = True

one_view = (
    dataset
    .select_fields("ground_truth")
    .match(F("ground_truth.detections").length() == 1)
)
# dataset.save_view("one-view", one_view)

session = fo.launch_app(one_view, desktop=True)
session.wait()

Describe the problem
COCO17 is ~26GB for the entire detection dataset, but FiftyOne is attempting to download 144GB. I was going to reduce the dataset down to images with only a single object instance, but I was trying to do that in the view.
Any help is appreciated!

The progress bar shows bits, not bytes. Does 25.20GB = 201.6Gb for the test + train splits sound more reasonable 😄
Related: the zoo docs explain how you can specifically download (only) a subset of COCO if you'd like.

Oh, shoot! Well, thank you!


fix(generator): 'generate container' produces unnecessary return - better solution

Replaces the solution merged with #1327
A better solution. Handles the problems and guides the user (dev).

Coverage remained the same at 98.585% when pulling f91f044c7489c168db6f2846d27a73fddb84b221 on gihrig:useless-return into 7e763894bba85401996ddeffe079bc35c8dd4124 on mxstbr:dev.


Sharing is Caring Power App Sample-GavinWarner

Category

[x] New Sample

What's in this Pull Request?
Power Apps Calendar Component

Thanks @GavinWarner05! 💜🧡


Linear Regression Error

Hi there,
It is a nice package and codes are easier.
But, why functions for loading the data arrays?
Also, can we use the in-built functions for different data manipulations.
Example, I tried basic "LinReg", both the data has same dimensions and still the following error pops up

That's really quick.
The shapes are as follows:
1)
[image: image.png]
2)
[image: image.png]
3)
[image: image.png]
I thought the first variable's latitudes weren't matching, so I used the
third one, yet, the same error persists.
Thanks for getting back :)
Girija Kalyani Burada
Doctoral StudentNew Zealand
On Mon, Apr 24, 2023 at 8:04 PM ZiluMeng @.***> wrote:

please give me the shape of siconc and u10
At 2023-04-24 15:58:39, "GK" @.***> wrote:
Hi there,
It is a nice package and codes are easier.
But, why functions for loading the data arrays?
Also, can we use the in-built functions for different data manipulations.
Example, I tried basic "LinReg", both the data has same dimensions and
still the following error pops up
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/ZiluM/sacpy/issues/1#issuecomment-1519583988, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWDHECMRYK2DJTTX73LOSDXCYX2BANCNFSM6AAAAAAXJGIGI4
.
You are receiving this because you authored the thread.Message ID:
@.***>

I can't see the image.png, please copy or upload a new one! Thank you!
I think the quick fix is switching the sconce and u10's locations.

sic= xarray.DataArray  time: 516      latitude: 361      longitude: 1440
u10 =
xarray.DataArray    time: 516        latitude: 281  longitude: 1440
z = xarray.DataArray'z'time: 516       latitude: 281         longitude:
1440

These are the quick sizes, plus, shift the location to? Package's folder?
On Mon, Apr 24, 2023 at 8:15 PM ZiluMeng @.***> wrote:

I can't see the image.png, please copy or upload a new one! Thank you!
I think the quick fix is switching the sconce and u10's locations.
—
Reply to this email directly, view it on GitHub
https://github.com/ZiluM/sacpy/issues/1#issuecomment-1519601921, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWDHEDG2YQ3NW3UTJLHTOLXCYZCRANCNFSM6AAAAAAXJGIGI4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Oh lovely, so, I can add another dimension as a time series.
Thanks for the clarification.
On Mon, 24 Apr 2023, 20:26 ZiluMeng, @.***> wrote:

I think you have not understood the function of LinReg. It calculates the
Y = aX + b, X is one dim and Y is one or two or three or more dims (plz see
the documents).
If you want to calculate the space correlation in each space point, I will
add this function in the next version!
Zilu Meng
At 2023-04-24 16:21:40, "GK" @.***> wrote:

sic= xarray.DataArray time: 516 latitude: 361 longitude: 1440
u10 =
xarray.DataArray time: 516 latitude: 281 longitude: 1440
z = xarray.DataArray'z'time: 516 latitude: 281 longitude:
1440

These are the quick sizes, plus, shift the location to? Package's folder?
On Mon, Apr 24, 2023 at 8:15 PM ZiluMeng @.***> wrote:

I can't see the image.png, please copy or upload a new one! Thank you!
I think the quick fix is switching the sconce and u10's locations.
—
Reply to this email directly, view it on GitHub
https://github.com/ZiluM/sacpy/issues/1#issuecomment-1519601921, or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ABWDHEDG2YQ3NW3UTJLHTOLXCYZCRANCNFSM6AAAAAAXJGIGI4
.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/ZiluM/sacpy/issues/1#issuecomment-1519616642, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWDHEBPPK5GCCE223OE7IDXCY2LZANCNFSM6AAAAAAXJGIGI4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Specify JWKS as format of didState.secret output field.

Addresses https://github.com/decentralized-identity/did-registration/issues/2, based on discussions during the 02 Jun 2021 DID Registration Work Item Call.

This is missing information on how the returned secrets can be encrypted (client needs to provide a public key).