Support signal and delete

Signed-off-by: Enis Inan<EMAIL_ADDRESS>

Blocked on https://github.com/puppetlabs/wash/pull/574


#460: deploy hotfix PRs to UAT

Resolves #460

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6470

Relevant Lines:
6579

💛  - Coveralls


Repository Type return as empty object when used with type-graphql field resolver

@Service()
@Resolver(of => CampaignItem)
export class CampaignItemsResolver implements ResolverInterface<CampaignItem> {
  constructor(@injectRepository() private readonly campaignItemReposiory: CampaignItemRepository) {
  }
@FieldResolver({name: "publisherMetrics"})
  async publisherMetrics(@Root() data: CampaignItem):Promise<CampaignItemPublisherMetrics[]> {
    return this.itemMetricsRepository.getMetricsByItemPublisher(data.id)
  }
}

When used with ResolverInterface & FieldResolver, the respository passed will be a function that empty object

Closing as solved, if your issue persists, please open a new issue.

Closing as solved, if your issue persists, please open a new issue.


CI for Node

Add js

Contains the compiled JavaScript
Initializes with two players
Starts the game with a random player
Sets the current player to be whoever it is not


Allowing other default values for postgres ID fields

Currently the postgres dialect will create a SERIAL for ID fields, which behind the scenes results in a default value being the nextval from a sequence. I would like to set a default value to be a custom function I  create for generating ids, but still use the name ID and have gorm know it is the PK.
So, I'd like to have a model like:
type User struct {
    ID        uint64 `sql:"DEFAULT:user_gen_id()"`
...
}

This results in errors when trying to create the table:
pq: multiple default values specified for column "id" of table "users"

I believe it should be straight forward enough in the postgres dialect (maybe https://github.com/jinzhu/gorm/blob/master/postgres.go#L59) to detect the string default in the StructTag and instead of creating the PK as a SERIAL create it as the appropriately sized integer. I could put a pull request together if people think this could work.
This is also my first day looking at gorm (evaluating it and ran into this snag for my use case)  so if there is a workaround for accomplishing this I'm all ears. Thanks.

You should be able to do it like this:
type User struct {
    ID        uint64 `sql:"type:integer PRIMARY KEY;default:user_gen_id()"`
...
}

Where to define that user_gen_id function?
Right here in Go source code?
@jinzhu


Delete unnecessary spaces

When copying from the web page, there is an unnecessary space after the code.

@changeworld : Thanks for your contribution! The author, @xujing, has been notified to review your proposed change.

@changeworld : Thanks for your contribution! The author, @xujing, has been notified to review your proposed change.

Verified the updates; merging the PR.
@changeworld Thanks!


Why after modification count is less than webapp's database's count

Before modification count:
{2: 130, 4: 1361, 3: 734, 1: 13, -8: 169, -9: 176, -7: 53}
Total 2636

After modification count:
{2: 130, 4: 1361, 3: 734, 1: 13, -8: 169, -9: 25, -7: 53, 10: 70, 13: 25, 14: 53}
Total 2633

Webapp's entries are based on existence of sessions in the backend. Sessions are synced from ERIS to VM every six hours. So it may be the case that there are more sessions in webapp's database than autoQC algorithm's. Hence, algorithm could not change all -9s.

Wrong!! There was a mistake in our counting algorithm.

Fixed by https://github.com/AMP-SCZ/eeg-qc-dash/commit/c46a6ecb2f4c15982b2dce57c44a5b31ff14ce2a


Project still has console logs

Hi, the project still logs to the console, needs to be fixed before shipping to final release :)

fixed


back button is not working

"header": {
"menu": {
          "items": [{
"type": "vertical",
"components": [{
  "type": "button",
  "url": "https://.....",
  "action": {
  	"type": "$back"
   }
}]

}]
}
}
This code isn't working. I got this directly from the jasonett website, does anyone know how to fix this?

hello, this is missing the basic structure.
{
   "$jason": {

    }
}

please go to the docs in https://jasonelle.com/docs
for more info.


Migration force overwrite data option

Provide an option to overwrite data when running the migration task.
When running the migration task, and it finds there is data that will be lost, it should alert to the developer, and require forcing overwrite. Could use a flag like forceoverwrite=1

Migration task will need to be completed overhaul.


Update Project Settings

Migrate to Java21
change database to file

Change the way register admin. Instead of register admin provide the admin the log in details and let him to change a password latter

#36 Update all deprecated configs and methods

Implement profiles


Possible to run liana_tensor_c2c function with external_resource/orthologs?

Dear developers,
Thank you for this great package and for your extensive effort to properly document it!
I have a question regarding the liana_tensor_c2c function/workflow: Is there somehow an implemented way to combine this with the use of an external resource (gene orthologs), e.g. for using it on mouse data?
All the best,
Stephan

Hi Stephan,
For R, check out this tutorial on ortholog mapping. For python, liana has a human consensus 'consensus' and mouse consensus 'mouseconsensus' resource options you can specify in the resource_name argument. You can also map genes as shown here. This is all @dbdimitrov's work, so Daniel correct me if I'm wrong.
For use of custom resources see these sections of our tutorials in Python and R. Let me know if this answers your question.

Hi, yes you could. Both the R and Python liana packages support conversion /w external orthologues. For R it's the one that Hratch shared, for Python you could check this tutorial.


Fixed markdown links in README and README.dev

The testing section was referring to old link in README.md itself. I have changed it to Read the Docs link.
I have left the typo in README.dev.md which is fixed by #492. So I may have to rebase once that is merged.

Automated message from Dropbox CLA bot
@akarambir, it looks like you've already signed the Dropbox CLA. Thanks!

Great, thanks @akarambir, I merged this with some small tweaks.


Tune shared_buffers and effective_cache_size based on given memory

@praiskup @bparees initial database tuning PR.
We already chat about this with @praiskup and his concerns are that we should not be changing the defaults and let users to provide the 'correct' value.
My thoughts are that we as a platform can deterministically get the memory (based on the resource quota set for the Pod) so we can provide better "default" based on upstream formulas. If we will not do it, users will have to change them every time they  adjust them in OpenShift. The assumption here is that they will follow the upstream formulas and so we can make their life easier by doing that for them.
Note that this auto-tuning only take account when you run the container with --memory=512MB or so. By default we don't do anything. Also note that users can still override the values manually (the user  provided env vars take precedence)
@hhorak ptal as well, we might do something similar to mysql :-)
For the reference, there is Trello card for this: https://trello.com/c/cmY00byV/675-8-update-images-to-autoconfigure-based-on-available-memory-template-evg

This is not a bad idea, but it is hard to decide what should be a 'correct' value.
As discussed over irc, https://github.com/gregs1104/pgtune might be interesting
project to cooperate with.

@praiskup agree, using that script will make it better, but it has to support cgroups ;-) (it does not from the source code). I can add that as trello card/follow up.

Generally, I think we can do some reasonable auto-tuning, if it is properly documented, users understand what is going on and they have way to change it. I see this change is documented in README.md, but this file is not part of the resulting image.
Well, I don't say it must be implemented as part of this PR, but we should start using some standard way, once it is set up (https://github.com/projectatomic/container-best-practices/issues/38) and document things from README.md in the image as well.

update the 9.2 readme also?

@bparees updated.

@mfojtik I'm working for an update for the script for sti-base. It'll address issues raised by @bparees and @php-coder as well.

@mnagy cool, can you tag me in the PR?

fyi, the PR for updated script is here: https://github.com/openshift/sti-base/pull/90

[test]

[test]

@php-coder @mnagy any final thoughts? ;-)

LGTM.

Issue with the settings in common.sh:
https://github.com/openshift/postgresql/issues/109


Object requirements

[Enter feedback here]
This does not work at all. I suspect that 'Object' has requirements, such as implementing INotifyProperty; but none of the help sites mention this. I have a simple Object, basically a structure with 5 Properties, and the bound form components are not notified when the data changes and no BindingSource events fire.
Can you expand on what the Object is required to implement?

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

ID: e890ee3b-412f-e524-f93a-28e7dad1faa0
Version Independent ID: ee48495d-3a11-1de2-0ed1-71e959630adc
Content: Bind Control to a Type Using the Designer - Windows Forms .NET Framework
Content Source: dotnet-desktop-guide/framework/winforms/controls/how-to-bind-a-windows-forms-control-to-a-type-using-the-designer.md
Product: dotnet-framework
Technology: dotnet-winforms
GitHub Login: @adegeo
Microsoft Alias: adegeo

Hi @Rose-Kenner-Biamp I'm assuming you're using Windows Forms for .NET 6 or 7? This was written for .NET Framework, and the data binding docs haven't been written for .NET yet. AFAIK there aren't any non-working parts of databinding in .NET winforms. There is a blog post that discusses some of the new .NET 6 features, but .NET 7 goes even further if you enable previewfeatures, which adds WPF-style command binding and base classes too.
Personally, I've never been able to get databinding working well in Windows Forms. Perhaps @KlausLoeffelmann can help?

I can do it by hand.
Following any common help like https://learn.microsoft.com/en-us/dotnet/desktop/winforms/how-to-implement-the-inotifypropertychanged-interface?view=netframeworkdesktop-4.8
The object must

