[Android] Fixed test error or XWalkRuntimeClient

Added the api related javascripts to the apk of XWalkRuntimeClientShell
BUG=XWALK-4711
(cherry picked from commit 3c0a6bf1d25fba6e5ca287684c071e9c76340dcb)

@halton It seems okay except the error of update scripts

@lincsoon The sysapps APIs are deprecated, I do not think we need test them. Presentation API should be cover.


JPEG 2000

JPEG 2000

This issue was imported from Google Moderator
Moderator votes: +3
Issue added by Jamie Mason (@fold_left) on 2014-11-16
To vote this issue up or down, simply include +1 or -1 in your comment.

Added recently with #1572


Git sidebar chapter: Clone gets 'Permission denied'

I tried the first clone of Microscope with this:
git clone git@github.com:DiscoverMeteor/Microscope.git github_microscope
and got this: 'Permission denied (publickey).'
Server is Centos 6.5.
I searched here and in stack overflow, but couldn't find anything that seemed relevant to my setup, as they were mainly to do with pull requests etc.
So I tried the github suggested clone address: https://github.com/DiscoverMeteor/Microscope.git
and that worked fine.
Is this a better address to recommend for readers (especially newbies like me), or is this likely to come back and bite me later.
Cheers
Peter

I think you are right Peter -- I guess the assumption was most people use github anyway and have a ssh user setup with it, but this is a more reliable way to do it.


Hook project up to CI to run tests

I personally use CircleCi but happy to use Travis...or some other tool.  Either way, we should get these tests running.

Travis should be easy because it was already running on travis.

Ah, so it was.  I missed that.  Didn't realise the travis config was a hidden file.  I was looking for travis.yml

I guess, let's use this issue to make sure it's still working ;)

@LevelbossMike I haven't used Travis before really.  Any idea why this can't find the .travis.yml? https://travis-ci.org/ember-cli/ember-cli-deploy

Closed in #44


[Bug] Black screen error - Android

Nothing to follow up on here. How do you expect for anyone to help here? Provide some steps how to reproduce what you are experiencing.

I also have black screen in android.
media_kit: 1.1.10 # Primary package.
media_kit_video: 1.2.4 # For video rendering.
media_kit_libs_video: 1.0.4 # Native video dependencies.


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py
Fixes #1

It's a good idea to open an issue first for discussion.

[x] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!

I don't know which changes to make on README


IdentityServer4.Samples ImplicitFlowAuthentication won't build

Quickstart3_ImplicitFlowAuthentication has compile errors.
Issue / Steps to reproduce the problem
Execute IdentityServer4.Samples\Quickstarts\3_ImplicitFlowAuthentication\src\QuickstartIdentityServer\updateUI.ps1
Build IdentityServer\IdentityServer4.Samples\Quickstarts\3_ImplicitFlowAuthentication\Quickstart3_ImplicitFlowAuthentication.sln
The solution has build errors since IdentityServer4 2.1.0 version is referenced in QuickstartIdentityServer.csproj and respective events are not presented there.
Possible fix would be to update the reference of IdentityServer4  to 2.2.0
Relevant parts of the log file
Error	CS0246	The type or namespace name 'GrantsRevokedEvent' could not be found (are you missing a using directive or an assembly reference?)	QuickstartIdentityServer	*\IdentityServer4.Samples\Quickstarts\3_ImplicitFlowAuthentication\src\QuickstartIdentityServer\Quickstart\Grants\GrantsController.cs	57	N/A
Error	CS0246	The type or namespace name 'ConsentDeniedEvent' could not be found (are you missing a using directive or an assembly reference?)	QuickstartIdentityServer	*\IdentityServer4.Samples\Quickstarts\3_ImplicitFlowAuthentication\src\QuickstartIdentityServer\Quickstart\Consent\ConsentController.cs	115	N/A
Error	CS0246	The type or namespace name 'ConsentGrantedEvent' could not be found (are you missing a using directive or an assembly reference?)	QuickstartIdentityServer	*\IdentityServer4.Samples\Quickstarts\3_ImplicitFlowAuthentication\src\QuickstartIdentityServer\Quickstart\Consent\ConsentController.cs	136	N/A

The .ps1 is meant for us to update the sames when the code's all in sync. It's not really meant for you to grab the latest of the QS UI.


init.d script improvements

Objectives

Clarify script changing variables to capital letter.
Allow to overwrite all the variables with default variables.
Check if config file exists and it's readable.
Check if pidfile is writable.

Motivation

The current init script use lower case for some variables which names match with the output messages, changing this variables to capital letter makes the script more standard and more readable.
The current init script does not allow the overwrite of all the variables with environment variables, this would make that some users need to change the init script (instead of add environment variables or edit /etc/default/influxdb) or would make harder to manage influxdb from automatisation systems like puppet.
The config file exists by default ( built from package.sh ), however could exist a race condition in some environments, better if we check if config file exists.
Like point 3, some race condition could happen, better if we check if pidfile is writable.

Changes

Change all the variables in lower case by capital letter.
Move the definition of the variables before load the $DEFAULT file
Check if config file is readable, if not exit 4.
If the pidfile doesn't exists, check if writable doing a touch with the proper user, exit 5 if it fails.

Backward compatibility
No changes in the way influxDB is launched neither how the init script is called neither in the value of default variables, so it's full backward compatible.
Testing
I didn't find integration test over the builded package and the CI test seems to test "only" the code, not the init script.
For this reason I run some manual test:

with empty /etc/default/influxdb

root@vm01:~# service influxdb version
InfluxDB v0.9.0 (git: .... )

root@vm01:~# service influxdb status
influxdb Process is not running [ FAILED ]

root@vm01:~# service influxdb start
Starting the process influxdb [ OK ]
influxdb process was started [ OK ]

root@vm01:~# service influxdb start
influxdb process is running [ FAILED ]

root@vm01:~# service influxdb stop
influxdb process was stopped [ OK ]

root@vm01:~# service influxdb stop
influxdb process is not running [ FAILED ]

with some config in /etc/default/influxdb
root@vm01:~# cat /etc/default/influxdb
CONFIG=/opt/influxdb/etc/influxdb.conf

root@vm01:~# service influxdb start
config file doesn't exists [ FAILED ]

root@vm01:~# cat /etc/default/influxdb
PIDFILE=/var/run/influxdb.pid

root@vm01:~# service influxdb start
/var/run/influxdb.pid now writable, check permissions [ FAILED ]

root@vm01:~# cat /etc/default/influxdb
CONFIG=/etc/opt/influxdb/influxdb.conf
PIDFILE=/var/run/influxdb/influxdb.pid

root@vm01:~# service influxdb start
Starting the process influxdb [ OK ]
influxdb process was started [ OK ]

Thanks @KoeSystems -- we'll definitely take a look. Have you signed the CLA?

@otoolep I've just sign the CLA.

Thanks again @KoeSystems -- can you rebase? We had to merge in some other changes to this file and there is now a conflict.

@KoeSystems if you can rebase we'll merge this in right away. Thanks!

@pauldix @otoolep Rebase done, thanks!

@KoeSystems -- unfortunately the patch has conflict markers embedded within it i.e. >>>> and this change is not mergeable. If you can fix that I will then merge.

Done, sorry for the fail when I did the rebase. Now should works.

Tested this on a Digital Ocean droplet and all functionality remains in place. Thanks @KoeSystems

I've just open an issue for this [3803] (https://github.com/influxdb/influxdb/issues/3803)

I created a new PR [3804] (https://github.com/influxdb/influxdb/pull/3804) with the solution.


⚠ Thumbnails API Endpoint has degraded performance

In 075f3cf, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1132 ms

Resolved: Thumbnails API Endpoint performance has improved in d07a204.


More helpful error message when accidentally overwriting reserved properties like self.container

class Foo(ui.Widget):
    def init(self):
        with ui.VBox() as self.container:
            # remainder of widget construction here

This seems like a natural name for the "container" of the subwidgets, but surprise: this will break your application, because self.container is already used internally. This could use a more helpful error message than Host is not attached.!

In this particular case, container should also be constrained to be a string, I guess.


VCloud Driver Features/Bugfixes

VCloud Driver Features/Bugfixes
Description
Fixed a few bugs and implemented new features needed in the VCloud driver.
Overview of changes:

Bug fixes

Fixed incorrect VCloud 5.5 auth headers error.
Added function for conditionally removing admin password element when it would be invalid to include it (rather than removing unconditionally).

Added VCloud node description handling.
Added lease settings tracking for VCloud nodes.
Added shutdown option when destroying and undeploying a VCloud node.
Added ability to specify guest script text for VCloud VMs.
New driver methods:

VCloud_1_5_NodeDriver.ex_find_vm_nodes(): Finds nodes that contain a VM with the specified name.
VCloud_1_5_NodeDriver._ex_get_node(): Get a node instance from a node ID.
VCloud_1_5_NodeDriver._remove_admin_password(): Bug fix mentioned above.

Added tests for changes where appropriate.
Fixed tests broken by changes.

These changes were made by the OpenText Corporation.
Status
Complete, ready for review
Checklist (tick everything that applies)

[x] Code linting (required, can be done after the PR checks)
[x] Documentation
[x] Tests
[ ] ICLA (required for bigger changes)

Codecov Report

Merging #1373 into trunk will increase coverage by <.01%.
The diff coverage is 86.66%.

@@            Coverage Diff             @@
##            trunk    #1373      +/-   ##
==========================================
+ Coverage   86.34%   86.34%   +<.01%
==========================================
  Files         372      372
  Lines       76156    76366     +210
  Branches     6971     6995      +24
==========================================
+ Hits        65755    65942     +187
- Misses       7605     7621      +16
- Partials     2796     2803       +7

Impacted Files
Coverage Δ

libcloud/test/compute/test_vcloud.py
94.71% <83.73%> (-2.93%)
:arrow_down:

libcloud/compute/drivers/vcloud.py
83.29% <90.8%> (+1.19%)
:arrow_up:

libcloud/storage/drivers/s3.py
89.15% <0%> (-0.51%)
:arrow_down:

libcloud/common/nttcis.py
73.71% <0%> (-0.09%)
:arrow_down:

libcloud/storage/types.py
100% <0%> (ø)
:arrow_up:

libcloud/test/compute/test_nttcis.py
86.18% <0%> (ø)
:arrow_up:

libcloud/storage/providers.py
50% <0%> (ø)
:arrow_up:

libcloud/test/storage/test_s3.py
91.2% <0%> (+0.21%)
: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 3a2bf30...70099bd. Read the comment docs.

@KYDronePilot Thanks for the contribution.
Since it's a bigger change, can you please sign an ICLA - https://libcloud.readthedocs.io/en/latest/development.html#contributing-bigger-changes.
In the mean time, I'll review the changes.

I've added a couple of small comments, mostly looks good :+1:
Please let us know when an ICLA has been submitted.

@KYDronePilot Have you had the change to sign an ICLA yet?

@Kami I apologize for the delayed response. I just got my ICLA submitted. We are currently working on getting a CCLA submitted as well, since this change is made on behalf of the OpenText corporation. I will let you know as soon as it goes through.

Great, thanks.
I can confirm we have received an ICLA. And, yes, as you said, CCLA would be great.
I will go ahead and merge this.

@Kami Could you please hold off on merging the changes? We would like to get the CCLA in before the changes are accepted.

@KYDronePilot Sure, no problem, I'll wait for a CCLA.

I can confirmed we have received CCLA so I went ahead and merged this.
Thanks again for the contribution.


Why is the training accuracy okay, but the testing accuracy is very low?

The testing accuracy is low because the normalization has been used in training set but not use in testing set in the last version of my repository. I have revised my code in main.py to make sure the same transformation of the train set and val/test set.

Okay, thank you


Disabling Desktop mode doesn't work anymore

What are you experiencing an issue with?

[x] Aurora
[ ] Aurora supported game, (Fill in the game)
[ ] Aurora Wrapper
[x] Aurora Profile
[ ] Aurora supported device, (Fill in problematic devices)
[ ] Aurora Scripts
[ ] Aurora Updater
[ ] Aurora Lighting issue
[ ] Other, (Fill in another issue here)

What kind of an issue is it?

[ ] Crash
[ ] Performance problems
[x] Visual bug
[ ] Localization issue
[ ] Enhancement suggestion
[ ] Device issue
[ ] Other, (Fill in another type of issue here)

Expected behavior:
Aurora lighting should be disabled at desktop if the option "Enable Aurora to provide lighting effects for desktop" is disabled.
Actual behavior:
The latest version of Aurora shows lighting effects for desktop.
How often does this problem occur?
every time at latest version
Please describe the steps to reproduce this issue:
The issue can be reproduced by doing following steps:

goto Desktop Settings and disable "Enable Aurora to provide lighting effects for desktop"
open Windows desktop

What is the version of Aurora you are experiencing this issue with?
v0.6.0-dev2
Does the previous version of Aurora have this issue?
previous version (v0.6.0b) worked fine
Please include any logs, screenshots, or files that are related to this issue:
Log has this error:
[ERROR] 31.05.2017 18:30:37: Exception in GetActiveWindowsProcessnameSystem.Exception: Eine Ausnahme vom Typ "System.Exception" wurde ausgelöst.    bei Aurora.Utils.ActiveProcessMonitor.GetActiveWindowsProcessname() in D:\Repos\Aurora\Project-Aurora\Project-Aurora\Utils\ActiveProcessMonitor.cs:Zeile 92.
System: Windows 7 Professional, 64 bit

i have this same issue


Apostrophe should be for mandatory only

And add a footnot saying it is mandatory

Apostrophe on the footnote looks weird, will skip this one


bugfix catch exception when getting hudi partition (#35027)

bp https://github.com/apache/doris/pull/35027
Hudi use a thread pool to get files for each partition. And use a countdown latch to wait all threads finish. But if the thread throw exception, the countdown latch will not be counted down, and thread will be blocked.
Proposed changes
Issue Number: close #xxx

run buildall

run buildall


s.c_r('s_vi') will only work on a first-party cookie (CNAME)

Doc below should point out that the s.c_r() method will only work on a first-party cookie.
https://github.com/AdobeDocs/analytics-1.4-apis/blob/master/docs/data-insertion-api/overview/c_visitor_id.md#reading-the-s_vi-cookie

Thanks for the report! Recently the data insertion API was revamped, and no longer includes references to cookies. instead, it provides information on how to identify visitors, leaving the specific visitor ID implementation requirements to the user.
https://developer.adobe.com/analytics-apis/docs/1.4/guides/data-insertion/variable-reference/


Re-running cd commands should handle relative path

Cool feature maybe?

In the above image, I navigate to ~\Source\github\microsoft one directory at a time. It would be cool if re-running any one of the cd commands would work no matter my current working directory. It seems the path is already remembered in the link decorator.

Right now it acts as you would expect from ctrl+r though. If you run a recent command and hit cd .., you generally wouldn't expect that to be resolved relative to the last cwd in which you ran that command from.

yea cd .. is special in that it can be run anywhere, but paths that were relative and no longer resolve in the cwd don't make sense to re-run without moving to the original working. i definitely view this as a cool-to-have feature, not something I expected to work.

Yes it would be cool but it would mean how only those specific directories is handled specially to other cd calls and every other recent command. I think we should keep this as is and if you want to go to a different dir you should use the go to recent directory command instead which uses an absolute dir.


Text and layout updates to Results page

Edits to Results page and some rearrangement of elements, to make the page read better.
See this document: https://docs.google.com/document/d/10ojqtr3Hfrbfq_opqKISXNP0YmNV-E5Qwct1xDCxTno/edit?usp=sharing

I took a stab at this and I think there's a lot of good stuff to consider here, but it's actually pretty complex to implement because a lot of the text is actually dynamic depending on some specifics of the type of place selected.  Deflecting for future consideration.


Determine how we're going to build and host Storybook

Can we use GitHub pages for this?
What do we use to build Storybook?
Can this be done in PRs as deploy previews as well?
What do we want the URL to be?

Follow-up issue will be doing it.

Hey team! Please add your planning poker estimate with ZenHub @alisonjoseph @andreancardona @francinelucca @jdharvey-ibm

Related to #519 in terms of the investigation and requirements

Created a test project https://github.com/francinelucca/storybook-test and added storybook to it.

confirmed can build and deploy to github pages through github actions both when merged to main and when pr is opened: https://francinelucca.github.io/storybook-test/?path=/story/example-newstory--default
...
from #320:  Can't use GH actions to build Storybook in PRs because of forks and secrets does this stand? we're currently not using forks and we're able to build our app through GH actions without much issues, not sure what the secrets part is about?

if we go PR previews for this we'd definitely have to host somewhere else 'cause gh pages won't work for multiple prs open at once? 🤔  so  IBM Cloud Object Storage (dotcom hosts here according to #320), code engine, kubernetes cluster, vercel... 👀

Another option is to mirror what system squad is doing and use Netlify.
https://github.com/carbon-design-system/carbon/pull/11570

Another option for PR previews is to mirror what system squad is doing and use Netlify. carbon-design-system/carbon#11570

yes! I understand that'd work. I know we had previous concerns about third party hosting due to having to set environment secrets but that wouldn't be a concern for storybook (or the web-app anymore for that matter I think because from my understanding from https://github.com/carbon-design-system/carbon-platform/issues/519#issuecomment-1093191028 it'd all be static data so I don't even think we'd need secrets)

Using GitHub pages for our deployed storybook seems like the best idea, now we need to decide on a url.
What about platform.carbondesignsystem.com? any other ideas?

I know we're using GH pages for our dev docs already (https://carbon-design-system.github.io/carbon-platform/), does that mean we can't use them also for SB?
I'm thinking we can probably have storybook as a sub-route there and have them co-exist, is that a thing 👀 ?
or would the idea  be we'd bring all of that info over to SB as well and stop using just-the-docs?
CC: @jdharvey-ibm

@francinelucca I'd recommend chatting with @joshblack on that to see if what he did for system squad could work to host multiple sites out on github pages. If it will, then that's definitely the best option. If not, then hosting on netlify seems like a close second.

Could also have a subroute for each PR so https://carbon-design-system.github.io/carbon-platform/deploy-previews/322 would contain storybook for PR #322

Since our community will be using our Storybook for MDX docs, is there a subdomain fitting for that?

platform
docs

Kindof makes me think we'd want two storybooks maybe? One is just the MDX documentation for platform at using something like:

mdx
mdx-docs
docs

For subdomain, and then we have a different Storybook that includes those MDX components plus our platform private components and pages and we use that full Storybook for development and testing.
Just something to think through.

Neat! Might be worth testing what happens if you try to make a page in the docs called "storybook". I'm guessing it'll be fine because one will end up as storybook.html and the other will be storybook/, but it might be good to test just to make sure they won't collide.

@jdharvey-ibm did that and basically the docs take precedence, so https://francinelucca.github.io/storybook-test/storybook will take you to the docs and to get to the storybook you'll have to use a more specific url like https://francinelucca.github.io/storybook-test/storybook/ (the extra / makes the difference) or a path. I'm guessing it won't matter tho' cause we're using a CNAME for the storybook and will we even have a storybook.md?
The only thing about this approach is everytime storybook builds I'm storing the build in /docs/storybook so everytime I push to main I have to pull first 'cause it's outdated, there's a couple workarounds for that like:

We could serve github pages out of a separate 'gh-pages' branch, we'd have to host the docs there tho', or create a workflow that copies them over everytime they change
We could see if there's a way to ignore that folder for branch syncing purposes?
We could do the CI through Jenkins
IDK if there's a way to build/deploy storybook to github pages without having to actually host the files somewhere, I can't find much on this

Will solve this with github actions:
1 - Create a separate repository on carbon-design-system branch for the purpose of hosting storybook/PR deployment files
2 - set up gh pages inside repo (main/docs or gh-pages/docs)
3 - Build and deploy storybook to docs/storybook
4 - Setup custom domain
5 - Create github action inside carbon-platform repo with the following specifications:
Every time a pr is opened, reopened or pushed to and storybook files are changed:
build storybook , output to docs-build folder
commit output folder to repo from step 1 (docs/pr/{PR_NUMBER}) (might or might not need a github token for this, TBD)
6 - create a github action inside carbon-platform repo with the following specifications:
Every time main is pushed to and storybook files are changed:
build storybook, output to docs-build folder
commit output folder to repo from step 1 (docs/storybook) (might or might not need a github token for this, TBD)

Optional but recommended: github action to delete PR folder in separate repo when PR is merged in or closed so that deployment gets deleted.
in the future: s3 or netlify shine because you can set up auto-delete policies


Update Application-toolkit-trace-cross-thread.md

Add how to use SupplierWrapper in Java 1.8+
Please answer these questions before submitting pull request

Why submit this pull request?

[3776] New feature provided

[ ] Improve performance

Related issues

Bug fix

Bug description.

How to fix?

New feature or improvement

Describe the details and related test reports.

Hi, you should modify the document in this PR #3783 .

Agree, please move this change inside your PR of feature change.


Hubitat can't find HA Bridge.

I have been using HA Bridge for a while and I just purchased a Hubitat Elevation. When I go to the Hubitat's Hue Bridge integration app it doesn't find HA Bridge.

I changed some values in Bridge Control Settings and Hubitat was able to find HA Bridge.


Migrate to Swift 2.2

This PR fixes all the build warnings/errors necessary to build with Xcode 7.3 and Swift 2.2
The biggest change here is that now the name property of XCTest is a String?. The fix for this was to implicitly unwrap it for the tests.
After these changes I have successfully run the tests with no failures on my machine. I would also suggest long term pulling in CryptoSwift as a dependency, rather than copying the code, which I've had to modify here as well.

For some reason the tests were failing with xctool. Possibly a side effect of the new Xcode version. I swapped out the travis command to use raw xcodebuild (along with xcpretty) to fix this. Let me know if you'd prefer to stick with xctool and we can try and find a different work around.
Also note that the OHHTTPStubs binary vendored from their GitHub releases wasn't compiled with Xcode 7.3 so it won't work. The tests pass because travis machines have exceeded the download limit. If you'd like to fix that more explicitly for now, I can add --no-use-binaries to the carthage bootstrap call.

thx!

Thank you! And of course it would be awesome if you could create a new release with this changes :smile: :pray:

Awesome!


[Storefront Preview] SCAPI cache breaker for older v2 PWA Kit (@W-15214446@)

Porting the changes from #1701. Unfortunately, while the solution/concept is still the same, the implementation had to be different.
In the older V2 PWA Kit:

The StorefrontPreview component does not have access to the commerce api clients because they're not in the same package
The Proxy mechanism has already been used. So we need to add the cache breaker using the existing mechanism (see willSendRequest)

Todos:

[x] Write tests
[ ] Write some documentation for existing projects to consume the update?
[x] Bump up version number? To release as a patch.

Types of Changes

[x] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Documentation update
[ ] Breaking change (could cause existing functionality to not work as expected)
[ ] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

Changes

(change1)

How to Test-Drive This PR
https://runtime.commercecloud.com/mobify/blitz/will/preview?path=/global/en-GB/product/25695327M?color%3DJJI15XX%26size%3D9SM%26pid%3D701644153083M

Note that the product price should be the regular price
In the Preview form, enter tester=yes as the custom qualifier
Click on the Preview button
The price is immediately updated with the discounted promo price.

Note: to verify that the environment is actually using a bundle for the older PWA Kit, you can add an item to the cart, and that should result in an OCAPI call (not SCAPI).
Checklists

General

[x] Changes are covered by test cases
[x] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[x] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

Opening a new PR #1740


Augment Expanding - Quantile (Polars Engine): More Elegant/Scalable Way of Handling Parameters

I suggest we come up with a more elegant solution that the "quantile" parameter.
I can see us adding more functions that have parameters down the road and adding parameters to the augment_expanding function isn't scalable.
One thought I had is allowing users to add polars functions like we do lambda functions with pandas as Tuples.

So these polars functions would throw an error if the engine was not set to 'polars'?
Something like this?
( 'quantile', rolling_quantile(quantile=0.75, window_size=<dataframe_length>, min_periods=1, interpolation='midpoint') )
The current implementation operates only with rolling_ functions in this manner df.group_by(group_names).agg(rolling_agg_function). May need to find a more flexible solution of the form df.group_by_rolling.agg(agg_function).
Right now series-based lambda functions will work with the polars backend through rolling_apply, albeit they are slower than pandas, which could be do to an inefficient implementation at the moment.
Haven't tried to implement a polars backend for the dataframe-based lambda functions in augment_expanding_apply (since presumed it will be slower) until we get the series-based lambda functions performing faster.

Yes I think you are on the right track.
We should have the column name and function linked so users can add them and control the parameters.
Something like:
("quantile_75", pl_quantile(x, quantile = 0.75))

Not technically a lambda function but a function that will spit out a polars function.

Ah, yeah, that's a nice solution.


Generate bracketless tag id in FreeMarker forms

Before this change if FreeMarker Spring form macro was bound to a path
which contains square brackets, those brackets would also appear in id
of generated tag, making the id invalid.
As of this fix all FreeMarker Spring form macros generate tag with id
that does not contain square brackets.
Issue: SPR-8732
I have signed and agree to the terms of SpringSource Individual Contributor License Agreement.

Looks good, thanks!


Is it possible to subscribe to multiple service characteristics of 'notify type' for a device

Prerequisites

[X] I checked the documentation and FAQ without finding a solution
[X] I checked to make sure that this issue has not already been filed
[X] I'm sure that question is related to the library itself and not Bluetooth Low Energy or Classic in general. If that so, please post your question on StackOverflow.
[X] I'm running the latest version

Question
I am able to connect to the device. Also monitorCharacteristicForDevice for device works for one serviceUUID and characteristicUUID but i want to get it working for 5 services and charecterisitcs. Is it possible to do using react-native-ble-plx library?
Question related code
const monitorMultipleCharacteristics = async () => {
    try {

      const monitorPromises = uuids.map(({ serviceUUID, characteristicUUID }) => {

        return new Promise((resolve, reject) => {
          BLEService.monitorCharacteristicForDevice(
            serviceUUID,
            characteristicUUID,
            async data => {

                case UUID:
                  const temperature = parse(data.value)
                  break

                case UUID:
                  console.log('dispatch service')
                  break

                default:
                  console.log('Unknown UUID', data.uuid)
              }
              setTimeout(resolve, 100, data)
              console.info('success', new Date().getTime())
            },
            async error => {
              console.error(error)
              await BLEService.finishMonitor()
              reject()
            }
          )
          // Store the subscription so it can be cleaned up later
          //subscriptions.push(subscription)
        })

        // Wait for each monitor to be set up
      })

      Promise.all(monitorPromises)
        .then(data => {
          console.log(data)
        })
        .catch(error => {
          console.error('Error in one of the monitors:', error)
        })
    } catch (error) {
      console.error('Error setting up characteristic monitoring:', error)
    }

    // Optionally return the subscriptions if you need to stop monitoring later
    //return subscriptions
  }

Yes, it is. Please remember that if you use Promise, it will resolve only once according to the contract. That's why monitorCharacteristic is callback based.


C++ 17 fixes

Fix missing C++17 headers in fw-logs-formatting-options

Can one of the admins verify this patch?

Thanks for the PR,
Closed and re-opened to trigger CI.
Can you add the compiler you use and error you get?


Drips - Prepare Drips Candidates

Overview
More ERC20 support coming soon, including USDC (which we hold 1,000,000 of)
We should have a list of Drips recipients for us to send to, focusing on any protocol/group empowering our governance
Budget to commit: 10%
Candidates

Gnosis
Snapshot
Sybil
Discourse

From @abbey-titcomb :

Create Drips community
configure splits to other projects

Wondering how we can drip to Discourse... it's a pretty big community that's relied on pretty heavily by the Web3 community. Might be worth it to see if we can find a way to get them funding. I can look into it but from first glance looks like they don't accept donations (https://meta.discourse.org/t/possible-to-donate-money-to-discourse/26208?page=2)
Probably should add Compound to the list for deploying GovernorAlpha.


Stage1

Commit initial modifications for custom detection, specifically for the BDD dataset.

Hi, it seems that this repo is inactive for a while... (more than a year :worried:)
Would you consider to pass your changes to this fork https://github.com/Borda/keras-yolo3 ?


Decorator for TaskRule dependency

i.e.
class R(TaskRule):
    ...
    @dependency
    def load_data(self):
        ...

By default, all methods are now dependencies.


Provide installer

macOS installer (dmg or pkg)

creates the named MIDI port
configures Synthesia for making use of that MIDI port for light guide control

is that even doable? can we safely patch Synthesia's configuration?

Added a basic DMG but that isnt what I really had in mind when saying "installer". We still need something that does the heavy lifting of that MIDI port setup as well as the Synthesia configuration.

HI! Thanks for awesome work! With an proper installer maybe it could be a better experience, but it works just fine with xcode project - finally can throw away printed numbers for the keys XD

For the Synthesia configuration;
We would need to assert this OutputDevice one in ~/Library/Application Support/Synthesia/multiDevice.xml
<?xml version="1.0"?>
<DeviceInfoList version="1" useSoftwareInput="1">
	<OutputDevice version="1" name="IAC Driver LoopBe" enabled="1" userNotes="0" backgroundNotes="0" metronome="0" percussion="0" lightChannel="-2" />
</DeviceInfoList>

Added a Synthesia configuration patch mechanism that works but is not perfectly user-friendly. The user has to accept patching the file in question in a file-dialog. But that seems to work fine then for me - see https://github.com/tillt/KompleteSynthesia/tree/till/0.30

I played a bit with the idea of providing a setup-dialog for giving users a neat and unified way for granting privacy related permissions. The inspiration came from looking at Alfred's permissions setup dialog.
Initially I was hoping I could get the current status of those permissions from macOS and then give users a way to add the missing permissions through a simple dialog. Trouble is, that is broken in macOS - one can not get the status of Accessibility permissions and then hope for achieving anything around Input Monitoring. See https://openradar.appspot.com/7381305
Next approach I tested was ignoring current permissions and simply giving users a unified surface for requesting those permissions;

If the user now was clicking on Accessibility first, the permission request dialog from macOS would correctly pop up and do its thing. All fine and dandy. When the user then clicks on Input Monitoring, nothing happens - macOS will not show the permission request dialog for that feature. Actually never ever -- even if the app gets restarted in between. macOS appears to mix the idea of Accessibility and Input Monitoring as soon as the former is setup before the latter.
The only way to get all those permissions requested and setup properly is when Input Monitoring gets requested first.
That in turn means such setup dialog would have to be a step-by-step guide -- first only asking for Input Monitoring and when that was setup by the user ask for Accessibility. Overall, the convenience gain is vastly diminished by that macOS bugfest and I decided to drop the entire idea. The flow would be very similar to what we got already, so why bothering with added dialogs.


warning: self-comparison always evaluates to true

The build logs on Debian report the following warnings:
bottleneck/src/move.c: In function ‘move_rank_int64’:
bottleneck/src/move.c:2009:24: warning: self-comparison always evaluates to true [-Wtautological-compare]
                 if (aj == aj) {
                        ^~
bottleneck/src/move.c: In function ‘move_rank_int32’:
bottleneck/src/move.c:2070:24: warning: self-comparison always evaluates to true [-Wtautological-compare]
                 if (aj == aj) {

Is it intended?

Yes, those are intended. They check for NaN.


Syntax-Highlight Color for "Invalid" Things Could Be More Legible

This theme is absolutely GREAT!  But the Syntax-Highlight color for "Invalid" elements has a red background that makes things very difficult to read, particularly JSON comments. This is what I'm referring to:
.invalid.illegal {
  background-color: #cc6666;
  color: #1d1f21;
}

Bring up any JSON file with lots of code comments. You will see what I mean.
I realize that Code Comments are not in the JSON spec, but that are a proposed addition to the spec.
The default "Atom Dark" Syntax theme has better-looking "Invalid" elements. They are a shade of purple, and they subdue the background, so it's more pleasant on the eye.  I recommend a similar approach... something like this:
.invalid.illegal {
  background-color: #330000;
  color: #990000;
}

Gonna close this as wontfix, see this comment for the reasoning: https://github.com/atom/base16-tomorrow-dark-theme/pull/22#issuecomment-138168945


Safe tx resubmit

This PR introduces a safe tx resubmit function that can be executed when Account Viewer is not sure if tx has been submitted successfully or not (ex. connection timeouts).
It will display the following message in such cases:

Please note that we display: "Send another transaction" link if we get a response body from horizon. We hide this link in this view to disallow users to creating another transaction.
Because of the fact that a lot of code has been moved I recommend reviewing this PR in the split view mode.

No problem Scott! Thank for the review!


Creating code tabs

Hi, how do I create code tab like this in minimal

There's no built-in way. You'll have to add your own HTML template and relevant JS / CSS around.

There's no built-in way. You'll have to add your own HTML template and relevant JS / CSS around.

Do you know any example, I'm not very proficient with frontend and static generator

Then learn the skills required for your needs and DIY.
The repository is a Python project, designed to be a plugin for Python Markdown. It's unlikely to be able to be integrated into Jekyll, which itself is a Ruby program.
Just FYI, Material for MkDocs supports this feature out of the box.

Found solution, closing!

Would you mind sharing your solution? That'd help others with the same issue.

Would you mind sharing your solution? That'd help others with the same issue.

I simply added css/js into assets


Looks nice but how it works?

Hello Fossdroid community!
The site looks awesome and I like the recommendations and categories but if I download and install a app how will be upgraded? Fdroid app or Fdroid like app stores?
Sorry if a noob question

As long as it mirrors  F-Droid’s APKs,  F-Droid will be able to update it. The APK only needs to be signed with the same key.
The burger menu also has a link to  F-Droid. If you have a way to open that in F-Droid, you can also install it via the F-Droid app. With fennec -> share URL -> LinkSheet -> F-Droid it works for me, but there are probably more ways to do that. F-Droid and Neo Store can work with F-Droid URLs.


Facilitate JSON logs

When errors are logged as multi-line text, it plays badly with system like Splunk that consume each line from the logs as a separate item. Specifically, it becomes difficult to access tracebacks.
Uvicorn supports log configuration involving custom Python objects. It may be convenient for Tiled to ship with a JSON logger to cover this important use case.

We have discussed creating a repository of deployment examples. I think this would be better placed there than in Tiled core. Closing.


2013 additional infrastructure investments in table not 0 in 2013

For an empty scenario the additional infrastructure investment in the table is not 0 for 2013

@antw can you please have a look?

I'm not too familiar with the Gqueries used to calculate these costs, but this seemed to become an issue when we added the explicitly-modelled electric technologies (the dynamic electricity curve project).
I think the problem is this line and similar lines in other peak load queries. The load of these technologies should perhaps be wrapped in a QUERY_DELTA like the other values.
Without a QUERY_DELTA that query returns the following:

Present: 49 869.86
Future: 49 889.94

With a QUERY_DELTA:

Present: 0.0
Future: 20.08

Assigning @DorinevanderVlies to take a look at this as part of https://github.com/quintel/mechanical_turk/issues/97

This issue (with this description) is fixed by https://github.com/quintel/etsource/commit/9ce203b63f090a834c9b8d84ec3265356e485798.
However there are two more issues which I will adress separately

Initial investment costs very different on beta and on live
with default settings additional investment costs are non-zero on beta


Adding solar system examples.

Code isn't perfect, but it works ;-)

@aldrinmartoq Thank you very much for providing this example for testing.
Just note this example adds lots of resources that increase a lot raylib size, it won't probably be included with raylib installer in a future, I need to decide it...


system start fails with empty platform flag

Signed-off-by: Neha Lohia<EMAIL_ADDRESS>Description
This PR fixes #3673
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

Codecov Report

Merging #3675 (064e4c4) into master (8f5e194) will decrease coverage by 7.89%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #3675      +/-   ##
==========================================
- Coverage   19.12%   11.23%   -7.90%
==========================================
  Files          39       39
  Lines        3613     3615       +2
==========================================
- Hits          691      406     -285
- Misses       2741     3118     +377
+ Partials      181       91      -90

Flag
Coverage Δ

gointegrationtests
?

unittests
11.23% <0.00%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

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

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

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

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

Continue to review full report at Codecov.

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

@nehaLohia27 There is lint warning on the lines surrounding this change. See your earlier PR, please fix that as well.

@navendu-pottekkat The lint errors are already fixed and merged in earlier PR. This PR does not have lint issues.


feat: add support to plugeth plugins

Description
Closes #5
Type of change
Please delete options that are not relevant.

[ ] 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

This PR is stale, and plugeth is generally used less. So, for now, I'll close the effort on this.


Fix Llama rope scaling factor, improve accuracy

Ticket
Fixes #15737, improves accuracy.
Problem description
Meta's reference model incorrectly used a rope scale_factor of 8 for the 1B and 3B models, it should be 32 as fixed after release in a patch to their HF repo that was not reflected in their GitHub.
What's changed
This applies a fix to our clone of their repo and to our own code. We regenerate all reference outputs and update our accuracy figures, which are now improved.
Performance was degraded on main on 4th December, the updated tables here reflect those performance figures but this patch does not degrade performance itself.
Checklist

[x] Post commit CI passes
[x] Single-card
[x] T3K

Tests pass

I'm rebasing this locally and testing if it still works. If it passes we can merge 👍


Multi-platform support

Currently there is only amd64 architecture in release repository.  According to this proposal https://github.com/kubernetes/kubernetes/blob/master/docs/proposals/multi-platform.md it would be beneficial to have multi-platform support for heapster.
Heapster is very important component (used in Dashboard or for Horizontal Pod Autoscaling). Lack of official support for other architectures will limit k8s cluster functionality.

cc @luxas

I have scripts already that shows how to cross-compile heapster, grafana and influxdb for all arches here: https://github.com/luxas/kubernetes-on-arm/tree/master/addons/images
ping @kubernetes/heapster-maintainers is this something you'd be willing to merge in v1.2?
I can probably make a PR for that probably, with full backwards compability.
After all, it won't be any functional changes, just release infrastructure changes,

This should be done together with building the image in container (in the similar way to how Kubernetes is built now). I'm happy to accept contributions!

@luxas did this ever happen?  As far as I can tell the latest ARM version of heapster is 0.8.0
Thanks!
--brendan

@brendandburns Well, kubernetesonarm/heapster:0.8.0 is v1.2.0-beta.2 of heapster. 0.8.0 is the release of Kubernetes on ARM it was shipped with. Also there are kubernetesonarm/grafana:0.8.0 which is 3.1.1 and kubernetesonarm/influxdb:0.8.0 which is at version v0.13.0.
However, it's of course best if we have official images for them, so here we go: #1387
I planned to do it earlier, but fully focused on sig-cluster-lifecycle here in between.
Cheers, Lucas


Add more tracking rules

This pull request adds more tracking rules.
In fact, I wrote a Python script to find all spy pixels in my inbox.
Feel free to use it if it is helpful at all: https://github.com/martbock/spy-pixel-finder

Awesome! Will check out your project too, thanks again


[New game]: Life of blobb

🎮 Game Request
It's hard being a blob -- there is always a bigger blob who may consume you. Even if you outgrow everyone, this doesn't mean that you will succeed - you need to be quick and fit to get through life. And in the end, it all depends on luck, doesn't it?
Point down the features
Walls kill you instantly, so go through the gap
Score points by growing and shrinking Blobb
Finish levels faster to earn bonus points
Move with arrow keys or ASDW.
Show the high scores with Esc / P
Show this info screen with I.
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @S-ishita !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @S-ishita ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


CI setup to formalise supported GHC versions

Hi! First of all, thanks for your work reviving HsQML! If you are interested I would like to submit a PR for a CI setup that would stay in sync with a cabal file's tested-with stanza, using https://github.com/Kleidukos/get-tested.

Sure, pull requests are always welcome.


Syntax error in IE 11 on development build

Basically because this isn't transpiled for some reason:
https://github.com/kybishop/ember-attacher/blob/a2f1cd0850c4afc6b371cf121ff35138e4c0eca3/addon/-debug/helpers.js#L11

It actually works on a newest version. Nevermind, sorry for the trouble!


feat:add cluster template api testcases

Signed-off-by<EMAIL_ADDRESS>
What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

Codecov Report

Merging #507 (b6356a8) into release-1.3 (2084b77) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           release-1.3     #507   +/-   ##
============================================
  Coverage        12.20%   12.20%
============================================
  Files              110      110
  Lines            16911    16911
============================================
  Hits              2064     2064
  Misses           14600    14600
  Partials           247      247

/lgtm
/approve


Error: libzmq.so.1: cannot open shared object file: No such file or directory

Hi I try to deploy meteor app that use a node js binding for the ZeroMQ library, i have properly installer the libzmq on the server but i got always this error
Error: libzmq.so.1: cannot open shared object file: No such file or directory
I have try to run my meteor app on the server with the meteor run commande and it work well.
Thanks if anybody can help.
=> Starting meteor app on port:80
/bundle/bundle/programs/server/node_modules/fibers/future.js:280
						throw(ex);
						      ^
Error: libzmq.so.1: cannot open shared object file: No such file or directory
    at Module.load (module.js:356:32)
    at Module.Mp.load (/bundle/bundle/programs/server/npm/node_modules/meteor/babel-compiler/node_modules/reify/node/runtime.js:16:23)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at bindings (/bundle/bundle/programs/server/npm/node_modules/bindings/bindings.js:76:44)
    at Object.<anonymous> (/bundle/bundle/programs/server/npm/node_modules/zmq/lib/index.js:6:30)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)

Your meteor app runs in docker. So installing the library on server doesn't work. Create a docker image with your library using meteorhacks/meteord as the base image and use it as the mup docker image.


⚠ Homepage (omg.lol) has degraded performance

In 505ab5f, Homepage (omg.lol) (https://8bit.lol/) experienced degraded performance:

HTTP code: 503
Response time: 2908 ms

Resolved: Homepage (omg.lol) performance has improved in 130bcd4 after 18 minutes.


feat(FMP-294): editable input element

Current dependencies on/for this PR:

main

PR #21   👈

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: 🔄 Waiting for checks to pass
This pull request is currently waiting for CI and mergeability checks to pass before being merged.
This comment was auto-generated by Graphite.

Job Reference: hO3oh8hmx69S4NBtVksc


[2.0] xread() halts when the connection is lost.

Describe the bug
If the connection with Redis is lost while the program is awaiting for xread() it get stuck until the connection is reestablished.
It doesn't raise any Exception, or at least timeout after the block time.
If I wrap xread() with asyncio.wait_for(), and force a disconnection, the wait_for() timeouts correctly, and the subsequent xread() calls raise ConnectionError. So it confirms that the bug only happens if the code is awaiting xread() when the disconnection occurs.
To Reproduce

You need a running Redis with a stream with entries
Set correct values for redis_dsn and stream. Then run:

import asyncio
import logging
from datetime import datetime

import aioredis

logging.basicConfig(level=logging.INFO)
logger = logging.getLogger()

redis_dsn = "redis://localhost:6379"
stream = "stream_name"

async def main() -> None:
    redis = aioredis.from_url(redis_dsn, decode_responses=True)
    index = "0"

    while True:
        try:
            response = await redis.xread(
                {stream: index},
                block=1000,
                count=10,
            )

            logger.info(
                f"{datetime.now()} : Read {len(response)} messages from stream."
            )

            for _stream_name, messages in response:
                for message_id, values in messages:
                    index = message_id
        except Exception as err:
            logger.warn(err)

asyncio.run(main())

The logs should be displayed as the entries are read.
Force a disconnection, such as stopping the Redis.
You should see that the logs have stopped and no exception is logged.
[Optional] If you force a reconnection, the program should resume so it start to display the logs again.

Expected behavior
After running the snippet, you should see the logs been displayed rapidly. After the disconnection the program will halt, so the logs will also stop, and no exception is raised.
Logs/tracebacks
It's not applicable.

Python Version
Python 3.8.10

aioredis Version
aioredis 2.0.0

Additional context
No response
Code of Conduct

[X] I agree to follow the aio-libs Code of Conduct

Hopefully this is resolved by #1156 as this seems to relate to #778


Integrate DynDNS/NoIP Services that also support ACME

There are at least two (free) services that offer DynDNS like features which seem to support ACME:

https://www.duckdns.org/domains
https://freedns.afraid.org/faq/#17 (might not make much sense; see my comment below)

Make a "HTTPS-Domain"-Subscription out of this - you dont choose a shop here - but you have DuckDNS and FreeDNS here as free options. Let the user enter the needed credentials and then trigger the ACME script accordently and exchange the cert nginx.

"Bad" news: https://github.com/acmesh-official/acme.sh/issues/2305#issuecomment-529165181

I ask because the dns_freedns plugin will not work with a "public" domain shared by FreeDNS... it will only work with a domain that you own.

This - kind of - defeats the idea to require almost "nothing" from our users. I will focus on Duckdns.

@rootzoll please be aware that the DuckDNS API does not work (with acme.sh) when more than one name is to be included in the certificate.
E.g. raspiblitz.duckdns.org will work, but myraspiblitz.duckdns.org + raspiblitz.duckdns.org will not.
The cause for this is that acme.sh tries to validate using different TXT records, but DuckDNS doesn't support multiple different TXT records to be set.
For our current setup this is just something to keep in mind (and maybe point out to users). But my idea was to always set the primary subdomain and a wildcard: raspiblitz.duckdns.org + *.raspiblitz.duckdns.org. Unfortunately this won't work like that.
Also bare in mind that the certificate for *.raspiblitz.duckdns.org will not considered valid if you point your browser to raspiblitz.duckdns.org.

https://github.com/acmesh-official/acme.sh/wiki/dnsapi

@frennkie ok I added dynu.com to for the v1.6.1 release - works fine on my tests with ACME and LetsEncrypt. Marking for final testing.

Tested it a few more times and works well. Will be part of v1.6.1 release. Cosing issue.


Which part of a string

In https://www.dynamodbguide.com/querying we use an expression on the OrderId key which is formatted as a datestamp and random generated number to retrieve a number of items in a specific time period.
aws dynamodb query \
    --table-name UserOrdersTable \
    --key-condition-expression "Username = :username AND OrderId BETWEEN :startdate AND :enddate" \
    --expression-attribute-values '{
        ":username": { "S": "daffyduck" },
        ":startdate": { "S": "20170101" },
        ":enddate": { "S": "20180101" }
    }' \
    $LOCAL

An OrderId would look like this "20170609-25875", then how does Dynamodb knows that only the first part should be considerated in the comparison not also the random number part? I am sort in the dark here.
And does BETWEEN sort of convert the string to number to do the comparison?

Hey @efleurine, great questions! I'll try to answer but let me know if it doesn't make sense.
The OrderId is a string type, not a number. When doing the BETWEEN operations, string values are ordered according to their UTF-8 bytes. Thus, you could imagine the ordering looking something like this:
20170101
20170101-00001
20170101-00002
20170101-00003
.....
20171231-99999
20180101

So when we specify "BETWEEN 20170101 AND 20180101", it will look for any values between those two UTF-8 bytestrings.
Does that make sense?

Yeah it make sense -
thanks @alexdebrie  for the quick reply and this great guide

You're welcome! Going to close this :)


Add option to set diff editor color for each use, not globally

I'm creating an extension and need to set the colors when my extension opens the vs code diff editor.
I know that there the diffEditor.insertedTextBackground and diffEditor.removedTextBackground settings, but I only want the colors changed in my extension and have the default diff colors be used if the user was diffing a file.

We currently have no plans to provide an API for editor specific colors...


Preview widget

Used for entire site, e.g. user profiles, forum posts, group pages, stories.

Related to #48. Created at skairunner/dj-preview-mixin


Wrap ${componentHTML} to kill checksum warning

Wrapping ${componentHTML} with an empty <div> gets rid of this console warning:
Warning: React attempted to reuse markup in a container but the checksum was invalid

This seems wrong to me, as it would remove any re-use of ssr code? Is it in both prod and dev you get these messages?

Correct. That's a bad solution, and this issue should probably be renamed to "Fix Checksum Console Warning"
Reading this has be wondering if it might be this:
"Your issue appears to be with CSS - if you are using autoprefixer and inline-styles that would explain your difference here."


Handle page numbering in the merge PDFs route

When merging several PDFs through the Merge PDFS route, it would be really useful to have the ability to reset the numbering of pages in the merged file.
For instance, in order to generate a PDF with a different header for each chapter, we first generate partial PDFs and then merge them. The issue is that page numbering in the merged footer keeps the inherited numbering, i.e. restarting at 1 for each chapter.
Would it be feasible to address this issue ? I'm thinking of an optional footer or a resetNumbering form field accepted by the merge route.
Thank you for the awesome work.

Hello @KojoEnch 👋
That would make a lot of sense, but I’m not sure this feature is available in the underlying tools used by Gotenberg. For instance, I did not find anything with QPDF.

@gulien thanks for your  reply.
I installed QPDF, read the docs and tried a few CLI. At one point, I thought I had found the Graal with the  --set-page-labels option, then I learned PDF page labels are NOT page numbers in footer ! :expressionless:
If you suggest any other library used in this repo where we might find something useful, I'd be glad to continue searching.
If renumbering while merging ends difficult to implement, maybe a kinda replace footer route after merging ?

Truth is, I'm not sure that's even doable. A footer is not a "standard" feature of a PDF, so I don't think automation is possible here 🤔

Closing this as there is no obvious or straightforward solution. Feel free to reopen if anything new!


yaml/json file backend

I've createt a simple file based yaml/json backend #361
It is working with json files:
{
    "MYAPP_DATABASE_URL": "db.example.com",
    "MYAPP_DATABASE_USER": "rob",
}

and yaml files:

---
  MYAPP_DATABASE_URL: "db.example.com"
  MYAPP_DATABASE_USER: "robby"

It has full watch support.
./confd -backend yaml -yaml-file test.yaml -watch

This change is

@HeavyHorst This is a good start. The questions I have are what is the format for the key names. My guess is people will want to use any key name they want. So we should support
---
foo: bar
test_foo: bar

Maybe we should run this by the person who file the original issue. But so far so good.

@HeavyHorst I'll be able to take a deeper review later this evening, but so far the use of fsnotify feels right.

@kelseyhightower it should now be possible to choose any key name.

Looking good, works as advertised and provides an extremely light way for people to store data.
Cosmetic issue:
2016-02-24T20:12:11-08:00 ireul.local bin/confd[13853]: INFO Backend set to yaml
2016-02-24T20:12:11-08:00 ireul.local bin/confd[13853]: INFO Starting confd
2016-02-24T20:12:11-08:00 ireul.local bin/confd[13853]: INFO Backend nodes set to

Would it be possible to change the text "Backend nodes set to" to "Backend source(s) set to", and to put the path of the json/yaml file afterwards?
User Experience:
It looks like you can pass in a JSON or YAML file, but the command line makes it seem like it only takes YAML. What do you think of changing the backend name to something like "file", and -yaml-file argument to -file?
Other Stuff:
I was only able to get flat data to work, deep structures ended up with this:
2016-02-24T20:45:00-08:00 ireul.local bin/confd[14332]: ERROR yaml: unmarshal errors:
  line 2: cannot unmarshal !!map into string

I used this the data.
---
myapp:
  database:
    url: url1
    user: user1

This might be fine if it's the desired behavior, since using multiple files can approximate the functionality. Downside is that it will require that all templates/configs be changed to remove nested data.

That is a limitation. For example in our use case we use etcd like so:
/deis/database/hosts/<IP_ADDRESS>
/deis/database/hosts/<IP_ADDRESS>
/deis/database/hosts/<IP_ADDRESS>
/deis/database/hosts/<IP_ADDRESS>

So if it was in file form I'd imagine it to work something like
deis:
  database:
    hosts:
      - <IP_ADDRESS>
      - <IP_ADDRESS>
      - <IP_ADDRESS>
      - <IP_ADDRESS>

To replicate the same behaviour.

Would you also mind adding integration tests inside integration/file as well so we can validate this? Thanks!

test file:
deis:
  database:
    hosts:
      - <IP_ADDRESS>: test1
      - <IP_ADDRESS>
      - <IP_ADDRESS>
      - <IP_ADDRESS>

myapp:
  database: {url: www.google.de, user: Boris}

Map: map[string]string{"/myapp/database/url":"www.google.de", "/myapp/database/user":"Boris", "/deis/database/hosts/<IP_ADDRESS>":"test1", "/deis/database/hosts/<IP_ADDRESS>":"", "/deis/database/hosts/<IP_ADDRESS>":"", "/deis/database/hosts/<IP_ADDRESS>":""}

I will add the integration tests soon.
thanks for the feedback !

LGTM! Awesome work. :)

LGTM, Rockin!

+1 (when is next release, and is this included?)

@kelseyhightower Thank you for creating confd, I'm working on a project where this is exactly the tool we needed.
@HeavyHorst +1
When will this be merged / released in next stable version? Looking to use this feature.

Looking forward to see this in upstream!

@kelseyhightower is this PR ready to merge?
/cc @HeavyHorst

I've gone ahead and made a v0.12.1 release on my fork that includes this change. See the v0.12.0 and v0.12.1 release notes.

This should be fixed and merged


Revert "[build-script-impl] Add support for building benchmarks on linux but disable it in all currently defined presets for linux."

Reverts apple/swift#7860
This broke the OSS - LLDB incremental linux bots.

@swift-ci please smoke test

Thanks!

On Mar 6, 2017, at 9:39 AM, Ben Langmuir<EMAIL_ADDRESS>wrote:
Merged #7931 https://github.com/apple/swift/pull/7931.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/apple/swift/pull/7931#event-988152162, or mute the thread https://github.com/notifications/unsubscribe-auth/AAee35h43uQLRgkYGnYc6h8N_V9HrZl-ks5rjETagaJpZM4MUZy7.


Drawscript

@jwittenbach As I am still drawscriptless, when you get a chance could you export the new curves to code and send it to me? I'll take care of putting them into the json file.

@bcoke Happy to!
shapes_raw.txt

@jwittenbach Great! Thanks!


System.CalloutException: Web service callout failed: Unable to parse callout response. Apex type not found for element includeRecordsOwnedByAll

While fetching sharing criteria Rules I'm getting error System.CalloutException: Web service callout failed: Unable to parse callout response. Apex type not found for element includeRecordsOwnedByAll
MetadataService.MetadataPort service = new MetadataService.MetadataPort();
service.SessionHeader = new MetadataService.SessionHeader_element();
service.SessionHeader.sessionId = UserInfo.getSessionId();
MetadataService.SharingCriteriaRule sh=(MetadataService.SharingCriteriaRule)service.readMetadata('SharingCriteriaRule', new String[]{'SharingRuleName'}).getRecords()[0];

I also faced this same exception. Looks like a new element was added to SharingCriteriaRule by Salesforce that hasn't been addressed in the SharingCriteriaRule subclass.


Variables no longer seem to work with inline chat

Copilot Chat Extension Version: v0.22.2024102201 (pre-release)
VS Code Version: 1.95.0-insider Commit: defe52dc43e6f5d861f55600ff01f7553fac7eaf
OS Version: macos 15.0.1

Steps to Reproduce:

Open Editor inline chat
Type #
See no variable suggestions

Previously, variable suggestions were provided. I noticed sometimes a pre-release was broken before and didn't include them, but usually this was only broken for a release or two. Now, it has been several.
In case, this is intentional, I would like to recommend, the editor inline chat continues to have an option to use variables inline with text. This enables the developer to write more natural instructions. For instance: Write an implementation for the interface defined in #file

Fixed in next insiders/stable

Tested this out (v0.22.2024102304 (pre-release))  and it does seem fixed.
However, I can only see suggestions for: codebase, terminalLastCommand, terminalSelection and vscodeAPI. I cannot select a file like I can in the sidebar chat?

@jrieken Can you confirm that the #file variable works? Tested out the new release: v0.22.2024102401 (pre-release) and I still only see the options mentioned above.

I cannot select a file like I can in the sidebar chat?

We don't have the 📎 button but you should be able to type # followed by a file name, like #app.js and intellisense should suggest files that are from your workspace. Do you not see that?

When I left my previous comment, I didn't see this. I just updated VSCode-Insiders and now I can!


Verify "official" multisend calls

On the frontend, we've been checking if a contract call is an official multisend deployment call by looking up the contract address in safe-deployments.
As @rmeissner pointed out, a client shouldn't be doing this logic. It would be good to add some sort of flag like isMultisend for txs targetting our official multisend deployments.
Currently, we seem to check only the method name when decoding things like actionCount.
References
Project docs: https://www.notion.so/safe-global/Verifying-decoded-multi-sends-b9966297978b443ea9dda704beb8f4b3
(Need feedback from mobile)

Closing because apparently this logic existed only in the web app and was added not for security but just to not ignore unsupported data shapes.


Fix url appending logic

Recently we received an ICM issue. After investigating we found when we sent the next page request we would append the $filter twice. This is because we compare the search param without considering the encoding. And we don't know that the $filter and %24filter are the same filter.
https://management.azure.com/subscriptions/subscription-id/resources?api-version=2021-04-01&%24filter=ResourceType+eq+'Microsoft.OperationsManagement%2Fsolutions'&%24top=1&%24skiptoken=Reacted
In this fix we include the both to do comparation.

API change check
API changes are not detected in this pull request.

I think the change is fine, but I'm curious how we were getting two versions of this query parameter. Is the API paging with a nextLink with an encoded param name and we were appending the query param again?
If so we should be able to resolve this by simply regenerating the affected libraries as we no longer process query parameters in nextLink-based operations.

Yes this is the case, and the impacted SDK is arm-resoureces. I'll regenerate the SDK again to verify if still existing!

The issue could be resolved after we refresh to the latest codege, I will close the pr.


Remove console.log

Description
Removes an unnecessary console.log introduced in https://github.com/material-table-core/core/commit/a8644bc31acf3ff26af59d357bea1cddaa839ec8
Impacted Areas in Application
MTableToolbar

TY!


feat: redesign of the alert component

The alert received  a slight redesign to align with the notification. With this story, it should be fixed according to Figma.

Closed by #1891


Demonstrate issue with large nokogiri documents

Large nokogiri documents expand to even larger when pp'd, and this produces very complex diffs that take a long time to produce, I'm not sure if we have a solution for this but it's the source of rspec/rspec-expectations#773

@JonRowe I don't see any activities on this issue since 4 years. I am in favor of closing it. What do you think?

@benoittgt this is a bit confusing because of the rebase, but its worth leaving open as its an issue rather than a stale PR


Support linking xcframework

Modern iOS libs ship as *.xcframework artefacts which are capable of targeting multiple platforms.
I am attempting to bind the cossacklabs/themis lib built by Carthage (also tried their xcframework creation scripts)
https://github.com/cossacklabs/themis
And my linker error is:
Building target "_LinkNativeExecutable" completely.
        Tool xcrun execution finished (exit code = 1).

        Undefined symbols for architecture arm64:
          "_OBJC_CLASS_$_TSCell", referenced from:
              objc-class-ref in registrar.o
          "_OBJC_CLASS_$_TSCellContextImprint", referenced from:
              objc-class-ref in registrar.o
          "_OBJC_CLASS_$_TSCellSeal", referenced from:
              objc-class-ref in registrar.o
          "_OBJC_CLASS_$_TSCellToken", referenced from:
              objc-class-ref in registrar.o
          "_OBJC_CLASS_$_TSCellTokenEncryptedData", referenced from:
              objc-class-ref in registrar.o
          "_OBJC_CLASS_$_TSCellTokenEncryptedResult", referenced from:
              objc-class-ref in registrar.o
          "_OBJC_CLASS_$_TSComparator", referenced from:
              objc-class-ref in registrar.o
          "_OBJC_CLASS_$_TSKeyGen", referenced from:
              objc-class-ref in registrar.o
          "_OBJC_CLASS_$_TSMessage", referenced from:
              objc-class-ref in registrar.o
          "_OBJC_CLASS_$_TSSession", referenced from:
              objc-class-ref in registrar.o
          "_OBJC_CLASS_$_TSSessionTransportInterface", referenced from:
              objc-class-ref in registrar.o
          "_TSGenerateSymmetricKey", referenced from:
             -u command line option
        ld: symbol(s) not found for architecture arm64
        clang: error: linker command failed with exit code 1 (use -v to see invocation)

Combining the dynamic libs of the framework to a single binary with lipo create seems like a flawed approach since both iPhone and Mac desktop might be using the same M1 chips in future (with slightly different API and different lib artefacts)


Ensure top-level permissions are not set to write-all

Check: CKV2_GHA_1: "Ensure top-level permissions are not set to write-all"
  	FAILED for resource: on(drawio-export-action)
  	File: /.github/workflows/drawio-export-action.yml:0-1
Check: CKV2_GHA_1: "Ensure top-level permissions are not set to write-all"
  	FAILED for resource: on(Lint Code Base)
  	File: /.github/workflows/linter.yml:10-11

:tada: This issue has been resolved in version 2.25.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Path to nginx and lighttpd sample config files -- error in INSTALL.txt

In line 77, install/INSTALL.txt says sample config files for nginx and lighttpd are in doc/install. But, they actually are in install/sample-nginx.conf and install/sample-lighttpd.conf respectively.

They used to be in doc/install, but were moved because it made more sense to keep them with the rest of the install related stuff. I've updated the text. Thanks.

Yeah, I thought so. Thank you! :-)


Make menu look more like minecraft ?

The current menu looks pretty nice, but I wonder if people have a clue what it is about if first coming there.
Maybe we can put the prismarine logo, the green color ?
Maybe some short explanation on what this is ?

We could render a cube map of the terrain with pviewer using https://threejs.org/docs/#api/en/cameras/CubeCamera so we could use it as a rotating background for the menu like vanilla minecraft.
And since the minecraft title screen seed has been found it could be made almost identical https://www.pcgamer.com/minecraft-title-screen-seed/
I think it would be funny.

Here is the render of the original title screen scene in pviewer https://github.com/PrismarineJS/prismarine-web-client/blob/master/assets/title.jpg
Generated using https://github.com/spite/THREE.CubemapToEquirectangular
And here is how to use it: https://threejs.org/examples/webgl_panorama_equirectangular.html

done, thanks @Karang


Building documentation for two framework versions fails (Support multiple simultaneous framework versions)

Our code library contains the same code across projects, but there are two project libraries loaded, one for .NET 4 and one for .NET 4.5. There are some issues with this when sharpDox loads the solution:

The type list contains two copies of the same classes with no way to distinguish them
Excluding one class (e.g. the one from .NET 4.0) excludes it from .NET 4.5 also, once the exclusion list is reloaded
Building the documentation with both loaded gives me the following error:

Starting step: "Parsing project file"
Starting step: "Parsing code"
Could not build documentation.
System.InvalidOperationException: Sequence contains more than one element
   at System.Linq.Enumerable.Single[TSource](IEnumerable`1 source)
   at SharpDox.Build.Roslyn.Parser.ProjectParser.MethodCallParser.ParseMethodCalls()
   at SharpDox.Build.Roslyn.RoslynParser.ParseMethodCalls(ParserOptions parserOptions)
   at SharpDox.Build.Roslyn.RoslynParser.GetParsedSolution(String solutionFile, ICoreConfigSection sharpDoxConfig, Dictionary`2 tokens, Boolean parseMethodCalls, Boolean ignoreExcludes)
   at SharpDox.Build.Context.Step.ParseCodeStep.RunStep(SDProject sdProject)
   at SharpDox.Build.Context.BuildContext.StartBuild()

Here is a sample of the structure we have:

MyLibrary-net4

MyClass

MyMethod()

MyLibrary-net45

MyClass

MyMethod()

Since you are only using fully qualified method names as "unique names", if two projects are loaded with the same code, it will fail, because that .Single(...) call inside of ParseMethodCalls() is expecting one result, but instead it finds two identical ones:

MyLibrary.MyClass.MyMethod() [net4]
MyLibrary.MyClass.MyMethod() [net45]

I think the fix here is to prepend the Framework version to the exclusion/inclusion list, or at the very least, prepend the project name (project names are guaranteed to be unique within a solution IIRC).
The workaround was to create another solution with only one set of projects in it, so that code was not duplicated. Not ideal, though - I would like to use my main solution instead.

This seems like a bug. sharpDox should know the difference between the different framework version. I will look into it as soon as possible.


popover isn't showing

I'm using bootstrap 3.3 and I can't get the popover element to show up, I get no errors just blank or if I use backdrop just focus on the first step.
Is this plugin bootstrap 3 compatible? I do have the popover plugin as I did a test popover snippet to test.
Thanks

I found a related issue that mentioned how to fix the problem. I got it working now.


feat(Shape): add component

WIP
Fixes #196.

Codecov Report

Merging #1751 into master will decrease coverage by 0.36%.
The diff coverage is 30.76%.

@@            Coverage Diff             @@
##           master    #1751      +/-   ##
==========================================
- Coverage   99.75%   99.39%   -0.37%
==========================================
  Files         142      144       +2
  Lines        2466     2481      +15
==========================================
+ Hits         2460     2466       +6
- Misses          6       15       +9

Impacted Files
Coverage Δ

src/modules/Shape/Shape.js
16.66% <16.66%> (ø)

src/modules/Shape/ShapeSide.js
42.85% <42.85%> (ø)

src/modules/Search/Search.js
100% <0%> (ø)
:arrow_up:

src/addons/TextArea/TextArea.js
100% <0%> (ø)
: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 91c3301...e2e3442. Read the comment docs.

How is this going ? Can we do something to help ?  Help pass the test or something else ?

Without progress as you can see, I have less time now while there are PRs with more priority. Help there is much appreciated.

Completely understand, i'll have a closer look and see. Thank you very much for your time.

Any news on Shape component? If it gets added, would be highly appreciated!

Please feel free to hop in!  Fork the repo, pull the branch and submit a PR to finish this one up.  That would be a huge help 😃

Closing due inactive, feel free to pick up it!


Multiple GPUs Encoder Decoder

If we use >1 gpu there is an error:
return type(out)(map(gather_map, zip(*outputs)))
TypeError: zip argument #1 must support iteration

@ibeltagy if i install pytorch-lightning-1.0.3, it works. I will try to train model and check correctness of result

I am using my own fork of PL https://github.com/allenai/longformer/blob/encoderdecoder/requirements.txt#L2. PL is full of bugs and I don't know which release to trust.

hey @ibeltagy! we've recently released PyTorch Lightning v1.0 with a bunch of stability fixes, and are releasing minor version releases to handle edge cases fixes/build towards new features. I'll make a PR to propose upgrading to the stable release!


cbgetlpsolution returns assertionerror

I'm trying to collect some info using an infocallback illustrated in the callback section of the JuMP document. When I try to access the fractional lp solution at the root node using cbgetlpsolution within the infocallback function, I get an assertion error. This is the function I use to add the infocallback to the model.
addinfocallback(mdl, infocallback, when = :Intermediate)
The objective is to access the fractional solution similar to this. How can I achieve this in JuMP?

Can you provide a complete exmaple that demonstrates the problem? See https://stackoverflow.com/help/mcve

A simple MWE (based on knapsack.jl) of the problem I have. I want to get the fractional LP value of variable x at the root node. However, I get a warning here: Variable value not defined for component of x. Check that the model was solved properly
using JuMP
using CPLEX

type NodeData
  time::Float64  # in seconds since the epoch
  node::Int
  obj::Float64
  bestbound::Float64
  x_val::Array{Float64,1}
end

m = Model(solver=CplexSolver(CPX_PARAM_NODELIM = 0))

@variable(m, x[1:5], Bin)

profit = [ 5, 3, 2, 7, 4 ]
weight = [ 2, 8, 4, 2, 5 ]
capacity = 10

# Objective: maximize profit
@objective(m, Max, dot(profit, x))

# Constraint: can carry all
@constraint(m, dot(weight, x) <= capacity)

bbdata = NodeData[]

function infocallback(cb)
  node      = MathProgBase.cbgetexplorednodes(cb)
  obj       = MathProgBase.cbgetobj(cb)
  bestbound = MathProgBase.cbgetbestbound(cb)
  x_val = getvalue(x)
  push!(bbdata, NodeData(time(),node,obj,bestbound, x_val))
end
addinfocallback(m, infocallback, when = :Intermediate)

solve(m)

I haven't used the callbacks in CPLEX, however it looks like the solution is only available for when=:MIPnode for the infocallback.
https://github.com/JuliaOpt/JuMP.jl/blob/37ab6795eaa269bb762eb11dbba350eb8379bfb1/src/callbacks.jl#L170
You could try addlazycallback(m, infocallback) instead.
The new version of JuMP will not support solver independent callbacks. The behaviour is too solver-dependent.

I haven't used the callbacks in CPLEX, however it looks like the solution is only available for when=:MIPnode for the infocallback

Is this behaviour independent of the solver? I mean if I switch to another solver, lets say Gurobi, the solution would still not be available?

Presumably you're trying to follow https://discourse.julialang.org/t/accessing-the-root-node-solution/10371/7?
This is not a CPLEX.jl issue. The infocallback only supports querying the LP solution in the following states
https://github.com/JuliaOpt/CPLEX.jl/blob/81d7dbb9947960f56c81f7b32dcdf612fa99f7ba/src/CplexSolverInterface.jl#L373-L374
The variable values are not available for when=:Intermediate.
I'm not sure what the story with Gurobi is, but according to
https://github.com/JuliaOpt/JuMP.jl/blob/37ab6795eaa269bb762eb11dbba350eb8379bfb1/src/callbacks.jl#L165-L170
the solution will not be available for when=:Intermediate.

Closing this since it relates to the old JuMP callbacks, which we won't be fixing. The new MOI wrapper implements solver-independent MOI callbacks, and provides access to solver-specific generic CPLEX callbacks.


Linux setup instructions outdated

The instructions for the Linux setup seem to be pretty old. Could you provide new ones?
Currently, the referenced linux/app_configuration.mk file is no longer used and though the plugin no longer works for Linux throwing a MissingPluginException...
Anyway, thank you for this great plugin!

Hi @TheOneWithTheBraid for linux you should use version 1.0.0-rc1 which not require any additional changes to work
https://pub.dev/packages/openpgp/versions/1.0.0-rc1


Improve diffing of values: Concrete Types

In our ongoing effort to improve the accuracy and reliability of our interface diffing tool, we need to ensure that our implementation can correctly identify modifications to values that reference concrete types (records or variants). At present, our tool may fail to detect certain modifications or incorrectly flag identical concrete types as different.
For example:
type t = { a: int; b: string }
val x : t

If we pass this same interface given above form two different cmi, they should be treated as identical and no modification should be detected in val x.
Only when we have modified the record defined in type t like in the interface given below, should a change in val f be detected.
type t = { a: int; b: float }
val x : t

Similarly in case of variants, when the interface given below is passed from two different cmi, no change should be reported for the val x.
type t = A | B
val x : t

This issue will address these shortcomings by adding relevant test cases and updating the implementation accordingly to make the added tests pass.

Only when we have modified the record defined in type t like in the interface given below, should a change in val f be detected.

I think in this specific case, that might not be true. We decided to mark the change if the declared type is just an alias because there are other legitimate ways to build values of those type, e.g. if somewhere one defines an exposed alias such as type t = int list and provide values and functions referencing the type t, they could be legitimately tempted to use it on arbitrary lists of integers.
Here, given the type is defined in this module, I don't think we should mark values referencing it as modified (unless they've been otherwise modified) since the type does not exist outside of this module, as opposed to lists of integers for instance.
I think in that case, an external user of the library defining t that updates to this new version that modifies the definition of t would have to fix pattern matchings over values of type t and "literals" of type t and that would be enough of a fix, they would not have to fix a call to a val f : t -> string function for example. What I'm trying to say is that in this case, from a library user perspective, x or the aforementioned f would not have changed.
Am I making sense here? I'm finding this harder to describe than I'd hoped.

Also to clarify, I think the expected semantic would be that considering the following two interfaces:
type t = {a : int; b : string}
val x : t

and
type t = {a : int; b : float}
val x : t

we should report a change in type t but no change for val x.

Okay @NathanReb, I think I understand what you mean! I see your point regarding the distinction between type aliases and defined types within the module. Thanks for providing such an insightful explanation!
I had a follow up question: If we want to detect just the changes in the type definitions in these cases mentioned above, that would be dealt when we are writing implementation for detecting type changes right? For now, we will just focus on not detecting the value changes from a variant or record change, am I correct?

Yes exactly, the change in the type will simply by marked as [Any] and indeed we're testing that no value change is reported!


[storage][doc] remove Special bundling notes for IE11

Following the bundling guide: https://github.com/Azure/azure-sdk-for-js/tree/master/sdk/storage/storage-blob#special-bundling-notes-for-ie11, running
tsc --allowJS --target es5 bundled-output.js --outfile final-output.js

gave "Unused '@ts-expect-error' directive" error for this line https://github.com/Azure/azure-sdk-for-js/blob/f169ccaee02ff59d0a3589a677272d0732a59a0d/sdk/storage/storage-file-share/src/utils/utils.common.ts#L385
We added this note because the open telemetry library we depend on is not compliled for es5. So we have to recompile it.
But now the bundling seems to work without this step in IE11.
I don't know why we don't have this issue before. Probably not important anyway.
I think we can get rid of the exception note since the old issue is fixed in https://github.com/Azure/azure-sdk-for-js/pull/10393 and the step is no longer needed. @ramya-rao-a @jeremymeng @xirzec @HarshaNalluru

