Define some Variables

E.g Main Brand Color

Brand Name (maybe this could be the rails app name)
Brand Logo

This way the style guide can be more related to the user's brand

If you place a copy of the engine provided layout file into app/views/layouts/mountain_view.html.erb then Rails should pick that up before the engine layout and use that. You can then modify that as you wish.

@OddEssay, sorry for not replying earlier. I've been away on holidays 100% disconnected the last month :)
That themed Styleguide looks awesome. Is the Styleguide public so I can check it out? If not, do you have a couple of screenshots?
Thanks!!!
Nacho

@devnacho Nice excuse! ;)
We don't have a public version as such, but right now it can be accessed on a test url of https://morning-shore-9193.herokuapp.com/mountain_view/styleguide/ - Which will be there until it's not (Might be a week, might be a month).
At some point, we'll be writing up a blog post (And maybe a talk/presentation for some of the user groups we're involved with) on how we've used components (powered by mountain_view)  and dig into some of the design principles we've used and the logic behind the direction we've taken some of the components.
If there is any bit's you like me to talk over how we did them, I'm more than happy to chat about them,  and if there is a pattern you think might be useful to maybe make a version as an example, I'd be happy to submit a PR with that in it.

Hey @OddEssay thanks for sending the link. The style guide looks awesome. It makes me happy to see how other people are using the tool.
Feel free to put the link to your blog post here when you have it. I'm eager to read about the design principles you used with components and Mt.View.
I don't think I have any particular subject in mind. (Maybe @kitop...anything you want to add?)
Cheers!
Nacho

Would be nice with a rails command to copy over the engine stuff, like rails mountain_view:copy_assets :)

@Frexuz great idea! Do you want to send a PR?

I can try, but not today :) Will start using MV later this week, can try then :)

Awesome, thanks!!!


Add dependobot config file

What does this PR do?
Add a Dependabot config file.
Motivation
Get notified when a new version of a GO module is available.
Additional Notes
Anything else we should know when reviewing?
Describe your test plan
Check dependabot alerts.

Also if I understand correctly, this will make dependabot update all dependencies as soon as they have new versions, regardless of whether they're affected by security advisories or not. Enabling this now would likely create tons of dependabot PRs (or one PR with a huge list of updates) immediately, which we're unlikely to be able to merge.
We should probably start with a more selective approach (example: use dependabot only for select deps we want to always keep on the latest version)

We should probably start with a more selective approach (example: use dependabot only for select deps we want to always keep on the latest version)

Agreed, for now all of tooling can be excluded for example (not a priority).
We should also add the labels option (probably changelog/no-changelog, qa/skip-qa and team/agent-platform would work) and the milestone one (which we will need to keep updated)

Enabling this now would likely create tons of dependabot PRs
What's the alternative? Being stuck with old versions is creating technical debt, and the longer we wait the worse it becomes. Updating everything will be a bit of effort initially and lots of changes but over time it will become a smaller effort as we only do incremental changes.

the milestone one (which we will need to keep updated)
We have a Triage milestone that we can use.


Merge conflicts in committed Solidity file

protocol-units/settlement/mcr/contracts/test/settlement/MCR.sol has been committed with merge conflicts in #369.
Apparently the PR was so big that nobody spotted this in review, and we still don't fully exercise MCR tests in CI as per #482.
The failure occurs in a suzuka-full-node-setup run though, so just about any cicd:* triggered workflow would have discovered the breakage.

Assigned to @l-monninger for the time being, as git blame points on him and I don't currently know which way the conflicts should have been resolved.


Added option for ArtifactsBucket to CFN

This PR adds an optional ArtifactsBucket parameter (and associated policies) to the AWS stack.
This is nice for uploading artifacts outside of buildkite agents...

Many thanks! Nicely done.

Nice!! I'd like to improve the label. So this sets up the correct policy, but doesn't create the bucket or set the environment variable to be used by the agent right? (we should be clear with the secrets bucket too)

Good point @toolmantim, is there an agent setting for setting up a custom bucket? This should absolutely do that. It doesn't create the bucket, but we should document that.

Yep https://buildkite.com/docs/agent/build-artifacts#using-your-own-s3-bucket
export BUILDKITE_S3_DEFAULT_REGION=eu-central-1 # default is us-east-1
export BUILDKITE_S3_ACL=private # default is public-read
export BUILDKITE_ARTIFACT_UPLOAD_DESTINATION=s3://name-of-your-s3-bucket/$BUILDKITE_JOB_ID

And then if you wanted to expose the artifacts as web-clickable straight from Buildkite, and have all their linked/embedded artifacts working (like a coverage html report type thing), you could spin up https://hub.docker.com/r/daveoxley/s3-proxy/ via EB or something, and serve them up with basic auth. If you export BUILDKITE_S3_ACCESS_URL we'll use that host to link to instead of s3.amazon.etc


Add Retina CLI command to create ephemeral container with network debugging tools

Is your feature request related to a problem? Please describe.
I want to be able to do adhoc, exploratory debugging of node and pod networking.

Describe the solution you'd like
kubectl retina sh pods/<pod>
kubectl retina sh nodes/<node>

Start an ephemeral container in the specified node or pod.
Use an image based on Azure Linux with built-in networking tools (ping, curl, nslookup, tcpdump,  conntrack, iproute2, iptables, etc.). If the cluster is using Retina, then it should already have firewall configuration to download this new image from the same source as the other Retina images.
Run an interactive shell (bash).
Assign the appropriate permissions to ensure the tools work (NET_ADMIN and NET_RAW).
(Optionally) mount host filesystem for inspecting CNI logs, which iptables executable is installed on the host, etc. Make this opt-in (--mount-host-filesystem flag) since most network troubleshooting doesn't need it.

Future work could expand this to support Windows (image with PowerShell, use host process containers for node access), but to start I care most about Linux support.
Describe alternatives you've considered

Node-shell gives full access to the node using "nsenter" on PID 1.

Overprivileged: Starts privileged pods on customer nodes (CAP_SYS_ADMIN and access to /proc filesystem).
Alpine Image: Defaults to an alpine image from DockerHub that is often blocked by customer firewalls or subject to DockerHub rate limits. Can override this using an env var, but still requires an image with nsenter installed (and Azure Linux 2.0+ do not have it installed).
No pod debugging: Cannot easily debug connectivity from within a pod network namespace. Users can find the netns for a pod, then use ip netns exec, but this is painful and error-prone.

Kubectl exec allows us to execute a process within a pod's namespace.

Pod image missing tools: Pod images may lack networking tools or even a shell (more common now with distroless images).
Host networking: If there's no host networking pod, we can't test connectivity from node IP.
Pod crashing: If the target pod is crashing, we cannot exec into it.

Kubectl debug creates ephemeral containers to get access to either a node or a pod. It provides "profiles" with different levels of permission.

Missing tools: Requires an image with network debugging tools installed. A popular image is netshoot, but there isn't currently an equivalent image in MCR. People often fallback to running Azure Linux and manually installing whatever packages they need, which is slow and tedious.
Confusing permissions: the "general" profile provides access to the node filesystem, but not NET_ADMIN capability required for iptables/nftables, leading to misleading permissions errors like Permission denied (you must be root) even for the root user. kubectl with k8s 1.27 and later have "netadmin" profile that grants NET_ADMIN capability, but not access to host filesystem.

Additional context
Retina has a unique opportunity to overcome the limitations for the other tools listed above. If people are already using Retina for monitoring/observability, then it would be very convenient to also use it for adhoc network debugging.
This would also pair well with Retina's existing packet capture command. I often find myself starting a packet capture, then using kubectl exec / kubectl debug to reproduce an issue. It would be really nice to do this without needing a separate tool.

Completed


[BUG]

Problem subject
Documentation:
In the file called "README.md" in the Tutorials section.
Description
In the file called "README.md" in the Tutorials section there is a link to a page that no longer works. I have been able to verify that in the README.md file in the C language section of the Tutorials folder there is another link that I think could serve to replace the one that does not work in the main Tuturials section. I have opened a Pull Request changing one link for the other to fix the problem.
Expected behavior/contents
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@NisarLDev
Can you implement this changes on my NEW Account
https://github.com/The-Young-Programer/C-CPP-Programming


$expand is reported as enum while it is actually string

Current behavior:
$expand is enum.
Expected behavior:
$expand is string because you can have complex expressions in $expand like for example $expand=Groups($expand=Members)

If we updated the schema to be like this it would allow us to provide a list of properties that are expandable but also indicate that each expand property could be a more complex syntax.
          schema:
            uniqueItems: true
            type: array
            items:
              anyOf:
                - enum:
                    - '*'
                    - serviceAnnouncement
                - type: string
              type: string

Thoughts?

We do have a hard dependency on that though, GE relies on this to auto completion.
We need to tie this new behaviour to a conversion setting switch (true by default, false for GE) "EmitExpandSelectAndOrderByAsStrings" (or a better name)

I propose we simply project those three as arrays of strings instead. Which will have the additional benefit of making Microsoft Graph descriptions quite smaller too.

+1


missing dependencies for running tests on macos

The following dependencies were required for me to run tests on macos:

gnu tar (gnu-tar in homebrew)
wget
sha1sum (coreutils in homebrew)

They seem to be tied to running the squad instance, but I did not see them listed in any documentation.

Is this squad or squad-client?

It should be squad, but if you try to test squad-client it will come up as well.

Maybe we should add a section for MacOS installs?

I would try to reproduce it in docker container. It should fail equally. If it doesn't it's Mac thing and I won't worry about it. We have too few people to cover all OSes. We also don't test on Windows. Sorry.

Perhaps we could change the squad deployment to use docker for testing squad-client? That would fix a lot of the problems.


webots.min.js: rendering issues on some systems

On my Linux desktop I get some issues when rendering specific objects, for example:

TruckSimple in the highway_driving benchmark

both BungalowStyleHouse objects in the humanoid_marathon

I tested both on Chrome and Firefox and I get the same issue.
The black dots are visible both the in the robotbenchmark animation and simulation.
@fabienrohrer tested on his macOS Chrome/Firefox and works correctly.

Operating System: Linux Ubuntu 18.04
Graphics Card: NVIDIA GeForce RTX 2060 6 GB

I tried to check if it was a Z-fighting issue and I enabled the logarithmic depth buffer, increased the camera near value and decreased the far value but none of these changes solved the issue.

This issue is also visible with the Door object, so I could easily simplify the world:
this is related to IndexedFaceSet (TexturedBox or any simple cube) and the PBRAppearances.normalMap.

This seems to be due to texture mapping.
Default IFS texture mapping shows the texture on "front"/"back" faces and the border value is copied along the whole side faces.
I get the issue on the side faces.
But if I set a custom texture mapping where an area (and not a line) of the texture is shown on each face then it works fine without black artifacts.

I can reproduce the exact same problem:
System: Linux 4.4.0-157-generic x86_64
Processor: Intel(R) Core(TM) i7-4790K CPU @ 4.00GHz
Number of cores: 8
OpenGL vendor: NVIDIA Corporation (0x10de)
OpenGL renderer: GeForce GTX 970/PCIe/SSE2 (0x13c2)
OpenGL version: 4.5.0 NVIDIA 384.130
Firefox

