[WIP] Fixing Geometry errors

Fixed a couple of issues, still have a couple more to go. WIP, hopefully can fix these today.

Main test issues are fixed. Now:

issue in #114 (and duplicates) will be fixed.

Adding skipIf statements

Ok, so I implemented a fix for the eligible torsion thing. However, it seems that the current criteria are then sufficient to cause some members of the ValenceSmallMoleculeLibraryTestSystem to become unreachable from the rest. I just need to correct the array there, and have a warning saying which molecules have been removed.

Ok, so I implemented a fix for the eligible torsion thing. However, it seems that the current criteria are then sufficient to cause some members of the ValenceSmallMoleculeLibraryTestSystem to become unreachable from the rest. I just need to correct the array there, and have a warning saying which molecules have been removed.

Do you mean that some molecules cannot be reached from any molecule in the set, or that some molecules cannot be reached from every molecule in the set?
Put another way: If I make a graph of molecules as nodes connected by edges that correspond to transitions GeometryEngine can handle, do I end up with disjoint subgraphs?  Or just a graph that isn't densely connected?

It actually ends up with disjoint subgraphs. I suppose this is the tradeoff between very conservative atom maps that delete most of the old atoms, and ones that leave old atoms in at the risk of creating untenable structures.
For now, I was going to just remove those molecules with a warning. Alternatively, we could alter the atom mapping criterion such that all molecules are connected to at least one other. Finally, we could also modify the GeometryEngine such that it can propose with less than one torsion (though I'd save this for later).
Next, I'll implement the visualization stuff so that we can have a better look at what's going on!

Thanks for the clarification!
My thinking (not necessarily correct) is to prioritize:
0. bugfixes to get things to run at all

visualization (so we know what is going on, can have things for paper, etc.)
minimal modifications to get essential systems to be used in the paper (amino acids, T4 lysozyme ligands) to run
optimizations to make MCSS criteria more permissive for other ligand sets (kinase inhibitors, etc)
optimizations to ligand proposal scheme to increase acceptance probabilities
considering how to allow GeometryEngine to make more flexible proposals if item 3 still doesn't help

I agree with your priorities, thanks for the input!

Ok, so I added in code to avert the torsion disaster. But now there is another issue: what if one of the chemical states that the ProposalEngine removes is the starting state? I'm tempted to say throwing an exception might be practical here. After all, the user has supplied input that cannot be used.
As an aside, the entire ValenceSmallMoleculeLibraryTestSystem is disconnected under the current atom mapping scheme.

Perhaps we should include specification of atom mapping criteria in the constructor.

Ah, there's another issue with it. Hang on...

Ok, I see what's wrong. One second

Currently we're retaining 54/115 molecules in the T4 inhibitor test system with current atom mapping options.

Currently we're retaining 54/115 molecules in the T4 inhibitor test system with current atom mapping options.

Do you think that's enough?  We might try some tinkering to see if we can improve that.

Is this guy good to go if you're up to 115/115 molecules now?

Yeah, I think we can merge. Going beyond that will require additional, more complicated changes.

Let's see what the current travis tests say about the last commit--should have fixed the sampler issue and topology proposal test issue.

Miniconda MD5 mismatch...

https://travis-ci.org/choderalab/perses/jobs/119378544#L178

Switch the devtools/install.sh to use Miniconda2-latest instead of
Miniconda-latest

ok

Nice, the ValenceSmallMoleculeLibraryTestSystem actually passes now!

Travis finally passes! Merging.

Woohoo!


Whitelist another known spurious curl error

Seen recently in comments on #6788

r? @Eh2406
(rust_highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit f54ac5a26cad6267338c8bd7ba6f47721eaf7465 has been approved by Eh2406

:hourglass: Testing commit f54ac5a26cad6267338c8bd7ba6f47721eaf7465 with merge 15d8044f05551446aeef638354b79aba3389a4e1...

:sunny: Test successful - checks-azure
Approved by: Eh2406
Pushing 15d8044f05551446aeef638354b79aba3389a4e1 to master...


Unbound Domain Overrides support for forward-zones instead of stub-zone

There seems to be an issue that some uses run in to where domain overrides in Unbound might point to DNS servers that are forwarders themselves (i.e. another OPNSense box with its own overrides and zones). Unbound stub-zones don't allow this as they can only point to authoritative name servers.
The forward-zone option does the same thing but allows non-authoritative to be used instead of just returning a SERVFAIL to the client.

Ping?
Perhaps the easier quick fix is to swap forward-zone for stub-zone for now?

This should be able to make 18.7.3 if testing goes well, thanks! :)

Will pull into 18.7.3 as promised. Just to be clear: 18.7.2 is next up. The addition comes afterwards. :)


Updated user-guide/getting-started markdown

Problem
Due to an additional padding of 60px for 'anchor' class, the user can not access links just above the anchor.
E.g. On User Guide -> Getting Started page, the link for Filters is not accessible.
Solution
Changed class name from 'anchor' to 'no-pad-anchor'.
Result
No visual change.
The links above anchors are accessible.

@ameyanaik11 Thanks!


hystrix not working with spring cacheable

Hi All,
I'm trying to use use spring cloud hystrix with spring cacheable annotation, it is not working.
following is code sample
@Cacheable(value = "test", sync = true)
@HystrixCommand(fallbackMethod="recoverGetValue")
public String getValue(int id) {
return id + "";
}
public String recoverGetValue(int id) {
System.out.println("getting data from different resource");
return id + "";
}
Cacheable annotation throws operationTimeoutException, but fallback method is not being called. How to solve this issue?

Make sure you order the aspects /interceptors so they run in the order you want. Eg hystrix before cache

I did but still it is not working
@Bean
@Order(value = Ordered.HIGHEST_PRECEDENCE)
public HystrixCommandAspect hystrixAspect() {
return new HystrixCommandAspect();
}
@EnableCaching(order= Ordered.LOWEST_PRECEDENCE)


Support destructured variable declarations

We need to support destructuring objects and arrays in all interpreters.
new StepFunction(({key} : Type => {
  const {val} = ..
  const [item] = ..
})

Event Bridge needs this too
.map({detail} => {
    const { { value } = detail;
    const [item] = detail.list;
    return value;
})


tsconfig automation

Thanks for the pull request. Very useful functionality. I have merged your first and third requests, but this one seems to be stepping onto toes of first one. Could you please submit new one that plays well with the earlier?


Trying to select an option other than "latest" will not work when installing or updating murkmod.

Describe the bug
Murkmod can only find the latest version of chromeOS. if you select options 1 to 4, or 6, it will result in an error when trying to install that complains about being unable to find the correct version, and it exits the program.
To Reproduce
Steps to reproduce the behavior:

Open the devmode terminal using CTRL+ALT+-> OR open mush
Login as root OR press 1 for Root Shell and use the command to open the murkmod installer
press keys 1, 2, 3, 4, or 6 (you need to select a version if pressing 6) and press enter.
answer the bootsplash prompt.
Wait a few seconds and see error

Expected behavior
It should install murkmod normally and proceed without an error.
Screenshots

System (please complete the following information):

Board [Dedede]
Version [v117 & v128] (tested w both)
Installation method used [devmode installer]

Additional context
-It works perfectly fine if you press 5 and use the latest version.
-All selectable versions are on chrome100.dev

Looks like Chrome100 changed how it stores stuff... I'll push a patch in a few minutes.

Bit more than a few minutes... but it seems to work now.


fix(List): saturating_sub to fix highlight_symbol overflow

An overflow (pedantically an underflow) can occur if the highlight_symbol is a multi-byte char, and area is reduced to a size less than that char length.
I am not sure of the best way to create a test for this, I only discovered it after resizing my ratatui window to as small as possible.
There might be a number of other potential overflows in ratatui, but I am not well versed enough in the source to know if they could ever actually occur. Basically anywhere that uses a Rect that calculates width by a subtraction, x-y, instead of using x.saturating_sub(y)

I am not sure of the best way to create a test for this, I only discovered it after resizing my ratatui window to as small as possible.

Probably it would work to create a test that makes the highlight symbol wider than the area that you're rendering the list to

There might be a number of other potential overflows in ratatui, but I am not well versed enough in the source to know if they could ever actually occur. Basically anywhere that uses a Rect that calculates width by a subtraction, x-y, instead of using x.saturating_sub(y)

Yup - there's definitely a few of these. We've gradually been adding bits and pieces to make it easier to do some of these calculations in ways that avoid overflows (e.g. using Rect::rows() / Rect::columns() for iterating instead of adding values to heights, or using Rect::clamp() to ensure that a rect cannot be moved outside a another one etc.)

That makes sense, I thought I had tried to write a test to do that, but I just realised that I had already implemented the saturating_sub, so of course it didn't overflow - even though I was looking for an overflow.
I'll update the PR with an overflow test, as I said there are some other places that maybe could do with a saturating_sub, but should I use a different PR for that?

That makes sense, I thought I had tried to write a test to do that, but I just realised that I had already implemented the saturating_sub, so of course it didn't overflow - hence my test was "failing" by passing.

I find a good way to handle something like that is always to show the output of the failing test before the change - regardless of the fix. Tests often have more value in their failing state than their passing state (the failing state helps you understand whether the test adequately explains the reason for failure, but the success state only shows that something succeeded).

as I said there are some other places that maybe could do with a saturating_sub, but should I use a different PR for that?

Sure. Another PR for these would be good.
As a side note, I wonder if there's a simpler way to write the list render implementation in a way that makes it easier to understand at a glance how it works. The underlying problem that might have made it easy to sneak that bug in was that understanding the list rendering requires understanding every line of code rather than being able to understand smaller extracted portions.


confusing UI for configuring allowed merge strategy

Description
The Default merge style for pull requests menu allows picking merge strategies that have been disabled by the checkboxes above. The wording of those checkbox items is different from the items in the menu (which is also the menu that appears on PRs), which adds to the confusion.
I suggest to replace both the checkboxes and menu with a list of items with enable/disable toggle switches, plus a radio button to pick the default.
Screenshots

Gitea Version
1.16
Can you reproduce the bug on the Gitea demo site?
Yes
Operating System
Fedora 36
Browser Version
Firefox 102.0

I think it's a ui design problem but a bug.


Missing evse_id and connector_id in PATCH location client calls

Both patchEvse and patchConnector are missing the ID for evse and connector in the path
https://github.com/IZIVIA/ocpi-toolkit/blob/486a2f3454ab6c185a10de8cfe221f29dbf9cc23/ocpi-toolkit-2.2.1/src/main/kotlin/com/izivia/ocpi/toolkit/modules/locations/LocationsCpoClient.kt#L188C1-L188C1
should be
      path = "/$countryCode/$partyId/$locationId/$evseUid",

https://github.com/IZIVIA/ocpi-toolkit/blob/486a2f3454ab6c185a10de8cfe221f29dbf9cc23/ocpi-toolkit-2.2.1/src/main/kotlin/com/izivia/ocpi/toolkit/modules/locations/LocationsCpoClient.kt#L211
should be
        path = "/$countryCode/$partyId/$locationId/$evseUid/$connectorId",

I could create PR for it, not sure what your contribution guidelines are

Thanks for your report :)
The project has not been prioritized by the company until recently, so there is no contribution guidelines defined yet. The project will undergo many changes in the coming months and should be ready for production in summer 2024 (2.1.1 support will be dropped). Expect some refactoring in version 2.2.1, but nothing that will completely change the approach. To get back on the subject, as long as the code is correct and roughly follows established conventions, there is no reason to reject a pull request :)
I hope that time will be allocated to properly set up this repository to make it more "open-source friendly"

