feat: frontmatter property to exclude notes from explorer

Is your feature request related to a problem? Please describe.
This request has its roots in #932 which was closed by #938 but I propose a cleaner solution for the underlying problem.
The underlying problem is the need to exclude specific notes from the explorer, without touching the explorer config. This is a common use case for everyone hosting their website in the EU (erquired to have a privacy policy), especially in Germany (required to have an Imprint page) and I am sure that there are many more cases where people would want to exclude notes from the explorer.
Describe the solution you'd like
I'd like to discuss the introduction of a custom boolean frontmatter in resemblance of enableToc (e.g. disableExplorer or excludeFromExplorer), which should default to false.
Describe alternatives you've considered aka Why not using the existing methods?
Using the filename requires the user to edit the config file for each page that is to be exclude. Using tags comes with two major disadvantages: 1. The tag is visible prominently on the excluded pages by default and you might not want to start you privacy policy with #excluded  2. A tag listing page is automatically created which might be undesired as well. Using directories is a clean solution, however, not as flexible as a custom frontmatter property.
Additional context
I think the implementation is fairly straight-forward but it has to be decided if such a filter should be part of the default filterFn or if it should be applied outside so that it persists a custom filterFn.
If this proposal is to your liking, I can prepare a PR soon™.

I found having the example that you wrote should be good enough.

This should be addressed by https://github.com/jackyzha0/quartz/pull/495


Make it possible to disable the IP reflection

Description
This pr allows the Liqo users to disable the IP reflection. The local copy of the offloaded pods will not have an assigned IP, and the endpoint slice reflection is disabled by default.
Ref #1671
How Has This Been Tested?

[x] locally on KinD with the reflection enabled
[x] locally on KinD with the reflection disabled

/build

/test

/rebase test=true

/merge


The Figure and the Description of the Gauge charts section don't match

[Enter feedback here]
Hi,
The Figure and the Description of the Gauge charts section don't match.
https://user-images.githubusercontent.com/110555152/182658019-26b859d3-9954-4680-9653-0cd0fe6944d4.png
https://docs.microsoft.com/en-us/power-bi/visuals/power-bi-visualization-types-for-reports-and-q-and-a#gauge-charts
Likely the wrong Figure is embedded in the webpage, because everything looks good if you forward to the external click (For more information, see Gauge Charts in Power BI) at the bottom of the Gauge charts section.

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

ID: 69212c7b-4492-a044-a07b-05266b20e370
Version Independent ID: cc3d001b-1d58-b3b5-62a6-95f3955bf733
Content: Visualization types in Power BI - Power BI
Content Source: powerbi-docs/visuals/power-bi-visualization-types-for-reports-and-q-and-a.md
Service: powerbi
Sub-service: pbi-visuals
GitHub Login: @mihart
Microsoft Alias: mihart

Thanks for your comment, @JennaW666. This issue has been assigned to the doc author for review.

Hi @JennaW666 @bishalgoswami ,
Thank you for noticing this and bringing it to our attention.  The documentation has been updated and set to republish tomorrow.


QuickActionBar/Grid/Widget wrong X location on Landscape

Tested on a Tablet 7", if you rotate to landscape, and have a item on the right side of the screen, the QuickAction popup loads in the wrong location (not on the right, but in the center).

In landscape, the problem continues with a ten inch screen tablet.


feat(remote): add new component and utils

Owner

[x] Nickname(Tuya): 庄子
[x] Intra-group review（组内审阅人员）：风眠

Description
add new component and utils
Package
Please select the relevant package.

[ ] tuya-panel-animation-sdk
[ ] tuya-panel-api
[ ] tuya-panel-electrician-sdk
[ ] tuya-panel-ipc-sdk
[ ] tuya-panel-lamp-sdk
[ ] tuya-panel-lock-sdk
[ ] tuya-panel-robot-sdk
[ ] tuya-panel-outdoor-sdk
[x] tuya-panel-remote-sdk

Type of change
Please select the relevant option.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Self CheckList
Please check all options.

[x] Intra-group review
[x] ESLint lint
[x] Example Run
[x] Jest Run
[x] No Grammer Error
[x] No Spell Error
[x] No Merge Conflict

Codecov Report

Merging #184 (1ad4540) into main (bb65f69) will decrease coverage by 0.22%.
The diff coverage is 39.06%.

@@            Coverage Diff             @@
##             main     #184      +/-   ##
==========================================
- Coverage   62.00%   61.78%   -0.23%
==========================================
  Files         342      346       +4
  Lines       12829    12957     +128
  Branches     2900     2942      +42
==========================================
+ Hits         7955     8005      +50
- Misses       4854     4932      +78
  Partials       20       20

Impacted Files
Coverage Δ

packages/tuya-panel-remote-sdk/src/res/index.ts
100.00% <ø> (ø)

...s/tuya-panel-remote-sdk/src/utils/getAirKeyData.ts
15.58% <15.58%> (ø)

packages/tuya-panel-remote-sdk/src/utils/index.ts
68.18% <68.18%> (ø)

...l-remote-sdk/src/components/search-input/index.tsx
78.57% <78.57%> (ø)

...mote-sdk/src/components/search-input/i18n/index.ts
100.00% <100.00%> (ø)

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 bb65f69...1ad4540. Read the comment docs.


Default selection of spectra

When we drag and drop a jcamp containing FT and FID we display the FT by default. This is perfect. We should also however select the first spectrum after FT by default. Currently nothing is selected.
1H_Cytisin.dx.zip

@lpatiny
because we agreed in previous ticket that if we in 1d mode and have more than on spectrum not to select any one.
https://github.com/cheminfo/nmr-displayer/issues/407#issue-609663177

You are right indeed. But if there is only once spectrum after FT it should still be selected ;)

It should be the one after FT and not before ft. Currently you display the FID (before FT)


Testbed folders

Why are these changes needed?
The current testbed templating format works great for single-file scenarios, but is less flexible when multiple files need to be included in a test (e.g., including a PDF or image to operate over). This PR moves the templating format to one that accepts whole folders. Backwards compatibility is maintained.
Related issue number
This PR will enable progress on #691, #692, and other benchmarks (e.g., newly released GAIA)
Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

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

Comparison is base (b30ead2) 27.61% compared to head (bf815aa) 27.61%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #792   +/-   ##
=======================================
  Coverage   27.61%   27.61%
=======================================
  Files          27       27
  Lines        3480     3480
  Branches      789      789
=======================================
  Hits          961      961
  Misses       2447     2447
  Partials       72       72

Flag
Coverage Δ

unittests
27.61% <ø> (ø)

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

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

I was looking at this testbed and has a question about Docker. Currently the run_scenario will pull the python:3.11 docker image and do pip install every time. Is it possible to pass in names of a customized docker images to run (or am I overlooking)? I think it is even better if users could just pass in a docker_build.txt with the required python packages to create a new docker image if the intention is to run a large scale experiment.

Yes, adding support for a custom Docker image, or a Docker file would be a logical next step. At present, there are a couple of options. (1) you can update or specify a requirements.txt file. Or (2) you can customize the global_startup.sh or scenario_startup.sh files to install by other means. The new folder-oriented specification means that you can include zips, packages, or other dependencies in the includes or scenario template folders as well, allowing for testing on private content.


Class transform - Remove React import only if it should be removed

When replacing React.createClass with  createReactClass, the React import was being erroneously deleted even though JSX existed in the file.
This change ensures that the React import is not erroneously removed. Added a corresponding test case.

Thanks for all of the recent fixes, @jsnmoon 😄

Glad to be of service! 🙂


vmlatency: Randomize VMI names

Currently, source and target VMI names are hard-codded. This prevents simultaneous execution of multiple checkup instances in the same namespace.
VMIs names will have a random string prefix:
latency-check-source-<random string>
latency-check-target-<random string>
Partially solves issue #164.
Signed-off-by: Orel Misan<EMAIL_ADDRESS>

Change: Moved nameGeneratorStub to checkup_test.go and launcher_test.go.

Change: Removed nameGenerator dependency from checkup.


Only change to master to keep down on the chaff.

DS_Store

This should only appear on my repo.


Add "Settings > Reverse Proxy" section

Issues are for bug reports only, please go to the "Discussions" tab to submit a Feature Request

[X] I understand

