Report code coverage when building

Codecov Report

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

:exclamation: Current head 9530e8d differs from pull request most recent head b23f5a7. Consider uploading reports for the commit b23f5a7 to get more accurate results

@@           Coverage Diff           @@
##             main      #96   +/-   ##
=======================================
  Coverage        ?   34.83%
=======================================
  Files           ?       42
  Lines           ?     3014
  Branches        ?        0
=======================================
  Hits            ?     1050
  Misses          ?     1964
  Partials        ?        0

Continue to review full report at Codecov.

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


refactor: ParquetExec logical expr. => phys. expr.

Which issue does this PR close?
Closes #4695.
Rationale for this change
Use Arc<dyn PhysicalExpr> instead of Expr within ParquetExec and move lowering from logical to physical expression into plan lowering (e.g. ListingTable).
This is in line w/ all other physical plan nodes (e.g. FilterExpr) and simplifies reasoning within physical optimizer but also allows correct passing of ExecutionProps into the conversion.
What changes are included in this PR?
 The tests still use logical expressions because they are easier to construct and its a simple conversion from logical to phys. (at least in the test cases).
Are these changes tested?
All existing tests pass. Removed tests that no longer apply. Added one to demonstrate PruningPredicate behavior when no columns are referenced (see https://github.com/apache/arrow-datafusion/pull/5386#discussion_r1117641476 ).
Are there any user-facing changes?

Phys. plan printouts look slightly differently.
BREAKING: Serialization has changed.
BREAKING: ParquetExec arguments changed.

I've also fixed a few test expressions that where buggy (or at least would require type coercion which would run on the logical plan), most notably:

negate(cast(some_int to UTF8)) => cast(negate(some_int) to UTF8)
some_bool.and/or(some_int % 2) => some_bool.and/or(some_int % 2 == 0)


Steam Deck - Windows - Input not detected from SWICD gamepad driver

Steam Deck Windows 10
Tried v1.2.0 and v2.0.0 beta of the xcloud client
Using v0.3.1 of the Steam Deck windows user mode driver here - https://github.com/mKenfenheuer/steam-deck-windows-usermode-driver

After launching and connecting to my console, most inputs from my Steam Deck are not recognized for the remote play session. Cannot scroll through the dashboard or interact with anything.
Using the official Xbox App to remote play works as expected. All other Windows games I've tried also detect the controller without issues.

The face buttons (A,B,X,Y) appear to work fine. Just cannot give any directional input


Hierarchical State Nodes - validation and lookup of state values

A means is required to resolve a state value (in the form of a full state ID) from a custom ID.  (see also Issue #4 ) And ideally a means is required to validate that a given state value exists in the machine config.
A solution approach was chosen using an initial pass thru the machine config to store state values and custom ids with thier corresponding paths (full state value Ids).   A recursive function getStateMap() creates a state mapping object with keys identifying the state ids (inc custom ids) with values containing the paths to those ids. The resulting state mapping object stateMap is built upon machine creation. using getStateMap(),  and attached to the initial and all subsequent transitioned states as an additional property.
An additional runtime function validateStateID()  validates/converts and returns full state value from varying types of state reference (eg custom id)  using lookups into stateMap.

The following function returns the stateMap object as above, including the initial mapping (see #4).
/* fsmPlus Function - returns a map of all state nodes in a machine
  * in the form of an object with a key set to ID of each state node (custom ID or full def ID).
  * the keys value is an object with optional properties:
  *  - pathID: state ID if state node ID is a custom ID
  *  - initial: id of the state nodes initial state/substate when node is compound
  *  - type: future uses ?
  * a recursive function , Called with arg obj set to machine config object, other args are empty on initial call
  * see functions: validateStateID(), for usage
*/
function getStateMap(obj, sofar,stateMap) {
  if (sofar === undefined) {
    if (typeof(obj.id) === 'undefined') {
      throw new Error ('(getStateMap) - Machine config requires an ID at top level');
    } else {
      sofar = ['#' + obj.id];
      stateMap = {};
    }
  }
  if (obj.states !== undefined) { // to do simplify
    Object.keys(obj.states).forEach(function (k) {
        if ('id' in obj.states[k]) { // add id as a custom
          node = '#'+ obj.states[k].id;
          stateMap[node] = {};
          stateMap[node].pathID = sofar.concat([k]).join('.');
        }
       else {
         node = sofar.concat([k]).join('.');
         stateMap[node]= {};
       }
       /* check if object has substates and assign type and initial */
        if (obj.states[k].states) {
          stateMap[node].type = 'C';
          stateMap[node].initial = getInitial(obj.states[k],sofar.concat([k]).join('.'));
        }
        else stateMap[node].type = 'A';

        if (typeof(obj.states[k].states) === 'object') getStateMap(obj.states[k],sofar.concat([k]),stateMap);
        else return stateMap;
    });
 }
  return stateMap;
}
`

the following function provides a runtime lookup into stateMap for validation and conversion of:

custom Ids
initial values (when called with mapInitial = true).

/* fsmPlus - validates/converts and returns full def ID from varying types of state reference
 * using lookups into stateMap. expects arg stateRef is in form of a full state id or a custom id
 *  - in all cases returns null if resolved state does not exist in stateMap
 *  - returns stateRef as full ID if exists as key in stateMap
 *  - returns full ID lookup in stateMap if stateRef is a custom ID
 *  - returns stateRef as full ID if stateRef is pathID of a custom ID
 *  - returns initial lookup in stateMap when mapInitial is true (used validating target paths)
*/
function validateStateID(stateRef,stateMap,mapInitial) {
  if (!(stateMap[stateRef])){
    /* stateRef not a full state ID - maybe a custom id */
    if (id = Object.keys(stateMap).find( k => stateMap[k].pathID === stateRef)){
      /* id assigned to key of a custom id */
      if (mapInitial && stateMap[id].initial) return stateMap[id].initial
      return  stateMap[id].pathID
    }
    else return null ;
  }
  else if (mapInitial && stateMap[stateRef].initial) return stateMap[stateRef].initial
  return (stateMap[stateRef].pathID) ? stateMap[stateRef].pathID : stateRef
}


removed MAC address checking, added check for port (62078) on iPhone

I was able to verify that this port is open even on a freshly restored iPhone.

Thanks for the patch!

We have test coverage on this file internally too; I'll try to get that open sourced soon...


[nrf fromtree] net: lib: coap: Use zsock_ functions

Use Zephyr internal zsock_ calls to remove dependency to NET_SOCKETS_POSIX_NAMES.

@rlubos can we get this merged now?


vendor: Vendor latest virtcontainers.

This is mainly for the changes introduces in hyperstart container
for specifying system mount information
Virtcontainers Shortlog:
71ce0c8 proxy: Implement Kata Containers proxy
8f9a2e0 proxy: Implement no_proxy generic case
2e2d33c agent: Introduce new functions to agent interface
05cec52 hyperstart: Change the path where rootfs is bind-mounted
d0051aa agent: Pass information about  the "-v /dev:/dev"  case
bdc815d mounts: Deduce additional information about system mounts
7c88153 vendor: switch to govmm
Signed-off-by: Archana Shinde<EMAIL_ADDRESS>

@sameo @grahamwhaley Can you take a look?

LGTM

lgtm


ghostText doesn't work with tiddlywiki

Setup
Browser: Chrome
Editor: macvim
Description
clicking on the ghosttext icon to launch ghosttext in tiddlywiki doesn't seems to have any effect.

Have you tried in Sublime Text?

Can you test the new version? It hasn't been published to the stores yet: https://github.com/GhostText/GhostText/issues/172


Handling --volume in kafka-mesos

We currently use a custom docker volume driver to mount remote/NFS directory inside docker container and was wondering how does one handle the
--volume              pre-reserved persistent volume id

flag in the CLI.
Also, what would be the pros or cons of making the framework spawn brokers inside a docker container?

Much better volume support is coming soon!  It sounds like for your use case you want to use the docker volume support built into mesos, thats next on my list of todos for docker support.


[FEATURE] Keep the original line break

Is your feature request related to a problem? Please describe.
I'm always frustrated when I find that after running this tool, the line break are all replaced by CRLF on my windows PC.
Describe the solution you'd like
Keep the original format of line break.

@BoYanZh Hi, I'm gonna work on that as soon as possible, thanks for being here!

https://github.com/hadialqattan/pycln/blob/db98579c0380b333441a6ca3b6ff8c3f6f7b9af0/pycln/utils/refactor.py#L113
After investigating, the problem is splitlines in Python uses CRLF for WindowsNT and LF for Unix-based systems by default!

@BoYanZh Please checkout Pycln beta.3 release!

Thanks! I really appreciate that.


Added cinsts in Readme

Squashed commits

You are going to need to learn how to do rebases and pushes back to the same branch at some point - it is a staple of git development. Let me know when you are ready to learn and we can walk through it together.

I am learning so lets not make a big deal out of this. I am feeling you are blowing it out of proportion.

Closed for the same reasons as https://github.com/chocolatey/chocolatey-test-environment/pull/16.


musllinux wait a little?

the musllinux builds take the majority of CI time, trumping even windows with openssl source builds etc.
the main issue seems to be that numpy has no binary wheels yet and the source build takes a good 6 mins for each python minor version. the issue is https://github.com/numpy/numpy/issues/20089 but not sure, doesn't look like there's been much traction. though they apparently already transitioned to use cibuildwheel for distribution but don't use it to build musllinux yet..
just a question if those wheels are important to you or if it could wait until there's numpy wheels for alpine? would save a few CI hours:)

Musllinux should already be turned off in pyproject.toml.
Haven't checked. Are you saying it is building on github actions?

that's what I thought, but maybe I was looking at an older build. you're right, it's actually turned off in the toml.

actually yes, see https://github.com/jonathf/pyvroom/actions/runs/1638879674. took > 30 mins for linux. the last full release build built all musllinux wheels


AssemblyDiagnosticsAccessor.AssemblyTitle returns product name, not title

I'm not sure how best to implement this fix or if a fix is truly necessary (AssemblyTitle is not ultimately used anywhere, at least in NetSparkle's code). Is a valid solution to load the assembly from disk, then use Assembly.GetCustomAttributes to get the title?

Would this code work?
var assembly = Assembly.LoadFrom(assemblyName);
Attribute attr = assembly.GetCustomAttribute(typeof(AssemblyTitleAttribute));

I really don't know enough to answer that question.

Not actually a problem -- AssemblyDiagnosticsAccessor is purposefully not using reflection. Assembly.LoadFrom returns reflection info, which we are avoiding, so we can't get the title this way. So, for now, I'm calling this "not a problem."


Update address form autofill

I want to merge this change because... it updates address form autofill with latest autocomplete spec https://html.spec.whatwg.org/multipage/form-control-infrastructure.html#autofill

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://qa.staging.saleor.cloud/graphql/

Task linked: SALEOR-2868 Autofill- swapped Country area and country fields


Unable to display calendar style and pop up

Describe the bug
Hello,
I can't display the style of the calendar and pop up. I tried to implement the style-css module without success.
I'm not using JQuery.
I am using de lastest version (v1.13.0).
Screenshots
Here is the display I get after implementing the library and adding imports:

@charlottecol123
Hello, I'm afraid that you don't give us enough information to check this issue.

I don't understand what "displaying the style of the calendar and popup". Does that mean you cannot customize styles of calendar and popup?

If so, you can read more about theme here

If you're saying about the other bug, Please describe details about the bug and steps to reproduce it. Such as,

What you really intended
What happened at real
The code example when you facing the bug
(Optional) How far did you even tried to resolve this problem

Hello, thank you for your answer.
Here is the display I would like to have :

And also with the different Popup

But for the moment I can only display this :

And when I click on an event or a place to create an event, the popups do not appear.
I installed the packages by using "npm install tui-calendar"  but when I'm trying to import these lines :
import "tui-calendar/dist/tui-calendar.css";

// If you use the default popups, use this.
import 'tui-date-picker/dist/tui-date-picker.css';
import 'tui-time-picker/dist/tui-time-picker.css'

I have this error that appears three times :
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|  */
| /* @charset 'utf-8'; */

I think the problem must come from the webpack.config.js.
Here is the code for the module part :
module: {
    rules: [
      {
        test: /\.(ts|js)$/,
        //test: /\.js$|jsx/,
        use: "ts-loader",
        exclude: /node_modules/,
      },
      {
        test: /\.styles.scss$/,
        // test: /\.css$/,
        use: [
          "sass-to-string",
          {
            loader: "sass-loader",
            options: {
              sassOptions: {
                outputStyle: "compressed",
              },
            },
          },
        ],
      },
      {
        test: /\.(scss|css)$/,
        exclude: [/\.styles.scss$/, /node_modules/],
        use: [MiniCssExtractPlugin.loader, "css-loader", "sass-loader"],
      },
      {
        test: /\.(woff|woff2|eot|ttf|otf|png|svg|jpg|jpeg|gif)$/i,
        type: "asset/resource",
      },
    ],
  },

The farst I've gone is trying to modify the css of the webpack.config.js.
Thank you again for taking the time to help me !!

@charlottecol123 Yeah, you picked the right point about this problem.
At the block of handling external css files like .scss & .css, you're excluding node_modules folder. Even though tui-calendar and their friends' css files located somewhere in node_modules folder.
So try to remove /node_modules/ regex in the exclude array.
Besides that, you also have to enable options in order to use popups.
const calendar = new Calendar('#app', {
  defaultView: 'week',
  useCreationPopup: true,
  useDetailPopup: true
});

Those options are disabled by default. Please check out the docs.

Hello,
Thank you again for your answer.
Indeed, removing /node_modules/ did remove the error. But that didn't bring out the style.
To be able to display it, I add the default.css file that I imported as follows:
@use "./default.css" as *;
I have the right style that is displayed.
Thank you for your time !

@charlottecol123 I see. That looks like you need to import related CSS files into your scss(sass) file so that your project environment can recognize third-party styles. Good to know :)


use display_data in plots.force if available

Fix #2096
Changed plots.force to use display_data in preference to data if it exists.

Great. Thanks! Just remerging with the latest master and then I will merge here.


Widget api call not working because of wrong url

@dirkjanfaber I noticed today that the url is not correctly build. With an & instead of an ? at the start of the parameters.
I allready created the pull request for the fix #21 ;)
Can you take a look and approve it?

Thanks. I'll create a new release.


 He didn't know Emmet exists, so he wasn't familiar with expansion on tab, he expected 4 spaces to be inserted. After his frustration, I guided him in the settings to turn of Emmet.
A solution I am thinking of would be to ask the user if he or she would like to use Emmet the first time a Emmet compatible file is opened. To give a user more context on what Emmet is, it could be useful to show build in extension in the extension view. The message could have a button which opens the Emmet extension page.
An extension page also solves a second concern. In my experience, for a new user it is weird turning self installed extensions off through the extension view and build in extensions trough settings.
Steps to Reproduce:

Normal Emmet expansion on tab in an html or php file

@svenvNL Thanks for reporting this
Emmet today is not implemented as an extension. It is built-in inside VS Code, and so has a setting to turn it off.
We are working on moving away from this tight coupling and move emmet to its own extension.
You can see this in the upcoming release of 1.13 (which you can try in the current Insiders build) after adding the setting emmet.useNewEmmet to true.
With this, emmet expansion will show up in the suggestion list instead of expanding on TAB.
As for the showing of built-in extensions in the extensions page and the ability to disable them, can you open a new issue?

@ramya-rao-a,
Thank you for your response, created #27935


pod install fails on RN 0.72.6 and lib version 2.0.5

react-native 0.72.6
react-native-ios-context-menu v2.0.5
react-native-ios-utilities v4.1.0
Xcode Version 15.0.1 (15A507)
pod 1.13.0
[!] CocoaPods could not find compatible versions for pod "ReactNativeIosUtilities":
  In Podfile:
    ReactNativeIosUtilities (from `../node_modules/react-native-ios-utilities`)

Specs satisfying the `ReactNativeIosUtilities (from `../node_modules/react-native-ios-utilities`)` dependency were found, but they required a higher minimum deployment target.

Let me know if I can provide any additional context.

Needed to install expo and expo modules in my bare RN project 🤦🏻♂

I have Expo installed and this is still not working.
[!] Unable to find a specification for `ReactNativeIosUtilities` depended upon by `ReactNativeIosContextMenu`

@danielrhodes
Yes. I have Expo and its modules installed. I'm still getting this error on installing pods.
[!] Unable to find a specification for `ReactNativeIosUtilities` depended upon by `ReactNativeIosContextMenu`

You have either:
 * out-of-date source repos which you can update with `pod repo update` or with `pod install --repo-update`.
 * mistyped the name or version.
 * not added the source repo that hosts the Podspec to your Podfile.

Couldn't install: ReactNativeIosContextMenu » ReactNativeIosUtilities. Updating the Pods project and trying again...
> pod install --repo-update
Using Expo modules
[Expo] Enabling modular headers for pod ExpoModulesCore
[Expo] Enabling modular headers for pod ReactNativeIosUtilities
[Expo] Enabling modular headers for pod DGSwiftUtilities
[Expo] Enabling modular headers for pod ContextMenuAuxiliaryPreview
Auto-linking React Native modules for target `ChatwootApp`: RNFlashList, RNGestureHandler, RNReanimated, RNSVG, RNScreens, react-native-blur, react-native-keyboard-controller, react-native-menu, react-native-pager-view, and react-native-safe-area-context
Framework build type is static library
[Codegen] Generating ./build/generated/ios/React-Codegen.podspec.json
Updating local specs repositories
Analyzing dependencies
[Codegen] Found FBReactNativeSpec
[Codegen] Found rncore
[!] Unable to find a specification for `ReactNativeIosUtilities` depended upon by `ReactNativeIosContextMenu`

You have either:
 * mistyped the name or version.
 * not added the source repo that hosts the Podspec to your Podfile.

Couldn't install: ReactNativeIosContextMenu » ReactNativeIosUtilities. Updating the Pods project and trying again...
Command `pod install` failed.
└─ Cause: Unable to find a specification for `ReactNativeIosUtilities` depended upon by `ReactNativeIosContextMenu`

You have either:
 * mistyped the name or version.
 * not added the source repo that hosts the Podspec to your Podfile.

Packages Installed
    "expo": "^49.0.16",
    "zeego": "^1.7.2",
    "@react-native-menu/menu": "^0.9.1",
    "react-native-ios-context-menu": "^2.3.0",


quickstart template deployment fails on Splunk 8.2.0

Hello,
I'm trying to deploy quickstart template in a new VPC following this steps mentioned here in us-east-1 and i selected c5.4xlarge instance for Splunk launch on EC2.
https://aws-quickstart.s3.amazonaws.com/quickstart-splunk-enterprise/doc/splunk-enterprise-on-the-aws-cloud.pdf#[{"num"%3A25%2C"gen"%3A0}%2C{"name"%3A"XYZ"}%2C69%2C709%2C0]
I'm getting the below error in SplunkCM
API: ec2:RunInstances Not authorized for images: [ami-0484972f36720ea7f]
I don't see this ami in my account. In my AWS account, I launched splunk on  ami-0a5b63581103ff2ae. Not sure why its picking the wrong ami.
Anyone else facing the same issue? Any pointers would be helpful!

replace in splunk-enterprise-template.yaml replace AMI number with new one you can find in AWS Marketplace after selecting Splunk ami+your AWS region

Hi everyone,
Having the same issue when I use the new AMI version 8.2.2 Ami Id: ami-04bd6f47d7c4931fc, it seems like the ansible playbooks runs after of user data code.
These lines can be seen in the /var/log/cloud-init-output.log
mv: cannot stat ‘/opt/splunk/etc/passwd’: No such file or directory
/var/lib/cloud/instance/scripts/part-001: line 17: /opt/splunk/etc/system/local/user-seed.conf: No such file or directory
sed: can't read /etc/init.d/splunk: No such file or directory
service splunk restart
Redirecting to /bin/systemctl restart splunk.service
Failed to restart splunk.service: Unit not found.
Like sourabhparvatikar said in his post https://github.com/aws-quickstart/quickstart-splunk-enterprise/issues/41 it's like splunk is not installed in the AMI.


Remove unused amdName property from package.json. Fixes #613

Coverage remained the same at 97.695% when pulling 1d9a4e34380cad6f790d8193b1b82ec75e7d7f5e on remove-amdname into 1444485fd482536717114ab675c9637c672939c9 on master.


Saving worksheet that contains image cells

Save images in .RGF format

Original post at https://reogrid.net/forum/viewtopic.php?id=504

Has the above problem been solved?

hello,
Excuse me, can I save images in .rgf file now ?

I think it is not supported yet.

Implemented by #200


MAE computation

Hi DCD-FG devs, I really admire your work on this project. I've been briefly trying it out over the past couple of days. From run_perturbseq_linear.py, I can see how to train it and how to compute log likelihoods on held-out expression. But how do I predict expression? Is there a built-in method to maximize eq. 8 with respect to most entries of X, while holding constant a few entries of X (those under intervention), and the graph structure, and the coefficients? EDIT: Or is there a way to sample non-intervened entries of X? I assume this would be needed to compute the MAE. Thanks!

I'm new to PyTorch but starting to understand the implementation better. Would this be a reasonable strategy?
def simulateKO(trained_model: MLPModuleGaussianModel, control_expression: np.Array, KO_gene_idx: int):
    x <- control_expression
    until convergence:
        x[KO_gene_idx] = 0
        x <- trained_model.module.forward(x)

We spoke about this on the phone and for anyone reading this in the future, the MAE results in the paper are based on conditional likelihood, not the steady state. In other words, rather than continuing the loop above "until convergence", only a single iteration is done.


socket.io.js not found when styleguide is put on server

Hey,
I don’t understand how socket.io.js is being referenced from index.html. I see these 3 lines at the bottom of the index.html:
<script src="/js/vendor.js"></script>
<script src="/js/app.js"></script>
<script src="/socket.io/socket.io.js"></script>

The /js/ folder being referenced exists. The /socket.io/ folder doesn’t. The styleguide works locally but not when I put the output folder on the server. I get this error:
GET https://styleguide.proxi.tools/socket.io/socket.io.js 404 (Not Found)
Is there something I need to do for installing socket.io.js?

Are you sure you need socket.ip on server? Usually for public hosting styleguide is generated as a static website. If you do not use server option, the tool makes static style guide web site in the given folder. Look like I do this https://github.com/varya/varya.github.com/blob/develop/gulpfile.babel.js#L14 The result is here http://varya.me/styleguide/#/
Will this help you?


"Leave an Issue" link on "Thanks for filling out survey" seems to 404?

Thanks, closing this as duplicate.


Console.ReadLine()  is not supported

Hi, I'm I'm trying to use try.net, I found that Console.ReadLine() cannot be used, So can I use it later? I hope try.net can be more perfect, thanks very much!

Supporting Console.ReadLine() would require try.net to leave a running instance of the application running on their server, awaiting I/O.  I don't think this feature is likely.

@briangithex This is exactly right ... for the server-side execution model. But we've been shifting code execution to the browser using Blazor, and we hope to revisit this.

Would love to have support for this for porting intro to C# courses. A lot use ReadLine for more advanced "hello world" scenarios. Some kind of virtual console they could type into would be amazing.

@jimbobbennett I've found that by re-thinking the tutorial code somewhat, I can avoid the use of Console.Readline. It may make this less important.
I'm also experimenting with more advanced tutorials where I use a data file (JSON, CSV, whatever...) and have readers open VSCode to modify input. There's a clunkiness there, but it opens a lot of possibilities.

We do want to add support for this in dotnet try. See #230.

We want to adopt dotnet try in our NServiceBus samples. It basically requires 'endpoints' to run until stopped, which in a console app is achieved by a Console.ReadLine().

Tried it today and this feature still not supported.

I am currently working on interactive materials to help teach C# to undergraduate game design students. Try .NET seems like it could be the best candidate for integration in our Learning Management System, but there is a user experience hurdle that is dissuading me from adopting it. Namely, while Console.ReadLine() works, it does not work as one would expect.
When the program reaches a Console.ReadLine() call, a browser pop-up asks for user input, but after sending input with a press of the enter key, the browser creates another input window and does not continue executing the code from the window-triggering ReadLine call. The user must hit the escape key to stop input, then the program continues running.
More confusingly, if the user inputs text 5 separate times (pressing the enter key to send input to Try .NET), these appear to be saved in a queue/buffer which ReadLine pulls from each time it is called in the program. This means a program with a single ReadLine will read the first entry in the queue, with any remaining input still in the queue. The program can finish, then a subsequent run of the program will not prompt the user for input but instead read directly from the queue. This becomes very confusing if the user accidentally inputs something multiple times as the program will not ask for input. In cases where programs have multiple ReadLine calls, the issue is more unintuitive as subsequent calls read input from a previous user context.
In essence, the issue I see is Try .NET's Console.ReadLine() handling does not reflect the same user experience one has with a console program offline. For myself, this is a deal breaker as we are teaching students to make interactive software. The current input handling does not satisfy that needs of an interactive console program. I wish it would, since otherwise it is quite good.
The browser's I tested in include:

FireFox: 110.0.1 (64-bit)
Chrome: 110.0.5481.104 (Official Build) (64-bit)

There are a number of known limitations with Try .NET for education scenarios. It's really oriented more towards documentation.
Have you taken a look at Polyglot Notebooks?
(Console.ReadLine also doesn't work there, for different reasons, but in the meantime it does support user input gestures.)

I would love some movement on Console.ReadLine() becoming available in Try . NET.

This would really be helpful also for experiments with string encodings directly in the web browser.


【AT】build undo log by delete target SQL

What would you like to be added:

Why is this needed:

please assgin to me

please assgin to me

looking forward your contribution


Support multi-root workspaces

Brief Issue Summary
Support the new multi-root workspaces
Other Notes/Information
Right now cmake-tools always builds the first directory.
VSCode now supports multiple workspaces, it would be nice if cmake-tools would use the CMakeLists.txt in the folder the user is currently in/the folder with the file the user currently has opened.
https://github.com/Microsoft/vscode/wiki/Extension-Authoring:-Adopting-Multi-Root-Workspace-APIs
Thanks

This is a big one, and I knew it would come up eventually. I'm in the middle of a refactor and am afraid this will have to wait for a while, but I really want to get to it as soon as possible.

@vector-of-bool Does your refractoring touch the multi-root thing, or can I start working on it based on your work?

@go2sh: I've done no investigation into what it would take to do multi-root. Feel free to investigate, just work on the refactor branch, since that's where the develop will be updated soon.

This is something that cmake tools NEEDS.

I've done no work on it, but I'm hoping to onboard some more collaborators and we'll hopefully have time to take a look at this.

Sorry all for the down time on this. I'm finally getting around to it. Expect something in the 1.1.0 update.

Turns out to be more work than anticipated. Lot of stateful stuff has to be moved around. This will be the big one for 1.2, since it's holding back the 1.1 and I have other things I want to get in user's hands for 1.1.0.

I've noticed that, in addition to the issue mentioned above, the automatic Intellisense configuration also breaks down when using a multi-root workspace, when an application in one workspace folder includes a library in a different folder.
I'm assuming this falls under the general "support multi-root" scope of this issue, just wanted to add my +1. Thanks!

This would be a mighty useful addition.

I created a new cmake extension about a year ago, because the pain to add this here was to big for me. It has nearly the same features plus a few extras, like multi-root, tasks support, finer configuration options and cmake file api support. The server api used here will be dropped in future versions of cmake.
Search for cmake integration in the vscode extensions panel.

We are planning on adding multi-root support soon, though I don't expect it to be ready for 1.2.0.

I have a multi-root workspace, where one of the roots is for Python code, one is for CMake, and one is just for a reference to common headers that are not under the CMake root. I use the 1.3.0-beta of CMake Tools and it works fine. It wold be even better if CMake Tools did not complain or otherwise react on roots that don't have CMakeLists.txt file.

Thanks for the feedback @astraujums. I have moved your comment to a new issue.

This was addressed in 1.3.0


Add a basic AIOHTTP integration

This PR adds a basic AIOHTTP integration, tests and documentation.
Description
As mentioned in #437 AIOHTTP does not support ASGI and thus needs a separate integration. This PR adds a basic aiohttp integration with query, mutation and file upload support (I will possibly attempt to add subscription support in a separate PR). This PR also includes tests and documentation based on the Sanic integration tests and docs.
Types of Changes

[ ] Core
[ ] Bugfix
[ ] New feature
[x] Enhancement/optimization
[x] Documentation

Issues Fixed or Closed by This PR

#437

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver,so make sure to pick the appropriate type. If in doubt feel free to ask :)

LGTM other than the merge conflict +1

I just rebased my branch on main.

Tests fail, but I'm not sure if they're related to your changes...

@DoctorJohn there might be an issue with python 3.7, unrelated to this PR, maybe it was due to the merge of the latest typing_extensions. Will try to fix this tomorrow!
Error is
E     ../../../../../../Users/patrick/Documents/github/strawberry-graphql/strawberry/.venv/lib/python3.7/site-packages/pydantic/typing:131: error: Module 'typing' has no attribute '_GenericAlias' (diff)

But it works when importing normally:
❯ python
Python 3.7.10 (default, May  3 2021, 01:28:14)
[Clang 12.0.5 (clang-1<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from typing import _GenericAlias

Also this error doesn't happen when running mypy normally. So maybe something is wrong with the test plugin

Not sure why we get this now and only when running the tests, but the error seems to be trigger by typeshed not having some private classes defined: https://github.com/python/typeshed/issues/4694
I've ignored the error in pydantic, so we are good I'd say :)


[SUPPORT] Metadata Table causes missing data.

Describe the problem you faced
Regardless the table type ( CoW, MoR ), I notice missing data when Metadata Table is enabled.
For example, If I ingest 100,000 records ( no dups ) with the batch size 10,000, the ingested records in Hudi are not 100,000.
I checked the number or records through Amazon Athena and also double-checked the count by running Spark Job as well.
Full Configuration
{
	'className': 'org.apache.hudi'
	'hoodie.datasource.hive_sync.database': 'hudi_exp'
	'hoodie.datasource.hive_sync.enable': 'true'
	'hoodie.datasource.hive_sync.support_timestamp': 'true'
	'hoodie.datasource.hive_sync.table': 'hudi_etl_exp'
	'hoodie.datasource.hive_sync.use_jdbc': 'false'
	'hoodie.datasource.write.hive_style_partitioning': 'true'
	'hoodie.datasource.write.partitionpath.field': 'org_id'
	'hoodie.datasource.write.recordkey.field': 'obj_id'
	'hoodie.table.name': 'hudi_etl_exp'
	'hoodie.bulkinsert.shuffle.parallelism': '24'
	'hoodie.delete.shuffle.parallelism': '24'
	'hoodie.insert.shuffle.parallelism': '24'
	'hoodie.upsert.shuffle.parallelism': '24'
	'hoodie.index.type': 'BLOOM'
	'hoodie.bloom.index.prune.by.ranges': 'true'
	'hoodie.datasource.clustering.async.enable': 'false'
	'hoodie.datasource.clustering.inline.enable': 'false'
	'hoodie.datasource.compaction.async.enable': 'false'
	'hoodie.clean.automatic': 'true'
	'hoodie.clean.async': 'true'
	'hoodie.keep.max.commits': 40
	'hoodie.keep.min.commits': 30
	'hoodie.cleaner.commits.retained': 20
	'hoodie.cleaner.policy': 'KEEP_LATEST_COMMITS'
	'hoodie.compact.inline': 'false'
	'hoodie.clustering.async.enabled': 'false'
	'hoodie.clustering.async.max.commits': 4
	'hoodie.clustering.inline': 'false'
	'hoodie.metadata.clean.async': 'true'
	'hoodie.cleaner.policy.failed.writes': 'LAZY'
	'hoodie.write.concurrency.mode': 'OPTIMISTIC_CONCURRENCY_CONTROL'
	'hoodie.write.lock.provider': 'org.apache.hudi.client.transaction.lock.ZookeeperBasedLockProvider'
	'hoodie.write.lock.zookeeper.port': '2181'
	'hoodie.write.lock.zookeeper.url': 'zookeeper_url'
	'hoodie.write.lock.zookeeper.base_path': 'zookeeper_base_path'
	'hoodie.write.lock.zookeeper.lock_key': 'hudi_etl_exp'
	'path': 's3://hello-hudi/hudi_exp/hudi_etl_exp'
	'hoodie.datasource.write.precombine.field': '_etl_cluster_ts'
	'hoodie.datasource.hive_sync.partition_extractor_class': 'org.apache.hudi.hive.MultiPartKeysValueExtractor'
	'hoodie.datasource.write.keygenerator.class': 'org.apache.hudi.keygen.SimpleKeyGenerator'
	'hoodie.datasource.hive_sync.partition_fields': 'org_id'
	'hoodie.combine.before.upsert': 'true'
	'hoodie.datasource.write.operation': 'upsert'
	'hoodie.datasource.write.table.type': 'COPY_ON_WRITE'
	'hoodie.table.type': 'COPY_ON_WRITE'
	'hoodie.metadata.enable': 'true'
}

To Reproduce
Steps to reproduce the behavior:

generates random 100 records
ingest 10 records per batch
count number of ingested records ( 10, 20, 30 )

Expected behavior
The all 100 records have to be on Hudi Tables
Environment Description

Hudi version : 0.9.0

Spark version : 3.1.1-amzn-0

Hive version : 2.3.7-amzn-4

Hadoop version : 3.2.1-amzn-3

Storage (HDFS/S3/GCS..) : S3

Running on Docker? (yes/no) : no

Can you not set this config "'hoodie.metadata.clean.async': 'true'". This should not have been exposed to end user. it has to be false. Please remove this and try it out.

@nsivabalan
Thank you. I will try with "hoodie.metadata.clean.async": false
Yep. I will let you know the result.

@nsivabalan
Hi, I ran the test
Test1: no metadata table - OK
Test2: metadata table + hoodie.metadata.clean.async=true - Missing Data
Test3: metadata table + hoodie.metadata.clean.async=false - Missing Data
I still see the missing data issue when Metadata table is enabled.

ok. when metadata table is enabled, and metadata.clean.async is false:
can you post the contents of .hoodie
and .hoodie/metadata.hoodie
and we can go from there. Ensure you when you do "ls", you sort based on file last mod time btw.

@nsivabalan
Hi,
here is the content in .hoodie
    0 Feb 26 23:34 archived_$folder$
    0 Feb 26 23:34 .temp_$folder$
  514 Feb 26 23:34 hoodie.properties
    0 Feb 26 23:34 .aux_$folder$
    0 Feb 26 23:34<PHONE_NUMBER>3429.commit.requested
    0 Feb 26 23:34 .heartbeat_$folder$
48321 Feb 26 23:34<PHONE_NUMBER>3429.inflight
68948 Feb 26 23:35<PHONE_NUMBER>3429.commit
    0 Feb 26 23:35 metadata_$folder$
    0 Feb 26 23:35<PHONE_NUMBER>3548.commit.requested
47671 Feb 26 23:36<PHONE_NUMBER>3548.inflight
69025 Feb 26 23:36<PHONE_NUMBER>3548.commit
    0 Feb 26 23:37<PHONE_NUMBER>3706.commit.requested
48322 Feb 26 23:37<PHONE_NUMBER>3706.inflight
69957 Feb 26 23:37<PHONE_NUMBER>3706.commit
    0 Feb 26 23:40<PHONE_NUMBER>4055.commit.requested
    0 Feb 26 23:41<PHONE_NUMBER>4100.rollback.inflight
15444 Feb 26 23:41<PHONE_NUMBER>4100.rollback
50285 Feb 26 23:41<PHONE_NUMBER>4055.inflight
72571 Feb 26 23:41<PHONE_NUMBER>4055.commit
    0 Feb 26 23:42<PHONE_NUMBER>4225.commit.requested
48970 Feb 26 23:42<PHONE_NUMBER>4225.inflight
70917 Feb 26 23:43<PHONE_NUMBER>4225.commit
    0 Feb 26 23:44<PHONE_NUMBER>4418.commit.requested
47670 Feb 26 23:44<PHONE_NUMBER>4418.inflight
69084 Feb 26 23:44<PHONE_NUMBER>4418.commit
    0 Feb 26 23:46<PHONE_NUMBER>4617.commit.requested
47013 Feb 26 23:46<PHONE_NUMBER>4617.inflight
68146 Feb 26 23:46<PHONE_NUMBER>4617.commit
    0 Feb 26 23:48<PHONE_NUMBER>4817.commit.requested
49625 Feb 26 23:48<PHONE_NUMBER>4817.inflight
71895 Feb 26 23:48<PHONE_NUMBER>4817.commit
    0 Feb 26 23:50<PHONE_NUMBER>5018.commit.requested
45706 Feb 26 23:50<PHONE_NUMBER>5018.inflight
66287 Feb 26 23:50<PHONE_NUMBER>5018.commit
    0 Feb 26 23:52<PHONE_NUMBER>5217.commit.requested
48977 Feb 26 23:52<PHONE_NUMBER>5217.inflight
70975 Feb 26 23:52<PHONE_NUMBER>5217.commit
    0 Feb 26 23:54<PHONE_NUMBER>5416.commit.requested
  256 Feb 26 23:54 ..
  128 Feb 26 23:54 .aux
 1376 Feb 26 23:54 .
  192 Feb 26 23:54 metadata

here is the content of .hoodie/metadata/.hoodie
   0 Feb 26 23:35 archived_$folder$
 373 Feb 26 23:35 hoodie.properties
   0 Feb 26 23:35 .temp_$folder$
   0 Feb 26 23:35 .aux_$folder$
   0 Feb 26 23:35<PHONE_NUMBER>3429.deltacommit.requested
1187 Feb 26 23:35<PHONE_NUMBER>3429.deltacommit.inflight
2195 Feb 26 23:35<PHONE_NUMBER>3429.deltacommit
   0 Feb 26 23:36<PHONE_NUMBER>3548.deltacommit.requested
1905 Feb 26 23:36<PHONE_NUMBER>3548.deltacommit.inflight
2443 Feb 26 23:36<PHONE_NUMBER>3548.deltacommit
   0 Feb 26 23:37<PHONE_NUMBER>3706.deltacommit.requested
1905 Feb 26 23:37<PHONE_NUMBER>3706.deltacommit.inflight
2518 Feb 26 23:37<PHONE_NUMBER>3706.deltacommit
   0 Feb 26 23:41<PHONE_NUMBER>4055.deltacommit.requested
1905 Feb 26 23:41<PHONE_NUMBER>4055.deltacommit.inflight
2590 Feb 26 23:41<PHONE_NUMBER>4055.deltacommit
   0 Feb 26 23:41<PHONE_NUMBER>4100.deltacommit.requested
 526 Feb 26 23:41<PHONE_NUMBER>4100.deltacommit.inflight
1297 Feb 26 23:41<PHONE_NUMBER>4100.deltacommit
   0 Feb 26 23:43<PHONE_NUMBER>4225.deltacommit.requested
1905 Feb 26 23:43<PHONE_NUMBER>4225.deltacommit.inflight
2667 Feb 26 23:43<PHONE_NUMBER>4225.deltacommit
   0 Feb 26 23:45<PHONE_NUMBER>4418.deltacommit.requested
1905 Feb 26 23:45<PHONE_NUMBER>4418.deltacommit.inflight
2742 Feb 26 23:45<PHONE_NUMBER>4418.deltacommit
   0 Feb 26 23:47<PHONE_NUMBER>4617.deltacommit.requested
1905 Feb 26 23:47<PHONE_NUMBER>4617.deltacommit.inflight
2817 Feb 26 23:47<PHONE_NUMBER>4617.deltacommit
   0 Feb 26 23:49<PHONE_NUMBER>4817.deltacommit.requested
1905 Feb 26 23:49<PHONE_NUMBER>4817.deltacommit.inflight
2893 Feb 26 23:49<PHONE_NUMBER>4817.deltacommit
   0 Feb 26 23:50<PHONE_NUMBER>5018.deltacommit.requested
1905 Feb 26 23:50<PHONE_NUMBER>5018.deltacommit.inflight
2967 Feb 26 23:50<PHONE_NUMBER>5018.deltacommit
   0 Feb 26 23:52<PHONE_NUMBER>5217.deltacommit.requested
1905 Feb 26 23:52<PHONE_NUMBER>5217.deltacommit.inflight
3042 Feb 26 23:53<PHONE_NUMBER>5217.deltacommit
 128 Feb 26 23:54 .aux
1280 Feb 26 23:54 .
 192 Feb 26 23:54 ..

@yihua : Can you assist in debugging data loss issue w/ metadata table. this is hudi-0.9.0 btw.

@nsivabalan @yihua Thank you
Please let me know if you need anything from me about this issue.

@Gatsby-Lee : Can you zip just the .hoodie folder contents and give us. Guess ethan has some hudi-cli tools to inspect the timeline.
so, .hoodie of data table and .hoodie of metadata table as well.

@nsivabalan : Hi, ok. I will do that.
BTW, I don't have the data that I reported last time.
Let me run the experimentation again and populate the data and share with you.

@nsivabalan shared the requested content through slack channel.

@yihua @nsivabalan
I haven't really confirmed yet.
but, I feel this issue happens when the table uses table partition.

hey @Gatsby-Lee : will close out the github issue for now. Once you have confirmed the issue and when you have sample data, feel free to open up a new issue. we can definitely follow up.
thanks!


Add keepNames to esbuild options

Hi @geoffrich, thanks for the work you do here. It's a little change, but I hope it's correct. I'm pretty new to SvelteKit, so admittedly I'm not quite sure how I could go about testing this adapter locally. Double whammy, I've also never contributed to anything open source before. If I need to do more, please just let me know and I'll learn and give a go this weekend 🏎
My use case was mentioned in #148 --this is causing a bug that prevents me from logging entries on my grandma's TV app (that's secretly a SvelteKit website)
Closes #148
Mentioned in #51 #56

Change looks good! Perhaps also update the relevant README section?

@tlaundal added!

Sure thing @geoffrich ! I'll try it out today
Thanks so much for including the command

Looks like these changes work for me 🥳 Here are some details that (hopefully) make it easy for you to confirm these changes are indeed what fixed it for me
Demo with forked adapter (working)
https://black-mushroom-06e3d7310-56.centralus.3.azurestaticapps.net/ask
Send a message a see that it works just fine
FYI, I receive your IP address and the text you typed, but I don't do anything with it
Demo of adapter ^0.16.0 (not working)
https://black-mushroom-06e3d7310-57.centralus.3.azurestaticapps.net/ask
Send a chat message, observe scary error
TypeError: Expected signal to be an instanceof AbortSignal

I have the bad site PR'd off the good site so you can see that the only change is the adapter


large pip cache by using default python_packages in cog.yaml

remove cog container pip cache.
in the container i see a large cache for pip modules
3.5G    ./root/.cache

solution: use the no cache flag when installing pip modules
pip install ... --no-cache-dir

I believe this has been fixed since https://github.com/replicate/cog/pull/1902


Problems using the module build

We're trying to embed Neuroglancer (master) into an Angular project.
Attempt: Consume the module build
Building the module with npm run build-module worked fine. But following the readme from examples/dependent-project I was unable to get the example to work. I think it's because the example uses webpack, while the main project has switched to esbuild.
So the dependent-project seems unmaintained? (If so, maybe add a note about that?)
Importing and calling setupDefaultViewer() from Angular (via npm link) didn't work:

Got many different typescript and build errors. After fixing includes and tsconfig, the main issue was css/svg import statements in the built module dependencies. Angular's Webpack didn't want to resolve those (build errors). Plus some issues with the codemirror dependency that emitted compatibility-import statements that work in the browser, but caused errors from when bundling with Angular. After fixing those directly in node_modules, @babel/runtime was missing, etc.
Also types didn't work, but that's easy to ignore for now. I understand we cannot generate type definitions because of this typescript issue with mixins.
Conclusion: We probably should consume a build already bundled by esbuild.

Which implies that the Angular project shouldn't depend on neuroglancer's dependencies (package.json) any more.

Attempted to change config/esbuild/ to create a module build with bundle: true,. Got it to build, but didn't find the module entry point in the bundled result.

Do you think it is useful to proceed with that approach? (Would you want a PR doing a bundled module build?)
Alternative: Embed the main build via
This is described in this comment from issue 324 and works just fine so far. We can access the viewer object from the <iframe>'s window, configure it, even subscribe to events.
The main drawback seems to be that we cannot import and  control the initial config / viewer creation.

You are correct that the dependent-project example is out of date.  Note though that the module build is unrelated to that example --- it was added to support https://github.com/neuroglancerhub/react-neuroglancer
In general there are 3 ways to consume neuroglancer as a dependency:

Bundle it directly as part of your project's build.  Cleanest and most flexible but requires most complex build configuration.  This is a what the dependent-project did, but as noted it is now out of date.
Bundle the workers ahead of time, consume main thread sources directly as esmodules.  This is what the module build supports, but it is somewhat specialized for Janelia's use case currently.
Use an iframe.  Simplest but least flexible.

Thanks for clarifying. We're still playing around with those options.
With the iframe approach we also got types on the viewer object to work, via /// <reference path="../lib/neuroglancer/typings/index.d.ts" />. This required a lot of fiddling with tsconfig.json, probably because we're using typescript 4.7.2, while neuroglancer is at 4.3.2. So here are the relevant options that work for referencing an unmodified neuroglancer (v2.31):
  "compilerOptions": {
    "strict": false,
    "noImplicitOverride": false,
    "noPropertyAccessFromIndexSignature": false,
    "strictPropertyInitialization": false,
    "useUnknownInCatchVariables": false,
    "strictNullChecks": true,
    "noImplicitAny": true,
    "allowSyntheticDefaultImports": true,
    "esModuleInterop": true,
    "noImplicitReturns": true,
    "noFallthroughCasesInSwitch": false,
    "downlevelIteration": true,
    "experimentalDecorators": true,
    "noImplicitThis": true,
    "noUnusedParameters": true,
    "noUnusedLocals": true,
    "preserveConstEnums": true,
    "moduleResolution": "node",
    "importHelpers": true,
    "target": "es2020",
    "module": "es2020",
    "lib": [
      "es2020",
      "dom",
      "dom.iterable"
    ],
    "typeRoots": [],
    "paths": {
      "neuroglancer/*": [
        "lib/neuroglancer/src/neuroglancer/*"
      ]
    }

What would we need to get an actual node package for Neuroglancer working, so that it could be imported like any other dependency? Is the problem just the css files and the different entry points for the web workers?
Right now I'm essentially forking Neuroglancer in order to use it as a "dependency".


[Not Recognized Modifier] - #% increased Physical Damage

Not Recognized Modifier
Expert Forge Maul
#% increased Physical Damage
Item Text
Please provide the item's description

Can be copied with ctrl+shift+c
Can also be copied from below the error message

Item Class: Two Hand Maces
Rarity: Rare
Plague Roar
Expert Forge Maul
--------
Physical Damage: 172-233 (augmented)
Elemental Damage: 16-35 (augmented), 1-33 (augmented)
Critical Hit Chance: 5.00%
Attacks per Second: 1.05
--------
Requirements:
Level: 67
Str: 174 (unmet)
--------
Item Level: 70
--------
54% increased Physical Damage
Adds 16 to 35 Fire Damage
Adds 1 to 33 Lightning Damage
+16% to Critical Damage Bonus

Trade site request failed
Error: Unknown stat provided: explicit.stat_419810844

duplicate of #82, I will update the title on that one though so it is easier to tell what it is about


Allowing PIL image to get return in command

Tasks
Implement the support of Image type from the Pillow library.
You might want to use the standard IO and not saving the image afile.
from PIL import Image

...

@Client.command()
async def image_example(self) -> Image:
    img = Image.new("RGBA", (200, 100), "white")
    ...

    # returning a single image
    return img

@Client.command()
async def image_example(self) -> Message:
    img = Image.new("RGBA", (200, 100), "white")
    ...

    # Sending the image within a custom message, as a file
    return Message(..., file=img)

For the person implementing this
Following the Discord API, the implementation should be sending an array of attachments instead of a single file.
@Client.command()
async def image_example(self) -> Message:
    img = Image.new("RGBA", (200, 100), "white")
    ...
    return Message(..., attachments=[img])

https://discord.com/developers/docs/resources/channel#message-object
https://discord.com/developers/docs/resources/channel#attachment-object

I want to contribute so Im going to look at this


First Commit - Skeleton Django Project

Hey Everyone - Completed Create a new Django project
Please review
all additional information is in README.md

LGTM, but lets have someone else have a look :)


conflict with slash command

it seems obsidian core plugin "Slash commands" was broken when various complements plugin was enabled. conflicted?

@lifesign
What version do you use?

I could reproduce it 👍

@lifesign
I have released to v3.4.0 🚀
Please confirm whether it works as you expected.

seems ok, thx~

I'm having this problem still. Using a fresh vault with slash commands enabled and Various Complements installed.
Create a note titled 'Internal Link'. Then type /int for the Add Internal Link command.

Typing /in will show Slash commands
Typing /int will show only VC

Versions:

Obsidian 0.13.19
VC 4.2.0

@keithadams
Thanks. I could reproduce it.
But, strangely, it occurs only in a new vault. I couldn't reproduce it in my main vault... 🤔

@tadashi-aikawa
Interesting, I was having the problem on my main vault as well.
I installed the beta and it's fixed. Thank you very much!

Thank you for your confirmation!
I will close this issue after release v5.0.0.

I have released to v5.0.0 🚀
So closed this issue.


[backport] Use pull_request_target instead for auto notify bot

Backport of #5541 by @kmaehashi

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit cb30b1b85d8566f5026a47b824b61c63ad872d1a, target branch v9) failed with status FAILURE.


Tweaked documentation of Model's fit, evaluate and predict.

Clearly documented what all the options are for x and all the implications for other arguments.
Also made the documentation more consistent between the arguments and between fit, evaluate and predict.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 59.84%. Comparing base (b91cfe5) to head (327fc99).

:exclamation:  There is a different number of reports uploaded between BASE (b91cfe5) and HEAD (327fc99). Click for more details.
HEAD has 6 uploads less than BASE

Flag
BASE (b91cfe5)
HEAD (327fc99)

keras
4
1

keras-torch
1
0

keras-tensorflow
1
0

keras-jax
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #20454       +/-   ##
===========================================
- Coverage   82.01%   59.84%   -22.17%
===========================================
  Files         515      515
  Lines       47275    47275
  Branches     7422     7422
===========================================
- Hits        38772    28292    -10480
- Misses       6706    17237    +10531
+ Partials     1797     1746       -51

Flag
Coverage Δ

keras
59.84% <ø> (-22.03%)
:arrow_down:

keras-jax
?

keras-numpy
59.84% <ø> (ø)

keras-tensorflow
?

keras-torch
?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Model signature changed

I noticed that model classes (since 0.9?) do have I different signature than standard Yii2 model classes.
Why do they require a Finder as the primary parameter? I think it's not a good practice to change the constructor signature of a class unless absolutely necessary. Can't this simply be done with static call and/or a property?
And if it's required for a reason I do not see at the moment, why not use it as a second param?
Related to: https://github.com/dektrium/yii2-user/issues/212

From version 0.9.0 Yii2-user uses dependency injection container, so all controllers', models' and components' dependencies are resolved automatically.
The order of constructors' arguments does not really matter, they will be resolved correctly. The only reason why it is done in that way is that documentation provides such example, where dependencies go before config argument.
I dont's see any problems with overriding constructors in overriden classes. Base model class does not have any dependencies, and that is why its' constructor has only config param. But the child classes have dependencies on other classes. So we are adding new arguments to constructor's signature. By the way, this can be discussed. If you can provide any significant reasons why it is a problem, code can be refactored to solve it.

From the docs:

You can declare the dependency through a constructor parameter and let the DI container to resolve it for you.

But why do you even need it in the constructors?
Why can't you declare the properties as public and use DI to define the Finder, ... in your module or application config if needed?
Like so:
    /** @var Finder */
    public $finder;

    /**
     * @param Finder $finder
     * @param array $config
     */
    public function __construct($config = [])
    {
        $this->module = \Yii::$app->getModule('user');
        parent::__construct($config);
    }

    \Yii::$container->set(LoginForm::className(), ['finder'=>...])

Of course it can be done in that way. But it makes us to manually configure each class' dependencies. However constructor injection allows to leave dependency configuration to yii.
Moreover, I prefer constructor injection because in that case we already know that dependency injected and it is correctly typed, but in case of setter/property injection we should check if dependency has already been injected or not.
So I still don't see any problem with constructor dependency injection. It is very common practice.

I am not very proficient in DI, I stumbled upon this because I had to change my admin user setup after upgrading to 0.9.
I'd have expected to see additional optional parameters for DI, especially for classes derived from commonly used Yii classes.
But now the from-model classes form yii2-user all take different parameters $finder, $mailer, $config in various combinations.
It looks overly complicated to me, but as mentioned before, I may not see the full picture.
But these are just my 2 cents, feel free to close the issue.

all take different parameters $finder, $mailer, $config in various combinations.
That is why constructor injection used. In order to make yii inject them automatically. We don't care about how they injected and where are they getting from. We just use Yii::createObject() or Yii::$container->get() in order to make yii build object automatically.


Introduce a simple prefetching solution

Basically this is a universal solution which works everywhere.
We simply invoke a XHR for all the prefetching URLs.
But we don't parse JSON until the it's being used.
Fixes #956

This is related to #956. 🤖

@sarukuku added to the introduction text so it's linked 👍

@rauchg I did a load test. Basically, prefetch a lot of pages in a setInterval.
This is the Chrome's timeline profiler view.

Basically, it spend a pretty small(micro-secs) time to send the request and get it.
It's actually, smaller than the setInterval call.
(So, I think with the performance wise this is fine)

I wonder how this will affect the limit of concurrent HTTP requests on browser. (Is there the limit if we use service worker ?)

@nkzawa I think I need to find more about that.

By the way, if we're going to ditch the SW, I'm ok with prefetching by default and ditching next/prefetch.

@rauchg I am not sure about that.
We can move next/prefetch logic into somewhere elas.
But I assume, user still has to tell us what to prefetch or not.

Does ditching the service worker cost us any gains/losses vs using the service worker?

We think this only has upsides so far

@nkzawa actually it depends. Usually, it's around 4-8 connections per origin.
So, yes there's an issue with that. (But it applies for SW as well)
But I couldn't find any docs saying SW and limitations.
But with HTTP2 there won't be such problem because of multiplexing. But that'll affect the time to other resources.
So, here's my idea.

We can throttle to load just 2 JSON pages at once.
I'll move the codebase into the router.
I'll add prefetch APIs to the next/link.
Let's deprecate next/prefetch

@nkzawa @arunoda
Even with HTTP2 multiplexing you have some device dependent limits on the number of concurrent requests, but it goes much higher. It can become limited by the memory allocated for the responses, somewhere around 25mb of pre-allocated header memory can easily cause problems even on desktop browsers. But with that you can easily afford 100+ concurrent requests, which should cover most needs. If you need more, it makes sense to use websockets and keep track of requests and responses in js.
These limits should apply both to service workers and other http requests. Also, when running out of your per-process memory, you will get e.g. net::ERR_INSUFFICIENT_RESOURCES in Chrome/ium.
https://chromium.googlesource.com/chromium/src/+/58.0.3001.2/content/browser/loader/resource_dispatcher_host_impl.cc#2341
Anyway, throttling prefetching at 2 concurrent requests is probably a sane default, considering mobile.

@nkzawa Implemented what I suggested here.
I've changed the core router API to use fetch always.
That's cleaner.
I need to do few more tests and write some test cases.
In the meantime, could you check this?

I think xhr and fetch is not equivalent when you need to abort requests. abort should be more efficient ?

@nkzawa Yep. With fetch we can't really abort. My suggestion is we don't need to abort.
With this, it's discard the response.
Usually, it's very rare xhr abort to happen. Even we do that, it might have reached the server.
So we don't get a big win.
Since we also do prefetching and has aggressive caching this won't be a problem.

@nkzawa This is ready.

Let's take this and keep iterate on this.


Deploy Redis

Context
Deploy Redis
Changes proposed in this pull request
Add redis resource in aks using terraform-modules
Guidance to review
make development_aks terraform-apply
See the already deployed resources (s189t01-gitapi-dv-rg)
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Deploy redis


Disable submit button after Submit

Fixes problem with the original PR #636. Now button changes on submit not click

 Temporarily disable submit button to guard against double clicks


Directory Environment

Often we want to change environment variables for a particular directory (e.g. nvm, rvm). A directory environment, which would merge itself into environment and automatically clean up when you switch out of a CWD could make this task easier.

This should be compatible with the direnv tool (which does exactly this).
cf. https://direnv.net/

direnv doesn't appear to work in upterm


Query distribution regarding group

We decided to distribute the queries regarding their group.
We have to implement this.

This will be dealt in #997


[BUG] assert_fail is called

I keep getting assert_fail called in my app.  I am using vcpkg, version  <IP_ADDRESS>
I don't have a complete stack track, just

OpenImageIO.dll!fmt::v11::detail::assert_fail(const char * file, int line, const char * message) Line 38	C++
OpenImageIO.dll!fmt::v11::detail::fwrite_fully(const void * ptr, unsigned __int64 count, _iobuf * stream) Line 77	C++
[Inline Frame] OpenImageIO.dll!fmt::v11::detail::print(_iobuf *) Line 1707	C++
OpenImageIO.dll!fmt::v11::vprint_buffered(_iobuf * f, fmt::v11::basic_string_view fmt, fmt::v11::basic_format_argsfmt::v11::context args) Line 1714	C++

Should I define FMT_THROW to something, so it will throw vs. crash?  It is currently crashing my app.
I do not have a repro case, it occurs every 10 minutes or so in my app.
Windows 11

My guess is that somewhere there is an error message or some other formatting that isn't needed very commonly and isn't encountered in the testsuite, in which the arguments don't match the format string.
You seem to be in a debugger, can you go up the stack frame and fine the format/print statement in OIIO that triggers the assert?
Also, in our fmt.h, there's a line that says #    define FMT_EXCEPTIONS 0, maybe you can change that to 1 to trigger the exception rather than the terminate?

It’s a release mode build with symbols in the debugger.  The stack that is displayed in the image is all I have.  I am trying to repro in debug mode, but I have not been able to so far.
I use vcpkg, I will see if there is a simple way to set that define.

Is this behavior something that could be moved to runtime vs. compile time?

I think that in sufficiently advanced C++ standard, a bunch of it may become constexpr and thus compile-time checkable. Short of that, I think it is possible to fix this to be compile checkable, if we can find the place where it happens (or the much larger job of fixing EVERY place where strings are formatted).
I think it would be fine to stop suppressing the fmt exceptions. I guess it would be problematic if we called it and it threw from within some API call we currently declare as noexcept, but probably not in a way that's worse than the current problem you're facing.

App still crashes in release mode when the throw occurs from unhandled exception.

More info.
buffer contents in vprint_buffered
"OpenImageIO exited with a pending error message that was never retrieved via OIIO::geterror(). This was the error message:\nCould not open file "D:\PhotosAndVideos\2024\Whale Watch\dng\_DHR9039.dng"
What is the solution?

My app is reading new files as them become available on disk.  They may not be completely written when OIIO tries to read them.   I have wrapped all my functions with
`
if (OIIO::has_error())
{
OIIO::geterror();
}
inp = ImageInput::open(diskFilePath.toStdString(), &configSpec);

if (OIIO::has_error())
{
    OIIO::geterror();
    return;
}

`

I don't think a support library should crash the program for any reason.  There should be an error log created if an error is ignored and second is generated.   If my app ignores errors, OIIO should not crash the app, IMHO.

I completely agree.

See #4400 for my attempt to address this:

No terminate
No actual thrown exception

-> should mean that a bad format won't crash the app.
But you will get one of those unretrieved global error messages (if you didn't already retrieve it) that will clue us into a bad format call.
In a debugger, if you set a breakpoint at the pvt::log_fmt_error function, you ought to be able to catch it in the act and figure out which format call is the one that was hitting the exception.

This error is format independent, but it was a DNG file as shown above in the error message.  As I mentioned, I think this is a result of reading partially written DNG files.  I have another app creating hundreds of DNG files and my app (Tinta) is monitoring the hard disk for files.  When new file is detected, Tinta attempts to read it to display its thumbnail.
I am assuming that the crash is caused because OIIO is given files by that are only partially created and causing repeated errors that I am not checking.

Also, I have not been able to repro this failure in debug mode.  I will set the breakpoint in release mode and provide an update soon.

Have I misunderstood issue the whole time?
The first post with the stack trace was of fmt itself hitting its internal assertion, which I thought could only happen in a case where fmt had a mismatched formatting string and arguments, or something of a similar nature. I thought we were chasing a bug in OIIO where we called some format function with incorrect arguments.
I'm not sure how that would be connected to trying to open an incompete file, which I would think would fail its open(), issue an error of some kind, and then move on without having any reason to call assert_fail.
So there may be a second bug somewhere in the mix, but I stand by that PR #4400 and its attempt to avoid any calls to std::terminate just because a fmt call gives bad arguments. You're right that exiting at that point would be an unhelpful behavior, independent of anything else we may need to track down.

The overall issue is that when repeated oiio errors occur, the app is terminated.  Thinking that OIIO shutdown by calling assert_fail when errors were not retrieved with geterror,  I changed the definition of FMT_EXCEPTIONS to 1.  This changed he call to do_throw, but that made no difference.  The app still crashes since this expection was not handled.
There are several things at play here.

Images are failing then open call and calling errorfmt, which sets the errors.
Images are loaded in a separate thread, so when the thread terminates, ~ErrorHolder is called.  This is a high frequency event.
The call to print in ~ErrorHolder is failing in fwrite_fully .   std::fwrite does not write all the requested data, then usesFMT_THROW`, which terminates the app.

It look suspicious to me that the thread_local function ~ErrorHolder uses a global variable pvt::oiio_print_uncaught_errors for error reporting.
The error checking message is not the issue, there should just be an informational message printed.
The issues as I see them are:
A) Why is the call to std::fwrite failing?
B) When std::write fails, the app is terminated.
C) Should a global variable be use in this way by ~ErrorHolder
inline void fwrite_fully(const void* ptr, size_t count, FILE* stream) {
  size_t written = std::fwrite(ptr, 1, count, stream);
  if (written < count)
    FMT_THROW(system_error(errno, FMT_STRING("cannot write to file")));
}
// ErrorHolder houses a string, with the addition that when it is destroyed,
// it will disgorge any un-retrieved error messages, in an effort to help
// beginning users diagnose their problems if they have forgotten to call
// geterror().
struct ErrorHolder {
    std::string error_msg;

