Please add an option to avoid creating short redirects

While I can see the value on them I don't really care about them, and all my redirection is done at Apache level, so I'd rather not have them to avoid filling in a table that I don't care about.

Hello,
shortened URLs are useful for people who post their articles on services like Twitter and who don't want to use third party URL shorteners. They are also useful for Typo users who post lots of links on Twitter and want their own URL shortener (like I do).
Shortener only adds one like per published post. There should be less than a thousand lines in the redirects table, which is not a big deal. Adding an option to disable this would make the code and interface more complicated, and raise the risk of bugs and errors. If you really want to get rid of redirects creation, I'd suggest you either comment the call to set_shortened_url in app/controllers/admin/content_controller or empty the redirects table from time to time.


Add tpl approach for plugin Goreleaser config

Description
Changes proposed in this pull request:

Add tpl approach for plugin Goreleaser config

Testing

CI

template will be added by this PR:  https://github.com/kubeshop/botkube/pull/1165
Archives can be added later.


[FLINK-33985][runtime] Support obtain all partitions existing in cluster through ShuffleMaster.

What is the purpose of the change
[FLINK-33985][runtime] Support obtain all partitions existing in cluster through ShuffleMaster.
Brief change log

Support obtain all partitions existing in cluster through ShuffleMaster.
TaskExecutor will not release partitions immediately when JM Failover enabled.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@zhuzhurk Thanks for your review, I've updated this pr accordingly, PTAL.

@zhuzhurk Fixed, PTAL~


Is debug.namespace a public api ?

Hi :)
Recently, I've noticed that most of my modules are using debug module.
const debug = require('debug')('moduname:xxx');

So I am wondering if I can reuse the debug.namespace in the same module ?
For example, maybe in a production environment, I need to log.info() or log.warn() something, can I write it like the following? So, I can keep debug namespace more consistent with logging information.
var libname = debug.namespace;
log.info(`${libname} underlying socket closed, reconnecting...`);

I know I can do it now. However, I am wondering if the namespace property will be changed in the future, because I see no documentation regarding this debug.namespace property.
Thank you for your answer and this awesome tool. :)

@benpptung not sure if this will change


[Material] ThemeData should use InkSparkle for splashFactory when useMaterial3 is true

InkSparkle should only be used as the splashFactory in ThemeData when useMaterial3 is true, the platform is Android, and the compilation mode is not web, i.e. kIsWeb is false.

I already asked this in the pr, but it was ignored, so maybe its better to ask questions in the dedicated issues:
I'm curious: is the sparkle effect not part of material design, or why is it only applied on Android?


Refactor the 'advanced types' page

Extract the very specific scenario documented on that page into a sub-page
Write an introduction of Prisma's generated types  - only basic example available now: https://www.prisma.io/docs/concepts/components/prisma-client/crud#create-a-single-record-using-generated-types

Related: #1265

Im going to go ahead and implement #1265 in this issue too.


Add kills and deaths stats