Hey, I've been trying to connect to dockge via Reverse Proxy Traefik + Authentication via Authentik.
This works great for Uptime Kuma, since I can set Settings > Reverse Proxy> HTTP Headers > Trust Proxy = true (Trust 'X-Forwarded-)
How ever this Settings section is missing for Dockge, and I can't establish a connection to Dockge as soon as I enable Authentik as a middleware.

updated


How to get the shipmentId?

How to get the ShipmentId displayed in the upper right corner of the shipment label through the API？
SHP#:  11 alphabets and numbers

Hi, please reference the UPS Developer Portal's Ship API documentation response section for the field "ShipmentResponse.ShipmentResults.ShipmentIdentificationNumber"


The version of react-components is not updated when changing Hilla version

Description of the bug
Hilla should define the version of react-components that it is compatible with. Currently the user has to manage both react-components and @vaadin/bundles manually. This mostly leads to warning messages about the bundle when upgrading Hilla, or just to the application using old react-components
Expected behavior
The versions should be upgraded with the Hilla version
Versions:
- Vaadin / Hilla version: 2.2 beta1

Fixed by https://github.com/vaadin/flow/pull/17572


Bug: TypeError: Linearization of inheritance graph impossible as of v1.10.0

🎉 Description

[X ] 🐛 This is a bug report.

💻 Environment
OZ version: v1.10.0 and v1.11.0
Network: Ganache
Tooling: Truffle
📝 Details
TypeError: Linearization of inheritance graph impossible as of v1.10.0
The contracts: WhitelistedCrowdsale & FinalizableCrowdsale with Ownable being the cause.
🔢 Code To Reproduce Issue [ Good To Have ]
CustomCrowdsale is CappedCrowdsale, MintedCrowdsale, WhitelistedCrowdsale, FinalizableCrowdsale

👍 Other Information
This was working okay in v1.9.0. WhitelistedCrowdsale was update in v1.10.0 which introduced this 🐛.

Thanks for reporting @mswezey23! I was able to reproduce this in 1.11.0, but not in 1.10.0. My understanding is that the bug was introduced in #981, which was only included in 1.11.0.
I think the problem is caused by this line:
https://github.com/OpenZeppelin/openzeppelin-solidity/blob/73be06412fdeb0453c11fa708d4e211947569ec3/contracts/crowdsale/distribution/FinalizableCrowdsale.sol#L13
Swapping Ownable and TimedCrowdsale makes linearization possible.
It should've been like that from the beginning because of the rule that "most base-like" goes to the left of the inheritance list.

Ah, that could have been my mix up. Yarn and I were having some issues & we had to duke it out. Apparently manually updating the versions in the package.json file to downgrade the OZ version doesn't work. It keeps the latest in the yarn.lock file. Had to run the commands yarn remove ... and yarn add ... for a successful downgraded version... back to topic now.
Thanks, @frangio, for the additional insight and confirming the bug!
Glad the fix was easy as swapping the order 😎


add docs tests into sub packages

The docs are owned and being generated by this repository. However, the sub-packages that are being documented should ideally not admit changes that break the documentation, so it would be nice to have tests in the sub-packages that ensure this.

addressed in #62


expose API as an interface

close #42

I started this PR before checking any pending PRs - sorry about that! There is already a PR addressing the same issue!

This approach does not make use of the KMM expect/actual feature. It is just a simple interface/implementation approach.
Also this PR is a breaking API change - there is no downgrade compatibility. The adjusted README.md highlights the change.
As the tests are green I have a good feeling. But I did not tested the library in a real Android app. I wish there was an Android example app (similar to the iostests app). But I had no luck to add this android-example-app. I don't know how to reference the library source code (./src) at this example app (gradle?).
So please review this approach carefully and let me know if this isn't the correct approach for multi-platform libraries. I took some recommendations from the kmm-documentation (w/o the factory class as the two kvault implementations need different parameters).

I was able to test the changed API in my real world app by installing this PR to my local maven repo via ./gradlew publishToMavenLocal and reference this version via mavenLocal().
As expected I had to change my dependency graph (we are using koin - but this doesn't really matter):
./shared/src/androidMain/package/di/PlatformModule.kt
single<KVault> { KVaultEncryptedSharedPreferences(context = get(), fileName = "any") }

./shared/src/iosMain/package/di/PlatformModule.kt
single<KVault> { KVaultKeychain("any") }

My Android and my iOS apps are running as expected. And unit tests can run w/ a mock of the interface KVault.

@cleemansen thanks for your efforts, I favour the variant with actual / expect, so I will look at #47 more deeply tomorrow. Thanks for your contribution!


New updates to context-match

New features added

custom context file
string separations
discounting property values for missing properties in q_nodes.

@HardiRathod your unit tests should be more extensive. Rather than checking for non zero values, you should check for exact values. Unit tests should have  a test for each of date, string, quantity etc and then a combination of these, type of context matches.
The tests should check for exact scores. Please add more precise testing


fix: UserModel::assignIdentities() always produces a DB query

In the UserModel - when we use withIdentities() method the query is produced properly.
The problem occurs in the assignIdentities() method, when we're trying to assign identities to the users.
We're working with the User entity, and there is a line:
$newIdentities   = $mappedUsers[$userId]->identities;

It will call getIdentities() method which will produce a query to the DB, since there are no identities assigned yet.
This PR solves this problem.

Am I correct in assuming that this PR prevents queries to retrieve unnecessary Identities from being executed?
It's just useless and there has never been a problem with the results returned.

The code looks good.

Am I correct in assuming that this PR prevents queries to retrieve unnecessary Identities from being executed?
It's just useless and there has never been a problem with the results returned.

There was never a problem with the correct result per se. The problem here was that we were producing n+1 queries for code like this:
$users = model(UserModel::class)->withIdentities()->findAll();
foreach ($users as $user) {
    $user->email;
}

That would be just fine, but since we use withIdentities() it should not work that way.

Thanks All!


Missing intellisense for secrets property

Describe the bug
Not sure if bug or feature request. Apologies if this is mis-categorized.
The documentation here details the syntax for passing secrets to composite workflows. This extension seems to be missing the intellisense for the feature.
To Reproduce
Steps to reproduce the behavior:

Create any workflow that consumes a composite workflow
Add secrets: inherit
The error will appear

Expected behavior
The schema for secrets should be implemented.
Screenshots

Extension Version
Name: GitHub Actions
Id: github.vscode-github-actions
Description: GitHub Actions workflows and runs for github.com hosted repositories in VS Code
Version: 0.25.3
Publisher: GitHub
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=GitHub.vscode-github-actions

Additional context
Add any other context about the problem here.

cc @clind-seismic

btw, I'm happy to do this myself if someone can point me to the json schema where this is defined

my bad. I was trying to use secrets in a composite action, instead this property only exists for reusable workflows. confusing 😅


Possible improvement for isEven

I have no idea what Haskell even is, but this seems like it would save lots of lines of code, since you already have isOdd defined, just rely on it and negate the result.
Trust me.
-- | Returns True if the number is even.
isEven :: (Integral a) => a -> Bool
isEven = not . isOdd
{-# INLINE isEven #-}```

genius
{-# LANGUAGE NoMonomorphismRestriction #-}
isEven, isOdd :: (Integral a) => a -> Bool
(isEven, isOdd) = (not . isOdd, not . isEven)


Disappearing content in Chrome, A's vs DIV's

I came across an issue that seemed only affect Chrome (53). When dragging the content, it is replaced by a gray representation of what is being dragged.
At first this appeared to be related to the "will-change" branch/fix that I saw in issue #857 and #852. However, it appears it's related to using an A tag for the items being dragged.
Here's an demo of it (using the will-change branch): http://jsbin.com/suvedezuro/edit?html,css,js,output. The top boxes are the A's and when dragged in Chrome you'll see a gray box replace the dragged content. The bottom boxes work as they should with DIV's.
Maybe this isn't an issue worth pursuing because it seems so easy to overcome by just using DIV's, however, I thought I'd report it if others come across this.

Alas, just so "awry" Chrome works with links.
will-change has no relation to this problem.


Is MFX_EXTBUFF_VPP_SCENE_ANALYSIS option supported?

It was discovered in #2035 that bSceneDetectionEnable field in mfxExecuteParams is not changed anywhere (except setting to 0). So what about MFX_EXTBUFF_VPP_SCENE_ANALYSIS option?

https://github.com/Intel-Media-SDK/MediaSDK/blob/master/doc/mediasdk-man.md:

MFX_EXTBUFF_VPP_SCENE_CHANGE
Deprecated.

Do we need to delete corresponding path in mediasdk library? Specifically:
https://github.com/Intel-Media-SDK/MediaSDK/blob/1f9e67b2db7b680af142376f264a61df8d5c50f1/_studio/mfx_lib/vpp/src/mfx_vpp_hw.cpp#L6179
once #2035 will be merged this if condition will become empty.

I think so


Incompatible with OptiFine

Latest release for 1.19 and 1.19.1 doesn't work with optifine installed. Crashes immediately. Please fix.

I have the same issue in 1.19.1.

This has been fixed for 1.3.6.


kdump [!] Failed to locate kernel

iPad:~ root# kdump
[!] Failed to locate kernel

kern-utils-utils 1.4.1-1
iOS 13.4.1
iPad 9.7" Pro

Why would you even want that?

I get the error when running nvpatch com.apple.system.boot-nonce on my SE when trying to downgrade 13.6 to 13.5

The method nvpatch uses has been fixed back in iOS 11.
And kdump has been unnecessary since iOS 10.
I'm closing this.


integrations: Add webhook payloads for mention

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

@anirudhjain75 Thanks!  This looks good.  We will probably keep this PR open for a while in case you move on the subsequent tasks related to this webhook.

@showell Check that too please

Can i post the link here

Can this PR be merged, and closed, because I'm done writing the integration, and need that merged.

OK, merged, thanks @anirudhjain75!


Feature request: Send cloud event to AWS sns

It seems like we can only send the event to the Google pub/sub service. any plan to support Aws sns service as well?

There is no AWS SNS bindings spec for CloudEvents, so we are not able to support it. There has been talk in the main group but so far no one has stepped up to support maintaining that spec.

There is some work starting to look at this binding!
https://github.com/cloudevents/spec/pull/1043


Uncaught Error: The specified module could not be found./?~\AppData\Local\atom\app-1.53.0\res...

[Enter steps to reproduce:]

I tried to save a file using ctrl + s, and this popped up.

Atom: 1.53.0 x64
Electron: 6.1.12
OS: Microsoft Windows 10 Home
Thrown From: Atom Core
Stack Trace
Uncaught Error: The specified module could not be found.
\?\C:\Users\itetb\AppData\Local\atom\app-1.53.0\resources\app.asar.unpacked\node_modules\keyboard-layout\build\Release\keyboard-layout-manager.node
At electron/js2c/asar.js:138

Error: The specified module could not be found.
\\?\C:\Users\itetb\AppData\Local\atom\app-1.53.0\resources\app.asar.unpacked\node_modules\keyboard-layout\build\Release\keyboard-layout-manager.node
    at process.func (electron/js2c/asar.js:138:31)
    at process.func [as dlopen] (electron/js2c/asar.js:138:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:828:18)
    at Object.func (electron/js2c/asar.js:138:31)
    at Object.func [as .node] (electron/js2c/asar.js:147:18)
    at Module.load (internal/modules/cjs/loader.js:645:32)
    at Function.Module._load (internal/modules/cjs/loader.js:560:12)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (internal/modules/cjs/helpers.js:16:16)
    at customRequire (~/AppData/Local/atom/app-1.53.0/resources/app/static/<embedded>:1:736531)
    at get_KeyboardLayoutManager (~/AppData/Local/atom/app-1.53.0/resources/app/static/<embedded>:14:2789479)
    at get_manager (~/AppData/Local/atom/app-1.53.0/resources/app/static/<embedded>:14:2789581)
    at Object.getCurrentKeyboardLayout (~/AppData/Local/atom/app-1.53.0/resources/app/static/<embedded>:14:2789707)
    at e.keystrokeForKeyboardEvent (~/AppData/Local/atom/app-1.53.0/resources/app/static/<embedded>:14:1057734)
    at KeymapManager.keystrokeForKeyboardEvent (~/AppData/Local/atom/app-1.53.0/resources/app/static/<embedded>:11:1227949)
    at KeymapManager.handleKeyboardEvent (~/AppData/Local/atom/app-1.53.0/resources/app/static/<embedded>:11:1225934)
    at WindowEventHandler.handleDocumentKeyEvent (~/AppData/Local/atom/app-1.53.0/resources/app/static/<embedded>:11:284919)

Commands
     -0:52.6.0 tree-view:remove-project-folder (span.name.icon.icon-file-directory)
  2x -0:48.9.0 core:save (input.hidden-input)
     -0:38.8.0 tree-view:remove-project-folder (span.name.icon.icon-file-directory)
     -0:16.2.0 core:save (input.hidden-input)

Non-Core Packages

Thanks for taking the time to contribute!
We noticed that this is a duplicate of #21285. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Add config watchers

Fixes #74
I've also modified the update-deps script to pull latest eventing version from master (like in eventing-contrib)
/assign @lberk

ok i've found that there is a not so funny data race in sdk-go, next release will be out soon with the fix

Ok now it should work fine :smile:

/assign

/lgtm
/approve


Add support for building with other C++ compilers?

make CXX=<path to some gcc version>/bin/g++ third_party seems to work, but
make CXX=<path to some gcc version>/bin/g++ cc does not, i.e., the changed compiler is ignored.
Can support for this be added?
I am asking because the standard compiler on one of the machine I am using is an old gcc (5.4.0), which does not work:
./ortools/constraint_solver/local_search.cc: In member function ‘void operations_research::LocalSearchFilterManager::Revert()’: ./ortools/constraint_solver/local_search.cc:3896:10: error: expected unqualified-id before ‘[’ token      auto [filter, event_type] = filter_events_[i];

You need  c++17 mode.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le mar. 16 févr. 2021 à 17:19, pfetsch<EMAIL_ADDRESS>a écrit :

make CXX=/bin/g++ third_party seems to work, but
make CXX=/bin/g++ cc does not, i.e., the
changed compiler is ignored.
Can support for this be added?
I am asking because the standard compiler on one of the machine I am using
is an old gcc (5.4.0), which does not work:
./ortools/constraint_solver/local_search.cc: In member function ‘void
operations_research::LocalSearchFilterManager::Revert()’:
./ortools/constraint_solver/local_search.cc:3896:10: error: expected
unqualified-id before ‘[’ token auto [filter, event_type] =
filter_events_[i];
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/2397, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3PV4WZPQWGV4EH7IELS7KLH3ANCNFSM4XWXJLGA
.

and you should use the cmake build.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le mar. 16 févr. 2021 à 17:45, Laurent Perron<EMAIL_ADDRESS>a écrit :

You need  c++17 mode.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68
53 00
Le mar. 16 févr. 2021 à 17:19, pfetsch<EMAIL_ADDRESS>a
écrit :

make CXX=/bin/g++ third_party seems to work,
but
make CXX=/bin/g++ cc does not, i.e., the
changed compiler is ignored.
Can support for this be added?
I am asking because the standard compiler on one of the machine I am
using is an old gcc (5.4.0), which does not work:
./ortools/constraint_solver/local_search.cc: In member function ‘void
operations_research::LocalSearchFilterManager::Revert()’:
./ortools/constraint_solver/local_search.cc:3896:10: error: expected
unqualified-id before ‘[’ token auto [filter, event_type] =
filter_events_[i];
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/2397, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3PV4WZPQWGV4EH7IELS7KLH3ANCNFSM4XWXJLGA
.

O.k.  - thanks! This should work.
I followed the documentation which currently only lists make. But you will probably change this.


Need to add RTMP-in controls

The teams PowerShell module now supports the parameter: AllowedStreamingMediaInput
it has two possible values, blank or RTMP.
if set to RTMP, users assigned that corresponding policy can use RTMP in services when they schedule meetings.

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

ID: 919bf080-1e20-1411-0d9e-41f559b883d8
Version Independent ID: 6a37be44-d957-2b7a-a563-53332cf59098
Content: Set-CsTeamsMeetingPolicy (SkypeForBusiness)
Content Source: skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md
Service: skypeforbusiness-powershell
GitHub Login: @tomkau
Microsoft Alias: tomkau

@get-itips

Thanks for your feedback!


build-angular 12 regression: Updates to node_modules properly trigger rebuilds, but vendor.js chunk is not updated by rebuild

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[x] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
Yes, the previous version in which this bug was not present was: @angular-devkit/build-angular version 0.1102.14.
Description
When running ng serve, changes to .js files in node_modules are properly detected and trigger a rebuild. However, the rebuild itself does not notice the changes and leaves everything unchanged.
This is particularly problematic when using the recommended flow to work on an angular library (using ng build --watch in the library to automatically rebuild it on change, and using npm link to link the library inside the main project's node_modules). In this particular scenario, updates to the libary's code trigger the library's build in its own ng build --watch. The build outputs data in the library's directory that the projects node_module symlinks to. The project's ng serve detects the update and recompiles. But vendor.js is not updated.
Sample output (trimmed newlines):
✔ Browser application bundle generation complete.
5 unchanged chunks
Build at: 2021-06-24T09:52:46.822Z - Hash: 10397e9fbdac4352dd17 - Time: 791ms
✔ Compiled successfully.

Same exact flow with version 0.1102.14:
✔ Browser application bundle generation complete.
Initial Chunk Files | Names  |    Size
vendor.js           | vendor | 8.34 MB
4 unchanged chunks
Build at: 2021-06-24T10:25:52.854Z - Hash: a8c2d9813f0281ecfff6 - Time: 4176ms
✔ Compiled successfully.

🔬 Minimal Reproduction
In an angular project that uses @angular-devkit/build-angular version 12 (I tried 12.0.3, 12.0.5 and 12.1.0-next.6):

Install, for instance, momentjs: npm install moment
Run ng serve.
While ng serve is running, make a meaningless edit to node_modules/moment/moment.js (like adding a whitespace at the beginning of the file). Using touch won't work, there has to be a small edit.
Observe that ng serve triggered a compile step, but did not update vendor.js.
Repeat every step with an older @angular-devkit/build-angular version (like 0.1102.14). Step 4 will now show an update to vendor.js.

🌍 Your Environment
ng version where the bug occurs:

Angular CLI: 12.0.5
Node: 14.15.5
Package Manager: npm 6.14.11
OS: linux x64

Angular: 12.0.5
... animations, cdk, cli, common, compiler, compiler-cli, core
... forms, google-maps, material, platform-browser
... platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1200.5
@angular-devkit/build-angular   12.1.0-next.6
@angular-devkit/core            12.0.5
@angular-devkit/schematics      12.0.5
@schematics/angular             12.0.5
rxjs                            6.6.3
typescript                      4.2.4

ng version where the bug does not occur (notice that this version of build-angular is not really supposed to be compatible with angular 12, but it still works, and it also works as expected within an angular 11 project):

Angular CLI: 12.0.5
Node: 14.15.5
Package Manager: npm 6.14.11
OS: linux x64

Angular: 12.0.5
... animations, cdk, cli, common, compiler, compiler-cli, core
... forms, google-maps, material, platform-browser
... platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1200.5
@angular-devkit/build-angular   0.1102.14
@angular-devkit/core            12.0.5
@angular-devkit/schematics      12.0.5
@schematics/angular             12.0.5
rxjs                            6.6.3
typescript                      4.2.4

Duplicate of https://github.com/angular/angular-cli/issues/20962

Indeed, my bad. Thank you for the quick answer.


Feat/meshmodel update

Description
This PR

Adds isNotDeployable attribute to MeshModel to control the provisioning of components.

Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Closing since we decided to move this attribute to metadata


publish: iterate over all h-entrys on a page (not just one)

Recently I started clustering adjacent lightweight interaction posts of the same type, e.g. likes, which then link to a separate page that lists them individually. E.g. http://tantek.com/2015/002/f
It would be handy to be able to use Bridgy Publish to automatically POSSE those all those likes to FB with one Bridgy Publish call/curl, rather than calling Bridgy Publish once per h-entry on that page.
If there's a worry of accidental misuse, the "publish all h-entrys on a page" feature could be constrained to:

must be all same "type" (Bridgy already has some notion of post type in its logic that uses to say what's going to happen in the Bridgy Publish UI)
must all be to same silo

thanks for filing!
see irc snippet below for my concers about the complexity. the key thing i'd want to wait on for a bit is to see what common indie like flow emerges. do most people like one or two things individually? do they cluster, then try to posse the cluster? or do they like and posse a bunch of things, then cluster afterward in presentation? etc.
discussion from irc:

snarfed
10:46 tantek: honestly i don't know. i suspect it would add a fair amount of complexity, at least to get the behavior "right" for all cases
tantek
10:46 it's not "just" a loop over the h-entrys on the page?
snarfed
10:46 …which i don't have to do, but i'd rather not have a bunch of undefined behavior at edges
10:47 it might be, i just have a feeling it will result in surprises
10:47 i'd need to think about it more
10:47 e.g. if a new user accidentally plugs in a feed permalink, and bridgy happily goes and posts 20 new tweets for them
tantek
10:48 got it
snarfed
10:48 i detect when people put in their home page, and other obviously wrong inputs, but still
tantek
10:48 yes I was thinking of more scoped use-cases like http://tantek.com/2015/002/f
snarfed
10:48 feel free to file an issue so we can discuss though!
10:48 definitely, i totally get your use case. it's just uncommon
tantek
10:50 snarfed, uncommon for now ;)
snarfed
10:50 sure!
10:50 i try not to predict the future too much :P
tantek
10:50 I see clustering of same-kind adjacent posts as natural step forward
10:50 which then implies a page for such clusterings
aaronpk
10:50 me too. I try to create the future. Easier to predict that way.
snarfed
10:50 definitely! clearly better ui. i just don't know that people would naturally want to bridgy publish those clusters instead of individually
10:51 the clustering seems more appropriate to do afterward, purely as presentation

I'm uncomfortable with Bridgy publishing an entry were the source parameter is not the permalink being published. I feel like the one input → one output (source url for syndication url) is nice guarantee/sanity-check, and it feels more in line with the way webmentions are used elsewhere.
Also this would add complexity in other places, not just for sanity checking inputs:

Preview UI would need to show multiple syndicated posts
API users that depend on the return value of the POST would need to check for a syndicated URL or a list of syndicated URLs

thanks for the input, @kylewm! agreed with the simplicity and robustness of one input -> one output, as well as the two complexity examples.

tentatively closing this, at least for now.


Start Candidate insertion searching Error

Dear developer.
I am doing a run with docker.
So I got an error in the Start candidate insertion searching process.
ERROR: Requested column 4, but database file - only has fields 1 - 0.
Maybe Use default mobile element reference:/root /DeepMEI/DeepMEI_model/reference/ME_add_ALU.fa
part of the file may be related to this?
For example, ME_chr.bed.
Best regards.

Could you please provide details about the input BAM file? For example, what species it belongs to (human or other), and which reference genome version you're using. Additionally, could you share the command you used to run DeepMEI?

Thanks for the quick response.
It is GRCh38 with the human bam file.
The commands used are as follows.

docker run -it -v /lustre/home/kouta/:/root/data xuxiaofeiscu/deepmei:latest /bin/bash  -c 'export PATH=/root/miniconda3/bin:$PATH; ./DeepMEI/DeepMEI -I /root/data/sample.bam -r 38 -w /root/data/'

Best regards

Thanks Developer.
I try that command, but get same Error message.
ERROR: Requested column 4, but database file -only has fields 1 - 0.
Best regards

If it's convenient, you could share the BAM file for this sample with me. Alternatively, you can extract the main chromosomes (chr1-22, chrX, chrY, chrM) from this BAM file into a new BAM file, and then reanalyze the newly generated file using DeepMEI. I suspect there might be some inconsistency between your reference genome and the contigs in the Docker image. Another option is to send me the intermediate files. To do this, you need to enter the recently run Docker container (docker ps -a), then run docker start container_id, followed by docker exec -it container_id /bin/bash. Navigate to /root/DeepMEI/DeepMEI_script/, and package the files candidate_pos and vsoft_pos in this directory and send them to my email at<EMAIL_ADDRESS>


adding Deep Red!

AFTER MANY MONTHS I FINALLY IMPART THE NEW THEME, DIRECTLY THROUGH neatnik/omg.lol TO YOU:
nothing b- DEEP RED!


ClientResetError.resetRealm

Return bool from ClientResetError.resetRealm to indicate if reset was initiated or not
Update CHANGELOG

Resolves: #1068

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 6 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 89.22%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
2814

Relevant Lines:
3154

💛  - Coveralls

Superceeded by #1093


Feature/add on terms

Implemented the ability for users to check terms & conditions after adding an addon #2160

@fayetality RE: The above ^^ there's a lot of white space. Can you try a single line for the add-ons and 'terms and conditions' like, "Google Drive (terms and conditions)" etc .. ? Thanks.

Continued in #2984.


Add enableFeedback param to MaterialButton

In some cases developers need to disable default tap feedback, e.g. if you want to implement custom feedback. This PR gives that opportunity

@googlebot I signed it!

Hi, this change looks good! Could you add a few tests to verify that the enableFeedback parameter is working properly for MaterialButton and RawMaterialButton? There should be some useful pointers in ink_well_test.dart.
Thank you!

How can i fix " Flutter build is currently broken" ?

That isn't an issue with your PR, that basically checks if Flutter's latest master is failing for whatever reason. This prevents new changes from being introduced if there is a potential existing issue in the codebase.

Hi, any updates on this PR? I just want to make sure we can get this one moving along since it's so close to the finish line!

Hi, any updates on this PR? I just want to make sure we can get this one moving along since it's so close to the finish line!

Fixed formatting; Seems some checks don't passed after merge master of origin repository

It seems to be failing the following test:
   info • Unused import: 'feedback_tester.dart' • packages/flutter/test/material/buttons_test.dart:13:8 • unused_import

Make sure to run flutter analyze --flutter-repo locally after your changes to make sure that all analyzer issues are properly addressed.
▌17:09:49▐ RUNNING: cd .; grep --line-number --extended-regexp [[:blank:]]$ packages/flutter/lib/src/material/button.dart packages/flutter/lib/src/material/icon_button.dart packages/flutter/lib/src/material/material_button.dart packages/flutter/test/material/icon_button_test.dart
packages/flutter/lib/src/material/material_button.dart:377:      enableFeedback: enableFeedback,
▌17:09:49▐ ELAPSED TIME: 0.025s for grep --line-number --extended-regexp [[:blank:]]$ packages/flutter/lib/src/material/button.dart packages/flutter/lib/src/material/icon_button.dart packages/flutter/lib/src/material/material_button.dart packages/flutter/test/material/icon_button_test.dart in .
Detected trailing whitespace in the file[s] listed above.

It shows '36816 issues found." if i run it locally

Is there any more detail as to what the issues are? Make sure that you've run flutter update-packages. It might be that your local copy of Flutter's dependencies are not available.

Yes, flutter update-packages helped, thank you!


Bug with named parameters in 3.3.0

Named parameters reformatted incorrectly:
% echo 'f(~commit=!build)' | esy refmt
f(~commit=~!build);

This is most likely my fault, and introduced in #1945
I’ll have a look soon; this is now highest on my list

@andreypopp fix submitted in https://github.com/facebook/reason/pull/2071


Free api for Movies/Tv shows

This API is free of cost for anyone who wants to create a movie/tv shows web application.
You can get the free API key by registering and creating an account on this website.
Link => https://developer.themoviedb.org/docs/getting-started

Link of the API => https://developer.themoviedb.org/docs/getting-started


Panic on merging contours.

[2021-08-18T20:22:19Z WARN  glifparser::glif::read] Skipped comment not marked !!MFEK!!
[2021-08-18T20:22:20Z ERROR mfek_ipc] Module MFEKmetadata is not available. MFEK is modular software; it will still run but some features will not be available. For the best experience, please install all available MFEK modules into your PATH.
[2021-08-18T20:22:20Z INFO  MFEKglif::settings] Configuration directory is "C:\\Users\\matth\\AppData\\Roaming\\MFEK\\glif"

panicked at 'index out of bounds: the len is 4 but the index is 4', C:\Users\matth\.cargo\git\checkouts\math.rlib-d2c76c88b18fa5ee\a5b3955\src\variable_width_stroking.rs:138:28

error: process didn't exit successfully: `target\debug\MFEKglif.exe` (exit code: 101)

Without a test case and with Matt having left I don't see a choice but to close this as I won't be able to reproduce it/debug it without knowing how it happened…

Confirmed.


[GHSA-j7h6-xr7g-m2c5] Code injection in Apache Struts

Updates

Affected products
References

Comments
Add some patch links related to CVE-2013-4316.

Hi @sunSUNQ, both commits appear to be consistent with the patch described in the Solution section of https://cwiki.apache.org/confluence/display/WW/S2-019. In addition to adding the commits as reference links, I changed the vulnerable version range to >= 2.0.0, < <IP_ADDRESS> because only versions on the 2.x branch are vulnerable and the previous vulnerable version range incorrectly named <IP_ADDRESS> as the patched version. Thank you for your contribution!


Reorganize directory structure

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

Comparison is base (e646696) 93.19% compared to head (8972f5b) 93.08%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
- Coverage   93.19%   93.08%   -0.11%
==========================================
  Files          24       22       -2
  Lines         720      709      -11
  Branches      133      133
==========================================
- Hits          671      660      -11
  Misses         34       34
  Partials       15       15

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


⚠ Fortnite has degraded performance

In 7ebff5d, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1547 ms

Resolved: Fortnite performance has improved in 330d02b.


Uploading not resume when i kill and restart app  in android

Scenario:
-> Uploaded a file 200+mb
-> on 16%, Internet disconnected and app killed from background
-> Once restarted app, its again uploading from 0% .
basically its not resuming in case of app kill

I'm also experiencing the same issue. Specifically, I get this crash report in Sentry when uploading using TUS and putting the app in the background
EXC_BAD_ACCESS: stream:handleEvent: > XTUM >
KERN_INVALID_ADDRESS at 0x4dad5b9db79e.

Can't seem to get much information when I search for this error. But it happens consistently when I put the app in the background. Eventually when I return, the app restarts and the file upload starts from 0%
The expected behavior is to continue uploads in the background, is this not a feature of TUS?


Document API.friends

Currently undocumented method that uses the friends/list endpoint:
https://github.com/tweepy/tweepy/blob/cc3c8e7cae73d28b9a75edb7342f89f04da99702/tweepy/api.py#L547-L557

Resolved by https://github.com/tweepy/tweepy/commit/1de28d24ec8fce97c3775c0795c97d70ca6d3a5c and https://github.com/tweepy/tweepy/commit/1a2fa720f16abeb1035b25d46ef8b904a691e1b0.


Responsiveness and Adaptability

Make your page responsive. Implement media queries in CSS to ensure the page looks good on various devices such as mobile phones, tablets, and desktop computers. Adjust the layout to be user-friendly and functional across different screen sizes.

Done correctly!


Fixed a misleading debug message.

While debugging with the -DDEBUG flag on, the log told me "FINE: Found 0 executable files" even though I have one configured.
Further investigation showed the debug log message referenced the wrong collection.

IzPack project » izpack #708 SUCCESS
This pull request looks good
(what's this?)

Is there an issue for this? Just to set it fixed.
Thank you for figuring this out.


Additions to Actor proficiencies should automatically update current owned Items which should now be flagged as proficient.

Originally in GitLab by @denisPoifol
What happened:
If my character is equiped with a leather armor and its proficiency to light armors is not yet set, when setting it the armor is still marked as "not proficient"
What I expected to happen:
Ideally setting light armor proficiency to the character would make him proficient with any light armor he owns
Additional notes:
This is also true when :

dealing with a weapon
removing proficiency and the object was marked as proeficient

System version:
Foundry v0.7.9
D&D5e v1.2.4

I think I have a working commit that just needs refactoring, JSDoc annotations, etc. - should be ready to submit the PR in a day or so!

Based on discussion in the original pass at this, we decided that rather than storing proficiency on the items, we would automatically calculate it during prepareData while adding a proficiencyMode property that allows an item to be set to "Automatic", "Always Proficient", or "Never Proficient".
This solution is currently blocked by #774

We might want a bit more flexibility depending on the final solution of #774. “Automatic, “Always”, and “Never” don’t allow the user to specify double/half proficiency, and it might be better to have proficiencyMode be a value from an enumeration, maybe something like config.dnd5e.proficiencyValues.{expertise|half|full|none|auto}. It would also give devs an easy way to modify the options and add something else (maybe like 1/3rd proficiency - nonsense in SRD RAW but maybe something a user might want to do).


Add a colormap for images that are label maps

A tentative binary map LUT to better see label images. Tell me what you think.
PS : by the way, this made us realize (me and RP) the LUT/transfer functions system in medinria is quite a bit crappy. @papadop : any chance your branch on that could be revived / finished soon ?

@LoicCadour : how does it compare to the discrete LUT feature you developed in MUSIC?

Actually I created a mode in the medClutEditor to switch between linear and discrete mode. I didn't create a color map, I just modify the current LUT according to the mode.

Well, here it's a LUT with a different color for each label.Since it is a LUT, it is adapted depending on the min max values as the other LUTs. It has advantages and disadvantages, let me know what you think of it.

In MUSIC, we needed this discrete mode to clearly visualize few (in most cases 2 or 3)  groups of pixels according to their intensity values. For instance, pixels whose intensities belong to [0;100] are displayed in red, [101;200] in green etc. And this is very easily done by placing vertices on the histogram.
I think your colormap is more useful for label images, in our case we are playing with CT/MR images.

@LoicCadour @ocommowi
What's the consensus here? Can we merge this, close it, or is more discussion needed?

I quickly tested this branch (with MUSIC) it was working fine. The things I've done with the ClutEditor are not really related, so I've nothing else to add.

OK - will merge after the meeting, unless there are objections.


Feature branch commits cause unwanted pushes

@dstillman, I just noticed that feature branch commits on https://github.com/citation-style-language/styles seem to be pushed to the "master" branch of the https://github.com/citation-style-language/styles-distribution repo. This is bad. Only commits to "master" should trigger a push.
See e.g. https://github.com/citation-style-language/styles/commit/45503dc0655a16133925335457490682e27116e9 (feature branch "travis-changes") which caused https://github.com/citation-style-language/styles-distribution/commit/22df00eb507a510561610fd77e1748f13dd35a00
(cc @adam3smith)

I guess one quick way to fix this is by limiting the Travis builds to "master" by whitelisting it in .travis.yml (per https://docs.travis-ci.com/user/customizing-the-build/#Whitelisting-or-blacklisting-branches ), although I find the feature branch Travis builds rather useful.

Hmm, I thought we fixed this...

Last time we were having issues with PRs: https://github.com/citation-style-language/distribution-updater/commit/861cb537d18a261175da04968f207a01de7b25a6
I wonder if the Travis payload mentions the branch.

I wonder if the Travis payload mentions the branch.

Looks like it does: https://docs.travis-ci.com/user/notifications/#Webhooks-Delivery-Format ("branch": "master")
Could we then just use (new lines indicated with "+"):
    if request_body:
        try:
            payload = json.loads(parse_qs(request_body)["payload"][0])
            environ["type"] = payload["type"]
            environ["commit_hash"] = payload["commit"]
            environ["build_status"] = payload["status"]
+          environ["branch"] = payload["branch"]

...

def update_styles(environ):
    if (environ["response_status"][0:3] != "200"
            or environ["build_status"] != 0
+          or environ["branch"] != "master"
            or environ["type"] == "pull_request"):
        return

Yup. That's done — if you want to trigger a feature branch commit, we can make sure it works, and then I'll commit it.

https://travis-ci.org/citation-style-language/styles/builds/111580613

Awesome, thanks.
https://www.zotero.org/styles?q=augmented doesn't yet show https://github.com/citation-style-language/styles-distribution/blob/master/dependent/augmented-human-research.csl from two hours ago, though. Is that plumbing all working?

Oops, wasn't working. Should be fixed now, with the page updating within a few seconds of commits — thanks.

My commit from just now went indeed live on zotero.org/styles right away. Thanks for that too!

@dstillman, sorry to bug you, but https://github.com/citation-style-language/styles-distribution/commit/8681c7b16274c16b4ed63528b572a6ee81a4181a hasn't made it to the ZSR yet. (see https://www.zotero.org/styles?q=clara)
This should have happened already, right? It's been over an hour.

(it's online now)


fix bug: modify unit test file path to temp dir

Signed-off-by: songtao98<EMAIL_ADDRESS>Ⅰ. Describe what this PR does
Previous PR left relative path "." in unit-test which makes that everytime one runs that test will leave a config file in os.
This PR replace it with t.TempDir()

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/lgtm


MSID: 93509 Version: 1 DOI: 2<IP_ADDRESS>5217

"msas": "Neuroscience"
"msid": "93509"
"preprintDoi": "10.1101/2<IP_ADDRESS>5217"
"Reviewed Preprint posted": "2023-12-12"
"Sent for peer review": "2023-10-19"
"Posted to bioRxiv": "2023-10-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>5217v2/")
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=93509)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/93509/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[ ] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>5217)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/93509/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/93509)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/93509

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/93509

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/93509/v1/93509-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/93509)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

v1 loaded instead of v2


moment alternative?

Timeline depends on moment.js for parsing dates. However, moment is a huge dependency!
See here the source map of the vendor bundle for Home Assistant. This bundle only includes Chart.js and the timeline (source).
Note that we do currently bundle ALL moment locales. 225KB. Stripping out the localses will help but still leaves us 51.53KB for all of moment.

Would you be open for this package to use fecha for formatting and date-fns for date manipulation? Both are very small utilities.

I like date-fns, but Chart.js requires moment currently. Seems that date-fns(25kb when import format) is not much smaller than Momentjs on my test.

Oh, I didn't realize Chart.js also requires moment. Guess we can stick with that then.


Fix Smokey Plek configuration for domain suffixes.

This makes Smokey's Plek configuration consistent with the applications being probed (apart from the workaround for the content-data-admin vs. content-data alias, which we still need).
Tested on the (not-yet-live) prod cluster.

Looks like you might need to add dummy helm values for the smokey chart for kubeconform.


Doxygen not going through all C++ files

Hi, I checked my Doxyfile, and it works, with an output to docs/html.
I use this into my existing github action
    - name: Doxygen GitHub Pages Deploy Action
      uses: DenverCoder1/doxygen-github-pages-action@v1
      with:
        github_token: ${{ secrets.GITHUB_TOKEN }}

after my tests pass.
What is possibly problematic? Token? I haven't set any as I saw somewhere it should be automatic. Is it true?
Also my branch gh-pages looks like this :

Thanks!

That should work, does any error come up?

Yes Doxyfile is set to "docs"

Note that my workflow (named CMake) triggers another which is not mined (I guess it is from Github Pages).

Alright my bad I assume, it was that Doxyfile was setup with a absolute path to my source (which is obviously bad !!).
Thanks for the github action !!


fix(about-modal): makes changes for Openshift POC

This is work requested by @kybaker for the Openshift POC.
The about modal has been simplified a bit by removing the secondary logo.
Also a bunch of minor design changes :)

Deploy preview for pf-next ready!
Built with commit 8e91c9c95e98dfc87b93feda0ea78a062cb4f132
https://deploy-preview-1135--pf-next.netlify.com

Can you remove these vars?
https://github.com/patternfly/patternfly-next/blob/034a935e4bec8b0968c61ffb443aea42037a95af/src/patternfly/components/AboutModalBox/about-modal-box.scss#L84-L99

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Lack of clear tutorial how-to

Hello,
I just wanted to raise the need of a straight forward tutorial inside a non-docker environment. Can we expect to see that?
I have tried her and there and centre trying to set up an environment and failed for that ...there are absolutely no installation-guide...
Appreciate that

You are right @hsafe the tutorial requires docker.
In the mean time you can explore other examples/ like hello-world that don't require docker, just to fetch the code and build the project.
I think is well explained but feedback is welcome.

@hsafe, Sorry for the frustration, we should have something obvious to explain how to deploy the standalone WAR, and optional action plugins, into WildFly.  We'll try and get something in place ASAP.


Ensure Tasks Can Be Filtered

Task

Ensure that tasks can be filtered based on tags assigned to the structure object and implement functionality to perform the filtering.

Deliverables

Functions that allow filtering based on the task object structure

Pushed to Sprint 4, delivery by 12/6

From PR by @Yuxuan-Wu:

Methods for task filtering based on type, status, time implemented
Methods for task sorting from new to old and new to old implemented


Add in-place LU decomposition option for linear solve

Adds an option for doing in-place LU decomposition that overwrites the original A matrix with the combined L and U matrices. Adds a test using in-place LU decomposition with the Backward Euler solver.
closes #695

Codecov Report
Attention: Patch coverage is 99.03846% with 2 lines in your changes missing coverage. Please review.

Project coverage is 93.93%. Comparing base (1cbd28e) to head (b529224).

Files with missing lines
Patch %
Lines

...e/micm/solver/lu_decomposition_mozart_in_place.inl
98.05%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #698      +/-   ##
==========================================
+ Coverage   93.68%   93.93%   +0.25%
==========================================
  Files          57       61       +4
  Lines        3941     4141     +200
==========================================
+ Hits         3692     3890     +198
- Misses        249      251       +2

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


MX-581 Re-broadcast all messages received

Updated the _handleMessage function to re-broadcast all messages received.

Why is this necessary? This removes the ability to later have message specific transforms - from the PR I'm guessing you're having an issue with the connected event (otherwise this behaviour already exists)?

It still works the same, the only difference is that added, changed and removed events are broadcasted too.

@ttacon it's opening the publication client up for proxying, see https://github.com/mixmaxhq/emailer-chrome-extension-source/pull/2559 and https://github.com/mixmaxhq/app/pull/4242. (Btw @gaastonsr it would have been nice to provide such description :) However please hold off on commenting on that approach, Gaston is writing a discussion doc.

Closing in favor of new implementation.


remove pull_request target that does not work with fork prs

What type of PR is this?
/kind bug
What this PR does / why we need it:
follow up of https://github.com/kubernetes-sigs/release-sdk/pull/91
the Identity token is not inject when the PR come from a fork. to we inject that we need to use the pull_request_target but that might generate possible security issues.
However this repository does not have any secrets or sensitive data, but the attacker can use the token for other things.
So removing the pull_request target for now.
/assign @ameukam @saschagrunert @puerco @jeremyrickard @palnabarun
cc @kubernetes-sigs/release-engineering
Which issue(s) this PR fixes:

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

NONE

Thanks for the follow up! It's unfortunate that we cannot e2e test forks now :thinking:


Output CSV format in abundance-dist.py

Add --csv argument to output CSV format with column headers

[ ] Is it mergable?
[ ] Did it pass the tests?
[ ] If it introduces new functionality in scripts/ is it tested?
Check for code coverage.
[ ] Is it well formatted? Look at make pep8, make diff_pylint_report,
make cppcheck, and make doc output. Use make format and manual
fixing as needed.
[ ] Did it change the command-line interface? Only additions are allowed
without a major version increment. Changing file formats also requires a
major version number increment.
[ ] Is it documented in the ChangeLog?
[ ] Was a spellchecker run on the source code and documentation after
changes were made?

ready for review

Here's the latest checklist:

[ ] Is it mergeable?
[ ] Did it pass the tests?
[ ] If it introduces new functionality in scripts/ is it tested?
Check for code coverage with make clean diff-cover
[ ] Is it well formatted? Look at make pep8, make diff_pylint_report,
make cppcheck, and make doc output. Use make format and manual
fixing as needed.
[ ] Did it change the command-line interface? Only additions are allowed
without a major version increment. Changing file formats also requires a
major version number increment.
[ ] Is it documented in the ChangeLog?
http://en.wikipedia.org/wiki/Changelog#Format
[ ] Was a spellchecker run on the source code and documentation after
changes were made?

FYI: latest checklist is here http://khmer.readthedocs.org/en/latest/dev/coding-guidelines-and-review.html#checklist

ready for review

LGTM, after comments addressed.  Could you also take a look at scripts/abundance-dist-single.py? :)

@ctb  it is ready

Plz review before I need to update the Changelog againnnn

LGTM.


Feature request: Execute plugin only programmatically, not on page load

I'd like to be able to set the plugin to a mode where it does nothing by default. I'd then like to execute it programmatically from script.
The use case for this is JavaScript bundling. In my specific case I have a hard time to ensure that my JavaScript runs before cookieconsent.js. That's a problem because I might not be in time to set the options.
The solution would be to set the options from my code and then to programmatically trigger cookieconsent.js processing.
On-the-fly options change does not seem to solve this scenario.

I also ran into that problem when I wanted to check the visitor's country before executing cookieconsent. I resolved it like this:
https://gist.github.com/saschalalala/18211d261a301dee1142
Then I call

cookieconsent_wrapper();

in the success callback of my location check ajax.
Hope that can help.

CookieConsent V2 struggles to do this.
CookieConsent V3 does not run automatically if you pass autoOpen: false as an option. Then you can control many aspects of its execution. CookieConsent V3 also has a location feature built into it by default


Run tools/test-backend in python 3 mode in Travis

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

#1225 fixes failures in around half of the TestCases containing skip_py3.

Ahh, OK, that makes sense.  Merged, thanks @sharmaeklavya2!


Enable buyback delegate metadata for pay() txs to juicebox

Changes

New encodeDelegateMetadata() pattern as a new catch-all metadata encoding util for all delegate types

Encodes metadata for multiple delegates, using the new metadata-lib pattern, and the juicebox-metadata-helper package
Also supports legacy encoding of jb721Delegate metadata where version < 3.4 (not compatible with buyback delegate)
Will be extensible for any future cases where any project makes use of multiple metadata-lib-compatible delegates

Enables buyback delegate metadata encoding for Juicebox

Minted tokens count must not exceed half of available AMM liquidity (arbitrary) to avoid slippage. If condition is not met, buyback delegate metadata will be omitted
If included, buyback delegate metadata will instruct protocol to use ALL ETH in pay amount to purchase tokens on AMM. In order to reduce arbitrage opportunities, the client will estimate the output of the swap based on the market price, and encode a minimum of 95% of that estimate in the delegate metadata. This is arbitrary—we only care about getting a minimum of the number of tokens that would otherwise be minted by the protocol, but we don't want to request a minimum that is significantly smaller than the actual swap output, as it would increase the likelihood of the tx being front run.
Support for buyback delegate with Juicebox project is hard coded to projectId. We could determine this dynamically, but since there's no expectation of any other project using the buyback delegate this is prolly fine for now

Adds a notice in Juicebox Pay > TokensReceived section that indicate a payer may receive more tokens than what are estimated.

@peripheralist heads up: pushing a rebase 🙏


Cannot upload files >10MB to Amazon Cloud

rclone 1.27 fails to upload any file larger than 10MB to Amazon Cloud.  It generates the following error message:
Failed to copy: HTTP code 400: "400 Bad Request", reponse body: {"message":"Cannot complete upload"}

It is working for me.
I suspect a temporary glitch at Amazon.
Can you post the output of the command with the -v flag if you are
still having a problem?  Also tell me which platform and whether there
is anything special about your Internet connection (eg a proxy).
$ rclone -v copy 100MB acd:100-mb-test
2016/02/07 20:08:56 acd: Saving new token in config file
2016/02/07 20:08:57 100-mb-test: Failed to read info: Node not found
2016/02/07 20:08:57 Amazon cloud drive root '100-mb-test': Modify window not supported
2016/02/07 20:08:57 Amazon cloud drive root '100-mb-test': Building file list
2016/02/07 20:08:57 Amazon cloud drive root '100-mb-test': Reading ""
2016/02/07 20:08:58 Amazon cloud drive root '100-mb-test': Finished reading ""
2016/02/07 20:08:58 Amazon cloud drive root '100-mb-test': Waiting for checks to finish
2016/02/07 20:08:58 Amazon cloud drive root '100-mb-test': Waiting for transfers to finish
2016/02/07 20:09:47 100MB: Copied (new)

$ rclone lsl acd:100-mb-test
104857600 2016-02-07 20:09:46.982000000 100MB

I have further investigated the issue. It actually has nothing to do with Amazon and it is not the write that fails.  Here is the situation that causes the failure:
The original file is on a NAS that is mounted via cifs. Reading a large file on the cifs share (>10MB) fails if done with rclone (and it is only rclone that cannot read the file):
$ rclone copy FAIL /tmp
2016/02/09 18:18:47 Local file system at /tmp: Building file list
2016/02/09 18:18:47 Local file system at /tmp: Waiting for checks to finish
2016/02/09 18:18:47 Local file system at /tmp: Waiting for transfers to finish
2016/02/09 18:18:49 FAIL: Failed to copy: read /afs/media/Pictures/2007/FAIL: input/output error
2016/02/09 18:18:49 Attempt 1/3 failed with 1 errors
...

That is interesting....
I presume you are using CIFS on a linux machine?
This thread  looks relevant maybe.
Just to be sure you can cp the file onto the linux machine?

Thanks for the hint with the thread.  It was not the problem mentioned in the thread, but it is due to a problem in the samba server of the NAS, where the server and the client negotiated too large rsize and wsize values.
So it is not a problem with rclone at all, it only appeared with rclone as no other tool reads files in chunks that large.
Thanks for the help!

Well done for finding the issue - a tricky one to track down.
I'll close this now we've come to the bottom of it.
Thanks
Nick


External dependencies

This PR hopes to add support for importing external dependencies within the service-worker by bundling dependencies into the sw.js bundle.
At the very beginning (August in the commit history) I implemented the same changes as @dadleyy. https://github.com/DockYard/ember-service-worker/issues/63
Inside the package.json of my Ember app, I was consuming the fork directly from GitHub which worked fine. After recently upgrading our Ember project to version 3.5.1, the fix stopped working and I got a lot of Rollup and dependency related issues. The package was tested with 3.5.1 but I do need some guidance on how to test different ember versions and write automated tests for my changes.
I also thought I could maybe fix another issue by allowing one to configure babel. As the issues were not related to each other, the changes didn't make it into the final version and I'm saving this issue for another day or someone else.
I wasn't able to run the tests yet due to build errors. The loader.js addon is missing from your project, please add it to your package.json and after adding it, another build error Missing template processor. If anyone can shed some light on what the best way is to get the tests up and running, please let me know.
This is my first non trivial PR so any feedback is greatly appreciated.

@CodingDive I wonder if this is still relevant given the dependency updates that have been made lately?

I'm no longer working with Ember or this project. @MarcoUmpierrez can you confirm that the PR can be closed and is no longer relevant?


New Developer

I cannot even call myself a developer yet, b/c I'm still brand new... so please forgive me for this....
When I Run Former  and even Former Demo,  it builds but there's no Simulator or even a Storyboard that I can find... I thought I was going to be able to see examples.
Thank you!

When you fire up the demo project in Xcode, have you tried changing the build scheme?

The demo project launches with it set to the framework rather than the demo app.

Thank you Brendan,
I’ve never seen that section of Xcode and it was defaulted to the Framework.  When changing it to the Former-Demo it worked perfectly.
Wow..
Thank you for helping and for showing me something new.
Michael

On Apr 4, 2016, at 3:35 AM, Brendan Kilfoil<EMAIL_ADDRESS>wrote:
When you fire up the demo project in Xcode, have you tried changing the build scheme?
https://cloud.githubusercontent.com/assets/749593/14245432/613b5936-fab5-11e5-893e-0bacb806f3fd.png
The demo project launches with it set to the framework rather than the demo app.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/ra1028/Former/issues/20#issuecomment-205234774

No problem, Michael.  Happy coding! :)

Could we close this issue?

Who is this and what issue are you talking about ?

On Nov 27, 2017, at 2:53 PM, Zachary Khan<EMAIL_ADDRESS>wrote:
Could we close this issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ra1028/Former/issues/20#issuecomment-347356191, or mute the thread https://github.com/notifications/unsubscribe-auth/AEg27gTsHDSHQB3gzqZbr_Gpc7x7iUduks5s6z1wgaJpZM4H-IBJ.

@GadgetAddict you asked a question about the "Former" iOS library through Github issues. You got an email from @ZacharyKhan because he commented on the issue, asking if someone (you?) could close this issue as it's been solved.
Hopefully this screenshot comes through in the email but your reply went straight onto the Github issue:

So, could you please login to Github and close this issue? Thanks muchly 😊

Yes.  Close


Fix Issue#28

@PiyushKalyanpy, I have added the right-padding side of the Explore Courses Button
Before Changes -

After Changes -

The changes requested were not done, so we acre closing this PR, please reopen the PR once you made the changes.


reduce chunk size for countries table

FROM https://trello.com/c/3JWvIC8T/8-aggiungere-una-faq-per-grandezza-pacchetti-mysql
May I suggest to change the chunk size of the countries query?
While this slow the initial load from 17 to 18 seconds on this machine (for the sqlite db) it should be a non issue.
With a chunk size of 50 instead of 500 the packet size should be less than 500KB instead than the actual more than 4MB
The committer has not checked other tables

Thanks! This would fix #457

caravel load_examples
SQLlite - 50: 31.669s
SQLlite - 500: 30.263s
-- pretty insignificant, merging


Syntax for prepared queries

I came across this little package and it's great! Exactly what I was looking for when working with LibPQ, which doesn't provide named parameters.
I would find it more useful to be able to pass parameters as a Dict{Symbol,Any}, rather than pulling them from the environment with a macro - seems cleaner and safer, and works well if your workflow looks like load SQL query template from file > interpolate variables > run query. I don't think this package supports that because of the macro approach (correct me if I'm wrong, not a macro wizard here).
So I put that together for myself. Concretely, it's a non-macro equivalent to @sql_str called something like render(sql::AbstractString, parameters::AbstractDict{Symbol,Any}, that renders a template given a Dict of parameters. While doing that, I also hacked in a way to reuse parameters if the same parameter appears more than once in the template.
Does that sound like something you'd like me to contribute to this package?

Hey, very sorry I missed your comment here. I'm glad you like this library :-)
I think what you're looking for is SQL prepared statements, but with named arguments rather than positional arguments?
With positional arguments, things are already simple enough.  Let's use the following basic example to discuss.
Some setup:
julia> c = LibPQ.Connection("...")
PostgreSQL connection (CONNECTION_OK) with parameters:
  user = postgres
  password = ********************
...

julia> LibPQ.execute(c, "create table test (x bigint, y text)")

julia> using SQLStrings

julia> function runquery(conn, sql::SQLStrings.Sql)
           query, args = SQLStrings.prepare(sql)
           LibPQ.execute(conn, query, args)
       end

julia> for i=1:10
           runquery(c, sql`insert into test values ($i, $("str_$i"))`)
       end

julia> using DataFrames

julia> runquery(c, sql`select * from test`) |> DataFrame
10×2 DataFrame
 Row │ x       y
     │ Int64?  String?
─────┼─────────────────
   1 │      1  str_1
   2 │      2  str_2
   3 │      3  str_3
   4 │      4  str_4
   5 │      5  str_5
   6 │      6  str_6
   7 │      7  str_7
   8 │      8  str_8
   9 │      9  str_9
  10 │     10  str_10

Now to create and use a prepared statement with bare LibPQ:
julia> statement = LibPQ.prepare(c, raw"select * from test where x > $1")
PostgreSQL prepared statement named __libpq_stmt_0__ with query select * from test where x > $1

julia> LibPQ.execute(statement, (7,)) |> DataFrame
3×2 DataFrame
 Row │ x       y
     │ Int64?  String?
─────┼─────────────────
   1 │      8  str_8
   2 │      9  str_9
   3 │     10  str_10

Possible design
So, what I think you want is basically prepared statements, but with more convenience and with named arguments? Here's a possible design I think could work:
# The following would be a representation of a prepared statement with named arguments,
# but without taking them from the macro environment, and without referring to any particular DB
# connection.
q = lazysql`select * from test where x > $x`

# Maybe the resulting type would be called `ParameterizedSql` or something?
q::ParameterizedSql

# This can be prepared with respect to a particular DB connection, resulting in a prepared statement
# which can be executed.
statement = prepare(c, q)

# The statement would need to be a custom type to allow us to manage the name=>positional argument mapping
statement::SqlStrings.PreparedStatement

# We should be able to execute this with a dictionary, as you've suggested:
execute(statement, Dict("x"=>7))

# Or with a named tuple
execute(statement, (x=7,))

# Symbol dictionaries should probably also work
execute(statement, Dict(:x=>7))

We could also support positional arguments in ParameterizedSql so that we can offer a standard notation for positional arguments (IIUC different DBs have different syntax for this.)

Thanks! Yes, I think you have partially understood what I'm getting at. Given that the underlying library doesn't support named arguments to an execute command, at least in the case of LibPQ, I think SQLStrings might be a good place to provide a convenience layer that does named=>positional argument mapping.
However I don't think prepared queries were what I had in mind. The recommended way of using SQLStrings right now seems to be your runquery example, where you:
a) run SQLStrings.prepare to generate a parameterised query + a list of positional arguments, taking the arguments from the environment;
b) execute the parameterised query with those arguments, by supplying the parameterised query and argument list to your database connection of choice.
I wondered if it would be possible to just have an alternate route in a) where you provide a Dict or similar, rather than taking from the environment.
So my proposed adaptation to your design might look something like this:
# A representation of a parameterised statement with named arguments, but without taking anything from the environment
q = lazysql`select * from test where x > $x`
# It would also be helpful to have the ability to parse a string into a LazySQL object, so
# that we can read a parameterised query from a file
q2 = LazySQL(readlines("query.sql"))