    ~ErrorHolder()
    {
        if (!error_msg.empty() && pvt::oiio_print_uncaught_errors) {
            OIIO::print(
                "OpenImageIO exited with a pending error message that was never\n"
                "retrieved via OIIO::geterror(). This was the error message:\n{}\n",
                error_msg);
        }
    }
};

In  ~ErrorHolder. is it possible that error_msg might cleared (or invalidated) by another thread after it is checked?

static thread_local ErrorHolder error_msg_holder;

The error holder itself is thread local.

Yes, but is it possible for a different thread to modify the value of error_msg when the destructor is executing?


How to crop input images properly?

Hi Oeslle,
Great work with FASNet! However, I tried running the model with some test images, and they always seem to be detected as fake. After reading your paper carefully, I found this:

In step two(face detection), we first used the OpenFace face detector [35] algorithm to find the region-of-interest (ROI) corresponding to a face. Next, using OpenFace algorithms we cropped to a window sized 96 pixels and aligned the faces to center based on the nose and eyes position.

How can I replicate this procedure for my own test images? Can you possibly share the code used to detect the face, crop the image and center align the images based on nose and eyes position?

Hi @aakashns , first of all, thank you for ur interesting in our work. I used the OpenFace algorithm, which is open source. I did not change anything in the code.


