Remove legacy extension for Firefox for Android pages

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

The legacy extension for Firefox for Android pages, those under https://developer.mozilla.org/en-US/docs/Archive/Add-ons/Legacy_Firefox_for_Android need to be removed as the technology they describe has being unsupported for some time.
To complete this work, translated pages need to be deleted before the reference/English pages

I have gone through and deleted all of the subpages of this page that I could delete. the following JSON indicates the ones that are left:
https://wiki.developer.mozilla.org/en-US/docs/Archive/Add-ons/Legacy_Firefox_for_Android$children
So basically it's the following three pages that are stopping us from getting this section complete deleted:

https://wiki.developer.mozilla.org/en-US/docs/Archive/Add-ons/Legacy_Firefox_for_Android/API/NativeWindow
https://wiki.developer.mozilla.org/en-US/docs/Archive/Add-ons/Legacy_Firefox_for_Android/API/NativeWindow/doorhanger
https://wiki.developer.mozilla.org/en-US/docs/Archive/Add-ons/Legacy_Firefox_for_Android/API/Home.jsm

I've been through and looked it over, and none of them have translations or children according to their $children JSON readouts or language menus. But I am still not able to delete them.
@escattone could you give me some help on this if you get the chance? Is this problem due to some kind of orphaned translations sat somewhere?
And if so, how do I find out what these are so I can delete them?

@chrisdavidmills I couldn't find anything suspicious about these documents, and I was able to successfully delete them manually using a production Django shell. Here are the five (5) documents I deleted:

https://wiki.developer.mozilla.org/en-US/docs/Archive/Add-ons/Legacy_Firefox_for_Android/API/NativeWindow/doorhanger
https://wiki.developer.mozilla.org/en-US/docs/Archive/Add-ons/Legacy_Firefox_for_Android/API/NativeWindow
https://wiki.developer.mozilla.org/en-US/docs/Archive/Add-ons/Legacy_Firefox_for_Android/API/Home.jsm
https://wiki.developer.mozilla.org/en-US/docs/Archive/Add-ons/Legacy_Firefox_for_Android/API
https://wiki.developer.mozilla.org/en-US/docs/Archive/Add-ons/Legacy_Firefox_for_Android

I did not purge them (which would irrecoverably remove them from the DB), but if you'd like me to do that I can do that as well. I hope that helps, and let me know if you need anything more. Cheers!

This is perfect, thanks @escattone !
I think that fixes this one. Closing.


Missing Assets not reloading

Hi, just discovered this really nice program.
Just one thing to add. If I add missing assets by hand which are not in the database, they're still marked as not available. Same when I change missing rolling stock with Loco Swap. Tried starting RailworksDownloader as Admin and even moved it to the desktop.

Installed files should be checked every single run, even when invoked manually by button.
I don't quite remember now if we do cache them somewhere, but just from quick look into the code I highly doubt so.
As of the second,
route, eventually scenario dependencies definitelly are cached.
However if any of the file changes (actually we only check last write attribute of the folder itself and yet if the time is newer than last run, we check its checksum) it will process the dependencies again.
Anyways we probably have small design flaw there - at least it seems to me rn - as we only add new dependencies, but do not actually remove ones, that aren't used anymore.
We will definitelly need to take a deeper look at this one 🧐

Found the reason for some of the missing assets that aren't really missing. Special Characters...


AssertionError: zero stage 1 requires an optimizer

An error occurred when using the evaluation code。
the command is:
python ./deepy.py evaluate.py   xxxx.yml --eval_tasks piqa

The training mode is multi-machine mode and the evaluation mode is single-machine mode.
Has anyone had a similar issue? thanks!

This is a known issue that is awkward to handle. Our current recommendation is to set ZeRO stage 0 when calling the evaluation script. We are working on integrating DeepSpeed Inference which will solve this issue and substantially accelerate inference tasks as well.

