TB_GETBUTTONINFOA and TB_SETBUTTONINFOA not working on 64bit application

Hi,
I have an application and would like to modify some style of buttons of a toolbar.
I raised another bug that I could not assign value to fsStyle but I fixed that on my local.
(https://github.com/dahall/Vanara/issues/326)
The issue is that it works fine when application is on 32bit but not 64bit.
You can compare the results of SendMessage for TB_GETBUTTON, TB_GETBUTTONINFOA and TB_SETBUTTONINFOA here:

And here is the code:
private void btnVanada_Click(object sender, EventArgs e)
{
    var tb = new ComCtl32.TBBUTTON();
    var tbi = new ComCtl32.TBBUTTONINFO();

    lblResult_TB_GETBUTTON.Text = User32.SendMessage(mainToolbar.Handle, ComCtl32.ToolbarMessage.TB_GETBUTTON, 0, ref tb).ToString();

    tbi.cbSize = (uint)Marshal.SizeOf(tbi);
    tbi.dwMask = ComCtl32.TBIF.TBIF_STYLE;
    lblResult_TB_GETBUTTONINFOA.Text = User32.SendMessage(mainToolbar.Handle, ComCtl32.ToolbarMessage.TB_GETBUTTONINFOA, tb.idCommand, ref tbi).ToString();

    tbi.fsStyle = tbi.fsStyle | ComCtl32.ToolbarStyle.BTNS_WHOLEDROPDOWN;
    lblResult_TB_SETBUTTONINFOA.Text = User32.SendMessage(mainToolbar.Handle, ComCtl32.ToolbarMessage.TB_SETBUTTONINFOA, tb.idCommand, ref tbi).ToString();

}

And here is the project I created.
ToolbarMessage.zip

I noticed this line of code:
https://github.com/dahall/Vanara/blob/bdf07f10f59c02669f6eef79a7b17734738e6c5d/PInvoke/ComCtl32/CommCtrl.Toolbar.cs#L3693
If I remove Pack=1 from StructLayout it works in both 32bit and 64bit.
Is there going to be any issue removing it?

All good


Quota by Priority

Feature Description

One-line feature description (can be used as a release note):
Primary contact (assignee): @bsalamat
Responsible SIGs: @kubernetes/sig-scheduling-feature-requests, @kubernetes/sig-node-feature-requests
Design proposal link (community repo):
Link to e2e and/or unit tests:
Reviewer(s) - (for LGTM) recommend having 2+ reviewers (at least one from code-area OWNERS file) agreed to review. Reviewers from multiple companies preferred: @davidopp, @derekwaynecarr, @erictune, @k82cn, @timothysc, @wojtek-t @dchen1107
Approver (likely from SIG/area to which feature belongs): @davidopp
Feature target (which target equals to which milestone):

Alpha release target: ???
Beta release target: 1.12
Stable release target: ???

rel: https://github.com/kubernetes/features/issues/268
/sig scheduling node
/kind feature
/stage beta
/milestone v1.12
/assign @bsalamat

@bsalamat @kubernetes/sig-scheduling-feature-requests @kubernetes/sig-node-feature-requests --
Any update on docs status for this feature? Are we still planning to land it for 1.12?
At this point, code freeze is upon us, and docs are due on 9/7 (2 days).
If we don't here anything back regarding this feature ASAP, we'll need to remove it from the milestone.
cc: @zparnold @jimangel @tfogo

This feature is implemented and docs are also written.
cc/ @vikaschoudhary16

And the doc is updated for 1.12.

The doc change that I updated is actually for 1.11. I am going to send a PR to update the version tag in the doc.

@bsalamat please also link it here, so we can track.

@justaugustus Done

Hi folks,
Kubernetes 1.13 is going to be a 'stable' release since the cycle is only 10 weeks. We encourage no big alpha features and only consider adding this feature if you have a high level of confidence it will make code slush by 11/09. Are there plans for this enhancement to graduate to alpha/beta/stable within the 1.13 release cycle? If not, can you please remove it from the 1.12 milestone or add it to 1.13?
We are also now encouraging that every new enhancement aligns with a KEP. If a KEP has been created, please link to it in the original post. Please take the opportunity to develop a KEP.

@ameukam - this is already in beta, @bsalamat do we plan to move this to GA in 1.13 ?

No, we don't. We should probably target 1.14 for GA of pod priority and quota by priority if things go well.

thanks for the update!
/milestone clear

I'm the Enhancement Lead for 1.15. Is this feature going to be graduating alpha/beta/stable stages in 1.15? Please let me know so it can be tracked properly and added to the spreadsheet.
Once coding begins, please list all relevant k/k PRs in this issue so they can be tracked properly.

Hi @bsalamat , I'm the 1.16 Enhancement Shadow.  Is this feature going to be graduating alpha/beta/stable stages in 1.16? Please let me know so it can be added to the 1.16 Tracking Spreadsheet. If it's not graduating, I will remove it from the milestone and change the tracked label.
Once coding begins or if it already has, please list all relevant k/k PRs in this issue so they can be tracked properly.
Milestone dates are Enhancement Freeze 7/30 and Code Freeze 8/29.
Thank you.


Cache invalid txns

Closes https://github.com/paradigmxyz/reth/issues/6350.
Removes transactions also from TransactionFetcher on_bad_import. On checking for vacant entry in pool imports or tx fetcher hashes, checks if tx is actually new by checking if it's in bad_imports cache already.

can be closed @emhane ?

can be closed @emhane ?

nope will fix conflicts and merge


Slicer instructions - incomplete?

https://github.com/shimming-toolbox/b0-fieldmap-realistic-simulation/blob/f166ff5f38b2c6ea46ebe020fcb0313add778e7f/smoothing_procedure.py#L25-L30
Hi @CharlesPageot  - I installed Slicer and am trying to replicate your smoothed skulls, however the instructions in these files are not working for me. Am I missing additional setup steps to run your script?

(base) mathieuboudreau@Mathieus-MacBook-Pro b0-fieldmap-realistic-simulation %  /Applications/Slicer.app/Contents/MacOS/Slicer --python-script smoothing_procedure.py -m data/sub-amuAL_T1w.nii.gz -s data/sub-amuAL_T1w_label-brain_dseg.nii.gz -o data/slicer_smoothing_output.nii.gz
Error #1 while writing setting "Modules/AdditionalPaths"
Error #1 while writing setting "Modules/IgnoreModules"
Error #1 while writing setting "Extensions/ManagerEnabled"
Error #1 while writing setting "Extensions/ServerUrl"
Error #1 while writing setting "Extensions/FrontendServerUrl"
Error #1 while writing setting "Extensions/InstallPath"
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
Switch to module:  "Welcome"
Populating font family aliases took 73 ms. Replace uses of missing font family ".AppleSystemUIFont" with one that exists to avoid this cost.
Traceback (most recent call last):
  File "<string>", line 5, in <module>
  File "<string>", line 3, in <module>
ModuleNotFoundError: No module named 'click'
Switch to module:  ""
Switch to module:  ""

click was installed locally through pipy in my base, but maybe Slicer doesn't see that path

Yeah it looks like Slicer doesn't see it. Maybe if you try to install click inside Slicer's embedded Python environment?


Replace svm-codec with SSZ

We are planning on replacing the encoding internally used by SVM both for storage and on-wire entities representation with SZZ. This change would:

Reduce our codebase size.
Standardize our technology stack.
Facilitate the creation of alternative SVM implementations.

SSZ would thus be used to encode:

Template sections.
Transaction execution receipts.
Transaction contents.

(1) and (2) are mostly implementation details, and changing the encoding doesn't have ripple effects across the whole stack. On the other hand, (3) transaction syntax is extremely delicate and it needs to evaluted carefully. A the time of writing, a detailed SMIP documenting the proposed transaction syntax with SSZ is not available, but there is some promising work done by @noamnelke and @lrettig in this regard. See https://docs.google.com/spreadsheets/d/1JjKL9GgYTgNRM_qrkCcpxGeSUnwuhOeTC0dfn16ivHM/edit#gid=1073868340 for more information.

See https://github.com/spacemeshos/SMIPS/issues/58 for details about how transaction receipts are currently encoded.


Implement dot1q encap for L3 sub interface - #546

Change Summary

Types of changes

[ ] 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 change)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes)
[ ] Documentation content changes
[ ] Other (please describe):

Related Issue(s)
Fixes #546
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes
Extend data model as per issue #546
ethernet_interfaces:
  ethernet1.101:
    encapsulation:
      dot1q:
        vlan: 101

Applied also on port-channel interfaces
How to test
Run Molecule scenario for eos-cli_config_gen
Checklist:

[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed (pre-commit, make linting and make sanity-lint).
[x] I have updated molecule CI testing accordingly

Will be re-submitted after enhancement of interface type enhancement to support more than switched and routed


Publish packages in ESM module format

Why

Run untranspiled on server environments (e.g. Next.js experimental.esmxternals). Could help with https://github.com/mui-org/material-ui/issues/29742#issuecomment-982597676 to give babel-plugin-styled-components a source that is easier to statically analyze.
Snowpack has trouble serving our library: https://github.com/mui-org/material-ui/issues/28628
It looks like it’s trying to convert back the commonjs version into ESM and getting confused (this is a non-deterministic process, best effort). It seems to resolve the package specifier to the commonjs module instead of ESM
Next.js supports ESM by default in v12

How

Ongoing implementation: https://github.com/mui-org/material-ui/pull/30510

Figure out what happened with these PRs:

https://github.com/mui-org/material-ui/pull/26310
https://github.com/mui-org/material-ui/pull/26656

package.json: dual ESM/commonjs: https://nodejs.org/api/packages.html#dual-commonjses-module-packages

probably needed in the top-level package.json as well as the ones in the nested folders

Internal imports will need file extensions https://github.com/jaydenseric/babel-plugin-transform-require-extensions

How do we deal with process.env.NODE_ENV (e.g. https://github.com/mui-org/material-ui/blob/1230b1a571942f1abf2fec81812b757ceb0e5d0b/packages/mui-system/src/spacing.js#L104)? It will break when run as a module in the browser.
Perhaps:
// This way it won't break uncompiled, but webpack can still
// replace `process.env.NODE_ENV`
import process from './process'

// ...
      if (process.env.NODE_ENV !== 'production') {
        if (typeof abs !== 'number') {
          console.error(
            `MUI: Expected ${propName} argument to be a number or a string, got ${abs}.`,
          );
        }
      }
// ...

// ./process.js
// Gracefully handle being run in the browser
export default typeof process === 'undefined' ? { env: {} } : process;

This doesn't work with react <=17: Bug: Cannot import 'react/jsx-runtime' from esm node/webpack 5 facebook/react#20235, We may be stuck on this until package exports are backported to react 17 (or when we can bump our react peer dependency >=18).

Publint is reporting that the components are written in ESM but getting interpreted as CJS.

I tried the MUI imports in a Next.js project with type: module and moduleResolution: node16 though, seems to work fine. (https://codesandbox.io/p/sandbox/angry-benji-oi33ry)
Is this because Next.js is reading the CJS exports under node/ directory instead of ESM ones?

In order to be able to use @mui/icons-material in ES module (with remix), I had to use named import:
import { LockOutlined } from '@mui/icons-material';

I believe it relies on the presence of "module": "./esm/index.js" field in the installed package.json which reexports everything. Reaching out to @mui/icons-material/LockOutlinedIcon like the examples do, fails.

This seems to be a blocker for getting MUI components to play nicely with Remix. Named imports appear to work fine.
❌ import Box from "@mui/joy/Box";
✅ import { Box } from "@mui/material";

As a workaround, you can do for now the following search-and-replace with regex to fix it:
Search import (.*)Icon from "@mui/icons-material/(.*)"; and replace with import { $1 as $1Icon } from "@mui/icons-material";

Output .cjs file, set it under package.json should make it possible use it with tsconfig esModuleInterop: true. Currently you can't do anything in esm configured project, because there is just a .js file in the build. And this .js file uses esm syntax.

Requirement for both CommonJS and ESM support from Docker: https://app.supernormal.com/posts/mui-docker-x-charts-6b722290c9c8013c6d350e8110019023

Would be interested in what those infrastructure requirements are exactly. What mandates the need for a commonjs bundle? Does their bundler not understand ESM? Or does their SSR setup not compile modules on the server? It's an odd choice to try and fix this by adding a commonjs version to their chart library, rather than to force their bundler to transpile the dependency.

Sadly, I'm removing this issue from the v6 milestone, tentatively moving it to v7. I'll write an in-depth document on what we tried, why we're blocked, and the steps forward. As a summary, we found a blocker with NextJS Pages Router: https://github.com/vercel/next.js/issues/65056. The probable path forward is removing the esmExternals: false from our documentation infra, but that's currently blocked by https://github.com/vercel/next.js/issues/49898#issuecomment-1762202697

Just recording some tips for anyone that wants to debug this:

Remove all pages and add just a single page with
import * as React from 'react';
import { DataGridPro } from '@mui/x-data-grid-pro';

export default function MyPage() {
  return <DataGridPro rows={[]} columns={[]} />;
}

This will drastically speed up the build and reproduce the error (pnpm docs:build)

Disable the minifier to be able to inspect the bundles more easily. Look into the server bundles (.next/server/pages/...) If you've only got 1 page, there should be no code splitting happening.
// next.config.mjs
  // ...
  webpack: (config, options) => {
	    // ...
    config.optimization.minimize = false;
    config.optimization.minimizer = [];
	    // ...

It looks like webpack isn't using the ESM bundle of @mui/x-data-grid. We may be able to fix this by:

Being strict about file extensions, our packages are type: 'commonjs', so our ESM files should have the .mjs extension.
For Node.js to be actually able to load these ESM files, we'll need to make sure the contain only fully resolved module specifiers.

Currently the issue is maximized in the sense of blocking every possible simple workaround at once, by exporting .js, not .cjs (for interop workaround TS) or .mjs and at the same time using esm syntax in the .js build:
https://github.com/mui/material-ui/issues/30671#issuecomment-1941097296
main points to .js:

ESM syntax in .js output:

I've gotten used to copy-pasting icons from the website into my project. I forgot about this issue and accidentally broke my app in dev. That's why I went out of my way to prevent this from happening in the future by adding an eslint rule to prevent imports of the icons as in the copypasta:
       "no-restricted-imports": ["error", {
          "patterns": [{
            "group": ["@mui/icons-material/*"],
          }]
        }]


Add iOS(13) to list of platforms

Resolves https://github.com/apple/swift-async-dns-resolver/issues/17

@swift-server-bot test this please


CD Releases to GitHub packages

Use release-please to generate review-gated PRs that tag releases, make changelogs, and submit the code to github packages

Close in favor of https://github.com/opentdf/java-sdk/issues/79


Change the format of object sent on completion

Currently you send this object;
{
  "desc":"Image was resized and saved to database.",
  "filename":"0_555bdf3b9331c0",
  "sizes":[
    {"width":800,"height":800,"name":"large"},
    {"width":500,"height":500,"name":"medium"},
    {"width":200,"height":200,"name":"small"},
    {"width":45,"height":45,"name":"thumbnail"}
  ]
}

Can we have the size object return this info?
{
  "width":45,
  "height":45,
  "label":"thumbnail", // the name given in the config
  "file": "thumbnail.jpg" // the file name that was created
}

Does this have anything to do with the bug #11 that you have reported?

Not really.  Its the addition of a "label" and "file" value to the object that this one is concerned about...


the environment passed is not a vectorized environment, however PPO2 requires it

Describe the bug
Unable to set_env in PPO2. The error thrown is " the environment passed is not a vectorized environment. However, PPO2 requires it", but the environment is vectorized !
Code example
env = gym.make('InvertedPendulum-v2')
    model = PPO2.load(
                'models/initial_policy_invpendulum.pkl',
                env=env,
                verbose=1)

    env = SubprocVecEnv([lambda: gym.make('InvertedPendulum-v2')])
    model.set_env(env)

    model.learn(total_timesteps=100000, reset_num_timesteps=False)

System Info
Describe the characteristic of your environment:

Describe how the library was installed (pip, docker, source, ...)
from source
GPU models and configuration
RTX 2060
Python version
3.6.8

Why closed? I'm having the same issue.

If your code is same as above, then the env is not vectorized in the load step. Set env=None in load, or change env to be VecEnv at the very beginning.


[firebase_auth] Invalid SMS verification code exception not handled on iOS

The invalid SMS verification code exception is not handled on the native side on iOS, and as such, when the wrong verification code is entered, onVerificationCode does not get called, and the plugin prints out the following log in the console:
Unhandled exception:
PlatformException(Error 17044, FIRAuthErrorDomain, The SMS verification code used to create the phone auth credential is invalid. Please resend the verification code SMS and be sure to use the verification code provided by the user.)

Please add the output of flutter doctor -v.

The output is as follows:
[✓] Flutter (Channel beta, v0.5.1, on Mac OS X 10.13.6 17G65, locale en-RO)
    • Flutter version 0.5.1 at /Users/alex/dev/flutter
    • Framework revision c7ea3ca377 (3 months ago), 2018-05-29 21:07:33 +0200
    • Engine revision 1ed25ca7b7
    • Dart version 2.0.0-dev.58.0.flutter-f981f09760

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.0)
    • Android SDK at /Users/alex/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.0
    • ANDROID_HOME = /Users/alex/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.4.1, Build version 9F2000
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 27.1.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] IntelliJ IDEA Ultimate Edition (version 2018.1.5)
    • IntelliJ at /Users/alex/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 25.0.2
    • Dart plugin version 181.4892.1

[!] VS Code (version 1.26.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected devices (1 available)
    • iPhone • 9f1683e8f02bc5200ca070e8648a23cd990a4759 • ios • iOS 10.3.2

! Doctor found issues in 1 category.

I have the same issue in the android side. And whats worst is that the exception comes with a generic code, "exception" so no way to correctly identify this.
If the sms code expires and that method is called, another exception is thrown, with this same code, and a different message.
My currently workaround (working in android, maybe not in ios) is check for the message:
try {
  final user = await auth.signInWithPhoneNumber(verificationId: verificationId, smsCode: smsCode);
  // ...
} on PlatformException catch (e) {
  print('platformException: $e');
  if (e.message.contains('The sms verification code used to create the phone auth credential is invalid')) {
    // ...
  } else if (e.message.contains('The sms code has expired')) {
    // ...
  }
}

I don't see the relevance in that case, but here is the output of flutter doctor:

[✓] Flutter (Channel beta, v0.7.3, on Linux, locale en_US.UTF-8)
• Flutter version 0.7.3 at /opt/flutter
• Framework revision 3b309bda07 (11 days ago), 2018-08-28 12:39:24 -0700
• Engine revision af42b6dc95
• Dart version 2.1.0-dev.1.0.flutter-ccb16f7282
[✓] Android toolchain - develop for Android devices (Android SDK 28.0.2)
• Android SDK at /home/gabriel/Android/Sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.2
• Java binary at: /opt/android-studio/jre/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
• All Android licenses accepted.
[✓] Android Studio (version 3.1)
• Android Studio at /opt/android-studio
• Flutter plugin version 28.0.1
• Dart plugin version 173.4700
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
[✓] IntelliJ IDEA Ultimate Edition (version 2018.2)
• IntelliJ at /home/gabriel/.local/share/JetBrains/Toolbox/apps/IDEA-U/ch-0/182.4129.33
• Flutter plugin version 28.0.4
• Dart plugin version 182.4129.13
[✓] VS Code (version 1.27.1)
• VS Code at /usr/share/code
• Flutter extension version 2.18.0
[✓] Connected devices (2 available)
• moto x4 •<PHONE_NUMBER>       • android-arm64 • Android 8.0.0 (API 26)
• moto x4 • <IP_ADDRESS>:5555 • android-arm64 • Android 8.0.0 (API 26)
• No issues found!

I've experienced the same. Here's my Sentry log:
PlatformException(ERROR_INVALID_VERIFICATION_CODE, The sms verification code used to create the phone auth credential is invalid. Please resend the verification code sms and be sure use the verification code provided by the user., null)
This was caught in:
firebaseUser = await firebaseAuth
            .signInWithCredential(credential)
            .catchError((err) async {
    SentryClient.report(err);
});

I also think that lots of phone_auth related error codes aren't documented properly. For example, there's no documentation of what ERROR_INVALID_VERIFICATION_CODE means.

I have the same issue in Android and IOS.

same issue with me.. any one got solution please share...

Unhandled Exception: PlatformException(ERROR_SESSION_EXPIRED, The sms code has expired. Please re-send the verification code to try again., null)
class PhoneSignInSection extends StatefulWidget {
var phonenumber;
PhoneSignInSection(this._scaffold, this.phonenumber);
final ScaffoldState _scaffold;
@override
State createState() => _PhoneSignInSectionState(phonenumber);
}
class _PhoneSignInSectionState extends State {
final _phoneNumberController = TextEditingController();
final _smsController = TextEditingController();
final GlobalKey _scaffoldKey = new GlobalKey();
var phonenumber;
String codeSent;
String _message = '';
String _verificationId;
String _code;
FirebaseAuth _auth = FirebaseAuth.instance;
_PhoneSignInSectionState(this.phonenumber);
@override
void initState() {
// TODO: implement initState
super.initState();
setState(() {
_verifyPhoneNumber();
});
}
@override
Widget build(BuildContext context) {
return Scaffold(
body: Stack(
children: [
new Container(
decoration: new BoxDecoration(
image: new DecorationImage(
image: new AssetImage("assests/background_shininghealth.png"),
fit: BoxFit.cover,
),
),
),
Column(
crossAxisAlignment: CrossAxisAlignment.start,
children: [
ListTile(
title: Padding(
padding: const EdgeInsets.only(top: 180),
child: Container(
alignment: Alignment.topCenter,
child: Text(
"OTP Verification",
maxLines: 1,
style: new TextStyle(
fontWeight: FontWeight.bold,
color: Color(0xFFBE3837),
fontSize: 35,
fontFamily: "montserrat"),
),
),
),
),
TextField(
controller: _smsController,
decoration: InputDecoration(labelText: 'Verification code'),
),
Container(
padding: const EdgeInsets.symmetric(vertical: 16.0),
alignment: Alignment.center,
child: RaisedButton(
onPressed: () {
_testSignInWithPhoneNumber(_smsController.text);
},
child: const Text('Sign in with phone number'),
),
),
],
)
],
),
);
}
// Exmaple code of how to veify phone number
void _verifyPhoneNumber() async {
setState(() {
_message = '';
});
final PhoneVerificationCompleted verificationCompleted =
(AuthCredential phoneAuthCredential) {
_auth.signInWithCredential(phoneAuthCredential);
setState(() {
_message = 'Received phone auth credential: $phoneAuthCredential';
});
};
final PhoneVerificationFailed verificationFailed =
    (AuthException authException) {
  setState(() {
    _message =
        'Phone number verification failed. Code: ${authException.code}. Message: ${authException.message}';
  });
};

PhoneCodeSent codeSent =
    (String verificationId, [int forceResendingToken]) async {
  widget._scaffold.showSnackBar(SnackBar(
    content:
        const Text('Please check your phone for the verification code.'),
  ));
  _verificationId = verificationId;
};

final PhoneCodeAutoRetrievalTimeout codeAutoRetrievalTimeout =
    (String verificationId) {
  _verificationId = verificationId;
};

await _auth.verifyPhoneNumber(
    phoneNumber: phonenumber,
    timeout: const Duration(seconds: 60),
    verificationCompleted: verificationCompleted,
    verificationFailed: verificationFailed,
    codeSent: codeSent,
    codeAutoRetrievalTimeout: codeAutoRetrievalTimeout);

}
/// Sends the code to the specified phone number.
Future _sendCodeToPhoneNumber() async {
final PhoneVerificationCompleted verificationCompleted =
(FirebaseUser user) {
setState(() {
print(
'Inside _sendCodeToPhoneNumber: signInWithPhoneNumber auto succeeded: $user');
});
} as PhoneVerificationCompleted;
final PhoneVerificationFailed verificationFailed =
    (AuthException authException) {
  setState(() {
    print(
        'Phone number verification failed. Code: ${authException.code}. Message: ${authException.message}');
  });
};

final PhoneCodeSent codeSent =
    (String verificationId, [int forceResendingToken]) async {
  this._verificationId = verificationId;
  print("code sent to " + _phoneNumberController.text);
};

final PhoneCodeAutoRetrievalTimeout codeAutoRetrievalTimeout =
    (String verificationId) {
  this._verificationId = verificationId;
  print("time out");
};

await FirebaseAuth.instance.verifyPhoneNumber(
    phoneNumber: phonenumber,
    timeout: const Duration(seconds: 60),
    verificationCompleted: verificationCompleted,
    verificationFailed: verificationFailed,
    codeSent: codeSent,
    codeAutoRetrievalTimeout: codeAutoRetrievalTimeout);

}
Future _testSignInWithPhoneNumber(String smsCode) async {
final AuthCredential credential = PhoneAuthProvider.getCredential(
verificationId: _verificationId,
smsCode: smsCode,
);
final FirebaseUser user = await _auth.signInWithCredential(credential);
final FirebaseUser currentUser = await _auth.currentUser();
assert(user.uid == currentUser.uid);
_smsController.text = '';
return 'signInWithPhoneNumber succeeded: $user';

}
/*
_testSignInWithPhoneNumber() async
{
/*
try {
final AuthCredential credential = PhoneAuthProvider.getCredential(
verificationId: _verificationId,
smsCode: _smsController.text,
);
  final FirebaseUser user = await _auth
      .signInWithCredential(credential)
      .then((user) => Navigator.of(context).pushReplacementNamed('/form'));
} catch (e) {
  print(e);
}

}

Issue still exists. Any timeline when it can be improved?

@derTuca
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/687. Any further collaboration will be done there.

i have solved this error by this method
signIn() async {
final AuthCredential credential = PhoneAuthProvider.getCredential(
verificationId: verificationId,
smsCode: smsCode,
);
try {
var firebaseUser =
await FirebaseAuth.instance.signInWithCredential(credential);
print("Login Successful");
print(firebaseUser.user);
Navigator.pushReplacementNamed(context, 'homepage');
} catch (e) {
  print("Login not successful");
  Navigator.pop(context);
}

}

i have solved this error by this method
signIn() async {
final AuthCredential credential = PhoneAuthProvider.getCredential(
verificationId: verificationId,
smsCode: smsCode,
);
try {
var firebaseUser =
await FirebaseAuth.instance.signInWithCredential(credential);
print("Login Successful");
print(firebaseUser.user);
Navigator.pushReplacementNamed(context, 'homepage');
} catch (e) {
  print("Login not successful");
  Navigator.pop(context);
}

}

@tanmoy27112000  Its works. Thanks!

There is a need of adding an error enum just like we have here PhoneAuth.ERROR_NUMBER_NOT_VALID


Specify Python3 language

Overview
This PR addresses concerns raised in https://github.com/Ouranosinc/pavics-sdi/issues/77
Changes:

Changed generic tags to Python3 only

Related Issue / Discussion
Additional Information
Links to other issues or sources.

@Zeitsperre simple change :) Feel free to merge.


upload date

the doujin info should contain the upload timestamp

This feature is going to be available in the next release

This feature is available on 0.0.19
pip3 install NHentai-API==0.0.19


SCRUM-1614

Ready for review.

Hey @ianlongden .  Just remembered that before Blue team started, the create reference API would take the next highest curie number, meaning that if someone created a reference, deleted it, and created a new reference, it would reuse the same curie.  I haven't looked at the PR files yet, but wanted to check in with you about whether the create endpoint should check the obsolete_reference_curie table when creating a new curie.  Alternatively, it should just be a sequence and not look up new curies however it used to do it ?

@ianlongden @valearna It all looks good to me.  I just want to point out the first comment I made before starting the PR review about how create makes new curies, and whether it should check the obsolete table.

Yep this is a good spot. I will add something to fix that after lunch. Thanks.
-Ian.

On Jun 24, 2022, at 12:55 PM, Juancarlos Chan @.***> wrote:
@ianlongden https://github.com/ianlongden @valearna https://github.com/valearna It all looks good to me. I just want to point out the first comment I made before starting the PR review about how create makes new curies, and whether it should check the obsolete table.
—
Reply to this email directly, view it on GitHub https://github.com/alliance-genome/agr_literature_service/pull/143#issuecomment-1165758438, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA7X4RQVRZWPG2XWZUFERN3VQXSAVANCNFSM5ZRQYDPQ.
You are receiving this because you were mentioned.

@ianlongden The UI expects specific status codes for doing different types of requests.  PATCH 202  DELETE 204  POST 201  .  It works with all previous POST requests returning 201, but the references/merge//  endpoint is returning 200 instead of 201.  https://restfulapi.net/http-status-codes/ seems like it should be a 201, maybe more so because it's returning the new curie instead of just 'OK'.  Does this sound right to you ?


nm_custom_draw should provide a mutable reference?

Hi!
Thanks for the great lib!
I am trying to change the colors for certain items in a list view.
So i used the nm_custom_draw event.
This is my custom draw code:
        self.list_view.on().nm_custom_draw({
            move |draw: &mut winsafe::NMLVCUSTOMDRAW| {
                match draw.mcd.dwDrawStage {
                    CDDS::PREPAINT => {
                        debug!("PREPAINT");
                        Ok(co::CDRF::NOTIFYITEMDRAW)
                    },
                    CDDS::ITEMPREPAINT => {
                        // set color here?
                        debug!("ITEMPREPAINT");

                        draw.clrText = COLORREF::new(255, 255, 255);
                        draw.clrTextBk = COLORREF::new(155, 0, 0);

                        Ok(co::CDRF::DODEFAULT)
                    }
                    _ => Ok(co::CDRF::DODEFAULT)
                }
            }
        });

In order to set the colors i needed to change the nm_custom_draw function to provide a &mut NMLVCUSTOMDRAW.
Original  nm_custom_draw:
	/// [`NM_CUSTOMDRAW`](https://learn.microsoft.com/en-us/windows/win32/controls/nm-customdraw-list-view)
	/// notification.
	pub fn nm_custom_draw<F>(&self, func: F)
		where F: Fn(&NMLVCUSTOMDRAW) -> AnyResult<co::CDRF> + 'static,
	{
		self.0.wm_notify(co::NM::CUSTOMDRAW,
			move |p| Ok(Some(func(unsafe { p.cast_nmhdr::<NMLVCUSTOMDRAW>() })?.raw() as _)));
	}

Changed:
	/// [`NM_CUSTOMDRAW`](https://learn.microsoft.com/en-us/windows/win32/controls/nm-customdraw-list-view)
	/// notification.
	pub fn nm_custom_draw<F>(&self, func: F)
		where F: FnMut(&mut NMLVCUSTOMDRAW) -> AnyResult<co::CDRF> + 'static,
	{
		self.0.wm_notify(co::NM::CUSTOMDRAW,
			move |p| Ok(Some(func(unsafe { p.cast_nmhdr_mut::<NMLVCUSTOMDRAW>() })?.raw() as _)));
	}

This works as expected:

Question:

Am i missing something or should it be a mutable reference?
If it should be mutable i can offer a PR if you want (also for the other nm_custom_draw events of other controls).

That was fast, thank you @rodrigocfd !
I can confirm, it works as expected.
One note: I needed to assign the colors this way:
let txt_clr = COLORREF::new(highlight.fg_color.0, highlight.fg_color.1, highlight.fg_color.2);
unsafe {
    *draw.clrText.as_mut() = txt_clr.raw();
}

If i do it any other way the compiler optimizes it away in release mode 🤷.

Won't this work for you?
self.lst_entries.on().nm_custom_draw(|p| {

	let blue = w::COLORREF::new(0x00, 0x00, 0xff);

	p.clrText = blue; // simple assignment!

	// ...

	Ok(co::CDRF::DODEFAULT)
});

Unfortunately not.
I made a quick repo reproducing the behavior if you want to try it out.
Do you have any ideas what is happening here?
This is a learning project for me, so it's not a problem if i have to use unsafe.
But it would be nice to understand what exactly is going on here.

The root cause was way deeper than the message handling itself. The lambda parameter was originally const, then later cast to mutable, thus causing undefined behavior.This is a rule that comes from C++, which I fortunately already faced in my C++ days.
I still don't know how this change will propagate throughout all WM_NOTIFY events, but so far everything seems fine.
Your example is working for me with all profiles now. Let me know if it works for you.

I've done some small testing with max optimizations enabled.
So far everything works fine. Thank you so much for the support!
Also thanks for the info. I checked your changes and the stackoverflow post you linked.
This makes sense to me now.
I still have a bunch of features to implement, so i keep an eye out for anything suspicious.


there is no requirements.txt file.

Hi, thanks for releasing the source code.
Could you share the requrements.txt file?
It doesn't exists in this repository and i have some trouble about package version.
Thanks,

I found it in ./downstream_imagenet


Update license in setup.py

As a complement to the license change. 🎉

Thanks so much!
On (Sep-24-20|15:28), Dale Roberts wrote:

Merged #3 into master.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://urldefense.com/v3/__https://github.com/daleroberts/hdmedians/pull/3*event-3805310456__;Iw!!Mih3wA!VnFOnW20mcuuLFfo6tshlKlCsi4yqwh1LvNZwgwOYf4WtEUHXRskRc9D7lmOHuA$

No problem. I’ve updated the pypi package now too.
On 25 Sep 2020, at 8:51 am, Yoshiki Vázquez Baeza <notifications<EMAIL_ADDRESS>wrote:
Thanks so much!
On (Sep-24-20|15:28), Dale Roberts wrote:

Merged #3 into master.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://urldefense.com/v3/__https://github.com/daleroberts/hdmedians/pull/3*event-3805310456__;Iw!!Mih3wA!VnFOnW20mcuuLFfo6tshlKlCsi4yqwh1LvNZwgwOYf4WtEUHXRskRc9D7lmOHuA$

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/daleroberts/hdmedians/pull/3#issuecomment-698627875, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAEOGJFXEL7DUJZANMTSGFLSHPENLANCNFSM4FQB64FQ.

That's spectacular, thanks!


blame: fix edge case with missing \n in content length causing mismatched length error

Fixes #725

Replace https://github.com/src-d/go-git/pull/849


Simplify proxy list

open a ticket to manage the issue
https://github.com/wildfly/jandex/issues/94


doc(README): Link to Gitpod

The link to https://gitpod.io/#https://github.com/leanprover-community/mathlib4 was present in the mathlib3 README. It is unclear why this was not ported.

bors merge


Example with MobX on v4

I thought it could be useful to everybody to have a simple example using MobX to manage the state.
Is anyone able to provide one example? or even an existing use case?

Duplicate of #2094


feat: remove the unnecessary await linter rule

Description
The dartunnecessary_await_in_return linter rule can be dangerous, since it messes with error handling in async functions. We should remove it.

https://github.com/dart-lang/linter/issues/2357


BUG: standalone map legend colors should NOT change as the subset changes

on qa.restricted.clinepidb.org, using the PRISM Resistance surveillance study, the legend colors change as the filtered data changes.
If I filter "District" = Kanungu & Kabale and use donut markers to look at Plasmodium genotype / k13 Ala675Val, the legend looks like this:

If I filter "District" = Lamwo and use donut markers to look at Plasmodium genotype / k13 Ala675Val, the legend looks like this:

I think we are sorting by abundance, but should instead order by vocabulary order: https://github.com/VEuPathDB/web-monorepo/blob/f16f3e6d5bb7ca9a3e001148f4db4ca076ab3eea/packages/libs/eda/src/lib/map/analysis/utils/defaultOverlayConfig.ts#L178-L180

I think abundance order is what we want for the map, at least for high-cardinality variables. We just don't want that to change for low.
Oh, looky here: https://github.com/VEuPathDB/web-monorepo/issues/279


[New] find out the best/worst scenarios

Summary of this new feature
Scenario.describe() calculates the following representative values and the new prediction method #1010 will produces many scenarios. Users will select the important representative values and covsirphy will find out the best/worst scenarios for scenario analysis.

max(Infected): max value of Infected
argmax(Infected): the date when Infected shows max value
Infected on …: Infected on the end date of the last phase
Fatal on …: Fatal on the end date of the last phase

CovsirPhy version: 2.23.0

To make it convenient, users will specify

included scenarios,
excluded scenarios,
reference variable,
assessment date,
new name of the best scenario, and
new name of the worst scenario.

When delete_unselected=True, unselected and included scenarios will be deleted.

To remove outliers, 0.05 and 0.95 quantiles (default) will be used.

snl.predict(days=30)
snl.find(excluded=["Main"], variable="Fatal", date="30May2022", quantile=0.05, renamed="Best")
snl.find(excluded=["Main"], variable="Fatal", date="30May2022", quantile=0.95, renamed="Worst")
snl.delete(name="Multivariate*)

Alternative will be used.
best, worst = snl.represent(q=[0.5, 0.95], variable="Fatal", date=None, excluded=["Main"])
snl.rename(old=best, new="Best")
snl.rename(old=worst, new="Worst")

q is the quantile and The number of variable will be compared on date (None means the end date of the last phase) and the scenarios which have the nearest values to the quantiles will be returned (to best and worst, strings). Users can specify the tartget scenarios to compare with included and excluded.

Users will rename the scneario name with Scenario.rename() #1022.


Add "Max Interval" flushing mechanism

This addresses the issue @ https://github.com/zachdaniel/spandex/issues/37 by configuring a 'max interval' for traces to stay waiting. After this period of time, they will be sent. This is important, as the datadog agent discards traces received that are older than 10 seconds.
Open to suggestions/changes - I just wanted to get this going for testing with some services locally.

Coverage increased (+0.7%) to 69.912% when pulling 8df97db8abbae7705bda8a7e0a8a4754c23ad8dd on aspett:max-interval into d5dd2f5d5c5afd28a76ba1be11f8d5dd7eccb13a on zachdaniel:master.

I'm worried that this won't have the advertised effects when there is large load. I agree that its not really a good thing to only consider sending spans when a new one is received, but in the case where the mailbox for this server is growing, this flush message could actually be received far later than intended.
I think that if your load isn't enough to fill up even a small batch within 10 seconds, then a batch size of 1 should work just fine. However, I believe you mentioned that it is seasonal/sporadic which this wouldn't account for. In order to implement this without causing issues during times of load (or at least it just not working under heavy load), I think you'd want to instead check every so often, say 1 second, for any traces approaching their max interval, and send those (this can be done by looking at the time stamp). At the same time, you'd want to duplicate this logic whenever you are sending spans, so that each message to send a batch also sends any additional batches it needs to flush them all.
It still isn't fully solved by that though, because the server could be so backed up that you just wouldn't have time.
What kind of load are you looking at? We were sending 200 traces per second, each with 5-10 spans with a batch_size of only 2. So if you're sending less data than that, then maybe you don't need this and can just set the batch_size to 1?

I'm worried that this won't have the advertised effects when there is high load. I agree that its not really a good thing to only consider sending spans when a new one is received, but in the case where the mailbox for this server is growing, this flush message could actually be received far later than intended.

Hmm this is a good point. One thing I'd considered doing is actually making a ref, storing the latest one, and sending a message more like {:interval, ref} and checking the ref matches the latest one in case it wasn't current anymore. The more I think about it, that would be a better solution than what this currently is.
I'm mostly solving this problem at the moment for dev/QA environment setups - but looking for it to scale better in the future for (yet to be released) prod. Hence I'd really like to take advantage of the current backpressure mechanism, while still catering for low points.
What do you think of the above idea as opposed to this?

I still think even with sending the ref that you could find yourself just not honoring the contract of "spans will only wait x amount of seconds maximum". Granted, that just isn't really a hard guarantee we could ever make, so a good faith effort to do so isn't necessarily a bad thing. I will say that if you're not planning on releasing your production piece for at least a couple weeks, I intend to have sampling (and really everything on the list I gave you) done by then. I think using a proper sampling strategy would just let you always send spans as soon as they are sampled, without incurring any load penalties.

👍 Sounds good. I think I'm going to make that refactor on my fork nevertheless so it can be used in the meantime. I look forward to seeing how those improvements go & look!


Breaks on RN26

As of<EMAIL_ADDRESS>you have to change the way you import React .
My changes are here
Can someone check for backwards compatibility before I sumbit PR?

I tested it. Works fine<EMAIL_ADDRESS>Do you submit a PR?

nope, just did.


⚠ Bofrost.lu has degraded performance

In c3ab16b, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1341 ms

Resolved: Bofrost.lu performance has improved in 5c4a329 after 14 minutes.


Add trail plotting option

While working on the different Lambert's issues I thought it may be useful to have a faded trail plotting option. Just pass the keyword  trail=True and the path of the orbit will be shown. As an example:
plotter = StaticOrbitPlotter()
plotter.plot(iss, trail=True, color="red")
plt.show()

This looks very nice! It reminds me of this library we mentioned in the chat a while ago: https://github.com/hannorein/rebound/blob/master/ipython_examples/OrbitPlot.ipynb
I have two requests:

From the implementation point of view, perhaps we could make it simpler? https://matplotlib.org/3.1.0/gallery/lines_bars_and_markers/multicolored_line.html
From the visual point of view, could we make the gradient in a way the complete orbit can be seen? (as in rebound)

Regarding the implementation of the algorithm I tried the suggested multicolored lines. The problem is that it does not work with colors but instead with cmap. Also, we can not make use of the method get_color in LineCollection instances... Both things increase the difficulty of working with multicolored lines.
But plotting the whole orbit is not a huge problem. In fact, it looks nicer than just plotting some part of it:

Also, we can not make use of the method get_color in LineCollection instances...

What do you mean? In principle it's listed there https://matplotlib.org/3.1.0/api/collections_api.html#matplotlib.collections.LineCollection.get_color and also https://matplotlib.org/3.1.0/api/collections_api.html#matplotlib.collections.LineCollection.get_colors

The problem is that it does not work with colors but instead with cmap

True, but maybe we can do a colormap between the target color and the same color with alpha=0?

What I'm concerned is that we are plotting the points one by one, and that makes the appearance of the line less smooth and perhaps difficults the rendering

Oh, we need to use pytest --mpl for that!

This will need a rebase after https://github.com/poliastro/poliastro/pull/693 is merged.

Ups! Sorry for the error uploading the test. Now it is working properly :+1:

Merging, thanks a lot @jorgepiloto!


add spatial property change notify, about transform

transform property editbox should automatically changed along with scale /translation ..

Sorry for the late review, we had a lot of backlog after the 3.0 release.
We discussed it with @reduz, and the property editor is due a full rewrite as it makes it quite difficult to keep track of changes, forcing to notify all the time as you're adding in this PR. So we'll close this PR pending the editor rewrite that is planned for Godot 3.1 and should solve all this. Thanks for the contribution nevertheless!


Needs support for reading configuration from YAML files.

I see that currently it just supports config from json file, it will be better to have it support yaml file as well.

I could see yaml parsing if there's need for it..
Is this something you're currently in need of support for?

Having a YAML config support is good and we use YAML extensively in our projects. So, it would be good to have sooner.

Alright, I personally haven't seen much use for it other than for service providers, but I can see why applications may use it for their own configuration.
Will look into this support for an upcoming release :+1:


i18n: How to generate dynamic urls per language?

I have an endpoint that generates urls for all my products:
nuxt.config:
sitemap: {
    sources: [
      process.env.NUXT_PUBLIC_API_URL + '/public/products/sitemap-urls',
    ],
  },

endpoint:
export default defineCachedEventHandler(async (event): Promise<SitemapUrl[]> => {
    const items = await useDb()...

    return items.map((item) => ({
         loc: `de/produkte/${getProductCleanUrl(item.name, item.company)}/${item.id}`,
         lastmod: item.lastUpdated.toISOString(),
    }));
});

But this generated the same urls for all languages, so they show up in de-DE-sitemap.xml and en-GB-sitemap.xml and fr-FR-sitemap.xml.
How can I generate urls per language?
UPDATE: Seems like it just works if I add to the same endpoint both /de and /en routes they get added to the corresponding sitemaps.

Hi, glad you could figure this out.
The guide for this can be found at Dynamic URLs with i18n, let me know if you have any further questions.

The docs dont say anything about it, maybe you want to add this information?
Also I was wondering if there is any interface for the handler to return, what I did to figure it out was trial and error :(

I think this is intuitive enough not to document, I think the only thing that tripped you up was the missing leading slash which could be more considered a bug.
There is guidelines for the types on the Dynamic URL Endpoint page.

I think the whole docs were missing a few informations for dynamic urls in general, even before I added i18n. I just found out by trial and error.
My biggest question was: What does url endpoint need to return? There was no interface or explaination. In the end I found out by trial and error that I can just return an array of string and later it can also be an object with loc and lastmod. Is there not Interface?

If you use any of the functions it tells you you'll have full typescript definitions of what to return.
Otherwise, just open the Schema documentation.

Oh now I understand, you mean the user should use defineSitemapEventHandler() instead of defineEventHandler()?
This wasnt clear for me. Personally I would argue that adding another eventHandler function creates an unnecessary abstraction. For example some users might want to use defineEventHandler() while others want to use defineCachedEventHandler().
Checking defineSitemapEventHandler()´ made me find the interface that I was looking for: SitemapUrlInput` is what the handler should return. Unfortunately the package does not export this interface so it cant be used.

You should use the asSitemapUrl composable,


Uri::path() and and Uri::path_and_query() have confusing semantics

path() returns &str which means it is never missing. Then why does path_and_query() return Option<&str> instead of &str? In other words, if the path is never missing then how can path_and_query() ever be missing? Under what circumstances would path_and_query() return None?
path() returns * when the URI is *. This seems really error prone. Regardless, what does path_and_query() return when path() returns *? This should be documented.
I would expect that url.path_and_query().unwrap().starts_with(url.path()) never panics and is always true, and that url.path_and_query().unwrap() == url.path() + url.query().map(|q| "?" + q) never panics and is always true. That is, I would expect that url.path_and_query() never returns None.

If the return type of Uri::path_and_query is intended to explicitly indicate when the query part is not present, I might expect it to return something like (&str, Option<&str>), where the path part is always returned and the query part is optional?

@briansmith you're not wrong...
The various APIs evolved over time. There should definitely be a re-evaluation of how everything works.
The problem is the HTTP uri component always includes a path except when * or for CONNECT requests. This means that for 99% of the use cases, there will always be a path. As such, having uri.path() return Option is going to be pretty annoying.
The question is if there is a way to have an "easy" API for the 99% cases while still exposing the subtleties when you care.
One option (without an opinion of quality of the option) would be to switch Uri::path to panic if there isn't a path component present.
Also, currently path_and_query does not return a &str, but a ref to a PathAndQuery. It is None in the two cases above and Some otherwise.

A question arises in https://github.com/carllerche/h2/pull/228#issuecomment-370584403:
Should a URI that does not include a path default to /?


Updated method of logging exceptions in Request.process

Exceptions raised while calling a method in Request.process were being logged using just the logging utility, which relied on a logging handler being defined. In the event a logging handler is not defined and an exception is raised while calling a method, an error about the lack of a logging handler is raised, and the original exception is never logged.
This change makes use of the _log utility, which takes into account situations where a logging handler is not defined. In the scenario described before, a stacktrace of the initial error is logged as expected.

Good pickup, thanks


Bring back and improve text for docket sheet buttons #51.

The old 'No' button actually had text that was confusing in the context of the new text. Also, the exit icon wasn't appropriate for the 'Yes' button.
Should they just be 'Yes' and 'No' at this point? The question at the bottom seems clear enough. Would it help to add a horizontal line before that last question?

Wasn't there another change in the original PR? To this line https://github.com/knod/docassemble-juvenilesealing/blob/master/docassemble/juvenilesealing/data/questions/petitioner-entrypoint-10.yml#L197 to make it read something like "What's so special about certified docket sheets?"

Yes! Good catch!

What are your thoughts on simplifying the buttons?

What are your thoughts on simplifying the buttons?

I'm not a UX person so discount what I say appropriately. I think the text you have is good for now. We can change it later once we have more user feedback.


ValueError: does not appear to be an ipv4 or ipv6 address

Hello, I have trouble running your ee-timeseries method. I am pretty new to coding and overall not a very tech savy guy, but I managed to install docker and launch your dockerfile.
I run into a problem at the last step when you get to jupyter notebook with
jupyter-notebook ee-atmcorr-timeseries.ipynb --ip='*' --port=8888 --allow-root
What I get is this
Traceback (most recent call last):
  File "/miniconda/lib/python3.6/site-packages/notebook/notebookapp.py", line 869, in _default_allow_remote
    addr = ipaddress.ip_address(self.ip)
  File "/miniconda/lib/python3.6/ipaddress.py", line 54, in ip_address
    address)
ValueError: '' does not appear to be an IPv4 or IPv6 address

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/miniconda/bin/jupyter-notebook", line 11, in <module>
    sys.exit(main())
  File "/miniconda/lib/python3.6/site-packages/jupyter_core/application.py", line 266, in launch_instance
    return super(JupyterApp, cls).launch_instance(argv=argv, **kwargs)
  File "/miniconda/lib/python3.6/site-packages/traitlets/config/application.py", line 657, in launch_instance
    app.initialize(argv)
  File "<decorator-gen-7>", line 2, in initialize
  File "/miniconda/lib/python3.6/site-packages/traitlets/config/application.py", line 87, in catch_config_error
    return method(app, *args, **kwargs)
  File "/miniconda/lib/python3.6/site-packages/notebook/notebookapp.py", line 1629, in initialize
    self.init_webapp()
  File "/miniconda/lib/python3.6/site-packages/notebook/notebookapp.py", line 1379, in init_webapp
    self.jinja_environment_options,
  File "/miniconda/lib/python3.6/site-packages/notebook/notebookapp.py", line 158, in __init__
    default_url, settings_overrides, jinja_env_options)
  File "/miniconda/lib/python3.6/site-packages/notebook/notebookapp.py", line 251, in init_settings
    allow_remote_access=jupyter_app.allow_remote_access,
  File "/miniconda/lib/python3.6/site-packages/traitlets/traitlets.py", line 556, in __get__
    return self.get(obj, cls)
  File "/miniconda/lib/python3.6/site-packages/traitlets/traitlets.py", line 535, in get
    value = self._validate(obj, dynamic_default())
  File "/miniconda/lib/python3.6/site-packages/notebook/notebookapp.py", line 872, in _default_allow_remote
    for info in socket.getaddrinfo(self.ip, self.port, 0, socket.SOCK_STREAM):
  File "/miniconda/lib/python3.6/socket.py", line 745, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -5] No address associated with hostname

Do you have any idea why is this happening? Is this an issue on my end or on yours?
Your gee-atmcor-S2 for single images is working perfectly fine for me, but I don't think I am capable of modifying it on my own to run it for a ImageCollection object.
I appreciate any help, thanks :)

hi, i'm having that problem also.
exactly the same, also no problem for the other repository.
i would love to use this tool!
thanks

I found the solution, when opening the jupyter notebook,
change the ip='*' por ip=<IP_ADDRESS>
i found that with either this one or the repo gee-atmcorr-s2, when the web is provided, instead of copy/pasting inside the web browser, open de Kitematic soft (i suppose that you should have it if you are running a docker container), in there there's going to be a list with containers, and the one active, y colour. In the right panel (web preview) you had an square with an arrow, hit there, and that would make the connection successfully with your browser, then you just need to copy/paste the token provided in the cmd, and the notebook works fine.


Discussion: Update list organization and sections for clarity

Problem
In the course of the discussion on PR https://github.com/shuaibiyy/awesome-terraform/pull/171, it came up that the current list sections and organizations may no longer be adequate given the size the list has now grown to (a good problem to have!). For example, the Tools section has a mix of open source and free tools as well as SaaS products with and without free tiers.
This can be confusing for an end-user looking for a specific type of tool/product, but it can also be confusing for contributors to the list to determine where a new list addition should be placed.
Potential Approach
I took a look at the existing list and had a few ideas for how to reorganize the sections, at least as a first pass to set the stage for further organization. Here's how we could organize the list:

Contents
Legend
Official HashiCorp Terraform Resources
Community
Resources

Books
Tutorials and Blog Posts

Beginner Guides
How-To
Cloud Providers

Azure
AWS
Google Cloud

Multi-Environment Configuration
Security & Compliance
Testing & Troubleshooting
Writing Custom Providers
Miscellaneous

Videos

Providers

HashiCorp Supported Providers
Vendor Supported Providers

Community Modules
Tools & Services

General Tools & Services
Terraform Enterprise
Boilerplates
Testing
Libraries
Private Module Registries
Private Provider Registry
Editor Plugins

License

@chenrui333 Opened this issue to continue our discussion and put up a draft PR as a proposal. I kept the majority of the tools in the "General Tools & Services" section for now, but I recommend breaking that up in follow-up PRs to avoid huge PR changes.

@chenrui333 just wanted to keep the conversation going. Like I said, this would be the first step to help organize the list, and then it'll (hopefully) be easier to improve organization after that.

close this for now due to inactivity, feel free to create a new one.


Added CORS as No

I checked the website by requesting from another site origin and It gives a CORS error.

@matheusfelipeog Oh Thanks I am now going to change it to yes.

@matheusfelipeog I changed it

Un subscribe
On Sun, Oct 10, 2021, 14:26 Matheus Felipe @.***> wrote:

Merged #2437 https://github.com/public-apis/public-apis/pull/2437 into
master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/public-apis/public-apis/pull/2437#event-5439083665,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV4FXTSQCTFK3V6Q57ZCM2LUGFIK5ANCNFSM5FWF2GRA
.


Error while validating the service descriptor exception after AddLockableNavigationManager

I get the following exception on the builder.Build() call.
Error while validating the service descriptor 'ServiceType: Mirality.Blazor.Routing.Internals.LockableNavigationManager Lifetime: Singleton ImplementationType: Mirality.Blazor.Routing.Internals.LockableNavigationManager': Cannot consume scoped service 'Microsoft.AspNetCore.Components.NavigationManager' from singleton 'Mirality.Blazor.Routing.Internals.LockableNavigationManager'.
I followed the basic setup from the readme in a Blazor Server app.
If I change the AddLockableNavigationManager extension method to use AddScoped it works.
services.AddScoped<LockableNavigationManager>();
services.AddScoped<ICustomNavigationManager>(sp => sp.GetRequiredService<LockableNavigationManager>());
services.AddScoped<ILockableNavigationManager>(sp => sp.GetRequiredService<LockableNavigationManager>());

This is actually a duplicate of #1, but I'll leave it open for now as this is a more concise description.


Helidon tuning guide

We are missing a Helidon tuning guide. Initially it can start with how to configure thread pool sizes and then go from there.

Basic tuning for Helidon MP

Use io.helidon.microprofile.bundles:helidon-microprofile-core dependency instead of full (helidon-microprofile) dependency.
Configure the size of the Netty worker thread pool
Configure the size of Helidon's server thread pool
Configure Netty's maxOrder setting if using Helidon 2.4.1 or earlier
Configure/tune Java heap size as per any Java application

1) Use io.helidon.microprofile.bundles:helidon-microprofile-core
Use helidon-microprofile-core dependency (and not the helidon-microprofile dependency) and add only what you use. For example:
        <dependency>
            <groupId>io.helidon.microprofile.bundles</groupId>
            <artifactId>helidon-microprofile-core</artifactId>
        </dependency>
        <dependency>
            <groupId>io.helidon.microprofile.metrics</groupId>
            <artifactId>helidon-microprofile-metrics</artifactId>
        </dependency>
        <dependency>
            <groupId>io.helidon.microprofile.health</groupId>
            <artifactId>helidon-microprofile-health</artifactId>
        </dependency>
        <dependency>
            <groupId>io.helidon.media</groupId>
            <artifactId>helidon-media-jsonp</artifactId>
        </dependency>

2) Configure Netty worker thread pool size
The Netty worker threadpool is what handles your incoming requests. To set it to something else you can set this property in microprofile-config.properties:
server.worker-count=4

3) Configure Helidon server pool size
The Helidon server threadpool takes requests from Netty and invokes your JAX-RS endpoints.  You can control lts configuration in microprofile-config.properties (Note that this threadpool does not exist in Helidon SE).
server.executor-service.core-pool-size: 2
server.executor-service.max-pool-size: 4

To verify settings increase the log level for Helidon's executor service by adding this to your logging.properties:
io.helidon.common.configurable.ThreadPool.level=FINE

4) Configure Netty's maxOrder (Helidon 2.4.1 or earlier)
In some situations Netty can aggressively allocate memory per request. This has been addressed in recent versions of Helidon and Netty, but if you are running an earlier version set this system property when you start your Helidon application:
-Dio.netty.allocator.maxOrder=6

You can try smaller numbers.


Crash : Failed to query our own IP from

report attached
coda_crash_report_2021-01-13_14-48-05.279476.tar.gz

Same for me..
coda_crash_report_2021-01-12_06-17-23.846208.tar.gz

Same for me..
coda_crash_report_2021-01-12_06-17-23.846208.tar.gz

btw, if node cannot query dns server somehow, crashing

btw, if node cannot query dns server somehow, crashing

Closing this issue as it is stale for a long time.


Link and bibtex to arxiv technical report on the image captioning module

Adding the link and bibtex to arxiv technical report on the image captioning module. The change affects only the documentation (README.md).
Pull Request Checklist

[x] Changes are complete (if posting work-in-progress code, prefix your pull request title with '[WIP]'
until you can check this box.
[ ] Unit tests pass (pytest)
[x] Were system tests modified? If so did you run these at least 5 times to account for the variation across runs?
[ ] System tests pass (pytest test/system)
[ ] Passed code style checking (./style-check.sh)
[x] You have considered writing a test
[x] Updated major/minor version in sockeye/__init__.py. Major version bump if this is a backwards incompatible change.
[x] Updated CHANGELOG.md

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I guess you are suggesting to close the pull request and open a new one in the website repository. Is that correct?

Ah sorry, I just noticed that the website is in this repo too :)


Remove trailing slash from YOURLS_SITE

We should be more forgiving with noobs who cannot RTFM (or maybe don't understand English) and define their YOURLS_SITE with a trailing slash (eg http://sho.rt/ instead of http://sho.rt as advised)
This is a very minor bug but it produces discrepancies in links, href and src's. For instance javascript and stylesheets are included correctly (


Add some extra debug logging during artifact uploads

The current non-debug output during artifact uploads looks like this:
--- Uploading artifacts
$ buildkite-agent artifact upload log/*.log
2020-06-23 17:46:52 INFO   Found 1 files that match "log/*.log"
2020-06-23 17:46:52 INFO   Creating (0-1)/1 artifacts
2020-06-23 17:46:52 INFO   Uploading artifact 4941dcd3-665d-445b-86b0-8d552b86dd53 log/foo.log (2275 bytes)

Without debug logging, there's no output once the uploads start, and if the upload process hangs for any reason it can be hard to see (based on log output only) where things have become stuck.
This adds a single Info-level log line when all artifacts have successfully been uploaded, so it's obvious when that process has
completed. Any stalled builds after that point will be easier to attribute to a non-artifact uploading issue.
I also added a couple of extra Debug-level logs at points where the agent is waiting for asynchronous network IO to complete. Just in case we want to see where things are at for customers in the future.

This seems uncontroversial and low-risk so I'm going to merge in the interests of easier debugging for customers, but I can revise if anyone has feedback!
/cc @sj26 @pda @lox


Docker image version with python3.8 (Ubuntu20.04)

Hello,
I'm working on a project where I want to take advantage of new feature of Python3.8, the multiprocessing shared memory library. Could you please provide a docker image based on Ubuntu 20.04, where default python is 3.8? I tried to do it myself, but I encountered some issues installing protobuf and clif after that.
Thank you!

You could still use Ubuntu 18.04 and install python 3.8 on it via an external repository (check here for an example).
For protobuf check this issue which says that python 3.8 is supported on version 3.11.2.
For CLIF, you should be careful if you have 2 python versions installed. Check the building instructions and you will probably be able to resolve the issue by changing in params in cmake.

We have now released whl files for Python 3.7, 3.8 and 3.9. That should make the installation much easier and you could install those in your own docker project as well!
Unfortunately the official pykaldi docker builds are unmaintained atm. If you'd like to maintain these, please get in touch.


flaky test: TestCache_Account_Add_NonceTooSmall

Description
Failed during unrelated change: https://github.com/spacemeshos/go-spacemesh/actions/runs/4231221021/jobs/7349474661
=== FAIL: txs TestCache_Account_Add_NonceTooSmall (0.00s)
    cache_test.go:655:
        	Error Trace:	/home/runner/work/go-spacemesh/go-spacemesh/txs/cache_test.go:655
        	Error:      	Target error should be in err chain:
        	            	expected: "bad nonce"
        	            	in chain:
        	Test:       	TestCache_Account_Add_NonceTooSmall

The test is still flaky: https://github.com/spacemeshos/go-spacemesh/actions/runs/5517581731/jobs/10060394729


lockfile not in sync with package.json

npm ci is needed for consistent deployments
Currently it seems the lockfile was not udpated for a while.
Running npm ci leads to cipm can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync. Please update your lock file with npm installbefore continuing.
https://github.com/hyva-themes/magento2-hyva-checkout/tree/develop/src/reactapp
So please don't forget to update the lockfile =)

@DanielRuf Thanks for reporting the issue. There is #130 with which all packages will be updated to the latest versions and also the package-lock.json too.

Awesome, thank you very much =)


Upgrade idb-storage dependencies

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.336%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
52464

Relevant Lines:
53352

💛  - Coveralls


Extensible offset storage, adds possibility to store pipeline offsets…

… in external storage, not only in file. As for now supports only storing offsets in files and in zookeeper. Offset storage is configured for whole sdc instance, not per-pipeline.

Guys, any ideas about this PR?

Thank you for the contribution @vkrot . Sadly, we currently do not have plans to make the offset storage pluggable as that would interfere with our future plans for the data collector.

Making the storage module pluggable doesn't really fit our plans with data collector going forward, so I will sadly have to reject the change. I will go ahead and resolve the pull request for now.


[3.7] fix gauge assignment operators

Scope & Purpose
(Please describe the changes in this PR for reviewers - mandatory)

[X] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[x] :book: CHANGELOG entry made

Backports:

[X] No backports required
[ ] Backports required for: (Please specify versions and link PRs)

Related Information
(Please reference tickets / specification / other PRs etc)

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket number:
[ ] Design document:

Testing & Verification
(Please pick either of the following options)

[X] This change is a trivial rework / code cleanup without any test coverage.
[X] The behavior in this PR was manually tested
[ ] This change is already covered by existing tests, such as (please describe tests).
[X] This PR adds tests that were used to verify all changes:

[X] Added new C++ Unit tests
[ ] Added new integration tests (e.g. in shell_server / shell_server_aql)
[ ] Added new resilience tests (only if the feature is impacted by failovers)

[ ] There are tests in an external testing repository:
[ ] I ensured this code runs with ASan / TSan or other static verification tools

Link to Jenkins PR run:
https://jenkins01.arangodb.biz/view/PR/job/arangodb-matrix-pr/12800/
Documentation

All new features should be accompanied by corresponding documentation.
Bugs and features should furthermore be documented in the CHANGELOG so that
support, end users, and other developers have a concise overview.

[ ] Added entry to Release Notes
[ ] Added a new section in the Manual
[ ] Added a new section in the HTTP API
[ ] Added Swagger examples for the HTTP API
[ ] Updated license information in LICENSES-OTHER-COMPONENTS.md for 3rd party libraries

External contributors / CLA Note
Please note that for legal reasons we require you to sign the Contributor Agreement
before we can accept your pull requests.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/12904/

http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr-windows/6474/
http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr-mac-onlycatch/10249/

http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr-windows/6477/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/12911/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/12918/

Tests blue.


[WIP] H-07: Minimum delegate time not set

The core system has setMarketMinDelegateTime() function which has to be called by integrating markets to set their minimum waiting time for the LPs to withdraw their collateral after they have delegated it.
The PerpsMarket doesn't call this function which lets LPs delegate and undelegate collateral in the same block. This can lead to some unexpected behaviors such as interest rate manipulation or risk-free yield opportunity by sandwiching a settlement to gain from fees by depositing a large amount of collateral and the withdrawing it.

Note that this PR passes CI but breaks all the local anvil tests due to time traveling issues


Fix 'your' error and add text in end.

 Change, in the submission form, the text above the box "DOI of your preprint..."


Fix - feature flags - Master

Codecov Report
Base: 32.54% // Head: 32.51% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (3ab0bc4) compared to base (ddef4dd).
Patch coverage: 25.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #180      +/-   ##
==========================================
- Coverage   32.54%   32.51%   -0.03%
==========================================
  Files          91       91
  Lines        3263     3266       +3
  Branches      550      552       +2
==========================================
  Hits         1062     1062
- Misses       2201     2204       +3

Impacted Files
Coverage Δ

trade_remedies_caseworker/core/views.py
46.87% <25.00%> (-1.52%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update unmanaged platform guidance

This updates the CI onboarding guidance with the following changes:

Specify current unmanaged platform used by other dotnet repos, i.e. runtime
Clarify the location for platform-related changes

I thought we want to change the location of this file to make it easier to find.

We decided to tackle doc organization and discoverability as a separate work. This PR addresses incorrect/obsolete guidance.


Non-existent 0.5.3 release

VS Code offers 0.5.3 release to me but I can't find this release in this repo.
This seems fishy to me so you might want to confirm if this release is legit or not.

There was no commit on the repo updating the version number to 0.5.3, this seemed fishy to me, that you might didn't release this version but someone else a malicious one.
The commit is there now and I'm seeing the same for the 0.5.4 release. Looks like you don't push after a release so the commit is missing on GitHub. All good now that I know that this is your usual routine.


Update timeout for .github/workflows/autogenerated-warning.yml

This is an automated pull request to update default timeouts for all jobs in .github/workflows/autogenerated-warning.yml that do not currently contain a default timeout property.
Please review these changes closely to ensure that all jobs have sufficient time to run. If a job needs more time, please add a timeout property to that job to override the organization default.
Additionally, please be attentive to unintended changes to the workflow YAML file, as the parser is not perfect.
Lastly, note that it is up to the reviewer to merge this pull request and delete the PR branch once they are satisfied with the changes.

Replaced by an explicit timeout setting: https://github.com/Kong/docs.konghq.com/pull/6912


update custom styles for Bar chart native

@Ahmad-Elsayed
@ConnorLand is off sick today. Should this be published immediately or can it wait?

 Should this be published immediately or can it wait?

yes it can wait

Confirmed over Slack


Can we use common artifact for Arbor?

Kind (Required)

Improvement

Overview (Required)

We use JVM artifact. But Probably In recent Kotlin MPPs, you can only refer to common artifact.
Like const val abor = "com.ToxicBakery.logging:arbor:1.35.72"

Please delete "com.ToxicBakery.logging:arbor-android" and "com.ToxicBakery.logging:arbor-jvm" dependency and use "com.ToxicBakery.logging:arbor" 🙏

Links

https://github.com/DroidKaigi/conference-app-2021/pull/221/files#diff-0f9d3ee70139178b0d6cca6bf4a909174a00934c74f3dc32c6afca2ed335d647R104

🙋♂

Thank you! Assigned 🚀

I added this library.
com.ToxicBakery.logging:arbor:1.35.72
But there was a problem.
Could not find com.ToxicBakery.logging:arbor:1.35.72.
Required by:
    project :data:api

I'm sorry. Do you know how to solve this problem?

Sorry, can you try com.ToxicBakery.logging:common? 🙇
https://search.maven.org/search?q=g:com.ToxicBakery.logging

Thank you. I will try.


Add camera_functioning properties in read_wi()

It seems like I missed to import camera_functioning properties from Wildlife Insight (see https://github.com/inbo/camtraptor/issues/129).
This properties can provide useful information in the filtering of the data (e.g. before uploading on GBIF in my case)

@peterdesmet , What do you think of adding this field to deployement$comments (https://tdwg.github.io/camtrap-dp/data/#deployments.comments) https://github.com/inbo/camtraptor/blob/HEAD/R/read_wi.R#L297

Issue solved via #180


incorrect view for more than 2 buttons

hi,
my problem is that i want to have both leftUtilityButtons and rightUtilityButtons in a UITableViewCell and i have achieved this matter. but there are 2 problems.
1- if i want to use a language that its direction is RTL (eg. Arabic) i wont be able to use accessory anymore because the xib flips automatically and accessory fall outside of view. see the pictures below :

2- so the first problem made me get rid of accessory type for the cells but then i ran into another problem this time without the accessory. the problem appears when i define 3 button for the leftUtilityButtons! it works fine when 2 buttons are defined but when it is 3, it looks like this :

can anyone help me resolve these issues ?
thanks

For me too same issue can any one help me plz


Guide: Securely store Micronaut Application secrets in Azure Key Vault

The screenshots' application name is DemoApp. Should it be micronautguide to match the value in bootstrap.yml?

Changed ib https://github.com/micronaut-projects/micronaut-guides/pull/1099/commits/3c3bafe47a4a8c1d66033dea8d7f5e0f53a621c0.

The screenshots' application name is DemoApp. Should it be micronautguide to match the value in bootstrap.yml?

Changed in https://github.com/micronaut-projects/micronaut-guides/pull/1099/commits/3c3bafe47a4a8c1d66033dea8d7f5e0f53a621c0

something went wrong with the merge?


How do i confirm that sw-toolbox urlPattern  handler strategy is working as designed?

I'm using sw-toolbox with sw-precache at runtime caching option. But when i visit the url and inspect the network in devtools it shows cached from service workers not from network first (as i desire). Maybe I just don't understand how these two libs work together.  My current implementation uses sw-precache to cache my entire webapp(very small webapp) and using runtime caching option i used a url pattern with handler of network first. My thought is that even-though the entire app is cached , when i go the specific url pattern the data used at the page will be fetched from network not cache. Is this incorrect?

This is expected.
All requests that passing through to the Service Sorkers will appears (from service workers)


download media

Issue that this pull request solves
Closes: #84
Proposed changes
This PR will let users download media from one contact
Types of changes
Put an x in the boxes that apply

[ ] Bugfix (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)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Screenshots
Please attach the screenshots of the changes made in case of change in user interface

Other
How to use: python -m wplay -wd AnyName or python -m wplay --download_media AnyName
Input number of media to be downloaded.

Please review @priyansh19 @xandao6 @rpotter12

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Complexity increasing per file
==============================
- wplay/download_media.py  19
- wplay/__main__.py  1

Clones added
============
- wplay/download_media.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Complexity increasing per file
==============================
- wplay/download_media.py  19
- wplay/__main__.py  1

Clones added
============
- wplay/download_media.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Complexity increasing per file
==============================
- wplay/download_media.py  19
- wplay/__main__.py  1

Clones added
============
- wplay/download_media.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- wplay/download_media.py  19
- wplay/__main__.py  1

Clones added
============
- wplay/download_media.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- wplay/download_media.py  19
- wplay/__main__.py  1

Clones added
============
- wplay/download_media.py  1

See the complete overview on Codacy


Prepare release 0.5.1

This PR should not be merged.
It contains only a single bug fix (b09b1fe54d901e3e6469bd4d190f7bc3191e1af1) and adds an export: (f14524f3415018c643778e5691b6e6fcd4c1f1fa)
When CI is green, this PR is pushed into the main repo, tagged as a release and then the ChangeLog and hie-bios.cabal changes will be cherry-picked on master.
This way we dont have any breaking changes but include the important bug-fix.
cc @Avi-D-coder

https://github.com/mpickering/hie-bios/tree/0.5.1


Print only Deleting Objects. Include other names in debug log

Fixes #4

@mkumatag Can you please take a look at this?


why set TF_BUILD_BAZEL_CLEAN=1 in ci_parameterized_build.sh

Hi,
I found TF_BUILD_BAZEL_CLEAN is always set to 1 in ci_parameterized_build.sh
https://github.com/tensorflow/tensorflow/blob/a6a265b61a9ad9510f45cf4c9032778bf2e042b9/tensorflow/tools/ci_build/ci_parameterized_build.sh#L99
This was introduced about 11 months ago.
The comment says this is temporary and will be removed later.
So is that still required?
Thanks.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Yeah, it's not needed anymore. Sent a change internally to delete it.

We are not using ci_parameterized_build.sh internally anymore.
So that script is rotting, with such pieces of code accumulating in it.

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

This is now fixed.


[CDS-62] Add module for syncing Shelley block.

https://github.com/input-output-hk/cardano-db-sync/issues/62

Closing this in favor of having a single branch containing the changes for Shelley.
Moving to https://github.com/input-output-hk/cardano-db-sync/pull/70

The base branch marked here - https://github.com/input-output-hk/cardano-db-sync/pull/72


Cannot use in MacOS arm64

I installed, imported and used like you specified in document. But when I run I got this error:
$> node index.js
node:internal/modules/cjs/loader:1048
  const err = new Error(message);
              ^

Error: Cannot find module '/Users/thanh_da/Documents/im_nodejs/node_modules/magickwand.js/lib/binding/darwin-arm64/magickwand.node'
Require stack:
- /Users/thanh_da/Documents/im_nodejs/node_modules/magickwand.js/lib/index.cjs
    at Module._resolveFilename (node:internal/modules/cjs/loader:1048:15)
    at Module._load (node:internal/modules/cjs/loader:901:27)
    at Module.require (node:internal/modules/cjs/loader:1115:19)
    at require (node:internal/modules/helpers:130:18)
    at Object.<anonymous> (/Users/thanh_da/Documents/im_nodejs/node_modules/magickwand.js/lib/index.cjs:21:13)
    at Module._compile (node:internal/modules/cjs/loader:1241:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1295:10)
    at Module.load (node:internal/modules/cjs/loader:1091:32)
    at Module._load (node:internal/modules/cjs/loader:938:12)
    at cjsLoader (node:internal/modules/esm/translators:284:17) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/Users/thanh_da/Documents/im_nodejs/node_modules/magickwand.js/lib/index.cjs'
  ]
}

Node.js v20.9.0

This is the only code that use the lib
import { Magick } from "magickwand.js";

const resize = async (imageArrayBuffer, specification) => {
	const wizard = new Magick.Blob(imageArrayBuffer);
	console.log("Before transfrom", wizard.length);
	const im = new Magick.Image();
	await im.readAsync(wizard);
	return await im.resize(specification).then(() => wizard.data);
};

Can someone please explain to me, is this a library error or did I do something wrong?
This is my package.json
{
  "name": "lambda-image",
  "version": "1.0.0",
  "private": true,
  "type": "module",
  "author": "",
  "license": "ISC",
  "dependencies": {
    "@aws-sdk/client-s3": "^3.454.0",
    "axios": "^1.6.2",
    "file-type": "^18.7.0",
    "magickwand.js": "^1.0.0-beta.6"
  }
}

As you may have seen from my profile, I am currently unemployed because of an extortion involving the French judicial system. Some of the people involved in the extortion are trying to make me shut up and they are throwing a promise for a mac arm64 into the mix. Does your posting of this issue have anything to do with this extortion?

As you may have seen from my profile, I am currently unemployed because of an extortion involving the French judicial system. Some of the people involved in the extortion are trying to make me shut up and they are throwing a promise for a mac arm64 into the mix. Does your posting of this issue have anything to do with this extortion?

No, please. I'm just a normal user who finds your project interesting. I'm not from your country and know nothing about your situation. How terrible it is for a person to be caught in that situation. I considered donating to you, but the minimum amount is 2/3 of my monthly income, so I can't now donate to you.

@mmomtchev I know that with the current situation, it is difficult for you to continue the great unfinished work. However, if at any moment you suddenly come up with an idea for the cause of the error I mentioned above, I would appreciate it if you could share it here.

Hello @mmomtchev
GitHub added support for free Apple Silicon runners for GitHub Actions https://github.com/actions/runner-images/issues/9254
Could you consider providing prebuilt darwin-arm64 packages now?

magickwand.js@next supports macOS arm64


Switch to Skeletor Makefile

/usr/bin/ld: anonymous version tag cannot be combined with other version tags


Updated Scikit converter code to fix the errors.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/convert-model-winmltools.md
:white_check_mark:Succeeded
View

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


Restart error in .Net

Hello
I get an error when trying to run the restart command.
I think this error occurs when outstation does not support this command (I just guess) and , although it does not happen in C ++.
https://github.com/dnp3/opendnp3/blob/44c06cb148d40ed150e539895b3fa951376e5f74/dotnet/CLRAdapter/src/CallbackAdapters.cpp#L50
I think this error is related to time converting.

What error do you get exactly? Can you provide logs showcasing the issue?
I quickly tried it on my machine and couldn't reproduce, but I'm not sure what error you are experiencing, nor what the repro steps are.

I using Axon Test 4.3 to simulate outstation for both .Net and C++ master example.
.Net Master Log:

C++ Master Log:

Notes:

The execution of the .Net example stops by an exception on the specified line before.
Exception: timespan overflowed because the duration is too long
C++ example runs under the same conditions without any fatal error.
Axon Test use opendnp3 as DNP library too.

Thanks for the detailed response.
I found the issue and written a bugfix. Have a look at the bug/issue-350 branch and PR #351. It only affects the 2.x releases.
Can you give it a try and tell me if it fixes your issue?
Thanks!

It fixed.
Thanks!

Great, the fix was released in 2.3.3.


Update to CNCF Network TAG

See https://github.com/cncf/toc/pull/654
Signed-off-by: Chris Aniszczyk<EMAIL_ADDRESS>

FYI @leecalcote as a heads up

Thank you, sir. 👍


"pipenv lock --requirements" leaks output from git, making the result invalid, only the first time

Issue description
One of my dependencies is in the format "lib" = {ref = "branch-name", git = "https://url/to/repo.git"}.
I run pipenv lock --requirements > requirements.txt for the first time inside a docker container.
Expected result
I get a valid requirements.txt
Actual result
See below
Steps to replicate
docker run -it --rm python:3.6 bash

Inside container
pip install pipenv==2018.11.26

echo '
[[source]]
url = "https://pypi.org/simple"
verify_ssl = true
name = "pypi"

[dev-packages]

[packages]
"numpy" = {ref = "maintenance/1.15.x", git = "https://github.com/numpy/numpy.git"}
' > Pipfile

pipenv lock --requirements > requirements.txt

Resultant requirements.txt (yes really):
Switched to a new branch 'maintenance/1.15.x'
Branch maintenance/1.15.x set up to track remote branch maintenance/1.15.x from origin.
-i https://pypi.org/simple
git+https://github.com/numpy/numpy.git@3430d78c01a3b9a19adad75f1acb5ae18286da73#egg=numpy

And of course:
root@3f46e6fed2ba:/# pip install -r requirements.txt
Invalid requirement: 'Switched to a new branch 'maintenance/1.15.x''
It looks like a path. File 'Switched to a new branch 'maintenance/1.15.x'' does not exist.

Workaround
I run pipenv lock --requirements > /dev/null 2>&1 once before running the actual command.

Please run $ pipenv --support, and paste the results here. Don't put backticks (`) around it! The output already contains Markdown formatting.
$ pipenv --support
Pipenv version: '2018.11.26'
Pipenv location: '/usr/local/lib/python3.6/site-packages/pipenv'
Python location: '/usr/local/bin/python'
Python installations found:

3.6.7: /usr/local/bin/python
3.6.7: /usr/local/bin/python3.6m
3.5.3: /usr/bin/python3
3.5.3: /usr/bin/python3.5m
2.7.13: /usr/bin/python2.7

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.6.7',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '4.15.0-39-generic',
 'platform_system': 'Linux',
 'platform_version': '#42~16.04.1-Ubuntu SMP Wed Oct 24 17:09:54 UTC 2018',
 'python_full_version': '3.6.7',
 'python_version': '3.6',
 'sys_platform': 'linux'}

System environment variables:

LANG
HOSTNAME
GPG_KEY
PWD
HOME
TERM
PYTHON_VERSION
SHLVL
PATH
PYTHON_PIP_VERSION
_
PIP_DISABLE_PIP_VERSION_CHECK
PYTHONDONTWRITEBYTECODE
PIP_SHIMS_BASE_MODULE
PIP_PYTHON_PATH
PYTHONFINDER_IGNORE_UNSUPPORTED

Pipenv–specific environment variables:
Debug–specific environment variables:

PATH: /usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG: C.UTF-8
PWD: /

Contents of Pipfile ('/Pipfile'):

[[source]]
url = "https://pypi.org/simple"
verify_ssl = true
name = "pypi"

[dev-packages]

[packages]
"numpy" = {ref = "maintenance/1.15.x", git = "https://github.com/numpy/numpy.git"}

Contents of Pipfile.lock ('/Pipfile.lock'):
{
    "_meta": {
        "hash": {
            "sha256": "e8d89524a159e6d44fa849ffd5e1bf02b13a73b773013996d77d5405776369eb"
        },
        "pipfile-spec": 6,
        "requires": {},
        "sources": [
            {
                "name": "pypi",
                "url": "https://pypi.org/simple",
                "verify_ssl": true
            }
        ]
    },
    "default": {
        "numpy": {
            "git": "https://github.com/numpy/numpy.git",
            "ref": "3430d78c01a3b9a19adad75f1acb5ae18286da73"
        }
    },
    "develop": {}
}

Wow that is incredibly frustrating. That is pip logging to standard output.

They log literally everything else to standard error....

I see that the update tries to output the git commit hash as the checkout path instead of the branch name, but I haven't dig deep enough to figure out which command is leaking.

It’s very indirect. It’s going to be in a super recent change in pipenv/utils.py basically via venv_resolve_deps, which now calls get_vcs_deps directly. This eventually calls get_locked_requirement which calls pipenv.vendor.requirementslib.models.requirements.Requirement() to get access to a contextmanager with access to a temporary checkout of the repo. That ultimately is done in requirementslib.models.vcs but is just delegated to pip’s native VCS handled. I’m pretty confident we moved this to make sure resolution happens in the venv; I guess that’s why we are capturing this...

hmmm the specifics are super helpful here, because the output only happens when you clone a branch directly in pip.  I have a fix for this but it just basically monkeypatches the defaults to suppress stdout in pip's command runner during vcs operations :|

I can confirm this in 2018.11.26.
[packages]
foo = {git =<EMAIL_ADDRESS>editable = true, ref = "dev"}

root@5a495b0ebc49:/server# pipenv lock --requirements > requirements.txt
Creating a virtualenv for this project…
Pipfile: /server/Pipfile
Using /usr/local/bin/python (3.7.6) to create virtualenv…
⠼ Creating virtual environment...Already using interpreter /usr/local/bin/python
Using base prefix '/usr/local'
New python executable in /root/.local/share/virtualenvs/server-CV4Qw303/bin/python
Installing setuptools, pip, wheel...
done.

✔ Successfully created virtual environment!
Virtualenv location: /root/.local/share/virtualenvs/server-CV4Qw303
Pipfile.lock not found, creating…
Locking [dev-packages] dependencies…
✔ Success!
Locking [packages] dependencies…  # This hangs for several minutes until I terminate
✘
root@5a495b0ebc49:/server# ls
Pipfile  dist  main.py  requirements.txt  server  tmp  wsgi.py
root@5a495b0ebc49:/server# cat requirements.txt
Switched to a new branch 'dev'
Branch 'dev' set up to track remote branch 'dev' from 'origin'.


Should address Issue #65 by compressing annotation in setup_query()

Should address Issue #65

Coverage increased (+0.3%) to 99.02% when pulling 91f3a491858d3cde31893e25edde9363c61b0b76 on denhartog:master into b3b45a0805e861fda22db1a3a1a011c7140fff69 on california-civic-data-coalition:master.

Coverage increased (+0.3%) to 99.02% when pulling 91f3a491858d3cde31893e25edde9363c61b0b76 on denhartog:master into b3b45a0805e861fda22db1a3a1a011c7140fff69 on california-civic-data-coalition:master.

Thanks, though I still don't understand why we need the clause. @gordonje, if you remember let me know so I can shut up about it.

It permits exporting annotations. See "Person.objects.annotate(name_count=Count('name')).to_csv('/path/to/my/export.csv')" in http://django-postgres-copy.californiacivicdata.org/en/latest/#how-to-export-data


Support for navigateTo function to open in new window

Description
navigateTo new window
Fixes #2644
Type of change

New feature (non-breaking change which adds functionality)
This change requires a documentation update

Checklist:

[ ] My code follows the style guidelines of this project
[ ] 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 changes generate no new warnings
[ ] 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

Docs PR appsmithorg/appsmith-docs#82

Docs PR appsmithorg/appsmith-docs#82