QUESTION: Django and Python version support

To the contributors:
Hi,
Could you let me know which Django versions are supported by the dj-database-url plugin? We are trying to upgrade to Django 2.2 (3.0.1 if possible).
Apologies if this information is already out there - I searched on PyPi and the documentation on the landing page for this project, but could not find it.
Also, the PyPi page for this shows Python 3.5 as the highest supported version. Is this correct, or can we use a higher version of Python with this? We are trying to upgrade to Python 3.7 (3.8 if possible).
Regards,
Manish Patel

This supports Python > 2.7, and Django > 1.11 (and I've just added some better CI to verify that I'm right about that).

This supports Python > 2.7, and Django > 1.11 (and I've just added some better CI to verify that I'm right about that).

Thanks a lot for the clarification!


Add missing @Nullable annotations on SourceSet

SourceSet.getTaskName(..) method was missing @Nullable on its nullable parameters.
It surfaced as Kotlin compiler warnings when passing null arguments.

Green build
Merging


Add environment element at boards and components level

The environment element is used in the standard at device and part level.
It may contains child elements that are specific for a development tool (identified by the environment's attribute name). The structure of an element inside the environment is not specified in the xsd schema file, which gives the development tool full control of the element usage.

So far, the environment can be used in elements:
/package/devices/family
/package/devices/family/device
/package/devices/family/device/variant
/package/devices/family/subfamily
/package/devices/family/subfamily/device
/package/devices/family/subfamily/device/variant
/package/parts/part
To me, it should be introduced also at boards and components level since, also for these elements, there could be the need to define some tool-specific information.
So my proposal is to extend the standard introducing the environment as (optional) child element of the following elements:
/package/components/bundle
/package/components/bundle/component
/package/components/component
/package/boards/board

@jkrech, @ReinhardKeil, what do you think?
The modification should enhance the standard. If you are ok with that, I can take charge of opening the PR.

@silviooliva, thanks for the systematic review of the  tag. Adding more places for environment specific information adds flexibility to capture additional tool dependent meta information in the package description.
My worry still is that despite us wanting to specify a standard format we end up with a level of flexibility which will make packs unusable in tools which will ignore this meta information.

@jkrech, I understand your concern, even if the presence of the mandatory elements (and their mandatory attributes) in the standard should provide the set of information ensuring that a pack is working in tools ignoring the meta information.
On the other hand, having such meta information through the environment element should favor the increase of the number of tools using the Open-CMSIS packs, so making the standard more and more popular.

Understood. @silviooliva  I did not mean to say that you should not prepare the PR adding environment more systematically throughout the hierarchy of the description.
I certainly prefer an environment tag being used instead of 'abusing' elements intended for other purposes.

Thanks for the PR. We had previously added <extensions> to <component> upon STMicroelectronic's request.
1. June 2022: v1.7.8
   - added extension points description for components as suggested by STMicroelectronics

Are we certain we need both?


ODK-X Survey: Add Back Navigation To The Top Appbar

I pulled the latest changes from the upgrade-jdk11 branch on the the Survey repo. While testing out the app and looking for bugs, I noticed there is no back navigation on the top appbar to lead users back to the main home screen
Here is the back navigation found in the Services App

And this is how the Survey App looks like with no back navigation ( UI Inconsistency )

My Point
Since this project is primarily UI Consistency, adding a back navigation is going to make the ui changes consistent, also improving the overall user experience
@maprehensive @Redeem-Grimm-Satoshi Can I work on this issue?

@juayuohcarineneng19 this is very well described, and you're right there's an inconsistency.
However, we usually use the Android back button, which is why there isn't a top nav back on most screens.
It would be complicated to add a top bar back button consistently across the apps, so I don't endorse this issue.

@juayuohcarineneng19 this is very well described, and you're right there's an inconsistency.
However, we usually use the Android back button, which is why there isn't a top nav back on most screens. It would be complicated to add a top bar back button consistently across the apps, so I don't endorse this issue.

Never thought of it this way makes  much more sense


New BVDP brand and style guide is incorporated into Saturn UI

Saturn is going to receive a new name. We need to revisit the logo, visual style guide, and design mocks to account for this change
Epic description in RPD here:
https://docs.google.com/document/d/10dkK5LtThALh9Za5LsaLtILkUxdkdDEbChNBYrGefyY/edit#bookmark=id.myrbbrni9otj

Epic complete


Inception V3 Example

I resolved some merge conflicts for @Ehud-Sharlin . Please confirm if anything is amiss.


Random com.google.cloud.storage.StorageException: Socket closed

We randomly receive com.google.cloud.storage.StorageException: Socket closed when uploading files.
Environment details

OS: Ubuntu 16
Java version: 11
google-cloud-java version(s): google-cloud-storage 1.53.0

Stacktrace
com.google.cloud.storage.StorageException: Socket closed
        at com.google.cloud.storage.spi.v1.HttpStorageRpc.translate(HttpStorageRpc.java:220)
        at com.google.cloud.storage.spi.v1.HttpStorageRpc.write(HttpStorageRpc.java:704)
        at com.google.cloud.storage.BlobWriteChannel$1.run(BlobWriteChannel.java:51)
        at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
        at com.google.api.gax.retrying.DirectRetryingExecutor.submit(DirectRetryingExecutor.java:105)
        at com.google.cloud.RetryHelper.run(RetryHelper.java:74)
        at com.google.cloud.RetryHelper.runWithRetries(RetryHelper.java:51)
        at com.google.cloud.storage.BlobWriteChannel.flushBuffer(BlobWriteChannel.java:47)
        at com.google.cloud.BaseWriteChannel.close(BaseWriteChannel.java:161)
        ....
Caused by: javax.net.ssl.SSLProtocolException: Socket closed
        at java.base/sun.security.ssl.Alert.createSSLException(Alert.java:126)
        at java.base/sun.security.ssl.TransportContext.fatal(TransportContext.java:321)
        at java.base/sun.security.ssl.TransportContext.fatal(TransportContext.java:264)
        at java.base/sun.security.ssl.TransportContext.fatal(TransportContext.java:259)
        at java.base/sun.security.ssl.SSLSocketImpl.handleException(SSLSocketImpl.java:1314)
        at java.base/sun.security.ssl.SSLSocketImpl$AppInputStream.read(SSLSocketImpl.java:839)
        at java.base/java.io.BufferedInputStream.fill(BufferedInputStream.java:252)
        at java.base/java.io.BufferedInputStream.read1(BufferedInputStream.java:292)
        at java.base/java.io.BufferedInputStream.read(BufferedInputStream.java:351)
        at java.base/sun.net.www.http.HttpClient.parseHTTPHeader(HttpClient.java:746)
        at java.base/sun.net.www.http.HttpClient.parseHTTP(HttpClient.java:689)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1604)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1509)
        at java.base/java.net.HttpURLConnection.getResponseCode(HttpURLConnection.java:527)
        at java.base/sun.net.www.protocol.https.HttpsURLConnectionImpl.getResponseCode(HttpsURLConnectionImpl.java:329)
        at com.google.api.client.http.javanet.NetHttpResponse.<init>(NetHttpResponse.java:37)
        at com.google.api.client.http.javanet.NetHttpRequest.execute(NetHttpRequest.java:143)
        at com.google.api.client.http.javanet.NetHttpRequest.execute(NetHttpRequest.java:84)
        at com.google.api.client.http.HttpRequest.execute(HttpRequest.java:1011)
        at com.google.cloud.storage.spi.v1.HttpStorageRpc.write(HttpStorageRpc.java:685)
        ... 105 common frames omitted
Caused by: java.net.SocketException: Socket closed
        at java.base/java.net.SocketInputStream.read(SocketInputStream.java:183)
        at java.base/java.net.SocketInputStream.read(SocketInputStream.java:140)
        at java.base/sun.security.ssl.SSLSocketInputRecord.read(SSLSocketInputRecord.java:448)
        at java.base/sun.security.ssl.SSLSocketInputRecord.bytesInCompletePacket(SSLSocketInputRecord.java:68)
        at java.base/sun.security.ssl.SSLSocketImpl.readApplicationRecord(SSLSocketImpl.java:1104)
        at java.base/sun.security.ssl.SSLSocketImpl$AppInputStream.read(SSLSocketImpl.java:823)
        ... 119 common frames omitted

Can you add SocketException to retryable exceptions? Or any hints how to diagnose the root cause?

Thanks for reporting this.
Can you give us a bit more information:

Is this running from GCP or externally?
Is the upload succeeding?
Is there a specific file size where you see this happening? (very large files, very small files)

GCP or externally?

Running on GCE.

Is the upload succeeding?

Unfortunately I can't tell. Have to increase diagnostic logging.

Is there a specific file size where you see this happening? (very large files, very small files)

Can't tell as well. All the files have a size of 10-20kB. Not very large, not very small.
A network problem is unlikely. There were multiple requests running in parallel, and only one failed.

Looking at the GCS load chart, there were request count peaks where we received the errors.
Once
ReadObject: 48/s
WriteObject: 25/s
another time:
GetObjectMetadata: 56/s
ReadObject: 56/s
WriteObject: 30/s

closing this as it seems fixed.

Hello,
I am still receiving this error. Is there a possibility that the issue is not fully resolved?
com.google.cloud.storage.StorageException: Socket closed
        at com.google.cloud.storage.spi.v1.HttpStorageRpc.translate(HttpStorageRpc.java:219)
        at com.google.cloud.storage.spi.v1.HttpStorageRpc.write(HttpStorageRpc.java:702)
        at com.google.cloud.storage.BlobWriteChannel$1.run(BlobWriteChannel.java:51)
        at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
        at com.google.api.gax.retrying.DirectRetryingExecutor.submit(DirectRetryingExecutor.java:89)
        at com.google.cloud.RetryHelper.run(RetryHelper.java:74)
        at com.google.cloud.RetryHelper.runWithRetries(RetryHelper.java:51)
        at com.google.cloud.storage.BlobWriteChannel.flushBuffer(BlobWriteChannel.java:47)

