Can't compile without strict typing of inport

Describe the bug
Can't compile code without strict typing of inport on flow example below Sworktest
To Reproduce
Steps to reproduce the behavior:

Run

flow Main(start) (stop) {
    nodes {
        Sworktest
        StreamPort<int>
        Println
    }
    :start -> [
        (1 -> streamPort[0]),
        (2 -> streamPort[1])
    ]

    streamPort -> sworktest -> println -> :stop
}

flow Sworktest(in) (out) {
	nodes {
		Add<int>
	}
	:in -> add -> :out
}

Expected behavior
Run and print out 3
Fix
Adding a type to the inport Sworktest like Sworktest(int stream<int)(out) allows the example to compile

Here's is the error that I got
Incompatible types: in:in -> add:UNKNOWN: Subtype and supertype must both be either literals or instances, except if supertype is union: expression any, constaint { idx int, last bool, data int }

I must admit that add:UNKNOWN part is very confusing and error message overall far from perfect. However, what it tells is that this connection right here
:in -> add

Is invalid because add wants int and you send any to it. It's any because you didn't specify port type, which is ok for many cases but here you have to do it.
However, there's a PR I made today that improves this error just a little bit (it removes the UNKNOWN word from message that comes from the fact that we didn't specify add node's port)
https://github.com/nevalang/neva/pull/676

That helps definitely. I wasn't sure if there was type inference or not (not saying there needs to be that is a design discussion)


Property-based testing for truncate

While working on #85, I noticed that test_scale_to_equivalent_to_truncate was broken (it had min_value == max_value for the length argument), so I had to go and fix it; while I was at it, I added some more tests.
There are a few behaviours that currently make the tests fail, that should be clarified:

[ ] test_truncate_equivalent_to_scale detects that x.truncate(0) always succeeds, whereas x.scale_to(0) fails when x is null.
Should we make scaling the null vector to length 0 succeed, or should truncate raise an exception?
[ ] test_truncate_length detects that sometimes scale_to rounds up, resulting in a vector slightly-longer than the desired length.
Should we make the test more lax (for instance, checking length <= (1 + 1e-9) * max_length) or should truncate make really sure that the resulting vector isn't larger than max_length?

So, what do you think should be the correct behaviours for truncate and scale_to?  (see first message in the PR)

I pushed 2 commits (51bcb2b and b5c4447) with a proposed fix:

modify Vector2.scale_to so it always succeeds when length == 0 (even if self == (0, 0));
admit and document the rounding behaviour of scale_to and truncate.

Hypothesis still found an exciting case where self.length == max_length and truncate and scale_to disagree.  Will fix but my train is arriving.

The current CI failure is due to a bug in Vector2.isclose, exposed and fixed in #96

PS: This got a bit out of hand (17 commits, not counting the merge from master), sorry about that.
I merged from master rather than rebasing, because the conflict resolution was a bit annoying and I couldn't justify to myself the extra effort in that case.  :3

I just realised there was no reply to last week's message.
Are the proposed behaviours the one we want?

They seem like reasonable behaviors.

OK, what's the blocker here?  CI is green, and you agreed the implemented behaviours are correct.


Add ability to configure your own AWSCredentialsProvider

The Spring boot auto configuration will always register an AWSCredentialsProvider making it impossible to register your own.
Adding a conditional to ContextCredentialsAutoConfiguration
@ConditionalOnMissingBean(AWSCredentialsProvider.class)
@Configuration
@Import({ContextDefaultConfigurationRegistrar.class, ContextCredentialsAutoConfiguration.Registrar.class})
public class ContextCredentialsAutoConfiguration {

would allow for something like
    @Bean(name = {CredentialsProviderFactoryBean.CREDENTIALS_PROVIDER_BEAN_NAME, AmazonWebserviceClientConfigurationUtils.CREDENTIALS_PROVIDER_BEAN_NAME} )
    public static AWSCredentialsProviderChain credentialsProvider(Environment environment) {
        return new AWSCredentialsProviderChain(
                new EnvironmentVariableCredentialsProvider(),
                new SystemPropertiesCredentialsProvider(),
                new ProfileCredentialsProvider(),
                new InstanceProfileCredentialsProvider(),
                new StaticCredentialsProvider(new BasicAWSCredentials(environment.getProperty("cloud.aws.credentials.accessKey",""),
                        environment.getProperty("cloud.aws.credentials.secretKey",""))));
    }

It also seems that the documentation is not in line with the behavior -
the documentation state at http://cloud.spring.io/spring-cloud-aws/spring-cloud-aws.html states:
"The com.amazonaws.auth.DefaultAWSCredentialsProviderChain is used by all the clients if there is no dedicated credentials provider defined."
The ContextCredentialsAutoConfiguration seems to always create its own credentials provider - but I really do want the DefaultAWSCredentialsProviderChain

Bump.
I need to use ContainerCredentialsProvider inside an ECS container to get the IAM role credentials assigned to the container.
How would I do this currently with 1.1.1?

I'm not one of the authors, @Qvazar - it seems that there's no way to avoid it registering a credentials provider factory in the bean registry with a certain name - but it seems like you can at least get it to not use that provider chain by making your own and marking it as @Primary
Try doing something like this:
    @Bean
    @Primary
    public AWSCredentialsProvider awsCredentialsProvider() {
        return new DefaultAWSCredentialsProviderChain();
    }

The DefaultAWSCredentialsProviderChain is pretty darn good. It has several providers it wraps and one of them is the EC2ConatinerCredentialsProviderWrapper that will should pull the credentials out for you...
(In #170 I have a patch that would make this thing use the DefaultAWSCredentialsProviderChain, but in the long term I really wonder if it wouldn't be better to remove all of this stuff altogether and have people either use the DefaultAWSCredentialsProviderChain or if they need a different one have them create their own @Bean)

That seems to work great, thank you @ryangardner !

I also had to name the bean org.springframework.cloud.aws.core.credentials.CredentialsProviderFactoryBean#CREDENTIALS_PROVIDER_BEAN_NAME for mine take precedence.

Sincere request to maintainers @spencergibb  to provide a way to allow to use custom awsCredentialsProvider, much more required with @Profile support for different work env

@rverma-nikiai this is a community maintained project, it is currently assigned to @aemruli

In case anyone still stumbles upon this and needs Default AWS chain, there is an option now to use it. Just set cloud.aws.credentials.useDefaultAwsCredentialsChain to true in your app properties.
See here: https://github.com/spring-cloud/spring-cloud-aws/blob/master/docs/src/main/asciidoc/spring-cloud-aws.adoc#configuring-credentials

Fixed in ee779018d38144d86a071ac708d48857b8328f37

You can provide custom credentials provider bean, it just has to have specific name:
@Bean(name = CredentialsProviderFactoryBean.CREDENTIALS_PROVIDER_BEAN_NAME)
AWSCredentialsProvider awsCredentialsProvider() {
    return mock(AWSCredentialsProvider.class);
}

Where CredentialsProviderFactoryBean.CREDENTIALS_PROVIDER_BEAN_NAME has value credentialsProvider.

I was unable to get this to work in auto-configuration without still resorting to @Primary as there seems to be a second credential provider created named org.springframework.cloud.aws.core.region.RegionProvider.BEAN_NAME.


Moving insights(1) resources to v1beta1 version

Moving insights(5) resources to v1beta1 version

[ ] azurerm_monitor_diagnostic_setting

Precondition: run kubectl annotate managed --all upjet.upbound.io/test=true --overwrite
Need investigate why we see True True for all resources but in logs have an error:
status:
atProvider:
id: /subscriptions/038f2b7c-3265-43b8-8624-c9ad5da610a8/resourceGroups/rg-monitor2/providers/Microsoft.KeyVault/vaults/svetaexdbvault
vaultUri: https://svetaexdbvault.vault.azure.net/
conditions:

lastTransitionTime: "2023-01-25T13:50:00Z"
reason: Available
status: "True"
type: Ready
lastTransitionTime: "2023-01-25T13:52:30Z"
reason: ReconcileSuccess
status: "True"
type: Synced
lastTransitionTime: "2023-01-25T13:49:27Z"
reason: Finished
status: "True"
type: AsyncOperation
lastTransitionTime: "2023-01-25T13:49:27Z"
message: 'apply failed: waiting for Vault: (Name "svetaexdbvault" / Resource Group
"rg-monitor2") to become available: connecting to "https://svetaexdbvault.vault.azure.net/":
Get "https://svetaexdbvault.vault.azure.net/": dial tcp: lookup svetaexdbvault.vault.azure.net
on <IP_ADDRESS>:53: no such host: '
reason: ApplyFailure
status: "False"
type: LastAsyncOperation

monitordiagnosticsetting.yaml.txt

Hello, any ETA when this functionality is merged?


Review support for WMQ-transport

Original issue 130 created by soi-toolkit on 2011-04-07T12:31:57.000Z:
See compensating code in:
org.soitoolkit.commons.mule.jms.ObjectToJMSMessageTransformer.setJmsProperties(...)

Won't fix.
WMQ-transport is a Mule EE feature and requires a WMQ-installation, which means we can't have continuous testing for it.


