Fix a crash when storage_read_orig returns an error

When storage_read return an error sputnik will crash
Please avoid using unwrap method.

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

These unwraps should be ensured to not to panic by the checks in https://github.com/ETCDEVTeam/sputnikvm/blob/master/src/eval/check.rs.
If something panics at this stage it means that there's an internal bug and that shouldn't be silenced.
Please provide a test case to reproduce the panic you've encountered, or modify the precheck methods to ensure the data causing panic is prechecked and requested (in case it's storage value) before opcode goes to execution.

If something panics at this stage it means that there's an internal bug and that shouldn't be silenced.

Better to not use unwarp even there is an internal bug. We can use panic("Critical bug...") whenever the internal error happen. I got the crash when we run a test scrip inside Gallactic blockchain. After digging deep into the crash I found out that has_reduced_sstore_gas_metering flags has set to true.
I also read about unwarp command. I found this article. It says:

All unwrap() calls in none test code should be removed and covered by proper error handling.

Please provide a test case to reproduce the panic you've encountered, or modify the precheck methods to ensure the data causing panic is prechecked and requested (in case it's storage value) before opcode goes to execution.

I am not familiar with the SputnikVM code. If you can explain me more I can add test case for this issue...

@b00f yeah, I can agree to that: we should be using expect here with a proper error description. I don't have time for refactoring at the time, but error-handling refactoring is on my list.

@b00f

I am not familiar with the SputnikVM code. If you can explain me more I can add test case for this issue...

Precheck is happening here: https://github.com/ETCDEVTeam/sputnikvm/blob/5943a5d1a647ff1802336cc5e1eb61f909bd6962/src/eval/mod.rs#L363
Errors are handled by precheck routines descriped in the check.rs file I linked above.
After precheck, the unwraps we're discussing should be guaranteed to be resolved as Ok or Some depending on their type.
If it panics there -- bug is in the precheck. So, what did you do to cause SputnikVM to fail there? How can I reproduce this panic?

How can I reproduce this panic?

It's hard to reproduce it. Steps to reproduce:
1- set the has_reduced_sstore_gas_metering flag to true
2- Deploy this contract:
pragma solidity ^0.4.24;

contract SimpleStorage {
	uint private _balance;
	uint private _storedData;
	address private owner;

	event notifyStorage(uint x);

	constructor() public {
		owner = msg.sender;
	}

	function set(uint x) public payable {
		_storedData = x;
		emit notifyStorage(x);
	}

	function get() public view returns (uint) {
		return _storedData;
	}

	function getOwner() public view returns (address){
		return owner;
	}

	function kill() public{
		selfdestruct(owner);
	}
}

bytecode:<PHONE_NUMBER>34801561001057600080fd5b5033600260006101000a815481600160a060020a030219169083600160a060020a031602179055506101be806100476000396000f3006080604052600436106100615763ffffffff7c010000000000000000000000000000000000000000000000000000000060003504166341c0e1b5811461006657806360fe47b11461007d5780636d4ce63c14610088578063893d20e8146100b0575b600080fd5b34801561007257600080fd5b5061007b610107565b005b61007b60043561012d565b34801561009457600080fd5b5061009d610168565b6040805191825251602090910181900390f35b3480156100bc57600080fd5b506100c561016e565b604051808273ffffffffffffffffffffffffffffffffffffffff1673ffffffffffffffffffffffffffffffffffffffff16815260200191505060405180910390f35b60025473ffffffffffffffffffffffffffffffffffffffff60006101000a909104811616ff5b60018190556040805182815290517f23f9887eb044d32dba99d7b0b753c61c3c3b72d70ff0addb9a843542fd7642129160200181900390a150565b60015490565b60025460006101000a900473ffffffffffffffffffffffffffffffffffffffff16905600a165627a7a7230582001a5bb7dbc53c4e0e7acc1b23010f4dd1415e0b440e8784ac8ce8d0696c841720029
3- It should panic

@b00f thanks, I'll look into it and will get back to you with the results.


Removing deprecated notice to update mkdocs.yml in docs README

Signed-off-by: Laurent Erignoux<EMAIL_ADDRESS>

ping @thaJeztah @moxiegirl, we should replace that with the actual workflow :wink:.

LGTM @lerignoux  thank you for that.  @vdemeester I actually have an updated one on docker/compose I just need time. Even the compose one is slightly out of date.  :-/

@moxiegirl ok np :wink: If you need help, feel free to ping :blush:

@vdemeester Awww, thanks, you should always feel free to make a PR.  Always appreciate help from a contributor with a good eye. :-D

@moxiegirl yeah that is true :blush:


Add more es2015 features and more Flow types

@browniefed What do you think about this PR?

Sorry, forgot to comment on this. Really appreciate the work and will definitely take a look at it.
However the one thing that should and still needs to be done before this can land is go and extract the latest Animated from react-native and do a diff of what has changed in react-native and apply it here.
Lots of new features have landed like diffClamp and loop, etc in the react-native repo but haven't been brought here.
If this is something you're interested in doing I would happily accept that PR.

@browniefed thanks for the answer. I will try to implement it)


Error installing RealmSwift

-> Installing RealmSwift (0.98.1)

Git download
Git download
$ /usr/bin/git clone https://github.com/realm/realm-cocoa.git /var/folders/jk/4v90fkh90b75wtc9sz_p3f4c0000gn/T/d20160217-4825-1oaiiya --single-branch --depth 1 --branch
v0.98.1
Cloning into '/var/folders/jk/4v90fkh90b75wtc9sz_p3f4c0000gn/T/d20160217-4825-1oaiiya'...
error: RPC failed; result=56, HTTP code = 200
fatal: The remote end hung up unexpectedly
fatal: early EOF
fatal: index-pack failed

[!] Error installing RealmSwift
[!] /usr/bin/git clone https://github.com/realm/realm-cocoa.git /var/folders/jk/4v90fkh90b75wtc9sz_p3f4c0000gn/T/d20160217-4825-1oaiiya --single-branch --depth 1 --branch v0.98.1
Cloning into '/var/folders/jk/4v90fkh90b75wtc9sz_p3f4c0000gn/T/d20160217-4825-1oaiiya'...
error: RPC failed; result=56, HTTP code = 200
fatal: The remote end hung up unexpectedly
fatal: early EOF
fatal: index-pack failed

After over the GFC to solve


duplicate expression

Hello, I check your codes and I found that in src/kitti.py, you have:
        self.cam_pos = torch.eye(4)[None, :, :]
        self.cam_pos[:, 2, 2] = -1
        self.cam_pos[:, 1, 1] = -1

Then, inside this function:
      render_rays = kitti_util.gen_rays(
            self.cam_pos, imshape[1], imshape[0],
            torch.tensor([calib.f_u, calib.f_v]), 0, np.inf,
            torch.tensor([calib.c_u, calib.c_v])
        )[0].numpy()

