feat: add withImports higher-order plugin

This lets other frameworks inject their own imports.
import mdx from "vite-plugin-mdx"

export default mdx.withImports({
  react: "React", // default import
  "@mdx-js/react": ["mdx"], // named imports
})

Then users can install a wrapper package for a framework that is not supported by default.
npm install @svelte/vite-plugin-mdx

Love it.
CC @sskyy.

Released in v3.5.0


Multi-upload: "Next" button sometimes greyed out after removing one picture from the upload

Steps:

Select about 10 pictures, share them to the app
Start filling metadata (I used "Copy to subsequent media" but not sure whether necessary).
After a few picture, in the top strip of thumbnails, tap the "X" at the corner of a thumbnail (to remove that picture from the multiupload).
Result sometimes: "Next" button is not usable, no clear way to finish the upload.

Workaround: Tap "Previous" then "Copy to subsequent media" then "Next".

I want to work on this .

@shaahmeer It is yours, thanks! 🙂

@shaahmeer Any progress to share maybe? :-)


StackNavigator layout glitch for cards with no nav bar when navigating to card with nav bar

Current Behavior
When using a StackNavigator and navigating between a card with no nav bar (i.e. header: null) to a card with a nav bar, the card with no nav bar is "squashed" (i.e. layout is re-rendered to the height of a card with a nav bar) while the new card slides in.
Expected Behavior
I would expect that the card with no nav bar would stay exactly the same during the slide to the new card.  That is, it should not re-render the layout or change height because the "incoming" card has a nav bar.
Your Environment

software
version

react-navigation
1.0.0-beta.11

react-native
0.45.1

node
6.9.1

npm or yarn
3.10.8

Adding headerMode: "screen" resolves this issue, but then the subsequent fades aren't very nice.
I will try two navigators to see if that improves it.


Added link to wiki page

Fixes #378
changes made

I added a link to the Documentation on the wiki page
I removed the Android_README.md file

@ijemmao kindly review this

Got it @ijemmao


Running Dashboards on Azure Databricks

Describe the bug
I am trying to run ResponsibleAIDashboard on Azure Databricks Notebook. I have a cluster and I have installed raiwidgets library in the cluster.
When I do this - ResponsibleAIDashboard(rai_insights, cohort_list=cohort_list) , I get the error
ResponsibleAI started at None

I bet it is a IP and port problem because the tutorial - https://github.com/microsoft/responsible-ai-toolbox/blob/main/notebooks/responsibleaidashboard/responsibleaidashboard-census-classification-model-debugging.ipynb- runs seamlessly on my VM or local.

hi @achandak33 We only support the explanation and fairness dashboards on Databricks (and only the static view for the explanation dashboard).  Since Azure Databricks does not allow us to open ports, we can’t actually allow the user to open a new tab and view the all-in-one ResponsibleAI dashboard with dynamic functionality enabled in Azure Databricks.
Please see related closed issue:
https://github.com/microsoft/responsible-ai-toolbox/issues/1488
Also, the Error Analysis dashboard has some initial limited support I’ve added a couple months ago to display on Databricks, but only when using a spark model/dataset.  I could help the customer try to run it.  It’s not advertised or well-tested though, and I do plan on doing more work on it (like adding categorical support) before putting it in public documentation.
I’ll reply to the github issue below as well.
See related open issue for Error Analysis Dashboard:
https://github.com/microsoft/responsible-ai-toolbox/issues/944
Since then I have merged this PR to add support for spark models in a static view of the Error Analysis Dashboard:
https://github.com/microsoft/responsible-ai-toolbox/pull/1388
Note we don't publicly advertise it on this repository yet as we still need to support the heatmap, add support for categorical columns, fix some minor issues and test the functionality more.


SassError: Can't find stylesheet to import. @import "~@carbon/type/scss/font-face/sans"

I have angular app with cli version 13.2.2 I am using carbon design system in my app which has below versions ```
"@carbon/icons": "10.45.0",
"@carbon/type": "10.40.0",
"carbon-components": "10.52.0",
"carbon-components-angular": "4.57.6"

I am importing this package like this way @import "~@carbon/type/scss/font-face/sans";
"stylePreprocessorOptions": { "includePaths": ["src/styles", "node_modules"] }
Now during jenkins build creation it is giving me SassError: Can't find stylesheet to import. @import "~@carbon/type/scss/font-face/sans" error. It was working fine from last 2 years.
I removed ~ from import statement but no luck. Please help

Hi team,
Any updates

Could you verify the version is in your package-lock.json (Or Yarn.lock)? Any chance that the package was updated? I am not able to reproduce this on my end.

Hey no chance of package updation because we are not using ~ or ^ in package.json so package can not be update to minor or patch version

What changes were made to get this result? I can't really think of another reason, we cannot update a npm package once it is released. Have you tried a clean install npm ci in your automation environment?

Lets leave it, I commented this line https://github.com/import "~@carbon/type/scss/font-face/sans
The response time from your side is too slow so no more interested on this topic now.


2021-11-25 Release

Release Notes:
New Terms:

specified double negative thymocyte (Homo sapiens)
committed double negative thymocyte (Homo sapiens)
rearranging double negative thymocyte (Homo sapiens)
monocyte-derived dendritic cell
type 1 spiral ganglion neuron
type 2 spiral ganglion neuron
cochlea auditory hair cell

Edits:

Temporary fix on enterocyte of the appendix (will break if uberon import is refreshed)
Made auditory hair cell more general
Fixed def of Cajal-Retzius cell
Added some comments on thyocte cells
Added various synonyms
Merged myoepithelial cell of lactiferous gland
Added layer axioms and equivalent classes for a bunch of neuronal cells

Currently enterocyte of the appendix temporarily fixed by forcefully placing a uberon mirror file that excludes offending axioms by removing all uberon_* terms in cl_import.owl in uberon (yes very cyclical)
Failure due to obsolete myoepithelial cell of lactiferous gland - will look into this, but im guessing its cause uberon axioms of these (hopefully not cyclical again), will look into this

Ok fixed :) all ready to go

@paolaroncaglia if all good, could you give a thumbs up in the review please, thanks


fix: types export

Explain your changes
Fix's exporting of types
Checklist

[x] I have read the “Pull requests” section in the contributing guidelines.
[x] I agree to the terms within the code of conduct.

🛟 If you need help, consider asking for advice over in the Kinde community.

This resolves https://github.com/kinde-oss/kinde-auth-nextjs/issues/143
Published under @kinde-oss/kinde-auth-nextjs@beta


[Migrated] Getting Gateway Timeouts (504), Despite Setting timeout_seconds to 120

Originally from: https://github.com/Miserlou/Zappa/issues/2182 by JordanTreDaniel
I have deployed a lambda that performs image manipulation, which can take a while sometimes. Longer than 30 seconds is a normal, expected time (for some photos)
At first, the timeout was showing in the zappa tail logs, but now, after setting the timeout_seconds option to 120, it seems as though the Amazon Cloudfront/Amazon Gateway is still enforcing a 30-second limit. I have dug around on AWS, and it seems I can only change this from a Cloudfront console. I don't believe there is an Cloudfront Distribution for this Lambda.
I am running on python 3.6
Expected Behavior
I would think that if I set the timeout_seconds to 120 in the Zappa settings, zappa would apply that setting to not only the lambda, but also whichever gates stand in front of it. (Gateway OR CloudFront Dist)
Actual Behavior
I get 504's when I don't choose to have the lambda downsample the image. (When it's normal for it to take long)
Possible Fix
Is there a way to apply the timeout_seconds to more than just the Lambda itself?
Steps to Reproduce

Go to http://www.rapclouds.com
Sign In (using oauth, super easy)
Search a song
Go to song
Click blue settings button by word cloud
Pick a "mask" (image to make the wordcloud from)
Try diff values between 3 & 0 for the downSample option.
0 downsampling is almost a guaranteed timeout exception. (Network tab in devtools to verify)

Your Environment

Zappa version used: 0.51.0
Operating System and Python version: I use Mac, but this is only a problem on AWS. (python 3.6)
The output of pip freeze:

argcomplete==1.12.0
boto3==1.14.31
botocore==1.17.31
certifi==2020.6.20
cfn-flip==1.2.3
chardet==3.0.4
click==7.1.2
cycler==0.10.0
docutils==0.15.2
durationpy==0.5
Flask==1.1.2
future==0.18.2
hjson==3.0.1
idna==2.10
itsdangerous==1.1.0
Jinja2==2.11.2
jmespath==0.10.0
kappa==0.6.0
kiwisolver==1.2.0
MarkupSafe==1.1.1
matplotlib==3.3.0
numpy==1.19.1
Pillow==7.2.0
pip-tools==5.3.0
placebo==0.9.0
pyparsing==2.4.7
python-dateutil==2.6.1
python-slugify==4.0.1
PyYAML==5.3.1
requests==2.24.0
s3transfer==0.3.3
scipy==1.5.3
six==1.15.0
text-unidecode==1.3
toml==0.10.1
tqdm==4.48.0
troposphere==2.6.2
urllib3==1.25.10
Werkzeug==0.16.1
wordcloud==1.7.0
wsgi-request-logger==0.4.6
zappa==0.51.0

Link to your project (optional): http://www.rapclouds.com
Your zappa_settings.json:

 {
	"dev": {
		"app_function": "app.app",
		"aws_region": "us-east-1",
		"profile_name": "default",
		"project_name": "wordcloudflask",
		"runtime": "python3.6",
		"s3_bucket": "word-cloud-bucket",
		"slim_handler": true,
		"cors": true,
		"binary_support": false,
		"timeout_seconds": 120
	}
}```

The API Gateway and Cloudfront timeouts are 30 seconds and cannot be changed. The timeout_seconds setting does have an effect when running async invocations (for example, issuing django manage commands).
https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/lambda-requirements-limits.html
https://docs.aws.amazon.com/apigateway/latest/developerguide/limits.html

APIGW has a 30 second timeout.
The zappa configurable timeout is for the lambda function only.
closing.

What can be done to avoid it? Any workarounds?

What can be done to avoid it? Any workarounds?

We can't modify APIGW max timeout, what's your use case and why do you need to run it for more than 30 sec?

@sridhar562345
My app makes a lot of api calls (5ish) during bootup (of the app).
I do have "keep_warm" enabled.
As I understand from here, only 1 lambda is warm and thus some of these api calls will need to startup a new lambda.
The startup takes some time (2/3 minutes), because I also use slim_handler.
But I suppose I should find a way to keep multiple lambda warm?

Hi @JacobDel, I think you can keep the main lambda warm and make those API calls async, Plus to keep lambda warm make those warm calls to an endpoint which doesn't ping your other APIs.

Thank you for your response @souravjamwal77 .
How can I choose which endpoints to keep warm?

@JacobDel, you can create a seperate dummy endpoint to keep warm. Or whenever you're getting a request check if it's coming a user or it's a keep warm call.
Then, if the call came from AWS do nothing and return a true response.
And if this call is from a user then start hitting your other lambdas asynchronously and load data on the front side asynchronously.
Finally, still if any of your API call fails then try to use zappa's async feature.

@JacobDel
Instead of using a slim handler, why don't you try to deploy your app as a docker image? So your initialization times will be reduced.

