Unnecessary installed files with NPM : Need NPM whitelisting

Nice package, very useful :bowtie:
But unnecessary files are installed during npm install :

bin/changelog
test/...
.jshintrc
.travis.yml

I see that you have an .npmignore file with this inside :
node_modules
.DS_Store

That's pretty useless as those files are ignored by default by NPM
I suggest whitelisting instead of blacklisting files to be installed by adding files to the package.json :
{
  "files": [
    "lib/",
    "decode.js",
    "index.js",
    "sign.js",
    "verify.js"
  ]
}

Please read documentation : https://docs.npmjs.com/files/package.json#files
Also remove .npmignore file
Some of us are using microservices (AWS Lambda, Azure Function, etc.) and we can benefit having small packages with only production files. Thanks. I maybe PR with that.
PS: And rename bin/changelog to scripts/changelog.js as bin is usually used to distribute bin files

Okay here is the PR https://github.com/auth0/node-jsonwebtoken/pull/245

But unnecessary files are installed during npm install :

I accept your opinion but I am against this kind of optimization in the repositories that I maintain.
The short answer is that I happen to use the things I maintain and this kind of optimization doesn't worth and it makes developers life harder.
Long answer:
These files sum a total of 120k and gzipped is 11k. This is nothing in term of band witch and storage nowadays.
Removing this mean that I can't check the "tests" or the CHANGELOG locally. Instead I have to:

go to my package.json to check what version of the library I am using
then go to the repository (or github) and select the version I want to get info

which is very inconvenient when you are developing in node.js.

@jfromaniello  This is not an optimisation, this is first of all about semantics :
source code !== distribution
Please have a look at some packages like :

babel-cli with its .npmignore
expressjs with its package.json file
bluebird with its package.json file
request with its .npmignore
etc.

I didn't add CHANGELOG.md in file by purpose as npm adds it by default:

Certain files are always included, regardless of settings:
package.json
README (and its variants)
CHANGELOG (and its variants)
LICENSE / LICENCE

Tests are for contributors (people that are cloning your repo).

The short answer is that I happen to use the things I maintain

All those files will still be in your repo and in your tags/releases..

this kind of optimization doesn't worth and it makes developers life harder.

Think about microservices (AWS Lambda, Azure Functions, etc..). I had to add a "cleanup" script on my CI environment not to deploy tests and extra useless files.. :-/
NPM is a package manager not a SCM

You can also look at your friend auth0/lock with its .npmignore file


Can we also register click listeners using this??

Yew ,you can get a View in the layout (that created with json2view) and register an click listener on it

Closing this.

how to get the view in the layout created by json2view  ?????   sample code please


Bump version of rekor to v0.5.0

Per a recent Sigstore blog post [1], it is recommended for all consumers
to migrate to a version of rekor >= 0.5.0 due to API and sharding
changes on the part of the rekor server. This commit bumps the version
to v0.5.0
[1] https://blog.sigstore.dev/sigstore-project-update-march-2022-bf34aa632388
Signed-off-by: Rob Nester<EMAIL_ADDRESS>

/assign @mpeters
as suggested by tekton-robot.

/ok-to-test

/retest

/lgtm

@wlynch, it looks like it needs an approval label.

I can't approve (yet) :(
/approve

cc @mattmoor @dlorenc for an assist?

approved 🙃

/lgtm


Update BubbleSort.py

improved docs!!!!

@codeshoper Can you please also mention the reference for the source code in your commit.

Hey dude I am new to GitHub  so if you can  briefly tell what are you
saying then that will be great
On Mon 19 Oct, 2020, 11:01 PM Tanmay Khandelwal<EMAIL_ADDRESS>wrote:

@codeshoper https://github.com/codeshoper Can you please also mention
the reference for the source code in your commit.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Dude-901/python-algorithms/pull/17#issuecomment-712318131,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ4G72P5G3WX6QCOLHUNIYTSLRZX7ANCNFSM4SVGQX7Q
.

@codeshoper I was asking if you could share the platform/portal/website (whatever applies) from where you have learned this concept.

I started coding  6 months back and everything thing I learnt was from
youtube or chrome
On Thu 22 Oct, 2020, 12:15 PM Tanmay Khandelwal<EMAIL_ADDRESS>wrote:

@codeshoper https://github.com/codeshoper I was asking if you could
share the platform/portal/website (whatever applies) from where you have
learned this concept.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Dude-901/python-algorithms/pull/17#issuecomment-714270390,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ4G72P4YW2PQTTSGPVSLADSL7IJBANCNFSM4SVGQX7Q
.

@codeshoper Okay great. It's just, No offense intended, but I think there are one or two glitches in the code (I could be wrong), and I currently cannot review the code properly due to some issues. It would be great if you could add some images of the code running with input and output shown so my little doubt could be cleared and the code's credibility can be maintained.

Ok I will try to do so
On Fri 23 Oct, 2020, 7:59 PM Tanmay Khandelwal<EMAIL_ADDRESS>wrote:

@codeshoper https://github.com/codeshoper Okay great. It's just, No
offense intended, but I think there are one or two glitches in the code (I
could be wrong), and I currently cannot review the code properly due to
some issues. It would be great if you could add some images of the code
running with input and output shown so my little doubt could be cleared and
the code's credibility can be maintained.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Dude-901/python-algorithms/pull/17#issuecomment-715377706,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ4G72L3HCGLCG75HBMA4D3SMGHNPANCNFSM4SVGQX7Q
.


Update local workflows with new input names

Fixes #62

Codecov Report

Merging #63 (fb0a4a4) into main (713c47f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #63   +/-   ##
=======================================
  Coverage   52.14%   52.14%
=======================================
  Files           3        3
  Lines         280      280
=======================================
  Hits          146      146
  Misses        134      134

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


psych doesn't work with rails

Please refer to http://stackoverflow.com/questions/4980877/rails-error-couldnt-parse-yaml/5107098#5107098
I wonder if it's a known issue.

Redcloth used to contain invalid yaml. Please update redcloth.


Update runtime protocol with attribute connectionStores

What type of PR is this?
Improvement
What does this PR do / why is it needed ?
Users can organize stores underneath connections within a runtime.
Does this PR introduce a user-facing change?
yes, DSL expansion

This PR is stale because it has been open for 30 days with no activity. Please remove stale label or add any comment to keep this open. Otherwise this will be closed in 5 days.

This PR was closed because it has been inactive for 35 days. Please re-open if this PR is still relevant.


Release: feature/genericRoles

Release: feature/genericRoles

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: shalini-peeka:white_check_mark: agnelpraveen:white_check_mark: aniruddha-k:x: araarumaraarum seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Travis - fix PHP7 build

Dammit, our codebase started failing on PHP 7 :/

and... fixed ;)

:+1:


Noticed that enchant count ends up being negative...good first issue?

Hi!  I am a C# programmer and would like to fix this.  Is there a reason we save files with negative enchant points?
Repro Steps

Import a new level 1 character
Use the editor to add two points to an item
Observe that the enchant points count will be -2.
This carries over after saving into the game as well

Proposal

Maintain all current logic
On saving, if count of enchant > 0, then save as 1 or 0.

Let me know if you like this, and I'll do it.

Could not replicate this issue on latest release, all enchantments are displayed and function correctly (I created a completely new item and added 3 enchantments, for reference)

That's the issue I was talking about, where it is possible to end up with negative enchants.
If you have a character with no spare points, and add a level of any enchant to a piece of gear, the character's overall enchantment point count decreases to below 0.
If this is intentional to prevent a buffer underrun or some kind of error in the game, then I understand it and we can just clarify this behavior with a tool tip on the enchant point field.
But if it isn't then I think a better option is to just save it at zero points.

The enchantment points are not saved in the character save file. They are computed based on the character level minus the total number of enchantment points used on items in the player's inventory.
Code from ProfileExtensions.cs:
public static int remainingEnchantmentPoints(this ProfileSaveFile profile)
        {
            int totalEnchantmentPointsUsed = 1;
            foreach (var item in profile.allItems())
            {
                totalEnchantmentPointsUsed += item.enchantmentPoints();
            }
            return profile.level() - totalEnchantmentPointsUsed;
        }

Thank you, that makes complete sense, not actually a bug then at all!


Blank browser screen on initial app launch in new browser

Describe the Bug
When I launch the app by opening a new browser (e.g. after clearing history) and type in URL, I get a blank page. The web console shows the following error:
Uncaught TypeError: Cannot read properties of null (reading 'fam_cognito_region')
Expected Behaviour
The application screen displays properly. This does happen when I refresh the page.
Actual Behaviour
Blank screen is displayed. In the code, it appears it is initializing main.ts and specifically awsExports before env.js is called to initialize the environment. Perhaps move initializing of env.js into main.ts???
Additional context
Observed in Edge and Chrome.

PR is ready.
In summary:
From observations, this initial blank page only happens during user's "first time" (particularly when no browser caching from previous loading) accessing the FAM site.
Found that web browser stack can use "preload" for preloading some content for performance and that seems to solve the problem. Tested few times on browsers (Chrome/Edge) with clear caching and inPrivate mode, I observed it is consistent now with no blank page issue.
Hopefully it does solve the problem.

As can be seen above, "env.json" is preloaded and ready first, and main "js" code is then queued and loaded later.

@basilv  - please review


in README/Duck Accessors, the acessor for actions is named 'creators' and not 'actions', right?

Right! Thank you for the heads up!


⚙ Warmup for /pages/index.vue failed with: At least one

Environment
❯ pnpx nuxi info
Packages: +1
+
Progress: resolved 2, reused 1, downloaded 0, added 1, done
Working directory: /home/everyx/Projects/fengsi/api.fengsi.io                                                                                9:47:43 AM
Nuxt project info:                                                                                9:47:43 AM

------------------------------
- Operating System: Linux
- Node Version:     v18.17.0
- Nuxt Version:     -
- CLI Version:      3.7.3
- Nitro Version:    -
- Package Manager:  unknown
- Builder:          -
- User Config:      -
- Runtime Modules:  -
- Build Modules:    -
------------------------------

Reproduction
https://codesandbox.io/p/sandbox/vibrant-greider-vk735p?file=%2Fnuxt.config.ts%3A5%2C1
Describe the bug
I want to add a web component to Vue, and I set below, then get error.
 vite: {
    plugins: [
      vue({
        template: {
          compilerOptions: {
            isCustomElement: (tag) => tag.includes('-'),
          },
        },
      }),
    ],
  },

Additional context
No response
Logs
[2:35:44 AM]  ERROR  At least one <template> or <script> is required in a single file component.

Kindly place script and template in your component. I think it works

