YAML support for monorepo style codebase?

We use a codebase that has one folder per project all sitting in a single git repo. Is it possible to have a YAML based deployments where each folder has a YAML file that describes build/release ?

@vtbassmatt Thanks for the response. I am not sure it was clear, but our different folder have different build+release definitions and they all get triggered with changes. My question was how can I have multiple YAMLs in a repo - each will be triggered corresponding to different build/release ?

The current model has each build pipeline corresponding to a YAML file that can be anywhere in your directory structure. So if you have N services/apps, you need N build pipelines. The YAML can specify path triggers so that a build is only kicked off when a commit touches a file in one of the paths. Ideally, your build pipeline only produces artifacts. Your release pipleine is then triggered on the appearance of those artifacts + other possibly other gates.
Some sort of tooling is needed to sync up the YAML path triggers with your csproj. For .net core, this is fairly easy -- you can use msbuild to produce the project dependency graph and then process the dependency graph into YAML files. (What's needed is a dotnet tool plugin... )


feat: tea.Batch() returns nil if all Cmds were nil

hi,
as far as i can tell, tea.Batch() semantically should return nil if there is no tea.Cmd. this is not documented, but i base this assumption on the length check in its implementation:
https://github.com/charmbracelet/bubbletea/blob/608fde59edde447d4f34ce7e8ddc8bb3d54b80fa/tea.go#L112-L128
however, if the []tea.Cmd passed to it contains only nil item(s), a batch cmd containing only nil is returned, instead of just nil.
a []tea.Cmd that just contains only nil can easily happen when you expect multiple tea.Cmd in your Update() function and start by initializing the slice, but then don't check every cmd returned by a subcall for nil first before appending to this slice.
this happens for example in bubbles.List:
https://github.com/charmbracelet/bubbles/blob/7ecce3fb97420ba2a44e3453c9291a98d5894d9d/list/list.go#L682-L683
this bit me a little bit, as i pass the tea.Msg to a submodel, and wanted to check if it "consumed" the message by checking if the returned tea.Cmd is nil. if it is not consumed, i consume it in the main model (think pressing : in vim insert mode vs normal mode). i used the pattern from the List bubble in my own code, but even when i checked everywhere for nil before appending, i realized that the List bubble itself returns a batch cmd with just a nil inside it and i'd have to check for that too.
for now i just worked around this by explicitly telling my main model if the message was consumed, which is fine too. but i think it would be more comfortable and true to the intention of the tea.Batch() function to e.g. check if the slice given to it contains only nils. not only can i just still check whether the tea.Cmd a model returns is simply nil or not without having to know that it is a batch cmd, but i also don't have to check every returned cmd before appending it to such a slice to make this work.
the reasons i can think of against this are:

the slight performance hit of iterating through the slice to check the items for nil

but i think generally the slice should be pretty short

the behaviour change

but not if you consider this a bugfix to implement the intended behaviour

what do you think?

I took an approach a bit different: #217

filtering out the nils and then checking if anything is left definitely seems more elegant than counting nils. how big is the impact of a nil command? an alternative to my solution could be cancelling that check at the first non-nil value encountered, so not the whole slice has to be traversed, if that matters in any way.

Good catch, @gkdr! I think I prefer #217 as a fix as well.

okay, merged #217, will close this one
answering: @gkdr I believe iterating here shouldn't be much of a problem
thanks for the issue/pr :)


First Row Padding with condensed={true}

I have noticed an issue with the padding of the first row (after the header row, that is) of a table that has condensed={true}:

See how the 1 row has slightly less padding than the others? I think this is because its padding is overlapping with the header row (notice how the green padding goes above the header row bottom border):

Thanks for bug report, I've fixed this styling issue and will release it on next version.

@AllenFang :+1:   :+1:   :+1:

Released on v1.4.0


Is setting consistent-read supported by the API  ?