@souravjamwal77 I created a timed event like you mentioned but that did not create a difference, even with an expression rate of 1minute.
@sridhar562345 this solved my problem! I also noticed in the logs that the warm calls now actually do something after disabling slim handler.
thank you guys for solving my problem!


all: fix some typos

@cuishuang Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@cuishuang Thank you for signing the Contributor License Agreement!

The _spans.adoc actually gets built from sources, so you could search for that text in the code so that we can fix it there.


Revert "Docs: Update 04_backup.rst"

Reverts restic/restic#3115. As I said in a comment on that PR, the old description was correct: --files-from reads filenames from a file. Also fixes the merge conflict with #3017.

Perhaps the text should say what you just wrote: "reads filenames from a file"

I'm not going to mess with this text while the other PR is close to merging. It changes this sentence to

The options --files-from, --files-from-verbatim and --files-from-raw allow you to list the files to back up in one or more files.

which should be a bit clearer.

@greatroar Sorry for not noticing your comment in the other PR, honestly I think I had a race condition with you there. Regardless, I'm closing this one for now given what you say and that #3017 isn't in conflict anymore. Thanks.


⚠ GitLab Server has degraded performance

In 0bc7fcc, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 8743 ms

Resolved: GitLab Server performance has improved in 89e33ab.


Can I customize the database?

My protein sequence is from a mutant library, is it possible to speed up my protein structure prediction by changing the dataset? For example, without using the BFD dataset, if it is possible, how should I implement it?

You can use the --db_preset=reduced_dbs flag to run a slightly more lightweight MSA search (doesn't search BFD, only its small clustered version).
However, running without these databases either requires modifying the data pipeline code to skip the databases you are not interested in searching, or by modifying the databases to contain only a single sequence (open the database fasta file in a text editor and remove all but the first two lines or use head -n2 <fasta file>).


Mac libs for Qt 6.4.3 are 2-3 times bigger than from Maintenance tool

put wrong build type

Glad you got it working!


copy matcher proto and dependencies from Envoy

Moves the Matcher proto into the udpa repo along with a few dependencies: TypedExtensionConfig, StringMatcher and RegexMatcher.

@htuch @markdroth

Nice, thanks! Yep, agree that this should be in the xds/ tree and TypedExtensionConfig should be core.

@htuch @markdroth ptal, i made the moves
Also FYI I dropped reserved fields, deprecated fields, udpa annotations, etc. when copying these over

Sorry about all the force pushes, I need to get my aliases right so I sign commits in repos without the pre-commit hooks :)

Harvey, I don't have merge access on this repo, so you'll need to do that.

@markdroth done, I've fixed it so you should be able to merge in the future.


support custom pane specification

