Source images not showing up in crate, thwomp & first person examples

Crate, Thwomp and First Person all appear to have errors with their source images.

http://elm-lang.org/examples/crate
http://elm-lang.org/examples/thwomp
http://elm-lang.org/examples/first-person

It happens in Firefox and Chrome. I took screenshots of what appears in the console.

Same issue with http://elm-lang.org/examples/resize-paint
http://elm-lang.org/paint.jpg

Thanks for the report @alexiamcdonald! I think this is a recent development, not sure the root yet.
@raine, this one is just a bad link, should be fixed now. Thanks for pointing it out!

@evancz yeah, it's definitely a recent development.
The console is actually helping me find the problem now for some reason (yay!)

http://elm-lang.org/examples/thwomp/result

Console says the problem for thwomp example is from line 12439 in result:
req.send();

Also, I was reading up on similar console errors of this type on stackoverflow somewhere and it said to make sure you have your adblocker off because sometimes it can interfere finding the problem. Not sure how much of that is true in this case but just a good idea just in case.

I wonder if it is related to #304?
If you take the code located at the following address in chrome (view-source:http://elm-lang.org/examples/thwomp/result) and make the modifications from #304 and run it locally, does stuff still mess up?
I have been holding off on doing a new release of core because it is a major version bump even though there are no real breaking changes. It's more that 0.15.1 made it way easier to catch errors in docs and exports, so we found a bunch of hidden things escaping. In any case, this might be a good kick to do it anyway.

It could also be from a new release of elm-webgl. I recently updated all the packages used on elm-lang.org, so if elm-webgl has been bumped perhaps it introduced the issue.

I'm clearly getting a timeout in Safari too. The WebGL code responsible is clearly following spec recommendation. When I manually enter the exact logic in the developer console it loads successfully. On the next refresh it's even cached.

I failed to notice it's in fact using a different implementation based on XHR in Elm Try.

I failed to notice Elm Try is in fact using a different implementation based on XHR. The new code is more flexible but sets a timeout of 30ms. I suspect it's simply too quick. @johnpmayer?

@dnalot pointed out the specific location here. The code causing issues is this:
function loadTexture(url) {
    return Task.asyncFunction(function(callback) {
      var req = new XMLHttpRequest;
      req.addEventListener('error', function() {
        return callback(Task.fail({ ctor: 'NetworkError' }));
      });
      req.addEventListener('timeout', function() {
        return callback(Task.fail({ ctor: 'Timeout' }));
      });
      req.addEventListener('load', function() {
        return callback(Task.succeed(toTexture(req)));
      });
      req.open('get', url, true);
      req.timeout = 30;
      req.responseType = 'blob';
      req.send();
    });
  }

It looks like the relevant elm-webgl code has been updated to not have that timeout anymore. Probably just need to update versions.

Alright, all the examples are fixed for me!
Thanks @alexiamcdonald for the report! Thanks @dnalot for finding the exact files and line numbers causing things, that made it super easy to figure out!

The XHR code looked newer than the Image code. The timeout is probably even a good idea.  A full second might ought to be permitted. In any case CC @johnpmayer in case XHR gets re-published.


Fix possible panic during resourceCloudflareSpectrumApplicationRead

Fixes a panic in resourceCloudflareSpectrumApplicationRead where application.EdgeIPs.Connectivity could be nil. This is impacting our team from upgrading to the latest provider version. We were previously on: v3.4.0.
Relavent logs:
Stack trace from the terraform-provider-cloudflare_v3.14.0 plugin:

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0xba37c0]

goroutine 279 [running]:
github.com/cloudflare/terraform-provider-cloudflare/cloudflare.resourceCloudflareSpectrumApplicationRead(0xc00072e080, {0xdada00?, 0xc0004a8480})
        github.com/cloudflare/terraform-provider-cloudflare/cloudflare/resource_cloudflare_spectrum_application.go:116 +0x10e0
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).read(0xf05178?, {0xf05178?, 0xc00043f1d0?}, 0xd?, {0xdada00?, 0xc0004a8480?})
        github.com/hashicorp/terraform-plugin-sdk/v2@v2.14.0/helper/schema/resource.go:712 +0x178
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).RefreshWithoutUpgrade(0xc000457340, {0xf05178, 0xc00043f1d0}, 0xc0004531e0, {0xdada00, 0xc0004a8480})
        github.com/hashicorp/terraform-plugin-sdk/v2@v2.14.0/helper/schema/resource.go:1015 +0x585
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*GRPCProviderServer).ReadResource(0xc000187230, {0xf050d0?, 0xc00071b840?}, 0xc00071b8c0)
        github.com/hashicorp/terraform-plugin-sdk/v2@v2.14.0/helper/schema/grpc_provider.go:613 +0x497
github.com/hashicorp/terraform-plugin-go/tfprotov5/tf5server.(*server).ReadResource(0xc000388a00, {0xf05178?, 0xc0004030e0?}, 0xc000791aa0)
        github.com/hashicorp/terraform-plugin-go@v0.9.0/tfprotov5/tf5server/server.go:746 +0x438
github.com/hashicorp/terraform-plugin-go/tfprotov5/internal/tfplugin5._Provider_ReadResource_Handler({0xd80140?, 0xc000388a00}, {0xf05178, 0xc0004030e0}, 0xc000791a40, 0x0)
        github.com/hashicorp/terraform-plugin-go@v0.9.0/tfprotov5/internal/tfplugin5/tfplugin5_grpc.pb.go:349 +0x170
google.golang.org/grpc.(*Server).processUnaryRPC(0xc0001ae8c0, {0xf07c00, 0xc000374d00}, 0xc0008959e0, 0xc000460f60, 0x14002f0, 0x0)
        google.golang.org/grpc@v1.45.0/server.go:1282 +0xccf
google.golang.org/grpc.(*Server).handleStream(0xc0001ae8c0, {0xf07c00, 0xc000374d00}, 0xc0008959e0, 0x0)
        google.golang.org/grpc@v1.45.0/server.go:1619 +0xa1b
google.golang.org/grpc.(*Server).serveStreams.func1.2()
        google.golang.org/grpc@v1.45.0/server.go:921 +0x98
created by google.golang.org/grpc.(*Server).serveStreams.func1
        google.golang.org/grpc@v1.45.0/server.go:919 +0x28a

Error: The terraform-provider-cloudflare_v3.14.0 plugin crashed!

It appears this regression was introduced here: https://github.com/cloudflare/terraform-provider-cloudflare/pull/1515/files#diff-98e27307f62b90eef57324e5dfb984be43e94aa8fcec759596265dd981f168efR116

@joshuamsager can you please add a test case exercising this new code path for coverage?

@joshuamsager can you please add a test case exercising this new code path for coverage?

Hey @jacobbednarz, when trying to come up with a test locally... I receive errors from the API server:
2022/05/09 11:20:03 [DEBUG] Cloudflare API Request Details:
---[ REQUEST ]---------------------------------------
POST /client/v4/zones/<REDACTED>/spectrum/apps HTTP/1.1
Host: api.cloudflare.com
User-Agent: terraform/1.1.9 terraform-plugin-sdk/2.10.1 terraform-provider-cloudflare/dev
Content-Length: 271
Content-Type: application/json
X-Auth-Email: <REDACTED>
X-Auth-Key: <REDACTED>
Accept-Encoding: gzip

{
 "dns": {
  "type": "CNAME",
  "name": "qvgfshhqbg.cdn2.rhopifycloud.com"
 },
 "origin_direct": [
  "tcp://<IP_ADDRESS>:23"
 ],
 "protocol": "tcp/22",
 "traffic_type": "direct",
 "tls": "off",
 "proxy_protocol": "off",
 "origin_port": 22,
 "edge_ips": {
  "type": "static",
  "ips": [
   "<IP_ADDRESS>"
  ]
 },
 "ip_firewall": true
}
-----------------------------------------------------
2022/05/09 11:20:07 [DEBUG] Cloudflare API Response Details:
---[ RESPONSE ]--------------------------------------
HTTP/2.0 400 Bad Request
Cf-Cache-Status: DYNAMIC
Cf-Ray: 708b6abfeb1586ae-ORD
Content-Type: application/json; charset=UTF-8
Date: Mon, 09 May 2022 15:20:07 GMT
Expect-Ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
Server: cloudflare
Set-Cookie: __cflb=0H28vgHxwvgAQtjUGU4vq74ZFe3sNVUZXW2QpHxQ52u; SameSite=Lax; path=/; expires=Mon, 09-May-22 17:50:08 GMT; HttpOnly
Set-Cookie: __cfruid=2e62042a3fc5e51dd91e036ed3b734de17760aa8-1652109607; path=/; domain=.api.cloudflare.com; HttpOnly; Secure; SameSite=None
Vary: Accept-Encoding
X-Envoy-Upstream-Service-Time: 67

{
  "result": null,
  "success": false,
  "errors": [
    {
      "code": 10012,
      "message": "json: unknown field \"traffic_type\""
    }
  ],
  "messages": []
}

Which happens when I run any of the tests in: terraform-provider-cloudflare/cloudflare/resource_cloudflare_spectrum_application_test.go. When I test this API call locally using Postman, the API succeeds only after removing all of the optional parameters from the JSON body.
In any case, I added the test case I believe to test the new codepath in: d666c6c1f9d4775e6d1e9ce550925f9f78d0cc3c

thanks, I'll pull this locally and have a look tomorrow.

i managed to get to the bottom of this one, and it was a combination of account entitlements and tighter validation on some endpoints for payloads.
i did have to mark your new test as pending in our acceptance test suite for now as it requires BYO IP setup and i don't yet have that (confirmed it was fixed in another account with BYO IP enabled though).
TF_ACC=1 go test $(go list ./...) -v -run "^TestAccCloudflareSpectrumApplication_" -count 1 -parallel 1 -timeout 120m -parallel 1
?   	github.com/cloudflare/terraform-provider-cloudflare	[no test files]
=== RUN   TestAccCloudflareSpectrumApplication_Import
=== PAUSE TestAccCloudflareSpectrumApplication_Import
=== RUN   TestAccCloudflareSpectrumApplication_Basic
--- PASS: TestAccCloudflareSpectrumApplication_Basic (29.45s)
=== RUN   TestAccCloudflareSpectrumApplication_OriginDNS
--- PASS: TestAccCloudflareSpectrumApplication_OriginDNS (13.53s)
=== RUN   TestAccCloudflareSpectrumApplication_OriginPortRange
--- PASS: TestAccCloudflareSpectrumApplication_OriginPortRange (14.28s)
=== RUN   TestAccCloudflareSpectrumApplication_Update
--- PASS: TestAccCloudflareSpectrumApplication_Update (19.57s)
=== RUN   TestAccCloudflareSpectrumApplication_EdgeIPConnectivity
--- PASS: TestAccCloudflareSpectrumApplication_EdgeIPConnectivity (10.48s)
=== RUN   TestAccCloudflareSpectrumApplication_EdgeIPsWithoutConnectivity
    resource_cloudflare_spectrum_application_test.go:228: pending getting BYO IP assigned to the acceptance testing account
