Dropping AlternateScreen breaks alternate screen mode with Crossterm instance

extern crate crossterm;
use self::crossterm::{Crossterm, Screen, ClearType};
use std::io::Read;
use std::{thread, time};

fn main() {
    let raw_mode = true;
    let screen = Screen::new(raw_mode);
    let crossterm: Crossterm;
    {
        let alternate_screen = screen.enable_alternate_modes(raw_mode).unwrap();
        crossterm = Crossterm::from_screen(&alternate_screen.screen);
    }
    // <-- Dropping alternate_screen here

    let mut input = crossterm.input().read_async().bytes();
    let terminal = crossterm.terminal();
    let cursor = crossterm.cursor();
    cursor.hide().unwrap();

    loop {
        terminal.clear(ClearType::All).unwrap();
        cursor.goto(1, 1).unwrap();
        match input.next() {
            // These user inputs are not in raw mode
            Some(ipt) => {
                terminal.write(format!("{:?}    <- Character pressed", ipt)).unwrap();
                match ipt {
                    // Does not return to original screen
                    Ok(b'q') => break,
                    _ => {},
                };
            },
            _ => {}
        };
        thread::sleep(time::Duration::from_millis(200));
    }
}

When creating a Crossterm instance from an AlternateScreen, dropping the AlternateScreen breaks both raw and alternate screen mode, see example above. This gets tricky when users assume it is possible to only store the Crossterm instance to draw to the alternate screen.

It turns out it is possible to stay in raw mode by calling disable_drop() on AlternateScreen's Screen like so:
...
{
        let mut alternate_screen = screen.enable_alternate_modes(raw_mode).unwrap();
        alternate_screen.screen.disable_drop();
        crossterm = Crossterm::from_screen(&alternate_screen.screen);
    }
    // <-- Dropping alternate_screen here
...

However this feels like a bit of a hack. Alternate screen mode still breaks after the drop.

It is 'normal' behavior for an AlternateScreen to be switched back to mainscreen when it is dropped. AlternateScreen is meant for quickly doing something and not have it open for a long time. So, for now, you are bounded to the scope in which you are allowed to use the alternate screen.
When Screen drops it will turn off the raw modes. There are usecases you don't want that from automatically happening so, therefore, you can call disable_drop().

In the future, this might be more flexible and I will build in the functionality to have multiple screen buffers who have their own settings. You can be using lazy_static this allows you to have your crossterm type living longer.
I get your argument that the user might be misled and I take a look to see if I can improve the docs here. Are there any problems you are encountering with otherwise I'll close this issue.

Thanks for the quick reply. I think AlternateScreen's behavior make sense and isn't the issue here. What threw me off is that if I create a Crossterm instance from an AlternateScreen's screen, the Crossterm instance 's behavior depends on whether the AlternateScreen has been dropped or not. I'm not very familiar with your code, but it looks like AlternateScreen and Crossterm share ownership of the TerminalOutput via Arc's, so they can both mutate it at will. In the worst case scenario you could create a Crossterm instance from an AlternateScreen's screen, send the AlternateScreen to whatever thread you like and use it to manipulate the screen, until you eventually drop it. The Crossterm could be doing the same in a completely different part of your code and work as expected, until the AlternateScreen drops and the behavior suddenly changes. It would be extremely difficult to figure out where this change comes from without going through the source, because it has nothing to do with what you do to the Crossterm locally.
Given that Crossterm seems to be a convenience wrapper, could it consume an AlternateScreen so that there's only one place to change behavior? Alternatively, could Crossterm retain a reference to AlternateScreen, making sure they're used in the same place?

closing this, because it is answered if you have any more question feel free to reopen.


Update README - Add Brew Support

Most users will know to check BREW but it can be helpful for folks to know that Brew is a viable option for installing this tool.

:+1:


ci: use large CCI hosts

Description of changes
I'm seeing e2e test hosts intermittently running out of memory (see example: https://app.circleci.com/pipelines/github/aws-amplify/amplify-cli/6860/workflows/e647898f-ad82-46fe-95e2-0735b3310940/jobs/178939). This PR bumps our resource class so that this doesn't happen.

Issue #, if available

Description of how you validated changes
Checklist

[x] PR description included
[x] yarn test passes

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

Codecov Report

Merging #9929 (144e9b8) into master (2bd4172) will increase coverage by 0.36%.
The diff coverage is 87.60%.

@@            Coverage Diff             @@
##           master    #9929      +/-   ##
==========================================
+ Coverage   53.00%   53.36%   +0.36%
==========================================
  Files         830      833       +3
  Lines       46012    46092      +80
  Branches     9826     9838      +12
==========================================
+ Hits        24390    24599     +209
+ Misses      19604    19489     -115
+ Partials     2018     2004      -14

Impacted Files
Coverage Δ

...rmation/cdk-stack-builder/apigw-stack-transform.ts
11.76% <0.00%> (ø)

...vice-walkthrough-types/cognito-user-input-types.ts
100.00% <ø> (ø)

...-graphql-auth-transformer/src/utils/definitions.ts
100.00% <ø> (ø)

...amplify-graphql-index-transformer/src/resolvers.ts
99.59% <ø> (ø)

...chable-transformer/src/cdk/create-cfnParameters.ts
100.00% <ø> (ø)

...earchable-transformer/src/generate-resolver-vtl.ts
90.47% <ø> (ø)

...appsync-simulator/src/velocity/value-mapper/map.ts
86.88% <50.00%> (+0.21%)
:arrow_up:

...plify-graphql-relational-transformer/src/schema.ts
94.35% <50.00%> (-0.57%)
:arrow_down:

...c/extensions/amplify-helpers/auth-notifications.ts
51.72% <66.66%> (+23.54%)
:arrow_up:

...sync-simulator/src/velocity/value-mapper/mapper.ts
95.65% <88.88%> (-4.35%)
:arrow_down:

... and 41 more

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 f0ddd2f...144e9b8. Read the comment docs.


License?

Awesome work!
Just wondering what's the license for this project, before going ahead forking etc.

Hi @RonnyO,
This project is licensed under the MIT License.

Cool! Thank you

Only the code should be considered as the governance of MIT License. However, the artwork SHOULD NOT be regarded. Explicit disclaimer about artwork and trademark information are necessary.

Hi @jserv,
Thanks for your kind reminder. License in README updated


Not showing correct temperature

In toolkit 5.2 and 5.0 my temperature it's always 25°c. It never changes even if ambient temperature it's 15°c. This happens to both joycons

I have two pairs of joycons
The firmware versions are all 4.07. One pair of them has a constant temperature of 25 ℃, and the other pair has a real ambient temperature

I have two pairs of joycons
The firmware versions are all 4.07. One pair of them has a constant temperature of 25 ℃, and the other pair has a real ambient temperature


GrammarLocatorDef Not Found

Hi, I'm getting same error. above code doesn't help either. any ideas?

Hello @A5H1Q ,
you need to check if:

your project build script includes the kapt:plugins {
    id 'com.android.application'
    id 'org.jetbrains.kotlin.android'
    id 'org.jetbrains.kotlin.kapt' // THIS ONE
}