done in 0.0.15


remove unused py package from requirements

This fixes the one remaining vulnerability in https://github.com/microsoft/Qcodes/security/code-scanning/52

This was a transitive dependency of pytest but is no longer in any version that we support


Start the move to yaml pipelines.

It's better to have the pipeline config in code, so let's move to yaml.

@sanoursa said:

Do you have any pipelines that are using these new yaml files? Everything seems fine, but I have no way to test or validate any of it

Yep: https://dev.azure.com/gvfs/ci/_build/results?buildId=4060&view=results

Ignore the failed check. I can't authorize access to the secret file without having the yaml out of my fork and into Microsoft/VFSForGit. Once I do, the checks should come online too.

@kewillford to set up the pipeline to post status, i first have to get the yaml file into the branch, as best as i can tell, because i need to authorize access to the secure file we have in the pipeline. Getting it all configured to post statuses will be my next PR. Thanks for the review!


Add files via upload

@Ahmed-SaeedM I am sorry, the domain h-eng.helwan.edu.eg is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Add tests with rust stable and nightly in CI

Pull Request
Related issue
Fixes #3402
What does this PR do?
In the test-linux job two tests are added :

one run with rust stable
one with rust nightly that runs once a day

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

[x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[x] Have you read the contributing guidelines?
[x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

bors try

Closing in favor of https://github.com/meilisearch/meilisearch/pull/3495, since this current PR was not working
Thanks anyway for the PR @hugoamalric!


Not installing under NuGet3

At least one package (Xam.Plugin.Connectivity) does not install under VS2015+NuGet3 due to the use of content files for the readme. I imagine other plugins are using the same technique.

Please see: https://github.com/jamesmontemagno/Xamarin.Plugins/issues/86


Massive amounts of random player heads

https://www.youtube.com/watch?v=y-bM6gVbWBQ
This happens when i switch tabs in chrome and click out of chrome, and seems that if the aboves true then it has to be more than 10ish players.

Confirmed.
I have noticed this too, and am investigating currently. It seems as though, for whatever reason, the nodes are being duplicated.
http://prntscr.com/fmwasd

This has been repaired in the newest commit.


Kylin-4037 Fix can't Cleanup Data in Hbase's HDFS Storage When Deploy Apache Kylin with Standalone HBase Cluster

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Pull Request Test Coverage Report for Build 4610

9 of 17 (52.94%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.007%) to 28.22%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

server-base/src/main/java/org/apache/kylin/rest/job/StorageCleanupJob.java
9
13
69.23%

storage-hbase/src/main/java/org/apache/kylin/storage/hbase/HBaseConnection.java
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
3
78.42%

Totals

Change from base Build 4606:
0.007%

Covered Lines:
23255

Relevant Lines:
82406

💛  - Coveralls


Add GDPR.md

Docs Build status updates of commit e006ee8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

DocsLegal/GDPR.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update readme file

After including Authy OneTouch two-factor authentication some of the development instructions should be updated to reflect the changes we made.
@kwhinnery, please have a look on this one.

:+1:


How does it work. is it detectable?

The vietnamese guy made one that loads it through a driver which is bound to be detected at some point. I was wondering if your approach is bannable. Idc about accounts i was just wondering

According to my analysis, Riot Vanguard's driver monitors almost all sensitive operation callbacks (processes, threads, objects, image loads, registry, etc.). I can only say that certain functions can be executed successfully if allowed, so theoretically, it knows what you are doing. If you haven't modified any sensitive data, I don't think it will issue a ban.

W

The vietnamese guy made one that loads it through a driver which is bound to be detected at some point. I was wondering if your approach is bannable. Idc about accounts i was just wondering

Please don't harbor too much malice. I took a quick look at its program, and actually doesn't use a driver. It's actually a disguise. Haha :)

yeah i figured it was a disguise as well. saw it was calling ornn.sys but not doing anything with it. i thought i didnt look far enough. thank you for doing service to this community.


Test feature

Target components
browser, docs, integration_tests
Why & who
Test feature
Feature or enhancement
Test feature
Solution & alternative
No response

Successful test


[CWS] small cleanup to hash resolver

What does this PR do?
This PR replaces all the string operations when computing the hash digest of a file by a string builder, reducing the number of allocations related to this code path.
Motivation
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes

/merge


App Crashes on IOS 15 and below

Each time i call controller.getSettings() the app crashes with this and i have no idea what this means.
023-02-01 15:20:28.375805+0100 Runner[62348:2123240] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[WKPreferences isTextInteractionEnabled]: unrecognized selector sent to instance 0x60000371f430'
*** First throw call stack:
(
	0   CoreFoundation                      0x00000001803f7978 __exceptionPreprocess + 236
	1   libobjc.A.dylib                     0x0000000180188800 objc_exception_throw + 56
	2   CoreFoundation                      0x0000000180406f14 +[NSObject(NSObject) instanceMethodSignatureForSelector:] + 0
	3   CoreFoundation                      0x00000001803fbbf0 ___forwarding___ + 1408
	4   CoreFoundation                      0x00000001803fde7c _CF_forwarding_prep_0 + 92
	5   flutter_inappwebview                0x000000010552ee30 $s20flutter_inappwebview20InAppWebViewSettingsC07getRealG03objSDySSypSgGAA0cdeF0CSg_tF + 6876
	6   flutter_inappwebview                0x0000000105508a34 $s20flutter_inappwebview12InAppWebViewC11getSettingsSDySSypSgGSgyF + 68
	7   flutter_inappwebview                0x00000001055668d0 $s20flutter_inappwebview22WebViewChannelDelegateC6handle_6resultySo17FlutterMethodCallC_yypSgctF + 16712
	8   flutter_inappwebview                0x00000001054d35dc $s20flutter_inappwebview15ChannelDelegateC7channelACSo013FlutterMethodC0C_tcfcySo0fG4CallC_yypSgctcACcfu_yAH_yAIctcfu0_TA + 44
	9   flutter_inappwebview                0x00000001054d3654 $sSo17FlutterMethodCallCypSgIegn_Ieggg_AByXlSgIeyBy_IeyByy_TR + 108
	10  Flutter                             0x0000000105ec2cd8 __45-[FlutterMethodChannel setMethodCallHandler:]_block_invoke + 112
	11  Flutter                             0x00000001059315fc ___ZN7flutter25PlatformMessageHandlerIos21HandlePlatformMessageENSt3__110unique_ptrINS_15PlatformMessageENS1_14default_deleteIS3_EEEE_block_invoke + 104
	12  libdispatch.dylib                   0x00000001081340b4 _dispatch_call_block_and_release + 24
	13  libdispatch.dylib                   0x0000000108135884 _dispatch_client_callout + 16
	14  libdispatch.dylib                   0x0000000108143f84 _dispatch_main_queue_callback_4CF + 1044
	15  CoreFoundation                      0x0000000180365d1c __CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__ + 12
	16  CoreFoundation                      0x000000018036025c __CFRunLoopRun + 2448
	17  CoreFoundation                      0x000000018035f3bc CFRunLoopRunSpecific + 572
	18  GraphicsServices                    0x000000018afdd70c GSEventRunModal + 160
	19  UIKitCore                           0x00000001843f03d0 -[UIApplication _run] + 964
	20  UIKitCore                           0x00000001843f51ac UIApplicationMain + 112
	21  Runner                              0x000000010436ad38 main + 64
	22  ???                                 0x000000010444de50 0x0 +<PHONE_NUMBER>
	23  ???                                 0xfb69800000000000 0x0 +<PHONE_NUMBER>3562886144
)

libc++abi: terminating with uncaught exception of type NSException
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[WKPreferences isTextInteractionEnabled]: unrecognized selector sent to instance 0x60000371f430'
terminating with uncaught exception of type NSException

I'm getting the same. Crashes on iOS 14.6 and 14.8.X whenever getSettings() is called (directly or indirectly, e.g. by calling getHtml()). Tried with Flutter 3.3.10 and 3.7.1, and package version beta.1 to beta.22.

The problem is coming from this line, if you fork it and comment it out it works buh i still don't know why i have to remove it in the first place since isTextInteractionEnabled and upgradeKnownHostsToHTTPS is available form iOS 14.5 above

cc @Manuito83

Thanks @donpaul120, this is exactly the same I've done (at least as a temporary fix). I've forked it, but instead of commenting it out, I changed it to a higher version both in InAppWebView.swift and InAppWebViewSettings.swift. With the platform versions I've got available, it seems that > 15.0 should work (or at least it does in the simulator). This matches what I'm seeing in Crashlytics, where only version 14.6, 14.8.0 and 14.8.1 seem affected.
if #available(iOS 15.0, *) {
    realSettings["isTextInteractionEnabled"] = configuration.preferences.isTextInteractionEnabled
}

if #available(iOS 15.0, *) {
    if newSettingsMap["isTextInteractionEnabled"] != nil && settings?.isTextInteractionEnabled != newSettings.isTextInteractionEnabled {
        configuration.preferences.isTextInteractionEnabled = newSettings.isTextInteractionEnabled
    }
}