Incorporate/enable/test the ability to specify a custom leaflet pane for a source (and it's underlying Overlay/Tile layer).

Ordering WMS layers within a source is discussed in #19
Ordering a WMS source among other leaflet layers is handled by #28 (edit: but see https://github.com/heigeo/leaflet.wms/issues/6#issuecomment-331259309)

Has any improvement on this been made yet? Tried to find a workaround on Stack Exchange but not found a solution. @sheppard did Leaflet.NonTiledLayer work for you?

This is still pending, PRs are welcome.  I don't think it will be a huge change.  I haven't tried Leaflet.NonTiledLayer directly but it appears they accept a pane argument.

As discussed in #19, multiple sources can get out of order when panning and it's not clear that #28 is sufficient to address.  Thus, panes are probably the easiest and most robust way to ensure that layer order remains the same when using multiple sources  (c.f. esri/esri-leaflet#1000).

I ran into the same problem. I want to use a wms source (only one layer) as the basemap but it ends up on top of everything else.
So I think this should be filed as a bug not an enhancement.

For simple use case with a L.WMS.Overlay you may find this simple four-line patch relevant:   https://github.com/GreenInfo-Network/leaflet.wms/blob/gh-pages/src/leaflet.wms.js#L405-L412
This is a patch into the L.WMS.Overlay update() method, to look for a pane option that was passed in, and move the resulting image overlay into the pane.
This does not address the more complex use cases, but hopefully someone using Overlay will find it useful.


Parse sql: no rows in result set error

The UPSERT ONE operation sometimes will return a sql: no rows in result set error.
Like:
err := client.User.
    Create().
    SetAge(30).
    SetName("Ariel").
    OnConflict().
    DoNothing().
    Exec(ctx)

Maybe the error should be parsed by ent like the NotFoundError error ?

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Tech
Version

Go
1.17.?

Ent
0.9.1

Database
Postgresql

Driver
https://github.com/lib/pq

Related: https://github.com/ent/ent/issues/1821.


Removed jcr:content definition while using sync with checkout filter roots

Assuming that we have already checked out page at path
/content/example/demo/a/b/c

the a/.content.xml will have jcr:content with data [jcr:title,...]
the b/.content.xml will have jcr:content with [jcr:title,...]
the c/.content.xml will have jcr:content with [jcr:title,...]

but if we run sh gradlew aemSync -Paem.checkout.filterRoots=[/content/example/demo/a/b/c]
*the a/.content.xml will no longer contain jcr:content and data below but only jcr:content tag
*the b/.content.xml will no longer contain jcr:content and data below but only jcr:content tag
It is needed to somehow prevent such situation to happen. Maybe it is possible by temporarily inserting .vltignore files to parent directories, but that could be dangerous if such files will not be cleaned up properly if sync will fail in the meantime.

implemented and released within 5.0.0


Add last time updated?

Hey! Loving the addon for home assistant was wondering if it would be possible to pull in the last time the price was updated? Not sure were to put this since it's not an "issue" so sorry for that but though it would be a cool addition!

This is already implemented, it's an attribute of the sensor entity.

Ahhh I see it now. I was looking at the home assistant last update which is showing longer, I see the sensor entity now. Thanks for the quick reply!


Alert from Intuit: "Action Required: Reminder that Intuit changes to identity signature keys may impact your application"

According to an alert email I received from Intuit:

In the OpenID Connect flow, after making the call to exchange your auth code for an ID token, your app needs to validate the ID token and verify that the signing authority for responses is from Intuit. The recommended way to check the signature is to scan through the array of keys at oauth.platform.intuit.com/op/v1/jwks for the public key information. The kid value should match the value that was returned in the ID token header.

Is this already handled in this package? If not, will this package be updated for this, or do we have to migrate to a new one?

Hi Json,
Yes, this is already handled in this package.
Thanks,
Rajesh
On Mon, Feb 12, 2024 at 8:48 AM Jason DePeaux @.***>
wrote:

According to an alert email I received from Intuit:
In the OpenID Connect flow, after making the call to exchange your auth
code for an ID token, your app needs to validate the ID token and verify
that the signing authority for responses is from Intuit. The recommended
way
https://app.dp.intuit.com/e/er?s=2332625&lid=34&elqTrackId=abce9ade8711403590505fb65b12467f&elq=f63b6b1bb6644b758efa4a61b6bbc019&elqaid=78&elqat=1
to check the signature is to scan through the array of keys at
oauth.platform.intuit.com/op/v1/jwks for the public key information. The
kid value should match the value that was returned in the ID token header.
Is this already handled in this package? If not, will this package be
updated for this, or do we have to migrate to a new one?
—
Reply to this email directly, view it on GitHub
https://github.com/intuit/oauth-jsclient/issues/154, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABY72WTUJHRPSNEJBOL4NMDYTJBVRAVCNFSM6AAAAABDFBTJKGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEZTANJUGA4TIOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Just for the sake of anyone who might find this issue in a Google search, this deadline has passed, and we've had no issues related to this alert so far.
Thanks!


language code doesn't work

hello,
I tried to call $weather = $owm->getWeather('Mende, Fr', 'metric', $lang); with $lang = 'fr', 'de', same, i always get the result in English.
Certainly a problem of parameter passed in the url because if I directly access to openWeather, I get translated results.

Hi @caBBAlainB,
the lang parameter should be correctly included in the url. Could you please try to add echo $url; here and post the url? Thanks.

Yes,
When when ask for a request with city name, units and language are not well passed:
$owm->getWeather($site->city .',' . $site->postcode, ',' . $site->country->code, 'metric', 'fr');
http://api.openweathermap.org/data/2.5/weather?q=Hures+la+Parade%2C48150&units=,fr&lang=metric&mode=xml&APPID=fr'
It's ok when we make the request with coordinates:
$owm->getWeather(['lat' => $site->latitude, 'lon' => $site->longitude], 'metric', 'fr);
http://api.openweathermap.org/data/2.5/weather?lat=44.221078&lon=3.356774&units=metric&lang=fr&mode=xml&APPID=5df9d03c4c6da5e34fa8368385e2778b
Regards,
Alain

Sorry, I reopen it, but again, I wonder if it comes from your side:
My code is the following one:
Line 389    $curWeather = $owm->getWeather($site->city . ',' . $site->postcode . ',' . $site->country->code, 'metric', 'fr');
Line 390    debug($curWeather->clouds->getDescription());
Line 391    $rawWeather = $owm->getRawWeatherData($site->city . ',' . $site->postcode . ',' . $site->country->code, 'metric', 'fr', '', 'json');
Line 392    debug($rawWeather);
And I inserted a debug($url) in your getRawWeatherData() line 352.
result is:
\vendor\cmfcmf\openweathermap-php-api\Cmfcmf\OpenWeatherMap.php (line 352)
'http://api.openweathermap.org/data/2.5/weather?q=Hures+la+Parade%2C48150%2Cfr&units=metric&lang=fr&mode=xml'
\src\Controller\DeviceconnectionsController.php (line 390)
'clear sky'
\vendor\cmfcmf\openweathermap-php-api\Cmfcmf\OpenWeatherMap.php (line 352)
'http://api.openweathermap.org/data/2.5/weather?q=Hures+la+Parade%2C48150%2Cfr&units=metric&lang=fr&mode=json'
\src\Controller\DeviceconnectionsController.php (line 392)
'{"coord":{"lon":3.43,"lat":44.25},"sys":{"message":0.0184,"country":"France","sunrise":1425881331,"sunset":1425923080},"weather":[{"id":800,"main":"Clear","description":"ensoleillé","icon":"01d"}],"base":"cmc stations","main":{"temp":13.637,"temp_min":13.637,"temp_max":13.637,"pressure":930.1,"sea_level":1038.49,"grnd_level":930.1,"humidity":40},"wind":{"speed":1.06,"deg":355.001},"clouds":{"all":0},"dt":1425901293,"id":0,"name":"Hures-la-Parade","cod":200}
'
As you can see, both urls seem to be correct but the first one returns a not translated result (clear sky and ensoleillé for the second one).
Regards,

So the XML version doesn't properly translate the result? This might be another bug with the Api.

Yes it appears so
 does have the correct translated info

hi, it reproduced, again..
my url :
http://api.openweathermap.org/data/2.5/weather?q=MINSK&lang=ru&appid=*********


-g argument not getting passed to juicer_postprocessing.sh

This is hopefully the last issue I post!
After upgrading to Juicer Tools 2.13.07 (Juicer was pulled from github using git clone), I successfully manage to pick up my process and run all steps until the very last. However, after writing norms, postprocessing fails. It seems -p hg38 is not getting translated to juicer_postprocessing.sh -g hg38.
$> ./opt/scripts/juicer.sh -S final -D $(readlink -f opt) -z $(pwd)/opt/references/hg38.p13.fa.gz -d $(pwd)/L79851_Track-123454/ -p hg38 -t 32
Finished writing norms

real    123m24.622s
user    714m45.769s
sys     18m23.893s
Juicer/opt/scripts/common/juicer_postprocessing.sh: option requires an argument -- g
Usage: Juicer/opt/scripts/common/juicer_postprocessing.sh [-h] -j <juicer_tools_file_path> -i <hic_file_path> -m <bed_file_dir> -g <genome ID>

Hi @sa501428
I'm part of agilly's team who intitiated the issue and i'm also working on implementing the juicer piepline in our lab.
I want to add a new finding to the bug issue he raised which might help in resolving the issue.
I rerun his command with adding -g h38 manually and then the postprocessing starts without any problem and the pipeline is running fine.
Still the function of giving the genome id with the -z command is not working when giving the path as agilly did in his code posted (the reference directory he is pointing to has all the bwa index files)
I also did a rerun with giving the argument -z hg38 which also didn't resolve the problem.


db2 support

@V3lop5 I cleaned up my code today and pushed the code here.
So now you can pull the code , change the db2 config and have a test.
There are some work need your help.

Some tests are still failed, most of them are about foreign key, index.
Some tests are changed or ignored for db2, you may check them again.

Last, wish this pull request helps.

@sola1tmy what version of db2 are you using?
I'm using the docker image ibmcom/db2:<IP_ADDRESS>.
It was a bit tricky to get that docker version finally running.
I received an exception in this org.jetbrains.exposed.DefaultsTest > testDefaultExpressions02 testcase and found the solution here.
Also I had a hard time with that docker image because its takes a lot of time to initialize the database.
I'll create a patch tomorrow.

I suppose the foreign key and index tests fails, because db2 does not allow unique constraints over nullable columns. (For my understanding.)
For example PlayerTable in CreateMissingTablesAndColumnsTests.kt contains
        val username = varchar("username", 10).uniqueIndex().nullable()

and the test org.jetbrains.exposed.sql.tests.shared.ddl.CreateMissingTablesAndColumnsTests#explicitFkNameIsExplicit fails with
com.ibm.db2.jcc.am.SqlSyntaxErrorException: DB2 SQL Error: SQLCODE=-542, SQLSTATE=42831, SQLERRMC=USERNAME, DRIVER=4.31.10
org.jetbrains.exposed.exceptions.ExposedSQLException: com.ibm.db2.jcc.am.SqlSyntaxErrorException: DB2 SQL Error: SQLCODE=-542, SQLSTATE=42831, SQLERRMC=USERNAME, DRIVER=4.31.10
SQL: [ALTER TABLE PLAYER ADD CONSTRAINT PLAYER_USERNAME_UNIQUE UNIQUE (USERNAME)]

https://www.ibm.com/docs/en/db2/11.5?topic=constraints-unique
We might need to change unique constraints to unique indices.
As far as I know from AS/400 (or i5) this should allow unique over nullable columns.

@sola1tmy

In my impression, It's a wrong way, because the foreign key needs unique constraint.（I'm not very sure)

Thats absolutly true! We might spoke past each other.
See test org.jetbrains.exposed.sql.tests.shared.ddl.CreateMissingTablesAndColumnsTests#explicitFkNameIsExplicit
https://www.sqlerror.de/db2_sql_error_-542_sqlstate_42831.html
For my understanding only columns defined as NOT NULL can be used for PRIMARY KEY, UNIQUE CONSTRAINT or PARENT KEY (foreign key).
So the Exposed definition is not allowed for db2:
val username = varchar("username", 10).uniqueIndex().nullable()

I guess we should disable these tests?

@V3lop5
My bad, I didn't describe it clearly.

val username = varchar("username", 10).uniqueIndex().nullable()

This code you said can be fixed by unique index.But you will more failed tests because you can't create table with foreign key without a unique constraint.
You can try override createIndex like this
override fun createIndex(index: Index): String {
    if (index.unique) {
        return super.createIndex(index.copy(unique = false)).replace("CREATE INDEX", "CREATE UNIQUE INDEX")
    }
    return super.createIndex(index)
}

Then run this test
org.jetbrains.exposed.sql.tests.shared.entities.EntityTests#callLimitOnRelationDoesntMutateTheCachedValue
Now, you should get the error
 DB2 SQL Error: SQLCODE=-573, SQLSTATE=42890, SQLERRMC=FK_POSTS_CATEGORY__UNIQUEID;INST.CATEGORIES, DRIVER=4.31.10
Now we know, UNIQUE CONSTRAINT disable uniqueIndex().nullable() and UNIQUE INDEX disable table create with foreign key. But in exposed we can not choose unique constraint or index right now.
Maybe we should throw an exception when people want to use  uniqueIndex().nullable()  for db2.

@V3lop5
My bad, I didn't describe it clearly.

val username = varchar("username", 10).uniqueIndex().nullable()

This code you said can be fixed by unique index.But you will more failed tests because you can't create table with foreign key without a unique constraint.
You can try override createIndex like this
override fun createIndex(index: Index): String {
    if (index.unique) {
        return super.createIndex(index.copy(unique = false)).replace("CREATE INDEX", "CREATE UNIQUE INDEX")
    }
    return super.createIndex(index)
}

Then run this test
org.jetbrains.exposed.sql.tests.shared.DDLTests#testCompositeFKReferencingUniqueIndex
Now, you should get the error
DB2 SQL Error: SQLCODE=-573, SQLSTATE=42890, SQLERRMC=FK_TABLEB_ID_A_ID_B__ID_A_ID_B;INST.TABLEA, DRIVER=4.31.10
Now we know, UNIQUE CONSTRAINT disable uniqueIndex().nullable() and UNIQUE INDEX disable table create with foreign key. But in exposed we can not choose UNIQUE CONSTRAINT or UNIQUE INDEX right now.
Maybe we should throw an exception when people want to use  uniqueIndex().nullable()  for db2.

@V3lop5
My bad, I didn't describe it clearly.

val username = varchar("username", 10).uniqueIndex().nullable()

This code you said can be fixed by unique index.But you will have more failed tests because you can't make a foreign key constraint column reference a nullable unique index column.（Sorry for my bad english, maybe try the test below is more clear)
You can try override createIndex like this
override fun createIndex(index: Index): String {
    if (index.unique) {
        return super.createIndex(index.copy(unique = false)).replace("CREATE INDEX", "CREATE UNIQUE INDEX")
    }
    return super.createIndex(index)
}

Then run this test
org.jetbrains.exposed.sql.tests.shared.DDLTests#testCompositeFKReferencingUniqueIndex
Now, you should get the error
DB2 SQL Error: SQLCODE=-573, SQLSTATE=42890, SQLERRMC=FK_TABLEB_ID_A_ID_B__ID_A_ID_B;INST.TABLEA, DRIVER=4.31.10
It seems that UNIQUE CONSTRAINT disable uniqueIndex().nullable() and UNIQUE INDEX disable FK reference to the column. But in exposed we can not choose UNIQUE CONSTRAINT or UNIQUE INDEX right now.
Maybe we should change the Column.uniqueIndex()  to Column.uniqueConstraint(),and keep the Table.uniqueIndex, so that they can work differently.But this idea have break changes.

I prefer the second option.
Sorry, got no time in the past two days. I ll merge this PR tomorrow & add my changes.


Index copy

Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

Fixes SNOW-1458120

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code

[ ] If this test skips Local Testing mode, I'm requesting review from @snowflakedb/local-testing

[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency
[ ] If this is a new feature/behavior, I'm adding the Local Testing parity changes.

Please describe how your code solves the related issue.

Implements Index.copy

Can we have for the next PR pls a JIRA ticket + a meaningful description @sfc-gh-samdesai1 ? This helps us troubleshoot in case.


Added call to free on layerInstance.intGridCsv when len != 0

LayerInstance.deinit() doesn't free int grids and only accounts for TileInstances, this fixes that memory leak.

Thanks for catching this! I should really get some tests made to detect these sorts of bugs.


Use firmware Node modules ABI version in binary module resolution. Fixes gh-978

Presently, only the latest versions of the existing precompiled modules are available for use with node 6.5.0 (because why would we try to precompile older versions?)
Smoke Test
The goal is to confirm that the new naming is being used when resolving precompiled binary modules.

Check out this branch

cd ~/.tessel/binaries && rm *.tgz

cd ~ && mkdir t2-978 && cd t2-978 && t2 init

npm install bufferutil

Edit index.js to:
var bu = require('bufferutil').BufferUtil;
var a = new Buffer([0, 1, 2, 3]);
var b = new Buffer([4, 5, 6, 7]);
var c = Buffer.alloc(8, 0);

bu.merge(c, [a, b]);

console.log(c);

(If that fails with TypeError: Buffer.alloc is not a function, then you'll need to t2 update)

t2 run index.js

Expected result:
$ t2 run index.js
INFO Looking for your Tessel...
INFO Connected to sonny.
INFO Building project.
INFO Writing project to RAM on sonny (39.936 kB)...
INFO Deployed.
INFO Running index.js...
<Buffer 00 01 02 03 04 05 06 07>

Then cd ~/.tessel/binaries && ls -la, which should look approximately like this:
rwaldron in ~/.tessel/binaries
$ ls -la
.
└── [drwxr-xr-x rwaldron staff     136 Sep 15 13:59]  bufferutil-1.2.1-Release-node-v46-linux-mipsel/

1 directory, 0 files

Signed-off-by: Rick Waldron<EMAIL_ADDRESS>

Just went through the smoke test:
hipsterbrown:t2-978 $ t2 run index.js
INFO Looking for your Tessel...
INFO Connected to tessel-router.
INFO Building project.
INFO Writing project to RAM on tessel-router (39.936 kB)...
INFO Deployed.
INFO Running index.js...
<Buffer 00 01 02 03 04 05 06 07>
hipsterbrown:t2-978 $ ls -la ~/.tessel/binaries/
total 0
drwxr-xr-x  7 hipsterbrown  staff  238 Sep 15 14:44 .
drwxr-xr-x  6 hipsterbrown  staff  204 Jul 18 23:23 ..
drwxr-xr-x  3 hipsterbrown  staff  102 Sep 15 14:44 bufferutil-1.2.1-Release-node-v46-linux-mipsel
drwxr-xr-x  2 hipsterbrown  staff   68 Jul 18 23:23 release-1.1.1-Release
drwxr-xr-x  3 hipsterbrown  staff  102 Jul 18 23:23 serialport-3.1.2-Release
drwxr-xr-x  4 hipsterbrown  staff  136 Sep  1 17:49 sqlite3-3.1.4-Release
drwxr-xr-x  3 hipsterbrown  staff  102 Jul 18 23:23 utf-8-validate-1.2.1-Release
hipsterbrown:t2-978 $

LGTM :+1: :clap:

@HipsterBrown thanks!
I see you didn't follow the first step ;)

@rwaldron I only deleted the bufferutil directory in ~/.tessel/binaries. I didn't want to get rid of my working sqlite directory yet. :laughing:


Better error message when running react-native run-android in linux and gradlew is not executable

Description
I cloned a git repo, then installed dependencies..  then ran react-native run-android
Building and installing the app on the device (cd android && ./gradlew installDebug)...
Could not install the app on the device, read the error above for details.
Make sure you have an Android emulator running or a device connected and have
set up your Android development environment:
https://facebook.github.io/react-native/docs/android-setup.html

Took me a while to figure out android/gradlew was not executable ... maybe the error message could be improved..
Additional Information

React Native version: 0.39.0
Platform: Android
Operating System: Linux

We're cutting down on the number of outstanding issues, in order to allow us to focus. I'm closing this issue because it has been open for over 60 days with no activity. If you think it should still be opened let us know why. PRs are always welcome.


bug: remapped i in visual mode has delay

Did you check docs and existing issues?

[X] I have read all the vscode-neovim docs
[X] I have searched the existing issues of vscode-neovim
[X] I can reproduce the issue with an empty init.vim/init.lua and no other vscode extensions (when applicable)

Neovim version (nvim -v)
NVIM v0.10.0-dev-1285+g032ab9d3e
Operating system/version
macOS 13.4.1
Describe the bug
When pressing i which is remapped in visual mode, there is a delay
Steps To Reproduce
vnoremap i :echo("hello world!")<CR>
Expected Behavior
No delay

I think this is XY problem. Try using <CMD> instead of :

I think this is XY problem. Try using  instead of :

There is delay regardless of whether :, <CMD>, or Lua function

@llllvvuu  Please execute the command :vmap i and check for any other mappings.

@llllvvuu Please execute the command :vmap i and check for any other mappings.

Looks like this is the issue (text objects), thank you.


Documentation Issue: Docs mention "audioOutput" prop, but not configured in v6.0.0 beta.

No reproducible code. Title states details.
Tried using the audioOutput prop, as it would serve a great need in my application, but the Video component does not support it.
Solution:
Either:
A.) Add the feature to the Video component,
B.) Update docs to reflect actuality.
 The react-native-video component is working excellently. Want to thank the open source community for this great package..
Thanks,
Chris

Fixed, will be available in 6.0.0-beta.4


Clang carried by LLVM will cause "could not find rodata struct member" on M3 MacOs to occur when debugging with dlv

When I install and configure LLVM17, I start dlv debugging and crash directly. The prompt is as follows
could not find rodata struct member

dlv version
Delve Debugger
Version: 1.22.1
Build: $Id: 0c3470054da6feac5f6dcf3e3e5144a64f7a9a48 $

go version
go version go1.21.4 darwin/arm64

mac version: MacOS 14.2 M3 MacBook Pro (2023)
uname -a
Darwin ... 23.2.0 Darwin Kernel Version 23.2.0: Wed Nov 15 21:54:55 PST 2023; root:xnu-10002.61.3~2/RELEASE_ARM64_T8122 arm64

the custom installation of clang (built from upstream LLVM) will make error
clang -v
Homebrew clang version 17.0.6
Target: arm64-apple-darwin23.2.0
Thread model: posix
InstalledDir: /opt/homebrew/opt/llvm@17/bin

Add the original clang environment to the debugging settings of vscode,  can debug normally
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "debug！",
            "type": "go",
            "request": "launch",
            "mode": "auto",
            "program": "${fileDirname}",
            "env": {
                "CC": "/usr/bin/cc",
                "CXX": "/usr/bin/c++",
            }
        }
    ]
}

