blackbox_exporter is reaching the nofile limit

What happened?
Lots of too many open files errors if too many blackbox checks are added to Prometheus
Mar 12 03:03:07 prom1 blackbox_exporter[37552]: 2020/03/12 03:03:07 http: Accept error: accept tcp <IP_ADDRESS>:9115: accept4: too many open files; retrying in 20ms

Did you expect to see some different?
Checks are done without error
How to reproduce it (as minimally and precisely as possible):
Add a few thousand blackbox checks.
Environment
The server is running Debian 10, the default nofile soft limit set by systemd seems to be 1024, hard limit is 524288.
Adding LimitNOFILE=65000 like in the Prometheus Ansible role to the blackbox_exporter systemd unit solves the problem permanently.

Seems like we are not setting LimitNOFILE in systemd service file, so it should be inherited from environment. In such case I don't think this is this role problem and it can be adjusted by using systemd "drop-in directory" (more in https://www.freedesktop.org/software/systemd/man/systemd.unit.html)
@SuperQ I see gitlab had some incident similar to this issue. Can you advise on some sensible value for LimitNOFILE?

This role has been deprecated in favor of a the prometheus-community/ansible collection.


install and uninstal Poetry on Windows

Hi, thanks for making this tool. I'm not very familiar with developing processes and still learning about it.
I can't tell if Poetry supports Windows from the documentation but I tried it anyway and it worked!
Great news. Maybe make it explicit in the docs? I think Poetry can easily become more popular if it can attract a tiny fraction of Windows users. (For example, I'm a teacher, 90% of my students use Windows and 100% of my school) I use both though.
I installed Poetry this way and it was installed to the Python directory in my home directory (Windows).
curl -sSL https://raw.githubusercontent.com/sdispater/poetry/master/get-poetry.py | python
What is the recommended way to uninstall it?
I find Poetry interesting and definitely will explore it further.
Thanks.

Thanks Victor-Savu. It worked.
Out of curiosity I tried pip install poetry again and got massive influx of packages (shown when I did pip list). I used pip-autoremove to save the day. One thing I like about poetry is that it can remove an installed package and its dependencies (pip/conda can't do it) but I don't think I could use poetry to remove itself.
Then I wondered how it hid these before and looked up "vendorizing" and found the _vendor folder in site-packages. (I thought it was in the home directory). Nice!
If PRs are welcome here, I can add a few lines here and there to README.md.

You're welcome, @jeremy886!
My guess would be that you could open a PR for the README and/or for the docs, but @sdispater is the one putting in most of the hard work, so he is the best person to ask about contributing to poetry :)

The fact that Poetry supports Windows is not specified in the README but is in the documentation: https://poetry.eustace.io/docs/#system-requirements

merici. :)

Closing this since the installer can now also uninstall Poetry.


MissingPluginException(No implementation found for method canMockLocation on channel safe_device)

As the title says, can't find canMockLocation channel.
Running on real developer enabled Pixel 4a device. Running Android 11.
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.2.3, on macOS 11.5.2 20G95 darwin-arm, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] VS Code (version 1.60.0)
[✓] Connected device (3 available)
Exception from Flutter:
I/flutter (22322): MissingPluginException(No implementation found for method canMockLocation on channel safe_device)
Tried deleting app, reinstalling, and restarting vscode. None of restarts or reinstalls worked.
Code I'm using:
static Future isDeviceCompromised() async {
bool isCompromised = false;
try {
isCompromised = await SafeDevice.isSafeDevice;
} catch (error) {
print(error);
rethrow;
}
print('isCompromised(): $isCompromised');
return isCompromised;

}
Please let me know if you need more info.

safe_device 1.0.8

not work in 1.0.8

same in 1.0.8

same in 1.1.1

same in 1.1.1
I don't know why @ufukhawk you close the issue before confirming that its solved already ?!


Help page on Mover.io for Office365

At the end, it seems like the Connect to Source step is repeated twice, while there is no step for Connect to Destination. Also, the whole thing ends abruptly at that point. I was really hoping to see what I needed (or was able) to do after the connections were made - selecting users and data, for instance.

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

ID: da7ce00b-f843-f8d1-c949-bf4a15a53fb6
Version Independent ID: abd7e5ff-1cee-2c1f-71ef-7b9c27b023b2
Content: Authorizing the Office 365 Connector - Migrate to SharePoint and OneDrive
Content Source: migration/mover-o365.md
Service: sharepoint-online
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

@mozzis Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Add AWS Amplify Gen 2 backend messaging to README.md

Description of changes
This PR makes changes to the README of the CLI repository to add messaging on the new Amplify Gen 2 backend building experience for customers
Issue #, if available
N/A
Description of how you validated changes
Verified the README on local environment
Checklist

[ x ] Relevant documentation is changed or added (and PR referenced)

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

Closing in favor of - https://github.com/aws-amplify/amplify-cli/pull/13422 that has the updated docs url


support http proxy settings

sometimes, direct internet accessing is not possible, toad will be deployed with proper proxy settings.

I'm working on adding traefik right now, hopefully that will simplify this request as well.

So based on what you're trying to do, I was wrong on traefik helping you at all. Per our conversation last night, I think you might want to open a PR with the changes you made to tell the jenkins-cli.jar pull to  skip using the HTTP proxy.


Add PPTP to ignore for IPfw

https://github.com/opnsense/core/issues/1907

@iMiMx sorry, I was reading through my email from oldest to newest, missed your pull. thanks anyway

l2tp != pptp

sorry, now I missed the second commit :D

Was scratching my head a little ;)

To be fair, the first commit message said pptp but fixed l2tp so that’s where I missed the second one that actually fixed it. :D
Thank you for the report!

On 31. Oct 2017, at 12:54, iMx<EMAIL_ADDRESS>wrote:
Was scratching my head a little ;)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


Documentation of the new raw key feature

IMHO the new raw key feature deserves a slightly more elaborate documentation regarding its behaviour in certain "special" cases:

If the length of the raw key is not exactly 32, it is handled as a normal key (including the "raw:" prefix).
If the length of the raw key plus salt is not exactly 32+16 (48), it is handled as a normal key (including the "raw:" prefix).
If the length of the hex encoded key is not exactly 64, it is handled as a normal key (including the "raw:" prefix).
If the length of the hex encoded key plus salt is not exactly 64+32 (96), it is handled as a normal key (including the "raw:" prefix).
If the hex encoding of key and/or salt is erroneous, the key is handled as a normal key (including the "raw:" prefix).
Specifying the salt together with the key makes only sense when creating a new database. For an existing database the salt stored in the database header is used and the specified salt is silently ignored, even if it is different from the stored salt.

I'm fine with the behaviour as implemented. However it should be documented, so that users are aware of it.

Addressed in commit 4c24a3d, hopefully adequately.
One comment though: specifying the salt makes also sense when re-encrypting (rekeying) the database.


BLE GATT with multiple identical uuid

Hi,
I'm testing BLE GATT, and I'm facing a problem.
Into a service one can have more than one characteristic with the SAME uuid (for instance two Analog_Input). It's accepted and working fine with the ESP32 stack.
But the event ESP_GATTS_ADD_CHAR_EVT and the data within the structure provided -esp_ble_gatts_cb_param_t- is not suffisant to distinguish between the first, then 2nd, ... differents characteristics with the same Id. So to obtain the attr_handle the event data are not enought.
Of course since the attr_handle given by the ESP BLE stack is always the same for a given configuration ... some 'working but not good' code can be used to go ahead.
I'm right ?

As a general rule, You should not use two characteristics with the same UUID in the same service in BLE. It can and will cause lots of issues, like the one mentioned above both internally in the esp32 and with connected central applications.
If you really need to use the same characteristic UUID, then perhaps use a difference service to house the characteristics with the same UUID.

@fchaxel esp_ble_gatts_create_attr_tab is preferred to create gatt service. it will return an array of handles, the same order with the table of attributes you created. identical UUID chars will have different handles, you can access with handles then.
if you use old add char API, the event won't give you enough info. need to add extra flag yourself.

Hi,
No, multiple characteristic UUID are acceptable. In fact why a service could'nt be used to gives access to 3 separeted LED, or 10 buttons for instance ? Of course we can put all into an unique charac., but if it's a standard UUID (for instance 0x2A59 for one and only one Analog Output) it's not possible without loosing genericity.
A service may have multiple characteristic definitions with the same Characteristic UUID (Bluetooth 4, Core specification, page 537)
https://www.bluetooth.org/docman/handlers/downloaddoc.ashx?doc_id=229737
I will try esp_ble_gatts_create_attr_tab
Thank's to all.

If you have a look at https://www.bluetooth.com/specifications/gatt/viewer?attributeXmlFile=org.bluetooth.service.environmental_sensing.xml the use of the same UUID is documented.
For environmental you have to add a ES_MEASUREMENT description when using the same UUID - but doing this is a common thing (search for if_multiple_instances_of_same_characteristic_optional_otherwise).
Imagine a weather station - you have a temperature from the outdoor sensor and on from indoor (same for humidity).


Enemies on this map default to walking upwards and get trapped in spawn on West side

hmm. Probably related to #176.
I'll take a look.

I think this is resolved. Pretty sure.
(But the AI have other issues. They still conga line along edges.)


Hybrid term/type tests

Clear and concise description of the problem
Currently type tests require being in a different module than term ("concrete" seems to the the term used by the docs) tests.
This creates a separation of function rathe than concern.
That is, there are times when a test case concerns assertions for both the terms and types.
Having to keep these in two separate modules is not ideal, since it creates distance between these concerns.
Suggested solution
I'm really not sure what design and/or implementation considerations would be at play off the top of my head. I think it is desirable to have test runner terminal/ui/etc. output "merge" the failures from a test case into a unified view, but I think its not simply a matter of taking the current outputs and "concating" them... Some thinking will be required probably how best to present them, grouping vs interleaved, etc.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Having to keep these in two separate modules is not ideal, since it creates distance between these concerns.

