Compile errors on Cygwin

Hi, it's Matthew. Thank you again for updating your wrapper. Unfortunately, the new version has a lot of errors on my side. These seem to be general syntax (at least from what CodeBlocks outputs). Here is the error readout: https://pastebin.com/dM12cv0A
I am running Windows 10, with CodeBlocks 17.12 and Cygwin version 2.11.2.
The code does not run currently, but my XFOIL version is 6.99.
Thanks,
Matthew

Hi Matthew, that is quite odd. It's compiling just fine on my end, and in Travis as well, both with g++ and with clang++. I'm not sure how you could be getting compiler errors. Could you post a pastebin of your main.cc as well?
One thing that comes to mind: under what C++ standard are you compiling the code? That really shouldn't be the issue (I'd expect many more errors that what you've posted if you're compiling under, say, C++98) though.

My main file below. Pretty much an exact copy of the readme just for testing. I did have to add "config" before the '.Naca' etc because it was throwing errors (error was "expected primary-expression before '.' token). Not sure if that helps narrow things down. I tried running on C++11 and C++14 with the same errors.
https://pastebin.com/ApbT9e4F

The only errors I can reproduce are the first two, which are because of the line double alpha = result["alpha"] and the one below. The reason for that error is that result is actually of type unordered_map<string, vector<double>>, and you were trying to assign the contained vectors to doubles. That's also how it's currently written in the readme, so I'll get to fixing that. I'll add some more detail to the readme while I'm at it.
The below main file runs without any errors on my end. Are you still getting those syntax errors with this code? If so, I'll spin up a Windows VM and see if I can reproduce there.
#include "CXXfoil/src/xfoil_config.h"

int main() {
  cxxfoil::XfoilConfig config("/usr/local/bin/xfoil") ; //Create xfoil object
  config.Naca("0015"); //Load specific NACA airfoil;
  config.AngleOfAttack(4.0);
  config.PaccRandom(); // generates a random file name under /tmp

  cxxfoil::XfoilRunner runner = config.GetRunner() ;
  cxxfoil::polar result = runner.Dispatch() ;

  std::vector<double> alpha_vec = result["alpha"];
  std::vector<double> CD_vec = result["CD"];

  double alpha = alpha_vec[0];
  double CD = CD_vec[0];
  // etc, other keys are CL, CDp, CM, Top_Xtr, Bot_Xtr, Top_Itr, Bot_Itr

  std::cout << "done" << std::endl;

  return 0 ;
}

Thanks for the sample main file and updated readme. I tried running again and I got the same errors are before (excepting the main file ones). I even tried reinstalling my compiler but the issue persists.

Very odd, it must be either a Cygwin or Windows specific thing then. I'll spin up a Windows VM and see what I can do.

I could reproduce your issues on Cygwin. It would seem that the compiler errors were caused by Cygwin or one of its headers globally defining stdin and stdout as pointers of some kind, so using those names as members of spawn was causing the errors. Additionally, it could not find execvpe (although that should be defined in unistd.h), so I replaced its usage with one of its relatives. I've also added a check to see whether you're using Cygwin, in which case it passes a different filename to xfoil if you're using PaccRandom() (since Windows doesn't have a /tmp/ directory).
I've managed to build and run the code successfully under Cygwin; you simply have to pass the XfoilConfig constructor the path to your xfoil.exe (e.g. "C:/users/jesse/xfoil/xfoil.exe") and it should work.

Thank you for all the work you've been doing on this! It now compiles successfully, however I am getting an  exception when running.
3 [main] XFOIL_Wrapper 39448 cygwin_exception::open_stackdumpfile: Dumping stack trace to XFOIL_Wrapper.exe.stackdump
I've attached the generated stackdump but don't know how to open it myself.
Main File:
https://pastebin.com/TTsdJQU5
Stackdump.zip

The stackdump shows that it crashed due to an access violation... My guess is that xfoil cannot open the polar file, and thus the result vectors are empty, causing the program to die when you try to access their contents.
Does the program still crash when you comment out lines 16, 17 and 20?

So upon commenting out those lines the program cannot run at all. the error the terminal shows is 'Failed to launch program.' Possibly the access violation stems from trying to read files that were never populated since xfoil isn't launching.

No, the access violation is definitely due to the empty vectors, which in turn are empty because Xfoil never ran. Which is strange, since it runs just fine on my Cygwin install.  Otherwise, I'm not sure what could be causing execv to fail.
I've also uploaded a version using system instead of execv, in this branch. You could try that one, though it really shouldn't make a difference - system just calls fork and execl. It's worth a shot though.

The file path is correct in my system, but when I ran your 'system' version the error was more explicit stating that the directory wasn't found so that is definitely the problem. However, the file path is correct and even placing the xfoil.exe in the same directory as the program and using a local path leads to the file still not being found. Could it be possible that CodeBlocks does not have the permissions to access other files?

Possibly, but I have to say I know very little about Windows file permissions. You could try running the exe from cmd or cygwin's shell.

So I figured out how to access xfoil.exe. Firstly, I was using a shortcut to the real application which it system() didn't like. Secondly, I need to have the program and the xfoil executable in Cygwin's home directory for it to be able to find the xfoil regardless if I am using an absolute path. In this directory relative paths work as well. However, upon generating a very simple makefile to test out the wrapper, the resulting executable doesn't run but for a split second. Running the program through CodeBlocks just says that the program cannot be found, but I have had success opening xfoil myself with system() now that I am in the correct directory. I don't get any build errors so I assume my makefile is operating well but just so you have it, it is linked below.
Makefile: https://pastebin.com/xvuRSyUL
P.S upon generating the makefile it gives me an error saying that the variable child_status is unused in xfoil_runner.cc.
P.P.S just saw your edits and I really don't see anything different with my setup. I have tried re-downloading Cygwin a few times with no change and I have all the CodeBlocks compiler settings as well.

I'll have a closer look at your Makefile tomorrow (it's quite late over here right now), however, one thing you could try is to use cmake instead of make. Nobody really writes Makefiles manually anymore. If you want to try your hand at this, just place this CMakeLists.txt file in your project directory: https://pastebin.com/SAzMEF98
Provided that the cxxfoil library is in its own CXXfoil folder in your project root, i.e. as if you cloned the repo directly into your project root. The directory structure should look like this:
.
├── CMakeLists.txt
├── CXXfoil
└── main.cc
Then, from your project root, enter the following commends:

mkdir build && cd build
cmake .. -G "Unix Makefiles"
make

This will create a file test_cxxfoil.exe under build. If it complains about missing dll's (e.g. cygwin1.dll), you can place those dll's (located at C:\cygwin64\bin) into your C:\Windows directory, or alternatively add their directory to your %PATH%.

Thanks for the info! When I run your sample CMakelists.txt it can't seem to find the header files (specifically xfoil_config.h but I suspect that that is just the first one that it tried and failed to find. I have the directory set up as you indicated above.


[New Feature]: fixing navbar

🎮 feature Request
i can add the animation and design the navbar as requirement.
Point down the features
adding the animation , fixing the navbar
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

As a Gssoc'24 contributor i want to work on this issue. so, i kindly request you to assign this to me

@Ramu-chiluveru @SougataXdev someone is already working on this issue. Please raise any other issue,  i'll assign you

If you have any doubt you can directly send message on our discord (https://discord.gg/rZb46cCMmK), we have created seprate chennals for all projects...
Thank you :)


refactor: load eq_param_index of BundleSolvers correctly

fixes: #211

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

Comparison is base (1b6d8c5) 84.46% compared to head (b5d457d) 84.46%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #212   +/-   ##
=======================================
  Coverage   84.46%   84.46%
=======================================
  Files          21       21
  Lines        3695     3695
=======================================
  Hits         3121     3121
  Misses        574      574

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


updateIphoneOSDeploymentTarget -> updateOSDeploymentTarget

Summary:
While merging new commits into React Native macOS, I noticed https://github.com/facebook/react-native/pull/39478/
I would like to also set MACOS_DEPLOYMENT_TARGET in our fork, and thought this slight rename would be something I can do upstream
Changelog:
[Internal] - updateIphoneOSDeploymentTarget -> updateOSDeploymentTarget
Test Plan:
CI should pass

@Saadnajmi For people have less context, this changes so that the method would cover both iphoneos and macos?

@Saadnajmi For people have less context, this changes so that the method would cover both iphoneos and macos?

Yeah, but I would only add the macOS deployment target in React Native macOS. Most of my changes to React Native are small renames / upstreaming bug fixes from React Native macOS. I was trying to be proactive about this one so we have one less diff between React Native and React Native macOS


Yac: switch supermajority checker to CFT

Signed-off-by: Mikhail Boldyrev<EMAIL_ADDRESS>

Description of the Change
While the code is not Bft, we should use Cft mode explicitly to avoid misunderstanding and tolerate more failing peers.

Benefits

Possible Drawbacks

Usage Examples or Tests [optional]

Alternate Designs [optional]

maybe it is better not to be using namespace std?

Does it imply that other calculations are going to be performed during consensus? If so, then I am not able to decide if that change is sufficient.


@-mention markers

This PR is built on top of #510
Fixes: #384

@gnprice ready for review again!

@gnprice Commit message updated per comment, now just waiting on #510


Update code-of-conduct.md

Signed-off-by: Matt Germonprez<EMAIL_ADDRESS>Updating the metric to coincide with the new metrics template.

This is great. Thanks @germonprez


Removed deprecated qiskit_ibm_runtime.IBMRuntimeService class

I have removed the deprecated IBMRuntimeService class as per Issue https://github.com/Qiskit/qiskit-ibm-runtime/issues/359. This PR will be on hold as per the discussion in PR #363.
Fixes #359

@bopardikarsoham This class has to be removed
https://github.com/Qiskit/qiskit-ibm-runtime/blob/6647f477209b92b97796671c223e25d4e4efb3ca/qiskit_ibm_runtime/qiskit_runtime_service.py#L1439
in this PR.

Ohh right I'll make the changes

I have updated the PR @rathishcholarajan

@bopardikarsoham Could you pls remove IBMRuntimeService from this line as well
https://github.com/Qiskit/qiskit-ibm-runtime/blob/315a088a844dc8aa4452bde6136b53694dfb3220/qiskit_ibm_runtime/__init__.py#L234
?

I had actually made a complete PR and removed all these lines perhaps a clash with my previous PRs have caused problems. I'll make a fresh PR for this issue