You can /toggle stats to toggle between sidebar, actionbar, and bossbar(I personally don't like this one).
A pic of how all of 3 at once would look like:

Note: don't expect this to be a feature of cardinal right away, for the time being it's here just to be tested and to see players' opinions

Woohoo, /stats is back!!

On 8 Jun 2016, at 12:13 pm, Pablete1234<EMAIL_ADDRESS>wrote:
You can /toggle stats to toggle between sidebar, actionbar, and bossbar(I personally don't like this one).
A pic of how all of 3 at once would look like:
Note: don't expect this to be a feature of cardinal right away, for the time being it's here just to be tested and to see players' opinions
You can view, comment on, or merge this pull request online at:
https://github.com/twizmwazin/CardinalPGM/pull/1024
Commit Summary
Add kills and deaths stats
File Changes
M src/main/java/in/twizmwaz/cardinal/chat/ChatConstant.java (1)
M src/main/java/in/twizmwaz/cardinal/command/SettingCommands.java (8)
A src/main/java/in/twizmwaz/cardinal/event/PlayerSettingChangeEvent.java (46)
D src/main/java/in/twizmwaz/cardinal/event/PlayerVisibilityChangeEvent.java (24)
M src/main/java/in/twizmwaz/cardinal/module/ModuleFactory.java (4)
M src/main/java/in/twizmwaz/cardinal/module/modules/scoreboard/ScoreboardModule.java (2)
A src/main/java/in/twizmwaz/cardinal/module/modules/stats/Stats.java (219)
A src/main/java/in/twizmwaz/cardinal/module/modules/stats/StatsBuilder.java     (14)
M src/main/java/in/twizmwaz/cardinal/module/modules/visibility/Visibility.java (5)
M src/main/java/in/twizmwaz/cardinal/util/PacketUtils.java (11)
M src/main/resources/lang/en.xml (1)
M src/main/resources/settings.yml (12)
Patch Links:
https://github.com/twizmwazin/CardinalPGM/pull/1024.patch
https://github.com/twizmwazin/CardinalPGM/pull/1024.diff
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@SungMatt but this time it's 200% better than a command :P

The group that I play with would be very happy to see this come back, since it disappeared a year ago!

On 8 Jun 2016, at 1:04 pm, Pablete1234<EMAIL_ADDRESS>wrote:
@SungMatt but this time it's 200% better than a command :P
―
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@twizmwazin fixed all the hashmaps

Great idea, love the implementation :tada:

@iPGz how would it not be a great idea if it was yours!? nice to know you like the way i implemented it :P

Tested with myself and a friend of mine, sidebar is working as intended! Will be playtesting more today, as I get more people.

@SungMatt i'll update it to 1.10

@Pablete1234 Tested it, there appears to be an issue in TDM where scores do not increment properly. Upon Team 1 player death via being killed by a Team 2 player, Team 2's score increments by 1 (Correct behaviour) but Team 1's score decreases by 1 (Incorrect behaviour)

merged


fix(encapsulate): ignore type not defined

Sorry for skipping issue.
Description
encapsulated schema fails to introspect if applyTo.{type} is true but {type} does not exist.
Type of change
Please delete options that are not relevant.

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

Screenshots/Sandbox (if appropriate/relevant):

How Has This Been Tested?
I had this problem in production and fixed it through patching node_modules. Commiting here a failing test first.
Test Environment:

OS: MacOS 11.2.3
@graphql-mesh/*: master
NodeJS: v14.16.0

Checklist:

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

Thank you @cometkim ! That's great <3

I just realized same patch should be applied to transformRequest() and transformResult(). I wanna fix it right away, but I don't know how to get the schema from there. @ardatan any idea?


Split header into 7Kb of data

This generates several X-Cattle-Smbios-NUMER headers when sending the
data to the operator in order to limit the max size of each individual
header to 7Kb. By default nginx is configured with a max of 8Kb header
and will return a 400 error if the headers excess that size.
As we currently use a GET+Websocket to go trougth the challenge/response
of TPM, this is the easiest implementation to workaround the issue.
Signed-off-by: Itxaka<EMAIL_ADDRESS>

Codecov Report

Merging #133 (0b1702b) into main (4c75b5c) will decrease coverage by 0.63%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #133      +/-   ##
==========================================
- Coverage   29.16%   28.53%   -0.64%
==========================================
  Files           5        5
  Lines         360      368       +8
==========================================
  Hits          105      105
- Misses        251      259       +8
  Partials        4        4

Impacted Files
Coverage Δ

pkg/server/register.go
12.76% <0.00%> (-0.57%)
:arrow_down:

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


Extra latency in comparison to Web Audio Api

I had a feeling that tone.js adds extra latency on top of the standard web audio api, so I wrote a small app to test it:
https://yonih.github.io/latency-test/
You can find the code here.
At least in my setup (Mac M1, Chrome browser) the difference between plain Web Audio Api and Tone.js is very audiable and unacceptable.
I recorded myself playing with the test app, so I can look at the recording's waveform and measure exactly how much time it took between the keystroke sound and the time that the sample sound came out of the speakers. That is, the latency.
Here is the recording.
And here are the results:
Web audio api: 76ms 61ms 57ms
Tone.js: 157ms. 162ms, 160ms
Am I missing something? Did I write the tone.js code wrong? Can I expect tone.js to respond as quickly as plain web audio api?

Looking at the results, it seems that the difference is about 100ms, which is exactly the default lookahead.
I did set the the lookahead to 0 though, and also called Tone.immediate().
Maybe I have a bug around that area?

You're right that the cause of the latency is lookAhead. But in your code you're creating your Players before instantiating a new audioContext with a lower lookAhead. That results in those players still using the older default Tone context with the higher lookAhead. You actually don't need to create a new context if all you want is to lower the lookAhead, that can be changed dynamically at any time Tone.getContext(). lookAhead = 0.

Yes, that was it 🎉🎉🎉
Now the latency I'm getting is similar to plain Web Audio Api.
Thanks 😄


Documentation or Tutorial on Implementing the t-J Model

Dear QuSpin Team,
I am currently trying to implement the t-J model using QuSpin, as described by the following Hamiltonian:

I tried to implement it using the following code:
import sys, os
import numpy as np
from quspin.basis import spinful_fermion_basis_general
from quspin.operators import hamiltonian

os.environ['KMP_DUPLICATE_LIB_OK'] = 'True'
os.environ['OMP_NUM_THREADS'] = '1'
os.environ['MKL_NUM_THREADS'] = '1'

Lx, Ly = 4, 4
N_2d = Lx * Ly
t = 1.0
J = 0.1

s = np.arange(N_2d)
x = s % Lx
y = s // Lx
T_x = (x + 1) % Lx + Lx * y
T_y = x + Lx * ((y + 1) % Ly)

basis_2d = spinful_fermion_basis_general(N_2d, Nf=(2, 2), double_occupancy=False)

hopping_left = [[-t, i, T_x[i]] for i in range(N_2d)] + [[-t, i, T_y[i]] for i in range(N_2d)]
hopping_right = [[+t, i, T_x[i]] for i in range(N_2d)] + [[+t, i, T_y[i]] for i in range(N_2d)]

J_XX = [[ J, i, T_x[i]] for i in range(N_2d)] + [[ J, i, T_y[i]] for i in range(N_2d)]
J_YY = [[J, i, T_x[i]] for i in range(N_2d)] + [[ J, i, T_y[i]] for i in range(N_2d)]
J_ZZ = [[J, i, T_x[i]] for i in range(N_2d)] + [[J, i, T_y[i]] for i in range(N_2d)]

static = [["+-|", hopping_left],
          ["-+|", hopping_right],
          ["|+-", hopping_left],
          ["|-+", hopping_right],
          ["xx|", J_XX],
          ["yy|", J_YY],
          ["zz|", J_ZZ]]

H = hamiltonian(static, [], basis=basis_2d, dtype=np.complex128)

E_GS, psi_GS = H.eigsh(k=4, which='SA')

print('E_GS', E_GS)

However, when comparing my results to other exact diagonalization (ED) codes, I get different results for the ground state energy. I'm wondering if I'm implementing something incorrectly or missing a step in setting up the model.
Is there any available documentation or tutorial that explains how to set up and implement the t-J model correctly in QuSpin? Any guidance or pointers would be greatly appreciated!
Thank you for your help!

please post this Q in the discussion forum, its place is not here ;)
https://github.com/QuSpin/QuSpin/discussions
the closest to t-J I know of is this test:
https://github.com/QuSpin/QuSpin/blob/master/test/test_sq_lat_Heis_double_occupancy.py


Revert update of codecov-action from v4 to v3

The v4 update for the codecov GH Action was prematurely done by https://github.com/dependabot. The tag has been removed (if it ever existed) and v4 is still (only) in beta.

Fixed by #406


Next row down from deleted row becomes "swiped"

I have a delete button that removes a row item based on index in the state array. After I swipe and delete, the next item in the list shows as "swiped", showing its Delete button. I am looking to keep the next item from showing swiped state after deleting the item above it.

I have a delete button that removes a row item based on index in the state array. After I swipe and delete, the next item in the list shows as "swiped", showing its Delete button. I am looking to keep the next item from showing swiped state after deleting the item above it.

Having the same issue.  did you found the fix .??

I had the same issue and ended up fixing it by passing the closeRow function into my hiddenItem component, and then calling it before deleting the item.
<SwipeRow
    rightOpenValue={-80}
    disableRightSwipe
    disableLeftSwipe={item.request !== true}>
            <HiddenItem data={item} closeRow={() => rowMap[item._id].closeRow()} />
            <TransactionListItem
              item={item}
            />
</SwipeRow>

and then
const HiddenItem = ({ data, closeRow }) => (
  <View style={styles.rowBack}>
    <View style={{...}}>
      <TouchableOpacity
        onPress={() => {
          closeRow();
          Alert.alert("Delete Request", "Are you sure?", [
            {
              text: "Confirm",
              onPress: () => {
                deleteTransaction(data._id);
              }
            },
            { text: "Cancel", style: "cancel" }
          ]);
        }}
        style={{
          alignItems: "center",
          justifyContent: "center",
          height: "100%",
        }}
      >
        <Text style={[styles.backTextWhite]}>Decline</Text>
      </TouchableOpacity>
    </View>
  </View>
);```

Hey @dontpressthat I'll admit I tend to manually close a row as it's being deleted so there's never a chance that I run into this issue. However I just ran my example and removed this line: https://github.com/jemise111/react-native-swipe-list-view/blob/master/SwipeListExample/examples/basic.js#L26 and I'm still not seeing the behavior you're talking about. Can you post some code to show how you're implementing it? Thanks!
(apologies for the delay in responding)

Closing as inactive

This fixed my issue.
https://github.com/jemise111/react-native-swipe-list-view/issues/455#issuecomment-618104389


⚠ Bofrost.at has degraded performance

In 3458d01, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1782 ms

Resolved: Bofrost.at performance has improved in 3760e6e after 11 minutes.


linter takes issue with the header names

A current header is for example:
VOXBLOX_CORE_ESDF_MAP_H_
and linter will prevent commits unless it is changed to:
CORE_ESDF_MAP_H_

Hmm this is incorrect, and a problem with the linter. @mfehr ? Can we either turn this feature off or fix it?

I'll look into it tomorrow. For now just fix whatever makes sense, then use the -n switch

This should be fixed #101


docs: privacy strategy document

I think there was a problem during the rebase, some commits are now duplicated between main and your branch.

I have asked for review - @SdgJlbl , can we still add co-contributors?

I have asked for review - @SdgJlbl , can we still add co-contributors?

Yes, we will add the co-authors on the squash commit during the merge I think


Remove bay.yaml

do we want to remove it entirely ? i was thinking this might still be a good way ( the only way ? ) to sync the two clusters..

Yeah, this has to be removed for bay to work properly..this is a leftover after old PR. I'll merge it now.


remove UtilsBuilder load, use torch (un)flatten ops

Torch (un)flatten currently has no performance issues, we have tested it on the CUDA device. Skip compiling unnecessary util.
It will not depend on the deepspeed compile process for many training processes. This may also be friendly to other device integration.

Thank you @inkcherry! This is a great reminder, I see that torch has updated their (un)flatten implementations https://github.com/pytorch/pytorch/pull/58006. I also did a benchmark just now and see no difference between versions.
In this PR can you also remove: op_builder/utils.py


Add deformation models to resources

Input files for IERS2010 Earth deformation model should be added to Tudat resources, and the associated unit test re-enabled

Done!


Bug Report: link validation doesn't run when link and screenshot are both provided

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Resolved via https://github.com/ssilve1989/ulti-project/commit/c36502118622a4a182c656b838769f17ba1ebc73


WebApp:az webapp config ssl import not importing certificate

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug
Im running few hundreds of az webapp config ssl import to import certificate from keyvault to webapp, and then bind them to the custom url
Oftentimes after uploading the certificate successfully, I'm getting
uploaded cert: 7BC11C3A2D6C5625E9E4FF6C3834055F1675DF4C
Certificate for thumbprint '7BC11C3A2D6C5625E9E4FF6C3834055F1675DF4C' not found.

when running
az webapp config ssl bind --certificate-thumbprint $thumbprint --ssl-type SNI ids  $webapp_id
First line is from script, getting the thumbprint of certificate that was sucessfully uploaded. Second one is from az webapp config ssl bind
To Reproduce
Attach few certificates to webapp from keyvault, and then bind them to custom domain.
Expected behavior
Certificate installed succesfully.
Environment summary

Additional context
Sometimes certificates are present on azure webapp website and can be bind to url manually. Sometimes they need to be connected from keyvault, throw out an error, and only then they can be connected. This works really wonky.

webapp

@kaletkamk - Could you please provide the az version, commands you are using and  --debug log to investigate further.

{
"azure-cli": "2.14.0",
"azure-cli-core": "2.14.0",
"azure-cli-telemetry": "1.0.6",
"extensions": {
"azure-devops": "0.18.0"
}
}
I believe this is related to the webapps themselves - sometimes you can't bind pre-existing ssl certificate without going to the certificates tab first.

Is there any secure channel I can share the deug log with you? It contains a lot of sensitive information, and I'm not sure if I'll be able to purge it all out.

@kaletkamk -- Is this still an issue? Was wondering if this was related to this: https://github.com/Azure/azure-cli/issues/21133 Currently I'm not able to repro your issue.


style :  optimized code & UI

Description
This PR refines the user interface and functionality of the SheSharp React.js web application. The following improvements have been implemented:

UI Enhancements: The user interface has been updated to provide a more modern and visually appealing look. The changes include a redesigned card layout and an attractive title to create a visually engaging experience for users.

Table Styling: The data display table has been styled for better readability. Thicker lines have been added between table cells to provide clear separation.

Comments and Documentation: Comments within the code have been improved for clarity and maintainability. Additionally, a descriptive punchline title has been added to the card for an engaging user experience.

Improved Fetching: The Fetch API has been integrated to fetch data from a public API (JSONPlaceholder) when the "Fetch Data" button is clicked. This provides users with the ability to experience data retrieval in real-time, enhancing the application's interactivity.

These enhancements collectively create a more engaging and user-friendly experience, making the SheSharp React.js application more modern and user-centered. The changes aim to elevate the overall usability and aesthetic of the application.

Hi @ZeeshanMukhtar1 , thanks for your interest in contributing. Currently we are focusing on assigning issues and accepting PRs from our community members and underrepresented genders. You can learn more about our organization on the SheSharp Website.

@ZeeshanMukhtar1 Thanks again for your interest. One thing is that we want to keep this repo true to the original design - which is not fetching from a public API, but using JSON.
We have assigned the header and footer issues to other community members. Would you be open to contributing a specific component for the UI or something else?
I don't want to totally shoot down your PR, I'm sorry for that. We are still figuring out how to best focus our efforts on our community members / underrepresented genders, but it's a process. We are open to contributions from others, if you have insight or suggestions to make it more collaborative we would love to hear from you.
If you want to continue the discussion, I invite you to join our Slack. We have an #open-source channel as well as a #devs channel.

@mariahlaqua Thank you for your response and clarification. I appreciate your commitment to maintaining the original design. I'm open to contributing in any way that aligns with your project's goals. If you have specific suggestions or tasks, please let me know. I'm here to help and support your efforts to make the project more collaborative.

Thank you for understanding @ZeeshanMukhtar1 , we're thinking on it today and I'll get back to you. 👍 Super appreciate your patience!


Startup error when tried to run on ios

I followed the setup instructions and tried to run on ios and I got the following error.
2016-01-19 22:29:06.967 [warn][tid:com.facebook.React.JavaScript] Invariant Violation: Application snowflake has not been registered. This is either due to a require() error during initialization or failure to call AppRegistry.registerComponent.

Did you 'npm install' and be sure you are running 'npm start' in snowflake
dir.
On Jan 20, 2016 12:35 AM, "muyen"<EMAIL_ADDRESS>wrote:

I followed the setup instructions and tried to run on ios and I got the
following error
2016-01-19 22:29:06967 [warn][tid:comfacebookReactJavaScript] Invariant
Violation: Application snowflake has not been registered This is either due
to a require() error during initialization or failure to call
AppRegistryregisterComponent
—
Reply to this email directly or view it on GitHub
https://github.com/bartonhammond/snowflake/issues/43.

Hi, thank you for your response. It worked after I issued 'npm start' command.


Separate chareges

Do you plan to add the separate charges option?

Please fill in the template when opening feature requests: https://github.com/stripe/stripe-firebase-extensions/issues/new?assignees=&labels=&template=2.Feature_request.md


Form Validation makes unwanted behavaior when used with list view

Steps to Reproduce
Enter following code and click on top and bottom validate button for the same form it showing different validation result in console

code sample
// Copyright (c) 2019, the Dart project authors.  Please see the AUTHORS file
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.

import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      debugShowCheckedModeBanner: false,
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
var formKey = GlobalKey<FormState>();
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Form(
        key: formKey,
        child: ListView(
          children: <Widget>[
                RaisedButton(
         onPressed: (){
           if (formKey.currentState.validate()){
             print("validated");
           }else{
             print("invalid");
           }
        },
        child: Text("Validate"),
      ),
            for(int i =0;i<10;i++)
           TextFormField(
             key:Key("f$i"),
            validator: (value) {
              if (value.isEmpty) {
                return 'Please enter some text';
              }
              return null;
            },
            ),
            for(int i =0;i<20;i++)
           TextFormField(
             key:Key("s$i"),
            validator: (value) {
              if (value.isEmpty) {
                return 'Please enter some text';
              }
              return null;
            },
             initialValue:i.toString(),
          ),
            SizedBox(height: 10,),
            RaisedButton(
         onPressed: (){
           if (formKey.currentState.validate()){
             print("validated");
           }else{
             print("invalid");
           }
        },
        child: Text("Validate"),
      ),
          ],
        ),
      ),

    );
  }
}

Hi @PratikMMohite
Thanks for filing the issue, it's definitely strangely but looks very, looks like a duplicate of https://github.com/flutter/flutter/issues/56159.
If not, can you please provide more details?

flutter doctor -v
[✓] Flutter (Channel beta, 1.25.0-8.1.pre, on macOS 11.1 20C69 darwin-x64, locale en-GB)
    • Flutter version 1.25.0-8.1.pre at /Users/tahatesser/Code/flutter_beta
    • Framework revision 8f89f6505b (2 weeks ago), 2020-12-15 15:07:52 -0800
    • Engine revision 92ae191c17
    • Dart version 2.12.0 (build 2.12.0-133.2.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.0

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (2 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 10 (API 29)
    • Chrome (web)     • chrome           • web-javascript • Google Chrome 87.0.4280.88

• No issues found!

Ya you can close it

Hi @PratikMMohite
Given your last message I feel safe to close this issue, if you disagree please write in the comments and I will reopen it.
Thank you


Authentication in react-admin

I tried to implement authentication with prisma with no success.
Do you have a working example with auth?

If you use it with a prisma token, you will receive an error message, and redirection to the login page will not occur (ra failed)
I think it is because ra trying to init but getting this error (at this point, we does't have token), so he can't redirect to login page.
Uncaught (in promise) Error: GraphQL error: Your token is invalid. It might have expired or you might be using a token from a different project.
    at new ApolloError (ApolloError.js:33)
    at eval (QueryManager.js:360)
    at eval (QueryManager.js:808)
    at Array.forEach (<anonymous>)
    at eval (QueryManager.js:807)
    at Map.forEach (<anonymous>)
    at QueryManager.broadcastQueries (QueryManager.js:803)
    at Object.next (QueryManager.js:887)
    at notifySubscription (Observable.js:152)
    at onNotify (Observable.js:196)
    at SubscriptionObserver.next (Observable.js:248)
    at eval (bundle.esm.js:98)
    at Array.forEach (<anonymous>)
    at Object.next (bundle.esm.js:97)
    at notifySubscription (Observable.js:152)
    at onNotify (Observable.js:196)
    at SubscriptionObserver.next (Observable.js:248)
    at notifySubscription (Observable.js:152)
    at onNotify (Observable.js:196)
    at SubscriptionObserver.next (Observable.js:248)
    at eval (bundle.esm.js:175)

My App.js
const authLink = setContext((_, { headers }) => {
    // get the authentication token from local storage if it exists
    const token = localStorage.getItem('token');
    // return the headers to the context so httpLink can read them
    return {
      headers: {
        ...headers,
        authorization: token ? `Bearer ${token}` : "",
      }
    }
  });
const client = new ApolloClient({
    link: authLink.concat(httpLink),
    cache: new InMemoryCache()
  });

My authProvider.js
import { AUTH_LOGIN, AUTH_LOGOUT, AUTH_ERROR, AUTH_GET_PERMISSIONS, AUTH_CHECK } from 'react-admin';
// import decodeJwt from 'jwt-decode';

export default (type, params) => {
    if (type === AUTH_LOGIN) {
        const { token } = params;
        localStorage.setItem('token', token);
    }
    if (type === AUTH_LOGOUT) {
        localStorage.removeItem('role');
        return Promise.resolve();
    }
    if (type === AUTH_ERROR) {
        debugger
    }
    if (type === AUTH_CHECK) {
        return localStorage.getItem('token') ? Promise.resolve() : Promise.reject();
    }
    return Promise.reject('Unknown method');
};

(I also use a custom login page where I just get the token from user)
How we can resolve this issue?

+1

Doc should be improved to showcase authentication example, it's not that straightforward.

Here's how I did it.
const httpLink = createHttpLink({
      uri: 'https://api-euwest.graphcms.com/v1/ck73ixhlv09yt01dv2ga1bkbp/master',
    });

    const authLink = setContext((_, { headers }) => {
      // get the authentication token from local storage if it exists
      const token = '';
      // return the headers to the context so httpLink can read them
      return {
        headers: {
          ...headers,
          authorization: token ? `Bearer ${token}` : '',
        },
      };
    });
    const client = new ApolloClient({
      link: authLink.concat(httpLink),
      cache: new InMemoryCache(),
    });

    buildGraphQLProvider({
      client,
    }).then((dataProvider) => this.setState({ dataProvider }));


Missing documentation for IntegralFunction and BatchIntegralFunction

The tutorial in the documentation for Intergral.jl recommends the use of the “nout” and “batch” keywords which are deprecated. When using these, a warning is given and it is recommended to use IntegralFunction and BatchIntegralFunction instead. There is, however, no documentation on how to use Integral function and BatchIntegralFunction.
I would recommend that the use of “nout” and “batch” be removed from the tutorial in the documentation of Integral.jl and that a example for how to use IntegralFunction and BatchIntegralFunction is added instead.
This is the first time I report an Issue likt this so I hope I did it correctly, if I did not please let me know. Below a link to discourse on the subject on the Julia forum. https://discourse.julialang.org/t/is-there-any-documentation-on-integralfunctions/108970

Generally the docstrings are kept with the types, and those are all separate from the solvers for deps reasons.


Call getCertificatePublisher only when publisher not set

When trying to sign MSIX with a certificate from the certificate store and not from PFX file, we can achieve this by setting our own signtool_options like this:
msix_config:
  display_name: Flutter App
  publisher_display_name: Company Name
  identity_name: company.suite.flutterapp
  msix_version: <IP_ADDRESS>
  logo_path: C:\path\to\logo.png
  capabilities: internetClient, location, microphone, webcam
  signtool_options: /v /fd SHA256 /a /sha1 xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx /tr http://timestamp.digicert.com
  install_certificate: false

However when running msix:create an error will be thrown:
Get-PfxData : The system cannot find the file specified. 0x80070002 (WIN32: 2 ERROR_FILE_NOT_FOUND)
Turns out the code tries to get the Certificate Publisher from a PFX - not set.
This patch mitigates the issue by taking the publisher from the config - if it is set.
msix_config:
  ...
  publisher: CN=Company, O=Company,...

I will try to prepare an alternative patch, that will allow alternative certificate selection method

Hello @zobo
I will think of a solution for this bug, thank you 👍

Yet another suggestion. If the user has a .crt file (only the public part - and thus no certificatePassword set) one could use something like this.
$crt=new-object System.Security.Cryptography.X509Certificates.X509Certificate2("file.crt")
$crt | fl -Property Subject

I think passing only the public part to signtool via /f option is also valid. The system will find the private key alone.
I think the whole thing is important as certificates should not lie around on the file system in PFX format. For EV code signing cert you can't even get the private key. You get a pre-populated certificate hardware token mailed to you, and cloud signing is very much a thing today.
In all cases, you do not have a PFX.
I can provide a patch with these ideas.

Moving related work to https://github.com/YehudaKremer/msix/pull/156.


Version notation fix

Description
If an image is already present in CML (i.e., already imported from replace ISO), it may have  notation with hyphens instead of dots. This small patch fixes the handling of such notation in two places:

Validation of proper image for deployment
Listing of available images

It may be just a base to fix the notation handling in other parts of the code as well, I added support for hyphens in those two places because I don't have access to a fresh CML install to test image import via "setup" command here.
With this patch, I'll be able to deploy the lab with "sdwan-lab deploy 20.13.1" and "sdwan-lab deploy 20-13-1" commands.
Type of Change

[X] Bug Fix
[ ] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)

Hi @WojciechowskiPiotr, thanks for opening a PR. The dotted notation is used in many places around the script. Instead of covering two use cases every time (with dot and hyphen), how about we add the step in setup task to convert all the SD-WAN images with hyphen to dotted format?

I thought about that, but I wonder if it's possible. The image definitions are read-only, but even if this flag is removed, the ID cannot be changed, at least not from the GUI. It makes sense as image ID is referred in node definition. I think that needs further investigation.
I'm open to other solutions because if it's not changed, the only way would be to remove existing SDWAN images and set them up again using setup, which is not convenient either.

@WojciechowskiPiotr I've implemented a converter in setup task on dev-2.0.10 branch. This should verify images and if there are any SD-WAN images using "-", it will convert them to ".". Could you please test it?

Make sure no SD-WAN labs are running on the server.
Install the tool from dev branch: pip install<EMAIL_ADDRESS>Run csdwan setup
Try deploying the lab

Let me know if that works!


Deprecate -LogsOnly and replace with -FileType

Type of Change

[X] Bug fix (non-breaking change, fixes #2590)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

Deprecate -LogsOnly, and replace with -FileType of either All, Data, or Log
Approach

All and Log use existing functionality.  I added a section for Data that shrinks every data file in the database.
Commands to test

Included in the test file.

OUU YESSS! I've been wanting this, thank you. Gonna merge when it passes.

 Thanks for updating the Test too.

Ah, so that's how that works :) .  As an FYI, I used Test-DbaDiskAllocation as and example, so looks like there are a few out there using an old method.  The ones I found in a search are Test-DbaDiskAllocation, Test-DbaPowerPlan, and Test-DbaValidLogin.


Urls api

[X]  Base URLs module with Put and Get endpoints
[x] Move create URL method to UrlsService
[x] Delete endpoint?
[x]  Docs
[ ]  Unit Tests

@achiyahb Hey,
After migrating the whole repo to nx we need to modify this PR.
Do you mind fix the conflicts?


Abstract the "get installation ID for username, then auth" dance

Feature Request
Is your feature request related to a problem? Please describe.
I'd like to make it easier to authenticate as the installation for a user, repo or org when you don't already have the installation ID. Currently, that looks cumbersome:
let octokit = await app.auth()
// This can be `findUserInstallation` or `findOrgInstallation`
const { data: installation } = await octokit.apps.findRepoInstallation({ owner, repo })
octokit = await app.auth(installation.id)

Describe the solution you'd like
Let's let app.auth() take an object (in addition to just the id as a number, or maybe that'd be part of the object, I don't know yet):
const authedAsUserInstallation = await app.auth({ username: 'JasonEtco' })
const authedAsRepoInstallation = await app.auth({ owner: 'JasonEtco', repo: 'top-10-cereal' })
const authedAsOrg = await app.auth({ org: 'github' })

Teachability, Documentation, Adoption, Migration Strategy
This should definitely be documented, as its a pattern that'd be used frequently. Assuming we still let folks pass just the id, this would not introduce any breaking changes.

I agree that the current "auth, find, reauth" pattern feels very awkward. It was definitely one of things that stood out most to me when I developed my first probot app.

This feels very similar to some work being done in octokit proper; see https://github.com/octokit/app.js/pull/1 for that proposal. Right now we're still in the design/beginnings of implementations phase, but feedback welcome!
cc/ @gr2m


[RFC] tools: Add script to generate all test cases in VMs

Add the generate-all-test-cases tool, which generates all defined image
test cases based on provided "distro x arch x image-type" combination
matrix and CLI arguments. Test cases are generated by running the
generate-test-cases tool inside an architecture-specific VM.
The tool requires a qcow2 image to be provided for each requested
architecture. These images are used to create ephemeral VMs used
for test case generation.
There appears to be an issue with entropy on all VMs for other
architecture than the host's one. It makes the generation of the first
test case usually fail due to the go get command failing with a "TLS
handshake timeout" error. There is a workaround to retry the generation
of a test case a maximum of 3 times in this case. In the usual case,
the test case generation passes on the second try. All subsequent test
case generations should complete successfully on the first try.
Add the distro-arch-imagetype-map.json file as the default source of
the "distro x arch x image-type" combination matrix for generating test
cases. Its content is based on the current content of the
test/data/manifests directory.
Update test/README.md to mention the generate-all-test-cases tool.
Add a note, that generate-test-cases must be run on a host with the
same architecture, as the intended test case's.
Needs discussion:

The content of distro-arch-imagetype-map.json and its idea in general.
Whether the tool works also on some else's computer 😇

Thanks for the feedback @teg. I added a note to test/README.md that the tool unfortunately does not work with RHEL images. The tool uses "9p virtio" to share directories from the Host to the Guest but unfortunately the "9p" filesysten is not supported on RHEL. The way that sources are shared with the Guest and the results are retrieved could be definitely reworked to support also RHEL images, but I would not spend more time on this at this moment. The PoC works fine with Fedora 33 images.

Thanks for the feedback @teg. I added a note to test/README.md that the tool unfortunately does not work with RHEL images. The tool uses "9p virtio" to share directories from the Host to the Guest but unfortunately the "9p" filesysten is not supported on RHEL. The way that sources are shared with the Guest and the results are retrieved could be definitely reworked to support also RHEL images, but I would not spend more time on this at this moment. The PoC works fine with Fedora 33 images.

(CI was unhappy but this was because of some Jenkins issues. As this change only code for development, we're safe to merge this.)


SQLAlchemy Error while running guardrails text2sql application

Describe the bug
Installed guardrails-ai, happened with both sqlalchemy versions 1.4.x and 2.x after I updated to try and fix this.
    179 class SqlDocument(Base):
    180     __tablename__ = "documents"
--> 182     id: orm.Mapped[int] = orm.mapped_column(primary_key=True)
    183     page_num: orm.Mapped[int] = orm.mapped_column(
    184         sqlalchemy.Integer, primary_key=True
    185     )
    186     text: orm.Mapped[str] = orm.mapped_column(sqlalchemy.String)

AttributeError: module 'sqlalchemy.orm' has no attribute 'mapped_column'

To Reproduce
Steps to reproduce the behavior:

Simply import from guardrails.document_store import DocumentStoreBase, EphemeralDocumentStore

Expected behavior
No exception
Library version:
Version 0.1.6 (latest)

Will try to fix this when I have a chance

@RamiAwar what's your SQLAlchemy version?
Can you try running:
pip uninstall -y sqlalchemy
pip install sqlalchemy>=2.0.9

and see if that fixes the issue?

Hey Shreya, interestingly, I just cloned the repo and installed it as an editable package and it now works fine.. My IDE also detected the mapping fine.
I have sqlalchemy set to ^2.0.13. Maybe I forgot to restart my kernel after updating sqlalchemy or something like this, I think it's a not an issue anymore, you can close this!
Thanks for the quick response! I'm planning to re-write my Text2SQL app https://dataline.app using guardrails instead of manual prompts, so I'm excited to contribute where I can!


small addition, to make things clear

Before it was:
"Interactive messages" is Slack's fancy way of saying "buttons." In order to enable buttons, under Request URL, add https://my-bot-url/slack/receive, then click save."
Now:
Interactive messages" is Slack's fancy way of saying "buttons." In order to enable buttons, click on the "Interactive Components" tab and under Request URL, add https://my-bot-url/slack/receive, then click save.

Thanks for this! We should go through all the docs and update little slack changes like this!


Error finding key for token

[Enter feedback here]
Sorry if this is the wrong place to ask question.
I follow the instructions to test command bot app, but when I use @mention to send "helloWorld" command to this bot, I
got no reponse, instead , I saw error messages in vscode terminal:
Error finding key for token. Available keys: -----BEGIN RSA PUBLIC KEY-----
MIIBCgKCAQEAzk1mzQsl8cCw7IuH6tP34+evkVTX/IORk1Yta7kHVbV3SoDrpJUE
C2Jufohp3Wr6iF1093y+WZl/UGPcOAVtcw10MeI9ZJbfW78uubo+OcduLqsqbYFJ
X9ixwOuEW1xytA8E79HDjJ/4Lwy89Oi3/xBdaEeOhz1bYMFVQ3NfQl4ShP+HPMbe
jP5w3s4DsJUwMa/f0xYtMS/bxqXQbTGEWvtZppSEZOgN81z+Nc8SnHGuUYgMwgCX
d6NC7DFZcGYPxqWXJQlzSw/BTaqo9wXKcvUryGR1t+DLGjgCGqdheGLzv45GUj6c
rFWzRnoAaJiCXPb998MUugqcsY9juRnsbQIDAQAB
-----END RSA PUBLIC KEY-----
I am a newbie in Teams app development, hope someone can help me out of this.
Thanks in advance.

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

ID: e7ac1764-9172-9f3e-2091-5dfd755eded4
Version Independent ID: e7ac1764-9172-9f3e-2091-5dfd755eded4
Content: Build command bot with JavaScript - Teams
Content Source: msteams-platform/sbs-gs-commandbot.yml
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: lajanuar

weird,  I try again, this time I won't get the error message about key and token , but local debug still fails.
Right now, when I add the command bot, teams only says "Run into problem" as the picture (it is written in chinese)

OS: Windows 11
VSCode + Teams Toolkit + Edge

@kenhutaiwan - We are not able to repro the issue as it is working fine from our end. Could you check it once Prerequisites and steps once in the given document?
Reference Document-https://learn.microsoft.com/en-us/microsoftteams/platform/sbs-gs-commandbot?tabs=vscode&tutorial-step=1
https://user-images.githubusercontent.com/93527375/219654917-531c8cbe-0237-460f-92a3-bf3d0195f44d.mp4

@kenhutaiwan -Did you get chance to check above suggestion?

Sorry, I am too late to see your feedback and I can't reproduce it myself because I lost the experiment installation.
Thanks for your help.

@kenhutaiwan -Could you please confirm if your issue has resolved or still looking for any help?


chore(package): update eslint-plugin-node to version 6.0.1

Closes #80

Codecov Report

Merging #82 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #82   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines         157    157
  Branches       23     23
=====================================
  Hits          157    157

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 30ec685...b2d38a9. Read the comment docs.


reinterpret retvar as godot_string and then convert it to Kotlin String in icalls returning String

We were trying to convert a pointer to KString, this fixes this behaviour by first reinterpret pointer as godot_string, then read its value and anding by converting value to Kotlin String.

I'm a bit confused, isn't Godot supposed to return us a Variant in any case, not directly a Godot_string ?

Yeah, I thought it was a godot_variant not a godot_string. Can you confirm @piiertho?

I have a better way to do, will be fixed with rework of allocations in icalls.


Create inquiry form in English.

@hkurokawa English version not exists?
https://docs.google.com/forms/d/1fpxZOqprZyzBgMBb7HCCrq538oOzfzWkOMpItRqF-ro/viewform

I think this link is.
https://docs.google.com/forms/d/1X9x9eb92_BeRaELjePq8FgeqTa70MyChkmUWFce47jo/edit

@konifar Sorry for the late reply. As hotchemi already pointed out, there is a form but not translated yet. As for now, we can just use the URL and update it later.


Access server object from server methods

Is there a way to access the server object from within server methods?
I guess it should be possible to  bind the server object (by setting options: {bind: server}, see here) but I haven't been able to do this with haute couture

Yes!
With haute-couture any time you would write this,
// methods.js
module.exports = [{
    name: 'someName',
    method: (x, y, next) => {/* ... */}
}];