But, as you said, iOS documentation states that it should be available > 14.5, so I'm not really sure why this is an issue (but I don't think we should submit for this until we find out). In any case, it doesn't look like we are missing much (except the ability to use this particular option in a couple of iOS versions).

Released new version 6.0.0-beta.23 that checks only for iOS 15.0+.


WebJobs with Node (and node modules)

I am bringing in all my NPM dependencies but it seems that makes the file paths too long.
Such as this is the error log. I even went into each node_module to bring their deeper node_modules into the topmost folder... and that didn't seem to work. I'd like to check that it's not because I'm compressing the files on a macintosh... I see some left overs in VS online indicating that azure puts node modules in a temp file and then converts them.
[01/23/2014 03:23:00 > 948219: SYS ERR ] Failed to copy job files: System.IO.PathTooLongException: The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters.
   at System.IO.PathHelper.GetFullPathName()
   at System.IO.Path.NormalizePath(String path, Boolean fullCheck, Int32 maxPathLength)
   at System.IO.Path.GetFullPathInternal(String path)
   at System.IO.File.InternalCopy(String sourceFileName, String destFileName, Boolean overwrite, Boolean checkHost)
   at System.IO.File.Copy(String sourceFileName, String destFileName, Boolean overwrite)
   at System.IO.Abstractions.FileWrapper.Copy(String sourceFileName, String destFileName, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Infrastructure.FileSystemHelpers.CopyDirectoryRecursive(IFileSystem fileSystem, String sourceDirPath, String destinationDirPath, Boolean overwrite)
   at Kudu.Core.Jobs.BaseJobRunner.CacheJobBinaries(IJobLogger logger)
[01/23/2014 03:23:00 > 948219: SYS ERR ] Failed to start job due to: System.InvalidOperationException: Missing working directory
   at Kudu.Core.Jobs.BaseJobRunner.InitializeJobInstance(JobBase job, IJobLogger logger)
   at Kudu.Core.Jobs.TriggeredJobRunner.StartJobRun(TriggeredJob triggeredJob)

Long Path Tool is a best software to solve this issue. 5++ Rating. Long Path Tool is a best software
it is a free software. You can Try it


Cannot paste with Ctrl+V

Please make this happen.

Turns out this is impossible using OpenFL legacy, which HaxeUI requires.

This will probably never be answered.


Empty submodule

The path /centralized-relay/centralized-relay.wiki is a submodule that is empty. I am not sure what it is, perhaps it should be removed.

Submodule is for it's wiki.
git submodule sync


Psych engine more keys  Null Object Reference

Describe your bug here.
I tried to play the mod on fnf more keys but when it gets to the middle of the level the game crashes

Command Prompt/Terminal logs (if existing)
editors/EditorPlayState.hx (line 456)
flixel/group/FlxGroup.hx (line 769)
editors/EditorPlayState.hx (line 361)
flixel/FlxState.hx (line 203)
flixel/FlxGame.hx (line 747)
flixel/FlxGame.hx (line 677)
flixel/FlxGame.hx (line 551)
openfl/events/EventDispatcher.hx (line 402)
openfl/display/DisplayObject.hx (line 1399)

Uncaught Error: Null Object Reference
Please report this error to the GitHub page: https://github.com/ShadowMario/FNF-PsychEngine

> Crash Handler written by: sqirra-rng

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No

Isn't that a fork? This report doesn't go here
You also posted two different errors

Describe your bug here.
I tried to play the mod on fnf more keys but when it gets to the middle of the level the game crashes
Command Prompt/Terminal logs (if existing)
editors/EditorPlayState.hx (line 456)
flixel/group/FlxGroup.hx (line 769)
editors/EditorPlayState.hx (line 361)
flixel/FlxState.hx (line 203)
flixel/FlxGame.hx (line 747)
flixel/FlxGame.hx (line 677)
flixel/FlxGame.hx (line 551)
openfl/events/EventDispatcher.hx (line 402)
openfl/display/DisplayObject.hx (line 1399)

Uncaught Error: Null Object Reference
Please report this error to the GitHub page: https://github.com/ShadowMario/FNF-PsychEngine

> Crash Handler written by: sqirra-rng

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No

Isn't that a fork? This report doesn't go here You also posted two different errors

sorry I don't know how to use github much.
but I managed to solve the problem myself!
This error is related to the game's NOTE_assets.png


LdSuperCtor: Update interpreter error message to match JIT

LdSuperCtor: Update interpreter error message to match JIT
TypeError thrown by operator 'LdSuperCtor' differs in Interpreter vs. in JIT.
Change operator code to align the messages.

@rajatd, @abchatra, this should be quick - could you take a look pls?
This is to fix the mismatch b.t.w. interpreter and JIT in TypeError message thrown by LdSuperCtor. Decided to take this route because the ROI of changing error-throwing helper call format from unary to binary (such that extra string can be passed) has not been justified.

The change itself looks good. I'm fine if you guys are fine changing the error message.

Changing the error-throwing helper for JIT may make sense, although I see it as lower priority unless we need it in more often than isolated cases like this.


Update gitextensions.json autoupdate for new naming convention

It looks like GitExtensions has changed their naming convention from GitExtensions-$version-Mono.zip to GitExtensions-$version-portable.zip as of version 2.51.02 (https://github.com/gitextensions/gitextensions/releases/tag/v2.51.02). So autoupdate isn't picking it up as I think it's supposed to

FYI, I ran scoop update and scoop status. I saw 9421d1b in the list of updates to the extras bucket, but when I ran scoop status I did NOT see an available update to git extensions

Updated to 2.51.02 in https://github.com/lukesampson/scoop-extras/commit/f27b61c8a50c5fab62a28240aa3f3544c9a94d31

Thanks! scoop update of git extensions worked for me after this change.


Feature/gradle kotlin dsl compatibility

Hi,
I was converting my groovy gradle build files and was not able to assign publications on the extension class. These are really small changes that make it possible to use this plugin when using kotlin gradle build files.
What changed:

replaced all String[] attributes on the BintrayExtension by List and initialized them with empty lists.
configurations and publications can now be properly added by using .add(...) on the initialized list attributes of BintrayExtension.
aligned the types on the BintrayUploadTask for the licenses and labels attributes.
updated the empty check on the publications and configurations attributes in the BintrayPlugin class.

Hello @teyckmans , do you know if this PR solves this issue.

Hi @GuilhE , I don't think I had a problem accessing the extension it was just not that handy to use with the Kotlin build scripts.
In this build https://github.com/teyckmans/ui-rig/blob/master/ui-rig-core/build.gradle.kts I was testing the changes.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


css for -webkit

maybe add follow style to html body make fonts clear:
-webkit-font-smoothing: antialiased;
-webkit-text-stroke-width: 0.2px;

Not sure about it. It is not required for default/system fonts, and could produce performance issues. So lets leave it up to developer, because it is always easy to add these two lines in custom app's CSS file

you are the pro. hahaha


navbar adding 'navbar-brand' class to horizontal list

When using the following ERB template (taken from the docs) with bh (1.3.1) in Rails (4.2.0)
<%= navbar do %>
    <%= vertical do %>
        <%= link_to 'Home', root_path %>
    <% end %>
    <%= horizontal do %>
        <%= nav class: 'navbar-right' do %>
            <%= link_to 'Profile', root_path %>
            <%= link_to 'Settings', users_path %>
        <% end %>
    <% end %>
<% end %>

I end up with

<button class="navbar-toggle" data-target="#navbar-collapse-8310700815" data-toggle="collapse" type="button">Toggle navigation

</button>

    Home

Profile

Settings

I will look into this issue and make a PR if needed

Indeed seems to be an issue!

Hello @zenry (and sorry for the delay).
The reason why I add navbar-brand is because, in the Bootstrap navbar documentation, whenever there is an element with class="navbar-header", then the links inside always have the class="navbar-brand".
So, I thought that to be consistent with Bootstrap CSS, you would only have links with the navbar-brand class inside the navbar header.
That is indeed the case in all my personal projects where I use Bh.
Could you show me an example that requires an item with class="navbar-header" which includes a link without the navbar-brand class? I could proceed from there…
Thanks!

@claudiofullscreen If you take a look at the menu on getbootstrap.com itself, they only have navbar-brand on their name, not all of their links.

Also note that the other li > a items are not contained inside of navbar-header.

If you take a look at the menu on getbootstrap.com itself, they only have navbar-brand on their name, not all of their links.

@jeroenvisser101 Oooook now I understand. I misread and I thought you were talking about the link inside the
 element, and that link has the class="navbar-brand" applied.
You are saying that the links inside horizontal should not have the navbar-brand class applied, but they do.
Well… I just tried again with a brand new Rails 4.2 app and the latest Bh gem and I actually don't see the navbar-brand class inside the horizontal links.
Can you download my test app and tell me if that matches your expectations?
This is the code that I use:
  <%= navbar do %>
    <%= vertical do %>
      <%= link_to 'Home', root_path %>
    <% end %>
    <%= horizontal do %>
      <%= nav class: 'navbar-right' do %>
        <%= link_to 'Profile', root_path %>
        <%= link_to 'Settings', '/#settings' %>
      <% end %>
    <% end %>
  <% end %>

and this is the resulting HTML that I see:

      <button class="navbar-toggle" ...>...</button>
      Home

Profile

Settings

If you still experience the bug, I will reopen the issue.


[Exploration] Build before test

I found out that in the xcodebuild commands it is required to have build test instead of test for Xcode to launch the simulator.
Otherwise, if your build is taking longer than 2 minutes, it aborts because of the timeout.
As I saw in your Makefile your are just calling test so that might fix the problem described in https://github.com/artsy/eidolon/pull/379#issuecomment-74949058?

Cool. Failure on CI is due to lack of access to Travis secure environment variables on PRs from forks. Meeerging thanks!

@pietbrauer Such spotting, amaze! :heart:

Did it work?


Question: why is compression disabled by default in the latest version?

This is a question rather than a issue: why are you disabling the compression in the latest version? I saw many changes in the driver about this and I was wondering if this brought issues either in the driver or in the cluster

Hi @paoliniluis thanks for bringing this up! You can also use the Discussion section instead of the Issues section for questions :)
We noticed that the benefits of compression vary depending upon network throughput and how much the returned data compresses. In general, we see performance benefits in lower throughput network environments (e.g, multi-region, customer VPN, etc.). Utilizing compression in higher throughput networks can make performance slightly worse if the overhead of using compression is higher than the time savings of sending less data.
Since we do not know which customers have higher/lower throughput networks, we decided to turn it off by default. Customers are welcome to enable it for their use cases by setting the connection property compression=lz4.


test(cli): setup login util for E2E

I added a isElementClickable function to prevent relying on the timeout.
It's a bit overkill but it works every time.


[FEATURE] Rebase Harvester on SLE Micro for Rancher

This feature request is to rebase Harvester's OS platform on the new operating system SLE Micro for Rancher.
Dev updates
Tasks

[x] Clarify how to install packages using zypper with SLE Micro for Rancher team

If we want to install from SLE Micro repository, seems like the only way is via OBS (Open Build System). Is it possible to pull the image and install packages using a Dockerfile?
Some packages required by the original Harvester OS image are already installed on SLE Micro for Rancher image. I've written a script to test what is missing, and those listed here should be it.
What about those packages that are missing SLE Micro repository? (nano and rng-tools)

[x] Double-check the unnecessary luet packages

Already done some cleanup. Still worth checking with SLE Micro for Rancher team to see if some of them will be covered by the image.

[x] Fix GRUB rebranding issues

After installation, GRUB menu does not get rebranded. Still shows Rancher OS
SLE Micro for Rancher contains a cOS config /etc/cos/config which has higher precedence than our cOS config /etc/os-release.
Solution: Remove /etc/cos/config while building the Harvester OS image. Most of the settings inside are default values anyway.

[x] Specify Rancher's version installed by Rancherd

New way: https://github.com/harvester/harvester-installer/pull/146
The current way still works. as Rancher's version is still pinned to the one we needed.

[x] Replace bootstrapResources with resources

bootstrapResources is deprecated by Rancherd. We need to replace them with resources.
This will not affect the upgrade path since bootstrapResources/resources are only for bootstrapping Rancher. Once the Harvester clusters are created, they are not used afterward.
If this also works in the older version of Rancherd we can merge this change before we even updated the Harvester OS.

Tentatively scheduled for v1.0.2.

Can I ask why SLE Micro and not openSUSE MicroOS? The former requires registering to get updates whereas the latter does not. Or perhaps you're planning on disabling transactional updates?

@simonflood SLE Micro for Rancher is different from SLE Micro. It's an OS with enterprise container focus. We will share more details later.
Rest assured. We won't require users to register to get updates. There should be no difference in the user experience. Harvester users will be always able to get updates for free.

Thanks. Perhaps you meant to write "SLE Micro for Rancher is different from openSUSE MicroOS"? SLE Micro for Rancher sounds a lot like SLE Micro which "is an ultra-reliable, lightweight operating system purpose built for containerized and virtualized workloads. It leverages the enterprise hardened security and compliance components of SUSE Linux Enterprise and merges them with a modern, immutable, developer-friendly OS platform." (from https://www.suse.com/products/micro/ )
If SLE Micro for Rancher is not the same as SLE Micro then I suggest renaming it to avoid confusion. I'll await further details before commenting further.

@simonflood Sorry for the confusion. I've reworded my comment. SLEMicro for Rancher is SLEMicro in a different delivery method, that will be freely available to Harvester users.

We also need to check if the interface name rule change also happens in the new OS: https://github.com/harvester/harvester/issues/2073

Add require/release-note label. We'll share with community about the rebase.

https://github.com/harvester/harvester-installer/pull/296

Test plan

Check can download support bundle correctly, check no empty file
Check debug command exists (include necessary packages)
Check the daily E2E test result
Can upgrade 3 nodes harvester correctly
Version check
$ cat /etc/os-release
Major feature smoke test

Images
Volumes
Virtual Machine
Live Migration
Backup
Rancher integration

Test Result

[ ] PASS Check can download support bundle correctly, check can access every file without empty

[ ] PASS Check debug command exists (include necessary packages)

k9s
kubectl
ip
lsblk, gisk
tar, gzip
ctr
zstd

[ ] PASS : Daily E2E test on Jenkins server

[ ] Ongoing: Upgrade 3 nodes harvester correctly

[ ] PASS : Version check

SLE Micro: 5.2
VARIANT_ID: Harvester-20220705

harvester-gv2ks:~ # cat /etc/os-release
NAME="SLE Micro"
VERSION="5.2"
VERSION_ID="5.2"
PRETTY_NAME="Harvester master"
ID="sle-micro-rancher"
ID_LIKE="suse"
ANSI_COLOR="0;32"
CPE_NAME="cpe:/o:suse:sle-micro-rancher:5.2"
VARIANT="Harvester"
VARIANT_ID="Harvester-20220705"
GRUB_ENTRY_NAME="Harvester master"

[ ] Ongoing Major feature smoke test

Images
Volumes
Virtual Machine
Live Migration
Backup
Rancher integration

Verified fixed on master-0f590757-head. Close this issue.
Result
According to the previous test result https://github.com/harvester/harvester/issues/1933#issuecomment-1182832981
We have passed all test plan scenarios.
Test Information

Test Environment: 4 nodes harvester on dolphin bare machine
Harvester version: master-0f590757-head

Verify Steps

Download support bundle in support page
Extract support bundle and check every file content
Vagrant install master release
Execute backend E2E regression test
Run frontend Cypress automated test against feature Images, Networks, Virtual machines
Run manual test against feature Volume, Live migration and Backup and rancher integration


update diagnosis key validation

Align diagnosis key validation to be more in line with apple+google export proto expectations

check that diagnosis keys are exactly 16 bytes

check upper end of invalid rolling period, not just lower end

0 is a valid transmission risk level, and what may be present if that is omitted (client is only sending reportType)

on the matching side, 0 is used as 1, at least on Android matching

update test data to have 16 byte keys by default, update associated test hash value

add additional test cases for key validation

mvn page
and
mvn test
succeed locally. Does anyone have advice here?

I restarted the job it succeded now

rebased

rebased

See also :  https://github.com/google/exposure-notifications-server/pull/1041
TRL Field was declared as "0x7FFFFFFF" for EU exchange in cause of different handling of different apps to avoid a misinterpretation and false alarms. (DSOS and RT Field has a more proper epidemilogically meaning during the exchange, better translateable in different models) Validation should exclude all these fields. DSOS can be overloaded to more than 14. (Normalized by the national backends)

How would you like to proceed?

I would hope that validating TEK length is not controversial - I propose to leave that in.

Rolling periods of 1-144 seems not controversial as well - I propose to leave that in.

TransmissionRisk level - the Google exposure-notifications-server accepts 0 and will attempt to pass 0 to this server. If this sever continues to reject 0, we will flip all of our 0s to 1s (which is what happens in client side matching). - I can remove this.

The DSOS field overloading is still not documented on this repository - I would like to propose changing the protocol to add proper semantic fields to capture the data that is being transited through field overloading. I am happy to file an issue to propose this.

We have observed bugs in the field where this field is being used incorrectly and devices are being told of DSOS of very large magnitude values (18,000+) - Failure to provide correct data may result in missed matches due to out of range fields.

We open a issue for that #185 Please add some technical comments there. Proposal of validation agreed for:

[x] Keylength must be 16
[x] 0<RP<=144
[x] TRL Field between 0<TRL<=8, BUT EFGS Architecture describes 0x7FFFFFFF as default, if the field is not properly or not used.
[ ] DSOS Field and Report Type is still under discussion and should be excluded from the validation. Please open a issue with your proposal.

Agree with @mikehelmick. Google uses 0 for TRL by default (we have all values with 0 in our db). Also take into account that this attribute is only for v1 compatibility. It is deprecated for new API version.
So... would you allow to send 0 in TRL?

Yes we can allow it. But i have to point out, that this values can have different meanings between the member states.  Some countries just using 8. Others using a range between 1 and 7. What 8 then means in this range is not clearly defined. So when we allow 0 to 8, its no quarantee that your epidemilogically algorithm has exact the same result as meant by the sender. (false alarms possible) We keep 0x7FFFFFFF as fallback to indicate that this value is not used.

The recommended path is to not use TR at all. In our server implementation, we do automatically backfill TR for different report types in the event that there are older clients (OS implementations) processing the files.
specifially: https://github.com/google/exposure-notifications-server/blob/main/pkg/api/v1/verification_types.go#L49

unkown -> 0
confirmed test -> 2
clinical -> 4
negative/revoked -> 6

These are the values set by the Google and Apple clinical teams.
If EFGS doesn't accept 0, the Google server implementation will pass 1 when 0 is in the database (which matches the Android client implementation anyway).

The commits of this PR are integrated with some further validation implementation in this PR: #194
Maybe we can continue the discussion over there and close this one?

Closing in favor of #194


romfs: expand file cache by CONFIG_FS_ROMFS_FCACHE_NSECTORS

Summary
The default size of file cache is size of a sector, it may not be good size for optimizing read speed in physical device. So we can set the config according to speed test profile to optimize access IO speed.
Signed-off-by: dongjiuzhu1<EMAIL_ADDRESS>Impact
improve read speed for romfs
Testing
Vela CI

let ignore the ci broken, which is fixed in:
https://github.com/apache/incubator-nuttx/pull/7409


Batch gradient descent theta does not updated

Hi @ageron , in chapter 4, section Linear regression using batch gradient descent following code does not update theta. Am a missing something?
eta = 0.1
n_iterations = 1000
m = 100
theta = np.random.randn(2,1)

for iteration in range(n_iterations):
    gradients = 2/m * X_b.T.dot(X_b.dot(theta) - y)
    theta = theta - eta * gradients

Hi @alparslanuysal ,
Well actually the last line of this code does update theta:  I could have written instead theta -= eta * gradients, but I wanted it to look like the equation.
Cheers,
Aurélien

Answer to this issue: https://github.com/ageron/handson-ml/issues/16 , adding:
from __future__ import division
to your notebook if using Python 2 also solves this.


Convert to pandas dataframes, update for Python 2.7 compatibility

Codecov Report

Merging #1 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master       #1      +/-   ##
==========================================
- Coverage   98.62%   98.59%   -0.03%
==========================================
  Files           4        4
  Lines         145      142       -3
==========================================
- Hits          143      140       -3
  Misses          2        2

Impacted Files
Coverage Δ

plsxml/data/loader.py
100% <100%> (ø)
:arrow_up:

plsxml/plsxml.py
97.82% <100%> (+0.07%)
:arrow_up:

plsxml/plsxml_test.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Updated README.md

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:

/kind documentation

What this PR does / why we need it: Updated README.md with community meeting information
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: None

/lgtm

/approve

/approve

/approve

/approve

/approve

/approve

/approve

/approve

/approve

/approve

/lgtm

/approve


add links to source code

looking at a random plugin (e.g. https://studiorack.github.io/studiorack-site/instruments/studiorack/adlplug/adlplug) there is no link to source code, nor any way to verify the build is from you, or how it is packaged or anything at all.

https://github.com/DISTRHO/Cardinal/issues/653

The link to the source code is already there, but linked from the Author.
I agree it could be made more prominent/obvious.
Adding to enhancements

Add a view source button which makes the link more obvious and easier to review before downloading any plugin:


Support Slack's app_mention Event

Bot Info

SDK Platform: Node.js
SDK Version: 3.14.0
Active Channels: Slack
Deployment Environment: Azure Bot Service

Issue Description
Add support for the app_mention event in Slack. Right now subscribing to app_mention doesn't do anything. This means that the bot has to be specifically coded for Slack. It would be nice if there was an option to just have the bot respond whenever it was mentioned in a channel instead of all the time.
Expected Behavior
Adding in the app_mention event in Slack would trigger a response from Bot Service whenever a bot is mentioned.
Actual Results
Bot Service doesn't receive the app_mention event so there is no response from the bot when mentioning if channel.messages is turned off. If it is turned on, the bot answers to all messages, not just ones where it is mentioned.

Turning on the app_mention event while channel.messages is on adds an item to the array "entities" that includes a field type within the message object when sent via slack. Unfortunately there is no current way to automatically filter only these messages without additional slack specific code.  However, when this event is on, it is possible to create middleware (v4 version) that checks if slack messages include this field with a mention.
Here's an example middleware implementation using v3 (requires using a message to avoid resetting the current dialog, however session.reset() can be used if all dialogs are 1-step)
var util = require('util');

module.exports = {
    isMentioned: function(session, next) {
        if (session.address.channelId === 'slack' &&
            !(
                session.entities[0] && session.entities[0].type === 'mention' &&
                session.entities[0].text === util.format("@%s",session.address.bot.name)
            )){
            console.log("not mentioned");
            session.send(); //send empty message if a slack message didn't include a mention
	 }
	else{
            console.log('mentioned');
            next();
        }
    }
}

We will look into including this natively later on. I will update if this is possible in v3 without sending a message. That said, it is already possible to do this in v4 without sending a message. (similar to the implementation above, but with "return" instead of sesssion.send())

Turning on the app_mention event while channel.messages is on adds an item to the array "entities" that includes a field type within the message object when sent via slack. Unfortunately there is no current way to automatically filter only these messages without additional slack specific code.  However, when this event is on, it is possible to create middleware (v4 version) that checks if slack messages include this field with a mention.
Here's an example middleware implementation using v3:
var util = require('util');

module.exports = {
    isMentioned: function(session, next) {
        try{
            if (session.address.channelId === 'slack' &&
                !(
                    session.entities[0] &&
                    session.entities[0].type === 'mention' &&
                    session.entities[0].text === util.format("@%s",session.address.bot.name)
                )){
                console.log("not mentioned");
            }
            else{
                console.log('mentioned');
                next();
            }
         }
         catch(err){
             console.error(err);
         }
    }
}

We will look into including this natively later on.

Thanks! I'll look into the middleware solution.

Thank you for opening an issue against the Bot Framework SDK v3. As part of the Bot Framework v4 release, we’ve moved all v3 work to a new repo located at https://github.com/microsoft/botbuilder-v3. We will continue to support and offer maintenance updates to v3 via this new repo.
From now on, https://github.com/microsoft/botbuilder repo will be used as hub, with pointers to all the different SDK languages, tools and samples repos.
As part of this restructuring, we are closing all tickets in this repo.
For defects or feature requests, please create a new issue in the new Bot Framework v3 repo found here:
https://github.com/microsoft/botbuilder-v3/issues
For Azure Bot Service Channel specific defects or feature requests (e.g. Facebook, Twilio, Teams, Slack, etc.), please create a new issue in the new Bot Framework Channel repo found here:
https://github.com/microsoft/botframework-services/issues
For product behavior, how-to, or general understanding questions, please use Stackoverflow.
https://stackoverflow.com/search?q=bot+framework
Thank you.
The Bot Framework Team


Address high bounce rate on /docs/get-started

Problem
/docs/get-started has the 4th-highest blended score (4152) for bounce/traffic/negative feedback.
Proposed Solution

[ ] Make 'cloud providers for other clouds' more discoverable with a CTA button instead of a link.

[ ] Add better intro text "Ready to get started using Pulumi on your preferred cloud using your favorite languages? Pick a cloud below for step-by-step instructions and you'll be up and running in no time."

Intro text has been updated to be more descriptive. See https://www.pulumi.com/docs/get-started/.
The other cloud providers link has remain unchanged (i.e. not a CTA button). Updating the getting started experience for additional cloud providers will be based on usage, customer feedback, and page view information.

Intro text has been updated to be more descriptive. See https://www.pulumi.com/docs/get-started/.
The other cloud providers link has remain unchanged (i.e. not a CTA button). Updating the getting started experience for additional cloud providers will be based on usage, customer feedback, and page view information.


DS-92 Distributed Architecture & Clustering

DS-92
Figure out how to distribute a sim over n peers in a way that is portable.

For docs, see: https://github.com/yetanalytics/datasim/blob/DS-92_distarch/README.md#distributed-generation

For docs, see: https://github.com/yetanalytics/datasim/blob/DS-92_distarch/README.md#distributed-generation


Script for creating new Pod.Cast annotation "rounds"

Simplify the process of creating new annotation "rounds".
@scottveirs maintains Orcasound hydrophone events in the Gsheet over time here -- and there's a column for prioritizing which events are optimal for annotation rounds in Pod.Cast | Help listen for Orcas!.
Creating a new round involves the following:

Pulling audio files from Orcasound AWS S3 and breaking them into 1 min *.wav chunks
Processing these *.wav chunks with a model and save predictions in corresponding JSON files
(optional) filter out very low confidence chunks to reduce annotation burden
Uploading these files in appropriate structure to Azure Blob
No update is required to the flask web-app, things will be available under the /roundN suffix to the URL

Identifying interesting events from the GSheet is OK to trigger manually now - this issue is for a prepare_for_podcast.py script to simplify the rest of it.

@scottveirs - I think we have some version of this, that hasn't been cleaned up & added to this repo yet.
It's likely based on this on aifororcas-livesystem that needs to be consolidated.


Allow one to remove the service from the catalog

Allow a deployer to not have the service resource from the catalog.
This feature is useful when one manages the service through another
program, like pacemaker.
With a pacemaker setup, a user ensure a service is stopped, pacemaker is
in charge of starting it. At a subsequent run, puppet will make sure the
service is stopped again. To avoid that, this feature offer an option to
the user to not manage the resource at all.

:+1:


Otio wont start

after a fresh install of opentimeline via
pip install opentimelineio
I tried to open it with
otioview path/to/your/file.edl
then it return
otioview /home/william/Documents/test_edl.edl
Traceback (most recent call last):
  File "/usr/bin/otioview", line 7, in <module>
    from bin.otioview import main
ImportError: No module named bin.otioview

i'm on Fedora 28, python 2.7.14, PySide2

This was fixed in https://github.com/PixarAnimationStudios/OpenTimelineIO/commit/f1e53e5de7985b63c440c280e4e6d7eb4c1c17fa on July 31, but the version that is tagged and on PyPi - https://github.com/PixarAnimationStudios/OpenTimelineIO/tree/v0.8 - is from July 24. It's up to the maintainers if they want to push a version 0.8.1 which would fix this, but if you need to install a working version, you can clone and pip install from the clone and this will be fixed.

@Kaiz3rTool are you able to get this working from the latest master branch?

Sorry i missed your answers
yes with installing the 0.9.0 version it work like a charm
thank you

pip install git+https://github.com/PixarAnimationStudios/OpenTimelineIO.git
pip install pyside2
otioview /path/to/file.edl

@melMass @Kaiz3rTool I've pushed a new version of OTIO to pypi (v0.8.1) which should fix the issue with the commandline scripts not working.


Changing default calendar permissions is broken

If you want to change the default calendar permissions in Dashboard/System&Settings/Calendar/Permissions the URL provided in this file does't work:
https://github.com/concrete5/concrete5/blob/develop/concrete/elements/permission/details/calendar_admin.php#L10
When you click on, lets say "View calendars" on the dashboard page, you get the error ".../concrete/elements/permission/details/calendar_admin.php:10 Call to a member function getID() on null (0)"
This error didn't come up in version 8.5.2 but occurs in 8.5.4 and 9.0.0a3.
Seems like the calendar ID (caID) is only needed if you edit a specific calendar's permissions in Dashboard/Calendars & Events but it's not needed if you edit the default permissions in Dashboard/System&Settings/Calendar/Permissions.

Confirmed by a Japanese user and reported to the Japanese forum
https://concrete5-japan.org/community/forums/beginner/post-19483

#8534 is the reason for this error.

The same error reported on Slack.
https://concrete5.slack.com/archives/CDJ4G93C3/p1617864602150500
Why this issue closed silently?

I believe this issue is not fixed yet. Tested on recent 8.5.x branch

It was closed because @KorvinSzanto said he fixed the issue already.

@aembler  Thanks for the reply, could you re-open it?

It might be fixed already on develop branch. Need to test


Fixes for some of the errors known at compile time

Add the missing static libraries and move them into nrf_sdks. Accordingly,the Makefiles have been updated.

2.The command mergehex is not usually installed in advance.A README.md has been added in order to make the compilation process smoother.
All these changes have been carefully validated.
The compiling environment is :
Linux 4.18.0-16-generic Ubuntu 18.04.1 x86_64
GNU Arm Embedded Toolchain: 6-2017-q2-update
nRF5 SDK: nRF5_SDK_v15.2.0_9412b96

@vinsec micro-ecc library is in the folder <nRF5 SDK path>/external/micro-ecc. Just change to the directory and run ./build_all.sh or .\build_all.bat. So, I think it is better to add a tutorial for this step rather than adding the pre-build libs.

@vinsec micro-ecc library is in the folder <nRF5 SDK path>/external/micro-ecc. Just change to the directory and run ./build_all.sh or .\build_all.bat. So, I think it is better to add a tutorial for this step rather than adding the pre-build libs.

That would be better :)


License needed

This repo needs a LICENSE.md :-)

Any thoughts on a code license (e.g. APACHE 2) vs a content license (e.g. CC-BY)?

I'm perfectly fine with using CC-BY for this. I'll add a file.

Thank you very much! Does the CC-BY also cover the source code part? I am not a super expert on licenses but often I see CC licenses covering the "content" part and one of the code licenses the source code part. I don't want to over-complicate this however. @rlonka

Well, we've hardly written any source code for this site. Most of it is just plain 'minima', which has MIT License (https://github.com/jekyll/minima). We've only changed the colors and inserted a few things like a logo and a twitter bar, nothing to worry about.

OK. Thank you again :-)