Just for note I reported the issue in the three.js GitHub (issue  #17559) but they are not interested in fixing it.
Here is the answer:

Unfortunately seems to be a bug in the driver and/or device. You may want to report the issue to Nvidia instead.


Snowplow extension 1.0.2

This PR is for reviewing purposes currently - PR will be made to amphtml repo once approved.
Rebasing and squashing is a bit of a nightmare as all the changes are in one file. The AMP project squashes them together on merge anyway so I think we're ok to do it this way.
Changes:

Fix bug where duid is not sent from AMP pages

The JS tracker's cross-domain linker passes duid to AMP pages, but for pages served from the AMP CDN these weren't being attached to events. This is caused by the fact that we set a cookie and retrieved the value from that cookie - but cookies can't always be set from the CDN.
Fixed by using the value directly from the querystring when we try to send data.
Also we prioritise the value from the querystring in setting the cookie, since shared devices may have changing duids.

Rename _sp_id cookie

This was a poor naming choice as it's the same cookie name that the JS tracker uses, but a different ID.
We avoid changing all AMP ID values, setting the new cookie to the value of the _sp_id cookie when the new one doesn't exist.
We only do this when it doesn't exist, to avoid accidentally writing the wrong value in when it does exist.
If the cookie does exist we attempt to use the AMP linker value that we set in the tracker. If that doesn't exist behaviour defaults to the AMP CLIENT_ID() macro, which will set the cookie for us.

Rename 'linker' to 'sp_amp_linker'

Another poor naming choice - using the default opens the door for conflicts. Ripping off the plaster now while we're at it.

Rename aaVersion to trackerVersion

Just sensible to have the variable named to what it is.

Can we adjust this url to docs with this PR? https://github.com/ampproject/amphtml/blame/7bfa178669f61d5151343c24ddcb0c452bf97384/extensions/amp-analytics/analytics-vendors-list.md#L467

I think that addresses everything. :)

Closing in order to create PR into upstream project

https://github.com/ampproject/amphtml/pull/32636


Remove instance variables in generated specs

