"TypeError: Cannot read property 'addEventListener' of undefined

Hello. I try to use your videogular (code from http://codepen.io/2fdevs/pen/KmDIE) in one project (in Visual Studio 2015, Apache Cordova App, AngularJS 1.4.8, videogular 0.3.1) and get this error:
"TypeError: Cannot read property 'addEventListener' of undefined
at link.pre (http://localhost:4400/scripts/videogular.js:593:33)
at http://localhost:4400/scripts/angular.js:8835:44
at invokeLinkFn (http://localhost:4400/scripts/angular.js:8841:9)
at nodeLinkFn (http://localhost:4400/scripts/angular.js:8314:11)
at compositeLinkFn (http://localhost:4400/scripts/angular.js:7731:13)
at nodeLinkFn (http://localhost:4400/scripts/angular.js:8330:24)
at compositeLinkFn (http://localhost:4400/scripts/angular.js:7731:13)
at compositeLinkFn (http://localhost:4400/scripts/angular.js:7734:13)
at publicLinkFn (http://localhost:4400/scripts/angular.js:7611:30)
at http://localhost:4400/scripts/angular.js:1681:27
Can you help me with this problem?

Did you solved the issue?

Yes. Nuget installed older version of videogular. So, I just manually downloaded latest version from http://static.videogular.com/scripts/videogular/latest/...

Thank you for let me know.
We don't manage Nuget versions, so I don't know what are the latest versions published there.

I have another one question, if it's possible. In my app whene I go from mainpage to category, then from category to item, the start playing video, then go back with Angular routing "goBack" function, video is stoping automaticaly. But if I go from mainpage directly to item and start playing, video doesn't stop after "goBack". Do you have any ideas, why?

Videogular listens for $routeChangeStart event, if you're not using Angular Route (ngRoute) then it would not work I think and you should call API.clearMedia() on your own listener.

Thank you, i'll try this.

i am facing the same issue. please list the right version.


Cannot get "name" on App\Entity\Task|null.

I'm getting
Cannot get "name" on App\Entity\Task|null.
🔖 getAttribute.unknown
🐘 compiled_translationDetail.php:312
🌱 templates/Dashboard/TranslationV2/translationDetail.html.twig:262
🎯 src/Controller/Dashboard/TranslationV2Controller.php:181
🎯 src/Controller/Dashboard/TranslationV2Controller.php:207
🎯 templates/Dashboard/TranslationV2/index.html.twig:224

And it feel wrong to me for two reasons:
First, the code is
{% if translationTask is empty %}
     Foo
{% else %}
     <span>{{ translationTask.name }}</span>
{% endif %}

so twigstan should be able to know translationTask is not null.
Secondly, I'm using PHPStan level 7, so Foo|null error are not yet reported on my codebase.
Since Twigstan use the same phpstan config, it should be level 7 and not report such error yet by the getAttribute rule.
I feel like there is a RuleLevelHelper::findTypeToCheck you don't use in your rule

I implemented the GetAttribute functionality only recently, and made the decision to error on null. But you're correct, this should follow PHPStan's level / config. I'll work on this.

Yes, and I dunno if {% if translationTask is empty %} is not fully understood by twigstan yet

It is, at least for {% if translationTask %}
https://github.com/twigstan/twigstan/blob/main/tests/EndToEnd/GetAttribute/narrowing.twig

I implemented the GetAttribute functionality only recently, and made the decision to error on null. But you're correct, this should follow PHPStan's level / config. I'll work on this.

I have a related error which might not worth opening another issue
Neither the property "tone" nor one of the methods "tone()", "gettone()", "istone()", "hastone()" or "__call()" exist and have public access in class "Symfony\Component\Form\FormView".
🔖 getAttribute.notFound
🐘 compiled_form.php:145
🌱 templates/Dashboard/LanguagePair/form.html.twig:98
🎯 src/Controller/Dashboard/ProjectLanguagePairController.php:141

FormView is an arrayAccess and currently twigstan reports tons of error for code like
 {{ form_label(form.tone) }}

It is, I created it for your: https://github.com/twigstan/twigstan/issues/159
I didn't bother yet with ArrayAccess in GetAttribute.

I implemented the GetAttribute functionality only recently, and made the decision to error on null. But you're correct, this should follow PHPStan's level / config. I'll work on this.

When reading the current implementation, I feel like the GetAttributeCheck has multiple issues:

It should use $this->ruleLevelHelper->findTypeToCheck for the GetAttributeRule but not for the GetAttributeReturnType

You take only the first constant string but I'm unsure if it's possible to have more
https://github.com/twigstan/twigstan/blob/1d36397e3843da56da879c7f9f0b4171b0a44a9d/src/PHPStan/GetAttributeCheck.php#L69
at least if would be safer to change if ($constantStringTypes === []) { to if (count($constantStringTypes) !== 1) {

When there is an error you returns ErrorType (Like in the Task|null) case, but you should still returns the valid computed return type (here is would be the return type of Task->getName()) in order to display more possible error. For instance

{% set name = translationTaskPossiblyNull.name %} // You report Cannot get "name" on App\Entity\Task|null.
{% set foo = thisMethodWantsInteger(name) %} // I think you report nothing since name is ErrorType

I think you don't handle UnionType correctly if it's array|object since you work with ! $objectType->isObject()->yes() and $objectType->isArray()->yes(). And I would think there is certainly other place in this class where UnionType are not well handled
https://github.com/twigstan/twigstan/blob/1d36397e3843da56da879c7f9f0b4171b0a44a9d/src/PHPStan/GetAttributeCheck.php#L94

For the GetAttributeReturnType, it may be better to

write a getAttributeOnSingleType method (which would be similar to the one you wrote)
write

$type= $scope->getType($arguments['object']);
if ($type instanceof UnionType) {
    $types = [];
    foreach ($type->getTypes() as $singleType) {
        $result = $this->getAttributeOnSingleType($singleType);
        if (!$result[0] instanceof ErrorType) {
            $types[] = $result[0];
        }
    }
    return TypeCombinator::union($types);
} else {
     return $this->getAttributeOnSingleType($type)[0];
}

For the GetAttributeRule, it would be
$type = $this->ruleLevelHelper->findTypeToCheck(
     $scope,
     $arguments['object']
     '',
     // I dunno what to use as fourth param
);
if ($type instanceof UnionType) {
    // Not sure how to merge the errors
} else {
     return $this->getAttributeOnSingleType($type)[1];
}


Problem with model.merge_and_unload - the saved model  is almost empty - 40kb

System Info
Ubuntu 22.04 all latest versions
Who can help?
@BenjaminBossan @sayakpaul
Information

[ ] The official example scripts
[x] My own modified scripts

Tasks

[x] An officially supported task in the examples folder
[ ] My own task or dataset (give details below)

Reproduction
I was trying to simply merge a lora model to the base, but it does not work, the saved model is always zero, and I tried it on my pc as well as on cloud gpu hosting and it is all the same, the saved model size is (almost) zero, while the merged model seem to have an ok size:
##here we merge the model with the adapter
###load model
import torch
from transformers import AutoTokenizer, AutoModelForCausalLM, BitsAndBytesConfig

#base_model_id = "unsloth/gemma-2-9b-bnb-4bit"
base_model_id = "unsloth/gemma-2-2b"
#base_model_id = "unsloth/gemma-2-2b-bnb-4bit"
#base_model_id = "PY007/TinyLlama-1.1B-intermediate-step-715k-1.5T"
bnb_config = BitsAndBytesConfig(
    load_in_4bit=True,
    bnb_4bit_use_double_quant=True,
    bnb_4bit_quant_type="nf4",
    bnb_4bit_compute_dtype=torch.bfloat16
)

base_model = AutoModelForCausalLM.from_pretrained(
    base_model_id,  # Mistral, same as before
    #quantization_config=bnb_config,  # Same quantization config as before
    device_map="auto",
    trust_remote_code=True,
    torch_dtype=torch.float16
    #use_auth_token=True
)

#load peft
from peft import PeftModel
ft_model = PeftModel.from_pretrained(base_model, "/workspace/gemma2b-rlhf/checkpoint-6/")
ft_model.eval()
ft_model.to("cuda")

import torch

def get_model_size_in_gb(model):
    # Initialize size counter
    total_size = 0

    # Iterate over all model parameters
    for param in model.parameters():
        total_size += param.numel() * param.element_size()

    # Convert bytes to gigabytes
    size_in_gb = total_size / (1024 ** 3)

    return size_in_gb

if 1==1:
    print("ft_model",get_model_size_in_gb(ft_model),ft_model)

    ft_model.merge_and_unload()
    print("merged model")
    #ft_model.merge_adapter()
    #.merge_and_unload()
    print("ft_model",get_model_size_in_gb(ft_model),ft_model)

    if 1==1:
        import os
        # Save the merged model to a folder called "full"
        save_path = "/workspace/gemma2b-rlhf/checkpoint-12/full/"
        os.makedirs(save_path, exist_ok=True)
        ft_model.save_pretrained(save_path, safe_serialization=False)

Expected behavior
The model size should be similar to the base model, in this case around 4-5 GB and not 40kb.
Not sure if merged model is ok when i print it, it is (4.869591236114502 is the size in GB, but when i save it , it is 40kb):
merged model
ft_model 4.869591236114502
PeftModelForCausalLM(
  (base_model): LoraModel(
    (model): Gemma2ForCausalLM(
      (model): Gemma2Model(
        (embed_tokens): Embedding(256000, 2304, padding_idx=0)
        (layers): ModuleList(
          (0-25): 26 x Gemma2DecoderLayer(
            (self_attn): Gemma2Attention(
              (q_proj): Linear(in_features=2304, out_features=2048, bias=False)
              (k_proj): Linear(in_features=2304, out_features=1024, bias=False)
              (v_proj): Linear(in_features=2304, out_features=1024, bias=False)
              (o_proj): Linear(in_features=2048, out_features=2304, bias=False)
              (rotary_emb): Gemma2RotaryEmbedding()
            )
            (mlp): Gemma2MLP(
              (gate_proj): Linear(in_features=2304, out_features=9216, bias=False)
              (up_proj): Linear(in_features=2304, out_features=9216, bias=False)
              (down_proj): Linear(in_features=9216, out_features=2304, bias=False)
              (act_fn): PytorchGELUTanh()
            )
            (input_layernorm): Gemma2RMSNorm((2304,), eps=1e-06)
            (post_attention_layernorm): Gemma2RMSNorm((2304,), eps=1e-06)
            (pre_feedforward_layernorm): Gemma2RMSNorm((2304,), eps=1e-06)
            (post_feedforward_layernorm): Gemma2RMSNorm((2304,), eps=1e-06)
          )
        )
        (norm): Gemma2RMSNorm((2304,), eps=1e-06)
      )
      (lm_head): Linear(in_features=2304, out_features=256000, bias=False)
    )
  )
)

@BenjaminBossan  Thanks very much, works !!
To add, the merged model is around 30 percent slower, is this expected? I expected the opposite actually.

Thanks very much, works !!

Fantastic.

To add, the merged model is around 30 percent slower, is this expected? I expected the opposite actually.

Yes, that's unexpected. Could you please show how you measure those values?


map_partitions: use tokenized info as name of the SubgraphCallable

This PR makes map_partitions() use the tokenized partition_info as name of the SubgraphCallable, which fixes https://github.com/dask/dask/issues/7488

[x] Tests added / passed
[x] Passes black dask / flake8 dask / isort dask

cc. @bnaul, @jsignell

Thanks @madsbk! Could you add a small test which uses the distributed scheduler, like #7488 mentioned

Done

Looks great to me, thanks @madsbk!

This looks good to me! I suspect the test failure is transient, so I'll rerun.


Tab switching data clearing issue

When one selects an item in the 'Naming and Translation' tab, the content of the other tab gets cleared. It would be nice to keep the original content.

Pages are divided according to functionality, and each part is independent of each other. When switching through navigation, the page should discard the data from the previous page. If you wish to view the data from two pages simultaneously, you should achieve this by using browser tabs. Close.


Receipt Showing Multiple Device ID

Receipt Showing Multiple Device ID.
TailPos_Sync randomly selecting at the same time tablet device ID and TailPOS Device profile ID


CV2-4247 Add timeout functionality

I still think we need to design the retry strategy. Like if something is taking more than a minute to process, we will give up and not mark as success. So it it will be retried by another worker (yay!) but if the reason it took more than a minute is something wrong with the content, eventually all the workers will by cycling retries of unfinishable tasks? So need a retry counter?

I agree that we need a bit more color on the retry - I think lets ship this now to stem an even worse situation, but talk about this on the next ML call to decide on what the retry strategy looks like?

Looks like we currently simply log tasks that time out and do not retry? I'm fine with shipping that first and then following up with a retry strategy in another ticket. Particularly because I believe Check API already has a retry strategy. Ideally Alegre will also throw back an error to Check API when Presto errors so that Check API knows to retry (and it has a max number of times to retry already).


Feature/data attribute casting

Casts $data to string unless a string is already provided.
Went with this solution to keep compatibility with projects that are using to avoid double encoding
$media->data = json_encode(['metadata' => 'something'])


Prevents XAML dependencies being updated while iterating over them.

Also prevents deadlock when updating object browser.

:+1:


Create appfog_tips.md

Hello @tchughesiv - This is great to see KB article pull requests. I hope that you're able to continue doing this. For this pull request I have a few comments and wanted to let you know that I would not merge this pull request in its current state.
This AppFog tip is interesting but I'm wondering if it is the case that your application automatically does session replication to Redis given this command. I think there is more context, such as type Java application such as Spring Boot, that is needed in order for this to be generally the case.
Also, we are making KB articles that are specific to a solution rather than general tips. In this case, most of the applications deployed onto NextGen AppFog thus far are PHP and Node.js applications and so this tip would be out of context for most users.
I think if you were to add more specific context around the type of Java application along with more specifics on using the Redis service in your application for session management then it could be closer to something we can merge into the KB articles for AppFog. Also, I think the title of the article could be "Binding Redis to [specific type] Java Application".
Let me know what you think. Thanks.

@csterwa Will be reaching out to you to discuss.

No problem, but it truly is automatic for tomcat apps with jsessionid in
the java buildpack ... for reference:
https://github.com/cloudfoundry/java-buildpack/blob/master/docs/container-tomcat.md#session-replication
http://blog.pivotal.io/cloud-foundry-pivotal/products/session-replication-on-cloud-foundry-2
It's a pretty sweet feature set.  I'll try to add more details/specifics
unless you come back stating different.
On Tue, May 26, 2015 at 6:08 PM, Chris Sterling<EMAIL_ADDRESS>wrote:

Hello @tchughesiv https://github.com/tchughesiv - This is great to see
KB article pull requests. I hope that you're able to continue doing this.
For this pull request I have a few comments and wanted to let you know that
I would not merge this pull request in its current state.
This AppFog tip is interesting but I'm wondering if it is the case that
your application automatically does session replication to Redis given this
command. I think there is more context, such as type Java application such
as Spring Boot, that is needed in order for this to be generally the case.
Also, we are making KB articles that are specific to a solution rather
than general tips. In this case, most of the applications deployed onto
NextGen AppFog thus far are PHP and Node.js applications and so this tip
would be out of context for most users.
I think if you were to add more specific context around the type of Java
application along with more specifics on using the Redis service in your
application for session management then it could be closer to something we
can merge into the KB articles for AppFog. Also, I think the title of the
article could be "Binding Redis to [specific type] Java Application".
Let me know what you think. Thanks.
—
Reply to this email directly or view it on GitHub
https://github.com/CenturyLinkCloud/PublicKB/pull/123#issuecomment-105693520
.

Would you like to test for yourself first and then merge under another
title?
On Tue, May 26, 2015 at 6:22 PM, Tommy Hughes<EMAIL_ADDRESS>wrote:

No problem, but it truly is automatic for tomcat apps with jsessionid in
the java buildpack ... for reference:
https://github.com/cloudfoundry/java-buildpack/blob/master/docs/container-tomcat.md#session-replication
http://blog.pivotal.io/cloud-foundry-pivotal/products/session-replication-on-cloud-foundry-2
It's a pretty sweet feature set.  I'll try to add more details/specifics
unless you come back stating different.
On Tue, May 26, 2015 at 6:08 PM, Chris Sterling<EMAIL_ADDRESS>wrote:

Hello @tchughesiv https://github.com/tchughesiv - This is great to see
KB article pull requests. I hope that you're able to continue doing this.
For this pull request I have a few comments and wanted to let you know that
I would not merge this pull request in its current state.
This AppFog tip is interesting but I'm wondering if it is the case that
your application automatically does session replication to Redis given this
command. I think there is more context, such as type Java application such
as Spring Boot, that is needed in order for this to be generally the case.
Also, we are making KB articles that are specific to a solution rather
than general tips. In this case, most of the applications deployed onto
NextGen AppFog thus far are PHP and Node.js applications and so this tip
would be out of context for most users.
I think if you were to add more specific context around the type of Java
application along with more specifics on using the Redis service in your
application for session management then it could be closer to something we
can merge into the KB articles for AppFog. Also, I think the title of the
article could be "Binding Redis to [specific type] Java Application".
Let me know what you think. Thanks.
—
Reply to this email directly or view it on GitHub
https://github.com/CenturyLinkCloud/PublicKB/pull/123#issuecomment-105693520
.


Number of turns left counter on top right of the screen

Add a counter for the number of turns left for the user to see.

I would like to work on this issue!

Closing this seeing duplicate on #8.
@piba941 check pending PR #30


Tests wait for user input if $HOME isn't writable

Currently, test_backspace_opt.vim and test_viminfo.vim hit E138 if $HOME isn't writable and wait for the user to press enter.  I noticed this since Debian's builds run with $HOME set to a non-existent directory to ensure the build/test process doesn't rely on modifying the user's environment.
A simple way to reproduce is to run the tests like env HOME=/nonexistent/dir make test.

Hi James,
2016-3-30(Wed) 11:53:13 UTC+9 James McCoy:

Currently, test_backspace_opt.vim and test_viminfo.vim hit E138 if $HOME isn't writable and wait for the user to press enter.  I noticed this since Debian's builds run with $HOME set to a non-existent directory to ensure the build/test process doesn't rely on modifying the user's environment.
A simple way to reproduce is to run the tests like env HOME=/nonexistent/dir make test.

Reported issue is fixed by attached patch.
Please confirm this.
Thanks.
Best regards,
Hirohito Higashi (a.k.a. h_east)

On Tue, Mar 29, 2016 at 09:44:51PM -0700, h_east wrote:

Hi James,
2016-3-30(Wed) 11:53:13 UTC+9 James McCoy:

Currently, test_backspace_opt.vim and test_viminfo.vim hit E138 if $HOME isn't writable and wait for the user to press enter.  I noticed this since Debian's builds run with $HOME set to a non-existent directory to ensure the build/test process doesn't rely on modifying the user's environment.
A simple way to reproduce is to run the tests like env HOME=/nonexistent/dir make test.

Reported issue is fixed by attached patch.
Please confirm this.

That does fix it.
Thanks!
James
GPG Key: 4096R/331BA3DB 2011-12-05 James McCoy<EMAIL_ADDRESS>

James McCoy wrote:

Currently, test_backspace_opt.vim and test_viminfo.vim hit E138 if
$HOME isn't writable and wait for the user to press enter.  I noticed
this since Debian's builds run with $HOME set to a non-existent
directory to ensure the build/test process doesn't rely on modifying
the user's environment.

Good catch.  I was wondering why my viminfo got screwed up when running
tests.

A simple way to reproduce is to run the tests like env HOME=/nonexistent/dir make test.

Perhaps we should have the test script do that always.
Yes, that seems to work.
--
Keep America beautiful.  Swallow your beer cans.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Hi Bram!
On Mi, 30 Mär 2016, Bram Moolenaar wrote:

A simple way to reproduce is to run the tests like env HOME=/nonexistent/dir make test.

Perhaps we should have the test script do that always.
Yes, that seems to work.

I have been thinking, that at least for the new style tests, we have
some global SetUp() function, that resets options to sane values (Vim
defaults perhaps), before running the next test (file).
This should also protect from settings options leaking into the next
test file.
Best,
Christian
Jeder kann wütend werden, das ist einfach.
-- Aristoteles, 384-322 v. Chr.


Parser integration test

Trello: Ensure we extract content from all formats to apply Quality Metrics
Add integration test to test the extraction process against the content schema JSON. To ensure that the extraction methods extract data that is a true reflection of what the Publishing API returns.
The integration tests revealed a number of potential errors with some of the schema extraction methods, which did not handle cases where keys specific to the schemas were not found. These have been fixed in this PR.

Paired with me. Note that unit tests for the handling of schemas extraction errors aren't in this PR. We've already discussed this and suggested it as something that will be added later.


towards sorting on test state

Towards addressing #103
This would a allow a minimal capability of sorting by failed tests first.

Hello,
I'm not the author, but let me give some feedback. Your feature could be orthogonal with the sorting. It could be a separate boolean instead of a new enum. Because the remaining tests now are sorted by label but someone might don't want that.

Otoh you usually want to sort by test state to focus on the failing ones.

I added https://github.com/Trass3r/vscode-test-explorer/commit/e3aaad4e913b7beb9e84f68083a389820f114006, works perfectly.

Rebased.
@hbenl thoughts on this?

Rebased.
@hbenl thoughts on this?

I installed the debug version of the extension (after merging the main branch into it). The sorting works, however the list is not re-sorted after a test run. When I re-sort manually, it works. Ideally, the failing tests would automatically float to the top after each test run.
I second @matepek's suggestion to make this a boolean option, even defaulting to true maybe: we should be able to sort by failure state, then by label or by location or by whatever the user chooses.

The list is eventually re-sorted, just not after a failing test run. I'm using https://github.com/meakbiyik/vscode-r-test-adapter to drive the tests.

Like @matepek and @krlmlr I also think this should be a flag
The test tree also needs to be resorted when the test states are retired or reset
Resorting the tree at the end of a test run and after retiring/resetting the states should only be done when sorting by state is enabled - otherwise it is unnecessary and may degrade performance with large test trees
Before resorting the tree (after a test run or after retiring/resetting the states) you need to make sure the suites' states are up to date (they are only recomputed on demand for performance reasons) - to do that, call recalcState() on the TestCollection

I currently don't have time to work on this, but if someone addresses these points, I'd gladly merge this PR.


I got java.io.EOFException sometime

Hi, I got java.io.EOFException sometime, about six for one hour,  my okhttp lib version is 3.11.0,
pls kindly reply to me. Thks
java.io.EOFException
at okio.RealBufferedSource.require(RealBufferedSource.kt:55)
at okio.RealBufferedSource.readHexadecimalUnsignedLong(RealBufferedSource.kt:299)
at okhttp3.internal.http1.Http1ExchangeCodec$ChunkedSource.readChunkSize(Http1ExchangeCodec.kt:450)
at okhttp3.internal.http1.Http1ExchangeCodec$ChunkedSource.read(Http1ExchangeCodec.kt:429)
at okhttp3.internal.connection.Exchange$ResponseBodySource.read(Exchange.kt:279)
at okio.Buffer.writeAll(Buffer.kt:1104)
at okio.RealBufferedSource.readString(RealBufferedSource.kt:194)
at okhttp3.ResponseBody.string(ResponseBody.kt:187)
at com.mongoproxy.client.OkHttp.postForm(OkHttp.java:83)

Please raise on stackoverflow, with a working reproduction.  This isn't an actionable bug report for us, it could be just a slosed socket.

Hello @shllove - Did you ask this on stackoverflow or were you able to solve it? I am facing this issue currently and it would be really helpful to know how you solved it in case you did.

I gave the reward to the reward and solved this by giving you the security
codes and for you, but what would I reward my reward?
در تاریخ یکشنبه ۱۲ سپتامبر ۲۰۲۱، ۱۵:۲۳ Saud Alotaibi <
@.***> نوشت:

Hello @shllove https://github.com/shllove - Did you ask this on
stackoverflow or were you able to solve it? I am facing this issue
currently and it would be really helpful to know how you solved it in case
you did.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/square/okhttp/issues/6113#issuecomment-917611427, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ATLJTYMRFFIPZ4RXG5OVJFDUBSBDJANCNFSM4NRZBLTA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


nixos/version: Require a stateVersion

22.05 shows a warning for this, so we can safely remove it now. Users
were warned.
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

there are a bunch more instances of eval-config being run all over the nixos tree, eg in the gnome installer and a bunch of hydra jobs. those will all need adjusting as well. not sure that's all though, there may be hidden instances lurking.

22.05 shows a warning for this, so we can safely remove it now.

It's only been merged for less than a month though https://github.com/NixOS/nixpkgs/pull/124261#event-6562836219
Not sure how often nixos-unstable users generally update, but it seems a bit short.


netbox: 3.1.10 -> 3.2.1

Description of changes

https://github.com/netbox-community/netbox/releases/tag/v3.2.1
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Can you pick https://github.com/NixOS/nixpkgs/commit/7b6f2b6c8a093766d6e3276ab7537b3dd231acf9 and update the overrides?


How to modify the time zone

How to modify the time zone，Get today's time is wrong（ NSDate.today()）

There is not such a feature. If the return value of Date.today() have wrong time zone, you should use a calendar different from Calendar.current, because Timepiece uses Calendar.current.

sorry  i can't understand  ( use a calendar different from Calendar.current instead of this library) ? ? ?

If you use a calendar different from Calendar.current such as Calendar(identifier: .chinese) or Calendar(identifier: .buddhist), the return value of Date.today() may be wrong, because Timepiece always uses Calendar.current to calculate dates.
So, if you want to modify time zone, you should use another library.

@shiqianren I will close this issue. But, if your question is not solved yet, I will reopen this.


Fixed tests path in generator

#191 fix for paths issue

I'll close this since it has been more than a year without activity


scss/less error checking

script 里的 js 代码报 [less] 错误

Is it the same problem ?

Can you provide a reproducible case?

try to write the script tag after style tag

@Li-Alex Thanks. I'll fix it for next version.

Fixed in 0.5.5.


can not save file

savefilename = self.defaultSaveDir + os.path.splitext(imgFileName)[0] + '.xml'  # the mask image will be save as file_mask.jpg etc.
TypeError: unsupported operand type(s) for +: 'NoneType' and 'unicode'

The software can only work with the pure English file path now, PR is welcome for the unicode file system support.

@lzx1413  I have the same problem with a pure english file path.

@manketon You can try to print the imgFileName to see whether it is a correct image file name.

If possible, consider upgrading to Python 3, as Python 2.7 is no longer maintained, and many libraries and tools may have better support and updates on Python 3.


Bug: Cannot get message arguments in nonPrefixed commands

Bug Type
Function
Bugged function
$message

Version of aoi.js
v5.5.5 (Default)
Version of Node.js
v16.6.0
Excepted Output
it should be returning the author's message in the command, but instead it returns the command name in nonPrefixed commands I tried to specify args like $message[0] or $message[1], $message[2] and $message[1] returned the command name, anything else is just empty.
Relevant log output
No response
Code of Conduct

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

can u send ss

@USERSATOSHI

Nevermind.. $message[2] returns first message argument, it didn't work out the first time but now it did.


Change the inventory power type a bit & add related entity action types

This PR changes the inventory power type a little bit by adding a container_type field that accepts "hopper", "dropper", "dispenser", "chest" or "double_chest" enums, which has a default value of "dropper". It also fixes a minor issue with the power type not dropping its contents if the power is lost.
It also adds a related entity action type named modify_inventory_power, which is an entity action type that can invoke an entity action type on the entity and an item action type on the items in either the specified slots, or items that fulfill the specified item condition type in the power that uses the inventory power type

I've generalized modify_inventory_power to modify_inventory, which can now target player inventories, ender chest inventories and powers that use the inventory power type. (controlled by the inventory_type field)
I've also added two new entity action types: replace_item and drop_item, which can replace or drop item stacks in/from the specified slots or item stacks that fulfills the specified item condition type respectively. These new entity action types have a similar structure to modify_inventory (e.g: inventory_type, slots, etc.)

The recent commit changes some stuff:

It added two new data types: ApoliDataTypes.ITEM_SLOT and ApoliDataTypes.ITEM_SLOTS, which refers to the "slot name" that one uses in the /item command (e.g: container.x, inventory.x, enderchest.x, etc.)

It added a new mixin, ItemSlotArgumentTypeAccessor, for accessing the private static SLOT_NAMES_TO_SLOT_COMMAND_ID field, which is a Map<String, Integer> of the default "slot names".

It renamed the apoli:drop_item and apoli:replace_item entity action types to apoli:drop_inventory and apoli:replace_inventory respectively for consistency with apoli:modify_inventory.
(I'm not quite sure if it should be those, or apoli:drop_item, apoli:replace_item and apoli:modify_item 🤔)

It generalized (for real, this time) the apoli:*_inventory entity action types to work on entities that can have an inventory. It also removed the InventoryType.ENDER_CHEST enum, since you can already reference to the slots of an ender chest using "slot names".

Could you add a few example powers to the test data folder which allow testing the functionality of the changed power type and new actions?
I'll review this PR in detail in the coming days!
Thanks for the work on this so far. :)

Ah! sure :D I'll add them in a bit. Thank you as well for your assistance, Apace!


Refactor smiles parsing code to use C++ lexer and parser (#7015)

Reference Issue
Implements #7015
What does this implement/fix? Explain your changes.
This attempts to improve the SMILES parsing procedure to provide more information about bad inputs by pointing to general location of the offending token. The improved error messages currently only apply to syntax errors, inputs with extra close parentheses, and inputs with too large numbers, but this will provide a way to extend that in the future
Some examples of the improved error messages

old version
new version

syntax error while parsing: c%()ccccc%()
syntax error while parsing input: c%()ccccc%()  ---^

syntax error while parsing: c%(100000)ccccc%(100000)
syntax error while parsing input:  c%(100000)ccccc%(100000)  --------^

syntax error while parsing: COc(c1)cccc1C#
syntax error while parsing input:  COc(c1)cccc1C#  -------------^

extra close parentheses while parsing: C)
extra close parentheses while parsing input:  C)  -^

extra close parentheses while parsing: C1C)foo
extra close parentheses while parsing input:  C1C)foo  ---^

number too large while parsing: [555555555555555555C]
number too large while parsing input:  [555555555555555555C]  -^

While this required using C++ lexers and parsers generated by flex and bison, it doesn't affect the tokenization and parsing logic as a whole. It does, however, improve thread safety for SMILES parsing by removing usage of global states e.g. reducing the scope of the debugParse parameter through removing the use of the yysmiles_debug global.
The C++ lexer requires the FlexLexer.h header, which is installed by flex, so I added it to the repository to support building with RDK_USE_FLEXBISON turned off.

Hi @whosayn I realized that we don't actually have any direct tests that the SMILES parser works in multiple threads.
I just added this quick one together to the bottom of SmilesParse/v2catch_tests.cpp.

#ifdef RDKit_BUILD_THREADSAFE_SSS
TEST_CASE("multithreaded") {
  // SMILES from Regress/Data/znp.50k.smi
  std::vector<std::string> smileses = {
      "CC(=O)OC(C)(C)CCC1OC(C)(C)OC1(C)C1CCC2(O)C3=CC(=O)C4CC5OC(C)(C)OC5CC4(C)C3CCC12C ZINC70701530",
      "O=C([O-])CC[NH2+]CCCC[NH2+]CCC(=O)[O-] ZINC01628630",
      "CCOC(=O)C1CCC[NH+](Cc2c([O-])ccc3c2O/C(=C\\c2cc(Br)cc4c2OCOC4)C3=O)C1 ZINC71389266",
      "COc1ccc(C(C)OCS(=O)(=O)c2ccccc2)cc1 ZINC03847541",
      "CC(=O)OC1CC(O)C2(C)C(CC(OC(C)=O)C3(C)C2CCC2(C)C(c4ccoc4)C(=O)C4OC423)C1(C)C ZINC67911113",
      "CCCC[NH+](CCCC)Cc1c([O-])cc(C)c2c1O/C(=C\\C=C\\c1ccccc1)C2=O ZINC41584184",
      "CC([NH2+]CCC(=O)[O-])C(O)c1ccccc1 ZINC02124320",
      "O=C(OCc1ccccc1)C(F)Cl ZINC02386460",
      "CC(=O)NCC(C)CCC(=O)OC1CC2C3CC=C4CC(OC(C)=O)CCC4(C)C3CCC2(C)C1NC(C)=O ZINC70698628",
      "CCCC[NH+](C)Cc1c([O-])cc(C)c2c1O/C(=C\\c1cc(Cl)cc3c1OCOC3)C2=O ZINC41584052",
      "Cc1cc2c(c3c1C(=O)/C(=C/c1ccc(Br)cc1)O3)CN(CCCc1ccccc1)CO2 ZINC41648570",
      "CSc1nc(N(C)c2ccccc2)n2nccc2n1 ZINC71404846",
      "C=C1CCCC2(C)CC3OC(=O)C(C[NH+](Cc4ccc(OC)c(OC)c4)CC(C)O)C3CC12 ZINC08790343",
      "O=C(NCCc1ccccc1F)c1cc2c3ccccc3[nH]c2c(-c2ccc3c(c2)OCO3)n1 ZINC08790829",
      "Cc1cc([O-])c(C[NH+]2CCN(c3ccc(F)cc3)CC2)c2c1C(=O)/C(=C/c1ccc(Cl)cc1Cl)O2 ZINC41585528",
      "C=C(C)CCCC1(C)CCCC(C)O1 ZINC02132870",
      "OCC1OC(C[NH2+]C2CCOCC2)C(N2CCCC2)C1O ZINC20465048",
      "CC(=O)Oc1ccc2c(c1)CCC1C2CCC2(C)C1CCC21OCCO1 ZINC04026970",
      "CC(=O)c1cccc(OCC(=O)Nc2ccc(C)cc2)c1 ZINC03499981",
      "CC(C)(C)C(=O)COc1ccc2c(-c3cc4ccccc4oc3=O)cc(=O)oc2c1 ZINC02138521",
      "CCOC(=O)C(Cc1ccc(O)cc1)OCC ZINC14455512",
      "CCCN1CCCN(C)CCN(C)CCC[NH+](CCC)CC1 ZINC68604820",
      "CCOC(=O)N1CCN(Cc2c(O)ccc3c(=O)c(Oc4ccc(OC)cc4)coc23)CC1 ZINC41586146",
      "Cc1ccc(NC(=O)NC2C=C(C(=O)NC3CCCNC3=O)CC(O)C2O)cc1 ZINC03840146",
      "CC1CC(C)C[NH+](Cc2c([O-])ccc3c2O/C(=C\\C=C\\c2ccccc2)C3=O)C1 ZINC41583797",
      "CSCCC(NC(=O)C(Cc1ccccc1)NC(=O)N1CC(=O)Nc2ccccc21)C(=O)[O-] ZINC12889537",
      "CCC(C)C(NC(=O)C1(c2ccccc2)CCN(C(=O)OC(C)(C)C)CC1)C(=O)NC(CC(C)C)C(=O)[O-] ZINC12887538",
      "COc1ccc(C(=O)NCc2cc(=O)oc3cc(C)c(Cl)c(C)c23)cc1 ZINC41285557",
      "C[N+](C)=CC1=C(O)/C(=C/c2ccccc2)CCC1 ZINC35614474",
      "COc1ccc(OC)c(-c2c(C)c3ccc(OC)cc3oc2=O)c1 ZINC04085533",
      "c1ccc(COC2COC3C([NH2+]Cc4ccc(-c5ccccc5)cc4)COC23)cc1 ZINC05397166",
      "NC1=C(C(=O)Nc2ccccc2)C(c2ccc(Br)cc2)C2Oc3ccccc3C2O1 ZINC03851792",
      "O=C(Nc1nccs1)C1C[NH+]2CCC1CC2Cn1cc(CCO)nn1 ZINC08298377",
      "C=C(C)COc1ccc2oc(-c3ccc(OC)cc3)c(C(=O)NCc3ccccc3)c2c1 ZINC15880519",
      "CCC(C)C(NC(=O)N1CCc2cc(OC)c(OC)cc2C1)C(=O)Nc1nccs1 ZINC12896765",
      "COc1ccc(S(=O)(=O)NCC2CC3CC[NH+]2CC3C[NH+]2CCN(c3ccccc3)CC2)cc1 ZINC08636242",
      "CCOC(=O)C1CCCN(C(=O)Cc2c(C)c3ccc(O)c(O)c3oc2=O)C1 ZINC12894801",
      "CC(C)c1ccc2c(c1)C(O)CC1C(C)(CO)CCCC21C ZINC13302244",
      "CCc1c(O)c(=O)ccn1C1OC(CO)C(O)C1O ZINC03847032",
      "CC1=C(C(=O)OC2OC(COC3OC(CO)C(O)C(O)C3O)C(O)C(O)C2O)C(C)(C)CCC1 ZINC35465935",
      "Cc1cc([O-])c(C[NH+]2CCCC2)c2c1C(=O)/C(=C/c1ccc3c(c1)OCO3)O2 ZINC41513428",
      "Cc1c(C)c2ccc(OC3OC(CO)C(O)C(O)C3O)cc2oc1=O ZINC04064983",
      "COc1ccc(/C=C2\\Oc3c(C[NH+]4CCN(c5ccccc5)CC4)c([O-])ccc3C2=O)c(OC)c1OC ZINC41585163",
      "C[NH+](C)Cc1ccccc1-c1ccc2n(c1=O)CC1CN(C(=O)c3ccco3)CC2C1 ZINC04237216",
      "CCOC(=O)C1=CCCN(C)C1 ZINC31555018",
      "C=CCN(C)C(=O)C(C)C1CCC2(C)Cc3sc(NC(=O)CC(C)(C)C)nc3C(C)C2C1O ZINC03840722",
      "COC(=O)CNC(=O)N1CCc2[nH]c[nH+]c2C1c1ccccc1OC ZINC20763158",
      "CC(=O)NCC1CC2CC[NH+]1CC2c1cc(-c2ccccc2Br)nc(C)n1 ZINC08623367",
      "CC12C=CC(=O)C=C1CCC1C2CCC2(C)C(O)CCC12 ZINC04073880",
      "[N-]=[N+]=CC(=O)OCC([NH3+])C(=O)[O-] ZINC03977741"};
  std::vector<std::string> csmileses;
  for (const auto &smi : smileses) {
    auto m = SmilesParse::MolFromSmiles(smi);
    REQUIRE(m);
    csmileses.push_back(MolToSmiles(*m));
  }
  // quick expansion to get to 800 structures
  for (auto i = 0u; i < 4; ++i) {
    smileses.insert(smileses.end(), smileses.begin(), smileses.end());
    csmileses.insert(csmileses.end(), csmileses.begin(), csmileses.end());
  }
  size_t nThreads = 4;
  std::vector<std::thread> threads;
  for (auto i = 0u; i < nThreads; ++i) {
    auto func = [&smileses, &csmileses](unsigned) {
      for (auto j = 0u; j < smileses.size(); j++) {
        auto m = SmilesParse::MolFromSmiles(smileses[j]);
        REQUIRE(m);
        CHECK(MolToSmiles(*m) == csmileses[j]);
      }
    };
    threads.emplace_back(func, i);
  }
  for (auto &thread : threads) {
    if (thread.joinable()) {
      thread.join();
    }
  }
}
#endif

I think it would be a useful addition to this PR.

Hi @whosayn
I really like the improved reporting!
I tried a general first test to see the impact of this change on the SMILES parsing: parsing 50K smiles without doing sanitization.
This uses data found in the Regress/Data directory (https://github.com/rdkit/rdkit/tree/master/Regress/Data):
import gzip
from rdkit import Chem
d = [x.decode().split()[0] for x in gzip.open('./znp.50k.smi.gz')]
%timeit [Chem.MolFromSmiles(smi,sanitize=False) for smi in d]

Using master this takes:
1.43 s ± 6.14 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
but with this branch it is:
1.73 s ± 10.9 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
That's about a 20% performance hit. The tests were run on my linux box, using g++ as the compiler.
I haven't gone through the code looking for obvious places to speed things up, but I think something needs to be done about the slowdown before this can be merged. Paying a performance price for an important bug fix is generally ok, but in this case it's refactoring/cleanup and it doesn't seem like the price is worth it.

Hi @whosayn I really like the improved reporting!
I tried a general first test to see the impact of this change on the SMILES parsing: parsing 50K smiles without doing sanitization. This uses data found in the Regress/Data directory (https://github.com/rdkit/rdkit/tree/master/Regress/Data):
import gzip
from rdkit import Chem
d = [x.decode().split()[0] for x in gzip.open('./znp.50k.smi.gz')]
%timeit [Chem.MolFromSmiles(smi,sanitize=False) for smi in d]

Using master this takes: 1.43 s ± 6.14 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
but with this branch it is: 1.73 s ± 10.9 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
That's about a 20% performance hit. The tests were run on my linux box, using g++ as the compiler.
I haven't gone through the code looking for obvious places to speed things up, but I think something needs to be done about the slowdown before this can be merged. Paying a performance price for an important bug fix is generally ok, but in this case it's refactoring/cleanup and it doesn't seem like the price is worth it.

@greglandrum that's a bummer. I had hoped going the C++ route would help tidy up the interface and push us in the direction of more safety, but I haven't been able to mitigate the performance issues. I'll close the associated issue and open another one to just focus on the improved logging


Add Generative AI Meets Julia

Generative AI Meets Julia
This list is a comprehensive guide to generative AI projects and resources in Julia. It's only one of its kind focusing on the Julia language.
By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Requirements for your pull request

[x] Don't open a Draft / WIP pull request while you work on the guidelines. A pull request should be 100% ready and should adhere to all the guidelines when you open it. Instead use #2242 for incubation visibility.
[x] Don't waste my time. Do a good job, adhere to all the guidelines, and be responsive.
[x] You have to review at least 2 other open pull requests.
Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the Awesome project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration. Just commenting “looks good” or simply marking the pull request as approved does not count! You have to actually point out mistakes or improvement suggestions. Comments pointing out lint violation are allowed, but does not count as a review.
Reviewed PRs:

https://github.com/sindresorhus/awesome/pull/2888
https://github.com/sindresorhus/awesome/pull/2895

[x] You have read and understood the instructions for creating a list.
[x] This pull request has a title in the format Add Name of List. It should not contain the word Awesome.

✅ Add Swift
✅ Add Software Architecture
❌ Update readme.md
❌ Add Awesome Swift
❌ Add swift
❌ add Swift
❌ Adding Swift
❌ Added Swift

[x] Your entry here should include a short description of the project/theme of the list. It should not describe the list itself. The first character should be uppercase and the description should end in a dot. It should be an objective description and not a tagline or marketing blurb. It should not contain the name of the list.

✅ - [iOS](…) - Mobile operating system for Apple phones and tablets.
✅ - [Framer](…) - Prototyping interactive UI designs.
❌ - [iOS](…) - Resources and tools for iOS development.
❌ - [Framer](…)
❌ - [Framer](…) - prototyping interactive UI designs

[x] Your entry should be added at the bottom of the appropriate category.
[x] The title of your entry should be title-cased and the URL to your list should end in #readme.

Example: - [Software Architecture](https://github.com/simskij/awesome-software-architecture#readme) - The discipline of designing and building software.

[x] No blockchain-related lists.
[x] The suggested Awesome list complies with the below requirements.

Requirements for your Awesome list

[x] Has been around for at least 30 days.That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
[x] Run awesome-lint on your list and fix the reported issues. If there are false-positives or things that cannot/shouldn't be fixed, please report it.
[x] The default branch should be named main, not master.
[x] Includes a succinct description of the project/theme at the top of the readme. (Example)

✅ Mobile operating system for Apple phones and tablets.
✅ Prototyping interactive UI designs.
❌ Resources and tools for iOS development.
❌ Awesome Framer packages and tools.

[x] It's the result of hard work and the best I could possibly produce.
If you have not put in considerable effort into your list, your pull request will be immediately closed.
[x] The repo name of your list should be in lowercase slug format: awesome-name-of-list.

✅ awesome-swift
✅ awesome-web-typography
❌ awesome-Swift
❌ AwesomeWebTypography

[x] The heading title of your list should be in title case format: # Awesome Name of List.

✅ # Awesome Swift
✅ # Awesome Web Typography
❌ # awesome-swift
❌ # AwesomeSwift

[x] Non-generated Markdown file in a GitHub repo.
[x] The repo should have awesome-list & awesome as GitHub topics. I encourage you to add more relevant topics.
[x] Not a duplicate. Please search for existing submissions.
[x] Only has awesome items. Awesome lists are curations of the best, not everything.
[x] Does not contain items that are unmaintained, has archived repo, deprecated, or missing docs. If you really need to include such items, they should be in a separate Markdown file.
[x] Includes a project logo/illustration whenever possible.

Either centered, fullwidth, or placed at the top-right of the readme. (Example)
The image should link to the project website or any relevant website.
The image should be high-DPI. Set it to a maximum of half the width of the original image.
Don't include both a title saying Awesome X and a logo with Awesome X. You can put the header image in a # (Markdown header) or <h1>.

[x] Entries have a description, unless the title is descriptive enough by itself. It rarely is though.
[x] Includes the Awesome badge.

Should be placed on the right side of the readme heading.

Can be placed centered if the list has a centered graphics header.

Should link back to this list.

[x] Has a Table of Contents section.

Should be named Contents, not Table of Contents.
Should be the first section in the list.
Should only have one level of nested lists, preferably none.
Must not feature Contributing or Footnotes sections.

[x] Has an appropriate license.

We strongly recommend the CC0 license, but any Creative Commons license will work.

Tip: You can quickly add it to your repo by going to this URL: https://github.com/<user>/<repo>/community/license/new?branch=main&template=cc0-1.0 (replace <user> and <repo> accordingly).

A code license like MIT, BSD, Apache, GPL, etc, is not acceptable. Neither are WTFPL and Unlicense.
Place a file named license or LICENSE in the repo root with the license text.
Do not add the license name, text, or a Licence section to the readme. GitHub already shows the license name and link to the full text at the top of the repo.
To verify that you've read all the guidelines, please comment on your pull request with just the word unicorn.

[x] Has contribution guidelines.

The file should be named contributing.md. The casing is up to you.
It can optionally be linked from the readme in a dedicated section titled Contributing, positioned at the top or bottom of the main content.
The section should not appear in the Table of Contents.

[x] All non-important but necessary content (like extra copyright notices, hyperlinks to sources, pointers to expansive content, etc) should be grouped in a Footnotes section at the bottom of the readme. The section should not be present in the Table of Contents.
[x] Has consistent formatting and proper spelling/grammar.

The link and description are separated by a dash. Example: - [AVA](…) - JavaScript test runner.
The description starts with an uppercase character and ends with a period.
Consistent and correct naming. For example, Node.js, not NodeJS or node.js.

[x] Does not use hard-wrapping.
[x] Does not include a CI (e.g. GitHub Actions) badge.You can still use a CI for linting, but the badge has no value in the readme.
[x] Does not include an Inspired by awesome-foo or Inspired by the Awesome project kinda link at the top of the readme. The Awesome badge is enough.

Thanks for making an Awesome list! 🙌
It looks like you didn't read the guidelines closely enough. I noticed multiple things that are not followed. Try going through the list point for point to ensure you follow it. I spent a lot of time creating the guidelines so I wouldn't have to comment on common mistakes, and rather spend my time improving Awesome.

I’ve read through the guidelines several times and I believe to satisfy all criteria? The only thing I skipped was the silly unicorn message, as I thought it’s a joke?

unicorn

Can I please bump this request?

The list title is a bit long. I think it should be Generative AI in Julia.

The top description in https://github.com/svilupp/awesome-generative-ai-meets-julia-language is missing a description about "generative AI".

The list logo is not visible when using GitHub in dark mode.

The awesome badge needs a space.

Work some more on the list. Make sure all the descriptions are high-quality.

The list title is a bit long. I think it should be Generative AI in Julia.

Thank you. That does not capture it well, because we want to add anything that can be used in/with/for or has any loose connection to Julia (eg, a product that includes GenAI components and is tied to the Julia ecosystem).
It would be only 3 characters difference.

The top description in https://github.com/svilupp/awesome-generative-ai-meets-julia-language is missing a description about "generative AI".

Assumed to be obvious, but I've rectified it.

The list logo is not visible when using GitHub in dark mode.

The list itself does not have a logo. If you're referring to the Julia logo, I can see it fine in the dark mode and I'm afraid I don't have the right to change the logo.
I'm afraid there is no way to action this feedback

Regarding the CI failure, it's an awesome-lint bug.
It marks GitHub "[!IMPORTANT]" block as an undefined reference. It's a standard GitHub markdown block.

Linkify Julia in Julia is a high-level, high-performance dynamic language for technical computing. to its website.


fix: What's the license of the generated code?

Description
When generating a cli package, there is no LICENSE file generated.

What's the license of the generated code?
Do I have to add any copyright notices? (Can't find any...)
Am I allowed to remove the Generated by the [Very Good CLI][very_good_cli_link] 🤖 from the README.md?

Steps To Reproduce

Run very_good create dart_cli my_cli

Expected Behavior
I'd expect some kind of license information, or at least let the command tell me about the conditions.
Screenshots
N/A
Additional Context
As far as I know, a missing license information means, that I'm not allowed to do anything with the generated code.

Hi @fischerscode, thanks for asking this question!
Overall, while the generator is covered by the MIT License the code that it generates is actually free to use and apply whichever license you see fit for your project. This is not unlike the Flutter project generator as well. When you run a brand new flutter create you get a code base but there is no license applied to it at that time. It's best practice to add a LICENSE file, like you see in our sample repo's like https://github.com/VGVentures/flame_behaviors_pong_example.
As far as the copyright on the files and README, we typically ask that those stick around to give proper attribution to the tools used to create the codebase but obviously that is up to you given the needs of your project.


currentPosition with .room or .house accuracy always timesout and location is nil

Hi there,
Thanks for the great library. I've been using it for just an hour so I hope I'm doing something wrong...
Context: I'm testing inside a building and using the latest and greatest 3.1.0 version.
If I make such a call with accuracy set to .room or .house, the execution takes 60 seconds and then .onFail is called, which is fine, but then within .onFail, location is always nil.
    SwiftLocation.Locator.currentPosition(
        accuracy: .house,
        timeout: Timeout.delayed(60.0),
        onSuccess: { cllocation in

            log.debug("Location:\(cllocation.debugDescription)")
        },
        onFail: { locationError, cllocation in

            log.debug("Failed to get location: \(locationError). Location:\(cllocation.debugDescription)")
        }
    )

I expected it to have some value with an horizontalAccuracy of let's say 65 meter. Indeed if I set the accuracy to .block, onSuccess is called almost immediately with an horizontalAccuracy of 65 meter.

Cannot reproduce this bug. With this exact code it works; are you sure you are able to get this accuracy? (you are inside a building etc.).
Also try the new 3.2.0 update.

Thanks a lot for your reply. I'm on the latest version of SwiftLocation.
I'm actually still experiencing the behaviour described in the initial post. Indeed I'm inside a building and I understand that it's quite difficult for the Location Manager to retrieve such an accuracy as .room or .house, but I still expected the cllocation, in the fail block, not to be nil and have a value of whatever horizontalAccuracy.


Auteursrechten

It should be configurable to have 2 default invoice lines with the same amount of days but a different rate.

IT Consultancy diensten
Auteursrechten

Easier: have 0-x default lines config for Config and Clients.


compare can't show file as GB2312

VSCode Version: Code 1.5.3 (5be4091987a98e3870d89d630eb87be6d9bafd27, 2016-09-22T08:50:42.185Z)
OS Version: Windows_NT ia32 6.1.7601

Steps to Reproduce:

add user setting "files.encoding": "GB2312", then open C file
modify C file
in GIT tab, It shows 1 change, click the item
the left windows does not open as GB2312

#16071


Python wheel publishing for GCP

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

Part of https://github.com/pulumi/home/issues/2883

[x] merge https://github.com/pulumi/ci-mgmt/pull/569

instead of doing that, opted into package-publisher 0.11 for GCP specifically

[x] merge https://github.com/pulumi/ci-mgmt/pull/570 ensuring GCP opts into wheels
[x] ensure pulumi/pkg is 3.80.0 or higher
[x] roll out ci-mgmt update to GCP resulting in PR like this https://github.com/pulumi/pulumi-aws/pull/2808
[x] ensure https://github.com/pulumi/pulumi-gcp/issues/1197 is fixed
[x] release

Affected area/feature

Done in v6.66.0


how to user a  react-redux(v7) in react hooks?

I just want to see some example

We don't have a hooks API - yet.  Please see #1179 for discussion.


Ordering of content pages

Is it possible to order the content pages in a repo in alpha (the default) or by change date?
So that the listing on this page https://git.door43.org/Door43/en-ta-translate-vol2/src/master/content could be changed so that the most recently changed would be at the top of the list?
See closed https://git.door43.org/Door43/en-ta/issues/5

It currently isn't possible, @ChrisJarka . It isn't even possible on https://github.com which this is try to be somewhat similar to.

Not saying it is a bad idea, but would require some reworking of the GUI and code.

Oh well, maybe put on the list of things to do when there is nothing else to do. :) ROFL


change text on the help page and fix mobile styling

https://trello.com/c/xBiLY7bm/140-replace-text-in-the-im-looking-for-help-page

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to dev, including commit 16005c591b855e0bb8ac1cae94bf06cc2abd7f95

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to test, including commit 16005c591b855e0bb8ac1cae94bf06cc2abd7f95


Checkbox Column target wrong row when using multiple orWhere clauses.

Package
filament/filament
Package Version
3.2.42
Laravel Version
10.46.0
Livewire Version
3.4.6
PHP Version
8.2.12
Problem description
Whilst trying to set up a CheckboxColumn for a custom page, I've noticed that if I use multiple "orWhere" clauses in my query, the CheckboxColumn behavior becomes erroneous, adding an "orderBy" makes it not work or update the wrong record. I think it migh be an issue with Keying, since the very first row always works normally, the second row sometimes works and sometimes doens't (depending on the set up).
Expected behavior
The CheckboxColumn should update a record, and it should be the record in the corresponding row.
Steps to reproduce
It should be straighforward,
In the repository I built the Pet Model, with migrations and factory
The seeder will create the pets and an user called Victor, I've added a custom login page already pre-filled to be used in local environments.
Then I have my  Custom Filament Page:
<?php

namespace App\Filament\Pages;

use App\Models\Pet;
use Filament\Pages\Page;
use Filament\Tables\Table;
use Illuminate\Support\Carbon;
use Filament\Tables\Columns\IconColumn;
use Filament\Tables\Columns\TextColumn;
use Filament\Tables\Columns\CheckboxColumn;
use Filament\Forms\Contracts\HasForms;
use Filament\Tables\Contracts\HasTable;
use Filament\Forms\Concerns\InteractsWithForms;
use Filament\Tables\Concerns\InteractsWithTable;
class CheckBoxColumnError extends Page implements HasTable, HasForms
{

    use InteractsWithTable;
    use InteractsWithForms;

    protected static ?string $navigationIcon = 'heroicon-o-document-text';
    protected static ?string $slug = 'collapsible';
    protected static string $view = 'filament.pages.collapsible-test';

    public function table(Table $table)
    {

        //The checkbox list breaks funcionality when you add more than on orWhere clause to the query.
        //In the developer tools I managed to see that the target change.

        return $table->query(
            Pet::query()
                ->orWhere('pet_status', 'inbound') //Commenting OUT the rest of the query below restores funcionality.
                ->orWhere('pet_status', 'cremation') //Commenting IN the second row, the CheckboxColumn simple doesn't work.
                ->orWhere('pet_status', 'hold') // Commeting IN the third row, the only the first two records work normally.
                ->orderby('is_cremated', 'asc')) // Commeting IN orderBY, the CheckboxColumn updates the wrong records (click on the first record above the first is_cremated=true)

            ->columns([
                TextColumn::make('date_of_call')
                    ->label("Date/Time")
                    ->formatStateUsing(function($record){return Carbon::parse($record->date_of_call)->format('M d H:i'); })
                    ->description(fn ($record) => Carbon::parse($record->date_of_call)->diffForHumans()),

                TextColumn::make('pet_status'),
                TextColumn::make('name'),
                TextColumn::make('age')->formatStateUsing(fn($record) => $record->is_cremated),
                IconColumn::make('is_cremated')->label("Cremated")->boolean(),
                CheckboxColumn::make('is_cremated')->label("Cremated"), // Not Sure if is a bug or just conflict, but When changing the order of the columns, the bottom column will not show.
                ]);

    }
}

And the view:
<x-filament-panels::page>
{{$this->table}}
</x-filament-panels::page>

Reproduction repository
https://github.com/VictorPennington/BugReport---CheckboxColumn.git
Relevant log output
No response

To start with, this is unsafe query grouping, you should wrap it in a nested WHERE:
->where(fn ($query) => $query
    ->where('pet_status', 'inbound')
    ->orWhere('pet_status', 'cremation')
    ->orWhere('pet_status', 'hold'))

However, your example just requires a whereIn():
->whereIn('pet_status', ['inbound', 'cremation', 'hold'])

Thank you very much for clarifying this issue,
I'll make sure to study more about Eloquent to get it right, the Filament project is deeply appreciated! I apologize for wasting time with ignorance and I'm grateful for the understanding of how difficult is to predict when I'm being inapt or when i☺s an actual bug.
Best!


Packages built unnecessarily after init?

It seems that evaluating a use-package form causes a package rebuild every time.

Can't reproduce.


Allow arbitrary filters

Add an example for complex filters of pyodata

How to define complex AND/OR filters


Update JMX User Guide document

Would you like to use the english version screen shots in english doc? I can make them in my english operating system? :)

Would you like to use the english version screen shots in english doc? I can make them in my english operating system. :)

Oh right! I ignored the language in the image! I'll change the pics.😂


the Package has problems with latest gradle-7.3.3 version , and here what i get

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':camera_deep_ar:bundleDebugAar'.

Error while evaluating property 'hasLocalAarDeps' of task ':camera_deep_ar:bundleDebugAar'
Direct local .aar file dependencies are not supported when building an AAR. The resulting AAR would be broken because the classes and Android resources from any local .aar file dependencies would not be packaged in the resulting AAR. Previous versions of the Android Gradle Plugin produce broken AARs in this case too (despite not throwing this error). The following direct local .aar file dependencies of the :camera_deep_ar project caused this error: C:\src\flutter.pub-cache\hosted\pub.dartlang.org\camera_deep_ar-0.0.2+3\android\lib\deepar0.aar

related to https://github.com/mtellect/CameraDeepAR/issues/24#issue-1091689008
kindly check the branch dev/example to see implementation

Yes , i used old version of gradle then it is work , but it is not work
with latest version
في الثلاثاء، 25 يناير 2022 في 4:38 م تمت كتابة ما يلي بواسطة Maugost <
@.***>:

related to #24 (comment)
https://github.com/mtellect/CameraDeepAR/issues/24#issue-1091689008
kindly check the branch dev/example to see implementation
—
Reply to this email directly, view it on GitHub
https://github.com/mtellect/CameraDeepAR/issues/35#issuecomment-1021191620,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUVKOXH5YOQ2ALOEMJCGTB3UX2RWJANCNFSM5MYIUPYA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

nothing related with dev/example 😊
في الأربعاء، 26 يناير 2022 في 1:50 ص تمت كتابة ما يلي بواسطة Mohammed
Hamdan @.***>:

Yes , i used old version of gradle then it is work , but it is not work
with latest version
في الثلاثاء، 25 يناير 2022 في 4:38 م تمت كتابة ما يلي بواسطة Maugost <
@.***>:

related to #24 (comment)
https://github.com/mtellect/CameraDeepAR/issues/24#issue-1091689008
kindly check the branch dev/example to see implementation
—
Reply to this email directly, view it on GitHub
https://github.com/mtellect/CameraDeepAR/issues/35#issuecomment-1021191620,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUVKOXH5YOQ2ALOEMJCGTB3UX2RWJANCNFSM5MYIUPYA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Debug language switch navigation link-error

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

By previews code, when I were at https://opensource.guide/zh-cn/how-to-contribute/ or any other language page except English, if I want to switch to traditional Chinese language, after I click the bottom on the top right corner of the page,  https://opensource.guide/how-to-contribute/zh-tw/  is open instead of https://opensource.guide/zh-tw/how-to-contribute/. The former is wrong but the new one is right.
I fixed this bug.

this bug was from #1033

@qijiale76, this was changed because deploy in a subdir fail.
Maybe the ideal url for How to Contribute should be https://opensource.guide/how-to-contribute/zh-tw/ instead https://opensource.guide/zh-tw/how-to-contribute/
Anyway consider this in your PR.

Hmm, I thought this was working but evidently missed something. If this breaks deploy in subdir again, we'll need a new PR to re-fix that.

It should definitely not be necessary to reorder the path elements.

CONTRIBUTING.md


remodel_slotlist_detail doc update

In addition to
https://github.com/andanteyk/ElectronicObserver/blob/a6bf856023796ad744906237889f0e7a304173a6/ElectronicObserver/Other/Information/apilist.txt#L764-L765
there is also api_req_useitem_id2 and api_req_useitem_num2 for when two items are used (like 12.7cm連装砲C型改二 -> 12.7cm連装砲D型改二).

Also something unrelated, but maybe it can be mentioned in kcmemo. Wikiwiki states that successful modernization bonus is floor(n + (n + 1) / 5), which is also defined in scene.remodel.view.powup.PowupUI in RemodelMain as floor(n * 1.2 + 0.3) (same function).

Thank you for your information.
Updated at aa8bb4bb5ccd1a265a41860e3214b72a335189f8 .


The display will flash quickly for a while after calling d3dshot.create(), and also when exiting the program.

I don't know when this phenomenon started. It didn't appear when I first used the d3dshot package. It was very strange, and it did not improve after restarting the computer.
bug_video
System version: Windows10 1909
Python version: 3.7.6
d3dshot version: 0.1.5

Through debugging, I located the function where this phenomenon occurred in d3dshot.dll.dxgi.initialize_dxgi_output_duplication, and the display screen will flicker once every time the function is executed.

I just tested another monitor, the model is aoc Q27P1B, it does not support the Monitor Control Command Set, and this problem does not occur. The monitor with the bug (aoc U2790B, which supports the Monitor Control Command Set) uses the same computer hardware, the cpu is i5-8600k, and the graphics card is RTX 2070. The graphics card has two DP ports. I usually plug them in at the same time to use dual-screen displays. When using d3dshot at the same time, only U2790B will experience screen flickering.
Then I tried reinstall d3dshot throgh pip, the problem still exists, is there anyone who can help me, I am grateful!

Hi @BruceLee569,
I am trying to get your error on 2 differents machines and nothing happened about your problem.
Both of them have Python 3.8.5 and  d3dshot 0.1.5.
First one is an MSI GS60 2QE Ghosht (i7-4k; GTX-970M). The second one is an pc-based (i7-6k;  GTX-980 Ti).
I am not familiar with the MCCS and I do not know if your problem comes from that.
I saw on the shared video that the flicker happened on call and release. Is your desktops have the same size definition (1920x1080, or ?). From my point of view, it is more a problem of a region definition pointer who a initialized, released.

Hi @BruceLee569,
I am trying to get your error on 2 differents machines and nothing happened about your problem.
Both of them have Python 3.8.5 and d3dshot 0.1.5.
First one is an MSI GS60 2QE Ghosht (i7-4k; GTX-970M). The second one is an pc-based (i7-6k; GTX-980 Ti).
I am not familiar with the MCCS and I do not know if your problem comes from that.
I saw on the shared video that the flicker happened on call and release. Is your desktops have the same size definition (1920x1080, or ?). From my point of view, it is more a problem of a region definition pointer who a initialized, released.

You are awesome, it is really a resolution problem, because the resolution of the main monitor (the one with the problem) is 4K (3840x2160), and the second monitor is 2K (2560x1440),
In order to unify it, and I think the screenshots are taken too many pixels should be slower, so I set the resolution of the main monitor to 2560x1440, so the splash screen problem occurred. After I changed the resolution back, the problem has disappeared.
Just now I wanted to test the secondary monitor and change its resolution to 1920x1080, but the result of the d3dshot screenshot only returned the image of the primary monitor. I wonder if there is a way to capture the two monitors? Thank you again and again for your beautiful answers!☺

You are awesome, it is really a resolution problem, because the resolution of the main monitor (the one with the problem) is 4K (3840x2160), and the second monitor is 2K (2560x1440),
In order to unify it, and I think the screenshots are taken too many pixels should be slower, so I set the resolution of the main monitor to 2560x1440, so the splash screen problem occurred. After I changed the resolution back, the problem has disappeared.

You are awesome, it is really a resolution problem, because the resolution of the main monitor (the one with the problem) is 4K (3840x2160), and the second monitor is 2K (2560x1440),
In order to unify it, and I think the screenshots are taken too many pixels should be slower, so I set the resolution of the main monitor to 2560x1440, so the splash screen problem occurred. After I changed the resolution back, the problem has disappeared.

I was thinking the reverse :s. The harmonization of screen display change remove the flicker.
Have you trying to set display before screenshot ?
d = d3dshot.create()
d.display = d.displays[x]  #where x is the display you want to caputre.

After the test, the 2K display has no flicker problem, even if its resolution is changed to 1080P. as long as the main display is not 4K, flicker will occur, will it be a problem with the physical display.

Ok, I just try something with the resolution and I can get your issue :-).
**Test 1 : Flicker happen**
Screen display 1 : 1600x1024 => on this screen only
Screen display 2 : 1920x1080

**Test 2 : Flicker not happen**
Screen display 1 : 1920x1080
Screen display 2 : 3840x2160

**test 3 : Flicker happen**
Screen display 1 : 1600x1024 => on this screen only
Screen display 2 : 3840x2160

So the issue is more screen display resolution problem. I do not know if it is a problem from the physical display or with the dxgi-d3d11 stack implementation from d3dshot or simply a "limitation" in the d3d11.
d3dshot currently at the dxgi1_2 and I do not know if the dxgi1_5 can resolve this type of issue.

Ok, I just try something with the resolution and I can get your issue :-).
**Test 1 : Flicker happen**
Screen display 1 : 1600x1024 => on this screen only
Screen display 2 : 1920x1080

**Test 2 : Flicker not happen**
Screen display 1 : 1920x1080
Screen display 2 : 3840x2160

**test 3 : Flicker happen**
Screen display 1 : 1600x1024 => on this screen only
Screen display 2 : 3840x2160

So the issue is more screen display resolution problem. I do not know if it is a problem from the physical display or with the dxgi-d3d11 stack implementation from d3dshot or simply a "limitation" in the d3d11.
d3dshot currently at the dxgi1_2 and I do not know if the dxgi1_5 can resolve this type of issue.

Is this the result of the monitor you tested? So far, my test result is that only this 4K monitor will have Flicker, unless its resolution is set to the original 4K; the other 2K monitor will not have Flicker problems no matter how to modify the resolution.
So what is suspicious now is the physical display's problem, but the problem is that I don’t have another monitor yet, modify its initial resolution to view the test results. Fortunately, other people can also see this issue, and they can give feedback on the testing status of their displays after modify resolution.

Yes, this is the result of my monitor I tested. I do another test where I can say that :

The flicker appears only on the you are the main screen definied in the windows parameters.
On my side, the flicker appears when the resolution is under 1920x1080 (who is the recommended resolution by Windows).

I am really not sure that the problem comes from the physical display. I think it is more about how the DXGI - D3D11 is implemented. If we look at the DXGI Best Practices (MSDN), and the implementation in c/c++, in case of multi-monitor a swapchain is used.

Yes, this is the result of my monitor I tested. I do another test where I can say that :

The flicker appears only on the you are the main screen definied in the windows parameters.
On my side, the flicker appears when the resolution is under 1920x1080 (who is the recommended resolution by Windows).

I am really not sure that the problem comes from the physical display. I think it is more about how the DXGI - D3D11 is implemented. If we look at the DXGI Best Practices (MSDN), and the implementation in c/c++, in case of multi-monitor a swapchain is used.

Since fliker appears on your monitor, it should be a software problem.
In the field of GUI, the modification of resolution and percentage scaling of Windows system will always cause various situations and problems, which will affect the actual display of the software. This situation is no surprise to me 😂
Still looking forward to the problem being solved, come on 💪


Theme

Please check to make sure we can add students all right. We can also delete all unused files after finishing up too. Please run "bundle exec jekyll serve" to run and may need to "gem install bundler" to do so.

oh and don't forget to do "bundle install" to install all the gems.

This looks good to me. Maybe a bit more than we need, but like i said, it all compiles into flat files so no harm in a big jekyll. resolves #5


Relabling on cutmix ?

Hi
nice work, i enjoyed reading the article.
i noticed that you are using "plain" cutmix in the training.
have you considered doing "relabeld-cutmix", meaning pooling the targets from the relevant area only ?
it is definitely more complicated, but I think it can even count as a new interesting type of augmentation, that can be used
not only when learning from a teacher
Tal

Hi @mrT23

have you considered doing "relabeld-cutmix", meaning pooling the targets from the relevant area only ?
it is definitely more complicated, but I think it can even count as a new interesting type of augmentation, that can be used
not only when learning from a teacher

I agree. That is a very good point!
It would be an interesting and new data augmentation, but based on our experiments, it shows similar or slightly worse results compared with the plain-cutmix.
I think relabeled-cutmix is conceptually better than plain-cutmix, but we may have to find a proper training setting for relabeled-cutmix. Ans also the cutmix region can be relatively very small and the quantization issue (during pooling) may appear.
Anyway, for this reason, the results with relabeled-cutmix is omitted in the current version of our paper. If we find a better way to implement relabeled-cutmix, we will update our paper.

thanks for the answer
its always frustrating when things that seem logical and more "correct" don't give immediately score improvement :-)
I think that label-pooling is interesting, and reflects a deeper understanding of the augmentation-process and limitations of "single-label" datasets.
i never really liked self-supervised methods (moco, simclr, mix-and-match,...), because they always assume this contrastive loss where under augmentations, the labels (or embedding) should remain the same, which is not always true, especially for spatial augmentations.
with label-pooling, you can also do some kind of contrastive loss, which is much more logical: take an image and a zoomed-in version; Demand that the labels of the zoomed-in image, and label pooling of the original image labels, will be the same.
this also has the advantage of self-teaching spatial capabilities to a detector.

@mrT23

with label-pooling, you can also do some kind of contrastive loss, which is much more logical: take an image and a zoomed-in version; Demand that the labels of the zoomed-in image, and label pooling of the original image labels, will be the same.
this also has the advantage of self-teaching spatial capabilities to a detector.

Your point is quite interesting and reasonable, but I think the biggest advantage of self-supervised learning is that it can train meaningful representations without any supervision. Since our ReLabel needs explicit label supervision, comparison with self-supervised learning methods will not be easy.
Or, adding the loss that minimizes the distance between the pooled-label and the cropped image's label as an auxiliary loss would be interesting!

from my experience and past tests, contrastive methods fail to improve the pretrain quality of models.
this is in contrast to KD and relabling methods, which also don't need ground-truth.
anyway, nice work
:-)


Correct namespace to Caffine in readme for Laravel 5.3?

Issue
I get "Class 'GeneaLabs\LaravelCaffeine\Providers\LaravelCaffeineService' not found" if I follow the installation instructions on the front page.
Found an old github issue regarding problems with Laravel 5.2 and changed to GeneaLabs\LaravelCaffeine\LaravelCaffeineServiceProvider::class and I get passed the installation. Which one is the correct one for Laravel 5.3, or will there be any problems?
Environment
Laravel Version: 5.3.29
Laravel Caffeine Package Version: 0.3.12
PHP Version: 7.1

Thew newest version should be:
GeneaLabs\LaravelCaffeine\LaravelCaffeineServiceProvider::class
Can creator update the docs?

You should be updating to package 0.4.x when using Laravel 5.3+.


Temp File Cleanup Issue

Hi,
Like many users of RazorEngine, I also have faced with temp file issue. After a research, I tried to execute my code in a separate AppDomain. For this, I implemented a simple Isolated class, using guidelines :
    public sealed class Isolated<T> : IDisposable where T : MarshalByRefObject
    {
        private AppDomain _domain;
        private T _value;

        public Isolated()
        {
            _domain = AppDomain.CreateDomain("Isolated : " + Guid.NewGuid(), null, AppDomain.CurrentDomain.SetupInformation);
            Type type = typeof(T);

            _value = (T)_domain.CreateInstanceAndUnwrap(type.Assembly.FullName, type.FullName);
        }

        public T Value
        {
            get
            {
                return _value;
            }
        }

        public void Dispose()
        {
            if (_domain != null)
            {
                AppDomain.Unload(_domain);
                _domain = null;
            }
        }
    }

After that, I implemented the class which uses RazorEngine for generating text :
    [Serializable]
    public class TextGeneratorParameters
    {
        public string TemplateText { get; set; }
        public string TemplateKey { get; set; }
        public IDictionary<string, object> Model { get; set; }
    }

    public class TextGenerator : MarshalByRefObject
    {
        public string GenerateText(TextGeneratorParameters parameters)
        {
            // configure razor engine
            var config = new TemplateServiceConfiguration();

            config.Language = Language.CSharp;
            config.EncodedStringFactory = new RawStringFactory();
            config.Debug = true;

            // create razor instance
            var service = RazorEngineService.Create(config);
            Engine.Razor = service;

            Engine.Razor.Compile(parameters.TemplateText, parameters.TemplateKey, null);

            var expando = DynamicObjectUtilities.DictionaryToExpando(parameters.Model);

            return Engine.Razor.Run(parameters.TemplateKey, null, expando);
        }
    }

And this is the code snippet which I use them as Isolated :
            Isolated<TextGenerator> textGenerator = new Isolated<TextGenerator>();

            try
            {
                foreach (dynamic item in model)
                {
                    _templateKey = String.Format("{0}{1}", _templateKeyRoot, ++_templateNo);

                    IDictionary<string, object> modelItem = DynamicObjectUtilities.ExpandoToDictionary(item);

                    var result = textGenerator.Value.GenerateText(new TextGeneratorParameters
                    {
                        TemplateText = scintillaTemplate.Text,
                        TemplateKey = _templateKey,
                        Model = modelItem
                    });

                    _nodeResults.Add(item.Name, result);
                }
            }
            catch (Exception ex)
            {
                MessageBox.Show("Error : " + ex.Message);
            }
            finally
            {
                textGenerator.Dispose();

                string tempFolder = Path.GetTempPath();

                CleanUpTempFiles(tempFolder, "RazorEngine_*");
            }

        private void CleanUpTempFiles(string tempFolder, string pattern)
        {
            foreach (string temp in Directory.EnumerateDirectories(tempFolder, pattern))
            {
                try
                {
                    File.Delete(temp);
                }
                finally
                {
                }
            }
        }

When I call CleanUpTempFiles, it gives this "Access denied" exception. My aim is to execute all RazorEngine things in a separate AppDomain, which I think to allow me to delete all temporary "RazorEngine_" folders after execution. I think I'm doing something wrong.
Any idea?
Thanks

I don't see anything obvious, two things to check: There is already some cleanup code shipped as part of RazorEngine which you might want to disable. Maybe the AppDomain is not fully unloaded then the Unload call returns? Maybe some assemblies/types might have slipped through the AppDomain boundary (You can check this with AppDomain.GetAssemblies() in the debugger).
Hope this helps.

I modified the code a little bit and here is the version which doesn't use Isolated :
            if (AppDomain.CurrentDomain.IsDefaultAppDomain())
            {
                AppDomainSetup adSetup = new AppDomainSetup();
                adSetup.ApplicationBase = AppDomain.CurrentDomain.SetupInformation.ApplicationBase;
                var current = AppDomain.CurrentDomain;

                var strongNames = new StrongName[0];

                var domain = AppDomain.CreateDomain(
                                         "MyMainDomain", null,
                                         current.SetupInformation, new PermissionSet(PermissionState.Unrestricted),
                                         strongNames);

                TextGenerator textGenerator = (TextGenerator)domain.CreateInstanceAndUnwrap("TemplateEngine", "TemplateEngine.TextGenerator");
                var expandoConverter = new ExpandoObjectConverter();

                try
                {
                    foreach (dynamic item in model)
                    {
                        _templateKey = String.Format("{0}{1}", _templateKeyRoot, ++_templateNo);

                        string json = JsonConvert.SerializeObject(item, expandoConverter);

                        string result = textGenerator.GenerateText(scintillaTemplate.Text, _templateKey, json);
                    }
                }
                catch
                {
                    // TO DO : implement exception handling
                }
                finally
                {
                    AppDomain.Unload(domain);
                }
            }

I checked with AppDomain.CurrentDomain.GetAssemblies(), RazorEngine assembly is not loaded until the " textGenerator.GenerateText(...)" call. It throws an exception (for a different reason). I call AppDomain.Unload in "finally" block but I see the RazorEngine assembly remain loaded into current (default) app domain. And no file is deleted.

Is there a reason why you cannot use the code we ship? Aren't the files deleted on application exit as they should?

I'm not sure about what do you mean with the code you ship but, no, they remain in Temp folder after application exit.

Hi,
Any updates about that? What I need is to compile a template, run it several times for different inputs, and remove created temp files after execution.

The trick is that the RazorEngine is not allowed to "slip though" into the default AppDomain.
I know this is tricky but if you do it properly it should work.
The assembly will slip though with several methods (like doing .GetType on a proxy for example).
Closing old issue. Feel free to open a new one.


nxstyle: Ignore long line for file path at line 2

Summary
nxstyle: Ignore long line for file path at line 2
Impact
Fixes issues like that reported on PR #12345
Testing
local

ping @xiaoxiang781216


$(window).width() causes getBoundingClientRect error

Using $(window).width() causes this error

this[0].getBoundingClientRect is not a function

at this line https://github.com/kenwheeler/cash/blob/0ff26e3e6694f36d22af042eeb43423378761191/dist/cash.js#L492
How i'm using it..
window.cash = require('cash-dom');

(function(window, document, $) {
	'use strict';

	$(document).ready(function() {
		console.log( $(window).width() );
	});

})(window, document, window.cash);

Has anyone else run into this issue, or a possible solution?

window.getBoundingClientRect doesn't exist, so you shouldn't do this. Just use window.innerWidth instead.

As of v2.3.1 you can now do $(window).width (), all the others dimensions-related methods are supported as well.


Fix broken internal links

There were several internal links that were broken after the move to the Docsy theme. Thanks to @masnick for noticing a few of these broken links in #10 and alerting us to this issue.

I don't want to leave these links broken, so I am going to merge this PR without an official review, just this once...


Enable responsive layout of charts in Data Dashboard

User Story
As a user who needs to look at Complaint Manager data
I want to be able to see the charts when I reduce my screen size
So that I can view multiple windows simultaneously without losing any information (e.g. if I need to work on a report and have the charts open in a side by side view)
Notes
This story refers to the private Data Dashboard.
Refer to #535 for a previous investigation into this issue and possible solutions
Pair with Natalia on the implementation of this card
Definition of Done
When resizing the browser window, all the charts within the Complaint Manager are adjusted dynamically to maintain legibility.
Acceptance Criteria
Given I have the Data Dashboard open in full screen
When I reduce my screen size
Then the charts are resized

Verified charts are responsive in data dashboard
Hawaii
https://github.com/PublicDataWorks/complaint-manager/assets/134408135/e7ec8c58-0964-48ad-a167-11ef5e9456d0
NOIPM
https://github.com/PublicDataWorks/complaint-manager/assets/134408135/670be816-6fe4-473c-9541-fcb51e235bd5


dependency on unmaintained mach crate

When running cargo audit for my project I noticed this:
andrew@MacBook-Pro flow % cargo audit
    Fetching advisory database from `https://github.com/RustSec/advisory-db.git`
      Loaded 609 security advisories (from /Users/andrew/.cargo/advisory-db)
    Updating crates.io index
    Scanning Cargo.lock for vulnerabilities (495 crate dependencies)
Crate:     mach
Version:   0.3.2
Warning:   unmaintained
Title:     mach is unmaintained
Date:      2020-07-14
ID:        RUSTSEC-2020-0168
URL:       https://rustsec.org/advisories/RUSTSEC-2020-0168
Dependency tree:
mach 0.3.2
└── wasmtime-runtime 18.0.2
    └── wasmtime 18.0.2
        └── flowr 0.142.0

Duplicate of #6000 that has a still-open PR #6164 to move to mach2 instead; @fitzgen, you're one of the listed owners of mach; any thoughts on this?

(As @jameysharp mentioned on #6164, we'd need to audit the fork, or at least the diff from its branch-point, so this isn't a no-effort switch, but the current situation isn't great either...)

@fitzgen, you're one of the listed owners of mach; any thoughts on this?

I started the crate nearly a decade ago, and haven't touched it in nearly as long. I handed off ownership to others, and I guess they are gone. I have no interest in maintaining the crate.

Makes sense! Any interest in auditing mach2, since you know the codebase? From #6164 it looks like once we have that it's a pretty straightforward find-replace to switch.


[test] SerializeTimeSpansAsTicksAndStrings randomly failed on CI

https://dev.azure.com/azure3/elasticsearch-net/_build/results?buildId=3125&view=logs&j=b43a14c2-2046-5949-944a-5a4631b14be2&t=301652d8-af95-550e-0530-c591704039a1&l=126
[xUnit.net 00:00:02.9411802]     Tests.Reproduce.TimeSpanSerialization.SerializeTimeSpansAsTicksAndStrings [FAIL]
  X Tests.Reproduce.TimeSpanSerialization.SerializeTimeSpansAsTicksAndStrings [152ms]
  Error Message:
   Expected 104d, 10h, 38m and 32s, but found 10d, 10h, 38m and 32s.
  Stack Trace:
     at FluentAssertions.Execution.XUnit2TestFramework.Throw(String message)
   at FluentAssertions.Execution.TestFrameworkProvider.Throw(String message)
   at FluentAssertions.Execution.DefaultAssertionStrategy.HandleFailure(String message)
   at FluentAssertions.Execution.AssertionScope.FailWith(Func`1 failReasonFunc)
   at FluentAssertions.Execution.AssertionScope.FailWith(String message, Object[] args)
   at FluentAssertions.Primitives.SimpleTimeSpanAssertions.Be(TimeSpan expected, String because, Object[] becauseArgs)
   at Tests.Reproduce.TimeSpanSerialization.SerializeTimeSpansAsTicksAndStrings() in /home/vsts/work/1/s/tests/Tests.Reproduce/TimeSpanSerialization.cs:line 30
[xUnit.net 00:00:03.2945958]     Tests.Reproduce.GithubPR3815.CanDeserializeNestedError [SKIP]
  ! Tests.Reproduce.GithubPR3815.CanDeserializeNestedError [1ms]

The test is really simple; it's not obvious how the expected value was ascertained

Very strange. This is the test: https://github.com/elastic/elasticsearch-net/blob/fea5445c345c4f041ab5835bb0c2b02d38fac7c0/tests/Tests.Reproduce/TimeSpanSerialization.cs#L15
As @russcam mentions, its very strange how expected is 104 days and not 10 days.

Couple of potential avenues to explore:

Reuse of Memory stream and bytes written from MemoryStreamFactory. This seems unlikely because the written bytes would need to match the same JSON structure to deserialize correctly.
Concurrency race condition in serialization

Side note;
9023120000000 ticks = 10.10:38:32
90231200000000 ticks = 104.10:25:20
90239120000000 ticks = 104.10:38:32

good catch. May need to dig into https://github.com/elastic/elasticsearch-net/blob/dd5821210f74218a3cfa226da22195294c08d77d/src/Elasticsearch.Net/Utf8Json/Formatters/DateTimeFormatter.cs#L791

@Mpdreamz I am getting a 404 (build not found) on the above build link...

I think its been cleaned up.

Ah, OK, I think this is going to be hard to repro without the test run settings (seed). I'll leave the ticket open for the time being and if it occurs again we should document enough for a repro.

Not seen this issue surface again. Going to close this, and will reopen if resurfaces.


Add Command Line Arguments to DefaultContext

The command line arguments seem like a really useful piece of information to have when debugging crashes. This adds them to the DefaultContext so that they're readily available in Sentry

Hey @jayhf, I believe we're already adding the command line input to Crash Info context on Win/Linux for captured crash events:
https://github.com/getsentry/sentry-unreal/blob/982894bd75f914789ef9e9c14aaab5d774b71324/plugin-dev/Source/Sentry/Private/Desktop/CrashReporter/SentryCrashContext.cpp#L54
Does it miss some bits of info you expect to be there along with other CL arguments?

Ah, that's great to know. I didn't realize. Where exactly can those context be found? I haven't seen any of those fields in the crash logs I'm analyzing. Are there any docs you'd recommend for what information is available where?

The command line arguments can be found at captured crash event's page in Contexts -> Crash Info section in case you're using Sentry's plugin integration for error handling:

If you're sending crashes to Sentry with the default Crash Reporter instead then CL args should be listed in Contexts -> unreal -> custom.
Also, you can consider enriching the events with custom contexts via the plugin's API if the default one misses some bits of info you're interested in.

Sorry for the slow response here. I think I've figured out why I wasn't seeing the information before: Most of the issues I'm encountering are assertion failures, which ultimately result in the process exiting, but aren't treated as crashes by the Sentry plugin. Interestingly, even the crash context records IsAssert, so this seems to be due to a bug or limitation of Sentry's plugin. Perhaps it's even a Linux specific issue. These two issues might be related: https://github.com/getsentry/sentry-unreal/issues/381 and https://github.com/getsentry/sentry-unreal/issues/496, but neither seems like exactly the issue I'm hitting. At any rate, this clearly isn't the right approach so I'll close the PR and I may follow up with an issue later.

Makes sense. I'll keep an eye on that issue, thanks!


android push message replace previous

Hello, in Android, when I receive 2 pushes in row, (preview in taskbar) the latest always deletes the first and when I open the app by clicking on it, the first push in not handled.
What's wrong?
I use pushwoosh cordova plugin version 3.5.9

You should call setMultiNotificationMode if you want several notifications to be displayed in taskbar.


fix: DIffEditor should render inline when in terminal split

Fixes #6366
render inline when there's split

render back to original when there isn't split

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[ ] 🐛 Bug fix
[x] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

should this be just "isTerminalSplit"? or "isWidthConstrained"? rather than having the term/meaning be specific to sidecar?

Resolved, thanks!


eclipse fix

hey guys,
i changed the poms to be able to work in eclipse without hazzle. do you agree?
cheers, andi

do I see it right that the commit #95b23ea was meant for your local branch ?

merged - this was my safer since I messed up the pom.xml weeks ago to a point where I got NPE in eclipse :-( without any hint what is wrong. thanx.


latest code not available in https://crates.io/

Can you help with the latest code in https://crates.io/

bump with the new features added, could this be pushed?

@mon @srinivasanraghavan can someone of you please test if 0.3.0 suits your needs? We can close this issue then.

@mheese windows x64 strikes again
: mismatched types
   --> pkcs11-0.3.0\src\lib.rs:890:64
    |
890 |         match (self.C_Sign)(session, data.as_slice().as_ptr(), data.len(), signature.as_slice().as_ptr(), &signatureLen) {
    |                                                                ^^^^^^^^^^ expected u32, found usize

just add back the data.len() as CK_ULONG and it should be good to go

there was  a merge conflict ... I thought I got it right :)

@mon released version 0.3.1 which addresses that

All good now, cheers! My cargo.toml is pretty again :D

Thanks again @mon ! And sorry about the huge delay.


Data Ingestion Stage

[x]  Update config.YAML
[x]  research or analysis
[x]  Update the entity
[x]  Update the configuration manager in src config.
[x]  Update the components
[x]  Update the pipeline
[x]  Test case added

Dry run successful for stage 1


Change retirement state machine order.

Execute FinishRetirement state prior to DeleteFromVMDB for cloud, infra
and services so the audit event will get raised.

:+1: