Handle annotations where multiple columns are IsAbout the same attribute

Context
Currently we store the first instance that appears in the TSV, e.g., for nb:Diagnosis
One example where this is bad:
The QPN TSV contains two columns about diagnosis, one of which has values PD and PSP (progressive supranuclear palsy), and the other column classes individuals as PD or healthy control. Both were annotated in the data dictionary, but only the PD/PSP diagnosis column is in the graph right now due to order of appearance in the raw data.
So in short: a subject who has no data on the first column (i.e. missing value), but has data on the second column (i.e. is a healthy control) will still not show up as a healthy control. -> not good
Desired treatment of multi-column annotation
According to our current data model, a subject can have a list of multiple diagnosis or assessments, but only one participant ID, age, or sex (?).

Multiple columns IsAbout an assessment is already supported ✔
Multiple columns currently can't be IsAbout diagnosis ❌

We should update the phenotypic column handling logic to return a list of values for a set of columns annotated as being IsAbout the same attribute (currently, this function is called for sex, diagnosis, and age, but not assessment):
https://github.com/neurobagel/bagel-cli/blob/4da00b6db4cce30d40f101c0c4e17be25db3828f/bagel/pheno_utils.py#L213-L238
Storing only the first ('transformed') value for variables that do not support multiple values (age, sex) should be done via conditionals outside of this utility function.
isSubjectGroup, for now, should remain mutually exclusive with any diagnoses. So, if the list of values for columns about diagnosis contain at least one instance of healthy control, we say the subject isSubjectGroup  and do not assign any diagnoses.

Blocked by #225 to avoid bagel pheno command logic conflicts


feat(vscode): Provide SettingsAdapter for VSCode

Context
This commit adds a new getVSCodeKaotoSettings method to the VSCodeKaotoEditorChannelApi class to build the Settings model.
It also deprecates the existing getCatalogURL method as the new way to go is using the settings adapter from now on.
Note
This PR needs to be merged after: https://github.com/KaotoIO/kaoto/pull/1264
relates: https://issues.redhat.com/browse/KTO-441
relates: https://github.com/KaotoIO/kaoto/pull/1221
relates: https://github.com/KaotoIO/kaoto/pull/1264

I am wondering should we mention the bump of @kaoto/kaoto to 2.2.0-RC3 in changelog and readme to have main branch up to date?

I'll take care :+1:

I am wondering should we mention the bump of @kaoto/kaoto to 2.2.0-RC3 in changelog and readme to have main branch up to date?

I'll take care 👍

Updated the CHANGELOG.md and the READMEmd


Issue pulling indicators from MISP

Hey, awesome package, i was keen to try it out so i built 2 new vm's one with bro 2.5.4 and latest misp with a dummy dataset. first issue is i couldnt install dovehawk via bro-pkg, the package doesnt exist on the cli or web interface to bro packages.
Once i downloaded it and installed manually i could get it to invoke fine and i was able to define my misp api key and IP address.  Then if i run bro on a pcap file or on a network interface i receive the below error
bro -C -i ens33 /usr/local/bro/share/bro/site/packages/dovehawk/                       listening on ens33
Downloading Signatures 2018/08/29 03:40:20 [1.00.002]
Local Directory: /usr/local/bro/share/bro/site/packages/dovehawk/./scripts
MISP Server: https://<IP_ADDRESS>/
Downloading Indicators...
Downloading Signatures...
1535539221.640420 error: curl --header "Authorization: APIKEY" -s -g -o "/tmp/bro-activehttp-C0OBIAJA1Me_body" -D "/tmp/bro-activehttp-C0OBIAJA1Me_headers" -X "GET" -m 60 "https://<IP_ADDRESS>/attributes/bro/download/all" && touch /tmp/bro-activehttp-C0OBIAJA1Me_body |/Input::READER_RAW: Child process exited with non-zero return code 60
1535539221.640420 error: curl --header "Authorization: APIKEY" -s -g -o "/tmp/bro-activehttp-hqdYPmgmf7i_body" -D "/tmp/bro-activehttp-hqdYPmgmf7i_headers" -X "GET" -m 60 "https://<IP_ADDRESS>/attributes/text/download/bro" && touch /tmp/bro-activehttp-hqdYPmgmf7i_body |/Input::READER_RAW: Child process exited with non-zero return code 60
1535539221.643126 error: /tmp/bro-activehttp-C0OBIAJA1Me_body/Input::READER_RAW: Init: cannot open /tmp/bro-activehttp-C0OBIAJA1Me_body
1535539221.643126 error: /tmp/bro-activehttp-C0OBIAJA1Me_body/Input::READER_RAW: Init failed
1535539221.643126 error: /tmp/bro-activehttp-C0OBIAJA1Me_body/Input::READER_RAW: terminating thread
1535539221.643126 error: /tmp/bro-activehttp-hqdYPmgmf7i_body/Input::READER_RAW: Init: cannot open /tmp/bro-activehttp-hqdYPmgmf7i_body
1535539221.643126 error: /tmp/bro-activehttp-hqdYPmgmf7i_body/Input::READER_RAW: Init failed
1535539221.643126 error: /tmp/bro-activehttp-hqdYPmgmf7i_body/Input::READER_RAW: terminating thread
If i curl my url with the correct authorization api key i can download all the bro signatures i need so the data is definitely coming down from misp. the command i tested api with was a little different
curl -k --header "Authorization: APIKEY" https://<IP_ADDRESS>/attributes/bro/download/all >> /tmp/bro-test.intel
That command downloads everything i need into bro-test.intel file. im skipping SSL check and -s -g -o options and not building a tmp-dir etc.
I can just wait until the bro-pkg version is put back up and test again or happy to debug this if you want to know any more info?
Thanks again!

Hello, thanks for trying out Dovehawk! I think the issue is the SSL certificate isn't passing validation.
There's a couple places in scripts/dovehawk.bro where you can add the -k:
--header \"Authorization: %s\"

change to:
-k --header \"Authorization: %s\"

Let me know if that works and I can probably add a config option for that situation. For the bro-pkg - I submitted a pull request to have it added, hopefully soon.

closing issue as using a non https url or a valid ssl cert would prevent this error.


Add a message after creating the repo to git init and git add

Help me not to forget to do this after creating the repo!

Also fixed by #129


can not find vtk.vtkglobaljavahash when turn on itkvtkglue and wrap_java

Description

Steps to Reproduce

Expected behavior

Actual behavior

Reproducibility

Versions

Environment

Additional Information

itk version 4.13.2

ITK's Java wrapping needs maintenance to get it into a working state.


Add option in source control view to copy filename

In the explorer view there is an option to copy the path or filename.

I would love to have this option in the source control view as well.

I often include the file name in the commit message and therefore need to type the filename manually for now.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Add Governor and Robot

I don't know why it's failing now, I think the specific issue with the Governor is resolved


balanced and redundant access to elasticsearch servers

I'm opening this issue to discuss/catalog/crystalize the issues around elasticsearch redundancy and load balancing.
My understanding is that our goal is to run three elasticsearch servers in a cluster, with all shards replicated across two servers.
This should make the loss of any single server survivable.
Having the indexer and news-search API services each point to any single elasticsearch server would be both fragile (requiring manual intervention/redeployment in the face of a server failure) and result in unfair distribution of workload.
In today's (2023-09-27) meeting I suggested using DNS "round robin" (DNSRR) for access to elasticsearch servers.
I did some investigation, and here's what I found:
DNSRR may be useless in some circumstances, due to implementation of RFC 3484 Rule 9 in getaddrinfo (the modern C library call to resolve a hostname to one or more addresses) to mean "select the address with the longest common prefix to the very last bit" making address preference ordering deterministic when the servers and the client are on the same subnet.
Some things I also learned (which only apply to servers running as Docker containers):

Docker swarm implements a "routing mesh" (for requests originating outside the server? via the "ingress" network??)

The docker-compose file has an "endpoint_mode" attribute for services:
https://docs.docker.com/compose/compose-file/compose-file-v3/#endpoint_mode
The default is "vip": docker assigns a virtual IP address for the service and tranparently routes requests to the Docker DNS name for the service.  The alternative is "dnsrr" where Docker DNS returns a list of addresses for each container.

Also:
The news-search-api  accepts multiple URLs for eshost configuration:
https://github.com/mediacloud/news-search-api/blob/main/api.py#L29
And passes them as a list to the Elasticsearch constructor:
https://elasticsearch-py.readthedocs.io/en/v8.10.0/api.html#module-elasticsearch

We should test to make sure things work if we bring down an Elastic Search server that the indexer and/or elastic-stats containers are connected to before closing this!

We are now connecting to all three Elasticsearch nodes on staging & prod, and the New Search API can also connect to the three.
Can we close this?

Can we close this?
yes


Extremely slow and "never-end" installation

Description
Hi. I find the library installation is very slow and never end, so far the installation on my side did not reach any completion. I am quite sure it is not due to my network.
Can someone help validate about this issue? Thanks.
To Reproduce

virtualenv env
.\Scripts\activate
pip install dwave-ocean-sdk

Expected behavior
Installation will stuck for longer time. As per my observation, the installation never complete even after more than 8 hours.
Environment:

OS: Windows 10
Python version: 3.7.2

Additional context
I am using "cmder" as an alternative to plain windows command line (cmd.exe) GUI.

Would it be possible to get the output of everything after the pip install command until it hangs?

Hi @arcondello. Any update on this issue?

Hi @thevenmuthu , this looks like something is going wrong with you pip, can you try pip install dwave-ocean-sdk --no-cache-dir? Also, for the sake of completeness, updating your pip?

Hi @arcondello. Thanks for your suggestion. It seems things worked well once I upgrade my Python from 3.7 to 3.9. I believe Pip has been upgraded in parallel with Python. Thanks again.


Add support for an "Undo" button

Allow the user to undo their last move (integration, transformation, etc).

Basically when an update is made to the mappings file or catalog ADM file, copy the existing file to .prev.  This will allow for a single level backup.


feat: make gradient end colors overridable

https://github.com/JimmyTai/react-native-color-picker-light/blob/8e1c46b6b401501fc9a37d568237eee981254d1b/ios/ColorPickerView.swift#L64-L73
It seems the colors in the RGB square picker are stuck at:

min 20% saturation and max 80% saturation.
min 90% light and max 100% light

For example, the "redest" red is #E6312E. Which is 1°, 80 sat, 90 light.
This limits an RGB LED object to tame colors, not allowing pure RGB colors like #FF0000, #00FF00 etc, or even pure white #FFFFFF
Could they be made overridable by an array in the React components props?

Maybe I could let you pass the color array to the picker.
By the way, If you want to help me improve this feature, you can send me a PR.


Add telocyte (CL:0017004)

Closes #1048 .
Telocytes are a rather common intersticial cell. It was also mentioned in #1155.
Note: a few spurious diffs appearing after running the normalisation script, but it seems manageable.
Description is a bit chunky, any ideas on shortening it are welcome.
I'm not sure who to assign as a reviewer, I'll try @nicolevasilevsky  but feel free to reassign (perhaps @gouttegd can help here?)

Thank you, David. I have added a NTR on GO's tracker: https://github.com/geneontology/go-ontology/issues/21973 .
I think connective tissue cell would be probably safer, I go with that too.

It seems GO has added telopode (geneontology/go-ontology#21974), so from the comments above it looks like what remains to be done here is to:

[ ] add the has_part relationship with the new GO term;
[ ] change the parent to connective tissue cell (CL:0002320);
[ ] fix the conflict against master;

and then this is ready for merge?
(No pressure, just your shepherd trying to tend his flock…)

@gouttegd Thank you for the shepherding :) I am not sure how to add new GO terms to CL (I guess it is similar to UBERON terms, but idk). Also, I imagine there is a GO release cycle or something, right? I wonder if the term is already "visible" from within CL

Hi @lubianat ,

I imagine there is a GO release cycle or something, right?

Yes, monthly. The latest release date is shown on the top right at http://geneontology.org/, and more detail is available at http://geneontology.org/docs/downloads/. I'm not sure how the general import of GO into CL works, whether it has to be triggered manually, and whether specific wanted terms need to be manually added to a list...? @matentzn @dosumis thanks :-)

Here are the instructions in our shiny new docs: https://obophenotype.github.io/cell-ontology/odk-workflows/UpdateImports.html

@gouttegd I've tried to make the suggested modifications (sorry for the long delay).
I am not quite sure on the next steps, can you help here with your outstanding shepherding? :)

The last modifications look good to me.
Do you know how to resolve Git conflicts? There are three that need to be fixed before we can merge.

The last modifications look good to me.
Do you know how to resolve Git conflicts? There are three that need to be fixed before we can merge.
If you don’t, ping me on Slack and I can guide you through it.

Wow, I am so sorry this took that long. I am not really sure how to solve ontology git conflicts, as these are often too large to fix manually.
I'll get back to this this week and see what needs to be fixed.

@lubianat I’ve refreshed the imports to fix the failed CI check. I think this is now ready to merge, unless anyone objects I’ll do that sometimes next week.


permalinks for headers not showing for .adoc

clickable permalinks next to headers appear in markdown, but not in asciidoc

Unfortunately, the geekdocs theme doesn't support permalinks for asciidoc, as per https://github.com/thegeeklab/hugo-geekdoc/issues/152. Will need to migrate to a different Hugo theme, or another Asciidoc supported SSG, s.a., https://github.com/DocOps/asciidocsy-jekyll-theme


Add JS polling to Cosmetics

Adds polling for Webpacker so you shouldn't need to restart web before seeing JS changes anymore

For reference, this is the mspsds counterpart - sorry for not including the cosmetics change straight away!
https://github.com/UKGovernmentBEIS/beis-mspsds/pull/659


Refactoring endpoint

Pull Request Test Coverage Report for Build 924490369

14 of 209 (6.7%)  changed or added relevant lines in 62 files are covered.
284 unchanged lines in 45 files lost coverage.
Overall coverage decreased (-36.1%) to 39.354%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/IMAPClient-Core.package/ICConnectionHandler.class/instance/checkConnected.st
0
1
0.0%

packages/IMAPClient-Core.package/ICConnectionHandler.class/instance/checkForParameterError.st
0
1
0.0%

packages/IMAPClient-Core.package/ICConnectionHandler.class/instance/checkLoggedIn.st
0
1
0.0%

packages/IMAPClient-Core.package/ICConnectionHandler.class/instance/fatalErrorOccured..st
0
1
0.0%

packages/IMAPClient-Core.package/ICConnectionHandler.class/instance/fetchNextResponse.st
0
1
0.0%

packages/IMAPClient-Core.package/ICConnectionHandler.class/instance/sendLoginCommand.st
0
1
0.0%

packages/IMAPClient-Core.package/ICConnectionHandler.class/instance/throwErrorIfAccountInvalid.st
0
1
0.0%

packages/IMAPClient-Core.package/ICEndPoint.class/instance/expungeEmails.st
0
1
0.0%

packages/IMAPClient-Core.package/ICEndPoint.class/instance/hierarchyDelimiter..st
0
1
0.0%

packages/IMAPClient-Core.package/ICEndPoint.class/instance/isConnected.st
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

packages/IMAPClient-Core.package/ICEndPoint.class/instance/readWriter..st
1
0%

packages/IMAPClient-Core.package/ICEndPoint.class/instance/hierarchyDelimiter..st
3
0%

packages/IMAPClient-Core.package/ICEndPoint.class/instance/parseListResponse.st
3
0%

packages/IMAPClient-Core.package/ICEndPoint.class/instance/readWriter.st
3
0%

packages/IMAPClient-Core.package/ICEndPoint.class/instance/rootFolder..st
3
0%

packages/IMAPClient-Core.package/ICCommandStorage.class/instance/add.for..st
4
0%

packages/IMAPClient-Core.package/ICCommandStorage.class/instance/getNextCommandFor..st
4
0%

packages/IMAPClient-Core.package/ICCommandStorage.class/instance/isEmptyFor..st
4
0%

packages/IMAPClient-Core.package/ICCommandStorage.class/instance/popNextCommandFor..st
4
0%

packages/IMAPClient-Core.package/ICEmail.class/instance/deleted.st
4
0%

Totals

Change from base Build 921577068:
-36.1%

Covered Lines:
573

Relevant Lines:
1456

💛  - Coveralls

Another idea: If you want to increase coverage a bit again, you might want to make the tests call the ICEndpoint methods (which in turn use the ICConnectionHandler methods) instead of calling the ICConnectionHandler methods directly :)


default-mysql-client cannot be installed anymore

Description
We are using the virtual environment ubuntu-20.04 to build our application and need the mysqldump to be available during our test execution.
For this reason we execute a run step that looked the following
run: |
          sudo apt-get update
          sudo apt-get install -y default-mysql-client

This worked just fine until the 13th of December this year using the following environment-virtual-version: 20211129.1
Since the 15th of December of this year using the following environment-virtual-version: 20211209.3, the run-command fails with the following message:
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 default-mysql-client : Depends: mysql-client-8.0 but it is not going to be installed
E: Unable to correct problems, you have held broken packages.

We have not changed the build-script that's why I assume something changed in the virtual environment.
We tried to install the mysql-client-8.0 package instead of the default-mysql-client package, which ended in the following error:
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 mysql-client-8.0 : Depends: mysql-common (>= 5.5)
E: Unable to correct problems, you have held broken packages.

What is the expected way to install the mysql-client on the ubuntu-20.04 virtual environment?
Virtual environments affected

[ ] Ubuntu 18.04
[X] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[ ] Windows Server 2016
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
it started to fail for us with the following image version: 20211209.3
it still fails using the following image version: 20211214.2
Is it regression?
No response
Expected behavior
The default-mysql-client package gets correctly installed by apt-get without any errors and the binary mysqldump is available.
Actual behavior
An error is shown and the build-script fails with the following error:
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 mysql-client-8.0 : Depends: mysql-common (>= 5.5)
E: Unable to correct problems, you have held broken packages.

Repro steps
Execute a workflow-build that runs on ubuntu-20.04
And add a step that looks like this
- name: Install mysql client for mysqldump
        run: |
          sudo apt-get update
          sudo apt-get install -y default-mysql-client

Hi
Try temporally remove this step from your workflow as mysql-clir4nt should be installed be default see https://github.com/actions/virtual-environments/blob/main/images/linux/scripts/installers/mysql.sh#L33-L44. Reason for this change is described in https://github.com/actions/virtual-environments/pull/4674 - this is a temporary change.

seems to work properly even without installing the package additionally. Thank you very much for the information


[RFR] Create a deploy gh-pages task

With some automatic commit inside. But it renders a nice GitHub page now.
Updating the README with this link will come later: it needs more fancy styling first.

:+1:

tests fail

Just added better cleaning of demo folder after a deployment and fixed tests.


optimiz: optimize function's parameters into one struct-rm_api.go#ResourceManagerInbound

What happened:
There are too many parameters in the method. I want to merge them into one class
rm_api.go#ResourceManagerInbound 接口的参数太多，希望能合并到一个结构体去

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:

你好，我想参与开发seata-go。从这个小的改动开始。

你好，我想参与开发seata-go。从这个小的改动开始。

ok， assigned to you

从Go开发层面请教一下，参数都放到context里是否可行呢

从Go开发层面请教一下，参数都放到context里是否可行呢

context已经存了全局事务ID，分支事务ID这些全局的信息，其他的就不放了


Lack of modules

Hello! I can't find a lot of modules in this version, for example no Bluetooth module is not available. Where can I get it?

@DerR1chter
Hello.
Because some of websploit dependencies were deprecated and i'm trying to write them by my self, they will add in future.

We are waiting for this. Thank you.

any news on this?


CELO <> tBTC on-chain exchange

Pairs to support

[ ]  CELO/tBTC - required
[ ]  CELO/cKEEP

TBC if https://koroswap.com will support

Aggregated into one issue: tBTC Ecosystem Support


li tags don't close with nuxt generate

Version
v1.4.2
Reproduction link
https://codesandbox.io/s/vue
Steps to reproduce
Add a ul > li in index.vue (with the basic start kit)
<template>

      <app-logo/>

        test

        Nuxt.js project

        Documentation
        GitHub

              Documentation

Github

</template>

When npm run generate is executed, the output in dist folder looks like,

  Documentation
  GitHub

Documentation

Github

What is expected ?
Tags should close correctly
What is actually happening?
Tags isn't closing properly
Additional comments?
Issue observed only with ul > li for now. Not sure how I can provide a reproduction link for a nuxt generate example - apologies in advance.