Problem
I noticed some generated scaffold specs contain explicit instance variable gets and sets. But rubocop-rspec gem, at least v2.10.0, prefers no instance variable usages in specs by default (RSpec/InstanceVariable cop, introduced in 2014 https://github.com/rubocop/rubocop-rspec/pull/3).
In fact, I confirmed that some specs such as spec/views/posts/edit.html.erb_spec.rb generated by bin/rails g scaffold post are offended by rubocop.
What I Changed
Therefore, I removed usage of instance variables from generated specs.
For historians, commits in 2010 (f06aa507280f8f9e6316c9bd5f72e640269a49b1 and 0d244a1fbd43427a83021c189f7bf6cc7d085457) introduce the use of instance variables, maybe without strong reasons.
Changes to Specs
The following table is a summary of changes to generated edit_spec.rb. I generated these results on my local environment with Rails <IP_ADDRESS> and rspec-{core,expectations,mocks,support} 3.12.0.pre. Specs in "After" column do not contain @post.

Context
Before
After

Without attributes

require 'rails_helper'

RSpec.describe "posts/edit", type: :view do
  before(:each) do
    @post = assign(:post, Post.create!())
  end

  it "renders the edit post form" do
    render

    assert_select "form[action=?][method=?]", post_path(@post), "post" do
    end
  end
end

require 'rails_helper'

RSpec.describe "posts/edit", type: :view do
  let(:post) {
    Post.create!()
  }

  before(:each) do
    assign(:post, post)
  end

  it "renders the edit post form" do
    render

    assert_select "form[action=?][method=?]", post_path(post), "post" do
    end
  end
end

With an attribute

require 'rails_helper'

RSpec.describe "posts/edit", type: :view do
  before(:each) do
    @post = assign(:post, Post.create!(
      content: "MyText"
    ))
  end

  it "renders the edit post form" do
    render

    assert_select "form[action=?][method=?]", post_path(@post), "post" do

      assert_select "textarea[name=?]", "post[content]"
    end
  end
end

require 'rails_helper'

RSpec.describe "posts/edit", type: :view do
  let(:post) {
    Post.create!(
      content: "MyText"
    )
  }

  before(:each) do
    assign(:post, post)
  end

  it "renders the edit post form" do
    render

    assert_select "form[action=?][method=?]", post_path(post), "post" do

      assert_select "textarea[name=?]", "post[content]"
    end
  end
end

With two attributes

require 'rails_helper'

RSpec.describe "posts/edit", type: :view do
  before(:each) do
    @post = assign(:post, Post.create!(
      content: "MyText",
      priority: 1
    ))
  end

  it "renders the edit post form" do
    render

    assert_select "form[action=?][method=?]", post_path(@post), "post" do

      assert_select "textarea[name=?]", "post[content]"

      assert_select "input[name=?]", "post[priority]"
    end
  end
end

require 'rails_helper'

RSpec.describe "posts/edit", type: :view do
  let(:post) {
    Post.create!(
      content: "MyText",
      priority: 1
    )
  }

  before(:each) do
    assign(:post, post)
  end

  it "renders the edit post form" do
    render

    assert_select "form[action=?][method=?]", post_path(post), "post" do

      assert_select "textarea[name=?]", "post[content]"

      assert_select "input[name=?]", "post[priority]"
    end
  end
end

Is it the only change needed for generated specs to pass the RuboCop check?
I can think of e.g. RSpec/HookArgument cop that might be triggered by before(:each), as its default enforced style is "implicit",  and it is enabled by default.

Is it the only change needed for generated specs to pass the RuboCop check?

Nice point. No, other cops such as Style/StringLiterals by rubocop gem are triggered. Almost all triggered cops are, however, able to auto-corrected by rubocop. The only exceptions I found are RSpec/InstanceVariable cop and Metrics/BlockLength cop.
So I want to change generated codes beforehand for RSpec/InstanceVariable. Also I don't touch for Metrics/BlockLength because many generated specs can be offended by that cop when attributes are too many (and I personally prefer to ignore Metrics/BlockLength for specs).
This is why I only changed usage of instance variables in this pull request. Thank you for pointing it out.

Thank you!


Manually include composer autoload

At the moment the autoloader is included by framework inside framework/core/Core.php
This poses problems when we need to include composer managed packages before Core.php is included. I've addressed this issue here https://github.com/silverstripe/silverstripe-framework/pull/5572 by moving the autoloader include to as early in the execution as possible.
To get the behat tests passing we need to include the autoloader as early as early as possible in the behat bootstrap.
This problem was solved for PHPUnit by adding the include to the cli-script.php in core and I expect there's a similar file we can do this in for this module.

This might be relevant: http://stackoverflow.com/questions/28852162/how-to-require-vendor-autoload-php-with-behat-yml

Composer's autoloader is used by default.

Yep, looks like a red herring


Dockerized, added license, added kerberos.

I quite enjoy running the wallet service under docker.  I think others might want to as well.
Included in this pull request are:

A Dockerfile to facilitate building the docker container
A .dockerignore file that ensures we do not copy too much into the container.
Some markdown to help bootstrap the container experience.
A patch to the package.json to help it stop complaining about the lack of a license.
A patch to the package.json to include kerberos as recommended in recent mongodb documentation

Upon further review, this is not very docker like.  I am going to re-work this to have individual running containers for each of the running services.  At that point and once I have it up and running on a large cluster I will re-submit the pull request.


Moving server FAQ from cepdnaclk.github.io to faq.ce.pdn.ac.lk

Move https://github.com/cepdnaclk/cepdnaclk.github.io/tree/master/servers to https://faq.ce.pdn.ac.lk/network-n-servers/
Delete the first one.
Inform Dr. @damayanthiHerath about this change so she can update the link on ce.pdn.ac.lk. >> about us >> Facilities >> GPU Server.

TODO:

[ ] Move content from https://ce-pdn-ac-lk.com/cewiki/server_use:use_of_servers to here

Added a redirection from https://cepdnaclk.github.io/sites/servers/ to https://faq.ce.pdn.ac.lk/network-n-servers/


'symbolic-immutables' attribute ignored in the kontrol.toml file

Adding symbolic-immutables = true in the kontrol.toml file does not have any effect. There might be other flags that are ignored.

The kontrol.toml file has symbolic-immutables = true. Based on its definition in KEVM, the flag should set the break_on_load_program variable to True.
After merging #778, I used -debug to see what options are computed from the kontrol.toml parsing.
ProveOptions(..., break_on_load_program: False, ..., symbolic_immutables: True,...)

It looks like the parsing of kontrol.toml rewrites from symbolic-immutables to symbolic_immutables instead of break_on_load_program.


scope issue of '_linear' method in 'tensorflow.python.ops.rnn_cell_impl' module

In 'core_rnn_cell_impl' module _linear method, the arg 'scope' is not used in the method, easily causing variable conflicts. eg:
with tf.variable_scope(scope):
zw = core_rnn_cell_impl._linear(self.hyper_output, self.hyper_embedding_size, False, scope=scope + "z")
alpha = core_rnn_cell_impl._linear(zw, dimensions, False, scope=scope + "alpha")
zw, alpha(forgive me for the indent in format) although defined in their specific scope, they will collide each other for using the same variable name.
I look up the 'core_rnn_cell_impl' code and I find the 'scope' arg has not been truly used.
This is the 1.0.0 version:
scope = vs.get_variable_scope()
with vs.variable_scope(scope) as outer_scope:
This is the 0.12.0 version:
with vs.variable_scope(scope or "Linear"):
Please check if it's truly the problem, thanks a lot !

@ebrevdo, could you comment please.

yes... it's an internal method and isn't meant for external use.  i think
we got rid of the scope= argument in master branch.
On Tue, Mar 28, 2017 at 4:28 PM, Andrew Selle<EMAIL_ADDRESS>wrote:

@ebrevdo https://github.com/ebrevdo, could you comment please.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/8777#issuecomment-289935941,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtim8eMrP1HXhMmCx_MMlCnohxGnhaHks5rqZeWgaJpZM4Mrtu_
.

Thanks


HDDS-5074. Bump Guava version

JIRA: https://issues.apache.org/jira/browse/HDDS-5074
What changes were proposed in this pull request?
Guava has a tmp directory related CVE (CVE-2020-8908).
Based on my best knowledge Ozone is not affected, but it's hard to explain this situation for all the automated tools. Let's just bump the version to the latest one...
How was this patch tested?
Full CI passed.
And checked if guava jar is really updated in the final distribution:

LGTM
I just want to say that the description of that alert isn't accurate. Simply updating to guava 30 doesn't help. It requires removing the API call usage in the code. Fortunately we don't seem to use it in Ozone.

Thanks, the review @adoroszlai and @jojochuang. Agree with the comment, it's not a security fix. Just makes less  false-positive problems reported... Merging it now...


crates.fyi links don't work anymore

It would be nice if we could still replace the crates.io with crates.fyi to find the correct docs.rs page. E.g.
https://crates.fyi/crates/serde/ -> https://docs.rs/serde/.

crates.fyi links should be working fine now


[Feature Request] How to get execution breakup (GPU vs CPU) when running llama model on android using OPENCL=ON

Hi,
I am running, llama model with q8f16_1 quantization on android device (build from source, both relax and mlc-llm).
It run successfully.
I would like to know, How to check the execution break up( which layers really ran on GPU? which layers fall back to CPU?).
Or is it like, when i use DUSE_OPENCL=ON, all the layers will run on GPU?
Regards

MLC-LLM runs the whole model on GPUs. We do not have CPU support right now

@Ramees025 may I ask if you could share the binaries and apk?

@Hzfengsy Do you have plan to support Android CPU backend? The GPU is used by other display process in our system, so we hope  MLC-LLM can run on CPU.

Closing as original question has been answered. As of now we are focusing on GPU and possibly NPU. CPU can in theory be supported through tvm but there is no specific plan to optimize yet.


Fix month view force unwrap crash

I think this should do it

You are right. Man, this is really confusing. I'm bouncing around 4 different repos. I just cherry-picked the commit. I hope this is all of the stuff

I just tested this PR with tap inspect on the main branch. We should be good to merge.

Missed one: https://github.com/TapInspect/JTAppleCalendar_OLD/commit/3edf8fba3d9314ff5e42626c945031668e54dd47

Dude, it's already included in this PR. It's the first commit.


cache FileSystem.SafeExists and FileSystem.GetLastWriteTimeShim during single LanguageService.ParseAndCheckFileInProject

It's temporary fix for https://github.com/fsprojects/VisualFSharpPowerTools/issues/1280

The issue was caused by FCS <IP_ADDRESS>, in https://github.com/fsprojects/VisualFSharpPowerTools/pull/1288 it's rolledback to 1.4.2


Can't pass through snow carpets placed on top of leaves

First of all, fantastic mod ! I really enjoy playing with it, it's just so much more entertaining than solid leaves.
To the matter that brings us here: you cannot climb through snow carpets placed on leaves, for example, when snow fell due to weather, which feel wrong as that snow carpet is very thin and should probably be able to be climbed through. similarly, if you are inside a leaves block and try to walk towards another one which is covered in snow, you'll hit it like a wall and won't be able to walk past it.
I have two possible solutions for this:

Make it so snow carpets are non-solid if they are placed on top of leaves blocks
Make it so snow carpets break into snowballs when a player's hitbox touches them, only if they are placed on top of leaves blocks.

I think I like your first suggestion more and I think it's feasible with a little bit of trickery. The idea is to register a non-solid version of snow that drops regular snow when players dig it up. This would also require some code to check for snow on leaves and convert it to non-solid snow. This gets the best of both worlds by allowing for soft snow on leaves while letting snow remain as it is elsewhere.
I'll take this under consideration for a future feature. I don't think it'll actually be too difficult to implement but I would need to do some testing.
Thanks so much for the report and I'm glad to hear that you're enjoying the mod!

You're welcome 🤗
Coming back to this, i think why i suggested the second option is because it would make for a pretty fun interaction and reaction, espicialy in multiplayer, seeing your friends's head pop up and break through the snow on top of a tree !
i understand it may pollute your inventory, but if you're climbing trees instead of building stuff, you probably don't car much about an optimized inventory anyway 😄
Anyway, whatever suits you best, as long as that issue is solved ! I figured if you didn't answes it could have been a way for me to get into Lua and try my hands at coding this myself. I'll take another opportunity then !


"Attempting to parent and popup a dialog that already has a parent" error raised for each asset import

Godot version
4.1.dev (dc625bcbfc5b587ba8867481ee7a1f6a567c25d0)
System information
Mageia 9, Linux
Issue description
When importing a project in current master, the following error is raised for each asset that gets imported:
Attempting to parent and popup a dialog that already has a parent.

It seems to be raised once per asset, caused by the progress dialog.
CC @bruvzg
Steps to reproduce

Import a new project in Godot that contains importable assets, e.g. https://github.com/RPicster/godot4-demo-desert-light

Minimal reproduction project
See above.

Seems to be a regression from #76025, CC @YuriSizov.
(Didn't do a clean bisect locally, but I downloaded a bunch of recent GHA artifacts and narrowed down on this which seemed a likely culprit. It was merged in a batch so I don't have a build from the direct previous commit to compare, but 9163d8c33650d4bb181a59b8d9838ebc61d5ecbb merged in the same batch and rebased a few hours before doesn't have the bug, while #76025 does).

I am having this issue again in 4.1.stable, July 6 2023 release.
OS: Pop!_OS 22.04

same problem on Linux, godot 4.1.1
trying to export android apk
  Attempting to parent and popup a dialog that already has a parent.
param: --xr_mode_regular
param: --use_immersive
  editor/export/editor_export_platform.h:178 - Export: Building of Android project failed, check output for the error. Alternatively visit docs.godotengine.org for Android build documentation.

I have this issue too in the 4.1.1
OS: Windows11 JDK: JDK-20
* Attempting to parent and popup a dialog that already has a parent.
0 param: --xr_mode_regular
1 param: --use_immersive
* editor/export/editor_export_platform.h:178 - Export: Building of Android project failed, check output for the error. Alternatively visit docs.godotengine.org for Android build documentation.```

i got the same message. i use Godot 4.1.1

why is this closed, did someone found solution i am still getting this error

Error opening file ''.
Attempting to parent and popup a dialog that already has a parent.
Exporting Poing AdMob '.cfg' file
0 param: --xr_mode_regular
1 param: --use_immersive
editor/export/editor_export_platform.h:178 - Export: Building of Android project failed, check output for the error. Alternatively visit docs.godotengine.org for Android build documentation.

why is this closed, did someone found solution i am still getting this error

That's a separate issue from the one reported and resolved here. As I asked responders before you, you should open a dedicated report to this.


Parcelable auto value entities are being generated incorrectly with the 1.0.0 version

When updating auto value moshi dependencies from 0.4.7 to 1.0.0 the auto value entities which are marked as Parcelable started failing (does not compile). Tests are done using the latest version (0.2.8) of auto-parcel. It seems that new version (1.0.0) is adding redundant $ sign while calling a constructor in one of the generated classes.
Here is a simple auto value class:
@AutoValue
public abstract class Foo implements Parcelable {
    abstract String bar();

    public static JsonAdapter<Foo> jsonAdapter(Moshi moshi) {
        return new AutoValue_Foo.MoshiJsonAdapter(moshi);
    }
}

Here is the generated $AutoValue_Foo class:
A comment is added to the line that has the wrong constructor call. It seems that $AutoValue_Foo(bar) should be AutoValue_Foo(bar).
abstract class $AutoValue_Foo extends $$AutoValue_Foo {
  $AutoValue_Foo(String bar) {
    super(bar);
  }

  public static final class MoshiJsonAdapter extends JsonAdapter<Foo> {
    private static final String[] NAMES = new String[] {"bar"};
    private static final JsonReader.Options OPTIONS = JsonReader.Options.of(NAMES);
    private final JsonAdapter<String> barAdapter;
    public MoshiJsonAdapter(Moshi moshi) {
      this.barAdapter = moshi.adapter(String.class).nonNull();
    }
    @Override
    public Foo fromJson(JsonReader reader) throws IOException {
      reader.beginObject();
      String bar = null;
      while (reader.hasNext()) {
        switch (reader.selectName(OPTIONS)) {
          case 0: {
            bar = this.barAdapter.fromJson(reader);
            break;
          }
          case -1: {
            // Unknown name, skip it
            reader.skipName();
            reader.skipValue();
          }
        }
      }
      reader.endObject();
      return new $AutoValue_Foo(bar); // $AutoValue_Foo(bar) -> AutoValue_Foo(bar)
    }
    @Override
    public void toJson(JsonWriter writer, Foo value) throws IOException {
      writer.beginObject();
      writer.name("bar");
      this.barAdapter.toJson(writer, value.bar());
      writer.endObject();
    }
    @Override
    public String toString() {
      return new StringBuilder().append("JsonAdapter(").append("Foo").append(")").toString();
    }
  }
}

Here is the generated code with old version (0.4.7) using the same auto value class definition.
abstract class $AutoValue_Foo extends $$AutoValue_Foo {
  $AutoValue_Foo(String bar) {
    super(bar);
  }

  public static final class MoshiJsonAdapter extends JsonAdapter<Foo> {
    private static final String[] NAMES = new String[] {"bar"};
    private static final JsonReader.Options OPTIONS = JsonReader.Options.of(NAMES);
    private final JsonAdapter<String> barAdapter;
    public MoshiJsonAdapter(Moshi moshi) {
      this.barAdapter = adapter(moshi, String.class);
    }
    @Override
    public Foo fromJson(JsonReader reader) throws IOException {
      reader.beginObject();
      String bar = null;
      while (reader.hasNext()) {
        switch (reader.selectName(OPTIONS)) {
          case 0: {
            bar = this.barAdapter.fromJson(reader);
            break;
          }
          case -1: {
            // Unknown name, skip it
            reader.skipName();
            reader.skipValue();
          }
        }
      }
      reader.endObject();
      return new AutoValue_Foo(bar); // This is correct constructor call
    }
    @Override
    public void toJson(JsonWriter writer, Foo value) throws IOException {
      writer.beginObject();
      writer.name("bar");
      this.barAdapter.toJson(writer, value.bar());
      writer.endObject();
    }
    private JsonAdapter adapter(Moshi moshi, Type adapterType) {
      return moshi.adapter(adapterType);
    }
  }
}

This looks to be an issue here:
https://github.com/rharter/auto-value-moshi/blob/master/auto-value-moshi-extension/src/main/java/com/ryanharter/auto/value/moshi/AutoValueMoshiExtension.java#L733
That line should be using the autoValueClassName, not the className. I'm on
the road for the weekend so can't take care of it, but PRs are welcome.
On Thu, Feb 27, 2020, 4:00 PM Hakan Bagci<EMAIL_ADDRESS>wrote:

When updating auto value dependencies from 0.4.7 to 1.0.0 the auto value
entities which are marked as Parcelable started failing (does not
compile). Tests are done using the latest version (0.2.8) of auto-parcel.
It seems that new version (1.0.0) is adding redundant $ sign while
calling a constructor in one of the generated classes.
Here is a simple auto value class:
@AutoValuepublic abstract class Foo implements Parcelable {
abstract String bar();
public static JsonAdapter<Foo> jsonAdapter(Moshi moshi) {
    return new AutoValue_Foo.MoshiJsonAdapter(moshi);
}

}
Here is the generated $AutoValue_Foo class:
A comment is added to the line that has the wrong constructor call. It
seems that $AutoValue_Foo(bar) should be AutoValue_Foo(bar).
abstract class $AutoValue_Foo extends $$AutoValue_Foo {
$AutoValue_Foo(String bar) {
super(bar);
}
public static final class MoshiJsonAdapter extends JsonAdapter {
private static final String[] NAMES = new String[] {"bar"};
private static final JsonReader.Options OPTIONS = JsonReader.Options.of(NAMES);
private final JsonAdapter barAdapter;
public MoshiJsonAdapter(Moshi moshi) {
this.barAdapter = moshi.adapter(String.class).nonNull();
}
@Override
public Foo fromJson(JsonReader reader) throws IOException {
reader.beginObject();
String bar = null;
while (reader.hasNext()) {
switch (reader.selectName(OPTIONS)) {
case 0: {
bar = this.barAdapter.fromJson(reader);
break;
}
case -1: {
// Unknown name, skip it
reader.skipName();
reader.skipValue();
}
}
}
reader.endObject();
return new $AutoValue_Foo(bar); // $AutoValue_Foo(bar) -> AutoValue_Foo(bar)
}
@Override
public void toJson(JsonWriter writer, Foo value) throws IOException {
writer.beginObject();
writer.name("bar");
this.barAdapter.toJson(writer, value.bar());
writer.endObject();
}
@Override
public String toString() {
return new StringBuilder().append("JsonAdapter(").append("Foo").append(")").toString();
}
}
}
Here is the generated code with old version using the same auto value
class definition.
abstract class $AutoValue_Foo extends $$AutoValue_Foo {
$AutoValue_Foo(String bar) {
super(bar);
}
public static final class MoshiJsonAdapter extends JsonAdapter {
private static final String[] NAMES = new String[] {"bar"};
private static final JsonReader.Options OPTIONS = JsonReader.Options.of(NAMES);
private final JsonAdapter barAdapter;
public MoshiJsonAdapter(Moshi moshi) {
this.barAdapter = adapter(moshi, String.class);
}
@Override
public Foo fromJson(JsonReader reader) throws IOException {
reader.beginObject();
String bar = null;
while (reader.hasNext()) {
switch (reader.selectName(OPTIONS)) {
case 0: {
bar = this.barAdapter.fromJson(reader);
break;
}
case -1: {
// Unknown name, skip it
reader.skipName();
reader.skipValue();
}
}
}
reader.endObject();
return new AutoValue_Foo(bar); // This is correct constructor call
}
@Override
public void toJson(JsonWriter writer, Foo value) throws IOException {
writer.beginObject();
writer.name("bar");
this.barAdapter.toJson(writer, value.bar());
writer.endObject();
}
private JsonAdapter adapter(Moshi moshi, Type adapterType) {
return moshi.adapter(adapterType);
}
}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rharter/auto-value-moshi/issues/173?email_source=notifications&email_token=AAJ4S3U2K3ILUYENTB52JTTRFAZYXA5CNFSM4K5EEGSKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IQ5572A,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJ4S3R7V3RQ4Y3GZTM7WCDRFAZYXANCNFSM4K5EEGSA
.

@rharter Thanks for the fix recommendation. Using autoValueClassName does not seem to fix the problem. I have checked the createReadMethod from AutoValueMoshiExtension.java in version 0.4.7.  I applied the similar approach from the following line and it seem that it fixes the problem.
https://github.com/rharter/auto-value-moshi/blob/ef6bcd70733e3570c7ee070d30bdea918003ce87/auto-value-moshi/src/main/java/com/ryanharter/auto/value/moshi/AutoValueMoshiExtension.java#L548

I think auto-value-moshi is working as intended here. It's hard to say as your issue seems to be with whatever "auto-parcel" is, but we are just invoking the constructor we generated. Can you give the full error and why it should be trying to shortcut to a class name we're not given? That's fairly brittle and almost certainly not correct in the way you have PR'd it.

@ZacSweers I understand that this in not the perfect solution. This was the working solution in 0.4.7 and I have just copied it to here. Here is the link to the working solution.
https://github.com/rharter/auto-value-moshi/blob/ef6bcd70733e3570c7ee070d30bdea918003ce87/auto-value-moshi/src/main/java/com/ryanharter/auto/value/moshi/AutoValueMoshiExtension.java#L548
Since you have better know-how in auto-value-moshi and auto-value_parcel it would be better you propose a better solution. But it is kinda obvious that when AutoValueMoshiExtension class is rewritten it is broken because the outputs are not same any more. You can check again the outputs  of 0.4.7 and 1.0.0 from the first comment.
I think auto-value-moshi should not make an assumption about single hierarchy. There might be other abstract classes in the hierarchy while generating the code as in the case of Parcelables.
Here is the full error code (Compile error) when I try to generate code for the Foo class in first comment.
.../entity/$AutoValue_Foo.java:42: error: $AutoValue_Foo is abstract; cannot be instantiated
      return new $AutoValue_Foo(bar);
             ^

Here is the generated class:
package de.meinfernbus.network.entity;

import com.squareup.moshi.JsonAdapter;
import com.squareup.moshi.JsonReader;
import com.squareup.moshi.JsonWriter;
import com.squareup.moshi.Moshi;
import java.io.IOException;
import java.lang.Override;
import java.lang.String;
import java.lang.StringBuilder;

abstract class $AutoValue_Foo extends $$AutoValue_Foo {
  $AutoValue_Foo(String bar) {
    super(bar);
  }

  public static final class MoshiJsonAdapter extends JsonAdapter<Foo> {
    private static final String[] NAMES = new String[] {"bar"};
    private static final JsonReader.Options OPTIONS = JsonReader.Options.of(NAMES);
    private final JsonAdapter<String> barAdapter;
    public MoshiJsonAdapter(Moshi moshi) {
      this.barAdapter = moshi.adapter(String.class).nonNull();
    }
    @Override
    public Foo fromJson(JsonReader reader) throws IOException {
      reader.beginObject();
      String bar = null;
      while (reader.hasNext()) {
        switch (reader.selectName(OPTIONS)) {
          case 0: {
            bar = this.barAdapter.fromJson(reader);
            break;
          }
          case -1: {
            // Unknown name, skip it
            reader.skipName();
            reader.skipValue();
          }
        }
      }
      reader.endObject();
      return new $AutoValue_Foo(bar);
    }
    @Override
    public void toJson(JsonWriter writer, Foo value) throws IOException {
      writer.beginObject();
      writer.name("bar");
      this.barAdapter.toJson(writer, value.bar());
      writer.endObject();
    }
    @Override
    public String toString() {
      return new StringBuilder().append("JsonAdapter(").append("Foo").append(")").toString();
    }
  }
}

It should not call $AutoValue_Foo(bar) but AutoValue_Foo(bar) in return statement of fromJson method.

I see. And there's no way to get AutoValue_Foo from the auto value extension API? That would be surprising

I see. And there's no way to get AutoValue_Foo from the auto value extension API? That would be surprising

Actually I could not find. If I am not mistaken, in current context  className  is $AutoValue_Foo and autoValueClassName is Foo, but we are looking for AutoValue_Foo. I think removing the leading $s looks like a solution since it is guaranteed that it will not call the abstract class constructor in any case , but I accept that is not the best.

Thing is we're not guaranteed That the one without the dollar signs is it

I am assuming that the purpose of auto-value-moshi is to generate an abstract class that injects serialization/deserialization methods. This abstract class starts with $, but the one which is being generated by auto-value does not have $ sign and it is always final. I think in this case calling the constructor without $ sign will be more guarenteed solution. The only concrete class in the hierarchy is AutoValue_Foo in all the scenarios whether we have extra Parcelable or not. Correct me if I my assumption is wrong.

I think @hakanbagci is correct here. Unless things have changed since my original extension architecture, the concrete classes guaranteed to be without a dollar sign, and extensions generate intermediate abstract classes with the dollar sign. The Auto value documentation requires that to instantiate your own instance of an auto value generated class.


Propagate Terraform Cloud Failures to CodeBuild

Describe the outcome you'd like
Currently, AFT CodeBuild jobs which run account customization and global customization Terraform simply start Terraform Cloud workspace runs and then exit successfully when Terraform Cloud or Terraform Enterprise are used as AFT's Terraform distribution.
This means that if a Terraform Cloud workspace fails to plan or apply, this issue is not reflected in CodeBuild or CodePipeline.
Ideally, the CodeBuild jobs in question should error out if the Terraform Cloud / TFE workspaces they run experience a failure.
Is your feature request related to a problem you are currently experiencing? If so, please describe.
Yes, currently we have to check Terraform Cloud manually to verify that customization changes deployed successfully.
Additional context
I was able to get Terraform Cloud errors to propagate properly using a fork of this repository. This requires a small change to workspace_manager.py which is shown here: https://github.com/aws-ia/terraform-aws-control_tower_account_factory/compare/main...addefisher:main

@addefisher thank you for reaching out and for providing the details.
I have gone ahead and created a backlog to address this request.


Shader Editor is always focused breaks shortcuts on Inspector

x11
https://github.com/godotengine/godot/commit/45e7306b5adec09746fefda902a5414e5b53fdfa
Shader Editor is always focused breaks shortcuts on Inspector
Reproduce:

Open Shader editor
Try to duplicate or delete a Node with CTRL+D or CTRL+ENTF
See changes applied to shader editor.

Seems related - #12916

See also #17591

I don't think it is related to an old issue.
When a shader material is open you can not leave the editor. Soit is not possible to edit the shader parameters which makes shader materials basically unuseable. Must have happend with yesterdays commits.
Steps to reproduce:

create a ShaderMaterial
load a shader file
try to edit shader parameters

Another related issue: #21592

Likely related to 472c94ce3 or 5b77aa6be. @reduz

looks good, thanks


parameter wrongly parsed as bit_typespec instead of logic_typespec

Test: FSMBsp13
https://github.com/chipsalliance/Surelog/blob/adbf40797649f890fdd79e35b6e6ffe30fdb2c72/tests/FSMBsp13/fsm1.v#L7-L10
https://github.com/chipsalliance/Surelog/blob/adbf40797649f890fdd79e35b6e6ffe30fdb2c72/tests/FSMBsp13/FSMBsp13.log#L6729-L6748

Both are wrong, the correct answer is unsigned:
6.20.2 Value parameters
A parameter constant can have a type specification and a range specification. The type and range of
parameters shall be in accordance with the following rules:
— A parameter declaration with no type or range specification shall default to the type and range of the
final value assigned to the parameter, after any value overrides have been applied. If the expression
is real, the parameter is real. If the expression is integral, the parameter is a logic vector of the
same size with range [size-1:0].
— A parameter with a range specification, but with no type specification, shall have the range of the
parameter declaration and shall be unsigned. The sign and range shall not be affected by value
overrides.
— A parameter with a type specification, but with no range specification, shall be of the type specified.
A signed parameter shall default to the range of the final value assigned to the parameter, after any
value overrides have been applied.
— A parameter with a signed type specification and with a range specification shall be signed and shall
have the range of its declaration. The sign and range shall not be affected by value overrides.
— A parameter with no range specification and with either a signed type specification or no type
specification shall have an implied range with an lsb equal to 0 and an msb equal to one less than the
size of the final value assigned to the parameter.
— A parameter with no range specification, with either a signed type specification or no type
specification, and for which the final value assigned to it is unsized shall have an implied range with
an lsb equal to 0 and an msb equal to an implementation-dependent value of at least 31.

Fixed by #2798


Implement fallback behaviour for version specific manifests

In addition to version specific manifests which match the current version, we will also consider a version specific manifest of the previous major version. If that one uses a newer tools version than the regular manifest, we'll prefer that.
rdar://problem/46189034

@swift-ci please smoke test

Actually, I can reproduce it, it works when running the tests inside Xcode, though 🤔

Ah, Versioning.currentVersion.displayString is different in that configuration.

@swift-ci please smoke test


Implement embedded-hal 1.0 traits

Now that embedded-hal 1.0 is released, this PR adds implementations of most of its relevant traits. For now I've added them alongside the existing embedded-hal 0.2 trait implementations, so that users can still use drivers based on the old traits. We can remove the old traits in a later release.
I haven't implemented the embedded-hal-nb and embedded-hal-async traits yet, and the I2C traits are also missing as the design has changed and implementing it correctly is non-trivial.

Hi, could I get a review of this soon? It's blocking me from updating some other things to embedded-hal 1.0.

@qwandor Hi, that depends a bit on: https://github.com/nrf-rs/nrf-hal/issues/432
In essence, this crate has no active maintainers...

I'll try these out with some of my stuff over the weekend and let you know if I see anything. I have only reviewed a couple of these so far — will get to the rest shortly.


inputs_depleted error when sending tx / estimating fee / running coin selection on wallet with total balance = 0

Context
The error used to be not_enough_money, now it is:
{"code":"inputs_depleted"
,"message":"I cannot select enough UTxO from your wallet to construct an adequate transaction. Try sending a smaller amount or increasing the number of available UTxO."}

Information
-

Version
2020-06-07

Platform
Linux, all

Installation
from source

Steps to Reproduce

Try to send tx / estimate fee or run random coin selection on either Shelley or Byron wallet with total balance = 0 (i.e. UTxO = 0 and rewards = 0)

Expected behavior
not_enough_money

Actual behavior
{"code":"inputs_depleted"
,"message":"I cannot select enough UTxO from your wallet to construct an adequate transaction. Try sending a smaller amount or increasing the number of available UTxO."}

Resolution

#1896

QA

After fixing the bug, I had to revert back some error messages in integration tests that were changed during the withdrawals introduction. Messages are now back to what they used to be, which the behavior detailed above.

lgtm


removing "dependencies" from config results in run failure

2.19.2 released

nice, works now :)


Create  initial sql script for yig backend before yig backend can be registered

Is this a BUG REPORT or FEATURE REQUEST?:
This is a BUG REPORT, and to track the initial sql script for yig backend database, and this script can be used before yig backend is registered.
this issue is to track the #709

Uncomment only one, leave it on its own line:
/kind bug
/kind feature

What happened:
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Gelato(release/branch) version:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

this issue has been solved. pls see the comments.


[feature request] how to sync lines in free drawing mode ?

set fabric isDrawingMode = true
user1  drawing  on webpage1
can user2 receive the drawing data from user1 , then render it to user2's webpage2 and keep the drawing animation?

Hey, this actually sounds like fun and is possible. Only problem could be performance, but I think one could solve this problem by a clever update method.
If I have time this weekend I'm going to implement it and update the repository.


I got blank white screen on my ios release

it's work by using "flutter run" on my iphone
but when I do "flutter build ios --release" and open xcode11.1 to test
it will show blank white screen on my iphone
And I archive to app store , I test on  TestFlight, it also has the issues
when i run in xcode:

2019-10-31 16:56:49.336432+0800 Runner[3397:1036892]  - <AppMeasurement>[I-ACS036002] Analytics screen reporting is enabled. Call +[FIRAnalytics setScreenName:setScreenClass:] to set the screen name or override the default screen class name. To disable screen reporting, set the flag FirebaseScreenReportingEnabled to NO (boolean) in the Info.plist
2019-10-31 16:56:50.065498+0800 Runner[3397:1036813] [VERBOSE-2:dart_vm_data.cc(19)] VM snapshot invalid and could not be inferred from settings.
2019-10-31 16:56:50.065600+0800 Runner[3397:1036813] [VERBOSE-2:dart_vm.cc(238)] Could not setup VM data to bootstrap the VM from.
2019-10-31 16:56:50.065647+0800 Runner[3397:1036813] [VERBOSE-2:dart_vm_lifecycle.cc(89)] Could not create Dart VM instance.
2019-10-31 16:56:50.065900+0800 Runner[3397:1036813] [VERBOSE-3:shell.cc(218)] Check failed: vm. Must be able to initialize the VM.

flutter doctor -v
[✓] Flutter (Channel stable, v1.7.8+hotfix.4, on Mac OS X 10.14.6 18G95, locale zh-Hans-CN)
    • Flutter version 1.7.8+hotfix.4 at /Users/sunjianan/Documents/programs/SDK/flutter
    • Framework revision 20e59316b8 (3 months ago), 2019-07-18 20:04:33 -0700
    • Engine revision fee001c93f
    • Dart version 2.4.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Volumes/Jianan/SDK/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.1)
    • Xcode at /Volumes/Jianan/APP/Xcode.app/Contents/Developer
    • Xcode 11.1, Build version 11A1027
    • CocoaPods version 1.8.3

[✓] iOS tools - develop for iOS devices
    • ios-deploy 1.9.4

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[!] IntelliJ IDEA Ultimate Edition (version 2018.1.5)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.dev/intellij-setup/#installing-the-plugins

[✓] Connected device (1 available)
    • Jianan 的iPhone 6s • bed1d946f16440e6af735de7e2f41411f514aab3 • ios • iOS 10.3.3

! Doctor found issues in 1 category.

@sunjianan9900 Please try and report if that will work with current stable or master channel? Also do flutter clean.
This issues duplicates https://github.com/flutter/flutter/issues/36247, closing it. Please follow it there and upvote.


Update merge on refresh and merge on commit on Opensearch (Lucene 9.3)

Is your feature request related to a problem? Please describe.
The Apache Lucene 9.3 made merge-on-refresh enabled by default [1].
[1] https://github.com/apache/lucene/pull/921/files
Describe the solution you'd like
The Opensearch merge policy has to be updated in order to accommodate this change.
Describe alternatives you've considered
N/A
Additional context
N/A

@nknize fyi

@reta is there anything we should consider documenting regarding this change? Does it have performance implications for users or anything else we should cover in opensearch.org/docs?

@elfisher yes, I will submit the documentation update for 2.1.0 / 2.2.0. Trying to find out the best place for it and not sure 9.3 will be released before 2.1.0 is out. In general, the change should have only positive performance implications, but we are providing the option to disable it.


Changes to Origin

The CheckpointHeaderV0 should never be used and is an artifacts of a strange couple of days from history that must not be spoken about. Origin should be unique and identify the log, thus such a constant is dangerous.
Fixed the const in the test data to use the origin string that is actually used.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 60.26%. Comparing base (07aa77d) to head (0c397b5).
Report is 22 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #104      +/-   ##
==========================================
+ Coverage   55.99%   60.26%   +4.26%
==========================================
  Files           8        8
  Lines         759      609     -150
==========================================
- Hits          425      367      -58
+ Misses        265      173      -92
  Partials       69       69

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Markup is broken

Link to article:

https://docs.microsoft.com/en-us/windows/desktop/sysinfo/merged-view-of-hkey-classes-root
https://docs.microsoft.com/en-us/windows/desktop/sysinfo/categories-of-data

Expected

Hi @otya128 -- thank you for letting us know! We'll work on getting these pages fixed.


New version: YoYoGames.GameMaker.Studio.2 version 2<IP_ADDRESS>

Created by 🥟 Dumplings.
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Save backups in 24h format.

At the current moment, backups are saved in DD-MM-YYYY 12h format, which is unfortunate because they aren't in alphabetical order that way and makes the order feel disorganized. I propose, for the sake of simplicity and ease for the server admin, that backups be saved in 24h format without an AM or PM displayed, at least as a config option.

In version 1.5.1 the format is now year-month-day hour-minute-second with hour in 24-hour time, so the backups should now be properly alphabetized by date. I'm not sure why I was using that other format.


Github Actions + lint com flake8

Adiciona flake8
Cria worklow para CI utilizando GitHub actions

Coverage remained the same at 98.95% when pulling 9dab0942d49b2bd8e909da47ddaf98328ff8773a on dev into 80540476e6a4897ff8f078cd66ad67ad38e87c41 on master.


Missing existing_payment views

The Solidus commit https://github.com/solidusio/solidus/pull/1707/files added a partials folder called existing_payment, but there are no such views in the master branch of solidus_gateway.
This means that users using Stripe and Quick Checkout will cause 500 errors when they try to check out.
This is a show-stopping issue, as it literally breaks checkout in some Stores.
What we need: views in the spree/checkout/existing_payment/ path for Stripe and Quick Checkout. I’m not sure what these views are supposed to look like exactly, so I’m not sure what the PR would be here. But this is an urgent issue for anyone using Solidus 2.2 and Stripe or Quick Checkout in production.

Fixed by #49


Validation every epoch with non-finite dataloader

🚀 Feature

Providing a way to do validation every epoch with non-finite (__len__ not implemented) dataloaders.
Motivation

Doing validation every epoch is a natural choice, and with finite dataloader you can do it easily by setting val_check_interval=1.0. However, with non-finite dataloader you cannot set val_check_interval to be float. There's no simple way to work around it yet.
Pitch

There can be several way to make it happen.
One solution on top of my head is to let val_check_interval to be None. if it is none, just do the validation according to the number of check_val_every_n_epoch at the end of every epoch.
Alternatives

Alternatively, you can let the user set val_check_interval to be 1.0 even with non-finite dataloaders. Anything below 1.0 would be invalid but only 1.0 can be valid. If it is 1.0 then do the validation according to the number of check_val_every_n_epoch at the end of every epoch.
Additional context

Not all dataloaders without __len__ implemented are infinite dataloaders. Some just cannot decide length in advance. With these dataloaders the concept of 'epoch' is still valid. Pytorch-lightning needs to serve this kind of dataloaders better.

Thanks for the issue! Totally agree that we should support this. It might be a bit painful with how validation works at the moment. Currently we validate when the batch index reaches a certain value. But with Iterable stuff we'd need to do it once the itetation is over (and we can't know ahead of time when that will be). I guess that the way to do it is to add a seperate clause which deals specifically with the case when val_check_interval=1.0, although it's not the most elegant solution :/

cool, @ethanwharris @dreamgonfly mind drafting a PR?
cc: @PyTorchLightning/core-contributors


Docker daemon hangs on stop and becomes defunct process, machine requires hard reset

I'm brand new to docker, and I'm going to try to file a proper bug report, but if I neglect something important, please let me know and I will try my best to add it back in.
I'm using a fully updated version of openSuse tumbleweed (kernel 4.11.8-1), and I recently installed docker (version 17.04.0-ce) using the standard method zypper in docker. I can start the docker daemon fine using sudo service docker start, and from my small amount of testing, docker seems to be working fine. I have not actually run any containers using docker; I have only installed it and nothing else (never done a docker pull or created/downloaded a docker file). However, when I go to stop the docker daemon (either manually with sudo service docker stop or try to reboot the machine, the docker daemon hangs and the machine hangs as well. If I do sudo service docker stop and then hit ctrl-c, the dockerd process becomes defunct, and the machine is useable but very slow and hangs on many commands. If I try to reboot the machine (before or after issuing the sudo service docker stop command) then the machine hangs on shutdown, and never reboots.
The only way around this hang is to kill the docker process with kill -9 $(dockerd-pid) before trying to stop the docker process with sudo service docker stop. Then the machine will reboot normally. If I first try sudo service docker stop, then there is no way that I have found to successfully reboot the machine short of a hard power cycle.
Just to be clear, this is a plain vanilla setup without any modification. Freshly installed from the official repositories for openSuse Tumbleweek. I don't have any proxies or anything.
Here is the requested information listed in the bug tracker instructions:

Docker version output:

Client:
 Version:      17.04.0-ce
 API version:  1.28
 Go version:   go1.7.5
 Git commit:   78d1802
 Built:        Tue Jul  4 16:31:44 2017
 OS/Arch:      linux/amd64

Server:
 Version:      17.04.0-ce
 API version:  1.28 (minimum version 1.12)
 Go version:   go1.7.5
 Git commit:   78d1802
 Built:        Tue Jul  4 16:31:44 2017
 OS/Arch:      linux/amd64
 Experimental: false

Docker info:

Containers: 0
 Running: 0
 Paused: 0
 Stopped: 0
Images: 0
Server Version: 17.04.0-ce
Storage Driver: btrfs
 Build Version: Btrfs v4.10.2+20170406
 Library Version: 102
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
Swarm: inactive
Runtimes: oci runc
Default Runtime: runc
Init Binary:
containerd version:  (expected: 422e31ce907fd9c3833a38d7b8fdd023e5a76e73)
runc version: N/A (expected: 9c2d8d184e5da67c95d601382adf14862e4f2228)
init version: N/A (expected: 949e6facb77383876aeff8a6944dde66b3089574)
Security Options:
 apparmor
 seccomp
  Profile: default
Kernel Version: 4.11.8-1-default
Operating System: openSUSE Tumbleweed
OSType: linux
Architecture: x86_64
CPUs: 8
Total Memory: 15.69GiB
Name: cyclon
ID: B2IN:EMBW:HQNI:3GK4:TFUV:L33J:ARBS:JNRR:DYVB:6DQJ:GRKC:Z3B3
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 14
 Goroutines: 20
 System Time: 2017-07-07T10:36:30.49992113-04:00
 EventsListeners: 0
Registry: https://index.docker.io/v1/
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

WARNING: No swap limit support

docker exec <registry-container> registry -version (not sure what this is)

-bash: registry-container: No such file or directory

I've attached a log file (entitled docker_logs_before_stop.txt) gotten with sudo journalctl - u docker from before issuing sudo service docker stop, and I've enabled debug mode on the docker daemon (using daemon.json) before doing so. I've also attached logs gotten after hard resetting the machine and running the same command (entitled docker_logs_after_stop.txt), but weirdly, they are less complete then the ones gotten before running the command...
docker_logs_after_stop.txt
docker_logs_before_stop.txt
Please let me know if there is anything else necessary to diagnose and fix the problem. Thanks!

I'm sorry, I think I put this on the wrong bug tracker. I opened another issue on docker/for-linux

I opened a bug report on openSUSE bugzilla as well (Bug 1047793), and it turns out that this is similar to another bug previously reported there (Bug 1047152). The issue is with the btrfs storage driver using some element of the btrfs filesystem that hangs if the filesystem is first mounted as ro and then later mounted as rw. This appears to be what's going on here, and it seems like it is a bug in btrfs, not docker. If anyone stumbles on this, be warned that it exists up to kernel 4.11.8.


Newsboat background problem

Is there an existing issue for this?

[X] I have searched the existing issues.

What is your concern?
Hey!
I tried to install the dark theme to newsboat is it said in the Usage. But my Newsboat background colour still white, not the full color scheme implemented.
Could you help me please?

Hey!
I'm using zsh (macOS terminal).
I downloaded the dark them and after that i inserted the following code to the newsboat config: include ~/.newsboat/dark.
What color code can i set up to zsh?


Txn is not cleared when load been cancelled.

You can reproduce the bug like this.
CREATE TABLE `table1` (
  `siteid` tinyint(4) NULL DEFAULT "10" COMMENT "",
  `citycode` smallint(6) NULL COMMENT "",
  `username` varchar(32) NULL DEFAULT "" COMMENT "",
  `pv` bigint(20) SUM NULL DEFAULT "0" COMMENT ""
) ENGINE=OLAP
AGGREGATE KEY(`siteid`, `citycode`, `username`)
COMMENT "OLAP"
DISTRIBUTED BY HASH(`siteid`) BUCKETS 1
PROPERTIES (
"replication_num" = "1",
"in_memory" = "false",
"storage_format" = "DEFAULT"
);

Data is like this
1111          1	'jim'	        2
2222	1	'grace'	2
3333	2	'tom'	2
4444	3	'bush'	3
5555	3	'helen'	3

The data will be encounter error when loading.
But the Txn is still exists until timeout.


Assigning long to value to Long field triggers VerifyError

What I am doing here is oviously wrong, but this should be caught at compile time I think.
michal@domov:~/Dropbox/dev/usedriven$ mirah -e "@a = Long(nil); @a = Long.parseLong('1297')"
java/lang/Class.java:-2:in `getDeclaredMethods0': java.lang.VerifyError: (class: DashE, method: main signature: ([Ljava/lang/String;)V) Expecting to find object/array on stack (NativeException)
from java/lang/Class.java:2427:in `privateGetDeclaredMethods'
from java/lang/Class.java:2670:in `getMethod0'
from java/lang/Class.java:1603:in `getMethod'
from /home/michal/mystuff/mirah/lib/mirah.rb:128:in `run'
from /home/michal/mystuff/mirah/lib/mirah.rb:125:in `each'
from /home/michal/mystuff/mirah/lib/mirah.rb:125:in `run'
from /home/michal/mystuff/mirah/lib/mirah.rb:21:in `run'
from /home/michal/mystuff/mirah/bin/mirah:9

Interestingly, while mirahc -e "@a = Long(nil); @a = Long.parseLong('1297')" gives you an invalid class file
$ mirahc -e "@a = Long(nil); @a = Long.parseLong('1297')"
Parsing...
  <inline script>
Inferring types...
Compiling...
  DashE
Done!
$ java -cp . DashE
Exception in thread "main" java.lang.VerifyError: (class: DashE, method: main signature: ([Ljava/lang/String;)V) Expecting to find object/array on stack

compiling to java source generates good output that compiles and runs correctly.
$ mirahc -j -e "@a = Long(nil); @a = Long.parseLong('1297')"

Parsing...
  <inline script>
Inferring types...
Compiling...
  DashE
Done!

$ javac -cp . DashE.java
$ java -cp . DashE

DashE.java
// Generated from DashE
public class DashE extends java.lang.Object {
  private static java.lang.Long a;
  public static void main(java.lang.String[] argv) {
    DashE.a = ((java.lang.Long)(null));
    DashE.a = java.lang.Long.parseLong("1297");
  }
}

I think it's a type coercion issue because when you use Long.new(str) it works.
$ mirah -e "@a = Long(nil); @a = Long.new('1297'); puts @a"
1297

Ran it w/ -V with locals and fields. The relevant log line I think is this one
* [Mirah::Typer] Learned field type under  : a = Type(long)

currently, inference on field assignment doesn't do all the type checking it should.
https://github.com/mirah/mirah/blob/master/lib/mirah/typer/simple.rb#L152-159
      def learn_static_field_type(cls, name, type)
        log "Learned field type under #{cls} : #{name} = #{type}" if type

        # TODO check for compatibility?
        static_field_type_hash(cls)[name] ||= known_types[type] || type

        type
      end

local inference does, which is why you get an inference error rather than a runtime error.
The other interesting thing is that cls is nil. Which is probably contributing to the problem.


Default product should not be hard-coded

When you visit the / page, the default product that is attempted to be loaded is ls7_nbar_scene.
This page should either load the first product of those available, or a list of available products to choose from perhaps.

Thanks Alex.
Closing this old issue — the current version picks the first available, starting with the defaults.

Nice work!


Can not import AutoAWQForCausalLM on google colab

I was able to easily import AutoAWQForCausalLM on google colab a few days ago but today I received the following error:
ImportError: libcudart.so.12: cannot open shared object file: No such file or directory
This is the entire code:
!pip3 install autoawq
from awq import AutoAWQForCausalLM

and I get this error:
`---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
 in <cell line: 1>()
----> 1 from awq import AutoAWQForCausalLM
5 frames
/usr/local/lib/python3.10/dist-packages/awq/modules/linear.py in
2 import torch
3 import torch.nn as nn
----> 4 import awq_inference_engine  # with CUDA kernels
5
6
ImportError: libcudart.so.12: cannot open shared object file: No such file or directory

NOTE: If your import is failing due to a missing package, you can
manually install dependencies using either !pip or !apt.
To view examples of installing some common dependencies, click the
"Open Examples" button below.
---------------------------------------------------------------------------`
I am using the free T4 on google colab.

The latest version uses CUDA 12.1 and torch 2.1.0, so you would need to install those dependencies in colab

@casper-hansen is it possible to make a version that is compatible with cu11.8 since the cuda driver might not be able to upgrade in some cases? thanks

@casper-hansen is it possible to release a version that is compatible with cu11.8 since the cuda driver might not be able to upgrade in some cases? thanks

Yes it might be possible, I just have to figure out how to do it practically. I think it may be possible to build 11.8 wheels and upload them to the GitHub release but not to PyPi.

You can now run the following to install torch 2.0.1 + cuda 11.8.0 + python 3.10 wheel. See the other wheels in the release for Windows and other Python versions.
pip install https://github.com/casper-hansen/AutoAWQ/releases/download/v0.1.6/autoawq-0.1.6+cu118-cp310-cp310-linux_x86_64.whl

Perfect, I have been trying to upgrade Pytorch Cuda on Colab for the last few hours and your code came in the right time.
It works perfectly now.
Thank you.


DelayVaultMigrator

need to use :
https://github.com/The-Poolz/DelayVault/blob/1ad776e119e41d2facd640177c840a09a8a3d67b/contracts/DelayManageable.sol#L104
the user will do this first:
https://github.com/The-Poolz/DelayVault/blob/1ad776e119e41d2facd640177c840a09a8a3d67b/contracts/DelayVault.sol#L106
the token transfers to the DelayVaultMigrator and he makes a new vault for the user.
we can reset the Data to fill by type, max out type 1, then i++ (the last one has no limit)

https://github.com/The-Poolz/DelayVaultProvider


PlayFabPatcher.dll issue

the playfabpatcher.dll does not enable special items when banned. if you can just send me all the dlls that have been used to compile that dll i will fix it myself

Hello, do not install the mods to your quest if your banned Thank you!


Fix discovery document link in kubernetes-api.md

Removed an extra "v" in the link to fix the broken link.

Modified one more broken anchor.

/lgtm
Thanks!

/label tide/merge-method-squash


CarPlay garage door  no response

Having issue with garage door in CarPlay Siri suggestions. So Garage door is shown in Siri suggestions but it's not working. it's only display card with message "no response". BUT the same door working fine from Apple home app.
I Tried to update iOS to latest version, delete cached accessory,still no luck. Issue is still persist. I don't know this is something from my side, Apple or plugin updates. A month ago all working fine(
Homebridge ltest 1.3.4
ConfigUI latest.
iOS latest.

I use sonoff dc device

So just to confirm, on the apple home app on your phone the accessory doesn't show as no response... but on the carplay screen it does show as no response?

This is strange. If it’s working from your phone but not car, this might suggest a connectivity issue between the home hub or your phone and your car.
I unfortunately have zero experience with CarPlay, I assume it connects to your phone?


A simple spinner component when the App is fetching data used as a fallback with suspense

Is your feature request related to a problem? Please describe.
A simple spinner component that adds a visual experience when the user waits for the application to load the data.
I have seen a PR related but didn't see an update from him based on the request asked. So I thought  I can reopen this
Additional context

I would like to /claim this

We already have content loaders for a few components. Which particular component do you want to add this to?

@shuklaayush in the app.js itself. I have tested this
return (
    <div className="App">
      <Suspense fallback={<Spinner />}>
        <LanguageSwitcher
          {...{showLanguageSwitcher, setShowLanguageSwitcher}}
        />
      </Suspense>

      <Navbar
        pages={pages}
        {...{darkMode}}
        {...{showLanguageSwitcher, setShowLanguageSwitcher}}
      />

      <Suspense fallback={<div />}>
        <Switch location={location}>
          {pages.map((page, index) => {
            return (
              <Route
                exact
                path={page.pageLink}
                render={({match}) => <page.view />}
                key={index}
              />
            );
          })}
          <Redirect to="/" />
        </Switch>
      </Suspense>
    </div>
  );

/answer

Closing due to inactivity


Regarding disabling of virtual nodes

@philippn commented on Mon Feb 10 2020
In your code example for removing the virtual network resources, the last command to execute is:
# Delete the subnet delegation to Azure Container Instances
az network vnet subnet update --resource-group $RES_GROUP --vnet-name $AKS_VNET --name $AKS_SUBNET --remove delegations 0

However, this doesn't work and results in the following error:

Subnet /subscriptions/.../resourceGroups/dye-aut/providers/Microsoft.Network/virtualNetworks/aks-vnet/subnets/virtualnode-subnet requires any of the following delegation(s) [Microsoft.ContainerInstance/containerGroups] to reference service association link /subscriptions/.../resourceGroups/dye-aut/providers/Microsoft.Network/virtualNetworks/aks-vnet/subnets/virtualnode-subnet/serviceAssociationLinks/acisal.

Dokumentdetails
⚠ Bearbeiten Sie diesen Abschnitt nicht. Er ist für die Verknüpfung von docs.microsoft.com zum GitHub-Artikel erforderlich.

ID: 916ac90f-8a94-5f3d-eb99-fe06c39eb6cf
Version Independent ID: 2aad2e47-7fde-8d24-f824-236a26e5d887
Content: Erstellen von virtuellen Knoten mit der Azure CLI in Azure Kubernetes Service (AKS)
Content Source: articles/aks/virtual-nodes-cli.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@srvbpigh commented on Mon Feb 10 2020
Hello, @philippn
Thank you for your feedback.
We are actively reviewing your comments and will get back to you soon.
Kind regards,
Microsoft DOCS International Team

@CeciAc Thank you for the feedback. We will review and update accordingly.

@CeciAc I am able to reproduce the issue, I have reached out to internal team to check on this.

@CeciAc This is a know issue from ACI.
Team has rolled out a fix for VNet deletion and expect behavior to be back to back to normal.
However, internal team has advised to open a support case for investigating this incase issue is still existing.

We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


SRA extraction for SOLID data is not compatable with TopHat

When extracting SOLID data from SRA for Tophat, need to use abi-dump not fastq-dump (see http://seqanswers.com/forums/showthread.php?t=44333) and retain quality files. Currently, following sra extraction, there is no attempt to detect whether the data is SOLID.
Propose check for SOLID data within SRA and then append correct fastqs to retain quality scores for downstream mapping.

See 1022967.
Currently no check to make sure the downstream mapper can handle SOLID data. Should probably incorporate this at this stage in pipeline_mapping to avoid waiting for SRA extraction to complete and then working it out from the mapper log file?


New topic: Soft Fork Activation

I think this is really putting the extended in the extended_summary field.  I can probably publish this excessive amount of detail about past soft fork attempts somewhere else if y'all think this should be more focused.
I also think this could maybe do a better job of describing current activation proposals, but they're all still being actively discussed and so I'm not sure it's worth writing about them now only to have to change it in a week or a month.  (Maybe that's why I geeked out about nice, static history.)

Wow @amitiuttarwar, thanks for the comprehensive review!  I applied all the easy code suggestions and answered a few of your questions.  I'll go through and work on the more substantive edits tomorrow.

@amitiuttarwar and @michaelfolkson I believe I've addressed all of your feedback.  Thank you both for the detailed comments!

Just a note that I'm still working on making edits for the above feedback.  I was optimistically hoping to finish my edits on Friday and again today; now I'm hoping to finish tomorrow.

Oh! I didn't notice the most recent push here. @harding is this ready for re-review?

@jnewbery

@harding is this ready for re-review?

Indeed it is.  I believe all feedback to date has been addressed.  Sorry I didn't say anything.

The new section on Emergency activations is great! I've left some minor review comments on the section.
Additionally, there are two previous review comments that I think still need to be addressed:

https://github.com/bitcoinops/bitcoinops.github.io/pull/531#discussion_r599592479
https://github.com/bitcoinops/bitcoinops.github.io/pull/531#discussion_r600429688

Pushed an edit that I hope fully addresess @jnewbery and @ajtowns feedback.  Thanks (and double thanks to John for pointing me at feedback from AJ that I had somehow not seen).
The same edit also addresses all comments from @luke-jr except two where I left a specific reply.  Thanks especially for this comment where I had indeed misconstrued the sequence of events.
If this is looking pretty good, I'd like to squash it tomorrow or Thursday.

This is ready for merge. Fantastic work as always, @harding! Thank you for documenting this.


Nokogiri::Reader unescape inner html/xml?

Nokogiri::Node has methods (content, and text) that unescape &.  Yet, Reader does not appear to have such methods -- inner_xml returned strings remain escaped.
Is there a method in Nokogiri::Reader that will unescape &.  If not, what is the recommended way to do so?

Hello!
Thanks for asking this question! Your request for assistance using
Nokogiri will not go unanswered!
However, Nokogiri's Github Issues is reserved for reporting bugs or
submitting patches. If you ask your question on the mailing list, Team
Nokogiri promises someone will provide you with an answer in a timely
manner.
If you'd like to read up on Team Nokogiri's rationale for this policy,
please go to http://bit.ly/nokohelp.
Thank you so much for understanding! And thank you for using Nokogiri.

OK. Though this is also an enhancement request if the method does not exist.


Update release to match tag.

The rpc-release variable is dropped into /etc/rpc-release, and thus should match the release's tag.

:+1:

:+1:


Link repository to zenodo DOI

tutorial documenting how to do this is here:
https://guides.github.com/activities/citable-code/
we want to make sure a release is made at the end of the hackweek so that all content contributors are listed

I wasn't seeing any of the snowex-hackweek repos logged into zenodo and came across this solution https://github.com/zenodo/zenodo/issues/1814#issuecomment-620555444

Plan for releasing a reproducible version of the website. cc @aaarendt @jomey

Need to move any datasets used to make plots into a single repository because s3://snowex-data/tutorial-data will not last forever and as we saw during tutorials, sometimes servers are down so pulling data when running notebooks can be unreliable. Finally, if people want to run these examples a year from now, what happens if servers no longer exist or data versioning and urls are different!

There are two strategies here and I think the best one is to create a zenodo record for tutorial data and a second record for the website. If there data is very small it can be stored in a github repo, but we have some large rasters. Here is a quick comparison between those options:

data store
total size	 limit
single file limit

Zenodo Record
50 GB
2 GB

Github Repository
1 GB - 5 GB*
100 MB**

* “strongly recommended”  ** strict
If we create a zenodo record the tutorials could start with this cell:
wget https://zenodo.org/record/RECORD_ID/files/snowex-hackweek/tutorial-data.zip

Many tutorials use the database. The location of the database may change, so what would it take to reserve a more permanant domain name that can map to any IP? For example snowex-data.io? cc @micahjohnson150 @hpmarshall

Need to lock in the environment in this repository from 'latest' to the date stamp used '2021.07.07'
https://github.com/snowex-hackweek/website/blob/391e0cd8bed13f6325777070e6fe0cad3aff01e3/.github/workflows/deploy.yaml#L12

Anything else i'm missing?

Another solution for the data could be to use a git submodule:
https://www.atlassian.com/git/tutorials/git-submodule
The module could be the https://github.com/snowex-hackweek/tutorial-data
I feel that the file size limit of 100MB for a GitHub repository sounds reasonable, given that we should aim for the notebooks to execute rather quick. Long running notebooks might deter people from looking at them as well.

This is awesome @scottyhq thanks for putting this together.  I'll figure out the domain name.  The last few letters don't matter correct?  Nothing special about .io?  Its more expensive...

And I agree that a zenodo for the tutorial data sounds like what we need for the larger datasets.  We can reduce the number of UAVSAR rasters on s3 before we do that...the tutorial only used a few and the project downloaded many more.

This is awesome @scottyhq thanks for putting this together. I'll figure out the domain name. The last few letters don't matter correct? Nothing special about .io? Its more expensive...

I've had colleagues use .pizza b/c it is hillarious and cheap :) It can be whatever you want, For data oriented things .io is common. I've also seen .xyz or even .org.  We pay $39 / year for  hackweek.io , which we obtained via https://docs.aws.amazon.com/Route53/latest/DeveloperGuide/registrar.html

You can pick any domain name.
My current provider is hover.com and I have only good things to day (Can't pick on Canada anyways).
I suggest: snowexdata.org
Cheapest option and very common suffix

Interesting arguments for a .org domain
https://thenew.org/org-people/domain-products/org/

Done!  We now have "snowexdata.org".  I like the org arguments.

Awesome! @micahjohnson150 could you please map the database IP to a URL under snowexdata.org and confirm the database tutorial notebook works with that new URL? We've used this https://dns.he.net/ service for mappings in the past, I'm sure there are alternatives or maybe whatever service was used to obtain snowexdata.org can do this mapping.
I'm envisioning https://database.snowexdata.org but again, this is up to you
https://docs.github.com/en/pages/configuring-a-custom-domain-for-your-github-pages-site/about-custom-domains-and-github-pages#using-a-subdomain-for-your-github-pages-site

Definitely. It will take me a couple days to make sure its full flushed out. Thanks for the resource suggestion!

I can vouch for the hover DNS (if purchased through there). Has also not failed me in the past.
As for url - I like the suggestion. You can also shorten it up and do db.snowexdata.org, but that's my fat fingers speaking that like to misspell things.

just let me know where to point snowexdata.org to.  I guess it would be the jupyterhub for the next month while we figure out how to host long-term?


Replace extra Floating Action Buttons with a menu item in a Drawer

Create a Drawer accessible from the app bar and move the navigation for the Display Settings into it (as a menu item) along with New Page.  This should leave Edit Page as the only FAB.

This was implemented back in Iteration 10.


Error on Second stage (pitch tracking)

Windows 10
Tried running on a YouTube video.
This is the traceback:
File "C:\Users\Dimi novo\polymath\polymath.py", line 690, in <module>     main()   File "C:\Users\Dimi novo\polymath\polymath.py", line 622, in main     audio_features = get_audio_features(file=file,file_id=vid.id)   File "C:\Users\Dimi novo\polymath\polymath.py", line 424, in get_audio_features     frequency_frames = get_pitch_dnn(file)   File "C:\Users\Dimi novo\polymath\polymath.py", line 317, in get_pitch_dnn     time, frequency, confidence, activation = crepe.predict(audio, sr, model_capacity="tiny", viterbi=True, center=True, step_size=10, verbose=1) # tiny|small|medium|large|full   File "C:\Users\Dimi novo\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\crepe\core.py", line 255, in predict     activation = get_activation(audio, sr, model_capacity=model_capacity,   File "C:\Users\Dimi novo\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\crepe\core.py", line 185, in get_activation     model = build_and_load_model(model_capacity)   File "C:\Users\Dimi novo\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\crepe\core.py", line 44, in build_and_load_model     from tensorflow.keras.layers import Input, Reshape, Conv2D, BatchNormalization ModuleNotFoundError: No module named 'tensorflow'

Make sure you have TensorFlow installed on your system! Will update the readme soon to cover this for windows systems

Okay, after installing TensorFlow, that stage went smoothly.
Now there's an error on 8/8 split steams stage.
Traceback (most recent call last):   File "C:\Users\Dimi novo\polymath\polymath.py", line 690, in <module>     main()   File "C:\Users\Dimi novo\polymath\polymath.py", line 622, in main     audio_features = get_audio_features(file=file,file_id=vid.id)   File "C:\Users\Dimi novo\polymath\polymath.py", line 449, in get_audio_features     stemsplit(file, 'htdemucs_6s')   File "C:\Users\Dimi novo\polymath\polymath.py", line 325, in stemsplit     subprocess.run(["demucs", destination, "-n", demucsmodel]) #  '--mp3'   File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.10_3.10.2800.0_x64__qbz5n2kfra8p0\lib\subprocess.py", line 503, in run     with Popen(*popenargs, **kwargs) as process:   File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.10_3.10.2800.0_x64__qbz5n2kfra8p0\lib\subprocess.py", line 971, in __init__     self._execute_child(args, executable, preexec_fn, close_fds,   File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.10_3.10.2800.0_x64__qbz5n2kfra8p0\lib\subprocess.py", line 1440, in _execute_child     hp, ht, pid, tid = _winapi.CreateProcess(executable, args, FileNotFoundError: [WinError 2] The system cannot find the file specified

@Dimimiha can you please provide the command you entered which produced your error? It looks like the file you specified to be added to the polymath library can not be found, possibly due to a spelling error in its path?

I just tried to add a song from youtube.
python polymath.py -a wNC9Njb5RME


Drop-in demo returns 'Error! Reason: Cancelled' using valid test cards

I have been trying out the drop-in demo using various test card numbers from https://docs.adyen.com/development-resources/test-cards/test-card-numbers. However, so far these have all returned the 'Error! Reason: Cancelled' message, even though I have successfully tested the same numbers using curl and prefixing the numbers with 'test_'. The response from the server is:
info: adyen_dotnet_online_payments.Controllers.ApiController[0]
Response for Payment API::
class PaymentResponse {
Action:
AdditionalData:
Amount:
Authentication:
Details:
DonationToken:
FraudResult:
MerchantReference: 39b82c21-fe7d-4d6e-a6ac-108cdb498d75
Order:
OutputDetails:
PaymentData:
PspReference:<PHONE_NUMBER>18467H
Redirect:
RefusalReason: FRAUD-CANCELLED
RefusalReasonCode: 22
ResultCode: Cancelled
}
Using curl with, for instance:
"paymentMethod": {
"type": "scheme",
"encryptedCardNumber": "test_5555444433331111",
"encryptedExpiryMonth": "test_03",
"encryptedExpiryYear": "test_2030",
"encryptedSecurityCode": "test_737"

}
(using the same number) I get the resultCode  "Authorised".

Thanks for reporting this. I'll take a look

Hi I tried this and couldn't reproduce. The test cards are working fine. Seems like in your case the request is going to fraud detection flow for some reason. Could be an intermittent test system issue. can you please try gain? if it still fails give me the psp ref from response i'll dig in further

Hi Deepu
I haven't had a chance to try this again. However, in the meantime, I did get this response from Adyen to a technical request I raised with them, which may throw some light on the issue. Note that I could get the same card numbers to work correctly using curl (in Postman) and prefixing the numbers with 'test_' without encryption.
Best regards,
Martin
| ------##- Please type your reply above this line -##------ |
|  |
|
Hi ,
This is an email to remind you that your request #1973819 has been pending for 3 days and awaits your feedback. This request will be closed in 2 days from now if no update on the ticket is provided.
|
|  |
Stella Liang (Adyen)
Mar 31, 2021, 20:12 GMT+2
Hi Martin,
Thanks for reaching out! The transaction was cancelled by our fraud engine, which happens when the total fraud score for the shopper exceeds 99 points. You can review the risk results for a shopper through Customer Area > search payments > click on PSP reference > fraud scoring > click on the fraud score.
You can disable risk through Customer Area:  Log in to Customer Area > Risk > Risk Settings. It will be the first item you see on the page.
You can also add the shopper to the trust list to decrease the fraud score. Please see documentation here: https://www.adyen.help/hc/en-us/articles/115001179984-How-do-I-block-or-trust-a-shopper-.
Please let us know if you have any questions.
Best,
Stella Liang
Technical Support Engineer
Adyen
|
|
|
|  |
Martin Vaughan
Mar 31, 2021, 14:24 GMT+2
I have cloned the https://github.com/adyen-examples/adyen-dotnet-online-payments.git repository and run it locally on my machine. I have been trying some of the test card numbers given on Test card numbers | Adyen Docs on the drop-in demo. However, using these returns the error that the card is cancelled.
|
|
|  |
Test card numbers | Adyen Docs
Test your integration with our test card numbers and payment method details.
|
|
|
The numbers I have tried are:
(Mastercard)
2222 40▇▇ ▇▇▇▇ 0005   Commercial Debit   CA   03/2030   737
5555 34▇▇ ▇▇▇▇ 1115    Consumer   NL   03/2030   737
5555 44▇▇ ▇▇▇▇ 1111   Consumer   GB   03/2030   737
Example server response:
info: adyen_dotnet_online_payments.Controllers.ApiController[0]
Response for Payment API::
class PaymentResponse {
Action:
AdditionalData:
Amount:
Authentication:
Details:
DonationToken:
FraudResult:
MerchantReference: 39b82c21-fe7d-4d6e-a6ac-108cdb498d75
Order:
OutputDetails:
PaymentData:
PspReference:<PHONE_NUMBER>18467H
Redirect:
RefusalReason: FRAUD-CANCELLED
RefusalReasonCode: 22
ResultCode: Cancelled
}
I cannot see anything on the web page to indicate that these are cancelled cards (although one would obviously want to test this use case). So are these intended to be cancelled? I am assuming that since this is the demo program written by Adyen (I have not altered the code at all) that the problem is not a bug in the code although obviously it could be.
Could you please let me know if there is a problem with these cards, if I am doing something wrong or if there are any known bugs with the demo code.
Many thanks,
Martin Vaughan
|
|
|
This email is a service from Adyen.[R5VL00-MDXG]
On Friday, 2 April 2021, 15:34:45 BST, Deepu K Sasidharan<EMAIL_ADDRESS>wrote:

Hi I tried this and couldn't reproduce. The test cards are working fine. Seems like in your case the request is going to fraud detection flow for some reason. Could be an intermittent test system issue. can you please try gain? if it still fails give me the psp ref from response i'll dig in further
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Have just tried again with details:
Card number: 5555 4444 3333 1111
Expiry date: 03/30
Security code: 737
The payment response (including Psp ref) was:
class PaymentResponse {
Action:
AdditionalData:
Amount:
Authentication:
Details:
DonationToken:
FraudResult:
MerchantReference: 4eda8298-30d9-4fc1-8016-49cb5d3f7270
Order:
OutputDetails:
PaymentData:
PspReference:<PHONE_NUMBER>26653G
Redirect:
RefusalReason: FRAUD-CANCELLED
RefusalReasonCode: 22
ResultCode: Cancelled
}

Ya seems like fraud setting issue and since this is running on localhost it could be detected as fraud. I'm closing this as its a setting related issue and Adyen support should be able to help you if you encounter this again

I think the fact that this is running on localhost may indeed be the problem. However, would it be possible to describe the settings you have on your merchant account for 3DS authorization so I could try these? Thanks.

well mine is a testing account with PCI compliance enabled so don't know if that matters, but in any case this normally works fine on new accounts as well. I think support would be your best bet in figuring out whats going wrong with your account


Introduce TravisCI

Are there plans to run syntax error checking and some linting on the code?
Maybe TravisCI?

I don't have the time or knowledge on this stuff, but feel free to contribute.

@thednp Thanks.
What linter tool does your code pass? eslint? jshint?
What build step or other check would you like me to implement?

As this isn't my area of expertise, this has to be decided by public vote.
I am reopening the issue with QUESTION tag, so everyone can share an idea/opinion on the matter.

Aren't these build*.js the build scripts?

Yes they are.

I don't have knowledge about this either. Maybe @szepeviktor could explain what it is and why we should make use of it?

@Vuurvlieg On every commit or - preferably - PR you benefit:

linter runs
builds run
any other checks run, e.g. static analysis

basically it like an assistant for you.
You may quickly discover syntax errors, wrong code styling, unused variables etc.
Are you interested?

For myself all these features are mostly useless because I triple check all components before commits, before publishing new versions and the final test is https://jshint.com/

OK.
I thought this is a community project.

Well, thats a lot of errors...

My finding - if you want to produce enterprise-level code - is that JSLint leads to very robust code.

It would hurt the size alot and I honestly don't see why to go back to that "corporate" level code, I don't think it's superior in any way. As a matter of fact, our code uses some already minified expressions, the reason for most of those errors.
For instance instead of doing this
lib/V3/tooltip-native.js: line 81, col 65, Expected an assignment or function call and instead saw an expression.
is the use of
if (condition) { that() }
we do
condition && that();
which is the minified version. takes less space and I find it elegant to use.

Yea, those results by given tool in bsn are more or less coding style issue things. There are other tools around, too, like standardJS for example.
IMO automatic checking is great way for large scale projects with many PRs of newcomers. This way, you can easily determine whether coding style (and maybe tests) pass properly. Overall in many cases it's quite an overhead just do it, because it's marketed as "professional".