Incorrect error spans

I'm seeing this on Sublime Text 2 on OS X currently with the latest 'master' bits.  It's very easy to get the error spans out of sync or confused.  For example, see the below 1 char error span left in a valid function after any errors were fixed (the return type), or the one below that where there is one error (no property by that name exists) but it's showing up as 3 separate error spans:

(Note: I just rolled back to the latest tag and see the same behavior, so this isn't a new issue in 'master').

This is caused by the way we do reloading differently on Sublime 2. I have a fix that will send out PR after the #205 is merged.

See commit 784e70c

Fixed in #205.


Ink-Stained Steel Sakura Event Maps Not Found

Seems like it affects E-A1 to E-B1 so far. Here is my config settings:
...
[Updates]
Enabled: True
Channel: Development
[Assets]
Server: EN
[Combat]
Enabled: True
Map: E-B1
KillsBeforeBoss: 0
BossFleet: True
OilLimit: 8000
RetireCycle: 1
RetreatAfter: 0
IgnoreMysteryNodes: False
FocusOnMysteryNodes: False
ClearingMode: False
HideSubsHuntingRange: False
SmallBossIcon: False
SirenElites: False
IgnoreMorale: False
LowMoodSleepTime: 1
SearchMode: 1
[Headquarters]
Dorm: False
Academy: False
SkillBookTier: 2
AvailableSupplies: 5000 3000 2000 1000
[Modules]
Commissions: False
Missions: False
[Enhancement]
Enabled: False
SingleEnhancement: False
[Retirement]
Enabled: True
Rares: True
Commons: True
...

Issue solved, please delete issue thanks!

What did you do to fix this? I have the same issue with B2.

What did you do to fix this? I have the same issue with B2.

@aodpip In combat.py change these values to the ones listed below (lines 260-266):
        Utils.wait_update_screen()
        map_region = Utils.find('maps/map_{}'.format(self.chapter_map), 0.96)
        if map_region == None:
            Logger.log_error("Cannot find the specified map, please move to the world where it's located.")
        while map_region == None:
            map_region = Utils.find('maps/map_{}'.format(self.chapter_map), 0.96)
            Utils.wait_update_screen(1)

I believe the reason having a lower value than 0.99 in "Utils.find('maps/map_{}'.format(self.chapter_map), 0.99)" lets the program see the event map number image through the flowers in the sakura event.

What did you do to fix this? I have the same issue with B2.

@aodpip In combat.py change these values to the ones listed below (lines 260-266):
        Utils.wait_update_screen()
        map_region = Utils.find('maps/map_{}'.format(self.chapter_map), 0.96)
        if map_region == None:
            Logger.log_error("Cannot find the specified map, please move to the world where it's located.")
        while map_region == None:
            map_region = Utils.find('maps/map_{}'.format(self.chapter_map), 0.96)
            Utils.wait_update_screen(1)

I believe the reason having a lower value than 0.99 in "Utils.find('maps/map_{}'.format(self.chapter_map), 0.99)" lets the program see the event map number image through the flowers in the sakura event.

I have just done done this change and that seems to fix the issue where it can't find the map but now it just keeps searching the map and doesn't select an enemy to battle.

I have just done done this change and that seems to fix the issue where it can't find the map but now it just keeps searching the map and doesn't select an enemy to battle.
@aodpip Try setting your config SearchMode: 0.
From wiki:
SearchMode: Specify which search method the bot will employ. Allowed values are 0 and 1.
0: The bot searches for the triangles which represent the enemy type and for the level, the coordinates obtained are then translated, in order to point to the center of the cell. This means that an enemy is picked up as long as its type or its level are visible. The downside is that this option is heavier (since the bot will also search with resized assets) and it's more susceptible to false positives when the similarity threshold is lowered.
1: The bot searches for the triangles which represent the enemy type and for the level: for each finding a rectangle is drawn. An enemy is recognized as such if there is an intersection between a rectangle drawn from the enemy type and one drawn from the level: the center of this intersection is the enemy's coordinates. This improves the reliability and the performance of enemies' search. The downside is that both the triangles and the level must be visible for an enemy to be picked up.
Default: 1.


Package request: quarto for darwin

quarto is currently available for "x86_64-linux" only. I tried to edit the default.nix file found here to make it work on macOS on my nixpkgs fork. I simply changed the download link to download the macOS binary and the platform on line 74 to "x86_64-darwin" and tried building it but that doesn't seem to work. The building process fails with:
error: builder for '/nix/store/qij4mc9nczy04hk8nr99fipcp1wq479d-quarto-1.2.475.drv' failed with exit code 1;
       last 10 log lines:
       > tar: Ignoring unknown extended header keyword 'LIBARCHIVE.xattr.com.apple.cs.CodeDirectory'
       > tar: Ignoring unknown extended header keyword 'LIBARCHIVE.xattr.com.apple.cs.CodeSignature'
       > tar: Ignoring unknown extended header keyword 'LIBARCHIVE.xattr.com.apple.cs.CodeRequirements-1'
       > tar: Ignoring unknown extended header keyword 'LIBARCHIVE.xattr.com.apple.cs.CodeRequirements'
       > tar: Ignoring unknown extended header keyword 'LIBARCHIVE.xattr.com.apple.cs.CodeDirectory'
       > tar: Ignoring unknown extended header keyword 'LIBARCHIVE.xattr.com.apple.cs.CodeSignature'
       > tar: Ignoring unknown extended header keyword 'LIBARCHIVE.xattr.com.apple.cs.CodeRequirements-1'
       > tar: Ignoring unknown extended header keyword 'LIBARCHIVE.xattr.com.apple.cs.CodeRequirements'
       > tar: Ignoring unknown extended header keyword 'LIBARCHIVE.xattr.com.apple.cs.CodeDirectory'
       > unpacker produced multiple directories
       For full logs, run 'nix log /nix/store/qij4mc9nczy04hk8nr99fipcp1wq479d-quarto-1.2.475.drv'.
error: 1 dependencies of derivation '/nix/store/inclgcmqnvdhwhpya91nfvp0nr2v1f8b-nix-shell.drv' failed to build

Here's my attempt: link to my fork. Being a beginner with Nix, I'm asking here for help :smile: But this seems to be related to tar that might work differently on macOS than on Linux? When decompressing the archive manually I do see the /bin and /share folders that are then copied during installPhase though.
I'm trying this on a hackintosh VM, as I don't own a Mac, but think it would be good if Quarto would be also available for macOS on nixpkgs.
Metadata

homepage URL: https://quarto.org/
source URL: https://github.com/quarto-dev/quarto-cli
license: gpl2+
platforms: linux, darwin

pinging package maintainers
@MrTarantoga @minijackson
fyi: @jbedo @kupac @philipp-baumann

so tar tzf on the archive for darwin returns:
tar tzf quarto-1.2.475-macos.tar.gz
./
./bin/
./share/
and then a long list of files

while on linux it returns:
quarto-1.2.475/
quarto-1.2.475/bin/
many files and then
quarto-1.2.475/share/

many files

so there's a top-level folder for the linux archive, but not for the macOS one, which I guess is the source of the problem.

So following the advice from this old thread: https://releases.nixos.org/nix-dev/2016-June/020541.html
solved the issue, quarto now successfully installs on macOS. Here's the line I added to the default.nix file: https://github.com/b-rodrigues/nixpkgs/blob/572a740bfffb56dbea63fda503754107d6c01e6b/pkgs/development/libraries/quarto/default.nix#L24
So now the question becomes: how can I rewrite this expression so that it works of both linux and darwin?

I'm trying to rewrite the expression using if then else statements like this: https://github.com/b-rodrigues/nixpkgs/blob/dca8ea46d50c9e586d4c6ccdced5cd0ea20928dd/pkgs/development/libraries/quarto/default.nix#L27 but this fails with `error syntax error, unexpected ID, expecting ":" or "@".

you can do something like:
src = {
    "x86_64-linux" = fetchurl {
       url = "...";
       sha256 = ...;
     };

     "x86_64-darwin" = fetchurl {
        ...
      };
    }.${system}

thanks, this now works on both linux and macOS (x86_64, I don't have the hardware to test aarch64).
would it be ok to open the PR?

@sagikazarmark I saw that you were also interested in getting quarto for macOS (https://github.com/NixOS/nixpkgs/pull/228984#issuecomment-1536278120), I can only test on a hackintosh VM but if you have access to the hardware and are still interested in getting quarto for macOS, that'd be great!

@b-rodrigues I use quarto on macOS (mostly ARM these days, but I have access to Intel), so yeah, I can test it on macOS.
In fact, I have one that runs on my Mac in my NUR repo: https://github.com/sagikazarmark/nur-packages/blob/master/pkgs/quarto/default.nix
Although it's a somewhat older version. Let me see if I can get #228984 working on darwin.

That's cool, thanks! But did you link to the correct expression? The one you linked to seems to download the linux binary?

It does, but it works all the same.

oh wow I wasn't expecting that! Are you also using some kind of compatibility layer?

Not really. It's just a nodejs app as far as I can tell, so it should run on most platforms just fine. The rest of the dependencies already come from Nix.

Ok so that certainly makes things easier! In that case it would just be a matter of adding the platforms to the expression I guess?

hi @sagikazarmark did you manage to update the package ?

I've updated quarto in my NUR: https://github.com/sagikazarmark/nur-packages/blob/master/pkgs/quarto/default.nix
It works just fine on darwin aarch64

@sagikazarmark that’s great! Since there’s no difference between the packages, could you open the PR to get this latest version of quarto on nixpkgs?

Fixed by #259443


[BUG] python setup.py generate_stubs fails with "No module named 'setuptools'"

Describe the bug
I'm following the instructions from Build from source and when I hit the step for Python API where I run python setup.py generate_stubs I hit this:
% python setup.py generate_stubs
Traceback (most recent call last):
  File "/Users/rudkx/repo/mlx/setup.py", line 11, in <module>
    from setuptools import Command, Extension, find_namespace_packages, setup
ModuleNotFoundError: No module named 'setuptools'

To Reproduce
Follow all the steps in Build from source up to the python setup.py generate_stubs.
Expected behavior
Successful installation.
Desktop (please complete the following information):

OS Version: macOS 14.5

Additional context
My repo was forked earlier today with top-of-tree at 95d11bd.

Looks like you need to install setuptools, e.g.  pip install setuptools


Translation and correction of translated texts

Complete translation and correction of poorly translated texts corrected for pt_BR.

Thank you!


Speedup polymod in ~15x times

Benchmark result:
$ SEED=64403a4d05e393ac93683b1ff825f342 node index.js
Seed: 64403a4d05e393ac93683b1ff825f342
Spent 5.000023416s for 256735 encode/decode: 51346.760 op/s

$ SEED=64403a4d05e393ac93683b1ff825f342 node index.js
Seed: 64403a4d05e393ac93683b1ff825f342
Spent 5.000157577s for 15747 encode/decode: 3149.301 op/s

Coverage remained the same at 100.0% when pulling 2d4385309c4f2ac0ab2bd9e49b72a3bf6425373d on fanatid:performance into bb2d2fc7bdaf3ad1846e5d8faf1ec6007e201653 on bitcoincashjs:master.

Bump. Can we get this merged in?

Compare the function polymod against the official spec: https://github.com/bitcoincashorg/bitcoincash.org/blob/master/spec/cashaddr.md#checksum.
The code in this library is essentially a straightforward translation of the code snippet provided in the spec into JavaScript, using a well-tested library for dealing with big integers.
With this PR's version of the polymod function, the process of verifying that the library matches the spec becomes less trivial.
For speed, you can use something like this, or something like this.

For notes, with node v12:
// bitcoincashjs/cashaddrjs
$ SEED=64403a4d05e393ac93683b1ff825f342 node index.js
Seed: 64403a4d05e393ac93683b1ff825f342
Spent 5.000286579s for 17785 encode/decode: 3556.796 op/s
// fanatid/cashaddrjs#performance-bigint
$ SEED=64403a4d05e393ac93683b1ff825f342 node index.js
Seed: 64403a4d05e393ac93683b1ff825f342
Spent 5.000098226s for 34732 encode/decode: 6946.264 op/s
// fanatid/cashaddrjs#performance
$ SEED=64403a4d05e393ac93683b1ff825f342 node index.js
Seed: 64403a4d05e393ac93683b1ff825f342
Spent 5.000026432s for 373435 encode/decode: 74686.605 op/s


Classic resources do not support tags

Adding a line to the limits section to call out that classic resources do not support tagging.

@tfitzmac - Would you review the proposed changes? Thanks

@vivekr20 - thanks for this addition! #sign-off

#sign-off


Remove IPMI mc info aux version check

Analysis and coding in progress but it's a bit complication scenario when Master vs other custom builds versioning
https://github.com/openbmc/openbmc-test-automation/issues/1297

This is breaking every test runs. so quick fix for now

https://gerrit.openbmc-project.xyz/#/c/10240/


Feature Request: Minimize to Tray on Startup

Feature Request: Minimize to Tray on Startup
Dear Mechvibes Development Team,
I hope this message finds you well. I would like to request an enhancement for Mechvibes. Currently, the application opens a visible window upon startup, which can be inconvenient for users who prefer a more seamless experience.
Would it be possible to add an option to minimize the application to the system tray upon startup? This feature would greatly improve user experience by allowing the application to run in the background without displaying the main window.
Thank you for considering my request. I appreciate all the hard work that goes into making Mechvibes a great application.
Best regards,
[A.Ramazan]

Yah! OBVIOUS!
I don't understand why this feature hasn't existed accross all these versions


Allow passing of Configuration object, not just resource path, for DropwizardAppRule

When testing it is often much more convenient to just construct Configuration object that is needed, exactly as is, instead of hand-crafting a YAML file to store as resource, and/or add dot-path overrides.
But currently it seems that only resource path approach is supported.
I will try to come up with a patch that offers alternative mechanism to use where full Configuration POJO instance is passed as is.

Thanks. Squashed and applied to master.


(A11y severity 2) Associate notifications dropdown content more closely with icon

The "Notifications" icon (), when selected, opens a drop down of recent notifications. While the drop down is accessible by keyboard, users can only access it after they have navigated through the entire page. Keyboard focus should be set to the dropdown content upon selecting the icon or, optimally, the underlying code should be rearranged to ensure the dropdown content comes immediately after the icon instead of at the end of the page.

Back to @sathomas for minor follow-up

followed up on review. To @nicolaasmatthijs to check

Looks good. Merged.


Remove unecessary explicit rubygems access

Since the dependencies are already specified in the .gemspec, there is
no need for this.

To ellaborate: having explicit rubygems stuff like this in the library code makes it difficult to use capistrano without Rubygems .For example, when capistrano is packaged in GNU/Linux distributions e.g. Debian, the dependencies are all installed through the native package manager and not by Rubygems.


clearTimeout will not clear timers created prior to clock installation

We understand you have a problem and are in a hurry, but please provide us with some info to make it much more likely for your issue to be understood, worked on and resolved quickly.

FakeTimers version : 8.0.1
Environment : node12, macOS Catalina 10.15.7
Other libraries you are using: several, this is happening in tests for a VS Code extension

What did you expect to happen?
clearTimeout to clear timers even if they weren't created with the mock.
What actually happens
The timer that should have been cleared still fires.
How to reproduce

Describe with code how to reproduce the faulty behaviour
or link to code on JSBin or similar

const timer = setTimeout(() => console.log('This should not be called'), 1);
const clock = FakeTimers.install();
clearTimeout(timer);
clock.uninstall();
// ... wait 1 ms + next event loop
// > 'This should not be called'

I can make a PR to fix this if the current behavior is not intended. I added a patch to my own source, though a patch here would be much more robust (need to consider things like duplicate handles, clearInterval, clearImmediate, etc.)

Closed by #407


instagram hashtag search depracted

Hi,
after API changes on Instagram side, searching for tags using /tags/tag/media/recent stopped working as It is depracated.

Please?! any updates?!
thanks

Facebook is now unifying all the api under the same graph api see https://developers.facebook.com/docs/instagram-api/hashtag-search, you should consider using a nodejs Facebook client instead of this lib

can u please tell me how to do it ?  and tits really sad that your library not supporting this feature :(

I am closing the issues in this repo:
The legacy Instagram API is deprecated and will be disabled on June 29, 2020. More information instagram.com/developer.


Match case feature broken

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.82
OS Version: Windows 11

Steps to Reproduce:

Unsure how the functionality broke
Modified key binds and it did not work.
I tried uninstalling and re-installing. It did not fix the issue

### Tasks

### Tasks

Are you talking about in the Find widget and/or the Search Panel?
And you should give an example text and find/search query where it doesn't work.

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Random failure when logging

Details
Hi!
I am facing a quite unexpected behaviour with the package.
Logging sometimes works, and some other times doesn't.
The message that is displayed when the log fails is this one: failed because of a processing error or invalid data.
I am only logging a simple String message, and I have also tried wrapping the log static method with a try catch without success.
Do you have any clue or suggestion on how to overcome this?
PS Thanks for the package!!!

Hi @mrverdant13 I don't have an answer for that and haven't seen the message. Do you have a stack trace? Or an example log line that precedes the error messaging?

I just found the root issue.
I am trying to switch environment by triggering DataDog initialization again.
Is there any way to do this?
In the meantime, I end up using a custom tag for my environment, but the client token cannot be changed.

@mrverdant13 maybe I could understand your use case better. Why do you want to switch environments? Could you instead use separate Datadog loggers and add your unique tags to those individual loggers?

@mrverdant13 heads up, I'm going to close this ticket in 72 hours


[stable/phpbb] allow use of external database

Since bitnami/phpbb:3.2.1-r2 it is possible to configure phpBB to use an already existing database.
This PR makes the required changes in the chart to create the database using the MariaDB chart and use it for phpBB.

/ok-to-test

@prydonius can you recheck this?

/assign @sameersbn

thanks for the tip @timothyclarke! I update the PR and this other too https://github.com/kubernetes/charts/pull/2814

/approve

/lgtm


Limit the number of elements acquired per cycle by LQ

Fixes #7509
It has to be merged after https://github.com/dmwm/WMCore/pull/7546
As Seangchan mentioned in the issue, if higher priority work keeps coming, it will eventually oversubscribe the agent(s). Anyways, that's the behaviour we want to keep.
availableWork is a joker method :-) since we use it to:

pull work from the parent queue (GQ)
pull work from the local queue (LQ)
same as 2, but only used for calculating the resources available before calling 1)

@ticoann I tested it in my VM and seems to be working fine. Let me know if I made any mistake or you have suggestions.

BTW, it will have pylint errors since we need to first merge https://github.com/dmwm/WMCore/pull/7546

All these unit test failures because I didn't based this PR in #7546.
Can you review that other PR and merge it if it's Ok? Then we can trigger these tests again.

test this please

test this please

As we discussed, we will try to twig the configuration to find the optimal number for wq element to pull.


update redis wrapper

更新 redis wrapper，更改 eval 接口参数列表，增加部分日志便于问题排查

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

Project coverage is 38.13%. Comparing base (c9f1f94) to head (b5b7e9b).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #864   +/-   ##
=======================================
  Coverage   38.13%   38.13%
=======================================
  Files          61       61
  Lines       10436    10436
=======================================
  Hits         3980     3980
  Misses       6156     6156
  Partials      300      300


Support the KSZ8795 phy in the phyHandling driver.

Description
Support the KSZ8795 switch in the phyHandling driver.
Driver's MIIM interface phyAddress for the 4 ports starts at 0x1 instead of 0x0.
Test Steps
Connected to the MIIM interface of a KSZ8795 switch.
Related Issue
N/A
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@AniruddhaKanhere Is there a way to see the uncrustify's corrections? Tried to run the tool locally and got no corrections as far as I can see (not a user of the tool normally so I might be doing something wrong!)

/bot run uncrustify

/bot run uncrustify

@ancantus, you can ask the GitHub action to uncrustify the source code for you by commenting /bot run uncrustify as i did here.


More then 19 devices causing issues

Has anyone gotten more then 19 devices working? I'm adding device 20 and only the on URL gets a hit from echo not the off. When I added device 24 the echo refused to discover. I can refresh the database and do additional tests. Just wondered if anyone has gone past 19.  Thanks.

For some reason creating a device with the name "Pause" seems to cause odd behavior. "Turn on Pause" works, but "Turn off Pause" does not. This appears to be an issue with Echo and not the bridge. I successfully added device 20 and all is well. I'll recreate an issue if I detect issues past 23.


v2024.6.12 -mapping issue

What happened?
This error originated from a custom integration.
Logger: custom_components.smart_irrigation
Source: custom_components/smart_irrigation/init.py:680
integration: Smart Irrigation (documentation, issues)
First occurred: 12:39:18 PM (1 occurrences)
Last logged: 12:39:18 PM
Unable to find mapping with id in update all: 1
How to reproduce
see above
Relevant log output
No response
Which version are you running?
v2024.6.12
Diagnostics file

[X] I have attached a diagnostics file

Additional information
No response

seems like just a configuration issue


Add return value for synchronous callbacks

calling into Haskell from JS is a common task, and often a return value is needed in some way. Synchronous callbacks don't return a value currently, but they could. This would require converting the signalling mechanism for blocking to an exception.
This should probably be added to the ghcjs-base library, with some documentation on the limitations of synchronous tasks.

done

Thank you so much -- we're using it already.  :)  We're building a language (vaguely ML-ish, compiles to JavaScript) written in Haskell and compiling the compiler to JavaScript with GHCJS for the browser playground ( http://cruxlang.org/try/ ) and for easy installation via npm ( https://www.npmjs.com/package/cruxlang ).   Just wanted to say thanks for your amazing work on ghcjs!


a function to remove vowels

Summary
A function that takes a string as an argument and returns the string with all the vowels removed.

thanks


Encounter the issue "Waiter StackCreateComplete failed" once execute the "Setup Cloud9 environment" in workshop

Encounter the issue "Waiter StackCreateComplete failed" once execute the "Setup Cloud9 environment" in workshop
1. Backgroud
Region:  us-east-2 (Ohio)
Account: Not Event Engine
I follow the steps in the link -- https://catalog.us-east-1.prod.workshops.aws/v2/workshops/31676d37-bbe9-4992-9cd1-ceae13c5116c/en-US/
2. Error/Symptom
1) Where I encountered the issue:
https://catalog.us-east-1.prod.workshops.aws/v2/workshops/31676d37-bbe9-4992-9cd1-ceae13c5116c/en-US/installation/not-using-ee/setup-cloud9
In the section -- Setup Cloud9 environment, after "copy and past the following commands into the terminal", and execute the command. The error is below in the Cloud9:
=========================================================================================
[cloudshell-user@ip-10-0-23-92 ~]$ curl -O https://raw.githubusercontent.com/aws-samples/one-observability-demo/main/cloud9-cfn.yaml
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 21102  100 21102    0     0   156k      0 --:--:-- --:--:-- --:--:--  157k
[cloudshell-user@ip-10-0-23-92 ~]$
[cloudshell-user@ip-10-0-23-92 ~]$ aws cloudformation create-stack --stack-name C9-Observability-Workshop --template-body file://cloud9-cfn.yaml --capabilities CAPABILITY_NAMED_IAM
{
    "StackId": "arn:aws:cloudformation:us-east-2:0************:stack/C9-Observability-Workshop/ddbe3810-6b1c-11ec-aa07-065ddcde578e"
}
[cloudshell-user@ip-10-0-23-92 ~]$
[cloudshell-user@ip-10-0-23-92 ~]$ aws cloudformation wait stack-create-complete --stack-name C9-Observability-Workshop

Waiter StackCreateComplete failed: Waiter encountered a terminal failure state: For expression "Stacks[].StackStatus" we matched expected path: "ROLLBACK_COMPLETE" at least once
[cloudshell-user@ip-10-0-23-92 ~]$
[cloudshell-user@ip-10-0-23-92 ~]$ echo -e "Cloud9 Instance is Ready!!\n\n"
echo -e "Cloud9 Instance is Readyaws cloudformation wait stack-create-complete --stack-name C9-Observability-Workshop\n\n"
Cloud9 Instance is Readyaws cloudformation wait stack-create-complete --stack-name C9-Observability-Workshop

=========================================================================================
2)Error in the cloudformation stack:
2022-01-02 00:09:31 UTC+0800	C9BootstrapInstanceLambda	CREATE_FAILED	Received response status [FAILED] from custom resource. Message returned: See the details in CloudWatch Log Stream: 2022/01/01/[$LATEST]5907c73b4e594e13a4fbb3da7fe072b5 (RequestId: 0985cb63-9839-46a6-973f-ff9c3d029217)

3)Error info in the cloudwatch:
2022-01-02T00:09:30.228+08:00

