[cxx-interop] Referencing Swift class from C++ causes a include cycle and crashes the compiler

Description
I'm trying to call a C++ function that takes a Swift reference as an argument. This causes an include cycle and crashes the compiler.
// SwiftClass.swift

public class SwiftClass {
  public func sayHi() {
    print("Hello from Swift!")
  }
}

// Dummy.hpp

#pragma once
#include "MyApp-Swift.h"

namespace cpp {
void useSwiftClass(MyApp::SwiftClass swiftClass);
}

// Dummy.cpp

#include "Dummy.hpp"

namespace cpp {
void useSwiftClass(MyApp::SwiftClass swiftClass) {
  swiftClass.sayHi(); // <-- call Swift func
}
}

// Dummy.swift

func callToCpp() {
  let swiftClass = SwiftClass()
  cpp.useSwiftClass(swiftClass) // <-- call C++ func
}

The problem lies in the generated MyApp-Swift.h header, because it doesn't generate useSwiftClass anymore.
Reproduction
see code above
Stack dump
PrecompileSwiftBridgingHeader normal arm64 (in target 'SwiftCxxPlayground' from project 'SwiftCxxPlayground')
    cd /Users/mrousavy/Projects/SwiftCxxPlayground
    /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/swift-frontend -target arm64-apple-ios18.0-simulator -Xllvm -aarch64-use-tbi -enable-objc-interop -cxx-interoperability-mode\=default -sdk /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator18.0.sdk -I /Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Products/Debug-iphonesimulator -F /Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Products/Debug-iphonesimulator -no-color-diagnostics -enable-testing -g -debug-info-format\=dwarf -dwarf-version\=5 -module-cache-path /Users/mrousavy/Library/Developer/Xcode/DerivedData/ModuleCache.noindex -swift-version 5 -enforce-exclusivity\=checked -Onone -D DEBUG -serialize-debugging-options -const-gather-protocols-file /Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Intermediates.noindex/SwiftCxxPlayground.build/Debug-iphonesimulator/SwiftCxxPlayground.build/Objects-normal/arm64/SwiftCxxPlayground_const_extract_protocols.json -enable-experimental-feature DebugDescriptionMacro -enable-experimental-feature OpaqueTypeErasure -enable-bare-slash-regex -empty-abi-descriptor -validate-clang-modules-once -clang-build-session-file /Users/mrousavy/Library/Developer/Xcode/DerivedData/ModuleCache.noindex/Session.modulevalidation -Xcc -working-directory -Xcc /Users/mrousavy/Projects/SwiftCxxPlayground -resource-dir /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift -enable-anonymous-context-mangled-names -file-compilation-dir /Users/mrousavy/Projects/SwiftCxxPlayground -Xcc -std\=gnu++20 -Xcc -D_LIBCPP_HARDENING_MODE\=_LIBCPP_HARDENING_MODE_DEBUG -Xcc -ivfsstatcache -Xcc /Users/mrousavy/Library/Developer/Xcode/DerivedData/SDKStatCaches.noindex/iphonesimulator18.0-22A3362-db63dc9361471f152f572502bdbfe70a.sdkstatcache -Xcc -I/Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Intermediates.noindex/SwiftCxxPlayground.build/Debug-iphonesimulator/SwiftCxxPlayground.build/swift-overrides.hmap -Xcc -iquote -Xcc /Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Intermediates.noindex/SwiftCxxPlayground.build/Debug-iphonesimulator/SwiftCxxPlayground.build/SwiftCxxPlayground-generated-files.hmap -Xcc -I/Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Intermediates.noindex/SwiftCxxPlayground.build/Debug-iphonesimulator/SwiftCxxPlayground.build/SwiftCxxPlayground-own-target-headers.hmap -Xcc -I/Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Intermediates.noindex/SwiftCxxPlayground.build/Debug-iphonesimulator/SwiftCxxPlayground.build/SwiftCxxPlayground-all-target-headers.hmap -Xcc -iquote -Xcc /Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Intermediates.noindex/SwiftCxxPlayground.build/Debug-iphonesimulator/SwiftCxxPlayground.build/SwiftCxxPlayground-project-headers.hmap -Xcc -I/Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Products/Debug-iphonesimulator/include -Xcc -I/Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Intermediates.noindex/SwiftCxxPlayground.build/Debug-iphonesimulator/SwiftCxxPlayground.build/DerivedSources-normal/arm64 -Xcc -I/Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Intermediates.noindex/SwiftCxxPlayground.build/Debug-iphonesimulator/SwiftCxxPlayground.build/DerivedSources/arm64 -Xcc -I/Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Intermediates.noindex/SwiftCxxPlayground.build/Debug-iphonesimulator/SwiftCxxPlayground.build/DerivedSources -Xcc -DDEBUG\=1 -import-objc-header /Users/mrousavy/Projects/SwiftCxxPlayground/Bridge.h -module-name SwiftCxxPlayground -frontend-parseable-output -disable-clang-spi -target-sdk-version 18.0 -target-sdk-name iphonesimulator18.0 -external-plugin-path /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/usr/lib/swift/host/plugins\#/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/usr/bin/swift-plugin-server -external-plugin-path /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/usr/local/lib/swift/host/plugins\#/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/usr/bin/swift-plugin-server -plugin-path /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift/host/plugins -plugin-path /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/local/lib/swift/host/plugins -index-store-path /Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Index.noindex/DataStore -emit-pch -serialize-diagnostics-path /Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Intermediates.noindex/PrecompiledHeaders/Bridge-26rwerqus5m3e.dia -pch-output-dir /Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Intermediates.noindex/PrecompiledHeaders /Users/mrousavy/Projects/SwiftCxxPlayground/Bridge.h

/Users/mrousavy/Projects/SwiftCxxPlayground/Bridge.h:8:10: note: in file included from /Users/mrousavy/Projects/SwiftCxxPlayground/Bridge.h:8:
#include "Dummy.hpp"
         ^
/Users/mrousavy/Projects/SwiftCxxPlayground/SwiftCxxPlayground/Dummy.hpp:16:40: error: no type named 'SomeClass' in namespace 'SwiftCxxPlayground'
void useSwiftClass(SwiftCxxPlayground::SomeClass swiftPart);
                                       ^
<unknown>:0: error: failed to emit precompiled header '/Users/mrousavy/Library/Developer/Xcode/DerivedData/SwiftCxxPlayground-bxhzaumzxkpricbyzwtutaumuopz/Build/Intermediates.noindex/PrecompiledHeaders/Bridge-swift_1E7HDKVNFKPAD-clang_7W2O1TJ10F43.pch' for bridging header '/Users/mrousavy/Projects/SwiftCxxPlayground/Bridge.h'
/Users/mrousavy/Projects/SwiftCxxPlayground/Bridge.h:8:10: note: in file included from /Users/mrousavy/Projects/SwiftCxxPlayground/Bridge.h:8:
#include "Dummy.hpp"
         ^

Expected behavior
I expect the generated Swift header to properly work both ways.
Environment
swift-driver version: 1.115 Apple Swift version 6.0 (swiftlang-<IP_ADDRESS>.10 clang-16<IP_ADDRESS>)
Target: arm64-apple-macosx15.0
Additional information
It does work when I don't use the Swift class in the useSwiftClass function and instead pass in a Double or something simple, because then it's only one way (Swift -> C++).
I am honestly not sure how to implement this behaviour right now, because I also cannot forward-declare SwiftClass right? At least then the useSwiftClass(...) C++ function will no longer be available from Swift, because it's an incomplete type.
Any ideas? Workarounds?

So traditionally in C++ you would just forward-declare the type to break the cyclic include.
But this does not work in Swift because if the function signature contains an incomplete type (the forward-declared swift type), it will not expose that C++ function to Swift!
So I cannot forward-declare the Swift types, and honestly I'm not sure how to even do bi-directional C++/Swift interop 🤔


Add .jpeg as a valid extension.

Add support for .jpeg images, like https://www.ptt.cc/bbs/Beauty/M.1599825532.A.E30.html this.

Thanks for contributing!


Item docorator

How to add item decorator to header and child separatedly in expandable recyclerview

I don't think recyclerview supports applying decorators to individual items. You'll need to write your own decorator that checks the position using getAdapterPosition (this will be the visible position) and choose whether or not to apply the decoration at that point.


(FM-8233) add litmus for future acceptance testing

Passing pipelines here https://pipelines.puppet.com/team-modules/builds/574823

what @eimlav said 👍


How do i disable responsive?

My site is only used in PC browsers. How do i disable responsive?

Even so, why to disable it, it can work very well in PC browsers.

Check this out, but its not recommend to do something like this.


AppArmor enabled on system but the docker-default profile could not be loaded

Using release v24. Docker 1.10.2 is unable to start because it's unable to find the docker-default apparmor profile:
time="2016-03-10T17:59:30.456330433Z" level=warning msg="/!\\ DON'T BIND ON ANY IP ADDRESS WITHOUT setting -tlsverify IF YOU DON'T KNOW WHAT YOU'RE DOING /!\\"
time="2016-03-10T17:59:30.549290724Z" level=info msg="Graph migration to content-addressability took 0.00 seconds"
time="2016-03-10T17:59:30.860662498Z" level=info msg="Default bridge (docker0) is assigned with an IP address <IP_ADDRESS>/16. Daemon option --bip can be used to set a preferred IP address"
time="2016-03-10T17:59:30.987538458Z" level=fatal msg="Error starting daemon: AppArmor enabled on system but the docker-default profile could not be loaded."

Tested this on a 3302 ubuntu stemcell.

Exactly the same for me on ubuntu stemcell 3130 and 3026 (on vsphere)
I have read - https://github.com/docker/docker/issues/15340
(see the bottom)
what would you recommend here ? Which release should I use for now.
I had a git clone of master so I am on v24 but can may be switch to another one without too much pain
Note that is my first attempt at this docker-boshrelease with swarm & broker manifest.

using release 25, I also met the same problem
ubuntu@ip-172-31-28-108:~/bosh_workspace/max-services-docker-boshrelease/tt/docker$ cat docker.stderr.log
time="2016-03-23T09:53:41.299283226Z" level=warning msg="/!\\ DON'T BIND ON ANY IP ADDRESS WITHOUT setting -tlsverify IF YOU DON'T KNOW WHAT YOU'RE DOING /!\\"
time="2016-03-23T09:53:41.599187072Z" level=info msg="Graph migration to content-addressability took 0.00 seconds"
time="2016-03-23T09:53:42.736214452Z" level=info msg="Default bridge (docker0) is assigned with an IP address <IP_ADDRESS>/16. Daemon option --bip can be used to set a preferred IP address"
time="2016-03-23T09:53:43.334475864Z" level=fatal msg="Error starting daemon: AppArmor enabled on system but the docker-default profile could not be loaded."
time="2016-03-23T09:53:46.073428052Z" level=warning msg="/!\\ DON'T BIND ON ANY IP ADDRESS WITHOUT setting -tlsverify IF YOU DON'T KNOW WHAT YOU'RE DOING /!\\"
time="2016-03-23T09:53:46.075740174Z" level=info msg="[graphdriver] using prior storage driver \"aufs\""
time="2016-03-23T09:53:46.076009848Z" level=info msg="Graph migration to content-addressability took 0.00 seconds"
time="2016-03-23T09:53:46.183688206Z" level=info msg="Default bridge (docker0) is assigned with an IP address <IP_ADDRESS>/16. Daemon option --bip can be used to set a preferred IP address"
time="2016-03-23T09:53:46.266488604Z" level=fatal msg="Error starting daemon: AppArmor enabled on system but the docker-default profile could not be loaded."

And the docker cannot be started.

@bodymindarts do you happen to remember which stemcell version you used to test the update to docker-1.10.2?

We have this running on vSphere with stemcells v3173 and v3202 and it was tested on most recent bosh-lite stemcells from the time of the update.

Good to know
I have the issue on
bosh-vsphere-esxi-ubuntu-trusty-go_agent/3130
I could find a way to have my own patch.

We are seeing this on vsphere stemcell version 3215

Final v26 contains docker 1.11.0. And the release is working fine for us though there were issues upgrading due to #60
Perhaps @avasseur-pivotal @StanleyShen @frodenas could confirm that the docker 1.11.0 resolves the issue on the stemcells they are using so that we can close this issue.

It works for with version v26.
| bosh-warden-boshlite-ubuntu-trusty-go_agent | ubuntu-trusty | 3147*   | 0636a704-8ac9-44a8-7f21-793f17310298 |

Using stemcell 3232.2 and v27 works for me - I guess you can close this issue


What is the difference between this work and K2?

I think this paper (ICLR 2021 under double-blind review) is related to this repo...

Thanks for the question. There is in fact a lot of overlap. In particular the high-level vision: seamlessly and efficiently using WFSTs with automatic differentiation is virtually the same. The two frameworks are being developed simultaneously and for most of their life without knowledge of one another. We only learned of K2 a couple of weeks ago :).
We started developing an earlier version of GTN in January 2020 and it looks like K2 started around the spring of 2020 (?) so on a similar time frame.
However, I am sure there are and will be some differences in the API and the implementation. The details remain to be seen pending the release of K2.
The linked ICLR submission is indeed related to this framework. The preprint is also available on arXiv.

@awni
I find the graph in Figure 2 (e) of the paper quite confusing. The result of composing the token graph and
the label graph is an FST with ilabel letters and olabels words. However, the emission graph is an FSA with label letters.
How can you compose an alignment graph and an emission graph?
I find in the example code that you are actually converting the alignment graph to an FSA before doing composing.
https://github.com/facebookresearch/gtn/blob/f7900f41edc65843515b1d2f02d6fa40846e33a4/bindings/python/examples/word_decompositions.py#L81-L84
Are you omitting something in the paper?

Oh right, thanks for catching that. It should probably be E_x \circ A \circ Y. For simplicity of notation we use \circ to be composition or intersection. Also if you have more questions about the paper specifically please email me. Let's keep this thread focused on the topic at hand e.g. differences between K2 and GTN.


python311Packages.formulaic: refactor

