iOS and Mac clients out of sync through Feedly

I've been noticing the past week or so that my iOS and Mac clients aren't staying in sync. iOS client version 6.0.2 (Build 610). Mac client 6.1 (6106)
Screenshots taken seconds apart after refreshing feeds:

I just marked all as read on the Mac client and then refreshed. Then I refreshed the iOS client. This is the result:

The iOS client increased the unread count!

That doesn't tell me anything. Could you do the starred test that I described above?

At the very least, sign on to the Feedly web interface and let me know if your iPhone or Mac matches the unread count you see there.

Okay, so here's what I did, and the results:

Signed into Feedly.com and marked everything as read.
Refreshed both iOS and Mac Clients
Both clients were reset to all read.
Waited for new updates to feeds (aka went to lunch)
Afterwards, checked on unread status on Feedly
Both iOS and Mac clients correctly matched Feedly unread count.
Picked one folder (F1) and marked as read on iOS, updated.
Refreshed Feedly and Mac client. F1 not update as being read on both Feedly and Mac client
Picked a different folder on Mac client (F2) and marked as read, then updated
Refreshed Feedly and iOS client. F2 did not update on Feedly and iOS client
On Feedly, marked F1 and F2 as read, updated
Refreshed iOS and Mac clients. Both clients now show F1 and F2 as being read.

So, it seems that both clients are reading from Feedly correctly, but the "read" statuses aren't making it back to Feedly.

On your Mac, I need you to get a file for me so that I can diagnose it.
Inside of ~/Library/Containers/com.ranchero.NetNewsWire-Evergreen/Data/Library/Application Support/NetNewsWire/Accounts you will find a folder that starts with 16_. Inside that folder is a file called Sync.sqlite3. I need you to zip that file and attach it with your next comment.

Sync.sqlite3.zip

Thank you so much for the database file. I was able to diagnose the problem and it wasn't what I thought it would be! I'll follow up with details in my next comment.
For you, I would remove the Feedly account in NetNewsWire from your iPhone and your Mac and you should be back in business. We'll release a NetNewsWire update in the future that prevents this issue in the future.

The root cause of this problem was the number of read statuses that were trying to be synced. Feedly doesn't have the limit documented, but we hit it. There were over 3400 entries trying to be marked as read. The NetNewsWire code for Feedly syncing doesn't try to chunk these into multiple requests, so we send the full shot and Feedly sends an error message back.
We should chunk these up into requests of 300 status updates so that we don't hit the Feedly limit.

Thanks for helping resolve this issue. It's much appreciated.

Thanks for the update, @vincode-io.
I had the same issue, and the reasoning behind this of course makes sense to me ( I was on vacation for a while and I don't read my feeds consistently so I forced "inbox zero" at some point and I'm pretty sure it was 1000~ articles).
If I'm reading this correctly this is planned to be fixed by version 6.2, right?

We are actually going to fix this in 6.1 now.


remote chrome support

I am new to scrapy-playwright.
I want to scrape using chrome on a remote machine as explained in https://blog.chromium.org/2011/05/remote-debugging-with-chrome-developer.html and using playwright.chromium.connect_over_cdp.
Here is a working snippet:
from playwright.sync_api import sync_playwright

with sync_playwright() as playwright:
    browser = playwright.chromium.connect_over_cdp("http://localhost:9222")

    page = browser.new_page()
    page.goto("https://example.com")
    print(page.title())
    browser.close()

What is the best way to use scrapy-playwright using a remote machine?
Thanks!

This is not possible at the moment. Currently the browser is closed when the crawl finishes, that might not be desired if the browser is remote.
Sounds like a good feature to have, thanks for the request.

期待这个功能

Support for connecting to a remote browser via the Chrome DevTools Protocol was added in version 0.0.32. See the docs for the PLAYWRIGHT_CDP_URL and PLAYWRIGHT_CDP_KWARGS settings.


BITMAKER-2338 estela-api: Create OwnerProject Role on user permissions

Description
estela-api: Create OwnerProject Role on user permissions
Issue
https://tasks.bitmaker.dev/issues/2338?tab=time_entries
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] My code follows the style guidelines of this project.
[x] I have made corresponding changes to the documentation.
[x] New and existing tests pass locally with my changes.
[x] If this change is a core feature, I have added thorough tests.
[x] If this change affects or depends on the behavior of other estela repositories, I have created pull requests with the relevant changes in the affected repositories. Please, refer to our official documentation.
[x] I understand that my pull request may be closed if it becomes obvious or I did not perform all of the steps above.

@JoseArmandoChavezQuijahuaman This PR is ready for review


GH1616 : Better error message for interdependent tasks

Error message was changed.
Fixes https://github.com/cake-build/cake/issues/1616
Added information was put on another line to maintain backwards compatibility with old message.

 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.:x: norpache sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

The case where a task depends on itself could probably also use a better message (Error "Reflexive edges in graph are not allowed."). Let me know if you want it changed in another PR.

@norpache Thanks for the PR!
I think you're spot on in the sense that we're trying to get a user-friendly message from a low-level graph API.
Perhaps the way to go about this, would be to throw custom graph exceptions as needed (e.g. CakeGraphCircularReferenceException) from the low-level code including the details that you'd need to craft a user-friendly message (e.g. edge names, etc.) that can bubble up the stack and be caught closer to the UI, and then a CakeException
@cake-build/cake-team What do you think?


Too many packets in payload error

Type of Issue
Select the type of issue:

[x] Bug report (to report a bug)
[ ] Feature request (to request an additional feature)
[ ] Tracker (I am just using this as a tracker)
[ ] Refactor request
[ ] Documentation Ask

Description
An error where there are "too many packets in payload" occurs occasionally when locobot_agent.py is run. It seems to resolve itself after waiting a while (~1 min) or after restarting.
Steps to reproduce
The error occurs somewhat randomly whenever locobot_agent.py is run. The stack trace is as follows:

INFO:root:LOG LEVEL: 20
2021-06-03 06:34:28,409 [locobot_agent.py:214 - () INFO]: LOG LEVEL: 20
Rootdir /raid/hollisma/droidlet
post request handler error
Traceback (most recent call last):
File "/private/home/hollisma/.conda/envs/droidlet_env/lib/python3.7/site-packages/engineio/server.py", line 412, in handle_request
socket.handle_post_request(environ)
File "/private/home/hollisma/.conda/envs/droidlet_env/lib/python3.7/site-packages/engineio/socket.py", line 124, in handle_post_request
p = payload.Payload(encoded_payload=body)
File "/private/home/hollisma/.conda/envs/droidlet_env/lib/python3.7/site-packages/engineio/payload.py", line 13, in init
self.decode(encoded_payload)
File "/private/home/hollisma/.conda/envs/droidlet_env/lib/python3.7/site-packages/engineio/payload.py", line 44, in decode
raise ValueError('Too many packets in payload')
ValueError: Too many packets in payload

can confirm, I'm starting to see this as well.

https://github.com/miguelgrinberg/python-engineio/issues/142


how do use proxy

how do use proxy in RpcClient？

Override RpcClient
Call like this： RpcClientProxy client = new RpcClientProxy(true, Cluster.MAINNET);

refer to：https://github.com/liuruibin1/solanaj/blob/main/src/main/java/org/p2p/solanaj/rpc/RpcClientProxy.java

SOCKS proxy support added in 1.18-staging


Zooming Issue in V2

Hi, there,
I found a zooming issue in v2. when I keep zooming in, eventually the lines are disappeared, although there is data point in it (I can find it when I use mouse cross the area).
FYI, I use y2 axis, and two different line types in the same chart (one series is line, another is interpolation: {mode: 'step-after'} ) .
Thanks!
Sam

anyone could take a look?
thanks,
Sam

Could you provide a Codepen please?


Does not support for "IF NOT EXISTS" syntax for DEFINE statements

Version 1.1.3 of surql-gen does not support the "IF NOT EXISTS" syntax for DEFINE statements,  introduced in SurrealDB version 1.3.0. It errors out as follows:
[Error: Parse error: Failed to parse query at line 5 column 16 expected ON
  |
5 | DEFINE USER IF NOT EXISTS myusername ON DATABASE PASSWORD 'secret' ROLES EDITOR;
  |
] {
  code: 'GenericFailure'
}

The issue should now be resolved.
See Version 2 Release Note.


yeelink.light.color5

Hi
I buy yeelink.light.color5 from internet
and use miio
It's can set power and Brightness
but Cant use "set_ct_abx"
can you help me ?
please dont mind my poor english  thx
yeelink.light.color5 is this
http://buy.mi.com/tw/item/3201200025

I have the same. How did you manage to turn it on?
You can only get the json information from device.


Invalid or unexpected token for images inside node-modules/

Bug report
Describe the bug
I use an npm module (react-images-upload) that uses an SVG icon internally. The package works fine with the svg icon, unless I reload the specific page where that pachage is used, or write its url directly.
The error:
Unexpected token '<'
I thought it might be a problem with the svg file itself, so I tried to replace the .svg file in the package with a .png file, but still gives me an error when I reload the page.
To Reproduce
Repo: https://github.com/moaaz-bhnas/hdomi

Clone the repo: git clone https://github.com/moaaz-bhnas/hdomi.git
Install required packages: npm install
run: npm run dev
Write the url direclty for the page that uses the package: http://localhost:3000/seller/add-product

Expected behavior
The package displays with the icon when I reload the page or enter its url directly in search tab.
Screenshots

System information

OS: Windows
Browser: Chrome
Version of Next.js: latest

The library loads the SVG icon with a simple require. For that to work you need to tell webpack to load SVG files.
See:
https://nextjs.org/docs/api-reference/next.config.js/custom-webpack-config
https://webpack.js.org/loaders/svg-inline-loader/

@lboecker
I have this as my webpack config:

Try to explicitly include the directory like so:
const path = require('path');

module.exports = {
  webpack(config) {
    // url loader
    config.module.rules.push({
      test: /\.(png|jpg|gif|svg)$/,
      include: [path.resolve(__dirname, 'node_modules/react-images-upload/src/component')]
      use: {
        loader: 'url-loader'
      }
    });

    return config;
  },
}

@lboecker
I tried this path as the svg icon lives directly inside the package (node_modules/react-images-upload/icon.svg):
const Dotenv = require("dotenv-webpack");
const path = require('path');

module.exports = {
  distDir: '../../dist/functions/next',
  webpack(config) {
    // url loader
    config.module.rules.push({
      test: /\.(png|jpg|gif|svg)$/,
      include: [path.resolve(__dirname, 'node_modules/react-images-upload')],
      use: {
        loader: 'url-loader'
      }
    });

    // Add the new plugin to the existing webpack plugins
    config.plugins.push(new Dotenv({ silent: true }));

    return config;
  },
}