g:wintabs_ui_active_left Not applied to first tab

I believe the first wintab doesn't show the active_left character:
let g:wintabs_display='statusline'
set hidden
let g:wintabs_autoclose=2
let g:wintabs_autoclose_vim=1
let g:wintabs_autoclose_vimtab=1
let g:wintabs_switchbuf='useopen,usetab'
let g:wintabs_ui_active_higroup = 'TabLineSel'

" If doing some custom rendered GUI use a not-actually space.
" let intertabSpace= ' '
let intertabSpace= ' '
let activeWinTabChar = '|'
" This must be different to distinguish.
let activeVimTabChar = '|'
let activeWinTabChar = '▎'
" This must be different to distinguish.
let activeVimTabChar = '▍'
let g:wintabs_ui_sep_leftmost=intertabSpace
let g:wintabs_ui_sep_inbetween=intertabSpace
let g:wintabs_ui_active_right=intertabSpace
let g:wintabs_ui_active_left=activeVimTabChar
let g:wintabs_ui_sep_rightmost=intertabSpace
let g:wintabs_ui_modified=' ◎'
let g:wintabs_ignored_filetypes=[]
" Annotating the active vimtab makes it always available for detection
let g:wintabs_ui_active_vimtab_left=activeVimTabChar
let g:wintabs_ui_active_vimtab_right=intertabSpace

