Protect against undefined values

cc @marcaaron @chrispader
Details
Explained in this thread: https://expensify.slack.com/archives/C01GTK53T8Q/p1673550288930829
This caused a crash in the app when clearing Onyx due to one of the values passed to multiSet() being undefined (which is normal behavior for a value that we are trying to clear out).

Related Issues
None
Automated Tests
None
Linked PRs
This will be tested and deployed with https://github.com/Expensify/App/pull/13886

OK, updated. Here is the output of stringifiedPairs:
[["session", "{\"loading\":false}"], ["account", "{\"errors\":null,\"success\":\"\",\"isLoading\":false}"], ["network", "{\"isOffline\":false}"], ["iou", "{\"loading\":false,\"error\":false,\"creatingIOUTransaction\":false,\"isRetrievingCurrency\":false}"], ["isSidebarLoaded", "false"], ["shouldShowComposeInput", "true"], ["preferredLocale", "\"es\""], ["activeClients", "null"]]


Have command/solution to find out Kubernetes release specific Rancher image versions

Problem
We are making releases for air-gapped environment and like to have release size as small as possible. As newer releases supports more and more different versions. It cause release size to grow significantly.
For Rancher Kubernetes Engine there command rke config --system-images --version  to get needed image versions for specific release which is quite nice
Solution
Rancher has guide to publish images to private registry but possibility to be more release specific would be nice.
https://ranchermanager.docs.rancher.com/getting-started/installation-and-upgrade/other-installation-methods/air-gapped-helm-cli-install/publish-images
Solution would be way to get image list for e.g. Rancher v2.6.11 to support only Kubernetes v1.22.17 similar as it has done in RKE or some other way.

This is still nice to have feature.

Can we have please?


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3763

Questions ?
Please refer to the Backport tool documentation


Fix: Removed chart for patients by age

Description of PR
As the sample size of patients by age was very small (~5%). The chart for Patients by age should be removed
Relevant Issues
Fixes #1734
Checklist

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

@jeremyphilemon @bee-rickey @shuklaayush Is something more required from my end here?

@jeremyphilemon @bee-rickey @shuklaayush Is something more required from my end here?

Seems like this PR is obsolete now since deep-dive was deprecated. Closing


Contribute and incorporate "unsafe checker" tool

Upon discussion with the Dragonwell team and a demo of their Unsafe Checker tool, we should consider where this tool can be contributed for use by the community and whether we can or should incorporate it in any of our test workflows.
What is it?  Unsafe checker is a Java agent that verifies the correctness of the usage of Unsafe API.  It can be used for:

Memory Leak Check
Range and Offset Check
Type Check

Next steps: let's indicate where such a tool can be useful at the project, what issues this tool currently surfaces and  identify use cases and value that it may bring to the project's main goals.

Closing as stale.


Dispute an agreement

It should be possible for either the issuer and beneficiary to dispute an agreement e.g. let's say one party wants to repeal it, and the other does not. In that case, there should be a nominated judge account that is able to either accept or reject the disputed repeal state.

Can the contract be sold or in case of force majeure allow for it to be fulfilled by another party?
Sent from my iPhone

On Oct 15, 2017, at 10:59, Alex Batlin<EMAIL_ADDRESS>wrote:
It should e be possible for either the issuer and beneficiary to dispute an agreement e.g. let's say one party wants to repeal it, and the other does not. In that case, there should be a nominated judge account that is able to either accept or reject the disputed repeal state.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

At this point I am thinking this is non-transferrable.


Allow including inside of extern C

I have production code in C and I'm using C++ test framework and that caused some linkage problems with nanopb. This patch fixed my problem and hopefully some others too.

Thanks, I didn't even know about extern "C++".


use startMirage function consistently

Ensure that in both the component test and the helper we use the same startMirage function name.

Thank you!


Dev

Your welcome, big thanks to you for this great package
في الخميس، 28 أكتوبر 2021 في 7:38 م تمت كتابة ما يلي بواسطة Đặng Ngọc
Đức @.***>:

@Hhazeen https://github.com/Hhazeen Thanks for your PR!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dangngocduc/comment_tree/pull/6#issuecomment-954015315,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMP43FQ5N2SDKJDERZGTWWLUJF4B7ANCNFSM5G5KP63Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Where's the source code of the demo?