But it somehow the loader doesn't work at all, even for images in my src folder. I'm unfortunately not good with webpack and configs ..

Sorry, I got the path wrong. Since your Next.js project lives in a subdirectory, the path should look like this: path.resolve(__dirname, '../../node_modules/react-images-upload').

But somehow the loader doesn't work at all, even for images in my src folder.

Just add your src directory to include and it'll work again.

@lboecker I'm sorry for the late response ..
First, thank u so much for taking the time to try solve my issue.

By the way, you're repo doesn't work without Firebase or with any Node.js version other than 8

My bad. I should've stated that, but this is the first time i write an issue ..

the error I got was that global CSS isn't supported in files inside node_modules.

Actually, that was another error that i solved by importing the css in _app.js file, and I dont know away to reproduce only the image issue without that CSS error.


Mark the rule '_if_null_expression' explicitly as inline

This marks the rule _if_null_expression explicitly as inline so as to not rely on tree-sitter auto-inlining it. We run into a problem when we unhide (automatically) the rule by removing the leading underscore. This causes the rule to no longer be inlined, resulting in the following conflict:
dart: Importing initial 'grammar.json'.
dart: Simplifying 'grammar.json' for ocaml-tree-sitter.
dart: Recovering informational JS grammars from JSON.
dart: Generating definitive 'parser.c'.
Unresolved conflict for symbol sequence:

  real_expression  if_null_expression__repeat1  •  '??'  …

Possible interpretations:

  1:  real_expression  (if_null_expression_  if_null_expression__repeat1)  •  '??'  …
  2:  real_expression  (if_null_expression__repeat1  if_null_expression__repeat1  •  if_null_expression__repeat1)

Possible resolutions:

  1:  Specify a left or right associativity in `if_null_expression_`
  2:  Add a conflict for these rules: `if_null_expression_`

The issue is tracked here: https://github.com/tree-sitter/tree-sitter/issues/1683

ping @UserNobody14
Could we merge this? This is useful to add support for Dart in semgrep.


SRTM download link doesn't respond

Hi,
SRTM.py inside StartMaja/prepare_mnt/mnt folder uses https://srtm.csi.cgiar.org/wp-content/uploads/files/srtm_5x5/TIFF/%s.zip to download SRTM files.
But SRTM download link has been not responsive.
Example: https://srtm.csi.cgiar.org/wp-content/uploads/files/srtm_5x5/TIFF/srtm_17_05.zip
There were some similar errors a few months back, where a new change was merged to Maja changing HTTP to HTTPS which fixed this.
Is this some similar error too?

Hi, I just tried  https://srtm.csi.cgiar.org/wp-content/uploads/files/srtm_5x5/TIFF/srtm_17_05.zip from my navigator and it worked. Was it a temporary issue on the server ?
Olivier

There seem to be some downtimes associated to the SRTM data. Depending on where you want to process Maja, you might want to switch over to EuDEM v1.1, also supported by Maja since v4.2 which covers the entirety of Europe.
Unfortuntely we cannot host a backup of SRTM data, which would be too costly for us. So if you are relying on it, the only solution is to wait until the server is up again.
Kind regards,
Peter

Thank you @olivierhagolle and @petket-5,
I tried loading the srtm_17_05.zip and it still fails.
I'll try soon to run this on server in a different location and see if it is my network.
@petket-5
Is it feasible if someone wants to host srtm data themselves for internal use? If yes can you please point me to any pointers?
Thanks a lot for the reply and appreciate all the efforts you guys put into :)

I tried this on the server and was able to wget it manually, and also some of the MAJA processes ran successfully. So I think it is something with my network.
Sorry for the false alarm.

Thanks for letting us know !


Search the web

Provide a description of the new feature / enhancement
I would want to be able to search the web (preferably Google), but I wouldn't want to have to say Google (something) or search (something).
FYI, I'm talking about PowerToys Run.

thanks for the request, tracked in either #3245 or #10619  depending on implementation


⚠ Bofrost.at has degraded performance

In 9b80b61, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1147 ms

Resolved: Bofrost.at performance has improved in b09bb3c after 6 minutes.


Release Google.Cloud.ApiKeys.V2 version 1.0.0-beta01

Changes in this release:
Initial release.

Created release for Google.Cloud.ApiKeys.V2-1.0.0-beta01


allow setting of libguestfs paths from env variables

What this PR does / why we need it:
allow setting of libguestfs paths from env variables
this will help to set different paths in ds
Release note:
NONE

/lgtm

/cherrypick release-v0.12

/retest

/retest

/retest

/retest

/retest

/retest

/lgtm

/retest

/cherrypick release-v0.9.0


Added files and updated projectData.json

Title and Issue number

Title : Black Jack
Issue No. :#268
Code Stack : HTML ,CSS, Javascript
Close #268

Video (mandatory)
https://github.com/apu52/METAVERSE/assets/118686637/9fe5379c-0631-40da-9ed2-03e3959b611e
Checklist:

[X ] I have mentioned the issue number in my Pull Request.
[X ] I have commented my code, particularly in hard-to-understand areas
[X ] I have created a helpful and easy to understand README.md
[X ] I have updated the projectData.json file for my contribution
[ X] I have added the img of my project in the assets/img section

Additional context (Mandatory )
Are you contributing under any Open-source programme?
Codepeak 23

@drockparashar  could you fix the conflict :)

Is is due to a previously added project?

@5h0ov Merge conflict resolved...u can merge it now....

Is is due to a previously added project?

ye but it ok now


[sw/exts] Optimize stack and .bss zeroing for sim targets.

As described in #9163, the current flash_crt.S zeros the .bss and stack memory segments before transferring control flow to the C program entry point (i.e., main()). For DV and verilator simulations this wastes simulation cycles since simulated memory is already initialized to zero in the testbench.
This commit optimizes the flash_crt by checking the device type the binary is compiled for, and skips the zeroing of memory segments if the device type is configured to DV or Verilator simulation.
Additionally, this commit adds static asserts to ensure that DV and Verilator simulation device enums remain assigned a specific value so the new flash_crt.S code is valid.
This fixes #9163.
Signed-off-by: Timothy Trippel<EMAIL_ADDRESS>

fyi @matutem

I have updated this PR to simply move the FreeRTOS heap to its own NOLOAD section as suggested by @mundaym. Will keep as draft for now until next SW WG, and we can merge after discussion if discussion is positive. Thanks @mundaym for the great idea!

After consensus in SW WG, this PR is ready for review.

Looks like a good idea, but could you expand the comment on lines 52-54 to explain how we configure things to zero memory in these cases?
My guess is that Verilator would bring the memory up as uninitialised, but uses 2-value logic and we run it in the "X's are zero" mode (the default). I'm slightly surprised about this being true for other simulators, though: are we doing some clever configuration to zero the memories at the start of time?

We backdoor load the RAM to 0s at boot time.


More features on compiler

May I know if it is intended that new constructs are added but not actually extracted?

Yes, for example the import declarations are added to the AST structure without being extracted. I intend to create a separate PR for the extraction part. The idea is that we could settle on the AST structure and resolve existing build errors in the repo first. This could reduce the dependencies between components.


style: call rubocop directly

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

This PR attempts to solve part of #10505
The idea is that we can initiate rubocop by calling RuboCop::CLI.new.run instead of using a system call to rubocop. Once this is done, we can selectively ignore the warnings to remove the message.
It appears that this first draft works well. I've noticed two issues, though:

Calling rubocop directly raises style warnings that were not raised previously. I think that this is likely due to a misconfiguration, but it is surprising.
Calling rubocop directly doesn't display rubocop errors. This isn't an issue for most brew style users, but for those who create and modify style rules, this lack of indication of a rubocop problem would be problematic.

I'm going to keep poking at this to see if I can figure it out. If anyone has any ideas, please share.

Review period will end on 2021-02-11 at 00:09:01 UTC.

Does running it in a more isolated environment (i.e. a new Ruby process running a custom .rb file that does the RuboCop::CLI.new.run) make any of the weirdness go away?
The one new style warning it raises in the CI does seem correct - not entirely sure why it wasn't caught before though.

Calling rubocop directly raises style warnings that were not raised previously. I think that this is likely due to a misconfiguration, but it is surprising.

Yes, no idea what's happening here.

2. Calling rubocop directly doesn't display rubocop errors. This isn't an issue for most brew style users, but for those who create and modify style rules, this lack of indication of a rubocop problem would be problematic.

Can you elaborate on with an example here?

Does running it in a more isolated environment (i.e. a new Ruby process running a custom .rb file that does the RuboCop::CLI.new.run) make any of the weirdness go away?

I'd be surprised if it did but copying the code in the bin stub (Library/Homebrew/vendor/bundle/ruby/2.6.0/bin/rubocop) with the warning gem added seems likely to.

Calling rubocop directly raises style warnings that were not raised previously. I think that this is likely due to a misconfiguration, but it is surprising.

The one new style warning it raises in the CI does seem correct - not entirely sure why it wasn't caught before though.

I think the change is that style warnings (as opposed to style errors) weren't shown by brew style before. I don't know why that was the case. Unless there's a reason we want to ignore style errors, I'd say that the "new way" is better (i.e. including the style warnings)

Does running it in a more isolated environment (i.e. a new Ruby process running a custom .rb file that does the RuboCop::CLI.new.run) make any of the weirdness go away?

I'll give it a shot

I'd be surprised if it did but copying the code in the bin stub (Library/Homebrew/vendor/bundle/ruby/2.6.0/bin/rubocop) with the warning gem added seems likely to.

Thanks for the tip. I'll see what I can do. Totally venturing into new waters here so I may need some guidance.

I think the change is that style warnings (as opposed to style errors) weren't shown by brew style before. I don't know why that was the case. Unless there's a reason we want to ignore style errors, I'd say that the "new way" is better (i.e. including the style warnings)

Agreed 👍🏻

Totally venturing into new waters here so I may need some guidance.

Happy to help however I can!