I might be missing the point of your suggestion as a feature request of Vitest typecheck mode, but I think it's perfectly reasonable to have expect and expectTypeOf for a concrete "term" to co-exist in the normal test file and let typescript spot type errors as usual without Vitest typecheck mode.
I was looking at TRPC code base the other day and I found such example:
https://github.com/trpc/trpc/blob/677dbe7edd470b12fa2255db7ae52fb81735887f/packages/tests/server/concat.test.ts#L76-L84
test('decorate independently', async () => {
  const result = await ctx.proxy.getContext.mutate();

  expect(result).toEqual({
    user: mockUser,
    foo: 'bar',
  });

  expectTypeOf(result).toEqualTypeOf<{
    user: User | null;
    foo: 'bar';
  }>();
});

I'm not sure this approach would fit your use case, but I thought I'd share since I found this pattern has nice ergonomics and I just started to employ this on my own project.

@hi-ogawa thanks, I have done similar in the past but what this issue is asking is to bring support for the test runner output too, not just the IDE or tsc checks.

I believe I just ran into the same issue / desire... I set test.typecheck.include to ['**/*.{test,spec}?.?(c|m)[jt]s?(x)'] (removed the -d), intending to pick up all my test files this way. I was very surprised to discover that this disabled the normal tests. I thought I could have have test files containing both normal and type-level tests and look at test output that combined these.
Is there a reason to prefer separated tests?

Is there a reason to prefer separated tests?

This is a technical limitation for now.

What version will this be released in? I'm eager to take it for a test drive because it's not clear to me how it all works together - @sheremet-va said test results are not merged so I want to see how I can use them/how they're reported.

What version will this be released in? I'm eager to take it for a test drive because it's not clear to me how it all works together - @sheremet-va said test results are not merged so I want to see how I can use them/how they're reported.

It will be released in the next beta - 2.1.0-beta.6. Probably, in a few days.


Replace the hacky calling cmake install with real ament_cmake environment hooks.

Replace the hacky calling cmake install with real ament_cmake environment hooks.
Targeted for post-ardent.

Resolved by #2


Running in Ubuntu 22.04 with a higher version of CUDA

Hi,
Thank you for sharing your work.
I am trying to run the code in Ubuntu 22.04.
System details:
OS: Ubuntu 22.04 LTS
CUDA: 12.3
However, Ubuntu 22,04 only supports CUDA version >=11.7, while the default CUDA version of this DriveAdapter repo seems to be 11.3.
I tried installing the cudatoolkit package (version 11,3) in the conda environment, but there is an error when running the setup.py file.
Error:

The detected CUDA version (12.3) mismatches the version that was used to compile
PyTorch (11.3). Please make sure to use the same CUDA versions.

Question:

Is there any method that I can use to either:

Run the code with CUDA version >=11.7
Run the code in Ubuntu 22.04

What is the fixed required setting: The CUDA version must be 11.3? or the Pytorch version must be 1.12.1?

Thank you!

@thunguyenth
Yes. The cuda version must be 11.3 as the dependency for mmcv and mmdet3d is quite complex.
However, if you are familiar with the installation of mm series, it may work with higher version of Pytorch and cuda with some minor modifications of the code.  There is no specific operators has certain cuda dependency.

@jiaxiaosong1002
Thank you for your help!


TimePickerThemeData action buttons styles aren't working

Steps to reproduce

Create custom ThemeData with useMaterial3: true,
Provide timePickerTheme, changing confirmButtonStyle or\and cancelButtonStyle
Provide created theme to the app and put TimePickerDialog in Scaffold.body

There was an issue #94927 however it was closed long time ago. The answer @TahaTesser gave, didn't help, as it changed the ButtonStyles for both - confirmButton and cancelButton. I think that those props must be removed (if they truly doesn't work) or if there is a solution to this it must be somewhere in docs
Expected results
The confirmButtonStyle and cancelButtonStyle of TimePickerThemeData should have been changed
Actual results
Nothing has changed. I think if those properties are in TimerPickerThemeData and we can provide our ButtonStyles to it, it should change.
Code sample
Code sample
import 'package:flutter/material.dart';