Related issue:https://github.com/go-delve/delve/issues/3701

dlv may not be applicable for llgo. We should probably use the llvm debugging toolchain instead. Anyway, it may not be the best time to consider this issue at the moment.

Removing Xcode fixed the problem (Temporarily), others says that downgrade 16 to 15 o 14 version of xcode also works.


Add SM3 implementation in RISC-V Zksh inline asm

This PR adds the support of SM3 for RISC-V platform with Zksh capability. Note that this works for both RV32 and RV64
The RV64 infra has been merged (#18275). RV32 infra will be contributed in further PRs (see #18285). So this PR is ready to merge.
This feature is enabled when using GNU compiler with -march=rv64gc_zksh.
This PR borrowed the idea from #16710 and #16638
Correctness has been verified as both C and inline asm version outputed the same digest.
Benchmark of openssl speed -evp sm3
With the setup in https://gist.github.com/ZenithalHourlyRate/7b5175734f87acb73d0bbc53391d7140
# pure C
sm3                 97.79k      258.58k      578.42k      969.12k     1223.19k     1199.24k
# with zksh asm
sm3                142.48k      404.12k      937.73k     1443.19k     1707.36k     1742.17k

Discussion on runtime selection
This inline asm feature is only enabled at compile time (the same way as #16710 and #16638), and can not be enabled at runtime with OPENSSL_riscvcap as #17640, #18197 and #18285.
If we implement runtime selection in current sm3_local.h, then we need to judge RISCV_HAS_ZKSH before each P0/P1, which is not acceptable for performance.
Another way is to implement rvi_zksh_{init,update,finish} along side current ossl_sm3_{init,update,finish} in providers/implementations/digests/sm3_prov.c. However, re-implementing all the functions just for one instruction is not maintainable. Also, there are two ways to implement it: C with inline asm, then we must use -march="zksh" when compiling even for pure C version and it is not portable; or perlasm, and we need to rewrite the whole thing in asm and need .word magic for compiler compatibility.
It seems there is no good way to implement it with small effort.
Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

The new commit add ROTATE inline asm support in md32_common. This benefits a lot of hash functions (e.g. SM3, SHA256, MD4, MD5) for platform without Zk (special SM3/SHA256 inst) but with Zbb.
Note that there are many ciphers using ROTATE around the code base (e.g. AES, CHACHA, ). They do not use the md32_common ROTATE. I will send other PR on this for them.
https://github.com/openssl/openssl/blob/cb2764f2a8165421dc5ab52159af99cbf766fa2c/crypto/aes/aes_core.c#L2000
https://github.com/openssl/openssl/blob/cb2764f2a8165421dc5ab52159af99cbf766fa2c/crypto/chacha/chacha_enc.c#L25
https://github.com/openssl/openssl/blob/cb2764f2a8165421dc5ab52159af99cbf766fa2c/crypto/des/des_local.h#L115
The benchmark now
# with zksh and zbkb (rorwi)
sm3                191.78k      569.17k     1357.40k     2079.74k     2457.60k     2495.83k
# with zbkb only (note that P0/P1 can also leverage ROTATE in rorwi)
sm3                150.81k      450.50k     1078.97k     1659.29k     1972.66k     2004.31k

Testing
The following testing has been done：
This PR was built and run successfully under qemu.
Here are some details:
The qemu was built with
-cpu rv64,zba=true,zbb=true,zbc=true,zbs=true,zbkb=true,zbkc=true,zbkx=true,\
zk=true,zkn=true,zknd=false,zkne=true,zknh=true,zkr=true,zks=true,zksed=true,zksh=true,zkt=true

then run the  test :
export CC="gcc -march=rv64gc_zbkb_zksh"
./Configure linux64-riscv64 no-afalgeng
make
make test

And here is the result
Result: PASS
All tests successful.
Files=248, Tests=3385, 7081 wallclock secs (115.03 usr 16.62 sys + 5781.35 cusr 1337.34 csys = 7250.34 CPU)

Benchmark of openssl speed -evp sm3

RV32 part of it
# pure C
sm3                116.59k      322.20k      738.98k     1159.62k     1408.26k     1425.41k
# zksh inline asm with zbkb (rori)
sm3                237.76k      688.12k     1527.21k     2199.19k     2495.96k     2506.75k

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This pull request is ready to merge

Merged to master.

Waiting on the repo to be unfrozen....

I am not sure that ROTATE should be presented as intrinsic.
For example, check here:
https://github.com/gcc-mirror/gcc/blob/00b26ae2d1c0e1d6cc990862f54dc3dbcbeed913/gcc/config/riscv/bitmanip.md#L180-L186
There are known issues:

Emit condition is "TARGET_ZBB", which does not (and should not) imply "TARGET_ZBKB".
GCC authors did not fix a known issue of emitting too big immediate shifts as li + small shift.

This code should be used in compiler in this file instead of all ror[i][w] stuff:

(define_insn "rotrsi3"
  [(set (match_operand:SI 0 "register_operand" "=r")
	(rotatert:SI (match_operand:SI 1 "register_operand" "r")
		     (match_operand:QI 2 "arith_operand" "rI")))]
  "TARGET_ZBB || TARGET_ZBKB"
  { return TARGET_64BIT ? "ror%i2w\t%0,%1,%2" : "ror%i2\t%0,%1,%2"; }
  [(set_attr "type" "bitmanip")])

(define_insn "rotrdi3"
  [(set (match_operand:DI 0 "register_operand" "=r")
	(rotatert:DI (match_operand:DI 1 "register_operand" "r")
		     (match_operand:QI 2 "arith_operand" "rI")))]
  "TARGET_64BIT && (TARGET_ZBB || TARGET_ZBKB)"
  "ror%i2\t%0,%1,%2"
  [(set_attr "type" "bitmanip")])

(define_insn "rotrsi3_sext"
  [(set (match_operand:DI 0 "register_operand" "=r")
	(sign_extend:DI (rotatert:SI (match_operand:SI 1 "register_operand" "r")
				     (match_operand:QI 2 "register_operand" "rI"))))]
  "TARGET_64BIT && (TARGET_ZBB || TARGET_ZBKB)"
  "ror%i2w\t%0,%1,%2"
  [(set_attr "type" "bitmanip")])

(define_insn "rotlsi3"
  [(set (match_operand:SI 0 "register_operand" "=r")
	(rotate:SI (match_operand:SI 1 "register_operand" "r")
		   (match_operand:QI 2 "arith_operand" "rI")))]
  "TARGET_ZBB || TARGET_ZBKB"
  {
    if (CONST_INT_P(operands[2])) {
      operands[2] = GEN_INT(32 - INTVAL(operands[2]));
      return TARGET_64BIT ? "roriw\t%0,%1,%2" : "rori\t%0,%1,%2";
    }
    return TARGET_64BIT ? "rolw\t%0,%1,%2" : "rol\t%0,%1,%2";
  }
  [(set_attr "type" "bitmanip")])

(define_insn "rotldi3"
  [(set (match_operand:DI 0 "register_operand" "=r")
	(rotate:DI (match_operand:DI 1 "register_operand" "r")
		   (match_operand:QI 2 "arith_operand" "rI")))]
  "TARGET_64BIT && (TARGET_ZBB || TARGET_ZBKB)"
  {
    if (CONST_INT_P(operands[2])) {
      operands[2] = GEN_INT(64 - INTVAL(operands[2]));
      return "rori\t%0,%1,%2";
    }
    return "rol\t%0,%1,%2";
  }
  [(set_attr "type" "bitmanip")])

(define_insn "rotlsi3_sext"
  [(set (match_operand:DI 0 "register_operand" "=r")
	(sign_extend:DI (rotate:SI (match_operand:SI 1 "register_operand" "r")
				   (match_operand:QI 2 "arith_operand" "rI"))))]
  "TARGET_64BIT && (TARGET_ZBB || TARGET_ZBKB)"
  {
    if (CONST_INT_P(operands[2])) {
      operands[2] = GEN_INT(32 - INTVAL(operands[2]));
      return "roriw\t%0,%1,%2";
    }
    return "rolw\t%0,%1,%2";
  }
  [(set_attr "type" "bitmanip")])

I am too lazy to commit to GCC because of their bureaucracy but still I think that proper ROTATE is more a compiler problem then openssl's one.
Of course this is just my opinion.

@ZenithalHourlyRate do you have any comments on that?

It would be better done by the compiler, but if they aren't doing it...

@ZenithalHourlyRate do you have any comments on that?

I'm just following the convention in OpenSSL, like https://github.com/openssl/openssl/blob/5317b6ee1fc3db20de5976fbb46cc49a45c0768a/crypto/aes/aes_x86core.c#L101-L109 and https://github.com/openssl/openssl/blob/5317b6ee1fc3db20de5976fbb46cc49a45c0768a/crypto/rc5/rc5_local.h#L136-L149

It would be better done by the compiler, but if they aren't doing it...

I do agree with pauli and I also prefer the toolchain way, but changes in the toolchain still have a long way to go to hit end users like linux distributions (e.g. https://github.com/openssl/openssl/pull/17640#issuecomment-1036454264). I think adding it for now and cleaning up them when the toolchain is ready (I'm afraid there is no timing plan on this) is an acceptable solution for end users and the toolchain.

Merged to master branch. Thank you for your contribution.

Also, there are two ways to implement it

Just a side note:
C with inline asm can be used even without -march using this. This allows to avoid gcc (binutils) from knowing instruction names but still use "=r"(out) : "r"(in) syntax (i.e. not calculate the whole .word with the register numbers).
All the zk-ext non-CSR insns are of r or i type so it should not require any additional knoweledge to that riscv-compiler already has.


cilium-operator.Dockerfile: create /tmp directory

This is a hack to avoid crashes introduced by glog, which is used by the
Kubernetes client libraries within cilium-operator. This crash is caused
because by default, glog assumes that the directory /tmp exists, and tries
to log to it. If said directory does not exist, glog has the unfortunate
behavior of crashing cilium-operator. The /tmp directory is created via
using multi-stage Docker builds to grab the /tmp directory from an Alpine
Docker image, as mkdir is not available within the scratch image which
is used for packaging cilium-operator.
Related-to: #7006
Signed-off by: Ian Vernon<EMAIL_ADDRESS>I've deployed an image with this fix on a cluster which was hitting this problem frequently, so I am waiting until that image does not crash after some time to establish whether this hack really works.
I manually tested this by applying the following diff:
diff --git a/operator/main.go b/operator/main.go
index e71b83935..c4b505b8e 100644
--- a/operator/main.go
+++ b/operator/main.go
@@ -17,6 +17,7 @@ package main
 import (
        "flag"
        "fmt"
+       "io/ioutil"
        "os"
        "os/signal"
        "syscall"
@@ -130,6 +131,12 @@ func initConfig() {
 func runOperator(cmd *cobra.Command) {
        logging.SetupLogging([]string{}, map[string]string{}, "cilium-operator", viper.GetBool("debug"))

+       d1 := []byte("hello\ngo\n")
+       err := ioutil.WriteFile("/tmp/dir", d1, 0644)
+       if err != nil {
+               log.Fatal("error writing file: %s", err)
+       }
+
        // Make sure glog (used by the client-go dependency) logs to stderr, as it
        // will try to log to directories that may not exist in the cilium-operator
        // container and cause the cilium-operator to exit.

And building the image:
$ docker build . -f cilium-operator.dockerfile

I then ran the image to see whether it would get past the creation of the file with the new patch to the Dockerfile, and it did:
$ docker run aa5bf0fa6097
level=fatal msg="Unable to setup kvstore" address= error="unknown key-value store type \"\". See cilium.link/err-kvstore for details" kvstore= subsys=cilium-operator

The fatal error above is because I ran the image without providing any kvstore configuration, which is expected. That code is executed after the creation of the file in /tmp so the creation of the file succeeded.
Building the image without the change to the Dockerfile, and the following occurred:
$ docker run 417ab42acd76
level=fatal msg="error writing file: open /tmp/dir: no such file or directory" subsys=cilium-operator

This change is

Coverage decreased (-0.03%) to 42.926% when pulling 4a72ec001aca842518d939ff9adaffb4f78f78f2 on pr/ianvernon/create-tmp-directory-for-glog-logs-cilium-operator into 6a8cb1f84a1444c68d87e2740b54597f10832eb0 on master.

@ianvernon this works
diff --git a/operator/main.go b/operator/main.go
index e71b83935..701145099 100644
--- a/operator/main.go
+++ b/operator/main.go
@@ -35,6 +35,7 @@ import (
        "github.com/sirupsen/logrus"
        "github.com/spf13/cobra"
        "github.com/spf13/viper"
+       "k8s.io/klog"
 )

 var (
@@ -110,6 +111,12 @@ func init() {
        option.BindEnv(option.DisableCiliumEndpointCRDName)

        viper.BindPFlags(flags)
+
+       // Make sure glog (used by the client-go dependency) logs to stderr, as it
+       // will try to log to directories that may not exist in the cilium-operator
+       // container and cause the cilium-operator to exit.
+       klog.InitFlags(nil)
+       flag.Set("logtostderr", "true")
 }

 // initConfig reads in config file and ENV variables if set.
@@ -130,11 +137,6 @@ func initConfig() {
 func runOperator(cmd *cobra.Command) {
        logging.SetupLogging([]string{}, map[string]string{}, "cilium-operator", viper.GetBool("debug"))

-       // Make sure glog (used by the client-go dependency) logs to stderr, as it
-       // will try to log to directories that may not exist in the cilium-operator
-       // container and cause the cilium-operator to exit.
-       flag.Set("logtostderr", "true")
-
        log.Infof("Cilium Operator %s", version.Version)

        if err := kvstore.Setup(kvStore, kvStoreOpts); err != nil {

@aanm how did you validate that this works?

@aanm how did you validate that this works? Looks promising, though!

add a klog.Warning("hello") in the code

Moving back to wip, as @aanm's fix is the correct one. Validated it locally.

@aanm yes, that is true, but that means that we may be filling up /tmp with logs from K8s over time, and we may be hiding errors from the logs we get from cilium from the K8s libraries.

test-me-please

Build timed out for all three stages:
https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/9859/
The build occurred on jenkins-fixed-4: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/9859/execution/node/3/log/
This node hit issues in other builds, so I'm going to mark it as offline and start the build again.

test-me-please


About cogdl's reproducibility

As mentioned in an issue of PyG,

I evaluated this problem and see perfect reproducibility on CPU when using the seed_everything code from you. However, on GPU, we can not guarantee determinism because we make heavy use of scatter ops to implement Graph Neural Networks which are non-deterministic by nature on GPU.

PyG's reproducibility is limited by the presence of the torch_scatter operation.
Will the reproducibility of cogdl be better?
I think this is an important question because it is closely related to the ablation analysis.

Hi @4fee8fea,
Thanks for your attention.
We try our best to (re-)implement GNN models in our library to reproduce similar results to the original results.
However, all GNN libraries (e.g., PyG, DGL, CogDL) cannot provide the exact reproducibility since the scatter operators on GPU are not guaranteed with determinism by nature. In reality, we can run the same experiments multiple times with different seeds (e.g., 10 seeds) to obtain a relatively accurate metric, which is enough for comparison and ablation analysis.

Hi @cenyk1230,
Thanks for your prompt reply and advice!
I noticed that taking ten different random seeds to report the average is also a recommended practice in OGB.
My experiments use the fixed seed (e.g., taking seed as 42) and run ten times within each experiment. Perhaps this is equivalent to the practice above.
However, the average results were also different each time under this setting.
I see now that it comes down to torch_scatter, which is somewhat frustrating.
Thanks for the reply, though, and all the best to you!


Caching

Browser & Node

Realised that caching is really something that , although it could be added, doesn't make sense to add here, as Tchef would have to maintain it's own cache, and this really should be one layer of abstraction higher, meaning the project using Tchef should decide on the cache and maintain the cache.
So, for now, closing this.
Will be working on a different project that incorporates caching on top on Tchef.


ciao-cert: Reduce cyclomatic complexity

Fixes #153
Signed-off-by: Samuel Ortiz<EMAIL_ADDRESS>

This was merged.


Update zio-kafka to 2.7.5

About this PR
📦 Updates dev.zio:zio-kafka from 2.7.3 to 2.7.5
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (2.7.3).
You might want to review and update them manually.
build.sbt

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-kafka" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio", artifactId = "zio-kafka" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Supersedes #1436

Still the case https://github.com/laserdisc-io/tamer/pull/1436#issuecomment-2099774474


CycleError is still here

Describe the bug
The is still some CycleErrors when using embedded loops. Especially, it looks like the host.loop_position is shared across all hosts.
To Reproduce
I can consistently reproduce the issue with following simple script and 2 hosts.
import random
import time

from pyinfra import host
from pyinfra.operations import server

for i in host.loop(range(0, 3)):
    for j in host.loop(range(0, 3)):
        print(f"{host.name} {host.loop_position}")
        server.shell(name=f"ls A", commands="ls")

        if random.randint(0, 1) == 0:
            time.sleep(1)
            server.shell(name=f"ls B", commands="ls")

        server.shell(name=f"ls C", commands="ls")

The print will show something going wrong in the handling for host.loop_position.
Meta
v2.5.2
Output of previous example shows issues in host.loop_position.
--> Preparing Operations...
    Loading: tasks/loop_test.py
host08 [0, 0]
host08 [0, 1]
host08 [0, 2]
host07 [0, 2, 0, 0]
host07 [0, 2, 0, 1]
host08 [0, 2, 1, 0]
host08 [0, 2, 1, 1]
host07 [0, 2, 1, 2]
host07 [0, 2, 1, 0]
host07 [0, 2, 1, 1]
host08 [0, 2, 1, 2]
host07 [0, 2, 1, 2]
host08 [0, 2, 2, 0]
host08 [0, 2, 2, 1]
host07 [0, 2, 2, 0]
host07 [0, 2, 2, 1]
host08 [0, 2, 2, 2]
    [host08] Ready: tasks/loop_test.py
host07 [0, 2]
    [host07] Ready: tasks/loop_test.py

--> Proposed changes:
    Groups: * / *
    [host08]   Operations: 22   Change: 22   No change: 0
    [host07]   Operations: 23   Change: 23   No change: 0

--> Beginning operation run...
Cycle detected in operation ordering DAG.
    Error: ('nodes are in a cycle', ['16914cca7f66c3611e9d2a416a7d621a277ee2c3', 'f4a17e92c61121c85c77d34bc15544d0679d5c50', '30096631874118bdb1cfbeaa427c57b569521c25', '7fa4d2f28e0ae70166e7b1a885755610166d49f2', '1552cc4ed1d48843b07f6e66ac114c6dfee0febe-0', '1e668c25af2946343637a2a0ee48d1b3dbec151d-0', '14a0639245fafec20a20e3c702021c28baca8a49', '5746348464227d74adbd59e0b377451a397f00b8', 'bbb49c1789858e4aafb6f0e471a1c2658512f809', '8eeaf6fd0106f688e69270ab5110c006b9dfa894', '75605df4dcce58cd84e134dd1fbbcc4f87c782ba', '67d0504000be158e31955ca4433465ff6fbb549c', '6d7d88a24e3b87c7d02003429c70be1a8f3a15ba', '12a41289448f557d0cd296011d41a36e3fd73bb2', '47155dc5a19d4758dac71657aac9458106fa8ecc', '5c96e4db6ad35ebd092fb183629e0752f9211fce', '16914cca7f66c3611e9d2a416a7d621a277ee2c3'])

    This can happen when using loops in operation code, please see: https://docs.pyinfra.com/en/latest/deploy-process.html#loops-cycle-errors

Gah this is a mutable class attribute 🤦 - https://github.com/Fizzadar/pyinfra/commit/6dc9fb8cee693e73a2c077be70fd0d7fcd983193. Will roll fix in 2.5.3 shortly.


Followed recipients query fix

https://www.pivotaltracker.com/story/show/188016941

Add query for Patient resources for patient(s) whose general practitioner is the user.  This may be the case where the patient is assigned the practitioner but doesn't have active careplan,e.g. careplan revoked.
Bump up _count, pagination may be a concern for large number of patients the user is following.

@achen2401  is this somewhere that @sierramatice  can test it?

@mcjustin this can be tested on Isacc shared dev instance if it is merged. Should I merge it?

@mcjustin this can be tested on Isacc shared dev instance if it is merged. Should I merge it?

Sounds good, since @ivan-c  and I approved it.


[mixcloud] Failed to parse JSON

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.08.13. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.08.13
[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--extract-audio', '--audio-format=mp3', 'https://www.mixcloud.com/zedsdead/zeds-dead-diplo-friends-mix-2016',
 '-v']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2017.08.13
[debug] Python version 3.4.4 - Windows-8.1-6.3.9600
[debug] exe versions: ffmpeg 2.8.4, ffprobe 3.3.2
[debug] Proxy map: {}
[mixcloud] zedsdead-zeds-dead-diplo-friends-mix-2016: Downloading webpage
ERROR: zedsdead-zeds-dead-diplo-friends-mix-2016: Failed to parse JSON  (caused by ValueError('Expecting value: line 1 column 1 (char 0)',
)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sur
e to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpc3a66tbr\build\youtube_dl\extractor\common.py", line 676, in _parse_j
son
  File "C:\Python\Python34\lib\json\__init__.py", line 318, in loads
  File "C:\Python\Python34\lib\json\decoder.py", line 343, in decode
  File "C:\Python\Python34\lib\json\decoder.py", line 361, in raw_decode
ValueError: Expecting value: line 1 column 1 (char 0)
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpc3a66tbr\build\youtube_dl\extractor\common.py", line 676, in _parse_j
son
  File "C:\Python\Python34\lib\json\__init__.py", line 318, in loads
  File "C:\Python\Python34\lib\json\decoder.py", line 343, in decode
  File "C:\Python\Python34\lib\json\decoder.py", line 361, in raw_decode
ValueError: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpc3a66tbr\build\youtube_dl\YoutubeDL.py", line 776, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpc3a66tbr\build\youtube_dl\extractor\common.py", line 433, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpc3a66tbr\build\youtube_dl\extractor\mixcloud.py", line 90, in _real_e
xtract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpc3a66tbr\build\youtube_dl\extractor\mixcloud.py", line 70, in _decryp
t_play_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpc3a66tbr\build\youtube_dl\extractor\common.py", line 680, in _parse_j
son
youtube_dl.utils.ExtractorError: zedsdead-zeds-dead-diplo-friends-mix-2016: Failed to parse JSON  (caused by ValueError('Expecting value:
line 1 column 1 (char 0)',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtub
e-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description of your issue, suggested solution and other information
When trying to download this sound file it is throwing an exception while parsing JSON for the file.  I had another issue for this but it was closed for using an outdated version - which it wasn't.  I've updated to 8.13 and it continues to happen.  @dstftw made a change (da20951a57bddd4a0102cd776ff93a2adc6db77d) three days ago but it's not available through the self updater yet.

It's not yet released.

@dstftw - verified it's working against master - thanks so much for your help on this project.  :1st_place_medal:


[1.11.2] Logistical Transporters Server Crash

While this may have to do more with mod interaction, when the Logistical Transporters are connected with a Crucible Inserter from Sky Resources, the server crashes. By the server crash log I can tell that i has to do with the way the transporter works, rather than the inserter.
https://pastebin.com/njYRHXZM

Not sure how this could be occurring on my end but I added a check to be safe. Should be fixed in next release.

Well piping systems from other mods don't crash it.

If you can give me an exact reproduction scenario with a picture or two I could take a look again.


Visual issues on Select component

Component's height ignores being out of view. A possible solution might be to check available height via JS. But issue is fixable on users' end, using custom css. ( As a side issue, you have !important here, which seems to be unnecessary, and forces user to use !important as well)

Text overflow is ignored when select is focused and options list is closed as seen here:

@nikoTM Please check latest release<EMAIL_ADDRESS>and<EMAIL_ADDRESS>The !important is removed and the text overflow is fixed.
I dont think this fully addressed the first point. What would be nice is to have popover above the input (which is possible when mdl-popover has bottom: 22px;) however I was not able to come up with good way to toggle this properly based on position and preserve proper behaviour. Here is some code I tried in popover:
private show(event: Event) {
    event.stopPropagation();
    this.updateDirection(event);
    this.isVisible = true;
}

private updateDirection(event: Event) {
    const parentRect = document.body.getBoundingClientRect();
    const targetRect = (<HTMLElement>event.target).getBoundingClientRect();
    const targetHeight = (<HTMLElement>event.target).offsetHeight;
    const height = this.elementRef.nativeElement.offsetHeight;

    const bounds = {
        top: 0,
        bottom: parentRect.height,
    };
    const spaceAvailable = {
        top: targetRect.top - bounds.top,
        bottom: bounds.bottom - (targetRect.top + targetRect.height)
    };

    this.directionUp = spaceAvailable.bottom < height;

    console.log('spaceAvailable', spaceAvailable);
    console.log('height', height);
    console.log('targetHeight', targetHeight);
    console.log('this.directionUp', this.directionUp);
}

On top of that offsetHeight does not work with mdl-popover display: none;, but when its visibility: hidden; the is issue with invisible div expanding the view.

@tb so I have looked into it. This partially solves the issue
private show(event: Event) {
        event.stopPropagation();
        this.isVisible = true;
        setTimeout(() => {
          this.updateDirection(event);
        }, 10);
    }

    private updateDirection(event: Event) {
      const nativeEl = this.elementRef.nativeElement;
      const targetRect = (<HTMLElement>event.target).getBoundingClientRect();
      const viewHeight = window.innerHeight;
      const height = nativeEl.offsetHeight;

      const spaceAvailable = {
          top: viewHeight - targetRect.top,
          bottom: viewHeight - targetRect.bottom
      };

      this.directionUp = spaceAvailable.bottom < height;

    }

Things that need to be improved here:

setTimeout is not reliable(on slower machines angular can take longer to digest the change, so we will end up with height 0), also causes a minor flicker. This can be solved for static popovers, by storing the height value, but those with dynamic height will still struggle.
We are not handling scenarios where available space on both sides is lower than height. I think we need something like:

let absoluteHeight = Math.max(spaceAvailable.top, spaceAvailable.bottom);
if( absoluteHeight < height) {
 nativeEl.style.maxHeight = absoluteHeight;
} else if(spaceAvailable.bottom < height) {
 this.directionUp = true;
}

And probably remove maxHeight on hide.

@nikoTM could you check if the setTimeout can be fixed using this.changeDetectionRef.markForCheck(); like here?
Can you also make a branch with this code so that I can checkout it :)

@tb sorry for the delay, I will try to make the branch today.


adding md-radio-button gives error

Bug:
I added a bunch of material2 components together.  I was able to add a toolbar and button to my app and see it in the view.  Then I later added @angular-material/radio.  References are there in angular-cli-build.js, package.json, and system-config.ts.
import { MD_RADIO_DIRECTIVES } from '@angular2-material/radio';
import { MdUniqueSelectionDispatcher } from '@angular2-material/core';
....
  providers: [MdUniqueSelectionDispatcher],
  directives: [
    APP_SHELL_DIRECTIVES,
    MD_TOOLBAR_DIRECTIVES,
    MD_BUTTON_DIRECTIVES,
    MD_CARD_DIRECTIVES,
    MD_RADIO_DIRECTIVES
  ]

I add md-radio-group and there are no issues.  But as soon as I add md-radio-button, there is an error.
What is the current behavior?
build error.
What are the steps to reproduce?
http://plnkr.co/edit/BS6JmEIIj639kicEXnWc?p=preview
Which versions of Angular, Material, OS, browsers are affected?
material2: 2.0.0-alpha.6
OS: mac
browser: chrome ( but this is on all browsers)
Is there anything else we should know?
file changed app/app.component.ts
The Broccoli Plugin: [AppShellPlugin] failed with:
Error: Command failed: /bin/sh -c node /Users/user-name/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/angular2-broccoli-prerender/dist/child_proc.js  --sourceHtml=/Users/user-name/coding/angular2/mobile-toolkit/walnutquiz-web/webui/tmp/app_shell_plugin-input_base_path-dwO6CAEZ.tmp/0/index.html --optionsPath=/Users/user-name/coding/angular2/mobile-toolkit/walnutquiz-web/webui/tmp/app_shell_plugin-input_base_path-dwO6CAEZ.tmp/0/main-app-shell --outputIndexPath=/Users/user-name/coding/angular2/mobile-toolkit/walnutquiz-web/webui/tmp/app_shell_plugin-output_path-efRlZmdx.tmp/index.html
DEPRECATION WARNING: `document` is no longer supported and will be removed in next release. Please use `template`
DEPRECATION WARNING: `Bootloader#serializeApplication` arguments has changed.Please use an `AppConfig` interface {providers: Array<any>, directives: Array<any>, template?: string}
[TypeError: str.replace is not a function]

    at ChildProcess.exithandler (child_process.js:213:12)
    at emitTwo (events.js:100:13)
    at ChildProcess.emit (events.js:185:7)
    at maybeClose (internal/child_process.js:827:16)
    at Socket.<anonymous> (internal/child_process.js:319:11)
    at emitOne (events.js:90:13)
    at Socket.emit (events.js:182:7)
    at Pipe._onclose (net.js:471:12)

The broccoli plugin was instantiated at:
    at AppShellPlugin.Plugin (/Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/broccoli-plugin/index.js:10:31)
    at AppShellPlugin.CachingWriter [as constructor] (/Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/broccoli-caching-writer/index.js:18:10)
    at new AppShellPlugin (/Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/angular2-broccoli-prerender/dist/prerender.js:15:16)
    at Angular2App._buildTree (/Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/angular-cli/lib/broccoli/angular2-app.js:161:48)
    at new Angular2App (/Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/angular-cli/lib/broccoli/angular2-app.js:53:23)
    at module.exports (/Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/angular-cli-build.js:10:10)
    at Class.module.exports.Task.extend.setupBroccoliBuilder (/Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/angular-cli/node_modules/angular-cli/lib/models/builder.js:55:19)
    at Class.module.exports.Task.extend.init (/Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/angular-cli/node_modules/angular-cli/lib/models/builder.js:89:10)
    at new Class (/Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/angular-cli/node_modules/core-object/core-object.js:18:12)
    at Class.module.exports.Task.extend.run (/Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/angular-cli/node_modules/angular-cli/lib/tasks/serve.js:15:19)
    at /Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/angular-cli/node_modules/angular-cli/lib/commands/serve.js:64:24
    at lib$rsvp$$internal$$tryCatch (/Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1036:16)
    at lib$rsvp$$internal$$invokeCallback (/Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1048:17)
    at /Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:331:11
    at lib$rsvp$asap$$flush (/Users/username/coding/angular2/mobile-toolkit/walnutquiz-web/webui/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1198:9)
    at nextTickCallbackWith0Args (node.js:453:9)

Looks like I'm having the same issue with md-checkbox also.  Am I doing something fundamentally wrong?

import { MdRadioButton, MdUniqueSelectionDispatcher, MdRadioGroup } from '@angular2-material/radio'; is the correct import.
Can u reproduce thin in plunkr?

I had added a plnk but bad formatting made it invisible.  Have fixed it now.

I think this issue is obsolete. Feel free to open a new issue if there is still a problem.


Update site-recovery-network-mapping.md

I think the intended string was "VMNetwork2-Chicago". Ignore this if I am wrong.

Hi @ambhat, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@ambhat : Thanks for your contribution to the Azure documentation! The author, @rayne-wiselman, has been notified to review your proposed change.

@Jim-Parker
#sign-off
Good to go


Update to recent 0.24.* versions

Please update with latest 0.24.* versions

@motiz88 do you need any help?
I am really new to docker (don't really use it) and work on OS X, but I will have to works with people that have Windows machine, so this project is my best shot.
Maybe you could give hints on how to make a PR for a new version?

I've sent a PR for all recent versions: #3

Thanks!


docs: mermaid support

found this also: https://stackoverflow.com/questions/53883747/how-to-make-github-pages-markdown-support-mermaid-diagram


Mapping of nodes and relations returned as lists or maps

Expected Behavior
Consider the following query:

MATCH (n:A) RETURN n, [(n)-[rn:REL_1]-(n2) | [rn, n2]]

I would expect OGM to map n2 to its @NodeEntity object and wire relations accordingly. Alternatives:

MATCH (n:A) RETURN n, [[(n)-[rn:REL_1]-(n2) | [rn, n2]]]
MATCH (n:A) RETURN n, [(n)-[rn:REL_1]-(n2) | {rn:rn, n2:n2}]

Current Behavior
None of the above queries is mapping the node n2 when calling session.query("...", Map.of(), true).
Possible Solution
If adapted to

MATCH (n:A) RETURN n, [(n)-[:REL_1]-(n2) | n2], [(n)-[rn:REL_1]-() | rn]

then it works but IMO this is an unnecessary repitition of the pattern comprehension. Please also note that the cypher generated by OGM when calling session.load has the same pattern as the first alternative query from above which implied my assumption that it should work.
Steps to Reproduce (for bugs)
Simple example project reproducing the issue:
demo.zip
Start gradle task :test in debug mode to fail the assertion, important code lines in NodeService class.
Context
See https://stackoverflow.com/questions/59481997 for background information.
Your Environment

OGM Version used: 3.2.3
Java Version used: 11
Neo4J Version used: 3.5

I wonder if that is related to https://github.com/neo4j/neo4j-ogm/issues/666. Could you please test if your issues was there in 3.2.3 respectively 3.1.15?

I don't think this is related to #666 because here (#737) it's a even a step before the actual wiring of @NodeEntity. Consider the case that REL_1 is not even modeled in my @NodeEntity but exists in the graph. Now if I query like above I still want to get the related node n2 to be mapped by OGM but of course OGM couldn't wire returned @NodeEntity with node n. And in fact that works when return statement is a list of nodes in contrast to list of lists.
In a nutshell, this issue is about OGM mapping "list of nodes" correctly but not "list of list of nodes" or "list of maps containing nodes" in query results.
Btw my demo project for reproduction uses OGM 3.2.3.

I cannot reproduce that issue in current 3.2 and master. I have attached an updated version of your demo project, that uses Neo4j-OGM 3.2.4 and it runs as well (
fixed-demo.zip)
See my article here if you need help upgrading Spring managed versions in you projects: https://michael-simons.github.io/simple-meetup/overriding-spring-managed-versions-in-gradle-projects.
Thanks for reporting.

For the record: Also fixed in Neo4j-OGM 3.1.16 (see
fixed-demo_boot2.1.11_ogm3.1.16.zip)

Thanks! In fact, it is working with OGM version 3.2.4, so I will update. And thanks for the spring boot dependency tip, I didn't know it was that easy to adapt the version.

If you can wait till tomorrow, we have 3.2.6 out. It fixes a performance and spring boot dev tools issue.
The upgrade with Maven is similar (use Maven properties in the pom.xml like <neo4j-ogm.version>3.2.6</neo4j-ogm.version>).
Thanks for your feedback, much appreciated.

Hello @steffen-harbich-cognitum Here's a heads up for you: in #902 it was discovered that we flatten lists of known objects which shouldn't be the case as we don't do this for non-domain objects.
We are gonna fix that, but for your example it means a breaking change:
MATCH (n:Movie{title:'Pulp Fiction'}) return n, [(n)-[r:UNKNOWN]-(p) | [r,p]] as relAndNode

with the array around it, it will map to a List of node and relationships as it is actually correct (that's what a pattern comprehension produces).
A workaround to not change your calling code will be
MATCH (n:Movie{title:'Pulp Fiction'}) WITH n, [(n)-[r:UNKNOWN]-(p) | [r,p]] as patterns UNWIND patterns as relAndNode RETURN n, relAndNode

the mapping itself won't change.
Otherwise, the calling / usage side need a bit of change: It must now get the first index of the created pattern. See
https://github.com/neo4j/neo4j-ogm/blob/60e5f51a3c499f756732004be5b9e0fa57e2f6a6/neo4j-ogm-tests/neo4j-ogm-integration-tests/src/test/java/org/neo4j/ogm/persistence/session/capability/QueryCapabilityTest.java#L955-L969
Thank you and sorry for any possible inconvenience.


Releases frees-rpc 0.12.0

This release brings, among others things the following patches, features, and additions:

Fixed stacktraces in tests caused by unclosed channels: https://github.com/frees-io/freestyle-rpc/pull/189
Upgrades to Freestyle 0.8.0: https://github.com/frees-io/freestyle-rpc/pull/193
[Docs] Split into different sections: https://github.com/frees-io/freestyle-rpc/pull/190
Sbt build config refactoring, with dependency updates: https://github.com/frees-io/freestyle-rpc/pull/188
Merge https://github.com/frees-io/sbt-freestyle-protogen into the freestyle-rpc codebase, and update @rpc processing to handle latest freestyle-rpc syntax: https://github.com/frees-io/freestyle-rpc/pull/184
Avro IDL support: https://github.com/frees-io/freestyle-rpc/pull/195

Thanks @L-Lavigne, @eperinan and everyone who made this possible.

👍


Firebase ML kit operation progress

As at the time of creating this issue, there's no way to monitor the progress of a Firebase ML kit operation such as text recognition performed over a network.
It's possible that I just use a CircularProgressIndicator until the Future in which I'm running the operation is complete but I think this would make for a good UX since users can now know how far their ML recognition process has gone in percentage.
Something like this is possible in Firebase Storage Upload, but I've scratched through the ML kit docs and found nothing like it.

@thedejifab
The issue at https://github.com/flutter/flutter/issues/26576 has been closed and moved here. Future collaboration on this issue will be done here.

Closing this one - currently it's not supported by the underlying SDKs so it isn't possible to implement.
If it becomes available, feel free to re-create a new issue as a feature request.


test-branch/execute-job/wrong-commit-sha-on-pr/d1e37a61-77b7-4997-a147-1988796edf03

PR for testing 0440a321c140dc59b26e4caff854af9e0679fd90

/canonical/self-hosted-runners/run-workflows 931ad21dd9ace68aaa0a7d341e94de96ad93fa6b


Vision integration tests / snippets are failing

Looks like an external link has been taken down. Checking.

Looks like this has been resolved.


Feature request: Export disassembly to text

Is your feature request related to a problem? Please describe.
I want to do some simple custom pattern matching on a program's assembly instructions.
Describe the solution you'd like
I would like to be able to export a disassembly of the entire program, including addresses, labels, etc., to a text file so I can then use standard text tools to analyze or share it.
Describe alternatives you've considered

Select All + Copy.  However, Select All was greyed out.  I tried selecting the entire program by clicking at the beginning then shift-clicking at the end, then pressing copy; however, this only copied about a screen or two's worth of diassembly, not the entire program.
I could do the pattern matching using BN's Python API.
I could implement export-to-text using BN's Python API.
I could use a different disassembler on the same file.
In practice I will probably do the last one since I'm lazy.

Additional context
Bells and whistles that would be nice to have along with this:

Option to export any of the IL views in addition to disassembly
Option to export with syntax highlighting in some suitable format (HTML or RTF)

IDA has two ways to export a disassembly to text.  The one I mainly used, 'Create LST File', exports the same text seen in the interface.  There is also 'Create ASM File' which I've never had a use for, but apparently excludes the address column so that it's closer to something that could be passed back to an assembler.  Beyond that, there is 'Create C File' to export a decompilation.

(in before someone points out there's some obvious way to do this already…)

FWIW, @psifertex has a snippet that does some text export here: https://gist.github.com/psifertex/6fbc7532f536775194edd26290892ef7#file-save_decompilation-py

Yeah, sorry for not chiming in sooner. I was actually working on a more fully featured bit of code during a vacation for a side-task, but forgot to finish it off. Part of the hold up is we're working on "display as C" in #1650 and I kinda wanted to wait and see if that changed any of the display APIs for output style before I finished off the plugin.
Anyway, I just finished the changes and it's available in: https://gist.github.com/psifertex/6fbc7532f536775194edd26290892ef7#file-export_to_text-py

This is live on dev now:


CRUD endpoints for pullquote Alarm

For V1, all we want to do is schedule an alarm using ALL quotes. no filtering... yet.

[x] read
[x] create
[x] update
~- [ ] replace~ not currently needed
[x] destroy

merged in #30


.Net: Temporarily excluding planners and flow orchestrator

Action, sequential, stepwise planners, and flow orchestrator are temporarily excluded from the solution.
Motivation and Context

Description

Contribution Checklist

[ ] The code builds clean without any errors or warnings
[ ] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

This makes it hard to keep these projects building. They're still being built by CI.  I'm seeing failures now like:
/__w/semantic-kernel/semantic-kernel/dotnet/src/Experimental/Orchestration.Flow/Extensions/PromptTemplateConfigExtensions.cs(21,60): error CS1061: 'PromptTemplateConfig' does not contain a definition for 'GetDefaultRequestSettings' and no accessible extension method 'GetDefaultRequestSettings' accepting a first argument of type 'PromptTemplateConfig' could be found (are you missing a using directive or an assembly reference?) [/__w/semantic-kernel/semantic-kernel/dotnet/src/Experimental/Orchestration.Flow/Experimental.Orchestration.Flow.csproj]


Add documentation page on subhalo nomenclature conventions

There is currently nowhere stating what Halotools means by "subhalo", and users coming from different halo-finders have terminology that differs from the rockstar nomenclature

The following page was added to the documentation by PR #399
http://halotools.readthedocs.org/en/latest/quickstart_and_tutorials/tutorials/catalog_analysis/halocat_analysis/rockstar_subhalo_nomenclature.html#rockstar-subhalo-nomenclature


Add Honeycomb!

:)
This is a modified version of the Contributing Guidelines.
This pull request adheres to the repository's Code of Conduct

[ ] This PR contains housekeeping (URL edits, copy changes etc)
[x] You know your alphabet - company is listed in alphabetical order in the README
[x] A company profile is included - Not essential but it really helps!
[x] The company directly hires employees - This may sounds odd but while awesome, this list is not for coding bootcamps, "teaching" positions a network (Teacher positions employed directly, such as Treehouse, are allowed), or networking sites for listing a CV looking for gigs
[x] The company added hires remote employees, or positions are available to remote workers and are clearly illustrated as such
[x] A company-profile has been included for each addition to the list
[x] I am an employee of the company mentioned and confirm all details are correct
[x] Remote status has details regarding how the culture includes remote employees, how the company integrated remote workers, etc
[x] Region details any restrictions to applicants based on geography
[x] How to apply details the best approach for new applications, link to open positions, and any other help available for job hunters
[ ] Any missing parts from the profile, please use <Unknown> to allow for future completion

Cracking! Thanks @lpil, out of interest, where did you find out about this repository?

A friend found it on a slack channel. I forget which I'm afraid

Probably Support Driven.
No worries, thanks again 👍


Remove package server subscription

This PR includes #931 but also simplifies our deploy manifests by removing the catalog and subscription for package-server.

/retest

/retest

/test e2e-aws-upgrade

/test e2e-aws

/refresh

/retest

/retest

/retest

e2e-aws is failing because an OLM OpenShift conformance test expects that at least one instance of each OLM kind exists on a given cluster. We could potentially ask @jianzhangbjz to remove or modify this expectation in openshift/origin #223224.

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest


Vuetable perPage prop reactivity

Hi,
I update perPage and send it down as a prop to the Vuetable component but nothing happens...
 Shouldn't updating prop make it reactive and update table accordingly?

in case of calling refresh() method, thy it still see's the prop perPage old value insted of updated one? If I call with setTimeout then it's okay.

Thanks

Can you please set up an example code in jsfiddle or codepen, so I can have a look?
Please also run npm run dev to checkout the example as well.


Attempt to add public variables

Here I propose a method to introduce also public variable into classes

I'm not sure this is required.
If you just need a collection of public variables, there's the is_struct property.
In case of class, opening only part of data members would be violation of encapsulation


Create Fake Pull Request to Demonstrate. Close me.

We are adding a new fake dataset to cf.gov data.json / data.gov. (but not really)
[Enter an explanation of what the pull request does and why.]

[Feel free to delete any sections that are not applicable to this PR.]
Additions

Removals

Changes

How to test this PR

Screenshots
Notes and todos

Checklist
[Feel free to delete any checkboxes that are not applicable to this PR.]

[ ] PR has an informative and human-readable title

PR titles are used to generate the change log in releases,
so good ones make that easier to scan.
Consider prefixing with the feature/section you're addressing,
e.g., "Mega Menu: fix layout bug" or "Paying for College: Update content on Repay tool"

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code follows the standards laid out in the CFPB development guidelines
[ ] Future todos are captured in comments and/or tickets
[ ] Project documentation has been updated

Potential places for documentation:

This repo’s docs (edit the files in the /docs folder) –
for basic, close-to-the-code documentation on working with this repo
CFGOV/platform wiki on GHE – for internal CFPB developer guidance
CFPB/hubcap wiki on GHE – for internal CFPB design and content guidance

Front-end testing
[When new or significantly modified front-end functionality is present,
the following things should be tested.
Feel free to delete this section if not applicable to this PR.]
Browser testing
Visually tested in the following supported browsers:

[ ] Firefox
[ ] Chrome
[ ] Safari
[ ] Edge 18
[ ] Internet Explorer 11 and 8 (via emulation in 11's dev tools)
[ ] Safari on iOS
[ ] Chrome on Android

Further guidance on browser support
Accessibility

[ ] Keyboard friendly (navigable with tab, space, enter, arrow keys, etc.)
[ ] Screen reader friendly
[ ] Does not introduce new errors or warnings in
WAVE

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Does not introduce new lint warnings
[ ] Flexible from small to large screens

Fake PR. Please do not merge. Thanks!


make ok and error tuples easy.

solved in 4115db7 with the introduction of make_ok_tuple, make_error_tuple, make_ok_atom, make_error_atom


Bump version numbers post 0.25.0rc1

Summary
This opens development on main for the next minor version of Qiskit (no longer to be called Terra).  The version number is jumping to 0.45 to bring the version of qiskit-terra in line with qiskit (currently the metapackge); starting from "Qiskit 0.45", this repository will be called qiskit only, and the packages qiskit and qiskit-terra will be unified, at least as far as Python packaging allows us to do.

Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
15 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.009%) to 85.912%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/parse.rs
6
97.58%

crates/qasm2/src/lex.rs
9
90.63%

Totals

Change from base Build<PHONE_NUMBER>:
0.009%

Covered Lines:
72872

Relevant Lines:
84822

💛  - Coveralls


Added p-values to Wilcoxon

Added p-values output to Wilcoxon tests, as discussed in #270. Left most of the code the same (including the chunking). Wilcoxon test now outputs pvals, adjusted pvals (Bonferroni correction), and log fold changes. The change in speed should be much smaller than before:
Original time:

New time:

I also fixed a bug where the Wilcoxon test would output the absolute values of the scores when rankby_abs=True, which shouldn't happen according to the function description.
As a side note, I was unable to find a quick/efficient way to rank the full sparse matrix for the wilcoxon test, which is why I kept the chunking along genes as suggested. Hope this helps!

That's a good point, @LuckyMD. I chose Bonferroni to have a more stringent correction (albeit with a loss in power), particularly due to the increased power inherent in the large sample sizes of single cell data. I might be wrong, but I think the Benjamini-Hochberg standard was established with bulk RNAseq, where limited sample sizes required an approach with more power.
However, I'm happy to change it to Benjamini-Hochberg if that's the consensus! It's a simple one-liner - we can even provide both and let the user choose by passing a parameter. Whatever is preferred!

I definitely don't define the consensus, but I normally prefer FDR correction. It makes a bit more sense to me to correct for a false discovery rate, rather than a test-based error, if you are only interested in the rejected null hypotheses.
They also test for FDR control in a recent comparison of differential testing methods.

That's a good point! I just added the Benjamini-Hochberg correction to the Wilcoxon code (as well as to the t-tests) and left that as the default. I left the Bonferroni code in there and marked the place with comments. If and when the pull request is complete, they can choose what to keep and remove the extra comments. Either way, since the uncorrected p-values are also outputted, the user can choose whichever correction they want downstream.

Looks very good to me, thank you very much!
Would you mind adding an option to select for the correction type that defaults to 'benjamini-hochberg' and can be set to 'bonferroni'?
In the best of all world's, you'd also extend the tests for rank_genes_groups so that the p values are tested and not messed up by pull requests in the future. We want people to get the same p values again and again. And as the whole module sort of involves a lot of custom code as the scipy alternatives are not there for mult-dimensional and sparse data, it's easy to mess this up in the future.
Thank you so much for the awesome addition @a-munoz-rojas , I'll add you both to the Scanpy author list and to the release notes.

Thank you very much, @falexwolf! I really appreciate the addition to the author list! I'm glad this is useful.
I just now added the option to choose which correction method to use, and set benjamini-hochberg as the default, so that should be all set. With regards to the test, I unfortunately don't have experience building those tests, and have limited bandwidth at the moment. So it would probably be best if someone else could extend those tests.

Done! :smile:


Automatically hide servers item in navbar if the api does not support it

Potential issues: If the api is momentarily not available, it could hide the page.

Looks good in staging


feature/dq_oauth

OAuth.
Still requires testing and integration into DQ.py file.

Requires testing.

As discussed @Ksteeds here is the propose re-working of the folder structure for the DataQuery interface:

macrosynergy

dataquery

api.py

Interface class

auth.py

OAuth class for OAuth authentication
CertAuth class for certificate authentication

To keep back-ward compatibility we can rework the current DataQuery interface file dq.py into:
import warnings
from ..dataquery.api import Interface as DataQueryInterface

warnings.warn(
    "DataQueryInterface has been moved to macrosynergy.dataquery.api"
   DeprecationWArning,
   stacklevel=2
)

__all__ = ["DataQueryInterface"]

This should insure old code will still work.


has no exported member 'MdToolbarModule'

Bug, feature request, or proposal:
What is the expected behavior?
What is the current behavior?
Any component I try to import is not exported.
What are the steps to reproduce?
Providing a StackBlitz/Plunker (or similar) is the best way to get the team to see your issue.
Plunker starter (using on @master): https://goo.gl/DlHd6U
StackBlitz starter (using latest npm release): https://goo.gl/wwnhMV
npm start
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Is there anything else we should know?
How do i format the package json so it doesnt look like poo?
{   "name": "stock-app",   "version": "0.0.0",   "license": "MIT",    "scripts": {     "ng": "ng",     "start": "ng serve --host <IP_ADDRESS> --port 8081 --public-host sayed-fall-2017-phortonssf.c9users.io",     "build": "ng build",     "test": "ng test",     "lint": "ng lint",     "e2e": "ng e2e"   },   "private": true,   "dependencies": {     "@angular/animations": "^4.3.6",     "@angular/cdk": "^2.0.0-beta.10",     "@angular/common": "^4.2.4",     "@angular/compiler": "^4.2.4",     "@angular/core": "^4.2.4",     "@angular/forms": "^4.2.4",     "@angular/http": "^4.2.4",     "@angular/material": "^2.0.0-beta.10",     "@angular/platform-browser": "^4.2.4",     "@angular/platform-browser-dynamic": "^4.2.4",     "@angular/router": "^4.2.4",     "core-js": "^2.4.1",     "rxjs": "^5.4.2",     "zone.js": "^0.8.14"   },   "devDependencies": {     "@angular/cli": "1.4.1",     "@angular/compiler-cli": "^4.2.4",     "@angular/language-service": "^4.2.4",     "@types/jasmine": "~2.5.53",     "@types/jasminewd2": "~2.0.2",     "@types/node": "~6.0.60",     "codelyzer": "~3.1.1",     "jasmine-core": "~2.6.2",     "jasmine-spec-reporter": "~4.1.0",     "karma": "~1.7.0",     "karma-chrome-launcher": "~2.1.1",     "karma-cli": "~1.0.1",     "karma-coverage-istanbul-reporter": "^1.2.1",     "karma-jasmine": "~1.1.0",     "karma-jasmine-html-reporter": "^0.2.2",     "protractor": "~5.1.2",     "ts-node": "~3.2.0",     "tslint": "~5.3.2",     "typescript": "~2.3.3"   } }

the Md/md- prefix has been removed. See changelogs

import {MatToolbarModule} from '@angular/material'
and then selector is  or no mat?

Selector should be mat-toolbar, module is MatToolbarModule


I can not package this software

Can you help me to package a program, running on Windows?

Windows is currently not supported.