opt(apps/prod/tekton/configs/tasks): add env var in build step to avoid cargo fetch error

Example error:
[build]     Updating git repository `https://github.com/tabokie/fs2-rs`
[build]     Updating git repository `https://github.com/pingcap/rust-protobuf`
[build]     Updating git submodule `https://github.com/google/protobuf`
[build] error: failed to load source for dependency `protobuf`
[build]
[build] Caused by:
[build]   Unable to update https://github.com/pingcap/rust-protobuf?branch=v2.8#6642ebaa
[build]
[build] Caused by:
[build]   failed to update submodule `google-protobuf`
[build]
[build] Caused by:
[build]   failed to fetch submodule `google-protobuf` from https://github.com/google/protobuf
[build]
[build] Caused by:
[build]   network failure seems to have happened
[build]   if a proxy or similar is necessary `net.git-fetch-with-cli` may help here
[build]   https://doc.rust-lang.org/cargo/reference/config.html#netgit-fetch-with-cli
[build]
[build] Caused by:
[build]   SSL error: unknown error; class=Ssl (16)
[build] + r=101
[build] + set +x

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


Deleting users is missing all the features of the old portal

I went to delete a user using the new interface and instead it asking whether I wanted to convert it to a shared mailbox, set up an auto reply and and grant access to a different user and what to do with the license,  it just deleted the user !  Where has all that functionality gone in the new interface ?

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

ID: 6c772841-0d77-98e6-3e0e-af6f5c9d06a1
Version Independent ID: 9e61ed55-3037-c7f2-4149-ca72ba789c9d
Content: What's new in the Microsoft 365 admin center?
Content Source: Office365-Admin/whats-new-in-preview.md
Service: o365-administration
GitHub Login: @adefowler
Microsoft Alias: anfowler

@officedocsbot assign @mypil

@technonath  - Thank you for submitting feedback.
I will get this issue over to the O365ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@technonath  Thank you for feedback. The option to convert to a shared mailbox or to grant someone else access is not available on the new interface. Instead you will see three different options
1)Email alias will be removed (shows the alias which will be removed).
2)Mailbox delegation permissions will be removed( shows the permissions which are present)
3)Give another user access to onedrive(If OD is provisioned)
Hope this helps!
Regards
Sri

@msbemba you have stated everything, I already knew,  so know that doesn't help at all!
What I wanted to know is do you plan to re-instate the features available in the old interface, if so when and if not why not?
That was the information I was seeking.

@adefowler - Can you please share your insights on this issue?
Thank you.

@adefowler so you closed this issue with out updating the documentation or replying to the feedback I gave through the give feedback link.
I am not impressed.

I'm sorry @technonath, I should've responded more clearly. That's not the purpose of this document. This document's purpose is to quickly let you know what is new - What it is and how to find the new feature in the UI. This document is an historic record and we don't go back and update previous months. Especially at the time of your comment, we were still in "Preview".
And you're actually asking for feature requests in your comment:
"What I wanted to know is do you plan to re-instate the features available in the old interface, if so when and if not why not? That was the information I was seeking."
In fact, I asked you to please provide product feedback as the product group will get back to you very quickly with the information you are seeking. Did you not have success doing that? If not, let me know which day you provided feature feedback in-product and I will find it and make sure it gets answered ASAP.

@technonath - I found your feedback on 8/13 that you submitted and I have replied you via email. I'm sorry that no one got back to you on that day. I've forwarded the comment on to the product team and you should get a response soon. And I'm very sorry about all of the confusion and the lack of response, especially since I had guaranteed it.  Let me know if you don't hear anything and I will definitely get it noticed!


Emoji error, type anything between colons and it repeats :likethis:likethis

Not fixing this now auto-complete is in, users have no excuse! :)


Allow to read files after mtime check

Thanks for awesome plugin. But I want to make it even faster. Right now we need to read all files in fulp-src and only then we will remove unchanged.
But we can read only changed files:
return gulp.src('src/*.js', { read: false })
  .pipe( changed('dest/') )
  .pipe( readFile )

Unfortunately, file.isNull() check in yor plugin broke my perfomance utopia :(.
Maybe we can move it to compareSha1Digest? Because compareLastModifiedTime doesn’t need file content.
@sindresorhus

Yes, I agree.

I will send a PR today


Add FDs to a process' FD table when transferred via SCM_RIGHTS

Motivation
While debugging an issue with missing FDs for an HAProxy process in the Sysdig Agent, we found that HAProxy utilizes the SCM_RIGHTS functionality to do hitless reloads when the HAProxy config changes. You can read more about that at https://blog.cloudflare.com/know-your-scm_rights/, https://man7.org/linux/man-pages/man7/unix.7.html, and https://copyconstruct.medium.com/file-descriptor-transfer-over-unix-domain-sockets-dcbbf5b3b6ec. To summarize them, SCM_RIGHTS allows processes to transfer open file descriptors between themselves via control messages in the sendmsg() and recvmsg() syscalls.
In looking at the library code, I don't see this functionality being handled at either the driver or userspace layer. Because of this, the FDs that are transferred to a process via SCM_RIGHTS do not show up in the FD table for that process until restarting the Agent. After restarting, the FDs appear since they are found during the initial procfs scan.
Feature
When open FDs are transferred to a process via SCM_RIGHTS, I'd like to see those FDs show up in the process' FD table.
At the driver layer, we'll need to return a new argument for recvmsg() which will be the control messages from msghdr. Those are fields msg_control and msg_controllen in the msghdr.
Then, in sinsp_parser::parse_rw_exit(), we need to check for SCM_RIGHTS control messages for recvmsg() syscalls. If we detect one, we need to extract the FDs out of it, lookup the FDs in procfs to construct their fdinfo, and add them to the FD table for the process.
Alternatives
Beyond a periodic procfs rescan, there's not really much else we can do to handle SCM_RIGHTS.
Additional context
The Sysdig Jira issue where this was found and discussed is SMAGENT-5353.

Hi! Thanks for such an in-depth analysis!
Yep this feat request totally makes sense; would you be willing to tackle it? :)

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

This was actually fixed in #1400.
/close

@FedeDP: Closing this issue.

In response to this:

This was actually fixed in #1400.
/close

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


vue-cli-service not creates new files for nested lazy loaded views

Version
4.4.6
Environment info
the below command shows nothing, but I don't think it's needed for this case.

Steps to reproduce
Having a router like this:
const router = new Router({
    mode: 'history',
    base: process.env.BASE_URL,
    scrollBehavior() {
        return { x: 0, y: 0 }
    },
    routes: [
        {
            path: '',
            component: () => import(/* webpackChunkName: "test1" */ './layouts/main/Main.vue'),
            children: [

                {
                    path: '/dashboard/analytics',
                    name: 'dashboardAnalytics',
                    component: () => import(/* webpackChunkName: "test2" */'./views/DashboardAnalytics.vue')
                },
            ]
        }
    ]
});

What is expected?
It's expected that new file will be generated for the parent route and child route separatelly since they both get loaded as lazy.
What is actually happening?
Only parent's js file gets created. Children's files don't get created even though they get loaded as lazy

I'm sorry but we do need a runnable reproduction repo to see what's going wrong. A simple code snippet is not enough.

Closing due to inactivity.


create category feed

I tried to set up a category feed using jekyll now, however the category feed ends up being empty. I followed the typical steps that are described in "how-to"s on the web like that one:
http://jwhollister.com/r/2014/10/09/Category-Feed.html
Here is my blog:
https://github.com/jmbh/jmbh.github.io
and the empty feed:
http://jmbh.github.io/feed.r.xml
any help is very much appreciated!

Not sure I'm afraid @jmbh, this isn't something I've really looked into.

It appears the xml feed is working now. Is it not still?


[BUG] No hits for keys following an array in a flat_object

Describe the bug
When using the flat_object type, keys that appear after an array within the object do not return any hits in search queries
Related component
Other
To Reproduce

create an index:

curl -X PUT "$OS_HOST/$INDEX_NAME" -H 'Content-Type: application/json' -d '{
    "mappings": {
      "properties": {
        "data": {
          "type": "object",
          "properties": {
            "detail": {
              "type": "flat_object"
            }
          }
        }
      }
    }
  }'

Add data

curl -X POST "$OS_HOST/$INDEX_NAME/_doc/1" -H 'Content-Type: application/json' -d '{
    "data": {
      "detail": {
        "val1": "val1",
        "array1": [
          {
            "key3": "value5",
            "key4": "value6"
          },
          {
            "key3": "value7",
            "key4": "value8"
          }
        ],
        "val2": "val2"
      }
    }
  }'

Query the data:

curl -X GET "$OS_HOST/$INDEX_NAME/_search" -H 'Content-Type: application/json' -d '{
    "query": {
      "match": {
        "data.detail.val2": "val2"
      }
    }
  }'

It retrieves 0 hits, although searching the value before the array works fine.

curl -X GET "$OS_HOST/$INDEX_NAME/_search" -H 'Content-Type: application/json' -d '{
    "query": {
      "match": {
        "data.detail.val1": "val1"
      }
    }
  }'

Expected behavior
Query should return matching values, despite the structure of the flat_object.
Additional Details
Plugins
No plugins enabled
Screenshots

Host/Environment (please complete the following information):

OS: Mac OS Sonoma 14.1.1
Version 3.0.0-SNAPSHOT

Additional context
Add any other context about the problem here.

Looks like a legit problem. Want to turn this into a (failing) YAML REST test?

[Triage - attendees 1 2 3 4 5]
Thanks for filing. Looks like a bug with clear reproduction steps.


Fix/ignore button via js

