Ethereum puzzle solver package

Master automated transfer
Manual transfer will not work, because most wallets does not allow entering private key in HEX format. May be still there exists such wallet.
Write a guide how to transfer assets from one Ethereum address to another.
For this, may seth or geth may be used:
https://docs.makerdao.com/clis/seth
https://github.com/ethereum/go-ethereum
Private keys and addresses:
private key:  23d4a09295be678b21a5f1dceae1f634a69c1b41775f680ebf8165266471401b
address:      0xCFcb12D849569A3d1Cf971721665B2dCe0279A91
balance:      61 wei

Source: https://privatekeyfinder.io/brainwallet/ethereum/?page=2
private key:  59b47e9ec520a9513f404b325fa84662202adbb134c73cc57cbb0f7cea25c689
address:      0xF08A9664E46bC52878C35894391AE520a1ebBd2A
balance:      0.00007664 ether

Source: https://privatekeyfinder.io/brainwallet/ethereum/?page=181
private key:  725544de96fa7e38e7c1e3911d79770f6de178c52d4b87702ad820f5fe7e135c
address:      0x854C77c8D57C770360fB00569C2fc29e03C089bE
balance:      0.00000023 ether

private key:  71099161d22b90d8d4d95922f3781027661f80107c2a7bf7bdecb1984455610c
address:      0x6CEbfF11d72A10Dc183CF61b813fA173eA9A1Bd2
balance:      10 gwei

private key:  49ae7eff833b4aa1400a87b9c12ff44ebd5349e87ea9ef20fa10f9ae8c0b44bb
address:      0xF0737b678059A2dEcc1293B755D481ad7301292b
balance:      0.00000234 ether

Source: https://privatekeyfinder.io/brainwallet/ethereum/?page=192
private key:  48047b49b975ee8880742ea2544b30e8965f1675df787e9d832f8e0c2db87630
address:      0x41791393b667240cED1F1AfD6bfedD91ad19425e
balance:      0.0001 ether

Source: https://privatekeyfinder.io/brainwallet/ethereum/?page=287
private key:  6d08a9aaf3685faba55c76e3722163be8305f819647eb983453c6f2cbb5731a8
address:      0xC260b9dbf37A879e4726fDac8E147cb8Fd08ed64
balance:      0.000001 ether

Source: https://privatekeyfinder.io/brainwallet/ethereum/?page=315
The first key in multiple networks: https://privatekeyfinder.io/key/0000000000000000000000000000000000000000000000000000000000000001
See also: https://github.com/deep-foundation/deep-packages/issues/68

https://learnmeabitcoin.com/technical/public-key
https://learnmeabitcoin.com/technical/ecdsa#elliptic-curve-mathematics

https://github.com/ryancdotorg/brainflayer


Add Windows blur

This is reallay cool, now I know how to make th tilebar work after unmaximizing!


Vagrantfile for general use...?

This requires further testing, but I think(?!) this should work to create a fully-functional VirtualBox VM for fmrbenchmark v0.0.1, with all dependencies included.
Anybody interested in trying it out on their machine and giving me some feedback? :)
https://github.com/cmcghan/vagrant-fmr2016

I skimmed the sources at the URL that you provide. Good work! I will review it more thoroughly. Is this the best place to give comments, or should I open issues in the tracker at https://github.com/cmcghan/vagrant-fmr2016?
Did you know about remote/Vagrantfile?

Sure, please do! :) You can give comments either here or at the tracker at https://github.com/cmcghan/vagrant-fmr2016 -- whichever is most convenient for you. I should see notifications in either place.
I did not know about the remote/Vagrantfile in the fmrbenchmark repo; guess I should've explored the subfolders a bit more :) Yours looks much more lightweight than mine. Mine is admittedly meant to be more of a 'one-click' creation that is fully set up and ready to go (or as close to as possible).
FYI, I'm planning on adding a 'FMRVERSION' command-line parameter to install_deps.sh, so that v0.0.1 or v0.0.2 can be chosen between. Hopefully this weekend.

In the section that contains git clone, vagrant up, and other commands of the README, should it use vagrant-fmr2016 instead of vagrant-rss?