derive from INotifyPropertyChanged
add an event public event PropertyChangedEventHandler PropertyChanged;
implement NotifyPropertyChanged like this
private void NotifyPropertyChanged([CallerMemberName]  String propertyName = "")
{
if (PropertyChanged != null)
{
PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
}
}
add using System.ComponentModel (for [CallerMemberName] recognition)
each property must now add a private backing variable e.g. for public Details there must be a member field details.
each property {get;set;} must be converted to
public { get => details;
set { if (value != this.details)
{ this.details = value;
NotifyPropertyChanged();
}
}
the form must subscribe to BindingSource_ListChanged which must have this code
if (e.ListChangedType == ListChangedType.ItemChanged)
{
if (e.PropertyDescriptor.Name == "Details")
{
text1.Text = (myBindingSource.DataSource as myBindingType)?.Details;
}
}

Ok, now it is bound.
But this is a lot of boilerplate code to write by hand - I thought there was a more convenient way to do databinding from an object.

I'm going to close this because I don't think there is anything to do on the old doc

I suppose "Implement NotifyPropery" covers what must be done.
From: Andy (Steve) De George @.>
Sent: Thursday, June 15, 2023 1:13 PM
To: dotnet/docs-desktop @.>
Cc: Rose Kenner @.>; Mention @.>
Subject: Re: [dotnet/docs-desktop] Object requirements (Issue #1625)
I'm going to close this because I don't think there is anything to do on the old doc

Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/dotnet/docs-desktop/issues/1625*issuecomment-1593667938__;Iw!!HBnMciuwfVSXJQ!Qv-bK0Ng6X5sjDONrVPOsanzEddQ7M3sUZoph2Xc0hrYKEKvcEW707tinOUiEOp0iDBgOJHrfKjJyD7-xUDUwnZr$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AYNDUH6TSBGUVYCXUOHJWBDXLNUD5ANCNFSM6AAAAAAXF4JRHA__;!!HBnMciuwfVSXJQ!Qv-bK0Ng6X5sjDONrVPOsanzEddQ7M3sUZoph2Xc0hrYKEKvcEW707tinOUiEOp0iDBgOJHrfKjJyD7-xQ6UqR7d$.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Something like oidc-client-js's querySessionStatus

do you have something like oidc-client-js "querySessionStatus"  which returns the user's current signin status in IdP?
querySessionStatus [1.1.0]: Returns promise to query OP for user's current signin status. Returns object with session_state and subject identifier.
source

We provide hasAccessToken and hasIdentityToken (perhaps it's name is a bit different) for this.

hasAccessToken and hasIdentityToken checks in the storage only. querySessionStatus fires an actual http request. maybe this checkSession but it does not return anything

I'm not sure what queryeSessionStatus does in the other lib, but to give an overview:

as Manfred mentioned, hasAccessToken and similar methods will give local (client side) status
you can force a silent refresh to ask for server side status and check what comes out of that

Hoping that answers the question? Let us know if we should keep this issue open, or feel free to open a fresh one.


fix mkdir in package.json

Issue
The package does not build as expected using Ubuntu 22.04 - the command shx mkdir -p lib/{cjs,esm} in package.json creates one folder lib/{cjs,esm} rather than two folders as expected (see image). This issue is consistent across npm, pnpm, and yarn.

Fix
Extended the script to specify two unique paths.

Thanks! What about passing two args to mkdir, would that work?

Yep that works - just updated it

tysm!


build.sh should create wasm subdirectory

After a fresh git clone, following the steps in README.md (including my pull request for including installing wasm-bindgen-cli), the build.sh script still fails:
$ ./build.sh
+ cargo build --target wasm32-unknown-unknown
    Finished dev [unoptimized + debuginfo] target(s) in 14.10s
+ wasm-bindgen ./target/wasm32-unknown-unknown/debug/wasm_tetris.wasm --out-dir ./wasm
error: failed to write `./wasm/wasm_tetris.js`
	caused by: No such file or directory (os error 2)

mkdir wasm

fixes this. Should probably be part of build.sh?

fixed, thank you.


[Bug] git config not detected when it contains includeif

What is the problem?
After selecting the router and database I see an error:
user.email is not set in git config.
Please set up git config before trying again.
The program encountered an unexpected issue and had to exit. The error was:
GIT CONFIG ISSUE: user.email is not set in git config.

If you continue to experience this issue, please post a message on our GitHub page or join our Discord server for support.

I'm not using global git config to avoid mixing 'private' commits with 'enterprise' ones.
My ~/.gitconfig is a  bunch of includeIf sections which choose correct .gitconfig based on a current working directory.
$ cat ~/.gitconfig
[includeIf "gitdir:~/github/"]
  path = .gitconfig_github
[includeIf "gitdir:~/personal/"]
  path = .gitconfig_gitlab
[includeIf "gitdir:~/proj1/"]
  path = .gitconfig_customer1
[includeIf "gitdir:~/proj2/"]
  path = .gitconfig_customer1
[includeIf "gitdir:~/proj3/"]
  path = .gitconfig_customer2
[core]
	autocrlf = input

Then my 'user' section is filled per-customer (different e-mail addresses or casing for the name).
I'm not familiar with the project, just saw the video: https://www.youtube.com/watch?v=1ZbQS6pOlSQ and though you might want to consider more complex git configs.
Actually I do not understand why on earth this tool tries to commit under my name?
 It also gives the option to integrate with one of the more popular Go frameworks (and the list is growing with new features)!

Not a single word about git or creating commits :no_mouth:
Operating System
Ubuntu 20.04
Architecture Version (x86, x64, arm, etc)
x86
Steps to reproduce

Start with your .gitconfig working as expected.
Rename it to .includeme
Create empty .gitconfig, which includes your original config

[includeif "gitdir:~/tmp/"]
    path = .includeme

cd /tmp
go run github.com/melkeydev/go-blueprint@latest create
choose whatever
it will fail

Relevant log output
user.email is not set in git config.
Please set up git config before trying again.
The program encountered an unexpected issue and had to exit. The error was:
GIT CONFIG ISSUE: user.email is not set in git config.

If you continue to experience this issue, please post a message on our GitHub page or join our Discord server for support.

Early on in the project, a decision was made that the initial commit would be executed upon project creation. The project is still in its early stages, and we didn't take 'advanced' git configuration into consideration. We can open a ticket to address these posible changes if someone wants to implement it.
For now it needs global user and email in .gitconfig
[user]
email =<EMAIL_ADDRESS>name = Global Name
git config --global user.name "Your Name"
git config --global user.email<EMAIL_ADDRESS>

Thank you for the explanations.
Feel free to close this issue.


Trigger CSS animation when slide open

How could I make an animation to trigger when the slide get focus? This should be fun, even more impressive using CSS libraries like Animate.css (https://github.com/daneden/animate.css) and Magic (https://github.com/miniMAC/magic).
Any ideas? :)

hi
you can easly solve this problem in fact @bartaz snippet is ok but should know that you most add animated class plus your desire animation class! with this trick u can use all animate.css stunning animations in impress.js ;)
example for shake animation:
$('#mystep').on('impress:stepenter', function(){
var step = $(this);
step.find('.button').addClass('animate shake');
});
//this remove section is for when you want to repeat animation each time user visits that step
$('#mystep').on('impress:stepleave', function(){
var step = $(this);
step.find('.button').removeClass('animate shake');
});
BTW: i write some snnipet that may help you even better ;)
$( document ).ready(function() {
function animation(stepId,animateClass){
$('#'+stepId+'.animation').on('impress:stepenter', function(){
var step = $(this);
step.find('i.fa').addClass('animated '+animateClass);
});
            $('#'+stepId+'.animation').on('impress:stepleave', function(){
                 var step = $(this);
                 step.find('i.fa').removeClass('animated '+animateClass);
            });
        };
            animation('contact','shake');

    });

have Fun

I guess the problem was fixed given the comments above. If not please let us know so that we can continue the discussion.


Notify the user when they are editing a shared question

Goal
Better surface to the user what the effects of editing a question / publishing all drafts will be
Improvement
Notify user when editing a shared question
Mock is here
Eng summary
As part of this PR, we now have the ability to easily tell which programs reference a given question. On the question edit screen, we can modify the Update buttons (depending on the result of the above open question) to pop the above modal (see mock). The modal data will be populated from the list of programs referencing the question.
Trigger times: The dialog appears as a confirmation when a user hits “update” on a new version of the question
Nice to have: Include a “don’t show me this message again” checkbox option in the dialog
More context
This improvement is part of Phase 1 of the Publishing Programs doc here.

Copied the screenshot in for posterity:

I believe @syntapy is working on this, so assigning. @syntapy, if that's not the case, feel free to unassign.

Am a see if I can make progress on this today

Assigning to bion@ since GitHub won't allow me to assign to @syntapy.

Compared to other modals we do this one  might be tricky as we don't want to display stale data.
In other places we statically create the modals on page load, then show them upon click.
Here if you have the entire question list loaded they could get out of date if there's edits in other windows.  It'd be nice to load the relevant data.
Note that this page has other staleness issues that we address when a mutate operation happens, such as if you are viewing a Question with Active state (no Draft), then a Draft is made in another window.  IN that case we'll load the draft rather than try to make a new draft as would otherwise happen

Would using e.g. a JS function to continuously ping for the number of shared questions be a good approach to keeping the data up to date?

Also in the case of the draft question, would it be appropriate for them to end up editing that draft, so long as they were informed? And maybe also add some indicator that there was an update in the state since they clicked the edit button? E.g. maybe like a red "updated at XX:YY:ZZ: question shared accross (1) program now has one draft created by another user. Continuing would edit that draft"
But yeah, this is a really good point you brought up

@clouser2 likely has the best guidance here, but we don't have good AJAX patterns to follow yet.

@clouser2 likely has the best guidance here, but we don't have good AJAX patterns to follow yet.

This is a consistent problem throughout the UI. We have similar challenges for the "programs referencing a question" popup as well as the "publish all drafts" popup. In each of these cases, stale data could be shown to the user. So far, we've deemed that stale data is acceptable in these places and we'd like to someday have a more holistic solution.
Given that, I think having a live view of the data for this specific use case would be inconsistent with the approach we've taken elsewhere. If possible, I'd consider allowing stale data for the purposes of this issue and then filing a separate issue to determine how we want to holistically tackle the problem across the entire app.

@nb1701 Ask Sijia about if we want to do this


Is-net-clock_TJB

Improved screen wake-up responsiveness.

Corrected second-skip bug?

Resolved conflicts, awaiting your approval @doubledave

I see wakeup is really responsive now.


Document JS dynamic creatives

Document new option to set up creatives with Prebid.js - https://github.com/prebid/Prebid.js/pull/10819
Note
There are many references to PUC in native documentation that shoud, but did not, get updated:
https://docs.prebid.org/prebid/native-implementation.html
I am not sure on how to approach it because:

the doc seem to assume that the only way to do native is to use Prebid.js. If that's true (I don't think it's supposed to be), the new approach is just better than PUC. I can't find an example on how to set up native creatives when using AMP/mobile SDKs.
there's a lot of emphasis put on the sendTargetingKeys and requestAllAssets options. Those are irrelevant for the new approach. The first may be relevant for AMP/mobile - again, I don't know if those work at all. The second is, to the best of my knowledge, just a relic and there's never a reason not set it to true.

To me it looks like native docs need an overhaul which is a bit more than I am prepared to chew for this feature.
🏷 Type of documentation

[x] new feature

📋 Checklist

[x] Related pull requests in prebid.js or server are linked -> Paste link in this list or reference it on the PR itself

Hi
Thanks for this initial docs. We are using prebid native through GAM native designs.

the doc seems to assume that the only way to do native is to use Prebid.js. If that's true (I don't think it's supposed to be), the new approach is just better than PUC.

Do you mean that there's a need for prebidjs in order to support native? This is IMHO not true, as native can be requested via prebid server as well, and thus rendered too.

I can't find an example on how to set up native creatives when using AMP/mobile SDKs.

We have never get passed web, as native demand was basically none existent for us . However the initial approach, send all assets via key values, should support AMP. If all assets are sent as key values, they can be accessed through macros and used in native design templates

there's a lot of emphasis put on the sendTargetingKeys and requestAllAssets options. Those are irrelevant for the new approach. The first may be relevant for AMP/mobile - again, I don't know if those work at all.

IIRC requestAllAssets only works with prebidjs, as this is basically a post message call into the ad iframe.
sendTargetingKeys is the initial way to send all assets as key values. Which IMHO doesn't work well, if the assets are too large. GAM sends GET requests and truncates key values if it gets too long.

The second is, to the best of my knowledge, just a relic and there's never a reason not set it to true.

That is true. I guess 🤔

Do you mean that there's a need for prebidjs in order to support native? This is IMHO not true, as native can be requested via prebid server as well, and thus rendered too.

That sounds right, but otoh, how? I was stuck in trying to update that page, since I've never seen it done and it only talks about JS.

Ok, worked with our PostIndustria contact and found that there are indeed issues with the docs. It's on my list, but may take a few weeks to resolve.


Refactor codebase to use the useDisclosure hook

Fixes #498
Thanks in advance :D

Changed some stories where required to use explicit open and close instead of toggle (Sidebar, Modal)

GJ


PipelineRun 's PodTemplate support affinity

Feature request
currently, PipelineRun support PodTemplate only three feature: tolerations, nodeSelector, and imagePullSecrets. and I would like to see affinity be supported.
Use case
In my case, I prefer all the task running on the high performance node which I will label it. but PipelineRun scheduler doesn't support affinity.nodeAffinity.
and I cannot guarantee the high performance node will avalible all the time, so nodeSelector makes assumption too strong.

/assign

any update? @yuzp1996

any update? @yuzp1996

Hi! I will start working on this issue today.

Hi @lookis,  Podtemplate supports affinity now. This document podtemplates could help you.
But if your PipelineRun contains PVC then affinity will be overwritten by affinity-assistant.
I tried and find two solutions to avoid being overwritten by affinity-assistant.

delete PVC in your pipelinerun
disable affinity-assistant by setting feature-flag disable-affinity-assistant to be true

@lookis  For more information about Affinity Assistant, you can refer to this document https://tekton.dev/vault/pipelines-v0.16.3/workspaces/#specifying-workspace-order-in-a-pipeline-and-affinity-assistants.

@yuzp1996 I think it would be better to merge two configurations, that's what nodeAffinity.weight for.
It's no reason for pvc case not to prefer a better node, thought it may failed, that's why it's nodeAffinity rather than nodeSelector

@lookis  Yes, I agree with you that merging the two configurations is the better approach. I made changes in my dev environment and tested some simple cases and everything seems to be working fine. Maybe tomorrow I will create the PullRequest because I need to add some unit tests and test some more complex cases.

PR https://github.com/tektoncd/pipeline/pull/5306 has merged
/close

Thx for the merge.


#88 Number contributor section steps - updated numbers in contribute …

Number contributor section steps #88
Replaced section numbers in the followings:
A working program

(1) The 'Hello World' program
(2) Containers
(3) Writing documentation
(4) Integrating a DevOps pipeline
(5) Finalize PR

Cloud deployments

(1) Pick just one cloud
(2) Finalize PR
(3) Enhance, enhance, enhance

@ktSuW I've updated #88 with comments. I made an incorrect assumption about how MkDocs works. Let's start a new PR with those changes. Sound good?

Thanks @ericis. It is good.  will work the updated task. I will ask you question if I have any when I start the task. Thanks!


Bug with query fragment

with the following query:
query roomQuery($roomId: String!) {
  room(id: $roomId) {
    account {
      name
      testAccount
      id
    }
    ...RoomJoinAttendeeCheckin_room
    id
  }
}
fragment RoomJoinAttendeeCheckin_room on Room {
  account {
    id
    timezone
    name
  }
}

I am expected to get this back
{
  "data": {
    "room": {
      "account": {
        "id": "QWNjb3VudDo1NzE2ZmIzNGQ1MjViM2VhNGY0MjMwZjQ=",
        "timezone": "Australia/Melbourne",
        "name": "Blu Property Malverned",
        "testAccount": true
      },
      "id": "Um9vbTozMg=="
    }
  }
}

But instead i get this back
{
  "data": {
    "room": {
      "account": {
        "id": "QWNjb3VudDo1NzE2ZmIzNGQ1MjViM2VhNGY0MjMwZjQ=",
        "timezone": "Australia/Melbourne",
        "name": "Blu Property Malverned"
      },
      "id": "Um9vbTozMg=="
    }
  }
}

testAccount is missing

same issue with:
query roomQuery($roomId: String!) {
  room(id: $roomId) {
    ...RoomJoinAttendeeCheckinB_room
    ...RoomJoinAttendeeCheckin_room
    id
  }
}

fragment RoomJoinAttendeeCheckinB_room on Room {
  account {
    id
    timezone
    name
    testAccount
  }
}

fragment RoomJoinAttendeeCheckin_room on Room {
  account {
    id
    timezone
    name
  }
}

Basically last fragment wins, for example this works:
query roomQuery($roomId: String!) {
  room(id: $roomId) {
    ...RoomJoinAttendeeCheckinB_room
    ...RoomJoinAttendeeCheckin_room
    id
  }
}

fragment RoomJoinAttendeeCheckinB_room on Room {
  account {
    id
    timezone
    name
  }
}

fragment RoomJoinAttendeeCheckin_room on Room {
  account {
    id
    timezone
    name
    testAccount
  }
}

I am solving this bug. 😁

amazing 💯

I fixed this bug, please upgrade to 1.13.4.

can confirm it's working thank you

The bug seems to be still there.
It does not appear for simple cases anymore, but it still appears for highly nested types.
Here is an example:
query {
  account(id: "QWNjb3VudDsvYWNjb3VudHMvMTVmZjlkZDYtMzJhNi00MTdiLWIwMDQtNmY2OWQ3Y2M0Yzhi") {
    ...FragmentA
  }
}

fragment FragmentA on Account {
    databases(last: 1) {
      edges {
        node {
          ...FragmentB
          ...FragmentC
          bindings {
            edges {
              node {
                engine {
                  id
                }
              }
            }
          }
        }
      }
    }
}

fragment FragmentB on Database {
  bindings {
    edges {
      node {
        engine {
          currentStatus
        }
      }
    }
  }
}

fragment FragmentC on Database {
  bindings {
    edges {
      node {
        engine {
          desiredStatus
        }
      }
    }
  }
}

Returns:
{
  "data": {
    "account": {
      "databases": {
        "edges": [
          {
            "node": {
              "bindings": {
                "edges": [
                  {
                    "node": {
                      "engine": {
                        "id": "RW5naW5lOy9hY2NvdW50cy8xNWZmOWRkNi0zMmE2LTQxN2ItYjAwNC02ZjY5ZDdjYzRjOGIvZW5naW5lcy8wMzdiZGI4OC00MTAzLTRkZDAtYWRkNi00NjliYzg2MzU3YjA="
                      }
                    }
                  }
                ]
              }
            }
          }
        ]
      }
    }
  }
}

I expected id, currentStatus and desiredStatus to be returned, but only id was returned.

Can you provide a test case that can reproduce the problem? This way I can solve it faster. 🙂

@sunli829 After more research, I think my bug is not the same as the one from this issue, so I created a new issue: https://github.com/async-graphql/async-graphql/issues/345


Add options "add this to method access" and "Class qualifier for stat…

…ic access"

LGTM
I'll try to merge and release soon


bug: initialized with all options selected, but "check all" checkbox not checked

to reproduce

var el = document.getElementById('sample-select');
var opts = [
        {label: 'Options 1', value: '1'},
        {label: 'Options 2', value: '2'},
        {label: 'Options 3', value: '3'},
    ];
var initialValue = opts.map(opt=>opt.value);

VirtualSelect.init({
    ele: el,
    multiple: true,
    enableSecureText: true,
    options: opts,
    selectedValue: initialValue,
});

and click the select component, after the dropbox expanded, you will see an unchecked checkbox aside the Search text.
with the options checkbox all checked.

@jonirrings thanks for letting me know it. I will fix it in the future release.

@jonirrings this issue has been fixed and available in the release v1.0.24

@sa-si-dev nice and thanks for your awesome work.


Getting prompt to "Please fill out all the fields" even when all fields are filled

getting stuck at the screen attached

@abhitheawesomecoder Thanks for the bug report! Our authentication provider that we use for developer accounts is actually experiencing a critical issue right now so we've disabled login until the issue is fixed. I'm really sorry about that.
@jimmylee Assuming this is related to the Auth0 outage, we probably should show a more accurate error message here.

We keep issues related to Expo on the repo for "expo" so I've recreated your report over here: https://github.com/expo/expo/issues/857 and am going to close this one.


Description of available images (aka ChangeLog)

Describe the solution you'd like:
[A clear and concise description of what you want to happen.]
The UI presents a selection of images (name + version) to choose from when building ISOs or triggering upgrades.
However, there's no information about the differences between images and how to choose one over the other.
Users can only choose the highest version and keep fingers crossed.
The UI should be enriched with 'ChangeLog' data, informing users about the main changes between
image versions and thus helping them to choose the right one (or to not trigger and upgrade at all).
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Traditional SUSE Linux Enterprise captures this information in 'patch' descriptions.

We need to include additional information in ManagedOSVersion resources:

Package list
Patches information

We need a process to collect the patches included in each build (I'd suggest asking maintenance if they know a process)

Would be interesting to embed this information into image itself, so we can easily extract it at the time of channel creation.

This issue needs to be refined further.
As for now we came up with some conclusions about packages information:

Adding the packages list into the ManagedOSVersion is going to add too much data to the k8s resource and degrade performance
A package list could be shipped using a "side" info image, but this is making the OS release process more complex. It would be best that any information we want to display can be extracted/produced from the OS image directly.
Providing a packages list is not really the goal of this issue (which is instead to provide a human readable summary of changes)
Providing a list of packages can be done running rpm -qa from any OS container images, this can be documented
We can provide this as a image feature by using ENTRYPOINT:

docker run --entrypoint "rpm" registry.suse.com/suse/sl-micro/6.0/baremetal-os-container:2.1.1-3.29 -qa

However this won't work for ISO images, so either we include this info into the ISO image itself, or maybe we could add information on the base OS image used for the ISO (better).

Regarding patches:

Need to find out whether adding RUN zypper --installroot /osimage patch -y is a required best practice or not. This seems to be the case for the registry.opensuse.org/opensuse/leap-micro/6.0/toolbox:latest](http://registry.opensuse.org/opensuse/leap-micro/6.0/toolbox:latest) image, as I found out latest still had some patches that could be applied.
The output of zypper patches could be used as a human readable changelog maybe:

Refreshing service 'container-suseconnect-zypp'.
Refreshing service 'openSUSE'.
Loading repository data...
Reading installed packages...
Repository      | Name | Category    | Severity  | Interactive | Status     | Since      | Summary
----------------+------+-------------+-----------+-------------+------------+------------+---------------------------------------------------------------------------------------------
repo-main (6.0) | 9    | recommended | low       | ---         | applied    | 2024-09-03 | Recommended update for bash, libcap-ng, libselinux, libselinux-bindings, libsemanage, zypper
repo-main (6.0) | 10   | security    | critical  | ---         | not needed | -          | Security update for qemu
repo-main (6.0) | 11   | security    | moderate  | ---         | not needed | -          | Security update for wget
repo-main (6.0) | 21   | security    | important | ---         | not needed | -          | Security update for skopeo
repo-main (6.0) | 22   | security    | moderate  | reboot      | not needed | -          | Security update for ucode-intel
repo-main (6.0) | 23   | security    | important | ---         | applied    | 2024-09-03 | Security update for python311, python-rpm-macros


[CI] Use micromamba instead of miniforge3

Describe the enhancement requested
There is a standalone no-dependencies binary installation of mamba called micromamba:
https://mamba.readthedocs.io/en/latest/installation/micromamba-installation.html
We could probably use it instead of our current miniforge install.
Of course, this is a minor improvement. It might make the setup step slightly faster, and would probably make the CI code slightly simpler.
Component(s)
Continuous Integration

We recommend using micromamba's dedicated GitHub Action, mamba-org/setup-micromamba.

Ah, but we're installing it in a Docker image.

In this case, mamba-org/micromamba-docker might be of an inspiration.

Apparently there are docs for it: https://micromamba-docker.readthedocs.io/en/latest/


Second try at fixing issue #9 after rebasing.

Concat also the failed ip-api.com responses so that missing columns will have null values. This way we'll keep the ip information at least for further stats if wanted. Will probably break some plotting functions. They will need checks for not operating on null values where applicable.

Passed the main tests, it's just failiing on the plots which I plan to factor out anyway. Looks good! I'll try to merge in a fix on the plots to green the tests, otherwise I'll just merge anyway


DRY up net.go

In response to #214. Abstacted out process of sending message to peer given by peer ID. Total of -113 lines of code, which is good, but I'm not sure it is really useful, as it makes code in those functions non-symmetrical and, as a result, harder to mentally parse as a whole (similar to @cpacia's note at #217). On the other hand, less code == less space to make something wrong, so either close or merge, depending on what is more suitable for the goals of the project.

Coverage remained the same at 60.427% when pulling de10f6e1f147a9a4c21079f6a9b40d60bf5b5917 on yurizhykin:net-dryup into d5293b70092d1d87ad1346f17b2d8f01675debbb on OpenBazaar:master.

What do you mean by "it makes code in those functions non-symmetrical"? Are you referring to SendOrder being exceptional?

@duomoney, yes, exactly that

Coverage remained the same at 60.427% when pulling 11d55c83be288284d07468c5e6ea05c991c823a4 on yurizhykin:net-dryup into d5293b70092d1d87ad1346f17b2d8f01675debbb on OpenBazaar:master.

@yurizhykin I find it strange that SendOrder uses SendRequest but the other functions use SendMessage or SendOfflineMessage. This is for two reasons:

It seems to suggest that SendOrder doe not work in the offline case. I though offline orders was part of OB 2.0.
It seems to suggest that the response for the actions other than SendOrder are not worth looking at (e.g. for errors). Error checking here seems to be missing.

Maybe @cpacia can clarify.

@JustinDrake In the offline case, SendOrder returns an error, at which point Purchase() resorts to using SendOfflineMessage. Why it is done this way (outside of net.go) unlike all other messaging I am not sure.

If the vendor is offline it changes the data we need to send so it doesn't work to just send the message as-is as an offline message. We need to modify it before sending the offline message.

I fixed the merge conflicts and and bugs created by other changes to net and merge.


Support determine file type and schema through catalog service

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6553/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6554/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6555/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6556/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6557/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6560/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6566/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6567/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6569/
Test FAILed.

checkstyle is failing

@yuzhu @gpang  Tested on my local machine, csv files can be converted to parquet files, but still need to update the serde and input format classes in layouts.
|                   createtab_stmt                   |
+----------------------------------------------------+
| CREATE EXTERNAL TABLE `csv`(                       |
|   `a` int,                                         |
|   `b` string,                                      |
|   `c` double)                                      |
| ROW FORMAT SERDE                                   |
|   'org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe'  |
| WITH SERDEPROPERTIES (                             |
|   'field.delim'=',',                               |
|   'serialization.format'=',')                      |
| STORED AS INPUTFORMAT                              |
|   'org.apache.hadoop.mapred.TextInputFormat'       |
| OUTPUTFORMAT                                       |
|   'org.apache.hadoop.hive.ql.io.HiveIgnoreKeyTextOutputFormat' |
| LOCATION                                           |
|   'file:/Users/alluxio/p/transform-csv'            |
| TBLPROPERTIES (                                    |
|   'transient_lastDdlTime'='1572352942')            |
+----------------------------------------------------+```

Merged build finished. Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6571/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6572/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6573/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6574/
Test FAILed.

Failed tests:
alluxio.job.transform.format.ReadWriteTest.readWrite(alluxio.job.transform.format.ReadWriteTest)
  Run 1: ReadWriteTest.readWrite:87 expected:<alluxio.job.transform.format.parquet.ParquetSchema@55006bff> but was:<alluxio.job.transform.format.csv.CsvSchema@3ced125b>
  Run 2: ReadWriteTest.readWrite:87 expected:<alluxio.job.transform.format.parquet.ParquetSchema@55006bff> but was:<alluxio.job.transform.format.csv.CsvSchema@3ced125b>

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6575/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6576/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6579/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6585/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6589/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6591/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6612/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6616/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6615/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6621/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6622/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6625/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6630/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6632/
Test PASSed.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.1 please

alluxio-bot, cherry-pick this to branch-2.1 please


Adjust the inner padding of the calendar

Thanks for this awesome calendar component.
I've been trying to reduce the padding between the calendar edge and the date table, but the values seem to be set dynamically during width and height calculation.
I don't want to hard code the height and width values of the calendar, because that makes things look weird when the number of days in the month change. I also thought of setting a max-height, but it makes the calendar padding inconsistent between months.
Here's a screenshot of what I'm trying to do. The padding I'm trying to reduce is shown by the red arrows.

Is there a way for me to reduce those values?
Thanks!

How can i increase the height and width of the calendar ?

Awesome. I'll make a pull request soon. Thanks!

Any update on this?

Since I couldn't add padding, I adjusted the size of the days using daySize


Update IPublishedContentExtensions.cs

Fix for NullReferenceException when calling GetVortoValue : allow return of defaultValue or if the recursive parameter set true traversal up the tree.

Thanks @drangus! 🎉  I've updated the PR with whitespace formatting - to match the current code style.
Once @mattbrailsford is happy, we can merge it in 👍

Was just about to submit a PR for this as well. This change should also be added to DoInnerHasVortoValue.

Was just going to raise an issue/PR for this!  Might it be approved/merged/released soon?  Anything I can do to help?

This has now been merged in (and tweaked as per @sniffdk's suggestions) and is available via the build server https://ci.appveyor.com/project/UMCO/umbraco-vorto/build/<IP_ADDRESS>/artifacts or via MyGet https://www.myget.org/gallery/umbraco-packages.
I'll have a chat with @leekelleher to see if we push a release out now, or try and address any of the open issues first.

The applied patch doesn't allow recursion!
Change to:
if (prop == null)
{
    // PR #100 - Prevent generation of NullReferenceException: allow return of defaultValue or traversal up the tree if prop is null
    //return defaultValue;
    return recursive && content.Parent != null? content.Parent.DoGetVortoValue<T>(propertyAlias, cultureName, recursive, defaultValue) : defaultValue;
}


[custom channels]: add decimal display to channel funding blob

Fixes https://github.com/lightninglabs/taproot-assets/issues/1067.
Adds the decimal display value to the channel funding blob, so it can be rendered for lncli listchannels and lncli pendingchannels.
To make the code in the funding manager as nice as possible, we first do some refactor.

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

57 of 242 (23.55%)  changed or added relevant lines in 12 files are covered.
96 unchanged lines in 9 files lost coverage.
Overall coverage decreased (-0.06%) to 40.71%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tapchannelmsg/custom_channel_data.go
0
1
0.0%

tapfreighter/parcel.go
0
4
0.0%

tapsend/send.go
13
18
72.22%

rpcserver.go
0
7
0.0%

proof/meta.go
0
16
0.0%

address/address.go
0
24
0.0%

address/book.go
2
34
5.88%

tapdb/addrs.go
0
44
0.0%

tapchannel/aux_funding_controller.go
0
52
0.0%

Files with Coverage Reduction
New Missed Lines
%

tapdb/assets_store.go
1
65.12%

rpcserver.go
1
0.0%

tappsbt/create.go
2
53.22%

asset/asset.go
2
81.43%

tapchannel/aux_leaf_signer.go
3
43.43%

tapdb/universe.go
4
80.91%

tapgarden/caretaker.go
4
68.87%

commitment/tap.go
4
83.64%

tapchannel/aux_funding_controller.go
75
0.0%

Totals

Change from base Build<PHONE_NUMBER>9:
-0.06%

Covered Lines:
25821

Relevant Lines:
63427

💛  - Coveralls


Not receiving data

Hi,
so i installed hass.io on a pi3 and added mitemp_bt to my sensors in yaml. Unfortunatelly I only get  an "Updating mitemp_bt sensor took longer than the scheduled" for both of my xiaomi sensor.
This is my config:
`sensor:

platform: yr

platform: mitemp_bt
mac: '4c-65-a8-d4-c5-xx'
name: Living Room Temp
force_update: false
median: 3
monitored_conditions:

temperature
humidity

platform: mitemp_bt
mac: '4c-65-a8-d5-41-xx'
name: Hall Temp
force_update: true
median: 1
monitored_conditions:

temperature
humidity`

As you can see, neither false nor true force_update is working. What am I doing wrong?

Try to use the official one, instead of my sensor.
https://www.home-assistant.io/components/sensor.mitemp_bt/
If also the official one doesn't work, then you should troubleshoot your bluetooth connection. Maybe you have a bluetooth device tracker that could interfere, or maybe the sensors are too far away?

Indeed I have file system access, but hass.io has no intern search, so i have to find the py-file manually. Do you have a hint where it might be located?

In a standard installation, the file is located in /srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/sensor but I don't know how Hass.io structure files...I'm sorry

I finaly got the solution, after a sleep and 2 coffees. Maybe I can help someone else with this looking for a solution opon the internet.

I made a big mistake. I copied blind the mac from my BLE sniffer (xx-xx-xx). After reading the mitemp_bt.py I saw it must be xx:xx:xx, so I changed that. -Restart- Still didnt work out.

After some google "research" I found out one should delete the device from within the xiaomi mi app. I restarted hass.io. From now on, hass.io found the devices, but still didnt show any data.

I renamed the names from "Livingroom Temp" to "LivingroomTemp" (no more space left). And after a reboot it worked like a charme. All data is now shown on my dashboard.


Use shorter, more generic post attribution

For comment and thread posts. Always use {time} by {author} {authorLabel}
https://openedx.atlassian.net/browse/MA-1223

:+1:


Add an option to color the starting node differently

It would be helpful in SFDP layout mode, since the starting node might get moved around...

Looks like included by #309? I'm not sure if we are referring to the same sfdp

Relates to #404


epoll event adding: no need to join, there's no old clock here

@bors r+

:pushpin: Commit 5177354b9bcb35fb2b1f0391c7b642677b9f4132 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 5177354b9bcb35fb2b1f0391c7b642677b9f4132 with merge 29f68ed5fd802d24b544ff9b1b18712cafb8ab26...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 29f68ed5fd802d24b544ff9b1b18712cafb8ab26 to master...


Feedback to the user when a background task fails

Problem
Sometimes, background tasks fail.
Like in the real world, uploads and downloads sometimes fail.
The problem is that we don't warn the user. And we just retry the task, again and again.
In some cases, retrying is the correct approach: if the connection was down, if the server was down.
But there are cases where the server does not accept our data, and never will. For instance, if the image is too small (cf. #4488).
And in that particular case, we just get a confuse status ('status not ok') and a localized message (e.g. 'Das Foto ist zu klein. Bitte beachten Sie, dass Sie kein Foto aus dem Internet, sondern nur Ihre eigenen Fotos hochladen dürfen.').
Proposed solution
Let's say that there are 3 possible cases:

the background task works OK - no problem
the background task throws an exception - typically connection / server down - we'll retry
the server returns status not ok - this is problematic

In case (3), we have good reasons to say that the background task will always fail, and it's no use running it again.
But how do we warn the user that the background task is in error and the local data is to be discarded?
A system notification? A dialog that gets opened from most pages if something like that happens?

Duplicate with #4638.


.Net 6 "Object reference not set to an instance of an object" on import statement

I am trying to make some extensions and testing
When i try evaluate this code
import clr from "clr";

var Random = clr.getClass("System.Random");
var r = new Random(); // this will create instance of `System.Random`.
var n = r.next();

console.log(n)

i get exception
Object reference not set to an instance of an object

  at YantraJS.Core.FastParser.Compiler.FastCompiler.VisitImportStatement(AstImportStatement importStatement) in D:\projects\yantra\YantraJS.Core\FastParser\Compiler\FastCompiler.VisitImportStatement.cs:line 29
     at YantraJS.Utils.AstMapVisitor`1.Visit(AstNode node) in D:\projects\yantra\YantraJS.Core\FastParser\Ast\AstMapVisitor.cs:line 117
     at YantraJS.Core.FastParser.Compiler.FastCompiler.VisitProgram(AstProgram program) in D:\projects\yantra\YantraJS.Core\FastParser\Compiler\FastCompiler.VisitProgram.cs:line 58
     at YantraJS.Utils.AstMapVisitor`1.Visit(AstNode node) in D:\projects\yantra\YantraJS.Core\FastParser\Ast\AstMapVisitor.cs:line 45
     at YantraJS.Core.FastParser.Compiler.FastCompiler..ctor(StringSpan& code, String location, IList`1 argsList, ICodeCache codeCache) in D:\projects\yantra\YantraJS.Core\FastParser\Compiler\FastCompiler.cs:line 118
     at YantraJS.CoreScript.<>c__DisplayClass0_1.<Compile>b__0() in D:\projects\yantra\YantraJS.Core\CoreScript.cs:line 43
     at YantraJS.Emit.DictionaryCodeCache.<>c__DisplayClass2_0.<GetOrCreate>b__0(StringSpan k) in D:\projects\yantra\YantraJS.Core\Emit\DictionaryCodeCache.cs:line 21
     at YantraJS.Core.Core.Storage.ConcurrentStringMap`1.GetOrCreate(StringSpan& key, Func`2 value) in D:\projects\yantra\YantraJS.Core\Core\Storage\ConcurrentStringMap`1.cs:line 93
     at YantraJS.Emit.DictionaryCodeCache.GetOrCreate(JSCode& code) in D:\projects\yantra\YantraJS.Core\Emit\DictionaryCodeCache.cs:line 20
     at YantraJS.Emit.DictionaryCodeCache.YantraJS.Emit.ICodeCache.GetOrCreate(JSCode& code)
     at YantraJS.CoreScript.Compile(StringSpan& code, String location, IList`1 args, ICodeCache codeCache) in D:\projects\yantra\YantraJS.Core\CoreScript.cs:line 46
     at YantraJS.CoreScript.Evaluate(String code, String location, ICodeCache codeCache) in D:\projects\yantra\YantraJS.Core\CoreScript.cs:line 70
     at YantraJS.Core.JSContext.<>c__DisplayClass96_0.<<Eval>b__0>d.MoveNext() in D:\projects\yantra\YantraJS.Core\Core\JSContext.cs:line 693
  --- End of stack trace from previous location ---
     at Microsoft.Threading.AsyncPump.Run[T](Func`1 func) in D:\projects\yantra\YantraJS.Core\Core\Promise\AsyncPump.cs:line 32
     at YantraJS.Core.JSContext.Eval(String code, String codeFilePath) in D:\projects\yantra\YantraJS.Core\Core\JSContext.cs:line 692
     at Program.<Main>$(String[] args) in D:\projects\yantra\test\Program.cs:line 10

During debugging, it was found that the variable "require" is null
and when calling require.Expression a NullReferenceException occurs
Please Fix

Additional Info
Code for test

My patch, that i want PR (this doent break code ExpressionCompiler)

Which version are you using? There is no error in the latest build as all unit tests run successfully.
For modules, do not use Eval instead use RunAsync or RunExportsAsync.

Do you want JSModuleContext without clr for enhanced security?
In that case, how will you support external modules?

Please provide your source code of how you are executing it, null may be due to some other error like corrupted async state or synchronization context may be null. In this case, please provide sample repo or code.
Also take a look at CSX Modules, may be you will find some hints. In source code for YantraTests.

i copy latest commit from this repo

Yes, i actually do this
I want support to external modules via register module method

upd. at current moment i work on RunAsync that execude string of code aka compiling module from string

Thanks for the response.

I will add support to register external modules manually.
And I will add support to run module from the string instead of file.

As soon as this is done, I will update this issue. Thanks for your contribution.

I am currently working on item 2.
Will be implemented soon (I want to implement it myself, for experience)

trying to implement it myself
however, unfortunately, there is not enough understanding of the system of modules and the internal structure of the compiler
I hope you implement this features

I have add comments in your pull request, if you can update it, I will merge it.
As of now, try to save the text in a temp file and give the path, and after execution delete the file.
I will try to provide an easier way in near future.

I have added RunScriptAsync method in JSModuleContext. You can use it this way.

using var context = new JSModuleContext( enableClrIntegration: false);
await context.RunScriptAsync( script , homeFolder );

RunScriptAsync return exported values (like export const, class, module.export, etc)

Yes it will return all exports from the module.


⚠ Bofrost.at has degraded performance

In bb39bb1, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2279 ms

Resolved: Bofrost.at performance has improved in 4a70650 after 7 minutes.


DBZ-6742 use JSON format for JMX Notitication userData

Closes: https://issues.redhat.com/browse/DBZ-6742

@vsantona Applied, thanks!
@mfvitale Feel free to merge the PRs next time, there is no need to wait for me in similar cases.


Sytax error in webpack package

Describe the bug
Installation fails due to a Syntax error in the webpack package
To Reproduce
Steps to reproduce the behavior:

Run install.sh

Stacktraces
SyntaxError: Invalid regular expression: /^[_\p{L}][_0-9\p{L}]*$/: Invalid property name in character class
    at internalCompileFunction (node:internal/vm:73:18)
    at wrapSafe (node:internal/modules/cjs/loader:1176:20)
    at Module._compile (node:internal/modules/cjs/loader:1218:27)
    at Module._extensions..js (node:internal/modules/cjs/loader:1308:10)
    at Module.load (node:internal/modules/cjs/loader:1117:32)
    at Module._load (node:internal/modules/cjs/loader:958:12)
    at Module.require (node:internal/modules/cjs/loader:1141:19)
    at require (node:internal/modules/cjs/helpers:110:18)
    at Object.<anonymous> (/home/luke/test/taxaminer-dashboard/react-frontend/node_modules/webpack/lib/Compilation.js:53:25)
    at Module._compile (node:internal/modules/cjs/loader:1254:14)

Upon first inspection, this is a compatibility issue with the most recent 18.15.0 versions of the Nodejs conda package. Forcing conda to install 18.14.0 fixes the issue


[WIP] Display item value heatmap in bank

TODO:

[ ] cleanup whitespace
[ ] add prices to heatmap maybe?
[ ] add minimum price threshold for display?
[ ] separate commits

Displays an item value heatmap in bank when hotkeys are held.

Ctrl = GE value heatmap
Ctrl + shift = Alch value heatmap

Closes #2062
Supersedes/Closes #9137
Supersedes/Closes #7129

This is nice, I like it. A few questions:

What do you envision the final product will look like? I’m assuming the colored circles are placeholders?
What do the different colors mean and how would users know?

One of the biggest things I wanted was to be able to not only see the most expensive items, but also see their ordering and how much they were actually valued. I wonder if there’s any nice way we could display that information.

Any progress on this?

This was added as an external plugin.


Flatten circular references

https://github.com/stoplightio/platform-internal/blob/fa5ef4a94ec892ba87050fad1ea66185ec9b2e20/packages/public-api/src/export/refs/resolve.ts#L111-L114
Overview
Currently it's really hard to understand where a circular reference is happening, because if we ignore circular references and one exists the we unwind the current stack and set leave the original reference.
For example when resolving kitchen-sink.v1.yaml and specifically indirect_circular_ref_file, the original references is left in place because there is a circular ref from kitchen-sink.v1.yaml -> indirect-circular.v1.yaml -> kitchen-sink.v1.yaml.
kitchen-sink.v1.yaml
title: Kitchen Sink
type: object
description: 'kitchen sink model test'
properties:
  basic_property:
    type: string
  simple_file_ref:
    $ref: ./simple.v1.yaml
  simple_file_as_http_ref_masked:
    $ref: 'http://stoplight-local.com:8080/api/v1/projects/bear/bear-cave/nodes/fixtures/simple.v1.yaml?mid=7'
  ref_to_self_as_file:
    $ref: ./kitchen-sink.v1.yaml
  ref_to_self_as_http:
    $ref: 'http://stoplight-local.com:8080/api/v1/projects/bear/bear-cave/nodes/fixtures/kitchen-sink.v1.yaml'
  ref_to_self_as_http_optomized_bundle:
    $ref: 'http://stoplight-local.com:8080/api/v1/projects/bear/bear-cave/nodes/fixtures/kitchen-sink.v1.yaml?deref=optimizedBundle'
  ref_to_self_as_http_with_mask_all_props:
    $ref: 'http://stoplight-local.com:8080/api/v1/projects/bear/bear-cave/nodes/fixtures/kitchen-sink.v1.yaml?deref=bundle&mid=8'
  indirect_circular_ref_file:
    $ref: ./indirect-circular.v1.yaml
  indirect_circular_ref_http:
    $ref: 'http://stoplight-local.com:8080/api/v1/projects/bear/bear-cave/nodes/fixtures/indirect-circular.v1.yaml'
  indirect_circular_ref_http_optomized_bundled:
    $ref: 'http://stoplight-local.com:8080/api/v1/projects/bear/bear-cave/nodes/fixtures/indirect-circular.v1.yaml?deref=optimizedBundle'

indirect-circular.v1.yaml
title: indirect-circular
type: object
properties:
  id:
    type: string
  circular_ref_as_file:
    $ref: ./kitchen-sink.v1.yaml
  circular_ref_back_as_http:
    $ref: http://stoplight-local.com:8080/api/v1/projects/bear/bear-cave/nodes/fixtures/kitchen-sink.v1.yaml
  circular_ref_back_as_http_optomized_bundle:
    $ref: http://stoplight-local.com:8080/api/v1/projects/bear/bear-cave/nodes/fixtures/kitchen-sink.v1.yaml?deref=optimizedBundle

Proposal
Flatten circular references, so when resolving kitchen-sink.v1.yaml and specifically indirect_circular_ref_file, we replace indirect_circular_ref_file with the contents of indirect-circular.v1.yaml.
    dereference: {
       circular: 'flatten',
    },

Example result of flattening indirect_circular_ref_file
  indirect_circular_ref_file:
      title: indirect-circular
    type: object
    properties:
      id:
        type: string
      circular_ref_as_file:
        $ref: ./kitchen-sink.v1.yaml
      circular_ref_back_as_http:
        $ref: http://stoplight-local.com:8080/api/v1/projects/bear/bear-cave/nodes/fixtures/kitchen-sink.v1.yaml
      circular_ref_back_as_http_optomized_bundle:
        $ref: http://stoplight-local.com:8080/api/v1/projects/bear/bear-cave/nodes/fixtures/kitchen-sink.v1.yaml?deref=optimizedBundle

Once this work is merged, will it make its way into Prism / Spectral? My OAS apparently has a bunch of circular refs (they do make sense unfortunately), and right now I'm staring down a nasty multi-step process to de-ref, I would love to just have support within the tools themselves.

@jhodges10 I don't know when prism will support circular refs, but it is on our road map. Spectral uses a different project to resolve refs, I don't know when and if it will be replaced or if it's on the road map. @P0lip does spectral not resolve refs?


LGTM fixes

I wouldn't just comment these things out, most of them can be deleted (like the CANPacker import)

You're right, even the commented code that needed the import is dead. Fixed in master.


reference/sse: clarify retry behavior

document initial wait time
provide an example for "infinite" retries

The initial wait time is 1s.

I find this comment confusing rather than clarifying. The initial wait time is retryScaler multiplied by retryMaxWaitMs, which happens to be 1 second given their default values, which are documented.

Use a large number (e.g. Number.MAX_SAFE_INTEGER) for "infinite" retries.

Does this really need to be stated? Definitely not “infinite”.

I was looking in the documentation for these two values: What does the backoff algorithm start with and how can I turn off the retry limit since I want a constant, predictable, infinite retry for the '/event' endpoint for screens where there is no user to hit F5. I cannot afford the screen to stall.
Initial retry time: Is this better? The default is not on the page. Probably in the code only.
As for disabling the retry limit: I think a sentinel value or a separate flag to disable the limit makes the intent more clear (and maybe less error-prone than data-on-load="sse('/events', {retryScaler:1,retryMaxCount:Number.MAX_SAFE_INTEGER})". #382 says that using Number.MAX_SAFE_INTEGER achieves virtually the same thing. It is logical but not obvious - at least not to me.

I know you mean well but I find the contributions misleading more than providing any real value. The “initial wait time” is not 1 second! It is a function of two settings.

So after looking into this a bit more, I realised that I misunderstood, and that the retry interval is hard-coded to be 1 second. I’ve created a PR to make this configurable at https://github.com/starfederation/datastar/pull/393. Apologies for not understanding you initially.


Expand Orchest API tracking

Description
This PR expands the anonymized tracking of the Orchest API. Changes to already existing events:

pipeline run --> pipeline_run start
creating (POSTING) a job will not incorrectly send a pipeline run event anymore, but will instead send a job create event. The job update event will allow to infer the type of job (immediate, scheduled,  cron job).

Checklist

[X] The documentation reflects the changes.
[X] I have manually tested the application to make sure the changes don’t cause any downstream issues.
[X] In case I changed one of the services’ models.py I have performed the appropriate database migrations.

The changes to existing events to my knowledge are:
pipeline run --> pipeline_run start
Is there more? I think it isn't problematic.

@ricklamers I've updated the PR description.


Implement generic git handler

This one should definitely use git proper for any requests. And probably some sort of specialized caching technique (since it's not just gonna go through the http client)

Closing this in favor of #4 as a joint issue, since these should be done together.


docs(tutorial/blog): Add guidance in case of GitHub auth error

Closes: #

[ ] Docs
[ ] Tests

Testing Strategy:

This should not happen. Can you provide reproduction steps?

@kentcdodds I it happened to me because I had an expired  GITHUB_TOKEN environment variable in my ~/.bash_profile and had to override it with an inline valid token. If I first unset the expired token then the create-remix command works as is:
unset GITHUB_TOKEK
npx create-remix --template remix-run/indie-stack blog-tutorial

Glad you figured out the issue.


Use database for Noid minter state

I'm trying to get the database minting working locally, with no luck so far.  I've seen a few people talking about using this on Slack, so I'll see if I can get some help getting started.

Upgrading to the latest sqlite3 gem fixed the issues I was having.


About  ccnet.py. 'return [x, x_dsn]'

def forward(self, x, recurrence=1):
        x = self.relu1(self.bn1(self.conv1(x)))
        x = self.relu2(self.bn2(self.conv2(x)))
        x = self.relu3(self.bn3(self.conv3(x)))
        x = self.maxpool(x)
        x = self.layer1(x)
        x = self.layer2(x)
        x = self.layer3(x)
        x_dsn = self.dsn(x)
        x = self.layer4(x)
        x = self.head(x, recurrence)
        return [x, x_dsn]

I do not know why return [x, x_dsn]? why not return (torch.cat(x, x_dsn))

@yearing1017 It's for deep supervision. Please refer to the loss function.


How to select all items with date "today"?

I have items with field date
Something like this, but for today
MyApplication.getInstance().getData().toBlocking().select(Actions.class)
                .where(Actions.TYPE.eq(DBHelper.ACTION_STEPS))
                .and(Actions.DATE.eq(Now.now(Date.class))).get().toList();

Now I have to create two dates(start of day and end of day) and select using this interval. It looks ugly)

You can do something like this:
        Date now = new Date(); // this can be changed to end of day if needed instead
        Calendar calendar = Calendar.getInstance();
        calendar.set(Calendar.HOUR_OF_DAY, 0);
        data.select(Actions.class).where(Actions.DATE.between(calendar.getTime(), now)).get());

The java time api isn't super easy to working with, may also want to consider using the Java 8 java.time instead if possible.


[PROJECT] Create Collective Flow (implementation)

Purpose
We want to make it easier for collectives to find the right path for them, thus aligning this with our new business model for fiscal hosts.
Owner
Project: @piamancini
Technical: @sbinlondon & @znarf
Prioritization

Does this serve our mission?

Yes, it aims to help us become more sustainable

Will this help to make us financially sustainable within a year?

Yes, it will hopefully have a direct impact on revenue by aligning new collectives with the new business model.

Is it feasible to build and maintain with our team and runway?

Yes, it is mostly creating a new flow using already created components with some new image design and a minimal amount of new components being created.

Can this make us move faster?

Yes, by taking away ambiguity in the signup process and decreasing the amount of people who get in touch.

Does this increase quality and not technical debt?

Yes, it will increase quality of the create collective flow and we'll aim to implement it in a sustainable way that won't increase technical debt.

Will this bring us more Collective and financial contributors?

Yes, hopefully, if they find it easier to sign up and figure out which path is correct for them.

Is this already solved elsewhere or is it our unique contribution?

Our unique contribution 😊
Roadmap & Timeframe

[x] Design (see #2366)
[x] Implementation of the final design
[x] Migrate Create Collective to GraphQL API v2
[x] Feedback round
[x] Making changes based on feedback round
[x] Final implementation
[x] Staging
[x] Last bug fixes
[x] Söft delivery date: end of Q1 2020 (March 31)

Success metrics

More self-hosted Collectives created
Increase per-Collective revenue

Pending:

review and merge tests from @jeffin143 for the onboarding modal
finish tests for updated CCF by @sbinlondon
finish new tests for AFC flow by @sbinlondon


Ansible Operator undefined variable using nested vars

Hey, I have error:
The task includes an option with an undefined variable. The error was: 'dict object' has no attribute 'tlsSecret'
within Ansible jinja2 templete:
apiVersion: v1
kind: Template
metadata:
  name: "wxu-web"
  annotations:
    description: "Openshift a set of object"
labels:
{% for k, v in default.items() %}
{% if v is not none %}
  {{ k }}: {{ v }}
{% endif %}
{% endfor %}
objects:
- apiVersion: networking.k8s.io/v1beta1
  kind: Ingress
  metadata:
    name: {{ default.release_name }}
  spec:
    rules:
    {% for hosts in ingress -%}
    {% for host in hosts.hosts -%}
    - host: {{ host }}
      http:
        paths:
        - backend:
            serviceName: {{ default.release_name }}
            servicePort: {{ service_port }}
          path: /
    {% endfor -%}
    {% endfor -%}
    tls:
    {% for hosts in ingress -%}
      - hosts:
      {% for host in hosts.hosts -%}
      - {{ host }}
      {% endfor -%}
      secretName: {{ hosts.tlsSecret }}
    {% endfor %}

and with following custrom_resources:
apiVersion: xxx.xxx.com/v1alpha1
kind: xxx
metadata:
  name: xxx-xxx
spec:

# Default Jinja2 Template
  namespace: tutorial
  default:
    release_name: xxx-stage
    app_name: xxx
    cluster: xxx-xxx
    region: xxx-xxx
    datacenter: xxx

  # Service Jinja2 Template
  service_port: xxx

  # Ingress Jinja2 Template
  ingress:
    - hosts:
        - xxx-stage.xxx.com
        - xxx-stage-xxx.xxx.com
        - xxx-stage-xxx-xxx.xxx.com
      tlsSecret: star-xxx-com
    - hosts:
        - xxx-stage.xxx.com
        - xxx-stage-xxx.xxx.com
        - xxx-stage-xxx-xxx.xxx.com
      tlsSecret: star-xxx-com

I am not sure but looks like Operator-SDK not handling nested variables with Ansible Template

secretName: {{ hosts.tlsSecret | default(none) }} fixed issue with undefined var but I have definition for this var and still not sure if it works correct

This seems like an upstream ansible issue rather than an SDK-specific one.


convertValuetoArray($value) not compatible with __call()

Hi @hopeseekr ,
Thank you for your work, this package. I am using this in favor of the Spatie package because I also think DTOs should be immutable.
I run into a problem with the $dto->toArray() method when I have a \Illuminate\Http\UploadedFile in the DTO. Because it has magic function __call() implemented (by the  \Illuminate\Support\Traits\Macroable trait), the next check always returns true (even when no toArray exists). Therefore I get this error:

Method Illuminate\Http\UploadedFile::toArray does not exist.

The problem is known: https://www.php.net/manual/en/function.is-callable.php#118623
In \PHPExperts\SimpleDTO\SimpleDTO::convertValueToArray there is this check for every data attribute value:
        if (is_object($value)) {
            if (is_callable([$value, 'toArray']) && !($value instanceof Carbon)) {
                return $value->toArray();
            }

            if ($value instanceof \stdClass) {
                return (array) $value;
            }
        }

source: https://github.com/PHPExpertsInc/SimpleDTO/blob/f7b027eae78d16f8042b12ff4942e61e7321cd8f/src/SimpleDTO.php#L328
The problem: is_callable has trouble figuring out that UploadedFile has no callable toArray, it says true but instead it is false.
I think the following would fix the problem:
        if (is_object($value)) {
            if (method_exists($value, 'toArray') && !($value instanceof Carbon)) {   // <=== this would work
                return $value->toArray();
            }

            if ($value instanceof \stdClass) {
                return (array) $value;
            }
        }

Would you consider implementing this fix?

Sure. I'll test it later tonight.

Available in version 2.6.0!

Thank you.


Add basic docker-compose.yaml for Jupyter notebooks

This can make easier to spin up a test infrastructure for jupyter testing.

How is this different from instructions here: https://github.com/jaegertracing/jaeger-analytics-java#run-jupyter-as-docker ?

It provides a way to pull up all components without building the project. It's almost identical, but this can be more convenient for some.
The idea came from https://github.com/jaegertracing/jaeger-analytics-java/pull/53, because that had individual docker commands in it, compared to those, a docker-compose is more simple.
I'd say merging this w/ #53 can make sense (if the idea of adding a separate deploy method is welcomed), as that has the instructions.


Allow clients to request specific manifest media types

The current registry/client sends the registered manifest types in
random order. Allow clients to request a single specific manifest type
or a preferred order as per the HTTP spec.
Fixes #2385
Signed-off-by: Clayton Coleman<EMAIL_ADDRESS>

LGTM

@smarterclayton Could you please rebase this? There was a fix in master to deal with some upstream changes to golint.

Updated

@dmcgowan PTAL


Add notice about cupy.array_api removal

context: https://github.com/cupy/cupy/issues/8470#issuecomment-2285383328.
Closes #8470 and #8747.
cc/ @leofang

No, supporting array API has been one of the goals of NumPy 2(.0), and it's been delivered
https://numpy.org/doc/stable/release/2.0.0-notes.html#highlights


set custom slide indicator

hi, is there any way i can custom my own indicator?  i need to change the shape of the indicator

Hi @rizkyputrapb,
Yes, you can use a custom indicator.
Please refer to the example in the package.
Thanks,
Nikhil


checkbox sharing the same name

254 Is there way to have different names? My problem right now is that I have two tables with radion buttons and when I clicked the radio on the other table it removes the check on the other table.

it's a bug, will be fixed on next version. Thanks your report :+1:

Okay thanks! :+1:

@gideonairex, fixed and released on v1.6.0. Thanks :)

@AllenFang Thank you! :)


data commit

Issue: 73
Short description of what this resolves:
Add data class
Changes proposed in this pull request and/or Screenshots of changes:
-add variables in the data classes

correct option should be Int, not string

Ok

I will  change and re commit


homeassistant 2021.7  device automation triggers

Logger: homeassistant.config
Source: components/device_automation/init.py:91
First occurred: 01:32:05 (3 occurrences)
Last logged: 01:32:05
Invalid config for [automation]: Integration 'xiaomi_gateway3' does not support device automation triggers (See ?, line ?).
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/device_automation/__init__.py", line 85, in async_get_device_automation_platform
    platform = integration.get_platform(platform_name)
  File "/usr/src/homeassistant/homeassistant/loader.py", line 498, in get_platform
    cache[full_name] = self._import_platform(platform_name)
  File "/usr/src/homeassistant/homeassistant/loader.py", line 503, in _import_platform
    return importlib.import_module(f"{self.pkg_path}.{platform_name}")
  File "/usr/local/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 855, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/config/custom_components/xiaomi_gateway3/device_trigger.py", line 2, in <module>
    from homeassistant.components.device_automation import TRIGGER_BASE_SCHEMA
ImportError: cannot import name 'TRIGGER_BASE_SCHEMA' from 'homeassistant.components.device_automation' (/usr/src/homeassistant/homeassistant/components/device_automation/__init__.py)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/automation/config.py", line 104, in _try_async_validate_config_item
    config = await async_validate_config_item(hass, config, full_config)
  File "/usr/src/homeassistant/homeassistant/components/automation/config.py", line 72, in async_validate_config_item
    config[CONF_TRIGGER] = await async_validate_trigger_config(
  File "/usr/src/homeassistant/homeassistant/helpers/trigger.py", line 49, in async_validate_trigger_config
    conf = await platform.async_validate_trigger_config(hass, conf)
  File "/usr/src/homeassistant/homeassistant/components/device_automation/trigger.py", line 17, in async_validate_trigger_config
    platform = await async_get_device_automation_platform(
  File "/usr/src/homeassistant/homeassistant/components/device_automation/__init__.py", line 91, in async_get_device_automation_platform
    raise InvalidDeviceAutomationConfig(
homeassistant.components.device_automation.exceptions.InvalidDeviceAutomationConfig: Integration 'xiaomi_gateway3' does not support device automation triggers

Same problem here. I have installed homeassistant core-2021.7.0 and supervisor-2021.06.8.

same here.
issue started after update to core-2021.7.0

Hi, same for me

exactly the same issue here... only after updated 2021.7

Same here for Xiaomi switch

Rename device trigger base schema to DEVICE_TRIGGER_BASE_SCHEMA

Replace in 'custom_components/xiaomi_gateway3/device_trigger.py' 'TRIGGER_BASE_SCHEMA' to 'DEVICE_TRIGGER_BASE_SCHEMA'
--- device_trigger.py.orig      2021-07-08 14:36:12.193019049 +0300
+++ device_trigger.py   2021-07-08 14:37:41.956014894 +0300
@@ -1,5 +1,5 @@
 import voluptuous as vol
-from homeassistant.components.device_automation import TRIGGER_BASE_SCHEMA
+from homeassistant.components.device_automation import DEVICE_TRIGGER_BASE_SCHEMA
 from homeassistant.components.homeassistant.triggers import \
     state as state_trigger
 from homeassistant.const import *
@@ -10,7 +10,7 @@
 from .core import zigbee
 from .sensor import BUTTON

-TRIGGER_SCHEMA = TRIGGER_BASE_SCHEMA.extend(
+TRIGGER_SCHEMA = DEVICE_TRIGGER_BASE_SCHEMA.extend(
     {
         vol.Required(CONF_TYPE): cv.string,
         vol.Required('action'): cv.string

Rename device trigger base schema to DEVICE_TRIGGER_BASE_SCHEMA

Replace in 'custom_components/xiaomi_gateway3/device_trigger.py' 'TRIGGER_BASE_SCHEMA' to 'DEVICE_TRIGGER_BASE_SCHEMA'
--- device_trigger.py.orig      2021-07-08 14:36:12.193019049 +0300
+++ device_trigger.py   2021-07-08 14:37:41.956014894 +0300
@@ -1,5 +1,5 @@
 import voluptuous as vol
-from homeassistant.components.device_automation import TRIGGER_BASE_SCHEMA
+from homeassistant.components.device_automation import DEVICE_TRIGGER_BASE_SCHEMA
 from homeassistant.components.homeassistant.triggers import \
     state as state_trigger
 from homeassistant.const import *
@@ -10,7 +10,7 @@
 from .core import zigbee
 from .sensor import BUTTON

-TRIGGER_SCHEMA = TRIGGER_BASE_SCHEMA.extend(
+TRIGGER_SCHEMA = DEVICE_TRIGGER_BASE_SCHEMA.extend(
     {
         vol.Required(CONF_TYPE): cv.string,
         vol.Required('action'): cv.string

Its realy work fine!

From the changes in 2021.7
Rename device trigger base schema to DEVICE_TRIGGER_BASE_SCHEMA

Replace in 'custom_components/xiaomi_gateway3/device_trigger.py' 'TRIGGER_BASE_SCHEMA' to 'DEVICE_TRIGGER_BASE_SCHEMA'
--- device_trigger.py.orig      2021-07-08 14:36:12.193019049 +0300
+++ device_trigger.py   2021-07-08 14:37:41.956014894 +0300
@@ -1,5 +1,5 @@
 import voluptuous as vol
-from homeassistant.components.device_automation import TRIGGER_BASE_SCHEMA
+from homeassistant.components.device_automation import DEVICE_TRIGGER_BASE_SCHEMA
 from homeassistant.components.homeassistant.triggers import \
     state as state_trigger
 from homeassistant.const import *
@@ -10,7 +10,7 @@
 from .core import zigbee
 from .sensor import BUTTON

-TRIGGER_SCHEMA = TRIGGER_BASE_SCHEMA.extend(
+TRIGGER_SCHEMA = DEVICE_TRIGGER_BASE_SCHEMA.extend(
     {
         vol.Required(CONF_TYPE): cv.string,
         vol.Required('action'): cv.string

works for me!  core 2021.7.0 | supervisor 2021.06.8.

From the changes in 2021.7
Rename device trigger base schema to DEVICE_TRIGGER_BASE_SCHEMA

Replace in 'custom_components/xiaomi_gateway3/device_trigger.py' 'TRIGGER_BASE_SCHEMA' to 'DEVICE_TRIGGER_BASE_SCHEMA'
--- device_trigger.py.orig      2021-07-08 14:36:12.193019049 +0300
+++ device_trigger.py   2021-07-08 14:37:41.956014894 +0300
@@ -1,5 +1,5 @@
 import voluptuous as vol
-from homeassistant.components.device_automation import TRIGGER_BASE_SCHEMA
+from homeassistant.components.device_automation import DEVICE_TRIGGER_BASE_SCHEMA
 from homeassistant.components.homeassistant.triggers import \
     state as state_trigger
 from homeassistant.const import *
@@ -10,7 +10,7 @@
 from .core import zigbee
 from .sensor import BUTTON

-TRIGGER_SCHEMA = TRIGGER_BASE_SCHEMA.extend(
+TRIGGER_SCHEMA = DEVICE_TRIGGER_BASE_SCHEMA.extend(
     {
         vol.Required(CONF_TYPE): cv.string,
         vol.Required('action'): cv.string

Working perfectly again! Thank you!

This is a plug-in problem, or a problem with the new version of HA

From the changes in 2021.7
Rename device trigger base schema to DEVICE_TRIGGER_BASE_SCHEMA

Replace in 'custom_components/xiaomi_gateway3/device_trigger.py' 'TRIGGER_BASE_SCHEMA' to 'DEVICE_TRIGGER_BASE_SCHEMA'
--- device_trigger.py.orig      2021-07-08 14:36:12.193019049 +0300
+++ device_trigger.py   2021-07-08 14:37:41.956014894 +0300
@@ -1,5 +1,5 @@
 import voluptuous as vol
-from homeassistant.components.device_automation import TRIGGER_BASE_SCHEMA
+from homeassistant.components.device_automation import DEVICE_TRIGGER_BASE_SCHEMA
 from homeassistant.components.homeassistant.triggers import \
     state as state_trigger
 from homeassistant.const import *
@@ -10,7 +10,7 @@
 from .core import zigbee
 from .sensor import BUTTON

-TRIGGER_SCHEMA = TRIGGER_BASE_SCHEMA.extend(
+TRIGGER_SCHEMA = DEVICE_TRIGGER_BASE_SCHEMA.extend(
     {
         vol.Required(CONF_TYPE): cv.string,
         vol.Required('action'): cv.string

It's OK,  thinks.

Fixed in latest master thanks to @aatrubilin

@AlexxIT Is there a timeline for when this new release will be pushed? I've changed my automations to work use states but would be nice to regain the ability to use device automations.

Fixed in latest release

Hi! After automation related to such a bug in the latest version of HA (2021.9.7). Gateway firmware version 1.5.0_0102. Can someone tell me a solution?
Logger: homeassistant.config
Source: config.py:464
First occurred: 15:23:54 (1 occurrences)
Last logged: 15:23:54
Invalid config for [automation]: Integration 'xiaomi_gateway3' does not provide trigger support. Got OrderedDict([('default_config', {}), ('lovelace', OrderedDict([('mode', 'yaml')])), ('tts', [OrderedDict([('platform', 'google_translate')])]), ('scene', {}), ('recorder', OrderedDict([('db_url', 'mysql://hass:hass@core-mariadb/homeassistant?charset=utf8'), ('purge_keep_days', 3), ('include', OrderedDict([('domains', ['switch', 'light']), ('entities', ['sensor.maria_db_size', 'sensor.0x00158d0001581109_temperature', 'sensor.0x00158d000159c7e6_temperature', 'sensor.0x00158d0001dcd47e_temperatu.... (See /config/configuration.yaml, line 24).

Seems this Issue is back with release  -
Core
2024.10.2
Supervisor
2024.10.0
Operating System
13.1
Frontend
20241002.3
Triggers: Integration 'xiaomi_gateway3' does not support device automation triggers


Placeholder hide when dropdown opened and not searchable

It is often happens, you don't need searchable input for selects with short options list(2-5). And you just set :searchable='false'. But it makes input text field to remove from DOM(because in source code it has v-if="searchable"). Also css has rule:
.multiselect--active .multiselect__placeholder {
  display: none;
}

so PLACEHOLDER is hidden, because input(and his placeholder) not rendered and placeholder slot hidden by css.
So I need select without search. My workaround is @open="onOpen" and
onOpen(id) {
    document.querySelector('#' + id).setAttribute('readonly', true);
}

It work - it just disable search functional as I need, but placeholder is visible.
But now I think, that many tabbind/searching issues can be fixed by using readonly attr of search input and always render input(modify v-if="searchable" to :readonly="!searchable" ).
Reproduction Link
https://jsfiddle.net/karakum/8npu3mgh/
Steps to reproduce

version 2.1.6
empty selection

Expected behaviour
Placeholder visible while dropdown opened
Actual behaviour
Placeholder invisible while dropdown opened

bump? I'm having the same issue.


Error message when selecting toggle

Just installed package and receive the following error when selecting git-time-machine -> Toggle

https://github.com/littlebee/git-time-machine/issues/56#issuecomment-215983124

Closing.  Added more detail to troubleshooting section of readme.  will publish today


Update installation instructions

These work in the current Tridactyl betas and Tridactyl stable (unreleased) 1.21.0
You might want to wait for Tridactyl 1.21.0 to be released (perhaps within the next couple of weeks) before merging it, or you might not.
I knew if I didn't make this PR now I would forget to tell you : )

Great to hear that these themes still work somehow! It's been a while since I've used tridactyl.


[suggestion] Should SumeragiHandle contain Arc?

This issue originates from the following discussion about SumeragiHandle.
queue: Arc<Queue>,
    _thread_handle: Arc<ThreadHandler>,
    message_sender: mpsc::SyncSender<MessagePacket>, // This must be kept alive until the thread exits, ergo it's after _thread_handle.

Erigara: Good point.
appetrosyan: Actually, the better way to express this is by impl Drop. Just because Arc::drop was called doesn't mean the thread is stopped, just that the refcount got decremented.
appetrosyan: In fact instead of silently shutting off the Sender, it's a good idea to block until the refcount reaches zero and panic if it doesn't in a reasonable amount of time.
SamHSmith: I disagree. There are no copies of the ThreadHandle arc outside of the SumeragiHandle. So there is a one to one relationship between the number of SumeragiHandle's, Arc and SyncSenders.
appetrosyan: In which case it's not supposed to be an Arc, which is a partial owning reference.

Alternatives
If it's an Arc then the reference count can go up and can go down, and when it reaches zero, the thing gets deallocated. This is useful whenever you don't know for sure which object will hold the reference last, but that one of a few objects will. It has reference semantics.
If you know for sure that a particular object will hold a reference to an object before deallocation, there's no need for an Arc. In this particular case you have several options:

Just use regular borrows. Best option in simple cases. Still doable in complex cases.
Use Weak. Difference is, it acts like a C pointer, except it returns None instead of NULL. Syntactically this has more overhead than borrows, because you have to null-check before dereferencing. Occasionally this can be optimised by the compiler.
Use unsafe regular pointer dereferencing. If you can prove (and the borrow checker can't) that the other objects would be constructed at a time after malloc and no use after free would take place, then eliding the Null check for performance reasons may be acceptable.

Now the whole question is the intent. We want to have multiple SumeragiHandles, hence Clone. We want for the thread to be shut down when all handles go out of scope. Final piece of the puzzle is that

there is a one to one relationship between the number of SumeragiHandle's, Arc and SyncSenders

Indeed. Why two objects which can almost always come only together are not grouped into a struct at least? Are there any other pieces of data similarly associated? What can this tell us about the refcounts for Kura, Queue etc. Can we consolidate refcounts?

But the bigger question is why do we even bother stopping the threads on drop? There's only one way to trigger graceful shutdown, and every other way involves being killed one way or another. Moreover, many data here are basically static and only need to be de-allocated on graceful shutdown.


In the general function, the calculation result of the trigonometric function with a coefficient before shows a precision error

Description
In the general function, the calculation result of the trigonometric function with a coefficient before shows a precision error.
Reproduction
Input the trigonometric function with a coefficient before.
Expected behavior
It is expected that in the general function, the calculation result of the trigonometric function with a coefficient before is correct.
Screenshots
[Example 1]

After enclosing the trigonometric functions in parentheses, the image renders fine:

[Example 2]

After enclosing the trigonometric functions in parentheses, the image renders fine:

This has something to do with the v8 engine


Modified tests June 17

Summary:Modified tests
TC-BOOl-1.1
TC-CC-1.1
TC-FLW-1.1
TC-I-1.1
TC-ILL-1.1
TC-ILL-2.1
TC-LVL-1.1 to TC-LVL-6.1
TC-MC-1.*
TC-MOD-1.1
TC-OCC-1.1
TC-OCC-2.1
TC-OO-1.1
TC-OO-2.4
TC-PCC-1.1
TC-PRS-1.1
TC-PS-1.1
TC-RH-2.1
TC-TSUIC-1.1
TC-ULABEL-1.1
TC-WNCV-1.1
TC-WNCV-2.1
TC-WNCV-4.3
Added auto generated files
Attached execution logs for reference:
Execution_log.txt

Fast tracking tests.


Refactoring build and test logic

Changes summary

Remove Dapper by relying on docker buildx and Makefile to download required tooling.
Migrate away from Drone to GitHub actions.
Refactor E2E testing logic. In CI it is now executed against versions 2.7, 2.8 and 2.9.
External dependency versions (e.g. sonobuoy, kind, etc) are centralised at hack/make/deps.mk.

Although the cross-compilation is verified as part of the new building process, the arm64 binary/image was also manually tested:
$ uname -a
Linux rpi-top 6.1.21-v8+ #1642 SMP PREEMPT Mon Apr  3 17:24:16 BST 2023 aarch64 GNU/Linux

$ podman run --rm quay.io/paulinhu/security-scan:dev kb-summarizer --version
kb-summarizer version 4168e18-dirty


Add function for changing charset with unit test.

Creates a function to change the charset
Adds a unit test to test if the sequences have invalid length when converting into tokens
Does not yet change the generation of tokens yet.

Thanks @Enet4, I don't have time today, so I'll look at this tomorrow 👍

@Enet4 I had a quick look at how I could add the flag to into_token. Could you have a quick look at my latest commit to give an indication if I'm going in the right direction? I don't fully understand how that code fits together. Can you tell me if this approach makes sense?


Battle for Azeroth PVP and PVE Milestones

As Battle for Azeroth nears release this bot needs to be updated with data from the new expansion, and data from the previous expansion needs to be removed. I'll be using this ticket here to track the progress.

Checklist
[ ] Add raid data for the first Tier, Uldir.
[ ] Update the keystone master achievements to their new ids.
[ ] Remove data related to the Mage Tower.
[ ] Add BFA PVE achievements.


feat: remove stubbed client for recherche entreprise

Amélioration technique.
Détails :

Change recherche entreprise snapshot to be saved before mapToDomain
Add interceptor for recherche entreprise

+13,548 −13,122
Oh wow.

Indeed ^^
That's because I use the snapshots and I needed to save it differently to be able to use them.