let g:wintabs_ui_arrow_left='«'
let g:wintabs_ui_arrow_right='»'

Not sure if this is one of the old design flaws that I now treat as a "feature". I rewrote some of the rendering steps to support unicode and custom rendering, which might be able to fix it. I haven't documented the changes yet, but you can take a look at https://github.com/zefei/vim-wintabs-powerline and see if that can help.

I see one can define their own custom renderer now. Nice.

Created a pull request in case you feel like it's worth merging: https://github.com/zefei/vim-wintabs/pull/28
Here's how I'm using it: To create the appearance of an IDE "currently selected tab" UI:
Without my PR, the blue selected line doesn't show up on the leftmost tab when active.

Manually merged.


Allow users to switch between different default search table configurations (e.g. specific data types)

Specifically, types such as Neuron Morphologies or Neuron Electrophysiologies might have different fields/columns that are not relevant for the other. The users would be able to switch between different configurations that have different: column ordering, hidden columns, filters, sorting.
At the deployment level, default configurations can be created and will be accessible to all users (anonymous as well).
A configuration saves the following parameters:

column order
hidden columns
filters
sort criteria

Expected behavior:

A default deployment configuration will have a name and a description
Multiple configurations can exist
Configurations can be selected in the search app (i.e. through the configuration dropdown menu)
Switching to a configuration will apply the corresponding above parameters to the interface