you could also write the same thing this way to access the server and plugin registration options,
// methods.js
module.exports = (server, options) => {

    return [{
        name: 'someName',
        method: (x, y, next) => {/* ... */}
    }];
};


C-PARSER cannot parse MSVC preprocessed files which have #ifdef in a struct or function

Describe the bug
The MSVC preprocessor will emit a #line directive when it sees an #ifdef.
If this #ifdef is inside a struct or function, then the C parser chokes.
This happens when using the -E preprocessor flag.
When using -EP, the #line directives are not emitted.
But then the c parser is unable to track the location of a struct/function definition.
I'm using the MSVC preprocessor because ghidra's preprocessor does not parse function macros (see #2665)
To Reproduce
Steps to reproduce the behavior:

Create this header (structif.h):#define USE_INT
typedef struct mystruct {
#ifdef USE_INT
  int a;
#else
  float a;
#endif
};

Preprocess this file with MSVC (e.g. inside a MSVC development prompt) with the following command: cl -E structif.h >structif.preprocessed.h.
The resulting file is:#line 1 "C:/Users/maarten/source/repos/ghidra_preprocess/structif.h"

typedef struct mystruct {

  int a;

#line 8 "C:/Users/maarten/source/repos/ghidra_preprocess/structif.h"
};

Add the structif.preprocessed.h file to the list in Parse C Source dialog
Click on Parse to Program/Parse to File + ok until it starts parsing

Expected behavior
Parsing works + struct with integer is added to the filetype archive.
Screenshots

Environment (please complete the following information):

OS: Windows 10
Java Version: 14.0.2 2020-07-14
Ghidra Version: 9.2.1

Additional context
This problem occurs in my quest to parse Microsofts' ddraw.h.

I guess much is explained here : #1784

I guess much is explained here : #1784

Thanks for the reply.
I am able to create a datatype archive by post processing the file(s).
I need to apply the following regex'es to the preprocessed file:

s/__declspec\(deprecated\([^)]*\)\)//g
s/#pragma[ \t]*(pack|warning)\((push|pop|(disable:[0-9]*))\)//g

