@ApiModelProperty value doesn't show in Swagger UI

Hi there, I got a problem that @ApiModelProperty value doesn't show in Swagger UI, demo code as below:
@ApiModel
public class RequestPayload<P> implements Serializable {

    @ApiModelProperty(value = "Search Parameters")
    private P param;

    @ApiModelProperty(value = "page size")
    private Integer pageSize;

    @ApiModelProperty(value = "ppage number")
    private Integer pageNum;
    @ApiModelProperty(value = "pageable")
    private Boolean countable = DEFAULT_COUNTABLE;

    @ApiModelProperty(value = "order by")
    private String orderBy;

    // other code goes below...
}

@ApiModel
public class TenantVo {

    @ApiModelProperty(value = "tenant name")
    private String name;
    @ApiModelProperty(value = "tenant code")
    private String code;
    @ApiModelProperty(value = "tenent icon")
    private String logo;
    @ApiModelProperty(value = "tenant detail desc")
    private String description;

}

Then I have a Object named Tenant, which I will use it in Controller as a request parameters like RequestPayload, but after Swagger UI show up, I found the description of param in RequestPayload was missing?

Is there wrong usage of @ApiModelProperty, if so, help. Thanks all.

I am having the same issue with both the value and required field


Bug/hra to publications

Moved HRA data to publications.

@vitorbaptista
please take a look

@roll If I understood correctly, the process for linking trials to HRA "publications" is:

For each HRA publication, do:
For each HRA identifier, do:
1. Look for a trial with that identifier
2. If found, link the trial with the publication
3. If not, create a new "empty" trial with just the identifier (similar to what we do with pubmed)

I'm not sure, but I'd think that it should be changed to:

For each HRA publication, do:
Look for all trial with any of the identifiers we got from HRA;
If found, link the trial with the publication and merge the HRA identifier list with the current trial identifier list (so we add any new identifiers from HRA);
If not, create a new "empty" trial with all identifiers we found from HRA.

The reasoning behind this is that I think there's a 1-to-1 relationship between a research summary and a trial, so it doesn't make sense to use all identifiers as if each of them were relative to a different trial.
Considering our deadline (and that I'm not sure if my interpretation is correct), I'm happy to simply keep the code as is removing the creation of new empty trials and create a new issue about this so we can do it later.
What do you think?

@vitorbaptista
Relationship is 1-to-many as minimum because there could be other research summaries sources. Also I'm not sure that one research summary could not be able to point to a few trials in some cases.
So I don't see a difference between research_summaries and publications for now. WDYT?

Duh, you're right. The relationship is 1 trial for many summ) research summaries. However, are you sure that the other way is 1 to many as well? As in, a single research summary from HRA can point to many trials?
If I remember correctly, there's only one identifier of each type (nct,...), right? If that's true, it seems to me as an indicator that the research summary only talks about a single trial. If that wasn't true, I'd expect it to be possible to link a research summary to multiple trial ids from the same registry.

We need an additional consultation about it when we will be adding new research_summary entity.
But for now I think we need to use this approach (pubmed like) because it's anyway will be slightly re-written after new entity will be added.


Question about the batch size.

Hi, thanks for sharing the fabulous work!
I wonder what batch size you guy used for the training. The paper does not mention the batch size.
Is it really that you guys used batch size of 1 for the training according to the config file?
Thanks!

Yes, we set the batch size to 1 during training and testing.


Start Releasing vscode-js-annotations VSIX files on Github

For any instance which the VS marketplace may be down or difficult to access, it may be good to host VSIX files for the extension here on GitHub as well.

0.6.1 is now in the releases page


[liblzma] build failure

Host Environment

OS: Linux
Compiler: gcc version 10.2.0

To Reproduce
Steps to reproduce the behavior:
./vcpkg install liblzma
Failure logs
In file included from /scratch/general/lustre/miheer/hw/timeloop-accelergy/vcpkg/buildtrees/liblzma/src/v5.2.5-77236e0e89.clean/src/liblzma/common/common.h:17:0,
                 from /scratch/general/lustre/miheer/hw/timeloop-accelergy/vcpkg/buildtrees/liblzma/src/v5.2.5-77236e0e89.clean/src/liblzma/common/block_encoder.h:16,
                 from /scratch/general/lustre/miheer/hw/timeloop-accelergy/vcpkg/buildtrees/liblzma/src/v5.2.5-77236e0e89.clean/src/liblzma/common/block_encoder.c:13:
/scratch/general/lustre/miheer/hw/timeloop-accelergy/vcpkg/buildtrees/liblzma/src/v5.2.5-77236e0e89.clean/src/common/mythread.h:138:52: error: expected ‘;’, ‘,’ or ‘)’ before ‘set’
 mythread_sigmask(int how, const sigset_t *restrict set,
                                                    ^
/scratch/general/lustre/miheer/hw/timeloop-accelergy/vcpkg/buildtrees/liblzma/src/v5.2.5-77236e0e89.clean/src/common/mythread.h: In function ‘mythread_create’:
/scratch/general/lustre/miheer/hw/timeloop-accelergy/vcpkg/buildtrees/liblzma/src/v5.2.5-77236e0e89.clean/src/common/mythread.h:162:2: warning: implicit declaration of function ‘mythread_sigmask’ [-Wimplicit-function-declaration]
  mythread_sigmask(SIG_SETMASK, &all, &old);
  ^
In file included from /scratch/general/lustre/miheer/hw/timeloop-accelergy/vcpkg/buildtrees/liblzma/src/v5.2.5-77236e0e89.clean/src/liblzma/common/block_encoder.h:16:0,
                 from /scratch/general/lustre/miheer/hw/timeloop-accelergy/vcpkg/buildtrees/liblzma/src/v5.2.5-77236e0e89.clean/src/liblzma/common/block_encoder.c:13:
/scratch/general/lustre/miheer/hw/timeloop-accelergy/vcpkg/buildtrees/liblzma/src/v5.2.5-77236e0e89.clean/src/liblzma/common/common.h: At top level:
/scratch/general/lustre/miheer/hw/timeloop-accelergy/vcpkg/buildtrees/liblzma/src/v5.2.5-77236e0e89.clean/src/liblzma/common/common.h:107:27: error: expected ‘;’, ‘,’ or ‘)’ before ‘in’
   const uint8_t *restrict in, size_t *restrict in_pos,
                           ^
/scratch/general/lustre/miheer/hw/timeloop-accelergy/vcpkg/buildtrees/liblzma/src/v5.2.5-77236e0e89.clean/src/liblzma/common/common.h:150:2: error: unknown type name ‘lzma_code_function’
  lzma_code_function code;
  ^
/scratch/general/lustre/miheer/hw/timeloop-accelergy/vcpkg/buildtrees/liblzma/src/v5.2.5-77236e0e89.clean/src/liblzma/common/common.h:268:51: error: expected ‘;’, ‘,’ or ‘)’ before ‘in’
 extern size_t lzma_bufcpy(const uint8_t *restrict in, size_t *restrict in_pos,
                                                   ^
/scratch/general/lustre/miheer/hw/timeloop-accelergy/vcpkg/buildtrees/liblzma/src/v5.2.5-77236e0e89.clean/src/liblzma/common/block_encoder.c:49:27: error: expected ‘;’, ‘,’ or ‘)’ before ‘in’
   const uint8_t *restrict in, size_t *restrict in_pos,

Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.

buildtree.tar.gz

buildtree.tar.gz

This prevents installing boost-iostreams.

This prevents installing boost-iostreams.

@vmiheer, it looks the gcc revision(4.8.5) is too old in your machine, I built it successfully with gcc 7.5.0, could you upgrade the gcc and try again?

@vmiheer, it looks the gcc revision(4.8.5) is too old in your machine, I built it successfully with gcc 7.5.0, could you upgrade the gcc and try again?

My bad. It would be great if vcpkg gives this as error. 😊

My bad. It would be great if vcpkg gives this as error. 😊

How do you change the gcc version to be used? I have gcc 7.5.0 in my current environment (checked using gcc --version)but during liblzma installation, it fails and the build log says -- The C compiler identification is GNU 4.8.5. I have
set(CMAKE_C_STANDARD 99)
set(CMAKE_C_STANDARD_REQUIRED ON)

inside vcpkg/build-trees/liblzma/src/...clean/CMakeLists.txt.

Check your environment variables. vcpkg will not override CC et al.

The weird thing is that, when I run ./bootstrap-vcpkg.sh before installing liblzma, I see The CXX compiler identification is GNU 7.5.0 but something changes during liblzma installation.
@dg0yt What environment variables are you referring to? Is there a way to force the gcc compiler to be used?

I named the variable... This issue was closed two years ago. Please open a new issue with a proper report of your problem.


chore: consolidate sell routes in app

This PR resolves ONYX-948
Description
This PR consolidates the different submit artwork routes in app to be using the same url. This will make it easier for us to track conversions and implement proper deep links.
Bonus:
I noticed that the sell tab route is /sales so I fixed that to be /sell like in web
Starting the flow from the sell tab
https://github.com/artsy/eigen/assets/11945712/21696d31-15ab-40c8-b797-445d3d9871dc
Starting the flow from my collection
https://github.com/artsy/eigen/assets/11945712/c233d776-2dd6-4f7b-af28-21c6085fa8ad

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

consolidate sell routes in app - mounir

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (consolidate sell routes in app - mounir)

  Generated by :no_entry_sign: dangerJS against e56ebbe411e0fce19b1a0ebf8faa24f2dd1b8f06


Possible Targeted-User Specific Features

[ ] Show all Runners with one command / keyboard shortcut
[ ] Show all Customers with one command / keyboard shortcut
[ ] Mark all jobs of a certain tag done
[ ] Mark all jobs assigned to a certain Runner done
[ ] Assign selected jobs to a Runner
[ ] Tag multiple Customers at once with one or more tags
[ ] Tag multiple Runners at once with one or more tags
[ ] Clear one or more tags from multiple Customers at once
[ ] Clear one or more tags from multiple Runners at once
[ ] Auto updating interest rates and actions to be taken