Out of scope:

column width
decimal digits
row height
searching on a config from the search bar
merging filters and sort criteria on config switch
temporary config save (user local storage)
sharing link to config (incl. deployment ones)

Hey team! Please add your planning poker estimate with ZenHub @adulbrich @imsdu @nicwells @smitfire @wwajerowicz


How to make it work both ways (Android Pixel 8 to MacBook Air M2 not working for me)?

Do I have to do anything special to make it work both ways?
My situation:
MacBook Air M2 → Pixel 8: Works
Pixel 8 → MacBook Air M2 → I don't get any popup or anything where I can add the pin and confirm receiving the files, so it's just trying to send until it aborts the job
I am not sure if I am simply doing something wrong, but I haven't seen a hint in the FAQs.

make sure your "do not disturb" is off

That did it! Thanks a lot - I didn't notice that I have "DND" permanently on on my MacBook


leave-delete-or-deactivate-an-app

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: leave-delete-or-deactivate-an-app
Document link: leave-delete-app.md
My Issue/Suggestion
i am not able to  switch back to Mendix Public Cloud for delete

Hi,
This issue mechanism is for reporting problems with the documentation. Your comment seems to be about a problem you are having with the way your app has been set up.
If you are still having an issue, please contact Mendix Support or, alternatively, ask a question on the Mendix forum where there are many community members who may be able to help you.
If you have a specific issue with the process description in the documentation, please feel free to raise a new issue and we will be happy to investigate for you and update the process to make it clearer or indicate additional steps which might be needed.
Yours
Mark van Ments


docs: Fix typo

Noticed a place in the docs where it references "Python" instead of "JavaScript", thought I'd submit a quick PR to fix it.
Thanks!

Thank you!


Inheriting GenericModel does not make the subclass Generic

Checks

[X] I added a descriptive title to this issue
[X] I have searched (google, github) for similar issues and couldn't find anything
[X] I have read and followed the docs and still think this is a bug

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.7.3
            pydantic compiled: True
                 install path: /Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/pydantic
               python version: 3.8.6 (v3.8.6:db455296be, Sep 23 2020, 13:31:39)  [Clang 6.0 (clang-600.0.57)]
                     platform: macOS-10.15.7-x86_64-i386-64bit
     optional deps. installed: ['typing-extensions', 'email-validator']

To start: I hope I have not misunderstood how Generics are supposed to work in pydantic. If I have, feel free to disregard this issue. My problem boils down to something simple. Let's say we inherit from a generic class in "plain python". If we do this, then the child class is also a generic, we can index it with a type parameter:
from typing import TypeVar, Generic
T = TypeVar("T")

class MyGeneric(Generic[T]):
    def __init__(self, field: T):
        self.field = field
    field: T

a = MyGeneric[str](field="abc")

class MyOtherGeneric(MyGeneric[T]):
    pass

b = MyOtherGeneric[int](field=1)

Trying almost the same thing in pydantic, we get an exception:
from pydantic.generics import GenericModel
from typing import TypeVar, Generic
T = TypeVar("T")

class MyGenericModel(GenericModel, Generic[T]):
    field: T

c = MyGenericModel[str](field="abc")

class MyOtherGenericModel(MyGenericModel[T]):
    pass

d = MyOtherGenericModel[int](field=1)  # this part breaks

Here's the error:
Traceback (most recent call last):
  File "........./generic.py", line 28, in <module>
    c = MyOtherGenericModel[int](field=1)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/pydantic/generics.py", line 52, in __class_getitem__
    check_parameters_count(cls, params)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/pydantic/generics.py", line 118, in check_parameters_count
    raise TypeError(f'Too {description} parameters for {cls.__name__}; actual {actual}, expected {expected}')
TypeError: Too many parameters for MyOtherGenericModel; actual 1, expected 0

The only workaround I've found is to use multiple inheritance and to keep adding Generic[T] as a second parent class on the child. In principle that works, but its easy to forget this (which would cause issues down the line that might not be caught right away).
Is this fundamentally supposed to work, and is there a way to make it work?

Hi @phdowling,
The docs point out that "To inherit from a GenericModel without replacing the TypeVar instance, a class must also inherit from typing.Generic." So this would resolve your issue:

from pydantic.generics import GenericModel
from typing import TypeVar, Generic
T = TypeVar("T")

class MyGenericModel(GenericModel, Generic[T]):
    field: T

class MyOtherGenericModel(MyGenericModel[T], Generic[T]):
    pass

d = MyOtherGenericModel[int](field=1)  # no longer breaks

Hi @tfwillems - thanks for the quick reply. I did notice that adding Generic as a parent on each subclass fixes it, sorry of course for missing it in the docs.
I guess that means it's intended behaviour, which makes this not so much a bug but rather a question: is there any chance this would ever be changed? Here's the issue I see: If I ship a library that uses pydantic types, with some of them being generic, users must always remember to subclass Generic even if their own code has no need for some given class to be generic (as long as the library needs the class to be generic somewhere down the line). It's easy for users to forget this, and potentially difficult for them to understand why it's be necessary.
I realize it might seem like a niche concern, but since it's somewhat unexpected behavior I'm wondering if this was a deliberate design decision or whether might be changed in the future (or, if say I manage to figure this out by myself, whether a PR that changes it would be welcome).

I don't know if it's possible to change this, or if there might be a really good reason why we would want to keep it the way it is (forcing us to reapply the "Generic" base class).  If not, I could at least try and look at what is possible.
If it doesn't work, perhaps we can add a nicer error message