Update media folder in cms example configuration

Should this not use the new media folder?

Thanks @thomwiggers !


Bundles are missing files on windows

Related to Issue #55
The archives contain files with * in the name. These don't appear on windows.

Fixed in v0.6.9


how can I run any commands which is not in this image?

Since the image is run as user/group of jenkins, I can't update and install any tools, such as python, node.
$ docker run -ti --rm jenkins/jnlp-slave bash
jenkins@290f78c72661:~$ apt upgrade
E: Could not open lock file /var/lib/dpkg/lock-frontend - open (13: Permission denied)
E: Unable to acquire the dpkg frontend lock (/var/lib/dpkg/lock-frontend), are you root?

$ docker run -ti --rm jenkins/jnlp-slave:alpine bash
bash-4.4$ apk update
ERROR: Unable to lock database: Permission denied
ERROR: Failed to open apk database: Permission denied
bash-4.4$
```'
what should I do? How can I run node or python in it? any hints for me?

Try
$ docker run -u 0:0 -ti --rm jenkins/jnlp-slave bash


implement DCO signoff

We want to add a DCO signoff for contributions. There's a bot that can check that for us.

(c) All new inbound code contributions to the CNCF shall be (i) accompanied by a Developer Certificate of Origin sign-off (https://developercertificate.org/) and (ii) made under the Apache License, Version 2.0 (available at https://www.apache.org/licenses/LICENSE-2.0), such license to be in addition to, and shall not supersede, obligations undertaken under the contribution license agreement(s) provided for in (b) above.

requirements: https://github.com/cncf/foundation/blob/main/charter.md#11-ip-policy
AC:

bot checks commit when opening a PR
missing sign off prevents merge

@gusfcarvalho i accidentally added the DCO GitHub App here: https://github.com/organizations/external-secrets/settings/installations/27752715

:point_up_2: the app is installed in all repositories. tested here: https://github.com/external-secrets/external-secrets/pull/1396


remove dead links

Removes dead links as specified in https://github.com/Leaflet/Leaflet/issues/9291

Also needs similar change in https://github.com/Leaflet/Leaflet/blob/main/docs/dialog/index.md

Thanks, I changed the index.md file as well.


⚠ Spryr has degraded performance

In 059ed02, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 15674 ms

Resolved: Spryr performance has improved in c517f4c after 7 minutes.


No active PythonEnv controller for new kernel pick

Partially addresses #11932
Requires more work in subsequent PRs

Codecov Report

Merging #11933 (2bd8207) into main (3c58033) will increase coverage by 0%.
The diff coverage is 83%.

@@          Coverage Diff           @@
##            main   #11933   +/-   ##
======================================
  Coverage     62%      62%
======================================
  Files        490      490
  Lines      34602    34605    +3
  Branches    5616     5617    +1
======================================
+ Hits       21691    21709   +18
+ Misses     10831    10808   -23
- Partials    2080     2088    +8

Impacted Files
Coverage Δ

src/notebooks/controllers/controllerLoader.ts
90% <83%> (-2%)
:arrow_down:

src/platform/common/configuration/service.base.ts
76% <0%> (-3%)
:arrow_down:

...c/platform/common/process/pythonDaemonPool.node.ts
77% <0%> (-2%)
:arrow_down:

src/platform/common/process/condaService.node.ts
67% <0%> (-2%)
:arrow_down:

src/notebooks/controllers/kernelRanking/helpers.ts
55% <0%> (-2%)
:arrow_down:

...els/jupyter/finder/remoteKernelFinderController.ts
61% <0%> (-1%)
:arrow_down:

src/kernels/kernelDependencyService.node.ts
83% <0%> (-1%)
:arrow_down:

src/kernels/jupyter/launcher/serverUriStorage.ts
61% <0%> (-1%)
:arrow_down:

...c/kernels/execution/cellExecutionMessageHandler.ts
77% <0%> (-1%)
:arrow_down:

src/notebooks/debugger/debuggerVariables.ts
67% <0%> (-1%)
:arrow_down:

... and 8 more


node unschedulable

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
1:- run 2 node with etc and control panel and worker node in both node
2:- both node having 4 core cpu each
3:- run 1 given below instance to node 1 and second instance to node 2
4:-create another pod with this configuration run elastic search with following option
`apiVersion: apps/v1
apiVersion: apps/v1
kind: StatefulSet
metadata:
name: es-test
labels:
app: es-test
spec:
serviceName: es-test
updateStrategy:
type: RollingUpdate
replicas: 1
selector:
matchLabels:
app: es-test
template:
metadata:
labels:
app: es-test
spec:
terminationGracePeriodSeconds: 300
nodeSelector:
lilnode: node2
initContainers:
- name: fix-the-volume-permission
image: busybox
command:
- sh
- -c
- chown -R 1000:1000 /usr/share/elasticsearch/data
securityContext:
privileged: true
volumeMounts:
- name: data
mountPath: /usr/share/elasticsearch/data
- name: increase-the-vm-max-map-count
image: busybox
command:
- sysctl
- -w
- vm.max_map_count=262144
securityContext:
privileged: true
- name: increase-the-ulimit
image: busybox
command:
- sh
- -c
- ulimit -n 65536
securityContext:
privileged: true
containers:
- name: elasticsearch
image: docker.elastic.co/elasticsearch/elasticsearch:5.5.3
ports:
- containerPort: 9200
name: http
- containerPort: 9300
name: tcp
resources:
requests:
cpu: "100m"
memory: "3Gi"
limits:
cpu: "500m"
memory: "3Gi"
env:
- name: ES_JAVA_OPTS
value: -Xmx1g -Xms1g
- name: "xpack.security.enabled"
value: "false"
- name: node.name
valueFrom:
fieldRef:
fieldPath: metadata.name
volumeMounts:
- name: data
mountPath: /usr/share/elasticsearch/data
- name: elasticsearch-exporter
image: justwatch/elasticsearch_exporter:1.1.0
ports:
- containerPort: 9114
name: http
command:
- /bin/elasticsearch_exporter
- --es.uri=http://<IP_ADDRESS>:9200
- --es.indices
resources:
limits:
cpu: 100m
memory: 500Mi
requests:
cpu: 25m
memory: 64Mi
volumeClaimTemplates:

metadata:
name: data
spec:
storageClassName: "longhorn-2-replica"
accessModes: [ "ReadWriteOnce" ]
resources:
requests:
storage: 10Gi

Result:
0/6 nodes are available: 1 Insufficient cpu, 1 node(s) were unschedulable, 4 node(s) didn't match node selector.
Other details that may be helpful:
kube schedular logs :-> of node 2
08:48:49.915466       1 authentication.go:267] No authentication-kubeconfig provided in order to lookup client-ca-file in configmap/extension-apiserver-authentication in kube-system, so client certificate authentication won't work.
W0227 08:48:49.915532       1 authentication.go:291] No authentication-kubeconfig provided in order to lookup requestheader-client-ca-file in configmap/extension-apiserver-authentication in kube-system, so request-header client certificate authentication won't work.
W0227 08:48:49.915552       1 authorization.go:146] No authorization-kubeconfig provided, so SubjectAccessReview of authorization tokens won't work.
I0227 08:48:49.915594       1 controllermanager.go:161] Version: v1.17.2
I0227 08:48:49.916618       1 tlsconfig.go:179] loaded serving cert ["Generated self signed cert"]: "localhost@1582793329" [serving] validServingFor=[<IP_ADDRESS>,localhost,localhost] issuer="localhost-ca@1582793328" (2020-02-27 07:48:48 +0000 UTC to 2021-02-26 07:48:48 +0000 UTC (now=2020-02-27 08:48:49.916594004 +0000 UTC))
I0227 08:48:49.916886       1 named_certificates.go:52] loaded SNI cert [0/"self-signed loopback"]: "apiserver-loopback-client@1582793329" [serving] validServingFor=[apiserver-loopback-client] issuer="apiserver-loopback-client-ca@1582793329" (2020-02-27 07:48:49 +0000 UTC to 2021-02-26 07:48:49 +0000 UTC (now=2020-02-27 08:48:49.916869107 +0000 UTC))
I0227 08:48:49.916956       1 secure_serving.go:178] Serving securely on [::]:10257
I0227 08:48:49.917503       1 tlsconfig.go:219] Starting DynamicServingCertificateController
I0227 08:48:49.918080       1 deprecated_insecure_serving.go:53] Serving insecurely on [::]:10252
I0227 08:48:49.918186       1 leaderelection.go:242] attempting to acquire leader lease  kube-s
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
2.3.4
Installation option (single install/HA):
single install

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
cloud 20 reserved cpu and 73.6 gb ram
Kubernetes version (use kubectl version):

 Kubernetes Version: v1.17.2