Copy
[ERROR]	2022-01-01T16:09:30.209Z	f405d19a-b691-4d29-a965-59e7ef1cae33	An error occurred (IncorrectState) when calling the AssociateIamInstanceProfile operation: There is an existing association for instance i-07f26f6008b399af4
Traceback (most recent call last):
  File "/var/task/index.py", line 43, in lambda_handler
    response = ec2.associate_iam_instance_profile(IamInstanceProfile=iam_instance_profile, InstanceId=instance['InstanceId'])
  File "/var/runtime/botocore/client.py", line 386, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/var/runtime/botocore/client.py", line 705, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.exceptions.ClientError: An error occurred (IncorrectState) when calling the AssociateIamInstanceProfile operation: There is an existing association for instance i-07f26f6008b399af4
[ERROR] 2022-01-01T16:09:30.209Z f405d19a-b691-4d29-a965-59e7ef1cae33 An error occurred (IncorrectState) when calling the AssociateIamInstanceProfile operation: There is an existing association for instance i-07f26f6008b399af4 Traceback (most recent call last): File "/var/task/index.py", line 43, in lambda_handler response = ec2.associate_iam_instance_profile(IamInstanceProfile=iam_instance_profile, InstanceId=instance['InstanceId']) File "/var/runtime/botocore/client.py", line 386, in _api_call return self._make_api_call(operation_name, kwargs) File "/var/runtime/botocore/client.py", line 705, in _make_api_call raise error_class(parsed_response, operation_name) botocore.exceptions.ClientError: An error occurred (IncorrectState) when calling the AssociateIamInstanceProfile operation: There is an existing association for instance i-07f26f6008b399af4