Caused by: java.net.SocketException: Socket closed
        at java.base/java.net.SocketInputStream.read(SocketInputStream.java:183)
        at java.base/java.net.SocketInputStream.read(SocketInputStream.java:140)
        at java.base/sun.security.ssl.SSLSocketInputRecord.read(SSLSocketInputRecord.java:448)
        at java.base/sun.security.ssl.SSLSocketInputRecord.bytesInCompletePacket(SSLSocketInputRecord.java:68)
        at java.base/sun.security.ssl.SSLSocketImpl.readApplicationRecord(SSLSocketImpl.java:1104)
        at java.base/sun.security.ssl.SSLSocketImpl$AppInputStream.read(SSLSocketImpl.java:823)
        ... 186 more

Thanks in advance!


Error form UglifyJs

Hello,
I am getting a strange error from UglifyJS.
Running this through webpack
Here is the error:
{ Error: 4.714f43602dc9cdd1454a.js from UglifyJs
Unexpected token: operator (>) [4.714f43602dc9cdd1454a.js:29,12]

It points to this section in the bss code:
const cssProperties = ['float'].concat(Object.keys(
  findWidth(document.documentElement.style)
).filter(p => p.indexOf('-') === -1 && p !== 'length'))

which looks fine.
I don't know what might cause this. The code works fine. It just wont minify.

UglifyJS can't handle ES6 IIRC

That's right Barney.
@smuemd what bundler are you using? If rollup, then bss uses rollup-plugin-buble for it's browser bundle, so you could use that too to make things work. (you can look at the bss rollup.config.js too)

It's a webpack project with standard configuration via spike (https://github.com/static-dev/spike)
@barneycarroll It's actually using ES6 all over the place. bss is the only package where this issue crops up. Emotion works fine for example. But who would want to use that? ;)

Oh I see..
I just tried a clean install of spike, and it doesn't complain using bss there.. Is the project one you can share?
Could you try to remove the node_modules folder package-lock.json and do npm install?
Googling for that specific uglify error also returns mostly 2017 results, so perhaps it's an old version causing this?

@porsager with spike the js minification only takes place in production mode:
you gotta run
$ npm run prod

I got another hint:
I just took the  ./lib folder and droped it from node_modules/bss into my local project directory
then:
import b from './lib'

minification worked without errors. Does Rollup give the files some special treatment? I am not too familiar with the tool.

Pointing main to bss.js and getting rid of the module declaration in package.json works too
what is the module declaration good for anyway?

For bss not that much currently since it only has a single default export, but if it had others, bundlers would use the module entry point to allow for tree-shaking (remove unused code paths).
A side-effect which is nice is that you'll also get ES6 code directly instead of transpiled ES5 which makes sense in cases where you just want to run on ES6 supported clients.
There's some more on the module field here https://github.com/rollup/rollup/wiki/pkg.module

I'm actually curious I can't find anything more about this anywhere.. I don't see why it would make sense for them "not" to transpile the module entry point...

interesting question indeed. I also  tried  theforceAllTransforms option for uglify-js. it just wont do it.

I think I'll keep this open until there's a resolution 😊 If you want to check in with webpack / spike, it's also good to refer here. I might also do that 😉

I think the issue lies in the transpilation step before uglify.js.
It's only uglify.es that supports ES6 (also outputs ES6)

I'm facing the problem using webpack before, looks like below two line:
https://github.com/static-dev/spike-core/blob/a2743fb7197a780be5e63fc641b7bb25c7813dd8/lib/config.js#L117
https://github.com/static-dev/spike-core/blob/a2743fb7197a780be5e63fc641b7bb25c7813dd8/lib/config.js#L227
Cause babel-loader to exclude the node_modules folder, thus also exclude the installed bss module, so bss kept as ES6 module untouched, then lead to the uglify error.
For you own webpack config, solution is remove the exclude rule for babel-loader, but for spike, maybe you should dive into the API to find a way...

Ah thanks a lot @futurist
The thing that confuses me is that if I remove the module field and point main to the ES6 entry point it actually does transpile it 🧐

@porsager If you use babel-preset-env, then it's only support transpile below formats:
"amd" | "umd" | "systemjs" | "commonjs" | false, defaults to "commonjs"
https://babeljs.io/docs/en/babel-preset-env.html#modules
The source is here:
https://github.com/babel/babel/blob/afa1207224a500d358b36c9b9a6e7167bb386e3d/packages/babel-preset-env/src/module-transformations.js#L4
Then follow into transform-modules-commonjs, below line:
https://github.com/babel/babel/blob/afa1207224a500d358b36c9b9a6e7167bb386e3d/packages/babel-plugin-transform-modules-commonjs/src/index.js#L122
seems the plugin skip any isModule format, then going on to find isModule definition:
https://github.com/babel/babel/blob/afa1207224a500d358b36c9b9a6e7167bb386e3d/packages/babel-helper-module-imports/src/is-module.js#L12
I'm lost in path.node.sourceType === "module" line...
Looks like module never be touched by babel-preset-env?

@futurist good find!
I opened an corresponding issue in the spike repo

@smuemd I looked at the spike issue, and I wasn't sure if my understanding that webpack 4 doesn't have this problem is correct? In that case I also think it would be fine to close this issue..

I also faced the problem when using webpack@3, for spike-core below line caused this issue:
https://github.com/static-dev/spike-core/blob/ad4debf3009a5ce819f60ff48c9bc3c551e1d9a2/package.json#L31
Seems webpack 3 don't support module field very well...

Thanks @futurist .. Do you have an easy workaround for a case like that? Maybe
import b from `bss/bss

I hope this  is resolved with Webpack 4, but I don't have time to test it right now..

Tested I have no issue again for webpack 3 when using import b from 'bss/bss', @smuemd could you have a test with spike?
Maybe can publish module with a version transpiled, without const etc, but have export default just for ES6?

@futurist It works! Uglify is not complaining anymore.

What is different now? What did I miss?

Hehe...
When you're doing bss/bss you're importing the bundled & transpiled file ;)

Ha. I had no idea. Thought `import b from 'bss' would actually do that as specified in package.json main

Yeah that's what you would think, but webpack chooses to pick the module field, and then not include it in it's transpilation pipeline 😔


Add support for feature flags

As per https://docs.gitlab.com/ce/api/features.html
I plan to implement this myself as I need it

Closing because the branch is merged in


Add missing support for tag bicycle:forward=no detected during test a…

As a result of the OSM note discussion http://www.openstreetmap.org/note/319765 and the resolution with bicycle:forward=no I detected that graphhopper does currently not support this tag.
This change does two things:

Add missing support for tag bicycle:forward=no detected during test after OSM issue http://www.openstreetmap.org/note/319765 got fixed.
Utilize the basic oneway logic in bike2 and handle the bike2 specific stuff afterwards such that we don't have to maintain the oneway stuff handling for bikes twice.

Thanks! Good catch regarding the code duplication! But I'm not sure regarding the bicycle:forward=no. Is this really the right tag to use, there are only 73 such values world wide :) !
BTW: also feel free for a lane>1 bike penalty as discussed on the mailing list :)

Thanks for detecting the infrequent usage of bicycle:forward=no. Even tough its infrequent usage I think we should add the support for it, because the generic vehicle:forward is also supported. But now I additionally implemented support for oneway:bicycle=yes, which is much more common.
I did not extend anything about lanes. On the mailing list I just wanted to inform the user that we do not evaluate any lanes for bicycle routing. As a first guess it might sound as a good idea to avoid ways with multiple lanes.
But it is not that simple:

On steep ways it is much better to have a road with two or even more lanes such that the faster traffic has more space for passing by.
There would need to be some handling at crossings, which have turn lanes. I'm currenlty not very familiar with the OSM tagging sheme for turn lanes.
Many lanes within residential areas indicate danger for traffic jams. Outside of residential areas the maxspeed for cars is much more important.
Especially with the racing bike it is a good idea to choose roads with many lanes, except for rushing hours. On a Sunday morning it is really great to leave big cities on roads with many lanes.

Therefore I have improved the avoidance handling depending on the car maxspeed tag instead.
See: https://github.com/graphhopper/graphhopper/compare/master...ratrun:master
For some unknown git reasons the two additional commits don't show up here under this pull request. Do I need to create a new pull request?

Okay, thanks for the explanations. Makes sense.

For some unknown git reasons the two additional commits don't show up here under
this pull request. Do I need to create a new pull request?

This pull request is against your branch ratrun:bicycleforwardno so you can add the changes there. BUT we should really separate every issue as a separate PR to avoid holding back changes that are okay because of changes that need work.

Thanks for pointing out the commit to the wrong branch. Could you please check once again? I added the patch with the maxspeed because of your comment about the lanes. Now I removed this change from this pull request. This pull request add now just the support for oneway:bicycle and bicycle:forwad=no.

Thanks!
One minor point: would you squash into one commit :) ? E.g. see here, if not, really no problem I can do this

Sorry, git only sometimes does what I intend, I always get errors which I can't interpret.