I opened #2665, #2666 and #2666 because I was able to pinpoint the root problems with the C parser.
If these get fixed, these workarounds are no longer needed (by me or anyone else).

Thanks for the reply.
I am able to create a datatype archive by post processing the file(s).
I need to apply the following regex'es to the preprocessed file:

s/__declspec\(deprecated\([^)]*\)\)//g
s/#pragma[ \t]*(pack|warning)\((push|pop|(disable:[0-9]*))\)//g

I opened #2665, #2666 and #2666 because I was able to pinpoint the root problems with the C parser.
If these get fixed, these workarounds are no longer needed (by me or anyone else).


Laravel Vite requests to not load on fresh Laravel Install on Homestead

Laravel Vite Plugin Version: 0.2.1
Laravel Version: 9.19
Node Version: v16.15.0
NPM Version: 8.5.5

Description:
Created a new laravel project, installed laravel breeze, run npm install and run dev, and JS and CSS files do not load in the browser. Using homestead as my local dev env running PHP 8.1. All requests to the vite dev server get blocked. Not using any SSL, just a local domain configured within the homestead.yaml. I've tried a couple of different config options within the vite.config.js file to specify my custom homestead domain but it did not fix it.
Console with blocked network requests

Styles not loading

Steps To Reproduce:
laravel new project
composer require laravel/breeze --dev
php artisan breeze:install
npm install
npm run dev
If I just run PHP artisan serve, styles and js load correctly - only issue while trying to visit a local domain that is running via homestead