Is this bug resolved? How do we pass the or set the ZeRo stage 1? I also see the same error during inference.
  File "generate.py", line 91, in <module>
    main()
  File "generate.py", line 33, in main
    model, neox_args = setup_for_inference_or_eval(use_cache=True)
  File "/localhome/local-vsabavat/ai/training/gpt-neox/megatron/utils.py", line 448, in setup_for_inference_or_eval
    model, _, _ = setup_model_and_optimizer(
  File "/localhome/local-vsabavat/ai/training/gpt-neox/megatron/training.py", line 647, in setup_model_and_optimizer
    model, optimizer, _, lr_scheduler = deepspeed.initialize(
  File "/localhome/local-vsabavat/.local/lib/python3.8/site-packages/deepspeed/__init__.py", line 186, in initialize
    engine = PipelineEngine(args=args,
  File "/localhome/local-vsabavat/.local/lib/python3.8/site-packages/deepspeed/runtime/pipe/engine.py", line 68, in __init__
    super().__init__(*super_args, **super_kwargs)
  File "/localhome/local-vsabavat/.local/lib/python3.8/site-packages/deepspeed/runtime/engine.py", line 309, in __init__
    self.optimizer = self._configure_zero_optimizer(optimizer=None)
  File "/localhome/local-vsabavat/.local/lib/python3.8/site-packages/deepspeed/runtime/engine.py", line 1468, in _configure_zero_optimizer
    assert not isinstance(optimizer, DummyOptim), "zero stage {} requires an optimizer".format(zero_stage)
AssertionError: zero stage 1 requires an optimizer```

@vsabavat In one of your yml config files you should have something that looks like
  "zero_optimization": {
    "stage": 1,
    "allgather_partitions": true,
    "allgather_bucket_size":<PHONE_NUMBER>,
    "overlap_comm": true,
    "reduce_scatter": true,
    "reduce_bucket_size":<PHONE_NUMBER>,
    "contiguous_gradients": true,
    "cpu_offload": false
  },

In my example, the stage is set to 1.


Release 0.4.0

== Release 0.4.0

Improvements

Use latest latest nokogiri version 1.6, thor 0.19.1 (#18)
Add support for latest rspec 3.x (#13)
Added --capture option to plsql-spec to indicate when to capture the output (#10)
Tested with Ruby version 2.2.0
Tested with ruby-oci8 version 2.1.7

Buf fixes

fix for cli diff issue related to color rendering (#14)

Internal (development) improvements

Ensure DATABASE_PORT is Fixnum (#16)
Coverage test fix: compile plsql with optimize level 1 (#13)
Tests: allow fully-qualified service name connection (#12)
Migrate tests to rspec 3.x, using expect syntax (#13)
Use latest jeweler 2.0.1 (#18)

@rsim, I'm planning to release 0.4.0. Any concerns?

@javornikolov Seems that Raimonds is now either very busy nowadays.
I hope everything is well and you will get a green light for a release.
@rsim If you don't have any objections, please confirm. It would be nice to have ruby-plsql-spec updated with newest library versions.

@javornikolov Sorry for not responding earlier.
It's OK to release the new version.

Thanks! I'll release shortly.


cmd/link: failed to set variable at build time if variable value is result from method call

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.10.3 linux/amd64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/cuonglm/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/cuonglm/go"
GORACE=""
GOROOT="/home/cuonglm/sources/go"
GOTMPDIR=""
GOTOOLDIR="/home/cuonglm/sources/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build755255777=/tmp/go-build -gno-record-gcc-switches"

What did you do?
https://play.golang.org/p/M9xGmognyrl
run as:
go run -ldflags "-X main.X=y -X main.MyX=myY" main.go

What did you expect to see?
y
myY

What did you see instead?
y
myX

I think that fixing this would require a completely different and less efficient approach to the implementation of the linker's -X option.  I'm inclined to just make this a documented limitation.

Every global variable has memory allocated in the data section of the program.  The contents of the data section are set to the initial value of the variable.  For a variable initialized with a function call, the variable's value is set by an initialization function.  The -X option simply changes the initial value stored in the data section, but does not affect the initialization function.
The implementation of -X is the function addstrdata in cmd/link/internal/ld/data.go, which sets the global variable strdata, and then dostrdata which calls addstrdata for each element of strdata.

@ianlancetaylor Thanks. I went through those codes already but don't fully understand them.
/me will send the CL this morning


Minerva refactor

Factors out reusable components from the MinervaMath implementation into reusabale ABCs and mixins.

MajorityVotingMixin implements majority voting.
SymbolicMathMixin implements functions for parsing TeX expressions from strings and determing sympy equivalence of expressions.
SymbolicMathTask is a template for implementing tasks whose answer is a mathematical expression.

I plan to write follow up PRs that create a NumericalMathTask. The harness implements a template for multiple choice tasks, but we may have to modify it for our use case.

@keirp @mcaleste  are you happy with the SymbolicMathTask template? Anything I should change to make it more useful to you?

This looks really nice! It'll also make supporting multiple samples in minif2f-isabelle much easier.


Live Data Source

Summary
This PR adds everything needed in the Beacon backend to use the dynamic module compilation approach for "live data" - assigns to use in page templates.  This allows users to change the value of assigns at runtime, whether the value is a simple primitive, the result of elixir code, or even dynamically generated based on a path param.
For an example of the dynamic case, imagine we have a route shop/products/:id
We could create a live data assign where:
%{format: :elixir, key: "product", value: "MyApp.Shop.Products.get_product!(id)"}

Then create a page with path shop/products/123 with template:

<%= @beacon_live_data[:product].name %>

Price: <%= @beacon_live_data[:product].price %>

etc...

and we have the expected results from LiveView, but everything is defined at runtime through the LiveAdmin interface.
Implementation

Added a LiveData schema which holds the path (scoped to site)
Added a LiveDataAssign schema which holds key/value + the format of the value (text or elixir)
LiveData has_many LiveDataAssigns
Added Content context functions for working at both levels
Changed Beacon.DataSource behaviour to no longer care about site because each site will have its own module
page_title/2 and meta_tags/2 callbacks are deprecated (in the aforementioned behaviour)
Page title now accepts Solid snippets
Beacon.Loader now loads a data source module per site
DataSourceModuleLoader generates data source modules
Deleted unused beacon_web/controllers/api/{page,component}_controller.ex and the related tests which were broken by these changes
Deleted beacon/data_source_test.exs which didn't seem relevant with the new approach
Added a bunch of new tests for context functions and validating the DataSourceModuleLoader outputs
Added test fixtures, removed the mock data source module

Data Source Module Example
Generated by the DataSourceLoader (whitespace added, maps truncated here for readability):
defmodule :"Elixir.BeaconWeb.LiveRenderer.82CBFE60DC5B9C46C4C0BC8ADDAB657ADataSource" do
  @behaviour Beacon.DataSource.Behaviour
  require Logger

  @impl Beacon.DataSource.Behaviour
  def live_data(path, params) do
    live_data(path, params, %{})
  end

  def live_data(["forum"], var!(params), var!(data)) do
    %{} = var!(params)
    Enum.reduce([%{..., key: "boards", value: "MyApp.Forum.Boards.get_board_list()"}], var!(data), fn assign, acc ->
      Map.put(
        acc,
        String.to_atom(assign.key),
        case assign.format do
          :text -> assign.value
          :elixir -> assign.value |> Code.eval_string([], __ENV__) |> elem(0)
        end
      )
    end)
  end

  def live_data(["forum", board_slug], var!(params), var!(data)) do
    %{} = var!(params)
    Enum.reduce([%{..., key: "board", value: "MyApp.Forum.Boards.get_board_by_slug(board_slug)"}, %{..., key: "posts", value: "MyApp.Forum.Posts.get_posts_for_board(board_slug)"}], var!(data), fn assign, acc ->
      Map.put(
        acc,
        String.to_atom(assign.key),
        case assign.format do
          :text -> assign.value
          :elixir -> assign.value |> Code.eval_string([board_slug: board_slug], __ENV__) |> elem(0)
        end
      )
    end)
  end

  def live_data(["forum", board_slug, post_id], var!(params), var!(data)) do
    %{} = var!(params)
    Enum.reduce([%{..., key: "post", value: "MyApp.Forum.Posts.get_post!(post_id)"}, %{..., key: "comments", value: "MyApp.Forum.Posts.get_comments_for_post(post_id)"}], var!(data), fn assign, acc ->
      Map.put(
        acc,
        String.to_atom(assign.key),
        case assign.format do
          :text -> assign.value
          :elixir -> assign.value |> Code.eval_string([board_slug: board_slug, post_id: post_id], __ENV__) |> elem(0)
        end
      )
    end)
  end

  def live_data(path, params, data) do
    Logger.warning("Unhandled Beacon Live Data request for site #{:dockyard_com} with path #{inspect(path)} and params #{inspect(params)}")
    data
  end
end

❤


Fix #23

Fixed https://github.com/ezura/spell-checker-for-swift/issues/23

@RustamG Thank you!
(And I'm sorry for late)


feat: unenroll without a refund

Pre-Flight checklist

[ ] Screenshots and design review for any changes that affect layout or styling

[ ] Desktop screenshots
[ ] Mobile width screenshots

[ ] Migrations

[ ] Migration is backward-compatible with the current production code

[ ] Testing

[ ] Code is tested
[ ] Changes have been manually tested

[ ] Settings

[ ] New settings are documented and present in app.json
[ ] New settings have reasonable development defaults, if applicable
[ ] Opened issue for DevOps regarding necessary configuration changes to deployed environments

What are the relevant tickets?
#1321
What's this PR do?
(Required)
How should this be manually tested?
(Required)
Screenshots (if appropriate)
(Optional)

Codecov Report

Merging #1333 (b75c57c) into main (b42a427) will increase coverage by 3.29%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1333      +/-   ##
==========================================
+ Coverage   83.20%   86.50%   +3.29%
==========================================
  Files         307      146     -161
  Lines       12789     4215    -8574
  Branches      918        0     -918
==========================================
- Hits        10641     3646    -6995
+ Misses       1891      569    -1322
+ Partials      257        0     -257

Impacted Files
Coverage Δ

users/models.py

cms/wagtail_hooks.py

...ement/commands/generate_legacy_enrollment_codes.py

hubspot_sync/tasks.py

flexiblepricing/factories.py

cms/constants.py

courses/__init__.py

main/context_processors.py

authentication/api.py

repl.py

... and 151 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Feature Request: read-only pre-0.10 BackupStorageLocation

I'm trying to transfer from an older cluster to a new cluster.   During the transition, both clusters need to be up.
The old cluster runs ark 0.8.1, the new one 0.10.0.   I figured I'd just take advantage of your new support for multiple locations, and add both the old and new buckets to the new ark setup.   However, when I do that, I get the error:
An error occurred: some backup storage locations are invalid: backup store for location "aws-old" is invalid (if upgrading from a pre-v0.10 version of Ark, please refer to https://heptio.github.io/ark/v0.10.0/storage-layout-reorg-v0.10 for instructions): Backup store contains 570 invalid top-level directories:
I can't transition the old bucket because the old cluster is still using it.   I don't want to upgrade the old cluster because the whole point is to get rid of it; also the cluster is somewhat fragile/broken so I want to touch as little as possible.
It'd be nice to have a way to use a pre-0.10 BackupStorageLocation.   This can & should be read-only.
Environment:

Ark version (use ark version): 0.8.1 and 0.10.0
Kubernetes version (use kubectl version): v1.11.2
Kubernetes version: v1.9.7 & v1.11.4-gke.12
Cloud provider or hardware configuration: metal/kubespray & GKE
OS (e.g. from /etc/os-release): Ubuntu 18.04.1

hi @bryanlarsen - my recommendation would be to do a full copy of your v0.8.1 bucket into a new bucket, and then to use the script here to upgrade the new bucket to be v0.10-compatible, which you can then add as a backup storage location to your v0.10 install.  So the steps would be something like:

copy the full contents of your v0.8.1 bucket to a new bucket (I'll call it migration-bucket). We won't touch the v0.8.1 source bucket in any of the subsequent steps
run our migration script against migration-bucket to upgrade it to be v0.10-compatible (note that the script will create a third temporary bucket in order to perform the migration)
configure a BackupStorageLocation that points to migration-bucket.
if appropriate, put the v0.10 ark instance into restore-only mode, as documented here in step 7b.

Hope this helps - let me know if you have any questions!

The procedure you're describing is a one time procedure, I need something ongoing.  For a ~ two week period, both the old & new clusters will be writing to their respective buckets.
It appears that I can just copy backup directories from the old bucket into the new bucket, though.   I lose some metadata (the created timestamp on the copied backup is 0001-01-01 00:00:00 +0000 UTC for some reason), but it still worked for a restore.

Perhaps I spoke too soon.    I was able to do this for one backup, but when I tried it for another, it's not showing up in ark backup get.

to get it to pick up the new directory I deleted metadata/revision and restarted the pod.

Ah, I see. Yes - if you manually copy backups into the v0.10 bucket, you either need to delete the metadata/revision file, or wait up to an hour for those backups to appear. Ark's using the revision file to determine if the bucket contents have changed, as it doesn't expect anyone else to be modifying the contents.
Perhaps you could set up a periodic s3 sync job to copy backups from the v0.8.1 bucket into the v0.10 bucket?

I liked the idea here but as we're approaching v1.0 this is no longer something we're going to prioritize. Closing as "Won't Fix".


Two Factor Authentication

Authy has a free plan for developers that seems to fit our needs. It can be found at https://www.authy.com/developers/

@nywillb I created an app on Authy, but it doesn't look like it's free.. However, we could use TOTP. This would give us compatibility with apps like Google Authenticator and Authy, but we wouldn't have text messages...

By created an app, you mean what?

@RobotXLabs created an application on the Authy Dashboard. It's marked as "Pay as you Go", so....

@thatoddmailbox TOTP looks hard...

@nywillb https://www.npmjs.com/package/notp

@thatoddmailbox Nevermind.

@thatoddmailbox Do you want to knock this out right now?

@nywillb Let me finish some theme color things first.

@thatoddmailbox Okay.

@nywillb The only problem is account recovery. How do we handle someone who doesn't have access to their device (lost, stolen, broke, etc..)?

@thatoddmailbox Give them a code?

@thatoddmailbox Have them contact us?

@thatoddmailbox Verify by email?

@nywillb but how? (if they've lost their device). And if they can contact us to disable it, what's stopping meanie hexr from doing the same?

@nywillb email could work, but we would need to set up an email system to send messages...

@thatoddmailbox It's not that hard.


Handle not finding config gracefully

Describe the bug
Currently when Paul cant find a PAUL.yaml he returns and error on the webhook, it would be nice if Paul handled this case gracefully
To Reproduce
Install paul and dont add a PAUL.yaml file
The error will be this:
paul | 2021-04-21 23:33:36 time="2021-04-21T21:33:36Z" level=error msg="webhook error occurred" error="unable to download config file: No file named PAUL.yaml found in .github"

Expected behavior
Still return a 200 in response to Githubs Webhook
Additional context
The easiest way to do this is to check the response of the configuration get and if its 400 return an empty configuration and no error

/label bug

/giphy bug

Fix Seems to not have worked


plugin setting buttons margin fix

changelog(Fixes): Fixed UI margin bug in Plugin Settings
margin top added on buttons div in plugin settings
before :

after:

@therealrinku you can claim a free tshirt for this PR contribution - find more at https://konghq.com/community/open-source-contribution

@therealrinku you can claim a free tshirt for this PR contribution - find more at https://konghq.com/community/open-source-contribution

Thank you!! I'll check it out


Crypto_material should be localised to a single folder

Ideally, we should save all crypto-material (MSPs and the genesis/channel blocks) should be placed in a single folder, such that it is easier to get rid of test examples.

Addressed by #48


Lifting

This is not yet fully ready, but I wanted to get some feedback from you, esp. @bmmoore, before tomorrow's iteration meeting.

@mattoxb thanks for taking a look! of course the structure is cleaner.  lifting gets rid of all the object patterns and stuff and the complications came from the need to be able to express both meta and object stuff in an unified way.  Therefore, I was able to use a standard Fix construction for meta patterns type and catamorphisms as (bottom-up) visitors.  I guess we could write a top-down visitor for it, too, but my primary focus so far was to implement lifting an unlifting and get some integration with existing stuff (such as parsing & unparsing)

@virgil-serbanuta @mattoxb @bmmoore please re-review.  I'll only be adding tests tomorrow, and I'd like to get it merged soon since it's been too long since I started it.
A lot of it are refactorings in my (unsuccessful so far) attempt to use Virgil's implicit kore safe pattern building methods for the lifting process.  I might want to retry that at some later moment.

BTW, you may still have comments from the previous review pass that are still neither answered nor fixed.


Slow to get a functional TS editor

VSCode Version: Code - Insiders 1.15.0-insider (9b6cc9f774f424b4e8a91d4a2d3d4ad8efb9cfb9, 2017-07-24T05:07:43.734Z)
OS Version: Darwin x64 16.6.0
Extensions:

Extension
Author (truncated)
Version

vscode-docker
Pet
0.0.16

quokka-vscode
Wal
1.0.41

crazy
kie
0.0.1

githistory
don
0.2.2

python
don
0.6.9

tslint
eg2
0.17.0

vscode-npm-script
eg2
0.2.0

vsc-travis-ci-status
fel
1.1.0

svgpreview
kis
0.2.0

Go
luk
0.6.62

mssql
ms-
1.1.0

github-issues-prs
ms-
0.3.1

debugger-for-chrome
msj
3.1.6

vscode-versionlens
pfl
0.19.1

Ruby
reb
0.12.1

kustovscode
sea
0.0.1

php-syntax-visualizer
vsc
0.0.1

vscode-open-in-github
ziy
1.2.7

(1 theme extensions excluded)

Steps to Reproduce:

open a VS Code workspace on master, no open editors
search for from 'electron' in all *.ts files
click on the first search result and wait until TS is properly initialized
click on the second search result and select ipcMain in the editor
-> caret stops blinking, it takes several seconds until the underlying word is highlighted and occurrences are shown in the scrollbar (only happens after the reference code lenses are shown)

Reproduces without extensions: Yes
This is the TS server log

Thanks @kieferrm. I've opened https://github.com/Microsoft/TypeScript/issues/17385 to track this upstream.
I was able to reproduce this after closing all editor windows which leads me to suspect that the slowdown may be caused by us reloading some or part of the vscode project when switching between the search results


CLI installation script fails on Amazon Linux 2

Current Behavior
Configu CLI installation script (curl https://cli.configu.com/install.sh | sh) is failing on Amazon Linux 2 (EC2, Bastion machine)
Expected Behavior
Installation script should work on AL2
To Reproduce

create a machine with Amazon Linux 2 (ec2 machine)
run cli installation script
see error

Context

Hey @rannn505 ! I would like to work on this please assign

@Sadaf-A All yours brother :)
Do you have access to any aws account in order to pick up an ec2 machine with al2?
DA, if you produce a machine in the micro tier, it should be included in the free aws offer.
In addition, I would be happy if in your test you could produce another machine with a newer Amazon Linux (I read that version 2023 was released) and check whether the script works on it or not on this occasion.
Thanks in advance.

@Sadaf-A All yours brother :)
Do you have access to any aws account in order to pick up an ec2 machine with al2?
DA, if you produce a machine in the micro tier, it should be included in the free aws offer.
In addition, I would be happy if in your test you could produce another machine with a newer Amazon Linux (I read that version 2023 was released) and check whether the script works on it or not on this occasion.
Thanks in advance.

yes, I have an aws account I will let you know about any progress

@rannn505 the error was resolved however i ran into another error that said node is not installed we can make our own AMI with node pre installed to resolve this issue or we could add a user data script

/usr/local/lib/configu/bin/node: /lib64/libm.so.6: version `GLIBC_2.27' not found (required by /usr/local/lib/configu/bin/node)
/usr/local/lib/configu/bin/node: /lib64/libc.so.6: version `GLIBC_2.28' not found (required by /usr/local/lib/configu/bin/node)
Configu CLI installed to /usr/local/bin/configu
node: /lib64/libm.so.6: version `GLIBC_2.27' not found (required by node)
node: /lib64/libc.so.6: version `GLIBC_2.28' not found (required by node)

I'm getting this output finally after installing node

@Sadaf-A this is very strange as the cli is compiled and shipped with node.js binary in it. can you try installing node.js 20.10 on the AL2 machine and let me know if it works?
also, when you said "resolved", what do you mean?

@rannn505 the error was resolved however i ran into another error that said node is not installed we can make our own AMI with node pre installed to resolve this issue or we could add a user data script

@Sadaf-A this is very strange as the cli is compiled and shipped with node.js binary in it. can you try installing node.js 20.10 on the AL2 machine and let me know if it works? also, when you said "resolved", what do you mean?

@rannn505 the error was resolved however i ran into another error that said node is not installed we can make our own AMI with node pre installed to resolve this issue or we could add a user data script

what I meant by error being resolved was that the Your path is missing usr/local/bin, you need to add this to use this installer error is not coming anymore


PLAT-88734 Remove IconItem from Agate (it is deprecated)

Deleted IconItems files in App

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Renuka Atale seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Cannot read property 'environment' of undefined at AngularCompilerPlugin

Hi.
After I've upgraded my Asp core project from Angular 5 to 6 I got TypeError: Cannot read property 'environment' of undefined only when I deploy/publish:
D:\_dev_\PosWeb\node_modules\@ngtools\webpack\src\packages\ngtools\webpack\src\angular_compiler_plugin.ts:581
    compiler.hooks.environment.tap('angular-compiler', () => {
                   ^
TypeError: Cannot read property 'environment' of undefined
    at AngularCompilerPlugin.apply (D:\_dev_\PosWeb\node_modules\@ngtools\webpack\src\packages\ngtools\webpack\src\angular_compiler_plugin.ts:581:20)
    at Compiler.apply
....
D:\_dev_\PosWeb\PosWeb.csproj(62,5): Error MSB3073: The command "node node_modules/webpack/bin/webpack.js --env.prod" exited with code 1.

thanks
Angular CLI: 6.0.0
Node: 9.11.1
OS: win32 x64
Angular: 6.0.0
... animations, cli, common, compiler, compiler-cli, core, forms
... http, platform-browser, platform-browser-dynamic
... platform-server, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.6.0
@angular-devkit/build-angular     0.6.0
@angular-devkit/build-optimizer   0.6.0
@angular-devkit/core              0.6.0
@angular-devkit/schematics        0.6.0
@angular/language-service         5.0.1
@ngtools/webpack                  6.0.0
@schematics/angular               0.6.0
@schematics/update                0.6.0
rxjs                              6.1.0
typescript                        2.8.3
webpack                           2.5.1

The webpack version appears to be 2.5.1.  CLI v6 depends on webpack 4.3+ and webpack should not generally be installed directly within the project.

Hi,can you give me a working solution running  npm install  webpack@latest not solving the problem.thanks

If webpack is present within the project's package.json, try removing it completely.  Then to ensure the packages are installed properly before running npm install remove both the node_modules folder and the package-lock.json file.

Hi.thanks for your time.There are 2 day spent trying to update from angular 5 to 6 :(
I think my my asp core project is based on webpack,for example if I update bootstrap I have to run  webpack --config webpack.config.vendor.js
I've removed and now got  Cannot find module webpack:
AggregateException: One or more errors occurred. (Webpack dev middleware failed because of an error while loading 'aspnet-webpack'. Error was: Error: Cannot find module 'webpack'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:548:15)


When minifying, it changes the css order!

I have a css file which has lots of @media queries. When setting minify: true it combines all the same @media query together which screws the order of my css rules!
Consider the following rules in a css file,
@media (min-width:46.25em) {
    .c-tooltip--hover:hover .c-tooltip__content, .is-active .c-tooltip__content {
        width: 20em
    }
}

@media (min-width:20em) {
    .c-class-name {
        height: 30em
    }
}

@media (min-width:46.25em) {
    .c-class-name {
        height: 40em
    }
}

when setting minify:true, the above code gets minified into the following,
@media (min-width:46.25em) {
    .c-tooltip--hover:hover .c-tooltip__content, .is-active .c-tooltip__content {
        width: 20em
    }
    .c-class-name {
        height: 40em
    }
}

@media (min-width:20em) {
    .c-class-name {
        height: 30em
    }
}

Now if you see the .c-class-name 's height: 40em  has been grouped with the other rules for the same @media query rule. This has changed the order of css which breaks styling of my elements on the page.

Minification is done through CleanCss so the problem is more related to cleancss than purifycss.
A solution might be to turn off the minification in your purifycss option and used a css minification tool afterward.


[Enhancement]: Exit Button in the game

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
It would be better to add Exit button in the game for more responsive to the user/gamer.
Please assign me this task.
Describe the solution you'd like
Adding the Exit button

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Though we can refresh the page to go back to home page, it will be user-friendly if there will be a 'Exit' button. Please assign me this issue.

pls assign me this issue

If you have any doubt you can directly send message on our discord (https://discord.gg/rZb46cCMmK), we have created separate channels for all projects...
Thank you :)


Scribble fails on ternary operator in tuple assignment

function f(bool cond) internal pure returns (uint[2] memory ret) {
  (ret[0], ret[1]) = cond ? (1, 2) : (3, 4);
}

results in
Error: Unexpected rhs in tuple assignment: Conditional#1710
    at getAssignmentComponents (/Users/alex/.nvm/versions/node/v16.8.0/lib/node_modules/eth-scribble/dist/instrumenter/state_vars.js:41:19)
    at getAssignmentComponents.next (<anonymous>)
    at getAssignments (/Users/alex/.nvm/versions/node/v16.8.0/lib/node_modules/eth-scribble/dist/instrumenter/state_vars.js:124:20)
    at getAssignments.next (<anonymous>)
    at Object.findAliasedStateVars (/Users/alex/.nvm/versions/node/v16.8.0/lib/node_modules/eth-scribble/dist/instrumenter/state_vars.js:272:21)
    at new InstrumentationContext (/Users/alex/.nvm/versions/node/v16.8.0/lib/node_modules/eth-scribble/dist/instrumenter/instrumentation_context.js:231:38)
    at Object.<anonymous> (/Users/alex/.nvm/versions/node/v16.8.0/lib/node_modules/eth-scribble/dist/bin/scribble.js:582:26)
    at Module._compile (node:internal/modules/cjs/loader:1101:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)

Thank you for reporting this! Will take a look and root-cause tomorrow!

First a bit of context: In order to detect when its safe to instrument state variables (for #if_updated annotations), we need to detect when a state variable may be aliased. To detect that we need to locate all of the places in the AST where the state variable may be assigned to a local/argument/return etc. This is done in getAssignments. This logic has to deal with tuple assignments, and so far it has only dealt with simple cases of tuple assignments - tuple to tuple and function call to tuple. It doesn't yet handle assignments where the RHS is a conditional (causing the exception you see).
To fix this, we must first add the following case to getAssignmentComponents:
--- a/src/instrumenter/state_vars.ts
+++ b/src/instrumenter/state_vars.ts
@@ -72,6 +72,9 @@ function* getAssignmentComponents(lhs: Expression, rhs: Expression): Iterable<[L

                 yield [lhsComp, [rhs, i]];
             }
+        } else if (rhs instanceof Conditional) {
+            yield* getAssignmentComponents(lhs, rhs.vTrueExpression);
+            yield* getAssignmentComponents(lhs, rhs.vFalseExpression);
         } else {
             throw new Error(`Unexpected rhs in tuple assignment: ${pp(rhs)}`);
         }

And must also modify explodeTupleAssignment to correctly handle this tuple case.

Thanks, this seems to be working with develop.


max_query_size URL parameter stopped working

We use max_query_size as JDBC URL parameter to increase allowed select query size:
jdbc:clickhouse://ch-host:8123/db?max_query_size=1000000
After switching from the deprecated package ru.yandex.clickhouse to the new one com.clickhouse.jdbc we discovered that this parameter is ignored and currently we have exceptions:
java.sql.BatchUpdateException: Code: 62. DB::Exception: Syntax error: failed at position 262144 ('86') (line 16, col 261384): 86. Max query size exceeded: '86'. (SYNTAX_ERROR) (version <IP_ADDRESS> (official build))
, server ClickHouseNode [uri=http://ch-host:8123/db, options={max_query_size=1000000,socket_timeout=30000,async=true,http_keep_alive=true}]@1448807023
	at com.clickhouse.jdbc.SqlExceptionUtils.batchUpdateError(SqlExceptionUtils.java:109)
	at com.clickhouse.jdbc.internal.SqlBasedPreparedStatement.executeAny(SqlBasedPreparedStatement.java:208)
	at com.clickhouse.jdbc.internal.SqlBasedPreparedStatement.executeQuery(SqlBasedPreparedStatement.java:235)

Position 262144 is related to the default limit of 256 KiB.

Hi @konstantinosss, you can add custom_http_params in connection property or url, for examples:

connection propertyProperties props = new Properties();
// comma separated custom settings, you can add custom_http_headers in a similar way as well
props.setProperty("custom_http_params", "session_check=0,max_query_size=1000");

connection urlString url = "jdbc:clickhouse://ch-host:8123/db"
  // same as above but url encoded
  + "?custom_http_params=session_check%3D0%2Cmax_query_size%3D1000";

Sorry this is more of a workaround at this point because it's limited to http protocol. I'll add custom_settings to support more protocols.

I have url string with such params:
/?socket_timeout=6000000&max_bytes_before_external_group_by=16000000000&optimize_aggregation_in_order=0&join_default_strictness=ANY&join_algorithm=auto&max_memory_usage=20000000000"

After switching from ru.yandex.clickhouse 0.3.1 to  0.3.2-patch11 these params don't work in all cases:

Using params in url connection string
Set every property individually and pass to ClickHouseDataSource
Set via custom_http_params

Checking with SELECT * FROM system.settings.  It returns e.g for max_bytes_before_external_group_by - value  "0" and changed 0 instead of "16000000000" and changed 1 when using old driver.

Hi @ilevd, will it work if you use custom_settings in latest nightly build at here? Feel free to open an issue if still not working.
https://github.com/ClickHouse/clickhouse-jdbc/blob/4d6fe526b38c1e8c64b50f871ca36c72c2c4e713/clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/internal/ClickHouseConnectionImplTest.java#L220-L221

but I had to remove socket_timeout option and set it in url

socket_timeout is not a server setting so you have to put it separately in connection string or properties.

Am I right that all settings from ClickHouse Settings should be in custom_settings?

Yes, maybe it should be renamed to custom_server_settings but it's getting too long to type in :p On a side note, if you have many settings to mange, you probably should consider to use server profile instead to benefit all type of clients instead of just JDBC.

Or could you please explain when to use custom_http_params, custom_settings and ordinary properties? And are there plans to support all params in url string?

custom_http_params only works when using http protocol and it should support arbitrary parameter regardless it's a server setting or not. The driver will simply append the parameters to server url and then send http request accordingly. custom_settings on the other hand is more generic and it's also supported by other protocols like grpc and tcp. I think in most cases we should use custom_settings, but if you're using http and no plan to switch to any other protocol, you may use custom_http_params as well.

@zhicwu, thanks!


Add test to showcase included build annotation processor issue

We are seeing issues with using annotation processor results from included builds when retrieving the information from the cache. I mirrored our setup in this test, by introducing two new projects:

A new included build called processor-build that defines an annotation processor and uses the Google AutoService to define it
A new submodule in this project, which uses the processor to process one of its fields

The problem is the following (although I couldn't fully define that in the test): when we run :projectHealth on this, it stores the information in our build cache. When we then attempt to retrieve it in a subsequent run, the :projectHealth task fails with the following error:
> Transitively used dependencies that should be declared directly as indicated:
    annotationProcessor project('my.custom.processor:sub-processor1')

It seems like the information retrieved from the cache is incomplete, where it somehow deduces that the annotation processor is unused. Again, if we run it without the cache, it is all fine. Only when the :user-of-processor:findDeclaredProcsMain is FROM-CACHE, it fails.
This only happens when using included builds. If the subprojects are part of the same build, we don't observe any caching problems.
Next to that, we also noticed that the usages-annotation-processors.json includes fully qualified project references. However, the cache key for this task does not take into account the fully qualified names of dependencies. In the sense that: when we run the task in 1 build (where the rootProject.group is set to Group1) and we run it again in another build where the group is set to Group2, the task would also fail, as it doesn't match the group.
We fixed this issue by aligning the group names, but I think that's a problem that can be fixed in the plugin as well. To do so, we can mark the rootProject.group as inputs.property() to these tasks. (Note that the rootProject.group is of the dependency, not the project that defines the plugin task, which makes it slightly more complicated).
I created this PR as a request of @jjohannes who wanted to see the example. I am not sure Jendrik if I did this correctly and if these test expectations are correct. I had trouble trying to formulate what the advice should be, as the test is now failing with an assertion error I don't understand. Hopefully it is clear enough with this setup.

I think #853 is related to this issue

This was included in #916


Deauth does not work for me...

I installed the bin with ESPFlasher and the acces point works but when I enter to the interface, i can't deauth the router and create the evil twin.

btw good work, keep working!!

Only problem is deauth not working,
Evil twin is working file, recheck again. If still also same problem then I'll change the bin file.
I will fix deauth in next coming month ✌. Thank you.

True, today Evil twin works for me.. but deauth still not working... Ill be waiting for next version. Good Work!!


eve_flash_pico.uf2 not doing anything

I wanted to try out flashing with a Pi-Pico.
When I use reset+boot the drive "RPI-RP2" pops up in my Windows 11.
When I drag eve_flash_pico.uf2 into that drive it vanishes and does not come back as "BTEVE-FLASH".
USB Device Tree Viewer shows that there is nothing on the USB after using eve_flash_pico.uf2.
Using eve_flash_pico.uf2 from EAB 2.8.0 has the same result - the USB port is just dead afterwards.
"Connection Status        : 0x00 (No device is connected)"
I also used flash_nuke.uf2 to make sure that the content of the flash of my original Pi-Pico is cleared out.

Ok, found the issue, it is required for eve_flash_pico.uf2 to work to actually have a display connected to the Pi-Pico first.

And I ran into the next issue.
I connected a RVT50HSBNWN00 with a BT817 and it works with my software on the Pi-Pico after I configured PD to pin 7.
I was using BT815-unified.blob and BT817-unified.blob from this repository.
BT817-unified.blob is 100% the same as EVE Asset Builder\tools\BT817-unified.blob and EVE Asset Builder\flash_blob\bt817.blob in EAB 2.8.0 - it does however not work.
BT815-unified.blob from this repository is the same as EVE Asset Builder\tools\BT815-unified.blob.
However, EVE Asset Builder\flash_blob\bt815.blob is a totally different file.
CMD_FLASHERASE
python uf2conv4eve.py -f BT815 bt815.blob -i flash.bin
-> works
CMD_FLASHERASE
python uf2conv4eve.py -f BT817 bt817.blob -i flash.bin
-> does not work
python uf2conv4eve.py -f BT815 bt815.blob -i flash.bin
-> works
python uf2conv4eve.py -f BT817 bt817.blob -i flash.bin
-> works
CMD_FLASHERASE
python uf2conv4eve.py -f BT817 bt817.blob -i flash.bin
-> does not work
python uf2conv4eve.py -f BT817 bt817.blob -i flash.bin
-> does not work
python uf2conv4eve.py -f BT815 bt815.blob
-> does somewhat work but the flash data is not correct
python uf2conv4eve.py -i flash.bin
-> works
python uf2conv4eve.py -f BT817 bt817.blob
-> still works
Reading back the flash with EAB 2.8.0 and a MPSSE adapter reveiled though that the bt817.blob was actually not written to the flash, the bt815.blob is still there.
CMD_FLASHERASE
python uf2conv4eve.py -f BT817 bt817.blob
Reading back the flash with EAB 2.8.0 and a MPSSE adapter does not only result in a 16 MiB file instead of the 64 MiB expected file, the file also is completely empty - filled with 0xff.
I generated a new flash image now with  /flash_blob/bt817.blob.
I flashed it with EAB, I checked the .bin, I read it back, the first page is from bt817-blob.
I switched back to the Pi-Pico - the software works.
CMD_FLASHERASE
Flashing the new flash image with EAB 2.8.0 and the Pi-Pico.
-> does not work
Reading back the flash image with EAB and MPSSE
-> the flash image is completely empty - filled with 0xff.
For some reason uf2conv4eve.py and uf2conv4eve.exe do not write the b817.blob to an empty flash.

When I drag eve_flash_pico.uf2 into that drive it vanishes and does not come back as "BTEVE-FLASH".
Ok, found the issue, it is required for eve_flash_pico.uf2 to work to actually have a display connected to the Pi-Pico first.

Hi RudolphRiedel, the eve_flash_pico.uf2 require Pi-Pico connecting to an EVE, no need LCD.

However, EVE Asset Builder\flash_blob\bt815.blob is a totally different file.

EAB developer recommend using blob files in "EVE Asset Builder\flash_blob"

For some reason uf2conv4eve.py and uf2conv4eve.exe do not write the b817.blob to an empty flash.

Please share me how you send CMD_FLASHERASE?
By the way, the eve_flash_pico.uf2 does not support updating the blob file yet. I will add this feature later.

Ok, found the issue, it is required for eve_flash_pico.uf2 to work to actually have a display connected to the Pi-Pico first.
Hi RudolphRiedel, the eve_flash_pico.uf2 require Pi-Pico connecting to an EVE, no need LCD.

Yes ok, I only do not have an EVE board that is not connected to a display, I am exclusively using complete modules.

However, EVE Asset Builder\flash_blob\bt815.blob is a totally different file.
EAB developer recommend using blob files in "EVE Asset Builder\flash_blob"

Ok.

For some reason uf2conv4eve.py and uf2conv4eve.exe do not write the b817.blob to an empty flash.
Please share me how you send CMD_FLASHERASE?

I was using EAB and a MPSSE adapter.
But switching between the MPSSE and the Pi-Pico is not something I like to do more often than necessary. :-)
So I also used EVE_cmd_flasherase(); from my library.
I erased the flash again and
python uf2conv4eve.py -f BT817 bt817.blob -i flash.bin
is not working.
I tried this:
python uf2conv4eve.py -p -f BT817 bt817.blob -i flash.bin
And now that flash is programmed with my flash image and my software works.
Including EVE4 firmware blob bt817.blob
Launching H: (BT8XX-RP2040-v0.0)
Setting SPI pin: [['MISO', 4], ['CS', 5], ['SCK', 2], ['MOSI', 3], ['INT', 6], ['PWD', 7], ['IO2', 14], ['IO3', 15]]
spin= MISO: 4
CS: 5
SCK: 2
MOSI: 3
INT: 6
PWD: 7
IO2: 14
IO3: 15
100%|████████████████████████████████████████████████████████████████████████████████████████████| 8/8 [00:00<?, ?kB/s]
Wrote 67 bytes to H:/SPI_PIN.CNF
Converting to uf2, output size: 131072
Flashing H: (BT817-RP2040-v1.0)
100%|███████████████████████████████████████████████████████████████████████████████| 128/128 [00:00<00:00, 134.74kB/s]
Wrote 131072 bytes to H:/NEW.UF2
Hmm, I just got this on the SPI:

Note that CS is not going low.
My software is using the exact same pin configuration and working just fine.
What I also noticed is that the eve_flash_pico.uf2 is no longer displaying anything on the screen,
the text and the matrix-like animation is no longer displayed.
If that is on purpose, ok, why not - but it looks like the backlight still gets activated, please turn it off as well.

By the way, the eve_flash_pico.uf2 does not support updating the blob file yet. I will add this feature later.

It does not?
I looked over the sources and it only seems to be using CMD_FLASHUPDATE.
And CMD_FLASHUPDATE works fine in flash state BASIC.

I looked over the sources and it only seems to be using CMD_FLASHUPDATE.

Yes, it's reuse EveApps to update blob.
Could you try 7b0a3c6? I've just updated the SPI pins configuration to default.

Opening two issues at the same time was probably not the best idea. :-)
The new set of uf2conv4eve.py + eve_flash_pico.uf2 does work better now including setting up the pins.
I can flash using this line:
python uf2conv4eve.py -p int-7 pwd-6 -f BT817 BT817.blob -i funtype.bin
There still is an issue though.
I always have to use that line twice.
BTEVE USB drive detected at ['H:']
No BTEVE drive is detected, existing...
It works on the second call, during flashing the display is active again including the "matrix-effect" style animation.
And after flashing there is a reset now with the drive switching back to RPI-RP2.
From uf2conv4eve.py line 344+:
if tries == 100:
print("Timeout! UF2 is just written but EVE usb drive is not detected")
return False
else:
print("BTEVE USB drive detected at", eve_drives)
return True
I added the last line, with this it is working on the first call.

Hi @RudolphRiedel ,

python uf2conv4eve.py -p int-7 pwd-6 -f BT817 BT817.blob -i funtype.bin
I always have to use that line twice.

Please try option "-w" to see if it works at the 1st time.

Hi @RudolphRiedel ,

during flashing the display is active again including the "matrix-effect" style animation.

I will hide this animation because it makes other users confuse what's going on.
I will replace it with a progress bar and thinking how to detect screen size to make it fit the LCD. May let user input the LCD size as a command argument.

Hi @RudolphRiedel ,

python uf2conv4eve.py -p int-7 pwd-6 -f BT817 BT817.blob -i funtype.bin
I always have to use that line twice.

Please try option "-w" to see if it works at the 1st time.

While this should help as the exit point from write_eve_flash_pico()
would then be line 318, there is a bug that I fixed on my side, see above.
The output is:
BTEVE USB drive detected at['H:']
No BTEVE drive is detected, existing...
This led me to line 344+ and I found that a "return True" line is missing after
the print("BTEVE USB drive detected at", eve_drives) line.

Hi @RudolphRiedel ,

during flashing the display is active again including the "matrix-effect" style animation.

I will hide this animation because it makes other users confuse what's going on. I will replace it with a progress bar and thinking how to detect screen size to make it fit the LCD. May let user input the LCD size as a command argument.

I would prefer if the display does not show anything and that the backlight also is disabled.
Perhaps implemented as an option.
The reason is that the displays need to be powered and that is increasingly difficult from 7" and up.
And I also would apreciate to have a second .uf2 file that is written to the flash of the RP2040 and therefore permanent,
manual installation would be fine.

Hi @RudolphRiedel,

would prefer if the display does not show anything and that the backlight also is disabled.
to have a second .uf2 file that is written to the flash of the RP2040 and therefore permanent

Applied in 4e91e78

to have a second .uf2 file that is written to the flash of the RP2040 and therefore permanent
Applied in https://github.com/Bridgetek/pico-brteve/commit/4e91e780f0dddb847f632e34614f7a7edeb31127

Nice, no complaints from me. :-)
I downloaded the branch and I am in the process of testing it.
But, isn't that a bit far for you?
My idea was to have a Pi-Pico dedicated to nothing else, connect it to an adapter, preferably apply some electrical isolation and have it ready when I want to flash any BT81x based module.
But then you have hardware like the IDM2040-7A and MM2040EV and users of these might prefer a non-permanent solution.
I also found it conveniant to test the display with the same Pi-Pico I was flashing the module with, I am only not using RP2040 controllers for anything else than testing since they do not have the hardware units I need.
And as I am still not sure if I even should try a pull-request against a branch and if I can actually fork a branch to try it,
since "uf2conv4eve.py" did not change it still is missing a line.
In the repository now:

The version with the additional "return True" I am using now:

Ok, I used my second Pi-Pico and an adapter board and soldered wires between these for a permanent connection.
And it works, but not entirely as expected.
I need to supply the "-p int-7 pwd-6" option or it will not flash the image even if the pins are already configured.
After flashing the image the Pi-Pico is reset by the application into Boot mode.
And when trying to flash another image the .uf2 gets written to the Pi-Pico which makes the persistent application kind of pointless. :-)
So, I guess with regards for boards like the IDM2040-7A and in order to not go deeper into the rabbit hole of changes,
it might be a good idea to revert the change of having the application persistent.
This works fine for me now:
python uf2conv4eve.py -p int-7 pwd-6 -f BT817 BT817.blob -i Test_Image_2.bin
And what I really should do is to find a way to build the application for the Pi-Pico so that I can change it to whatever I prefer without bothering you with things that you do not actually need.
I feel like I should do pull requests and not bug reports, at least for things that qualify as bug fixes.

Hi @RudolphRiedel ,

And what I really should do is to find a way to build the application for the Pi-Pico

Please follow this  guideline: https://github.com/Bridgetek/pico-brteve/blob/main/tools/uf2/source/README.md#Build

Hi @RudolphRiedel,

it might be a good idea to revert the change of having the application persistent

Reverted as your suggest. please check 3d78fa3

Please follow this guideline

I am a bit reluctant to do that and this entirely on my side. I am not at all happy with the RP2040 toolkit for Windows.

Reverted as your suggest. please check https://github.com/Bridgetek/pico-brteve/commit/3d78fa33191bc6405a70ecaab7453fe56ffb0556

I just grabbed uf2conv4eve.py and eve_flash_pico.uf2 from that branch.
I had no issue programming the flash image and after a reset my application was running again.
So thank you, I will close the issues now.


Clarification on R10.4 and polya

It is clear that eventalign-dependent modules does not support R10.4, but it looks like polya uses non-eventaligned durations. Does this mean that polya should be fine to run on R10.4 data, or should it be considered untrustworthy?

The polyA module is only for direct RNA sequencing, not DNA, so it doesn't work with any of the DNA chemistries (R9.4, or R10.4). Since poly-A calling is now available in dorado, we have not updated the polyA caller for RNA004. I hope that helps.

Yes, though it is partly my fault for not having a good handle on the nomenclature Nanopore uses - I thought the R was for RNA. Can results still be trusted for RNA004 kits?

No, you should use dorado for RNA004On Sep 4, 2024, at 1:03 PM, bnpapas @.***> wrote:
Yes, though it is partly my fault for not having a good handle on the nomenclature Nanopore uses - I thought the R was for RNA. Can results still be trusted for RNA004 kits?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>


how to run from exe

how to connect the app from published .exe?
.connectTo('dotnet', 'run', '--project', 'app.exe') does not work

the syntax for connectTo is: .connectTo(executableOrFullPathToExecutable, Parameter1, Parameter2, ...) so if app.exe is your published executable you can do:
connectTo('pathToApp/app.exe')

thank you so much... it works like charm


site propertypromotionad pilot

Docs Build status updates of commit 8316e22:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Advertising/bingads-13/guides/hotel-group.md
:white_check_mark:Succeeded

Advertising/bingads-13/guides/hotel.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Make Firehose subgraphs have their sync status be updated correctly

What?
Firehose subgraphs don't have their sync status correctly updated, they're never synced.
How?
Currently the responsibility of updating the subgraph sync status is in the PollingBlockStream. That should be moved to the instance manager so that Firehose can also benefit from it.
Obs: This issue doesn't fix the block number the subgraph is at for Firehose, that's responsibility for the FirehoseBlockIngestor that's on review at the moment.
Implementation approaches/possibilities
There are a few options to do this as discussed with @leoyvens and @mangas:

Add a new "Done" variant to the BlockStreamEvent enum
Add a new listener to the notification queue (questionable because Firehose doesn't need this)
Make a timer to compare the chain head pointer with the deployment head

I think 3 with a tolerance of chain head - 1 being considered synced will be a simple and reliable implementation.

#3108 ended up reveiling a bug on the promotion of a subgraph from pending to current when it gets synced. @lutter solved the issue with #3238


Bad gateway error

When tried to access the api, I'm getting 502 bad gateway error

Same here, has been like this for like 3 weeks:


Export sheets associated with new AE module

Can test with LOFT-HF Certification, which now has AEs in the new module and associated sheets. There are sheets entered by reporters (highest priority) and sheets entered by team reviewers (lower priority).

So with the next update, the following sheets will be exportable, those created by reporters, reviewer admins, and principal reviewers. The "duplicate" sheets by reviewers are specifically excluded. Let me know if that needs to be changed.

Also, a side-effect of this change is that the sheets index now includes AE sheets, let me know if those should be hidden again there.


[SPARK-6910] [WiP] Reduce number of operations to the cluster.

Here is my workaround for SPARK-6910.
It doesn't prune early. It takes a different approach. Instead our investigation showed that during a query w/ hivecontext on a parquet partitioned table w/ many partitions and many files per partitions (totalling ~40K files), the metastore was poked a lot, but more importantly the datanodes were solicited A LOT. Turns out when doing a query spark would read the footers of ALL parquet files in order to build the schema. This PR read to footers lazy only if needed.
Also:

The metastore already contains the schema, so we use it when available (always when using metastore tables?) to avoid reading the footers.
We don't need to poke that much files in order to figure out the schema, unless we do schema merging, but this in the normal path (w/o schema merging), however, there is a bug or at least a big caveat in parquet-mr. Spark when using readAllFootersInParallelUsingSummaryFiles will end up reading ALL the footers for that tables. This because when there is no summary, parquet-mr revert to readAllFootersInParallel

Improvement:
One our benchmark query that would take 40s on the first call, but then a few seconds (when all the metadata would be cached), now takes 8s consistently. This is not a big query, just that the target table have ~40K files.
I'm not sure if it's mergeable as-is, this PR fit our requirements and we use it in our build, but maybe not those of the whole community.

Can one of the admins verify this patch?

@oliviertoupin thanks for sharing your patch.
But I think SPARK-6910 is about (1) avoiding making the getAllPartitions() call to Hive metastore not about (2) improving file listing.
Can you open a separate jira for your patch? I think both are good optimizations though.

As for getAllPartitions(), my tests show that's it's not so slow, and I think what the users are reporting in this bug (long latency) is actually caused by the reading of the footers. At least, that's what we experienced. With this patch, latency issues are gone.

@oliviertoupin I have a 1.6M partitions table in production and I cannot even query that table because it gets stuck in the Hive metastore call.
This jira is originally from the following conversation on the dev mailing list-
http://goo.gl/2DHPqb

Yeah, it seems this is more closely related to https://issues.apache.org/jira/browse/SPARK-8125
/cc @liancheng

Can one of the admins verify this patch?

Can we close this now that #7396 is merged?

I think so, I will try #7396, and reopen a PR if the issue is still present in our use case.


Disable cache for github action

As Docker's cache subsystem cat't detect update to external sources it's better just disable it.

Thank you


How to create a dataset?

I was wondering whether this repo contains code that helps with the generation of a dataset? Can you or do you plan on open sourcing the code used to create a dataset? Or is it already in the package and I just didn't find it?

Hi， @Kaju-Bubanja @Abdul-Mukit ，I want to creat a new datasets ,and I met the same problem that how to annotating the poses of the objects in the first frame of each video.
Do you know how to get ground-truth annotations for object poses?

I moved on to DOPE and NDDS both from Nvidia research. NDDS  is great for synthetic data generation.

Feel free to have a look at my master thesis, especially the section 3.4 Dataset generation.
You can find my thesis Textureless, Symmetric, Multiple Instances 6D Pose Estimation in Cluttered Scenes Using RGB-D Data here.

@Kaju-Bubanja thanks for your share
@Abdul-Mukit  thanks your advice
I finally choose labelfusion and NDDS
THANKS

@Abdul-Mukit What was your last choice for 6D pose estimation ? PoseCNN or DOPE ? I am planning to train it for a single custon object

@pra-dan DOPE. I got the job done quickly with it. I am sure there are better and faster pose estimators out there now. I have been busy with other stuff. Can you let me know which estimator you end up working with later?


Custom encoding of variants

Hi,
I've the following json format to parse:
I can have either:
{"type":"address", "content":string}

or:
{"type":"coord", "content":{"latitude":float, "longitude":float}}

And what I would like to do is something like that:
type geo_coord =
  { latitude : float
  ; longitude : float
  } [@@deriving yojson]

type coord =
  | Adresse [@name "address"] of string
  | GeoCoordinates [@name "coord"] of geo_coord
  [@@deriving yojson]
  [@@deriving_field_name "type"]

Is it already possible to do something like that or otherwise, is it possible to add such a feature ?

No, this is not possible.
The problem with adding this as a feature is that it is not flexible; there is no single format JSON APIs use for serializing sum types.

Ok, no problems. I just used Yojson.Safe.json

I'm actually open to ideas in regards to this, I just don't know how/if it can be implemented without bloating the interface terribly.

Mmmh at least allowing [@name "name"] on record fields would be very helpful.
The field has name type, but it's an ocaml keyword so I can't create the field.

By the way, generally speaking, with ppx is it possible to enforce that attributes (such as ```[@name "name"]``) have to be used by a ppx module ?

https://github.com/whitequark/ppx_deriving_yojson#options
And to the second question, no. There's been discussion on schemas on mantis but those are vaporware.

Ohh thanks a lot !

I have to deal with similar json schemes, I usually handle this the following way:

I define functions adresse_of_yojson : json -> [> Ok of string | Error of string]andgeo_coord_of_yojson :  json -> [> Ok of geo_coord | Error of string] and have to bind them in a monadic way, which – without cutting through monadic operators – boils down to

match adresse_of_yojson json  with
| `Ok(a) -> `Ok(Adresse(a))
| `Error(_) ->
  begin match geo_coord_of_yojson json with
  | `Ok(c) -> `Ok(GeoCoordinates(c))
  | `Error(_) as error -> error
end

It is not very easy to figure out a solution that will work with all popular APIs because they all have their pecularities. Maybe an acceptable solution would be the ability to specify for a specific field a family of deserialisers that would have to be tried sequentially deserailse the json structure.
I am not sure how this feature should be combined with variants – should they return Adresse(a) or just a ?  In the former case the extension interface would be clean and lean, in the latter case, it would save red typing for the programmer.


Locale

Duplicate: https://github.com/GuVAnj8Gv3RJ/NeosAccountDownloader/blob/main/AccountDownloader/Properties/Resources.resx
Save it as Resources.<code>.resx
Translate
PR it

Even though a good amount of the application is translated, there are still other components that are still untranslated. How are you going to solve this issue?

This is a French translation; why is the entirety of this not translated? How come you did not provide additional elements in the Resources file to translate this?

Locale is community driven. I can machine translate but don't want to rely on that.

I have received info that someone who can do French translations wanted to (and still is happy to) translate the English words and phrases in that dialog above. However, you specified that cannot be done due to the "design of the application." So what is your actual answer? Honestly, I do not think you want to put in the work to provide the availability for other translators to translate other strings unless you put those in the Resource files.

That translator should make a separate issue describing their problems and we'll do our best to resolve them.
I personally use: https://marketplace.visualstudio.com/items?itemName=TomEnglert.ResXManager to manage the locale files.

French appears complete.
If a string is missing from the EN file, its unlocalalizable for any locale. This is very similar to how Neos does things where some areas periodically go through improvements to become localizable. I haven't touched the Library portion of the code here in terms of localization at all. When I do, i'll update things here.

Closing, localization instructions are now in the readme.
Additional localization issues need to occur in their own issue.


Update cats-effect, cats-effect-laws to 3.2.8

Updates

org.typelevel:cats-effect
org.typelevel:cats-effect-laws

from 3.2.5 to 3.2.8.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel" } ]

labels: library-update, semver-patch

Codecov Report

Merging #544 (5c13b14) into main (4e9e60a) will decrease coverage by 0.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #544      +/-   ##
==========================================
- Coverage   90.23%   89.97%   -0.27%
==========================================
  Files          15       15
  Lines         758      758
  Branches       43       43
==========================================
- Hits          684      682       -2
- Misses         74       76       +2

Impacted Files
Coverage Δ

core/src/main/scala/io/iteratee/Enumeratee.scala
99.14% <0.00%> (-0.86%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4e9e60a...5c13b14. Read the comment docs.

Superseded by #552.


Saml2pAuthenticationOptions for dynamic schemes

Hello,
I'm trying to setup dynamically registered external SAML IdPs into IdentityServer4. I've opened a question on stackoverflow.
I suppose I'm missing some service registered in the DI container, but i could not find anything on it.
I can provide any additional info if required.

I've answered on Stack Overflow.

Thanks again, i'll be closing it since it is resolved.


3.3 Data Sharing and Conclusion - Answers showing automatically (bug)

Issue found here - https://eo-college.org/courses/cubes-and-clouds/lessons/3-3-data-sharing-and-conclusion/topics/data-sharing-and-conclusion/
Once you drag an item, the correct answer for it is shown in the exercise. It didn't happen in the previous exercises of this type.

Interesting, we need to investigate why this happens.


geodesics_improvements

includes:

conditional geodesics, with corresponding optimization step
PC plot
factorized steerable pyramids
notebook
bunch of minor merge conflicts

I am closing this PR. The Geodesic object, which this was primarily related to, is deprecated as of #288 , and has been moved to a new repo. The relevant changes from this PR have been included in that new repo.


Horizontal Scrolling on Mobile

On iPhones you can sometimes scroll sidewards (seen on Homepage)

Wait for homepage redesign before seeing whether this is still an issue.


Debian Wheezy repo does not yet have docker.io package

Currently the docker module will attempt to install docker using the docker.io package. Currently this is not present and this will fail:
e.g.
==> apt:
==> apt: E: Package 'docker.io' has no installation candidate

The instructions for installing docker on Wheezy involve some slightly different steps which are a little more crude than running apt-get install.
This is not necessarily a problem of the puppet module, but perhaps we should either:
a) Fail with "wheezy is not yet supported".
b) Attempt to run the docker wheezy installation steps within the manifest.

Debian is works but isn't supported at the moment as noted. This will change in 5.0, which you can track in #311.
I'll close this issue just to avoid duplication, but if you try out the branch in the above PR it should hopefully work out.

Thanks, I'll follow that PR instead..


Pulsar Automated Release

Original issue: https://github.com/AtomLinter/linter-eslint-node/issues/43
We need to automate the publishing of releases to ppm. @confused-Techie made semantic-release-ppm.
We should look into using that.

Since there's some interest around this repo now, I'll try to find some time to test this on my end as well, to get some of the kinks ironed out

🤔 Considering that https://github.com/AtomLinter/linter-eslint-node is now archived. Can we point pulsar-edit ~ linter-eslint-node to this repo?
Or does ppm support scoped packages? (I would make this @pulsar-linter/eslint)

So we could go ahead and change where that package is pointing to. Since the backend does not currently support any scoped packages.
Really the hardest part would be figuring a secure way to do so.

Just looking at semantic-release-ppm src/publish.js:31
I assume we could use release-please, then on gh tag an action can just do something like:
curl -X POST -H "Authorization: $PULSAR_TOKEN "https://api.pulsar-edit.dev/api/packages/$PULSAR_NAME/versions"

Just looking at semantic-release-ppm src/publish.js:31
I assume we could use release-please, then on gh tag an action can just do something like:
curl -X POST -H "Authorization: $PULSAR_TOKEN "https://api.pulsar-edit.dev/api/packages/$PULSAR_NAME/versions"

That very possibly could be the case. I'll be honest, testing more with my plugin gave me some issues, where the recommended code from semantic-releases docs don't work as expected and lead me down some rabbit holes. If we could have something as simple as that, then I'm more than happy to implement it instead.
Since really, under the new backend, very little needs to be done to publish a new release:

Update the version in the package.json
Tag this release via git
Push that tag to your repo
Hit the backend with authentication to a specific endpoint.

So I'll investigate release-please and see if we can make it work, as I'm rather unfamiliar with the project.

release-please makes a PR for the release.
Once that PR is merged it will:

Make a change log for the release.
Update the version in the package.
Make a git tag
Make a git release

From there we should just need to listen for the update and inform Pulsar of the change
This means something like this should work:
on:
  push:
    branches:
      - main
      - release-please
name: release-please
jobs:
  release-please:
    runs-on: ubuntu-latest
    steps:
      - id: release-please
        name: Make the Github Release and Tag
        uses: GoogleCloudPlatform/release-please-action@v3
        with:
          release-type: node

      - if: ${{ steps.release.outputs.release_created }}
        name: Inform Pulsar of the Release
        uses: actions/github-script@v6
        env:
          PULSAR_NAME: linter-eslint-node
          PULSAR_TOKEN: ${{ secrets.PULSAR_TOKEN }}
        with:
          script: |
            const { PULSAR_NAME, PULSAR_TOKEN } = process.env;
            await fetch(
              `https://api.pulsar-edit.dev/api/packages/${PULSAR_NAME}/versions`,
              {
                method: 'post',
                headers: { 'Authorization': PULSAR_TOKEN },
              },
            );


Numpad input in VSTGUI

Numpad input doesn't work for me, VSTGUI does not forward the keycodes.

I've made a fix to that. But also it needs to rebase it to vstgui's latest.
vstgui-numpad.diff.gz

Patch to fix the vstgui latest, currently broken. Also it forces the upgrade to c++ 17.
https://github.com/steinbergmedia/vstgui/pull/229

Fixed following the update #987

I'm reopening to remember to patch the 4.10.2 VSTGUI fork until the corruption issue is handled with 4.11.


Refactor tests

Split the tests out into different test files. This will allow us to run specific tests when altering code without having to run all the other, unnecessary tests, as well as making creating tests more simple. e.g. when altering the decoration functionality, the color tests don't have to run

Refactor completed in ed8c5b2, tests ran successfully in Travis CI


Sections with "-" in the title don't link correctly

I have a README with a section with a "-" in the title:
# Development - IntelliJ

The link generated by markdown-toc looks like:
https://github.com/me/myproject#development-intellij

But the link generated by GitHub when it renders the README needs to look like
https://github.com/me/myproject#development---intellij

I think I know the right way to slugify this now. I can update when merge your pr - probably today. thanks

let me know if this doesn't work for you.


Add SharedTree to bundle-size-tests

This should give the SharedTree team slightly more visibility on impact of various changes.

here's current status:

gzipped 62kb vs. SharedString 44kb. not too bad considering larger scope.

⯅ @fluid-example/bundle-size-tests: +239.12 KBMetric NameBaseline SizeCompare SizeSize Diff
aqueduct.js
437 KB
437 KB
⯅ +6 Bytes

connectionState.js
680 Bytes
680 Bytes
■ No change

containerRuntime.js
229.71 KB
229.71 KB
⯅ +2 Bytes

loader.js
160.81 KB
160.82 KB
⯅ +6 Bytes

map.js
44.07 KB
44.07 KB
⯅ +2 Bytes

matrix.js
137.65 KB
137.65 KB
⯅ +2 Bytes

odspDriver.js
92.35 KB
92.36 KB
⯅ +6 Bytes

odspPrefetchSnapshot.js
43.58 KB
43.59 KB
⯅ +4 Bytes

sharedString.js
158.21 KB
158.22 KB
⯅ +2 Bytes

Total Size
1.38 MB
1.61 MB
⯅ +239.12 KB
Baseline commit: 309fe9801f48d9cdec4571c55242c0d529c3109e

  Generated by :no_entry_sign: dangerJS against f9a4a068d7c0f0dc8a8f808cc2aaebfcbe501587

Shouldn't tree be a line item here?

⯅ @fluid-example/bundle-size-tests: +239.12 KBMetric NameBaseline SizeCompare SizeSize Diff
aqueduct.js
437 KB
437 KB
⯅ +6 Bytes
connectionState.js
680 Bytes
680 Bytes
■ No change

containerRuntime.js
229.71 KB
229.71 KB
⯅ +2 Bytes

loader.js
160.81 KB
160.82 KB
⯅ +6 Bytes

map.js
44.07 KB
44.07 KB
⯅ +2 Bytes

matrix.js
137.65 KB
137.65 KB
⯅ +2 Bytes

odspDriver.js
92.35 KB
92.36 KB
⯅ +6 Bytes

odspPrefetchSnapshot.js
43.58 KB
43.59 KB
⯅ +4 Bytes

sharedString.js
158.21 KB
158.22 KB
⯅ +2 Bytes

Total Size
1.38 MB
1.61 MB
⯅ +239.12 KB

Baseline commit: 309fe98
Generated by 🚫 dangerJS against f9a4a06

Shouldn't tree be a line item here?

I did notice the same thing and glance at the diff logic when I opened this. It makes sense it doesn't appear now--look at handling of commonBundleMetrics. It's a bit odd that the total size shows up larger, but should be transient...
I'll look into it if it doesn't start appearing in the diff for PRs with baseline above this one's CI, though.

⯅ @fluid-example/bundle-size-tests: +239.12 KBMetric NameBaseline SizeCompare SizeSize Diff
aqueduct.js
437 KB
437 KB
⯅ +6 Bytes
connectionState.js
680 Bytes
680 Bytes
■ No change

containerRuntime.js
229.71 KB
229.71 KB
⯅ +2 Bytes

loader.js
160.81 KB
160.82 KB
⯅ +6 Bytes

map.js
44.07 KB
44.07 KB
⯅ +2 Bytes

matrix.js
137.65 KB
137.65 KB
⯅ +2 Bytes

odspDriver.js
92.35 KB
92.36 KB
⯅ +6 Bytes

odspPrefetchSnapshot.js
43.58 KB
43.59 KB
⯅ +4 Bytes

sharedString.js
158.21 KB
158.22 KB
⯅ +2 Bytes

Total Size
1.38 MB
1.61 MB
⯅ +239.12 KB

Baseline commit: 309fe98
Generated by 🚫 dangerJS against f9a4a06

Shouldn't tree be a line item here?

New PRs include it now that it's in the baseline, see here. so we should be in good shape.


no_default_bound: remove Default bound in Deref/DerefMut

I suspect this bound was retained in error, probably a copy-paste from the default version. :)

