[General] Add Solution for Detecting a Loop in a Linked List

📋 General Issue or Question
Issue Description
This issue proposes adding a solution to detect a loop in a linked list using Floyd's Cycle-Finding Algorithm (Tortoise and Hare algorithm). This is a common problem in data structures, and providing a clear, efficient implementation would be beneficial for users.

Would you like to work on this issue?

[x] Yes
[ ] No

#570  i have already done it can you change it my label to level 2


Does not support cn.bing.com

Microsoft Bing has a Chinese version website cn.bing.com, but the Display Banner feature does not support this website.

Thanks for reporting this! I'll see if I can get IWB working for cn.bing.com in the next update.

Indie Wiki Buddy should now work on cn.bing.com as of v3.2.0. Please let me know if you run into any issues. Thank you!

Thanks, it works now


Fix EnqueueAction randomly throwing if called from (async) disposal thread

This may not fix every incorrect case, but will fix the disposal path (as seen here).
Probably "good enough" for the time being?

It's not. In single threaded move it's feasible for the disposal to run on the same managed thread as the other four threads. Because the thread static bools only reset at the start of the next execution, it's stuck in IsDrawThread at that point in execution.
Maybe we want to reset these variables immediately?

But they're thread static. Whatever state they're run stuck in is irrelevant, unless that same threadrunner thread is somehow running disposals at the end of the frame/in the next input frame?

Precisely that, yeah. It's the input thread and we yield it away. I'm pretty sure in such a case the thread statics will be read on the disposal thread?


Update num tokens from text

Why are these changes needed?
Address #63, added a parameter for injecting customized token count function to support accurate token count for non-OpenAI models.
Related issue number
Closes #63, #90
Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[x] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report

Merging #149 (f72992f) into main (5ff85a3) will decrease coverage by 2.03%.
The diff coverage is 70.58%.

@@            Coverage Diff             @@
##             main     #149      +/-   ##
==========================================
- Coverage   41.03%   39.01%   -2.03%
==========================================
  Files          17       17
  Lines        2091     2084       -7
  Branches      469      466       -3
==========================================
- Hits          858      813      -45
- Misses       1156     1191      +35
- Partials       77       80       +3

Flag
Coverage Δ

