FR: Setting to add delay to highlighting

Not only would it remove distractions when writing, but having a short delay would also help with performance. In case you are considering on expanding this highlight the current sentence as well, a short delay would be very fine and would probably give the time needed for performance?

Addressed in 0.0.5

works like a charm


create relocation pom's for nanohttpd. com

Release a relocation pom for every module in 2.1.1 so the nanohttpd.org groupid is evaluated as follow up of nanohttpd. com

for the protocol.. relocation pom's deployed to central


Add to README a reminder on Win10 and internet downloaded chm

I need to add a line on the README explaining that a chm downloaded from the web may need to be marked in it's file properties to be able to read. Failing to do so results in blank file, on Windows 10.

It isn't a simple thing to explain, but I've previously written some things here:
https://www.adventuregamestudio.co.uk/wiki/Prevent_AGS_from_triggering_antiviruses
(ignore the page title, I was correcting information which was pre-existing)
It isn't limited to Windows 10, certain Windows programs will set or maintain a flag that represents a Security Zone, and the HTML rendering engine used by the CHM viewer checks for it.

Because the Wiki doesn't work for me unless I am incognito, I am going to paste current text here

Introduction
With the move to a more open source style of development for the AGS engine, game developers may use AGS builds that don't correspond to an official release. These are usually downloaded and extracted manually rather than installed with an installer and this can cause some problems with Windows security settings. Specifically, attempts to access the help file (ags-help.chm) from inside or outside the AGS editor may be blocked.
This is caused by something called "alternate data streams".
##What are "alternate data streams"?
When a file is stored on a computer by the Operating System it's written in a particular way. In order to read a file back there is a need to know where it begins and ends on the disk, so a File System is used to store data. On Windows the most popular File System is NTFS which supports the use of alternate data streams. Normally the contents of a file will be saved into the NTFS $DATA attribute in the default stream but it is also possible to specify an alternate stream name and store data in a different location within the same file. One common alternate stream name is named "Zone.Identifier" which is used to indicate the Windows Security Zone that a file originates from. This is commonly used to indicate that a file originates from another computer, the origin being indicated by a zone number. The zone numbers are defined by Zone Security settings in Internet Options.
It's typically up to individual web browsers and download utilities running on Windows as to whether they write to the "Zone.Identifier" stream when a file is downloaded, and equally any application could choose to check the "Zone.Identifier" stream of a file when it is being opened. The help file viewer is one such application and will, by default, refuse to open a file if the "Zone.Identifier" indicates it originated from a different computer (i.e. it is potentially unsafe).
When looking at a file's properties, if a restrictive zone number is present in the "Zone.Identifier" stream the file will be listed as "blocked". Choosing to unblock the file will re-write the "Zone.Identifier" stream to indicate the file should not longer be restricted (i.e. programs shouldn't be able to identify the origin of the file
How do I clean my AGS editor files of "alternate data streams"?
Usually it's just the single "Zone.Indentifier" alternate stream on the help file that causes a problem.

Right click the ags-help.chm file in Windows Explorer and press the unblock button.
If the unblock button doesn't work this is usually because the AGS folder has been located somewhere where the user attempting to unblock does not have write permission (if this is the case it's likely that the user extracting the AGS Editor would have encountered a UAC prompt when extracting to the AGS Editor files and approved the process to run with elevated permissions). The easiest solution is to run the Windows program "streams.exe" with elevated permissions. It can be download from here.

Download the zip file and extract it to any temporary directory, e.g. the Desktop.

Open a Windows Command prompt using "run-as administrator"

In the command prompt, change to to directory where the files were extracted:
cd "<EXTRACTION FOLDER>"
(you need to replace "" with the path to where streams.exe was extracted. For example "C:\Users\MyAccount\Desktop\Streams")

(use double quotes when the path contains spaces)
Run the following command to recursively remove any alternate data streams:
streams.exe -s -d "<AGS EDITOR FOLDER>"
(you need to replace "" with the path to where the AGS Editor folder was extracted. For example "C:\Users\MyAccount\My Documents\AGS Builds\ags_3.4.12")
(use double quotes when the path contains spaces)
An alternate solution is to move the AGS Editor files to a location that doesn't use an NTFS file system, and then move the files back to the computer again. Most USB memory sticks will be formatted with the FAT32 File System instead of NTFS. FAT32 does not support alternate data streams so they cannot exist on the memory stick.

I am curious of how to name this. Fixing AGS Windows Security Zone maybe? And the file being FixingAGSWindowsSecurityZone.md under a trouble shooting area, and simplified explanation for manual only in both the README and the FAQ.
I still feel the items on the index are not good, like, the stuff under Other Features is kind of hidden.

It should probably go into an 'installation' section which says how to install the program, and this should probably only be interested in discussing files which make up a release.

It isn't an issue if you use the InnoSetup installer because when the files are unpacked the files won't be present.

It is a problem where someone downloads the Editor as a zip file, because the security zone specified on the archive is added to all extracted files by the unzipping process that is built into Windows. So the important part is to remove the zone identifier before you extract it or use an unzipping tool that isn't interested in the identifier.

It is a confusing problem when someone extracts the Editor zip file to somewhere where an unelevated process cannot write such as "C:\Program Files" (Windows will helpfully prompt to elevate the copy process if you do this manually). This means that removing the zone identifier will need to be done by a process that is elevated, because otherwise you don't have permission to modify the files. The Editor tries to remove the zone identifier when the file is opened by pressing F1, but this will fail unless the Editor is running as an elevated process (which isn't an advisable thing to do, except to fix this specific problem).

So what I get is, we need to add a install h2 here on introduction, that tells to use the installer, no need to mention it comes from InnoSetup I guess, just it's a installer and where to get it. It should also have a section on getting from zip, maybe also hinting Betas usually only comes in zip form. There a very small and short item to deal with security zone.
And on a different md file like InstallationTroubleshooting add more details on the problems you stated above, and the contents of the security zone full explanation. This InstallationTroubleshooting would then be linked by the introduction too.

Só, apparently this problem of the security zone on Windows blocking a file downloaded from the internet happens not only with the manual, but also plugins downloaded from the internet. So I think it's easier to add this more general entry and then point in the needed places to this entry (the README, the AGS installation, and the area related to plugin installation).

I created the entry TroubleshootingWindowsZoneID.md and for now the only link to it is the plugin installation entry. I just roughly messed with the introduction to account for the plugin case too.


Add resolved tag to markets in portfolio/ My Bets

Turbo
Add 12px above market title
https://www.figma.com/file/6y4nvjfeVZwzwKcoXB0neq/Augur-Simplified-UI?node-id=319%3A4430

SB
Add 8px above market title
https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Sportsbook?node-id=11063%3A3130

@JohnDanz tags aren't showing up

no resolved markets so


Add build support for .NET Core

