Repo sync for protected CLA branch

The pull request is created from main638042365062721757sync_temp to main to fix git push error for protected CLA branch

Learn Build status updates of commit f63fe51:
:white_check_mark: Validation status: passed
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


Karpenter images have a zero build date

Version
v0.27.1 onwards
Expected Behavior
docker inspect public.ecr.aws/karpenter/controller:v0.27.5 -f '{{ .Created }}' should return a date in the last few weeks****
Actual Behavior
docker inspect public.ecr.aws/karpenter/controller:v0.27.5 -f '{{ .Created }}'
0001-01-01T00:00:00Z

Steps to Reproduce the Problem
Run docker inspect
Resource Specs and Logs
N/A
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Looks like this could be a problem with ko configuration
$ ko build -L -B github.com/aws/karpenter/cmd/controller
2023/05/29 12:01:46 Using base gcr.io/distroless/static:nonroot@sha256:9ecc53c269509f63c69a266168e4a687c7eb8c0cfd753bd8bfcaa4f58a90876f for github.com/aws/karpenter/cmd/controller
2023/05/29 12:01:48 Building github.com/aws/karpenter/cmd/controller for linux/amd64
2023/05/29 12:01:52 Loading ko.local/controller:3ff06d81a3fec41f8b71fd5ec943bb567f4765831e73d411b67ab4bc9b69b27d
2023/05/29 12:02:01 Loaded ko.local/controller:3ff06d81a3fec41f8b71fd5ec943bb567f4765831e73d411b67ab4bc9b69b27d
2023/05/29 12:02:01 Adding tag latest
2023/05/29 12:02:01 Added tag latest
ko.local/controller:3ff06d81a3fec41f8b71fd5ec943bb567f4765831e73d411b67ab4bc9b69b27d
$ docker inspect ko.local/controller:3ff06d81a3fec41f8b71fd5ec943bb567f4765831e73d411b67ab4bc9b69b27d -f '{{.Created}}'
0001-01-01T00:00:00Z

It's been this way for a very long time:
$ docker inspect public.ecr.aws/karpenter/controller:v0.2.0 -f '{{ .Created }}'
1970-01-01T00:00:00Z

Do you see something different?

@tzneal - ah, I hadn't checked older versions - I misunderstood a comment in slack.
So if we set SOURCE_DATE_EPOCH when building the final containers, we should be sorted

@engedaam is this actually sorted - can you point me to a fixed container image?

This is not fixed
docker inspect public.ecr.aws/karpenter/controller:v0.28.0 -f '{{ .Created }}'
0001-01-01T00:00:00Z

Yeah, looks like this was erroneously closed. We'll put out a fix. Apologize for the churn with open/close on this one.

FYI: this is where the SOURCE_DATE_EPOCH value needs to be set in KO to utilize this: https://ko.build/advanced/faq/. We need to utilize this in our release.sh script so that we don't end up with zero build dates that are blocking Karpenter install


Segmentation Fault when testing

I successfully built up the dependencies but failed in test phase, anyone knows what caused this?
`Installed /home/yuankunhao/DCNv2
Processing dependencies for DCNv2==0.1
Finished processing dependencies for DCNv2==0.1
(yuankunhao) [root@localhost DCNv2]# python test.py
Segmentation fault
(yuankunhao) [root@localhost DCNv2]# python test.py
Segmentation fault
`

get the same result when run test.py, my gpu driver version is: 384.130, cuda version is V9.2.148, any suggestion?

get the same problem

@SCoulY  @guanlinting Have you solved the problem?

@mangdian noop

get the same problem

@SCoulY @guanlinting @mangdian Have you solved the problem?

@linzhi123  @SCoulY  @guanlinting  @mangdian You will need gcc's version larger than 4.9 to compile the extension.

Must GCC be version 5.0 or higher? I also get segment default. My GCC is 4.8

Must GCC be version 5.0 or higher? I also get segment default. My GCC is 4.8

it seems so, i successfully compiled it under gcc 7.3.0.

when gcc is 4.8.0, i meet "Segmentation Fault" error, and i solve this problem when i change gcc version to 5.8.0,

Meet this problem using gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0


Bug: releaseCommitMessageFormat preset option doesn't work

The option releaseCommitMessageFormat doesn't seem to have any impact on the generated commit message.
My options look like this:
"options": {
        "preset": {
          "name": "angular",
          "releaseCommitMessageFormat": "chore(release): {{currentTag}} [skip ci]",
          "commitUrlFormat": "{{host}}/{{owner}}/{{repository}}/commits/{{hash}}",
          "compareUrlFormat": "{{host}}/{{owner}}/{{repository}}/branches/compare/{{currentTag}}%0D{{previousTag}}",
          "issueUrlFormat": "https://barbora.atlassian.net/browse/{{id}}"
        },
        "tagPrefix": "${projectName}@"
      }

All the other formatting options work fine.

@m3Lith I have a similar problem, but my issue is that none of the preset options have any impact on the generated changelog.
Just out of curiosity, which version are you using?

Apparently there's an option called commitMessageFormat, which is taken into account, and not the conventional commits option releaseCommitMessageFormat under preset.


WIP: Release changelog

Can we filter out stuff that's not relevant to users? We've had a lot of churn on things like Gemfile changes that are more noise than signal in here. I understand we want to auto-generate the changelog, but I think it's resulting in an increasingly unusable document.

Yup. Note this is WIP. I'm just getting started here :)

ok maybe not perfect but WIP removed

:+1:

👍


As a CF Dev user, downloading resources should automatically retry download if there are network issues

GIVEN I have run cf dev start
AND I am downloading resources for the first time
WHEN CF Dev loses network connectivity
THEN it will try to resume downloading resources up to 10 times before an error is thrown

Added in v0.0.4


Table Widget: core functionality

Screenshot from integration in carto-react-template sample app 3

This PR introduces TableWidgetUI as a new UI component and TableWidget as a new widget.
For now, supports sorting, pagination and click on row behavior.
Selection, search and export will be managed in further PRs.
TableWidgetUI behavior is fully controlled, so filtering, sorting and pagination has to be done by the parent component. This allows TableWidget to use the new getRawFeatures API from https://github.com/CartoDB/carto-react/pull/225, to access and sort features in an efficient way.

Pull Request Test Coverage Report for Build 960390032

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 76.973%

Totals

Change from base Build 950500057:
0.0%

Covered Lines:
610

Relevant Lines:
748

💛  - Coveralls

Shortcut link: https://app.shortcut.com/cartoteam/story/183138/implement-table-widget-basic-functionality

Shortcut link: https://app.shortcut.com/cartoteam/story/183138/implement-table-widget-basic-functionality

Looks good to me for a first version. Remember to update the changelog.

@padawannn about your comment on pagination vs. viewport, I've added this piece of code https://github.com/CartoDB/carto-react/pull/154/commits/17454dd4cdb14306df6c5f104583ab0c0d17dcc8 which resets the page to 1 when the features change.
It uses the fact that the ready state of the source goes through the cycle true -> false -> true when the viewport changes.
Do you think it's robust enough?

@padawannn about your comment on pagination vs. viewport, I've added this piece of code 17454dd which resets the page to 1 when the features change. It uses the fact that the ready state of the source goes through the cycle true -> false -> true when the viewport changes. Do you think it's robust enough?

@bbecquet we also need to reset the page to 1 if filters change. For example, in your branch template when you click hover a category in the category widget. We also should reset the page to 1  if the datasource change although this case is not common

@padawannn Thanks for the explanation about filters. I've changed the reset callback.


resumable upload response 416 after retry with status code 503

I am testing upload large file into Onedrive, e.g. 10G file.
but I suffered a  situation: if I got 503 status code and retry the request, I'll get 416 range erroor with error message: {"error":{"code":"invalidRange","message":"Uploaded fragment overlaps with existing data.","innererror":{"code":"fragmentOverlap"}}}
and during my test, I could not finished uploading 10G file more than 10 times, I'll get 503 error often and 416 error after retry.
please check this and I hope this is not a bug but just my mistake. Thanks!

Hi @mikechiu1012, it sounds as though you've obviously undesirable scenario whereby although the PUT for the fragment returned an error it actually succeeded under the covers. We'll dig in to why that occurred, but for now you may find you can work around the issue by, if you get a 416 on a PUT, doing a GET against the upload URL and parsing the "nextExpectedRanges" in the response body. Have a look at http://onedrive.github.io/items/upload_large_files.htm#request-upload-status and see if that will work for you.

hi @ificator, thanks for response, I'll test what you suggest for and I think it might related with https://github.com/OneDrive/onedrive-api-docs/issues/70. cause I use fragment size as 4MB.
I will try and report the latest result here, thanks for help!

Hey @mikechiu1012, I just wanted to follow up and see if you were still seeing issues uploading large files.

hi @ificator , since I used 5MB as fragment size, the error response is lesser to be seen.
thanks for ur great help!


Exclude non-essential files

This way the users of this package doesn't have to download non-essential files which aren't necessary for the package to function. Such as files used for development (docs, tests, .travis.yml, etc).
Happy Hacktoberfest!

Closing due to inactivity.


Investigate location typo

I'm assuming this is a typo, but it somehow has lasted in the code for 4 years...
https://github.com/microsoft/vscode-azureappservice/blob/11cf1f8855d1e4eef560dd4360eaca597ce967b7/src/commands/createWebApp/setPrePromptDefaults.ts#L34


Return the llm response when there's no tool result

I see that in the logging, that the llm still creates a response when theres no tool usage. Anyway you can expose that? It would be helpful, instead of chaining another claude response.

@gbertb that can be exposed, will try to add it soon


Fix function call highlighting for C-Functions

Some function names do not contain the parameter list (mostly C
functions). Thus, the disassembly view highlighter fails matching
those functions when occuring in a call.
This change restructures matching, such that not the complete match
will get highlighted, but the outermost capture group.
This allows more flexible describtion around the match without
highlighting this (similar to lookarounds, but without the restriction
of fixed size lookaheads).
With this change, we can now simplify the call target regex
to "anything that is inside parantheses and comes after a call".
Test: CodeViewerDemo + Disassembly of some functions
Bug: http://b/217849922