@everyx
Nuxt sets up @vitejs/plugin-vue internally, and we are supposed to pass its options via Nuxt config.
Try the code below:
export default defineNuxtConfig({
  vue: {
    compilerOptions: {
      isCustomElement: (tag) => tag.includes('-'),
    },
  },
})

It's worth checking the following options of Nuxt config related to Vite and Vue.

vite option of Nuxt config
vue option of Nuxt config

@NozomuIkuta Thank you, that is indeed the case.


Started duplicator tool

Going to need client to server RCP before file transfers will work. I can at least do the ghosts and add duplicator support to all the scripted entities.

Thanks a ton for starting work on this! What you had largely did seem to work, after updating for the API changes. I added constraint support, and my NData Server -> Client networking library for saving dupes :)

Awesome


[WIP] Categorical Feature Concatenation

Closing as this will now need to be re-factored into OperationsNew


NONANTIALIASED_QUALITY misspelled in os_mswin.c

And, less important as it doesn't impair functionality, also in options.txt.
patch.txt

Kim Brouer wrote:

And, less important as it doesn't impair functionality, also in options.txt.

Thanks!
--
"I've been teaching myself to play the piano for about 5 years and now write
most of my songs on it, mainly because I can never find any paper."
Jeff Lynne, ELO's greatest hits
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Mup deploy constantly failing.

Guys,
I'm having a devils own job trying to get mup to deploy my latest server.
I've been at it all day and have now got to the following error and can get no further.
I've had to install all of the node modules by hand using an updated version of npm (the only one I can get to work 1 out of 3 times without failing without getting connection resets).
I now have no idea where to go from here to get this working ?
Any clues? Please? Anyone?
Peter.
x Invoking deployment process: FAILED
-----------------------------------STDERR-----------------------------------
lib_file=node.lib',
gyp info spawn args   '-Dmodule_root_dir=/opt/taralgadash/tmp/bundle/programs/server/npm/npm-bcrypt/node_modules/bcrypt',
gyp info spawn args   '--depth=.',
gyp info spawn args   '--no-parallel',
gyp info spawn args   '--generator-output',
gyp info spawn args   'build',
gyp info spawn args   '-Goutput_dir=.' ]
gyp info spawn make
gyp info spawn args [ 'BUILDTYPE=Release', '-C', 'build' ]
gyp info ok
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN<EMAIL_ADDRESS>No license field.
stop: Unknown instance:
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (7) Failed to connect to localhost port 3080: Connection refused
stop: Unknown instance:
Latest deployment failed! Reverted back to the previous version.
-----------------------------------STDOUT-----------------------------------
MODULE(target) Release/obj.target/bcrypt_lib.node
COPY Release/bcrypt_lib.node
make: Leaving directory `/opt/taralgadash/tmp/bundle/programs/server/npm/npm-bcrypt/node_modules/bcrypt/build'
><EMAIL_ADDRESS>install /opt/taralgadash/tmp/bundle/programs/server/node_modules/fibers
> node ./build.js

`linux-x64-v8-3.14` exists; testing
Binary is fine; exiting
<EMAIL_ADDRESS>/opt/taralgadash/tmp/bundle/programs/server
├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
Waiting for MongoDB to initialize. (5 minutes)
connected
taralgadash start/running, process 26450
Waiting for 350 seconds while app is booting up
Checking is app booted or not?
taralgadash start/running, process 27004

Please post your settings file too.
I always recommend using the development branch mupx if having problems with plain old regular mup.
I suspect you need to set the node version to 0.10.40 as its the one that works with Meteor 1.2

Sorry, mup.json is
"servers": [
4     {
5       "host": "<IP_ADDRESS>",
6       "username": "root",
7       // "password": ""
8       // or pem file (ssh based authentication)
9       "pem": "/var/root/.ssh/id_dsa"
10     }
11   ],
12
13   // Install MongoDB in the server, does not destroy local MongoDB on future setup
14   "setupMongo": false,
15
16   // WARNING: Node.js is required! Only skip if you already have Node.js installed on server.
~  17   "setupNode": false,
18
19   // WARNING: If nodeVersion omitted will setup 0.10.36 by default. Do not use v, only version number.
20   "nodeVersion": "0.10.36",
21
22   // Install PhantomJS in the server
~  23   "setupPhantom": false,
24
25   // Show a progress bar during the upload of the bundle to the server.
26   // Might cause an error in some rare cases if set to true, for instance in Shippable CI
27   "enableUploadProgressBar": true,
28
29   // Application name (No spaces)
30   "appName": "taralgadash",
31
32   // Location of app (local directory)
33   "app": "/Users/peternunn/Documents/src/hardsoftware_infolite/meteor/twfdashboard",
34
35   // Configure environment
36   "env": {
37     "ROOT_URL": "https://taralga.hardsoftware.com",
38     "PORT": 3080,
39     "MONGO_URL": "mongodb://localhost:27017/infolite/"
40   },
41
42   // Meteor Up checks if the app comes online just after the deployment
43   // before mup checks that, it will wait for no. of seconds configured below
44   "deployCheckWaitTime": 350
45 }
I'm not using 1.2, I'm still on <IP_ADDRESS> due to some issues (still unresolved) when we tried to migrate to 1.2.
I don't want to use mupx because I don't want to set up docker containers (I think that's what mupx does?) as I'm already using lxc containers for this project.
I've tried to run this manually by copying the bundled code into the right place but am getting
Error: /opt/taralgadash/app/programs/server/npm/npm-bcrypt/node_modules/bcrypt/build/Release/bcrypt_lib.node: invalid ELF header
when I run node main.js
I've tried removing and installing<EMAIL_ADDRESS>(globally) but its telling me I have unmet dependency somewhere... sigh.
Thanks.
Peter.

All I can think of to try in your complicated setup is the nodejs version. Maybe 0.10.40 will help there too. I hope your solution is simpler then the typical route which is rebuilding with mupx, docker and meteor 1.2. Unofficially your no longer supported if you can call it that.
I didnt grasp your reasons why, but I'd alao spend some more time to make sure trying mupx is truely not an option.

Thanks MasterJames, Just wondering what's complicated about the setup?  I've used exactly the same setup (infact an exact copy of this machine) on two other installs with some minor problems last time getting the deployment happening, but its basically just a single server with meteor and mongo on it. Its an app at 1.1 rather than 1.2 but I'm not sure how that qualifies as complicated.

I've just cleared the directory in /opt on the server away and done a mup setup, mup deploy. Now I'm back to npm errors !! This is nuts.
Invoking deployment process: FAILED
-----------------------------------STDERR-----------------------------------
registry.npmjs.org/fibers/-/fibers-1.0.5.tgz
npm ERR! fetch failed https://registry.npmjs.org/source-map/-/source-map-0.1.32.tgz
npm ERR! fetch failed https://registry.npmjs.org/source-map/-/source-map-0.1.32.tgz
npm ERR! fetch failed https://registry.npmjs.org/source-map/-/source-map-0.1.32.tgz
npm ERR! network read ECONNRESET
npm ERR! network This is most likely not a problem with npm itself
npm ERR! network and is related to network connectivity.
npm ERR! network In most cases you are behind a proxy or have bad network settings.
npm ERR! network
npm ERR! network If you are behind a proxy, please make sure that the
npm ERR! network 'proxy' config is set properly.  See: 'npm help config'

npm ERR! System Linux 3.13.0-74-generic
npm ERR! command "/usr/bin/node" "/usr/bin/npm" "install"
npm ERR! cwd /opt/taralgadash/tmp/bundle/programs/server
npm ERR! node -v v0.10.36
npm ERR! npm -v 1.4.28
npm ERR! syscall read
npm ERR! code ECONNRESET
npm ERR! errno ECONNRESET
npm ERR! not ok code 0
-----------------------------------STDOUT-----------------------------------
 > ./bcrypt: npm install due to binary npm modules
<EMAIL_ADDRESS>node_modules/bindings

<EMAIL_ADDRESS>node_modules/nodeunit
└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>make: Entering directory `/opt/taralgadash/tmp/bundle/programs/server/npm/npm-bcrypt/node_modules/bcrypt/build'
  CXX(target) Release/obj.target/bcrypt_lib/src/blowfish.o
  CXX(target) Release/obj.target/bcrypt_lib/src/bcrypt.o
  CXX(target) Release/obj.target/bcrypt_lib/src/bcrypt_node.o
  SOLINK_MODULE(target) Release/obj.target/bcrypt_lib.node
  COPY Release/bcrypt_lib.node