Yikes! Thanks -- fixed. Let me know if you find anything else? (And yeah, I'll need to get this working for v0.0.2-0.0.4 at some point...)

@cmcghan thanks for your work on this. following results #12, it is best to use Docker containers to provide for reliable reproduction of benchmarks and examples, e.g., domains/integrator_chains/Dockerfile. Using Docker has the additional advantage that controllers under evaluation can be separated from the benchmarks rather than having all code run as the same user on the same machine.
have you used this Vagrantfile since you opened this issue, and if so, are there some good use-cases you found that are not addressed by Dockerfiles?

closing this for now as it appears abandoned. @cmcghan feel free to open the discussion again.


build on linux

Hi, trying to install on linux (Linux 4.13.0-38-lowlatency x86_64 GNU/Linux). I followed the instruction in README, but I get on the first taktil build the following error:
[taktil] building...
TypeError: Cannot read property 'chunkGroups' of undefined
    at module.exports.dependency (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/html-webpack-plugin/lib/chunksorter.js:30:35)
    at BitwigWebpackPlugin.optimizeChunkAssets (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/bitwig-webpack-plugin/index.js:6:31)
    at compilation.hooks.optimizeChunkAssets.tap.chunks (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/bitwig-webpack-plugin/index.js:24:26)
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:9:1)
    at AsyncSeriesHook.lazyCompileHook [as _callAsync] (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/Hook.js:35:21)
    at hooks.additionalAssets.callAsync.err (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compilation.js:952:36)
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:6:1)
    at AsyncSeriesHook.lazyCompileHook [as _callAsync] (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/Hook.js:35:21)
    at hooks.optimizeTree.callAsync.err (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compilation.js:948:32)
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:6:1)
    at AsyncSeriesHook.lazyCompileHook [as _callAsync] (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/Hook.js:35:21)
    at Compilation.seal (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compilation.js:890:27)
    at hooks.make.callAsync.err (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compiler.js:481:17)
    at _err0 (eval at create (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:11:1)
    at _addModuleChain (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compilation.js:758:12)
    at processModuleDependencies.err (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compilation.js:697:9)
/usr/local/lib/node_modules/taktil/bin/tasks/build.js:46
            stats.toString(
                  ^

TypeError: Cannot read property 'toString' of undefined
    at onBuild (/usr/local/lib/node_modules/taktil/bin/tasks/build.js:46:19)
    at finalCallback (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compiler.js:156:39)
    at onCompiled (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compiler.js:164:20)
    at compilation.seal.err (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compiler.js:482:22)
    at hooks.optimizeChunkAssets.callAsync.err (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compilation.js:954:14)
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:12:1)
    at AsyncSeriesHook.lazyCompileHook [as _callAsync] (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/Hook.js:35:21)
    at hooks.additionalAssets.callAsync.err (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compilation.js:952:36)
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:6:1)
    at AsyncSeriesHook.lazyCompileHook [as _callAsync] (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/Hook.js:35:21)
    at hooks.optimizeTree.callAsync.err (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compilation.js:948:32)
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:6:1)
    at AsyncSeriesHook.lazyCompileHook [as _callAsync] (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/Hook.js:35:21)
    at Compilation.seal (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compilation.js:890:27)
    at hooks.make.callAsync.err (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/webpack/lib/Compiler.js:481:17)
    at _err0 (eval at create (/home/xxx/uh18-osc2midi-bitwig-controller/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:11:1)

The second error I can easily prevent with commenting out the console.log(). But on the first I'm stuck. Do you have a hint?

I recently updated the BitwigWebpackPlugin to work with Webpack v4, and this error looks like it might be connected to that. Can you put you're project up somewhere for me so that I can load it locally and try and reproduce the problem?

thanks for the fast reply!
=> https://github.com/comixtrip/taktil-test

@comixtrip I just pushed out a patch release of bitwig-webpack-plugin that should solve your problem. Running npm upgrade from your project root should  get things working for you.
The bitwig-webpack-plugin depends on an undocumented function from inside html-webpack-plugin and the way I had the dependency setup it was installing the latest 3.x.x version, but they pushed a breaking change in a patch release and it broke my plugin. I've now locked it down to a specific version, so it shouldn't happen again.
Once you let me know it's working for you I'll close this. Thanks for the heads up! Also if you have any other questions about Taktil in general, let me know. I'm also on the Bitwig discord server if you ever want to chat there.


Better naming of sub-packages

Examples of very generic names:

go/lib/pathdb/conn
go/lib/pathdb/query
go/lib/pathdb/sqlite (even clashes with go/lib/sqlite)
go/lib/overlay/conn
go/lib/ctrl/extn
go/lib/ctrl/ifid
go/lib/ctrl/seg
go/lib/infra/transport

As an example, if a reader sees conn.Conn in a go file, that gives them zero idea what that might actually be. They have to consult the import list, on a per-file basis.
To put it another way, any package that's imported by something other than its parent should have an identifiable name. E.g. go/lib/pathdb/conn could be go/lib/pathdb/pdbconn. That way pdbconn.Conn is unambigious.

And, while we're at it, let's rename go/lib/infra/disp to perhaps go/lib/infra/infradisp, to reduce confusion with the scion dispatcher.

I agree that package names are not ideal right now. The golden rule for package naming should be https://github.com/golang/go/wiki/CodeReviewComments#package-names.
Since this is an ongoing effort (that is also mentioned in the style guide we reference in https://github.com/scionproto/scion/wiki/Coding-Guidelines), we should create individual issues for each package name that should be refactored. That will make it much easier to track than a catch-all issue. It will also make it easier to discuss what represents a good package name, because every instance of the naming problem is slightly different.
Regarding the suggestions in this issue, pathdb/conn.Conn should maybe become pathdb.Conn (similarly, pathdb.Query). The other packages will probably be going away altogether with the move to gRPC/standard UDP underlay sockets.

The coding guidelines document you linked to seems to be down?

The coding guidelines document you linked to seems to be down?

Yes, sorry about that. We recently moved our documentation to a Read the Docs site. The new location for the Go style guide is here.

The package name guidelines are here. Closing this because it's not actionable.

The package name guidelines are here. Closing this because it's not actionable.


KernelMemory: Include tags in answer's "RelevantSources"

I load text and data from different sources into memory. These are PDF's, text files but also parsed website data.
For every webpage, I already have a plain-text representation and know the Url for it, but I don't need to fetch it because I get the data from an export of Adobe Experience Manager.
In a chat application, I would like to render buttons that point to relevant web sources (url's).
When importing data, I add a tag containing the source (Url) for a webpage. But unfortunately, RelevantSources does not return the tag information.
As workaround, I encode the URL to base64, append ".txt" and use it as file path which is quite a hack.
Would be nice to be able to the tags as part of the relevant sources when asking memory.

Sorry, I should have better post this in the Kernel-Memory repo:
https://github.com/microsoft/kernel-memory/issues/247


[BUG] URI fragments don't get handled properly

General Information
Version: 0.6.0
Installation Method: cargo
Operating System: macOS
Backend (If changed during install): default
Describe the bug
When the target URL of a link contains an URI fragment, it doesn't get handled nicely by the opening dialog (it is left in its raw form, see the screenshot below) and when opening said article, it stays at the top of the article and doesn't scroll to the required position.
To Reproduce
Steps to reproduce the behavior:

Open the Atom (text editor) wiki page in wiki-tui
Scroll down to the link unique UUID v4 random identifier (it is almost at the bottom of the article)
Open said link

Expected behavior
The user should be informed by the dialog box that the link is pointing to a specific position in the article and the program should scroll down to said position when opening the article.
Screenshots

Checklist

[x] checked other issues for the same bug
[x] read CONTRIBUTING.md

Blocking this issue because the following issues have to be completed first:

#135


Add bespoke deepfloyd license

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


[$250] Chat - System message about adding tag in Parent: Child format shows Parent: Child

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.81-4
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/43392
Action Performed:

Go to URL https://staging.new.expensify.com/
Log with any account
Go to account settings -> Workspaces -> Create WS if needed
Enabled Tags
Add a tag in Parent: Child format
Go to #admins room and check the system message

Expected Result:
System message about adding tag in Parent: Child format shows as Parent: Child, not Parent\: Child
Actual Result:
System message about adding tag in Parent: Child format shows Parent\: Child
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014a67ec2b09731f24
Upwork Job ID:<PHONE_NUMBER>428158713
Last Price Increase: 2024-06-11
Automatic offers:

eh2077 | Reviewer | 102720454

Issue OwnerCurrent Issue Owner: @eh2077

@sakluger  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

FYI @lanitochka17 - GitHub was removing the \ from your comment because it's a Markdown formatting character in GH - it's an escape character used to ignore other markdown formatting (more details here).
I updated your comment to add that character back. 😄

Proposal
Please re-state the problem that we are trying to solve in this issue.

System message about adding tag in Parent: Child format shows Parent: Child

What is the root cause of that problem?

With the system message about adding tag, BE returns a message like:

{
    "actionName": "POLICYCHANGELOG_ADD_TAG",
    "message": [
        {
            "html": "added the tag \"name\\: age\" to the list \"Tag\"",
            "text": "added the tag \"name\\: age\" to the list \"Tag\""
        }
    ]
}

And we display this message by using ReportActionItemMessage:
https://github.com/Expensify/App/blob/8d11d0b185083a6a813daeb0fe7c0697a5627ef6/src/pages/home/report/ReportActionItem.tsx#L647
that will display a message base on raw reportAction.message data.
So that leads to the bug.

What changes do you think we should make in order to solve the problem?

We need to add an additional else logic to this:
https://github.com/Expensify/App/blob/8d11d0b185083a6a813daeb0fe7c0697a5627ef6/src/pages/home/report/ReportActionItem.tsx#L638

 else if (action.actionName === CONST.REPORT.ACTIONS.TYPE.POLICY_CHANGE_LOG.ADD_TAG){
            children = <ReportActionItemBasicMessage message={PolicyUtils.getCleanedTagName(action.message?.[0]?.text ?? '')} />;

Then, we add an additional else logic to this:
https://github.com/Expensify/App/blob/8d11d0b185083a6a813daeb0fe7c0697a5627ef6/src/libs/SidebarUtils.ts#L378

else if (lastAction?.actionName === CONST.REPORT.ACTIONS.TYPE.POLICY_CHANGE_LOG.ADD_TAG) {
            result.alternateText = PolicyUtils.getCleanedTagName(lastAction?.message?.[0]?.text ?? '');

to fix the LHN issue.
What alternative solutions did you explore? (Optional)

There are a few component where we have the same issue, for example, copy to clipboard action and we need to address them as well.

Thank you for your proposals!
@devguest07 Your proposal is close to success. You dived into the code and found a line to fix the effect, though that's not the best way to fix it. No worries, as you seem new to the project. Just a small tip for you: try to find out if there are similar existing coding patterns in the codebase when you're going to make changes. I believe you will soon land on a successful proposal.
@truph01 's proposal looks good to me. Their solution fixes both report and LHN message. More detail changes can be discussed in PR.
🎀👀🎀 C+ reviewed

Looks good, thanks!

@truph01 Kindly let us know when we can expect a PR to be ready for review, thanks!

@eh2077 I created PR https://github.com/Expensify/App/pull/43765. It can be reviewed now.

@sakluger this issue was fixed by some other PR and is not reproducible anymore, but we had a working PR and all, can you proceed with payment to @truph01 and @eh2077 ?

Summarizing payment on this issue:
Contributor: @truph01 $250, payable via Upwork
Contributor+: @eh2077 $250, paid via Upwork

Not sure why Melvin think this is overdue

@truph01 could you please apply to the job on Upwork (https://www.upwork.com/jobs/~014a67ec2b09731f24) or share your Upwork profile link so that we can pay you for your work and close out the GH issue? Thanks!

@truph01 could you please apply to the job on Upwork (https://www.upwork.com/jobs/~014a67ec2b09731f24) or share your Upwork profile link so that we can pay you for your work and close out the GH issue? Thanks!

@sakluger I actually messed up and got things wrong, the main issue is still reproducible, and we are fixing it in the PR, I am really sorry about that.

@sakluger I accepted that offer, TY

Thanks, all paid!
@eh2077 we're just holding for the BZ checklist before we can close this issue.

Bump @eh2077

Checklist

[x] [@eh2077] The PR that introduced the bug has been identified. Link to the PR: There's not such PR that causes this bug because the system message if from backend.
[x] [@eh2077] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: N/A
[x] [@eh2077] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
[x] [@eh2077] Determine if we should create a regression test for this bug. No

cc @sakluger


[GIT PULL] Fix madvise syscall wrapper

The current wrapper uses __do_syscall2 (wrong arity, should be 3) and ignores the advice parameter.

git request-pull output:
The following changes since commit fd1d6cfafb4e9fb4d826175247527c7d1ca4ffc1:

  test/pollfree: use mmap(2) instead of a manual syscall (2022-02-21 05:11:29 -0700)

are available in the Git repository at:

  https://github.com/Spasi/liburing.git fix_madvise_syscall

for you to fetch changes up to e05652f22ee5b5bcc78f69a09ed7a7ba883a928a:

  Fix madvise syscall wrapper (2022-02-21 15:33:09 +0200)

----------------------------------------------------------------
Ioannis Tsakpinis (1):
      Fix madvise syscall wrapper

 src/arch/x86/syscall.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Click to show/hide pull request guidelines
Pull Request Guidelines

To make everyone easily filter pull request from the email
notification, use [GIT PULL] as a prefix in your PR title.

[GIT PULL] Your Pull Request Title

Follow the commit message format rules below.
Follow the Linux kernel coding style (see: https://github.com/torvalds/linux/blob/master/Documentation/process/coding-style.rst).

Commit message format rules:

The first line is title (don't be more than 72 chars if possible).
Then an empty line.
Then a description (may be omitted for truly trivial changes).
Then an empty line again (if it has a description).
Then a Signed-off-by tag with your real name and email. For example:

Signed-off-by: Foo Bar<EMAIL_ADDRESS>
The description should be word-wrapped at 72 chars. Some things should
not be word-wrapped. They may be some kind of quoted text - long
compiler error messages, oops reports, Link, etc. (things that have a
certain specific format).
Note that all of this goes in the commit message, not in the pull
request text. The pull request text should introduce what this pull
request does, and each commit message should explain the rationale for
why that particular change was made. The git tree is canonical source
of truth, not github.
Each patch should do one thing, and one thing only. If you find yourself
writing an explanation for why a patch is fixing multiple issues, that's
a good indication that the change should be split into separate patches.
If the commit is a fix for an issue, add a Fixes tag with the issue
URL.
Don't use GitHub anonymous email like this as the commit author:
<EMAIL_ADDRESS>
Use a real email address!
Commit message example:
src/queue: don't flush SQ ring for new wait interface

If we have IORING_FEAT_EXT_ARG, then timeouts are done through the
syscall instead of by posting an internal timeout. This was done
to be both more efficient, but also to enable multi-threaded use
the wait side. If we touch the SQ state by flushing it, that isn't
safe without synchronization.

Fixes: https://github.com/axboe/liburing/issues/402
Signed-off-by: Jens Axboe<EMAIL_ADDRESS>

By submitting this pull request, I acknowledge that:

I have followed the above pull request guidelines.
I have the rights to submit this work under the same license.
I agree to a Developer Certificate of Origin (see https://developercertificate.org for more information).

Sorry for introducing this bug.


Month scroll right two at a time

When I scroll months by swiping to the right on months view, they scroll two at a time. It works fine when swiping left or when swiping right on the calendar itself. Here's a video demonstration: https://gfycat.com/UnfoldedCanineClownanemonefish

Thank you, I noticed this bug, it's because I use pagingEnabled of the UIScrollView.
#65


How replace tags  and ?

Hi friends!
I'm having a problem with paste text from other pages when it's contain old tags like  and .
How can I replace these old tags to  or
 tags?
When I paste this example, I lost the color green:

Inquire about VIP Status. *Existing clients ONLY!

Thanks!

You can do this by customizing a quilljs module to listen to the clipboard and replace those labels before pasting.
https://github.com/kensnyder/quill-image-drop-module/blob/master/index.js#L21
https://quilljs.com/docs/modules/


Rifts

I have a big problem. I have many Rifts in my base and dont know how to close them.
Is there a way to close them in the newest beta build?

Currently there's no way to close them. Azanor plans to add that, hopefully in B17
#954

If you switch to creative, theres a flux scrubbing sponge, you can give yourself that, then switch back to survival, Ive just done that on my world until something gets added for survival

Sponge Flux - removes the Flux.

I'd still like a non-cheat version. I've read that rifts used to go away on their own, so I guess they not doing that now is a bug?

Actually intended, as Azanor wrote in #954.
This behavior changed in B14 I believe (I'm still running B13), and currently there's no survival way to get rid of them.

Just to be clear, ways of getting rid of rifts are coming in B17 or future betas, it's not intended that they stay there permanently. Just for now.

Actually intended, as Azanor wrote in #954.
Indeed. Thank you. Ah well, I was waiting for a "fix" anyway, so I guess now I'm waiting for a regular update instead. ^^


Error when converting a distribution from WSL1 to WSL2

After installing WSL2 I tried to convert an existing WSL1 to WSL2 using this command:

C:\Users\warpd>wsl --set-version Ubuntu 2

And after a long time (~20 minutes) I got the following error:
La conversion est en cours. Cette opération peut prendre quelques minutes...
Pour plus d’informations sur les différences de clés avec WSL 2, visitez https://aka.ms/wsl2
Échec de l’importation de la distribution.
./var/lib/gems/2.5.0/doc/bundler-2.1.4/rdoc/js/jquery.js: Hard-link target './var/lib/gems/2.5.0/doc/rubygems-update-3.1.3/rdoc/js/jquery.js' does not exist.
./var/lib/gems/2.5.0/doc/bundler-2.1.4/rdoc/fonts/Lato-Light.ttf: Hard-link target './var/lib/gems/2.5.0/doc/rubygems-update-3.1.3/rdoc/fonts/Lato-Light.ttf' does not exist.
./var/lib/gems/2.5.0/doc/bundler-2.1.4/rdoc/fonts/Lato-LightItalic.ttf: Hard-link target './var/lib/gems/2.5.0/doc/rubygems-update-3.1.3/rdoc/fonts/Lato-LightItalic.ttf' does not exist.
./var/lib/gems/2.5.0/doc/bundler-2.1.4/rdoc/fonts/Lato-Regular.ttf: Hard-link target './var/lib/gems/2.5.0/doc/rubygems-update-3.1.3/rdoc/fonts/Lato-Regular.ttf' does not exist.
./var/lib/gems/2.5.0/doc/bundler-2.1.4/rdoc/fonts/Lato-RegularItalic.ttf: Hard-link target './var/lib/gems/2.5.0/doc/rubygems-update-3.1.3/rdoc/fonts/Lato-RegularItalic.ttf' does not exist.
./usr/lib/ruby/2.5.0/rdoc/generator/template/darkfish/js/jquery.js: Hard-link target './var/lib/gems/2.5.0/doc/rubygems-update-3.1.3/rdoc/js/jquery.js' does not exist.
./usr/lib/ruby/2.5.0/rdoc/generator/template/darkfish/fonts/Lato-Light.ttf: Hard-link target './var/lib/gems/2.5.0/doc/rubygems-update-3.1.3/rdoc/fonts/Lato-Light.ttf' does not exist.
./usr/lib/ruby/2.5.0/rdoc/generator/template/darkfish/fonts/Lato-LightItalic.ttf: Hard-link target './var/lib/gems/2.5.0/doc/rubygems-update-3.1.3/rdoc/fonts/Lato-LightItalic.ttf' does not exist.
./usr/lib/ruby/2.5.0/rdoc/generator/template/darkfish/fonts/Lato-Regular.ttf: Hard-link target './var/lib/gems/2.5.0/doc/rubygems-update-3.1.3/rdoc/fonts/Lato-Regular.ttf' does not exist.
./usr/lib/ruby/2.5.0/rdoc/generator/template/darkfish/fonts/Lato-RegularItalic.ttf: Hard-link target './var/lib/gems/2.5.0/doc/rubygems-update-3.1.3/rdoc/fonts/Lato-RegularItalic.ttf' does not exist.
bsdtar: Error exit delayed from previous errors.

During the conversion my machine slowed down a lot: it seems the process was using most of my 8Gb:

I am running Windows 10 Pro version 2004 (19041.264) on a Surface Book.

I think you have to use Powershell. for that command to work here

I think you have to use Powershell.

No cmd.exe is fine. The linked documentation is just expository.
This is /dupe #4909 LZ #4457


GraphQL stack resource naming causing drop table warning

How did you install the Amplify CLI?
homebrew
If applicable, what version of Node.js are you using?
v20.3.0
Amplify CLI Version
12.2.3
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No
Describe the bug
We are in the process of updating the the new v2 transformer and updated GraphQL schema.  We have successfully deployed to our dev backend with no apparent issues, however in the process of deploying to the test backend we have hit a snag.  When attempting to deploy or even run gql-compile we get the following warning:
Removing a model from the GraphQL schema will also remove the underlying DynamoDB table
This is shown for all current tables we have in our app.  Some minor debugging tells me that the stack resource names (seen under backend/api/app_name/build/stacks/tablename.json have new numeric postfixes appended to them.  This is triggering the warning, however I am uncertain whether or not the warning is a false detection or if I should be worried about losing user data on deploy.
Expected behavior
Updating GraphQL schema and rebuilding stacks should not incur drop tables warning.
Reproduction steps

Update GraphQL schema from old @key syntax and adding new @auth directives.
Utilize amplify migrate api to facilitate graphql cli transformer and parsing.
Switch backend environments and deploy

Project Identifier
No response
Log output
Table names removed below

% amplify api gql-compile
⚠ WARNING: owners may reassign ownership for the following model(s) and role(s): .... If this is not intentional, you may want to apply field-level authorization rules to these fields. To read more: https://docs.amplify.aws/cli/graphql/authorization-rules/#per-user--owner-based-data-access.
🛑 Removing a model from the GraphQL schema will also remove the underlying DynamoDB table.
This update will remove table(s) [...]
ALL EXISTING DATA IN THESE TABLES WILL BE LOST!
If this is intended, rerun the command with '--allow-destructive-graphql-schema-updates'.

Additional information
For extra information, while deploying to the test backend I had several issues and ended up removing the whole api/build directory at some point, as well as bounced between the new change branch and our current development branch to compare things. I'm not sure if something in the build directory, amplify-meta.json or another file not in git lost some setting or what happened. There are no tables that were removed nor renamed in updating our schema, it is simply the data source mapping names that have changed.
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Just updated to 12.2.4 and the issue is gone.


Update for castore version 1.0.0

I believe there needs to be a small update to the mix.exs to either state that the :castore version can be >= 0.0.0 or to want at least version 1.0.

Done and released v1.2.1 of Redix!

Words cannot express my appreciation. Thank you so much.


Kubelet does not register host

Rancher Version:
v1.1.0-dev5
Docker Version:
1.11.1
OS and where are the hosts located? (cloud, bare metal, etc):
Ubuntu 16.04. Scaleway - Cloud
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single Node
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Kubernetes
Steps to Reproduce:

Create the Kubernetes Environment
Add two or three hosts
View the Kubelet log on each node.

Results:
Only the node in which the kubernetes master services are running shows the following line:
kubelet.go:1150] Node  was previously registered
Alternatively it says "successfully registered"
No other nodes show that they were registered in their respective kubelet logs.
kubectl get nodes  only shows the master node
Expected:
Node  was successfully registered should appear on every log of each kubelet.
kubectl get nodes should display all nodes in READY state
Thanks!

Are all your host names unique?  For kubernetes to work properly they must be unique.

Yes, the cloud provider generates a random set of 6 alphanumeric chars as a
name for the host.
I used to prepend a common name but after many tries I simply used that
random name when adding hosts.
On Thursday, June 30, 2016, Tani Fiduccia<EMAIL_ADDRESS>wrote:

Are all your host names unique? For kubernetes to work properly they must
be unique.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rancher/rancher/issues/5294#issuecomment-229825253,
or mute the thread
https://github.com/notifications/unsubscribe/ABduYl-aTQ95E16mbzuFrI9oqGFcWpRrks5qRF-ngaJpZM4JCch3
.

--
Ricardo Román | INT
|<EMAIL_ADDRESS>|

So instead of using Scaleway's docker image, I used a regular Ubuntu 15.10 image and installed docker on it.
Kubelet and Proxy start on the first try and the node gets registered.
Also tried with CentOS and got same successful results.
Could the image be the problem here?
They provide Ubuntu 16.04 + Docker 1.11.2 + aufs
And if so, how can I test it?

Currently Kubernetes doesn't support docker 1.11.  You can try a Scaleway image with a 1.10 version of docker and it should work.  We have a bug already entered that you can track, #4428.  I'm going to close this issue, if after using 1.10 version and if you still aren't registering, please reopen.

Thank you @tfiduccia
I went ahead and tried it. Here is my current configuration:
Two Ubuntu 16.04 Hosts with Docker 1.10.3 installed. This is what I am seeing:

Then the logs from kubelet:
Kubelet on master node -

Kubelet on other node - (This one does not output that it registered host)

And finally, output from kubectl get nodes -

Deploying a couple of items from catalog results in them being scheduled only on the master node, not the other node.

Thanks again!


What does s do? Can we get short docs/comments on the attributes?

I was playing with the custom-grid sample and found that two of the grid tiles use and display hex values as follows:

1, -1, 0 (ie. q=1, r=-1, s=0)
1, 0, -1 (ie. q=1, r=0, s=-1)

It's not clear what s is doing here. The difference in position seems to be based entirely on the q and r values, and a look at the source code confirms that. Modifying the s value does not change position, size, opacity, or z-index, and I can't think of anything else it might affect.
Additionally, s seems to be used in certain methods such as HexUtils.lengths() or HexUtils.round(), but given that it's not used in the HexUtils.hexToPixel() method, I'm not sure why it would appear there (or in HexUtils.getID(), for example, where its usage suggests it's important to the identity of the hex).
The request:

Clarify the usage of the s value specifically.
Document the component APIs in the README, or perhaps in the component's proptypes.
Update the custom-grid sample to use more clear/meaningful values

This library is great, and I'll be ignoring the s value for now. Any guidance could help us get more use out of it.

q and r are Axial, s appears to be extra.  From https://www.redblobgames.com/grids/hexagons/ :

In my projects, I name the axes q, r, s so that I have the constraint q + r + s = 0, and then I can calculate s = -q - r when I need the third coordinate for algorithms that work better with cube coordinates.

You can see this confirmed in pixelToHex: const hex = new Hex(q, r, -q - r);
One of the examples contradicts this with <Hexagon q={-2} r={0} s={1} />, but likely that's a typo.  I'm not sure why it was written as an explicit param instead of implicitly calculated when needed.

this grid uses a coordinate system with three variables q, r and s. There is now a picture in the readme that should make it clearer where these coordinates are.


Fix include error in android cmake example

Describe the pull request

What does your PR fix?

The android cross compile example has an include error when running compile.sh
my_lib.cpp:1:10: fatal error: 'json/json.h' file not found
[build] #include <json/json.h>
[build]          ^~~~~~~~~~~~~

Which triplets are supported/not supported? Have you updated the CI baseline?

All android

This LGTM, thanks for the fix!


Added Profile Page 2

Ref - #62

This part is overflowed
please fix this @chandansgowda

Hey @Roshaen
May I know in which device dimensions it's overflowing??

Hey @Roshaen
I have resolved the error now. Please check.

Hey @Roshaen
Thanks for reviewing and merging the PR.
My Profile URL - https://www.clueless.tech/profile/41890434


Add History page.

Description
LPVS Front-End History Page.
History Page provides a history of generated PRs.

My Pull Request - History of PR I created
My Repo PR - History of PR created in My Repository
My Org PR - History of PR created in my organization

Result, Dashboard pages will be uploaded.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Code cleanup/refactoring
[ ] Documentation update
[x] This change requires a documentation update
[ ] CI system update
[ ] Test Coverage update

How Has This Been Tested?

Update environment variables of docker-compose.yml file.
Build the lpvs container : docker compose build
Run containers: docker compose up -d
See the lpvs home page with any browser: http://localhost:7896

For using social login with Naver, Kakao, and Google, fill the necessary information:

Update the application.properties located in src/main/resources:
Enter client-id and client-secret. (Kakao doesn't require a client-secret.)
Update the .env file located in frontend:
Enter your REST_API_KEY.

The values required to implement the login function can be obtained from the link below.
Naver: https://developers.naver.com/products/login/api/api.md
Kakao: https://developers.kakao.com/docs/latest/ko/kakaologin/common
Google: https://cloud.google.com/identity-platform/docs/web/google

Test Configuration:

Java: v11
LPVS Release: v1.1.0

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

An error occurs in the Spring test code as sender, owner are added to the Pull Request table.
So in Dockerfile # Build LPVS-open-source application part by writing it as follows.
RUN mvn clean install -Dmaven.test.skip=true

[License Pre-Validation Service]  No license issue detected
Detected licenses:
File: src/test/java/com/lpvs/util/WebHookUtilTest.java
License(s): MIT (permitted)
Component: LPVS (test/java/com/lpvs/util/WebHookUtilTest.java)
Matched Lines: 25-120
Snippet Match: 79%

[License Pre-Validation Service]  No license issue detected
Detected licenses:
File: src/test/java/com/lpvs/util/WebHookUtilTest.java
License(s): MIT (permitted)
Component: LPVS (test/java/com/lpvs/util/WebHookUtilTest.java)
Matched Lines: 25-120
Snippet Match: 79%

@Basaeng  Please check build errors and fix them.

@Basaeng Please check build errors and fix them.

@o-kopysov
With the addition of pull_request_base and pull_request_head to the pull_request table, errors occurred in the test code.
Now resolved these errors by modifying the test code.


Rewrite uses search for performance, simplicity & correctness

I did some profiling for the A1 test and found that the currently slowest check by far is the UnusedCodeElementCheck (and the UseDifferentVisiblityCheck, which also searches for code element uses). On my machine (i7-12700K) it looks like this:

With a profiler attached, the entire analysis takes about 17.5s, of which the UnusedCodeElementCheck takes ~7s, and the UseDifferentVisiblityCheck takes another 2.5s. For reference, the next slowest checks are the ConstantNamingAndQualifierCheck (0.84s), ReassignedParameterCheck (0.78s), and the UnusedImportCheck (0.58s). Model building takes only about 1s, which is faster than the compilation step! (probably due to laziness within Spoon)
Anyway, we should improve the performance of the usage search. The current implementation sweeps across the entire model for each code element that we want usage information for, which leads to a runtime of something like O(n²). The goal of this PR is to rewrite usage search with a single pass over the model and a caching strategy for different checks.

Also, there is a failure case within the current usage search around instanceof patterns. This is valid Java code where i is used:
Object o = "Hello";
if (!(o instanceof String i)) {
    throw new IllegalArgumentException();
}
System.out.println(i);

The pattern variable i is introduced to the outer scope since the then-block cannot complete normally, so we statically know that o is a String when printing. (see JLS §6.3.2 https://docs.oracle.com/javase/specs/jls/se22/html/jls-6.html#jls-6.3.2)

@Luro02 I've just seen that you are doing something similar in #503. Am I right in that this rewrite is only for readability/code structure, and not performance? If so, I think we should prefer my implementation since it is significantly faster than the old implementation in first tests.

Mine is just a new api, so we don't have like 20 different methods in SpoonUtil.

The implementation now works in all cases (i.e. passes all existing tests and some more, and generates the same problems on the A1 task as the main branch, aside from a bug on main). It's also a lot faster compared to main and the scope-limiting implemented in #503:
Caching Uses:

Executed in    4.30 secs    fish           external
usr time   28.60 secs  411.00 micros   28.60 secs
sys time    1.70 secs  216.00 micros    1.70 secs

Scope-Limiting:

Executed in   12.01 secs    fish           external
usr time   36.66 secs  643.00 micros   36.66 secs
sys time    1.75 secs  164.00 micros    1.75 secs

Baseline on main:

Executed in   14.50 secs    fish           external
usr time   40.10 secs  692.00 micros   40.10 secs
sys time    1.62 secs    0.00 micros    1.62 secs

Looking at the profiler, the checks take now less than 25% (1.2s) of the total runtime (5.3s), with the remaining large chunks being Spoon's model building (0.9s), PMD (1s), javac (0.9s), and Lingua (0.3s), which is disabled in practice anyway:

The API is in no way finished, and I'm very unhappy with the current split of the model-related functionality into SpoonUtil, CodeModel, and now Uses/MethodHierarchy. It's especially inconvenient that the CodeModel has to be passed around everywhere where Uses may be needed. I've implemented a shortcut by caching the CodeModel inside the root CtPackage's metadata, but that feels hacky and may break sometime down the line.


Bring back the about tab in settings

The question is... where should we put it, it looks a little weird to have 4 options here!

maybe in the settings tab? as a button?

yeah, or like next to the "What's New" button in settings (the megaphone one).

maybe in the settings tab? as a button?

Maybe look at the image from Burkart above your message? It looks weird.

maybe in the settings tab? as a button?

We could possibly make an additional info page that has all the version and about stuff, and have that as a button in the settings tab.

I think I might close this as not planned as it is stale. You can reopen it but I don't think any activity is going to happen.

I'll just probably put it on the todo list beecause I was thinking it would be nice to have it, I'll re-introduce it with the new svelte settings menu.

| add enhancement tag

Something like d68ba15 ?

can you send ss as I cannot compile it right now.

wait i got it

although it should go next to the megaphone in the top right corner

although it should go next to the megaphone in the top right corner

Try f985f94 ?

yep. that works except it's not really about but what new instead.


CORS and/or other error causing loading issues

Using latest version of Edge on windows 11

Yeah, the records over time section is broken. Thinking of replacing it with a height-time graph per day.
I'm currently recording live heights data every 20sec and dumping it to a GCS bucket. Shouldn't be too hard to render it

Fixed with /player?q=<name> pages. eg: https://deepdip2.live/player?q=BrenTM


Update SharePoint hybrid support to reflect what's the real story in regards SP 2019

Hybrid support pages should be updated to reflect any concerns in regards of hybrid features with SP 2019. It seems all the pages reflect the support provided by SP 2013 and SP 2016, but nothing about SP 2019 is being said.

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

ID: 04f79181-8511-5f1f-e227-b3edd79abaf7
Version Independent ID: 4733698e-bf05-fd61-ff87-f86fc2ad3e72
Content: Plan hybrid SharePoint taxonomy and hybrid content types
Content Source: SharePoint/SharePointServer/hybrid/plan-hybrid-sharepoint-taxonomy-and-hybrid-content-types.md
Product: sharepoint-server-itpro
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

What needs to be changed? The features are identical with 2016.

@jcgonzalezmartin We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Specify resources for the proxy_init container

The resources can we set with helm through
--set global.proxy_init.resources

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ostromart
If they are not already assigned, you can assign the PR to them by writing /assign @ostromart in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @wmuizelaar. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/assign @ostromart

We're trying to limit the helm options as it's already getting out of hand. This strikes me as a rather specific/narrow rather than general option since it's just one particular container.
What's the use case and who's asking for it?

Sure, that seems reasonable. @costinm thinks there may be something already doing this in the pipeline but I couldn't find it so please go ahead.

@ostromart ok, pushed a new commit!

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ostromart
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [ostromart]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@wmuizelaar: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
c5a6428c15e081663ce13fa882af2aedbd399864
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ostromart
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ijsnellf
If they are not already assigned, you can assign the PR to them by writing /assign @ijsnellf in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [ostromart]
pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/retest

@wmuizelaar: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test message.

In response to this:

/retest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/assign @ijsnellf

/ok-to-test

@wmuizelaar does make test pass locally for you?  The the helm inject unit tests are failing in prow and circleci

@GregHanson I'm sorry - no. I'm trying to get all the local things setup now to get them fixed, will update the PR when done!

/retest

@wmuizelaar: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
471ac3ebbef46bcd6fd4978450f47422a99e71af
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@GregHanson I saw the e2e-bookinfoTests-v1alpha3.sh fail on other PR's as well - couldn't find a link in the output to my specific changes. Can you confirm? Thanks!

anyone? @GregHanson @ostromart

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ijsnellf, ostromart
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [ijsnellf,ostromart]
pilot/OWNERS [ijsnellf]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@wmuizelaar: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
fde6a2b0c470c2918a2db0d4115038a2bfa5c3b9
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

New changes are detected. LGTM label has been removed.

@wmuizelaar: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
03676941cb66c01c20d7cf437d0a7b8efa333601
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-unit-tests

@wmuizelaar: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
03676941cb66c01c20d7cf437d0a7b8efa333601
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wmuizelaar the prow unit test job had a bug, should be fixed now.  Please resolve the conflicts in your branch and then it should be good to go after the tests rerun

@wmuizelaar: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
080d1ad3e9788fcf425a92d124aace5ba0e2b19e
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wmuizelaar: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
080d1ad3e9788fcf425a92d124aace5ba0e2b19e
link
/test istio-unit-tests

prow/istio-presubmit.sh
080d1ad3e9788fcf425a92d124aace5ba0e2b19e
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ijsnellf, ostromart
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [ijsnellf,ostromart]
pilot/OWNERS [ijsnellf]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@wmuizelaar: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
080d1ad3e9788fcf425a92d124aace5ba0e2b19e
link
/test istio-presubmit

prow/istio-unit-tests.sh
2a49de6ee9438170b028ada3753eaa7b6525da56
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wmuizelaar: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
2a49de6ee9438170b028ada3753eaa7b6525da56
link
/test istio-unit-tests

prow/istio-presubmit.sh
2a49de6ee9438170b028ada3753eaa7b6525da56
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@GregHanson I think I hit another test-failure on master when I rebased...  :/

@wmuizelaar: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
f8e93203a12239f66023886443969f3dbc02cfb1
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
f8e93203a12239f66023886443969f3dbc02cfb1
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@GregHanson and did a rebase again, it now looks better, at least all the required tests are succesful now!

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: andraxylia, ijsnellf, ostromart
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [andraxylia,ijsnellf,ostromart]
pilot/OWNERS [andraxylia,ijsnellf]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/retest


Feature Suggestion : Video.js

The actual player is really basic, maybe using video.js will externalize the part of player and give you more time for the core
Cheers !

Thanks for the suggestion. We'll take a look.

@feross Feel free to ping me if you're ever interested in trying it out. It'd be super cool to see VJS used :)

@mmcc Will do! I think what we have right now is working pretty well. Do you know what new features or improvements we'd gain by switching?

@feross It's honestly probably one of those "not broke don't fix" kinda deals for you, but theoretically you would get better support for things like WebVTT captions. Beyond that, things like custom styling could be nice for you, along with easily being able to add plugins long term.
It's a little hard for me to say what the biggest benefit would be since the Electron App use case is a little out of our usual wheel house. Selfishly, that's one reason I'm interested in seeing you try it out :)

We already have support for WebVTT and our styling is already pretty nice, I think. 👍 But if we were doing it from scratch, I'd give it a try. Maybe a future project?

Didn't mean to imply the current styling isn't already nice (it is!), meant more from the end user's perspective.
Sounds good!

@mmcc Yep! I didn't take what you said to be negative :)


Guides - Add Image or Analyze Link to Code Splitting

Code Splitting page example is already very good, but do we need a more intuitive view to help us understand?
We use webpack-bundle-analyzer to help us more intuitively
Use it we can get such a view：

Hmmm I think we would be open to PR that mentions the webpack-bundle-analyzer or one of the other similar tools... are you suggesting adding an image to the guide?

@skipjack Yes, this will be better to see the  Code Splitting

Hmmmm @TheDutchCoder what do you think?
Personally I actually like some other bundle analyzers more and I think maybe keeping it simple within the guide (i.e. not adding any extra images) would be better. However what I do think would be good is linking to some bundle analyzers so people can try them out themselves.

@skipjack I agree. I understand that visuals help, but I feel like the guides will get too cluttered.
It also would be more logical to have images like these in the documentation. The guides are really a "how to" sort of thing.
I also agree that we could link to bundle analyzers, they're very useful.

I think the image helps to understand, so that we clearly see each file contains dependencies, and I also agree that we could link to bundle analyzers, they're very useful.

Yea once mermaid, or something similar, is in place (#469) a small graph to illustrate this should be an easy add.

I'm going to submit a PR with links to analyzers shortly.


Proposal: emit files without .ts extension to get custom file extension

Search Terms

file extension
.mjs
.esm

Suggestion

It is very useful if you can customize the output extension.
But mapping extension is boring and painful.
Maybe there is another way to make this easier.
If we don't map the extension, we just erase the .ts or .tsx extension to achieve it.
A compilerOptions is needed to prevent break existing TypeScript code.
Use Cases

Output the .mjs file for Node Module,
Output .es or other file extensions without additional steps. (e.g. useful for tsc watching mode)

Examples

input:
// src/lib.js.ts
export const result = 42;

// src/index.js.ts
import { result } from "./lib.js";

console.log(result);

output:
// build/lib.js
export const result = 42;

// build/index.js
import { result } from "./lib.js";

console.log(result);

input:
// src/lib.mjs.ts
export const result = 42;

// src/index.mjs.ts
import { result } from "./lib.mjs";

console.log(result);

output:
// build/lib.mjs
export const result = 42;

// build/index.mjs
import { result } from "./lib.mjs";

console.log(result);

input:
// src/lib.myext.ts
export const result = 42;

// src/index.myext.ts
import { result } from "./lib.myext";

console.log(result);

output:
// build/lib.myext
export const result = 42;

// build/index.myext
import { result } from "./lib.myext";

console.log(result);

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

Related issue:

https://github.com/Microsoft/TypeScript/issues/27957
https://github.com/Microsoft/TypeScript/issues/18442

just add a compiler option --outSuffix so tsc --module es6 --outSuffix mjs...
https://github.com/microsoft/TypeScript/issues/32373

Problem is wider, I need in --outSuffix /index.js =)


Speed of ball

The speed needs to be fun and incrementing. Right now it's quite static. We need to increase the speed more than right now

Fixed


Lower NuGet requirement to 2.8.1

Hey there! There was a bug in my templates (I have since updated them), but 2.8.1 is what is in Xamarin Studio, so you if you can modify and re-upload the nuspec that would be great.
Thanks!
James

It was 2.8.1 in template but I changed it because of hangouts video ;)
I don't have my mac build host with me but I think VS do not need that.
I uploaded <IP_ADDRESS> version to nuget.
Best,
Ufuk ARSLAN

Ahhh perfect, yeah that was my fault. I found out about 2.8.1 later. I will
leave a comment on the video :) Yeah you don't need a build host to
compile, only to test and deploy.
Thanks!
On Mon, Jan 5, 2015 at 5:07 PM, Ufuk ARSLAN<EMAIL_ADDRESS>wrote:

It was 2.8.1 in template but I changed it because of hangouts video ;)
I don't have my mac build host with me but I think VS do not need that.
I uploaded <IP_ADDRESS> version to nuget.
Best,
Ufuk ARSLAN
—
Reply to this email directly or view it on GitHub
https://github.com/ufuf/AdvancedTimer/issues/1#issuecomment-68790829.

--
James Montemagno
http://www.montemagno.com
@jamesmontemagno http://www.twitter.com/jamesmontemagno


AttributeError: 'module' object has no attribute 'version'

Hello, edd is an useful tool, and i really need it. Thank you for your contribution and sharing.
When i run edd, the following error was made. How can i solve this problem? As a beginner,  i hope to get a detailed solution.
Looking forward to  your reply. Thank you.
edd --write-log-ratios --write-bin-scores
chrNameLength.txt hg19.blacklist.merge.bed
H3K9me2_ab.rmdup.bam input.rmdup.bam H3K9me2_ab
Traceback (most recent call last):
File "/data/User/limingsen/.conda/envs/EDD/bin/edd", line 133, in
import eddlib.experiment
File "/data/User/limingsen/.conda/envs/EDD/lib/python2.7/site-packages/eddlib/experiment.py", line 18, in
import logit
File "/data/User/limingsen/.conda/envs/EDD/lib/python2.7/site-packages/eddlib/logit.py", line 3, in
from statsmodels.stats.proportion import proportion_confint
File "/data/User/limingsen/.conda/envs/EDD/lib/python2.7/site-packages/statsmodels/stats/init.py", line 1, in
from statsmodels.tools._testing import PytestTester
File "/data/User/limingsen/.conda/envs/EDD/lib/python2.7/site-packages/statsmodels/tools/init.py", line 1, in
from .tools import add_constant, categorical
File "/data/User/limingsen/.conda/envs/EDD/lib/python2.7/site-packages/statsmodels/tools/tools.py", line 8, in
from statsmodels.compat.python import lzip, lmap
File "/data/User/limingsen/.conda/envs/EDD/lib/python2.7/site-packages/statsmodels/compat/init.py", line 1, in
from statsmodels.tools._testing import PytestTester
File "/data/User/limingsen/.conda/envs/EDD/lib/python2.7/site-packages/statsmodels/tools/_testing.py", line 11, in
from statsmodels.compat.pandas import assert_equal
File "/data/User/limingsen/.conda/envs/EDD/lib/python2.7/site-packages/statsmodels/compat/pandas.py", line 4, in
import numpy as np
File "/data/User/limingsen/.conda/envs/EDD/lib/python2.7/site-packages/statsmodels/compat/numpy.py", line 46, in
NP_LT_114 = LooseVersion(np.version) < LooseVersion('1.14')
AttributeError: 'module' object has no attribute 'version'

hello, do you solve the problem? I also have the same problem and do not know how to solve. If it is convenient, could you please help me? I will be very grateful.

I am so sorry. I also do not solve this problem.

------------------ Original ------------------
From: @.>;
Date:  Sat, Jan 8, 2022 11:15 AM
To: @.>;
Cc: @.>; @.>;
Subject:  Re: [CollasLab/edd] AttributeError: 'module' object has no attribute 'version' (#17)

hello, do you solve the problem? I also have the same problem and do not know how to solve. If it is convenient, could you please help me? I will be very grateful.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

Thank you very much!
Best wishes!
At 2022-01-22 17:42:05, "Mingsenli" @.***> wrote:
I am so sorry. I also do not solve this problem.

------------------ Original ------------------
From: @.>;
Date:  Sat, Jan 8, 2022 11:15 AM
To: @.>;
Cc: @.>; @.>;
Subject:  Re: [CollasLab/edd] AttributeError: 'module' object has no attribute 'version' (#17)

hello, do you solve the problem? I also have the same problem and do not know how to solve. If it is convenient, could you please help me? I will be very grateful.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.Message ID: @.***>

As far as my investigation went I found that it is actually not EDD that is causing the problem but rather statsmodels which apparently is incompatible with python 2.7 from it 0.11 version (also see here). Thus, I would say a quick solution might be getting an older version of statsmodels (like 0.10)

Thank you very much, I have found  the statsmodels==0.6.1 can work well.
At 2022-02-13 17:03:06, "Daniel Malzl" @.***> wrote:
As far as my investigation went I found that it is actually not EDD that is causing the problem but rather statsmodels which apparently is incompatible with python 2.7 from it 0.11 version (also see here). Thus, I would say a quick solution might be getting an older version of statsmodels (like 0.10)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.Message ID: @.***>

Thank you for helping to solve this problem.
Your help has brought me great convenience.

------------------ Original ------------------
From: @.>;
Date:  Mon, Feb 14, 2022 09:31 AM
To: @.>;
Cc: @.>; @.>;
Subject:  Re: [CollasLab/edd] AttributeError: 'module' object has no attribute 'version' (#17)

Thank you very much, I have found  the statsmodels==0.6.1 can work well.
At 2022-02-13 17:03:06, "Daniel Malzl" @.***> wrote:
As far as my investigation went I found that it is actually not EDD that is causing the problem but rather statsmodels which apparently is incompatible with python 2.7 from it 0.11 version (also see here). Thus, I would say a quick solution might be getting an older version of statsmodels (like 0.10)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.Message ID: @.>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.>

No thanks, I am very glad to help you.
Best regard,
Shuli
At 2022-03-07 11:31:40, "Mingsenli" @.***> wrote:
Thank you for helping to solve this problem.
Your help has brought me great convenience.

------------------ Original ------------------
From: @.>;
Date:  Mon, Feb 14, 2022 09:31 AM
To: @.>;
Cc: @.>; @.>;
Subject:  Re: [CollasLab/edd] AttributeError: 'module' object has no attribute 'version' (#17)

Thank you very much, I have found the statsmodels==0.6.1 can work well.
At 2022-02-13 17:03:06, "Daniel Malzl" @.***> wrote:
As far as my investigation went I found that it is actually not EDD that is causing the problem but rather statsmodels which apparently is incompatible with python 2.7 from it 0.11 version (also see here). Thus, I would say a quick solution might be getting an older version of statsmodels (like 0.10)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.Message ID: @.>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.Message ID: @.***>


NSwag studio generates wrong HTTP Headers for IEnumerable collections

Hi Team.
I have Web API with action which have Http Header parameters with IEnumerable types, like:
public IActionResult Get([FromHeader]List<string> idList)
When I generate client, using NSwag studio (based on Swagger Json specification)
it creates API methods with folowing code line:
request_.Headers.TryAddWithoutValidation("idList", idList != null ? idList.ToString() : null);
Nswag supose that i passing string which is wrong.
Could someone suggest me how to fix it.
(Probably, I'm doing something wrong with Nswag config)
Thanks a lot in advance.

This is currently not supported...
What would be the expected output?

Thanks a lot for the quick answer.
Expected output should be:
request_.Headers.Add("idList", idList);

Is it also ok to use
request_.Headers.TryAddWithoutValidation("idList", idList);

?

Yeah, this one should be ok.
Thanks.

Sorry I have one more (probably stupid :) ) question:
Should I expect this fixed in next version(s) of Nswag studio ?
Thanks.

Yes, v11... released next week.
Until then you can use the latest CI builds:

Clear and Thank you so much !
 :-)

Yeah, almost :) (only my life and my colleagues :)) ) And not only life but time also !!!
We are building some complex, AR product (Unfortunately, I can't say now what exatly the name of product / company etc. (( )
Cheers !!

Sounds interesting :-). Then I wish you success with your project!

Thanks man ;)
p.s
Totaly forgot that we have web site :)
https://www.re-flekt.com/

Just FYI: v11 is released, v11.1 incoming ;-)

Thanks a lot for the update. ;-)


Support enabling alternator

since in ccm we listen of specific address, we need to make sure
alternator is listening on the same address

would need to backport to branch-4.0


Role activation should load balance a little better

I noticed that the master server who is responsible for activating roles on servers doesn't account for the already activated roles on the server.  This could lead to a case where the master server activates roles on an already bogged down appliance, while another appliance that is not doing much work doesn't receive the role activation.
cc @gtanzillo @jrafanie

@jrafanie Technically this is still open, so I'm assigning to you.  This might go away with the rearch.
@miq-bot rm-label stale
@miq-bot assign @jrafanie


Add unix socket support in php_fpm plugin

What does this PR do?
It adds unix sockets support to php_fpm plugin
Motivation
Company PHP fpm setup which does not use plain sockets.
Review checklist

[x] PR has a meaningful title or PR has the no-changelog label attached
[ ] Feature or bugfix has tests
[x] Git history is clean
[ ] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo

Additional Notes
This is our first contribution, do not hesitate to contact us !

Codecov Report

Merging #2636 into master will increase coverage by 3.11%.
The diff coverage is 41.66%.

@@            Coverage Diff             @@
##           master    #2636      +/-   ##
==========================================
+ Coverage    83.9%   87.01%   +3.11%
==========================================
  Files         644        5     -639
  Lines       36468      208   -36260
  Branches     4439       25    -4414
==========================================
- Hits        30597      181   -30416
+ Misses       4555       16    -4539
+ Partials     1316       11    -1305

Tests fixed !

This change will probably require mentioning the unix socket syntax in documentation, not sure how do that.


[cmd/maya-exporter] The volume utilization metrics aren't representing the used capacity correctly

Description
The maya-exporter container is deployed as a side-car to the target/controller pods and are responsible for exposing volume metrics.
Example of such is openebs_volume_status, openebs_actual_used, openebs_size_of_volume, openebs_volume_uptime
Expected Behavior
OpenEBS' storage engines viz Jiva and Cstor both have maya-exporter deployed as a sidecar container and it's metrics. Both the storage engines expose a common set of volume-metrics for clients which can be used by clients like Grafana to provide visualization.
A typical use case is figuring out remaining space, which is volume_used - volume_free.
As of now there are some discrepancies and it isn't easy to get what is expected.
There's an openebs_actual_used and openebs_size_of_volume, which can be used to figure out the same.
Current Behavior
At times the openebs_actual_used capacity has exceeded the openebs_size_of_volume, which isn't possible until the former is telling physical volume-size used.
Steps to Reproduce

It has been seen in the openebs-workloads cluster, which also takes regular volume-snapshots.

@vharsh can you please post the output, which you've got.


[6621] Added skip to results for the filter

Context
Skip to results
Changes proposed in this pull request
Added skip to results for the filter
Guidance to review
Use the keyboard tab key, as it is hidden from view.
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Add hidden 'skip to results' link at start of filters

@defong Can you add the missing data-module="govuk-skip-link" attribute but apart from that it's testing well thanks. :shipit:


feat(plugin): allow to filter hosts

@tmieulet thanks for your pull request) I'll take time tomorrow to review and then give you the feedback)


make required use the valid html5 format

Also added in a couple of other HTML5 elements; readonly, disabled and autofocus.  Afraid I couldn't find a spec to get them all :/

resolves #23

@skinofstars please squash all into 1 commit and I'll get it merged

Sure, I'll do this in the morning (UK time)
On 27 Jul 2015 00:22, "Trung Lê"<EMAIL_ADDRESS>wrote:

@skinofstars https://github.com/skinofstars please squash all into 1
commit and I'll get it merged
—
Reply to this email directly or view it on GitHub
https://github.com/lotus/helpers/pull/25#issuecomment-125047698.

For documentation purposes: https://github.com/kangax/html-minifier/issues/63#issuecomment-18634279 list of "boolean attributes"

Hello and thanks for your PR, but this tries to solve an invalid ticket.
For reference, please see:

https://github.com/lotus/helpers/issues/23#issuecomment-125116084
https://github.com/lotus/helpers/issues/23#issuecomment-125116742

My sincere apologises.

@jodosha ok, thanks anyway


WIP overhaul error stuff

Run perltidy on util/mkerr
Change some mkerr flags, write better documentation
Make generated tables "const" when genearting lib-internal ones.
Re-order creation/destruction of the hashtable, to simplify the code.

sorry, this is a WIP.  Will clean it up later when ready.


artifacts v3 deprecation

GitHub
Artifact actions v3 will be deprecated by December 5, 2024. You are receiving this email because you have GitHub Actions workflows using v3 of actions/upload-artifact or actions/download-artifact. After this date using v3 of these actions will result in a workflow failure. Artifacts within their retention period will remain accessible from the UI or REST API regardless of the version used to upload.
To raise awareness of the upcoming removal, we will temporarily fail jobs using v3 of actions/upload-artifact or actions/download-artifact. Builds that are scheduled to run during the brownout periods will fail. The brownouts are scheduled for the following dates and times:
November 14, 12pm - 1pm EST
November 21, 9am - 5pm EST
What you need to do
Update workflows to begin using v4 of the artifact actions as soon as possible. While v4 improves upload and download speeds by up to 98% and includes several new features, there are key differences from previous versions that may require updates to your workflows. Please see the documentation in the project repositories for guidance on how to migrate your workflows.

can we just change the version number?
https://github.com/EbookFoundation/free-programming-books/blob/fa55eb0e61535ae74dd9e1168701b4ba1a519073/.github/workflows/check-urls.yml#L107

I can test this out this week. We had some issues with v4 but hopefully they are resolved on the most recent version.

Opened a PR with the fix. This seems to work consistently in my testing. https://github.com/EbookFoundation/free-programming-books/pull/11754


Big files: support sliced read

The idea is to be able to deal with big files (that would not fit in memory).
A possible API could be:
sliceString : Int -> Maybe Int -> File -> Task x String
sliceString start end file =

Seems the Blob.slice function allows to implement this efficiently.

We're also running into this, for now we're going through a port and slice it on JS side.


Milestone 2 - Work Plan

[x] Writing distance function.
[ ] Testing distance function.
[x] Writing kmedians function.
[ ]  Testing kmedians function.
[x] Writing summary function.
[ ] Testing summary function.

Following GitHub Flow workflow:
remember to use branches
Code review and feedback for each function:

[x] Distance
[x] KMedians
[x]  Summary

Python Package

[x]  Make the package importable
[ ]  Update README with instructions on how to import package

I need to make some minor changes to the test cases written for the summary function. Since it randomly initializes the medians some of the test are failing.

[ ] upload copy of pytest report


Annoying error message whenever my laptop is offline.

An uncaught exception was reported.
Info:
OS: darwin
Arch: x64
GPMDP Version: 3.0.0
Time: Fri Mar 11 2016 03:46:38 GMT-0800 (PST)
Error:
The Internet connection appears to be offline.: (null)
Error: The Internet connection appears to be offline.: (null)
    at Error (native)

This issue was created automatically inside the uncaughtException handler

Please accept the update notification and this error will go away


Dragon orbs don't abide by regular entity moving rules

Quick look at the code makes me think its just exp orbs with the move to player effect ripped out, this makes them act in weird ways compared to other entities,

visually being in places they aren't
not being moveable by pistons by the same rules item entities are.

For consistency's sake we may want to look into them being more moveable like other mob drops.

fixed for next release


Faster clones, parser naming

Another day, another PR
I'm attempting to port the great improvements you made to the swift and json parser Makefiles back to the others. While doing this, I noticed that the parser name cannot always be determined from the git url. This makes a small change to add some forwards compatibility, so it will not disrupt builds of gomod when its new Makefile is ready.
Also, changed the clone depth to 1, because building the xcframework is already slow enough...

actually hang on...


Event Request codeunit 5704 "TransferOrder-Post Shipment" OnAfterDeleteTempHandlingSpecificationAfterTransferTransferToTransfer

Hello. Could you please add event OnAfterDeleteTempHandlingSpecificationAfterTransferTransferToTransfer to the TransferTracking procedure in the codeunit  5704 "TransferOrder-Post Shipment"?
local procedure TransferTracking(var FromTransLine: Record "Transfer Line"; var ToTransLine: Record "Transfer Line"; TransferQty: Decimal)
    var
        DummySpecification: Record "Tracking Specification";
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeTransferTracking(FromTransLine, ToTransLine, TransferQty, IsHandled);
        if IsHandled then
            exit;

        TempHandlingSpecification.Reset();
        TempHandlingSpecification.SetRange("Source Prod. Order Line", ToTransLine."Derived From Line No.");
        if TempHandlingSpecification.Find('-') then begin
            repeat
                ReserveTransLine.TransferTransferToTransfer(
                  FromTransLine, ToTransLine, -TempHandlingSpecification."Quantity (Base)", "Transfer Direction"::Inbound, TempHandlingSpecification);
                TransferQty += TempHandlingSpecification."Quantity (Base)";
            until TempHandlingSpecification.Next() = 0;
            TempHandlingSpecification.DeleteAll();
        end;

        //new event
        OnAfterDeleteTempHandlingSpecificationAfterTransferTransferToTransfer(FromTransLine, ToTransLine, TransferQty);

        if TransferQty > 0 then
            ReserveTransLine.TransferTransferToTransfer(
              FromTransLine, ToTransLine, TransferQty, "Transfer Direction"::Inbound, DummySpecification);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnAfterDeleteTempHandlingSpecificationAfterTransferTransferToTransfer(var FromTransLine: Record "Transfer Line"; var ToTransLine: Record "Transfer Line"; TransferQty: Decimal)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10806.

Availability update: We will publish a fix for this issue in the next update for release 22.
Build ID to track: 56637.


Nested Patch tooltip title not picked up correctly from md file

should be called "Nested Patch"

The tooltip title comes from the name of the plugin (in
https://github.com/vizorvr/vizor/blob/develop/browser/plugins/plugins.json)

careful when editing this file though as syntax errors will break
everything :)
ᐧ

On Thu, Feb 18, 2016 at 12:10 PM, Matt Innes<EMAIL_ADDRESS>wrote:

[image: screenshot 2016-02-18 23 07 49]
https://cloud.githubusercontent.com/assets/6858525/13142964/ceb1d2a0-d694-11e5-9d67-a7647c56039d.png
should be called "Nested Patch"
—
Reply to this email directly or view it on GitHub
https://github.com/vizorvr/vizor/issues/1315.


Event stream test

It required a little bit more refactoring for me to test this. I abstracted the payload handling code into its own function so I could test that directly without having to run all of handleMessage every time.
The only thing I'm feeling a little iffy about is in the fact that I'm passing around getEntities as a call back, instead of calling entify directly, but that was somewhat of a kludge so that I could pass in a stub and not have to set up an actual model and entify it in the tests, which seemed unnecessarily complicated.

Creating a mock model for the sake of the test wouldn't be overkill IMO, still fine to do it how you did though. Thanks a lot for this, it's much needed!


Do not create shaded artifact which prevents dependency override.

Currently your project shades the whole depenencies in particular org.ow2.* which means a consuming project can not override the used version by newer ones in cases where errors etc. occur.

That's the exact idea of shading the dependencies. The library becomes a black box without transitive dependencies and the fixes have to be made by releasing the library. Overriding anything but minor releases of dependencies is a bad idea anyway. Which is why I am not really convinced it's worth maintaining another artifact. Not even sure how this would work with maven central.
Can you make a good case for it other than "I want to upgrade asm1 for java compatibility"?
Also see https://github.com/tcurdt/jdependency/issues/11

Unfortunately it's not a black box cause you inherit the classes including the ones given inside the jar and furthermore the problem is if you have a issue with an older version of the library like ASM which is shaded you can't overwrite it in a simple manner (which is advantage of transitive dependencies). I'm aware of the problems based on minor versions etc. which would be my own responsibility ...but at the moment I'm trying to upgrade maven-shade-plugin (trying to make a new release) to newer versions of ASM based on JDK9 / JDK10 issues which is at the moment not possible...cause I can't overwrite the versions of ASM cause it's shaded inside of jdependency...

The classes should be shaded that means they have completely different class names.
https://github.com/tcurdt/jdependency/blob/master/pom.xml#L143
Only inheritance would expose the rewritten package for ASM. The jdependency classes do not inherit from ASM only the internal ASM helper classes do that no one should use.
https://github.com/tcurdt/jdependency/tree/master/src/main/java/org/vafer/jdependency/asm
That said I think we should mark/document them as such.
So inheritance should not be a problem.
It should be a blackbox or otherwise there is a bug.
Which of course means if we need a newer version of ASM we also need to make a new jdependency release. Which to me sounds like the right thing to do.

Ah..haven't read the pom file carefully enough...It would be great If you could do a new release of jdependency...

Right now I am on vacation with little/bad internet activity. I try my best. Otherwise it might have to wait 2 weeks. Can you confirm that master is all you need for the new release, @khmarbaise?

@khmarbaise I am back in civilization. How about master and a release from it?

Just release 1.4. This closes #15 and includes #14


pascal panics in some cases during generating files

Hi!
Specifying empty string for the field cause panics.
Steps to reproduce

Create schema with empty enum field name:

// Fields of the Pet.
func (Pet) Fields() []ent.Field {
	return []ent.Field{
		field.Enum("").Values(
			"A",
			"B",
			"C",
		),
	}
}

entc generate ./ent/schema
see it panics

Reason
https://github.com/facebookincubator/ent/blob/b54007ef5686f1bcfaff0e8280b718252f49e079/entc/gen/func.go#L108
In any of this cases pascal will panic:

empty string
starting from _underscore
ending with underscore_

Test file:
func Test_pascal(t *testing.T) {
	type args struct {
		s string
	}
	tests := []struct {
		name string
		args args
		want string
	}{
		{
			name: "empty string",
			args: args{s: ""},
			want: "",
		}, {
			name: "starting from underscore",
			args: args{s: "_example"},
			want: "_Example",
		}, {
			name: "ending with underscore",
			args: args{s: "example_"},
			want: "Example_",
		},
	}
	for _, tt := range tests {
		t.Run(tt.name, func(t *testing.T) {
			defer func() {
				if err := recover(); err != nil {
					t.Fatalf("recovered: %s", err)
				}
			}()
			if got := pascal(tt.args.s); got != tt.want {
				t.Errorf("pascal() = %v, want %v", got, tt.want)
			}
		})
	}
}

Possible solutions how to fix:

return error when using empty field name (best solution for me, because for empty field name is difficult to generate schema, and etc.)
check w is not empty before passing to rules.Capitalize call
create PR to https://github.com/go-openapi/inflect or fork-and-fix

If the first solution is ok, i can send PR

I think the easiest way to handle this, is simply to reject fields without name.
#273 should fix this.

return error when using empty field name (best solution for me, because for empty field name is difficult to generate schema, and etc.)

Oh, I just saw this option.
I really sorry about this, I saw the bug and just fixed this automatically (while working).
I hope you'll have more opportunities to contribute code at the future.
Thanks and sorry (again!)


docs: correct the transform-tracking-logs do command name

The transform-tracking-logs do command in the README was incorrectly spelled as transform_tracking_logs. I've updated this in the README.

@Danyal-Faheem thanks for the correction


Show data fetched from server

In my project I need to show some predefined tags fetched from the server using network call, the user input will be taken along with the fetched tags and new array of tags will be sent to the server using PUT request. How can I get the data fetched from the server and show in the tags?
Here is my code so far:
constructor(props){
    super(props);
    this.handleSubmit = this.handleSubmit.bind(this);
    this.state = {
      tags: [ ],
    };
      this.handleDelete = this.handleDelete.bind(this);
      this.handleAddition = this.handleAddition.bind(this);
  }
handleDelete(i){
        const tags = this.state.tags;
        tags.splice(i, 1);
        this.setState({tags});
    }

    handleAddition(tag){
        const tags = this.state.tags;
        tags.push({
            id: tags.id,
            text: tag
        });
        this.setState({tags});
    }
fieldData(){
      const fields = Object.assign(this.props.synonym.table_fields).map(h => ({
          id: h.id,
          synonyms: this.state.tags
      }));
      return{
          fields,
          table_id: this.props.synonym.selectedTable.id,
      }
  }

  handleSubmit(e){
    e.preventDefault();
    this.props
      .tableData(this.fieldData())
      .then((res) => {
        swal({
          title: 'Synonym data is sent to the server',
          icon: 'success'
        });
      })
      .catch((error) => {
        // eslint-disable-next-line no-console
        console.error('Synonym Updating failed: ', error.response.data);
        swal({
          title: 'Error sending synonym data',
          icon: 'error'
        });
      });
  }

  tableRender(){
    const placeholder = 'Write a synonym and hit enter';
    const { tags } = this.state;
    if (!this.props.synonym.selectedTable) return;
    const Fields = this.props.synonym.table_fields;
    return Object.assign(Fields).map(t =>

          <input type="number" className="border-none" id="field_id" value={t.id} readOnly />
          {t.name}
           {t.alias}

                   <ReactTags
                              placeholder={placeholder}
                              tags={tags}
                              handleDelete={this.handleDelete}
                              handleAddition={this.handleAddition}
                   />

    );
  }

@prakhar1989 need help badly.

The easiest way imo is to fetch tags from backend at the very beginning of mounting component i.e. and just put them in tags state.
Also could you please format your code for better readability? :)

Thanks for the response, could you please elaborately tell me where to change things in my code block? @mrdannael

Axios package should be enough for that. Let's assume that you have endpoint for fetching tags from backend i.e. /api/tags. So we want to make api call in componentDidMount() and update our state.
componentDidMount() {
   axios.get('/api/tags')
      .then(resp => this.setState({ tags: resp.data.tags }))
      .catch(err => { console.log(err) }) // handle error properly
}

Response from backend should look like this:
{
   tags: [{
      id: 1,
      text: 'Tag 1'
   }, {
      id: 2,
      text: 'Tag 2'
   }]
}

Hope it helps :)

Thanks, I am trying it now.

Hi @mrdannael is there any way I can save these tags data in redux store instead of fetching them from setting state as in the same component I am showing data using redux store like tableid, tablename, ailas and then there is tags for individual rows of the table, now I am facing problem getting the tags using state as my other table components of the same table are loaded from redux store.

I see two options for it now @ashfaqrafi:

Create simple action that will put your tags into specific reducer and then just call this.props.dispatch with that action and proper arguments i.e. response.data.tags. (or refer to this)
Create action/function with redux-thunk. Inside it just make an api call and inside then() call dispatch where you will put payload into specific reducer (similar to 1 option).

I'm closing this issue because it's not related strictly to react-tags itself.
Please see the rest of docs for React, Redux and/or redux-thunk.
Also look for answers on StackOverflow too :)
Cheers!

Thanks @mrdannael :)


bob is before leap in exercise order

Bob is a fair bit harder IMO. Other tracks (such as xhaskell, xocaml) have bob after leap.

All of the exercises really need to be reordered at some point. I took a swag at estimating the difficulty and topics for all the exercises here a month ago, but I didn't get around to changing the order.
I don't just want to reorder them based solely on my swag difficulty ratings, both because I don't fully trust them and because I haven't yet taken the time to understand if there's a good ordering for the exercises that build on top of each other.
I am fully open to accepting PR's that make obvious fixes like the one above 👍

I added PR #127 for this.

Merged the linked PR so I'm closing this. Thanks!


Spaces in between section headers

Currently getting space between the section headers when the rows are closed. No spaces appear between the rows under the section. But can't seem to get rid of the spaces between the section headers.

@marknf94 how to reproduce this please? any sample project or code?

Closing this since no answer from @marknf94


Ordered path maps

Fixes #199 by changing the path_maps option from a dictionary, to an ordered list of dictionaries. This means paths will be matched in the order they are defined, allowing you to map specific paths before they are altered by more general ones:
let g:vdebug_options['path_maps'] = [
    \  {"/var/www/libs/some-lib": "/home/jon/libs/some-lib/dist"},
    \  {"/var/www": "/home/jon/project/www"}
    \ ]

This would introduce a breaking change, so it might be a good idea to include a deprication notice.

Hi @adambiggs, thanks for taking this upon yourself to sort out!
The solution makes complete sense, and I can see fixes the problem. However, my thoughts are towards backwards compatibility. A lot of people use path maps, and I'd quite like them to be able to use the existing syntax if they aren't currently struggling with the issue of mappings overriding each other.
My thought is to keep your solution for the most part, but also to check the type for the path mapping that's defined by the user. If it's a list, then fine. If it's a dictionary (i.e. the old syntax) then convert it to the newer list format, and the order will be random.
I'm happy to look into this when I can, otherwise feel free to go ahead on it.
Thanks a lot!

Hi @joonty, glad to see you're still checking in on this project!
After opening this PR, I noticed there was a similar fix in the version-2.x branch: 64b703a. What are your plans for that branch? I tried using that branch when I noticed it, but couldn't get it to work.
Your comment above makes perfect sense, and I'm happy to make the change, but seems like this whole PR might cause conflicts if the version-2.x branch is merged into master...

It looks like the fix in 2.x would be easy to backport into the current version, and would likely solve most people's problems - it sorts the dictionary entries by length, descending, which places the longer, more specific paths before the shorter ones.
This doesn't give the full flexibility of user-specified ordering, but has the benefit of "just works" without people having to worry about ordering.
I also don't' see any technical reason that prevents both fixes being combined (keep user sorting for lists, sort by length for hashes), but that would add to code complexity, and if the only use case for user-specified ordering is solved by sorting by length, it might make sense to just backport the change from 2.x.

Thanks @cincodenada, I agree with your take on the situation. I'll backport the version 2 fix, and notify everyone here when that's done.
Thanks

Erm. It seems like this has already been done. Or rather, the fix was added to the version-2.x and dev branches at the same time. So @adambiggs you can try the dev branch - that's less disruptive than version 2 and has the same fix.
Incidentally, @adambiggs could you open an issue for version 2 as to why it wouldn't work for you? As in, what the error was? That would be a massive help.
I'm going to close this PR on the basis of the fix that will go into the next release (1.5.0). Thanks for your contribution, nevertheless!

@joonty sounds good. I haven't had to do any PHP debugging for months now, but I'll be sure to use the dev branch next time I do and will report any issues I run into. Cheers.