GitHub Action to run OWASP Dependency-Check

Feature Request
Create a GitHub Action (as independent GitHub repository, following GitHub Actions development best practices) that scans a repository using the OWASP Dependency-Check
Tasks

[ ] Develop Action
[ ] Test Action
[ ] Document Action on ODP Wiki page
[ ] Enable Action on FINOS projects
[ ] Add Action to project-blueprint
[ ] Announce and socialise

Are you aware of https://github.com/Sburris/dependency-check-action ? This might fit your needs or can act as a starting point..

@MarcoEidinger , yes, definitely! Would be good to see it in action, maybe on this repository?
If you or anyone else would like to submit a PR to this repo, so that we can play a bit around it and build some docs to host on odp.finos.org (or even add it to the FINOS project blueprint!), that would be awesome.

We've delivered github actions around the OWASP DC plugin, checkout https://github.com/maoo/security-scanning
Closing this issue, we can create new items in repo mentioned above.


Pass original message down through conversion for storage write api

Enable users to specify an alternate way to generate the table row for the error output for BQIO's storage write api.
The user passes in a function of ElementT -> TableRow, and we maintain an index of the original elements passed in to BQIO. If the function exists, we use it to generate the error row, instead of the default behavior of emitting the failure directly.
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

@Abacn @ahmedabu98 could you take a look at this?

Also going to run some load test to see if it has performance implications

There was an unresolved discussion about maintaining update compatibility without duplicating a lot of code: https://github.com/apache/beam/pull/31106#discussion_r1587792477


Parse expression macro expansion buffers as expressions