This PR allows us to target .NET Core (#140) and to track progress towards compiling successfully for .NET Core

Adds configurations to build for .NET Core
Ports Microsoft.Build.Framework to build successfully for .NET Core
Starts porting Microsoft.Build to .NET Core
Adds compilation constants and MSBuild defines for features not available in .NET Core (we will need to add more as we hit them while porting)
Adds Port-Progress configuration to build against .NET 4.6 but using the features flags targeting .NET Core, so we can successfully compile and track our porting progress with ApiPort.  Added batch file to run ApiPort against this build

Hi @dsplaisted, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

LGTM. We'll need to fix the CI build though. It would be good to at least have something in place.


Babel has deoptimised the styling

[BABEL] Note: The code generator has deoptimised the styling of<EMAIL_ADDRESS>as it exceeds the max of 500KB.
I am using the package and Material UI.

Why I see this behavior? It's an issue?

@jesusr00 You should go to the @babel organization to ask this question.
Oh, Maybe it is the log typed out by next.js. 😯

This is clearly caused by using next-remove-imports, so it's probably worth a check @jaywcjlove .

@szilarddoro

@szilarddoro @jesusr00 Can you provide an example for me to test.

@jaywcjlove - Please find my repository which tries to reproduce these issues:
https://github.com/szilarddoro/react-md-editor-demo
Node version: v14.17.0
Yarn version: 1.22.17
@uiw/react-md-editor version: 3.9.3
next-remove-imports version: 1.0.6
You can find configuration for Storybook as well so that you can try it in a normal React environment as well as a Next.js one.
My original repository is a bit more complex (also contains @mui/material, etc.), but in this case I could also reproduce an issue where react-markdown/index.js throws an error regarding ES Module requires, which are obviously not supported.
Stack trace:
Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: /path/to/project/react-md-editor-demo/node_modules/react-markdown/index.js
require() of ES modules is not supported.
require() of /path/to/project/react-md-editor-demo/node_modules/react-markdown/index.js from<EMAIL_ADDRESS>is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename /path/to/project/react-md-editor-demo/node_modules/react-markdown/index.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from /path/to/project/react-md-editor-demo/node_modules/react-markdown/package.json.

One of the solutions I could come up with was to use next-transpile-modules package and transpile both @uiw/react-md-editor and @uiw/react-markdown-preview packages and also enable the experimental esmExternals configuration of Next.js like this:
const withTM = require('next-transpile-modules')([
  '@uiw/react-md-editor',
  '@uiw/react-markdown-preview',
])

// ... other plugins ...

const nextConfig = {
  // ...other configuration...
  experimental: {
    esmExternals: `loose`
  }
}

module.exports = withTM(nextConfig)

I'm happy to help with additional information if you need anything else.

I would advise against using this lib, or any of the author's markdown libs for that matter. Or just fork them and remove the bad code practices. It will save you a lot of headache in the long run.

@gnovotny can you be more specific what headaches are you talking about? I am about to embark on this beautiful journey, so any heads up will be appriecated


build: update to Angular 8

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/ngneat/transloco/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
There's a lot to do about the dependencies of this repo
What is the new behavior?
Initially upgrade from Angular v7 to v8 and refresh some old dependencies
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Maybe the CI needs to check a correct build of all the packages too right?

Hi @itayod, could you review this PR? Thanks
Also, i think it's time to start contemplating pulling out a major version. The last LTS version of Angular is 9 and ends in August of this year. In addition there is the release of the next version 12.
Currently it is a bit difficult to work with such obsolete versions and node 14 with npm 7. What do you think?

@matheo @seravifer we will upgrade the version as part of Transloco's v3, therefor I will close this pr for now, thanks for you effort guys!


Host range overall call

There should be an overall host range call which uses the NCBI taxonomy and the literature evidence to produce an overall call. I think that the overall call should be whichever of the two is less restrictive. Biologically it makes sense to set the maximum resolution to the family level since it is unlikely to have plasmids that can only replicate in a single species.

MOB-suite v. 3.0.0 now provides an overall call for host-range


Issue with Data Augmentation in LOVE Reproduction

Hello! I am currently trying to reproduce the LOVE model, but I have encountered an issue with data augmentation.
Specifically, the paper mentions that one of the strategies for data augmentation is to replace the original word with a synonymous word. However, I noticed that the 'data/synonym.txt' file does not contain the full set of 2M vocabulary as expected.
Could you please provide the complete 'data/synonym.txt' file or, alternatively, share the code that can be used to generate this file? Thank you for your assistance!

Hi,
The synonym.txt sample file is not the final one used in our experiment. For your reference, you can use the code below to generate synonymous words:
from itertools import chain
from nltk.corpus import wordnet

synonyms = wordnet.synsets('car')
lemmas = set(chain.from_iterable([word.lemma_names() for word in synonyms]))
print(lemmas)

Best,
Lihu

Thanks, it's really helpful, and I close the issue.


Dask Device Quantile D Matrix not recognizing "qid" as key word

I am using 1.3.1, and am using the following code to make a dask Device Quantile D Matrix, and I get the following error:
dTrain = xgb.dask.DaskDeviceQuantileDMatrix(client=client, data=xTrainG, label=yTrainG, feature_names=features, qid = db.Date)

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-25-cee1058d5efe> in <module>
----> 1 dTrain = xgb.dask.DaskDeviceQuantileDMatrix(client=client, data=xTrainG, label=yTrainG, feature_names=features, qid = db.Date)

TypeError: __init__() got an unexpected keyword argument 'qid'

Based on the documentation, 'qid' should work for dask device quantile d matrix since 1.2.0?

No.  The support for qid is very recent and haven't been released yet: https://github.com/dmlc/xgboost/pull/6576 .  Then accepting all qid in constructor of DMatrix and friends is another recent PR: https://github.com/dmlc/xgboost/pull/6601 .

oh ok I must have misread the documentation, so should I close this?

@9849842 The link to document is always pointing to latest build instead of stable release.  You can choose stable release at lower right corner on the site.


Crash on multiplayer server

latest.log
This mod is barely playable with others. Random interactions, keybinds, or items crash the game - all with the same exact message:
java.lang.NullPointerException: Cannot invoke "net.minecraft.server.MinecraftServer.m_6846_()" because "com.seniors.justlevelingfork.JustLevelingFork.server" is null 	at com.seniors.justlevelingfork.registry.RegistryClientEvents.onClientChatReceivedEvent(RegistryClientEvents.java:36) ~[justlevelingfork-1.1.3.jar%23489!/:1.1.3] {re:classloading} 	at com.seniors.justlevelingfork.registry.__RegistryClientEvents_onClientChatReceivedEvent_ClientChatReceivedEvent.invoke(.dynamic) ~[justlevelingfork-1.1.3.jar%23489!/:1.1.3] {re:classloading,pl:eventbus:B}
Includes but is not limited to:

Crafting locked modded items
Using modded items (even if unlocked)
Right-clicking items unrelated to this mod on pedestals and cutting boards (?)
Pressing 'I' to enable/disable Do a Barrel Roll (???)


🛠 Tooling: Find a way around using shell: true in setup-test-e2e.js

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
See discussion in https://github.com/JoshuaKGoldberg/template-typescript-node-package/pull/349/files#r1156109852. We use shell: true in a grep command in script/setup-test-e2e.js because without it, depending on the person running the script, the command either hangs or fails with no such file or directory. Fun!
https://github.com/sindresorhus/execa/blob/main/readme.md#shell recommends against using shell: true. So it'd be nice if someone could figure out how to get the setup-test-e2e.js file working.
Additional Info
script/setup-test-e2e.js is the file run to test that script/setup.js works as expected. I generally run the file locally with pnpm run setup:test. See https://github.com/JoshuaKGoldberg/template-typescript-node-package#setup for an explanation of why we have a setup command.

Created an issue in execa for a solution - https://github.com/sindresorhus/execa/issues/555

This works without {shell: true}
const files = await globby(["*.*", "**/*.*"], {
	gitignore: true,
	ignoreFiles: ["script/setup.js", "script/setup-test-e2e.js"],
});

for (const search of [
	`/JoshuaKGoldberg/`,
	"template-typescript-node-package",
]) {
	const { stdout } = await $`grep -i ${search} ${files}`;
	assert.equal(
		stdout,
		`README.md:> 💙 This package is based on [@JoshuaKGoldberg](https://github.com/JoshuaKGoldberg)'s [template-typescript-node-package](https://github.com/JoshuaKGoldberg/template-typescript-node-package).`
	);
}

Reference: https://github.com/sindresorhus/execa/issues/555


Incompatibility between cordova-sqlite-storage and cordova-plugin-firebase

Scenario
When installed cordova-sqlite-storage in a project using cordova-plugin-firebase, app crashes when run. Only white screen is displayed.
Tested
Phonegap 6.4.3
Android 6.0.0 and Android 5.2.2

Sorry, searching for solution I found this https://github.com/fechanique/cordova-plugin-fcm/issues/213#issuecomment-274296132 the problem is with Google Firebase Panel, the file to download "google-services.json" become without app_id, so cordova-plugin-firebase can't work and crash app.


Update peer dependencies to support newer Angular versions

Hey @Allisone,
I didn't think anyone was actually using this library to be honest. 😅
I've fooled around a bit but in the end managed to add proper support for versions 16 & 17: a few testing util methods were deprecated in angular-devkit/schematics, so while your update would've worked like a charm when building the package, once I added the new versions to GitHub actions that ran tests on versions both before and after v16, things quickly went south. :D
This should now do the trick, hopefully by the time you read this you'll be able to install the new version. :)


WIP, grouping occurrences by the grouped artifact for git resources

In order to group together the occurrences related to a produced artifact, this is the code. Definitely could use some refactoring but I don't think this will end up being much smaller than it currently is. Testing is also going to be a nightmare with this, so I just want to push this out and see what we think before I commit to the whole ordeal.

Had some discussions on 06/03 around this. Seems like the grouping by produced artifact is needed. With the possibility of tfsec scans producing a build occurrence, the current logic might not work because we are looking for the built artifacts (and the built artifact is the same git resource/TBD). I'm going to leave this in draft until more thoughts are gathered around how to handle tfsec stuff.


SolverHandlerInspector Fix

Overview
The current implementation does work on a scene object, but it fails when editing a prefab. I am not sure if that's a bug, at least one of my Unity bug reports considered something similar as not being a bug, but this way the value stays.
Verification
Without the fix, creating a prefab with the SolverHandler script on it will loose the chosen TrackedTargetType every time you reselect it (deselect -> select).
With this fix, the value is remembered.

This was fixed in #5514 PR which got merged to the stabilization branch. The proper fix is actually to use EditorGUI.BeginProperty/EndProperty to get all of the prefab tracking. It failed for me too otherwise.
Once stabilization branch merges back to development again it should be there

@Troy-Ferrell thanks for calling this out! We currently are merging stabilization back into dev, which should have this fix at that point.


High Contast Dark theme looks fuzzy

#8229
Safari
Change the theme to High Contrast and text looks fuzzy

Moved to Microsoft/monaco-editor#141
Due to so many issues being opened straight in Microsoft/monaco-editor let's also move this one over there


typo in "could not be activated" issue

fixes #497. Thanks @ankon!

Still seeing this typo.

Should be fixed now. It just wasn't deployed 😉


bug: preset.lua:12 attempt to index nil value

Did you check docs and existing issues?

[X] I have read all the noice.nvim docs
[X] I have searched the existing issues of noice.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.9.0-dev-d1e0f74
Operating system/version
Ubuntu 20.04.6 LTS
Describe the bug
Hi, first of all thank you very much for this awesome plugin.
I am not familiar with lazy (I use another plugin manager), so please forgive me if that is a rookie mistake. I read the docs and afaik this is the way to configure plugins. When trying to write a reproducible example for another bug I encountered following issue. When I open nvim with attached config noice fails with preset.lua:12 attempt to index local 'options' a nil value
Steps To Reproduce

nvim -u repro.lua

Expected Behavior
There should be no errors.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
    vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
    vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
    "folke/tokyonight.nvim",
    {
        "folke/noice.nvim",
        tag = "v1.9.5", -- tested other versions as well
        config = function()
            require("noice").setup()
        end,
        dependencies = {
            -- if you lazy-load any plugin below, make sure to add proper `module="..."` entries
            "MunifTanjim/nui.nvim",
            -- OPTIONAL:
            --   `nvim-notify` is only needed, if you want to use the notification view.
            --   If not available, we use `mini` as the fallback
        }
    }
    -- add any other plugins here
}
require("lazy").setup(plugins, {
    root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Hi @folke thanks for taking a look at it. It is weird because I am able to reproduce it each time with the attached config.
After looking at some other issues I realized that I am missing one option that others have been using event = "BufReadPost", in plugin configuration.
After adding that option the issue disappeared. Maybe it is worth adding that into the minimal reproduction template?

As I said, there is simply no code path that can trigger that issue with the latest version.
Don't use that tag= thing. IT may have been broken at some point, but I'm obviously not going to look into bugs in older versions.

You get the error for options being nil at this line here:
https://github.com/folke/noice.nvim/blob/1da4edfba3e9278e98fc9723d7235d9ab3889744/lua/noice/config/preset.lua#L12
That function is called here https://github.com/folke/noice.nvim/blob/1da4edfba3e9278e98fc9723d7235d9ab3889744/lua/noice/config/init.lua#L243
Options is ALWAYS set
https://github.com/folke/noice.nvim/blob/1da4edfba3e9278e98fc9723d7235d9ab3889744/lua/noice/config/init.lua#L224
So as you can see the error you get is simply not possible.
I also tested your repro and as I figured, I don't get an error.
So whatever you're doing, you're doing something very wrong.

just fyi: I found the culprit. In nix neovim is wrapped with a script that modifies the runtime path. Even though lazy was pulling the newest version the old one from the nix store was taking precedence.
I didn't know about it until now, sorry for bothering you.


[VL] Support BroadcastNestedLoopJoinExec and CartesianProductExec

Description
These two join operators can be mapped to NestedLoopJoin  in Velox

I would like to work on it. Please let me know if I can go ahead since I was already looking into it before the issue was created.

@SurbhiVijayvargeeya cool! I just started working with some basic C++ code with the NestedLoopJoin in Velox and as far as I know there's no other people working on this. Could you please send a draft design/doc on this first, if it's convenient for you?
thanks, -yuan

Please allow me sometime. I will try to share a draft in this week.

This the initial draft doc based on my current understanding. I have started working on code changes and will add more details into it as I get more clarity.

This the initial draft doc based on my current understanding. I have started working on code changes and will add more details into it as I get more clarity.

@SurbhiVijayvargeeya thanks for this write up. Overall it looks good. One possible missing piece is on how to handle IdentityBroadcastMode broadcast xchg for BNLJ, as the current impl doesnt support this
https://github.com/oap-project/gluten/blob/a7b68c3dc0c88d117e2b6c1fdaaf8f5dd011da8c/gluten-core/src/main/scala/org/apache/spark/sql/execution/ColumnarBroadcastExchangeExec.scala#L138
The related code on broadcast small table is here:
https://github.com/oap-project/gluten/blob/main/backends-velox/src/main/scala/io/glutenproject/backendsapi/velox/SparkPlanExecApiImpl.scala#L274C40-L274C40
thanks, -yuan

Thanks @zhouyuan for pointing out. Yes, I also had a look at it. I am planning to first work on supporting BroadcastNestedLoopJoinExec to cover all these cases and then CartesianProductExec.


import-from-folder - import error

Hey Blackary,
I'm trying to use that tricks but it seams that the .py cannot find utils folder when it runs :

ModuleNotFoundError: No module named 'utils'
Do you have any ideas of something i'm missing ?

I'm on the last version of streamlit in SiS.

The first thing I would check would be to make sure your zip contains the folder utils, and not just the raw files in it. Locally, you can do this by unzipping the .zip, and seeing what comes out.
On SiS, you can do this
import zipfile
util_zip = zipfile.ZipFile('utils.zip')
st.write(util_zip.namelist())

Yes, thanks for your help, indeed, instead of doing
zip -r utils.zip ./utils/*py
probably better to do zip -r utils.zip ./utils to keep the same structure than in local mode.
Thanks for your help :)


[DOCS][SQL] Fix JDBC example

Test build #29191 has started for   PR 5192 at commit b48a33d.

This patch merges cleanly.

Test build #29191 has finished for   PR 5192 at commit b48a33d.

This patch passes all tests.

This patch merges cleanly.

This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29191/
Test PASSed.


[1.0] Do not require prefixing ::content if native support of polyfill available

.content-wrapper > ::content .special is really ugly, would be nice do not require prefix if native Shadow DOM support available or full WebComponents.js polyfill included.
(from https://github.com/Polymer/polymer/issues/1698#issuecomment-108030075)

Actually, when using Shadow DOM this recommendation can be just ignored, so I'm fine here.
But ideally this should work the same under Shady DOM too (at least if it is planned to be supported in long-run), just to be closer to native standard behavior

I've approached this issue second time and found very simple way to fix this inconvenience.
@sorvell, take a look at #2419, please.

@sorvell, this was fixed by #2419 on 30 September, can you take a look at it?
PR is really trivial.

Fixed by merged #2419, thanks


Replace path separator on windows.

Fixes https://github.com/kiteco/kiteco/issues/11149
The hover Def link was broken on Windows, causing either the link text to be incorrect or the link to not work. It appears that the call to the VSCode command (command:kite.def) itself was incorrectly formatted such that the command was being called with the argument undefined rather than the expected args. Ultimately, the issue was that the filename needed a forward slash path-separator rather than the os-specific one. Replace \\ with / fixes the link.
You can test on Windows by clicking the hover Def link and confirming that it works with this change.

@metalogical I wasn't able to find what could have changed that caused this to break, let me know if you have any ideas!


ios-snapshot-test-case does not work properly if UIApperance attributes change intrinsic content size of view being tested

I was having a problem regarding a view that wasn't being properly recorded when I changed an attribute of the view using the UIAppearance proxy, if that attribute would change the intrinsic content size of the view.
The culprit seems to be in the following method in UIImage+Snapshot.m:
+ (UIImage *)fb_imageForView:(UIView *)view
{
  CGRect bounds = view.bounds;
  NSAssert1(CGRectGetWidth(bounds), @"Zero width for view %@", view);
  NSAssert1(CGRectGetHeight(bounds), @"Zero height for view %@", view);

  // If the input view is already a UIWindow, then just use that. Otherwise wrap in a window.
  UIWindow *window = [view isKindOfClass:[UIWindow class]] ? (UIWindow *)view : view.window;
  BOOL removeFromSuperview = NO;
  if (!window) {
    window = [[UIApplication sharedApplication] fb_strictKeyWindow];
  }

  if (!view.window && view != window) {
    [window addSubview:view];
    removeFromSuperview = YES;
  }

  UIGraphicsBeginImageContextWithOptions(bounds.size, NO, 0);
  [view layoutIfNeeded];
  [view drawViewHierarchyInRect:view.bounds afterScreenUpdates:YES];

  UIImage *snapshot = UIGraphicsGetImageFromCurrentImageContext();
  UIGraphicsEndImageContext();

  if (removeFromSuperview) {
    [view removeFromSuperview];
  }

  return snapshot;
}

The problem is the following:

In the beginning of the method the current bounds of the view to be tested are stored in the bounds variable.
The view is added to a window (if it still didn't belong to one). This fires the appearance mechanism, that may set a property that calls setNeedsLayout
Then an image context is created with the size of the original bounds.
Then layoutIfNeeded is invoked on the view being tested. This may change the original dimensions of the view if the intrinsic content size changes
The view is drawn using its current bounds, but the image context is still using the original bounds

As a result, if the view has a greater intrinsic content size due to some change in appearance properties (for example, a greater font) it is clipped as it's being drawn in a smaller image context.
Is it possible to invoke layoutIfNedeedbefore UIGraphicsBeginImageContextWithOptions to use the updated size of the view? Or am I missing anything? BTW, the view I'm testing seems to be behaving correctly when resized after changing the font to a greater one in an application, so I think it's correctly implemented (I may share it if you want)

I have just noticed that this also happens with any property that changes the intrinsic content size. For example, setting a large text that makes the view grow.

@jgongo could you provide sample app?

Here you have it. I'm using Nimble-Snapshots as a wrapper over ios-snapshot-test-case, but I've checked that the parameters are being correctly passed down. If they weren't, appearance properties wouldn't be applied.
Here you have the relevant piece of code:
                UIButton.appearance().contentEdgeInsets = UIEdgeInsets(top: 15, left: 15, bottom: 15, right: 15)

                let button = UIButton(type: .system)
                button.setTitle("Click me!", for: .normal)
                button.layer.borderColor = UIColor.red.cgColor
                button.layer.borderWidth = 2

                button.snp.makeConstraints({ (make) in
                    make.width.greaterThanOrEqualTo(65)
                })
                button.sizeToFit()

                expect(button).to(recordSnapshot(usesDrawRect: true))

I've added a border to the button so you can check its bounds and a width constraint so it gets resized when auto layout does its magic. If you comment the first line (setting the content edge insets through the appearance proxy) you'll see the button snapshot gets properly recorded. If you leave it you'll see the button gets clipped on the bottom and right.
SnapshotTestCase.zip

I have just created a PR to solve the issue: #228

Hi... is this project alive? I opened this issue more than 4 months ago, I provided a PR solving the issue, and yet no progress :(

Hey, I understand and I am really sorry for this. Unfortunately I recently archived the project as we no longer use this internally and deciding about API improvements without being a consumer is incredibly difficult. I am working on a solution though to provide a better owner for it. I'll keep you posted.

I'm curious... you no longer do view based testing? Or do you use anything else?

Our number of tests grew (>10k) and intentional changes were slowing us down so we built another system (similar to this one) that doesn't rely on XCTestCase. It simply records images and stores them in the DB. This runs on CI job and If there is a change, we file a task and bisect to the offending commit.

@nscoding Is it possible to share the timeline for new FBSnapshotTest framework? And will FB opensource it?

@nscoding I imagine you won't Open source this new piece of software, but any chance you can give a quick rundown (or just notes) on what was the selected solution? How do you make snapshots of your Views/Layers without a dependency on XCTestCase? Sounds very interesting and I'd love the opportunity to dig deeper into it.

@freak4pc I want to write a lengthy post about it but I haven't found the time to do so. Generally speaking this system made it really hard for us to scale it because it requires having to record the screenshots for intended changes.
As our numbers of snapshot tests grew we needed a more async system that files tasks for changes instead of making the diffs land blocking. So our new infra is running the screenshot tests on a cron job and if there is a visual change it creates tasks and assigns them to the author. The author can then either close the task or treat it as a regression and follow up with a fix.

@nscoding I understand that the rest of the world probably hasn't the needs of Facebook :) So I guess a lot of people would benefit from this pod still being available and maintained by someone. Do you have any plans for transferring its ownership or should we fork it to go ahead?

@jgongo yes I've been trying to do this for quite some time now and we just manage to finish it
https://github.com/facebookarchive/ios-snapshot-test-case/pull/245

Thanks @nscoding  - First of all thank you so much for finding the time to reply. I really look forward to reading your article, when you get the time; whenever you publish it, would appreciate if you could share it here so other people coming from Google search can find it :)

As our numbers of snapshot tests grew we needed a more async system that files tasks for changes instead of making the diffs land blocking. So our new infra is running the screenshot tests on a cron job and if there is a visual change it creates tasks and assigns them to the author. The author can then either close the task or treat it as a regression and follow up with a fix.

This entire bit entirely makes sense. Our snapshot test cases are large at number and take up most of the time to a successful/failed test suite.
The interesting part about this is you writing this isn't depending on XCTest. This is really the part I don't entirely understand, since even on a cron job, these snapshot tests will have to run inside a simulator, unless you built something entirely from scratch instead of XCTest. If you'll have a few minutes to share a few words about that, I'd really appreciate it.
Thanks for all your work on this project !


Generating data availability roots from shard block roots

In a beacon block, we have N data roots r[1] ... r[n] (in the worst case, N = 256), each of which represent data d[1] ... d[n] of different sizes: on average 128 kB, in the worst case 512 kB. We want to combine these roots into a combined root R = f(r[1] ... r[n]) which is a Merkle root of some kind of concatenation of d[1] ...d[n].
The main desiderata are:

R can be computed from r[1] ... r[n] either directly or without adding much auxiliary data
Minimal "wasted space" in R (consider that if we force all r[i] to have the same height, on average there will be 75% wasted space)

Possibilities I see:

Allow r[i] to have different heights depending on its size, so there's on average ~30% maximum 49% wasted space. Then create R by combining all r[i] values of each size, adding a bit of padding if needed and then combining the different sizes. Main weakness is that variable heights are not SSZ-friendly.
Expose a maximum of 4 data sub-roots where r[i] = root(r[i][0], r[i][1], r[i][2], r[i][3]). Reduces wasted space even further, and is cleaner (including being SSZ compliant), but increases number of block hashes from 1 to average 1.5 max 4, increasing beacon chain load by 1.2x average-case and 2.33x worst case.
Expose 3 data sub-roots where r[i][0] is the root of the first half of the data if the data is more than half the max size and otherwise zero, r[i][1] is the root of the first quarter of the data not covered by r[i][0], and r[i][2] is the root of the first eighth of the data not covered by r[i][1]. Reduces wasted space heavily (maybe ~10% wasted), but increases beacon chain load ~1.89x, and is more complicated (including non-SSZ-friendliness from (1)).

For (1) here is an example algorithm:
def combine_roots(roots: List[Hash], sizes: List[int]) -> Hash:
    roots = []
    cur_size = 1
    ZERO_HASH = b'\x00' * 32
    while cur_size < max(sizes) * 2 or len(roots) > 1:
        if len(roots) % 2:
            roots.append(ZERO_HASH)
        roots = [hash(roots[i] + roots[i+1]) for i in range(0, len(roots), 2)]
        ZERO_HASH = hash(ZERO_HASH + ZERO_HASH)
        roots.extend([roots[i] for i in range(len(roots)) if cur_size//2 < sizes[i] <= cur_size])
    return roots[0]

For (2), here are concrete estimates of data overhead. With 64 shards, and a MAX_CATCHUP_RATIO of 4, and 4x slack for different attestations, there are a maximum of 1024 data roots in a block, plus 1024 state roots and 1024 lengths (that's 32 + 32 + 8 = 72 kB). Proposal (2) would increase the former to 1536 data roots hence 48 + 32 + 8 = 88 kB average case and 160 + 32 + 8 = 168 kB worst case. Note that average-case figures are expected to be ~8x less than these worst-case figures.
I am currently favoring (2) but open to other ideas.

I think (2) sounds very reasonable. Could we save a lot of beacon chain load by making the target shard block size just a little bit smaller than 128 kB, say something in the range of 100-120 kB? I we are expecting that the distribution around the target does not have a huge variance, then this would make a lot fewer of the second data roots necessary (not sure if there is an obvious way to get the expected distribution from EIP1559?)

I do expect the distribution to have a considerable variance! Sure, if you assume it's a normal distribution, on average a block has ~120 txs so the standard deviation should be ~10, but that ignores uneven sizes of transactions, multi-transactions, etc.


How to get the consumer lag by passing topic and cluster info.

I am new to kafka manager and from last few days going through its code to understand its functionality. I have a requirement where I need to get consumer information for different topics present in the cluster and then pass the lag information(both topic and its partition wise) to another application ( Hubble in my case). I am going through ActorModel and KafkaStateActor class to implement this but any input in this regard will be great.

u could use the api to get the info and push it wherever
/api/status/:cluster/:consumer/:topic/:consumerType/topicSummary
/api/status/:cluster/:consumer/:consumerType/groupSummary

http://<IP_ADDRESS>:9002/api/status/KafkaClusterSg/flink/client-log-id/KF/topicSummary
Action not found
For request 'GET /api/status/KafkaClusterSg/flink/client-log-id/KF/topicSummary'


Inspector

It'd be nice to have an Inspector for EarlGrey. Similar to uiautomatorviewer for Android or appium's inspector.
Key features

UI tree that displays element info
Screenshot that represents current app state

Another benefit of inspector:

Generate gesture code based on user interactions with the simulator

We have this planned for EarlGrey+XCUITest :) but I'd love if someone wants to contribute this to EarlGrey contribs until EarlGrey+XCUITest is ready.


Implement str.format()

There's a lot stdlib code (especially in the unittest library) that uses str.format() so this will become very important soon.

 Comment by S-YOU
Friday Jan 20, 2017 at 01:16 GMT

Yeah, hit that a lot when adding new modules.

 Comment by S-YOU
Friday Jan 20, 2017 at 03:08 GMT

I think we have three options.

port CPython C version to go
port pypy's newformat as python library.
use external stringformat library as python library.

What would be Ideal solution? I guess CPython version?

 Comment by trotterdylan
Friday Jan 20, 2017 at 17:35 GMT

I think newformat will be problematic because of all the pypy specific extensions used in that file.
The stringformat lib is a good find! It would require a little hackery because the grumpy Go package would need to somehow patch str.format with the generated code for stringformat.py. But it is doable and it could be a good stopgap until we can implement something ourselves.

 Comment by S-YOU
Saturday Jan 21, 2017 at 03:27 GMT

Monkey-patching method on builtins is easier on grumpy than (CPython](https://gist.github.com/mahmoudimus/295200)
Can we keep as a feature or will forbid like CPython?
str.__dict__["format"] = lambda self: "formatted: " + self
print "hello".format()

formatted: hello


plot_surface does not produce expected results

Hi,
thanks for your work on ipyvolume. I've just spent a couple of tens of minutes getting it to work and didn't manage to do it. I'm using a test case from matplotlibs gallery (here: http://matplotlib.org/gallery/mplot3d/surface3d.html#sphx-glr-gallery-mplot3d-surface3d-py).
My first test works.

However, when I'm changing the size of the array X by changing its step size in the arange method, I get weird results.

What am I doing wrong?
Here's the source:
# test case 1
from mpl_toolkits.mplot3d import Axes3D
import numpy as np

fig = plt.figure()
ax = fig.gca(projection='3d')

# Make data.
X = np.arange(-5, 5, 0.5)
Y = np.arange(-5, 5, 0.5)
X, Y = np.meshgrid(X, Y, indexing='xy')
R = np.sqrt(X**2 + Y**2)
Z = np.sin(R)

# Plot the surface.
surf = ax.plot_surface(X, Y, Z)

p3.figure()
p3.plot_surface(X, Y, Z)
p3.show()

# test case 2
from mpl_toolkits.mplot3d import Axes3D
import numpy as np

fig = plt.figure()
ax = fig.gca(projection='3d')

# Make data.
X = np.arange(-5, 5, 0.25)
Y = np.arange(-5, 5, 0.5)
X, Y = np.meshgrid(X, Y, indexing='xy')
R = np.sqrt(X**2 + Y**2)
Z = np.sin(R)

# Plot the surface.
surf = ax.plot_surface(X, Y, Z)

p3.figure()
p3.plot_surface(X, Y, Z)
p3.show()

Thanks for your help (this is an amazing project, I'm just sad it didn't work in my case... ^^)

(sorry for the late reply, was off the radar for a while)
Regarding a new version, I'd like to get that out soon, so stay tuned!

Thanks, that would be great. I really appreciate the work you've put into ipyvolume. Best regards. Florian

4.0.1 is release on pypi (conda-forge might take a day)


(python3-streams) fix AU script

Description
The zip directory naming is inconsistent, this directly uses the filenames to find the appropriate license and copyright files.
Motivation and Context
Fixes #2555
How Has this Been Tested?
Tested on Win10 and in test env
Screenshot (if appropriate, usually isn't needed):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[ ] I have updated the package description and it is less than 4000 characters.
[x] All files are up to date with the latest Contributing Guidelines
[x] The added/modified package passed install/uninstall in the Chocolatey Test Environment(https://github.com/chocolatey-community/chocolatey-test-environment/). Note that we don't support the use of any other environment.
[x] The changes only affect a single package (not including meta package).

:white_check_mark: Package verification completed without issues. PR is now pending human review

Is there any ETA on this PR? Since there has been no activity in 10 days

@JPRuskin could you kindly review? it would be nice to have a package for 3.13 almost 2 months after its release

@JPRuskin @sigma67 Just a +1 here. I'm not sure there's anything I can do to help except vote this PR up.

@AdmiringWorm Perhaps you could review this?

@rob-neobim @sigma67 Please don't ping people on PR's. The community team will get to it whentime allows.

Thanks! I can confirm revised update.ps1 works for 3.13.1. However, there is another error related to zip files for 3.14.0a2:
Downloading to python-3.14.0a2.exe - https://www.python.org/ftp/python/3.14.0/python-3.14.0a2.exe
Downloading to python-3.14.0a2-amd64.exe - https://www.python.org/ftp/python/3.14.0/python-3.14.0a2-amd64.exe
MethodInvocationException: C:\Users\yen\Downloads\chocolatey-packages\automatic\python3-streams\update.ps1:41
Line |
  41 |    $response_stream = $webrequest.GetResponse().GetResponseStream()
     |    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Exception calling "GetResponse" with "0" argument(s): "The remote server returned an error: (404) Not Found."

In https://docs.python.org/3.14/archives/, there is python-3.14.0a0-docs-text.zip but no python-3.14.0a2-docs-text.zip. I'm not sure if missing files are caused by some unexpected errors or intended changes. If the latter is the case, this pull request may need major revisions.

Thanks for checking, @yan12125! I attempted to check this out, but there are no instructions on how to test anything. Everything I try falls over with an exception An exception ocurred during Chocolatey AU update. Cancelling all other checks. or The term 'updateall' is not recognized as a name of a cmdlet, function, script file, or executable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again. Could someone who knows what they're doing explain how to test this particular package?

From https://github.com/python/pythondotorg/issues/2672

To summarise, there are no official docs built for alpha and beta pre-releases. Release candidates and full releases do come with official docs builds.

Looks like I"ll have to update this PR to take that into account.

I attempted to check this out, but there are no instructions on how to test anything.

This package is an automatic package, in that the package version (and other associated files) is updated by a CI build that uses the Chocolatey-AU module with the update.ps1. I thought there was an explanation somewhere on how to setup and test the update scripts, but the module documentation itself is the best I can find:
https://github.com/chocolatey-community/chocolatey-au/
In short, you will need to have the Chocolatey-AU module installed and imported into in powershell, and then run the update.ps1 script.
I think this package may also needs some addtional helper scripts imported as well: https://github.com/chocolatey-community/chocolatey-packages/tree/master/scripts

Thanks, @TheCakeIsNaOH. That's really helpful information, and it should probably be included somewhere in this repository. I'll have a tinker at some point next week.

Looks like I"ll have to update this PR to take that into account.

Are txt files required here? Otherwise the following files may be an option:
https://github.com/python/cpython/blob/main/Doc/copyright.rst
https://github.com/python/cpython/blob/main/Doc/license.rst
If I understand CPython's Makefile correctly, txt files are generated from corresponding rst files.

I think the license is simple enough, as that's currently just a web link: https://docs.python.org/3.13/license.html
Copyright is somewhat trickier, but we can use a regex to (badly) strip the RST stuff, preamble, and postamble:
((Invoke-WebRequest -Uri 'https://raw.githubusercontent.com/python/cpython/refs/heads/main/Doc/copyright.rst').Content) -replace "\n", " " -replace "\*+ Copyright \*+  Python and this documentation is:  " -replace "  -------.*$"

I've submitted a replacement PR #2594.

I also raised an issue upstream, noting that Appveyor is testing the wrong version of Python.

I also raised an https://github.com/chocolatey-community/chocolatey-au/issues/73 upstream, noting that Appveyor is testing the wrong version of Python.

The fix for that would be to return an ordered dictionary of the steams (in the correct versioning order), instead of returning a hashtable.

Superseeded by #2594


Can't add to HiveList if HiveObject is in inner collection

Steps to Reproduce
I want to store a box where key is user ID and value is List of Wallet (every Wallet is a HiveObject).
I also want a user to have a HiveList of its wallets.
The problem is adding to HiveList requires to check if HiveObject is inside of box.
And when I add to box not a HiveObject directly, but a List<HiveObject>, the box property of every HiveObject is null!
Desired behaviour: if I put a collection into box, every element should also have a reference to this box in its box property
Code sample
Box<List<CryptoWallet>> walletsBox;
walletsBox = await Hive.openBox<List<CryptoWallet>>('wallets');

user.wallets = HiveList(walletsBox);

var newWallet = CryptoWallet(address: '1dalrkje;j34;lkj;lkd;35234');
walletsBox.put(userId, [...existingWallets, newWallet]);
user.wallets.add(newWallet); // throws HiveError: HiveObjects needs to be in the box "wallets"
print (newWallet.box); // --> null

Version

Platform: Android
Flutter version: 1.22.6
Hive version: 1.5.0-pre

@subzero911 I found a workaround for your case. Already tried and tested.
user.wallets = HiveList(walletsBox).cast();

@subzero911 I found a workaround for your case. Already tried and tested.
user.wallets = HiveList(walletsBox).cast();

Thank you for your reply.
I found in a docs, that HIve doesn't support collections as a Box generic parameter. So it's my mistake.
The workaround is to create a wrapper WalletList class, that contains this List<CryptoWallet>.
So the box type is Box<WalletList>

user.wallets = HiveList(walletsBox).cast();
It's not intended use. HiveList is for relations (list of other objects stored in Hive).


Removing an original skill does not properly update project and user points for its imported counterparts

Create 2 projects; project A export and project B import
report few events for the original skill
verify they propagated
remove an original skill

The issue is in the import project:

overall project and subject points are not updated
users' points are not updated

PR: https://github.com/NationalSecurityAgency/skills-service/pull/1291

:+1:


PR tets

This is a fake PR to test CLA stuff.

 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.

 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.


Feature/#31 closes #31

👍
I'll keep an eye for the usage for this account and make sure the API limits won't be a problem for this project


Hardcode to retry 3 times for now

Opened chore #128286679 for further enhancement and correction.

No need to make this change, cancelling...


Add region tags and Service/Deployment to autopilot tutorial

As per discussion with stakeholders,

Added region tags
Added a basic Deployment and Service to make the Terraform complete. I tested them myself, but I am not an expert so feel free to make suggestions

Thank you for these changes!

Were you planning on also adding region tags to the two .tf files to make sure that the copyright header doesn't render here?
https://cloud.devsite.corp.google.com/kubernetes-engine/docs/tutorials/configure-networking
Could you add a link to the tutorial in the README.md? "Please follow the tutorial at ......"

Added region tags to the other TF files, let me know if I did it wrong.
The tutorial is not published yet, so probably better to wait until it is published to add the link, right?


Unity WebGL support

Really need it ASAP.
There is already "Unity - Webgl Support" item in FEATURE_REQUESTS.md,
but it links to https://github.com/aws/aws-sdk-unity/issues/50
which is a - 404 page not found.
So there is really no information on progression or problems with this feature.
As long as SDK is pure C# what is the problem with WebGL target considering
iOS (IL2CPP) is supported?
Thanks!

@AlexVK79 your feedback is appreciated on the webgl, in the meantime we are happy to accept pull requests.
The link that is present on the FEATURE_REQUEST.md points to the issue on our older git repo which is no longer maintained. I'll update the link to the current issue.

I'm waiting for the WebGL support too.
I made a couple of changes in the SDK to support Amazon Cognito (credentials, datasets, sync) and generic Async request (only tested for Dynamo) for WebGL.
The high level API for Dynamo is too complex to be patched.
The main issue was WebGL didn't support Threads (lock, WaitHandle, ThreadPool, ...) and synchronous request.
I had to remove any call to "InvokeSync" and use only one thread (main thread).
This implies to add callbacks in a lot of methods, remove call to others threads and only use the *ASync methods.
The codebase became messy that's why I can't make a pull request here. If you want a version of my updated version PM me.
Also I tried to compile the SDK for Windows Universal 10 with IL2CPP (.NET is not compatible due to the lack of System..... namespace used in the SDK).
It worked well for Cognito (credentials, dataset, sync) , I didn't test the other services.

If you have a working codebase, consider uploading that as a GitHub repo and linking it here. That will help us or a community member to get started on this work.
This issue was added to FEATURE_REQUESTS.md, closing it. Feel free to comment, of course.

Hey guys, I added a pull request for unity WebGL. It is not covering every aspect, but I think it is at least a beginning on which we can build :)
https://github.com/aws/aws-sdk-net/pull/898

Any progress with that feature?

Any updates on supporting this feature request?  Or does anyone know of a shim to get this to work?  thanks


Node loading is slow.

Loading this customer node takes more than 10 seconds, perhaps some adjustments to the logic could be considered

The fact is that the argotranslate uses a neural network and it installs all sorts of different things, you can simply delete it if you don’t use it, it just doesn’t save all the dependencies in the python environment.

Yes, this node took me about six or seven seconds to load, which is the longest load time of any node I have installed so far.

This is due to the import of ArgosTranslate specified in init.py and insertion into mapping.

This is due to the import of ArgosTranslate specified in init.py and insertion into mapping. https://github.com/AlekPet/ComfyUI_Custom_Nodes_AlekPet/blob/master/init.py#L231-L234
https://github.com/AlekPet/ComfyUI_Custom_Nodes_AlekPet/blob/master/__init__.py#L250-L251
https://github.com/AlekPet/ComfyUI_Custom_Nodes_AlekPet/blob/master/__init__.py#L266-L267

I don't think importing class and updating dict are time cost operations. In fact, I suspect the slow load is due to the module install procedure when checking modules as the following code shows:
https://github.com/AlekPet/ComfyUI_Custom_Nodes_AlekPet/blob/d8629270c944f2328d890d03a6a73e14c428ef75/__init__.py#L79-L95
The module_install function creates one subprocess and two threads which are a little heavy. Maybe the two threads are not necessary if using subprocess.check_call or subprocess.check_output instead of subprocess.Popen

This is due to the import of ArgosTranslate specified in init.py and insertion into mapping. https://github.com/AlekPet/ComfyUI_Custom_Nodes_AlekPet/blob/master/init.py#L231-L234
https://github.com/AlekPet/ComfyUI_Custom_Nodes_AlekPet/blob/master/__init__.py#L250-L251
https://github.com/AlekPet/ComfyUI_Custom_Nodes_AlekPet/blob/master/__init__.py#L266-L267

I don't think importing class and updating dict are time cost operations. In fact, I suspect the slow load is due to the module install procedure when checking modules as the following code shows:
https://github.com/AlekPet/ComfyUI_Custom_Nodes_AlekPet/blob/d8629270c944f2328d890d03a6a73e14c428ef75/__init__.py#L79-L95
The module_install function creates one subprocess and two threads which are a little heavy. Maybe the two threads are not necessary if using subprocess.check_call or subprocess.check_output instead of subprocess.Popen

There is no line from the code I provided, I thought you wanted to turn off ArgosTranslate. Let's see what we can do.

This is due to the import of ArgosTranslate specified in init.py and insertion into mapping. https://github.com/AlekPet/ComfyUI_Custom_Nodes_AlekPet/blob/master/init.py#L231-L234
https://github.com/AlekPet/ComfyUI_Custom_Nodes_AlekPet/blob/master/__init__.py#L250-L251
https://github.com/AlekPet/ComfyUI_Custom_Nodes_AlekPet/blob/master/__init__.py#L266-L267

I don't think importing class and updating dict are time cost operations. In fact, I suspect the slow load is due to the module install procedure when checking modules as the following code shows:
https://github.com/AlekPet/ComfyUI_Custom_Nodes_AlekPet/blob/d8629270c944f2328d890d03a6a73e14c428ef75/__init__.py#L79-L95
The module_install function creates one subprocess and two threads which are a little heavy. Maybe the two threads are not necessary if using subprocess.check_call or subprocess.check_output instead of subprocess.Popen

There is no line from the code I provided, I thought you wanted to turn off ArgosTranslate, after delete folder in custom_nodes. Let's see what we can do.

Oh, I misunderstood it. After deleting sub folder, some code in init.py should also be deleted. What I mentioned foreward may be another perfomance optimization point.

Yes, that’s right, you need to remove the folder from the ComfyUi_Custom_Nodes_Alekpet/ArgosTranslate folder and delete or comment out those lines where argos is imported into _init.py. There is another way - download the old init.py and replace the original one, where nodes were added to my mapping automatically and also delete the folder.

Updated init.py, added check for already installed modules and other code optimizations


Closes #5 by enforcing ipywidgets<6.0

For now this is easier than the full JS re-write that has been planned.

Coverage remained the same at 77.011% when pulling 49f0324f04d1fadf29d0a81564daf2d6463eb540 on tjduigna:master into 2819069a92eb56e5b323e860df9ca7523e7113d2 on exa-analytics:master.

Coverage remained the same at 77.011% when pulling 49f0324f04d1fadf29d0a81564daf2d6463eb540 on tjduigna:master into 2819069a92eb56e5b323e860df9ca7523e7113d2 on exa-analytics:master.


Enabling server-side-rendering for boardgame.io (fixes #22)

Fixes #22 !!!

Creates test for making sure we are not throwing errors when doing SSR. (Cool stuff that jest can emulate node!)
Replaces react-json-view with simple
 and JSON (react-json-view throws "window is not defined")
Avoid establishing socket.io connection to the server if you are, er, executing on the server.

Coverage remained the same at 100.0% when pulling 39b1e9927a1fa7b4b37d8e2e2ab21d099718dc57 on Felizardo:ssr into ed09f514ce4c882d2f0452b00ea9ef697efc2add on google:master.

Why is that node environment necessary? Jest doesn't use node?

@Stefan-Hanke Jest by default uses a browser-like environment with JSDOM, which means that it has a window object available as global. See here: https://facebook.github.io/jest/docs/en/configuration.html#testenvironment-string
With this test, we want to specifically catch those usages of window or other global objects that won't run on a node.js server. More specifically, react-json-view using the window object and trying to connect to the server using socket io were the culprits of boardgame.io not working for SSR before.

Awesome!
Will take a closer look soon. In the meantime, did you check that Mousetrap doesn't cause any problems for SSR?

@nicolodavis It is passing the test with the debug view enabled, so it does not seem to be an issue :man_shrugging:

Then I'm curious why react-json-view was the problem? I'm OK getting rid of the dependency, but I'm just wondering.

They are applying their styles using window object, and there is other people with the same issue:
https://github.com/mac-s-g/react-json-view/issues/121
Another option would be to help them making their library work with SSR too, but I am kind of prioritizing the AI stuff now :P

Btw, this PR is already live in my website:
curl https://turnato.com
curl https://turnato.com/g/chess

@nicolodavis Another option would be to render an empty debug view for SSR, so it does not render react-json-view and maybe does not cause this issue. Let me try it here :)

That's what I tried in https://github.com/google/boardgame.io/commit/252ab4055dd92ca0a8ca02da919f255a8786995b (now reverted).
I think I was running into some Rollup issues as well. You can verify that by running:
$ npm run prepare
$ npm run docsify:update
$ npm run docsify

and then navigating to the tutorial page to see that the examples work properly.

No luck with this approach indeed. I changed the code to be like this:
    let gJSON = (
JSON.stringify(this.props.gamestate.G, null, 2)
);
    let ctxJSON = (
JSON.stringify(this.props.gamestate.ctx, null, 2)
);
    if (typeof window !== 'undefined') {
      gJSON = (<ReactJson
                src={this.props.gamestate.G}
                name="G"
                enableClipboard={false}
                displayDataTypes={false}
               />);
      ctxJSON = (<ReactJson
                  src={this.props.gamestate.ctx}
                  name="ctx"
                  enableClipboard={false}
                  displayDataTypes={false}
                 />);
    }

And that's what I got:
 FAIL  src/client/react.ssr.test.js
  ● Test suite failed to run

    ReferenceError: window is not defined

      at node_modules/react-json-view/dist/main.js:1:150634
      at node_modules/react-json-view/dist/main.js:1:150596
      at e.exports (node_modules/react-json-view/dist/main.js:1:151098)
      at Object.<anonymous> (node_modules/react-json-view/dist/main.js:1:146290)
      at t (node_modules/react-json-view/dist/main.js:1:466)
      at Object.defineProperty.value (node_modules/react-json-view/dist/main.js:1:41419)
      at t (node_modules/react-json-view/dist/main.js:1:466)
      at Object.defineProperty.value (node_modules/react-json-view/dist/main.js:1:40080)
      at t (node_modules/react-json-view/dist/main.js:1:466)
      at t.exports (node_modules/react-json-view/dist/main.js:1:823)

@nicolodavis I did run npm run prepare and npm run prepublishOnly with no problems after these changes to generate the content of this branch https://github.com/Felizardo/boardgame.io/tree/package that I use as dependency on https://github.com/Felizardo/turnato

@Felizardo The people over at react-json-view got it working by delay loading it.

@Stefan-Hanke I saw that, but I think it is hack-ish and also I am not sure if it will work well with our rollup setup (the rollup project says we should avoid mixing require() and import statements)

OK, let's just remove react-json-view for now then. It's also a fairly large dependency. We can replace it with a SSR-compatible renderer in the future (or just write some code ourselves to enable collapsing / expanding of keys).


Hi, How do we learn about token balances and prices in NFT?

I need balance and price information of assets in NFT? So I'm working on a structure where I have to constantly query this, how do I ensure this?

Hi descartes, sorry this question is best suited for the developer channel in our discord. Good luck!


Extended DNS Errors a la RFC8914

Still TODO:

[x] Answer EDE code 4 Forged for local-data answers
[x] Fix unit tests
[ ] Configure option(s?) whether or not local-data should result in EDE code inclusion
[x] Write subroutine to parse packets which are REFUSED caused by an acl to either return REFUSED with or without EDE 18
[x] figure out how to do ACL PROHIBITED reply FORMERROR
[ ] Document how the log-val-level: config options influences the returned EDE option text

Not sure if I forgot to submit my previous comment a couple of weeks ago, or if it got overridden as the diff took shape.
TL;DR I am looking forward this making in into the main branch and was wondering if EDE stale answer (code 3) was going to be supported from the get go as the current diff has it commented out.
Thanks @TCY16 for tackling this!

Hi @chantra, please don't let the commented-out code dishearten you, this is very much WIP 😄
Although I can't guarantee that EDE stale answer will be in the eventual release, we'll definitively look at this particular piece of code again!


Moncole 3 part does not work well in hosted container

The monocle 3 part requires a shiny pop up to work. These pop ups are not forwarded because ports are not bound. Look for solution.

Solution is hard to find. Will close for now.


extract instruction formats

Extract instruction formats, like "R-type", "I-type", "S-type", "U-type".
Unfortunately, these are not present in the RISC-V Sail specification, so this issue is a sort of placeholder / reminder to pursue adding these to the Sail specification hosted at https://github.com/riscv/sail-riscv.

Deprecating / archiving project.


Enable option to open in system browser

Hello!
I suggest adding an option to let the developer choose whether or not to open in the inappbrowser or in the system browser (iOS: Safari etc).
http://docs.phonegap.com/en/2.9.0/cordova_inappbrowser_inappbrowser.md.html#window.open
target="_system"

I think that'd help many people. Thank you!

System browser has limitations that won't work correctly with this library.


Added borderWidth, and borderColor to roundRect

Now the roundedRect PageMenuCursor has the ability to have define the borderWidth, and borderColor of the cursor. They are set to optional just in case someone does not require them. It would have been nice to define them as nil by default not to have any API breaking change; however, that is not possible with a Tuple.

@serjooo Okay I will check

@serjooo Thank you for your cooperation


Tenant quota UI

Wip - to add validation, specs and styling. depends on  #4085
@miq-bot add_label wip
@miq-bot add_label ui
@miq-bot add_label enhancement

@lgalis #4085 is merged.

@miq-bot remove_label wip

@dclarizio tested Manage Quotas, looks good.


Fixed Cubes rendering slow

I told you this shouldn't be part of cubic chunks mod...

I am going to work on a better solution, and try to get it in Forge


Not generate message model, just example when using allOf

When I tried allOf to refer an object, the async api schema just generated empty model like below:

but the example has full properties, including extending model

Great catch. Thanks for reporting, @vodanh109!

I was unable to reproduce this. I used the followings API definition:
...
  schemas:
    lightMeasuredPayload:
      allOf:
        - type: object
          properties:
            lumens:
              type: integer
              minimum: 0
              description: Light intensity measured in lumens.
            sentAt:
              $ref: "#/components/schemas/sentAt"
        - type: object
          properties:
            command:
              type: string
              enum:
                - on
                - off
              description: Whether to turn on or off the light.
            sentAt:
              $ref: "#/components/schemas/sentAt"
...

which generated:

Can one of you provide an example API definition which causes this problem?

Yeah, I also can't reproduce it. @vodanh109, do you have an example?

Sorry for late reply, I tried with following files and run command:

ag -o ./docs test.yaml html

generator.zip

Can you please provide the content of test.yaml?

Sorry for late reply but it's already inside the zip file I attached

Sorry, I just asked for the content because it's not very secure to download and uncompress a zip file from an unknown source. Hope you understand.

Sorry. But in the zip has a lot of sub files becaus my docs have dependent hierarchy. If you can suggest any convinient way to share the source code for u without zip file, I will try :) Here is the test.yaml content
asyncapi: '2.0.0-rc1'
id: 'urn:array.example.com'

info:
  title: Very simple example
  version: '1.0.0'

channels:
  myChannel:
    publish:
      message:
        description: Simple example
        payload:
          $ref: './strategy/echo.json'

components:
  schemas:
    myObject:
      type: object
      properties:
        key1:
          title: A key
          type: string
          default: "foo"
          enum:
            - foo
            - bar
          description: Just an example enum

What about a Github repo? 😊

https://github.com/vodanh109/async-sample
Sorry for late reply. Here it is

I see. Please, do not use version 2.0.0-rc1 but instead use 2.0.0. Release candidates are not supported and some things have changed.

@vodanh109 I just used html template from latest master using your example and works like a charm imho:

Can you confirm?
also, notice Fran's comment about the spec version

unable to duplicate and anyway related issue is here already https://github.com/asyncapi/html-template/issues/11 so closing this one

I experience the issue when using nested allOf: statements.
Here is an example:
asyncapi: '2.0.0'
id: allof:test
info:
  title: AllOf Test
  version: '0.1.0'

servers:
  test-local:
    url: "localhost:{port}"
    protocol: ws
    variables:
      port:
        default: "8000"

channels:
  /signaling:
    bindings:
      ws:
        method: "GET"
        headers:
          type: object
          required:
            - Sec-WebSocket-Protocol
          properties:
            Sec-WebSocket-Protocol:
              type: array
              items:
                - type: string
                  enum: ["test-v{MAJOR}.{MINOR}"]
                  description: |
                    This informs the API that you want to talk using Protocol specified in this document.
                    The version part MUST be a valid MAJOR.MINOR release of this specification.

                    The server will respond with the same version as part of the WebSocket sub-protocol negotiation.

    publish:
      summary: Send messages nested API message
      operationId: onNested
      message:
        oneOf:
          - $ref: '#/components/messages/nested_message'

components:
  messages:
    nested_message:
      name: "Composed Message"
      payload:
        allOf:
          - $ref: '#/components/schemas/specific_envelope'
          - type: object
            properties:
              payload:
                type: string
  schemas:
    common_envelope:
      type: object
      required:
        - namespace
        - payload
      additionalProperties: false
      properties:
        namespace:
          type: string
        payload:
          type: object
      descriminator:
        propertyName: namespace

    specific_envelope:
      type: object
      allOff:
        - $ref: '#/components/schemas/common_envelope'
        - type: object
          properties:
            namespace:
              enum: ['my_variant']

This has been solved.

@jonaslagoni After two years 😂


Not returning a snowflake?

I updated to version 2 from 1 and in my testing I am not getting a snowflake, see the code I am testing with:
import { Snowflake } from "nodejs-snowflake";

const uid = new Snowflake();

console.log(
    uid.getUniqueID().toString(),
)

This yields an id such as 4101121

Hi @ghostdevv, can you try the following which should restore the previous behaviour:
- const uid = new Snowflake();
+ const uid = new Snowflake({
+    custom_epoch:<PHONE_NUMBER>000 // Default epoch < v2
+});

From version 2, the default epoch is Date.now() and that's why the IDs are smaller in size because the epoch is smaller (hence lesser bits filled). This change was made because Snowflake can generate ids that can fit inside 64 bits, so to ensure that the new users aren't starting with a huge ID (because of the 2019 epoch) and hence should be able to use the package for a longer period of time (albeit this period would be 60+ years - mathematically).
I apologize as this should change should have been documented somewhere but wasn't.

Ah makes sense, I am used to a snowflake being 18 digits long so it threw me off. I do need the 18 digits as all ids I generate need to be the same shape so will try that!

@ghostdevv feel free to reopen the issue in case your issue isn't resolved yet.


Tracking: sambamba 0.6.1 segfaults in variant2 coverage step

This is to track a (seemingly) random segfault issue in sambamba that occurs during depth/coverage counting in QC with large samples (or long BED files). The linked issue upstream is https://github.com/lomereiter/sambamba/issues/223, I will update once I know more.

Seems to be fixed upstream in sambamba 0.6.3; waiting for bioconda release to confirm.

Thanks for testing this, and to @lomereiter for the new release. I bumped the bioconda version to 0.6.3 so you should be able to update tools in bcbio and get the latest for testing. Hope this fixes the problem.

Sven-Eric and Brad,
the issue seems to have disappeared with upgrading to more recent LLVM used for compilation. Please bump the bioconda version to 0.6.5, and I mildly hope to never hear of random segfaults again.

Thanks a bunch: @chapmanb, feel free to close the issue once the version requirements have been bumped.

Artem -- brilliant, thank you so much for tracking this down. It looks like you beat me to the update in bioconda so updating bcbio will pull down the latest version. Much appreciated.


added windows 2008, 2008r2, and 7 with winrm support

This doesn't do cygwin ssh yet, as I'm trying to do this via a new (as of yet unwritten) vagrant winrm provisioner. However these boxes are up enough to 'knife bootstrap windows winrm'

Does your winrm piece work with domain controllers? Or is it using local accounts on the windows box? I haven't had luck with 'knife bootstrap windows winrm' when the VM is joined to a domain.

nm, i just found this:
winrm=WinRM::WinRMWebService.new(endpoint, :plaintext, :user => 'Administrator', :pass => 'vagrant', :basic_auth_only => true)


Error in a full session Musig2 using ed25519

When performing a full Musig2 session, error occurs with curve ed25519, while functional with secp256k1.
Every function being wrapped by SCL_ecc.mjs, it seems like an incorrect encoding

the concerned test is in src/libMPC/test_Musig2.mjs

It appears that it is related to xonly handling, when all element of partial signatures have even coordinates, the full session succeeds.

solved by
https://github.com/get-smooth/crypto-lib/commit/4194f211a8012f8d08397d05d966469f75d37738


Update Microsoft store ratings image

Here is one I got from a screenshot that you can use, taken at 9/7/2022

Hello,
The Microsoft Store, for some reason that I don't know, only shows reviews for the current country your Windows is associated to (I believe USA in this case). So, the 288 reviews are only made by people in USA. The screenshot I have on the README is from the Microsoft Partner Center, which is the portal that allows me to manage the app on the store and look at all the reviews in the world (not only a specific country).
Here are the global reviews. As you can see, there are 1.4k+, which I think is better to show in the README than the reviews from a specific country:

Off-topic
Side-rant:
This mechanism from the Microsoft Store also mean that if in a specific country (let's say Country A), one of the early adopter shared a bad review (1 star for example), but that in every other countries the app gets great reviews, then the amount of download of the app may get screwed in Country A for a long time because people may perceive the app as a bad one that shouldn't be installed.
KPI
While we're talking about reviews and acquisitions, for whoever is interested, here are some stats about the app:
Acquisition

Daily usage
It's funny how we can clearly see a drop over the weekend, which makes sense since it's a productivity app.

Retention

Engagement
One to two minutes is very short, but it makes sense since it's the kind of app people open spontaneously to do a short task (format a JSON, decode a text...) and close it as soon as they're done.

Thank you so much for this, I didn't even know about that, I left my good review and the app is just chef's kiss Thanks!


v3.0.1

Questions
Answers

Description?
PR for release v3.0.1

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?

Sponsor company

How to test?
Test the whole module behaviours

Hello @lartist ,
I found an issue that was not correct related to this this PR:
when you make changes in the wishlist configuration especially for the Wishlist default title .

steps to reproduce:

go to modules > module manager > search for the wishlist module
choose the french language > change the Wishlist default title to "Wishlist" then click on save .
FO create a new customer > choose a product to add to the wishlist > see the wishlist the Wishlist default title is changing .
try to change the language [EN ] > SEE same Wishlist default title !!

See the screen record for more informations :
Untitled_ Aug 10, 2023 6_00 PM.webm
Could you please check it
Thanks

Thanks for the test @djoelleuch. The bug is still semi-persistent so it's not really blocking the release. There are other fix worth the release. We will fix this issue on another release ;)
Could you make a dedicated issue ?
@lartist go release. !

Thanks @lartist


Add onScroll prop to Panel component

Since the Panel component already allows you to set a scrollY prop it makes sense to allow an onScroll prop as well.
Use Case

Scollspy

Example
<Panel onScroll={(event)=> console.log('scroll', event)} scrollY>...<Panel>

Great! Thanks!


ui: tooltips for database size, replication factor, and target range size

At database and table levels in admin ui, though replication factor and range size aren't yet available at database level via the ui.

Very old and out-dated. Closing.


Uncaught TypeError: Cannot read property 'error' of null at AnimationItem.triggerEvent

I'm getting this error when trying to call a goToAndPlay method on the animationRef element.
Uncaught TypeError: Cannot read property 'error' of null
    at AnimationItem.triggerEvent (lottie.js:236)
    at AnimationItem.triggerRenderFrameError

Based on the blow code the errors goes away if I remove my set state call in eventHandler.
setPath(collection[index]);
Obviously I need that to keep my queue loop going of new loottie files.
Anyone might know why this error is happening ?
const LottieCollection = ({ status}) => {
  const ref = useRef<AnimationItem>(null) as React.MutableRefObject<
    AnimationItem
  >;
  const queueStatus = useRef(status);
  const queue= [data1, data2];
  const [path, setPath] = useState<string[]>(queue[0]);
  const [state, setState] = useState<ReactLottiePlayingState>("paused");

  useEffect(() => {
    if (status === "goTo") {
      queueStatus .current = status;
      ref.current.goToAndPlay(0, true);
    }
  }, [status]);

  const eventHandler= (event: { type: string }) => {
    if (event.type === "complete") {
      if (queueStatus .current === "start") {
        console.log("loop");
        index = (index + 1) % 2;
        setPath(collection[index]);
      }
    }
  };

  return (
      <Lottie
        animationRef={ref}
        config={{
          animationData: path,
        }}
        playingState={state}
        lottieEventListeners={[{ name: "complete", callback: eventHandler}]}
      />
  );
};

Also do we really need to add the below work around to fix the animationRef typescript warning you get when you don't add this.`
useRef<AnimationItem>(null) as React.MutableRefObject<
    AnimationItem
  >;

It looks like because i'm using a queue to update the lottie animationData i'm seeing that the currentFrame of ref only updates for the first lottie animation after that it stops updating. It would seem the ref is old and not getting update internally ?


support for unicode characters

let h = [':ship:', '世' , '界']
Assuming the source code is encoded in utf8, these characters should be parsed as Const_int (code_point).
I suggest we enforce that all source code must be encoded in utf8 which is similar to other languages like golang

How would you handle the difference in encoding in the file (utf-8) and make it compatible with the representation in the JS engine (utf-16)? The above example would be a single utf-8 character but in the JS runtime it would be represented as two characters as emojis don't fit the 16 bits.

there is a mis-conception. The source encoding is not relevant to string encoding, we choose utf8 just to follow the existing conventions, we do this which is exactly the same as ts/js: whose source are encoded in utf8 yet the string are utf16.
In rescript char will be int which is large enough for any code point.

@jfrolich if you're interested you can read more about it here: https://262.ecma-international.org/5.1/#sec-6

If an actual source text is encoded in a form other than 16-bit code units it must be processed as if it was first converted to UTF-16.

The unicode topic is a bit confusing as there can be a difference between your javascript source file encoding and the encoding of strings at runtime. When you send JS files encoded as UTF-8 (which is for example the default when I open a JS file with VSCode) and send the appropriate headers, then the browser will decode the UTF-8.
@bobzhang will take care of this :+1:

there is a mis-conception. The source encoding is not relevant to string encoding, we choose utf8 just to follow the existing conventions, we do this which is exactly the same as ts/js: whose source are encoded in utf8 yet the string are utf16.
In rescript char will be int which is large enough for any code point.

I didn't say anything that contradicts what you are saying here, and I agree what you are saying. The javascript engine will transparently encode the file and external inputs from utf-8 to utf-16 (internal representation).
The the thing to reconcile is character size (a rescript char is 32-byte but a js char is always two byte). This also means that the emoji as stated above will be represented as two characters in JavaScript. This will cause issues when calculating the length of the string, or iterating over characters of a string. Perhaps @bobzhang has a solution already for this, but that was what I was alluding to.

Would it be possible to give a concrete example of what you're trying to do? Forgive me, but I always get infinitely confused about what length and character means in the context of encoded strings.
Let's say you have a plain JS file in UTF8 with an emoji.

It is logical that x.length and x[i] will be wrong here. Wouldn't you write your custom function to get the length and characters in JS?
We could compile all this info into a forum post one day 🙌

@jfrolich I am not sure if I follow you, we are just doing things exactly the same as ts/js to avoid an abstraction layer for such primitive types. we are not saying we are doing unicode right, we are aiming to be the same as JS

How would it work with emojis "composition", such as this one 👨👩👦? Would writing it as a char be a syntax error?

Writing as a char would be a syntax error, a char is a Unicode code point

Would it be possible to give a concrete example of what you're trying to do? Forgive me, but I always get infinitely confused about what length and character means in the context of encoded strings.
Let's say you have a plain JS file in UTF8 with an emoji.

It is logical that x.length and x[i] will be wrong here. Wouldn't you write your custom function to get the length and characters in JS?
We could compile all this info into a forum post one day 🙌

Yes I was talking about @hongbo example because he is using a char type. In a string this is absolutely correct behavior, x.length in JS also gives the utf16 length and not the real length in characters. That is just an inconvenience of the platform.

@jfrolich I am not sure if I follow you, we are just doing things exactly the same as ts/js to avoid an abstraction layer for such primitive types. we are not saying we are doing unicode right, we are aiming to be the same as JS

I agree that it is correct behavior with strings, but this becomes problematic if you have a char type as in the example. How would you handle chars here?

Writing as a char would be a syntax error, a char is a Unicode code point

Which unicode though? Utf-8 (outside world: files, network) or Utf-16 (internal representation).

Unicode is standardized which does not depend on the encoding method. For source code we will use utf8. This is all the same approach used by js/ts

Unicode is standardized which does not depend on the encoding method. For source code we will use utf8. This is all the same approach used by js/ts

So you'll have conversion functions between array of int32 to strings and the other way around? @bloodyowl's comments are interesting, because a single utf-8 character can actually be more than 4 bytes because it can be composed of multiple unicode scalars. But I guess that can be out of scope for the character type?

Unicode is standardized which does not depend on the encoding method.
Javascripts unicode codepoints are 16-bit, so for an emoji you need two codepoints while in utf8 it fits in a single codepoint (or 4 bytes).

Let’s keep the discussion focused on the implementation details. We can discuss more about Unicode on the forum

Yes I agree. I am not keen in talking unicode in general. But how rescript is planning to reconcile the char type in the current proposal (four byte code points, because you used a four byte emoji as a char), and the javascript utf16 two byte codepoints.

hi @IwanKaramazow I double checked the theory is sound, so I guess you can start working on it when you have time
I scratched some suggestions for your references
https://docs.google.com/document/d/1V99iVjIncN7seNe-oECa_PU7Sx4ajz8xIH-qDg1vS6Y/edit?usp=sharing


Redesign Footer

the footer is an important part of the website, we would like to request a new and more attractive design for it.

Hey @Khizarshah01 do you have a design for it? I can work on development part if you have any design you want to be build.

What important information would you like to include in your footer?

@TJRelly @shyamtawli we don't have a specific design in mind for the footer at this time.
We would appreciate it if you could use your best judgment to create a footer that is visually appealing and consistent with the overall design of the website. Please keep in mind the following elements that we would like to include:

A copyright notice, including the year the website was created or last updated.
A link to our website's contact page.
A link to our website's about page.

If you have any questions or need any additional resources to complete the task, please let us know. We appreciate your help in improving our website!


How to create repo with readme file

Here i try to create a repo on azure devops using rest api, i successfully create empty repo but i need to create repo with README.md file. None of part in that docs should not explain these. Any idea about that?

Well solve the issues with push with empty file.


Implement failure reporting

Currently when running Cucumber..run() nothing it reported back to the caller. This changes this behavior by making the result value a result with either () in the regular case (arguably could be the number of successes or something) and u64 if any errors have been found. This number also equals the total number of failure found.
To implement this a handy function failed() is added to the various events allowing for checking down the tree whether the currently reported item reports a failure or not. All functions have minimal rustdoc documentation added.

Sorry, only saw now that there is another PR pending implementing result reporting on run. However, it was moved into draft without further notice. Is this generally unwanted? What is the approach thought to be used instead?

Thanks for the contribution! The preferred approach has been discussed here: https://github.com/bbqsrc/cucumber-rust/issues/84#issuecomment-721998855.
I think with a little rework your PR can solve this issue. :smile:

yeah, that's easy enough to implement. So I guess this means we want it to report that new Output-type in any case (rather than a Result) and have that inform us whether failures where found? Only remaining question then would be whether to implement that reporting on the EventHandler or raw in the Cucumber-Command. I'd argue the latter as it is generic over different handlers, but that also means the current "tracking" would happen twice or we'd have to alter the EventHandler to take the final counts from Cucumber and not count itself internally... not a clean design either way. Thoughts?

Not duplicating the count is the best option. Other APIs can be refactored later. Ideally counting things would be functionally separate to the event handlers, as they exist to enable pretty output, XUnit and other 'outputters', and should not have to care about reimplementing the counting logic, as it must be the same anyway.
Perhaps some kind of counter type can be passed around with an Arc to deduplicate the counts in the first place.

@bbqsrc what do you think about this way?

Can you rebase onto the current main and force push your branch? CI should hopefully run then. 🙂

Nice. :smile: Any other changes or do you think it's ready?

Should be good to go :) .

Thanks! Only thing left from my original suggestion is a run_and_exit function, so I would happily accept a PR for that, otherwise I'm sure someone will get to it, if not me. 😄


Add [WazirX wazirx.com] #exchange #memo-required GA6SXI…UFIMFF

Address: GA6SXIZIKLJHCZI2KEOBEUUOFMM4JUPPM2UTWX6STAWT25JWIEUFIMFF
Tags: exchange memo-required
Title: WazirX  Domain: wazirx.com
Requested by @dhoppus

1527135629

Hello @dhoppus
This issue report was combined with https://github.com/stellar-expert/public-directory/issues/552
which seems to be the same request.
Looking Towards The Future With You,
Rob

1527135629


.drag() with viewBox set

I'm using the default Element.drag() handler with a viewBox applied and it doesn't scale the mouse delta's accordingly, so the drag effect is amplified by 2-3x what it should be, is there an easy way to take the viewBox scaling factor into account?

This issue has opened on 31 Oct 2013 and I think it's still a critical issue.
Is it planned to fix this soon @DmitryBaranovskiy ?

@DmitryBaranovskiy  also experiencing the same issue +1

same issue +1!

Tried with this... http://svg.dabbles.info/snaptut-dragplugin ?

@ibrierley Thanks this works well... :)

I am also experiencing the mentioned issue. The solution of @ibrierley worked like a charm. This isn't very nice behaviour though. +1

This is not an issue and implemented like this by design.


Error start_with invalid byte sequence in latest_version_finder

Got the following error on a yarn dependabot job :
/usr/local/bundle/gems/dependabot-npm_and_yarn-0.117.6/lib/dependabot/npm_and_yarn/update_checker/latest_version_finder.rb:339:in `start_with?': invalid byte sequence in US-ASCII (ArgumentError)

I experienced the same issue with a PHP package few months ago but upgrading dependabot-core fixed the issue. Now I reproduce the problem on a yarn package.
Traceback (most recent call last):
	14: from main.rb:101:in `<main>'
	13: from main.rb:101:in `each'
	12: from main.rb:111:in `block in <main>'
	11: from /usr/local/bundle/gems/dependabot-common-0.117.6/lib/dependabot/update_checkers/base.rb:27:in `up_to_date?'
	10: from /usr/local/bundle/gems/dependabot-common-0.117.6/lib/dependabot/update_checkers/base.rb:165:in `version_up_to_date?'
	 9: from /usr/local/bundle/gems/dependabot-common-0.117.6/lib/dependabot/update_checkers/base.rb:207:in `numeric_version_up_to_date?'
	 8: from /usr/local/bundle/gems/dependabot-npm_and_yarn-0.117.6/lib/dependabot/npm_and_yarn/update_checker.rb:22:in `latest_version'
	 7: from /usr/local/bundle/gems/dependabot-npm_and_yarn-0.117.6/lib/dependabot/npm_and_yarn/update_checker.rb:197:in `latest_version_details'
	 6: from /usr/local/bundle/gems/dependabot-npm_and_yarn-0.117.6/lib/dependabot/npm_and_yarn/update_checker.rb:170:in `latest_released_version'
	 5: from /usr/local/bundle/gems/dependabot-npm_and_yarn-0.117.6/lib/dependabot/npm_and_yarn/update_checker/latest_version_finder.rb:33:in `latest_version_from_registry'
	 4: from /usr/local/bundle/gems/dependabot-npm_and_yarn-0.117.6/lib/dependabot/npm_and_yarn/update_checker/latest_version_finder.rb:99:in `valid_npm_details?'
	 3: from /usr/local/bundle/gems/dependabot-npm_and_yarn-0.117.6/lib/dependabot/npm_and_yarn/update_checker/latest_version_finder.rb:271:in `npm_details'
	 2: from /usr/local/bundle/gems/dependabot-npm_and_yarn-0.117.6/lib/dependabot/npm_and_yarn/update_checker/latest_version_finder.rb:316:in `check_npm_response'
	 1: from /usr/local/bundle/gems/dependabot-npm_and_yarn-0.117.6/lib/dependabot/npm_and_yarn/update_checker/latest_version_finder.rb:339:in `private_dependency_not_reachable?'
/usr/local/bundle/gems/dependabot-npm_and_yarn-0.117.6/lib/dependabot/npm_and_yarn/update_checker/latest_version_finder.rb:339:in `start_with?': invalid byte sequence in US-ASCII (ArgumentError)

For me it helped to set the proper locale on the host where dependabot runs.
As example following helped to solve this problem:
ENV LANG C.UTF-8

Hi @laurieplo. I think @fatroom's advice to set a UTF-8 locale in your runner environment should fix your issue. You may also try this solution by @wengjq:
export RUBYOPT="-KU -E utf-8:utf-8"