cc @xirzec


added truncated normal distribution

Oy. I did not check! I'll do the test.
Sent from my iPhone

On 12 Mar 2017, at 21:30, Giacomo Vianello<EMAIL_ADDRESS>wrote:
@giacomov commented on this pull request.
In astromodels/functions/functions.py:

   self.mu.unit = x_unit

   # The lower_bound has the same dimensions as the x

   self.lower_bound.unit = x_unit

   # The upper_bound has the same dimensions as the x

   self.upper_bound.unit = x_unit

   # sigma has the same dimensions as x

   self.sigma.unit = x_unit

noinspection PyPep8Naming

def evaluate(self, x, F, mu, sigma, lower_bound, upper_bound):
thanks for this addition!

Is this faster than using scipy.stats.truncnorm?
https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.truncnorm.html
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@giacomov yes, it is a few microseconds faster on average :)


Dashboard timesheet updates

[x] Cleanup/remove clients/projects/charge codes from get dashboard time response

[ ] We may need to keep projects/activities so we can populate the activities dropdowns however, but since we switched to autocomplete most of these I don't believe are being used, if we need client/project names we can add to get charge code response and pull from the selected charge code instead

[ ] Move charge code into staff dashboard service as shareReplayed observable
[ ] Duplicate time show date modal and let staff enter new date, add min date validator for staff cutoff date
[ ] Need a Reset button that clears search, sets period to Day and date to Today