This started out as a crash, where an expression macro could not be defined in terms of one of the builtin macros (e.g., #line), because we were expecting a macro expansion expression but didn't get one. Easy fix.
However, this uncovered a second bug, which is that we couldn't handle an expression macro expansino to #line. This is because we were parsing the macro expansion buffer as "top level items", which treats #line at the start of a line as a deprecated alias of #sourceLocation. Switch over to parsing a single expression in these contexts, and fix up an issue where #isolation didn't even have that expression.
Fixes rdar://139372780.

@swift-ci please smoke test


Recipes that include variables with generic level coordinates (e.g. cl) might not work anymore

After the merge of ESMValGroup/ESMValCore#598 recipes which use variables with generic level coordinates (e.g. cl) might not work anymore, since that PR included a checker for issues that have been detected before.
Example recipe:
https://github.com/ESMValGroup/ESMValTool/blob/master/esmvaltool/recipes/recipe_ecs_constraints.yml.
Example issues:
esmvalcore.cmor.check.CMORCheckError: There were errors in variable cl:
lev: units should be 1, not hPa
 lev: has values < valid_min = 0.0

wait - why should lev be in units 1 and not hPa?

also note that these recipes are still functional with --check_level ignore as an interim solution :+1:

wait - why should lev be in units 1 and not hPa?

cl uses a hybrid pressure coordinate (see https://scitools.org.uk/iris/docs/latest/iris/iris/aux_factory.html#iris.aux_factory.HybridPressureFactory), so this lev coordinate is only an auxiliary coordinate used to define the actual air_pressure coordinate.

ach so, fair dos, the intricacies of CMOR are getting at me, cheers dude :beer:

Found another related problem:
See ESMValGroup/ESMValCore#598:
I still have this weird issue that (I think) depends on the order of which the datasets are read:
The following recipe sometimes works, sometimes fails with
esmvalcore.cmor.check.CMORCheckError: There were errors in variable cl:
lev: units should be 1, not m
 lev: has values > valid_max = 1.0

for HadGEM3-GC31-MM and sometimes with the same error for ACCESS-ESM1-5.
Each of theses dataset does NOT throw an error when used alone.
What is going on here???
The recipe:
# ESMValTool
---
documentation:
  authors:
    - schlund_manuel

  maintainer:
    - schlund_manuel

  references:
    - gregory04grl

  projects:
    - crescendo

diagnostics:

  diag_test:
    variables:
      cl:
        project: CMIP6
        exp: historical
        mip: Amon
        start_year: 2004
        end_year: 2004
    additional_datasets:
      - {dataset: ACCESS-ESM1-5,   ensemble: r1i1p1f1, grid: gn}
      - {dataset: GISS-E2-1-G,     ensemble: r1i1p1f1, grid: gn}
      - {dataset: GISS-E2-1-H,     ensemble: r1i1p1f1, grid: gn}
      - {dataset: HadGEM3-GC31-LL, ensemble: r1i1p1f3, grid: gn}
      - {dataset: HadGEM3-GC31-MM, ensemble: r1i1p1f3, grid: gn}
    scripts:
      null

@schlunma Is this still an issue?

Yes!


Argument of type 'null' is not assignable to parameter of type 'MsalGuardConfiguration'

I tried to create a sample angular application as per this page. But in app.module.ts. I am getting the following error. This is a simple SPA in angular latest.

Please let me know if this is not enough details.
[Enter feedback here]

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

ID: 28c13084-08f6-c015-7769-c2ab9170c1d2
Version Independent ID: 44d5c264-8c95-0ae5-be9c-8982cea33223
Content: Tutorial: Create an Angular app that uses the Microsoft identity platform for authentication using auth code flow - Microsoft identity platform
Content Source: articles/active-directory/develop/tutorial-v2-angular-auth-code.md
Service: active-directory
Sub-service: develop
GitHub Login: @joarroyo
Microsoft Alias: joarroyo

I found the answer for this, as for first null, I used
{
        interactionType: InteractionType.Redirect,
        authRequest: {
          scopes: ['user.read'],
        },
      }

and for the second null, I used
{
        interactionType: InteractionType.Redirect,
        protectedResourceMap: new Map([
          ['Enter_the_Graph_Endpoint_Here/v1.0/me', ['user.read']],
        ]),
      }

The error went away.
Thank you

Thanks for sharing the update.


fix: use static variable incompatible with PHP8.1

Codecov Report
Base: 93.97% // Head: 93.97% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (41e1579) compared to base (4a59f62).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #168   +/-   ##
=========================================
  Coverage     93.97%   93.97%
  Complexity      478      478
=========================================
  Files           109      109
  Lines          1643     1644    +1
=========================================
+ Hits           1544     1545    +1
  Misses           99       99

Flag
Coverage Δ

phpunit
93.97% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

...raphqlite-bridge/src/ScalarType/AbstractScalar.php
100.00% <100.00%> (ø)

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

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


Remove stabilized #![feature]s, fix compilation

(Made Overlay's data field ManuallyDrop)

FYI, @CAD97's storages-api repository is a better approach to the problem.
The key idea behind Christopher's approach is that there's no need to type the storage like I did -- which ultimately leads to the requirement for GATs, and makes things complicated -- and instead a lower-level API similar to the Allocator API should be favored.
With this key idea, a Storage is just an Allocator which returns an abstract Handle, rather than a pointer.
This simplification allows trimming down the API massively, and does not prevent building a higher-level API on top, though it's unclear if such an intermediate between Storage and Box/Vec/... is necessary, or just cruft.

Thanks!


Fluent Settings page (instead of settings.json and defaults.json)

Description of the new feature/enhancement
As the title says, it would look better and be easier to use for both programmers (who are familiar with JSON) and non-programmers. It would also solve many problems.
For example, I have 6 different profiles (3 default and 3 installed), and I want to style all of them at the same time and/or in the same way. Now, the problem is I have to do the same thing 6 times (not all styles are applied by using a common color scheme). But instead, if we have a settings page which lets us choose styles for individual profiles AND/OR lets us make a standard master style template for all profiles, it would make styling easier and much organized.
Also, everyone loves Windows 10's Fluent Design System, and therefore, a Fluent Settings page would make changing settings look much easier and more modern, and in the background, this also easily gets the work done.
Proposed technical implementation details
At the same time, these JSON files may not be removed.
We can create a Settings page (which opens as another tab OR as a completely different section in the same window), with options as checkboxes, color-pickers, float value text boxes (for floating points like "acrylicOpacity", "padding", etc.), etc., and the respective settings will be applied (after clicking on an "Apply" button at the end of the settings page OR dynamically applying them as soon as a setting is changed) by changing the settings.json file (which, if wished, can be accessed by the user by opening the Windows Terminal's directory).
TL;DR
We create a Fluent Settings interface with AcrylicBrush and RevealBrush, etc., which in the background dynamically modifies the settings.json file, as it fixes many boring problems (one of which has been mentioned before as an example), gives a good look and impression on Windows Terminal to many users and will be neatly welcomed.

See issue #1564.

@j4james Oh I did not know that existed. And btw, I was kinda talking about a slightly different UI than #1564
Should I close this issue in that case?

Yeah, if you have suggesting for how the UI should work, it's best to add them to the original issue. You'll see there are already a bunch of comments there from different people with their own ideas. As for closing the issue, someone from MS will probably close it with a dup link eventually, but you can also close it yourself if you want.

Yep, this is definitely a dupe. We should continue the discussion over on that thread. Thanks!
/dup #1564


Hive MetaStore threads leak after upgrading JuiceFS 1.0.2 SDK

What happened:
Upgrading JuiceFS SDK from 1.0 to 1.0.2, Hive MetaStore can't serve new request and keep throwing OOM error with "unable to create new native thread" error message. We dump java stack of Hive MetaStore JVM to analyze. We find there are 14394 threads in total, 6036 threads named "Uid and group refresher" and 6040 threads named "Node fetcher".  After downgrading to 1.0 SDK, Hive MetaStore works fine and the thread number keeps around 400+.
What you expected to happen:
Threads of Filesystem should be shutdown correctly.
How to reproduce it (as minimally and precisely as possible):
Debugging following code with 1.0.2 sdk, you will find thread named "Uid and group refresher" and "Node fetcher" after fs.close(). And there are not those threads debugging with 1.0 sdk.
    FileSystem fs = FileSystem.get(new URI("jfs://jfs-test"), hadoopConf);
    Path p = new Path("jfs://jfs-test/hello.txt");
    FSDataInputStream in = fs.open(p, 1024);
    byte[] buffer = new byte[1024];
    in.read(buffer);
    in.close();
    fs.close();

    FileSystem fs2 = FileSystem.get(new URI("jfs://jfs-test"), hadoopConf);
    FSDataInputStream in2 = fs2.open(p, 1024);
    buffer = new byte[1024];
    in2.read(buffer);
    in2.close();
    fs2.close();

Anything else we need to know?
This issue is introduced by #2512 . FileSystem object with fetcher thread and refresher thread can't be closed.

Environment:

JuiceFS version (use juicefs --version) or Hadoop Java SDK version: 1.0.2
Cloud provider or hardware configuration running JuiceFS: Not related
OS (e.g cat /etc/os-release): CentOS 7.3
Kernel (e.g. uname -a): Not related
Object storage (cloud provider and region, or self maintained): Not related
Metadata engine info (version, cloud provider managed or self maintained): Not related
Network connectivity (JuiceFS to metadata engine, JuiceFS to object storage): Not related

#2512

#3090


Create nesting for Network Policy providers

This helps shortening the /docs/tasks→Administer Cluster section by creating
a section for the current and future networking providers.
/cc @caseydavenport, @chrismarino, @bboreham

This change is

/assign @chenopis


Manual page for lychee

As a user of lychee I'd like to read the manual with man lychee as is possible with other programs.
Personally, I've never written a man page before, but I can give it a shot when I find the time. Most content from the README can probably be reused. I guess the sections and structure would be the most difficult part to get right.
@mre do you have any experience with writing man pages?

Good idea. I don't have any experience with man pages yet, but the README and the docs are a good start.
As for tooling, here's a survey: https://stackoverflow.com/a/78226570


Can't interact with the UI on VR

Hey! love the mod. i was wondering if you could add some support for VR and the UI. i really enjoy this mod cause if you go to charge and item and there are suits in the way. if you could make some sort of support for VR that would be lovely
Here is the VR repo: https://github.com/DaXcess/LCVR

Unfortunately I do not own any VR headsets and therefore would not be able to implement compatibility. You can implement it yourself and make a pull request if you know how to code.


Prevent EVM account nonce to be zero again if account gets reaped

Otherwise the EVM account becomes unusable until the proper nonce is restored.

Fixed


Upgrade bindgen to 0.43

Issue: #5
The rust-bindgen package is not yet updated to 0.43.2 on crates.io, so this PR currently gets it from github. We can wait for the update before merging.

These will need to be added, but it's buggy in combination with ParseCallbacks::item_name
.whitelist_type("(?i)mbedtls.*")
.whitelist_var("(?i)mbedtls.*")
.whitelist_function("(?i)mbedtls.*")
.whitelist_recursively(false)

I made some more changes, but still waiting for a couple bindgen bugs to be resolved:
diff --git a/mbedtls-sys/build/bindgen.rs b/mbedtls-sys/build/bindgen.rs
index ec0a1c7..454ae73 100644
--- a/mbedtls-sys/build/bindgen.rs
+++ b/mbedtls-sys/build/bindgen.rs
@@ -99,14 +99,15 @@ impl super::BuildConfig {
                     .to_str()
                     .expect("failed to convert header path to string"),
             ).use_core()
-            .derive_debug(false) // buggy :(
             .disable_name_namespacing()
             .prepend_enum_name(false)
             .ctypes_prefix("raw_types")
             .parse_callbacks(Box::new(ParseCallbacks{}))
-            // max_align_t is causing bindgen generated tests to fail an alignment check and
-            // is not needed by the bindings.
-            .blacklist_type("max_align_t")
+            .blacklist_type("mbedtls_iso_c_forbids_empty_translation_units")
+            .whitelist_type("(?i)mbedtls.*")
+            .whitelist_var("(?i)mbedtls.*")
+            .whitelist_function("(?i)mbedtls.*")
+            .whitelist_recursively(false)
             // Including the comments breaks the generated code because it contains formatting
             // that is interpreted as escaped characters.
             .generate_comments(false)
@@ -122,7 +123,7 @@ impl super::BuildConfig {

         let mod_bindings = self.out_dir.join("mod-bindings.rs");
         File::create(&mod_bindings)
-            .and_then(|mut f| f.write_all(b"mod bindings;\n"))
+            .and_then(|mut f| f.write_all(b"#[allow(bad_style)] mod bindings;\n"))
             .expect("mod-bindings.rs I/O error");
     }
 }

https://github.com/rust-lang/rust-bindgen/pull/1452
https://github.com/rust-lang/rust-bindgen/issues/1454
https://github.com/rust-lang/rust-bindgen/issues/1455

@jethrogb any update on this?

I rebased and fixed the original branch from this PR: https://github.com/awakecoding/rust-mbedtls/tree/bindgen-upgrade-v2

@awakecoding Thanks, but I'm still waiting for https://github.com/rust-lang/rust-bindgen/issues/1454 and https://github.com/rust-lang/rust-bindgen/issues/1455.

Obsoleted by #72

Final PR: #152

Fixed in #152


LOADING Redis is loading the dataset in memory

What's the best way of handling this exception?

Hi, the one thing you can do is to wait until data loading is complete.
You can catch this particular error, and retry after some idle (sleep) time.
Also, you can use info persistence command and wait for loading flag to become 0.
Check the command documentation (see persistence section) https://redis.io/commands/info


Caps Lock no longer works as caps_lock with to_if_alone

I've used the following stanza to change the behavior of the Caps Lock key in the following ways:

When pressed alone, caps_lock toggles Caps Lock.
When held down while another key is pressed, caps_lock acts as the left_control modifier

This behavior works just fine in 11.6.0 but seems broken in 12.0.0 - 12.0.7.
In 12.0.0 - 12.0.7, caps_lock is still seen as a key event when pressed alone, but it no longer activates Caps Lock. Caps Lock seems to behave normally when it isn't assigned.
{
    "description": "Change caps_lock key to left_control if pressed with other keys. (Post caps_lock when pressed alone)",
    "manipulators": [
        {
            "from": {
                "key_code": "caps_lock",
                "modifiers": {
                    "optional": [
                        "any"
                    ]
                }
            },
            "to": [
                {
                    "key_code": "left_control",
                    "lazy": true
                }
            ],
            "to_if_alone": [
                {
                    "key_code": "caps_lock"
                }
            ],
            "type": "basic"
        }
    ]
}```

Please use hold_down_milliseconds
https://github.com/tekezo/Karabiner-Elements/issues/1349#issuecomment-381379081

Thanks, that did it! For anyone interested, here's the fixed stanza:
{
    "description": "Change caps_lock key to left_control if pressed with other keys. (Post caps_lock when pressed alone)",
    "manipulators": [
        {
            "from": {
                "key_code": "caps_lock",
                "modifiers": {
                    "optional": [
                        "any"
                    ]
                }
            },
            "to": [
                {
                    "key_code": "left_control",
                    "lazy": true
                }
            ],
            "to_if_alone": [
                {
                    "key_code": "caps_lock",
                    "hold_down_milliseconds": 100
                }
            ],
            "type": "basic"
        }
    ]
}


Formatter: include filenames in ruff format --check

Apologies if this is a duplicate or already fixed; the closest issue I found to this one was #7231.  This issue is similar, but much smaller scoped.  When I run black --check, it gives me a list of filenames that would be reformatted, not just a count.  When I run ruff format --check, it only gives me statistics.  The list of filenames is much more useful when the format check is run during CI, and it'd be nice to add it to ruff.
I'm running ruff 0.0.291.  Here's a sample output from black and from ruff:
~/my_code % ruff format --check .
warning: `ruff format` is a work-in-progress, subject to change at any time, and intended only for experimentation.
5 files would be reformatted, 264 files left unchanged

~/my_code % black --check .
would reformat /path/to/my_code/my_file.py

Oh no! 💥 💔 💥
1 file would be reformatted, 268 files would be left unchanged.

Good call, we should fix this!

Awesome, thanks!

No prob, thanks for reporting.

@charliermarsh It would also be useful to have that when running it without --check (or at least allow to enable it with an option).
Usecase: I use the pre-commit config to run Ruff locally before each commit, but also on the CI (for all files).

Locally, I want to automatically change the files with the formatter on a commit.
In the CI, I want to see what files are not correctly formatted.

And thanks for the amazing tools (Ruff and uv) @charliermarsh! 😁


Clarify documentation regarding templates customization

Description
Improve documentation especially for people not used to the django CMS jargon
References
Ref #595
Checklist

[x] I have read the contribution guide
[ ] Code lint checked via inv lint
[x] changes file included (see docs)
[x] Usage documentation added in case of new features
[ ] Tests added

I added the link to the django documentation, thanks for the review!


Unload button doesnt seem to work?!

Hey, I'm finding the unload button doesnt seem to work? No movment from the extrusion servo. Works on loading but not unloading. Im using Marlin 2.x on an ender 3.
Any suggestions?

It's probably a limitation of your firmware preventing the configured length of the unload extrude command. See #15


Add tweet badge flow

Use my Twitter guide docs

[![Tweet](https://img.shields.io/twitter/url?style=social&url=https%3A%2F%2Fmichaelcurrin.github.io%2Fpython-twitter-guide%2F)](https://twitter.com/intent/tweet?url=https%3A%2F%2FMichaelCurrin.github.io%2Fpython-twitter-guide&hashtags=twitter-api,tweepy,python,guide,tutorial)

tweet
https://twitter.com/intent/tweet?original_referer=https%3A%2F%2Fnecolas.github.io%2Fnormalize.css%2F&ref_src=twsrc%5Etfw&text=Normalize.css%3A%20a%20modern%20HTML5-ready%20alternative%20to%20CSS%20resets&tw_p=tweetbutton&url=https%3A%2F%2Fnecolas.github.io%2Fnormalize.css%2F

Tweet

Tweet

See also existing tool for generating Tweets based on URL, title, description, image URL
https://www.sociallinkgenerator.com/#generated-links

e.g.
https://twitter.com/intent/tweet?url=https%3A%2F%2Fgithub.com%2FMichaelCurrin&text=MichaelCurrin on GitHub%3A My description

Documentation
https://developer.twitter.com/en/docs/twitter-for-websites/tweet-button/guides/web-intent

Related
https://publish.twitter.com/#
https://help.twitter.com/en/using-twitter/twitter-buttons
https://help.twitter.com/en/using-twitter/add-twitter-share-button

Follow button docs https://developer.twitter.com/en/docs/twitter-for-websites/follow-button/overview

Example

URL https://michaelcurrin.github.io/badge-generator/
Title Badge Generator
Magically generate custom badges that you can paste in your docs

Result:
https://twitter.com/intent/tweet?url=https%3A%2F%2Fmichaelcurrin.github.io%2Fbadge-generator%2F&text=Badge Generator%3A Magically generate custom badges that you can paste in your docs


Adding a working Password Manager Desktop app

Is your feature request related to a problem? Please describe.
I want to add a password-manager app in which you can add accounts which will be stored in a local encrypted sql database. It will also have a built-in password-generator, password-strength-checker.
Describe the solution you'd like

I will use electron to make a desktop app with react frontend.
I will use built in crypto(nodejs) library to encrypt all the emails and passwords before saving in database.

Additional context/ Screenshots
I want to submit this project under GSSOC'22
Landing Screen (if user didn't already created a master password)

A cool password generator which can be used across all the app e.g when adding a new account.

Very Simple Form to add new accounts to your vault.

Clean looking UI and very easy to understand. User can do full CRUD operations on any of his accounts.

So sorry! I raised the same issue on a different repo but didn't knew it is not allowed to share the same projects on different repos under GSSOC!
Again I am truly sorry.


Switch active network

This PR can switch active network.
https://user-images.githubusercontent.com/47471299/231543022-e4d22e4f-dcf6-44b8-9dc9-18c4214d6977.mov
This is sample duplicate network definition.
import { EVMNetwork } from './pages/Background/types/network';

const networks: EVMNetwork[] = [
  {
    chainID: '5',
    family: 'EVM',
    name: 'LocalGoerli',
    provider: 'http://xxxxxxxx',
    entryPointAddress: '0x087654323456789098765,
    bundler: 'http://xxxxxxxxxxxxxx',
    baseAsset: {
      symbol: 'ETH',
      name: 'ETH',
      decimals: 18,
      image:
        'https://ethereum.org/static/6b935ac0e6194247347855dc3d328e83/6ed5f/eth-diamond-black.webp',
    },
  },
  {
    chainID: '11155111',
    family: 'EVM',
    name: 'LocalSepolia',
    provider: 'http://xxxxxxxx',
    entryPointAddress: '0x045676543456787654345678,
    bundler: 'http://xxxxxxxxxxxxxx',
    baseAsset: {
      symbol: 'ETH',
      name: 'ETH',
      decimals: 18,
      image:
        'https://ethereum.org/static/6b935ac0e6194247347855dc3d328e83/6ed5f/eth-diamond-black.webp',
    },
  },
];

// eslint-disable-next-line import/no-anonymous-default-export
export default {
  enablePasswordEncryption: false,
  showTransactionConfirmationScreen: true,
  factory_address: '0x09c58cf6be8E25560d479bd52B4417d15bCA2845',
  stateVersion: '0.1',
  networks,
};

Thanks, this is amazing! Can we add support for "wallet_switchEthereumChain" as well, this will also allow the auto-change of chains by the Dapp.
EIP - https://eips.ethereum.org/EIPS/eip-3326#specification
Line of code where to make change: https://github.com/plusminushalf/trampoline-example/blob/master/src/pages/Background/services/provider-bridge.ts#L564


Configserver firewall, CSF, does not show configuration button/page

System:
Latest theme (18.32) and Webmin (1.830) versions installed.
Centos 7.3
Nginx latest
PHP 5.6.9
MariadB 5.5
Epel, remi, virtualmin, and centos repos
I did a fresh install of Centos 7.3 and Virtualmin/Webmin Pro yesterday.  I installed configserver firewall, CSF, and the CSF module was working, including the configuration button on the first/landing page.
Then I disabled Apache, installed and enabled Nginx and did a yum update to assure all are latest versions.
After the change from Apache to Nginx, the configuration button is no longer visible.  This is not a 'problem' in that I had already configured CSF and can edit the csf.conf directly.  However, I would very much like to have the web interface through Webmin again.

https://github.com/qooob/authentic-theme/issues/663

OK, I see the COG button... it changed from a text menu button.  Not as obvious but that works.


Document test case for csmock

An example can be seen here.

And here is the output for something more interesting.


MAT-2306 Execution of CV measure observations

MAT-2306 Execution of CV measure observations

updated measure parameter parsing

Pull requests into cqm-parsers require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Submitter:

[ ] This pull request describes why these changes were made.
[ ] Internal ticket for this PR:
[ ] Internal ticket links to this PR
[ ] Code diff has been done and been reviewed
[ ] Tests are included and test edge cases
[ ] Tests have been run locally and pass

Reviewer 1:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Reviewer 2:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Codecov Report

Merging #123 (b704bbe) into bonnie-on-fhir (0577930) will increase coverage by 0.18%.
The diff coverage is 100.00%.

@@                Coverage Diff                 @@
##           bonnie-on-fhir     #123      +/-   ##
==================================================
+ Coverage           90.21%   90.40%   +0.18%
==================================================
  Files                  13       13
  Lines                 593      594       +1
==================================================
+ Hits                  535      537       +2
+ Misses                 58       57       -1

Impacted Files
Coverage Δ

lib/measure-loader/bundle_loader.rb
85.47% <100.00%> (+0.98%)
:arrow_up:

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 0577930...b704bbe. Read the comment docs.


Fast Toggling of section activates markdown edit mode

When I quickly (or too quick) click on the open/close arrow of a section, the section heading goes into markdown edit mode (and I have to execute it to be formatted again)
Here a piece of my notebook:

Here is how it looks after two quick clicks on the arrow:

It would be nice to fix this behaviour since it prevents fluid navigation in the document. Thanks for the great extension!

I also seem to experience this issue?  It does appear to relate to the "speed" of multiple clicks on the open/close arrow. This mainly happens when you are playing with it & testing it out. =)
Overall this is a VERY welcome extension, @aquirdTurtle - thank you!
I'm sure you have little time to address this issue and I'm not sure where I'd start to help out myself but if you'd like more documentation of the issue from my POV let me know.

So, I just took a look at this, and it's proving to a bit tricky to properly fix. The problem is that the button is a child of the cell class and when you double-click the button, it also calls the cell's default double-click handler - or something in that spirit - which I don't know how to override. The only workaround I found was to set a timer to re-run the markdown cell after the double-click, but this is very hackish and I don't think it's a good idea for my extension to run peoples cells unexpectedly.
However, it is true that this is only an issue with the click-to-toggle functionality. If you toggle using anything else, including the "T" keyboard shortcut, your own shortcut, or the left/right arrows, then there's no problem. I'll leave this issue open for now in case I or anyone else finds a proper solution.


Is bilingual GLM with smaller model size like 1.5B, 2.7B available?

Hi!
I have checked #9. But there are all monolingual models. Is bilingual GLM with smaller model sizes like 1.5B, 2.7B available?

@BrightXiaoHan There is a multilingual (104 languages) 1B GLM model listed in our README. We haven't trained or planned to release other bilingual GLMs yet.


Maintain grounders

@zhengzhongliang, to what extent are you still working on grounding?  That single grounding file should have been split into parts some time ago, but not while so many different people were working on it.  Have things slowed down enough that this PR won't cause you problems?
@BeckySharp just FYI.  The ontologies are split up as well.  For both grounders and ontologies there are additional variations in different eidos forks and this hopefully helps with isolating the changes.

Hi Keith,
Yes I am working on some final ideas of the grounder. It would be great if
I can have one more week to do those experiments. I think they will be done
by the world modeler meeting next week.
Best regards,
Zhengzhong
On Tue, Jun 9, 2020 at 10:16 AM Keith Alcock<EMAIL_ADDRESS>wrote:

@zhengzhongliang https://github.com/zhengzhongliang, to what extent are
you still working on grounding? That single grounding file should have been
split into parts some time ago, but not while so many different people were
working on it. Have things slowed down enough that this PR won't cause you
problems?
@BeckySharp https://github.com/BeckySharp just FYI. The ontologies are
split up as well. For both grounders and ontologies there are additional
variations in different eidos forks and this hopefully helps with isolating
the changes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/875#issuecomment-641456680, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVRS5YWGEEEYNBWGZVDHW3RVZU6DANCNFSM4NYZMQZA
.

It can wait a little for Zhengzhong.  I've been lulled into complacency by Scala's C-style ability to include numerous classes in a single file.  It starts out being handy, but at some point there's an overload.  The JLD serializers will eventually meet a similar fate.

Hi Keith,
Since I do not intend to change any major things in this branch (i.e., done with this for now), maybe you can start modifying it. Thanks for waiting for me.
Best regards,
Zhengzhong

Thanks, @zhengzhongliang.  I'll merge soon.  It shouldn't be too difficult to figure out where things have moved to.


Update Button Text on Main Page

The current text displayed on buttons on the main page could be optimized for better clarity and user experience. This issue aims to make these buttons more concise and straightforward, enhancing the overall usability of the main page.
Proposed Changes:
We recommend shortening the following button texts:
From: "Features Sign Up"
To: "Features"
From: "Code & Coffee Chapters"
To: "Chapters"
From: "Photos from past events"
To: "Past Events"

I will make the changes according to the updated Figma design.

PR - https://github.com/CodeandCoffeeCommunity/boston-code-and-coffee/pull/49


[topgen] topgen flow

Currently, topgen's flow goes something like the following

gather all non-top hjson configurations (ips, including top generated ips and xbar)
validate all hjsons, including consistency checks between top and xbar/ips
if validation passes, merge all hjsons into top_earlgrey.gen.hjson
generate other downstream collateral such as top related ips

This generally works, however issues pop-up for top generated ip's. Specifically..

The first time top generated ips are included (rvplic, alert_handler) there is no hjson to be validated and merged in steps 2/3 above
If a particular hjson is generated based on gathering results of other ip.hjson (alert_handler, clks/pwr/rst may all fall into this category), it creates a dependency where either the hjsons are not there (problem above), or the hjsons included may be outdated.

There are many ways to solve this, but one thought that was quickly discussed with @eunchan is to allow for a generate step between 1/2.  There are some blocks (for example clk) who don't actually need ip information to perform generation (it only needs top information).
For other blocks that do have dependencies, it might make sense to some kind of dependency graph and handle those later or in a second / third pass of topgen.  For example, assuming the dependency is as follows:
ips -> pwrmgr -> alert_handler -> rvplic

This potentially could be 4 passes of topgen (or we hardcode the order of when things are run) and then the final topgen merge is done.
There are definitely many other ideas, so this issue mostly is just trying to frame the problem so it is not forgotten.

Thanks. As we get more generated IPs, I keep thinking that splitting out the generated part into separate modules in topgen makes sense.
For instance, clkmgr generation (validation also) codes is in util/topgen/clock_manager.py and it has common function such as validate(), elaborate(), generate() etc. These functions are same across all the topgen plug-able libraries such as interrupt, alert, intermodule, etc.
Topgen, then, should have a code that calls the functions  at certain stages in the main code. As you mentioned, it may calls prior to load ip objects, after loaded ip objects, etc..

yeah i tend to agree with this. I feel like if we have separate
clk_manager.py, then that same code could also be used for the examples in
the individual hw/ip/clkmgr directory.
That way everything stays consistent.
On Wed, Apr 29, 2020 at 5:52 PM Eunchan Kim<EMAIL_ADDRESS>wrote:

Thanks. As we get more generated IPs, I keep thinking that splitting out
the generated part into separate modules in topgen makes sense.
For instance, clkmgr generation (validation also) codes is in
util/topgen/clock_manager.py and it has common function such as validate(),
elaborate(), generate() etc. These functions are same across all the
topgen plug-able libraries such as interrupt, alert, intermodule, etc.
Topgen, then, should have a code that calls the functions at certain
stages in the main code. As you mentioned, it may calls prior to load ip
objects, after loaded ip objects, etc..
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/2083#issuecomment-621548204,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSUSFY4L6CGJMDS3W6DRPDDWBANCNFSM4MTJLM5A
.


[1.3.11] Update manifest with tags 1.3.11

Description
Update manifest with tags 1.3.11 as one of post release processes
Issues Resolved
N/A
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Please rebase and only change the refs

Thanks, @gaiksaya ..there were some duplication records from the previous PR to update manifest; hence; this PR is also going to remove those duplications..

Please rebase and only change the refs

Thanks, @gaiksaya ..there were some duplication records from the previous PR to update manifest; hence; this PR is also going to remove those duplications..
Let me update description to note that as well

Lets not change the manifest now. Because this is what we shipped with. We need to add mechanism to check for duplicate values in yaml file for each component. Can you please create an issue for this? To add a check at CI level for duplicate values?
Thanks!


zh format example_id=30 does not contain in evaluation

This sample does not contain in gpt4_based_evaluation:
https://github.com/YJiangcm/FollowBench/blob/master/code_zh/gpt4_based_evaluation.py#L214
and does not contain in rule_based_evaluation:
https://github.com/YJiangcm/FollowBench/blob/master/code_zh/rule_based_evaluation.py#L30
So I wonder how to calculate the score for this sample?

Thanks for pointing out this. We have fixed the bug by using gpt4_based_evaluation for zh format example_id=30.


document "identifier" and drop "metadata"

Started working on the first one, then realized I need to fix the second too.
Fixes #440
Fixes #433

@majamassarini targets accepts a list of strings:
diff --git a/.packit.yaml b/.packit.yaml
index 44a17f3..52ae9a8 100644
--- a/.packit.yaml
+++ b/.packit.yaml
@@ -19,11 +19,11 @@ upstream_project_url: https://github.com/majamassarini/knx-stack
 jobs:
 - job: copr_build
   trigger: pull_request
-  targets: fedora-stable
+  targets: [fedora-stable]
   owner: mmassari
   project: knx-stack
 - job: copr_build
   trigger: commit
-  targets: fedora-stable
+  targets: [fedora-stable]
   owner: mmassari
   project: knx-stack

After doing that change, I got an SRPM:
$ packit srpm
2022-06-01 11:59:59.834 upstream.py       INFO   Version in upstream registries is '0.9.2'.
2022-06-01 11:59:59.834 upstream.py       INFO   Version in spec file is '0.9.2'.
2022-06-01 11:59:59.834 upstream.py       INFO   Picking version '0.9.2' found in spec file.
2022-06-01 12:00:01.310 api.py            INFO   Directory with sources: /home/tt/g/knx-stack
2022-06-01 12:00:01.565 command_handler.py INFO   Nothing to clean.
2022-06-01 12:00:01.565 srpm.py           INFO   SRPM: /home/tt/g/knx-stack/python-knx-stack-0.9.2-1.20220601115959869882.remove_metadata.8.gef2686f.fc36.src.rpm

I updated the examples using targets: [fedora-stable] syntax. @majamassarini @lachmanfrantisek please let me know if this is okay with you


Update declarative-ids.mdx

Call out that extras.graphql needs to be referenced in your schema configuration or it will be silently ignored!

@FredLoh: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/


sphinx incorrect URL

Platform
Which platform does the bug apply to?

Website

Describe the bug
sphinx redirects to http://sphinx.pocoo.org/ which is not working and it should be https://www.sphinx-doc.org/en/master/ according https://github.com/sphinx-doc/sphinx.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://www.wappalyzer.com/technologies/documentation-tools/sphinx'
Click on 'Visit webiste'
See error

Expected behavior
Should redirect to correct webiste.
Additional context
I would would make a PR if anyone points out to a how-to change technology url.

Thanks!
Pull request #3190 will fix this issue.

I would would make a PR if anyone points out to a how-to change technology url.

See https://www.wappalyzer.com/docs/dev/contributing
For this case, just edit https://github.com/AliasIO/wappalyzer/blob/eb2dfc714aa03232dc7e6c87a9e807bff2369426/src/apps.json#L10581


combine commands.LargeInt with commands.Range

Summary
LargeInt cannot be restricted like Range, and Range cannot be a big int.
What is the feature request for?
disnake.ext.commands
The Problem
LargeInt is an integer bigger than 2 ** 53, and Range only works on numbers below 2 ** 53. It would be nice to have a combination of the two, possibly even merge them into one class.
We also don't have LargeFloat right now, so a modified Range could potentially support that.
The Ideal Solution
Continue to keep LargeInt around, but merge support into Range. This means that if Range has more than the limit provided by discord, we instead use an underlying string type and manually compute the value. In order to constrain the value further, we should provide min_length and max_length computed to the length of the object.
For example:
# setup variables
# for those wondering, these are the shortest and current longest possible snowflake values
min = int("1" + "0" * 16)
max = int("9" * 19)

commands.Range[min, max]

This would create a Range object that is rendered as a string but has a min_length set to 17, and a max_length set to 19, as that is how long the beginning number and second number are after converted to a string. After that we would compute the limitations as normal once we received an interaction. We should also keep in mind https://github.com/python/cpython/issues/95778
The Current Solution
None
Additional Context
No response

I would like to implement this!

Since #991 made commands.Range require taking the actual type as first param, I think it makes sense to do extend this to take LargeInt: commands.Range[LargeInt, min, max]
Exceeding the bounds on normal int should raise an error, telling the user to use LargeInt if they really want the greater range
This way it's an explicit opt-in to use a string type to represent larger numbers


SEO URL for detail pages

currently the URL of a product page in the shopware-pwa is the like this:
http://localhost:3000/detail/8290109126884ca09733057358e31843
In Shopware Backend Settings -> SEO I created the URL Template for Product Detail Page like this:
{{ product.translated.name }}/{{ product.productNumber }}
Preview: Main-product-with-advanced-prices/SWDEMO10002
The SEO Index is correctly done and all urls are in table set_url.
In Shopware-PWA the URL of product detail and category pages should be always like the settings in backend.

hi @theomanus, thanks for the details regarding seoUrls.
Yes, we are aware of lacks of SEO Urls within the listing, so far we are using only the technical URLs because we can't get the seoUrls object in product-listing/page request.
I will be back with some updates when we prepare mutual solution with Shopware core team :muscle:

@mkucmus any news on this? :)

@mkucmus any news on this? :)

Yep, It's on the way to be merged right now :+1:
https://github.com/DivanteLtd/shopware-pwa/pull/1134
will be released in v0.5.0, together with SW v6.4 adjustments stuff
stay tuned :muscle:


Fixed typo

Coverage remained the same at 100.0% when pulling 0c35e881310e14d9de2ad4c55e8d36aeca83277e on 3-add-testing-coverage into 6b8cb39320671b3c43f80b2ba563521913b9ac91 on master.


Readme.md: linked encoding explanation

Just added a nice explanation, as I could myself not figure out how it worked initially.

Please clean/delete, as I renamed the branch (naming convention)


BloodHound-CE No Data Available after uploading JSON file

Hi,
I'm unable to view the data after uploading the JSON files generated by the bloodhound.py collector (bloodhound-ce branch).
Command used to collect data:
bloodhound-python -c All -d "academy.ninja.lan" -u 'frank' -p 'P@ssw0rd' -dc dc-ac.academy.ninja.lan --dns-tcp -ns <IP_ADDRESS>

Docker log from bloodhound-bloodhound-1 container after uploading the JSON files
bloodhound-bloodhound-1  | {"level":"error","time":"2024-01-06T11:14:35.832660728Z","message":"Error processing ingest task 19: json: cannot unmarshal number into Go struct field Trust.Trusts.TrustDirection of type string"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":0.005749,"time":"2024-01-06T11:14:37.975877997Z","message":"Starting analysis"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":153.482291,"time":"2024-01-06T11:14:38.129805231Z","message":"Fix well known node types"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":76.029569,"time":"2024-01-06T11:14:38.206247341Z","message":"Domain Associations"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":30.340705,"time":"2024-01-06T11:14:38.236599767Z","message":"Link well known groups"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":69.349496,"time":"2024-01-06T11:14:38.305968105Z","message":"ClearSystemTagsIncludeMeta"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":185.475518,"time":"2024-01-06T11:14:38.422094811Z","message":"Updated asset group isolation tags"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":22.649391,"time":"2024-01-06T11:14:38.472677129Z","message":"Finished tagging Azure Tier Zero"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":603.239174,"time":"2024-01-06T11:14:39.076410026Z","message":"Finished deleting transit edges"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":0.010463,"time":"2024-01-06T11:14:39.084927722Z","message":"DCSync Post Processing"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":0.007671,"time":"2024-01-06T11:14:39.092261453Z","message":"SyncLAPSPassword Post Processing"}
bloodhound-bloodhound-1  | {"level":"info","time":"2024-01-06T11:14:39.09230606Z","message":"Expanding all AD group and local group memberships"}
bloodhound-bloodhound-1  | {"level":"info","time":"2024-01-06T11:14:39.138389534Z","message":"Collected 61 groups to resolve"}
bloodhound-bloodhound-1  | {"level":"info","time":"2024-01-06T11:14:39.471615261Z","message":"Finished submitting all groups to be traversed. Waiting..."}
bloodhound-bloodhound-1  | {"level":"info","elapsed":392.091685,"time":"2024-01-06T11:14:39.484411527Z","message":"ResolveAllGroupMemberships"}
bloodhound-bloodhound-1  | {"level":"info","time":"2024-01-06T11:14:39.582584825Z","message":"Finished post-processing 6 active directory computers"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":173.376634,"time":"2024-01-06T11:14:39.688252414Z","message":"LocalGroup Post Processing"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":411.779032,"time":"2024-01-06T11:14:40.202253789Z","message":"Finished deleting transit edges"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":0.092729,"time":"2024-01-06T11:14:40.214513257Z","message":"Azure User Role Assignments Post Processing"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":0.011542,"time":"2024-01-06T11:14:40.259564892Z","message":"AZAddSecret Post Processing"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":0.061585,"time":"2024-01-06T11:14:40.265064973Z","message":"AZExecuteCommand Post Processing"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":0.007889,"time":"2024-01-06T11:14:40.272101812Z","message":"Azure App Role Assignments Post Processing"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":148.63538,"time":"2024-01-06T11:14:40.420805158Z","message":"Asset Group Isolation Collections"}
bloodhound-bloodhound-1  | {"level":"info","time":"2024-01-06T11:14:40.420851544Z","message":"Started Data Quality Stats Collection"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":161.364257,"time":"2024-01-06T11:14:40.58222651Z","message":"Successfully Completed Data Quality Stats Collection"}
bloodhound-bloodhound-1  | {"level":"info","time":"2024-01-06T11:14:40.582774165Z","message":"Cache successfully reset by datapipe daemon"}
bloodhound-bloodhound-1  | {"level":"info","elapsed":0.001757,"time":"2024-01-06T11:14:40.582804467Z","message":"Analysis run finished"}

Thank you in advance for your help.

It looks that that Bloodhound went back to their old format.
This issue is related :
https://github.com/dirkjanm/BloodHound.py/issues/84
Current output of Boodhound.py
                {                     "TargetDomainName": "REDACTED",                     "TargetDomainSid": "REDACTED",                     "IsTransitive": true,                     "TrustDirection": 2,                     "TrustType": 2,                     "SidFilteringEnabled": true                 }
Current output of Sharphound
                {                     "TargetDomainName": "REDACTED",                     "TargetDomainSid": "REDACTED",                     "IsTransitive": true,                     "TrustDirection": "Outbound",                     "TrustType": "Forest",                     "SidFilteringEnabled": true                 }
Fix your domains.json yourself for now using this commit : https://github.com/dirkjanm/BloodHound.py/commit/273984883d9ca3dd21f6fca35ec88671cac3fc87
It worked for me.

Check PR https://github.com/dirkjanm/BloodHound.py/pull/160

For anyone who finds this thread and is facing this issue, here is a simple command to use the reverted version identified by ipfyx.
Hopefully the changes get merged soon!
git clone -b bloodhound-ce_fix#157 https://github.com/ipfyx/BloodHound.py.git

Should be fixed since the PR was merged now

This seems to not be working again. With v1.7.2 and BHv6


feat: document cardBrand onChange event property

Description

Adds Elements documentation for new cardBrand onChange event property when using a CardElement.

Testing required outside of automated testing?

[X] Not Applicable

Screenshots (if appropriate):

Rollback / Rollforward Procedure

[X] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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


🐛 BUG: kv - local list output different schema than typical listing

Which Cloudflare product(s) does this pertain to?
KV, Wrangler core
What version(s) of the tool(s) are you using?
building off main
What version of Node are you using?
No response
What operating system are you using?
Mac
Describe the Bug
The recent addition of the --local flag is lovely and I am a big fan. However the format it outputs list operations in is different from that which the non---local list operation uses. Specifically a --local list looks like:
{
    keys: {
        name: string,
        metadata?: string, // note the raw JSON, not parsed value
        expiration?: number
    }[],
    list_complete: true
 }

whereas a standard list looks like:
{
    name: string,
    metadata?: any, // parsed JSON
    expiration?: number
}[]

It'd be nice if they had the same output format so tools building on wrangler don't have to have a special case for parsing local commands.
cc @jspspike
Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
No response

I see, I'll try to put up a PR sometime this week

cool, I can do it if you'd prefer


Change test for no content when parsing response

When parsing the response for a DELETE request to GitHub, I found I had
an empty byte-string instead of an empty string for my response data.
This caused OAuthlib to try and parse it with JSON, which will not work.
Changing the test to allow for any Falsey value will allow empty strings
or byte-strings to return an empty dict.

Merged in https://github.com/lepture/flask-oauthlib/commit/32e799bf795f9e49f7bb85c0262b5af29a0fd9a4


No tests for s_server -WWW/-HTTP

As illustrated by #17231 it seems that we don't have any test coverage for s_server -HTTP or s_server -WWW.

I will take a look for this issue.

Take a look at PR #8962 which (among other things) is introducing some s_server/s_client tests for a different reason. It might be you can re-purpose some of the same code - or at least some of the ideas.

I've hijacked this issue to be more general. We do not actually have any s_client and s_server tests apart from using these by TLSProxy.
We seriously need them.

I've been looking into the feasibility of factoring out the s_server launch code in TLSProxy. The situation is surprisingly hairy to solve in a way that meets the project's portability requirements and doesn't impose an unreasonable burden.
We could simply refactor the current s_server launch code out of TLSProxy, but the existing code doesn't seem safe to me. It scans the output of s_server for an ACCEPT: line to determine what port it's listening on. The problem is that after this it stops reading from s_server's output, which means if s_server generates more output, it may deadlock on writing to its stdout and stop responding to whatever network traffic is being directed on it. Probably this simply hasn't happened to be occurring up until now due to relatively little output produced by s_server. I'm not sure it's desirable to preserve this issue in a refactor though, and in general if we want an ability to launch processes and check their output, which is more important for testing s_client, a better approach needs to be found.
This particular bug may become more of an issue, as for example a recent fix to s_server's KTLS sendfile support causes it to emit a warning per request if KTLS is not available, so this deadlocking could easily happen.
There are third party modules for spawning processes but they don't really offer a solution to the buffering problem. Possibly we should just direct the stdout/stderr from s_server to temporary files so that the output buffers infinitely and the output can be examined easily.
The only concern here is if an application doesn't flush its write buffers immediately when its stdout is a file, but as we control the code of the processes being tested this isn't an issue.
This is probably the most portable solution, especially if we consider VMS support. TLSProxy is currently not supported and skipped on VMS but OpenSSL::Test's run function does support VMS, so there is some desire for VMS support here.


Unable to set linker symbol

This is my app.json. I'm deploying directly by pushing from git but it seems that $SOURCE_VERSION is not being set. It just runs Running: go install -v -tags heroku .
Also, would it be possible to have it set to the latest tag instead of the commit sha1?

Unfortunately git push heroku master doesn't consider app.json in any way.
You need to run heroku config:set GO_LINKER_SYMBOL=main.version for the app in question before you push.
Please let me know if that doesn't work.

Closing since I haven't heard back. Please re-open if the config changes did not solve your issue. Thx.


Added support for multiple code languages

Added support for multiple code languages
the plugin now retrieves the active files comment type from atoms api rather than only working with double forward slash ( // ).

Found bug when non-editor tab is open, or on first open of atom with no active files.
plugin threw an exception running function atom.workspace.getActiveTextEditor().getRootScopeDescriptor()
Caused by atom.workspace.getActiveTextEditor() returning undefined when there is no active text editor object.
Fixed in next commit, and creating a new pull request


Bump golinstor to v0.34.3

A bug in previous golinstor versions meant that responses that
included Volume information always created errors. The broken
golinstor version was only included in 8653ba34f

doesn't actually work, needs more work :/

Nevermind, I was just using test data I modified to pinpoint the issue, the change itself is fine.
@rck sorry for the noise, this is ready for review


fix: PushSamples issue.

fix: PushSamles
To ensure that a variable holds a specific value rather than a pointer value.
Before:

After:

thank you @shardingHe


Markdown preview UX

The markdown preview is currently laid out like the old Stack Exchange editor's preview - editor on top, preview on bottom. We should look into a way to improve the UX here, potentially making it an expandable pane, making it configurable, etc.

@b-kelly I am happy to take the lead on this, I just need some guidance on how to get started.
I am assuming:

Expandable pane for preview section, with internal scrolling
Button on editor panel to toggle it on/off (ie: even when turned on in the editor, that just makes the button visible).
Save value of button in cookie (for starters, can be user pref at a later date)

Questions:

Configurable - what types of configuration options where you thinking of? I thought it is just on or off.
We'd need a new icon for a button, right?
Do you view these integration steps as something to go into the core editor, or into a plugin?

There needs to be some investigation here. There's on/off ofc, but also things like "always shown", side-by-side vs above-below, etc. These probably wouldn't be included in the first pass, but it is something we'd want to think about
I don't know if we even want to do a button - adding a new button to the menu on non-mobile causes some scrolling, so we'd potentially want to collapse other items into a menu to make room for this

Additionally, adding a new button would cause some menu jumping when toggling between modes

Whatever we decide on for options, that support would go into the editor. As far as "how to store the preference" (e.g. cookie), that should not go into the editor codebase

As a side note regarding using a cookie, that would have some bureaucratic overhead as we'd need to get the cookie categorized for GDPR and other legal mumbo-jumbo.

@yellis Still planned for Beta 2 or should we push this out to a different milestone?

Pushing out to 2.5 then so we can cross off all the boxes for Beta 2.


Performance drop after the latest update

Hi, I took the last update and restarted the game and I did not leave the feeling that the game began to go slower. To confirm my feelings, I turned on the TicksPerSecond and Faster Dev Speed mods (https://steamcommunity.com/sharedfiles/filedetails/?id=1579446607 and https://steamcommunity.com/sharedfiles/filedetails/?id=1579958166) and started playing at 4x speed with just:
Core
HugsLib
Rim War
Faster Dev Speed
TicksPerSecond

Measurements showed that by eye with the previous version (Sep 28, 2019), the average TPS is 2100 TPS. With the latest version (Oct 3, 2019), the average TPS is 1700 TPS.
Previous version:
Rim War.zip
Latest version:
Rim War.zip

In the latest version, with default settings, the average TPS is 1700. However, when changing settings to performance, the average TPS is 2100. I think the performance option solves the performance problem.


CustomPainter & BlendMode.multiply clipping artifacts

I am trying to mask one rectangle with another one using CustomPainter with BlendMode.multiply. When I embed it in ListView (also happens when embedded in GridView or PageView) then I see strange white artifact on top of the list and on left border of masked rectangle:

Steps to Reproduce
Run following code:
void main() => runApp(new MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return new MaterialApp(
      title: 'Flutter Demo',
      home: new Container(
        color: new Color.fromARGB(255, 128, 128, 128),
        margin: new EdgeInsets.all(50.0),
        child: new ListView.builder(
          itemBuilder: (context, index) {
            return _buildRow();
          },
          itemCount: 50,
        ),
      ),
    );
  }

  Widget _buildRow() {
    return new Container(
      height: 100.0,
      child: new CustomPaint(
        painter: new MaskingPainter(),
      ),
    );
  }
}

class MaskingPainter extends CustomPainter {
  @override
  void paint(Canvas canvas, Size size) {
    final Rect rect = new Rect.fromLTRB(
      5.0,
      5.0,
      size.width - 10.0,
      size.height - 10.0,
    );

    canvas.drawRect(
      rect,
      new Paint()..color = new Color.fromARGB(255, 255, 255, 255),
    );

    canvas.drawRect(
      rect,
      new Paint()
        ..blendMode = BlendMode.multiply
        ..color = new Color.fromARGB(255, 128, 0, 0),
    );
  }

  @override
  bool shouldRepaint(CustomPainter oldDelegate) {
    return true;
  }
}

Reproducable on Xiaomi Redmi4A
Flutter Doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, v0.2.11, on Mac OS X 10.12.6 16G29, locale en-PL)
[✓] Android toolchain - develop for Android devices (Android SDK 28.0.0-rc1)
[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
[✓] Android Studio (version 3.1)
[✓] VS Code (version 1.22.2)
[✓] Connected devices (1 available)
• No issues found!

Flutter version seems a bit outdated. Can you try with a newer version?

Can't reproduce in
[✓] Flutter (Channel master, v0.5.8-pre.142, on Mac OS X 10.13.6 17G65, locale en-AT)
• Flutter version 0.5.8-pre.142 at /Users/zoechi/flutter/flutter
• Framework revision f72c514c59 (27 hours ago), 2018-07-26 05:08:11 -0700
• Engine revision 95e91e31fa
• Dart version 2.0.0-dev.69.0.flutter-937ee2e8ca
[✓] Android toolchain - develop for Android devices (Android SDK 28.0.1)
• Android SDK at /usr/local/opt/android-sdk
• Android NDK at /usr/local/opt/android-sdk/ndk-bundle
• Platform android-28, build-tools 28.0.1
• ANDROID_HOME = /usr/local/opt/android-sdk
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
• All Android licenses accepted.
[✓] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 9.4.1, Build version 9F2000
• ios-deploy 1.9.2
• CocoaPods version 1.5.3
[✓] Android Studio (version 3.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 23.1.2
• Dart plugin version 173.4700
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
[✓] IntelliJ IDEA Ultimate Edition (version 2018.2)
• IntelliJ at /Applications/IntelliJ IDEA.app
• Flutter plugin version 26.0.3
• Dart plugin version 182.3569.4
[✓] Connected devices (1 available)
• Android SDK built for x86 • emulator-5554 • android-x86 • Android 8.1.0 (API 27) (emulator)
• No issues found!
Can this be closed?

Thanks, please close this.


syntaxcomplete ignores buffer-local ignorecase setting

Here the setting is read:
https://github.com/vim/vim/blob/b98cec28d95b2184c64a0646458c1e62deb2524b/runtime/autoload/syntaxcomplete.vim#L150-L156
But then when it should be used, it reads it from the global variable:
https://github.com/vim/vim/blob/b98cec28d95b2184c64a0646458c1e62deb2524b/runtime/autoload/syntaxcomplete.vim#L201-L209
Note that in the commented part just above it uses b: but then in the real code it uses g:.

ping @dfishburn

Thanks for the bug report, I will address it.

I did quite a bit of testing on this, but if you could give it a quick run through before I upload it that would be good.
The upload  will expire 20200503.

For my use case I can confirm it works now.

let me close this and I let David handle sending a new file to Bram for inclusion :)

@dfishburn thanks for the fix by the way.


docker refactor

I've reworked the Docker images a bit to get them up to date. Here's a summary of my changes:

Upgraded dlib to v19.21
Upgraded the GPU base image to use cuda 11 and cudnn8 (nvidia/cuda:11.2.0-cudnn8-devel)
I. Its has been supported by dlib for around ~6 months
II. There are several performance gains from this upgrade.
Implemented multi-stage builds to shrink image sizes.
I. The CPU image went from ~2GB to 350MB
II. The GPU image went from ~7GB to ~4GB
Added a Kubeflow Jupyter image for easy development on platforms like GCE.

I've setup some autobuilds to keep these images up-to-date. I'm currently hosting these images on my docker hub. I would be happy to help set it up on another account as well.

Thanks!


Makefile: --use-mirrors has been deprecated

The --use-mirrors option should be replace by something other, because with pip 6.0.8 there is the following deprecation warning:
DEPRECATION: --use-mirrors has been deprecated and will be removed in the future. Explicit uses of --index-url and/or --extra-index-url is suggested..
And with new version of pip like 7.0.3 it ends with an error: no such option: --use-mirrors.

Now fixed in dev. Ta.


feat: set monolog.extra

https://github.com/getsentry/sentry-php/issues/1318#issuecomment-1129865484

Duplicate of #1244, already merged in develop.

Hello @Jean85 I checked the PR and does not seem to read $record['extra']. Can you review?

We are not reading extra purposefully, as it's meant to be used by processors and we absolutely want to avoid logging anything blindly as it may result in exceptionally big messages

The main difference is that context can be supplied in user land (it is the 3rd parameter to Psr\Log\LoggerInterface methods) whereas extra is internal only and can be filled by processors. The reason processors write to extra and not to context is to prevent overriding any user-provided data in context.

The extra values are provided by processors, but they are provided with the intention of being logged. See

ProcessIdProcessor or
HostnameProcessor or
MemoryUsageProcessor or
MemoryPeakUsageProcessor

You are right. I have no opinion against implementing support at this point, as long as logging is dependent on the fillExtraContext flag, so that in the worst case the user can still turn it off. @ Jean85 WDYT?

@ste93cry in that case the PR should be good as is
@JanTvrdik thanks for input

Because the branch got deleted, I cannot reopen this PR. If you can't too, please open a new one

mb. https://github.com/getsentry/sentry-php/pull/1330


Sideloading APK fails on Android 6.0.1

I downloaded the APK from here and tried to install on my Android 6.0.1 tablet.
This is the error I receive:
Failure [INSTALL_PARSE_FAILED_NO_CERTIFICATES]

OK, will take a look. I think I have to sign the APK will the older signature style...

OK, will take a look. I think I have to sign the APK will the older signature style...

Just updated the APK on the download page to v1.0.1 with many improvements and using the older signature system, it's on the download page now. Let me know if that works.

Just updated the APK on the download page to v1.0.1 with many improvements and using the older signature system, it's on the download page now. Let me know if that works.

Confirmed working. Thanks.


Don't require a name in tye.yaml and default to the current directory

We currently validate that a name is present in the tye.yaml, however a good default would be the directory name of the current directory (ToLower).

I'm pretty sure we actually don't validate this. I think the work that's needed will be to update the schema.


blockages on larynx

when I validate Larynx as a TTS interface I have a sign indicating that addons must be downloaded .... but it remains blocked !!!
rhasspy/fr_larynx-siwis/checkpoint_270000.pth.tar
rhasspy/fr_larynx-siwis/config.json

rhasspy/fr_larynx-siwis/vocoder/checkpoint_450000.pth.tar
rhasspy/fr_larynx-siwis/vocoder/config.json
the download is not done !!!

Please try again in 2.5.9 and re-open the issue if you still have problems. I went through all of the download links before the update.

Please try again in 2.5.9 and re-open the issue if you still have problems. I went through all of the download links before the update.


return string response instead of writing always to file system

when using spectaql as a dependency in a node project we do not get a stringified response of the html document generated. This makes it difficult to integrate in an api.
can we have some feature similar to @asyncapi/generator where we have possbility to get string response instead of getting a HTML file saved to file system always?

yeah makes sense for embeddable mode or if we have inline or no css and JS.
This feature would allow me to generate documentation for multiple different introspection.json on the fly without having to always read the output HTML file .

OK, this will be coming soon:
https://github.com/anvilco/spectaql/pull/497
The run function will return an object {html: '<your html here>'}, which you can do what you please with.
I'll let you know when it's been released.

@parmeet22 this is now in main and available in v1.5.4


[pytorch] Fixes model loading bug for 1.11.0

Change-Id: Ic08086214a5826d10ca2f40b61a7b56cd7b21a97
Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report

Merging #1705 (1e85e81) into master (bb5073f) will decrease coverage by 1.63%.
The diff coverage is 69.50%.

@@             Coverage Diff              @@
##             master    #1705      +/-   ##
============================================
- Coverage     72.08%   70.45%   -1.64%
- Complexity     5126     5517     +391
============================================
  Files           473      523      +50
  Lines         21970    24446    +2476
  Branches       2351     2646     +295
============================================
+ Hits          15838    17224    +1386
- Misses         4925     5913     +988
- Partials       1207     1309     +102

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <ø> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

...nslator/InstanceSegmentationTranslatorFactory.java
14.28% <0.00%> (-3.90%)
:arrow_down:

.../ai/djl/modality/cv/translator/YoloTranslator.java
8.33% <0.00%> (-0.50%)
:arrow_down:

.../modality/cv/translator/YoloTranslatorFactory.java
8.33% <0.00%> (-1.67%)
:arrow_down:

...i/djl/modality/cv/translator/YoloV5Translator.java
5.38% <0.00%> (-0.31%)
:arrow_down:

... and 287 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 49d7bd8...1e85e81. Read the comment docs.


Saving Texture3D resource through script does not work in Godot 4

Godot version
4.0.beta.3
System information
Vulkan, Windows 10 GTX970m
Issue description
Saving Texture3Ds from script is does not work in Godot 4. Someone apparently already fixed it but there's no pr of it yet it seems: https://github.com/Game3DEE/godot/commit/68bb829d49afcd4f0bd6516abbed84f1a27df61f
Edit: Turns out there's already a pr here: https://github.com/godotengine/godot/issues/66558 but it's not merged yet.
Steps to reproduce
First declare an array of type Image to hold each layer of the 3D Texture.
Initialize the Image array in a triple nested for loop where the outermost loop iterates through each each Image creating a new Image for the current layer, and the two inner loops iterate through each Image's x and y pixel coordinates setting the pixel at those coordinates.
Then, create an ImageTexture3D by first declaring ImageTexture3D.new() and then initializing it with ImageTexture3D.create() setting the format to Image.FORMAT_RGBA_8, its dimensions and passing in the image layer array to it.
Finally save the 3D texture using ResourceSaver.save(). Saving as tres outputs a 1x1x1 pink texture and saving as png or tga causes the saver to not write anything to the disk.
Minimal reproduction project
Create a new empty project and follow the steps to to reproduce.

Beta 3, really? There's an RC already out, double check if the issue still exists.

It's still not working on the latest snapshot beta 17

Do you mean PR #71394?

Yes that's it.

Duplicate of https://github.com/godotengine/godot/issues/66558.

I'm trying to understand the process of creating the data array for the image.

First declare an array of type Image to hold each layer of the 3D Texture.

Initialize the Image array in a triple nested for loop where the outermost loop iterates through each each Image creating a new Image for the current layer, and the two inner loops iterate through each Image's x and y pixel coordinates setting the pixel at those coordinates.

Could someone help me with this part? Before reading this, my code looked like this:
var arr = []
	for x in range(-bounds,bounds):
		arr.append([])
		for y in range(-bounds,bounds):
			arr[x+bounds].append([])
			for z in range(-bounds,bounds):
				var loc = Vector3(x,y,z)
				var item = get_cell_item(loc)
				if item == 0:
					# cell is full
					arr[x+bounds][y+bounds].append(Color(1,1,1,1))
				elif item == -1:
					arr[x+bounds][y+bounds].append(Color(0,0,0,1))

How do I create an array of type image? And is this the correct loop arrangement?

The array should be used to store individual images. The outer most loop appends a new image to the image array. Then an inner loop iterates through each row of the current image and then the innermost loop iterates through each column of that current row. Once done you can create a new Texture3D passing along the array of images to it.

A Texture3D is just an array of 2d images. The array should be used to store individual images. The outer most loop appends a new image to the image array. Then an inner loop iterates through each row of the current image and then the innermost loop iterates through each column of that current row setting each pixel of that image. Once done you can create a new Texture3D passing along the array of images to it.

Thank you! This solved my problem, here is the new code:
func make_array_from_gm():
	var arr = []
	for z in range(-bounds,bounds):
		var arrimg = Image.create(2*bounds, 2*bounds, false, Image.FORMAT_L8)
		for x in range(-bounds,bounds):
			for y in range(-bounds,bounds):
				var loc = Vector3(x,y,z)
				var item = get_cell_item(loc)
				if item == 0:
					# cell is full
					arrimg.set_pixelv(Vector2i(x+bounds,y+bounds), Color.WHITE)
				elif item == -1:
					arrimg.set_pixelv(Vector2i(x+bounds,y+bounds), Color.BLACK)
		arr.append(arrimg)
	return arr

I believe this issue deserves re-opening, as PR #71394 does not fix the problem for ImageTexture3D in 4.1.1 (a278c1b)
The following code should produce a tres file containing the data. Instead, it produces an tres file without data:
var i := Image.create(1, 1, false, Image.FORMAT_RGB8)
i.set_pixel(0,0, Color(1,0,0))
var l := ImageTexture3D.new()
l.create(Image.FORMAT_RGB8, 1, 1, 1, false, [ i ])
ResourceSaver.save(l, "res://texture3d.tres")

The equivalent for Texture2DArray works fine:
var i := Image.create(1, 1, false, Image.FORMAT_RGB8)
i.set_pixel(0,0, Color(1,0,0))
var l := Texture2DArray.new()
l.create_from_images([ i ])
ResourceSaver.save(l, "res://layered.tres")

I'm not sure if this is really how I'm meant to save as I am still fairly unfamiliar with godot, but it does feel like there is an issue there.

Yeah, I agree. I'm thinking of contributing a fix if no one gets to it yet.

Can confirm that ImageTexture3D instance generated from code can't be saved into tres file

I believe this issue deserves re-opening, as PR #71394 does not fix the problem for ImageTexture3D in 4.1.1 (a278c1b)
The following code should produce a tres file containing the data. Instead, it produces an tres file without data:
var i := Image.create(1, 1, false, Image.FORMAT_RGB8)
i.set_pixel(0,0, Color(1,0,0))
var l := ImageTexture3D.new()
l.create(Image.FORMAT_RGB8, 1, 1, 1, false, [ i ])
ResourceSaver.save(l, "res://texture3d.tres")

The equivalent for Texture2DArray works fine:
var i := Image.create(1, 1, false, Image.FORMAT_RGB8)
i.set_pixel(0,0, Color(1,0,0))
var l := Texture2DArray.new()
l.create_from_images([ i ])
ResourceSaver.save(l, "res://layered.tres")

I'm not sure if this is really how I'm meant to save as I am still fairly unfamiliar with godot, but it does feel like there is an issue there.

Can confirm that ImageTexture3D generated from code can't be saved into valid tres/res file.
File generated as tres just be like:
`[gd_resource type="ImageTexture3D" format=3 uid="uid://1brwgfcw8yvu"]
[resource]
`
Using godot 4.1.1 bd6af8e0e

This is a duplicate, please see:

https://github.com/godotengine/godot/issues/66558


Dragging causes logging in console

Noticed that current version 4.3.0 will log 'DraggableCore' stuff in console while dragging element.

4.3.0 also not works with grid-layout, took my half day to find out, seems the style not pass down to child. pin to 4.2.0 solved

Woof. Thanks all, unpublishing and will fix this in 4.3.1 soon (i.e. not on a holiday).

The curiosity was killing me. Both issues were quite strange (cache invalidation and what appears to be a bug in React.cloneElement). They are both fixed in 4.3.1.
Unfortunately I couldn't pull 4.3.0 as something is depending on it, but I've added a deprecation message.


Unable to update Office with Windows 10 multi-session image with ProPlus

I have used the Windows 10 image with ProPlus and got out of the box Office applications installed on all my sessions hosts. But the user is unable to configure any office applications or even update Office on those session hosts.

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

ID: d2eeca14-80ef-c0b4-2719-80ceb286ba3d
Version Independent ID: b7d3df23-3158-2807-eecc-73e7230c2033
Content: What is Windows Virtual Desktop? - Azure
Content Source: articles/virtual-desktop/overview.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

Thanks for the feedback! We are currently investigating and will update you shortly.

Please take a look at the page: https://docs.microsoft.com/en-us/azure/virtual-desktop/install-office-on-wvd-master-image which covers off the installation of office and confirm via the necessary policies/registry values as to the current activation method.
Here's another link on SCA: https://docs.microsoft.com/en-us/DeployOffice/overview-of-shared-computer-activation-for-office-365-proplus which may help.
Are they seeing anything in the event logs on the host by any chance?

Is there any update?

We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


refactor: Refactor the code and remove the examples

I have read the CLA Document and I hereby sign the CLA


typos; add train_batch_size back to base config

Does train_enwik8.py not run for you if train_batch_size is not included in base_model.json?

/gpt-neox/src/deepspeed/deepspeed/runtime/config.py", line 721, in _set_batch_related_parameters     'Either train_batch_size or micro_batch_per_gpu needs to be provided' AssertionError: Either train_batch_size or micro_batch_per_gpu needs to be provided
is what it throws for me

Superseded by codebase refactoring.


fix hardcode namespace in 2.1 branch

Signed-off-by: yahliu<EMAIL_ADDRESS>

For consistency between releases, would it be okay for us to close this PR and I'll cherry-pick when the change was originally made (https://github.com/open-cluster-management/grc-ui/pull/577) down to this release? Is this a change that also needs to be in 2.2?

Ah, especially since 2.2 and 2.1 can't accept PRs from external forks since the builds are in Travis and not Prow. I'll do the cherry-pick now and close this PR.

Sorry--I wasn't thinking. The cherry-picks conflict because of the move to Cypress. 🤦  Reopening and approving. 🙂

The build failed due to
$ npm ci
npm ERR! Error while executing:
npm ERR! /usr/bin/git ls-remote -h -t<EMAIL_ADDRESS>npm ERR!
npm ERR! Warning: Permanently added the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts.
npm ERR! Permission denied (publickey).
npm ERR! fatal: Could not read from remote repository.
npm ERR!
npm ERR! Please make sure you have the correct access rights
npm ERR! and the repository exists.
npm ERR!
npm ERR! exited with error code: 128
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/travis/.npm/_logs/2021-06-15T03_02_39_590Z-debug.log

@dhaiducek, could u help to take a look?

seems the build failed due to the fork repo, may need to re-create the PR for 2.1  branch.

@dhaiducek  I already applied request to join the team. Can you or someone approve it ? or you can help to make the PR for this change ?

Sorry, I hadn’t merged it because I was thinking there was a code freeze, but that’s for 2.2. Merging now.


[BUG] Add proper error message for mono TrueHD encoding

Description of the the bug
deew tries to encode mono TrueHD
Version
Latest
OS
Every

And 44.1k too. Even though ffmpeg can encode that, no problem.
ffmpeg -h encoder=truehd
General capabilities: delay small exp
Threading capabilities: none
Supported sample rates: 44100 48000 88200 96000<PHONE_NUMBER>00
Supported sample formats: s16 s32
Supported channel layouts: stereo 5.0 5.1

BTW, 44.1 is supported by HW bitstream players like LG C9 internal speackers, no problem.

from the readme:

automatic sample rate conversion using ffmpeg's soxr resampler in case of an unsupported sample rate

for DD/DDP unsupported rates will be converted to 48 kHz
for TrueHD unsupported rates will be converted to 48 kHz if source sample rate is lower than 72 kHz, otherwise will be converted to 96 kHz

DEE doesn't support anything else other than 48kHz for DD and DDP and only 48 and 96kHz for TrueHD

Dolby Reference player supports playing mono with this script, just change truehddec-presentation=2 or 6 or 8 instead of 16
https://gist.github.com/Plazik/e950267cc580bcde371ea7879aaa3d5d


New Test Grid for Simona

Resolves : #753
New Grid
CHANGELOG.md

Some first points, have still to check this more detailed:

when publishing a branch the CI/CD pipeline is checking for a certain naming pattern <initials>/#<issueId>-<description> looks like you missed a # infront of the issue number. Thus the pipeline couldn't be build. Maybe you push it again with corrected branch name.
maybe you can adapt the coordinates for the line that is still not matching with distance of its coordinates

Note for self: Still have to check results.

Okay, I'll check that right away. I want to solve the issue with the cable :).


filesListFolderLongpoll() causes CORS preflight error

XMLHttpRequest cannot load https://notify.dropboxapi.com/2/files/list_folder/longpoll.
Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource.
Origin 'http://localhost:8080' is therefore not allowed access. The response had HTTP status code 501.

Tested on macOS Chrome (54.0.2840.71 (64-bit)), Safari (10.0.1 (116<IP_ADDRESS>.7)) & Firefox (49.0.2).
This seems unrelated to #42 (or associated issues) because domain is notify.dropboxapi.com not api.dropboxapi.com.
Can also be reproduced with following XHR request:
var xhr = new XMLHttpRequest();
xhr.open("POST", "https://notify.dropboxapi.com/2/files/list_folder/longpoll");
xhr.setRequestHeader("Content-Type", "application/json");
xhr.send(JSON.stringify({cursor: "AAG9UMjHQxjlfNX_rOPsVO0W27HIkPlD-xBu0rETcMhuWsprxpgE-94NVr29SIgJfGlFokxPbJSuTG6ryGD01d_amqnijqi0fcXp2cdkYeq63krcuD9jKVeU4kbBNSLfXzswS-NXCJJF1IA_AUeL_h4ZKFyuJIuVRQNodUu8EOo0Vw"}));
// I assume you'll need to provide a cursor value that matches your own account

It seems it is the Content-Type "application/json" header that is triggering the CORS prefilght. The suggested Content-Type "text/plain;charset=dropbox-cors-hack" hack doesn't work as all 3 browsers I tested rewrite to "text/plain" which result in a 400 Bad request.

Thanks! We'll look into it.

I'm also having what I think is this issue - same errors as above:
OPTIONS https://notify.dropboxapi.com/2/files/list_folder/longpoll 501 (Not Implemented)
XMLHttpRequest cannot load https://notify.dropboxapi.com/2/files/list_folder/longpoll. Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost' is therefore not allowed access. The response had HTTP status code 501.
When I then installed "Allow-Control-Allow-Origin: *" Chrome plugin as a test, longpoll still returns:
XMLHttpRequest cannot load https://notify.dropboxapi.com/2/files/list_folder/longpoll. Response for preflight has invalid HTTP status code 501

@MaxwellK For chrome and firefox use the Fetch API.
For IE use a fetch polyfill.
For Safari use a different api method or longpoll implementation

@grazinskis what exactly do we need to do for Safari? Thanks!

I don't understand what "for IE use a fetch polyfill" means.  Can you give me a pointer?
This looks like it was reported four months ago and is still not working?  Am I missing something? @greg-db

I just got lonpoll working by hacking the .min version from distribution (since non-min does not work in a browser) to effectively change rpcRequest() to do apiRequest.type("text/plain; charset=dropbox-cors-hack") and apiRequest.send(JSON.stringify(body))
This apparently avoids the CORS preflight, thus avoiding the 501 response from the server to the preflight.
I hope this can make it into a distribution soon (or a fix to the server accepting preflight).  I'm trying really hard to get off of v1 before it's shutoff in June!

@episodeyang I have come up with a couple ways of dealing with it:

The first method described here (number 1)
Periodically call filesListFolderContinue() with your latest cursor and check how many changes it reports.

It's ugly though. Amazing for how long this issue is still relevant. Let's hope the fix of @kennellink works, doesn't break anything else and can make it to distribution.

Thanks for following up, everyone. This issue is still open with the team on our side. I'll follow up here once we have an update.

Thanks @episodeyang.  My hack does not work in Firefox and perhaps others since they rewrite the content-type.  Works in IE.

Hey @kennellink and others, we are in the midst of a backend rewrite that will fix this for free, but I'm unable to give you an eta yet. Give me a couple of days for an ETA.
Thanks a lot for the patience, and thanks for reporting.

Thanks @pran1990, I appreciate the help!  Can I look forward to having a working filesListFolderLongpoll() (with some time to test) before the June deadline to be off API v1?

I don't have a timeline for a fix right now. I'll check in with the team again.

@greg-db Thanks! :)

Hi everyone.  I appreciate the work you're doing here.  We have a deadline, however, from Dropbox to be off API v1 by June and I am in dire need of filesListFolderLongpoll() to work before then.  Do you know if Dropbox will extend that deadline or if Longpoll is expected before then?  I'm sorry to push, but we have an app in production that our customers rely on and I need a plan to provide uninterrupted service.
I'm afraid the active polling technique would trigger throttling, so I don't have an alternative at this point.

@kennellink Thanks for following up, and apologies for the delay. Work on this is ongoing, but it requires some infrastructure changes to do right. I can't offer a guaranteed timeline, but we do expect this to be done this month, well ahead of the v1 retirement.

Thanks @greg-db ! It's really good to hear.

Thanks @greg-db that sounds great!  Best wishes.

Hey @greg-db and @pran1990 , I hope all is well.  I just received a reminder from Dropbox that the v1 API is going away soon.  I have about a month, which is not long enough to completely rewrite to another SDK.  Is this SDK going to have longPoll working in the near near future so I can finish testing and finalize the upgrade?  If not, do you know who I would contact at Dropbox to extend access to v1 API or if they'll even entertain the idea since I've been depending on this SDK working?

Hey @kennellink, thanks for reaching out! I'm in the process of testing the
new backend, I'll update this thread when that is over. The change on your
side will be minimal (upgrading the SDK) at this point.
On Wed, May 24, 2017 at 9:42 AM, kennellink<EMAIL_ADDRESS>wrote:

Hey @greg-db https://github.com/greg-db and @pran1990
https://github.com/pran1990 , I hope all is well. I just received a
reminder from Dropbox that the v1 API is going away soon. I have about a
month, which is not long enough to completely rewrite to another SDK. Is
this SDK going to have longPoll working in the near near future so I can
finish testing and finalize the upgrade? If not, do you know who I would
contact at Dropbox to extend access to v1 API or if they'll even entertain
the idea since I've been depending on this SDK working?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dropbox/dropbox-sdk-js/issues/85#issuecomment-303782039,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE2ScMBpANk5RhDaLZvxP26-wOVBO2V5ks5r9F4LgaJpZM4Kq9jB
.

--
Pranesh P
http://linkd.in/pran1990

We are reviewing requests for extensions to API v1 access though. If you'd like an extension for access to API v1, please open an API ticket:
https://www.dropbox.com/developers/contact
Please include:

The app key(s) for the app(s) you need an extension for.
The reason(s) for the extension.
How long of an extension you're asking for.

Longpoll still 501 (Not Implemented) -  any ETA please?
Would be really nice with some advance against the v1 deprecation ...

I don't have an update on this yet unfortunately. We'll follow up here once we do.

This is really critical for our applications. Glad that you postponed the depreciation date, but we would love to see this implemented in new API.

Hi pavol, when our fix for this issue is ready, I expect it to be a noop
(except for removing workarounds). That fix is not tied to the deprecation
date.
On Jun 29, 2017 4:49 AM, "Pavol Rusnak"<EMAIL_ADDRESS>wrote:

This is really critical for our applications. Glad that you postponed the
depreciation date, but we would love to see this implemented in new API.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dropbox/dropbox-sdk-js/issues/85#issuecomment-311943422,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE2ScM95LOEijYvoRQdXlrHpjQMN7MCHks5sI485gaJpZM4Kq9jB
.

So filesListFolderLongPoll was reported broken 9 months ago, is not fixed, and has no fix in sight.  There is no unit test for it, so it's not clear if it was ever tested or if it ever worked.  Am I missing something?  As someone new to this SDK and assuming it was generally stable, that's pretty disconcerting.

API v1 cutoff date until September 28th, 2017 ???
great, just still missing a few basic functions! Please, fix it at least a few weeks before deadline, we need to test our apps as well before cutoff.

Work on this is still in progress. If you need an extension for access to API v1 please refer to this comment for instructions for requesting one.

This should be working now. Please let us know if you're still seeing any issues.


New Enum in Asset Type

An applicant of a mortgage can bring unpledged building plot as part of his assets (Eigenmittel). This enum is missing in the assetType.
Please add the enum: unpledged_building_plot

Not sure if this is a "mainstream" thing. AssetType "other" is available.

Unpledged Building Plot is not mainstream. Can be reality, though. The more precise, the better.
AssetType "other" is not available. see Code line 2677-2694

Agreement MP / TKB on 29.09.22: Ok


Hugepage support for LCOW

This adds hugepages support for LCOW.
A hugetlbfs drive can be mounted inside a LCOW. This is backed by a location in UVM. Currently, linux kernel shipped with containerd only supports hugepage size of 2M.
In order to specify a container path is a hugepage mount, hostpath needs to be specified as hugepages://2M.
In order for container to use hugepage successfully, it is expected UVM will be created appropriate KernelOptions annotation.
"io.microsoft.virtualmachine.lcow.kernelbootoptions" : "hugepagesz=2M hugepages=10"
UVM also has to be created with sufficient memory to host desired number of hugepages.
Ex:
"io.microsoft.virtualmachine.computetopology.memory.sizeinmb": "2048",
"io.microsoft.virtualmachine.lcow.kernelbootoptions" : "hugepagesz=2M hugepages=10"
In order to realize the performance of using hugepages inside the LCOW, memory backing the UVM has to ensured to use an approriate page size. The default page for physical memory backed UVM is 2MB. Hence, to ensure performance with using hugepage of size MB, UVM should be physcial memory backed.
"io.microsoft.virtualmachine.fullyphysicallybacked": "true"
Verification: Adhoc testing using updated containerd binaries/linux kernel /container image with a test app that tries to huge page.
Newly added functional test also passes.

Oddly enough some of the bridge gcs tests are failing

Oddly enough some of the bridge gcs tests are failing here

Kathryn, reran the tests, and they passed.

Oddly enough some of the bridge gcs tests are failing here

They are passing now, after pulling in latest changes.

This needs to get rebased to get Kathryns recent fixes

Still lgtm


Received SMS wont display on Serial Monitor

Hi, thanks for this awesome library of yours. I've tried this, everything's working fine. But the only problem is that, the text messages that I send to GSM A6 wont display on the serial monitor. Do you happen to encounter this problem? Thank you.

just by changing storage location of SMS msg (line 72 of A6lib.cpp), I'm now able to receive SMS
    if (A6_OK != A6command("AT+CPMS=SM,SM,SM", "OK", "yy", A6_CMD_TIMEOUT, 2, NULL))

@1technophile Thank you for the update! @vanjsy, does that work for you?

Hi @skorokithakis , I didn't try it yet. Once I did I'll update you if it works, thank you.

I dont have my ESP8266 with me though, but will this library works on Arduino Uno? If so, which part will i need to modify? Thanks.

As far as I know, it works unmodified. Try the suggestion above to see if it works.

Even using Arduino Uno?

Yeah, it should work, but I haven't tried it personally.

Hello, live message still not displaying when texting the GSM using my mobile.

I know, please don't post messages like that.

Like what sir? Apology.

Comments that don't add any new information.

Here are some useful info about SMS Storage area: https://www.developershome.com/sms/cpmsCommand.asp
As @1technophile mentioned, you should set Preferred Storage to SIM (mine can handle about 15 and don't know why ME doesn't work!).maybe Preferred Storage should be default to SM or control by some Macros.
And remember that SMS index for A6 & A6 mini start at 1!
This lib definitely needs some modification to remove blocking operations, enabling async and callbacks , ...

you should set Preferred Storage to SIM

I am a bit wary of doing that because I remember my module working with ME but not with SM, and I don't have my A6 to test with, so I can't verify that this fixes things for more people than ME... There's a comment in the code to change that if it doesn't work, but I agree that it would be better as a parameter. Can someone create a PR with storage selection, perhaps?

And remember that SMS index for A6 & A6 mini start at 1!

Is this currently set to 0 somewhere? I didn't find any mentions, but I'd appreciate someone pointing out any bugs.

This lib definitely needs some modification to remove blocking operations, enabling async and callbacks , adding UTF8 etc.


FIX: imp module deprecated

(cherry picked from commit cd8c3ad2a2b62ad71f4eb2cc187ce6c59a348ee9)
@atlanswer I have cherry picked your PR to trigger the CICD. Thank you for your contribution
Problem: The imp module has been deprecated since version 3.4 and removed in 3.12^1.
It's used in #2949:
https://github.com/ansys/pyaedt/blob/da0f3f4070070d6da5b480369160f9608803c7df/pyaedt/emit_core/__init__.py#L25
Changes: Replace the usage of imp.find_module() according to the official guide^2.

Oh, I missed #4762. Nice to see that 3.12 support has been worked on.


Document supported games

Document games verified as functional.

Wiki started for this: https://github.com/HazeCrate/HazeCrate/wiki/Verified-Games


Fixing a java error.

Was picked up by our internal java compiler: a proto returned value
cannot be null.

LGTM

LGTM