Added lightweight javascript (no framework other then already included axios) to handle request for both "Ignore" and "Un-Ignore", removing item from the list and prevents the "jump to top" effect.
(closes #24 )

I've submitted the requested changes, and tried to format the tabs/spaces thing.
My formatter settings in VS Code try to change almost all lines of the files, so I had to turn the autoformat off.
And yes, many years since I wrote some basic js like this, but when you don't need a framework, why both? :)


Subshell Expansions Often Highlight as Strings

I'm not sure if this is intended or not, but for example:
#!/bin/bash

function my_func() {
	echo "hello!"
}

output="$( my_func )"
output=$( cat <<<"$( my_func )" )
output=$( my_func "$( my_func )" )
output=$( cat <( my_func ) )

The only literal string that's present is "hello!" but the following is how this is highlighted:

Quoting the expansions as in the following doesn't change the highlighting:
#!/bin/bash

function my_func() {
	echo "hello!"
}

output="$( my_func )"
output="$( cat <<<"$( my_func )" )"
output="$( my_func "$( my_func )" )"
output="$( cat <( my_func ) )"

TL;DR
wbond (one of ST maintainers) and facelessuser (the author of BH) said they have a way to fix this issue. And I am just waiting for it.
--
Yes. Currently we have some problems about this. The root cause is that I try to use meta.reset.color to get things back to their default color (mostly white for a dark theme). In recent ST release, clear_scope is introduced and it can indeed resolve this in a decent way. But, unfortunately, that way breaks the way BracketHighlighter(BH) works. BH is a top-25 plugin so the fix may not be acceptable. People in ST forum thought this issue could be resolved so I am waiting for an example of a solution.
--
Ref:

Problem with clear_scopes and BracketHighlighter
An experimental branch with clear_scope used.

@jfcherng understood

I have tweak the scope in 1.2.21 to do coloring better.
But this issue still remains in more complex codes (in that case, codes are written badly I guess).

Feel free to report here if this issue remains.

If you didn't use the bundled color scheme in this package, you would have to add the following settings into your color scheme.
<dict>
    <!-- force codes to be shown as the default color (white) -->
    <key>name</key>
    <string>Embedded</string>
    <key>scope</key>
    <string>meta.embedded</string>
    <key>settings</key>
    <dict>
        <key>foreground</key>
        <string>#FFFFFF</string>
    </dict>
</dict>

Cool, thanks!


Web-API webserver could not start

Hi,
I just try'd you plugin and all goes well til the server has to start.
This is the following message given:
[04:34:29] [Server thread/ERROR] [webapi]: Web-API webserver could not start, probably because one of the ports needed for HTTP and/or HTTPS are in use or not accessible (ports below 1024 are protected)
I've try'd several different ports and even placed the machine in a DMZ and still it refuses to operate on any port listed in the cfg.
I'm really interested in this plugin and its features.
If someone can help I would be in your debt.

Hi,
What ports did you try ? There are usually no trouble with default port and numbers around 8000 (not 8080 or 8888).
This plugin is dope, how do you plan to use it ? (Basic, api, dev?


Screen size measuring

Currently, projects can get the size of the window using Window Controls and the size of the entire display using Screen Resolution. But what if more information about the screen, window, and project player could be obtained through a single extension?
Here are all the interfaces we can use to get different size measurements, and some ideas for what we could call them:

API
My term
Access

🖥
window.screen.width/height
Display
✔

🖼
window.screen.availWidth/availHeight
Desktop
✔

🟦
outerWidth/outerHeight
Window
✔

📄
innerWidth/innerHeight
Webpage
🛡

📽
Scratch.renderer.canvas.width/height
Player
🛡

And here's how I represent them visually:

Which of these do you think would be useful for Scratch projects?

This does exist in Window Controls, I still never understood the point of Screen Resolution standalone, considering it already concisely in Window Controls.

Good idea.
Scratch.renderer.canvas.width/height is technically already possible using of Simple3D + high quality pen

https://github.com/TurboWarp/extensions/blob/a83405803fe28a940c6b3998992a76c916f7655c/extensions/Xeltalliv/simple3D.js#L959-L965
https://github.com/TurboWarp/extensions/blob/a83405803fe28a940c6b3998992a76c916f7655c/extensions/Xeltalliv/simple3D.js#L4314-L4329

 This does exist in Window Controls, I still never understood the point of Screen Resolution standalone, considering it already concisely in Window Controls.

That's not what OP is asking for. In fact, they acknowledge in the original post that Window Controls has these blocks. What they're looking for is an extension that can detect the size of different parts of the screen, such as the project player, window, display/monitor, etc.

 This does exist in Window Controls, I still never understood the point of Screen Resolution standalone, considering it already concisely in Window Controls.

That's not what OP is asking for. In fact, they acknowledge in the original post that Window Controls has these blocks. What they're looking for is an extension that can detect the size of different parts of the screen, such as the project player, window, display/monitor, etc.

No I understood that, I lamented about a different fact, that the blocks in the other listed extension also existed in WC. But whatever. Let's move on.


Inject CSI and CP vSphere Creds (#3424)

The controller was not injecting the extra CSI/CP credentials.
Issue #, if available:
https://github.com/aws/eks-anywhere/issues/3404
Description of changes:
The EKSA Controller didn't have necessary vsphere credentials. As a result, when it reconciled eksa-system:csi-vsphere-config it deleted the username and password.
Testing (if applicable):

create:

create cluster on main

upgrade:

create cluster on v0.11.1
upgrade on release-0.11 with my changes on top of it. Secret eksa-system:vsphere-credentials was correctly updated.
ran old controller while upgrading. eksa-system:vsphere-credentials was unaffected but controller continued to did still overwrite csi-vsphere-config secret
ran new controller after upgrade and csi-vsphere-config reconciled correctly with appropriately populated config information derived from eksa-system:vsphere-credentials

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Standard cherry-pick flow didn't work due to conflicts: https://github.com/aws/eks-anywhere/pull/3424#issuecomment-1261311211

/approve
/lgtm


floatPrecision should ignore gradient stops

Hi,
I noticed that when setting floatPrecision to 0, it will totally mess up gradient stops.
It does work well on path/shapes position precision optimization, but not so well on gradient stops.
Perhaps we could have a flag to exclude or set separate floatPrecision for gradient stops?
before
<linearGradient id="a" x1="2" x2="139.4" y1="43.4" y2="-.9" gradientUnits="userSpaceOnUse">
      <stop offset="0" stop-color="#fc0"/>
      <stop offset=".1" stop-color="#ff7600"/>
      <stop offset=".2" stop-color="#ff6d1e"/>
      <stop offset=".3" stop-color="#ff556c"/>
      <stop offset=".4" stop-color="#ff2fe6"/>
      <stop offset=".5" stop-color="#ff29f9"/>
      <stop offset=".6" stop-color="#ca36fb"/>
      <stop offset=".8" stop-color="#5e50fe"/>
      <stop offset=".9" stop-color="#335bff"/>
      <stop offset="1" stop-color="#237aff"/>
      <stop offset="1" stop-color="#217dff"/>
    </linearGradient>

after
<linearGradient id="a" x1="2" x2="139" y1="43" y2="-1" gradientUnits="userSpaceOnUse">
      <stop offset="0" stop-color="#fc0"/>
      <stop offset="0" stop-color="#ff7600"/>
      <stop offset="0" stop-color="#ff6d1e"/>
      <stop offset="0" stop-color="#ff556c"/>
      <stop offset="0" stop-color="#ff2fe6"/>
      <stop offset="0" stop-color="#ff29f9"/>
      <stop offset="1" stop-color="#ca36fb"/>
      <stop offset="1" stop-color="#5e50fe"/>
      <stop offset="1" stop-color="#335bff"/>
      <stop offset="1" stop-color="#237aff"/>
      <stop offset="1" stop-color="#217dff"/>
    </linearGradient>

Not so sure that it should. E.g. you would want instead of something like 0.66667 to use .67. Zero digits is actually quite extreme setting, though it can have it uses.

I was just comparing output from this library vs https://svgomg.net.
On svgomg I do get a benefit of of smaller filesize when setting precision to 0 and still having intact gradients.

https://github.com/svg/svgo/pull/1843


Top and bottom black lines are displayed unconditionally

There is an issue with your example. I've just cloned repo and run it and here's what I see:

There are black lines at top and bottom of Segmentio while nothing is on GIF-demo. Could you please clarify is it a bug or feature? And in case if it's feature - let me know how to disable it?

Hi @gooooer. Thanks for interest to out component!
I cloned and didn't see the line (it mustn't be there). @houze2311, could you also check it?

Hi @serejahh . I'm running Xcode 7.3.1, iPhone 6s simulator with iOS 9.3 on board. I'm also able to see those two black lines even if I just put Segmentio to Interface Builder (i.e. add UIView and assign Segmentio class to it)

I'm facing the similar issue after upgraded to 1.0. For my case, I put the segmentio down the bottom of a view, then I got two horizontal lines at the top of the view. I reckon that's the horizontal separators.
downgraded the version to 0.15 fixed the issue.

Acknowledge having the same issue

Hello, guys! Check new version 1.1

Fixed 👍 Thank you!


NYISO interconnection queue columns have changed

NYISO interconnection queues are distributed as 3 sheets in an excel file, one sheet for each of active, withdrawn, and completed projects. Each sheet has slightly different columns, and it appears they may have changed since the NYISO.get_interconnection_queue() method was written.
For example, active projects use the label "Points of Interconnection" whereas the other two sheets use "Interconnection Point". This is not currently accounted for in the code, resulting in active projects losing all information about their interconnection points. There may be other differences as well.

I too am running into this issue.

@TrentonBush @APIKreative Are there other differences you have noticed? I've opened #329 but it only addresses the "Points of Interconnection" difference.

I'm going to close this since #329 is merged, but feel free to comment if you find other issues.


Suggest: Clear All Problems

Situation

Run tslint task get some error.
Want to see tsc error, and clear last tslint error message
Run tsc task, get new error messages, but is merged with last tslint error

Nowadays I can only achieve this by restart VSCode.
Suggest
Add a action: Clear all problems
So that user can manually clear problems.

Related: https://github.com/Microsoft/vscode/issues/15524

Agree. Facing the same issue.
Most of the time run webpack which also runs lint and displays its warning, but once in a while I need to execute just lint only task. After I do it - problems found during lint only task never go away, displaying merged with webpack duplicates until I restart vscode

Duplicates #15524


Explicit coverage over creation of parameterized URLs

Description
This is to ensure we have the right helpers (and test coverage) to build URLs in the right way, and not experience code defects as the ones reported recently.

CC @jonatascastro12 @PaulAsjes @southgate


Use current login option

https://ocd.appscale.com:8080/job/Daily Build/4262/
https://ocd.appscale.com:8080/job/Hawkeye/12061/
https://ocd.appscale.com:8080/job/Migration/2265/


DOC-667 | Cluster metrics about read-only shards and collection shard locks

Description
Waiting for https://github.com/arangodb/arangodb/pull/20726
Upstream PRs

3.10:
3.11:
3.12:
3.13:

Upstream PR has been closed


The engine "pnpm" appears to be invalid.

Bug report
Description / Observed Behavior
I get a warning when I do "yarn add swr".
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
Expected Behavior
That the warning does not appear.
Repro Steps / Code Example
1.. "yarn add swr"
Additional Context
Useful data :
    Operating System:
      Platform: win32
      Arch: x64
      Version: Windows 10 Education
    Binaries:
      Node: 16.14.2
      npm: N/A
      Yarn: 1.22.19
      pnpm: N/A
    Relevant packages:
      next: 13.0.6
      swr: 2.0.0
      react: 18.2.0
      react-dom: 18.2.0

We use pnpm as our package manager, and that is for pnpm.
https://pnpm.io/package_json#engines
I'd say there is no problem even though yarn displays the warning.


AVRO-2428: Update Hadoop version in the MR document

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Avro Jira issues and references them in the PR title. For example, "AVRO-1234: My Avro PR"

https://issues.apache.org/jira/browse/AVRO-2428
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

No additional test since it's mainly a documentation fix. Instead, I walked through the MR document (http://avro.apache.org/docs/current/mr.html) and confirmed all of the commands in that document (mvn compile and mvn exec:java) worked with the updated pom.xml. Also, I ran ./build.sh dist
and confirmed that avro-doc-1.10.0-SNAPSHOT was successfully generated.
Commits

[x] My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain Javadoc that explain what it does

This looks good, however I'm wondering if the code snippets in the doc are still up to date or not? On the link mentioned in the Jira (http://avro.apache.org/docs/1.9.0/mr.html) there are several MR examples, I bet they were not updated since ages, and maybe they are very out of date. Anyway, we can track it in a separate Jira, this PR is only about version updates.

@Fokko should we backport this to 1.9 branch too?

Yes, I think it would be good to cherry-pick -x this to the 1.9 branch as well. I think we should not wait too long to start the 1.9.1 process because of the regression issues that have been discovered after the release.


Fix typo

implmentation -> implementation
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[X] Docs change / refactoring / dependency upgrade

Motivation and Context / Related issue

How Has This Been Tested (if it applies)

Checklist

[ ] The documentation is up-to-date with the changes I made.
[ ] I have read the CONTRIBUTING document and completed the CLA (see CONTRIBUTING).
[ ] All tests passed, and additional code has been covered with new tests.

Closing this as it was fixed in https://github.com/facebookresearch/Private-ID/commit/fe511dccd70084eeee85c8871792377f66058f1e.


Stop catching things like KeyError in most integrations

Historically, a number of our older incoming webhook integrations have caught KeyError/ValueError/etc. in order to avoid throwing a 500 when receiving an unexpected request.  However, our new strategy is to throw the 500 and automatically log the request to /var/log/zulip/webhook_errors.log, so that we can fix the exceptions.  We should probably go through our older webhook integrations and eliminate the bad pattern of suppressing errors inside the view code (and don't forget to consider updating the docs to describe this strategy!).
Part 2 of making this work well is to add code to send a PM to the bot's owner when one of these 500s occurs letting them know about the issue, to make debugging easier, but let's leave that for its own follow-up issue since there are a few complicated design questions to decide.
https://chat.zulip.org/#narrow/stream/integrations/subject/bitbucket.20pull.20requests/near/271437 has some discussion and an example use case.
I think a side effect of fixing this is that it'll be easy to move our webhook integrations codebase to 100% test coverage, since most of the coverage gaps are these exception handlers.

@roberthoenig  I grepped for "KeyError" I can see few exception still uses that inside Webhooks integrations. Shall I clean that up? or those are the different case to handle?

git grep except shows a good number.  I think any that are intended to cover bugs in the integration (as opposed to variation in the format) should probably be changed; these usually have no test coverage, because we didn't have a payload that would actually fail the test.
For example, this code:
    try:
        # The event identifier is stored in the X_GROOVE_EVENT header.
        event = request.META['X_GROOVE_EVENT']
    except KeyError:
        logging.error('No header with the Groove payload')
        return json_error(_('Missing event header'))

I think we want to replace with something that isn't per-integration code and has unit tests.  E.g. maybe a good way to do those would be to write:
event = validate_extract_webhook_http_header("X_GROOVE_EVENT")
And that function would take care of handling the KeyError and doing appropriate error reporting (probably something that effectively sends a notice to the webhook owner that their payloads are invalid for the webhook).  We might do a part 2 migration to make that run as a REQ style converter, but definitely not worth working on until we have the function path working.
@eeshangarg pinging you for thoughts on this.

Okay, so I think there is a part of this discussion that I misunderstood. Are you saying that we should notify users of missing event headers regardless of whether they are explicitly caught using except KeyError currently? If that's what you meant, I think that makes sense!
What I meant was more so that currently the KeyErrors being caught aren't about event headers at all, so we don't need to account for KeyErrors per se but for missing event headers in general.
If you think we should send a user a message "hey your payloads are missing the event header this might mean you are using an older version of the third-party's product", I think that is useful in a way. But I still believe that either way the payloads should be reported to us using webhook-logger so that we can be sure we aren't doing anything wrong!
Let me know what you think! :)

Right, so my philosophy here is that there's basically a few categories of issues with a webhook integration, and we want to route them to the right person:

If the payload is clearly for the integration, but we're not parsing it correctly -> 500 and notify server admins to fix the bug.  So we should get rid of all the KeyError things that look like defensive programming -- we want those to 500 and have the payloads be fixed (these predate our current strategy).  I think a good signal for which is which is whether we have test coverage for both code paths, since that tells you whether there's a valid payload that is different.  (And it's just not KeyError; also ValueError, etc.).  Though maybe it'd be better to write most of these with an if 'foo' in bar form than just looking up invalid keys; I think catching KeyError is generally an ugly solution.  A goal of the effort of removing unnecessary conditionals is to get more of our integrations to have 100% test coverage.
If the payload is a new/unsupported event type, we should probably return a 200 (not a 500), but log something for the server admin to address with just like we would have for a 500.
If the payload is clearly not for the integration (e.g. is missing the field through which we identify the event type), we give the user a notification so they can repoint their integration.
For special cases like the old GitLab integration, try to provide a custom error message about the specific issue.

Does that make sense?


How do you do the alignment between audio features and posture frames?

This is an excellent work. As a student working on gesture generation, I am very appreciate of this code. Thank you.
However I've got a question for you guys.
I notice that in the file create_vector.py, specifically in create_vectors function, there is a step 3, which is commented as "align vector length". It is written as,
# Step 3: Align vector length
input_vectors, output_vectors = shorten(input_vectors, output_vectors)

and,
def shorten(arr1, arr2):
    min_len = min(len(arr1), len(arr2))

    arr1 = arr1[:min_len]
    arr2 = arr2[:min_len]

    return arr1, arr2

In my view it was supposed to be doing some kind of alignment, because in your work, all MFCC features were concatenated as one vector during training. Therefore it is crucial to know which part of MFCC features is corresponding to posture frames.
The problem is that if the vectors were simply cut by shorter length of MFCCs and posture frames, aren't some information lost? For example, if the length of MFCCs is longer than posture frames, the part in posture frames exceeding the length of MFCCs will be deleted.
Looking forward to your reply!

Thank you for your question!
You are right. But the difference between those vector lengths is negligible (it comes from the sliding window in MFCC processing). You can print it out and convince yourself that it is just a few frames difference.
Apart from that, we are taking 1.5s of the past and 1.5s of the future for each sliding window, so even if there is a slight misalignment (1 frame out of 60) - all the relevant information will still be taken into account.
Does this answer your question?

Thank you for your reply!
Do you mean the alignment are done by sliding the window in MFCC processing at certain stride and window size? I see, thank you!
I also have another question.
Have you guys tried to train on one audio file at a time corresponding to one series of motion frames? I think the reason that in your work all audio are concatenated is that the conversation is continuous. What do you think of the performance of training the model on only one series of audio at one time.

Yes, you are right again: every sequence we obtain using a sliding window belongs to single audio. But the dataset contains sequences from many different audio files and they are being shuffled every epoch. So every batch during training contains sequences from different audios.

If you have questions on different aspects of the model - please, pose them in a separate issue

@wubowen416 , did I answer your question fully?

I understand! Thank you very much!


chore(combine-service): update and simplify Python environment

Removed constraint on version of requests_cache; closes #3080
Updated Python environment for COMBINE service
Changed pipenv to install into system Python environment

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


UHF-X: remove uninstalled module helfi_school_addons for good.

Remove remaining code of previously uninstalled module helfi_school_addons

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

Comparison is base (5987e30) 81.63% compared to head (1b9d66d) 81.63%.

:exclamation: Current head 1b9d66d differs from pull request most recent head c984d6a. Consider uploading reports for the commit c984d6a to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #163   +/-   ##
=========================================
  Coverage     81.63%   81.63%
  Complexity      361      361
=========================================
  Files            50       50
  Lines          2881     2881
=========================================
  Hits           2352     2352
  Misses          529      529

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


Static AGG Data

I have about 4.5 million rows in my database, showing 500 per page, and its working fine.. I want to aggregate "totals" for a column. My data is grouped, and is working. So for that "group" footer, is there a way to tell it what my aggregate values are, rather than let ag-grid calculate them?

yes, either:
a) provide the data to the grid already grouped (like the file explorer does). that way, you put whatever you like into the group fields. (ps - i've not used this with paging before, so I'm only 75% sure it will work, but good chance it will work without issues)
b) provide your own 'groupAggFunction' - although it will be provided with all the nodes to group, you just ignore that and get your results from somewhere else.


mne.viz.ica._plot_ica_overlay_evoked() broken (no red color)

Hello,
running this example
martinos.org/mne/stable/auto_examples/preprocessing/plot_ica_from_raw.html
the call of
ica.plot_overlay(eog_evoked, exclude=eog_inds)
(2nd last command)
produces only black lines.
It looks like this could be fixed by using
show=False
in line 457 of mne.viz.ica._plot_ica_overlay_evoked()
Cheers,
Sebastian

the figures do not render as in the web page for you?

I'm surprised, this code is in use every day. Can you say more about how
you use it and on which system with wich python distribution?
Maybe a reproducible example?
2015-02-22 9:53 GMT+01:00 Alexandre Gramfort<EMAIL_ADDRESS>
the figures do not render as in the web page for you?
—
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/1819#issuecomment-75426218
.

You can use the plot_ica_from_raw example. These are the problematic figures:

I'm on Mac OS 10.8.5 using system Python 2.7.8. mne is the current master version through git, matplotlib (1.4.2) is build with homebrew using the backend MaxOSX ("version unknown"). Same result if I force "Qt4Agg" as backend.

hum... so you get this just when running the untouched example?
if so can you try with anaconda?

That's crazy. We cannot reproduce this. I suspect it has to do with the
matplotlib build. You should use the 'osx' with mavericks.
2015-02-22 16:56 GMT+01:00 Sebastian Stober<EMAIL_ADDRESS>
You can use the plot_ica_from_raw example. These are the problematic
figures:
[image: figure_1]
https://cloud.githubusercontent.com/assets/8196250/6319138/cf495ce2-ba7f-11e4-9368-89a430ed854b.png
[image: figure_2]
https://cloud.githubusercontent.com/assets/8196250/6319139/cf497c04-ba7f-11e4-89a5-71e62420eaa4.png
I'm on Mac OS 10.8.5 using system Python 2.7.8. mne is the current master
version through git, matplotlib (1.4.2) is build with homebrew using the
backend MaxOSX ("version unknown"). Same result if I force "Qt4Agg" as
backend.
—
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/1819#issuecomment-75441801
.

I'm afraid there's no way I'm upgrading to Mavericks as it does not properly support some software I still depend on. I'd also rather not tinker with my otherwise perfectly functioning Python setup as it took me some time to get Theano and Pylearn2 working nicely together with Cuda support on my GPU.
Guess I'll use my custom plotting function (with show=False) for now.

Sorry my mistake I thought you were using Mavaricks. It would nevertheless be good to explore what's going on. I fear the problem is in matplotlib not in the mne-python code. I cannot really help a lot because I cannot reproduce it.

sounds fair.
Sorry if we cannot help more.

If you cannot reproduce it, I'd suspect mpl as well. I'll see if I can find out more about this. But for now I got more pressing stuff to attend to and I can live with my workaround.

FWIW I'm also experimenting this on macOS 10.12 (I do not see a red trace)
However it works again when I do this:
import matplotlib.pyplot as plt
...
ica.plot_overlay(eog_evoked, exclude=eog_inds, show=False)
plt.show()


DO NOT MERGE Match image versions to current downstream [backplane-2.5]

For testing the Konflux build only.

/hold


fix(install.sh): fix paths to Brew, return to current folder after execute .zshrc

Thank you for your commitment @urnix ! 👍


HOFF-158 added checkboxes

What
Added checkbox page with validation if user does not select an option
How
Based off RIL form where user is asked for income
Why
Demonstrate checkboxes for the basic form

Looks good to me


Mavenize docker images

Build pulsar docker images with maven. This will make it easier to
pull them in as a base for the maven integration tests.
I didn't touch the publish script, because it has handling for private
registries which I wasn't able to test.

@merlimat @sijie

retest this please


update im_journal params for RHEL7 versus others

From the module load pattern, it appears that im_journal only is appropriate for RHEL 7 - added if statement to that effect.

Closing in favor of a previous PR


[BUG]EFS storage block cloudformation stack deletion, even delete EFS by hand.

What happened:
EFS storage block Cloudformation stack deletion, even delete EFS by hand.
To Reproduce
Steps to reproduce the behavior:
Delete Cloudfromation stack

Attach logs
What you expected to happen:
Cloudformation stack can be deleted normally without manual operation.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

These CFN object is PV/PVC in EKS cluster. It should be automatically deleted with the EKS cluster.
In current implementation, we create static PV/PVC pair on CDK as a shared resource among runtimes. This is because prior of v1.7, Amazon EFS CSI Driver doesn't support customize sub-path for each PVC with same filesystem. On v1.7 update, Amazon EFS CSI driver can dynamic provision PV with same sub-path on actual filesystem.
The creation of PVC should be moved to Helm Chart, and use dynamic provisioning to create PV.

Close as #104 removed dependency of EFS.


BOUNDARY Function

Proposal:
I would like a function that you can add to an InfluxQL SELECT statement or similar FLUX statement that generates points at the time boundaries (ie. start and end time) of a data request.
Current behavior:
If I'm recording data by exception (ie. only when there is a change in the data), Influx will return no data for large sections areas of history. If I have a history point at 1:00pm (10 Deg) and one at 4:00pm (40 Deg), then request data between 2-3pm, I show nothing at all.
Desired behavior:
A 'BOUNDARY' function would specify that we would like to automatically generate data at the upper and lower boundary of the query. So in the example case, a record would be generated at 2:00am (20 Deg) and one at 3:00am (30 Deg).
Alternatives considered:
One of the only real alternatives is to add additional queries asking for the FIRST and LAST values on either edge of the query, then re-integrating the data. This can't be done server-side using InfluxQL. This might be possible using joins in Flux, but is still very awkward and time-consuming, particularly if you are querying many different measurement points (our product might request 1000 different points at any one moment).
Many process historian products - products that have done this sort of work for decades - have this important feature to ensure the graphs are informative. It's built into OSI Pi, Wonderware eDNA, even the ancient OPC HDA standard understands that this is an important feature to include.
Use case:
This helps resolve many, many issues in the database where people are requesting that the FILL function works with data outside the specified range. It allows Grafana dashboards to actually show the entire graph, rather than having blank points at the beginning of the graph ('fill' tends to give you data all the way to the most recent edge, but the data might be misleading when there is a radical change just out-side of the fill area).

Similar to Fill, it could have parameters to control the method of interpolation (NONE - disable the boundary, LINEAR - Linear interpolation, EXP - Exponential interpolation, STEP or LAST - hold the previous / last value).
You would have to specify your END date, or it would (like your GROUP BY statements currently do) assume that now() is your end date.
So a 'SELECT value FROM temperature WHERE machine="AR9-203" WHERE time > now() - 1h AND time <= now() BOUNDARY(LINEAR)'

I'm thinking that we can look at updating the Locations passed from the locations call in file_store.go.
If the location class included a reference to the first outside point(s) on either side of the range, we could then adjust the behaviour of queries that used KeyCursor, which appears to be all of the queries.
That data could potentially be used for boundary point generation.


[#779] feat: Grpc server support random port

What changes were proposed in this pull request?

grpc server port support random port
grpc sever will retry binding with increased port num if configured port is already occupied

Why are the changes needed?

improve robustness for server startup
sync with netty server functionality

Fixs #779
Does this PR introduce any user-facing change?
Yes. rss.rpc.server.port could be set as zero.
How was this patch tested?
UT

Codecov Report

Merging #820 (cad1903) into master (cae7cd9) will increase coverage by 1.33%.
The diff coverage is 73.07%.

:exclamation: Current head cad1903 differs from pull request most recent head 00f606e. Consider uploading reports for the commit 00f606e to get more accurate results

@@             Coverage Diff              @@
##             master     #820      +/-   ##
============================================
+ Coverage     57.63%   58.97%   +1.33%
  Complexity     2058     2058
============================================
  Files           306      292      -14
  Lines         14871    12909    -1962
  Branches       1221     1224       +3
============================================
- Hits           8571     7613     -958
+ Misses         5808     4864     -944
+ Partials        492      432      -60

Impacted Files
Coverage Δ

.../java/org/apache/uniffle/common/util/RssUtils.java
58.10% <0.00%> (-1.00%)
:arrow_down:

...java/org/apache/uniffle/common/rpc/GrpcServer.java
80.26% <81.81%> (+3.45%)
:arrow_up:

.../java/org/apache/uniffle/server/ShuffleServer.java
68.01% <100.00%> (-1.36%)
:arrow_down:

... and 19 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@leixm @advancedxy @zuston

Sorry for the late review.


Additional Graphs

The conclusion section of my SRP still lacks some analysis. More suggestions for graphs that could lead to further insight would be appreciated. This issue is for suggestions for graphing the COSORE data.

January 2021 suggestion:

For the WorldClim climate data, let's compute the standard deviation in addition to the mean, i.e. annual s.d. of the monthly climate data. This should just require adding one additional apply() computation in the Rmd file for each variable.
Then, it would be fascinating to look at annual Q10 of Rh (you can also try Rs, but Rh seems a cleaner test) versus s.d. of T and P, because we might hypothesize that microbial populations in highly variable systems would be better adapted to take advantage of those variations.

January 2021 suggestion:

For the WorldClim climate data, let's compute the standard deviation in addition to the mean, i.e. annual s.d. of the monthly climate data. This should just require adding one additional apply() computation in the Rmd file for each variable.
Then, it would be fascinating to look at annual Q10 of Rh (you can also try Rs, but Rh seems a cleaner test) versus s.d. of T and P, because we might hypothesize that microbial populations in highly variable systems would be better adapted to take advantage of those variations.

I was thinking of air temperature for this analysis, because it'll have the biggest swings.

I was thinking of air temperature for this analysis, because it'll have the biggest swings.


Duplicate event in SQLTracker

Rasa version: 1.0
Python version: 3.6.8
Operating system (windows, osx, ...): osx / postgreSQL
Issue: Duplicate event in SQLTracker. It seems the FLOAT precision can't handle the timestamp correctly. Actually, the field has a 5 digits precision, it needs at least 7.
The column type should be change to numeric.

Thanks for raising this issue, @wochinge  will get back to you about it soon.

This issue is also arising during interactive training since https://github.com/RasaHQ/rasa/blob/master/rasa/core/training/interactive.py#L235 expects the entire event history to be replaced (as done in the InMemoryTrackerStore https://github.com/RasaHQ/rasa/blob/master/rasa/core/tracker_store.py#L131) but the SQLTrackerStore only appends events that are after the greatest timestamp (https://github.com/RasaHQ/rasa/blob/master/rasa/core/tracker_store.py#L411) without replacing previous events

@claudio-7lieues In which setting did you see this problem?
@feynmanliang Thanks for the digging into this problem and providing the fix.
I have a look at your pr immediately.

During normal usage.


Allow PALLADIUM_CONFIG to be a list of config files

Implements 'configuration inheritance'.  This allows us to write:
PALLADIUM_CONFIG=etc/myconfig.py,etc/mybaseconfig.py
Entries in the configuration files earlier in the list will override those later in the list: If the contents of etc/mybaseconfig.py is "{'a': 42, 'b': 6}" and the contents of etc/myconfig.py is {'b': 7}, then the resulting configuration will be {'a': 42, 'b': 7}.
Let me know what you think.
EDIT: Had the example the wrong way around.

Coverage remained the same at 100.0% when pulling f9bea69ff297cab1bebf94fa5c94d8341529b896 on samsungaccelerator:feature-multiple-configs into 3e1c89daecf8df97205c4b2019db9b0001de2e97 on ottogroup:develop.

Look good. I would rather expect that configurations that occur later overwrite the ones to the left. Any reason why you reversed the list?

Look good. I would rather expect that configurations that occur later overwrite the ones to the left. Any reason why you reversed the list?

Interesting.  :-)
I did it because it's how class inheritance works in Python.  But it may well be less intuitive.  Let's wait for a third opinion?

One could think this might not be an independent third opinion, but maybe @BenjaminBossan wants to share his thoughts?

I didn't look too closely at the examples but since you ask, my intuition would be that we start from the left and update the dictionary with the consecutive new dictionaries.

OK I flipped the order.  :-)