--- SKIP: TestAccCloudflareSpectrumApplication_EdgeIPsWithoutConnectivity (0.00s)
=== CONT  TestAccCloudflareSpectrumApplication_Import
--- PASS: TestAccCloudflareSpectrumApplication_Import (14.03s)
PASS
ok  	github.com/cloudflare/terraform-provider-cloudflare/cloudflare	101.917s
?   	github.com/cloudflare/terraform-provider-cloudflare/tools/cmd/changelog-check	[no test files]
?   	github.com/cloudflare/terraform-provider-cloudflare/tools/cmd/maintainer-only-file-check	[no test files]
?   	github.com/cloudflare/terraform-provider-cloudflare/tools/cmd/tf-log-check	[no test files]
?   	github.com/cloudflare/terraform-provider-cloudflare/version	[no test files]

thanks for getting this one over the line @joshuamsager, you rock!


 It'd be nice to have the same methods on the gzip DecoderReader as well.
Relatedly, it would also be really nice to have traits that all the decoders and encoders here (and in your bzip2-rs repo, but not sure how easy that would be) implement, so that users could (statically or dynamically) switch out compression engines :)

Sounds good to me! I may not have time to implement this soon but it should be a relatively easy PR!

Any thoughts on exposing a common interface? (Right now I'm doing it within my own project.)
Still working through some internal paperwork required to contribute :)

I'd personally prefer to hold off on that for now, but if it ends up being popular we can certainly add it!
Let me know if you need any help though!

I believe this was effectively done in #66, so closing


Сover code with tests

Hi @OrkoHunter
I think it's reasonable to write some unit test and add them to CI because now it's like minefield.

True! I verify every patch with manually creating a test PR on OrkoHunter/test-pep8speaks and sending the payload (copying it from the github developer settings) on the local server through a jupyter notebook to verify if everything worked properly.
This sure is a minefield. :D

Some of the core features are now being tested on CI! #100


Add tests for pipeline binaries, host image copy and shader objects

CI gfxreconstruct build # 5297 running.

CI gfxreconstruct build # 5297 failed.


mbed-cli should list compatible pip version as a dependency

Through the hardship we have learned about https://github.com/ARMmbed/mbed-cli/blob/66f734d1cfb889a1f09e577d61e53c3006e3fcaf/README.md#mbed-cli-and-pip-compatibility, however for the better user experience, I think, it would be more useful, if its compatible version subset was included to install_requires in the setup.py. Then, possibly, pip itself would warn us, if we try to break the compatibility.

Hmm... So on the one hand, I completely agree, but with release 1.6.0, the pip issue has been resolved. And as far as I know, the fix is backwards compatible across older pip versions.
@screamerbg Thoughts on adding pip version to setup.py?

If definately should be improved. There is no guarantee that it would not break again later, or break a bit differently in future.

@teetak01 Agreed and following that thought, all requirements.txt in Mbed OS should be updated to limit to major releases of Python dependencies they have. As you said, there's no guarantee that they won't break in the future.
cc @cmonr @theotherjimmy

Issue header has missing or incorrectly filled in fields! Please fix.


panic in VSCode terminal

Hi I wanted to follow the tutorial and installed miniredis server
cargo install mini-redis
then I ran the command mini-redis-server which resulted in panic.
The command worked fine in a normal terminal though.
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Syntax(
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
regex parse error:
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 1: (?x)
 2:                 ^(?P<global_level>(?i:trace|debug|info|warn|error|off|[0-5]))$ |
                                          ^
 3:                  #                 ^^^.
 4:                  #                     `note: we match log level names case-insensitively
 5:                 ^
 6:                 (?: # target name or span name
 7:                     (?P<target>[\w:-]+)|(?P<span>\[[^\]]*\])
 8:                 ){1,2}
 9:                 (?: # level or nothing
10:                     =(?P<level>(?i:trace|debug|info|warn|error|off|[0-5]))?
11:                      #          ^^^.
12:                      #              `note: we match log level names case-insensitively
13:                 )?
14:                 $
15:
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
error: Unicode-aware case insensitivity matching is not available (make sure the unicode-case feature is enabled)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
)', /Users/user/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tracing-subscriber-0.2.25/src/filter/env/directive.rs:167:14
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

feel free to close this as won't fix so this issue at least serves for informational purpose :)

This isn't specific to the vscode terminal. I think this is https://github.com/tokio-rs/tracing/issues/2573
The root cause seems to be a semver bug in an outdated dependency but you can fix this by installing the exact versions specified in the lockfile via
> cargo install mini-redis --locked

cargo install mini-redis --locked --force if it's already installed).
The tutorial should probably change the suggested command to that.


python312Packages.python-calamine: Fix Darwin build

Description of changes
Add missing dependency required on Darwin systems.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 342382 run on aarch64-darwin 1

22 packages built:

python311Packages.layoutparser
python311Packages.layoutparser.dist
python311Packages.pandantic
python311Packages.pandantic.dist
python311Packages.pandas-stubs
python311Packages.pandas-stubs.dist
python311Packages.pdfplumber
python311Packages.pdfplumber.dist
python311Packages.python-calamine
python311Packages.python-calamine.dist
python312Packages.layoutparser
python312Packages.layoutparser.dist
python312Packages.pandantic
python312Packages.pandantic.dist
python312Packages.pandas-stubs
python312Packages.pandas-stubs.dist
python312Packages.pdfplumber
python312Packages.pdfplumber.dist
python312Packages.python-calamine
python312Packages.python-calamine.dist
tocpdf
tocpdf.dist

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

22 packages built:

python311Packages.layoutparser
python311Packages.layoutparser.dist
python311Packages.pandantic
python311Packages.pandantic.dist
python311Packages.pandas-stubs
python311Packages.pandas-stubs.dist
python311Packages.pdfplumber
python311Packages.pdfplumber.dist
python311Packages.python-calamine
python311Packages.python-calamine.dist
python312Packages.layoutparser
python312Packages.layoutparser.dist
python312Packages.pandantic
python312Packages.pandantic.dist
python312Packages.pandas-stubs
python312Packages.pandas-stubs.dist
python312Packages.pdfplumber
python312Packages.pdfplumber.dist
python312Packages.python-calamine
python312Packages.python-calamine.dist
tocpdf
tocpdf.dist

Closing since since another PR added the missing dependency: #342563


made PageUser non cascading

Description
A PageUser gets created for every user that is created, with a created_by to the original user.
On deleting the original user all PageUsers would be deleted by cascade.
And thus also the user connected to there PageUsers as well.
This sets created_by to on_delete=models.SET_NULL stopping this problem

Related resources

#6776

Checklist

[x] I have opened this pull request against develop
[x] I have updated the CHANGELOG.rst
I can make a test if that is required
[] I have added or modified the tests when changing logic

@FinalAngel i saw there was an unrelated commit in the last PR, this is just the commit that changes the delete beheviour

Coverage remained the same at 78.215% when pulling 6e9ea589bb07835d9bdb6d55cc75d045c72e6eb9 on ThunderbyteAI:develop into e428cb356d14591baedc93bfd7dda2b4bc5645e9 on divio:develop.

thank you @robinvd, would you also be able to rebase develop and add a changelog entry? /cc @Aiky30 @victor-yunenko

@FinalAngel is this changelog entry discriptive enough?

thank you @robinvd, that seems fine. Let's wait for some feedback from @Aiky30

Created by is a required field and the only breaking change to the api that v4 introduces. I don't think this should be set to null and there may be other logic in the current codebase that requires created by to be populated.

Hi @robinvd, did you see the above feedback on this change?
I'd agree that setting to models.PROTECT instead of SET_NULL makes sense.

Closing this as I've raised the recommendation in #7206


Explore ways to link Datasets with projects/studies/samples

https://github.com/NFDI4Chem/nmrxiv/pull/695

@NRayya Update please?

Pending on enabling datacite schema update