Description of changes
Fix build (https://hydra.nixos.org/build/246580506)

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 281129 run on x86_64-linux 1

12 packages failed to build:

python311Packages.heudiconv
python311Packages.heudiconv.dist
python311Packages.nipype
python311Packages.nipype.dist
python311Packages.pybids
python311Packages.pybids.dist
python312Packages.heudiconv
python312Packages.heudiconv.dist
python312Packages.nipype
python312Packages.nipype.dist
python312Packages.pybids
python312Packages.pybids.dist

8 packages built:

python311Packages.formulaic
python311Packages.formulaic.dist
python311Packages.lifelines
python311Packages.lifelines.dist
python312Packages.formulaic
python312Packages.formulaic.dist
python312Packages.lifelines
python312Packages.lifelines.dist


OpenStack Automated Deployment Guide

Related to #2, please do not merge yet

I am calling this PR ready for review. This introduces the automated deployment guide using kolla-ansible. I'll follow up with the overview, planning, and operations guides in subsequent PRs.


Add activity feed API

#261  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @lllamnyp and the rest of your teammates on  Graphite


strange CDB bug

Reported by Jonathan S Callahan on 29 Mar 2005 23:45 UTC
Ferret is returning "No Valid Data" when it should not.  Try the following:
use "/home/raid1/data/OSMC/ncep/2005_GODAE_OBS.cdf"
shade/pal=redscale sst_obs["This one works"
shade/pal=redscale sst_obs[l=50@CDB:32](l=50@CDB:31]
message)
message "This one doesn't work"
shade/pal=redscale sst_obs[l=49@CDB:32]
message "So why does this one work?"
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1207

Comment by<EMAIL_ADDRESS>on 29 Mar 2005 23:54 UTC
Interestingly, the behavior seems to be tied to the total number of points
being accessed in this 1x1 deg dataset:
works               fails
l=50@CDB:31         l=50@CDB:32
l=60@CDB:21         l=60@CDB:22
l=70@CDB:11         l=70@CDB:12
l=80@CDB:1          l=80@CDB:2

Comment by<EMAIL_ADDRESS>on 19 Apr 2005 16:42 UTC
fixed in offset_ss.F  Return one-sided offset for transformations CDB, CDA, CIB,
CIA

Comment by<EMAIL_ADDRESS>on 19 Apr 2005 18:04 UTC
additional example using monthly_navy_winds (the example Jon quotes is a file
which is updated, so the particular values dont replicate the bug)
yes? use monthly_navy_winds
yes? let ts =  UWND[let gappy = if ts gt 0 then ts
yes? list/l=1:15 ts, gappy
DATA SET: /home/ja9/tmap/fer_dsets/data/monthly_navy_winds.cdf
TIME: 01-JAN-1982 14:45 to 03-APR-1983 04:15
LONGITUDE: 100E to 180E
LATITUDE: 0
Column  1: TS is UWND[Y=0,X=100:180@AVE](Y=0,X=100:180@AVE]
yes?)
Column  2: GAPPY is IF TS GT 0 THEN TS
TS   GAPPY
16-JAN-1982 /  1: -0.4817    ....
16-FEB-1982 /  2: -0.3769    ....
18-MAR-1982 /  3: -0.0354    ....
18-APR-1982 /  4: -0.4059    ....
18-MAY-1982 /  5: -0.7301    ....
18-JUN-1982 /  6: -0.4212    ....
18-JUL-1982 /  7:  0.4259  0.4259
17-AUG-1982 /  8:  0.5553  0.5553
17-SEP-1982 /  9:  0.5186  0.5186
17-OCT-1982 / 10:  0.5577  0.5577
17-NOV-1982 / 11:  0.9512  0.9512
17-DEC-1982 / 12: -0.6394    ....
17-JAN-1983 / 13: -0.5386    ....
16-FEB-1983 / 14: -0.9079    ....
18-MAR-1983 / 15: -0.9929    ....
yes? list gappy[L=1@cia:10]
VARIABLE : IF TS GT 0 THEN TS
FILENAME : monthly_navy_winds.cdf
FILEPATH : /home/ja9/tmap/fer_dsets/data/
LONGITUDE: 100E to 180E
LATITUDE : 0
TIME     : 16-JAN-1982 20:00 (closest index above by 10 pts)
....
Missing result, though the answer should be 6 -


Added missing export: 'lib/function-configuration'

Description

When I upgraded from<EMAIL_ADDRESS>to 3.15.4, I could no longer deploy function with this command:
$ firebase deploy --only functions:myFunction

The errors are:
i  deploying functions
i  functions: ensuring required API cloudfunctions.googleapis.com is enabled...
i  functions: ensuring required API cloudbuild.googleapis.com is enabled...
v  functions: required API cloudbuild.googleapis.com is enabled
v  functions: required API cloudfunctions.googleapis.com is enabled

Error: Error occurred while parsing your function triggers.

Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './lib/function-configuration' is not defined by "exports" in /home/ben/repos/my-project/src/functions/node_modules/firebase-functions/package.json

    at throwExportsNotFound (internal/modules/esm/resolve.js:299:9)
    at packageExportsResolve (internal/modules/esm/resolve.js:522:3)
    ...

Code sample
N/A

Replaced by: https://github.com/firebase/firebase-functions/pull/963


available_locales() includes :de when it shouldn't

I'm assuming that if you don't explicitly set I18n.available_locales= then I18n will automatically set it based on the translation YAML files in found the load path.
In my load path, I only had files to support locales :en and :es
But when I called I18n.available_locales it returned [:en, :es, :de] . Even thought :de was no where in the load path. I have no idea where it was getting :de from. Does anyone have a guess?
The workaround was that I explicitly set the value using this statement:
config.i18n.available_locales = [:en, :es]
This could very well be something wrong with my server. But it seems strange to me that the incorrect locales persisted.

This extra locale might be from some gem somewhere in your application, I18n should not be adding anything to available_locales, it only loads the ones available in locale files.
Can you search for the gems in your project and trying to find which one might be the culprit? I'd bet that's the case.
Thanks.

Agree with above. Likely comes from a gem.


Position on response body resets if moving to another tab/request

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
Hi.
When I issue a request and scroll down through the response body, if I switch to another request and come back to the first one (or if I click on response's headers, Timeline or Tests tab), Bruno lost track of the position where I left.
I added a bunch of screenshots to help you guys understand of the issue. Let me know if more clarification is needed.
I used Mockoon to simulate an API server, and https://json-generator.com/# to generate a test JSON response.
Thanks!
.bru file to reproduce the bug
No response
Screenshots/Live demo link
Switching between requests

Switching between Response's tabs

Hey @Nahuel92, We appreciate you highlighting this issue. Our team will look into it shortly.


Auto Cancellation support

Added auto-cancellation via Lifecycle.
Any component like Activity, Fragment, or Service that has a Lifecycle can be now used to auto-cancel the inflation in the onDestroy state.
In the case of a Fragment, we use viewLifecycleOwner.lifecycle as it makes more sense to cancel inflation there.

Removed unnecessary dependencies

Updated Readme.md

Also fixes: #1

Any updates on this PR?

Updated the sample & certain dependencies.


Removed custom css properties for now

Fixes #141

This change is

Review status: 0 of 4 files reviewed at latest revision, 1 unresolved discussion.

vaadin-date-picker-overlay.html, line 157 [r1] (raw file):
border-left-color can't be customised anymore. Deliberate?

Comments from the review on Reviewable.io

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

demo/index.html, line 118 [r1] (raw file):
Would it be more consistent to use --paper-gray-something?

demo/index.html, line 120 [r1] (raw file):
Would it be more consistent to use --paper-gray-something?

Comments from the review on Reviewable.io

Review status: 2 of 4 files reviewed at latest revision, 3 unresolved discussions.

vaadin-date-picker-overlay.html, line 157 [r1] (raw file):
Done. Added a new mixin for it. It's the color for the year scroller arrow indicator.

demo/index.html, line 118 [r1] (raw file):
See below

demo/index.html, line 120 [r1] (raw file):
The color is --dark-theme-secondary-color which is fixed in the main theme file https://github.com/PolymerElements/paper-styles/blob/master/default-theme.html#L60 and there's no value for it in https://github.com/PolymerElements/paper-styles/blob/master/color.html
Can't use "--dark-theme-secondary-color" here as value either because it's changed few lines above and that would create a conflict. I think it's ok to demonstrate use of inline color values also (not only variables)

Comments from the review on Reviewable.io

Reviewed 2 of 4 files at r1, 2 of 2 files at r2.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

Comments from the review on Reviewable.io

Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending.

Comments from the review on Reviewable.io

Reviewed 2 of 4 files at r1, 2 of 2 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io

Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from the review on Reviewable.io


Dynamically creating actors

Hi
I need help in Dynamically creating the actors according to the user requirement (no of actors they need for the job to complete )
as i am placing them in the form id's and names

__


New functionality for N-1 histograms, Including dimuon categories for cutflows, Changed all Selections to Cuts

I've changed the n-1 histograms so I only fill the histograms corresponding to the cut not applied. It uses the same names of the cuts specified in the config, same as for the TTAlpsDimuonSelections.
Some other changes:
Including the dimuon categories (pat-pat, pat-dsa, dsa-dsa) in the cutflows
I changed all Selections to Cuts as requested in the last PR
including new signal cross sections
small changes in objects selections

Changed Selections to Cuts in some tea files as well, see PR in tea: https://github.com/jniedzie/tea/pull/20


Fixed Citrix Workspace appNewVersion

cut didn't work for appNewVersion so I changed it to awk.

Thanks!


useQuery data type not nullable

In Nuxt 3.5.1 the return type of useAsyncData was changed and is not nullable by default anymore. we should define it as nullable in our types:
https://github.com/wobsoriano/trpc-nuxt/blob/adeb5a5afdd4ebde7743ca9e16148a0f134cb448/src/client/types.ts#L63
should be:
... AsyncData<PickFrom<DataT, PickKeys> | null, DataE>

Hey, thanks! Would you like to open a PR?


"CustomHTML2Text" is not defined in synchronous WebCrawler

This issue occurred  we use synchronous WebCrawler in crawl4ai==0.3.73

@aadityakanjolia4 Thanks for using the library. This issue is already fixed in the new version, which will be released in a day or two.


gnome3: Include gnome-shell-extensions in session path

This is required to get the "window list" extension to work

Pushed it directly into master.


Schedule for event, Issue #180

@mananwason
Please have a look.

Here are the screen shots for the schedule fragment.

If there are any issues please report.


Infinite loop when setting data on ChartDataEntry object

I'm setting additional data on the ChartEntryObject to pass it to the Marker view:
for accountBalance in accountBalances
{
    var entry = ChartDataEntry(value: accountBalance.amount.floatValue, xIndex: count++)
    entry.data = dateFormatter.stringFromDate(accountBalance.date)

    yVals.append(entry)
}

This causes EXC_BAD_ACCESS infinite loop in ChartDataEntry.swift:

The problem is fixed by changing line 64 of ChartDataEntry.swift from:
if (object!.data !== data && !object!.isEqual(self))

to
if (object!.data !== data)

Is it safe to do so?

Actually it isn't, the correct fix would be to changed line 54 to be === instead of ==

Hi, thanks for the quick reply.  I changed that line as you suggested but it's now causing EXC_BAD_ACCESS on the line that I thought may be the problem:

Making this change does appear to fix it:
if (object!.data !== data)

I'm new to Swift so I'm just not sure of the full implications of this.  Is !object!.isEqual(self) not somehow causing an infinite isEqual() loop on itself?

It was just another thing I missed. It should have been isEqual(data)...

Thanks!


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_eawy22s9gf/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_eawy22s9gf/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_eawy22s9gf/clone/recipe/meta.yaml:31: missing_imports_or_run_test_py: Python packages require imports or a python test file in tests.
ERROR: /tmp/abs_eawy22s9gf/clone/recipe/meta.yaml:37: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_eawy22s9gf/clone/recipe/meta.yaml:37: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_eawy22s9gf/clone/recipe/meta.yaml:37: missing_description: The recipe is missing a description
ERROR: /tmp/abs_eawy22s9gf/clone/recipe/meta.yaml:37: missing_dev_url: The recipe is missing a dev_url
Errors were found


Site Navigation gets shortened too early

For Bug Reports

BookStack Version (Found in settings, Please don't put 'latest'): 0.18.3

Current Behavior

Should be shortened using CSS or give us an option after how many signs it should be cutted of, or that we can define if it should get cut off or not.
Where can i find in code the shortening to change it right now for me?

Thanks for the request @Wachert. Just came across this again.
Looks like it was fixed as part of a duplicate request (#1206) and has been part of BookStack since v0.27. Will therefore close this off.

Thanks for the request @Wachert. Just came across this again.
Looks like it was fixed as part of a duplicate request (#1206) and has been part of BookStack since v0.27. Will therefore close this off.


Refine publication attribute names

From the 2023-10-10 HTAN operations meeting

Eg: publications, de-id issues
Could we alter publications manifest column header “Publication-associated HTAN Parent Data File IDs” to something like “Publication-associated HTAN Data File IDs” in both the RFC and manifest example?  The description of the column in Table 1 of the RFC seems fine but the naming doesn’t seem to jibe with the naming in the id_provenance.upstream_ids table in bigquery. (Alex)

The discussion was to keep ID's as is but to open this issue to confirm that @elv-sb is happy with this.
With @clarisse-lau we need to check that this will be implemented in the upstream IDs OK

No action required here.


Please, explain how does copying memory on each draw operation stops crashing

https://github.com/jcupitt/ruby-vips/blob/4d9340ff6aa19c6f29300a198994b03948b01893/lib/vips.rb#L416-L417
I want to understand how ruby-vips works to make smth about my code that does lots of drawing and so leaks a lot.

For example, I disabled this line
https://github.com/jcupitt/ruby-vips/blob/143e4133af85aefff4bcecde42f30c833195904c/lib/vips/operation.rb#L148
for draw_line and draw_circle, and it didn't crash.

Hello @Nakilon, when you draw on a computed image, libvips has to allocate some memory, render the image into that memory, then adjust the image structure so that it refers to the memory area and not the pipeline.
For example:
a = Vips::Image.new_from_file "x.jpg"
b = a + 10
c = b

So b is now the head of a pipeline of delayed computations and c is a reference to it. But if you now do:
b.draw_line 255, 10, 10, 110, 110

Now b has been changed internally to be a memory area and no longer references a, but c is still referencing the delayed computation. If you try and use c, you'll probably get a crash, or at least strange behaviour.
It can be hard to see when images are being shared like this, especially in a threaded program with the libvips cache turned on. You need to .copy() before using any of the draw operations if there's any chance there could be another reference to the image somewhere.
I think we were talking a few months ago about a better way to handle this. It would be much faster to make a new block, called mutable or destructive, that you used like this:
b = a.mutable do |x|
    x.draw_line 255, 10, 10, 100, 100
    x.draw_circle ...
end

With some kind of lock so that draw operations could only be used inside a mutable block, and the mutable does the copy only once and passes the copied image in.

Perhaps as a first step we could add a nocopy optional argument? It would disable the extra copy if set true. We could then implement the ! versions, and document it as the caller's responsibility to call .copy before the first !.
It's not really a memory leak, it's just very heavy memory use that the Ruby GC does not know about.

Here's a branch that adds ! operators:
https://github.com/jcupitt/ruby-vips/tree/add-destructive
Before:
require 'vips'

Vips::leak_set true

im = Vips::Image.new_from_file ARGV[0]

im = im.copy
(0..im.width).step(10) do |i|
  im = im.draw_line 255, i, 0, 0, im.height - i
  im = im.draw_line 255, i, 0, im.width, i
  im = im.draw_line 255, i, im.height, 0, i
  im = im.draw_line 255, i, im.height, im.width, im.height - i
end

im.write_to_file ARGV[1]

$ time ruby lines.rb ~/pics/k2.jpg x.jpg
memory: high-water mark 425.07 MB
real	0m2.360s
user	0m3.782s
sys	0m0.875s

If I change it to:
require 'vips'

Vips::leak_set true

im = Vips::Image.new_from_file ARGV[0]

im = im.copy
(0..im.width).step(10) do |i|
  im.draw_line! 255, i, 0, 0, im.height - i
  im.draw_line! 255, i, 0, im.width, i
  im.draw_line! 255, i, im.height, 0, i
  im.draw_line! 255, i, im.height, im.width, im.height - i
end

im.write_to_file ARGV[1]

I see:
$ time ruby lines.rb ~/pics/k2.jpg x.jpg
memory: high-water mark 19.32 MB
real	0m0.966s
user	0m0.954s
sys	0m0.098s

Still not especially quick, but much better memory use.
It's still possible to write a program that will crash, which seems bad.

See https://github.com/libvips/ruby-vips/issues/139


now we just need to figure out how to output the structs

Signed-off-by: Kris Nóva<EMAIL_ADDRESS>

Closes #64

This is an interface change, so we will need to bump to 1.0.0 for this update 🎉
we add app.Objects() []runtime.Object to the interface
we also add the ability to pass in nil for a kube client to register the app objects


Silent Payment send suport

Silent Payments are getting support in an ever growing list of wallets right now. Implementing the ability to receive Silent Payments is a huge feature but being able to send to a Silent Payment address sounds doable and would help adoption in the ecosystem

Ack for Send Support.
Receive Support later

https://docs.rs/silentpayments/0.3.0/silentpayments/

Silent Payments would be a nice feature add IMHO.
https://github.com/bitcoin/bips/blob/master/bip-0352.mediawiki
https://bitcoin.design/guide/how-it-works/silent-payments/
https://silentpayments.xyz/docs/wallets


add homework5

https://video.drift.com/v/ablzNEwW1QDoe2auwZwEFP6W6pKhpAJU5V6vD2lLrqwU/

Metric
Value

alert_status
:white_check_mark: OK

bugs
1

code_smells
18

reliability_rating
C (at least 1 Major Bug)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:x: ERROR

bugs
1

code_smells
2

reliability_rating
C (at least 1 Major Bug)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:x: ERROR

bugs
0

code_smells
1

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


Sticky steps?

Hi, first of all I wanted to say I love the plugin! I use CodeTour for private commenting, to help me personally explain bits of code without cluttering the repo with comments.(In my use case I don't commit the .tour folder. I also don't "run" tours, I just use them as hidden/popout comments)
What would you say is the best way to make a step "sticky" so that file changes won't drift a tour step from its associated code block? I have mostly used line based steps but now see that maybe selecting a code block first will better connect the step to it. Is that true?
If the code block is modified is there potential for the step to still drift? What type or level of modification would cause that?
Thanks!

Hey!  Or are you creating tours as a series of comments for a single codebase?
Also, how are you storing/persisting the tours if you don't commit them? Do you record the tours, and then move the files someone else? I just ask since you can export tours, and CodeTour allows storing tours in GitHub Gists, and I'd be curious to hear if that's helpful for your workflow 👍
Regarding making the steps sticky, there are a few options currently:

You can export a tour, which will include the associated code in the tour file itself, and therefore, can always be replayed, even if the annotated code changes.

Additionally, when you record a tour, you can choose to associate it with a specific commit, and whenever you replay the tour, it will use the version of the files in the original commit.

Would either of those work for your workflow? Both of these options rely on "snapping" the tour to a specific version of the codebase, and therefore, might not be great if you want to have your tours be "sticky" to the codebase. But they have the benefit of not requiring the codebase itself to be modified for your personal notes.
In order to allow a tour to remain in sync with the latest codebase changes, we're exploring two options:

Injecting a comment "marker" into the codebase, so that the step can be associated with the code block, not a specific line. This might not work for you, since you don't want to include any changes into the codebase?

Associating the step with a line of code via a regular expression. That way, if the line number changes, then the tour would remain "sticky" to it.

In both of these cases, we'd need to make the recorder allow authoring tours easily, but I think we could address that challenge.
In general, I'd love to hear your thoughts, since this is an area where we need a ton of great feedback on. Thanks! 😁

Hi Jonathon,
What I do is create a single tour with a bunch of steps. In the future I might group them depending on code base section but right now I use a single tour.
The tours are persisted locally only. I use a global .gitignore rule to ignore .tours   Your ideas are also great, I will look into both in a few months when I get a second machine.
Regarding making comments sticky, regex matching would require more work and in my view reduce the ease of use of creating quick, private comments on a line or code block. I value the speed of a simple GUI click, typing of the private comment, and save. Easy to comment, easy to return to later.
So what could improve the current drift issue? Comment markers on code blocks sound ideal. To handle updates to changing code blocks, perhaps the code block/marker is flagged as "altered". It then has a visual change marker at an alert/caution/yellow level to visually indicate medium priority (not all changes outdate a private comment).
CodeTour also references this altered code-blocked tour/comment under a "altered code blocks" section. Ideally this section allows jumping through altered steps just as it would with a normal "tour."
When a change is complete, a "mark updated" button is clicked to move the step out of altered status. The button could also be clicked if updating the block is not needed or desired.
The changed section could continue to nest the tour. Or it could dump the changed sections globally. Nesting is probably more usable for the use cases of others. In my case a global dump would work well.
Hope that helps and great job so far with CodeTour!
On Sun, 31 Jan 2021 18:01:55 -0800
Jonathan Carter<EMAIL_ADDRESS>wrote:

Hey!  Or are you creating tours as a series of comments for a single codebase?
Also, how are you storing/persisting the tours if you don't commit them? Do you record the tours, and then move the files someone else? I just ask since you can export tours, and CodeTour allows storing tours in GitHub Gists, and I'd be curious to hear if that's helpful for your workflow 👍
Regarding making the steps sticky, there are a few options currently:

You can export a tour, which will include the associated code in the tour file itself, and therefore, can always be replayed, even if the annotated code changes.

Additionally, when you record a tour, you can choose to associate it with a specific commit, and whenever you replay the tour, it will use the version of the files in the original commit.

Would either of those work for your workflow? Both of these options rely on "snapping" the tour to a specific version of the codebase, and therefore, might not be great if you want to have your tours be "sticky" to the codebase. But they have the benefit of not requiring the codebase itself to be modified for your personal notes.
In order to allow a tour to remain in sync with the latest codebase changes, we're exploring two options:

Injecting a comment "marker" into the codebase, so that the step can be associated with the code block, not a specific line. This might not work for you, since you don't want to include any changes into the codebase?

Associating the step with a line of code via a regular expression. That way, if the line number changes, then the tour would remain "sticky" to it.

In both of these cases, we'd need to make the recorder allow authoring tours easily, but I think we could address that challenge.
In general, I'd love to hear your thoughts, since this is an area where we need a ton of great feedback on. Thanks! 😁
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/microsoft/codetour/issues/126#issuecomment-770513911


Add an example for force clean checkout at step level

Currently in docs there is no example on how to use environment variable BUILDKITE_CLEAN_CHECKOUT to set force clean checkout so adding one through this example and also information on how to confirm from logs if clean checkout happened.

@mbelton-buildkite I accepted the suggestion and updated the PR with it.

@mbelton-buildkite To unblock this doc PR I removed the use of that emoji as that is not essential part of this doc change.

👍🏻 Feel free to merge when you're ready


Stubs (.h and .cpp) files does not update when i build

I am following the tutorial below but when i make make changes to my .idl file, the .h an .cpp files in 'Generated files' does not get updated.

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

ID: cd002fb0-9c40-6faa-9c83-b6cf7430bbe1
Version Independent ID: 416d6f14-82b4-38c9-a8b2-2daf5cf2a37c
Content: XAML items controls; bind to a C++/WinRT collection - Windows UWP applications
Content Source: windows-apps-src/cpp-and-winrt-apis/binding-collection.md
Product: uwp
Technology: platform
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Hi, @petrejonn, they should be being generated in `\Bookstore\Bookstore\Generated Files\sources'? I'll make the topic clearer about the exact path. ~Steve

I guess its an issue with my visual studio. i will try try reinstalling it. Thanks anyway

Hi, @petrejonn, I'll go ahead and close this issue now since it's been a while, and it feels resolved. Please reopen if I made a mistake in closing. ~Steve


Question regarding restarts and sunrise before triggertime

Hi
If using the ON/OFF mode. Will it function properly if there is a restart in the active range?
E.g ON: 06:00 OFF Sunrise +60
What happens if there is a restart at 07:30. Will it still trigger at sunrise +60?
What happens if sunrise +60 is before 06:00 ? Will it realize this and not trigger?
(This is a very common thing when living close to the poles :) )

Will test this and let you know 😄


Version 3.3 forces X and Y to be 0 on child svg

Discussed in https://github.com/svg-net/SVG/discussions/948

Originally posted by rparussel January 27, 2022
Previously, when using version 3.2.3, I made use of a trick to create an svg that would be resized with a new aspect ratio to fill it's container, which contained text that retained their aspect ratio. The text was placed inside child svg's within the parent svg. In order to do this it was necessary to set the X and Y values on the child svg's to position the text. Here's a snippet of the html that it generated ...

However, I've since upgraded to version 3.4 and I notice that all X and Y values on svg's are forced to be 0. Looking at the release notes this seems to have been done as a fix in version 3.3 ...

ignore x and y attributes in outermost svg as per standard (see #886 ([https://github.com/svg-net/SVG/issues/886]))

In my case these child svg's, which are not outmost svg's are also affected. Looking at the source for SvgDocument, the cause seems clear ...

Would it be possible to include a fix in the next release to only set X and Y to zero on the outermost svg rather than on the child svg's as well?

Closing again, should remain a discussion item.


Enabling XSLT visualization from b2 module

can we not say "crappy?" maybe "preliminary" or "initial"

Sorry, that was my original commit not meant to ever be merged. It still
won't be merged; I believe this is a branch so I can help Dmitry
troubleshoot issues he's running into. I'll watch my tongue in the future
in any case. :-)
On Wed, Nov 3, 2021 at 9:15 AM Vinnie Falco @.***>
wrote:

can we not say "crappy?" maybe "preliminary" or "initial"
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/boostorg/docca/pull/116#issuecomment-959604675, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAETWLS4JV435KRONQ7KOQDUKFUZ7ANCNFSM5HJATFDA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Yes, once everything works we'll do some clean-up.

Sure. We have to keep up appearances :)


Feature Request: add support for Strict Transport Security

All SSL, all the time, is pretty much a standard for anyone with authenticated traffic. HSTS provides this in a way that is much better than simply redirecting non-ssl requests (which really don't add much protection anyways).
What would be the correct place for this? It could be argued that it should be set in session_hijacking.rb but that's not my call :)

Well then. Sorry for not closing this out earlier!


Disallow def foo=(a:A, b:B)

foo= should take only one argument.

Oh I forgot []=. Yes, that should be allowed.


cli: set click_types.Path for -r and -e options

The issue
pipenv can generate its own shell completions using the click framework. Prior to this PR, there are no file/path completions for the -r and -e options. See #3478 for a discussion.
The fix
This PR changes the option type to click_types.Path for -r and -e. This is enough for click to know that file/path completions are desired here.
And this isn't only beneficial for fish shell, it automatically applies to bash and zsh also.
This fixes partially issue #3478 but not the part concerning pipenv run completions.
The checklist

[x] Associated issue
[ ] A news fragment in the news/ directory to describe this fix with the extension .bugfix.rst, .feature.rst, .behavior.rst, .doc.rst. .vendor.rst. or .trivial.rst (this will appear in the release changelog). Use semantic line breaks and name the file after the issue number or the PR #.

I'm not sure whether the news fragment is mandatory and which type would be appropriate. If you tell me, I could add one.

Can you please add a news file?

Can you please add a news file?

done


Migration Offerings Table Incorrect

The table under 'Migration Offerings'  looks wrong. The headings and values make no sense in relation to each other. Would be great to fix this please!

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

ID: 9ee6b3fe-cc73-cf65-8f91-aa75b2bda8f8
Version Independent ID: a49a2330-4e5b-922e-870c-26aca6f63892
Content: Migrate file shares to SharePoint and OneDrive - Migrate to Microsoft 365
Content Source: migration/Fileshare-to-ODSP-migration-guide.md
Service: sharepoint-online
Sub-service: sharepoint-migration
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

Hello @briggsb9 Ben, thanks for your feedback, will propose to improve the table reading.


Textbox not editable?

I'm not sure what I'm doing wrong with this.  I am populating a Datatable with AJAX data, and that's working fine.  When I bring the editor up, I can modify the "select" columns, but not the text ones.
The only thing I can think of is that I had to force a higher z-index for the modal dialog since the Datatable is inside a modal dialog (created using Magnific Popup) and the editor was originally coming up behind the popup the Datatable was contained in:
var modal = '<div class="modal fade" id="' + modal_id + '" tabindex="-1" role="dialog" style="z-index:1500;!important">' +
var columnDefs = [
        { data: "id", title: "ID", type: "hidden", className: "hideColumn" },
        { data: "firstname", title: "First Name", type: "text" },
        { data: "lastname", title: "Last Name", type: "text" },
        { data: "primary_phone", title: "Phone Number", type: "text" },
        { data: "primary_phone_type", title: "Type", type: "select", options: types },
        { data: "location", title: "Location", type: "select", options: locations }
    ]

    $('#manageTable').DataTable({
        ajax: '/StatusSite/secure/getEMUsers',
        dataSrc: '',
        columns: columnDefs,
        dom: 'Bfrtip',
        select: 'single',
        altEditor: true,
        buttons: [
            {
                extend: 'copy',
                text: 'Copy to clipboard'
            },
            {
                extend: 'excel',
                text: 'Save as XLS'
            },
            {
                text: 'Add',
                name: 'add'
            },
            {
                extend: 'selected',
                text: 'Edit',
                name: 'edit'
            },
            {
                extend: 'selected',
                text: 'Delete',
                name: 'delete'
            }
        ],
        order: [[2, 'asc']]
    });

edit:  it was definitely because of Magnific Popup.  Once I got rid of that the editor started working as expected.

I suppose this can be closed, please reopen if needed.


Add query_balance subcommand

CHANGELOG:

Add support for new node RPC method query_balance, used in the binary's new query-balance subcommand.

Closes: 2876

Closing in favor of #27


Fix issue

Description

Made the script that checks the disk space and send a email if it is below 20% space!
Additional Context

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


TestRunDuplicateMount requires the same host

Signed-off-by: John Stephens<EMAIL_ADDRESS>@yongtang

LGTM (not a maintainer). Thanks for taking care of it! ❤

LGTM

Failure in windows should be unrelated (cc @jhowardmsft )
...
01:14:10.606 SKIP: docker_cli_ps_test.go:831: DockerSuite.TestPsFormatSize (Test requires a Linux daemon)
01:14:10.784 SKIP: docker_cli_ps_test.go:547: DockerSuite.TestPsGroupPortRange (Test requires a Linux daemon)
01:14:10.947 SKIP: docker_cli_ps_test.go:666: DockerSuite.TestPsImageIDAfterUpdate (Test requires a Linux daemon)
01:14:11.117 SKIP: docker_cli_ps_test.go:528: DockerSuite.TestPsLinkedWithNoTrunc (Test requires a Linux daemon)
01:14:42.205 PASS: docker_cli_ps_test.go:20: DockerSuite.TestPsListContainersBase	28.788s
01:15:26.409 PASS: docker_cli_ps_test.go:309: DockerSuite.TestPsListContainersFilterAncestorImage	43.876s
01:15:26.997 PASS: docker_cli_ps_test.go:568: DockerSuite.TestPsListContainersFilterCreated	0.318s
01:16:10.984 PASS: docker_cli_ps_test.go:449: DockerSuite.TestPsListContainersFilterExited	42.436s
03:20:31.926 Build timed out (after 200 minutes). Marking the build as failed.


Provision: Replace virtualenv by python native venv.

Fixes #625

Merged, thanks @LoopThrough-i-j!  I made a few edits to the commit message, mainly for technical English ("Replace virtualenv with python native venv.", i.e. I did s/by/with/).

Thanks, will keep it in mind.


The project 'precise-truck-742' is not in the requisite Stackdriver service tier.

The system test that creates and deletes a metric descriptor is failing as below. This is what would happen if Stackdriver had not been enabled for this GCP project, so that's probably what's wrong.
Could one of you with access to the precise-truck-742 project please use the cloud console to enable Stackdriver Monitoring? (This is not about enabling the API.) Just go to "Monitoring" in the left nav, choose to create a new Stackdriver Account, and click through the remaining screens.
test_create_and_delete_metric_descriptor (monitoring.TestMonitoring) ... ERROR
test_fetch_metric_descriptor (monitoring.TestMonitoring) ... ok
test_fetch_resource_descriptor (monitoring.TestMonitoring) ... ok
test_list_metric_descriptors (monitoring.TestMonitoring) ... ok
test_list_metric_descriptors_filtered (monitoring.TestMonitoring) ... ok
test_list_resource_descriptors (monitoring.TestMonitoring) ... ok
test_query (monitoring.TestMonitoring) ... ok
======================================================================
ERROR: test_create_and_delete_metric_descriptor (monitoring.TestMonitoring)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/GoogleCloudPlatform/gcloud-python/system_tests/monitoring.py", line 174, in test_create_and_delete_metric_descriptor
    descriptor.create()
  File "/home/travis/build/GoogleCloudPlatform/gcloud-python/.tox/system-tests3/lib/python3.4/site-packages/gcloud/monitoring/metric.py", line 152, in create
    data=self._to_dict())
  File "/home/travis/build/GoogleCloudPlatform/gcloud-python/.tox/system-tests3/lib/python3.4/site-packages/gcloud/connection.py", line 343, in api_request
    error_info=method + ' ' + url)
gcloud.exceptions.BadRequest: 400 The project 'precise-truck-742' is not in the requisite Stackdriver service tier. (POST https://monitoring.googleapis.com/v3/projects/precise-truck-742/metricDescriptors/)
----------------------------------------------------------------------
Ran 7 tests in 11.414s
FAILED (errors=1)

On it.

Done. Please re-open if it continues to fail. Can you send the failing build so I can retry it?

It was build 3615 that failed.

OK. Just restarted https://travis-ci.org/GoogleCloudPlatform/gcloud-python/builds/127310753

Thanks! The test now passes. (Subsequent failures have been on other other modules.)


2D function, look_at() is absent.

2D function, look_at() is absent.

#1843 ?

close to there was no double issue ...


DBDAART-10305 - For June TAF Run, update subData to remove 'CSO' file…

… submission types for New Mexico (state 35), because all four file types are 'TFFR' in June.
What is this and why are we doing it?
For New Mexico (state 35), all four file submission types are 'TFFR' in June. This change is to delete the lines of New Mexico from subData where its ELG and TPL were previously assigned as 'CSO'.

Link to the Jira ticket for this change: https://jiraent.cms.gov/browse/DBDAART-10305

What are the security implications from this change?
None.
How did I test this?
I created a notebook in VAL with the new version of the function, ran the function, viewed the output table, and confirmed it is as expected. The link to the notebook - https://cms-dataconnect-val.cloud.databricks.com/?o=955724715920583#notebook/3300450104828381/command/3300450104828382
Should there be new or updated documentation for this change? (Be specific.)
None.
PR Checklist

[ x ] The JIRA ticket number and a short description is in the subject line
[ x ] My code follows any applicable style guides
[ x ] I have performed a self-review of my own code
[ x ] I have commented my code, particularly in hard-to-understand areas
[  ] I have made corresponding changes to the documentation

@a-pfeiffer @alihashmi-wh @fumanjie @hls0231 Please review at your earliest convenience. Once approved, I will need to merge the change to 'main', and build/upload a new wheel file for the June TAF run, scheduled to begin on 6/5. Thanks in advance!
cc: @sgreiner-cmcs


Adding all AllJoyn samples as source code

This creates a source tree structure instead of copies of .zip files for all AllJoyn samples.
Addresses
https://github.com/ms-iot/samples/issues/112
https://github.com/ms-iot/samples/issues/64

Hi @zhuridartem, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


Sharing a TensorRT implementation of MODNet

@ZHKKKe  Thanks for sharing your inference code and the pre-trained model.
I took your pre-trained PyTorch model and was able to build a TensorRT engine from it.  I shared my TensorRT implementation at jkjung-avt/tensorrt_demos, or more specifically, Demo #7: MODNet.  Feel free to give it a try.

@jkjung-avt
It is wonderful! Thanks!
I hope to integrate it into this Github project after I check it.
As I am busy these two weeks, I will add it later. See what you think.

I hope to integrate it into this GitHub project after I check it.

I put my code under MIT License.  You are welcome to integrate the code into this repo by following the license.
Thank you again for sharing the model.

Hey, I referenced your project in our README. Thanks again.


Add handling for Youtrack Auth header

Hey! I needed to add authentication for my YouTrack installation, here are the changes if you are interested :)

Looks useful! Does it behave properly if auth_header_youtrack has an invalid value (e.g. an empty string)?

With b07be91 that should be taken care of. Also since token_youtrack defaults to None it is compliant with previous versions.

Thanks!


[SPARK-27184][CORE] Avoid hardcoded 'spark.jars', 'spark.files', 'spark.submit.pyFiles' and 'spark.submit.deployMode'

What changes were proposed in this pull request?
For SPARK-27184
In the org.apache.spark.internal.config, we define the variables of FILES and JARS, we can use them instead of "spark.jars" and "spark.files".
private[spark] val JARS = ConfigBuilder("spark.jars")
  .stringConf
  .toSequence
  .createWithDefault(Nil)

private[spark] val FILES = ConfigBuilder("spark.files")
  .stringConf
  .toSequence
  .createWithDefault(Nil)

Other :
In the org.apache.spark.internal.config, we define the variables of SUBMIT_PYTHON_FILES  and SUBMIT_DEPLOY_MODE , we can use them instead of "spark.submit.pyFiles" and "spark.submit.deployMode".
private[spark] val SUBMIT_PYTHON_FILES = ConfigBuilder("spark.submit.pyFiles")
    .stringConf
    .toSequence
    .createWithDefault(Nil)

private[spark] val SUBMIT_DEPLOY_MODE = ConfigBuilder("spark.submit.deployMode")
    .stringConf
    .createWithDefault("client")

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9076/
Test PASSed.

Test build #103679 has started for PR 24123 at commit 37c16db.

Test build #4644 has started for PR 24123 at commit 37c16db.

Test build #4648 has started for PR 24123 at commit 37c16db.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9127/
Test PASSed.

Test build #103749 has started for PR 24123 at commit 7a88f79.

Test build #103749 has finished for PR 24123 at commit 7a88f79.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103749/
Test FAILed.

retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9133/
Test PASSed.

Test build #103755 has started for PR 24123 at commit 7a88f79.

Test build #103755 has finished for PR 24123 at commit 7a88f79.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103755/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9135/
Test PASSed.

Test build #103757 has started for PR 24123 at commit 7a88f79.

Test build #103757 has finished for PR 24123 at commit 7a88f79.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103757/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9138/
Test PASSed.

Test build #103761 has finished for PR 24123 at commit 7a88f79.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103761/
Test FAILed.

Test build #4650 has started for PR 24123 at commit 7a88f79.

Test build #4650 has finished for PR 24123 at commit 7a88f79.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9148/
Test PASSed.

Test build #103770 has started for PR 24123 at commit 7a88f79.

test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9157/
Test PASSed.

Test build #103783 has started for PR 24123 at commit 7a88f79.

Test build #103783 has finished for PR 24123 at commit 7a88f79.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103783/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9175/
Test PASSed.

Test build #103802 has started for PR 24123 at commit c229cdc.

Test build #103802 has finished for PR 24123 at commit c229cdc.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103802/
Test FAILed.

retest this please

Test build #103810 has started for PR 24123 at commit c229cdc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9184/
Test PASSed.

Test build #103810 has finished for PR 24123 at commit c229cdc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103810/
Test PASSed.

Merged to master.


Ideas for use-case driven demos

Financial services / banking:

customer 360 (join mortgage, retail, investment, commercial, wealth management)
investigate accounts based on credit score changes

Logistics:

notify based on changes in shipment status
3rd party logistics

@chuck-alt-delete are they related to data science, data vis, analytics, or any vertical in particular?

@joacoc I think the idea from the sales side is it would be nice to have a handful of off-the-shelf demos that we can use to connect with prospects a little more directly. We obviously can’t do a tailored demo for every prospect, but we can show use cases for a couple of common industries and use cases.
We could (should?) even have them follow a general formula:

show index and how it lives in a cluster
Show materialized view and how it lives in persist
Show index in serving cluster on top of MV
Show subscribe + zero downtime resizing
show client application / BI (visualization)

verticals are maybe not the right approach right now. Feedback from slack. I'll close this for now.


Geatures/ea 79 shared read write access

I think we need to add tests for concurrent access (at least 2 concurrent writers) emitting log messages at high frequency. It is my assumption that the output will then be mangled.

It is fine to have the output out of sink (ha!) in my opinion. Having a new file created on each app pool recycle will probably cause issues for DV as they are expecting one file per day.

if you think the messages will be serialised correctly then code looks fine. :shipit:

What do you mean serialised correctly? The ordering or just that they are written to the file correctly?

I mean that the streams don't flush at the same time, mangling the log events from 2 writers.

To elaborate more, i think that the events are linearised (is that a word?) in the sink. So the sink owns it's resource, and assumes exclusive access.

Yeah, I understand you now.


Bug in Static Property Node Accessor VI Creation

(this bug originally posted here: https://forums.ni.com/t5/GDS-Goop-Development-Suite/Bug-in-Static-Property-Node-Accessor-VI-Creation/m-p/4142629 )
When you create a static dispatch property node accessor VI with regular non-GDS classes (i.e. using the native class editor features in LabVIEW), the new property VI is marked as Inline. The same does not happen when creating a static dispatch property node accessor VI with OpenGDS. It is created as a non-reentrant, non-inlined VI. From what I can tell this is because the property node accessor VI template is set to non-inlined to support being dynamic dispatch.
To solve the issue for static dispatch property node accessor VIs, I propose a fix to the following VI:
resource\Framework\Providers\Open_GDS\ClassProviders\Provider_LvNativeClass\ClassWriterNative_class\CreateMethod.vi
I made the following edit to the VI, and now my static property node accessor VIs are marked as inline when they are created:

I assume a similar fix would need to be made to scripting for other class types (GOOP3, GOOP4, etc.), but I didn't dig into their scripting code so I wasn't sure.

Done, in the 2020 branch.
Planning of building a new version next week

It seems that this bug is still active in the latest LV2020 branch v1.2.72. I checked the code and it seems to be as shown in the image, so i'm not sure what the problem is. I can reproduce it using the LVClass by value and the GOOP4 class provider.

It is there i the code, if it's a Property Node Method and Not Dynamic Dispatch, it should set it to that?!?

Found it, you can get an error if you do this in parallel, since the dynamic dispatch setting is still being turned off.
V1.2.74 is coming tomorrow


📝 add README

Current dependencies on/for this PR:

main

PR #7   👈

This comment was auto-generated by Graphite.


Token not found in Error Callback instead of notFoundHandler

If call a page that are not maped instead of notFoundHandler is called the error callback of this library.
I.e.: if call https://foo.bar/public/pageitenrionalynotmaped
I receive: {"status": "error", "message": "Token not found"}
I expected: Slim 404 Handler to be called
PHP 7.2.7
Slim 3.1
JWT-Auth: 3.0

Confirmed, I do not remember why this is the case. Seems to date back to 2.x branch. Changing this behavior by default would be a BC break. So maybe add a configuration switch to honor 404.
Will investigate a bit.

Now I remember.
By default middleware does not have any information about current route. Middleware is also executed for all requests, even those which are made against nonexistent route. As a workaround you could set determineRouteBeforeAppMiddleware => true and throw a NotFoundException when route does not exist. This will exit the middleware stack early.
For example:
<?php

require __DIR__ . "/vendor/autoload.php";

$config = [
    "settings" => [
        "determineRouteBeforeAppMiddleware" => true,
    ],
];
$app = new Slim\App($config);

$app->add(new Tuupola\Middleware\JwtAuthentication([
    "path" => "/",
    "secret" => "supersecretkeyyoushouldnotcommittogithub",
    "error" => function ($response, $arguments) {
        $data["status"] = "error";
        $data["message"] = $arguments["message"];
        return $response
            ->withHeader("Content-Type", "application/json")
            ->getBody()->write(json_encode($data, JSON_UNESCAPED_SLASHES | JSON_PRETTY_PRINT));
    }
]));

$app->add(function ($request, $response, $next) {
    $route = $request->getAttribute("route");

    if (empty($route)) {
        throw new Slim\Exception\NotFoundException($request, $response);
    }

    return $next($request, $response);
});

$app->get("/test", function ($request, $response, $arguments) {
    print "Brawndo!";
});

$app->run();

$ curl --include http://localhost:8080/test

HTTP/1.1 401 Unauthorized
Host: localhost:8080
Date: Tue, 14 Aug 2018 09:50:40 +0000
Connection: close
X-Powered-By: PHP/7.1.14
Content-Length: 60

{
    "status": "error",
    "message": "Token not found."
}

$ curl --include http://localhost:8080/nosuch

HTTP/1.1 404 Not Found
Host: localhost:8080
Date: Tue, 14 Aug 2018 09:50:46 +0000
Connection: close
X-Powered-By: PHP/7.1.14
Content-type: text/html;charset=UTF-8
Content-Length: 879

...


Update packages

Signed-off-by: Kevin Cormier<EMAIL_ADDRESS>Related Issue: https://github.com/open-cluster-management/backlog/issues/16315
Description of Changes

Update packages

/cc @fxiang1


Make Mini Chatbot

Finally, a friend who won’t ignore your “how’s the weather” questions.
Create a mini chatbot that responds to simple questions or can conduct small talk, using basic string matching or a small rule-based system.

please assign this issue to me!


python3Packages.nghttp2: fix build

Description of changes
Apparently #205395 broke the nghttp2 python package (and made it to stable too, hence the backport label). Python bindings now need an explicit flag to enable.
However, note from https://github.com/nghttp2/nghttp2/pull/1846 that these bindings are going to be removed entirely in the next release.
Added python package to passthru.tests to prevent this happening again.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

OK.  I suppose it will soon get dropped on nixpkgs master.


Q: Extra mode for the end

Hi Team,
how it is possible to define the ending mode, similar to the starts:
begin: /~\[/,
className: "subst smartquote",
starts: {
  end: /\]/,
  subLanguage: 'javascript',
  excludeEnd: true,
  className: 'subst',
}

This should be the interpolation quotes within string mode and with javascript mode inside. Like this:
'Hello ~[foo + 2] bar'

Now the problem I have, the end quote ] does not belong to "subst smartquote" mode, but to the parent ("string"). And when I set excludeEnd: true, to false, the ] belongs to javascript mode. How I can highlight the end matching?
Thanks in advance!

From my understanding of the problem something as simple as this should work:
{
  className: 'subst',
  begin: /~\[/, end: /\]/,
  subLanguage: 'javascript'
}

Thanks Ivan, but how I put the interpolation quotes (the beginning match and the ending) to the seperate mode, for instance the "smartquote", but not to the javascript content itself.
And how I make the content to be ungreedy, I thought actually the starts feature is for this, sothat the sublanguage mode is ended when we are in the root mode of that language and the ending is matched, like in TextMate.
So that such things would work:
"Be ~[ users[0].name ] good"

When we use begin: /~\[/, end: /\]/,, then we will stop the interpolation after users[0].

Oh, that's a whole different can of worms then :-(
Unfortunately starts won't help here, it starts a new mode unconditionally but that mode is just a regular one, it still needs end or endsWithParent to be closed. A sub-language doesn't affect parsing of the outer language at all, it's actually applied during a second pass when the whole outer mode is parsed up to the next terminator. Which means that dealing with nested brackets and a sub language is impossible as every part between [ ... [ ... ] ... ] will be treated separately.
To make it work we'll need to rework this logic so a sub-language syntax tree would actually be a part of the outer language, and it's going to be quite an undertaking!

Thank you, Ivan, I appreciate your explanation. And such sub-language tree nesting would be much desired. As for now, I'm trying to create the syntax highlighting for a template markup, which accepts not only the javascript interpolations, but also javascript and css style blocks. For instance:
header {
  style scoped {
       div.panel { background-image: url ('foo.png') }
       /* css comments */
       a { text-decoration: underline }
  }
  // ....
}

So as I understood, it is not possible as for now, to select the whole content inside style scoped {} block as css sub-language, while we have to select the block with regexp first, and it is not possible to select recursively the content in curly braces.

Hello again! It's been a while, but after our recent big release I've got a chance to look over this issue again and I think I can offer a solution here:
/*
Language: my-template
Requires: css.js
*/

function (hljs) {
  return {
    contains: [
      hljs.C_LINE_COMMENT_MODE,
      {
        begin: /style\s+scoped\s*\{/, end: /\}/,
        keywords: 'style scoped',
        contains: hljs.getLanguage('css').contains
      }
    ]
  }
}

By using Requires: in the language header you can ensure that the language being embedded is compiled into the hljs object by this moment and then you can simply reuse its whole structure in your contains.
There might be some corner case bugs that I'm not aware of, but the solution itself seems sound to me, and it works perfectly with the snippet you've provided earlier.

Or even this way:
contains: [
  hljs.C_LINE_COMMENT_MODE,
  hljs.inherit(hljs.getLanguage('css'), {
    begin: /style\s+scoped\s*\{/, end: /\}/,
    keywords: 'style scoped'
  })
]

hljs.inherit copies over the entire language mode, so you get not only contains but all the other top-level stuff like keywords, illegals, etc.

Woow, Ivan, that's awesome!! Thanks, I have tested a little bit, and it works) Later I will make the deeper check.

Hi Ivan, sorry for bothering you again, but it seems something wrong with the recursion and the scope in javascript syntax.
{
  className: 'string',
  begin: '"',
  end: '"',
  contains: [
    hljs.inherit(hljs.getLanguage('javascript'), {
      begin: /~\[/, end: /\]/,
      className: 'subst'
    })
  ]
}

The string interpolation works here just fine, until we have a bracket notation inside:
" Foo ~[ baz['qux'] + 2]  "

Then only ~[ baz['qux'] is highlighted as subst, and  + 2] belongs to the string scope.
But the css include from the example above works as expected - } closes the css block,  when being only in the most top level scope.
Have you any ideas what can be here with javascript syntax wrong?
Thank you in advance!

It seems javascript doesn't handle the brackets [], the above javascript interpolation works, when I add to the javascripts syntax simple snippet inside
{
    begin: /\[/, end: /\]/,
}

Do you know any solutions, how to solve this without changing the javascript syntax, or would it be OK to add those changes to the javascript syntax?

Something like this then:
var js = hljs.getLanguage('javascript');

{
  className: 'string',
  begin: '"',
  end: '"',
  contains: [
    {
      className: 'subst',
      begin: /~\[/, end: /\]/,
      illegal: js.illegal,
      keywords: js.keywords,
      contains: [
        {begin: /\[/, end: /\]/, relevance: 0, contains: ['self']}
      ].concat(js.contains)
    }
  ]
}

I had to give up hljs.inherit and assign individual properties because otherwise contains would've been a reference to the javascript's own contains, not a copy, and modifying it would've affected pure JavaScript as well.

Hi Ivan, thanks a lot for your support with this. Returning to the initial subject - how it would be possible to define the extra mod for the ending match only. Something like this would be perfect
    {
      className: 'subst',
      begin: /~\[/, end: /\]/,
      beginClassName: 'meta', endClassName: 'meta',
      contains: [ /*.. */]
    }

I was trying to achieve this with such a workaround:
begin: /~\[/,
className: 'meta',
starts: {
  contains: [
    /* js Modes */
    {
      begin: /\]/,
      endsParent: true,
      className: 'meta'
    }
  ]
}

But having an infinite loop by matching.
Cheers, Alex

OK, it looks like I have managed to find out a woraround to achieve the issues subject.
function createBlock(def) {
  return {
    begin: def.begin, end: def.end,
    className: def.className,
    endsParent: def.endsParent,
    returnBegin: true,
    contains: [
      {
        begin: def.begin,
        className: def.beginClassName,
      },
      {
        begin: def.end,
        className: def.endClassName,
        endsParent: true,
      },
      {
        end: def.end,
        returnEnd: true,
        contains: def.contains
      }
    ]
  };
}

Usage sample:
/*
Language: my-template
Requires: javascript.js
*/

function (hljs) {
  return {
    contains: [
      createBlock({
        begin: /~\[/,
        end: /\]/,
        beginClassName: 'meta',
        endClassName: 'meta',
        contains: hljs.getLanguage('javascript').contains,
      })
    ]
  }
}

@isagalaev If you find this solution appropriate for usage, please close the issue as it is resolved then, and if I find later some issues with the solution I will reopen the issue.
And thank you for your support  :smiley:

Closing this in favor of the pull request using all these ideas — #1037. It's better to work on that one.


android 6.0, ERROR :stop failed  and start failed

due to Authorization ???
if so ,
How to use    "AudioRecorder.requestAuthorization","AudioRecorder.checkAuthorizationStatus"?

start failed due to Authorization!  but how to use  checkAuthorizationStatus???
stop failed is bugs in android  ,resolved  ==>pull request #108

You may need to add this to your AndroidManifest.xml: <uses-permission android:name="android.permission.RECORD_AUDIO" />
Feel free to reopen again if you still have problems!

stop failed is bugs in android  ！
try{
recorder.stop();
}catch(final Exception e){
}

@ReactMethod
  public void stopRecording(Promise promise){
    if (!isRecording){
      Log.e("INVALID_STATE", "Please call startRecording before stopping recording");
      promise.reject("INVALID_STATE", "Please call startRecording before stopping recording");
      return;
    }
    recorder.stop();
    isRecording = false;
    recorder.release();
    promise.resolve(currentOutputFile);
    sendEvent("recordingFinished", null);
  }

I don't understand this bug report. Can you clarify? If you have a solution, also, please submit as a PR. Thanks!

I'm experiencing the same issue here. If you call the stopRecording method when the startRecording promise resolves it throws the "Please call startRecording before stopping recording".
I think the state needs to change before you resolve the promise. You should have to be able to cancel the recording in the promise resolution e.g.
audioRecorder.startRecording().then(res => {audioRecorder.stopRecording()})
The above line throws throws the "Please call startRecording before stopping recording" error at the moment.
@jsierles is this something you can look into?


Add expiry for tokens

adds a configuration value and check for expired tokens, to create auth tokens that expire at a custom time

Hey @lauraannwilliams,
thanks a lot for providing this PR! Great!
Could you have a look at the failing specs on Travis and add some information to the readme about the new configuration option?

@lauraannwilliams Isn't this the same functionality as the devise timeoutable module (http://www.rubydoc.info/github/plataformatec/devise/master/Devise/Models/Timeoutable) gives us?

I actually think this would be a good improvement. The devise timeoutable module doesn't quite accomplish the same thing. Timeoutable will invalidate a session after a period of inactivity, but if you include the same auth token in every request, a new session will be created each time and thus the token will never be reset.

@baschtl I've resurrected this PR (and fixed the specs) https://github.com/baschtl/devise-token_authenticatable/compare/v0.4.6...TetrationAnalytics:v0.4.7 but since there isn't a branch for v0.4.6 I can't submit a PR.

Changes were incorporated in version 0.4.9. Thanks @lauraannwilliams !

Does the introduction of this PR requires a authentication_token_created_at on the users table? I'm haven't change any setting, other than upgrading the gem and I'm getting:
NoMethodError: undefined method authentication_token_created_at=' for #User:0x007f4eff0fb388`
…ctivemodel-4.2.6/lib/active_model/attribute_methods.rb: 433:in `method_missing'
…atable-0.5.0/lib/devise/token_authenticatable/model.rb:  62:in `reset_authentication_token'
…atable-0.5.0/lib/devise/token_authenticatable/model.rb:  73:in `ensure_authentication_token'
/app/app/controllers/api/v2/registrations_controller.rb:  17:in `create'

Yes @leonelgalan See here:
https://github.com/baschtl/devise-token_authenticatable/releases/tag/v0.4.9
Should we make it so this field isn't required, @baschtl ?

I missed there. Specially being a 0.0.x increase, I didn't expect breaking changes. My gut says yes, as long as the setting is optional and off by default the field should too. I'll take a look.

@mikwat Yes, optional would be good. I just did not have the time to reason about it.
@leonelgalan Thanks for the input. You are also welcome to improve the readme via a pull request. :-)

@baschtl Here's a fix (if you create a branch for 0.4.9 I can send a proper PR): https://github.com/baschtl/devise-token_authenticatable/compare/v0.4.9...TetrationAnalytics:optional-token-expires-in
CC: @leonelgalan

Specs are passing: https://travis-ci.org/TetrationAnalytics/devise-token_authenticatable/builds/138157233

@mikwat See https://github.com/baschtl/devise-token_authenticatable/tree/v0.4.9.


Introducing packages in the order of their installation dependency in README

I encountered the following error when following instructions in README to run remotes::install_github("blind-contours/CVtreeMLE@main"):
ERROR: package installation failed
Error: Failed to install 'CVtreeMLE' from GitHub:
  Failed to install 'sl3' from GitHub:

Please check dependencies among the following packages introduced in README file, and describe them in the order that, if installation of A will fail because B hasn't been installed, introduce B before A.
remotes::install_github("blind-contours/CVtreeMLE@main")
remotes::install_github("tlverse/sl3@devel")
install.packages("partykit")
install.packages("pre")
install.packages(c("kableExtra", "ranger", "arm", "xgboost", "nnls", "hrbrthemes", "viridis"))

Sorry about that - I've just updated the readme to have things in the correct sequence for installation.


[Print] - a marker is printed

Randomly a marker is printed

This happens when we rigth-clic the map: no marker is displayed but however it is printed

errata: it happens also with left-clic (without dragging)

I may have seen this problem in Geoadmin as well. It is hard to reproduce though.

You are right, I think it is something similar as #2163.
In emapis, I can reproduce it everytime.

I think this is browser specific in geoadmin.

It was fixed in geoadmin


Support for folder.jpg

What TMM version are you using?
tmm 2.9.5
release, pre-release, nightly, or directly from GitHub/branch?
Release
What is the actual behaviour?
When scraping artwork for shows and seasons, no folder.jpg is generated
What is the expected behaviour?
The show directory should contain a folder.jpg with a scaled down version of poster.jpg with a maximum width/height of 256px and correct aspect ratio.
Each season directory should contain a folder.jpg with a scaled down version of season-poster.jpg with a maximum width/height of 256px and correct aspect ratio. (eg. Season 01/folder.jpg generated from season01-poster.jpg)
Steps to reproduce:

Go and scrape a TV show
folder.jpg should be generated immediately after the mentioned artwork is downloaded and kept in sync when images are changed

is that just your wish or is there any need for Kodi (or any other media center) for this?

I'm sorry but I think nobody will be able to list ALL media players supporting this de facto standard feature for folder images. Even my old Archos AV 320 supported it back in the old days. Other examples are the mentioned Windows Explorer, WMP, Winamp, Poweramp, most cheap chinese hardware DivX players.
Just write the contents of poster.jpg in a low-res copy to folder.jpg whenever the user wants to assure, that the correct folder image is used. Use it when writing season posters, show posters and posters for folder-based movies or movie-sets.
It's less lines of code as this discussion ...

I'm sorry but I think nobody will be able to list ALL media players supporting this de facto standard feature

To be clear, i asked for Media Centers you used and know that implement this feature.

Just write the contents of poster.jpg in a low-res copy to folder.jpg whenever the user wants to assure, that the correct folder image is used. Use it when writing season posters, show posters and posters for folder-based movies or movie-sets.
It's less lines of code as this discussion ...

User feedback and discussion is very important to develop a feature. As we clearly hadn't the idea of doing that, we are asking exactly all the implementation details and also other software implements already this feature to have the correct idea of what you are asking. A huge problem on software development is always communication.
I won't speak directly for tMM but generally for Software Companies having clients ask them to develop a software. While the client has always the correct idea on their minds, they don't express it always as they want it , or express it with less details thinking that everything else they said are enough to fully understand what they ask. Which most of them time is not happening, thats why there are dozens of revisions till the final product.
Not implying this is the case really, we really need to fully understand what you ask. Believe me there's always time for communication. It's better to over discuss something, rather than implementing it way differently than implied.
To proceed with an add-in feature we need to ensure that this feature will be useful for more than one user and that add-in feature will be useful. We have an agenda with things needed to be completed so every suggested idea is being discussed and if agreed to be implemented, is being pushed on a queue based on it's priority.
Having said that, thank you for your examples we will be having a look at it, and will let you know as soon as possible we figure out what to do.

we've migrated v3 to GitLab (https://gitlab.com/tinyMediaManager/tinyMediaManager) along with all issues. The issues for v2 at GitHub will be closed - we will respond to the corresponding issue in GitLab in the future
br
Manuel


Remove unused import

This is not a direct backport but we did this fix on master already. Just remove the import statement to shoosh the linter.

WASM failure is expected, fixed in #3527.

I must have been on crack when I opened this.