Docker version (use docker version):

Docker version 1.10.3, build 20f81dd

/sig scheduling


Implement new twf estimation

Thanks to the repository created by @patrickltobing, we integrated a technique to align feature vector to source or target length in sprocket.

Coverage decreased (-0.8%) to 86.387% when pulling 50f0810c2f8c8b8a477874ff648b5a41799c4760 on tar_align into ada7b884e358b56cec5acd200af246df34f1e9c2 on master.

Thanks @tats-u . Merge this PR.


I dont know, look...

I tried it with Mozilla Firefox, Edge... nothing works ^^

But in Chrome, Android works

Yeah same issue i guess the developer has stopped working out on this project or maybe he's already busy in developing something new

Hi @CruptexDE @darshil5p, apologies to such a long response, I have been busy with other development projects.
Thanks for reporting! Fixing it now :)

@CruptexDE @darshil5p I fixed it in https://github.com/uilicious/inboxkitten/pull/22, you should be able to view the email now


Unable to import Myo using Anacoda

Good morning,
I am a novice. I am trying to read data from the Myo Wristband. I have read the documentation on how to import packages into my anaconda distribution. I can see the module called myo when I list the packages within the terminal but when I try to run any of the examples, I get the following error:
runfile('/Users/edgarcolin/Documents/3D printing/myo-python-master/examples/feed_myo.py', wdir='/Users/edgarcolin/Documents/3D printing/myo-python-master/examples')
Traceback (most recent call last):
File "", line 1, in
runfile('/Users/edgarcolin/Documents/3D printing/myo-python-master/examples/feed_myo.py', wdir='/Users/edgarcolin/Documents/3D printing/myo-python-master/examples')
File "/Users/edgarcolin/anaconda/lib/python2.7/site-packages/spyderlib/widgets/externalshell/sitecustomize.py", line 699, in runfile
execfile(filename, namespace)
File "/Users/edgarcolin/anaconda/lib/python2.7/site-packages/spyderlib/widgets/externalshell/sitecustomize.py", line 81, in execfile
builtins.execfile(filename, *where)
File "/Users/edgarcolin/Documents/3D printing/myo-python-master/examples/feed_myo.py", line 2, in
import myo as libmyo; libmyo.init()
ImportError: No module named myo