Remove Clients/Projects/Activities/ChargeCodes from GetDashboardTimeV1
Make ProjectId nulllable on GetProjectActivitiesV1 (so we can filter to it based on the projectId)
Add ProjectId to the GetProjectActivitiesV1 record model (So we can filter client side)


Links should (probably) be sorted alphabetical order by key

It would make it easier to locate keys when the list gets longer.

It already works like that!


GEOPY-1781: tests of las-goh5 fail with geoh5 0.10.0

GEOPY-1781 - tests of las-goh5 fail with geoh5 0.10.0

relates to https://github.com/MiraGeoscience/geoh5py/pull/638


add entry for 'vchw'

Amendment 1 of ISO/IEC 14496-22:2019 adds the 'vchw' feature with this description:
Friendly name: Vertical Contextual Half-width Spacing
Registered by: Adobe/W3C
Function: Contextually respaces glyphs designed to be set on full-em heights, fitting them onto individual
half-width vertical heights, to approximate more sophisticated text layout, such as what is described
in Requirements for Japanese Text Layout (JLREQ [21]) or similar CJK text-layout specifications that
expect half-width forms of characters whose default glyphs are full-width. This differs from 'vhal' in
that the respacing is contextual. This feature may be invoked to get better fit for punctuation or symbol
glyphs without disrupting the monospaced alignment.
Example: When PRESENTATION FORM FOR VERTICAL RIGHT PARENTHESIS (U+FE36; "︶", vertical
form of FULLWIDTH RIGHT PARENTHESIS U+FF09; ")") is followed by PRESENTATION FORM FOR
VERTICAL IDEOGRAPHIC COMMA (U+FE11; "︑", vertical form of IDEOGRAPHIC COMMA U+3001; "、"),
the latter is respaced to remove half-em of height between them.
Recommended implementation: The font stores a set of adjustments for pairs of glyphs (GPOS lookup type
2 or 8). These may be stored as one or more tables matching left and right classes, &/or as individual
pairs. Additional adjustments may be provided for larger sets of glyphs (e.g. triplets, quadruplets, etc.)
to overwrite the results of pair kerns in particular combinations.
Application interface: The application passes a sequence of GIDs to the 'vchw' table, and gets back
adjusted positions (XPlacement, XAdvance, YPlacement, and YAdvance) for those GIDs. When using the
type 2 lookup on a run of glyphs, it’s critical to remember to not consume the last glyph, but to keep it
available as the first glyph in a subsequent run (this is a departure from normal lookup behavior).
UI suggestion: This feature would be off by default.
Script/language sensitivity: Used mostly in CJKV fonts.
Feature interaction: This feature is mutually exclusive with all other glyph-height features (e.g., 'valt',
'vhal', 'vpal'), which

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