 Could you re-compile the model with python -m mlc-llm compile ...? Also see this thread where the same issue happens. I believe model recompilation can help.

Ha still same

I see. To help confirm the issue, could you help check the file mlc_llm/nn/kv_cache.py on your local side and see whether the Line 351 is the following? If it is not the following, it means your mlc_llm is not up to date, and please update mlc_llm to the latest nightly or the latests commit on the main branch.
bb.add_func(_copy_single_page(num_key_value_heads, page_size, head_dim, dtype, target), "kv_cache_copy_single_page"),

https://github.com/mlc-ai/mlc-llm/blob/5bc3ffa6f682a4cf42fdeba3a4c505d0e7c08c3c/python/mlc_llm/nn/kv_cache.py#L351

I see. To help confirm the issue, could you help check the file mlc_llm/nn/kv_cache.py on your local side and see whether the Line 351 is the following? If it is not the following, it means your mlc_llm is not up to date, and please update mlc_llm to the latest nightly or the latests commit on the main branch.
bb.add_func(_copy_single_page(num_key_value_heads, page_size, head_dim, dtype, target), "kv_cache_copy_single_page"),

https://github.com/mlc-ai/mlc-llm/blob/5bc3ffa6f682a4cf42fdeba3a4c505d0e7c08c3c/python/mlc_llm/nn/kv_cache.py#L351

bb.add_func(_copy_single_page(num_key_value_heads, page_size, head_dim, dtype, target), "kv_cache_copy_single_page"),

mlc_llm version
pip show mlc_llm
Name: mlc-llm
Version: 0.1.dev1068+gb7416c02
Summary: MLC LLM: an universal LLM deployment engine via ML compilation.

@Vinaysukhesh98 @sygi could you share your commands to run things, together with the full logging? From the information we have so far I am unable to find the reason.

clone the repo:
git clone --recursive https://github.com/mlc-ai/mlc-llm/
add tvm, java, android, etc to bashrc
compile tvm:
chmod +x 3rdparty/libbacktrace/configure && \  mkdir -p cmake-build && \  cmake -H. -Bcmake-build -DUSE_LLVM=ON && \  cmake --build cmake-build --target all -- -j 4 && \  mv cmake-build build
download configs etc for android + llama 7B f16 from here.
start virtualenv, install dependencies (attrs, numpy, typing_extensions, psutil, decorator)
go to android/library, add:

set(JAVA_AWT_LIBRARY NotNeeded)
set(JAVA_JVM_LIBRARY NotNeeded)
set(JAVA_INCLUDE_PATH2 NotNeeded)
set(JAVA_AWT_INCLUDE_PATH NotNeeded)

to CMakeLists.txt
7. modify app-config.json to only include the llama model and point to my android.tar.
8. Run prepare_libs.sh, make sure the libraries appeared.
9. Compile things in android studio, send to device.
10. Download the weights, start conversation.

Hello, same issue encountered when building and testing the android app as instructed in https://llm.mlc.ai/docs/deploy/android.html. The error message shows as the chatting UI initializes.
We use prebuilt models and libs:

https://huggingface.co/mlc-ai/RedPajama-INCITE-Chat-3B-v1-q4f16_1-MLC
https://huggingface.co/mlc-ai/phi-2-q4f16_1-MLC
https://github.com/mlc-ai/binary-mlc-llm-libs

We use the latest prebuilt packages:

mlc_llm v0.1.dev0
tvm 0.16.dev0.

We are attempting to fix this with earlier versions. No prebuilt packages are provided, so we have to build from source.

Hello, same issue encountered when building and testing the android app as instructed in llm.mlc.ai/docs/deploy/android.html. The error message shows as the chatting UI initializes.
We use prebuilt models and libs:

huggingface.co/mlc-ai/RedPajama-INCITE-Chat-3B-v1-q4f16_1-MLC
huggingface.co/mlc-ai/phi-2-q4f16_1-MLC
mlc-ai/binary-mlc-llm-libs

We use the latest prebuilt packages:

mlc_llm v0.1.dev0
tvm 0.16.dev0.

We are attempting to fix this with earlier versions. No prebuilt packages are provided, so we have to build from source.

Update:
Still no luck. We've managed to build the android .tar with tvm v0.15 (after fixing some version mismatch issues with mlc and llvm...), but it seems not compatible with the latest prebuilt weights and libs, and we got error message as in #2031. It looks like the deployment workflow is undergoing significant changes recently, and we are not able to find a walkaround for now by ourselves.
We understand that the android pipeline might not be the priority, but still appreciate it if you could look at this issue and provide a valid android building & deploying pipeline :)

I actually looked into #2031, @xxxxyu a bit and it looks like the config format has changed a bit recently. Can you confirm that you have the last version of mlc-chat-config (which has conv_template as a string), as well as a new version of tvm (which parses the object here)?

@sygi Hi, the mlc-chat-config is exactly the same as you provided, but mlc/tvm version is earlier.
We actually tested the following 3 settings (windows + wsl + Pixel 6 Pro):

latest mlc + tvm 0.16: bug reported in this issue .
mlc commit #1659 + tvm 0.15 (built from source): bug reported by #2031.
latest mlc + tvm 0.15: mismatch.

I also encountered the same problem, which can be solved by modifying mlc-llm/3rdparty/tvm/src/runtime/relax_vm/paged_kv_cache.cc :
line 1814:
TVM_REGISTER_GLOBAL("vm.builtin.paged_attention_kv_cache_create_reduced")
.set_body_typed([](ShapeTuple cache_config, int64_t num_layers, int64_t num_qo_heads,
int64_t num_kv_heads, int64_t head_dim, int rope_mode, double rotary_scale,
double rotary_theta, NDArray init, PackedFunc f_transpose_append,
PackedFunc f_attention_prefill, PackedFunc f_attention_decode,
PackedFunc f_attention_prefill_sliding_window,
PackedFunc f_attention_decode_sliding_window,
PackedFunc f_attention_prefill_ragged, PackedFunc f_merge_inplace,
PackedFunc f_split_rotary, PackedFunc f_copy_single_page) {
//Optional f_debug_get_kv
CHECK_EQ(cache_config.size(), 5);
int64_t reserved_num_seqs = cache_config[0];
int64_t total_token_capacity = cache_config[1];
int64_t prefill_chunk_size = cache_config[2];
int64_t page_size = cache_config[3];
bool support_sliding_window = cache_config[4];
int64_t num_total_pages = (total_token_capacity + page_size - 1) / page_size + 1;
if (support_sliding_window) {
// When sliding window is enabled, each sequence may use two more pages at most.
num_total_pages += reserved_num_seqs * 2;
}
ObjectPtr n = make_object(
page_size, num_layers, num_qo_heads, num_kv_heads, head_dim, reserved_num_seqs,
num_total_pages, prefill_chunk_size, support_sliding_window, RoPEMode(rope_mode),
rotary_scale, rotary_theta, init->dtype, init->device, std::move(f_transpose_append),
std::move(f_attention_prefill), std::move(f_attention_decode),
std::move(f_attention_prefill_sliding_window),
std::move(f_attention_decode_sliding_window), std::move(f_attention_prefill_ragged),  //
NullOpt, NullOpt, NullOpt, NullOpt, NullOpt, NullOpt,                                 //
std::move(f_merge_inplace), std::move(f_split_rotary), std::move(f_copy_single_page),
NullOpt);
//std::move(f_debug_get_kv));
return AttentionKVCache(std::move(n));
});

https://github.com/mlc-ai/mlc-llm/issues/2076#issuecomment-2056249208 works fine for me, thanks!

Not sure if this is what @xxxxyu meant, but for me:

the compiler for tvm wasn't able to infer the types until I changed to:

make_object<PagedAttentionKVCacheObj>

When I ran ./prepare_libs.sh, I got:

In file included from /home/sygi/code/mlc-llm2/cpp/llm_chat.cc:26:
/home/sygi/code/mlc-llm2/cpp/./metadata/model.h:18:7: error: typedef redefinition with different types ('std::unordered_map<std::string, value>' (aka 'unordered_map<basic_string<char>, picojson::value>') vs 'value::object' (aka 'picojson::object_with_ordered_keys'))
using object = std::unordered_map<std::string, value>;
      ^
/home/sygi/code/mlc-llm2/3rdparty/tvm/3rdparty/picojson/picojson.h:326:23: note: previous definition is here
typedef value::object object;

Hi @sygi

Regarding the make_object type error, I've fixed it as you did to make it work.
I didn't encounter any other issue at compiling time, including the redefinition error you mentioned. You might need to check if the brackets still match after replacing the codes.

The error I mentioned in the update shows only at runtime, and only when I attempt to compile the model libs on my own (the prebuilt model libs on https://github.com/mlc-ai/binary-mlc-llm-libs are OK), and the error message is still "...relax.vm.AttentionKVCache expects 19 arguments, but 18 were provided".

Thank you for confirming. To clarify, my error doesn't appear during compilation of tvm (which runs fine), but when running ./prepare_libs.sh from the android folder (presumably during linking with picojson). Did you also do this? Could you confirm that you're at 0f67508 commit of tvm submodule?
Fwiw, I only want to do the prebuilt libraries, compiling them yourself seems like another ordeal ^^

@sygi
I'm using mlc-ai-nightly==0.15.dev275, which should contain a prebuilt tvm package. I didn't compile tvm myself either, I only attempted to compile mlc-llm myself.
python -c "import tvm; print('\n'.join(f'{k}: {v}' for k, v in tvm.support.libinfo().items()))" gives: GIT_COMMIT_HASH: ae057a2e74e895a846df958c19ff342505131a65.
./prepare_libs.sh was successful to me.
BTW, I've noticed a small difference between our codes. You might want to try ObjectPtr<PagedAttentionKVCacheObj> n = make_object<PagedAttentionKVCacheObj>? But I'm not sure if this matters  :)

latest android sdk might help address related issues https://llm.mlc.ai/docs/deploy/android.html

Hit the same issue with the latest code at<PHONE_NUMBER>d5309ec87dc32c373e1759813ee84e for iOS app with a customized llama model
Function vm.builtin.paged_attention_kv_cache_create_reduced(0: runtime.ShapeTuple, 1: int64_t, 2: int64_t, 3: int64_t, 4: int64_t, 5: int, 6: double, 7: double, 8: runtime.NDArray, 9: runtime.PackedFunc, 10: runtime.PackedFunc, 11: runtime.PackedFunc, 12: runtime.PackedFunc, 13: runtime.PackedFunc, 14: runtime.PackedFunc, 15: runtime.PackedFunc, 16: runtime.PackedFunc, 17: runtime.PackedFunc, 18: runtime.PackedFunc) -> relax.vm.AttentionKVCache expects 19 arguments, but 18 were provided.


Resolve RUSTSEC-2023-0044

Changes

Current dependencies on/for this PR:

main

PR #282   👈

This stack of pull requests is managed by Graphite.


Fix codec string for H.264 High Level 4.1

H.264 High Level 4.1 should be avc1.640029 instead of avc1.640028.
Here's the Google docs on the subject: https://developers.google.com/cast/docs/media
Interestingly enough, the Apple documentation says it should be 28!
https://developer.apple.com/library/ios/documentation/NetworkingInternet/Conceptual/StreamingMediaGuide/FrequentlyAskedQuestions/FrequentlyAskedQuestions.html
However, the Apple mediastreamvalidator tool reports an error for a 4.1 stream with 28 saying that it expected 29. I think the Apple docs are incorrect, and have submitted a bug report.

Good catch, thanks!


Version number plugin support

Needed this for local purposes, thought it would be best to contribute it back!

Thank you for contributing to openstack-infra/jenkins-job-builder!
openstack-infra/jenkins-job-builder uses Gerrit for code review.
Please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


[Keyvault] Security domain changed to LRO

Resource Provider

Keyvault
Description of Feature or Work Requested

Keyvault security domain download have changed operations to async and match remaining to current service behavior
https://github.com/Azure/azure-rest-api-specs/pull/12863
https://github.com/Azure/azure-rest-api-specs/pull/12966
Minimum API Version Required

Swagger Link

Target Date

Keyvault is using vendor SDK now, no need official released data plane SDK.


Swipe doesn't work for iOS 13

Same problem here. On iOS 13 build on xCode11 swipe don't work.
Any ideas about how to fix?

I made a pull request that fixes this: https://github.com/Ramotion/paper-onboarding/pull/124#issue-445602674.


Other languages

What about examples for Python or Ruby with PIDOR pattern ?

Translate this word Pidor to russian language))) you will excited :D