I think you just don't have the myo package in your PYTHONPATH.  See the installation instructions.

Closed due to lack of feedback.


⚠ Bofrost.com has degraded performance

In 4cc158c, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 1284 ms

Resolved: Bofrost.com performance has improved in 0595a4c after 6 minutes.


gcc/11: apply upstream fix 103910 so openjdk builds

Description of changes
Our openjdk derivations fail to build with -march=opteron (via either NIX_CFLAGS_COMPILE or hostPlatform.gcc.arch).  This was fixed upstream in gcc12 but not carried back to gcc11:
https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=d243f4009d8071b734df16cd70f4c5d09a373769
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103910
Since gcc11 is still nixpkgs' primary compiler, this PR cherry-picks the fix out of gcc12.
Unfortunately since -march= can be added after the gcc11 derivation is built we can't apply this patch conditionally (which we don't usually do anyways).
Things done

Built on platform(s)

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[ ] Fits CONTRIBUTING.md.

(On current staging-next any gcc11 changes shouldn't cause a big rebuild anymore.)

(On current staging-next any gcc11 changes shouldn't cause a big rebuild anymore.)

Sorry about the slow reply.  I'm back now and dealing with the backlog.

Well, maybe this won't be important anymore, as gcc12 is becoming the default and shouldn't need this fix...

OK, why not.

Backport to staging-22.11, I guess?  There gcc11 remains the default for x86_64-linux.


PTR: Fix high contrast list item display, update PTR test page

Summary
Fixed displaying of PTR in high contrast mode.
Remove background color for PTR in PTR test page and add border to show borders of PTR.
Motivation
Fixes #1227.
Testing methodology
Start MUXControlsTestApp and visit PTR -> RefreshContainer.
Ran test suite:

MUXControls.InteractionTests.RefreshContainerTests
MUXControls.ApiTests.RefreshVisualizerTests

/azp run

Thanks for contributing this! I looks right and I've verified it locally. Once the pipelines passes we should be able to merge it in.


Update/fedora 38

Update to ublue-os/silverblue-main:38

Sorry! False repository! Please don't mind.


[New game]:  Avax Gods

🎮 Game Request
A basic NFT battlecard game using web3 technology and blockchain

Point down the features
great UI/UX
Easy to play
Based on blockchain.
 Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Write recursive verifier

Write the recursive verifier. The checklist below is probably incomplete and in some places too coarse-grained. It will be updated and broken up into additional issues as subtasks become clearer.
Triton Assembly Verifier

[x] decompress partial authentication paths (not TASM)
[x] sample {indices, weights, challenges}

[x] Fiat-Shamir

[ ] Merkle tree authentication path verification
[ ] quotient building

[ ] evaluate AIR (see also: #135)
[ ] divide by zerofier

[ ] non-linear combination
[ ] {base, extension, combination} tree root comparison
[ ] FRI verify

[ ] co-linear point compute
[x] #5

sample weights
input: seed, num_weights

list_of_weights = []
for i in {0, …, num_weights-1}:
    digest = xlix(seed, i)
    weight = XFieldElement(digest)
    list_of_weights += [weight]
return list_of_weights

sample indices
See document on index sampling.
Revealed indices
For the recursive verifier, we need to do something different with revealed indices: We currently sort and deduplicate, which is going to be expensive/hard when coding this in TVM.
(Originally issue number 24 in the internal issue tracker.)

Instead of translating our current rust verifier into Triton assembly in one straight go, it might be worth considering to do an intermediate step. For example:

Write a standard library for Triton VM, exposing commonly used data structures and functionalities.
Write a stand-alone, no_std Triton rust verifier.

Tasm-lib now has a recursive verifier. One of the next steps is to bring it into Triton VM. In any case, this issue is superseded.


Migration: Restore corrupted archives

We found a lot of corrupted archives that don't have index.json. And, if I try to access it from the UI, I receive an internal server error for that reason.
Example of a corrupted archive
[error] ** (File.Error) could not read file "/home/bookmark/volume/bookmark/priv/static/archive/archive/1668599147.343435/index.json": no such file or directory

URL -> http://<IP_ADDRESS>/archive/1668599147.343435
Actions
Restore and check corrupted files are working okay. Example of corrupted file:
(File.Error) could not read file "/app/lib/bookmark-0.1.0/priv/static/archive/archive/1666399521.029474/index.json": no such file or directory

https://bookmark.lol/archive/1666399521.029474
http://<IP_ADDRESS>/archive/1666399521.029474

Report status cc @joedavison
Status Summary
Corrupted/broken archives in SQL without index.json before new migration: 149
Corrupted/broken archives in SQL without index.json after new migration: 16
Current broken archives (16)
["1667253001.750881", "1668519899.701167", "1669024891.211075",
 "1669046776.22335", "1669773495.275819", "1669888545.514206",
 "1672013472.890044", "1672075043.855209", "1673858330.130801",
 "1674304230.990355", "1675322778.686017", "1675328403.82425",
 "1675890277.929317", "1676422736.664493", "1676800127.355979",
 "1678129156.641021"]

All those, are also broken in bookmark.lol, for different reasons:
index.json: no such file or directory
 ["1675322778.686017", "1675328403.82425",
 "1675890277.929317", "1676422736.664493", "1676800127.355979",
 "1678129156.641021" ]

errors were found at the given arguments:  * 1st argument: not a bitstring
["1667253001.750881", "1668519899.701167",
 "1669046776.22335", "1669773495.275819", "1669888545.514206",
 "1672013472.890044", "1672075043.855209", "1673858330.130801",
 "1674304230.990355"]

no route found for GET ....html (BookmarkWeb.Router)
 "1669024891.211075"

Detected problems (already solved):

Some folder names were corrupted (for example, they had an extra character \" (example: "1667064899.824599\"", "1668807645.621577\"")
Titles longer than 255 varchars were broken in Postgres. Those were trucked to 200 characters

I verified the errors were found at the given arguments:  * 1st argument: not a bitstring archives, and the error was provoked due to those archives doesn't has the wget_path in the index.json, for example, this ->
archive_1667253001.750881_index.json.txt
I deleted those corrupted archives from the SQL registries, so I proceed to close this issue.


Fix x-aws-keys field name

The current documentation suggests listing SecretsManager secret JSON keys in a keys field, but this is not correct.
The correct field is x-aws-keys.
The keys field as written is not supported by the compose spec, and attempting most docker compose commands would result in failure if one named the field as written.

Proposed changes
Replace the incorrect keys field with x-aws-keys.

Tagging @ndeloof as they appear to be responsible for all compose-cli issues

Help us merge your changes more quickly by adding details and setting metadata (such as labels, milestones, and reviewers) over at the right-hand side.

That sure would be a good suggestion if I had the permissions to do that


build: Add Log level help text

Signed-off-by: Murilo Belluzzo<EMAIL_ADDRESS>

+1

Merged with suggested fixes


An error occurs when debugging the Java/JavaScript/TypeScript project

OS: Mac (Pro)
Build Version: 20220906.4
Regression: Not a Regression
Repro Steps:

Create and open a Java project in VS Code.
F5.
Check whether no error occurs.

Expect:
No error occurs.
Actual:
An error occurs.

More Info:

This issue also reproduces for JavaScript and TypeScript projects.
This issue can be fixed after deleting the below content in the host.json file.

"extensionBundle": {
    "id": "Microsoft.Azure.Functions.ExtensionBundle",
    "version": "[3.*, 4.0.0)"
  }

@v-xinda I'm not able to reproduce this for JS or TS projects on my M1.  (My Java environment doesn't seem happy, so I can't yet test that scenarios.). I've tried using both existing released Functions CLI (4.0.4483) as well as the latest PyStein-compatible release (4.0.4776).  I see you're using a slightly earlier version (4.0.4736); I wonder if you might try pulling down the latest?

@philliphoff  Verified this issue on the latest build, it doesn't reproduce for JavaScript and TypeScript projects after install Functions CLI (4.0.4776). But it still reproduces for Java project.


[WASM][Currency Converter] Time appears at the bottom is incorrect

Steps To Reproduce:

Visit WASM Calculator.
Open Currency converter.
Look at the bottom left corner and observe the time.

Actual behavior:

It shows the wrong time.

Expected behavior:

Time appears on the Currency converter page is same as the local time and it is mentioned in the unit. (for eg. UTC).

Attachment:

Device and Application Information:

OS Build: Windows 10
WASM Build Version: Version 0.1.0-uno.1+506
Environment: Chrome, Firefox and Microsoft Edge

Note:

Refer the attached image to get better idea.

When following repro steps in Edge (Version 88.0.705.50 (Official build) (64-bit)) using this: https://calculator.platform.uno/
I find that there is no Currency converter.
There is a Currency converter in the Windows 10 Calculator 10.2012.21.0 app.

When following repro steps in Edge (Version 88.0.705.50 (Official build) (64-bit)) using this: https://calculator.platform.uno/
I find that there is no Currency converter.
There is a Currency converter in the Windows 10 Calculator 10.2012.21.0 app.


Update sql-query-join.md

Removes duplicate example description and minor formatting changes.

@aakashbhardwaj619 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good to me. The PR is ready to merge. Thanks @aakashbhardwaj619
Publishing in this repository is suspended until Tuesday, May 25. Merging will resume Tuesday, and we'll merge this approved pull request at that time. Thanks.


Init byte_infer_perf/llm_perf

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


[fix] Nav dropdown button not closing when clicking outside of it

ReactUSWDS Version & USWDS Version:
ReactUSWDS: 2.8.0
USWDS: 2.13.1
Describe the bug
Nav dropdown button doesn't close when clicking outside of it. Instead, it stays open until clicking the button again.
To Reproduce
Steps to reproduce the behavior:

Go to the header storybook
Click on a nav dropdown button
Click outside the menu/button
See that the meny does not close until you click the dropdown button again

Expected behavior
It should close upon clicking anywhere else:

Device and Browser Information (please complete the following information if describing a UI bug):

Firefox 98.0.2, Chrome 99.0.4844.83, Safari 15.3
Mac 12.2.1

I'm taking a look at this right now. Looks like the escape key should also close the menu, however unfocusing the button should not close the menu unless the entire nav is not in focus.

I should note that this behavior is a function of how we've set up the component in storybook. Consumers of React-USWDS can (and in fact, must) create their own handlers for when to open and close the nav. Borrowing from a conversation on Truss Slack, we have the option to

Leave it alone. Let consumers of react-uswds set their own open/close togglers.
Use the uswds-made listener
Write our own listener, trying to mirror the behavior of the listener above.

@kmars2022 I've updated the link above, which has since been refactored and can now be found here. We have proceeded with option 1, leaving it to consumers to write their own togglers.


chore(main): release 1.4.1

:robot: I have created a release beep boop
1.4.1 (2024-08-14)
Code Refactoring

merge all errorkeys + clean value... (369816e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Basalt-Lab/basalt-auth/releases/tag/v1.4.1 :sunflower:


A piece of dead code

The following code never gets executed, since the err != nil case is handled above.
https://github.com/reddit/baseplate.go/blob/b28862699aee56a1b96c4bf94b7032bb81f71dd2/thriftbp/client_pool.go#L362-L367
cc @fishy

This is not dead code. This is a deferred function with named return. err will be whatever error https://github.com/reddit/baseplate.go/blob/b28862699aee56a1b96c4bf94b7032bb81f71dd2/thriftbp/client_pool.go#L371 returns.

Yeah, I see. Sorry 😐.


ufrag renamed to usernameFragment

'candidate' message needs updating.
See https://github.com/w3c/webrtc-pc/pull/1479

Good thing I didn't already implement this everywhere 😅


Updated with Table filteration

Now user can easily filter and search with their requirements.
GSSOC'24

ISSUE CLOSED - #25

@mukilan2815 Please Mention the Issue Number in Your PR Request. After That, I will Merge it. Without Mentioning the Issue Number I Cannot Merge it.

updated the comment , please check it out and close it under GSSOC'24

@mukilan2815 Sorry, but this is looking complex for the users (as users are very naive).

Having filter options can be very helpful right


Typo in xml leads to error

my xml (with a typo in the closing tag of three
<?xml version="1.0"?>
<Scenarios>
    <scenario>
        <one>1</one>
        <two>2</two>
        <three>3</tree>
    </scenario>
</Scenarios>

when I do this:
	allScenarios := xmlquery.Find(memXML, "//scenario")
	fmt.Println(len(allScenarios))

It doesn't come to to printing of the len, but it raises me an error.
(And I'm not capable of handling it)

@marcelloh , Its your XML document format error:
Unable to perform XPath operation. The element type "three" must be terminated by the matching end-tag "</three>".
You can try XPath Tester to test your XPath expression.

I know I have this error in the xml. It's handtyped, and every human can make mistakes. However, the error was not thrown by the find option, it's just panics and close down my application.
It would be nicer if find gave me the error too, like this (as example)
allScenarios, err := xmlquery.Find(memXML, "//scenario")
if err != nil {
// inform the maker of the xml it has an error
}
fmt.Println(len(allScenarios))

Can you give an example in how the recover would work? (I'm a bit unfamiliar with that)

s=`
<?xml version="1.0"?>
<Scenarios>
    <scenario>
        <one>1</one>
        <two>2</two>
        <three>3</tree>
    </scenario>
</Scenarios>
`
doc, err := xmlquery.Parse(strings.NewReader(s))
if err != nil {
    panic(err)
}
allScenarios := xmlquery.Find(doc, "//scenario")
fmt.Println(len(allScenarios))

xmlquery.Parse() will return error if an XML document is invalid, so you can check err!=nil to determine XML format is incorrect.

the err after parse is perfect :-)
Thanks


Array - add support for ordered feature - tuple

This PR adds support for ordered feature in handler of Joi.array().
I added new joinOperation for this - tuple

thankyou for the contribution


vite: a11y warnings during build

svelte-dialogs 1.2.2
vite-plugin-svelte reports accessibility warnings during bundling:
[vite-plugin-svelte] /node_modules/svelte-dialogs/src/components/DialogCore.svelte:119:0 A11y: visible, non-interactive elements with an on:click event must be accompanied by an on:keydown, on:keyup, or on:keypress event.
[vite-plugin-svelte] /node_modules/svelte-dialogs/src/components/DialogCore.svelte:128:2 A11y: visible, non-interactive elements with an on:click event must be accompanied by an on:keydown, on:keyup, or on:keypress event.

The two divs at line 119 and 128 need keypress handlers.
In the first case I guess it suffices to add on:keypress={handleBgClick} but in the second case I'm not sure.