ID: dd547c7a-0679-c1f5-9bd8-6e0249218aca
Version Independent ID: 38e91354-2503-9b0c-c0da-a0cd808e1e35
Content: Registered features, u-z - Typography
Content Source: typographydocs/opentype/spec/features_uz.md
Product: typography
GitHub Login: @PeterCon
Microsoft Alias: PeterCon

Cf #535

Investigate this part further:
"it’s critical to remember to not consume the last glyph, but to keep it
available as the first glyph in a subsequent run (this is a departure from normal lookup behaviour)."

Changed for next version.

See my note in https://github.com/MicrosoftDocs/typography-issues/issues/536 on the example in the description ("former" vs "latter"). It applies here too.

Based on feedback and additional info from Koji Ishi, I've revised the draft as follows:

Friendly name: Vertical Contextual Half-width Spacing
Registered by: Adobe/W3C
Function: Contextually re-spaces glyphs designed to be set on full-em heights, fitting them onto individual half-width vertical heights, to approximate more sophisticated text layout, such as what is described in [Requirements for Japanese Text Layout (JLREQ)][1] or similar CJK text-layout specifications that expect half-width forms of characters whose default glyphs are full-width.* This differs from 'vhal' in that the re-spacing is contextual. This feature may be invoked to get better fit for punctuation or symbol glyphs without disrupting the monospaced alignment.
* In JLREQ, see, in particular, [3.1.4 Positioning of Consecutive Opening Brackets, Closing Brackets, Commas, Full Stops and Middle Dots][2] and [B. Spacing between Characters][3]. See also [Requirements for Chinese Text Layout (CLREQ)][4], [Requirements for Hangul Text Layout (KLREQ)][5].
Example: When PRESENTATION FORM FOR VERTICAL RIGHT PARENTHESIS (U+FE36; “︶”, vertical form of FULLWIDTH RIGHT PARENTHESIS U+FF09; “)”) is followed by PRESENTATION FORM FOR VERTICAL IDEOGRAPHIC COMMA (U+FE11; “︑”, vertical form of IDEOGRAPHIC COMMA U+3001; “、”), the former is re-spaced to remove half-em of height between them.
Recommended implementation: The font stores a set of adjustments for pairs of glyphs (GPOS lookup type 2 or 8, XPlacement, XAdvance, YPlacement, and YAdvance). These may be stored as one or more tables matching left and right classes, and/or as individual pairs. Additional adjustments may be provided for larger sets of glyphs (e.g. triplets, quadruplets, etc.) to overwrite the results of pair kerns in particular combinations.