I'm sorry but I think I figured it out. Looking through some previous GitHub issues it looks like this is a Brave-specific issue! I turned off the "shield" and now things are working. Sorry for creating this ticket in haste but for anyone else having this issue on Brave this helped me: https://github.com/laravel/vite-plugin/issues/47

Glad you got it sorted @gradedcatfood For yourself and future visitors, you may also configure Brave to allow the Vite dev server to run on your specific project URLs.


FEAT-#2274: support encoding='unicode_escape' for read_csv

What do these changes do?

add test cases with encoding equals unicode_escape, raw_unicode_escape, utf16, utf32.
add support for encodings in which the end-of-line character is encoded in several bytes.

[x] commit message follows format outlined here
[x] passes flake8 modin
[x] passes black --check modin
[x] signed commit with git commit -s
[x] Resolves #2274
[ ] tests added and passing

There was found the bug in Pandas: https://github.com/pandas-dev/pandas/issues/37414

@anmyachev could you also please post the time measurements of read_csv with encoding="unicode_escape", comparing it with encodings, whose read logic was not affected by these changes

@anmyachev could you also please post the time measurements of read_csv with encoding="unicode_escape", comparing it with encodings, whose read logic was not affected by these changes

rows_count: 2000000, encoding: unicode_escape, quoting: True, nrows: None
.....
.....
pandas.read_csv: 4.086673974990845
modin.read_csv: 3.378664016723633

rows_count: 2000000, encoding: unicode_escape, quoting: True, nrows: 2000000
.....
.....
pandas.read_csv: 4.046497344970703
modin.read_csv: 5.378690958023071

rows_count: 2000000, encoding: unicode_escape, quoting: False, nrows: None
.....
.....
pandas.read_csv: 4.0841968059539795
modin.read_csv: 3.005202293395996

rows_count: 2000000, encoding: unicode_escape, quoting: False, nrows: 2000000
.....
.....
pandas.read_csv: 3.9735991954803467
modin.read_csv: 5.374706745147705

rows_count: 2000000, encoding: utf8, quoting: True, nrows: None
.....
.....
pandas.read_csv: 3.504272222518921
modin.read_csv: 2.897287130355835

rows_count: 2000000, encoding: utf8, quoting: True, nrows: 2000000
.....
.....
pandas.read_csv: 3.338742256164551
modin.read_csv: 4.659839153289795

rows_count: 2000000, encoding: utf8, quoting: False, nrows: None
.....
.....
pandas.read_csv: 3.2649505138397217
modin.read_csv: 2.537930965423584

rows_count: 2000000, encoding: utf8, quoting: False, nrows: 2000000
.....
.....
pandas.read_csv: 3.125645875930786
modin.read_csv: 4.3011555671691895

@anmyachev what about measurements of encodings like cp1252 which should not be affected by this changeset?

Much better, but what about splitting refactoring and feature changes?

I want to do this after everything works correctly here.
An error appeared after the rebase.

What is the status of this PR @anmyachev ?

There was some unsolved problem. I need to remove conflicts and look more closely.

Python unicode_escape fix: https://github.com/python/cpython/pull/28939
Will be available only for python>3.9.

@vnlitvinov ready for review


Added a Gregorian Calendar Arithmetic class

The class Gregorian allows easy addition of dates just use +

Calendar Arithmetic


Challenge http://www.freecodecamp.com/challenges/waypoint-create-a-text-field has an issue

Is this a bug?  Im reasonably sure my code is correct but I can't go on to the next waypoint.
My code is
<link href='http://fonts.googleapis.com/css?family=Lobster' rel='stylesheet' type='text/css'>

CatPhotoApp

Click here for cat photos.

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

<form>
<input type="text" name="text">
</form>

But I still get a "Syntax error, unrecognized expression"
I popped it into Brackets IDE and it seems to work o.k. though?
Thanks

We've fixed this. Thank you for your patience, and for bringing this to our attention!


Server Rendered Example

Hello, Can somebody create an example of choo server rendered using expressjs??
Thanks

We lost a few examples in the recent updates to Choo, but here are the old examples from Choo 4 that need to be brought forward to Choo 5/6:

https://github.com/choojs/choo/tree/v4.1.0/examples/server-rendering
https://github.com/jondashkyle/jon-kyle.com <- static ssr example

Its on the TODO list, but if you come up with a good example we can add it to the example section.

Here's my example for choo ssr. Any suggestions?
https://github.com/hdriqi/choo-ssr-example

I like it from the look!  I'll (or whoever beats me to it) will look at this more soon.  We should probably have an SSR example with bankai, but I think the express example is good too.

There is express one here as well:
https://github.com/graforlock/choo-bandwagon


Cannot print legacy certificate

Describe your problem and - if possible - how to reproduce it
I am not able to print my legacy certificate. I am trying to print it using Ctrl + P, and Print option in the right click dropdown menu. The pdf appears blank.
Add a Link to the page with the problem
Here is the link to my certificate.
Tell us about your browser and operating system

Browser Name: Google Chrome
Browser Version: 66.0.3359.181
Operating System: Windows 8.1 Single Language 64-bit

If possible, add a screenshot here

P.S.: I am not sure if it is a bug or intended to be unprintable, but I found why I can't print it.
@media print {
  * {
    display: none !important;
  }
}

Thanks.

@mouri11 If you remove that JavaScript, are you able to print it? If so, how does it look?

@QuincyLarson  I tried to print his certificate to see if it was a local issue or not. I am not having issues seeing a preview or a printer listed on my Win10 in either Edge or Firefox... but chrome gives me this.

This is the same for me. As far as I'm aware, @QuincyLarson certificates were specifically not allowed to be printed and this was decided during the process. Do we want this to change?

@QuincyLarson Hi. Removing the javascript messes up the layout of the document. The signature becomes huge, the date gets skewed, etc.

I guess not being able to print would be fine, except if we have to carry hardcopies to interviews.

How about taking a screenshot? :wink:

@mouri11 Sorry, I meant taking a screenshot of the certificate, save it as png or something, then print that photo out. Firefox has the 'take a screenshot' feature built in, if you're not aware of.

Oh right. Sorry to misinterpret it. I already took a fullscreen screenshot of the certificate. I had asked this to know if this blocking of printing is intended or just a bug. Thanks though.

Oh cool! Glad you could find a way around it.

@jonathan-grah @huyenltnguyen I've changed the title of this issue to "make certificates more printable". I believe since they're just HTML and CSS, with a couple images, we should be able to make them reliably print. This has always been a very common request among campers, and it would be worth fixing for the new certificates :) Would either of you be interested in tackling this?

@QuincyLarson, sure, I can tackle this soon. This has been highly requested by campers.
If this needs to be done quickly, it might be better for @huyenltnguyen to give this a go, if they want, because I mightn't be able to get this done this week.

@jonathan-grah I'll gladly take it from you :) Will try to submit a PR asap.

