Allow for passing sensitive settings through files

As in some images provided on Docker Hub allow for setting *_FILE environment variables for passing sensitive data like passwords.
Example from PostgreSQL https://github.com/docker-library/postgres/blob/master/docker-entrypoint.sh#L8

This is already possible in a way.
You can specify any neo4j configuration option via environment variables (https://neo4j.com/docs/operations-manual/current/installation/docker/#docker-environment-variables) and docker run supports --env-file argument to set environment variables from a file.
(https://docs.docker.com/engine/reference/commandline/run/#set-environment-variables--e-env-env-file)
The security implications seem slightly better here than in the approach taken by postgres:

Host environment: You have a file with the password in it (both approaches)
Docker environment: You have an environment variable with the password in it (both approaches)
Docker environment: You have a file with the password in it (only postgres approach)

but I might be missing something else. I'm not sure what

[...] Docker's secrets feature)

is referring to and how that operates.

Yeah, I was thinking about --env-file but my current setup doesn't allow me to do it easily.
I'm using Rancher and their Secrets (https://docs.rancher.com/rancher/v1.6/en/cattle/secrets/) implementation allows only mounted /run/secrets/mysecret (inside container) files at the moment.
Kubernetes (https://kubernetes.io/docs/concepts/configuration/secret/) has both options for file + environment variable so it will be working even now.
Ideal solution for me would be for images to allow passing sensitive data through files mounted by docker secrets. This would lead to situation when the only place sensitive data is visible is inside docker container through file (where file permissions could be used to further secure it).
Feel free to close this issue. I'll write a wrapper that will suite my needs for now. Hopefully some standard way of dealing with secrets will emerge soon.


Bugfix for GlyphLayout.wrap

GlyphLayout: wrap:
(Bug only visible with Align.left and wrapped lines)
On wrap the xAdvance of the seconds line should be 0.
The xAdvance of the second+ lines is aligned with that of the first line, so it should be 0.
In BitmapFontTest it can be observed:
Adjust text to be Align.left, the top left text should look like this:
Sphinx of black
quartz, judge my
vow.
The second and third line are drawn 1 unit too far left because of:
GlyphLayout line 322:
xAdvances2.set(0, -glyphs2.first().xoffset * fontData.scaleX - fontData.padLeft);
-glyphs2.first().xoffset * fontData.scaleX - fontData.padLeft  = -(-19)*1 - 20 = -1
The second and third line start -1 units left of the first line.
It gets more obvious with bigger fontData.scaleX or not nearly equal glyphs2.first().xoffset and fontData.padLeft

Updated initial pull request message to make it easier to reproduce the bug

Updated initial pull request message to make it easier to reproduce the bug

An executable example would make it easy to reproduce. Since it was not provided, I had to make one. Here is a modified BitmapFontTest. Move the mouse so the text wraps where you see the problem. I do not see a problem:

Consider this wrapping without your change:

And with your change:

Your change causes the j to be too far left.
The line of code you set to zero is aligning the glyph's pixel bounds so they start at the drawing position. The glyph pixels are not guaranteed to be to the right of the drawing position. Some fonts are more extreme with this than others.


NilPointer dereference on BlobIndexTag deletion

Hello,
in the latest versions removing all BlobIndexTags leads to a Panic due to a built-in incompatibility which was introduced with https://github.com/Azure/azure-sdk-for-go/commit/5e4becb50868a1ac3a825313e273b0716e687615
// SetTags operation enables users to set tags on a blob or specific blob version, but not snapshot.
// Each call to this operation replaces all existing tags attached to the blob.
// To remove all tags from the blob, call this operation with no tags set.
// https://docs.microsoft.com/en-us/rest/api/storageservices/set-blob-tags
func (b *Client) SetTags(ctx context.Context, tags map[string]string, options *SetTagsOptions) (SetTagsResponse, error) {
	serializedTags := shared.SerializeBlobTags(tags)
	blobSetTagsOptions, modifiedAccessConditions, leaseAccessConditions := options.format()
	resp, err := b.generated().SetTags(ctx, *serializedTags, blobSetTagsOptions, modifiedAccessConditions, leaseAccessConditions)
	return resp, err
}

Calling it now without any tags set (empty map) the  shared.SerializeBlobTags(tags) returns nil which will be dereferenced on line 4.

Hi @daniel-meyer-dme. Thanks for reporting this issue. We have merged the fix and it will be part of our next release.


OpenTok library not found problem

Hi, I am developing an app using phonegap. In the app I need to use Twilio + OpenTok SDK for iOS and Android. The thing is I cannot make it to work with both plugins at the same time. I think there may be conflicts between libraries. Because when I only use openTok it works perfect.
Any suggestions?

I am developing a phonegap app for android and iOS. I am also using Twilio and OpenTok SDK. After struggling some time I could make it both SDK to work in IOS. There was a problem with openssl libraries, because both sdk use it. However, When testing on Android, I have linker problems. Linker is not able to find opentok.so, in presence of twilio-native.so. After removing twilio references, it all works correctly.
Need help asap.
Thanks in advance.


Rework the Operator to call Grafana API directly

Having deployed the Grafana Operator I am concerned about its architecture for future releases.
Most problematic would probably be the grafana-dashboards ConfigMap which in my case already is stuffed with 20 objects.
Every time some dashboard changes the deployment gets re-deployed entirely, which might not preferred when deploying often and by different teams.
I propose to have the Operator read GrafanaDashboards CustomResources and when reconciling directly call the Grafana API and add dashboards that way. It should work similarly for datasources and other things too.
Let me know what you think.

The deployment should only get redeployed if plugins have changed because Grafana can currently not reload plugins at runtime. If you update a dashboard without plugins or not modify the list of plugins then no redeploy should happen.
But I agree with you that we should avoid importing all dashboards into a config map. I'm concerned that at some point we reach a size limitation on config maps as dashboards can get pretty large. I'll try to look into this as soon as I get a chance.

If you update a dashboard without plugins or not modify the list of plugins then no redeploy should happen.

Ok, then it redeployed for another reason for me.

I'm concerned that at some point we reach a size limitation on config maps as dashboards can get pretty large.

That's exactly why I also bring it up here. We have hit that limit with https://github.com/brancz/kubernetes-grafana and now deploy every dashboard as separate ConfigMap which would cause a rollout every time.
I am happy to take how far I can get with a proof of concept! 😊

We approached the size issue by using multiple config maps. Each config map has its own dashboard provider. Our fork of this operator distributes the dashboards over the config maps. Thus we only need another rollout, if the number of dashboard config shall decrease / increase.
After some more testing I will contribute the changes in a PR. Though I like metalmaze's approach to use the Grafana API more than ours.

@lukasgr90 Yes, that exactly the same approach that we use with kubernetes-grafana :)
I have a very very instable first version of this working and currently fight with the Authorization Bearer not working, but that should be solvable.

@metalmatze that sounds great! If you'd like to send a PR i'd be happy to help you get it over the line.

@metalmatze The new release (https://github.com/integr8ly/grafana-operator/releases/tag/v3.0.0) is now using the Grafana API directly and brings some other improvements (e.g. no more finalizers on dashboards and data sources). Would you mind giving this another try?


Support for --ext-code

I'm curious if there would be support for adding a jsonnet.extCode setting to generate --ext-code command line arguments.  In my case, I rely on non-string std.extVar values, so I cannot configure the extension to populate them with the jsonnet.extStrs setting.  Ideally, I could have settings.json like this:
{
  "jsonnet.extCode": {
    "objVar": {
      "foo": "bar"
    }
  }
}

Then the extension would not generate errors with jsonnet like this:
local obj = std.extVar('objVar');

{
  foo: obj.foo
}

I also wonder if there is any workaround to disable the linting in jsonnet. I want to ignore the error raised by std.extVar.

Perhaps std.extVar(whatever) should be treated as "any value" by the vscode extension by default, when extVars are not specified.

Any of these options would be great. So many false error messages cluttering up the code when working with many different jsonnet files. The ability to disable the error checking would be phenomenal

Is there anyone working on this issue? If not, I can try to implement it and send a PR maybe?


Can you update template tunnel project ?

Can you update template tunnel project

Sorry, I don't understand what you mean. Could you be more specific?


Installing or removing Lens metrics crashes

Describe the bug
Trying to change the installation of lens metrics results in a console error of cannot read 'getPath' of undefined. This is a regression from #4653

How to reproduce?

Could not reproduce again


feat: port rich text tracking events

Closing it in favor of https://github.com/contentful/field-editors/pull/1027


Performance Testing

Create JMH perf tests.
Right now we have JMH 0.9 so we need to upgrade that: https://github.com/ReactiveSocket/reactivesocket-java/blob/e38b757ca02e597712139dfb9d01ba5dc7079c40/src/main/java/io/reactivesocket/ReactiveSocketClientProtocol.java#L132

Done.


Prepare 0.9.0 release

This commit starts the preparation for the 0.9.0 release. It moves all
the release notes for the release into a separate folder (to distinguish
them from notes for future releases) and also reworks the content of the
release notes and updates the documentation for the release. This commit
should be the last one merged before releasing 0.9.0 to ensure we've
moved and updated all the release notes before the release.
Fixes #328

Pull Request Test Coverage Report for Build 883143126

0 of 0   changed or added relevant lines in 0 files are covered.
17 unchanged lines in 1 file lost coverage.
Overall coverage increased (+1.1%) to 96.654%

Files with Coverage Reduction
New Missed Lines
%

src/lib.rs
17
99.31%

Totals

Change from base Build 882184262:
1.1%

Covered Lines:
8146

Relevant Lines:
8428

💛  - Coveralls

Leaving this as on hold until we're ready to release 0.9.0 since we'll want to merge this last

@ajavadia updated the class docstrings as part of this PR to make how node and edge indices work a bit clearer. Let me know if I need to tweak it more.

Ok, all the tracked PRs have merged so this is unblocked. Once this is approved and merged I'll tag main as the 0.9.0 release.


Fix #639

Resolves https://github.com/IronLanguages/ironpython2/issues/639
@jacks-bush Is this something you can test with your application? The fix is in https://github.com/IronLanguages/ironpython2/commit/2e8aa226427ea602f6f1567462ad86cda5641fa5 the rest is just cleanup.

@slozier I think this is something I should be able to test with our application.

@slozier This fix appears to work with our software.


Grocery outlet

some typos and sentence structure suggestions:  About us: We are brother And sister Djibia And should be small a;
Our journey with grocery outlet should be either a or the; Haoua started as Heath should be a health and Beauty Department Head and Djibia as security guy should be as a. Both worked We both worked our way up to management. March 2017 should be in March we decided to take it to the next Step should be small s and began our journey as independent Owners of Grocery Outlet.On July 2017 we got our 1st store in La Habra Grocery Outlet up to January 2020. should be From July 2017 new sentence In January 2020 we got our current second Grocery Outlet store in Irvine until present.


Added IllegalPathError

Created a IllegalPathError struct, as well as a the IsIllegalPathError helper function.
Note that the struct has an additional field, AbsolutePath, that is not used for printing, but might be useful for for debugging purposes to library users.

Thanks @ibraimgm. This looks good to me.
There is one suggestion I have, but it's not a blocker:
Rather than checking the string, I believe you should be able to directly check the type:
_, ok := err.(ErrInvalidPath)
return ok

What do you think?

I'm going to go ahead and merge this in as-is so that it's not blocking others. I'll open a new issue for using type checking over string checking.


Add type hints to be compatible with symfony/serializer on PHP8.1+

Fixes #9
composer test passes cleanly against Laravel-10 on PHP-8.3 with these changes

lol look how naive I was an hour ago when I opened this PR. Yes these changes were enough to get the unit tests to pass and my code to instantiate a new Client, but the same issue is present in all the files under Api/Normalizer. Since these files are generated by Jane anyway, probably best to just re-generate everything instead of hand-edit.

OK I didn't have docker readily available to me on the remote host I am working on, so was unable to run generate.sh.
HOWEVER, I was able to apply the necessary syntax changes to all the files in Api/Normalizer using this sed command:
sed -i -e 's#public function denormalize($data, $class, $format = null, array $context = array())#public function denormalize(mixed $data, string $type, ?string $format = null, array $context = []): mixed#' \
    -e 's#public function normalize($object, $format = null, array $context = array())#public function normalize(mixed $object, ?string $format = null, array $context = []): \\ArrayObject|array|string|int|float|bool|null#' \
    -e 's#array()#[]#' \
    *.php

I am going to push a commit with the results, as that seems to be working fine for me to be productive in my project. Though I fully understand if you would prefer to use Jane to regenerate these files and decline this PR.

...and then this command has to be run to hit all the files under src/Normalizer/ ☹
sed -i -e 's#public function denormalize($data, $class, $format = null, array $context = \[\])#public function denormalize(mixed $data, string $type, ?string $format = null, array $context = []): mixed#' \
    -e 's#return parent::denormalize($data, $class,#return parent::denormalize($data, $type,#' \
    *.php */*.php

Hi, thanks for your contribution.
I've just released version 2.0 which should be compatible with Symfony 6 and 7, there were differences in interfaces between the two versions, but Jane handles that now. Thanks!


TWatch-2020-V1 won't boot after firmware update

Tried 0.9 alpha in both Windows 10 and Ubuntu 20.04.
When I run Firmware > Update (Clear device flash), the watch won't boot any more.
Tried with no views at all and with 1 basic view, with same results:

Erasing flash ...
Serial port /dev/ttyACM0
Detecting type...
Connecting

After that, nothing, the watch won't boot.

Ok, so I followed this steps (both linux and windows):

Run TWatch SK Designer as sudo/admin
Click on Firmware > Update
Select the proper port (there's only 1, so i assume it's ok)
Checked "Clear device flash prior upload"
Click "Upload" button
Wait for the process to finish
Wait up to 5 minutes while looking at the watch screen

Results:
· Nothing is displayed on the watch during the entire process
· Nothing seems to happen after 5 minutes
· Quick press the watch button does nothing
· Long press the watch button does nothing
· Double tap watch screen does nothing
To make sure the issue wasn't on the device itself, I tried uploading another image (not SK related) with platformio without issues, so I guess Watch, cable, and usb port is fine.
Let me know if I understood correctly what you asked me to test and if there is any further info I can provide.

So, I cloned the repo and test with the latest commit, in debug mode, and this is the output, in case it helps:

PROGRESS: Checking if esptool is downloaded...
PROGRESS: esptool exists at C:\Users\user\AppData\Local\TWatchDesigner\Esptool\esptool.exe...
PROGRESS: Downloading firmware from https://dev.dytrych.cloud/twatchsk.zip
PROGRESS: Extracting firmware archive...
PROGRESS: Erasing flash...
PROGRESS: esptool.py v3.2
PROGRESS: Serial port COM6
PROGRESS: Connecting....
PROGRESS: Chip is ESP32-D0WDQ6-V3 (revision 3)
PROGRESS: Features: WiFi, BT, Dual Core, 240MHz, VRef calibration in efuse, Coding Scheme None
PROGRESS: Crystal is 40MHz
PROGRESS: MAC: 08:3a:f2:66:3d:50
PROGRESS: Uploading stub...
PROGRESS: Running stub...
PROGRESS: Stub running...
PROGRESS: Erasing flash (this may take a while)...
PROGRESS: Chip erase completed successfully in 29.3s
PROGRESS: Hard resetting via RTS pin...
PROGRESS:
PROGRESS: Uploading new Firmware version 0.9...
PROGRESS: esptool.py v3.2
PROGRESS: Serial port COM6
PROGRESS: Connecting....
PROGRESS: Detecting chip type... Unsupported detection protocol, switching and trying again...
PROGRESS: Connecting....
PROGRESS: Detecting chip type... ESP32
PROGRESS: Chip is ESP32-D0WDQ6-V3 (revision 3)
PROGRESS: Features: WiFi, BT, Dual Core, 240MHz, VRef calibration in efuse, Coding Scheme None
PROGRESS: Crystal is 40MHz
PROGRESS: MAC: 08:3a:f2:66:3d:50
PROGRESS: Uploading stub...
PROGRESS: Running stub...
PROGRESS: Stub running...
PROGRESS: Changing baud rate to 800000
PROGRESS: Changed.
PROGRESS: Configuring flash size...
PROGRESS:

I waited for ~10 minutes, but it stayed there

When I start the project, the file C:\Users\user\AppData\Local\TWatchDesigner\Firmware\firmware.json is created with the arguments:

"arguments" : "--baud 800000 --before default_reset --after hard_reset write_flash -z --flash_mode dio --flash_freq 40m --flash_size detect"

The baud rate won't work, but if I modify it it to 112500 before flashing, everything runs as expected.

Sorry, I was little confused, from your description, I thought the issue was after firmware upload is completed.
I'm glad you've figured it out. Baudrate 800000 works fine for our 3 development watches, but I could change json file to lower the baud rate, no issue here.
Could you please test 400000 bauds for upload if it works for you?
Maybe there is some HW revision that is using different USB to UART chip that doesn't handle high speed transfer.
Please let me know!
Thanks

Tested with 400000, and it works fine too

OK, I've changed to 400000 bauds!
Thanks for testing TWatchSK!


PowerToys DSC Resource

Description of the new feature / enhancement
I would like a PowerShell DSC Resource for configuring PowerToys to my preferences.
Scenario when this would be used?
When I want to use PowerShell DSC to configure an environment, I would like to be able to apply the configuration for PowerToys in that environment.
Supporting information
Related to:

https://github.com/microsoft/winget-cli/issues/2845

Desired State Configuration v3

This has been worked on during the 0.80 release cycle.
Please update to the latest release: https://github.com/microsoft/PowerToys/releases


Add STAT version to exception messages

Fixes #87

I've had a few situations come up where I am getting exceptions sent to me to troubleshoot and while the exception is clear, it's unclear which version of STAT it's happening and in a couple cases it's been bugs that were already fixed.  This adds the installed version to the exception information.
I also refactored the code that checks the local version out of the base module and into the data module so it was accessible regardless of which module throws the exception
This includes an update for the test cases to test the new function and ensure it returns a version correctly

I'm not sure why the actions didn't run on this PR.. going to try again with a new one


align failed

mydata.zip
I used Nimbus to analyze my data, with a bed file and pair end fastq data, using the hg38 reference.
I checked my fast data, the read ends could match the bed file.
Whereas, after running Nimbus, I get error information: Loaded 0 bases in 0 amplicons.
it seems the alignment failed. #2

110882 + 0 in total (QC-passed reads + QC-failed reads)
0 + 0 secondary
0 + 0 supplementary
the flagstat report:
0 + 0 duplicates
0 + 0 mapped (0.00% : N/A)
110882 + 0 paired in sequencing
55441 + 0 read1
55441 + 0 read2
0 + 0 properly paired (0.00% : N/A)
0 + 0 with itself and mate mapped
0 + 0 singletons (0.00% : N/A)
0 + 0 with mate mapped to a different chr
0 + 0 with mate mapped to a different chr (mapQ>=5)

It seems that the tools does not support hg38, please use hg19.

Dear booew,
The standard GRCh38 reference sequence includes text after the chromosome qualifiers. For example ">chr1  AC:CM000663.2  gi:568336023  LN:248956422  rl:Chromosome  M5:6aef897c3d6ff0c78aff06ac189178dd  AS:GRCh38"
Everything after the chrN should be removed from the header rows.
This can be done with the following code:
cat Homo_sapiens_assembly38.fasta | sed 's/ +.*$//g' > Homo_sapiens_assembly38.clean.fasta
With the Homo_sapiens_assembly38.clean.fasta file, 99 amplicons can be loaded.
I assume the design that attached to the message is not complete as only 36 reads aligned. Otherwise the amplicon design could be hg19 (the standard genome at Agilent) while the reference is GRCh38.
with kind regards,
Rutger


Login - "Remember Me" brakes login after closing/opening the browser

Steps to reproduce:

Login setting "Remember Me"
Close the browser
Reopen the browser

Error only applies to Windows version 22H2.

Cannot replicate on macOs or Linux
Browser Chrome version 125.0.6422.142

Error is: TOO MANY REDIRECTS.
Deleting the REMEMBER_ME Cookie we can login again but the issue is not solved


[Feature Request] Pick multiple values from  and

Similarly to sd-dynamic-prompts, it should be possible to write syntax that allows <random:...> and <wildcard:...> to randomly pick more than one value from their respective lists.
Perhaps something like <random(2):cat,dog,fish> and <wildcard(2):animals> to pick two items at random, but not the same item twice.
Additionally, a range like <wildcard(1-3):animals> should be defined to randomly pick one to three values from the list.
For a more concrete example:
Prompt: <wildcard(2-4):scene_natural>, <wildcard:season>, <wildcard:time>
Output: forest, river, heavy clouds, autumn, evening
(The three forest, river, heavy clouds having been picked from scene_natural)
This feature would be useful for easily creating more diverse batches.

I spent a few days uncertain how to do this best, then decided to go ahead and add multiple variants.
First:
You can do <random:1-5> to pick a random number from 1 to 5.
You can do <repeat:3,cat> to get cat cat cat
You can combine things and do eg <repeat:1-3,<random:cat,dog>> to get eg cat dog cat
This is generic and can be mixed however you want to get whatever result you want.
This also allows for eg maybe in the future we go crazy and add math tags or whatever, this would automatically support that.
Second:
For the more specific case you're actually asking about, I used square brackets [] instead of () parens but otherwise works mostly how you were thinking.
You can do <random[2]:cat,dog,fish> and <wildcard[2]:animals> to get any two animals, eg cat dog
It uses spaces by default, if you prefer commas you can just add a , to the input, ie <random[2,]:cat,dog,fish> will get eg cat, dog
(The reasoning for this is that the value of commas in a prompt is a common debate, some people think they help, others say they don't... in modern XL they seem even less valuable than they did in SDv1... and even if you prefer them there's plenty of cases where you'd still not want them while otherwise combining random things. So space feels a smarter default in the long run, and this way you can easily choose to have them with no more than a single extra keystroke of the comma key)
You can of course also do eg 1-5 and it does of course also apply to wildcards.
Also
A notable distinction between option 1 and 2: <repeat:1-3,<wildcard:animals>> might give you cat dog wolf or cat cat cat, but <wildcard[1-3]:animals> will avoid repetition, ie cat cat cat won't happen.

Perfect, thank you! Good point about making the commas be optional.


New mini book:  OS setup for Fortran developement

Hi all, this PR is a follow-up to Issue #134.
The goal is to provide, in a beginners-friendly format, all the necessary steps to choose and install a compiler, as well as it includes information about text-editors and IDEs.
It is not complete and definitely needs "attention"... take it as a template to start building on it.
The part with the text-editors needs a few of you with experience with some of those editors to contribute with "how to configure them".
I am aware that some parts are opinionated, but the topic itself is subjective by nature... choosing compilers, text-editors, or IDEs. Let's discuss them and find a common path.

Nice to see this - I will have a closer look at the text.
Op di 6 okt. 2020 om 02:02 schreef smeskos<EMAIL_ADDRESS>
Hi all, this PR is a follow-up to Issue #134
https://github.com/fortran-lang/fortran-lang.org/issues/134.
The goal is to provide, in a beginners-friendly format, all the necessary
steps to choose and install a compiler, as well as it includes information
about text-editors and IDEs.
It is not complete and definitely needs "attention"... take it as a
template to start building on it.
The part with the text-editors needs a few of you with experience with
some of those editors to contribute with "how to configure them".
I am aware that some parts are opinionated, but the topic itself is
subjective by nature... choosing compilers, text-editors, or IDEs. Let's
discuss them and find a common path.
You can view, comment on, or merge this pull request online at:
https://github.com/fortran-lang/fortran-lang.org/pull/152
Commit Summary

add new _site to master
Revert "add new _site to master"
test
Revert "test"
Merge remote-tracking branch 'upstream/master'
Merge remote-tracking branch 'upstream/master'
Merge remote-tracking branch 'upstream/master'
Merge remote-tracking branch 'upstream/master'
added main index page
added second page: choosing compilers
added third page: installing gfortran
added fourth page: installing opencoarrays
added fifth page: texteditors
added sixth page: IDEs
added final page: Tips
added new minibook structure

File Changes

M _data/learning.yml
https://github.com/fortran-lang/fortran-lang.org/pull/152/files#diff-793049ea95168d552f25dbf988e2b6f3
(12)
A learn/os_setup/choose_compiler.md
https://github.com/fortran-lang/fortran-lang.org/pull/152/files#diff-3f295dde916c8c2704b83a487a362e44
(18)
A learn/os_setup/ides.md
https://github.com/fortran-lang/fortran-lang.org/pull/152/files#diff-2681dbd91fd0b562e296b3b4441cf8ec
(20)
A learn/os_setup/index.md
https://github.com/fortran-lang/fortran-lang.org/pull/152/files#diff-2da110db96cec689bc5276893e1f7924
(27)
A learn/os_setup/install_gfortran.md
https://github.com/fortran-lang/fortran-lang.org/pull/152/files#diff-b3e6b9920d8f42caf950376c2e530f2e
(85)
A learn/os_setup/install_opencoarrays.md
https://github.com/fortran-lang/fortran-lang.org/pull/152/files#diff-96fdd7a54c435d91e866957d4eac37c5
(16)
A learn/os_setup/text_editors.md
https://github.com/fortran-lang/fortran-lang.org/pull/152/files#diff-88cbca544ed22b504a0f0bf812b14f34
(54)
A learn/os_setup/tips.md
https://github.com/fortran-lang/fortran-lang.org/pull/152/files#diff-88f51d3e1e97db0cafb7b34ac5ba2ebd
(13)

Patch Links:

https://github.com/fortran-lang/fortran-lang.org/pull/152.patch
https://github.com/fortran-lang/fortran-lang.org/pull/152.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fortran-lang/fortran-lang.org/pull/152, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN6YR4CW2UVBDYNB4YMFNTSJJNCHANCNFSM4SFKITAA
.

#build_preview

The link at the preview doesn't work. I am now sure why. I've built it locally and the site appears fine.

#build_preview

@certik thank you for the review and your comments. I will try to address them one by one and discuss them. I would also like to repeat, that this is just a draft and it definitely needs improvements. Furthermore, I wrote about some topics even if I am not an expert, just because I believe, they should be addressed in this tutorial, with the expectation, that the more experienced developers in this community will help me correct them.

Thank you @milancurcic for this detailed review. Following your recommendation, I also added a link to a tutorial for update-alternatives.

#build_preview

@arjenmarkus Do you need more time to review this or it's good to go?

Let me read it now - I have some time at the moment.
Op di 20 okt. 2020 om 18:22 schreef Milan Curcic<EMAIL_ADDRESS>
@arjenmarkus https://github.com/arjenmarkus Do you need more time to
review this or it's good to go?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fortran-lang/fortran-lang.org/pull/152#issuecomment-712977104,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN6YR5QBCYPTXIRCFQE3NDSLW2MZANCNFSM4SFKITAA
.

I read through it, this is good enough to merge and we can improve it with future PRs. After @arjenmarkus approves, let's merge it.
I think our tools should support all editors, but I think we should ensure that at least one editor works first class.
I think we should pick an editor and provide tutorials with screenshots using it, and we should also write extensions to such an editor with fpm support, with formatting support, with Flang and LFortran support etc. Such an editor should work cross platform, and be in wide use. I think it should be VSCode. But as I said, we should write our tools in an editor independent way, but by ensuring that things just work in VSCode, we can simply direct new users to it, and make our lives simpler. And we can support other editors as we have time, but don't have to stress about it too much, since we have a nice default that will do the job for new users.

@smekos, I have just read the entire booklet in its present form. Nice work
:), but I do have a comment: MinGW-w64 and MSYS2.
First of all, under the threading Windows, MinGW-w64 is mentioned, but not
its relation to MSYS2.
Secondly. if you compile under MSYS2, then you get binaries that depend on
the MSYS2 environment, but if you use the MinGW-w64 environment, you can
freely distribute the programs. So, just the other way around.
Thirdly, the latest version, at https://github.com/msys2, gives you the GCC
compiler suite 10.2.
Regards,
Arjen
Op di 20 okt. 2020 om 21:04 schreef Arjen Markus<EMAIL_ADDRESS>
:

Let me read it now - I have some time at the moment.
Op di 20 okt. 2020 om 18:22 schreef Milan Curcic<EMAIL_ADDRESS>
:

@arjenmarkus https://github.com/arjenmarkus Do you need more time to
review this or it's good to go?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fortran-lang/fortran-lang.org/pull/152#issuecomment-712977104,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN6YR5QBCYPTXIRCFQE3NDSLW2MZANCNFSM4SFKITAA
.

@Ondřej Čertík<EMAIL_ADDRESS>see my comments - the text is fine as it
is, with just this inaccuracy/update for MinGW-w64/MSYS2. I particularly
focused on this, as it has confused me for years :).
Op di 20 okt. 2020 om 21:23 schreef Ondřej Čertík<EMAIL_ADDRESS>
:

I read through it, this is good enough to merge and we can improve it with
future PRs. After @arjenmarkus https://github.com/arjenmarkus approves,
let's merge it.
I think our tools should support all editors, but I think we should ensure
that at least one editor works first class.
I think we should pick an editor and provide tutorials with screenshots
using it, and we should also write extensions to such an editor with fpm
support, with formatting support, with Flang and LFortran support etc. Such
an editor should work cross platform, and be in wide use. I think it should
be VSCode. But as I said, we should write our tools in an editor
independent way, but by ensuring that things just work in VSCode, we can
simply direct new users to it, and make our lives simpler. And we can
support other editors as we have time, but don't have to stress about it
too much, since we have a nice default that will do the job for new users.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fortran-lang/fortran-lang.org/pull/152#issuecomment-713085855,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN6YR4J5ZMQBHLRNR4U76DSLXPTTANCNFSM4SFKITAA
.

Hi @arjenmarkus, the current text on MSYS2 and MinGW was written by myself. I will address your comments with an update. Can you be more specific about what you understand the relationship between MSYS2 and MinGW-w64 to be?
My understanding is that they are separate with MSYS2 providing one (of three) environments that is built atop MinGW-w64, hence why I have written that you can freely distribute programs. Yes, there is a 'native' MSYS2 environment that has it's own DLL dependency, however this environment is only recommended for developing tools for the MSYS2 .

Okay, minor comments/suggestions added.
Op di 20 okt. 2020 om 21:36 schreef Arjen Markus<EMAIL_ADDRESS>
:

@Ondřej Čertík<EMAIL_ADDRESS>see my comments - the text is fine as
it is, with just this inaccuracy/update for MinGW-w64/MSYS2. I particularly
focused on this, as it has confused me for years :).
Op di 20 okt. 2020 om 21:23 schreef Ondřej Čertík <
<EMAIL_ADDRESS>
I read through it, this is good enough to merge and we can improve it
with future PRs. After @arjenmarkus https://github.com/arjenmarkus
approves, let's merge it.
I think our tools should support all editors, but I think we should
ensure that at least one editor works first class.
I think we should pick an editor and provide tutorials with screenshots
using it, and we should also write extensions to such an editor with fpm
support, with formatting support, with Flang and LFortran support etc. Such
an editor should work cross platform, and be in wide use. I think it should
be VSCode. But as I said, we should write our tools in an editor
independent way, but by ensuring that things just work in VSCode, we can
simply direct new users to it, and make our lives simpler. And we can
support other editors as we have time, but don't have to stress about it
too much, since we have a nice default that will do the job for new users.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fortran-lang/fortran-lang.org/pull/152#issuecomment-713085855,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN6YR4J5ZMQBHLRNR4U76DSLXPTTANCNFSM4SFKITAA
.

Sure, I ran into this when I tried to understand the way MinGW-w64/MSYS2
handle Windows-style paths.
If you install MinGW-w64/MSYS2, you get a number of shells, each giving
access to their own versions of the various packages. To keep it simple, I
will describe what I understand regarding the GCC compiler suite:
mingw64.exe: a program compiled with the GCC compilers understands the
Windows-style path names, the home directory I get with "echo $HOME" is
"/home/markus", but if I ask in a C program for the current directory if I
am in my home directory, I get "c:\msys64-2020\home\markus".
I can copy the program to another machine or run it in a plain command
window without any problem.
msys2.exe: if I do the same under this shell, I have a different version of
the compiler and the run-time libraries. The result of getcwd() would then
be Unixy - "/home/markus". And the program does not work outside the MSYS2
environment, as it would miss a crucial DLL.
Op di 20 okt. 2020 om 21:50 schreef Laurence Kedward <
<EMAIL_ADDRESS>
Hi @arjenmarkus https://github.com/arjenmarkus, the current text on
MSYS2 and MinGW was written by myself. I will address your comments with an
update. Can you be more specific about what you understand the relationship
between MSYS2 and MinGW-w64 to be?
My understanding is that they are separate with MSYS2 providing one (of
three) environments that is built atop MinGW-w64, hence why I have written
that you can freely distribute programs. Yes, there is a 'native' MSYS2
environment that has it's own DLL dependency, however this environment is
only recommended for developing tools for the MSYS2 .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fortran-lang/fortran-lang.org/pull/152#issuecomment-713099601,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN6YRZ7S72BU2D6ERTU22DSLXSYJANCNFSM4SFKITAA
.

Thanks @arjenmarkus, that essentially aligns with my understanding. However it should be clear that the MinGW-w64 toolchain is separate to MSYS2 and can be distributed independent of it.
Further, when you use the native MSYS2 shell, the compiler toolchain is not MinGW, but an MSYS2 native build of gcc.
The text I have written makes the assumption that the reader will use one of the MinGW shells in MSYS2 as opposed to the 'native' MSYS2 shell which should only be used for writing programs to distribute in MSYS2.
To be clear, the following compile environments are available:

MinGW-w64: compiler toolchain only (no shell), links with Microsoft libs (native Windows), called from Windows shell
MSYS2/mingw64: MinGW-w64 compiler toolchain provided as an MSYS2 package, links with Microsoft libs, called from MSYS2 shell
MSYS2/mingw32: same as above with 32bit x-compile target
MSYS2/MSYS2: GNU compiler collection compiled within the POSIX environment provided by MSYS2, links with msys-2.0.dll

I will update the text tomorrow to make it simpler and clearer to a beginner.

@LKedward:

I will update the text tomorrow to make it simpler and clearer to a beginner.

Please try to do that.
I am also reading info from here msys2.org/wiki/History and trying to understand it as well.
My main concern is that every attempt to clarify all these options will lead to content outside of this tutorial's scope.
Honestly, I'd rather remove the whole last paragraph and don't even mention about differences between MSYS2, Cygwin, and MinGW-w64.

Why don't we remove the paragraph, merge this PR, and then let's open another PR with just the mingw clarification and we can figure it out there?

I agree, unless we can come up with a concise and clear description. A
clear description is of importance to the stdlib-os subproject as well and
given the confusion we all have about it, it might even become a topic in a
separate minibook on popular OSes.
Op di 20 okt. 2020 om 22:51 schreef Ondřej Čertík<EMAIL_ADDRESS>
:

Why don't we remove the paragraph, merge this PR, and then let's open
another PR with just the mingw clarification and we can figure it out
there?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fortran-lang/fortran-lang.org/pull/152#issuecomment-713131982,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN6YR23TWPCRW2GFFSGKNLSLXZ4HANCNFSM4SFKITAA
.

Thanks @smeskos - I agree, removing that paragraph is the clearest we can be for the scope of this tutorial.

This is now good to merge, correct?

As far as I am concerned, yes.
Op wo 21 okt. 2020 om 17:12 schreef Milan Curcic<EMAIL_ADDRESS>
This is now good to merge, correct?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fortran-lang/fortran-lang.org/pull/152#issuecomment-713648760,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN6YR4XQ47CUDDQ5LQZUCTSL325RANCNFSM4SFKITAA
.

Thank you all, and especially @smeskos who led the effort!

Thanks everybody, I am glad this is in.

#delete_preview


Add partial bitstream generation support

This PR adds changes which are essential to generate a partial bitstream using modifications introduced in https://github.com/SymbiFlow/prjxray/pull/1492

CI is red?


support get reference chunkGroups via chunk.groupsIterable

What problem does this feature solve?
https://github.com/vuejs/preload-webpack-plugin/blob/master/src/lib/does-chunk-belong-to-html.js#L45

What does the proposed API of configuration look like?
just like webpack:
https://github.com/webpack/webpack/blob/1f99ad6367f2b8a6ef17cce0e058f7a67fb7db18/lib/Chunk.js#L516

also need chunkGroup.getParents()

https://github.com/vuejs/preload-webpack-plugin/blob/master/src/lib/does-chunk-belong-to-html.js#L53

closed by https://github.com/web-infra-dev/rspack/pull/4641


Cannot unpickle saved .pt dataset with torch.save()/load()

Saving a formatted torch dataset to file using torch.save(). Loading the same file fails during unpickling:
>>> import torch
>>> import nlp

>>> squad = nlp.load_dataset("squad.py", split="train")
>>> squad
Dataset(features: {'source_text': Value(dtype='string', id=None), 'target_text': Value(dtype='string', id=None)}, num_rows: 87599)
>>> squad = squad.map(create_features, batched=True)
>>> squad.set_format(type="torch", columns=["source_ids", "target_ids", "attention_mask"])
>>> torch.save(squad, "squad.pt")

>>> squad_pt = torch.load("squad.pt")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/vegarab/.conda/envs/torch/lib/python3.7/site-packages/torch/serialization.py", line 593, in load
    return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)
  File "/home/vegarab/.conda/envs/torch/lib/python3.7/site-packages/torch/serialization.py", line 773, in _legacy_load
    result = unpickler.load()
  File "/home/vegarab/.conda/envs/torch/lib/python3.7/site-packages/nlp/splits.py", line 493, in __setitem__
    raise ValueError("Cannot add elem. Use .add() instead.")
ValueError: Cannot add elem. Use .add() instead.

where create_features is a function that tokenizes the data using batch_encode_plus and returns a Dict with input_ids, target_ids and attention_mask.
def create_features(batch):
    source_text_encoding = tokenizer.batch_encode_plus(
        batch["source_text"],
        max_length=max_source_length,
        pad_to_max_length=True,
        truncation=True)

    target_text_encoding = tokenizer.batch_encode_plus(
        batch["target_text"],
        max_length=max_target_length,
        pad_to_max_length=True,
        truncation=True)

    features = {
        "source_ids": source_text_encoding["input_ids"],
        "target_ids": target_text_encoding["input_ids"],
        "attention_mask": source_text_encoding["attention_mask"]
    }

    return features

I found a similar issue in issue 5267 in the huggingface/transformers repo which was solved by downgrading to nlp==0.2.0. That did not solve this problem, however.

This seems to be fixed in a non-released version.
Installing nlp from source
git clone https://github.com/huggingface/nlp
cd nlp
pip install .

solves the issue.


Typescript Definitions: Remove 'I' prefix and make methods as methods

Addressing the great feedback from @janaagaard75 (thank you!!). Unfortunately, I don't think we can keep the PR'd changes as is since we want to keep Interfaces.d.ts strictly generated. We reference many of those interfaces internally as well to make sure what we are returning truly contains values as expected.
Includes changes from:
https://github.com/Azure/azure-functions-nodejs-worker/pull/161
https://github.com/Azure/azure-functions-nodejs-worker/pull/159
And resolves: https://github.com/Azure/azure-functions-nodejs-worker/issues/162

Hey @christopheranderson! We got some good feedback on the types package we're going to release. If you haven't already passed the phase of pushing out bits somewhere, we could incorporate that feedback? Should have reached out earlier but your feedback would also be very much appreciated!


Added telegram message_thread_id option

Description

I wanted to sort alerts sent to Telegram, because I have different log sources and there are so many types of them, therefore so many chats. But having multiple Telegram chats for every source type is not convenient (chaotic chats order, difficult managing telegram_room_id in rules).
Now Telegram has a feature of supergroups/forums where you can create threads. Using this supergroup/forum I can have all my alerts in 1 chat and sort them in threads. So I decided to add an option for message_thread_id of Telegram sendMessage function.
Checklist

[x] I have reviewed the contributing guidelines.
[x] I have included unit tests for my changes or additions.
[x] I have successfully run make test-docker with my changes.
[x] I have manually tested all relevant modes of the change in this PR.
[x] I have updated the documentation.
[x] I have updated the changelog.

Questions or Comments

Thanks for this enhancement. The PR looks good. Can you confirm that alerts show up properly in Telegram when telegram_thread_id is not provided (default behavior)?

Yes, it works as usual
I performed following tests:

Send alert to common group chat, no telegram_thread_id option in a rule. Result: works as usual, alert is sent to the group chat.
Send alert to supergroup chat, no telegram_thread_id option in a rule. Result: alert is sent to general thread of supergroup.
Send alert to common group chat, telegram_thread_id: 2 in a rule. Result: not working, because common group chat obviously has no threads. Writes an error like this: ERROR:elastalert:Error while running alert telegram: Error posting to Telegram: 400 Client Error: Bad Request for url: https://api.telegram.org/bot<BOT_TOKEN>/sendMessage. Details: {"ok":false,"error_code":400,"description":"Bad Request: message thread not found"}
Send alert to supergroup chat, telegram_thread_id: 2 in a rule. Result: alert is sent to thread with ID 2.


Working version for Bedrock Anthropic models

Hi, I assume this PR will not actually get merged as-is, but I wanted to share the changes I made in order to get Anthropic models to produce reasonable evaluation outputs:

Added option for custom prompts
Defined and tested prompts for Claude models
Added logging

I also copied and updated your aws-bedrock.ipynb to reflect the changes. Hope this may help! 😊

Hey @austinmw
Thank you so much for helping us improve ragas with your PR ❤. I know we didn't merge this one yet but this has been really useful for us as we prototyped different ways of customizing prompts (will share this shortly with you)
Now since it's Christmas and all we wanted to send you a postcard and a couple of stickers as our way of saying thank you for your contribution. If you are interested could you shoot an email at<EMAIL_ADDRESS>and I'll tell you more?
Cheers 🙂
Jithin

Hey @jjmachan , sorry for the late reply, must have missed the thread subscription update. Will do, thanks!

hey @austinmw, I'm closing this now 🙂
btw would love to catchup sometime if your interested, for a short call


add token expiration support to prevent nodepool upgrades from killing in flight provisions.

This pr enhances how ignition tokens are cleaned on nodepool upgrades. Previously on an upgrade: all ignition tokens were revoked as soon as the config was updated. The problem with this approach is outlined in issue: https://github.com/openshift/hypershift/issues/1020. To summarize: immediate revocation does not account for in flight provisions that were executed with the old token. This means these in flight operations would fail. This is a problem for consumers of the offering that should be addressed: provisions should never have the potential to fail. To fix this: when an upgrade occurs: old tokens are marked for expiration. The expiration time is based on the amount of time necessary to allow all in flight provisions to complete (new provisions will use the updated token). After that time: the ignition-server will proceed to delete the secret. The behavior on nodePool delete is unchanged: all tokens are revoked when the nodePool is deleted
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes # https://github.com/openshift/hypershift/issues/1020
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

can we please include the final state of this PR into the ignition-server Readme?

yep I will update the doc for deletion flow

/unhold

/retest

/retest

/retest

all comments addressed

/approve
/lgtm


As an application, I want apply theme for preview an publish so that user can see posts with its own layout

This has been separated from the story #13
Preview and Publish should apply different layout – theme from the main admin layout.

[x] Implement theme
[x] Apply theme into preview
[x] Apply theme into publish

Implemented basic business logic for theme.
Implemented the theme on the preview page.

Implemented logic that applies theme into publish posts.


Rename Memories => Conversations in menu

I have no idea if i did it right

agreed on doing in later + facts


Add GitHub topics to make it easier for other people to find this project

For example, I suggest the topic: https://github.com/topics/caffeine

Done.


Can I use wllvm with afl-clang-fast?

Hi. I want to use afl-clang-fast to instrument the program first, and then, use wllvm to get llvm IR code.  To finish this, I try to set:
export LLVM_COMPILER=afl-clang-fast
CC = wllvm

But it didn't work. Here is the error message:

I don't know what should I do to achieve this. Could you please help me?

Please read the README.
LLVM_COMPILER  sets the type of compiler. In your case it should be clang. Then you would set LLVM_CC_NAME or LLVM_CXX_NAME or both.

Thank you for your reply. I try to set:
export LLVM_COMPILER=clang
export LLVM_CC_NAME =afl-clang-fast
CC = wllvm

But it didn't work. Here is the error message:


Not working at win10

When execute docker-toolbox-portable.exe first time, always failed with
Docker: Host does not exist: “default”

Reason:
https://github.com/portapps/docker-toolbox-portable/blob/68b41a5f312760a8cfb3b7580ebf692806d47dd4/res/start.sh#L45
this line seems like to check default vm exists / storage path exists.
But second condition does not work.
After I change this as below, I can get work.
if [ $VM_EXISTS_CODE -eq 1 ]; then

Also, I always get error
bad argements: [

Would you please take a look and fix ?
Thank you.

@zl910627 See #22 and also https://portapps.io/news/2020-11-04-docker-toolbox-portable-abandoned/


Add Config.FileContent

I need to be able to embed a swagger.json file into a Go program using the embed package. Config.FileContent enables that.

Summary by CodeRabbit

New Features

Introduced the ability to directly specify Swagger file content in configurations, enhancing flexibility.

Tests

Added comprehensive tests for Swagger middleware configurations, including custom paths and multiple instances.

Note that I tried to follow the way the existing code does and tests things.

@willfaught Want me to take over and get this merged?

I forgot I had this PR open. If a maintainer is willing to attend to this PR to get it merged, I can reopen the PR.

@willfaught I will take a look if you can re-open it, thanks!

GitHub won't let me reopen this PR. Opened a new PR: https://github.com/gofiber/contrib/pull/1085.


docs(secondary-nav): add docs page

What I did

Created a new page for the Secondary navigation component

[ ] Can the primary navigation link in the sidebar be changed to "Navigation (primary)"? You had that in a mockup.

[ ] Under Style > Theme options, I think a comma can be added before "unless" in theses: The drop shadow is always visible when scrolling or not unless covered by an expandable tray. and The thin gray line is always visible when scrolling or not unless covered by an expandable tray. The wording of the second half was a little confusing.

[ ] Under Usage > Secondary navigation, did you mean to include the helpful tip? It's in a previous section.

[ ] Under Usage > Writing navigation content, change "less" to "fewer" in * If there are fewer menus and links in Zone 2, more characters can be written; if there are more menus and links, less characters can be written. It's a picky note, but I figured it should be consistent since you used "fewer" in the same sentence.

[ ] For the images that have notes in fuschia, do you want to change those to blue now, or do you want to make that change in the bigger site redesign?  We can always pick a different color for notes in the redesign too.

[ ] Under Behavior > Collapsing the expandable tray, should Clicking or tapping anywhere outside of the expandable tray should collapse it. say Clicking or tapping anywhere outside of the secondary navigation... instead? I tried clicking on an empty space in the secondary nav, and it didn't close the tray.

[ ] Under Interaction states > Focus, it looks like the text in Zone 1 is a different color from the links in Zone 2, but the table says they should be the same?

[ ] Under Accessibility > Keyboard events, I think the table headings should change to "Action" and "Event." Also, are the last two rows supposed to say Active (all themes)?

Not an edit: I like how you explained the expandable tray scrolling. Not a super easy thing to convey.

Please squash when merging, thanks


FETCH_KEY_PAIR / FETCH_UNIQUE / FETCH_GROUP fetch modes

In writing framework benchmark, one of the test cases required data that would have been more conveniently fetched using the FETCH_KEY_PAIR mode.   I worked around it by using array_column.
It would be more convenient to either specify the fetch mode as an optional parameter to select or have additional methods (like selectColumn).
public static function fortunes(App $app) {
        $fortunes = Controller::getDb()->select(
            'SELECT * FROM Fortune'
        );
        $fortunes = array_column($fortunes, 'message', 'id');
        $fortunes[0] = 'Additional fortune added at request time.';
        asort($fortunes);
        echo $app->view('fortunes.html.twig', ['fortunes' => $fortunes]);
    }

benchmark code: https://github.com/bfinlay/FrameworkBenchmarks/blob/delight.im/frameworks/PHP/delight.im/app/Controller.php

Excellent point, thank you!
While we only use \PDO::FETCH_ASSOC internally so far, we definitely want to support the following additional “fetch styles”:
\PDO::FETCH_OBJ
    => selectAsObjects + selectRowAsObject
\PDO::FETCH_NUM
    => selectNumbered + selectRowNumbered
\PDO::FETCH_CLASS
    => selectIntoClass + selectRowIntoClass
    or
    => selectAsInstances + selectRowAsInstance
\PDO::FETCH_CLASS | PDO::FETCH_PROPS_LATE
    => selectIntoConstructedClass + selectRowIntoConstructedClass
    or
    => selectAsConstructedInstances + selectRowAsConstructedInstance
\PDO::FETCH_KEY_PAIR
    => selectAsPairs + selectRowAsPair
\PDO::FETCH_UNIQUE
    => selectUnique + selectRowUnique
\PDO::FETCH_GROUP
    => selectGrouped + selectRowGrouped

Yes, some of these are a bit verbose. But you have IDEs with autocomplete everywhere, and we currently can’t think of any solution other than these separate methods that we like as much.
What do you think?

I agree that the IDE autocomplete addresses verbosity concerns.  These would work great!
From: Marco<EMAIL_ADDRESS>Reply-To: delight-im/PHP-DB<EMAIL_ADDRESS>Date: Wednesday, May 29, 2019 at 1:16 PM
To: delight-im/PHP-DB<EMAIL_ADDRESS>Cc: Brion Finlay<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [delight-im/PHP-DB] FETCH_KEY_PAIR / FETCH_UNIQUE / FETCH_GROUP fetch modes (#14)
Excellent point, thank you!
While we only use \PDO::FETCH_ASSOC internally so far, we definitely want to support the following additional “fetch styles”:
\PDO::FETCH_OBJ
=> selectAsObjects + selectRowAsObject

\PDO::FETCH_NUM
=> selectNumbered + selectRowNumbered

\PDO::FETCH_CLASS
=> selectIntoClass + selectRowIntoClass

or

=> selectAsInstances + selectRowAsInstance

\PDO::FETCH_CLASS | PDO::FETCH_PROPS_LATE
=> selectIntoConstructedClass + selectRowIntoConstructedClass

or

=> selectAsConstructedInstances + selectRowAsConstructedInstance

\PDO::FETCH_KEY_PAIR
=> selectAsPairs + selectRowAsPair

\PDO::FETCH_UNIQUE
=> selectUnique + selectRowUnique

\PDO::FETCH_GROUP
=> selectGrouped + selectRowGrouped

Yes, some of these are a bit verbose. But you have IDEs with autocomplete everywhere, and we currently can’t think of any solution other than these separate methods that we like as much.
What do you think?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/delight-im/PHP-DB/issues/14?email_source=notifications&email_token=AANNFIHKH632SULN47UVOADPX3B7HA5CNFSM4HQH4WY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWQGLAI#issuecomment-497051009, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AANNFIGX2T5YF36YGBIR3TDPX3B7HANCNFSM4HQH4WYQ.


Range expression with hexadecimal numbers does not work

Description:

The following code snippets currently do not work. But they should be supported.
    foreach int i in 0x1 ... 0x5 { }

    foreach int i in 0x1 ..< 0x5 { }

Given error message:
Compiling source
        temp.bal
ERROR [temp.bal:(4:13,4:16)] incompatible types: expected 'other', found 'int'
ERROR [temp.bal:(4:22,4:33)] operator '..<' not defined for 'byte' and 'byte'
error: compilation contains errors

Affected Versions:
Ballerina Swan Lake Beta 3
Suggested Labels (optional):
Team/CompilerFE

Another concern is that whether we can put byte i instead of int i there, or any other subtype of  int such as int:Unsigned8,int:Signed32, etc.

May be related to #33139

Another concern is that whether we can put byte i instead of int i there, or any other subtype of int such as int:Unsigned8,int:Signed32, etc.

In the range-expr section of the spec says,

The result of a range-expr is a new object belonging to the object type Iterable<int,()> that will iterate over a sequence of integers in increasing order

Since the resultant type is Iterable<int,()> we can't hold the iterator value with a subtype of int.


Migrate kafka connect terraform integration doc

What should we add to our content collections?
Migrate https://help.aiven.io/en/articles/4187903-kafka-connect-terraform-integration
Is this something you would work on yourself?
[ ] I plan to open a pull request for this

@ftisiot Would https://developer.aiven.io/docs/products/kafka/kafka-connect/howto be an ideal new home for this article?

I'm not really sure if this belongs to the kafka connect subtree or the /docs/tools/terraform subtree...
@lornajane what do you think?

I think terraform, we're trying to collect a series of reference implementations and this a really good example. Especially because I think the service integration concept is something we need more content on. Find the issue about terraform for more of an overview of the cookbook format and to link this issue to it (I'm on mobile, sorry). Let's also make sure to put this as a clear resource on the Kafka Connect landing page too, I agree it could go in either place but hoping to keep the tools in their own spaces for the most part


Rename getting started vigniette

If you rename your getting started vignette into the package name it would be automatically detected by pkgdown and it will create a separate menu item on the top in the documentation website
more details here: https://pkgdown.r-lib.org/articles/customise.html#:~:text=built-in components%3A-,intro,-%3A “Get Started”%2C which
Example:

because of this: https://github.com/tidyverse/dplyr/blob/main/vignettes/dplyr.Rmd

FYI @harriscw


chore(package): update standard to version 10.0.0-beta.0

https://greenkeeper.io/

Coverage remained the same at 89.286% when pulling f6b88ec3b3de5fce5cc331e7bd609ba6add7c595 on greenkeeper/standard-10.0.0-beta.0 into 113975834e61cec94dd18b4980fa7b88a2f1ddf0 on master.


Url does not open up on my phone.

Hi! I've been trying to use this add-on and when I scan the URL on my Iphone it says "safari cannot open page" I tried it on two different phones, one android and one iphone, and it says the same thing. When i plug the url into my macbook it works but of course this is not helpful for me. Some assistance would be greatly appreciated as I think this is a wonderful and helpful add on. :)

Hi. There's a few reasons this could be happening. First make sure you don't have a VPN enabled on your laptop or phone.
Secondly make sure your phone and laptop are on the same wifi network and that your wifi doesn't block communication across devices. Some public, school, and apartment managed networks block communication so it's best to try it on a home wifi router that you own.
If none of this works, reply with the address under the QR code, and the internal IP address for your phone and computer that you can find using this guide: https://lifehacker.com/how-to-find-your-local-and-external-ip-address-5833108

Thankyou!.<IP_ADDRESS>. This is my internal IP adress. This is the QR code. http://<IP_ADDRESS>:61349/

Those addresses look reasonable, the another reason this could happen is the firewall on your computer.
If you're on windows, try allowing anki access on both public and private networks. Follow this guide https://pureinfotech.com/allow-apps-firewall-windows-10/

Im not on windows, is there another option for ios?

What platform is your computer anki on, Mac?
If your firewall is on you can try letting anki through -
https://support.apple.com/en-in/guide/mac-help/mh11783/mac

Also if your phone is at <IP_ADDRESS>
Try opening terminal and typing "ping <IP_ADDRESS>" and hitting enter. This will tell us whether the devices are able to talk to each other over your router

Yea, its on mac. i typed it in and the first thing that comes up in the search engine is "Bogon IP adresss

You'd have to type it into the terminal app on Mac, not your browser

It says "Host is down/ Request timeout for icmp_seq" repeatedly

Ok so that shows us that either the devices arent on the same local network / wifi, or that the wifi stops devices from talking to one another.
I think the best way forward would be to talk to whoever manages the wifi, or try a different home wifi network.

I see. Thank you.

Tried a different 5g network and it works. :D thank you!


Temporarily disable dev runs

Re-enabling dev build testing with https://github.com/oracle/graalvm-reachability-metadata/pull/242.


spurious messages when USB port already used

If another program is already using a serial port (e.g., owlcms-firmata), version 1.1 correctly detects that the port is used,
In the following screen shot, COM9 is already used.  COM5 is the blue-owl device.  Cannot tell if the firmware timeout messages pertain to COM9 or COM5, but both devices work.

Unfortunately these logs come from Johnny-Five and can't be disabled. However, future versions of the Blue Owl app will be packaged in a way that does not display the full terminal logs and only displays the Blue Owl logs, so these messages will not be displayed (there will be a true native application that does not run a script in the terminal and does not open a web browser).
blue-owl-core will continue to display these logs in the terminal, at least as long as is remains built on top of Johnny-Five. I would like to eventually build this directly on top of firmata.js.

Closing as the issue in Blue Owl is resolved. blue-owl-core will continue to display these messages.


Add block support for Colophon

Description
Custom block that will render the Colophon in both the editor as well as the frontend.

This won't live in the monorepo, but will instead live in the Colophon repo.
It looks like @georgestephanis requested changes here: https://github.com/a8cteam51/colophon/pull/29
Will close this one.


Add a Powershell script for Container Preview of Website

Closes #42861
This adds a Powershell script for launching a Container Preview of Website on Windows

I've changed the approach slightly as per feedback https://github.com/kubernetes/website/pull/43461#issuecomment-1786037133
Instead of invoking wsl, We'll work with bash and Powershell scripting

/close


Correct sys.periods table column description

The column name and description are mismatched and I have corrected this in the pull request.

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

Reviewed and confirmed correction.


issue with eddystone

sir the eddystone i made in the same way is detected sometimes and in some apps and sometimes not detected . what can be the reason and how can i fix this issue?

@cheena19 please share the sketch you are using, and a list of apps with issues.


updated the js file

I made it translatable

Please, put all files related to the same patch in the same PR.


MySQL charset option

Need to add charset option

This option has been added over a year ago. Closing.


Footnote styling

Add inline dropdown-style footnotes a. la. Tufte, but have all set to display hidden by default.  Incorporate background colour to the element.

There doesn’t appear to be any way of accomplishing this without shortcodes, and both Blackfriday and Goldmark markdown renderers don’t seem to offer customization of the footnote html elements.
Closing this.


Add an option to remove the microseconds from the filename.

Having the microseconds in the filename is a bit too extra for my liking. 😝

Updated.

Updated.

What does this mean?

I see what you mean now:<PHONE_NUMBER>26d4068cb8d910e8b8ece41fe652ac
You should have just merged my pr. You missed the changes needed in timelapse.py and usersettings.py
Also, %f are MICROseconds with 6 digits of accuracy, milliseconds have 4 digits. To get milliseconds you need to do dt.datetime.now().strftime('%Y%m%d_%H%M%S_%f')[:-3]. Emphasis on the [:-3].


Can't deselect an item in the selection group menu

Describe the bug
Can't deselect a selected item in the selection group menu.
To Reproduce
Steps to reproduce the behavior:

Go to https://trimox.github.io/angular-mdc-web/#/angular-mdc-web/menu-demo/examples
Click on 'Show Selection Group Menu'
Select anything.
Now try to deselect.

Expected behavior
To select an item and to deselect it.
Also it would be a nice touch if I don't have to reopen the menu every time I check something.
What Angular MDC version are you using?
v3.2.0
What browser(s) is this bug affecting?:
I only tried with the latest Google Chrome.

@TryHardDood Thanks for reporting this.
Problems seems to be in getSelectedSiblingOfItemAtIndex(index: number); always returning -1.
https://github.com/trimox/angular-mdc-web/blob/master/packages/menu/menu.ts#L108
Let ya know..

Got it. PR incoming.

Also it would be a nice touch if I don't have to reopen the menu every time I check something.

I'll have to submit separate PR for surface closing.


Python wrapper, pointcloud as numpy array has numpy.voids in it!

Required Info

Camera Model
{ D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (10)

Kernel Version (Linux Only)
()

Platform
PC/

SDK Version
{ legacy  }

Issue Description
I slightly change the python example saving a ply file:
I want to make a python-"realtime" viewer of the point cloud based on open3d.
Now the problem is that the numpy array I get from the pointcloud has numpy.voids in it.
I only get rid of those by using the command tolist and casting back to numpy.array.
Now this procedure is like a factor 100 slower than normal numpy operations.
Therefore, I only would get 5-Hz framerate on a possible viewer.
 points = pc.calculate(depth)     pc.map_to(color)     print("Saving to %d.ply..." % ind)     #points.export_to_ply("data/%d.ply" % ind, color)      vtx = np.asanyarray(points.get_vertices())     print(vtx.dtype)     print(type(vtx),type(vtx[0]))     vtx2 = np.array(vtx)     print(vtx2.dtype)     #vtx2 = vtx.astype(dtype=[('f','f','f')])     t0 = time.time()     vtx2 = vtx.tolist()     vtx2 = np.array(vtx2)     print(time.time() -t0)     raise "s"
Printout
Saving to 0.ply...
[('f0', '<f4'), ('f1', '<f4'), ('f2', '<f4')]
<class 'numpy.ndarray'> <class 'numpy.void'>
[('f0', '<f4'), ('f1', '<f4'), ('f2', '<f4')]
0.24765872955322266
0.2 seconds this is slower than snail  time.

I can use the pointcloud generation in open3d as in: https://github.com/IntelVCL/Open3D/issues/374

[Realsense Customer Engineering Team Comment]
hi @ralphybbbaer,
this is interesting, so do you make it and is it still slow or not?
I want to see how it works too.

When postprocessing the depth image through librealsense and calculating the pointcloud with open3d and using the viewer in open3d I get an acceptable 10-20 Hz viewer in python.

Is there a way to get good fps using the pointscloud calculated by open3D converted in numpy?
The compute time to convert the numpy from the librealsense to the one compatible with open 3D is very long.
I've found this on github, have a look at this :
https://github.com/baptiste-mnh/pySoRo#adding-two-dimensional-data-protocols
It explain how to speed up the compute time.

Hi there, I was also wondering if there may have been any new updates on improving the fps for real-time point cloud solutions.

@karlita101  An easy fix for getting NumPy arrays without void dtypes is provided in #7629.


WIP: Remove api streams sensor

Breaking Change:
api_streams sensor has been removed as it wasn't working and wasn't used. It can be recreated but it will need a redesign, if anyone uses it.
Description:
Note: This currently branches from #22109.
Related issue (if applicable): fixes #22147 #21815
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io home-assistant/home-assistant.io#8972

@balloob and @amelchio  have suggested to remove it.

Actually, that was @awarecan :-)

I will wait util the upstream PR merged.

I'm not following all discussions if there was a decision made then I back it up.
In the current state this can't be merged.

@fabaff the thing is that the sensor has been broken for a long time, no one bothered to fix it, so we might as well remove it.

I'm not following all discussions if there was a decision made then I back it up.

That's why I made this PR, to discuss :smile:

In the current state this can't be merged.

Hence WIP.

Another comment:
$ ls $(cat ../izone_custom_component/movers.txt) | grep -v __init__.py | grep -v __pycache__ | grep -v services.yaml
homeassistant/components/air_quality:

homeassistant/components/alarm_control_panel:
manual_mqtt.py
manual.py

homeassistant/components/binary_sensor:

homeassistant/components/calendar:
demo.py

homeassistant/components/camera:
demo_0.jpg
demo_1.jpg
demo_2.jpg
demo_3.jpg
demo.py

homeassistant/components/climate:
const.py
demo.py
generic_thermostat.py
reproduce_state.py

homeassistant/components/cover:
demo.py
group.py

homeassistant/components/device_tracker:

homeassistant/components/fan:
demo.py

homeassistant/components/geo_location:
demo.py

homeassistant/components/image_processing:

homeassistant/components/light:
demo.py
group.py
light_profiles.csv
switch.py

homeassistant/components/lock:
demo.py

homeassistant/components/mailbox:

homeassistant/components/media_player:
const.py
demo.py
reproduce_state.py

homeassistant/components/remote:

homeassistant/components/scene:
homeassistant.py

homeassistant/components/sensor:
mold_indicator.py
strings.moon.json
strings.season.json

homeassistant/components/switch:

homeassistant/components/vacuum:
demo.py

homeassistant/components/water_heater:
demo.py

homeassistant/components/weather:

OK this is ready to merge now, I'll leave it up to more experienced people what to do with it.

@awarecan @balloob
Actually I've had more of a think about this and redone the whole thing properly. PR here: #22200. Will close this PR for now because I think it's better off fixed properly, and plus the events are useful too.


Fix images not loading

https://github.com/cdr/code-server/issues/3410

Yeah I think this is better because then you can see the change in contrast with the CDN code so it might make more sense why we have to add self.


Feature request - Allow passing patterns to watch

It would be good to support patterns for watch. I.e.
scripts.json
{
  "scripts": {
    "start": {
      "cmd": "deno run --unstable mod.ts",
      "watch": "*.ts,*.json"
    },
  }
}

Based on https://github.com/denoland/deno/issues/2401#issuecomment-859901801 Deno supports deno run --watch=./**/*.ts,./**/*.tsx server.ts kind of syntax.
Looking at the code, the command builder would have to be extended to support strings beyond booleans (case DenoOptions.watch: { in code).

Hi @bebraw, thanks for the suggestion! 🙂
It looks like the --watch options is still a boolean flag though. You can verify this by passing a file to --watch which is not part of the entry point module graph and then edit that file. You'll see that only the entry point and its dependencies are watched. The example in the issue you linked is probably working because the watched .ts files are already part of the module graph (and the value of the boolean flag is ignored I guess?).

@umbopepato Thanks for the heads up. I opened a feature request at https://github.com/denoland/deno/issues/12197 .
I guess another option would be to leverage<EMAIL_ADDRESS>. I wonder if I could do that in user space even. In this case I would have to trigger whatever Deno's watch is triggering right now and if we're lucky there's a programmatic hook for that.

Awesome, thank you @bebraw! Let's see what they say and sure, if the flag is not going to change we can evaluate a custom implementation.


Hardware Specs...24V?

Hi, what is the limiting factor for only 24V? I would be in need of 30V (8s Lifepo4 battery)...

TESTING! I'm not sure of the hard voltage limit, should be over 34 volts, I just don't have the tools to do a lot of testing and rate the board higher.

Thanks


Move tenant & timeline dir method to NeonPageserver and use them everywhere

Problem
In many places in test code, paths are built manually from what NeonEnv.tenant_dir and NeonEnv.timeline_dir could do.
Summary of changes

NeonEnv.tenant_dir and NeonEnv.timeline_dir moved under class NeonPageserver as the path they use is per-pageserver instance.
Used these everywhere to replace manual path building

#5258
Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

@koivunej could you pls review this

Minor approved-for-ci-run issues, but hang tight...


Tooling/script for listing forbidden exports from backend packages

It should list tasks for stopping support for the legacy backend, such as missing default exports, remove unused exports or that are only used by create routers, etc.
Experiment using ts-morph library to create the script.

Pull request: https://github.com/backstage/backstage/pull/26130


Lazy item loading from Kopia

Lazily loading items from kopia seems to cut down the load time to just the time needed for looking up collections which is very minimal. It also ensures that at a time we only have a few open file objects and that they are closed after consumption.
More details: https://github.com/alcionai/corso/pull/3506#discussion_r1209446233

related to #3440


[build-presents] Enable SwiftPM in incremental OS X builds.

What's in this pull request?
This PR enables the SwiftPM tests on the OS X builders. I believe it is an oversight that it has been disabled.
Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

This is enabled for the package builder bots and the Linux, I think it is an
oversight that it has been disabled on the OS X bots for all this time.

@swift-ci please test and merge

@swift-ci please test


Multiple {{dynamic-link}} in template supported?

I have an odd issue where I am setting the query params dynamically which seems to generate the link href correctly however when I click the link the url that is transitioned to is always the url of the last href {{dynamic-link}} instance on the page.
Is this helper stateless?
e.g. template.hbs
{{#dynamic-link route='sysmngt.devices' queryParams=(dynamic-link-params "groups" "groups1")}} groups1{{/dynamic-link}}
{{#dynamic-link route='sysmngt.devices' queryParams=(dynamic-link-params "groups" "groups2")}} groups2{{/dynamic-link}}

and dynamic-link-params.js helper:
import Ember from 'ember';
const {Helper} = Ember;
/**

A helper to convert a name value pair to a an object that is suitable for the dynamic-link component
@param {array} params
@returns {{}}
*/
export function dynmicLinkParams(param) {
const paramsObj = {};
if (params.length === 2) {
paramsObj[params[0]] = encodeURI(params[1]);
}
Ember.Logger.debug(paramsObj=${JSON.stringify(paramsObj)});
return paramsObj;
}

export default Helper.helper(dynmicLinkParams);

Hmm, interesting issue. I'm pretty sure the component itself is stateless -- I've used multiple per page with different queryParams before.
Is it possible that your dynamic-link-params.js isn't stateless, e.g. somehow the second time you call it, it modifies the first paramsObj you return? You could also try exposing the params objects as properties on the parent component/controller.

That does not seem to be the problem as the mouseover and generated src (viewed in chrome) show distinct and expected links.  It seems to be in the transition.

I forked the component and added some debug in the transitionRoute() and can confirm that the link href is fine but queryparams are always the ones for the last instance in the template.

Cool, thanks for all this investigation. I will take a look at it soon and try to reproduce/fix it (although if you fix the issue in your fork, feel free to make a PR). Just to clarify, in those debug statements are you seeing the wrong values in routeArguments/routingArguments, or is the issue that the transitionTo call does not perform correctly?
Also, what version of Ember are you running?

Ember	2.12.1
Ember Data	2.12.2
jQuery	3.2.1
I think the transitionTo is ok actually but the routingArguments and routeArguments that it passes to the router.transitionTo() seem to be incorrect even though the generated urls appear ok.  I have tried to add debug to the href computed prop but that results in an assert error from ember.

The following might be related - multiple ember component instances on a page https://github.com/octobercms/october/issues/737

Ok, cool. I wonder if the problem could have something to do with a version incompatibility -- I'm not actively developing any apps in Ember right now, so I think this library hasn't been run against 2.12 or 2.13. And it looks like the dependencies are slightly out of date as well.
I'm a bit busy today but I'll try to get this resolved soon. If you're able to create a test case in your fork that reproduces the issue (and if it only fails on recent versions), then that would definitely speed things up. If you make partial progress that isn't ready for a full PR, you can make a PR into the queryparams-fix branch and we can merge to master once we've fully resolved the issue.

I'm assuming you've probably moved on from this now, but I have a branch where I've fully upgraded Ember CLI and, on all the tests, added a second dynamic-link to the end of the page. I'll push this up soon, but I don't think it will solve your problem -- the tests still pass even though there are now two links on the page.

I'm afraid I could not make it work so went back to using the OOTB link-to helper in the template and figured out how to make it work using https://discuss.emberjs.com/t/passing-dynamic-query-params-as-an-object-in-link-to-helper/9136/3

Gotcha. Glad you were able to get it to work somehow, and thanks again for reporting this.


Updating package.json version to 0.7.0

This looks good to me, @msecret. You wrote the release so I'll let you go ahead and merge this when you publish the release notes.


[ ] I have followed the contribution guidelines for this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that new and existing unit tests pass locally with my changes

We updated cargo locks in bindings but not in our main crate? is that intentional?

At the moment it's git ignored so I can't update it. Whether it should be ignored or not is another question...


ITHC Network ACL tightening

User Story
As a modernisation platform engineer
I want to control network traffic entering and exiting my private and data subnets
So that traffic directly to or from the internet is not permitted
Value
Our IT Healthcheck identified some points of configuration that could be adjusted to improve our security posture. Since our private and data subnets should not ordinarily need to communicate with internet addresses, we can replace any open network ACL entries with ones referencing private IP addresses instead. This will prevent any unintentional ingress to or egress from the internet.
Questions / Assumptions
See ITHC report for further details
Definition of done

[x] remove any <IP_ADDRESS>/0 entries from data & private ACLS
[x] add entries for <IP_ADDRESS>/8, <IP_ADDRESS>/12, <IP_ADDRESS>/16
[ ] another team member has reviewed
[ ] tests are green

Reference
How to write good user stories

https://github.com/ministryofjustice/modernisation-platform/pull/1536


[Android] Implement generate certificiation API

Similar to the part implemented in Darwin's MTRCertificate class, I have implemented an API to generate a certificate in Android Application.
Refer : https://github.com/project-chip/connectedhomeip/blob/b611144792c9f572676837582eb1d91b7200db31/src/darwin/Framework/CHIP/MTRCertificates.h#L29

@mergifyio refresh

https://github.com/Mergifyio refresh


Calculating FLOPs and training memory per GPU

Hi,
Would you please give some instructions about the way you calculated FLOPs for a single forward pass during training and training memory per GPU? Thanks.

Hi,
We used fvcore from facebook research to automatically estimate FLOPs. For the GPU memory usage, we reported the maximum GPU memory occupied by tensors with torch.cuda.max_memory_allocated.

Thanks so much for your response. I tried fvcore but I keep getting this error message: "RuntimeError: Detected that you are using FX to torch.jit.trace a dynamo-optimized function. This is not supported at the moment." Could you please share the details of where do you call FlopCountAnalysis() and what do you pass as the input args?

Just disable torch.compile().


Fixed readme errors

Readme errors were fixed in.

Repository readme:

About Section
picture URLs

Documentation: Board schematics

Fixed Picture links
added new layout for front views.

One picture is missing in documentation/images/readme.md, will source that and mark the PR done.

The changes made look ok to me. Proceed with #18

I've named this branch readme-fixes, but the proceeding tasks require me to name the branch gh-pages should I proceed nevertheless or make a new one?


Access to local devices is not very friendly

Golang's ability to access local devices is very limited (such as local cameras, microphones, network card IP settings, etc.), and often requires the use of third-party libraries, which leads to the introduction of a bunch of huge things. I really hope that this problem can be improved in future versions.

This is not actionable, closing.


celebA downloaded from G-drive, but md5 not match.

🐛 Describe the bug
celebA = datasets.CelebA(root='../../data',download=True, transform=None)
RuntimeError: The MD5 checksum of the download file ../../data/celeba/img_align_celeba.zip does not match the one on record.Please delete the file and try again. If the issue persists, please report this to torchvision at https://github.com/pytorch/vision/issues.
Versions
Collecting environment information...
PyTorch version: 2.0.1
Is debug build: False
CUDA used to build PyTorch: 11.8
ROCM used to build PyTorch: N/A
OS: Ubuntu 22.04.3 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.35
Python version: 3.9.16 (main, Mar  8 2023, 14:00:05)  [GCC 11.2.0] (64-bit runtime)
Python platform: Linux-6.5.0-15-generic-x86_64-with-glibc2.35
Is CUDA available: True
CUDA runtime version: 11.6.124
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration: GPU 0: NVIDIA GeForce RTX 3090
Nvidia driver version: 545.23.08
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
CPU:
Architecture:                       x86_64
CPU op-mode(s):                     32-bit, 64-bit
Address sizes:                      43 bits physical, 48 bits virtual
Byte Order:                         Little Endian
CPU(s):                             32
On-line CPU(s) list:                0-31
Vendor ID:                          AuthenticAMD
Model name:                         AMD Ryzen 9 3950X 16-Core Processor
CPU family:                         23
Model:                              113
Thread(s) per core:                 2
Core(s) per socket:                 16
Socket(s):                          1
Stepping:                           0
Frequency boost:                    enabled
CPU max MHz:                        4839.8428
CPU min MHz:                        2200.0000
BogoMIPS:                           6999.61
Flags:                              fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good nopl nonstop_tsc cpuid extd_apicid aperfmperf rapl pni pclmulqdq monitor ssse3 fma cx16 sse4_1 sse4_2 x2apic movbe popcnt aes xsave avx f16c rdrand lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs skinit wdt tce topoext perfctr_core perfctr_nb bpext perfctr_llc mwaitx cpb cat_l3 cdp_l3 hw_pstate ssbd mba ibpb stibp vmmcall fsgsbase bmi1 avx2 smep bmi2 cqm rdt_a rdseed adx smap clflushopt clwb sha_ni xsaveopt xsavec xgetbv1 cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local clzero irperf xsaveerptr rdpru wbnoinvd arat npt lbrv svm_lock nrip_save tsc_scale vmcb_clean flushbyasid decodeassists pausefilter pfthreshold avic v_vmsave_vmload vgif v_spec_ctrl umip rdpid overflow_recov succor smca sev sev_es
Virtualization:                     AMD-V
L1d cache:                          512 KiB (16 instances)
L1i cache:                          512 KiB (16 instances)
L2 cache:                           8 MiB (16 instances)
L3 cache:                           64 MiB (4 instances)
NUMA node(s):                       1
NUMA node0 CPU(s):                  0-31
Vulnerability Gather data sampling: Not affected
Vulnerability Itlb multihit:        Not affected
Vulnerability L1tf:                 Not affected
Vulnerability Mds:                  Not affected
Vulnerability Meltdown:             Not affected
Vulnerability Mmio stale data:      Not affected
Vulnerability Retbleed:             Mitigation; untrained return thunk; SMT enabled with STIBP protection
Vulnerability Spec rstack overflow: Mitigation; safe RET
Vulnerability Spec store bypass:    Mitigation; Speculative Store Bypass disabled via prctl
Vulnerability Spectre v1:           Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:           Mitigation; Retpolines, IBPB conditional, STIBP always-on, RSB filling, PBRSB-eIBRS Not affected
Vulnerability Srbds:                Not affected
Vulnerability Tsx async abort:      Not affected
Versions of relevant libraries:
[pip3] flake8==6.0.0
[pip3] numpy==1.23.1
[pip3] pytorch-fid==0.3.0
[pip3] pytorch-ignite==0.4.12
[pip3] torch==2.0.1
[pip3] torchaudio==2.0.2
[pip3] torchvision==0.15.2
[pip3] triton==2.0.0
[pip3] vit-pytorch==1.2.1
[conda] blas                      1.0                         mkl
[conda] cudatoolkit               10.1.243             h6bb024c_0
[conda] ffmpeg                    4.3                  hf484d3e_0    pytorch
[conda] ignite                    0.4.12                     py_0    pytorch
[conda] mkl                       2023.1.0         h6d00ec8_46342
[conda] mkl-service               2.4.0            py39h5eee18b_1
[conda] mkl_fft                   1.3.6            py39h417a72b_1
[conda] mkl_random                1.2.2            py39h417a72b_1
[conda] numpy                     1.23.1                   pypi_0    pypi
[conda] pytorch                   2.0.1           py3.9_cuda11.8_cudnn8.7.0_0    pytorch
[conda] pytorch-cuda              11.8                 h7e8668a_5    pytorch
[conda] pytorch-fid               0.3.0                    pypi_0    pypi
[conda] pytorch-mutex             1.0                        cuda    pytorch
[conda] torchaudio                2.0.2                py39_cu118    pytorch
[conda] torchtriton               2.0.0                      py39    pytorch
[conda] torchvision               0.15.2               py39_cu118    pytorch
[conda] vit-pytorch               1.2.1                    pypi_0    pypi

Thank you for the report @supershiye . This is the same issue as in https://github.com/pytorch/vision/issues/8204#issuecomment-1935737815 so I'll close this one. Please see my message there for the recommended temporary fix and the future plan. Thank you

Hi  @supershiye ,  torchvision 0.17.1 is out and the download util should now be fixed. For it to work, you'll need to pip install gdown


switch to toml file

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.37%. Comparing base (2852799) to head (8cc49f9).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #34   +/-   ##
=======================================
  Coverage   82.37%   82.37%
=======================================
  Files          17       17
  Lines         913      913
=======================================
  Hits          752      752
  Misses        161      161

Flag
Coverage Δ

unittests
82.37% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[bug] single_class_element_per_statement - handle arrays

The fixer doesn't detect the statements correctly when a statement is an array, so it will break the array into pieces.
Input:
<?php
class Test {
    private static $content = array(
        "     * @param string \$hello\n",
        "     * @param bool \$test Description\n     *        extends over many lines\n",
        "     * @param adkjbadjasbdand \$asdnjkasd\n",
        "     * @throws \Exception asdnjkasd\n     *\n     * asdasdasdasdasdasdasdasd\n     * kasdkasdkbasdasdasdjhbasdhbasjdbjasbdjhb\n",
        "     * @return void\n",
    );
}

Output:
1) PhpCsFixer\Tests\Fixer\ClassNotation\SingleClassElementPerStatementFixerTest::testFix with data set #0 ('<?php\nclass Test {\n    /**\n  ...}    \n')
Code build on expected code must not change.
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
     private static $content = array(
-        "     * @param string \$hello\n",
-        "     * @param bool \$test Description\n     *        extends over many lines\n",
-        "     * @param adkjbadjasbdand \$asdnjkasd\n",
-        "     * @throws \Exception asdnjkasd\n     *\n     * asdasdasdasdasdasdasdasd\n     * kasdkasdkbasdasdasdjhbasdhbasjdbjasbdjhb\n",
-        "     * @return void\n",
-    );
+        "     * @param string \$hello\n";
+    private static "     * @param bool \$test Description\n     *        extends over many lines\n";
+    private static "     * @param adkjbadjasbdand \$asdnjkasd\n";
+    private static "     * @throws \Exception asdnjkasd\n     *\n     * asdasdasdasdasdasdasdasd\n     * kasdkasdkbasdasdasdjhbasdhbasjdbjasbdjhb\n";
+    private static "     * @return void\n";
+    private static );
 }

master only, tested on:
possum@LandOfOz:~/work/PHP-CS-Fixer$ php -v
PHP 7.0.4-7ubuntu2.1 (cli) ( NTS )

ping @phansys
(if you have no time in the near future please let me know, I might find so time)

I'll have some time this weekend @SpacePossum, so please, if you can work on a fix before, it would be great.
Thank you in advance.

ref #2044


test for loading error added

test provided in #267 by @prettycoder

It solved my problem

@kradima since I'm basically re-fix @265 in this PR could you please test it with RubyMine on your tests.
I'll wait couple days before integrating it into master and publishing a new beta.

pushed directly to master to avoid redundant merge

1.3.2.beta1 has been release, please test

@os97673 Thank you for the notification, I asked our QA team to check and got report that everything works fine in RubyMine.


Coding Foundation

Thanks for submitting your project to be featured in the Open Leadership Zone at MozFest 2017 :sparkles:
This is open to any project in Mozilla Open Leaders round 4 -- even if you're not coming to MozFest. This process ensures that you're following open practices and will help set you up for a sustained contributor community afterwards. If you have any questions, feel free to reach out to @acabunoc or ask questions in our chat room.
Open Project Checklist :clipboard:
Your project must complete the following to be a featured on Mozilla Pulse and the Open Leadership Zone at MozFest 2017. As you complete each exercise, check off the box and comment with a link to your completed resource. This template repository is here to help you if you get stuck!

[x] Complete Open Leadership 101
[x] Provide a GitHub repository for work and discussion on your project in a comment

new to GitHub?  Here's a step-by-step guide on using the #mozfest template

[x] Create file: README.md in your project repository. This file should help newcomers understand what your project is, why it's important, and kinds of help you're looking for.
[x] Create file: LICENSE to give your project an open license, allowing for sharing and remixing.
[x] Create file: CONTRIBUTING.md so others know how they can contribute. If you'd like, you can remix this template
[x] Create file: CODE_OF_CONDUCT.md. This can be the Mozilla Community Participation Guidelines or a code of conduct of your choice
[x] Turn on your Issue Tracker and create issues to describe each task that you need help with and how a contributor can get started on that task. If you're participating in Hacktoberfest, Create a label called hacktoberfest and apply it to your issues.

Ready to be featured :tada:

[x] Fill out this form to submit your project to Mozilla Pulse. Add the tag open leadership zone.
[x] Leave a comment on this issue with the text This is ready for the Open Leadership Zone. @acabunoc will review this issue. If accepted, your project will be approved on Mozilla Pulse and featured in the Open Leadership Zone.

If you get stuck at any point, feel free to look at the project templates or reach out to your mentor or @acabunoc. We're here to help you through this process.

This is ready for the Open Leadership Zone
Also, the repo for this project is https://github.com/trallard/Coding-foundation 🦊
@acabunoc

This is great, thanks @trallard! 🎉  Accepted 😄 I sent over a small PR
Pulse: https://www.mozillapulse.org/entry/487