make: Leaving directory `/opt/taralgadash/tmp/bundle/programs/server/npm/npm-bcrypt/node_modules/bcrypt/build'

><EMAIL_ADDRESS>install /opt/taralgadash/tmp/bundle/programs/server/node_modules/fibers
> node ./build.js

`linux-x64-v8-3.14` exists; testing
Binary is fine; exiting
----------------------------------------------------------------------------

AMAZING!! I just, in desperation ran deploy again and its started... no idea why!! This is going to bite me again I'm sure. Thanks for the help though.  Peter.

That error you posted speaks of temporary networking problems while fetching from the registry. Anyway I'm glad it's working now.

There seems to be a real issue with registry.npmjs.org. If I wget a file from it, it can take 6 or more attempts before one works without a connection reset in header error.
I also just noticed that when mup fails it is still using https:// despite me setting the registry config item on the target server to http:// I'm running npm config set registry http://registry.npmjs.org as a normal user, then forcing a cache clear, is that correct or should I be running this somewhere else (on the bundling server perhaps?).

OK, on another machine I'm having the same issue. It seems mup deploy is ignoring the registry setting no matter if I set it on the target or the development machine. Where the hell do I need to set this to make it stick?


Feature Request: Enhanced Control Over Checkboxes in Timeline View

Feature Request: Enhanced Control Over Checkboxes in Timeline View
Current Behavior
As it stands, checkboxes that originate from the current daily note interaction in the timeline view doesn't matter. However, checkboxes that are derived from other source need the interaction in the timeline. To change the status of these checkboxes (e.g., marking a task as completed), one must navigate away from the timeline view to the original note where the checkbox is located. This process introduces unnecessary friction and disrupts the flow of managing tasks efficiently across different notes within the timeline view.
Suggested Feature
I propose the development of a feature that enables users to change the status of any checkbox directly from the timeline view. This enhancement would allow for a more unified and efficient task management experience, as users could interact with tasks derived from various notes without the need to navigate away from the timeline.
Thank you very much.

Hello, @Kang-geophysics!
This is a duplicate of #304.
In the meantime, you can use the hover-editor plugin to complete tasks quickly:


Remove backlight pin from pin assignments when disabling backlight control

Backlight control appears when control of the display backlight by using an GPIO is disabled, it shouldn't.

Fixed with 404c4a1


pwsh -f <ps1-file> fails to run in system lockdown environment if the ps1-file is signed (trusted)

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
This is a report from internal customer:
On a machine with Applocker (WDAC) enforcement, PowerShell runs in ConstainedLanguage mode by default. Running pwsh -file <ps1-file> fails with the following error when the ps1-file is signed (trusted).

The problem is -- the user doesn't dot-source CreateShortCut.ps1, but just run it with powershell -file.
The root cause here is that powershell.exe -f <ps1-file> attempts to invoke the ps1 file without using a local scope, which is the same behavior as if it’s dot-source invoked.
In ConsoleHost.cs, it creates a Command instance with Command(filePath, isScript: false, useLocalScope: false).
PowerShell core (pwsh) also does the same thing, and hence pwsh -f <ps1-file> will fail too in this case:
https://github.com/PowerShell/PowerShell/blob/8d25b5ae6e2eb83a957eac823c27546a70c229a0/src/Microsoft.PowerShell.ConsoleHost/host/msh/ConsoleHost.cs#L1907-L1909
Expected behavior
`powershell -f <ps1-file>` and `pwsh -f <ps1-file>` should run no matter `ps1-file` is signed (trusted) or not in the WDAC locked down environment.

Actual behavior
`powershell -f <ps1-file>` and `pwsh -f <ps1-file>` fail to run.

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.2.6
PSEdition                      Core
GitCommitId                    7.2.6
OS                             Microsoft Windows 10.0.19044
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

I see the file is on network share? Can the customer put the file locally and check?

It works fine when the script is not signed (trusted). Quote the customer's words here:

If he makes script not trusted, he gets event that script not allowed to run in applocker log, but it works as expected.

So as not to introduce a breaking change, I feel the best solution is to add a new command parameter, something like: -FileNewScope, which runs the file script in a new scope rather than the global scope as it does now.
This is not a security issue, so I am removing the @wg-security tag.

Note that there is a workaround (using -command '& c:\filepath\filename.ps1), so I feel this is not super high priority.

@sdwheeler @michaeltlombardi
The @PowerShell/wg-powershell-engine discussed this a we're worried about making a behavioral change, but the way PowerShell behaves when using -file should definitely be documented.

@daxian-dbw Can you open a doc issue for this?

It turns out the -File already calls out the dot-sourcing behavior in its doc:
https://learn.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_pwsh?view=powershell-7.3#-file---f

This is the default parameter if no parameters are present but values are present in the command line. The specified script runs in the local scope ("dot-sourced") of the new session,

@JamesWTruher @SeeminglyScience Do you think we should add additional information about this particular scenario -- using a singed .ps1 file for pwsh -file in a locked down system?

@JamesWTruher @SeeminglyScience Do you think we should add additional information about this particular scenario -- using a singed .ps1 file for pwsh -file in a locked down system?

Eh, I think it's fine as is.


[1500] Move copy content warning into partial

Context
Cleaner code
Changes proposed in this pull request
Move copy content warning into partial to avoid duplicated code
Guidance to review
No visual or functional change, just DRYer code.

 DRYup Refactor copy_form warning message

The template DRYing is definitely a good improvement, but now _copy_content_warning.html.erb has some logic belonging to the individual pages.
How about using content_for to keep the page-specific bits (probably the list items) in the original template files?

Nice idea @benilovj. I deffo don't like that logic but it was an improvement on the previous code. Another iteration to come :)


[flash_ctrl/dv] Move callback to apply_reset from dut_init

Hi,
This PR move the call of flash_ctrl_callback dut_init_callback method in flash_ctrl_base_vseq from dut_init to apply_reset.
The reason for this, is that this method should be called after every reset.
Thanks!
Signed-off-by: Eitan Shapira<EMAIL_ADDRESS>

@TimEwins : Please check if this affects RMA and OTP tests.

@rswarbrick I understand it might be problematic, but I need to make sure that whenever a reset is occuring, the callback method will be called, since it is doing some register configurations in the extending partner env needed to be done after each reset.
There are sequences that specifically calling apply_reset and this what caused some problems for me in some scenarios.
I specifically failed in some seed when I ran the hw_sec_otp test, because it calls apply_reset as below:
https://github.com/lowRISC/opentitan/blob/9f60770816a837425c5cc3faf71d748ae7e7f3b9/hw/ip/flash_ctrl/dv/env/seq_lib/flash_ctrl_hw_sec_otp_vseq.sv#L196
When I ran it with this fix it passed.
Do you have a suggestion how to bypass it?

@rswarbrick As I see it, there are 2 options - to disallow any direct calls to apply_reset, or to move the callback to the apply_reset as I did in this PR, which is still the correct approach to my opinion. Please let me know what you think.
@cindychip @weicaiyang @sriyerg Does anyone else has anything to comment?
Please review this and if all is good lets merge this PR.
Thanks!

I am happy to go for your suggestion.  The tests I have written so far call apply_reset directly, so this is the
least disruptive to my tests.
TimE
Get Outlook for iOShttps://aka.ms/o0ukef

From: eshapira @.***>
Sent: Sunday, March 6, 2022 07:16
To: lowRISC/opentitan
Cc: Tim Ewins; Mention
Subject: Re: [lowRISC/opentitan] [flash_ctrl/dv] Move callback to apply_reset from dut_init (PR #11137)
@rswarbrickhttps://github.com/rswarbrick As I see it, there are 2 options - to disallow any direct calls to apply_reset, or to move the callback to the apply_reset as I did in this PR, which is still the correct approach to my opinion. Please let me know what you think.
@cindychiphttps://github.com/cindychip @weicaiyanghttps://github.com/weicaiyang @sriyerghttps://github.com/sriyerg Does anyone else has anything to comment?
Please review this and if all is good lets merge this PR.
Thanks!
—
Reply to this email directly, view it on GitHubhttps://github.com/lowRISC/opentitan/pull/11137#issuecomment-1059909689, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWDNPW4NXZV7W27QTI2LJSLU6RLVZANCNFSM5PRIJLGA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

If all is good here, can we please merge this PR?

Can someone please merge this PR?

Thanks @weicaiyang !


Problem in running gzsd.py file

Hi @salman-h-khan
It would we great if you could help me out , while running gzsd.py file, ealier I got an Import error for "keras-retinanet ",  after which I tried by installing using "pip install keras-retinanet==0.5.1", this problem resolved but a new error is coming shown below:
File "gzsd.py", line 6, in
from keras_retinanet.models import retinanet_vocab_w2v as retinanet # retinanet_vocab_glo
ImportError: cannot import name 'retinanet_vocab_w2v'
I am not getting solution for this, please help me out,also does the keras-retinanet version same or different ( I had setup the environment as stated).
Thanks,
Sushil Kumar

got solved.


Not starting on macOS 10.11 with default config

This is the error I receive:

Config file not found; using defaults
Alacritty encountered an unrecoverable error:
Error creating glutin::Window; Couldn't create OpenGL context: Some of the requested attributes are not supported

I'm on a 2015 MBP 13". It started once. I used ctrl+D to exit, created a ~/.alacritty.yml and changed some options, tried to start it, which failed, then removed ~/.alacritty.yml. It's still failing with the same error message, and I'm 100% certain that my custom options aren't being used because of the first line of the output!

And for reasons I completely do not understand, after I quit Atom accidentally, Alacritty started right up, much faster than it had the previous few times.
I cannot explain this, other than perhaps Atom was somehow occupying some level of system resources that Alacritty needed.


CORS for get endpoints

Unless I'm missing something, it seems like there's no supported way to access the endpoints like /_stats and /_live outside of the analytics domain in a browser. It seems relatively harmless for these to be access-control-allow-origin: *. However that should be different configuration than cors for post requests, since while it makes sense to make the auditing info available wherever, I'd likewise want to be able to restrict analytics post requests to a single domain.
I'd be happy to make a PR if we can agree on what the API would look like. What do you think? I'm wondering if it makes sense to add a new flag called endpointsOrigin which is the same as origin but only for the non-/ GET endpoints?

hey @benwiley4000 thanks for the input.
I think what you said makes perfect sense. We definitely want to have those endpoints available from other domains.
Since the change is harmless, as you mentioned, I'd change the default CORS config for those specific endpoints rather than introducing a new configuration flag.
What do you think?

Sounds fine to me! Thanks!
Le lun. 1 avr. 2019 03 h 47, Alessandro Arnodo<EMAIL_ADDRESS>a
écrit :

hey @benwiley4000 https://github.com/benwiley4000 thanks for the input.
I think what you said makes perfect sense. We definitely want to have
those endpoints available from other domains.
Since the change is harmless, as you mentioned, I'd change the default
CORS config for those specific endpoints rather than introducing a new
configuration flag.
What do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vesparny/fair-analytics/issues/21#issuecomment-478472236,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AM7h7UnPHDzWrYMXRQIQOKZTXNg1iD1Lks5vcbl1gaJpZM4cUDMW
.


Updated nuget reference for NReco.LambdaParser.

:white_check_mark: Build ExtendedXmlSerializer <IP_ADDRESS>-qdifoiwe completed (commit https://github.com/ExtendedXmlSerializer/home/commit/4c9f31a0c3 by @Mike-EEE)
Install it by running the following command in Package Manager Console:
Install-Package ExtendedXmlSerializer -Version <IP_ADDRESS>-qdifoiwe -Source https://ci.appveyor.com/nuget/extendedxmlserializer-preview


alpha is not added in correctly

The role of alpha is to act as a weight between the two classes (pos and neg class have weights alpha,1-alpha respectively; so alpha should be chosen between 0 and 1)
In the current implementation, alpha is the same for both classes; so effectively does nothing to the loss function

I have corrected alpha value according to @Atomwh's implementation.


Description of Introductory Assignments

I worked out some exercises for the introductory assignments, especially at Architecture.
@Blazefrost and I talked about different exercise styles. Currently, I am writing on another version where I describe more than just the exercises. @ckirsch would that be better, or is it enough to link to resources?
We thought that we should save time and concentrate more on the coding exercises.

@aemonk Exercises that we can use in the upcoming class have highest priority. Think of producing material that I can use in class right away. When writing about numbers use correct prefixes: 0b1010 for binary, 0x1010 for hexadecimal, 0o1010 for octal. Remove the $ signs. They are confusing.

Try to write up self-contained exercises where the provided information is sufficient and necessary, not more, not less.

Think of a good sequence of introductory assignments at the beginning of the class. Remove all planned stuff from introductory-assignments.md Instead make that file self-contained so that I can point the students to it.

Create a GitHub project in the selfie repo to keep track of what you plan to do. @ChristianMoesl can help you with that. All files in the selfie repo should be ready for use. I will keep merging your changes in the grader branch into the master branch for the students to look at.

Think of a good sequence of introductory assignments at the beginning of the class. Remove all planned stuff from introductory-assignments.md Instead make that file self-contained so that I can point the students to it.

Removed all planned stuff from the file and made a new first version of how it could work.
I referenced all the material to the selfie manuscript (book), everything is beautifully described there.
@ckirsch please let me know what you think about this style.

Version 2 is better. But don't mix low- and high-level questions. Creating semantics is very high-level and difficult to answer.

I recommend to focus on low-level questions for now.


ENH: add QProbe.oversample

@pkienzle, is this PR the correct way of going about oversampling? It would be nice to have it for the validation suite that checks if resolution smearing is done correctly. (It is, but oversampling is required for the test cases)

Yes, this is fine. Unfortunately, I missed seeing it, and ended up reimplementing it in response to issue #95.
I'm adding your changes to .gitignore, so this PR is no longer needed.


Add geo types support

Added geo types support from PostgreSQL and geo_types module in Rust.
Description

Motivation and Context

How has this been tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Btw. Looks great! Hope to see the final version soon!

Today will update on using exceptions that changed after main branch merge

Add tests, please

Added fixes, now all geo types except polygon are working well!
Have some troubles with writing polygon to databases, trying to figure out how to do it:
DriverError: Database engine exception: db error: ERROR: invalid number of points in external "polygon" value.


Not getting latest data of sat-search

I'm trying to get latest sat-search data from 1 May 2023 to till date but it is showing empty
but old data I'm able to access.
I'm using this api url : https://earth-search.aws.element84.com/v0
sample code:
import satsearch
timeRange = '2023-05-01/2023-05-24'
bbox = [72.78151203878225, 19.46368082936482, 72.78289605863394, 19.464469853150305]
SentinelSearch = satsearch.Search.search(
    url = "https://earth-search.aws.element84.com/v0",
    bbox = bbox,
    datetime = timeRange,
    collections =  ["sentinel-s2-l2a"] #['sentinel-s2-l2a-cogs']
)
Sentinel_items = SentinelSearch.items()
print('summary: ', Sentinel_items.summary())

Output like this:
summary:  Items (0):
date                      id

I am facing a similar issue as I attempted to acquire satellite imagery after May, but it does not meet my requirements.

Anyone have any solution for this or any other similar way to get satellite data

Hello @anilmohite22 and @aimad1234 ,
We actually have a new version of the API (v1), and you can consider v0 being deprecated.  You can read about the updates in the blog post here:
https://element84.com/blog/introducing-earth-search-v1-new-datasets-now-available
Also, sat-search is currently unmaintained and I'm not sure it will work with the new API.  The replacement for sat-search is pystac-client: https://github.com/stac-utils/pystac-client

@matthewhanson Thank you, sir, for providing this answer. I am grateful for your assistance, and I am pleased to inform you that I was able to successfully implement the solution using the pystac_client.


Update build gradle

get error No resource found that matches the given name: attr 'android:keyboardNavigationCluster' with RN 0.56
Updating build gradle solve this problem

I'm getting the same error with
"react-native": "0.57.1",
"react-native-admob": "2.0.0-beta.5",
"react": "16.5.1",

Your update fixed it for me. Please accept the pull request asap, modifying the node_modules to make the build work should be a solution I have to rely on.

I've merged #393 so I think it should now be possible to override the versions from your project.


fix(line): soft clipping gradient.

Brief Information
This pull request is in the type of:

[x] bug fixing
[ ] new feature
[ ] others

What does this PR do?
In the cases like #15680. The coordinates range of gradient objects is very large. Which leads to blurry transition when GPU acceleration is enabled in chrome. Only version 5.1 doesn't have this issue because we clamped the data. But it will cause infinite value wrong appearance. So in this PR https://github.com/apache/echarts/pull/15416 we removed the clamping and this blurry issue appears again.
In this PR. I clamp the color stops and recalculate the clamped color in LineView.ts. It should fix both issues mentioned above.
Fixed issues
#15680
Details
Before: What was the problem?

After: How is it fixed in this PR?

Misc

[ ] The API has been changed (apache/echarts-doc#xxx).
[ ] This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs
NA.
Others
Merging options

[ ] Please squash the commits into a single one when merge.

Other information

It can solve but doesn't seem to solve perfectly...

@plainheart Yes, the leaking issue is hard to be avoided because of the limitation of the gradient.

大佬牛逼

@plainheart I did some search on how skia draws the gradient with OpenGL. Seems it will use a 256 width texture to cache the gradient pixels when it has more than 10 color stops. The size is much smaller than the canvas width. So it can't be pixel-accurate.
https://source.chromium.org/chromium/chromium/src/+/main:third_party/skia/src/gpu/gradients/GrGradientShader.cpp;l=29;drc=a9641800e0e8f7a2ea9a5f55f831b632e990a0e0;bpv=1;bpt=1
One way to solve this issue I can think of is to split the gradient into multiples and draw them on a full-size canvas. Then use this canvas as a CanvasPattern.


Optimize symbolic execution using depth-based call graphs

Description
The idea is to use a depth-based call graph of the Foundry test suite in order to manage how we do symbolic execution. Instead of starting symbolic execution from the top of the test, you would create a dependency list of functions, then execute them starting with the functions at the lowest level/highest depth and generate simplification lemmas for them. Next, you would execute the next functions up the call graph tree, importing the simplification lemmas for the dependent functions.
Example:
Given this Solidity smart contract
contract BranchingDepthExecutionExample {

    // State variables
    uint256 public countA = 0;
    uint256 public countB = 0;
    uint256 public countC = 0;
    bool public branchCondition = false;

    // Set the branch condition
    function setBranchCondition(bool _condition) public {
        branchCondition = _condition;
    }

    // Function A: Calls both B and C, but B's call is conditional based on branchCondition
    function A() public {
        countA++;
        if(branchCondition) {
            B();
        }
        C();
    }

    // Function B: Calls C
    function B() public {
        countB++;
        C();
    }

    // Function C: A basic function with no further calls
    function C() public {
        countC++;
    }
}

And a test contract
contract BranchingDepthTest is Test, KEVMCheats {
    BranchingDepthExecutionExample c;
    function setUp() public {
        c = new BranchingDepthExecutionExample();
    }
    function testExample(bool branchCondition) external {
        kevm.infiniteGas();
        vm.assume(branchCondition == true);
        c.setBranchCondition(branchCondition);
        c.A();
        c.B();
        c.C();
        assertEq(c.countA(), 1);
        assertEq(c.countB(), 2);
        assertEq(c.countC(), 4);

    }
}

With our current symbolic execution, the order of the functions looks like:

And by using the call graph and lemma generation we could have:

Plan:

When building the Foundry object in kevm-pyk during kontrol foundry-kompile, generate the call graph of the test functions in the test suite. The call graph depth should start at 0, representing the level of the test function.
When foundry-prove --test MyContract.test1 --test MyContract.test2 ... is called, generate a list of the functions that need to be tested: [MyContract.test1, MyContract.test2].
For each test function MyContract.f in the list, parse the call graph and append all the functions that MyContract.f would call to the list. This list should then be sorted by the depth level such that the functions with the highest depth (or the most low-level functions) are first.
Start to execute the functions in the list symbolically. Functions on the same depth layer/level should be executed in parallel.
When the proof execution is finished, generate a simplification lemma for the function and store it locally.
After all the functions from the highest depth layer are finished, go to the next layer. Import simplification lemmas generated at 5. only for the calls executed inside the function.

This should improve the symbolic execution process considerably, avoiding execution for duplicated calls.

@anvacaru and @PetarMax let's please flesh out a workplan for this one? It's critical for performance and for some of our engagements.

Next steps

Modify how foundry_prove generates the initial state when executing a contract method.
Refactor kontrol show --to-module file_path to:

avoid writing the kcfg in the file_path
generate a valid module name (currently includes invalid characters in the module name, i.e.module SUMMARY-CSE.F(UINT256,UINT160,BOOL):0)
generate rewrite rules only from the initial node towards the terminal nodes/leaves.

Have the ability to include a module using the add_module endpoint in foundry_prove()

Subsumed by #284 and #340.


Fix bbc.co.uk playlist support

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
This PR fixes two problems:

Downloading playlists from BBC iPlayer did not work at all due to the page layout changing completely since the extractor was written.
Downloading playlists from other parts of bbc.co.uk was partially broken – only the first page would be downloaded due to some whitespace being inserted between elements since the extractor was written.

Example URLs:

https://www.bbc.co.uk/programmes/b00mfl7n/clips (All 142 clips should be downloaded, not just the first 24)
https://www.bbc.co.uk/programmes/b00mfl7n/clips?page=3 (Only the 24 clips on the specified page should be downloaded)
https://www.bbc.co.uk/iplayer/episodes/b07xd230/josh (All 18 episodes should be downloaded)
https://www.bbc.co.uk/iplayer/episodes/b07xd230/josh?seriesId=p047lvcs (Only the six episodes of season 2 should be downloaded)
https://www.bbc.co.uk/iplayer/episodes/b06kw5fq/scot-squad (Should download whatever episodes are currently available, time sensitive)

I pushed an additional change that fixes a crash when downloading shows that have multiple pages of episodes but only a single season, for example: https://www.bbc.co.uk/iplayer/episodes/b05qzmgd/louis-theroux


Selecting specific element also applies pseudostate to all its descendants

Because of this rewrite logic if .pseudo-* class is applied to a specific element it triggers all descendants' pseudostate
https://github.com/chromaui/storybook-addon-pseudo-states/blob/main/src/rewriteStyleSheet.test.js#L26-L30
when: parameters = { pseudo: { hover: '.someSelector' } } or parameters = { pseudo: { hover: ['.someSelector', 'div'] } }
Current state:
a:hover -> a:hover, a.pseudo-hover, .pseudo-hover a
Proposed change:
a:hover -> a:hover, a.pseudo-hover
when: parameters = { pseudo: { hover: true } }
no changes - a:hover -> a:hover, a.pseudo-hover, .pseudo-hover a

I can work on this, just want to confirm that this is a valid option that you want to implement 😄

@ghengeveld What do you think about it?

Hi! We faced the same issue with pseudostates and snapshotting with Chromatic. Play functions are not working for us (esp with hover) and this addon is the only solution. @ghengeveld, can you take a look, please?

@ghengeveld I can add this functionality, but I think it will break backward compatibility. Is it ok to up to a major version or it's better to add a different way to enable this behaviour?

I'm starting this today

You can use the canary version here https://www.npmjs.com/package/storybook-addon-pseudo-states/v/2.1.1--canary.85.01f67b4.0, I am still working on support for custom elements but it should work otherwise. Please let me know if you have any feedback #85

^^ @poalrom @filipw01

I'm closing this issue because it has been fixed and is now available in v2.1.1


WIP OTel-Go code sample

Design principles

Keep GCP-specific logic out of the application binary
Configure things at runtime, ideally using OTel env variable conventions, rather than baking decisions into the binary
Use slog to stdout for structured logging rather than OTLP (to align with the convention in containers)

To do

[ ] Fix trace-log correlation

Superseded by #3312 .


Multiple node cluster

This is great project, thanks for that :+1:
However could you provide some simple tips for multiple node cluster?
Currently the single-node cluster is working fine, however would like to increase at least to two-nodes cluster.
Appreciate it,
Thanks!

Would this work for you?
https://github.com/deviantony/docker-elk#how-can-i-scale-out-the-elasticsearch-cluster

I have forked your project and adjusted it to multiple node cluster.
https://github.com/kallasramon/docker-elk-cluster

Cool, thanks for sharing your approach!


Show all analysis nodes in the source options

Fixes #8265
At the time of implementation it was only allowed to select layers' head nodes as input sources, this changes the options to include all nodes (except for the self).
Also fixed the bug where source options weren’t shown in the options if the query-schema was already fetched.
E.g.

@javisantana @saleiva  Right now only the input source is filtered out (A2 in the example above), but we could potentially have trouble if we don't filter out nodes that creates cycles (e.g. a node in layer A, which has a node that points to a node in B…), but we can solve that separately…
@xavijam review, please

:+1:

Is this still showing your of the datasets like before?

@saleiva take into account the analysis it's showing is link to layer (pending a PR to change the name) and that's only meant to nodes but it should show the datasets as well

Ok, thanks!
2016-06-28 11:46 GMT+02:00 javi santana<EMAIL_ADDRESS>
@saleiva https://github.com/saleiva take into account the analysis it's
showing is link to layer (pending a PR to change the name) and that's only
meant to nodes but it should show the datasets as well
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/pull/8266#issuecomment-229002849, or mute
the thread
https://github.com/notifications/unsubscribe/AAIENQOiGdXHiUBSyxtChwHfVI-6GLqdks5qQO2SgaJpZM4I_6dr
.

--
Sergio Alvarez Leiva

@saleiva @javisantana yes, by default datasets are also included, but for this particular analysis is filtering out the datasets (I guess that was done on purpose?)

yes @viddo, I did it on purpose

👍  for intersection you'll get both for example:

Merging since frontend tests already passed


Mouseflow documentation

What do these changes do?

Added documentation for Mouseflow destination

Type of documentation

[x] New documentation
[ ] Documentation update
[ ] Hotfix

Moving this back to engineering review as there are more changes expected in the doc. cc @sauravlal-rudderstack @ItsSudip


Fatal Error since containerd.io update

I am running kylemanna\openvpn in OpenMediaVault 7. Recently there was an update to containerd.io and since then I am getting the following error message and I cannot start my container
Running 'openvpn --config /etc/openvpn/openvpn.conf --client-config-dir /etc/openvpn/ccd --crl-verify /etc/openvpn/crl.pem '
Tue Dec 10 02:57:00 2024 OpenVPN 2.4.9 x86_64-alpine-linux-musl [SSL (OpenSSL)] [LZO] [LZ4] [EPOLL] [MH/PKTINFO] [AEAD] built on Apr 20 2020
Tue Dec 10 02:57:00 2024 library versions: OpenSSL 1.1.1g  21 Apr 2020, LZO 2.10
Tue Dec 10 02:57:00 2024 Diffie-Hellman initialized with 2048 bit key
Tue Dec 10 02:57:00 2024 CRL: loaded 1 CRLs from file /etc/openvpn/crl.pem
Tue Dec 10 02:57:00 2024 Outgoing Control Channel Authentication: Using 160 bit message hash 'SHA1' for HMAC authentication
Tue Dec 10 02:57:00 2024 Incoming Control Channel Authentication: Using 160 bit message hash 'SHA1' for HMAC authentication
Tue Dec 10 02:57:00 2024 ROUTE_GATEWAY <IP_ADDRESS>/<IP_ADDRESS> IFACE=eth0 HWADDR=02:42:ac:11:00:02
Tue Dec 10 02:57:00 2024 ERROR: Cannot open TUN/TAP dev /dev/net/tun: Operation not permitted (errno=1)
Tue Dec 10 02:57:00 2024 Exiting due to fatal error

Apparently you can fix it by adding /dev/net/tun to the compose file but as I built this without a compose file, I am lost on how to fix it

--device=/dev/net/tun:/dev/net/tun should work to allow the device to run inside the container with docker run command.

--device=/dev/net/tun:/dev/net/tun should work to allow the device to run inside the container with docker run command.

I tried docker run --device=/dev/net/tun:/dev/net/tun but I get an error saying docker run requires at least 1 argument

Maybe it can help somebody: I had same issue in a plesk server running Ubuntu Server 22.04 with this docker (ERROR: Cannot open TUN/TAP dev /dev/net/tun: Operation not permitted (errno=1)
Problem started one week ago with an upgrade of containerd.io package to 1.7.24-1. Using apt I downgraded to 1.7.23-1 and it started to work again.

I tried docker run --device=/dev/net/tun:/dev/net/tun but I get an error saying docker run requires at least 1 argument

This is not surprising if you run the command without defining an image. The --device parameter is an option(al) parameter for the docker run command. You have to add it to the server process. But I'm not sure how to add additional paramters to the docker run command in OMV - perhaps, it's only possible via docker compose.

I tried docker run --device=/dev/net/tun:/dev/net/tun but I get an error saying docker run requires at least 1 argument

This is not surprising if you run the command without defining an image. The --device parameter is an option(al) parameter for the docker run command. You have to add it to the server process. But I'm not sure how to add additional paramters to the docker run command in OMV - perhaps, it's only possible via docker compose.

My OpenVPN is run via command line, not a compose file. So I SSH to the server and run commands via CLI

I fixed this issue using a command where I added the flags --cap-add=MKNOD and --device /dev/net/tun (see more details at https://openvpn.net/as-docs/docker.html). As a result, the following command was used to start the server:
docker run -d -v $OVPN_DATA:/etc/openvpn -p 1194:1194/udp --device /dev/net/tun --cap-add=MKNOD --cap-add=NET_ADMIN kylemanna/openvpn
I also added two options to the client's configuration file to disable compression (Windows 11 OpenVPN 2.6.12)
allow-compression no
disable-dco


AccessControl bug fixing

It was not reading the last recorded ID.

Q
A

Bug fix?
yes

New feature?
no

Doc update?
no

BC breaks?
yes/no

Deprecations?
yes/no

Fixed tickets
#...

Duplicate of #389
The fix do not solve the bug properly. Described in #403.


NSIS Installation directions

Hi @negrutiu ,
I would like to use this with a current NSIS installer I'm working on, though I'm not sure if this is installed as a plugin in that I'm looking for dll files and .nsh files. If you can could you provide a set of instructions for NSIS newbs like myself.
Thanks again and much appreciated.
-Brian

Hi @bparbhu,
I've just written a getting-started wiki page that sheds more light on this topic...
Short answer:
NScurl is a 3rd party plugin, it's not included in the official NSIS distro. You'll need to download/compile the latest binary (NScurl.dll) and call it in your NSIS script... Once your installer (Setup.exe) gets built, NScurl.dll is automatically embedded inside it and called at runtime.
Alternatively, I've started an NSIS fork project that includes NScurl as built-in plugin. You may want to check it out...

Thanks for the clarification @negrutiu , I appreciate it. Actually I'm trying to get a hold of people from the NSIS community to ask advice about developing installers within NSIS and I'm having trouble finding a good forum or place. Do you have any suggestions?


Fix some graphical issues with themes that have a light primary color

This PR fixes a couple of issues when using themes with a bright primary color, specifically:

icons in the bottom bar remain white
About activity title remains white
bottom app bar has no elevation shadow (and therefore poor contrast with the content)

It also improves colors of Changelog screen a little bit for increased readability:

@Fs00 the work you've put into this app is amazing, and I have one request. Could you target the dev branch instead of master?

Ok. Only this PR or all the others too?

Ok. Only this PR or all the others too?

All of them would be great


Update djunits-demo to Java-17

Following the update of djunits to Java-17, djunits-demo should also be updated.

Completed.


Scrum 2973 - sort topic and entity tags with null values

added special case to sort null values in entity and topic tags
for now, sorting still uses a hardcoded map to convert curie to names. I'm waiting for the A-team to enable admin tokens (this will be discussed at a future dev coordination meeting) to convert this map into a dynamic one fetched from the A-team api
fixed a bug: now the api does not break when the A-team api returns empty result sets

Thanks Juancarlos. I tried an easier solution (assign 'aaa' or 'ZZZ' to None and use the same sorting mechanism for the in place for the other values) and it didn't work, so I had to find a more complicated solution


H26x: uvgRTP seems to sometimes send the same video frame multiple times

This isn't causing frame loss, but it doesn't really serve any purpose to send more video data than necessary.
This was indicated in format tests with many duplicate fragment prints when the amount of sent and received packets was the same.

This was caused by an automatic buffer increasing feature and was relevant only for large frame sizes. I removed this feature in 0d795f5f8ae8c008282edaebef0fc554c305f0a3 since implementing it correctly would have slowed down the reception too much.
If experiencing frame loss, please increase the buffer with configure_ctx()-function on the receiver with RCC_UDP_RCV_BUF_SIZE  variable and adopt the fps api on the sending side with RCC_FPS_ENUMERATOR.


Disconnected adapters are still shown

My expectation is that they won't show while disconnected

Adding and removing graphs from the sysinfo window is not yet supported when they're visible. You have to close and reopen the window.
There's also the issue of some types of adapters adding/removing the adapters multiple times a second moving the window and graphs up and down causing seizures.

Adding and removing graphs from the sysinfo window is not yet supported when they're visible. You have to close and reopen the window.

I don't understand this in full.
I'll just say here that the last time those adapters were used was maybe even 2 years ago.

There's also the issue of some types of devices (e.g. USB) adding/removing the adapters multiple times a second moving the window and graphs up and down causing seizures.

I get if updating the graphs works only between PH invocations. But, as I said above, those adaptors have not been plugged in nice never.
It was just disorienting to have to manually add them each time.

I don't understand this in full.

Disconnected devices are always hidden on the sysinfo window for both the network and disk devices. I assumed they were being removed when you closed the window - which is how this feature used to work previously.

those adaptors have not been plugged in

Windows used to return an error when the adapter was disconnected which is what we used to hide the network adapter from sysinfo window but seems to have changed. I've added a second check for the adapter state which fixes the issue in the latest nightly 👍


Does WIPrint work for chrome/safari/mac ?

There's no reason why it shouldn't work in Chrome or Safari on Mac, but I don't have a Mac so I can't test it. If it does not work, I would be more than happy to fix it.

Thanks for the quick response.
Actually, I've found out that if I access my company tfs server via https, the plugins don't work.
It seems that mixed content ( http x https ) is somewhat buggy, or the initial setup was not done
correctly.
I was able to use your plugin successfully, accessing my tfs server via http.
By the way, congratulations for your plugin, it's very useful.

I haven't tried it over https on TFS, but VSTS is secure so it should work similarly. There's definitely no hardcoded URLs in my extension.
Thanks for the feedback!


Request to make columns as equal height as possible

It will work perfectly but if some items of particular columns make the column abruptly longer move the last elements of column to other columns.

Can you clarify what "column abruptly longer" mean? Maybe with code and a screenshot of the unexpected behaviour is possible

and it's even more below.

Thanks for the screen shot, I understand what you're asking now.
I'm aware of this behaviour and it's occurring because we don't actually handle this case, but i'm open to revisiting.
To implement it's slight performance trade off, as we need to calculate all the item heights and adjust the layout again, then again after each image has loaded. So for this plugin, avoiding all these re-layouts was actually the motivation behind making this plugin originally.. to be a fast and performant foremost.
If we were to do this I'd consider implementing it to be an "opt-in" feature to retain performance but support this feature to the smaller set of users who would need it. happy to discuss more

yeah, that would be very convenient.

@paulcollett having the same issue as here and in #73. The problem becomes especially bad if you use this package for something like unsplash.com. The more items you display, the larger the difference between the columns can become until you end up with just a single column that is actually filled till the end.

this is a great alternative to the traditional masonry library 🙌 However, with such uneven columns, it's visually unusable for anything more than a few items as the page looks broken. Is this opt-in feature in the works by any chance?

Can we at least get it to sort by the height on load?
A lot of us aren't using images, or already know the size of our images, so layout shift isn't an issue.

Considering this is, understandably, intended behaviour we'll likely have to switch to infinite loading of our content but as @jaschaio pointed out, that only delays or potentially magnifies the issue.
Keeping an eye on this for future projects 🙏

Does anybody have an alternative library they can recommend to get around this issue? It's bad enough with a few elements like in the screenshot above, but if you have over 100 (not uncommon for an image gallery) it leaves columns comically uneven, several screens of height different.

did anyone find a solution for this?

@mui/lab/Masonry Can it be handled?

https://github.com/GoncharukBro/react-smart-masonry library works, but not sure about the performance.

Does anybody have an alternative library they can recommend to get around this issue? It's bad enough with a few elements like in the screenshot above, but if you have over 100 (not uncommon for an image gallery) it leaves columns comically uneven, several screens of height different.

Absolutely True! I'm removing this library immediately.
This is what I need https://www.npmjs.com/package/react-photo-gallery?activeTab=readme


bpf: Handle ICMPv6 NS/NA in host firewall

In IPv6 mode, when the host firewall is enabled and rules are enforced, we start dropping ICMPv6 packets that are required to route packets. In particular, we can notice the following drops in cilium monitor:
xx drop (Policy denied) flow 0x1fc0ef0 to endpoint 0, identity 1->0: fd00::c -> f00d::a0f:0:0:dfa1 DestinationUnreachable(NoRouteToDst)
xx drop (Policy denied) flow 0x0 to endpoint 0, identity 0->0: fd01::c -> fd01::b NeighborAdvertisement

The nodes need to be able to exchange ICMPv6 NS and NA messages to establish routes. We already handle the response to NS messages on ingress, but when the egress policies are enforced, we start dropping outgoing NS and NA messages.
This commit fixes that by allowing and rejecting ICMPv6 messages according to RFC4890 Section 4.4. No other verifications than the types' are performed on the messages' correctness or their source IP addresses. Such messages from the pods are already handled on their egress, so we're not at risk of spoofing from pods here.
Handling of echo request and reply messages does not conform to RFC4890 as they can be filtered by the host firewall. That is to be consistent with our handling of ICMPv4 messages.
With this commit, we also stop answering to NS and echo request messages from the BPF program on ingress to the host. This behavior had been broken by a695f53, but we will now explicitly stop replying to those messages and pass them up the stack instead.
Updates: #11799
Fixes: #10994
Fixes: #11507

test-me-please

Coverage increased (+0.007%) to 37.045% when pulling 465977d3f8600a3fc7ab7439866bcfd6675a5db8 on pr/pchaigno/host-fw-handle-icmpv6 into 8aade7de709d5e30c6bc5dc247c934717ec4da20 on master.

test-me-please

K8S-1.17-Kernel-4.19 hit #10231. All other pipelines are passing.

@jedsalazar @joestringer Since you looked into these sort of things recently, are there any concerns with allowing ICMPv6 NS/NA messages through the host firewall?

The recent issues were specifically for router solicitations/advertisements, for what it's worth. But similar to the logic you point out in your commit messages, this program is in the wrong place to have any security impact from pods generating such messages. (I note also that the reason this was bad for pods was based on a threat model of someone gaining arbitrary execution access to a pod; that's a different threat model than someone having access to your physical network)
I see the Host FW as an L3 security feature here, where routed traffic destined to the node is subject to policy. ICMPv6 NS/NA, RS/RA are more about L2 adjacency.  I don't think it's our job to handle L2 network security here; users could legitimately configure their nodes directly adjacent to a router that sends IPv6 RAs and we will trust the underlying network. I think it's up to the user to configure their Linux boxes with the right settings for handling these messages. I note that in the PR as-is, this network configuration would actually break because we'd be dropping the router advertisements from the network. Fortunately, Cilium will log the drops pretty clearly in cilium monitor so if someone hit an issue with this, they should be able to find out reasonably quickly.
If we want to involve ourselves in L2 security features like this, we probably need to think it through and actually design it, complete with some clear way for users to configure it with what they anticipate the underlying network to legitimately allow; if we step in here and start messing with people's ICMPv6 peer discovery then I think we'll more likely break IPv6 in weird&wonderful ways while not actually improving security.

test-me-please

retest-4.9

Just a friendly ask: Is there anything blocking this change to be merged?

Just a friendly ask: Is there anything blocking this change to be merged?

@anfernee only the review from @joestringer  :smile:

All tests are passing. The pending tests are just because some of the GitHub Actions were renamed, but they were already executed under their old names before.

test-me-please


preserve file permissions on lxc file push

When pushing a file to a container it's permissions are reset.
When a source file is 400, the destination becomes 644.
File permissions should be preserved in pushing files in the same way permissions are kept when copying files.

Yep, that'd be good. A branch would be welcome (API already supports passing the wanted mode, so we just need to change the default to the source file's current value).

I'll work on a pr somewhere this week.
Op 6 jan. 2016 18:17, bij 18:17, "Stéphane Graber"<EMAIL_ADDRESS>schreef:

Yep, that'd be good. A branch would be welcome (API already supports
passing the wanted mode, so we just need to change the default to the
source file's current value).

Reply to this email directly or view it on GitHub:
https://github.com/lxc/lxd/issues/1472#issuecomment-169393545

Hey @jsimonetti any progress on this?

Unfortunately none sofar. I have been otherwise occupied. I should have some time this weekend or next week at the latest to get this done.


AKS: Subnet CIDR details not being displayed correctly

Setup
Rancher version: v2.7-0b2f7fa0f878c3402ddf416a5b622327949b9348-head
Browser type & version: Chrome latest
Describe the bug
Subnet CIDR details not being displayed as 'undefined'
To Reproduce

Create downstream AKS cluster and select Network policy & Network plugin as Azure, under Networking
Fill necessary fields and create clsuter
Wait for cluster to become Active
Edit cluster and check Networking details

Screenshots
While cluster creation:

Edit Cluster after creation:

Found while validating https://github.com/rancher/rancher/issues/42496

Adding MS

Blocked on https://github.com/rancher/dashboard/issues/10431

Validation results on build: v2.9-c1d92781561c0d4d5a18b32b2f312655dfe67e9c-head
Virtual Network (Subnet CIDR) displaying - None

Spec:
aksConfig:
    authBaseUrl: null
    authorizedIpRanges: null
    azureCredentialSecret: cattle-global-data:cc-dcgtt
    baseUrl: null
    clusterName: cpinjani-aks1
    dnsPrefix: cpinjani-aks1
    dnsServiceIp: <IP_ADDRESS>
    dockerBridgeCidr: null
    httpApplicationRouting: null
    imported: false
    kubernetesVersion: 1.28.9
    linuxAdminUsername: azureuser
    loadBalancerSku: standard
    logAnalyticsWorkspaceGroup: null
    logAnalyticsWorkspaceName: null
    managedIdentity: null
    monitoring: null
    networkPlugin: azure
    networkPolicy: calico
    nodePools:
      - availabilityZones:
          - '1'
          - '2'
          - '3'
        count: 1
        maxPods: 110
        maxSurge: '1'
        mode: System
        name: agentpool
        orchestratorVersion: 1.28.9
        osDiskSizeGB: 128
        osDiskType: Managed
        osType: Linux
        vmSize: Standard_DS2_v2
        vnetSubnetID: >-
          /subscriptions/<REDACTED>/resourceGroups/<REDACTED>/providers/Microsoft.Network/virtualNetworks/<REDACTED>/subnets/default
    outboundType: loadBalancer
    podCidr: null
    privateCluster: false
    privateDnsZone: null
    resourceGroup: <REDACTED>
    resourceLocation: eastus
    serviceCidr: <IP_ADDRESS>/16
    subnet: null
    tags:
      Account Type: group
    userAssignedIdentity: null
    virtualNetwork: null
    virtualNetworkResourceGroup: null

@cpinjani how was the cluster configured when created? The 'Virtual Network' input field will set aksConfig.virtualNetwork aksConfig.virtualNetworkResourceGroup and aksConfig.subnet - all of which are null in the cluster spec you shared, so the displayed value 'None' appears correct.

@mantis-toboggan-md Good call, so its worth checking from back-end perspective.


Clarify Difference Between timeSeriesPlayer and playClip

Both of these tools have the same help text:
    /**
     * Starts playing a clip or adjusts the frame rate of an already playing clip.  framesPerSecond is
     * optional and defaults to 30 if not specified.  A negative framesPerSecond will play the clip in reverse.
     * The element must be a stack of images
     */

but I assume they are used in different contexts.

I've merged #173, so I'm closing this. Thanks!


Installer: incorrect assumptions when going back from the bitcoind installation step

When clicking on "previous" while downloading Bitcoin Core:

It also happens if you go back without having pressed "Next". (Failed to find cookie file.)

Tested today:

press Previous while downloading => stop downloading and go to previous step.
cannot press Next while downloading.
press Previous after downloading and then press Next => bitcoind restart well.
Next is greyed out until bitcoind start

This issue looks fixed, i close it.


(MAINT) Use jruby-gem-list for all versions >= 5.3.0

This reverts commit d27a8cfadf097cd07ce9c78dc58292087b41d1d8.
Then just changes the gem-list file name to read from

Test FAILed

CLA signed by all contributors.

Added don't merge because I have a new branch for to target 5.1.x. Going to let the tests run to see if they work on master anyways

Test PASSed

Merged into 5.1.x as 40ece647953e4d117fbd691a4ed2f465df7009b2 and then merged up.


Issues with KivyMD DataTables in Metadata view

There are some known issues with KivyMD DataTables as of 0.104.1. It appears that they will be fixed in the next release, however.
I will probably hold off on the next naturtag release until those fixes are released.

Fixed in KivyMD 0.104.2


add demo png in docs

add demo png in docs.
So that the users can view the finial UI be intergrated.

LGTM


Added capital category and corrected existing file formatting

Corrected formatting in anime.py
Added capital.py for capital cities around the world. #42
Added the new category into pattern.py
Added test_capital.py

@gabrielbarker Sure, I didn't see in the docs that it could not be multiple words.  I can do a quick go back and remove the capitals that are combinations of words.  If you don't mind tagging this PR hacktoberfest-accepted, I'd appreciate that! :)

Great! Thanks @digitalvirtuoso ! I think this PR should count anyway as the repo is tagged by I'll add the tag just to be sure

Updated requested changes!  I appreciate the tag!

This looks great now @digitalvirtuoso , thanks so much for the contribution! 🎉


Cannot download package

I try to download any package via install command but it can not
Example display (I try to using Power shell and command prompt that same answer)
PS C:> vcpkg install grpc:x64-windows
Computing installation plan...
PS C:>

Duplicate of #18636: Don't use cmake 21, use cmake 20.x

Hi @yajokboy
Thanks for posting this issue.
Could you please help confirm if you encountered the same problem as #18636? If yes, please try to use cmake 3.20.x to rebuild.
https://github.com/microsoft/vcpkg/issues/18636#issuecomment-868555508

Hi @yajokboy
Thanks for posting this issue.
Could you please help confirm if you encountered the same problem as #18636? If yes, please try to use cmake 3.20.x to rebuild.
#18636 (comment)

Hi @NancyLi1013 I try to down version to 3.20.5 that worked , thank you


Allow the embedding of inline rolls in Item descriptions.

Journal Entries currently allow for the following:

It would be great if the same functionality could be implemented for spells and rituals (and other relevant Items), e.g.:

Currently works at some places in item descriptions. will be completed together with the rules schema changes.
Should also work in the chat item cards.

Replacement complete for Spells and Rituals.

Should work in all places, reopen if it's missing anywhere


Reorganize structs a bit.

This just moves Header and ShapeEntry into cacheformat namespace.
I cherry-picked this from v2 newCache work I did from scratch after the bug.
Next one would use these structs to pass filePointer, Header and ShapeEntry for ShapeRange constructor
but since getCubesByShape() is in .cpp now, I don't see the point doing it?

sry, I forgot your PR. Should I merge this as is, or could you rebase on byshape?

sry, I forgot your PR. Should I merge this as is, or could you rebase on byshape?

You can merge this onto byshape branch as-is, if it still applies cleanly otherwise I'll rebase.
(I'm techinally forced to do rebase at this point, since the branch is merged to upstream, but this is just a single commit)


Palindrome Pair

📚 Documentation
Given 'n' number of words, find if there exist any two words which can be joined to make a palindrome or any word, which itself is a palindrome.
The function should return either true or false.
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


Repository and license

When i put npm install discord.io winston --save it downloaded all of these files but it says "No repository field" and "No license field".

This is not a problem


Fix compaction threshold not configured in metadata namespace

Fixes https://github.com/streamnative/kop/issues/1272
Motivation
https://github.com/streamnative/kop/pull/448 introduced a bug that the
compaction threshold was not set in metadata namespace.
// NOTE: compactionThreshold is always null unless it has been configured
Long compactionThreshold = namespaces.getCompactionThreshold(kafkaMetadataNamespace);
if (compactionThreshold != null && compactionThreshold != MAX_COMPACTION_THRESHOLD) {
    // it never gets here unless compaction threshold has been configured
    namespaces.setCompactionThreshold(kafkaMetadataNamespace, MAX_COMPACTION_THRESHOLD);
}

In addition, currently there is no test that ensures the metadata has
been initialized correctly.
Modifications

Configure the compaction threshold when the metadata namespace is
created. Since the policies could only be configured when the
namespace didn't exist before, there is no need to check the existing
policy.
Add MetadataInitTest to test the metadata namespace has been
configured correctly. It also verifies the
kafkaManageSystemNamespaces config, which disables the
initialization of the metadata namespace.
Refactor the internalSetup method to avoid the redundant start of
broker.

Documentation
Check the box below.
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[x] no-need-doc
(Please explain why)

[ ] doc
(If this PR contains doc changes)

This PR relies on the changes of #1274.


make only output the libeuler_spiral_lib.a file

I executed the command but only get the libeuler_spiral_lib.a. No executable file generated. Could any one teach me the following step to get the plot shown in ReadMe? Thanks very much!

Thank you for your feedback.
How about this procedure?
$ mkdir build
$ cd build
$ cmake ../test/
$ make

Fixed so that demo can be built on Ubuntu as well


Add InteractiveExplorer, InteractiveEditor.  Creating a new branch, PR seems to have fixed the issue...

I was running into some odd issues with travis hanging before starting the tests and this PR is blocking #1694 . Since these visualizers are not used in any of the tests, I am assuming that they did not change anything.


Ref-RHIROS-1215 added link to group name column value

Add group link to column :boom:
https://issues.redhat.com/browse/RHIROS-1215
Why do we need this change? :thought_balloon:
It is a bug fix we need as part of card https://issues.redhat.com/browse/RHIROS-1215
Documentation requires update? :memo:

[ ] Yes
[ ] No

Security Checklist :lock:
Upon raising this PR please go through RedHatInsights/secure-coding-checklist
:guardsman: Checklist :dart:

[ ] Bugfix
[ ] New Feature
[ ] Refactor
[ ] Unittests Added
[ ] DRY code
[ ] Dependency Added

Additional :mega:
Feel free to add any other relevant details such as links, notes, screenshots, here.

Thank you for the review @saltgen 🙌🏻


Set up Intercom clone example

Set up an example project in the /examples directory  that shows how to design a chat widget that looks like Intercom's.

Grouping by date and showing avatar:

Show how long since last message and sender name (e.g. Bot - 1w ago)


[BUG] Some JSFX plugins GUIs are not showing correctly in BespokeSynth

Describe the bug
I tried a few JSFXs from the JoepVanlier JSFX repo and when opening them in the JoepVanlier ysfx vst3 they don't show the GUI correctly. With the jpcima ysfx the plugin's GUI is showed correctly.
This problem seems to happen only in BespokeSynth, AFAIK.
In Ardour the JoepVanlier ysfx vst3 load the JSFX plugin's GUI correctly.
To Reproduce
Steps to reproduce the behavior:

Load JopeVanlier ysfx vst3 in BespokeSynth
Click on "Load" end select Saike_Yutani.jsfx
The plugin is loaded but the GUI is not showing correctly

On the left jpcima ysfx vst3, on the right JopeVanlier ysfx vst3
See screenshot:

Desktop:

OS: Linux, Debian Sid
DAW: BespokeSynth Nightly
YSFX v0.0.23

Interesting. I just tried with bespoke 1.2.1 (on windows and ubuntu) and the latest nightly on windows. For me it's exactly the opposite. jpcima's comes up blank and ysfx-s shows the UI.
I think I need a little more information. Did you compile these from source or did you use released binaries?

What version of ysfx is this? Does it also happen if you load only ysfx-s (never load ysfx first in that same session)?
Is it possible something got cached funny?
Do you know at which release it stopped displaying the UI?

Did you compile these from source? Is it possible something funny happened with linking JUCE? For me, I end up having to wipe my build folder every time I switch between a commit before or after the transition to JUCE8. It seems that not everything triggers correctly to fully rebuild on my end (shouldn't be an issue on CI where the artefacts are produced though).
What version of ysfx from jpcima is this? Do you know which commit it was at?

I tried to delete the old jpcima ysfx and compiled again, starting fresh from scratch, the last jpcima's commit.
Now also the the jpcima's ysfx can't display Yutani GUI:

Does it also happen if you load only ysfx-s (never load ysfx first in that same session)?

Yes, compiled the latest JopeVanlier's ysfx, cleared the plugin cache, restarted BespokeSynth and the result is the same.

Is it possible something got cached funny? Is it possible to reset the VST cache in bespoke?

I tried to clear the cache and even removing all the plugins in .vst3, also removed every plugin cache file in BespokeSynth, just in case.

Do you know at which release it stopped displaying the UI?

No version is displaying Yutani's GUI. I tried also 0.02 and 0.04.

Can you show me the right side of the UI of ysfx-s? What happens if you click the scaling button?

latest ysfx-s, right side:

latest ysfx-s, scaled:

latest ysfx-s, resized

For completeness of information I'm using Xorg.
If in your tests with Debian 12 everything worked I suspect it could be some newer package in Sid that might create this issue.

Interesting. Just to be clear, yesterday, I didn't recompile on debian. I used the artefact from CI.
I just compiled it on Debian (in virtualbox) to see if the plugin works there and it did. So for me, it's really a mystery.
Would it be worth giving the artefact a shot? Or would that not be compatible?

It's a mistery indeed.
I just tried this one and it doesn't work. Feel free to close this issue, as for now I don't have any clue why it should work on Debian 12 and not on Sid, beside some newer package causing this issue. This case seems now most likely since I tried on another machine with Sid and I have the same problem, so it's not specific of my main machine. If I find some new clues I will let you know.

Curious. I'll leave it open for now, in case someone else notices the issue and has some ideas on things to try.

This issue is not host-related for me, I see the same error in every host.
Looks like the problem is this update commit https://github.com/JoepVanlier/ysfx/commit/ccc39a46c02f33ad35ee71c3f36b47a7582a5710
I reverted it manually and now all seems to work: https://github.com/KottV/ysfx/commit/b637ec16b193aeeeb96c2507d2b8458bf64d1ad9


Update template for LaxarJS v1.0.0

The application template needs to be modified because of:

LaxarJS/grunt-laxar#39
LaxarJS/grunt-laxar#29
LaxarJS/laxar#180
LaxarJS/laxar#88

Also, the latest ^1.0.0-alpha.* LaxarJS artifacts should be specified as dependencies in the bower- and npm-configuration.
These are mostly breaking changes, so we need to branch off master-0.x before implementing them.

Also incorporate changes for LaxarJS/grunt-laxar#44

Upgrade Information
In preparation for LaxarJS v1.0.0, the following changes have been made to the grunt-init template for LaxarJS applications.

The es5-shim has been removed (see LaxarJS/grunt-laxar#39)
The grunt-task portal_angular_dependencies is now named laxar_application_dependencies (LaxarJS/grunt-laxar#29)
The CommonJS module laxar-application has been added to the RequireJS configuration to represent the application itself (see LaxarJS/grunt-laxar#44).
The directive axPageFade has been moved to LaxarJS UiKit and is no longer included by default (LaxarJS/laxar#180).
The underscore library is no longer included by default (LaxarJS/laxar#88).
For production, CSS is now minified by default using grunt-contrib-cssmin.
The watch task is no longer triggered repeatedly by running the test task (due to test-results.xml files being created).
Inessential configuration options have been removed.
The LaxarJS artifact dependencies (core, UiKit, Patterns) have been changed to ^1.0.0-alpha.x.
Copyright year in the template sources has been bumped.
Use separate connect ports for grunt test and grunt server, so that they cannot interfere.

Feel free to modify your application after instantiating the template to undo any changes that you do not like.


How to set Set Rectangular overlay on Camera Preview

taking a look at this now. Would add this functionality to the next update.

I just added setCustomDetector(Detector<Barcode> customDetector) to help achieve this.
Here is an example of how you can achieve this:

public class CustomBarCodeDetector extends Detector<Barcode> {

    public SparseArray<Barcode> detect(Frame frame) {
        // do something with frame
        return null;
    }

    public boolean isOperational() {
        return super.isOperational();
    }
}

see this on for further clarification on how you can use this.
don't forget to call barcodeCapture.refresh() after calling  setCustomDetector()

Can you please release this feature

@kprathap23 available in the latest release.

I also encounter this issue. When can we use the new release?

Still getting this error
E/OpenCameraSource: Exception thrown from receiver.
java.lang.IllegalStateException: Detector processor must first be set with setProcessor in order to receive detection results.
at com.google.android.gms.vision.Detector.receiveFrame(Unknown Source)
at com.google.android.gms.samples.vision.barcodereader.ui.camera.CameraSource$FrameProcessingRunnable.run(CameraSource.java:1222)
at java.lang.Thread.run(Thread.java:818)

Hi @jpabanil, do you achieve the success result now?
Hi @KingsMentor, i can't find the setCustomDetector method (using gradle dependency). Would you mind to provide more details in the readme section and the params (preferably in Kotlin version though)? Thanks.


Single tile error stops whole render

Hello @neocturne, first of all congratulations on the Rust release! It might actually push me to learn some Rust 😀
I have noticed that the new version is more strict about errors, and if a problematic region means that a tile can't be rendered, the whole process fail — leaving no map at all. The C version would skip the tile, leaving a hole on the map, but the process would complete.
Like the "processing" phase, tile rendering could continue even if a tile is missing, leaving a hole in the map.
2023-10-08T12:55:14.767085Z  INFO Processing region files...
[...]
2023-10-08T12:57:55.734375Z DEBUG Processing region r.8.-2.mca
2023-10-08T12:57:55.734911Z ERROR Failed to process region (8, -2): Failed to read region header

Caused by:
    failed to fill whole buffer
[...]
2023-10-08T12:57:55.739075Z  INFO Processed region files (106 processed, 3 unchanged, 3 errors)
2023-10-08T12:57:55.739372Z  INFO Rendering map tiles...
[...]
2023-10-08T12:58:06.095434Z DEBUG Rendering tile map/0/r.7.-7.png
Error: Failed to render tile (8, -2)

Caused by:
    0: Region (8, -2) from previous step must exist
    1: Failed to get modified timestamp of file /home/fran/var/minecraft/minedmap/processed/r.8.-2.bin    2: No such file or directory (os error 2)

https://github.com/neocturne/MinedMap/blob/ceb71f035caa5214e19dda49c63b6ea4b36c200b/src/core/tile_renderer.rs#L327-L330
The problematic regions have size 0. I don't know I common this is, but it happens sometimes on my server 🤔

I have prepared a fix in #33. Will run a few more tests and likely make a fix release soon.

Thank you!


Azure CLI command for creating a Dynamic Scope with --filter-tags not working correctly

The Azure CLI command for creating a Dynamic Scope seems to have an issue.
The --filter-tags example is not working when I use the example from the documentation:
$ az maintenance assignment create-or-update-subscription --maintenance-configuration-id "/subscriptions/.../resourcegroups/foo/providers/Microsoft.Maintenance/maintenanceConfigurations/bar" --name cli_dynamicscope --filter-tags {{tagKey1:[tagKey1Val1,tagKey1Val2],tagKey2:[tagKey2Val1,tagKey2Val2]}}
Failed to parse '--filter-tags' argument: Shorthand Syntax Error: Expect character ',' or ']':
        {tagKey1:[tagKey1Val1}
                             ^

When I use only one value for the tags, the command is accepted, but only the second tag "tagKey2" is added to the filter options:
$ az maintenance assignment create-or-update-subscription --maintenance-configuration-id "/subscriptions/.../resourcegroups/foo/providers/Microsoft.Maintenance/maintenanceConfigurations/bar" --name cli_dynamicscope --filter-tags {{tagKey1:[tagKey1Val1],tagKey2:[tagKey2Val1]}}  --filter-tags-operator All -l global
This command is experimental and under development. Reference and support levels: https://aka.ms/CLI_refstatus
{
  "filter": {
    "tagSettings": {
      "tags": {
        "tagKey2": [
          "tagKey2Val1"
        ]
      }
    }
  },
  ...
}

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

ID: 92aebd16-d63e-55a1-05b5-9db3a6c2df93
Version Independent ID: 203fb0e1-0980-83e1-4afb-7cfc6653ff60
Content: Schedule updates on Dynamic scoping.
Content Source: articles/update-manager/tutorial-dynamic-grouping-for-scheduled-patching.md
Service: azure-update-manager
GitHub Login: @SnehaSudhirG
Microsoft Alias: sudhirsneha

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Gcode lines getting skipped

I'm not sure what's going on, so I made a video to show you the problem.
I think that some gcode lines are getting skipped.
Video
To be clear, this problem happened not every time I started the cycle.
Just by stopping the cycle and restarting it, the problem happened like 75% of the time the other 25% it worked just fine. I tested it around 50 times. It happened with the 2.0.37p1/2.0.36 but not with the 2.0.35 version of iosender. A full reset of the teensy and the iosender didn't change anything.
And one drilling operation of two holes ended up by drilling one hole two times.
Besides that I also got the error:33, error:36 and one time even error:2 in different gcodes and sometimes different errors or no error in one gcode then I ran the program multiple times.
By changing minimal things like radii or distances the gcode files sometimes worked, sometimes they didn't. The marked lines from the errors didn't differ from the other ones above or below.
Is it possible that this could be caused by a bad Ethernet connection or something related to the transmission.
A problem with the internal grblhal should reproduce the problem every time I guess as well as the iosender.
Everything I tried changing at fusion 360 didn't help.
Then I'm remembered correctly since I updated grblhal I didn't run a program to verify if it's a sender or a grblhal problem or both.
The gcode from the video and from the boring.
OneHoleTwoTimes.txt
RadialDesaster.txt

Hmmm, are you using telnet or websocket?  There was a telnet bug reported that was causing comm errors, iirc. If you are using prebuilt binaries, they might be older than that.  If telnet, try websocket.

Websocket solved the problem.
Thx a lot.


Pr/95

Updates https://github.com/JamesPHoughton/pysd/pull/95

Coverage increased (+0.02%) to 94.905% when pulling af497e824b3e5da5286d3b496b7011b1fe0fd55d on pr/95 into 88ae9ff49ca4d885b2f1cb8813afe7f79bca6ec9 on master.


Export Data (a)

In preference page UI. Add a tile saying 'Export all data'

Hey! I'm a beginner, can I work on this?
I'll add a Privacy tab under which there will be the required Export all data tile.

sure, I'll create a mock UI for you. for better understanding. :)

you can also work on #237 in a single PR. Task is just to implement UI with no (onPressed or onTap) logic.

Will this do?

yesss perfect


CHANGELOG: Update change log for 0.2.9 release

Signed-off-by: Fernando Fernandez Mancera<EMAIL_ADDRESS>

Coverage remained the same at ?% when pulling a75af8a8f6469dcc9520aeaa2cc154715c4e2770 on ffmancera:changelog029 into d23c782e2676f12560f0501b2b3526c6e6fa8592 on nmstate:nmstate-0.2.

Coverage remained the same at ?% when pulling a75af8a8f6469dcc9520aeaa2cc154715c4e2770 on ffmancera:changelog029 into d23c782e2676f12560f0501b2b3526c6e6fa8592 on nmstate:nmstate-0.2.

If you can add references to the fixes, it will be nice.

Yes, added. Thanks!


Training data for base model?

Hi!~Thank you for sharing your excellent work.
May I ask for training data for base model for my further research?

Please refer to https://github.com/sstzal/DFRF/releases/tag/Base_Videos.


Ampersands not rendered in tray icon tooltip

OS: Windows 10 Pro x64
Issue Descriptions: When a song title (and presumably an artist name, but not tested) contains an ampersand (&), it isn't rendered in the tray icon's tooltip.

Steps to Reproduce: Play a song that has an ampersand in the track title, then hover over the tray icon.

This appears to be a windows thing 😆 "&&" as "&" will render and "&" will not 😕 ....
Will add a simply replace to fix this 😆

Hey I've never helped with open source, can I give this a whirl?

@fraserhaer Sure, you probably want to look at the two lines in this file where we set the tooltip
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/blob/40ee551fef2520b928d2175b3ab4b885d6dd51f8/src/main/features/core/windowTitleInfo.js#L13
And then do string replacement on the generated string to replace all "&" with "&&" 👍


Compatibility with Qwik

Hello,
I’m interest in know if Tremor have compatibility with new Framework Qwik.
Regards thank you very much

Qwik supports both React and Tailwindcss, so Tremor should work there without any problem.
The only thing is that some Tremor components have "use client" at the beginning of their files to make them compatible with Next.js 13. If that's not a problem I guess you are good to go!

Closing as covered by @amirhhashemi 🙏


Small but annoying typo

In this line there is a #0 and not a #10 that cut the project in a new wizard:
https://github.com/VSoftTechnologies/DUnitX/blob/402bfc20da7e1ddb3b94086cb13aaff65ff92764/Expert/DUnitX.Expert.CodeGen.Templates.pas#L58

Thanks, just pushed a fix.