@jonathan-grah OK - thanks for being candid about your availability.
@huyenltnguyen Awesome - thanks for your help with this. Please keep us posted and let us know if we can be of any help during the process :)


added react native link in the step

they should run react-native link so that this could work

Thanks @hiaw
I have merged now :)


Expand test coverage for null-safety to all configurations

[x] CFE
[x] Analyzer
[x] VM (this will need to be broken down further)
[ ] DDC
[ ] dart2js
[x] Ensure that there is a way to compare the results of the new builders to their legacy counterparts.
[ ] Check that the remaining configurations are either out-of-scope or add coverage for them as well.

@franklinyow FYI

In the scope of this issue, builders will start to be added as nightly builders. Once it has been determined which have the greatest discrepancies, some builders will be moved over to be commit triggered.

@vsmenon Unfortunately not. We still need to do more work there.

@athomas - is this still a P1?


Fix: Navigation Bar Breaking

When clicking on 'tools' for example, you can't go back to home. This is true for other parts of the navigation as well

Can I be assigned to this please

is the issue still open

Closing this for now


[release-1.20] install: add envoyfilter to prune list

Please provide a description of this PR:

@hanxiaop: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio_release-1.20
1354d8fd239024d9adae4f430b6171854d00e559
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio_release-1.20
1354d8fd239024d9adae4f430b6171854d00e559
link
true
/test lint

unit-tests-arm64_istio_release-1.20
1354d8fd239024d9adae4f430b6171854d00e559
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio_release-1.20
1354d8fd239024d9adae4f430b6171854d00e559
link
true
/test lint

unit-tests-arm64_istio_release-1.20
1354d8fd239024d9adae4f430b6171854d00e559
link
true
/test unit-tests-arm64

gencheck_istio_release-1.20
1354d8fd239024d9adae4f430b6171854d00e559
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio_release-1.20
1354d8fd239024d9adae4f430b6171854d00e559
link
true
/test lint

unit-tests-arm64_istio_release-1.20
1354d8fd239024d9adae4f430b6171854d00e559
link
true
/test unit-tests-arm64

gencheck_istio_release-1.20
5b1810f9fa9ee99372d58a49bb348ab9831e199e
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.20
1354d8fd239024d9adae4f430b6171854d00e559
link
true
/test unit-tests-arm64

gencheck_istio_release-1.20
5b1810f9fa9ee99372d58a49bb348ab9831e199e
link
true
/test gencheck

unit-tests_istio_release-1.20
5b1810f9fa9ee99372d58a49bb348ab9831e199e
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio_release-1.20
5b1810f9fa9ee99372d58a49bb348ab9831e199e
link
true
/test gencheck

unit-tests_istio_release-1.20
5b1810f9fa9ee99372d58a49bb348ab9831e199e
link
true
/test unit-tests

unit-tests-arm64_istio_release-1.20
5b1810f9fa9ee99372d58a49bb348ab9831e199e
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


feat: 🎸 add concept of Resource

This long PR creates a new concept, Resource. The resource is aimed at being allocated and then released after use: connection to a database, modification of the datasets library config, or creation of storage directories...
Before this PR, it was done in the Config step. Now the Config step should never raise and should be immutable, and the Resources are created based on that config.
It makes it "easier" to understand what is failing in the tests and in the app.
GOOD LUCK to the reviewers, and sorry for the length.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Codecov Report
Base: 90.69% // Head: 92.57% // Increases project coverage by +1.88% :tada:

Coverage data is based on head (4681b97) compared to base (3ca72c6).
Patch coverage: 99.07% of modified lines in pull request are covered.

:exclamation: Current head 4681b97 differs from pull request most recent head 2301eb9. Consider uploading reports for the commit 2301eb9 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #784      +/-   ##
==========================================
+ Coverage   90.69%   92.57%   +1.88%
==========================================
  Files          70       35      -35
  Lines        3493     2385    -1108
==========================================
- Hits         3168     2208     -960
+ Misses        325      177     -148

Flag
Coverage Δ

services_admin
?

services_api
?