I have the feeling we are overcomplicating the Syntax Highlighting with this.
There is an operator "?:" which literally means "Match expression but do not capture it". What are we doing wrong so this operator is not useful for us? Therefore we can match ?:callˆ( without capturing it.

This is the lookahead operator mentioned in the PR description, it does only work if we know how many characters are there before the opening "(".

Everything seems perfect, but did you test it using the CodeViewerDemo?

As discussed offline, I tested the changes with all examples we have for CodeViewerDemo


Add FirebaseFirestore.h to fix Firebase module in source build

Add a second umbrella header so that the Firebase module will include the Firestore umbrella header in source builds.
Changing Firebase.h to check for Firestore-umbrella.h would not work because umbrella headers only get generated by CocoaPods when use_frameworks! is specified and the name of the umbrella header may or may not be qualified by a target if the Podfile specifies multiple Apple targets.
More details internally at b/70281259

It's intentional to exclude the version declaration since our internal build system doesn't generate those.


feat: add convert_method_to_trt_engine() for dynamo

Description
Add a function convert_method_to_trt_engine() for dynamo.
I'm not familiar with this part, so detailed reviews and suggestions are greatly welcome.
Fixes #2410
Type of change

New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project (You can use the linters)
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas and hacks
[ ] I have made corresponding changes to the documentation
[ ] I have added tests to verify my fix or my feature
[x] New and existing unit tests pass locally with my changes
[x] I have added the relevant labels to my PR in so that relevant reviewers are notified

@gs-olive @narendasan Another question - Regarding testing, how to invoke this function like directly calling torchtrt.dynamo.convert_method_to_trt_engine? and where should I put the test cases?

I think it could potentially go in tests/py/dynamo/runtime? Then, you can directly call the function to generate the TRT engine which can be run directly in Python for inference purposes.

Got a Python library import error:
==================================== ERRORS ====================================
_ ERROR collecting tests/py/dynamo/runtime/test_convert_method_to_trt_engine.py _
ImportError while importing test module '/__w/TensorRT/TensorRT/pytorch/tensorrt/tests/py/dynamo/runtime/test_convert_method_to_trt_engine.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/opt/python/cp38-cp38/lib/python3.8/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
runtime/test_convert_method_to_trt_engine.py:4: in <module>
    import pycuda.driver as cuda
E   ModuleNotFoundError: No module named 'pycuda'

Can we add pycuda in CI?

Hi @zewenli98 - sure, I think we can add this dependency at the path tests/modules/requirements.txt, and it should be reflected in the CI afterward.

Thanks! Naren suggested that PythonTorchTensorRTModule should be used instead of directly calling native functions, so I modified accordingly.


Adding CVSS to the list of findings

Hey,
starting with a vulnerability management in a quite big environment can be challenging. Specially when tools are used which are scanning everything and creating big lists of different, yet unknown, vulnerabilities for an import in defectdojo. Depending on the size of the environment, patch management and vulnerability management before using such tools, you could be overwhelmed by a big list of vulnerabilities found in your environment.
To have an easier mechanism for prioritizing each finding, it would be very helpful to have the CVSS score also listed in the list of findings. Thus making it more easy to filter and prioritize the fix of each finding, specially when you think that some of them might have an other score depending on the situation.
Kind regards

In order to prioritize findings, we added EPSS score recently (look at release 2.32). To me it can be closed. What is your opinion @mtesauro ?

@manuel-sommer Yes, I agree. We also have vunerability ID which can hold any vulnerability identifier you want so I think we're good here.
Additionally, you can provide CVSS (including environmental, etc) scores and DefectDojo will adjust the Severity of a finding accordingly so you can still utilize CVSS as long as you're OK with it being 'translated' into DefectDojo's severity levels.


DDI-396 Update SDK Nav

Updating the SDK navigation links so it behaves in line with expectations. Involves some heavy updates to accomodate breadcrumbs and work around limitations of MkDocs.
Deadline
ASAP
Change type

[X] Non-documentation related fix or update.

PR checklist:

[X] My documentation follows the style guidelines of this project.
[X] I previewed my documentation on a local server using mkdocs serve.
[X] Running mkdocs serve didn't generate any failures.
[X] I have performed a self-review of my own documentation.

Linkcheck errors are all due to includes files. They work on preview.


Why remove video and time components from display pages

Why remove video and time components from display pages

We do not remove these two types of component, if these components do not be shown on Display, it might be caused by kind of bugs.

I used davinci0.3, but have been removed.

https://github.com/edp963/davinci/blob/dev-0.3/webapp/app/containers/Display/components/util.ts#L33

maybe my version is old


Using PROST in Error

Seems like if the crate is built with prost-codec feature - then prost error should be there https://github.com/tikv/raft-rs/blob/2357cb22760719bcd107a90d1e64ef505bdb1e15/src/errors.rs#L27

You need to change the prost version to be same with this library

We use protobuf-build to unify the APIs and use rust-protobuf versions, that's why there is only one type of error.

Closed as answered.
Out of thread, I wonder about the rationale of choosing between rust-protobuf and prost. Is there some best practices or benchmark?

We prefer rust-protobuf because it's more function-complete and well proved in production. In the long term, I think the official protobuf implementation is preferred.


Various network Tweaks

Most of these are small fixes in preparation for moving IP allocation to the master.

Current coverage is 55.69% (diff: 59.74%)

Merging #705 into master will decrease coverage by 0.47%

@@             master       #705   diff @@
==========================================
  Files            84         84
  Lines          6945       6862    -83
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           3901       3822    -79
+ Misses         2736       2732     -4
  Partials        308        308

Powered by Codecov. Last update 3cdc486...ee630ee


[Request] CSD-G6 unused BSicon redirects

It would be nice if it were possible for redirects not used outside the User namespace on any WMF wikis and not contained in any of the exception categories to be automatically tagged for speedy deletion under criterion G6. Currently I manually filter the complete list every few months using BBEdit, a Quarry query and PetScan queries.

Please start a discussion about having a bot do this so that I can link it in the approval request. I expect that it would be uncontroversial.

This issue was moved to JJMC89/bsiconsbot#5


[java] Create a central cacerts store

[x] convert java to v2 tools (#487)
[ ] create /opt/buildpack/ssl/cacerts on prepare-tool java(-jre)
[ ] symlink cacerts to   /opt/buildpack/java(-jre)/lib/security/cacerts on install-tool java(-jre)

part of #3

Thinking if we really need this. for gradle and maven a user can set:

MAVEN_OPTS=-Djavax.net.ssl.trustStore=/my/cacerts.jks

GADLE_OPTS=-Djavax.net.ssl.trustStore=/my/cacerts.jks

JAVA_OPTS=-Djavax.net.ssl.trustStore=/my/cacerts.jks (only gradle)

https://maven.apache.org/guides/mini/guide-repository-ssl.html#setting-properties

https://docs.gradle.org/current/userguide/build_environment.html#sec:gradle_configuration_properties

My goal was that the user can just add the custom certs to the image and the rest is taken care of. Can we export these environment variables automatically if we find they have the custom certs file, once we do the ubuntu update command?

we need java keytool to create such a store or to import certs to an existing key store 😕


Update repo link (master => main)

Closes #1978

@danieljurek, @weshaggard do we have some strategy to keep links always pointing to main going forward? I imagine link verification doesn't flag these as invalid since GitHub re-routes it.

We don't have a strategy. The thought is once there is a one-time clean-up the most new links will just point to main.


Add function to determine ITCZ edges based on CWV

This PR:

adds a function to determine the ITCZ edges based on the column water vapour field
reformats (ruff) the flightplan.py file
sets the requirements for easygems to >=0.0.8

This closes #1, closes #4

Both the old implementation as well as my update don't work.... I will investigate this further

With the latest fix, the following code snippet works for me:
import easygems.healpix as egh
import intake

from orcestra import flightplan

cat = intake.open_catalog("https://tcodata.mpimet.mpg.de/internal.yaml")
ds = cat.HIFS(datetime="2024-08-11").to_dask().pipe(egh.attach_coords)

flightplan.find_edges(ds.tcwv.isel(time=1), 48)

(-16.334822780685997, 8.989299345162806)


General Cleanup

Logs stdout and stderr to JSON return, closes #5.
Moves error return field from error to error_message.
Provides choice to fail or not via input, closes #4.

Codecov Report

Merging #6 into master will increase coverage by 1.26%.
The diff coverage is 79.31%.

This pull request introduces 3 alerts when merging 4aeb6bb9e12d1651099f74d7ee1d9b948cb8e0b9 into af1805b07b6d8d2af83281f5e784fa6678a22f12 - view on LGTM.com
new alerts:

2 for Unused import
1 for Unused local variable

Comment posted by LGTM.com

This pull request introduces 1 alert when merging 5081fb88dc00c7e5a1eed702091488db55d25fd3 into af1805b07b6d8d2af83281f5e784fa6678a22f12 - view on LGTM.com
new alerts:

1 for Unused import

Comment posted by LGTM.com


3 node cluster being created instead of a single notebook VM

I created a new notebook VM in a new Terra project and when I look at the instances list in the Cloud console, I see 3 VMs created:

saturn-616cedf5-cd3d-4dee-a860-be597706708f-m
saturn-616cedf5-cd3d-4dee-a860-be597706708f-w-0
saturn-616cedf5-cd3d-4dee-a860-be597706708f-w-1

Configure as Spark cluster is NOT checked.
A co-worker also created a notebook runtime and we observe 3 VMs created.

Hi @mbookman,
I looked at this and unfortunately I think it's a Leo bug. :-( A change released last week allowed for user-specified Spark properties. Unfortunately I think it was overwriting the dataproc:dataproc.allow.zero.workers property which is needed for 0-worker clusters. I made a PR (https://github.com/DataBiosphere/leonardo/pull/907) to fix this. We'll get it release ASAP next week.

@mbookman the fix is live in prod as of 5/5 2:30pm EST. I made a cluster in Terra and verified I get only 1 instance as expected:

The dataproc properties are set as expected too.
So you should be free to recreate clusters. We'll follow up with a post mortem as mentioned via email.


Fixed key error on pusher:pong

Just noticed this isn't actually an issue on official pusher server, it's on the pusher custom implementation i'm using (beyondcode/laravel-websockets) 🤦♂
So... not strictly necessary to adapt pysher to it but it's still better practice to use get() (?)

Just noticed this isn't actually an issue on official pusher server, it's on the pusher custom implementation i'm using (beyondcode/laravel-websockets) man_facepalming

No worries :smile: it happens.

So... not strictly necessary to adapt pysher to it but it's still better practice to use get() (?)

hm, I'd say it depends. In principle, I'd say it should throw an error, since the key is expected to be there. Its absence shouldn't have to be handled by the downstream function (which is what returning None from dict.get() would do). On the other hand, it shouldn't just throw a KeyError for custom implementations like the one you're using.
Initially, I was thinking of catching the KeyError and calling the function anyway if it's a pong request, but again - that seems too dependent on the custom implementation.
In the very least, it should perhaps raise a better exception.

Thing is pusher is vague on their documentation about data field:
Here you can see that ping & pong messages should have an empty data field

This means that Pusher Channels will respond to a WebSocket protocol ping message with a pong message, and also it will respond to apusher:ping event with a pusher:pong event (both have empty data).

On same doc page you can later see them telling data field is optional:

All events received and sent by clients can contain a data field.

Edit: If pusher allows event data field to be optional the checks should be handled by the functions bound to those events thinking

That's true. Let's merge it then. :)


[TYPO] Undefined variable $path

It's a tiny one and i think it's self-explained :)
https://github.com/Laravel-Backpack/CRUD/blob/c7c74166244f7995a07abb83f2a83e914e515c9f/src/app/Library/CrudPanel/Traits/Views.php#L295

Fixed in https://github.com/Laravel-Backpack/CRUD/pull/4184 - will tag & merge in a few minutes.
Thank you @dickalexei !


Problem when deserialising large numbers