Thanks for the catch @Jaltaire -- this does look like an unneeded type constraint :)
I added a little fix for the conventional commit message but once the tests run I'll be happy to push this forward.


trade_disputes_controller.rb

module Api::V1
  class P2pTrades::TradeDisputesController < Api::BaseController
    before_action :find_trade

    def index
      render json: @trade.disputes, each_serializer: P2pTradeDisputeSerializer
    end

    def create
      p2p_trade_dispute = P2pTradeDispute.new(create_service.p2p_trade_dispute_params)
      if p2p_trade_dispute.save!
        P2pApi::TradeDispute::CreateJob.perform_later(current_user.id, @trade.id, p2p_trade_dispute.id)
        render json: p2p_trade_dispute, serializer: P2pTradeDisputeSerializer
      else
        errors = if p2p_trade_dispute && ResourceErrors
                   ResourceErrors.format(p2p_trade_dispute)
                 else
                   Rails.env.production? ? 'An error occurred' : e
                 end
        render json: { errors: errors }, status: :bad_request
      end
    rescue StandardError => e
      Rails.logger.info e

      render json: { errors: e }, status: :bad_request
    end

    private

    def create_service
      P2pApi::Requests::TradeDispute::CreateService.new(
        user: current_user,
        trade: @trade
      )
    end

    def find_trade
      @trade = P2pTrade.find_by(id: params[:trade_id])
      @trade if allow_trade_access
    end

    def allow_trade_access
      @trade&.user == current_user || @trade&.offer_user == current_user
    end
  end
end

The P2pTrades::TradeDisputesController manages disputes related to P2P trades. Here’s a detailed overview of its functionality:
P2pTrades::TradeDisputesController Overview

Namespace: Api::V1
Base Class: Api::BaseController
Before Action: find_trade

Actions

index

Purpose: Retrieves and renders all disputes related to a specific trade.
Implementation: Uses the @trade.disputes association and renders it using the P2pTradeDisputeSerializer.

create

Purpose: Creates a new dispute for the trade and triggers an asynchronous job.
Implementation:

Initializes a new P2pTradeDispute with parameters from create_service.
Saves the dispute and triggers P2pApi::TradeDispute::CreateJob if successful.
Handles errors by formatting them or providing a generic error message based on the environment.

Private Methods

create_service

Purpose: Provides an instance of P2pApi::Requests::TradeDispute::CreateService with the current user and trade.
Usage: Used to fetch parameters and other required data for creating a dispute.

find_trade

Purpose: Finds the trade by params[:trade_id] and checks access permissions.
Usage: Sets the @trade instance variable if access is allowed.

allow_trade_access

Purpose: Checks if the current user has access to the trade.
Usage: Ensures that only authorized users (either the trade’s owner or the offer user) can access the trade.

Error Handling

Error Handling in create:

Catches and logs StandardError.
Formats errors using ResourceErrors or provides a generic message in production.
Logs errors and returns a bad request status in case of exceptions.

Example Usage

Index:

Request to /api/v1/p2p_trades/:trade_id/trade_disputes
Returns a list of disputes related to the specified trade.

Create:

Request to /api/v1/p2p_trades/:trade_id/trade_disputes with appropriate parameters.
Creates a dispute, triggers a job, and returns the created dispute or errors.

Summary

Purpose: Manages disputes for P2P trades.
Data Handling: Retrieves disputes and handles creation, including job triggering and error management.
Security: Ensures that only authorized users can access and create disputes for trades.

If you need further details on specific parts or have additional questions, let me know!


Can't use mathematics notation in prefixes

First of all, thank you very much for creating and maintaining such a helpful tool. I would like to report a bug as follows.
Describe the bug
It's not possible to use mathematics notation in prefixes.
Additional information

TeX distribution: TeX Live 2023
Package citation-style-language version: 0.4.5
LaTeX engine: xelatex

To Reproduce
\documentclass{article}

\begin{filecontents}[overwrite, noheader]{\jobname.json}
[
    {
        "id": "ITEM-1",
        "type": "book",
        "author": [
            {
                "family": "D’Arcus",
                "given": "Bruce"
            }
        ],
        "issued": {
            "date-parts": [
                [
                    2005
                ]
            ]
        },
        "publisher": "Routledge",
        "title": "Boundaries of dissent: Protest and state power in the media age"
    }
]
\end{filecontents}