This looks relevant:
https://github.com/samuelcolvin/pydantic/blob/619ff261c9bdc2a612ef2fcfa17a005dfefbcf21/pydantic/generics.py#L39
I assume it refers to this line:
https://github.com/samuelcolvin/pydantic/blob/master/pydantic/generics.py#L84
I guess if Generic[T] (with the according type vars) were added as a base to the subsequent create_model call whenever there are such typevars still present in the params of the class_getitem, this should work. I think create_model does not accept tuples in its __base__ parameter though, so that would seem to be the root cause (related old issue: https://github.com/samuelcolvin/pydantic/issues/2029).
So to me it looks like the solution is to either let create_model accept a tuple of BaseModel types, or to change that create_model call to somehow create the new type by just using a type(...) call (which seems tricky to get right).

GenericModel is removed from Pydantic on V2.

In v2, even though we don't have GenericModel and you just use BaseModel, it's still the case that you need to add Generic[<parameters>] to any subclasses (as noted here). We are interested to remove the need for this, but when I last tried I couldn't get it to work after a moderate amount of effort.
I believe (not 100% sure though) there is already some issue already for tracking this, so I don't think you need to worry too much about this being forgotten; more likely, the fact that it can be worked around by just re-inheriting from Generic means that it won't be a high enough priority to get fixed given all the other priorities.
A PR fixing this in v2 would definitely be welcome, but I'll just give the warning that this code is pretty nasty and complex, so it may be hard to get it to work without breaking some of the more complex examples.
inheriting from Generic with all parameters


Barcode Scanning UI Update

This is complete when: https://its-pro.ucsd.edu/browse/MA-74

[x] The camera view does not display when the SDK is initializing
[x] The camera view appears only when the SDK has initialized

Deployed on 2020-10-08


Progress bar colour not sticking

Load Progress Bar 0.9
Firefox 62
Linux Mint 19 x64 Cinnamon
If I change the setting for bar colour, and then close the addOn window, then, when I go back to the preference window, the colour has reverted to what it was previously.

Do you click the "Save" button below?

Oh. No, I had not. That button was off the bottom of the screen . . Thanks.


merge flow-path into Flow

Path merged into Flow
Any object can be used as a key; a Path is a hierarchical key.
Services can be configured for any key via a supplied factory.
This makes Path essentially free to use.
All Context creation is internalized and managed by Flow.
Setup/teardown is no longer the app author's concern.
Each key can now have an arbitrary state Bundle associated with it, in
addition to ViewState.
Keys are reference-counted and retained until all uses are gone.
This enables apps to have multiple "live" keys, including Path keys
with shared hierarchy prefixes. You can finally have a tree of scopes
live in the app.
Key services are set up on demand, and torn down only when the
traversal queue is completely flushed.
This enables safe redirects without thrashing on scope setup/teardown.
Introduced KeyChanger interface and KeyDispatcher which uses it.
KeyChanger is a simplified Dispatcher interface, intended for use by
Dispatchers which only care about the keys at the top of the incoming
and outgoing histories (most of them)

still TODO: samples, tests, docs

updated for copyright headers

updated to fix static imports of checkNotNull

updated for feedback. @rjrjr PTAL.

I don't have the big picture yet. Fine looking trees, unclear about the forest.
:shipit:

I'll definitely do a PR to review the forest when all the trees are more or less in place.


Add EXIF metadata

See https://github.com/rwcarlsen/goexif
Looks a little bit early, but promising.

Add as metadata (params?) to the image resource for JPEG and TIF.
Probably also add it to the imageConfig template func?

Add as metadata (params?) to the image resource for JPEG and TIF.

I would expected the name params in a context where the user can set and retrieve custom data. For Exif data I would call the meta or metadata instead.
But this are just my two cents :)

But this are just my two cents :)

But you are right, I didn't think. It needs its own header. This will require some thinking, I think.

Metadata may be confusing with the existing implementation of .Resources metadata.

.. or just call it what it is...EXIF.