Note: When using a GPOS type 2 lookup for this feature, it is recommended that no positioning adjustment be applied to the second glyph in a pair. (That is, that valueFormat2 be set to 0.) As a result, the next glyph pair to be processed after the lookup has been applied to this pair will start at the second glyph. In this way, every glyph in a sequence can undergo positioning adjustment as the first glyph in a pair. Otherwise, if positioning of the second glyph is adjusted, then the next glyph pair to be processed will begin with the glyph following the second glyph. That will result in spacing adjustment happening only for every other pair of glyphs.

Application interface: If a layout engine supports advanced layout for CJK text as described in [CLREQ][4], [JLREQ][1] or [KLREQ][5], this feature should not be used. Otherwise, this feature should always be applied in vertical layout of CJK text.
UI suggestion: This feature should not be used in combination with a layout engine that independently provides advanced layout as described in [CLREQ][4], [JLREQ][1] or [KLREQ][5]. For applications that provide such advanced layout, it may appropriate not to expose control of this feature to users. In applications that do not support such advanced layout, this feature should be enabled by default for vertical layout of CJK text.
Script/language sensitivity: Used mostly in CJKV fonts.
Feature interaction: This feature is mutually exclusive with all other glyph-height features (e.g., 'valt', 'vhal', 'vpal'), which should be turned off when this feature is applied. It deactivates the 'vkrn' feature. See also 'chws'.


flutter_gallery__transition_perf regression

flutter_gallery__transition_perf average_frame_build_time_millis regressed after https://github.com/flutter/flutter/pull/12058.
cc @a-siva

https://github.com/flutter/flutter/commit/03121ffa79f1bd3266221f91d383e10d3baa858e


Sorry.. the Pirateweather forecast is not available.

Hi, I'm running Pw within a WordPress site. Pw files have been uploaded and edited using a WordPress plugin "Advanced file manager" which is a bit like a Cpanel utility. I've run into 2 issues.

I've changed the list of towns but it doesn't show on the website; it still shows the default list
the above message ("Sorry.. the Pirateweather forecast is not available.") has now been showing. Is this a cache related issue (as maybe the first issue is?)

Sorry this is regarding the stand alone application found at saratoga-weather.org

Thanks for clarifying the issue here! It sounds like you've got it sorted out, so I'm going to close for now, but let me know if anything else comes up