# This can be prepared in the same way as the current `@sql_cmd` object,
# but explicitly providing the named parameters - maybe even have a kwarg route
query, args = SQLStrings.prepare(q, Dict(:x => 7))
# Then it's up to you to use your DB connector to execute the query with parameters, as normal.
# At this point query looks like `select * from test where x > $1` and args looks like `(7,)`
LibPQ.execute(conn, query, args)

Does that make sense? Maybe a prepared query is the way to go about this, but it seems to me that the above approach sidesteps needing to care about how different backends might construct a prepared query, and keeps all the action in SQLStrings.prepare instead.

Maybe a prepared query is the way to go about this, but it seems to me that the above approach sidesteps needing to care about how different backends might construct a prepared query, and keeps all the action in SQLStrings.prepare instead.

The downside here is that you loose some efficiency of prepared statements — normally those would involve transmitting the SQL to the database a single time, letting the database's SQL parser (and maybe query planner) process the prepared statement and return you a handle. Then you can execute the same prepared statement many times with different input data, but without resending the query.
Having said all that, I think it'd probably be fine to have a utility function to turn a LazySQL/ParameterizedSql into an SQLStrings.Sql object. Perhaps it'd be called prepare or perhaps something else.
BTW prepare is currently only a semi-public API; needed for use when connecting SQLStrings to database backends, but shouldn't be needed when just using queries. I think it needs to be slightly database-specific because the placeholder syntax isn't standardized — see the to_placeholder argument to prepare().


Remove redundant '?' in regex.

Python versions before 2.7.6 complain about '(.*)?' since
the '?' is redundant.  Regex compilation should strip this, but didn't
before then, and reported 'sre_constants.error: nothing to repeat'.
Clients should use more recent versions of the python 2.7 stream as
there might be more issues than this one; I fixed this mostly as reference.

Such a weird warning / error!


hack/generate/samples: strip operator-sdk metrics annotations

Description of the change:

hack/generate/samples: added SampleContext.StripBundleAnnotations() which removes all operator-sdk metrics labels from metadata and manifests in a bundle
*: renamed exported testutils to align with their functionality

Motivation for the change: This PR removes metrics annotations from samples in testdata/, which are inherently versioned by git. Removing these annotations permits proper release tagging of samples.
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[ ] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

/cc @camilamacedo86 @joelanford @asmacdo

/cherry-pick v1.2.x