The demo project ( https://grapesjs.com/demo-mjml.html )  provides some useful information on how to use the grapesjs-mjml. However, I cannot find source code of the demo project.  Where can I find it?
Especially, the demo project has custom ckeditor. I don't know how to integreate ckeditor in my project. It'd be very helpful to find an example of it.

I don't think the demo page is open source, it also might need a upgrade since it is running a pretty old version of grapesjs-mjml and grapesjs. I would recommend cloning this repro and running the dev server to test things out for now

I don't think the demo page is open source, it also might need a upgrade since it is running a pretty old version of grapesjs-mjml and grapesjs. I would recommend cloning this repro and running the dev server to test things out for now

@DRoet I think the demo pages are open source. All the demo pages are in gh-pages branch in grapesjs repo.
@KevinShea1990 Here's the source code for the demo you are looking for
https://github.com/artf/grapesjs/blob/gh-pages/demo-mjml.html

@DRoet I think the demo pages are open source. All the demo pages are in gh-pages branch in grapesjs repo.
@KevinShea1990 Here's the source code for the demo you are looking for
https://github.com/artf/grapesjs/blob/gh-pages/demo-mjml.html


update description of Period type once issue-1372 is merged

I actually don't think anything will need to be changed for this in the docs, based on just looking through. It is just that the output will look different in the text editor. Do you think we need to update the corresponding video(s) accordingly @Jdville03?

I don't think we need to update the video for that in particular right now. But a project in the new year will be updating all the videos so we can get rid of the outdated ui in all of them.

Sounds good!


[BUG] Extra argument passed to transactions in Stable Cadence Pre-release version

Problem
When trying to send a transaction with parameters, I consistently get
❌ Command Error: error parsing transaction arguments: argument count is [n+1], expected [n]

Steps to Reproduce
With the Stable Cadence CLI preview release installed, try the following transaction:
transaction(s: String) {
    prepare(signer: &Account) {
        log(s)
    }
}

flow transactions send ./transactions/test_param.cdc hello

And it should output:
❌ Command Error: error parsing transaction arguments: argument count is 2, expected 1

Confirm it works without the transaction parameter:
transaction {
    prepare(signer: &Account) {
        log("hello")
    }
}

flow transactions send ./transactions/test.cdc

And it should succeed.
Acceptance Criteria
Transactions with and without params should execute.
Context
Using the preview release to update FLIPs and contracts.
Machine Info
OS: macOS 13.6 (22G120)
CLI: v1.5.0-stable-cadence.2

This has been fixed in head i belive

Just tried with fresh install of v1.5.0-stable-cadence.2 and still experiencing the issue.

@SupunS Is it possible that the latest Stable Cadence release, v1.5.0-stable-cadence.2, does not yet have the Flow CLI master code which includes #1179 and #1182? Could you please merge master into the Stablwe Cadence feature branch and create a new preview build?

I took a stab at this @turbolent @SupunS

Merging the master seems to fix it.
We should probably add some test cases around this. Didn't see any test in the above two PRs (#1179 and #1182) either.


couple ideas

Few ideas for your review

This draft can be closed, I think I've implemented everything onto my branch now


Is it possible to set an environment variable

I need to have some of the environment variables set for the cgi application. How would I do that? For example, if I have this URL, then the cgi script expect certain environment variables to be set namelt PATH_INFO and REQUEST_URI
 http://localhost:8080/cgi-bin/test.cgi/the/rest/of/the/url

 PATH_INFO = /the/rest/of/the/url
 REQUEST_URI = /cgi-bin/test.cgi/123/456/ttt

So how do I set the enviromnet for the cgi script before invoking it.

http://www.cgi101.com/book/ch3/text.html

Was able to solve. Duh! Noticed I just pass my environment variables as the second option.
 {env:{foo:"bar"}}


Indentation of interpolated expression missing an indent level

Hi @Sec-ant 👋 Hope you're good!
I encountered a problem with indentation with interpolation in a condition of a WHERE line:
Expected:
sql`
  SELECT
    *
  FROM
    longer_table_name_with_many_characters
  WHERE
    longer_table_name_with_many_characters.id IN ${sql(
      longerTableNameWithManyCharactersIds,
    )}
`

Actual (the 2 lines after sql( are missing 1 indent level):
sql`
  SELECT
    *
  FROM
    longer_table_name_with_many_characters
  WHERE
    longer_table_name_with_many_characters.id IN ${sql(
    longerTableNameWithManyCharactersIds,
  )}
`

Workaround
Wrapping the interpolation in parentheses works around the problem:
sql`
  SELECT
    *
  FROM
    longer_table_name_with_many_characters
  WHERE
    longer_table_name_with_many_characters.id IN (
      ${sql(longerTableNameWithManyCharactersIds)}
    )
`

But this workaround fails with a similar problem if the JS expression is too long and Prettier wraps it:
sql`
  SELECT
    *
  FROM
    longer_table_name_with_many_characters
  WHERE
    longer_table_name_with_many_characters.id IN (
      ${sql(
    evenLongerTableNameWithManyCharactersKeepsGoingEvenLongerAndLongerAndLongerIds,
  )}
    )
`;

I'll need some help on this one. If anyone can offer a general solution, I'd be happy to merge.


Fix package.json push

Is there an existing issue for this?

[X] I have searched the existing issues

Current behavior
None
Minimum reproduction code
http://example.com
Steps to reproduce
None
Expected behavior
None
@polar-rules/nest-task version
0.1.0
Node.js version
20.x.x
In which operating systems have you tested?

[X] macOS
[ ] Windows
[ ] Linux

Other
None

Not done.

Fix package.json version

Correct tag issue


enable L3VXLAN

Signed-off-by: Boris Glimcher<EMAIL_ADDRESS>works outside Azure:
Attaching to opi-evpn-bridge-opi-test-1
opi-evpn-bridge-opi-test-1  | + sleep 60
opi-evpn-bridge-opi-test-1  | + ping -c 3 <IP_ADDRESS> -I br100
opi-evpn-bridge-opi-test-1  | PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes
opi-evpn-bridge-opi-test-1  | 64 bytes from <IP_ADDRESS>: seq=0 ttl=64 time=0.408 ms
opi-evpn-bridge-opi-test-1  | 64 bytes from <IP_ADDRESS>: seq=1 ttl=64 time=0.212 ms
opi-evpn-bridge-opi-test-1  | 64 bytes from <IP_ADDRESS>: seq=2 ttl=64 time=0.181 ms
opi-evpn-bridge-opi-test-1  |
opi-evpn-bridge-opi-test-1  | --- <IP_ADDRESS> ping statistics ---
opi-evpn-bridge-opi-test-1  | 3 packets transmitted, 3 packets received, 0% packet loss
opi-evpn-bridge-opi-test-1  | round-trip min/avg/max = 0.181/0.267/0.408 ms
opi-evpn-bridge-opi-test-1  | + ping -c 3 <IP_ADDRESS>
opi-evpn-bridge-opi-test-1  | PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes
opi-evpn-bridge-opi-test-1  | 64 bytes from <IP_ADDRESS>: seq=0 ttl=64 time=0.177 ms
opi-evpn-bridge-opi-test-1  | 64 bytes from <IP_ADDRESS>: seq=1 ttl=64 time=0.172 ms
opi-evpn-bridge-opi-test-1  | 64 bytes from <IP_ADDRESS>: seq=2 ttl=64 time=0.184 ms
opi-evpn-bridge-opi-test-1  |
opi-evpn-bridge-opi-test-1  | --- <IP_ADDRESS> ping statistics ---
opi-evpn-bridge-opi-test-1  | 3 packets transmitted, 3 packets received, 0% packet loss
opi-evpn-bridge-opi-test-1  | round-trip min/avg/max = 0.172/0.177/0.184 ms
opi-evpn-bridge-opi-test-1  | + ping -c 3 <IP_ADDRESS> -I br20
opi-evpn-bridge-opi-test-1  | PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes
opi-evpn-bridge-opi-test-1  | 64 bytes from <IP_ADDRESS>: seq=0 ttl=64 time=0.239 ms
opi-evpn-bridge-opi-test-1  | 64 bytes from <IP_ADDRESS>: seq=1 ttl=64 time=0.239 ms
opi-evpn-bridge-opi-test-1  | 64 bytes from <IP_ADDRESS>: seq=2 ttl=64 time=0.229 ms
opi-evpn-bridge-opi-test-1  |
opi-evpn-bridge-opi-test-1  | --- <IP_ADDRESS> ping statistics ---
opi-evpn-bridge-opi-test-1  | 3 packets transmitted, 3 packets received, 0% packet loss
opi-evpn-bridge-opi-test-1  | round-trip min/avg/max = 0.229/0.235/0.239 ms
opi-evpn-bridge-opi-test-1 exited with code 0

but fails in Azure:
modprobe -v vrf
modprobe: FATAL: Module vrf not found in directory /lib/modules/5.15.0-1036-azure

VRF kernel module is not present ion azure images
find /lib/modules/`uname -r` -type f

but
grep CONFIG_NET_VRF /boot/config-$(uname -r)
2023-05-02T15:28:33.5108804Z CONFIG_NET_VRF=m

@mardim91 FYI

fixed by https://github.com/opiproject/opi-smbios-bridge/pull/103


[NEW ALGORITHM] Suffix Array And LCP Array

Issue will be closed if:

You mention more than one algorithm. You can create a separate issue for each algorithm once the current one is completed.
You propose an algorithm that is already present or has been mentioned in a previous issue.
You create a new issue without completing your previous issue.

Note: These actions will be taken seriously. Failure to follow the guidelines may result in the immediate closure of your issue.

Name:
[NEW ALGORITHM]  Suffix Array And LCP Array
About:
Useful for pattern matching and solving substring problems efficiently.

Labels:
new algorithm, gssoc-ext, hacktoberfest, level1

Assignees:

[x] Contributor in GSSoC-ext
[x] Want to work on it

Do not create multiple issues.


Unable to use the resize function

I am unable to use the resize function in iOS... it is giving the following error:
TypeError: undefined is not an object (evaluating 'window.ImageResizer.resize')
please advice..

Do you have more then just this one line?

@JoschkaSchulz no getting only this and error is raised by jquery.min.js

@supportreq are you using a framework? Can you post a snippet of your code?


Investigate Horizon High Memory Issues

Partners and users on Discord have reported very high memory consumption from Horizon when the state rebuild happens.
### Tasks
- [x] Setup a dev environment in EC2 for testing
- [x] Recreate the high memory consumption of Horizon
- [x] Memory profile Horizon during state rebuild

Tamir's new change has significantly improved the RAM usage and also the state rebuild times. The time decreased from ~4hrs to ~1.1hrs. While memory usage dropped from ~25GB to ~5GB.


PP: edit step 7. Enter vehicle details content

could you add: at this point the user should also be allowed to select the permit start date and choose the length of permit (if available)

@EUzkuraityte Permit lengths - what form do they take? i.e:
Select the permit length you need:

1 month
3 month
6  month
12 month

Also how many LAs offer permit length/want to offer permit length?


Demangle issue thrown in log, plus crashes

I am trying to analyze the RepetierServer binary provided in the 'Linux 64 bit intel' debian installer of Repetier Server found here, and am receiving Unsupported demangle type: transaction clone in the binja logs.
Currently running dev build 2461 (personal) on Ubuntu.
This analyze has also crashed a few times, but might be due to resource issues. dual core VM with 4 GB ram and no swap.
Binary will be uploaded shortly.

crashes likely due to out of memory error. here is the binary for the demangle issue:
repetier.tar.gz

Duplicate of #1016


Write sub-page  ABOUT The Current Team and Management structure

Current DRAFT is taken from FreeREG.  There is also a page DRAFT About Us.  Introductions to the Project Team.  which is very out of date - much is now History - some needs incorporating here, and some on FreeUKgenealogy.org.uk
Include on this page link to the FUG website page on the Team.
Remaining issues

[ ] corrections needed to personnel
[ ] contact details
[ ] conformance to style, typos, etc.

@PatReynolds
Firstly, unfortunately I don't know what FUG is, so I can't do anything about that.
There are numerous problems with this page, as a whole. As you say, the current draft appears to have been taken from another site. The individual pages from FreeBMD(1) identified in the hyperlinks audit were moved onto Refinery back in May to June but have never been incorporated into the draft page and another FAQ page was created and also moved into Refinery but never incorporated; and all the FreeBMD(1) pages did not use active-case and the new guidance issued in October requires active-case.
So, there are currently three inductions to the Current Team and I think two to the current management structure (which may be in the wrong case - non-active). Comfort and I, via Git, were asked to give priority to Help pages  and some of the About pages that were allocated to Transcriptions, so I don't think any editorial work has been done on the About tag, other than on the FAQ page; and that FAQ page is fairly close to completion. The FAQ version of about the Current Team, may be the most update, but it is un-liked to the top-level page. Many of these problems are common to the Help page.
Can you advise where the most up to date version of the Current Team and Management structure can be found?

@PatReynolds
Firstly, unfortunately I don't know what FUG is, so I can't do anything about that.
There are numerous problems with this page, as a whole. As you say, the current draft appears to have been taken from another site. The individual pages from FreeBMD(1) identified in the hyperlinks audit were moved onto Refinery back in May to June but have never been incorporated into the draft page and another FAQ page was created and also moved into Refinery but never incorporated; and all the FreeBMD(1) pages did not use active-case and the new guidance issued in October requires active-case.
So, there are currently three inductions to the Current Team and I think two to the current management structure (which may be in the wrong case - non-active). Comfort and I, via Git, were asked to give priority to Help pages  and some of the About pages that were allocated to Transcriptions, so I don't think any editorial work has been done on the About tag, other than on the FAQ page; and that FAQ page is fairly close to completion. The FAQ version of about the Current Team, may be the most update, but it is un-liked to the top-level page. Many of these problems are common to the Help page.
Can you advise where the most up to date version of the Current Team and Management structure can be found?

First draft done.  Remaining work noted aboce.

For discussion: I have removed most of the contact details because I believe this is intended to be a publicly facing page -  contacts for volunteers will be developed as another task (would need a new story creating).

@AlOneill to proof it and @PatReynolds to create the new story.

@PatReynolds I will assume, unless I hear otherwise, that this is the page you wish me to proof:
https://test3.freebmd.org.uk/cms/about/draft-about-us-introductions-to-the-project-team
There are some unexpected issues with this page, which is why I am asking if I'm looking at the correct one:

h1 heading includes two full-stops, i.e. "DRAFT About Us.  Introductions to the Project Team."

top level heading used in Refinery is h3

no layout

In addition, the casing of "linBMD" is different from the casing of other project software such as: WinBMD, MacBMD and so on. It does not seem to be a Linux thing to use lower-case like this, so I'll change it to LinBMD (which is incidentally easier to parse) in a sans-serif font).
I was taught that a page should still make sense if all the headings are taken away (because it would be a mistake to read the headings out loud). That could be an old-school way of looking at things, but I'll expand the opening sentence of each section so that it can stand alone without any loss of information or meaning. And after all, the actual content is pretty sparse.

@PatReynolds
Initial checking of page completed. The address is now (provisionally):
https://test3.freebmd.org.uk/cms/about/draft-introducing-the-project-team
Issues noted above have been fixed. There are two queries near the top of the page — both are marked with highlighter.
The other thing to consider is whether and how we use email links — there are pros and cons. (There was one malformed email link already on the page, so I've marked up both addresses to be valid, at least for now.)

selecting the email link is convenient if you use an email client (e.g. Mail or Outlook) but might produce an unhelpful result if you do not. For instance, opening Mail and asking if you want to set up an account
mobiles should be OK — they will take a user to the default app, whatever that is
copying an email address from the link is possible but awkward for some
will our users be familiar enough with the vagaries of an email link on their device to not let any odd behaviour bother them?
it might be better for desktop and laptop users if we ditch the email links — but how awkward would that be for mobile users, if at all?

FreeBMD1 links to a form - which might be more convenient for all?

@PatReynolds to check how this operates on mobiles.

@PatReynolds I have un-linked the second email address so that you can compare the two approaches.

On Chrome the 'mail to trustees link' does not engage with Gmail. But the address can be copied and pasted.  I much prefer the 'unlinked' version of presentation of information used at the bottom of the page.
Test on Android showed first type opens up gmail with the correct address.  The bottom of page version of the markup offers a choice of gmail, copy, share or "select all".
Because of increased ease of use on Chrome, and increased range of utilities on Android, I suggest we go with the bottom of page implementation.
Minor edits made where requested,

@PatReynolds
Thank you. I've removed the link for the first email address.
Next I'll make the page sync-ready which will include listing a link to it in 'Related pages' on the About page. Let me know if that is not what you want.

Thanks, @AlOneill yes, that's what is needed.


mindustry,mindustry-server: 104.6 -> 120

Motivation for this change
After a ton of betas, the "codename 6.0" release is finally stable: https://github.com/Anuken/Mindustry/releases/tag/v120
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Builds and starts on NixOS x86_64-linux. The audio sounds a bit crackly to me, not sure if it's part of the soundtrack though.

@petabyteboy it's just how it is I think

Result of nixpkgs-review pr 105590 run on x86_64-linux 1

2 packages built:

mindustry
mindustry-server

Result of nixpkgs-review pr 105590 run on x86_64-darwin 1

2 packages failed to build:

mindustry
mindustry-server

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.7/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 1m 15s
10 actionable tasks: 7 executed, 3 up-to-date
installing
post-installation fixup
strip is /nix/store/pd852x17wab1glhw1ahqacsnlvcidr7a-cctools-binutils-darwin-949.0.1/bin/strip
patching script interpreter paths in /nix/store/hfmns925g1qlbcflhmjp7sbrlz0ry49g-mindustry-deps-120
hash mismatch in fixed-output derivation '/nix/store/mm5dpz4q45mm45xrcmsgc00x9x1vg54d-mindustry-deps-120':
  wanted: sha256:1yv9l8zdml6drmvlgv45w3qas9qmb654x4kja3an4d16k020khr7
  got:    sha256:0w63syrqvq2g7ra99xr18mhinrkhckvvhy6c4psfivgk71080jj3
cannot build derivation '/nix/store/4l0kws7c7nln04m7b7ciamk2x69vl51d-mindustry-120.drv': 1 dependencies couldn't be built
error: build of '/nix/store/4l0kws7c7nln04m7b7ciamk2x69vl51d-mindustry-120.drv' failed

A hash mismatch? I don't think I can fix that for every new release, especially with the broken ofborg darwin builder (I don't have a darwin machine). So I think I'll just mark the package as broken for darwin, unless someone volunteers to generate the hash at each new release.

Maybe this can be fixed by changing to gradle2nix at some point.

Indeed, I hope it gets integrated into nixpkgs soon

A hash mismatch? I don't think I can fix that for every new release, especially with the broken ofborg darwin builder (I don't have a darwin machine). So I think I'll just mark the package as broken for darwin, unless someone volunteers to generate the hash at each new release.

I could generate it but with some delay.

The deps directory contains com/github/Anuken as well as com/github/anuken. @SuperSandro2000 can you post a directory listing of the -deps derivation? I would like to see if these lowercase/uppercase names differ.

@petabyteboy
anuken~nix~case~hack~1

https://termbin.com/bv8w
😂
these derivations will be built:
  /nix/store/7pzl8gisaly7r7xrd4aihgy47f4713wc-mindustry-120.2.drv
building '/nix/store/7pzl8gisaly7r7xrd4aihgy47f4713wc-mindustry-120.2.drv'...
unpacking sources
unpacking source archive /nix/store/jdxyya7q9gicm4rdnkwjxlzv1g2zs0xd-source
source root is source
patching sources
configuring
no configure script, doing nothing
building

Welcome to Gradle 6.7!

Here are the highlights of this release:
 - File system watching is ready for production use
 - Declare the version of Java your build requires
 - Java 15 support

For more details see https://docs.gradle.org/6.7/release-notes.html

To honour the JVM settings for this build a new JVM will be forked. Please consider using the daemon: https://docs.gradle.org/6.7/userguide/gradle_daemon.html.

FAILURE: Build failed with an exception.

* What went wrong:
Unable to start the daemon process.
This problem might be caused by incorrect configuration of the daemon.
For example, an unrecognized jvm option is used.
Please refer to the User Manual chapter on the daemon at https://docs.gradle.org/6.7/userguide/gradle_daemon.html
Process command line: /nix/store/54l615jr0sdlzs91vxwncs9a3kab90bm-zulu14.28.21-ca-jdk14.0.1/zulu-14.jdk/Contents/Home/bin/java --add-opens java.base/java.util=ALL-UNNAMED --add-opens java.base/java.lang=ALL-UNNAMED --add-opens java.base/java.lang.invoke=ALL-UNNAMED --add-opens java.prefs/java.util.prefs=ALL-UNNAMED -Xms256m -Xmx1024m -Dfile.encoding=UTF-8 -Duser.country=US -Duser.language=en -Duser.variant -cp /nix/store/rhmg9cp3pzw81ly0m8aq9k3d3lfknlsc-gradle-6.7/lib/gradle/lib/gradle-launcher-6.7.jar org.gradle.launcher.daemon.bootstrap.GradleDaemon 6.7
Please read the following process output to find out more:
-----------------------

FAILURE: Build failed with an exception.

* What went wrong:
java.net.SocketException: Operation not permitted

* Try:
Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Exception is:
org.gradle.api.UncheckedIOException: java.net.SocketException: Operation not permitted
        at org.gradle.internal.UncheckedException.throwAsUncheckedException(UncheckedException.java:61)
        at org.gradle.internal.UncheckedException.throwAsUncheckedException(UncheckedException.java:41)
        at org.gradle.internal.remote.internal.inet.TcpIncomingConnector.accept(TcpIncomingConnector.java:62)
        at org.gradle.launcher.daemon.server.DaemonTcpServerConnector.start(DaemonTcpServerConnector.java:86)
        at org.gradle.launcher.daemon.server.Daemon.start(Daemon.java:169)
        at org.gradle.launcher.daemon.bootstrap.DaemonMain.doAction(DaemonMain.java:125)
        at org.gradle.launcher.bootstrap.EntryPoint.run(EntryPoint.java:50)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:564)
        at org.gradle.launcher.bootstrap.ProcessBootstrap.runNoExit(ProcessBootstrap.java:60)
        at org.gradle.launcher.bootstrap.ProcessBootstrap.run(ProcessBootstrap.java:37)
        at org.gradle.launcher.daemon.bootstrap.GradleDaemon.main(GradleDaemon.java:22)