no problem, I know what you mean ;) (I'll do it tomorrow)

@karussell thanks for the git squash hint!

Merged here: https://github.com/graphhopper/graphhopper/commit/1df47ec6dae1f07579b168921bc4e42d6ebea040


run_smartseq2 wants UMIs

Dear Velocyto Team,
It looks like that version 0.17.10 introduced a bug into run_smartseq2.
When running
velocyto run_smartseq2 -o OUTPUT -m mm10_rmsk.gtf -e 35_rmsk ../bam-star/35/Aligned.sortedByCoord.out.bam gencode.vM18.primary_assembly.annotation.gtf
I get the following (shortened) error:
DEBUG - Cell barcodes will be determined while reading the .bam file
DEBUG - The multi input option
DEBUG - Peeking into /home/ubuntu/data/bam-star/35/
Aligned.sortedByCoord.out.bam
WARNING - Not found cell and umi barcode in entry 0 of the bam file
WARNING - Not found cell and umi barcode in entry 1 of the bam file
WARNING - Not found cell and umi barcode in entry 2 of the bam file

This used to be working in 0.17.8.
Thank you!

Thank you for reporting.
From the log I see some warnings but not an error. How does the log
continues? Does it throw this kind of warnings without ever stopping?

Sorry, I forgot to include the last bit of the error log:
  File "/usr/local/bin/velocyto", line 11, in <module>
    sys.exit(cli())
  File "/usr/lib/python3/dist-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3/dist-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/lib/python3/dist-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/lib/python3/dist-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/velocyto/commands/run_smartseq2.py", line 71, in run_smartseq2
    samtools_memory=1, dump=dump, verbose=verbose, additional_ca=additional_ca)
  File "/usr/local/lib/python3.6/dist-packages/velocyto/commands/_run.py", line 155, in _run
    exincounter.peek_umi_only(bamfile[0])
  File "/usr/local/lib/python3.6/dist-packages/velocyto/counter.py", line 186, in peek_umi_only
    raise IOError("The bam file does not contain umi barcodes appropriatelly formatted. If you are runnin UMI-less data you should use the -U flag.")
OSError: The bam file does not contain umi barcodes appropriatelly formatted. If you are runnin UMI-less data you should use the -U flag.

Everything before that are just warnings for each entry of the bam file. After this output velocyto stops without producing any output.

I recall this bug was discussed already in a previous issue, but actually I
couldn't find it myself now by quickly skimming through... . Anyways it is
a bug affecting using smartseq2 with only one file, where it expects
multiple files instead if you use 2 or more (e. G. with glob expansion) the
problem is solved. Otherwise use 'run' with the appropriate combination of
parameters, see the docs for further info.
I will try to bugfix in 0.17.11


Add semanticaly clickable button element to result list

improves usability with keyboard #258
improves accessbility

See #259 for other relevant discussion

Closes #258
Closes #189

(Edited the description to auto-close relevant issues.)

(Edited the description to auto-close relevant issues.)

Good work. Details to fixup in the styles, but otherwise appropriate approach.

Good work. Details to fixup in the styles, but otherwise appropriate approach.


test: fix ssz types in fork_choice spec tests

Motivation
Fixes issue brought up by @jeluard  in fork choice spec tests
First offset must equal to fixedEnd 229 != 228

Description
Select correct ssz types for Attestation and AttesterSlashing in fork_choice spec tests

:tada: This PR is included in v1.22.0 :tada:


key.example.txt syntax help

I'm looking to get more clarity on how to read/use a "key.txt" file. The file contains the line
bottom drive obey lake curtain smoke basket hold race lonely fit walk//Alice
This is a 12 word mnemonic.
1 can I use a hex private key instead?
2 what does "//Alice" denote?
3 if there are multiple accounts being used, how do you make a proper array?
4 are there differences for a Kusama and Polkadot key?
As a slight aside, the syntax for multiple stash accounts in the "stashes.example.json" file is
[
"kldfjfaklsdklsdf"
"sdfjklakldsjdfkl"
]
OR
[
"jdkasfdjklss", "kdffadjkljsd"
]
Thank you kindly for this help.

I haven't tested, but it should not be an issue
This is an example of a hard derive, if you don't know what it is you probably can ignore it. Here is a section with some more info: https://substrate.dev/docs/en/knowledgebase/integrate/subkey#hd-key-derivation . The exact key I use here is the canonical "Alice" for substrate development.
You can only make payouts from one account. This account should have just enough to cover transaction fees.
Nope, same syntax.

As a slight aside, the syntax for multiple stash accounts in the "stashes.example.json" file is

The latter with a ,. So [ "jdkasfdjklss", "kdffadjkljsd" ] should work fine. (It expects a JSON array.)
Closing for now, but please reopen if you have any follow up questions and fee free to open another issue for different questions or feedback :)

Thank you thank you kindly for your quick response and help! And I apologize for my slow reply.
For all the payment questions I was confusing myself, yes, all payments should only be coming from 1 account (controller Accout) and therefore an array wouldn't be necessary. My mistake. Thank you again and for the use of your code!


Add two sparse matrix

can i take the issue?

Yes sure. I have assigned the issue to you

Can I work on this?

@amritansh22 I would like to take this up!


unable to define individual position