[Fix #12138] Fix a false positive for Layout/LineContinuationLeadingSpace

Fixes rubocop#12138.
This PR fixes a false positive for Layout/LineContinuationLeadingSpace when a backslash is part of a multiline string literal.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

@koic Thanks for the review! I've amended the commit.

Thanks!


strangely, Laravel + franken PHP not autoreload even has been implemented docker mounted volume

So the mounted volume implementation works well using the compose.yaml code below
# compose.yaml

services:
  laravel:
    container_name: laravel
    # uncomment the following line if you want to use a custom Dockerfile
    build: .
    # uncomment the following line if you want to run this in a production environment
    # restart: always
    ports:
      - "80:80" # HTTP
      - "443:443" # HTTPS
      - "443:443/udp" # HTTP/3
    volumes:
      - ./:/app
      - caddy_data:/data
      - caddy_config:/config
    depends_on:
      - database
    # comment the following line in production, it allows to have nice human-readable logs in dev
    tty: true

# Volumes needed for Caddy certificates and configuration
volumes:
  caddy_data:
  caddy_config:

but strangely, when I make changes to code, the app is not reloaded in browser, I have to make manual reload to view the changes. actually when I make changes in the code, because I have implemented docker mounted volume, the changes is already maded, but on browser I still need to manually refresh it to view the change.  is it possible to make it automatic (view changes without manual reload)?

I'm not sure this is a FrankenPHP question, but more of a PHP question.
That being said, it would be really neat if someone made a composer library that listened for file changes, announced it to mecure, then a simple js script that subscribed to mecure and reloaded the page when files were changed.

Wait, are you expecting it to reload the page for you? Is that a Laravel thing? I've never seen that in 20 years of programming PHP, and I've minimal experience with modern Laravel.
Ah, I see. It appears this only works in dev mode (via artisan serve) and requires some configuration on your end. This is a pretty Laravel-specific question, and not a FrankenPHP question. I'm going to close this for now, since there isn't much we can help with. Feel free to reopen if this is a FrankenPHP issue.

If you use Octane, the php artisan octane:start --watch will do the trick.


Ubuntu -> Ubuntu Linux

:-1 : "Ubuntu Linux" may be used sometime, but it is not commonly used on https://ubuntu.com/, https://en.wikipedia.org/wiki/Ubuntu.
If the goal is to list all linux distribution easily, it would be best to use tags (not yet implement, #2225).

If the goal is to list all linux distribution easily, it would be best to use tags (not yet implement, https://github.com/endoflife-date/endoflife.date/issues/2225).

Yes it is :smile_cat: :heavy_heart_exclamation:

If the goal is to list all linux distribution easily, it would be best to use tags (not yet implement, https://github.com/endoflife-date/endoflife.date/issues/2225).

Yes it is :smile_cat: :heavy_heart_exclamation:


Query builder grafana beta integration

PR to start the packaging job

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


Fixed the scale of bcmod

Fixes the issue, that a previous bcscale screws with the precision of the bcmod function.

@jschaedl Would it be possible to get this fix in ASAP?
I've been having strange issues with many of my tests that deal with the IBAN validation. For some odd reason, with PHPUnit running a test suite / directory, the IBAN validation was failing in places where it should not. After much investigation, I came to realise that it had to do with the bcscale being set to 2, which caused 1.00 to be returned instead of 1.
I could not find where this is happening in my code, which leads me to believe it's something in configuration tucked away somewhere or a change happening due to another package. I was about to make a PR when I found this fix PR.
Regardless of the cause, I believe this change will let us use the IBAN validator without issue and without causing other packages relying on bcmath to break.

Fixes #14

Thank you @CodeDuck42


styled-components as peerDependency

Shouldn't styled-components be a peerDependency ?

It’s a dependency for the CLI to work, so no. That said, there might be a case for making the components available as a standalone library

Maybe mdx-deck should hold the standalone library and mdx-deck-cli a separate module ?
Currently using mdx-deck as a library might create a ton of duplicate modules depending on the project it is used.

That’s sort of what I was suggesting, except the new package would be mdx-deck-components

Would be nice !
I can try to give it a shot if you create the repository ?

or i create it and you'll fork it later ?

or lerna ?

I'm thinking the package could be generated within this repo's prepare script – I can look into it over the weekend if you wanted to take a shot at it

Closing in favor of #132


basic_value& operator= not guarding against self-assignment

In operator=(const basic_value& v) and operator=(basic_value&& v), if v is the same object as this (&v == this), a segmentation fault occurs (Tested on Ubuntu 20.04 with gcc 9.4.0) when invoking operator=.
To reproduce the issue:
toml::value tmp_tab{{"variable", "test"}};
tab_ = toml::value{{"table", tmp_tab}};
tab_["table"] = tab_["table"];

Thank you for reporting! I reproduced the bug and added a quick patch.


Feat: Support versioning of user project

Create two gits: one for df_designer/frontend_flows.yaml and another for bot
Connect the two gits to get the corresponding graph of a particular build when needed. (During the build process, two commits are made—one in each Git repository. Both commits share the same build_id tag)

TODO: Update frontend to apply the changes
Notes for frontend:

/flows endpoints take now parameter build_id

Included in #97


Create Windows images for release automation

Describe the feature
We want to be able to build and test on Windows platforms.
We need to create docker images for Windows and supported swift versions.
Windows images can only be created from Windows machines.
Ideally the work here will also result in a process/automation that will enable team members to easily create additional Windows images in the future.
Use Case
N/A
Proposed Solution
No response
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

Not a priority for GA as of now.
We don't have customers asking to run Swift SDK on Windows. If we get enough requests, we will re-prioritize it.

We currently have no plans to support Windows.


Add Set Target Level to EXP tracker.

I am frustrated at the time it takes for me to currently set target levels with runelite.
There are currently two ways to do this,

Use the runelite calculator and do the math on how much EXP is left

Use the vanilla client and configure your exp drops and set a target level that way.

I dislike having to interrupt my game by pulling up a vanilla client interface, and taking a long time to set exp trackers.
Describe the solution you'd like
A clear and concise description of what you want to happen.
I'd like to be able to right click any of the runelite exp trackers and click 'set target level' type in a level, and press enter in order to see my remaining exp for a higher level.
This is much easier when doing something like say, early crafting, when I want to set a target at multiple increments, to keep track of when I'm able to make new items.
Currently using the vanilla client feature it is too clunky and interrupts the game which makes me able to only set an end goal, instead of multiple intermediate goals.

The way it works is intentional, and its not like you are changing your goals every minute. It is integrated with vanilla in nice way, and it will stay like that. If you want to improve how the vanilla goal interface works, that is not really something we shoudlt ake care of. Also, the vanilla goals are persistent, while our re-implementation of them would not be, and would be generally worse and not integrated into the game (e.g into the in-game xp tracker as well).

I am changing my goals literally every minute, especially in DMM. It's pretty much useless to use the ingame interface, because its not worth the time it takes to set up a target level.


Check if -h or -help is used on version command

Is your feature request related to a problem? Please describe.
Check if -h or -help is used on version command

Fixed on master


support cert-manager for generating tls and ca

closes #554
Added new flag to Capsule enable-secret-controller, which enables or disables secretcontroller reconcilers.
added 2 new options to Capsule helm chart:

manager.options.enableSecretController - configures enable-secret-controller flag
certManager.generateCertificates - enables cert-manager certificates yaml and other stuff

Besides the ongoing discussion, I think we should change the logic here:
https://github.com/clastix/capsule/blob/9f6883d309368a9dc0dda2c79a436837dc924c55/controllers/secret/ca.go#L178-L182
Actually, the code is not aware we have to ignore the secret since managed externally, maybe I could address this in a different issue: WDYT @MaxFedotov ?

Besides the ongoing discussion, I think we should change the logic here:
https://github.com/clastix/capsule/blob/9f6883d309368a9dc0dda2c79a436837dc924c55/controllers/secret/ca.go#L178-L182
Actually, the code is not aware we have to ignore the secret since managed externally, maybe I could address this in a different issue: WDYT @MaxFedotov ?

Completely ok :) you make a separate PR or add a commit to this?

Completely ok :) you make a separate PR or add a commit to this?

@MaxFedotov we can fix it later, let's get this merged.


Create a hashing state store

Create a state store which hashes the content and stores that, instead of a low/high watermark like the old version.
This will be useful for things like generic RAW rows and data models, where we don't always have an incremental field to use as a high watermark.

split statestore.py into multiple files

Ideally, I'd want to rename the old ones and reorganize a bit, but that would be breaking. I could try to split this up and keep the same behavior with some __init__ magic

split statestore.py into multiple files

Ideally, I'd want to rename the old ones and reorganize a bit, but that would be breaking. I could try to split this up and keep the same behavior with some __init__ magic, but that won't fix the two base class issue.

Moving the hashing state stores to a submodule or another module at the same level would be preferable.


Fixed imports serialization

I've faced issue with imports serialization. We don't serialize import names due to the fact that name equals to the latest identifier in the path. Sometimes this rule doesn't work. For example, if we create new TransactionKernel::assembler() we can find this different name in its imports:
"wallet": LibraryPath { path: "miden::contracts::wallets::basic", num_components: 4 }

Thus, sometimes name differs from the last path element.
This PR should fix this issue by optionally serializing name of import (if it differs).

I'm unfamiliar with this bit of code, and can't really tell how it should behave. Can you give a concrete example of what is this fixing? Perhaps a unit test?

I'm unfamiliar with this bit of code, and can't really tell how it should behave. Can you give a concrete example of what is this fixing? Perhaps a unit test?

When I tried to instantiate TransactionKernel::assembler() and then serialize and deserialize it, I've faced that deserialized assembler doesn't equal to initial one in part of its imports. This is because, according to comment in the current code, author assumes that import name always equals the last segment of import's path and current solution doesn't serialize name of the import at all, calculating it on deserialization. But import of basic wallet has path "miden::contracts::wallets::basic" and name "wallet" which, obviously, can't be figured out from the path.
This fixes incorrect (de)serialization of assembly imports by serializing imports names when it can't be calculated automatically.

so, if I understood correctly this happens when we serialize a reexport, e.g. use.miden::contracts::wallets::basic->wallet? I still think this needs a unit test. And it should probably be done after https://github.com/0xPolygonMiden/miden-vm/pull/1277

so, if I understood correctly this happens when we serialize a reexport, e.g. use.miden::contracts::wallets::basic->wallet?

Yes, it seems so. Thank you, I will add a unit test for this.

I've found that code:
use.miden::contracts::wallets::basic->wallet

Thus, it renames import. Will create unit-test for this.

@hackaugusto, I've added a unit-test.

@polydez I suspect this won't be relevant anymore after #1277 is merged, since the way imports are represented has changed (the ModuleImports type doesn't exist any more for example). I think the issue you hit should be resolved as a result of the changes that were implemented in that PR, even if the representation of things has been reworked.
I know we're hoping to get that merged by next week, but if this issue is blocking you until that happens, it's not a big deal if this gets merged in the meantime.

Thank @bitwalker, I will wait for merging of solution you mentioned.


Avoid internal error when compiling traceback info for an unknown OSError

The added test reproduces the error: when an unknown OSError is raised, friendly greets with a TypeError: dict.update() argument after ** must be a mapping, not str. The reason for this is the cause info being returned as a plain string, not dict.

Thank you very much for the report ... and even more so for the pull request!

@aroberge gladly, and thank you for the fast response! :rocket:


Pull request for Hello Microverse Project

Hello Microverse Project Milestone
In this Milestone I:

Set up Github Action
Configured (Set-up) linter for my local environment
Installed packages for stylelint and hintrc
Added index.html to the project
Added styles.css to the project
Added test.md to the project
Created .gitignore file and added node_modules and test.md
Added descriptive README.md file in professional way


Waypoint: Manipulate Arrays With unshift....Wrong example, description is vague.

Challenge http://freecodecamp.com/challenges/waypoint-manipulate-arrays-with-unshift has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
The example shown here:
var ourArray = ["Stimpson", "J", ["cat"]];
ourArray.shift();
ourArray.unshift(["happy", "joy"]);
// ourArray now equals [["happy", "joy"], "J", ["cat"]]

Is wrong. When I try to do as shown it results in error.
myArray.unshift("Paul"); this is correct, and not shown in the example, yet neither description nor example shows this.
Please fix.

Hey @guavajellyf1sh thanks for posting this!
The example you mentioned is in the comments just to show you how you might use the unshift function, however it is not the answer as you have pointed out.
I see why the wording of the example might be confusing, do you have any suggestions that might make it easier to understand?

Yes instead of showing the incorrect example ourArray.unshift(["happy", "joy"]);
Can you please show the correct way ourArray.unshift("happy", "joy"); ?

Yeap, no square brackets when adding values at the beginning of an array, which is confusing, since when you declare an array, you use square brackets but there you go... Example tricks people (line 4). It made me go round in circles for a while before I realised what it was.

Same here. I had to look it up on W3schools.com for their example and viola it worked. More of a typo than a bug.

The example is not incorrect, just confusing.  I think the confusion is that it adds an array as an element to an array of strings so it goes from:
[STRING, STRING]
to
[ARRAY, STRING, STRING].
This is definitely confusing to a beginner, I would suggest that the lesson just add a string instead.

var ourArray = ["Stimpson", "J", ["cat"]];
ourArray.shift();
ourArray.unshift(["happy", "joy"]);
// ourArray now equals [["happy", "joy"], "J", ["cat"]]
Hi Ben,
Why is it when I copy the same exact example code for code it doesn't show up as it should? It only works when remove the square brackets. Yet the example shows it with the the string with the square brackets [()]?

Hi @guavajellyf1sh,
Where are you copying it such that it doesn't work?

I copied the form myArray.unshift(["string"]); which didn't work. myArray.unshift("string") worked as it is without the square brackets.
This has the ([""]) brackets in the example which I copied, and apparently wasn't correct.
ourArray.unshift(["happy", "joy"]);

Hey @guavajellyf1sh, I'm going to try to chat with you on Gitter to see if we can figure this out

Alright cool. What's your chat username?

Okay, so after working through it with @guavajellyf1sh , we've confirmed that the instructions are correct but confusing since it prepends a nested array in the example and just asks the user to prepend a string in the challenge.
I propose changing the example to prepend a string or a number to remove this confusion (or adequately explain nested arrays/objects).


No ActivityHandler events raised for new messages in group chats with bot installed

Version
4.14.1
Describe the bug
I created a group chat and installed my bot into it.  I can post proactively using a connectorclient, and I get events for participants being added and removed, but there are no activities raised for new messages in the chat thread.
To Reproduce
Using graph, create a group chat and add a bot like this:
                Chat c = new Chat
                {
                    ChatType = ChatType.Group,
                    Topic = "Group chat:" + Guid.NewGuid(),
                    Members = new ChatMembersCollectionPage()
                    {
                        new AadUserConversationMember
                        {
                            Roles = new List<String>()
                            {
                                "owner"
                            },
                            AdditionalData = new Dictionary<string, object>()
                            {
<EMAIL_ADDRESS>"https://graph.microsoft.com/v1.0/users/"+ userID}
                            }
                        },
                        new AadUserConversationMember
                        {
                            Roles = new List<String>()
                            {
                                "owner"
                            },
                            AdditionalData = new Dictionary<string, object>()
                            {
<EMAIL_ADDRESS>$"https://graph.microsoft.com/v1.0/users/" + AppId}
                            }
                        }
                    }
                };
                //requires TeamsAppInstallation.ReadWriteForChat
                Chat resp = await m_graphClientUser.Chats.Request().AddAsync(c);
                m_chatID = resp.Id;
                var teamsAppInstallation = new TeamsAppInstallation
                {
                    AdditionalData = new Dictionary<string, object>()
                {
<EMAIL_ADDRESS>"https://graph.microsoft.com/v1.0/appCatalogs/teamsApps/" + catalogAppID}
                }
                };
                await m_graphClientUser.Chats[resp.Id].InstalledApps.Request().AddAsync(teamsAppInstallation);