unittests
39.01% <70.58%> (-1.93%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

...gen/agentchat/contrib/retrieve_user_proxy_agent.py
4.39% <0.00%> (-0.03%)
:arrow_down:

autogen/retrieve_utils.py
68.75% <80.00%> (+5.05%)
:arrow_up:

... and 2 files with indirect coverage changes


Write more specific error messages for failing to read files

eg. IsADirectory

We can write more of these, but not all of them. Some of these, like IsADirectory, are unstable APIs.
Once the API https://github.com/rust-lang/rust/issues/86442 becomes stable, we can add the rest.


Follow #605 BitMapScanner clear bmFooterFiber status to enable re-entry

What changes were proposed in this pull request?
When test the combing-index like select xx from table where a = xxx and b = xxx , there's a problem like #605( #604 ), In this sql a use btIndex and b use bmIndex, if sql like  select xx from table where b = xxx , there's no problem.The reasons for the above problems are as follows:

In our scene, bitmap index alway not trigger skipIndex read Behavior, so it will always invoke scanner.initialize method clear BitMapScanner status even if there more than 1 file in 1 task

But when use combing-index, some times btIndex trigger skipIndex read Behavior and bitmapscanner no chance to invoke scanner.initialize method, if there 2 files in 1 task, the 1st file skip index but not clear BitMapScanner bmFooterFiber status, the 2nd file do bitmap analyzeStatistics will use previous bmFooterFiber to wrap bmFooterCache, then there may occur read eof problem.

So only change of this pr is clear BitMapScanner. bmFooterFiber status after do analyzeStatistics to fix  above problems .
How was this patch tested?
mvn test pass

@adrian-wang this pr always need merge to v0.3

@adrian-wang thx~


Overview dataset / tutorial

Description:

what is windower wrapping?
what is the purpose of this class? what does it produce?
what is the input to windower class, moabb dataset or anything?

We have two examples in https://github.com/braindecode/braindecode/blob/49b770ef5226c64c28a1bdea6c6dcfde2567a736/examples/plot_custom_dataset_example.py and https://github.com/braindecode/braindecode/blob/master/examples/plot_dataset_example.py

This would be a tutorial for the doc that would explain a bit more also the internal structure of datasets


[CPO] Fix cinder-csi-plugin related jobs

The CI for old branches is broken(https://github.com/kubernetes/cloud-provider-openstack/pull/1029).
Currently, all CPO jobs are supposed to be running against all branches (except the periodic jobs for different branches), this PR:

Use image-csi-plugin target in Makefile, which is supported in all branches.
Use correct image tag for corresponding branches, e.g. release-1.18 is using k8scloudprovider/cinder-csi-plugin:v1.18.0 docker image rather than k8scloudprovider/cinder-csi-plugin:latest

@ramineni @wangxiyuan PR is ready for review.


Using DeviseTokenAuth::Concerns::User breaks Devise::confirmable and Devise::reconfirmable

Culprits being :
    # don't use default devise email validation
    def email_required?
      false
    end

    def email_changed?
      false
    end

    def will_save_change_to_email?
      false
    end

why is that

Hi @Marinlemaignan , I haven't been here since day 1, but when going through the repo I did find a few references to confirmableand reconfirmable being broken.
We'd welcome your pull request if this is truly an issue!

This breaks this part of the reconfirmable module:
        if Devise.activerecord51?
          def postpone_email_change?
            postpone = self.class.reconfirmable &&
              will_save_change_to_email? &&    ######### HERE
              !@bypass_confirmation_postpone &&
              self.email.present? &&
              (!@skip_reconfirmation_in_callback || !self.email_in_database.nil?)
            @bypass_confirmation_postpone = false
            postpone
          end
        else
          def postpone_email_change?
            postpone = self.class.reconfirmable &&
              email_changed? && ######### HERE
              !@bypass_confirmation_postpone &&
              self.email.present? &&
              (!@skip_reconfirmation_in_callback || !self.email_was.nil?)
            @bypass_confirmation_postpone = false
            postpone
          end
        end

I'll try something out and will keep you updated as i can

In the meantime this issue should be referenced in the wiki, so that no one will loose time figuring out something that is already known

Sure we currently have nothing in the Wiki most is in the README (I know, it's really hard to read but that's where we're at) - maybe a quick PR into there!

Appreciate your possible contribution @Marinlemaignan !

Not sure if I'm missing something, but this is still an issue, right? So the workaround for now is to reimplement this methods and return provider == 'email' instead of false, right? Creating a PR for this is something you would still be interested? @lynndylanhurley @Marinlemaignan @zachfeldman

Just to provide an update if it helps anyone.
Just adding this method on the mounted model (eg: User) fixes the problem
# On ActiveRecord 5.1 and newer
def will_save_change_to_email?
  super
end

# On ActiveRecord < 5.1
def email_changed?
  super
end

You just need one of those depending on your AR version. I tested that it doesn't break any email authentication, registration, password recovery end points. I'm not using Omniauth so it might break something there as that is precisely why making this method always return false was introduced.
Just make sure this method definition goes below this line
include DeviseTokenAuth::Concerns::User
you will also need the default confirm_success_url on your initializer
# config/initializers/devise_token_auth.rb
DeviseTokenAuth.setup do |config|
  config.default_confirm_success_url = 'https://google.com'
end

@mcelicalderon I've tried your method but apparently breaks the update on the email field (put '/api/v1/auth/' params: {email: ....})
Any solution?
Thanks.


blob get fails and locks up SDK

The Issue
While this is running, the sdk fails to shutdown on first attempt.
C:\Users\thoma\Desktop>lbrynet blob get 17aa0c77c404bed99c84c35b1d3621244a611ef9bb522b8db7f3bacaf64c54098b5fe4c1ccabaf656cf843f916fc5e8a --timeout=5
Traceback (most recent call last):
  File "lbrynet\extras\cli.py", line 322, in <module>
  File "lbrynet\extras\cli.py", line 310, in main
  File "asyncio\base_events.py", line 573, in run_until_complete
  File "lbrynet\extras\cli.py", line 32, in execute_command
  File "site-packages\aiohttp\client.py", line 1005, in __aenter__
  File "site-packages\aiohttp\client.py", line 497, in _request
  File "site-packages\aiohttp\client_reqrep.py", line 857, in start
  File "site-packages\aiohttp\helpers.py", line 585, in __exit__
concurrent.futures._base.TimeoutError
[57460] Failed to execute script cli

System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Internal Use
Acceptance Criteria

Definition of Done

[ ]  Tested against acceptance criteria
[ ] Tested against the assumptions of user story
[ ] The project builds without errors
[ ] Unit tests are written and passing
[ ] Tests on devices/browsers listed in the issue have passed
[ ] QA performed & issues resolved
[ ] Refactoring completed
[ ] Any configuration or build changes documented
[ ] Documentation updated
[ ] Peer Code Review performed

Hi! I just (forked) cloned lbry-sdk commit be544d6d899d2fdcfdf5e91bd091f97656de5809 and started locally in debug mode.
I can confirm, the bug is still present.
When I try to download something (eg using get) everything works as expected. When I am trying to download a blob using its blob_hash and blob_get method, the daemon outputs lbry.dht.protocol.iterative_find:379: blob peer search bottomed out
So, @lyoshenka  I would kindly suggest opening the bug again.
@belikor are you actively working on this bug now? I will start a bit of investigation on my own, but please stop me if you found a solution/are close/etc.

I think I got it but the problem is a bit trickier. Found the actual cause (One has to load some default peers into the DHT node before starting the actual blob download) but the root cause is bigger. Some larger code shifting must be performed, that is a bit beyond my knowledge about lbry-sdk.
There are two paths towards BlobDownloader.download_blob() The right one, that works, is from StreamDownloader that loads some fixed peers into the node before triggering the download. The "wrong" one, that timeouts, is from `blob_exchange.downloader.py.download_blob() that does not do this peer loading step.
Loading some peers (from config object) into the node, solves the problem.
I am going to clean up a bit the code and make a PR. I would recommend NOT to be accepted as is but suggest some refactoring to proper prime the node for download both from StreamDownloader and other "random" places (like downloader.py.download_blob()) After all, the know how is in the config object that is available.
Maybe a static method in the Node class? (There are some processing needed from Config.fixed_peers to KademliaPeer objects)

@belikor are you actively working on this bug now? I will start a bit of investigation on my own, but please stop me if you found a solution/are close/etc.

I am not working on this at the moment, so feel free to propose your own solution.
I noticed that the script located in scripts/download_blob_from_peer.py is precisely able to download an arbitrary blob (which is then immediately removed). So my intention was to compare the code in this script with the code in daemon.jsonrpc_blob_get, and try to figure out what needs to be fixed in the latter, but I've been busy with other things, so no progress on this specific issue from my side.


Add wlr_input_device_is_libinput

And corresponding functions for wlr_output, etc

The idea is to expose the actual libinput_device to library users so they can do the configuration on the device themselves.
Users can use wlr_libinput_get_device_handle() to get the libinput_device but this will segfault if the device is not a libinput device, hence the new function.
The requirements for libinput device configuration can be seen at the sway-input man page. Each of these configuration options has a corresponding function in libinput.h to set the configuration.
Examples: libinput_device_config_accel_set_profile(), libinput_device_config_click_set_method(), libinput_device_config_tap_set_drag_lock_enabled(), need to be called on the raw device.
Rootston should take a similar approach in the [device:<name>] block.
Example:
[device:mouse1]
accel_profile=adaptive
drag_lock=disabled
click_method=button_area

@martinetd make sure to mention any kind of configuration you want to add to this.

I'd rather keep both issues separate, as I'd do this in two PR (especially since the original comment from @SirCmpwn asks for corresponding wlr_output/etc functions which have nothing to do with libinput configuration)
That aside the plan looks good to me and corresponds to the idea I had, I'll check which configurations I depend on exactly tonight as well.
What about what I said about only doing a few arbitrary settings for rootston just to show how it should work (that'll save effort on config parsing boilerplate/adding new enums mostly)? Do you agree there or do we want to be as feature complete as possible?

Yeah just do whichever ones you want to do. They'll all be pretty similar.


Adding metrics endpoint for interoperator pod

Pull Request Test Coverage Report for Build 3281

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.007%) to 89.82%

Totals

Change from base Build 3278:
-0.007%

Covered Lines:
9371

Relevant Lines:
10082

💛  - Coveralls

@amshuman-kr Can you please take a glance at this ?

thanks @amshuman-kr 👍


Remove unnecessary apt update

apt_repository should update apt package cache by itself
Closes #406

I just tried 6efd18f locally and it failed. It looks like FC023 is deprecated now so adding ~FC023 to .foodcritic would be fine instead. Also test in kitchen - looks good.

I haven't reach to kitchen yet, but I've understood that FC023 should be turned off. Attempt to comply to it doesn't work.

NP, I took care of the integration testing, saw it do the apt-get update when the new ppa was added. Thanks for the contribution and bearing with me!

@erichelgeson, thanks to you for cooperation!


Syntax for .ftpignore

What's the syntax for .ftpignore, can't find anything in the docs. Thank you.

Check readme https://github.com/icetee/remote-ftp/commit/aac1d8437671d86c8b01a6cd7244cabacf4162c8 ;)
Related #714


BUG: Render texture is cropped if draw after a bit of time [code example & reproducible]

Example: https://codepen.io/anon/pen/RYwpBq?editors=0010

Note changing the width and height of the game config will increase this area.
Even this hack will kind of fix it, but the width and height has to match the render texture width and height, otherwise the clipped area increases just a bit (so strange):
 // scene.game.config.width = 15000;
 // scene.game.config.height = 15000;

Note: Something very strange happens. So basically I was playing with it and was working outside the cropped area. Now is not working anymore, no code change. I wonder if is a chrome bug or memory related issue?
The thing is that if you draw something in the create method, or in the update method (here only works once) it renders outside the crop area like this:

In the update method renders once outside the render method:

Code:
  var config = {
      type: Phaser.AUTO,
      parent: 'phaser-example',
      width: 800,
      height: 600,
      scene: {
          preload: preload,
          create: create,
          update: update
      }
  };

  var rt1;
  var rt2;

  var game = new Phaser.Game(config);

  function preload ()
  {
      this.load.image('bunny', 'https://labs.phaser.io/assets/sprites/bunny.png');
      this.load.image('grid', 'https://labs.phaser.io/assets/sprites/128x128-v2.png');
  }

  function create ()
  {
      this.bunny = this.textures.get('bunny').get();
      //this.gridImage = this.add.image(0, 0, 'grid').setVisible(false).setOrigin(0);

      rt1 = this.add.renderTexture(0, 0, 1024, 1024);
      //rt1.draw(this.bunny, 0, 0);
      //rt1.draw(this.gridImage, 128, 0);
      //rt1.draw(this.gridFrame, 256, 0);

      rt1.draw(this.bunny, 0,0);
      rt1.draw(this.bunny, 555,555);
      this.cameras.main.zoom =0.5

      // rt2 = this.add.renderTexture(200, 200, 800, 600);
      // rt2.draw(bob, 200, 200);
      // rt2.draw(rt2, 0, 0);
  }
  let i = -100;
  function update ()
  {
      rt1.draw(this.bunny, 666 + i, 666+ i);
     // this.cameras.main.scrollX++;
      rt1.draw(this.bunny, i++, i++);
      //rt.camera.rotation -= 0.01;
      // rt.clear();
       //rt.draw(graphics, 0, 0);
  }

Another example (remove the setTimeout and notice how it renders outside of the crop area)
  var config = {
      type: Phaser.AUTO,
      parent: 'phaser-example',
      width: 800,
      height: 600,
      backgroundColor: 0x2d2d2d,
      scene: {
          preload: preload,
          create: create,
          update: update
      }
  };

  var rt;
  var graphics;
  var offset;

  var game = new Phaser.Game(config);

  function preload ()
  {

            this.load.image('bunny', 'https://labs.phaser.io/assets/sprites/bunny.png');
      this.load.atlas('megaset', 'assets/atlas/megaset-0.png', 'assets/atlas/megaset-0.json');
  }

  function create ()
  {
            this.bunny = this.textures.get('bunny').get();
      rt = this.add.renderTexture(0, 0, 800, 800);
      this.cameras.main.zoom = 0.5

      var atlasTexture = this.textures.get('megaset');

      var frames = atlasTexture.getFrameNames();

  setTimeout(()=>{

      for (var i = 0; i < frames.length; i++)
      {
          var x = Phaser.Math.Between(0, 800);
          var y = Phaser.Math.Between(0, 600);

          rt.drawFrame('megaset', frames[i], x, y);
          rt.draw(this.bunny, x, y)
      }
  }, 0)

      graphics = this.add.graphics();

      var cropWidth = 290;
      var cropHeight = 120;

      offset = rt.getTopLeft();

      this.input.on('pointermove2', function (pointer) {

          rt.setCrop(
              (pointer.x - offset.x) - cropWidth / 2,
              (pointer.y - offset.y) - cropHeight / 2,
              cropWidth,
              cropHeight
          );
      });
  }

  function update ()
  {
      //graphics.clear();
      //graphics.lineStyle(1, 0x00ff00);
      //graphics.strokeRect(offset.x + rt._crop.x, offset.y + rt._crop.y, rt._crop.width, rt._crop.height);
  }

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Internally a Render Texture creates a Frame Buffer Object. FBO maximum sizes are GPU dependent, so it'll always 'crop' a Render Texture to this max size, regardless.
https://webglstats.com/webgl/parameter/MAX_RENDERBUFFER_SIZE
It's probably worth forcing RTs to limit themselves to what the GPU can handle, so if a value is given in code beyond this it just resets the width / height of them to force them to be the right allowed size.
I'll check the examples tomorrow to see if there is something else going on, but it's probably related to this somehow.

But how do you explain that if you change the game.config.width it allows bigger render textures?

Looked at this today, some more findings:

Only happens in WebGL. Canvas is fine.

The trigger is creating a Render Texture that is larger than the Canvas it is being drawn to. An RT of the same size, or less, works as expected.

Creating an FBO beyond what the GPU can support throws all kinds of fun errors (from both the Camera FBO and the RT), which is to be expected really, but needs catching.

@photonstorm

The time of the draw appears to be irrelevant and not the trigger.

What do you mean? I wonder because if you render a few 100s of times in the update method it will render only a few of them outside the crop area. So I assume for a few milliseconds the crop is not applied, only after that.
If you have a update method and there you draw 1000 times per update, you will see what I mean.

Hi! I was wondering when do you think this will be fixed?

Hi @photonstorm is this still being worked on are there any news?

Hi! I was wondering if you have any news about this @photonstorm thank you

Hello @photonstorm is there anything I can do to help on this issue? It's really blocking my game at the moment.

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.
You can find a demo here: http://labs.phaser.io/view.html?src=src\bugs\rt crop.js&v=dev
Use the arrows to move the camera and Q/E to zoom in/out.
The Render Texture is 1024 x 1024, larger than the game canvas. The Phaser logo is being drawn randomly in the range -3000 to 3000, so well outside the size of the Render Texture. It no longer draws anything outside of the RTs size.

Thank you


remote-complete issue - results are not set if term does not perfectly match part of the choice

When using remote complete on choice or choices, with either reference or reference_many, the behavior is such that the returned values has to match a Contains predicate in order to work, otherwise the results do not show up, this is due to the | filter in the directive's template.
the | filter is needed for local filtering of options, but when using a remote call, the remote server is in charge of filtering, so local filtering gets in the way
A sample scenario: when adding a reference to something with a title, the backend might treat incoming request (like ang adm) as prefix terms, returning {1, "Angular Admin" } as the result. Since "Angular Admin").indexOf('ang adm') < 0, the value that was returned would not show up on the dropdown, and would not be selectable.
A proposed solution (in an upcoming PR) removes the filter in case of a remote-complete instance of the field.

oh man the CI server refuses to build again (UnknownError: unknown error: Chrome failed to start: crashed)


Huge performance concern for directives

So I was poking around the code for unrelated reasons and came across this line: https://github.com/luisfarzati/angulartics/blob/master/src/angulartics.js#L260
You are essentially adding watchers for every single property of an element that has analytics. This is an insane amount of watchers and is completely unnecessary until the event tracking trigger is actually initiated. You need to change this to be evaluations that perform at event-track time and not as constant watchers that update data that may very well never get used (if the tracking event never even fires).

Yes, that's a very good point. It seems this was introduced here: https://github.com/luisfarzati/angulartics/commit/6a925390758c5d6e5e013dfa60d5a76e6446f58f. @pcwinters do you remember if there was any specific reason to watch this all the time? Otherwise I guess it's safe to assume evaluation only at event-track time, as @ProLoser suggests.

Sorry for the delay, @luisfarzati. I'd like to believe it was intentional, but it's been over a year. The commit sure seems intentional. At the time I was working on a rather large and deep angular app, and I believe there was an issue with analytics properties as expressions.
I'd need an angular refresher, but maybe this was the only issue?
properties[name.slice(9).toLowerCase()] = $attrs[name];


Use node: protocol when importing builtin modules

Refactor
Description
Update all the code to use node: imports for all builtin modules (e.g. fs to node:fs or path node:path). For a motivation, see the (linked) documentation.
Considering the version history of node: imports (taken from the (linked) documentation):

Version
Changes

v14.18.0, v16.0.0
Added node: import support to require(...)

v14.13.1, v12.20.0
Added in: v14.13.1, v12.20.0

Partial support (i.e. only with import x from "y") for node: imports was added in later v12 (12.20.0 and up) and v14 (14.13.1 and up) releases and full support is only present since later v14 (14.18.0 and up) releases and all releases since v16 (16.0.0 and up). This means package code should only adopt node: imports once their engines.node value (in packages/[package]/package.json) is such that consumers can be "guaranteed" to support it.


Add feedback on "You cannot alias an alias" error message to say what the original command was

and prompt for aliasing to the source
https://discord.com/channels/746131494875168770/746551007567413258/848026589991796796


bug: crash on some small contract

Component
Heimdall (Core)
Have you ensured that you are up to date?

[ ] Bifrost
[X] Heimdall

What version of Heimdall are you on?
8.4
Operating System
Linux
Describe the bug
Seems like some proxy  or something. Reporting, since it's a long time since I've seen crash:
( https://etherscan.io/address/0xb8d5c36e1bd74e9dd1dbdbe938884f538c73a3a8#code ) :

./heimdall-rs/target/release/heimdall decompile 0xb8d5c36e1bd74e9dd1dbdbe938884f538c73a3a8 -r http://localhost:8545 -vvvv
2024-11-17T21:50:50.744633Z TRACE heimdall_common::utils::http: GET https://api.github.com/repos/Jon-Becker/heimdall-rs/releases/latest
2024-11-17T21:50:51.037982Z TRACE heimdall_common::utils::http: GET https://api.github.com/repos/Jon-Becker/heimdall-rs/releases/latest: Response { url: "https://api.github.com/repos/Jon-Becker/heimdall-rs/releases/latest", status: 200, headers: {"date": "Sun, 17 Nov 2024 21:50:50 GMT", "content-type": "application/json; charset=utf-8", "cache-control": "public, max-age=60, s-maxage=60", "vary": "Accept,Accept-Encoding, Accept, X-Requested-With", "etag": "W/\"...\"", "last-modified": "Mon, 26 Aug 2024 20:25:02 GMT", "x-github-media-type": "github.v3; format=json", "x-github-api-version-selected": "2022-11-28", "access-control-expose-headers": "ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset", "access-control-allow-origin": "*", "strict-transport-security": "max-age=31536000; includeSubdomains; preload", "x-frame-options": "deny", "x-content-type-options": "nosniff", "x-xss-protection": "0", "referrer-policy": "origin-when-cross-origin, strict-origin-when-cross-origin", "content-security-policy": "default-src 'none'", "server": "github.com", "x-ratelimit-limit": "60", "x-ratelimit-remaining": "56", "x-ratelimit-reset": "1731883716", "x-ratelimit-resource": "core", "x-ratelimit-used": "4", "accept-ranges": "bytes", "content-length": "6424", "x-github-request-id": "..."} }
2024-11-17T21:50:51.038648Z DEBUG heimdall_cache: cache hit for key: 'chain_id.http-localhost-8545'
2024-11-17T21:50:51.038670Z DEBUG heimdall_cache: cache hit for key: 'contract.1.0xb8d5C36E1BD74E9dd1DBdBe938884F538c73A3A8'
2024-11-17T21:50:51.038676Z DEBUG heimdall_decompiler::core: fetching target bytecode took 65.183µs
2024-11-17T21:50:51.038682Z DEBUG heimdall_common::ether::compiler: detected compiler unknown unknown.
2024-11-17T21:50:51.038687Z  WARN heimdall_common::ether::compiler: unknown is not fully supported by heimdall
2024-11-17T21:50:51.038714Z DEBUG heimdall_disassembler::core: fetching target bytecode took 2.35µs
2024-11-17T21:50:51.038727Z DEBUG heimdall_disassembler::core: disassembly took 7.981µs
2024-11-17T21:50:51.038730Z  INFO heimdall_disassembler::core: disassembled 49 bytes successfully
2024-11-17T21:50:51.038733Z DEBUG heimdall_disassembler::core: disassembly took 20.771µs
2024-11-17T21:50:51.038744Z TRACE heimdall_vm::ext::selectors: optimistically assuming instruction 000004 PUSH4 57e871e7 is a function selector
thread 'main' panicked at crates/vm/src/core/vm.rs:1161:39:
index out of bounds: the len is 49 but the index is 49
stack backtrace:
   0: rust_begin_unwind
   1: core::panicking::panic_fmt
   2: core::panicking::panic_bounds_check
   3: heimdall_vm::core::vm::VM::_step
   4: heimdall_vm::core::vm::VM::step
   5: heimdall_decompiler::core::decompile_impl::{{closure}}
   6: heimdall::main::{{closure}}
   7: heimdall::main

looking into it! thanks for opening


Not an issue: Can you do this for the "this is fine" meme?

I'm not sure of the best way to make a suggestion for your next project, but I would love to see a similar Tidbyt app for the "this is fine" meme: https://giphy.com/gifs/perfect-loops-kspVl6FzbdblOMKRmM. I would have written it myself, but then I saw your work using Pixilart to make the frames and got intimidated :)

a suggestion for your next project

This is fine (pun intended).
I'll add it to my TODO list, but no promises it gets done soon, if ever.

@mickeyboxell, as it turns out, you can just use the existing photo app by Tidbyt to display GIFs.
I saved the GIF you linked above to my camera roll and then selected it from within the Tidbyt app. Here's what it looks like:


Unable to parse Seattletimes.com

Greetings, I've noticed newspaper failing to parse a link from Seattletimes.com. How can I dive a bit deeper to find out what's wrong?
urls:
https://www.seattletimes.com/seattle-news/politics/the-irony-of-the-no-cop-chop-it-showed-how-much-we-still-need-the-police-after-all/
https://www.seattletimes.com/seattle-news/health/coronavirus-daily-news-updates-june-24-what-to-know-today-about-covid-19-in-the-seattle-area-washington-state-and-the-world/
https://www.seattletimes.com/entertainment/visual-arts/seattle-art-museum-isnt-dissolving-despite-a-fake-news-release-with-a-point-to-make-imagining-so/

good find, thanks for filing this. we will take a look

great, thanks for your work, much appreciated!
I also found forbes links not working:
https://www.forbes.com/sites/jackbrewster/2020/06/27/trump-most-republicans-silent-about-reports-russia-paid-taliban-to-kill-us-troops/#7acb8c7c6a45
https://www.forbes.com/sites/adrianbridgwater/2020/06/17/world-economic-forum-data-flows-project-hopes-to-globalize-information-exchange/?ss=big-data#523c3904745f

This may be due to user agent. Try this, it should work
url = 'https://www.seattletimes.com/seattle-news/politics/the-irony-of-the-no-cop-chop-it-showed-how-much-we-still-need-the-police-after-all/'
article = Article(url, browser_user_agent='Mozilla')
article.download()
article.parse()

Another:
https://www.nasaspaceflight.com/2020/06/starship-sn5-test-campaign/

this one doesn't pick up "top image"

There are several sites that seem to have this issue. I noticed this when trying to parse NYTimes, https://github.com/codelucas/newspaper/issues/645. From what I found, there is a decision to figure out where to start parsing an article and sometimes it doesn't get it right. In doing so leads to getting partial or no content from the page.

@sagunsh thanks that solved it

@sagunsh thanks that solved it


configstore/gateway: move projectgroups/projects between users/orgs

For moving the project we can user the api PUT /projects/{projectref} that already exists es. PUT https://agoladev.sorintdev.it/api/v1alpha/projects/org%2FTest%2Fproject1 body: {"parent_ref": "user/username1/userprojectgroup"}

Since this can be a move between different with the actual datamanager and related locking implementation it'll be quite cumbersome to properly handle locking. So let's wait when datamanager will be removed.

For getting the list of projectgroups I would like to create this api GET /user/projectgroups?grouptype
es. /user/projectgroups?grouptype=user%2Corg
The parameter grouptype is an array, we can use "user" and/or "org" value.
With user value the api return the list of user projectgroups recursively
With org value the api return the list of all organization projectgroups where the user is owner
The default value for grouptype empty is "user"

The user is already able to navigate between projectgroups/projects. I don't see the need for this new api that will require e recursive fetching.


ci: fix file comparison check

Fixes #10

CI logging remains a mystery but it's not that important anyway so this can be closed.


Merge Nx branch to master.

There will be many breaking changes in this pull request. Major file structure updates along with new monorepo approach. Everything on the Nx branch should be accepted over the  master branch, as master has not been truly updated in a long time.
This change will bring commitlint, Nx, Docker, the ConfigModule, the DatabaseModule, and GraphQL to the tabletop-companion (originally Zeldaplay) project; however, it will also break the front end as the code still needs to be re-written to follow better practices.

Closes #332 and #310


#15: Remove dependency on Avalon Framework

Avalon Framework is now retired so any dependency on it should be removed

I was curious how you were able to remove the avalon framework. So I checked the code and basically you copied the used classes from the avalon framework to the barcode4j library:

/**

This class is essentially a copy of org.apache.avalon.framework.configuration.ConfigurationException
*/

Doesn't this beats the purpose and the avalon framework better remains like it is, to reflect that the framework is used?
In case some security issue are found in the framework, probably nobody remembers that there are still vulnerable copies in the barcode lib...


Add Python (and other core dependency) support policy to our docs

IMO we should follow the Numpy policy and see how that works for us.
Follow up from discussion in #571
### Tasks
- [x] Adopt a policy by consensus. Scientific Python SPEC0?
- [x] Add Python support policy to docs (e.g. in "User Guide > Policies"?)
- [ ] Update project dependencies to follow new policy
- [ ] #471

@mfisher87 Hey, Is this the policy that you are talking about https://jax.readthedocs.io/en/latest/deprecation.html. If yes then how should we implement this in our context?

Numpy's Python version support policy is:
https://numpy.org/neps/nep-0029-deprecation_policy.html
Similarly, the scientific python policy is:
https://scientific-python.org/specs/spec-0000/

I think we're not decided yet on what policy we want to use. I personally think Scientific Python SPEC0 (as referenced by your link above, and based on NEP29) is where we want to be.

Support for Python versions be dropped 3 years after their initial release.
Support for core package dependencies be dropped 2 years after their initial release.

This means we can drop support for Python 3.9 now.

@'ing some folks I think may have opinions :) @betolink @andypbarrett @JessicaS11 @chuckwondo @asteiker @danielfromearth

I added a task list to the original post of this ticket. I think if whoever takes this up wants to break off the piece about updating our deps to another ticket, please do :)

Fairly positive emoji responses to request for consensus on SPEC0, maybe we can decide on the hackathon call rq. :)

I think we can call that complete :) I wonder about number 3, are our dependencies already in compliance or do we need to make changes?

This was completed https://earthaccess.readthedocs.io/en/latest/user_guide/backwards-compatibility/#our-python-and-dependency-support-policy
#763


Dds_v5 Max Cached Disk Throughput chart is missing

Dds_v5 disk throughput chart is missing the entire Cached disk performance throughput numbers.
Currently the chart is only showing values for
"Max uncached disk throughput: IOPS/MBps"
"Max burst uncached disk throughput: IOPS/MBps"
However, these VMs support Disk Caching but the performance values are not displayed in the performance chart.
These two values are missing
VM Limit - Max Cached IOPS	308000
VM Limit - Max Cached MBps	2000
These values are already listed
VM Limit - Uncached Disk IOPS	51200
VM Limit - Uncached Disk MBps	865

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

ID: b040274d-e9e8-5ede-7ffe-2858dfeced77
Version Independent ID: 681ed815-340e-0e50-2634-a7b29e3604a3
Content: Ddv5 and Ddsv5-series - Azure Virtual Machines
Content Source: articles/virtual-machines/ddv5-ddsv5-series.md
Service: virtual-machines
Sub-service: sizes
GitHub Login: @andysports8
Microsoft Alias: shuji

Cache disk burst performance is also missing.

@phyoungmsft
Thanks for your feedback! We will investigate and update as appropriate.

@phyoungmsft
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Thanks for your contribution to our documentation
We are moving to a new user feedback system. As part of that effort, we are creating a content request in our backlog to address this gap and are closing this GitHub issue. The updated content will be published shortly.

Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close

@phyoungmsft did you ever get this resolved? It looks like cached limits are missing in the documentation for every VM SKU.


Outputing Hebrew Chars

hello i try to use your amazing platform:
i write the simple module:
   Get["/hebrew"] = x =>
   {
           return "עברית";
   };

but unfortunately i was getting as output: ×¢×‘×¨×™×ª
i was sure it's an encoding issue so i download the source and debug and found this:
in the Nancy.Response line 103 at the GetStringContents if i make the following modification it's solve the issuse:
before:
       protected static Action<Stream> GetStringContents(string contents)
    {
        return stream =>
        {
            var writer = new StreamWriter(stream) { AutoFlush = true };

            writer.Write(contents);
        };
    }

after:
    protected static Action<Stream> GetStringContents(string contents)
    {
        return stream =>
        {
            var writer = new StreamWriter(stream, System.Text.Encoding.UTF8) { AutoFlush = true };

            writer.Write(contents);
        };
    }

just adding the System.Text.Encoding.UTF8 to the StreamWriter constructor solve it.
i also want to notice that routes with Hebrew chars are don't work:
example:
     Get["/עברית"] = x =>
     {
             return "hebrew route";
     };

don't get fire thanks!

I think the proper solution to this is #2020


application crash after switching view from terminal

If you open an SSH connection to an SFTP Server and then navigate to the terminal view, the app crashes after selecting any other view.

So due to the server settings you had no access to the terminal?

Exactly. It was a user account restricted to only use the sftp server.

fixed in fbf3c41


Problem with scanning qr codes

Problem
We actively using codemagic-preview-app for organizing continuous testing process, but users are experiencing difficulties in scanning the QR code generated by the Codemagic Preview app due to its low resolution (200 pixels), it getting worse when we working in extreme conditions (night mode, sunshine, etc) This issue affects usability and can hinder efficient workflow.
Proposed Solution:
Implement a feature to make the QR code's resolution configurable, allowing users to adjust it based on their needs. Additionally, increase the default resolution of the QR code to ensure better scanability across various devices.
Expected Outcome:
By making the QR code resolution adjustable and enhancing its default quality, users will have an easier and more reliable experience when scanning QR codes, leading to improved efficiency in app development and testing processes.

codemagic by default generating 380x380 pixels, checked their panel

Hey @kcrebound, thanks for the suggestion! I added a --qr-code-size option which allows you to set the width and height of the qr code. I also increased the default from 150 to 200.


Tiny issue with documentation

Hi,
Tiny issue with documentation. In your sample config, module.exports.keyfile should be module.exports.keyFile
Thanks! :)

Hi there,
I've made the change to the readme 👍
Thanks for pointing that out!
Cheers :D


How to make zone.js not to patch some async tasks like settimeout

Hi Folks,
We have a scenario where we are continuously polling data from server in frequent calls of setTimeout.
We don't want the change detection to run after each setTimeout.
I know we can run setTimeout outsideAngularZone. But we have setTimeout at multiple places in our code. So putting each setTimeout in outsideAngularZone will be time taking process.
So what is the best way to tell Zone.js not to patch some async tasks.
If there is way to set the blacklist required async tasks something like following.
Zone.setBlacklist('setTimeout','requestAnimationFrame', 'XHR')
Please suggest your way to do this kind of stuff
Reason why we need it because continuous calls of setTimeout and XHR gives sudden memory spikes in IE10 after a long run.
But when we remove the __load_patch calls for timers, requestAnimationFrame and XHR in Zone.js, there are no memory spikes at all.

here is the document, https://github.com/angular/zone.js/blob/master/MODULE.md
Just add code before import 'zone.js/dist/zone'; in polyfills.ts
(window as any).__Zone_disable_timers = true;
(window as any).__Zone_disable_requestAnimationFrame = true;
import 'zone.js/dist/zone';

Thanks @JiaLiPassion  for quick response.
I hope for XHR(window as any).__Zone_disable_XHR = true; will work.
and this won't trigger changeDetection at all after XHR completion?

Yes, it should work, but I only disable this XHR may not enough, because I remember Angular also use xhr.onload, so you may also need to disable onload
    (window as any).__Zone_ignore_on_properties = [
      {
        target: XMLHttpRequest.prototype,
        ignoreProperties: ['load']
      }
    ];

what about onReadyStateChange?

Yeah, you may need to disable that too.

HI @JiaLiPassion
Polyfills.ts
(window as any).__Zone_disable_timers = true;
import 'zone.js/dist/zone';

App.Component.ts

class AppComponent implements OnInit {
  value = 0;
  ngOnInit() {
    setInterval(() => {
      this.value = +new Date();
    }, 500);
  }
}

This is updating the value in template after each interval, that's means Angular Change detection is running after each interval.
But instead of making changes in Polyfills.ts, If I directly comment out code in Zone.js
// Zone.__load_patch('timers', function (global) {
//     var set = 'set';
//     var clear = 'clear';
//     patchTimer(global, set, clear, 'Timeout');
//     patchTimer(global, set, clear, 'Interval');
//     patchTimer(global, set, clear, 'Immediate');
// });

The value is not updated in template and angular change detection is not running.
Please let me know what am I missing?

Zone.__load_patch = function (name, fn) {
                console.log(__Zone_disable_timers);
......
}

This was giving undefined, So for the time being I have defined it in index.html
 <link rel="icon" type="image/x-icon" href="favicon.ico">
  <script>
    __Zone_disable_timers = true;
    __Zone_disable_requestAnimationFrame = true

  </script>

and this works fine


RecyclerView.ViewHolder already has its View definition

RecyclerView.ViewHolder already has its View definition (@see https://developer.android.com/reference/android/support/v7/widget/RecyclerView.ViewHolder.html#itemView)

@kimbeejay Hi thx a lot for commit! I'll get your changes to the experimental branch okay?

@clockbyte it is up to you ^^


[4.0] libnv_vulkan_wapper.so error

Godot version
v4.0.dev.20211015.official [f113dc986]
System information
Debian 11
Issue description
When start:
Godot Engine v4.0.dev.20211015.official.f113dc986 - https://godotengine.org
Can't open bumblebee display.
ERROR: GENERAL - Message Id Number: 0 | Message Id Name: Loader Message
	loader_scanned_icd_add: ICD libnv_vulkan_wrapper.so.1 doesn't support interface version compatible with loader, skip this ICD.
	Objects - 1
		Object[0] - VK_OBJECT_TYPE_INSTANCE, Handle 139067664
   at: _debug_messenger_callback (drivers/vulkan/vulkan_context.cpp:157)
MESA-INTEL: warning: Ivy Bridge Vulkan support is incomplete
Vulkan API 1.2.162 - Using Vulkan Device #0: Intel - Intel(R) HD Graphics 4000 (IVB GT2)

ps:. I use already vulkan drivers with DOTA2 (set it in video settings)
Steps to reproduce
NA
Minimal reproduction project
NA

This isn't a Godot error. This is an information message from your GPU driver. Typically this comes from having drivers configured from a device that is not connected (which can happen if you have hybrid graphics on a laptop, for example)


libaom: 2.0.2 -> 3.0.0

Motivation for this change
Things done

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

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

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

Result of nixpkgs-review pr 117466 at 5ad0f55e run on x86_64-linux 1

24 packages marked as broken and skipped:

autotrace
gimpPlugins.exposureBlend
gimpPlugins.texturize
glimpsePlugins.exposureBlend
glimpsePlugins.texturize
gnome3.gnome-books
gnome3.gnome-documents
gnunet-gtk
hhvm
invoice2data
...

557 packages skipped due to time constraints:

MIDIVisualizer
abiword
adapta-gtk-theme
almanah
apostrophe
aravis
arc-theme
areca
arx-libertatis
asc-key-to-qr-code-gif
...

2 packages built successfully:

libaom
libavif

1 suggestion:

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/development/libraries/libaom/default.nix:13:15:
   |
13 |   patches = [ ./outputs.patch ];
   |               ^


ic: fix use of possibly uninitialized variable

Found when doing OpenWrt build:
  ic/ovn-ic.c: In function 'main':
  ic/ovn-ic.c:689:5: warning: 'ls' may be used uninitialized in this function [-Wmaybe-uninitialized]
    689 |     nbrec_logical_switch_update_ports_addvalue(ls, lsp);
        |     ^
  ic/ovn-ic.c:430:40: note: 'ls' was declared here
    430 |     const struct nbrec_logical_switch *ls;
        |                                        ^

Likely a false report from the compiler.

@yousong I'm quite surprised that you don't get the same warning for other instances of *BREC_*_FOR_EACH_EQUAL, e.g.:
$ grep -rn "BREC_.*_FOR_EACH_EQUAL" ic/
ic/ovn-ic.c:432:    NBREC_LOGICAL_SWITCH_FOR_EACH_EQUAL (ls, key, ctx->nbrec_ls_by_name) {
ic/ovn-ic.c:757:    ICSBREC_PORT_BINDING_FOR_EACH_EQUAL (isb_pb, isb_pb_key,
ic/ovn-ic.c:777:        ICSBREC_PORT_BINDING_FOR_EACH_EQUAL (isb_pb, isb_pb_key,
ic/ovn-ic.c:1333:        ICSBREC_ROUTE_FOR_EACH_EQUAL (isb_route, isb_route_key,
ic/ovn-ic.c:1451:    ICSBREC_ROUTE_FOR_EACH_EQUAL (isb_route, isb_route_key,
ic/ovn-ic.c:1632:    ICSBREC_PORT_BINDING_FOR_EACH_EQUAL (isb_pb, isb_pb_key,

Most of these don't have an explicit initialization of the first argument either.

Closing due to inactivity.


How to stop VS Code Search Feature

VSCode Version: 1.12.1
OS Version: macOS 10.12.4

Steps to Reproduce:

Run a search inside VS Code

Hopefully this is super simple. It should be easy to Stop the search. How do I stop the search in VS Code? I can't find a stop button anywhere. I must be missing something here. Please advise. Much appreciated.

You can use ESC button. This is works previously.

Great. that was easy. Works perfect.

It's a design flaw if it's not obvious how to stop a search from proceeding. There should be a visual indication of how to stop a search. You shouldn't just "know".

Maybe @roblourens give more details for this issue 👍


Train road AI for waterways too

From some HOT mapping it is kinda common for me to see the AI thing confusing waterways for track roads. It is bad data, but I do appreciate something having done the job of tracing lines for me.
Maybe the AI should just be trained with a category of waterway=stream? Streams look like track roads, but they usually are curvier, and tend to come with a bunch of tell-tale plant cover to the sides when the surroundings are dry. I think the AI can tell them apart given correctly labeled data. (For human editors in RapiD, the OpenTopomap SRTM data can be checked foe a valley too. Adding the DEM to the AI would be great too, but that could be a lot of restructuring?)
From a utilitarian perspective waterways are quite important for disaster management too. They also help provide a key metric for evaluating roads.

I fully agree. I cannot imagine that this hasn't been discussed by the team in the past.

Will be beautiful such feature. However right now several water courses are recognized by Rapid as roads, it{s up to the mapper to convert them to waterways...

PLEASEEEEE!!!

this is really waited since this wame year


Automatically caching viewport

Description
Creates a new constructor for Viewport widgets to automatically calculate a caching extent based on the main axis size as provided during layout.
Updates to the relevant render object(s) to achieve this.
The plan is to use this in service of creating a PageView that can participate in a11y implicit scrolling, e.g. for a carousel widget. We could also potentially add it as a feature on page views for pre-caching the next/previous page(s) if we can fix the iOS bridge to respect hasImplicitScrolling.
Splitting this out to ease review.
Related Issues
https://github.com/flutter/flutter/pull/44645
https://github.com/flutter/flutter/issues/44484
https://github.com/flutter/flutter/pull/42107
Tests
I added the following tests:
Rendering tests for cache extent and automatic caching on RenderViewport.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

This seems to have caused a 2x regression on simple_animation_perf_iphonexs gpu_percentage. Can we revert?

I'll revert it but it seems super fishy to me :\

https://github.com/flutter/flutter/issues/45641 has more details - cause was not this PR.
Was there an issue opened for the regression?

I never got around to filing an issue. Glad to hear it wasn't caused by the PR!111111111


Revert "Storage: Refactor DMFileReader"

Reverts pingcap/tiflash#8854
Highly suspected to be related to https://github.com/pingcap/tiflash/issues/8904
Release note

None

/run-all-tests

/run-all-tests


JavaScript AST inconsistent about trivia vs JSDoc

The parsed JavaScript AST sometimes includes JSDoc in its trivia, and sometimes it doesn't.  This seems to be a regression from TS 1.9ish to current tip of tree.
If you have an input file like
/** @a */
var a = 3;
/** @b */
function b() { }

and you parse this with TypeScript ts.createSourceFile and reserialize it out via a recursive loop like:
function dump(node: ts.Node) {
  let pos = node.getFullStart();
  for (const child of node.getChildren()) {
    dumpRange(pos, child.getFullStart());
    dump(child);
    pos = child.getEnd();
  }
  dumpRange(pos, node.getEnd());

  function dumpRange(start: number, end: number) {
    process.stdout.write(node.getSourceFile().text.slice(start, end));
  }
}

The output is:
/** @a *//** @a */
var a = 3;
/** @b */
function b() { }

You'll see that one of the comments was duplicated, and the other wasn't.
I think this is because in the FunctionDeclaration case, the trivia of the FunctionDeclaration node stops at the JSDocComment, but in the VariableStatement case, the trivia of the VarKeyword still includes the jsdoc despite it also having a JSDocComment.

(If it'd be helpful, I have a small program I can upload somewhere that does the AST dumping I mentioned.)

I think the underlying cause of this is using node.getChildren() rather than ts.forEachChild.
It appears the former actually re-parses the source code and it produces bad parses in this case and in other cases (https://github.com/angular/tsickle/commit/c410541cbd102da60ca3e8be63ce632c3802f7ff )


[cppwinrt] port rewrite to use cppwint.exe tool from NuGet Gallery

The existing cppwinrt port really does nothing beyond verifying the installed Windows SDK has pre-generated cppwinrt headers. The Windows SDK headers for C++/WinRT are old, and the recommendation is to use the Microsoft.Windows.CppWinRT NuGet instead. This package contains the latest cppwinrt.exe compiler tool that is then executed using the installed Windows SDK to generate the latest C++/WinRT projections from the Windows SDK .winmd files.
This updated port now does this as well for the installed Windows SDK headers when the port is installed.

This port is not as full-featured as the NuGet package which leverages MSBuild machinery to automatically generate projections for Xaml and authored WinRT components, but this is a big step up from what was there before.

@BillyONeal @kennykerr Please review this as well.

Validated the supported triplets:
vcpkg install cppwinrt:x86-windows
vcpkg install cppwinrt:x86-windows-static
vcpkg install cppwinrt:x86-windows-static-md
vcpkg install cppwinrt:x64-windows
vcpkg install cppwinrt:x64-windows-static
vcpkg install cppwinrt:x64-windows-static-md
vcpkg install cppwinrt:arm64-windows
vcpkg install cppwinrt:arm64-windows-static
vcpkg install cppwinrt:arm64-windows-static-md
vcpkg install cppwinrt:x86-uwp
vcpkg install cppwinrt:x64-uwp
vcpkg install cppwinrt:arm-uwp
vcpkg install cppwinrt:arm64-uwp

Note I've verified consuming the headers works as expected using:
find_package(cppwinrt CONFIG REQUIRED)
target_link_libraries(<target> PRIVATE Microsoft::CppWinRT)

This results in
(a) the generated C++/WinRT headers are in the INCLUDE path before the Windows SDK
(b) the target links against cppwinrt_fast_forwarder.lib
(c) the target uses C++17 or later

Reverified all supported triplets:
vcpkg install cppwinrt:x86-windows
vcpkg install cppwinrt:x86-windows-static
vcpkg install cppwinrt:x86-windows-static-md
vcpkg install cppwinrt:x64-windows
vcpkg install cppwinrt:x64-windows-static
vcpkg install cppwinrt:x64-windows-static-md
vcpkg install cppwinrt:arm-windows
vcpkg install cppwinrt:arm64-windows
vcpkg install cppwinrt:arm64-windows-static
vcpkg install cppwinrt:arm64-windows-static-md
vcpkg install cppwinrt:x86-uwp
vcpkg install cppwinrt:x64-uwp
vcpkg install cppwinrt:arm-uwp
vcpkg install cppwinrt:arm64-uwp

No idea why the automatic builds are failing now.

After all code review, revalidated supported triplets and checked consuming of this port.
vcpkg install cppwinrt:x86-windows
vcpkg install cppwinrt:x86-windows-static
vcpkg install cppwinrt:x86-windows-static-md
vcpkg install cppwinrt:x64-windows
vcpkg install cppwinrt:x64-windows-static
vcpkg install cppwinrt:x64-windows-static-md
vcpkg install cppwinrt:arm-windows
vcpkg install cppwinrt:arm64-windows
vcpkg install cppwinrt:arm64-windows-static
vcpkg install cppwinrt:arm64-windows-static-md
vcpkg install cppwinrt:x86-uwp
vcpkg install cppwinrt:x64-uwp
vcpkg install cppwinrt:arm-uwp
vcpkg install cppwinrt:arm64-uwp

As I was looking into why the Windows SDK env vars wouldn't work, I noticed a bunch of other little things that could be touched up:

The CMake build was entirely unnecessary -- it's just running the header generator and doesn't need a compiler at all!
The examples online indicate that the headers should be included via #include <winrt/...> -- this means we want them at include/winrt/... instead of include/cppwinrt/winrt/....
I have switched to enumerating the Windows SDK contents in the CMake instead of the cppwinrt tool -- this avoids touching the registry and so opens the possibility for consuming arbitrary copies of the WinSDK instead of requiring the installer.

The examples online indicate that the headers should be included via #include <winrt/...> -- this means we want them at include/winrt/... instead of include/cppwinrt/winrt/....

The reason I had it install to include/cppwinrt/winrt and set the include path to include/cppwinrt was so that:
(a) You can still use #include <winrt/ to get the headers
(b) You would not get the updated cppwinrt headers if you had them installed, used some other port that pointed to include but had NOT added the specific CppWinRT target to your project.

LGTM, thanks!


Add JSON Structured Logging to data plane components

Proposed Changes

Add JSON Structured Logging to data plane components

/lgtm
/approve


Allow JSON Matcher to accept string pointers

Seems pretty straight forward.

Ugh, this reminds me that Travis CI is dropping support for FOSS projects.
FUN.
lgtm on CI :green_circle:


Correct Citadel server log.

Resolves https://github.com/istio/istio/issues/11386

@myidpt: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
cdc24de5834e1e7af03b670437ac8f437c35fc57
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
1f1610f5fec2c2270bdfc6d1c7c234c31b7e991c
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


feat(android): simple installation

Hello Team, thank you for always supporting high-quality open source!
I got the idea from react-native-gradle and simplifies the installation.
It works, but I'm not sure of any side effects because I don't have deep knowledge of groovy.
so feel free to comment. thank you!

ooh thanks for this p/r 😍  it looks like it should work 👀  I'll double-check locally, and I'll kick off the jobs.
Would you be able to sign the cla?
Thank you!

I attached a jest result screenshot ran from my local !

If there is a high rate that tests cannot be performed because the ts-jest path cannot be found, we may consider specifying the path directly💡

thank you!! Merging! And about to release 😎

@mikehardy  @helenaford Thank you!


Not compatible with FancyMenu (I am using Sinytra Connector)

Minecraft Version
1.20.x
Mod Loader Version
forge 47.2.19
Mod Version
dungeonz-1.0.5
Describe the Issue
When I press the compass on the cartographer's table, the game crashes. I noticed such a problem at the Fabric, and it seems to be the culprit FancyMenu (as written in Crash_Report)
latest.log
crash-2024-05-30_01.35.29-client.txt
Do you use any other mods except the required ones?
Yes, and quite a lot

Same here on Fabric.
MC 1.20.1


Add issue template for CDN sync issues

Checklist

[X] I added a descriptive title
[X] I searched open requests and couldn't find a duplicate

What is the idea?

Why is this needed?

What should happen?

Additional Context

@dholth is there information that you would find useful to capture in CDN GH issues?


Showcase: Material-UI

Request to be added to the showcase on nextjs.org

Name of company: Material-UI
Url: https://material-ui.com/
Testimonial: Next.js is a great fit for building documentation websites. It can help in achieving insane performance:

800x460
Speaking of performance, https://developers.google.com/speed/pagespeed/insights/

First Contentful Paint (FCP)
DOM Content Loaded (DCL)

Bootstrap

React (Gatsby)

Material-UI (Next.js)

Hi! I use material-ui and i love it. But, I have a question. Do you use next export for improving the web performance? or you serve it like usual react app?

This is a static website. I'm also using next for a dynamic one. You can see it in https://www.onepixel.com.

The showcase has been added in https://github.com/zeit/next-site/pull/77, you will see it on the website soon 😉


Implement tag lookup on recipe curation

When a user creates a recipe, we need to look at tag nodes and see if the tag(s) already exist. If they do, we need just a relationship from the recipe to the tag. If not, we need to be able to create a new Tag node with the corresponding tag name.
To standardize tags, we will accept any case in the request, set all characters to lowercase and name the tag accordingly. This way, duplicates because of weird case do not occur. This will not account for misspellings. However, future improvements that provide a GET /tags endpoint for faster lookup and for common tags in the database should cause these type of errors to go down.

I wrote this query so that it adds a new tag only if one does not exist in DB.


Camera - App passed NULL surface

Hi,
I am using the latest version of ZXing.Net.Mobile (v2.2.9) in my Android application to perform QR code scanning.
Even though I am being able to scan every QR code successfully so far, the app freezes randomly after the scan and the device logs a camera related error - "app passed NULL surface". After this message the app stays frozen for a few seconds until another message is logged into the device logger - "[W/CameraBase] Camera service died!". After this happens the app resumes its normal behavior.
How can I solve this problem?
In my core project I have an event to initiate the scan:
private void StartScan()
        {
            try
            {

                MobileBarcodeScanningOptions oOptions = new MobileBarcodeScanningOptions();
                oOptions.PossibleFormats = new List<BarcodeFormat>() { BarcodeFormat.QR_CODE };
                oOptions.DelayBetweenContinuousScans = 3000;

                _oScanner = new MobileBarcodeScanner();
                _oScanner.TopText = Notices.TITLE_SCANNER_QRCODE;
                _oScanner.CancelButtonText = Notices.LABEL_CANCEL;

                _oScanner.ScanContinuously(oOptions, HandleScanResult);

            }
            catch (Exception ex)
            {
                Logger.Debug("StartScan[ScannerViewModel]", ex);
            }
        }
private void HandleScanResult(Result oResult)
        {
            try
            {
                if (oResult != null && !string.IsNullOrEmpty(oResult.Text))
                {

                       //In here I am handling the result of the scan and navigating to another view
                }
            }
            catch (Exception ex)
            {
                Logger.Debug("HandleScanResult[ScannerViewModel]", ex);
            }
            finally
            {
                if (_oScanner != null)
                    _oScanner.Cancel();
            }

        }

Any help is appreciated.

I have also seen this in the log for Android using the latest version when trying to use the Xamarin Forms wrapper ZXingScannerView and create the control at the same time as the page was created. In a similar way the MobileBarcodeScanner creates a new page/activity to show the scanner.
In my case I changed the strategy and delayed creating the scanner view until the page was visible and things became stable. So I keep my ZXingScannerView active and have no issues. Whenever I turn of continious scan I remove my control and create a new one for the next round. This appeared to help.
Another suspect I had was what I did in my event handler. I had some async code there and this somehow caused issues.
I would try to comment out you code that actually does the handling of the result with just some debug logging and try to see if it becomes stable. Maybe add some logging with Environment.CurrentManagedThreadId to see if you switch thread? Or are you trying to call UI code from another thread than the main thread?
Even if my scenario is very different at least this proves that you can work around this.

Another thing you should try is to rollback to nuget package version 2.1.47 and see if the problem also exists there or not.
A huge refactoring was released just a two weeks ago and new issues and some regression was introduced.

Hi Fredrik,
Thanks for the reply.
My initial approach would fall in your scenario, since it involved calling async methods, doing work on the UI thread while waiting and then resolve a different view. But then I started to simplify the handling of the QR code but I could find this problem even when I did no handling at all. Just null reference validations and then closing the scanner.
I followed your suggestion and rolled back to version 2.1.47 and I am having better results. Not only the QR code scanning is getting much faster, but also the problem I had before seemed to vanish.
I still notice that there is something wrong with the camera service by looking up the device logs (again this happens randomly), but the application does not freeze and keeps it's normal behavior. the error and ZXing stack trace when this sittuation occurs is:
05-02 18:30:56.825 D/ZXing.Net.Mobile( 5892): Found 0 cameras...
05-02 18:30:56.825 D/ZXing.Net.Mobile( 5892): Finding Back camera failed, opening camera 0...
05-02 18:30:56.835 W/CameraBase( 5892): An error occurred while connecting to camera: 0
05-02 18:30:56.845 W/CameraBase( 5892): Camera service died!
05-02 18:30:56.845 E/ZXING   ( 5892): Setup Error: Java.Lang.RuntimeException: Fail to connect to camera service
05-02 18:30:56.845 E/ZXING   ( 5892):   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in /Users/builder/data/lanes/4009/3a62f1ea/source/mono/mcs/class/referencesource/mscorlib/system/runtime/exceptionservices/exceptionservicescommon.cs:143
05-02 18:30:56.845 E/ZXING   ( 5892):   at Java.Interop.JniEnvironment+StaticMethods.CallStaticObjectMethod (Java.Interop.JniObjectReference type, Java.Interop.JniMethodInfo method, Java.Interop.JniArgumentValue* args) [0x00082] in /Users/builder/data/lanes/4009/3a62f1ea/source/Java.Interop/src/Java.Interop/Java.Interop/JniEnvironment.g.cs:12649
05-02 18:30:56.845 E/ZXING   ( 5892):   at Java.Interop.JniPeerMembers+JniStaticMethods.InvokeObjectMethod (System.String encodedMember, Java.Interop.JniArgumentValue* parameters) [0x0001b] in /Users/builder/data/lanes/4009/3a62f1ea/source/Java.Interop/src/Java.Interop/Java.Interop/JniPeerMembers.JniStaticMethods.cs:97
05-02 18:30:56.845 E/ZXING   ( 5892):   at Android.Hardware.Camera.Open (System.Int32 cameraId) [0x00017] in /Users/builder/data/lanes/4009/3a62f1ea/source/monodroid/src/Mono.Android/platforms/android-23/src/generated/Android.Hardware.Camera.cs:4478
05-02 18:30:56.845 E/ZXING   ( 5892):   at ZXing.Mobile.ZXingSurfaceView+<SetupCamera>d__26.MoveNext () [0x0029e] in <04a6c4a6536f4e76a10998845fe4c337>:0
05-02 18:30:56.845 E/ZXING   ( 5892):   --- End of managed Java.Lang.RuntimeException stack trace ---
05-02 18:30:56.845 E/ZXING   ( 5892): java.lang.RuntimeException: Fail to connect to camera service
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.hardware.Camera.<init>(Camera.java:568)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.hardware.Camera.open(Camera.java:405)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at md55109d95aac470e32f9a4031a908f0227.ZXingSurfaceView.n_surfaceChanged(Native Method)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at md55109d95aac470e32f9a4031a908f0227.ZXingSurfaceView.surfaceChanged(ZXingSurfaceView.java:69)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.view.SurfaceView.updateWindow(SurfaceView.java:740)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.view.SurfaceView$3.onPreDraw(SurfaceView.java:215)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.view.ViewTreeObserver.dispatchOnPreDraw(ViewTreeObserver.java:1014)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:2599)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:1524)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:7520)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.view.Choreographer$CallbackRecord.run(Choreographer.java:911)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.view.Choreographer.doCallbacks(Choreographer.java:686)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.view.Choreographer.doFrame(Choreographer.java:622)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:897)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.os.Handler.handleCallback(Handler.java:739)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.os.Handler.dispatchMessage(Handler.java:95)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.os.Looper.loop(Looper.java:148)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at android.app.ActivityThread.main(ActivityThread.java:7303)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at java.lang.reflect.Method.invoke(Native Method)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1230)
05-02 18:30:56.845 E/ZXING   ( 5892): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1120)
05-02 18:30:56.845 D/ZXing.Net.Mobile( 5892): Checking android.permission.CAMERA...

After this happens, the camera takes a little longer to start since the camera service gets restarted.
Is there anything I should be doing to prevent this error?

I too have this error, but after IsScanning=false, IsScanning=true switch. The image freezes and that's that. Downgrading solved it.

Hi frnds,
I am using zxing scanner view in my project (xamarin forms). It working fine till android version 7.0 but higher version like 7.1.1 and above my app crashed.
after scanning i got scan result but after 5-6 sec app got crashed.Any solution ?
Exception is Camera : App passed null surface.
Any help is appreciated.
thanks

Deleting line 137 (Camera.SetPreviewDisplay(null);) from ZXing.Net.Mobile.Android/CameraAccess/CameraController.cs seems to fix this issue for me.
According to the Android documentation the parameter to SetPreviewDisplay must be a fully initialised SurfaceHolder object, so I don't think passing null is valid.

HI, I am also facing it seems a similar issue. My app uses ZXing.Net.Mobile.Forms version 2.3.2. I tried to downgrade but without success. The ZXingScannerView scans the code successfully and calls OnScanResult. In the debug console it says
[ZXing.Net.Mobile] Barcode Found: 990616200
[Camera] app passed NULL surface
and then I get a NullReferenceException and the app freezes.
Any suggestions?

In the OnScanResult handler I replace the ZXingScannerView, which resolves the issue

@philmoz Does deleting that line cause any other effects? Like would something else not work because of that deletion? I'm wondering why it was there if it causes this error.
I have yet to pull the project down to try deleting that myself to see if it works.

So far I have not had any crashes since removing the Camera.SetPreviewDisplay(null); line.
I suspect it was added to 'clean up' the camera object when the scanning stops.

I'm not to familiar with github so I may be saying this wrong, but do you think if it works that you should do a pull (push?) request to get it merged?

I'm having this issue as well. Any word on the status of fixing this? Thanks!

I'm using version 2.4.1 and still facing this problem..could you guys please update the nuget version for this issue? Thanks

This is causing some issues for me as well. A publish of the NuGet would really help us out.

+1 for publish.

I too had the same problem, Scan works but when i try to capture the image I got same error as mentioned above.
Any help is really appereciated!

The only way I have been able to get around it is to Unload and Load the controls from a Grid each time I set IsScanning to False. I keep handles to the controls with member variables and manually hook up the event. It took me a while to get it working correctly, you might have to fill in a few parts of the code but this should help. It's slow to reload depending on the phone, but at least it's working for now until the new package is built.
private void UnloadCameraScanner()
{
    if (_cameraScanner != null)
    {
        ScannerGrid.Children.Remove(_cameraScannerOverlay);
        ScannerGrid.Children.Remove(_cameraScanner);
        _cameraScannerOverlay = null;
        _cameraScanner = null;
    }
}

private void LoadCameraScanner()
{
    UnloadCameraScanner();

    _cameraScanner = new ZXingScannerView
    {
        Options = _vm.ScanOptions,
    };
    _cameraScanner.SetBinding(ZXingScannerView.IsAnalyzingProperty, "IsAnalyzing");
    _cameraScanner.SetBinding(ZXingScannerView.IsScanningProperty, "IsScanning");

    _cameraScanner.OnScanResult += (result) => { HandleScanResult(result.Text); };
    ScannerGrid.Children.Add(_cameraScanner, 0, 0);

    _cameraScannerOverlay = new ZXing.Net.Mobile.Forms.ZXingDefaultOverlay
    {
        TopText = "Top Text",
        BottomText = "Bottom Text",
        ShowFlashButton = false
    };

    ScannerGrid.Children.Add(_cameraScannerOverlay, 0, 0);
}

<Grid x:Name="ScannerGrid" Grid.Row="0" VerticalOptions="FillAndExpand" HorizontalOptions="FillAndExpand">

</Grid>

Thanks for reporting this issue!  Unforunately it took me way too long to respond 😭.  Sorry, I apologize!  Recently the source code for this project was completely refactored to modernize it.  Many PR's were included in this effort, and many bugs were hopefully fixed.  Please try out the latest 3.x series of NuGet packages (currently in prerelease).  To try and make the project more maintainable in my spare time going forward, I've decided to close all existing issues to start with a clean slate.  If you're still experiencing this issue on the newest version, please open a new issue with as much detail as possible.  Thank you for your patience and understanding!  Happy scanning!

I face the same issue in my Xamarin forms project. I have been using the Zxing scanner view for the scanner control.
In my case the issue I have pushed the scanner page inside the Navigation page.
Once the scanning is done, I have set the IsScanning = false.  Then I have popped the page.
Due to this, the Null surface is passed to the camera.
This caused the null reference exception.
To Fix this:
I have removed the IsScanning = false statement. Then it worked perfectly.
Note:
I didn't face this issue when I used the Modal page for the scanner. This happened only when I pushed the normal page in the navigation stack.


feat: Drop the hostname requirement when handling PV topology for release-v0.34.x (#1018)

Fixes #N/A
Description
Cherry-picks over #1018 into the v0.34.x release branch
How was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


The speed of predict is so slow!

Hi, everyone. I am meeting a really confusing question!
I use the Super Computing Center in China to accelerate my training process, which deploys the DCU Card and installs the pytorch ROCm whose version is 4.2(beta).
But I encountered a srange issue:

As the picture above shows, The value in left circle  is time of out = model(in) , and the valu in right circle is time of backward().
In NVIDIA's 960, the two value above are both 0.0x. I think this is so weired. The input that was feed to network is an array whose shape is (2500, 12).
Another, I test on two different Super Compute Center in China. They all have the same problem.
Actually, I know that maybe I have to make some optimization if I want to move from cuda to dcu. But I can't find the solution and I wanna know why.
Thanks vvvvvvery much!!!!!!

Thanks @Joevaen for reaching out.
Can you please share the exact steps you followed to reproduce the problem so that I will try the same and update.
Thank you.

https://github.com/RadeonOpenCompute/ROCm/issues/1565 is the same question. We have dealt with this issue via modify network architecture. Maybe you can make a test and I think it is an urgently needed bug.

Hi @Joevaen
Can you please validate with the latest ROCm 4.5 and update.
Thank you.

I am closing this as there is no response from user for the last 4 weeks.
Request to open a new ticket, if any, with all details steps, for quick resolution. Thank you.


AI-Powered shooter based on field location

Based on the robot's estimated field location (taken from limelight and swerve data, but perhaps not entirely accurate) we might be able to train a model using evolutionary learning in a simulator (I'm eyeing IsaacSim and PhysX by NVidia, but a game engine might work) to learn the angle and shooter wheel speeds needed to score a note at distance into the speaker (and potentially, in a secondary model, into the amp). Of course this only works if we have an angleable shooter in the robot design. But I think the soft kinematics can be accounted for by a good simulator.

This does seem quite complex. I also don't have access to NVidia GPUs for IsaacSim or PhysX, so it might be slow or impossible to do it with those. Something else, like MuJoCo or PyBullet, might work, but it involves a decent amount of work to understand how to model everything with materials that accurately deform. It's quite a cost-benefit analysis thing.

This could allow me to take an OnShape model of our shooter, the game piece, and the speaker and convert them to the URDF file format that MuJoCo or PyBullet can use. I'm concerned that note deformation throughout an event might affect the model's accuracy, but that problem is true of any way we do the shooter, and I might be able to simulate this in the training.

An alternative that does seem simpler is to program a bunch of positions with known values for shooter speed and angle and robot angle. Give the driver a selection they can choose from. Don't know how advanced the widgets on SmartDashboard can get, but with the touchscreen driver's station and a second driver, we should be fine. The driver could have an indicator (on-screen/through LEDs on robots) that indicates the closeness to the position, and the driver could hit a button to make the robot do the final piloting when it's close enough.

The one drawback is that we likely can't take moving shots (unless those were additional positions). But at least for the first few competitions, missing this capability will not be detrimental. For context, BOB is considering this method. While less impressive and robust, as long as we can get time on the field to test all of these, we should be fine.

I'm going to sweep this idea under the rug. Judging by the complexity, it's likely our competition—unless we get to a very high level—won't be doing anything like this. The complexity of simulating everything has so many unknowns, so I'll say we focus our resources on something less complex and more repeatable.


docs: cache tweets

Closes #
✅ Checklist

[ ] I have followed every step in the contributing guide (updated 2022-10-06).
[ ] The PR title follows the convention we established conventional-commit
[ ] I performed a functional test on my final commit

Changelog
[Short description of what has changed]

Screenshots
[Screenshots]
💯

Bumped a few versions too while I'm at it

Reduced headers in the Folder Structure to fix the line-breaks on small devices, i think they're plenty big still


Tests for Parser.SpecialChar() and extension options

This pull request contains tests for Parser.SpecialChar() and extension options (It can detect bugs like #159 and #161).

Instead of those formOptions and collectPairs, I think combinations could be used to generate extension option pairs.

 I'll fix it.

If I merge this PR into the current master these new tests fail.
Unfortunately I don't have the capacity to look into the cause of the problem.
Can you help?

@sirthias Oh! OK, that's good! I will. :smiley: Thank you!


Step and max value bug

Steps to reproduce

Create slider with step 50
set max value at 80, min at 0
drag slider to the very end

Demo: http://jsfiddle.net/9hc8ypk4/1/
Expected behaviour
Value should go like so: 0 -> 50 -> 80.
Actual behaviour
Current value goes like so: 0 -> 50 -> 100 -> 80.

This is not a big, your ceil is not coherent with the step, that's why...

It is a bug when you're dealing with mobile data transfer where max is dynamic, step is static and when max should NEVER go over the bounds. Expecting for step to be coordinated with max is simply unreasonable.

I see, then you should set enforceRange to true.

That doesn't seem to do anything for me. It still stops at 100 before 80.

Indeed. The issue is that you can also argue that the value should ALWAYS match the step...

So do you have a workaround or a fix for this? Because in my app it's a bug.

You could use a virtual limit by using the new maxLimit option: http://jsfiddle.net/0vc6axxc/
Thus you set the ceil, the the actual maximum (that should match the step) and you can set any maxLimit  you want.

That worked. Thank you.


Any roadmap for 2.* release?

Wondering if it's possible to get some visibility on the 2.* roadmap and release schedule. If it's still in the works.
If not anytime soon, it would be great to have a Timber update for Twig 3.* and other dependency updates.

I second that. I am also willing to spent some time as well to drive development forward but I don't know where to start at this moment. I would love to see a day organized where we rally some Timber users and contributors and iron out the release. We would need some guidance from the devs though. @jarednova , would you mind to give your thoughts on this?
And are there people interested in that?

I'd also be interested in having some visibility on the 2.* roadmap and I could also consider contributing to this project, if the development is still active

@jarednova Should I start a super-simple (6 page types) theme today with Timber v2?

@szepeviktor I've been using Timber v2 in production on several blogs for months now, everything seems good. If you're starting something new I would totally use v2 to avoid upgrade hassles, keeping in mind that some things might still change before the official release.

We just released a first beta for Timber 2.0: 2.0.0-beta.1 (release).
Unfortunately, we have no final release date yet and we don’t want to commit to one, because we all work on it in our free time. Thank you so much for understanding.
We made good progress in the last couple of weeks. If we can keep that momentum, I think you can expect a stable release soon.
For now, it would help us the most if you can test out the beta and report any bugs. Or if you feel like tackling known bugs for 2.0, there’s a 2.0 label in the issues. Feel free to work on any issues that have no assignees yet. I tagged some of the with the "Help wanted" and "Good first issue" labels, because I think they would be easy to tackle.
I’ll leave this issue open for now in case you have additional questions.

test out the beta and report any bugs

You can use PHPStan for that. It tells you a few dozen problems right away.

@szepeviktor Yep, working through the issues that PHPStan reports up to a certain level is also on our list. 👍

Start on Level 0, it is less frightening.

Hi everyone, the team is hard at work to make Timber 2.0 happen. We've just created an updated roadmap with open tickets, pull requests, etc. If you would like to help out on Timber, now is the perfect timing. See #2741
Closing this ticket in favor of the mentioned ticket above.


added home section details and language switching to it

can you share the screenshot or recording?

_ No description provided. _

screenshot???

Resolve conflicts !!


mouse motion to scroll: disable inertia

First, thank you for restoring mouse-motion-to-scroll functionality.
I'd like to add an option to disable inertia. My device is a marble mouse, which already has physical inertia. So the simulated inertia is unwanted.

I looked at the code.  It's possibly something I can contribute.
But I think some refactoring is needed (e.g. there are unused parameters such as "momentum_minus_"), and @tekezo is modifying this code very recently, so I'll defer to him.

OK, I'll add an option. Please wait for a while.

I've added options.momentum_scroll_enabled at v12.3.2.
How to upgrade to the latest beta version:
https://pqrs.org/osx/karabiner/document.html#check-for-updates
You can use it by this json.
https://github.com/tekezo/Files/blob/master/karabiner-elements-extra-json/mouse_motion_to_scroll_ex.json
You can import it by copy karabiner://karabiner/assets/complex_modifications/import?url=https://raw.githubusercontent.com/tekezo/Files/master/karabiner-elements-extra-json/mouse_motion_to_scroll_ex.json and paste the text to web browser's location bar.

Thank you.  I've tried it but the behavior seems quite nonlinear.  I'll try to describe it:

slow mouse movement causes high speed scroll
quick mouse movement causes low speed scroll
scroll seems to retreat slightly after quick mouse motion (which gives me some sickness feeling)

I'm using speed parameter set to 400%.  (By the way, could "no momentum" be a parameter instead of separate rule?)
I'm comparing this behavior to a commercial product ("Smart Scroll") which I started using after the transition from Karabiner to Karabiner-Elements.  It has a smooth 1:1 mapping between mouse motion and scroll, which works very well (though I don't like trusting a proprietary binary).

Please make a pull request.
You can adjust the behavior by replacing karabiner_grabber.
https://github.com/tekezo/Karabiner-Elements/blob/master/docs/DEVELOPMENT.md#replace-karabiner_grabber


server::kv_service::test_split_region failed

Latest build: https://internal.pingcap.net/idc-jenkins/job/tikv_ghpr_test/37456/display/redirect

Traced by #9151.


Updated dependencies and fixed the demo tests

Updated demo tests to work with the newer seleniumeasy website popups

Only found single whitespace in the built artifact.
So I dont think new release is needed. It might be good psychologically to confirm wdio 6 support with a new build though. I dont know.
Anyhow .. its good that the demo tests are updated as well, since they help with validating functionality for both jasmine style tests and cucumber.


drivers: hwinfo: Add support for generating device id from device addr

In some ICs (including nRF54H20) the DEVICEID register is not part of FICR, and thus it is not accessible to applications. Use instead the device address, along with a couple of bytes from ER and IR, to generated a unique device id.
At the same time update the pointer to the hal_nordic repo to pull in https://github.com/zephyrproject-rtos/hal_nordic/pull/196.

Last push: remove change to west.yml, since https://github.com/zephyrproject-rtos/zephyr/pull/76330 covered it.


[New game]: Add Feed the snake game

🎮 Game Request
Players have to guide the snake to eat food that randomly appears on the game board. The game will end if the snake hits a wall or its own body.
Point down the features
When the snake eats food, the score should increases by one.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @suhanipaliwal !
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🕹

/assign

Hey @suhanipaliwal ! 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! 😇


walletName undefined causing errors during connections

Description of problem
Older wallets running the latest build will overwrite the default value of walletName with undefined causing errors during connections due to the label missing
Expected behavior
The label should always be defined and the default value should not be overwritten with undefined during initialization
Steps to reproduce
Complete onboarding with an older version of the wallet (pre-wallet naming PR), then switch to the latest version without wiping your storage. After reloading, try to make a connection.
Screenshots and/or log output
Environment

[x] Occurs on Android
[x] Occurs on iOS

Build #: 998
Android Device Model:
iOS Device Model:
Workaround
N/A
Severity

[x] High
[ ] Medium
[ ] Low

assigned back to @bryce-mcmath to confirm if bug is fixed, if yes mark as closed


How to access "Business Central Administration tool". (to enable API)

Hello,
I am following this guide to Enable access to the APIs.
For that, I need access to Business Central Administration tool.
I do not see a link to this tool is created. I tried both v.14 & 20 images.
Please let me know a way to enable though command line if the tool is not available
I use below script to create containers.
Thanks
Bsr
$containerName = 'bc14-dev'
$password = 'Pas$Word!'
$securePassword = ConvertTo-SecureString -String $password -AsPlainText -Force
$credential = New-Object pscredential 'admin', $password
$auth = 'UserPassword'
$artifactUrl = Get-BcArtifactUrl -type 'OnPrem' -country 'na' -select 'Closest' -version '14.0.40464.0'
New-BcContainer `
    	-accept_outdated `
    	-accept_eula `
    	-containerName $containerName `
    	-credential $credential `
    	-auth $auth `
    	-artifactUrl $artifactUrl `
    	-isolation hyperv `
    	-memoryLimit 8G `
    	-includeCSIDE `
    	-doNotExportObjectsToText `
	-WebClientPort 8082 `
	-FileSharePort 8280 `
	-ManagementServicesPort 7245 `
	-ClientServicesPort 7246 `
	-SoapServicesPort 7247 `
	-ODataServicesPort 7248 `
	-DeveloperServicesPort 7249 `
    	-updateHosts `
    	-publishPorts @(8082,8280,7245,7246,7247,7248,7249) `

You are looking for the ApiServicesEnabled setting.
You can extend your creation script from above with the following powershell lines after New-BcContainer.
Invoke-ScriptInBCContainer -containerName $containerName -scriptblock {
     Set-NAVServerConfiguration -ServerInstance $serverInstance -KeyName "ApiServicesEnabled" -KeyValue "true" -Force
     Set-NAVServerInstance -ServerInstance $serverInstance -Restart
}

ApiServicesEnabled should be enabled for all BC versions by default.
With the command above - you should be able to connect to APIs on port 7248 on the host (or on the container) using basic authentication.


Write integration test for InstanceAdmin::DeleteCluster.

write integration tests for DeleteCluster.
This task is depndent on #422

Yes.  Fixed by #696.


Fix flex layout for repo list icons

#20241 Added a tooltip, which does not satisfy the flex layout, and the icons are not aligned

before
after

repo list icon

Signed-off-by: Xinyu Zhou<EMAIL_ADDRESS>

🎺 🤖


docs: Uniformize usage of quotes in links of CodeQL CLI

Why:
When browsing the documentation, there seems to be some inconsistency on the links, some have quotes and some do not have. If I understood correctly links to docs.github.com should have quotes, while to external sources should not.
With this I hope it uniformizes the usage of quotes in the doc under CodeQL CLI
Closes: https://github.com/github/docs/issues/28908

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[ ] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@gonmmarques Thanks for submitting a PR against your issue 👍
However, unfortunately we won't be able to accept these changes. CodeQL docs are autogenerated and will require internal updates. I'm going to go ahead and close this PR 💛
We truly appreciate your efforts in improving the docs. If you are participating in Hacktoberfest, we are going to label this as "accepted" so you still get credit for it, as it was a great effort and not your fault we can't accept these changes. If you are looking for more ways to contribute, check out our help wanted section to find open issues you can work on. ✨

Hello, totally makes sense. I was also going on a hunch that these changes were needed and not really sure.


solver_stats is None for CVXOPT

Is your feature request related to a problem? Please describe.
Following is my MWE.
import cvxpy as cp
import numpy as np

p = cp.Parameter()
x = cp.Variable()
quadratic = cp.square(x - 2 * p)
problem = cp.Problem(cp.Minimize(quadratic), [x >= 0])
p.value = 3.0
problem.solve(solver=cp.CVXOPT)

print(problem.solver_stats.solve_time)
print(problem.solver_stats.setup_time)
print(problem.solver_stats.num_iters)
print(problem.solver_stats.extra_stats)

The output is:
None
None
None
None

Describe the solution you'd like
I would like to extract the solver statistics.
Describe alternatives you've considered
N/A.
Additional context
Add any other context.

As far as I am aware, CVXOPT does not return those stats. For details on what the CVXOPT solvers output, please consult the documentation: https://cvxopt.org/userguide/coneprog.html
Those stats are returned by solvers such as ECOS and SCS if I am not mistaken.

So CVXOPT might not return those stats, but we could consider intercepting CVXOPT's logging stream (not sure how) and recovering information from that. If the user solved the problem with verbose=True then we'd print out what we intercepted to stdout as usual.

That will be really helpful.
On Sat, Mar 19, 2022, 10:36 AM Riley Murray @.***>
wrote:

So CVXOPT might not return those stats, but we could consider intercepting
CVXOPT's logging stream (not sure how) and recovering information from
that. If the user solved the problem with verbose=True then we'd print out
what we intercepted to stdout as usual.
—
Reply to this email directly, view it on GitHub
https://github.com/cvxpy/cvxpy/issues/1458#issuecomment-1073020911, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAC2A7ABDPMKXIGLZPA2P3VAXQ6JANCNFSM5BRSQLXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Executing a re-proposed block that uses an expired blob fails.

It must always be possible to re-propose the locked block, even if the committee that certified a blob which the new block reads has been deprecated in the meantime.
Let's either send along the ExecutionOutcome with the proposal, so the validator doesn't have to execute it again until it is confirmed. (Alternatively, we could pass the provided blobs into execute_block, so they can be read even though they are not in storage.)

This will have a negative effect on the latency. Probably also on a throughput.

This will have a negative effect on the latency.

We have to do one of the options to guarantee liveness. It will affect only the (hopefully rare) cases where rounds have failed in the first place, and then it is a tradeoff:

Do we want to redundantly execute the block again?
Or do we want to use more bandwidth but send along the execution outcome?

Regarding the blobs themselves, we could also consider adding back the blobs cache at some point.