When using ppx_deriving with js_of_ocaml I have a problem when deserialising not-too-small integers (with ten digits or more). The following code is a minimal example:
open Printf

type x = {
  a: int;
} [@@deriving yojson]

let json =  {json|{"a":9999999999}|json}

let () =
  eprintf
    "test_integer: %S: Back and forth.\n"
    (Yojson.Safe.(to_string(from_string json)))

let () =
  let x =
    x_of_yojson (Yojson.Safe.from_string json)
  in
  match x with
  | Result.Ok(actualx) -> eprintf "test_integer: %d\n" actualx.a
  | Result.Error(mesg) -> eprintf "test_integer: %s\n" mesg

The first test demonstrates that decoding and recoding json text using Yojson works, and especially that js_of_ocaml has no problem with the not-so-small integer constant.
node test_integer.js
test_integer: "{\"a\":9999999999}": Back and forth.
test_integer: Dynamodb_TestInteger.x.a

Replacing the integer type by int64 let the program run without errors:
test_integer: "{\"a\":9999999999}": Back and forth.
test_integer:<PHONE_NUMBER>

Is there any special processing in ppx_deriving that could explain this discrepancy? Or is this more likely a js_of_ocaml bug? (The latter hypothesis seems to be ruled out by the first test.)

System information
opam list ppx_deriving ppx_deriving_yojson  yojson
# Available packages for 4.02.3:
ppx_deriving           4.0  Type-driven code generation for OCaml >=4.02
ppx_deriving_yojson    3.0  JSON codec generator for OCaml >=4.02
yojson               1.3.2  Yojson is an optimized parsing and printing library