Caused by: java.net.SocketException: Operation not permitted
        at java.base/sun.nio.ch.Net.bind0(Native Method)
        at java.base/sun.nio.ch.Net.bind(Net.java:479)
        at java.base/sun.nio.ch.Net.bind(Net.java:468)
        at java.base/sun.nio.ch.ServerSocketChannelImpl.bind(ServerSocketChannelImpl.java:220)
        at java.base/sun.nio.ch.ServerSocketAdaptor.bind(ServerSocketAdaptor.java:88)
        at java.base/sun.nio.ch.ServerSocketAdaptor.bind(ServerSocketAdaptor.java:80)
        at org.gradle.internal.remote.internal.inet.TcpIncomingConnector.accept(TcpIncomingConnector.java:59)
        ... 11 more

* Get more help at https://help.gradle.org

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org
builder for '/nix/store/7pzl8gisaly7r7xrd4aihgy47f4713wc-mindustry-120.2.drv' failed with exit code 1
error: build of '/nix/store/7pzl8gisaly7r7xrd4aihgy47f4713wc-mindustry-120.2.drv' failed


currying functions with optional arguments

When a function is curried that does not require all of its arguments to be successfully invoked, the bassline.curry causes the function to never be invoked. Consider the following example.
var nothing = require("bassline").nothing;

function next(error) {
  console.log("in nothing");
}

nothing().fold(next, function(){console.log("in just");});

The expected output when this is run is in nothing, but the actual output is empty.

The caller must now specifically curry functions for folding, functors, and monads.


Adding Additional Tricks

Added a few simple to implement tricks

Mockball
Sword Only Dark Rooms
Light World South Fake Flippers

I updated the version number as discussed in the other PR. Wasn't sure if you meant updating to 3.5.0 or 4.0.0, so I did this for now. Feel free to update (or tell me to update) if you'd prefer 4.0.0.

4.0 is probably fine considering the amount of technically breaking updates we've done.

Updates made


Site is built only when there are version changes

The current action only runs the build-ui job if the version changed and it should be executed either the versions changed or the ui code changed.

Done now!


Updating readme

The port for postgres was incorrect and set to the mysql port. This was updated along with removing extra postgres config from the end of the mysql line
./cloud_sql_proxy -instances=${PGSQL_CONN_NAME}**=tcp:5432**

I signed it!


Remove scheduleSyncPeriod & hard-code to 1 minute

Users don't need to be able to customize how frequently Ark checks if it's time to create a new backup for a schedule. This needs to be hard-coded to 1 minute.

We're now tracking this as part of #538 so closing this issue.


fix(antlr): Map "BSD License" to ANTLR-PD for version 2.7.7

Version 2.7.7 declares "BSD License" in its POM 1, which is wrong when looking at the LICENSE.txt contained in the source artifact. Instead, it is the ANTLR-specific Public Domain license, see 2

FYI @willebra.


Replace local type declarations for polyfill APIs with those imported from the polyfills.

This PR removes the local types describing the polyfill APIs in favor of the new TS externs from the polyfill repos.

[ ] https://github.com/webcomponents/polyfills/pull/457
[ ] Remove local tarball and use released polyfill packages instead.

For some reason, I can't add @sorvell as a reviewer.


map::tree_bins::concurrent_tree_bin: attempt to subtract with overflow

Hit two of these at the same time. This is after #85.
test map::tree_bins::concurrent_tree_bin ...
thread '<unnamed>' panicked at 'attempt to subtract with overflow', src/map.rs:1163:17
stack backtrace:
...
  13: core::panicking::panic
             at src/libcore/panicking.rs:54
  14: flurry::map::HashMap<K,V,S>::add_count
             at src/map.rs:1163
  15: flurry::map::HashMap<K,V,S>::replace_node
             at src/map.rs:2626
  16: flurry::map::HashMap<K,V,S>::remove
             at src/map.rs:2366
  17: flurry::map::tree_bins::concurrent_tree_bin::{{closure}}
             at src/map.rs:3429

thread '<unnamed>' panicked at 'attempt to add with overflow', src/map.rs:1159:17
...
  13: core::panicking::panic
             at src/libcore/panicking.rs:54
  14: flurry::map::HashMap<K,V,S>::add_count
             at src/map.rs:1159
  15: flurry::map::HashMap<K,V,S>::put
             at src/map.rs:1970
  16: flurry::map::HashMap<K,V,S>::insert
             at src/map.rs:1625
  17: flurry::map::tree_bins::concurrent_tree_bin::{{closure}}
             at src/map.rs:3419

Current best guess for the order of events would be as follows:

We have a (regular) bin with 1 element, which is the only element in the map (count == 1).
That element gets removed by thread 1, which is paused before the call to add_count.
An element for the same key is inserted by thread 2, which also is paused before add_count (note that all the count updates happen outside of the respective critical sections of the corresponding methods).
Thread 3 now removes this element again, and decrements count to 0.
Thread 1 gets to run again and decrements count to usize::MAX.
Thread 2 gets to run and increments count to 0.

As of yet unsure as to why this is be a problem for us, but wouldn't be for the Java implementation. There is a validated boolean in Java's replaceNode which is omitted in our implementation due to match/continue, but I don't see how that would be the culprit.

The Java implementation has all the size information as long, it's possible that they just
allow this. See also their implementation of size, which essentially clamps the actual computed value to between 0 and Integer.MAX_INT. The tree bin test may just be the first to trigger this for us.
It is also possible that the shared counters we don't yet have influence this, there seems to be some kind of contention detection there. There is also this annotation on the counter cells. But I think it is still possible for the computed value to be negative upon call to size, and that they use long just so they can perform bounds checks on int.