Just realized that I forgot to give an example of the issues not being reported.
What I did was add a line with something like a = 1 / 0 to one of the cop files (I think I used homepage.rb to force an exception. Then, running rubocop directly or with the old brew style shows an error but the new brew style doesn't and just says 0 files checked.

Then, running rubocop directly or with the old brew style shows an error but the new brew style doesn't and just says 0 files checked.

Hmm, that seems non-ideal unless we can figure out a way to optionally show it again.

Agreed. This is its current state is not something that should be merged. I'd rather live with a warning than have errors silenced.
I looked at the code a little more. It looks like the error message are displayed using the warn method. I think this also puts them to stderr. So, it seems that the warn method isn't working properly when we're calling it (this is reproducible in brew irb as well). Do we override/modify warn anywhere? I wonder if rubocop does? It's also possible that stderr output is being silenced somehow. I'll be home in ~30 minutes and I'll spend some more time working on this.

So, it seems that the warn method isn't working properly when we're calling it (this is reproducible in brew irb as well).

It's because we run brew with -W0.
The warn method should work with HOMEBREW_DEVELOPER=1 HOMEBREW_RUBY_WARNINGS=-W1 brew irb

Bingo, thanks!
I modified this to use with_warnings 1 to always show warnings generated with #warn just for the rubocop run. This seems to mostly solve that issue, but it's not perfect. I'm still getting this message when running an intentionally broken brew style:
$ brew style
An error occurred while FormulaAudit/Homepage cop was inspecting /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/htop.rb:1:0.
To see the complete backtrace run rubocop -d.
An error occurred while FormulaAudit/Homepage cop was inspecting /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/htop.rb:1:0.
To see the complete backtrace run rubocop -d.

1 file inspected, no offenses detected

1 error occurred:
An error occurred while FormulaAudit/Homepage cop was inspecting /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/htop.rb:1:0.
undefined method `metadata' for nil:NilClass
/usr/local/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/rubocop-1.9.1/lib/rubocop/cli/command/execute_runner.rb:79:in `display_error_summary'
/usr/local/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/rubocop-1.9.1/lib/rubocop/cli/command/execute_runner.rb:58:in `display_summary'
/usr/local/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/rubocop-1.9.1/lib/rubocop/cli/command/execute_runner.rb:27:in `block in execute_runner'
/usr/local/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/rubocop-1.9.1/lib/rubocop/cli/command/execute_runner.rb:52:in `with_redirect'
/usr/local/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/rubocop-1.9.1/lib/rubocop/cli/command/execute_runner.rb:25:in `execute_runner'
/usr/local/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/rubocop-1.9.1/lib/rubocop/cli/command/execute_runner.rb:17:in `run'
/usr/local/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/rubocop-1.9.1/lib/rubocop/cli/command.rb:11:in `run'
/usr/local/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/rubocop-1.9.1/lib/rubocop/cli/environment.rb:18:in `run'
/usr/local/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/rubocop-1.9.1/lib/rubocop/cli.rb:65:in `run_command'
/usr/local/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/rubocop-1.9.1/lib/rubocop/cli.rb:72:in `execute_runners'
/usr/local/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/rubocop-1.9.1/lib/rubocop/cli.rb:41:in `run'
/usr/local/Homebrew/Library/Homebrew/style.rb:147:in `block in run_rubocop'
/usr/local/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/activesupport-6.1.1/lib/active_support/core_ext/kernel/reporting.rb:28:in `with_warnings'
/usr/local/Homebrew/Library/Homebrew/style.rb:146:in `run_rubocop'
/usr/local/Homebrew/Library/Homebrew/style.rb:54:in `check_style_impl'
/usr/local/Homebrew/Library/Homebrew/style.rb:16:in `check_style_and_print'
/usr/local/Homebrew/Library/Homebrew/dev-cmd/style.rb:75:in `style'
/usr/local/Homebrew/Library/Homebrew/brew.rb:122:in `<main>'

It seems unable to display the normal rubocop error message:
Errors are usually caused by RuboCop bugs.
Please, report your problems to RuboCop's issue tracker.
https://github.com/rubocop-hq/rubocop/issues

Mention the following information in the issue report:
1.9.1 (using Parser <IP_ADDRESS>, rubocop-ast 1.4.1, running on ruby 2.6.3 universal.x86_64-darwin20)

The traceback points here. Looks like a gem issue. I need to research more here I'm not that familiar with how gems interact with ruby code, etc.

Bundler's standalone setup.rb (the require "vendor/bundle/bundler/setup") skips spec loading.

Basically Bundler's standalone mode is a very basic version of how gem loading works so it doesn't support the full feature set.
But it means that it works without having Bundler/Rubygems installed. Which matters less nowadays because Bundler ships with Ruby now. In fact, require still does gem searching under standalone mode (fixed in Bundler 2.2.7).

Review period ended.

Does running it in a more isolated environment (i.e. a new Ruby process running a custom .rb file that does the RuboCop::CLI.new.run) make any of the weirdness go away?

I went back to give this a try and it worked! I've updated the PR to implement this. Interestingly, though, the warning that reappeared is now gone again. Not sure why.
I didn't know where to put the rubocop.rb file that actually runs rubocop. For now, I put it in Library/Homebrew/rubocops/bin/rubocop.rb. I could use some advice on that, though. Maybe something like Library/Homebrew/bin/rubocop.rb?
If this looks good, I'll try to do a similar thing with brew tests. I haven't looked there at all to see what the situation is, though, so that may go in a separate PR.

Interestingly, though, the warning that reappeared is now gone again. Not sure why.

Probably because Rubocop doesn't know what a FormulaUnavailableError is by just parsing the file under an isolated environment. When called within the brew environment, it will know since it will have been defined beforehand.

I didn't know where to put the rubocop.rb file that actually runs rubocop. For now, I put it in Library/Homebrew/rubocops/bin/rubocop.rb. I could use some advice on that, though. Maybe something like Library/Homebrew/bin/rubocop.rb?

I think utils is currently the dumping ground for scripts like that.

Probably because Rubocop doesn't know what a FormulaUnavailableError is by just parsing the file under an isolated environment. When called within the brew environment, it will know since it will have been defined beforehand.

👍 makes sense

I think utils is currently the dumping ground for scripts like that.

Thanks, sounds good

One other thing I just noticed: should test/rubocop.rb be updated as well?
I quickly tried to modify the Open3.capture command to call ruby $HOMEBREW_LIBRARY/Homebrew/utils/rubocop.rb instead of ruby -S rubocop but it seems that the rubocop.rb file doesn't actually exist in the test HOMBREW_LIBRARY. Not sure what the best way to handle this is.

Edit: looking again at the test, I'm not sure this should be changed after all. It says when calling `rubocop` outside of the Homebrew environment, so I think it should use ruby -S rubocop after all.

Agreed 👍🏻
--
Another thing that I don't think requires a fix but is just worth thinking about is dev-cmd/rubocop.sh: does that need any changes/thought/consolidation?

Another thing that I don't think requires a fix but is just worth thinking about is dev-cmd/rubocop.sh: does that need any changes/thought/consolidation?

This seems to work, so let's give it a go!


Cannot automate gulp task cuz lack of css in the bower.json

"angular": "^1.5.5",
"angular-bootstrap": "^1.3.2",

Why is only included the .js in the bower ?
Angular-bootstrap  bower.json
{
    "author": {
        "name": "https://github.com/angular-ui/bootstrap/graphs/contributors"
    },
    "name": "angular-bootstrap",
    "keywords": [
        "angular",
        "angular-ui",
        "bootstrap"
    ],
    "license": "MIT",
    "ignore": [],
    "description": "Native AngularJS (Angular) directives for Bootstrap.",
    "version": "1.3.2",
    "main": ["./ui-bootstrap-tpls.js"],
    "dependencies": {
        "angular": ">=1.4.0"
    }
}

We don't take opinions on what version of bootstrap people use, whether the less, sass, or css version.


OD invalid inputs

Catch those invalid inputs.

Hmmm for grid_shape,
the thing is the toolkit won't throw any error or crash when it is a list of string.
And also 'a' > 0 will produce True in Python.
So I think we should still need to make sure that it contains int ?
Any thought?

Or we can check it in c++.
I don't know which one is better :) .

This is good. Actually we don't need to check > 0,
since it is check in c++.


[CCI] Move OuiBottomBar default inline styles to the CSS file

Is your feature request related to a problem? Please describe.
It is not possible to override the position of OuiBottomBar component without !important in the CSS file because the component uses inline styles. We have the option of not using !important in our styles, but then we would have to remove the default inline styles and use them in the CSS file so we have more flexibility.
Describe the solution you'd like
Move default inline styles to the CSS file
Additional context
The need for this issue was caused by https://github.com/opensearch-project/OpenSearch-Dashboards/pull/3804, and may also be caused in the future by something similar.

@SergeyMyssak Thanks so much for opening the issue and PR over here!

We can close this issue as https://github.com/opensearch-project/oui/pull/694 has been merged


Graph drops to 0 values at the end

The time range for the Metric.php getValues function goes from 0 to 10, I think it needs to go from 1 to 11 instead so that the average of the current period (i.e. the last value in the graph) still gets reported and is not set to 0.

I'll have a play with that :)

Yeah, this is far better. Thanks :beers:


MessageSend json payload does not decode MessageComponents

When using the library in a microservice design, I've tried sending MessageSend JSON payload via a PubSub broker.
When sending a normal message with just message content, the JSON decoding work correctly and is able to be sent to Discord.
When sending the payload with MessageComponents, the components do not get decoded and are set to an empty array, this also causes the Discord API to refuse the message because message components are set to nil.
I'd expect the message send unmarshal to be able to decode the components from a JSON payload.
Example MessageSend payload through PubSub:
{
   "channelId":"855166114047852588",
   "messageSend":{
      "content":"This is a test message which should (hopefully) contain message components/buttons",
      "tts":false,
      "embeds":null,
      "components":[
         {
            "type":1,
            "components":[
               {
                  "type":2,
                  "label":"Some Button",
                  "style":1,
                  "custom_id":"some_button"
               }
            ]
         },
         {
            "type":1,
            "components":[
               {
                  "type":2,
                  "label":"A second button",
                  "style":1,
                  "custom_id":"some_other_button"
               }
            ]
         }
      ],
      "message_reference":null,
      "allowed_mentions":null
   }
}

Example code:
type ChannelMessageSend struct {
	ChannelId string                   `json:"channelId"`
	MessageSend *discordgo.MessageSend `json:"messageSend"`
}

commandListener := redisClient.Subscribe("commandResponse")
discordMessage := ChannelMessageSend{}
for message := range commandListener.Channel() {
    err := json.Unmarshal([]byte(message.Payload), &discordMessage)
    // Error handling omitted
    _, err = discord.ChannelMessageSendComplex(discordMessage.ChannelId, discordMessage.MessageSend)
    // Error handling omitted
}

discordMessage.MessageSend.Components will be nil after, and unmarhsal will throw the following error:
json: cannot unmarshal object into Go struct field MessageSend.messageSend.components of type discordgo.MessageComponent
Discord would also refuse the message with the following error:
HTTP 400 Bad Request, {"code": 50035, "errors": {"components": {"0": {"_errors": [{"code": "LIST_ITEM_VALUE_REQUIRED", "message": "List item values of ModelType are required"}]}, "1": {"_errors": [{"code": "LIST_ITEM_VALUE_REQUIRED", "message": "List item values of ModelType are required"}]}}}, "message": "Invalid Form Body"}

Sorry for a late response.
MessageSend is mostly used for sending stuff, so we haven't implemented unmarshalling mechanism. You can implement your own though.

This is no longer an issue, I believe a solution was implemented for me a while ago, cheers though!


[Feat]: Add ui config for snowflake datasource form to enable key pair auth

Update snowflake datasource config to enable key pair authentication.

Key pair auth will have

Username text input field (mandatory)
Private key file picker to accept file for private key (mandatory)
Private key passphrase as text input (optional)

This should be behind feature flag for now : release_snowflake_key_pair_auth_enabled
https://www.notion.so/UI-for-key-pair-auth-ee41bf8f3b6c4fc4a71caf77aa4c4552

Added test case notion issue : https://www.notion.so/appsmith/42ab11bf621a42fb9670196b065d792e?v=2100cc622ba140d0b4dca0bd881a391a&p=c074b8bb44784390b84a441adc89e78f&pm=s


parse-base clarification

This issue is for my comment on https://github.com/perl6/doc/commit/4b9944d69fe1bb244bde49315284230a642dd8b7 (so that it does not slip through the cracks).

Performs the reverse of Numeric by converts a string with a base-$radix number to its base-10 Numeric equivalent.

What is a “base-10 Numeric equivalent”? Does not sound right.

There was a typo: it's

Performs the reverse of base by converting a string string with a base-$radix number to its base-10 Numeric equivalent.

Contains a number in base-10? Like in BCD? Heh
If we say that its .Str returns a number in base-10, fine. Or, we say that it is stored in binary, that's fine as well (although then you'd have to get into more details, so I'd rather not mention that at all). But “base-10 numeric” doesn't mean anything (the number itself is practically baseless from the user point of view).

Would you, please, change the text to how it would make sense to you?


fix: Deprecated warnings fixed

Fixes #905
- What I did

Replace depreciated withOpacity method with withValues in at_common_flutter, at_onboarding_flutter.
These fixes on the beta channel results in errors on the stable channel.

- How I did it
- How to verify it
- Description for the changelog

Thanks @CurtlyCritchlow
My inclination here is to hold off on making these changes until Flutter 3.25 is stable. We have --no-fatal-infos for the tests that run against stable. So the deprecation warnings won't cause stable channel static analysis to fail. We can then make changes that will work for both 3.25 and 3.26 as they'll both support withvalues.
👀 @gkc @XavierChanth


Spike: Site labels / translation

The storefronts is driven by different content streams. Most of the content is driven by either the CMS or by the product content. There's however a portion of content that is provided by so-called site labels.
Site labels are in the current storefront solution (spring mvc based) based on java resource bundles, which gives some issues:

they cannot be controlled by business users and require a full development cycle
they aren't exposed through a webservice
they do not have any structure (it's a flat list, although they're separated over multiple files)

There's a lot of room for improvement. We like to setup a solution that provides:

runtime language switching
lazy load languages
avoid hardcoded text

Angular doesn't provide these features yet, but have hired the author of translate to build this is as a standard feature in Angular. This takes longer then expected and it's recommended to start embracing ng-translate so that we can relatively smoothly transition to the standard solution (probably beyond ng7 and in parallel with ngyvi).

Prerequisites to investigate before we decide to use ngx-translate:

can we use ngx-translate library both in shell app and in the library? Doesn't it cause any problems?
can we use ngx-translate library together with SSR?

Acceptance criteria - when we are done:

no wordings are hardcoded in our code - all taken from JSON with translations
translation mechanism used everywhere:

in components' templates
in the code

we provide a few JSON files with translations in English (for Spartacus)
Customer can provide JSONs for other languages (for Spartacus)
Customer can overwrite our keys or extend (introduce new keys) with his own JSONs
Customer can reuse our keys in his custom components
Customer's lazy loaded components can use it's own isolated translations
current language used for translations is taken from site context
Customer can provide JSONs with translations for his custom components, by using the same library as us - ngx-translate

Other research:

how to support plural forms of translations using ngx-translate
how to structure the keys of translations and split into separate files - we should avoid breaking changes of the strucutre in the future
what are best practices for migrating from ngx-translate to the native i18n baked in the Ivy

Assumptions:

JSON files are just static assets served by the the shell app

Possible future outlook:

fetching translations from some occ endpoint and manage in CMS

Out of scope (beacause Ivy should be relased in the end of 2019):

migrate from ngx-translate to the native i18n baked in the Ivy

Questions:

should we provide JSON with translations for other languages before Launch 1.0?

i closed this ticket as a dupe of all the localization efforts, but will reopen if needed


Incompatibility with Firebase

I'm trying to add to an App google_mlkit_image_labeling in its version 0.5.0, but when I run "pod install" there is a problem with the dependencies of the latest versions of firebase plugins:

CocoaPods could not find compatible versions for pod "GTMSessionFetcher/Core":
  In Podfile:
    firebase_auth (from `.symlinks/plugins/firebase_auth/ios`) was resolved to 4.0.1, which depends on
      Firebase/Auth (= <IP_ADDRESS>) has been resolved to 10.0.0, which depends on
        FirebaseAuth (~> 10.0.0) has been resolved to 10.0.0, which is dependent on
          GTMSessionFetcher/Core (~> 2.1)

    google_mlkit_image_labeling (from `.symlinks/plugins/google_mlkit_image_labeling/ios`) has been fixed in 0.5.0, which depends on
      GoogleMLKit/ImageLabeling (~> 3.2.0) has been resolved to 3.2.0, which depends on
        MLKitImageLabeling (~> 2.2.0) has been resolved in 2.2.0, which depends on
          MLKitCommon (~> 8.0) has been resolved in 8.0.0, which depends on
            GTMSessionFetcher/Core (~> 1.1)

Could you please update the necessary dependencies so that the GTMSessionFetcher/Core versions match and can be used together?

I have found a temporary solution at https://issuetracker.google.com/issues/254418199#comment48

I have the same issue, thanks for the workaround but it would be nice to update the dependency inside the project directly


Update the Title in the partners section in the home page

In the "partners" section on the homepage, the title is currently: "Hosted on the ThreeFold grid "
As we want to be more specific, we should update it to " Partners Building on the ThreeFold Grid" ( or any other suggestion, that not imply that they fully hosted on the grid)

[ ] Please change "Hosted on the ThreeFold grid" -> "Utilizing the ThreeFold Grid"


First Microverse Pull Request

This is a simple HTML and CSS file

I have added the test.md file in the htmlcss-branch. I have also added the microverse template.


Pipes get broken when water level is reached

If I try to pipe into a WritableStream, if the stream on the other end of the pipe emits a chunk of data that exceeds the high water mark, the WritableStream stops listening to future chunks.  Here is a reproducible test case, where I compare the behavior of this module with another popular module exposing similar functionality, "stream-buffers":
const fs = require("fs");
const memoryStreams = require("memory-streams");
const streamBuffers = require("stream-buffers");

const reader1 = fs.createReadStream("test.jpg");
const writer1 = new (memoryStreams.WritableStream)();
reader1.pipe(writer1);

const reader2 = fs.createReadStream("test.jpg");
const writer2 = new (streamBuffers.WritableStreamBuffer)();
reader2.pipe(writer2);

setTimeout(() => {
	console.log("Bytes in write1 stream: " + writer1.toBuffer().length);
	console.log("Bytes in write2 stream: " + writer2.size());
}, 100);

Output:
Bytes in write1 stream: 65536
Bytes in write2 stream: 454528

In other words, this implementation of WritableStream stops consuming data after it first hits the high water mark, whereas the other implementation finishes reading the entire 454 KB image file.

I can confirm the same

Hello! As a work-around you can pass in highWaterMark with the options param when you construct the appropriate stream (node docs). If this does not meet your needs then feel free to create a pull request and I'll happily review and merge in with the project.


sdk version problem

Hi I couldn't use this package, I get error below:
depends on invert_colors >=0.0.2 which requires Flutter SDK version >=1.20.0 <2.0.0, version solving failed.

Lol this means , I should upgrade my own sdk , Oops

@AliRn76 sorry about that -- when I tried to publish with a lower requirement for dart pub.dev wouldn't let me.


Node type in export

Pretty sure that setting the inclusion of the node type in the export output doesn't actually do anything / isn't used. Fix this.

fixed in<PHONE_NUMBER>b509bcda7d0f41fa4fb34178dcf192


Feat: Improve Navbar

Describe the feature
Add Login/button
it it now normal  but i wish to make it a button and place it on second left from right.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

appreciate your recommendation but not planned yet!


README to get started isn't clear

I have really basic app using create-react-app.
Followed the README and have my index.js to look like this:
import React from 'react';
import ReactDOM from 'react-dom';
import StyletronClient from 'styletron-client';
import {StyletronProvider} from 'styletron-react';
import './index.css';
import App from './App';
import registerServiceWorker from './registerServiceWorker';

const styleElements = document.getElementsByClassName('_styletron_hydrate_');

ReactDOM.render(
  ,
  // <App />,
  document.getElementById('root')
);
registerServiceWorker();

I get this error when I add

 into index.html:
Uncaught TypeError: Cannot read property 'insertRule' of undefined

Without adding that div, I would get:
styletron-client.js:95 Uncaught TypeError: Cannot read property 'sheet' of undefined

All this error will only show up the moment I use styled:
import {styled} from 'styletron-react';
const Container = styled('div', () => ({
  width: '16px',
  height: '16px',
  display: 'inline-block',
  textAlign: 'center',
}));

const Foo = () => <Container />;  // this one causes the error there

For the purpose of hydrating server-side rendered styles, StyletronClient constructor takes an optional collection of style DOM elements, that should correspond to the server-side rendered style elements.
If you aren't doing SSR, you don't need to pass in anything to the constructor (it will create it's own style element for you).


Not working with forced HTTPS

Hey I'm a bit new to this whole thing but I got through the install process and have it running on mydomain.me:8000. What I am trying to do is have a subdomain point to that address which I have setup, something along the lines of music.mydomain.me.
Right now if I go to that subdomain it shows a white screen and checking the networking tool in chrome shows that I'm getting connections refused because Koel is requesting files through HTTP. I am using Caddy as my webserver and have not found an option to do 'Override ALL' as said in #360. Is there a way to use HTTPS natively with Koel or is this something I'm going to have to go to the Caddy people for?
Thanks!

Koel is just another PHP installation, so as long as you can run HTTPS+PHP in Caddy (I'm sure you can!), you can do HTTPS Koel. Something like this (I'm don't use Caddy, so fix where you need):
music.mydomain.me. {
    root /www/root/of/your/koel/installation/where/index.php/is/found
}

It was recommended to me to use Caddy's fastcgi directive for this, but I went ahead and tried that out for the Caddyfile:
music.mydomain.me {
    root /home/jack/koel/
}

and I just got the default PHP file:
<?php

/**
 * Laravel - A PHP Framework For Web Artisans.
 *
 * @author   Taylor Otwell<EMAIL_ADDRESS> */

/*

And this is while Koel is running at mydomain.me:8000

mydomain.me:8000 works because it's PHP's internal server. music.mydomain.me doesn't, because you haven't configured your system correctly to run PHP with fastcgi. Please google how to, since I don't use Caddy.


Lags

Please search to check for an existing issue (including closed issues, for which the fix may not have yet been released) before opening a new issue: https://github.com/k9mail/k-9/issues?q=is%3Aissue
Expected behavior
Tell us what should happen
Actual behavior
Tell us what happens instead
Steps to reproduce

Environment
K-9 Mail version:
Android version:
Account type (IMAP, POP3, WebDAV/Exchange):
Please take some time to retrieve logs and attach them here:
Very more lags in the job of program and with scrollings. Very bad optimization?

Sorry for my over quotting!... I writed: Very more lags in the job of program and with scrollings. Very bad optimization?

Please edit your first post and fill out the complete template. It's there for a reason. Without filling out, nobody can help you and the issue will be ignored or closed. There are no lags on my device. To find out what is different on your device, we need all the steps you do and information about your device.
Are you talking about the list of messages? Does turning off contact pictures and names of known contacts fix the lag?

Duplicate of #747


update boilerplate to current standards

Description

Updated CHANGELOG.tmpl
Updated Makefile
added tools.go
Restructured Docs folder
Updated .drone.star

Cool! You found a copy&pasta


Zoom into selected area

Explore the possibility for the map to zoom into the area selected (whether by point,
bounding box, polygon drawn on screen).

Use osmdata e.g. q <- getbb("Iles de la Madeleine") then use q (binding box) to determine zoom and loc.

https://github.com/r-spatial/mapedit/pull/100 Maybe some ideas for other features.

Now added as part of the validate tab in commit 326dd50e049017c828d77b3254cde2bf30af38e7


Crash when committing large files.

Occurred after https://github.com/Murmele/Gittyup/issues/522 completed. The window eventually closed, when it finally crashed.

Can you send us the logs shown when you run from the terminal?

@lonix1 and @Murmele, I don't possess the files anymore, although they were approximately a few gigabytes in size. They may well have been binary files primarily.

@Ionix1 in this case it is better to not commit such files directly into git, because they will slow down git extremely. Consider using git lfs for such files. Neverthless we have to check if it is possible to do it asynchronly so that the application wont crash

Small addendum to this: I've experienced crashes (process exit, not unresponsive) also with large files that are not committed or staged - the crash happens as soon as gittyup attempts to load the repository when a large file is in the working dir. It happens even when the diff is not shown.
I'm fully on board about not committing such large files to git - but it would be great if Gittyup would do a quick check for file size and if files are too big to handle, don't process it for diff and instead show a warning like File is too big for diff if someone makes the mistake of actually trying to view the file's diff. Since Gittyup crashes immediately I can't even ignore or delete the file through Gittyup (I can of course do this in other ways but it'd be very nice if Gittyup wouldn't crash so I could git-ignore it or simply not stage/commit the file).
Probably not helpful but this is the command line output:
Name:  "Staged" Current Collapsed:  6619245 ; New Collapsed:  0
Name:  "Unstaged" Current Collapsed:  6619245 ; New Collapsed:  0
Name:  "Staged" Current Collapsed:  0 ; New Collapsed:  0
Name:  "Unstaged" Current Collapsed:  0 ; New Collapsed:  0
terminate called after throwing an instance of 'std::bad_alloc'
  what():  std::bad_alloc
[1]    41006 IOT instruction (core dumped)  ./gittyup -d


ili9341 video splash screen not working. Ways to fix?

Hi there.
I was planning on adding a video splash screen to my pi zero Gameboy replica. I enabled splashscreen and chose a video of the splash screen but after I turn it on again the splash screen does not appear. Boot text still show and system still boots normally.
I have seen a similar issue that another user has posted awhile back. I was wondering if there is a way to fix this issue?
Thank you, I look forward to solving this issue.

Check out https://github.com/juj/fbcp-ili9341/issues/164 and https://github.com/juj/fbcp-ili9341/issues/20 , maybe one of those helps? I don't know of anything newer on this front.

Just using...
-DDMA_TX_CHANNEL=7 -DDMA_RX_CHANNEL=1
... worked for me.  This issue was driving me mad for ages as I couldn't show a splash screen either. Using this method is the only one that has worked for me.

apologies for the late reply, had a busy few weeks. I will give it a go and see if it works!

Still doesn't work : (

I ran into  other issues using the RX and TX channels. I may have found an easier way though.
Remove the RX and TX channels from cmake again.
Edit the asplashscreen.sh file within /opt/retropie/supplementary/splashscreen
fine the line:
omxplayer -o both -b --layer
and change "both" to "alsa"
This solved it for me and I can now use video splashscreens with the ili9341 driver

omg!!! it works!!! thank you so much!!!

You are welcome. Took me ages to find this out, but solved the last issue I had with this driver :)


Simplify extract_referer_site

We don't need to change the url, so just use the referer is fine? (actually I just want no goto)

I'm closing this as I think the issues brought up have been fixed. Otherwise, feel free to reopen it. Thanks!


Merge staging into main for a new docs release

lgtm, no errors in the staging deploy and the individual PRs have been reviewed already when merged into staging!


[New game]: TOWER BLOCKS 🚀

🎮 Game Request
Game logic and basic description

Tower Blocks is a simple yet addictive game where you have to stack blocks to build the tallest tower possible.

To play Tower Blocks, simply tap on the screen to drop a block. The blocks will fall straight down, so you need to be careful not to let them hit the ground or other blocks. As you stack more blocks, the tower will become more unstable, so you need to be strategic about where you place each block.

The goal of the game is to build the tallest tower possible. The game ends when the tower falls over. You can earn points for each block that you stack, and your score will be displayed at the end of the game.

Here are the features

Simple and addictive gameplay: Tower Blocks is a simple game to learn, but it can be challenging to master. The gameplay is addictive and will keep you coming back for more.

Variety of levels: There are a variety of levels to play, each with its own unique challenges. This keeps the game fresh and prevents it from becoming repetitive.

Free to play and fully Offline: Tower Blocks is a free game, so you can try it out without any risk.

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @0xabdulkhalid !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @0xabdulkhalid!
We are already having a similar game request in #481 👀
Make sure you come up with a cool unique idea 😀
Waiting for your new game idea 💗.

Hey @0xabdulkhalid ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Thanks for pointing this out!, sure i have other games let me raise an issue for them.


Update Jumpshare.app to version 2.1.1

Closing this in favour of #14593

Thank you for the contribution. It was merged directly as b2e309d10530f3b8b7ce33f53723493c0a433b06 to keep commit history cleaner. Your contribution is still credited to you.

:+1:


s value is undefined in ProofGen

In (currently) step 11 of ProofGen:
B = P1 + H_s * s + H_d * domain + H_1 * msg_1 + ... + H_L * msg_L

The scalar s is undefined. Is it supposed to be s~ generated in step 9?

Ah, that's the s from the signature; missed that.


Fixing ux/live-component "main" path in package.json

Q
A

Bug fix?
yes

New feature?
no

Tickets
Fix #179

License
MIT

Just an oversight after redoing the build process.

Thank you Ryan.


TOC doesn't contain  anymore (0.62.2)

What version of Hugo are you using (hugo version)?

Hugo Static Site Generator v0.62.2/extended windows/amd64 BuildDate: unknown
GOOS="windows"
GOARCH="amd64"
GOVERSION="go1.13.4"

Does this issue reproduce with the latest release?
Yes, it's actually a new issue.
I could not find in the release notes documented changes to TOC's  behaviour.
Before
Earlier, when our Markdown headings contained code, that code would appear as  in the TOC.
For example:
## Hugo's `where` function
Performs filtering.

## `.RenderString` is new

Renders the following HTML on Hugo 0.55.6:

Hugo’s where function

.RenderString is new

Note the  around the words with a backtick.
Now
With Hugo 0.62.2,  doesn't appear in the TOC anymore. For example, the same Markdown now makes this TOC:

    Hugo's where function

.RenderString is new

Repository to replicate
See https://github.com/Jos512/hugo-toc-issue

Yes, it's actually a new issue.

So, in Hugo 0.61 it behaved differently?

So, in Hugo 0.61 it behaved differently?

Sorry, I answered from my own perspective. :man_facepalming:
To answer it properly, the 'issue' first happened in:
Hugo Static Site Generator v0.60.0/extended windows/amd64 BuildDate: unknown
GOOS="windows"
GOARCH="amd64"
GOVERSION="go1.13.4"

And the last good version was:
Hugo Static Site Generator v0.59.1/extended windows/amd64 BuildDate: unknown
GOOS="windows"
GOARCH="amd64"
GOVERSION="go1.13.1"


Multiline string variables

Is your feature request related to a problem? Please describe.
I could not find a way to define multiple line string variables. Something like:

var test = @"
asdas
asdasasd
asdasasd
"@

or
var test = '
asdas
asdasasd
asdasasd
'

Describe the solution you'd like
A clear and concise description of what you want to happen.

To clarify, do you want the same functionality we have in ARM templates today? It keeps the line breaks when the string is sent to ARM. Alternatively, you may want us to drop the line breaks.

I would prefer to have the string defined in multiple lines but when send to ARM the line breaks are removed. If we need to add line breaks we can do it explicitly whit escape characters. For me this is the more natural as when I have some long text (descriptions, kusto queries or something) for better viewing of the code and less horizontal scrolling this is the better behavior for me. Of course other folks can chime in and express different opinion. I am fine with both approaches as long as it is supported.

Would you expect the multi-line strings to allow interpolation?

Yes, Also will look weird when you have mixture of objects and arrays within each other and the arrays have to be one liner.

OK, just been pointed to this, thanks :-)
My requirement is for inline scripts and cloud-init yaml files so would expect line breaks to be preserved and for the text to include references that are interpolated if I understand the term right so that it may be customised during deployment

Would you expect us to normalize line endings or preserve them in the file?
If it's the latter, be aware that Git does its own line ending normalization unless you opt-out. If it's the former, would \n be sufficient or would you want to control this?

If I understand that is would I want any CRLF mapped to CR then I'm ambivalent. I try to only use CRs so would be happy to correct my own code to get the right output but if it was normalised automatically that would be fine.

For now I am using:
param customDataRaw string = '#cloud-config\n\nruncmd: \n- mkdir -p /mnt/%dir%\n'
which is sort of ok for a small config but not terribly accessible. I would like to be able to define this more like:
param customDataRaw string = '
#cloud-config
runcmd:

mkdir -p /mnt/%dir%
'

HTH

Something to be careful about regarding whether to output \n vs \r\n in the generated output - I recall hearing that some APIs are fussy about getting the right character.

Sorry, my - example keeps being turned into a bullet :)
I'm currently using this:
param customDataRaw string = '#cloud-config\n\nruncmd: \n- mkdir -p /mnt/%dir%\n'
but would prefer to use something more like this:
`param customDataRaw string = '
#cloud-config
runcmd:

mkdir -p /mnt/%dir%
'`

The content I want to inject runs to ~50 lines of cloud-init yaml file

@lr90 FYI you can use markdown's fenced code blocks to paste code without having GitHub messing up the formatting:
https://docs.github.com/github/writing-on-github/creating-and-highlighting-code-blocks#fenced-code-blocks

I think triple quoted strings would be great, especially if we're talking about embedded bash/pwsh scripts
https://github.com/Azure/bicep/blob/main/docs/examples/101/deployment-script-no-auth/main.bicep

Team Discussion - 02/03/21
Multiline strings
Proposed syntax
var myString = '''
this
is
a
multiline
string
'''

Multi-line strings begin with 3 consecutive single quotes (''') followed immediately by a newline character sequence, and are terminated with 3 consecutive single quotes (''') immediately following a newline character sequence.
Considerations
What should the opening/closing syntax be?

We agreed on ''' as it aligns with the current string syntax.
We discussed whether we could allow the user to optionally pick n number of ' characters (n >= 3) - e.g. '''' or ''''' in order to include strings containing '''. We decided this was overkill for now, but we could consider it for a future improvement if necessary.

Should we give users the flexibility to pick the newline character?

We decided that we simply use the newline sequence from the .bicep file - CRLF or LF would be converted into the appropriate JSON-escaped newline character on conversion. For example, in a file with LF newline sequences, Bicep would consider these two strings equivalent:var multiline = '''
hello
Anthony!
'''
var singleline = 'hello\nAnthony!'

Should we support string interpolation?

We decided that for now, we would not support string interpolation or any form of escaping. The reason being that if you have an escape sequence, you lose the ability to paste in any old text without formatting it correctly. If users need variable replacement, it would still be possible using the format() function - e.g.:var unformatted = '''
hello {0}!
'''
var formatted = format(unformatted, 'Anthony')

A potential option for interpolation in the future we discussed is to match the number of $ characters with the number of single quotes - e.g.:var myString = '''
hello $$${name}
'''

How should we handle whitespace?

Users may want to indent the mutliline string block to match the formatting of the document. We decided that rather than trying to second-guess the intention, we would simply use the whitespace as provided. This means that the following two strings would be regarded as equivalent:var multiline = '''
    I'm
    Indented!
'''
var singleline = '    I\'m\n    Indented!'

Summary
We decided to go with the proposed syntax, leaving some options open for future extension w.r.t interpolation & using different multiline string opening/terminating sequences.
Samples
Valid Strings
// basic sample
var basic = '''
hello
   this
       is
           valid
'''

// some edge cases - this is all going to be treated as one valid multiline string
var horribleSample = '''
 ''' <-- it's totally fine to do this
 ' <--- or this
 \n <---- this does not need escaping
'''we still
haven't '''
terminated
  the ''' string
'''

// comment-line syntax will be included in the string output and not treated as comments
var commentsAreIgnored = '''
  these // comments
  are /* going to be
  included in the verbatim */
  string
'''

Invalid Strings
// the initial ''' must be followed immediately by a newline
var invalidStartSequence = '''this
is
invalid
'''

// the final ''' must be immediately after a newline
var invalidEndSequence = '''
this
is
invalid'''

// ''' may not be used inside a multiline string
var disallowedCharacters = '''
oh
'''
no
'''

FunctionApp with JavaScript
resource testResourceFunc 'Microsoft.Web/sites/functions@2018-11-01' = {
  name: '${backingSite.name}/${testResourceName}'
  properties: {
    config: {
      bindings: [
        {
          authLevel: 'function'
          type: 'httpTrigger'
          direction: 'in'
          name: 'req'
          methods: [
            'put'
            'get'
            'delete'
          ]
        }
        {
          type: 'http'
          direction: 'out'
          name: 'res'
        }
      ]
    }
    files: {
      'index.js': '''
module.exports = async function (context, req) {
    console.log(context);
    setResponse(context, 200, req.body);
};

function setResponse(context, code, body) {
    context.res = {
        body: body,
        status: code,
        headers: {
            \'Content-Type\': \'application/json\',
        }
    }
}
'''
    }
  }
}

DeploymentScript with PowerShell
resource scriptName_mysqlShareName 'Microsoft.Resources/deploymentScripts@2020-10-01' = {
  name: '${scriptName}-${mysqlShareName}'
  location: location
  kind: 'AzurePowerShell'
  identity: {
    type: 'UserAssigned'
    userAssignedIdentities: {
      '${identityName.id}': {}
    }
  }
  properties: {
    forceUpdateTag: '1'
    azPowerShellVersion: '3.0'
    arguments: ' -storageAccountName ${storageAccountName} -fileShareName ${mysqlShareName} -resourceGroupName ${resourceGroup().name}'
    scriptContent: '''
param(
    [string] $storageAccountName,
    [string] $fileShareName,
    [string] $resourceGroupName
)
Get-AzStorageAccount -StorageAccountName $storageAccountName -ResourceGroupName $resourceGroupName | New-AzStorageShare -Name $fileShareName
'''
    timeout: 'PT5M'
    cleanupPreference: 'OnSuccess'
    retentionInterval: 'P1D'
  }
}

We passed this plan by Anders, who suggested we make the following changes:

Newline after the opening ''' should be made optional, but will not be treated as part of the string if it is present. This means that the following strings are equivalent:var string1 = '''hello
'''

var string2 = '''
hello
'''

Newline should not be considered part of the closing set of '''. This means that if it occurs after a newline, the newline will be considered part of the string. This makes pasting in verbatim text simpler and more predictable:// evaluates to 'hello\n'
var string1 = '''
hello
'''

// evaluates to 'hello'
var string2 = '''
hello'''

As a consequence of the above two points, it should be possible to have a one-line verbatim string: var string1 = '''hello'''
Because we are loosening the restrictions on where the ''' can go, we should allow the user to specify 3+ quotes to open/close the string. This gives users the freedom to paste strings containing ''' by picking a different number of opening/closing quotes. So the following are all valid:var string1 = ''''
I can paste ''' here
''''

var string2 = ''''''''
hello!
''''''''

We should stick with the plan to disallow interpolation as it makes pasting in verbatim text more straightforward.
The lexer should treat the full verbatim string as a single token, including the opening and closing quotes

Also this:

If a multi-line string is not terminated correctly (that is starts with ''' but never ends), the remainder of the file becomes part of the string token and a lexical error about the unterminated string is emitted.

I also don't think that we need to introduce a new type of AST node to represent these strings. We should try to incorporate this into the existing StringSyntax.

For bonus points, we can also have the decompiler generate a multi-line string when a string literal in an expression exceeds some threshold.

Only to say I’m struggling to contain my excitement at this and have got some great scenarios to start testing it!!

One more serious comment. Disallow interpretation? Please clarify but with my multi line string I would like (require!) placeholders such as ${targetName} or some such that will be evaluated during the deployment

One more serious comment. Disallow interpretation? Please clarify but with my multi line string I would like (require!) placeholders such as ${targetName} or some such that will be evaluated during the deployment

Would you mind sharing some example use cases for this? When we discussed this, our thoughts were:

Allowing interpolation means that there's more to think about when copy+pasting preformatted text into a multiline code block, as you need to consider escaping. We wanted to keep it very simple so that you'd just have to consider whether your text contains '''.
If using this to generate e.g. script blocks, interpolation replacement may make it easy to create subtle escaping issues, because it's not going to be aware of the semantics of the underlying language - for example:var name = 'Bruce "The Boss" Springsteen'
var powerShellScript = '''
Write-Output "Hello ${name}"
''' // this is not going to escape the " characters, so the powershell script won't have the desired effect

It's possible (though less nice) to achieve a similar effect by DIYing it with the format function - e.g:var powerShellScript = format('''
Get-AzStorageAccount -StorageAccountName "{0}" -ResourceGroupName "{1}" | `
    New-AzStorageShare -Name "{2}"
''', storageAccountName, fileShareName, resourceGroup().name)

One possible workaround we discussed for the 1st point was that we could require a matching number of $ characters for interpolation to make it at least configurable - e.g.:
var output = '''
Hello $$${name}
'''

// this is a Bicep string containing valid Bicep code
var manyQuotes = ''''''
var name = 'Anthony'
var thisIsEscapedBicepCode = '''
Hello $$${name}
'''
''''''

Sure, my current burning use case is Cloud-Init and CSE scripts. While many don't require any interpolation there is a working example here:
(https://github.com/Azure/bicep/blob/b2de1c425a461950b03bc34bd2c23b7177f2a7d8/docs/examples/101/cyclecloud/main.bicep#L12) where I am replacing placeholders in the script with values determined at deployment time.
You can also get a sense of how much fun it isn't turning a perfectly good 90+ line yml file into a single line string variable :-) IIRC I had to also escape some of the ' in that line but from what I'm seeing as proposed at the moment that burden would also be removed and reduce the risk of introducing errors to the yml.

I would agree with @lr90 here. I would say the need for interpolation would be higher than to be able to copy+pasting preformatted text which might need escaping.

We're planning to go with verbatim strings (no escaping) for the first implementation, and plan to collect feedback to make the case for adding interpolation.
Something I realized after my above suggestion using the format function with multiline strings is that a lot of the benefit is lost, since you need to escape { -> {{ - e.g. the below would not work:
var jsCode = format('''
function sayHello() {
  console.log(`Hello {0}!`);
}
''', myName)

To make this work, you would need to modify to use the following:
var jsCode = format('''
function sayHello() {{
  console.log(`Hello {0}!`);
}}
''', myName)

Another workaround (though definitely not pretty to look at) would be to use concat:
var jsCode = concat('''
function sayHello() {
  console.log(`Hello ''', myName, '''!`);
}
''')

I could live with verbatim multi line strings. For a limited number of replacements I could just iterate through the replace() function
  var mltext = '''
  Line 1
  {0}
  Line 3
  '''

  var parsedtext = replace(mltext, '{0}', 'Line 2')

Key thing is it will make editing the original multi line string a lot easier

@anthony-c-martin it appears expressions are not evaluated in multiline? Was this intended? If so it severely limits the benefit of multiline.
Also maybe there should be an option to control whether you use \r\n vs \n? This can affect if using a multiline string for userdata in a VM to preconfigure information.
Bicep
output fgaManagementSSHConfig string = '''
Host ${fortigateA.outputs.fgName}
  HostName ${fqdn}
  Port ${fgaManagementSshPort}
  User ${adminUsername}
'''

##Expected ARM
subexpressions get evaluated
##Actual ARM
    "fgaManagementSSHConfig": {
      "type": "string",
      "value": "Host ${fortigateA.outputs.fgName}\r\n  HostName ${fqdn}\r\n  Port ${fgaManagementSshPort}\r\n  User ${adminUsername}\r\n"
    },

##Workaround
Continue to use single line formatting with \n separators

How about doing something python's f-strings or C#'s $-string interpolation:
var interpolated = 'interpolated'
var s = $'''
this is ${interpolated}'''

var ns = '''
this is not ${interpolated}'''


Action workflows refined: linting, coverage, and Black

Actions separated
More work on GitHub Actions workflows for the project.
With this update, the original main.yaml has been broken down, with new workflow functionality added to different files (filenames don't matter to Actions). While much of this could be accomplished in the same workflow, I wanted to break down those concerns into more succinct workflow files that can handle separate concerns, potentially having different triggers in future.
Plus, the testing suite runs across 3 OSes and 4 Python versions, and we don't need 12 separate calls for generating coverage reports for each push. Best to have a single VM do a single task.
Breakdown

main.yaml renamed testing.yaml, and workflow renamed to simply "Testing" (from "CI Testing").
pytest-cov and codecov removed from "Testing" workflow. Now reside in their own workflow, coverage.yaml / "Coverage report".
Added an initial workflow for checking Black formatting, formatting.yaml / "Code formatting check"

This marks the start of a preference for code that is formatted with Black before being merged, with the goal of reducing the size of code diffs and removing a concern over code formatting issues. Contributors will be encouraged to update future PRs to match this style.
Currently disabled, to be un-commented and enabled in a future change.

State of the code base is not quite ready to have this check in place. We will format existing modules and re-enable the Black check in a future PR.

Added flake8 linting back to the tests, in linting.yaml / "Linting"

Configuration in setup.cfg has been updated to ignore some errors on which black and flake8 might disagree (we prefer that the formatting by black never be the cause of an error in flake8).

Initial Python code formatting
Per the addition of the flake8 checks, any files that were showing linting errors in this update have been formatted by black to resolve them. All changes to Python code in this update are purely for formatting purposes.

Changes noted earlier, regarding formatting.yaml, have been added to a newer branch based off this one. Reviewing and merging this one would be a wasted effort, as these changes are incorporated into task-paint-it-black.
Closing this, opening the newer one.


Inconsistent theme and display of docstrings

Details

Read the Docs project URL: https://distributed-malaria-detection.readthedocs.io/en/latest/index.html
Build URL (if applicable): https://readthedocs.org/projects/distributed-malaria-detection/builds/8984800/
Read the Docs username (if applicable): 2fasc
Github: https://github.com/2fasc/Distributed_Malaria_Detection

Expected Result
I expected RTD to apply the rtd theme and display sites with the documentation for my functions.
Local builds work just fine.
Actual Result
The displayed theme is a mixture of alabaster and the rtd theme and there is no documentation generated for the functions that show the alabaster theme. I don't know if this is related though.
Thank you for your help!

The read the docs theme doesn't support sphinx 2.x yet https://github.com/2fasc/Distributed_Malaria_Detection/blob/1d30cfd9a94809ba39d1ca42532d294f1a8697a1/requirements_rtd.txt#L78
https://github.com/rtfd/sphinx_rtd_theme/issues?utf8=✓&q=is%3Aissue+is%3Aopen+sphinx+2

Thank you!

I checked a couple hours later and everything is displayed as it should.

I checked a couple hours later and everything is displayed as it should.

I think the mixture about sphinx and alabaster was because of cache :)

I think the mixture about sphinx and alabaster was because of cache :)

That makes sense.
I don't know if this is of general interest but adding a sentence in the FAQs that make clear that updates might not occur immediately could be helpful for others as well :sweat_smile:
I really appreciated your support!


Still necessary to install filo for groupBy?

@arq5x, could you clarify the status of groupBy? I was under the impression that users need to install groupBy separately by downloading filo (https://github.com/arq5x/filo) and running make on it. But to my surprise my newly installed system (in which I first ran make for filo and then reinstalled bedtools per the instructions at http://bedtools.readthedocs.io/en/latest/content/installation.html) reports:
$ groupBy
***** ERROR: -opCols parameter requires a value.

Tool:    bedtools groupby
Version: v2.26.0

But http://bedtools.readthedocs.io/en/latest/content/history.html#version-2-11-0-january-21-2010 says:

Major Changes:
The groupBy command is now part of the filo package (https://github.com/arq5x/filo) and will no longer be distributed with BEDTools.

The filo package at https://github.com/arq5x/filo hasn't been updated in years, though, and seems to be stuck at v1.1.0.

(I started to rely on groupBy after finding instructions on calculating mean coverage at https://groups.google.com/forum/#!msg/bedtools-discuss/WSNEVuHIpz0/qPPlm2XSDQEJ, which mention that groupBy is part of filo)

@arq5x, thanks for clarifying.
For anyone stumbling on this thread, @arq5x wrote in #418:

The groupby tool in filo is no longer maintained and has been replaced by the groupby tool in bedtools.


Remove version: CrystalDewWorld.CrystalDiskMark version 8.0.4

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://osdn.net/frs/redir.php?m=ipconnect&f=crystaldiskmark%2F75540%2FCrystalDiskMark8_0_4.exe
The SSL connection could not be established, see inner exception.
(Automated message - build 897)

URL: https://osdn.net/frs/redir.php?m=ipconnect&f=crystaldiskmark%2F75540%2FCrystalDiskMark8_0_4.exe
The SSL connection could not be established, see inner exception.
(Automated message - build 897)

URL: https://osdn.net/frs/redir.php?m=ipconnect&f=crystaldiskmark%2F75540%2FCrystalDiskMark8_0_4.exe
The SSL connection could not be established, see inner exception.
(Automated message - build 897)

URL: https://osdn.net/frs/redir.php?m=ipconnect&f=crystaldiskmark%2F75540%2FCrystalDiskMark8_0_4.exe
The SSL connection could not be established, see inner exception.
(Automated message - build 897)

URL: https://osdn.net/frs/redir.php?m=ipconnect&f=crystaldiskmark%2F75540%2FCrystalDiskMark8_0_4.exe
The SSL connection could not be established, see inner exception.
(Automated message - build 897)

URL: https://osdn.net/frs/redir.php?m=ipconnect&f=crystaldiskmark%2F75540%2FCrystalDiskMark8_0_4.exe
The SSL connection could not be established, see inner exception.
(Automated message - build 897)

URL: https://osdn.net/frs/redir.php?m=ipconnect&f=crystaldiskmark%2F75540%2FCrystalDiskMark8_0_4.exe
The SSL connection could not be established, see inner exception.
(Automated message - build 897)

URL: https://osdn.net/frs/redir.php?m=ipconnect&f=crystaldiskmark%2F75540%2FCrystalDiskMark8_0_4.exe
The SSL connection could not be established, see inner exception.
(Automated message - build 897)

URL: https://osdn.net/frs/redir.php?m=ipconnect&f=crystaldiskmark%2F75540%2FCrystalDiskMark8_0_4.exe
The SSL connection could not be established, see inner exception.
(Automated message - build 898)


add prefect to qhub

This adds prefect to a qhub deployment when the flag prefect: True is set

[ ] TODO for myself: Check the the Github actions file have prefect token as variable

[ ] TODO for myself: Check the the Github actions file have prefect token as variable

LGTM

LGTM


Fix error using latest VS2019 to build ponyc

Fixes #3366

CirrusCI failures are unrelated. I just merged a fix to master for those (purely a setup issue when trying to do CI for PRs from forked repositories).


[Bug]: Clicking on the debug icon breaks App

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Clicking on the debug icon breaks App
Steps To Reproduce

Go to this App (https://release.app.appsmith.com/app/b01b5ee3-2/page1-62b99aac91ee3a540c639771)

Create an Api without a URL and run

Click the debug button

Public Sample App
https://release.app.appsmith.com/app/b01b5ee3-2/page1-62b99aac91ee3a540c639771
Version
Release

Closing in favour of https://github.com/appsmithorg/appsmith/issues/14836


remove rethinkdb course

What does this PR do?
Remove Resource
For resources
Description
This PR removes Rethinkdb course as the authors has removed it from their website.

Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

Thanks, and welcome to free-programming-books!


JB-570 - [Mobile & Desktop] Auto scroll to 'about' tab on-click of 'read more'

What does this PR do and why?
Closes https://linear.app/peel/issue/JB-570/[mobile-and-desktop]-auto-scroll-to-about-tab-on-click-of-read-more
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3848

PR #3851   👈

This comment was auto-generated by Graphite.

Graphite rebased this pull request as part of a merge.

@wraeth-eth merged this pull request with Graphite.


feat: typed API definition

🔗 Linked issue
#364
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[x] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This change will allow developers to define their api schema at one place and get type-safe fetch client.

Resolved #364
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

really looking forward to use this!


Latino .NET event

Hi, @jorgelevy I have added some comments about the API ID. If you want to integrate your agenda with the GlobalAzure.net website that would be awesome, if not, just comment here and I will approve this for you as it is.


Refactor

Separated the scripts and attempt to use classes, DI

Refactor: Introduced classes and DI using kink library


Remove doutcloud packages

the dotcloud github repo redirects to docker, and is no longer valid.
use the go-dockerclient package to handle registry auth entirely, so
we no longer need to import the docker auth package (which you can't
directly go get)
Don't try to use the docker term package, since IsTerminal doesn't
even work with our non-cgo, cross-platform build.

:+1:


Cow: Watch odooconsultant.com

Cow requests the watch of the watch_keyword odooconsultant\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

odooconsultant\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Nick in Charcoal HQ


Update installation and release instructions

Closes #38

Weird that appveyor flaked out on the branch for this change set

Yeah the test are a bit fragile. This problem also exists on the upstream repo.


[Fixes #15911] Examples — Fixes navbar issues if brand is empty

This includes the navbar fix. I did not notice any difference with the footer code so I did not include that.
Fixes #15911.

Note that :empty isn't supported by IE8, but we can probably spin this as a progressive enhancement.

That's interesting, thanks for sharing that. I wonder why IE was so late when it was supported by FF 1.0 / Chrome 1.0 per MDN

Simpler fix in 05513c52c4b31db3646639ba4e1b8a5ee7271ae9.
Rather than account for weird empty states, this just makes the component more durable. The padding goes on the heading and the header gets a clearfix to ensure floats don't bust the flow.

Sounds good. Thanks for that info @mdo

@kkirsche In general, it's best to solve things like this with fewer exceptions to individual components. The presence of an element shouldn't have much of an unexpected affect on it's parent like that. It was an oversight on my part of not clearing floats properly. Hope that helps shed a bit of light on it :).


subcall examples

This implements the following runtime-sdk functions:

consensus.Undelegate
consensus.Delegate
consensus.Withdraw
accounts.Transfer

It also supports generation of staking wallet addresses. via Subcall.sol:ConsensusUtils.generateStakingAddress
consensus.Deposit isn't supported.
I will need to update the documentation to include some subcall examples using the Subcall.sol library.
I should probably remove subcall from Sapphire.sol, as all the functionality for subcalls is implemented in Subcall.sol.

Please rename StakingPublicKey to StakingAddress. Also let's use term "address" for destination address instead of "account" for delegation/undelegation/withdrawal.

I'm not sure why I used public key, I even had a function called derive address from public key...


[Activity Port] Fraction Bounce

This is an initial build of Fraction Bounce game for issue #205 implementing the single player mode. Adding this to get an initial review on the progress made so far.

Single player mode is completely playable through keyboard with working options such as changing ball & background images, percentage mode, etc. The smileys are logged similar to the python activity. Custom fractions can also be added and will be saved in the journal.
Yet to implement:

[x] Sectors mode for the ball
[x] Toolbar help text
[x] Adding custom ball/background from journal
[x] Add Touch Controls
[x] Tutorial for the activity

This is an initial build hence it might contain some bugs. Please review the progress @llaske and @ashish0910 and let me know the changes if any.

Very good start. Congrats!

I have implemented most of the functionalities of the activity. I have also made the activity touch-friendly, i.e., if the user touches to the left/right of the ball, the ball will move in the respective direction.
The workflow of multiplayer isn't completely clear to me, @llaske can you please clarify exactly how the multiplayer is going to work?
Please review the current build @llaske and @ashish0910.

Very cool.
I don't think a multiplayer version make sense for this activity.
Few remarks:

[ ] The game should be playable with the mouse too
[ ] The current mode (ball, fraction, percent) should stay selected in the toolbar (use active property in toolbar item)
[ ] There is no visual indication that the answer is correct: I suggest to replace red line by a green line when the answer is correct
[ ] The user should have a way to pause the game (may be by clicking again on the ball)
[ ] The context of the game is not saved in the Journal
[ ] The localization file refer to chess string :-)
[ ] In percent mode the 1 to the right of the screen should be replaced by 100%
[ ] The Add fraction don't seem to work
[ ] When the game is played for a long time it hang the browser (look like there is an infinite loop somewhere)
[ ] I suggest to stop the game after 10 fractions
[ ] There is a scrollbar in the Journal popup

[ ] When a custom background is set, I suggest to fit the image to the screen
[ ] Remove debug messages in the code
[ ] Miss a tutorial

In my latest set of commits:

[x] The game should be playable with the mouse too
[x] The current mode (ball, fraction, percent) should stay selected in the toolbar (use active property in toolbar item)
[x] There is no visual indication that the answer is correct: I suggest to replace red line by a green line when the answer is correct
[x] The user should have a way to pause the game [Can be paused by clicking the ball again.]
[x] The context of the game is not saved in the Journal [Custom fractions will be saved in the journal]
[x] The localization file refer to chess string :-) [An honest mistake. xD Fixed it.]
[x] In percent mode the 1 to the right of the screen should be replaced by 100%
[x] The Add fraction don't seem to work [Fixed a bug using parseInt]

[ ] When the game is played for a long time it hang the browser (look like there is an infinite loop somewhere)
[ ] I suggest to stop the game after 10 fractions

I haven't been able to replicate this issue. How long did you play the game for?
Maybe we can make other improvements rather than stopping the game like reducing delay between bounces the longer you play as mentioned in the wiki notes. (https://wiki.sugarlabs.org/go/Activities/Fraction_Bounce)

[x] There is a scrollbar in the Journal popup

[ ] When a custom background is set, I suggest to fit the image to the screen

The current behavior is to cover the screen with the image. If I fit the entire image, there will be blank spaces on the sides if the image's aspect ratio doesn't match the screen.

[x] Remove debug messages in the code
[x] Miss a tutorial

Good job. It's cool to be able to play to this nice Sugar activity on Sugarizer.
My remarks:

Pausing the game by clicking on the ball again is not a good idea. The user could accidentally click on the ball instead of to the left and to the right and it's frustrating to see the game paused. I suggest to include a play/pause button in the toolbar (like in the Physics activity)
The 1 at the bottom right of screen should be replaced by 100% as soon as the Percent mode is selected
Do not display decimal on percent value: 33,33% = 33%
Mark unselectable text/content for score else it tend to be selected when clicking on the board

Good idea to increase complexity the longer you play, let's go
I'm not comfortable with an infinite game. I suggest the game stop after 10 plays. Any attempt to start the game will reinit the score board
Changing ball by an image from the Journal has a strange behavior before start

Regarding background image, may be you could repeat the image if it don't fit the screen?
I think the speed of the ball should be adapted to the height of the screen else the game is very complex on small screens
I suggest to rename the activity Fraction instead of Fraction Bounce (just the title, you could keep the same directory and id)

In the last series of commits:

[x] Added play/pause button in toolbar similar to Physics activity.
[x] The 1 at the bottom right of screen should be replaced by 100% as soon as the Percent mode is selected
[x] Do not display decimal on percent value: 33,33% = 33%
[x] Mark unselectable text/content for score else it tend to be selected when clicking on the board
[x] Reducing delay of launch by 100ms after each attempt to increase complexity
[x] The game now stops after 10 attempts. Added a help message in the toolbar

[x] Fixed strange behavior after image upload before starting the game
[x] Repeating the custom image if it doesn't fit.
[x] Added a formula to adapt speed as per height of the screen. Might not be perfect but should work with most screen sizes.
[x] Changed the name to Fraction

Very good job. Will try how it works on mobile device.

I've done some testing on tablets, it's pretty usable but, to move the ball the user should let its finger on the screen (instead of trying to touch several times).
BTW I've two more remarks.

There is a bug when the game is restarted the green/red indication no longer appears (see below):

In the initial Sugar game, a sound is played when the ball touch the ground. See here and here. You could copy this sound from the TamTam micro activity, it's crash.mp3 (if fail) and bottle.mp3 (if success).

Made the required changes @llaske.

[x] Fixed ball not moving on multiple taps
[x] Green/red bars will now appear as soon as the ball touches the slope
[x] Added success/failure sounds

Nice.
The original version also include support for accelerometer to move the ball. See here.
Do you think you could add it too? You could take inspiration on Physics activity that rely on Cordova accelerometer support. It should not be so complex. See here.
One more thing: the tutorial should be updated to mention the new Play/Pause button.

I have added the accelerometer support to the game, taking inspiration from Physics activity. Also updated the tutorial for play/pause buttons and accelerometer. Looking forward to your review @llaske.

It's perfect.
Thanks a lot to port this mythic Sugar activity in Sugarizer.


Datatypes with duplicate names

Your README says:

Dot not parse datatypes with names that already exist anywhere, regardless of in which category, otherwise type might end up with wrong name

I think the solution for this is to use an appropriate DataTypeConflictHandler in this line instead of null
https://github.com/Katharsas/ghidra-struct-importer/blob/1ae2ec69d723d25ec22da390c32920bcb2d08f02/ImportCStruct.java#L241
The default one implicitly used does a "rename and add" and probably leads to the issue described in the README.

Cool thanks, ill try that!

Hey @fmagin
The big problem im seeing here is that the "Parse" Button is already seemingly modifying the state of the global DataTypeManager. Simply executing the line type = parser.parse(text); can already override existing types, if a struct with existing name was parsed. The apply button does not need to be pressed at all.
The reason seems to be that the parser actually adds/overwrites datatypes on parsing. The line type = parser.parse(text); will call dtMgr.startTransaction("Parsing"); by itself, the apply button is not needed. This happens because the parser is instantiated with our global DataTypeManager (var parser = new CParser(dtm);).
I think to make this work as expected (only modify program state when apply button is actually pressed) i have to pass a temporary DTM that acts as dataType buffer, but can still resolve referenced types from the global DTM. I will look into that.

Ok i implemented BufferedDataTypeManager  in the latest commit, and it fixes the the main issue (datatypes do no longer get added on parse). I tried to basically implement a combined view onto two underlying DataTypeManagers: an standalone in-memory DTM as buffer and the global program DTM. It does resolve existing program DataTypes for the parser, so that is great.
However, i have a problem with the current implementation:
The datatypes do not actually get saved inside the buffer, they simply disappear. The only reason why i do no loose them entirely is that i get the last parsed DataType from the parser on parsed, just as in the original implementation.
So now, we cannot:

parse multiple structs at once because all except the last one disappear
have multiple parsed structs depend on each other because that would require BufferedDataTypeManager to resolve them to each other which it does not do because it is always empty

I guess the way in which i implemented BufferedDataTypeManager breaks the parent class StandAloneDataTypeManager somewhat, but im not sure why.

Actually what i said earlier about the parser adding/overwriting datatypes on parsing is probably not true.  CParser literally has methods that allow getting all parsed DataTypes without having to use the storeDataType flag at all, which means that dependency resolution also works. Should now be properly implemented.