It is my understanding based on your report that this problem manifests exclusively on js_of_ocaml, and not natively. In this case this is a js_of_ocaml bug, that is not exposed for some reason when you write the deserialization code explicitly; OCaml does not (apart from the Obj module that I don't use) have a concept of "undefined behavior", so if it works on ocamlc, it ought to work the same way on js_of_ocaml by definition (of the OCaml language and standard library semantics).
Please reopen if you have a compelling counterargument, and I'll take a look.

Incidentally, the code generated by ppx_deriving_yojson for your record basically consists of match `Int x -> Result.Ok (x) >>= ... with some fluff around it to handle the field name. ppx_deriving_yojson doesn't do any manipulation of the integer value whatsoever, that's all inside yojson. So I really don't see how it can be responsible here.


Allow cross-compile Android while compiling libicu in Linux.

Before this commit, if one used the --libicu parameter of the build script to compile ICU in Linux, the variable SWIFT_PATH_TO_LIBICU_BUILD is set. While evaluating which link flags to use in stdlib/public/core/CMakeLists.txt, the presence of that variable will set all the SDKs to use -licuuc style link libraries, which will not work while cross-compiling Android (since Android libraries have a swift suffix, and they are not in the library search path).
This commit delays the selection of -licuuc style or full paths to later in the process, where the current SDK is already know, and the script can distinguish between the Android SDK case and other SDKs.
This change will become more important if #19860 is merged. It might need to be modified slightly to always use the swift suffixes, though.
/cc @spevans

Just to note to say that swift/cmake/modules/FindICU.cmake and
swift-corelibs-foundation/cmake/modules/FindICU.cmake are completely different using different methods to find ICU. I think we should make them both the same, probably using the one from swift/cmake/modules/FindICU.cmake in both places. This means that the -DICU_... variables set in build-script-impl are actually used differently for swift v swift-corelibs-foundation.
I think they each use a  different subset of the variables presented. So a followup PR would be required to fix up swift-corelibs-foundation after this one is done.

@spevans we can include that in the cmake distribution for swift. Then Foundation can just import it itself from swift.

I’m aware that the way ICU is linked in swift and in swift-corelibs-foundation is different. And it will probably will have to keep changing if ICU is integrated more in the swift build system. I don’t like the special support for Linux, and the weird setup for Android (but I like more the weird setup for Android than the special support for Linux, since it is more flexible and works with any SDK).
Anyway, this change was a minimal change to allow compiling both Linux and Android, Linux with a self-compiled ICU, Android with a pre-compiled ICU (which should be the typical setup for many Linux distributions).
I didn’t noticed that you finally merged #19860, but this one will probably have to change now. From the recommended scripts to build ICU for Android (SwiftAndroid/libiconv-libicu-android), I don’t think it would be easy to self-compile ICU for Android, but I will see if there’s a way of modifying the scripts to build something that works with FindICU.cmake


RAM leak

@yuki-sakura-chan . When a new song is launched, the RAM increases by 10 megabytes. And so with each new music. This problem has grown in my new pull request to the fact that with each new slider movement, the RAM increases by 10 megabytes.

@Ai-Vonie
Since I store the file list in memory after loading, I plan to optimize it after updating the cloud music library.
Recently, the game I’ve been playing released a new version, and my focus has been on the game. I’ll start coding once I have some free time.

@yuki-sakura-chan , I fixed the RAM leak in the new commit. You didn't clear the audio channels for the songs.


New pixel

Checklist

[ x ] I ran npm test locally and it passed without errors.

[ x ] I only edited the _data/pixels.json file.

[ x ] I entered the username in the pixels.json that I'm also using to create this pull request.
[ x ] I acknowledge that all my contributions will be made under the project's license.

I am not following the issues in my Pull Request details; I only made a change to the _data/pixels.json file; added my pixel at coordinates x:50 y:50

Looks like this is an issue with GitHub action or node itself, danger is erroring with JavaScript heap out of memory.
From a visual check there doesn't seem to be anything wrong with your pixel.
I don't work at Twilio, so I can't really help with getting this approved, but if you make another commit or amend your existing commit and push back up to this branch (or close and make a new PR), it should kick off the CI process again and hopefully danger will pass.


Add multiple requests for www::mechanize

See if we can get www::mechanize to allow multiple requests on a single GET operation to prevent the occasional
Error GETing [ blah blah blah ] Status read failed: Connection reset by peer at sensor2splunk/lib/Apc.pm line 48.
Update: commit on 2017-11-02 was a first pass at attempting to resolve this issue.  If this update solves the issue after a few weeks, close this feature request.

After update on 2017-11-02, the logout function for APC was still using a single request, which resulted in errors.  This was updated on 2018-02-06.  If the errors go away, this issue will be closed.

This appears to be fixed now.


ZZUTPRE has 2 problem in mapping %ut in Cache

Line 32: hardcoded to a database name of "VISTA"
Line 37:
S %=##Class(Config.MapGlobals).Get(NMSP,"%ut*",.PROP)

Always returns with an error of:
 Error=ERROR #5002: Cache error: <UNDEFINED>zCreate+13^Config.CommonMapMethods.1
*Properties

I suggest reverting to the previous API.

This problem was fixed by the update in April 2016.

This problem was fixed by the update in April 2016.


Add the ability to override default ssh and api access cidr

Hey @scholzj,
Love the project. Thanks for all of your hard work. I need the ability to override the default CIDRs for ssh and api access. It seemed like a simple enough tweak to submit.
Let me know if I can do anything to facilitate getting it merged.
Thanks,
Chris

Thanks for the PR.


Add common Makefile/dependabot

This includes:

Adding Dependabot for GitHub Actions
Upgrading controller-gen
Adding (and addressing) linting
Adding a version check for operator-sdk so the correct version is always used
Updating the bindata flow to run it directly during the CI

ref: https://issues.redhat.com/browse/ACM-10714

Missed the go.mod path update in the golangci-lint config. This PR should be ready for review now--sorry for the churn there!


Research: Ninbot hugemoji command

Discord emotes can be made bigger. Normal emojis is a tougher area that I don't feel like dealing with right now. Closed with https://github.com/Nincodedo/Ninbot/commit/f1471af0e777bd1ea3e5ff82d1bf08ca49db1a08 for now.


[FR] please add a option to exclude polyfills for min version

export default {
    input: 'src/index.js',
    name: 'xxx',
    globals: {
    },
    polyfills: false, // HERE!!!!!
    plugins: [
        sass({
            insert: true
        })
    ]
}

I actually think the coolest thing would be a 3rd build > like xx-xx-plugin.raw.min.js, that contains no polys!!! Coders than can choose to add or not them with external link :)).. depending on browser requirements.

Minified bundle (especially transpiled from ES8 to olders versions) without polyfills can causing errors such as asyncIterator is not defined or similar. This is an extra complexity that gives more errors than good.
I think the best solution would be to add "module" property to package.json of each plugin.
"module": "build/some-plugin.debug.js",
This way when building your application using webpack it will import *.debug.js for import SomePlugin from 'rete-some-plugin';
*.debug.js does not have polyfills, since the source code is not converted to older versions. Converting to old versions is going with webpack (if you configure it)


About the chart animations ?

Heyya guys,
I had a try to generate a 3D Donut chart but the plugin rendered immediately without any animation. So I need a module in additional ?
Thx a lot.

No one met the prob as mine ?

See if setting neverReflow to true would work for you

I found no option called neverReflow.
If you meant "reflow" to false, unfortunately it still doesn't work for me.

https://github.com/kirjs/react-highcharts/blob/master/src/chartsFactory.jsx#L11

Please let me know if this worked for you

Feel free to reopen

So sorry for coming late, but this props was never gonna work for me.
Finally, I wrote my own components to implement highchart.

@be-next-hotdog (or others in the future) you may try this workaround we used here
https://github.com/kirjs/react-highcharts/issues/171
hope it helps someone out 👍


3D bounding boxes not displaying correctly

Describe the bug
Hi, I've been experiencing another minor issue. After I draw a 3d bounding box in the 2d viewer, the bounding box disappears if I change the z-slice. If I switch the viewer to 3d mode and back to 2d, the bounding box displays over all slices as expected.
Steps to reproduce
Using the napari-bbox plugin, create a new layer with ndims=3. After drawing the bounding box, adjust the z-slice a few steps and the box disappears. Switch to 3D view then back to 2D, and the box displays correctly. (Alternatively, the issue is fixed by transposing or switching the axes).
I'm using napari version 0.4.18.
Any thoughts on this?
Thanks,
Graham

Hey,
yeah, I remember that I encountered this issue. Unfortunately, I couldn't find out what's the cause of this issue. I'm quite busy nowadays, but I'll try to look into the problem soon.
Best,
David

Hi @bauerdavid, I've found the source of this issue. When drawing bounding boxes, the transform function is called which updates the data points correctly but does not update the bounding box slice key. The slice key then stays at its initial value until the data method is called.
I can create a pull request with the fix for this if I have push access to the repo.
Thanks,
Graham

Great, thanks for looking into it! You don't actually need push access, just create a fork from the repo, and edit there. You can find some more info here.

Solved in #17


Update sbt to 1.4.3

Updates org.scala-sbt:sbt from 1.4.2 to 1.4.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, semver-patch

java.lang.NoSuchMethodException: sbt.test.ScriptedTests$.runInParallel(java.io.File, boolean, [Ljava.lang.String;, java.io.File, [Ljava.lang.String;, java.util.List, int)

:thinking:

It also didn't even run in the proper order?  It's meant to run sbt test scripted mimaReportBinaryIssues "$PUBLISH" but it looks like it published and then blew up for scripted?
[info] + PreviousVersionSpec.on commit with local changes: OK, proved property.
[info] + PreviousVersionSpec.contains unrelated tags in-between version tags: OK, proved property.
[info] + PreviousVersionSpec.on tag v1.0.0 with local changes: OK, proved property.
[info] Passed: Total 62, Failed 0, Errors 0, Passed 62
[success] Total time: 29 s, completed Nov 16, 2020 10:49:40 AM
[info] Main Scala API documentation to /home/travis/build/dwijnand/sbt-dynver/target/scala-2.12/sbt-1.0/api...
model contains 21 documentable templates
[info] Main Scala API documentation successful.
[info] :: delivering :: com.dwijnand#sbt-dynver;4.1.1+31-14cf2dac :: 4.1.1+31-14cf2dac :: integration :: Mon Nov 16 10:49:44 UTC 2020
[info] 	delivering ivy file to /home/travis/build/dwijnand/sbt-dynver/target/scala-2.12/sbt-1.0/ivy-4.1.1+31-14cf2dac.xml
[info] 	published sbt-dynver to /home/travis/.ivy2/local/com.dwijnand/sbt-dynver/scala_2.12/sbt_1.0/4.1.1+31-14cf2dac/jars/sbt-dynver.jar
[info] 	published sbt-dynver to /home/travis/.ivy2/local/com.dwijnand/sbt-dynver/scala_2.12/sbt_1.0/4.1.1+31-14cf2dac/srcs/sbt-dynver-sources.jar
[info] 	published sbt-dynver to /home/travis/.ivy2/local/com.dwijnand/sbt-dynver/scala_2.12/sbt_1.0/4.1.1+31-14cf2dac/docs/sbt-dynver-javadoc.jar
[info] 	published ivy to /home/travis/.ivy2/local/com.dwijnand/sbt-dynver/scala_2.12/sbt_1.0/4.1.1+31-14cf2dac/ivys/ivy.xml
[error] java.lang.NoSuchMethodException: sbt.test.ScriptedTests$.runInParallel(java.io.File, boolean, [Ljava.lang.String;, java.io.File, [Ljava.lang.String;, java.util.List, int)
[error] 	at java.lang.Class.getMethod(Class.java:1786)
[error] 	at sbt.ScriptedPlugin$.$anonfun$scriptedRunTask$1(ScriptedPlugin.scala:115)
[error] 	at scala.Function1.$anonfun$compose$1(Function1.scala:49)
[error] 	at sbt.internal.util.$tilde$greater.$anonfun$$u2219$1(TypeFunctions.scala:62)
[error] 	at sbt.std.Transform$$anon$4.work(Transform.scala:68)
[error] 	at sbt.Execute.$anonfun$submit$2(Execute.scala:282)
[error] 	at sbt.internal.util.ErrorHandling$.wideConvert(ErrorHandling.scala:23)
[error] 	at sbt.Execute.work(Execute.scala:291)
[error] 	at sbt.Execute.$anonfun$submit$1(Execute.scala:282)
[error] 	at sbt.ConcurrentRestrictions$$anon$4.$anonfun$submitValid$1(ConcurrentRestrictions.scala:265)
[error] 	at sbt.CompletionService$$anon$2.call(CompletionService.scala:64)
[error] 	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
[error] 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
[error] 	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
[error] 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
[error] 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
[error] 	at java.lang.Thread.run(Thread.java:748)
[error] (scriptedRun) java.lang.NoSuchMethodException: sbt.test.ScriptedTests$.runInParallel(java.io.File, boolean, [Ljava.lang.String;, java.io.File, [Ljava.lang.String;, java.util.List, int)
[error] Total time: 4 s, completed Nov 16, 2020 10:49:44 AM

https://github.com/sbt/sbt/issues/6090


Add Unique Tag

Adds a unique tag to parameters with label-info color.

Looking to find out what Travis doesn't like about this PR


undocumented API - support

There are some useful, but undocumented api urls, like
https://api.github.com/user/:id. Is there a trick to request these apis?
Also works for:
https://api.github.com/repositories/:id

@PatWie you can retrieve an repository's details using its id after this PR has merged.

@PatWie I think if API is not documented on Github then it's not supported officially by Github so they might update it or remove it any time which could break functionality of the apps using it so developers should not use such API's. Thoughts?

Well, if you want to robustly deal with repository, it is a much better idea to just store the repo-id rather than everything else. In addition, GitHub allows users to change their nicknames. This also would screw up any tight integration.
I currently extent this library using
module GithubRepoExtension
  def get_by_id(*args)
    arguments(args, required: [:id])
    get_request("/repositories/#{arguments.id}", arguments.params)
  end
end

But looking forward the merge.

@Shwetakale I think the risk is the same as for any other documented method call. GitHub chops and changes stuff quite frequently so I don't see a problem merging this in.
@prasadsurase Thanks for working on this PR.
@PatWie Done.


Server errors on stopping and starting

Server starts for the first time and generates new world, no errors
228 recipes
27 achievements
2021-10-28 12:09:25 [INFO] Starting minecraft server version 1.5.2
2021-10-28 12:09:25 [INFO] Loading properties
2021-10-28 12:09:25 [INFO] Default game type: SURVIVAL
2021-10-28 12:09:25 [INFO] Generating keypair
2021-10-28 12:09:25 [INFO] Starting Minecraft server on xxxx
2021-10-28 12:09:26 [INFO] Preparing level "world"
...Add-On Handler Initializing...
2021-10-28 12:09:26 [INFO] ...Add-On Handler Initializing...
Better Than Wolves Community Edition Version 1.2.1 Initializing...
2021-10-28 12:09:26 [INFO] Better Than Wolves Community Edition Version 1.2.1 In
itializing...
Better Than Wolves Initialization Complete.
2021-10-28 12:09:26 [INFO] Better Than Wolves Initialization Complete.
...Add-On Handler Loading Custom Language File With Prefix: BTW...
2021-10-28 12:09:26 [INFO] ...Add-On Handler Loading Custom Language File With P
refix: BTW...
...Add-On Handler Initialization Complete...
2021-10-28 12:09:26 [INFO] ...Add-On Handler Initialization Complete...
2021-10-28 12:09:26 [INFO] Preparing start region for level 0
2021-10-28 12:09:27 [INFO] Preparing spawn area: 11%
2021-10-28 12:09:28 [INFO] Preparing spawn area: 25%
2021-10-28 12:09:29 [INFO] Preparing spawn area: 42%
2021-10-28 12:09:30 [INFO] Preparing spawn area: 56%
2021-10-28 12:09:31 [INFO] Preparing spawn area: 68%
2021-10-28 12:09:32 [INFO] Preparing spawn area: 79%
2021-10-28 12:09:33 [INFO] Preparing spawn area: 92%
2021-10-28 12:09:34 [INFO] Done (8,169s)! For help, type "help" or "?"

Server stops for the first time nad gives error
stop
2021-10-28 12:12:19 [INFO] Stopping the server
2021-10-28 12:12:19 [INFO] Stopping server
2021-10-28 12:12:19 [INFO] Saving players
java.net.SocketException: Socket closed
        at java.base/sun.nio.ch.NioSocketImpl.endAccept(NioSocketImpl.java:689)
        at java.base/sun.nio.ch.NioSocketImpl.accept(NioSocketImpl.java:762)
        at java.base/java.net.ServerSocket.implAccept(ServerSocket.java:681)
        at java.base/java.net.ServerSocket.platformImplAccept(ServerSocket.java:
647)
        at java.base/java.net.ServerSocket.implAccept(ServerSocket.java:623)
        at java.base/java.net.ServerSocket.implAccept(ServerSocket.java:580)
        at java.base/java.net.ServerSocket.accept(ServerSocket.java:538)
        at if.run(SourceFile:61)
2021-10-28 12:12:19 [INFO] Saving worlds
2021-10-28 12:12:19 [INFO] Closing listening thread
2021-10-28 12:12:19 [INFO] Saving chunks for level 'world'/Overworld
2021-10-28 12:12:19 [INFO] Saving chunks for level 'world'/Nether
2021-10-28 12:12:19 [INFO] Saving chunks for level 'world'/The End
2021-10-28 12:12:19 [INFO] Stopping server

Server starts again and gives errors (this happens every time from now on)
228 recipes
27 achievements
2021-10-28 12:13:41 [INFO] Starting minecraft server version 1.5.2
2021-10-28 12:13:41 [INFO] Loading properties
2021-10-28 12:13:41 [INFO] Default game type: SURVIVAL
2021-10-28 12:13:41 [INFO] Generating keypair
2021-10-28 12:13:41 [INFO] Starting Minecraft server on xxxx
2021-10-28 12:13:41 [INFO] Preparing level "world"
java.lang.NullPointerException: Cannot invoke "java.util.Map.entrySet()" because
 "FCAddOnHandler.addonWorldDataMap" is null
        at ajv.LoadModInfoFromNBT(WorldInfo.java:711)
        at ajv.<init>(WorldInfo.java:173)
        at aju.c(SourceFile:56)
        at ajq.b(SourceFile:100)
        at net.minecraft.server.MinecraftServer.b(SourceFile:109)
        at net.minecraft.server.MinecraftServer.a(SourceFile:146)
        at hz.c(SourceFile:177)
        at net.minecraft.server.MinecraftServer.run(SourceFile:306)
        at gp.run(SourceFile:573)
java.lang.NullPointerException: Cannot invoke "java.util.Map.entrySet()" because
 "FCAddOnHandler.addonWorldDataMap" is null
        at ajv.LoadModInfoFromNBT(WorldInfo.java:711)
        at ajv.<init>(WorldInfo.java:173)
        at aju.c(SourceFile:67)
        at ajq.b(SourceFile:100)
        at net.minecraft.server.MinecraftServer.b(SourceFile:109)
        at net.minecraft.server.MinecraftServer.a(SourceFile:146)
        at hz.c(SourceFile:177)
        at net.minecraft.server.MinecraftServer.run(SourceFile:306)
        at gp.run(SourceFile:573)
java.lang.NullPointerException: Cannot invoke "java.util.Map.entrySet()" because
 "FCAddOnHandler.addonWorldDataMap" is null
        at ajv.LoadModInfoFromNBT(WorldInfo.java:711)
        at ajv.<init>(WorldInfo.java:173)
        at ajt.d(SaveHandler.java:130)
        at net.minecraft.server.MinecraftServer.a(SourceFile:155)
        at hz.c(SourceFile:177)
        at net.minecraft.server.MinecraftServer.run(SourceFile:306)
        at gp.run(SourceFile:573)
java.lang.NullPointerException: Cannot invoke "java.util.Map.entrySet()" because
 "FCAddOnHandler.addonWorldDataMap" is null
        at ajv.LoadModInfoFromNBT(WorldInfo.java:711)
        at ajv.<init>(WorldInfo.java:173)
        at ajt.d(SaveHandler.java:146)
        at net.minecraft.server.MinecraftServer.a(SourceFile:155)
        at hz.c(SourceFile:177)
        at net.minecraft.server.MinecraftServer.run(SourceFile:306)
        at gp.run(SourceFile:573)
...Add-On Handler Initializing...
2021-10-28 12:13:41 [INFO] ...Add-On Handler Initializing...
Better Than Wolves Community Edition Version 1.2.1 Initializing...
2021-10-28 12:13:41 [INFO] Better Than Wolves Community Edition Version 1.2.1 In
itializing...
Better Than Wolves Initialization Complete.
2021-10-28 12:13:42 [INFO] Better Than Wolves Initialization Complete.
...Add-On Handler Loading Custom Language File With Prefix: BTW...
2021-10-28 12:13:42 [INFO] ...Add-On Handler Loading Custom Language File With P
refix: BTW...
...Add-On Handler Initialization Complete...
2021-10-28 12:13:42 [INFO] ...Add-On Handler Initialization Complete...
2021-10-28 12:13:42 [INFO] Preparing start region for level 0
2021-10-28 12:13:42 [INFO] Done (1,098s)! For help, type "help" or "?"

Server stops and gives same error as before
Is it something not to worry about™ or not to worry about?

Fixed for next release


The configuration file does not exist.

Hello,
Check the link in description download it directly from our forum, maybe the pull of github have made a mistake on one file.

ohh okey, but i havent got a permission to download :(

You need create an account in order to download something from forum.
Today this CMS has been downloaded by 4 users.
Regards,


set working directory automatically

When I use SublimeREPL: R ("driven" by R-BOX of course), I manage to tell it to use the current folder as home folder.  It saves a lot of confusion. The setting is "cwd": "$folder",.
Sadly I haven't found how to do the same in RBox when using the R app (I'm using OS X).
Is it possible, and if yes, how?

This is something I've wanted as well.  I have a fork that does something like this.  It sets the working directory to the project directory every time you send a command.
https://github.com/hafen/R-Box
You can see the relevant changes in rbox/send_text.py at the bottom of here:
https://github.com/randy3k/R-Box/compare/master...hafen:master
The way you activate it is to set a project variable "R_box_setwd" to true.  So your x.sublime-project file might look something like this:
{
	"folders":
	[
		{
			"follow_symlinks": true,
			"path": "."
		}
	],
  "settings":
  {
     "R_box_setwd": true
  }
}

This is pretty unobtrusive but I haven't gotten around to doing a PR for it.

Not sure if I follow, but is what you are looking for different than CMD+\  ?

First create a project in Sublime Text in a folder foo.
Open a file, as R extended (even without saving the file).
In REPL R, thanks to the setting that I mentioned above, by default the current working directory (cdw) is set to the level of the folder foo.
It makes Sublime Text work with projects in the same way as Studio does (the setting of wd is totally transparent to the user and I can refer to any subfolder with indirect addressing, e.g.  ./data/somefile.csv)
@hafen code (that I confess I haven't tried yet but is on my todo list!) should be doing something similar for R terminal (i.e. when we send the code to the R UI): automagically the wd is set to the folder level.
I have never used CMD+\ but it seems to be trying to save a file?

@smartinsightsfromdata
where did you add that pref? and at what point does it change directory? when ou open a project?
I modified my R-box user setting to:
{
"prog": "SublimeREPL",
"cwd": "$folder",
}
i'm not getting anything different.
The problem with CMD+\ is it uses the source files directory.  I usually keep my project-template file in projectdir, in the below structure. in fact, is there a way to turn it off because I hit \ accidentally all the time.
projectdir/
src/
data/

Sending code feature is exported to SendREPL.
Please open an issue there on your will.


Extrenal Feathers installation support

Hi!
Plugin is really great! Thank you!
A few suggestions:

Checking of air SDK path by mxmlc instead of air-sdk-descriptor.
Explicit enumeration of anes.
This two fixes allow to use external Feathers SDK.

Hi,
thanks for the feedback!
Can you explain more about your suggestions

do you mean you aren't using AIR SDK? i mean if you're using a pure AIR SDK or a merged one with Flex SDK you'll get an air-sdk-description.xml xml in both case?
you want to include ANEs with a file path insteadof a folder path? maybe check Gradle-dependencies. You can add ANEs in different way and even separatly for each platforms and or variants:

dependencies {
  nativeLibrary files( "app/src/myANEFolder/myANEFile.ane" , "app/src/myANEFolder/myOtherANEFile.ane" )
  iosNativeLibrary fileTree( dir: "app/src/myANEFolder", include: "*.ane" )
  [variant]NativeLibrary files( "app/src/myANEFolder/myANEFile.ane" , "app/src/myANEFolder/myOtherANEFile.ane" )
}

Hi!

Both Flex SDK and Feathers SDK contains flex-sdk-description, no air-sdk-description. There is no problem to copy air-sdk-description from pure Air SDK folder, but...
You are right, I should enumerate anes in dependencies section, sorry.
Thank you!

I`ve tryed to build my project rigth now with Feathers SDK 3.0beta, official installation with Feathers SDK Manager. Part of my build script:
dependencies {
nativeLibrary files( "app/libs_ane/aqua.ane", "app/libs_ane/flair.PhysicalDensity.ane")
}
compilerOptions "-external-library-path+=$projectDir/app/libs_ane/aqua.ane"

So, if I comment line with compilerOptions, build fail, compiler cant find some classes from ane

ANEs from app/libs_ane are automatically include for compilation, usually you should use dependencies block if you want to add ANEs (or others libraries) outside of default library paths (libs_as, libs_swc and libs_ane) or to make available a library only for a specific platform/variant.
Anyway this doesn't explain why you get that compilation error, are you compiling with gradle task or launcher from intellij/FDT?
Could you as well send me logs from compilation so i can see which arguments are passed to mxmlc.

There is a little difference between mxmlc included in Air SDK and mxmlc from Flex/Feathers SDK.
Pure Air mxmlc accept dir path with -external-library-path, Flex/Feathers not, only paths to files. I made a batch file and tried it with Air mxmlc and Feathers mxmlc. Below version working in both cases (take a look at 4th line from the end, without it Flex/Feathers mxmlc fail).
mxmlc	+configname=air
-debug=true
-source-path+=pathToProject\build\desktop_debug\classes
-source-path+=pathToProject\build\desktop_debug\asLibraries
-library-path+=pathToProject\build\desktop_debug\libraries
-external-library-path+=pathToProject\build\desktop_debug\extensions
-define+=PLATFORM::ANDROID,false
-define+=PLATFORM::DESKTOP,true
-define+=PRODUCT_FLAVOR::ARMV7,false
-define+=PRODUCT_FLAVOR::X86,false
-define+=BUILD_TYPE::DEBUG,true
-define+=BUILD_TYPE::RELEASE,false
-external-library-path+=pathToProject/app/libs_ane/aqua.ane
-output
pathToProject\build\desktop_debug\package\desktop_debug.swf
pathToProject\build\desktop_debug\classes\com\implex\station\MainDesktop.as

Ok thanks for report, i must admit i'm not used to work with pure flex projects, i'll have a look to make plugin compatible with pure flex SDK.

Thank you!

I've been looking to make the changes but i can't figure out how you generate your desktop build (from your mxmlc logs it looks like you're doing a Desktop build cf. -define+=PLATFORM::DESKTOP,true), how can you get rid of AIR SDK? or are you trying to make a web project? but with web projects you can't use ANE so there is something i can't understand here :)


Domain resolution

Description
I've added the domain resolution library and used the same for destination address resolution. The input field is disabled when the domain is resolving and enabled when the resolution is complete. The resolution status is also shown below the input field.
Refers/Fixes
#1
Resulting gif as per the description of #1

Hi, I appreciate the work you've done here but if you're doing work for the Gitcoin bounty please don't start work until you're approved. I approved a different worker for this task who has completed this bounty. Be on the lookout for future bounty postings by Unstoppable Domains! I'll keep you in mind for the next bounty :)


Compiling Issue: haxe.ui.core.Component has no field _allowDispose

I have tried compiling on windows and it said this

D:/FnF-Eternal/.haxelib/haxeui-flixel/git/haxe/ui/backend/ScreenImpl.hx:223: characters 40-53 : haxe.ui.core.Component has no field _allowDispose

please don’t put your compiling issue in the title like that, but also did you run the setup?

Use git versions of HaxeUI:
haxelib git haxeui-core https://github.com/haxeui/haxeui-core
haxelib git haxeui-flixel https://github.com/haxeui/haxeui-flixel

If it asks you about installing haxelib versions, just ignore or type n.

yeah it still  says the same thing

idk what i did wrong here..

and i did run setup.bat

try running these:
haxelib set haxeui-core git
haxelib set haxeui-flixel git

thanks, it worked bro

np


I lost all the devices. Help me Please.

Hi. I have 3 devices (1 smartphone and 2 papers) I lost both papers, and I sold the phone without adding my PC or my new phone. If I reset the account, will I lose the airdrop?
Is there any way to enter the APP without having any of the devices? Thanks

I don't think it's possible. I re-added my phone many times (had to reset my system) and I had to use either the paper or my notebook to validate the new device and use the account I have.

I don't think it's possible. I re-added my phone many times (had to reset my system) and I had to use either the paper or my notebook to validate the new device and use the account I have.

Thanks for answering.  If you reset the account, do you lose the airdrop?


Challenge "No repeats please" do not process the correct numbers

The code is giving the right numbers but the page does not process the tests or is doing it incorrectly, the only correct test is 'returning a number' even that all the others numbers a have tried are OK.
First time I have a problem like this in freecodecamp tests.
Challenge No repeats please has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:47.0) Gecko/20100101 Firefox/47.0.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
var permArr = [], usedChars = [];
function permute(input) {
  var i, ch, chars = input.split("");
  for (i = 0; i < chars.length; i++) {
    ch = chars.splice(i, 1);
    usedChars.push(ch);
    if (chars.length === 0)
      permArr[permArr.length] = usedChars.join("");
    permute(chars.join(""));
    chars.splice(i, 0, ch);
    usedChars.pop();
  }
  return permArr;
}

function factorial(num)
{
    // If the number is less than 0, reject it.
    if (num < 0) {
        return -1;
    }
    // If the number is 0, its factorial is 1.
    else if (num === 0) {
        return 1;
    }
    var tmp = num;
    while (num-- > 2) {
        tmp *= num;
    }
    return tmp;
}

function repet(myarr){
  var cant = 0;
  for(var i = 0; i < myarr.length;i++){
    for(var j = 0; j < myarr[i].length - 1;j++){
      if(myarr[i][j] == myarr[i][j + 1]){
        cant++;
        j = myarr[i].length;

      }
    }
  }
  return cant;
}

function permAlone(str) {
  var cant = factorial(str.length);
  var final = cant - repet(permute(str));
  return final;
  //if(final === 12) return 12;
}

permAlone('aaabb');

@ronaldblanco you have global variables in your code. The tests run the function permAlone over and over so any global variables in your code will change with each test, which will have undesirable results. Move your global variables into your function and you should be able to debug more effectively. Your code is incorrect as permAlone('aab') returns 6 and not 2, as specified in the challenge.
If you get stuck or have questions with a challenge in the future, please first ask questions to the Help Room. Happy coding!

Thanks erictleung, the global variable was important for the correct result, I could eliminate it but having your comment in mind I close it in another function and it worked OK.
So, thank you very much.


Make incremental sync opt-out and fix line index rebuild

Is there a test that would cover this?

No. I might try to add one, but it involves a bit of byte and code unit counting.

bors r+


Use git2 library for git commands

Replace git process commands with git2 lib methods.
https://docs.rs/git2

https://github.com/infinyon/cargo-cvm/pull/7 starts to implement; git2 does not seem to implement all the same functionality needed, namely having difficulty reading the diff blob between trees.


CAN I GET SOME HELP PLS

Hi, SXK here
I don't do JS, so I need help on Step 3 and up.
The Bot I am trying to hack is Myuu Bot.
PLEASE HELP ME!!!

read the readme file, you need the bot token .

read the readme file, you need the bot token .

help me about this
Cannot read property 'get' of undefined

help me about this
Cannot read property 'get' of undefined


Add Hue Iris and Hue Centura Spot

Add Hue Iris and Hue Centura Spot
When create a PR for new measurements to the library, please go the the Preview tab and select the power-profile template:
If you are issuing a new PR for other purpose you can remove all this text.

Power profile

@Morpheus361, thanks for the PR!
Please follow the instructions in the PR body:

please go the the Preview tab and select the power-profile template

That will load a template containing some checks and some more information which needs to be provided.
One of the checklist items is there should only be 1 power profiles submitted per PR. For now it's ok to have two, but please keep that in mind for future contributions, as it makes things easier for me.
One first comment on the PR, the *.csv files must be removed, only *.csv.gz files must be submitted.


Please use native for() loop instead of angular.forEach()

By using the native for() loop and not angular.forEach() (or the native forEach) it is possible to speed up the iterations by more than 60% in Chrome 45. This would indeed speed up the rendering of large datasets.
Test it: https://jsperf.com/angular-foreach-vs-native-for-loop/3

:+1:

Have you identified that the loops are the main cause of lower speed?
(they could represent less than 1% of the overall speed drop)

As mentioned above this issue lacks any thought through insight or information.
There are existing already issues about performance, given that angular foreach is suitable for use by angular, I think its suitable for this library so I'm going to close this.
If someone is willing to profile the library and submit a PR it can be reviewed by the team.

I'm not sure if this qualifies or not. Not to mention if the methodology is correct or not, but you may observe a 50% performance increase by using for() on larger data sets:
http://plnkr.co/edit/NHke0iFeJmcaurJclsvc?p=preview

Real application example of registerState seemingly being slow due to angular.forEach().

The problem is that no attempt has been made to determine if the angular.forEach loops are the cause of any major performance issues in the ui-select library. This isn't demonstrated in your plunkr either as far as I can tell.

@user378230 I'm terribly sorry. It was really late at night and I unintentionally dropped in on this project while I was experiencing the same problem with ui-router. The issue was too similar to what problem I'm facing with so I didn't notice that it's not the repo I should've been commenting on :/


[Bug]: Pause button is covered during pre-download on small window size

Affected Version
Collapse 1.70.11
Bug Behavior

When I switch to the normal window size everything shows up just fine.
Expected Behavior
Just show up completely on small window size.
Steps to reproduce

Switch to small window size
Start Genshin Impact pre-download

Related Issues
No response
Screenshot(s)
No response
Additional Information
No response

i have the same problem here

Hello, sorry for taking a bit to respond
Thank you for reporting the issue, we will look into it!

FYI, a fix will ship as part of the 1.71.3 hotfix. Closing issue, but feel free to reopen if the issue persists after the update.


VS Code Output panel opens to TypeScript Hero on launch

System information

OS: Mac OS 10.12.6
TypeScript Hero version: 2.1.1
VSCode version: 1.19.3

Description
When I open VS Code, it opens the Output panel and selects TypeScript Hero. Does not matter whether I have the entire bottom panel open or closed, it always goes to TS Hero.
Steps to reproduce

Close the Output panel in VS Code
Quit out of VS Code
Open VS Code and wait a few seconds

It's certainly not the end of the world but I keep forgetting to switch back to my Problems panel

Yea that's the logger :-)
I'll have a look at it.

Pretty sure it's your call to show - documentation says this reveals the channel in the UI ;)
https://github.com/buehler/typescript-hero/blob/develop/src/utilities/logger.ts#L78
I'd make a PR but I'm not really in a position to verify the fix at the moment 🤔


Add Miden SDK dependency and update the sample code

This PR updates the new project template for the compiler alpha milestone:

update build options;
add Miden SDK dependency;
update sample code;

This PR is a counterpart of the https://github.com/0xPolygonMiden/compiler/pull/164 and should be merged in sync.

@bitwalker Ping for review.

@bitwalker Ping for review.


ToDo: Brush/Primitive creation

ToDo for the brush/primitive system:

[x] *Creation of an object programmatically
[ ] *Ability to move and resize primitives
[x] *Proper UV mapping for primitives (Required for texture support)
[x] Proper Normal mapping for primitives (Required for planned lighting preview/shaded 3d view)
[ ] Add shapes other than a cube
[ ] *Face specific textures

Not all of these features are required for a Proof of Concept release of Chisel. Only entries marked with a * are needed

One thing that I'm not the best at still is things like uv mapping. so if someone could help get the UV mapping to work consistently across all of the faces that would be very nice!

update: my dumbass didn't realize you could CubeMesh.new() so uh commit coming soon containing a complete overhaul of the primitive creation system using existing godot primitive meshes

#8 is a requirement for this to be closed


[feat][Windows] Change default install location

Describe the problem
Somewhat related to https://github.com/tauri-apps/tauri/issues/7603, but many apps (iirc it's also the default for some electron bundlers?) Install the programs in AppData/Local/Programs instead of AppData/Local and considering the linked issue we may wanna think about having an actual config for it.
Describe the solution you'd like
A config that allows users to select between these for example:

AppData/Local/<appname>/
AppData/Local/<company>/<appname>/
AppData/Local/programs/<appname>/
AppData/Local/programs/<company>/<appname>/

And imo the company name should be an option for the Program Files variant too.
Alternatives considered
No response
Additional context
No response

This seems easy enough to cover by providing an option that uses a base directory
"windows": {
  "defaultInstallLocation": "$LOCALAPPDATA/Programs/$PUBLISHER/$APPNAME",
}

and if unspecified, it fallsback to the current behavior, i.e. based on the install mode.

Does that work for MSI, since when I tried it. I can get it to work for NSIS but not for MSI install?


Handle UnhandledPromiseRejectionWarning in routes

Right now errors are handled in express if they are thrown sync:
app.get("/health", (req, res) => {
  throw new Error("fake error");
});

but not if they are inside an async function
app.get("/health", async (req, res) => {
  throw new Error("fake error");
});

Others have made small wrappers that catches those errors like this:
https://github.com/Abazhenov/express-async-handler
I am making this issue because I searched and could not find that it has been discussed (other than 4 question issues)
Would it be an idea to add this code to express to help? Or add something about async/await here: https://expressjs.com/en/guide/error-handling.html

As for me, docs are covering async error handling pretty good. More over, looks like this issue already fixed in upcoming v5 - #4196
Latest discussion on this can be found in #4186

Yep, the latest version 5 release already has this covered. If the 5.x docs on expressjs.com website do not help clarify this, please by all means open an issue at https://github.com/expressjs/expressjs.com so we can continue to improve our documentation 👍


I think it would be more user friendly if on package manager selection, (pnpm, yarn etc.) it stores selection on localStorage and next time visiting, docs start with last package manager selection.

I think it would be more user friendly if on package manager selection, (pnpm, yarn etc.) it stores selection on localStorage and next time visiting, docs start with last package manager selection.
On another note, can it be synced with other code-group sections so on selection it automatically selects that package manager on others too.
Originally posted by @doganalper in https://github.com/nuxt/nuxt/discussions/25785

I think we can improve this by creating the components similar to undocs:
https://github.com/unjs/undocs/blob/main/layers/core/app/components/global/Pm-Install.vue


New gxapi constants doc feedback

@jbgeosoft compare, for example:
https://geosoftinc.github.io/gxpy/9.3/python/GXAGG.html#agg-render-constants
https://geosoftinc.github.io/gxpy/9.2/python/GXAGG.html#agg-render-constants
gxapi.AGG_RENDER_ADD = 0
In 9.2 I liked the reminder that the constant is gxapi.AGG_RENDER_ADD, including the gxapi. This makes it clear that it is not the constant alone, or gxapi.GXAGG.AGG_RENDER_ADD, which to a Python developer would seem more logical. So the transparency of use is useful.
In 9.2 it was very useful that the value was exposed as a python developer experimenting (which is an important aspect of Python) simply enters the value, in this case 0.  I often just open a Python or iPython console and poke at a gxapi function to see how it behaves, and without auto-complete (or the habits of using it), it is fastest to just enter 0.
Worth a discussion?

Hi Ian,
I can see the argument for keeping the namespace in the docs but I do not agree with your point on using values instead in code. It is a bad programming practice to use a magic value in code instead of the more descriptive constant name if available. When I read a line with the value 0 or 1 or -1 passed to a function I have to consult the docs to figure out what that means. Normally the named module attribute constant is descriptive enough to make that clear.
I also thought of putting the constants within their respective GX classes as attributes at some point but that would lead to either duplication or break backward compatibility if we remove them from the main gxapi module namespace.
I will look at the options of restoring the namespace and values (in the interest of transparency) in the docs.
Cheers,
Jacques

Yes, I completely agree on your "bad practice", and in delivered modules this is important.  But when one is simply experimenting with the Python interpreter, or iPython, we are after speed of experimentation and anything that slows experimentation reduces the efficiency of experimentation. Further, these environments do not have auto-complete, so spelling mistakes further reduce speed and increase frustration for the experimenter.

Ah okay. Note however that iPython has auto complete (hit tab while typing). And it should actually function better than before now that the wrappers are encapsulated with Python code including the docstrings. I think with everything inside a binary module there were some issues with this in the past.

I'll switch to using iPython as my primary experimentation tool.


Skip staging for DOS search result changes

This should have been part of https://github.com/alphagov/digitalmarketplace-functional-tests/pull/475

But we don't have any version of those tests for staging now?  Generally you'd want a skip-preview version of the test that works for staging still, so we maintain test coverage of the features.

@TheDoubleK Have updated with what I think will work?


Bump 1.16.3

This PR upgrades cilium to upstream release v1.16.3

Checklist

[x] I added a CHANGELOG entry
[x] I ran E2E tests in the CI pipelines

Add the following comment to trigger the E2E tests:
/run app-test-suites

/run app-test-suites


update golang and debian version

Hi there!
This is the issue that I encounter when running build command, due to old version of golang:1.16 in Dockerfile:
 => [build 6/7] COPY *.go ./                                                             0.4s
 => ERROR [build 7/7] RUN go build -o /docker-gs-ping-roach                              3.3s
------
 > [build 7/7] RUN go build -o /docker-gs-ping-roach:
#14 3.109 main.go:14:2: //go:build comment without // +build comment
------
executor failed running [/bin/sh -c go build -o /docker-gs-ping-roach]: exit code: 1

Could you update please?

Hello!
This repository had now been migrated to Docker GitHub org: https://github.com/docker/docker-gs-ping-roach
I'm going to archive the current repo, so please direct all pull request to the official Docker repo from now.
:shipit:


Create an engagement phases widget

Issue #:
https://github.com/bcgov/met-public/issues/670
Description of changes:

Add engagement phases widget in public view
Add styled popper and arrow components

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #974 (fdc75d3) into main (3f7d3a5) will increase coverage by 0.02%.
The diff coverage is 74.56%.

@@            Coverage Diff             @@
##             main     #974      +/-   ##
==========================================
+ Coverage   73.58%   73.60%   +0.02%
==========================================
  Files         198      210      +12
  Lines        5318     5430     +112
  Branches      329      333       +4
==========================================
+ Hits         3913     3997      +84
- Misses       1363     1391      +28
  Partials       42       42

Flag
Coverage Δ

metweb
66.62% <74.56%> (+0.36%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.


shorten packet to fit into BLE large packets

Use BLE large packets and use 242bytes or something.

What do you mean? Isn't the MTU of BLE really small? https://stackoverflow.com/questions/38913743/maximum-packet-length-for-bluetooth-le

Do all devices support the ATT packet extension?

James to look at this.

@pelikhan @mmoskal
I've spoke to Martin Woolley at the Bluetooth SIG and he says that not every device will have the same MTU. This means that packet chunking will be required for some devices anyway.
As a result, I think we should account for packet chunking in the design of the BLE physical layer. This means we can keep the current packet size and layout.

(this is a similar approach to what we already do with say USB + HF2)

isn't there a common size? we don't really care about all devices, I think we care about nrf52 and maybe esp32; I assume all laptops/phones would support the max size?

@mmoskal It depends which version of BLE we are talking about. BLE 5.2 (Vol 3 Part F) supports an ATT value of up to 512 bytes.
For version 4.2, the LE data extension has variable support. From reading around, iPhones support an LE extension of around 185 bytes. I believe this is fixed and cannot be modified by applications.
We could design only for BLE 5.2 if we think that's the way to go...

OK, let's go with fragmentation then

I think either way, the current packet and frame layout does not need to be modified to account for BLE specifically. Agree?

Correct.


IDEA : Pull Mitsubishi Into It's own repo

Hi Scotty, I was wondering if you can start a new repo with just the mitsubishi custom component? And then make it so we can import it through Hacs (Home Assistant Community Store). I'm more than happy to help contribute.

To be honest it has been on my to-do list for a while. Sounds like a plan.


States not visible in the initial zoom level

Set the minimum width of a state to 1 so that small states with negligible widths are visible in the initial zoom level
Signed-off-by: Satish Muddana<EMAIL_ADDRESS>

@tahini I couldn't find any trace in 'tracecompass-test-traces' where I can reproduce this issue. I noticed that there are few states that doesn't show up in the time-graph on the initial zoom level though the api returns few states.
The same states starts appearing as you zoom in. This is because the width of those states on the initial zoom level is negligible(almost 0). Hence we need to have a limit on the minimum width to let the user know that there are states present in the chart.

I've just tested it as well and it looks good to me too. Thanks for the patch and the reviews!


Improve styling for long tag names

Currently, the input for entering tag names doesn't fill all available space, which leads to strange behavior when typing long tag names.

Thanks 👍

Awesome, thanks for the fast response!


Document platforms supported

Implement result of discussion.

Suggest disabling Windows2019 CI as that is also not tested in liboqs. @dstebila : What level of Windows (CI) support do we want to attain? fwiw, latest CI problems in oqsprovider were triggered by a failed liboqs Windows2019 CI check (now disabled for oqsprovider0.5.3/liboqs0.9.2 RC).

Contrary to the last comment and as per separate discussion suggest to align platforms supported by OpenSSL with those supported by oqsprovider to ease usage of both at the same time. Depends (mostly) on https://github.com/open-quantum-safe/liboqs/issues/1691.

Moving this issue forward also depends on the level of commitment by the community. Yours truly does not have the bandwidth to provide support for more than basic Ubuntu support. Hence, documenting more platforms as properly supported would be misleading. Same issue as with #94.

@dstebila OK to close this issue? Your demand in yesterday's meeting to spend more cycles on a platform we documented as not CI-supported (IMO effectively disregarding/valuing as irrelevant the PLATFORMS.md file of liboqs) leads me to wonder why we should document the same here knowing it's getting ignored if it can/would be used to guide decisions as to what to focus on?

I was caught off guard yesterday and didn't realize that Ubuntu x86 was not in our PLATFORMS.md file.  Obviously we had x86_64 Ubuntu, and we had x86 Zephyr and x86 Windows, so I was surprised that x86 Ubuntu wasn't in there.  However unless anyone speaks up (with a willingness to contribute) I don't think there's a need to revisit and I'm okay with letting it go as per our PLATFORMS.md file.  @SWilson4 any thoughts?


Paging is incorrect

Where the library builds up the list of files to delete, the paging is incorrect. You don't look at isTruncated unless delimiter is set. We should be looking at the last key.
Deliverables

[ ] Replicate behavior in unit testing
[ ] Address issue
[ ] Update to respect a 'maxDeleteCount` (and set a default for it) such that if there are many, many, many items to delete, it'll stop the building of the list, send a delete command, and then resume building a new list.

Work for this issue will begin begin on branch work-liquid-labs/s3-empty-bucket/21.


admin can turn off events

Related to #3283
User story
As an admin I want to disable events using the page sections settings, e.g. make events appear on this list (even if separating it from contributions is another issue, it could appear as a subheading under contributions until we fix that).

This is requested by a host admin user whose Collectives are meetups but they need them to post the events on meetup.com instead and need to disable them on Open Collective to avoid confusion.

This is done now, right @Betree ? In the new profile page settings.

This is done now, right @Betree ? In the new profile page settings.

This is done now, right @Betree ? In the new profile page settings.

It is indeed, events can now be turned off form the settings:

This is done now, right @Betree ? In the new profile page settings.

It is indeed, events can now be turned off form the settings:


[stable/sealed-secrets] Fix sealed secrets service selector

What this PR does / why we need it:
The current service's selector won't match the pod's name as it is randomly generated.
Using a static annotation fixes the issue.
Which issue this PR fixes
None open
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/assign @olib963
/assign @stefanprodan

@stefanprodan sorry for the somewhat out of topic question but shouldn't the update be available straight away from the repo? Any idea why the sync CI job didn't run on the merge commit?

Nevermind, I check cci config and it seems to be set as a cronjob. Sorry for the noise.


Call chdir to change working directory in older versions of glibc

Motivation:
SPM_posix_spawn_file_actions_addchdir_np_supported and SPM_posix_spawn_file_actions_addchdir_np simply return error for glibc versions < 2.29. This causes SwiftPM tools such as swift package-registry publish to be unusable on Amazon Linux 2, which has an older version of glibc installed.
Modifications:
For older versions of glibc, call chdir to change working directory as an alternative to posix_spawn_file_actions_addchdir_np. This way child process will inherit the working directory path.

@swift-ci test windows

@swift-ci please test

@swift-ci please test Windows

@swift-ci please test

@swift-ci please test Windows

@swift-ci please test

@swift-ci please test

@swift-ci please test

Can we make the switch to fork accessible in some way such that we can run the tests on Linux in both posix_spawn and fork modes?
Separately, I am a bit concerned that we're introducing yet another capability that Foundation doesn't have, it will make it even harder to ever switch away from maintaining our own process code.

Can we make the switch to fork accessible in some way such that we can run the tests on Linux in both posix_spawn and fork modes?

Will look into this once we think the fork impl generally looks ok

https://github.com/apple/swift-package-manager/pull/7187 is alternative approach that implements workaround just for unblocking swift package-registry publish tool.

Doing https://github.com/apple/swift-package-manager/pull/7187 instead. Closing this.


merge from arlac77/npm-package-template

package.json

chore(package): add ^1.31.0 remove ^1.30.1 (devDependencies.rollup)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Post a feed of manual PRs to the #pkg-registration-manual-prs channel on Slack

@logankilpatrick @ericphanson Currently, it looks like these messages are getting posted to the #new-packages-feed channel.
What do I need to modify in order to get the messages to post to the #pkg-registration-manual-prs channel?


[7zip] Version info resource not included and problem with static lib

CMakeLists.txt for 7zip doesn't include rc file, so resulting dll lacks VERSION resource. resource.rc should be added.

diff --git a/ports/7zip/CMakeLists.txt b/ports/7zip/CMakeLists.txt
index 7e93a3bf0..eca6aa3aa 100644
--- a/ports/7zip/CMakeLists.txt
+++ b/ports/7zip/CMakeLists.txt
@@ -311,6 +311,7 @@ add_library(7zip
${SRC_CRYPTO}
${SRC_C}
CPP/7zip/Archive/Archive2.def

CPP/7zip/Archive/7z/resource.rc
)

If you try to build 7zip as static library (with VCPKG_LIBRARY_LINKAGE static) result doesn't work. First lacks headers, and second I believe there is a problem with source code of 7zip itself (it uses global variables to register objects and these global variables are eliminated by linker during linkage of that static lib into resulting exe). So I believe that port should give an error on attempt to build static library.

@elviiix, thanks for reporting this issue.
I will take a look at this issue.

About the static issues : I made a PR but I was not able to do this without "patching" the code so it got rejected.
It could help you: https://github.com/microsoft/vcpkg/pull/8758


connect timeout error

Why do I get a connect timeout error when running your project locally and accessing api/models? Should I use Nginx to proxy port 3000 to openai on my local machine? Or is there something else wrong?

Can you share the response you are getting back here

Thank you for your response. I tried again at the company and it seems to be a network issue in mainland China, rather than a code defect. I will have to find a solution myself. Thank you again.

I met the same issue.
I try to set a proxy to the container ,but not work:

docker run --env ALL_PROXY=socks5://host.docker.internal:7890 -p 3000:3000 -p 7890:7890 chatgpt-u

ME TOO.how to fix it ?


Log module throws error when trying to build basic ble advertising application.

I tried to build a simple ble advertiser application in apps/advertiser and got the error related to implicit declaration for the function and type cast of a variable.
Here is what my target looks like.
targets/blues
    app=@apache-mynewt-nimble/apps/advertiser
    bsp=@apache-mynewt-core/hw/bsp/nordic_pca10056
    build_profile=debug

The error when trying to build the application is as follows. For me the application worked when I did some hacks by commenting the code that throws error.
Error: repos/apache-mynewt-core/sys/log/full/src/log.c: In function 'log_module_get_name':
repos/apache-mynewt-core/sys/log/full/src/log.c:236:12: error: implicit declaration of function 'logcfg_log_module_name' [-Werror=implicit-function-declaration]
  236 |     name = logcfg_log_module_name(module);
      |            ^~~~~~~~~~~~~~~~~~~~~~
repos/apache-mynewt-core/sys/log/full/src/log.c:236:10: error: assignment to 'const char *' from 'int' makes pointer from integer without a cast [-Werror=int-conversion]
  236 |     name = logcfg_log_module_name(module);
      |          ^
cc1: all warnings being treated as errors

GCC version.

/gcc-arm-none-eabi-10.3-2021.10/bin/arm-none-eabi-gcc

OS
Linux mint 64-bit

Hi,
This seems to work for me on master branch. Could you paste output of 'newt info' command (executed in project repo)

yes, here it is.

Apache Newt 1.12.0 / unknown / 2024-06-18_23:28
Repository info:
* apache-mynewt-core: fd2519971422867204b94786be6b274804495a57, (dirty: local changes)
* apache-mynewt-mcumgr: 1c9b642fa3bfb3048b7ed7e75a278667230816c2, 0.4.0
* apache-mynewt-nimble: 675452b62822efd3e3fd28ac55eb166baf8461f4, 1.7.0
* mcuboot: 304fd41980ed929533b9f387dde1b463b0be5b90, 2.0.0

Hi,
Looks like you are using apache-mynewt-core from master branch,  this will not work.
If you want to play with mynewt from master branch you need to used newt tool also from master branch.
NimBLE should likely be also from master branch

Yes, I am using a master branch.

please reopen if this is stil an issue for you


How to customize previous/ next icon on Android 13+

Hi media team 👋  I have a similar question to https://github.com/androidx/media/issues/751, but it was closed, so I decided to file one new issue.
I extended DefaultMediaNotificationProvider and override getMediaButtons to provide our icons for the actions, but the icons are applied only on Android 12 and before, the default skip previous/ forward icons are still displayed on Android 13+, I wonder what's the correct way to customize them? Thanks!

You can't if the media session expose that skip next / previous are available then the buttons are shown.
https://developer.android.com/media/implement/surfaces/mobile
The hack is to not expose the commands are available and publish custom commands.

Thanks @Tolriq , I wonder if there is any plan to provide a proper way to override those icons?

Hi @kevinguitar,
Before Android 13, the buttons are derived from the actions added with the notification. And starting from Android 13, the system derives media controls from PlaybackState, and the available commands (specifically to the "seek to prev" and "seek to next" buttons) - https://developer.android.com/media/implement/surfaces/mobile. So that's why overriding DefaultMediaNotificationProvider.getMediaButtons only works on Android 12 and below.
I think @Tolriq's answer is right here, to customize the icons for "seek to prev" and "seek to next" buttons, you may want to:

Do not expose the commands COMMAND_SEEK_TO_PREVIOUS, COMMAND_SEEK_TO_PREVIOUS_MEDIA_ITEM, COMMAND_SEEK_TO_NEXT and COMMAND_SEEK_TO_NEXT_MEDIA_ITEM as available, thus the system buttons of "seek to prev" and "seek to next" won't display. Please refer to the point 3 in this comment on how to hide these buttons with ForwardingPlayer.
Then you can set the custom buttons (with your customized icons) of the session and the available commands of controllers accordingly. You can refer to this documentation for this step.
Also as stated in https://github.com/androidx/media/issues/751, with the above steps, the commands of "seek to prev" and "seek to next" sent from the bluetooth headset will be ignored  as we don't expose them as available. So from media3 1.2.0, MediaSession.onMediaButtonEvent was added, by overriding this method, the button events of "seek to prev" and "seek to next" can be handled by app before being handled by the library which ignores the unavailable commands.

Hope the above steps can help in your case!

@tianyif Thank you for your reply 👍  I'm good to close the issue


[Bug]: fail to pull from Github

Describe the bug
Hi thank you very much for your program! Could you kindly help me? Thank you again!!
This is a bug and I do not know how to fix it. Here is what happened:

I created some notes in Obsidian in my MacBook and pushed them to GitHub.
Two weeks after, I tried to pull them in my Lenovo ThinkPad but failed (which succeeded all the time before these new notes). The marks in the path also appeared in other notes. And in the screenshots, it is mentioned that only one note cannot be pulled but actually there are also others that are not pulled which I can see in GitHub. So the bug screenshots are as follow:

Relevant errors (if available) from notifications or console (CTRL+SHIFT+I)
No response
Steps to reproduce
I only created notes.
Expected Behavior
No response
Addition context
No response
Operating system
Windows
Installation Method
None
Plugin version
2.22.0

Your file name includes a question mark "What is El Nino and La Nina?.md"  <--
Question marks are special characters on most operating systems, making them part of a filename creates "mild problems" at best.
Consider to use remove the question mark from the file name, and have a top-level heading (# Text) which includes the question mark, or any special character you want.


Include flip_filters=False in convolutional layers

Closes #58. (Sorry the branch is in this repository, I did it via the web interface.)

Thanks for taking of it!


Unreadable files/directories lead to a crash, not a note in the results

At the moment, if there is a file or directory where the user does not have 'read' permission, something like the following happens:
me@pc:~/Cathy-main$ python3 cathy.py scan /media/me/My\ Book/
Scanning: /media/me/My Book ...
Traceback (most recent call last):
File "/home/me/Cathy-main/cathy.py", line 658, in
cat = CathyCat.scan(scanpath)
File "/home/me/Cathy-main/cathy.py", line 567, in scan
t_cat.info.append(t_cat.scandir(0, start_path))
File "/home/me/Cathy-main/cathy.py", line 539, in scandir
(did, fcnt, tsiz) = self.scandir(keepdir, elem)
File "/home/me/Cathy-main/cathy.py", line 539, in scandir
(did, fcnt, tsiz) = self.scandir(keepdir, elem)
File "/home/me/Cathy-main/cathy.py", line 539, in scandir
(did, fcnt, tsiz) = self.scandir(keepdir, elem)
[Previous line repeated 5 more times]
File "/home/me/Cathy-main/cathy.py", line 526, in scandir
for el in os.listdir(start_path):
PermissionError: [Errno 13] Permission denied: '/media/me/My Book/Core2home/.google/picasa/3.0/dosdevices/z:/tmp/systemd-private-774a0ba670754d1fb3a141d9747bc7dc-systemd-logind.service-ieuL7m'
So you either have to run as root, leading to the results being owned by root, or alter the file/directory permissions, which may have Consequences if you need them in the future.
Ubuntu 22.04 so Python 3.10.6

I worked around this with a try block inside the scandir function:
    def scandir(self, dir_id, start_path):
    # the recursive function for scanning a disk
    # it is better to do the recursion yourself instead of using os.walk,
    # because then you can build the Cathy tree more easily (filecount and dirsize)
    tsize = 0
    tsiz = 0
    filecnt = 0
    fcnt = 0
    try:
        for el in os.listdir(start_path):
            elem = os.path.join(start_path, el)
            if os.path.isfile(elem):
                filecnt = filecnt + 1
                cursize = os.path.getsize(elem)
                tsize = tsize + cursize
                dat = os.path.getmtime(elem)
                self.elm.append((int(dat), cursize, dir_id, el))
            if os.path.isdir(elem):
                self.totaldirs = self.totaldirs + 1
                keepdir = self.totaldirs
                dat = os.path.getmtime(elem)
                self.elm.append((int(dat), -keepdir, dir_id, el))
                (did, fcnt, tsiz) = self.scandir(keepdir, elem)
                self.info.append((keepdir, fcnt, tsiz))
            filecnt = filecnt + fcnt
            tsize = tsize + tsiz
    except PermissionError as e:
        print(e)
    return (dir_id, filecnt, tsize)

I'm sure someone could incorporate this into a PR at some point, I didn't have the time, but wanted this here for future reference in case I needed it again at least.


An unexpected network error occurred. Error code: Wsl/0x8007003b

Windows Version
10.0.19045.2728
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 20
Other Software
No response
Repro Steps
open terminal and see the error today:
An unexpected network error occurred.
Error code: Wsl/0x8007003b

[process exited with code<PHONE_NUMBER> (0xffffffff)]

It works normal yesterday, I didn't change anything.
Expected Behavior
see a shell
Actual Behavior
see above
Diagnostic Logs
No response

Sorry I reboot my computer and it came back again.


Allows cannot be done from cmdlet

Updated document to remove examples of allow except advanced delivery.

Docs Build status updates of commit 86853b4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-TenantAllowBlockListItems.md
:white_check_mark:Succeeded
View (exchange-ps)

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 docs.microsoft.com contributor guidesPost your question in the Docs support channel

@dhairyya Please implement the suggested changes. Thanks

Docs Build status updates of commit b2a8f60:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Match JSON structures

Is it possible to match two JSON documents to find the difference using the parser?

json-iterator does not provide such function.


Build apoc.schema.assert

Potential customer says that we're missing apoc.schema.assert

We have another community report -> https://discord.com/channels/842007348272169002/890969003374289016/1171708792040329236
I would like to emphasize that there is no easy way to drop all constraints/indexes without this. The workaround combination of running and passing SHOW INDEX INFO; and DROP INDEX queries. Similar goes for the constraints.
We have that capability in GQLAlchemy, but if users are migrating, this is probably not their current driver used.

@as51340 How much more time do you need to finish this?

I will close this issue here and open it on Memgraph repo.


Not asking variables

Hi, in your readme i can read: 'To integrate your Samsung Soundbar with Home Assistant using YASSI, you will be asked for the following variables'. But my HA does not ask for these variables. What am i doing wrong? What lines i have to put into my configuration.yaml?
Best Regards,
Pim

Currently the preferred way of using the Integration is through the web interface of HomeAssistant.
When adding the integration through the "Devices" Tab, you should be prompted with a form to input your settings.
Which version are you using?

Thnx for your quick reply. I'm using 2024.6.4 and i used HACS to install the integration.

Hi, i tried it again (uninstalling, re-installing) and now it works!
Thnx!!
Pim

Happy to hear that :)
Have fun with the integration ✌

Thnx!!
Van: Samuel Spagl @.>
Verzonden: zaterdag 29 juni 2024 07:54
Aan: samuelspagl/ha_samsung_soundbar @.>
CC: PimStevens @.>; Author @.>
Onderwerp: Re: [samuelspagl/ha_samsung_soundbar] Not asking variables (Issue #37)
Happy to hear that :)
Have fun with the integration ✌
—
Reply to this email directly, view it on GitHub https://github.com/samuelspagl/ha_samsung_soundbar/issues/37#issuecomment-2197995446 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABTBCH7PSKA5MRW6UUDHGH3ZJZDXTAVCNFSM6AAAAABKBOCMWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJXHE4TKNBUGY .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/ABTBCHYEC3IMVRT6BX5S4CLZJZDXTA5CNFSM6AAAAABKBOCMWOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUDAK73M.gif Message ID: @.*** @.***> >