4)To be honest, I have tried this in the other region us-east-1, us-east-2, I encountered the same issue.

screenshots:
cloud_shell_error
cloudformation_error
cloudwatch_error

screenshots:
cloud_shell_error
cloudformation_error
cloudwatch_error

Could not reproduce or haven't heard from others. Closing this issue. Please update this and reopen the issue if you still see it.


Update download-and-install-windows-powershell-5-1.md

#3728
Fixed link.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.


fix: clarify validate konnect behaviors

Few quick fixes around Konnect behavior:

The help description for gateway validate vs file validate was incorrect (said both would contact a running CP to validate, file validate does not do this, gateway validate does).
Message about the konnect section being missing has been changed to a warning (as you can specify konnect cli flags still, and it now returns a 0 exit code for automation purposes) and the message has been clarified.
--konnect-compatibility and --parallelism flags gated to their relevant parent commands (file validate and gateway validate respectively).

@johnharris85 Could you please check why the tests are not running for this particular PR?


[Feature] Add linkedin profile

Description
Hey, @rohansx It would be nice if we add the LinkedIn profile of the project admin in our team cards next to the name of the admins. Please assign me this issue if you think it'll be appropriate Thnak you!
Screenshots

Checklist

[X] I have checked the existing issues
[X] I have read the [Contributing Guidelines]
[X] I have read the discussion tab thoroughly and got the project idea
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

Actually #1136 is the similar issue and your's was assigned later. Still can approve for twitter links!
Requesting to change title.


Cesium Looks Good To Me

The page https://lgtm.com/ (Looks Good To Me) performs code analysis on various open source projects.
The good news: Cesium is one of the projects with the highest code quality:

This grade refers to JavaScript projects, and for now, let's skip the discussion about what that means, exactly, and instead, just celebrate that for a moment:
🎉
However, there are some "alerts" (i.e. "possible issues") raised by this system. The list is at https://lgtm.com/projects/g/CesiumGS/cesium/alerts/?mode=tree
Most of them are "Useless assignment to local variable". Some of them are caused by the pattern of assignments like result = CartesianX.compute(..., result). These do make make sense under certain conditions, but ... not always (and it's hard to know which of them make sense, actually...)
Many of them are "Superfluous trailing arguments", which are (uncontroversially) "wrong".
Others might be worth a closer look. There certainly will be "false positives" (e.g. from tests, or Sandcastles). But some of these alerts could actually be the reasons for known (or yet undiscovered) bugs.
It might be worth considering to do a pass over these alerts, and see whether there are some that could trivially be fixed, or raised into dedicated issues.
Yes, the title is a clickbait. Repository maintainers HATE this trick...

lgtm.com has been closed, apparently acquired by GitHub (aka Microsoft), and... now the scope of the analysis has been narrowed down, to a point where there doesn't seem to be any notion of "quality" any more. Soooo ... what now? Well, we'll have to discover all the pending bugs manually, or ... try to apply something like https://codeql.github.com/ with all the custom rules that lgtm.com already had in place...
In any case: This issue does no longer make sense.

@ggetz I was actually reminded of this issue by a recent PR ( https://github.com/CesiumGS/cesium/pull/11206 ). I had seen the forum question, and ... zoomed ... into the code quite a bit, but didn't notice that missing zooming parameter there.
I think that many of the quirks of JS revolve around code being wrong in a way that could trivially be detected, but all the tools are just shrugging and saying "I don't know what a 'type' or a 'method signature' is".
Specifically: That 'lgtm.com' site did warn about some functions in CesiumJS being called with unnecessary parameters - and I wanted to look whether it was also able to detect missing parameters.
(I think that if some parameter is supposed to be undefined, then the function should explicitly be calledWith(undefined). Just leaving it out is misleading and error prone. And I'm reasonably sure that there are tools that can detect stuff like this)


Revert "Fix issue with pointerlock movement not working"

Reverts BabylonJS/Babylon.js#10942

cc @PolygonalSun


Change max_dim to maxdim

Ripser.py uses the keyword maxdim while cechmate uses max_dim. We should standardize this across both projects. Since Ripser.py has a userbase already, I vote we change cechmate.

Yes, definitely!
On Wed, Apr 17, 2019, 1:31 PM Nathaniel Saul<EMAIL_ADDRESS>wrote:

Ripser.py uses the keyword maxdim while cechmate uses max_dim. We should
standardize this across both projects. Since Ripser.py has a userbase
already, I vote we change cechmate.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/scikit-tda/cechmate/issues/6, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABNh5sN_a17KLpong4CRDaHpFbV-d-6Fks5vh1qAgaJpZM4c1lSF
.

As a related issue, cech and alpha don't have max_dim in their constructor; only base does


warpsql/docker-compose: Added pgwatch2 service

Added pgwatch2 service to the Docker Compose file.
Configured pgwatch2 to use the cybertec/pgwatch2-postgres:latest image.
Exposed port 3000 for the pgwatch2 service.
Set up PW2_ADHOC_CONN_STR environment variable for pgwatch2 to connect to the timescaledb service.
Configured pgwatch2 to restart automatically in case of failure.
Added timescaledb as a dependency for pgwatch2 to ensure timescaledb starts before pgwatch2.

Fixes: #52
CC: @singhalkarun

Hey @jayanth-kumar-morem this looks good. Let's also move the POSTGRES_USER and POSTGRES_PASSWORD to environment (let's also add a sample.env for the same)

Yes @singhalkarun , I beleive we should clean this. I'll create an another ticket to clean this as its unrelated cleanup to the current one.

Yes @singhalkarun , I beleive we should clean this. I'll create an another ticket to clean this as its unrelated cleanup to the current one.

Sure, merging this one.


Add setTabColor and resetTabColor actions

Summary of the Pull Request
Adds a pair of ShortcutActions for setting the tab color.

setTabColor: If color is null or omitted, this will open the color picker. If this is set to a value, this will set the tab color to that color immediately.
resetTabColor: Immediately resets the color of the focused tab.

FOR DISCUSSION: Should it instead be setTabColor(null) = "reset the tab color", and showTabColorPicker = "show the tab color picker"?
References
PR Checklist

[x] scratches my own itch
[x] I work here
[x] Tests added/passed
[x] https://github.com/MicrosoftDocs/terminal/pull/69

Detailed Description of the Pull Request / Additional comments
Validation Steps Performed

hey look there are tests
Tested with the following:

        // { "command": "setTabColor", "keys": [ "alt+c" ] },
        { "keys": "alt+c", "command": { "action": "setTabColor", "color": "#123456" } },
        { "keys": "alt+c", "command": { "action": "setTabColor", "color": null} },
        { "keys": "alt+shift+c", "command": "resetTabColor" },

FOR DISCUSSION: Should it instead be setTabColor(null) = "reset the tab color", and showTabColorPicker = "show the tab color picker"?

+1 on what you put up for discussion. Then we can remove resetTabColor.
Side Note 1: when we/you implement the keybinding for renaming a tab, it should be the same behavior too.
Side Note 2: It'd be neat if we could set color to an array of values. Then multiple invocations of setTabColor cycles around the array. This would obviously be a separate PR, but I think it's a design that makes sense and could be applied to several other keybindings (like a setSettings keybinding, idk).

@carlos-zamora tab renaming is actually already in PR over at #6557, that's what inspired me to put this together 😆
I want to make sure that we pick a consistent style for both. Right now, I'm definitely leaning away from my current proposal and towards the setTabColor/openTabColorPicker & renameTab/openTabRenamer style, but lets get some more opinions before we commit to either.

Spoke with @DHowett verbally, and he agreed the setTabColor/openTabColorPicker & renameTab/openTabRenamer style was better, so I'm gonna change it to that one.

I guess update the PR name/comment and fix that misspelling but I'll trust you'll do that...


Create array lists with a capacity when it's known

Parceler writes the size of lists but does not pass the size of the list when creating a a new ArrayList.

Ah, that would be a nice touch...

I added it for all the applicable collections.  Let me know what you think.


Column Types Reference?

I am working on custom JSON formatting for a SharePoint site. I am utilizing internal column types in the custom view. Is there anywhere for me to see the schema for these fields? For example, I can get the name of a Person column type by "[$Person.title"], but I found that randomly. How would I get the image associated? Is there some kind of reference page for this, or how are we supposed to know the internal fields?

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

ID: f60068c3-4534-a400-350b-fdbcefd684b6
Version Independent ID: 228accdb-a7de-0671-34ad-60a37866c380
Content: Use column formatting to customize SharePoint
Content Source: docs/declarative-customization/column-formatting.md
Product: sharepoint
GitHub Login: @VesaJuvonen
Microsoft Alias: vesaj

Hi. You can go to List Settings, and click on the column you want. The internal name of the column will be displayed in the corresponding URL after the &Field= parameter.

Thanks. I have utilized that trick before to identify the internal name. However, this does not capture the internal fields of an internal name. Here's what I mean:

We have a field, 'Contact' which is an O365 user/group. The internal name is 'Contact'.
However, to access the Contacts name, you must reference 'Contact.title'. I found this by guessing semi-randomly. I am unsure how to get the Contact profile photo. I have tried 'Contact.image, Contact.picture, Contact.url' and various others. I would like a link to some sort of reference or some way to find the name of these fields without guessing.

This information can be found at the bottom of the Use column formatting to customize SharePoint article, under the section "@currentfield".
For example, the article provides a definition for the People object you're referring to:
{    "id": "122",    "title": "Kalya Tucker",    "email"<EMAIL_ADDRESS>   "sip"<EMAIL_ADDRESS>   "picture": "https://contoso.sharepoint.com/kaylat_contoso_com_MThumb.jpg?t=63576928822",    "department":"Human Resources",    "jobTitle":"HR Manager" }


Adding Push Notification functionality to the Application

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
As the application we are targetting will be serving a lot amount of people and may be able to change their lives in developing themselves throught our content  so in this case , I came up with this below feature :
Describe the solution you'd like
A clear and concise description of what you want to happen.
I would like to implement the feature of Push Notification to keep the User of the Application Active ,  about  :

New Updates to the Application
Progress of his/her enrolled course
Messages from any course- mate or associated teacher or mentor
Profile Updations
Important Deadlines
Schedule of the next class

Similarly , many things can be conveyed to the user through Push Notifications
I request all the project maintainers , mentors and contributors to give their views and reviews on this feature request of mine
Additional context
Add any other context about the feature request here.

@Abhishek-kumar09  Can you please review this ?

Sounds good. Nice feature to look upon.
We can use firebase push notifications or maybe anything else, it just came up to my mind right now.
Have to figure out how the feature can be implemented with capacitor/ionic

Yeah sir @Abhishek-kumar09  Firebase was the option that was coming to my mind while thinking of this feature
But once it's implemented , it will definitely give a boost to the application

@Abhishek-kumar09  Can you please give your reviews on #211 also


ci(.github): add dispatch workflow

Adds a dispatch.yml GH workflow for detecting file changes and dispatching an event as appropriate
First use case is to detect doc updates and then dispatch an event to a destination repo (which handles (re-)deploying the docs site)

Tested on fork: https://github.com/vdice/spin/actions/runs/2111223700

(Note: build failing due to unrelated issue)


Extend github actions to run all tests and build, and to run on PRs t…

…o any branch
Closes #20 #21.

Amazing, thanks.


Request: Allow feat level above softcap

Various leveled feat in the game can be added with toybox, but their level is softcapped by the same limit a regular character/class normally allow. Because the engine support higher values with no issue (tested with memory alteration programs like CE), allowing to remove the softcap and increase the feats above the normal limit seems a reasonable option to add.
Example being 'Weapon Training' normally softcapped at 10, but altering the value to 40 register perfectly with no problem, same for 'Armour Training', tested up to 100 with no issue.

After 2 weeks i remembered this request and like to clarify how this is not a feature that would require adding the extra level of the feats above the default cap, those feats are added trough a progression table and therefore any valid N value is technically functional as it is, the request is limited in allowing the toybox to set a value higher then the game normally allow to reach
I understand that for some feat it could bring problems but for most repeatable feats i tried it work just fine, adding a small check to allow the toybox to increase the level of feats above the softcap threshold should be sufficient warning, perhaps marking it as experimental for the time being.


[virtualbox] Multiple machines have the same IP address

Through some (don't remember) combination of stop/restart Docker Machine, duplicate IP addresses are assigned to two machines now. docker-machine ls shows:
> docker-machine ls
NAME      ACTIVE   DRIVER       STATE     URL                         SWARM
default   *        virtualbox   Running   tcp://<IP_ADDRESS>:2376
lab       *        virtualbox   Running   tcp://<IP_ADDRESS>:2376

Ouch, this is totally broken!

shut one down and bring it back up... does that solve the issue?

Other users have reported, this duplicate ip assignment.
But we lack more evidences of how it happens.
@arun-gupta Closing for now, re-open if you manage to reproduce it.

Duplicate of #1892


Exit game button has artifacting (playing in 4k)

(actually applies to any button that is in that spot)

Manually edit the Interface Resolution in options.txt to 1.0 and see if this continues (Idk if you changed this)
Also, were there artifacts in 4k resolution in the previous version?
And please don't report anything related to artifacts on GitHub, "fixing" this is out of my scope as I don't have all the screen resolutions, the only thing I can do is make small tweaks.

Manually edit the Interface Resolution in options.txt to 1.0 and see if this continues (Idk if you changed this) Also, were there artifacts in 4k resolution in the previous version? And please don't report anything related to artifacts on GitHub, "fixing" this is out of my scope as I don't have all the screen resolutions, the only thing I can do is make small tweaks.

Changing this has corrected that specific issue, but now most things appear slightly too small and other things simply look off. This was not an issue in the previous update

Manually edit the Interface Resolution in options.txt to 1.0 and see if this continues (Idk if you changed this) Also, were there artifacts in 4k resolution in the previous version? And please don't report anything related to artifacts on GitHub, "fixing" this is out of my scope as I don't have all the screen resolutions, the only thing I can do is make small tweaks.

Changing this has corrected that specific issue, but now most things appear slightly too small and other things simply look off. This was not an issue in the previous update

However, Changing this setting, then modifying the same setting in game to "100%" has fixed my issue

Interesting, I applied the tweak in 4k resolution because someone told me that in this resolution there were artifacts in the previous versions, it will be removed in the next version


Create angellist.data.json

Process the startup job postings at AngelList, and store the results in angellist.data.json.

https://github.com/radgrad/internbit-scraper-mm-puppeteer/blob/master/angellist.canonical.data.json


dev and preview produce unhelpful error message when bucket directory is missing

dev and preview output Error: No such file or directory (os error 2) without any further indication on what's wrong. After lot of trial and error I ran publish which actually produced a meaningful error message that helped me solve the issue.
% wrangler build
✨  Built successfully, built project size is 494 KiB.

% wrangler dev
🌀  Using namespace for Workers Site "…"
Error: No such file or directory (os error 2)

% wrangler preview
🌀  Using namespace for Workers Site "…"
Error: No such file or directory (os error 2)

% wrangler publish
✨  Built successfully, built project size is 494 KiB.
Error: ⚠  bucket directory "release/browser" does not exist

wrangler 1.12.3

Yes! The issue is we need to run npm export - hope this helps someone in future!

Thanks for creating this issue! Wrangler v1 is now deprecated and support is only being provided for critical updates or security concerns. As such, we are closing this PR.
New versions of Wrangler are maintained in the workers-sdk repo. If you are running into a similar issue with wrangler v2, please report it or create a new PR in the workers-sdk repo. For more info about wrangler v1 deprecation, please check out our blog post.


Code completion support for Flutter @macros

The code completion help for a symbol does not use the substituted @macro text for Flutter elements. Flutter defines @macro text in elements in their dartdoc and then references the text in the docs for other elements (a simple textual substitution.
There is support for @macro substitution in hover text, but not for code completion docs.

/cc @bwilkerson, @devoncarew

@bwilkerson - is this something you could take on?
Do you have a sense whether this would have any performance cost - something to be aware of, or likely a non-issue?

Yes, I'll take a look at it.
I don't know what the cost is, but (if we haven't already done so) we should be able to write it such that if the data we need in order to perform the substitution isn't already available we will just degrade to not performing the substitution rather than wait for the data. The substitution itself is a simple string manipulation, so it shouldn't be expensive.

https://dart-review.googlesource.com/c/sdk/+/143322
Let me know if you find any cases that were missed, but to the best of my knowledge macros should now be getting substituted correctly.


Link for non-DockerHub tools is incorrect

Describe the bug
If a workflow references a non-DockerHub Docker image, the link incorrectly has hub.docker.com as the domain.
To Reproduce
Steps to reproduce the behavior:
curl -s 'https://dockstore.org/api/workflows/5874/tools/13615' -H 'Accept: application/json' | jq '.'
[
  {
    "id": "icgc_pcawg_dkfz_workflow",
    "file": "dkfz/icgc_pcawg_dkfz_sbg.cwl",
    "docker": "images.sbgenomics.com/bogdang/pcawg-dkfz:1.0",
    "link": "https://hub.docker.com/_/images.sbgenomics.com/bogdang/pcawg-dkfz"
  },
  {
    "id": "pcawg_icgc_delly_workflow_1",
    "file": "delly/icgc_pcawg_seqware_delly_sbg.cwl",
    "docker": "images.sbgenomics.com/bogdang/pcawg-delly:10072019",
    "link": "https://hub.docker.com/_/images.sbgenomics.com/bogdang/pcawg-delly"
  }
]

Note how the links are to DockerHub, even though the images are hosted on SB.
Visible in the UI by navigating to https://dockstore.org/workflows/github.com/sbg/sbg_dockstore_tools/icgc_pcawg_dkfz_embl_sbg_modified:master?tab=tools
Expected behavior
Links should be to SB Docker image repository.
Additional context
Happens on prod and dev. Dev info:
Webservice - 2589683
UI - 2.6.1-42-g3b3889cf
Compose Setup - 8c083bddfa2e8cfcb517792d45b03b95dc3dda5d
Deploy - 1.9.2-24-g3fa31af
┆Issue is synchronized with this Jira Story
┆Fix Versions: Dockstore 1.9.X
┆Issue Number: DOCK-1588
┆Sprint: Sprint 45- Trumpetfish
┆Issue Type: Story

Current logic for making the link doesn't yet support non-DockerHub/Quay entries (the code just reaches the last else clause):
https://github.com/dockstore/dockstore/blob/96424cf3dc0f7ca623efc17670613db659242b55/dockstore-webservice/src/main/java/io/dockstore/webservice/languages/LanguageHandlerInterface.java#L361
A simple solution for now could be to check if the entry looks like a URL with regex?

Current logic for making the link doesn't yet support non-DockerHub/Quay entries (the code just reaches the last else clause):
https://github.com/dockstore/dockstore/blob/96424cf3dc0f7ca623efc17670613db659242b55/dockstore-webservice/src/main/java/io/dockstore/webservice/languages/LanguageHandlerInterface.java#L361
A simple solution for now could be to check if the entry looks like a URL with regex?

Didn't do much with it, but wrote some code that tries to detect the other image registries we support as an FYI https://github.com/dockstore/dockstore/blob/96424cf3dc0f7ca623efc17670613db659242b55/dockstore-webservice/src/main/java/io/dockstore/webservice/languages/LanguageHandlerInterface.java#L373

I may be oversimplifying this, but when discussing with Andy and we were trying to figure this out for the spreadsheet, it seems that if the image has one slash in it, it's DockerHub; if it has two, it's not (unless they redundantly specify DockerHub).

In dev, the link is no longer DockerHub and is to sbgenomics.
However, the http link for SB doesn't work, and per this page, SB does not have a web page, so there is no place to link to. Options:

It's OK as is
Link should be null
?

In dev, the link is no longer DockerHub and is to sbgenomics.
However, the http link for SB doesn't work, and per this page, SB does not have a web page, so there is no place to link to. Options:

It's OK as is
Link should be null
?

Both (1) and (2) make sense in their own ways.
My case for (1) is that assuming the link works in the default case is probably a good thing for using new/unsupported registries.

Going to close. We can revisit link discussion if it comes up from a user, otherwise, I think it's good for now.


Unnecessary indentation in JS when line starts with semicolon

If I have JS code like this and save the file, because I've got automatic formatting on save enabled it will indent the second line (because it starts with a semicolon), which should not happen.
const a = 'b'
;(() => { console.log('c') })()

That's the default of TS formatter.

Opened https://github.com/Microsoft/TypeScript/issues/16909 to track this upstream


made header positioned absolute

this pull request redoes the positioning of header elements to make them absolute

This pull request adds the changes from safari debug work into the main brance


[Elasticsearch 2] Renamed search backend classes

Suffixing them with the word Backend

It's odd. Some of build jobs were failed. I don't think that the reason in this PR. I've rebased cleanup/search-class-names branch on top of master.

I don't understand... I've pushed a copy of this branch into my fork and the build is green, but in the main repository build fails (again!).
It looks like Travis tries to build c30f9c6c18e370852ee8246e10b34641ea63d7ad commit, but I have not idea why: there is no such commit in this branch!

Rebased on top of master (#2567, #2568 were merged into master).

Rebased on master

@m1kola
Looks like the tests are fixed now. Are you happy with the deprecation warnings?

@kaedroho yep, I think we can merge this PR ;)

Thanks @m1kola!


Port voltage tuning algorithm limitations

Following initial tests on both the 64-channel and RHS2116 headstages, the current port voltage auto-tuning algorithm implemented in ConfigureFmcLinkController was found to have significant limitations.
For the 64-channel headstage, the link voltage settle times were found to be on the order of at least 4000 milliseconds. Given the scale of these time intervals, this will significantly impact the time it takes to do port scanning. Since these are electrical properties specific to individual headstage design, they should probably be exposed as additional configurable parameters in the link controller itself so they can be tuned per headstage.
For the RHS2116 the minimum voltage for establishing the SERDES lock (1.8V) is too low to enable full headstage functionality (minimum is 5.0V). This can be partially addressed by the VoltageOffset property, but it seems like there are additional limitations on minimum port voltage for resetting the headstage, as well as a potential requirement to do an extra context reset before the headstage can be found, both of which require further investigation.

Given the widely different port voltage scanning algorithms which have been empirically discovered, we discussed making the FMC link controller into an abstract base class where each headstage will be able to implement its own specific bespoke logic for configuring the port voltage without impacting implementation details of other headstages.


AnchorJS should not be active in live examples

For example, it's currently active on the example panel titles in http://getbootstrap.com/components/#panels-alternatives
CC: @XhmikosR

Unfortunately, there doesn't seem to exist any way currently to skip specific selectors. Maybe we could submit a patch upstream?

Also, we really need https://github.com/twbs/bootstrap/tree/bump-anchorjs. It has an important change like avoiding duplicate IDs which we all know are invalid.

I submitted a PR to add unwanted selectors on anchorjs
https://github.com/bryanbraun/anchorjs/pull/33

@cvrebert couldn't we just add this to our docs css in the short term:
.bs-example .anchorjs-link {
  display: none !Important;
}


Remove instance Foldable

Or is it a bad idea, given that most of the folds diverge? On the other hand, we already provide instance Foldable.

I think it's better to remove instance Foldable, but provide foldr :: (a -> b -> b) -> Chimera a -> b similar to infinite-list.

I'd rather avoid a breaking change right now.


IO-747 Make commons-io a proper JPMS module

This PR is similar to https://github.com/apache/commons-fileupload/pull/107
If commons team likes the approach I could work on a PR that moves the logic to commons-parent and is reused for all commons-** projects.

Coverage decreased (-0.1%) to 89.122% when pulling cc02a8c2cc68ec6461146c8c1b550aa32923e34a on martin-g:io-747-make-commons-io-proper-jpms-module into 70f3ce0b23de3180cb7e821e6792f086616e951a on apache:master.

Thank you @martin-g for raising this! We also need JPMS-zed version of commons-io and commons-collections4.
There are some forks like guicedee and jwebmp at Maven Central but it is time the originals to evolve too!

Why does this patch not follow the recommeded pattern from https://maven.apache.org/plugins/maven-compiler-plugin/examples/module-info.html ?

I used Moditect because it generates the module-info.java. With maven-compiler-plugin module-info.java must be in src/main/java, i.e. require JDK 9+ for development.
Also I am not sure how to setup toolchains in GitHub Actions.
I don't mind closing this PR if someone provides different way to achieve the end goal!

One minor nitpick: As I understand it, there is now a hardcoded version 8 for the Java compiler. IIRC, there is a property in commons-parent for that. Please, change that to use that property. Thanks!

@jochenw I guess you mean https://github.com/apache/commons-parent/blob/dd3e4bc64628c0e1ae15cf16a48a04bdfd8dbcc9/pom.xml#L109-L110
Those are overwritten here - https://github.com/apache/commons-io/blob/319a5404b657e3859bad4216fb6ca499e072b18d/pom.xml#L297-L298
But there is no such for maven.compiler.release which uses 8 as a value vs. 1.8 used by source and target.

But there is no such for maven.compiler.release which uses 8 as a value vs. 1.8 used by source and target.

If so, you can override the property maven.compiler.release with the value "8". Should work fine, as far as I know.

FYI: This is how we use toolchains over at Log4j, see the master and release-2.x branches: https://github.com/apache/logging-log4j2/tree/release-2.x/.github/workflows

If so, you can override the property maven.compiler.release with the value "8". Should work fine, as far as I know.

@jochenw I don't follow you! There is no mention of maven.compiler.release in the parent pom, so there is nothing to override.
In this PR I set <maven.compiler.release>8</maven.compiler.release> in the JDK 9+ profile. I cannot move it to the common properties because javac 8 will complain that it has not a parameter --release.


os: process.Wait marks the process as already finished on ptrace stop signals on Linux

What version of Go are you using (go version)?

$ go version
go version go1.20.4 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/mitar/.cache/go-build"
GOENV="/home/mitar/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/mitar/.go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/mitar/.go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.20.4"
GCCGO="/usr/bin/gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-O2 -g"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build3740724317=/tmp/go-build -gno-record-gcc-switches"

What did you do?
https://go.dev/play/p/PYGxV0Kg3pF
What did you expect to see?
2009/11/10 23:00:00 status: true 5
2009/11/10 23:00:00 end

Or nothing, if process.Wait returns only on exit status and not stopped status.
What did you see instead?
2009/11/10 23:00:00 status: true 5
2009/11/10 23:00:00 os: process already finished

Thanks for the bug report.  I don't understand why this is happening.  We call waitid with WEXITED, which is documented as waiting for children that have terminated.  Why does it return for a child that has been trapped but not terminated?
strace -f reports
pid 557792] waitid(P_PID, 557797, {si_signo=SIGCHLD, si_code=CLD_TRAPPED, si_pid=557797, si_uid=100, si_status=SIGSTOP, si_utime=0, si_stime=0}, WEXITED|WNOWAIT, NULL) = 0

but why does waitid return?

A good catch. Go uses waitid in blockUntilWaitable which does return only on process exit, but then uses syscall.Wait4 in Process.wait itself, which does return for both exited and stopped. This is why I used process.Wait call in the first place, because I thought that I will be getting exited and stopped (which I do). But this seems to break some assumptions in the code.
So one way to fix this is to use waitid instead of syscall.Wait4 and really wait only for exited processes in both places (given that we figure out why waitid returns for stopped processes as well).
BTW, I could not find anywhere explicitly documented that waitid does not return for stopped processes. In Linux man page for waitid it is unclear what is the behavior here, because for waitpid it says:

WUNTRACED --- also return if a child has stopped (but not traced via ptrace(2)). Status for traced children which have stopped is provided even if this option is not specified.

So maybe the same "Status for traced children which have stopped is provided even if this option is not specified." holds for waitid as well?
BSD seems to have clearer options here:
WTRAPPED	 Report	the status of selected processes which are being
		 traced	via ptrace(2) and have trapped or reached a break-
		 point.	 This flag is implicitly set for the functions wait(),
		 waitpid(), wait3(), and wait4().
		 For the waitid() and wait6() functions, the flag has to be
		 explicitly included in	options	if status reports from trapped
		 processes are expected.

BTW, I could not find anywhere explicitly documented that waitid does not return for stopped AND traced processes.

Isn't that required by POSIX?
https://pubs.opengroup.org/onlinepubs/9699919799/functions/waitid.html:

The options argument is used to specify which state changes waitid() shall wait for. It is formed by OR'ing together the following flags:
…

It goes on to describe WCONTINUED, WEXITED, and WSTOPPED, and to state that “[a]pplications shall specify at least one of the flags WEXITED, WSTOPPED, or WCONTINUED to be OR'ed in with the options argument.” Nothing there about ptrace because ptrace isn't in POSIX.
https://pubs.opengroup.org/onlinepubs/9699919799/functions/wait.html does give some detail about ptrace, but only to say:

Most applications do not need to concern themselves with such extensions because they have control over what extensions they or their children use. However, applications, such as command interpreters, that invoke arbitrary processes may see this behavior when those arbitrary processes misuse such extensions.

At any rate, I wonder if the Linux waitid behavior was broken by the patch described in (and linked from) https://lwn.net/Articles/688624/.

@bcmills Nice find.  Seems like on Linux we can't rely on waitid to do what we need.    I wonder if using pidfd_open would fix that problem (#60320)?

A nice find indeed. Now that I know what to look for it is interesting that this man page says nothing about it, but this one does:

Since Linux 4.7, the __WALL flag is automatically implied if the child is being ptraced.

I made a draft in #60461 which addresses this. See Gerrit for some of my comments. It is not yet perfect but it does fix the issue here.

Interesting; the man-pages update containing that line was commited back in 2016, so the pages on linux.die.net must be quite old. 😵💫
(https://man7.org/linux/man-pages/man2/waitid.2.html also includes the update.)

I just finished the comments now.

So while working on this I think there are two ways to address this: pidfd helps here because it allows us to really detect exit and just exit so we do not have to call wait unnecessary. But another solution is to simply expand cases when we loop around wait and do another wait if wait returned for any other reason than exit/signal (we could do such a loop in blockUntilWaitable as well, which is not necessary with pidfd approach).

A loop sounds good if it works, but would it work?  If a process is stopped due to ptrace will waitid immediately return a notification that the process is stopped?  In other words, would a loop just be a busy wait?

(This is helpful behavior for this particular problem, but for my own init program in fact means that only one wait in the process succeeds and if there are multiple waits for the same process (e.g., one by pid and one using -1, like a wait loop to reap unknown reparented processes to the init), it is arbitrary which one will succeed and which one will stay blocked. And there is no timeout to blocking.)

I have read through kernel code and I do not think that the patch linked above broke this. That patch deals with the question if threads or not are waited on by default. And if thread is ptraced, then it is waited on.
From looking at the code it seems that wait simply always waits on all signals from ptraced processes. I also tested this by calling waitid with WEXITED|__WALL options and sent a SIGSTOP signal. Wait didn't exit on that. But it did exit if I added WSTOPPED to options. So I think BSD has it better here where they provide WTRAPPED.


Improve type information for react-dnd

Updated the react-dnd definitions based on existing TypeScript
definitions from DefinitelyTyped. Type definitions are now complete
based on react-dnd documents.
Added unit tests exercising this functionality.
Also fixed incorrect signatures for DragLayer.

cc @gaearon

Nice!

@thejameskyle Two questions:

What's the right way to define a higher-order component in flow? Is the following the right way to type a higher-order component?

The DragSource is a higher-order component that takes three arguments and it transforms a component C into a draggable componet (typed as DndComponent).
type DragSource = <P, C: React$Component<any, P, any>>(
  type: DragSourceType<P>,
  spec: DragSourceSpec<P>,
  collect: DragSourceCollector,
  options?: DndOptions<P>
) => (component: Class<C>) => Class<DndComponent<P>>;

declare class DndComponent<P> extends React$Component<any, P, any> {
  static DecoratedComponent: Class<this>;
  getDecoratedComponentInstance(): this;
  getHandlerId(): Identifier;
}

How to test the types of the two instance methods on DndComponent? As an example, dk below is an instance of the DndKnight component. Flow doesn't like dk.getHandlerId() because the type of dk is React$Element<{}>.

const DndKnight = DragSource('knight', knightSource, knightCollect)(Knight);
const dk = <DndKnight />;

@thejameskyle @gaearon
I've been trying to get the DecoratedComponent (named DndComponent below) in react-dnd to type check. React-dnd extends React's Component class with methods such as getDecoratedComponentInstance and getHandlerId. These methods are then used on a ReactElement. See here for an example.
I started with something like this:
declare class DndComponent<D, P, S> extends React$Component<D, P, S> {
  static DecoratedComponent: Class<this>;
  getDecoratedComponentInstance(): this;
  getHandlerId(): Identifier;
}

But this is not getting me anywhere, because as far as I can tell, the React and Flow definitions do not allow changing the result type of React.createElement beyond passing in a Config, which only affects the props.
I think to support this, it would be necessary to add a parameter to ReactComponent for the "extension" methods. It may also fit on the Config. Something like:
// Extensions to React$Element enabled by react-dnd:
type DndExt<C> = {
 getDecoratedComponentInstance : () => C
 getHandlerId: () => Identifier
};

// Additional parameter to React$Component specifies these extensions:
declare class DndComponent<D, P, S> extends React$Component<D, P, S, DndExt<this>> { ... }

// Type of React.createElement changes
// from React.createElement : ReactClass<C> -> ReactElement<C>
// to React.createElement : ReactClass<C, X> -> X<ReactElement<C>>
// Where X is a higher-order type that adds additional methods to the ReactElement.

Since the DecoratedComponents are only used in tests, I've dropped them from the type definitions for now.

React-dnd extends React's Component class with methods such as getDecoratedComponentInstance and getHandlerId. These methods are then used on a ReactElement.

Hmm, I don’t think this is correct. These methods are defined on the component instance, not element. So it shouldn’t actually involve ReactElement.

In other words, in your example, those methods should be on class DndComponent, not the element it uses.

Gotcha @gaearon. Fixed in the last check-in. Added (back) the DndComponent and ContextComponent classes and parameterized them over types of default props, props and state. DndComponent for DragSource, DropTarget and DropLayer. ContextComponent for ContextComponent.

@thejameskyle @gaearon does the PR look good to merge?

Looks good to me, and we can always improve this in the future!

Great work by the way, thank you so much for taking the time to do that (even for undocumented methods). 💯

Ping @thejameskyle  +  @marudor, the original author of the react-dnd flow definition.
Is this PR ready to go in?


Issue with Azure Storage Emulator, VSO and Integration Tests

Firstly please accept my apologies if this is not the right place to ask this question...
We have some integration tests that relies on Azure Storage Emulator which fail when we run them using VSO Build with the following:
-------- System.Net.Sockets.SocketException : No connection could be made because the target machine actively refused it <IP_ADDRESS>:10002
These test for fine on our local machine(s). I'm suspecting that it's something to do with loopback adapter or such like as I've had problems like this before with server's which have been hardened as I assume the environment hosting VSO build agent's are.
Does anyone have any ideas on how I could run the Azure Storage Emulator successfully using hosted VSO agent ?

We don't support running the azure storage emulator on the hosted build agent. It requires higher privileges than what we allow for the buildguest on those machines.
Sent from my Windows Phone

From: John<EMAIL_ADDRESS>Sent: 8/25/2015 6:55 PM
To<EMAIL_ADDRESS>Subject: [vso-agent] Issue with Azure Storage Emulator, VSO and Integration Tests (#72)
Firstly please accept my apologies if this is not the right place to ask this question...
We have some integration tests that relies on Azure Storage Emulator which fail when we run them using VSO Build with the following:
-------- System.Net.Sockets.SocketException : No connection could be made because the target machine actively refused it <IP_ADDRESS>:10002
These test for fine on our local machine(s). I'm suspecting that it's something to do with loopback adapter or such like as I've had problems like this before with server's which have been hardened as I assume the environment hosting VSO build agent's are.
Does anyone have any ideas on how I could run the Azure Storage Emulator successfully using hosted VSO agent ?
—
Reply to this email directly or view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fvso-agent%2Fissues%2F72&data=01|01|Chris.Patterson%40microsoft.com|666b8dc2ce1448a05a9708d2ada03a55|72f988bf86f141af91ab2d7cd011db47|1&sdata=zXQ3216BLA4T8xur1%2FHYHt5Dy4t0tJR2J2fSCC4hM1w%3D.

@chrisrpatterson - Thanks for confirming, I suspected as much and its understandable. I'll build out a VM with a build agent on it and run them from there.

@chrisrpatterson Just wondering if VSTS build agents still block Azure Storage Emulator? I'm trying to run automated tests in VSTS using Azure storage emulator without setting up a separate build server.


Console spamming errors

Minecraft Version: 1.12.2
Sponge Version: 1.12.2-2768-7.1.5
Forge Version: <IP_ADDRESS>15
Plugin Version: 1.12.2-<IP_ADDRESS>8
Issue: I made a server in my Windows10 pc to test out some plugins and after i join the server console started spaming this errors:

If you wanna see the log file:
latest.log
(I couldn't upload it to pastebin or ftb paste cuz file is too big, sorry)

Did you figure out a fix for this? I am having the same issue


Error when connecting USB device

RancherOS Version: (ros os version)
0.5.0.
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
bare metal.
I'm attempting to use rancher/os on a personal home server. I have a Z-wave pen drive (this one) and it was working fine on a debian installation.
When I plug it in on the machine with rancher/os 0.5.0, I get this on dmesg:
[  390.229144] usb 1-3: new full-speed USB device number 5 using xhci_hcd
[  390.405438] usb 1-3: device descriptor read/64, error -71
[  390.681588] usb 1-3: device descriptor read/64, error -71
[  390.901188] usb 1-3: new full-speed USB device number 6 using xhci_hcd
[  391.073234] usb 1-3: device descriptor read/64, error -71
[  391.292906] usb 1-3: device descriptor read/64, error -71
[  391.509089] usb 1-3: new full-speed USB device number 7 using xhci_hcd
[  391.514439] usb 1-3: Device not responding to setup address.
[  391.721397] usb 1-3: Device not responding to setup address.
[  391.937188] usb 1-3: device not accepting address 7, error -71
[  392.293114] usb 1-3: new full-speed USB device number 9 using xhci_hcd

Any ideas on how to fix this?

After quite a bit of investigation, I found that the rancher-os kernel was simply missing the module cdc-acm, which was needed by my serial USB device.
I've compiled the rancher-os kernel myself for now and have since added a PR to include the module in a future release.

+1. Wish I had known about this before getting everything else running first.


Photoview on roku tv does not pick up all photos

My Photo view only picks up the first 274 photos, then stops and plays them again. The other photos are ones I want in the program also. What am I doing wrong?

Forgot to say it does a good job of only picking one of two photos.....I have duplicates.


SequencerDemo iOS example app crashes

Xcode 10.2.1, both device and simulator.

The AudioKitUI framework was not linked for some reason. Fixed it on the develop branch. Thanks for the heads up.


remove separate twitter bundle; use exception trapping to only regist…

…er AdapterFactory when Twitter4J is available

Upon installation, if Twitter4J isn't available, you get a single log message:
com.adobe.acs.commons.twitter.impl.TwitterAdapterFactory Twitter4J Library not found. Not registering TwitterAdapterFactory.

Note that due to #1570, this isn't actually testable as-is.
Also, this pull request doesn't remove the min packaging. I think that warrants keeping around for legacy reasons.

Coverage increased (+0.03%) to 41.183% when pulling 55ed1c69f8194b9eb246cfa2e7610c05536bb84f on justinedelson:feature/better-twitter-loading into f3ba4f3dbeb23f8f8c92301ab3b2bb053c9fa275 on Adobe-Consulting-Services:master.

Code climate strikes again... 🤕

We're doing a major version bump so we could remove it now, but after 4.0.0 it will be a while before we get another option to consider it.

+1 for removal of the twitter4j bundle, thus eliminating the need for the min bundle

I'll merge this in once the confirmation build passes (which I expect it to)


openvswitch*: fix 3.13 compatibility, apply RFCs, add self

Description of changes
Fix broken tests on python 3.13 using commit from upstream.
Added self as maintainer, moved to by-name and applied nixfmt-rfc-style

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

@ofborg build openvswitch openvswitch-lts


remove duplicate qlog finish_frames() call

@LPardue I didn't test this, but the same call is done like 10 lines above this?

@LPardue it seems to work fine to me... Also, this uncovered another small problem, so added a commit to fix that.

Weird, I'm sure it was broken at one point without that extra call. Maybe then double it got fixed in code review before merging.
I've tested your changes locally and the generated qlogs are all good.


CLI Quick Start instructions have confusing headers and header styles

Describe the bug
On the Dockstore Quick Start instructions page in dev environment https://dev.dockstore.net/quick-start the headers have a few problems that are distracting for the user:

Step 2 appears twice
Step 1 has a different style from Step 2

See screenshot below showing problem.

To Reproduce
Steps to reproduce the behavior:

Visit https://dev.dockstore.net/quick-start
See error

Expected behavior
Header styles should be consistent and Step 2 should not appear twice.
Build info:
Webservice - 6d891ef
UI - 2.8.4-107-g97af6aee
Compose Setup - 5454f4a
Deploy - 1.11.2-106-gb5fc8b9
(dev environment)
┆Issue is synchronized with this Jira Story
┆friendlyId: DOCK-2022
┆sprint: Backlog
┆taskType: Story

Seems to apply to prod as well


it's necessary code for Mocha?

I think I found the old code while analyzing it. It's probably been under 2.x, but I'm still wondering if it's necessary for Mocha, so I'm leaving a message.
runnable.js 385
  function callFn(fn) {
    var result = fn.call(ctx);
    //my custom log
    console.log("ret : ", result)
    if (result && typeof result.then === 'function') {
      self.resetTimeout();
      result.then(
        function() {
          done();
          // Return null so libraries like bluebird do not warn about
          // subsequently constructed Promises.
          return null;
        },
        function(reason) {
          done(reason || new Error('Promise rejected with no or falsy reason'));
        }
      );
    }

that.. "Return null so libraries like bluebird do not warn about"
I think this has something to do with bluebirds, so I did a bluebird example, and that part of it is undefined. So I wonder if it's necessary. Is this not necessary? Or I'd appreciate it if you let me know if you've handled any other exceptions.
test code
const Promise = require('bluebird')
const {expect} = require('chai')
describe('BlueBird AsyncTest', function() {
    it('should be resolved value : 10', function() {
        Promise.resolve(10).then(a =>{
            expect(a).to.equal(10)
        })
    });
});

result

  BlueBird AsyncTest
ret :  undefined
    ✓ should be resolved value : 10
  1 passing (9ms)

@wnghdcjfe
I don't know wether this line of code is needed or not, it certainly does no harm.
IMO this single line is not worth taking any risk by removing it.
So I'm closing ths PR, sorry.


Update time-sync.md

Tweak the text slightly to refer to a PTP clock source, which is what Linux guests have, rather than the PTP protocol, which is related but is really something completely different.

@kelleymh : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@kelleymh Thank you!
#sign-off


Configure firebase through config file

Description
LEARNER-6725
Configure the firebase parameters through app config file.

@saeedbashir ready for another pass.

@saeedbashir fixed comment, ready for another pass.


How to completely disable Flash, while enabling YouTube?

With a site that we are developing we have encountered an issue that when we try to play a Youtube hosted video under IE 11 on Windows 7, the video will consistently attempt to fall back to Flash, crash, and then use the standard Youtube skin (without controls) rather than the desirable html5 video.js player. We discovered that if we disable flash in the IE browser everything works as expected. Unfortunately, as this will be a public website, that will not be a realistic solution.
Our site uses a mixture of self-hosted MP4 videos, MP3 audio files, and YouTube-hosted videos. To completely disable Flash from being used, at the server-level, I've tried every combination of techOrder possible, along with trying to set the techOrder settings without Flash like this:
videojs.options.techOrder = ["html5", "youtube"];
I've also tried everything with and without the techOrder option: "sourceOrder": true
None of my efforts to disable Flash at the server-level have been successful. Note that my efforts also include disabling the video-js.swf file from loading in the first place.
Perhaps an answer to this question might resolve the related VideoJS YouTube plugin issues #342 (Video not playing Internet Explorer), and #361 (Not compatible with latest video.js 5.7.0?).
Here are examples of the latest VideoJS YouTube plugin working (at least when using pure html5 browsers) and various versions of VideoJS libraries:
http://5-0-2.videojs-test-xyz.org
http://5-1-0.videojs-test-xyz.org
http://5-2-4.videojs-test-xyz.org
http://5-3-0.videojs-test-xyz.org
http://5-4-6.videojs-test-xyz.org
http://5-5-3.videojs-test-xyz.org
http://5-6-0.videojs-test-xyz.org
http://5-7-1.videojs-test-xyz.org
And to try these same versions without any influence due to caching:
http://5-0-2.dev.videojs-test-xyz.org
http://5-1-0.dev.videojs-test-xyz.org
http://5-2-4.dev.videojs-test-xyz.org
http://5-3-0.dev.videojs-test-xyz.org
http://5-4-6.dev.videojs-test-xyz.org
http://5-5-3.dev.videojs-test-xyz.org
http://5-6-0.dev.videojs-test-xyz.org
http://5-7-1.dev.videojs-test-xyz.org
The sub-domain URLs indicate which version of the VideoJS library is used, with all else being equal.
What is the appropriate mechanism to prevent Video.js from falling back to flash on the site, to ensure IE only displays using the html5 videojs player, just like other common up-to-date browsers?

Yeah, I wonder if the issue you're seeing is actually related to the other youtube-tech isn't working on IE issues.
As for how to force flash from loading, theoretically, it should be just removing Flash from the tech order. However, I think the youtube tech just loads in whatever youtube wants and so if youtube decided to try and load in flash, it can. There may be some kind of option that can be given to youtube's API to have it only load in html5 content but I'm not sure.

Since late last week we've noticed that all of our code is working, again. Meaning our website versions from last December along with the website versions from February, that prompted me to write this issue in the first.
I thoroughly tested every conceivable iteration of techOrder, with no desirable results. I am positive techOrder had no relevance on this particular issue.
I swear, during December all of the website code using both VideoJS player and the YouTube plugin worked, however in February everything failed, causing me to question my earlier results, and my sanity. In retrospect, now that everything works, (all versions of code, old and new), we're squarely pointing the finger at Microsoft Internet Explorer and its 'updates'. I've searched the internet for Internet Explorer Release Notes, and Microsoft doesn't even publish them.
The moral to this sad story is: how much it costs to support Microsoft and its web browsers. The End. This issue is now closed, thank you very much.

@Amstercad heh, yeah, I noted in #342 it's been working for me in latest IE and videojs.
Glad to hear that it's working for you.


Remove rocblas on windows

Added MIGRAPHX_USE_ROCBLAS build option which is automatically ON on Windows and OFF on Linux.
When MIGRAPHX_USE_ROCBLAS is OFF, rocBLAS is removed on MIGraphX.
@apwojcik

Please change the code to default the CMake variable to ON for now.


azure-mgmt-recoveryservices: LICENSE missing from PyPi

Package Name: azure-mgmt-recoveryservices
Package Version: 2.0.0
Operating System: Linux
Python Version: 3.9.6

Describe the bug
The LICENSE file is missing from azure-mgmt-recoveryservices in PyPi.
Additional context
I am making a PR to fix this now.

same issue : https://github.com/Azure/azure-sdk-for-python/issues/20183


postgres icon

Describe the icon you'd like to be added
A clear and concise description of what you want to have.
an icon for postgres
Sample icon
Attach any samples/screenshots here.
https://www.postgresql.org/media/img/about/press/elephant.png

@antoni909 Hi, this icon has been added to the library.


aws-amplify-angular is only support angular5 now. will that package update soon?

In angular 6.x , aws-amplify-angular is no longer supported

Hi @caojianfei ,
Thanks for reaching out.  Angular 6 should be supported.  Due to the elimination of the 'global' object in Angular 6 (which some of our dependencies rely on) you'll need to add a shin to your application as described here.
Please let us know if this doesn't help.


Support yadm

yadm is a Git-based dotfiles manager. Basically, it manages a Git repo whose work tree is the $HOME directory. yadm can be used to run any git commands, such as yadm status, yadm add, yadm diff. It would be great to support it.
https://github.com/lewis6991/gitsigns.nvim supports yadm. Maybe https://github.com/lewis6991/gitsigns.nvim/blob/a6d1574babd96b53102677e50c65f916d388708e/lua/gitsigns/git.lua#L261 could be helpful.

this tutorial accomplishes a similar goal without requiring a package install:
https://www.atlassian.com/git/tutorials/dotfiles
Essentially, it would be nice to be able to configure vgit such that files in ~/.config/**/* would have their git dir in ~/.cfg instead of ~/.git

@ouuan @bennypowers, try out #218, works great so far.


UE55H6400 wrong work

after then as I pairing (by pair1 method) my device successfully
(no one else method of pairing does not work for me, i had a pairing error)
and plugin successfully send mute to my tv
i have some issues:
1) button POWER in widget don't turn off my tv
2) also INPUTS don't work as expected
 { // it turn on/off "FOOTBALL" mode
   "name": "HDMI 1",
   "keys": "KEY_HDMI1"
},
{ // it do nothing
   "name": "HDMI 2",
   "keys": "KEY_HDMI2"
},

but next input works well
{ // it works
   "name": "MUTE",
   "keys": "KEY_MUTE"
},

how i can fix it?
i will be grateful for the help

Your TV might not support the keys hdmi1 and hdmi2... unfortunately samsung's inofficial api is pretty inconsistent across devices. Can you try the hdmi key (without number)? This might also be the reason for not being able to turn off the tv, the library tries to find out if your tv is turned off with KEY_POWER or KEY_POWEROFF (in your case it uses power as you have a H-Series as far as I remember). Maybe your tv has to be turned off with power instead...

yes you are right, the KEY_HDMI key works for my TV, it switches the HDMI inputs one by one, thanks!
but I still have a problem with the big POWER button
maybe you have another solution to fix this?
as an example
     "platform": "SamsungTVControl",
     "devices": [
         {
             "usn": "uuid: ---",
             "token": "eyJz ---",
             "name": "TV Sumsung 55",
             "customKeyForPowerButton": "KEY_POWER", // <- some like this
             "inputs": [
                 {
                     "name": "HDMI",
                     "keys": "KEY_HDMI"
                 },
                 {
                     "name": "MUTE",
                     "keys": "KEY_MUTE"
                 },
                 {
                     "name": "POWER",
                     "keys": "KEY_POWER"
                 }
             ]
         }
     ]
}

and, if it posible, can i configurate remout control for recognises hold volume buttons?
now volume changed up/down on 1 point only when i release volume button. holding has not effect. can we fix/improve it?


bug: hero image overlaps nav in dev branch

Currently, working on this. Will pr a fix.

Fixed in #46


Update to consul.json does not trigger service restart

Had to update the node name to the hostname to support our deployment, but when the node name (i.e. node['consul']['config']['node_name']) attribute was overridden, it did not restart the consul service. Had to manually restart to capture the new node name.
Running on ubuntu jessie with chef client version 12.16.42.

Hi, @errygg
This issue duplicates #307 and #341
Service restart was disabled intentionally by https://github.com/johnbellone/consul-cookbook/commit/80f4a1ce4edb1638482e2391dabd1ba547ed7fad and https://github.com/johnbellone/consul-cookbook/pull/309
Please, refer to the discussion: https://github.com/johnbellone/consul-cookbook/issues/288
TL;DR: Automatic restart of Consul servers could cause a cluster downtime when you run chef-client simultaneously on your nodes. Also, if Consul version has been changed (upgrade scenario), then Consul servers should be restarted manually, according to the upgrade manual.

Thanks @legal90! I guess I wasn't the first to see this. I'll go ahead and close the issue out. Automate all the things (except consul restarts 😆 )


Release drafter

Hi @manno, I want to give this a try, but I have a couple of questions about how it works:

Does it rewrite the draft releases notes from scratch every time a PR is merged?
So can I add labels to already merged PRs and the draft will get updated to reflect that?
Can that be triggered by adding the label, or does it require merging another PR?
Will the release notes body be updated from the template each time the template is updated (e.g. bumping Quarks should update not just dependencies.yaml, but also .github/release-drafter.yaml?
How do you make manual changes to the release notes? Do you have to do them in one sitting, and then publish them, to prevent them from getting overwritten if anybody merges another PR?

I've looked at all the docs, but this is still not clear to me...

@manno I've merged the PR now to see how it works, but if you can, please still answer the questions I asked above...

Does it rewrite the draft releases notes from scratch every time a PR is merged?

The workflow is triggered for every push to master

So can I add labels to already merged PRs and the draft will get updated to reflect that?

Yes.

Can that be triggered by adding the label, or does it require merging another PR?

You would have to trigger the workflow for label changes on merged PRs, not sure if that's possible.

Will the release notes body be updated from the template each time the template is updated (e.g. bumping Quarks should update not just dependencies.yaml, but also .github/release-drafter.yaml?

The release-drafter.yaml is just the config for the run.

How do you make manual changes to the release notes? Do you have to do them in one sitting, and then publish them, to prevent them from getting overwritten if anybody merges another PR?

Once you are ready to release you edit the draft. If another push to master triggers the workflow it might overwrite your edits.
I think you can have multiple drafts, so just change the version.


Use struct with pid and Go routine addr for Go BPF maps

After merging #1158, we change behaviour for Go tracer to be shared among different processes.
Currently we are using the goroutine address as key to store data in BPF maps.
This PR changes the key to use the goroutine address and PID of the process in order to avoid
colissions when storing data to maps.
Fixes #1159

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

Project coverage is 53.55%. Comparing base (008f4f2) to head (0fcc3c1).

:exclamation:  There is a different number of reports uploaded between BASE (008f4f2) and HEAD (0fcc3c1). Click for more details.
HEAD has 2 uploads less than BASE

Flag
BASE (008f4f2)
HEAD (0fcc3c1)

oats-test
1
0

integration-test
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1182       +/-   ##
===========================================
- Coverage   76.83%   53.55%   -23.28%
===========================================
  Files         136      136
  Lines       11451    11451
===========================================
- Hits         8798     6133     -2665
- Misses       2121     4916     +2795
+ Partials      532      402      -130

Flag
Coverage Δ

integration-test
?

oats-test
?

unittests
53.55% <ø> (+<0.01%)
:arrow_up:

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

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

I'm missing something. Shouldn't be each goroutine address unique in the system? I would have expected that the goroutine pointer contains a unique virtual memory address

Looks great Marc! There are few other maps we need to fix, but please let me know if you don't want to use this PR and you want to follow-up with another instead and I'll flip this to Approve.

I see, thanks for the feedback. Yeah, I think it'd be better in a different PR

Please don't merge this yet, it seems to be tracking/comitting binary .o files.


IfcStreamer GeometryCullerRenderer does not respect coordinate when loading models.

Describe the bug 📝
When using the IFCStreamer the model is getting culled according to its "centered position".
When loading multiple models they are placed in the world correctly when using components.get(CustomIfcStreamer).load(data, true).
But the culler is not respecting this and seems to cull as if the models where loaded using components.get(CustomIfcStreamer).load(data, false).
Objects that are hidden by other models in the centered position are being culled and it's impossible to get them to be visible.
Reproduction ▶
No response
Steps to reproduce 🔢

System Info 💻
-

Used Package Manager 📦
npm
Error Trace/Logs 📃

Validations ✅

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a repository issue and not a framework-specific issue. For example, if it's a THREE.js related bug, it should likely be reported to mrdoob/threejs instead.
[X] Check that this is a concrete bug. For Q&A join our Community.
[X] The provided reproduction is a minimal reproducible example of the bug.

Solved! Let us know otherwise :)


Validation Gas Fees

Summary

RFCs: $\emptyset$.
Categories: protocol-units
Associated PRs: https://github.com/movementlabsxyz/aptos-core/pull/81

Integrates validation_only gas fee into maptos_opt_executor and adds tests.
Testing

Unit and e2e tests covering gas charges.

Outstanding issues

Tests are still WIP.

Marking this as draft for reasons discussed internally.


Fix build.sh copying

The quoting means that the globbing of the artifact path fails.

@facebook-github-bot shipit


Add SaveState and picture tests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+6.6%) to 76.546%

Totals

Change from base Build<PHONE_NUMBER>:
6.6%

Covered Lines:
768

Relevant Lines:
987

💛  - Coveralls


feat(qwik): add plugin to nxkit

Closing as Nx team is working in a Vite plugin, will wait to leverage on that or evaluate the Nx + Qwik landscape.


Rename broke rtx self-update

Describe the bug
Running rtx self-update results in "0: ReleaseError: No asset found for target: macos-arm64".
To Reproduce

Have rtx installed, (rtx 2023-09-13 in my case)
Run rtx self-upgrade

Expected behavior
Update to latest version of rtx
mise doctor output
rtx version:
  2023.9.1 macos-arm64 (c6d59d7 2023-09-13)

build:
  Target: aarch64-apple-darwin
  Features: DEFAULT, NATIVE_TLS, OPENSSL, SELF_UPDATE
  Built: Wed, 13 Sep 2023 14:12:15 +0000
  Rust Version: rustc 1.72.0 (5680fa18f 2023-08-23)
  Profile: release

shell:
  /bin/zsh
  zsh 5.9 (x86_64-apple-darwin22.0)

rtx data directory:
  /Users/_/.local/share/rtx

rtx environment variables:
  RTX_SHELL=zsh

settings:
  {"always_keep_download": "false", "always_keep_install": "false", "asdf_compat": "false", "disable_default_shorthands": "false", "disable_tools": "[]", "experimental": "false", "jobs": "4", "legacy_version_file": "true", "legacy_version_file_disable_tools": "[]", "log_level": "INFO", "missing_runtime_behavior": "warn", "plugin_autoupdate_last_check_duration": "10080", "raw": "false", "trusted_config_paths": "[]", "verbose": "false", "yes": "false"}

config files:
  /Users/_/.config/rtx/config.toml
  /Users/_/.tool-versions
  /Users/_/code/_/main.tf
  /Users/_/code/_/.rtx.toml

plugins:
  awscli           https://github.com/MetricMike/asdf-awscli.git#2e31396
  go               (core)
  java             (core)
  jq               https://github.com/lsanwick/asdf-jq.git#f02032e
  k2tf             https://github.com/carlduevel/asdf-k2tf.git#345a297
  kubectl          https://github.com/asdf-community/asdf-kubectl.git#cbe6df4
  node             (core)
  pipx             https://github.com/joe733/asdf-pipx.git#81cb547
  pulumi           https://github.com/canha/asdf-pulumi.git#99d675b
  python           (core)
  rbac-lookup      https://github.com/looztra/asdf-rbac-lookup.git#4d9ceb3
  redis            https://github.com/smashedtoatoms/asdf-redis.git#5e0a853
  ruby             (core)
  terraform        https://github.com/asdf-community/asdf-hashicorp.git#197e3ec
  terraform-docs   https://github.com/looztra/asdf-terraform-docs.git#acfd01c

toolset:
  rbac-lookup@latest, pipx@latest, node@latest, jq@latest

1 problem found:
new rtx version 2024.0.0 available, currently on 2023.9.1

there is unfortunately probably no way I can fix this since I can't patch the old clients

going to close since I don't think there is anything I can do here

That's fair, it might be a good idea to create a pinned issue titled something like "rtx has been renamed to mise" that links to https://mise.jdx.dev/rtx.html