This bug report is available on Nuxt community (#c7538)

Can reproduce this with latest nuxt-edge

This is intended, as HTML is minified (see the docs). The HTML is also valid, as the
 tag is optional in your case source.


Open SAP Web IDE Full-Stack

Tutorial URL: https://developers.sap.com/tutorials/webide-multi-cloud.html
Step 1: This refers to the "SAP Web IDE", however this has been replaced by "SAP Business Application Studio". This affects the following step, also.

Thanks for reporting the issue. I updated the tutorial according to the latest changes, they should be live within the next 15-20 minutes.


WHITESPACE_ONLY and ES6 modules emits incorrect "Illegal redeclared variable" errors

When you have two ES6 modules which declare a const or let at the top scope of the module, and both use the same name e.g. ...
// module A
const xxx = "a";
...

// module B
import A from "./A"
const xxx = "b"

... they conflict in WHITESPACE_ONLY mode, and you get:
ERROR - Illegal redeclared variable: xxx

I can't find how to do WHITESPACE_ONLY and ES6 modules either the service or the debugger. But the local test case is very simple.
Are ES6 modules are simply not supported in WHITESPACE_ONLY mode? Previously (my last version of the compiler was around Feb this year) this used to 'compile' successfully in ws-only mode (but probably would never have worked at runtime) but today's compiler fails with this error.
Or is it the case that ws-only mode should now work with ES6 modules, and this is simply a bug with that? I think I recall seeing some commits go past relating to whitespace mode and ES6 modules.
I'd definitely be interested in using ws-only mode with ES6 modules to shorten dev build times, but can live without it.

Equally:
Variable Tab first declared in closure-externs/browser/chrome.js

@Dominator008, I believe this is another report of the bug you fixed earlier today.
Do you agree?

@brad4d I think so, @supersteves can you confirm it is now fixed?

@Dominator008 Yep, fixed - thanks!


skopeo: support copy to OSTree storage

Currently blocked on: https://github.com/containers/image/pull/228
Signed-off-by: Giuseppe Scrivano<EMAIL_ADDRESS>

ACK on the skopeo parts

@mtrmac I have updated containers/image and amended in the patch

why is github always saying branch is out-of-date :confused:

LGTM, for the record.

why is github always saying branch is out-of-date 😕

Well, because it is? Giuseppe, please rebase on top of current master and we’ll merge.

Of course, I thought it was like the last time in c/image eheh

@mtrmac @runcom I've rebased and pushed the new version


Fix admin favicon URLs

This fixes the admin favicon URLs.  The broken URLs appear to have been causing the express JS server to hang or take long periods to time-out.

awesome!


Fix: OpenFAST can't find tail fin aerodynamics file (when absolute)

This PR is ready to merge
Feature or improvement description

Fix for issue https://github.com/OpenFAST/openfast/issues/2359. When reading in file path, check for relative path was missing.
Related issue, if one exists

https://github.com/OpenFAST/openfast/issues/2359
Impacted areas of the software

AeroDyn15
Additional supporting information

Test results, if applicable

Caught when updating openfast_io python library. Will be covered by tests for the lib.

Another possible issue here: https://github.com/OpenFAST/openfast/blob/c2df793c300215b5eff3a1b91e9ef4721ec1cbc0/modules/subdyn/src/SubDyn.f90#L1110
Currently there are no r-test that reach this part of the code.

Another possible issue here:
https://github.com/OpenFAST/openfast/blob/c2df793c300215b5eff3a1b91e9ef4721ec1cbc0/modules/subdyn/src/SubDyn.f90#L1110
Currently there are no r-test that reach this part of the code.

I don't think the SSI capability is fully functional at present.  I'm inclined to leave this one as is until we revive SSI.


mAP score

I am using [SSD MobileNet V2 FPNLite 320x320] to train my model. I have chest x-ray to detect Covid-19. There 1349 Normal chest x-rays and 3883 Covid-19 chest x-rays. I have used different Augmentations to increase my Normal chest-xray from 1349 to 2215. and pneumonia images from 3883 to 4032.
I have trained my model 4000 training steps, and it has training loss of 0.21 and evaluation loss of 0.23.
I am confused that I have gotten 0.837 mAP. Is it a good result? I need to compare my results with some other papers, but they have accuracy nearest 96 or 98. There result is in accuracy and mine is in mAP. How am I going to compare them??

Another question is that most of the Tensorflow object detection API has mAP between 0.20 to 0.50, but mine is 0.83. So is their some issue with my model or its fine? Because it is detecting and classifying most of the images accurately.
Please please any expert guide me regarding all of the issues I have asked.

It's actually 83% not 0,83.
The values from tensorflow are for the mAP from scratch not transfer learning


successful purchase not fired in purchaseUpdatedListener

Thanks for great package and sorry for my English.
Version of react-native-iap
4.4.4
Version of react-native
0.60.3
Platforms you faced the error (IOS or Android or both?)
iOS
Expected behavior
Actual behavior
Tested environment (Emulator? Real Device?)
Real Device
Steps to reproduce the behavior
I try to buy subscription. Enter password and commit. But has not event in purchaseUpdatedListener.
If I return
[[SKPaymentQueue defaultQueue] addTransactionObserver:self];
in RNIapIos.m - that's OK!
I tried add
[IAPPromotionObserver startObserving];
in didFinishLaunchingWithOptions - but I has't any effect.

@AndyYudkin Thanks for the issue and I am suspecting it causes by the last releases trying to fix the #950. I've just pushed a new commit with release 4.4.5. Could you try this and come back?

@hyochan I was install 4.4.6 and that's OK!

So this is resolved~!


Fixes #3306. Tweaks R# settings and applies to View.cs and ViewLayout.cs

Fixes

Fixes #3306

Proposed Changes/Todos
Pull Request checklist:

[ ] I've named my PR in the form of "Fixes #issue. Terse description."
[ ] My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
[ ] My code follows the Terminal.Gui library design guidelines
[ ] I ran dotnet test before commit
[ ] I have made corresponding changes to the API documentation (using /// style comments)
[ ] My changes generate no new warnings
[ ] I have checked my code and corrected any poor grammar or misspellings
[ ] I conducted basic QA to assure all features are working

When I get a chance, I'm gonna remove the impacted files from this PR and just change the settings.


configure max-scenarios-per-file for whether each row of a Scenario Outline is counted

Adds another configuration option to the max-scenarios-per-file rule so that you can specify whether each row in a Scenario Outline table should be counted towards the number of scenarios (the current, and still default, behaviour) or whether the whole outline block should just count as 1.

Pull Request Test Coverage Report for Build 523

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 97.505%

Totals

Change from base Build 520:
0.005%

Covered Lines:
635

Relevant Lines:
644

💛  - Coveralls

Pull Request Test Coverage Report for Build 523

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 97.505%

Totals

Change from base Build 520:
0.005%

Covered Lines:
635

Relevant Lines:
644

💛  - Coveralls


Not going to publish to wordpress.org?

Is there any reason not publishing this plugin to wordpress.org? Is it support? Or low request?

It's just because of due to low request.


Update manage-queues-app.md

Updating to say that VDI is now supported

Already done in another PR in the private repo


Not able to give custom Config file for Prometheus Adapter

I am using this Prometheus Adapter to expose celery_tasks_total as a custom metric from Prometheus.
This adapter is installed using Helm 3 as follows :
helm install --generate-name prometheus-community/prometheus-adapter
I want to provide my custom Config-Map to this adapter. I have tried the way instructed in the Readme for the Prometheus Adapter repository. But I am unable to do so.

I tried using --config argument which is not a valid argument in Helm 3.
I also tried giving it as followshelm install --generate-name prometheus-community/prometheus-adapter --set-file config='/home/user/path_to_config.yaml'but when I check the Config-Map in the Kubernetes Dashboard, I still see this default Config-Map that gets shipped with the prometheus adapter helm chart.

Can anyone help me with the process of supplying a custom config map manifest to this Helm Chart?

Please open an issue in the helm repository itself. We are not maintaining the helm charts here.


Reapply does not work when adding slave to newly created bridge

When creating new bridge, ethernet section is not created in NM profile. When adding slave to this
bridge, the merge state brings the ethernet config which cause reapply fail.
NM bug is https://bugzilla.redhat.com/show_bug.cgi?id=1786547
but NetworkManager team seems reluctant to fix it.

To fix from nmstate part, don't create ethernet when user does not explicitly desired.


Code ends when no dates are avaliable

Hello right now there is not a single availability for Austin and instead of retrying until there is, the code just ends and I have to start it again manually.

Have you tried to use the city name option on config.yml instead of zipcode?


cleanup(bigtable)!: make *Client::BackgroundThreadsFactory() pure

Change the recently-added BackgroundThreadsFactory() functions (see
#5967, #6004, and #6012) in bigtable::AdminClient, bigtable::DataClient,
and bigtable::InstanceAdminClient to be pure virtual.
This restores these class to be pure interfaces, rather than having one
small piece of concrete behavior.  A default implementation that returns
a null factory, which throws when called, is never useful.
This may seem like a breaking change, but it was actually the introduction
of the member function that was breaking, as derived classes already had
to override the function in order to use the client (for example, to pass
a DataClient to the bigtable::Table constructor).
Failure to override will at least now be a compilation error, rather than
a runtime error (std::bad_function_call).
This was discovered while trying to use the latest Bigtable client in
another package, and seeing exceptions thrown.

This change is

I'm open to addressing this in a different way, but at least this PR raises the issue.

I'm open to addressing this in a different way, but at least this PR raises the issue.

I think your analysis is correct.  A couple of things:

This PR should be marked as refactor(bigtable)! (or maybe cleanup(bigtable)!) to signal it is a breaking change.  I understand the previous PRs should have been marked that way too, but we can make this one right.
Please update CHANGELOG.md explaining the breaking change.
As you probably noticed, you need to update the API baseline.

Thanks.  Done.  PTAL.


Cant add it to my server

i cant add it

At least bother to read the !help command.

At least bother to read the !help command.

i did
and stop ignoring me on discord

You clearly did not read the help command considering its right there

i saw it now

You could've also used https://bot.rc24.xyz/ and the link is on our site 😂


⚠ T:Riza Api has degraded performance

In 45e8b98, T:Riza Api (https://infra.triza.dev/api) experienced degraded performance:

HTTP code: 200
Response time: 3028 ms

Resolved: T:Riza Api performance has improved in 5dd3a4a.


asyncio.Task.current_task PendingDeprecation fix

Issue #, if available:
Resolves #216
Description of changes:
Change asyncio.Task.current_task invocation to asyncio.current_task to alleviate PendingDeprecationWarning
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Yeah so unfortunately simply changing the invocation won't work with 3.5 and 3.6. I think the 2 solutions would be to either do a try/except catch of an AttributeError and then fall back to the compatible version or use and if/else off of sys.version_info. The latter would be more clean imo.

Yes. I also think use of sys.version_info would be more cleaner.

Codecov Report

Merging #217 into master will decrease coverage by 0.12%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master     #217      +/-   ##
==========================================
- Coverage   84.55%   84.43%   -0.13%
==========================================
  Files          80       80
  Lines        3120     3128       +8
==========================================
+ Hits         2638     2641       +3
- Misses        482      487       +5

Impacted Files
Coverage Δ

aws_xray_sdk/core/async_context.py
85.96% <75.00%> (-3.84%)
:arrow_down:

aws_xray_sdk/core/sampling/local/reservoir.py
88.88% <0.00%> (-11.12%)
:arrow_down:

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 88406fd...00595d9. Read the comment docs.

Looks like it is all back to green in the test suite. Not sure why the check isn't working.


How to include scrollify on more than one page?

I need to target specific classes on 2 different pages.
At the moment the 2nd instance is causing the first one to not work.
I couldn't see in the documentation how this is supposed to be written?
Something like:
jQuery.scrollify({         section : ".page-template-home ._home-section",         interstitialSection : ".page-template-home footer#colophon",         // easing: "easeOutExpo",         section : ".page-template-event ._home-section",         interstitialSection : ".page-template-event footer#colophon",         // easing: "easeOutExpo",         scrollSpeed: 1500,       });

You'll just need a comma between the class names: section : ".page-template-home, ._home-section"

That's great, thank you!


Uncaught SyntaxError: indirect export not found: Lexer

Hello, i have some trouble to use this library with Vite.js v2.4.2. i have this error with Firefox
Uncaught SyntaxError: indirect export not found: Lexer markdown-parser.js:68:9 and with chrome Uncaught SyntaxError: The requested module '/node_modules/marked/src/Lexer.js?v=4ea01e18' does not provide an export named 'default'

Seems like the logged error is on markedjs which isn't a dependency of svelte-toast - it's probably caused by another package you installed.


Add check: Marathon in upgrade state

In the past and more recently we've come across issues where customers could not get their cluster back up and running due to a Migration already in progress state which will then cause the upgrade process to fail.
The Marathon log will display an error similar to:
7-26 13:40:39,434] INFO Leadership ended with failure; exiting (mesosphere.marathon.core.election.ElectionServiceImpl:marathon-akka.actor.de
ere.marathon.MigrationFailedException: Migration Failed: Migration is already in progress; /migration-in-progress

This has been implemented during the last PR merge. Issue can be closed.


Enhance guards and asyncability

Added new AuthenticationGuard (AuthenticationGuardWithoutLoginRedirect) that does not redirect the user if not authenticated.
Also added new function (getAuthenticatedAsPromise) to AuthenticationState to asynchronously get the value of authenticated property (in router setup for example)

Still missing documentation for getAuthenticatedAsPromise

Moved to #415


MacOS Download link is broken

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[x] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[x] Reinstalled the latest version of Anaconda
[x] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
The new 4.2.1 download doesn't work. 404 not found.
What steps reproduce the problem?

Click download link on GitHub or the official website.
404 not found.
Be sad.

What is the expected output? What do you see instead?
A download of a dmg file.
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version:
Python version:
Qt version:
PyQt version:
Operating System name/version:

Dependencies

PASTE DEPENDENCIES HERE

Solved now.

Thanks for the feedback @pixeluk42 just in case, as an explanation, this happens since the workflow that builds the installers (which usually takes 20-30 mins for Mac) runs  after the Github Release is created so a little wait is needed before being available to actually download the installer :)


Support for external URL

Please test new version https://github.com/baraja-core/image-generator/releases/tag/v1.0.4.


fix: set priority for messages to sleeping nodes again

#6944 broke queuing message to sleeping nodes due to too much code being removed. This PR adds a test for the situation and fixes the break.
fixes: #6952

@zwave-js-bot automerge

Thanks for the quick fix!


Sort By in Pivot Table

Sort by field in pivot table does not work.
Expected results
Enter a metric/custom sql for the pivot table to sort its entries.
Actual results
The sort by field is not implemented into the pivot table chart but it is visible in underlying query
Screenshots
Pivot table not sorted by max(rank)

Visible in query

so... there's a sortby metric added to the south table/data, it's just not impacting the pivot table itself....
it seems to be a simple fix BUT we are not fixing it. pivot v2 doesn't support sort by metric yet, and it should.
@kgabryje see if we can reach feature parity by mid august.

@matthewphua @junlincc Sort by in pivot table v1 seems to be working though
https://user-images.githubusercontent.com/15073128/126996248-4cd71144-38ab-4a13-8e26-52dcf026d1ee.mov
Anyway, implemented in v2 as well
https://github.com/apache-superset/superset-ui/pull/1240

Pivot v1 and Table has different behavior on sort by metric(when it's different than the metric) .. anyway, for V2 we should at least show the data in the south table


Show tutorial on first login of user is included in expriment

info comes with /me request in format. There is an obm field.
Details in the backend docs.
First user experiment number is 74
If user is included in experiment they should be show the start tutorial when they login to toggl desktop the first time. Lib should send notification to UI. Currently the experiment works in Windows.

“obm”: {“included”: true/false, “nr": integer, “actions”: stringarray/hasopmitempty/canbemissing}
will be included in backend response.
how will i pass it on to the ui?
thanks

Send it with login callback that makes the UI show listing view.

I think it would be good to add a run_experiment(int, other params maybe) call, which we can reuse for all experiments in the future.

can we use the already there promotion thing in the lib?
at the moment it has
#define kPromotionJoinBetaChannel 1
toggl_on_promotion event
with
const int64_t promotion_type
parameter.
I wonder if 1 is already used by the OBM backend.

experiment number is 74 this is something that comes from obm and is setup in backend

ok then we can use the promotion stuff, as number 1 is not taken.

do we need to pass the actions to UI as well? or what does it do.

Yes, in the future we'll need the actions as well with the first experiment we don't need them in UI.


"Run to a breakpoint" Is missing a step

Visual Studio can't run/debug the code unless a startup object is also selected.  The article does not include this step, but it should.

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

ID: b1abc350-6abd-b03b-7a76-2022928271ae
Version Independent ID: 52c4a7e4-f0e0-6c41-5bbd-5ad2efce752f
Content: Build and debug projects - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/dev-tools/build-debug-project.md
Service: dynamics-365-cross-app
Product: ****
Technology: ****
GitHub Login: @pvillads
Microsoft Alias: pvillads

Created bug 571648 to track this issue.

Agree. It should. At the same time as you set the startup project, you should set the startup item (form, menu item or class).


[KOGITO-4307] Add tags params

and some other useful params
https://issues.redhat.com/browse/KOGITO-4307
Many thanks for submitting your Pull Request :heart:!
Please make sure your PR meets the following requirements:

[x] You have read the contributors' guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains a link to the JIRA issue
[x] Pull Request contains a description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

tested ok

@sutaakar please review last change: https://github.com/kiegroup/kogito-cloud-operator/pull/744/commits/91a473e85f3386091c62c58c4f6871f8ae319168

Looks good.


Cannot add a string message to bbs_sign_context

I am currently writing a wrapper in c++ for this library (so it can eventually be used in React Native), but I am running into an issue that I do not know how to resolve.
Currently, I can call bbs_sign_context_init() without any errors like this:
double NativeBbsSignatures::bbs_sign_context_init(jsi::Runtime &rt) {
    ExternError *err = new ExternError();
    uint64_t res = ::bbs_sign_context_init(err);
    return res;
}

Then when calling bbs_sign_context_add_message_string():
double NativeBbsSignatures::bbs_sign_context_add_message_string(jsi::Runtime &rt, const jsi::Object &options) {
    uint64_t handle = TurboModuleUtils::jsiToValue<uint64_t>(rt, options.getProperty(rt, "handle"));
    std::string message = TurboModuleUtils::jsiToValue<std::string>(rt, options.getProperty(rt, "message"));

    ExternError *err = new ExternError();

    int32_t res = ::bbs_sign_context_add_message_string(handle, message.c_str(), err);

    // error code : -1000
    // error message: "handle has stale version number"
    return res;
}

The error comes from ffi-support when checking the validity of the handle. It says that the handle is stale and it indicates something equivalent to a use-after-free / double-free, etc.
Did this happen to anyone else writing a wrapper for this library? @tmarkovski @alexksg

Hey @blu3beri im unsure exactly of the issue you are facing here, I've not encountered this before, but I've also not tried the library with C++. If you are looking for RN support I'd recommend taking the objective-c and java wrappers published by these libraries and bringing them together in a TS/JS package using the react native native module bridge (https://reactnative.dev/docs/native-modules-ios)

I have not encountered this, seems odd that the library would report this. Does this happen if you make a call to context_init and add_message_string right after, not in a different API method? Could it be that RN unloads the library domain after each call?

Thanks for the responses. It seems to only error when we pass the handle over the bridge and then back when we want to do the next step (add_message_string). Keeping the handle inside the cpp environment and then continuing the flow seems to work. The library is not closed as the handle is still in the concurrentHandleMap but it seems that it refers to something else (Not the best explanation).
Keeping the handle inside cpp and only return back to RN when bbs_sign_context_finish() is called seems to be the trick. Thanks for the help :).
pseudo code:
void main() {
  ExternError *err = new ExternError();
  auto handle = ::bbs_sign_context_init(err);
  ::bbs_sign_context_add_message(...);
}

btw @tplooker for this library we are using React Native TurboModules instead of the NativeModules. They are for faster synchronous code but it is rather undocumented. Hopefully it will get a proper public release soon.


Color on timeIndicators timelabel

Hi
Is it possible to change to color of the timelabel on the timeindicator showing the current time?
Alternatively, is there any way to hide it?

I just figured out that timeline.timeIndicator.font = UIFont.systemFont(ofSize: 0)is a way to hide the label.

Alternatively, you could use isHidden property of the UIView to achieve the same result.


Add support for nats server request profile

This adds support for nats server request profile, as per the new PROFILEZ endpoint in nats-io/nats-server#3774. The profiles are written to files in the supplied directory, so that they can be used with go tool pprof.

Looks good, I am seeing build/test failures:
Error: ../cli/account_command.go:393:17: undefined: server.UserInfo
Error: ../cli/account_command.go:405:20: undefined: server.UserInfo

fix those, squash commits and you can merge when green


WC: load also the config.js

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
12418

Relevant Lines:
12420

💛  - Coveralls


Support MDX

Unless I've done something wrong, the extension doesn't seem to work on MDX files. Would you consider supporting this file type?

@axelboc
I don't know MDX file. Is it a foo.mdx?
This extention doesn't work with .mdx files, because it isn't a common markdown file extention.
FYI, you can add unique file extention to vscode by setting 'files.associations' preference.
(https://code.visualstudio.com/docs/languages/identifiers)
You possibly use this extention by adding '*.mdx' to 'markdown' in 'files.associations'.

MDX is a language that allows writing JSX code in Markdown files. If I treat *.mdx as markdown, I loose MDX support. 😢
MDX is becoming quite mainstream in the React ecosystem. It's supported by Next.js, Create React App, Gatsby and Storybook to name a few.


Alternate way of calling api from overview plugin

More abstraction and less use of IPC throughout. :+1: or :-1:?

#25 more favorable


why the size of ssl log is 0?

I use sslsplit to sniffer the data transfering using ssl, however, I've found out most of logs ending with 443.log is empty, why? any solutions?

Pretty tough to solve this without some more information.
How are you invoking sslsplit ?  What does your setup look like ?

Please post the debug log, what kind of connections cause empty logs, how you invoke sslsplit, and the output of sslsplit -V.

Over three months no response, closing issue, feel free to re-open or comment if this is still an issue for you.


Add XLMRoberta models support

now merged in llama.cpp: https://github.com/ggerganov/llama.cpp/pull/8658

that was fast, thanks!


Duplicate question having different answers

What currencies are supported by Azure Marketplace?

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

ID: 50f6ee16-5202-f3b3-bf68-d2aca21f2ea8
Version Independent ID: 08853ce1-8448-f09c-4236-5dfc3e2ff3bb
Content: Microsoft commercial marketplace FAQ
Content Source: articles/marketplace/marketplace-faq-publisher-guide.md
Service: marketplace
Sub-service: partnercenter-marketplace-publisher
GitHub Login: @qianw211
Microsoft Alias: dsindona

@Jont-K there's an FAQ question answering this, did you find your answer or do you have additional questions?  Thanks!

sorry I wasn't clear.  I'm pretty sure that question is listed in the FAQ
twice but will different answers.  which one is correct?
Jonathan Kingsepp
e<EMAIL_ADDRESS>m:<PHONE_NUMBER>
https://tackle.io
On Tue, Apr 21, 2020 at 6:13 PM Qian Wen<EMAIL_ADDRESS>wrote:

@Jont-K https://github.com/Jont-K there's an FAQ question answering
this, did you find your answer or do you have additional questions? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/52989#issuecomment-617459240,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANWNOUJSVRSDTKW33V3W64LRNYR75ANCNFSM4MNV3TGQ
.

Hey @Jont-K we support 17 currencies: Transactions can be conducted in the following 17 currencies: AUD, BRL, CAD, CHF, DKK, EUR, GBP, INR, JPY, KRW, NOK, NZD, RUB, SEK, TWD, USD, RMB.

Thanks, Brian, this is taking up more of people's time than it's probably
worth.
I was just pointing out that the same question is listed twice, and in one
of the answers, the currency RMB is missing from one of them.  We can
close this - thank you.
[image: image.png]
[image: image.png]
Jonathan Kingsepp
e<EMAIL_ADDRESS>m:<PHONE_NUMBER>
https://tackle.io
On Tue, Apr 21, 2020 at 6:28 PM Brian Levenson<EMAIL_ADDRESS>wrote:

Hey @Jont-K https://github.com/Jont-K we support 17 currencies:
Transactions can be conducted in the following 17 currencies: AUD, BRL,
CAD, CHF, DKK, EUR, GBP, INR, JPY, KRW, NOK, NZD, RUB, SEK, TWD, USD, RMB.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/52989#issuecomment-617463789,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANWNOUNK3XDWSQSUMWREES3RNYT3FANCNFSM4MNV3TGQ
.

no worries, and i'm nearly finished with a refresh of the FAQs, so i'll make sure they're only listed once (and include the right currencies)
#please-close


Correct Example 3 switch

The switch Title does not work. The switch -Identity is correct as referenced in Example 1 and 2
Before creating a pull request, make sure that you have read the contribution file located at
https://github.com/pnp/powerShell/blob/dev/CONTRIBUTING.md
Type

[ ] Bug Fix
[ ] New Feature
[ ] Sample

Related Issues?
Fixes #X, partially fixes #Y, mentioned in #Z, etc.
What is in this Pull Request ?
Please describe the changes in the PR.
Guidance

You can delete this section when you are submitting the pull request.*
Please update this PR information accordingly. We use this as part of our release notes in monthly communications.
Please target your PR to Dev branch. If you do not target the Dev branch we will not accept this PR.

Well noticed @pagejustin, thanks for correcting!


Fix docker build

As far as I can tell, the current docker build is not working. This PR is an attempt to fix that and improve the Dockerfile.
Some of the changes are based on my personal preference. I'm open to feedback about those choices.
I tried to split this in multiple small commits to make the review easier (I'm not sure if you prefer a single large commit, but I can squash the changes later if you prefer).
Nevertheless, here's an overview of the main changes:

Add the js/lab directory as a build dep.: yarn install was not being called for js/lab. (If you had already called yarn install during develop, you would not notice this);
Merge base docker image into the main image: The build cache should always cache those initial commands, so I think that the base image only makes the build process more complex without any gains;
Split install operations: Use multiple RUN commands instead of a single one to avoid trashing the build cache if a single dependency is changed;
Add required files to the docker file: Adds only the required files to the docker image. They are also split in a way that avoids trashing the build cache when only the documentation files are changed.

Also need to update the docker part of the Architecture and Code Overview in README.md.

Thanks @JD557 the new script looks great. I think the only remaining work is documenting


Update page to reflect latest update.

This page needs updating to describe new PIM expericens. Screenshots look nothing like on this page.

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

ID: d8c2e8de-6dac-e81d-e58e-1ff6b8c5cb3a
Version Independent ID: 3f4af047-05af-3631-1bba-c6cf6b41fed4
Content: Grant access to manage PIM - Azure Active Directory
Content Source: articles/active-directory/privileged-identity-management/pim-how-to-give-access-to-pim.md
Service: active-directory
Sub-service: pim
GitHub Login: @curtand
Microsoft Alias: curtand

@zlDev , Thanks for your feedback. We are looking into this query and will update you as soon as possible.

@curtand Please check on the same


depreciation issue: componentWillRecieveProps is depreciated

depreciation issue: componentWillRecieveProps is depreciated and will be removed in the next release warning pops up
react-native: 0.60.5

Looks like duplicate of #314 for now.
Haven't seen any movement on this project for awhile to indicate it might be resolved anytime soon?

You can use alternative this library https://github.com/santomegonzalo/react-native-floating-action . There is no deprecated lifecycle method warning.

Should be taken care of in #333

+1

Hey Guys,
i know this solution is temporary but if you change your code with below code you can't see warning.
into node_modules/react-native-action-button/ActionButton.js
componentWillReceiveProps(nextProps) { .... }

to
UNSAFE_componentWillReceiveProps(nextProps) { .... }

Thank you. :+1:

Any update here on the release?

Apparently this issue is fixed in the latest version but not updated in npm.
Use this to get the latest version
npm install https://github.com/mastermoo/react-native-action-button

I followed @wniroshan's recommendation:

npm install https://github.com/mastermoo/react-native-action-button

Then I applied the fix for add explicitly - useNativeDriver: false, then I used patch-package to keep everything versioned.

This is still an issue. Is this library not maintained?

seems abandoned...

@gregfenton - this is a great tip to solve this issue. Thanks! Nice to have patch-package in the toolbelt =]


maya-nodebot daemonset yaml creation

Is this a BUG REPORT or FEATURE REQUEST?
Choose one: BUG REPORT or FEATURE REQUEST
FEATURE REQUEST

What happened:
maya-nodebot runs on every node and it supports only cli now.
What you expected to happen:
maya-nodebot should run as daemonset in all nodes of kubernetes with incluster config settings.
How to reproduce it (as minimally and precisely as possible):
kubectl get ds should list maya-nodebot

This has been deprecated. node-disk-manager will cover this.


Doc minor setting changes

view button added (requires the newest furo, will work correctly once merged to main)
GitHub icon in the footer
showing tips delayed, top position

could you try pip install --upgrade furo?

could you try pip install --upgrade furo?

Thanks, that solved it! Maybe we should mention that we should use the updated version of furo?

Did you mean in the how-to?


GoogleFit.Authorize is hanging

I've installed & linked react-native-google-fit in order to allow my app to read Google Fit data. Specifically, I need to read the user's heart rate data (it's possible that I will need more than this at some stage).
I've created an OAuth 2.0 Client ID (although I just created it in the console and then downloaded a file - I haven't actually put that file anywhere as none of the instructions I followed told me what to do with it).
In my app I have:
import React, { useEffect } from 'react'
import GoogleFit, { Scopes } from 'react-native-google-fit'

const Navigator = props => {

  useEffect(() => {
    GoogleFit.checkIsAuthorized().then(() => {

        if(!GoogleFit.isAuthorized) {
          const options = {
            scopes: [
              Scopes.FITNESS_ACTIVITY_READ,
              Scopes.FITNESS_BODY_READ
            ],
          }

          console.log("Attempting to authorize");

          GoogleFit.authorize(options)
            .then(authResult => {
              if (authResult.success) {
                console.log('Success')
                dispatch("AUTH_SUCCESS");
              } else {
                console.log('Denied')
                dispatch("AUTH_DENIED", authResult.message);
              }
            })
            .catch(() => {
              dispatch("AUTH_ERROR");
            })

        }else {
          console.log("Already authorized")
        }
      })
  },[])

When I run this, I get a screen which asks which Google account I wish to use to continue to my app; I select my account and then I just get a white box with the Google spinner in it.  In the console I can see "Attempting to authorize", but then nothing else happens.
Am I doing something wrong?  I believe I have followed all steps in the instructions, but I must be missing something.

First, you don't need to link if your RN version is above 0.6.
You need to follow the instruction https://developers.google.com/fit/android/get-api-key#find_your_apps_certificate_information then request your OAuth 2.0 Client ID .
the development keystore is located in android\app. You need to get your SHA-1 from the keystore file and use it during your authtication. You also need to get your app name from  android/app/src/main/AndroidManifest.xml. Both are mentioned during the process. You just need to carefully follow the instruction then you should be fine,
In you google console, there is a library section under Dashboard, make sure to add fitness api there.

Thanks for your reply.  I went through all of that to check, and it's all in, but still not working.  I wonder whether it's to do with how I've configured the API.  I'm on a debug version of my app for starters, and running on a local device (connected via USB). Does it need to be a production version?
I had it set to "External testing" but changed that to "Internal Testing" - once I did that, I opened my app again, and tried to authorise Google Fit (with my own gmail address - the app is set up under the company account).  This time I was denied access.
Does it have to be set up in a particular way?

Hmm, it's really hard to tell. I will try to create a fresh new RN app recently and go through the entire auth process.
And then rewrite the auth instruction doc.

Hi there!
i got some issue maybe related to Authorizing Or Somethingeles that i cant figure out!
That i can get stepCountSample for sure but heartRate and blood pressure samples is always return empty array, though my google fit app already have heart rate data.
Here is my google fit app data.

@NguyenHoangMinhkkkk  you are missing the scope
  Scopes.FITNESS_HEART_RATE_READ,

I hadn't managed a proper way to return error when scope is missing.

@NguyenHoangMinhkkkk you are missing the scope, also read the doc carefully. It's on the doc but ppl keep missing those lines.
  Scopes.FITNESS_HEART_RATE_READ,

I hadn't managed a proper way to return error when scope is missing.

Thanks for your replying! but i already add it at the authorize function!

@NguyenHoangMinhkkkk you are missing the scope, also read the doc carefully. It's on the doc but ppl keep missing those lines.
  Scopes.FITNESS_HEART_RATE_READ,

I hadn't managed a proper way to return error when scope is missing.

Thanks for your replying! but i already add it at the authorize function!

Also include  Scopes.FITNESS_ACTIVITY_READ to see if it works, it can be the problem that your old fitness data is from the old scopes, and new heartrate data will be in new scope. Depending on how developer implements it, it's confusing but Google seems like going to enforce the change in the future.

@NguyenHoangMinhkkkk you are missing the scope, also read the doc carefully. It's on the doc but ppl keep missing those lines.
  Scopes.FITNESS_HEART_RATE_READ,

I hadn't managed a proper way to return error when scope is missing.

Thanks for your replying! but i already add it at the authorize function!

Also include Scopes.FITNESS_ACTIVITY_READ to see if it works, it can be the problem that your old fitness data is from the old scopes, and new heartrate data will be in new scope. Depending on how developer implements it, it's confusing but Google seems like going to enforce the change in the future.

It also not work,  may google did some change which i dont know. i need more time to reseach other solutions.
Thanks

@NguyenHoangMinhkkkk  Maybe you should try to inspect the background process in Android studio to check if the data exist but failed to pass to the RN.
I just try the sample code and was able to receive data which I added from rest api.

@SharonGilmore Did you find a solution? I'm having the same problem

Hi All,
For me, the issue was in the google console while setting the project I had made the project OAuth consent screen External so, I had to add a test user account (google account id) to access the API.
After adding that I got a screen similar to

and after clicking continue I was able to use the google fit APIs.

Hi All,
For me, the issue was in the google console while setting the project I had made the project OAuth consent screen External so, I had to add a test user account (google account id) to access the API.
After adding that I got a screen similar to

and after clicking continue I was able to use the google fit APIs.

Did you manage to find a workaround for this? This screen suddenly appeared in our apps. It is causing alarm among our users.

@mabc21  I believe these are the test users who are testing your application, If not then your app deployment on the play store is not done correctly. In the debug environment it is fine see this screen.

@minkeshj I think we need our app to be verified by google since they changed the scopes of Google Fit to be a sensitive scope, they just changed it this year. Your apps are verified?

No, Actually I was testing certain features on the debug build only that's
when I encountered this issue. Have not published app on production yet so
can not comment on it.
On Tue, Jun 15, 2021 at 12:35 PM Angela Cabonce @.***>
wrote:

@minkeshj
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_minkeshj&d=DwMCaQ&c=kscRujJyAkQsYKgOOQVwjhZ5JZcK7vY7fLntrz10DKg&r=ZNRD4o2TfflFgv9V9Qedy_o4l-8CtfI1KuSoQUhDZHk&m=vibJgtw_yT8R3Fta1f96Vpa2N4ORKutsg2OWEnq7gkw&s=3Xr_SXPTurr5xy3ML5CCJaz-_4qLZ5BzVmUCZOkxXAE&e=
I think we need our app to be verified by google since they changed the
scopes of Google Fit to be a sensitive scope, they just changed it this
year. Your apps are verified?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_StasDoskalenko_react-2Dnative-2Dgoogle-2Dfit_issues_217-23issuecomment-2D861237733&d=DwMCaQ&c=kscRujJyAkQsYKgOOQVwjhZ5JZcK7vY7fLntrz10DKg&r=ZNRD4o2TfflFgv9V9Qedy_o4l-8CtfI1KuSoQUhDZHk&m=vibJgtw_yT8R3Fta1f96Vpa2N4ORKutsg2OWEnq7gkw&s=tllkIV-FsgoDIXFwEF8W-_I1anfo_vpw4LKviBWbXxE&e=,
or unsubscribe
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AOQYON36WXSYMWTTKZJS3QDTS33S7ANCNFSM4YCZXP2Q&d=DwMCaQ&c=kscRujJyAkQsYKgOOQVwjhZ5JZcK7vY7fLntrz10DKg&r=ZNRD4o2TfflFgv9V9Qedy_o4l-8CtfI1KuSoQUhDZHk&m=vibJgtw_yT8R3Fta1f96Vpa2N4ORKutsg2OWEnq7gkw&s=9LfWwSKliWlAcoPXdKNp21DODewXribu1NUuGXwyvUc&e=
.

--
This message and its
attachments are confidential (or legally privileged)
information and are meant
solely for the addressee of such message. Any
unauthorized use of the message /
its attachments is strictly prohibited.

I tried by adding my login email id to test users. If you are not an organizational user you can not make your oAuth consent screen to internal and so for external you either need to add a test user or publish the app and send it for review. Though I do not know the steps of sending the app to review in google console.
Internal Testing link
Verify app link


how to prevent testcafeLighthouseAudit from rendering new page to test the url.

how to prevent testcafeLighthouseAudit from rendering new page to test the url
it redirected the url to another url

Hi @essaabouasali-code, can you provide a bit more context? You can also share a repository where I can reproduce the bug.

I can add my $.02 because I am running into the same issue. It looks like lighthouse always runs in a fresh browser session, so if I am ether using BasicAuth or trying to login prior to running Lighthouse the tests will fail.

I've just experienced this same issue. Unfortunately, if the page to be audited is sat behind a login page of any kind, it appears that currently this won't work, as the page to be audited pops up in a new tab and needs to be authenticated again.

I've just experienced this same issue. Unfortunately, if the page to be audited is sat behind a login page of any kind, it appears that currently this won't work, as the page to be audited pops up in a new tab and needs to be authenticated again.

Sorry my english is bad, you just launch your page first and access to Network tab in Inspect -> in Headers (Request Headers) you will see your authorization code -> Copy it to here! Good luck!

I'm also facing a similar issue. could anyone please provide workaround if any.


⚠ Guya.moe Proxied has degraded performance

In 3cc692e, Guya.moe Proxied (https://ice.guya.moe/) experienced degraded performance:

HTTP code: 200
Response time: 13260 ms

Resolved: Guya.moe Proxied performance has improved in 2f76953 after 2 days, 5 hours, 20 minutes.


build.graddle

I added the dependency line into my existing project graddle build file on Android Studio beta 0.8.14
dependencies {
compile fileTree(dir: 'libs', include: ['*.jar'])
compile "com.android.support:appcompat-v7:21.0.+"
compile 'com.getbase:floatingactionbutton:1.2.0'
}
and I got this error  when build. Note that I haven't even added any codes. What could I be doing wrong?
Error:Execution failed for task ':app:processDebugResources'.

com.android.ide.common.internal.LoggedErrorException: Failed to run command:
C:\Users\vnguyen5\android-sdks\build-tools\20.0.0\aapt.exe package -f --no-crunch -I C:\Users\vnguyen5\android-sdks\platforms\android-21\android.jar -M D:\dev\MyTestApplication\app\build\intermediates\manifests\full\debug\AndroidManifest.xml -S D:\dev\MyTestApplication\app\build\intermediates\res\debug -A D:\dev\MyTestApplication\app\build\intermediates\assets\debug -m -J D:\dev\MyTestApplication\app\build\generated\source\r\debug -F D:\dev\MyTestApplication\app\build\intermediates\res\resources-debug.ap_ --debug-mode --custom-package com.vnguyen.mytestapplication -0 apk --output-text-symbols D:\dev\MyTestApplication\app\build\intermediates\symbols\debug
Error Code:
-1073741819

I have the same error and I'm trying to import version 1.9.0 as indicated under Usage. Build tools are up to date as well and mavenCentral() added to the repositories
compile 'com.getbase:floatingactionbutton:1.9.0'

Any ideas? hi danijoo!

It was a build tools version issue, updated the version to the latest in gradle and it ran fine, thanks.


Sometimes stderr is expected but not important

Attempts to address https://github.com/Bachmann1234/diff-cover/issues/7
Hides stderr output that only confuses the output of a test run. We don't care about the output, we just care that when we are handed invalid args we don't attempt to continue running. We can trust parse_args to handle invalid args

Coverage increased (+0.0%) to 99.23% when pulling 7c709060b07dd67c884d577b33dfe3c58adb216d on suppress_std_err into fbbbf8c1c4cfa5495f553f6f7cd85c5539d74aec on master.

Cool. :)


OwnerID was not set

Hey...
I did everything good, and when I launch the runbot.bat,
It says "OwnerID was not set"
....
What can I do ?
Thanks,
Maxime

In options.ini ensure you have write correctly your ID in OwnerID =

Enable dev options, go and copy your own id and paste that number next to the OwnerID =

Had same problem, everything set just perfectly. Same error whether file exist and filled or not

Can't reproduce this. Ensure that you have filled out options.ini correctly. Make sure it is not named options.ini.ini (file extensions may be hidden on your system).


Add support for Slim CI

Describe the feature you'd like
Currently, the dbt-run hook will pass all staged models, possibly causing a re-run of the project that is not necessary, which can be problematic on large projects with a very long runtime.
It would be ideal to be able to support dbt's built-in Slim CI support. This way, if the developer has already ran and tested the project and made no further changes, the hook will detect this and skip the run. If, however, the developer did not run the project, the hook would run the changed models automatically, ensuring that nothing is being committed without successfully running.

Hey @MartinGuindon I just fixed that in bleeding edge version. You can run it with:
 hooks:
 - id: dbt-compile
   args: ["--models", "state:modified", "--cmd-flags", "++defer", "++state", "path/to/artifacts", "--"]

beware you need to change --defer and --state to ++defer and ++state since there are 2 argument parsers.


Travisci sonarcloud

[skip it]

SonarQube analysis reported 5 issues

 2 critical
 1 major
 2 minor

Watch the comments in this conversation to review them.
4 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 ServerConfig.java#L152: Define a constant instead of duplicating this literal "Command must not be null" 6 times.
 ServerConfig.java#L153: Define a constant instead of duplicating this literal "Command must not be empty" 6 times.
 ServerConfig.java#L18: Remove this empty statement.
 ServerConfig.java#L18: Remove this redundant "static" qualifier.

SonarQube analysis reported 1 issue

 1 minor

Watch the comments in this conversation to review them.

ok to test


Zooming causes off-screen image parts to be displayed wrongly until zoomed again while on-screen

Sometimes I have a big image and zoom in on something specific, but then also want to pan around to other parts of it. However, zooming seems to displace parts of the image that are currently not visible on the screen, I then have to zoom again once they are visible to fix that, which can be a bit annoying.
Example:

just opened

pulled offscreen and zoomed out

pulled back on-screen

fixed after zooming again

I've noticed this recently too, It's pretty annoying. For some reason zooming just started bugging out a while ago... Not sure when, maybe after some particular Windows update.
I haven't gotten around to debugging it yet though.


loss explosion

Hi, there. I tried to train your model on my own dataset following the default setting but found that the loss decreases gradually before about 80 epochs while suddenly exploded to thousands. Many tries meet the same problem and the same on the rain13k dataset. Do you have any ideas of this issue or ever met with such problem when you trained your model for the final 4600 epochs?
Thank you.

Hi, sorry for the delay, as reported by others, the issues should be solved if you change to a larger batch size. Thanks!


Fix layer 43

This is just a reminder to fix layer 43 that doesn't look like it should.
This will probably also affect other layers.
Edit (more test cases: Thanks, Bruno!):

[222,221] ; [224,223] ; [225,0] ; [226,0] ; [227,0] - or the six last suggested layers from Garen's code, excluding for the 265 one

After inspecting this a little closer, it seems that this issue is caused by wrong datatypes. The original code used a weird mixture of Uint8/Int16 for representing distortion effects (in two different files for some reason). I'll try and fix this ASAP.

Fixed with 116ba970740833b9c48d24ff3e740a2d0af94c50.


Use go 1.16 embed for templates

Changes

:gift:  YAML helpers now take an fs.FS, allowing us to use the new embed package for bundling yaml into test binaries

/kind enhancement
Fixes #193

Not sure if we need to bump all the downstreams first, this is a breaking change

https://github.com/knative-sandbox/reconciler-test/pull/195 needs to merge

I think for the action to update from #195, you need to rebase this PR

We should get this in first thing after 0.23 is released.


Check BTC/PTS addresses on key/balance import?

Latest CLI has the ability to import BTC/PTS keys. Is it possible with GUI?
See
https://github.com/cryptonomex/graphene/pull/558
https://github.com/cryptonomex/graphene/issues/557

This issue was moved to bitshares/bitshares-ui#50


Check capacity enough for tx

Problem
Closes #670
Solution
Before submitting capacity transaction via payWithCapacity or payWithCapacityBatchAll check if tx fee in capacity is enough as compared for remaining capacity from provider
Tests

[x] Test full e2e and checks are working

@JoeCap08055 read your comment late, I did little bit of manual testing at the edge, I don't think that covers,
I plan to write e2e tests but need some familiarity with them first so a PR will come next week


runtime: error returned by os.OpenFile blocks panic handler

go1.6 windows/amd64 on Windows 8
The following program tries to create a new file with os.O_EXCL and panics if the file already exists. When panic(err) is called, "panic:" is written to stderr and the program blocks. You have to terminate it with Ctrl-C. If you uncomment err.Error() then you get a normal backtrace and the program exits.
I'm guessing that there is a problem with retrieving the error information in the panic handler, but if done before panicking, then everything works as expected.
package main

import "os"

func main() {
	_, err := os.OpenFile("tmp", os.O_RDWR|os.O_CREATE|os.O_EXCL, 0666)
	if err != nil {
		//err.Error()
		panic(err)
	}
}

/cc @alexbrainman

Interesting. The program could be further simplified to:
package main
import "syscall"
func main() {
panic(syscall.Errno(0x50))
}

The FormatMessageW syscall returns successfully,
but after that the runtime deadlocks in stoplockedm
(line 1682 of runtime/proc.go, notesleep(&_g_.m.park))

OK, it's because startpanic_m has already called
freezetheworld() (and set sched.gcwaiting to 1).
Calling any cgocall after that point is going to deadlock.
Normally this won't be a problem, but Windows
syscall.Errno.Error() relies on cgocalls to retrieve
the error messages.
I don't know how to fix this problem. Any suggestions?
/cc @aclements

Interesting. I wonder if we shouldn't be calling in to user code to format the panic message after freezing the world. Presumably on any platform, if an Error method calls cgo or does something nontrivial with goroutines or something, it would deadlock the system in the same way.
We could format all of the panic messages first, then freeze the world, then print them.

This also means we're calling in to user code while holding a runtime lock (paniclk). For example, the following results in a "panic: fatal error: schedule: holding locks", which should not be triggerable by user code:
package main

import "runtime"

type e struct{}

func (e) Error() string {
	runtime.Gosched()
	return "error"
}

func main() {
	panic(e{})
}

This one deadlocks on Linux (I haven't looked in to why the Println is necessary):
package main

/*
char *geterror() {
	return "error";
}
*/
import "C"
import "fmt"

type e struct{}

func (e) Error() string {
	fmt.Println("Hi!")
	return C.GoString(C.geterror())
}

func main() {
	panic(e{})
}

And this one panics with another internal error, "fatal error: stopm holding locks":
package main

import "fmt"

type e struct{}

func (e) Error() string {
	fmt.Println("Hi!")
	fmt.Println("Bye!")
	return "x"
}

func main() {
	panic(e{})
}


Ticket clipped out of board in board preview

From @Monttinen on March 19, 2015 11:33
<EMAIL_ADDRESS>pw: peruna123
http://student.labranet.jamk.fi/~G3318/testing/ClippedTicket.png
Copied from original issue: N4SJAMK/teamboard-feedback#80

This is probably due to some teamboard-api shenanigans. There actually isn't a check whether or not the ticket is inside the bounds of the board, so if you POST a ticket directly there, not using the browser client, there is no check.
If this bug can be reproduced directly from teamboard-client-react, reopen this issue.

Waiting for more incidents..


computeDestinationPoint uses a different hardcoded earth radius value instead of the constant

in computeDestinationPoint.ts
const computeDestinationPoint = (
    start: GeolibInputCoordinates,
    distance: number,
    bearing: number,
    radius: number = 6371000
) => {

instead of using  export const earthRadius = 6378137; from constants.ts
this issue appeared in v2.0.25

I submitted PR #296 to fix this, could someone give it a review and merge when possible please?


feature/Scroll To Top Button!

scroll to the top button allowing the user to reach the top easily without scrolling.
it would be great if we add this button to our website.
assign this issue to me, if you want to implement it.

Hi @mohitparmar1, this issue is already addressed in #269


[darwin] rustPlatform.buildRustPackage: do not add libiconv to buildInputs

Motivation for this change
Many buildRustPackage-based derivations require libiconv as a build
input on Darwin. This is caused by libc unconditionally linking
against libiconv. However, this was recently changed:
https://github.com/rust-lang/libc/pull/2150
We will probably see this dependency disappear over the coming months
once a new libc crate version is released and other crates adopt
this new version.
Since this is already an optional dependency and it will disappear
in the coming time, we should probably not unconditionally add it
to all Rust crates.
Can't test, because I don't have a Darwin machine.
cc @jonringer
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

0.2.95 was released two weeks ago, so taking this PR out of draft.
https://github.com/rust-lang/libc/releases/tag/0.2.95
This is the first version that does not unconditionally link libiconv.
Of course, downstream crates have to update to this version of the libc crate, but the number of crates requiring iconv on macOS should shrink drastically over time.

rustPackages is currently broken on 21.05 because libiconv was removed. Can we please introduce it again?

rustPackages is currently broken on 21.05 because libiconv was removed. Can we please introduce it again?

Seems to be there in 21.05?
https://github.com/NixOS/nixpkgs/blob/6613a30c5e3ee59753181512b4bedd4121569925/pkgs/build-support/rust/default.nix#L122

From a quick look, lorri seems to use Carnix? Carnix doesn't use buildRustPackage, but buildRusetCrate, so an override needs to be added to

The PR I linked actually switches it over to crate2nix since carnix is super out of date at this point and failed with the newer lock file format.

crate2nix also uses buildRustCrate.

tbh I stopped caring about Darwin and call upon @NixOS/darwin-maintainers

I mean this has been broken since May or even earlier, and is exactly why https://github.com/NixOS/rfcs/pull/112 exists.

@Profpatsch I'd kindly ask you to be less hateful and show some empathy for others.

This is still required.

(and by "this" I mean that libiconv still needs to be present)

For anyone else having issues with buildRustCrate, crate2nix or carnix and ending up here, the fix is in nixpkgs master: https://github.com/NixOS/nixpkgs/commit/c9f0c6f115f4369b5047c3c3086518294541d0bf.

@Profpatsch I'd kindly ask you to be less hateful and show some empathy for others.

apathy ≠ hate


Default filename parameter for save file dialog

Default filename parameter for save file dialog.
This is a solution for issue #140.
Supplied together with PR for Photino.Native (https://github.com/tryphotino/photino.Native/pull/140).

This looks good all in all. I just added a small change request to the PR. If you can resolve that we will merge it for the next release.

Can't found that change. Anyway i turned on "Allow edits by maintainers" option for my branch.

Can't found that change. Anyway i turned on "Allow edits by maintainers" option for my branch.

Sorry about that. I forgot to submit the review


stabilize chef_index_batch tests

A number of timeouts we were seeing seems to be a race condition in
shutting down the gen_server. To avoid this, we monitor the gen_server
pid and wait to get notified of its exit.

:+1:


Adds dyno type tag

As an alternative to #22: add a type tag to metrics, so that metrics are tagged both with the dyno instance (e.g. source:web.12) and type (e.g. type:web).
@abtris, @w-vi, does this work better for you?

Looks good to me. @w-vi can you check and merge?

@mezis Cheers, this will work for all of us.


Java EE to Quarkus 2 Migration Recipe Request

What problem are you trying to solve?

Looking to create a generic JavaEE to Quarkus 2 composite recipe who's primary purpose would be to move off EJBs. Planning on incrementally contributing to various parts of the upgrade. Will start with Adding Quarkus dependencies, removing JEE dependencies and converting Stateless, Stateful and Singleton EJBs to reasonable defaults on CDI. Eventually planning on upgrading Message Driven Beans to Camel Quarkus JMS/ActiveMQ, EJB Schedules to Quarkus Scheduler
What precondition(s) should be checked before applying this recipe?

That it is in-fact a JavaEE (Not JakartaEE) application to begin with
Describe the situation before applying the recipe

Working JavaEE application
Describe the situation after applying the recipe
Ideally, a fully transformed and working Quarkus 2 application
Are you interested in contributing this recipe to OpenRewrite?

Yes. Would be looking for feedback on best practices. I would like to stick to as many Declarative recipes as possible.

For folks following along: some initial work has already been merged in through
-https://github.com/openrewrite/rewrite-quarkus/pull/73
That means the initial recipes added there should become available for folks looking to move from Java EE to Quarkus.
Would appreciate anyone chiming in here with steps missed that ought to be added for a smooth transition.


Fix improper streaming behavior in data handling

Fix improper streaming behavior in data handling
Resolved an issue where data was not being streamed correctly due to improper use of the reduce function in the stream(Prompt prompt) method. This was causing data to be processed in a blocking manner, contrary to the expected reactive streaming behavior.
Changes made:

Removed the reduce operation which aggregated all data into one chunk, thus blocking the streaming.
Replaced the reduce with a direct handling of each individual item within the window to maintain continuous stream flow.
Added comprehensive tests to ensure streaming is performed as intended, and to verify the non-blocking behavior with multiple emitted items.

Resolves: #764

@joshlong @markpollack I have taken the liberty of opening a PR that attempts to address a problem I've been seeing with production usage where the streaming mode for the Azure OpenAI completions would, instead, return the entire response at once, which was not the desired behavior.
I will be happy to fix any potential issues and/or addressing any comments you might have!

I could be wrong, but wasn't the removed concatMapIterable code with window.reduce part needed for function calling?
Also, I'm under impression that the only thing needed to fix streaming issue in Azure OpenAI model was in this code from AzureOpenAIChatModel:
			.windowUntil(chatCompletions -> {
				if (isFunctionCall.get() && chatCompletions.getChoices()
					.get(0)
					.getFinishReason() == CompletionsFinishReason.TOOL_CALLS) {
					isFunctionCall.set(false);
					return true;
				}
				return false;
			}, false)

to replace return false; with return !isFunctionCall.get();. At least this is what I've used in my code.

Seems fixed by: https://github.com/spring-projects/spring-ai/commit/178a607cf6fc65e302b7420fe50cb8dff7e2df2d


fix: avatar display size

Currently avatars are being crop and display incorrectly, this small fix address this issue by changing the objectFit property to cover.

Hello! The view is different in the card than in the profile view. Could you make it so that the pic is equally shown in both cases? Thanks a lot for your work! 🙌🏻

Fixing it ASAP. I'll send a path soon.

Ha ha ha, at the end keeping the circular shape for all avatars it's kinda of tricky, some vertical images look cropped :(

This is the best I can make them look, using object-position: "0 0".

@spacecowb0y Hmm, yes, this might cause problems for people vouching for new profiles. Good point.
Do you think we should add some sort of tool for helping people crop their uploaded images?

That could help, there are some client side tools for that. Also validating the dimensions of the images might help as well.

That could help, there are some client side tools for that. Also validating the dimensions of the images might help as well.

@spacecowb0y Okay, could you create a new GitHub issue for that? And please put the relevant client-side tools in the issue description ☺

@spacecowb0y Hmm, yes, this might cause problems for people vouching for new profiles. Good point.
Do you think we should add some sort of tool for helping people crop their uploaded images?

Currently working on this, as well as many other UI features. ETA a few weeks.


Update package.json remove vulnerabilites

Motivation
When npm audit was is run any errors showing
Solution
Update packages by more recent versions without errors of vulnerabilities

Coverage remained the same at 74.667% when pulling e0f3c01a3e17c263848bdba72dc52ce6a72eea69 on reginaldojunior:master into ce60bd121cb888bbd339c4b1240cb1d9f19ca3cf on benbria:master.

@reginaldojunior I've added support for CoffeeScript 2.6.1 and fixed up the dependency on my fork. Try out the prerelease branch 4.0.0-pre and let me know if it meets your needs: https://github.com/DanielXMoore/CoffeeCoverage


fixing broken package in streamlit

Pull Request
fixing broken package in streamlit
As it is now, this example doesn't run.
Description
This fixes the broken import

Type of Change
Indicate the type of change by choosing one (or more) of the following:

🛠 Bug fix

Testing
Launched webpage successfully with this fix.

I know this will fail the linter right now but I wanted to get this in, as it's currently broken (not just a lint thing).


Fix more setup.cfg formatting

Oops, sorry - too late!

Thanks for catching it!  https://github.com/jupyter/terminado/pull/134


Simplifying where expressions

Before Kysely supported or and and compound conditionals, I had written my own filter supporting compound conditionals. I'm grateful that Kysely now provides these, because it allows me to ditch a lot of code and rely on someone else to maintain this functionality.
However, I'm a bit unhappy about one thing. My filter syntax was more concise and therefore easier to read. Consider the following v0.24 Kysely query:
    await db
      .deleteFrom("some_table")
      .where(({ or, and, cmpr }) =>
        or([
          cmpr("columnA", "=", 100),
          cmpr("columnB", ">=", 32),
          cmpr("columnC", "in", ["a", "b", "c"]),
          and([
            cmpr("columnD", ">", 0),
            cmpr("columnE", "<", 100),
            cmpr("columnF", "is", null),
          ]),
        ])
      );

With two syntactic changes, this could become:
    await db
      .deleteFrom("some_table")
      .where(({ or, and }) =>
        or(
          ["columnA", "=", 100],
          ["columnB", ">=", 32],
          ["columnC", "in", ["a", "b", "c"]],
          and(
            ["columnD", ">", 0],
            ["columnE", "<", 100],
            ["columnF", "is", null],
          ),
        )
      );

The two changes are:

Replace cmpr() with tuples.
Have or and and accept rest parameters instead of arrays.

This would be another breaking change, but now would be the time to do it, before too many people have converted to the new syntax.

No thanks 😄

No thanks 😄

I see you found the proposal tempting. 🤣


Revert "Make the driver aware of all the flags the selected toolchain's swift-frontend supports."

Reverts apple/swift-driver#482
Making SwiftScan internal instead of private seems to be breaking the ability to build SwiftDriver from SwiftPM PRs.

@swift-ci please test

The error that this triggers in SwiftPM self-hosted PR testing is:
14:44:40 [836/922] Compiling SwiftDriver PredictableRandomNumberGenerator.swift
14:44:40 [837/922] Compiling SwiftDriver DependencyGraphBuilder.swift
14:44:40
/Users/buildnode/jenkins/workspace/swift-package-manager-with-xcode-self-hosted-PR-osx/branch-main/swiftpm/.build/checkouts/swift-driver/Sources/SwiftDriver/SwiftScan/SwiftScan.swift:69:7: error: cannot use struct 'swiftscan_functions_t' here; 'CSwiftScan' has been imported as implementation-only
14:44:40   let api: swiftscan_functions_t;
14:44:40       ^    ~~~~~~~~~~~~~~~~~~~~~
14:44:40 CSwiftScan.swiftscan_functions_t:1:15: note: type declared here
14:44:40 public struct swiftscan_functions_t {
14:44:40               ^
14:44:40 /Users/buildnode/jenkins/workspace/swift-package-manager-with-xcode-self-hosted-PR-osx/branch-main/swiftpm/.build/checkouts/swift-driver/Sources/SwiftDriver/SwiftScan/SwiftScan.swift:72:7: error: cannot use type alias 'swiftscan_scanner_t' here; 'CSwiftScan' has been imported as implementation-only
14:44:40   let scanner: swiftscan_scanner_t;
14:44:40       ^        ~~~~~~~~~~~~~~~~~~~
14:44:40 CSwiftScan.swiftscan_scanner_t:1:18: note: type declared here
14:44:40 public typealias swiftscan_scanner_t = UnsafeMutableRawPointer
14:44:40                  ^
14:44:40 [838/922] Compiling SwiftDriver SwiftScan.swift
14:44:40

Which is not right because using an implementationOnly type in an internal SwiftScan should be fine (this change made it internal from private). Those PR tests use a 5.3 compiler though to build swift-driver and this seems to be hitting a compiler bug in 5.3 that's been fixed since then, which is why we didn't catch it in our PR testing.
Until SwiftPM PR testing is updated to use a newer toolchain, it's probably best to still revert this and try to work around this bug.

Closing, instead submitting a workaround in:
https://github.com/apple/swift-driver/pull/487


Is there a way to mark code as ignored by the language server?

Most code analysis tools that I know have some sort of magic comment to mark code that should be ignored by the linter.

LuaCheck has -- luacheck: ignore
TypeScript has //@ts-ignore
ESLint has /* eslint-disable */
etc.

Is there such a syntax for EmmyLua/your language server?
If there is, I cannot find it documented anywhwere (in English at least).
If there isn't, I think there should be one.

I want to supports these:

ignore (code1(, code2)*)? : ignore in next line
disable (code1(, code2)*)? : disable in following code
enable (code1(, code2)*)? : enable in following code

But Im not sure how to mark them.

-- diagnostic ignore
-- lint ignore
---@lint ignore

or any others.

Maybe like this? (this is sort of how ESLint names them)
---@diagnostic disable-next-line: Disables diagnostics in the line after the comment
---@diagnostic disable-line: Disables diagnostics in the line the comment is on
---@diagnostic disable: Disables diagnostics for the following code
---@diagnostic enable: Enabled diagnostics for the following code


Receiving empty new tweet "None" from Stream

First thanks for this lib it's the easiest I found to manage Twitter Stream.
I have a strict Stream filter rule on my account, but I receive new tweet signals with empty tweet multiple times a day (tweet=None) from Stream, as there are no now tweets corresponding to this filter (I checked carefully).
Is it a normal behavior ? Or it can a bug from Twitter or the lib ?
Thanks.

Yes, stream in library only return the data. And not handle the data, Maybe you need filter the None data.

Okay thanks, so strange behavior from twitter data.