Then, override OnTurnAsync in a bot that derives from ActivityHandler (I just left the default and set a breakpoint to inspect activities:
    public class ACSBot : ActivityHandler
    {
        public override Task OnTurnAsync(ITurnContext turnContext, CancellationToken cancellationToken = default)
        {
            return base.OnTurnAsync(turnContext, cancellationToken);
        }

Note if you add/remove participants from the chat in Teams, OnTurnAsync (and the appropriate other events) will fire.  Reactions to bot messages also fire events.  If another user sends a message to the chat thread though, nothing is fired on the bot, so the bot can't react to the new messages like it would in a 1:1 chat.
Expected behavior
I'd expect to get message activity for any other user message in the chat.

@chrisbardon, are you still needing assistance on this issue?

@stevkan - it seems like this is by design, so at this point the only other assistance would be to amend that design so that bots installed to a group chat with chat.readwrite should get notified of all messages in the chat as they happen.  It's possible to do this with a change notification subscription, but it's harder than it should be (I actually put together a blog post describing how to do this here)

Copying from https://docs.microsoft.com/en-us/learn/modules/msteams-conversation-bots/2-conversation-bots:

Group chats are non-threaded conversations between three or more people. They tend to have fewer members than a channel and are more transient. Similar to a channel, your bot will only have access to messages where it's @mentioned directly.
Scenarios that work well in a channel will usually work as well in a group chat.

I think this is by-design and is a feature request: Bot on Teams should react/receive messages without explicit at-mentions.


[Feature Request]: Add a new page to display all contributors

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I want to add a page to showcase all the contributors of this website.
@mansiruhil13  please assign me this task.
Use Case

..
Benefits
No response
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

@mansiruhil13

contributors on website?? its' already mentioned on readme file of repo

contributors on deployment site?? its' already mentioned on readme file of repo. i think, there is no need.

Ok ok


regarding the documentation for using the custom activation strategy

I am struggling to update the context in case of using a custom activation strategy. I went through the documentation and did not find any examples there. There exists one example for Node though. It was mentioned that clients provide the abstraction for using the custom activation strategies, but I could not figure out how. If there are any pointers for the same would be greatly helpful. For now, I am using existing strategies with different values to match my need.

@santhosh-challa I am also looking for a way to work with the strategies defined in the Unleash UI and I have found none so far. Can you show me somehow they way you are dealing with the strategies in the frontend? Thanks in advance.

I am struggling to update the context in case of using a custom activation strategy. I went through the documentation and did not find any examples there. There exists one example for Node though. It was mentioned that clients provide the abstraction for using the custom activation strategies, but I could not figure out how. If there are any pointers for the same would be greatly helpful. For now, I am using existing strategies with different values to match my need.

Hi. In order to create a custom strategy for the client side you need to add it to the unleash-proxy. It will accept a customStrategies field or environment variable: UNLEASH_CUSTOM_STRATEGIES_FILE that takes an array of paths to exported javascript strategies. Are you running your own proxy?

@santhosh-challa I am also looking for a way to work with the strategies defined in the Unleash UI and I have found none so far. Can you show me somehow they way you are dealing with the strategies in the frontend? Thanks in advance.

Is this related to custom strategies?

Hi, @FredrikOseberg. Nope. I had misunderstood and by now we have made it work with the standard strategies. Thanks anyway.

Thanks for the pointer, I will check that. Is that documented somewhere? Currently, we are running our won proxy.

I am struggling to update the context in case of using a custom activation strategy. I went through the documentation and did not find any examples there. There exists one example for Node though. It was mentioned that clients provide the abstraction for using the custom activation strategies, but I could not figure out how. If there are any pointers for the same would be greatly helpful. For now, I am using existing strategies with different values to match my need.

Hi. In order to create a custom strategy for the client side you need to add it to the unleash-proxy. It will accept a customStrategies field or environment variable: UNLEASH_CUSTOM_STRATEGIES_FILE that takes an array of paths to javascript files with the exported strategies you have written.

Thanks for the pointer, I will check that. Is that documented somewhere? Currently, we are running our won proxy.

I think you have found a weakness in our documentation and I have alerted our excellent community manager @thomasheartman  who is currently working on updating this. In the meantime, I've looked at the code and it seems that if you provide a filepath to file that exports an array of instantiated strategies that should work. Something like this:
// customStrategies.js

class MyCustomStrategy extends Strategy {
    //...
}

export default [new MyCustomStrategy()]

Ok, thanks for the example. I will check this. Thanks for the quick help!

No problem. Closing this for now, if you still have issues please let me know.

Hey, folks! Just wanted to chime in and say that the updated documentation is well underway, but due to Christmas holidays it likely won't be released until January. In the meantime, feel free to check out the PR and the prerendered update.
I've updated the Custom Activation Strategies section, the Unleash Proxy docs, and added a new "How to use custom activation strategies" document.
@santhosh-challa Feel free to check it out if you have time; I'd love some outside feedback on this as well: do the new updates make sense?
Note that I haven't had it reviewed yet, so there may still be some minor errors hanging about that'll get fixed before we release.

Hi @thomasheartman thanks for your efforts in making the documentation great!. I just quickly looked at it. Everything looks good! I have a doubt only at the below statement in  "How to use custom activation strategies" document:
You mentioned that:

Front-end client SDK don't evaluate strategies directly, so you need to implement the custom strategy in the Unleash Proxy. The below steps assume you're running the Unleash Proxy as a Docker container. If you're running the Unleash Proxy via Node directly, follow the steps for server-side SDKs instead.

I felt a little disconnected here. If we are running the proxy using node, the example code looks like this:
const app = createApp({
    unleashUrl: 'https://app.unleash-hosted.com/demo/api/',
    unleashApiToken: '56907a2fa53c1d16101d509a10b78e36190b0f918d9f122d',
    proxySecrets: ['proxy-secret', 'another-proxy-secret', 's1'],
    refreshInterval: 1000,
    // logLevel: 'info',
    // projectName: 'order-team',
    // environment: 'development',
});

And you mentioned referring to the previous step, which is talking about using unleash on the server side, not setting up unleash proxy using node, right?
I feel if we have another section option B and to show an example passing custom strategies to createApp would make it perfect. Am I making sense? Please let me know if I am not making a point here.
Thanks!

@santhosh-challa Yeah, that's great input! I was wondering about that too. Thanks for letting me know; I'll sort it out 😄

@santhosh-challa Thanks again for the input 🙏🏼 I've added an extra section on using custom strategies with the proxy when running it via Node now. Thanks for helping us make the docs better. And any more feedback you've got is always welcome 😄


Stacking of Classifiers that Operate on Different Feature Subsets

I have a dataset with say 200 features. What I want is to give 30 features to 1 classifier, 90 to another and 80 to another in one layer of ensembled clfs and then take their outputs to and give them to a meta classifier. I believe this is achievable via Subset Class available in your library but  can't figure the right way.
I have found a similar way in another library 'mlxtend' code of which is available below. However, I'd lie to do my work via your library.
Thanking you in anticipation.
from sklearn.datasets import load_iris
from mlxtend.classifier import StackingCVClassifier
from mlxtend.feature_selection import ColumnSelector
from sklearn.pipeline import make_pipeline
from sklearn.linear_model import LogisticRegression

iris = load_iris()
X = iris.data
y = iris.target

pipe1 = make_pipeline(ColumnSelector(cols=(0, 2)),
                      LogisticRegression())
pipe2 = make_pipeline(ColumnSelector(cols=(1, 2, 3)),
                      LogisticRegression())

sclf = StackingCVClassifier(classifiers=[pipe1, pipe2],
                            meta_classifier=LogisticRegression(),
                            random_state=42)

sclf.fit(X, y)

Same doubt !

@m-mohsin-zafar  thank you for sharing mlxtend code! Have you tried pystacknet, vecstack and scikit - learn

@m-mohsin-zafar mlens documentation is seriously very confusing and not up to the mark

Hi there,
Thanks for reaching out!
We can achieve what you're looking for easily by adding using dicts to specify pipelines when adding a layer to the ensemble:
from sklearn.datasets import load_iris
from sklearn.linear_model import LogisticRegression
from mlens.ensemble import SuperLearner
from mlens.preprocessing import Subset

iris = load_iris()
X = iris.data
y = iris.target

ens = SuperLearner()
ens.add(estimators={"pipe-1": [LogisticRegression()],
                    "pipe-2": [LogisticRegression()]},
        preprocessing={"pipe-1": [Subset([0, 2])],
                       "pipe-2": [Subset([1, 2, 3])]})
ens.add_meta(LogisticRegression())
ens.fit(X, y)

The key to note is that the values in these dicts should be lists:
ests = {pipe_1: [est_1, est_2, ...], pipe_2: [est_1, est_2, ...]}
prps = {pipe_1: [trans_1, trans_2, ...], pipe_2: [trans_2, trans_2, ...]}

So if we feed an input X to this layer, it will get processed in parallel through pipe_1 and pipe_2. In each of these, we obtain preprocessed features X -> trans_1 - > trans_2 - > X_[1,2] that we feed to the list of estimators in that pipeline. The output of a layer is the concatenation of all predictions:
P = [pipe_1_est_1(X_1), pipe_1_est_2(X_1), ..., pipe_2_est_1(X_2), pipe_2_est_2(X_2), ...]

Note that you can also propagate features from the input array X to the output array P by using the propagate_features argument when adding a layer to the ensemble:
ens.add(estimators=ests, preprocessing=prep, propagate_features=[0, 1, 2])

The reason for using this logic is that it allows us to run a preprocessing pipeline just once and then have many estimators using those features. The sklearn version would require us to re-run the preprocessing step for every estimator, which isn't efficient.
Having said that, you can mix and match between mlxtend, mlens, and scklearn:
from sklearn.datasets import load_iris
from mlxtend.feature_selection import ColumnSelector
from sklearn.linear_model import LogisticRegression
from mlens.ensemble import SuperLearner
from sklearn.pipeline import make_pipeline

iris = load_iris()
X = iris.data
y = iris.target

pipe1 = make_pipeline(ColumnSelector(cols=(0, 2)),
                      LogisticRegression())
pipe2 = make_pipeline(ColumnSelector(cols=(1, 2, 3)),
                      LogisticRegression())

ens = SuperLearner()
ens.add([pipe1, pipe2])
ens.add_meta(LogisticRegression())
ens.fit(X, y)

Hope this helps! Feel free to reopen this issue otherwise : )


Can protractor absorb 404 error?

we have a page that contains 404 error for a not found image. The protractor test case fails on this page. When if I upload the image, protractor is happy. Is there any way protractor can by pass this error?

Please direct general support questions like this one to an appropriate support channel, see https://github.com/angular/protractor/blob/master/CONTRIBUTING.md#questions
Thank you!


Action for onExit not triggered when transition out on higher level

Bug or feature request?
Bug
Description:
The onExit action in a nested, parallel state is not triggered when transitioning out on a higher level
(Bug) Expected result:
onEntry and onExit actions should be triggered
(Bug) Actual result:
onExit action is not triggered, onEntry is though.
Link to reproduction or proof-of-concept:
https://repl.it/@rodin/xstate-example
It logs D2 entry, but not D2 exit, which I think it should?

You're right, it should. I'm working on a major refactor that will address this (and also simplify the algorithms).

Thanks, that's all I needed to know.

Fixed in 05a19b8c4c20f4051fee67022a2f67a5d5de5861 (and available in xstate@next)


Redstone Conduits are preventing chunks from being unloaded

Issue Description:
Redstone Conduits are preventing chunks from being unloaded.
What happens:
EnderIO has checks in place to verify if a chunk is loaded but it lacks checks for if the chunk is queued for unload, when getBlockState is called it results in the chunk having its unloadQueued field set to false, This is preventing the chunk from being unloaded.
On a Forge server all Redstone Conduits will prevent chunks from being unloaded.
On a SpongeForge server only Redstone Conduits with a Redstone Sensor Filter will prevent chunks from being unloaded.
What you expected to happen:
EnderIO shouldn't be ticking conduits that are in chunks queued for unload.
Steps to reproduce:
Forge:

Place a Redstone Conduit.

SpongeForge:

Place a Redstone Conduit.
Right click with a Yeta Wrench.
Insert a Redstone Sensor Filter.

I disconnected from the server and used IntelliJ to set breakpoints so I could see what chunks were still loaded.

Affected Versions (Do not use "latest"):

EnderIO: 1.12.2-5.1.52
EnderCore: 1.12.2-0.5.73
Minecraft: 1.12.2
Forge: <IP_ADDRESS>47
SpongeForge? yes and no
Optifine? no
Single Player and/or Server? I've only tested this on a Dedicated Server

Your most recent log file where the issue was present:
Redstone Conduit Stacktrace (Forge):
getLoadedChunk:92, ChunkProviderServer (net.minecraft.world.gen)
loadChunk:107, ChunkProviderServer (net.minecraft.world.gen)
loadChunk:101, ChunkProviderServer (net.minecraft.world.gen)
provideChunk:147, ChunkProviderServer (net.minecraft.world.gen)
getChunk:374, World (net.minecraft.world)
getChunk:366, World (net.minecraft.world)
getBlockState:1007, World (net.minecraft.world)
neighborNotifyEvent:212, RedstoneConduitNetwork (crazypants.enderio.conduits.conduit.redstone)
notifyConduitNeighbours:191, RedstoneConduitNetwork (crazypants.enderio.conduits.conduit.redstone)
tickEnd:268, RedstoneConduitNetwork (crazypants.enderio.conduits.conduit.redstone)
onServerTick:57, ServerTickHandler (crazypants.enderio.base.handler)
invoke:-1, ASMEventHandler_124_ServerTickHandler_onServerTick_ServerTickEvent (net.minecraftforge.fml.common.eventhandler)
invoke:90, ASMEventHandler (net.minecraftforge.fml.common.eventhandler)
post:182, EventBus (net.minecraftforge.fml.common.eventhandler)
onPostServerTick:266, FMLCommonHandler (net.minecraftforge.fml.common)
tick:787, MinecraftServer (net.minecraft.server)
run:592, MinecraftServer (net.minecraft.server)
run:748, Thread (java.lang)

Redstone Conduit with Redstone Sensor Filter Stacktrace (SpongeForge):
getLoadedChunk:92, ChunkProviderServer (net.minecraft.world.gen)
redirect$zmg000$impl$ProvideChunkForced:145, ChunkProviderServer (net.minecraft.world.gen)
provideChunk:147, ChunkProviderServer (net.minecraft.world.gen)
getChunk:374, World (net.minecraft.world)
getChunk:366, World (net.minecraft.world)
getBlockState:1970, WorldServer (net.minecraft.world)
apply:15, ComparatorInputSignalFilter (crazypants.enderio.base.filter.redstone)
getNetworkInput:432, InsulatedRedstoneConduit (crazypants.enderio.conduits.conduit.redstone)
updateInputsForSource:114, RedstoneConduitNetwork (crazypants.enderio.conduits.conduit.redstone)
tickEnd:260, RedstoneConduitNetwork (crazypants.enderio.conduits.conduit.redstone)
onServerTick:57, ServerTickHandler (crazypants.enderio.base.handler)
invoke:-1, ASMEventHandler_145_ServerTickHandler_onServerTick_ServerTickEvent (net.minecraftforge.fml.common.eventhandler)
invoke:90, ASMEventHandler (net.minecraftforge.fml.common.eventhandler)
forgeBridge$post:253, EventBus (net.minecraftforge.fml.common.eventhandler)
post:203, EventBus (net.minecraftforge.fml.common.eventhandler)
onPostServerTick:266, FMLCommonHandler (net.minecraftforge.fml.common)
tick:787, MinecraftServer (net.minecraft.server)
run:592, MinecraftServer (net.minecraft.server)
run:748, Thread (java.lang)

seems to be SpongeForge problem bc are server doesn't have any problem with redstone conduits

The code is there in vanilla/Forge, too:

and is documented:

But absolutely nothing but the ChunkProvider accesses that field:

Also, queueUnload() is called:

When a player logs out; on all chunks within their view distance (even if there are other players there).
When the server is stopped.
When the admin issues a command to unload one or all chunks.
After the pregen command has generated a chunk.

All of them are "unload if the chunk is idle" events.

I've retested this in a production environment with only Forge, EnderCore, EnderIO and a custom mod to list loadedChunks via commands, I've found that moving away from the chunk so that it's outside the view distance will cause it to get unloaded instantly, However disconnecting while the chunk is still within view distance will keep the chunk loaded even with no other players present on the server.

seems to be SpongeForge problem

Nope, As mentioned above and in the original post I've tested this without SpongeForge being present.

But absolutely nothing but the ChunkProvider accesses that field

Correct but if you check the World::getBlockState method which EnderIO calls and follow how it loads chunks you will end up at the ChunkProviderServer::getLoadedChunk which does change the unloadQueued field, I did provided a Stacktrace in my original post which shows the RedstoneConduitNetwork::tickEnd method resulting in ChunkProviderServer::getLoadedChunk getting called.

All of them are "unload if the chunk is idle" events.

Yes but the issue here is that EnderIO is marking the chunk as active when the chunks in question shouldn't be active.

If the chunks are unloaded properly when moving away, this sounds more like a vanilla or Forge bug. There's absolutely no indication that a mod should check that field in addition to world.isBlockLoaded(). There are also plenty of calls to isBlockLoaded() in vanilla code and not a single one does any additional check.

Looking at the order in which notable things are ticked:

Scheduled tasks which include packets are processed first, This means that the CPlayerPacket which gets sent when the client moves is processed, part of the processing involves updating the PlayerChunkMap to mark chunks that are non longer within the players view distance as inactive.

Worlds are ticked and this also include the ChunkProvider, it's at this point chunk are unloaded if they have been marked as inactive.

World Entities and TileEntities are ticked.

NetworkSystem is then ticked which handles player disconnections and part of this processing is removing the player from the PlayerChunkMap which again will mark chunks as inactive.

Finally the RedstoneConduitNetwork tickEnd method is called.

This explains why moving away from chunks unloads them correctly as it's happening within the same tick however when a player disconnects the chunks that they were loading are marked as inactive but won't be unloaded until the next tick.
Because the RedstoneConduitNetwork runs at the end of the server tick any chunks that were marked as inactive due to a player disconnecting are marked as active by EnderIO which then prevents the chunk from being unloaded on the next tick.
I wouldn't call it a Forge or Vanilla bug, Simply put EnderIO is essentially performing TileEntity logic where it shouldn't be performed it, at least not without additional checks in place, no mod should have to check the unloadQueued field but again you've shifted a ton of logic to be outside the order in which vanilla Minecraft does things and as such you can't expect it to work with zero issues.

ItemConduits on RFTools Crafters dont Unload Chunks.
Energy Conduits on Mekanism Machines dont Unload Chunks.
Using Sponge Forge RC4007
Forge 2847
Minecraft 1.12.2
Just adding this as my case was duplicate of this


Not An Issue in This Project : This is a Great Project

Hi,
i like this "Nuitka" transpiler ( Python to C++ ) project.
its useful.
i have Starred this project.
it needs to be improved further.
i have mentioned it here in SO(StackOverflow) site, would you please upvote it here (please upvote both of my Q+A) if you find it useful or if you think it will be helpful for other users related to this type of transpiler projects.
( and please downvote user Caleb's answer, as that is incorrect answer . Correct-answer must answer my questions first , My question is about "Transpiler" tools & which "transpiler" tool can keep high-level algorithms/structures intact , Question/article is NOT about language learning, when to transpile & when-not, etc.
In SO site, user Flimzy (and Caleb) doesn't give advice When "C/C++",etc to "Go" conversion related question is posted or when "C/C++" is criticized , But when "Go"-to-"C/C++",etc conversion Question is posted or when Google based products (like "Go"-language) are criticized , then those users, especially Flimzy "Close" the question & downvotes, & give many un-asked advices, etc, Because those users are Google PUSHERS & google based product (like "Go") pushers . Google Crimes1, 2, 3, 4, 5, 6. Problems in "Go"1. )
Thanks.
EDIT : added few more lines, i Starred this project, etc.

I consider this to be spam.


Allow client to match replay file with model hash

I was able to get this inside the file itself.
But it is also passed to the upload method inside server_converter.py


Minor tweaks to the tutorial notebook to support Spark 1.3.0

Also a couple of minor grammar tweaks in the first paragraph.

Thanks for recreating PR. LGTM.


Location of Nginx configuration file?

I'm asking because I want to submit a pull request letting static assets be compressed with gzip.

Our operational provisioning isn't part of this repository. We're happy to have any suggestions, but the provisioning and other operational pieces are all private projects.

Can it be made open-source? There is a campaign going on to open-source not just the code but all the things used to maintain it. As of now, readthedocs.io is quite slow and I want to submit a PR that does some major performance tweaks.

What in particular are you wanting to have gzipped? We already have this config:

    gzip on;
    gzip_vary on;
    gzip_types    text/plain text/html text/css
                  application/x-javascript text/xml
                  application/xml application/xml+rss
                  text/javascript;

I've updated it in a PR to:
    gzip_types
    application/atom+xml application/javascript application/json application/ld+json application/manifest+json
    application/rss+xml application/vnd.geo+json application/vnd.ms-fontobject application/x-font-ttf
    application/x-web-app-manifest+json application/xhtml+xml application/xml font/opentype image/bmp
    image/svg+xml image/x-icon text/cache-manifest text/css text/plain text/vcard text/vnd.rim.location.xloc
    text/vtt text/x-component text/x-cross-domain-policy;

*.js ≈ 269KB
https://kintojs.readthedocs.io/en/latest/js/highlight.pack.js   183KB
https://kintojs.readthedocs.io/en/latest/js/jquery-2.1.1.min.js 53 KB

cache *.js *.css

google has a NGINX module that does all of this automatically https://www.modpagespeed.com/doc/build_ngx_pagespeed_from_source
https://developers.google.com/speed/pagespeed/insights/?url=https%3A%2F%2Fkintojs.readthedocs.io&tab=desktop

I'm not following. Are you wanting us to minify the JS along with gzipping it? I don't feel comfortable changing the actual served JS content of folks, other than compressing it, at scale.

Google PageSpeed insights suggests:

Enabling compression
Eliminating render-blocking JavaScript and CSS in above-the-fold content
Leveraging browser caching
Minifying CSS and JS

And Google provides NGINX and Apache modules to apply those optimizations automatically

@CrazyPython I have just some comments/questions on this:

Enabling compression

This is already done. Just #3564 missing

Eliminating render-blocking JavaScript and CSS in above-the-fold content

What's needed for this? I think with the public code you can propose a PR, right? How complicated it is? How much work does this involve?

Leveraging browser caching

What is needed for this?

Minifying CSS and JS

From Eric's reply I think this is not going to be done in the near future.

Would you consider using the NGINX module google made that does the optimizations automatically?

I largely agree with most of this. However some of it may go into other places.

Eliminating render-blocking JavaScript and CSS in above-the-fold content

I don't think RTD should modify the HTML output of docs builds to this extent. If an author has put inline JS into their docs (using a retructuredtext raw block) it might depend on loading libraries before executing that inline JS. Moving JS and if possible CSS down would improve performance, but I think individual Sphinx themes should do that, not RTD. In fact, there were bugs around this in the RTD theme. See https://github.com/rtfd/sphinx_rtd_theme/pull/545.

Minifying CSS and JS

Similar to the above point, this should happen but I would argue the theme should do it. I don't think RTD should be modifying people's build output in that way.

Leveraging browser caching

This is something that RTD can do, although I think it might be a bit more complex than just adding an nginx module. Of these 3 things, this will also make the largest performance impact for visitors on repeat visits to RTD.
Take for example, the pageload for https://docs.readthedocs.io/en/latest/ :

Loads 2 CSS files  from media.readthedocs.org and 1 (typically a 404) from the build output
Loads 7 font files all from media.readthedocs.org
Loads 5 JS files from media.readthedocs.org and 3 from the build output

The default browser caching on media.readthedocs.org caches for 7 days and it uses eTags. There is no caching on the generated build output although there are eTags. The generated build output could change build to build (for example, some custom CSS a user has for overriding a theme) so I don't think RTD can do much there in terms of caching other than eTags which are already on. The files on media.readthedocs.org in theory can change on a new deploy of RTD although some are static forever in practice. This is where the opportunity for caching is, IMO.
One of the easiest optimizations would be to set multi-year/forever cache times for files we know never change. RTD could do this for the fonts, for doctools.js, underscore.js, jquery-2.0.3.min.js, and for jquery-migrate-1.2.1.min.js. readthedocs-doc-embed.css and readthedocs-doc-embed.js will need shorter cache times as those are subject to change. This could be done one-off in the nginx configs although it would be a bit ugly. This would reduce the necessary HTTP requests on repeat visits from 18 to 5. From a size perspective, the fonts are by far the largest files and they would be cached forever leading to significant improvement.
Since these files are written by ./manage.py collectstatic however, a smarter way might be to leverage Django's ManifestStaticFileStorage. Perhaps there's a way to read staticfiles.json and replace the the filename with a version that is cached forever. For example, rather than loading underscore.js we would load underscore.13ac50b.js which is cached forever and never changes unless the actual file contents change. This could possibly happen at docs build time or when the files are served (using ngx_http_sub_module maybe?).

Is there anything else actionable here that we want to do?


Add marl::containers::list, use for marl::ConditionVariable

Provides substantial performance improvements.

Benchmarks show performance improvements ~ 1.4x for certain tests (mostly Schedule/Event).

Looks good, one simple comment about interop with std::list.


type SupportedSocialApps = 'twitter' | 'facebook' | 'whatsapp' | 'googleplus' | 'email';

Steps to reproduce
1.no support INSTAGRAM

that's not true, we already have support for instagram
check this: https://github.com/react-native-community/react-native-share#static-values-for-social


missing script(s) and instructions to fine-tune models using the generated dataset?

@bin123apple could you please add them to the repo?
I guess you used the 2.5K dialogue dataset to fine-tune models.
please add a new section into README.md , how to fine-tune an open-weight model

list the software you used: deepSpeed and version
show example command lines and options used to run fine-tune
share the training dataset or link to it if it is already in the repo

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Document all the steps for finetuning on Dr.Ding's server. / The version etc.

I just uploaded the fine-tuning code in https://github.com/bin123apple/Fortran2Cpp/tree/main/training. Thanks for all the help!

readme.md still needs detailed steps (including scripts) on how to prepare the 2.5k dialog dataset to be a multiple-dialog csv file for fine-tuning the deepseek-coder model.

made some progress to updated the readme for the steps.


Array response

Hi! i am using select and where adaptive expressions to filter in in an array and getting an id if i type in name, the function is stored in a property, and when i send the property as a response i get this:
{
“lgType”: “Activity”,
“text”: [
"976588",
"002848"
]
}
Is there any way i can get this formatted as text:
976588
002848

Hi, you could try using join() on the text property of the result with the newline (or return) character as the joining character.
Example:

The Microsoft Bot Framework team prefers that how to questions be submitted on Stack Overflow. The official Bot Framework Github repos are the preferred platform for submitting bug fixes and feature requests.


Can't login to iMessage.

I can log into Settings with my Apple ID, but not the Messages app.
I don't know if I did the SMBIOS step correctly. I downloaded and unzipped the latest version of the EFI from the releases page, wrote the SMBIOS via GenSMBIOS to EFI\OC\config.plist, and then ran the script. Was that correct or wrong?

Nvm I regenerated SMBIOS and everything's working again.


AttributeError: 'Api' object has no attribute 'protocol_state'

Version information
The pypi version or branch of snappi-ixnetwork
Describe the bug
I am trying to i=open the IxNetwork API  by using Snappi-Ixnetwork  and while confighuring the endpoint getting the atribuite error
Traceback (most recent call last):
File "C:\WS\myproject\idea\S32.venv\lib\site-packages\snappi\snappi.py", line 103, in api
return lib.Api(**params)
File "C:\WS\myproject\snappi-ixnetwork\snappi_ixnetwork\snappi_api.py", line 70, in init
self._protocol_state = self.protocol_state()
AttributeError: 'Api' object has no attribute 'protocol_state'
To Reproduce
Steps to reproduce the behavior. Prefer code snippets, pytest test cases.
import snappi
host is IxNetwork API Server
#api = snappi.api(location='https://localhost:443', ext='ixnetwork')
api = snappi.api(location='https://localhost:20320', ext='ixnetwork')
Expected behavior
A clear and concise description of what you expected to happen.
I want to open the ixnetwork API while running the code
Additional context
Add any other context about the problem her

Issue is fixed in snappi-ixnetwork v1.13.0.
It incorporates support of new APIs.
protocol_state is obsolete in OTG model 1.x onwards.
Sample new api:
# start transmitting configured flows
control_state = api.control_state()
control_state.choice = control_state.TRAFFIC
control_state.traffic.choice = control_state.traffic.FLOW_TRANSMIT
control_state.traffic.flow_transmit.state = control_state.traffic.flow_transmit.START  # noqa
api.set_control_state(control_state)


Building on MinGW32 fails

error: failed to run custom build command for `rust-crypto v0.2.36`
Process didn't exit successfully: `C:\Users\kaoD\rustocin\target\debug\build\rust-crypto-0caf103d2b9a229d\build-script-build` (exit code: 101)
--- stdout
TARGET = Some("i686-pc-windows-gnu")
OPT_LEVEL = Some("0")
PROFILE = Some("debug")
TARGET = Some("i686-pc-windows-gnu")
debug=true opt-level=0
TARGET = Some("i686-pc-windows-gnu")
HOST = Some("i686-pc-windows-gnu")
CFLAGS_i686-pc-windows-gnu = None
CFLAGS_i686_pc_windows_gnu = None
HOST_CFLAGS = None
CFLAGS = None
running: "cc" "-O0" "-ffunction-sections" "-fdata-sections" "-g" "-m32" "-o" "C:\\Users\\kaoD\\rustocin\\target\\debug\\build\\rust-crypto-0caf103d2b9a229d\\out\\src\\util_helpers.o" "-c" "src/util_helpers.c"

failed to execute command: El sistema no puede encontrar el archivo especificado. (os error 2)
Is `cc` not installed? (see https://github.com/alexcrichton/gcc-rs#compile-time-requirements for help)

According to the link in the error message gcc-rs should be using gcc under my system. This hardcoded "cc" in build.rs seems to be the problem.
Setting envvar CC=gcc fixes it, but AFAICT it probably should use the default gcc-rs strategy for that else branch.

This is not so much a bug as a missing dependency. I was experiencing the same problem but a little searching around the web pointed me towards installing this:
http://win-builds.org/doku.php/download_and_installation_from_windows
Also ensure the 'bin' directory residing within the installation path is present in your PATH - as usual - and away you go. You'll find 'cc.exe' within the bin directory.

I'm suffering from this error too.Thanks for the tip:"Setting envvar CC=gcc fixes it."@alvaro-cuesta.

I think gcc should be use as default also.

How do I set the envvar CC=gcc under Windows 7/8.1/10???

http://www.computerhope.com/issues/ch000549.htm
@Trodis

Yep, just hit this problem myself on Windows 10. I'll set the env var for now, is there a proper fix coming for this?

Setting envvar 'CC=gcc' fixes it,
What does it means? How can I do it too?

I ran across this problem in appveyor, where CC doesn't seem to be available with TARGET=x86_64-pc-windows-gnu. I found the solution in the appveyor.yml of this project:
environment:
  matrix:
    - TARGET: nightly-x86_64-pc-windows-msvc
    - TARGET: nightly-x86_64-pc-windows-gnu
      MSYS_BITS: 64
  [...]
install:
[...]
- if defined MSYS_BITS set PATH=C:\msys64\mingw%MSYS_BITS%\bin;C:\msys64\usr\bin;%PATH%

In thise file, every mingw build has MSYS_BITS defined which adds the mingw bin folder to the PATH prior to starting the build. This fixes the mingw build without breaking the -msvc builds.

how do you set cc=gcc ? what does it mean.
I have installed migwin. nothing resolved after.

@steelx
https://rustsec.org/advisories/RUSTSEC-2016-0005.html

@steelx
https://rustsec.org/advisories/RUSTSEC-2016-0005.html

I'm using openssl. And internally it uses this it seems

No, OpenSSL does not use rust-crypto, neither the other way around. If you have issues with OpenSSL bindings try asking in respective repositories, e.g. in https://github.com/sfackler/rust-openssl.


Feature request: support SPIR-V input for vulkan backend.

As of right now it appears that RGA only takes glsl files for the vulkan backend. Ideally I'd feed it my own .spv files. It looks like beProgramBuilderVulkan.cpp is only missing handling for in.spv and in.spvText cmd's into amdspv.exe so it should be easy to add as lon as amdspv.exe properly supports it.

Hi Jasper-Bekkers,
We will add this feature to RGA's feature list.

Sounds good, should be a relatively small code-change since amdspv.exe alread supports the hard work from what I can see (might be mistaken).

Just wanted to post that we really need this or a similar tool as well! We've run into some shaders that are performing significantly worse than expected and we currently have no real visibility into what might be going on.

+1 from me too.  This would be super useful.

Hi punkUser, danginsburg,
SPIR-V as an input language will be included as a beta feature in the upcoming RGA release, which is coming soon (a matter of a few weeks).

Hi punkUser, danginsburg,
SPIR-V support was added as a beta feature to the RGA version that was released today (RGA-1.1).
We hope that you will find it useful.

Bumping this because binary spirv seems to be crashing i the latest released version. The text spv does work though so can always pass it through spirv-dis.exe first to work around.

Hi Jasper-Bekkers,
There is going to be an RGA release soon that will include fixes to the Vulkan backend.
If you can provide more details about the shaders and the command that you were using, we will try to make sure that the specific problem that you are seeing is resolved.

Hi Jasper-Bekkers,
Please let us know if the fixes in RGA 1.2 solved your problem.


psutil: 2.1.1 -> 3.4.2

Successfully built locally.
Notably, version 2.1.1 seems to be incompatible with Let's Encrypt client (https://github.com/letsencrypt/letsencrypt/issues/1080).

Thanks @k0ral
I've included this one in https://github.com/NixOS/nixpkgs/pull/12552 which I hope to merge soon.
Please always check whether you can enable tests after an upgrade.


Convert interpolation to concatenation in mysql driver

Why? The interpolation is alright. Unless there is a bigger reason, we should leave it as is.

I see the other config params are being concatenated like database and extra I figured it will make it unified but if you think its not vital thats fine

Ok, closing then! Thank you! :heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Can I define an event that fires when the tab is changed?

Thank you for the wonderful project! 👍
I am using multiple tabs. And I want to fire an event when the focused tap changes.
I've been looking for a guide, but I can't find an answer, so I'll ask. 🙏

That is only available on the Pro distribution onopenworksheet

Thank you for your reply!

Thank you for your reply!


sick_scan: 1.7.6-1 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository sick_scan to 1.7.6-1:

upstream repository: https://github.com/SICKAG/sick_scan.git
release repository: https://github.com/SICKAG/sick_scan-release.git
distro file: melodic/distribution.yaml
bloom version: 0.9.7
previous version for package: 1.6.0-1

Holding for sync.


Issue saving as PDF from Nextcloud (Hidden letters)

I have an interesting issue. When I try to Save As PDF this same spreadsheet, there are issues rendering some letters.
Testing Notes:

The first letter of the spreadsheet is always invisible (It's still there as I can search for it)
This doesn't happen on merged cells (If first letter of spreadsheet is within a merged cell, it is the next cell over that is affected)
The letter that is affected is  affects all cells that contain the same case-sensitive letter. (Doesn't have to be the first letter of the cell)
This issue occurs when creating a document from scratch within OnlyOffice and when importing existing documents created from other sources. (AKA It's the Save As PDF that is not working correctly)
This issue is not present when saved as PDF in another document reader and then viewed in Open Office (more proof its the Save As PDF that is not working correctly)
This issue is not present when viewed in another document reader (such as Excel)

Screenshot
(The letter "S" is invisible)

Links & Downloads
Nintendo Switch.xlsx
The following links will take you to my Nextcloud Server.
Spreadsheet (Edit Mode) https://cloud.betaleaf.net/apps/onlyoffice/6370
Spreadsheet https://cloud.betaleaf.net/s/hC1LsnTTVJBW89W
PDF (OnlyOffice)https://cloud.betaleaf.net/s/bnqExmsytuQQdg0
PDF (Excel)https://cloud.betaleaf.net/s/6hOTNetNJRVHB7I
If prompted to log in, use guest:guest
Don't worry about messing up the spreadsheet, its a copy.

@BetaLeaf Yeah, I can confirm size difference but can't find if we missed something, Onlyoffice PDF looks almost exactly the same with PDF visually


Unable to install 12.6.3 (or 12.6.2) version

My Environment

[X] Windows 10

[X] My Windows installation is non-English.

I'm using NVM4W version: 1.1.7
I have already...

[x] read the README to be aware of npm gotchas & antivirus issues.
[X] reviewed the wiki to make sure my issue hasn't already been resolved.
[X] verified I'm using an account with administrative privileges.
[X] searched the issues (open and closed) to make sure this isn't a duplicate.
[X] made sure this isn't a question about how to use NVM for Windows, since gitter is used for questions and comments.

Expected Behavior
Running nvm install 12.6.3 should install node 12.6.3
Actual Behavior
I get the error message:
Version 12.6.3 is not available. If you are attempting to download a "just released" version,
it may not be recognized by the nvm service yet (updated hourly). If you feel this is in error and
you know the version exists, please visit http://github.com/coreybutler/nodedistro and submit a PR.
Even tough nvm list available show this version as available as shown below:
Oddly enough, installing node 14.2.0 with nvm install 14.2.0 worked like a charm.

Steps to reproduce the problem:

Install nvm on Windows 10
Run nvm install 12.6.3

Looked at https://nodejs.org/dist/index.json and realized latest available to install is 12.6.0.


refactor: Update examples to use new addons module repo

What does this PR do?

Updates examples to use the new https://github.com/aws-ia/terraform-aws-eks-blueprints-addons repo for addons.
Replaces decommissioned add-ons with examples that show use of the helm_release resource or https://github.com/aws-ia/terraform-aws-eks-blueprints-addons/eks-blueprints-addon helper module in-cases where IRSA would be needed (e.g. appmesh-controller)

Motivation

Part of the in-progress v5 refactoring epic.
Partially fixes #1394

More

[x] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[ ] Yes, I have updated the docs for this feature
[x] Yes, I ran pre-commit run -a with this PR

For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes

Replacing the cluster_oidc_issuer_url for oidc_provider parameter in the following files:

examples/agones-game-controller/main.tf
examples/amp-amg-opensearch/main.tf
examples/appmesh-mtls/main.tf
examples/argocd/main.tf
examples/blue-green-upgrade/modules/eks_cluster/main.tf
examples/external-secrets/main.tf
examples/fargate-serverless/main.tf
examples/karpenter/main.tf
examples/stateful/main.tf
examples/tls-with-aws-pca-issuer/main.tf

FROM:
  cluster_oidc_issuer_url     = module.eks.cluster_oidc_issuer_url

TO:
  oidc_provider     = module.eks.cluster_oidc_issuer_url

Adjusted the README.md file in the examples/blue-green-update directory.

examples/blue-green-upgrade/README.md

Copy the terraform.tfvars.example to terraform.tfvars on each core-infra, eks-blue and eks-green folders, and change region, hosted_zone_name, eks_admin_role_name according to your needs.

cp terraform.tfvars.example core-infra/terraform.tfvars
cp terraform.tfvars.example eks-blue/terraform.tfvars
cp terraform.tfvars.example eks-green/terraform.tfvars

Please urgently see https://github.com/aws-ia/terraform-aws-eks-blueprints/issues/1630, thank you.


[Dialog][1.0.0-beta.5] The example from the documentation is incorrect

Problem description
The documentation for the <Dialog> tag doesn't seem to be valid with the latest beta. Using the example from the documentation, only its children are displayed and the console shows the following error :

Warning: Unknown props actions, modal on <div> tag. Remove these props from the element.

Steps to reproduce
Copy paste the Simple Dialog example from the documentation. Fix the buttons because the documentation is also outdated for them.
Versions

Material-UI: 1.0.0-beta.5
React: 16.0.0-beta.5
Browser: Vivaldi | 1.11.882.4 (Official Build) (64-bit)

Oh, it's just the website that is outdated. The documentation from the repository isn't.


glCreateBuffers and OpenGL 4.4

Hi,
We should make it clear what OpenGL copatability the project should have. The moment we changed glGenBuffers to glCreateBuffers things went south (for me) with the exception:
Exception thrown at 0x0000000000000000 in Sandbox.exe: 0xC0000005: Access violation executing location 0x0000000000000000.

Some quick research suggested that it's sue to me running OpenGL 4.4 on my laptop. This of course means that the solution does not run out-of-the-box in this case, and based on what I've read, it's due to GL_ARB_direct_state_access entering the core profile only as of 4.5. So:

Are there any reasons for not sticking with glGenBuffers ?
Might it be worth differentiating the implementation based on the GL version?
Should anything below 4.5 be unsupported all together?

"Are there any reasons for not sticking with glGenBuffers ?"
I believe that he used glCreateBuffers just to match the same function names for generating buffers and shader. As he shown, glCreateBuffers does not bind the buffers as he expected.
The difference between glCreate and glGen is that glCreate returns buffers that are completely new and unused, unlike glGen, which reuse buffer names.
Somebody please correct me if i'm wrong, but there is no practical difference between glGen and glCreate
"Might it be worth differentiating the implementation based on the GL version?"
"Should anything below 4.5 be unsupported all together?"
I really don't see a reason for that. You could base your build on the GL version, but I don't think that is really necessary to limit the build to a GL version so recent. Older versions still working pretty fine

glCreateBuffers is useful when working with other functions in GL_ARB_direct_state_access, such as glBindNamedBuffer. Since it's not being used at the moment, there is no difference. See #66.

This was fixed in #137


Problem with running application

Hi
I created application with "Building a public Shopify application" tutorial and this repo.
App works properly locally with ngrok and on server when I start it in SSH console with "ruby app.rb" or "ruby app.rb &".
But when I close console or put another command, application stop working.
I can get it from App store but after clicking "Install app" I have an error:

when I put another command to console:

ActiveResource::UnauthorizedAccess at /giftbasket/auth Failed.
Response code = 401. Response message = Unauthorized.
app.rb in create_order_webhook
unless ShopifyAPI::Webhook.find(:all).any?
sinatra.error:#<ActiveResource::UnauthorizedAccess: Failed. Response code = 401. Response message = Unauthorized.>
sinatra.route: GET /giftbasket/auth

when I close console:

Errno::EIO at /giftbasket/auth
Input/output error @ io_write - <STDERR>
.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/webrick/httpserver.rb in service
si.service(req, res)
sinatra.error:#<ActiveResource::UnauthorizedAccess: Failed. Response code = 401. Response message = Unauthorized.>
sinatra.route: GET /giftbasket/auth

Please help me run the application to work all time.

Hey Pelcomp,
This is a known limitation of this naive example application – when the application dies, all of the tokens stored in local memory will be discarded. A token is needed to authenticate with the shop, hence the 401 error.
In order to have this continue working, you would need to build some logic into the app to redirect back to the auth route when authentication fails. However, this will cause issues with processing webhook data. You may want to look into https://github.com/janko-m/sinatra-activerecord or another method of more permanent token storage.


alsa-utils are install but can't play samples from volsa2-gui app

I can successfully run aplay from the command line, so I know that parts working. But the music note icons in this app are grayed out and unresponsive.

Doh! it only plays the file if on the local drive. Got it.


Refactoring Numeric comparisons and arithmetic operations into NumberOperations

Currently we define basic arithmetic operations (+,-,*,<,<=..) in each instance of Number.
It would be nice to move these implementations from the actual instances of UInt32,UInt64 etc into NumberOperations so we don't have so much boilerplate/redundant code inside of NumberType.scala

Fixed in #99


GitHub pages

There is some sort of GitHub action going on, that generates a GitHub pages site based on main. See https://pcmdi.github.io/input4MIPs_CVs/
I'm not really sure what this is doing, but I think switching to using mkdocs with GitHub pages is the much more modern solution: https://www.mkdocs.org/user-guide/deploying-your-docs/#github-pages
@durack1 do you know anything about this was set up? Is it just a placeholder or is it performing some key function?

Yep, this is just the default GH Pages instance - see https://github.com/PCMDI/input4MIPs_CVs/settings/pages. Keep it as simple as possible is always my preference

Keep it as simple as possible is always my preference

Can do. Just running mkdocs gh-deploy will be pretty simple

This way GH pages works is for each merged commit, the pages are regenerated - no additional step required. If we can batch mkdocs into a merge action then we're at the same point as GH Pages

Is there more than one page on the GH page? At the moment, all I see is the README rendered as a web page, which doesn't seem that helpful, but maybe I am missing something?

When I had this configured, it also published the docs/*html contents, which was already in HTML (not markdown) so didn't have very much to do - you now have a couple of HTML files in docs/ so not sure where this is at, what it is doing

Closing, replaced by more clear issues #86 and #87


Debounce send link code button on Auth0 with a button disable.

@raisedadead Is this still an outstanding issue?

So far we haven't hit any limits. Only concern is the abuse of SES. I would like add the debouncing sometime soon.


can't receive full varbinary data without SET TEXTSIZE

which is a quick fix but whenever i want to save data to a file or share it, it always creates the files as TEXTSIZE size. Like if i set it to 4MB then all of the files are 4MB no matter if it's less then that

Interesting. Do you think the problem is in SQLClient or FreeTDS?


MobileBarcodeScanner

Hi guys.
I have updated the package in my Xamarin Forms app.
In Xamarin Forms I have always used
ZXing.Mobile.MobileBarcodeScanner scanner = new ZXing.Mobile.MobileBarcodeScanner();
scanner.FlashButtonText = "Flash";
scanner.TopText = "";
scanner.BottomText = "";
var result = await scanner.Scan();

and it has always worked well. But today nothing happens... (I am testing it in Android Prj)
I have seen that in your samples you don't use MobileBarcodeScanner but other things (ZXingScannerPage with PushAsync....)
Can I use MobileBarcodeScanner again?

Ok, I have add this and now it works (strange because before updating  the packages, this line of code was not necessary)
MobileBarcodeScanner.Initialize(Application);