That's fascinating... I mean, I suppose we could just move it to an AtomicIsize instead... I guess they decided it wasn't worth the cost to keep the count accurate at all times. Mind sending a PR?

Sure. I'll put together a PR for this and one for #83 when I'm back home. Should work out to fit in tomorrow.

I can confirm that this was fixed by #88 (perhaps unsurprisingly) after having run it in a loop for a while.


Bumped mypy-extensions to 0.4.3

I have made things!
Checklist

[ ] I'm sure that I did not unrelated changes in this pull request
[ ] I have created at least one test case for the changes I have made
[ ] I have added my changes to the CHANGELOG.rst

Related issues
Format is:

Closes #759
Refs #759

I have manually updated the mypy-extensions entry to the 0.4.3 version in the Pipfile.lock in order to make the tests pass in Linux.
The build still doesn't pass because of the Windows 10 environment with Python 3.7. I'm sorry but I'm not familiar with Windows so I can't reproduce it.

@ceccoemi It seems like it did not help.

@lk-geimfari as you can see here, the build fails because of the Windows 10 environment, but the tests with Linux now work

@ceccoemi Great! Thanks for fixing this issue!


Is there an example with animations?

I'm trying to evaluate this library for animated gltf2 support.

https://github.com/KhronosGroup/glTF-Sample-Models


Windows Icon Scaling

Feature/enhancement description:
In the File explorer, when I'm in Small icon mode, the icon of Lorien is in good resolution but when I switch to Medium or Large icon mode, it become pixelated.

Fixed in #265


Deprecate copy of common.Point in xref.proto [phab:D1552]

Created by schroederc at 2017-04-20 18:21:23:
This reuses the common.Span message for all pairs of points in xref.proto and deprecates the current Location.Point usages.

schroederc wrote at 2017-04-20 18:24:18:
I'm going to copy the span/point proto fields to keep existing clients working for a while.

schroederc wrote at 2017-04-20 19:30:33:
Leave existing usages of Location.Point deprecated

schroederc wrote at 2017-04-20 19:32:22:
Description

fromberger wrote at 2017-04-20 19:33:24:
Please also update the generated code before landing this.

fromberger commented inline (context lost during migration) at 2017-04-20 19:36:29:
[deprecated = true] should work on the fields, also.

fromberger accepted this revision at 2017-04-20 19:36:40

schroederc wrote at 2017-04-20 19:38:10:
Update generated source files

schroederc wrote at 2017-04-20 19:43:19:
Annotate deprecated fields

schroederc closed via commit fd74eeac448e65704ba4c95f96bbe582f14cc2f4 at 2017-04-20 19:46:38


Throw an error if a second join call is fired when one already in progress

already throws an error if join is in progress


media() assumes min / max values when not declared

the media() fuction would assume min/max values when I only give two values similar to just min when there is one value.
@include media(em(600) em(900)) {
   //style
}

would be the same as:
@include media(min-width em(600) max-width em(900)) {
   //style
}

I noted this in our upcoming discussions.
Closing for now.


Install expo-snack- sdk on app/showcase and render component as a expo snack

render components imported from 'packages/example' as a expo snack

Render with Expo web without using expo-snack


distributed TP model forward output's requires_grad is False

Hi, thanks for the nice work!
I've been trying to optimize the performance of the TP wrapper here, and the first thing that came to mind was balancing out the compute on each rank using distributed / multiprocessing (as opposed to threading).
I've been wrapping my model like the following
tp.tensor_parallel(model, distributed=True, device_ids=device_ids)

But it seems the final model output from the forward pass doesn't require grad anymore, which makes it impossible to call loss.backward().
Code below to reproduce
# torchrun --nproc-per-node 2 --master_port=1234 training/tp/standalone.py
import os

import tensor_parallel as tp
import torch
import transformers
from torch import distributed as dist

from peer_lm import constants

def get_local_rank():
    return int(os.getenv("LOCAL_RANK", -1))

def get_world_size():
    return int(os.getenv("WORLD_SIZE", 1))

dist.init_process_group(backend="nccl", rank=get_local_rank(), world_size=get_world_size())
pg = dist.distributed_c10d._get_default_group()
torch.cuda.set_device(get_local_rank())

current_device = torch.device(torch.cuda.current_device())
device_ids = [current_device]
print(device_ids)

model = transformers.LlamaForCausalLM.from_pretrained(
    constants.SHARED_MODEL_DIR / "llama-teeny", torch_dtype=torch.bfloat16
)
model, _ = tp.tensor_parallel(model, distributed=True, device_ids=device_ids)
tokenizer = transformers.AutoTokenizer.from_pretrained(constants.SHARED_MODEL_DIR / "llama-teeny", use_fast=True)
if tokenizer.pad_token is None:
    tokenizer.pad_token = tokenizer.eos_token
tensors = tokenizer(["how are you?", "I love you and you too."], return_tensors="pt", padding=True)
tensors = {
    "input_ids": tensors["input_ids"],
    "attention_mask": tensors["attention_mask"],
    "labels": tensors["input_ids"],
}

with torch.enable_grad():
    model.train()
    tensors = {k: v.to(current_device) for k, v in tensors.items()}
    outputs = model(**tensors, return_dict=True, output_hidden_states=True)
    print(outputs.logits)
    print(outputs.logits.requires_grad) # False
    print(outputs.keys())

    hs = outputs.hidden_states
    s0 = hs[0]
    print(s0)
    print(s0.requires_grad) # False

Finally, pasting some of my system configs here

pip dump

tensor-parallel          2.0.0
termcolor                2.3.0
tiktoken                 0.4.0
tokenizers               0.13.3
tomli                    2.0.1
tomlkit                  0.11.8
toolz                    0.12.0
torch                    2.0.1
torchvision              0.15.2
tqdm                     4.65.0
transformers             4.30.0.dev0

hardware: 2 A100 gpus nvlink interconnect

I can confirm that the intermediate hidden_states within the forward pass of the submodules still has requires_grad=True, so it's likely something related to the final step (communication?).

outputs.logits.requires_grad=True for Bloom models. The problem must with LLaMA/LLaMA-2 models specifically. I'll look into it.

Made some progress, it seems the activation doesn't require_grad anymore after going through lm-head.

Ok it's caused by the last lm-head being sharded and all gather of the activations along the column dimension doesn't prop gradients.

A simple workaround for now would be to not shard the lm-head; another solution would be to have run the all gather through the custom a torch.autograd.Function with backward pass implemented. I'll test out the first solution for now.


Add jemoji support

This adds support for markdown emoji.

And just so it's clear. https://github.com/jekyll/jemoji

@erikng Thanks a lot, I added this in https://github.com/nielsenramon/chalk/commit/2ad1014fb2b09b75cf7423164659b1b1594a407b
added a bit more styling and info about it ;-)

Nice.


Add redirect support

Currently, redirects received from the NextJS server are not supported. More specifically, I'm referring to this kind of redirects:

I have roughly added support for 308 codes, yet the implementation is not tested nor does it feel complete. Could you please guide me on how to complete this implementation?

Hi Silviu,
thank you for your contribution. I've implemented this in 8e2344c990506a606b79b0579a9eb36335c8ad58 since it was easier to implement than explain. Hope that it's going to be useful.

Hi Danial,
Thank you for the solution, looking forward to using it!


Add CRYSTAL_LIBRARY_PATH for lookup static libraries

The compiler will use the CRYSTAL_LIBRARY_PATH environment variable as a first lookup destination for the static libraries that are wanted to be linked.
It follows the other configuration options conventions:

Upon building the compiler the CRYSTAL_CONFIG_LIBRARY_PATH variable can be used to define the default embedded.
The Crystal::LIBRARY_PATH constant is defined on compiled programs.

Fixes #251 :-)

If the goal is to fix #251 then this is fine, otherwise I fail to see to point. We can already set LIBRARY_PATH (for static libraries) and LD_LIBRARY_PATH (for shared libraries) when building, to help the linker find libraries.

The goal is to have a priority lookup of static libraries. Independent of having pkg-config or not.
That will also allow

linking against embedded shipped libraries easier.
end user to link against other builds of the library without learning to write a .pc and setup pkg-config.

I'm following what it was more or less discussed in the forum.
After this I would like to implement some version of link annotation override from compiler options.

If this only applies to static libraries, should that maybe be reflected in the env var's name? Like CRYSTAL_STATIC_LIBRARY_PATH?