const Color darkBlue = Color.fromARGB(255, 18, 32, 47);

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData.dark(useMaterial3: true).copyWith(
        scaffoldBackgroundColor: darkBlue,
        timePickerTheme: TimePickerThemeData(
          backgroundColor: Colors.blue,
          confirmButtonStyle: ButtonStyle(
            backgroundColor:
                MaterialStateColor.resolveWith((states) => Colors.grey),
            foregroundColor:
                MaterialStateColor.resolveWith((states) => Colors.red),
          ),
        ),
      ),
      debugShowCheckedModeBanner: false,
      home: Scaffold(
        body: Center(
          child: TimePickerDialog(
            initialTime: TimeOfDay.fromDateTime(
              DateTime.now(),
            ),
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration

Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.6, on macOS 13.5 22G74 darwin-x64, locale en-UA)
    • Flutter version 3.10.6 on channel stable at /Users/me/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (5 weeks ago), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/me/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/me/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] VS Code (version 1.81.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.70.0

[✓] Connected device (3 available)
    • iPhone 14 Pro (mobile) • 48896B42-4960-4EB8-9A92-B04AB52D8E18 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-0 (simulator)
    • macOS (desktop)        • macos                                • darwin-x64     • macOS 13.5 22G74 darwin-x64
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 115.0.5790.170

[✓] Network resources
    • All expected network resources are available.

• No issues found!

The solution provided in https://github.com/flutter/flutter/issues/94927 allows you to customize both action buttons but you're right TimePickerThemeData.confirmButtonStyle & TimePickerThemeData.cancelButtonStyle properties aren't working as expected.

Facing same issue, and setting textButtonTheme is not enough because for my usecase these buttons should be different.

...or if there is a solution to this it must be somewhere in docs

I don't think there should be one - these parameters are already exposed as parameters in TimePickerThemeData, so you'd expect they would be prioritized over any other theme props 🤔


Removed dependencies from test-manifest.yml for Integ and BWC tests

Signed-off-by: Owais Kazi<EMAIL_ADDRESS>Description
Currently,  Integ test pulls the common dependencies at once before the run for any plugin and that specific plugin can pick up the dependencies required from there. Same goes for BWC as well.
Issues Resolved
https://github.com/opensearch-project/opensearch-build/issues/521
Check List

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

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

Codecov Report

Merging #524 (cb2ca43) into main (54db534) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #524   +/-   ##
=======================================
  Coverage   69.65%   69.65%
=======================================
  Files          59       59
  Lines        1575     1575
=======================================
  Hits         1097     1097
  Misses        478      478

Impacted Files
Coverage Δ

bundle-workflow/src/manifests/test_manifest.py
100.00% <ø> (ø)

Continue to review full report at Codecov.

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

Thanks @owaiskazi19


G815/G915

Hello, I am using your application for my G815 keyboard. However there is no profile for this keyboard. I use that of the G910 but it does not have the buttons to manage the music. Is it possible to add them? Thank you
Device Name: Logitech G815
Device Website: https://www.logitechg.fr/fr-fr/products/gaming-keyboards/g815-low-profile-rgb-mechanical-gaming-keyboard.html

Logitech's SDK doesn't allow for individual addressing of the media keys.


Make modelling in mm possible

Request from a client to be able to import models in mm (since the Tekla model is in mm). I think we should wait with this and see if we can get some input on why scaling in GH is a problem. It would be possible for us to create a scaling component, or if Budapest change the struxml to allow different units

This is actually quite a big task (8-40h) for us I think. Since all our elements needs to be scaled differently.
Currently we don't have any (or very little) geometry operations.

For now, will not fix this. There are a lot of caveats with this.


Multiline in examples tables

Scenario: [case]
    Given I need to transpile
    When EcmaScript6=[EcmaScript6]
    Then EcmaScript5=[EcmaScript5]

Examples:
  case             | EcmaScript6              | EcmaScript5
  --------------------------------------------------------------------------------
  arrow function   | var r=arr.map((x)=>x*x); | "use strict";
                   |                          |
                   |                          | var r=arr.map(
                   |                          |   function(x){
                   |                          |     return x*x;
                   |                          | });
  --------------------------------------------------------------------------------
  template strings | var s=`x=${x}            | var s='x='.concat(x).concat('\n')
                   | y=${y}`;                 | .concat('y=').concat(y);

Why:

It's very useful
Simple to parse, just use a blank or a dashed line before each example.

Note:

Identation it's important
Allow one or more empty lines between rows for pretty code
Sample working here

Cucumber and Gherkin is designed to bridge the communication gap between non-technical and technical people by providing a syntax that is easy for non-technical people to learn.
It seems you want to use it more as a scripting tool. Adding code to Gherkin is not a good example of how it is intended to be used. No non-technical person would understand this.
It's the first time I've come across this feature request, so I'm not convinced it's something that more people would find useful - especially not non-technical people.

I disagree Aslak - I think this is a really cool use of Cucumber. We want to use it to write executable specifications - why not executable specifications of technical tools too?
I also like the way the tables look without the preceding column separator.

@aslakhellesoy, I sorry about disagree and and sorry too about my poor examples. I'm not a Gherkins founder and I'm learning it now. But, would be very nice if we can use it too in more complicated things, like transpiling, systems integration, text conversions, etc.
I don't wanna a scripting tool, sorry again about my examples. But it's my business work with complex data internally, with vendors and with customers.
@mattwynne , I work with medical systems. Often I need integrate with machine or systems others vendors. Below is a real sample.
Scenario: HL7 Clinical Chemistry Integration
   Given a LIS integrated by HL7
   When LIS recieve [message]
   Then LIS must emit a [report]

Examples:
   message                                                                   ┆ report
   ----------------------------------------------------------------------------------------------------
   PID|1|1034157|012993817||SNOW^JOHN||19520101|M|||1234 MAIN^^DEARBORN      ┆ JOHN SNOW, Male, 63 years
   OBX|2|NM|2093-3^CHOLESTEROL^LOINC|138.000|MG/DL|100.000 - 200.000|N|||F|  ┆ Cholesterol 138 mg/dl

+1

I'd be happy to consider a pull request for this. No amount of +1 will persuade me to do it for you though ;-)

Mmh, for me it easier to read if you would put the different cases into 2 scenarios where the when/then steps have text parts. I think that is much easier to read (and understand) than long multiline rows in table (especially in your example).


Kubernetes Components Low Image Visibility

The image of the "The components of a Kubernetes cluster" on the link https://kubernetes.io/docs/concepts/overview/components/ has a grey background that hurts the eyes a little due to containing the blue and white icons. The color combination may be improved for better visibility.
Kindly, address the issue if you feel the same about the image.

Hi @Farial-mahmod , there is an issue which address a similar problem. after zooming in , it will help seeing the image better: https://github.com/kubernetes/website/issues/38051

if the issue address different problem you can suggest changes you recommend. any suggestion or contribution is welcome.

/area web-development


BUG: Fix python wrapping for classes with only default constructor

This PR includes the following:

Fixes regular expression in ctkWrapPythonQt.py which parses the class constructors to populate the classes to be passed to PythonQT for wrapping. It was skipping the classes which only had default constructors (i.e. no constructors of the type className(QObject* ...). Also, the closing parenthesis was not identified. @jcfr: you might want to check the RE! I have compared the build outputs (by switching on verbose in the ctkMacroWrapPythonQt.cmake), and the following classes are added to Python wrapping now: ctkErrorLogLevel, ctkErrorLogQtMessageHandler, ctkErrorLogTerminalOutput, and ctkEventTranslatorPlayerWidget. ctkErrorLogAbstractMessageHandler is skipped because it has a pure virtual method, and not because the constructor type is wrong.  The other classes are unaffected.

Adds a default constructor to ctkCheckableHeaderView class so that it will be included for Python wrapping. This is actually a hack. Some methods of the class are enabled for Python wrapping as well.

👍 This looks good to me, but let's hold off until @jcfr is back to look at it.

@jcfr : When you get time, could you please look at this PR? Thanks! CircleCi failed on uploading the report at the end, but other than that all the tests were passed.

Thanks Hina. Travelling , I will have a look when I am back in office
around May 7.
Thanks
Jc
On Mon, Apr 23, 2018, 7:34 AM hina-shah<EMAIL_ADDRESS>wrote:

@jcfr https://github.com/jcfr : When you get time, could you please
look at this PR? Thanks! CircleCi failed on uploading the report at the
end, but other than that all the tests were passed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/commontk/CTK/pull/796#issuecomment-383597442, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AANXo61JY8SsWg_OP9ErpWpQshhgBlxhks5treZ5gaJpZM4TQvUN
.

Hi @jcfr, could you take a look at this again? Thanks!

@jcfr Please have a look at it whenever you have time. Thanks!

Superseded by https://github.com/commontk/CTK/pull/985 and https://github.com/commontk/CTK/pull/1078


Exception when pipe is added

Hi
please have a look at these modifcations:
https://github.com/frankbenoit/angular2-authentication-sample/commit/5910ed3f19a45a84e338463ac1a5cee0fc97f477
I don't know if this is a problem in my pipe usage, in the auth sample code, or in angular2?!
I added a list of text item and a pipe to 'home'.
Good cases:

If logged in, home can be opened normally.
Before having the pipe, when not logged in; opening 'home' directly,
the routeroutlet navigates to 'login'.

Now the bad case:
Now with the pipe, not logged in and opening 'home' directly, an exception happens and
the app crashes
Can you help?

to change between good case and bad case, just change in home.html the line 15:

{{item}}

{{item}}

Hi,
I also work on a testing project where I added angular2-jwt. I have the exactly the same error like @frankbenoit
Tried to add pipe import in several places to see if it changes behaviour. Still the error.

Would you be able to post the stack trace here? What is the exception that is thrown?
Thanks!


Add various crosses endpoints

@BrapiCoordinatorSelby  @trife
I'd like to start this issue to discuss necessary/potential changes for the v2.1 update.
It would be nice to have the following endpoints for the crosses API; although these are fairly low priority:
Projects in a specific program:
GET /crossingprojects/{programDbId}
GET /crosses/{programDbId}   or this could be a /search/crosses
GET /plannedcrosses/{programDbId}
Some less important but 'discussion worthy' endpoints:
For the Cross object, we have crossDbId, what would the ramifications be for replacing this or adding an observationUnitDbId to show that this Cross is actually a new observation unit and not just an event from two parents?
Cross:
      allOf:
      - $ref: '#/components/schemas/CrossNewRequest'
      - properties:
          crossDbId:
            description: the unique identifier for a cross
            example: d105fd6f
            type: string
        type: object

The ability to get parents:
GET /parents/        returns CrossParent objects for a given project or planned cross
GET /parents/{parentType}
I'm not sure if something like this is possible as an endpoint but returning the difference of /plannedcrosses and crosses.
GET /plannedcrosses/todo/{plannedCrossesDbId} or {crossingProjectDbId}
and GET /plannedcrosses/complete/{plannedCrossesDbId} or {crossingProjectDbId}
Basically, the ability to return the plannedcrosses that have not been crossed yet. I'm open to ideas on this one, it might be too optimized for the spec.

A lot to unpack here, but I think there would be more value in the ability to link a crossDbId to a seedLot rather than a new observationUnit. If measurements were being taken on that actual seedLot, it would be the observationUnit, but it's basically its own standalone entity once created.

Sorry I'm late to comment here, this one slipped off my radar

Your first point has already been covered by issue "#480". The URL will be GET /crosses? programDbId=xyz123 but I think the results will be the same as what you are looking for.

Regarding a connection to ObservationUnits, I agree with @trife. As far as I know, a cross is just an event between two parents and produces seeds. With issue "#446" you can now create a seed lot based on a crossDbId. In this case, I think of a cross as sort of a proto-germplasm, it is theoretically a new line, but has not yet been named/ID'd as an official accession yet. Following that logic, I guess an ObservationUnit could be created from a cross too. @chaneylc Would it suit your needs to add crossDbId to the ObservationUnit model, so an ObservationUnit can be created from a germplasm OR a cross?

Regarding parents, I'm not really sure why you need this? Using GET /crosses? crossingProjectDbId=xyz123 will get you all the cross data for that project, including all the parent entities. The only advantage I could see would be to get an array of parents without any duplicates. I would want to see proof of a serious performance issue before adding CrossParent as a primary BrAPI entity. It is designed to be a simple pointer to a germplasm/OU.

Regarding the difference between plannedCrosses and crosses, I agree that your proposal is a little too specific. But I have some general ideas that could be used to solve the same problem. What if we add a status field to the plannedCross object to record if it is "TODO", "DONE", or "SKIPPED". A "SKIPPED" status would have to be triggered manually, but we could also add plannedCrossDbId into the cross object so when the cross is created it can be linked back to the plan that was used, and when that new cross is uploaded, the server can mark the appropriate plans as "DONE" automatically. With all that, we get the URL GET /plannedcrosses? crossingProjectDbId=xyz123 & status=TODO and I think that gets the list you are looking for?

@trife @chaneylc
If my proposals above sounds like it will suit your needs, please let me know and I will implement them in the spec.


vscode-extensions.jackmacwindows.vscode-computercraft: init at 1.1.1

Description of changes

Homepage: https://github.com/MCJack123/vscode-computercraft
Description: Adds autocompletion for ComputerCraft/CC: Tweaked APIs to Visual Studio Code
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

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

1 package built:

vscode-extensions.jackmacwindows.vscode-computercraft


Composer requirements

The requirement "silverstripe/cms": "~3.1" is not necessary, I use the framework without the cms and I had to manually delete the cms folder when I added the taxonomy module with composer.

Thanks I've now removed this.


Opera Startscreen more Searchengines

In the Opera starting screen is a search bar that only supports google search.
I'd like to see an option to use it in the customisation bar on the right.
Ps.: mine is in german sorry for that.
http://image.prntscr.com/image/62e9d227d0304acab5770f7b6540a87c.png

Please address your issues in Desktop Team in blog or on Opera Forums.


(#63) Add Newly-Required Dependency of dotnet4-sdk to CCM Setup

Fixes #63
Added install of dotnetcore-sdk V 3.1.410 from the CCR to the C4bCcmSetup.ps1. Reason for this is in CCM DB 0.6.2+ this is listed as a dependency.
Also added comment explaining reasoning behind why we install the other packages in this section as well.

Apologies, @ryanrichter94. On further reflection, it would be better if we could internalize this dependency as a part of the initial Start-C4bSetup.ps1 bootstrap script. That way, it's available for the CCM script to utilize. This also keeps us more closely aligned with the CCM setup docs.
Hope that makes sense. I'll close out this PR for now.


Fix the height to end with

Fix the height to end with

Nice catch, @superboyiii, thanks.
There was a minor problem when we requested the number of blocks equal to the last height, it was not returning the Genesis Block.

Awesome fix 😝. Agree with you.

No problem.😀 I'll test it tommorrow.

:D Tested! Thanks, @superboyiii.


feat: allow highlighted numeric filters

In some use-cases, the Engine can return highlighted values for numeric filters as well, like price > <b>50</b>.
This PR allows to consider them valid when passing an appropriate flag while initializing a Parser.

Hi there ; not sure who to ping for a review on this one? @sarahdayan @DevinCodes @maxiloc 👀 🙇

Closing in favor of #15


batect not propagating host's /etc/resolv.conf to containers

batect is not propagating host's /etc/resolv.conf to the container. As a result, the DNS resolution fails. The example using alpine image is shown below.
Note that on running the same container using docker, the name resolution succeeds.
Further, on mounting host's /etc/resolv.conf to the container in the batect container definition and login to the container, the DNS resolution succeeds.
batect version = 0.38.0
docker image = alpine:3.10.3 (this is just an example but issue is seen with other images also)
$ ./batect shell
Running shell...
deploy-env: running bash
bash-5.0$ nslookup forwardproxy
nslookup: can't resolve '(null)': Name does not resolve
nslookup: can't resolve 'forwardproxy': Try again

$ docker run -it alpine:3.10.3 sh
bash-5.0# nslookup forwardproxy
nslookup: can't resolve '(null)': Name does not resolve
Name:      forwardproxy
Address 1: <IP_ADDRESS> ip-10-39-66-83.ap-southeast-2.compute.internal
Address 2: <IP_ADDRESS> ip-10-39-66-137.ap-southeast-2.compute.internal
Address 3: <IP_ADDRESS> ip-10-39-66-227.ap-southeast-2.compute.internal

Hey @chandantp, thanks for the feedback. This may be an issue with how Docker does DNS resolution in non-default networks. By default, it will set up a DNS server of its own that will delegate to the host's resolv.conf if it can't answer the query. (https://docs.docker.com/v17.09/engine/userguide/networking/configure-dns/ has more details.)
If you attempt to resolve the name with the full domain name (eg. instead of nslookup forwardproxy, use nslookup forwardproxy.mycompany.com), does it work?

Docker version: 19.03.5, build 633a0ea (missed providing this earlier)
If the full domain name is provided, the DNS resolution is successful as can be seen below.
$ ./batect shell
Running shell...
deploy-env: running sh
/code $ nslookup forwardproxy.extnp.mycompany.com.au
nslookup: can't resolve '(null)': Name does not resolve
Name:      forwardproxy.extnp.mycompany.com.au
Address 1: <IP_ADDRESS> ip-10-39-66-137.ap-southeast-2.compute.internal
Address 2: <IP_ADDRESS> ip-10-39-66-227.ap-southeast-2.compute.internal
Address 3: <IP_ADDRESS> ip-10-39-66-83.ap-southeast-2.compute.internal

Apparently, when the container is run using batect, the DNS resolution is not being delegated to the host's resolv.conf when the embedded DNS server is unable to resolve the hostname. The same however works when the container is run directly using docker CLI.
Another thing to be noted is the difference in the content of resolv.conf when the container is run using batect versus docker. Per the link you sent above, I expected the nameserver to be the embedded DNS server (<IP_ADDRESS>) in both cases. However, as you can see below, the batect run alpine container shows the nameserver to be the embedded DNS server where as the docker run alpine container shows the host's nameservers. Is this significant?
$ ./batect shell
Running shell...
deploy-env: running sh
/code $ cat /etc/resolv.conf
search extnp.mycompany.com.au
nameserver <IP_ADDRESS>
options timeout:2 attempts:2 rotate ndots:0

$ docker run -it alpine:3.10.3 sh
/ # cat /etc/resolv.conf
options timeout:2 attempts:2 rotate
; generated by /usr/sbin/dhclient-script
search extnp.mycompany.com.au
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>

The other big difference between docker run ... and batect is that batect also starts a separate Docker network for each task, which results in the behaviour described in that page I linked to.
Can you please try running the following and let me know if you run into the same behaviour?
docker network create test-network
docker run --rm -it --network test-network alpine:3.10.3 sh
# To clean up once you're done:
docker network rm test-network

This will reveal whether it is an issue with batect or Docker itself.

Yes, I run into the same behaviour with the above commands as can be seen below:
$ docker run --rm -it --network test-network alpine:3.10.3 sh
/ # nslookup forwardproxy
nslookup: can't resolve '(null)': Name does not resolve
nslookup: can't resolve 'forwardproxy': Try again

OK, in that case, it's not an issue with batect, but an issue with Docker. I can't find this issue reported elsewhere, so I'd suggest filing an issue with them.
In the meantime, configuring --dns-search for your Docker daemon (see https://docs.docker.com/engine/reference/commandline/dockerd/) may resolve the issue.


Investigate long response time of IconApi.getIconUrl() after a config is imported

Description
I'm doing some work to address https://github.com/digidem/comapeo-mobile/issues/686 and based on my testing, I'm seeing the following:

Import a config
Go to a screen that contains a react-query query that calls IconApi.getIconUrl(). The request to get the url times out (the relevant react-query query with default configuration gives AbortError: The operation was aborted error)
Exit screen and return to it
Relevant query succeeds much faster and the url that's retrieved works well.

Tasks

[ ] Write a test simulating the above. Would be interested to see if the timeout in 2 is expected. If it is, we can probably account for it and at least have context (we kind of already do, but it doesn't have any contextual information for why it's configured that way).
[ ] Address the issue causing the resolution of IconApi.getIconUrl() taking a long time

Hm maybe the problem lies somewhere in the IPC boundary? From what I can tell,  the IconApi.getIconUrl() implementation is basically string concatenation 🤔

Could it be that the server hasn't started yet? The abort could be coming from this code:
https://github.com/digidem/comapeo-core/blob/7e27b5aa9300dcac8a5ec46361e9cd49794a1f1d/src/fastify-plugins/utils.js#L19-L21
You could test that by logging when the Fastify server starts listening. If it starts listening after the timeout, that would explain it. If it starts listening before the timeout, then the error lives elsewhere.

does the fastify server (or the relevant plugin?) restart after importing a config? i noticed that the icon url port changed before and after step 1 e.g. went from http://<IP_ADDRESS>:39819/... to http://<IP_ADDRESS>:42747/...

Going to close this since I think it's more of an app-level concern. Have a strong feeling that what's explained in the comments above is what's happening

Maybe one last thing I'll note is an open question about whether the timeout here should be configurable or increased?
https://github.com/digidem/comapeo-core/blob/7e27b5aa9300dcac8a5ec46361e9cd49794a1f1d/src/mapeo-manager.js#L250

I think it makes more sense to increase it than to add an extra configuration option, but I don't feel strongly.


fix: sg arn output

what

When you're using external sg with setting create_security_group = false and don't want to create default one which is included in module you get response with nullable element and can't concatenate null values.

│ Error: Invalid function argument
│
│   on .terraform/modules/elasticache_memcached/outputs.tf line 12, in output "security_group_arn":
│   12:   value       = join("", module.aws_security_group[*].arn)
│     ├────────────────
│     │ while calling join(separator, lists...)
│     │ module.aws_security_group is object with 4 attributes
│
│ Invalid value for "lists" parameter: element 0 is null; cannot concatenate null values.

This should fix a long standing issue https://github.com/cloudposse/terraform-aws-elasticache-memcached/issues/45.

can you please run:
make precommit/terraform

@aknysh ping 🙂

@aknysh ping

/terratest

@jamengual can you merge?

can you please run:
make precommit/terraform

@dmitrijn

This very issue is handled nicely in the redis module so I'm frustrated as to why this isn't merged so we can actually use this module instead of forking it and fixing it ourselves.  https://github.com/cloudposse/terraform-aws-elasticache-redis/blob/main/main.tf#L152
Please merge? Pretty please??

@adamwshero thanks for the bump. We get a lot of PRs + issues with the hundreds of modules, so it's typically a "squeaky wheels gets the oil" type of situation. If you're ever annoyed by this, feel free to ping in the #pr-reviews channel in the SweetOps slack and we're pretty responsive there.
Sadly, this PR is a bit out of date and our CI has changed which has caused it to get into a base state (or at least it looks that way to me). @dmitrijn would you mind pulling in main into your branch (or rebasing) and pushing those changes? That should restart the process and we'll hopefully be able to get this merged once our CI goes green. Thanks!

bump

@adamwshero mind forking / copying @dmitrijn's change and putting up a PR yourself that is forked from main? We can get this merged quickly, we just need CI checks to not be in the way and that means a new branch or rebase off main. Add me as a reviewer and I'll work to get it shipped 👍

@adamwshero mind forking / copying @dmitrijn's change and putting up a PR yourself that is forked from main? We can get this merged quickly, we just need CI checks to not be in the way and that means a new branch or rebase off main. Add me as a reviewer and I'll work to get it shipped 👍

Not @adamwshero , but I have this problem so I opened https://github.com/cloudposse/terraform-aws-elasticache-memcached/pull/67 since it's such a small change. Looking forward to seeing it get merged!

@adamwshero @dmitrijn this was fixed in #67 and have been released in https://github.com/cloudposse/terraform-aws-elasticache-memcached/releases/tag/0.19.1  -- Sorry for the delay and that we didn't get it done as part of this PR, but at least it's fixed! Thanks for your work here -- closing out 👍


casing of extension used for new r markdown files

Please briefly describe what you were doing that led to the issue, if applicable:
In vs17 rtools using file | new | file | R | R Markdown creates boilerplate file with ".rmd" as its extension.   In rstudio using file | new file | R Markdown creates boilerplate file that when you save it for first time gets ".Rmd" as its extension.
Not sure if casing difference is intentional or is something that should be aligned with casing rstudio defaults to using.   Switching the casing to the ".Rmd" format doesn't appear to affect any vs17 rtools ability to recognize and handle it as r markdown.
If you have any screenshots demonstrating the issue, please include them as well to help us diagnose it better.
Additional information:
OS Information
Version:       Microsoft Windows NT 10.0.15063.0
RTVS Information:
Assembly: Microsoft.VisualStudio.R.Package, Version=1.1.30523.1233, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a
Active R URI:
Microsoft R Client (<IP_ADDRESS>): C:\Program Files\Microsoft\R Client\R_SERVER\

Looks like GIT configuration issue since it defaults to ignore file name casing

https://github.com/Microsoft/RTVS/commit/dad3e65a0ab9ae9f936772f00a846038c3d324da


Add explicit schema for "sign" function

In https://postgrest.org/en/v5.0/auth.html#sql-user-management, inside jwt_test and login.
Related to https://github.com/PostgREST/postgrest/issues/1156

Also add schema for login in https://postgrest.org/en/v5.0/auth.html#logins
Related to https://github.com/PostgREST/postgrest/issues/1155


docs: update README.md

What:

Update Read me.

Issue number: N/A

Codecov Report

Merging #538 into master will decrease coverage by 0.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #538      +/-   ##
==========================================
- Coverage   69.65%   69.47%   -0.18%
==========================================
  Files          65       65
  Lines         557      557
  Branches       35       35
==========================================
- Hits          388      387       -1
- Misses        161      162       +1
  Partials        8        8

Impacted Files
Coverage Δ

...tarter/src/app/core/animations/route.animations.ts
70.00% <0.00%> (-10.00%)
:arrow_down:

Continue to review full report at Codecov.

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


Define and create Lasair 2.0 announcement

When everything is ready, we need to publicise the official release of Lasair 2.0 and Sherlock 2.0. By default this will be (a) On the LSST:UK public site, and (b) as some kind of news item on the Lasair website. We can also consider other mechanisms.

I started writing my own wee plan for this, which I shall put together and share shortly. My thinking is:

make LSST-UK news item
a flash on "home" page
make a "Lasair 2.0" release page

link to this on "release notes" page
link to this on home-flash
list new things in this release
(Sherlock, documentation, recent additions)
(implementation on IRIS)
(stability and performance)
link to roadmap page

I think this gives me enough to count this as "in progress"

Lasair 2.0 release news. I suggest :
On front page put text :
On 7th April 2020 we released the stable, functioning version 2.0 of the Lasair broker for processing ZTF alerts. The additions and improvements in this release are found on this page.
Create a page, link from Lasair home with content:
Lasair 2.0
On 7th April 2020 we released the stable, functioning version 2.0 of the Lasair broker for processing ZTF alerts. The additions and improvements in this release are

Documentation  *Insert links
A new, upgraded version of the context classifier Sherlock Version 2.0 Insert link
Lasair is now running on the the UK's IRIS high performance computing facility insert generic link to IRIS
We provide a forward look to our broad plans to develop Lasair for the LSST alert stream
Anything else (e.g. stability/performance) that Roy/Ken want to highlight

Suggest putting the above text under the heading Lasair 2.0 on the page
https://lasair-dev.roe.ac.uk/about
and
https://lasair.roe.ac.uk/about

Lasair is not running on IRIS, it is on the LSST testbed at ROE

That'd fine, just write whatever it is running on

It says "Lasair provides Kafka output for query/filters: contact us for more information."
The reason I did not put the kafka endpoint address is that it is not a secure installation. Stellios has it on his to-do list to add access control so that only ourselves can put events into the kafka.

Things remaining to do before this ticket is done:
(1) Draft text for LSST:UK web page news item
(2) Draft text for LSST:UK consortium mailshot
(3) Draft text for mailshot to Lasair registered users.
Then to send out those items!

Would suggest putting in the LSST:UK August Newsletter and the website and not do mail shots :
(1) Draft text for LSST:UK web page news item
(2) Draft text for LSST:UK August Newsletter


Remove usage of backticks

**I admit a somehow frivolous PR here. But is it that frivolous to display good habits? :) **
There are several reason to always prefer the other form
for command output substiution.  See e.g.
http://unix.stackexchange.com/questions/126927/have-backticks-i-e-cmd-in-sh-shells-been-deprecated
for a starting point.

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at<EMAIL_ADDRESS>Thanks!

@michipili Thanks! This is the TIL for me today (was just using backsticks for the ease of typing).


Document the need to encrypt secrets in etcd

Fixes https://github.com/external-secrets/kubernetes-external-secrets/issues/538

Thanks! :)


The build scheme file (.xcscheme) made by generate-xcode-support can have unescaped spaces in the paths

Sonoma 14.4.1, Xcode 15.3, Swift 5.10, swift-bundler 2.0.3
To see it:
swift bundler create HelloBusted --template SwiftUI && \
cd HelloBusted && \
swift bundler generate-xcode-support && \
open Package.swift

Hit the big blue button to build & run... & it'll fail with the only semi useful error: Executable Path is a Directory.
Cause:
BuildAction & PathRunnable in HelloBusted.xcscheme both have paths to directories in ~/Library/Application Support/, & the space in Application Support isn't escaped.
Workaround:
Edit the file to escape any spaces in paths, eg Application\ Support,  restart xCode & all's well. Application Support's likely to be there for everyone.
Fix:
Add another escape in Sources/swift-bundler/Bundler/XcodeSupportGenerator.swift I guess.
Extremely cool project, had it recommended to me yesterday. Thank you!

Thanks for reporting this issue! Must've been introduced by a newish version of Xcode, cause I can reproduce the issue but last time I tested Xcode support it worked (which was probably 6 months ago at this point).
I've added an extra escape which resolves the issue on my end, hopefully that resolves it for you too 🤞

Extremely cool project, had it recommended to me yesterday. Thank you!

Thanks! I'm always interested how people find out about Swift Bundler, did you get recommended it by a colleague?

Fixed in v3.0.0! Quick turn around, ta. 😄
Found you via social media, Mastodon, which if you haven't heard of it's a Twitter clone. (Clone is probably not doing it justice.) I was complaining about rolling packages by hand & got the recommendation.

Fixed in v3.0.0!

Woohoo!

I was complaining about rolling packages by hand & got the recommendation.

Ah, good old Sven spreading the word 😅


TODO: Refactoring of Volumetric Lighting Demo

broken due to the new Post Processing stack logic

not a priority, currently the scene is removed


Issue generating Swagger with WebApiToSwaggerGenerator

Hello,
I don't really know if this is an issue, but I'm not being able to generate the swagger and I do need some help. I have a really simple ApiController with two POST methods, one receives a file (so no arguments on the method, and the other a string (string as argument).
I followed the sample code from the Wiki on this matter. When I call the URL for the swagger on :
return new FileContentResult(_swagger.Value, "application/json");
I get:
<Error>
<Message>An error has occurred.</Message>
<ExceptionMessage>
The 'ObjectContent1' type failed to serialize the response body for content type 'application/xml; charset=utf-8'.
</ExceptionMessage>
<ExceptionType>System.InvalidOperationException</ExceptionType>
<StackTrace/>
<InnerException>
<Message>An error has occurred.</Message>
<ExceptionMessage>
Type 'System.Web.Mvc.FileContentResult' cannot be serialized. Consider marking it with the DataContractAttribute attribute, and marking all of its members you want serialized with the DataMemberAttribute attribute. If the type is a collection, consider marking it with the CollectionDataContractAttribute. See the Microsoft .NET Framework documentation for other supported types.
</ExceptionMessage>
<ExceptionType>
System.Runtime.Serialization.InvalidDataContractException
</ExceptionType>
<StackTrace>
at System.Runtime.Serialization.DataContract.DataContractCriticalHelper.ThrowInvalidDataContractException(String message, Type type) at System.Runtime.Serialization.DataContract.DataContractCriticalHelper.CreateDataContract(Int32 id, RuntimeTypeHandle typeHandle, Type type) at System.Runtime.Serialization.DataContract.DataContractCriticalHelper.GetDataContractSkipValidation(Int32 id, RuntimeTypeHandle typeHandle, Type type) at System.Runtime.Serialization.DataContractSerializer.GetDataContract(DataContract declaredTypeContract, Type declaredType, Type objectType) at System.Runtime.Serialization.DataContractSerializer.InternalWriteObjectContent(XmlWriterDelegator writer, Object graph, DataContractResolver dataContractResolver) at System.Runtime.Serialization.DataContractSerializer.InternalWriteObject(XmlWriterDelegator writer, Object graph, DataContractResolver dataContractResolver) at System.Runtime.Serialization.XmlObjectSerializer.WriteObjectHandleExceptions(XmlWriterDelegator writer, Object graph, DataContractResolver dataContractResolver) at System.Runtime.Serialization.DataContractSerializer.WriteObject(XmlWriter writer, Object graph) at System.Net.Http.Formatting.XmlMediaTypeFormatter.WriteToStream(Type type, Object value, Stream writeStream, HttpContent content) at System.Net.Http.Formatting.XmlMediaTypeFormatter.WriteToStreamAsync(Type type, Object value, Stream writeStream, HttpContent content, TransportContext transportContext, CancellationToken cancellationToken) --- End of stack trace from previous location where exception was thrown --- at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task) at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at System.Web.Http.WebHost.HttpControllerHandler.<WriteBufferedResponseContentAsync>d__1b.MoveNext()
</StackTrace>
</InnerException>
</Error>

Besides I've debugged and the JSON for the Swagger is being correctly generated.

The return type of the action method must be ActionResult... Can you post the signature of the action method?

        public HttpResponseMessage AddManifFile()         {
and
        public HttpResponseMessage AddManif(string _manifest)         {
Using this for WebAPI.

The problem is the method with "new FileContentResult" not the other methods... Try using FileContentResult as return type...


Looking for a Code Explorer/Harness

I'm looking for some project I can use as a harness to showcase the source code of my JavaScript project on my website as it's running. E.g. click a link and present a file list in one pane and CodeMirror in the other. Any leads?

This is not the place for such requests, sorry.


Mark Request.queryParameters as deprecated in v2

This change will alert developers that in 2.0 and beyond they should package query parameters in the url property.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Daniel Larsen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #61 into master will decrease coverage by 0.16%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
- Coverage   96.39%   96.23%   -0.17%
==========================================
  Files          33       33
  Lines        1192     1194       +2
==========================================
  Hits         1149     1149
- Misses         43       45       +2

Impacted Files
Coverage Δ

Sources/Hyperspace/Request/Request.swift
94.02% <0%> (-2.9%)
:arrow_down:

Continue to review full report at Codecov.

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


GPU Acceleration

Describe what you are looking for
Is usearch support backend of gpu ,  run in gpu can be faster?
Can you contribute to the implementation?

[ ] I can contribute

Is your feature request specific to a certain interface?
It applies to everything
Contact Details
No response
Is there an existing issue for this?

[X] I have searched the existing issues

Code of Conduct

[X] I agree to follow this project's Code of Conduct

https://github.com/rapidsai/cuvs
Also look into how ArangoDB does GPU with cuvs. Or Use GPU.js


Installation issues with Rails 4

I've been struggling with this for a couple of days now.  I installed the gem, but Rails seems to be having a lot of trouble finding the files.  After repeated install / uninstalls, it stopped throwing errors on
*= require froala_editor.min.css
*= require froala_style.min.css
*= require font-awesome
It still can't find any of the plugin files.  I get messages like:
couldn't find file 'plugins/align.min.js' with type 'application/javascript'
And when I try to run it, it get:
Uncaught TypeError: $(...).froalaEditor is not a function
Am I missing something here?
Bundle install shows "Using wysiwyg-rails 1.2.8"
This is how I have the requires set up:
application.css.scss:
*= require froala_editor.min.css
*= require froala_style.min.css
*= require font-awesome
*= require_tree .
*= require_self
application.js:
//= require jquery
//= require jquery.turbolinks
//= require jquery_ujs
//= require 'typeahead.jquery'
//= require 'bloodhound'
//= require bootstrap-wysihtml5
//= require bootstrap-sprockets
//= require froala_editor.min.js
//= require turbolinks
//= require_tree .

We have updated last week this repo to V2 RC3. The current README is for V2 RC3 and you should specify the version when initializing the editor gem "wysiwyg-rails", "2.0.0.pre.rc.3".


'Question: [Whats the recommended way to play videos?]'

can I use iOS/Android RN components like

You would need to implement React Native Video (or some comparable library) for macOS. I did this for Windows back in the day. It isn't too difficult as long as you can find comparable native APIs to use.

You would need to implement React Native Video (or some comparable library) for macOS. I did this for Windows back in the day. It isn't too difficult as long as you can find comparable native APIs to use.

I tried using react Native video, and got undefined constant. So I would have to write the native code as well?
Also I would want this to be cross platform, so I would do this in windows as well?

The vast majority of React Native projects that use native code will not support macOS, so yes, you would need to write the native code yourself or wait until someone else does it. You can usually tell which platforms are supported by the native folders in the root of the project. React Native Video, for example, has iOS, android, and windows folders, so those are the supported platforms (although, if a package has no native platform folders it probably is a javascript-only package and may work on any platform).

I feel like if the process for adding macOS support is a bit more clearly documented (as per #467), it would give developers a good starting point.
In React Native Video's case specifically, I believe the underlying AVFoundation classes are the same for both iOS and macOS, so it may be possible to simply enable macOS as a valid build target.


[vm] FlowGraphCompiler::EmitMove does not support kUnboxedFloat from const source to stack

For const moves, it uses the constant instruction:
https://github.com/dart-lang/sdk/blob/5403dc49273dec83d2f22fa3035acf5debc564b3/runtime/vm/compiler/backend/flow_graph_compiler_arm.cc#L906-L916
And this instruction doesn't support kUnboxedFloat, and silently does the wrong thing on some archs:
Silent failure on arm32, assuming kTagged:
https://github.com/dart-lang/sdk/blob/5403dc49273dec83d2f22fa3035acf5debc564b3/runtime/vm/compiler/backend/il_arm.cc#L903-L917
Discovered on https://dart-review.googlesource.com/c/sdk/+/284300/59

Our IL tests don't give us access to the assembler. Our assembler tests don't have a flow graph. There is actually no need for ConstantInstr::EmitMoveToLocation to take a FlowGraphCompiler as argument, it only uses the Assembler. However, then we'd lose the parallel with all EmitNativeCode methods. And it would subsequently make more sense to move the implementation into the assemblers instead of the il_xx.cc files.
Not sure if that refactoring would be worth it.

So far the Assembler does not depend on Location. (And introducing that dependency currently creates an import cycle between the header files.)
The assembler is context agnostic, e.g. it doesn't know about IL, the FlowGraph, etc.
Location is mostly context agnostic. However, constant locations refer to a ConstantInstr which encapsulates the actual value and representation.
If we ignore the constant locations being backed by an instruction, then the target destination of a move is completely context agnostic, and we could have destination locations in the assembler.
(Thanks @sstrickl for some of these observations!)

CL without refactoring: https://dart-review.googlesource.com/c/sdk/+/332761


Add support solar (or green) power, home battery, charging modes, nordpool_diff

I've been using a similar automation to control my charging schedule using entso-e pricing.
More info see: https://community.home-assistant.io/t/ev-charging-scheduling-solution-with-solar-power-and-a-home-battery/482225
To be fair a in my opinion a few essential things are missing from your integration:

Green charging support: start charging when free solar power > kwh
home battery support: control charging based on the home battery level
Charging modes: a lot of chargers have interesting modes - normal, eco mode, peak saving mode, ...
norpool_diff: this integrating gives a 'score' of the current price compared to the price for the next few hours. This is VERY handy to determine the best moments to charge your car. See https://github.com/jpulakka/nordpool_diff

I use the following options:

automatic / manual schedule
only charge at night (which is 1900 to 0700)
use nordpool_diff 6 hours (= interval, 6 hours)
use nordpool_diff 20 hours (= interval, 20 hours)
charge from home battery only, ignore previous options together with a min battery level option to start and stop (default = 40% min to start and 25% min to stop)
only start charging when the home battery is full (meaning >98%)
Ignore all options and start charging when the car is connected

Small note: I have a Polestar 2 which doesn't have a SOC available. To be fair, you don't need it to run efficiëntly.
The result:

Thanks for sharing. To support home batteries is already on my to-do list, so that will probably be added. To do even more advanced control is something I need to think about.

Hi,
I also love to have support for soar energy, in my case which don´t have batteries I would like to use the solar energy as long it´s available. That´s is In´m producing more energy that I´m using and then using the lowest price.
A question, could I use manually start and stop the charging to use the solar energy during the day, will it then automatically recalculate the hours it should charge after I stop with the manually charging?

Hi, I also love to have support for soar energy, in my case which don´t have batteries I would like to use the solar energy as long it´s available. That´s is In´m producing more energy that I´m using and then using the lowest price.
A question, could I use manually start and stop the charging to use the solar energy during the day, will it then automatically recalculate the hours it should charge after I stop with the manually charging?

Create an automation that triggers when you have excess power from solar energy and force start the charger. You need to create a sensor that measures the excess power you have from solar and counts the power you put into your EV.
Adding solar support etc. is something I think should be seriously reconsidered. Is it really the purpose of this integration? There are so many ways to add solar/battery support automation in HA, that could trigger this integration, which currently has the main focus of "charge your EV when the electricity price is low".

A question, could I use manually start and stop the charging to use the solar energy during the day, will it then automatically recalculate the hours it should charge after I stop with the manually charging?

I would recommend to make an automation that when you want to manually start charging does

Start charging
Turn switch.ev_smart_charging_smart_charging_activated off

and another automation that when you want to manually stop charging does

Stop charging
Turn switch.ev_smart_charging_smart_charging_activated on

This will ensure that integration will recalculate the planned charging when you manually stop charging.

A question, could I use manually start and stop the charging to use the solar energy during the day, will it then automatically recalculate the hours it should charge after I stop with the manually charging?

I would recommend to make an automation that when you want to manually start charging does
* Start charging

* Turn `switch.ev_smart_charging_smart_charging_activated` off

and another automation that when you want to manually stop charging does
* Stop charging

* Turn `switch.ev_smart_charging_smart_charging_activated` on

This will ensure that integration will recalculate the planned charging when you manually stop charging.

Great, I will try that. One more question, could you use start/stop manually charging if you have stopped the automatically charging?

Great, I will try that. One more question, could you use start/stop manually charging if you have stopped the automatically charging?

Yes, you can.


fixed filepath bug

there are two new issues:

api\tests\routes\export_requests\list.test.ts timed out but works if you increase timeout to 10k ms
I can't export anything maybe I think the problem is in the file: run-entrypoint-file.ts

We can increase the timeout for the test inside the workflow (not a big deal) do you want to make that edit to see if the tests pass? I didn't understand your other issue, the diff is a bit hard to read because of a formatting issue, maybe you could run bun run format

@seveibar I used to review my own code I don't why I stopped doing that, sorry for that

@ShiboSoftwareDev all good, we should enable a lint rule for console log lines anyway so that its not easy to leave them in- i think biome has a setting


fix broken gcloud auth login command in configuring OIDC

Why:
In an example YAML file of "List services in GCP" workflow, there is an incorrect bash syntax "$".
https://docs.github.com/en/actions/deployment/security-hardening-your-deployments/configuring-openid-connect-in-google-cloud-platform

It should be "${{ steps.auth.outputs.credentials_file_path }}".
What's being changed:
I escaped {{ and }}.
Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Modified:

@shogo82148
Thanks so much for opening a PR! I'll get this triaged for review :zap:

Hi @shogo82148, thank you for creating this PR ✨
It looks like a similar fixed was already merged to close this issue https://github.com/github/docs/issues/12026 and the article looks correct on the live docs site, so I'm closing this PR. If you're still seeing the issue, please feel free to reopen this and we can take another look. 😄
Thank you for contributing to the GitHub docs ⚡


Material Specifications: cannot unpack non-iterable ValueNode object

@MicahGale How are material cards supposed to be formatted for MontePy to parse them? I keep getting the error in the subject line. When I inspect the isotope_fractions variable in \data_inputs\material.py, it is a ListNode object containing ValueNode objects. For any ValueNode object, I see no attribute that corresponds to the isotopic fraction (at least not one intuitively named). Additionally this object is non-iterable so the usage isn't correct either. Am I missing something?

This seems like a bug most likely. MontePy should be able to handle a material definition as long as the parameters come at the end.
So this should work
m1 1001.80c 0.33
      8016.80c 0.666667
      plib=80p

This syntax breaks it:
m1 1001.80c 0.33
      plib=80p
      8016.80c 0.666667

These object types that you are seeing are data types from the syntax tree. A bad syntax should be handled with syntax errors. So this is definitely odd.
Could you provide an example made up material that causes this bug?

Is the plib keyword required? I specify my materials like (without plib):
m1 1001.80c 0.33
      8016.80c 0.666667

No plib is not at all required. It was just an example of key-value parameter that you can use.
That looks like it should work. I'll have to look into this further.

I've never used SLY so I'm just throwing ideas at the wall but is the issue possible linked to _flush_phrase never being called for the isotope_fractions (since _flush_phrase is what generates the ValueNode - although this doesn't make a ValueNode iterable but mahybe you could zip the ZAID and fractions ValueNode's)?

That example code I just dropped in MontePy and it didn't have any issues as I suspected. Could you get us a stack trace and MWE?
As for:

I've never used SLY so I'm just throwing ideas at the wall but is the issue possibly linked to _flush_phrase never being called for the isotope_fractions (since _flush_phrase is what generates the ValueNode - although this doesn't make a ValueNode iterable but maybe you could zip the ZAID and fractions ValueNode's)?

So _flush_phrase isn't a SLY thing, that's just a utility function I made. First this guide should give a quick intro to it. Basically _flush_phrase is just used for generating ValueNode as leaves of the tree quickly. isotopes_factions which is of type IsotopeNode is a list of pairs of ValueNodes that represent the ZAID and then the atom/weight fraction. So the issue is that somehow a ValueNode is getting put too high in the syntax tree that the Material object as it tries to walk the Syntax Tree is finding it instead of a IsotopesNode (which IIRC is a subclass of ListNode) for some reason.
Dynamic typing strikes again.

@MicahGale Ahah! Found the issue - I lied to you. I did not, in fact, supply the abx suffix - which, as you probably know:

If .abx is omitted the first entry from the xsdir file corresponding to the proper library class will
be used. This same approach will be used for all libraries not specified.

@MicahGale By the way, are we adding of these to your unit tests (or maybe these aren't worthy)?

So to confirm the problem is:
m1 1001 0.333
    8016 0.6667

Yes I know this absolutely valid and can't believe I missed this as a test.
So yes we will absolutely add this to our test suite. It is the expectation for any bug fixes. What I will usually do first is write/modify a test to replicate the bug. Ensure that it fails, then push that test to the PR so that way a record of the test failing is preserved. This just makes it nicer for the the reviewer ok really @tjlaboss to verify that the test is effective. I think 100% of the tests in tests/test_edge_cases.py comes from bug reports.

Error confirmed. Full stack trace:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In[3], line 1
----> 1 material = montepy.data_inputs.material.Material(input)

File ~/dev/montepy/montepy/data_inputs/material.py:40, in Material.__init__(self, input)
     38 set_atom_frac = False
     39 isotope_fractions = self._tree["data"]
---> 40 for isotope_node, fraction in isotope_fractions:
     41     isotope = Isotope(node=isotope_node)
     42     fraction.is_negatable_float = True

TypeError: cannot unpack non-iterable ValueNode object


Unescaped 'type' keyword in package (directory) name

Hello! I tried to generate scala files with latest mu-scalaand I've faced this error:
Failed to parse package name: <input>:1: error: identifier expected but type found
google.logging.type
               ^

This error raises when it tries to process  this file - google/logging/type/http_request.proto. Package name (directory) contains type keyword so we are not able to properly parse it here:
val packageName = protocol.pkg // "google.logging.type"
      .getOrElse("proto")
      .parse[Term]
      .toEither
      .leftMap(e => s"Failed to parse package name: $e")
      .flatMap(_.as[Term.Ref])

As far as these protos comes with kinda "standard" google library (https://github.com/googleapis/api-common-protos) skeuomorph have to process it properly

@juanpedromoreno  Is there anything I could help with?
It seems like skeuomorph already has some "escaping" stuff, maybe if you can point me where to start I can fix&PR it by myself

Hi @Dr-Nikson , thanks for reporting this bug, I triaged it but completely forgot to reply/address the issue.
I've just submitted https://github.com/higherkindness/skeuomorph/pull/305

@juanpedromoreno so, thank you :) You're doing a really good job with this lib 👍🏼
As I can see mu-scala was released today, but https://github.com/higherkindness/sbt-mu-srcgen wasn't. Can I ask you to bump dependencies to ship this release to production? Or I can make a PR by myself if it's make sense somehow

Yeah, the mu-scala artifacts were just released, and they should go on its way to maven central. As soon as they are ready, @ scala-steward (or any of us) can submit to PR updating the libraries.

https://github.com/higherkindness/sbt-mu-srcgen/pull/68

@juanpedromoreno thank you so much for this!)


Deprecate ETL page

Problem
I believe we should deprecate: https://github.com/hashgraph/hedera-docs/blob/master/core-concepts/mirror-nodes/hedera-etl.md
We have deprecated the underlying pub-sub service.
Solution
Investigate with mirror node team if we should deprecate page.
Alternatives
No response

Waiting on the green light from the Mirror Node team to move forward with deprecation.

Resolved in dc87265


feat(gridsome): Collapsible sections on detail pages

One suggestion from the design review was to introduce collapsible sections in the detail page rendering:

We can probably use https://tailwind-elements.com/docs/standard/components/accordion for this, with the .accordion-flush class, and data-bs-parent attributes removed to let multiple elements be open.


Bai19

Review dùm em class LoginIceHrmTest. Em có tách ra 2 bộ data cho 2 case là success và failse nhưng run báo lỗi em chưa fix được.
Do em tách ra để chạy file Smoke.XML và file Regrestion.XML ( theo anh Khánh có chỉ thì smoke chỉ để happy case nên em mới tách riêng 2 dataprovider là success và false )

cmt: 5
"Hi ,
You are providing an incorrect PR link.
Here is an example of a correct pull request.
Plz resubmit with the correct link.
Example (DO NOT USE):[https://github.com/your-github-username/automation-course-template/pull/id_of_your PR]

https://github.com/trongtuyen271199/automation-course-template/pull/4

Hi @trongtuyen271199
I have checked your homework.
Cause of error:
In line 33, You declared the testdata array with a size of new Object[3][3], meaning it only has 3 rows (index 0 to 2). However, in the value assignment, you attempted to access testdata[3], which caused an error.
hint:
you can fix this error like this image
https://share.zight.com/v1uqZob1

//Comment from HW team,
Hello, @trongtuyen271199
Your homework has been reviewed.
 Keep up the good work!
https://share.zight.com/Apuq5wOG
Best wishes from the Automation Test Pro team (Son Le<EMAIL_ADDRESS>


Fetch & cache consensus parameters when Provider is initialized

This API change/feature attempts to solve #1050
Similar to the Rust SDK's Provider::connect:
/// Connects to an existing node at the given address.
    pub async fn connect(url: impl AsRef<str>) -> Result<Provider> {
        let client = FuelClient::new(url).map_err(|err| error!(InfrastructureError, "{err}"))?;
        let consensus_parameters = client.chain_info().await?.consensus_parameters.into();
        Ok(Provider::new(client, consensus_parameters))
    }

In the TS SDK, we could have:
// new API
const provider = await Provider.connect('https://node.url');

// old API
const provider = new Provider('https://node.url')

Motivation behind this API change
Making the initialization of the provider asynchronous will let us internally fetch consensus parameters from the chain and cache them on our provider object for later use.
This means that all of our internal logic related to contract calls can stay synchronous as it is with minimal modifications, and just consume these cached consensus parameters from the provider object whenever needed.
Please correct me if I am wrong - but I think the consensus parameters of the chain should not change often. Still, if the user wants to invalidate the cache and wants the provider to use the latest consensus parameters, we can expose a method called invalidateConsensusParamsCache from the Provider class that lets them do that explicitly.

great


Idea: Combine web, cli, mobile codebases into one

It might be a good idea to abstract the core functionality of the kalk into one library and then import it in cli, web and mobile wrappers. Otherwise you have to fix all bugs and implement all features 3x times.
The easiest choice would be to implement the core lib in JS, because it can easily run on cli, web and mobile. However, it might be possible to stick to rust and compile to WebAssembly and maybe something like this https://github.com/mozilla/rust-android-gradle for mobile.

Ah nevermind, I just read that you already do so.


The bot owner is the user who is archiving the channel

Subject of the issue
The owner of the bot, the one who reinstalled it, is the one that is archiving the incident channel in Slack.

Steps to reproduce
Using the command /hellper_close in any incident.
Expected behaviour
We have two options, that should be discussed:

The bot user archives the channel
OR
The user that called the close command archives the channel

Actual behaviour
The owner of the bot is archiving the channel.
Possible Solution
We need to revise the scopes of our Oauth in the Slack App, one of them should permit the bot to archive a channel as itself, or as the user that called him.

Fixed by using OAuth Token.


Patch value with Diff

This is a placeholder to gather thoughts but the initial idea is to:
Once #78 is implemented, we can add a method to the Schema[A] trait:
trait Schema[A] { self =>
  def patch(diff: Diff): Either[String, A => Either[String,A]]
}

which returns a Left(error message) if the diff cannot be applied to the schema structure (e.g. if I try to apply a Diff.Number to a Schema.Record and otherwise returns a function which will apply the specified diff to value of type A (possibly failing).
And associated syntax
implicit class DiffOps[A: Schema](a: A) {
   def patch(diff): Either[String,A] = Schema[A].patch(diff).flatMap(_.apply(a))
}

Laws:

Given a: A then a.patch(Diff.Identical) == Right(a)
Given a1: A and a2: A we should have a1.patch(a1.diff(a2)) == Right(a2)
Given a1: A and a2: A we should have

val diff1 = a1.diff(a2)
val diff2 = a2.diff(a1)
a1.patch(diff1).flatMap(_.diff(diff2)) == Right(a1)

Some questions:

Should the patching be able to fail if the the structure is correct. That is, should the signature be def patch(diff: Diff): Either[String, A => A] instead?

I would like to give this one a try.
The 2 proposed signatures are:
def patch(diff: Diff): Either[String, A => Either[String,A]]
and
def patch(diff: Diff): Either[String, A => A]
If we have a type A when can the function fail going from A to A?
Not sure how to deal with String...  For example:
val diff1 = "abc" diffEach "xyaz"
which produces: Myers(Chunk(Insert(x),Insert(y),Keep(a),Insert(z),Delete(b),Delete(c)))
How do you apply diff1 to some other random String like "hello"?
Or should this fail when one of the steps (Insert, Keep or Delete) do not match and are not doable?
Any other thoughts or guidance would be appreciated as I am new to the Zio community.

Thanks!
The way I was thinking about it is that the patch is intended to be applied to the original value so applying to completely different value creates a "merge conflict" which would fail.
In general, I think the main guideline for this one is that the patch itself should be pure data so we want to make it operation based and serializable. The string diff is already operation based but other diffs are not explicitly encoded that way currently. As an example, check out how we encode (and then apply) migrations in zio.schema.ast.Migration.

Closed by https://github.com/zio/zio-schema/pull/132


felix/bpf: fix for WG no longer uses failsafes

Previously, WG traffic was allowed by patching failsafe ports. It was broken by https://github.com/projectcalico/calico/pull/6250
We allow wireguard traffic between our nodes even if a user accidentally
block the traffic by a HEP policy.
We do that only if there was an actuall DENY by a bad policy to save
ourselves constantly testing whether to let it pass or not at the fast
path. The test requires 2 RT lookups.
When WG traffic is blocked by a bad policy, we allow it and only then
create conntrack entry to allow subsequent packets. Those entries are
kept alive by constant traffic (most of the time) or recreated after
they expire after policy is applied again.
Description

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

ebpf: WG traffic is allowed by the HEP programs in case of a conflicting policy.

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

/merge-when-ready

OK, I will merge the pull request when it's ready, leave the commits as is when I merge it, and leave the branch after I've merged it.


[bug] The sessionId in Session is too long and truncated by DB field's 65535 limit

https://github.com/casdoor/casdoor/blob/925679142021a65fecceab94ee2868fe4988f0f2/object/session.go#L28-L35
In MySQL, the sessionId field is text type and has 65535 as limit. However, if there are too many strings in this slice, the DB will truncate it and it's not valid JSON any more. We should limit the slice element count to 100 for example.

@leo220yuyaodog

Fixed by: https://github.com/casdoor/casdoor/commit/c9b990a3194d3073061f9b9cb404c01f14f82e68


tslib should follow standards for ESM/CJS detection

tslib's package.json has a few issues RE detecting whether it's CJS or ESM. Due to lack of standards support, it has to be special-cased in a number of places, eg jest-preset-angular. Workaround like this are needed in any projects using jest, typescript, and tslib.
The following changes to the package.json would fix things - happy to provide a PR if it will be considered:

There is no "type": "module" field, which tells node code that .js files support ESM
The ESM tslib (currently "tslib.es6.js") should have an .mjs file extension
The CJS tslib (currently `"tslib.js") should have a .cjs file extension.

To be clear, just the type field would help, but I think it would be preferable to complete all 3 at once.

What I experienced was: If using esbuild or swc to compile tslib.es6.js, Jest cannot process the compiled output. However, if I use TypeScript API to compile tslib.es6.js, Jest is happy with the output.

So are there any workarounds right now? I'm an indirect consumer of tslib, does anybody have a working npm package I can enforce using instead?

I tried changing tslib.js so it exports a plain object with the functions in there including __decorate, and Jest still doesn't work, same error 😓

as a workaround I made https://github.com/StratoKit/tslib

Hi @wmertens - there are 2 fixes I've used:

Explicitly link tslib.es6.js in your jest.config.js moduleNameMapper section, eg:

  moduleNameMapper: {
    tslib: 'tslib/tslib.es6.js',
  },

This is hard-coded in the jest-preset-angular b/c the ability to find tslib ES6 in jest is broken (thus this issue).

Create a copy of tslib.es6.js and rename it to tslib.mjs - jest (and node) always treat .mjs files as ESM. Then put it in a place, or update webpack config, so that it's linked instead of node_modules/tslib.

@johncrim when I try option 1. it complains that node_modules isn't transpiled, and I'd prefer to keep it that way, unless there's a way to tell jest to only transpile tslib?
Option 2 is a variation of the package changes I made, so basically the same maintenance burden.

What I experienced was: If using esbuild or swc to compile tslib.es6.js, Jest cannot process the compiled output. However, if I use TypeScript API to compile tslib.es6.js, Jest is happy with the output.
Possibly related to #161 (comment)

So are there any workarounds right now? I'm an indirect consumer of tslib, does anybody have a working npm package I can enforce using instead?

In case it matters to anyone, I've also posted most of what's written below to https://github.com/evanw/esbuild/issues/2296, as that issue is also quite related in my view.
My main project uses just tsc and esbuild, with no further tooling.  (So, no Jest.)  We recently found a way forward in ES5:

https://github.com/keymanapp/keyman/pull/8904

It's probably a bit hacky, but this relies on two details:

esbuild version 0.15.16 and above now support alias options in their build configuration parameters.  With this, we can alias tslib imports to a custom package that wraps the ES5-based tslib.js file found in the tslib distribution.  esbuild can't handle the object-destructuring modules/index.js syntax in ES5 mode, as it requires ES6 - and that's what esbuild picks up when doing module-resolution.
To ensure that we don't reproduce the same problem, the wrapping package's tsconfig.json uses "Node" module-resolution mode, not "Node16" or higher - we wish to ignore import maps.  This instead connects us to tslib's package.json main entry, which is the CommonJS import leading to the pure-ES5 tslib.js, which esbuild can handle.

On second thought, it may be possible to add a resolution-phase plugin to do the redirect - the issue is about which tslib script the import resolves to, after all.  But, what we have is working now, "if it's not broke, don't fix it", and it provides a decent site to add the plugins added by the next entry.
For further enhancements, we've also developed this one:

https://github.com/keymanapp/keyman/pull/8964

As is, the wrapping package approach unfortunately does not work well with esbuild treeshaking... but we can utilize esbuild's plugin architecture and the patterns in the backing tslib script to do it ourselves without too much issue.  This requires two plugins and an extra build pass for the first:

First phase:  Sets write: false, scans for all tslib helper function names used by the codebase, then determines which ones can be safely tree-shaken.  An 'ignore' pattern may also be set - we use this to avoid picking up names from a file defining embedded worker code.
Second phase:  now that we know which tslib helpers can be eliminated, the second plugin looks for tslib.js and erases the relevant method definitions and the exporter statements.


chore: publish to npm with release-please

Checklist

[ ] npm install && npm run lint && npm test passes
[ ] tests are included
[ ] documentation is changed or added
[x] commit message follows commit guidelines

Description of change
Add npm publish step after release-release.

NPM_TOKEN repo secret is created.


Uncaught TypeError: Cannot read property 'spinnerDialog' of undefined

Hi ,
I want to add your plugin but i am getting this error "Uncaught TypeError: Cannot read property 'spinnerDialog' of undefined",please give me any suggestion sort out this one.
Thanks
Shivam

Ha ha! I see nobody wants to help you. Friend I catch same error

I understood why it happened so. You need call function so:
window.SpinnerDialog.show("bla-bla-bla", "Confirm");
but it exists another problem. I can not see this "dialog", it are not reflected


NDArrayUtil implementation to convert matrices is wrong

Issue Description
NDArrayUtil.toNDArray(int[][] nums) implementation is wrong and give bad dimensions. The only place where this methos is called in deeplearning4j, datavec and nd4j is in ImageLoader.asMatrix method. To reproduce an issue, run MnistImagePipelineExample with these modifications before initializing the record reader:
Configuration conf = new Configuration();
conf.setBoolean(RecordReader.APPEND_LABEL, true);
conf.set("org.datavec.api.records.reader.RecordReader.imageloader", "imageio");
recordReader.initialize(conf ,train);

The images read will have bad shapes.
Version Information

version: master branch

Thanks for the fix!


Make it so _Quizzes repositories are optional

If course creators don't care about quizzes being private then they should be able have quizzes and connect this repository to Leanpub.
This would make a lot of things simpler so that someone can have just one repository if they want.

YAY! This was very easy to do. Now to document it all!

This was remarkably easy to do! (Hopefully it continues working well)
See the documentation updates on https://github.com/jhudsl/OTTR_Template/wiki/Publishing-on-Leanpub

@tmm211 The one repository option is now the default (so no need to use the _Quizzes repository)
If you want to try it out and tell me what you think, I would love any feedback you have. See updated docs here: https://github.com/jhudsl/OTTR_Template/wiki/Publishing-on-Leanpub

No way! Exciting!! I will try it out!

Tiffany Miller (she/her)
Data Sciences Platform
https://www.broadinstitute.org/data-sciences-platform
Broad Institute of MIT and Harvard
On Wed, Jan 26, 2022 at 7:45 PM Candace Savonen @.***>
wrote:

@tmm211 https://github.com/tmm211 The one repository option is now the
default (so no need to use the _Quizzes repository)
If you want to try it out and tell me what you think, I would love any
feedback you have. See updated docs here:
https://github.com/jhudsl/OTTR_Template/wiki/Publishing-on-Leanpub
—
Reply to this email directly, view it on GitHub
https://github.com/jhudsl/OTTR_Template/issues/417#issuecomment-1022737998,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGZ6N4GKEYWAGXJVGRSAXDUYCIS5ANCNFSM5MYX5KJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

This is done for now (unless/until we encounter any problems with it).


default amountToUnits decimals to 18, FRE consumerMarketSwapFee witho…

…ut basetoken
Fixes # .
Changes proposed in this PR:

FRE calcBaseInGivenOutDT method with consumeMarketSwapFee non-token specific, since is % of defined price
amountToUnits with default token decimals to 0

Don't change amountToUnits function, just use Web3.utils.toWei

amountToUnits with default token decimals to 0

the code does not reflect this as decimals is immediately set to 18 as soon as decimals is '0'. Commented on respective code line above.
And then what Mihai said, I don't have any context on this PR as no issue to be found describing what it tried to solve so no amountToUnits usage as it looks like

Yes it's a simple solution somthing like
.calcBaseInGivenOutDT(
        exchangeId,
        await this.amountToUnits(fixedRateExchange.datatoken, datatokenAmount),
        Web3.utils.toWei(consumeMarketFee)
      )

@mihaisc ya, that's the right way to go, earlier got blinded by the existing conversion method

Test integration fails because of compute timeoute issue


About page added aith responsiveness

This pull request adds an about page to the website. The about page provides a brief overview of the project, its goals, and how to contribute.

Please read this:
Important Update