workers_datasets_based
92.57% <99.07%> (+0.49%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...s/datasets_based/src/datasets_based/worker_loop.py
50.00% <83.33%> (+2.50%)
:arrow_up:

...atasets_based/src/datasets_based/worker_factory.py
92.30% <93.75%> (+1.19%)
:arrow_up:

workers/datasets_based/src/datasets_based/asset.py
100.00% <100.00%> (ø)

...orkers/datasets_based/src/datasets_based/config.py
100.00% <100.00%> (+1.02%)
:arrow_up:

...kers/datasets_based/src/datasets_based/features.py
73.84% <100.00%> (+0.40%)
:arrow_up:

...ers/datasets_based/src/datasets_based/resources.py
100.00% <100.00%> (ø)

...c/datasets_based/workers/_datasets_based_worker.py
100.00% <100.00%> (ø)

...ets_based/src/datasets_based/workers/first_rows.py
81.96% <100.00%> (+0.30%)
:arrow_up:

...datasets_based/workers/parquet_and_dataset_info.py
91.37% <100.00%> (ø)

workers/datasets_based/tests/conftest.py
92.77% <100.00%> (+2.00%)
:arrow_up:

... and 51 more

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

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

some of the resources you define do not follow this design pattern

If you use a context manager, we expect the resource to be deleted on exit, which is not the case here. Therefore I'm not sure this one should be a Resource.

Yes, you're both right. I tried to manage the log, storage and access to the databases from the same concept of Resource, as context managers. But it does not work so well. I'll change this, thanks.

Should be good for review now @lhoestq @albertvillanova @AndreaFrancis 🤞

Thanks for the reviews. Merging


Tabs vs spaces

When I was using Bender, I was confused with using indentations. I think Bender only uses spaces as indentation. But I think placing white space (tabs or spaces) is much easier for programmers.

I assume you mean in the generated scripts? I don't quite know how you would like to tailor to users' preferences, but feel free to propose a change in a PR.

Not in the generated scripts. When I was making a bender.yml file, I faced that issue. It is not a major issue but It would be great to have white spaces instead of tabs and spaces.

The Bender.yml is read using the serde_yaml package (see https://github.com/pulp-platform/bender/blob/master/src/cli.rs#L375-L387). If there is a configuration that is missing to allow your whitespaces, let me know (easiest with a PR that can be integrated), but fixing serde_yaml within bender does not make sense.


Check for bitstreams in ORIGINAL bundle only

Fix for https://github.com/ufal/clarin-dspace/issues/999.
Tested with metadata quality curation check, not sure if this is enough.

seems ok; will check next week


.Net: KernelMemoryBuilder: Unknown field 'embedding' in vector field list. Status: 400 (Bad Request) ErrorCode: InvalidRequestParameter

Describe the bug
var verification = await kernelMemory.SearchAsync("what is my Decutible?");
Result in the following error.
Unknown field 'embedding' in vector field list.
Status: 400 (Bad Request)
ErrorCode: InvalidRequestParameter

To Reproduce
Steps to reproduce the behavior:
var AZURE_SEARCH_ENDPOINT = _configuration.GetSection("AzureSearchAI:EndPoint").Value!;
                var AZURE_SEARCH_API_KEY = _configuration.GetSection("AzureSearchAI:ApiKey").Value!;
                var AZURE_SEARCH_INDEX_NAME = _configuration.GetSection("AzureSearchAI:IndexName").Value!;

#pragma warning disable SKEXP0001, SKEXP0010 // Type is for evaluation purposes only and is subject to change or removal in future updates. Suppress this diagnostic to proceed.

                var embeddingConfig = new AzureOpenAIConfig
                {
                    APIKey = _configuration.GetSection("AzureOpenAI:ApiKey").Value!,
                    Deployment = _configuration.GetSection("AzureOpenAI:EmbeddingModelName").Value!,
                    Endpoint = _configuration.GetSection("AzureOpenAI:EndPoint").Value!,
                    APIType = AzureOpenAIConfig.APITypes.EmbeddingGeneration,
                    Auth = AzureOpenAIConfig.AuthTypes.APIKey
                };

                var chatConfig = new AzureOpenAIConfig
                {
                    APIKey = _configuration.GetSection("AzureOpenAI:ApiKey").Value!,
                    Deployment = _configuration.GetSection("AzureOpenAI:DeploymentModelName").Value!,
                    Endpoint = _configuration.GetSection("AzureOpenAI:EndPoint").Value!,
                    APIType = AzureOpenAIConfig.APITypes.ChatCompletion,
                    Auth = AzureOpenAIConfig.AuthTypes.APIKey
                };

                var searchConfig = new AzureAISearchConfig
                {
                    APIKey = _configuration.GetSection("AzureSearchAI:ApiKey").Value!,
                    Endpoint = _configuration.GetSection("AzureSearchAI:EndPoint").Value!,
                    Auth = AzureAISearchConfig.AuthTypes.APIKey,
                };

                var memoryConfig = new KernelMemoryConfig
                {
                    DefaultIndexName = AZURE_SEARCH_INDEX_NAME,
                };

                var kernelMemory = new KernelMemoryBuilder()
                    .With(memoryConfig)
                    .WithAzureOpenAITextGeneration(chatConfig)
                    .WithAzureOpenAITextEmbeddingGeneration(embeddingConfig)
                    .WithAzureAISearchMemoryDb(searchConfig)
                    .WithSearchClientConfig(new SearchClientConfig { MaxAskPromptSize = 5000, AnswerTokens = 4096 })
                    .Build<MemoryServerless>();

My additional question here is, that i want to use this a Plugin.
I will set
   OpenAIPromptExecutionSettings openAIPromptExecutionSettings = new()
   {
       ToolCallBehavior = ToolCallBehavior.AutoInvokeKernelFunctions
   };

But my question here is how will Semantic kernel find out that it need to execute this Plugin for the Question asked ?
When we are creating our custom plugin, we do describe about it and that is what is used by semantic kernel to decide if it should call this plugin? Where would i provide the description in this case ?
Expected behavior
It should search the configured index and return appropriate results.
Screenshots
If applicable, add screenshots to help explain your problem.
Platform

OS: [e.g. Windows, Mac]
IDE: [e.g. Visual Studio, VS Code]
Language: [e.g. C#, Python]
Source: [e.g. NuGet package version 0.1.0, pip package version 0.1.0, main branch of repository]

Additional context
Add any other context about the problem here.

@nssidhu It looks like you are using KernelMemory package. Both Semantic Kernel and Kernel Memory packages don't support custom Azure AI Search index schema at the moment. We are working on design of memory-related abstractions to allow support of custom schema, but as for now, your index should have the same schema as defined in packages (including embedding field).
If you want to use custom schema in Azure AI Search, here is an example of such SK plugin:
https://github.com/microsoft/semantic-kernel/blob/main/dotnet/samples/KernelSyntaxExamples/Example84_AzureAISearchPlugin.cs

But my question here is how will Semantic kernel find out that it need to execute this Plugin for the Question asked ?
When we are creating our custom plugin, we do describe about it and that is what is used by semantic kernel to decide if it should call this plugin? Where would i provide the description in this case ?

When you register a Plugin in Semantic Kernel, you can call your function directly from prompt. For example, if you define a function in plugin with name Search, you can call it directly like this:
var result = await kernel.InvokePromptAsync("{{search 'my query'}}");

But if you want to use AI to decide whether you need to call it or not, you need to enable function calling ToolCallBehavior.AutoInvokeKernelFunctions and you need to provide meaningful description for your function so AI will be able to call it when needed. To describe your function, you can use Description attribute at place of function definition. For example:
https://github.com/microsoft/semantic-kernel/blob/043e49b6c51500150f578323c422cf6790fab3a5/dotnet/samples/DocumentationExamples/Plugin.cs#L104-L106

@dmytrostruk My Second question is related to using AzureAISearch with kernel memory and not the custom function.
Say you got the fix for custom schema, How would semantic kernel decide it has to use or invoke the Azure Search AI for the question asked. I don't want to invoke (var result = await kernel.InvokePromptAsync("{{search 'my query'}}");) i am using
OpenAIPromptExecutionSettings openAIPromptExecutionSettings = new()
 {
     ToolCallBehavior = ToolCallBehavior.AutoInvokeKernelFunctions
 };

Most of my Azure Search AI schemas were created from the portal.
Any time lines on when the schema fix will be available ?

My Second question is related to using AzureAISearch with kernel memory and not the custom function.

My apologies, probably I didn't get it right.

Say you got the fix for custom schema, How would semantic kernel decide it has to use or invoke the Azure Search AI for the question asked.

As for now, all memory-related features are registered in Semantic Kernel as Plugins. Which means that when you enable function calling (e.g. ToolCallBehavior = ToolCallBehavior.AutoInvokeKernelFunctions), AI model will decide whether to call your Plugin (i.e. make a request to Azure AI Search) based on Plugin description and your prompt. For example, if your prompt contains an ask to find some information, and memory plugin has description, that it is responsible for finding some information, then most probably AI model will call the Plugin, which means it will make a request to Azure AI Search.
I'm not sure how memory will be integrated into Semantic Kernel in the future, but as for now memory in SK is registered as Plugin.

Most of my Azure Search AI schemas were created from the portal.
Any time lines on when the schema fix will be available ?

Taking into account, that memory in SK is registered as Plugin, in order to work with your custom schema, you can define your own Plugin, that will work with Azure AI Search and desired schema. I shared an example above, it should help you to achieve this scenario without waiting for schema fix.

@nssidhu - please post this over to the Kernel Memory repo - https://github.com/microsoft/kernel-memory/issues


[Sozialpass] Localize dates

Is your feature request related to a problem? Please describe.
At the moment all dates in the app use german date format (DD.MM.YYYY).
Describe the solution you'd like
The format of the shown dates should depend on the current locale.
Describe alternatives you've considered
None.
Additional context
There is built in flutter functionality to do this: https://docs.flutter.dev/ui/accessibility-and-internationalization/internationalization#messages-with-dates
Part of #1174.

It might actually be better not to localize the dates, at least not the ones shown on the card itself. It might be very confusing for German verifiers to see an American date format (month/day/year).
Not sure, if you already discussed this, though.

what is your opinion @dkehne ?
Can we close this issue?

my opinion is to stay with the german-date-format everywhere. It should not be a problem if we have the English-Version of the App and German locaization of dates. I also see the confusion-aspect by switching month and day-position.

perfect so i'm gonna close that issue

perfect so i'm gonna close that issue


Add JMD to currency option to movemoney screen

Currently only USD and BTC options, want to add JMD as an option. Potential display modes:

JMD display, pegged to JMD rate || USD display, pegged to USD rate || BTC display pegged to BTC rate

JMD display, pegged to USD exchange rate  || USD display, pegged to USD rate || BTC display pegged to BTC rate

JMD display, pegged to BTC exchange rate  || USD display, pegged to USD rate || BTC display pegged to BTC rate

An upcoming update to the galoy backend will have multiple currencies enabled. May make sense to wait on this update and merge.

hello Ayush there, almost 3 week ago  we have a chat on mattermost galoy server about bounties on making components for inflash payement app https://github.com/lnflash/flash-mobile , now after two weeks i have made a contribution in galoy   mobile  https://github.com/GaloyMoney/galoy-mobile/pull/1172   and now i have got  enough confidence to make some components for your app . since you were not active on galoy server . thats why  im commenting here.

handled by galoy-mobile


LG S4NQ09WA51A.EB2GAMZ - Works fine.

Just to let you guys know, the LG S4NQ09WA51A.EB2GAMZ 220v 9000btu works fine.
The hardware doesn't heat, just cool, and in HK it shows "Heating/Cooling/Auto", but's ok.
I hope in a future we can hide the Heat mode.

Good to know! I can add this to the readme.
Does it support a “dry” or “dehumidify” option? That’s what I have mine mapped to right now for “heat” since HK doesn’t let us change the label

Yeah it does suport the Dehumidify! But, unfortunaly, i don't know how to map to this option.
Btw, i'm getting this "error" from time to time =
"[homebridge-lg-thinq-ac] This plugin generated a warning from the characteristic 'Filter Life Level': characteristic value expected valid finite number and received "NaN" (number). See https://git.io/JtMGR for more info."
The AC doens't show when the filter needs to be cleaned, so maybe that's why this shows.

Thanks! The log error has been fixed in 22f0aa6007dc4ee993aed383e3c35afcc2707fd4, follow #94 for a fix for filter life being accurate
What model is your unit? It should display in the Home app under "Model"

Note added in ffedffd


add requirement for SSA tags on lists

This avoids always using the default of atomic, which is often not what is actually desired.

/lgtm


picocom: fix substitution

Description of changes
The upstream code to be replaced changed in upstream commit 8f41c344.
Things done

Built on platform(s)

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

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

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

[ ] 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/)
22.11 Release Notes (or backporting 22.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Oh, how I would like a --abort-on-no-subst flag to substituteInPlace and friends.....


Add Benchmark against turbodbc

It would be nice to see a benchmark against turbodbc since it was often proclaimed to be the fastest way of fetching data to pandas.
https://github.com/blue-yonder/turbodbc

Hi @DD5HT , we just added the benchmark against turbodbc. You can find the result here.


Boids

This PR adds a new constraint to the list which simulates flocking behavior of bird using algorithm described by Craig Reynolds which can be found here: Boids - Wikipedia

https://github.com/anonthedev/particleLife/assets/32013268/ff0140fe-8b5e-4da2-b475-d906f7e8a14b

awesome PR man just add some comment in the utils.js last function

@anonthedev I added the comments. You can merge now. Tell me if you want something changed.

merging now, thanks for the PR man

Found this repo in the apna college PR spam thread on X. Thought I should give it a try.

Glad you did


Integration of Elasticsearch to the Frontend

Reason (Why?)
This feature is required and a mandatory need for our application to query posts from the backend. Without integration with elasticsearch no posts can be displayed on frontend site
Solution (What?)
Interact with the elasticsearch backend server via queries written in Dataservice.ts. The queries are written with usage of es-query-builder: https://github.com/hochschule-darmstadt/es-query-builder. Query results should be displayed at the ResultPage.vue as posts
Not doing
Do not autocomplete results via searchbar
Relation to other Issues
#8
Acceptance criteria
A simple search should yield results queried by ES search that are displayed on the ResultPage.vue

This issue has been completed and the progress can be viewed on the staging server


[itk] build failure

Operating system
Windows 10 home
Compiler
MSVC++ 2019, cuda v11.7
Steps to reproduce the behavior
vcpkg install itk[rtk]:x64-windows

Failure logs
E:\vcpkg>vcpkg install itk[rtk]:x64-windows --recurse
warning: The vcpkg E:\vcpkg\vcpkg.exe is using detected vcpkg root E:\vcpkg and ignoring mismatched VCPKG_ROOT environment value "E:\vcpkg". To suppress this message, unset the environment variable or use the --vcpkg-root command line switch.
Computing installation plan...
The following packages will be rebuilt:
itk[core,cuda,opencl,rtk]:x64-windows -> 5.3rc02
Detecting compiler hash for triplet x64-windows...
Restored 0 package(s) from C:\Users\sapak\AppData\Local\vcpkg\archives in 479 us. Use --debug to see more details.
Removing 1/2 itk:x64-windows
Elapsed time to handle itk:x64-windows: 900 ms
Installing 2/2 itk:x64-windows...
Building itk[core,cuda,opencl,rtk]:x64-windows...
-- Using cached InsightSoftwareConsortium-ITK-v5.3rc02.tar.gz.
-- Cleaning sources at E:/vcpkg/buildtrees/itk/src/v5.3rc02-2745f7bb96.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source E:/vcpkg/downloads/InsightSoftwareConsortium-ITK-v5.3rc02.tar.gz
-- Applying patch double-conversion.patch
-- Applying patch openjpeg.patch
-- Applying patch openjpeg2.patch
-- Applying patch var_libraries.patch
-- Applying patch wrapping.patch
-- Applying patch opencl.patch
-- Applying patch use-the-lrintf-intrinsic.patch
-- Applying patch dont-build-gtest.patch
-- Applying patch cxx17.patch
-- Using source at E:/vcpkg/buildtrees/itk/src/v5.3rc02-2745f7bb96.clean
-- Found external ninja('1.10.2').
-- Configuring x64-windows-dbg
-- Configuring x64-windows-rel
CMake Warning at installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_configure.cmake:318 (message):
The following variables are not used in CMakeLists.txt:
  BUILD_PKGCONFIG_FILES
  DCMTK_USE_ICU
  INSTALL_GTEST
  ITK_USE_SYSTEM_GOOGLETEST
  ITK_USE_SYSTEM_ICU
  ITK_USE_SYSTEM_SWIG

Please recheck them and remove the unnecessary options from the
vcpkg_cmake_configure call.
If these options should still be passed for whatever reason, please use the
MAYBE_UNUSED_VARIABLES argument.
Call Stack (most recent call first):
ports/itk/portfile.cmake:127 (vcpkg_cmake_configure)
scripts/ports.cmake:168 (include)
CMake Deprecation Warning at scripts/ports.cmake:98 (message):
vcpkg_apply_patches has been deprecated in favor of the PATCHES argument
to vcpkg_from_*.
Call Stack (most recent call first):
scripts/cmake/vcpkg_apply_patches.cmake:2 (z_vcpkg_deprecation_message)
ports/itk/portfile.cmake:189 (vcpkg_apply_patches)
scripts/ports.cmake:168 (include)
-- Applying patch rtk/already_defined.patch
-- Applying patch rtk/unresolved.patch
-- Applying patch rtk/Add-missing-include-for-Cuda.patch
-- Building x64-windows-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:134 (message):
Command failed: "C:/Program Files/CMake/bin/cmake.exe" --build . --config Debug --target install -- -v -j9
Working Directory: E:/vcpkg/buildtrees/itk/x64-windows-dbg
See logs for more information:
E:\vcpkg\buildtrees\itk\install-x64-windows-dbg-out.log
Call Stack (most recent call first):
installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_build.cmake:74 (vcpkg_execute_build_process)
installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_install.cmake:16 (vcpkg_cmake_build)
ports/itk/portfile.cmake:191 (vcpkg_cmake_install)
scripts/ports.cmake:168 (include)
error: building itk:x64-windows failed with: BUILD_FAILED
Elapsed time to handle itk:x64-windows: 6.4 min
Please ensure you're using the latest port files with git pull and vcpkg update.
Then check for known issues at:
https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+itk
You can submit a new issue at:
https://github.com/microsoft/vcpkg/issues/new?title=[itk]+Build+error&body=Copy+issue+body+from+E%3A%2Fvcpkg%2Finstalled%2Fvcpkg%2Fissue_body.md
Additional context
install-x64-windows-dbg-out.log

nvcc fatal   : Unsupported gpu architecture 'compute_30'
According to the error log, your GPU does not support the current version of CUDA. Please check the CUDA version supported by your GPU.

C:\Users\sapak>nvidia-smi
Wed Oct 25 12:51:46 2023
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 545.84                 Driver Version: 545.84       CUDA Version: 12.3     |
|-----------------------------------------+----------------------+----------------------+
| GPU  Name                     TCC/WDDM  | Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |         Memory-Usage | GPU-Util  Compute M. |
|                                         |                      |               MIG M. |
|=========================================+======================+======================|
|   0  NVIDIA GeForce GTX 1050      WDDM  | 00000000:01:00.0 Off |                  N/A |
| N/A   47C    P0              N/A / ERR! |      0MiB /  4096MiB |      0%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+
+---------------------------------------------------------------------------------------+
| Processes:                                                                            |
|  GPU   GI   CI        PID   Type   Process name                            GPU Memory |
|        ID   ID                                                             Usage      |
|=======================================================================================|
|  No running processes found                                                           |
+---------------------------------------------------------------------------------------+
-gencode;arch=compute_30,code=sm_30;  <--- This item seems to be interfering with my build. Installing CUDA version 10.2 gives another error:
The graphics card is using gtx1050.
What is the exact installation environment?

The GTX 1050 graphics card supports up to CUDA version 11.4, so you should install a CUDA version less than or equal to 11.4 so that the GPU can be correctly identified.

Even if I reset the cuda version to 11.4, the same problem still appears.
Searching the Internet, I found that Compute_30 was removed from cuda 10.2 and later versions.
How can I install it?
E:\vcpkg\buildtrees\itk\src\v5.3rc02-2745f7bb96.clean\Modules\Remote\RTK\cmake\FindCUDA_wrap.cmake
if("${CUDA_VERSION}" VERSION_LESS 8.0)
set(CUDA_NVCC_FLAGS ${CUDA_NVCC_FLAGS}
-gencode arch=compute_20,code=sm_20
-gencode arch=compute_30,code=sm_30
-gencode arch=compute_35,code=sm_35
-gencode arch=compute_35,code=compute_35
)
elseif("${CUDA_VERSION}" VERSION_LESS 9.0)
set(CUDA_NVCC_FLAGS ${CUDA_NVCC_FLAGS}
-Wno-deprecated-gpu-targets
-gencode arch=compute_20,code=sm_20
-gencode arch=compute_30,code=sm_30
-gencode arch=compute_35,code=sm_35
-gencode arch=compute_35,code=compute_35
)
else()
set(CUDA_NVCC_FLAGS ${CUDA_NVCC_FLAGS}
-gencode arch=compute_30,code=sm_30                                           <<---------- This item seems to be problematic.
-gencode arch=compute_35,code=sm_35
-gencode arch=compute_35,code=compute_35
)
endif()

You can download and install it from the cuda official website.

cuda has already been installed separately.
I was using version 12, but it didn't seem to work because the version was too high, so I reinstalled 11.4.

Can you compile normally if you install version 11.4 of cuda now?

Thanks for posting this issue. Please reopen this issue if this is still a problem for you.


fix: fixed some errors sonar cloud pickedup

Fixed three problems Sonarcloud presented:
Activity/tests/test_serializers.py:
Removed code that has been marked as unreachable for three years. My reasoning was that if nobody used this code for three years it probably is not that important.
Generics/serializers.py:
Made __init__ return none, if this is not the case, __init__ returns a type error(unless nothing is returned which is basically the same as return none)
Organisation/views.py:
Fixed a function that kept calling itself forever. It now returns the variable it sets in the function.

Can you make separate commits for each separate problem that SonarCloud shows? That way if we need to revert one of the changes if necessary without having to then re-alter the other two things. In this PR would then expect:
chore: remove unused test in test_serializers
chore: ... etc

relevant code moved to #2606 - others will be cherrypicked on a bugfix branch


Feature request: Proxy only for initial request/dial out

A flag to specify a proxy to be used only for the initial request/dial out to Hola's servers and to make the connection, and all subsequent requests be made without it. Could help with things like: https://github.com/Snawoot/hola-proxy/issues/90

Could help with this too:
MAIN    : 2023/09/24 17:30:25 main.go:208: INFO     Initializing configuration provider...
CRED    : 2023/09/24 17:30:26 credservice.go:33: ERROR    Configuration bootstrap error: temporary ban detected. Retrying with the fallback mechanism...
CRED    : 2023/09/24 17:30:30 credservice.go:33: ERROR    Configuration bootstrap error: temporary ban detected. Retrying with the fallback mechanism...
CRED    : 2023/09/24 17:30:31 credservice.go:33: ERROR    Configuration bootstrap error: temporary ban detected. Retrying with the fallback mechanism...
CRED    : 2023/09/24 17:30:33 credservice.go:33: ERROR    Configuration bootstrap error: temporary ban detected. Retrying with the fallback mechanism...
CRED    : 2023/09/24 17:30:35 credservice.go:33: ERROR    Configuration bootstrap error: temporary ban detected. Retrying with the fallback mechanism...
CRED    : 2023/09/24 17:30:37 credservice.go:33: ERROR    Configuration bootstrap error: temporary ban detected. Retrying with the fallback mechanism...

I am receiving these messages after 24 hours of use with the default options.


Add Material UI Styled Example

Add Razzle Material UI Styled Example as a submodule to the examples.

Thanks for this.
A couple of things:

The submodule approach is not going to work with the new create-razzle-app #267 that's coming because of how the install from example flag (--example) will work.
I think this should be split this into two distinct examples: Material UI (with-material-ui) and Styled Components (with-styled-components).

I see, I can add it directly as well.
I see. Basically I have created this example project for my self. So from now on I can start to work on a new project of mine. Which will be a more current version of Search is Back! with more possibilities by the way.
What do you think about adding it as a combination? Like with-material-ui-with-styled-components or like with/material-ui-and-styled-components.
However in this case the full name would be something like this: with-material-ui-and-styled-components-and-compression-and-html-minifier
Note: I am leaving out Express because it is part of the basic example too and it is just called basic not with-express for example.
In this case we could might shorten the -and- parts in the folder name like with-material-ui--styled-components--compression--html-minifier or with-material-ui.styled-components.compression.html-minifier.
I can split my project, but in this case should I split compression and html-minifier as well or they can be included because they related to Express so it is might considered a basic thing. What do you think?

The styled-components example should only include the code required to use styled component isomorphically. It should not include any other deps. Same goes for MUI.

I see. I will may add these 2 separately then sometime.


fix/envoyconfig-field-name

The name of the property was missing an 's'
/kind bug
/priority important-soon

/auto-cc

/lgtm

/ok-to-test

/approve


Is this project still actively maintained?

@bastilimbach For over a month there are two pull request waiting for your feedback as maintainer of this project. So I'm asking if you still consider maintaining this project? Or if the community should focus on a fork or other docker images for Magic mirror such as the one of @khassel https://hub.docker.com/r/karsten13/magicmirror/? I don't want to bother you, I just want to know the current status.

I believe it's safe to assume this project is no longer maintained.

Ok, then we need a new project which maintains the "official" docker image for MagicMirror. I think it's best to start as a fork of this project. I can be a maintainer, @sthuber90 and @khassel would you also like to be maintainers?

This may should be discussed as issue in the mm project.

I looked at the mm project repo and there is no real reference to the docker image, only in the changelog there is a link to this project. So I think they are not really interested in this.

see https://docs.magicmirror.builders/getting-started/installation.html#alternative-installation-methods

OK then we should move this discussion

Should we open an issue or create a forum post?

Maybe the forum is the best option as it is not directly related to the MM repo?

I think the issues on the repo are only for bug reports

not only real issues, we had already some general discussions there and I think it is the best place to discuss this with the main contributors and MichMich (only some of them are active in the forum). If its not the right place, they will tell us ...