Coverage remained the same at 100.0% when pulling 433c705205bf23e16f1f9e3a1640944bce47ce29 on samsungaccelerator:feature-multiple-configs into 3e1c89daecf8df97205c4b2019db9b0001de2e97 on ottogroup:develop.

Can we get this merged?  Thanks.

Coverage remained the same at 100.0% when pulling fbc1fd38742b06f50f2cb1f3a26b7b20aa8eac0a on samsungaccelerator:feature-multiple-configs into 8d6e2d6ba447ba2c1ed45f5fc10ccfe6c1a6694a on ottogroup:develop.

Sure, I am going to merge it. I have a bad Internet connectivity during my vacation this and the upcoming week.

@alattner Oh sorry had no idea. :-)


Keyboard accessibility in dialogs

I just tried to hit right+enter to say no to one of the "Are you sure?" dialogs
that don't work 😅
so I got a save file dialog instead to close instead of discarding the changes - not the worst outcome, but the "dialog" failed it's job
So basically, left/up cycles backwards, and right/down cycles forwards.
I already have cycling set up for tabbing, but not shift+tabbing.

Turns out there was a regression, I had actually already implemented arrow keys for dialog windows.
I fixed it in https://github.com/1j01/jspaint/commit/acb1f19abfd41110b071fac5e13b963b01937ac6.
But I'll leave this open for the further improvements I described in the above comment.