your have kapt 'io.noties:prism4j-bundler:2.0.0' in your dependencies block
you put @PrismBundle(includeAll = true) annotation of one of your classes (without which annotation processing won't be triggered), includeAll = true would include all know grammars, you can configure this as needed by specifying other languages in include parameter:// include all know languages
@PrismBundle(includeAll = true)

// include just languages specified
//@PrismBundle(include = ["kotlin", "java"])

// include all languages and generate `MySuperGrammerLocator`
//  in the same package as annotated class, instead of `GrammarLocatorDef`
//@PrismBundle(includeAll = true, grammarLocatorClassName = ".MySuperGrammerLocator")
class MyApp: Application() {

}

If you use java the same applies except the first point - you just need to specify annotationProcessor 'io.noties:prism4j-bundler:2.0.0' in your dependencies block


Iteration

So this isn't so much of an issue but I was trying to figure out how to iterate over a parsed JSON for like ten minutes, and I was wondering if it was possible.

never mind, I though of how to do literally as I pressed submit issue.

if anyone's wondering I usedfor i in 0..parsed.len(){ println!("{i}"); }

@RrayAgent saved my life - the docs don't mention that at all.


ModuleNotFoundError: No module named '_pywrap_s3_io'

Hi,  I run into en error of "ModuleNotFoundError: No module named '_pywrap_s3_io'", is it part of the module? or where can I find it?

How did you install the package? Can you provide some more info?

Hi, I'm having the same problem while attempting to package this for the nix package manager.
Would you mind running ls on the build path so that I could compare my build output? (shown below)
$ ls -R build
build:
bdist.linux-x86_64  lib.linux-x86_64-3.9

build/bdist.linux-x86_64:

build/lib.linux-x86_64-3.9:
aws_io.cpython-39-x86_64-linux-gnu.so  awsio

build/lib.linux-x86_64-3.9/awsio:
__init__.py  _version.py  python

build/lib.linux-x86_64-3.9/awsio/python:
__init__.py  lib

build/lib.linux-x86_64-3.9/awsio/python/lib:
__init__.py  io

build/lib.linux-x86_64-3.9/awsio/python/lib/io:
__init__.py  s3

build/lib.linux-x86_64-3.9/awsio/python/lib/io/s3:
__init__.py  s3dataset.py

I am having the same issue

Hi all, I've updated the installation guide for the plugin: https://github.com/aws/amazon-s3-plugin-for-pytorch#installation
However, we're deprecating support for the S3 plugin, and I suggest to try the new torchdata S3 IO datapipes. The S3 IO is upstreamed into torchdata package. We're supporting the new torchdata package, and continuously improving user experience and performance of the S3 IO datapipes.


Updating version to 0.3.1811.1501

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/concepts/oracles.md
:white_check_mark:Succeeded

articles/install-guide/command-line.md
:white_check_mark:Succeeded

articles/install-guide/vs-code.md
:white_check_mark:Succeeded

articles/relnotes/0.3-migration.md
:white_check_mark:Succeeded

articles/relnotes/index.md
:white_check_mark:Succeeded

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/concepts/oracles.md
:white_check_mark:Succeeded

articles/install-guide/command-line.md
:white_check_mark:Succeeded

articles/install-guide/vs-code.md
:white_check_mark:Succeeded

articles/language/expressions.md
:white_check_mark:Succeeded

articles/language/file-structure.md
:white_check_mark:Succeeded

articles/relnotes/0.3-migration.md
:white_check_mark:Succeeded

articles/relnotes/index.md
:white_check_mark:Succeeded

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


Empire C2

Description
Empire C2 Servers ( hxxps://bc-security[.]gitbook[.]io/empire-wiki/ )
Censys Search Query
Empire C2.
same_service(services.http.response.body_hash: "sha1:bc517bf173440dad15b99a051389fadc366d5df2" AND services.http.response.headers.expires: 0 and services.http.response.headers.cache_control: "*")

Thank you for your submission! I will add it shortly!

Added in 5a24af8


gitlab support

Sources of information

http://blog.wjlr.org.uk/2016/08/16/fast-rust-gitlab-ci.html
https://www.reddit.com/r/rust/comments/8twmub/even_faster_rust_builds_in_gitlab_ci/

That first link seems to be dead, but here's a similar one: https://noyez.gitlab.io/post/2018-06-15-rust-plus-diesel-plus-gitlab/


[Tabs] Tab switch will cause render on styled element

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
When a Tab in Tabs switches, an irrelevant styled('div') and all its subcomponents are re-rendered.
Expected Behavior 🤔
When a Tab in Tabs switches, only re-render the tab component.
Steps to Reproduce 🕹

https://codesandbox.io/s/mui-tabs-cause-render-dec0d?file=/src/App.tsx
Steps:

Open the link
Switch tab and see console log.
Change the StyledDiv to a plain div tag and style it with inline style={{/*...*/}} can remove this problem. Wrapping it indo another div won't resolve this..

Context 🔦

I am making my webpage snippethub.dev (see my repo).
I used a fetch method with an API to generate user icon.
When I switch tab, the fetch method run again.
I searched on google, stack overflow and issues in this repo (spent 3h)
Ended up found the error by removing&refactoring all the codes. (spent 2h)
The bug seems new.
I report it here. (I reported another misbehavior and the experience was top-tier)

Your Environment 🌎

`npx @mui/envinfo`
    System:
    OS: Windows 10 10.0.19042
  Binaries:
    Node: 14.17.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.15 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 6.14.13 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 92.0.4515.159 (I used this one)
    Edge: Spartan (44.19041.1266.0), Chromium (94.0.992.50)
  npmPackages:
    @emotion/react: ^11.4.1 => 11.4.1
    @emotion/styled: ^11.3.0 => 11.3.0
    @mui/core:  5.0.0-alpha.51
    @mui/icons-material: ^5.0.3 => 5.0.3
    @mui/material: ^5.0.4 => 5.0.4
    @mui/private-theming:  5.0.1
    @mui/styled-engine:  5.0.1
    @mui/styled-engine-sc: ^5.0.3 => 5.0.3
    @mui/system:  5.0.4
    @mui/types:  7.0.0
    @mui/utils:  5.0.1
    @types/react: ^17.0.27 => 17.0.27
    react: ^17.0.2 => 17.0.2
    react-dom: ^17.0.2 => 17.0.2
    styled-components: ^5.3.1 => 5.3.1
    typescript: ^4.4.3 => 4.4.3

Thanks, mnajdova. It is so obvious but maybe I was confused that moment. This issue shall be closed.


Generic plugin API (for output processors and such)

Add a more generic plugin API that would allow adding new output processors.
(There already is an API for auth plugins).
Candidates / use cases

[x] msgpack support #284
[x] http over unix domain sockets #209
[ ] Akamai debugging https://github.com/jakubroztocil/httpie/compare/master...matteck:master
[ ] WebSocket #371
[ ] triggering action upon receiving specific header #369

There is now a way to define auth, formatter, convertor and transport plugin. See "documentation" in the code.
The API is not considered stable, thus not publicly documented. I am closing this issue as it's "good enough" for now, but feedback on the API is welcome here!


About MovieGenreViewModel.SearchString

It seems I've already reported a similar issue (#16526) some time ago :) #17957 is new and I was guided to send a feedback from the page itself rather than submitting a PR.
The problem is that you used the SearchString property of movieGenreVM inside the view (even though you haven't specified the Model), and yet this property is not set in the controller.

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

ID: f9b9859c-b83c-5fc3-41d5-15531232185f
Version Independent ID: 5438de31-8562-3747-d1a5-2a4622252fc1
Content: Add search to an ASP.NET Core MVC app
Content Source: aspnetcore/tutorials/first-mvc-app/search.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Duplicate of #16526


Fix for WFLY-14582, Upgrade galleon-plugins to 5.1.1 [HOLD]

Issue: https://issues.redhat.com/browse/WFLY-14582
diff: https://github.com/wildfly/galleon-plugins/compare/5.1.0.Final...5.1.1.Final
[HOLD] do not merge

@jfdenise BTW you can use 'convert PR to draft' if you want to avoid accidental merge instead of commenting 'hold'.

@rhusar , thank-you!


Please provide Dockerfile with working installation

There are a lot of issues with problems with installation, and I'm facing also a lot of problems installing this library.
I think it would be good to have a Dockerfile with installation and a minimal test to show at least some exact CUDA, torch, python versions in which it can be installed.
Thank you.

Good idea. We recommend the
Pytorch
container from Nvidia, which has all the required tools to install FlashAttention.
I've added this to the README.


1.5.11 Play Store submission prep

[ ] README.md
[ ] (maybe) update privacy policy
[ ] Play Store copy
[x] en
[x] de
[ ] screenshots
[x] en
[x] da
[x] de
[x] fr
[x] gl
[x] nl
[x] pl
[x] tr
[x] zh-rCN
[x] zh-rTW
[ ] localizations
[x] de
[x] nl (#862)
[x] fr (#863)
[x] gl (#864)
[x] da (#865)
[x] tr
[x] pl (#867)
[x] zh-rCN (#869, #874)
[x] zh-rTW (#873)

For the translations, pinging
@javierpico (gl) @demokritos, @YunsuChoi (ko) @MadsAndreasen (da) @ptkdev (it) @shadoll (uk) @skolima (pl) @Ferri64 (ca) @niknetniko (nl) @Loffe (sv) @gLes (hu) @saratovout (ru) @marcher233, @CzBiX, @liweitianux (zh) @pejakm (sr) @werdeil (fr)
if you have time, it would be great if you could adapt the translation. if not, don't worry!

@marcher233 @liweitianux @MadsAndreasen @javierpico @werdeil @skolima @niknetniko
thanks for your help, it's getting closer to release time!
I've changed the Google Play store listing copy and added it to the project (in metadata/play/text). I used Google translate for the translations. If needed you can submit PRs. There is also a directory with screenshots of your translations, please check if everything looks ok there.

1.5.11 is now available in Google Play, update in F-Droid pending (!3104). Thanks everyone for your help! 🙂


Validate dot_axes argument in cos mode and fix output shape

Back when I wrote the cos mode it worked only with 2D tensors so it did not need the dot_axes argument. Now that cos mode uses this argument, we have to do sanity check in cos mode too.
Edit:
Fix output shape in cos mode too.

LGTM, thanks


Feature request - service

Fantastic!
I've used AVR Audio Guard to run a 'silent' audio stream continuously to work around this problem in the past, but over the last 3 or 4 years I've been unable to find anything that works in windows 10.  Until this!
Thank you so very much!
I'm using nssm to create a service out of the exe, but it would be great if it could be installed as a service natively.

Very interesting!
Thanks for sharing such an interesting use case! I would never have guessed that REAL could be useful in this way.
When I started this project, I never intended for people to have this application running in the background at all times. It was designed as something you would turn on and off as required.
With time I came to a realisation that this is not how people want to use this application. I'll consider implementing this functionality as a Windows service as you have suggested.


Sentinel retry logic

Currently, I have the following Redis setup on 3 different hosts.
Host1: sentinel on port 26379, master on port 6379
Host2: sentinel on port 26479, slave on port 6479
Host3: sentinel on port 26579, slave on port 6579

I am using socket.io-redis with ioredis adapter
Sample code with the above setup
const io = require('socket.io')(server);
const socketredis = require('socket.io-redis');
const ioredis = require('ioredis');

let redis = new ioredis({
  sentinels: [{ host: 'machine-1.test.com', port: 26379},{ host: 'machine-2.test.com', port: 26479}, { host: 'machine-3.test.com', port: 26579}],
  name: 'master01',
  showFriendlyErrorStack: true,
  retryStrategy: function (times) {
    Logger.info(`Redis: Connection retry #${times}`);
    if(times < 10) {
      return 20000;
    } else {
      Logger.error(`Redis reconnect max retry limit reached...aborting app`);
      process.exit(1);
    }
  },
  sentinelRetryStrategy: function (times) {
    Logger.info(`Redis Sentinel: Connection retry #${times}`);
    if(times < 3) {
      return 3000;
    } else {
      Logger.error(`Redis Sentinel reconnect max retry limit reached...aborting app`);
      process.exit(1);
   }
  }
});

io.adapter(socketredis(redis));

redis.on('connect', () => {
  console.log('Connected to Redis --->');
});

With the above sample code, I have no issue with the node app when the master redis instance (6379) is running where the node app is running - say I have node app running on host:1
When I shutdown the Redis instance on port 6379 on host:1 (where the node app is running), I don't see the retry logic kicking in and I see the node app shuts down with the error message Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED <IP_ADDRESS>:6379

Here is some error log with the above setup.
Exiting...listener count 1
events.js:182
      throw er; // Unhandled 'error' event
      ^

Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED <IP_ADDRESS>:6379
    at Object.exports._errnoException (util.js:1022:11)
    at exports._exceptionWithHostPort (util.js:1045:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1146:14)

Any thoughts on why the app would shutdown with the ECONNREFUSED <IP_ADDRESS>:6379 message and/or modifying the adapter code so that the retry logic would work?

I'm not sure but should this be io.adapter(socketredis({pubClient: redis, subClient: redis})) instead of io.adapter(socketredis(redis))?

Thanks for your quick response @luin
Your suggestion on using pubClient/subClient seems to be keeping the node app UP without shutting it down.

Another related question  - so keeping it in the same thread.
Continuing to test the retry logic with just one redis instance and one sentinel. The idea is to see the retryStrategy in action by setting the failover-timeout to 30000 and down-after-milliseconds to 3000 in the sentinel config
let redis = new IOredis({
  sentinels: [{ host: 'abc.xyz.com', port: 26379 }],
  name: 'master01',
  showFriendlyErrorStack: true,
  retryStrategy: function (times) {
    Logger.info(`Redis: Connection retry #${times}`);
    if (times < 20) {
      return 5000;
    } else {
      Logger.error(`Redis reconnect max retry limit reached...aborting app`);
      process.exit(1);
    }
  },
  sentinelRetryStrategy: function (times) {
    Logger.info(`Redis Sentinel: Connection retry #${times}`);
    if (times < 50) {
      return 5000;
    } else {
      Logger.error(`Redis Sentinel reconnect max retry limit reached...aborting app`);
      process.exit(1);
    }
  },
  reconnectOnError: function (err) {
    var targetError = 'READONLY';
    if (err.message.slice(0, targetError.length) === targetError) {
      // Only reconnect when the error starts with "READONLY"
      return true; // or `return 1;`
    }
    console.log(err);
  }
});

io.adapter(socketredis({pubClient: redis, subClient: redis}));

Interestingly, when I bring the only redis instance down, the node app seems to fail with the error message "All Sentinels are unreachable. Retrying from scratch after 5000ms" - even though the redis-sentinel is still UP.
Any thoughts on what I could be missing?

This issue is everywhere with this package..... Cant find a solution yet


[MCTB-468] Implementation of Users

Add a new terraform resource 'pingdom_user' to cover both Solarwinds user invitation and active users.
Change the previous meta object to cover new Solarwinds client.
Add utility functions for hashing of 'TypeSet' resources.

Need to hold off this PR until its dependent changes in the go-pingdom repository are merged.

@mLupine Currently the dependency declaration for 'go-pingdom' is replaced with the one on my personal repository. Otherwise the build will fail. Will change this once the dependent PR is merged.

Library PR is merged, please update the declaration

@mLupine Currently the dependency declaration for 'go-pingdom' is replaced with the one on my personal repository. Otherwise the build will fail. Will change this once the dependent PR is merged.

Library PR is merged, please update the declaration

Done, the dependency has been replaced with the one from nordcloud repository.


Improvements: Updating Spike and RVLS Submodules

Hi @Dolu1990 ,
Would you like to update the riscv-isa-sim and rvls submodules to the latest commit hash?
I tested this update on my side, and it works well. Could you please let me know the steps to follow to create a pull request for your review and validation?
Looking forward to your feedback, thank you!

Hi,
Sure ^^
Hmm so to get it through smoothly, the way to do is :

Create the PR in NaxRiscv with all the required changes

If a submodule need to merge a PR first, indicate it.

Wait that the github action success
Then on my side i will run some additional regression with randomized NaxRiscv configurations
Then it should be good for merge

The one thing which is important, is that that NaxRiscv PR has all the info, so that in some way, the maintener (me) can be next to amnesic and still be able to get the context. (Mainteners have to do a lot of switching context all the day, it break brains  cells XDXD)

Also note, you have to poke mainteners when things get lost.

Hi @Dolu1990,
Thank you for the detailed steps.
To proceed smoothly :

We first need to validate the pull requests for the submodules:

SPIKE https://github.com/SpinalHDL/riscv-isa-sim/pull/2#issuecomment-2506607012
RVLS https://github.com/SpinalHDL/rvls/pull/1#issue-2588700654

After the submodule updates are validated, we can move on to the NaxRiscv pull request.
For this step, NaxRiscv will need to point to the updated submodules hashe. Could you clarify if this update should be handled on my side or yours? how can i change it ?

NAXRISCV https://github.com/SpinalHDL/NaxRiscv/pull/125#issue-2588582034

Commit updates for FPU testing in RVLS and NaxRiscv:
After validation, I will make the necessary changes to properly support FPU testing using socSim.

Complete the addr_to_mem function in RVLS:
This function is essential for enabling the lrsc test and Linux boot functionality with socSim and the updated riscv-isa-sim. Here is the current implementation I tried:
// should return NULL for MMIO addresses
char* SpikeIf::addr_to_mem(reg_t addr)  {
    auto r = getRegion(addr);
    if (r == NULL || r->type != RegionType::mem) {
        return NULL;
    }
    return (char*)memory->memory.get(addr);
}

Unfortunately, it doesn't work. If you have any suggestions for improvement, I’d appreciate it.
For reference:

Definition in RVLS: https://github.com/Bill94l/rvls/blob/b5c15e129f8168c2317b2c129aef91adf935bfeb/src/hart.cpp#L41C1-L46C2

Usage in riscv-isa-sim: https://github.com/Bill94l/spinal-riscv-isa-sim/blob/cbc35ae812cdca97380417575df0cb16716ec96d/riscv/mmu.h#L299C3-L311C4

Definition in riscv-isa-sim: https://github.com/Bill94l/spinal-riscv-isa-sim/blob/cbc35ae812cdca97380417575df0cb16716ec96d/riscv/simif.h#L17C1-L19C70

Looking forward to your feedback and validation to move forward! 😊
Thanks :D

Hi ^^

We first need to validate the pull requests for the submodules:

What's about having NaxRiscv branchs in rvls / riscv-isa-sim ? This way it won't interfer too much with other projects ?

After the submodule updates are validated, we can move on to the NaxRiscv pull request.
For this step, NaxRiscv will need to point to the updated submodules hashe. Could you clarify if this update should be handled on my side or yours? how can i change it ?

I use smartgit to handle git submodule, not sure exactly how to handle that in command line.
I can do it if you want to ^^

Complete the addr_to_mem function in RVLS:

Ahhh there the idea is to specify more precisely which region of the memory i capable of what, right ?
I think addr_to_mem should always return null, to force spike checking every address.
Instead, checks should be added in mmio_xxx. (?)

Hi

What's about having NaxRiscv branchs in rvls / riscv-isa-sim ? This way it won't interfer too much with other projects ?

That's a great idea! However, in that case, I would need to create a new pull request for the NaxRiscv branch that you plan to create. Could you confirm if this is what you'd like me to do?

I use smartgit to handle git submodule, not sure exactly how to handle that in command line.

If you want to create a NaxRiscv branch in the riscv-isa-sim and rvls submodules. For each submodule after cloning the repository, we need to use:
git checkout NaxRiscv

I think addr_to_mem should always return null, to force spike checking every address.

If addr_to_mem always returns NULL, here’s the relevant implementation for RVLS in ext/rvls/src/hart.cpp:
ext/rvls/src/hart.cpp
// should return NULL for MMIO addresses
char* SpikeIf::addr_to_mem(reg_t addr)  {
    //if((addr & 0xE0000000) == 0x00000000) return NULL;
    //printf("addr_to_mem %lx ", addr);
    //return (char*) memory->memory.get(addr);
    return NULL;
}

To make the lrsc test and Linux boot work correctly with RVLS, I needed to modify the riscv-isa-sim source code as follows:
--- 'a/riscv/mmu.h'
+++ b/riscv/mmu.h
@@ -304,7 +304,7 @@ public:
     }

     reg_t paddr = translate(generate_access_info(vaddr, STORE, {}), 1);
-    if (sim->reservable(paddr))
+    if (!sim->reservable(paddr))
       return load_reservation_address == paddr;
     else
       throw trap_store_access_fault((proc) ? proc->state.v : false, vaddr, 0, 0);

However, when I apply this modification to riscv-isa-sim, the lrsc test and Linux boot fail to work properly with the regression test simulator, as its addr_to_mem does not always return NULL. Here’s the regression test simulator's addr_to_mem implementation in src/test/cpp/naxriscv/src/main.cpp:
src/test/cpp/naxriscv/src/main.cpp
// should return NULL for MMIO addresses
    virtual char* addr_to_mem(reg_t addr)  {
        if((addr & 0xE0000000) == 0x00000000) return NULL;
        return (char*)memory.get(addr);
    }

I’d like to find a solution that works both for RVLS and the regression test simulator.
Let me know if you have any suggestions or if further clarification is needed! 😊

I created the rvls / riscv-isa-sim branches :

https://github.com/SpinalHDL/riscv-isa-sim/tree/naxriscv
https://github.com/SpinalHDL/rvls/tree/naxriscv

They should be on the proper git hash for you to open a PR :D

Hi @Dolu1990
Thank you for setting up the branches!
- For `riscv-isa-sim`, I would prefer the branch to be created from commit `5f4eca2c0a5ee7808ad235f5230f3bd2c14a6ffa` (the second-to-last commit).
For `RVLS`, the branch should be created from the latest commit `b5c15e129f8168c2317b2c129aef91adf935bfeb`.

Thanks :D

The --enable-commitlog flag is no longer required when building the updated riscv-isa-sim :
cd $NAXRISCV/ext/riscv-isa-sim
mkdir build
cd build
(---) ../configure --prefix=$RISCV --enable-commitlog  --without-boost --without-boost-asio --without-boost-regex
(+++) ../configure --prefix=$RISCV --without-boost --without-boost-asio --without-boost-regex
make -j$(nproc)

Proposal:
Would it make sense to create a new branch named naxriscv in this repository?
The idea is for this branch to point to the naxriscv branches of both submodules, riscv-isa-sim and rvls.
What do you think?

Hi @Dolu1990
Thank you for setting up the branches!

For riscv-isa-sim, I would prefer the branch to be created from commit 5f4eca2c0a5ee7808ad235f5230f3bd2c14a6ffa (the second-to-last commit)
For RVLS, the branch should be created from the latest commit b5c15e129f8168c2317b2c129aef91adf935bfeb

Thanks :D

Ahhh damned, i have seen the PR before this message, the PR already got merged :/
Is it ok ?

Proposal:
Would it make sense to create a new branch named naxriscv in this repository?
The idea is for this branch to point to the naxriscv branches of both submodules, riscv-isa-sim and rvls.

Hmmm, let's name it "rvls-update" instead ?


Satisfy tslint

I'm submitting a ...
bug report

What is the current behavior?
Tons of warnings after completion of tslint script.

What is the expected behavior?
No warnings after completion of tslint script.

@lexzhukov @nnixaa Any plans on this? Are you interested in an appropriate PRs on that?

@Deilan , @lexzhukov , @nnixaa -  I'd be happy do go through the current code base and fix all the issues. I would then create a PR. Let me know if that would be ok before I start on it or if you would like to have some other things in place before it is done.

Let's probably close it as we switched to ngx-admin.


Fix install on Windows

Fix the following error when trying to install on Windows:
the package 'true' is not recognized as an internal or external command

Fix install on windows:

Pull Request Test Coverage Report for Build 59

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

Totals

Change from base Build 55:
0.0%

Covered Lines:
1

Relevant Lines:
1

💛  - Coveralls


platformsh: 4.10.0 -> 4.11.4

Description of changes
Package update.
Just tested a build on x64.
I see there's auto-update information there but I still didn't learn how it works. Not sure if this should've been auto-updated or something. I was working with the package so why not PR this.

You can run the script manually with:
nix-shell maintainers/scripts/update.nix --argstr package platformsh

The bot does run them after a while.

Result of nixpkgs-review pr 273601 run on aarch64-darwin 1

1 package built:

platformsh


Pal 390 resource service springboot cacheable

Change to the ResourceService interface to allow caching to work

closing while design issues are resolved that have come up from this work


Emformer forward and infer output difference

🐛 Describe the bug
Hello (@hwangjeff ), I noticed discrepancy between Emformer forward and infer outputs when using non-zero right context:

encoder_right_context_size = 1
encoder_left_context_size = 4
chunk_size = 4

emformer = Emformer(
    input_dim = 8,
    num_heads = 1,
    ffn_dim = 8,
    num_layers = 1,
    left_context_length = encoder_left_context_size,
    right_context_length = encoder_right_context_size,
    segment_length = chunk_size
)
emformer = emformer.eval()

x = torch.randn(1, 100, 8)
x_lengths = torch.tensor([100]).long() # something realyyyyyy long so that we do not have to bother with paddings or lengths

offline_output, _ = emformer(x, x_lengths)

online_states = None
for i in range(0, x.shape[1], chunk_size):
    s = i
    e = i + chunk_size
    x_chunk = x[:, s:e]

    online_output, _, online_states = emformer.infer(x_chunk, torch.ones(x_chunk.shape[0], device=x.device) * x_chunk.shape[1], online_states)
    break # we do not need to continue, but we could :)

print(offline_output[:, :chunk_size - encoder_right_context_size] - online_output)

Ouputs something like:
tensor([[
        [ 0.0156, -0.1142,  0.0152, -0.1302, -0.0019,  0.0077,  0.1108, 0.0971],
        [ 0.0376,  0.0665, -0.0177, -0.2270, -0.0756, -0.1156,  0.1082, 0.2236],
        [-0.0721,  0.0115,  0.0060, -0.2362,  0.1189, -0.0636,  0.0615,  0.1741]
]], grad_fn=<SubBackward0>)

I would expect having offline_output and online_output exactly the same values which is true only if encoder_right_context_size = 0. What am I missing or doing wrong? The difference is substantial on real data.
Versions
The current version in main

@Tomiinek thanks for flagging! There are a few things to point out:

segment_length is exclusive of right_context_length.
Accordingly, the input to infer should contain the right context and be of length segment_length + right_context_length.
Also, for forward, the length input should exclude the length of the right context, whereas for infer it should include it. This is admittedly a little confusing, and I will update the documentation to clarify this.

Accounting for these in edits to the code you provided, we have
import torch
from torchaudio.models import Emformer

encoder_right_context_size = 1
encoder_left_context_size = 4
chunk_size = 4

emformer = Emformer(
    input_dim=8,
    num_heads=1,
    ffn_dim=8,
    num_layers=1,
    left_context_length=encoder_left_context_size,
    right_context_length=encoder_right_context_size,
    segment_length=chunk_size,
)
emformer = emformer.eval()

x = torch.randn(1, 100, 8)
x_lengths = torch.tensor([100 - encoder_right_context_size]).long()

offline_output, _ = emformer(x, x_lengths)

online_states = None
for i in range(0, x.shape[1], chunk_size):
    s = i
    e = i + chunk_size + encoder_right_context_size
    x_chunk_rc = x[:, s:e]

    online_output, _, online_states = emformer.infer(
        x_chunk_rc, torch.ones(x_chunk_rc.shape[0], device=x.device) * x_chunk_rc.shape[1], online_states
    )
    break

print(offline_output[:, : chunk_size] - online_output)

, which yields a much more reasonable output:
tensor([[[ 1.1921e-07,  2.9802e-08, -2.9802e-08,  5.9605e-08,  1.1921e-07,
           2.9802e-08,  1.1921e-07,  1.1921e-07],
         [ 1.1921e-07, -5.9605e-08,  0.0000e+00,  0.0000e+00, -1.1921e-07,
           0.0000e+00,  0.0000e+00,  0.0000e+00],
         [ 0.0000e+00,  0.0000e+00,  0.0000e+00,  6.7055e-08,  3.5763e-07,
          -1.1921e-07,  0.0000e+00,  5.9605e-08],
         [ 1.1921e-07,  5.9605e-08,  4.4703e-08, -1.1921e-07,  5.9605e-08,
           5.9605e-08,  1.1921e-07,  1.0431e-07]]], grad_fn=<SubBackward0>)

Let us know if you any other questions or thoughts.

Thanks for the prompt reply! Makes sense, my fault :slightly_smiling_face:

No problem!

If you don't mind I will just describe my reasoning, maybe it help somebody who is in a late-night debugging rage as I was, or could help to clarify the documentation :slightly_smiling_face:
In Emformer constructor, there are:
right_context_length (int, optional) – length of right context. (Default: 0)
segment_length (int, optional) – length of each input segment. (Default: 128)

In the infer method, there are:
input ([torch.Tensor]) – utterance frames right-padded with right context frames, with shape (B, T, D).

And returns:
output frames, with shape (B, T - ``right_context_length`, D)`.

From this, I deduced that segment_length corresponds to T because of the mention "length of each input segment".
However, it should correspond to T - right_context_length exactly.
Maybe some asserts in infer could be helpful and made the code bullet (or my stupidity :smile:) proof? If I am not wrong, currently, it seems that it is possible to push through it arbitrarily-sized chunks.

@Tomiinek this is very helpful — thanks! Sorry to hear about the debugging issues; I see where the confusion lies and agree that adjusting the documentation and adding some asserts would make this more usable. Will make the changes.

Addressed in #2207


I added two questions to the Python file.

Summary of changes
I added two questions the Python file
Checklist

[x] I have read and followed the contribution guidelines.
[x] I have read through the Code of Conduct and agree to abide by the rules.
[x] This PR is for one of the available issues and is not a PR for an issue already assigned to someone else.
[x] My PR title has a short descriptive name so the maintainers can get an idea of what the PR is about.
[x] I have provided a summary of my changes.

closes #XXXXX

I am also going to close this PR because it has duplicate questions about css and agile that you have another PR
Please open up another PR that only has your Python questions and nothing else.
My suggestion would be to create a new branch off of main that just has your Python questions.
Thank you.


Resetting/wiping the map

Hi,
we want to add the possibility to reset all data(markers/routes) on the map, without having to close the window, to our program. Yet, i could not find a method to do so, and when we tried to do it by reassigning the variables, it didn't work either.
So, did we miss something, or do you have any idea, how we could do this?
Any help is very much appreciated

You can clear all markers in the latest version, but routes are different. Setting the map to null for the directions renderer is supposed to clear them when using pure JS, and there is the option to do so with the DirectionsRenderer in this library. Whether it has the desired effect is another matter, I haven't looked into the directions side of things. You could store your DirectionsRenderer(s?) (assuming you're using one somehow) and call setMap(null).
The other option would be creating a new GoogleMapView and initialising it, then add it in place of the existing one in your UI. This essentially recreates the WebView and reloads Google Maps within it.

Thanks for the quick answer. For some reason we still have Problems with your second Proposition, so i tried to do it with your first one.
First, calling setMap(null) on a renderer threw a NullPointerexception, since setMap() uses the variable name of the Parameter. Then i tried to set it to a new GoogleMap, which was nowhere displayed, which worked fine(new routes wouldn't appear anymore), but the old routes did remain on my map.
We will try to get your second guess running, but if anyone could make it possible, that the setMap(null) way works, that would be magnificent.

in the 2.11.0 release, clearing directions should work by calling either setMap(null) or the clearDirections() method on the DirectionsRenderer object
https://github.com/rterp/GMapsFX/wiki/Create-and-Clear-Directions-and--Route


Optimize memory usage with go-shadowsocks2 in bloomring for defense replay attack.

Hi, go-shadowsocks2 is such a brilliant project that really inspired me.
15Mb memory limit on iOS platform is a big challenge, and I think the dependency go-shadowsocks2 bloomring module alloc about 1Mb memory to defense replay attack in client is a little bit luxurious.
Could you please change the bloomring feature to optional?

I mean add build flags for this feature, so project using go-shadowsocks2 as dependency needn't to modify the source code. :)

@riobard

The Bloom filter is optional, but you have to opt out by setting an environment variable. See README.
I'm curious how do you use go-ss2 on iOS?

I'm curious how do you use go-ss2 on iOS?

FYI.
https://github.com/Jigsaw-Code/outline-go-tun2socks
https://github.com/eycorsican/go-tun2socks/tree/v1.16.0/proxy/shadowsocks

I see. Since you can opt out of Bloom filters, I assume this issue is now resolved. Feel free to reopen if there's anything else you have in mind.


support arm64-v8a - INSTALL_FAILED_NO_MATCHING_ABIS

Hi, relative newbie WRT android development so apologies in advance if I'm missing something obvious.
I've successfully built and run sokol-samples/triangle-glfw for linux-make-debug
I can also build sokol-samples/triangle-sapp for sapp-android-make-debug, but adb install fails:
adb: failed to install ./sapp-android-make-debug/sapp/triangle-sapp.apk: Failure [INSTALL_FAILED_NO_MATCHING_ABIS: INSTALL_FAILED_NO_MATCHING_ABIS: Failed to extract native libraries, res=-113]
I believe this is because only 32bit armeabi-v7a objects are generated, and my phone (pixel8) is arm64-v8a:
$ file lib/armeabi-v7a/libtriangle-sapp.so
lib/armeabi-v7a/libtriangle-sapp.so: ELF 32-bit LSB shared object, ARM, EABI5 version 1 (SYSV), dynamically linked, BuildID[sha1]=7305bf118b322127895e37407f73084504f97987, with debug_info, not stripped

I did try setting ANDROID_ABI: arm64-v8a in configs/android-make-debug.yml and this did not work - GLuint was undefined in sokol_gfx.h and I haven't yet looked further into why.

Hmm, I haven't run into issue yet (I have a Google Pixel 4a with a somewhat recent Android).
One thing you could try is also changing to a more recent platform than android-28 (e.g. the default config looks like this currently:
https://github.com/floooh/fips/blob/55f8f170f569aa7ed4cb517288672c21d290567f/configs/android-make-release.yml#L7-L9
If sokol_gfx.h fails with an undefined GL type, it looks like it's skipping the include here (maybe SOKOL_GLES3 is not defined?):
https://github.com/floooh/sokol/blob/7f7cd64c6d9d1d4ed08d88a3879b1d69841bf0a4/sokol_gfx.h#L4351-L4355

I was able to try those steps today and it worked - thank you for all the pointers! My fault for not reading far enough into how fips works. I have enough to iterate on now. Thanks again!

Good to know, I wonder if it's time to generally switch to 64-bit ARM in the default fips build configs and fade out 32-bit support.


Signing signature and digest algorithm configuration

Enable XML signature signing and digest algorithm configuration.
A per-SAML2 IdP frontend default may be configured and per-SP
overrides may be configured. With no configuration the default
for signature algorithm is
http://www.w3.org/2001/04/xmldsig-more#rsa-sha256
and the default digist is
http://www.w3.org/2001/04/xmlenc#sha256

I think this looks ok - @johanlundberg @Gijutsu @jkakavas - anyone wanna take a look before we merge?

I will take a look by EOB today.


Added resources section and additional tools

Description of changes:
Added the resources page, the route in the navigation and 3 additional tools.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Probably unrelated to this PR, but what is this vercel bot doing here 😮 ?

I did a test deployment via Vercel. I think it's from my Forked repo.

I did a test deployment via Vercel. I think it's from my Forked repo.

Ahh, right :)
Not sure if you noticed, but Netlify provides preview deployments as well

Alright, package-lock.json is gone. 👍 And Tailwind UI looks great, that will look a lot better than the tabs component that I found. The second tabs look great: Tabs with underline and icons


Added

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
All Promptflow Contribution checklist:

[x] The pull request does not introduce [breaking changes].
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request to get dedicated review from promptflow team. Learn more: suggested workflow.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

upadate issue

https://app.bluevine.com/contracts/accountverificationeua

review my code

IS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
DISCLAIMED.


Missing steps for xfs

Customer had xfs file systems but the document only provides steps for ext4. Customer had to run #xfs_growfs /... instead of #resize2fs /dev/...
Please include steps for xfs like in the below page (step-7)
https://docs.microsoft.com/en-us/azure/virtual-machines/linux/resize-os-disk-gpt-partition
Thanks!
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 06c61c5a-d9e4-742b-36e5-b9effa33d903
Version Independent ID: cb9dbd59-460d-a1a0-9863-5213d6a1093e
Content: Expand virtual hard disks on a Linux VM - Azure Virtual Machines
Content Source: articles/virtual-machines/linux/expand-disks.md
Service: virtual-machines-linux
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

#reassign: @ericd-mst-github

#reassign: roygara

#please-close This article has been updated multiple times to cover more distros and scenarios. I think this issue is now "fixed", but please feel free to reopen if it hasn't. Thank you!


Pull request for Jest issue - SecurityError: localStorage is not available for opaque origins #119

Pull request for Jest error - SecurityError: localStorage is not available for opaque origins #119
Jsdom just recently upgraded to version 11.12, which includes support for window.localStorage and other features. See https://github.com/jsdom/jsdom/blob/master/Changelog.md.
Jest does not raise above error when Node is specified as the test environment.
Pull request raised to add above change to documentation if appropriate.
Please excuse any inconvenience I may cause. This is my first pull request on a public github repository.
Thanks.

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

Thanks @NileshParkhe83 this looks good!


In tests, setMockMethodCallHandler interferes with text state

Calls to SystemChannels.textInput.setMockMethodCallHandler cause the test to be in a weird state where subsequent calls to enterText, even in other tests in the same file, will fail with this error:
Tried to use TestTextInput with no keyboard attached. You must use WidgetTester.showKeyboard() first.

Here is a simple test file that reproduces the failure:
import 'package:flutter/cupertino.dart';
import 'package:flutter/services.dart';
import 'package:flutter_test/flutter_test.dart';

void main() {
  testWidgets('text field respects keyboardAppearance from theme', (WidgetTester tester) async {
    final List<MethodCall> log = <MethodCall>[];
    SystemChannels.textInput.setMockMethodCallHandler((MethodCall methodCall) async {
      log.add(methodCall);
    });

    await tester.pumpWidget(
      const CupertinoApp(
        theme: CupertinoThemeData(
          brightness: Brightness.dark,
        ),
        home: Center(
          child: CupertinoTextField(),
        ),
      ),
    );

    await tester.enterText(find.byType(CupertinoTextField), 'asdf');
  });
}

It seems like there is some cleanup that needs to happen.  The docs state that passing null to the same call will unassign the callback, but that doesn't fix the problem.
I found this bug while working on https://github.com/flutter/flutter/pull/41326.  Adding a test that used enterText passed or failed depending on where in the file I put it.  This is a dangerous bug that could cause tricky phantom test failures in the future.

The referenced error no longer occours and the test passes when tester.testTextInput.register(); is added.
Can we close this @justinmc?

Sounds good, thanks for checking.  I'm glad this is fixed somehow.


Compliance test AddIPv4ToMultipleNHsMultipleRequests() does not flush the server when done

Most tests start with a defer flushServer() but this function does not.  That means that entries created by this test may still exist on the server for the next test.  I suspect this is an oversight and should be fixed.

This issue is resolved now.


No binding for evil-easymotion in orgmode using bepo layout

What did you expect to happen?
Access evil-easymotion bindings using either g s (original binding) or g é (suggested in the bepo layout documentation).
What actually happened?
g s is bound to org-backward-element, and g é is not bound.
Additional details:
In an org-mode file,  here are the bindings for evilem-motion-find-char-to:
Key Bindings
evil-collection-evilem-map-backup-map t
evil-motion-state-map g s j
evilem-map j

In some other modes, such as emacs-lisp-mode, evil-easymotion is accessible through g s, with the same bindings (for instance for evilem-motion-find-char-to):
Key Bindings
evil-collection-evilem-map-backup-map t
evil-motion-state-map g s j
evilem-map j

Steps to reproduce:

Open an org file
Type g s or g é

System information:
https://pastebin.com/LBg9mnqy

Looks like (after! evil-easymotion …) isn't the correct feature, I'll try to find this out

I think this can be closed, evil-easymotion is accessible both through g s and g é.

gé has probably been fixed by https://github.com/hlissner/doom-emacs/commit/9ac79c6e861ecfcf9b21f26678046f31b37dc536
and gs in org-mode probably still is the motion thing, but outside of orgmode it should be fine hopefully

I just tested it, and g s in org-mode is org-backward-element (g é is evilmotion prefix). There is a little bit of inconsistency here as g t is workspace switching, except in motion mode (where it is org-forward-element).

I'll need to look into what motion-state really is to fix this issue, because it feels like gt should actually take org-forward-element instead of "change tab" default vim binding. I'll check if it's something from :ui workspaces to adapt here.


fix: change boundActionCreators to actions

Changes boundActionCreators to actions per Gatsby v3 breaking changes: https://www.gatsbyjs.com/docs/reference/release-notes/migrating-from-v2-to-v3#removal-of-boundactioncreators

Oh boy. Thanks for this pull request.
Can anyone else confirm that there are no further problems with gatsby v3?
This is a breaking change that makes the code no longer compatible with gatsby v2 so I'll probably need to be careful with the next version's numbers.

Nobody has made a comment against this, so I'm merging it to master. Not publishing a new NPM version until I manually check compatibility issues, which may take a while, as I am busy running a kickstarter.
Thanks!

Nobody has made a comment against this, so I'm merging it to master. Not publishing a new NPM version until I manually check compatibility issues, which may take a while, as I am busy running a kickstarter.
Thanks!

I can confirm that this works in gatsby v3.

Thanks for fixing this @jeffwinegar, I am just running into this while upgrading to Gatsby v3 and you saved me quite some time :)
I can confirm that it works with v3.
For anyone who wants to pin the package unitil there as a new release on npm you can set it to this commit in your package.json:
{
  "dependencies": {
    "gatsby-source-github-api": "ldd/gatsby-source-github-api#fe8fa58d89be8c2f49ea277c58bbd369f8abbda0"
    ...
  }
  ...
}

I can also confirm that this works with Gatsby v3!

I've just published version 1.0.0 which includes gatsby v3 support.


[RFE] Export Image Gamma Correction Toggle

I started using MLOPS2 today. The first thing I noticed was the images were washed out. Tracking it down to the image utils, I connected a check box to the export_image node to allow gamma correction to be optional.
Here's my code change to mlops_image_utils.py
def colors_numpy_array_to_pil(input_colors, gamma_correct=0):
# Transpose into (width, height, channels)
input_colors = input_colors.transpose(1, 2, 0)
if gamma_correct!=0:
# Gamma Correct
input_colors = pow(input_colors, 1.0 / 2.2) #Pass float instead?
# Convert to RGB space
input_colors = (input_colors * 255).round().astype("uint8")
return Image.fromarray(input_colors)
Inside the HDA I altered the calling code.
# Export
gamma_correct = node.parent().parm("gamma_correct").evalAsInt()
image = mlops_image_utils.colors_numpy_array_to_pil(input_colors, gamma_correct)
It might be better to simply pass along a float instead of a toggle. Then you could pass 1.0 for no gamma correct and 2.2 for full gamma correct..?

Fixed in 3.0


Webauthn/Passkeys are a shattered dream. Remove text from header of README and move section further down the page.

Currently the README implies that Webauthn/Passkeys are the way forward.
Consensus in 2024 is that passkeys/webauthm is a shattered dream. Related Front Page discussion on HN.
"The biggest issue with passkeys is that I just can't trust the companies offering them"
-- Top comment on HN
Request that this front matter is removed and the section on this moved further down the readme, maybe including that issues around passkeys/webauthm are still being resolved and that there is still no mass adoption of this approach.
MFA TOTP remains best practice for password security in 2024 and this project is doing excellent work in this space.

First off, thanks for opening this issue and sorry for the delay in responding. I think it's a good issue to discuss and wanted to take some time to write up my thoughts on this.
In some ways I agree; Passkeys are a bit of a disaster UX wise, at least at the moment. Adoption of the full "Passkey" login has not been great, and it's definitely confusing for users. But while I can't predict Passkey adoption with consumers, for more security focused users (enterprise/corporate) it's already the standard (hardware keys via Webauthn).
Even as bad as the Passkey UX is, there's no way I could recommend TOTP in 2024 for most users.

OTP's are very phishable, so much so that most high profile orgs have banned their internal use - Google did this more than 6 years ago ^2. Cloudflare, Okta, Twitter - This trend is headed in only one direction.
It's a shared secret system. As a developer, this puts the onus on me to come up with a way to store it securely and retrieve the plaintext anytime a user logs in as well as prevent replays.
OTP's also seem to be heading towards UI enshittification. Nearly every enrollment will point a user to a mainstream OTP authenticator that then tries to tie this into the provider's backend. Google Authenticator has 100M downloads and a 3.8 rating, mostly due to cloud sync issues.
It's more work for the end user: install an app, scan a code, verify the code, get your phone out every time you need to login (or when the security team calls you and asks you to read off the OTP to them).

If you've got a user's email and security concerns are low, I think sending them a magic link is probably as safe for most users as TOTP, easier to use and easier for developers to implement (no need to encrypt and manage shared secrets and users don't need a separate app).
For users in more security critical roles, Webauthn/Passkeys is the solution while OTP is actually a liability ^1.
From personal experience I've moved every provider I can to Passkeys and haven't had much trouble. However it's clear from adoption rates that Passkeys are still not great for many users. But I think that's starting to change. For users with a password manager, like 1Password or Bitwarden (the beta on Testflight just added this on iOS and it works great), it's incredibly easy to use Passkeys now. It mostly just works, and works well.
While I think Passkeys still have many UX issues to solve, falling back to TOTP isn't an option. Put simply, if security is a big concern for your organization, YOU SHOULD NOT BE USING TOTP, or any OTP for that matter. For users that just want a simple 2FA solution, I think you'd be better off just using a simpler solution like magic links.


Tidy up Program.cs for better readability and maintainability

• Moved Azure Key Vault configuration to a separate method (ConfigureKeyVault).
• Moved service configurations to a separate method (ConfigureServices).
• Added comments to explain each section of the code.

Hi,
Thank you for pointing that out! I've reviewed and corrected the formatting issues by removing the multiple consecutive blank lines. The code should now conform to the single blank line preference.
The updated code is ready for your review. I appreciate your feedback and look forward to getting this merged and working more with you.
Thanks again!

Thank you so much for your contribution, all good and merged 👍


Import excel into database azure

Hi,
I stuck with problem importing 10mb excel table in to azure database with time out error.
I do this code, localy I manage to import , but on azure, no.
**<system.web>

**
Please help.

What is the error that you get? Can you check Azure web app and database logs too...

It is probably taking too long to import. Try to split it into multiple iterations (like import only, say 1000 rows in single service call)

just loop through rows and only send 10 000 to webservice processing them.
check bulk action demo in Serenity

Also, what size Azure DB are you using, and is the Web App and DB hosted in the same region?
I would be interested to learn about your solution as I also host on Azure and have similar requirements (for large Excel file uploads).

My database and app are located South Central US
Right now I move from basic to standard price , 5GB database and 20DTU
My monthly cost should be around 150Eur , Pay as you go...

I will try to do some upload now.

This is the error when upload 24000 rows ...
500 - The request timed out.
The web server failed to respond within the specified time.

The server don't respond in time out period, because is busy. You can try using an async call and periodic response from server to inform client that server is working.
See sample https://serenity.is/demo/BasicSamples/CancellableBulkAction
or advanced sample (at payment) https://serenity.is/demo/AdvancedSamples/LongRunningAction

I see example , and I use it for bulk operation on products, but how to use it with excel table on import dialog?

Closing due to inactivity.


launch in Visual Studio Code on Linux fails with index.html not found

How to launch the SignalR sample from Visual Studio Code on Linux? It doesn't work as it opens up browser pointing to http://localhost:5000/index.html not found.

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

ID: 0dd2991d-c22d-ad0f-80b6-bd1b503db8f3
Version Independent ID: 5e3fbe1d-c97f-9940-4f0c-067d4b2112b1
Content: Get started with ASP.NET Core SignalR
Content Source: aspnetcore/tutorials/signalr.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @bradygaster
Microsoft Alias: bradyg

thanks for the report @khteh - could you let us know which distro of linux you're using?

https://github.com/aspnet/AspNetCore/issues/10873

Switch to use IPAddress.Any works.


Where is test runner file?

Readme.md said:
Access test runner
To access the test runner and perform tests, open web-platform.test/_wave/.
However I couldn't find this in my container image. WebGL test runner was found, but is there whole test runner?

Could you try using port 8000? web-platform.test:8000/_wave/

Could you try using port 8000? web-platform.test:8000/_wave/

Thanks!! It worked.


Array Index Out of Bounds in scene overworldNW

Did you search for similar issues before submitting this one?
NO, I've tested all the other scenes and no any had the same issue.
Describe the issue you encountered:
When I run the scene overworldNW, I have an "Array index out of range" error. (More exactly 55 errors like this).
I've added some lines to the code in MapCollider.cs to test that issue and I've found that the issue happens always with z = 72.
Expected behavior:
When you run the scene overworldNW, you receive an "Array index out of bounds" error.

Platform (Win 10):

Windows 10 64 bit

Branch commit SHA:

I didn't create any branch.

Steps to reproduce:

Just run the scene overworldNW

Screenshot if needed (error files can be attached):

Screenshot of the error and some Logs made by me.

Screenshot of the lines added to MapCollider.cs to test the error.

@joseka1234 Can you compile #24 and see if it fixes any issues for you?

I have the error yet.

I'm not getting the error anymore on my local project with it. It might not be uploading to git correctly since the project is binary files? Is there a reason for using binary instead of text with git? Usually unity projects use text...

@mjohn314 The unity files are scenes, is there a way to make them text?

@superusercode yes, in the unity editor go to edit > project settings > editor. Then under asset serialization, change the mode to force text.

Fixed.
https://github.com/PokemonUnity/PokemonUnity/pull/33


SfHero component [WIP]

Related issue
#149
Scope of work
I have started working on Hero Component. For now idea is to use SfCarousel and SfBanner components, to create SfHero component.
Screenshots of visual changes
Checklist

[ ] I followed composition rules for my component
[ ] I tested the component in most common device sizes (can be tested in Storybook from viewport addon in top menu)

The reason I've chosen to use SfCarousel is to follow the principles of atom design, so that organisms are built from molecules and atoms. Correct me if I'm wrong, but here SfCarousel is similar to designed SfHero - basically it is a carousel with only one visible picture and the pagination.
I am not gonna argue that it is the best path, because I had a hard time to decide how to approach it myself.
Basically I wonder if:

maybe SfCarousel should be more flexible to used in SfHero, or as you suggested create a new organism/molecule SfHero "from scratch".
SfArrow should be extended with styling that allows the transparent background.

@michasik imo SfHero is not similar to SfCarousel, actually the only similarity between them is that they're swiping elements, but it's handled by glidejs, not by the carousel component itself.
Among the components, the arrows and the container is different, also the hero component has bullets for pagination and the carousel does not.
In addition, by default they should use glidejs in different ways, so they have much more differences than similarities I think.
About SfArrow, take a look at the sf-arrow--long CSS modifier, it makes the background transparent, I'm just not sure about the hover effect styles, as we don't have it from figma designs.


Extend timeout for location test

Change timeout for location test to reduce flakiness

Will it help, @jonathanmos , any idea?


Add Header Test

This PR adds test cases for \Slim\Psr7\Header.

Coverage increased (+0.5%) to 96.196% when pulling 4f3e252a705d798468ba40f7137787299cebd718 on adriansuter:patch-headerTest into 3f9b6ae466bae72a846e5ba7f85a2519497a83d0 on slimphp:master.


[Bug] Status of compatibility with Kubernetes v1.24

Page link
https://kyverno.io/docs/installation/#compatibility-matrix
Description
Hi devs,
thank you for the great project and amount or work you put in it.
I am not sure it is a bug or request from my side, but could you clarify the status of 1.24 Kubernetes?
Compatibility matrix shows it is not supported, but I found that this version is being tested e2e here:
https://github.com/kyverno/kyverno/blob/v1.7.2/.github/workflows/e2e.yaml#L25
So it makes me wonder that documentation for this is probably lagging behind.
Thank you in advance.
Expected behavior
Update compatibility matrix
Slack discussion
No response

Thank you for the answer @chipzoller . Is there any ETA for 1.8.0 yet?

No ETA as of yet.

Not an actionable docs issue for now. Will update support matrix with each release.


Support GEE feature/featurecolleciton/geometry when plotting with cartoee

Description
Describe the feature (e.g., new functions/tutorials) you would like to propose.
Tell us what can be achieved with this new feature and what's the expected outcome.
Source code
Paste your source code here if have sample code to share.

You can use features.style() or image.blend() to convert FeatureCollection to Image for plotting. See https://geemap.org/notebooks/57_cartoee_blend

Dear Dr. Wu,
It would be nice if the feature class was also supported by the add_layer
API, similar to get_map API.
On Thu, Mar 18, 2021 at 12:03 AM Qiusheng Wu @.***>
wrote:

Closed #368 https://github.com/giswqs/geemap/issues/368.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/giswqs/geemap/issues/368#event-4471160545, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AE5J3HTAMMCG7OCXALPAFMDTEDG4RANCNFSM4ZKQJWEQ
.

The cartoee add_layer function now supports Earth Engine vector data.


Add prometheus

This adds the prometheus demo from the dreamathon. I like to deploy that to our new cluster.
Let me know if we can merge it or not, given it's not officially supported.

@discordianfish we are moving to a model to add services into the multiverse first... especially for new or experimental services.   could you send a PR against https://github.com/mesosphere/multiverse/ on the master?

Yes, sounds good!


Create driver samples

hoping to find a wayto bridge the gap between windows and us common people and provide driver updates that cost nothing

Hi, i want to write a windows driver in WDM, please suggest me how to start preparation and how to write a code for that,
Thanx in advance

Maybe you opened this pull request by mistake


latest PR failed

Making note that:
NPM failure to rebuild oxrs.io in PR #51
https://github.com/OXRS-IO/OXRS-IO-WEBSITE/runs/7057593918?check_suite_focus=true

Spoken with MitchellR who is going to look into updating the build environment to get around the current NPM build issue.

Thanks for doing that @mrinc - I've been trying to keep up on the side bar and top bar navigation and to keep it functional in a logical way.
Glad it can be working again as we'll be having another large push of updates somewhat soon.

The sidebar does seem to be a bit of a pain to keep up with... We could maybe change the way its loaded to make it easier to manage.
So instead of managing the sidebar in the config.sidebar.js file (new file) - we move one to each of the main folders.
So like inside the docs folder, there is a sidebar file there
This would allow for a bit easier to manage context wise me thinks.
Nav bar doesn't change too much (well shouldn't have to) compared to the sidebar
What do you think?

In order to fix the build I had to do a bunch of stuff and during so noticed the nav bar some how stopped working.
But you will notice the search now works - which it didn't before (due to a breaking config change).... just no one noticed it lol
So thus why the split of the config files.
But yea, it's not live since PR - but make sure you npm i once you pull it down since package changes
Ideally, clear all temp files and then npm i, and run the project.


Flow 100% CPU, flow stop not working

I have encountered weird behaviour in flow. We use flow in two relatively similar repos, and flow works fine in one of them. However when I run flow in the other it spawns lots of flow processes, some of which can't be killed with flow stop
flow start:

Notice the 4 processes using over 50% of CPU.
flow stop:

OS: El Capitan
flow logs:
[2016-02-03 12:48:23] Initializing Server (This might take some time)
[2016-02-03 12:48:23] executable=/usr/local/Cellar/flow/0.21.0/bin/flow
[2016-02-03 12:48:23] version=0.21.0
[2016-02-03 12:48:23] Parsing

rm -rf repo
git clone repo
3 everything working normally

closing

This is related to #2828.


gcp-cups-connector does not work with OS X El Capitan 10.11

gcp-cups-connector
echo $?
0
Nothing happens. I followed the steps given in https://github.com/google/cups-connector/wiki/Build-from-source . In addition I had to # brew install bazaar. What else am I missing?

Yes, I fumbled a commit yesterday. It's fixed now.
I haven't updated the OSX instructions to include bazaar, and others. I thought I was the only one using OSX.

@8cH9azbsFifZ Have you tried this again?

Are there any more detailed or step by step instructions for installing and using the cloud-print-connector on an OS X El Capitan CUPS server? I'm not sure if I need to try to find and install the avahi-libs. Do I install the xcode command line tools and then pickup further instructions somewhere else?

You'll need to build from source, yes. You won't need avahi because OS X has Bonjour.
It's been a while since I built on OS X. Try this and let me know how far you get:
https://github.com/google/cloud-print-connector/wiki/Build-from-source
I don't know how to properly setup a service like this on OS X in a production environment.


Unable to persist state during docker build

I expected that I can modify the initial state of database in the Dockerfile, e.g. creating some indexes. It seems, however, that the state that I persist during the build steps is lost, maybe due to the entrypoint script?
Steps to reproduce
Following Dockerfile:
FROM neo4j:4.0.4-enterprise
ENV NEO4J_ACCEPT_LICENSE_AGREEMENT yes

COPY neo4j.conf /conf/
COPY schema.cypher /

RUN neo4j-admin set-initial-password test \
 && neo4j start \
 && sleep 10 \
 && /bin/bash -c "cat /schema.cypher | cypher-shell -u neo4j -p test --format verbose" \
 && neo4j stop

Following schema.cypher:
:begin
CREATE CONSTRAINT ON (node:SomeNode) ASSERT (w.uuid) IS NODE KEY;
:commit

CALL db.awaitIndexes(300);

The resulting database image, after a docker build -t <image> . however is empty, without any indexes. If I run the RUN steps manually in a Docker container, it works as expected.
Tested with both neo4j:4.0.4-enterprise and neo4j:3.5.14-enterprise.

What's the output of the docker build? Neo4j needs to be started as the neo4j:neo4j user, but your Dockerfile will be running it as root, which might be causing problems.
Could you also try adding USER neo4j:neo4j before your RUN?

Sure:
 ---> Running in 05bf08ff960e
Changed password for user 'neo4j'.
Active database: graph.db
Directories in use:
  home:         /var/lib/neo4j
  config:       /var/lib/neo4j/conf
  logs:         /var/lib/neo4j/logs
  plugins:      /var/lib/neo4j/plugins
  import:       /var/lib/neo4j/import
  data:         /var/lib/neo4j/data
  certificates: /var/lib/neo4j/certificates
  run:          /var/lib/neo4j/run
Starting Neo4j.
Started neo4j (pid 121). It is available at http://localhost:7474/
There may be a short delay until the server is ready.
See /var/lib/neo4j/logs/neo4j.log for current status.
0 rows available after 212 ms, consumed after another 0 ms
Added 1 constraints
0 rows available after 171 ms, consumed after another 0 ms
Stopping Neo4j.. stopped
Removing intermediate container 05bf08ff960e
 ---> e33d33c85441
Successfully built e33d33c85441

Adding the USER directive didn't change anything. The contents of all the directories and logs are empty on start of the image (here e33d33c85441)...

Hello @sdaschner, seems like you have the same isssue as here: https://github.com/neo4j/docker-neo4j/issues/166
Please try to follow this workaround and let us know if it works for you.
Thanks.


Very less font and colour options

There are only few font and colours options availabe ,it can be added with atleast of 10 choices

Hey, can you assign me to this issue? If yes, where do I find the Toolbar with the stroke and color options in the code? I already checked the files and somehow, I couldn´t find it or perhaps I have overlooked it. Thank you!

Thanks for the feedback, it is on the roadmap!


feat(#major); The Graph; Add The Graph Subgraph

What has been done:

Set up the subgraph and made compatible with CI/CD.
Started indexing rewards, usage metrics, and TVL from events from the Staking, Curation, and RewardsManager contracts

To Do:

Make sure that rewards, usage metrics, and TVL are not being double counted anywhere.
Determine if all reward streams are present.
Decide whether to include pools for the different roles. If any, pools for delegators and curators might make sense.

For todos, check out: https://docs.github.com/en/issues/tracking-your-work-with-issues/about-task-lists


Textures are not written correctly

I am able to reproduce this bug with this simple code
#include <bsa/fo4.hpp>

int main()
{
    using namespace bsa::fo4;
    file f;
    f.read("textures/in.dds", format::directx);
    f.write("textures/out.dds", format::directx);
}

This corrupts all tested textures. : Example
By looking at it in a hex editor, the header appears to be fine. The corruption starts at the end of the header, as soon as we enter the data section
The issue appears to be in write . Creating a textures BSA, and extracting it with another tool seems to work

Well, I investigated a bit more. By looking more closely in a hex editor, the issue is simpler than I thought
Some (20) CD bytes were inserted right after the header and before the data. The rest of the data is fine, just shifted

These bytes are in the place of the DX10 header. Maybe we need to force creation of a DX10 header

fixed by #6


NO-JIRA fix typos in key-differences.md and authentication.md

change
Another big difference that's good to cover early on is the difference is how message addressing and routing is done
to
Another big difference that's good to cover early on is the difference **of** how message addressing and routing is done
and
If your deployment is more complicated that this and requires some advanced JAAS configuration
to
If your deployment is more complicated **than** this and requires some advanced JAAS configuration

Squashed and pushed in https://github.com/apache/activemq-artemis/commit/e15bc1b35fef5553239d28b353c62dc488b3aedf, thanks.


Feature:  optional auto completion

Can we have vscode auto completion settings like this for nvim-cmp?
"editor.quickSuggestions" : [
   "other": true,
   "comments" : false,
   "strings" : false
]

https://github.com/hrsh7th/nvim-compe/issues/120#issuecomment-772108521
Well, it needs source code parse.

Related https://github.com/hrsh7th/nvim-cmp/issues/632


Current shelf of book

API Endpoint or Feature
If i get my list of Books, it would be nice to know in what shlef they currently are.
Use-Case
Yes, Books should be unique by name, but for in my case, I have a few books with the same name in different shelves. Would be nice to differentiate them by shelf.
Additional context
No response

Thanks for the suggestion.
I'm not sure on where we should expand the scope for the books - list endpoint, showing assigned shelves seems like it would be a bit out of scope since it's one-to-many parental detail. This is not a format we ever have in the interface itself at all.
It would make more sense to add a list of assigned shelves to the books - get endpoint, which would achieve the same thing but may require additional API calls.
You technically could already achieve this via pre-fetching the details of each shelf in the system, which lists assigned books, but I can see that's not ideal.

I'm going to therefore close this off as I do feel that a many-parent relation is out of scope for a listing endpoint, but if the alternative books - get option as mentioned above would suffice feel free to re-request for that.


Autogeneration of LF/SPINE fabric/access profiles fails when suffix is added

For example, in the aci_fabric_policies.tf module "aci_fabric_leaf_switch_profile_auto" (starting code line 368), the replace statement does not account for when a suffix is added in the default values for autogenerating fabric/access leaf/spine profiles, etc.
Working "LEAF\g" -> output is LEAF1001, LEAF1002 etc..
Not working "LEAF\g_SwPro" -> output is LEAF
Expected behaviour is to account for suffixes added to the naming

Thanks for raising the issue! This has already been fixed (https://github.com/netascode/terraform-aci-nac-aci/commit/c4be586c33f570dc408118bed8b0602b4bc4e14c) and will be included in the next release.

Hi,
Any update on when the new release will be available ?
Thanks
Alexander
From: Daniel Schmidt @.>
Date: Monday, 2 October 2023 at 10:02
To: netascode/terraform-aci-nac-aci @.>
Cc: Alexander Deca @.>, Author @.>
Subject: Re: [netascode/terraform-aci-nac-aci] Autogeneration of LF/SPINE fabric/access profiles fails when suffix is added (Issue #20)
Closed #20https://github.com/netascode/terraform-aci-nac-aci/issues/20 as completed.
—
Reply to this email directly, view it on GitHubhttps://github.com/netascode/terraform-aci-nac-aci/issues/20#event-10523076144, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEAJEUKC3MPSVDQY36BXCELX5JYJBANCNFSM6AAAAAA5PA4V6Y.
You are receiving this because you authored the thread.Message ID: @.***>

We are aiming for end of October, beginning of November.

Hey Daniel,
Any update on the drop for the new release? 😊
thanks
From: Daniel Schmidt @.>
Date: Monday, 16 October 2023 at 10:05
To: netascode/terraform-aci-nac-aci @.>
Cc: Alexander Deca @.>, Author @.>
Subject: Re: [netascode/terraform-aci-nac-aci] Autogeneration of LF/SPINE fabric/access profiles fails when suffix is added (Issue #20)
We are aiming for end of October, beginning of November.
—
Reply to this email directly, view it on GitHubhttps://github.com/netascode/terraform-aci-nac-aci/issues/20#issuecomment-1763944361, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEAJEUIJCMQWE2U6VGFFW5LX7TTFXAVCNFSM6AAAAAA5PA4V62VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRTHE2DIMZWGE.
You are receiving this because you authored the thread.Message ID: @.***>

We are working towards it. It should be ready by end of this week or latest next week. If you want to test the latest enhancements, you can always source the module directly from git:
module "aci" {
  source = "github.com/netascode/terraform-aci-nac-aci.git?ref=main"

  ...
}

Thanks!
From: Daniel Schmidt @.>
Date: Tuesday, 7 November 2023 at 11:08
To: netascode/terraform-aci-nac-aci @.>
Cc: Alexander Deca @.>, Author @.>
Subject: Re: [netascode/terraform-aci-nac-aci] Autogeneration of LF/SPINE fabric/access profiles fails when suffix is added (Issue #20)
We are working towards it. It should be ready by end of this week or latest next week. If you want to test the latest enhancements, you can always source the module directly from git:
module "aci" {
source = "github.com/netascode/terraform-aci-nac-aci.git?ref=main"
...
}
—
Reply to this email directly, view it on GitHubhttps://github.com/netascode/terraform-aci-nac-aci/issues/20#issuecomment-1798195112, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEAJEUNT4HYDV2OQ4PW5LRLYDICAHAVCNFSM6AAAAAA5PA4V62VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJYGE4TKMJRGI.
You are receiving this because you authored the thread.Message ID: @.***>


Ensure get_roles_reminders is sent even if no guest_role_received_time can be found

when this is merged we need to raise the logging level back to warning

when this is merged we need to raise the logging level back to warning

The logging is removed. Not finding the audit log is no longer seen as an error because it may be a common occurrence that user's were given the guest role longer ago than the storage of audit logs.


Allow rabbitmq username and passwords to be specified

Right now, these are programmatically generated. It would be nice if we could provide username and passwords to the spec instead of just generating them.

Do you think the credentials should always be required to be provided, rather than generated?

Most operators I've seen let you do either or. i.e. if you don't provide an externalSecretRef, then one is generated for you. A lot of helm charts follow this pattern as well.


[Doc fix]- Added Installation instructions for Windows OS

Closes #323
Closes #345

Please check it @VictorOdede @abhigya-sodani

Also this only closes #345

@VictorOdede

Hey @daredevilteja I'm getting this error while trying to run the script

Hi @VictorOdede I have fixed the error and updated the readme. Please run the tests now.


Add schema description for duplicate Postgres table names

I think you should wait to merge this one https://github.com/microsoft/vscode-cosmosdb/pull/1420 since it's using the same duplicate resource logic

This logic right here:
https://github.com/microsoft/vscode-cosmosdb/pull/1420/files#diff-466895c380cb7ae7167922111578c77eR81-R88
Rather than creating two sets every time the extension lists all of the resources, it uses a hashmap/object to store the name of resource and the schemes associated with it.  I personally find the duplicate set logic kind of 😬
Changing the loadMore logic seems like something we could do in a separate PR, but I don't have strong enough opinions to block this PR for that 😄

Could we change this duplicate duplicate (haha) logic before we merge this PR?
https://github.com/microsoft/vscode-cosmosdb/blob/master/src/postgres/tree/PostgresFunctionsTreeItem.ts#L56-L64
Including this PR and Will's (#1427), there are 3 separate times this gets called.  I'd like to change it mostly because it's a waste to create 2 new Sets for everytime a PostgresTreeItem loads or refreshes.
It also seems likely that other Postgres resources will use this logic, so I'd rather just consolidate it and make it a function on the PostgresDatabaseTreeItem.
Could either of you create a smaller PR that implements this logic?
It will probably be a while before Will's Functions PR gets merged since it's a pretty complex feature. so it makes sense to just add that specific part in now.

@neelip Could you use the new function that Will added to PostgresDatabaseTreeItem to add children rather than the duplicate set logic?

Sorry Nathan I missed your msg from yesterday. Thanks Will :)


Intersector.intersectPolygons fails

Using Intersector.intersectPolygons to find intersection of square and rotated trapezoid gives wrong overlapping.
In code
val trapezoid = Polygon(FloatArray(8))
val square = Polygon(FloatArray(8))
val overlap = Polygon()
// Initialize trapezoid.vertices and square.vertices with xy cordinates in CLOCKWISE order.
Intersector.intersectPolygons(trapezoid, square, overlap)

When initializing trapezoid with -2.8626027, -3.054608, 0.09345288, 14.654785, 18.01647, 4.9480224, 4.8018613, -7.2055407
and square with -8.879811, -14.852028, -8.879811, 5.1479716, 11.120189, 5.1479716, 11.120189, -14.852028,
I got overlap of
-3.4702098, 5.220678, -3.470294, 5.2206793, 0.0, 0.0, 3.4595287, 5.220678, 3.4597168, 5.220703, 0.0, 0.0
Here is grafical image of input objects

There is blue trapezoid intersecting with light-gray square, and YELLOW overlap result.
What can be wrong?

I did my best to reproduce your conditions, but did not get anything like what you showed:

Here's the code I used; it uses some libraries that I already was using, because I just wanted to use what I knew and get this tested quickly.
The overlap Polygon is drawn in red here, and seems correct, so it definitely isn't doing what your code does. We probably would need a simple, self-contained example that demonstrates the exact problem, and you haven't yet provided that. It seems like there could be some unrelated code that is also writing to the overlap Polygon.

Thank you so much for example! Finally I have found problem in my code.


Show badge in a tab

Hi there... is it possible to show a badge in a tab option?

Or is there any possibility of joining a future roadmap?

Hi! Sorry for the late reply. You can achieve this by creating your own PagingCell subclass. See UnsplashExample target for an example.
I want to keep Parchment as generic as possible, so I don’t have any plans of adding built-in support for this. Hope that makes sense 🙂
Let me know if you need any help implementing the custom cell

Hi @rechsteiner, I had not really thought about the option of creating my own PagingCell. Looking at the examples, including what you mentioned, I was able to simulate the badge and it worked.
Tks!

Hi there... is it possible to show a badge in a tab option?

Or is there any possibility of joining a future roadmap?

menuItemSource = .nib(nib: UINib)


Search spam bugs and bugs without steps to reproduce only between bugs filed by people without editbugs

We can assume bugs filed by people with editbugs are good enough, as they are expert enough bug filers.

Coverage increased (+0.01%) to 25.005% when pulling ae7a14fb719c9840f6393e325b0d9234f10285c0 on marco-c:noeditbugs into 28e6aefc2b3dfcca389d16370ab38df15e63f353 on mozilla:master.


Update investigate-threats.md

Based on text provided by Product team on PCX-8795
Updated the Categorization description

Tracking with #12987.


2D shadow issues with multiple viewports and particles with enabled SDF collision

Tested versions

Introduced with 5216ef5f9cb9479989b58aaab797a87a61899630 (#98701)
Still reproducable in 76fa7b291455a8ba24c50005072ebdb58f8a5984 (→ #99073 did not fix this)

(CC @DarioSamo)
System information
Godot v4.4.dev (76fa7b291) - Arch Linux #1 ZEN SMP PREEMPT_DYNAMIC Fri, 08 Nov 2024 17:57:58 +0000 on Wayland - X11 display driver, Multi-window, 2 monitors - Vulkan (Forward+) - dedicated Intel(R) Arc(tm) A770 Graphics (DG2) - Intel(R) Core(TM) i9-9900K CPU @ 3.60GHz (16 threads)
Issue description
https://github.com/user-attachments/assets/cfefbb56-9569-4ab1-a2c8-4c9a9e3ae421

There is a classic split screen setup (2 viewports, same world_2d) as this bug seems to only appear when rendering in multiple viewports
There is a particle system with particle collisions enabled
The particle system is not enabled initially
The left viewport has a camera, that moves from position A to position B
The right viewport is fixed at position A
There is one PointLight2D L1 attached to the camera/godot icon, and one PointLight2D L2 at position B

Steps to reproduce

When the camera has moved to position B (= left viewport only shows light L1 and right viewport only shows light L2) and the particles are enabled, the light L1 in the left viewport has shadows from light L2.

Observations:

When you enlarge the right viewport to render both lights, the issue disappears:
https://github.com/user-attachments/assets/ac4068c1-56d3-46a0-821c-8cb26a12ebda

Minimal reproduction project (MRP)
light-bug-repro.zip

Should be fixed in https://github.com/godotengine/godot/pull/99426.
The problem was a bit complex, but basically there was a wrong dependency detection if the following steps were taken:

The render target is used as a uniform texture for sampling (first viewport).
The render target is switched to a color attachment, but the first draw list that does so didn't cover the entire area.
The next draw lists would fail to detect an intersection and wouldn't insert themselves as being dependent on the transition caused by the previous step.

So this problem popped up often with resources shared between subviewports, as it reuses the same atlas and therefore causes a dependency on the writes.
There's potential to optimize this to do more drawing in parallel if different textures are used per sub viewport! But that's outside the scope of my fix and something the canvas renderer would have to detect if it's worth doing.

Thanks a lot! :heart:

Unfortunately, there are still problems with shadows in multiple viewports. @Calinou please re-open this issue as I don't have permission to do so. I will provide a new MRP tomorrow once I've figured out and isolated the specifics.

Scene:

Result:

MRP:
light-bug-repro-2.zip
CC @DarioSamo

@timoschwarzer 4.3 stable shows me this and it's what I see when I disable the intersection code.

So is it supposed to show up on the left or the right?

Please confirm if https://github.com/godotengine/godot/pull/99463 fixes the issue for you.

@DarioSamo Sorry, my screenshots are a bit confusing. The left viewport has the camera attached that shows the right light. So yes, your screenshot in 4.3 renders correctly. I will test your PR in a bit.

@DarioSamo I can confirm the issues are gone with https://github.com/godotengine/godot/pull/99463


Async support

Hello,
As far as I can tell, this can block as it uses the sync rust-postgres crate.
Is there any chance to make this asynchronous?
Thank you

Yep, I definitely plan to have a tokio-postgres version.

@sfackler Cool! AFAICT it doesnt seem like much effort.

Hi @sfackler, I'm happy to do this but need some clarification:

Errors: From<Error> is no longer implemented for Errorin postgres 0.18 and tokio-postgres. I'm inclined to say that ergonomically, a tokio_postgres::error::Error or postgres::error::Error is ideal, otherwise wrap in a postgres_large_object::Error type.
Should the interface for this crate be sync or async?

This seems like fairly straightforward code. I guess an alternative idea is to put a large_object feature into the parent crates?


Capacity Reduction for Kernnetz not necessary

In prepare_sector_network in add_storage_and_grids if config["sector"]["H2_retrofit"] the variable p_nom_extendable is already multiplied by the options["H2_retrofit_capacity_per_CH4"] factor.
This means that multiplying it again in modify_prenetwork is not necessary.
Only bugfix, so none of the points below are done.
Before asking for a review for this PR make sure to complete the following checklist:

[ ] Workflow with target rule ariadne_all completes without errors
[ ] The logic of export_ariadne_variables has been adapted to the changes
[ ] One or several figures that validate the changes in the PR have been posted as a comment
[ ] A brief description of the changes has been added to Changelog.md
[ ] The latest main has been merged into the PR
[ ] The config has a new prefix of the format YYYYMMDDdescriptive_title

Looks good to me.  @JulianGeis, please have a look if you agree with these changes, then the PR can be merged

Thank
  you for your mail. I am out of the office until Monday 9th
  September, and will respond to you after that.
Best
  wishes,
Julian
  Geis

--

Julian Geis (he/him)
PhD candidate in Energy System Modelling
Department of Digital Transformation in Energy Systems
Institute of Energy Technology
Technische Universität Berlin
Group website: https://www.tu.berlin/ensys
Einsteinufer 25 (TA 8)
10587 Berlin

This makes sense. The kernnetz lines are already in capacity of hydrogen and need no more conversion. PR can be merged.


Added separate methods for Levenshtein with search

Pull Request Test Coverage Report for Build 690295038

12 of 14 (85.71%)  changed or added relevant lines in 3 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.06%) to 87.197%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/natural/distance/levenshtein_distance.js
3
5
60.0%

Files with Coverage Reduction
New Missed Lines
%

lib/natural/brill_pos_tagger/lib/Brill_POS_Trainer.js
1
65.48%

lib/natural/brill_pos_tagger/lib/Corpus.js
1
74.0%

Totals

Change from base Build 686358762:
-0.06%

Covered Lines:
9366

Relevant Lines:
10369

💛  - Coveralls

Pull Request Test Coverage Report for Build 690306785
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

12 of 14 (85.71%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.003%) to 87.256%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/natural/distance/levenshtein_distance.js
3
5
60.0%

Files with Coverage Reduction
New Missed Lines
%

lib/natural/brill_pos_tagger/lib/Corpus.js
1
75.0%

Totals

Change from base Build 686358762:
0.003%

Covered Lines:
9368

Relevant Lines:
10369

💛  - Coveralls


Improve file upload on application form

Small improvement, I have changed the file upload input to follow the design of the rest of the application form

Codecov Report

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

@@           Coverage Diff           @@
##           master      #17   +/-   ##
=======================================
  Coverage   92.39%   92.39%
=======================================
  Files          23       23
  Lines         355      355
  Branches       31       31
=======================================
  Hits          328      328
  Misses         21       21
  Partials        6        6

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 13a66c8...4964cff. Read the comment docs.


Add support to create and publish NuGet package for xmlunit-constraints

Resolves issue #3
Tested build on Windows 7/.NET 4.x and Ubuntu 14.04/Mono 3.2.8

There are still issues with running nuget pack on Linux. The path separator switch works and nuget pack runs without error but no files get put into the generated nupkg. It only contains metadata.

I wonder whether you should try to "fix" the XmlUnit.Core dependency in the constraints nuspec file to become project.semantic.version.
The problem you see on Linux sounds like http://nuget.codeplex.com/workitem/4116
Finally I thought I had upgraded to NUnit 2.6, looks as if I had forgotten to :-)

Nuget pack is now working on Linux! I think the issue was NuGet issue 4116. I still needed to do the patch separator switch but it now create a nupkg with metadata and the file!
I also corrected the XmlUnit.Core dependency version in the constraints nuspec - Good catch!
For NUnit, do you want the latest (2.6.4)?

Great, now if you could squash your changes ... :-)
Yes, using the latest NUnit has always been my intention, I just seem to have slipped a few updates.   I guess it could be fixes as a by-product of #11.

Squashed! :-)


Remove visitor control mutability

Changes all the visitor functions to take immutable references
to their control nodes. This forces using the Action::Change pattern
to update nodes which allows the visitor framework to define what happens
when part of the AST changes during traversal.

LGTM


NSFileManager.defaultManager is not a method

Hi,
I've installed the plugin but working with iOS10 (I've not tested with previous versions of the SDK) I got an error stating that defaultManager is not a function but a property of NSFileManager:
[https://developer.apple.com/reference/foundation/nsfilemanager/1855948-defaultmanager?language=objc](Objective-C definition
)
The Objective-C code is
NSFileManager *fileManager = [NSFileManager defaultManager];
so it worked as a function... maybe Apple has changed something in the latest release...
Deleting the parenthesis the code works, but I think you'll have to modify the source code.

XCode 8 changed how to access some things, been a pain.   Thanks for letting me know that sqlite has one of them that needs to be fixed...

It should be fixed for iOS now on v1.1.2


Extend PlaceInfo

I would like to add the following properties to PlaceInfo:

Property
Data Type

parking
Parking[ ]

IMO, this should stay inside accessibility as it's an accessibility detail of a place.