Plunkr demo here: http://plnkr.co/edit/VFcfYRe8jY53Mkup6UeK?p=preview
the issue is my inability to dynamically define where the notification will appear. Passing in different numbers for the "startTop" and "startRight" parameters do not affect where the notification appears (as seen in the plunkr example. In my application I need to make the notification appear in different places (to correspond with specific elements) but can not define the position through these (startTop, startRight) parameters. Simply using "positionX: 'center'" is not specific enough of a position.

+1 same here

+1

Same here, I think the issue is you're using an assignment operator(=) where you maybe meant to use a conditional operator(==) in this line: https://github.com/alexcrack/angular-ui-notification/blob/master/src/angular-ui-notification.js#L85

What is the solution?
I have the same problem.
The notification appears in the same place despite I had defined the 'positionY' and 'positionX'.


Support split layer

:octocat: cibot: Thank you for posting issue #1208. The person in charge will reply soon.


Advanced Analytics for Instructors using SandDance and Data Query API

Is your feature request related to a problem?
Good exercise design can be developed and improved by analyzing how exercises are rated, corrected and how well test cases work, how exercises compare with each other and so on. This issue suggests using SandDance, which is also available as VS Code extension and a powerful visualization for plain CSVs. This is much more flexible and efficient than trying to implement all sorts of data visualizations manually into Artemis.
Describe the solution you'd like
Allow instructors of a course to:

Select a dataset (that is generated by the server using the DB and sent to the client).
Could be:

row per exercise
row per result [for a set of exercises]
row by feedback / test case in case of programming exercises [for a set of ...]
SGI usage / distribution
...

Select some additional properties; we could send all but that would always be expensive for Network and Client and DB.
(like LOC, Submission count, automatic & manual result, difference, Model Elements (or count), ...)
Being able to explore the data in https://github.com/microsoft/SandDance
Once the client has the data, no more ReST-calls are needed, unless the instructor wants to change the dataset.
Instructors can also download the generated CSV to process it further

Notes
A very interesting approach would be to realize this with e.g. a GraphQL-API to retrieve the data. This GraphQL API could also be used by programmatic clients (but limited to retrieval first). This would allow instructors to select only needed attributes in a powerful way that also helps to reduce the load on the DB or server. I would then advise to make this an officially supported Artemis API. At least I could imagine this working great, as the Artemis Objects as well as the proposed analytics are very data-centric, and this would allow to be selective, minimalist and still provide the user with all they need. The schema would make it fairly user-friendly as well (as only instructors have access, this should be ok as an option). Some basic and quick calculations could be done on the server side to decide whether a request would be too expensive (or warn the user beforehand). This API could also be decoupled without too much effort from the internal representation of the classes in Artemis/DB, as far as I know. Having an API like that would also reduce the number of calls needed to get the desired data.
As a side effect, this data export API could offer a file download of the results as well, effectively solving #1034.
I am happy for more feedback and ideas on this, we had this idea when analyzing data in SandDance retrieved using the browser current client API in PGdP and believe this would be extremely helpful for many other courses as well (looking at the before mentioned issue and some conversations in Slack).

This would also allow other chairs to build and research systems that don't need to be integrated into Artemis (directly). One could e.g. offer bachelor thesis topics on data mining on Artemis data, or static code analysis without requiring the solution to be an Artemis feature first, which would reduce the scope and the effort needed to supervise students and developers.
Once those systems have proven themselves, they could be integrated in a similar manner as the text clustering already is via a PR.
This lowers the barrier of entry into advanced Artemis work because no proficiency with Artemis code itself is needed.
Furthermore, such an integration could then be done by someone else as bachelor thesis, for example, decoupling Artemis development and feature development/research.

out of scope at the moment

It might be interesting to note that more clients start to appear. Apart from our unmaintained, closed-source Java API I found this: https://github.com/kit-sdq/artemis_client.


Simplify check set parameters in acmedns config

Bash 5.2 contains the following change (from their NEWS section):

j. Associative array assignment and certain instances of referencing (e.g.,
test -v' now allow @' and `*' to be used as keys.

… which breaks the existing check. Simply reference check for a set array item directly.

Thanks!


Python: Select Interpreter doesn't list all pyenv interpreters

Environment data

VS Code version: 1.21.1
Extension version (available under the Extensions sidebar): 2018.3.1
OS and version: OS X 10.13.3
Python version (& distribution if applicable, e.g. Anaconda): Multiple
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): pyenv(-virtualenv)
Relevant/affected Python packages and their versions: N/A

Actual behavior
See https://github.com/Microsoft/vscode-python/issues/363#issuecomment-378057027
Expected  behavior
All pyenv (and pyenv virtualenvs) interpreters should appear in list of interpreters.
Steps to reproduce:

With multiple pyenv versions and virtualenvs, run pyenv versions to get a list of all versions
View list from "Python: Select Interpreters"
Notice that the list in VSCode is incomplete

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
N/A, only contains linting output
Output from Console under the Developer Tools panel (toggle Developer Tools on under Help)
None

The issue is that you're detecting all of the non-virtual interpreters and some of the virtual ones, when what I believe is expected is that all of both kinds would be listed.

Hi, I've the same issue on mac.
When I've defined the pyton.venvPath on Settings->Python section, I could see the list of my venvs,
but from the last updates of VSCode, I can't get the venvs list from cmd+shift+P:Select Interpreter.
I get only the system installation, I have only virtualenv generated envs
Looking around I found that opening a terminal,
then activating a virtual env with  "workon mypyenv"
and then start VSCode with "code -a ."
I can see the active virtual env on  venvs list

@gbonline That does sound frustrating, however I want to make sure not to confuse the issues, as the one I'm brining up is specifically pyenv-related (whereas it looks like you're using virtualenv-wrapper) and doesn't involve the venvPath setting. I might suggest filing a separate issue? Best of luck with your problem.

@gbonline you're looking for #1192

@neverfox So you're saying we actually detect some of the virtual environments created using pyenv-virtualenv? I'm honestly surprised we pick up any without a configuration setting since we only pick up virtual environments in your workspace folder or if you have a Pipfile and so we know to go look for pipenv, hence why I want to make sure I understand exactly what is going on (screencast would be awesome).

@brettcannon Yes, some are picked up and some aren't. Did you see the screenshot and pyenv output I posted in the other issue?

@neverfox I did see it, but that window can be scrollable so it's non-obvious if that's the complete list or not.
Regardless, we think we might know the cause and fixed it in #1353 . If you have time, please try out the development build of the extension and see if this solves your problem.

Yeah, I thought you might wonder about that. I did scroll for sure and it's not complete. I will check out the dev build, thanks.

The alpha version appears to list all interpreters in pyenv. Now if there's ever a solution to #1167, pyenv users will be mostly set.


chore(dev/ci): set expiry date for access token

As of GitLab v16.0, all access tokens must have an expiry date. Otherwise the access token will not be created. This wasn't being done in the script that sets up a GitLab instances with an access token so that the token can be used for all acceptance tests via API calls. This worked fine before but started failing recently when latest version of GitLab was being used in the CI because the latest version is 16 and up. As a result, all the PR validations are failing because the acceptance tests are failing with '401 Unauthorized' error.

Hi @gdubicki 👋,
I'm not sure why codecov is reporting a big decrease in test coverage. This PR makes no changes to the codes that it's reporting a coverage decrease. Not too familiar with codecov, but I think this shouldn't be a blocker for this PR.
Also, because of GitLab v16 requirements of bot users must have an expiry date, couple of existing acceptance tests are failing in this PR.
=========================== short test summary info ============================
ERROR tests/acceptance/standard/test_group_members_users.py::TestGroupMembersUsers::test__remove_user_keep_bots - gitlab.exceptions.GitlabCreateError: 400: expires_at is missing
ERROR tests/acceptance/standard/test_members_enforce.py::TestMembersEnforce::test__enforce_keep_bots - gitlab.exceptions.GitlabCreateError: 400: expires_at is missing
============== 102 passed, 2 errors, 6 rerun in 223.73s (0:03:43) ==============

I believe this should be fixed as part of #554.
So, I'm leaning towards merging this PR soon even with the reported failures. It will at least resolve the issue with acceptance tests not running in the CI and unblock #554 . Thoughts?


[Feature] Generate visual plant sheet as PDF

Offer the possibility to:

Export a list of plants with their preview images and some details to PDF from the location lists
Export the current history (opt: up until a specific year) to PDF with preview images and plant name ordered by year

This way users can export something about their plants that they can backup somewhere or share with others.
Steps:

Generate the HTML content from the related data
Convert the HTML contents to PDF

Suggested HTML2PDF converter: https://github.com/dompdf/dompdf

After consideration and a talk it seems that this feature would not be of much use to users. Closing.


Possible typo in synopsys decorator

This line in gate_decorator_bdd_synopsys seems incorrect:
https://github.com/emsec/hal/blob/25094f4e0d4befd52139851c8a164bd73d68bc9c/plugins/gate_libraries/src/gate_decorator_bdd_synopsys.cpp#L94
Both sides of the OR statement are equal.
@swallat since you introduced this piece of code in 09f820a9bc0227daa87c2bfe2dde270c49009847, could you please check if the right side of this statement should rather say AOI22X2?

@RenWal Actually the line was introduced by @devhoffmann before the transition to Github. @devhoffmann could you check that?

Right literal should be "AOI22X2", I will fix that


backend changes

hello genuis, please can you give me the backend repo

You can find it here

you're the best genuis..
can i implement the backend of the reactjs project on the native project?

bro please help!
saving the user credentials and logging in from physical device requires asyncStorage?

yes


KITTI

Hello， How Can I set the configure files if I only want to train 'Car' about KITTI datasets？

Please try to change CLASS_NAMES and ANCHOR_GENERATOR_CONFIG to make them only include 'Car' category.


Failed to work after 5000 lines of code

Language: C
Problem: As stated in the title. Any bracket pair after line 5000 fails to be colored

I am seeing the same thing happen in .less files, as well

I hard coded this to prevent performance degradation, but I can move it to
a setting
On Wed, 6 Jun 2018, 3:08 PM Dennay<EMAIL_ADDRESS>wrote:

I am seeing the same thing happen in .less files, as well
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/CoenraadS/BracketPair/issues/154#issuecomment-395061971,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFVa4QpOkLGlmsVJYwZ3Ki4AzrfNyKVNks5t59Q6gaJpZM4UcEJn
.

I hard coded this to prevent performance degradation, but I can move it to
a setting

Well, in all honesty, there are still a lot of performance issues. I was forced to uninstall the extension (although I love it and keep using it at home) seeing that my work laptop with i5-5200U cannot handle it properly in any file larger than 1k lines. It gets exponentially worse as the files become larger. Our unit test files regularly exceed 2-3k lines of code and the extension simply kills the performance on that machine to a point that when editing the code that's in a more broad scope I have to wait up to a second or two for the inputs to register.
Maybe it would be nice to have a user preference where the user could limit the scope of the extension? For example, I usually only need the bracket highlighting for the function that I'm currently editing and a scope or two outside of that function.
Also, is it possible to improve the performance by detecting the collapsed code blocks and skip the processing of those?
These are just some random suggestions and thoughts. I didn't want to open a new issue seeing that I know that improving performance on an extension like this simply has it's limits but I still wanted to give you my two cents on the current status of the performance.

Yes - please move this to a setting as I am experiencing it as well. Thanks!


web: Case Studies md doc in Operating section.

As  Wiard proposed in our community meeting, it would be nice to start good example and have case studies with details like configuration setting.
Do you mind Wiard to starting this one? And propose some template? 🤗
Also help wanted everyone else, especially those who runs Thanos on Prod. We are happy to provide Red Hat one as well.

Currently working on public relations approval for our company, but nonetheless I'll make a template proposal.

Looking at other companies that are willing to share some details and existing blog post we have linked in Thanos website (e.g https://build.thebeat.co/monitoring-the-beat-microservices-a-tale-of-evolution-4e246882606e) I think it has to be bigger than a single page (:
I would propose another section: Case Studies and doc How to add case study under contributing. Then each doc per company WDYT?

I can start some initial work on website on separate branch (we have preview) and anyone can propose PR against that so we can incrementally work on this 🤗

Looking at other companies that are willing to share some details and existing blog post we have linked in Thanos website (e.g https://build.thebeat.co/monitoring-the-beat-microservices-a-tale-of-evolution-4e246882606e) I think it has to be bigger than a single page (:
I would propose another section: Case Studies and doc How to add case study under contributing. Then each doc per company WDYT?

Even better.

Still valid.

We are also interested in contributing in this way.

Super nice!

If it would help I can contribute our use case, still very fresh though ;) Our stack includes multi-tenant thanos receive, OpenID Connect integration (Keycloak) to authenticate the remote Prometheus shippers (which we don't fully control), the label-proxy-enforcer and Minio for long-term storage.
Let me know as soon as you want me to write up something, happy to help in any way I can, ❤  this project :)

Thanks @hoempf - I'll block some time ASAP so I can create a template page and we can get this rolling!

Still needed


[Release] <IP_ADDRESS>

Release tools version <IP_ADDRESS>.
Notes:

#107 [Comparison] ComparisonPerformers buildGetters breaks the filtering

Some major behavior bug fixed, with kind of a feature added as a fix.

Fix was backported to the 0.0.9 version and released as <IP_ADDRESS>
Master was not affected by release, and still have development iteration 0.0.10


Delegate + aliases not working as expected

I have a AR model that has a db column called state, and I want to transform it before serializing the data. That method that transforms data lives on the AR model
When delegating the method to the AR object + aliasing it, panko is not working the way I would expect. it's returning the actual persisted database value for object#state
class StateSerializer  < Panko::Serializer
  attributes :state

  aliases friendly_state: :state
  delegate :friendly_state, to: :object
end

class ActivityLog < ApplicationRecord
   def friendly_state
    case state
    when "error"
      "User Error"
    else
      state
    end
  end
end

I can accomplish what I want by doing but I think it would be a better approach to be able to rely on aliases + method delegation
class StateSerializer  < Panko::Serializer
  attributes :state

  def state
	object.friendly_state
  end
end

@danielnc Thanks for reporting this issue!
I see what you say and totally agree, this happens because aliases works on attributes and not on methods.
Since serializer is not strict to specific model (as of today), I can't know if "friendly_state" is a method on a model and then alias to it's method.
Making serializers strongly typed, is something I want to achieve in the future but not now.
I'll try to think about an options how to solve this without complicating the library.


showTextDocument & vscode.diff always default to ViewColumn.One

VSCode Version: Code - Insiders 1.13.0-insider (770206ab9c6357943a2145bd3ea49b667f3ff539, 2017-05-26T05:14:23.944Z)
OS Version: Windows_NT ia32 10.0.16199

These 2 apis should honor the ViewColumn if specified, but if left undefined they should default to the column of the active editor
Having this will eliminate code in many extensions such as:
(window.activeTextEditor && window.activeTextEditor.viewColumn) || 1

Which fails for anything that isn't a text document (since viewColumn will be missing)

Which fails for anything that isn't a text document (since viewColumn will be missing)

Unsure what that means?

Sorry, I meant that the window.activeTextEditor could be undefined if the actual active "editor" is a non-text document (image, extension, etc). And window.activeTextEditor.viewColumn can be undefined for a diff editor (maybe other cases?). So an extension can't reliably get the real "active" column

+1 for not setting ViewColumn.ONE as default if not specified so that we prefer the currently active editor column when documents open.

@bpasero PR https://github.com/Microsoft/vscode/pull/27409 addresses it, but it conflicts with another change see: https://github.com/Microsoft/vscode/pull/27409#issuecomment-304564580
Any thoughts on coming to a resolution on that -- I would LOVE to see this fixed.

@eamodio we have an option for editors to reveal if they are open in another group (see revealIfOpened).
I am actually surprised that https://github.com/Microsoft/vscode/issues/25801 was fixed and marked as important because the normal workbench behaviour is to open an editor in the currently active editor group. On top of that, the solution also does not seem right because what if I start in column 2 and I open a document in column 3 and then open the document from column 2 again. It opens in column 1?

@bpasero I was surprised as well as that was my argument too :)

I could rebase that PR again (since its quite old) if you'd like, but it will still fail because of a test checking for https://github.com/Microsoft/vscode/issues/25801

I am actually surprised that #25801 was fixed and marked as important because the normal workbench behaviour is to open an editor in the currently active editor group.

Well, we had this behaviour in place, extensions rely on it https://github.com/Microsoft/vscode/issues/25801#issuecomment-329824653, and we don't want to break them. That's why this is tricky


[BACKLOG-19744] Remove the source fields from XML Join preview generation

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
mvn -Dsurefire.runOrder=alphabetical -B -fn -f 'pom.xml' -pl 'plugins/xml/core' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'plugins/xml/core' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'plugins/xml/core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/xml/core/src/main/java/org/pentaho/di/trans/steps/xmljoin/XMLJoinMeta.java
  plugins/xml/core/src/test/java/org/pentaho/di/trans/steps/xmljoin/XmlJoinMetaGetFieldsTest.java

@pentaho/ackbar please review

This looks really close to the "fix" that was originally done which caused a performance regression on PDI-16721. transMeta.getStepFields calls the step's getFields method. Can you validate that PDI-16721 is not reintroduced with that fix ?

will do


load lab_4

your decorator should raise n case of invalid IP address, otherwise in unclear what's the function return type is


e2e tests not working on fresh install

I am using gulp, typescript, bootstrap and sass and when I run gulp test:e2e on a fresh install (and updates selenium) I get:
UnknownError: unknown error: Chrome failed to start: crashed   (Driver info: chromedriver=2.21.371459 (36d3d07f660ff2bc1bf28a75d1cdabed0983e7c4),platform=Windows NT 10.0 x86_64) (WARNING: The server did not provide any stacktrace information) Command duration or timeout: 60.73 seconds Build info: version: '2.51.0', revision: '1af067d', time: '2016-02-05 19:15:17' System info: host: 'DESKTOP-79F9HEB', ip: '<IP_ADDRESS>', os.name: 'Windows 10', os.arch: 'amd64', os.version: '10.0', java.version: '1.8.0_77' Driver info: org.openqa.selenium.chrome.ChromeDriver

This looks like Chrome failed to launch after a 60 second timeout. I don't think this is related to this plugin, but rather seems to be an issue with a Chrome installation.
I'm going to close this for now, but if you have additional information (gulpfile, Chrome version, replication steps) I'll reopen.

It was releated to chrome and not the plugin.


Add pprof endpoint

What would you like to be added:
Add an endpoint that allows to capture a profile.
Core Kuberentes components have this:
https://github.com/kubernetes/kubernetes/blob/407bd6a4afa5f7d7cb2a596dda09cacc30828f09/cmd/kube-scheduler/app/server.go#L282
Not sure if there is a way to do this via controller runtime https://github.com/kubernetes-sigs/controller-runtime/issues/1779
Why is this needed:
To inform performance improvements
Completion requirements:
This enhancement requires the following artifacts:

[ ] Design doc
[ ] API change
[x] Docs update

The artifacts should be linked in subsequent comments.

There is WIP https://github.com/kubernetes-sigs/controller-runtime/pull/1943

/remove-lifecycle stale

The next release of controller-runtime should include this https://github.com/kubernetes-sigs/controller-runtime/pull/1943

/remove-lifecycle stale

It should be doable with kubernetes-sigs/controller-runtime#1779 , we just need to add support for this in the config.

/assign


why do the loss equal nan when i don't use clipnorm ?

i know that clipnorm fix this issue and i know that clipnorm clip the big number of the gradients but i want to know why the nan is produced?
why do i see loss=nan when i don't use clipping of the gradient?

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[ ] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[x] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[ ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

the gradient gets really really large, which changes the weight values a LOT... and this makes the loss NaN because it's usually too large of a number for python to represent.

but what is the maximum number python can handle
i googled it and found
 import sys;   print(sys.maxsize);<PHONE_NUMBER>854775807
i mean, may you tell me an example of that big number in loss that make it show NaN?


After max concurrency is reached, jobs stop being processed

Any ideas why after the max concurrency setting for a specific job is reached, it seems to no longer accept new jobs, even after it's gone down.  For example, if set to 20, it'll process 20, then start to complete some, inactive jobs stay inactive and don't automatically start processing.

if your claim was true, then all Kue users would stop using it since this would be critical bug :)
Please post a sample code that can regenerate the problem of your case.

Yeah..... I can't seem to reproduce it, I'm thinking maybe it could have been related to high CPU load.  If it occurs again, I'll dig around a little more to see if I can find more info.  I'll close this issue out, thank you.


Revisit signup error copy

See here

Is this still an issue with the new onboarding flow?

nope!


Use StringView for protocol stack to avoid calling strlen on strings with a known length.

This is an alternative proposal to #1520.

I checked the compiler output and AFAICT passing a StringView by value is the same cost as passing two integers. That is, passing a StringView by value has the same performance as passing a pointer and a length by value.

FreeBSD build failed! See https://ci.trafficserver.apache.org/job/freebsd-github/1665/ for details.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/97/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1561/ for details.

FreeBSD build failed! See https://ci.trafficserver.apache.org/job/freebsd-github/1666/ for details.

clang-analyzer build failed! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/229/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1562/ for details.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/98/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1563/ for details.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/99/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1667/ for details.

clang-analyzer build failed! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/230/ for details.

clang-analyzer build failed! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/231/ for details.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/100/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1668/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1564/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/232/ for details.

👍
Looks good.
Just one question. Should the internal "protocol_contains" functions return "StringView" as well?
Obviously we should not be changing the return type of public API. But internally should we be using "StringView" for those internal api calls?

Hmmmm. I think I'd leave that for another day. Those aren't really used internally, only to forward the API request. The return value isn't examined except to check for nullptr which is just as fast without the StringView. Further the StringView can't be returned via the TS API without changes, making that not particularly useful. A key goal of that call is to normalize the string to allow direct pointer comparison, in which case the length is superflous.

👍

It even works:
odin (08:54) 283/0 $ curl -D - -o /dev/null -s  https://www.ogre.com/ | grep Via:
Via: http/1.1 tls/1.2 tcp ipv4 Cosmo (ApacheTrafficServer/7.2.0 [cRs f ])

Question: Do we really want to modify the Via: header format in the mid-release cycle? I guess i'm ok with it, but, a concern would be if anyone parses the Via: header for whatever reason, it might break things?

I'm terrible at reading RFC docs, so, fwiw, the Via header structure is described here:
https://tools.ietf.org/html/rfc7230#section-5.7.1

Linux build successful! See https://ci.trafficserver.apache.org/job/RAT-github/1/ for details.

Upon further review of this, #1520, and #1506, I think this is just a bug fix, in that there was already code in place to report the protocol stack in the VIA header. Therefore this is primarily an update to use StringView (and incidentally fix the #1506 bug).

FreeBSD11 build failed! See https://ci.trafficserver.apache.org/job/freebsd-github/1700/ for details.

clang-format check successful! See https://ci.trafficserver.apache.org/job/clang-format-github/5/ for details.

RAT check successful! See https://ci.trafficserver.apache.org/job/RAT-github/18/ for details.

AU check successful! See https://ci.trafficserver.apache.org/job/autest-github/4/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1595/ for details.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/131/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/263/ for details.

Gonna build it again anyways ... [approve-ci]

clang-format check successful! See https://ci.trafficserver.apache.org/job/clang-format-github/6/ for details.

RAT check successful! See https://ci.trafficserver.apache.org/job/RAT-github/19/ for details.

FreeBSD11 build failed! See https://ci.trafficserver.apache.org/job/freebsd-github/1701/ for details.

AU check failed! See https://ci.trafficserver.apache.org/job/autest-github/5/ for details.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/132/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1596/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/264/ for details.

It looks Http2ClientSession::PROTO_TAG is empty to me. I see two spaces between "http/1.1" and "tls/1.2".

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/16/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/29/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/15/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/142/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/1606/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/1711/

@maskit I'll look at that, but zwoop's test output has the 'h2' in it.

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/274/

I can see "h2" if I build it on Linux with gcc but not on Mac with clang.

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/22/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/35/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/21/

Linux build failed! https://ci.trafficserver.apache.org/job/linux-github/1612/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/22/

Linux build failed! https://ci.trafficserver.apache.org/job/linux-github/1613/

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/23/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/36/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/1717/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/147/

Intel CC build failed! https://ci.trafficserver.apache.org/job/icc-github/148/

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/25/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/38/

Intel CC build failed! https://ci.trafficserver.apache.org/job/icc-github/150/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/23/

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/26/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/39/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/1718/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/1614/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/25/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/279/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/151/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/1720/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/1721/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/280/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/1616/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/282/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/283/

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/31/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/44/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/30/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/1726/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/1621/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/156/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/288/

@bryancall says StringView should be changed to be conformant with string_ref from C++17. This would involve changing the method naming scheme and adding some additional methods to cover those in string_ref.

Squashing and verifying.

RAT check failed! https://ci.trafficserver.apache.org/job/RAT-github/53/

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/40/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/38/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/1735/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/164/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/1630/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/296/


Images are broken

Description:
Currently, the image links in the documentation is broken. Seems like it is broken with the change https://github.com/wso2/docs-choreo-dev/pull/32
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Fixed via https://github.com/wso2/docs-choreo-dev/pull/50


chore(demo-site): load font-awesome from Design Library site

:tada: This PR is included in version 4.0.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Thermocycler: Driver: Initiate firmware update

overview
As a robot user, I would like to have the TC driver be able to initiate a firmware update via gcode

Not MVP


localeData().monthsShort() throw exception in es, es-do, nl, nl-be and fy

Previously reported in this issue but they're two separate issues.
localeData().monthsShort() appears to be broken for:

es
es-do
nl
nl-be
fy

Both m and format are undefined when getting the locale data:
    monthsShort : function (m, format) {
        if (/-MMM-/.test(format)) {
            return monthsShort[m.month()];
        } else {
            return monthsShortDot[m.month()];
        }
    }

One or more localeData methods is also broken for:

be
el
hr
hy-am
ka
lt
pl
ru
uk


Inclusion of GitHub Dependencies in "GitHub Data from Pedigree" Data Source

@pvanoverbeek commented on Thu Jun 22 2017
Looks like GitHub has a nifty new feature to map dependencies between tickets:

We should definitely begin using this and ensure that the dependency relationships established there are reportable in our master data source. This would really be the cherry on top for GitHub data to allow for complete visibility into our queue to enable better decision making and prioritization.

Waiting for Github to include this in their api (v 3+)

Issue moved to tableau-mkt/marketing-etl #196 via ZenHub


Facilitate sass port, and better ide support

Could you consider dropping usage of .loadUIOverrides and other dynamic imports in theme.less, and hard code imports instead. It would facilitate porting to sass (sass/sass#739 would no longer be required). And it would also allow better ide support (inspection, code navigation...)
Suggested implementation:
1/ move site imports from theme.less to definition files.
2/ move component theme imports from theme.less to site files
To change a component theme, one would change the import statements in site files. It would be a little more messy but the benefits seem worth it.

Precision: in order to get rid from loadUIOverrides, theme import could be moved to the beginning of @{siteFolder}/<EMAIL_ADDRESS>files (theme.config could then be dropped)

Great suggestion. I'd love be able to choose whether i want to use less or sass.

Piling on. Sass support is a must for me, and I'd choose Semantic UI but for that. Please?

Hard coding imports would prevent themeing which is a major feature for Semantic UI.

According to my tests, theming is possible. But it should happen in site files, not theme.config. Eg:
1.breadcrumb.overrides could start with:
/*-------------------
Theme Overrides
---------------------*/
@import (optional) "../../themes/default/collections/breadcrumb.overrides";
/******************
Site Overrides
*******************/

2.breadcrumb.overrides could start with:
/*--------------------
Global variables
---------------------*/
/* nb: component theme */
@import (optional) "../../themes/default/globals/site.variables";
/* site */
@import (optional) "../../_site/globals/site.variables";
/*-------------------
 Component variables
---------------------*/
/* default theme */
@import "../../themes/default/collections/breadcrumb.variables";
/* nb: Component theme */
@import (optional) "../../themes/default/collections/breadcrumb.variables";
/******************
Site Overrides
*******************/

It's a little messier as I said but I find it worthy. There may be however other implications I'm not aware of.

Maybe theming is a thing that could be done outside the sass compiler. Maybe there is a nice possibility to include this into the build process? So configuration could be done in a JSON and a smart build could choose the files that are needed for building the themed version out of the JSON.

+1 @DaSchTour, it's looks like all porting process stops because theming couldn't be implemented like in less. Maybe we need use more common way to make theming (e.g. build process)?

Seems like hardcoded themes could work if symbolic links were used?


[VS Code] Starting collaboration session: Could not connect to the server. HTTPS connection failed, verify there are no self-signed certificates or custom cert validation policies blocking Live Share.

Error:
Starting collaboration session: Could not connect to the server. HTTPS connection failed, verify there are no self-signed certificates or custom cert validation policies blocking Live Share.
Steps to Reproduce:
1.
2.

Version Data

extensionName
VSLS

extensionVersion
1.0.419

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.35.1

platformName
Linux

platformVersion
4.19.0-kali5-amd64

It looks like it is a duplicate dupe of https://github.com/MicrosoftDocs/live-share/issues/2216#issuecomment-501372336 . Please follow the instructions there to get unblocked. Thanks for using Live Share and please keep the feedback coming!


Unable to size window dimensions with setSize

For a minimal repro, try this:
zig-webui/src/examples/call_js_from_zig
pub fn main() !void {
    var nwin = webui.newWindow();

    _ = nwin.setSize(10, 10);     // try resize before show

    _ = nwin.show(html);

   _ = nwin.setSize(10, 10);     // resize after .show

    _ = nwin.bind("MyButton1", my_function_count);
    _ = nwin.bind("MyButton2", my_function_exit);

    webui.wait();

    webui.clean();
}

In neither case does nwin.setSize(...) actually change the window size.
I'm not sure if this is a bug with this library (which essentially wraps WebUI) or with WebUI itself, but I'm posting it here just in case.

@hassandraga Honestly it might be browser related. After it failed in the console, I found this discussion:
https://stackoverflow.com/questions/65260529/how-to-resize-a-html-browser-window-with-javascript-using-chrome
Apparently you're (in many cases) only allowed to modify windows the size of windows opened with window.open(), as a security measure.

em.. linux chrome is available with zig

I will still test it using other browser.
To be honest, this function doesn't even do wrapping, it just converts the integer type

weird, setsize is not available with firefox
@hassandraga maye we should test it with C on firefox

I have test it with C on firefox, it really doesn't work

I will open an new issue in webui  repository.


⚠ ClamAV has degraded performance

In 511cb79, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 322 ms

Resolved: ClamAV performance has improved in fcaaa9d after 5 minutes.


Disable forwarding of splunk-internal logs

The majority of data ingested is currently from splunk-internal logs

This change disables all forwarding of splunk-internal logs except those needed for version tracking. It has been tested on running pods in production to alleviate pressure on indexers, last month and again last week

I have not reviewed this for content. However...
/hold
This definitely needs to go after #6. Otherwise it will not cause new images to be produced.

/approve

/hold cancel
dependency has merged

Looking over this, I am not sure we want it exactly like this, people using this container may way those logs sent over, Is there a way to do it based on an ENV var on startup(Default ship those logs, but option to disable it by some method that we can expose in the operator)?
Basically as pod starts up it checks to see if something like SHIP_INTERNAL_LOGS is set to false, if it is it copies over the inputs.conf to the splunk directory, or something else.

Basically as pod starts up it checks to see if something like SHIP_INTERNAL_LOGS is set to false, if it is it copies over the inputs.conf to the splunk directory, or something else.

This part's done - what's the best way to get an environment variable in the uf/hf pods? Does it need to go into the operator/CRD or is there another way?

This part's done - what's the best way to get an environment variable in the uf/hf pods? Does it need to go into the operator/CRD or is there another way?

This looks like one of those cases where you would provide the conf file via a config map that sfo would mount into the sf/shf container.


graphql_ppx is in peerDependency, devPendency and ppx-flags of root project

I don't quite remember if it's being used, but if it's not, it would be better to remove it as it causes conflicts when using the new recommended library graphql_ppx_re

Fixed by #124.


Please slow down the update frequency

Please slow down the update frequency! For beginners, the files downloaded before will not work in a few days! Have you considered the sense of experience?

Hi @pyp123,
CesiumJS typically releases once a month. 1.105.1 was an out-of-band release outside of the normal frequency.
CesiumJS is a big project, so we use GitHub for feature requests and bug tracking exclusively. In the future, please take any questions to the Cesium Forum where there are members of the community and developers from the team who can help. Thanks!


[firebase_messaging]: add additional documentation for the data in notifications

Description
In the documentation of the firebase_messaging plugin could have a more detailed section about accessing the data send alongside the notification. It would help not to waste time debugging and finding the errors if the documentation would explain it a bit better..
Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See [Contributor Guide]).
[ ] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (flutter analyze) does not report any problems on my PR.
[ ] I read and followed the [Flutter Style Guide].
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
[ ] I updated CHANGELOG.md to add a description of the change.
[x] I signed the [CLA].
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

I signed it!
:)

i am the only author of the change. Maybe the problem is that i pushed it using another computer than normal and therefore commited strange author informations..

Thanks for the contribution.
I'd like to consider making some breaking changes to allow more re-use of code across platforms, but I maybe for now documenting the current behavior is better than having it be confusing.
Can you please fix the analyzer failures?

@collinjackson done


Mypy annotation zerver.lib.cache helpers

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

Merged, thanks @medullaskyline!


more robust statement defining global based on browserify (#50)

though ordering has been switched to make the more likely use case come first

Maybe better something Add support for web worker global?

reworded

ok added

Great! Thanks a lot.

publish a new version ?


ROI Implementation

This is a PR for the implementation of ROI (see https://github.com/PX4/Firmware/issues/4584).
The gimbal driver will be implemented at a later stage (and a separate PR)
Progress:

[x] Implement the MAV_ROI enum in the vehicle_command topic
[x] Create a new uORB topic for the vehicle ROI (named vehicle_roi)
[x] Implement the parsing of MAV_CMD_NAV_ROI and MAV_CMD_DO_SET_ROI in commander.cpp and the advertising of the ROI on the vehicle_roi topic
[x] Rename the current gimbal driver to something else like RC_Gimbal

This should be good to go, please confirm. I can do some testing once I have implemented the mount (gimbal) driver.

Superseded by https://github.com/PX4/Firmware/pull/4624.


[mesheryctl] Display Loadbalancer IP for mesheryctl system status

…atus
Signed-off-by: ashish<EMAIL_ADDRESS>Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@leecalcote  After this I can use mesheryctl instead of kubectl in one more place on the core workflow

Codecov Report

Merging #4791 (27f5803) into master (0dcf288) will decrease coverage by 6.85%.
The diff coverage is 0.00%.

:exclamation: Current head 27f5803 differs from pull request most recent head 1230b38. Consider uploading reports for the commit 1230b38 to get more accurate results

@@            Coverage Diff             @@
##           master    #4791      +/-   ##
==========================================
- Coverage   32.01%   25.15%   -6.86%
==========================================
  Files          61       61
  Lines        5148     5159      +11
==========================================
- Hits         1648     1298     -350
- Misses       3074     3543     +469
+ Partials      426      318     -108

Flag
Coverage Δ

gointegrationtests
?

unittests
25.15% <0.00%> (-0.06%)
:arrow_down:

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (-47.44%)
:arrow_down:

mesheryctl/pkg/utils/platform.go
13.85% <0.00%> (-0.15%)
:arrow_down:

mesheryctl/internal/cli/root/system/start.go
1.15% <0.00%> (-58.47%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
2.50% <0.00%> (-47.50%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.05% <0.00%> (-34.74%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
1.78% <0.00%> (-23.22%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.38% <0.00%> (-22.23%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
3.29% <0.00%> (-20.88%)
:arrow_down:

Continue to review full report at Codecov.

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

@Revolyssup Why an additional function? Shouldn't this be the IP used in the meshconfig file as well?

@Revolyssup Why an additional function? Shouldn't this be the IP used in the meshconfig file as well?

Tbh I looked at the meshkit function being used. And that's not quite correct. It assumes a lot of things. Plus the way it is used as well is not very correct. I can talk about it in some meeting later. But I wanted a quick fix so did this. For now I'm closing this PR


Will not let me select a location

Subject: Will not let me select a location
Description:
As above
Results:
As above
Backend version: develop-b3cee755
Frontend version:
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.88 Safari/537.36
Page: https://couchers.org/signup?token=gVcuJQiM-219rFA-TB3LWKAvBI7_NfigyEVMDByZPyw=
User:

not enough info or can't reproduce


Change mod directory to be the Ultimate Skyrim directory instead

Currently, the mod directory points to the "content" directory and assumes a full MO2 installation already exists.
This needs to be changed to reference the US directory (containing the content and installer directories).
This will prepare for the launcher automatically creating these directories and running WabbaJack

This isn't needed actually because the US directory can now just contain the MO2 files. There is no need for an "install" and "content" folder because the launcher can install and keep track of WabbaJack separately


Build audit task

We should be able to give some statistics to the users around test and documentation coverage for their models we should have some stuff around the following:

documented vs undocumented columns (per models + across the dbt project)
tested vs untested models (per model + across the dbt project.

Ideally we should present it a bit like a traditional test coverage a la pytest
users should do something like dbt-sugar audit [-m model_a] and get statistics about the model or all of the models in the debt project that is in scope.
Documentation Coverage

table
% coverage
undocument columns

table_a
90%
column_b

table_b
10%
column_a, column_b

=======
==========
==================

80%

Test Coverage

table
% coverage
untested columns

table_a
90%
column_b

table_b
10%
column_a, column_b

=======
==========
==================

80%

Caveat

When running audit on the entire project, we should not show the undocumented/untested columns
However, when we run audit on **a single model ** -m my_model then we can show the list of undocumented and untested
We should maybe offer an optional argument like --save-verbose-output filename that would output the full list to a file?

@virvirlopez check this very old issue on the dbt repo: https://github.com/fishtown-analytics/dbt/issues/589
It's about someone asking for test coverage. Maybe there are some good point in there we can implement.


Version 0.9.14?

I'm a little confused by the release of 0.9.14 here:
https://github.com/graphite-project/carbon/releases/tag/0.9.14
It looks like it is identical to the 0.9.12 release.

They appear to be substantially different to me: https://github.com/graphite-project/carbon/compare/0.9.12...0.9.14
Can you help set some more context about what you're seeing?

@erikanderson as @bmhatfield mentioned, there are tons of differences between the two releases. You're looking only at the tag, not compare view.


It is not clear from the Response.java documentation if head and body calls should be ordered

org.takes.Response documentation misses information whether head and body calls should be ordered or not. For example, if body call should be forwarded by head call or vice versa or those calls should be completely independent and it is safe to call them in whatever order.
If it depends on the implementation - it is also should be stated.

@ekondrashev milestone set to 1.0 (correct me if I am wrong)

@yegor256 this is yours, pls go ahead

@ekondrashev thank you for reporting this, I added 15 mins to your acc, transaction 57288754

@davvd assign someone else pls

@davvd assign someone else pls

@yegor256 -30 points to your rating

@davvd assign someone else pls

@yegor256 right, I will find someone else, no problem

@kpbird please pick this up, and keep in mind these instructions. Any technical questions - ask right here. The budget of this task is 30 mins. This is exactly how much time will be compensated, when the task is completed. Read about our Definition of Done

@bdragan it's yours now, please proceed keeping in mind our principles. Feel free to ask any technical questions right here in the ticket. The budget of this task is 30 mins. This is exactly how much time will be compensated, when the task is completed. Read about our Definition of Done

@ekondrashev PR #409 has been merged successfully. Please review the solution and close this issue if you find the solution acceptable.

@bdragan nice! thank you.

@elenavolokhova please, review this ticket for compliance with our QA rules

@davvd Looks good!

@davvd Looks good!

@elenavolokhova thank you :)

@bdragan 10 mins was added to the account of @elenavolokhova (for QA review), in transaction 68334673
Thank you, I have added 30 mins to your account in payment/transaction "68334683", time consumed: 167 hours and 33 mins
added +30 to your rating, now it is equal to +83


https://github.com/mochajs⁹

Is your feature request related to a problem or a nice-to-have?? Please describe.
A clear and concise description of what the problem is. E.g. I'm always frustrated when [...]
**De[clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
https://github.com/mochajs/mocha-examples

https://github.com/mochajs

https://github.com/mochajs/mocha-examples

https://github.com/mochajs/mocha-examples

https://github.com/mochajs

https://github.com/mochajs/mocha-examples


feat: break up api rendering

Break up the rendering of a packages documentation into multiple pages.
The root package page now renders the readme, and each type within the
packages API reference is rendered on its own page. Items within the
readme and types are still linkable via hash links that are scrolled to
by the PackageDocs component.
The rendering is broken up by splitting the incoming markdown on the
API Reference header and then on subsequent headers within the api
reference markdown string. These are then parsed into menu items and a
hash map of typeid -> type markdown content.
Adds unit tests for markdown parsing logic to ensure outputs remain
stable.
fix: #560

Yeah!
Curious- did you have a chance to benchmark the load time for large modules against the previous version?

@MrArnoldPalmer It should be very easy to adapt the performance test script to test and compare the performance before and after.

Yeah!
Curious- did you have a chance to benchmark the load time for large modules against the previous version?

Only with local builds, which is not a super accurate number because of dev assets instead of our optimized product assets. Though the difference was clear and that should stand.
I can run some comparisons in my account and report back.

@eladb @addihorowitz here are numbers comparing the ec2 package rendering performance before and after this PR.
Root documentation page rendering time with loading assets:
New: approx 1.5 seconds

Old: approx 14.5 seconds

These screenshots are the browser tools timeline showing the first frames that include rendered markdown content of the readme.
Here are also the lighthouse performance reports that you can view in your browsers dev tools for more detailed information. You can see that though performance is obviously greatly improved, the performance score in fact goes DOWN. This is (as far as we know) because the old version of the site takes so long to render that lighthouse and pagespeedinsights actually assume that it is done much earlier than in reality, and show first contentful paint around 2 seconds, and time to interactive on desktop around 5, though we know this isn't true.
ec2reports.zip


camera cant show

sorry, i know what happened. case close

Ha haa! Can you please add a fix here? Someone might find that useful.

Can you share it ?
I Have same Problem

Can you share it ?
I Have same Problem

@souleateryx can you explain what happened and how you solved it? I might have a similiar or same issue at #162

FYI: A possible solution might have been to use queries in the manifest like mentioned at #143


search should set exit status on success/fail

Currently search always return 0 no matter if the search finds any packages or not. A package search should return a 0 exit code if it succeeds and a non-zero exit code if it does't find any matches. This would allow putting logic in scripts to check the exit code of the search command.

Fixed in master, latest nightly build aptly_0.9~dev+180+g7925af9 contains the fix.


Error: Unable to initialize Hardhat Runtime Environment

Using the latest version (0.5.5) of the plugin on VSCode v1.72.2, I am receiving the below error after the IDE starts up. I am not receiving any linting information from the plugin, but other features like import autocomplete are still working.
I enabled verbose mode and it indexed all the solidity files in the project before the below error displayed.
Error: Unable to initialize Hardhat Runtime Environment
    at initialiseWorkerState (c:\Users\Joe\.vscode\extensions\nomicfoundation.hardhat-solidity-0.5.5\server\out\worker.js:1:15983)
    at initialiseWorker (c:\Users\Joe\.vscode\extensions\nomicfoundation.hardhat-solidity-0.5.5\server\out\worker.js:1:17449)
    at Object.<anonymous> (c:\Users\Joe\.vscode\extensions\nomicfoundation.hardhat-solidity-0.5.5\server\out\worker.js:1:17793)
    at Module._compile (node:internal/modules/cjs/loader:1165:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1220:10)
    at Module.load (node:internal/modules/cjs/loader:1035:32)
    at Module._load (node:internal/modules/cjs/loader:876:12)
    at Function.c._load (node:electron/js2c/asar_bundle:5:13343)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12)
    at node:internal/main/run_main_module:17:47

@joejordan Hi! normally this can happen due to one of the following reasons:

you are not on a hardhat project

node modules are not installed

loading the hardhat config fails

The loading process can be a bit fragile because it uses the local installation of hardhat to perform compilation (i.e. linting, compiler errors). To detect what's happening on your end, please run npx hardhat compile on your project and see if it works. It can give a hint on what's failing.

Ah, my mistake was assuming that having the plugin installed and editing .sol files would be sufficient to get it to work, similar to Juan Blanco's plugin. Maybe the title of the plugin, Hardhat for Visual Studio Code should have been enough to tell me that I would need an installation of Hardhat to get this working.
FYI after adding hardhat to my package.json, npx hardhat compile instigates the "Create a Hardhat Project" as I would expect it to in this situation.
I am working in a Foundry project and was hoping to give this plugin a try alongside my current workflow. It looks like my current roadblock is that Hardhat does not support library path remapping, best described in Issue 2527. Ideally I would want to point Hardhat to look for libraries in the /lib folder, and then respect the entries in remappings.txt.

That's the case as of right now. But fortunately, we are working on the next big release that's coming within a few weeks, which gives full support (analysis, navigation, linting, etc) both to "standalone" solidity files and foundry projects as well. That involves supporting remappings, solc version, etc, from foundry projects.
Separately, we are also about to release a hardhat plugin for giving support to mixed hardhat-foundry projects, which you may be interested in in the future in case you want to use add hardhat tools to your existing project.
Stay tuned!


Some signals not emitted on Godot 3.6

Hi, thanks for the plugin! I was testing it in godot 3.6 and the Snapshot signals are not emitted.
I have already tested the Sign-in functions and signals work fine, I have not tested the Leaderboards or Events, but trying to use Snapshot the signals do not work, but methods work, for example I can save snapshots or open the dialog popup to select a snapshot.

Hi! Thanks for using the plugin! I'm sorry to hear that signals are not working properly. Maybe it's because I haven't updated the plugin to Godot 3.6.
I don't have the bandwith to test it now, but maybe you can try with this version of the aar file  of the plugin, which I compiled with Godot 3.6 SDK. Please let me know if this fixes the issue!

I tried, doesn't work, but if I use load_snapshot directly it emits the signal, so the issue is with the native dialog used to select a snapshot

I see, now I know where's the problem. That part is implemented a bit different due to restrictions of the Godot Android SDK. I will try to get a look as soon as I can, but in the meantime, do you see anything relevant in the logs?
If you're curious and you know kotlin and want to give it a look, the flow in  the code of the plugin is:

You call the snapshotsShowSavedGames method
That goes to the SnapshotsProxy class, that calls the Google sdk here
That opens the dialog in your game, and when you select a Snapshot, it returns an Intent to the plugin, that is consumed in the onActivityResult method, which in turn, emits the snapshotsGameLoaded signal with the Snapshot.

You should see these logs in logcat while doing all of that:

Showing save games
Loading snapshot with name $fileName.

Let me know if you find anything else, I will keep you posted as well if I find anything!

Hi @dannygaray60 I've found the problem and fixed it! You have a new release of the plugin here. Hope it works now! Let me know if it was fixed so we can close the issue 😄

wow! that was fast, thank you so much! it's working well now! Tested achievements, snapshots, and sign-in, methods and signals working :D

I'm happy to hear it's working!


Speed up tests by re-using the database

Currently, each acceptance and integration tests use a new database. We use "CREATE DATABASE FROM template", but there is still a non-negligeable time used to setup an entirely new database for each test.
I think (without being 100% sure) a good number of tests would behave just fine if we ran them in a re-used database, in a transaction that we would rollback afterwards.
We should try having a new fixture that would reuse the same DB and rollback, and see.
An alternative strategy would a fixture that empties our 2 tables before and/or after each test, using either DELETE or TRUNCATE (though... I think I recall TRUNCATE can be slower in some cases).
Also, any kind of DB optimization disabling postgres "durability" features in tests is welcome in that PR.

https://docs.docker.com/registry/storage-drivers/inmemory/ ?

postgres.conf:
  fsync = off                # turns forced synchronization on or off
  synchronous_commit = off   # synchronization level; on, off, or local
  full_page_writes = off     # recover from partial page writes

I've note it as both "Good for: newcomers" and "Good for: Occasional contributors", depending on how far you're willing to go :)
Adjusting the docker-compose.yaml for better performance and not touching the code is a perfectly valid contribution !

I think (without being 100% sure) a good number of tests would behave just fine if we ran them in a re-used database, in a transaction that we would rollback afterwards.

The problem is that we cannot rollback the transaction. Our aiopg connector uses this to issue queries to the database:
https://github.com/peopledoc/procrastinate/blob/77e036337441129f48a021bc5b8e6a8978aa3a78/procrastinate/aiopg_connector.py#L249-L252
with await self.pool.cursor() acquires a connection from the pool and creates a cursor on that connection. The connection works in autocommit mode, meaning that the query sent to Postgres has immediate effect.
Bottom line is that we cannot have control on the connection (and transaction) our aiopg code uses for issuing queries to the database system.

I am currently looking at the TRUNCATE option…

I am currently looking at the TRUNCATE option…

With https://github.com/peopledoc/procrastinate/commit/49f0098bb610428ca2a4d7fabd13dc05af0541cb running all the integration tests takes ~7 seconds on my machine, while it takes ~34 seconds with the current master branch. So a 5x speed-up.

With https://github.com/peopledoc/procrastinate/pull/315, and the use of Postgres 12, we made our test suite run in ~30 seconds rather than ~1 minute. And with this, the use of TRUNCATE and ALTER SEQUENCE to kind of reset the database (https://github.com/peopledoc/procrastinate/commit/46d2b22ab84f8537412c92c929e156918e279ff8) does no longer significantly speed up the execution of the test suite. I can observe a small improvement, but I don't think it justifies switching to a less robust mechanism.

So, shall we close ?

So, shall we close ?

Yep, I think we can close.


Satisfy Table and Figure Requirements

From NT:

Tables and figures should be designed to fit one- or two-column widths (85 and 173 mm, respectively). The height of lowercase letters should not be less than 1.5 mm after reduction; thus, a figure or a table that must be reduced to one-third its original size should have lowercase letters at least 4.5 mm high. Tables must be numbered in Roman numerals in the order they are called out in the text. A title must accompany each table. Authors are strongly encouraged to submit figures in a high-resolution electronic format. Preferred electronic formats for figures include TIFF (line drawings at least 600 dpi, halftone or grayscale images at least 300 dpi); EPS (with embedded fonts); Postscript or PDF (with embedded fonts). Lines and rules in figures should be at least 0.5 points (half-point rules) with an absolute minimum 0.25 points. Figures must be numbered in Arabic numerals in the order they are called out in the text. A descriptive caption must accompany each figure.

Color figures will be placed online in the journal without an additional fee (print version will still be black and white). For black-and-white (grayscale) reproduction of color figures, a test printing should be done in black and white (light blue, yellow, and green neon colors are to be avoided) to ensure that no information is lost when printed in black and white. Two sets of figures can be submitted: one set for black-and-white printing and one set for the online color version. These need to be clearly marked. Figure captions should avoid reference to color. Should an author want color in print, prepayment of the costs associated with color print will be required, after obtaining approval from the journal editor (current cost estimated at $525/page).

pretty sure this is ok.


access to program file or ocr result?

Excellent dataset!
I'm from Music & Technology Lab in academic institute, and we're interested in analyzing the performance history of New York Phil using machine learning approaches.
Would it be possible that we can access the file (e.g., .jpeg, tiff etc) of programs, instead of preview using web browser?
And have you done OCR yet? Is it possible that we can get access to the results of OCR?
Thank you

Hello,
Apologies for the delay in my response.
We are interested in learning more about your research. Could you share more information with us? Please email us at<EMAIL_ADDRESS>at your convenience.
We have done OCR on most, if not all, of our Programs. In the browser you can access the OCR data by clicking the icon at the bottom right of the viewer (see screenshot below).
We are also planning to add IIIF support to the Digital Archives, which may be helpful to you. This will take us some time to implement, but we can let you know when it’s ready for testing.
-Bill Levay

Thanks for the reply.
I've sent details about our research to your email, and hope that we can potentially collaborate for the further exploration of the data!


how to increase the height of the image as well as segment.

want to set the height of image as well as segment to achieve below UI or similar to this.Help would be really appreciated.Thank you in advance

What is it that you want to accomplish? If the images are part of the segment and you have a vertical menu like that and every time you tap a segment you have images as well I would suggest using a TableView that expands. Maybe checkout XMExpandableTableView