This is done.


[build] code coverage checker is missing

Recent PR opened misses code coverage checker via coveralls, shows travis build checker only. Creating issue for reference.
May related with https://github.com/ReactiveX/RxJS/issues/1304, temporal failure of coveralls.

As mentioned in #1304, migrate into codecov (https://codecov.io) is considerable option in further.

I just did a quick try to codecov and migration path was surprisingly easy, example result can be seen at https://codecov.io/github/kwonoj/RxJS/src?ref=258c5fb2ab4335eb17cfaa72cecccdb0071943e4 . If migration's needed, I'll create PR.

Seems checker's coming back finally, closing issue. #1449 can be used to discuss if migration would be required or not.


fix: remove key collision logging

While this log message is nice to have, it's not worth having an infinite loop over
Alternative solutions:

Consider renaming colliding field to avoid collision (changing the key name isn't ideal)

Issue #, if available: Fixes #33
Description of changes:
Remove log message that may cause infinite loop
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I'll merge this while we determine if #35 can be a solution. Thanks!


fix: avoid flash or non-reponsiveness of color mode toggle button using the ColorScheme component

according to the nuxt-color-mode docs, without the ColorMode component, there will be either a flash or non-responsiveness of the color mode toggle button because the package is yet to determine the system's color theme and this behavior is as so on the live env:
https://github.com/danielroe/roe.dev/assets/44336070/515c36fa-fc09-4446-a68f-e02aef6927a9
current behavior 👇🏽
https://github.com/danielroe/roe.dev/assets/44336070/03b7200a-2019-48d2-ab76-6973ce7b3233

Summary by CodeRabbit

Refactor

Enhanced the color mode toggle feature for improved usability and visual appeal.

okay, but you undid it 🤔

oh my bad, i looked at the wrong commit
apologies 🙏🏽

i'd appreciate your feedback/star on this (if you think it deserves it) — https://github.com/idorenyinudoh/adura-portfolio

Nice! 👌

appreciate it, Daniel! 🙏🏽


Add and enable 9009 rings

This commit creates new rings that say "AWU" and "CWA 9009" in regular (white-on-red), trans, and pride variants.

Thanks!


Allow empty filter in ValueSet 'expand' search

Coverage decreased (-0.001%) to 94.605% when pulling 504822022aaecb6f233c97bdb2824050eaa40b13 on cody-allow-empty-expand-search into 2601e4eee5cd60b7349fc6395111a7be1150a9de on main.


When completing a function, and the input matches at least one candidate exactly, function snippets are not included

The title is a bit of a mouthful, but the problem is as follows. Say you have this Rust file (where | signals the cursor position):
use std::alloc::Layout;

fn main() {
    Layout::|
}

When triggering completion, all the expected candidates show up: Layout::from, Layout::from_size_align, etc. When choosing a completion item, snippets and such are all processed as expected. For example, if we choose Layout::from_size_align, we end up with this:
use std::alloc::Layout;

fn main() {
    Layout::from_size_align(size, align)
}

The problem arises when one types Layout::from and then triggers completion. In this case rust-analyzer still shows the 3 methods you'd expect: Layout::from, Layout::from_size_align, and Layout::from_size_align_unchecked. The problem is that when you choose an entry, snippets are not applied. The result is that you end up with something like this:
use std::alloc::Layout;

fn main() {
    Layout::from_size_align
}

This happens every time the input of the completion query matches at least one function entry. So if the input is Vec::with_capacity, you can't complete the method with snippets and all that.
I confirmed this is happening both in NeoVim and when using vscode. Here's a recording showing that happening in vscode:
https://gfycat.com/simplisticidleguernseycow
Here's the same issue happening in NeoVim:

The expected behaviour here is to get snippet completion, regardless of whether the input is Layout::fr, Layout::, Layout::from, or something else.
My version of rust-analyzer is rust-analyzer 617535393 2021-04-26 dev, using Rust rustc 1.51.0 (2fd73fabe 2021-03-23) and Cargo cargo 1.51.0 (43b129a20 2021-03-16).

The problem here seems to be that the expected type for completions becomes the already written function's type which then causes us to skip the the snippet completion as we do not want to add calling parens when completing functions where a function is expected

Yeah, that's what I expected. We probably need to record more detailed information about what type was expected and what type we actually got during inference, but I'm not completely sure what the best way for that is (i.e. whether the 'obvious' way will give us the information we hope). It might need some experimenting.

This shouldn't be too hard to fix in this case I believe, currently the code checks the trailing expression of the function to determine its return type here https://github.com/rust-analyzer/rust-analyzer/blob/544a93ee0815697ff42b79e54d1a7a5a743de1f9/crates/ide_completion/src/context.rs#L379-L387
Instead we should lower the actual return type of the function and use that if I'm not mistaken(which probably wasn't done yet as sema doesn't expose a way to go from arbitrary ast::Type to hir::Type.

That would only fix it for this specific case, but already not work for
fn main() {
    Layout::from|;
}

, I believe. I don't think we should be lowering any types in this code.

Mmh that's a good point I guess

Though actually, reading that code, I'm not sure whether the bug would actually happen in that situation? Seems that the "expected type" logic already involves more special-casing than I thought, so maybe it's fine for now :man_shrugging:

Ye it already quite special cased, which is unfortunately necessary. We can just map the function node to its hir def and take out the return type that way though, so no need to lower anything extra actually.

Fixed in https://github.com/rust-analyzer/rust-analyzer/pull/8715(relevant test is https://github.com/rust-analyzer/rust-analyzer/pull/8715/files#diff-651615d626a57d22f045214c88f3eba75d41e96c0f2f96d121bd2a42b621f520R866-R872)

I just confirmed https://github.com/rust-analyzer/rust-analyzer/pull/8715 solves this issue. Thanks all! :tada:


Add support for custom scripts

Is there an existing feature request for this?

[X] I have searched the existing issues.

Command
flutterw run

This can be solve by adding a custom command to flutterw.
See doc: Adding Custom Command

Closed as commented above. Anyone who have requests that custom command cannot cove can fire a new issue.


entity:viewport

http://docs.autodesk.com/ACD/2011/ENU/filesDXF/WS1a9193826455f5ff18cb41610ec0a2e719-79c6.htm

Hi,
Not sure if it has some utility, it just creates view ports, also its complicated 🤔.


Expression Bodied Members w/ String Interpolation breaks Syntax highlighting

This is similar to issue #249 however in this scenario, it stops performing any syntax highlighting for the rest of the source file. This is using 1.0.10

It looks like this is specific to the string interpolation, since Expression Bodied Members without string interpolation appear to work fine.
// Works fine
public IConnection[] GetConnections() =>
    this.playerConnections.Select(keyValuePair => keyValuePair.Value).ToArray();

while string interpolation breaks it
// Breaks
public override string ToString() => $"{this.game.Name} - Adapter: {this.Name}";

This is a duplicate of #249. We'll use that issue to track this.


Implicit grant

What do you think about implementing implicit (token) grant?
https://tools.ietf.org/html/rfc6749#section-4.2

+1


Python3.10 support.

Fixes: #78

pre-commit.ci has upgraded to 3.10, so this is now broken. Also, this is my pet peeve - it's artificially broken by adding <3.10, instead of actually being broken. If you must put an upper cap (you never must put one), then please release it as soon as the betas start coming out, or the RCs. I'd much rather see a real error on Python 3.10 than just having it break like this.

This PR will be merged after supporting the new case-matching syntax (very soon).


[FEATURE] Placeholder type THIS to be replaced in derived classes.

Java does not have a generic placeholder to represent the actual current class, as opposed to the declaring class of any field, method, and thelike. This would be tremendously helpful e.g. with methods that are supposed to return the object which's method is called, similar to a builder-pattern:
MySubclass myObject= ...;
myObject.refresh().doSomethingElse();

While the above will work, if doSomething is defined on or above the same type as refresh, this is  - more often than not - not the case, especially if MySubclass is derived from a type declaring many standard functions for it's derivates. Assuming that refresh returns the called upon instance, it could be the same type as myObject, assuming at the time of invokation, I knew that type.
To achieve this, and handle similar issues, a placeholder annotation type could be introduced, similar to the var and val types, which is then replaced in the declaring class (and all compilation units marked with a certain annotation). Consider the following example:
@...
public class MySupertype {
    // will become signature:
    // public MySupertype refresh();
    public THIS refresh() {
        //...
        return this;
    }
}

@Redeclare
public class MySubtype  extends MySupertype {
    // will redeclare all fields and methods with 'THIS':
    // public MySubtype refresh() {
    //     return (MySubtype) super.refresh();
    // }
    // this implicitly declares the refresh method again, with the more specific return type of this class.
}

In this example, both MySupertype and MySubtype  declare a refresh method, but with different return types. The only issue I see is that the compilation units are separate. However, since MySubtype is a subtype of MySupertype it should theoretically be possible to leave some trace in the supertype, which could then be used in the subtype to save the day.

Sounds like a straightforward variant of
https://github.com/rzwitserloot/lombok/wiki/LOMBOK-CONCEPT:-Resolution
.. which means lombok probably can't solve this problem, sorry.
..
On Sat, Apr 18, 2020, 09:28 TreffnonX<EMAIL_ADDRESS>wrote:

Java does not have a generic placeholder to represent the actual current
class, as opposed to the declaring class of any field, method, and thelike.
This would be tremendously helpful e.g. with methods that are supposed to
return the object which's method is called, similar to a builder-pattern:
MySubclass myObject= ...;
myObject.refresh().doSomethingElse();
While the above will work, if doSomething is defined on or above the same
type as refresh, this is - more often than not - not the case, especially
if MySubclass is derived from a type declaring many standard functions for
it's derivates. Assuming that refresh returns the called upon instance,
it could be the same type as myObject, assuming at the time of
invokation, I knew that type.
To achieve this, and handle similar issues, a placeholder annotation type
could be introduced, similar to the var and val types, which is then
replaced in the declaring class (and all compilation units marked with a
certain annotation). Consider the following example:
@...public class MySupertype {
// will become signature:
// public MySupertype refresh();
public THIS refresh() {
//...
return this;
}
}
@Redeclarepublic class MySubtype  extends MySupertype {
// will redeclare all fields and methods with 'THIS':
// public MySubtype refresh() {
//     return super.refresh();
// }
// this implicitly declares the refresh method again, with the more specific return type of this class.
}
In this example, both MySupertype and MySubtype declare a refresh method,
but with different return types. The only issue I see is that the
compilation units are separate. However, since MyDerivedType is a subtype
of MyHibernateType it should theoretically be possible to leave some
trace in the supertype, which could then be used in the subtype to save the
day.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rzwitserloot/lombok/issues/2432, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABIERPLYUKFVWUKUJ4YIZDRNFJDRANCNFSM4MLHZKUA
.

Except... you do it the @SuperBuilder way: Create a static inner class ThisTypeHelper (with fancy type params), which you could use as return type. That class provides a get() method, which has the correct return type (with the help of the type params).
I've not tested this, so I'm not sure that this really works. Similar to @SuperBuilder, you'd need it on every class in the hierarchy, and you'd have to call the get() method instead of just using the returned value.

Well, if anyone has the expertise to implement something like that it's you
;)
On Sat, Apr 18, 2020, 12:17 Jan Rieke<EMAIL_ADDRESS>wrote:

Except... you do it the @SuperBuilder way: Create a static inner class
ThisTypeHelper (with fancy type params), which you could use as return
type. That class provides a get() method, which has the correct return
type (with the help of the type params).
I've not tested this, so I'm not sure that this really works. Similar to
@SuperBuilder, you'd need it on every class in the hierarchy, and you'd
have to call the get() method instead of just using the returned value.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rzwitserloot/lombok/issues/2432#issuecomment-615836953,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABIERKULU4XZS6O65STCELRNF42NANCNFSM4MLHZKUA
.

I thought again and tried around with this. I'm pretty sure this will only work if we put the type param directly on the annotated class. Generating a parameterized inner helper class will not work. So in the end the code will look like this:
public static class Superclass<THIS extends Superclass<THIS>> {
	@SuppressWarnings("unchecked")
	public THIS refresh() {
		return (THIS) this;
	}
}

public static class Subclass<THIS extends Subclass<THIS>> extends Superclass<THIS> {
}

public static class Subsubclass<THIS extends Subsubclass<THIS>> extends Subclass<THIS> {
	public THIS refresh() {
		// Do something else here.
		return super.thisReturn();
	}
}

public static void main(String[] args) {
	Subclass<?> sub1 = new Subclass<>();
	Subclass<?> sub2 = sub1.refresh();

	Subsubclass<?> subsub1 = new Subsubclass<>();
	Subsubclass<?> subsub2 = subsub1.refresh();
}

