create new class

i  want to inherit from the class FConvModell,
the code is
captionmodel,parent=torch.class('captionmodel','FConvModel')
what else should i configure?

Sorry for the late reply -- it depends on the exact changes you want to make to the model, or how you want to extend it. FConvModel inherits from AvgpoolModel and I'd recommend looking at both models to see what functions are used. For example, if you want to change the layout of the encoder, you might want to override makeEncoderStack; if you want to provide alternative input alongside the source sentence you would also override prepareSource (defined in avgpool_model.lua).

in the dictionary of model in fairseq，there is a lua script named init,should i configure new class in the script,what else should i configure?

Ah, you're asking whether you need to change anything in order to be able to use your new model? Sorry for the misunderstanding. Adding your model to models/init.lua should be sufficient.

Also, and the file name format has to be <name>_model.lua. See make_model_fn in train.lua for the code that creates a new model based on the -model command-line argument.

thank you very much

@jgehring
I want to create new class inherited from FConvModel,I want to override the makeEncoder method,
MineModel.makeEncoder = argcheck{
{name='self', type='FConvModel'},
{name='config', type='table'},
call = function(self, config)
local sourceIn = nn.Identity()()
local tokens, positions = sourceIn:split(2)
local embed = self:makeEmbedding(config, config.srcdict,
config.dropout_src)({tokens, positions})
    local cnn
    if #config.nhids > 0 then
        cnn = nn.Sequential()
        cnn:add(nn.Bottle(
            self:makeLinear(config.nembed, config.nhids[1],
                config.dropout_src)
        ))
        cnn:add(nn.Contiguous())
        cnn:add(self:makeEncoderStack(config, config.nhids))
        cnn:add(nn.Bottle(self:makeLinear(config.nhids[#config.nhids],
            config.nembed)))
    else
        cnn = nn.Identity()
    end

    -- The encoder stack will receive gradients *twice* for each attention
    -- pass: dot product and weighted sum.
    local nattn = #attentionLayers(#config.nlmhids, config.attnlayers)
    cnn = nn.GradMultiply(cnn, 1 / (2 * nattn))

------------------------------------- here is my added code
local r=embed.output:size(2)
i will use r variable to create a new Linear layer
but the embed doesn't have the output field
so what  can i do to implement the code that has the similar function
    local outputA = cnn(embed)
    -- Source feeding for weighted sum in attention: add embeddings to CNN
    -- output
    local outputC = nn.CAddTableMulConstant(
        math.sqrt(0.5))({outputA, embed})
    return nn.gModule({sourceIn}, {outputA, outputC})
end

}

@jgehring @michaelauli I want to create new class inherited from FConvModel,I want to override the makeEncoder method,
MineModel.makeEncoder = argcheck{
{name='self', type='MineModel'},
{name='config', type='table'},
call = function(self, config)
local sourceIn = nn.Identity()()
local tokens, positions = sourceIn:split(2)
local embed = self:makeEmbedding(config, config.srcdict,
config.dropout_src)({tokens, positions})
local cnn
if #config.nhids > 0 then
    cnn = nn.Sequential()
    cnn:add(nn.Bottle(
        self:makeLinear(config.nembed, config.nhids[1],
            config.dropout_src)
    ))
    cnn:add(nn.Contiguous())
    cnn:add(self:makeEncoderStack(config, config.nhids))
    cnn:add(nn.Bottle(self:makeLinear(config.nhids[#config.nhids],
        config.nembed)))
else
    cnn = nn.Identity()
end

-- The encoder stack will receive gradients *twice* for each attention
-- pass: dot product and weighted sum.
local nattn = #attentionLayers(#config.nlmhids, config.attnlayers)
cnn = nn.GradMultiply(cnn, 1 / (2 * nattn))

------------------------------------- here is my added code
local r=embed.output:size(2)
i will use r variable to create a new Linear layer
but the embed doesn't have the output field
so what can i do to implement the code that has the similar function
local outputA = cnn(embed)
-- Source feeding for weighted sum in attention: add embeddings to CNN
-- output
local outputC = nn.CAddTableMulConstant(
    math.sqrt(0.5))({outputA, embed})
return nn.gModule({sourceIn}, {outputA, outputC})

end
}

@jgehring @michaelauli I want to create new class inherited from FConvModel,I want to override the makeEncoder method,
MineModel.makeEncoder = argcheck{
{name='self', type='MineModel'},
{name='config', type='table'},
call = function(self, config)
local sourceIn = nn.Identity()()
local tokens, positions = sourceIn:split(2)
local embed = self:makeEmbedding(config, config.srcdict,
config.dropout_src)({tokens, positions})
local cnn
if #config.nhids > 0 then
cnn = nn.Sequential()
cnn:add(nn.Bottle(
self:makeLinear(config.nembed, config.nhids[1],
config.dropout_src)
))
cnn:add(nn.Contiguous())
cnn:add(self:makeEncoderStack(config, config.nhids))
cnn:add(nn.Bottle(self:makeLinear(config.nhids[#config.nhids],
config.nembed)))
else
cnn = nn.Identity()
end
-- The encoder stack will receive gradients twice for each attention
-- pass: dot product and weighted sum.
local nattn = #attentionLayers(#config.nlmhids, config.attnlayers)
cnn = nn.GradMultiply(cnn, 1 / (2 * nattn))
------------------------------------- here is my added code
local r=embed.output:size(2)
i will use r variable to create a new Linear layer
but the embed doesn't have the output field
so what can i do to implement the code that has the similar function
local outputA = cnn(embed)
-- Source feeding for weighted sum in attention: add embeddings to CNN
-- output
local outputC = nn.CAddTableMulConstant(
math.sqrt(0.5))({outputA, embed})
return nn.gModule({sourceIn}, {outputA, outputC})
end
}

@jgehring @michaelauli I want to create new class inherited from FConvModel,I want to override the makeEncoder method,
MineModel.makeEncoder = argcheck{
{name='self', type='MineModel'},
{name='config', type='table'},
call = function(self, config)
local sourceIn = nn.Identity()()
local tokens, positions = sourceIn:split(2)
local embed = self:makeEmbedding(config, config.srcdict,
config.dropout_src)({tokens, positions})
     local cnn
     if #config.nhids > 0 then
           cnn = nn.Sequential()
           cnn:add(nn.Bottle(
                self:makeLinear(config.nembed, config.nhids[1],
                config.dropout_src)
           ))
          cnn:add(nn.Contiguous())
          cnn:add(self:makeEncoderStack(config, config.nhids))
          cnn:add(nn.Bottle(self:makeLinear(config.nhids[#config.nhids],
          config.nembed)))
     else
           cnn = nn.Identity()
     end

     -- The encoder stack will receive gradients *twice* for each attention
     -- pass: dot product and weighted sum.
    local nattn = #attentionLayers(#config.nlmhids, config.attnlayers)
    cnn = nn.GradMultiply(cnn, 1 / (2 * nattn))

------------------------------------- here is my added code
local r=embed.output:size(2)
i will use r variable to create four new layer ,but the embed doesn't have the output field,
so what can i do to implement the code that has the similar function
  local outputA = cnn(embed)
  -- Source feeding for weighted sum in attention: add embeddings to CNN
  -- output
  local outputC = nn.CAddTableMulConstant(
  math.sqrt(0.5))({outputA, embed})
 return nn.gModule({sourceIn}, {outputA, outputC})

end
}

@jgehring @michaelauli

Do you need the embedding size? That's available in config.nembed.

when network  input is  single sample,i want to know  embed.output:size(1),when network input is batch sample,i want to know embed.output:size(2),i  want to use this number to dynamically set the input dimension of nn.linear layer

@jgehring

@linhanxiao you can't do that while wiring up the nngraph structure with nn.Module objects; the exact input and output shapes are only available during forward and backward passes. I suggest reshaping single examples so that they have a batch dimension of 1, which is also done in generate-lines.lua#L119.


Can you add a note to say this doesn't work from a tenanted admin

Or in other words we don't care about partners!

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

ID: 90059adc-9f5f-bc6c-caab-695e7ca1933a
Version Independent ID: d71bf1c7-ee9d-4c7c-5b65-a8765366b064
Content: Let users reset their own passwords in Office 365
Content Source: Office365-Admin/add-users/let-users-reset-passwords.md
Service: o365-administration
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

@webstaff - Thank you for submitting feedback.
Can you please share what specific issue you have with this doc? Or are you having some technical issues which requires technical support?
Please assist to provide more details so we can investigate further.

@webstaff - We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to open a new issue if you find any documentation error or if there is a specific area of the docs that we can improve on and we will be glad to help fix it.
Thank you!


Adding Type2 Decompositions

This set of changes adds the Type2 decompositions to the runtime. It also includes the splitting of Qsharp.Core into Qsharp.Core + Qsharp.Base, which allows other packages to define a replacement core with alternate intrinsics on top of Base. The Type2 decompositions are the first example of such an alternate core.

Thanks everyone for the great comments and feedback! I filed https://github.com/microsoft/qsharp-runtime/issues/249 to capture/track this work, and based on some of the feedback and design discussions, I will be breaking this change apart into a few PRs to a feature branch, each with more targeted focus. I'll still respond to the comments here so we can capture the discussion and to help guide my approach to the next PR.


Fix Back to Top link issue

Hi:)
It seems that Back to Top link is linked to the index header, which is not in this repo. (Seems like the index header is from EMNLP-2019-Papers repo)
So I've change the link to Accepted Papers with arXiv Link header.
[Back to Top](#accepted-papers-with-arxiv-link)

Thank you for the great project!

Thanks a lot!


workflows: fix docker tag

https://github.com/Homebrew/brew/runs/595913492
https://github.com/Homebrew/brew/pull/7379

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

The first part homebrew of homebrew_sut is the name of the current working directory, translated to lower case. This PR implies that the current working directory is not /home/linuxbrew/.linuxbrew/Homebrew. Should it be?

Also, thanks for the fix, Dawid!


use tokio::fs for all filesystem operations

https://docs.rs/tokio/latest/tokio/fs/index.html

I remember reading an article about tokio::fs is slower than std::fs. Ref: https://users.rust-lang.org/t/why-tokio-fs-is-slower-than-std-fs/39610

I remember reading an article about tokio::fs is slower than std::fs. Ref: https://users.rust-lang.org/t/why-tokio-fs-is-slower-than-std-fs/39610

The async code in the link is slow is because there is no parallelization. The author called .await serially on each entry.
Again. As with all things, we need to benchmark to find out.


Gen2 Multi Levelling: Add flows

When the user attaches a gen2 multi in the change pipette screen, it should

prompt the user to screw in the pipette only lightly before checking connection
display a leveling instruction video after checking connection

screw in lightly design: https://app.zeplin.io/project/5aa97729db58a2192f10d0d6/screen/5e839ed0f792546d4c858d30
leveling instruction design: https://app.zeplin.io/project/5aa97729db58a2192f10d0d6/screen/5e839ed07319a9140ea11305
The videos will be dependent on the type of pipette and the mount; for now, use a dummy video.


Add cjs-based getCommands() and getHelpers()

This enables us and others to make use of some of the features directly in code rather than to go through the CLI interface

Giving it another look, createReport() is just a thin wrapper on top of socketSdk.createReportFromFilePaths(packagePaths, cwd)
The better solution is probably to move the getPackageFiles helper to the SDK and point everyone to the SDK 🤔

is this missing the index.cjs output?


Adding support for reading and writing to multiple tfrecords in nsl.tools.pack_nbrs

The current implementation of nsl.tools.pack_nbrs does not support reading and writing to multiple tfrecord files.
Given the extensive optimizations made available by the tf.data API when working with multiple tfrecords, supporting this would yield significant performance gain in distributed training. I would be willing to contribute to this
Relevant parts of the code

for reading
https://github.com/tensorflow/neural-structured-learning/blob/c21dad4feff187cdec041a564193ea7b619b8906/neural_structured_learning/tools/pack_nbrs.py#L63-L71
for writing
https://github.com/tensorflow/neural-structured-learning/blob/c21dad4feff187cdec041a564193ea7b619b8906/neural_structured_learning/tools/pack_nbrs.py#L264-L270

Thanks for your interest in contributing, @srihari-humbarwadi! :) To begin with, could you share more specifics on what APIs you plan to change and how you plan to change them? Once we come to an agreement there, you can go ahead with the implementation. I've assigned one of my colleagues, @aheydon-google to this thread, who will be able to work with you on this.

Here is the signature of the current implementation of  pack_nbrs ---
def pack_nbrs(labeled_examples_path,
              unlabeled_examples_path,
              graph_path,
              output_training_data_path,
              add_undirected_edges=False,
              max_nbrs=None,
              id_feature_name='id'):

labeled_examples_path and unlabeled_examples_path are paths to a single TFRecord file, one for each of them.
The proposed modification changes this to additionally support a list of paths. This would enable users to load examples that are split across multiple TFRecord 'shards'; which is often the case when training on multiple accelerators.
labeled_examples_path = 'train.tfrecord'  # current implementation support this.

# proposed modification adds support for the following as well
labeled_examples_path = [
    'train-0001-of-0004.tfrecord',
    'train-0002-of-0004.tfrecord',
    'train-0003-of-0004.tfrecord',
    'train-0004-of-0004.tfrecord'
    ]

The _read_tfrecord_examples  function (defined here) that reads examples currently from a single file would require minimal changes to support reading from multiple files.
https://github.com/tensorflow/neural-structured-learning/blob/c21dad4feff187cdec041a564193ea7b619b8906/neural_structured_learning/tools/pack_nbrs.py#L66-L68
This modified code would look something like this
for raw_record in tf.data.TFRecordDataset(filenames):  # filenames is list of tfrecord paths
  tf_example = parse_example(raw_record)
  result[get_id(tf_example)] = tf_example

For writing the newly generated examples, the current pack_nbrs implementation writes them into a single TFRecord at a path given by the output_training_data_path argument. The proposed modification adds an optional functionality to split the newly generated examples across multiple TFRecord shards. num_shards, a new argument in the pack_nbrs will control the number of TFRecord shards generated. This again would require minimal code addition, changing
https://github.com/tensorflow/neural-structured-learning/blob/c21dad4feff187cdec041a564193ea7b619b8906/neural_structured_learning/tools/pack_nbrs.py#L264-L266
to
writers = []
for i in range(num_shards):
  # there could be a better way to generate output TFRecord names
  output_path = '{}-{}-of-{}'.format(output_training_data_path, i, num_shards)
  writers.append(tf.io.TFRecordWriter(output_path))

for i, merged_ex in enumerate(_join_examples(seed_exs, nbr_exs, graph, max_nbrs)):
  writers[i % num_shards].write(merged_ex.SerializeToString())

#  close all writers
for writer in writers:
  writer.close()

Hi, Srihari.
Thanks for supplying those details and for offering to contribute to NSL! What you propose sounds like a nice improvement. If you send me a pull request with your proposed changes, I can review it.
Thanks!

Allan

Thank you @aheydon-google, I will start working on this!

Looking forward to this feature.
For bigger datasets packing all the examples into a single TFRecord file will introduce a substantial amount of bottleneck in the overall data pipeline.

Hi, Srihari. Do you have any updates to report on this issue? I think it could be quite useful! Thanks, - Allan

Hi again, Srihari. Are you still planning to work on this issue? I think it would be a great contribution if you can do it!
Thanks,

Allan

Since this issue has been dormant for quite some time, I'm going to close it. Feel free to send a pull request if you want to implement the improvement!

Closing the issue for now.


WIP: Add TLV support and general refactorings

Hi,
I was looking for proxyprotocol support for Golang where I could encapsulate a listener like what is possible with https://github.com/armon/go-proxyproto , except I needed to use version 2 with TLV.
Therefore I was happy to find your repo, and took the liberty to extend it a bit here and there. It's still a WIP, though I'm curious what you think about the (albeit also still-in-progress) API.
Dolf

Freeaqingme

I like the idea of the connection wrapping even though it seems out-of-scope for this library. However, don't like the fact the API is broken with this change.
It's been some time since I last picked on this so I'll need to take a few hours to review properly.

@pires are you planning on merging these changes? I am also interested in support for v1 and v2 with the Listener interface.

I'd like to merge it, yes but, unfortunately, I haven't had the time to properly review it. The fact it breaks the existing API is also doesn't help.

Sorry for closing, it was accidental.

Would be cool if you could resume the TLV support based on current master branch 🙇

Good job on #8 , was planning of reviewing it later, but you move fast :)
Right now I've got different priorities so can't tell when I'll revisit this (same goes for https://github.com/golang/go/issues/20956 ). Having said that, I do expect to be working on some HTTP projects later this year when I'll surely need something similar.
Though, if that takes too long, I by no way would take offence if someone else would continue this work.

Sorry, I should've waited indeed. I may be back to this PR and try and adapt your TLV work. Thank you very much!

Apologies for the drive-by I was looking into the package for a Vault issue I'm curious about. But how might this compare against the PR on elastic's fork here? It seems to have a similar goal of supporting TLVs in go-proxyproto.

@dekimsey I'd ask the author of that PR. Chances are he's seen this PR as well, and he can explain why he thought one way to go about it would be better than another.

Hey @Freeaqingme! Thank you for your involvement so far in helping w/ this library. Is there anything from this PR that you think is missing from the current tip?

Thank you very much for your contribution even if it didn't get merged.


20 graphql schema generation

I added a run configuration "Generate GraphQL" which generates the graphql in sdl format to the frontend folder, where it can be used to generate the queries.
I had to upgrade artemis to make this possible (I don't know, why we used such an old version, maybe @PaulaWi do you know why?)

I had to upgrade artemis to make this possible (I don't know, why we used such an old version, maybe @PaulaWi do you know why?)

No particular reason, the article I read just used this older version.


[TOPI] Use cblas for dense and batch_matmul when "cblas" is in the target libraries

Use cblas for dense and batch_matmul when "cblas" is in the target libraries. This is already supported for cublas and rocblas, it's just not plugged in for cblas. Also added overridden batch_matmul compute for x86.
@ajtulloch @yinghai would you be able to review?

@hlu1, since you have a similar version. Could you take a look?

batch_matmul doesn't have any tuning schedules defined right now. Also, this code path is only hit when you explicitly specify "cblas" in the libs. If the user sets that as their library, I feel like it's expected to use BLAS. I think we can do tuning of cblas vs TVM schedules in another PR.
Also, dense already has this support for cublas and rocblas. It's just missing for cblas.

Please fix CI issue.


⚠ Secret Website 2 has degraded performance

In 3925eae, Secret Website 2 ($SECRET_WEBSITE_2) experienced degraded performance:

HTTP code: 200
Response time: 8366 ms

Resolved: Secret Website 2 performance has improved in c8fd80e.


Fragment index is not working

I also tried to test fragment orders, but it did not work for me with the following examples. Am I not doing it correctly?
- Appears last <!-- .element: class="fragment" data-fragment-index="3" -->
- Appears first <!-- .element: class="fragment data-fragment-index='1'" -->
- Appears second <!-- .element: class="fragment data-fragment-index='2'" -->

Originally posted by @wenlzhang in https://github.com/MSzturc/obsidian-advanced-slides/issues/6#issuecomment-996078208

Currently it's not possible to add attributes through annotations. I will add this feature in the near future.

But it works well for me with this codes:
## Pointer Animation

Appear First  <!-- .element: class="fragment" data-fragment-index="1" -->

Appear Second  <!-- .element: class="fragment" data-fragment-index="2" -->

Appear Third  <!-- .element: class="fragment" data-fragment-index="3" -->

Appear Fourth  <!-- .element: class="fragment" data-fragment-index="4" -->

I can confirm that the following also works for me:
- Permanent item
- Appear Fourth <!-- .element: class="fragment" data-fragment-index="4" -->
- Appear Third <!-- .element: class="fragment" data-fragment-index="3" -->
- Appear Second <!-- .element: class="fragment" data-fragment-index="2" -->
- Appear First <!-- .element: class="fragment" data-fragment-index="1" -->

Ohh wow, guess i had a typo in my test.


Fix hare message validation

The original issue was that starting a new iteration causes the hare not to terminate.
The actual reason is a bug in the message validation.


Avoiding allocating a handle in the Windows RNG.

Issue #, if available:
Description of changes:
This PR uses the BCRYPT_USE_SYSTEM_PREFERRED_RNG option in the call to BCryptGenRandom to avoid allocating an algorithm handle, and simplifying the code a bit.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Can you please explain why you would want to avoid allocating an algorithm handle?

Can you please explain why you would want to avoid allocating an algorithm handle?

It reduces global state and slightly simplifies the code. This approach is being used by OpenSSL and .NET among others.

Thanks!


Update uralex_documentation.md

Updated the article referring to analysis of the loanword layers > from Manuscipt to the published article (De Heer et al. 2023)

Updating a reference for the Readme

Attempted format the "quote all text cells" to clean up the datasheet.


fix default value for RebootOk

From my observations RebootOk was always initialized to $false so there was no way to override this behaviour when calling Install-BoxstarterPackage.
See this discussion where the regression was introduced.
I added a test to prove this behaviour and suggested a fix.

The boxstarter rebooting guard is a boolean that lives in the top level $Boxstarter dictionary. The Boxstarter.Chocolatey module sets this to $true as soon as the module is loaded.
You should be able to see this just after calling Import-Module Boxstarter-Chocolatey:
C:\dev\windows [master +0 ~2 -0]> Import-Module Boxstarter.Chocolatey
C:\dev\windows [master +0 ~2 -0]> $Boxstarter

Name                           Value
----                           -----
IsRebooting                    False
NugetSources                   http://chocolatey.org/api/v2;http://www.myget...
BaseDir                        C:\Users\matt\AppData\Roaming\Boxstarter
LocalRepo                      C:\Users\matt\AppData\Roaming\Boxstarter\Buil...
RebootOk                       True
Log                            C:\Users\matt\AppData\Local\Boxstarter\boxsta...
SuppressLogging                False

In this repo is a test package that I run in a VM during integration tests and it reboots the machine a couple times.
If you find that boxstarter is telling you that reboots are not enabled and is not rebooting, it would be great if you could provide a gist of a test package that illustrates this and I could run in a VM.

If I explicitly import the Boxstarter-Chocolatey module first (via $env:AppData\Boxstarter\Boxstarter.Chocolatey\Boxstarter.Chocolatey.ps1), than the RebootOk is indeed set to true. If I only import the BoxstarterShell.ps1 or open the Boxstarter Shell via the shortcut on the desktop then RebootOk is set to false.
This behaviour was different with 2.6.25 where in al situations the RebootOk was already set to true.
Is it advised to load the Boxstarter-Chocolatey manually? Or should it kick in the process later on? Because from my observations it is never automatically set to true if not specified in the gist explicitly.
In this paste I show the behaviour of 2.6.25 first and the behavior of 2.8.29 later on. In this paste you can see the behaviour as you described above with manually loading the Boxstarter-Chocolatey module.

Thanks for pointing out that the shell shortcut opens with RebootOK set to $false. That is indeed a bug. Most boxstarter use cases involve the following:

Using the web launcher which imports the modules for you.
Maually importing the boxstarter.chocolatey module (which will load all other modules it needs)
Using the boxstarter shell.

The third route above is clearly doing it wrong and I can easily fix that.

When do you think we might see this fix to the shell?  When I launch the shell I still see RebootOk as false which was mighty confusing for me as to why my boxstarter packages started failing.  I now run $Boxstarter.RebootOk=$true in the shell before I run my package.
My workflow is to get a new machine, launch the shell and go from there.


Version 1.16 has lost support for the Alt-Gr characters on a German keyboard

After updating to version 1.16 I cannot type keys like @ or [] on the German keyboard that require the Alt-Gr. I think that worked in the previous version since it was introduced as a fix for #572

Curious. I'll see if I can reproduce this.

Fixed in v1.16.1.

thank you


Disable soft geocoding for users without block price set

Fixes #10507

@rafatower Can you take a quick look? This is a just-in-case fix, but I don't think it will actually be needed. I just added it as a precaution against free geocoding.
I am also fixing the affected users (manually since there are very few).

:+1:
Quick question: did you check that fresh users get a block price?

Out of the scope of the review but to take into account: the soft_limit attribute is used beyond rails app, so I'd advocate against putting logic in such checks cc/ @juanignaciosl

There were some users getting null block prices (from the price plans). I have fixed the price plans to avoid it.
Also, if the check is being used elsewhere, what if we remove this check and we put a validation in central that disallows setting soft geocoding limit with nil block prices? This way we ensure users are not created with invalid combinations of parameters, no check needed. I'd also pursue making these kind of attributes NOT NULL in the database, to avoid this kind of issue.

Ok, trashing this then, and moving the check to central. Thanks!


Missing instructions for contributors

The project is missing a document that would help to understand rules required by this community. Or, if such document is present somewhere, then it is difficult to find.

It is not clear who is the person or mailing list to contact before contributing to the project, are there any documents required to sign before the contribution, which kind of copyright headers do you require or accept or do you accept ad hoc pull requests.
It is not defined which testability criteria the main code should meet, where are interfaces required. Due Go having no virtual methods it is easy to get a code that is completely non Unit-testable.
It is unclear which types of Unit tests are required, which are optional and which would not be accepted at all that is a grave enough experience.
It must be clearly documented on which standard functions should not be used, and which replacements should be used instead (like So -> SoMsg in Unit tests, or maybe closures in the main code).
Any coding style rules that are still an issue after both code formatter and linting tools are applied must be explicitly listed and the less of them the better.
Ideally the document should also list some tasks the newcomer could take over. Even if no longer up to date, these may serve as examples.

See GNU Classpath or OpenJDK for the examples of such documents. The code review is not a replacement for them.

Fixed in #3744. Some bullet points (testability, interfaces) are best discussed depending on the issue the contributor is working on, but the documentation now contains details on how to reach out for feedback early in the contribution process.


Extended Stable Branch

https://www.ghacks.net/2021/03/05/google-decides-to-switch-to-a-4-week-release-cycle-for-chrome/
Chromium is somewhat difficult to rebase the patchset onto new versions, and upstream plans to speed up the release cadence. This makes it harder to keep track of potentially privacy-invasive features that ship.
It would be useful if there's support for Chromium's 8 week extended stable branch.

I believe we are coping with 4-week release cycle rather swift, as such there is no need for the Extended Stable presently.


keyboard shortcuts

Collection of keyboard shortcut ideas:
Online only
Desktop App only
Save (Ctrl-S)
Common
Open (Ctrl-O)
Load (Ctrl-L)
From #34 :

When I am working on an elaborate timing diagram, I'd much rather prefer to work on the local app so as to not lose my work. It would be awesome to have the capability of quickly save (ctrl-s) and load from file, even auto-save perhaps. Also being able to copy paste within the json window would be immensely useful.

I can add an open/load feature, if you'd like? They are one and the same in this application, though, so if you let me know which you'd like, I can add it :)

Hi Aliaksei, my manager was just wondering how the builds (releases of desktop application) of WaveDrom are updated on GitHub/how often? Thanks for all of your help.


Initial adaptations for Server API version 0.3.0

In order to prepare to transition to v0.3.0 of the Superstaq API, this PR adds a temporary _SuperstaqClient object (named _SuperstaqClient_v0_3_0 which adheres to the new API and a css.job._Job object to work with it (an equivelant update for qiskit-superstaq will follow). This also involves adding a gss._models module containing the send/receive data structures that are needed for the new API.
This PR does not affect the current operation of any existing parts of client superstaq. A warning has also been added to indicate to users that using v0.3.0 is not stable and that they should continue using v.0.2.0 for now.

From Dev meeting: Move models to a separate PR to be merged in so server can import.

Updated with testing for the general-superstaq. Still requires testing for qiskit-superstaq


Why does the tutorial not work? I

Why does the tutorial at https://webpack.github.io/docs/tutorials/getting-started/ not work? It never sees the style sheet and I fixed the css-loader issue?

Do you want to request a feature or report a bug?

What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.

https://webpack.github.io/docs/tutorials/getting-started/  => webpack v1 (legacy)
https://webpack.js.org => webpack v2 (recommended)

Here's another tutorial that does not work:
https://www.sitepoint.com/beginners-guide-to-webpack-2-and-module-bundling/.
It breaks at the point where lodash is installed and a javascript is
changed.
Undoubtedly terrible documentation for webpack. Should you ever find a
tutorial that works, please publish that information.
Here at Disney we are passing on webpack.
On Mon, Apr 10, 2017 at 11:47 PM, Tobias Koppers<EMAIL_ADDRESS>wrote:

Closed #4689 https://github.com/webpack/webpack/issues/4689.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/webpack/webpack/issues/4689#event-1037716508, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZ-JbQBT5D-jM4F5ZlvZML9qKwpftHpmks5ruyHugaJpZM4M5Y9U
.


Remove version: Ziad.TCG version 6.6

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: URL works;


Current changelog process requires creating PR and then adding changelog file

We're asking each non-trivial PR to have a changelog file in the format <pr #>-.  The PR # isn't known until the PR is created (it can be guessed, but this is obviously not ideal).  So in the normal case, this always requires a minimum of two commits/pushes.
It'd be nice to simplify this.  I haven't spent any time yet thinking about possible alternatives yet, but hoping to collect input here.
The reason we're not using issue # is that not every PR has a corresponding issue.  We could require that an issue exist, but this adds overhead to small contributions.

Hi, @skriss I ended up here because of this tweet :)
Anyway, I want to share something that we made at GEUT, it's a tool called chan that maybe can result helpful to you. It was made to help developers manipulate and maintain changelogs, following "keep a changelog" style, i.e.: human-readable changelogs. In companion with chan, we have a github app, called chaas which can be used to check if a changelog has changed in the current PR.
We are working on new features for chaas, like: changelog message suggestion and a next version helper (for releases mostly).
If you want to give it a go, feel free to do it. It should adapt very easily to any workflow. Issues/features/ideas are super welcome too. 👍

Closing as inactive, we may revisit down the road.


[resotocore][feat] Add a csv output option to the list command

Description

Added a new --csv flag for the list command. Once specified, query will result in CSV formatted output.
To-Dos

[x] Add test coverage for new or updated functionality
[x] Lint and test with tox
[ ] Document new or updated functionality (someengineering/resoto.com#XXXX)

Issues Fixed

Fixes #653

Code of Conduct
By submitting this pull request, I agree to follow the code of conduct.

Codecov Report

Merging #672 (3e0c534) into main (2d3b380) will increase coverage by 0.15%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #672      +/-   ##
==========================================
+ Coverage   89.01%   89.16%   +0.15%
==========================================
  Files          64       64
  Lines        8606     8670      +64
==========================================
+ Hits         7661     7731      +70
+ Misses        945      939       -6

Flag
Coverage Δ

resotocore
89.16% <100.00%> (+0.15%)
:arrow_up:

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

Impacted Files
Coverage Δ

resotocore/resotocore/cli/command.py
86.76% <100.00%> (+0.79%)
:arrow_up:

resotocore/resotocore/cli/__init__.py
92.68% <0.00%> (+2.43%)
:arrow_up:

resotocore/resotocore/web/content_renderer.py
93.61% <0.00%> (+3.54%)
:arrow_up:

Continue to review full report at Codecov.

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


Updating List Synonyms from a json file

Hey guys,
I would like to update the synonyms of a "list entity", but it seems that when I try to upload the file it wants me to create a totally new entity.  So I get this error:
"BadArgument: List entity "first_name" cannot contain duplicate canonical forms: (first_name)."
How do I add to the synonyms of an already pre-existing list type entity via file upload?
Thanks

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

ID: 3578db8f-e411-b0e6-4990-dd3c49f6a41f
Version Independent ID: 21e19094-e487-a942-13f6-4fc1f0129feb
Content: Exact text match - Azure Cognitive Services
Content Source: articles/cognitive-services/LUIS/luis-quickstart-intent-and-list-entity.md
Service: cognitive-services
GitHub Login: @diberry
Microsoft Alias: diberry

@Gariantroll Thank you for asking this question. You won't be able to upload a file from the LUIS portal. You can add the information through the APIs (either REST-based or SDK). The quickest way is to use the APIM console. Fill out the parameters and change the request body.
Patch entire list
Update sublist
#in-progress

Hey, I am testing it out and it seems I am getting this response.  "Message": "The request is invalid."
I can't exactly figure out what is going wrong here. any ideas?
Thanks

@Gariantroll My guess is the JSON is invalid somewhere with either encoding from the text box to true HTTP encoding or string quotes are off somewhere. Can you please open F12/devtools on the browser, use the console again to send the request, then when you get the error, save the Har file and the console log and send them to me (diberry@microsoft.com)?

I sent the file, but I changed the query to a simply changing the name of an entity since the error was the same.
Thanks

@Gariantroll It appears that you are using the entity name instead of the entity id. The following image shows where to get the entity id.

Ahhh, could I be any more oblivious? lol.  Where do I get the sublist ID?

I am receiving this error which I think is associated with the sublistID value.
"error": {
"code": "BadArgument",
"message": "Failed to parse closed list extractor update object. Parameter name: wordListBaseUpdateObject"
Any ideas?
Alexis

Hey guys,
I figured out how to get the sublist and is a bit tricky because I couldn't initially find it in the Developers Tools.  To specifically find your sublistID you have to go to DevelopersTools->NetworkTab-> Click on one of the Recommended Synonyms.  You should see the sublist popup in the Name queue.
Thanks for your help.  Can't thank you enough.
Alexis

Thanks,
I found finding the sublist a little tricky.  Here is a quick rundown.    A sublist should pop-up in the Name column in the Network tab -> Click on the sub list -> you should see the list ID from there.
Thanks for everything!!!!

@Gariantroll - thank you for posting your answer. #please-close

@Gariantroll We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Use head block instead of safe block for snap sync

Change snap sync to use head block instead of safe block for the snap sync.
Using the head allows for larger download windows between two pivot blocks. With larger windows, we will have fewer pivot blocks and therefore more time for the heal. This could allow us to finish faster. If we download the storage of an account and then change the pivot block, we need to request the account again to get the new storage root of the account in the block, as this can cause a lot of storage requests to fail.
Acceptance Criteria

Verify that this has the improvement on healing and syncing that we are expecting

take a look at PivotBlockSelector

Adding some color to the acceptance criteria, I would suggest a couple ways to quantify the benefit of head versus safe snap-sync pivot blocks

we can just empirically count the number of re-pivots we have when syncing a large network like mainnet, sepolia or holesky.  This misses some of the point about whether this represents a performance improvement or not, but it is a good metric to capture.  For PoS networks, the logic that controls pivot selection can be found and instrumented here

there are two snap sync stages that should in theory benefit from fewer and longer-lasting re-pivots.  Those are the worldstate download and the worldstate heal stages.  Again as long as we are syncing a large enough network over a long enough time period, we can just observe the time it takes for each stage to complete when we are using head versus safe for the pivot block, on separate besu instances.

I made changes to sync to head block and added a configuration option to revert to syncing to safe if needed: Xsnapsync-to-head-enabled=false
To test I ran 2 nodes at the same time one syncing to head and the other to safe.  As expected there was a drastic reduction in the number of pivots: anywhere from 11 to 50% decrease. However the actual duration to sync was minimal at just 2% decrease in time. One run was even slower but perhaps something else was going on.


ADOT Collector message for writing to Prometheus remotely.

After launching ADOT Collector with sudo /opt/aws/aws-otel-collector/bin/aws-otel-collector-ctl -c $HOME/adot-config.yaml -a start, the log messages seen are as follows.
tail -f /opt/aws/aws-otel-collector/logs/aws-otel-collector.log
{"level":"info","timestamp":"2022-09-07T03:49:23.999Z","caller":"pipelines/pipelines.go:78","message":"Exporter is starting...","kind":"exporter","data_type":"metrics","name":"prometheusremotewrite"}
{"level":"info","timestamp":"2022-09-07T03:49:23.999Z","caller":"pipelines/pipelines.go:82","message":"Exporter started.","kind":"exporter","data_type":"metrics","name":"prometheusremotewrite"}
{"level":"info","timestamp":"2022-09-07T03:49:23.999Z","caller":"pipelines/pipelines.go:86","message":"Starting processors..."}
{"level":"info","timestamp":"2022-09-07T03:49:23.999Z","caller":"pipelines/pipelines.go:98","message":"Starting receivers..."}
{"level":"info","timestamp":"2022-09-07T03:49:23.999Z","caller":"pipelines/pipelines.go:102","message":"Receiver is starting...","kind":"receiver","name":"prometheus","pipeline":"metrics"}
{"level":"debug","timestamp":"2022-09-07T03:49:24.000Z","caller":"discovery/manager.go:289","message":"Starting provider","kind":"receiver","name":"prometheus","pipeline":"metrics","provider":"static/0","subs":"map[onpremises_exporter:{}]"}
{"level":"debug","timestamp":"2022-09-07T03:49:24.001Z","caller":"discovery/manager.go:323","message":"Discoverer channel closed","kind":"receiver","name":"prometheus","pipeline":"metrics","provider":"static/0"}
{"level":"info","timestamp":"2022-09-07T03:49:24.002Z","caller":"pipelines/pipelines.go:106","message":"Receiver started.","kind":"receiver","name":"prometheus","pipeline":"metrics"}
{"level":"info","timestamp":"2022-09-07T03:49:24.003Z","caller":"service/collector.go:215","message":"Starting aws-otel-collector...","Version":"v0.21.0","NumCPU":1}
{"level":"info","timestamp":"2022-09-07T03:49:24.003Z","caller":"service/collector.go:128","message":"Everything is ready. Begin running and processing data."}

Can you please elaborate what does Everything is ready. mean? Is ADOT Collector able to read from receiver and write to exporter successfully? In my case, I am writing to AMP; however, the awscurl does not report any metrics. I am not sure where to start debugging. This is my configuration file.
receivers:
  prometheus:
    config:
      global:
        scrape_interval: 15s
        external_labels:
          monitor: 'onpremises_exporter'

      scrape_configs:
        - job_name: 'onpremises_exporter'
          static_configs:
            - targets: ['localhost:9103']

extensions:
  sigv4auth:
    service: "aps"
    region: "ap-southeast-1"

exporters:
  prometheusremotewrite:
    endpoint: "https://aps-workspaces.ap-southeast-1.amazonaws.com/workspaces/workspaceId/api/v1/remote_write"
    auth:
      authenticator: sigv4auth

service:
  extensions: [sigv4auth]
  pipelines:
    metrics:
      receivers: [prometheus]
      exporters: [prometheusremotewrite]

Can you please elaborate what does Everything is ready. mean?

It means that you have no mistakes in your collector config and the pipeline you defined is up and running (in your case the one you named metrics.

Is ADOT Collector able to read from receiver and write to exporter successfully?

For that to answer we would need to see more of the collector logs, you only shared the start-up sequence.

In my case, I am writing to AMP; however, the awscurl does not report any metrics. I am not sure where to start debugging

First check if the receiver works as expected (do metrics arrive in the collector?) and the exporter as well. You can do that be enabling DEBUG level logs:
 exporters:
      logging:
        loglevel: debug
...
    metrics:
      receivers: [prometheus]
      exporters: [logging, prometheusremotewrite]

With above you should see the DEBUG level collector logs on stdout.
In my experience troubleshooting OpenTelemetry collectors it's often the case that IAM settings are lacking or misconfigured. In your case, for example, did you check if the machine you're running the ADOT collector on has an IAM role attached?

It means that you have no mistakes in your collector config and the pipeline you defined is up and runnin

I don't intend to split hairs on the language. However, it would have been more intuitive to simply report Valid configuration found. since, there are log messages already available that report the starting of receiver and exporter.

You can do that be enabling DEBUG level logs:

I will enable logging and report back.

did you check if the machine you're running the ADOT collector on has an IAM role attached?

No. I wanted to see the logs of a "mis-configured" instance. (It is a part of a guide/tutorial I am doing.)

I terminated my EC2 instance and started afresh.
I don't specify an IAM role; I had the credentials (aws configure) instead. So, collector was able to write to AMP remotely. If neither the role nor the credentials were applied, I can see the error message. This is valid behavior.
I was hoping for some indication of writing to AMP as is seen in case of Prometheus server. Example below.
...
ts=2022-07-24T22:43:13.588Z caller=main.go:958 level=info msg="TSDB started"
ts=2022-07-24T22:43:13.588Z caller=main.go:1139 level=info msg="Loading configuration file" filename=prometheus.yaml
ts=2022-07-24T22:43:13.596Z caller=dedupe.go:112 component=remote level=info remote_name=2156e9 url=https://aps-workspaces.ap-southeast-1.amazonaws.com/workspaces/ws-191a9a06-efa9-4441-bff3-65d6a9372fcc/api/v1/remote_write msg="Starting WAL watcher" queue=2156e9
ts=2022-07-24T22:43:13.596Z caller=dedupe.go:112 component=remote level=info remote_name=2156e9 url=https://aps-workspaces.ap-southeast-1.amazonaws.com/workspaces/ws-191a9a06-efa9-4441-bff3-65d6a9372fcc/api/v1/remote_write msg="Starting scraped metadata watcher"
ts=2022-07-24T22:43:13.596Z caller=dedupe.go:112 component=remote level=info remote_name=2156e9 url=https://aps-workspaces.ap-southeast-1.amazonaws.com/workspaces/ws-191a9a06-efa9-4441-bff3-65d6a9372fcc/api/v1/remote_write msg="Replaying WAL" queue=2156e9
ts=2022-07-24T22:43:13.601Z caller=main.go:1176 level=info msg="Completed loading of configuration file" filename=prometheus.yaml totalDuration=13.011172ms db_storage=1.584µs remote_storage=7.480765ms web_handler=1.495µs query_engine=1.808µs scrape=5.116067ms scrape_sd=46.547µs notify=2.442µs notify_sd=3.899µs rules=2.59µs tracing=10.092µs
ts=2022-07-24T22:43:13.601Z caller=main.go:907 level=info msg="Server is ready to receive web requests."

Could you provide the debug logs from the collector as requested earlier? This will help identify whether the issue lies in the receiver or the exporter.


[url_launcher_android] Issue regarding REQUEST_INSTALL_PACKAGES permission after uploading the application to PlayStore

Steps to Reproduce

Upload an application using the package url_launcher/url_launcher_android on the PlayStore
Expected results: Uploading my application without a Google alert
Actual results: I received an alert saying that my application is not compliant with one or more of Google Developer Program Policies.
The reason is that my application seems to use REQUEST_INSTALL_PACKAGES permission. I checked my AndroidManifest, I never use this permission.
I searched online how to find where in my application permissions are used even by me or other packages, and I found this:

in the file "/My_Flutter_App/build/app/outputs/logs/manifest-merger-release-report.txt".
I already tried to fill their form trying to explain where in my application this permission is used, in vain. They sent me this instead:

and I cannot remove it from my manifest of course.
Does anyone know what to do in this situation?

[✓] Flutter (Channel stable, 3.3.0, on macOS 12.6 21G115 darwin-x64, locale fr-QA)
• Flutter version 3.3.0 on channel stable at /Users/.../flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision ffccd96b62 (3 weeks ago), 2022-08-29 17:28:57 -0700
• Engine revision 5e9e0e0aa8
• Dart version 2.18.0
• DevTools version 2.15.0
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0-rc3)
• Android SDK at /Users/.../Library/Android/sdk
• Platform android-33, build-tools 33.0.0-rc3
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 14.0)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 14A309
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.2)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
[✓] VS Code (version 1.71.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.48.0
[✓] Connected device (2 available)
• macOS (desktop) • macos  • darwin-x64     • macOS 12.6 21G115 darwin-x64
• Chrome (web)    • chrome • web-javascript • Google Chrome 105.0.5195.125
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!

Hi @tel-kho, please provide the versions of url_launcher and url_launcher_android that you are using. You can find the version of url_launcher_android in your pubspec.lock file.
Thank you

Hi @tel-kho, please provide the versions of url_launcher and url_launcher_android that you are using. You can find the version of url_launcher_android in your pubspec.lock file.
Thank you

Hello, yes sure:
url_launcher:
dependency: "direct main"
description:
name: url_launcher
url: "https://pub.dartlang.org"
source: hosted
version: "6.1.5"
url_launcher_android:
dependency: transitive
description:
name: url_launcher_android
url: "https://pub.dartlang.org"
source: hosted
version: "6.0.19"

Hi @tel-kho, if you create a new project and add the url_launcher plugin, do you see the REQUEST_INSTALL_PACKAGES permission in your merged manifest? You can see this in Android Studio when you open AndroidManifest.xml and click on the Merged Manifests tab.
I did this with a new project and I do not see that permission. I also do not see that permission used in the source for url_launcher or url_launcher_android.

I searched online how to find where in my application permissions are used even by me or other packages, and I found this:

in the file "/My_Flutter_App/build/app/outputs/logs/manifest-merger-release-report.txt".

That file has the opposite order of how you are reading it; the permission (or other entry) comes first, and then the description of where it comes from. The two lines you've shown are the end of a previous section that you've trimmed out, which would say what is being included from url_launcher_android, and then the start of a new section that would tell you where REQUEST_INSTALL_PACKAGES came from (which you've also trimmed out). You need to look at the part below that excerpt to see what the source is.
Closing as this isn't coming from url_launcher, or anywhere else in the plugins repository, so isn't a Flutter bug.

I searched online how to find where in my application permissions are used even by me or other packages, and I found this:

in the file "/My_Flutter_App/build/app/outputs/logs/manifest-merger-release-report.txt".

That file has the opposite order of how you are reading it; the permission (or other entry) comes first, and then the description of where it comes from. The two lines you've shown are the end of a previous section that you've trimmed out, which would say what is being included from url_launcher_android, and then the start of a new section that would tell you where REQUEST_INSTALL_PACKAGES came from (which you've also trimmed out). You need to look at the part below that excerpt to see what the source is.
Closing as this isn't coming from url_launcher, or anywhere else in the plugins repository, so isn't a Flutter bug.

Yes, you are totally right, my bad the problematic permission is from another package: open_file
I made some research and indeed there is already an issue made about it and another linked to the package itself.
Thank you all for your time


Coverity scan

Run coverity scan for PX4. It is rather heavy process so this is ran on self-hosted PC

tiiuae-coverity-scan-image.yaml creates coverity scan container (ran manually as needed)
tiiuae-coverity-scan.yaml runs the actual scan

Here is an example of coverity scan output: https://github.com/tiiuae/px4-firmware/actions/runs/5722447503
The results are visible with more details also in coverity.ssrc.fi


How to access cells of Segmentio's collectionview?

Goal:
I want to dynamically add badges to some cells of Segmentio's collectionview
How do I access Segmentio's collectionview and cells to modify the UIImageView?

Hi @BabyAzerty! @PravdaEvgen has added the feature. Please update and check it out


Null frames hello realsense C++ : "null pointer passsed for argument "frame_ref"

Required Info

Camera Model
{ T265 }

Firmware Version
<IP_ADDRESS>1

Operating System & Version
Linux (Ubuntu 18.04LTS)

Kernel Version (Linux Only)
4.9.253-tegra

Platform
NVIDIA Jetson Nano.

SDK Version
latest default

Language
C++

Segment
Robot

Issue Description
Not able to get the C++ examples for the T265 camera working. Frames retrieved are null.
Steps:

Downloaded newest version of Jetson Nano operating system from Nvidia site as of 21.09.28
Followed https://dev.intelrealsense.com/docs/nvidia-jetson-tx2-installation as  of 21.09.28
realsense-viewer works, shows 3D tracking and 2 fisheye camera views in BW
Attempted to get the example: https://dev.intelrealsense.com/docs/rs-hello-realsense working
compiles fine using the command: g++ -std=c++11 hellors.cpp -lrealsense2  -o helloex
Execution fails with the message: "null pointer passed for argument "frame_ref"". This message can be traced to the depth frame depth being returned as 0, NULL.
Tried to get around this by using rs2::config cfg;  cfg.enable_all_streams();  p.start(cfg); However this still does not fix the problem.

Thanks in advance ^_^

RealSense error calling rs2_get_frame_width(frame_ref:nullptr):
null pointer passed for argument "frame_ref"
also occurs when running the installed binary (in /usr/bin) $rs-hello-realsense

Semi-solved this issue.
The  depth doesn't work because the  T265 camera doesn't output depth data, the OpenCV example also doesn't  work because the camera also does not output color.
It be nice if there were some elementary examples of what the T265 actually does output. Which would be grayscale image/images & position data.

Closing this out.
Fixed the issue. The code for running C++ test of the pose and image functions of the T265 is available here:
https://github.com/BessPtck/RSCam
hellors.cpp just uses the  realsense SDK and outputs pose (camera x,y,z location) data.
rsImg.cpp uses the camera with openCV to output the grayscale image created by  one of the lenses to screen.


Memory telemetry

Memory Telemetry

feat: resolve #148 - MemTel: Implement the memory usage Telemetry auto-collection based on the config options.
feat: resolve #147 - MemTel: Implement necessary Telemetry auto-collection options config settings with the first available option being the memory usage collection.
feat: resolve #146 - MemTel: Define custom data fields for a Manual Telemetry event to keep the collected data. Implement helpers to manage that data.
test: resolve #148 - MemTel: Add unit tests.

Type of change

[ ] Bug fix (non-breaking change that fixes an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Maintenance
[x] New release

Related issues

Shortcut stories and GitHub issues (delete irrelevant)

Feat #146
Feat #147
Feat #148

Checklists
Development

[x] Lint rules pass locally
[x] The code changed/added as part of this pull request has been covered with tests
[x] All tests related to the changed code pass in development

Code review

[ ] This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[ ] "Ready for review" label attached to the PR and reviewers assigned
[ ] Issue from task tracker has a link to this pull request
[ ] Changes have been reviewed by at least one other engineer

Looks awesome! Only concern is related to how to structure memory data, ie. as typed code structure or as a dynamic data map.

@matux , thanks for the review! Correct, for now, we are not using any custom typed data elements for memory capture and reusing a Manual telemetry event that is basically a string-keyed dictionary of object values. Once, the backend is ready to ingest a new custom type-safe Telemetry event type dedicated to memory stats, we will change it that way.

Had convo with @akornich, we're good to go!


Inconsistent naming (win/pane)

Functions take a win (window) argument but the documentation refers to windows as panes, and the window id returned is "pane_UID".
It's inconsistent and confusing.

Good point. I would propose to adapt the phrasing in the documentation (rather than changing the win name, which would be a breaking code change).


iOS homescreen option

Add some code in the html so Reacticz can be added to the homescreen on iOS hiding the Safari interface

Thanks!

Yeah, your icon has indeed a much better resolution :-)


Proposal: Link should provide an API to get info about its current state

What problem does this feature solve?
With info about Links active state you can trigger animations or something else.
Please see this issue from someone else:
https://github.com/vuejs/vue-router/issues/774
https://forum.vuejs.org/t/get-active-state-of-router-link-using-javascript/1462
and his gist: https://gist.github.com/aplr/b49b74103a5c4adba57379cc7ce932da
My use case is this component from vue-mdc-adapter
https://github.com/stasson/vue-mdc-adapter/blob/master/components/tabs/mdc-tab.vue
It is using https://github.com/material-components/material-components-web for rendering and I need to trigger foundation.setActive() from mdc when the underlying router-link changes to active
What does the proposed API look like?
An event on active state change and a data var or computed var for active and exactActive

Hello! Watching the current route or using afterEach is the actual solution. Firing an event may work now but it will not work when we move router-link to a functional component.
Your use case is very specific so you'll have to handle it be calling setActive on the right component whenever necessary. But don't worry, using a watcher or afterEach should be straightforward, even with an event you still will have to add references or keep track of the elements associated with a route 🙂

Thank you for your answer :) watching current route is ok. But i don't know how to comparing to path in router-link.
In router-link there are functions isSameRoute and isIncludedRoute. But they seems to be private. Is it possible to get Access to that functions?
But if my use case realy specific I will copy these functions in my code.

You can use router.resolve

Exposing utilities like isSameRoute() and isIcludedRoute() is necessary as comparing route is not trivial.

Exposing utilities like isSameRoute() and isIcludedRoute() is necessary as comparing route is not trivial.


Paypal Button renders then immediately disappears

I installed the paypal express plugin for my woocommerce site:
You can reproduce it by following that url and quickly scrolling down to the "add to cart" button. The paypal buttons appear and disappear just below it in less than a second, once the page is finished loading.
I believe this might be the same issue as #416
However, I haven't been able to gather any information using the break on method that is suggested there.
If i could isolate the function that is causing the button to disappear, I would be able to prevent it from occuring.
It'd be a big help if you could help me debug this

Hey @laphilia, please see this issue: https://github.com/paypal/paypal-checkout/issues/1006.
@tomasmd since we haven't heard back in a while, I'm going to close this issue out. If you have any updates on your side that need our eyes, feel free to open up a new ticket :).


Docs: styling

Maybe a small section in the example page about how to style it?
🤷♂

https://devlop.github.io/dialog/

Great idea! I have updated the docs.
https://devlop.github.io/dialog/

🙌


Bel generates weird output with void html elements and unquoted attributes

In the HTML spec, some elements are void and don't have closing tags:

Void elements
area, base, br, col, embed, hr, img, input, keygen, link, menuitem, meta, param, source, track, wbr

But in bel, they cause weird problems:
const bel = require('bel')
const element = bel`<span><input type=checkbox>some content</span>`
console.log(element.outerHTML)

It should be valid, but for some reason the content gets dropped from the output. It generates:
<input type="checkbox">

Weirdly if I put quotes around the attribute (<input type="checkbox">) then it works. It also works if I put in a spurious /> - ie, <input type=checkbox />

This has been a known issue for a while now; we should definitely fix this!
I think the reason this hasn't been fixed so far is b/c workarounds are
quite
straight forward, and people have been constrained on time lately (e.g.
building out an ecosystem of plain HTML modules). A PR to fix this would
be super helpful; and I'd be thrilled to assist in getting it landed (and
I'm sure
I wouldn't be the only one).
Hope this somewhat makes sense; I agree it'd be heaps neat if this could
be solved! :D
On Mon, Oct 2, 2017 at 11:13 PM Seph Gentle<EMAIL_ADDRESS>wrote:

In the HTML spec
https://www.w3.org/TR/html51/syntax.html#writing-html-documents-elements,
some elements are void and don't have closing tags:
Void elements
area, base, br, col, embed, hr, img, input, keygen, link, menuitem, meta,
param, source, track, wbr
But in bel, they cause weird problems:
const bel = require('bel')const element = bel<input type=checkbox>some contentconsole.log(element.outerHTML)
It should be valid, but for some reason the content gets dropped from the
output. It generates:

Weirdly if I put quotes around the attribute ()
then it works. It also works if I put in a spurious /> - ie,
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/shama/bel/issues/94, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWleuxyrZ8p7p6DlP7KsV9459481E92ks5soaZZgaJpZM4Prhzx
.

It looks like its actually a bug in hyperx and I've filed an issue there.

I'll see if I can find it and make a PR. Its too annoying.

https://github.com/choojs/hyperx/pull/56

<EMAIL_ADDRESS>landed which should close the self-closing tags; not sure if this also fixes the unquoted string situation?

Oh yeah, I should have closed this ages ago. Thanks!


fix: use import instead of module in package exports

Using "module" as a conditional export (package.json#exports) is non-standard. According to the node documentation, the correct key here is import.
This also upgrades resolve.exports from v1.1.0 to v2.0.2.

Thanks for the PR @emonadeo. We've had the intention to update Vanilla Extract's dev and build tooling for a while now. We'll likely update the outdated export conditions alongside that change, rather than in isolation.
I would be happy to accept just the resolve.exports change however.


ISM/IC issue

Quick stubby for me to chase down whether this is a policy change issue or a P2GO glitch

@sjm41

Emailed Alex 12th as seems to be a P2GO issue

emailed Alex on 30th Aug: See that there has been some resolution to the IC evidence code from ECO
Not seeing it filtering through to P2GO yet, but I expect that will take some time. Just checking that this will fix the error we see with IC withs e.g. PMID:12093363

done


Allow configuring accept-encoding when proxying

It can be convenient, when proxying or recording, to limit the encodings the target server can use when responding regardless of what the client can support.
This allows WireMock to capture the response traffic in a form it can understand.
Submitter checklist

[X] Recommended: Join WireMock Slack to get any help in #help-contributing or a project-specific channel like #wiremock-java
[X] The PR request is well described and justified, including the body and the references
[X] The PR title represents the desired changelog entry
[X] The repository's code style is followed (see the contributing guide)
[X] Test coverage that demonstrates that the change works as expected
[x] For new features, there's necessary documentation in this pull request or in a subsequent PR to wiremock.org

Documented in https://github.com/wiremock/wiremock.org/pull/269


Fix minor issues with riak implementation of query.

drop 2 to remove leading q= is a little hacky, but works for now.
fl arguments are just messy so we now ignore them. What's a little inefficiency between 10GbE-connected servers?

After https://github.com/markhibberd/riak-haskell-client/pull/72 lands, fl queries will work as expected. So, I don't think the right solution is to just remove fl entirely!
And re: drop 2, I think a better fix is to expose some more building-blocks style compile functions in solr-query, as I've done:
-- compiles only a query, and does not prepend "q="
compileQuery :: Query -> Builder

@xnomagichash I made the aforementioned changes to your branch. How's that look?

It looks fine, but when will riak-haskell-client#72 land? We stop-gap until then.

Not sure, but it doesn't matter because we're depending on my fork with the fix.

Ah good point!


Add scrollable alignment option

This PR adds a new Alignment option to both directions of the scrollable.
It's sometimes preferred to have the content aligned to the end of the scrollable by default, where the offsets are based on the end of the scrollable. New content getting added will cause the scrollable to expand up instead of down. Alignment::End enables this behavior.

I have also made the fields in Viewport private again, since I'm not sure which use cases warrant exposing those. Hopefully we can tackle them directly!

@hecrj I was using the bounds & content_bounds fields for calculating the new absolute offset needed to keep me anchored in the correct location when toggling between start / end alignments.
This is needed in a chat application with end alignment by default. When scrolling up from the bottom, we convert to start alignment so new messages don't change the position of the viewport. When swapping alignments, we need to issue scroll_to w/ an absolute offset to achieve this effect. Those bounds are necessary for this computation.
See https://github.com/squidowl/halloy/blob/caa024de2962d9646d62c34bcf565b1c7e9056b8/src/buffer/scroll_view.rs#L298-L305 for how we do this.
So I guess a direct use-case solution might be to expose a swap_alignment type method on Viewport which will return a new AbsoluteOffset which can be returned in scroll_to to keep the viewport stationary w/ the new alignment.

@tarkah I think we can expose a translation method that returns a Vector in Viewport to satisfy that use case. Would that work?

@tarkah I think we can expose a translation method that returns a Vector in Viewport to satisfy that use case. Would that work?

I don't believe this works because translation is "normalized" but scroll_to / AbsoluteOffset is relative to the Alignment.
Since Offset doesn't encode which Alignment it's for, we would need to store Alignment in Viewport. Then maybe the correct API is to just have with_<direction>_alignment methods that can return a new Viewport transformed for that Alignment and then caller can get it's absolute_offset for use w/ scroll_to.

@tarkah In the snippet you shared, it seems the only thing we'd need to expose is (viewport.content_bounds.height - viewport.bounds.height).max(0.0). This is the vertical translation.
It seems you are computing the alignment from the Status itself, which makes sense since the alignment is driven by your application logic and not the scrollable itself. Adding alignments to Viewport seems a bit redundant and unnecessary.
Am I missing something?

it seems the only thing we'd need to expose is (viewport.content_bounds.height - viewport.bounds.height).max(0.0). This is the vertical translation.

Having that exposed is all I need. However, this isn't the scroll translation, it's the max scroll height.
When flipping between alignments, I take max scroll height minus current absolute offset to match the change in alignment.

This is the vertical translation.

It's not, my bad! I think a simple absolute_offset_reversed method would work.


Issue #1090 Reduce log level to debug

Includes some logging related code cleanups.

Wow, managed to botch the branch origin ... will re-open a new one once fixed.

I think you're going to need to make changes to logging in offheap-store otherwise you might find all your good work undone when using offheap.

Can we just use the {} mechanism everywhere? Sometimes it's used, sometimes not.

Opened PRs to fix the remarks made here.


Update dependencies

Update lodash and inquirer

can we have this released? would be cool to have the new inquier


How to set a start time for a song?

Hello, I have this range as my song duration slider:
<input type="range" id="song-time-slider" class="amplitude-song-slider no-drag" amplitude-singular-song-slider="true" value="0">

And I'm trying to do something like:
$('#song-time-slider').val('17').trigger('input') // 17%

But it dosen't works. Could someone help me here?
Thank you!

Hey guys, any news here? I just patched a feature to allow set the start time for a song in #54. It has been a month after I did this PR and there's no updates from you guys there.

Sorry for not getting back to you sooner, it's been really busy for me the past 12 months or so. I was able to address this issue in Amplitude 3.0. If you do a skip to link, documented here: https://521dimensions.com/open-source/amplitudejs/docs#skip-to-link it should address the issue.


[WIP] Add a cache busting string generator

This PR aims to make the generation of the cache busting string configurable with maximum flexibility (that a config can't quite provide).
It introduces a CacheBusterInterface that any CacheBuster can implement with a single method that should return the cache busting string.
Easy I guess.
Will talk with @promatik to know if he foresees any downside, but I think we are good.

Any news on when this is going to be released?


Prevent vectors.normalize from returning NaN on vectors with zero length

Have vectors.normalize return 0,0,0 if the vector length is 0

Thanks for this PR, good catch!


Added optional delay between repeats.  -MW

This is sometimes useful for GST experiements.  Any interest in adding this to develop?

it seems a good option to have


Custom emailEntryViewController not showing custom view

I have my project set up just like the sample project provided. I have used the work-around (stated in https://github.com/firebase/FirebaseUI-iOS/issues/128 ) to get the Custom authPickerViewController to successfully show. When I tap the Sign in with email button, it brings me to the default email entry view controller, rather than the custom i have set up.
The below function seems to never trigger, but the similar authPickerViewController function does trigger.
func emailEntryViewController(forAuthUI authUI: FUIAuth) -> FUIEmailEntryViewController {
    return CustomEmailEntryVC(authUI: authUI)
}

@ryandailey100 Have you assigned authUI.delegate?
Please check this sample.

Yes I have. Like i said, the authPickerViewController function gets successfully called from the delegate, but the emailEntryViewController doesn't get called when it should.

Hm... let's go over check list:

Are you using exactly this method (it's different method signature than you specified). Is next method was called?

func emailEntryViewController(forAuthUI authUI: FUIAuth) -> FUIEmailEntryViewController {
    return FUICustomEmailEntryViewController(nibName: "FUICustomEmailEntryViewController",
                                             bundle: Bundle.main,
                                             authUI: authUI)
  }

How are you presenting authViewController? You should use:

      UIViewController *controller = [self.authUI authViewController];
      [self presentViewController:controller animated:YES completion:nil];

Do you retain AuthUI.delegate? This property is weak, so you should take care of it's ownership.
I recommend analyze code where we present email controller (assuming you're presenting AuthPicker which has 'sign in with email' button).
Can you provide error? Console logs? If no messages, can you send link to your repo?

Okay thanks! It looks like i didn't retain AuthUI.delegate properly. Thanks for the help!


Duplicating efforts

Why duplicate efforts if there is https://github.com/kuba/simp_le? I think we pretty much have the same goal...

Always more room for more clients :) The goal of this project is minimalization.
Sysadmins should be able to ssh into their box, highlight the 200 lines of this script, paste it into a file, run it, and set it as their cron job. Boom free, automated certs. No installing dependencies or setup.py required. I want sysadmins to be able to read this script and understand what it's doing in 10 minutes or less, then start using it without having to do anything else.


[Feature Request]: Add light / dark mode effect in Privacy and T&C Pages.

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I want to add light/dark mode toggle effect on Privacy as well as T&C page.
Use Case
This will make the website look more attractive .
Benefits
No response
Add ScreenShots

Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSOC'24 contributor
[X] I have starred the repository

@apu52 @itsekta please assign this issue to me.


Add screenshots to readme

Solves the problem referenced in https://github.com/Watts-Lab/surveys/issues/93, closes https://github.com/Watts-Lab/surveys/issues/93

It seems that the GH action screenshot path doesn't use the same directory structure as the local. This may be because the action is running a different version from us. We should update our cypress, and see if we are behind or ahead of the action. We can also use an environment variable to turn off our manual screenshotting during the GH action.

Also missing the team viability screenshots


[CCI] Update developerGuide regarding yarn troubleshoot steps

Description
Currently the Developer's Guide solution to Windows not letting you running scripts on client PCs is to set the policy to Unrestricted. Running the yarn install script should be possible even with the execution policy set to RemoteSigned, and this will also prevent users from opening themselves up to potential malicious unsigned scripts that they can run.
Besides this, by scoping the poilcy change to current user in the guide, there's no need to run PowerShell with administrator privileges (especially relevant when user doesn't have access to them)
Issues Resolved
Closes #455
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I think it might be worth it to just remove the part about setting 'Unrestricted' policy completely, everything seems to run fine with RemoteSigned anyway (but I left it for now in case someone has the problems somehow, as a last resort).
Should we remove it?


How to pass arguments

And so there is the question of how to convey some arguments when creating an actor?
thx

this a simple code snippet：
props := actor.FromProducer(createPayApple(channel))
//other code

func createPayApple(channel config.PayChannel)actor.Producer{
	server := PayApple{
		available:  channel.Available,
		urlProduct: channel.UrlProduct,
		urlTest:    channel.UrlTest,
	}
	return func() actor.Actor {
		return &server
	}
}

For a more systematic understanding, it is advisable to look at akka's data, many of which are the same.

Marking as answered


kristianwhittick:patch-1

    kristianwhittick:patch-1

Originally posted by @mlkmooeeeed in https://github.com/github/docs/issues/22080#issuecomment-1368653382

This issue may have been opened accidentally. I'm going to close it now, but feel free to open a new issue with a more descriptive title or ask any questions in discussions!


Develop

fix hdf5 taxonomy in save
add sort_ids function
enable local config file var

Coverage decreased (-0.3%) to 78.3% when pulling 8c2dec422cea0bc900d26a0b9593970471c3e50e on develop into 4c4b1597f17c6a929be6d421b53ece0e8efba061 on master.

Coverage decreased (-0.3%) to 78.3% when pulling 8c2dec422cea0bc900d26a0b9593970471c3e50e on develop into 4c4b1597f17c6a929be6d421b53ece0e8efba061 on master.

Coverage decreased (-0.2%) to 78.321% when pulling becb99417301c600bd78de37aedff35b7a06e68d on develop into 4c4b1597f17c6a929be6d421b53ece0e8efba061 on master.


Fix rendering of CPM packages

Description

Switches to the appropriate Sphinx directives for the desired rendering.
Checklist

[x] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[ ] The documentation is up to date with these changes.
[ ] The cmake-format.json is up to date with these changes.
[ ] I have added new files under rapids-cmake/

[ ] I have added include guards (include_guard(GLOBAL))
[ ] I have added the associated docs/ rst file and update the api.rst

Closing in favor of #637


Documentation for Socket.Receive(Byte[], Int32, Int32, SocketFlags, SocketError)  is lacking specific details w.r.t exceptions

Link to article:
cf. Socket.Receive(Byte[], Int32, Int32, SocketFlags, out SocketError)
Problem:
The specific overload of Socket.Receive(...) returning a SocketError via an out parameter seems useful in order to explicitly avoid exceptions in a tight loop. Unfortunately, the documentation of this specific overload seems to have been copied and pasted from another, similar overload. The section thus still references SocketExceptions but doesn't mention whether they actually still might occur (and if so, in which edge cases this might be the case).
On a similar note, an overview which SocketError codes exactly are to be expected in which cases would be very useful. I encountered a case where said method would return SocketError.IOPending, although the socket was used in a purely synchronous fashion on a separate thread. Sadly, most documentation on the topic describes the lower level C API and there is no obvious way to tell which parts of it apply to its managed counterparts and which ones don't.

This issue was moved to dotnet/dotnet-api-docs#3693


createServerData$ runs twice

I wanted to add a 'viewCount' to my app showing the number of times a given thread was viewed. The approach I took was to update the database when fetching the list of messages in createServerData$:
export function routeData({ params }) {
  return createServerData$(
    async ([, id], { request }) => {
      const session = await getServerSession(request);
      await runQuery<{ thread: Post }>({
        operationName: 'BumpCount',
        query: threadQuery,
        session,
        variables: {
          postId: Number(id),
        },
      });
      return runQuery<{ thread: Post }>({
        operationName: 'ThreadQuery',
        query: threadQuery,
        session,
        variables: {
          postId: Number(id),
        },
      });
    },
    { key: () => ['thread', params.postId] }
  );
}

The problem I have, however, is that this code runs twice for every page that is rendered, which means that the view count gets bumped by 2.
It seems really inefficient to be double-fetching records from the database. The 'ThreadQuery' is a particularly expensive query because it's fetching an entire thread (including all the user reactions and comments), and it doesn't make much sense to fetch the data twice.

Seems like the fetcher is being re-executed on the client even though the key is the same.
This only happens when createRouteData is used with a function as the key.
This does not happen when using a static key or when using a function key with createResource.
https://stackblitz.com/edit/create-route-data-fn-key-bug?file=src%2Froutes%2Findex.tsx


Containers not working with 'Azure Cosmos DB for MongoDB'

The current published Docker containers can't operate with "Azure Cosmos DB for MongoDB" as MongoDB backend. The logs show the error  Command failed with error 115 (CommandNotSupported): ''text' is not supported' on server <Azure Cosmos DB for MongoDB name>. The full response is {"ok": 0.0, "errmsg": "'text' is not supported", "code": 115, "codeName": "CommandNotSupported"};
The reason seems to be on the site of Azure. The current version 4.2 of the Cosmos DB does not support all MongoDB features (Microsoft Documentation). However it might be desireable for BaSyx users to be able to use this Azure Resource.

Thank you very much for pointing this out. We are aware of this issue - Azure Cosmos DB does not support text based indexing which we are using at the moment. We will look for alternatives and update the images.

The fix was merged and the SNAPSHOT images were updated.

The new Docker images did not work for me. I set up a Azure Cosmos DB for Mongo DB and tried to connect a submodel repository and an aas repository. Both fail, the logs are attached.
docker-compose:
  basyx:
    internal: false

services:
  submodel-repo:
    container_name: submodel-repo
    image: eclipsebasyx/submodel-repository:2.0.0-SNAPSHOT
    environment:
      BASYX_BACKEND: MongoDB
      SPRING_DATA_MONGODB_DATABASE: submodels
      SPRING_DATA_MONGODB_URI: mongodb://${MONGODB_USERNAME}:${MONGODB_PASSWORD}@${MONGODB_HOST}:${MONGODB_PORT}/?tls=true
    ports:
      - 8081:8081
    networks:
      - basyx

  aas-repo:
    container_name: aas-repo
    image: eclipsebasyx/aas-repository:2.0.0-SNAPSHOT
    ports:
      - 8082:8081
    environment:
      BASYX_BACKEND: MongoDB
      SPRING_DATA_MONGODB_DATABASE: shells
      SPRING_DATA_MONGODB_URI: mongodb://${MONGODB_USERNAME}:${MONGODB_PASSWORD}@${MONGODB_HOST}:${MONGODB_PORT}/?tls=true
    networks:
      - basyx

aas-repo.log
submodel-repo.log

Unfortunately CosmosDB is again handling something differently than MongoDB. This issue was already reported in a separate ticket, see https://github.com/eclipse-basyx/basyx-java-server-sdk/issues/80

Thanks for the quick answer and pointing that out. For the AAS Repo I am having the same issue as in the other ticket but the submodel repo does not even connect to Azure Cosmos DB. There seems to be another issue:
The bean 'getSubmodelServiceFactory', defined in class path resource [org/eclipse/digitaltwin/basyx/submodelrepository/MongoDBSubmodelRepositoryConfiguration.class], could not be registered. A bean with that name has already been defined in class path resource [org/eclipse/digitaltwin/basyx/submodelrepository/component/SubmodelRepositoryConfiguration.class] and overriding is disabled.

@mm-mse Are you trying to run directly the code form IDE or you are using the Docker? Is your docker image updated?
I haven't tested it with the docker, but I can run the code directly without any issue.
Regarding Azure Cosmos DB compatibility issue, follow the other issue. #80

Thanks for the quick answer and pointing that out. For the AAS Repo I am having the same issue as in the other ticket but the submodel repo does not even connect to Azure Cosmos DB. There seems to be another issue:
The bean 'getSubmodelServiceFactory', defined in class path resource [org/eclipse/digitaltwin/basyx/submodelrepository/MongoDBSubmodelRepositoryConfiguration.class], could not be registered. A bean with that name has already been defined in class path resource [org/eclipse/digitaltwin/basyx/submodelrepository/component/SubmodelRepositoryConfiguration.class] and overriding is disabled.

This was fixed in PR https://github.com/eclipse-basyx/basyx-java-server-sdk/pull/82. However, we did not push a new snapshot version until now. This will happen at the end of this week.

alright, thanks


feat(examples): add serverless

Serverless Framework example with a calculator REST API.
It's using AWS provider and can be deployed into multiple environments. (refer Readme.md)

Hi there, sorry that this got lost. I am going to close this due to the age but I would encourage you to open a new PR with these changes rebased and I will make sure to review it personally. Thanks!


gtf assertion error when installing ensembl release

This is on Python 3.
$ pyensembl 75 install
INFO:root:Annotation data for release 75 is already downloaded
INFO:root:Transcript sequence data for release 75 is already downloaded
INFO:root:Cached file Homo_sapiens.GRCh37.75.gtf from URL ftp://ftp.ensembl.org/pub/release-75/gtf/homo_sapiens/Homo_sapiens.GRCh37.75.gtf.gz
Creating database: /Users/tim/Library/Caches/ensembl/Homo_sapiens.GRCh37.75.db
INFO:root:Reading GTF /Users/tim/Library/Caches/ensembl/Homo_sapiens.GRCh37.75.gtf into DataFrame
INFO:root:Extracting attributes for 0 entries in GTF DataFrame
INFO:root:Adding attribute columns: []
Traceback (most recent call last):
  File "/Users/tim/sinai/git/datacache/venv/bin/pyensembl", line 9, in <module>
    load_entry_point('pyensembl==0.5.12', 'console_scripts', 'pyensembl')()
  File "/Users/tim/sinai/git/pyensembl/pyensembl/shell.py", line 59, in run
    data.install()
  File "/Users/tim/sinai/git/pyensembl/pyensembl/ensembl_release.py", line 152, in install
    self._index(force=False)
  File "/Users/tim/sinai/git/pyensembl/pyensembl/ensembl_release.py", line 163, in _index
    if self.db.create(force=force):
  File "/Users/tim/sinai/git/pyensembl/pyensembl/database.py", line 515, in create
    self._create_database(force=force)
  File "/Users/tim/sinai/git/pyensembl/pyensembl/database.py", line 153, in _create_database
    df = self.gtf.dataframe()
  File "/Users/tim/sinai/git/pyensembl/pyensembl/gtf.py", line 241, in dataframe
    self._dataframes[key] = cached_dataframe(csv_path, local_loader_fn)
  File "/Users/tim/sinai/git/pyensembl/pyensembl/compute_cache.py", line 92, in cached_dataframe
    df = compute_fn()
  File "/Users/tim/sinai/git/pyensembl/pyensembl/gtf.py", line 215, in local_loader_fn
    full_df = self._load_full_dataframe()
  File "/Users/tim/sinai/git/pyensembl/pyensembl/gtf.py", line 178, in _load_full_dataframe
    return cached_dataframe(csv_path, self._load_full_dataframe_from_gtf)
  File "/Users/tim/sinai/git/pyensembl/pyensembl/compute_cache.py", line 92, in cached_dataframe
    df = compute_fn()
  File "/Users/tim/sinai/git/pyensembl/pyensembl/gtf.py", line 185, in _load_full_dataframe_from_gtf
    return load_gtf_as_dataframe(path)
  File "/Users/tim/sinai/git/pyensembl/pyensembl/gtf_parsing.py", line 309, in load_gtf_as_dataframe
    list(sorted(df.columns)))
AssertionError: Missing required column 'gene_name', available: ['end', 'feature', 'frame', 'score', 'seqname', 'source', 'start', 'strand']

Does this still happen? I wasn't able to recreate it and am not sure if I fixed it previously.

I think it's fixed. Will reopen if I hit again.


Failing to compile on ios and android after update to 0.2

Details:

error on ios: you can convert an usize to i32 - https://gist.github.com/r10s/56dfccebbc2d65b52e774ccde933bd5a
error on android: undefined reference to sethostname - https://gist.github.com/r10s/3c620bddc01d8640e62287b966b10f93

worked fine with 0.1. 5

maybe i did sth. wrong ianar (i am not a rust-expert  :) if some additional information are needed, please just ping me :)

Thanks for a bug report! It was not really clarified what targets are supported for 0.1 version, apparently, this information was lost during the ownership transfer, sorry about that.
arm-linux-androideabi and aarch64-linux-android Android targets are already tested in CI and I dropped few iOS targets too (armv7-apple-ios and x86_64-apple-ios), so far everything compiles correctly, so it would be really helpful to provide a bit more information about your environments.
From what I understand, Android build targets Android 4.3 Jelly Bean (API 18) on a armv7-linux-androideabi platform?
It is also a bit unclear how exactly it compiles for iOS target, could you maybe provide executed command or smth, so I'll try to reproduce the issues.

hi @svartalf thanks for your help!
on android we use armv7-linux-androideabi, aarch64-linux-android, i686-linux-android, x86_64-linux-android using the most recent ndk. __ANDROID_API__ is set to 18 for 32bit and to 21 for 64bit. rust compiling succeeds, however, at the end linking fails, ndk-make complains about that sethostname is not there, see https://gist.github.com/r10s/3c620bddc01d8640e62287b966b10f93 (the first warning is not releated), with hostname 0.1.5, there is no problem.
on ios, rust compiling fails with the error shown above, the used targets are  ´aarch64-apple-ios´ and ´x86_64-apple-ios´ - at least this is what xcode claims.

Yes, hostname::set, which is utilizes sethostname for *nix systems was introduced in 0.2.0, that's why linking fails in your case.
It is still unclear to me, why CI builds are green, but at least we have a starting point for investigation now.

Alright, here goes the follow-up:
For iOS we could extend the following check to include the ios targets also: https://github.com/svartalf/hostname/blob/b7b4faeee5de0fa8e3e3542c5029713f2175b415/src/nix.rs#L42-L46,
same as nix crate does, that should handle the issue.
As for Android, I see that sethostnamefunction was introduced in Android API 23 (see NDK sources), and it is probably the reason why linking fails.
Not sure how to handle that exactly, though: dynamic function loading seems rather complicated in that case, I would probably go with some sort of set feature, which will enable the hostname::set function.
@r10s, any thoughts about it?

Okay, ce41122c6fd421b3f87ed66eb6c5c0e355bb3c1d should handle the iOS issue and 72fc33faf7aca32043efebad4d33d9b4bcc478a6 hides hostname::set function under a #[cfg(feature = "set")] feature (disabled by default), so it should handle any linking problems.
@dignifiedquire @r10s can you test it? Following patch section in the Cargo manifest should properly override the dependency:
[patch.crates-io]
hostname = { git = "https://github.com/svartalf/hostname.git", branch = "ios" }

@svartalf thanks, that looks good! unfortunately, my rust-cargo-skills are not sufficient to try this out easily, the hostname dependency is not used directly by deltachat. maybe we do a pr on deltachat that reverts the change to 0.1.5 we've just done for now to get things out,

Compiled it successfully for ios (I don't have android setup)

@dignifiedquire thanks a lot for testing!

Quick follow-up: how does the testing going on? Had you any chance to test it both for iOS and Android?

So, I hide the hostname::set function with a feature gate; as it was a bit of a rush to publish it as a default part of v0.2.0 and I tried to test the Android/iOS compatibility locally as much as I can, it should be okay now.
@hpk42, @dignifiedquire, can we test your case once again with the hostname = "0.3" version?

sorry, i did not came back to this later. not sure if we're back on the hostname-origin-crate, however. but if so, we will see if things work :) thanks a lot for helping :)


How to use 2 GPUs?

python train.py --cfg_path ${cfgs/anet_c3d_pdvc.yml}
But only one GPU is used!

OK！Thanks!


Question/RFE

Great design. Also loved the reference to s3s3mirror thanks.
Similar quesiton/RFE.

You only mention name+size, there is no check of ETAG or checksum or other method to make sure files of the same size but different content are not neglected ?
Does this handle S3 versions in any way ?
Are User meta data values copied when going from S3 to S3 ?
Any provisions for mapping filesystem attributes to user metadata ?

Hi @DALDEI

Correct, those attributes are not checked. See https://github.com/bitsofinfo/s3-bucket-loader/blob/master/src/main/java/org/bitsofinfo/s3/toc/TocInfo.java
However it might not be that difficult to extend this to add those attributes. Feel free to submit a PR. Note that ETAG is not relevant for the source of the data (if the source is from a non-s3 filesystem). Checksum could be, but you would have to match the same algo s3 uses for checksums.

S3 versions? No

If your goal is to put a bunch of stuff into S3 using a s3 filesystem as the conduit, and then access it via that same S3 filesystem across many machines once loaded, then yes. If you execute the uploads to S3 via an S3 filesystem target (such as yas3fs) and specify in the config to use rsync as the copying mechanism then yes, UID/GID are preserved (if that is what you are asking) assuming you will access the files in S3 from that same s3 filesystem impl; the UID/GID file stat info is stored in the metadata. (The meta-data writing is a feature of s3 uploading impl of a s3 filesystem such as yas3fs, not this program)

Mapping FS attrs to S3 meta-data? Yes, that occurs if you use yas3fs as the conduit target for the s3 uploads


add more

please add more I love them

It's open source, you can add some yourself ;)
But I'll still try to add more 👍

Added some more hope you like it!
And while you're at it could you buymeacoffee 👉👈


ci: prevent benchmarks job for running on an action for fork

Information

Type
Breaking change

Chore
No

See details in actions/github-script#42

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4233

Relevant Lines:
4243

💛  - Coveralls

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

v6.98.2
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.0.0-beta.4 :tada:
The release is available on:

v7.0.0-beta.4
GitHub release

Your semantic-release bot :package::rocket:


Fix loading for sites

$rootScope.isLoaded = true
right now stays true after the first view you visit. Originally it was $scope.isLoaded and therefore worked as intended, to show the loading screen on long angular processing pages.

possible fix would to simply set the $rootScope.isLoaded as false before navigating to any other page

it now only happens for the first page, is the hourglass that appears when it says Eyra is retrieving the tokens for the first time


Async-ified linterTests.ts

PR checklist

[x] Addresses an existing issue: #2328 (indirectly) / #3333

Overview of change:
Uses asynchronous file reading & writing. Very mild performance boost for writing the test's temp files in parallel.

this one makes no sense to be async and awaited immediately

I'll just rescind. Thanks for the review!


Introduce edge._id generation by graph

The graph class should only be accepting adding edges by string, which from now on will only be handled as label.
Then internally, we are going to assign an edge _id which for reasons of simplicity will just be a number that gets incremented (we don't worry about graph's with #edges > 2^64 edges right now...).
Programmers will then have to user the edge_objects getID() method for the graph class to be able to look it up... plus: adapt has/get edge methods accordingly...

CLOSED -> not entirely sure what I wanted to achieve by this... for testing purposes, the "A_B_(u|d)" string type ID actually allows comfortable lookup & naive duplicate handling, so why not leave it...?
However, we 1) cannot assume that a user will stick to this naming convention and hence cannot count on it; 2) when adding edges via JSONInput, we will thus generate a UUID for each edge and handle duplicates via a more involved checking method also involving type & weight, as handled in this issue: #89


Use type assertion to sort keys during diffing

Use type assertion to sort keys during diffing and fallback to Marshal if key is not a string (shouldn't happen, but just to be sure)

Closing in favour of #23