Build as static library with the new CocoaPods version

Goals
Use static libraries to reduce the launch time on a app (https://github.com/artsy/eigen/pull/2561)
Expected Results
RealmSwift would build
Actual Results
Redacted/Pods/RealmSwift/RealmSwift/LinkingObjects.swift:56:57: error: cannot convert value of type 'UInt' to expected argument type 'Int'
                                                 count: UInt(len)))
                                                        ^~~~~~~~~

Redacted/Pods/RealmSwift/RealmSwift/LinkingObjects.swift:120:48: error: cannot convert value of type 'Int' to expected argument type 'UInt'
        return notFoundToNil(index: rlmResults.index(of: object.unsafeCastToRLMObject()))
                                    ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                    UInt(                                               )
Redacted/Pods/RealmSwift/RealmSwift/LinkingObjects.swift:129:48: error: cannot convert value of type 'Int' to expected argument type 'UInt'
        return notFoundToNil(index: rlmResults.indexOfObject(with: predicate))
                                    ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                    UInt(                                    )
Redacted/Pods/RealmSwift/RealmSwift/LinkingObjects.swift:138:48: error: cannot convert value of type 'Int' to expected argument type 'UInt'
        return notFoundToNil(index: rlmResults.indexOfObject(with: NSPredicate(format: predicateFormat,
                                    ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                    UInt(
Redacted/Pods/RealmSwift/RealmSwift/LinkingObjects.swift:152:45: error: cannot convert value of type 'UInt' to expected argument type 'Int'
            return unsafeBitCast(rlmResults[UInt(index)], to: Element.self)
                                            ^~~~~~~~~~~

Redacted/Pods/RealmSwift/RealmSwift/List.swift:31:40: error: cannot convert value of type 'Int' to expected argument type 'UInt'
        return descriptionWithMaxDepth(RLMDescriptionMaxDepth)
                                       ^~~~~~~~~~~~~~~~~~~~~~
                                       UInt(                 )
Redacted/Pods/RealmSwift/RealmSwift/List.swift:35:62: error: cannot convert value of type 'UInt' to expected argument type 'Int'
        return RLMDescriptionWithMaxDepth("List", _rlmArray, depth)
                                                             ^~~~~
                                                             Int( )
Redacted/Pods/RealmSwift/RealmSwift/List.swift:85:47: error: cannot convert value of type 'Int' to expected argument type 'UInt'
        return notFoundToNil(index: _rlmArray.index(of: dynamicBridgeCast(fromSwift: object) as AnyObject))
                                    ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                    UInt(                                                                 )
Redacted/Pods/RealmSwift/RealmSwift/List.swift:94:47: error: cannot convert value of type 'Int' to expected argument type 'UInt'
        return notFoundToNil(index: _rlmArray.indexOfObject(with: predicate))
                                    ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                    UInt(                                   )
Redacted/Pods/RealmSwift/RealmSwift/List.swift:118:75: error: cannot convert value of type 'UInt' to expected argument type 'Int'
            return dynamicBridgeCast(fromObjectiveC: _rlmArray.object(at: UInt(position)))
                                                                          ^~~~~~~~~~~~~~

Redacted/Pods/RealmSwift/RealmSwift/List.swift:122:41: error: cannot convert value of type 'UInt' to expected argument type 'Int'
            _rlmArray.replaceObject(at: UInt(position), with: dynamicBridgeCast(fromSwift: newValue) as AnyObject)
                                        ^~~~~~~~~~~~~~

Redacted/Pods/RealmSwift/RealmSwift/List.swift:304:81: error: cannot convert value of type 'UInt' to expected argument type 'Int'
        _rlmArray.insert(dynamicBridgeCast(fromSwift: object) as AnyObject, at: UInt(index))
                                                                                ^~~~~~~~~~~

Redacted/Pods/RealmSwift/RealmSwift/List.swift:318:36: error: cannot convert value of type 'UInt' to expected argument type 'Int'
        _rlmArray.removeObject(at: UInt(index))
                                   ^~~~~~~~~~~

Redacted/Pods/RealmSwift/RealmSwift/List.swift:342:37: error: cannot convert value of type 'UInt' to expected argument type 'Int'
        _rlmArray.replaceObject(at: UInt(index), with: dynamicBridgeCast(fromSwift: object) as AnyObject)
                                    ^~~~~~~~~~~

Redacted/Pods/RealmSwift/RealmSwift/List.swift:358:34: error: cannot convert value of type 'UInt' to expected argument type 'Int'
        _rlmArray.moveObject(at: UInt(from), to: UInt(to))
                                 ^~~~~~~~~~

Redacted/Pods/RealmSwift/RealmSwift/List.swift:374:38: error: cannot convert value of type 'UInt' to expected argument type 'Int'
        _rlmArray.exchangeObject(at: UInt(index1), withObjectAt: UInt(index2))
                                     ^~~~~~~~~~~~

Redacted/Pods/RealmSwift/RealmSwift/ObjectiveCSupport.swift:151:24: error: invalid redeclaration of 'convert(object:)'
    public static func convert(object: @escaping (Int, Int) -> Bool) -> RLMShouldCompactOnLaunchBlock {
                       ^
Redacted/Pods/RealmSwift/RealmSwift/ObjectiveCSupport.swift:144:24: note: 'convert(object:)' previously declared here
    public static func convert(object: @escaping RLMShouldCompactOnLaunchBlock) -> (Int, Int) -> Bool {
                       ^
Redacted/Pods/RealmSwift/RealmSwift/ObjectiveCSupport.swift:146:27: error: cannot convert value of type 'UInt' to expected argument type 'Int'
            return object(UInt(totalBytes), UInt(usedBytes))
                          ^~~~~~~~~~~~~~~~

Redacted/Pods/RealmSwift/RealmSwift/Results.swift:96:87: error: cannot convert value of type 'UInt' to expected argument type 'Int'
        return Int(rlmResults.countByEnumerating(with: state, objects: buffer, count: UInt(len)))
                                                                                ^~~~~~~~~

Redacted/Pods/RealmSwift/RealmSwift/Results.swift:130:48: error: cannot convert value of type 'Int' to expected argument type 'UInt'
        return notFoundToNil(index: rlmResults.index(of: object as AnyObject))
                                    ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                    UInt(                                    )
Redacted/Pods/RealmSwift/RealmSwift/Results.swift:139:48: error: cannot convert value of type 'Int' to expected argument type 'UInt'
        return notFoundToNil(index: rlmResults.indexOfObject(with: predicate))
                                    ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                    UInt(                                    )
Redacted/Pods/RealmSwift/RealmSwift/Results.swift:148:48: error: cannot convert value of type 'Int' to expected argument type 'UInt'
        return notFoundToNil(index: rlmResults.indexOfObject(with: NSPredicate(format: predicateFormat,
                                    ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                    UInt(
Redacted/Pods/RealmSwift/RealmSwift/Results.swift:161:72: error: cannot convert value of type 'UInt' to expected argument type 'Int'
        return dynamicBridgeCast(fromObjectiveC: rlmResults.object(at: UInt(position)))
                                                                       ^~~~~~~~~~~~~~

Redacted/Pods/RealmSwift/RealmSwift/Sync.swift:595:77: error: cannot convert value of type 'Int' to expected argument type 'UInt'
                                                block(Progress(transferred: transferred, transferrable: transferrable))
                                                                            ^~~~~~~~~~~
                                                                            UInt(      )

Steps to Reproduce

Create a Gemfile:

source 'https://rubygems.org'

gem 'cocoapods', git: 'https://github.com/CocoaPods/CocoaPods.git'
gem 'cocoapods-core', git: 'https://github.com/CocoaPods/Core.git'

Run bundle install
Change your Podfile to use

use_frameworks!(false)
use_modular_headers!

Run bundle install && bundle exec pod install
Try to build the app

Version of Realm and Tooling
Realm framework version: ? 3.1.1
Realm Object Server version: ? -
Xcode version: ? 9.2
iOS/OSX version: ? 11.2
Dependency manager + version: ?  The documentation claims that NSUInteger is bridged to Swift as Int, and that does in fact seem to be the case when I create a brand new Xcode project. However, using the actual Realm xcode project or the cocoapods-generated project for a dynamic framework, it's actually bridged to UInt, and I don't see anything obvious that could be causing this to be different.

@ainopara Thanks! I've reported your findings in the CocoaPods issue https://github.com/CocoaPods/CocoaPods/issues/7589

CocoaPods has now merged a fixed for this, so it should work if you use CocoaPods master.

RealmSwift compiles with the fix from CocoaPods, but two warnings have appeared in the client app:
Umbrella header for module 'Realm' does not include header 'RLMMigration_Private.h'
Umbrella header for module 'Realm' does not include header 'RLMSyncManager_Private.h'

I'm using Cocoapods 1.5.0 and RealmSwift with use_modular_headers!, and still run into this issue. How can I fix this issue

Still having the issue with Cocoapods 1.5.2, any updates ?

Tried on CocoaPods 1.5.3 with RealmSwift 3.3.2 and with use_modular_headers! and it is working.

If the warnings are still present, please create a new issue if this is to be resolved in Realm.
Thanks!


[branch-2.0](publish version) reduce publish version not exist log #27332

pick: #27332
when publish version on not continous mow tablets, it will try to publish multiple times. reduce their log.
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Preserve Gmail labels in the mbox as sub-labels of the main mbox label.

Also ignore messages in Trash/Spam.
Don't bother creating Unread label.

I signed the CLA.

Here's what I've implemented:
  --takeout-labels {True,False,sublabels}
                        Preserve Gmail labels from Takeout mbox files. Or
                        optionally, transform them into sublabels. (default:
                        keep them)
  --takeout-no-unread   Don't preserve read/unread state from Takeout mbox
                        files (default: keep it)
  --takeout-spam-trash  Import messages from Spam/Trash labels in Takeout mbox
                        files (default: skip them)

...still hafta test that last one!

Hi @dotysan, please let me know if you don't have time to make the changes I requested so I can take it up myself if needed. :) Thanks!

Unfortunately, I haven't had a chance to test much. Plus, I got hung up trying to decide what SYSTEM labels to honor when importing into a sublabel...
For example, in the current PR, when importing --takeout-labels sublabels without --takeout-no-unread it will honor the UNREAD system label instead of creating a sublabel. This preserves the bold formatting of that status. But it will still create 'Starred' and 'Important' sublabels.
In my un-pushed tree, I've also honored the STARRED and IMPORTANT system labels and subsequent status formatting. Still fiddling with it a bit more. Feedback?

Hi, sorry for the delay in response, I was away.
I think treating STARRED and IMPORTANT as system labels (i.e. not prefixing them) is good, and IMHO there's no need in adding a flag on whether to preserve them like read/unread.
Thanks!

So I'm finally back on this and testing/refining. And just stumbled across an annoying (redundant?) label that is inconsistently applied. https://productforums.google.com/forum/#!topic/gmail/eKveqdKxZDM
Think I'll add the code to totally ignore "Archived" label since it's already implied with the absence of the "Inbox" label. Sound good?

Think I'll add the code to totally ignore "Archived" label since it's already implied with the absence of the "Inbox" label. Sound good?

Yes, if there's no other label it will go to All Mail which is the same as archived. Thanks!


Resolve a11y warnings from Svelte version 3.58

Now that Svelte version 3.58 has been released, a new slew of static a11y warnings needs to be addressed.
To repro the warnings, run the following at the root of this repository on master:
yarn add -D svelte; yarn build:lib;
# ... a11y warnings emitted

This PR is best reviewed by commit because it actually contains a fix.
For each warning, I referenced the upstream Carbon React implementation. There are a couple of cases where the a11y warning can be ignored. There is one fix for ListBoxMenuItem. For the tabIndex -> tabindex capitalization, I don't think there is any user-facing change (Svelte Language Server does not seem to pick up tabIndex).
@theetrain If approved, should this PR not be squash merged? Will standard-version automatically pick up the fix for de921e63ec6bf6862f5e4f8cbb042cea939f8ec9?

Now available in v0.74.0

Thank you!


#256 JSON-RPC Package (& Commons)

Overview
This PR introduces a full compliant JSON-RPC 2.0 interface for request and responses. Specs can be found at jsonrpc.org
What changed

Introduces the Commons and JSONRPC packages.
Adds RPCRequest and RPCResponse compliant with JSON-RPC 2.0 specifications.
Adds RPCRequestConvertible to facilitate decouple of protocol methods from RPC requests.
AnyCodable type moved to Commons package.
Reintroduces Either type.
Also enables the code coverage report for the project. (Was previously missing in SPM)

Notes
Types were introduced in a way to interfere as little as possible with current SDK code. Current request objects can be migrated over time, and ID generation logic can be wrapped over the RPC models.
The JSONRPC types can be considered foundational code and are build with the intention to be flexible, solid, and allow use in other types of application. Feedback over the API and its usage is appreciated.

Looks, very clean and thoroughly tested, I just wonder if it is not too complex fro our needs but looking forward how it will work consumed by SDKs.

We could close https://github.com/WalletConnect/WalletConnectSwiftV2/pull/155 after that changes?

We could close #155 after that changes?

Absolutely

Looks grate to me! Could you please commit a little example of integration JSON-RPC Package in out codebase?

Sure, I'll try doing something on a next PR


Update Publish-a-package.md - VSTS doc

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Create-Packages/Publish-a-package.md
:white_check_mark:Succeeded

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Create-Packages/Publish-a-package.md
:white_check_mark:Succeeded

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Create-Packages/Publish-a-package.md
:white_check_mark:Succeeded

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

@anangaur FYI, always check your changes on the staging server, e.g. https://review.docs.microsoft.com/en-us/nuget/create-packages/publish-a-package?branch=anangaur-vsts-latest-nuget-client, as in this case the indentation was incorrect which resulted in incorrect rendering. I've fixed that, but wanted to make sure you knew.


ENT-6774 Teach Tom to use sha256 when computing checksums

sed 's/md5/sha256/' * && git commit && git push

Thank you for submitting a pull request! Maybe @olehermanse can review this?


Reload issue page after logging time

Currently after logging time, issue form doesn't updated to reflect changes.

Could be dangerous if the user was also editing something on the page?

I think the issue form state (is it in edit mode or not) could be determined by  element visibility.


ID is not defined.

No matter what I do I keep getting this issue.

Wish I could use this tool but for now I am not capable. Help is needed urgently and would be greatly appreciated.

Here is the line that it is referring to.

is it possible for you to share an example .fla and scripts generating this error?

I suspected the problem was my module bundler, browserify. When the libraries where not being imported with browserify it worked. Never got to the bottom of why it didn't work with browserify though.

@cjgammon It is happening again. Every time i try to include it using browserify i get the error shown above.

I have an Animation class which calls the SnapSVgAnimator. and it looks like this
import 'whatwg-fetch';
import config from 'components/config';

export default class Animation {
  constructor(name, width, height, container, callback, fps=30) {
    this.name      = name;
    this.width     = width;
    this.height    = height;
    this.container = container;
    this.fps       = fps;

    this.play = this.play.bind(this);

    let cachedAnimation = window.animationCache[name];
    if(!cachedAnimation) {
      fetch(`${config.basePath}/deps/animations/${name}.json`)
        .then(res => res.json())
        .then(instructions => {
          this.instructions = instructions;
          callback();
        });
    } else {
      this.instructions = cachedAnimation;
      callback();
    }
  }

  play() {
    this.comp = new SVGAnim(this.instructions, this.width, this.height, this.fps);
    this.container.appendChild(this.comp.s.node);
  }
}

If you inspect the generated SVG element does it have an ID?  also if you log out this.comp.s.id what does it return?  The id should be generated by Snap.svg.  Also, there may be some issues with Snap.svg and browserify.  https://github.com/adobe-webplatform/Snap.svg/issues/344

That is probably what is causing the problem. I'll try and see if I can find a work around for it.


Fix findCycle when called following a prior forced build action

@swift-ci please smoke test

Can we get a test case for this?

Yeah, I'll look into it.


New alert route in alertmanager laa apply for legal aid

What:
See ticket https://waffle.io/ministryofjustice/cloud-platform/cards/5c8777ee7a4a6a0039ba24e8
New alert manager route

@pwyborn you've raised this PR against the master branch, can you please do another one for live-0 as well? Thanks!

@alkar - will do - just changed this one slightly to separate out the environments for the relevant slack channels

@alkar - think I have done this https://github.com/ministryofjustice/cloud-platform-infrastructure/pull/202 ?


Repackaging - FireEye Network Security (MMA to AMA Migration)

Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

/Package


make flag -f optional

and use by default label/atlas
we want to "force" the user to run this function with label/atlas due to the ML/MAP methods

Fixed in commit 52a2167.


basePath / assetPrefix is not respected in generated public-files.json

basically, just what the title says.
if you have a basePath or assetPrefix set in your next.config.js the generated public-files.json do not include said basePath or prefix.
workaround is to manually modify the public-files.json to include said prefix.
i can/am happy to create a PR to resolve this.

i'll get on it at some point today

public-files.json should not be used anymore


FDP_DDM_EXT.1 need more information

I am trying to find more info about FDP_DDM_EXT.1, but cannot any publications with them, only another components of FDP class can find be. Could you prompt me where I can find info about FDP_DDM_EXT.1?

@amkgi Are you sure FDP_DDM_EXT.1 is correct? That's not in the Operating System Protection Profile and FDP_DDM isn't even part of the Common Criteria catalog. Where did you see this requirement listed?

@zsmi Yes, I wrote it correctly. But you really won't find FDP_DDM in the CC. I have seen this requirement in this PP and probably this extension is a special case of this PP. But I was hoping that this is not the case and tried to find sources describing FDP_DDM.

Closing ticket as this appears to be a phishing attempt.


Removing defer in hot code - incCollected and incEmitted

After a bit profiling I see that incEmitted and incCollected is still hot, I saw that defer takes most of the time, after reading a bit it looks that defer has a bit of overhead and in small functions/hot code isn't recommended (should be checked in benchmark before checking, this is not a rule)
Reference: http://lk4d4.darth.io/posts/defer/
Diff between master and this branch with defaultEdgeBufferSize = 0:
benchmark                              old ns/op        new ns/op        delta
Benchmark_T1000_P5000_Matches-4<PHONE_NUMBER>3<PHONE_NUMBER>0      -3.70%
Benchmark_T1000_P5000_NoMatches-4<PHONE_NUMBER>1<PHONE_NUMBER>5      +15.88%
Benchmark_T100_P5000_Matches-4<PHONE_NUMBER>       773756805        -32.67%
Benchmark_T100_P5000_NoMatches-4<PHONE_NUMBER>       805973738        -32.47%
Benchmark_T10_P5000_Matches-4          189739122        152311075        -19.73%
Benchmark_T10_P5000_NoMatches-4        155396278        169503289        +9.08%
Benchmark_T10_P500_CountTask-4         20287106         19785667         -2.47%
Benchmark_T10_P50000_CountTask-4<PHONE_NUMBER><PHONE_NUMBER>       +5.87%
Benchmark_T1000_P500-4<PHONE_NUMBER><PHONE_NUMBER>       -3.39%
Benchmark_T1000_P50000_CountTask-4<PHONE_NUMBER>81<PHONE_NUMBER>39     +1.72%

benchmark                              old allocs     new allocs     delta
Benchmark_T1000_P5000_Matches-4        69144          69130          -0.02%
Benchmark_T1000_P5000_NoMatches-4      69146          69132          -0.02%
Benchmark_T100_P5000_Matches-4         56523          56490          -0.06%
Benchmark_T100_P5000_NoMatches-4       56548          56496          -0.09%
Benchmark_T10_P5000_Matches-4          55226          55211          -0.03%
Benchmark_T10_P5000_NoMatches-4        55219          55212          -0.01%
Benchmark_T10_P500_CountTask-4         10940          10940          +0.00%
Benchmark_T10_P50000_CountTask-4       1154692        1154689        -0.00%
Benchmark_T1000_P500-4                 542557         542934         +0.07%
Benchmark_T1000_P50000_CountTask-4     61019034       61024719       +0.01%

benchmark                              old bytes      new bytes      delta
Benchmark_T1000_P5000_Matches-4        4477632        4475088        -0.06%
Benchmark_T1000_P5000_NoMatches-4      4476112        4476880        +0.02%
Benchmark_T100_P5000_Matches-4         3668616        3666392        -0.06%
Benchmark_T100_P5000_NoMatches-4       3671944        3666944        -0.14%
Benchmark_T10_P5000_Matches-4          3585190        3584180        -0.03%
Benchmark_T10_P5000_NoMatches-4        3584604        3584320        -0.01%
Benchmark_T10_P500_CountTask-4         1624756        1624821        +0.00%
Benchmark_T10_P50000_CountTask-4       114281120      114279504      -0.00%
Benchmark_T1000_P500-4                 128125336      128148064      +0.02%
Benchmark_T1000_P50000_CountTask-4<PHONE_NUMBER><PHONE_NUMBER>     +0.00%

I closed #402 because I accidently created the branch based on fork-by-measurement, this is now on based on master.

@nathanielc by the way, I found out a lot of methods (which aren't hot) with one line, like this:
lock()
defer unlock()
return ..

I can change those, but this can too much aggresive

Yeah, let's leave those alone since they are not hot.
On Apr 4, 2016 12:14 PM, "Yosi Attias"<EMAIL_ADDRESS>wrote:

@nathanielc https://github.com/nathanielc by the way, I found out a lot
of methods (which aren't hot) with one line, like this:
lock()
defer unlock()
return ..
I can change those, but this can too much aggresive
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/influxdata/kapacitor/pull/403#issuecomment-205428173

I added comment, hope it explains and warns  :)


Support https://console.cdc.cloud.sap

Please support also https://console.cdc.cloud.sap/url and not only https://console.gigya.com/

The issue is fixed on release 1.7.0
https://github.com/SAP/sap-customer-data-cloud-toolkit/releases/tag/1.7.0


(Base) Lights shine through walls

Light bulbs and tubes shine through walls.

duplicate of  #2576


Compilation error

http://pastebin.com/GCFcQ0Rg

I'm sorry, apparantly I missed to update the commit hashes for the bukkit and craftbukkit repositories in the version information files on the last upstream merge from the spigot repo. I just corrected that and it should compile.
Note: The current version of Migot does not yet contain all the changes that you may know from the MinetickMod repo. For about 3 weeks now, I am having trouble to apply the queued player chunk loading feature, as the client starts doing something that it is not supposed to, as soon as I change the player chunk loading behaviour. This did not happen at all with the 1.7 client. This delays everything, and it is quite frustrating as this is one of the major features that I do not want to leave out.

Thank you! And hopefully you'll be able to find a way to fix the chunk loading soon!

yo do u still support this ?


https for vimeo

The vimeo template is not using https, this vimeos videos won't work on websites that use https

Thanks @Zauberfisch we will update the template to use https as there isn't any reason not too.

Fixed with 32f6d5eabe774f41c3318fcb940415f46b03163b.


Luis AI parity

Feature parity with https://github.com/Microsoft/botbuilder-dotnet

Solved in 4.4.0b1


Add Sections

add section headings to the side of the page (Case Docket Number; Property Address; Defendant Information; Plaintiff Information; Service of Process Address

added sections and made sure questions fit within each section heading


assert failure on modern intel cpus with avx512_bf16

running into this issue: https://github.com/gz/rust-cpuid/issues/45#issuecomment-873001709
you'll probably need to update to a later version of wasmtime once the fix is included in the version released on PyPI.

Got a link to a wasmtime issue?

not yet, but here's the irc log:
https://libre-soc.org/irclog/latest.log.html#t2021-07-02T10:43:34

wasmtime bug: https://github.com/bytecodealliance/wasmtime/issues/3053

Thank you for filing!

Apparently this can be fixed just by updating the wasmtime dependency to v0.23 or later which no longer depends on raw-cpuid:
https://github.com/bytecodealliance/wasmtime/issues/3053#issuecomment-873045298

Fixed in 505780d.

Released on PyPI.

Thanks!


release the code

Is there a plan to release the code?

Is there a plan to release the code?

Hi, the code has been released just now. Thank you for your interest in our work!
Thanks
Yiming


Handling multiple bundles?

In rollup.config.js, I have export default [appBundle, dashboardBundle]
I know I can run plugins: [     livereload()   ] for a single bundle, but since the livereload will listen on a port (and conflict with itself if it's configured in each bundle) what's the best way to run livereload for multiple bundles?

The server variable should be one scope higher:
https://github.com/thgh/rollup-plugin-livereload/blob/master/src/index.js#L16
let server

export function livereload() {
  ...
  if (server) {
    // close server
  }
  server = createServer()
  ...
}


Add filetype settings for vim

One way I could do this is with vim's modeline magic, but I wish there was a more editor generic method.

This was simple. All I had to do was add a .zsh suffix to the end of the temp files used for editing.


Mongoose fails to build

Message:
==> /home/bob/.linuxbrew/bin/gcc-5 -dynamiclib mongoose.c -o libmongoose.dylib
cc1: warning: unrecognized gcc debugging option: y
cc1: warning: unrecognized gcc debugging option: n
cc1: warning: unrecognized gcc debugging option: m
cc1: warning: unrecognized gcc debugging option: i
cc1: warning: unrecognized gcc debugging option: c
cc1: warning: unrecognized gcc debugging option: l
cc1: warning: unrecognized gcc debugging option: i
cc1: warning: unrecognized gcc debugging option: b

I've already got a fix for this; I'll submit it after the freeze is over.

This issue was moved to Linuxbrew/homebrew-core#79


error: no field named 'sec' in struct 'os.linux.x86_64.timeval'

when I try to build this lib in zig 0.13.0 it fails to build with the following:
/home/codespace/.cache/zig/p/1220df8995ceea78a4a37a505fc779ded75725d0606c33fded26103953524dde1619/src/reader.zig:75:22: error: no field named 'sec' in struct 'os.linux.x86_64.timeval'
                    .sec = @intCast(@divTrunc(ms, 1000)),
                     ^~~
/home/codespace/.vscode-remote/data/User/globalStorage/ziglang.vscode-zig/zig_install/lib/std/os/linux/x86_64.zig:274:28: note: struct declared here
pub const timeval = extern struct {
                    ~~~~~~~^~~~~~
referenced by:
    auth: /home/codespace/.cache/zig/p/1220df8995ceea78a4a37a505fc779ded75725d0606c33fded26103953524dde1619/src/auth.zig:21:15
    auth: /home/codespace/.cache/zig/p/1220df8995ceea78a4a37a505fc779ded75725d0606c33fded26103953524dde1619/src/conn.zig:167:25
    remaining reference traces hidden; use '-freference-trace' to see all reference traces

There's a branch for Zig 0.13
For now, master is tracking Zig's master.
When 0.14 is released, I'm going tie my master to Zig 0.14.


Figure out some wrapper for Update/Render clock operations and updating counters/duration variables

This issue is connected to #13
Closing this one.


Null User Object in Result for Get Friends Request

Calling
GET /user/friends/{username}
results in several "results" array entries that contain no user object.
This conflicts with the documentation which suggests the "user" field is mandatory and cannot be null.
Here is an example of one such invalid result:
https://www.deviantart.com/api/v1/oauth2/user/friends?access_token=<...access_token...>&expand=user.details,user.geo,user.profile,user.stats&limit=50&mature_content=true&offset=0
{
    "is_watching":true,
    "lastvisit":null,
    "user":null,
    "watch":
    {
        "activity":true,
        "collections":true,
        "critiques":true,
        "deviations":true,
        "forum_threads":true,
        "friend":true,
        "journals":true,
        "scraps":false
    },
    "watches_you":false
}

After some investigation, this appears to happen at least for some users who've had user name changes.  I noticed there was an entry with a null user object both where an object with their old username should've been and where an object with their new username should've been, making their accounts completely inaccessible.

I can list some of the users who aren't showing up if desired, but am not doing so without permission for privacy concerns.

Closing this ticket due to its age. If this is still an issue, please feel free to reopen. Thanks!


The forecast dataset path in the evaluation.ipynb quick setup guide seems to be wrong

When running the quick setup guide evaluation.ipynb in Google Colab, for variable forecast_path seems to be incorrect.
The command xr.open_zarr(forecast_path) results in error:
xr.open_zarr(forecast_path)
---------------------------------------------------------------------------
FileNotFoundError                         Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/fsspec/mapping.py](https://localhost:8080/#) in __getitem__(self, key, default)
    142         try:
--> 143             result = self.fs.cat(k)
    144         except self.missing_exceptions:

22 frames
<decorator-gen-119> in _request(self, method, path, headers, json, data, *args, **kwargs)

FileNotFoundError: https://storage.googleapis.com/download/storage/v1/b/weatherbench2/o/datasets%2Fhres%2F2016-2022-0012-64x32_equiangular_with_poles_conservative.zarr%2F.zmetadata?alt=media

During handling of the above exception, another exception occurred:

KeyError                                  Traceback (most recent call last)
KeyError: '.zmetadata'

The above exception was the direct cause of the following exception:

KeyError                                  Traceback (most recent call last)
KeyError: '.zmetadata'

During handling of the above exception, another exception occurred:

GroupNotFoundError                        Traceback (most recent call last)
GroupNotFoundError: group not found at path ''

During handling of the above exception, another exception occurred:

FileNotFoundError                         Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/xarray/backends/zarr.py](https://localhost:8080/#) in open_group(cls, store, mode, synchronizer, group, consolidated, consolidate_on_close, chunk_store, storage_options, append_dim, write_region, safe_chunks, stacklevel, zarr_version)
    445                     )
    446                 except zarr.errors.GroupNotFoundError:
--> 447                     raise FileNotFoundError(f"No such file or directory: '{store}'")
    448         elif consolidated:
    449             # TODO: an option to pass the metadata_key keyword

FileNotFoundError: No such file or directory: 'gs://weatherbench2/datasets/hres/2016-2022-0012-64x32_equiangular_with_poles_conservative.zarr'

While it does work for for the obs_path and climatology_path variables, with both datasets being read correctly.

Thank you for pointing this out. I forgot to update the data paths. We switched to using a 64x32 grid without poles, rather than with. This is because with poles, the lat and lon spacings are actually not equal. So just remove the _with_poles from the data paths. I will create a PR that fixes the notebook.


[Suggestion] Make the JEI Cheat not work if the server doesn't have it enabled

Heya,
i'm currently switching from Embers Rekindled over to this mod, and was wondering if you could change the JEI Cheats setting to be server sided, not client sided.
I would prefer to have it disabled on my server, so that no one is at an disadvantage if they enable this setting.
Cheers,
SC

As of version 1.24.1 I disabled this option in general as it's currently broken and doesn't report back the correct amount of ash. Once I get to fix this I'll probably implement it in a way that gives the server setting the priority.


Progress Presentation

The project requires a presentation of progress. Ideally, three slides and a demo should cover it.

Slides

UML
Noteable Achievements
Division of Efforts

Here is the slides to get started.
https://docs.google.com/presentation/d/1ATQ7jU7zqY9q-YHkYRMNu-PTagomzoQq74fDAY7vGIk/edit?usp=sharing

The slides are looking good!


Fix window rank integration tests on [databricks]

Workaround for #4936.
This reverts a change made in PR #4924 to these 2 integration tests regarding running window rank tests that fails on Databricks.
Signed-off-by: Navin Kumar<EMAIL_ADDRESS>

build


ceph-ansible hangs when the command 'ceph --cluster ceph osd pool stats rbd' hangs

When the above command hangs for any reason, cpeh-ansible is stuck waiting forever, Instead it should timeout and throw the fatal warning.

Assigning to Andrew since he knows the background.

the last output i see that hangs forever is

We should identify all the places where this could timeout (a lot of the Ceph CLI commands can hang) and do the same

It seems to me we have a couple options to add a timeout to our ceph commands.

use an approach with until like we do here https://github.com/ceph/ceph-ansible/blob/master/infrastructure-playbooks/rolling_update.yml#L146-L149

use async and poll http://docs.ansible.com/ansible/playbooks_async.html

Does anybody have a preference? Also, what's a good timeout default for these ceph commands?

For context, this is the upstream ceph bug related to the ceph command making ceph-ansible hang. http://tracker.ceph.com/issues/19600

None of these commands should take longer than 5 minutes (exaggerating here).  I don't feel authoritative enough to pursue a blanket solution like that one. @dmick do you know of anything or any environment that could take longer than 5 minutes?

Although until is more Ansible and possibly more elegant too, the timeout command is part of coreutils on RedHat/Debian based systems so widely available.
That could be a simpler solution.


Java: support Map

Map<String, Integer> m = new HashMap<>();
m.put("two", 2);
m.put("three", 3);
System.out.println(m);

has output
{two=2, three=3}

which I think is quite acceptable


Skip empty query parameters

Currently, a key without a value fails to deserialize, even if the field is wrapped in an Option. Semantically, no key and a key with an empty value is the same (I think?). This patch filters keys without values before the deserialization starts.
This allows us to deserialize things like value= if the value is an Option. If value is not an option, it will fails as expected that it is missing.
Let me know what you think.
I stumbled across this when implementing a form that has optional number fields. It would fail to deserialize with "cannot parse float from an empty string".

I don't agree that empty values should be treated as None or skipped when deserializing a Vec of values, sorry. This would also break deserialization of non-optional &str fields that are sometimes empty.

I don't agree that empty values should be treated as None or skipped when deserializing a Vec of values, sorry.

Okay, how would you handle the case of an optional number field then? Currently, deserialization of my form (in axum) fails if the field is empty. I'd have to make it a Option<String> and then manually parse as f64.

This would also break deserialization of non-optional String fields that are sometimes empty.

Does it? You can just put #[serde(default)] on it, right?
I tried to come up with a case where an absent parameter (no key & value) is semantically different from just the key being present but I couldn't think of anything. Perhaps boolean values like "path/path/path?foo" but I am not even sure whether that deserializes today!

I don't agree that empty values should be treated as None or skipped when deserializing a Vec of values, sorry.

Okay, how would you handle the case of an optional number field then? Currently, deserialization of my form (in axum) fails if the field is empty. I'd have to make it a Option<String> and then manually parse as f64 because I'd get Some("") then.

You can use a custom deserialization function for that, like this one.

This would also break deserialization of non-optional String fields that are sometimes empty.

Does it? You can just put #[serde(default)] on it, right?

Yes, it wouldn't make it impossible to let people work around it but I would definitely call it a bug if this didn't work by default.

I don't agree that empty values should be treated as None or skipped when deserializing a Vec of values, sorry.

Okay, how would you handle the case of an optional number field then? Currently, deserialization of my form (in axum) fails if the field is empty. I'd have to make it a Option<String> and then manually parse as f64 because I'd get Some("") then.

You can use a custom deserialization function for that, like this one.

Thanks, will try that!

This would also break deserialization of non-optional String fields that are sometimes empty.

Does it? You can just put #[serde(default)] on it, right?

Yes, it wouldn't make it impossible to let people work around it but I would definitely call it a bug if this didn't work by default.

Fair. Would you accept a PR that fixes the bug I am experiencing without this downside?

Would you accept a PR that fixes the bug I am experiencing without this downside?

You would first have to convince me that it's a bug at all. My current stance is that this is working as it should be.

Okay, this is my attempt 😁

I have an HTML form with an optional input of type number
My typed form in axum uses Option<f64> for this field
Submitting the form without a value makes the browser send value=
The request fails with "cannot deserialize f64 from an empty string"

I don't see how this is working as it should be. How am I meant to handle optional fields if not by wrapping them in Option?

Alright, convinced 😄
How about just changing the new Option deserialization code from #3 to call visit_none if the value is empty?

Alright, convinced :smile:

Yey!

How about just changing the new Option deserialization code from #3 to call visit_none if the value is empty? (although I wonder what will happen with Option<Vec<u8>> and value=&value=2 then)

That code branch unfortunately never gets called. The problem is within the MapDeserializer where we have a key but the value is an empty string. I am not a serde expert but by the time we are within our code again, we no longer have knowledge of the Option I think.

False alarm, it does work actually!

I pushed another commit. Can you re-open the pull-request?

I opened a new PR here: https://github.com/jplatte/serde_html_form/pull/5


Implement fuzzy search

Here's a wish: Could you implement sublimetext-like fuzzy search when giving column/table/etc auto-completion suggestions?
For example, in a WERE clause,  when i type "pid" and pres Ctrl+Space, the popup window should contain person_id and image_profile_id since they match (fuzzily).
I feel this would be a great addition.

Good point, I like it.

It's really nice!
It seems with auto complete implemented in Eclipse Mars.


how to promisfy Twig.renderFile

there is a way to promisy  Twig.renderFile or  create a new method like node 8
await Twig.renderFileSync ()?
I've tried to follow this method:
https://stackoverflow.com/questions/46717503/how-to-promisify-a-function-that-has-multiple-callback-parameters
but renderFileSync  need a function like parameter with content :
Twig.renderFile(fSourceFile2, templateGlobalValues, (err, txtContent) => {
(...)
anyone has did it ?

Hi @webmobiles ,
Twig object already has a method that is promisified.
Try to this:
Twig.renderFileAsync(fSourceFile2, templateGlobalValues)
.then(txtContent => /*do something*/)
.catch(err => /*do other things*/);


Support streaming services for JSON transcoding

Currently, Armeria's HttpJsonTranscodingService doesn't support streaming messages and throws an exception.
https://github.com/line/armeria/blob/104855af85783b40aba4750a15b5b79159f65e3a/grpc/src/main/java/com/linecorp/armeria/server/grpc/HttpJsonTranscodingService.java#L162-L165
Recently, we saw an issue where some users with streaming services in their proto files weren't able to use transcoding at all since the above exception was thrown. https://github.com/line/armeria/issues/5828 https://github.com/line/armeria/pull/5829
Although a workaround was added to log a warning instead of throwing an exception, this also probably isn't ideal since users will still see warning logs for which they can't (and shouldn't) take action.
Moreover, it seems like grpc-gateway supports streaming as seen in the following issue where ndjson is used: https://github.com/grpc-ecosystem/grpc-gateway/issues/581
In an effort to 1) remove this discrepancy with upstream and 2) remove unnecessary warning logs, I suggest that we also support streaming messages for HttpJsonTranscodingService.

Note: At least to follow-up on https://github.com/line/armeria/pull/5829, we may want to consider returning a 405 Method Not Allowed when a request to a streaming service is made until this issue is fully addressed.

Hi @jrhee17 , Is this still available?? I would like to contribute to this in case it's available.

Sure, assigned

Hi @jrhee17 ,
I followed the docs and tried to build Armeria locally using ./gradlew --parallel build but faced some failed test cases.

Hi @jrhee17, I have successfully set up the dev environment in locally.
I went through the HttpJsonTranscodingService.java file. Currently, we are just logging in case of nonunary.
Currently, two things I can see are mentioned, but I am not sure how to approach them yet. Can you please explain in brief or redirected me to some resource which might help me.

In an effort to 1) remove this discrepancy with upstream and 2) remove unnecessary warning logs, I suggest that we also support streaming messages for HttpJsonTranscodingService.

Currently once HttpJsonTranscodingService receives a request, the request is aggregated in memory until the end of stream is received.
https://github.com/line/armeria/blob/main/grpc/src/main/java/com/linecorp/armeria/server/grpc/HttpJsonTranscodingService.java#L640
In order to support streaming client streaming rpc, we will likely need to

divide the request according to a delimiter (newline)
convert each json into a protobuf
delegate the newly created request body to the logic processing gRPC requests (ref)

In order to support server streaming rpc, we likely need to do something similar to the response body. i.e. modify to convert each protobuf message to a json, and then send each newline delimited json.
https://github.com/line/armeria/blob/1e5cc2848ad2f153792ee1c34df7c535438e4c73/grpc/src/main/java/com/linecorp/armeria/server/grpc/HttpJsonTranscodingService.java#L455
Lastly, using reactive streams may not be performant so we may want to keep the current logic for unary and add a separate code path for streaming requests/responses.
We can probably check the content-type in the headers to determine whether a request is meant to be unary or streaming.

I missed this comment: https://github.com/line/armeria/issues/5832#issuecomment-2387838672
You may specify ./gradlew -Pretry=true -PfailOnPassedAfterRetry=false to retry flaky tests automatically

I would like to contribute to this issue. Can you please assign it to me.?

First, thanks for your interest. However, I don't think this issue is good for the first contributor. This issue requires some experience with Armeria internals and gRPC specifications to implement.
What do you think of finding other issues that are simpler and do not need to modify many files? It might be good to warm up and get familiar with the Armenian code base while working on an easy issue.

What do you think of finding other issues that are simpler and do not need to modify many files? It might be good to warm up and get familiar with the Armeria code base while working on an easy issue.

How can I find easy or good to-start issues in armeria??

We've added a good first issue label to issues that we thought had low difficulty and its scope is limited. https://github.com/line/armeria/issues?q=sort:updated-desc+is:issue+is:open+label:"good+first+issue"


Consider forking workflow

We need a way for projects to allow work to be done in a fork that does not create inconsistent states of the main project.
The goal would be that the main project should always be in a releasable state. This is needed for both higher frequency in releases, e.g. weekly, more involvements from more people and also an extended catalogue which automatically augments data more frequently.
A forking workflow with check ins and outs would also allow us to record smaller "commits" that document the changes applied to a project. This could replace the manually created, singular whats new document in our metadata for a release. Instead every "commit" would be documented (e.g. updated ITIS to version xyz, added new sector, etc).

@olafbanki @dhobern @thomasstjerne we should discuss/design in this issue a better workflow for projects with multiple editors. It is important to start this, I don't see that the current state can be scaled to multiple editors!


How to add description to a group

Hi guys, unable to find an option to put any additional info for the whole group info. Is it possible?
What I am looking for is something like this:
config.define_group(:public) do |public|
    public.docs_dir = config.docs_dir + 'public/'
    public.api_name = "Public API Header"
    public.api_explanation = <<~EXP
... html markup for the into ...
EXP
    public.filter = :public
  end

is there a way to do this?

There is not a way to do this currently. I'd be happy to review a PR if you added it though!

Ok, Im on it
ср, 16 авг. 2017 г. в 21:03, Eric Oestrich<EMAIL_ADDRESS>
There is not a way to do this currently. I'd be happy to review a PR if
you added it though!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zipmark/rspec_api_documentation/issues/354#issuecomment-322852289,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkTuGFF1UV3wG1bHL0gHrD08XzXMrsyks5sYy77gaJpZM4O3Yl-
.

@oestrich Sumbmitted #356, would you mind to take a look


PySide6 support

Adopted from #178, as I want to not mix this up with other changes. Currently blocked by https://github.com/spyder-ide/qtpy/issues/487. Running debug.py with qt_api="pyside6" will fail because of this, and I couldn't figure out why.

qtpy issues resolved by using pyside6 <6.7 https://github.com/spyder-ide/qtpy/issues/487#issuecomment-2118829698

Looks good to me now, didn't spot any obvious issues, but haven't looked closely at the displaying problems that were mentioned in the original PR. Since it works as before with pyside2, I think this can be merged.


Refactoring stuff

Refactor Header and Subscribe components to remove code duplication
Improve responsive layout for subscribe buttons
Save reused boolean expression to a variable in Show component.

@davidleger95 Hey David 👋🏼 Thanks for taking the time to do this. Would you be interested in doing a bigger scope re-factoring project. Getting rid of stylus and moving over to styled-components? Tracking that here: https://github.com/wesbos/Syntax/pull/162

@416serg yeah for sure! sounds fun! :)


Error: uncaught at _default at _default

Happens the second time you navigate on a carrousel

Fixed at https://github.com/wp-pwa/wp-org-connection/commit/1c53c6772746023925c7a31b0529fea8cfb492d2


Go syntax highlighting support for IntelliJ/Goland 2019.3

Related to #69, #70

Like already documented and fixed in #70, IntelliJ/Goland version 2019.3 also changes in Go's syntax highlight for the default bundled color schemes.
This again requires to explicitly define the values for some attributes in order to achieve the same highlight like in previous versions that are matching Nord's style guidelines.

reinstall plugin, it's work well

Yeah, but that's only a workaround. I've submitted #109 instead to ensure it works out-of-the-box after an IDE version update.


Release sdk/resourcemanager/streamanalytics/armstreamanalytics v0.1.0

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

/azp run go - armstreamanalytics - ci


Allow customization of SSL client verification

It would be really useful to have one or more of the following for dealing with non-public certificates (which often come up in corporate environments):

specifying a trust store
turning off host verification

Very good idea! I'm really busy at the moment with other projects so it might take a while until I can implement this feature. If you need it faster, then don't hesitate to send a pull request. It would be very much appreciated :-) Thanks.


Add a Buildkite pipeline

We should also have CPU linux

Windows isn't necessary

Thanks, looks good to me


Where can I download the engine?

you'd better add the website of the third-libraries.

check the README.md file plz.


query for time spent in state

Feature Request
Proposal:
A query that returns the time elapsed between state changes, and ideally including the start and end of each state.
Current behavior:
This is a common IoT use case, measuring how long a system is in one state or another, and also recording the times when state changed. Currently there's no way to measure elapsed time except with ELAPSED(), which only works on successive points.
Desired behavior:
Assuming field has possible values of open, closed, and halfway, then there should be a function FOO() such that
SELECT FOO(value) FROM bar WHERE time > 00:00 and time < 01:00 would return something like:
open
00:00-00:10, 10 minutes
00:45-00:50, 5 minutes

closed
00:10-00:30, 20 minutes

halfway
00:30-00:45, 15 minutes
00:50-01:00, 10 minutes

and perhaps SELECT SUM(FOO(value)) FROM bar WHERE time > 00:00 and time < 01:00 would return:
open
15 minutes

closed
00:10-00:30, 20 minutes

halfway
35 minutes

Use case:
Primarily for graphing of state changes and for calculating time spent in state. This is a common request from customers and I am happy to provide specific use cases.

@toddboom you had some ideas for this issue, yes?

I would try to return something like this (in pseudo-data):
[{
  time: "2016-07-19 12:00:00",
  duration:<PHONE_NUMBER>,
  value: "open"
},
{
  time: "2016-07-19 12:00:05",
  duration: 500000000,
  value: "closed"
}]

So, for a given time range, you would see the start time and duration (both in nanos) and the value for that interval. It should be pretty easy to build by iterating through the data in a straight line.
Maybe the duration isn't even necessary?

We would be very happy about a function like the above mentioned one. The solution proposed by @toddboom is fine as long as the returned points/ranges cover the whole query interval.
The duration can be omitted if no duplication of information is wanted. Alternatively, the following response data would be great:
[{
  "from": "2016-07-19 12:00:00",
  "to": "2016-07-19 12:00:05",
  "value": "open"
},
{
  "from": "2016-07-19 12:00:05",
  "to": "2016-07-19 12:00:20",
  "value": "closed"
}]

One additional thought: it would be great if successive data points with the same value would not result in multiple return intervals with the same value but be merged automatically.

closing in favor of #7581


Call loop in provided uptime/boot_time methods

We are currently using this crate and are implementing the Platform trait for a dummy/test platform for our unit tests.
The trait provides methods uptime and boot_time, but they each call each other, leading to a stack overflow unless one overrides either of them (as is done in the implementation for Linux where uptime is overridden).
See also: https://github.com/myfreeweb/systemstat/blob/master/src/platform/common.rs#L37 and https://github.com/myfreeweb/systemstat/blob/master/src/platform/common.rs#L45.
In our opinion, the uptime method should not be provided but required or this should be documented at the minimum.

the uptime method should not be provided but required

But on many other platforms, it's the boot_time that's overridden.
I do not want to pick either of them as the "blessed" one that everyone should implement. That's the whole point, you need to implement one, but it doesn't matter which one, use whatever is easier on that platform.

this should be documented at the minimum

Sure, I can do that.


Most container fields should not be nullable but are

A lot of the container fields are nullable.  There is different between required and nullable.  Most fields should be optional, but there is no reason to accept null as a valid value.  This causes a specific issue with restartPolicy today, if you set it to null it will fail in python agent.

Closed in favor of rancher/rancher#3851


Bot Responses and User Input navigation pane not aligned

Describe the bug
This is not a functional issue. It's just that the navigation panes are not aligned as they used to be.
In my personal opinion, the previous alignment looks better.

Current Bot Responses navigation pane

Previous Bot Responses navigation pane

Current User Input navigation pane

Previous User Input navigation pane

Version
The latest master branch.
Browser

[x] Chrome

OS

[x] Windows

To Reproduce
Steps to reproduce the behavior:

Go to 'Bot Responses...'/ 'User Input'
See the described issues.

Expected behavior
Should be aligned.

addressed via https://github.com/microsoft/BotFramework-Composer/pull/2799


Feature: at-styled time arithmetic operations

Description
I frequently use the Unix at command for scheduling jobs to start during dynamic time windows. Often times, it's important to delay the time windows by grouped time deltas. Could I humbly request help from fend to make this easier in shell scripting? Adding time blocks (like at this website ) would make a big QoL improvement for me.
Does this seem like something that is an appropriate feature request?

That’s a great idea, thanks! I’ll look into adding this soon.

That’s a great idea, thanks! I’ll look into adding this soon.
Thank you so much!!

Maybe when introducing time, timezone conversion could also be something great to have


Custom Vision Service Documentation: There are no separate Training and Prediction keys.

The documentation mentions, and shows screenshots for, separate Training and Prediction keys, however both the Azure Portal and customvision.ai project settings show a single key.

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

ID: 0b47b105-296c-139f-7e2c-d4d143aa1ccd
Version Independent ID: 7c55bf3c-739f-8e7d-6dee-40409ffbe548
Content: Quickstart: Create object detection with Custom Vision SDK - Azure Cognitive Services
Content Source: articles/cognitive-services/Custom-Vision-Service/quickstarts/object-detection.md
Service: cognitive-services
Sub-service: custom-vision
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@david-cornish
Thanks for the feedback! We are currently investigating and will update you shortly.

@david-cornish Hi David, sorry for your experience. The reason why you haven't got two keys is because when you trying to create a new resource, there are 3 options like the screenshot:

You can select both or do it separately.
If you do it as both, you will get result as following:

Please try again and let me know if you have any question. Thanks.

@david-cornish
We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion. Let me know if you have other question.


Face2Face dataset in your paper？

Hi, how can I present the result in table 4 in your paper? Your download link does not obtain face2face dataset！Thanks！

3.1.2 Face2Face dataset

[...]

the FaceForensics dataset [20] contains over a thousand forged videos and
their original using the Face2Face approach.

[...]

References

[...]

[20] A. Rossler, D. Cozzolino, L. Verdoliva, C. Riess, J. Thies, ¨
and M. Nießner. Faceforensics: A large-scale video dataset
for forgery detection in human faces. arXiv preprint
arXiv:1803.09179, 2018.


Restores default dispatch return value

Fixes #24

Could you make sure this also works on the Main process? (electronBrowserEnhancer)

Also could you give a brief description of what you did to test this?  Sorry I don't have unit tests 😞

@samiskin, no problem, should be good now! Tested both the same way to make sure it was working correctly.


[HOLD for payment 2023-09-07] [$1000] Android - Back button is overlapping with notification bar

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open app and sign in with an account that doesn't have lounge access
Using deeplink, open https://staging.new.expensify.com/settings/profile/lounge-access

Expected Result:
Back button must be visible to user
Actual Result:
Back button is overlapping with notification bar
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: v1.3.55-3
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

https://github.com/Expensify/App/assets/115492554/0b91ef9f-ce7c-4d9a-b03c-f775c6d15f69
Expensify/Expensify Issue URL:
Issue reported by: @rushatgabhane
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1691486921365289
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01995c8bbede070619
Upwork Job ID:<PHONE_NUMBER>241669120
Last Price Increase: 2023-08-25
Automatic offers:

hoangzinh | Contributor | 26334470

Proposal
Please re-state the problem that we are trying to solve in this issue.
The back button is overlapping with the notification bar in lounge acess page when we are not allowed
What is the root cause of that problem?
There is no ScreenWrapper surrounding the FullPageNotFoundView
What changes do you think we should make in order to solve the problem?
Add a ScreenWrapper around the FullPageNotFoundView in LoungeAccessPage
What alternative solutions did you explore? (Optional)

Yeah, the button is definitely in the wrong place here.

The button is wrong-aligned, There are lots of ways.
1- Margin from top
2- Use CSS positioning

I am out of upwork connects. 😒
Please send me an invitation on this upwork profile: https://www.upwork.com/freelancers/nekores
I will fix this.

I actually don't reproduce

@ShogunFire this only happens in native, not in mobile web. To reproduce, you can send the deeplink as a message to another user in the native app, and then click on it there.

@joekaufmanexpensify Ok so that's what I did but didn't reproduce

I just tried again and was able to reproduce on android native.
https://github.com/Expensify/App/assets/54866469/0a7bb021-e3d0-4459-9326-195cfb9d2cab

@eVoloshchak could you take a look at the above proposals when you have a sec? TY!

Pending review from @eVoloshchak

Bumped via slack

Both proposals are effectively achieving the same result, but I think @hoangzinh's proposal is better.
We should indeed use NotFoundPage, it's consistent with other pages and it's the same thing as wrapping FullPageNotFoundView with ScreenWrapper, but with a single component.
🎀👀🎀 C+ reviewed!

Sounds good to me 👍 Thanks for reviews.

@eVoloshchak mind completing the BZ checklist here so we can prep to issue payment?

This one qualified for speed bonus, so we need to make the following payments here:

@hoangzinh - $1,500 ($1,000 for PR + $500 speed bonus). Paid via Upwork.
@eVoloshchak - $1,500 ($1,000 for C+ review + $500 speed bonus). Paid via NewDot.
@rushatgabhane - $250 for reporting bonus. Paid via NewDot.

The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/23061
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/23061/files#r1318512682
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: this is purely a visual bug, I don't think an additional discussion is needed
Determine if we should create a regression test for this bug.
Is it easy to test for this bug? Yes
Is the bug related to an important user flow? No
Is it an impactful bug? No
This isn't an impactful bug, it doesn't prevent users from using the app, I don't think regression test is needed here

Thanks! BZ checklist complete. All set to issue payment.

@hoangzinh $1,500 sent and contract ended!

@rushatgabhane could you please request $250 via NewDot and confirm here once complete?

@eVoloshchak could you please request $1,500 via NewDot and confirm here once complete?

@joekaufmanexpensify, requested the payment

Reviewed the details for @eVoloshchak. $1,500 payment approved for payment via NewDot based on BZ summary.

Great, thanks for confirming! Closing as all that's left is to pay Rushat's request. Payment summary message is here.

$250 payment approved for @rushatgabhane based on BZ summary.


Support "Arbitrary Module Namespace Identifiers"

This PR implements Arbitrary Module Namespace Identifiers, which is a JavaScript syntax feature that was added to JavaScript over two years ago (in ES2022). It allows import and export aliases to be strings instead of identifiers.
Before this PR:

Notice that the string content is incorrectly colored. In particular, the characters default in the middle of the string are confusingly highlighted as a keyword. The preceding type keyword is also not highlighted as a keyword when it should be.
After this PR:

All strings are highlighted as strings, and the type keyword is highlighted as a keyword.

Update: This syntax feature has now been added to TypeScript and shipped in VSCode. However, VSCode shipped it with broken syntax highlighting because this PR wasn't merged. Merging this PR would fix syntax highlighting for this feature.


Karma fixes for latest versions

So, here is some fixes that closes #2035 and #2044
First off, one of the issues were that we are testing across 3 versions of npm. They have different behaviours when encountering peerDependencies. So ideally I would almost say we should upgrade npm to at least npm@2 for node 0.10. But open to opinions there. This was the least disruptive change, in my opinion.
Second, one of the issues were pointing out that the response object changed for one of the tests. So I ran the same test code with node, where the result was illuminating the fact that the browser test was testing for something only happening in the browser. So I changed that test, so it can check for a result that also is the same result when running on node.

Also, let me know if I should squash these commits. Had to do a couple of runs to get the suite to run it on travis to be sure :)

Your commits are great for explaining what's going on. I often find myself scrolling through the last commits for a project, so from a usability point of view clicking on a single and more meaningful commit there is easier. Of course this can be approach in many different ways, but in this case one commit might be better.
Also see how I'm using the Fixes and Closes to actually close those PRs and Issues once this gets merged in. Probably another syntax might work too, but I don't think GitHub is smart enough to link those issues from:

So, here is some fixes that closes #2035 and #2044 and #2056

Thanks for taking the time to work on this :+1:

Also see how I'm using the Fixes and Closes to actually close those PRs and Issues once this gets merged in

I am aware of the syntax :) Just wanted to make sure the issues were being referenced, for easier navigation.
Updated the branch and used this to indicate what is being closed and fixed.

Thanks :cat2:


Set $proxy_upstream_name before location directive

When nginx performs ssl redirect, $proxy_upstream_name used in log
is not initialized because it is set after nginx matched a location directive,
which is not the case when performing a ssl redirect.
refs #711

This change is

Coverage decreased (-0.03%) to 46.901% when pulling a83f17c716d8b82ba692762795f751e2a612ab4c on cheungpat:issue-711 into 12d2c4f6891bb83e5f62f35d34024ee8fd81b046 on kubernetes:master.

/lgtm

@cheungpat thanks!


Create dialog fixes [no merge]

UI bug styles (https://github.com/CartoDB/cartodb/issues/2330).
Add submit button near the URL input when connecting a dataset (https://github.com/CartoDB/cartodb/issues/2398).
Incorrect link on connect dataset modal (https://github.com/CartoDB/cartodb/issues/2397).
Disable click on the dataset title on the create new map modal window (https://github.com/CartoDB/cartodb/issues/2393).
Typo in services header template ((https://github.com/CartoDB/cartodb/issues/2350)

retest this please

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

retest this please

Frontend tests were OK :+1: (details)


BIDS compatability

add flag so that ndmg_setup will grab all appropriate filenames and generate list files from a base directory input if data is organized according to bids spec

closed with https://github.com/neurodata/ndmg/commit/a0516b832f62cd180c19fd43c08e972295657af2


[Component Proposal] Placeholder Image

What package is this for?

[x] React
[ ] Angular

Summary
Placeholder images indicate to users that an image (that is not currently displayed) is meant to occupy a designated space.

Available extra resources
PAL guidelines & Specs: https://pages.github.ibm.com/ai-applications/design/components/placeholder-image/style

@davidicus I guess we could implement this into  or a few other places, but PAL does not have the concept of an  component, which we do have in Graphite and does follow this pattern, making me think this is a bit of a redundant story

This is something that was contributed from TRIRIGA. You make a good point that we don't have a generic image component, but would it make sense to make this a separate placeholder image component? or does it need to be tied to an image component, and this is just the "empty state" variations of it


HAWQ-1107 - enhance PXF HDFS plugin documentation

added more examples, restructured the content, removed SequenceWritable references.

made the requested changes.  and also restructured the content to include the specific file creation/manipulation steps in the relevant profile section.
note: needed to make a subnav change as well.
ready for final review.  thanks.

Thanks for the edits, Lisa.  I think moving the sample file creation (and clearly labeling the associated procedures as Examples) really helps the flow.  I just had one last minor edit, that you can choose to deal with later or within this PR.  I noticed the use of:
Note: (italicized, to call out a note)
Notice:  and,
Notice that (not a note callout).
It would be great if this could be consistent.  In general, we're trying to limit the use of Note: callouts somewhat, preferring to call out Warning: and Caution: instead.  But if it's truly a note that you don't want to make into a regular sentence, let's stick with Note: in bold.

thanks for reviewing, kavinder.


Fix docstrings

Hi @goedman, just fixed a few typos in the docstrings.

Thanks again Pietro!

You're welcome. Let me know if you want me to join the organization with "write access" so that you don't even need to review such minor changes.

Hi Rob, thank you for the invitation.
I'll start the focused reading of the textbook in a few months... so don't worry about that.


bugfix: ModuleNotFoundError: No module named 'ruamel' (#97)

Codecov Report

Merging #101 (386b49a) into 0.5.x-rc (004500d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           0.5.x-rc     #101   +/-   ##
=========================================
  Coverage     81.19%   81.19%
=========================================
  Files            33       33
  Lines           755      755
=========================================
  Hits            613      613
  Misses          142      142

Continue to review full report at Codecov.

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


[Cognitive] remove unneeded parameter in Account Create

Per ./services/cognitiveservices/mgmt/2017-04-18/cognitiveservices/models.go#L371, when a CognitiveServicesAccount is created it must include a user-provided empty interface{} as one of its parameters.
This parameter appears to be useless. Is it, and if so can it be removed from the signature (as a breaking change I suppose)? Thanks!
@tombuildsstuff thanks for pointing this out!

@felixwa why is an empty object required and will it be fixed any time soon?

Swagger reference here.

This has been moved to the specs repo, see https://github.com/Azure/azure-rest-api-specs/issues/3157.


UI Shows double file names for some songs

Basically some songs show double file name, while in spotify for example, this doesnt happen with that same MP3

Just noticed its the same with Album names

@antonioxsimic Can you send me a sample song?

I uploaded it here in drive
https://drive.google.com/file/d/1rN4-Wa1VFcuE6lIJxabjrd2oNTGgICPF/view?usp=sharing

@antonioxsimic

Nora is showing the correct metadata from your song.
It has only stripped \\ characters from metadata.

What do you think why isn't the same thing visible in Foobar or other metadata editors? I have never typed "Pjevacica", i typed "Pjevačica", with european characters


yarn doesn't always install dependencies

VSCode Version: 1.19-Insider
OS Version: macOS

Node: 8.9.1
yarn: 0.16.0
Followed steps in https://github.com/Microsoft/vscode/pull/38481

nvm install 8.9.1
npm i -g yarn
git clean -xfd
yarn

Most of the time I got errors when installing vscode-ripgrep
[3/4] 🔗  Linking dependencies...
[4/4] 📃  Building fresh packages...
[8/16] ⠐ v8-profiler: CXX(target) Release/obj.target/profiler/src/profiler.o
[9/16] ⠐ vscode-fsevents: spawn args   '-Goutput_dir=.' ]
[10/16] ⠐ vscode-ripgrep
[4/16] ⠐ native-keymap: CXX(target) Release/obj.target/keymapping/src/keyboard_mac.o
error /Users/penlv/code/vscode/node_modules/vscode-ripgrep: Command failed.
Exit code: 1
Command: sh
Arguments: -c node ./dist/postinstall.js
Directory: /Users/penlv/code/vscode/node_modules/vscode-ripgrep
Output:
Downloading ripgrep failed: Error: Missing ./_node_modules...

I succeed installing all the packages twice, after that, I ran yarn watch or gulp watch, I got following errors
yarn watch v0.16.0
$ gulp watch --max_old_space_size=4096
[11:47:44] Node flags detected: --max_old_space_size=4096
[11:47:44] Respawned to PID: 3858
child_process.js:644
    throw err;
    ^

Error: Command failed: yarn list --json
    at checkExecSyncError (child_process.js:601:13)
    at Object.execSync (child_process.js:641:13)
    at getYarnProductionDependencies (/Users/penlv/code/vscode/build/dependencies.js:46:17)
    at Object.getProductionDependencies (/Users/penlv/code/vscode/build/dependencies.js:62:15)
    at Object.<anonymous> (/Users/penlv/code/vscode/build/gulpfile.vscode.js:38:37)
    at Module._compile (module.js:635:30)
    at Object.Module._extensions..js (module.js:646:10)
    at Module.load (module.js:554:32)
    at tryModuleLoad (module.js:497:12)
    at Function.Module._load (module.js:489:3)
    at Module.require (module.js:579:17)
    at require (internal/module.js:11:18)
    at require.sync.forEach.f (/Users/penlv/code/vscode/gulpfile.js:76:17)
    at Array.forEach (<anonymous>)
    at Object.<anonymous> (/Users/penlv/code/vscode/gulpfile.js:76:4)
    at Module._compile (module.js:635:30)

vscode-ripgrep tries to check node_modules in postinstall, which doesn't exist when doing yarn install.

Yarn's cache leads to this problem

yarn cache clean
yarn add vscode-ripgrep
vscode-ripgrep has a folder _node_modules, it has a postscript which will remove this folder and add a new folder called bin
if we run yarn add vscode-ripgrep again, yarn will load the module directly from cache (which has the bin folder instead of _node_modules) and try to execute postscript again.

Besides, it only happens on an old version of yarn, upgrading to the latest one solves the problem.

Should be fixed now

Great job guys!


Make LogLevel.style public

I think it's quite useful to have access to LogLevel.style outside of this file as well, thus this PR. If there is a reason for this being private, please let me know, I'll close the PR again 😄

The logging stuff will be going away next release in favor of apple/swift-logging. So unless we want to target this to 3.0 specifically, this will end up being changed.


Non-rails project throws NameError "uninitialized constant Class::JSON"

A simple non-rails project throws a NameError from `include 'neo4j':
/Users/ben/.rvm/gems/ruby-2.2.2/gems/neo4j-6.1.12/lib/neo4j/shared/type_converters.rb:243:in `convert_type': uninitialized constant Class::JSON (NameError)
	from /Users/ben/.rvm/gems/ruby-2.2.2/gems/neo4j-6.1.12/lib/neo4j/shared/type_converters.rb:343:in `register_converter'
	from /Users/ben/.rvm/gems/ruby-2.2.2/gems/neo4j-6.1.12/lib/neo4j/shared/type_converters.rb:312:in `block in included'
	from /Users/ben/.rvm/gems/ruby-2.2.2/gems/neo4j-6.1.12/lib/neo4j/shared/type_converters.rb:310:in `each'
	from /Users/ben/.rvm/gems/ruby-2.2.2/gems/neo4j-6.1.12/lib/neo4j/shared/type_converters.rb:310:in `included'
	from /Users/ben/.rvm/gems/ruby-2.2.2/gems/neo4j-6.1.12/lib/neo4j/type_converters.rb:5:in `include'
	from /Users/ben/.rvm/gems/ruby-2.2.2/gems/neo4j-6.1.12/lib/neo4j/type_converters.rb:5:in `<module:TypeConverters>'
	from /Users/ben/.rvm/gems/ruby-2.2.2/gems/neo4j-6.1.12/lib/neo4j/type_converters.rb:2:in `<module:Neo4j>'
	from /Users/ben/.rvm/gems/ruby-2.2.2/gems/neo4j-6.1.12/lib/neo4j/type_converters.rb:1:in `<top (required)>'
	from /Users/ben/.rvm/rubies/ruby-2.2.2/lib/ruby/site_ruby/2.2.0/rubygems/core_ext/kernel_require.rb:68:in `require'
	from /Users/ben/.rvm/rubies/ruby-2.2.2/lib/ruby/site_ruby/2.2.0/rubygems/core_ext/kernel_require.rb:68:in `require'
	from /Users/ben/.rvm/gems/ruby-2.2.2/gems/neo4j-6.1.12/lib/neo4j.rb:28:in `<top (required)>'
	from /Users/ben/.rvm/rubies/ruby-2.2.2/lib/ruby/site_ruby/2.2.0/rubygems/core_ext/kernel_require.rb:133:in `require'
	from /Users/ben/.rvm/rubies/ruby-2.2.2/lib/ruby/site_ruby/2.2.0/rubygems/core_ext/kernel_require.rb:133:in `rescue in require'
	from /Users/ben/.rvm/rubies/ruby-2.2.2/lib/ruby/site_ruby/2.2.0/rubygems/core_ext/kernel_require.rb:40:in `require'
	from test.rb:1:in `<main>'

Additional info:
Code example
require 'neo4j'

class Item
  include Neo4j::ActiveNode
  property :name
end

Runtime information:
Neo4j database version: 3.0.3
neo4j gem version: 6.1.12
neo4j-core gem version: 6.1.4

I think we fixed something like this in the past.  have you tried the latest version of the neo4j gem (7.1.1)?  There were also fixes in 7.0 which contain support for Neo4j 3.0.

Ah yes, got myself confused about neo4j and neo4j-core version numbers - neo4j 7.1.1 does the trick. Thanks!

Awesome!


Stop testing trim with Block DVs

after kubevirt/containerized-data-importer#2001, CDI will always
preallocate such DVs, and so all the test cases for non-preallocated
fail all the time.
Remove also the preallocated test - it's a negative test, and now
we would never get the feature on Block DVs.
Another reason to do this is that these tests did a lot of unusual
things like using rook toolbox, and despite a lot of effort, they
were unstable and quarantined for a long time.
Release note:

NONE

/retest
/assign

/approve
Test failures look like flakes when looking at flakefinder 7d and flakefinder 4w.
Let's see how the next run goes.

/test pull-kubevirt-check-tests-for-flakes

Some notes for the  pull-kubevirt-check-tests-for-flakes. Looking at the logs, there are 5 VMIs, 4 are running (maybe from previous tests?)
The VMI status from failed test shows this:
"type": "PodScheduled",
"status": "False",
"lastProbeTime": null,
"lastTransitionTime": "2021-12-21T11:39:15Z",
"reason": "Unschedulable",
"message": "0/2 nodes are available: 1 Insufficient memory, 1 node(s) had volume node affinity conflict."```

/lgtm


a broken test case revealed a bug in souper's test harness where it

wasn't checking that an APInt in test metadata had a width matching
the APInt computed by Souper
this patch fixes both problems

merging as obviously ok....


Allow specifying custom sevice_token instead of autogenerated one

Thank you for contributing to openstack/fuel-plugin-contrail!
openstack/fuel-plugin-contrail uses Gerrit for code review.
Please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


iOS: New Hash Algorithm in Log Files?

With recent iOS update I noticed however, that the algorithm is used by apple to compute the hashes must have changed:
Currently it worked by computing the sha256 hash of the bin file inside the zip archive.
With a recent exposure log from iOS 14.1, I tried to pinpoint two examples (part of log file):
"Files" : [
        {
          "Hash" : "D6C9ADEF176488BB7BCFA9F3A01EB5A9F692817441B777A630A1F74131A03CDF",
          "MatchCount" : 0,
          "KeyCount" : 16122,
          "AppBundleIdentifier" : "de.rki.coronawarnapp",
          "Timestamp" : "2020-10-23 18:59:04 +0200"
        },
       ...,
        {
          "Hash" : "41444FD9FFAEFDD39A78B11E59DD58EF2146D5F65D39E7607C0E1997712B737F",
          "MatchCount" : 0,
          "KeyCount" : 43934,
          "AppBundleIdentifier" : "de.rki.coronawarnapp",
          "Timestamp" : "2020-10-23 18:59:03 +0200"
        }]

Looking at the key count:

first example with 16122 keys most likely refers to 2020-10-22.zip, where the algorithm sha256 on bin file yields 80f50120713fa19c7ba96639c87ed2dab3f770c4ef8546c4f0ed3804c3c0b834.
second example with 43934 keys most likely refers to 2020-10-16.zip, where the algorithm sha256 on bin file yields 256addf082571d25d9db244b85de520aa8822ad12613dea0433dc7bf736d233f.

Unfortunately Apple is not very helpful, I did not find explanations beyond detectExposures on the log files.
I know that this is completely unrelated to your dashboard, but would very much appreciate if we can find out the hashing algorithm and then update the mapping file / create a new mapping file.
Please feel free to ignore this comment and keep the issue closed if you feel that it is too unrelated to your dashboard.
Originally posted by @felixlen in https://github.com/janpf/ctt/issues/3#issuecomment-715545706

Workaround should be good for now until a better understanding is available on how iOS >= 14.1 computes hashes.

The source of this issue is due to changing to country EUR instead of country DE, see https://github.com/janpf/ctt/issues/8. Once there is a decision on how to proceed, the code here can be adapted accordingly.

https://ctt.pfstr.de/json_EUR/filehashes.json ;)
as discussed in janpf/ctt#3 and janpf/ctt#8

@janpf Thanks a lot, much appreciated! Could you kindly allow CORS on that file?

allow CORS on that file

Whoops. Done. (in about 15 minutes)

Thanks again, works very well!


"Lunca Corbului" is listed in the drop-down menu

Description:
"Lunca Corbului" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Lunca Corbului" in the search bar.

Expected results
The location is listed in the drop-down menu as "Lunca Corbului, Argeș (Lunca Corbului)".


Lab 04L02, az140-21-p1-2 does not exist

Module: 4
Lab/Demo: 04L02, Package Azure Virtual Desktop applications (AD DS)
Exercise: 1, Task: 4
Step: 11
az140-21-p1-2 is not created in the previous labs
Same issue in exercise 2, task 1, step 1, 2. exercise 3, task 1, step 6

az140-21-p1-2 is created in Lab 02L01, this issue report it not correct


Zombie job during merge_publications

I noticed that the merge_publications step works fine when using DEV_LIMIT=1000 on my laptop but fails in our production Airflow:
https://sul-rialto-airflow-dev.stanford.edu/dags/harvest/grid?dag_run_id=manual__2024-07-10T11%3A06%3A13.060630%2B00%3A00&tab=logs&task_id=merge_publications
The error is:
[2024-07-12, 15:47:01 UTC] {scheduler_job_runner.py:1737} ERROR - Detected zombie job: {'full_filepath': '/opt/airflow/rialto_airflow/dags/harvest.py', 'processor_subdir': '/opt/airflow/rialto_airflow/dags', 'msg': "{'DAG Id': 'harvest', 'Task Id': 'merge_publications', 'Run Id': 'manual__2024-07-10T11:06:13.060630+00:00', 'Hostname': '0b19fcec1ff1', 'External Executor Id': 'ae9582b5-0a58-43ca-8729-b7f721a3be08'}", 'simple_task_instance': <airflow.models.taskinstance.SimpleTaskInstance object at 0x7fc84c2d5850>, 'is_failure_callback': True} (See https://airflow.apache.org/docs/apache-airflow/stable/core-concepts/tasks.html#zombie-undead-tasks)

After retrying a few times I noticed that the server is running out of available memory, which I believe results in the task being killed (either by Airflow, Docker or the VM).

The dimensions and openalex CSVs are quite large:
-rw-rw-r-- 1 rialto root 8.8M Jul 10 09:51 dimensions-doi-orcid.pickle
-rw-rw-r-- 1 rialto root  11G Jul 11 16:52 dimensions-pubs.csv
-rw-rw-r-- 1 rialto root  20M Jul 11 10:50 doi-sunet.pickle
-rw-rw-r-- 1 rialto root  11M Jul 11 10:49 openalex-doi-orcid.pickle
-rw-rw-r-- 1 rialto root  12G Jul 11 18:31 openalex-pubs.csv
-rw-rw-r-- 1 rialto root 2.5G Jul 10 05:07 sulpub.csv

We currently only have 8GB of RAM available. Since we are using polars and streaming it's not exactly clear to me how much RAM we might need. We could try doubling it and see if that helps?

That sounds like it would be worth trying.

I put in a request to double the memory 🤞

We could also experiment with setting low_memory to True when calling scan_csv https://docs.pola.rs/api/python/stable/reference/api/polars.scan_csv.html

We seem to have resolved this for now by lowering the amount of data we are writing to the merged publication parquet file #90
At some point we may want to revisit why we can't use df.collect(streaming=True).sink_parquet instead of df.collect().write_parquet() since the former should stream the data to the parquet file rather than requiring it to be built in memory?


Minimum Spanning Tree

💥 Proposal
There are n vertices and e edges and their weights, you need to construct minimum spanning tree.
Proposed Solution:
Scan the array elements, and push it in a vector, sort the array with their weights which takes O(e log e).
Now iterate through the vector and check whether or not this edge should be added using union - find with path compression
algorithm which takes O(e alpha e).
Efficiency: O(e log e + e alpha e)
Efficiency of Traditional approach: O(e log e + e log e)
This solution has a slight edge over other algorithms.
 Have you read the Contributing Guidelines on Pull Requests?
Yes

@Kajol-Kumari @plazzy99 @siddharth25pandey @Abhijit2505 @HarshCasper @iamrajiv
Please assign me this issue
I would like to implement this in C++ and please add labels.

Hi again  @Abhijit2505 @HarshCasper @anushkrishnav, would like to solve this issue in python. Can you assign me this?

Go ahead with it


DG: update appendix Instructions for manual testing

Dasha: Person features, Vendor features
Ricco: Wedding features, Task features, Tag features


DeleteByQuery don't set the generic as the type

I noticed another difference between NEST 1.7.1 and NEST 2.0 namely that DeleteByQuery don't set the generic as the type. So this query will delete all documents of every type in the index PersonData:
client.DeleteByQuery<Person>("PersonData", dq => dq.MatchAll());

but if I add Type it works like it did before, i.e. only deletes all documents of the Person type:
client.DeleteByQuery<Person>("PersonData",
                   dq => dq.Type(Types.Type(typeof(Person))).MatchAll());

Thanks @tomasfalt the deletebyquery in 2.x now takes an explicit Types argument so this becomes less ambiguous. e.g "mytype, mytype2" or Types.All or Type<Project>() if you statically import Nest.Infer


chore(client/conn): SendRequest conversion glue

users of the backports feature that are attempting to migrate from 0.14 to 1.0 may need to convert from a legacy SendRequest (returned by some of the connector builders) to an instance of the new protocol specific conn::http2::SendRequest<B> or
conn::http1::SendRequest<B> senders.
this adds From<T> implementation to the type provided by the backports feature, to help facilitate incremental migration.

This one I'm struggling to see the benefit. Do you have more of an example?

pardon the delay in following up on this pull request.
i've been working to upgrade from hyper 0.14 to hyper 1.0. along the way, i've been using the deprecated feature flag to incrementally prepare for this, but on further reflection i believe that you are right.
i'm going to close this particular pull request, pardon the trouble and thank you for your time!


Non-haskell dependencies

The proper way to bring in non-haskell dependencies when using cabal2nix is to again use package.yaml. This means we don't add dependencies directly into  the default.nix.
For example to bring in clang. We need to add it to the build-tools or system-build-tools in the package.yaml.
Note that system-build-tools is what it should be at the latest version of hpack. On older versions it's build-tools. The change was added here: https://github.com/sol/hpack/commit/c1d4ba7d19526075da01ce66d92b3c003aac6e53 This can be added like:
build-tools:
- clang

system-build-tools:
- clang

Then you update the cabal.nix and also the cabal file using cabal2nix and hpack. And then you should see clang appear in both the buildInputs of default.nix and shell.nix.

https://gist.github.com/CMCDragonkai/8b5cc041cea4a7e45a9cb89f849eaaf8

For non-haskell dependencies that are libraries. Apparently the proper way is to put down:
extra-libraries:
  - libmnl
  - libnftnl
  - linuxHeaders

This will put the dependency into librarySystemDepends which mean it will be available in both nix-build and nix-shell.
It also automatically makes cabal configure try to find these libraries. However I have not figured out how to resolve this since cabal then fails to compile. I'm guessing I'm missing something else that requires to refer to these libraries.

Ok I found the problem: https://github.com/NixOS/cabal2nix/issues/413
Basically Cabal/Hpack expects that extra-libraries refers to actual C libraries that will be linked.
So if you state extra-libraries: libmnl, it will actually try to link to -llibmnl. However this is problematic, because often the name of the library to be linked is something different like -lmnl. So -llibmnl will result in an error.
But if we change to -lmnl, the problem is that cabal2nix will generate a cabal.nix that wants mnl as a package attribute. And in nixpkgs, the package is called libmnl.
So what does cabal2nix do? It maintains a mapping of common library names to the package names in Nixpkgs.
This means we have to request them to maintain these mappings everytime we meet a dependency that isn't mapped already. However... as a stop-gap, we can fill in these mappings ourselves when we load the cabal.nix. So we use extra-libraries: mnl and then when importing the cabal.nix, we have to put mnl = pkgs.libmnl;.

The #24 takes over from #23. #23 shows an example of how to use CPP to put compile time macros into the Haskell program. So I'm going to close that one, but it is still useful reference for everybody in case they need a compile time macro. @DrFacepalm @nzhang-zh @Zachaccino

Solved this for Haskell Demo.
But in general there is 2 solutions:
A. You consider project to be a Haskell dependency (something distributed on hackage)
B. You consider the project to be a standard package (something that is on Nix)
In the A side, you must keep default.nix a callpackage derivation.
In the B side, you can dispense with that requirement, make default.nix something that can use stdenv.mkDerivation, and then you are using a Haskell project within it.
B may be relevant if there's a lot of non-Haskell stuff, a multi-language and many dependency project.
If you are doing B, you will end up using the haskellPackages.callPackage which will be whatever haskell compiler is considered the "default" compiler for that package set.


Tapping admin link in concierge page, directed to production app

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: V9. 0.54-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Go to https://staging.new.expensify.com/home
Login with a new gmail account
Tap fab - create a new workspace
Submit an expense
Go to LHN
Open the concierge chat
Tap #admin link

Expected Result:
Tapping admin link in concierge page, user must be directed to workspace admin room.
Actual Result:
Tapping admin link in concierge page, directed to production app instead of workspace admin room.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/a502aab8-a88e-4fb9-b26c-e4ede6a1fb38
View all open jobs on GitHub

Production
https://github.com/user-attachments/assets/71f0c536-7527-4406-a4c2-0f1524dba128

how you getting on with investigating this one @youssef-lr ?

This isn't a bug. When this hits production it will redirect properly to the admin room.


Stop using mozPrintCallback in generic builds

Over in https://bugzilla.mozilla.org/show_bug.cgi?id=1524640 the fix is only allowed for the built in pdf.js viewer, so if fonts are cleaned before the callback, printing will break. There's also been a little discussion on moving mozPrintCallback to only be allowed by the internal PDF.js viewer since no other browser implement it nor do they plan to.

The generic printing code, which is found in web/pdf_print_service.js, hasn't been using mozPrintCallback ever since PR #7697 :-)
The only usage of mozPrintCallback is found in https://github.com/mozilla/pdf.js/blob/master/web/firefox_print_service.js, a file which is only bundled into MOZCENTRAL builds: https://github.com/mozilla/pdf.js/blob/7d24fc34f9fb4045d49025512e5bb6989709eb13/web/viewer.js#L44-L47

Hah, missed that one.


Fix numerical validators with precision=None

If precision is None, then I believe the sensible behaviour is not to
apply a round, but to instead maintain the precision of the given
value.
This change corrects the base class to apply round only in the
circumstance that precision is not None. Also true when using
Validator.dump.

Need to bump version before merge


Explain why autohooks-plugin-black is marked as vulnerable

Dependency check reported CVE-2024-21503 in black, this way it is suggested to update to use version 24.3.0.
I made the update in the project, but dependency checker still reports a problem with black:23.10.0:
File Path	/home/*/dependencies/site-packages/autohooks/plugins/black/__init__.py
I suspect because autohooks-plugin-black = ">=23.10.0" in use.
See package details in this comment: https://github.com/greenbone/autohooks/issues/650#issuecomment-2033903967
I reported the issue to the project provider: https://github.com/greenbone/autohooks/issues/650, but he is also confused why is this package vulnerable.
Could you please explain us why is this package is problematic?
What is the logic in the checker?
Considering we do not see this as a vulnerable point I added this alert as a false positive.

Due to how dependencycheck works your autohook plugin is seen as the black library itself. That plugin is at version 23.10.0.
See also http://jeremylong.github.io/DependencyCheck/general/suppression.html and the also there linked http://jeremylong.github.io/DependencyCheck/general/internals.html

Thank you for your help.
I've opened a request to release a new version from autohooks-plugin-black: https://github.com/greenbone/autohooks-plugin-black/issues/560