@straight-shoota maybe, but it was too long for my eyes.
We could also append it to -L so it can be used for shared library also if it is prefered. (👍/👎 in comment if this is/isn't wanted)

When the upcoming changes of a custom GC and embedded compiler-rt, I think we need to push this.
If the user has a bdwgc package installed, or another llvm version that is not the one the compiler is used, the pkg-config, if configured, will default to those. Ignoring the ones shipped with the compiler.
We need some setting for libraries that should be prioritized more than pkg-config.

I think I would have preferred a CRYSTAL_LIBRARY_OVERRIDES="foo=/path/too/foo.a bar=..." variable, the name seems clearer and easier to use. It'd just override the entire linking lookup logic.


feat: progress plugin

Summary
#4973

Last time in #3803, we only supported the activeModule report and progress_handler compatibility. However, it had some issues, such as the default_handler not correctly reflecting the build duration. Due to several missing sealing hooks, the process duration wasn't accurate either.

[ ] default handler
[ ] options
[ ] setup
[ ] dependencies count
[x] modules count(support in the last version)
[ ] entries count(no support hook)
[x] activeModule(support the last version)
[ ] sealing(lots of hooks)
[ ] emitting

Test Plan
Require Documentation?

[ ] No
[ ] Yes, the corresponding rspack-website PR is __

@ahabhgk  @hardfist  I've encountered an issue, which can be described as follows: Due to rspack using multiple threads for building modules, it means that during the build process of one module, other modules may complete their builds. Since the profile report's time is calculated by subtracting the time of the last handler trigger from the time of the current handler trigger, both succeed_module and build_module will trigger the handler. Through these two hooks, the module's build time can be obtained. However, if other modules trigger succeed_module in the middle, the accuracy of the module's build time may be compromised. I'm not quite sure how to achieve the best compatibility with webpack plugins and ensure the accuracy of module build times. Therefore, I wanted to ask if you have any recommended solutions or best practices for this situation.

I suggest update the progress at make start and make end, missing the each module's progress is acceptable.
The progress of seal phase should be more accurate

I added a time parameter to default_handler. If this parameter is_some, the report time will use this time.The self.active_modules will store the build start time.The succeed_module will use the build start time to get the real build time and finaly pass the time to handler.

!bench


Upgrade feast to 0.25

Companion PR to https://github.com/NVIDIA-Merlin/systems/pull/208

https://github.com/NVIDIA-Merlin/systems/issues/174


Corrections

I would suggest using xtc format for pentapeptide again to save resources (traffic). The dcd files are aligned, right?

LGTM

I would suggest using xtc format for pentapeptide again to save resources (traffic). The dcd files are aligned, right?

The dcd files are, the xtc files are not. If we exchange everything that requires alignment, like the backbone atom positions in 00, we can switch back.

Do we need both, aligned and non-aligned?


Update readme with credit to the music.

Please update the readme with the credit for the audio.  I am going to chop it up but credit still goes to the creatot

https://pixabay.com/music/suspense-dark-mystery-trailer-taking-our-time-131566/

Good sir did you update the readme?


Add support for build variables

Adds support for CRUD functionality on a projects' build variables. Tested with unit tests and Gitlab version 8.10.2.
Fixes issue #127

Amazing! Thanks!


Idea: doFetch returns a promise

What do you think about having doFetch return a promise that resolves with the same object that the afterFetch function receives?
My use case is that I have a form that's a child of two composed react-request Fetch components (one to GET the initial form values, and one to PUT the updated values). I need the form's submission handler to call myPutFetch.doFetch() and then call a setSubmitting(false) function (which is provided as an arg in the submission handler) when the fetch is complete. I also would like to be able to have that submission handler have access to the PUT fetch's response when it's complete, so I can show errors from the server, etc.
So I'd like to be able to do something like this:
<MyComposedFetches>
  {([ myGet, myPut ]) => (
    <MyForm
      initialValues={myGet.data}
      onSubmit={(values, { setSubmitting }) => {
        myPut.doFetch({ body: JSON.stringify(values) })
          .then(({ failed, error, didUnmount }) => {
            setSubmitting(false);
            if (!didUnmount && !failed && !error) {
              alert('success');
              myGet.doFetch();
            }
          })
      }}
    />
  )}
</MyComposedFetches>

Hopefully that makes sense. Let me know what you think - thanks!

I see what you’re trying to do, @shanecav . I think a change is needed somewhere, possibly the one you proposed. I’m going to need to put more thought into this one, though 🤔

Hey @jamesplease, any new thoughts on this? I'm happy to help out however I can, just let me know - thanks!

Haven’t had time to think about it, sorry! I’ll post an update here once I have one

As a workaround @shanecav , would something like this work for you?

Store the failed, error, and didUnmount state in the component's state.
Access that state in the callback

In this code, I didn't add in the afterFetch call that would set the state, but it could be consumed like the following:
<MyComposedFetches>
  {([ myGet, myPut ]) => (
    <MyForm
      initialValues={myGet.data}
      onSubmit={(values, { setSubmitting }) => {
        myPut.doFetch({ body: JSON.stringify(values) })
          .then(() => {
            // pull this off of state
            const { failed, error, didUnmount } = this.state;
            setSubmitting(false);
            if (!didUnmount && !failed && !error) {
              alert('success');
              myGet.doFetch();
            }
          })
      }}
    />
  )}
</MyComposedFetches>

As an aside, I admit I am a little hesitant to make big changes to how the requests work here, mostly because of React Suspense. I think of changing the signature of doFetch to be a pretty big change, so I would love if we could find an alternative way for you to do what you are trying to do.
If the above system doesn't work for you, maybe we could set up time to do a GHangout to go over what you are trying to do, and what the different options are?
Thanks!

Unless I'm mistaken, I can't do myPut.doFetch(...).then(...), since doFetch doesn't return a promise (or anything else) currently - correct?
I'm currently doing a similar work-around though: in my form's onSubmit handler, I set some state in an ancestor component, which I can then access from the Fetch component's afterFetch prop callback. It's working fine for now, just seems a little dirty 😃.
Totally understand about React Suspense anticipation though, and I definitely appreciate the care around making substantial API changes. My general goal is to allow the caller of doFetch to be able to react to the fetch's completion. If there's a better way to accomplish that, I'm all for it!
Thanks @jamesplease!

Unless I'm mistaken, I can't do myPut.doFetch(...).then(...), since doFetch doesn't return a promise (or anything else) currently - correct?

Ah, you are right.
I think that should at least be updated to return the native promise from fetch. What do you think?

I'm currently doing a similar work-around though: in my form's onSubmit handler, I set some state in an ancestor component, which I can then access from the Fetch component's afterFetch prop callback. It's working fine for now, just seems a little dirty 😃.

Makes sense. I see why that would feel dirty. If doFetch returned a Promise – even if it doesn't have the same data that you have in afterFetch, would that clean up your code at all, or no?

I think that should at least be updated to return the promise from fetch. What do you think?

That would work, too. I initially suggested the same signature as afterFetch since it seems like doFetch's "completion" serves the same purpose, and that extra data is nice to have. Are you trying to keep doFetch more inline with the fetch API?

If doFetch returned a Promise – even if it doesn't have the same data that you have in afterFetch, would that clean up your code at all, or no?

Yes, definitely. I'd have to use both afterFetch and doFetch.then simultaneously in order to cover everything I need to do, but at least I'd be able to stop manually managing that data through an ancestor component's state. That alone would be a big improvement.

Are you trying to keep doFetch more inline with the fetch API?

Yeah, that's what I was thinking, although it may not be a good idea.
I have to apologize, because I was mistaken going into this issue. I was under the impression that doFetch already returned a Promise that matched the fetch API. A big change, in my opinion, would have been going from that to the afterFetch signature. But since it returns nothing right now, that opens the door for us to pick whichever one that we think is best.

That alone would be a big improvement.

Okay, awesome.
At this point, I am 100% certain that doFetch should return a Promise. I am not entirely certain if it should act like a fetch() wrapper, or use the same afterFetch signature. I can see it going both ways.
Do you have any more arguments to make in favor it following the afterFetch signature? If not, no big deal. I'll spend a few more days thinking about this, and by this weekend I think we could get something released. Does that timeline work for you?
Thanks for your patience and help with this, @shanecav ! I appreciate it! And sorry again for the delayed response.

The arguments I see in favor of the afterFetch signature are:

There wouldn't be a need to use both doFetch().then(...) and afterFetch simultaneously (in order to access the extra data afterFetch provides that fetch's API doesn't). Having to use both in parallel would be a bit complex, and it wouldn't be obvious to someone reading the doFetch().then(...) code that the request is also being responded to elsewhere (in some parent Fetch's afterFetch prop).
IMO it would be easier to understand/remember that doFetch's promise resolution and afterFetch use the same signature. I'd have an easier time thinking "this is how react-request always handles completed requests" as opposed to having to remember which one uses which signature.
The render prop and afterFetch already set a precedent for expanding on the standard fetch API for the sake of added convenience. I personally see doFetch as falling in the same category.

And no apology necessary, I'm just grateful you published and are supporting this awesome library! I'm happy with any timeline, and am more than happy to help out with PR/docs/whatever 😃 Thanks for your time on this!

@shanecav , I am working hard on fixing #151 before returning back here, but I haven't forgotten about ya! This bug fix has taken a lot of time to do, which is why I haven't gotten around to this yet.
Sorry about the delay!!

No worries, and thanks for the update!

@shanecav thanks for sharing your arguments for doFetch following afterFetch’s signature.

Agreed. Using both sounds really bad.
Agreed, I can see how this would be easier for someone to remember.
Interestingly, I don’t think of afterFetch nor the render prop as expanding on the standard fetch API. I have just thought of it as being the API that is a necessity for the React lifecycle (fetch-dedupe, on the other hand, is what I think of as expanding on the standard fetch API).

I asked some folks at work what they think. One good argument I heard is that if doFetch returns the afterFetch arguments, then folks could be confused about which to use.
I’m still not decided either way. I can see it going either way. I think I need to think more about, and optimize, for the most common use case for this, which I think is something like your use case. At the moment that I am typing this, I am leaning toward following the afterFetch API, but a few days ago I was thinking of making it return the regular Promise from fetch-dedupe so...we’ll see.
Sorry for the slowness here, btw. Still focused on that bug (which I believe is fixed. We’re going to be testing it at work today before I cut a release). This, and updating to fetch-dedupe for your awesome responseType fixed are my main priorities after the bug fix lands!

Interestingly, I don’t think of afterFetch nor the render prop as expanding on the standard fetch API. I have just thought of it as being the API that is required to map a request to the React lifecycle (fetch-dedupe, on the other hand, is what I think of as expanding on the standard fetch API).

That's a much better way of putting it! 😄

One good argument I heard is that if doFetch returns the afterFetch arguments, then folks could be confused about which to use. This could be resolved with good documentation stating the right and wrong way to use both, though.

I see the difference in use cases between doFetch's returned Promise vs. afterFetch as similar to the difference between doFetch's options argument vs. the Fetch props that they can override: It comes down to whether you need access to data in the child scope where you call doFetch.
A form is a great example of where I need access to data in a child scope when making & responding to a request, and where the afterFetch API in doFetch's returned Promise would be extra helpful:

When I call doFetch, I include the form values in the body of the request via doFetch's options argument.
I then need to respond to that request by setting the form's submitting property to false, and potentially set some form field errors if the server returned any. This is where the afterFetch API would be really helpful - it makes it easy to do things like checking if it failed and then setting form field errors based on data.

Thank you for the work and thought you're putting into this, by the way - it's very much appreciated! And again, happy to help out however I can with implementation or docs - just let me know what I can do 😄.

@shanecav , are you thinking that the doFetch promise will always resolve with the after fetch props, and never reject?
i.e.;
doFetch({ ...stuff })
  .then(({ error }) => {
    if (failed) {
      console.log('ya it failed', failed);
    }
  })

Yeah, that's what I was thinking. My understanding is that's how afterFetch currently works: if the underlying fetch promise rejects, afterFetch still gets called, and the error property of its argument comes from that fetch rejection - is that correct?
I'd personally expect the doFetch promise resolution to behave the same way if it's receiving the same argument as afterFetch - i.e. it's basically the same as afterFetch, it just lives in a different scope.
But maybe there's some reason for wanting a promise rejection that I haven't thought of. Can you think of any?

@shanecav , I am going to go ahead and release 3.0.0 with the responseType change, but without this. I agree with this issue for two separate reasons:

doFetch absolutely needs to return a Promise
the serial fetch story is not clearly outlined ( #59 )

Serial fetches is one thing I have been meaning to sit down and spend more time thinking about, but I just haven't had the time. Some folks at work and I are going to spend time in the next week deciding what we think the best approach is.
There will likely be two stories: one for the doFetch() use case, and the other for the props use case (as these are the two ways you can make a request using this lib).
Sorry for the extra delay...it's just that I want to make sure this is implemented right 🙈 and, as silly as it sounds, it wasn't until a few days ago that I made the connection between this issue and #59 😓

Just wanted to +1 this, it would be great for handling success/failure function props that the parent has passed down.

Let’s go with the afterFetch signature. Is anyone up for making a PR?

Happy to give it a go, but probably won’t have time for a week or two.

Resolved in #183 ; released as 3.1.0

Awesome! Happy to help, and thank you for the time and thought you put into this issue!


Update offer to have attached icon

20211102-liberty-was-500-blue.svg.gz

This icon is the wrong colour.  I'll update to use the icons from wasdev/azure.liberty.aro#40


🍒 Cherry pick PR #9555 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/9555 to staging 🍒

🚀 Cherry-picked to staging by @francoisl in version: 1.1.78-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @sketchydroide in version: 1.1.78-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Sample JACK client (sfizz_jack) goes into infinite loop on EOF

sfizz-1.2.3 compiled with gcc 11. If I start sfizz_jack and then hit ctrl-d to try to end the program, sfizz_jack instead goes into an infinite loop and I’ll have to hit ctrl-c to stop it.
Example (ctrl-d was pressed on line 54, i.e., immediately after I got a prompt):
[...]
    42 Positional arguments: /opt/sfz/data/VCSL-1.2.2-RC/Chordophones/Zithers/Grand Piano, Kawai - Legacy.sfz,
    43 Instrument loaded: /opt/sfz/data/VCSL-1.2.2-RC/Chordophones/Zithers/Grand Piano, Kawai - Legacy.sfz
    44 ===========================
    45 Total:
    46         Masters: 0
    47         Groups: 3
    48         Regions: 126
    49         Curves: 7
    50         PreloadedSamples: 126
    51 ===========================
    52 Unknown opcodes:
    53
    54 >
    55 >
    56 >
    57 >
    58 >
[...]
 11039 >
 11040 >
 11041 >
 11042 >
 11043 > ^C>
 11044 >
 11045 >
 11046 > Signal received
 11047 Closing...

Just tried this on current git version. Same behaviour and can also be reproduced by redirecting stdin from /dev/null.

@acli I love the potential of this project, but I'm not sure how active development is now. Have you had any success with getting a response from the developers?

@RustoMCSpit I'm hitting this, do you know anyone who can solve it?

CC @redtide @KKQ-KKQ Any help with this? Makes it impossible to run as a background job (launching with " &", etc)

@acli Do you know which source lines govern the REPL loop?

The REPL loop seems to be here if anyone knows how to fix it to properly detect EOF and exit: https://github.com/sfztools/sfizz/blob/806bf1dde49621d08c50a6a496b25c42b1afa8b8/clients/jack_client.cpp#L222
In the meantime, my workaround is to send it an empty string and newline once per minute like this:
( ( while true; do echo $'\n'; sleep 60; done ) | /home/pi/sfizz-ui/build/library/bin/sfizz_jack ...) &

@acli In my personal opinion, the client should not quit by Ctrl-D or EOF in order to run as a background job.
Could you use a "quit" command instead? Or the client freezes?

@KKQ-KKQ I have a patch too (actually three; I tried a few different ways). It’s still open after one and a half months; no one seems to be here to merge my patch or ask me any questions.

@GeoffreyPlitt Fixing this will not (and should not) allow it to run as a demon. When it detects EOF the CLI thread ends the program will (should) exit.
I think what you need is a flag to disable the CLI. I should be able to make you an unofficial patch but it’s been more than a month since I touched this. (My computer is too slow to do SFZ; I’ve sort of given up.)


Werkzeug problems with python 2.7.9

Hi,
I had an error "TypeError: 'unicode' does not have the buffer interface"
I updated werkzeug to 0.10.4 and the problem does not exist anymore.
I dont' know how to make changes and ask for my code update in requirements.txt

This issue does not apply anymore.


When using a custom Dafny.dll and language server reset is hard

If I experiment with a custom version of Dafny,
reseting the IDE back takes reseting the config values to default
AND uninstall/reinstall.
I would be nice if I could just reset it in the config

Sorry but I have a hard time following your description.
Do you mean that by default you're using the automatically installed Dafny version? And once you switch to a custom installation and back to the automatic installation you need to re-install the extension to re-trigger the automatic installation?
If I'm right, I suppose you're changing the paths to Dafny.dll and DafnyLanguageServer.dll through VSCodes UI:

I guess if you remove the manual entry in these two fields it's neither null nor undefined but an empty string from the extension's perspective.
You could probably click on the gear to the left and hit Reset setting to correctly reset it:

Alternatively, you could remove the entry from the underlying settings.json of your VSCode extension.
Restarting VSCode will still be necessary, but it should save you from re-installing the extension.

Yes, you have described what I do.
With the addition, that even when I reset the setting,
sometimes I have to uninstall/reinstall to get the plugin to download Dafny again...

I've quickly tested what you're describing. The problem is, that when you "clear" the text field by removing its contents, VSCode treats the value as an empty string instead of an unconfigured setting (you can see that by checking settings.json).
As I suggested, clicking the gear next to the setting and selecting Reset Setting correctly removes the configuration entry.
You can also double-check this behavior by opening the tab Output and select Dafny VSCode.
For example, with the automatic installation:

Removing the contents of the setting by deleting the content instead of resetting it:

I guess it makes sense to treat the empty string the same as if it was unconfigured.


Expanding a graph of language and automaticaly recognizing the same name recurrence, is it possible ?

"Due to the potentially unlimited complexity of interactions between language features defined by different dialects, there is by design no automation for combining dialects. In the general case, this is something that requires human intervention. "
Did you think the question i just made in the title is improbable or impossible ? iam interested in this, see that is important to expand artificial intelligence computer graphs languages to an automaticaly nameable architectonic computer language (similar to what is done with ASTs and in pyan).
By example see this feature request in tensorflow:
https://github.com/tensorflow/tensorflow/issues/30991

I'm sorry, I don't understand the question. I looked at the tensorflow feature request, but it didn't help much. I suppose I'm missing the required context.
Do you mean automatically avoiding some sort of naming conflicts?
If not, could you explain the question in more detail, or are there more examples?

yes,also automaticly recognize functional structures of graph, idepedently
of any parametric numerical value
Em qui, 25 de jul de 2019 19:40, Juha Jeronen<EMAIL_ADDRESS>escreveu:

I'm sorry, I don't understand the question. I looked at the tensorflow
feature request, but it didn't help much. I suppose I'm missing the
required context.
Do you mean automatically avoiding some sort of naming conflicts?
If not, could you explain the question in more detail, or are there more
examples?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Technologicat/pydialect/issues/1?email_source=notifications&email_token=AAOOCX4XXUZC36G5LYZ6SWLQBITUDA5CNFSM4IG5XH52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD227D6A#issuecomment-515240440,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOOCX7JJ6G3ZQ752MZYPTDQBITUDANCNFSM4IG5XH5Q
.


Add e2e tests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 2 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 50.946%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/server/options.go
0
1
0.0%

pkg/server/server.go
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
862

Relevant Lines:
1692

💛  - Coveralls

I see there is no easy way of rebuild the server image and get it into the cluster without recreating it. I would like to work on a script to implement that if you mind.

Please work on that, but please do that in different file because at the same time, I'd like to keep github action simple as we can.

Sure I will!


:gift: Add conditional to run correct command for worker

This commit adds a conditional to run the correct command for sidekiq or good_job, when running docker compose up.

@laritakr good point, I'll make a commit to hyku's WIP PR: https://github.com/samvera/hyku/pull/1997


Unable to predict on single channel images

I was able to train the PSNR model for single channel image but it breaks in the prediction part. Any clue?
I'm passing in_channel and out_channel as 1 in the yml file, but it breaks with this log:
      (conv1): Conv2d(64, 64, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1))
      (conv2): Conv2d(64, 64, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1))
      (relu): ReLU(inplace=True)
    )
  )
  (upconv1): Conv2d(64, 256, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1))
  (upconv2): Conv2d(64, 256, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1))
  (pixel_shuffle): PixelShuffle(upscale_factor=2)
  (conv_hr): Conv2d(64, 64, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1))
  (conv_last): Conv2d(64, 1, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1))
  (lrelu): LeakyReLU(negative_slope=0.1, inplace=True)
)
2021-01-26 09:10:03,507 INFO: Loading MSRResNet model from /home/jovyan/BasicSR/experiments/tif_nir_psnr_model/models/net_g_10000.pth.
2021-01-26 09:10:03,521 INFO: Model [SRModel] is created.
2021-01-26 09:10:03,521 INFO: Testing test_tif_nir_psnr_model...
  0%|                                                                                | 0/550 [00:00<?, ?image/s]Traceback (most recent call last):
  File "test.py", line 59, in <module>
    main()
  File "test.py", line 51, in main
    model.validation(
  File "../basicsr/models/base_model.py", line 49, in validation
    self.nondist_validation(dataloader, current_iter, tb_logger,
  File "../basicsr/models/sr_model.py", line 139, in nondist_validation
    self.test()
  File "../basicsr/models/sr_model.py", line 117, in test
    self.output = self.net_g(self.lq)
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 532, in __call__
    result = self.forward(*input, **kwargs)
  File "../basicsr/models/archs/srresnet_arch.py", line 65, in forward
    feat = self.lrelu(self.conv_first(x))
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 532, in __call__
    result = self.forward(*input, **kwargs)
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/conv.py", line 345, in forward
    return self.conv2d_forward(input, self.weight)
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/conv.py", line 341, in conv2d_forward
    return F.conv2d(input, weight, self.bias, self.stride,
RuntimeError: Given groups=1, weight of size 64 1 3 3, expected input[1, 3, 32, 32] to have 1 channels, but got 3 channels instead
  0%|

Same problem, have you solved it?


Uninstalled module disappears from backoffice modules and marketplace section.

Prerequisites

[X] I understand and accept the project's code of conduct.
[X] I have already searched in existing issues and found no previous report of this bug.

Describe the bug and add attachments
If you uninstall a module from the backoffice, it disappears completely even though the directory is still on the server.
Expected behavior
Previously you could find it in the marketplace section to reinstall.
Steps to reproduce

Install a module by uploading a zip
Uninstall the module
Try to find it in the marketplace to reinstall

PrestaShop version(s) where the bug happened
<IP_ADDRESS>
PHP version(s) where the bug happened
7.4
If your bug is related to a module, specify its name and its version
No response
Your company or customer's name goes here (if applicable).
No response

Hello @roja45,
Thanks for your feedback.
We indeed noted that regression.
Working on it to be able to fix it through the MBO module, or the PrestaShop project quickly.

Still an issue here, it's a regression.
Ping @sowbiba 🙏

Hello here, @micka-fdz, @roja45,
With the new direction given to the module ps_mbo, to be a real Marketplace, it will be difficult for us to find room re-adding this feature.
In the meantime, there are multiple solutions for this :

Update to PS 8.x, this is fixed in this version, as they will be shown in the module manager page.
Upload the module zip. PS will install it, and then, you'll be able to find it in the module manager page.
Comment or remove this line in the PS ModuleController.php will allow you to see them in the module manager page. As there will only be security updates on 1.7.8.x, you should be fine.
Install modules via command line php bin/console prestashop module:install/enable/uninstall module_name.

Hope this will help you and lead you to an acceptable compromise.

As discussed yesterday on the Slack OS, we are looking to a better solution to handle this.
Display a tab in the module manager page that will display the list of "Uninstalled modules" seems reasonable for everyone.
In the meantime, please consider using the workarounds specified above, as they are quite easy to implement, and will get you unstuck you faster.

Fixed in v2.4.0


Share managed and unmanaged package

Hi! I would like to ask you a question about sharing a managed an and unmanaged package in the same codebase.
I have two folders src and src_unmanaged, each with its own package.xml and I would like to deploy both to the same package org that has a managed and an unmanaged package.
I am able to deploy the managed package easily by calling cci flow run ci_master --org package, but I can't make the second part work.
So far, I tried replicating the same flow using the unmanaged counterparts like this:
flows:
    deploy_unmanaged_package:
        tasks:
            1:
                task: deploy_pre
            2:
                task: update_dependencies
                options:
                    path: src_unmanaged
                    package: My Unmanaged Package
            3:
                task: uninstall_packaged_incremental
                ignore_failure: true
                options:
                    path: src_unmanaged
                    package: My Unmanaged Package
            4:
                task: update_package_xml
                options:
                    path: src_unmanaged
                    package: My Unmanaged Package
            5:
                task: create_unmanaged_ee_src
                options:
                    path: src_unmanaged
                    package: My Unmanaged Package
            6:
                task: deploy
                options:
                    path: src_unmanaged
                    package: My Unmanaged Package
            7:
                task: revert_unmanaged_ee_src
                options:
                    path: src_unmanaged
                    package: My Unmanaged Package
            8:
                task: uninstall_packaged_incremental
                options:
                    path: src_unmanaged
                    package: My Unmanaged Package
            9:
                task: update_admin_profile

But when I go to the unmanaged package in my org, it is still empty.
Could you please point me in the right direction?
Thanks!

Please have look on the tasks you use for your custom flow.
cci task info deploy for example has only one argument (path)
Also, it seems your flow is missing 'create_package'  which would create a package in your org.
I suggest starting to run two task manually at first: Create package and then deploy your unmanaged package src.

The ci_master flow is definitely specific to uploading the managed metadata to the packaging org for upload as a managed package.
The dev_org flow is really the equivalent to ci_master if you're just dealing with unmanaged metadata.  Since cci now supports passing task options when running a flow, you should be able to now reuse the dev_org flow with these commands:
cci task run update_package_xml -o path src_unmanaged -o package YourPackageName
# Check src_unmanaged/package.xml to make sure the <fullName> element contains YourPackageName
cci flow run dev_org --org packaging -o deploy__path src_unmanaged


medspace UI 3

reduce the size of column no.of kgs/no.of grams
role->HCF->Biomedical waste list
http://medspace.cloudkoza.com/hospital/bio_medical_view

This is not a issues (it depend upon the lable width)


RuntimeError: CUDA out of memory. Tried to allocate 24.00 MiB (GPU 0; 3.95 GiB total capacity; 3.07 GiB already allocated; 20.31 MiB free; 3.09 GiB reserved in total by PyTorch)

Traceback (most recent call last):
  File "run_seq_labeling.py", line 811, in <module>
    main()
  File "run_seq_labeling.py", line 704, in main
    args, train_dataset, model, tokenizer, labels, pad_token_label_id
  File "run_seq_labeling.py", line 230, in train
    scaled_loss.backward()
  File "/home/shreyas/anaconda3/envs/layoutlm/lib/python3.6/site-packages/torch/tensor.py", line 195, in backward
    torch.autograd.backward(self, gradient, retain_graph, create_graph)
  File "/home/shreyas/anaconda3/envs/layoutlm/lib/python3.6/site-packages/torch/autograd/__init__.py", line 99, in backward
    allow_unreachable=True)  # allow_unreachable flag
RuntimeError: CUDA out of memory. Tried to allocate 24.00 MiB (GPU 0; 3.95 GiB total capacity; 3.07 GiB already allocated; 20.31 MiB free; 3.09 GiB reserved in total by PyTorch)

Model I am using LayoutLM
Platform: ubuntu 18.04, GTX 1050, CUDA 10.1, PyTorch 1.4
I followed the exact steps given in the read section, "run_seq_labeling.py" continues giving me the above error. I have tried changing batch size, clearing pytorch cache but none of them work for me.
Please guide me if anyone had encountered such issue before, and have fixed this.
Thank you

@shreyaschim We recommend using GPU with at least 8GB memory and make sure using "--max_seq_length 512"

@shreyaschim We recommend using GPU with at least 8GB memory and make sure using "--max_seq_length 512"
thanks for reply.


Add DOI prettification and allow http:// protocol cleanup

Please see #1 for the details. In this pull I also slightly modified the first gsub to allow either https:// or http:// protocols. I tested using pandoc:
▶ pandoc -L pretty-urls.lua -t html

Here is a http link: <http://test.org>.

Here is a https link: <https://test.org>

Here is a DOI: <http://dx.doi.org/10.1002/icd.2359>

DOI without the dx: <http://doi.org/10.1002/icd.2359>

Here is a http link: test.org.

Here is a https link: test.org

Here is a DOI: DOI:10.1002/icd.2359

Without the dx: DOI:10.1002/icd.2359

Thanks. The omission of http:// links was intentional, because they are rare nowadays and should stand out due to being insecure.
I seemed to vaguely remember something about doi: to being lowercase, and indeed, the official recommendation is to use lowercase: https://www.doi.org/doi_handbook/2_Numbering.html#2.6.1
It's probably easier for both of us if I merge this now and make the changes after that.

I've made the changes. URLs with http:// get an extra marker to signal that they are unsafe. I should probably make this configurable though.
archive.org 🔓


Archiving minirepos

Changes introduced with this PR
Archiving the repositories merged into libcontainerssh.

By contributing to this repository, I agree to the contribution guidelines.

Can we eventually delete them so the org clutter goes away?


WIP: Respond to breaking changes

https://developer.github.com/changes/2018-08-16-changes-to-the-checks-rest-api/

Pull Request Test Coverage Report for Build 45

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 32:
0.0%

Covered Lines:
54

Relevant Lines:
54

💛  - Coveralls


Exit code: 50

I have a customer whose install of the AAD extension failed with exit code: 50
Install failed for plugin (name: Microsoft.Azure.ActiveDirectory.AADLoginForWindows, version <IP_ADDRESS>) with exception Command C:\Packages\Plugins\Microsoft.Azure.ActiveDirectory.AADLoginForWindows\<IP_ADDRESS>\AADLoginForWindowsHandler.exe of Microsoft.Azure.ActiveDirectory.AADLoginForWindows has exited with Exit code: 50
Turns out they were running Windows Server 2016 Datacenter which is currently not supported. I believe this is the same as error code 51 which is shown in this document.

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

ID: c946fe0f-e18d-2529-2fbf-bce04759fbca
Version Independent ID: 885a61d1-6096-5aa0-fe8d-f1ec8d55e542
Content: Sign in to Windows virtual machine in Azure using Azure Active Directory (Preview)
Content Source: articles/active-directory/devices/howto-vm-sign-in-azure-ad-windows.md
Service: active-directory
Sub-service: devices
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@Jhines09 Thanks for reaching out. We will investigate and update the thread accordingly.

@Jhines09  Yes that would mean the same unsupported thing.  DO you see any other error message in event logs.
You can find them here :
Open Event Viewer.
Expand Windows Logs, and then expand Application.
In the Actions pane, select Filter Current Log.
In the Event sources box, select the Directory Synchronization check box.
Select OK.

Hi, Thanks for the reply!
I can try to get that,however the case is already closed as we already identified that the Customer was using Windows Server 2016 Datacenter VM which is not supported, so we did not proceed to collect the event logs.
Best Regards,
Jeffrey
From: VipulSparshMSFT<EMAIL_ADDRESS>Sent: Thursday, August 13, 2020 7:04 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Jeffrey Hines (MINDTREE LIMITED<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Exit code: 50 (#60658)
@Jhines09https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJhines09&data=02|01|v-jehine%40microsoft.com|c206e91735bc402e0b1f08d83f91b861|72f988bf86f141af91ab2d7cd011db47|1|0|637329242376130004&sdata=Lg6ZKjbKfuqph69Jb5bI43TTV4kDVO31aN%2FtHD1xvN0%3D&reserved=0 Yes that would mean the same unsupported thing. DO you see any other error message in event logs.
You can find them here :
Open Event Viewer.
Expand Windows Logs, and then expand Application.
In the Actions pane, select Filter Current Log.
In the Event sources box, select the Directory Synchronization check box.
Select OK.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F60658%23issuecomment-673498246&data=02|01|v-jehine%40microsoft.com|c206e91735bc402e0b1f08d83f91b861|72f988bf86f141af91ab2d7cd011db47|1|0|637329242376130004&sdata=jvwFWnWmv95w6FcGwtqk9QOUpBMm%2BtafKei1dNnf6eo%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAONHPTT75UHJGSCDKD6GWKLSAPXEZANCNFSM4P3NGC7Q&data=02|01|v-jehine%40microsoft.com|c206e91735bc402e0b1f08d83f91b861|72f988bf86f141af91ab2d7cd011db47|1|0|637329242376139960&sdata=msceIiJTILUM7IVBo4FfkswK6s5rRxDJVI0beRJhanc%3D&reserved=0.

@Jhines09  Sure no worries, if you get a chance to collect that, it might yield some additional data.  but moreover, it does signify unsupported part. We will close this now, please tag me in your comment if you have any questions.


App crashes when server returns HTTP 502 response

I use a reverse proxy in front of my Home Assistant instance. When Home Assistant is not running, the reverse proxy returns a 502 status code. This is making the Home Assistant app crash. Instead, the app should not crash but simply show an error message.

This should be fix.
I also added Crashlytics to monitor crash.


How do you update individual charts?

I see https://github.com/emn178/angular2-chartjs/issues/2 where the following is utilized to update chart.
import { ChartComponent } from 'angular2-chartjs';

//...

@ViewChild(ChartComponent) chart: ChartComponent;

//...

this.chart.chart.update();

However, does calling this.chart.chart.update() all charts in the component? How do one target a single chart?
@emn178

You can use template reference variables.
<chart #chart1 ></chart>

@ViewChild('chart1') chart1: ChartComponent;

@emn178  I created a pull request removing else from ngOnChanges which auto updates the chart on update to [data] attribute.

Please follow this and it will work!

Import these ones...
import { Component, OnInit, ViewChild } from '@angular/core';
import { BaseChartDirective } from 'ng2-charts';

Into the export class (in your ts file) add this...
@ViewChild(BaseChartDirective) chart: BaseChartDirective;

then, first you should initialize the object that contains the data and label info.
1, 1, 1 is an example, you can use 0 if want...
public barChartData: any[] = [{
    data: [1,1,1,1,1,1,1],
    label: 'this is an example'
}];

in your ngOnInit method add a setTimeout function, or use the call to the backend by http;
setTimeout(() => {
this.chart.chart.data.datasets[0].data = [55, 355, 35, 50, 50, 60, 10]
this.chart.chart.update()
}, 2000);

in your html file, make sure to add baseChart to the canvas tag

just to explore a little more, in the ngOnInit method, you can perform a console.log of (this.chart.chart) and you will find more information about the object...

Hope it helps!


Pod restarts

Proposed changes
Pod restarts

Types of changes

Bugfix (non-breaking change which fixes an issue)

Linked Issues
https://github.com/rancher/rancher/issues/17891

Further comments

Updated


need clarification

if i modify registry key like
HKLM\SOFTWARE\Microsoft\Security Center\Feature DisableAvCheck (DWORD) = 1
windows security stops checking of third party antivirus also ?

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

ID: b3d6356b-87ff-8c3a-56ef-1270a823d3ab
Version Independent ID: 4d6ae878-66cf-0e10-bdf3-bdd6054cbc9d
Content: What's new in Windows 10, version 1809
Content Source: windows/whats-new/whats-new-windows-10-version-1809.md
Product: w10
GitHub Login: @dawnwood
Microsoft Alias: trudyha

@officedocsbot assign @mypil

@RAJU2529 - Thank you for submitting feedback.
I will get this issue over to the Win10 ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

Dear @RAJU2529 ,
yes, it will, according to this article
Dear @SusanBradley , it was fixed in commit https://github.com/MicrosoftDocs/windows-itpro-docs/commit/a79f88000ef735a643cae9c4c8d787aab346e410#diff-a9ee0a47fba86a9999f1da444bbff8e3
Dear @rswgnu ,
there is no such fix in the list of changes for this release. However, file system performance increase is announced in new WSL 2
Thank you

Thanks sir, now I understood real reason after reading the article mentioned in your reply.
Thanks for clearing my doubt. Thanks a lot sir. You can close this issue

@officedocsbot close


when only part of the data set is trained, loss reaches 0

Hello and thanks for publishing the code of your research. After the pre-training, I had some questions.
iter     0 | loss 3.823 | ppl 45.743
Train   1: iter  1000 | loss 0.003 | ppl 1.003
Train   1: iter  2000 | loss 0.001 | ppl 1.001
I want to do the pre-training by myself, but during the training, the loss value drops rapidly.I don't know if there is something wrong with training corpus(chembl_25. csv) or if it is normal.The training corpus(chembl_25. csv) refers to your prepare_data.ipynb script in expriments folder.
Part of  training corpus is shown in the following figure:

Is the file chembl_25. csv generated in the same way as the file chembl_24. csv?

It does reach the zero training loss when the model is large enough, but I don't really remember at which iteration it reaches. So my advice is to keep training for long enough and go ahead to the downstream tasks.
Yes, chembl_25. csv is generated in the same way as chembl_24. csv

thanks for your reply


Switch back to default flux type option for interactive runs

Switch back to CombinedStateFluxes as the default option for interactive debug runs. This is faster and more stable for the current coarse benchmarks.

[x] I have read and checked the items on the review checklist.

bors r+


How to hot reload on non .js files?

I'm making a SSR app with Elm and Razzle, everything is fine, I just need to make work HMR with all .elm files.
Actually I can make hot reload to specific files, like this:
if (module.hot) {
  module.hot.accept();
  module.hot.accept('./Main', () => {
    const nextElm = require('./Main').Main;
    nextElm.embed( document.getElementById( "root" ) );
  })
}

If I change the code of Main.elm it hot reload, but if I edit any other child elm file it dosen't work. Is really annoying add every file (much more if the project is big) so I wan't to know if is possible reload on all .elm files
Greetings

Sorry if my English writing skills are poor, I still learning ^^'

Solved, if you're curious how to make work Elm and Razzle take a look to this repo

Want to submit a PR to make this an official Razzle example?

Sure, I'll do it tonight, just a question
I'll add it on the example folder or the link to the repo in Readme?

Examples folder so people can install it with the CLI. I will add a link to the initial repo in the readme in the example too

@jaredpalmer I just created the pull request in the example folder ^^


Add frame for displaying health of serial com

Need a frame with
kB/s of received data
amount of characters/frames in each reception buffer

Done with an alive signal


How to get access to global glEventHub, from React container NOT inserted into a golden-layout

For React components inserted into glLayout myLayout, they automatically receive glEventHub at this.props.glEventHub.
But, I have a React component not inserted into golden layout myLayout. I want this component to be able to emit / receive events from glEventHub.
How do I get access to glEventHub from outside golden layout components?
(ie something like:  window.glEventHub  or   myLayout.glEventHub )
Thanks!

I think you can always export the glEventHub object as you want :)
The simplest way is, as in your example, to attach it to window when you init GoldenLayout:
import GoldenLayout from 'golden-layout';

class App extends React.PureComponent {
  constructor(props) {
    super(props);
    this.setNode = this.setNode.bind(this);
  }
  componentDidMount() {
    const instance = new Provider(config, this.node);
    /* register components or bind events to your new instance here */
    window.glEventHub = instance.glEventHub;
    instance.init();
  }
  setNode(node) {
    this.node = node;
  }
  render() {
    return
;
  }
}

Now you can access to window.glEventHub anywhere

Or, you can also export it as a part of your module. For example we can wrap the GoldenLayout inside our "MyLayout" component, so it looks much "React"-ly:
import GoldenLayout from 'golden-layout';

const ref = { glEventHub: null };

export const getEventHub = () => ref.glEventHub;

export default class MyLayout extends React.PureComponent {
  constructor(props) {
    super(props);
    this.setNode = this.setNode.bind(this);
  }
  componentDidMount() {
    const instance = new Provider(config, this.node);
    /* register components or bind events to your new instance here */

    ref.glEventHub = instance.glEventHub;
    instance.init();
  }
  setNode(node) {
    this.node = node;
  }
  render() {
    return
;
  }
}

Now you can access glEventHub by importing that module
import { getEventHub } from './MyLayout';

const eventHub = getEventHub();

Note that this must be changed slightly if you have multiple instances of GoldenLayout. However I think you only have one in your case.


Deploy proxy plugin as part of toolchain for development clusters

As per this doc a proxy plugin is needed to access the tekton-results API.
Adding one so it works out of the box on development clusters.

/ok-to-test

/retest

@jrichter1 I see you are adding proxy-plugin for tekton-results here. Doesn't this affect the results test in e2e https://github.com/redhat-appstudio/e2e-tests/blob/main/tests/build/build_templates.go#L155-L172 . In case it affects the test would you mind updating those test as well accordingly.

@jrichter1 I see you are adding proxy-plugin for tekton-results here. Doesn't this affect the results test in e2e https://github.com/redhat-appstudio/e2e-tests/blob/main/tests/build/build_templates.go#L155-L172 . In case it affects the test would you mind updating those test as well accordingly.

I am holding it until we test it with the pipeline labeled e2e-test as this change is directly related to pipeline. Since on Infra-deployment AFAIK we don't test with pipeline label, ci might pass. My other concerns is it might break the other ci for ex periodic ci, after merge if we don't test the related test.
/hold

closing in favour of https://github.com/openshift/release/pull/41724


iOS support ability to launch Google Maps?

Currently, if you only have Google Maps installed, it will ask you to restore Apple Maps.  It would be nice to have the option or ability to have it also launch Google Maps or use whatever other map app is installed.  We have project requirements to have this ability because Apple Maps is not seen as reliable.

You will need to launch them via URL scheme: https://developers.google.com/maps/documentation/ios-sdk/urlscheme


Error installing SSHJ plugin

Windows 10 x64, DBeaver Community 5.1.0. I'm trying to install the SSHJ plugin from https://dbeaver.io/update/sshj/latest/, but I'm receiving the error below. I tried while running DBeaver as admin, same error.
An error occurred during the org.eclipse.equinox.internal.p2.engine.phases.CheckTrust phase.
    session context was:(profile=DefaultProfile, phase=org.eclipse.equinox.internal.p2.engine.phases.CheckTrust, operand=, action=).
    Error reading signed content.
    Exception in opening zip file: C:\Program Files\DBeaver\plugins\org.jkiss.bundle.bouncycastle_1.59.0

Check this - https://github.com/dbeaver/dbeaver/wiki/Optional-extensions
IMPORTANT: installation won't work if DBeaver is installed in write-protected folder (like Program Files, /Applications, /usr/). To proceed you need to run DBeaver from some non write-protected folder or run it as Administrator/root.

Yes, I put in my issue that the problem still occurs when running as administrator.

Hmm, can't reproduce that. Just installed this on fresh DBeaver copy.
Perhaps something went wrong during sshj plugin download.
Please post full error stacktrace.
Also - did you use DBeaver installer or you have some other preinstalled Java?
Also you could try to reinstall DBeaver and install SSHJ again.

Also this looks similar to #3624

I tried again today and it worked. No idea why it didn't work previously. I'll close this, sorry, and thank you!


add good rust documentation

We need to show good documentation for https://docs.rs/embed_anything/0.3.4/embed_anything/

I've been playing around with the Rust API for a bit now, I'd be happy to take this!
Would you prefer multiple smaller PRs (probably for each file?), or one big one for everything?

Since it's not too big a project right now, everything at once is fine.

I'm seeing a mix of embeder and embedder - is there a distinction to be made here, or should these be made consistent?

Ya. It has been a typo that has propagated for a while. "embedder" should
be the right word.
On Tue, 17 Dec 2024, 11:53 am Jack Boswell, @.***>
wrote:

I'm seeing a mix of embeder and embedder - is there a distinction to be
made here, or should these be made consistent?
—
Reply to this email directly, view it on GitHub
https://github.com/StarlightSearch/EmbedAnything/issues/88#issuecomment-2548135026,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOS3NIA6ZKCJSH2PIQ3S5732F77BBAVCNFSM6AAAAABRQE3WVOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBYGEZTKMBSGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


how to divide the testing set into Indoor and outdoor scenes

The paper report the metrics of indoor and outdoor scenes， could you provide the lists of indoor and outdoor scenes in the testing set?

Sure, please find at this [Link] .npy arrays. It is an array of numbers [1,76] where the number indicate the image order when the images are sorted by names.

Hi Abuolaim,
Thank you for your great work.
It seems the link you provided is invalid now. Would you please update the link?