Defer to v2.0?


New extension: HasLanded(for PlatformerObject behaviour)

Description
this extension checks if an object(which has PlatformerObject behavior) has landed on the floor
here is a playable example https://liluo.io/instant-builds/f4559a13-797a-4c3d-8339-b8aafdaf71ae
How to use the extension

open or create a project(well basically a platformer game will do(as long as the object you wanna check if landed has the platform object behavior
create an event
add the condition "HasLanded"
setup the condition to how you like it
test

Checklist

[X] I've followed all of the best practices.
[X] I confirm that this extension can be integrated to this GitHub repository, distributed and MIT licensed.

Example
Example
Extension file
Extension

!update
Extension.zip


tests: kernel: timer: timer_api: disable CONFIG_PM for it8xxx2_evb

The it8xxx2 chip needs more time to resume from low-power mode and it results in the failure of the kernel.timer.tickless test. This commit disables CONFIG_PM for the it8xxx2_evb platform to address this issue.

After addressing the timer drift issue in PR#69891, the problem about timer_api tickless test has been resolved. Consequently, this PR can be removed.


Prevent SQL injection

https://github.com/May-Take-A-Second/ReviewsAPI/blob/f66e922cdc3da5ef3a3776896943ac5f8f3e44cd/server/models/reviews.js#L21
You should be using either ? or connection.escape() to prevent malicious values being passed into your variables.
Please see npm mysql docs.

Thanks @eric-do for the suggestion. I went ahead and use escape() for all of my INSERT queries. You can refer to this commit f1c9c4813e5bcb8e55f6b90548d795d1f50ba1ec to see the changes.


Worker options to opt-in the behavior to get empty entries in payload

We recently updated host to include empty entries when sending the message payload to out of process layer when the IncludeEmptyEntriesInMessagePayload capability is present. We need to allow users to opt in to this behavior via worker options when they bootstrap the function app.
Something like
var host = new HostBuilder()
    .ConfigureFunctionsWorkerDefaults(builder =>
    {
    }, options =>
    {
        options.IncludeEmptyEntriesInPayload = true;
    })
    .Build();

Needs to agree on naming of property.
Should it be on the root level or should we move it under some other property to be better organized.

This is the last step to resolve bug #876 and enable us to enable batching again on the eventhub trigger.

@satvu @kshyju: is there any contribution I can do to speed this up a little bit? If you have any guidance on the two open questions I could look if I can make a contribution for this.


Address book: transaction list isn’t displayed for regular and maximum anonymity addresses

Bag description
On the address details screen the transaction list with incoming UTXO isn’t displayed and there is a possibility to expire address while transaction to this address is in progress
Steps

Create regular or max anonymity address in the IOS wallet
Send from another wallet transaction to this address
Go to address book in the IOS wallet > address details

Actual: on the address details screen the transaction list with incoming UTXO isn’t displayed and there is a possibility to expire address while transaction to this address is in progress
Expected: on the address details screen the transaction list with incoming UTXO is displayed and there isn’t possibility to expire address while transaction to this address is in progress
https://user-images.githubusercontent.com/68267190/151929064-5f3149bd-eda5-4ef0-9989-cdf881772ebc.mp4

Checked


win 64bit link is broken

I get a 404 when i click on it

Fixed by merging PR #13. Thanks for reporting it.


Flutter Web runner closes silently on (probably JS) exception

I have some issue in my Flutter Web app.
But I can't debug it, because when issue occurs chrome window closes itself silently, without any logs in stdout.
I can see issue in JS console, if I use flutter build web and run index.html manually, but it's minified so not useful at all.
It's serious bug that drives development nearly impossible and frustrating.
Please add copy of JS console output to stdout or just don't close window.

Ping @jonahwilliams cause I know, that he wrote Chrome runner logic

I can see issue in JS console,

Could you include that issue in this report?

@jonahwilliams yeah, sure
Uncaught TypeError: J.ag(...).i is not a function
    at Object.b9 (main.dart.js:2238)
    at Bo.$2 (main.dart.js:34217)
    at Object.Tu (main.dart.js:1527)
    at Object.Oh (main.dart.js:3050)
    at cl (main.dart.js:3764)
    at RTCPeerConnection.<anonymous> (main.dart.js:34139)
main.dart.js:18036 Uncaught NoSuchMethodError: method not found: 'call'
Receiver: Closure 'minified:Bs'
Arguments: [Instance of 'minified:cA', Instance of 'minified:C']
    at Object.d (http://localhost:63342/flutter-webrtc-master2/flutter-webrtc-master2/example/build/web/main.dart.js:1567:3)
    at Bs.kS (http://localhost:63342/flutter-webrtc-master2/flutter-webrtc-master2/example/build/web/main.dart.js:18036:26)
    at Object.RW (http://localhost:63342/flutter-webrtc-master2/flutter-webrtc-master2/example/build/web/main.dart.js:2272:32)
    at Object.hI (http://localhost:63342/flutter-webrtc-master2/flutter-webrtc-master2/example/build/web/main.dart.js:1521:10)
    at Object.Ts (http://localhost:63342/flutter-webrtc-master2/flutter-webrtc-master2/example/build/web/main.dart.js:1544:10)
    at Object.Tu (http://localhost:63342/flutter-webrtc-master2/flutter-webrtc-master2/example/build/web/main.dart.js:1529:41)
    at Object.Oh (http://localhost:63342/flutter-webrtc-master2/flutter-webrtc-master2/example/build/web/main.dart.js:3050:27)
    at cl (http://localhost:63342/flutter-webrtc-master2/flutter-webrtc-master2/example/build/web/main.dart.js:3764:19)
    at RTCPeerConnection.<anonymous> (http://localhost:63342/flutter-webrtc-master2/flutter-webrtc-master2/example/build/web/main.dart.js:34143:85)

As I said, it's minified
I stuck on this issue while working on unpublished fork of flutter-webrtc plugin, it's open source, so I can provide zip with whole project, if you wish

Could you edit your build.yaml and change the dart2js_args options to include --minify=false?
Here's an example build.yaml file: https://github.com/dart-lang/build/blob/master/build_web_compilers/build.yaml

@yjbanov I can't find build.yaml file anywhere
Did I mentioned, that I'm using unforked flutter web on master and not the fork?

I've updated flutter to v1.9.1-pre.59, issue is gone


Track by in multiple select

If specify multiple and track by simultaneously I recieve the error "Error: [ngRepeat:dupes] Duplicates in a repeater are not allowed."
Demo. Just select 2 items in select, and you will get the error.
http://plnkr.co/edit/taoUPj40sWnjuRD5zVCh?p=preview

Same here.

Use the forked version and everything works great with Angular 1.3.
Try it: https://github.com/richardallen/ui-select

@chuyik But, won't load a selected values.

Can this be fixed? Currently multiple entries are unsupported because of this.

same here, using angular 1.2.25, ui-select latest version


Fixes view dragged title has a number at the end

This PR fixes #86029

Closing this as this no longer applies because the entire header is now draggable.


Do not rely on ansible_user

Proposed Changes
Instead of reading ansible_user (which may or may not be defined), this patch lets the roles rely on Ansible facts.
Checklist

[x] Tested locally
[x] Ran site.yml playbook
[x] Ran reset.yml playbook
[x] Did not add any unnecessary changes
[x] Ran pre-commit install at least once before committing
[x] 🚀

Thank you!

First off - Thanks for the great playbook!
When running the playbook and following the video, everything worked great until I went to copy the Kube Config file
scp ansibleuser@<IP_ADDRESS>:~/.kube/config ~/.kube/config
I was expecting the config file to be on the master node within my ansible user home directory.  It wasn't there but instead it was in the root directory.
It appears that when using become: true the cube config is copied to the root directory instead of the user home directory.
Maybe I have this wrong but in attempting to look this up what I have found so far from really old posts is become: true changes the ansible_user_dir to root instead of the expected ansible_user home directory.  The end result is that you can't scp the kube config file from the master node root directory to the ansible machine.
Thanks for looking into this.


fix: use unified memo field and  extract memo value from different coin transaction implementation/labels

✅ Checklist

[x] npx changeset was attached.
[ ] Covered by automatic tests.
[ ] Impact of the changes:

...

📝 Description

Handle memo tag values extracted from different transaction properties
Unify memo tag field for SOL, TON, STELLAR

https://github.com/user-attachments/assets/9cd8ee64-92b8-488f-a419-28a4eea4c4d1
https://github.com/user-attachments/assets/816b45e4-970c-4873-bd0d-ceb07c73dc64
https://github.com/user-attachments/assets/7a35935b-5d81-4a81-90e9-3447dcd4a4b2
https://github.com/user-attachments/assets/f1fa17e0-3a78-449b-9080-47cdde56ec8c
https://github.com/user-attachments/assets/810b725d-2a38-49c9-9115-a6561ad7ffcb
https://github.com/user-attachments/assets/b992eacc-ec81-4d93-afcb-3f669dd775c2
https://github.com/user-attachments/assets/95883f91-79b4-41aa-a7cb-1c3199d9d8c9

❓ Context

JIRA or GitHub link:

https://ledgerhq.atlassian.net/browse/LIVE-14902
https://ledgerhq.atlassian.net/browse/LIVE-14910

🧐 Checklist for the PR Reviewers

The code aligns with the requirements described in the linked JIRA or GitHub issue.
The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
There are no undocumented trade-offs, technical debt, or maintainability issues.
The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
Any new dependencies have been justified and documented.
Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

@KVNLS i've fixed this https://github.com/LedgerHQ/ledger-live/pull/8392#pullrequestreview-2444699596 review , it's an existing functional bug that in my opinion should be fixed without feature flag because it has an impact on the old and new memoTagFlow


.cmake Adaptor?

The main reason why I still stick to CMake is its builtin modules which provide compiler detection/package search/etc. It is unnecessary to reinvent the wheels, so I was wondering is it possible to provide (at least kind of) .cmake support and wrap these modules into xmake utilities?

xmake also provide these features, so don't intend to wrap them.

Compiler detection of xmake is perfect
"Package search" is working on the way. You could have a look branch repo
Thanks for caring :smiley:

Thanks for reply. I'm just a little bit worried that there might be a lot more to do than you think. Take C++ as an example, we may need:

feature test: whether this compiler supports variadic template, or is it claimed supported but actually buggy?
compiler detection: we may be running clang frontend of MSVC, and many other corner cases
compiler*platform-specific options: including sanitize/floating point accuracy/etc.

Supporting .cmake may reduce some kind of work, I guess.

I'd like to have a look into it this week

And you can add  add_cxflags("-fsanitize=address", "-ftrapv") directly.
If compiler does not support these flags, It will ignore it automatically.

xmake will detect msvc on windows by default automatically. If you want to use other compiler, you can set it manually. for example:
$ xmake f --cc=clang --cxx=clang --ld=clang
$ xmake

@waruqi
IIUC, according to your example xmake currently has no plan for builtin feature detection based on compiler version, instead, it provides basic utilities like add_cxxfuncs to allow custom detection. The main drawbacks of the absence of builtin has_variadic_template/support_cxx_11 include:

We have to compile during configuration for many times if we have a lot of orthogonal features to detect
Some feature detections are not as simple as add_cfuns(sigsetjmp((void*)0, 0) and requires heavy templates and domain knowledge

Actually I'm more willing to see something like enable_sanitize_address which adds proper flags for every compiler supporting this feature. Hard-coded/compiler-dependent flags are not encouraged from my perspective, and I want to completely eliminate these in my build scripts.
I'm referring to Clang/C2 frontend of MSVC. Take a look into CMake's patch set, and you'll find that checking _MSC_VER is not enough.

@htfy96 Yes, xmake currently has no plan for builtin feature detection based on compiler version.

We have to compile during configuration for many times if we have a lot of orthogonal features to detect

This is only configured and detected once

Some feature detections are not as simple as add_cfuns(sigsetjmp((void*)0, 0) and requires heavy templates and domain knowledge

This is not a problem, the detection rules can be improved more flexibly.

Hard-coded/compiler-dependent flags

xmake takes the gcc flags as the standard and will map to other compiler flags smartly.
So these flags (-fsanitize=addres, -fomit-frame-pointer) are not hard-coded/compiler-dependent, you can see #71
About enable_sanitize_address ,  why not do the testing directly in the code?
#if (defined(__has_feature) && __has_feature(address_sanitizer)) || \
        defined(__SANITIZE_ADDRESS__)
...
#endif

@waruqi
Thanks for detailed reply. One more question, how can I switch flags based on whether a code snippet can compile without error? For example, I want to detect constexpr support:
option("test_cxx11_constexpr")
    add_defines_if_ok("CXX11_CONSTEXPR")
    add_cxxfuncs("has_cxx11_constexpr{ constexpr int f(int x) { return x ? x+f(x-1) : 0; } constexpr int x = f(5); static_assert(x == 15); ")

option("test_cxx14_constexpr")
    add_defines_if_ok("CXX14_CONSTEXPR")
    add_cxxfuncs("has_cxx14_constexpr{ constexpr int f(int x) { int sum=0; for (int i=0; i<=x; ++i) sum += i; return sum; } constexpr int x = f(5);  static_assert(x == 15);")

This doesn't seem to work.

Actually I'm more willing to see something like enable_sanitize_address

Now for flags adding, xmake uses gcc flags mapping
Advantage:

easier to remember

Disadvantage:

Some flags are not mapped yet. See #71 I wanna to add a flag for coverage but -coverage is not mapped. And compilers are difficult so some flags cannot be implemented on some compiler

I agree with generic flags as what you said as a second choice

A good problem @htfy96
Seems that not supported yet, right? @waruqi

And consider this case: user specified -fsanitize=address in xmake.lua, but actually the machine is running an older version of C2 on MSVC(e.g. Clang/C2 in MSVC14 Update 1) which has not sanitize support. How do you deal with such kind of issue?
The initiative of this issue is that compatibility/detection is extremely difficult. Manual checking could be very time-consuming and complicated. It is difficult to implement your own compiler database like CMake  from scratch as well, since if you want to support N compilers, each of which has K versions, and you have P features to check, the database will soon balloons to N * K * P. Therefore, the best choice in this case is to adapt existing CMake databases.

@htfy96 add_cxxfuncs and add_cfuncs only used to detect if the function interface exists. I will add add_snippet() to detect other feature support in the future, for example:
option("constexpr")
    add_snippet("cxx", "constexpr int f(int x) { int sum=0; for (int i=0; i<=x; ++i) sum += i; return sum; } constexpr int x = f(5);  static_assert(x == 15);")

And consider this case: user specified -fsanitize=address in xmake.lua, but actually the machine is running an older version of C2 on MSVC(e.g. Clang/C2 in MSVC14 Update 1) which has not sanitize support. How do you deal with such kind of issue?

If has not sanitize support, add_cxflags("-fsanitize=addres") will ignore it when building project

The initiative of this issue is that compatibility/detection is extremely difficult. Manual checking could be very time-consuming and complicated. It is difficult to implement your own compiler database like CMake from scratch as well, since if you want to support N compilers, each of which has K versions, and you have P features to check, the database will soon balloons to N * K * P. Therefore, the best choice in this case is to adapt existing CMake databases.

Therefore, even using the CMake database does not guarantee all compilers. The most stable way is that try to compile it to detect it

@waruqi
Validity check of snippets cannot be bulk-executed since an error in one snippet can easily affect others.
The -fsanitize example actually targets the underlying problem compiler detection: in order to detect whether this flag is supported, you need to compile with it during configuration instead of mapping it manually in scripts, as what you just mentioned.
Moreover, some features cannot be detected at compile-time. For example: <regex> in g++-4.8 defined all functions with empty implementation.

@htfy96 add_cxxfuncs and add_cfuncs only used to detect if the function interface exists. I will add add_snippet() to detect other feature support in the future, for example:

:+1: @waruqi
add_snippet() is great. Just like my way to check if libreadline exists in Makefile in #80 . I'd like to try writing a first implement these days

The initiative of this issue is that compatibility/detection is extremely difficult. Manual checking could be very time-consuming and complicated. It is difficult to implement your own compiler database like CMake from scratch as well, since if you want to support N compilers, each of which has K versions, and you have P features to check, the database will soon balloons to N * K * P. Therefore, the best choice in this case is to adapt existing CMake databases.

You're right @htfy96

in order to detect whether this flag is supported, you need to compile with it during configuration instead of mapping it manually in scripts, as what you just mentioned.

xmake do you say, compile with it during configuration to check whether a flag is available @htfy96

Moreover, some features cannot be detected at compile-time. For example:  in g++-4.8 defined all functions with empty implementation.

Great example :+1: @htfy96
Maybe best solution is

detection first, database second
flag mapping first, generic flags second

Validity check of snippets cannot be bulk-executed since an error in one snippet can easily affect others.

You can define one option with a snippet to detect only one feature.

The -fsanitize example actually targets the underlying problem compiler detection: in order to detect whether this flag is supported, you need to compile with it during configuration instead of mapping it manually in scripts, as what you just mentioned.

You also can use option to detect flags. Although I don't support it now, I'll improve it.
option("fsanitize_address")
      add_cxflags("-fsanitize=address")  <-- I'll improve it here.
      add_ldflags("-fsanitize=address")

@TitanSnow

Just like my way to check if libreadline exists in Makefile in #80 . I'd like to try writing a first implement these days

I've detected it in xmake.lua, just old core/makefile doesn't support. You can see core/src/xmake/xmake.lua
add_cfunc("API", "readline", nil, {"readline/readline.h"}, "readline")

@htfy96

Moreover, some features cannot be detected at compile-time. For example:  in g++-4.8 defined all functions with empty implementation.

But many vendors will modify the compiler, even the same version of the compiler does not guarantee that many implementations exist or not.

Yep @waruqi  but this is just a example that not so exact  and other code snippets could be used

So there is the same problem with the database.

Database is not no use at all! @waruqi

@TitanSnow  @htfy96  So I'm still more inclined to use compile detection. Perhaps xmake detection is not flexible enough, but I will improve it.

@waruqi
It is OK to just provide minimal feature detection support, I guess. Users have to deal with it something like Boost.config, though in many cases we don't want to introduce Boost dependency. Quite looks like the way of premake.
Thanks for your effort. It would be piles of work to implement proper compiler detection.

@htfy96 😄

add_snippet() is great. Just like my way to check if libreadline exists in Makefile in #80 . I'd like to try writing a first implement these days

Sorry, but for me there's still trouble to implement add_snippet()

@TitanSnow I will implement it in last these days.

@htfy96 I have quickly looked through *.cmake in cmake/modules
I found that your idea is great :+1:
Many things are useful like findQt.cmake
In fact trying to link with Qt in xmake might be troublesome or I don't get the way? @waruqi
And FindGit.cmake might be useful for you @waruqi . I know you recently work on branch repo requires git
If .cmake adapter comes up in the future, I think things will be easier and better.
In this way, I agree with @htfy96
My option is on-fly detection is the first hand, off-fly data & script is the second hand
This idea should be minded @waruqi
Does you have a preliminary way to implement the adapter, @htfy96 ? Or could you do some work on this and finally give a PR?

A preliminary roadmap

[ ] .cmake adapter

[ ] cmake syntax support
[ ] .cmake files include
[ ] compiler database support

[ ] work with xmake detection

[ ] find_package support

[ ] work with xmake-repo

@TitanSnow

And FindGit.cmake might be useful for you @waruqi . I know you recently work on branch repo requires git

Git is a necessary component for xmake require action, not just looking for it. So it's not enough to use FindGit.cmake. xmake will install winenv with git when using xmake require.
xmake also support tool detection and only for internal detection. In the future I will provide public interfaces to find tools like find_tool , find_package ..

If .cmake adapter comes up in the future, I think things will be easier and better.

Xmake has its own design pattern, not depend on .cmake or wrap it. But I will provide similar functionality in the future.

@htfy96 I have implemented add_cxxsnippet to detect compiler feature. for example:
-- define option for detecting constexpr feature
option("constexpr")
    set_languages("cxx14")
    add_defines_if_ok("CONSTEXPR_ENABLE")
    add_cxxsnippet("constexpr", "constexpr int f(int x) { int sum=0; for (int i=0; i<=x; ++i) sum += i; return sum; } constexpr int x = f(5);  static_assert(x == 15);")

-- add target
target("test")

    -- set kind
    set_kind("binary")

    -- add files
    add_files("src/*.cpp")

    -- need constexpr
    add_options("constexpr")

I'll focus on improving option to provide more flexible detection, but now I need to finish xmake require first.
for example, provide some builtin options in xmake/options directory, like .cmake/Modules
option("find_qt")
      on_check(function (option)
             -- check qt ...
             -- add links, linkdirs .. to option
      end)

option("xxx_interface")
      add_links("xxx")
      on_check(function (option)
             -- try compile codes
             -- try run and test codes (empty implementation?)
      end)

Maybe just translate some .cmake files manually is enough to finish the goal
I found cmake is licensed under BSD-3 so there's no problem

@TitanSnow But I don't like cmake's grammar and design, and don't want to be overly reliant on and use them.

For grammar, I do not like cmake's as well
But at least we could quickly look through them and come up with some ideas

Yes, it provides some good features and ideas. I will refer to them when I begin to improve option.

But I don't want to parse them for fast implementation, I need design the whole architecture first. Even if it will delay some time.

Yeah, it might be best solution

I've started to implement some detection modules and
I'll add a new interface has_feature to make it easier to judge all compiler features in the future. for example:
target("test")
     set_kind("binary")
     if has_feature("const_expr") then
          add_defines("-DCONST_EXPR")
          add_files("src/*.cpp")
     end

detect modules roadmap:

[x] basic modules (lib.detect.*)

[x] find_file

[x] by os.files
[ ] find from winreg

[x] find_path

[x] by os.filedirs
[ ] find from winreg

[ ] find_library

[ ] by os.files and os.filedirs
[ ] find from pkg-config
[ ] find from winreg

[x] find_program

[ ] find from which
[ ] find from winreg

[ ] find_toolchains (cross, toolchains dir, ...)
[ ] check_features for (compiler, linker ..)

[x] detect tools (detect.tool.*)

[x] git
[x] curl
[x] wget
[x] unzip/tar/gzip/7z ..
[x] git
[x] curl
[x] wget
[ ] compilers (detect.tool.compiler.*)

[ ] gcc
[ ] clang
[ ] msvc(cl.exe)
[ ] dmd
[ ] rustc
[ ] go
[ ] cgo

[ ] linkers (detect.tool.linker.*)

[ ] gcc
[ ] clang
[ ] msvc(lib/link)
[ ] ...

[ ] detect sdk (detect.sdk.*)

[x] xcode
[ ] android ndk (will handle)
[ ] vstudio (will handle)

[ ] detect features (detect.feature.*)

[ ] compiler (detect.feature.compiler.*)

[ ] flags features
[ ] language features

[ ] const_expr (c++)
[ ] ..

[ ] linker (detect.feature.linker.*)

[ ] flags features

[ ] lua api in xmake.lua

[ ] has_features("const_expr", "..")
[ ] option.on_check()
[ ] target:add_xxx() in before_build()
[ ] option:add_xxx in on_check()

winreg roadmap:

[ ] get

[x] path parse
[ ] type support

[ ] REG_BINARY
[x] REG_DWORD
[x] REG_EXPAND_SZ
[ ] REG_LINK
[ ] REG_MULTI_SZ
[ ] REG_QWORD
[x] REG_SZ

[ ] set

@htfy96 @TitanSnow I have implemented these features, please see: new-features-v2.1.5


Add support for Unicode normalization output iterators

I would argue that normalization shouldn't be an output iterator, but rather a transform algorithm.

I tend to agree that, in most cases, treating normalization as a transcoding operation is probably what is desirable.  Such an interface can be specialized for particular iterators (pointers) to provide higher performance as well.  The benefit of an output iterator (or a proxy input iterator) is that the transformation can be done lazily.  I think both interfaces have their uses.

I think that is an academical use case. In practice the reason why you need
to normalize the text is because you will perform an operation that
requires the text to be normalized.
Making the normalizing into an output iterator would be technically valid,
but semantically confusing.
Em sáb, 24 de set de 2016 18:14, Tom Honermann<EMAIL_ADDRESS>escreveu:

I tend to agree that, in most cases, treating normalization as a
transcoding operation is probably what is desirable. Such an interface can
be specialized for particular iterators (pointers) to provide higher
performance as well. The benefit of an output iterator (or a proxy input
iterator) is that the transformation can be done lazily. I think both
interfaces have their uses.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tahonermann/text_view/issues/20#issuecomment-249390570,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE9K53M6Ig3W4DnWU9QRItO7KXiHy39ks5qtaCmgaJpZM4Hoxuy
.

I can see use cases for wanting to perform an operation on normalized text in a lazy fashion.
Regardless, I agree that a transcoding interface has more potential uses.  That falls under issue #4.


auth2 Jump error after binding

When a new account is logged in with an account password, a github account is bound. Then click Complete Submission without jumping to the corresponding app's callback link. Instead, it jumps to the /null page

source code lastest

@seriouszyx

@AubreySte Can you provide a demo video (including the URL bar), because I wasn't able to reproduce this bug?

@AubreySte this should already be fixed after PR: https://github.com/casdoor/casdoor/pull/999


assets_version breaks "Notes" feature

assets_version set to not null leads to "Notes" can't be added (tinymce unable to load files) and no input showing in add note popup window

@ishakuta can you please check if this reproduces on the current master? We've updated Notes recently.

seems like fixed


fix: path to solution; added references

Hi,
here is the fix for your zkdoc problem. the source code to your plc project was moved -> zkdoc does that need to know too. i also added the references in order to be able to build the library eventually later with zkbuild, if you like.
filepath in documentation.yml can be . only, it is only needed if no config.json is provided.
Regards
M.

That sounds logical! Thanks for fixing it 👍
However, now the Action fails because of authentication:

My secrets are the same from the last successful run.
Thanks a lot for your time, by the way! You've invested quite a bit now in this project.

Hi,
can you try to trigger the merged action manually. I think this happened because it was executed in "branch-scope".
Regards M.

Now it fails for a different reason:

Hi, we had a problem regarding your folder structure, can you please try once more?
Thanks...

Thanks! It's working!
Regarding the question of excluding private and internal methods / types etc. from the documentation - is that something you'll be working on?
Or what's your strategy of documenting the public API of a library?

Hi,
currently everything which is marked as INTERNAL (e.g. for DataTypes) or with attribute hide, is not shown in the documentation. In the case, if something is marked with the above keywords and you have a xref in your documentation on one of those DUTs for example, zkdoc will generate an error.
Is that what you wanted? In our case (Zeugwerk Framework) all open, API-things are documented, and of course everything which describes the API. All internal things are marked with the mentioned keywords above and therefore not shown in our API-doc.
Regards
M.

Hi,
yes, with the pragma {attribute 'hide'} everything seems to work except methods. I'd like to hide private helper methods within public function blocks. These still make it into the docs:

Also, I found that for interfaces and functions INTERNAL alone was not enough, I also had to use {attribute 'hide'}.
And last thing, for cherries on the cake, it would be nice to show the values of constants in the docs, maybe in an additional column.

Hi,
thanks for your reply. We are working on those things and made some Issues -> i will inform you if there are any changes on this. This could take some time, so i hope thats ok for you.
Regards
M.


Fix issue (Content-Type-related) reported by Blackbox Testing .

This fixes the following issue: https://github.com/edgexfoundry/blackbox-testing/issues/434

Essentially, added back in a missing line in func restGetCommandsByDeviceName() for setting the ContentType. This was missed during one of the several refactorings which were done in the recent past.
It applies to the function restGetCommandsByDeviceName(), and the missing line (w.Header().Set(clients.ContentType, clients.ContentTypeJSON)) has been reinstated. Note that since the execution of a core-command is effectively a pass-through, a JSON result can be passed back to the caller as a string.
Ran the blackbox tests, which passed. Here's the individual command, which resulted in the expected response body and application/json for Content-Type, which addresses the issue reported per black-box testing:
curl --location --request GET 'localhost:48082/api/v1/device/name/Random-Binary-Device'

Sample test results for GET Headers:


Version 1.0.43 does not seem to emit "triStateChanged"

dispatch "triStateChanged" to me with the switchValue of me, the id of me
This is the current version of triStateChange in 1.0.43
Does that work for you?
I have tried but the message does not seem to be emitted. Neither on the card script level nor at the widget script level.
I had tried that version before but it did not work for me then either. That was the reason I dispatched it to the owner of me as in the now out commented version in the Diff does
dispatch "triStateChanged" to the owner of me with the switchValue of me, the id of me
That worked on a the card level.

If you also find that triStateChanged message is not sent please consider that in its current state it would also fire on initialisation. That may or may not be wanted.
If not wanted I suggest that you set in handler "openControl" at the end of the handler  "sIsOpening" to false.
Then you can wrap triStateChanged in handler "updateIndicator" in a conditional.
if not sIsOpening then
   dispatch "triStateChanged" to the owner of me with the switchValue of me, the id of me
end if

This would avoid the message to be emitted when opening a stack with triState widget.
Here is a dictionary entry for triStateChanged
/**
Type: Message
Summary: <triStateChanged> is sent to the owner of the widget when user changes state

Syntax: triStateChanged { left | center | right }, { the id of the sender }
Example:on triStateChanged pWhichWay, pSenderID
  put pWichWayy && pSenderId
end triStateChanged

Parameters:
Param1: text: left|center|right
Param2: ID: the short id of sender
Description: use <triStateChanged> message to react to user action on widget.
Put the handler one level higher than the widget:
If widget is placed on a card use card script to trap the <triStateChanged> message
*/
on triStateChanged -- fake handler for dictionary
end triStateChanged -- fake handler for dictionary

private command updateIndicator -- current code

Oddly enough I had to add the fake handler on triStateChanged at the end of the dictionary entry for the dictionary creation to pick up the triStateChanded as Message. As far as I can see it does no harm.
Of course this assumes that you adopt the message to be sent to the owner of the widget.

Bernd, I know I've said it many times, but once again thank you.
Not only have you helped make this a much better widget but you're upping my coding game at the same time :-)
I'll update to include the code above.

On further issue with triStateChanged is that it fires even if indicator hasn't changed position (i.e. if the user clicks on the indicator but doesn't move it) which I don't think is correct as the state hasn't changed.
I've added a new script local variable sPreviousValue which stores the switchValue on mouseDown and resets it on mouseUp/mouseRelease and added a check before emitting triStateChanged:
if not sIsOpening and sPreviousValue <> the switchValue of me then
      dispatch "triStateChanged" to the owner of me with the switchValue of me, the id of me
end if

This seems to work fine - I'll roll these changes up into next version and post.
Many thanks once again!

Stam,
I noticed that the message triStateChanged does not show up in the dictionaries.
The reason is a missing slash before the two asterisks in the opening block comment for triStateChanged
it should be:
/**

then it works
I think it is pretty much ready then.
I learned a lot from your script widget which I find a useful widget for special cases.
And I understood Ali's motivation better to develop script widgets. The uptake might be better than for LCB widgets.
But it is in no way trivial to do a script widget right.
I thank you for your patience and persistence. It was fun to delve into the code.

Hi Bernd, already spotted and updated ;)

The uptake might be better than for LCB widgets.
But it is in no way trivial to do a script widget right.

Agree completely. I just wish there was more  reference material for LCB, more examples online as currently it's an exercise in frustration trying to get into it... (there are examples, but because the basics aren't covered adequately, trying to apply to different use-cases just draws blanks...)
The script widgets are a nice addition and just want to see a bit more polish in the IDE support for these...
Once again thank you for your invaluable help with this widget - it would probably have taken me a whole lot longer to work around the issues (assuming I'd have been able to in the first place!).


Update Create docs + examples

Current documentation is very old and doesn't represent reality it's needs to be updated + visual examples added

@aurimasmi what should be tested?

@RosaryDarling can you give this PR a look? We discussed that a non-technical should have a look at it https://github.com/flexn-io/create/pull/115

Documentation pages for testing:

[ ] Getting started
[ ] Example app
[ ] Components
[ ] ActivityIndicator
[ ] App
[ ] ImageBackground
[ ] Image
[ ] Modal
[ ] Pressable
[ ] FlashList
[ ] Screen
[ ] ScrollView
[ ] Switch
[ ] TextInput
[ ] Text
[ ] TouchableOpacity
[ ] View
[ ] Guides
[ ] Flexn Focus Manager

In this step it is a bit unclear - assuming this is only if I want to build a project which has TVs or does it have to be added to any project?

In this step it would probably be a good  thing to mention that rnv has to be installed?
Also I'm getting 'command not found: rnv' after trying to run this one - works with npx + [copied command]

Just to be sure - in App page, there's a link to react native docs - View component, so just want to double check if this is the exact page/component that should be used?


KYC Match Readiness Checklist

Include the checklist for meta-release Fall24, including documentation, ATPs and alignment with commonalities (0.4.0)

Code action points:

(once #103 & #104 are merged)
Include clarification of the scoring algorithm into API documentation: "Unless otherwise captured in the specification, score will use the Jaro-Winkler distance algorithm for all countries."
Align with commonalities 0.4.0 (error handling mainly @rartych ?)

Documentation & versioning convention (Items 4-5) action points:

N/A, ready

Basic API test cases & documentation  (item 7) action points:

Pending to include 0.2.0 basic ATP proposal

* Align with commonalities 0.4.0 (error handling mainly @rartych ?)

I will check error responses.
I see that info and servers objects need to be aligned.

Documentation & versioning convention (Items 4-5) action points:

N/A, ready

Ready

Code (item 1-2-3) action points:

(once Align yaml with commonalities v0.3 #103 & Include score functionality proposal #104 are merged)
Include clarification of the scoring algorithm into API documentation: "Unless otherwise captured in the specification, score will use the Jaro-Winkler distance algorithm for all countries."
Align with commonalities 0.4.0 (error handling mainly @rartych ?)

Points fixed in d014a3c and 1a9c4c9

To be included by meta-release version, after 0.2 publication

Fixes for info and servers objects and security scope names proposed in https://github.com/camaraproject/KnowYourCustomer/pull/114


Documentation: New Success Stories page needs some cleanup

Now that the new Success Stories page is up on Spinnaker.io (https://www.spinnaker.io/success-stories/), it could use some cleanup. With more case studies and mini-testimonials coming as well as the new video section, the page is getting very long. Some changes I would suggest that I will start working on:

Remove the name of the company above the case study (e.g. "Waze" above the Waze case study). It is superfluous given the company logo below it and takes up space
Collapse the mini-testimonials into a logo, a quote, and "click here to read more" to uncollapse the section
Add a TOC at the top, ability to hyperlink down the page directly to each section

Other improvements to the design and layout are welcome.

#1603


After setting fill color,add shape and reselect the shape,the fill color display is not correct

@Edric-Li
It's the same issue as the previous #719. I'm closing it.


Colab & Bokeh persistence

Issue Description
Persist Bokeh figures in Colab without having to execute the plot cell.

In order to persist interactive plots in colab you have to execute the notebook in colab, and then save it to GitHub from Colab. Doing so will persist all types of plots we currently have; matplotlib, plotly, and bokeh.
Thanks @horizon-blue for helping debug this issue.
@michaeltingley We will need someone that has access to save Colab run notebooks to GitHub after their execution. You can see here an example where the notebook has been run in Colab, and then saved to GitHub. The Bokeh plot should be rendered without having to execute the nb.
https://colab.research.google.com/github/ndmlny-qs/bokeh-penguins/blob/main/bokeh_penguins.ipynb

The downside to doing this is that when notebooks change, we will need to rerun them in Colab, and then save them back to GitHub. @horizon-blue pointed out that this will become tedious, but the current workflow of saving notebooks from Colab to GitHub is a quick work-around for getting the documentation figures to stay rendered.

Could we diff them to see if we can write a script to add whatever metadata Colab needs to be happy?

@zaxtax There may be a way to update metadata in the notebook for Colab to render the images. I'll investigate further.

@horizon-blue @zaxtax Can y'all look at this notebook?
https://colab.research.google.com/github/ndmlny-qs/bokeh-penguins/blob/main/Hierarchical_regression.ipynb
I think I have figured out the mix of metadata needed to display plots without executing cells in Colab.

I think I have figured out the mix of metadata needed to display plots without executing cells in Colab.

That’s awesome! And yes, the plots in that notebook are rendered on my side.

From this thread, https://github.com/holoviz/holoviews/issues/3551, in order to render Bokeh plots in Colab we have to place the output_notebook() call in each cell that has a Bokeh plot in it. This is basically adding an extra output to the cell that contains the Bokeh JavaScript necessary to render the images, which Colab treats as a security feature for cell outputs.
The only way to not have to make this call in each Bokeh cell is to embed the JavaScript needed directly to the cell output in the notebook JSON. We could write a script to do this, but I think just adding the output_notebook() call to each cell will be sufficient. The notebook to mdx converter continues to work after adding the output_notebook() call in a cell, so we will need to update the notebooks to contain the Bokeh call in them if we want them to render in Colab without execution.


Items is not showing on Skillet

Version: <IP_ADDRESS> - 1.19-1.19.2
Items is not showing on Skillet when cooking
Screenshots:
Cooking on Stove

Portable Cooking

It's a bug. Don't know when it'll get fixed. I've been patient all this time, and I ask for you to do the same. The team behind Farmer's Delight's Fabric port will fix this bug. It'll just take a long time because it's currently not their highest priority.

Hello @Ross2750,
Thank you for reporting this issue, it is kwown and an issue is already created, you can see the advancement on #41.


Please return this back up

The only lyrics api that I understand how to use

it's back now (with a ratelimit of 150 requests per hour)


Tweaked build setup

using cross-platform packages (rimraf and cross-env)
updated deprecated babel-preset-es2015 to the preferred one - babel-preset-env
transpiling for 2 targets - commonjs and es modules, latter gives better tree-shakeability (nice to have if you decide to land this PR) and often it allows bundlers like webpack or rollup to optimize the output in various ways, like i.e. it doesnt have to be wrapped when webpack performs scope hoisting with ModuleConcatenationPlugin plugin

@rafrex friendly 🏓 about this one :)


git archive should have an option to also put submodule content into archive

It would be useful if there was an option
git archive --recurse-submodules

to package the entire content of a git repository commit, without leaving the submodule directories as empty directories in the archive as it does now.

Hey, this is the issue tracker for the git-scm.com website, not Git itself. If you still want to discuss this issue, you should reach out to the community.


.rev content file updated by new hash commit but no files are transferred

All are in title.
My dploy.yaml :
my_app:
host: "ftp.domain.com"
user: "user"
pass: "password"
path:
local: "/volume1/web/my-app"
remote: "/www/my-app"
exclude: ["dploy.yaml"]

Any idea ?
Thank you guys !

@lucasmotta

Same problem :|

@valentinoeval This fixed my problem #Issue 19

Hi @valentinoeval , that occurs when you made changes in the file you're exluding and looks normal to me! Do you think that might have been the case here?


Handle failed inputs in DirectoryContentsTask

I'm not really sure how to construct a test for this, but I think it makes sense to handle failed inputs here. The current behaviour is a runtime assertion if failed inputs are encountered by DirectoryContentsTask.

@swift-ci please smoke test


Support HTML parsing, directly

The Harlow JSON exporter project hasn't been updated in 3 years, so maybe we should support reading the official format, directly.
Tasks:

[ ] Choose an HTML parsing library
[ ] Convert to JSON
[ ] Define structs
[ ] Refactor library to account for this (maybe a harlow mod with json and html mods?)

This looks promising: https://docs.rs/html_parser/latest/html_parser/struct.Dom.html

Or these:

https://crates.io/crates/quick-xml
https://crates.io/crates/html5ever
https://crates.io/crates/roxmltree

Started with html_parser but think I want to move to roxmltree.


Release v6.0.7

Version bump for parity with Solnet

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 96.864%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
733

Relevant Lines:
751

💛  - Coveralls


"/packit test" does not trigger builds if there were none before

Pull request https://github.com/psss/tmt/pull/187 has been created by @FrNecas who is not a contributor of tmt, so tests were not scheduled. I manually asked packit to run tests using /packit test but tests failed because the copr build has not been created.

Using both /packit build and /packit test helped but I believe /packit test should be just enough ;-)

@psss thanks. The /packit test now only retrigger the test (send a new request to the TF).
We should:

[ ] Try to load the info about build from the database. (Search builds for the PR model.)
[ ] Act as \packit build if there is no build.

=> We probably need to do the search during the parsing to call correct job.

It seems this is not fixed. Just ran into the same issue with the https://github.com/psss/tmt/pull/457 where /packit test had no effect and I had to use both /packit build and /packit test to run the tests.

It seems this is not fixed. Just ran into the same issue with the https://github.com/psss/tmt/pull/457 where /packit test had no effect and I had to use both /packit build and /packit test to run the tests.

dang, @psss sorry about that, we'll take a look

dang, @psss sorry about that, we'll take a look

@psss You don't need to use both /packit build and /packit test. The build itself will trigger the tests.
I am moving to our upstream project so we can take a look into that.

Does that mean that /packit test will also trigger both build and test?

@psss No, no. That part needs to be fixed.
But for now, when you want to trigger tests when no builds are there, use /packit build (only) instead -- it will run both build and test. (That was just a note that using /packit test together with /packit build is not necessary.)

I see, thanks for clarification.

@psss Sorry for the confusion, but we removed the check for the PR-author in the meantime (=external contributors are allowed automatically). So, this situation should not happen anymore.
I am going to close the issue but let us know if you found some problem in the future -- I really hope we haven't missed any corner-case.


LDAP Auth: Fix WWW-Authenticate header with custom cred type

NOTE: Please read the CONTRIBUTING.md guidelines before submitting your patch,
and ensure you followed them all:
https://github.com/Kong/kong/blob/master/CONTRIBUTING.md#contributing
Summary
When using custom credential type for LDAP authentication (the config.header_type setting),
the WWW-Authenticate header that is sent back when auth failed should be accordingly set.
Using Basic as the header type should allow the use of any web browser as a client. When there's no (or an invalid) authentication data in the query, Kong replies with the WWW-Authenticate header. Currently, this header doesn't use  config.header_type so it sets WWW-Authenticate: LDAP, which is not understood by the browser, hence the login popup won't show.
Full changelog

Fix in kong/plugins/ldap-auth/access.lua
Add related tests in spec/03-plugins/21-ldap-auth/01-access_spec.lua

Issues resolved
I don't think a proper issue has been open, but it has been reported on discuss.konghq.com

hmm... @thibaultcha , I followed your last proposal, and reworked my commits and targeted the master branch. It looks like it broke our discussion stream though: our messages don't appear anymore in the discussion. Sorry about that :(

Manually merged, thank you!
As discussed, will you now be able to provide a subsequent patch for the migration towards the capitalized default value @francois-maillard? That'd be greatly appreciated, thanks!

@francois-maillard PS: cool picture :)

Leaving in the south of france can be tough ;)
I'll try do so next week, if I can't find the time, it might then take longer as I'll be off for two weeks.


regression: ctrl-click in terminal doesn't handle trailing ':' in file:line:col: links

Issue Type: Bug
regression: https://github.com/microsoft/vscode/issues/101155
i use this all the time, and this got broken again with the latest update.
VS Code version: Code 1.65.1 (8908a9ca0f221f36507231afb39d2d8d1e182702, 2022-03-08T02:06:27.846Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-9400 CPU @ 2.90GHz (6 x 2904)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
47.66GB (15.92GB free)

Process Argv

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: CentOS7

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
Intel(R) Core(TM) i5-9400 CPU @ 2.90GHz (6 x 2904)

Memory (System)
37.34GB (19.61GB free)

VM
0%

Item
Value

Remote
WSL: CentOS7

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
Intel(R) Core(TM) i5-9400 CPU @ 2.90GHz (6 x 2904)

Memory (System)
37.34GB (19.61GB free)

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

gitkraken-authentication
git
1.0.3

remote-ssh
ms-
0.76.1

remote-ssh-edit
ms-
0.76.1

remote-wsl
ms-
0.64.2

gitlens
eam
12.0.2

autoscrolldown
fox
1.2.1

gitkraken-authentication
git
1.0.3

cpptools
ms-
1.8.4

ok, it looks like it's a little more subtle than that issue. sometimes ctrl-clicking a link in the terminal window shows the pre-populated search-by-files popup, and then navigates to the correct location. other times it just opens the file directly and puts the cursor at the stsart of line #1 regardless of the value in the terminal link.

it also seems like there's some kind of background processing going on. sometimes the links in the terminal don't update immediately and sometimes hovering on a newly typed line shows the link in the location as if that line had not been typed (ie in the location of where the previous line was before it scrolled up).

here's a demo: https://youtu.be/Mtd5mNdnR_s

first i click a link to main.cpp:5, it opens the popup as usual, and then correctly navigates to line 5.
then i click a link to platformio.ini:5. this time it immediately opens the file (no popup) and fails to navigate to the correct line. note that the clicked link in the terminal does underline the :5 part.

I can reproduce this in stable and it appears to be fixed already in Insiders, could you verify your problems is fixed in https://code.visualstudio.com/insiders?

no. this is NOT fixed in 1.66.0-insider. please reopen.

Can reproduce, here are more specific steps:

Ensure shell integration is disabled
Create a file in the root folder that will not match any other file in the workspace
# <thefile>:5, 🐛 it will open to line 1

i can confirm that the problem seems to go away if i enable shell integration.

The supposed fix has broken other link detection.VS Code no longer picks up
"test-suite/tools/gen_rules/gen_rules.ml", line 309, characters 6-13

Only the file name is highlighted.

@Alizter I don't think the , characters x-y part was ever supported but line works for me:

This works with and without shell integration enabled.
Did you create the terminal in the parent folder of test-suite?

@Tyriar In previous versions of code, the entire thing was highlighted. Notably before ctrl was required to click on links.

Did you create the terminal in the parent folder of test-suite?

Yes.

In previous versions of code, the entire thing was highlighted. Notably before ctrl was required to click on links.

@Alizter the patterns are defined here:
https://github.com/microsoft/vscode/blob/f52f2ed7b5f50dac9aa71e899d12078c31020ca0/src/vs/workbench/contrib/terminal/browser/links/terminalLocalLinkDetector.ts#L49-L59
The file did move I think 6-12 months ago, but searching recent git history for "characters" comes up with zero results.
Also ctrl was always required to click links if I'm remembering right (maybe not very early on several years ago?).

Did you create the terminal in the parent folder of test-suite?

Yes.

Can you create a new issue with repro steps? It's a different problem then what's tracked here.

@Alizter "<path>", line <n> is supported as shown above from this line:
https://github.com/microsoft/vscode/blob/96f310e3ebee05189f03e974713ec1c666b15b6a/src/vs/workbench/contrib/terminal/browser/links/terminalLocalLinkDetector.ts#L53
, lines x-y however isn't, it was a feature request here https://github.com/microsoft/vscode/issues/144837 but got closed off as it didn't get enough votes. Is this confusion around both characters and lines not being supported but line being supported?

Found my way here because V1.71.2 doesn't seem to understand Jest references like "myfile.ts:12:56".
It DOES underline the right characters, right up to the '6', but when ctrl-clicked it pops the 'partial match' selector.
It's a perfect match to a local file - I expect it should just jump straight to the line, right?

I fixed the trailing colon problem yesterday in https://github.com/microsoft/vscode/pull/168240 which will land in insiders in a few days (after 1.74.0 stable goes out).
This is specifically about the issue in the top comment which points at https://github.com/microsoft/vscode/issues/101155:
File.cpp:52:32: error: blah blah blah

The fix specifically targeted word links, this is the type of link above as it's missing the path indicators (eg. ./<path>, /<path>, etc.). There are other types of links which I also believe are fine but please open an issue with a specific repro if you find an edge case that isn't working.

@Tyriar just to make sure I test the right thing, can you give steps?

A word link (those that need ctrl/cmd held) should include the :line:col. Just verified it myself:


gpsgridder error

Hi again, I received an error for strain calculation with gpsgridder. My files are attached and so is the message error. I'll be happy for your advice. Thanks!
strain_config_gpsgridder_Isr.txt
StrainTensor_input.txt
D:\Ariel\Phd\Strain_software\Strain_2D\example>python ..\Strain_Tools\bin\strain_rate_compute.py strain_config_gpsgridder_Isr.txt

Hello! We are...
Computing strain using : gpsgridder
Input data from        : ../test/testing_data/StrainTensor_input.txt
Calculation range      : [34.0, 36.1, 32.1, 33.9]
Putting the outputs    : Output/gpsgridder/

Reading file ../test/testing_data/StrainTensor_input.txt
11 stations before applying cleaning.
11 stations after imposing bounding box.

Computing strain via gpsgridder method.
writing vector output file tempgps.txt
gmt gpsgridder tempgps.txt -R33.98/36.120000000000005/32.08/33.92 -I0.04/0.04 -S0.5 -Fd0.01 -C0.0001 -Emisfitfile.txt -fg -r -Gnc_%s.nc
gpsgridder [WARNING]: (x_max-x_min) must equal (NX + eps) * x_inc), where NX is an integer and |eps| <= 0.0001.
gpsgridder (gmt_api.c:2126(gmtapi_init_grdheader)): Please select compatible -R and -I values
gpsgridder [WARNING]: (x_max-x_min) must equal (NX + eps) * x_inc), where NX is an integer and |eps| <= 0.0001.
gpsgridder (gmt_api.c:2126(gmtapi_init_grdheader)): Please select compatible -R and -I values
Traceback (most recent call last):
File "D:\Ariel\Phd\Strain_software\Strain_2D\Strain_Tools\bin\strain_rate_compute.py", line 11, in
internal_coordinator.strain_coordinator(MyParams);
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\site-packages\strain_2d-1.1.1-py3.10.egg\strain\internal_coordinator.py", line 26, in strain_coordinator
[Ve, Vn, rot, exx, exy, eyy, vels, resids] = constructed_object.compute(velField);  # computing strain
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\site-packages\strain_2d-1.1.1-py3.10.egg\strain\models\strain_gpsgridder.py", line 24, in compute
[Ve, Vn, rot_grd, exx_grd, exy_grd, eyy_grd] = compute_gpsgridder(myVelfield, self._strain_range,
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\site-packages\strain_2d-1.1.1-py3.10.egg\strain\models\strain_gpsgridder.py", line 66, in compute_gpsgridder
subprocess.call(['rm', 'tempgps.txt'], shell=False);
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 345, in call
with Popen(*popenargs, **kwargs) as p:
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 966, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 1435, in _execute_child
hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] The system cannot find the file specified

@odito2 it may have been a problem because I copied files with subprocess instead of native python libraries like shutil (my own less-functional legacy code, sorry).  I tried to fix it with https://github.com/kmaterna/Strain_2D/commit/e466e90be0aeeba62fe75e1ca17d58761e2c5536
Any change in results?

Thank you so much for your answer. I just returned from a long vacation and
saw your reply. I tried to run the gpsgridder but still receive the same
error.
Odelya
בתאריך יום ו׳, 19 באפר׳ 2024 ב-5:42 מאת kmaterna <
@.***>:

@odito2 https://github.com/odito2 it may have been a problem because I
copied files with subprocess instead of native python libraries like shutil
(my own less-functional legacy code, sorry). I tried to fix it with
e466e90
https://github.com/kmaterna/Strain_2D/commit/e466e90be0aeeba62fe75e1ca17d58761e2c5536
Any change in results?
—
Reply to this email directly, view it on GitHub
https://github.com/kmaterna/Strain_2D/issues/39#issuecomment-2065692643,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXV7LOTPNBW3TS6UWX2IP7DY6CHDLAVCNFSM6AAAAABGCUNJUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRVGY4TENRUGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

On Windows, I get the following error:
'awk' is not recognized as an internal or external command,
operable program or batch file.
Traceback (most recent call last):
  File "D:\Ariel\Phd\Strain_software\Strain_2D-master\Strain_Tools\bin\strain_rate_compute.py", line 11, in <module>
    internal_coordinator.strain_coordinator(MyParams)
  File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\site-packages\strain\internal_coordinator.py", line 26, in strain_coordinator
    [Ve, Vn, rot, exx, exy, eyy, vels, resids] = constructed_object.compute(velField)  # computing strain
  File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\site-packages\strain\models\strain_gpsgridder.py", line 26, in compute
    [Ve, Vn, rot_grd, exx_grd, exy_grd, eyy_grd] = compute_gpsgridder(myVelfield, self._strain_range,
  File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\site-packages\strain\models\strain_gpsgridder.py", line 86, in compute_gpsgridder
    xinc = float(subprocess.check_output('gmt grdinfo -M -C '+file1+' | awk \'{print $8}\'', shell=True))  # x-inc
  File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 420, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 524, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'gmt grdinfo -M -C Output/gpsgridder/nc_u.nc | awk '{print $8}'' returned non-zero exit status 255.

Can you remove the dependency on awk?
Switching to Linux, delauny works, but when I try to use gpsgridder I get the following error:
Computing strain invariants for 2d dataset.
Traceback (most recent call last):
  File "/mnt/d/Ariel/Phd/Strain_software/Strain_2D-master/example/../Strain_Tools/bin/strain_rate_compute.py", line 11, in <module>
    internal_coordinator.strain_coordinator(MyParams)
  File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/strain/internal_coordinator.py", line 27, in strain_coordinator
    output_manager.outputs_2d(Ve, Vn, rot, exx, exy, eyy, MyParams, vels, resids)  # 2D grid output format
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/strain/output_manager.py", line 27, in outputs_2d
    ds = Dataset(
         ^^^^^^^^
  File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/xarray/core/dataset.py", line 693, in __init__
    variables, coord_names, dims, indexes, _ = merge_data_and_coords(
                                               ^^^^^^^^^^^^^^^^^^^^^^
  File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/xarray/core/dataset.py", line 422, in merge_data_and_coords
    return merge_core(
           ^^^^^^^^^^^
  File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/xarray/core/merge.py", line 705, in merge_core
    dims = calculate_dimensions(variables)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/xarray/core/variable.py", line 2984, in calculate_dimensions
    raise ValueError(
ValueError: conflicting sizes for dimension 'x': length 53 on 'x' and length 54 on {'y': 'Ve', 'x': 'Ve'}

@kmaterna
'awk' is not recognized as an internal or external command,
operable program or batch file.
Traceback (most recent call last):
File "D:\Ariel\Phd\Strain_software\Strain_2D-master\Strain_Tools\bin\strain_rate_compute.py", line 11, in
internal_coordinator.strain_coordinator(MyParams)
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\site-packages\strain\internal_coordinator.py", line 26, in strain_coordinator
[Ve, Vn, rot, exx, exy, eyy, vels, resids] = constructed_object.compute(velField)  # computing strain
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\site-packages\strain\models\strain_gpsgridder.py", line 26, in compute
[Ve, Vn, rot_grd, exx_grd, exy_grd, eyy_grd] = compute_gpsgridder(myVelfield, self._strain_range,
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\site-packages\strain\models\strain_gpsgridder.py", line 86, in compute_gpsgridder
xinc = float(subprocess.check_output('gmt grdinfo -M -C '+file1+' | awk '{print $8}'', shell=True))  # x-inc
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 420, in check_output
return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 524, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'gmt grdinfo -M -C Output/gpsgridder/nc_u.nc | awk '{print $8}'' returned non-zero exit status 255.

@kmaterna
On Windows, I get the following error:
'awk' is not recognized as an internal or external command,
operable program or batch file.
Traceback (most recent call last):
File "D:\Ariel\Phd\Strain_software\Strain_2D-master\Strain_Tools\bin\strain_rate_compute.py", line 11, in
internal_coordinator.strain_coordinator(MyParams)
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\site-packages\strain\internal_coordinator.py", line 26, in strain_coordinator
[Ve, Vn, rot, exx, exy, eyy, vels, resids] = constructed_object.compute(velField)  # computing strain
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\site-packages\strain\models\strain_gpsgridder.py", line 26, in compute
[Ve, Vn, rot_grd, exx_grd, exy_grd, eyy_grd] = compute_gpsgridder(myVelfield, self._strain_range,
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\site-packages\strain\models\strain_gpsgridder.py", line 86, in compute_gpsgridder
xinc = float(subprocess.check_output('gmt grdinfo -M -C '+file1+' | awk '{print $8}'', shell=True))  # x-inc
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 420, in check_output
return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
File "C:\Users\Odelya\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 524, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'gmt grdinfo -M -C Output/gpsgridder/nc_u.nc | awk '{print $8}'' returned non-zero exit status 255.
Can you remove the dependency on awk?
Switching to Linux, delauny works, but when I try to use gpsgridder I get the following error:
Computing strain invariants for 2d dataset.
Traceback (most recent call last):
File "/mnt/d/Ariel/Phd/Strain_software/Strain_2D-master/example/../Strain_Tools/bin/strain_rate_compute.py", line 11, in
internal_coordinator.strain_coordinator(MyParams)
File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/strain/internal_coordinator.py", line 27, in strain_coordinator
output_manager.outputs_2d(Ve, Vn, rot, exx, exy, eyy, MyParams, vels, resids)  # 2D grid output format
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/strain/output_manager.py", line 27, in outputs_2d
ds = Dataset(
^^^^^^^^
File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/xarray/core/dataset.py", line 693, in init
variables, coord_names, dims, indexes, _ = merge_data_and_coords(
^^^^^^^^^^^^^^^^^^^^^^
File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/xarray/core/dataset.py", line 422, in merge_data_and_coords
return merge_core(
^^^^^^^^^^^
File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/xarray/core/merge.py", line 705, in merge_core
dims = calculate_dimensions(variables)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/xarray/core/variable.py", line 2984, in calculate_dimensions
raise ValueError(
ValueError: conflicting sizes for dimension 'x': length 53 on 'x' and length 54 on {'y': 'Ve', 'x': 'Ve'}

Hi @odito2,
You may keep working on Unix-based machines since it is optimal work in this environment.
Actually, I used the earlier version, then there is no issue using all methods.

@kmaterna
Switching to Linux, delauny works, but when I try to use gpsgridder I get the following error:
Computing strain invariants for 2d dataset.
Traceback (most recent call last):
File "/mnt/d/Ariel/Phd/Strain_software/Strain_2D-master/example/../Strain_Tools/bin/strain_rate_compute.py", line 11, in
internal_coordinator.strain_coordinator(MyParams)
File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/strain/internal_coordinator.py", line 27, in strain_coordinator
output_manager.outputs_2d(Ve, Vn, rot, exx, exy, eyy, MyParams, vels, resids) # 2D grid output format
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/strain/output_manager.py", line 27, in outputs_2d
ds = Dataset(
^^^^^^^^
File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/xarray/core/dataset.py", line 693, in init
variables, coord_names, dims, indexes, _ = merge_data_and_coords(
^^^^^^^^^^^^^^^^^^^^^^
File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/xarray/core/dataset.py", line 422, in merge_data_and_coords
return merge_core(
^^^^^^^^^^^
File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/xarray/core/merge.py", line 705, in merge_core
dims = calculate_dimensions(variables)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/odelya/anaconda3/envs/Strain_2D/lib/python3.12/site-packages/xarray/core/variable.py", line 2984, in calculate_dimensions
raise ValueError(
ValueError: conflicting sizes for dimension 'x': length 53 on 'x' and length 54 on {'y': 'Ve', 'x': 'Ve'}


Wifi and snap packaging för scrcpy

So I dropped my old computer and smashed it into oblivion.  Now I found the snap packaging and it was easy to install.
But I can not seem to get the wifi function to work.
Anyone ?
I get "unable to connect  to x.xx.xx.xx:5555: Network is unreachable"
When I run abd connect.
Anyone?

Nevermind good people. A restart of the system fixed it.
So it is no longer an issue.


Implement Possibility to answer in chat an process the answer in iobroker

It would be nice to have the possibiliy to answer in the signal chat and process the answer in iobroker.
Like...
"Do you want to start the washing mashine now?"
"Yes"
"ok, washing machine will be turned on"

Hello,
That's not possible at the moment. In your second request/issue I wrote to you, why it is not possible -> https://github.com/derAlff/ioBroker.signal-cmb/issues/7#issuecomment-1294714212
But. I write it down as a feature request 👍


Mangrove Staff

there are staffs for the rest of the vanilla woods so why not a mangrove one?

I forgot to implement it because I'm very smart


[Vendor] mssqldb: 2019-11-28 -> 2020-04-28

update go-mssqldb 2019-11-28 (1d7a30a10f73) -> 2020-04-28 (06a60b6afbbc)
it close eventualy #10258

Codecov Report

Merging #11364 into master will increase coverage by 0.00%.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##           master   #11364   +/-   ##
=======================================
  Coverage   43.83%   43.83%
=======================================
  Files         613      613
  Lines       87226    87229    +3
=======================================
+ Hits        38235    38240    +5
- Misses      44266    44268    +2
+ Partials     4725     4721    -4

Impacted Files
Coverage Δ

modules/setting/setting.go
45.53% <50.00%> (+0.25%)
:arrow_up:

modules/indexer/stats/queue.go
62.50% <0.00%> (-18.75%)
:arrow_down:

modules/indexer/stats/db.go
40.62% <0.00%> (-9.38%)
:arrow_down:

modules/git/utils.go
65.67% <0.00%> (-4.48%)
:arrow_down:

models/unit.go
41.97% <0.00%> (-2.47%)
:arrow_down:

modules/queue/unique_queue_disk_channel.go
53.84% <0.00%> (-1.93%)
:arrow_down:

modules/log/event.go
64.61% <0.00%> (-1.03%)
:arrow_down:

routers/repo/view.go
35.96% <0.00%> (-0.68%)
:arrow_down:

models/gpg_key.go
53.42% <0.00%> (-0.53%)
:arrow_down:

models/issue.go
52.41% <0.00%> (+0.46%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


folder does not exist for iOS apiview

upon building the project, I got the following errors and app crashed:

there's no "travisprescott" folder on my laptop

FYI @tjprescott

@jpeng-ms what are you attempting to do?

You will need to update the arguments passed on launched to point to whatever source you are trying to generate for.


Respect Shell Globs when running Docker cp...or make a sensible error.

I was trying to copy some files out of a container with globs.
[ec2-user@ip-172-31-17-218 ~]$ sudo docker cp jenkins:/opt/jenkins/data/com* .
2014/05/23 16:28:40 Tarball too short
[ec2-user@ip-172-31-17-218 ~]$ sudo docker cp jenkins:/opt/jenkins/data/* .
2014/05/23 16:28:51 Tarball too short
[ec2-user@ip-172-31-17-218 ~]$ sudo docker cp jenkins:/opt/jenkins/data/ .

Tarball too short is sort of meaningless....I didn't know I was using tar. There were files matching the com* pattern. There were files and directories matching the * pattern.

@jlhawn does this work with your cp patch

@jfrazelle nope, file globs are an extra feature

cool made it a black belt

The issue with this case is specifically that * is being interpreted by your shell before the docker client ever sees it.
When quoted, the error is ok:
$ docker cp $(docker ps -lq):'/tmp/*' .
Error response from daemon: lstat /mnt/sda1/var/lib/docker/overlay/d7e7e7be1a08e595d95ee18f3ec0745d9570f63dc077866759f859ba6bc51817/merged/tmp/*: no such file or directory

Closing this since there is already an open issue about supporting shell globs in the API.


Fix a typo in a method name

Any idea why check #20181116.3 failed ? I don't have access to the details.


"close the repository" or "clone the repository"?

should it be "clone the repository", instead of "close"?

Thanks @EdisonDa! Appreciate the help.


Sample require update

The management library is outdated (0.0.1-preview, where current version is 0.2.0-preview).
The following exception is thrown:
Microsoft.Rest.Azure.CloudException: 'The api-version '2015-08-01' is invalid. The supported versions are '2017-05-10,2017-05-01,2017-03-01,2016-09-01,2016-07-01,2016-06-01,2016-02-01,2015-11-01,2015-01-01,2014-04-01-preview,2014-04-01,2014-01-01,2013-03-01,2014-02-26,2014-04'.'

@jtaubensee who could look into this?

Management library for SB version 1.0.0 has been released.
I've updated PR #6 to use the latest version of SB library.


[SPARK-49257][INFRA] Make bundler install can retry

What changes were proposed in this pull request?
The pr aims to make bundler install can retry.
Why are the changes needed?
Reduce the occurrence of similar issues in GA below:
https://github.com/panbingkun/spark/actions/runs/10413547981/job/28841142085

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.

Merged to master.


Documentation: Update UserGuide

#46
Change title to cherbook v1.2
Add Introduction section
Add About section
Convert Command Summary to Command Shortcuts
Add Glossary and some definitons

Coverage remained the same at 92.689% when pulling 4f50786e866c63e51bbdca9f9d4e66561f6b7fa9 on lincredibleJC:feature/Update_UserGuide into 849671be194b12817b2d581549e3357429aef301 on CS2103AUG2017-T16-B2:master.

Coverage remained the same at 92.689% when pulling 49469bd752e86adfc90ecee4ffb61b79e8b83517 on lincredibleJC:feature/Update_UserGuide into 849671be194b12817b2d581549e3357429aef301 on CS2103AUG2017-T16-B2:master.


Move to .NET Core 3.1

pmcenter is planning to upgrade pmcenter's target .NET Core version from 2.1 to 3.1 with these following changes:

Add HTTP/2 support
Move to .NET Core's new built-in JSON support
R2R images based on AOT compilation available through GitHub Releases

More features may be added in the future, stay tuned!

Seems like the last update broke systemd support. Investigating.

pmcenter 1.9.280.15 is now officially the final version. Preparing to merge into the master branch.
Oh, the newly added dice feature in Telegram 6.0 will soon be supported for /info command. But it's only for pmcenter v2, stay tuned!

Opened 2 related issues #27 and #28.

No, pmcenter will not migrate to System.Text.Json yet, since Telegram.Bot still requires Newtonsoft.Json as a dependency and the framework is also pretty decent, so we're not moving yet.


modified zh.yml

@sot528
Sorry I forgot to add header link text on zh.yml. Please check it.

@sot528 Hi sota, is there any reason you don't review this task?


numpy.inf is needed

numpy.inf isn't supported by pythran, yet it's a very useful thing to have.
import numpy as np
# pythran export test()
def test():
    N = np.inf

This seems to be well supported, what makes you think it is not?

The issues is that np accepts np.Inf (uppercase i) and np.inf, but not pythran. I hadn't realized that when I submitted the bug
import numpy as np

# pythran export test(float)
def test(x):
    if x < np.Inf: return 0 # Fails
    if x < np.inf: return 0 # OK
    return 1


Sspx update

Masses, volumes, surface areas, resources are updated to new 1.8 scaling
Latest SSPX parts, 1.875m (3.38m in RO) and 5m (9m in RO) are now supported
Tubes, adapters etc. have habitable volumes
Habitat modules have pre-filled resources for long duration flights
More advanced habitats provide water purification
Greenhouses provide resource conversion and comfort
Centrifuges work and provide comfort
Extendables carry enough Nitrogen to be used for their extension
Also, see RO and RP-1 changes

Is it intentional that the centrifuges have no radiation shielding ability? I could see it for the inflatable ones, but the fixed, mechanically deployed ones should have that ability.

Is it intentional that the centrifuges have no radiation shielding ability? I could see it for the inflatable ones, but the fixed, mechanically deployed ones should have that ability.
This wasn't intentional and most likely happened because shielding is automatically assigned to modules with habitats and expandable modules have habitats turned off by default. I'll try to fix it.
The greenhouses / bioreactor for me also seem to be pitifully weak - even 16 of them barely puts a dent towards kerbal food consumption. Not sure if that's on my end or just not configured yet.
I didn't pay enough attention when configuring reactors, I will check it and raise numbers.