\usepackage[style = apa]{citation-style-language}
\addbibresource{\jobname.json}
\usepackage{hyperref}

\begin{document}
\cite[prefix = {That typically happens in 10$^{-15}$ seconds. See }]{ITEM-1}

\printbibliography
\end{document}

...exmf/scripts/citation-style-language/citeproc-output.lua:1981: bad argument #1 to 'gsub' (string expected, got table)

Thanks for your report and this is fixed in 7f295a5.


Allow properties to be added to Geo types

Is there a PostGIS (or other) limitation that would prevent us from adding properties to the Geo structs?
For example it would be very nice to be able to do the following:
%Geo.Point{coordinates: {1, 1}, properties: %{time: "12:01"}}
|> Geo.JSON.encode()

There is no limitation that would prevent that. It would just be ignored unless used by something like the Geo.JSON module. I think adding properties is fine.
If another request for adding a GeoJSON specific item comes up, I'm leaning towards making something that follows the spec exactly that wraps around the Geo structs.

@jerel

Is there a PostGIS (or other) limitation that would prevent us from adding properties to the Geo structs?

The GeoJSON RFC states that "FeatureCollection and Geometry objects, respectively, MUST NOT contain a "geometry" or "properties" member." This came up in a discussion I had with one of the RFC authors.
It seems that the general theme here is a lack of support for metadata. Your cases is pretty custom, but there are some cases where metadata is vital to retaining information when converting between formats (GeoJSON to/from EWKT/EWKB, for example). Relevant issues:

https://github.com/geomet/geomet/issues/28
https://github.com/bryanjos/geo/issues/76

I would really like to see some official guidance for how to include metadata/extensions in the GeoJSON schema. I'll have to go and do a bit of research.

Hi,
That's exactly what the Feature geojson type is for. It has two members, geometry and properties.
FeatureCollection has only one member (besides type) : features :: [Feature].
Any plans to implement it ?

Looking at the spec and thinking on it a bit, I think adding properties to the types should be fine.

Fixed in 4647ed357a0ffdb3376c64204d9b79521caef67e


'http://' prepended to host template

From the custom-baseURL-paging.json on the test server:

What this creates is:

So, there is a prepended http:// on the front. So, if I have this in the generated code:

You'll end up with http://http//localhost:17607/paging/customurl/partialnextlink, which obviously won't work. In this case, I'm using an accountName of empty string, so I'd like to simply have http://localhost:17607/paging/customurl/partialnextlink be the URL. As it stands, I'll parse the http:// out of the host value, so that it is provided properly. But, that seems confusing since host for the other tests means the full URL.

Closing. I realized that the problem was not looking at host versus $host. So, I need to do that in my generator to detect this properly.


Add support for Markdown output

This library currently supports JSON, & YAML output.
We need to support Markdown output as well.
Check https://github.com/asyncapi/diff/issues/75
Suggested solution:

Headings are names of changes, and then under each heading I guess you get a table of changes? could be nicely handled with IonicaBizau/json2md, so you just first slightly modify the default JSON output that you get as input for Markdown, and then run it through json2md to get markdown. No complicated Markdown templating needed, you only need to add some props to JSON

The format of md output
Subject to change, but my initial thoughts are as follows:

If user request a breaking, non-breaking or unclassified changes:

Breaking Changes

Action
Path
Before
After
isArrayIndex

remove
/servers/google
{}

false

add
/server/google/protocol

http
false

edit
/server/google/something/1
some value
new value
true

If user requests all changes

All changes

Type
Action
Path
Before
After
isArrayIndex

breaking
remove
/servers/google
{}

false

non-breaking
add
/server/google/protocol

http
false

unclassified
edit
/server/google/something/1
some value
new value
true

If you have other suggestions, drop them below :)

I'd love to work on this! Following up on our slack discussion, the structure of a Markdown diff will be similar to this, as it's hard to display large objects in the after and before columns of the table in a lucid format. The type of the data inside the dropdowns can be configured to be either JSON or YAML using the markdownSubtype option.
The WIP can be found at https://github.com/shivensinha4/diff, and I'll create a PR once the subtype option is added :)

@shivensinha4 Looking forward to the PR :slightly_smiling_face:

@shivensinha4 how is it going, do you need some help?

Sorry for the delay! Got caught up in some stuff. I'm done with most of the code, will look for and make any documentation edits soon : )


Returning Binary Data

How do I return binary data?
I've tried this and it does not work. What am I doing wrong?

@app.get('/binary')
def mybindata():
    data = b'123'
    return bottle.Response(body=data, status=200)

I was using requests wrong. Sorry.


The example doesn

[Enter feedback here]

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

ID: 5cceddcd-1431-ab61-df75-7ccf74a84fba
Version Independent ID: d5833a30-c43e-8048-63b3-d94d26410a3b
Content: Quickstart: Sign in users and call Microsoft Graph in a Universal Windows Platform app - Microsoft identity platform
Content Source: articles/active-directory/develop/quickstart-v2-uwp.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

@NovotnyMarek
Thank you for time, can you please provide some more details on issue or request?


Update README.md

Reckoned an example was needed there.

@Projjol the new CSP prevents injecting from the URL hence the comment to say 'copy paste gmail.js source code'. That codeblock in the readme is intentional


[🐛] Fixed fullscreen button not appearing

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Updated PlayFab UnrealCppSdk to 4.12

Updated use of TSharedPtr and USTRUCT assignment operator in several places of PlayFab UnrealCppSdk. Syntax of TSharedPtr and USTRUCT assignment has become stricter as of UE4.12.
Plugin code has been confirmed to compile with UE 4.11 and 4.12. Has not tried compiling with 4.9, but will most likely work.

I've attempted to port these lines into our SdkGenerator.
However, it seems that your changes target specific lines, where the generator wants to make these changes on thousands of lines, which probably isn't right.
What is most likely the case is that these are a datatype which requires a special case, and our generator hasn't identified that case.

Here is the destination branch for the 4.12 changes, built by the generator.
https://github.com/PlayFab/UnrealCppSdk/tree/Testing4-12
It's quite a bit more changes than you suggest, so we need to figure out if my attempt to port your changes in has improved a case which was not seen.
Optionally, the other possibility is that the generator doesn't have enough information to decide why those specific classes/variables are a different case.

I wasn't aware that the code was produced by a generator, which does make sense to have. The 23 compile errors that occur when trying to use the PF SDK with UE4.12 actually boil down to two syntax errors that happen in many places:
(1) The generated code tried to assign the value of some struct directly to a TSharedPtr<> of some struct. UE pre 4.12 had some syntactic sugar that allowed this direct operation, but as of 4.12 you have to deference the pointer first.
So now either the generator has to generate code in which TSharedPtrs are deferenced before being assigned a new value. Or consider to stop using TSharedPtrs altogether, since they aren't being used consistently. Being able to pass around a smart pointer to the same data serves to save some memory and to allow late modifications to the data that are visible to all with the pointer. In this case I don't think TSharedPtrs are providing any benefit? I could be wrong.
(2) The generated code tried to use assignment between two structs of different types. So lets call them FSomeRequestParams and FClientSomeRequestParams, because in all the cases its about a client version of request params trying to be assigned directly to the generic version. FClientSomeRequestParams would have just one field
FSomeRequestParams Data;
Assignment between the two would work before, because their size and layout in memory are the same. But as of UE4.12 they don't allow such assignment anymore, The left hand and right hand side have to be the same struct type now, so the generator should refer to the client param struct's Data field directly.

I hope the above comments helps you to update the generator code. That's as far as my contribution goes for now, because my goal was just to get it working with my UE4.12 project. :)

I was able to use your suggestions to build a working 4.12 project.  It will be published today.  Thanks!
(Closing this pull request, because as we discussed, the changes you provided went into the generator, which will in turn, mirror to here later today)

Glad I could help!


[CI] Publish a slack notification on each release

Information
The idea is to publish a message on slack when a release is publish on NPM.
We can use the following semantic release plugin for that:

https://github.com/juliuscc/semantic-release-slack-bot
https://github.com/intuit/semantic-release-slack

released


License Editing implementation

Coverage decreased (-0.06%) to 63.503% when pulling 04f2929d781b4c3c1d29ec9263bb9c9fe52ac11e on license-editing into 1d23029ce64309ef9b70bceca7466360f4b3d580 on master.

Please update the automated server tests here for new prepaids API as well:
https://github.com/codecombat/codecombat/blob/master/spec/server/functional/prepaid.spec.coffee


feat: add white value to service data

White value is available as a channel for brightness. This adds the support for lights that use it, using the same calculated value as brightness.
Fixes #26

Great work!

I'd like to leave it for a day so I can observe the full cycle, but quick on -> manual change -> off -> on tests are working.

After using it for a day, LGTM

Great!

I synced this back to the PR https://github.com/basnijholt/home-assistant/commit/ca7a01dd4d994891d859dd3a2e7c6e78976027ff.


[SPARK-20647][core] Port StorageTab to the new UI backend.

This required adding information about StreamBlockId to the store,
which is not available yet via the API. So an internal type was added
until there's a need to expose that information in the API.
The UI only lists RDDs that have cached partitions, and that information
wasn't being correctly captured in the listener, so that's also fixed,
along with some minor (internal) API adjustments so that the UI can
get the correct data.
Because of the way partitions are cached, some optimizations w.r.t. how
often the data is flushed to the store could not be applied to this code;
because of that, some different ways to make the code more performant
were added to the data structures tracking RDD blocks, with the goal of
avoiding expensive copies when lots of blocks are being updated.
Tested with existing and updated unit tests.

For context:

Project link: https://issues.apache.org/jira/browse/SPARK-18085
Upcoming PRs that build on this code: https://github.com/vanzin/spark/pulls

Note that unit tests will fail; they depend on changes being added in #19678. I'm posting this to speed up the review process.

Test build #83509 has started for PR 19679 at commit 7147bd2.

Test build #83509 has finished for PR 19679 at commit 7147bd2.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

I figured out what is going on with StreamBlocks now, both before and after this change, so I'm OK with it

Test build #83560 has started for PR 19679 at commit 7744c6c.

Test build #83560 has finished for PR 19679 at commit 7744c6c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #83604 has started for PR 19679 at commit e257a1a.

Test build #83604 has finished for PR 19679 at commit e257a1a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test PASSed.

Test build #83619 has started for PR 19679 at commit fd59a24.

Test build #83619 has finished for PR 19679 at commit fd59a24.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

merged to master


[WIP] Enhance TraceTimeline component

Summary

Use TypeScript for TraceTimeline components
Simplify tree-generation algorithm.
Use normal HTML tags instead of using SVG for trace tree graph.
Resolve some known issues.

Screenshots
Before

After

Description
Simplify tree-generation algorithm
The algorithm for generating trees has been complex.
So I have greatly simplified it and made it more readable and testable.
Use normal HTML tags instead of using SVG for trace tree graph
Trace trees were drawn using SVG. This was for rendering efficiency.
But compared to the regular DOM, styling with SVG is a bit more difficult, and some styling problem like #2948 occurs.
So, I used regular HTML tags instead of SVG in this PR.
This was an issue that had already been registered in #2997 .

fix #2948
fix #2997
fix #3118

However, the rendering of regular HTML tags is slower than SVG, and if you have to draw a lot of spans, the browser will get stuck. So I used a technique called "virtualization" to avoid the browser getting stuck for drawing.
Resolve some known issues.
fix #2509
Now annotations are marked (white line).

and... error spans will be marked with an exclamation mark.

fix #2906
Probably the shared trace can be drawn normally.

Combining #3349 and #3348 with this PR change, trace page will look like this:


Use type hints?

see https://www.python.org/dev/peps/pep-0484/
@stijnvanhoey @peterdesmet any complaints if we use type hints?  i think it means python 3.5 or greater

I don't think this should be an issue. Fine for me

thanks for the feedback @stijnvanhoey

not going to implement, can reopen if there's interest