Hello,
is it somehow supported (planned) to allow setting ConsistentRead=true ?
(e.g.: http://docs.aws.amazon.com/amazondynamodb/latest/APIReference/API_GetItem.html#API_GetItem_RequestSyntax)
I searched the sourceode but could't find anything that could solve this.
Thank you in advance!
Cheers,
Fabian

I am also looking for an answer to the same ,looking at the code and setting DynamoDbConfig seems the mapper used in the api is not using the bean provided with CONSISTENT reads . Will dig through the code more to find a way to inject the same .Though any help will be great
@derjust

Yes @gauravbrills is right you have to do it with those two things (configuration bean + configRef) - annoying and should be improved - but that's the way to do it right now.


Incrementing and Decrementing Numeric Attributes GraphQL

Which Category is your question related to?
GraphQL
Amplify CLI Version
4.13.1
What AWS Services are you utilizing?
Cognito, AppSync, DynamoDB, API Gateway, Lambda
Is it possible to increment or decrement a number in a graph model like you can do with the dynamodb SET Update Expression (https://docs.aws.amazon.com/amazondynamodb/latest/developerguide/Expressions.UpdateExpressions.html#Expressions.UpdateExpressions.SET.IncrementAndDecrement)?
For example I have this model:
type ResetHistory @model @auth(
  rules: [ {allow: owner} ])
{
  id: ID!
  resetCount: Int!
  totalResetTime: Float!
}

I currently have a function that updates the dynamodb table directly using the following command:
...[snip]
const params = {
          TableName: `ResetHistory-${apiRoomvioapiGraphQLAPIIdOutput}-${environment}`,
          Key: {
            id: unmarshalledNewRecord.id
          },
          UpdateExpression: 'SET resetCount = resetCount + :count, totalResetTime = totalResetTime + :resetTime',
          ExpressionAttributeValues: {
            ":count": 1,
            ":resetTime": parseFloat(resetTimeMinutes)
          },
        };
        try {
          //Write updates to agg table
          await documentClient.update(params).promise();
        ....[more code below]

This works but I need to create a subscription on this data for my app so I am looking to rewrite the function to execute an AppSync function (updateResetHistory) so it can trigger subscriptions. Is there a way to accomplish the same increment functionality within the amplify framework?
At the end of the day I really just need to set a field = field + [some number] but I am not seeing anything in the docs. I have been reading over posts about custom resolvers. Is that the only way to get this to work?

Is this not possible with Amplify? It is such a simple operation to not be able to do.

Hi @tfendt if you are referring to setting this kind of increment type operation in the schema, a custom implementation of this could be done through a customer transformer. Another way would be to create a custom resolver which would include this set expression in the VTL code.

I am looking for this same feature as well. DynamoDB supports Atomic Counters. Can Amplify support this? It is such as common features to keep track of upvotes, likes, etc that any app would use nowadays.

Just an idea and not sure if this would solve your issue. May be use @function and invoke an lambda to perform the increment operation for resetCount and totalResetTime.

Thanks @SwaySway I will give it a try.
@xitanggg looks interesting and easier than the other options. I'll give this a try first.

@SwaySway I tried the following but for some reason when executing the mutation it doesn't trigger the subscription. Am I missing anything? I would expect calling the updateResetTime would then trigger the subscription. If it makes a difference I am calling updateResetTime from a lambda function.
`type Mutation {
updateResetTime(owner: String!, id: String!, resetCount: Int!, resetTime: Float!): String @function(name: "updateResetTime-${env}") @aws_iam
}
type Subscription {
onUpdateResetTime(owner: String!, id: String!, resetCount: Int!, resetTime: Float!): String @aws_subscribe(mutations: ["updateResetTime"])
}`

So I ended up creating a custom Mutation to do this. I made it pretty generic so it should help others. It's not great - but it is a start and should be able to help people. I have left out the authentication and some other boilerplate included with the default update Mutation. You can copy it from what is generated in the amplify/backend/api/<ApiName>/build/resolvers/Mutation.<operation>.req.vtl. You should see where to start inserting this code by looking at the boilerplate. You will want to put your custom file at amplify/backend/api/<ApiName>/resolvers/Mutation.<operation>.req.vtl so it gets built every time you build the graphql schema.
The code looks at the values provided and instead of overwriting numbers will add them to the value that is already there. So if you pass 5 it will add 5 to whatever value is already in the field. It works with negative numbers as well. The caveat is that ALL NUMERIC FIELDS ARE ADDITIVE. There is no way to simply override the value with these operations.
#set( $expNames = {} )
#set( $expValues = {} )
#set( $expSet = {} )
#set( $expAdd = {} )
#set( $expRemove = [] )
#set( $expNumbers = [] )
#if( $modelObjectKey )
  #set( $keyFields = [] )
  #foreach( $entry in $modelObjectKey.entrySet() )
    $util.qr($keyFields.add("$entry.key"))
  #end
#else
  #set( $keyFields = ["id"] )
#end
#foreach( $entry in $util.map.copyAndRemoveAllKeys($context.args.input, $keyFields).entrySet() )
  #if( !$util.isNull($dynamodbNameOverrideMap) && $dynamodbNameOverrideMap.containsKey("$entry.key") )
    #set( $entryKeyAttributeName = $dynamodbNameOverrideMap.get("$entry.key") )
  #else
    #set( $entryKeyAttributeName = $entry.key )
  #end
  #if( $util.isNull($entry.value) )
    #set( $discard = $expRemove.add("#$entryKeyAttributeName") )
    $util.qr($expNames.put("#$entryKeyAttributeName", "$entry.key"))
  #else
    $util.qr($expSet.put("#$entryKeyAttributeName", ":$entryKeyAttributeName"))
    $util.qr($expNames.put("#$entryKeyAttributeName", "$entry.key"))
    #if ( $util.isNumber($entry.value) )
      $util.qr($expNumbers.add("#$entryKeyAttributeName"))
    #end
    $util.qr($expValues.put(":$entryKeyAttributeName", $util.dynamodb.toDynamoDB($entry.value)))
  #end
#end
#set( $expression = "" )
#if( !$expSet.isEmpty() )
  #set( $expression = "SET" )
  #foreach( $entry in $expSet.entrySet() )
    #if ( $expNumbers.contains($entry.key) )
      #set( $expression = "$expression $entry.key = $entry.key + $entry.value" )
    #else
      #set( $expression = "$expression $entry.key = $entry.value" )
    #end
    #if( $foreach.hasNext() )
      #set( $expression = "$expression," )
    #end
  #end
#end
#if( !$expAdd.isEmpty() )
  #set( $expression = "$expression ADD" )
  #foreach( $entry in $expAdd.entrySet() )
    #set( $expression = "$expression $entry.key $entry.value" )
    #if( $foreach.hasNext() )
      #set( $expression = "$expression," )
    #end
  #end
#end
#if( !$expRemove.isEmpty() )
  #set( $expression = "$expression REMOVE" )
  #foreach( $entry in $expRemove )
    #set( $expression = "$expression $entry" )
    #if( $foreach.hasNext() )
      #set( $expression = "$expression," )
    #end
  #end
#end
#set( $update = {} )
$util.qr($update.put("expression", "$expression"))
#if( !$expNames.isEmpty() )
  $util.qr($update.put("expressionNames", $expNames))
#end
#if( !$expValues.isEmpty() )
  $util.qr($update.put("expressionValues", $expValues))
#end
{
  "version": "2017-02-28",
  "operation": "UpdateItem",
  "key": #if( $modelObjectKey ) $util.toJson($modelObjectKey) #else {
  "id": {
      "S": $util.toJson($context.args.input.id)
  }
} #end,
  "update": $util.toJson($update),
  "condition": $util.toJson($condition)
}

Closing this issue as it's not a bug. Should you have any other comments on discussion on this we recommend our community discord.


feat: handle OS term signals

Nice. Exactly what I needed.
Is there a release planned adding this functionality to an official release?
The latest official release was the release of v0.7.0 on 5th of May, but this was merged later on 18th of May. It would be nice if there was an official release with this added functionality.
Without support for graceful stop on OS signals it is quite hard to work with profiling tools.


K8SSAND-1524 ⁃ Add a resource configuration block for every spawned POD

What is missing?
Configuration for the resources for the PODs managed by the operator
Why do we need it?
The best practice is to set resource requirements for every POD running to allow the correct resource placement.

Also our operations team require every resource deployed (via the admission controller) to have such configuration otherwise it is rejected by the policy.

Environment

K8ssandra Operator version: v1.1.1

┆Issue is synchronized with this Jira Task by Unito
┆friendlyId: K8SSAND-1524
┆priority: Medium

I agree that container resources need to be configurable. I read some interesting stuff earlier today that suggests it could be a bad idea to set limits. See https://www.reddit.com/r/kubernetes/comments/all1vg/on_kubernetes_cpu_limits/. With that said, I do think they should be configurable.

Just got into the issue while enabling Reaper as reaper deployment does not have resources defined.
Behaviour is following:

Operator is stuck in Deploying forever without throwing any errors
Once limit enforcement is suspended on the gatekeeper Reaper is able to proceed

There are 2 issues linked to that

Operator likely should have some sort of error handling/log trail to handle the faulty deployments of the Reaper resource (maybe a new issue?)
We should be able to control the limits of PODs (current issue)

Hey team! Please add your planning poker estimate with ZenHub @jsanda @Miles-Garnsey

@jsanda @bradfordcp, I think we have enough info to move forward and estimate that ticket. I think we should be able to implement this before v1.2 ships out. wdyt?

I agree that this would be a nice enhancement for 1.2.

We may want to set request defaults as well while we implement these changes.

A few things here as I'm looking to pick this up.

I think the original scope needs revision. We probably want resources blocks for every spawned container - pods don't have resource blocks.
Do we want initContainers to also have resource blocks? I think yes since the motivation is to ensure that they can be scheduled in systems where a validating webhook rejects requests for containers without resource limits.
But if we do include initContainers, this is a very broad ticket. Maybe we should do containers first? In either event, I suggest we make this an epic and ticket it out pod by pod to keep PR sizes more manageable.

I think the original scope needs revision. We probably want resources blocks for every spawned container - pods don't have resource blocks.

Agreed

Do we want initContainers to also have resource blocks? I think yes since the motivation is to ensure that they can be scheduled in systems where a validating webhook rejects requests for containers without resource limits.
But if we do include initContainers, this is a very broad ticket. Maybe we should do containers first? In either event, I suggest we make this an epic and ticket it out pod by pod to keep PR sizes more manageable.

Interesting question. If that's going to be more involved, I agree that we should separate concerns and create a follow up ticket for init containers 👍

I don't think the motivation is to be able to have a webhook to reject pods without limits. This ticket is about exposing the resources APIs for containers and init containers created by the operator. It would be good to see proposed API changes first. I don't think exposing resources for init containers makes this more complicated, does it?

I don't think exposing resources for init containers makes this more complicated, does it?

No, just a larger blast radius, but the work for initContainers/Containers is equivalent as far as I can see.

Well, if you need to be able to set distinct resource requirements for every specific init container we have, it makes it more involved for sure (and it'll make more CRD changes as well).
@Miles-Garnsey, I'm ok with making this an epic and creating sub tickets per pods. It'll allow to track work more efficiently and create smaller PRs.


Sum up fees before transferring bulkBuy and bulkSell

Hey @garyghayrat, there were a couple things that seemed off with this approach that I wanted to address—I had some time today so I spiked out my take on it here: https://github.com/showtime-xyz/creator-tokens/pull/45.
In short:

This avoids adding overhead to the non-iterating buy and sell methods by not currying to the bulk buy methods (felt regressive to do that in the name of "optimizing")
This avoids changing the return values of the public methods
This does all three transfers in one place (felt weird to do some in one place, but others not)
This renames the internal methods to indicate the transfers (i.e. the payment) have not occurred when the method returns

Because only internal non-public behavior was modified, no changes to the tests was required. Check out my PR and see what you think.


Delegate to facter if installed

This restores the behavior that was removed in 9b7fe39, whereby the
shell and powershell implementations of the facts task will detect
whether facter is available and call out to it if possible.
This was removed because it leads to a situation where you can get
different kinds of output from the same implementation of the task
depending on where it's being run. But since the unit of abstraction is
the task itself, and not an implementation, it's better to provide the
full facter output in as many cases as possible, without the user
having to explicitly mark the target as having the puppet-agent
feature first. This makes the output less consistent between targets for
the same implementation, but more consistent for the same target between
implementations.

This will break some things I'm working on, because I expect that explicitly calling facts::bash will give me the bash-specific facts. e.g. codename.  Could there be a way of handling this with a flag or option?

I like the idea in https://github.com/puppetlabs/puppetlabs-facts/pull/36/commits/cf0625da200979e1b16da11b48b5b473887ef8d3


PaymentSheet: place a hold

Describe the bug
How do I place a hold using PaymentSheet?
The paymentIntent API has clear instructions, however, I would need the payment method_id and PaymentSheet doesn't have it at creation.
        paymentIntent = stripe.PaymentIntent.create(
          confirm=True,
          capture_method='manual',
        )

To Reproduce
    const { error } = await initPaymentSheet({
      customerId: customer,
      customerEphemeralKeySecret: ephemeralKey,
      paymentIntentClientSecret: paymentIntent,
      // Set `allowsDelayedPaymentMethods` to true if your business can handle payment
      //methods that complete payment after a delay, like SEPA Debit and Sofort.
      allowsDelayedPaymentMethods: true,
    });

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
https://stripe.com/docs/payments/capture-later#authorize-only

seems to work when I remove confirm=true.


Separate notions of trusted peers and validators set

According to the RFC https://wiki.hyperledger.org/display/iroha/P2P+network

This is just about naming conventions for the peers


merge from template-sync/template-arlac77-github

.github/workflows/update_package_lock.yml

chore: add<EMAIL_ADDRESS>(jobs.update_package_lock.steps.uses)

.github/workflows/update_readme_api.yml

chore: add<EMAIL_ADDRESS>(jobs.update_readme_api.steps.uses)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

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

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
510

Relevant Lines:
539

💛  - Coveralls


traverse parent scopes to evaluate the expression given in init-date

If init-date is set and contains a valid angular expression, parent scopes are evaluated for this expression. First $parent may not contain the expression thus this method traverses all parents for the given expression.

I believe this is now fixed as a result of 98e2bdfc8b4dd108acdf3fac75a795675889bbc7 .  Thanks for the effort here though!


System support for GURPS 4e

System Name:
GURPS 4th Ed. Game Aid (Unofficial)
System Repository or Foundry VTT address:
https://foundryvtt.com/packages/gurps
Does the system has the languages set up on Foundry?
No
Languages in GURPS are interpreted as advantages and formatted like such as actor data:

Languages can only be added as advantages.
I see that GURPS is already a supported system, but have yet to find anyone who got the module working for GURPS.

Change the name from Language: Common to Language (Common) and it'll show up.


about extensions.md in doc

Hi Hero:
Firstly, in this very thank yours project  to bring their own power. It's very cool. So I have example demo project base Sanic with vueJS + webpack and hope update to  extensions.md in doc.
    - [SanicCRUD-vue](https://github.com/boylegu/SanicCRUD-vue): A example demo base Sanic with vueJS + webpack

I hope pass thought this example demo project for Sanic to be a better promotion.
So may i can take it merge your extensions.md ?  Thanks.

I just added this to the wiki, soon the extensions.md file will be removed. Only Sanic core related info will be in docs.
https://github.com/channelcat/sanic/wiki/Extensions
😄

Awesome! Closing this for now


[Feature Request] Introducing Conversational Task in Hugging Face transformers.js

Name of the feature
In general, the feature you want added should be supported by HuggingFace's transformers library:

If requesting a model, it must be listed here.
If requesting a pipeline, it must be listed here.
If requesting a task, it must be listed here.

I request the task: conversational
å
Reason for request
Why is it important that we add this feature? What is your intended use case? Remember, we are more likely to add support for models/pipelines/tasks that are popular (e.g., many downloads), or contain functionality that does not exist (e.g., new input type).

The addition of the Conversational task to the Hugging Face transformers.js library is important for several reasons. Firstly, it addresses the need for enhanced privacy in conversational AI applications. By allowing users to utilize Language Models (LLMs) in the front-end without sharing their OpenAI key or any private information, the Conversational task ensures that users can engage in conversations while safeguarding their privacy. This is crucial for users who are reluctant to disclose sensitive data and enables the application of LLMs in various sensitive areas.
Secondly, enabling the execution of the ChatGPT model in the front-end extends its capabilities to information-sensitive fields such as the medical domain and other domains where data privacy is critical. My intended use case for this ChatGPT model, which can run in the front-end, is as an AI data science assistant on Electronic Health Records (EHR). I plan to train LLMs for research purposes on EHR data and make them work in the front-end environment without sending sensitive patient health information to third-party servers. This allows for the development of real-time applications that leverage conversational models to provide valuable insights and assistance while maintaining data privacy and security. The Conversational task empowers developers to build conversational AI systems that adhere to strict data privacy requirements and can be used in sensitive domains where privacy is of utmost importance.
Furthermore, integrating the Conversational task into the Hugging Face transformers.js library expands its functionality and provides developers with a privacy-focused solution for implementing conversational AI applications. By leveraging the library's rich ecosystem of pre-trained models and tools, developers can create sophisticated conversational systems that respect user privacy, deliver seamless user experiences, and cater to a wide range of industries and domains.
Additional context
The addition of the Conversational task to the Hugging Face transformers.js library not only addresses the demand for privacy-conscious conversational AI but also contributes to the development of ethical and secure systems. It enables developers to build innovative conversational applications that prioritize user privacy, empower users to have control over their data, and ensure secure interactions. This feature is valuable across various industries, including healthcare, customer service, education, and more, where privacy and data protection are paramount. By supporting the Conversational task, the Hugging Face transformers.js library becomes a versatile and privacy-enhanced platform for developing cutting-edge conversational AI solutions.
Add any other context or screenshots about the feature request here.

This is definitely something we plan to add! Do you know which model you are planning to use? If so, we could try prioritize adding support for it. The main limiting factor for feasibly running these very large models in the browser is the lack of WebGPU support, which we are working on. And once it is supported, this will be a much greater priority.

Thank you for your prompt response. Our team is planning to utilize the Llama-65B and Alpaca-30B models for our project.

Consider adding support for bloomz . It supports multilingual(56 languages) generation on all models and even multilingual prompt on 7.1B and 176B models (I guess those 2 models will only be usable on server and not in browser).
Also, the smallest model 560m will very easily run in the browser after quantixation and you do not have to wait till webgpu support


Update gitignore

@mlehotskylf - just wanted to make sure that some files I use locally are not committed to the repo.

Still I can't merge @mlehotskylf due to CI failures not related to this PR - can u merge?

lint-test-build never finished, is this a bug @mlehotskylf - it is listed as "required" and didn't finish since yesterday...

@mlehotskylf this is still not resolved.

lint-test-build still blocks this @mlehotskylf


Pinpoint resource ID not found.

Amplify CLI Version
12.7.1
Question
When I try to pull amplify environment I get an error during initializing environment.
✔ Successfully pulled backend environment mainz from the cloud.
✖ There was an error initializing your environment.
🛑 Pinpoint resource ID not found.

Resolution: Run "amplify add analytics" to create a new Pinpoint resource.

I would love to run amplify add analytics to resolve the issue, but the amplify folder is not created so it cannot be done.
We have not added analytics to this environment. We did add it to another environment though.
Web console lists 4 categories as expected:
API
Authentication
Functions
Storage

Nothing about analytics or notifications is mentioned.
How can we recover the environment? There is no way for us to pull it right now.

Hey @krosiek, thank you for reaching out. Wanted to get some information on this behavior.
what command is being run to pull the environment (for example: amplify pull --envName --appID)?
is the environment being pulled from a different account or application?
was this environment created on the AWS Amplify console or using the CLI amplify add env?
As when using amplify CLI to create a new environment the CLI should create the Analytics resource for the environment.
was the resource previously removed from the environment?

The environment was created with amplify add env and it was working for long time till we merged PR was merged that has no amplify categories related changes. (it changed schema.graphql).
The command used to pull the environment is amplify pull --envName --appID.
We also have Amplify CI connected to it, which gives the same error.

Hey @krosiek, could you run amplify diagnose --send-report at the root of the project and provide us the Project Identifier output from your terminal or if was one already created on the build or pull command.

Closing due to inactivity


Object detection file names required to be numeric

When creating an Object Detection dataset, it seems like DIGITS requires the file names to be numeric. When I tried to create a dataset with filenames that contained alphabetic characters, I got the error ERROR: ValueError: invalid literal for int() with base 10: 'ba' in the sidebar under Create train_db DB where ba.txt and ba.bmp were the names of two of my files. When I changed the filenames to include only numeric characters (eg. 00.txt or 00.bmp), the dataset was successfully created.
I'm not sure if this requirement actually exists or is the result of a different bug, but it would be nice if DIGITS could handle alphabetic file names or if there was at least a warning on the UI.

Thanks for the report. As hinted on this page we expect a numerical identifier and the error is indeed not self-explanatory at all. We might relax this constraint in the future. In the mean time, can you stick to the expected format?

Thanks, I can stick to using numeric identifiers for now. I know the example names provided (000001.png and 2.jpg) are exclusively numerical, but for someone just reading the page it seems like the only requirement is that the file names for the labels and images must match

@darreneng I actually thought the same at first, too: https://github.com/NVIDIA/DIGITS/pull/909


cmd/compile: inlining of empty functions computes arguments

14531:7b037816cd5c tip linux/amd64

The following code:

func raceAcquire(addr unsafe.Pointer) {
}
func (rw *RWMutex) Lock() {
  raceAcquire(unsafe.Pointer(&rw.readerSem))
}

leads to generation of the following machine code:

  raceAcquire(unsafe.Pointer(&rw.readerSem))
  450e3e:       48 8b 44 24 20          mov    0x20(%rsp),%rax
  450e43:       48 83 c0 0c             add    $0xc,%rax
  // later %rax is not used

I would expect raceAcquire() call to yield no code at all.

This should be fixed with the SSA backend at tip.

Go tip generates a call to the empty function if the function is only empty e.g. after some dead code elimination (e.g. when used with a compile time debug constant) in the function: #4563
Maybe that is easier to remove with SSA now too.


Roll engine 223d6d3e4278..b520d7fd4b6d (1 commits)

git@github.com:flutter/engine.git/compare/223d6d3e4278...b520d7fd4b6d
git log 223d6d3e4278..b520d7fd4b6d --no-merges --oneline
2019-09-26<EMAIL_ADDRESS>Roll src/third_party/dart 53564db036..6ff8d2199d (1 commits)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Auto-roller completed checks. About to merge.


Tag support for crds and update from runtime and codegen v0.6.0

Enables tag supports for crds and adds them to the e2e testing.
Additionally updates to codegen and runtime v0.6.0

/test sagemaker-kind-e2e

/test sagemaker-kind-e2e

/test sagemaker-kind-e2e

/test sagemaker-kind-e2e

Too many file changes to review
Can you mention which files did you modify to get the Tags to work ?
Integration test changes look fine

/approve
/retest

/test sagemaker-kind-e2e


Added next tutorial chapter(toh-pt5) link

toh-pt4.html was missing next tutorial chapter link and tutorial flow was getting interrupted by this.
I've added next tutorial chapter link(toh-pt5) with corresponding text.

I signed it!

Good catch


"Fix-on-Save" Produces Incorrect Output

When enabling "fix on save", vscode-eslint appears to produce different results than it would if you were to explicitly run the "ESLint: Fix all auto-fixable problems" command.
I've managed to correct this behavior by overriding the mode in (resolveSettings](https://github.com/microsoft/vscode-eslint/blob/master/server/src/eslintServer.ts#L1933) by forcing it to treat all fix requests as though they originated from the command.
By the time resolveSettings completes in the attached example, "\n;\n" is included in the results.
The attached example illustrates my point a bit better. Using the included workspace settings, saving index.js will produce a different result than if a user were to explicitly call "ESLint: Fix all auto-fixable problems" or run ESLint from the CLI.
Original Source
const myconst = "hello"
const anothervar = 4

"ESLint: Fix all auto-fixable problems" output:
(eslint cli produces these same results)
const myconst = "hello";
const anothervar = 4;

Fix on Save Output
const myconst = "hello";
const anothervar = 4
;

I've also included lint.js which can be used to see what ESLint's raw message set is, as well as to see how ESLint's CLI resolves file edits (apply_edits is taken almost entirely directly from ESLint's CLI source).
vscode-eslint-bug.tar.gz

@SE2Dev thanks for the great test case.

@SE2Dev thanks for the great test case.

@dbaeumer I just wanted to give a heads up. Although this issue was resolved in the 2.1.20 release, 2.1.25 appears to have a regression regarding this issue.

@SE2Dev does the same test case reproduces it?

Yep

Can again reproduce. Thanks for reporting.


adds .components.yaml for being referenced in the nicegui solutions bazaar

aill automaticall picked up on update at http://ngdemo.bitplan.com/solutions

@WolfgangFahl Can you, please, elaborate what this file is supposed to do? Can you link to some online documentation explaining the "NiceGUI solutions" project? And I don't mean https://github.com/zauberzeug/nicegui/discussions/1618, which can't serve as a documentation page in the long run, but a standalone explanation, e.g. hosted at http://ngdemo.bitplan.com/solutions?
At the moment I'm confused that nicegui-highcharts is already listed, even though .components.yaml isn't merged. And the guide at https://github.com/zauberzeug/nicegui/discussions/1618#discussioncomment-7869878 doesn't explain why, e.g., livesync is listed, even though it doesn't have a "nicegui" tag. It only mentions it somewhere in the README.

@falkoschindler i already started doing this at https://github.com/WolfgangFahl/nicegui_widgets/issues/52


Contribution page docs

For #101

I noticed npm run lint won't run using just the instructions in CONTRIBUTING.md. I think a dependency is missing.
▶ npm run lint

><EMAIL_ADDRESS>lint /Users/marcel/Documents/code/tmp/filestack-react
> eslint src/**/*.js src/**/*.jsx tests/**/*spec.js && tsc --noEmit

Oops! Something went wrong! :(

ESLint: 7.5.0

ESLint couldn't find the config "prettier" to extend from. Please check that the name of the config is correct.

The config "prettier" was referenced from the config file in "/Users/marcel/Documents/code/tmp/filestack-react/.eslintrc".

If you still have problems, please stop by https://eslint.org/chat to chat with the team.

npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>lint: `eslint src/**/*.js src/**/*.jsx tests/**/*spec.js && tsc --noEmit`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>lint script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/marcel/.npm/_logs/2020-10-19T21_15_40_508Z-debug.log

I figured it out, eslint-config-prettier is missing. Should I go ahead and add that to devDependencies as part of this PR? I hesitate because it pulls in a few other libs, I see a ton of warnings about peer dependencies, and it looks like eslint warns on the existing code.
▶ npm run lint

><EMAIL_ADDRESS>lint /Users/marcel/Documents/code/tmp/filestack-react
> eslint src/**/*.js src/**/*.jsx tests/**/*spec.js && tsc --noEmit

/Users/marcel/Documents/code/tmp/filestack-react/src/ReactFilestack.jsx
   6:3  error  'defaultProps' should be declared outside the class body  react/static-property-placement
  22:3  error  'propTypes' should be declared outside the class body     react/static-property-placement

/Users/marcel/Documents/code/tmp/filestack-react/tests/components/ReactFilestack.spec.js
  47:21  error  Prop spreading is forbidden  react/jsx-props-no-spreading


Present the list of dependencies for each repository

Being able to see dependency information at a glance will be one of the primary benefits of Towtruck. This PR presents a first iteration for displaying this information, which is sourced from the Renovate Dependency Dashboard (if Renovate is configured to create one for a given repository). It does not currently display information about new releases or expected end-of-life dates.
This PR implements issue #17.
Below is a preview of these changes:

When the dependency list is collapsed:
When the dependency list is expanded:

@richpjames I think Renovate is probably our best bet.

Most of dxw's repos don't have the dashboard, I had a look at 200 and only 3 of them had a dashboard.

There could be an argument to be made here that setting up Renovate to automatically update minor dependency versions and automatically create PRs for major versions would be an important step to reduce the manual workload in maintaining our repos.

Of the dashboards that do it exist, it doesn't seem able to show the version for a lot of the dependencies. For example for a lot Ruby Gems it just shows undefined:

Looking at some Gemfiles where Renovate reports the version as undefined, there's no version specified (for example here). I don't know enough about Ruby to know if there is even a way to know what versions these will resolve to without actually building and running the project.

We are using regex to parse the body of the Renovate dashboard. If the format of this body changed then the dashboard would break. This dashboard data could be useful to guide decision but I'm not sure it would be the primary source of information.

Parsing them from the issue body text isn't ideal. The alternative would be to use the Renovate CLI tool, but then we'd need Towtruck to clone every repository and run it. Directly parsing dependency information from build scripts to me is a non-starter as at that point we're building a worse Renovate that will require manual intervention if we want Towtruck to be useful for a project in a new language.


imdb.com downloading a trailer from a specific webpage

[x] I've verified and I assure that I'm running youtube-dl 2018.04.25
[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser
[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

C:\Users\Computer User\Downloads>youtube-dl -v https://www.imdb.com/videoplayer/
vi539015193?ref_=ttvi_vi_imdb_2
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.imdb.com/videoplayer/vi539015193?
ref_=ttvi_vi_imdb_2']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2018.04.25
[debug] Python version 3.4.4 (CPython) - Windows-7-6.1.7601-SP1
[debug] exe versions: avconv 11.3, avprobe 11.3, ffmpeg N-83657-g7e4f32f, ffprob
e N-83657-g7e4f32f
[debug] Proxy map: {}
[imdb] 539015193: Downloading webpage
WARNING: unable to extract description; please report this issue on https://yt-d
l.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to
update. Be sure to call youtube-dl with the --verbose flag and include its compl
ete output.
[imdb] 539015193: Downloading player page
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on 'https://imdb-video.media-imdb.com/vi539015193/hl
s-1520347889278-master.m3u8?Expires=1524906680&Signature=KBqFL0ToS36FQ8lCSvlXxLP
TxPOKXEDExKq9GOqDLJkP6QoMg4jIWlM3XQ3ceu3KE0W8ryBCoeiX1Ol2PSS93KO9SfplpJHfC9JN2Xm
eMK1DfjuKH2Aza9uVqlv9IAHGi6GM2U272bXcriRMC~KKnyQ9f5tRYWpjRQbHsOdS~x5RP74DcND3CZv
ao22bLodFUlt~kph~6XUuv~b8FjUacLnTgHbFVuLxKU0WLn7sGI71h8g~sQ7sXgfQE1ienT8CAQwoYaA
3M8a~s-YZVmoVj3Spt5QAoZWGuEegHsHO~yRSW3ISXC8d3v6c448L18P2gigNJCg38mw1Juy0krYJmQ_
_&Key-Pair-Id=APKAIFLZBVQZ24NQH3KA'
[download] Destination: Official Trailer from Lost in Space (2018-)-539015193.m3
u8
[debug] avconv command line: avconv -y -loglevel verbose -headers "Accept-Langua
ge: en-us,en;q=0.5
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Accept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7
Accept-Encoding: gzip, deflate
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:59.0) Gecko/20100101 Firefox/59.0
 (Chrome)
" -i "https://imdb-video.media-imdb.com/vi539015193/hls-1520347889278-master.m3u
8?Expires=1524906680&Signature=KBqFL0ToS36FQ8lCSvlXxLPTxPOKXEDExKq9GOqDLJkP6QoMg
4jIWlM3XQ3ceu3KE0W8ryBCoeiX1Ol2PSS93KO9SfplpJHfC9JN2XmeMK1DfjuKH2Aza9uVqlv9IAHGi
6GM2U272bXcriRMC~KKnyQ9f5tRYWpjRQbHsOdS~x5RP74DcND3CZvao22bLodFUlt~kph~6XUuv~b8F
jUacLnTgHbFVuLxKU0WLn7sGI71h8g~sQ7sXgfQE1ienT8CAQwoYaA3M8a~s-YZVmoVj3Spt5QAoZWGu
EegHsHO~yRSW3ISXC8d3v6c448L18P2gigNJCg38mw1Juy0krYJmQ__&Key-Pair-Id=APKAIFLZBVQZ
24NQH3KA" -c copy -f mp4 "file:Official Trailer from Lost in Space (2018-)-53901
5193.m3u8.part"
avconv version 11.3, Copyright (c) 2000-2014 the Libav developers
  built on Mar  9 2015 02:27:07 with gcc 4.8.3 (Gentoo 4.8.3 p1.1, pie-0.5.9)
  configuration: --cross-prefix=i686-w64-mingw32- --arch=i686 --sysroot=/usr/i68
6-w64-mingw32/ --enable-runtime-cpudetect --enable-shared --enable-cross-compile
 --enable-w32threads --prefix=/usr --target-os=mingw32 --enable-memalign-hack --
enable-gpl --enable-version3 --enable-libmp3lame --enable-libopus --enable-libvo
-aacenc --enable-libvorbis --enable-libvpx --enable-libx264
  libavutil     54.  3. 0 / 54.  3. 0
  libavcodec    56.  1. 0 / 56.  1. 0
  libavformat   56.  1. 0 / 56.  1. 0
  libavdevice   55.  0. 0 / 55.  0. 0
  libavfilter    5.  0. 0 /  5.  0. 0
  libavresample  2.  1. 0 /  2.  1. 0
  libswscale     3.  0. 0 /  3.  0. 0
https://imdb-video.media-imdb.com/vi539015193/hls-1520347889278-master.m3u8?Expi
res=1524906680&Signature=KBqFL0ToS36FQ8lCSvlXxLPTxPOKXEDExKq9GOqDLJkP6QoMg4jIWlM
3XQ3ceu3KE0W8ryBCoeiX1Ol2PSS93KO9SfplpJHfC9JN2XmeMK1DfjuKH2Aza9uVqlv9IAHGi6GM2U2
72bXcriRMC~KKnyQ9f5tRYWpjRQbHsOdS~x5RP74DcND3CZvao22bLodFUlt~kph~6XUuv~b8FjUacLn
TgHbFVuLxKU0WLn7sGI71h8g~sQ7sXgfQE1ienT8CAQwoYaA3M8a~s-YZVmoVj3Spt5QAoZWGuEegHsH
O~yRSW3ISXC8d3v6c448L18P2gigNJCg38mw1Juy0krYJmQ__&Key-Pair-Id=APKAIFLZBVQZ24NQH3
KA: Protocol not found

ERROR: ffmpeg exited with code 1
  File "__main__.py", line 19, in <module>
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\bu
ild\youtube_dl\__init__.py", line 471, in main
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\bu
ild\youtube_dl\__init__.py", line 461, in _real_main
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\bu
ild\youtube_dl\YoutubeDL.py", line 1993, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\bu
ild\youtube_dl\YoutubeDL.py", line 800, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\bu
ild\youtube_dl\YoutubeDL.py", line 854, in process_ie_result
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\bu
ild\youtube_dl\YoutubeDL.py", line 1627, in process_video_result
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\bu
ild\youtube_dl\YoutubeDL.py", line 1900, in process_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\bu
ild\youtube_dl\YoutubeDL.py", line 1839, in dl
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\bu
ild\youtube_dl\downloader\common.py", line 365, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\bu
ild\youtube_dl\downloader\external.py", line 64, in real_download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\bu
ild\youtube_dl\downloader\common.py", line 166, in report_error
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\bu
ild\youtube_dl\YoutubeDL.py", line 617, in report_error
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpad1hmn8r\bu
ild\youtube_dl\YoutubeDL.py", line 579, in trouble

C:\Users\Computer User\Downloads>
...
<end of log>

youtube-dl is a great program, because it has lots of features, and usually downloads videos reliably from a variety of websites in a wide range of formats, but on "https://www.imdb.com/videoplayer/vi539015193?ref_=ttvi_vi_imdb_2" which is a page that has a specific movie trailer, it doesn't seem to be able to download the video, but the video plays fine in browsers, and other programs and browser add-ons can download it fine as well. Is youtube-dl having trouble downloading it, because it doesn't have the correct download url, and if so, how can an average user know how to find the correct download url for videos on different websites, if they put the url for the page where the videos play properly, and youtube-dl doesn't download them, shouldn't youtube-dl be able to automatically find the correct download url for a video if given the url for the page where the video is? Is there a way to fix that? Thanks.

avconv built without https support. Either fix or use ffmpeg.

I'm just an average computer user and I don't really know what that means. I think what you're saying is that avconv is a command line program that youtube-dl uses to convert the media files it downloads, and that it doesn't support https which somehow is related to the format that the imdb page with the video uses. I wouldn't know how to fix avconv to use https, but I'm familiar with some basic command line options for ffmpeg, so if you could please tell me how to use ffmpeg with youtube-dl to download this particular video trailer from the imdb page I'd appreciate it, or if you can tell me where I can find out information about how to do that, that would be good as well. Thanks.

Do you know how to download videos from "https://rg3.github.io/youtube-dl/supportedsites.html" and "https://www.imdb.com/title/tt5232792/?ref_=nv_sr_1" with youtube-dl? Thanks.


Add support for CSP script-src 'report-sample'

The 'report-sample' directive causes Chrome to send the first ~20 characters of the script that violated the CSP directive, which is amazingly useful for tracking down the inline script you missed. (Firefox used to do that anyway, but I believe requires the directive now).
Can this be added to NWebsec please?

Relevant links:

https://www.chromestatus.com/feature/5792234276388864
https://github.com/mdn/browser-compat-data/pull/2761


Build failure with aeson 0.10

Building ndjson-conduit-<IP_ADDRESS>...
Preprocessing library ndjson-conduit-<IP_ADDRESS>...
[1 of 1] Compiling Data.Conduit.JSON.NewlineDelimited ( src/Data/Conduit/JSON/NewlineDelimited.hs, dist/build/Data/Conduit/JSON/NewlineDelimited.o )

src/Data/Conduit/JSON/NewlineDelimited.hs:45:19:
    Not in scope: ‘A.encodeToByteStringBuilder’

Also see http://matrix.hackage.haskell.org/package/ndjson-conduit#GHC-7.10/ndjson-conduit-<IP_ADDRESS>
Stackage just upgraded to aeson-0.10 which means I need to disable ndjson-conduit for now. Please send a pull request to re-enable it when this is fixed. If this happens before new years ndjson-conduit will still be included in LTS 4.

Thanks, I'll see what I can do.

@bergmark Published a new version to cabal (<IP_ADDRESS>) that should fix the issue.

Thanks!


chore: prepare for public beta

Types

[x] 🧹 Chores

Codecov Report

Merging #532 (51b79f5) into main (b291e8c) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

:exclamation: Current head 51b79f5 differs from pull request most recent head 1940964. Consider uploading reports for the commit 1940964 to get more accurate results

@@            Coverage Diff             @@
##             main     #532      +/-   ##
==========================================
- Coverage   59.01%   59.01%   -0.01%
==========================================
  Files        1200     1200
  Lines       73924    73926       +2
  Branches    15300    15301       +1
==========================================
  Hits        43625    43625
- Misses      27617    27619       +2
  Partials     2682     2682

Impacted Files
Coverage Δ

...xt/src/browser/services/file-tree-model.service.ts
54.00% <50.00%> (-0.01%)
:arrow_down:

packages/core-common/src/node/port.ts
43.93% <0.00%> (-3.04%)
:arrow_down:

...erences/src/browser/preference-settings.service.ts
79.86% <0.00%> (+0.69%)
: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 b291e8c...1940964. Read the comment docs.


#34 feature/34 full redux javascript implementation

Good stuff, reviewed, checked locally, and played with the redux dev tools :)


New dataset: A29 Täglich aktualisierte Luftqualitätsmessungen, seit 1983

Proposal to include dataset: Täglich aktualisierte Luftqualitätsmessungen, seit 1983
Dataset properties

URL: https://data.stadt-zuerich.ch/dataset/ugz_luftschadstoffmessung_tageswerte
Organization: Stadt Zürich
format: csv
size: 20MB (daily data, will be aggregated monthly)
dimensions: CO, O3, O3_max_h1,O3_nb_h1>120, NO, NO2, NOx, SO2, PM2.5, PM10, PN
units: 1/cm3, ug/m3,
lang:d e

Additional notes

Dataset is provided with one file per year. They will need to be downloaded sequentially and combined
Daily records -> aggregate by month

Questions

There are multiple locations within Zurich which are not present in our spatialunits. Should we keep them (as strings) or average them into a single "Stadt Zürich" location
Should we create additional indices upon aggregation (min/max/mean)?

@cmdoret
In general I think the proposals in both questions are reasonable. My suggestions are only driven by simplification:

I would pick one station which is present over all datasets (all years) and only use this as a representation of the city of Zurich. This can be explained in the metadata table (if needed)
I would not create further indices as this could also be created with SQL Code/by the model.

But in the end I guess it is something also the team of the ZHAW could weigh in @nooralahzadeh
Thanks for the effort :)


feat(launch): better docker image job name handling, tag -> alias

Fixes
https://wandb.atlassian.net/browse/WB-13933
Description
Remove tag from the job name, move to alias. Launching a new docker image with the same name but different tag logs a new version, with correct aliases.

🤖 Generated by Copilot at 0d7c045
This pull request enables job artifacts to include docker image tags as part of their metadata. It updates the job_builder.py module to handle the tag attribute of image sources.

We'll need to ship a docs PR change for this as well @gtarpenning

https://github.com/wandb/docodile/pull/292

/fix-title


Reordering Doc Type Editors on a page not allowed

When trying to reorder a Doc Type editor in a page, I cannot seem to move it anywhere. In umbraco 8.11.1

That sounds like an Umbraco bug


[BUG] Assertion failure, file index :cufio-udev

Describe the bug
error when importing cucim:
Python 3.8.10 (default, Jun 22 2022, 20:18:18)
[GCC 9.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import cucim
Assertion failure, file index :cufio-udev  line :134
Aborted (core dumped)

Expected behavior
no error when importing
Environment details (please complete the following information):

Environment location: Ubuntu 20.04.4 LTS
NVIDIA-SMI 515.48.07    Driver Version: 515.48.07    CUDA Version: 11.7
Method of cuCIM install: pip install cucim (cucim-22.6.0)

Sorry this ran into issues at import.
Does running nvidia-smi work? If so, what does that show?

nvida-smi works fine, correctly display the gpu info. here is the first row from nvidia-smi NVIDIA-SMI 515.48.07 Driver Version: 515.48.07 CUDA Version: 11.7

Hi @wyli Thank you for the report
The error seems to be related to GDS (GPUDirect Storage) library that would be installed by default from CUDA 11.7.
Could you please share the output of the following commands?
/usr/local/cuda/gds/tools/gdscheck -v
/usr/local/cuda/gds/tools/gdscheck -p

lspci -tv

If you can find cufile.log file in the current folder, please also share the content.
cucim is imported in the bare metal system (not docker container)?
What happens if you install cucim inside container?

thanks, I put the logs here:

nvidiauser@ams-monai:~$ /usr/local/cuda/gds/tools/gdscheck -v
Assertion failure, file index :cufio-udev  line :134
Aborted (core dumped)
nvidiauser@ams-monai:~$ /usr/local/cuda/gds/tools/gdscheck -p
 ============
 ENVIRONMENT:
 ============
 =====================
 DRIVER CONFIGURATION:
 =====================
 NVMe               : Unsupported
 NVMeOF             : Unsupported
 SCSI               : Unsupported
 ScaleFlux CSD      : Unsupported
 NVMesh             : Unsupported
 DDN EXAScaler      : Unsupported
 IBM Spectrum Scale : Unsupported
 NFS                : Unsupported
 BeeGFS             : Unsupported
 WekaFS             : Unsupported
 Userspace RDMA     : Unsupported
 --Mellanox PeerDirect : Disabled
 --rdma library        : Not Loaded (libcufile_rdma.so)
 --rdma devices        : Not configured
 --rdma_device_status  : Up: 0 Down: 0
 =====================
 CUFILE CONFIGURATION:
 =====================
 properties.use_compat_mode : true
 properties.force_compat_mode : false
 properties.gds_rdma_write_support : true
 properties.use_poll_mode : false
 properties.poll_mode_max_size_kb : 4
 properties.max_batch_io_size : 128
 properties.max_batch_io_timeout_msecs : 5
 properties.max_direct_io_size_kb : 16384
 properties.max_device_cache_size_kb : 131072
 properties.max_device_pinned_mem_size_kb : 33554432
 properties.posix_pool_slab_size_kb : 4 1024 16384
 properties.posix_pool_slab_count : 128 64 32
 properties.rdma_peer_affinity_policy : RoundRobin
 properties.rdma_dynamic_routing : 0
 fs.generic.posix_unaligned_writes : false
 fs.lustre.posix_gds_min_kb: 0
 fs.beegfs.posix_gds_min_kb: 0
 fs.weka.rdma_write_support: false
 profile.nvtx : false
 profile.cufile_stats : 0
 miscellaneous.api_check_aggressive : false
 =========
 GPU INFO:
 =========
 GPU index 0 Tesla T4 bar:1 bar size (MiB):256 supports GDS
 ==============
 PLATFORM INFO:
 ==============
Assertion failure, file index :cufio-udev  line :134
Aborted (core dumped)
nvidiauser@ams-monai:~$ lspci -tv
-+-[b69d:00]---02.0  Mellanox Technologies MT27710 Family [ConnectX-4 Lx Virtual Function]
 +-[0001:00]---00.0  NVIDIA Corporation TU104GL [Tesla T4]
 \-[0000:00]-+-00.0  Intel Corporation 440BX/ZX/DX - 82443BX/ZX/DX Host bridge (AGP disabled)
             +-07.0  Intel Corporation 82371AB/EB/MB PIIX4 ISA
             +-07.1  Intel Corporation 82371AB/EB/MB PIIX4 IDE
             +-07.3  Intel Corporation 82371AB/EB/MB PIIX4 ACPI
             \-08.0  Microsoft Corporation Hyper-V virtual VGA

nvidiauser@ams-monai:~$ cat cufile.log
 26-07-2022 20:51:21:996 [pid=3955 tid=3955] NOTICE  cufio-drv:705 running in compatible mode
 26-07-2022 21:02:43:171 [pid=4944 tid=4944] NOTICE  cufio-drv:705 running in compatible mode
 27-07-2022 17:31:02:141 [pid=2223 tid=2223] NOTICE  cufio-drv:705 running in compatible mode
 27-07-2022 17:31:10:495 [pid=2228 tid=2228] NOTICE  cufio-drv:705 running in compatible mode

I ran into the same problem, any solution available yet?

I also get this error:
>>import cucim
Assertion failure, file index :cufio-udev  line :134
Aborted (core dumped)

I'm running a NV6ads_A10_v5 virtual machine with ubuntu 22.04.
NVIDIA-SMI 535.154.05             Driver Version: 535.154.05   CUDA Version: 12.2
RAPIDS and cucim installed with conda
Name                    Version                   Build  Channel
cucim                     24.04.00        cuda11_py39_240410_ga24abfd_0    rapidsai
libcucim                  24.04.00        cuda11_240410_ga24abfd_0    rapidsai
Any solution to this problem in the pipeline? I really want to run cucim.


Add a starfield generator for background (given vector)

Make a background mapper which can deterministically recreate a starfield given some R3::vector as input.

This will require a method to convert the vector to some 2d spherical coordinates which are then fed into a function to produces deterministic stars. Perhaps a seed can be given.


v0.2.1

Description
In this release, we have added

added support for scikit-learn<=1.4.2 and pandas<=2.2.2,
removed Python 3.7 support,
moved from setup.cfg/setup.py to pyproject.toml, and
migrated to ruff for QA checks and formatting.

There should be no user-facing API changes in this release.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.85%. Comparing base (826df41) to head (674d8d2).
Report is 3 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #19      +/-   ##
==========================================
+ Coverage   97.48%   97.85%   +0.36%
==========================================
  Files           3        5       +2
  Lines         239      280      +41
==========================================
+ Hits          233      274      +41
  Misses          6        6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Integer field where clause doesn't work properly

I created a class with a long field, accountId
I inserted some records
when I try to query those records it doesn't work as I expect.
All records has the same accountId, 232330
I have total of 16 records.
The class name is edgebase
I tried the below queries:
select count(*) from edgebase where accountId = 232330
PROPERTIES
count
6 (expected 16)
select count(*) from edgebase where accountId = '232330'
PROPERTIES
count
6 (expected 16)
select count(*) from edgebase
count
16 (this is ok)
select count(*) from edgebase where accountId like '232330'
PROPERTIES
count
16 (this is the correct result, but why is it different from =)
select count(*), accountId from edgebase group by accountId
PROPERTIES
count accountId
16    232330
(when using group by, the DB group all the rows with the same accountId)
Please help
using:
Studio version : 2.1
OrientDB version : 2.1.8
thanks

hi @arikalon
can you provide a script sql to populate the database and reproduce this issue?

below is the DB dump.
once imported, you can run the above queries on edgebase class
export.json.gz

some additional input ...
When I removed the index from edgebase class, the queries worked properly
Adding it back, brought back the problem

ok @arikalon
i'm checking it.

hi @arikalon
Which is the index?
EdgeBase.edgeKey?

hi @arikalon
I think you used a wrong index type for EdgeBase.edgeKey.
You used dictionary that has this exact behaviour
http://orientdb.com/docs/2.1/Indexes.html#index-types
You should use NOT_UNIQUE instead as you can have duplicates for the same
key [accountId,srcNode,destNode]

I think it's really bad that choosing the wrong index type can lead to such a behavior
I'll check the NOT_UNIQUE index type
thanks

hi @arikalon
It is just a type of index (DICTIONARY) that act in this way. For the same key just replace the indexed document instead of give the duplicate exception. In your case you have duplicate keys so you should go with NOT_UNIQUE

hi @arikalon
did you solve with NOT_UNIQUE?

yes, it did
Thanks for the help

:+1:


Dependecy upgrades

Instead of many small PR's with version updates, this will update most of our dependencies, so if we can test this branch and make it work we are a big step in the right direction. @roblabs  @ntadej
The CI/CD environments are generally a bit outdated, so let's update those first ( see #387 ).

Do not update maplibre-gl-js in this one. It will not work due to shaders.

Okay, I have reverted the change to submodule.

I reverted the bump of benchmark from 1.5 to 1.7, because google apparently against semver introduced breaking changed in the 1.6 minor release.

The CI OS versions commit may be more suitable for a separate PR, but it's probably fine here :)

The CI OS versions commit may be more suitable for a separate PR, but it's probably fine here :)

I think you are very right about that. There is a PR in here #388

closed in favour of smaller increments.


add 24.1 known limitations; reorganize existing KLs

DOC-10100
Preview

@DrewKimball @yuzefovich This looks like a bunch of changes, but all that needs to be reviewed is in the Preview link in the description, or known-limitations.md. Let me know if you have questions.
@michae2 Feel free to just review the addition to read-committed-limitations.md. I didn't include it in the "New" section since it was existing behavior (IIUC).

@mikeCRL PTAL at my includes here -- the idea is that the new KLs are added to:

the main Known Limitations page, directly.
the existing includes that appear on the respective (PL/pgSQL, UDF, stored procs) pages as well as under the "existing" heading on the Known Limitations page.

Within the includes, the new KLs for 24.1 are contained in a liquid block (which mentions v24.1 in an inline comment) that prevents them from being displayed on the Known Limitations page (i.e., appearing twice). The idea is that when we move the "new" KLs into the "existing" section later, we can:

delete everything under the "new" heading on the Known Limitations page.
move them out of the liquid blocks on the includes, thus causing them to appear in the Known Limitations "existing" section.

This sounds complicated but is hopefully intuitive once you look at it. Open to counter-suggestions & adjustments ofc.


locale file not found

  ERROR  Failed to resolve import "monaco-editor-nls/locale/zh-cn.json" from "node_modules\.pnpm\nuxt-monaco-editor@1.2.6_monaco-editor@0.45.0_vite@4.5.1\node_modules\nuxt-monaco-editor\dist\runtime\nls.mjs?v=a493b7b0". Does the file exist?

the doc need update. the right locale name is zh-hans.
==========
when use this settings:
  monacoEditor: { locale: 'de' },

will get

500
nls.localize2 is not a function

at new SelectToBracketAction (http://localhost:3000/_nuxt/node_modules/.pnpm/monaco-editor@0.45.0/node_modules/monaco-editor/esm/vs/editor/contrib/bracketMatching/browser/bracketMatching.js?v=05d667ad:47:34)
at registerEditorAction (http://localhost:3000/_nuxt/node_modules/.pnpm/monaco-editor@0.45.0/node_modules/monaco-editor/esm/vs/editor/browser/editorExtensions.js?v=05d667ad:318:20)
at http://localhost:3000/_nuxt/node_modules/.pnpm/monaco-editor@0.45.0/node_modules/monaco-editor/esm/vs/editor/contrib/bracketMatching/browser/bracketMatching.js?v=05d667ad:340:1

the nls.localize2 error seems appear in monaco-editor-0.45.0, when goes back to version 0.39.0, everythings ok.

It should be fixed in v1.2.9. Please try it.

locale file is fixed.
but a new problem is: the context menu not closed when click the blank.


Fix transparency in TMS demo page

The logic for the demo page using TMS doesn't currently support transparency due to incorrect logic

Thanks for fix!


.mkv attachements not coped properly

bug: in .mkv file format stream attachemet (picture, png) coped, but not set disposition to attached_pic
direct key like "-disposition:v:1 attached_pic" also not worked

Check this

Is that an issue specific to these builds? If not, please report it on ffmpeg trac and close this issue.

Yeah he's right

update issue: if .mkv file file contains a tag or global tag, bug is work. Delete tags from file, and attached pic coped properly.

Is that an issue specific to these builds? If not, please report it on ffmpeg trac and close this issue.

I'm use only these builds

That's still not an issue specific to the builds. Please report issues with ffmpeg itself on FFmpeg trac or ask for support on ffmpeg-users.


pkg/references: Replace trace package with custom complex errors

Example of the errors which the damon prints on the startup:
Error:
(UnableToResolveFlow) failed to resolve flow 'FetchLatestProject'
        Name: "failed to resolve flow 'FetchLatestProject'"
(UndefinedReference) undefined reference 'user:password' in 'output.Password'
        Reference: {"resource":"user","path":"password"}
        Breakpoint: "output"
        Path: "Password"
(UndefinedReference) undefined reference 'user:password' in 'output.Password'
        Reference: {"resource":"user","path":"password"}
        Breakpoint: "output"
        Path: "Password"
(UndefinedReference) undefined reference 'user:password' in 'output.Password'
        Reference: {"resource":"user","path":"password"}
        Breakpoint: "output"
        Path: "Password"
(UndefinedResource) undefined resource 'user:password' in 'output'
        Reference: {"resource":"user","path":"password"}
        Breakpoint: "output"
        KnownReferences: ["stack","input","error","query","user"]

the repeated UndefinedReference error is related to the references package: that's how the Resolve method resolves the flow. We should probably investigate what happens here.

Hi @azzz, thanks for your PR. The tests inside the reference package are currently failing could you please fix these test?: https://github.com/jexia/semaphore/pull/117/checks?check_run_id=1047248501#step:5:1329
--- FAIL: TestUnmarshalFile (0.10s)
    --- FAIL: TestUnmarshalFile/proxy_forward_header.fail.hcl (0.00s)
        references_test.go:82: unexpected error message failed to resolve flow 'echo', expected undefined reference 'input:unknown' in 'echo.output.Authorization'

@jeroenrinzema yes, sure, I will fix the tests. But what do you think about the general idea?
BTW, I feel NiceError should be moved to a package as a standard way to wrap and print errors with several formatters: text, json, grpc, whatever.

Codecov Report

Merging #117 into master will decrease coverage by 2.42%.
The diff coverage is 22.64%.

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
- Coverage   85.57%   83.15%   -2.43%
==========================================
  Files          82       86       +4
  Lines        4687     4874     +187
==========================================
+ Hits         4011     4053      +42
- Misses        411      551     +140
- Partials      265      270       +5

Impacted Files
Coverage Δ

pkg/references/errors.go
2.17% <2.17%> (ø)

pkg/references/references.go
60.90% <21.91%> (-11.19%)
:arrow_down:

pkg/pretty_errors/text_formatter.go
43.75% <43.75%> (ø)

pkg/pretty_errors/pretty_error.go
55.55% <55.55%> (ø)

pkg/pretty_errors/prettifier.go
100.00% <100.00%> (ø)

Codecov Report

Merging #117 into master will decrease coverage by 2.42%.
The diff coverage is 22.64%.

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
- Coverage   85.57%   83.15%   -2.43%
==========================================
  Files          82       86       +4
  Lines        4687     4874     +187
==========================================
+ Hits         4011     4053      +42
- Misses        411      551     +140
- Partials      265      270       +5

Impacted Files
Coverage Δ

pkg/references/errors.go
2.17% <2.17%> (ø)

pkg/references/references.go
60.90% <21.91%> (-11.19%)
:arrow_down:

pkg/prettyerr/text_formatter.go
43.75% <43.75%> (ø)

pkg/prettyerr/error.go
55.55% <55.55%> (ø)

pkg/prettyerr/prettifier.go
100.00% <100.00%> (ø)

Codecov Report

Merging #117 into master will decrease coverage by 1.98%.
The diff coverage is 30.66%.

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
- Coverage   86.28%   84.29%   -1.99%
==========================================
  Files          85       89       +4
  Lines        4957     5144     +187
==========================================
+ Hits         4277     4336      +59
- Misses        413      536     +123
- Partials      267      272       +5

Impacted Files
Coverage Δ

pkg/references/errors.go
16.30% <16.30%> (ø)

pkg/references/references.go
61.81% <21.91%> (-11.44%)
:arrow_down:

pkg/prettyerr/text_formatter.go
43.75% <43.75%> (ø)

pkg/prettyerr/error.go
77.77% <77.77%> (ø)

pkg/prettyerr/prettifier.go
100.00% <100.00%> (ø)


Smart quotes not treated properly?

when i try to use quotes like this:
md = """
'single'

"double"

'"double in single"'

"'single in double'"

"""
settings =
  typographer:  true
  quotes:       """«»()"""
MarkdownIt  = require 'markdown-it'
parser      = new MarkdownIt settings
console.log parser.render md

the result looks like this:

(single)

«double»

("double in single")

«’single in double’»

(Note that i used these specific replacements only in order to get a clearly discernible output).
i think this is problematic on more than one count.
First, how quotes are translated seems to be depending on whether they're enclosed in another pair of quotes. If there is any argument for this kind of behavior, it should be configurable.
Next, replacing a " in the source by a &quot entity (i.e. U+0022) may make sense in the context of an HTML attribute; in the context of an HTML text node, it's clutter.
Also, single quotes within double quotes do get replaced—but not by what was configured; rather, ’ characters (Right Single Quotation Marks, U+2019) appear in the target text. This may be justified by saying that some ASCII single quotes cannot be unambiguously interpreted as either opening or closing, and, hence, should be read as an apostrophe. Then again, the typogr module appears to get these cases right.
(As an aside, the configuration format—a single string with four characters (codepoints?)—is quite limiting and the documentation not very clear: "when typographer enabled, and smartquotes on"—where do i have to turn smartquotes on?).

I'd prefer js example + demo permalink.
At first glance things you describe are ok. Smartquotes are matched by pairs, and sequence is important escaping is done because HTML require it. Logic is here https://github.com/markdown-it/markdown-it/blob/master/lib/rules_core/smartquotes.js and it doen't make escaping itself.
About configurable options - i'm against cluttering API. Idea is to keep options count minimal, and move everything else to plugins.

@loveencounterflow , please create a demo permalink with examples, it will be more easy to understand.
- input1
- `expected1

- input2
- `expected2`

...

If smartquotes have problem in logic, those will be fixed.

care to explain what you mean by "demo permalink"? I looked at https://markdown-it.github.io/, but i don't get it to work.

Open https://markdown-it.github.io/ (it works, at least in my FF, just checked)
Type your exa,ple text
Copy URL of small permalink tav above text area.

If something doesn't work - let me know.

I've patched CLI script with «»() quotes option, end executed this example:
'single'

"double"

'"double in single"'

"'single in double'"

Output is:

(single)

«double»

("double in single")

«’single in double’»

We have test for nested quotes https://github.com/markdown-it/markdown-it/blob/master/test/fixtures/markdown-it/smartquotes.txt .
As far as i see, in current example, difference is coused by improper detection of opening/closing pair.

nested pairs are not processed
the rest of single quotes are interpreted as apostrophes (looks like mistake)

May be, we could use the same logic as for emphasis. Tough, it will fail in case of nested markup, because it's hard to ignore such case as "opening":
foo"bar

Fixed in https://github.com/markdown-it/markdown-it/commit/552c130b21b2c731a597b00b17d4db045c1099ec

This markdown text gives a 500 : Internal Server Error
nbconvert failed: PDF creating failed
on trying to Download from an IPython Notebook as PDF via LaTeX:
Any old text 'Some text' some more text.
Converting it to LaTeX and adding \usepackage{textcomp} fixes it as suggested at https://github.com/ipython/ipython/issues/7793 but it's tortuous having to do that every time.


InfluxData downloads page points to wrong URL of InfluxDB build

When I navigate to https://portal.influxdata.com/downloads I select:
Influxdb > Stable > Amd64 Ubuntu

and click the "Download" button which brings me to a page that reads:
Download Influxdb (Stable) for Amd64 Ubuntu
wget https://dl.influxdata.com/telegraf/releases/influxdb_1.2.0_amd64.deb
sudo dpkg -i influxdb_1.2.0_amd64.deb

MD5: f79959802930c7e9d6e81ffd366c0629

The URL provided is incorrect and returns a 404 Not Found response from the file server. The correct URL should be:
wget https://dl.influxdata.com/influxdb/releases/influxdb_1.2.0_amd64.deb

I suspect there are few other broken links as well.

See #7993

Stumbled across this as well, please fix.

Closing this since download links are maintained in a different repo: https://github.com/influxdata/docs.influxdata.com/issues/1014


<EMAIL_ADDRESS>update hash

issue https://github.com/lukesampson/scoop-extras/issues/4167

/verify

Duplicate of #4168


shims/super/cc: pass -ld_classic if needed for -dead_strip_dylibs

[x] Have you followed the guidelines in our Contributing document?
[x] 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.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Good catch, thanks @cho-m!


[time] basic implementation

This looks good to me.  It needs to be refreshed to latest main branch, to pull in CI fixes.

Is this ready for merging?  It is still marked 'Draft'

Yes, sir


#169: Fixed checking first if there is a logged in user before using the Entry API to access the Culturefeed Entry.

fixed by #175


How to redirect a page using GH pages?

Hello,
How to redirect a category and page using GH pages?
If I just replace url in jekyll, then it changes, but there will be no redirect from the old page. Is it possible to redirect using  GH pages?
Examle: https://blogi-kriptovalyut.github.io/exchanges/okex/ -->> 301 redirect https://blogi-kriptovalyut.github.io//markets/okex/

Check out jekyll-redirect-from which is bundled with GH Pages.
Next time we'll appreciate it if you ask general Jekyll support questions on Jekyll Talk. Issues are for bug tracking only.

Check out jekyll-redirect-from which is bundled with GH Pages.
Next time we'll appreciate it if you ask general Jekyll support questions on Jekyll Talk. Issues are for bug tracking only.

Thanks

Thanks


search tweets and store the results

I'm using the code as follows for getting the tweets with the respective search strings but it's not working
- (IBAction)searchButton:(id)sender
{
    [self.view endEditing:YES];
    NSString *searchString = self.searchField.text;
    [_twitter getSearchTweetsWithQuery:searchString successBlock:^(NSDictionary *searchMetadata, NSArray *statuses)
     {
         NSLog(@"Tweets for search field are : %@", [searchMetadata description]);
         self.tweetDictionary = searchMetadata;
         self.tweets = statuses;
         NSLog(@"tweets status array : %@", statuses);
         [self.tableView reloadData];
     }
    errorBlock:^(NSError *error)
     {
         NSLog(@"Oops! : %@", error);
     }];
}

it's not producing any logs and also i have checked it with breakpoint's the function's are not called need help please.

http://www.ioscodes.com/store/p6/iOS_Twitter_Login_.html


Fix the release scripting for mac builds

This fixes the package build failures that were called out on the 2.14.4 release issue.
A few changes were needed to fix SSL failures during the mac build process.  The SSL failures were not present on the build host, but were coming up during the build process.  The project uses https://github.com/kontena/ruby-packer to build the ruby code into a redistributable package, but the inner ruby build thats included in the package doesn't include certs by default as mentioned in the README.
I've done a few things here to fix the situation

Updated the release workflow to use ruby/setup-ruby instead of actions/setup-ruby, to make sure that the most recent and up to date ruby code was being brought onto the build host
Included gem update --system and gem update bundler in the mac build script, again to make sure that the most recent versions of each are being used in order to update any certificates or expectations related to certificates
Set the --openssl-dir argument when running rubyc to the parent directory of ruby's openssl default certificate directory.

With all of the above, I was able to successfully build both the linux and mac distributable exes which I minimally tested to ensure they were working.  I've attached those built packages to the 2.14.4 release and released the built gem to rubygems

CI failures are known and unrelated to this change.  Going to move ahead with merging this to unblock future releases.


Exploring reimplementation using bqplot

Today I spend some time to explore the possibility of using bqplot for implementation, image from FITS has to be convert to other standard format such as PNG, JPEG for it to work, my question is there better to convert FITS to these format?
My current way of converting it is using pillow.

I think @mwcraig or @astrofrog would have to advise on the bqplot side. I think they use Qt mainly, but I am not sure. In the meantime, we ask for your patience.

The bqplot implementation would probably use https://github.com/glue-viz/ipyastroimage as the Mark on the bqplot plot. That can take a numpy array, so I think we can avoid the conversion to another image format. I haven't had a chance to experiment much with https://github.com/glue-viz/ipyastroimage
I have a rough draft implementation of a really basic image viewer based on bqplot somewhere, will try to dig it up tomorrow.

Hi, have you digged up the draft implementation? I would like to have a look! :)

Found it, will upload soon (it had zero comments and didn't actually quite work 😮)

Look in this, don't judge the code too harshly 😀
https://gist.github.com/mwcraig/303cfb6e3e970d137e91789245ff259d
You will need to change the image you are using, let me know if you need me to send an image.

@mwcraig , what is the overlap with https://github.com/glue-viz/glue-jupyter ? Maarten seems to be involved in that too.

@pllim -- not too much from what I understand.
Maarten developed ipyastroimage as a more astronomy-centric replacement for the Image mark in bqplot because they needed that for the jupyter version of glue. He released as a separate package so that we could use it also.
When I asked Tom R. about the api for astrowidgets his response was that glue didn't really have an API for images like that.
I don't think it makes much sense to do a ton of work on making new selection tools or trying to generate images with a bunch of linked plots, since glue is focusing on that.
I could see adding a glue-based image widget in the future and/or replacing whatever bqplot-based widget developed this summer with something from glue down the road. But my impression is that we would still need to do a fair bit of wrapping because the API here is so much different than what glue provides.

Looks like they renamed ipyastroimage to bqplot-image-gl. I kinda went a slightly different route at #121 mainly because I forgot about this issue.
@mwcraig , I see that in your gist above, you had to do some Event hacking to get the cursor info. Do you think that will work inside the ImageWidget class or it has to be done inside the notebook itself?

Looks like they renamed ipyastroimage to bqplot-image-gl. I kinda went a slightly different route at #121 mainly because I forgot about this issue.
@mwcraig , I see that in your gist above, you had to do some Event hacking to get the cursor info. Do you think that will work inside the ImageWidget class or it has to be done inside the notebook itself?


Update getting-started-guide.md

Removed reference to the Community Edition as EOL and updated the installation link

Status
Ready/In Progress/In Hold (Reason for hold)
Related Issues
fixes: link to the issue
Description
A few sentences describing the overall goals of the pull request's commits.
Screenshots
Paste here any images that will help the reviewer

Preview Site Available
Congratulations! The automatic build has completed successfully.
A preview site is available at: https://xsoar-pan-dev--pull-request-1666-8ydpho1b.web.app

Important: Make sure to inspect your changes at the preview site.
Detected modified urls:

Getting Started Guide


feat: Add map designer

Issue #2
TODO: we have to make the towers place on the grid on clicking on induvidual grid

Codecov Report

Merging #4 (4d3b82e) into main (2ec6dbb) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main        #4   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            2         2
=========================================
  Hits             2         2

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 2ec6dbb...4d3b82e. Read the comment docs.


Typo in docs for config option for Naming/BlockParameterName

Fix a typo in docs for config option name for Naming/BlockParameterName in the code example

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!


Classic-Load balancer is getting created

When we deploy the solution, only classic load balancer is getting created, is there any way we can update the chart to create NLB when deploying the solution.
Whether we need to deploy aws-load-balancer controller and modify values.yaml accordingly.

Hi Murali,
The AWS load balancer configuration, including if classic or NLB, is controlled by annotations on the load balancer service. This can already be customized using the route.loadBalancer.annotations property. You will see that we configure a separate annotation in the AWS sample here, and can use a similar mechanism to change to an NLB.
Closing as resolved - feel free to re-open as needed.


Incompatible with latest experimental

From Yuri Spitzyn on facebook:
Great work, Lucas, thanks for sharing this with the community! As a bit of a feedback: for what it's worth the functionality seems to be broken by the latest touchdesigner update (2020.46540). The Web Render TOP inside the "container_UG_V4" comp simply displays a blank white rectangle now. This was checked on Windows 10 with the "UberGui_V4_Release.toe" network from the original Uber GUI github package.

Ok so this bug seems to stem from a TD bug in the experimental branch with the webrender top.
You can read more about it At the TD Forums.
For now until that bug is fixed, the only work around is to use the latest recommended stable version (2020.28110).

In the latest stable 20201.10330 this has now been fixed on Derivative's side:
https://forum.derivative.ca/t/2020-46540-x64-win10-fixed-strange-but-reproducible-webrender-top-thread-crashes/170801


Allow ToResult to accept a value

Would it be suitable to add a ToResult<T>(T value)  option?  Currently, when using the ToResult it is just using the default value, or the converter is supplied.  For a scenario I have, I would like something other than the default.  e.g.,
public Result<bool> DoStuff()
{
    if (something)
    {
        Result fooResult = Foo();

        return fooResult.ToResult(true);
    }

    return Result.Ok(false);
}

I have currently worked round this by doing fooResult.ToResult<bool>().WithValue(true).  The implementation would be something like:
public Result<TNewValue> ToResult<TNewValue>(TNewValue newValue)
{
    return new Result<TNewValue>()
        .WithReasons(Reasons)
        .WithValue(newValue);
}

Hi.
If Foo() returns a failed result (=result with one or more errors) then ToResult(true) would copy the errors and would set the Value of Result to true. The caller of DoStuff() have in this scenario no chance to access the value because the Result object contains one or more errors. The reason behind that: If you try to access Value of a failed result you get an exception. You can only access Value without exception if you have a success result. This is by design to prevent hidden bugs, see Fail Fast principle.
I would try something like that. Please write me back if it was helpful for you or if this suggestion doesn't help you in your scenario.
public Result<bool> DoStuff()
{
    if (something)
    {
        Result fooResult = Foo();
        if (fooResult.IsFailed) return fooResult;

        return Result.Ok(true);
    }

    return Result.Ok(false);
}

Sorry, my explanation may have been a little unclear.
What you have explained is what I expect the functionality to be.  For the Result<T> type, you have a public Result<TNewValue> ToResult<TNewValue>(Func<TValue, TNewValue> valueConverter = null) method to convert the value if it is not failed.  My suggestion is to try to replicate the same functionality, but instead of taking a converter, take a raw value.
Based on the ToResult() that accepts the converter, my example implementation should have been.
public Result<TNewValue> ToResult<TNewValue>(TNewValue newValue)
{
    return new Result<TNewValue>()
        .WithValue(IsFailed ? default : newValue)
        .WithReasons(Reasons);
}

Ah I see your point. I will introduce this method in the next days.

I will sort it if you want and put a PR in if you want.

Thanks again for your pull request. The new version is building and its on the way. I think it will be on nuget in the next hour.


Fix: Don't delete raw files

@lcardno10 This PR creates a new function that can delete raw files only - it is needed for the deleted protected endpoint work that is coming.


do not put method (https/ssh) into URL for submodule

do not put method (https/ssh) into URL for submodule; use relative path instead which will choose the method used for the main repo

sounds reasonable... but can you tell me, how I see if this works....?

Well - for me it made the checkout work: since I did checkout the main repo using https and did not have ssh+github setup, the submodule failed (silently at that). With this change, it made it work. I guess you could, after removing (renaming) the 'bibliography' subdirectory re-initialize the submodules and see if this works for you too (using ssh).

OK, let's doe experiments and merge this and see if it works....

so, it works in github....

fabby, that was on my list of stuff to figure out.

I noticed that this change was later reverted by @jngrad in https://github.com/the-teachingRSE-project/competencies/commit/264b2821f2ad5c89df3d5a6a264ef5aa0f4459d8. Was that in context, or are we walking in circles?

@MakisH Sorry, I was unaware of this PR. We seem to be walking in circles.
CI cannot run on forks when using a relative path, if the bibliography repository wasn't forked (logfile<PHONE_NUMBER>5):
repository 'https://github.com/jngrad/bibliography.git/' not found
clone of 'https://github.com/jngrad/bibliography.git' into submodule path '/home/runner/work/RSE_competencies/RSE_competencies/bibliography' failed
The process '/usr/bin/git' failed with exit code 1

We cannot require contributors to fork the bibliography repository, because its name might clash with another bibliography repository they might already have.
Maybe we could make CI clone both repositories, as outlined in actions/checkout user guide, section Checkout multiple repos (side by side). Although, it's unclear if we have to then cd .. to go into the correct folder... I need to test this out.


[Doc] Update "Prerequisites" in brokerLoad

What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_admit_test


Added ability to change week day background color and row height

I just release a new version.
Check the example projet.


Removed values from env example

Scope of changes
Removed values from env example file.
Type of change

[ ] bug fix
[ ] new feature
[ ] documentation
[x] other (describe)

Acceptance criteria
Describe how reviewers can test this change to be sure that it works correctly. Add a checklist if possible
Author checklist

[ ] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[x]   Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.

Codecov Report
Base: 43.94% // Head: 43.92% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (22ba1d1) compared to base (1e00fa8).
 Consider uploading reports for the commit 5147444 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #850      +/-   ##
==========================================
- Coverage   43.94%   43.92%   -0.02%
==========================================
  Files         624      624
  Lines       23091    23093       +2
  Branches     1417     1416       -1
==========================================
- Hits        10147    10144       -3
- Misses      11613    11618       +5
  Partials     1331     1331

Flag
Coverage Δ

unittests
43.92% <ø> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

...b/gds-user-ui/src/components/BasicDetail/index.tsx
31.42% <0.00%> (-1.91%)
:arrow_down:

web/gds-admin-ui/src/utils/index.js
44.05% <0.00%> (-0.70%)
:arrow_down:

...com/trisacrypto/directory/pkg/gds/certman/certs.go
54.91% <0.00%> (-0.32%)
:arrow_down:

web/gds-user-ui/src/utils/theme.ts
100.00% <0.00%> (ø)

...r-ui/src/components/NetworkAnnouncements/index.tsx
0.00% <0.00%> (ø)

...nents/OrganizationProfile/OrganizationalDetail.tsx
0.00% <0.00%> (ø)

web/gds-user-ui/src/utils/utils.ts
39.02% <0.00%> (+0.31%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Grimoire manifest definition is object hell.

https://destiny.plumbing/en/raw/DestinyGrimoireDefinition.json
I dont think I want to work on this...

https://destiny.plumbing/en/raw/DestinyGrimoireCardDefinition.json
oh god...


Expose from client when it's not up-to-date with server

This could be due to temporary network/server issues or if the client is coming back from being offline and hasn't yet downloaded & applied all the updates.

[ ] isUpToDate should be set to false
[ ] expose variable with the number of ms since the last successful sync

Nice idea to expose a variable that is the time elapsed since last sync!
Regarding isUpToDate, i'm not sure about setting it to false.
Internally, it's used to determine whether to use live mode or not. If it turns out we're not up to date, we will get the part of the log we are missing and will then go back into live mode on the latest offset. So internally, i don't think there's a need to put isUpToDate to false.
However, we also expose isUpToDate externally, which may lead users to think they are up to date while it may be that they were up to date 1 hour ago and that we failed to sync since then. But it's hard to tell when isUpToDate should become false... because there's no way to know for sure other than contacting Electric.
The point i'm trying to make is that if we pessimistically put isUpToDate to false because the last N fetch requests failed, that doesn't necessarily mean we are out of date, we wil only know when we manage to reach Electric again. So maybe, we want an internal _isUpToDate that we keep as it is now, and have an external isUpToDate that is true if we managed to sync recently, e.g. if timeSinceLastSync < UP_TO_DATE_THRESHOLD. One could also argue that exposing the time elapsed since last sync is enough for this purpose and clients can decide for themselves whether they are up to date or not.

I like the idea of an UP_TO_DATE_THRESHOLD, perhaps a configurable one with a sensible default?

But it's hard to tell when isUpToDate should become false...

Wouldn't it just be anytime there's a fetch error? https://github.com/electric-sql/electric/blob/b322b957a1010c6450175001b5519b9567558fbc/packages/typescript-client/src/client.ts#L390-L412
But yeah, I see you're point. Maybe isUpToDate is the wrong variable name for what I'm thinking. Basically what I want is what we had in the old electric where you could see if you were connected to Electric or not. There's not an exact analogy to http long-polling as we don't have a persistent connection but isConnected could be true once we've successfully fetched data and get switched to false as soon as there's any fetch errors.

UP_TO_DATE_THRESHOLD

NO OPTIONS! 😛

@KyleAMathews i see!
So, let's keep isUpToDate but make it private, and introduce a public isConnected variable that is put to false on a fetch error.

Great!


New subcommand: sevctl-attest (with initial support for QEMU)

sevctl should enable users to perform attestation routed through hypervisors that support AMD SEV virtual machines.
Example usage, for inspiration:
$ sevctl attest qemu --socket=$QEMU_QMP_SOCKADDR --ovmf=/path/to/ovmf

sevctl will need to learn the QEMU machine protocol so that it can attest through QEMU's API.

On top of OVMF attestation, there is work under review in QEMU and OVMF to support measured (attested) SEV boot with -kernel / -initrd / -append. In order for that to be secure, QEMU injects the hashes of those three blobs into a special SEV hashes page (designated in OVMF) before launch, and encrypts it as part of the initial guest memory so it is part of the measured memory. OVMF then makes sure that the kernel/initrd/cmdline it reads from QEMU indeed match those hashes, refusing to boot otherwise.
To verify such measurements, the Guest Owner needs the OVMF file and the three blobs. In such case, the command-line may look like:
$ sevctl attest qemu --socket=$QEMU_QMP_SOCKADDR \
    --ovmf=/path/to/ovmf \
    --kernel=/path/to/vmlinuz \
    --initrd=/path/to/initrd.img \
    --cmdline="console=ttyS0 loglevel=6"

This qemu patch includes the SevHashTable which is the memory that is added to the measurement after OVMF. Special care should be taken with empty cmdline and empty initrd.
/cc @fitzthum @jimcadden @jejb

For upstream posterity:
As I mentioned over emails, I will sketch out what this looks like in terms of sevctl's implementation of this.


Hide status element from mobile screen readers

This change address an issue reported to affect mobile screen readers when swiping to traverse focusable elements. In this condition, the loading status element would be focused by the screen readers, causing the status to be announced a second time by the screen reader.
Fixes #445

Milestones! 🙌

I'm learning, slowly ;)

cc @svinkle

Please read https://github.com/GoogleWebComponents/model-viewer/issues/445#issuecomment-478709003 before merging.

Please do not merge this PR. I will be reviewing the methodology to see if there is a good alternative, and if not then we can close this off.