So, to show you how random my priorities are. I'm going to implement this as an "experimental feature" (I will add some flag to turn it on). Not because I'm going to remove it, but because of the library I'm going to use is a little bit alpha.
There are possible additions to this (makernote data etc.), but I will start with EXIF.
The library has a JSON marshaler, but it puts everything in slices, which I guess is correct (in the sense that every field can have more than one value). Given a fairly representative image (se below), I think that will not be very user friendly for the average Hugo user. I think we rather should add a ´isSlice` func and add all EXIF fields with 1 element as a e single int etc. into a map.
Input on the datastrcuture would be cool. I suggest adding a map to Resource, say Data (which matches nicely with Page.Data).
So: Params is custom data from the user, Data is custom data from Hugo.
This way you can safely do with $myResource.Data.EXIF and it would not crash for non-image resources. Maybe.
type EXIF struct {
Lat float64
Long float64
Date time.Date
Values map[string]interface{} // See below

/cc @digitalcraftsman @kaushalmodi @regisphilibert @onedrawingperday and gang. Input and suggestions welcome.
GPSSatelites => "06" Format: string
GPSDateStamp => "2018:04:11" Format: string
FocalPlaneResolutionUnit => 3 Format: int
ExposureMode => 0 Format: int
Contrast => 2 Format: int
GPSLongitudeRef => "E" Format: string
GPSAltitude => "70/100" Format: rational
GPSTimeStamp => ["12/1","12/1","26000/1000"] Format: rational
ShutterSpeedValue => "8965784/1000000" Format: rational
Make => "RICOH IMAGING COMPANY, LTD." Format: string
GPSLongitude => ["6/1","446111/10000","0/1"] Format: rational
DateTimeOriginal => "2018:04:11 14:12:26" Format: string
FocalLength => "1600/100" Format: rational
GPSLatitudeRef => "N" Format: string
GPSSpeedRef => "K" Format: string
GPSMapDatum => "WGS-84" Format: string
Software => "Adobe Photoshop Lightroom Classic 7.3 (Macintosh)" Format: string
ExifIFDPointer => 292 Format: int
FNumber => "9/1" Format: rational
DateTimeDigitized => "2018:04:11 14:12:26" Format: string
Sharpness => 2 Format: int
SubjectDistanceRange => 2 Format: int
GPSProcessingMethod => "ASCIIGPS" Format: undefined
XResolution => "140/1" Format: rational
ISOSpeedRatings => 100 Format: int
FocalPlaneYResolution => "83927223/32768" Format: rational
SensingMethod => 2 Format: int
GPSTrack => "16169/100" Format: rational
ApertureValue => "633985/100000" Format: rational
Flash => 16 Format: int
GPSStatus => "A" Format: string
FocalPlaneXResolution => "83927223/32768" Format: rational
SceneCaptureType => 0 Format: int
Saturation => 0 Format: int
GPSTrackRef => "T" Format: string
Model => "PENTAX K-3 II" Format: string
GPSInfoIFDPointer => 850 Format: int
GPSSpeed => "1/100" Format: rational
DateTime => "2018:05:02 16:09:05" Format: string
ExposureTime => "1/500" Format: rational
LensModel => "smc PENTAX-DA* 16-50mm F2.8 ED AL [IF] SDM" Format: string
GPSImgDirectionRef => "T" Format: string
ResolutionUnit => 2 Format: int
Artist =><EMAIL_ADDRESS>Format: string
ThumbJPEGInterchangeFormatLength => 11537 Format: int
ExifVersion => "0230" Format: undefined
ColorSpace => 1 Format: int
GPSAltitudeRef => 0 Format: int
YResolution => "140/1" Format: rational
MeteringMode => 5 Format: int
WhiteBalance => 0 Format: int
FocalLengthIn35mmFilm => 24 Format: int
GPSVersionID => [2,3,0,0] Format: int
GPSLatitude => ["61/1","247253/10000","0/1"] Format: rational
ThumbJPEGInterchangeFormat => 1326 Format: int
ExposureProgram => 3 Format: int
ExposureBiasValue => "0/10" Format: rational
CustomRendered => 0 Format: int
GPSImgDirection => "27339/100" Format: rational
GPSMeasureMode => "3" Format: string

Taken:  2018-04-11 14:12:26 +0200 CEST
lat, long:  61.41208833333334 ,  6.743518333333333

Not sure what to do with the rational numbers. In the above, they are math/big.Rat which is, of course, the most precise value. That Rat has methods to convert it to a float etc., but it may be hard to use from the templates. But there will be people needing this kind of precision, so ...

That is really nice!

I think that will not be very user friendly for the average Hugo user. I think we rather should add a isSlice func and add all EXIF fields with 1 element as a single int etc. into a map.

Yep makes sense.

Not sure what to do with the rational numbers. In the above, they are math/big.Rat which is, of course, the most precise value.

I think people will expect raw values. I'm sure users will find ways to format it to their likings. If they cannot, maybe some template func could be introduced later on.
This will make some nice "photography" themes...

Hi, I'm afraid that I don't agree about the raw values. Most photo's and photo-using sites are not likely to want such horrible numbers to deal with. I am not even sure they may any sense at all in fact given that the difference in visual data would be infinitesimal.
F-stop differences below 0.1 are very unlikely to be useful. Shutter-speed differences below 1 thousandth of a second (maybe 1/10,000) would only be useful in limited scientific settings. Generally, cameras will use well-recognised settings since F-stop/shutter-speed interactions are well understood.
It would be worth noting that one of the leading experts on these things is Phil Harvey who has spent years developing the definitive exiftool. So worth looking at the outputs that produces.

@TotallyInformation My example above is not fantastic, but I agree that for a presentation I would only want to output f2.8 it that was the aperture.  I don't need gazillions of decimals and precision. That said, I have not thought too hard about what people want to use this feature for. Maybe output some JSON file as an input to some image processing library and that library really need the accuracy... I will have a look at exiftool.

No worries, I just know how easy it is for us techies to get carried away :)
I can see that a portfolio site would find this useful as would some sites that publish photos of the natural world (I have a friend who is an insect specialist for example). In those cases, telling people something about the settings that produced the photo can help in understanding.
For me personally, the only EXIF data I would want to make use of would be date and comment. I actually use IPTC tags a lot more as these provide a much richer set of metadata.

Yea, well:

I guess float values makes most sense for most people.
For me the EXIF value is focal length, aperture and shutter speed. For me and the common blogger I guess the GPS position and date is also useful.

So there is shutter speed value and ExposureTime...
I would say that 1/500 reads better than 0.0015625

I am no photographer, but I am used to see Exposure Time and Shutter Speed Value displayed as fraction on my camera. As this how we set them, it would make sense to read them this way too, in my opinion.

I have tested a little bit more, and concluded that: If the numerator is 1 (as in 1/500), I will use the rational number, else convert it to a float. Quckly looking at the exiftool, I'm not too far off. But remember that that is a specialist tool that the author has spent years fine-tuning ...
The "ExposureTime vs ShutterSpeed" ... there are some complex explanations online. The conclusion is: I'm using the ExposureTime.

: If the numerator is 1 (as in 1/500), I will use the rational numbe

Spot on, this is what we you're used to see in our camera settings.

Coincidentally this came up in the forum: https://discourse.gohugo.io/t/how-to-use-image-orientation-with-image-processing/11823.
Looks like none of the EXIF variables listed above store the Orientation info? Or may be it's just the test image you happened to pick?

After some more digging, it looks like the Orientation flag will show up automatically if present?
https://github.com/rwcarlsen/goexif/blob/d4c99c2ae043cae162d53024b1dc50c3a189a27c/exif/fields.go#L16

Nice to see things progressing on the EXIF front! I have to say though that selling that DA AL 35/2.4 seems like a terrible mistake to me ;-) cheap but lovely lens that is as long as you are wearing ear protection.
Sticking to fractions and generally following photographic convention must the by far the most useful way forward? In terms of useful tags the following matter to me. The technical photo ones not so much.

0x013b Artist
0x010e ImageDescription
GPS*
subject

The last one I gather might not me exif but xmp?! Being able to grab the gps data for mapping is a great feature.

seems like a terrible mistake to me ;-) cheap but lovely lens that is as long as you are wearing ear protection.

I also have the 35/2.8 macro -- which is probably my sharpest lens, and just as portable.

Would really love to see this added.

Would really love to see this added.

Yea, I have an implementation floating around. But I got hindered by a bug in a third-party library (which I fixed, but I wait for a merge of a PR). I will eventually get back to it.

FYI, seems like there's a (possibly?) less "alpha" exif tool for go than what you started with:
https://github.com/dsoprea/go-exif

I've recently started building a multilingual photography theme with hugo and came across this thread. Being able to use this EXIF data would be super helpful and reduce the need for an author to manually populate the data. Looks like some great progress has been made. I'd love to see this fully implemented.
Some of the fields I'd personally find helpful (with expected output format) as I've been looking at this theme, most of which you've already mentioned:

Date created / taken time
Camera Make (FUJIFILM)
Camera Model (FinePixS2PRO)
Focal length (97.0mm)
Aperture (ƒ/4.8)
Shutter Speed (1/350s)
ISO (200)
GPS Location (Latitude/Longitude)
Dimensions (3036 × 2030)


feature(action): Adds @action decorator

Adds an @action decorator, which works by creating a passthrough function on the actions hash of the decorated class.

re: Optional name param - Do we have a compelling use case for that? I'm always hesitant to add to the public API when all it does is add more ways to do the same thing 😛 but it's not entirely out there, I could see people wanting it for one reason or the other.

re: Optional name param - Do we have a compelling use case for that?

Nope! I agree, we should only consider adding it if/when we have a compelling use case...


Update portuguese translation

References:

https://github.com/hexojs/site/commit/99475ef34f4799e63cb5dab873753c18f8aac47f#diff-eff7591e393c65c451c299bb90cb91c4
https://github.com/hexojs/site/commit/19fdeadc22d6f1e064aa80523d7d0f87511e303b#diff-eff7591e393c65c451c299bb90cb91c4
https://github.com/hexojs/site/commit/7306f40b6a902ec010605ac6e340168a502e000f#diff-eff7591e393c65c451c299bb90cb91c4
https://github.com/hexojs/site/commit/beeeeaabb7e787d77e42420e3ef6bea56f7af0ac#diff-eff7591e393c65c451c299bb90cb91c4

Thank you so much for your contribution.


CIM Add ability to create Customer Profile with OpaqueData from Accept.js

I made an example app that showcases the accept.js and this addition that you can check here:
https://github.com/alberto1el/omnipay-example

Yes, I think this is the problem. I've made a fix and will merge it into master.

Try pulling in the master branch now. Should be fixed. I assumed it would be affecting other people running tests, so got the fix (hopefully :crossed_fingers:) in quickly. Merry Christmas :-)

@judgej Yes it was that, great catch thank you very much!
I am just missing a couple tests I will add them later on today, I will since I do want this to be merged as soon as possible :)

Just a heads up though - the CIM Abstract Request class extends the AIM Abstract Request, and that already supports some of the Opaque Data methods you have added (if I'm reading them correctly). It may be worth checking if duplicate and redundant methods have been been added here.

@judgej Thanks.
Yes, the methods were being duplicated so I removed them, I actually had copied them from that commit but didn't realize that the CIMAbstractRequest was extending it.
I added the tests and added an example to the README file I hope it is clear enough.

Looks great, thank you.
I'm going to try to merge in a few more PRs, but give the master branch a try in the meantime. If all is working for you, we can release this as version 3.1.0

I tested the master branch with the merged PR and it's working great :)
And yes I have been using this driver for years now, I believe 2019 will be a good year for cleaning it up, I read the really long issue were you talk about the accept.js library and the 'changes' on the Autorize.net API were that guy from authorize.net also comments, and I was wondering, which is the 'roadmap' for this library? , since I saw you worked on a couple repositories (probably to be able to use omnipay 3 with authorize.net which is now possible), will you continue with those or do you want to clean this one?.
Thank you very much for your work @judgej and may you have a very happy new year!


Redirect the HTTPS overview

What's changed, or what was fixed?

See https://github.com/GoogleChrome/web.dev/issues/2530

Fixes: https://github.com/GoogleChrome/web.dev/issues/2530
Target Live Date: Q2 2020

[x] This has been reviewed and approved by @kaycebasques
[x] I have run npm test locally and all tests pass.
[x] I have added the appropriate type-something label.
[ ] I've staged the site and manually verified that my content displays correctly.

:+1:

:+1:

Whoops!
There were 1 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/fundamentals/security/encrypt-in-transit/why-https.md - Unable to read file, was it deleted?

Whoops!
There were 1 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/fundamentals/security/encrypt-in-transit/why-https.md - Unable to read file, was it deleted?

Thanks @petele


EBD versions

The EBD is released monthly and it's valuable to keep track of which version of the data a user has. An easy approach would be to parse the version from the tar filename then include it as part of the parquet/ directory name. One other thought would be to have a small metadata file in the parquet/ directory storing the name of the file that generated that parquet file and the date that the data were imported. Do you have any thoughts on a better approach?
This also brings up the question of multiple data versions. Do we want to remove an old version of the EBD when a new one is imported? Keep multiple and allow users to specify the version they want somehow in ebird()? Always load the most recent version with ebird()?
Almost all users will only ever want a single version of the EBD at a time, but some may periodically download a newer version to get the most recent data.

Yup, that's a good point. Like you say, a user will probably not want to start accumulating multiple versions, but definitely ought to be able to easily look up which version they were using.  Definitely support the idea of just writing a small metadata file to BIRDDB_HOME with the release version, and ideally the sha256 hash of the downloaded tarball (ideally we'd check hashes of downloaded objects against known values to ensure integrity, but at least recording this value is best practice since it gives a verifiable way to confirm the file is the one you think it is).  It may also be wise to record the date the metadata entry was generated, and perhaps the filesize of the tarball.  Perhaps adding a friendly ebord_citation() function would be the way a user could see a formatted citation, including the version currently in use.
When a user downloads a new version, I think the default behavior should probably be to overwrite the old version, though with a utils::AskYesNo() confirmation before overwriting if R is in interactive() mode.  I imagine adding the new version would create a new entry in the metadata file, so a user could dig back if they like and see when they had the old version, but it would not keep the old version around.
Note that a user intent on having two versions on the same machine could manage this already by explicitly setting BIRDDB_HOME to separate paths for each version.  We can document that, but it's probably an edge case.  I imagine most users won't ever set BIRDDB_HOME, though it's also useful if you're on a multi-user server and you don't want each user downloading and storing their own copy of eBird.


Resource cloudflare_api_token fails due to schema issues with permission_groups.id

Confirmation

[X] This is a bug with an existing resource and is not a feature request or enhancement. Feature requests should be submitted with Cloudflare Support or your account team.
[X] I have searched the issue tracker and my issue isn't already found.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.8.5
on windows_amd64

provider registry.terraform.io/cloudflare/cloudflare v5.0.0-alpha1
provider registry.terraform.io/hashicorp/local v2.5.2

Affected resource(s)
cloudflare_api_token
Terraform configuration files
resource "cloudflare_api_token" "api_token" {
  name = "api_token"
  policies = [{
    effect            = "allow"
    permission_groups = [
      {
        id = "dfe525ec7b07472c827d8d009178b2ac"
      }
    ]
    resources = {
      resource = "com.cloudflare.api.user.${var.jr_user_id}"
      scope    = "*"
    }
  }]
}

Link to debug output
https://gist.github.com/jrogers-dlp/a2e4e2d6dabcc623a770ee36c597cf73
Panic output
No response
Expected output
Expected id to be set for permission group and terraform plan shown like so:
Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # cloudflare_api_token.api_token will be created
  + resource "cloudflare_api_token" "api_token" {
      + condition    = (known after apply)
      + expires_on   = (known after apply)
      + id           = (known after apply)
      + issued_on    = (known after apply)
      + last_used_on = (known after apply)
      + modified_on  = (known after apply)
      + name         = "api_token"
      + not_before   = (known after apply)
      + policies     = [
          + {
              + effect            = "allow"
              + id                = (known after apply)
              + permission_groups = [
                  + {
                      + id   = "dfe525ec7b07472c827d8d009178b2ac"
                      + meta = (known after apply)
                      + name = (known after apply)
                    },
                ]
              + resources         = {
                  + resource = "com.cloudflare.api.user.f4fdf41e55edfb26ba7e63c86053c9f9"
                  + scope    = "*"
                }
            },
        ]
      + status       = (known after apply)
      + value        = (known after apply)
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Actual output
Terraform error on attempting to set a read-only attribute.
╷
│ Error: Invalid Configuration for Read-Only Attribute
│
│   with cloudflare_api_token.api_token,
│   on main.tf line 60, in resource "cloudflare_api_token" "api_token":
│   60: resource "cloudflare_api_token" "api_token" {
│
│ Cannot set value for this attribute as the provider has marked it as read-only. Remove the configuration line setting the value.
│
│ Refer to the provider documentation or contact the provider developers for additional information about configurable and read-only attributes that are supported.

Steps to reproduce

terraform init with cloudflare provider version 5.0.0-alpha1
Create a resource with cloudflare_api_token that creates an api token
Run terraform apply

Additional factoids
Feels like with the current way that it is written, there is no valid resource configuration for cloudflare_api_token, as policies.permission_groups.id is listed as computed in the schema so it cannot be set.
I also tried modifying internal/services/api_token/resource.go and internal/services/api_token/model.go to change id from computed to required and rebuilt, and the resource does then succeed to at least the plan step. However, I'm currently having issues with the api not accepting anything I give the resources attribute so I havent gotten the resource to fully create yet, but that part is potentially just my own ignorance.
References
No response

thanks but we're not accepting bugs for the alpha release at the moment. these are known issues and will be resolved in later releases.