It works, but the price is to high in my opinion, because you need those seemingly useless type params everywhere you use that class.

Thank you for trying though, it seemed simpler when I thought about it. But I understand that - more often than not - the restraints lombok must function under prevent such things. Hopefully, one day there will be a solution to this, but for now it seems we have to stick with the verbose redeclaraton.

Let's wait what the Lombok maintainers say. Maybe they have another idea.

There is a possible solution that works without those type params:

Rename the annotated class (e.g. AbstractMySupertype), make it abstract, and add the <THIS extends A<THIS>> type param and use it as return type.

Subclasses inherit from this abstract type (and will also be processed accoring to this steps).

Create a public concrete implementation MySupertype without a type parameter.
Create an interface with the orginal inheritance, so that instanceof on the concrete classes still works as expected.

However, there is a technical problem: lombok does not generate new compilation units, it just modifies existing ones. As the JLS does not allow more than one public class in a compilation unit, all other have to be package-private. So if you want to inherit from such a class across package borders, that won't work.

@janrieke

So if you want to inherit from such a class across package borders, that won't work. That's a show-stopper IMO.

On the opposite, it's a feature. You need threes type per class, which is a lot and making them all top-level makes it even worse. When you want to extend them in a different package, you need to make the abstract scaffold public, too. So I'd go for classes nested in an interface which would be IMHO much nicer as the interface is what's needed most of the time (actually, all the time; see below). It could look like
public interface Node {
    public static abstract class Scaffold<THIS extends Node.Scaffold<THIS>>
        implements Node {....}
    public static class Impl extends Node.Scaffold<Impl> {....}

    public static Node create(....) {....}

    .... declarations
}

public interface SpecialNode extends Node {
    public static abstract class Scaffold<THIS extends Scaffold<THIS>>
        extends Node.Scaffold<THIS>
        implements SpecialNode {....}
    public static class Impl extends SpecialNode.Scaffold<Impl> {....}

    public static SpecialNode create(....) {....}

    ....
}

....

The nesting and the use of strange name helps to prevent mistakes like o instanceof Impl. Normally, all you need is the top-level interface, only when extending, you need the nested classes. But this extending gets done by Lombok, the user could write
@THIS interface VerySpecialNode extends SpecialNode {
    /// define methods like in a class, use `lombok.THIS` as the return type
    THIS clone() {....}
}

or something like this. No generics, no references to either Scaffold or Impl anywhere.
I may be missing something....

So users of the annotated class should only work with the interface? Then you need the type params on the interface, otherwise the problem with the unnecessary <?> remains. Or did I get you wrong?

@janrieke
So users of the annotated class should only work with the interface? Then you need the type params on the interface, otherwise you won't have the actual return type. And then the problem with the unnecessary <?> remains. Or did I get you wrong?
I wanted to hide all generics in the generated code, but I may be completely wrong....
It should go as follows: The user defines the classes using e.g.,
@THIS
@RequiredArgsConstructor
@Getter
interface SpecialNode extends Node {
    public THIS clone() {
        return new SpecialNode(parent);
    }

    private final THIS parent;
    @Setter private THIS otherNode;
}

@THIS
@Getter
interface VerySpecialNode extends SpecialNode {
    VerySpecialNode(THIS parent, String someString) {
        super(parent);
        this.someString = someString;
    }

    public THIS clone() {
        return new VerySpecialNode(parent, someString);
    }

    public THIS clone() {
        return new VerySpecialNode(parent);
    }

    private final String someString;
}

and Lombok creates the nested Scaffold class and move the code therein. Additionally it adds the Impl class and the static create method to the interface. The user would write things like
VerySpecialNode vsn = VerySpecialNode.create(null);
vsn.setOtherNode(VerySpecialNode.create(null, "someString");
assertTrue(vsn instanceof SpecialNode);
assertTrue(vsn.clone() instanceof VerySpecialNode);
assertEquals("someString", vsn.getOtherNode().getSomeString());

with no generics at all.
What obviously can't work ideally, is using .getClass(). This may be a problem in @EqAHC, too.


Ie8 support?

There is no polyfill for object.defineProperty. Is there a version that does support ie8?
We absolutely need to have ie8 support and rivets is the only viable databinding solution for backbone in a production environment. I would be happy to help fix this. We would all love to not support ie8 but the sad reality or corporate customers refuse to upgrade and demand support.

@MiguelCastillo Any chance those files could be place somewhere else? I've been looking for this solution also.

@Jmeyering I have been meaning to do that.  Let me see if I can do it sometime tomorrow. At the very least, I will put them on a small git repo.

@MiguelCastillo Thanks so much!

@MiguelCastillo I can't find the repository with ie7 support. Did you create it?

Hey folks, I am a bit embarrassed to say this but I don't have the files anymore...  @timothyswt any chance you saved those files?
I don't want to make promises because I am really struggling to find time, but if nobody else has the files I will try to find some time for this during the weekend.


Update README.md

As discussed: improved Usage section and Design Goals mentions other library where the toarray struct tag idea came from.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #71   +/-   ##
=======================================
  Coverage   97.42%   97.42%
=======================================
  Files           6        6
  Lines        1361     1361
=======================================
  Hits         1326     1326
  Misses         23       23
  Partials       12       12

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 1113a86...636098f. Read the comment docs.


⚠ Bofrost.fr has degraded performance

In 69dfadf, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1491 ms

Resolved: Bofrost.fr performance has improved in 7040aca after 311 days, 19 hours, 35 minutes.


Delete all records in an array

I am working with JSON data that contains arrays of class data. At some point, I need to remove all of the items from the arrays to start processing over on new data. I want to be able to remove all of the items in the array with one call instead of having to make a call for each item since each array can contain up to 100,000 items each.
How can I make my JSON data look like the following data use the minimal amount of calls?
{
"mse_points": [],
"rop_points": []
}
A sample of the fill out JSON data that looks like:
{
"me_points": [
{
"x": 26,
"y": 12,
"value":<PHONE_NUMBER>03733,
"dataCount": 132,
"id": 2612
},
{
"x": 26,
"y": 13,
"value":<PHONE_NUMBER>03733,
"dataCount": 132,
"id": 2613
},
{
"x": 26,
"y": 11,
"value":<PHONE_NUMBER>03733,
"dataCount": 132,
"id": 2611
}
],
"rp_points": [
{
"x": 26,
"y": 12,
"value":<PHONE_NUMBER>03733,
"dataCount": 132,
"id": 2612
},
{
"x": 26,
"y": 13,
"value":<PHONE_NUMBER>03733,
"dataCount": 132,
"id": 2613
},
{
"x": 26,
"y": 11,
"value":<PHONE_NUMBER>03733,
"dataCount": 132,
"id": 2611
}
]
}

Have you found an answer for this? I am wondering the same thing as well.


Capitalize public/private status that appears on the character detail page

Closes #267
Description
The string public or private that appears on the character detail page now is now capitalized.

@m-triassi is this allowed? does BE allow status to be sent as Private or Public, if it is allowed i guess this is a solution

@m-triassi is this allowed? does BE allow status to be sent as Private or Public, if it is allowed i guess this is a solution

iirc any where queries we use in the back end are not case sensitive (The test suite would fail if they were), in an ideal world, the text displayed would be upper case but the data returned to the back-end would be lowercase -- personally its not a big deal for me though.

@AnnieTran13 , let's send backend lowercase and have display text to be uppercase for the first letter

haha! HUGE PR!! 🚀 ill say this is fine for merge :)