Added java (#18)


[NFC] Separate special builtin types into a lazily initialized vector

In preparation for future patches where debug info generation will need to access the special builtin types, lazily emit them into a separate lazily initialized vector.

@swift-ci smoke test

@swift-ci smoke test


added models for R1.1'19

@atsyplen please change the header to "for R2'19"
cc @jkamelin
why 3 flavors of densenet and 1 facenet are not in this PR?

also, please make sure that license file is properly updated.


Support pulseaudio

This is a kind request to also support pulseaudio.
I know that arecord normally requires ALSA, but using the alpine package alsa-plugins-pulse it is possible to use alsa clients (like sox, arecord, aplay) over pulseaudio.
FYI this is the message the node is reporting:

I see that this node is using the below command to find the device list:
bash-5.0$ arecord -l
arecord: device_list:272: no soundcards found...
bash-5.0$

FYI the command arecord -L returns:
bash-5.0$ arecord -L
null
    Discard all samples (playback) or generate zero samples (capture)
default
    Default ALSA Output (currently PulseAudio Sound Server)
bash-5.0$

I managed to get it working for pulseaudio.
See the changes in my forked branch : https://github.com/janvda/node-red-contrib-sox-utils/tree/pulseaudio_support
The main change is that in case of error due to arecord -l : not to hide the options so that I can choose the default device and specify the options I want.

Partially fixed in 0.4.1 see continuing discussion in https://github.com/johanneskropf/node-red-contrib-sox-utils/issues/4

I suggest to check my comment https://github.com/johanneskropf/node-red-contrib-sox-utils/issues/4#issuecomment-740610793
The default device setting is reporting a warning.


User annotations on address spaces and address can be added or updated but not removed.

The AddressSpace and Address APIs allow the user to put arbitrary metadata annotations on the objects.  This presumably to allow the user to add business domain specific annotations to drive business process etc.

Currently, the user can add or change annotations via the replace operation (and now patch), but removal is disallowed.  This seems to be a defect.
EnMasse ought to continue to protect is own annotations from modification, but allow the user to add/change/remove others without restriction.
Originally posted by @k-wall in https://github.com/EnMasseProject/enmasse/pull/2369#issuecomment-469157179

I think this is fixed with the move to CRDs.


Toggle completion source

Hey,
I'm currently trying to toggle the completion from copilot on or off, but it seems, that even when toggling to off the completion from copilot.lua, cmp still uses copilot as a source.
Is there any way to toggle the source on or off?

Should be possible once https://github.com/zbirenbaum/copilot.lua/pull/127 gets merged.

This solved my issue. Thanks 👍


update ui

closed #143

Codecov Report

Merging #168 (d0f5f73) into master (bd01f81) will increase coverage by 2.08%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #168      +/-   ##
============================================
+ Coverage     62.65%   64.73%   +2.08%
- Complexity      574      584      +10
============================================
  Files           101      101
  Lines          1971     1971
  Branches        228      228
============================================
+ Hits           1235     1276      +41
+ Misses          661      612      -49
- Partials         75       83       +8

Impacted Files
Coverage Δ

src/main/java/seedu/address/ui/BudgetDisplay.java
0.00% <0.00%> (ø)

.../java/seedu/address/logic/commands/AddCommand.java
88.88% <0.00%> (ø)

...dress/logic/commands/PaidPersonWhoOwedCommand.java
17.64% <0.00%> (+17.64%)
:arrow_up:

...eedu/address/logic/commands/EditPersonCommand.java
41.86% <0.00%> (+39.53%)
:arrow_up:

.../address/logic/parser/EditPersonCommandParser.java
100.00% <0.00%> (+100.00%)
:arrow_up:

...s/logic/parser/PaidPersonWhoOwedCommandParser.java
100.00% <0.00%> (+100.00%)
:arrow_up:

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 bd01f81...d0f5f73. Read the comment docs.

There are some issues with the Tabs. The top left help and file don't work properly. Also when no JOSN is missing/ fresh load , there is a Disclaimer that pops up which is blank

There are some issues with the Tabs. The top left help and file don't work properly. Also when no JOSN is missing/ fresh load , there is a Disclaimer that pops up which is blank

Hi, neither is an issue on my system. Can you help me elaborate the issues on your end. Also, the disclaimer is never empty on my end when there is no JSON

Can update the Ui in UG so to reflect the new Ui you are trying to bring in?

Can update the Ui in UG so to reflect the new Ui you are trying to bring in?

Yep doing, will push soon


CUPS-Supplied Brother DPL3510CDW PPD Cannot Print Colours Accurately

Describe the bug
CUPS itself seems to work fine ... except that ALL colours are wrong.
I have & am willing to upload Microsoft-supplied PPDs that (mostly) work OK with the printer
(unfortunately, this page will NOT accept a ZIP file, nor bare PPD)
(I will try again with a bare PPD renamed as TXT; nope, it refused that file as well)
To Reproduce
Steps to reproduce the behavior:
(using DCP3510W-RGB obtained from Brother website & installed shortly after obtaining printer)

Print anything from anywhere containing a colour
The printed colour does not match the screen colour (nor the actual colour values)
Install (an older) PPD extracted from the Microsoft driver supplied on CD with the printer when purchased
(this is BRPPL3510CDW-CMYK)
Most colours are now accurate
Install CUPS-supplied PPD
(this turns out to be byte-identical to DCP3510W-RGB)

Expected behavior
That printed colours are accurate
Screenshots
No point. The screen preview looks fine, the actual printouts are wrong.
System Information:
Linux ng3 6.0.0-0.deb11.6-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.0.12-1~bpo11+1 (2022-12-19) x86_64 GNU/Linux
LibreOffice <IP_ADDRESS>
CUPS: 2.3.3op2-3+deb11u2
Additional context
$ lpstat -e
BRPPL3510CDW-CMYK
DCP3510W-RGB
$ lpstat -a
BRPPL3510CDW-CMYK accepting requests since Sat 04 Feb 2023 02:33:08 GMT
DCP3510W-RGB accepting requests since Sat 04 Feb 2023 02:39:20 GMT
$ lpoptions -p BRPPL3510CDW-CMYK
copies=1 device-uri=usb://Brother/%20series?serial=E79016L8N315916 finishings=3 job-cancel-after=10800 job-hold-until=no-hold job-priority=50 job-sheets=none,none marker-change-time=0 media=3 number-up=1 output-bin=face-down print-color-mode=color printer-commands=AutoConfigure,Clean,PrintSelfTestPage printer-info=DCPL3510CDW printer-is-accepting-jobs=true printer-is-shared=true printer-is-temporary=false printer-location printer-make-and-model='Brother DCP-L3510CDW BR-Script3' printer-state=3 printer-state-change-time=1675477988 printer-state-reasons=none printer-type=8425564 printer-uri-supported=ipp://localhost/printers/BRPPL3510CDW-CMYK sides=two-sided-long-edge

Hi @alexkemp9 ,
I'm not sure if I understand - so you have one PPD which is used for BRPPL3510CDW-CMYK queue and for BRPPL3510CDW-RGB queue, both uses usb device uri, but you get a different color output? And *-CMYK is the one which works and *-RGB doesn't?
Would you mind attaching the exact PPD file (you can rename it to have .txt suffix, then Github accepts it. .zip files are okay as well, so I'm not sure what kind of problem you hit with it, maybe you tried to load a different suffix?) for both queues?
Additionally do enable debug logging by cupsctl LogLevel=debug2, print again to both queues and attach log files here as well.
Thank you in advance!

Steps taken as requested, zip of all files attached.
On 08/03/2023 11:33, zdohnal wrote:

|cupsctl LogLevel=debug2|

AK

@alexkemp9 I'm sorry, could you attach it via web interface? Otherwise I don't see it in email notification and web page.


Remove zenmine.pro from available endpoints

https://github.com/ZencashOfficial/zencash-mobile/blob/c6626d0048312a531a69a124cd35c30dc6746a88/src/containers/SelectInsightPage.js#L60

This was addressed in the last release.


Support dynamic responses

Again, like WebMock, it would be great to be able to provide a block that would be evaluated at runtime when the endpoint is hit so you could customize the response at runtime using particulars of the request.
This might be kinda clumsy, due to the particulars of how you store the desired responses into the mock service itself, but I don't think it's impossible.
Whatdya think? (Last one! Miss you dude!)

I think this exists, but it might not look like webmock.  Are you familiar with:?https://github.com/stevenjackson/pinch_hitter/wiki/RegisterModule
The block syntax should be doable with a bit of syntax sugar.
Miss you too!  We should set up a time to chat.  You can hit me up at stevejxsn in gmail land.

Ah, sorry, I missed that part of the wiki! I do think some syntax sugar would be nice, so I'll leave this open for that


Update and simplify

Remove a big chunk of code now that https://github.com/chef/chef-server/pull/988 has shipped.
Update AMIs and chef-server-core to the latest version.
make ContactDept configurable
Also update some examples.

cc @itmustbejj for review


Filesystem read/write starter on top of the electron boilerplate.

Please check packages/idoc/src/chapters and packages/idoc/src/essentials for naming conventions and folder structure.
When opening a project folder we need:

[ ] ability to read children json files as data
[ ] ability to write over children json data files from within the app

Also… we don’t need to be running any Gatsby tasks till Preview or Export being triggered from the app.

This is what we are reading from and reading to:
https://github.com/storycopter/storycopter/tree/next/packages/idoc/src/site — global settings for the idoc
https://github.com/storycopter/storycopter/tree/next/packages/idoc/src/essentials — related mainly to Home/Credits pages
https://github.com/storycopter/storycopter/tree/next/packages/idoc/src/chapters — all the user generated pages, we need to be able to create similar jsons when creating new pages…

So we need to be able to scan through these folders https://github.com/storycopter/storycopter/issues/44#issuecomment-581469245, load all these jsons, construct a  state object that looks like this: https://github.com/storycopter/storycopter/blob/feature/51-document-inspector/packages/app/src/reducers/data.js#L24-L32 and save changes to disk before each preview and export


koord-scheduler: remove CompatibleDefaultPreemption plugin

Ⅰ. Describe what this PR does
Compatible with policyv1.PodDisruptionBudget of versions below k8s v1.22 using the same method as compatible with CSIStorageCapacity. So we can remove the compatibledefaultpreemption plugin.

Versions below k8s v1.22 need to enable the feature-gate CompatiblePodDisruptionBudget to custom policyv1.PodDisruptionBudget Informer compatible with policyv1beta1.PodDisruptionBudget.
Users also can disable the PodDisruptionBudget informer via feature-gate DisablePodDisruptionBudgetInformer

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in make test

/lgtm


[Discussion] Questions regarding pre-trained models

So, first of all, thanks for releasing this!
This is a super amazing project that I hope to see improve in the future.
I'm opening this issue because I read that newer pre-trained models were gonna be released soon, and I was wondering if it'll happen in the near future or if this is gonna be kept like this. Seeing that only one commit has been done (to the README file) over in 23 days, I was wondering if this is gonna be worked on in the future.
Again, thanks a lot for the effort you've all put into this amazing project.

Hi @NickAcPT
Thanks for your message.We've just released new models allowing higher frequencies to be generated (in version 1.4.9) but we cannot guarantee that newer models will be released on a regular basis.


How to detect protection key?

First of all, sorry if this is a dumb question. I know nothing about programmings and stuff.
There's a game made with WolfRPG Pro that I'd want to decrypt but when I try to decrypt it with this tool, it says this:

And then an empty Data file gets generated.
Do that means it's impossible to decrypt this game? What should I do?

Hmm, that's strange. Can you give me the name of the game so I can take a look and try to fix the problem?

Umm... don't know if it's ok, since it's a free trial, here's the link:
ttps://ci-en.dlsite.com/creator/813/article/1265100.
Thank you  m(_ _)m

The trial version you linked decrypts just fine for me; you might need to wait a bit for the decryption to finish (~1 minute). The message regarding the protection key can be ignored because although the game is made with WolfPro, the game data is not protected (probably because it is a trial version).
If nothing is extracted even after waiting several minutes, please let me know, and I will look into it.

Sadly, it still did not work for me... The program just shuts off after like 5 seconds I pressed progress button, leaving an empty data file.
I recorded the thing, maybe the video can help you figure out what is happening:
https://imgur.com/a/HHHpGIM
Thank you!

Very strange, the application closing itself implies that it crashes somewhere during the extraction process. I will try to look into it.
Could you try to unpack this game? It's just the example game shipped with the latest version of the free Wolf RPG Editor. It would be interesting if this also crashes the application.
In the meantime, I took the liberty of uploading the unpacked data of the trial version for you here.

I honestly could not thank you for all that you are doing for me...
As for the test game, I was able to decrypt it with no problem. All the decrypted data files appeared normally in the generated folder.
Thank you so much for uploading th unpacked data... I hope you're able to find out what's wrong for my case, too. Again, thank you so much!!!

I have been rather busy lately, and after fixing the problem, I forgot to create a new release. The new version 0.4.1 should no longer crash when trying to unpack this game or any game with a single large file.


ROX-19027: Rename generate netpol to netpol generate

Description
Aliasing seems to be impossible :(
In this PR, I refactored the functionality, so that the binding between cobra commands and NP-Guard integration is minimal. Then, I connected the function to both commands: generate netpol and netpol generate.
Checklist

[x] Investigated and inspected CI test results
[x] Unit test and regression tests added
[x] Evaluated and added CHANGELOG entry if required
[x] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

I think, there are no docs for the technical preview features, but I have updated the readmes

N/A

[ ] Determined and documented upgrade steps

Testing Performed

[x] Bats tests locally and in CI
[x] Unit tests in CI
[x] Running the command manually

@dhaus67 would roxctl generate generate also work if we add the aliases as you suggested?

@dhaus67 this is a bad idea. All combinations of commands would trigger the feature. PTAL:
$ bin/darwin_amd64/roxctl generate netpol
ERROR:  accepts 1 arg(s), received 0   // expected error from the feature, as I provided no input

$  bin/darwin_amd64/roxctl generate generate
ERROR:  accepts 1 arg(s), received 0

$  bin/darwin_amd64/roxctl netpol generate
ERROR:  accepts 1 arg(s), received 0

$  bin/darwin_amd64/roxctl netpol netpol
ERROR:  accepts 1 arg(s), received 0

Hm, one could argue that this may lead to mistakenly doing roxctl generate generate or roxctl netpol netpol, but chances are that this is quite low.
I suppose we can leave the changes as-is for now, I'll review them again.

I addressed all the suggestions. Let's wait for the CI now

/retest


Add Additional views for Helping with maintaining ACL

Adds an ACL model
Adds a tab on the Staff View for WHC Tools that "Lists all ACLS"
that shows all current members of the ACL for the app
added a filter after list that can show all the actions that have to be taken to update the ACL in game to match what Auth knows to be true
includes being kicked from the group (application rejected)
includes removing oneself from the group if allready a member

** IMPORTANT NOTE ** - The Migration contain a default WHC listing because alot of the functions are still semi hardcoded with WHC acl in mind
be very mindful of this line:
migrations.RunPython(insert_default_acl), in the migrations - if it doesnt run then we have to add an ACL titled "WHC" (all caps) to the database (can be done through the Admin panel)
TO DO:

Need to make it so characters can be added to different ACLs even within WHC.
Need to tie into  the signal for state change to Non Member to remove them from the ACLs as well.

All in all I think this is good work. Glad to see more development happening.


"Return to My app at the end of a call"  is not Working

Calling a remote assist app from a unity app is working fine, but i need to return back to my app after ending the call its not working
Can anyone please help me regarding my concern

Can anyone please help me regarding my concern

Hello, I did not test this scenario recently but as far as I know, the way to get back to the app is to ask the user to launch it again.
Sebastien.

@sbovo Thank you for your reply on my concern.
Our aim is to search contacts in AAD and call them. After the end of the call, we want to resume our application.
We are using this code to relaunch our application at the call end.
“string uriToLaunch = @"ms-voip-video:?contactids=bf538576-xxxx-yyyy-zzzz-1d548b9e7010&returnto=alsdk:appName.App";
To search any contact in AAD itself, we are removing “contactid” in uri (contactids=bf538576-xxxx-yyyy-zzzz-1d548b9e7010). However, in doing so, the app is unable to search the contacts and is directly returning to our app after showing the message “no contact id found”.

@sbovo Thank you for your reply on my concern.
Our aim is to search contacts in AAD and call them. After the end of the call, we want to resume our application.
We are using this code to relaunch our application at the call end.
“string uriToLaunch = @"ms-voip-video:?contactids=bf538576-xxxx-yyyy-zzzz-1d548b9e7010&returnto=alsdk:appName.App";
To search any contact in AAD itself, we are removing “contactid” in uri (contactids=bf538576-xxxx-yyyy-zzzz-1d548b9e7010). However, in doing so, the app is unable to search the contacts and is directly returning to our app after showing the message “no contact id found”.
Can you please guide me how to do that..

https://github.com/MicrosoftDocs/dynamics-365-mixed-reality/blob/master/mr-docs/remote-assist/protocol-activation.md
You may see this artical. But my problem is how to resume my own app to the state when I leave.


Build fails when using EME

The build fails when using the latest version from master of shaka-player-embedded and prebuilt_cdm with the following error:
[... output removed ...]
.../utah/vendor/prebuilt_cdm/shaka_plugin/cdm_host.h:25:3: error: unknown type name 'NON_COPYABLE_OR_MOVABLE_TYPE'
  NON_COPYABLE_OR_MOVABLE_TYPE(TimerAndClockImpl);
  ^
.../utah/vendor/prebuilt_cdm/shaka_plugin/cdm_host.h:25:31: warning: redundant parentheses surrounding declarator [-Wredundant-parens]
  NON_COPYABLE_OR_MOVABLE_TYPE(TimerAndClockImpl);
                              ^~~~~~~~~~~~~~~~~~~

.../utah/vendor/prebuilt_cdm/shaka_plugin/cdm_host.h:44:3: error: unknown type name 'NON_COPYABLE_OR_MOVABLE_TYPE'
  NON_COPYABLE_OR_MOVABLE_TYPE(StorageImpl);
  ^
.../utah/vendor/prebuilt_cdm/shaka_plugin/cdm_host.h:44:31: warning: redundant parentheses surrounding declarator [-Wredundant-parens]
  NON_COPYABLE_OR_MOVABLE_TYPE(StorageImpl);
                              ^~~~~~~~~~~~~

.../utah/vendor/prebuilt_cdm/shaka_plugin/widevine_implementation_factory.cc:58:10: error: cannot initialize return object of type 'shaka::eme::Implementation *' with an lvalue of type 'shaka::eme::WidevineImplementation *'
  return ret;
         ^~~
[... output removed ...]

I'm using the following commands to build the frameworks:
$ ../../vendor/shaka-player-embedded/configure --ios --release --disable-demo --disable-tests --ccache-if-possible --gn-args "ios_enable_code_signing=false" --eme-impl ../../vendor/prebuilt_cdm/shaka_plugin/release_cdm.json --cpu x86
$ ../../vendor/shaka-player-embedded/build.py

Building the Shaka Player Embedded framework with no EME integration works flawlessly.

I forgot the Widevine plugin used those macros when I changed their names.  Should be a quick fix.  For the moment, you can just add SHAKA_ to before each of those usages.

It should be fixed if you pull the latest master from the prebuilt_cdm repo (it has the same CDM version).


Change the icons to ones that don't infringe copyright

To be honest, I'm not sure if they do. But it's better to be safe.
Any thoughts? I'm thinking it needs to be circular
Requirements are 2 images, each with two different sizes:

Monochrome system tray icon (16x16 px @1x, 32x32 px @2x).
Colour app launch icon (512x512 px @1x, 1024x1024 px @2x).

Should follow Apple's HMI guidelines.

Thanks to @FiveClawD for the icons:

They look great!


Implement the Llama 3.2 vision models

🚨🚨🚨Model is working and ready for imminent release!🚨🚨🚨
Last few steps:

[x] Forward pass runs
[x] Correct values confirmed from inputs processor
[x] Correct values confirmed from vision model
[x] Correct values confirmed from text model

Implementation status:

[x] MLlamaModel

[x] MLLamaVisionModel

[x] MLlamaPrecomputedPositionEmbedding
[x] MLlamaPrecomputedAspectRatioEmbedding
[x] MLlamaVisionEncoder (tanh gated attn and feedfwd)

[x] MLlamaVisionAttention
[x] MLlamaMlp

[x] MllamaForCausalLM

[x] MllamaCrossAttentionDecoderLayer (tanh gated attn and feedfwd)

[x] MllamaTextRMSNorm
[x] MllamaTextMLP
[x] MllamaTextCrossAttention

[x] MllamaSelfAttentionDecoderLayer

[x] MllamaTextRMSNorm
[x] MllamaTextMLP
[x] MllamaTextSelfAttention

[x] MllamaRotaryEmbedding

[x] _prepare_cross_attention_mask for text <> vision

[x] ImageProcessor

[x] ImagePreProcessor (process images and inputs)

[x] resize
[x] pad
[x] rescale
[x] normalize
[x] split_to_tiles
[x] pack_images
[x] convert_aspect_ratios_to_ids
[x] build_aspect_ratio_mask
[x] InputsProcessor::process_inputs (from seqs)
[x] convert_sparse_cross_attention_mask_to_dense
[x] get_cross_attention_token_mask

[x] Process (apply chat template)

[x] process (from messages)

RUST_BACKTRACE=1 cargo run --features cuda -- --port 1234 vision-plain -m meta-llama/Llama-3.2-11B-Vision -a mllama

Run:
cargo run --features cuda --release -- -i vision-plain -m meta-llama/Llama-3.2-11B-Vision-Instruct -a mllama

And then...
> \image https://www.nhmagazine.com/content/uploads/2019/05/mtwashingtonFranconia-2-19-18-108-Edit-Edit.jpg <|image|>Where was this photo most likely taken?
The photo appears to be taken in the White Mountains of New Hampshire, USA. The mountain in the background is likely Mount Washington, which is the highest peak in the Northeastern United States and is known for its iconic summit and challenging weather conditions.
>

> Hello!
How can I assist you today?
> What is the date?
The current date is September 29, 2024.
> \image https://www.nhmagazine.com/content/uploads/2019/05/mtwashingtonFranconia-2-19-18-108-Edit-Edit.jpg <|image|>Where was this photo most likely taken?
The photo appears to be of a mountainous landscape with a snow-covered peak in the background. Based on the scenery, it's likely that the photo was taken in a region with mountains, possibly in North America or Europe. However, without more information or context, it's difficult to pinpoint the exact location.

That being said, the mountainous landscape and snow-covered peak remind me of the Northeastern United States, particularly the Appalachian Mountains or the White Mountains in New Hampshire. The White Mountains are home to Mount Washington, the highest peak in the Northeast, which is known for its rugged terrain and snowy winters.

If I had to take a guess, I would say that the photo was likely taken in New Hampshire or another mountainous region in the Northeastern United States.
> \image https://upload.wikimedia.org/wikipedia/commons/f/fd/Pink_flower.jpg <|image|>What type of flower is this?
The flower in the photo appears to be a type of wildflower or a daisy-like flower. However, based on the shape and color of the petals, it's difficult to determine the exact type of flower without more information or a closer look.

That being said, the flower's white petals and yellow center remind me of a daisy or a sunflower. However, the petals seem to be slightly more delicate and have a more rounded shape than a typical daisy or sunflower.

If I had to take a guess, I would say that the flower is likely a type of wildflower, such as a buttercup or a dandelion. However, without more information or a closer look, it's difficult to determine the exact type of flower.
>


 Please note, there’s no option to disable this feature. \n \n Note: New text formatting is only available on Web and Mac desktop. \n \n Italic \n To italicize your message, place an underscore on both sides of the text: _text_ \n \n Bold \n To bold your message, place an asterisk on both sides of the text: *text* \n \n Strikethrough \n To strikethrough your message, place a tilde on both sides of the text: \n ~text~ \n \n Monospace \n To monospace your message, place three backticks on both sides of the text: \n ```text``` \n \n Bulleted list \n To add a bulleted list to your message, place an asterisk or hyphen and a space before each word or sentence: \n * text \n * text \n Or \n  - text \n - text \n \n Numbered list \n To add a numbered list to your message, place a number, period, and space before each line of text: \n 1. text \n 2. text \n \n Quote \n To add a quote to your message, place an angle bracket and space before the text: \n > text \n \n Inline code \n To add inline code to your message, place a backtick on both sides of the message: \n `text`'

Done Merged @FZFR thank you, this PR will be included in next release


phone-otp-verification-added

🛠 fixes issue #73
✔ Check List (Check all the applicable boxes)

[x ] My code follows the code style of this project.
[ x] This PR does not contain plagiarized content.
[ x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
@KaizenGirl1111  maam, I added phone otp  verification , you can use it according to your convenience. My code is completed with each and every functionality require to send as well as verify sent otp. So, kindly merge it and 1 more thing I removed my personal account key of that API , you can add yours, You just contact me, whenever you feel difficulty to use my integrated functionality any time after completion GSSOC 22, I will be there.
Thank you @KaizenGirl1111  maam
📷 Screenshots

@KaizenGirl1111  maam, you can merge it , it is a separate file , I haven't commented anyone's code, do not disturbed any pathways

Hello @mayank-del !
Thank you for being a consistent Contributor to our project!
Here is a opportunity I believe you deserve -
https://digitalproductschool.io/apply/
Apply for Digital Product School
Deadline for September cohort - 30


vault_node_name unused

Setting vault_node_name has no effect because it is not implemented in the template for vault_main.hcl.
https://github.com/brianshumate/ansible-vault/search?q=vault_node_name&unscoped_q=vault_node_name

Apparently, there's no node name option in vault config or vault's consul integration config
I had thought setting the node_name would affect the consul service ID of each vault instance.
However, vault's consul service id is hardcoded to be service_name:host:port. Node name doesn't come into play.
It might be good to remove any reference to vault_node_name from the readme file.

Hi, thanks for the heads up.
vault_node_name is indeed nonfunctional; it has been removed in Master and will no longer be present in the v2.2.2 role version when it is released.


Added kinetic dockerfile

ros-kinetic-rospy-message-converter does not seem to be in the official repositories yet. I don't know this procedure but I alread created a kinetic docker file :)

Thanks @reinzor! And good reminder I need to release this for Kinetic. Will work on that this weekend.

@reinzor I released rospy_message_converter to Kinetic today. Thanks for your patience.

Thanks for this library :)
-Rein

On 02 Jun 2016, at 06:55, Brandon Alexander<EMAIL_ADDRESS>wrote:
@reinzor I released rospy_message_converter to Kinetic today. Thanks for your patience.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


I do not see Kubernetes

Only Fabric and docker

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

ID: 25b96907-35cf-ff1f-d7ad-a231aa78418b
Version Independent ID: 620e59a0-5658-9a05-c177-8d0687656e2a
Content: Kubernetes tools tutorial - Visual Studio
Content Source: docs/containers/tutorial-kubernetes-tools.md
Product: visual-studio-windows
Technology: vs-azure
GitHub Login: @ghogen
Microsoft Alias: ghogen

I upgrade my visual studio and now only see one.

@masters3d -- Thanks for your question. It looks like you're having an issue with the product itself, rather than an issue with the documentation. Here are a few options where you might consider asking your question:
https://social.msdn.microsoft.com/Forums/vstudio
https://www.stackoverflow.com
https://www.reddit.com/r/VisualStudio/
As well, you might also consider using the Report a Problem tool to report the issue.
I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.

Documentation says that I can do this. Are there limitations ?


3dmigoto not working

when i launch with cultivation 3dmigoto just doesnt work when i have it checked but when i launch normal genshin with 3dmigoto it works and i made sure to do the proper steps for launching 3dmigoto with cultivation and i tried used both versions of 3dmigoto(Playing mods and development) but it wont work when i launch with cultivation would there be a way to fix this

Are you sure you're launching from the + menu with 3dmigoto checked? If you aren't seeing the 3dmigoto loader open before the game, you're not launching with it. You could also simply launch the 3dmigoto loader before launching the game from Cultivation.

The same thing is happening to me, I ended up here looking for a solution. I even opened 3dmigoto launcher directly as you said, the log is as follows:
"d3d11.dll description: "3Dmigoto - d3d11.dll"
3DMigoto Version 1.3.16
Loaded E:.3dmigoto\d3d11.dll
3DMigoto ready - Now run the game.
Target process found (4480): D:\Genshin Impact\Genshin Impact game\GenshinImpact.exe"
But it does not actually works with cultivation.

This actually solved it, thank you!

yo, so I'm new to modding and I got some questions on how to use 3dmigoto AND how to use HSRGameStarter, if anyone can help me, pls help on how to do it step by step, because my W11 loader on my 3dmigoto ain't popping up

Never heard of HSRGameStarter (you are in a repo for a Genshin launcher, just so you know :P), but you can (and should) post your problems to the GI-Model-Importer Github instead. The same channel with the video of setting up Cultivation also has a video for setting up mods for the game.

I did everything correctly as I saw in the videos and even checked on the page, but whenever I open the game it not only takes a long time to load but also crashes whenever the world is about to open and returns to the launcher screen. How do I resolve this? Is it something in the game? In the program? In mods? On the computer? I do not know what else to do...

oh gosh,i can't find my W11Loader.exe,I'm a noob for using computer


Unable to Install

File "/home/muezza013/Cyberonix/cyberonix.py", line 5, in
from main import *
^^^^^^^^^^^^^^^^^^
File "/home/muezza013/Cyberonix/main/arguments.py", line 6, in
from ipwhois import IPWhois
ModuleNotFoundError: No module named 'ipwhois'
I am Getting this Error, Please tell me the solution to fix it.

Just install that package by using these commands: pip install ipwhois or pip3 install ipwhois.


eslint errors airbnb in atom

i have folowing errors  after open any js-file
Error: /Users/jt3k/Work/_playground/react-boilerplate/node_modules/eslint-config-airbnb/rules/react.js: Configuration for rule "react/jsx-sort-props" is invalid: Value "data["0"].shorthandLast" has additional properties.

Referenced from: airbnb Referenced from: /Users/jt3k/Work/_playground/react-boilerplate/package.json
Error: /Users/jt3k/Work/_playground/react-boilerplate/node_modules/eslint-config-airbnb/rules/react.js:
	Configuration for rule "react/jsx-sort-props" is invalid:
	Value "data["0"].shorthandLast" has additional properties.
Referenced from: airbnb
Referenced from: /Users/jt3k/Work/_playground/react-boilerplate/package.json
    at validateRuleOptions (/usr/local/lib/node_modules/eslint/lib/config/config-validator.js:115:15)
    at /usr/local/lib/node_modules/eslint/lib/config/config-validator.js:162:13
    at Array.forEach (native)
    at Object.validate (/usr/local/lib/node_modules/eslint/lib/config/config-validator.js:161:35)
    at load (/usr/local/lib/node_modules/eslint/lib/config/config-file.js:522:19)
    at /usr/local/lib/node_modules/eslint/lib/config/config-file.js:391:36
    at Array.reduceRight (native)
    at applyExtends (/usr/local/lib/node_modules/eslint/lib/config/config-file.js:362:28)
    at load (/usr/local/lib/node_modules/eslint/lib/config/config-file.js:529:22)
    at /usr/local/lib/node_modules/eslint/lib/config/config-file.js:391:36
Close All
Error: /Users/jt3k/Work/_playground/react-boilerplate/node_modules/eslint-config-airbnb/rules/react.js: Configuration for rule "react/jsx-sort-props" is invalid: Value "data["0"].shorthandLast" has additional properties.

Referenced from: airbnb Referenced from: /Users/jt3k/Work/_playground/react-boilerplate/package.json
Error: /Users/jt3k/Work/_playground/react-boilerplate/node_modules/eslint-config-airbnb/rules/react.js:
	Configuration for rule "react/jsx-sort-props" is invalid:
	Value "data["0"].shorthandLast" has additional properties.
Referenced from: airbnb
Referenced from: /Users/jt3k/Work/_playground/react-boilerplate/package.json
    at validateRuleOptions (/usr/local/lib/node_modules/eslint/lib/config/config-validator.js:115:15)
    at /usr/local/lib/node_modules/eslint/lib/config/config-validator.js:162:13
    at Array.forEach (native)
    at Object.validate (/usr/local/lib/node_modules/eslint/lib/config/config-validator.js:161:35)
    at load (/usr/local/lib/node_modules/eslint/lib/config/config-file.js:522:19)
    at /usr/local/lib/node_modules/eslint/lib/config/config-file.js:391:36
    at Array.reduceRight (native)
    at applyExtends (/usr/local/lib/node_modules/eslint/lib/config/config-file.js:362:28)
    at load (/usr/local/lib/node_modules/eslint/lib/config/config-file.js:529:22)
    at /usr/local/lib/node_modules/eslint/lib/config/config-file.js:391:36

may be let remove airbnb config?  its config make more problem for developers

npm install -g eslint-config-pedant

and change airbnb to pedant in package.json  here
  "eslintConfig": {
    "parser": "babel-eslint",
    "extends": "pedant",
    "env": {

fix this problem.

I'm also experiencing this. I don't love the solution of changing away form airbnb to solve...

I use Atom with react-boilerplate and don't have this problem.

Note: If you installed ESLint globally (using the -g flag) then you must also install eslint-config-pedant globally.
A globally-installed instance of ESLint can only use globally-installed ESLint plugins. A locally-installed ESLint can make use of both locally- and globally- installed ESLint plugins.

That npm install -g eslint-config-pedant works suggests that you have eslint installed globally. Is that right? If so, you probably need to npm uninstall -g eslint, or npm install -g ... all the eslint related packages used in react-boilerplate.

may be let remove airbnb config? its config make more problem for developers

While linting style is a matter preference, this project has standardized on airbnb and I don't see that changing. Still, it's nice to have an example that shows how easy it is to change to pedant.
Thanks for sharing.


parameter set auto generation

I think we need a mechanism for defining each param and metadata once, but with a configurable number of each (that can be overridden per board).
Many parameters in the system have an index.

CAL_ACCN_*
CAL_GYRON_*
CAL_MAGN_*
COM_FLTMODEN
PWM_AUX_*N
PWM_MAIN_*N
RCN_*
RC_MAP_AUXN
RC_MAP_PARAMN
temp cal (TC_*)

Why?

removes a huge amount of manually maintained param boilerplate and metadata, the list above accounts for > 1/3 of all params in the system

in the past some of the metadata became noticeably out of sync

better scalability and control per board

large configurations with more than 8 PWM outputs per output module
extremely small configurations where limiting the param generation to a more realistic maximum would be a worthwhile savings

future expansion

multi-EKF
parameter configuration for an arbitrary number of airspeed sensors, GPS, lidar units, etc

Thoughts? @LorenzMeier @bkueng @davids5

Sounds good. Something like
/**
 * ID of Magnetometer the calibration is for.
 *
 * @group Sensor Calibration
 * @array-min 0
 * @array-max 3
 */
PARAM_DEFINE_INT32(CAL_MAG*_ID, 0);

?
That's simple and we can generate the same XML from that.
Some questions:

Do we need more than one dimension?
Does the size need to be per board? What are the savings and are they worth it? It will add complexity and is a potential source or errors, since testing is usually done on one board.

That should work. There aren't many cases where more than one dimension would help (TC polynomial coefficients).
Override per board is a bit of a stretch goal. One not entirely contrived example would be if someone wanted to squeeze PX4 into something tiny and flash constrained with a single IMU. Not having to pay for the additional sensor params in flash (several hundred) would save a few kilobytes. Let's not worry about it for now, I think it's solvable later going in this direction.
The other minor piece would be specifying the index in the metadata.

@dagar Regrading the parameters definition. Why is it on a .c file? that is not a code, its not even tries to be compiled. there might be other implications on the past that might be irrelevant now?
From what I understand the data from the .c files are only taken by scripts to the XML file.
can't this data be on other way? something like small XML files maybe? that will make your ideas much more easier.
(I understand the compatibility issues, and that work is not trivial)
About this issue too:
PX4_PARAM_DEFINE.. is almost not used and can be removed, are you agree?

It's historical, and obviously not C code anymore. The issue is we want to move to defining them in json or xml directly, so it's not worth the project churn to rename them .c to .param (or something), and then rename then yet again right after.


IOS-1368: Removed resources from BarcodeHeroUI

JIRA URL
https://spothero.atlassian.net/browse/IOS-1368
Description
Since SPM doesn't support resources and BarcodeHeroUI is a pretty light project, I implemented everything programmatically and ripped out resources.
There are some declarations for requirements of iOS 9.0 and iOS 11. Because BarcodeHeroCore doesn't require bumping to iOS 9.0, this is important to allow both packages to coexist in a single repository. Eventually, SPM will support monorepos and potentially platform specifications between library, target, and module definitions. Until then, I want to continue to make all of our frameworks as accessible as we can.
Tests are failing, so I'll follow-up with another PR to change how Fastlane is running tests (to work like SpotHeroAPI). I'm not concerned about the tests failing temporarily--it's a very simple test and we need more coverage to make it worth it.

          2 Messages

:book:
👍 Pull request title and body JIRA tickets match!

:book:
👋 👋 Great job!

  Generated by :no_entry_sign: Danger

          2 Messages

:book:
👍 Pull request title and body JIRA tickets match!

:book:
👋 👋 Great job!

  Generated by :no_entry_sign: Danger

          3 Warnings

:warning:
.zinc/Dangerfile#L1 - The name of this source file (Dangerfile) should use snake_case.

:warning:
.zinc/Dangerfile#L90 - Favor modifier if usage when having a single-line body. Another good alternative is the usage of control flow &&/||.

:warning:
.zinc/Dangerfile#L95 - Favor modifier if usage when having a single-line body. Another good alternative is the usage of control flow &&/||.

          1 Message

:book:
👍 Pull request title and body JIRA tickets match!

  Generated by :no_entry_sign: Danger

          3 Warnings

:warning:
.zinc/Dangerfile#L1 - The name of this source file (Dangerfile) should use snake_case.

:warning:
.zinc/Dangerfile#L90 - Favor modifier if usage when having a single-line body. Another good alternative is the usage of control flow &&/||.

:warning:
.zinc/Dangerfile#L95 - Favor modifier if usage when having a single-line body. Another good alternative is the usage of control flow &&/||.

          1 Message

:book:
👍 Pull request title and body JIRA tickets match!

  Generated by :no_entry_sign: Danger

          3 Warnings

:warning:
.zinc/Dangerfile#L1 - The name of this source file (Dangerfile) should use snake_case.

:warning:
.zinc/Dangerfile#L90 - Favor modifier if usage when having a single-line body. Another good alternative is the usage of control flow &&/||.

:warning:
.zinc/Dangerfile#L95 - Favor modifier if usage when having a single-line body. Another good alternative is the usage of control flow &&/||.

          1 Message

:book:
👍 Pull request title and body JIRA tickets match!

  Generated by :no_entry_sign: Danger

          6 Warnings

:warning:
.zinc/Dangerfile#L1 - The name of this source file (Dangerfile) should use snake_case.

:warning:
.zinc/Dangerfile#L90 - Favor modifier if usage when having a single-line body. Another good alternative is the usage of control flow &&/||.

:warning:
.zinc/Dangerfile#L95 - Favor modifier if usage when having a single-line body. Another good alternative is the usage of control flow &&/||.

:warning:
BarcodeHero.podspec#L1 - The name of this source file (BarcodeHero.podspec) should use snake_case.

:warning:
BarcodeHero.podspec#L11 - Use the new Ruby 1.9 hash syntax.

:warning:
BarcodeHero.podspec#L12 - Use the new Ruby 1.9 hash syntax.

          1 Message

:book:
👍 Pull request title and body JIRA tickets match!

  Generated by :no_entry_sign: Danger


select value is not reflected if the list of options are loaded at a later time

Describe the bug
I am loading a list of select options from an API so the options get populated a bit after the element is rendered. The value of the select is already set but is not reflected after the list is loaded.
Your Example Website or App
https://playground.solidjs.com/anonymous/afff844c-d8cb-40ab-b368-7b0fe268ab6d
Steps to Reproduce the Bug or Issue

create a
set its value
use  to populate its options from an array
load the array of options using a timeout so that is populated after the render

Expected behavior
I expect the value of the select to be reflect the correct option regardless of when that option is rendered
Screenshots or Videos
No response
Platform

Browser: [Chrome]

Additional context
No response

You can make it work with either:

Update state.color to different value, so that the <select> gets notified of update, like so:

const state = createMutable({
  color: undefined, //undefined, so later it we can assign different value
  options: [];
})
setTimeout(() => {
  state.options.push(Color.Red)
  state.options.push(Color.Blue)
  state.color = Color.Blue
}, 1000)

A bit more implicit, but subscribe to state.options.length change in <select>'s value:

return <select value={state.options.length && state.color} ... >

Or similarly you could use index instead of value of your color (but this requires new object for state.options, as we don't subscribe to it's length):

const state = createMutable({
  colorIndex: 1,
  options: [];
})
setTimeout(() => {
  state.options = [Color.Red, Color.Blue]
}, 1000)
return <select value={state.options[state.colorIndex]} ... >

Take bit different approach and instead of using value on <select>, use selected on <option> with use of createSelector (or state.color === o):
https://playground.solidjs.com/anonymous/85464d7a-2dfa-479d-93fb-8e93f04b745d

But I wonder if it's possible to make it work for your code

@maciek50322  thank you for the detailed workarounds. Since this works automatically with Vue and React it would be nice to have one less gotcha for devs making the transition to Solid.

We don't re-render like React or Vue so there has to be some level of different expectations here. My usual go to is how VanillaJS works. If you have a select that has a value and options are added later does it work?
https://playground.solidjs.com/anonymous/70df1286-3b10-48c6-9b5a-c7c2ae93d018
Apparently not.. value has to always be reset after options change.
There is a big difference to me between things that are conceptually synchronous not working as that makes no sense(order shouldn't be your concern), and things that happen asynchronously.
I do think select is an awkward design. The fact it forces you to set its value again is tedious. I'd be willing to look at solving this but I think it starts with being able to come up with a vanilla approach. The problem is we will not be re-rendering, and reactively these things can be changed independently as they can exist in different files even with no connection. They can be wrapped in components even or rendered using Dynamic.

https://playground.solidjs.com/anonymous/70df1286-3b10-48c6-9b5a-c7c2ae93d018 Apparently not.. value has to always be reset after options change.

Interesting thing here is that it won't change even if you do
select.value = Color.Red

At the end, after adding options. I assumed solid does some custom magic for select value prop (similar to svg props),
value isn't html's attribute for select, as mentioned in thread here
https://stackoverflow.com/a/44798498

We don't use attributes for value and always we use the property.. That works. Also If you move the select.value into the timeout then it works. But we have no way of know we need to do that. The problem is the value must be set after the option is present otherwise it won't show it even if an option of the same name is added later. That is weird DOM behavior but also has us in a weird place unless we want to maybe like listen to DOM events.. like create mutation observer maybe? All we can do is declaratively make sure it runs last. Which we can do. But that doesn't help if someone tries to add options after the fact without setting the value as well.

Something forgotten this days, is that you can just tell which value is selected when creating the options.
https://playground.solidjs.com/anonymous/425eb7c6-1a16-49ed-8eaf-f49cb207dd86
The only thing I have changed, is to add a selected attribute to the option as in
<option value={o} selected={state.color === o}>


Question, is there a way to have a condition on a variable?

I'm looking for something like:
{{ if eq getv "/datacenter" "mydc" }}
    {{ "we are in our dc!" }}
{{ else }}
    {{ "some default value" }}
{{ end }}

but its not working i'm just guessing here the syntax... is there a syntax for conditioning?
thanks

Try separating it in parentheses like {{ if eq (getv "/datacenter") "mydc" }}

thanks indeed!


Fix multi-line parameter defaults

I noticed a bug with the new dynamic parameter handling feature where default values containing newlines were breaking the JS. This PR addresses that very simply by escaping newlines. I don't know enough about AlpineJS to know if there's a more elegant way to accomplish this escaping. Happy to adjust if there's something cleaner.

Hey @GabeIsman many thanks for this. I think this looks good as a fix for now, the whole parameter escape process definitely needs a clean up and extracting into a service or similar where it can be better tested but until then this is just fine.
Appreciate you taking time to figure the issue out and open the PR, thanks again 👍


add PHP7.4

https://github.com/tightenco/overload/pull/3 will allow the tests to run on PHP7.4

Thanks @chapeupreto!


Buffer Limit?

My neovim tends to get sluggish with many buffers open (>15), which happens pretty fast when coding. What would be the best way to auto-close "old" buffers when opening a new one? Is this something this plugin could do or do I need to do some hacking elsewhere?
Thanks again for your awesome plugin, does the job perfectly!

Closing "old" buffers (whatever the logic used to determine if a buffer is old might be) is definitely out of the scope of this plugin.
You could write a function that closes the oldest buffer if there are more than 15 buffers opened and execute it everytime a new buffer is opened with an autocmd, something like
_G.close_oldest_buffer_or_cpu_will_explode = function()
  -- ...
end

vim.cmd('autocmd BufAdd * silent call v:lua.close_oldest_buffer_or_cpu_will_explode()')


Updated comment.

Corrected capitalization in comment.

@jasonjyu is it for the hacktoberfest t-shirt? :thinking:hack

😅


Testing/issue 1194 add typedefs test

Summary
Add tests for the primitive's typedefs.hpp
Tests
Tested instantiations for size_type, matrix_d, vector_d, row_vector_d
Side Effects
Longer jenkins runs due to more tests.
Checklist

[x] Math issue #1194

[x] Copyright holder: Andre Zapico 2019
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

inactive


References to undefined CAMetalLayer, etc. symbols

As of ac4103a733c479b599aae8d42c08cabd7d5cf48a vulkan-zig generates some references to undefined macOS Zig symbols like CAMetalLayer:
https://github.com/hexops/vulkan-zig-generated/blob/836219ff4305889c4634e90c0d2e9038b8d56c37/vk.zig#L2364
We workaround this currently by appending them to the generated file:
https://github.com/hexops/vulkan-zig-generated/blob/836219ff4305889c4634e90c0d2e9038b8d56c37/update.sh#L18-L24
I've had a TODO to clean this up for a while https://github.com/hexops/mach/issues/822 but just remembered it, so figured I'd file an issue here. Hope that's okay!

Hmm, these were previously supplied by the user. Fyi, there are probably done other types that are broken like this. I wonder why the stuff to define them isn't being generated anymore...

As far as I can see, these types are still generated. Furthermore, this is actually intentional. The vulkan-zig definition for these types is as follows:
pub const CAMetalLayer = if (@hasDecl(root, "CAMetalLayer")) root.CAMetalLayer else @compileError("Missing type definition of 'CAMetalLayer'");

For some foreign types a default is provided, where reasonable. I do not know what type CAMetalLayer is, nor does the Zig standard library provide it. This is why its supposed to be provided as an external type. I guess this is rather obscure functionality.
This should also be the case in your version. @slimsag is this still an issue on your end? It seems to be generating fine for me, on both the xml in the repository and the latest vk.xml. Does the above mechanism provide enough integration for you? I guess if the Mach application defines the main(), then it will be relatively weird to have the user define it. If you have an idea for how to better provide it, please let me know.
I guess for now, we can use opaque{} as a default for CAMetalLayer, but there are still some other types where that will be relatively hard for.

Ok, perhaps I don't have the latest vk.xml or something. As long as it works for you, I can figure out whatever is broken on our end.

I guess for now, we can use opaque{} as a default for CAMetalLayer, but there are still some other types where that will be relatively hard for.

Providing these types, as opaque{} or otherwise, would definitely be my preference. If there are other types vulkan-zig cannot define/provide, perhaps generic parameters somewhere could be used?


Flat and Sharp at the same time

Hey there!
First of all, congratulations for the awesome work you did here.
Second, I've been reading chord.js. I see ninths are either aug, m or M (lines 103-116). This way you either have #9 or b9, not both.
So, in the sharp11-client if you write C11b9#9, it'll be assumed to be a C11#9. However, if you add the note C#, the output you get is indeed C11b9#9.
Shouldn't that be solved?

Hi Francisco,
Thanks for reaching out!  The b9#9 is an interesting case, because it's the only time that you can have an interval appear twice in a chord with two different alterations (unclear whether this is even technically "allowed" but I'm sure you can find it in jazz somewhere).  I'm not too concerned about the inconsistency, because the relationship between chord names and groups of notes can never be one-to-one, since you can have different names for the same chord and also have the same chord represented as different groups of notes.  The real question is is it worth having the chord name parser recognize b9#9.  It would require a special case, which is not out of the question, but I'm not sure if it's pressing.  I'm actually in the middle of somewhat of a sharp11 overhaul at the moment, so I'll leave this issue open and possibly get to it as part of this overhaul.  Thank you for bringing this to my attention.
Best,
Julian

Indeed, the inconsistency is not an issue in this case. However, when trying to identify one chord, sometimes you don't get the expected output. Although it may just be a matter of opinion, when you use C E G# A# D F# A, you get C13b5#5 instead of Caug13#11 (or C+13#11). I'm not sure that's an issue but I'd rather get Caug13#11. This happens because b5=#11 and aug implies #5.
There are a couple of other cases like this. However, ofc you can't just please everyone when it comes to chord names. But I'd still like to hear your opinion on this.

I agree that you should get Caug13#11 in that example.  Currently I choose #11 when there is a perfect fifth present and b5 when there isn't.  I should change the case to perfect fifth or raised fifth.


Problem with rules update - getting error "Rules are stale: Rules are older then two weeks"

Hi,
since today I'm getting error Rules are stale: Rules are older then two weeks. Since it is not common error when dealing with php version audit, I start digging deeper. First of all, I tried call composer update but version was the newest available, which is 1.20240802.0.
This forced me to look closer to the package itself and I found out that there is a problem with github actions, specifically it's call of docker-compose.
Can you look at it?

Thanks for letting me know. I don't use this anymore personally, so was not aware that updates have broken. I'll try to get it fixed in the next couple days

@chrudosvorlicek should be fixed now, thanks for the bug report!===

@lightswitch05 thanks for fast fix


Questions I met when running this code

I'm sorry to say that I don't understand the fourth step in the preparation of Trident.What would I do for choosing the LLVM IR instructions?And when I run the ./test，the log file "fi_breakdown.txt" didnot update the probabilities of SDC,Benign,Crash and breakdown.It just equals to the log file  "fi_breakdown.txt" generated in ./inst_count.

The fourth step in the preparation in the README says that you need to specify the instruction which is considered as the output of the program. For example, if you examine the SDC of your program under test by matching the standard output between the fault-free run and the fault injection runs, you need to specify all the "printf" instructions in the LLVM IR code of your program. That way Trident knows where to stop when tracking error propagation.
Hope this helps.

Thank you for your explanation about the steps.I will try my programs again.Thanks.
------------------ 原始邮件 ------------------
发件人:                                                                                "DependableSystemsLab/Trident"                                                                                @.>;
发送时间: 2021年6月7日(星期一) 中午12:56
@.>;
<EMAIL_ADDRESS>主题: Re: [DependableSystemsLab/Trident] Questions I met when running this code (#7)
The fourth step in the preparation in the README says that you need to specify the instruction which is considered as the output of the program. For example, if you examine the SDC of your program under test by matching the standard output between the fault-free run and the fault injection runs, you need to specify all the "printf" instructions in the LLVM IR code of your program. That way Trident knows where to stop when tracking error propagation.
Hope this helps.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Add support for a special header x-minio-purge-on-delete

this header can be passed to purge the objects on deletion

Corresponding server PR is closed.


Export gRPC server

WHY
WHAT
REF

Codecov Report

Merging #44 into master will increase coverage by 2.32%.
The diff coverage is 89.33%.

@@            Coverage Diff             @@
##           master      #44      +/-   ##
==========================================
+ Coverage   46.76%   49.08%   +2.32%
==========================================
  Files          35       36       +1
  Lines        1251     1316      +65
==========================================
+ Hits          585      646      +61
- Misses        623      624       +1
- Partials       43       46       +3

Impacted Files
Coverage Δ

pkg/grapiserver/gateway.go
72% <ø> (-0.55%)
:arrow_down:

pkg/grapiserver/config.go
85% <100%> (+5.68%)
:arrow_up:

pkg/grapiserver/builder.go
38.8% <100%> (+9.49%)
:arrow_up:

pkg/grapiserver/grpc.go
100% <100%> (ø)
:arrow_up:

pkg/grapiserver/mux.go
100% <100%> (ø)

pkg/grapiserver/engine.go
87.5% <83.67%> (+0.83%)
:arrow_up:

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 21ded8c...12bb4df. Read the comment docs.


Elaborate on goals

I think it would be an idea to elaborate a bit on the goals of this package - and hence on the goals of defining and implementing interfaces in Julia.
(as this package has claimed a rather generic name - and is making good use of it 👍)
I.e. to compile a more elaborate list of goals and get these in the README/documentation.
To compile answers for issues brought up e.g. in

https://github.com/JuliaLang/julia/issues/6975
https://invenia.github.io/blog/2020/11/06/interfacetesting/
https://viralinstruction.com/posts/badjulia/#abstract_interfaces_are_unenforced_and_undiscoverable
https://discourse.julialang.org/t/towards-the-creation-of-an-interface-testing-package/82182

E.g., the obvious one:

An interface should define which methods to implement.

Defining and implementing interfaces should be adoptable by the ecosystem, so the method should be compatible with the current direction of the community:
A. Using traits to define interfaces informally.
B. Reducing TTFX.
C. Enabling static analysis.
D. Enabling static compilation.

These are great, thanks.
We can probably finish them up in a readme PR where anyone can suggest edits and additional points?

We can probably finish them up in a readme PR where anyone can suggest edits and additional points?

Yes 👍

An interface should provide low/no overhead traits so that it can be checked if an interface is implemented for a type

An interface may also specify which types to inherit from

An interface may specify the types (and values??) a method must return

4. There should be (semi)automated testing to prove that an interface is implemented for a specific type, strongly coupled to the interface definition.

Can you elaborate on "(semi)automated" (and testing) and "strongly coupled"?
I guess this means something like:

The interface definition should provide enough information, that it enables the Interfaces package to provide a test method (Interfaces.test), that makes it easy for users to implement tests of their interface implementations? (this is probably too specific for a goal)
The interface definition should provide enough information, that it enables compile-time enforcement/testing?
The interface definition should enable test suites as described in https://invenia.github.io/blog/2020/11/06/interfacetesting/

Or maybe a rephrase of what is currently in the README?

Defining an interface should enable tests.
Or the long form: Defining an interface and stating that the interface is implemented for a type should enable testing that the latter is the case - by exploiting the information in the interface definition.

Sorry - this got unnecessarily long for a "please elaborate" comment :-)

5. An interface should provide low/no overhead traits so that it can be checked if an interface is implemented for a type

Can you elaborate on the concern here?
This is implementation that supports 4? And 2 (the community using traits already?)

6. (Maybe this should be 2)

We can always reorder in the PR.

6. An interface may also specify which types to inherit from.

I.e. the goal is to support hierarchies of interfaces (as mentioned in #6)?
E.g. to support:
@interface AnimalInterface ...
@interface BirdInterface <: AnimalInterface ...
@implements BirdInterface Duck

and thus inheriting AnimalInterface in BirdInterface - and stating that both AnimalInterface and BirdInterface is implemented wrt. Duck.

Interface definition - and interface implementation statement - should support multiple dispatch.

I.e., methods dispatch wrt. multiple arguments, so the interface definitions should support defining methods wrt. tuples of types.

I guess its hard to be more specific because there are various ideas floating around, and annoying constraints on optimal designs.

I was trying to capture that the interface definition contains tests. The strong coupling is via being in the same macro so the structure is forced. Previously the @implements macro actually put all the tests in a precompile function and ran it, so there was no ambiguity at all about the interface being tested - it was "automated" testing.
Now what we have is semi-automated, in that you get a function defined that runs tests, but you have to manually run it somewhere in your test suite and we have to trust that everyone does that. This is a compromise for performance.

The interface definition should provide enough information, that it enables the Interfaces package to provide a test method (Interfaces.test), that makes it easy for users to implement tests of their interface implementations? (this is probably too specific for a goal)

This might be the most accurate goal though

The interface definition should provide enough information, that it enables compile-time enforcement/testing?

I don't think the compile time testing part is broadly possible in practice

The interface definition should enable test suites as described in https://invenia.github.io/blog/2020/11/06/interfacetesting/

The invenia blog style is the basis of a lot of this. But it hasn't lead to a consistent recognizable way of reading and running interface tests across the ecosystem. So we are aiming for a little more structure than that.
Or maybe a rephrase of what is currently in the README?

Defining an interface should enable tests.

As mentioned above it used to, but currently it does not automatically enable them, and its an open question if it should.

Or the long form: Defining an interface and stating that the interface is implemented for a type should enable testing that the latter is the case - by exploiting the information in the interface definition.

As a generic statement this one is pretty good :)

This was to make it clear that traits and tests are not the same thing, and probably need to stay decoupled. So its really in opposition to 4. I have just found this to be a common point of confusion.

This was simply saying that interfaces are not just about defining methods, and point 1 is incomplete. You can define all the methods in the AbstractArray interface of your own type, but it only follows the interface by also being <: AbstracatArray.

7. An interface may specify the types (and values??) a method must accept and return

I would leave the "values" part out of this to make it less controversial :-) Maybe put the "interface provides values" in a list of candidate goals being considered.

hah sorry I should have broken that up

hah sorry I should have broken that up

Feel free to edit (for the time being) :-)

This was to make it clear that traits and tests are not the same thing, and probably need to stay decoupled. So its really in opposition to 4. I have just found this to be a common point of confusion.

This was simply saying that interfaces are not just about defining methods, and point 1 is incomplete. You can define all the methods in the AbstractArray interface of your own type, but it only follows the interface by also being <: AbstracatArray.

This was to make it clear that traits and tests are not the same thing, and probably need to stay decoupled. So its really in opposition to 4. I have just found this to be a common point of confusion.

Yes - good to get things clarified! 🙏 👍
So this was a response to 5 (not 6)? I.e.

An interface should provide low/no overhead traits so that it can be checked if an interface is implemented for a type

Can you elaborate on the concern here?
This is implementation that supports 4? And 2 (the community using traits already?)

yeah, 5 :sweat_smile:


Bootstrap Ecto example app with defaults

This PR adds essential defaults (docs, config, and seed script) to the example Friends Ecto app for quicker bootstrapping when testing various Ecto features within the sample app.

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:


Enable to bulk clear ignore area for test runs list and details view

Let's consider following conditions

I want to verify mobile images where I want to ignore the header (contains dynamic value like network strength, battery level, time etc.)

Manually applying ignore area for all images is troublesome especially if we have a lot of images in the build. There should be some mechanism to apply ignore all for all images. Similarly, if I want to clear/modify ignore areas for all, there must be some way to achieve that.

Something like this image

@suratdas the idea of having some bulk operations helping to manage results on UI side sounds really appealing to me
I would suggest to separate it onto smaller parts something like

clear ignore area
copy ignore area from one to another
bulk operation with those

meanwhile have you tried to add ignore areas programmatically via SDK?
this might help to accomplish something similar while this is not yet implemented

@suratdas thanks for contribution!


Add RPC support for unconfrimed Omni transactions

This PR adds support for unconfirmed transactions via RPC, as well as for the transaction details shown in the UI, when clicking on a pending transaction or trade.
The base fields were slightly reordered, to group related values, and the fields "blockhash" and "block" were added.
Only if a transaction is confirmed, the fields "valid", "blockhash", "blocktime" and "block" are included. Unconfrimed DEx payments are rejected, indicating that the transaction is unconfrimed. Because the effect on the global state is only avaiable for confirmed transactions, the extended mode, which is usually interacting with global state, is downgraded to basic mode, if the transaction is unconfirmed. This has an impact on "omni_getsto" or "omni_gettrade", but in any case, the basic mode also returns at least the information that would be available via the pending map.
Because the data can now directly be retrieved, there is no longer need to fetch pending transactions from the pending map, which allows to remove the description of pending transaction objects.
The PR resolves #171 for the most part.

From this point on it looks more or less straight forward to provide full support of pending transactions in the UI, namely by adding a pending entry on every send. I tested it and it works, however, it wasn't entirely clear how the pending-amounts-to-subtract, the exception for MetaDEx cancels in PendingAdd, and the amount [...] labels in the UI fit together. For some transaction types the amount and property shown in the UI should be set to "N/A" or "Unknown". It's further not clear to me how to deal with positive pending amounts.
Another interesting question comes to my mind:
With full parsing support of unconfirmed transactions, do we even need to keep track of the details of pending transactions and handle them differently in the UI?

Note: I submitted this PR seperately from #142 or #173, and I don't intend to add the UI support in this one, to keep the size and impact of each PR smaller.

Reviewed, looks great to me bud :)
Tested with pending simple sends & trades, seems all good...
Really cool stuff dude thanks :)
This'll help nicely with OmniChest.info unconfirmed support !!!  Also I'm pretty sure I can make some adjustments to the history to make pending wallet txs persist in UI through shutdowns of the client.
One thing I'm not sure about, you removed the ***THIS TRANSACTION IS UNCONFIRMED*** notification on pending transactions in the UI - do you think this was overkill?
Interesting timing on the removal of the need to check levelDB and if (parseRC < 0) return MP_INVALID_TX_IN_DB_FOUND; , I'm actually trying to troubleshoot an issue where someone is stuck getting this exact error hehe...

With full parsing support of unconfirmed transactions, do we even need to keep track of the details of pending transactions and handle them differently in the UI?

I'd like to move away from custom handling in the UI, but we do always need to keep track of at least the hash and amount so that we can track available balances (which should decrease with sent pending txs, 100% agree we should add PendingAdd to RPC layer too)...
Overall I think this is good to merge :)

Awesome, thanks!

One thing I'm not sure about, you removed the ***THIS TRANSACTION IS UNCONFIRMED*** notification on pending transactions in the UI - do you think this was overkill?

Yeah, I was really thinking about it, but I didn't find a satisfiying way to include it. Since populateX doesn't indicate, whether the transaction is confirmed or not, it would need additional means to determine, whether the transaction is unconfrimed. So I was thinking: the "confirmations" field in the result already provides a hint; the icons in the UI do so, too. And to open the transaction info dialogs, explicit user actions are necessary. In the end I decided to just remove it for simplicity.
But it should be noted that having some further indication of the confirmation status might be a good thing. Ideally, in my opinion, as middle ground, maybe there could be a "status" field, which shows something like "pending", "orphaned", "confirmed" or so. Unfortunally that conflicts with the MetaDEx status. :(

which should decrease with sent pending txs, 100% agree we should add PendingAdd to RPC layer too

Yeah, my main issue with it was that I didn't really know how/where the UI constructs the entries (in particular: color, positive/negative amounts etc.), so I just mirrored the original behavior for now. I added the flag bool fSubtract to PendingAdd, to indicate whether the amount should have an effect on the pending balance.


Question About Metrics

Hi there,
Can I get the idea how you implement evaluating metrics the FID, FVD,  Sync-C etc.?
If you can provide related script, that would be even better.
Thanks

FID: use python lib named pytorch_fid
FVD: you can refer  to the implementation of Disco(https://github.com/Wangt-CN/DisCo/tree/main)
Sync-C and Sync-D: refer to https://github.com/joonson/syncnet_python
E-FID: we use insightface to get landmarks, and then you can refer ER-Nerf (https://github.com/Fictionarry/ER-NeRF/blob/main/data_utils/process.py) to implement face_tracking. After that, you can load 'exp' from saved file, and calculate fid according to its definition.


Fix deserialize packed transaction

Deserializing packed transactions was throwing a null reference exception due to the ReadAbiStruct method being called with object as the type argument. Also changed type evaluation that wasn't working for IDictionary<string, Object>.

Thanks for finding this bug.
I'm not happy with the dictionary usage but it is probably the only option to make it cross-platform. This bug prob was introduced when I removed ExpandoObject that is not supported in Unity3D or any dynamic object types.
Can you provide a test example to test your fix?

Hi Mário,
I found this bug when trying to deserialize a packed transaction directly through the AbiSerializationProvider. In my particular use case, I had a propose transaction sent to eosio.msig and was trying to deserialize the packed transaction (a simple transfer transaction in eosio.token) included in the proposal table, so trying to deserialize a packed transfer transaction should serve to test it.
Please let me know if you need more details or if something needs to be changed.

Did a little optimization by avoiding the FindObjectName on the case of IDict type.


Update variables for Ansible v2

Thanks for the PR. Looks like I need to bump the Ansible version in the requirements file as well to get Travis green again.


OVN packages version bump to 22.12

Need to bump to 22.12 to get to the same level as already deployed on many customers yoga-20230905T130221.

Built new containers centos and rocky for ovn and openvswitch: yoga-20231019T102525


The latest dev build works perfectly with DevHunter (the Easter Egg in Excel 2000)

Thanks for your efforts of creating DDrawCompat! I have recently extracted the DevHunter game from Excel 2000 and sought to make it run on Windows 10. I tried several compatibility layers and only this one works! I will soon create a repo to share what I have done.
P.S.: The last stable build also works, but produces amusing graphics rather than the correct ones.

Thanks for the feedback! Since this isn't really an issue, I'll close it now. I'll think about adding a wiki, though I haven't really got time to maintain one, and I'm not sure there are enough interested people who would want to do it or even use it, either.


fix: fix token expiration

Issue #13
Happy to entertain how you'd like to unit test this change.  We could setup boundary in a test container I guess but I'll leave that to you, just let me know if you want unit tests and how to proceed.

@devops-rob Any chance this can get looked at?


No error detail overlay display in webpack v5

Current behavior
when using webpack v4:

when using webpack v5:

Expected behavior
show normally
Steps to reproduce the issue
here is the demo:
demo.zip
yarn
yarn start

update deps version to v4 show differents:
{
    "webpack": "^4",
    "html-webpack-plugin": "^4",
    "ts-loader": "^4",
}

Issue reproduction repository
Environment

fork-ts-checker-webpack-plugin: v6
typescript: 4.3.2
eslint: 7.28.0
webpack: 5
os: macos big sur 11.3

This seems to be due to the "webpack-dev-server": "^3.11.2" dependency, which does not play well with Webpack 5.
If you update to "webpack-dev-server": "^4" (and the corresponding webpack.config.js to devServer: {}) this works fine.

thx a lot~


[UWP] Problem with FFImage control in new instance of app

Hi, I tried to open xaml Page (which contains FFImage xaml control) in new window using this code from Template10. Unfortunately it throws System.Exception:

System.Exception: The application called an interface that was marshalled for a different thread. (Exception from HRESULT: 0x8001010E (RPC_E_WRONG_THREAD))

It'd be great if someone could help me with this. 😃
Regards.

It's this: https://github.com/luberda-molinet/FFImageLoading/issues/166