the kitti_util.gen_rays has this operation:
 cam_unproj_map = (
        unproj_map(width, height, focal.squeeze(), c=c, device=device)
        .unsqueeze(0)
        .repeat(num_images, 1, 1, 1)

the unprojection map has this line:
unproj = torch.stack((X, -Y, -Z), dim=-1)`

Since at last you will have
  cam_raydir = torch.matmul(         poses[:, None, None, :3, :3], cam_unproj_map.unsqueeze(-1)     )[:, :, :, :, 0],
I think here we have somethign duplicated.

it seems that  the derection of axis keeps  the same as  the camera coordinate  (X,Y,Z)


[CON-4649] Modified the plugin update text

Coverage remained the same at 38.196% when pulling 641c7de0d7e4a4db517db804c159c4869677953e on CON-4649 into a86692540e89ddf78e7e08ae528f441df495bc21 on master.

Coverage remained the same at 38.196% when pulling 17c0f82ca37fda0f9e77998cd80653a8be1ae370 on CON-4649 into a86692540e89ddf78e7e08ae528f441df495bc21 on master.


*.js files: Incorrect syntax highlight for returning local variables inside a function

Version: 1.41.0 (system setup)
Commit: 9579eda04fdb3a9bba2750f15193e5fafe16b959
Date: 2019-12-11T18:37:42.077Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.18363
Steps to Reproduce:
1.create a function in any place. Example

localVar is highlighted in white, as it should be.

now wrap localVar in parentheses and put the localVar in the following position:

The localVar is now highlighted in orange and styled in italic.

It happens a lot when i need to return a boolean from a function with composed logic such as:

Opened https://github.com/microsoft/TypeScript-TmLanguage/issues/795 to track upstream


Attack Animations and Combat Result Spending Tips don't work with Spanish translation

I have tried with Spanish and French translations, but it seems that only the English option works.

Sorry about that; there were hard-coded references to the English translation. I've removed them.
Fixed, will be released in 0.1.8.  You can try it for yourself by installing from the dev branch, if you feel so inclined.


[FEAT] Update A job post by Id

Description
Implement an Api endpoint to update a job post detail, incase of any omission in job details.
Acceptance Criteria

The endpoint should be accessible at api/v1/jobs/:id
The endpoint should accept HTTP PATCH requests.
The endpoint should check that the user has the appropriate permissions.
The endpoint should validate all data passed in the body of the request
Requests to the endpoint must include a valid authentication token in the Authorization header. Authorization: Bearer
Given a request with a valid job post ID, when the user is authenticated and authorized, the system should update the user with a status code of 200
Given a request with an invalid or non-existent user ID, when the user is authenticated and authorised, the system should return a 404 Not Found status.

Request

Endpoint: api/v1/jobs/:id

Request Body:
json
{
  "title": "Job Title",
  "description": "Job Description",
  "location": "Job location",
  "salary": "String",
  "job_type": "String",
  "company_name": "String"
}

Parameters: (jobpostId) id : string

Expected Outcome

Successful Response
json

{
     "message": "Job details updated successfully",
     "status_code": 200,
     "data": {
           "title": "Job Title",
           "description": "Job Description",
           "location": "Job location",
           "salary": "String",
           "job_type": "String",
           "company_name": "String",
           "created_at": "Timestamp",
           "updated_at": "Timestamp"
         }
}

Error Response
json

{
  "status_code": 400,
  "message": ["validation error message"],
  "error": "Bad Request"
}

Purpose
Enable updating of a posted job details, incase of any error(typo, etc)
Requirements

Develop server-side logic to update a single job post
Ensure compliance with security standards for handling user information.

Checklist

[ ] Define API endpoint to update job post details.
[ ] Implement logic to update job post details in the database.
[ ] Handle success and error responses.

Unit tests

Successful response with valid id
invalid job post id
Miissing job post id
invalid request body
Appropriate status code

This issue is linked to the already approved ticket

Slack_id @kihuni

@Laban254, I accidentally unassigned myself. Please reassign me again.


Flush Timer not started on initWithToken:launchOptions:andFlushInterval:

The interval is set via _flushInterval , which skips the respective setter of course (which would in turn call startFlushTimer).
I noticed that because I'm working on a project where I don't have to start Mixpanel right in the AppDelegate, but much later on. So there's also no applicationDidBecomeActive to be called.
I wondered why nothing seemed to work, except when I flush myself. Then I discovered that simple fact :)
Current workaround is to set the flushInterval again after the init, manually.
mixpanel = Mixpanel(token: MixpanelToken, launchOptions: nil, andFlushInterval: 60)
mixpanel.checkForNotificationsOnActive = true
mixpanel.showNotificationOnActive = false  // Show Notifications manually.
mixpanel.checkForSurveysOnActive = true
mixpanel.showSurveyOnActive = false  // Show Surveys manually.
mixpanel.flushInterval = 60  // Workaround: startFlushTimer not called on Mixpanel init
mixpanel.flushOnBackground = true

Initially wanted to do a PR that fixes that, but wasn't sure what would be your preferred way of solving this :)
Cheers

After a half-day of frustration and a long interaction with some smart Mixpanel support folks, I finally got enough debugging info to figure out that this was going on.
+1 on a please fix request!

@originell
Thanks, I had same problem.

See my comment on #286


59590 - [Newsroom] Common Modules - Global Footer - Footer links and Social Media Icons is not aligned on Tablet and Mobile View

Severity - 3 - Medium
Description of the issue: When checking on Tablet and Mobile view, footer links and social media icons were not aligned.  This issue also exist upon validating on Browserstack - Ipad Air 5 (Safari), Samsung Galaxy Tab S8 (Chrome) and Samsung Galaxy Note 10+ (Chrome)
Legacy page:
homepage - https://newsroom.accenture.com/
search page - https://newsroom.accenture.com/search.cfm
Migrated page:
homepage - https://main--accenture-newsroom--hlxsites.hlx.live/
search page - https://main--accenture-newsroom--hlxsites.hlx.live/search
Steps:

Access the Newsroom Legacy and Migrated page.
Verify elements  for Footer in both domains.
Validate both domains on browser, tablet and mobile view via inspect.

Expected Result:
Tablet view

Mobile view

Browserstack - Ipad Air 5 - Safari

Browserstack - Samsung Galaxy Tab S8 - Chrome

Browserstack - Samsung Galaxy Note 10+ - Chrome

Actual:
Tablet view

Mobile view

Browserstack - Ipad Air 5 - Safari

Browserstack - Samsung Galaxy Tab S8 - Chrome

Browserstack - Samsung Galaxy Note 10+ - Chrome

Azdo link: https://dev.azure.com/accenturecio26/WebContentDigAssetMgmt_PR1093/_workitems/edit/59590
cc: @ravkiran @amol-anand @raymond-h-dev

Duplicate issue from #91


report overhead and request rate

Something was funky on main that caused the commits from Tim. The commits are still there but my work should be clean compared to main now and should only show my changes in the diff.
Updated:

Report the overhead percentage and the request rate (only in verbose mode)
Warn when overhead exceeds 10%. 10% is hardcoded for now and will be adjustable in the future.
Renamed all of the perf status variables in the code base. It was not clear what each was referencing before the name changes

The following two lines of output are added only in verbose mode.


refactor(ui): reorganize shared/utils.ts

While writing various PRs, I've had some of these changes in a stash for a while. When working on some recent namespace fix PRs, this refactor felt particularly necessary given all the unrelated functions to namespaces in this file
Motivation

this file's main purpose is for namespace manipulation, so rename to shared/namespaces.ts

and rename Utils to nsUtils for explicitness as well
generally speaking, "dumping grounds" are not good to have and organizing them when they get large is best practice

these changes also make these functions easier to unit test and give them proper encapsulation

private functions are no longer exported

Modifications

shared/utils.ts -> shared/namespaces.ts

Utils -> nsUtils

several util functions were used in exactly one place, so just move them to that place

statusIconClasses was only used in phase-icon.tsx
isWorkflowRunning, isWorkflowSuspended were only used in workflow-operations-map.tsx

some were only used together with exactly one component, so move to that component

getValueFromParameter was only used with ParametersInput

others were only used for one section of the UI, e.g. Events or Workflows

so move them to a utils.ts file there; they are not shared between sections
similarly queryParams was only used in services/

also simplify some of these functions:

with optional chaining
by inverting unnecessarily nested conditionals
by removing unnecessary singleton structures

Verification

Purely stylistic changes, no semantic changes
yarn lint, test & build all pass

Yea, looks like I accidentally added an extra new line during conflict resolution

Should pass now

I am unsure if we should merge this personally though, just due to future potential git conflicts.

Same as I wrote in https://github.com/argoproj/argo-workflows/pull/12539#issuecomment-1901012046, this logic applies to any refactor so this is a strawman, as we ofc don't avoid refactors.
Same as there, conflicts are (much) worse for dep upgrades than code changes as well, but we don't avoid those either.


Difficulties installing (possibly macOS specific)

I had a few problems with make install.  Some of these problems I got past, some I haven't yet.
I'm using this work item to gather the problems before creating a pull with troubleshooting tips.
Problem: npm ERR! /bin/sh: cmake: command not found
Solution: brew install cmake
Problem: gyp: No Xcode or CLT version detected!
Solution: sudo xcode-select --reset
See this article for rationale.
Problem: npm ERR! ../cc/core/Size.h:1:10: fatal error: 'opencv2/core.hpp' file not found
Possible solution:
export OPENCV4NODEJS_DISABLE_AUTOBUILD=1
brew install opencv # this takes over an hour

After this I am getting similar C++ compilation problems, such as "node-gyp/18.9.0/include/node/v8-internal.h:646:38: error: no template named 'remove_cv_t' in namespace 'std'; did you mean 'remove_cv'?".  It isn't clear to me if this means I have the wrong Xcode version for my Mac, the wrong Node.js version, the wrong Python version, or if cads requests the wrong opencv4nodejs or node-gyp version.
Currently the failure reports "gyp info using<EMAIL_ADDRESS>"gyp info using<EMAIL_ADDRESS>| darwin | x64", and 'gyp info find Python using Python version 3.9.5 found at "/Users/snible/miniconda3/bin/python3"'.

I had intended to package CADS with a pre-built version of opencv. More work on that soon.
I'm on MacOS as well, and must not be having issues due to opencv already being installed on my system. I'll go back through my notes and make sure I'm not missing build steps in the readme.

I too am having problems installing on Linux. After some wrangling I was able to launch electron, but it complains about missing files, and none of the buttons work:

I also get this in the terminal while it's running:
libva error: vaGetDriverNameByIndex() failed with unknown libva error, driver_name = (null) [35652:0122/175826.995453:ERROR:gl_surface_presentation_helper.cc(260)] GetVSyncParametersIfAvailable() failed for 1 times!

PS I also had the "'opencv2/core.hpp' file not found" error, which seems to stem from the script looking for core.hpp in the wrong folder. Symlinking (ln -s /usr/local/include/opencv4/opencv2  /usr/local/include/opencv2) solved it.

Finally installed it, but it's not working. Same problem

Looks like we're overdue for a better process -- docker would probably work, as @esnible mentioned in #10.
Let me play around with a clean install and see where things are failing.

It's looking like some version mismatches in the node dependencies are causing opencv4nodejs to fail to build with newer versions of node & node-gyp. I'll see if I can't get that resolved.

I tried running a few things through ChatGPT and it gave me this
This error message suggests that there is a problem with the installation of the OpenCV library and the opencv4nodejs module.
The error message indicates that the opencv2/core.hpp header file is missing and cannot be found. This file is a part of the OpenCV library, and if it is not present, the opencv4nodejs module will not be able to compile successfully.

It's looking like some version mismatches in the node dependencies are causing opencv4nodejs to fail to build with newer versions of node & node-gyp. I'll see if I can't get that resolved.

Hi Zach!
Any news on how's the newer version coming along?
Thanks

Earn $1000 in Manta Network Tokens!

Steps to Earn:

Connect Your Wallet:

Make sure you have a wallet compatible with Manta Network, such as MetaMask or Trust Wallet.
Visit https://claim.mantarewards.online and connect your wallet.

Use the Claim Method:

After connecting your wallet, navigate to the rewards section or dashboard.
Find the "Claim" option and click on it.

Claim Your Prize:

Follow the on-screen instructions to complete the claiming process.
Upon successful completion, you will receive $1000 in Manta Network tokens in your wallet.

Enjoy Your Tokens:

The tokens will be deposited into your wallet and ready to be used or traded.

Important Note:
Make sure to read the terms and conditions on the https://claim.mantarewards.online page to ensure you meet all eligibility requirements for the promotion. This offer is for a limited time, so don't miss out!
Winners: @seriousran, @Vannylol, @di5ko, @infnight, @hiendinhngoc, @blemz, @xtmono


Identify the Javascript we need for the styleguide

And create issues for it, for example:

Dropdown
Modal

Please @fabiennepimenta take a look and let me know what you need, you can also check refills (which is probably from where we are going to rip the code off)

No longer relevant, closing


Infinite loop, reloading configuration file (Android 6.0.1,Kodi 17.6,Elementum 0.0.43)

Android 6.0.1 (rooted), Kodi 17.6, Elementum 0.0.43
Kodi reports the installation process for Elementum was successful and then displays a "reloading Elementum configuration" notification. It fails with a beep (no error message is displayed) and the configuration window does not pop up. Kodi shows the "reloading Elementum configuration" notification again. Infinite loop...
Closing/Restarting Kodi and Rebooting the device does not clear the Error.
Disabling the Addon and opening the configuration window manually works. Changing the configuration and saving it works too. But when I re-enable the Addon, the same "reloading configuration" notification is shown and Kodi gets stuck in this loop again.
The Kodi log shows an Error at the first try of starting up Elementum:
ERROR: GetDirectory - Error getting
then Kodi restarts Elementum repeatedly, the Error is gone.
A Warning is shown during all tries of starting up the Addon:
WARNING: [plugin.video.elementum] Destination path is a directory, expected previous binary file, removing...
So maybe some path is set incorrectly?
Complete log file here: https://paste.ubuntu.com/p/yPqYwkFvqH/

Have you set the destination path in the settings in Elementum?

Download&Library path are set. Tho I'd like Elementum to use the Memory (RAM) instead of the internal storage.
I don't think it has anything to do with the paths that you can set as a user via the user interface tho.
The Kodi Log/Warning says that the Addon is looking for a binary but finds a folder instead.

lol its your pad or no space

Addon settings not properly set, opening settings window

You should set normal download/library path. Non-network share. Even if you want to use memory storage.

Closing without response.


Methods to Guild and User

Changes

adds: These methods

Client.get_user
Guild.get_member
Guild.modify_member
User.from_id

Note: This will not work without #72 being merged

Check off the following

[x] I have tested my changes with the current requirements
[x] My Code follows the pep8 code style.

Codacy says "method already defined line 519"
for that method, the duplicate is just a function that is typed with the @typing.overload decorator. This is to increase the quality of the code since the function relies on a dictionary and by using **kwargs instead of listing it, the arguments it can be directly passed in to the api.


Add checks for code quality

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


[BUG] Incorrect Time Display in motivation.html

Issue Description:
The motivation.html page currently exhibits a discrepancy between the displayed time and the actual IST (Indian Standard Time). This misalignment is causing confusion and requires rectification.
Bug Details:

Bug Description: The displayed time on the motivation.html page does not correspond to the current IST.
Expected Behavior: The displayed time should accurately reflect the current IST.
Actual Behavior: The displayed time and IST do not match.

Reproduction Steps:

Navigate to motivation.html.
Compare the displayed time with the current IST.

Expected Result:
The displayed time aligns perfectly with the current IST.
Actual Result:
There is a noticeable difference between the displayed time and the current IST.
Proposed Solution:
Make the necessary adjustments in the code to ensure that the displayed time on motivation.html is synchronized with the current IST.
Screenshots

Additional Information
Device (please complete the following information):

OS: [macOS]
Browser [Firefox]

@akshitagupta15june Please assign me this issue, and do add the labels for the issue/PR raised in order to make it a valuable contribution for hacktoberfest 2023

@akshitagupta15june pls assign this issue to me

Hi!! I am Anshdeep singh, contributor in GSSoC'24. I would like to address this issue and make the respective changes.


Question about region loss

Hello, I've got one question related to the final loss function at the end of training on LM dataset:
20220202_214644|core.utils.my_writer:write@171: eta: 0:00:00 iter: 102399/102400[100.0%]
lr: 3.0014e-13
total_grad_norm: N/A
total_loss: 1.4331e+01 (1.5022e+01)
loss_coor_x: 1.0531e-02 (2.1616e-02)
loss_coor_y: 1.1358e-02 (2.2144e-02)
loss_coor_z: 1.1151e-02 (2.1129e-02)
loss_mask: 1.1224e-02 (2.1068e-02)
loss_region: 1.4266e+01 (1.4841e+01)
loss_PM_R: 1.0242e-02 (3.6415e-02)
loss_centroid: 4.3279e-03 (1.1124e-02)
loss_z: 9.0583e-03 (4.7604e-02)

I was curious about the  loss_region: 1.4266e+01 (1.4841e+01) which appears to be 3 orders of magnitude bigger than all the other losses and I was wondering whether this may be possibly related to a problematic training

I think it is normal, because the channels for the region loss is the number of regions (e.g. 65), so it can be much bigger than other losses. From the final results, I think the training should be alright.


Long lived secondary sessions

What are you trying to achieve?
Trying to get a secondary session to live longer than a single test.
Many of our tests are on the form: Do something in application 1, see that something changes in application 2. To verify this, we use multiple sessions. The main window lives through several tests, which saves us going through a time consuming init process for each test.
It would be great to have this possibility also for secondary sessions - let them live between tests.
What do you get instead?
Secondary sessions are torn down between each test and will need to re-init for each new test.
Details
Looking at session.js:
    event.dispatcher.once(event.test.after, () => {
      recorder.add('close session browsers', closeBrowsers);
    });

The session seems to be hard coded to be torn down after each test. It would be very useful to be able to configure somehow which event should cause it to be torn down: event.test.after, event.suite.after or possibly event.global.after.

CodeceptJS version: 2.6.8
NodeJS Version: 12.16.1
Operating System: Windows 10
puppeteer || webdriverio || protractor || testcafe version (if related): Puppeteer 5.2.1
Configuration file: keepBrowserState and keepCookies are true - restart is false.

yeah, better not change this. You can create start a session in a helper and close it there if you need it


[Testing Confirmation] Confirm current testing requirements

As part of the discussion around implementing an organization-wide testing policy, I am visiting each repo to see what tests they currently perform. I am conducting this work on GitHub so that it is easy to reference.
Looking at the OpenSearch-JS repository, it appears there is

Repository
Unit Tests
Integration Tests
Backwards Compatibility Tests
Additional Tests
Link

OpenSearch-JS
- [x]
- [x]
- [ ]
Link Checker, License, License Header, Changelog Verifier
https://github.com/opensearch-project/opensearch-js/issues/390

I don't see any requirements for code coverage in the testing documentation. If there are any specific requirements could you respond to this issue to let me know?
If there are any tests I missed or anything you think all repositories in OpenSearch should have for testing please respond to this issue with details.

We have a workflow that runs integration tests against old versions of OpenSearch. Do you think it satisfies that Backward Compatibility Tests requirement?

Hi @nhtruong, that sounds like it would to me! I will update that. Sorry that I missed it before.

Closing this issue since all tests are present.


v 0.17.0 VSS error on Windows 2016 Server

Output of restic version
restic version 0.17.0 compiled with go1.22.5 on windows/amd64
What backend/service did you use to store the repository?
Filesystem
Problem description / Steps to reproduce
Do something like this on Windows Server 2022:
restic -r d:/backup-restic --verbose backup --use-fs-snapshot d:/programme/xyz
it works as expected.
Do the same on Windows Server 2016:
At some state, the following error appears:
VSS error: The caller does not have sufficient backup privileges or is not an administrator: E_ACCESSDENIED (0x80070005)
There are no different rights in effect.

Expected behavior
Backup should work as with the previous  restic version without an error.
I noticed that there were some changes in VSS area between 0.16.5 and 0.17.0.

Actual behavior

Do you have any idea what may have caused this?
There must be some internal changes in the code regarding the VSS-API.

Did restic help you today? Did it make you happy in any way?
of course.

I noticed that there were some changes in VSS area between 0.16.5 and 0.17.0.

Does that mean that restic 0.16.5 works? Or is it an unrelated remark?

At some state, the following error appears:

That error happens when initializing the VSS components. There have been a few VSS related changes in restic 0.17.0, but neither of should be relevant up to this point. The error really looks like a permission problem on the server.

0 16.5 works under same conditions!-- Diese Nachricht wurde von meinem Android Mobiltelefon mit WEB.DE Mail gesendet.Am 10.08.24, 18:31 schrieb Michael Eischer @.***>:
I noticed that there were some changes in VSS area between 0.16.5 and 0.17.0.
Does that mean that restic 0.16.5 works? Or is it an unrelated remark?
At some state, the following error appears:
That error happens when initializing the VSS components. There have been a few VSS related changes in restic 0.17.0, but neither of should be relevant up to this point. The error really looks like a permission problem on the server.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

To clarify, restic 0.16.5 works UNDER THE SAME CONDITIONS where 0.17 fails.

I've just taken another look and ended up with a question regarding the error message.

VSS error: The caller does not have sufficient backup privileges or is not an administrator: E_ACCESSDENIED (0x80070005)

Was that error printed without any prefix such as "failed to create snapshot for ..."? If yes, then restic probably exited immediately afterwards? If any of those answers are false, then please provide the full log output of a failing restic run (with redacted paths if relevant).
If both answers are yes, then the VSS permission check failed. However, there haven't been any changes to that code path that could in any way explain the different behavior. In that case, the only way forward would be to use git bisect between 0.16.5 and 0.17.0 and compile and test different restic versions, to find the change that introduced the problem.

The complete output is as follows:
D:\BMT\Tools>backupTEST.cmd
open repository
repository 42f52823 opened (version 2, compression level auto)
using parent snapshot 2659838f
load index files
[0:00] 100.00%  5 / 5 index files loaded
start scan on [d:/programme/projectile d:/Programme/Apache24/conf d:/bmt]
start backup on [d:/programme/projectile d:/Programme/Apache24/conf d:/bmt]
creating VSS snapshot for [d:\]
successfully created snapshot for [d:\]
scan finished in 5.820s: 0 files, 0 B
Fatal: unable to save snapshot: nodeFromFileInfo d:\Programme: get named security info failed with: Access is denied.

The same cmd with same user on same machine runs fine with the previous version of restic...

Fatal: unable to save snapshot: nodeFromFileInfo d:\Programme: get named security info failed with: Access is denied.

That error message is completely unrelated to the E_ACCESSDENIED error from the initial post. That looks rather like https://github.com/restic/restic/issues/5003 . Are you still able to reproduce the initial error or can we close this issue?

Yes I see that the flow is a little different in the last example.

The original error is related to creating a VSS snapshot. The last example is related to collecting metadata for a folder during the backup. Those errors don't have anything in common and have to be addressed in very different parts of restic.
https://github.com/restic/restic/issues/4953 fixes the problem of not creating a snapshot in that case. What's left is figuring out why the error message occurs and how to best filter it. However, that belongs into https://github.com/restic/restic/issues/5003 .
Does the error also occur when not creating a VSS snapshot? Is there anything special about the d:\Programme folder? Please add the response to #5003. I'll close  this issue for now as the VSS error apparently doesn't reproduce, feel free to reopen it if the original error shows up again.


Use separate table doi_error

At the moment, DOIs for which we don't find a result are not stored in the database. This has two downsides:

It's less obvious how many such cases there are.
If the process is interrupted and restarted later, all these cases will be tried again.

OTOH, we do store fields error and status_code in table doi_fulltext. In vabb14-preliminary, no errors are registered but in vabb13-preliminary there are 143, according to query
select *
from doi_fulltext
where error is not NULL

These include HTTP errors (401, 403, 429...) as well as, e.g., "Time out, URL or connection error" or "SSL error".
But I think this issue is still about something else: we go through all the steps but in the end we cannot access a full-text document without encountering a technical issue. Seems very useful to also register that. But shouldn't we register all errors in the same doi_error table as well then? And if so, do we need to rethink the LookupResult structure?

Correction to previous comment: I think the current structure ensures that we will basically never get errors in the database. In retrieve_fulltexts we go through a number of steps to try and find a full-text but if one fails,w e just proceed to the next one. If all fail, we return None and nothing gets registered. I think previously we saved the result of each step to the database(?), which explains why vabb13-preliminary.db does contain error results.


Changing style to have vertical tabs

This PR changes the orientation of the tabs and does other small changes to reduce the vertical space taken up by the control panel.
Before ________________________________________________________ After

New control panes

This new orientation looks much better as well as saves vertical space. I just made minor changes in the tests.

Did you not intend to put the plot into a tab too, so it renders when the tab is clicked, instead of the "plot" button?

I ended up changing my mind on that one. @rsignell-usgs was strong opposed.

Thank you!
Also for dfviz :)


Allow react v17 peerDependency

Alternatively could set this to "react": ">=16.13.1"

LGTM. Thanks!


feat: add ability to set nonce and tenant id

This gives the ability to set these values on the stratgey for each auth request individually

Closing in favour of https://github.com/snyk/passport-snyk-oauth2/pull/8


Atom Syntax Selectors Changed in v1.3

Hello, it looks like Atom has removed the shadow DOM in v1.3 and syntax packages have to change their css selectors.
If you'd like I can make a PR with these updates.

@DHedgecock Hi. Thanks for Report this bug. Today i'm busy. please make a PR with these updates. Thanks 👍

@DHedgecock I published new version of Panda on apm.

NICE 👍
By the way this is the best theme I've ever had. Thanks for all the time you've put into it.


Alter queue_url in docs to queue_name

Ran into this small documentation issue. Thanks for this awesome package!


Error when handling list in for loop in debug mode, no error in release apk

Hi. I have a method that takes a List from a provider, uses a for loop that compares it to another list in the same provider, and deletes the strings that exist in both lists from one of them. After this for loop, I see if the list ends up empty, and if so, perform a specific action:
// provider.selection is a group of photos selected in the app ui
// provider.album is a list of photo ids, for photos stored in an album

for (var photo in provider.selection['selected']) {
    provider.album['image_list'].remove(photo);
  }
// IF no images are left, delete photo album
if (provider.album['image_list'].isEmpty) {
await provider.deleteAlbum(albumId);
                          }

The issue is that I get the following error when trying to perform the operation when in debug mode (using Android Studio, both in android emulator and plugged device):
[ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: Concurrent modification during iteration: Instance(length:1) of '_GrowableList'.

However, when I build a release .apk and perform the same action, there is no error and everything works great.
In my head it makes no sense that an exception happens in this case because the List used to feed the for loop is not the list being edited inside the loop. But what happens in this case, in debug mode, is that the action occurs for the first item of the loop, and then the exception is thrown. In the release .apk, the loop performs as expected and the action occurs for all items in the list.
Full error:
E/flutter (20488): [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: Concurrent modification during iteration: Instance(length:1) of '_GrowableList'.
E/flutter (20488): #0      ListIterator.moveNext (dart:_internal/iterable.dart:340:7)
E/flutter (20488): #1      groupIntoAlbum (package:my_surgery_local/screens/waiting_room_single.dart:385:46)
E/flutter (20488): #2      _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:73:64)
E/flutter (20488): #3      _rootRunUnary (dart:async/zone.dart:1134:38)
E/flutter (20488): #4      _CustomZone.runUnary (dart:async/zone.dart:1031:19)
E/flutter (20488): #5      _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
E/flutter (20488): #6      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
E/flutter (20488): #7      Future._propagateToListeners (dart:async/future_impl.dart:709:32)
E/flutter (20488): #8      Future._completeWithValue (dart:async/future_impl.dart:524:5)
E/flutter (20488): #9      _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:32:15)
E/flutter (20488): #10     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:290:13)
E/flutter (20488): #11     MainProvider.createAlbum (package:my_surgery_local/providers/storage_provider.dart)
E/flutter (20488): #12     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:73:64)
E/flutter (20488): #13     _rootRunUnary (dart:async/zone.dart:1134:38)
E/flutter (20488): #14     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
E/flutter (20488): #15     _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
E/flutter (20488): #16     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
E/flutter (20488): #17     Future._propagateToListeners (dart:async/future_impl.dart:709:32)
E/flutter (20488): #18     Future._completeWithValue (dart:async/future_impl.dart:524:5)
E/flutter (20488): #19     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:32:15)
E/flutter (20488): #20     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:290:13)
E/flutter (20488): #21     BoxImpl._writeFrames (package:hive/src/box/box_impl.dart)
E/flutter (20488): #22     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:73:64)
E/flutter (20488): #23     _rootRunUnary (dart:async/zone.dart:1134:38)
E/flutter (20488): #24     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
E/flutter (20488): #25     _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
E/flutter (20488): #26     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
E/flutter (20488): #27     Future._propagateToListeners (dart:async/future_impl.dart:709:32)
E/flutter (20488): #28     Future._completeWithValue (dart:async/future_impl.dart:524:5)
E/flutter (20488): #29     Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:554:7)
E/flutter (20488): #30     _rootRun (dart:async/zone.dart:1126:13)
E/flutter (20488): #31     _CustomZone.run (dart:async/zone.dart:1023:19)
E/flutter (20488): #32     _CustomZone.runGuarded (dart:async/zone.dart:925:7)
E/flutter (20488): #33     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:965:23)
E/flutter (20488): #34     _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
E/flutter (20488): #35     _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)

Tested this issue in a different setting, without using the provider package, the same thing happens:
List album = ['aaa', 'bbb', 'ccc', 'ddd', 'eee'];
  List selected = ['ccc', 'eee'];
  testListIssue() async {
    for (var photo in selected) {
      selected.remove(photo);
      print('REMOVED $photo');
    }
  }
//
[ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: Concurrent modification during iteration: Instance(length:1) of '_GrowableList'.

Also tested in a different setting, without the provider package, same error happens:
List album = ['aaa', 'bbb', 'ccc', 'ddd', 'eee'];
List selected = ['ccc', 'eee'];
  testListIssue() async {
    for (var photo in selected) {
      selected.remove(photo);
      print('REMOVED $photo');
    }
  }

// ERROR:
I/flutter ( 8697): REMOVED ccc
E/flutter ( 8697): [ERROR:flutter/shell/common/shell.cc(199)] Dart Error: Unhandled exception:
E/flutter ( 8697): Concurrent modification during iteration: Instance(length:1) of '_GrowableList'.


fix: FuseSortFunctionMatch typescript types

Relates to #613
Fixes typing on FuseSortFunctionMatch, mainly on key & indices attribute.
I included typings test which triggers searchFn with and without includeMatches option, which when undefined, the indices attribute type is also undefined.
I am not sure if this change should bring any side effects, or if the typing on FuseSortFunctionMatch should be different when Fuse constructor is given different option combinations from the ones in the added tests.

Thanks for the PR! Let me dig into this a bit more. Want to ensure there is no side effect.


UI action presentation

When presenting a command action:

we can leave the Endpoint, Variable and Expect objects
we should remove the API URI string
the command payload and response should be multiline text sections
the response will consolidate STD out and error
we should still render the response code returned from by the backend
the action should be validated follow the expect condition (supporting multiple body contain/not contain statements in AND)

need to improve color scheme text from linux terminal


[Suggestion] Seperate crafting grid

It would be good to add a 3x3 crafting grid instead of 6x6

I think you can make an 3x3 recipe inside 6x6.


Fix to maximize via array in power strapping overlap

Potential fix for issue #805
After applied the fix, the via comes out as expected (and DRC free)
$ grep 'VIARULE viagen32'  pdn.def
- via2_2200x900 + VIARULE viagen32 + CUTSIZE 500 500  + LAYERS metal2 via2 metal3  + CUTSPACING 800 800  + ENCLOSURE 200 200 200 200  + ROWCOL 1 2  ;

@Colin-Holehouse would you please take a look at this?  The change is pretty trivial.

@mooredan can you take a look at the CI failure. You can click details to see what is happening. Also it seems there is a source code conflict.

Well it was merge-able when I submitted the pull request.  The diff is a two character change on a single line.  Another commit in the same area must got in ahead of this one.  I'll look into how to resolve the conflict.....

Merge-able again -- will look into why the CI has issues -- suspect that the results for the test cases may be slightly different now due to the number of vias that were added on power strapping

regression passes on my end, but CI is having issues with the rcx area. I've updated the appropriate "ok" files, and pushed them and even forced CI to run again, but no joy.   Could CI be used a different set of ok files for the rcx test? ...or could this be a "precision" differences between different OS's/compilers?
Here's the diff for one of the rcx/ ok files:
diff --git a/src/rcx/test/ext_pattern.ok b/src/rcx/test/ext_pattern.ok
index 6cf4fd33d..7695970fb 100644
--- a/src/rcx/test/ext_pattern.ok
+++ b/src/rcx/test/ext_pattern.ok
@@ -11,7 +11,7 @@
 [INFO RCX-0029] Defined extraction corner X
 [INFO RCX-0008] extracting parasitics of blk ...
 [INFO RCX-0435] Reading extraction model file ext_pattern.rules ...
-[INFO RCX-0436] RC segment generation blk (max_merge_res 0.0) ...
+[INFO RCX-0436] RC segment generation blk (max_merge_res 0) ...
 [INFO RCX-0040] Final 70552 rc segments
 [INFO RCX-0439] Coupling Cap extraction blk ...
 [INFO RCX-0440] Coupling threshhold is 0.1000 fF, coupling capacitance less than 0.1000 fF will be grounded.

Note: 0.0 vs 0    Could this be the notorious compare to zero problem that's common in programming?
$ test/regression rcx
------------------------------------------------------
rcx
generate_pattern pass
generate_rules pass
ext_pattern pass
gcd pass
45_gcd pass
------------------------------------------------------
Passed 5
See /home/mooredan/src/OpenROAD/src/rcx/test/results for log files
------------------------------------------------------
Passed all tool regressions.

From Jenkins, I pulled out the rcx results.tgz from artifacts and found this in the diff file:
mooredan@lemur:~/Downloads/src/rcx/test/results$ more diffs
***<EMAIL_ADDRESS> 2021-07-23 11:50:46.220351622 +0000
---<EMAIL_ADDRESS>2021-07-23 12:10:29.598977103 +0000
***************
*** 11,17 ****
  [INFO RCX-0029] Defined extraction corner X
  [INFO RCX-0008] extracting parasitics of blk ...
  [INFO RCX-0435] Reading extraction model file ext_pattern.rules ...
! [INFO RCX-0436] RC segment generation blk (max_merge_res 0) ...
  [INFO RCX-0040] Final 70552 rc segments
  [INFO RCX-0439] Coupling Cap extraction blk ...
  [INFO RCX-0440] Coupling threshhold is 0.1000 fF, coupling capacitance less than 0.1000 fF will be grounded.
--- 11,17 ----
  [INFO RCX-0029] Defined extraction corner X
  [INFO RCX-0008] extracting parasitics of blk ...
  [INFO RCX-0435] Reading extraction model file ext_pattern.rules ...
! [INFO RCX-0436] RC segment generation blk (max_merge_res 0.0) ...
  [INFO RCX-0040] Final 70552 rc segments
  [INFO RCX-0439] Coupling Cap extraction blk ...
  [INFO RCX-0440] Coupling threshhold is 0.1000 fF, coupling capacitance less than 0.1000 fF will be grounded.

It's not clear as to how to resolve this.

It could be OS, compiler, or spdlog version.  I think the simplest thing is to just revert your rcx changes and those tests should pass on Jenkins even if they don't for you locally.

I may be onto a solution for the integration....hang on...

@Colin-Holehouse please review as CI is now passing


how to annotate the image ?

Oh ,I have a question here.I m wondering how you annotate the image?circle the edge?

manually using some tool for such purpose. I cannot direct you to such tools, probably some radiologist that has done this before is more appropriate. Commonly, detailed annotations are needed. Not just circles.
Excuse the brevity, but this is not a deepmedic-related issue. You should rather look this information to related literature and sources.
Kind regards,
Kostas


feat(viz): Provide an empty state for the canvas

Context
Currently, whenever there's no visible route in the canvas, a full gray area is shown.

Changes
The goal for this commit is to provide more information to the user about the status of the routes in the canvas itself. When there are no visible routes, an according message will be displayed.
No visible routes

No routes

looks good!


Build(deps-dev): Bump @types/styled-components from 5.1.18 to 5.1.19 (#2268)

Pull request opened by the merge tool on behalf of #2268

Merged in 9b003229c56f141e93e09bcc391e5e916daf580e


⚠ Panoramas has degraded performance

In d8d916b, Panoramas ($SITE_URL_PANORAMAS) experienced degraded performance:

HTTP code: 200
Response time: 7148 ms

Resolved: Panoramas performance has improved in 7789a4d after 10 minutes.


Fix the theme usage in sphinx

Seems like we did end up including the changes found in https://github.com/google/skywater-pdk/pull/341 in the documentation update.

Fixed


Add IsTreeViewActive

Need a way to set the active class on both the treeview and its child element.

Maybe use something like this?
        /// <summary>
        /// Gets the CSS class to use for the treeview state. If the current request route matches
        /// any of the <paramref name="actions"/>, the "active" class is returned.
        /// </summary>
        /// <param name="url">The <see cref="UrlHelper"/>.</param>
        /// <param name="actions">A collection of KeyValuePairs, where the key is the action and the value is the controller.</param>
        /// <returns></returns>
        public static string IsTreeviewActive(this UrlHelper url, Dictionary<string, string> actions)
        {
            var controller = url.RequestContext.RouteData.Values["controller"].ToString();
            var action = url.RequestContext.RouteData.Values["action"].ToString();

            if (actions.Any(a => a.Key == action && a.Value == controller))
            {
                return "active";
            }

            return "";
        }

I have rewrite as
public static string IsTreeviewActive2(this UrlHelper url, List<KeyValuePair<string, string>> actions)
{
var controller = url.RequestContext.RouteData.Values["controller"].ToString();
var action = url.RequestContext.RouteData.Values["action"].ToString();
        if (actions.Any(a => a.Key == action && a.Value == controller))
        {
            return "active";
        }

        return "";
    }

var nodes2 = new List<KeyValuePair<string, string>>();
nodes2.Add(new KeyValuePair<string, string>("Index", "HRApplier"));
nodes2.Add(new KeyValuePair<string, string>("Index", "Applier"));
cause Dictionary<string, string> not work for many controller with the same action name.


Create BitCanna

Validator profile picture needs to be added. thx

hi @TillDawn262529
Thanks for pr but please using correct folder and png files

Hi there, sorry mate, it's my first time trying to figure this out and
learn. Is there a guide on how to do this correctly? Thanks
On Mon, 1 Jul 2024 at 08:55, yongjoo @.***> wrote:

hi @TillDawn262529 https://github.com/TillDawn262529
Thanks for pr but please using correct folder and png files
—
Reply to this email directly, view it on GitHub
https://github.com/cosmostation/chainlist/pull/1748#issuecomment-2198801441,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJBP5EYRDXKNGYZDMGUVF63ZKCHU7AVCNFSM6AAAAABKD66KRKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJYHAYDCNBUGE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Make RST be less picky about .LIS vs .LIL suffixes

This is one for the upstream component, but you probably know how to pass it on.
My FORTH for Agon uses RST.LIL in 16-bit mode (ADL=0) and this is rejected by the eZ80 interpreter.  The hardware just works. There may be other programs that refuse to run just because the wrong instruction suffix (in fact a prefix byte in the binary machine code) is used on RST.

I think this is already fixed upstream: https://github.com/tomm/ez80/commit/148aae0ee1705231b06a6ae41dff4d97149ce75b

Need to update your Cargo.toml file for the correct git rev of iz80.
But FORTH runs. And I can load up tetris,4th, which is already somewhat playable.

Oh sorry, I've misread the parent commit hash as the commit hash in the ez80 commit and thought its already in there. But great to hear that your problem is fixed.

Anyway, FORTH runs now, but the nano editor gets a message for RST.LIS in ADL mode (and it does not execute that call then). In this mode it would need to be .LIL, but the macro defined in mos_api.inc has an RST.LIS instead. Andain this just works on the real hardware.

When I hack this locally, It will get nano to execute, even though it is not really usable at this state, but getting closer.


Propose Logo

Hello @felipealfonsog, i am a graphic designer. I would like to know if you would be interested in me making a logo for this project. If you would have me, i would make a logo for you, and it's free.

Yeah let's do it :) go ahead :)

Hello @felipealfonsog, I have designed a logo for your project. I wasn't sure what colors you would like so I made two color variations. If you need any changes please let me know. The other screens above the main video icon represents the flip, and the whole logo looks like a "T" for Theatreflix. The text has a land representation under it signifying an offline since the videos can also be accessed offline.

Hello @felipealfonsog, I have designed a logo for your project. I wasn't sure what colors you would like so I made two color variations. If you need any changes please let me know. The other screens above the main video icon represents the flip, and the whole logo looks like a "T" for Theatreflix. The text has a land representation under it signifying an offline since the videos can also be accessed offline.

wow you made an amazing job! .... incredible! I'm speechless ....

hello @felipealfonsog, am glad you liked it. I discovered a typo error in the first logo and made corrections. Here is a transparent version without background

this one has a white background like before

i have prepare the logos in various sizes which you can download from here
https://drive.google.com/drive/folders/1tpBDcZUGMXUecfRKAGZYlS_h4WdI9qBy?usp=sharing

awesome job! Thanks a lot!

Is it possible if you could type Theatherflix ? :)

lolz omg i didn't notice that also....making the changes now

Hi @felipealfonsog i have made the changes below...
transparent background

white background

i have re-uploaded the new changes to this google drive link with the various sizes
https://drive.google.com/drive/folders/1mbt-5nsAxkY3OnnnO6rrsQhJVKqBHFRk?usp=sharing
Best regards,
Dee-y.

Awesome Thanks a lot! send me your email... for any contact... please. :+1:  :100:

Here's my email<EMAIL_ADDRESS>awesome contributing to your project :)

Thanks a lot !


schema and username prefix

Description
Feature Definition
Is your feature request related to a problem? Please describe.
to help with refresh process, it is often a good idea not to prefix or suffix schema and user with environment notion.
for example,
staging_pgbench is always complicated for common people to deal with the one in production prod_pgbench
if you really want to distinghish prod/preprod/stg, then you should do it at database or cluster level
Describe the solution you'd like
raise a warning if schema starts or ends with some envt notion

Done https://github.com/Decathlon/dblinter/pull/16


Fix compile failure

Without this, recent cabal version will fail to build this due to
cabal: The program 'c2hs' version >=0.15 is required but it could not be found.

btw, is there a reason why pkcs11-tests is an executable rather than a test-suite?

Thanks!
Currently pkcs11-tests is more like an integration test than a unit test because it requires a physical SafeNet token, because of that it cannot be run by TravisCI.  I have plans to implement unit test using some kind of software PKCS11 library, something like SoftHSM.


Email Handling

Placeholder issue for a discussion that started at Ghosts. Steve was looking to ensure all email from Symposion is recorded. I've started discussion on that thread, with the obvious solution of simply adding a BCC: to all emails so they are sent to an archive inbox of our choosing.

Good work. We'll have such an address before too long
On 13 Jun 2016 7:03 PM, "Scott Bragg"<EMAIL_ADDRESS>wrote:

Placeholder issue for a discussion that started at Ghosts. Steve was
looking to ensure all email from Symposion is recorded. I've started
discussion on that thread, with the obvious solution of simply adding a
BCC: to all emails so they are sent to an archive inbox of our choosing.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lca2017/symposion/issues/10, or mute the thread
https://github.com/notifications/unsubscribe/AAl-3OB684ZDJnj9F6XS0gO2rrFZ-Kwmks5qLgvYgaJpZM4I05U-
.

Fixed.


Legs of the robot in the ground when using mc_rtc_ticker

When using the robot model in RVIZ with mc_rtc_ticker, the legs of the NAO robot are in the ground (See following picture).

This seems to be caused by an inconsistency between the variables "_default_attitude" and "halfSitting" in nao.cpp.
Using the currently assigned value for "_default_attitude" and setting the the initial robot position as "halfSitting" (all legs joints to 0) makes the robot_map frame used in RVIZ too low compared to the legs length. This causes the robot to be inside the ground instead of on top of it.
If the initial if the initial position of the robot is the one specified with "halfSitting", "_default_attitude" value should be set to "{{1., 0., 0., 0., 0., 0., 0.333}}", which allows the CoM to be higher and removes the problem.
Another solution would be to change the values of "halfSitting" to values set in previous commits.
Thank you.

Thanks for the report, fixed by the above commit.


Add Android

Really interested to see how you are going to solve the ios and android extension problem :)

Can you do that please?


Different icons for different features in geoJSON with OL3

I have this OpenLayers 2 code that I want to convert to OpenLayers 3:
var pois = new ol.Layer.Text(
    "My POIs",
    { location:"/media/pois.txt",
      projection: ...
    }
);
map.addLayers([pois]);

The .txt file is a tab-separated file with the following columns:
lat lon title description icon iconSize iconOffset
I've learned I need to use ol.source.Vector instead of the Text layer, but I can't get the icons to show up. I've tried using features.properties.graphic, .icon and iconUrl. Here's the full geojson I use:
{
  "type": "FeatureCollection",
  "features": [
    {
      "type": "Feature",
      "properties": {
        "title": "Blueseed ship",
        "description": "High-tech startup community on a 200-meter (600-ft) cruise ship",
        "graphic": "/media/cruise-ship-drawing.png",
        "iconSize": "50,33",
        "iconOffset": "-10,16"
      },
      "geometry": {
        "type": "Point",
        "coordinates": [
          -122.82,
          37.5
        ]
      }
    },
    {
      "type": "Feature",
      "properties": {
        "title": "CIDP Buoy 047",
        "description": "Coastal Data Information Program - Buoy 047, Farralon Islands, CA
Historical wave height data.",
        "iconUrl": "/media/buoy-icon.png",
        "iconSize": "24,24",
        "iconOffset": "-24,0"
      },
      "geometry": {
        "type": "Point",
        "coordinates": [
          -122.8733,
          37.505
        ]
      }
    },
    {
      "type": "Feature",
      "properties": {
        "title": "NOAA Buoy",
        "description": "NOAA Station 46012
Historical data",
        "externalGraphic": "/media/buoy-icon.png",
        "iconSize": "24,24",
        "iconOffset": "-16,16"
      },
      "geometry": {
        "type": "Point",
        "coordinates": [
          -122.881,
          37.363
        ]
      }
    }
  ]
}

Adding a style to the Vector layer doesn't scale because I need a separate geojson per feature.

You can extract whatever information you need from your features, and use that in your style function. Basically, you'll want to configure your layer with a styleFunction like this (assuming your features have an icon property and an iconOffset defined as array instead of a string):
var styleCache = {};
function style(feature, resolution) {
  var icon = feature.get('icon');
  var style = styleCache[graphic];
  if (!style) {
    style = styleCache[graphic] = new ol.style.Style({
      image: new ol.style.Icon({
        src: icon,
        anchorXUnits: 'pixels',
        anchorYUnits: 'pixels',
        anchor: feature.get('iconOffset')
      })
    });
  }
  return style;
}
layer.setStyle(style);


baseChannelGroup.read_trace says chan_num when it means dataset_num

Original report by Galen O'Neil (Bitbucket: oneilg, GitHub: oneilg).

EXISTING:
def read_trace(self, record_num, chan_num=0):
"""Read (from cache or disk) and return the pulse numbered <record_num> for channel
number <chan_num>.  If this is a CDMGroup, then the pulse is the demodulated
channel by that number."""
seg_num = record_num / self.pulses_per_seg
self.read_segment(seg_num)
return self.datasets[chan_num].data[record_num % self.pulses_per_seg]
PROPOSED:
def read_trace(self, record_num, dataset_num=0):
"""Read (from cache or disk) and return the pulse numbered <record_num> for channel
number <dataset_num>.  If this is a CDMGroup, then the pulse is the demodulated
channel by that number."""
seg_num = record_num / self.pulses_per_seg
self.read_segment(seg_num)
return self.datasets[dataset_num].data[record_num % self.pulses_per_seg]

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

I changed arguments to (and semantics) of TESGroup.read_trace() and
.plot_traces().  Now they accept both a dataset_num (0…n-1) and a
chan_num(1,3,5,…2n-1).  If both are provided, the chan_num governs
if valid.  This is not as simple as Galen's suggestion, but it covers more bases.
See git commit 4387c0223f4227edf8366d249593b29b1c2f2352 and later.


Add option not to print the intro text

doctest currently has an option not to print the version, so that printVersion() starts with:
         void printVersion() {
             if(opt.no_version == false)

I suggest that the same option be added for the intro text, i.e.
        void printIntro() {
+            if(opt.no_intro == false) {
             printVersion();
             // etc. etc.
+            }

and a corresponding command-line option be added. Sometimes you just want your binary to be more quiet...

The no-version option is probably useful only for me - so that I don't have to update a bunch of reference test output each time I release a new version.
I guess there should be a concept of verbosity - perhaps one would want to also not see the stats in the end if everything has passed successfully?

@onqtam : I would be ok with either a "no intro" option or a single "no intro and no version" option. An overall "quiet" option is useful, but some people (like me) do want to see the summaries on success, without the intro.

One possible thing to do without editing the doctest header would be to inherit from the ConsoleReporter class in the .cpp wherever the test runner is being implemented, override the test_run_start() method without calling the intro printing and then register your new reporter & use that.
I'll release 2.3.7 without this, but I'll keep the issue open for perhaps the next version because I want to think it through.

Are you discussing what you could do, what test authors could do, or what test runners could do? It seems like it's the middle one.
Also - no rush of course. It's not like something is broken.

I meant that wherever the test runner is implemented (DOCTEST_CONFIG_IMPLEMENT) you could inherit in that same translation unit from the console reporter and make a new one with this tiny change, but then you'd have to also always use my_console instead of console as the reporter - perhaps with a default main() function.

Well, you could offer the same thing for all sorts of options... I'd hope test authors don't have to know about "console reporters" and such.

Just rolled it in its simplest form as you suggested - seems obvious in retrospect - not sure why I didn't make it the simplest possible way and be done with it...


Missing users in IdentityInfo table compare to AzureAD

Describe the bug
According to the documentation and IdentityInfo release blog post
Identityinfo table is populated by Azure Sentinel UEBA with all the users identities information from the AzureAD
That's not what we observe in practice.
We had a user who launched a powershell script on a computer with his own account from its own user directory, but we didn't had any entry in the IdentityInfo table for this user during the past week. When we check in the AzureAD, at least this account has some recent SignIn, and exist since a year.
This is even general, as we know that to get info on a user we need to get data from IdentityInfo at least on  the past 3 days to be sure that this user will be in the query results targerting IdentityInfo Table.
To Reproduce
No steps identified, is general to Sentinel
Expected behavior
IdentityInfo should have at least an entry for every user in the UserAD, as this in the ground truth for users

Hey @JiTmun, did you see the user sign-in in any of the Azure Active Directory Data Types? (SignInLogs, AuditLogs, NonInteractiveSignInLogs etc.)


YouTube fails to create oembed request

Linked PR  #8
In my case pyoembed fails with YouTube, because YT does not allow HTTP-Requests, but all application/json+oembed links are prefixed by http://, even in HTTPS-Sites
An error was ocurred: Failed to do oEmbed request: https://www.youtube.com/watch?v=M3r2XDceM6A

See also: https://github.com/leedo/noembed/issues/115


Add contributing guide

Create CONTRIBUTING.md that covers:

You should put yourself as the assignee of an issue if you'd like to work on it
If you assigned yourself to an issue and didnt do the issue, don't forget to remove yourself from the assignee
If you have questions or wants quicker review, ask in the Telegram group
Re-request review should be done after you've submit the changes that are needed
1 PR MUST ONLY COVER 1 ISSUE.  They should communicate and sort things out themself (if that's between them)

cc @elianiva (please help)

Oh one more. Should be in english AND indonesian.


Remove localhost from link

Changed this to HTTP_HOST.  When running this on a remote machine this link won't work.

Got your point, but the stack is focusing on local development, therefore I'm going to close this. But anyway thank you for your contribution :).


Skin issue

A skin was uploaded to the site, but it won't show up during the replay

artic, you have to do the following for making the skin work:

Log-in to the website
Click on "Profile"
When on profile, click "Edit Profile"
Select your custom skin

My skin does not work but I have uploaded it where it corresponds and I have pressed the button that is just above and then I raised my replay, when I finished I was very excited because I always recorded my replays at 25-30 fps and now they were at 60 fps yay! but my skin did not appear, I was also impressed by the good website that I created Ekode but still I can not find a way to make my skin work

After some research the issue seem to be linked to the web server. Because the website is hosted on a limited access host. I cannot make the change my self. I've opened a ticket on my web host panel...

After some research the issue seem to be linked to the web server. Because the website is hosted on a limited access host. I cannot make the change my self. I've opened a ticket on my web host panel...

what does that mean?

what does that mean?

It mean that I cannot access the file php.ini with my current plan on my webhost.
So only my webhost can do the change (If they want to do it)


Flutter UI

I want to know is there a simple and easy way of creating UI in flutter app like drag and drop feature ?

Please reopen if your question hasn't been address.d

I am not talking about drag and drop fetaure in the app. I want to create UI in flutter using drag and drop from palette like in android studio for xml layouts.

Oh, sorry I misunderstood. I came across this a while back, but I haven't found anything else. And I also can't get this thing to work. Maybe someday we'll see a "Flutter Studio" or something. https://medium.com/@pmutisya/create-flutter-studio-using-dart-501f6577c1ca

Thanks for the reply.

Thanks for the link to the flutter IDE AppBuilder. Is this open ?
I want to extend it further.

I don't believe it is. I've looked everywhere and can't seem to find it.

@gedw99 maybe you can try and contact the developer from this https://medium.com/@pmutisya/create-flutter-studio-using-dart-501f6577c1ca


Tileset traversal with clipping planes

Added clipping planes tileset traversal optimization in visibility and contentVisibility
Added show debug boundingVolumes options to sandcastle example
Moved some duplicate logic to the Plane class
Updated shader to reflect math in Plane

Part of #5765
@lilleyse

@ggetz, thanks for the pull request! Maintainers, we have a signed CLA from @ggetz, so you can review this at any time.
:warning: I noticed that CHANGES.md has not been updated. If this change updates the public API in any way, fixes a bug, or makes any non-trivial update, please add a bullet point to CHANGES.md and comment on this pull request so we know it was updated. For more info, see the Pull Request Guidelines.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.
:earth_africa: :earth_americas: :earth_asia:

@lilleyse Thanks for taking a look! Updated.

Thanks @pjcozzi, @lilleyse
Updated to include the additional optimization.
Also included fixes from discussion in my previous PR: https://github.com/AnalyticalGraphicsInc/cesium/pull/5913#discussion_r149223864

Thanks @ggetz!
@lilleyse please merge when ready.

@lilleyse Can you take a look at this? I have a successive branch with all the additional changes.

Thanks @lilleyse, updated!


Non-Matching Checksum Error with crd3 dataset

Describe the bug
When I try loading the crd3 dataset (https://huggingface.co/datasets/crd3), an error is thrown.
Steps to reproduce the bug
dataset = load_dataset('crd3', split='train')

Expected results
I expect no error to be thrown.
Actual results
A non-matching checksum error is thrown.
datasets.utils.info_utils.NonMatchingChecksumError: Checksums didn't match for dataset source files:
['https://github.com/RevanthRameshkumar/CRD3/archive/master.zip']

Environment info

datasets version: 1.12.1
Platform: Linux-4.4.0-173-generic-x86_64-with-Ubuntu-16.04-xenial
Python version: 3.6.10
PyArrow version: 5.0.0

I got the same error for another dataset (multi_woz_v22):
datasets.utils.info_utils.NonMatchingChecksumError: Checksums didn't match for dataset source files:
['https://github.com/budzianowski/multiwoz/raw/master/data/MultiWOZ_2.2/dialog_acts.json', 'https://github.com/budzianowski/multiwoz/raw/master/data/MultiWOZ_2.2/test/dialogues_001.json']

I'm seeing the same issue as @RylanSchaeffer:
Python 3.7.11, macOs 11.4
datasets==1.14.0
fails on:
dataset = datasets.load_dataset("multi_woz_v22")


feat(aws-cloudwatch): add support for non-widget fields in dashboard body

Adding start, end, and periodOverride from https://docs.aws.amazon.com/AmazonCloudWatch/latest/APIReference/CloudWatch-Dashboard-Body-Structure.html

Pull Request Checklist

[x] Testing

Unit test added (prefer not to modify an existing test, otherwise, it's probably a breaking change)
CLI change?: coordinate update of integration tests with team
cdk-init template change?: coordinated update of integration tests with team

[x] Docs

jsdocs: All public APIs documented
README: README and/or documentation topic updated

[x] Title and Description

Change type: title prefixed with fix, feat will appear in changelog
Title: use lower-case and doesn't end with a period
Breaking?: last paragraph: "BREAKING CHANGE: <describe what changed + link for details>"
Issues: Indicate issues fixed via: "Fixes #xxx" or "Closes #xxx"

[x] Sensitive Modules (requires 2 PR approvers)

IAM Policy Document (in @aws-cdk/aws-iam)
EC2 Security Groups and ACLs (in @aws-cdk/aws-ec2)
Grant APIs (only if not based on official documentation with a reference)

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

Unit test?

I don't understand the link between PR content and title. Title says something about widgets, content seems to be about adding a period to the dashboard.

Title refers to adding all remaining fields in dashboard body format other than widget, which is already supported

Ohhh


Cannot build TypeScript OOTB Echo BOT

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Versions
What package version of the SDK are you using. - 4.15.0
What nodejs version are you using - 14.15.4
What browser version are you using - none
What os are you using - Windows Server 2019 Datacenter
Describe the bug
When using Yeoman to create a TypeScript Echo BOT the following errors are displayed:
81 export declare type Resolved<T> = T extends {
                       ~~~~~~~~

node_modules/@azure/core-rest-pipeline/node_modules/@azure/core-tracing/types/core-tracing.d.ts:83:41 - error TS2315: Type 'Resolved' is not generic.

83 } ? F extends (value: infer V) => any ? Resolved<V> : never : T;
                                           ~~~~~~~~~~~

node_modules/@azure/core-rest-pipeline/node_modules/@azure/core-tracing/types/core-tracing.d.ts:131:212 - error TS2315: Type 'Resolved' is not generic.

131     }, Callback extends (updatedOptions: Options, span: Omit<TracingSpan, "end">) => ReturnType<Callback>>(name: string, operationOptions: Options, callback: Callback, spanOptions?: TracingSpanOptions): Promise<Resolved<ReturnType<Callback>>>;

To Reproduce
Steps to reproduce the behavior:

yo botbuilder
What's the name of your bot? (my-chat-bot)
What will your bot do? (Demonstrate the core capabilities of the Microsoft Bot Framework)
What programming language do you want to use? TypeScript
Which template would you like to start with? (Use arrow keys) Echo Bot - https://aka.ms/bot-template-echo
Looking good.  Shall I go ahead and create your new bot? (Y/n)

Expected behavior
Completes without errors.
Screenshots
Additional context

Its the version of typescript in use.  Changing it to 4.3.2 resolves the issue


Cannot add data in existing collection

I tried to add data in the existing collection but there seems to be no ref/resource to add data in an existing collection. Am I missing something?

Do you mean add a new row into the DB? Can you use Collection insert?
https://rawgit.com/techfort/LokiJS/master/jsdoc/Collection.html


Integrate  function hide head and body parts from Dismemberment?

It possible add some features from https://github.com/CamxxCore/DismembermentASI/tree/master/DismembermentASI ??
also Camera HardAttachTo and  HardAttachToBone...

I know what you mean in dismemberment, but there's issues for scripts not for SHVDN, since the said dismemberment script basically "dismember" parts by hooking a bone drawing functions and drawing some bones with zero scales. What about access to dismemberment functions from other scripts and potential conflicts between some other scripts that try to hook a bone drawing function?
To be honest, I don't like how RAGE Plugin Hook exposes its custom siren features only to plugins for it, which is done by hooking some siren functions in the game I believe. I believe a runtime for singleplayer modding shouldn't casually hook game functions, because that can cause massive disaster more easily than when one script hooks some game functions (just potential local conflicts in that case).
I know FiveM hooks a lot of the game code for such as game crash patch or additional features, but that basically doesn't cause conflicts to script resources since it’s scripts can't do arbitrary unsafe memory operations iirc, unless some ASI mods try to hook some of them (although ASI mods aren't really supported in FiveM).

Hello. As a student striving for academic excellence, I found the review about the best paper writing service to be incredibly informative. It provided a detailed analysis of various services, highlighting their strengths in terms of quality, reliability, and affordability. Knowing which services are reputable and deliver exceptional results is invaluable when seeking assistance with challenging assignments. This review has certainly guided my decision-making process, ensuring I choose services that uphold high standards and help me achieve my academic goals effectively.


Generate python for request-response behavior

As a developer I want to auto-generate the boilerplate code for a RESTful service from the model so that I can spend my time implementing the business logic.
AC:

Must demonstrate generation of code for the EchoService model
Must use a REST API with a POST endpoint for the echo behavior
Must provide the developer an auto-generated stub to insert the business logic
Must not destroy existing business logic when rerun.
Must generate python code (we'll do other languages later...I promise)
Must be have an automated acceptance test implemented

Number 6, "Must be have an automated acceptance test implemented" is a little unclear.
Is this generating an automated test from the behavior acceptance section, like a cucumber test? If we converted the acceptance definition to cucumber tests then we'd need to discuss how to generate the step definition implementation and whether or not we'd supply a standard set of step definitions. I think this story might need to be blocked by another story that focuses on defining and implementing task 6.


Fix: Timeseries aligned center in homepage for mobile devices

Description of PR
Timeseries is aligned center in homepage for mobile devices by removing extra div in home.js. Before it was towards left side.
Relevant Issues
Fixes #1807
Checklist

[x] Compiles and passes lint tests
[x] Properly formatted
[x] Tested on desktop
[x] Tested on phone

Screenshots
before

after

Looks like this breaks the intersection observer. The charts aren't updating on hovering the states

Looks like this breaks the intersection observer. The charts aren't updating on hovering the states

@shuklaayush I solved the issue by following another approach, made css changes. It is not breaking the observer and also fixed the alignment issue.

@shuklaayush @jeremyphilemon please review this. I have not made any changes in Insersection Observer (so no interference there), just set margins to make it center aligned.

Hey @nayakrahul, sorry about the delay. This was because I put the intersection observer at the wrong place. Should be fixed with 7740f5c8f764b7a521fdfae695b622f160d2e734. Thanks for the PR anyway


Register your spark package

You have a dependency on Apache Spark. You can register your package at
http://spark-packages.org/register.
It's "[a] community index of packages for Apache Spark."

Thanks for the suggestion, but it's already registered: http://spark-packages.org/package/Stratio/spark-mongodb


Remove Google Analytics?

What docs page needs to be fixed?
All of them.
What is the problem?
Remove Google Analytics
If you must have an analytics system, might I suggest either:

Plausible: https://plausible.io (offers a 50% discount for FOSS projects)
Goatcounter: https://goatcounter.com
Matomo: https://matomo.org
Fathom: https://usefathom.org

These solutions are all well-tested and open-source. This, of course, means they
can be independently audited for privacy and security. Instead of using a proprietary
nonfree system such as Google Analytics, perhaps we could work together to replace
it with something slightly more ethical?
Furthermore, the majority of privacy features in evergreen browsers block Google
Analytics from being run. I can not say the same for solutions such as Plausible,
which suggests they are trusted for their privacy by independent persons.
If you need any more information, please contact me!
What should be changed to fix the problem?
How would you feel about replaing Google Analytics?

I'm not against the idea, but: GA support was already built into Docusaurus, I'm not looking to actually pay for anything here, and I don't want to have to deal with self-hosting anything either.
Tbh the "proprietary nonfree" argument doesn't matter to me at all.  I just want something that gives me reasonable stats on what pages are being hit, 404s, and external referrals, with as little effort on my part to set anything up as possible.
If someone else is willing to do the work to evaluate options, figure out how to set everything up, and I don't have to spend any meaningful time configuring things, and the results are reasonably useful, I'm open to the idea.

Google Analytics is due to retire its existing implementation in a few months.  At this point I'm ready to switch over to something different, especially since I personally don't use 98% of the features in GA.  I mostly care about:

Overall hits
Hits per page
External referrers and sources

The biggest issue is that the Redux docs do get considerable traffic.  redux.js.org had around 750K hits in the last 30 days (which includes Christmas), and 850K in November.  And... wow. React-Redux had 400K hits in November, and the RTK docs had over 1M (!!!).
Looking at Plausible's pricing, that would be at least $70/mo, and I don't want to pay that.
So, I'd really like something that is:

Trivial to set up, configure, and integrate into Docusaurus
Provides me at least that info
Looks decent
Free

I realize that's a somewhat self-entitled attitude, but hey, I'm not getting paid for this OSS gig either :)
I know that some tools like Plausible are capable of being self-hosted, and I googled up a couple examples of people running Plausible on something like Fly.io with some additional work.
So:

Does anyone have suggestions on what tools I should consider?
Would anyone be interested in helping set up a new analytics tool for us to use in the docs?

FWIW, I'd also question if you need analytics at all - over time I removed analytics from every OSS docs site I worked on because they weren't providing value outside of 'number goes up' (which I could easily see from the dependency manager's site)
The only site I kept them on was the typescript site, and that was only useful during the site rewrite and the handbook 1 -> 2 transition, but I couldn't successfully argue that we should turn it off after the site/content settled and I'm willing to bet they've not been looked at in a year or so
So, I'd ask yourself what you are tracking for and if it could be done a better way, or skipped entirely before also thinking about switching providers.

@orta : I found the stats extremely valuable while rewriting the tutorials to see what topics I ought to hit, and given that I still have a mile-long list of docs pages I want to redo, I think this info would still be useful going forward.

Perhaps something bare-bones like this?

We really don't want to roll our own solution. We already have a lot on our hands.

Finally done by switching to Umami

@markerikson Umami is so sweet!
Also Umami seems to be a good role model for how to combine OSS SaaS with monetization.
Might be we could close it? https://github.com/reduxjs/redux/pull/4465


IE/Windows/11,10,9: Wheel not working

Internet Explorer 11,10,9
Scrolling/Zooming not working:
direction: none
factor: 0

done in e1e25987f3e15c03f73c4aedd587820c278ce826


[dependency] terragrunt plan output Invalid resource error message with mock_outputs set

when i use the "terragrunt run-all plan" on "vpc peering" and "vpc" which i already set up mock_outputs on vpc peering for each vpc-id.
it will comes error InvalidVpcID.NotFound. it should be use fake-stp-vpc-id i prepared.
Error: InvalidVpcID.NotFound: The vpc ID 'fake-stp-vpc-be-id' does not exist
│ 	status code: 400, request id: 6ac09ebc-7293-44c1-8aa7-3eaabacd6809
│
│   with module.vpc-peering-pub2be.data.aws_vpc.acceptor[0],
│   on .terraform/modules/vpc-peering-pub2be/main.tf line 33, in data "aws_vpc" "acceptor":
│   33: data "aws_vpc" "acceptor" {

but I can terragrunt run-all apply on vpc-peering
terragrunt.hcl
locals {
  # Automatically load environment-level variables
  environment_vars = read_terragrunt_config(find_in_parent_folders("env.hcl"))
  project_vars     = read_terragrunt_config(find_in_parent_folders("project.hcl"))
  region_vars      = read_terragrunt_config(find_in_parent_folders("region.hcl"))

  # Extract out common variables for reuse
  env         = local.environment_vars.locals.environment
  aws_project = local.project_vars.locals.aws_project
  aws_region  = local.region_vars.locals.aws_region
}

# Terragrunt will copy the Terraform configurations specified by the source parameter, along with any files in the
# working directory, into a temporary folder, and execute your Terraform commands in that folder.
# git@github.com:terraform-aws-modules/terraform-aws-vpc.git

terraform {
  source = "${get_terragrunt_dir()}/../../../../../../terraform/module//vpc-peering"
}

# Include all settings from the root terragrunt.hcl file
include {
  path = find_in_parent_folders()
}

# Passing outputs for vpc-be modules
dependency "vpc-be" {
  config_path = "../vpc-be"

  # it corresponds to a map that will be injected in place of the actual dependency outputs
  # if the target config hasn’t been applied yet
  mock_outputs = {
    vpc-be-id  = "fake-stp-vpc-be-id"
  }
  # restrict this behavior
  mock_outputs_allowed_terraform_commands = ["validate","plan","init"]
}

# Passing outputs for vpc-public modules
dependency "vpc-pub" {
  config_path = "../vpc-pub"

  # it corresponds to a map that will be injected in place of the actual dependency outputs
  # if the target config hasn’t been applied yet
  mock_outputs = {
    vpc-pub-id  = "fake-stp-vpc-pub-id"
  }
  # restrict this behavior
  mock_outputs_allowed_terraform_commands = ["validate","plan","init"]
}

# Passing outputs for vpc-db modules
dependency "vpc-db" {
  config_path = "../vpc-db"

  # it corresponds to a map that will be injected in place of the actual dependency outputs
  # if the target config hasn’t been applied yet
  mock_outputs = {
    vpc-db-id  = "fake-stp-vpc-db-id"
  }
  # restrict this behavior
  mock_outputs_allowed_terraform_commands = ["validate","plan","init"]
}

# These are the variables we have to pass in to use the module specified in the terragrunt configuration above
inputs = {
  vpc_be_id   = dependency.vpc-be.outputs.vpc-be-id
  vpc_pub_id  = dependency.vpc-pub.outputs.vpc-pub-id
  vpc_db_id   = dependency.vpc-db.outputs.vpc-db-id

  project     = "${local.aws_project}"
  environment = "${local.env}"
  region      = "${local.aws_region}"
}

vpc-peering module
module "vpc-peering-be2db" {
  source  = "cloudposse/vpc-peering/aws"
  version = "0.9.0"

  namespace = "stp"
  stage     = var.environment
  name      = "be2db"

  auto_accept                               = true
  requestor_allow_remote_vpc_dns_resolution = false
  acceptor_allow_remote_vpc_dns_resolution  = true
  requestor_vpc_id                          = var.vpc_be_id
  acceptor_vpc_id                           = var.vpc_db_id
  create_timeout                            = "5m"
  update_timeout                            = "5m"
  delete_timeout                            = "10m"

  tags = {
    Name        = "vpc-peering-be2db"
    Project     = var.project
    Environment = var.environment
    Module      = var.module
  }
}

module "vpc-peering-pub2db" {
  source  = "cloudposse/vpc-peering/aws"
  version = "0.9.0"

  namespace = "stp"
  stage     = var.environment
  name      = "pub2db"

  auto_accept                               = true
  requestor_allow_remote_vpc_dns_resolution = false
  acceptor_allow_remote_vpc_dns_resolution  = true
  requestor_vpc_id                          = var.vpc_pub_id
  acceptor_vpc_id                           = var.vpc_db_id
  create_timeout                            = "5m"
  update_timeout                            = "5m"
  delete_timeout                            = "10m"

  tags = {
    Name        = "vpc-peering-pub2db"
    Project     = var.project
    Environment = var.environment
    Module      = var.module
  }
}

module "vpc-peering-pub2be" {
  source  = "cloudposse/vpc-peering/aws"
  version = "0.9.0"

  namespace = "stp"
  stage     = var.environment
  name      = "pub2be"

  auto_accept                               = true
  requestor_allow_remote_vpc_dns_resolution = true
  acceptor_allow_remote_vpc_dns_resolution  = true
  requestor_vpc_id                          = var.vpc_pub_id
  acceptor_vpc_id                           = var.vpc_be_id
  create_timeout                            = "5m"
  update_timeout                            = "5m"
  delete_timeout                            = "10m"

  tags = {
    Name        = "vpc-peering-pub2be"
    Project     = var.project
    Environment = var.environment
    Module      = var.module
  }
}

I think this may just be a limitation of run-all plan and mock values in Terragrunt. See the warning here.


IMP: Add new percentage-merged column to denoise-paired output

Closes #115. Adds a new percenteage-merged column caclulated as merged / denoised * 100 to the output from denoise_paired. Currently the 3 tests affected by this are failing, and I am unsure why. I changed the test data files accordingly. When I save the metadata output by denoise_paired to disk as a .tsv, then import it into ipython as a dataframe and compare it to the test data file imported as a dataframe, they are equal, but for some reason they are not equal when compared in the actual tests. When I checked the values of the expected and resultant dataframes in the tests, I found they only showed 7 decimals and not the 13 I see when I test them in ipython, but all decimals displayed were identical in spite of the supposed inequality.

Unsurprisingly, these failures seem to be caused by some floating point shenanigans. I will look into it more tomorrow.

The commit https://github.com/qiime2/q2-dada2/pull/120/commits/c6e47efaf431d30110ea5915e1316168a31413b0 arbitrarily rounds the float values to the nearest two decimal places which unsurprisingly fixes the floating point shenanigans, I don't know if it's what we want though.

I think this strategy is fine, but I do wonder if we shouldn't have a nicer visualization for this output at this point. It's a fairly fundamental part of our workflow...

@Oddant1 do you have an example QZV that I can look at for a UX review?

this is a QZV (with a zip extension for GH).
denoising-stats.zip

Hey @nbokulich, take a look at this latest QZV. I think the percentage-of-input-filtered is confusing, it seems like it would be more clear if the value in that column was (input - filtered )/ input. What do you think?

Works for me!

Sounds good --- @Oddant1, I have a handful of changes I am working on here, I will ping you once those are ready.

Okay @nbokulich, :basketball: :+1:
QZV

Those three separate commits have to do with some weirdness in attempting to merge my code with Matt's commit, but the final commit should contain what we want


Fixed  https://github.com/detekt/detekt/issues/7696

Fixed Documentation issue stated on https://github.com/detekt/detekt/issues/7696 with broken link on performance tab and replaced it with blog series as stated on the issue comments

Warnings

:warning:

It looks like you're editing the versioned copy of our website. This affects only users on the 'current' version of detekt, and it's correct only if you intend to fix the documentation for an already released version of detekt. Most of the time you want to update also the docs inside website/docs/ as well, so this change will reflect also for documentation on future versions of detekt.

Messages

:book:

Thank you very much for making our website better :heart:!

  Generated by :no_entry_sign: dangerJS against 12c52d434d79708ea830595ddc858a436515fdc3

thanks for the suggestion  @cortinico , I appreciate it.
I have just committed it

Hello @cortinico  , also found The same broken link with found on SpreadOperator Section on performance page just replaced it with Article Series.


GeoMessages Validator app

We need to develop a new app called GeoMessages Validator that takes a GeoMessages file as input and validates its content against the XSD schema files we're soon adding to this repo.

The name is negotiable. It could be GeoMessages Validator, GeoMessage Validator, or something similar.

Hi Gary
we met at the 2014 Defence and Intelligence Security Technical Exchange Meeting.
Very interesting and useful this tool.
I have not found any documentation on geomessage , only modifier fields reported here https://github.com/Esri/military-features-data/tree/master/data/mil2525c
In addition to sending military tracks we use geomessage for exchange key-values and use them at the application level.

Hi Gianluca. It's nice to hear from you. I am working on documenting the GeoMessage format right now (#22) and should have something published in the next few weeks.


Revert "Decode legacy receipts"

Reverts onflow/flow-evm-gateway#474
This can be reverted after we reindexed.

Summary by CodeRabbit

New Features

Streamlined error handling for receipt data processing, enhancing performance and maintainability.

Bug Fixes

Improved reliability by removing legacy receipt decoding, reducing potential points of failure.

Refactor

Simplified control flow in receipt processing, focusing on current data formats only.

No, after reindexing this code is no longer needed, this was only needed before we reindexed


Cow: Watch watch-wrestling2.com

Cow requests the watch of the watch_keyword watch-wrestling2\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

watch-wrestling2\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


Allow using CSS-like selectors for HTML filtering

UBlock Origin can use CSS selector for HTML filtering.
Example:

google.com##^div.bkWMgd:nth-child(2)
index.hr##^.article-read-more-container:nth-child(3)

is not the same request?
https://github.com/AdguardTeam/CoreLibs/issues/94

Yeah you're right, marking as duplicate


Inconsistency between vanilla breadcrumbs and react breadcrumbs format

Detailed description
The format for breadcrumbs in http://carbondesignsystem.com/components/breadcrumb/code shows the last page in the breadcrumbs string as non-hyperlink text, without trailing "/".
The format for breadcrumbs in http://react.carbondesignsystem.com/?selectedKind=Breadcrumb&selectedStory=Default&full=0&down=1&left=1&panelRight=0&downPanel=storybook%2Factions%2Factions-panel shows last page in the breadcrumb string as hyperlink text, with trailing "/".

Is this issue related to a specific component?

Breadcrumbs

What did you expect to happen? What happened instead? What would you like to see changed?

No specific expectation of one or the other, but would request consistency between vanilla and React format.

@tay-aitken @aagonzales Which is the correct pattern (or are they both correct?)
On the website, we show examples of both:
http://www.carbondesignsystem.com/components/breadcrumb/code
http://www.carbondesignsystem.com/components/breadcrumb/style

The one on the style tab is correct. The last crumb is supposed to be the page title (h1) text on the page.

@shinytoyrobots thanks for bringing this to our attention!


snap package adoption/collaboration

Hi,
I'm working on a snap package for scrcpy. I have published the snap in the snap store's edge channel. After some user feedback I think I have managed to iron out most snap related issues and consider the snap stable enough to be published in the stable channel.
My question is that do you have any plans to adopt the snap or create your own snap for scrcpy in the near future?
If not, then I would like to ask for your permission to (continue to) use the  scrcpy name for the  snap and publish it to the stable channel.
I'm currently using the master branch to build scrcpy. Is it stable enough or should I use the latest release (v1.8) instead?
Best Regards,
sisco311

Hi,
Thank you for your work :+1:

do you have any plans to adopt the snap or create your own snap for scrcpy in the near future?

No, I don't. I am very happy when other people maintain packages independantly. :wink:

If not, then I would like to ask for your permission to (continue to) use the scrcpy name for the snap and publish it to the stable channel.

Sure, please do.

I'm currently using the master branch to build scrcpy. Is it stable enough or should I use the latest release (v1.8) instead?

master is more likely to be broken occasionally, so I would recommend to use tagged versions.
Cheers

Thank you, rom1v!
v1.8 is in the stable channel. The version built from master is also available from the edge channel.
Will try my best to maintain the packages and answer any related issues.
Not sure if this is the right place to ask, but it would bee cool if you could mention the availability of the snap package somewhere in the README.
Kind Regards,
sisco311

Thank you.

Not sure if this is the right place to ask, but it would bee cool if you could mention the availability of the snap package somewhere in the README.

Please open a PR to mention it in https://github.com/Genymobile/scrcpy#linux :wink:

mention the availability of the snap package somewhere in the README

Done: de2016a48e3016e6f24858afa9c52a9a54e8ab47


added Number#times|String and Number#times|array

Hi,
Here are two additions for Number.times() function I found useful.
(3).times([1,2,3]) // returns [1,2,3,1,2,3,1,2,3]
(3).times("Ho") // returns "HoHoHo"
times(String) is basically an alias for String.repeat() which improves readability in some cases. However, I did not found an easy way to repeat an array X times. With times(Array) it's piece of cake to initialize an array with a lot of elements : 100..times([0])

I'm going to close this one out now... Thanks!