I do not see where you are running Ubuntu Desktop linux?

I tried your Linaro Debian version but if I try to install Gnome or the Ubuntu desktop, it gets a massive  number of errors. You refer to a build from scratch but then say it does not work.
So, where or how are you running Desktop Ubuntu on the ZCU104?
I installed Ubuntu and then Ubuntu desktop, but it fails on startx with an error that it cannot load driver "xlnx". Petalinux does not include that as far as I can tell, and it is not standard to any version of Ubuntu. So, I am unclear how you are running Ubuntu Desktop on the ZCU104.
I am using 2019.1 of Petalinux. I see that 2018.2 did not work because of missing files, but 2019.1 is supposed to be valid for this.

Paul,
I haven't touched that in a while but it is the petalinux flow I still use all the time.  You are running Petalinux 2019.1 and I am using 2019.2 (Vitus) but it should work the same.
I have never tried to install a desktop.  I always work by using ssh to log onto the ZCU104.  I'm willing to work with you if you are still interested.
Pete

From: Paul Kimelman<EMAIL_ADDRESS>Sent: Tuesday, March 3, 2020 2:17 PM
To: hdlguy/zcu104_ubuntu<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [hdlguy/zcu104_ubuntu] I do not see where you are running Ubuntu Desktop linux? (#4)
I tried your Linaro Debian version but if I try to install Gnome or the Ubuntu desktop, it gets a massive number of errors. You refer to a build from scratch but then say it does not work.
So, where or how are you running Desktop Ubuntu on the ZCU104?
I installed Ubuntu and then Ubuntu desktop, but it fails on startx with an error that it cannot load driver "xlnx". Petalinux does not include that as far as I can tell, and it is not standard to any version of Ubuntu. So, I am unclear how you are running Ubuntu Desktop on the ZCU104.
I am using 2019.1 of Petalinux. I see that 2018.2 did not work because of missing files, but 2019.1 is supposed to be valid for this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/hdlguy/zcu104_ubuntu/issues/4?email_source=notifications&email_token=AB6I7Z4KS7Z4MT4FNV3Q753RFVXVBA5CNFSM4LATLMH2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ISFBTVA, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB6I7Z5PWMXSU4PHUA2VFITRFVXVBANCNFSM4LATLMHQ.

Hi Pete. I installed Ubuntu and booted it up. I then installed the desktop (gnome and all of the apps) and all went fine. Then, I login via ssh -X which sets up the remote Xwindows server so I don't have to worry about HDMI. I then used startx, but it blows up with a segfault because it wants a driver called "xlnx". I cannot see how it could want an HDMI driver (what I assume this is) because this is remote. So, you would expect it to not need anything from the Xilinx board. I note that xterm works! That is, I can run xterm and it shows up on the server that I am ssh-ing in from. But, I do not know why the desktop cannot be started since I do not know what the "xlnx" driver is that it needs (or it seg faults).
Any help would be much appreciated.

Paul,
I also had a similar problem just using xterm.  I have never tried to run the graphical desktop like gnome.  I just use ssh. Sorry.

From: Paul Kimelman<EMAIL_ADDRESS>Sent: Tuesday, March 3, 2020 5:47 PM
To: hdlguy/zcu104_ubuntu<EMAIL_ADDRESS>Cc: Pedro<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [hdlguy/zcu104_ubuntu] I do not see where you are running Ubuntu Desktop linux? (#4)
Hi Pete. I installed Ubuntu and booted it up. I then installed the desktop (gnome and all of the apps) and all went fine. Then, I login via ssh -X which sets up the remote Xwindows server so I don't have to worry about HDMI. I then used startx, but it blows up with a segfault because it wants a driver called "xlnx". I cannot see how it could want an HDMI driver (what I assume this is) because this is remote. So, you would expect it to not need anything from the Xilinx board. I note that xterm works! That is, I can run xterm and it shows up on the server that I am ssh-ing in from. But, I do not know why the desktop cannot be started since I do not know what the "xlnx" driver is that it needs (or it seg faults).
Any help would be much appreciated.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/hdlguy/zcu104_ubuntu/issues/4?email_source=notifications&email_token=AB6I7ZYRDZK37ZKD3CYYVOLRFWQKVA5CNFSM4LATLMH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENVX4SI#issuecomment-594247241, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB6I7Z2QZEZZAC433OAL3OLRFWQKVANCNFSM4LATLMHQ.

On AskUbuntu someone said to use gnome-session vs. startx. This yielded a bunch of errors of the form: "X11 connection rejected because of wrong authentication". So, this is maybe hopeful. I have searched on the web and none of the causes listed are mine (ie. my xauth is all good, there is no rc file, etc). So, I am still stuck with no explanation, but maybe am closer to a solution.
As I said, xterm is working for me, as are xedit and xeyes and various other xwindows apps. So, something about Gnome is the issue.

That sounds hopeful.  If you figure it out I would like to learn what you had to do.
I have run graphical interfaces to linux on Zynq boards before.  I don't know if it was gnome but similar.  I used the HDMI output to a monitor and attached keyboard and mouse. That approach is very similar to what the Raspberry Pi folks do.
FYI, I have installed a sata ssd in the m.2 slot of the ZCU104 and now I boot my OS from there.  I love a non-volatile and fast filesystem.
Pete

From: Paul Kimelman<EMAIL_ADDRESS>Sent: Wednesday, March 4, 2020 12:10 AM
To: hdlguy/zcu104_ubuntu<EMAIL_ADDRESS>Cc: Pedro<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [hdlguy/zcu104_ubuntu] I do not see where you are running Ubuntu Desktop linux? (#4)
On AskUbuntu someone said to use gnome-session vs. startx. This yielded a bunch of errors of the form: "X11 connection rejected because of wrong authentication". So, this is maybe hopeful. I have searched on the web and none of the causes listed are mine (ie. my xauth is all good, there is no rc file, etc). So, I am still stuck with no explanation, but maybe am closer to a solution.
As I said, xterm is working for me, as are xedit and xeyes and various other xwindows apps. So, something about Gnome is the issue.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/hdlguy/zcu104_ubuntu/issues/4?email_source=notifications&email_token=AB6I7Z7KXFEEHUQXKHDKMELRFX5GVA5CNFSM4LATLMH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENWT75Y#issuecomment-594362359, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB6I7Z6OVJCW5FSFQTNZP6TRFX5GVANCNFSM4LATLMHQ.

That sounds hopeful. If you figure it out I would like to learn what you had to do. I have run

I will post here as soon as I get an answer. I want to be clear that my intent was to run the Zync board "headless" with the display via ssh -X and Vino. To run on the HDMI with keyboard, one needs to get the Petalinux drivers over I guess - this may have been done on 2019.2. I am assuming someone is likely working on getting that into Debian linux-next for Linaro device-tree; it is a shame that Petalinux is so out of date in terms of kernel and other pieces, as it makes this all a lot harder than it needs to be.
Truthfully, i thought this headless thing would have been as easy as it used to be for such headless systems, but somehow Gnome/Ubuntu-desktop has internal expectations that are not as cleanly divided as they should be. Hopefully this will actually get some of this cleaned up for others doing the same.
One other note: Xilinx has messed up the hardware description/config file model. It should be that I can run Vivado 2019.1 and you can use Petalinux 2019.2 and vice-versa, but that is not the case. It will just not work unless every part of the flow is from the same generation. I would hope that the broader community gets this resolved.

Paul, I experienced the same with tool version compatibility. I'm not a big fan of Yocto and Petalinux takes that to another level of complexity. Unfortunately, there is almost no alternative.
Pete

From: Paul Kimelman<EMAIL_ADDRESS>Sent: Wednesday, March 4, 2020 10:24 AM
To: hdlguy/zcu104_ubuntu<EMAIL_ADDRESS>Cc: Pedro<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [hdlguy/zcu104_ubuntu] I do not see where you are running Ubuntu Desktop linux? (#4)
That sounds hopeful. If you figure it out I would like to learn what you had to do. I have run
I will post here as soon as I get an answer. I want to be clear that my intent was to run the Zync board "headless" with the display via ssh -X and Vino. To run on the HDMI with keyboard, one needs to get the Petalinux drivers over I guess - this may have been done on 2019.2. I am assuming someone is likely working on getting that into Debian linux-next for Linaro device-tree; it is a shame that Petalinux is so out of date in terms of kernel and other pieces, as it makes this all a lot harder than it needs to be.
Truthfully, i thought this headless thing would have been as easy as it used to be for such headless systems, but somehow Gnome/Ubuntu-desktop has internal expectations that are not as cleanly divided as they should be. Hopefully this will actually get some of this cleaned up for others doing the same.
One other note: Xilinx has messed up the hardware description/config file model. It should be that I can run Vivado 2019.1 and you can use Petalinux 2019.2 and vice-versa, but that is not the case. It will just not work unless every part of the flow is from the same generation. I would hope that the broader community gets this resolved.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/hdlguy/zcu104_ubuntu/issues/4?email_source=notifications&email_token=AB6I7Z43E2QULPLYFQNE42TRF2FGXA5CNFSM4LATLMH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENZBYCY#issuecomment-594680843, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB6I7Z22E3UU3HBIS4RBUJ3RF2FGXANCNFSM4LATLMHQ.

OK, I have found that I can use remote-desktop but not with Gnome-session due to some bug in xrdb or gnome in Ubuntu 18.04.
I can use xfce instead and it all works fine. I would hope that whatever bug is stopping gnome-session working is fixed eventually as I prefer it.
I was not able to use remote X because GNOME has an issue where some parts want to use one scheme and some another in that case. Xrdb uses Xorg and that just works fine. I though I could use Vino and VNC for this, but I do not know how to tell vino to launch the desktop - the instructions I was given segfault. There may be some other VNC which can work but all the instructions I see is from a few years ago, and all complaints about it not working applies to 18.04.
Obviously it would be nice if this were possible since Remote desktop is limiting as a client model.

Hi Pete. I installed Ubuntu and booted it up. I then installed the desktop (gnome and all of the apps) and all went fine. Then, I login via ssh -X which sets up the remote Xwindows server so I don't have to worry about HDMI. I then used startx, but it blows up with a segfault because it wants a driver called "xlnx". I cannot see how it could want an HDMI driver (what I assume this is) because this is remote. So, you would expect it to not need anything from the Xilinx board. I note that xterm works! That is, I can run xterm and it shows up on the server that I am ssh-ing in from. But, I do not know why the desktop cannot be started since I do not know what the "xlnx" driver is that it needs (or it seg faults).
Any help would be much appreciated.


Make game sessions persistent by storing the answers in localStorage

Each session would be good until midnight GMT when presumably a new puzzle will be released.

Put a TTL in localStorage and constantly check whether current time has reached the TTL?

So I'd imagine each 'session' would be represented as an object in localStorage, containing:

The time (or day) for which the session was started (in UTC)
The time remaining from the last session
The answers from the last session

Validity for the session would be checked checked at App mount time, if the session was made in the same day (additionally check for whether each answer in the cache is valid).
If a valid session was found, the welcome screen would be presented with either a 'Resume Session' or 'New Game' option.
The goal being that someone should be able to simply close the window and come back to it later.


EVA-3269: Adding publications, phenotype mappings, functional consequences

Closes #4, closes #6, closes #7
Example evidence string from tests:
{
  "datasourceId": "pharmgkb",
  "datasourceVersion": "2023-03-23",
  "datatypeId": "clinical_annotation",
  "studyId": "1449309937",
  "evidenceLevel": "1A",
  "literature": [
    "27857962",
    "11389482"
  ],
  "variantId": "19_38499645_GGAGGAG_GGAG",
  "variantRsId": "rs121918596",
  "targetFromSourceId": "ENSG00000196218",
  "variantFunctionalConsequenceId": "inframe_deletion",
  "variantOverlappingGeneId": "ENSG00000196218",
  "genotype": "del/GAG",
  "genotypeAnnotationText": "Patients with the rs121918596 del/GAG genotype may develop malignant hyperthermia when treated with volatile anesthetics (desflurane, enflurane, halothane, isoflurane, methoxyflurane, sevoflurane) and/or succinylcholine as compared to patients with the GAG/GAG genotype. Other genetic or clinical factors may also influence the risk for malignant hyperthermia.",
  "drugText": "enflurane",
  "drugId": "http://purl.obolibrary.org/obo/CHEBI_4792",
  "pgxCategory": "Toxicity",
  "phenotypeText": "Malignant Hyperthermia",
  "phenotypeFromSourceId": "http://www.orpha.net/ORDO/Orphanet_423"
}

Note targetFromSourceId is mapped from the gene provided by PharmGKB, whereas variantOverlappingGeneId is what we get from VEP based on the variant definition.

cc @ireneisdoomed @DSuveges @tskir for the example PharmGKB evidence string above... anything that should be changed just let me know (in particular field names - I've been reusing ones from ClinVar and making up some of my own with impunity).

@M-casado

How does having targetFromSourceId being the one mapped by PharmGKB and variantOverlappingGeneId from VEP affect the way we would interpret targetFromSourceId in evidence strings from ClinVar?

This is an excellent point and maybe something to put on the agenda for a subsequent meeting... I don't know if we'll be able change the ClinVar field names, but if not maybe we should name them differently here so they're consistent in the way you describe.  (We should also check how well these two gene IDs are aligned across PGKB, it's always possible that we don't actually need both of them...)

@apriltuesday

This is an excellent point and maybe something to put on the agenda for a subsequent meeting... I don't know if we'll be able change the ClinVar field names, but if not maybe we should name them differently here so they're consistent in the way you describe. (We should also check how well these two gene IDs are aligned across PGKB, it's always possible that we don't actually need both of them...)

Agreed, I think maintaining the source of the keywords and finding a different name for them would make the future maintainer not pull their hair off if there is a time this gets curated.

I've linked to this in the meeting minutes, will merge this PR and make a subsequent one for any naming changes we decide on.


API 3 stopping soon and V4 didnt seem to work

Hi many thanks for a great plugin, I could not get the V4 API to work 3 works fine but emailed climacell and they advise that V3 will be stopping soon.
Paul,      You may register to V3 here: https://developer.climacell.co/register
Having said that, V3 will be discontinued in the next few week and we   strongly recommend you sign up to our V4.
  Jacob

Best Regard,   ClimaCell Customer Success Team

Thank you, I know, but I am very busy at the moment. I hope to work on the update soon.

Thank you, I know, but I am very busy at the moment. I hope to work on the update soon.

Hi no problems many thanks great app, it was more just popping the update from Climacell many thanks great work have a good weekend.

Hi no problems many thanks great app, it was more just popping the update from Climacell many thanks great work have a good weekend.

I think can be helpful to have a full list of fields available in Climacell's V4 API.
@r-renato all those are already working with your integration (tested).
All the listed fields listed are part of the Free Climacell API plan.
    fields:
      - temperature
      - temperatureApparent
      - dewPoint
      - humidity
      - windSpeed
      - windDirection
      - windGust
      - pressureSurfaceLevel
      - pressureSeaLevel
      - precipitationIntensity
      - precipitationProbability
      - precipitationType
      - visibility
      - cloudCover
      - cloudBase
      - cloudCeiling
      - moonPhase
      - weatherCode
      - sunriseTime
      - sunsetTime
      - treeIndex
      - grassIndex
      - weedIndex
      - particulateMatter25
      - particulateMatter10
      - pollutantO3
      - pollutantNO2
      - pollutantCO
      - pollutantSO2
      - mepIndex
      - mepPrimaryPollutant
      - mepHealthConcern
      - epaIndex
      - epaPrimaryPollutant
      - epaHealthConcern
      - fireIndex
      - solarGHI

I think can be helpful to have a full list of fields available in Climacell's V4 API.
@r-renato all those are already working with your integration (tested).
All the listed fields listed are part of the Free Climacell API plan.
    fields:
      - temperature
      - temperatureApparent
      - dewPoint
      - humidity
      - windSpeed
      - windDirection
      - windGust
      - pressureSurfaceLevel
      - pressureSeaLevel
      - precipitationIntensity
      - precipitationProbability
      - precipitationType
      - visibility
      - cloudCover
      - cloudBase
      - cloudCeiling
      - moonPhase
      - weatherCode
      - sunriseTime
      - sunsetTime
      - treeIndex
      - grassIndex
      - weedIndex
      - particulateMatter25
      - particulateMatter10
      - pollutantO3
      - pollutantNO2
      - pollutantCO
      - pollutantSO2
      - mepIndex
      - mepPrimaryPollutant
      - mepHealthConcern
      - epaIndex
      - epaPrimaryPollutant
      - epaHealthConcern
      - fireIndex
      - solarGHI


Adding new Github Security Policy doc

@wetneb Please review and provide feedback for accuracy and correctness.

Given our current workflows we are basically not able to release fixes for any previous version. We would need to use branches for the major versions we want to support on the long term. That's definitely something we can consider. Each security patch would be merged in all supported major release branches to create the corresponding versions. That would probably mean using more granular release numbers (such as 3.2.0).


Provide String constants corresponding to MediaType constants

From @tobias-

Are there any plans for adding the more common types of media types as string constants for consumption with e.g. @Produces ? I know there's a pretty comprehensive list in com.linecorp.armeria.common.MediaType, but those are unusable by annotations.

Sounds like a good suggestion, as long as we can add some test cases that ensure the strings and MediaTypes are in sync.

This can be closed?

Yes. :smile:


Geospatial analysis by Kaggle free course added

This course is for data science aspirants. A very well-defined course by Kaggle for analysis of Geographical dataset analysis. Best domain to grow your skills

also add in readme

added


ezPlus with multiple images

I have problems using ezPlus with multiple images on my page.
I have nailed down the problem to line 320 in (current) version 1.2.4
"$('.' + self.options.container).remove();
If this line exist, all (previous added) classes are removed and only the last image-container stays.
Why is this line in the script?
If I run the script with this line, and I have 2+ pictures I would like to zoom (e.g. $('#pic1').ezPlus(); $('#pic2').ezPlus();), the container for the first pic (#pic1) is removed with the second image and therefore I am not able to zoom the image #pic1.
To run both images I have to add at least the "ZoomContainer" option with different values - which is not what I expected (e.g. $('#pic1').ezPlus({ZoomContainer:a}); $('#pic2').ezPlus({ZoomContainer:b}); will work). But this is complicated and I would like to run multiple images with the standard settings (if possible :-) )
Can I (safety) remove this line? What is the purpose of this line?

same problem on multiple images

after comment this
"$('.' + self.options.container).remove();
all images are work!
volkerdobler,  thank you!

Excelent info my friend

depois de comentar isso
"$ ('.' + self.options.container) .remove ();
todas as imagens são funcionais!
volkerdobler, obrigado!

Thanks very good friend that your information

thanks @volkerdobler  for the fix :)


Right/Left loss their meaning

Consider this css file:
.float-right {
    float: right;
}
.float-left {
    float: left;
}

rtlcss convert that to:
.float-right {
    float: left;
}
.float-left {
    float: right;
}

As you can see now .float-right class make an item left floating. That is not natural and intuitive. It's better to use direction agnostic vocabulary for classes' name. e.g. .far and .near. far is right in LTR mode and left in RTL direction.
Keeping old right/left classes make the new vocabulary backward compatible.
Please also take a look at this: https://stackoverflow.com/questions/12875800/effective-development-and-maintenance-of-large-scale-css-for-multi-language-webs

This conversion is made so you can author one markup and serve both LTR/RTL languages. Changing the class names - in this case - requires different markup for each language direction.
This is explained in details here.
If you look at it from content perspective it would make sense, so float-right means float to the right of the content. which in LTR is to your right and in RTL to the screen right :)

This issue has been discussed in a lot of threads


Members in PublicIPAddress of NetworkInterfaceIPConfiguration are 'null'

I'm trying to fetch PublicIPAddress of an Azure Network Interface. But, all the members are shown as 'null' except 'id' in the object of NetworkInterfaceIPConfiguration.
I'm interested in member PublicIPAddress.IpAddress.
Note that the Azure Portal shows Public IP address of the same network interface. Hence, public IP address is assigned to this interface.
Package Details:
Microsoft.Azure.Management.Network
Version: 7.1.1-Preview
Date Published on NuGet: 09/21/2016

@DeepakRajendranMsft Hey Deepak, would you mind taking a look at this issue?

outdated

@dsgouda  please close

@MikhailTryakhov please provide details about the issue or link if it has been discussed elsewhere

This is the general model for all resource references in Microsoft.Network. If you get a nic, only the ID of public IP address is given out. You can get the public IP address using Get-AzureRmPublicIPAddress to get the IP address. This is by design

Thanks @avijitgupta
please make sure this is fixed in the rest spec moving forward


Allow bot to automatically resume operation after "Error codes to stop the bot" being triggered

I want to be able to resume the bot operation (within X minutes) after I auto-stop it for recieving X amount of error codes.
This feature should be very useful if you set the bot to stop after recieving "Too many requests from this user", so you can resume operation after X minutes automatically.

Added in lastest release


service complete version commit

@adamdbrw
I have implemented the service.
When you have time, could you please check it for me?

Now that we have completed the implementation of the service, we have closed this draft PR and created a new PR.

I closed this by mistake, so I will reopen it.

@miyakoshi-dev Thanks for the nice PR! I'm currently looking into the code. I will finish my review tomorrow :+1:

Thank you for your review.
I will reflect your comments when I have time.
Currently, it only supports galactic, so humble has not been verified.

Addressed non-Humble related review comments.

@pijaro @adamdbrw
hello
It's been a while, but now it supports humble.
In addition to humble support, it supports the PR I received on my github.
I have confirmed that the service works on humble/galactic on ubuntu/windows.

I will close this PR since the work is being continued on #36 by @Deric-W :+1:
@miyakoshi-dev let's continue our work there :medal_sports:


Deprecate Server and HTTP Client mode?

At present, extruct supports a HTTP API for "testing", but that carries a maintenance burden, and it invites feature-requests that may nudge it more and more into becoming a monolithic proxy service. That's not really where we want Extruct to be, I think.
Similarly with the HTTP-Client mode and the CLI tool that offers it - it's a mode of operation which probably shouldn't be our priority with extruct. I feel that if we provide a CLI client for extruct, it should probably just accept HTML through a Unix pipe or from a file, and operate on that. That way, people can use curl or wget or whatever else they like, and they won't worry about extruct's support for various HTTP client features.
Thoughts? :)

@cathalgarvey I'm probably too biased to provide valuable feedback on this, as I've implemented the CLI in the past, but I think that it is an useful feature (at least I find myself using it use it frequently 😄).
But I totally agree with you on that extruct should just read stuff from a file/pipe, as it makes it easier to customize the requests as much as we need using a proper tool such as curl/wget.
IMHO, we should keep the CLI, but maybe change it so that extruct doesn't make the HTTP requests.


[Proposal] Integrate NFT721 as a Data Provider

Integrate NFT721 as a Data Provider will enable to create new Groups from holders of a specific ERC-721 with a min balance and a certain attribute.
Specifications
{
  "name": "NFT721",
  "iconUrl": "",
  "providerClassName": "NFT721Provider",
  "functions": [
    {
      "name": "Get holders of NFT",
      "functionName": "TODO",
      "countFunctionName": "TODO",
      "description": "Returns all holders of a specific ERC-721 with a min balance and a certain attribute",
      "args": [
        {
          "name": "chain",
          "argName": "TODO",
          "type": "string",
          "required": "true",
          "example": "Ethereum | BSC | Polygon | Avalanche | Gnosis | Fantom | Arbitrum One | Arbitrum Nova | Celo | Harmony | Optimism | Moonbeam | Metis | Cronos | Boba | Goerli | Base Goerli",
          "description": "A specific chain"
        },
        {
          "name": "address",
          "argName": "TODO",
          "type": "string",
          "required": "true",
          "example": "TODO",
          "description": "A specific contract address"
        },
        {
          "name": "amount",
          "argName": "TODO",
          "type": "TODO",
          "required": "false",
          "example": "2",
          "description": "A minimum amount"
        },
        {
          "name": "attribute",
          "argName": "TODO",
          "type": "TODO",
          "required": "false",
          "example": "TODO",
          "description": "An attribute identifier"
        }
      ]
    }
  ]
}

Tips on how to integrate the Data Provider
The chains supported are: Ethereum, BSC, Polygon, Avalanche, Gnosis, Fantom, Arbitrum One, Arbitrum Nova, Celo, Harmony, Optimism, Moonbeam, Metis, Cronos, Boba, Goerli and Base Goerli.
The easiest way to extract the list of holders’ addresses of a specific NFT is by using BigQuery & Etherscan.
Have a look at Sismo documentation to understand what is a Data Provider, and checkout the tutorial to know how to integrate a Data Provider.
Have fun!

Now that we have the Ankr Provider and the Token Provider (built using bigquery), this issue is outdated


Embedding model is not working

This is, Most probably, because the embedding model is not available anymore.
Maybe we need to save it somewhere in the repo?

Change the link in the  line https://github.com/ARBML/bayanat/blob/05ff4c25102627c3fbcd7fc16456e6630f2916e7/bayanat/bayanat.py#L154 to https://archive.org/download/full_grams_cbow_300_twitter/full_grams_cbow_300_twitter.zip


fix(ngResource): resource instances should be instantiated with the corr...

The problem
Consider a simple Resource 'class' with a custom method added to the prototype:
var User = $resource('/user/:userId', {userId:'@id'});
User.prototype.sayHello = function(){};

Then this will work:
var user = new User();
user.sayHi();

But when one or more instances are retrieved using .get() or .query(), the wrong constructor is used, causing this functionality to break:
// When using .get()
var user = User.get({userId:123}, function() {
    user.sayHi(); // This will not work
});

// When using .query()
users = User.query(function(){
    var user = users[0];
    user.sayHi(); // This will not work
});

This is a result of instantiating the new resource instances directly from the Resource constructor defined in the resourceFactory() function and not the constructor of the current Resource 'class'.
The fix
This PR makes sure the correct constructor is used and the prototype is chained correctly if resource instances are created by .get() or .query().
// When using .get()
var user = User.get({userId:123}, function() {
    user.sayHi(); // This will now work as expected
});

// When using .query()
users = User.query(function(){
    var user = users[0];
    user.sayHi(); // This will now work as expected
});

This PR replaces and extends #5168 in that it now also includes support for objects instantiated by query() and is using a newer version of ngResource.

The test passes in 1.5.6, so I assume this has been working for a while.


Prediction Transparency

Provide a screen to interactively combine and visualize each of the glucose effects.

Is this a different concept to #109


Update prevent-changes-to-security-settings-with-tamper-protection.md

Added Windows 10 and Windows 11 Enterprise multi-session as they are to be treated as separate OS editions.

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/7147

@denisebmsft Please merge this PR. Thanks


System colors

So @kevinSuttle dug up this old (2009!) post about using the system colors in CSS. And the cool part is that it adapts to the changes in the settings:

CodePen Demo

Might be neat to use them in this theme?

Works.


Makefile should gracefully handle the absence of PackageMaker.app

Since using pkgbuild is the default action, we should not assume that PackageMaker is installed when 'make bootstrap_files' is executed, but if present we should set up the sym link as we have been doing.


substitute command replaces non-selected lines

Thanks for this great plugin. The lack of the :s/ command was keeping me from using Sublime full-time.
Also, FYI: I'm using Sublime 2 build #2161 on Ubuntu 10.04 64-bit. My VintageEx version is 11.10.3 according to PKG-INFO
Sorry I don't have the expertise to patch this myself ..
Say you have this code:
foo.bar = foo.baz;
bar.foo = baz;

Let's say I need to rename the foo variable on the first line.
If I enter visual-line-mode (shift+v), then :s/foo/whatever/g, the output is:
whatever.bar = whatever.baz;
bar.whatever = baz;

The expected output is:
whatever.bar = whatever.baz;
bar.foo = baz;

The bug only "bleeds over" 1 extra line. Also, the bug does not happen in regular visual mode.
Please let me know if I can help further.
Thanks again,
-- Matt

I've fixed this. Let me know if it works for you now.

Thanks for addressing this so quickly. It works perfectly :)


Default Document tab displays application settings snapshot

Azure documentation issue guidance
Thanks for opening an issue in the Azure technical documentation repository.
We use GitHub issues as the primary channel for customer and community feedback about the Azure documentation.
Creating an issue
We prefer that you create documentation feedback issues using the Feedback link on the published article - the feedback control on the doc page creates an issue that contains all the article details so you can focus on the feedback part.
You can also create a feedback issue here in the repo. If you do this, please make sure your issue lists:

[ ] The relevant Azure service or technology.
[ ] A link to the published documentation article that you have feedback about.
[ ] Clear, specific feedback that the author can act on.

Pull requests and article contributions
If you know the change that is needed in an article, we encourage you to submit the changes directly using a pull request. If the change is large, or if you want to contribute an entire article, follow these guidelines:

[ ] Don't surprise us with a big pull request or a pull request with a new article! Submit an issue that describes the details of the proposed large change or new article.
[ ] Include the service or technology area.

We'll route the issue to the appropriate content team for review and discussion.
Tech support and product feedback
If you would like to contact Microsoft about other things, such as product feedback or tech support, please review these guidelines:

If you need technical support using Azure, the paid and free support options are described here: https://azure.microsoft.com/support/options/.

Each article in the Azure technical documentation contains a product feedback button - it's best to submit product feedback directly from a relevant article. Otherwise, you can submit product feedback for most Azure products in the following product feedback forum: https://feedback.azure.com/d365community/forum/79b1327d-d925-ec11-b6e6-000d3a4f06a4.

@TarangSinghalAzr
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!!

@TarangSinghalAzr
Since we have not heard back from you, we will now proceed to close this thread.If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


[codelab] Issue in the documentation

As we have installed the new components we have close the running instance and need to do npm install and npm run.

No action required.


Added a Trigger to MouseOver Property of ValidationMessage

Describe the changes you have made to improve this project
Added a MouseOver Trigger to the Validation Template to show the Error Message.

Add a new ValidationHelper static class for Validation.
Add an attached property ShowValidationErrorOnMouseOver which can control this feature for a single control.
Add also the same property to the CustomValidationPopup class to allow setting this behavior to all controls.
Add an attached property CloseOnMouseLeftButtonDown to the ValidationHelper to enable setting this behavior for a single control.

Additional context

Closed Issues
Closes #3539

@timunie your changes have been merged, thanks for your contribution 👍


[MM-26824] [MM-25990] Add channel filters to searchAllChannels

Summary

Builds off of https://github.com/mattermost/mattermost-redux/pull/1190
Adds more filter options for search all channels and merges them all into one options hash for the searchAllChannels.
Accompanying WebApp PR will modify all places where search all channels is called to support this change
Related server changes https://github.com/mattermost/mattermost-server/pull/15009/files

Ticket Link
https://mattermost.atlassian.net/browse/MM-25990

@sudheerDev Kind note for reviewers that feature complete for v5.26 is tomorrow,

Looks good. Please just make sure we create tickets to update the channel and teams page on system console to pass in the new opts cause they are using these functions currently.

I have done this in the corresponding webapp pr

@fmunshi
Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-#1192-upstream-release-5.26-1595515632
Branch 'automated-cherry-pick-of-#1192-upstream-release-5.26-1595515632' set up to track remote branch 'release-5.26' from 'upstream'.
+++ Downloading patch to /tmp/1192.patch (in case you need to do this again)

+++ About to attempt cherry pick of PR. To reattempt:
  $ git am -3 /tmp/1192.patch

Applying: Add filters for getProfilesInChannel and getProfilesInTeam
Using index info to reconstruct a base tree...
M	package-lock.json
M	src/selectors/entities/common.ts
M	src/selectors/entities/users.test.js
M	src/selectors/entities/users.ts
Falling back to patching base and 3-way merge...
Auto-merging src/selectors/entities/users.ts
CONFLICT (content): Merge conflict in src/selectors/entities/users.ts
Auto-merging src/selectors/entities/common.ts
CONFLICT (content): Merge conflict in src/selectors/entities/common.ts
Auto-merging package-lock.json
Patch failed at 0001 Add filters for getProfilesInChannel and getProfilesInTeam
Use 'git am --show-current-patch' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

+++ Conflicts detected:

UU src/selectors/entities/common.ts
UU src/selectors/entities/users.ts

+++ Aborting in-progress git am.

+++ Returning you to the master branch and cleaning up.


[Snyk] Upgrade csrf-csrf from 3.0.5 to 3.0.6

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to upgrade csrf-csrf from 3.0.5 to 3.0.6.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.

The recommended version was released on 22 days ago.

Release notes

Package name: csrf-csrf

3.0.6 - 2024-05-17

3.0.5 - 2024-05-15chore(release): 3.0.5

    from csrf-csrf GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

1 version


RMT transmission with open drain adds pulse after transmission ends (IDFGH-7275)

Environment

Development Kit:      ESP32-DevKitC
Kit version (for WroverKit/PicoKit/DevKitC): v4
IDF version (run git describe --tags to find it): v4.4.1 (but same code in master)
Build System:         idf.py
Compiler version (run xtensa-esp32-elf-gcc --version to find it):  xtensa-esp32-elf-gcc (crosstool-NG esp-2021r2-patch3) 8.4.0
Operating System:     Linux
Using an IDE?: No
Power Supply:    USB

Problem Description
When the RMT module is set to output and the same pin is set to open drain, then rmt_write_items adds an
end-marker 0. However, when the idle-level is high, then this leads to the output being pulled down.
It is crucial that the pin is set to open drain and pulled high externally.
Expected Behavior
The rmt_write_items should not add additional signals.
Either the rmt_write_items should write an end marker that corresponds to the idle level, or the lower level shouldn't emit anything when seeing an end marker.
Actual Behavior
The pin is pulled low for a short period of time.
See the attached screenshot.
Steps to reproduce

Pull pin 25 high with an external resistor (I used 15KOhm).
Execute the given code.

Code to reproduce this issue
#include "sdkconfig.h"
#include "freertos/FreeRTOS.h"
#include "freertos/task.h"
#include "driver/rmt.h"

void app_main(void)
{
    gpio_num_t gpio_num = 25;
    rmt_config_t config_tx = RMT_DEFAULT_CONFIG_TX(gpio_num, 0);
    config_tx.tx_config.idle_level = 1;
    config_tx.tx_config.idle_output_en = true;

    ESP_ERROR_CHECK(rmt_config(&config_tx));
    ESP_ERROR_CHECK(rmt_driver_install(config_tx.channel, 0, 0));

    // Enable open drain
    GPIO.pin[gpio_num].pad_driver = 1;

    while (1) {
        rmt_item32_t pulse[] = {
            {{{ 50, 0, 50, 1 }}},
        };
        ESP_ERROR_CHECK(rmt_write_items(0, pulse, sizeof(pulse) / sizeof(pulse[0]), true));
        vTaskDelay(1000 / portTICK_PERIOD_MS);
    }
}

Possible fix
The following patch potentially fixes the issue. If not, it might help to diagnose it.
It did remove the blip for me.
--- a/components/driver/rmt.c
+++ b/components/driver/rmt.c
@@ -1140,7 +1140,8 @@ esp_err_t rmt_write_items(rmt_channel_t channel, const rmt_item32_t *rmt_item, i
         p_rmt->tx_sub_len = item_sub_len;
     } else {
         rmt_fill_memory(channel, rmt_item, len_rem, 0);
-        rmt_item32_t stop_data = {0};
+        uint32_t idle_level = rmt_ll_tx_get_idle_level(rmt_contex.hal.regs, channel);
+        rmt_item32_t stop_data = {{{0, idle_level, 0, 0}}};
         rmt_ll_write_memory(rmt_contex.hal.mem, channel, &stop_data, 1, len_rem);
         p_rmt->tx_len_rem = 0;
     }

Thanks for your effort in debugging this issue, we will create a bugfix and backport to earlier release branches as well.


update for icpx 2024.1

Resolves issue blocking: https://github.com/spack/spack/pull/43431
icpx 2024.1 no longer accepts sycl::ext::intel::ctz
@eugeneswalker

This is included in #1396

This is included in #1396

OK. Closing


Conversation - Receipt image does not load and download file not working in old conversation

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.72-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Open the app and log in<EMAIL_ADDRESS>Go to any conversation and have a owed message (Paid + have with image) or create a new scan expense
Tap on detail
Try to tap on image

Expected Result:
Receipt image can be open and download
Actual Result:
Receipt image does not load and the download file not working in the  conversation
Also console error appier
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/4ebdf711-e368-4761-860e-1991063ac130
https://github.com/Expensify/App/assets/115492554/3727e608-c279-439e-8c81-6ed4d14151b3

View all open jobs on GitHub

I couldn't upload the image for this issue here because it's more than 10 MB

Not able to reproduce on v1.4.72-0 with my personal account. I agree that this isn't a deploy blocker and might be an issue specific to that account.

I supportal'd into their account can see and download the receipt on OldDot. Unfortunately I can't get anything to load when supportal'd into NewDot. This is probably a legit bug but needs repro.

I can't reproduce this either. I tried to supportal into NewDot but it crashed. What should I do next, @arosiclair ? thanks!

I think it's okay to close this out if it's not reproducible for you either.


BytecodeListing.java:34-38: Decompile Bytecode Listing....

The puzzle 845-380c5294 from #845 has to be resolved:
https://github.com/objectionary/jeo-maven-plugin/blob/3636715ee1aa2ff83f67a8c1d1ac18759e8ec747/src/main/java/org/eolang/jeo/representation/BytecodeListing.java#L34-L38
The puzzle was created by volodya-lombrozo on 13-Nov-24.
Estimate: 90 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@volodya-lombrozo can I take this?

@h1alexbel sure! Please, go ahead.

@volodya-lombrozo should we decompile bytecode listing obtained after ASM transformation? For instance, listing for MethodByte.class:
// class version 55.0 (55)
// access flags 0x21
public class org/eolang/jeo/MethodByte {

  // compiled from: MethodByte.java

  // access flags 0x1
  public <init>()V
   L0
    LINENUMBER 3 L0
    ALOAD 0
    INVOKESPECIAL java/lang/Object.<init> ()V
    RETURN
    MAXSTACK = 1
    MAXLOCALS = 1

  // access flags 0x9
  public static main([Ljava/lang/String;)V
   L0
    LINENUMBER 5 L0
    GETSTATIC java/lang/System.out : Ljava/io/PrintStream;
    INVOKESTATIC org/eolang/jeo/MethodByte.method ()B
    INVOKEVIRTUAL java/io/PrintStream.println (I)V
   L1
    LINENUMBER 6 L1
    GETSTATIC java/lang/System.out : Ljava/io/PrintStream;
    LDC "passed"
    INVOKEVIRTUAL java/io/PrintStream.println (Ljava/lang/String;)V
   L2
    LINENUMBER 7 L2
    RETURN
    MAXSTACK = 2
    MAXLOCALS = 1

  // access flags 0x8
  static method()B
   L0
    LINENUMBER 9 L0
    BIPUSH 52
    IRETURN
    MAXSTACK = 1
    MAXLOCALS = 0
}

or we can decompile the original bytecode instead? Original MethodByte.class:
package org.eolang.jeo;

public class MethodByte {
    public MethodByte() {
    }

    public static void main(String[] var0) {
        System.out.println(method());
        System.out.println("passed");
    }

    static byte method() {
        return 52;
    }
}

Also, I think we can maintain both listings: <listing/> (bytecode listing), and <decompiled/> (decompiled Java code). WDYT?

@h1alexbel I've already added bytecode listing, as you can see. The purpose of this issue is to try to decompile it to a Java, or at least to try it. We can do it by using https://github.com/fesh0r/fernflower or https://github.com/Storyyeller/Krakatau, or something else.
As for <decompiled/>, we just can't add it, we are limited by XMIR format that allows only <listing/>.


Move hand-written codes into separate folder from auto-generated codes

This PR closes https://github.com/kubernetes-client/ruby/issues/31
Moved hand-written codes into src folder to keep lib folder contains only auto-generated codes.

kubernetes

lib

kubernetes

config

error.rb
incluster_config.rb
kube_config.rb

utils.rb

kubernetes.rb

src

kubernetes

utils.rb
config

error.rb
incluster_config.rb
kube_config.rb

/lgtm
/approve
Thanks!


Similar names in functions causes crash (LAB5)

Stack<type> *init(int maxSize, type a)
Queue<type> *init(int maxSize, type a)
Program can't see the difference between these two when i call one of them

There're two ways:

Rename functions
Understand, what's wrong and fix it


call domain.com/index.php failed

I want o redirect all http to https requests when i edit .htaccess file :
RewriteRule ^(.*)$ https://mydomain.com/index.php [QSA,L]
it not working , because when i call mydomain.com/index.php its failed!.
i think index.php proccess as a route!

Hey @saeedIT ,
this is not a problem with the router. This is because you want to solve many problems in one single line. Try to isolate the https rediret from the index.php rewrite like in this file (Not tested):
DirectoryIndex index.php

# enable apache rewrite engine
RewriteEngine on

# set your rewrite base
# Edit this in your init method too if you script lives in a subfolder
RewriteBase /

# Redirect to https
RewriteCond %{HTTPS} off
RewriteRule (.*) https://%{HTTP_HOST}%{REQUEST_URI} [R=301,L]

# Deliver the folder or file directly if it exists on the server
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d

# Push every request to index.php
RewriteRule ^(.*)$ index.php [QSA]

This is because the webserver will now send a 301 Redirect back to the browser. The bowser will then connect to the new https verion of the webpage. In this second request the server will jump over the https rule because the request is already https and will then rewrite to the index.php
You cannot do a rewrite and a redirect in one line. Because the browser has to reconnect to the server after a redirect.

I think, this issue was solved.


ValidatingWebhookConfiguration asterisks/wildcards do not appear correctly

This is a Bug Report

Problem:
Under webhooks.rules.apiGroups on the API reference for ValidatingWebhookConfigurations, it reads:

APIGroups is the API groups the resources belong to. '' is all groups. If '' is present, the length of the slice must be one. Required.

The docstring from the code reads
APIGroups is the API groups the resources belong to. '*' is all groups.
If '*' is present, the length of the slice must be one.
Required.

(funny enough, GitHub also removes the asterisk if I use a quote instead of a code block)
It seems somehow the '*' is being converted into just ''.
Proposed Solution:
Not sure how this part is generated, I tried to build the api-reference but am getting errors.
Page to Update:
https://kubernetes.io/docs/reference/kubernetes-api/extend-resources/validating-webhook-configuration-v1/
I wouldn't be surprised if this affects more than just this page.

Thanks. The generation of these documentation uses a lot of heuristics, and there's no clear agreement within the project about whether or not we can treat the upstream code as Markdown or not.
/triage accepted
However, this is really an issue with the code in https://github.com/kubernetes-sigs/reference-docs/

@benmoss I would like to work on that issue
Can you please tell me exactly what I have to change?

/assign

If I knew what to change I would just change it 😄 . I don't know where the code that controls this is, @sftim suggested its in https://github.com/kubernetes-sigs/reference-docs/ but I'm not sure which part. I think it might be a slightly complicated problem since it sounds like the bug is arising from treating the Go comments as Markdown, and there's ambiguity as to whether that's okay or not.

@benmoss  I am looking a problem here is as it is starting the sentence '' is not showing
but if we use '*' instead of '' it is showing correctly

'\*' instead of '*'

I don't think we want to change the comments in the Go code, but I don't really know

@benmoss I am not saying to change in the Go code. I can use '\*' in README file

The upstream source code has to be changed. https://github.com/kubernetes/kubernetes/blob/master/staging/src/k8s.io/api/admissionregistration/v1/types.go#L26-L27
There are many similar cases in the same file.
This is not an ideal solution. We could have a separate repo hosting the API specification because the API is the core of the container orchestration platform. We do not want the API specification to be generated from source code. However, that won't happen any time soon. We have to live in an imperfect world and carry on.

We might be able to improve things incrementally with more heuristics in the code generator. I don't object to those incremental improvements, but it's a lot of effort per fixed *.

We might be able to improve things incrementally with more heuristics in the code generator. I don't object to those incremental improvements, but it's a lot of effort per fixed *.

Do you mind elaborate the "heuristics" we can employ? A '*' can be used to for italics, it can be used to mark bullet items, and it can be used as a literal string.

We could give special treatment to quoted single asterisks:
foo bar (wibble): '*' != "wibble"
something something
_emphasized text_ more text *more emphasis*

---
# now this is YAML but the code generator wouldn't know that
# and that's OK
foo:
  bar: "example value"
  wibble: "*"

Some of those asterisks mean emphasis; others don't.

@sftim Wow, what you are suggesting is already a syntax parser. We will have to tokenize the comment string(s) and identify cases where a '*', "*", '*/scale' or '*/*' sub-string shows up. At least, such strings must have an enclosing quotation marks (either single quotes or double quotes).
In my view, it is much more straightforward to fix cases like this in the source code (if we insist that docs come from source comments). We show case this to the developers and let the team know: "Hey, we are treating these comments seriously, as documents to be published. Please help write them in markdown style. What we have today is something semi-markdown, not recognized as correct syntax."

If I had a magic wand to make it happen, I'd pick doc comments that are explicitly identified as Markdown. The Rust model for these is great, and could be an inspiration on how to do something similar for this project (no need to change Golang itself).
If we could assume reliably that we're parsing Markdown, we get to draw on lots of existing, well-tested code for parsing, linting, etc.
However, we don't have that magic wand.
Both (all?) options are a compromise: accepting the toil of individual fixups, vs the yak shaving stack of steps that are increasingly far away from “fix this one asterisk”.
So, whilst I'm reluctant, I'm still OK with individual fixes to the upstream code comments.

/reopen
/priority backlog
/remove-lifecycle rotten
It's not urgent that we fix this. Nice if we can though.

/language en


what is the use of dependency injection on repositories when we need to beforehand declare and inject those dependencies

what is the use of dependency injection on repositories when we need to beforehand declare and inject those dependencies  :
while have UserModel as a possible injection when we are going to hard define that in the container injecting section and link it by Name so if I want to inject an other object I need to change it manually. ????

I'm not sure what you mean, can you give an example of the "change it manually" part?

okey,
let's take the CreateUser Operation, it has a injected dependency named UserRepository. UserRepository is already declared and fixed. So if I want to use CreateUser with an other Repository namely FireBaseUserRepository I need to create a new Operation that has FireBaseUserRepository as dependency and declare it in the container.js file.
So technically I can't use the same operation with different repositories. which is exactly why dependency injection is used.
so event thought we have  :
class CreateUser{
    constructor({UserRepository]){}
}

we can't do this :
CreateUser({
UserRepository : UserRepository
});
CreateUser({
UserRepository : FireBaseUSerRepository
});

because the dependencies are defined beforehand in the injection container.
A question here is, can we override the container's default injection ?

Oh, I see what you mean now. This question would be better answered in the awilix repository, since it's the library used to achieve dependency injection here.
Anyway, I'd suggest one of these approaches:

Use the container to resolve the dependency you need and then inject it manually:

const firebaseUserRepository = container.resolve('firebaseUserRepository');
const createUser = new CreateUser({ userRepository: firebaseUserRepository });

Assign the changing dependency later:

class UsersController {
  constructor({ createUser, firebaseUserRepository }) {
    createUser.userRepository = firebaseUserRepository;
  }
}

Pass the dependency as a parameter of the execute method:

createUser.execute({ userRepository: firebaseUserRepository, userAttrs });


[now-next] Add support for src dir in now-dev

Fixes: #3133
Fixes: https://github.com/zeit/next.js/issues/9007

Codecov Report

Merging #3140 into canary will decrease coverage by 0.09%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           canary   #3140     +/-   ##
========================================
- Coverage   13.29%   13.2%   -0.1%
========================================
  Files         268     268
  Lines       10332   10312     -20
  Branches     1218    1222      +4
========================================
- Hits         1374    1362     -12
+ Misses       8895    8887      -8
  Partials       63      63

Impacted Files
Coverage Δ

src/util/prefer-v2-deployment.ts
56.25% <0%> (-5.66%)
:arrow_down:

src/util/projects/get-project-by-id-or-name.ts
25% <0%> (-3.58%)
:arrow_down:

src/util/dev/builder.ts
76.1% <0%> (-0.83%)
:arrow_down:

src/commands/dev/index.ts
0% <0%> (ø)
:arrow_up:

src/util/metrics.ts
70.58% <0%> (+2.4%)
:arrow_up:

Continue to review full report at Codecov.

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


Add support for dict Input to DataFeeder, StreamingDataFeeder and allow for different TensorSignature for each of ModeKeys

Problem
The Estimator.fit() function takes as argument either

(x, y, and batch_size) where x and y could be numpy arrays or iterators.

PROS

Easy to use.
Allows feeding data from arbitrary source as long as problem can be decomposed into x and y.

CONS

No provision to provide epoch
In case, x and y are arrays, the data aggregate must be available as opposed to reading on the fly ( say from database)
Whether array or iterator, x and y can't be dictionaries. Most complex problems cannot be reduced to input matrix and output matrix and may require multiple input features matrices.

input_fn - this is callback function which must return features and target tensors or dictionary of tensors.

PROS

Allows feeding data from arbitrary source (in theory).
returned features and targets can be dictionary thus allowing to solve complex problems which takes multiple inputs.

CONS

Only found support for reading files using read_batch_examples(), read_batch_features(), read_batch_record_features(), etc.
No support for passing placeholder and feed_fn to allow for arbitrary source of input data which don't require queue.

Relevant discussions

https://github.com/tensorflow/tensorflow/pull/4696#issuecomment-253632403
http://stackoverflow.com/questions/39855375/how-to-use-streamingdatafeeder-as-contrib-learn-estimator-fits-input-fn

Additional problem
Currently the self.feature_info (feature signature) must be same for training as well as evaluation. However, there can be cases where evaluation if done differently than training. For example, DualEncoderLSTM model where training just requires Context and Utterance, while Evaluation (Validation) requires context and multiple utterances.
Solution
This PR basically combines the pros of using (x, y, and batch_size) with pros of input_fn by allowing x, y to be dictionary of multiple arrays. This also works for x and y iterators returning dictionary of streaming data.
Also, added support for different feature/target signature for training and evaluation.

Can one of the admins verify this patch?

@martinwicke - This is updated version of #5000 where I accidentally deleted my branch. I have also resolved conflicts with the main in this PR.

Copied from #5000 where @ilblackdragon commented -
"Here is an update on current state:
we just split up SK compatibility from Estimator into separate class SKCompat. E.g. if one wants to use fit(x, y) they need to wrap their estimator with SKCompat(Estimator(...)). Otherwise preferred way will be .
there is a pending change (hopefully goes in today) to add pandas_input_fn that would allow you to est.fit(pandas_input_fn(x_df, y_series), ...) which would pass all the columns of the data frame as dict of features. It will also use local queue to feed data, which increases performance a bit compared to current data feeder solution.
Going forward we are thinking to remove data feeders, and rely on input functions similar to pandas_input_fn for other types of data. And SKCompat interface can just seamlessly wrap the passed data into this functions."

@ilblackdragon - fine if you remove the DataFeeders completely. What will be the equivalent for StreamingDatafeeder then? Last I checked, input_fn only support input from TFRecords files.
For people like me, who have data coming from database, StreamingDataFeeder with dictionary support( this #5546) is a blessing.
In any case, even you plan to introduce SKCompat and want to let go of DataFeeders in future, I think for now this PR could still be relevant?
@martinwicke - it's been over a month. :(

@abhitopia +1

input_fn is not limited to any particular data source. You can look at the pandas_input_fn that @ilblackdragon mentioned, it reads from pandas Dataframes. A similar function reading from a dict would be trivial.
We do want to remove data feeders, so relying on them may be short-lived. If you fix the conflicts, I'm happy to accept this change though.
Sorry for the wait.

The idea to remove all DataFeeders - and replace them with input functions that are using feeding_functions.
See example for pandas_input_fn - it already supports dicts passing.
What we need now to implement is enqueue_data to support also an iterators and write a something like iter_input_fn.

@martinwicke - I completely agree that we should let go of DataFeeders and have support for only for input_fn (which already supports dict). Meanwhile, the transition is still in progress, I guess it still makes sense to update DataFeeders with this PR. ( I have resolved the conflicts)
@martinwicke , @ilblackdragon  - Thanks for the pointers. I will definitely checkout pandas_input_fn and also see if I can contribute in writing iter_input_fn soon.

Thanks for updating the PR. I left review comments.
Please do evaluate pandas_input_fn, it is significantly easier, and should be easy to adapt into a new function dict_input_fn or similar which does exactly what you want. We'd love to have that function if you end up writing it.
We will eventually remove DataFeeder for the betterment of the word, so relying on this functionality will be brittle.

@martin -  I have updated with changes you asked. And left comment on where I have question from you. Please review. Thanks.

@martinwicke , @ilblackdragon - Once this is accepted, I will try and work on proposed dict/iter_input_fn and also update my private code to use same.

Jenkins, test this please.

Jenkins, test this please.

Jenkins, test this please.

@martinwicke, @ilblackdragon  - I have fixed all the bugs due to which some tests were failing. I also want to emphasise that this PR also adds support for different input signatures for ModeKeys.TRAIN and ModeKeys.EVAL and nicely extends it to ModeKeys.INFER.
As such, even if DataFeeders are going away ( and rightly so),  this PR still adds necessary support which makes already awesome contrib.learn even better.
Can you please review this soon? Many thanks.

Jenkins, test this please.

Bunch of things to fix.
I don't think we should allow letting the input signatures vary by mode. It seems to me this makes it more fragile -- is there a use case you have in mind for which this is useful?
@ilblackdragon do you have an opinion on this?

@martinwicke, @ilblackdragon  - I can give you at least one real life example where this is required.
Consider the case of Response Selection in conversation.  ( See http://www.wildml.com/2016/07/deep-learning-for-chatbots-2-retrieval-based-model-tensorflow/)
During training:
X: context
y: response
During evaluation:
X: context
y: k responses which are ranked against X to give k@recall metric.
Also, I don't understand how having different signature makes it fragile. If for instance, the tensor signature in ModeKeys.EVAL is not compatible with computation graph, the error will be generated any how. Similarly, for ModeKeys.INFER the code first tries to match it up against signatures for ModeKeys.TRAIN/EVAL. If it doesn't match, warning is generated. But if the user doesn't pay heed to the warning and continues, that error should automatically get generated down the line if signature of ModeKeys.INFER is incompatible with computation graph.

@martinwicke - I am still waiting for your response to my previous comment, meanwhile, I just ran the tests on docker container using
tensorflow/tools/ci_build/ci_build.sh CPU bazel test //tensorflow/...
and seems that 8 tests TIMEOUT, and no tests failed. I am not sure if that is the reason for some checks not passing. Any ideas on how to fix TIMEOUT tests?

The sanity checks failed, can you fix these errors? It may be that the other tests are stale.

@martinwicke, @ilblackdragon - struggling with running ci_build on mac #5759 . Any advise?

@martinwicke, @ilblackdragon, @caisq  - I need help here. There were no errors but only two TIMEOUT errors when i run
tensorflow/tools/ci_build/ci_build.sh CPU bazel test //tensorflow/...

But I am absolutely unable to figure out why sanity checks are failing only for datafeeder.py.  The complain seems to be about Bad indentation. But when I compare for instance other file like graph_io.py, indentation looks same to me. Further more, if I run pylint with ci_build/pep8 configuration file, then the same indentation errors appear for pretty much all other files in Tensorflow (which I did not modify).

I really need some input from you.

@martinwicke, @abhitopia whats the status of this?

@abhitopia Please do not worry about testing it yourself. We will run test suites on our Jenkins to test this PR before merging it. Flaky timeouts like the ones you see do occur and in this case they seem to be unrelated to your changes.

@tensorflow-jenkins test this please

@abhitopia It looks like that the sanity check passed on Jenkins: https://ci.tensorflow.org/job/tensorflow-pull-requests-sanity/2181/console

@caisq , @martinwicke - The only tests that failed were due to Python3 compatibility. I have fixed them now. Please check again.

@tensorflow-jenkins test this please

@caisq - Can you test this again please?

@tensorflow-jenkins test this please.

@caisq - Thanks for testing. It seems the error testing is a bit flaky as now one Mac OS test is failing.
//tensorflow/python/kernel_tests:cwise_ops_test                          FAILED in 1 out of 50 in 28.8s
This test passed before, and now without any change, failed. This test is not related to this PR,
can we get it merged please?
CC: @martinwicke

@martinwicke  - any comments please?

MacOS tests were broken at head. I just pushed a fix for them.
Jenkins, test his please.

Jenkins, test this please.

@martinwicke , @gunan, @ilblackdragon, @caisq - Tests passing now. 😄  Merge?

All the team is out on thanksgiving, and I am not very familiar with the code.
I would feel more comfortable once we have approval from Martin.

@gunan - thanks for letting me know. I agree, let's wait for @martinwicke's approval.

@martinwicke - Can you check this please?

Thank you, and sorry this took so long.

@martinwicke - Thank you.

@martinwicke, @davidsoergel  - When I submitted, all the tests pass, and now you have reverted the changes.  :(

Yes, I am sorry.
All the tests passed but it turns out the tests were not comprehensive
enough. We should try this again. Can you make another PR with exactly the
same changes, and assign me?


ivh-treeview-children-attribute does not override default config

While wanting to change the children-attribute for a specific tree, i noticed my getChildren().length was always zero.
<div class="streamgroups-tree"
     ivh-treeview="$ctrl.streamgroups"
     ivh-treeview-children-attribute="'childStreamGroups'"
     ivh-treeview-on-cb-change="$ctrl.treeUpdated()">

With previous config, i received following (example) output
node.childStreamgroups.length = 3
getChildren().length = 0

When i updated the option in the ivhTreeviewOptionsProvider it worked perfectly and getChildren().length was also 3.
ivhTreeviewOptionsProvider.set({
   childrenAttribute: 'childStreamGroups'
});

Things i've tried:

Using another key name
Setting the value via a controller variable instead of plain string
Adding options via ivh-treeview-options with controller variable & object directly in attribute
Printing out multiple variables

Hmm... that's interesting. Nothing jumps out at me but I also see we don't currently have any tests for overriding this option. I'll take a look but if you were interested in submitting a PR with a failing test that would help fast tract things :).

Thank you for the fast reply.
I will try to set up a test case a.s.a.p.

@iandroogmans any luck reproducing this on your end? I had a chance to take quick look but the test I threw together seems to be passing:
https://github.com/iVantage/angular-ivh-treeview/commit/0531a2ee97fa64a48ee57d94b6d928de9e834226

@jtrussell Haven't had the chance to do this. I'll add it to my agenda for this weekend.
Kind regards

Hey @jtrussell, I've managed to reproduce this issue in the starter template provided. See the following link:
http://jsbin.com/bolesaboxa/1/edit?html,js,output

Feel free to re-open if this is still an issue :).


Inability to navigate back to homepage from other pages [Technical Issue]

In all of your pages you have a line that is the 1st snippet. Change that line of code to the 2nd snippet in order to allow users to be able navigate back to homepage. You had #home as the href to go back to the homepage, without #home being defined.
https://github.com/MWeygandt/ProjectWebsite/blob/e88ac229bdf3622c2d220615d633177b1729fe89/cheeses.html#L103

Home

Fixed thank you!


⚠ Samsung has degraded performance

In 1599502, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2242 ms

Resolved: Samsung performance has improved in a4e5fd5.


Drag and Drop feature #67

Description

Drag and drop functionality +
Kanban Design +
Column added to Data Services, Router, Server routes, & Controller +
TaskModal +
Overall site theming

Related Issue

[ X ] Is this related to a specific issue? If so please specify: Closes #67 Drag and Drop feature#67

Type of Changes
Use a check for ✓ the following type of changes:

Type

:bug: Bug fix

✓
:sparkles: New feature

:hammer: Refactoring

:100: Add tests

✓
:link: Update dependencies

:scroll: Docs

Updates
Before

After

Testing Steps / QA Criteria
n/a
Checklist:

[ X ] This PR is up to date with the development branch, and merge conflicts have been resolved
[ X ] My code follows the style guidelines of this project
[ ] I have executed npm run lint and resolved any outstanding errors.
[ ] I have performed a self-review of my own code
[ X ] I have commented my code, particularly in hard-to-understand areas
[ X ] My changes generate no new warnings

@mjbramich changes are ready for review.

Great Work! @AitchGrasso


Does the code (training or testing) compatible with newer version of matconvnet, such as v1.0-beta25?

Does the code (training or testing) compatible with newer version of matconvnet, such as v1.0-beta25?

We haven't tried it yet.
Please feel free to report bugs if they arise.


Document Error

Hello! Thanks for taking the time to read my issue, I found that there is one area of the document that needs to be fixed: as shown in the figure, all hyperlinks similar to this one are invalid on this page. Thanks!

We're aware of problems on this page, those responsible for it have been reminded ...
That said, which links do you mean ? The expandable sections on the page seem to work

Thank you for your work, the best thing about flyway is the high efficiency and excellent organization members.

Closing this issue as you seem to be confirming that this is fixed. If i am mistaken, please reopen.


[Bug]: Call to ContentAsync deadlocks and won't return.

Playwright version
1.14.1
Operating system
Windows
What browsers are you seeing the problem on?
Chromium
Other information
Win 10: 10.0.19043 Build 19043
VS 2019: Community 2019 Version 16.10.2
Code: .Net 5.0 C# (<IP_ADDRESS>312)
Chrome is up to date -- Version 94.0.4606.54 (Official Build) (64-bit)
What happened? / Describe the bug
Seems like the code is running into a deadlock.   Having just spun up Playwright, registered a callback for the PageLoad event,  navigating to a page, calling ContentAsync from the PageLoad callback function it appears the code is deadlocks and won't return.
Expected no deadlock and the respective page source returned.
Would think you have all what is needed, if any more info is needed, will provide it.
Code snippet to reproduce your bug
//
//  Init code
//
...
this.browser = await browserInfo.browserType.LaunchAsync(browserOptions);

this.Page = await this.context.NewPageAsync();
this.Page.Load += this.Page_Load;
await this.Page.GotoAsync( WebSiteUrl );
return ;  // construction/init finished...
}

//
//  PageLoad callback handler
//
private void Page_Load( object sender, IPage e )
{
    var pageSource = Utils.GetPageContent(e);
}

//
//   Utility method for page source...
//
public static string GetPageContent( IPage page)
{
    return Task.Run<string>(async() =>
    {
        try
        {
            return await page.ContentAsync(); // never returns from here...
        }
        catch(Exception ex)
        {
            Console.WriteLine( $"{ex.Message}" );
            return null;
        }
   }).Result;
}

Relevant log output
No log to speak of, just appears a deadlock will occur and never returns once.

This is expected - you're mixing async code with a .Result, which is never a good idea. In this case, when the event is triggered, on the main thread, it also blocks that same thread and never yields, hence causing a deadlock. I suggest you redo your logic so that it's all proper async.
Is something like this not an option?
            await Page.GotoAsync(WebSiteUrl);
            await Page.WaitForLoadStateAsync();
            // do you asserts here

Thanks @avodovnik, interesting this is the first time running into an issue doing this.   Honestly, the only reason i've been doing this was trying to be lazy and avoid needing to put await all over my code and wanted various utility methods, so the subtle allure sent me down the wrong path.   Just read a number of articles saying what i'm doing is bad...  Just wish the pages on the web where i found doing this made comments "DON"T DO THIS".
Again Thanks for the help :)


@amazee/react-scripts – npm

I found a bug in the package @amazee/react-scripts this repo is using. Unfortunately the github-project (https://github.com/AmazeeLabs/amazee-js) is not longer available. Any suggestions how to file the bug now? Would be great if @Schnitzel or @fubhy could bring it back. 🙊

It was never public AFAIK. It's a custom fork we made from CRA. I am going to move this project towards next.js in the next days anyways so it will not be using it for much longer anyways. Stay tuned!

@signalwerk I've started to work on it here: https://github.com/drupal-graphql/drupal-decoupled-app/pull/399
Just a few final tweaks left. Works already.

😻 wow. massive pr. I'm right now on a productive project with the current setup. I think we try to switch with our codebase as well, since we are not nearly finished now. and with next in place we can configure the webpack like we need it. that sounds good! thx a lot for your work!

@signalwerk I've merged the pull request. There are a few things missing I'd like to add later: https://github.com/drupal-graphql/drupal-decoupled-app/issues/400

@fubhy thanks a lot! looks like there is still a way to go! unfortunately we (the company attribute from zurich & me as a freelancer) are quite new in the game with decoupled-drupal and we are not sure if we are really a help at that stage. But we try whenever possible to bring stuff upstream.
do you think at the current state it makes sense to switch to the next.js setup? or do you think we should role with the old one?


Is the .article supposed to upload to the cloud with the meta data?

I'm just curious if the .article file is supposed to be passed along to AEM content or if the process is supposed to be: 1. download the article file and then 2. upload to an AEM article? The meta data, titles, etc. all get pushed to AEM correctly, so I'm assuming the functionality just doesn't exist yet or maybe I'm missing something. Thank you in advance!

@kaynenh The process should be as follows:

Click on a single Article
Then click the Cloud Icon
Then choose Create in the Cloud
Click the cloud icon again and then choose "Update ALL" that will push to AEM Mobile the metadata, images and the .article file.

We provide a download of the .article file as a way for someone to download the article and attach it manually, as well as diagnosing purposes.
Thank you,
Studio Mercury

Thanks for the info. I figured something was going wrong. When I attempt to follow your steps, I get the following response. And yeah, I've tried to refresh and update and I've tried removing the article from AEM and going through the steps again. I'm reaching out to you all in a separate email/issue so we could discuss further there.
`Failed. Could not perform action (push_entity). The version trying to update is not the latest version. Please refresh the page and try updating again. [ Show Raw Response ]
Headers
HTTP/1.1 409 Conflict
Access-Control-Allow-Origin: *
Access-Control-Expose-Headers: Link
Content-Type: application/json;charset=UTF-8
Date: Thu, 21 Apr 2016 17:57:24 GMT
Server: APIP
x-dps-request-id: 41b56473-b66d-4c8e-afb8-e40d33e05ac1
Content-Length: 148
Connection: keep-alive
Body
{"code":"VersionConflictException","message":"VersionConflictException errorCode=409 error=version conflict exception currentVersion=1461261306029"}
URL
https://pecs.publish.adobe.io/publication/d408e5cc-af88-4bcb-a613-cbad96871c27/article/promotional-sunglasses-make-a-statement;version=1461261200281`

A version conflict can happen because of 2 reasons:

The version is out of date. You just need to refresh the page and push the change again
If step 1 doesn't work, the article you're trying to update has the same article name (entity name) as another article already in the AEM cloud. You will want to make sure there isn't a duplicate entity name. If there is, you can unlink the article, change the entity name and re-push the article / changes.


digest auth not complete

thanx for all your previous replies
i tried with the new modifications but no luck the digest auth should be 2 steps but in the tool it halts after the first step
[root@localhost src]# '/root/Desktop/tr064/build/src/tr064tool' LANConfigSecurity:SetConfigPassword -u "dslf-reset" -p "admin" -v  -L http://<IP_ADDRESS>:37215/tr064dev.xml
[LOG:trace] tr064.cc:281 : Found action in service!
[LOG:trace] Session.cc:47 : Going to execute Service: LANConfigSecurity with action: SetConfigPassword
[LOG:trace] Auth.cc:121 : Current state: IDLE
An error occured: Received response does not contain an envelope_node, unable to proceed, received:
HTTP/1.1 401 Unauthorized
Connection: Keep-Alive
WWW-Authenticate: Digest realm="HuaweiHomeGateway",nonce="955cd72720e2bfcc406b1a842eb1ef06", qop="auth", algorithm="MD5"
Content-Length: 0
the app should now calculate the response and send a new request with the response on it  then it shall be auth and the action done
can u please mail me on<EMAIL_ADDRESS>i need to give u some stuff that may help but i cannot post in public
thnxxxxxxxx

right .. I just changed it to use curl so far, gonna add a new parameter for the http digsest auth. Is the username and pw the same as for the SOAP authentication?

see manpage for contact info.

"right .. I just changed it to use curl so far, gonna add a new parameter for the http digsest auth. Is the username and pw the same as for the SOAP authentication"
there are 2 username and pwd sets
1st username is  :dslf-config  and its pwd is changeable the admin changed it in the current (new challenge)
and it is responsible for all the actions and services of the tr064 but one service&one action  (the LANConfigSecurity:SetConfigPassword) that is only accessible using the
2nd set of username :dslf-reset and its pwd in static cannot be changed most likely will be blank or admin
so i need to use dslf-reset to change the dslf-config pwd and from the dslf-config i would setuserpassword and solve the challenge
all i need is that the tool get the nonce from the 1st 401 reply and calculate a response that would be accepted by the tr064 server in the router
will mail u some stuff that may help
tnx alooooooooooooooooooot

sent u a mail
tnx

will be w8ing the new branch
thank you


Add static framework targets

Thanks @younata
For the peanut gallery, this also includes a few fixes to target membership we discovered along the way.
We're doing this in order to help a project migrate away from CocoaPods, and on the road to PCK supporting Carthage


Bug 2037274: pkg/security/ldapclient: add configuration for connection verification

This adds the possibility to add connection verification in the ldap client configuration.
See https://github.com/openshift/enhancements/pull/980 for context.

/lgtm

/lgtm

/lgtm

/lgtm

/approve

/hold

The change should also be in master.

@sttts as suggested by @stlaz these changes (amongst others) are not supposed to be in master. These helper functions help to identify legacy certificates in 4.9/go1.16. In master these functions have no value.

/hold cancel
The OWNER file changes were copied to master.


Timothy Kessler didn't see a registration button on mobile.

Probably will never be resolved. Discord bug?


There is no option to specify admin password on the pi-hole sensor

Problem/Motivation
pi-hole addon is working fine, but I can not setup the sensors on a different home-assistant machine.
Sensors are displayed as not available
Expected behavior
I want to see my pi-hole sensor information on another machine (not localhost) running home assistant.
Steps to reproduce
Setup an admin password and try to add sensors on a different home assistant machine.
I have defined the admin port like this:
sensor:
  - platform: pi_hole
    host: hassio.local:4865
    monitored_conditions:
      - ads_blocked_today
      - ads_percentage_today
      - dns_queries_today
      - domains_being_blocked
      - queries_cached
      - queries_forwarded
      - unique_clients
      - unique_domains

But there is no option to setup admin_password, so I guess that is the reason why it is failing.

I don't see how that is a problem on the add-on side.
However, please provide logs.

Well, I just took for granted that the addon and the sensor will be handled on the same repository, sorry about that.
In any case, maybe there is something I can configure on the addon so I can white-list some ips or to just expose some information without pass?
What kind of logs do you want? I mean, from which side? How can I get them? Just through the hassio interface?

Ok, here are some logs from the "client" side.
By the way, the iframe integration is working with the same config (hassio.local as hostname)
2019-07-28 23:38:26 ERROR (MainThread) [hole] Can not load data from *hole: hassio.local
2019-07-28 23:38:26 ERROR (MainThread) [homeassistant.components.pi_hole.sensor] Unable to fetch data from Pi-hole
2019-07-28 23:38:31 WARNING (MainThread) [homeassistant.components.weather] Setup of platform met is taking over 10 seconds.
2019-07-28 23:38:31 ERROR (MainThread) [metno] https://aa015h6buqvih86i1.api.met.no/weatherapi/locationforecast/1.9/ returned
2019-07-28 23:38:31 ERROR (MainThread) [homeassistant.components.met.weather] Retrying in 20 minutes
2019-07-28 23:43:39 ERROR (MainThread) [hole] Can not load data from *hole: hassio.local
2019-07-28 23:43:39 ERROR (MainThread) [homeassistant.components.pi_hole.sensor] Unable to fetch data from Pi-hole
2019-07-28 23:48:47 ERROR (MainThread) [hole] Can not load data from *hole: hassio.local
2019-07-28 23:48:47 ERROR (MainThread) [homeassistant.components.pi_hole.sensor] Unable to fetch data from Pi-hole
2019-07-28 23:53:54 ERROR (MainThread) [hole] Can not load data from *hole: hassio.local
2019-07-28 23:53:54 ERROR (MainThread) [homeassistant.components.pi_hole.sensor] Unable to fetch data from Pi-hole

Ok, here is one update:
I changed the host part of the config to use the direct IP and it is now working.
Let me revert it to the local name and see if it fails again.

Yep, I can confirm that using hassio.local has host (even if you include the port) it does not work. You need to use the IP.
Feel free to close the issue.

Just as a side comment, wouldn't both instances attempt to use the same mDNS name? which I would guess is the root cause.

Just as a side comment, wouldn't both instances attempt to use the same mDNS name? which I would guess is the root cause.

@sinclairpaul what instances do you mean?


v2.0.1

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for features)
[x] Build was run locally and any changes were pushed
[ ] Lint has passed locally and any fixes were made for failures

Pull request type
Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[x] Documentation content changes
[ ] Other (please describe):

What is the New behavior?
New feature added to assign named poolname for engines. Job will run only on designated engine with same poolname
Issue Number: N/A
Does this introduce a breaking change?

[ ] Yes
[x] No

Other information
Instructions.txt for more info on command change

I need to update version. I will reopen after updating version number


Collection.remove behavior in 1.2.X

In previous versions, when model was removed from collection, you first cleaned up the index (this._byId), and only then you trigger remove event. Which was okay.
In 1.2, you first remove the model, trigger remove, and only then clean up the index in _removeReference. Thus, in the remove event handler model is still present in the index (while absent in collection), resulting in successful get. Which leads to weird things, like infinite loop when trying to remove the same model again in 'remove' event handler.
Is there any specific reason you now triggering 'remove' while collection is in inconsistent state, or it's just a bug?
    _removeModels: function(models, options) {
      var removed = [];
      for (var i = 0; i < models.length; i++) {
        var model = this.get(models[i]);
        if (!model) continue;

        var index = this.indexOf(model);
        this.models.splice(index, 1);
        this.length--;

        if (!options.silent) {
          options.index = index;
          model.trigger('remove', model, this, options);
        }

        removed.push(model);
        this._removeReference(model, options);
      }
      return removed.length ? removed : false;
    },

Are you referring to #3835?

Are you referring to #3693 and #3803?

Yes, I guess it's the same.

Cool. Feel free to weigh in there.

Thanks for invitation. Quite recently I have decided that I don't need dependency from backbone in my framework any more. I would rather have my own implementation of collections. Have fun with tickets.


Unable to uninstall extensions

uninstall vim, and other extensions, have this issue.

VSCode Version: 1.19.1
OS Version: mac os 10.13.2

Steps to Reproduce:

disable the extension
reload
uninstall the extension

then the extension is uninstalling, and there is no response.

What error messages do you get? Have you looked in the debug console?

No error messages.

@vincen have you tried starting code with all extensions disabled?

Disabled all extensions
uninstall four extensions
and, as shown in picture below.

and I removed these extentions from ~/.vscode/extentions/ folder. and I checked logs, there is no messages about this problem.

Also getting this on windows with any extension.
OS: Windows 10 64-bit build 16299
VS Code: 1.19.1


[Request] add Persian language

Please add Persian language

Hi @kookma , I've now added the persian language. Please tell us how it goes :)

Thank you Simon!
Sure I will share my experiences here!

@kookma, I've added the possibility to add user-defined keywords in addition to Ok and Wiki so that you can define for example باشه ویکی as your additional keywords
you would only need to create the tiddler $:/config/speech-to-text/keywords , add the fields ok-keywords and wiki-keywords and populate those fields with a LIST of keywords you would like to have instead of Ok and instead of Wiki

@BurningTreeC  thank you it works like a charm.
I will add خب ویکی to see how it works!
Thank you for all your efforts!

That's great @kookma

I'll close this issue for now, if you encounter problems @kookma , please open a new issue

Thank you @BurningTreeC . I could add some Persian words for ok wiki and it works fine!

Glad you understood how the mechanism works @kookma !


File not imported correctly

Hi,
when importing this obj I get wird results.
First to notice is the indicies count not dividable by 3 for left sphere even though only triangles are present it seems to be capped at 1999.
The position array contains 1883 values, the obj file contains 546 vertices for this object so that should be 1638 position values shouldn't it?
Im am running arch linux.
Programming in Clion with the included compiler.

Hello. It looks OK to load with the recent version of tinyobjloader in my side(added dummy .mtl file to load .obj correctly).
What is the result with recent master of tinyobjloader?

[LoadOBJ] # of shapes in .obj : 8
  shape[0].name = leftSphere
  shape[0].indices: 3264
  shape[0].vertices: 1884
  shape[0].normals: 1884
  shape[1].name = rightSphere
  shape[1].indices: 3264
  shape[1].vertices: 1884
  shape[1].normals: 1884
  shape[2].name = floor
  shape[2].indices: 6
  shape[2].vertices: 12
  shape[2].normals: 12
  shape[3].name = ceiling
  shape[3].indices: 6
  shape[3].vertices: 12
  shape[3].normals: 12
  shape[4].name = backWall
  shape[4].indices: 6
  shape[4].vertices: 12
  shape[4].normals: 12
  shape[5].name = rightWall
  shape[5].indices: 6
  shape[5].vertices: 12
  shape[5].normals: 12
  shape[6].name = leftWall
  shape[6].indices: 6
  shape[6].vertices: 12
  shape[6].normals: 12
  shape[7].name = light
  shape[7].indices: 6
  shape[7].vertices: 12
  shape[7].normals: 12

Hi,
This is embarrassing.
I am so sorry, I guess I stared too long at it yesterday. It loads fine for me too, I just accessed the wrong pointer.
Again, sorry for bothering you.

Good now you can load .obj correctly!


Handle Json ExtensionData in sql translation visitor

Describe the bug
Given an object like this:
class MyEntity
{
    public string MyTypedProperty { get; set; }

    [JsonExtensionData]
    public Dictionary<string, System.Text.Json.JsonElement>? ExtensionData { get; set; }
}

Querying against an entity with extensiondata is not possible within a LINQ expression
{l => (Convert(l.ExtensionData.get_Item("MyTest"), Object) == Convert(100, Object))}

Should convert to:
WHERE root["MyTest"] == 100

Instead converts to
WHERE root["ExtensionData"]["MyTest"] == 100

The underlying entity in Cosmos will serialize like this;
{
    "MyTest": 100
}

So it is impossible to query for the "MyTest" field without dropping down to writing raw QueryDefinitions, in which case you do not have the benefit of strongly-typed expressions for the rest of your query

Workaround: have your type extend IReadOnlyDictionary<>, however then you would have to provide your own custom JSON serialization/deserialization for this type in order to still serialize the additional (non-extension data) properties

Note: I would be happy to fix this, but I already have 3 open pull requests on this repository that have not been looked at. Is it worth my time to try to contribute?

Hello @onionhammer my apologies that the ball was dropped on your other PRs. I will ping the people who were reviewing those PRs, and we will try to get them merged. We can take this particular issue forward once the other PRs have been resolved. Please note that the other PRs will likely not be merged before next week. Thanks.

@neildsh Any progress on this front? Again, I would definitely be willing to take a crack at this if it means getting it done sooner, but this PR approval process is pretty arduous.

Hey @onionhammer, I am taking a look at this item. I have looked and approved 2 of your other prs.

@adityasa ok, let me know if  you want any help/tests written

@onionhammer are you talking about Newtonsoft's JsonExtensionData attribute or System.Text.Json.Serialization [https://docs.microsoft.com/en-us/dotnet/api/system.text.json.serialization.jsonextensiondataattribute?view=net-7.0]?

@onionhammer if you are open to making this change, we will be happy to review & approve it. Let me know.

@onionhammer are you talking about Newtonsoft's JsonExtensionData attribute or System.Text.Json.Serialization.JsonExtensiondata?

Both should be handled

@onionhammer fyi test below baselines current behavior which shows that Newtonsoft property is already honored:
https://github.com/Azure/azure-cosmos-dotnet-v3/pull/3830

@adityasa so could the code just not ignore the STJ attribute? Would that be the change?

@onionhammer, I am not sure if it's that that straightforward. The sdk uses Newtonsoft.JSON for serialization/deserialization between client and the backend. Newtonsoft library doesn't understand/honor STJ attributes. As a result, this logic needs to be custom implemented on top of existing logic, with due back-compat considerations.

@adityasa OK I misunderstood what you meant previously. The functionality proposed in this issue doesnt work for either STJ or Newtonsoft attributes - right? I'm not proposing any changes to the way serialization of objects is done, only the behavior of generating cosmos SQL from LINQ.

@adityasa After looking into this, supporting this is trivial - just a slight modification to GetMemberName and VisitMemberAccess should be needed.
Supporting STJ is still necessary, but support could be split into a separate PR.

And then there were 4 (open PRs)


updated to fint-buildscripts v1.4.0

Some minor changes, also added a few TODOs.
Can you have a quick look @asgeirn to see if this can be merged?

Thanks!  Moving @Value expressions to ProviderProps sounds like a good idea.


IE11 template innerHTML issue

When using Polymer.dom with a template element in IE11, innerHTML returns empty string.
I think it's because of https://github.com/Polymer/polymer/blob/273ab0fbe641209fa3fc3ce8368bb78f83db9b10/src/lib/dom-innerHTML.html#L115-L116. document.createElement('template') instanceof HTMLTemplateElement == false in IE11. It's an HTMLUnknownElement.
See http://jsbin.com/yukokoseda/1/edit?html,output. Notice that in the first test, the whole element comes back empty. In the second, only children templates are empty.

Looks like a webcomponents.js bug, not a Polymer bug. http://jsbin.com/siduvaququ/1/edit?html,output


add all the OpenFloodAI social links to Connect Navigation

1.Create a section named as Connect
When Connect is clicked on navigation bar it should move to Connect Section
2.add social links to Connect section with Icons
// when the Icon is clicked it redirects to the link
eg. When the Twitter Icon is clicked it should redirect to OpenFloodAI Twitter handle!
Twitter: https://twitter.com/OpenFloodAI
Facebook: https://www.facebook.com/OpenFloodAI
Linkedin: https://www.linkedin.com/company/openfloodai
Github: https://github.com/OpenFloodAI
Website Design as follows:
         Home
          ↓
        About
          ↓
       Connect

Can I take this?

Hi @BhathiyaTK Sure! Can you have a look at #23 too?

I have assigned you!

@adventuroussrv Sure I will look at #23 too.

PR done: #25


N-Gage Font

Add N-Gage font from dafont

homepage seems defunct?

It's the author address on dafont page. I couldn't accesss too but added
for reference.
On 22 Ara 2015 Sal at 05:15, Aditya Dalal<EMAIL_ADDRESS>wrote:

homepage seems defunct?
—
Reply to this email directly or view it on GitHub
https://github.com/caskroom/homebrew-fonts/pull/423#issuecomment-166491818
.


Cigar generation

Hello,
This is a really useful package for my team which I'm glad to have come across - is it still being actively maintained/developed? If so, might there be scope to add functionality for generation of random CIGAR strings?
Thanks!

I've rolled my own very simple implementation of this, I will open a pull request for consideration/discussion.