[bug] import hook ignores functions with only keyword-only args

Hi!
It seems that Beartype import hook ignores (doesn't generate type-checking wrapper) functions with only keyword-only arguments and without return type annotation. It can be reproduced with the following snippet:
# module.py

def foo_bad(*, arg: str):
    print(arg)

def foo_good(arg: str):
    print(arg)

-----------------------------------------------

# main.py

from beartype.claw import beartype_all

beartype_all()

from module import foo_bad, foo_good

# passes type-checking
foo_bad(arg=42)

# beartype.roar.BeartypeCallHintParamViolation: Function module.foo_good() parameter arg=42 violates type hint <class 'str'>, as int 42 not instance of str.
foo_good(arg=42)

Oh, Gods! You're so right. This is appalling. It's been years since I've fielded an issue like this. I thought we'd finally nailed the low-level fundamentals of @beartype down to the rotting floorboards of GitHub. I was wrong. Now, we all pay the terrifying price.
Thanks for the superb minimal reproducible example (MRE), @avolchek. I'll resolve this immediately. If a miracle happens, I'll push this out with our new upcoming beartype 0.18.0 release. It may be cold, but I'm still sweating through my flannel. :cold_sweat:

Resolved by 3926b09acc4c. Praise Hypnobear! All is well. This has been resolved for our upcoming @beartype 0.18.0 release – to be published before I lose my last hair follicle. So... any day now.
Until then, please circumvent this shameful display of personal ineptitude and moral failing by annotating returns (i.e., typing the return values of all callables). Of course, annotating returns is really something everybody wants to do anyway. Now, you really really want to:
# Instead of this unannotated return...
def foo_bad(*, arg: str):  # <-- wat? no "-> None"!? O_O
    print(arg)

# Please annotate this return! Voilà. Experience the magic of worky.
def foo_bad(*, arg: str) -> None:  # <-- all's right with the world
    print(arg)

I apologize for this inconvenience with a scrunched-up face that's inducing a horrible migraine.


Implementing fix for Issue #150

Fixes #150

Do you need to re-enable any test as part of this fix?

@rfrenchseti No.


Unknown loss function for Ranking

I am looking to implement a model that uses a custom tensorflow loss function defined here:  https://github.com/tensorflow/ranking/blob/master/tensorflow_ranking/python/losses.py
which is blocked by:
Caused by: org.deeplearning4j.nn.modelimport.keras.exceptions.UnsupportedKerasConfigurationException: Unknown Keras loss function _pairwise_logistic_loss

@Jbiloki unfortunately, we don't support custom losses as of now. you'd have to implement that in dl4j first, as it's not an official keras loss.

@maxpumperla Thank you for the update, adding it myself is a fine option. How can I register it as a custom loss function to use for the imported model once created?

@Jbiloki that's a good point. we have a way of registering custom layers, but not custom losses.
I'll have to look into that first.


Show new SLA parameters in market proposals

The Chore
The launch of liquidity SLAs will introduce new parameters to market proposals, which we need to display in Governance.
Tasks

[ ] Show new SLA parameters in market proposals

Acceptance Criteria

[ ] I can see all 5 of the new fields listed under market parameters above in the governance app on new proposals

Additional details / background info

Planning thread in Slack

I want to double check this is ok to close

This should be on the Governanfe App not the BE.

Confirmed with Madalina that the fields are there on the API so we just need to expose them in the governance app.

Madalina accordion probably does tis already, but we should confirm before closing

Confirmed, seen on stagnet1


UI: need separation between taxi and twitter filtering UI

We could put them in two separate panels, or two tabs within the same panel. Right now it is not clear what each filter button is referring to.
Also, one other minor nit: a lot of analysts did not recognize that the filter button was a button, presumably because it was the same color as the surrounding area. I would suggest making it blue like the other buttons in the app.

@manthey I'm happy to fix this if you describe your preferred solution :)

It looks like we are getting new data for July.  Once we figure out what
that fully implies, I'll have a better idea of what we should do.
On Mon, Jun 29, 2015 at 11:32 AM, Zach Mullen<EMAIL_ADDRESS>wrote:

@manthey https://github.com/manthey I'm happy to fix this if you
describe your preferred solution :)
—
Reply to this email directly or view it on GitHub
https://github.com/XDATA-Year-3/geoapp/issues/145#issuecomment-116733251
.

--
David Manthey
R&D Engineer
Kitware Inc.
(518) 881-4439

One of my users also suggested a taxi panel and a message panel. Same user as #164.

Some filtering options apply to both.  For instance, the July data set will have multiple cities' data, so we will have a region selector that lets you pick 'All' or a specific city.  The date current can be grouped, but doesn't have to be.
Some display options apply to both (you can load more points than you display, for instance, so that you can have accurate heat map bins without crushing your GPU when looking at just points).
To further complicate things, some of the data for July is point-to-point data that ISN'T taxi data (Washington D.C. bike share data, for instance).
We've had the thought of having the panels have a simple mode and a lots-of-features mode.  This would hide the rarely used controls, but allow them to be shown when a + button was selected.  For instance, the taxi trip length and number of passenger controls would fall into this group.
One possibility would be instead of "Filters" and "Display", there was "General", "Trips", and "Messages".
"General" would have Region (what city), Date, Map Source, and the Location buttons (and the legend?).
"Trips" and "Messages" would have advanced (default hidden) options for "Use a different date" and their own date range.  Otherwise, they would have a prominent "Show" button (which, when toggled would grey out the rest of the controls).  Each would have the Filter and max controls.  The Trips would have the taxi-specific controls (the trip length, number of passengers, Display Type, and Processing), and the Messages would have the text search.
I currently have the "Max Points" control in the Display section (which changes to a "Max Vectors" control at times.  I guess this would remain in the Trips section and a dual of it would be in the messages section (so you can load more messages for binning or graphing than are drawn on the map).
Thoughts?

We've had the thought of having the panels have a simple mode and a lots-of-features mode. This would hide the rarely used controls, but allow them to be shown when a + button was selected. For instance, the taxi trip length and number of passenger controls would fall into this group.

At least one analyst requested such a "simple vs. advanced" mode switch.

One possibility would be instead of "Filters" and "Display", there was "General", "Trips", and "Messages".
"General" would have Region (what city), Date, Map Source, and the Location buttons (and the legend?).

This sounds good. If this moves us toward increased flexibility as far as what sorts of data we can handle, I'm all for it. I think the power of this application will be in how easily we can tailor it to be useful on new datasets that might be a little different from what we have currently. I do like having just a single date range for data filtering -- I can't imagine someone would want to load data from two different date ranges at once depending on the type of data.

People may want to look at the same data set at two different points in time on the same map, but that is a different situation.

This has been addressed in PR #201.


6.0: [PrunedLiveness] Fix boundary check for dead-ends.

Explanation: Fix an edge-case in a fundamental SIL utility.
The PrunedLiveness utility is the fundamental tool used in the SIL optimizer to track where a value is live.  For example, CSE uses it by way of the tool used to replace uses in OSSA, namely OwnershipRAUWHelper.  PrunedLiveness allows clients to specify the points at which a value is known to be live.  It uses those points to determine all places in the function where a value is live.
Often (and as used by OwnershipRAUWHelper) this is done by only considering the consuming uses.  So long as a value must be consumed on all paths this makes sense: the consuming uses are the last uses of the value; everything before them is live, everything after them is dead.  While soon (via LifetimeCompletion) it will be a correct and verified assumption that a value is consumed on all paths, unfortunately, currently, it is not: a value needn't be consumed on all paths.  In particular, it can be "leaked" in so-called "dead-end blocks".  A "dead-end block" is a block from which there are no function exiting paths--infinite loops, blocks that trap, and blocks all paths out of which lead into one or the other of those.
The lack of consumes in dead-end blocks means that it's not enough to only consider only consuming uses to determine whether a value is live.  Special care must be taken with the dead-end blocks.  Previously, it was only checked whether the block in question was a dead-end block; if so, the value was determined to be live within the block.  That was not correct, however.  If the value is consumed in a dead-end block, it is not available after that consume.  Additionally, if the value is consumed on some but not all paths into the dead-end block, the value would not be available in it.  Here, this check is tightened to only consider a dead-end block to be live if (1) the value is not consumed in the block and (2) the value is actually available in it (i.e. if the value is not consumed on any path into the block).
Fixes a SIL verifier error when building a simple project with an infinite loop for embedded.
Scope: Affects SIL optimizations.
Issue: rdar://126965232
Original PR: https://github.com/apple/swift/pull/73647
Risk: Low.  Corrects and tightens an edge case.
Testing: Added a SIL test and the original source-level test.
Reviewer: Andrew Trick ( @atrick )

@swift-ci please test

@swift-ci please test


Crash when the app running

I drap the AudioKit folder to my project then run and build.
It crash when the app start.
The error is follow:
dyld: Library not loaded: @rpath/CsoundLib.framework/CsoundLib
Referenced from: /Users/xxx/Library/Developer/CoreSimulator/Devices/51776503-6943-4AC0-903F-FE08A6AFB058/data/Containers/Bundle/Application/FBB6B689-8CE9-406D-B768-DEC2FD580054/audioSame.app/audioSame
Reason: image not found

Did you follow all of the steps in the 'Installation Screencasts and Walkthroughs' section on the website? http://audiokit.io/tutorials/

This issue needs more information since we can't act on it.  Feel free to reopen it with more details.

I would guess that you probably forgot to include the CsoundLib.framework in your app, since it seems like it can't be loaded. You also need to include the run script build phase as explained in the instructions (check out INSTALL-OSX.md).

Maybe you should take a look on this
http://stackoverflow.com/questions/24993752/os-x-framework-library-not-loaded-image-not-found


uuids is being cached

I'm not sure why but it seems like the uuids are being cached somewhere. Not sure if its the device or bleno itself but changing the uuids doesn't change what's actually being advertised (even though when I console.log from the Node side of things it's reflecting the new values). But when testing via a Cordova app, the old uuid is still being advertised. I'm also changing the advertisement name. Part of the name is used for constructing the uuid but the odd thing is that the advertisement name is changing while the uuid remains constant. This is why the app fails to connect because the actual uuid being advertised is no longer the same with the uuid being constructed by the app (as based from the advertisement name).
Here's what the code looks like:
var bleno = require('bleno');
var BlenoPrimaryService = bleno.PrimaryService;
bleno.on('stateChange', function(state) {

    if (state === 'poweredOn') {
        console.log('peripheral id: ' + config.peripheral_id);
        bleno.startAdvertising(config.name, [config.peripheral_id]);
    }
});

bleno.on('advertisingStart', function(error) {
    console.log('primary service uuid: ' + config.primaryservice_uuid); // shows the correct uuid but still advertising the old one
    if (!error) {
        bleno.setServices([
            new BlenoPrimaryService({
            uuid: config.primaryservice_uuid,
                characteristics: [some characteristics here]
           })
        ]);
    }
});

I already tried the following but nothing seems to work:

restarting the Pi (sudo shutdown -r now)
shutting down the Pi then turning on again (sudo shutdown -h now)
shutting down the Pi and then unplugging, then plug again
clear the cache of the Cordova app.
uninstalling the Cordova app and reinstalling again.

Has anyone ever encountered this issue? Anything else that I could try or somewhere I've missed? Thanks in advance.

Got same issue, I think this is why I can't write to my RPi https://github.com/sandeepmistry/bleno/issues/280

@anchetaWern could you provide an HCI dump capture for this? sudo hcidump -t -x
What version of Android are you using? Could you also try turning bluetooth off and on on the Android device?

I had the same problem, running bleno on mac with an android device. Turns out Android is the one doing the cashing, starting the bluetooth on the android device fixed the problem

This is on purpose, and should happen. In most cases in today's BLE stacks, once you do a scan, it will cache the data unless a service changed packet comes across invalidating the service.
This is one of the reasons I wrote #128


Json attributes not accessible

I use this project with sailsjs and mongodb and tried to setup a basic request which works fine.
Here is my User.js model:
module.exports = {
  tableName: 'users',
  attributes: {
    profile: {
      type: 'json',
      defaultsTo: {
        'firstName': null,
        'lastName': null
      }
    }
  }
}

I can request the id attribute and the profile attribute like this (Postman body):
{"request": "query getUser { users { id, profile } }"}
What I can't do is request a specific json attribute like so:
{"request": "query getUser { users { id, profile { firstName } } }"}
I already tried waterline-to-graphql (doesn't return json at all), sails-hook-graphql (no docs provided) and sails-grapqhl (fails at request).
How can I do this?

@benediktdertinger formatting your model like below should fix the problem:
module.exports = {
  tableName: 'users',
  attributes: {
    firstName: {
      type: 'string',
      defaultsTo: null
    },
    lastName: {
      type: 'string',
      defaultsTo: null
    }
  }
}

@pierreburgy thanks for your reply! Does it mean, that i can't use nested objects at all?

@benediktdertinger unfortunately not.


fix: move dev docker compose to better name

Description
Moves our development docker-compose.yml to docker-compose.dev.yml to more clearly mark that it's meant for development and to prevent docker compose up from even starting any containers by default in the project root.
This now also adds an FAQ item for how to remove the neo4j-data volume without restoring to a full wipe (docker compose down -v)
Motivation and Context
There is sometimes confusion between our root docker-compose.yml (which is used for development and requires more setup to use) and our examples/docker-compose/docker-compose.yml (which is the way to run BloodHound CE as an end user). In addition, running docker compose up without specifying profiles doesn't really do anything helpful, as it will only start the common containers and not the API/UI, so making it harder to run without just seems warranted.
How Has This Been Tested?

Ran through our just commands for development
Attempted to run docker compose up to ensure an error is presented rather than starting dev mode containers

Types of changes

[x] Chore (a change that does not modify the application functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] Documentation updates are needed, and have been made accordingly.
[ ] I have added and/or updated tests to cover my changes.
[x] All new and existing tests passed.
[ ] My changes include a database migration.

Added a small fix to our just init command to be idempotent (your config files will no longer be overwritten if they exist) and to ensure it can clean up after running a docker compose command without the default config file existing.


CHEMINF resolution to move from semanticscience resolver to Ontobee

Owing to end-of-life support for PHP 7, there are strict requirements on PHP code such that the semanticscience ontology resolution service will no longer function owing to a lack of update in its library dependencies. For this reason, we have decided to forward CHEMINF IRI resolution requests to Ontobee, where the high level ontology page is
https://ontobee.org/ontology/CHEMINF
and individual concept resolution pages are available, for instance
https://ontobee.org/ontology/CHEMINF?iri=http://semanticscience.org/resource/CHEMINF_000550
ontobee also offers rdf descriptions of these concepts with content type negotation and with links:
https://ontobee.org/ontology/rdf/CHEMINF?iri=http://semanticscience.org/resource/CHEMINF_000550
ontobee also has a SPARQL endpoint for all the ontologies, which is really nice, and you can see an example of querying for a CHEMINF concept using yasgui:
https://api.triplydb.com/s/UKbhkRgEW
we'll be putting this into production shortly, but any questions or comments are welcome.

@egonw

@micheldumontier, is there anything we need to do on the CHEMINF side? Do we want to keep the current IRIs? I think you suggest yes, correct?

@egonw we have now switched to ontobee for CHEMINF.
http://semanticscience.org/resource/CHEMINF_000550
nothing needs to be done.
cheers,
m.


Fixed issue with adding "onCondition" to "where".

Developers should know when the want to add a onCondition different from when they want to add a where clause

this is correct according to the documentation, onCondition is meant to be used in the relation definiotion.

The condition will be used in the ON part when yii\db\ActiveQuery::joinWith() is called. Otherwise, the condition will be used in the WHERE part of a query.
http://www.yiiframework.com/doc-2.0/yii-db-activequery.html#onCondition()-detail

@cebe, you probably should have tried with the joinWith before closing this pull request.
Of course I did with joinWith and still had it in the where condition.
See https://github.com/yiisoft/yii2/issues/8496


Support iPhone13

iPhone 13 has been released for some time. I hope to support iPhone 13 as soon as possible

https://github.com/lmirosevic/GBDeviceInfo/pull/124


calling SankeyWidget keeps kernel busy

Hi,
I am new to jupyter notebooks, so this problem might be something I'm doing wrong. I've installed the widget and can see it listed and enabled in 'Environment' parameters page using anaconda. I've opened the shared notebook and can import the libraries etc. When I get to calling SankeyWidget (third cell), the kernel is kept busy (I assume this is what is happening seeing I get an asterisk?). Either way, I can't get the diagram to show.
Could you advise?
Thanks
D

What version of the notebook are you using?
You could open the developer console in the browser and check if there are any errors reported.
I'm not sure what the Environment parameters page is with anaconda. You can check the nbextension is installed correctly by running jupyter nbextension list, which shows this on my computer:
Known nbextensions:
  config dir: /home/rick/.jupyter/nbconfig
    notebook section
      jupyter-sankey-widget/extension  enabled
      - Validating: OK
  config dir: /home/rick/anaconda3/etc/jupyter/nbconfig
    notebook section
      jupyter-js-widgets/extension  enabled
      - Validating: OK

Experienced today this on Safari on Mac. Switching to Chrome solved the issue.

Hi ricklupton,
I'm using 4.2.2.
I tried what aneverov4vmw suggested it's working now. It doesn't seem like this widget runs on Safari (9.1.2).
Thanks for your help!

Glad it's working now. I'll add a note to the README that it's not working on Safari.
I don't have a Mac to test on at the moment. Did you see anything in the error console that might indicate why it's not working?

Me too! Thank you.
Unfortunately not. I will be making extensive use of this widget, so when I have a moment I'll try to dig deeper.

Some feedback. I had to bump my python version down to 2.7 again -- turns out the component is now working in Safari.


[Alerting 2.x] Wait before checking update status for composite monitor

Description
After updating composite monitor added some wait time before checking if update was successful
Issues Resolved
https://github.com/opensearch-project/alerting-dashboards-plugin/issues/732
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@amsiglan would you mind checking on the linter?

@amsiglan would you mind checking on the linter?

Fixed it


i18n(fr): update guides/dev-toolbar.mdx

Description (required)
The file was marked as outdated because of #9240 but the changes are not applicable. Instead I fixed a malformed Markdown link.

Related issues & labels (optional)

Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

fr/guides/dev-toolbar.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


starlark: reject duplicate keyword arguments in function call even when callee has **kwargs

$ cat a.star
def f(*args, **kwargs):
        print(args, kwargs)

f(x=1, y=2)
f(x=1, x=2)

$ starlark a.star
() {"x": 1, "y": 2}
() {"x": 2} # should be an error

Python2, Python3 and Starlark-in-Java all reject repetition of a keyword argument in a function call. Starlark-in-Go should to.  (It does when the callee is a regular function but not when it has a **kwargs parameter.)

Note that you'll need a runtime check, in case there's a **kwargs.

Should the same apply to Go-implemented callables? I'm getting duplicate kwargs in them.
func (zzz) CallInternal(thread *starlark.Thread, args starlark.Tuple, kwargs []starlark.Tuple) (starlark.Value, error) {
	fmt.Fprintln(os.Stderr, kwargs)

zzz(arg = "value1", arg = "value2")

[("arg", "value1") ("arg", "value2")]

Maybe it's not the nicest (in terms of code) to prevent this, but otherwise every implementation of a callable would have to be wary of this.

Hm, maybe it's some kind of bug, because when I try to show a problem on a minimal test case, it fails "correctly"
And there's certainly code to handle this
https://github.com/google/starlark-go/blob/6ddc71c0ba77217b518c586dc8589d627c92ebfb/resolve/resolve.go#L735-L736
I'll look into this more

Ah right, here's an actual repro
zzz(
    foo = "a",
    **{"foo": "b"}
)

[("foo", "a") ("foo", "b")]

The calling convention says a callee gets a list of positional arguments and a list of named arguments. It's up to the callee what they do with them. Starlark functions, and builtins that use UnpackArgs to interpret parameters, both reject duplicates, but one could imagine a built-in that put them to good (albeit possibly surprising) use.


Dart initialization not working when using FirebaseAuth and FirebaseMessaging together on real device.

Some additional info. The error only occurs when running on a real device. Here is some context from my experience:
Console Log messages
2022-02-23 11:59:46.861192-0500 Runner[70126:84723762] Metal API Validation Enabled
2022-02-23 11:59:47.063866-0500 Runner[70126:84723999] 8.11.0 - [Firebase/Core][I-COR000012] Could not locate configuration file: 'GoogleService-Info.plist'.
2022-02-23 11:59:47.214326-0500 Runner[70126:84723762] : Please set a value for FacebookAutoLogAppEventsEnabled. Set the flag to TRUE if you want to collect app install, app launch and in-app purchase events automatically. To request user consent before collecting data, set the flag value to FALSE, then change to TRUE once user consent is received. Learn more: https://developers.facebook.com/docs/app-events/getting-started-app-events-ios#disable-auto-events.
2022-02-23 11:59:47.214379-0500 Runner[70126:84723762] : You haven't set a value for FacebookAdvertiserIDCollectionEnabled. Set the flag to TRUE if you want to collect Advertiser ID for better advertising and analytics results.
2022-02-23 11:59:47.217694-0500 Runner[70126:84724032] flutter: Observatory listening on http://<IP_ADDRESS>:54307/Aiipqazh3ak=/
2022-02-23 11:59:47.230417-0500 Runner[70126:84723762] FBSDKLog: Missing [FBSDKAppEvents appID] for [FBSDKAppEvents publishInstall:]
2022-02-23 11:59:47.448249-0500 Runner[70126:84723762] [VERBOSE-2:FlutterObservatoryPublisher.mm(101)] Failed to register observatory port with mDNS with error -65555.
2022-02-23 11:59:47.448295-0500 Runner[70126:84723762] [VERBOSE-2:FlutterObservatoryPublisher.mm(103)] On iOS 14+, local network broadcast in apps need to be declared in the app's Info.plist. Debug and profile Flutter apps and modules host VM services on the local network to support debugging features such as hot reload and DevTools. To make your Flutter app or module attachable and debuggable, add a '_dartobservatory._tcp' value to the 'NSBonjourServices' key in your Info.plist for the Debug/Profile configurations. For more information, see https://flutter.dev/docs/development/add-to-app/ios/project-setup#local-network-privacy-permissions
2022-02-23 11:59:47.448404-0500 Runner[70126:84723762] FBSDKLog: Missing [FBSDKAppEvents appEventsState.appID] for [FBSDKAppEvents flushOnMainQueue:]
2022-02-23 11:59:47.448431-0500 Runner[70126:84723762] FBSDKLog: Missing [FBSDKAppEvents appEventsState.appID] for [FBSDKAppEvents flushOnMainQueue:]
2022-02-23 11:59:47.456331-0500 Runner[70126:84723762] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'The default FirebaseApp instance must be configured before the default Authinstance can be initialized. One way to ensure this is to call FirebaseApp.configure() in the App Delegate's application(_:didFinishLaunchingWithOptions:) (or the @main struct's initializer in SwiftUI).'
*** First throw call stack:
(0x1a436e708 0x1b8e787a8 0x1a42637a0 0x101025b84 0x1000d7d90 0x1050f8c98 0x1a6d85028 0x1074500b4 0x107451de0 0x1074620a8 0x1a42e8340 0x1a42e2218 0x1a42e1308 0x1bb964734 0x1a6d5f75c 0x1a6d64fcc 0x100008638 0x1a3f9dcf8)
libc++abi: terminating with uncaught exception of type NSException
*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'The default FirebaseApp instance must be configured before the default Authinstance can be initialized. One way to ensure this is to call FirebaseApp.configure() in the App Delegate's application(_:didFinishLaunchingWithOptions:) (or the @main struct's initializer in SwiftUI).'
terminating with uncaught exception of type NSException
(lldb)
flutter doctor -v
[✓] Flutter (Channel stable, 2.10.2, on macOS 11.6.1 20G224 darwin-x64, locale en)
    • Flutter version 2.10.2 at /Users/sstakoff/dev/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 097d3313d8 (5 days ago), 2022-02-18 19:33:08 -0600
    • Engine revision a83ed0e5e3
    • Dart version 2.16.1
    • DevTools version 2.9.2
[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
• Android SDK at /Users/sstakoff/Library/Android/sdk
• Platform android-32, build-tools 32.0.0
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• CocoaPods version 1.11.2
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
[✓] VS Code (version 1.63.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension can be installed from:
🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[✓] Connected device (2 available)
• iPhone 13 (mobile)    • 452E84A5-5DC5-449B-B5C9-FB7B21008BB6 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
• Chrome (web)          • chrome                               • web-javascript • Google Chrome 98.0.4758.102
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!

Dependencies from pubspec.yaml
  cupertino_icons: ^1.0.2
firebase: ^9.0.2
firebase_core: ^1.12.0
firebase_database: ^9.0.6
firebase_auth: ^3.3.7
syncfusion_flutter_core: ^19.4.42
syncfusion_flutter_calendar: ^19.4.43
cloud_functions: ^3.2.7
modal_progress_hud_nsn: ^0.1.0-nullsafety-1
syncfusion_flutter_datepicker: ^19.4.43
swipe_image_gallery: ^0.0.5
cached_network_image: ^3.2.0
url_launcher: ^6.0.18
filter_list: ^1.0.1
firebase_messaging: ^11.2.6
flutterfire_ui: ^0.3.4
shared_preferences: ^2.0.13
icon_badge: ^1.0.0
flutter_app_badger: ^1.3.0
provider: ^6.0.2
firebase_analytics: ^9.1.0

Originally posted by @sstakoff in https://github.com/FirebaseExtended/flutterfire/issues/8119#issuecomment-1049015768

I was looking through some of the framework code and came across these 2 interesting comments:
/// The default app instance cannot be initialized here and should be created
/// using the platform Firebase integration.
  static Future<FirebaseApp> initializeApp({

This is odd since I call this code in main() as follows:
await Firebase.initializeApp(
    options: DefaultFirebaseOptions.currentPlatform,
  );

which I believe is in fact attempting to initialize the default app?????
I also noticed this in the FirebaseMessaging code:
  //  Messaging does not yet support multiple Firebase Apps. Default app only.
  /// Returns an instance using a specified [FirebaseApp].
  factory FirebaseMessaging._instanceFor({required FirebaseApp app}) {
    return _firebaseMessagingInstances.putIfAbsent(app.name, () {
      return FirebaseMessaging._(app: app);
    });

which makes reference to the fact that Messaging is currently ONLY supported using the default app. So is it possible that in my call to initializeApp, I am NOT actually initializing the default app - causing an issue with messaging? And then messaging uses the info in GoogleService-Info.plist as the default app config that it need???? Just a theory :)

@sstakoff see #7983 for potential workaround

@sstakoff see #7983 for potential workaround

Thank you! I will def try this out... Much appreciated

Closing this as duplicate of https://github.com/FirebaseExtended/flutterfire/issues/7983


BLEServer::on_disconnect lacks disconnect reason

The reason is given in nimble, but it's not passed along to the callback:
https://github.com/taks/esp32-nimble/blob/develop/src/server/ble_server.rs#L198
The disconnect struct contains a reason field that can be passed along.

Thank you for raising the issue.
I have changed the program and added the disconnect reason parameter.


Add Voting to tool request page

Migrate the list from personal graph

People will vote on GH issues instead


fix(md-select): fire blur after dropdown hides

#248

Coverage remained the same at 95.154% when pulling 6fae9690589e7276e5733f12eb42d04e0c668c34 on Ullfis:good-news-everyone into 1e1079dd7a177ce69d702dc71398543d89ed6256 on aurelia-ui-toolkits:master.

Just noticed the branch name. :joy:

Firefox sometimes fire blur several times in a row observable at http://localhost:3000/#/samples/select/ when enable 'Disable Functionality', open that list and then open 'Basic use' list.

Chrome - ok
IE 11 - ok
Edge ?


AttributeError: 'Credentials' object has no attribute 'to_json'

Expected Behavior
create 'token.json' file and then print the next 10 calendar events, from quickstart.py, https://developers.google.com/calendar/api/quickstart/python.
Actual Behavior
creates an empty 'token.json' with the following error:
Traceback (most recent call last):
  File "google_quickstart.py", line 69, in <module>
    main()
  File "google_quickstart.py", line 49, in main
    token.write(creds.to_json())
AttributeError: 'Credentials' object has no attribute 'to_json'

Then running again gives an error for an empty 'token.json'. I delete it to try again, but obviously same error as above.
Steps to Reproduce the Problem

Use quickstart.py from https://developers.google.com/calendar/api/quickstart/python.
Used my 'credentials.json' file from the Google Cloud Platform.
Downloaded required pakckages.
Run in python.

Specifications

Python version 3.9.7.
Windows 10 Pro Version 2004.

I started a fresh virtual environment. Issue solved.


Tables unreadable

Please use valign="top" on your HTML tables. They are hard to read with the default (valign="middle").
This applies to all pages in the OpenType spec.

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

ID: e8d3d5b8-059a-613d-8666-80438ebd7aa7
Version Independent ID: 6a0f2873-97c3-833c-45e4-340ea96667a0
Content: GDEF — Glyph Definition Table (OpenType 1.8.4) - Typography
Content Source: typographydocs/opentype/spec/gdef.md
Product: typography
GitHub Login: @PeterCon
Microsoft Alias: alib

@pgundlach :  I don't see the same result you show in the screenshot you provided. I've tried different browsers on both Mac and Windows, and see clear column separation and top-aligned cells.

Perhaps when you were viewing the page the stylesheets weren't getting loaded?
If you continue to have the same issue, please provide more details.

I see that it works in all browsers tested except for  Safari version 14.1.2 (146<IP_ADDRESS>.5) on macOS 10.14.6 (18G9323). It works on the current macOS (Big Sur) Safari.
With the current Safari, the overall page looks different. I have made sure that no ad blocker is in the way.


Support for fdg files

Support for fdg files that are the native files printed by Voxelab printers.

Ps. I reverted to stock firmware the time being

Hi ! How can I update ?

@doucettom, see #301


build: upgrade github.com/vmihailenco/msgpack to v5

msgpack is now using modules as of v5+, this PR upgrades the library to the latest version v5.3.5. From the changelog, there are a few breaking changes, however, none of them affect the usage of the library here.

go test ./...
ok  	github.com/eko/gocache/lib/v4/cache	1.544s
ok  	github.com/eko/gocache/lib/v4/codec	0.828s
ok  	github.com/eko/gocache/lib/v4/marshaler	0.587s
ok  	github.com/eko/gocache/lib/v4/metrics	1.512s
ok  	github.com/eko/gocache/lib/v4/store	1.659s

@eko 👋 would you consider merging this PR?

Thank you for this upgrade of msgpack @jaimem88


Added Split archive support with unit tests. …

…Added ArchiveFactory.IsArchive() and minor refactor.
ArchiveFactory.cs has new static methods to test if a file is an archive and will also return "out" the ArchiveType. ArchiveFactory.cs was refactored around this to simplify it a bit. A new Open() method was added to allow split archives and multipart rar to be opened - also supporting single file archives too. This method is able to open all types.
A SplitStream class has been added that allows split archives to be read as one file.
Rar4 unit tests have been added, along with split archive tests.
I'm not sure how this change affects some of the other classes and properties with regard to indicating that an archive IsSplit etc. A minor enhancement would be be able to give the library the first part of a multipart or split archive and have the library find the other parts. Currently you have to pass them all in to the new Open() method.
Hope this is useful for the repo.

Wow, blown away by this.  Thank you!


Test if headers return last added header value

If you add same header multiple times with different values, you will not be able to get last added header value;

Thanks for the test, @thestanislav! I was able to find a workable solution, and will release shortly.


[Urgent] Polygon Transaction data duplicated and data quality issue

As you can see at this polygon transactions, you can see that the hash is on 2022-02-09 00:26:21 UTC in 24714823
select
* from `public-data-finance.crypto_polygon.transactions`
where TIMESTAMP_TRUNC(block_timestamp, hour) = TIMESTAMP_TRUNC("2022-02-09 00:00:00", hour)
    AND `hash` = "0xd481ff0f96cdea342e8baa7d3ca65d59589cdec2782b0b7b4f65731bfae0e63e"
    ;

But when you look at https://polygonscan.com/tx/0xd481ff0f96cdea342e8baa7d3ca65d59589cdec2782b0b7b4f65731bfae0e63e
The transaction happened in Feb-09-2022 12:34:55 AM +UTC
So we deep dive into the logs, we found the transaction is logged in two different blocks and two different block timestamp as well.
select
    *
from `public-data-finance.crypto_polygon.transactions`
where TIMESTAMP_TRUNC(block_timestamp, hour) >= TIMESTAMP_TRUNC("2022-02-09 00:00:00", hour)
    AND `hash` = "0xd481ff0f96cdea342e8baa7d3ca65d59589cdec2782b0b7b4f65731bfae0e63e"

but we couldn't find the transaction in https://polygonscan.com/txs?block=24714823 at all.
Can anyone help deep dive in this more?

This seems like a forked block likely caused due to the following fork in the network
https://polygonscan.com/block/24714822/f
It will be automatically removed when our sync runs

@araa47 can u elaborate more on this? We waited more than 10 hours for the sync to happen but the results are still the same. When is the sync? And what's going to change after the sync? Can we expect the same results from polygonscan?


Automatically detect if the given text is Japanese or English with Python

I think I'm going to release the Twitter-based version of this product for Japanese and English first. So, we should be able to detect if a given tweet is written in Japanese or English with Python. This way, we can only show Japanese tweets coming from Japanese learners to native speakers of the language. Same with English.

Nice. @ykdojo

@ykdojo does it mean whenever there is a japanese tweet from a person,the person who is familiar with Japanese will only be able to see that.?or all the members in the community?If we notify only japanese familiar people,then while using this twitter app,they must be registered as learning English knows japanese?Is your thought process is the similar to this?,What I have understood.By the way I am very much interested in contributing to this app idea from which I can gain more  knowledge.we can do this to other languages aswell here in India :)
Small doubt :(

Hmm here's an example to clarify.
Suppose User A is learning Japanese, and her native language is English.
She starts using one of her Twitter accounts, say, @uesr_a_jp to start tweeting stuff in Japanese.
Then, Japanese native speakers should start seeing these tweets so they can fix them.
However, I'm only concerned that, what if @user_a_jp starts tweeting stuff in both Japanese and English? We should probably be able to ignore all English tweets in that case.

Oh yeah, the langdetect library looks good!

Would you like me to go ahead and create a few functions that make use of the library? @ykdojo

Yeah that would be awesome! Thank you.
On Thu, Nov 8, 2018 at 9:37 AM ratdog45<EMAIL_ADDRESS>wrote:

Would you like me to go ahead and create a few functions that make use of
the library? @ykdojo https://github.com/ykdojo
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ykdojo/editdojo/issues/23#issuecomment-437014861, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABukw50Jcr8W-oejIEfxqlCOD4m4-Edrks5utEGPgaJpZM4YM9Wv
.

NOTE: there's already a PR for this. https://github.com/ykdojo/editdojo/pull/29
Will come back to this when it's more immediately useful.

would it be easier to implement google traductors feature of automatic language detection or its something extra and unnecessary ? @ykdojo

Yeah, actually I think that will be ideal.


Embed() support for other file types (mp3)

Is your feature request related to a problem? Please describe.
I am able to successfully embed image files using this code, but not mp3 files.
How this code works: A note has a corresponding image and mp3 of the same file name, just a different extension. I grab the filename from a dataview query using YAML tags, add a jpg extension, and embed the image. This works. But it doesn't work for mp3 files.
LIST embed(link(file.name + ".jpg"))
FROM ("dir1" or "dir2") and #tagA/subtag
sort file.name asc

Describe the solution you'd like
I'd like embed to support other file types, such as .mp3 files.
It might also be nice if dataview had two additional options

An option to just list query results on a new line without a bullet point or table.

PLAIN embed(link(file.name + ".jpg"))

RESULT:
File 1
File 2
File 3

A RANDOM sort option, and/or RANDOM + SEED sort option

sort file.name rand 52

RESULT:
File 3
File 1
File 2
Describe alternatives you've considered
I don't know any alternatives. Dataview is all I got man!! I need my fix!

There's more information here:
https://forum.obsidian.md/t/how-do-i-embed-a-list-of-mp3s-queried-from-dataview-or-dataviewjs-query/56416


Cannot see failed tests for dependent gems for pull requests

Tests are passing for pull requests, while dependent tests are actually failed.
This might be a correct approach actually, sometimes I need to update dependent gems to make them pass.
But I need to go to github "Actions" every time to see if they are passed.
Is it the best solution we can have or there are better options?

@andrew2net is the person to ask here. He has a similar workflow that has been refined over the years, and is one that we should likely adopt for pubid gems.

@mico In my workflow I install PR using rake install locally and test all dependent gems. If it needs to update any depended gems, I release all Relaton gems with new minor version.
To automatically release gem when GHA test passing use following workflow:

create commit git commit -m 'fix issue'
create tag git tag -a v1.2.0 -m "v1.2.0"
push update with the tag git push --follow-tags

@mico I've created a bash script to run rspec in all relaton-* repos. You can adopt it for the Pubid project.


s3 delete-objects pass OK for fail to delete non-empty folder

https://github.com/chrislusf/seaweedfs/issues/1857

@chrislusf  Its ok ?


[logs] change repo_id to string in ghe schema

to: @jacknagz
cc: @airbnb/streamalert-maintainers
size: small
resolves N/A
Background
We noticed the value of repo_id can be empty in log coming from GHE when the event is to deletion a repository. Change type of repo_id to string, so that the rule classifier will handle empty value properly.
Changes

Change type of repo_id to string in GHE schema.

Testing

Rule testing

python manage.py lambda test --processor all
...
StreamAlertCLI [INFO]: (60/60) Successful Tests
StreamAlertCLI [INFO]: (93/93) Alert Tests Passed
StreamAlertCLI [INFO]: Completed

Unit testing

./tests/scripts/unit_tests.sh
...
Ran 529 tests in 8.475s

OK

Coverage remained the same at 95.4% when pulling 886d8662ecc299fb69ea3ece287a04b84fad60c7 on change_repo_id_to_string_in_ghe_schema into ab6a667196ca4db24427914e22dc8ada1bbc6e61 on master.


Any plans for an entire goodreads user review dataset?

The API has been discontinued, but it's actually faster to collect reviews from RSS feeds.
If there's interest, I started a script here
https://colab.research.google.com/drive/1uOyVlKaT4QFtce9yQpKj9hRtj5z8Uyta
It still needs work in confirming it has gotten all the books from a user (I think there might be timeouts) and issues with books that have several versions/editions. But the biggest bottleneck is collecting reviews from all 100 million users

Hello @SantoshGuptaML this is a project I would be interested in participating in! if you are still interested in scrapping this data please contact me i would be happy to help you in this endeavor!

HI @jonvlcs07 , somehow I missed your reply. How can I contact you? Feel free to email me at<EMAIL_ADDRESS>

@SantoshGuptaML My apologies to the late reply as I'm mostly maintain this dataset in my limited spare time and do not have any plan to extend the dataset. Do you need to keep this issue open in case other folks may want to join your project?

@SantoshGuptaML My apologies to the late reply as I'm mostly maintain this dataset in my limited spare time and do not have any plan to extend the dataset. Do you need to keep this issue open in case other folks may want to join your project?

Sure sounds good. At the moment I'm just trying to contact jonvlcs07 as they expressed interest in the dataset development. I can't seem to find any contact info on the profile, but if convenient feel free to pass them mine if possible<EMAIL_ADDRESS>

Reopen this issue - for anyone who's interested in collect a more recent and comprehensive Goodreads review dataset, feel free to contact @SantoshGuptaML<EMAIL_ADDRESS>

Hi @cantonalex  - we (the original author group) don't have any plan on continue collecting the data at the moment. If you're interested in a more recent data, probably can contact @SantoshGuptaML on that.

@cantonalex @jonvlcs07 I started scraping the data. Could use some help, currently projected to finish in many years lol


Proposal for Py2/3 Support

I have tested it in 2.7 and 3.4, mostly. I don't have a good test set, as I'm just starting to use it for a new Py3 project (which was my motivation for porting this), so I don't know it well. Also, not sure if there's a reason for using distutils over setuptools, but I love the "develop" support in setuptools, so I threw it in, we can always back that out.

I'm going to work on nfldb now, as that is a better interface. (btw, it's for a plugin for this bot I am writing: https://github.com/datamachine/telegram-pybot)

Rebased on new changes.


adjust indentation when multiple lines are selected

#602

Thanks for pointing that out, have updated the PR.

Yes, there is. Can try to do that in a separate PR.

@virattara merging this now to get it in ahead of #613. That's going to get merged sometime tomorrow, so be aware of that if you're doing any followup. Thanks again! :)

Okay, will keep that in mind. Also, I think I need to make changes suggested by Raph too.


add support for django 3.2

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e7d24e3). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #19   +/-   ##
=========================================
  Coverage          ?   90.20%
=========================================
  Files             ?       10
  Lines             ?      143
  Branches          ?        0
=========================================
  Hits              ?      129
  Misses            ?       14
  Partials          ?        0

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 e7d24e3...af97bbc. Read the comment docs.


Subtle ABI Break on "ClassicConfiguration"

Which version and edition of Flyway are you using?
Previously using 9.8.3, trying to upgrade to 9.10.0
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Java Configuration API
Which database are you using? (Type & version)
Postgres ~42
Which operating system are you using?
Development is done on a Mac, prod is Unix
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
Upgraded the Java library.
What did you expect to see?
No compile time breaks on a minor version.
What did you see instead?
There is a subtle ABI break on the ClassicConfiguration object where the argument signature of setBaselineVersion(MigrationVersion version) was changed from MigrationVersion to String. The fix seems trivial enough to stop wrapping the configured String in a MigrationVersion, but that also seems to be the point of setBaselineVersionAsString(String version). The newly added override seems to prevent Lombok from generating the appropriate setter. This type change broke some of our library upgrade automation due to failing compilation. Not sure if this is an intentional usability "break", but it was introduced in a minor version so it was a surprise.
Thanks!

This is currently in our pipeline to fix

Hey! Thanks a lot for the quick fix. Appreciate the tool!


Platform forwards should be awaited

https://developers.home-assistant.io/blog/2022/07/08/config_entry_forwards/
https://github.com/kvj/hass_openwrt/blob/8eb8a37c629bd2df933e282ab34005a665e46d2c/custom_components/openwrt/__init__.py#L33
Replace this block with await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS)

Fixed in 0.2.0


Unable to update the password from "My account"

Click 'Account'
From 'Edit user' enter:

Password (leave blank if you don't want to change it)
Password confirmation
Current password (we need your current password to confirm your changes)

Click 'Update'

Observation:
Unable to save the new password - user sees:
"1 error prohibited this user from being saved: Name can't be blank"

I think this is because your account is currently invalid, because you registered before we added a required name. Can you add a name, then try updating?


Remove etcd tweaks

We've seen lots of etcd/fleet hiccups while testing recently, so this attempts to fix that by removing the timing tweaks Deis had in user-data. The default etcd2 service lets the binary use its default values, so the hope is that will encourage better behavior.
Also updates the etcd version in the graceful-shutdown stanza to 2.1.12.

lgtm

we are also experiencing flaky etcd2 behavior recently.
Did switching to 2.1 mitigate the effect?

LGTM

Code LGTM.

Accidentally closed the PR.

Closing this in favor of #4444, which also removes these etcd timing tweaks.
@pmoust honestly we have not seen version 2.x behaving much differently from 0.4.9 with regard to Deis' behavior, so it's hard to say if would improve stability for you.


Configure Azure Pipelines

The Pipeline configuration build the application and produces the build for release configuration

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


daemon: Decrease log level for svc not found msg

This PR changes a log level from "warn" to "info" of a "Service frontend not found" message.
The SVC-related refactoring in v1.7 has fixed a few race conditions which could have been a culprit for the message. Unfortunately, it'd be quite difficult to backport the changes due to the heavy refactoring. Instead, we change the log level, as in (probably) most cases the warning is harmless.

test-me-please


Differentially private CIFAR 10 not training

When I run examples/dp_cifar10/cifar10_transforms.py locally, I see that for each epoch the loss gets higher and higher and the accuracy stays below 10% (which it should be, approximately, if we just kept guessing 0 because there's 10 classes). Given that we have a passing test that checks per sample gradients and that I'm not seeing this happen with Opacus, this is suggesting that our recreation of the clipping + noising of the per sample gradients is incorrect. The task is to diagnose and fix this
Output:
  Train Epoch: 1  Loss: 7.328635 Acc@1: 0.000000
  Train Epoch: 1  Loss: 37673.759659 Acc@1: 0.088727
  Train Epoch: 1  Loss: 57173.875967 Acc@1: 0.087619
  Train Epoch: 1  Loss: 94578.966633 Acc@1: 0.090323
  Train Epoch: 1  Loss:<PHONE_NUMBER>62 Acc@1: 0.085756
  Train Epoch: 1  Loss:<PHONE_NUMBER>67 Acc@1: 0.084706
  Train Epoch: 1  Loss:<PHONE_NUMBER>83 Acc@1: 0.082426
  Train Epoch: 1  Loss:<PHONE_NUMBER>43 Acc@1: 0.080845
  Train Epoch: 1  Loss:<PHONE_NUMBER>64 Acc@1: 0.080840
  Train Epoch: 1  Loss:<PHONE_NUMBER>25 Acc@1: 0.082593
  Train Epoch: 1  Loss:<PHONE_NUMBER>17 Acc@1: 0.083010
  Train Epoch: 1  Loss:<PHONE_NUMBER>72 Acc@1: 0.082739
  Train Epoch: 1  Loss:<PHONE_NUMBER>89 Acc@1: 0.082347
  Train Epoch: 1  Loss:<PHONE_NUMBER>34 Acc@1: 0.082626
  Train Epoch: 1  Loss:<PHONE_NUMBER>78 Acc@1: 0.083603
  Train Epoch: 1  Loss:<PHONE_NUMBER>16 Acc@1: 0.079735
  Train Epoch: 1  Loss:<PHONE_NUMBER>22 Acc@1: 0.074783
  Train Epoch: 1  Loss:<PHONE_NUMBER>02 Acc@1: 0.070409
  Train Epoch: 1  Loss:<PHONE_NUMBER>92 Acc@1: 0.066519
  Train Epoch: 1  Loss:<PHONE_NUMBER>36 Acc@1: 0.063058

The calculated noisy gradient needs to be divided by batch_size here:
https://github.com/pytorch/functorch/blob/23e8c6641b4f938b98dc4f510c300c0a3a0555c4/examples/dp_cifar10/cifar10_transforms.py#L79
The analogous operation in Opacus is performed by the scale_grad() method.
I am able to train with high accuracy by fixing the code with something like param.grad = param_grad/batch_size

Great catch @spliew! We'd welcome a PR if you want to make the change (especially if you can share the loss/accuracy you're seeing locally) or I can make it

Closing, thanks again @spliew!


blog: Project-Scoped Stacks in Self-Managed Backends

Adds a new blog post intended to be published this Wednesday (3/29) on the same day the feature will be released.
Part of https://github.com/pulumi/pulumi/issues/12173
Related Issues/PRs:

https://github.com/pulumi/pulumi/issues/2522
https://github.com/pulumi/pulumi/pull/12437
https://github.com/pulumi/pulumi/pull/12438

Per Slack exchange with @justinvp, I took a pass over the post.
I've incorporated most of the feedback (and probably introduced some new issues 😅), rephrased a bunch, deleted the "old CLIs" section in favor of an "info" note, and spaced out independent ideas a bit more so that paragraphs aren't too information dense, and added a couple diagrams to help users visualize what's changing.
Please let me know what y'all think.

Holistic feedback:
I think it would be pertinent to add a section that details the continuing improvement into the service backend and also reiterates our commitment to open source.
Something like:

Pulumi recently announced a host of improvements to the service backends such as (link)[deployments] and (link)[insights] which continue to provide value and engineering productivity to our users. As part of our continuing commitment and investment in the core OSS Pulumi products, we’re excited to announce an improvements into the self managed backends…

Building on Lee's feedback, I would prefer that this post doesn't just focus on filestate. Instead, we should retell the story of why projects and stacks management in Pulumi is awesome, and that we are happy to now support it equally across all backends. We should also take an opportunity to promote best practices as laid out in several recent Zephyr posts.
Happy to dive into more detailed thinking here, but at a high level, I would like to see us take that approach.

I think I understand the point. On a second read, the intro does read a bit like "filestate has problems but we fixed it". It's too negative. Rather, it should be "we're been hard at work constantly improving both httpstate and filestate. here we're adding one of the highest requested features of filestate." I'll take another stab at it.
I'll take another pass over this.

@justinvp I've incorporated comments and taken another pass.
I've changed the intro a fair bit to talk about how we're committed to both backends,
and the rest to talk about projects, stacks, etc. in general, covering them for both backends.
For "referencing stacks", it's now "referencing stacks in the CLI" to explain that this is in context of the CLI.
We may also want a "referencing stacks inside other stacks" section to link to stack references.
Please look when you have a chance and then we can get another review from everyone else.

@abhinav love it! Reads fantastically and really drives home all the improvements as well as the benefits of both approaches <3

This reads much better now! Nice work!

Tossing a do-not-merge on here so this doesn't go out before Monday.

Dropping do-not-merge label. This is planned to go out once we tag the release.


Increase cmake minimum required [16437]

Related:

https://github.com/eProsima/Micro-XRCE-DDS-Agent/pull/323
https://github.com/eProsima/Micro-CDR/pull/72
https://github.com/eProsima/Micro-XRCE-DDS-Client/pull/335

Build status:

Linux
Windows

Build status:

Linux
Windows

Build status:

Linux
Windows

Build status:

Linux
Windows

Build status:

Linux
Windows


Missing brand name in name resolution service

Hi, querying the API, i don't get a suggestion for the label "Ongentys" , which is a brand/synonym for OPICAPONE https://pubchem.ncbi.nlm.nih.gov/compound/135565903

Same for "Sertraline Tablets"
But Sertraline map to PUBCHEM.COMPOUND:68617

Thanks for the report.   Spot checking, I'd say we're currently pretty weak on brand names for drugs overall.  The only one I found was lipitor.  It doesn't seem that the sapbert that we're playing with does very well either (though it does handle the "sertraline tablets" query fine).

RxNorm has a lot of brand names, so once we've added that to Babel (https://github.com/TranslatorSRI/Babel/issues/150) that will hopefully mostly solve this problem.

@genomewide I remember you mentioning that you had a list of the top-selling drugs in the US for use in autocomplete checking -- I'd love to use that list to test this issue. Could you please send me that list or add it to this issue?

The list of top-selling drugs @Genomewide mentioned is https://www.drugdiscoverytrends.com/50-of-2022s-best-selling-pharmaceuticals/. I'll add these to the Babel validator and report back on where we currently are with this.

We support most of the top 50 best selling US drugs now. The missing ones:

Paxlovid (CHEBI:192712)
Spikevax (RXCUI:2601552)
Biktarvy (RXCUI:1999667)
Trikafta (Kaftrio, PUBCHEM.COMPOUND:165363555)
Gardasil (RXCUI:659877)
Prevnar (MESH:D022242)
Botox (MESH:C542869)
Entresto (CHEBI:9927)
Januvia (CHEBI:40237)
Xeplion (PUBCHEM.COMPOUND:115237)
Symbicort (PUBCHEM.COMPOUND:5281004)

Some of that might be because we're filtering for SmallMolecule, but it's most likely either because we don't have the correct synonyms.


Location data missing when crawling tags

Hey there,
when I crawl an account name, I get a json with the Location data. When I use --tag, these element is missing. Is there a way to include the location data this way?
Thanks for your great work!

You can specify the --include-location option in v1.5.0 to pull the location data.


Counter example doesn't compile

I'm was trying to use the library but couldn't get past the compilation of tiny_skia. I'm getting the following error with Rust 1.75 as well as 1.78:
error[E0308]: mismatched types
   --> tiny_skia/src/lib.rs:422:58
    |
422 |                 let pixmap = self.cache_glyph(cache_key, glyph_run.color);
    |                                   -----------            ^^^^^^^^^^^^^^^ expected `floem_peniko::Color`, found `peniko::color::Color`
    |                                   |
    |                                   arguments to this method are incorrect
    |
    = note: `peniko::color::Color` and `floem_peniko::Color` have similar names, but are actually distinct types
note: `peniko::color::Color` is defined in crate `peniko`
   --> [REDACTED]/.cargo/registry/src/index.crates.io-6f17d22bba15001f/peniko-0.1.0/src/color.rs:12:1
    |
12  | pub struct Color {
    | ^^^^^^^^^^^^^^^^
note: `floem_peniko::Color` is defined in crate `floem_peniko`
   --> [REDACTED]/.cargo/registry/src/index.crates.io-6f17d22bba15001f/floem-peniko-0.1.0/src/color.rs:9:1
    |
9   | pub struct Color {
    | ^^^^^^^^^^^^^^^^
note: method defined here
   --> tiny_skia/src/lib.rs:297:8
    |
297 |     fn cache_glyph(&mut self, cache_key: CacheKey, color: Color) -> Option<Rc<Glyph>> {
    |

Then I tried just running the example and that didn't work, so here I am 😅
EDIT: Getting the same error in the released version as well as main. Don't know how that works, you shouldn't be able to release something to crates.io that doesn't compile.

It will be fixed by #432
Use cargo update --package floem-cosmic-text --precise 0.7.1 to fix it

After passing that command cargo update --package floem-cosmic-text --precise 0.7.1 cargo is still complaining..
Any idea how to fix that?


refactor: Refactor parsec-api service initialization

:pencil: Description

:link: Related Issues

:tada: This PR is included in version 2.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix the inconsistent scaling_rule.adjustment setting in API server and Scaling Engine

#156429831

@cdlliuy Lets discuss this in detail before we go ahead.

LGTM

LGTM


Segmentation fault (core dumped)

Hi,
I was interested in running pstools with Omni-C data but I had the next error segementation fault error running hic_mapping_haplo.
... [M::tb_pipeline::10452.864*42.22] processed 1333334 sequences; [M::tb_pipeline::10462.076*42.23] processed 1333334 sequences; [M::tb_pipeline::10470.184*42.24] processed 1333334 sequences; [M::tb_pipeline::10474.139*42.25] processed 671892 sequences; /var/spool/slurmd/job106890/slurm_script: line 11: 12235 Segmentation fault      (core dumped) ./pstools_1 hic_mapping_haplo -t64 02.4_Nypro_Flye_sspace_lr1_gc1_SLRr1_gc1.scaff_seqs <(zcat /cluster/home/shared/nypro/08.8_ALLHiC/fastq/RacoonDogHiC_EKDL200002762-1a_R1.fastq.gz) <(zcat /cluster/home/shared/nypro/08.8_ALLHiC/fastq/RacoonDogHiC_EKDL200002762-1a_R2.fastq.gz) -o scaff_connections.txt /var/spool/slurmd/job106890/slurm_script: line 12: 48458 Killed
"hic_mapping_haplo" ran out of RAM with a 1 TB server. I tried to follow https://github.com/shilpagarg/DipAsm/issues/16 like you suggested in a previous issue but it didn't work.
Do you know how could I solve it?
Cheers,
Luis

Thank you for the issue. I will be happy to help if you could share Omni-C data and graph.

Many thanks for your help. I would be happy to share the data with you. How could I do it?
Best,

Any links to download data will be useful to help you.


Lib doesn't compile with Swift 4

It causes various errors, mostly of type "....HotKey/Sources/HotKey/NSEventModifierFlags+HotKey.swift:12:19: 'ModifierFlags' is not a member type of 'NSEvent'"

Latest pod HotKey 0.1.0 successfully compiling and working like a charm


Investigate origin of AWS Lambda timed out (#1754)

I found a solution to #1754 reproduction is here
https://github.com/prisma/prisma2/issues/1754#issuecomment-601271228
// Set to false to send the response right away when the callback executes, instead of waiting for the Node.js event loop to be empty.
context.callbackWaitsForEmptyEventLoop = false;

But this means that the Node.js event loop is not empty and it should be empty.
Without context.callbackWaitsForEmptyEventLoop = false
A POST request works perfectly. Only the GET requests in this example have a non empty event loop.
We could start investigating with:

Try out pure Prisma Client in lambda without the apollo server overhead, get the GET and POST event running
Instead of the callback based approach, try using a promise and play with async: true as described here https://github.com/serverless/serverless/issues/3171#issuecomment-540101244
Try to get serverless offline running with a simple prisma client example https://serverless.com/plugins/serverless-offline/

Marking it as a confirmed bug I can reproduce the timeouts.

@pantharshit00 You mean confirmed with reproduction https://github.com/UWellFail/prisma2--1754 I guess?

Yes, that one.

But this means that the Node.js event loop is not empty and it should be empty.
As we have an open handle to the database with an active connection, the event loop can't be empty, therefore this is not a bug.


Unity 2017.2 editor doesn't connect

Version Number and Operating System(s):
24.2 Windows 10
Expected behavior:
Connect from one editor server/client to other editor/build server/client in different PCs.
Actual behavior:
It does not connect.
Steps to reproduce:
Open Unity editor and try to connect to another PC running a build or other editor.
[Optional] Discord Username:
Nesvi

DEEP_LOGGING Doesn't outputs nothing because it doesn't even connects.

Try it with two editors on the same PC. If that works, then connecting between two PCs is likely a networking / firewall issue. To test with two editors on one PC, simply copy the entire project folder, open two instances of Unity, and open one project in one instance of Unity and the copied project in the second instance of Unity.

Closing this issue, as it was likely an error with the set up, not with Forge.


Material-Sprint: app/styleguide/icons

demo.html

Class attribute quotes should be double quotes not single for consistency.

Are the divs needed? I may have missed something here, but I think you could apply style of "preview" class to the anchor tag instead?

Hi @addyosmani, according to you latest commit you have this "Please note that the icons here have been removed as they have changed once again. We will not be including them
for another few days until the newer set has landed. Please ignore for now." Should I fix the commented  code or leave as is?

We'll be shipping support for the Material Design icons font in the coming months as part of #674. Given that this work is being tackled there, closing this.


Quick Guid generator

Description of the new feature / enhancement
A keyboard shortcut that allows to create a Guid/Uuid. Powershell already have this generator (using New-Guid cmdlet), but when editing mocks for unit testing, I need to changing from Powershell to IDE/Editor screens a lot of times.
An optional selection screen, allowing to edit the format to generate - like the formats available on C# Guid.ToString(string) method - could be opened, like the one available on Color Picker tool.
Scenario when this would be used?
When I press a key combination, generate a new guid and copy it to my transfer area, so I can easily past on IDE/editor.
Supporting information
No response

I'd loop this back to #5668 which would be an engine to do this.  totally understand the ask
/dup #5668


fix: set adapter.connected = False on channel closing

Description of PR...
Changes

Item 1
Item 2

API Updates
New Features (required)
This section should include details regarding new features added to the library.
This should include both open and private APIs. If including both, create two
sub headers underneath this one to label updates accordingly.
Deprecations (required)
All deprecations should be listed here in order to ensure the reviewer understands
which sections of the codebase will affect contributors and users of the library.
Enhancements (optional)
The enhancements section should include code updates that were included with this
pull request. This sections should detail refactoring that might have affected
other parts of the library.
Checklist

[ ] Unit tests
[ ] Documentation

References
(optional)
Include important links regarding the implementation of this PR.
This usually includes and RFC or an aggregation of issues and/or individual conversations
that helped put this solution together. This helps ensure there is a good aggregation
of resources regarding the implementation.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

4 of 5 (80.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.08%) to 96.66%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyzeebe/grpc_internals/zeebe_adapter_base.py
4
5
80.0%

Totals

Change from base Build<PHONE_NUMBER>9:
-0.08%

Covered Lines:
1013

Relevant Lines:
1048

💛  - Coveralls


S1 | Mock Create Hunts API

Story

As a project
I would like to mock the create hunt api
So that I can have a base API to test against

Acceptance Criteria

An /hunts api that accepts a POST request and returns a mock hunt

Approach

This route should be in a file like  hunts-api/routes/hunts.js
This file would be required by hunts-api/app.js to expose the route

@Kyle-Omara @mclark1996


Not possible to modify existing VIP

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
Not possible to modify VIP since update to 24.1.5.
To Reproduce
Steps to reproduce the behavior:

Go to 'Interfaces => Virtual IPs => Settings'
Click on "Edit" an existing virtual IP

Expected behavior
We should be able to edit an existing virtual IP.
Everything is empty, nothing on the dropdown lists...
Describe alternatives you considered
No alternative
Screenshots

Environment
OPNsense 24.1.5_2 (amd64).

Same with Edge, Firefox and Chrome... Also tested on 24.1.5_3, same issue.
No errors on the browser console.

Hello, any hint ? Does somebody was able to reproduce it ?
Thanks !

I've just spun up a fresh 24.1.6 and I can create new CARP VIPs, edit them, remove them.
Maybe run a health check in System: Firmware, to see if you have missing files or bad checksums somewhere.

I can create,edit and remove new VIP.
But it's not possible for VIP created before the update. And I cannot remove them !!!
The update has broke something...

To be frank it looks like a local issue of some sort. If you want to discuss a specific code change be my guest, but "update" "broke" "something" isn't going to cut through this.

I just say that I had no issue with modifiying existing VIP in 22.7.
Then I updated to 24.1.5, it does not work anymore...
So my thought is that the update is the reason why it's not working anymore...

I‘m merely saying if there is a bug it’s not possible to find with the information currently given.
And since nobody chimes in and the report is for the GUI misbehaving a local browser issue is the most likely.
You’d expect the GUI/API not working at all to be a widespread issue.

Can you test your API?
Open this link in your browser, replace <IP_ADDRESS> with your IP:
https://<IP_ADDRESS>/api/interfaces/vip_settings/get
From one of the list items, select the UUID and list it, replace 211c9748-651a-40cf-bbd6-918cedd845ec with one of your UUIDs from the prior list.
https://<IP_ADDRESS>:4444/api/interfaces/vip_settings/getItem/211c9748-651a-40cf-bbd6-918cedd845ec

I've found a workaround :

Create a new CARP IP adress
Save
=> Then you can now modify/delete previous created CARP IP adresses


process query strings

consider processing query strings
www.example.com/?this&that
www.example.com/?this
www.example.com/
example.com/
also, not sure this is doable with the webext API, but i recall other extensions like this where you could click on a url segment right in the address bar to go up a level - no pop-up needed
if that's not possible, and i'm guessing it isn't, you might consider essentially duplicating the url in a single-line pop-up rather than a vertical list - somehow i think that would be more attractive, but that's just me

i recall other extensions like this where you could click on a url segment right in the address bar

Sadly as you already guessed, the web-ext api does not allow such url bar manpipulations.
If that ever changes, i'll gladly take another look into this.


[Feature Request]Add tolerance threshold settings for screenshots

Description

Fixes # (issue)

Added threshold tolerance functionality on screenshots to avoid tests from failing due to minor pixel changes.
I have already created the issue on this repo already. Please read it for more details: https://github.com/FormidableLabs/react-native-owl/issues/140

Type of Change

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

How Has This Been Tested?

The changes have been tested via doing monkey patch on our app repo and verified that tests are passing now after this change. As I added an optional prop named threshold, therefore custom jest matcher toMatchBaseline function would be called as follow:
 const result = toMatchBaseline(latestPath, { threshold: 0.2 });

Checklist: (Feel free to delete this section upon completion)

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My code follows the style guidelines of this project (I have run yarn prettier:apply)
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes (I have run yarn test)
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

Screenshots (for visual changes):

@muhammadkhan0009 Thanks so much for your PR.
I've opened #142 which takes your changes, plus adds a default value, updates the types and updates the docs.

@robwalkerco
Thank you for adding the PR.


infinite $digest when 'put' missing translations

Subject of the issue
I want to put the missing translations in the database using the useMissingTranslationHandler() function. But I'm getting an infinite $digest loop when I add the $http code in the function. When I only log the missing translation like in the docs, there is no problem.
https://docs.angularjs.org/error/$rootScope/infdig?p0=10&p1=[]
Your environment

angular-translate: 2.15.2
angular 1.6.1
chrome 61

Steps to reproduce
You can see the code here: https://plnkr.co/edit/nl7GD7kgoMXS6XTt8Hl2
I have comment out some code in the app.js file, where the error happens. But when you enable it, there is a change that your browser will hang or crash. I have tested it in plunker, and despite the fact that the url gives an 404 error, my error is still going on.
Expected behaviour
I want to post the missing translation ID so I can handle it in my backend controller. (saving to the database for example)

Sorry for the delay.
Short: no idea.
Long: The error is something AngularJS internal, but mostly when digest are.. well.. in a loop.
Also I would recommend NOT sending each error directly via HTTP, because you can produce a flood attack against your own server. That's bad.
So a proper solution (which also works around your issue): On error, invoke only a service of you (adding the label in question to an internal stack/queue). Independently and periodically, you check for content and send them as a bunch to your server. Maybe every 10 seconds (if more critical), or only every 60 seconds.


Cleanup

Bring scripts/cleanup.sh up to date.

I was doing a lot of installs and uninstalls on a cluster, and I noticed that even after I remembered to change the confirmation setting, when I ran an uninstall via uninstall.yaml there were a lot of things left over, including some that caused a subsequent installation using Helm to fail.  In my case, even ignoring events, settings, and leases, there were still these left around:
`NAME                                               READY   STATUS      RESTARTS   AGE
pod/longhorn-admission-webhook-7c9b76b495-28jjg    1/1     Running     0          36h
pod/longhorn-admission-webhook-7c9b76b495-l4kbj    1/1     Running     0          36h
pod/longhorn-conversion-webhook-5dfc7849d4-4cv4s   1/1     Running     0          36h
pod/longhorn-conversion-webhook-5dfc7849d4-fbrp4   1/1     Running     0          36h
pod/longhorn-recovery-backend-6777c85b75-kp6p7     1/1     Running     0          36h
pod/longhorn-recovery-backend-6777c85b75-mnvjt     1/1     Running     0          36h
pod/longhorn-ui-7b7c86c7cf-jvcs6                   1/1     Running     0          36h
pod/longhorn-ui-7b7c86c7cf-lhwj9                   1/1     Running     0          36h
pod/longhorn-uninstall-xncz6                       0/1     Completed   0          2m5s
NAME                                  TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)    AGE
service/longhorn-admission-webhook    ClusterIP   <IP_ADDRESS>           9443/TCP   36h
service/longhorn-backend              ClusterIP   <IP_ADDRESS>             9500/TCP   36h
service/longhorn-conversion-webhook   ClusterIP   <IP_ADDRESS>             9443/TCP   36h
service/longhorn-engine-manager       ClusterIP   None                  36h
service/longhorn-frontend             ClusterIP   <IP_ADDRESS>             80/TCP     36h
service/longhorn-recovery-backend     ClusterIP   <IP_ADDRESS>             9600/TCP   36h
service/longhorn-replica-manager      ClusterIP   None                  36h
NAME                                          READY   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/longhorn-admission-webhook    2/2     2            2           36h
deployment.apps/longhorn-conversion-webhook   2/2     2            2           36h
deployment.apps/longhorn-recovery-backend     2/2     2            2           36h
deployment.apps/longhorn-ui                   2/2     2            2           36h
NAME                                                     DESIRED   CURRENT   READY   AGE
replicaset.apps/longhorn-admission-webhook-7c9b76b495    2         2         2       36h
replicaset.apps/longhorn-conversion-webhook-5dfc7849d4   2         2         2       36h
replicaset.apps/longhorn-recovery-backend-6777c85b75     2         2         2       36h
replicaset.apps/longhorn-ui-7b7c86c7cf                   2         2         2       36h`
I figured that it would be a good idea to have a foolproof "hammer" script that would clean up everything regardless, and that seemed to be the purpose of cleanup.sh.  Admittedly, it hadn't been touched for a while.
If you think this is a bad idea, I can drop the PR, and then the right thing to do is delete the obsolete script.

I've been thinking about this a bit @james-munson, and it seems your bad uninstall experience might be attributable to using one version of uninstall.yaml to clean up another version of Longhorn.
For example, if you used uninstall.yaml v1.5.0+ to clean up Longhorn v1.4.2-, you would be missing the logic to delete v1.4.2's recovery backend, admission webook, and conversion webhook deployments, as these no longer exist in v1.5.0.
https://github.com/longhorn/longhorn-manager/commit/2a0f924060e57252f218b23aacbf522cd509baa6#diff-9809693b91d35e4d0e3e4658840003aa5555c1f73772b141cb227add23dd8bfb
Since this cleanup script deletes the longhorn-system namespace, it is likely capable of getting rid of these deployments, which are deleted in a cascading fashion once the namespace is deleted.

This PR maybe helpful for situation like this https://github.com/longhorn/longhorn/issues/6259#issuecomment-1627470852

It looks like this script would also be unable to proceed in this situation (e.g. we will wait for volumes to delete forever because the volumes have finalizers). In my opinion, we should remove this script from the repo and make improvements to uninstall.yaml if we feel it needs them.

Agree. This script does not make too much sense since it cannot handle the broken uninstallation/Longhorn system cleanup. I prefer to remove it then open a separate ticket to improve the uninstallation.

Revised to simply remove the obsolete script.


Masonry component with async measurements

The doc says

Support for this may be added in the future but for now the use of the CellMeasurer render callback's async measure parameter is not supported.

Following is my way to use async measurements by modifying the cellRenderer with measure parameter.

suppose there is a components can lazy load image and has a callback method once it is loaded.

<LazyLoadImage loadedCallback={callbackFn} src={some url} />

use a function instead of a plain component as the child of  CellMeasurer

const cellRenderer = ({ index, key, parent, style }) => {
  const material = materials[index]
  return (
    <CellMeasurer
      cache={this.cache}
      index={index}
      key={key}
      parent={parent}
    >
      {
        ({ measure }) => {
          const updateFn = () => {
            measure()
            debouncedRecompute()
          }
          return (
            <LazyLoadImage
              loadedCallback={updateFn} width={(width - 40) / 5} src={material.url_128} />
          )
        }
      }

    </CellMeasurer>
  )
}

before that

const {
  materials, width
} = this.props

and

const debouncedRecompute = debounce(this.shouldRecompute, 250) // lodash.debounce

at last

shouldRecompute () {
  const { width } = this.props
  if (this.masRef) { // masRef is the ref of this Masonry component
  // <Masonry ref={ele => (this.masRef = ele)} ... />
    this.cellPositioner.reset({
      columnCount: 5,
      columnWidth: (width - 40) / 5,
      spacer: 10
    })
    this.masRef.recomputeCellPositions()
  }
}

about this.cache and this.cellPositioner

constructor (props) {
  super(props)
  const {
    width
  } = this.props
  const cache = new CellMeasurerCache({
    defaultHeight: 250,
    defaultWidth: (width - 240) / 5,
    fixedWidth: true
  })
  this.cache = cache
  this.cellPositioner = createMasonryCellPositioner({
    cellMeasurerCache: cache,
    columnCount: 5,
    columnWidth: (width - 40) / 5,
    spacer: 10
  })
  this.shouldRecompute = this.shouldRecompute.bind(this)
}

hope it helps

Thanks for sharing your approach, @chux0519! Any chance you'd be willing to share it as a Plnkr (or even a standalone demo that we could put in the playground folder for others to see? I'd be happy to link to it from the documentation. 😄
Closing this issue for now since it's not actionable- but I'd love to see something to add to the docs.

Here is my own approach, and it may still has some layout problem because of the debounced compute method.
But this approach is better than doing nothing.
online demo

Hi there. I'm trying to use Masonry to implement an infinitely scrollable display of images like Pinterest.  The problem is, I don't know the dimensions of the images beforehand because they are stored in another domain, so it's not possible for me to set the height and width of a cell before the image is loaded. This seems to have as a consequence that CellMeasurerCache is caching the wrong dimensions (the dimensions before the image is rendered), and therefore when the actual images start to show up they overlap each other in multiple places like so:

@bvaughn, what would you recommend to do in this situation?

Sorry @PabloMessina. Masonry does not support async measurement. This was a conscious choice I made when designing the component in order to simplify the API and optimize performance.
It's also mentioned in the docs:

Cell measurements must be synchronous. Size impacts layout and async measurements would require frequent layout invalidation. Support for this may be added in the future but for now the use of the CellMeasurer render callback's async measure parameter is not supported.

Just so everyone knows, I managed to workaround this limitation by requesting the image dimensions from the server without needing to download the whole image by using the npm module request-image-size (https://www.npmjs.com/package/request-image-size). So when the client lazy loads new image urls from the server on demand as the user scrolls down, the server requests the image sizes before sending the image urls to the client to make sure the client receives both the urls and the dimensions.

@PabloMessina You can also get the image sizes using an Image() instance's .onload event. In this example I have an array of json objects which represents my image feed items (the title, description, some other information, as well as the image urls for the images to be used in the feed). You can see here that I process through every item in my feedData array, and then I load the image using javascript ( var img  = new Image(); img.src = feedItem.thumbnailUrl;).
I then define a function in this  image instance's onload event. When the onload event fires, it means the image has been downloaded, meaning its properties have been populated with information (.height, .width, etc.) You can use this information to prepare your data for the grid (or if you are using an array of image urls, create an array of objects containing all of this data). In this onload function I see what the width and the height of the images are and inject this information as a new property into each feed item explicitly, so when I go to use them in my Masonry grid I know exactly what the widths and heights are. I also am storing the height to width ratio as heightRatio, so if I have responsive column sizes in the grid I can calculate what the height should be for each image at any given column width. (I determine the column widths through querying for the screensize, etc.). You can see I also set this data into state once the last feed item has been processed, and thus once I have all of the information I need.
For example:
      var forEachIndex = 0;

      feedData.forEach((feedItem,index) => {
          var img  = new Image();
          img.src = feedItem.thumbnailUrl;

          img.onload = () => {
              ++forEachIndex;
              var heightRatio = img.height / img.width;

              feedItem.height = img.height;
              feedItem.width = img.width;
              feedItem.ratio = heightRatio;

              if(forEachIndex == feedData.length){
                  me.setState({
                      currentDataset: feedData,
                      feedIncrements: 1,
                      feedLoading: false
                   });
               }
          }
    });`

@PabloMessina I'm having trouble using request-image-size, can you give a quick example on how/where you implemented it? Thanks!

@davidwu220, I ended up implementing a scrapper that would fetch all image dimensions off-line and save the dimensions in a .json file. The code I used is more or less like this:
const requestImageSize = require('request-image-size');
const queue = require('queue');
const fs = require('fs');
const path = require('path');

const urlsArray = require('./urlsArray.json');
const imageSizeCache = require('./imageSizeCache.json');

const q = queue({ concurrency: 30 });
const failedUrls = [];

function tryGetJob(url) {
    if (imageSizeCache[url])
        return null;

    return () => requestImageSize(url)
    .then(res => {
        imageSizeCache[url] = { width: res.width, height: res.height };
    })
    .catch(err => {
        console.error(err);
        failedUrls.push(url);
    });
}

for (const url of urlsArray) {
    job = tryGetJob(url);
    if (job) q.push(job);
}

function dump2jsonFile({_data, _path}) {
    fs.writeFile(_path, _data, (err) => {
        if (err) throw err;
    });
}

q.start(err => {
    console.log('all done!');
    console.log('dumping to json files ...');

    dump2jsonFile({
        _data: JSON.stringify(imageSizeCache),
        _path: path.join(__dirname, './imageSizeCache.json'),
    });

    dump2jsonFile({
        _data: JSON.stringify(failedUrls),
        _path: path.join(__dirname, './failedUrls.json'),
    });

    if (err) throw err;
});

Then my backend server would read the dimensions from the .json file and send the info to the React app before the masonry component is loaded.


Add documentation for trace

Overview
This PR:

Adds a separate documentation for trace command.
Adds a short information about the trace command to main readme as well as a reference to the seperate documentation.

Notes for reviewers
The main readme file still needs to be updated for new build/install process which is not included in this PR.

The main readme file still needs to be updated for new build/install process

@turkenh Now that #17 is merged, what do you think of whether we're able to include that in this changeset too?

@suskin thanks for the review.
All should be resolved now, including updates for new build/install process.


Add callback when editing on cell is cancelled

In version 1.2.2 of react-bootstrap-table2-editor, it is possible to be notified when a cell enters the editing state using onStartEdit. However, if blurToSave is set to false, there is no way to determine that editing was cancelled when the user clicks away from the editing cell and the cell transitions back to a non-editing state. It would be very useful to have a onEditCancel (or some other appropriate name) callback method that could be set for this case. When blurToSave is true, all is fine as any change in focus, leads to the afterCellSave method to be called.

@robowen5mac I see, I will consider to add a new listener for this case: onEscapeEdit ASAP, thanks

Hi @AllenFang is this issue still open?
I would like to work on this issue if anyone hasn't picked this up yet. Thanks!


[BUG] Importing data with index name with special chars creates a new index with encoded special chars

Context
I have a index with special characters in the index-name (it is a ";"). I exported the index with multielasticdump.
Afterwards I tried an import in an empty elasticsearch instance.

ElasticDump version 6.104.2
Elasticsearch version 7.8.1
Node.js version (note that nodejs v10+ is required for this tool) v20.9.0
Full command you are having issue with (with inputs and outputs): >multielasticdump --direction=load --input=testdata --output=http://<ip>:9200 --includeType=settings,data
A data-set you can share publicly which reproduces the problem:

test-data;info;version=1.settings.json
"{\"test-data;info;version=1\":{\"settings\":{}},\"test-data%3binfo%3bversion%3d1\":{}}"

test-data;info;version=1.json:
{"_index":"test-data;info;version=1","_type":"_doc","_id":"1234","_score":1,"_source":{},"fields":{}}

Describe the bug
When importing the index is created with the settings file and correctly named test-data;info;version=1.
When the data file is imported elasticdump can't find the index and creates a new index with the encoded special characters (test-data%3binfo%3bversion%3d1) instead of using the already existing index with the decoded component (test-data;info;version=1).
Expected behavior
When the index exists elasticdump should use the existing index. Otherwise a index with the decoded special characters should be created.
Additional context
Apparently when using the _bulk import the index name is retrieved from the encoded uri of the index. It would be better to either decode the uri or prefer the (correct) value of the key "_index" in the data file.
The solution can be here:
https://github.com/elasticsearch-dump/elasticsearch-dump/blob/7c4fb60941eae1baa2b42e8fa5e46366f3e595de/lib/transports/__es__/_data.js#L127
Suggestion 1:
 actionMeta[bulkAction]._index = decodeURIComponent(this.base.index) || elem._index

Suggestion 2:
 actionMeta[bulkAction]._index = elem._index || this.base.index

This isn't a bug. This is a feature request

Okay, but in my specific case this is an breaking error, because the exported files are not able to be imported.
Should I make a PR for a change? What suggestion do you prefer?

Here is a PR https://github.com/elasticsearch-dump/elasticsearch-dump/pull/1006. Test and let me know if this works for you.

changes are in https://github.com/elasticsearch-dump/elasticsearch-dump/releases/tag/v6.105.0


Refactor to separate UI code from core code

• refactored the project to separate UI code from the core functionality code.
• slight refactor of the UI code
• switched to SFSafariViewController to remove depreciated warning on openURL:
• added support for URL Schemes i.e App-Installer://Your_Download_Link
• overall easier extensibility

A new version of App Installer was released. Do you think you could update your code to be compatible with the new version? Sorry for the inconvenience, I appreciate your work.

I've updated my code to be compatible with the latest update and resolved the conflicts. I'm not sure if you were able to get the share menu working on your end, but it is not working for me so I can't test if the pasteboard option is working in this, though it should.

I'm so sorry, I pushed a bug fix for a crashing issue that simply required a reconnection of a button. If you could just update your Storyboard one more time it would be much appreciated. As long as there isn't any more crashing, your pull request is at the top of my to-do list.

I think you'll have to manually resolve these conflicts, both yours and my commit remap buttons in the storyboard and for some reason, it won't let me resolve the conflicts from my end. I double checked and my storyboard file works fine in place of yours.

You can resolve conflicts locally

@nullpixel care to elaborate a bit? I was able to resolve the last conflict in the browser, but this one says it's to complex. when trying to resolve locally all I know to do is make sure everything is operational and push the commit, but it still conflicts.

@nullpixel nevermind, I figured it out. I forgot to pull the latest update back into my fork.

Also, I was able to get the share menu working with version <IP_ADDRESS>, but I can't test the new version with your edits, because I don't permanently have access to a 3D Touch capable device.

I see that it shows up in the shortcut menu now, but it's not working properly. I've never worked with the 3D touch so I'm not very familiar with it.

It only works when it's used at app launch (not from background), because it uses didFinishLaunchingWithOptions and not applicationDidBecomeActive in the AppDelegate.

turns out they offer performActionForShortcutItem for this reason

Had no idea that existed (I just learned how to do 3D Touch shortcuts yesterday). Nice work!