use git dependency

This is a temporary fix as it is probably not a good idea to track the main branch. But I assume it will be fixed shortly? In any case this makes cargo-leptos work on my machine.

Thanks a lot!


Minor improvements on chore search

cater for numeric parameter values
add readme for tests
fix all chore tests

successor of #663

@adscheevel
Do these small changes look OK to you?

Nice fixes, thanks!


Library Request: alpaca-py

alpaca-py is required for me replicate trades from my alpaca account to IBKR from quantconnect.
Checklist:
I have completely filled out this template
I have confirmed that this issue exists on the current master branch
I have confirmed that this is not a duplicate issue by searching issues
I have provided detailed steps to reproduce the issue

Hi @ganeshvenepally ,
QuantConnect doesn't allow REST API POST calls to third parties.
We are working on supporting Alpaca:
https://github.com/QuantConnect/Lean.Brokerages.Alpaca


Shell unexpectedly exits or crashes when elevating a new PowerShell instance.

Environment
Windows build number:
PS >[Environment]::OSVersion
Platform ServicePack Version      VersionString
-------- ----------- -------      -------------
 Win32NT             10.0.19041.0 Microsoft Windows NT 10.0.19041.0

Windows Terminal version (if applicable):

Any other software?
PS > $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.19041.1
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.1
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

PS > $PSHOME
C:\Windows\System32\WindowsPowerShell\v1.0

Steps to reproduce

Run the command Start-Process PowerShell -Verb RunAs "-NoProfile -ExecutionPolicy Bypass -Command `"cd '$pwd'; & echo hello world`"";
Approve the windows prompt for elevation
Repeat steps 1 and 2 between 2-5 times in a row
The host tab will close even though the user did not exist the shell.

This was discovered with an auto elevating script such as this example. The script will run 1 or 2 times and then the host Windows Terminal tab will close unexpectedly. I have tried replicating with other shells with no luck. I am not certain this is only a PowerShell issue.
Expected behavior
The user should be able to run the command Start-Process PowerShell -Verb RunAs "-NoProfile -ExecutionPolicy Bypass -Command `"cd '$pwd'; & echo hello world`""; any number of times and not have Windows Terminal close the tab.

Actual behavior
Run the command Start-Process PowerShell -Verb RunAs "-NoProfile -ExecutionPolicy Bypass -Command `"cd '$pwd'; & echo hello world`""; between 2-5 times in a row and the host tab will close.

Windows PowerShell
Copyright (C) Microsoft Corporation. All rights reserved.

Try the new cross-platform PowerShell https://aka.ms/pscore6

PS > Start-Process PowerShell -Verb RunAs "-NoProfile -ExecutionPolicy Bypass -Command `"cd '$pwd'; & echo hello`"";
PS > Start-Process PowerShell -Verb RunAs "-NoProfile -ExecutionPolicy Bypass -Command `"cd '$pwd'; & echo hello`"";
PS > Start-Process PowerShell -Verb RunAs "-NoProfile -ExecutionPolicy Bypass -Command `"cd '$pwd'; & echo hello`"";
PS >
[process exited with code 0]

I was able to reproduce this in PowerShell v7 and v5 terminals. So this might not be a WT problem. I also noticed this morning that this might be related to an overly aggressive antivirus. I will post again/reopen if that proves not to be the case.

Ah! Thanks for following up. :smile:


Outdated .net core 3

https://docs.microsoft.com/en-us/dotnet/core/whats-new/ this page does not have  description for .Net core 3 Preview 2 yet)
Also it would be good to add the expected releasedate for RTM version
Regards

Thanks for your feedback @owingruters. I've opened a PR to fix the link description and other little things: https://github.com/dotnet/docs/pull/10317
The content inside https://docs.microsoft.com/en-us/dotnet/core/whats-new/dotnet-core-3-0 is already for preview 2.
Schedule information is always posted on the roadmap page on the dotnet/core repo.


change license specification in gemspec

for compatibility with license_finder gem, allowing automatic license checks.
Bundler already proposes the same format in it's newgem.gemspec template:
https://github.com/bundler/bundler/blob/master/lib/bundler/templates/newgem/newgem.gemspec.tt#L16
Let use it for bundler itself too.

@homu r+

:pushpin: Commit 2238385 has been approved by segiddins

:zap: Test exempted - status


Forward port fixes done for DAS 3.1.0 to APIM-Analytics 2.1.0

Description:
$subject
Suggested Labels:
improvement, das-3.1.0
Suggested Assignees:
@minudika
Affected Product Version:
2.1.0
OS, DB, other environment details and versions:
N/A
Steps to reproduce:
N/A
Related Issues:
N/A

Forward porting fixes is done. Hence closing this issue.


trim requirements.txt

I have no idea why most of these requirements are there.

Coverage remained the same at 97.505% when pulling faf6d957f454ffbc1e63135c531750089ce2bee1 on smurfix:master into 7fd738bb98e3e464538b11fc62a2473fd174486a on hMatoba:master.

Thank you!


Video unavailable

Script version
5.6
Describe the bug
Video unavailable
Playback on other websites has been disabled by the video owner
[Watch on YouTube]
This is what pops up on a grey background on some videos.
Expected behavior
I would expect to watch the video normally without any issues. Upon turning the script off, everything is fine
Screenshots
Desktop (please complete the following information):

OS: Windows 10
Browser firefox
Version 126.0.1 (64-bit)

Additional context
Not sure if it happens elsewhere.

I can confirm, I am seeing the same error.
OS: Windows 11,
Browser: Microsoft Edge
RemoveAdBlockThing version: 5.6


possible to do a http2 websocket example? current one is only http1.1

pls help get this up. do also pls put up a section for sponsorship at least for us to buy u coffees. it's really good.
i see greater adoption with actual use case scenario (case studies/simple info of current users) and appreciation page for xitca.

if websocket http2 is available. i can put in example use case for reference with results of production run/use.
thx

The websocket example work for http2 if you enable xitca-web's http2 crate feature.
This project is aiming to be 100% free and we can't dedicate extra time/effort to it even with extra funding. Contributions like Issues and PRs are welcomed and it'll be the main drive force of this project.

@fakeshadow ok.


program gets into an infinite loop

Hello,
I'm trying to run your pre-trained model on my test dataset, to see the effect on it.
This is the content of the subfolders:
(vodkaplus) eden@eden-xps:~/ToDayGAN$ ls checkpoints/robotcar_night2day/
150_net_D0.pth  opt.txt

(vodkaplus) eden@eden-xps:~/ToDayGAN$ ls datasets/tmp/*
datasets/tmp/test0:

datasets/tmp/test1:
hamburhnight1.jpg

The program just doesn't pass this step:
(my_env) eden@eden-xps:~/ToDayGAN$ python test.py  \
    --phase test  --which_epoch 150  \
    --serial_test  \
    --dataroot ./datasets/tmp  \
    --name robotcar_night2day  \
    --n_domains 2  \
    --loadSize 512

/home/eden/anaconda3/envs/my_env/lib/python3.8/site-packages/torch/cuda/__init__.py:143: UserWarning:
NVIDIA GeForce RTX 3050 Ti Laptop GPU with CUDA capability sm_86 is not compatible with the current PyTorch installation.
The current PyTorch install supports CUDA capabilities sm_37 sm_50 sm_60 sm_61 sm_70 sm_75 compute_37.
If you want to use the NVIDIA GeForce RTX 3050 Ti Laptop GPU GPU with PyTorch, please check the instructions at https://pytorch.org/get-started/locally/

  warnings.warn(incompatible_device_warn.format(device_name, capability, " ".join(arch_list), device_name))
------------ Options -------------
aspect_ratio: 1.0
autoencode: False
batchSize: 1
checkpoints_dir: ./checkpoints
dataroot: ./datasets/22
display_id: 0
display_port: 8097
display_single_pane_ncols: 0
display_winsize: 256
fineSize: 256
gpu_ids: [0]
how_many: 50
input_nc: 3
isTrain: False
loadSize: 512
max_dataset_size: inf
nThreads: 2
n_domains: 2
name: robotcar_night2day
ndf: 64
netD_n_layers: 4
netG_n_blocks: 9
netG_n_shared: 0
ngf: 64
no_flip: False
norm: instance
output_nc: 3
phase: test
reconstruct: False
resize_or_crop: resize_and_crop
results_dir: ./results/
serial_test: True
show_matrix: False
use_dropout: False
which_epoch: 150
-------------- End ----------------
/home/eden/anaconda3/envs/my_env/lib/python3.8/site-packages/torchvision/transforms/transforms.py:287: UserWarning: Argument interpolation should be of type InterpolationMode instead of int. Please, use InterpolationMode enum.
  warnings.warn(

I also notices cuda warning.
Could you please help?
Thank you

I figured out that the problem was coming from the cuda version. I fixed it with
pip3 install torch==1.10.0+cu113 torchvision==0.11.1+cu113 torchaudio==0.10.0+cu113 -f https://download.pytorch.org/whl/cu113/torch_stable.html
Just a remark, the train0 folder shouldn't be empty. Otherwise the program crashes.
Thank you again