Scraping Song

Description
Scraping Songs from and mailing the list.
Fixes #1132

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Documentation Update

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] I have added the project meta data in the PR template.
[x] I have created the requirements.txt file if needed.

Project Metadata
If there is no-file/nothing to fill the below fields with, then type: none
Example: If no requirements.txt needed/present, then type none in Requirments.
Category:

[ ] Calculators
[ ] AI/ML
[x] Scrappers
[ ] Social_Media
[ ] PDF
[ ] Image_Processing
[ ] Video_Processing
[ ] Games
[ ] Networking
[ ] OS_Utilities
[ ] Automation
[ ] Cryptography
[ ] Computer_Vision
[ ] Fun
[ ] Others

Title: ScrapingSongs
Folder: Scrapingsongs
Requirements: importing bs4,requests,smtplib,tabulate,MIMEText,MIMEMultipart
Script: ScrapingSongs.py
Arguments: None
Contributor: anumshka
Description: Scraping Songs from and mailing the list.

@anumshka update the PR before 2 pm so that we can review the code and merge it on time

Hey @anumshka are you going to update the PR..or Should I close it?


Why does this not follow the standard Ghost folder structure?

I'm very new to Ghost so my assumptions may be way off base, but I have a few questions about this repo that if answered may prove useful in the README for newcomers.
My first question is why the installation of Ghost is not /var/www/ghost which is the recommended installation location according to this article from Ghost support docs.
Also, I have a single theme located in a very basic folder structure ~/foo/ghost/content/themes/mytheme . When I try to map the volume of /var/lib/ghost to this folder using the given example: docker run --name some-ghost -v ~/foo/ghost/:/var/lib/ghost ghost I would expect my theme to show up in the Settings dashboard. Is this because I don't provide my own config.js? Is there a better way to include themes in the blog? When I enter into the running container I notice the structure is more /var/lib/ghost/themes which completely bypasses the content folder I would expect to see. Am I way off base here?

I'm having a similar question about adding our own themes.
It would be nice that have example where the entrepoint.sh loads a theme (not caspter obviously) via git repo (wget or something alike).

My 5 cents https://github.com/docker-library/ghost/issues/25#issuecomment-247216227

As of 1.x, we now use the official Ghost CLI tool for installing Ghost, which sets up a directory structure in a standard way. :+1:


Add support for high level volume operation metrics

Feature Description

One-line feature description (can be used as a release note): Add support for high level volume operation metrics
Primary contact (assignee): Brad Childs (@childsb)
Responsible SIGs: sig-storage
Design proposal link (community repo):
Reviewer(s) - @saad-ali @piosz
Approver (likely from SIG/area to which feature belongs):  @saad-ali
Feature target (which target equals to which milestone):

Stable release target (x.y) = 1.8

/sig storage

cc @kubernetes/sig-storage-feature-requests

@idvoretskyi primary contact would be me. Can you update the github assignee?

@gnufied any updates for 1.8? Is this feature still on track for the release?

Yes has been implemented for 1.8. We will open a follow up documentation PR for this.  https://github.com/kubernetes/kubernetes/pull/50036


[emacs-support] Basic flymake support

@swift-ci Please smoke test and merge

@swift-ci Please smoke test and merge

@swift-ci Please smoke test and merge


Crushed skeletal mesh on import

Hi!
I ran into a problem with importing skeletal meshes to unreal. Upon import, all the skeletal meshes are crushed/distorted for some reason.
For representation, I just exported the default UEFN mannequin to blender.

When trying to import it back unreal with Pipeline/Export/Push the result is this

Any idea on this?
I'm using:
Blender 4.3.2
Unreal 5.5.1
send2ue 2.6.1

Hey there. Make sure you apply scale to the skeleton after importing to blender. Also it appears there's an issue with UE5.5's interchange that could also possibly be messing some things up. I hear 5.5.2 should fix the interchange issues.

Hey Josh, yeah I applied the transforms, its gotta be the interchange then, guess I have to wait till .2, thanks!

I can also take a quick look to see if the UEFN skele is different from manny. Can you please share the UEFN character? I dont wanna install to get a copy heh.

Hey, no worries, here is the UEFN exported, this is the one I tested with.
https://drive.google.com/file/d/1oiZAtWxsKHGtwtYL0UZ8NOhy0LKPgcHx/view?usp=drive_link

thank you!

Yup it's all good in 5.3 without doing anything. So this is almost definitely the interchange issue. Should work in .2 🤞

Great! Thank you for checking!

I’ve also encountered the same problem and discovered that in my case it was due to the scene scale.
In Blender, I have the Unit Scale set to 0.01, and when I export with send2ue, the animations work fine, but the skeletal mesh appears collapsed, as described in the post.
I tried disabling the Apply Unit option and directly setting the Scale to 0.01, but it didn’t work.
However, if I disable the Apply Unit option and set the scale to 0.02, everything works fine (aside from the fact that the scale is, of course, not the desired one).
0.01 seems to be the threshold where this issue occurs.
I also tried exporting the model directly from Blender in FBX format, setting the scale to 0.01 in Blender's export dialog. In Unreal, the import works with the correct scale and without the collapsed mesh.
I also tried with other meshes, and the problem occurs as well.
I'm using:
Blender 4.4
Unreal 5.3.2
send2ue 2.6.1

I found its from applying scale. If you have a mesh parented to a skeleton and Apply Scale to the mesh, or Apply Armature Scale to the rig, Blender will go mess with the other part to maintain the same relative scale - this is what I think is happening at least. What I do is apply the armature scale first, then mesh scale, then double check that both have scale of 1,1,1

I tried several times to apply the scale to both the armature and the mesh, but nothing worked. Both are correctly set to 1 during the export phase. As I mentioned before, if I set the export scale to 0.02 or higher, everything works fine.
The problem occurs when I use an export scale of 0.01 (only with send2ue export; if I use Blender's classic export to then import the FBX, everything works).
Could it be caused by numerical precision? Some error propagating during the export that causes a mismatch in the scale of the mesh and the armature? I’m not sure.

May I ask why you're changing the export scale at all?
As long as scene scale is correct (metric, 1.0 unit scale) and your objects scales are at 1, then you should get accurate scale in unreal.

I created the mesh and animations using Rigify with a larger scale for the meshes, so when I import them into Unreal, they are too big. I should rescale the animations, but every time I try, something breaks. However, scaling the entire scene during export works, except that the mesh collapses.
If it helps, I did some testing today and found out that by changing the Forward and Up settings in the export dialog under the Transform section, the issue doesn’t occur (though, of course, the mesh appears rotated in Unreal).

Have you tried re-importing the animations from an FBX via the Import Asset operator? Its basically an FBX import plus the scale corrections to Skeletons and animations that comes from unreal.
Here is the call it uses to scale the actions
https://github.com/poly-hammer/BlenderTools/blob/fff28ece139ab8e88a56c9b21912c42a0a21eeee/src/addons/send2ue/core/utilities.py#L1572
Could probably fix them in your scene with this; provided the right armature object is selected and the action name is changed.
from send2ue.core.utilities import scale_object_actions
scale_object_actions(
    bpy.context.selected_objects,
    [bpy.data.actions['MyActionNameHere']],
   1.0
)

Haven't tested that code, but I think that would work to re-scale your actions (maybe play with the last number 1.0)

I created the mesh and animations using Rigify with a larger scale for the meshes, so when I import them into Unreal, they are too big. I should rescale the animations, but every time I try, something breaks. However, scaling the entire scene during export works, except that the mesh collapses. If it helps, I did some testing today and found out that by changing the Forward and Up settings in the export dialog under the Transform section, the issue doesn’t occur (though, of course, the mesh appears rotated in Unreal).

Something else to consider for ya; Blender's bone system uses different axis from Unreal, which is why the skeleton "fixes" when you change the forward/up axis. UE2Rigify was made to solve this issue to give you a Rigify rig that will bake animations correctly for UE.
It might be worth baking your animations to your skeleton, throw away your manual Rigify rig, and create a new template for your characters with UE2Rigify. You are then able to bake your animations back to the new rig automatically to continue modifying your animations with the new rig.


Reset "Upload key log csv" button after upload. Fixes #6

Thanks

I couldn't install you blog locally to test my changes, and now that you've deployed this commit I see that it doesn't work as expected. Unfortunately, the "change" event is not triggered at all, so the input element is never reset.
I should've added a new "click" event listener, to reset the input element.
Sorry about that.

It works correctly on Firefox, that's why I merged it ¯\_ (ツ)_/¯
I forget about the pains of browser compatibility every time


Come on...

Do it with at least with some style...
https://github.com/horefice/proof/blob/62ea86278d9e6baaf74482801e11768cc63611e1/index.html#L83

lol fixed it ;)


Exiting loop on closed channel, fixes mesos/mesos-go#182

minor comment remaining to be addressed. once fixed, and commits sqashed, lgtm

Addressed comment, squashed commits into one

thanks. LGTM

/cc @spacejam

fixes #182


Problem rotate preview in landscape

Hi.
In Manifest,  i set screenOrientation = Portrait
In my app, when want livestream in portait or landscape, i set rotation = 0 or 90 in rtmpCamera2.prepareVideo.
When watch livestream, everything work. But in preview with landscape, camera rotate incorrect.
How can the camera rotate correct in my case ?
I see in class Camera2Base ->
startPreview(CameraHelper.Facing cameraFacing, int width, int height, int rotation) {
...
boolean isPortrait = CameraHelper.isPortrait(context);
if (isPortrait) {
    glInterface.setEncoderSize(height, width);
} else {
   glInterface.setEncoderSize(width, height);
}
glInterface.setRotation(rotation == 0 ? 270 : rotation - 90);
glInterface.start();

Do not know if this is the problem ?
Please help me.

Hello,
Camera (preview and stream) is rotated according with current UI state (portrait, landscape, reverse portrait, reverse landscape).
To handle the correct rotation and let user select if stream is portrait or landscape. I recommend unlock activity rotation and lock it when you configure video settings and press start stream. For example:
startStreamButton.setOnClickListener {
     lockCurrentUIState()
     if (prepareAudioVideoConfiguration()) {
       startStream(endpoint)
     }
}

stopStreamButton.setOnClickListener {
     stopStream()
     unlockUIState()
}

In startPreview and prepareVideo methods. You should use rotation provided by CameraHelper.getCameraOrientation(context)

That is a great suggestion.
Thank you so much.


2FA verification using email instead of username is it possible?

General Question
Read the Notes and fill out the form.
Switch to Preview for reading.
Notes
Your issue will be closed if you violate any rule below.

You need to include a meaningful descriptions and e.g. a screenshot.
You do not include screenshots of your code.
Read the Docs.
Capturing endpoints and similar can be found here
(read it if you are going to ask about it).
You can delete this section after reading it.

Form
Put an [x] if you meet the condition, else leave [ ].

[x] I've searched the Issues
[x] I've read the basic concepts
[x] I'm using the latest version

Question
I tried using the twoFactorLogin method with the email and I get 400 response code, I understand then that it is only possible to use it with the username.
Is there a way to obtain the username through email while using 2FA and without having logged in?

Ok, it is possible to do it, taking it from the response.
const username = get(err, 'response.body.two_factor_info.username');


Update Python versions to include 3.13

SSIA

It is not entirely clear to me where the tests fail, with another package that also uses a DNSResolver I do not have this error 🤔


Added vtm5e files

Two files one for portuguese and other to english speakers

Is there any chance you'd be willing to include images and add these to the TEMPLATE_LIST.md?

For sure!

Done!

Thanks for the contribution!!


Doc updates mkdocs migration

Overhaul of all CM-Well Markdown documentation files and structure, for display with mkdocs.
Fixed all broken links.

Where is the mkdocs.yml file?

@Ivan-Shestakov , we reviewed the code and saw that there is no usage for length in next-chunk api. basically, the length is determined in the create-iterator only.


Add overlap parameter to improve the quality of inference results.

Add "overlap" feature to reduce the unnatural feeling of each chunk boundaries.
Add parameter overlap_rate to config.py
left: 0% overlap(like before)
right: 50% overlap

@dingjibang hi 请问你是tango.io的开发者吗？

@dingjibang hi 请问你是dango.io的开发者吗？

是的，这个项目很有趣所以想把它加入到团子里，顺便尽我可能帮忙做点贡献（


Libriheavy recipe (zipformer)

@pingfengluo from community wants to try this experiment quickly, so, if there is no major errors, I suggest to merge it as soon as possible, I will upload more models and update the results in the following RP.


os/mac/hardware: use Westmere on >= Ventura

Although we can't use anything with AVX in order to support Rosetta (similar to how Apple also avoid AVX in their own tools), we can do a minor bump to Westmere which adds CLMUL instructions that Rosetta 2 supports.
Not much of a difference to Nehalem, but it's something.

Thanks again @Bo98!


fix: make links always a single line

This matches the behavior of Prettier, which never introduces newlines into the link title.
resolves #53

Is there a way to use the changes made in this branch in my local dprint installation to try them out on my own documents?

Is there a way to use the changes made in this branch in my local dprint installation to try them out on my own documents?

Yup! First ensure you have wasm32-unknown-unknown target installed:
rustup target add wasm32-unknown-unknown

Then do:
cargo build --target wasm32-unknown-unknown --release --features "wasm"

This will create a .wasm file in the target/wasm32-unknown-unknown/releasefolder. Then just reference that in the "plugins" array of your dprint config (or specify it on the CLI in--plugins`) via a relative or absolute path.
Will take a look at this PR later. Thanks for it! I am still investigating if this should be the behaviour though, but I'm like 80% sure it should given prettier works this way.

Looks great. Thanks @kevgo!


Epdcs 1011 externalize hardcoded test configs

Changes are in acceptance-test module only

These changes are merged with Epdcs 1024 eventhub automation PR


Handle line sections

Builds on #141

!test

Oh and btw, I think it would make sense to note these additions and changes in CHANGELOG.md


[RFW] Improve the developer ergonomics

Problem
Currently RFW allows developers to write
widget root = Container(
  color: 0xFFFFEBEE, // red[50],
  child: Icon(
     icon: 0xe2a0, // flutter_dash
     fontFamily: 'MaterialIcon',
  ),
);

which is awesome but feels like a downgrade coming from flutter where one can write
Container(
  color: Colors.red[50],
  child: Icon(icon: Icons.flutter_dash),
)

Proposal
I have a couple of ideas
Extend the material library
Extend widgets to support something like
Container(
  color: 'red500',
  // or
  color: {
     name: 'red',
     intensity: '500',
  },
  child: Icon(name: 'flutter_dash'),
),

of course the local widgets will have to now infer user intent, if a valid "string" (like "red" or "red500") is passed do the right mapping; otherwise (a number is passed) use it. A similar treatment should be applied to the Icon param.
Provide argument encoders that can be passed as data
Similar to the argument decoders API we can create its encoder counterpart
class ArgumentEncoder {
   // returns red500 or red
   static String color(double number);

   // returns { color: 'red', intensity: 500 },
   static DynamicMap color(double number); // returns red500 or red

   static DynamicMap icon(IconData iconData) => <String, Object?> {
      codePoint: iconData.codePoint,
      fontFamily: iconData.fontFamily,
   };
}

So that it can be used as follows
final data = DynamicContent()
  ..update('icons', {
    'flutter_dash': ArgumentEncoder.icon(Icons.flutter_dash),
  })
  ..update('colors', {
    'red': ArgumentEncoder.color(Colors.red),
    'red500': ArgumentEncoder.color(Colors.red[500]),
  });

const rfw = '''
  widget root = Container(
    color: data.colors.red,
    child: Icon(icon: data.icons.flutter_dash),
  );
''';

Introduce the colors and icons identifiers.
Similar to args and state we can introduce new identifiers and give developers control over what to put there.
Color(
   color: colors.red,
   child: Icon(icon: icons.flutter_dash),
);

and in dart
final runtime = Runtime..updateColors('colors', <String, double>{
  'red': Colors.red,
  'red500': Colors.red[500],
});

Allow data definitions and library aliases to avoid duplication and maximize reuse
// library theme.rfwtxt
define colors = {
  'red': 0xFFFFEBEE,
};

// library main.rfwtxt
import theme as theme;

widget Foo = Container(color: theme.colors);

data.colors.red.500 should work today using data references, I think, without needing anything in package:rfw.
That said if the goal is easier authoring, maybe the right solution is just to create a new format (there's been some discussion in #hackers-rfw of a dart-like format) that can then be compiled to the binary form, which is never sent to the client. The client shouldn't have to know about things like Colors, that kind of authoring detail should not have performance implications on the client IMHO.

Sorry I should have stated my proposal more clearly. Color and IconData just where examples and each subheader in the issue refers to a different proposal to tackle the same problem. What I get from your comment is that you prefer options that don't have meaningful changes on the language nor the runtime. In that case would you be ok with adding a ArgumentEncoder helper?

What I was trying to convey is I don't think this needs a change to package:rfw at all. If someone wants to use data.colors.red.500 they can already do that in their client by adding a root key to their DynamicContent objects called "colors" that contains the various colors and shades they want to support. Personally I would recommend doing that entirely on the server and sending an integer to the client rather than an enum-like value. The only time I think it'd be better to send an enum-like value is to identify a color from a scheme, e.g. to say "use the app's background color" or "use the branding highlight color" or whatever, and that can already be done using data.


Allow shift+click on input to create new link from origin node

Supports shift+click on the input slot to create a new link from the origin node

This PR change only affects the input slot interaction. The shift behavior on output slot is not affected.


Interpolate raster - function for storing georeferenced images is needed

At this point array interpolation interpolate_raster function output is:
[[numpy array of interpolated values, numpy array of interpolation errors], [pixel size, min x, max x, min y, max y]]

It'd be nice to have function which saves this array to some georeferenced image, as example, to jpg + jgw pairs (without additional external packages needed to be installed).

Ok, function is the next candidate for development after the multiprocessing enhancement and Python 3.8+ compatibility development. If you know something about the raster processing in Python then don't wait and please, help community :)


Can't disable Scroll

When using the prop scrollEnabled={false} in SwipeListView, the scroll is enabled again after a swipe gesture has happened? How can I solve this?
Thanks

@fmcruz good call, this was definitely a bug.
This should be fixed in v1.3.1, thanks!


how-tos: use better cluster profile example

The pull request currently linked does not contain the Boskos reaper
configuration changes which should be added for new resources.  Even though the
link right next to it describes the requirements, resources are still added
without the corresponding configuration.
Replace the link with one to a pull request containing all required changes.

/lgtm because wandering around our clusters trying to understand what's going on with Boskos might not be fun in the long run.

/lgtm
does /lgtm deserve its own line?


Deploy the exonum-java-app module [ECR-3987]

Overview
Deploy the exonum-java-app module, upon which
the testkit depends.

See: https://jira.bf.local/browse/ECR-3987
Definition of Done

[x] There are no TODOs left in the code
[x] Changelog is updated if needed (in case of notable or breaking changes)
[x] The continuous integration build passes

Coverage remained the same at 86.397% when pulling 29767d5ed581c2e8be51d5e73c773466ae553fc2 on dmitry-timofeev:fix-testkit-deploy into 3e420982fbeb5d599a750f7135826f14c6d48faf on exonum:master.


Add built-in greptime_private schema by default

What problem does the new feature solve?
This built-in schema is designed to store tables that created by our own applications and do not visible to user by default. It also prevents us from creating tables in public schema that has a potential risk to conflict with user's table.
What does the feature do?
Create an empty greptime_private schema by default.
Implementation challenges
No response

Is it the same as the system schema in some DBMS?

I think it's a little different from system that:

this schema is designed to store tables created by greptime applications, while system schema is for storing tables of database itself
system schema is for database administrators, while in most case user won't need to care about tables in this schema
this schema is to be created for each catalog in multi-tenant environment, which system is created for default catalog only

Example cases for greptime_private:

Our dashboard will ship a Saved Query feature allows user to save a SQL/PromQL script for future use
Out cloud will write additional alerting logs into tables under this schema


TypeScript wrong highlighting when using if within for

From @igabesz on November 26, 2015 11:26
TypeScript highlighter interprets if calls within a for loop as function definitions if the if has middle parameter and brackets. The interpretation starts failing in the for loop for the last parameter.
Not the consequence of installed extensions.
Highlighting fails for other color themes as well, not only Monokai.
// OK
for (var i=0;; i++) {
	if (i===0) {}
}
// Partially wrong, see for loop 3rd parameter
for (let i=0; i<5; i++)
	if (i===0) {}
// Wrong
for (var i=0; i<5; i++) {
	if (i===0) {}
}
// Wrong
for (let i=0; i<5; i++) {
	if (i===0) {}
}
// Wrong
for (; i<5;) {
	if (i===0) {}
}

Copied from original issue: Microsoft/vscode#704

From @igabesz on November 26, 2015 11:56
OK, messing with this a little more I came across that highlighting breaks if there is a variableName< text in the middle parameter of for. I think it's something with the parsing of templates.

Not happens if there is whitespace, e.g. variableName <
Not happens for funcCall(variable)<
Not the < operator triggers the error, variableName<< also works

Also the consequences are not affecting only to the for body, although they are more limited afterwards.

From @igabesz on November 26, 2015 11:34
This can get really ugly for more complex cases. (Logic mostly omitted.)
Highlighting works fine again after removing the middle parameter of the for cycle.

From @egamma on November 26, 2015 14:52
The TypeScript Sublime plugin renders this correctly.
Should we update the textmate bundle with the one from Microsoft/TypeScript-TmLanguage.

I verified that we are using the latest version from https://github.com/Microsoft/TypeScript-TmLanguage.
Installing that same grammar file also in Sublime shows there is the same bug:
VS Code on the left - Sublime 3 on the right:

@aeschli should we close this as fixed?

To me it looks like a bug in the grammar. Do you disagree?

Sorry, I misread that last comment; I thought VSCode was not up-to-date. I guess there's been a regression, but I think this is due to the way in which we handle < next to an identifier. If you put spacing around it, does the problem go away?

Yes, with a space in front of the < things look fine.

What if you have a space before your < sign? There is a known issue that the current grammar cannot differenciate the casting angel bracket and the less-than sign.

@zhengbli Space in front of the < solves the issue. You can also see this in the 3rd screenshot

Duplicate #11


(fleet) support installing compliant datadog packages in the updater

What does this PR do?
This PR adds support for installing Datadog Packages in the updater.
Motivation
We want to open the door to storing datadog packages in OCI compliant registries (gcr.io & co). The previous package format and code handling that format was not fully compliant with the OCI spec and would not have allowed us to easily migrate to OCI registires.
Additional Notes
This PR also adds unit tests for all the updater code (except local_api).

/merge -p urgent

/merge -p urgent

/merge

/status

/merge


chore: make compat build side-effect free

changes

Make rapier-compat builds side-effect free. They no longer modify files outside the compat directory. It has caught be off guard several times that building rapier3d-compat "breaks" the rapier3d pkg
Easier dev setup for rapier packages by adding a root package.json with scripts and that testbeds can reference (this will be needed for a workspace setup anyway)
Use local rapier2d and rapier3d version in testbeds

Hmm think I found some issues with the tests... let me have a look

Thank you for this PR @alexandernanberg! Is this ready for review/merge or are there still issues with the tests?

@sebcrozet Yeah, dont' remember exactly what it was but can't reproduce any issues now!

Perfect, thanks!


Bug: "Error in atexit._run_exitfuncs" when running unit tests

Description

I'm using the package within a download function like so:
def download(url):
  spinner = Halo(text='Downloading', spinner='dots12')
  spinner.start()
  (...)
  spinner.succeed('Download Complete')
  spinner.stop()

While, the package works as expected when running the program, when running unit test with pytest the test output is followed by the error message pasted below.
System settings

Operating System: macOS
Terminal in use: iTerm2
Python version: 3.5, 3.6, 3.7
Halo version: 0.0.17
pip freeze output:

Error
Error in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/halo/halo.py", line 78, in clean_up
    self.stop()
  File "/usr/local/lib/python3.7/site-packages/halo/halo.py", line 419, in stop
    self.clear()
  File "/usr/local/lib/python3.7/site-packages/halo/halo.py", line 311, in clear
    self._stream.write('\r')
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 40, in write
    self.__convertor.write(text)
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 141, in write
    self.write_and_convert(text)
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 169, in write_and_convert
    self.write_plain_text(text, cursor, len(text))
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 174, in write_plain_text
    self.wrapped.write(text[start:end])
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 40, in write
    self.__convertor.write(text)
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 141, in write
    self.write_and_convert(text)
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 169, in write_and_convert
    self.write_plain_text(text, cursor, len(text))
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 174, in write_plain_text
    self.wrapped.write(text[start:end])
  File "/usr/local/lib/python3.7/site-packages/_pytest/capture.py", line 358, in write
    self.buffer.write(obj)
ValueError: I/O operation on closed file
Error in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/halo/halo.py", line 78, in clean_up
    self.stop()
  File "/usr/local/lib/python3.7/site-packages/halo/halo.py", line 419, in stop
    self.clear()
  File "/usr/local/lib/python3.7/site-packages/halo/halo.py", line 311, in clear
    self._stream.write('\r')
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 40, in write
    self.__convertor.write(text)
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 141, in write
    self.write_and_convert(text)
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 169, in write_and_convert
    self.write_plain_text(text, cursor, len(text))
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 174, in write_plain_text
    self.wrapped.write(text[start:end])
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 40, in write
    self.__convertor.write(text)
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 141, in write
    self.write_and_convert(text)
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 169, in write_and_convert
    self.write_plain_text(text, cursor, len(text))
  File "/usr/local/lib/python3.7/site-packages/colorama/ansitowin32.py", line 174, in write_plain_text
    self.wrapped.write(text[start:end])
  File "/usr/local/lib/python3.7/site-packages/_pytest/capture.py", line 358, in write
    self.buffer.write(obj)
ValueError: I/O operation on closed file

Expected behaviour

I wouldn't expect the package to have any impact on these tests that isn't present when running the program.
Steps to recreate

Run pytest unit tests on a program that uses halo as shown above

People to notify

@tterb Thanks for reporting. I'll check if I can reproduce this issue on my end and get back to you on this.

@manrajgrover Okay, let me know if more info is needed

@tterb I would need more information here. Could you please create a sample code to reproduce this issue? I tried doing it but couldn't.

Presuming halo runs some cleanup using an atexit func, this sounds more to me like halo writing to what it thinks is sys.stdout, but was really pytest's caplog — which presumably is a StringIO long-since closed by the time halo cleans up.

Here ya go: https://repl.it/repls/SorrowfulNaturalDisassembler


Parking lane and street side should be inverted

According to the wiki, you could map the whole city with parking=lane, but I'm sure that in most cases, when a mapper wants to add parking, it's probably parking=street_side. This is confirmed by taginfo.
The current drop-down menu looks like this:

It's even worse in French because the words are basically synonyms, I can't even tell if the translations are inverted. (I'm French btw)
This has led me and I think many other mappers to favor "lane" instead of "street_side".
Also, not sure if this is intended, but when you select "Street-Side", the Type field disappears.

For sure street_side should be higher in the list because it's one of the most, if not the most, popular type of parking lot.

See https://github.com/openstreetmap/id-tagging-schema/pull/1326#issuecomment-2322775871 for further suggestions on how to improve this UI.


Migrate batch to testProxy (Do not merge)

TestProxy

caused by azure-batch

/check-enforcer override


198MHz on i5 using Vexriscv in Litex, How?

Hi,
Thanks for this repo it is very informative.
I noticed that the litex_with_dram.bit demo file manages to run the Vexriscv processor at 198MHz and achieves high rates of memory access as well. Would it be possible to make the Litex python file available for this demo?
Cheers,
Derek

hello, please check these two examples
litex without dram
https://github.com/wuxx/litexOnColorlightLab004
litex with dram
https://github.com/ghent360/riscvOnColorlight-5A-75B

That info was great. Exactly what I needed.


Support diacritical marks for resource group names

Bug Report

import path of package in question: github.com/Azure/azure-sdk-for-go/profiles/latest/web/mgmt/web
SDK version: v68.0.0

Specify the exact commit: 2187aca40bbf6931ffac666977e5d29128d82585

output of go version: go version go1.21.3 darwin/arm64

What happened?

web.AppsClient#Get: Invalid input: autorest/validation:
validation failed:
parameter=resourceGroupName constraint=Pattern value="testötest" details:
value doesn't match pattern ^[-\w\._\(\)]+[^\.]$

What did you expect or want to happen?
Support diactrical marks, as it's supported, at least in the swedish regions (a-la swedencentral)
How can we reproduce it?
Create a resource group with diactrical marks from the portal, try to retrieve it with the sdk.
Anything we should know about your environment.
N/A

The package being imported is from our legacy SDK which is no longer supported.
I believe that armappservice is the latest SDK that has replaced /web/mgmt/web.

Hey @jhendrixMSFT !
This is an SDK-wide issue actually, the webapps usage is just an example.
It stems from the swagger validation, for instance here:
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/web/resource-manager/Microsoft.Web/stable/2023-01-01/WebApps.json#L30858

We've removed client-side validation in our latest SDKs, sorry I didn't clarify that earlier.
You can globally disable client-side validation in the legacy SDKs by setting validation.Disabled to false (I forgot about this).
Please keep in mind that v68.0.0 is the legacy SDK which is out of support.


It's difficult for me to tell what the problem is, since I can't read the whole sentence.
Some I can infer, but others are hard to guess...
The text appears to be "clickable," but clicking on it does nothing...
My coworker also appears to have this problem

Seems to be an issue with the status-bar package we're using: https://github.com/atom/status-bar/issues/38

There needs to be some other way of displaying linting errors. Like maybe if you click on it it displays a popup notification.

Closing as this project is deprecated. 🌵


HTML output uses removed HTML5 attribute: @compact

Expected Behavior
HTML output should use standard HTML markup supported by major browsers. @compact is no longer supported by Chrome, Firefox, Safari, Edge, Opera.
Actual Behavior
HTML output for html5 and xhtml uses @compact attribute, which was removed from HTML5. It was deprecated in HTML 4 about 20 years ago.
HTML5 spec, section 16.2 Non-conforming features:
https://html.spec.whatwg.org/#non-conforming-features
Paraphrase of the spec:

The following attributes are obsolete (though the elements are still part of the language), and must not be used by authors:
compact on dl elements
compact on menu elements
compact on ol elements
compact on ul elements

Possible Solution
Add a compact value in @class, and make the CSS match that?
Steps to Reproduce

In DITA 1.3 XML topic, add @compact to the root element of a list like
:

text

Run the html5 or xhtml transtype on it.
The HTML output uses @compact.
html5:

xhtml:

CSS:*[compact=yes] > li {
margin-top: 0; }
*[compact=no] > li {
margin-top: 0.53em; }

The Oxygen XML editor gives an error if you open the HTML file:

The “compact” attribute on the “ul” element is obsolete. Use CSS instead.

Environment

DITA-OT version: 4.1.1
Operating system and version:
Windows 10
How did you run DITA-OT?
Oxygen GUI
Transformation type:
html5, xhtml

The @compact is one of those features of DITA that I think is very rarely used. We should replace this with outputting class="compact" in HTML5 and XHTML.

@compact in DITA XML is still used quite a bit, according to the OASIS DITA TC. @fwegmann asked them. The TC plans to keep @compact in DITA 2.0. But it's long gone in HTML. We've been discussing compact lists in the Lightweight DITA Subcommittee and I discovered this.

Testcase that applies @compact="yes" to
,
, <sl>, and
: 4298.zip

I have a request on this. Recently in the Lightweight DITA Subcommittee we decided to add the attribute @data-compact to the HTML5 flavor of Lightweight DITA (aka HDITA), instead of adding a fixed @class value of "compact". Can we get you to use @data-compact="yes" or @data-compact="no" instead of the @class syntax above?
Example:
HTML5 and HTML5 XML syntax ("XHTML")

CSS
*[data-compact=yes] > li {
margin-top: 0; }
*[data-compact=no] > li {
margin-top: 0.53em; }

@jelovirt, @robander - this represents a departure from precedent for capturing element characteristics in the HTML. I think we should discuss it in the next monthly call.

I would not mind attending this meeting, can I?


Add support for lang separation and cleaning

Adds support for language separation and cleaning from the data curator.

@dimapihtar please review when you get the chance. Feel free to ping me with any questions.


Ability to Disable certain commands

I am running the bot on my server and would like to disable commands like //urban but am unable to do so which is causing people to spam it. Is there anyway to disable commands like this? I've seen other bots do so, so was wondering if this bot could or able to add a function to disable certain unwanted commands?

This is a planned v5 feature


Load root with uproot ok?

https://github.com/ricmperes/PyLArS/blob/23fee5e0929f40ec76895efb25d88ec36a93c13e/pylars/utils/input.py#L48
Is it ok to leave only uproot.open(...) or should it be in a with statement? Or uprrot.close?

You may want to check out the implementation here.


I love to help

Hi, I am glad to know you are doing a Game Server in C#.
I am a full-stack developer with different skill sets on both front-end and back-end.  C#, Kotlin, Swift, Angular, Unity3d to name a few.
Please let me know how can I help.  Thank you.

Hey, you're welcome 😀
There are several features which still need to be implemented. You can find most of them in the issues and in the classic project boards. I try to keep them up-to-date regularly.
For the start, I would recommend getting comfortable with the source code by implementing some example chat command plugins or debugging through some existing ones. If you have any idea how I could improve the docs or how to make it easier to start working on it, please just tell me.
I have put some documentation together here: https://munique.github.io/OpenMU/

Thank you.  I agree, I need to familiarize myself with the source codes, specially the architecture.  I have compiled and tried connecting it with the client.  I will get in touch if anything needs to discuss.

can your help to solve #355 issue .
i want to play this in other languages。


[Table] - Prevent selection on some places of md-table

Bug, feature request, or proposal:
Bug.
What is the expected behavior?
I'd expect to select the rows only clicking in checkbox or maybe in rows, but not on buttons.
What is the current behavior?
Rows are being selected if you click on md-menu.
What are the steps to reproduce?
Providing a Plunker (or similar) is the best way to get the team to see your issue.
https://plnkr.co/edit/iFVIE17s5T2WAmRyurRX?p=preview
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
2.0.0-Beta8.
Is there anything else we should know?
It could be added to docs on how to do it (if it's currently possible). I can see a lot of use cases that dev just want to allow user to select a row if click specifically on checkbox and nowhere else.

The table does not come with selection. Sounds like you have a click handler that does not stop the propagation of the click event so it is raised to the row.


docs(spin): Adds spin pipeline templates docs.

Agreed on the /reference reorganization, though I think I'll save that for when I automate the spin CLI documentation.


Point Load

Hello Sir,
How can we apply point load on hexahedral mesh. I have tried to implement "0" in the second column of boundary and also arranged it in required manner. But, I am unable to visualize it's geometry in GLVis and perform ex2.
Would please suggest me how can I apply point load in linear elasticity model?
I am adding the mesh file, where have I tried to consider point in boundary section for beam-hex.mesh. Please look into it.
Thank you very much.
# POINT       = 0
# SEGMENT     = 1
# TRIANGLE    = 2
# SQUARE      = 3
# TETRAHEDRON = 4
# CUBE        = 5
dimension
3
elements
8
1 5 0 1 10 9 18 19 28 27
1 5 1 2 11 10 19 20 29 28
1 5 2 3 12 11 20 21 30 29
1 5 3 4 13 12 21 22 31 30
2 5 4 5 14 13 22 23 32 31
2 5 5 6 15 14 23 24 33 32
2 5 6 7 16 15 24 25 34 33
2 5 7 8 17 16 25 26 35 34

boundary
36
1 0 0
3 0 9
3 0 18
3 0 27
3 0 1
3 0 10
3 0 19
3 0 28
3 0 2
3 0 11
3 0 20
3 0 29
3 0 3
3 0 12
3 0 21
3 0 30
3 0 4
3 0 13
3 0 22
3 0 31
3 0 5
3 0 14
3 0 23
3 0 32
3 0 6
3 0 15
3 0 24
3 0 33
3 0 7
2 0 16
3 0 25
3 0 34
3 0 8
3 0 17
3 0 26
3 0 35

vertices
36
3
0 0 0
1 0 0
2 0 0
3 0 0
4 0 0
5 0 0
6 0 0
7 0 0
8 0 0
0 1 0
1 1 0
2 1 0
3 1 0
4 1 0
5 1 0
6 1 0
7 1 0
8 1 0
0 0 1
1 0 1
2 0 1
3 0 1
4 0 1
5 0 1
6 0 1
7 0 1
8 0 1
0 1 1
1 1 1
2 1 1
3 1 1
4 1 1
5 1 1
6 1 1
7 1 1
8 1 1

Hi @rajpradi, this is an invalid mesh, which appears to be derived from the beam-hex.mesh file.
The boundary conditions in MFEM are implemented in the source code, not the mesh.
Do I understand correctly that you want to apply an elasticity Dirichlet (essential) boundary conditions in a single degree of freedom? If yes, how do you identify that degree of freedom (by its coordinates?)

Hi @rajpradi, were you able to resolve this issue?

Hi @rajpradi, this is an invalid mesh, which appears to be derived from the beam-hex.mesh file.
The boundary conditions in MFEM are implemented in the source code, not the mesh.
Do I understand correctly that you want to apply an elasticity Dirichlet (essential) boundary conditions in a single degree of freedom? If yes, how do you identify that degree of freedom (by its coordinates?)

Hi @tzanio , I have a same case when I resolve my problem. Do I understand correctly that you want to apply an elasticity Dirichlet (essential) boundary conditions in a single degree of freedom? If yes, how do you identify that degree of freedom (by its coordinates?) this is correctly my question. May you recomend me a solution for it? Thank you very much!!!


Run clean before --fetch

Hi, I'm @JPMartha.
I should run --fetch twice with this.
By the way, I have concerns about the SwiftPM design.
I understand the SwiftPM is still in early design and development.
The way I see the current implementation, it seems to be designed to be available for running --fetch again.

https://github.com/apple/swift-package-manager/blob/swift-DEVELOPMENT-SNAPSHOT-2016-03-24-a/Sources/Get/Fetcher.swift#L78-L86

But, I have no objection to that if acceptable.

I don't think this is desired. The real solution is that --update should exist and --fetch should error properly if deps have already been fetched.

I have some questions.
Let's suppose the following situation:

MyApp depends on PackageA and B.
PackageB depends on C.

MyApp
├── Package A
└── Package B
    └── Package C

Questions

If PackageA requires update, then SwitPM should fetch PackageA only, or all packages?
If PackageB requires update, then SwitPM should fetch both PackageB and C?
if PackageA-1.0.0 requires update to 2.0.0, then SwiftPM shoud remain PackageA-1.0.0 directory, or override?

I send the questions to the mailing lists again.
Because I feel questions might be mentioned on the mailing list better than GitHub PR. 🙏


Add-SPOFile doesn't work for style library

Add-SPOFile -Path $files[$i].FullName -folder $TargetDir -Checkout
Even when I have an explicit -Checkout parameter in the command, it throws an exception that file should be checked out first. it works fine for Site Assets library but doesn't work on style library and difference I can see is that in style library, "Require documents to be checked out before they can be edited?" is marked as Yes.

I am using:
Add-SPOFile -Path $source -Folder $folder -Publish -Checkout
to add the the Style Library and it seems to work.
Example values:
Path="./styles.css"
Folder="Style%20Library/_someFolder/css"

I'm having the same issue in PublishingImages Library. The "require documents to be checked out before they can be edited" is set to Yes.

I just tried here but I'm not able to reproduce the issue. This is what I used:
Add-SPOFile -Path $source -Folder $folder -Publish Checkout
my $folder variable contains:
"Style Library/somefolder/css"
It creates the somefolder folder, in there a css folder and it uploads the file and publishes it. Running the cmdlet again updates the file as expected.

We implemented additional fixes to the Add-SPOFile cmdlet, which can be found in this pr: https://github.com/OfficeDev/PnP-PowerShell/pull/411. If these fixes do not solve the report issue, do not hesitate to reopen this issue.


DXGI_COLOR_SPACE_TYPE is undefined.

I'm trying to use DDSTextureLoader.
When I include DDSTextureLoader.h, Visual Studio complains that "DXGI_COLOR_SPACE_TYPE" is missing at "c:\program files (x86)\windows kits\10\include\10.0.10586.0\um\d3d11_1.h(2680)".
Logically, I can force to compile this by adding my own DXGI_COLOR_SPACE_TYPE definition before the include macro, but I want to run it without the obnoxious hack.
What gives?

Check your include path - I suspect you are pulling in files from more than one different version of the DirectX headers, or perhaps you've manually copied DirectX headers into your project or to some unusual place on your machine?
DXGI_COLOR_SPACE_TYPE is defined by dxgitype.h, which is include by dxgi.h , which is included by dxgi1_2.h, which is included by d3d11.1.h.  To be missing that definition, you must somehow be pulling in a wrong version of one of those files.

Most likely you are mixing the Windows 8.x SDK and the legacy DirectX SDK incorrectly. See MSDN. The DirectX Tool Kit does not require any use of the legacy DirectX SDK except when using XAudio 2.7 on Windows 7. See The Zombie DirectX SDK and XAudio2 and Windows 8

Thanks for the advise. Actually, my include path was fine (I think) and I wasn't using legacy DirectX.
After some working around, I was able to solve the problem by re-targeting the platform sdk version to 10.0.10586.0 in the project's context menu and things started to be fine.
Dunno exactly why it worked though.


WIP: Axis divider in a few places

@dengemann suggested in #2897 we use the axis divider pattern instead of plt.subplot calls. I've refactored epochs.plot_image accordingly. Besides for slightly neater code, the main benefit is that this should in principle allow 3-axis epochs image plots in a topographical layout.

The very appealing example for the pattern in the tutorials has been broken for a few versions now because someone forgot to turn off show:
http://martinos.org/mne/stable/auto_tutorials/plot_stats_spatio_temporal_cluster_sensors.html
http://martinos.org/mne/dev/auto_tutorials/plot_stats_spatio_temporal_cluster_sensors.html
This fixes it.

Also working on the "significant clusters" terminology ...

Also the example can be simplified substantially now thanks to plot_compare_evokeds

https://7333-1301584-gh.circle-artifacts.com/0/tmp/circle-artifacts.nZMt4OX/html/auto_tutorials/plot_stats_spatio_temporal_cluster_sensors.html
good

https://7339-1301584-gh.circle-artifacts.com/0/tmp/circle-artifacts.C0uGUts/html/auto_tutorials/plot_stats_spatio_temporal_cluster_sensors.html
Most recent version .... the scipy link doesn't work yet though.

This is probably ready for review.

here is the rendered doc: https://7339-1301584-gh.circle-artifacts.com/0/tmp/circle-artifacts.C0uGUts/html/auto_tutorials/plot_stats_spatio_temporal_cluster_sensors.html#sphx-glr-auto-tutorials-plot-stats-spatio-temporal-cluster-sensors-py
good to go from my end.
2nd review? @dengemann maybe?

I milestones this because the example really should be fixed.

Found a problem: the sensor plots are given bad dimensions.
https://7376-1301584-gh.circle-artifacts.com/0/tmp/circle-artifacts.dlgPx4n/html/auto_examples/visualization/plot_channel_epochs_image.html

https://stackoverflow.com/questions/49500637/aspect-ratio-vs-size-and-position-of-inset-axis-of-axes-divider-host-axes
:(

@jona-sassenhagen I know, changing the size and position of insets can be an horror. I remember I spent about an hour first time trying to change the position of topo legend inset in erp spatial colors erp plots. I gave up and thought these insets should be normal axes (they are of some stranger type) but never brought that up here. So now might be a good time. :)
I only skimmed through the linked question so if my comment is unrelated feel free to ignore it. :)

I undid the original axis_divider changes for the time being. However, the fix and improvements to the example should still be merged.

I have rebased. I forgot to fix the scipy reference, but feel it's not worth stressing Travis over.

Ah, so I just add it to the commit message?
git commit -m "stuff [ci skip]"

Yes. Also supported by local convention are [circle front] to build the front-page examples, and [circle full] to have CircleCI run all examples (slow).

Alas, I had already committed it and amending stuff would have been a bit too much. We should still be green ...

Thanks @jona-sassenhagen

Thanks!


pagination and filtering not working properly.

Hi im using v 0.5.3.
The pagination and filtering working properly at first, but after filtering if i refresh or reload the page pagination not working properly and for every  page i click only the first page data getting displayed. And also if i choose any filter from drop down menu which i've in my project, the filter is applying automatically before i click search button.
Pls provide a solution.

Can you please use the latest version 0.5.8 and see if the error it still there.


Cleanest way to disable drag and drop on mobile device

Hi, I would know how to fully disable drag and drop on mobile device, i use touchpunch.js but I only need it on tablet.
What I mean by "fully disable" is to not only disable item with drag or drop attribute but the whole component.
Any suggestions?
Thank you in advance.

@elarion do not load jQuery, jQueryUI, and angular-dragdrop on Mobile.

Ok I though there was a method or something to disable it, and even if I don't like these alternatives it seems good. Thank you


Allows specifying api_version as default instead of specifying api_version for each request

The following processes that are possible in xrpl.js should also be possible in xrpl-py.
import { Client } from 'xrpl'
const client = new Client('custom-api-v1-server')
client.apiVersion = 1

await client.connect() // api_version = 1

await client.getServerInfo()  // api_version = 1
await client.request({ comand: "server_info" }) // api_version = 1

Hello,
In the Python codebase, api_version is set inside the Request class, it is not a part of the Client class.
These Request(-child) instances do not persist between RPC invocations, so the above idea cannot be achieved in xrpl-py.
rippled structures all the RPC requests with individual api_version fields. I mirrored that level of granularity in the Python SDK. Is this a pressing issue?
Let me think of a solution that doesn't change the architecture substantially :)

Hello, I acknowledge that there is no "default setting" for the api_version value in xrpl-py's Client class. This is also true for clients communicating with rippled directly via JSON-RPC or WebSocket connections.
If you have any use-cases which are hindered by this design, please let me know.
I'm happy to review any code-suggestions pertaining to this issue, however I cannot take this up at the moment.


Change clockwise initiative to alphabetical order

This is a change to PR #592
I've modified the clockwise initiative code so that player character tokens are sorted alphabetically rather than based on user configuration.
This also makes the system a bit more robust as it should always put every combatant in the initiative order, even if something is preventing them from rolling. For instance, I tested on a group of combatants with various "problems":

In this picture, Aboleth is a regular NPC (rolls normally), Basilisk is an NPC whose actor has been deleted and therefore has no roll data (goes after the other NPCs), Light is a light actor with no roll data (goes at the end of the GM's turn), Elbin is a PC belonging to a logged-in player, and Jorbin is a PC with no attached player.
This way, the behavior of the clockwise initiative tracker more closely replicates the default initiative system, which will roll for all combatants even if their actors don't have roll data. By default, combatants without actors are assigned initiative 0, so in clockwise initiative I just grouped them at the end of the GM's turn.
The only thing remaining before implementing this is to integrate the i18n changes in the *.yaml files into the *.json files, but I'm not sure how to do that.

The only thing remaining before implementing this is to integrate the i18n changes in the *.yaml files into the *.json files, but I'm not sure how to do that.

All you need to do for that is add the English translations to en.yaml and Crowdin will take care of the rest once it has been merged and they'll just show up on Crowdin as new translations needed for the other languages and the default English will be added to their specific files.
So, just to clarify, once players/NPCs have rolled, the highest roll goes first and then the rest are looped alphabetically from that position?  For example.
ActorA rolls 7
ActorB rolls 11
ActorC rolls 5
Then the order will be: ActorB, ActorC, ActorA
Is that corrent?

Correct, in that case it would go B, C, A. And if, for example, A represented an NPC, then the order would be B, C, NPC1, NPC2, etc.

I screwed something up with the merge since the changes were spread across two pull requests. I will resubmit this entire change as one PR.

This should be okay to merge now. This includes the changes from both PRs.


Stop shadowing CF dataflow library

We used to do shadowing to avoid a conflict with the version of Checker Dataflow used by Error Prone.  But more recent versions of Error Prone depend on dataflow-shaded, so that conflict is no longer an issue.  We do not expect anything else in the annotation processing path to pull in a conflicting version of Checker dataflow, so we can probably just stop shadowing that dependence.  This will significantly simplify our build config and help us move to modern implementation and api dependencies in the nullaway module.

We stopped this as of 0.9.0


Supervisor 2023.08.2 breaks hass_nuki_bt

System Health details
System Information

version
core-2023.8.3

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.11.4

os_name
Linux

os_version
6.1.21-v8

arch
aarch64

timezone
Europe/Berlin

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

GitHub API Calls Remaining
5000

Installed Version
1.32.1

Stage
running

Available Repositories
1270

Downloaded Repositories
7

Home Assistant Cloud

logged_in
false

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Home Assistant OS 10.5

update_channel
stable

supervisor_version
supervisor-2023.08.1

agent_version
1.5.1

docker_version
23.0.6

disk_total
439.4 GB

disk_used
9.1 GB

healthy
true

supported
true

board
rpi4-64

supervisor_api
ok

version_api
ok

installed_addons
deCONZ (6.20.0), RaspberryMatic CCU (<IP_ADDRESS>30626), Home Assistant Google Drive Backup (0.111.1), Advanced SSH & Web Terminal (15.0.7), File editor (5.6.0), Samba share (10.0.2)

Dashboards

dashboards
1

resources
2

views
7

mode
storage

Recorder

oldest_recorder_run
12. August 2023 um 19:13

current_recorder_run
21. August 2023 um 21:55

estimated_db_size
281.29 MiB

database_engine
sqlite

database_version
3.41.2

Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue
After installing Supervisor 2023.08.2 from 2023.08.1 nearly all entities from hass_nuki_bt are gone.
After restoring my latest backup, hass_nuki_bt works as it should.
Reproduction steps

install Supervisor 2023.08.2

...
Debug logs
Sorry, no logger

Diagnostics dump
No response

From what I can see, the latest Supervisor is 2023.08.1. Supervisor version 2023.08.2 is not out yet
I have Home Assistant 2023.8.3 with Supervisor 2023.08.1, and for me everything works

You are right, as of today latest Supervisor version is 2023.08.01.
But last Monday evening I got version 2023.08.02 ready for installation, maybe accidently.


Introduced CommandValue

so we can properly differentiate between CommandError and
CommandSuccess.
Previously on the client side one would have needed to try to parse an
error, catch that parsing error and then try to parse success. With this
change, the client can simply parse CommandValue and get the result
(either success or error). CommandSuccess is now no longer a type of
its own, but a constructor of CommandValue. I kept CommandError, so
we don't have partial accessor functions for CommandValue.
What I am not sure about is:

Should we really return CommandValue Value from /local or should it be the more
type safe: CommandValue (Term Name)? For now I kept it as it was
before, which is CommandValue Value.
If we are going to return Term Name, I think CommandValue no longer
needs to be a type constructor, but could be a simple type isomorph
to CommandValue (Term Name).
Is it really necessary to have _crResult to be of type Value,
can't it be CommandValue directly? This would make things more typesafe and the
REST API a bit more approachable.

I am more on board with just returning Value in this case, because Term Name has some issues with non-exhaustive JSON specification, and the fact that it adds a little "too much" information. I'm also more a fan of a tentative Either-based approach

Oh, I almost missed that one. I don't really get that. The returned Value is an encoded Term Name so it holds exactly the same information, but less type safe - which is not an issue for a JS API, but it is for the typed servant API. So if we really don't  want to return a Term Name we should make it something else, just dropping type information only obscures the API, makes it more error prone and harder to figure out.

The issue here is the Pact API is not supposed to be a Haskell API but instead a REST/JSON API: the results should be plain JSON types and not some schema. So Value is accurate in this regard (with the caveat that Null is pathological here). We can simply newtype that for the purposes of the Servant API.

I'm fine however with CommandResult being monomorphic. Generally, I prefer Either in Haskell and customizing the JSON serialization for CommandResult to case on Either to keep the wire format the same. I

The issue here is the Pact API is not supposed to be a Haskell API but instead a REST/JSON API: the results should be plain JSON types and not some schema. So Value is accurate in this regard (with the caveat that Null is pathological here). We can simply newtype that for the purposes of the Servant API.

I beg to disagree. What good is an API if it is not properly specified? What JS sees in the end is JSON anyway, but types provide us wit a proper specification that is enforced and could also be used to provide a generated specification that is useful even to JS developers.
The lack of a proper specification already resulted in real world issues: The docs on read-the-docs are lacking:
https://pact-language.readthedocs.io/en/latest/pact-reference.html#listen
As you can see, only the success case is documented, the assumption that the failure case looks like the failure cases of the other responses, is wrong - it looks different, as can be seen by the now available JSON instance of CommandValue.
This and the fact that it is very hard without proper types to figure out what actually gets send here made the development of pact-web unnecessary hard - and in fact resulted in me parsing the JSON by hand, except Tom for example assumed that the result of (describe-module) is in fact a serialized Module and parsed it that way. That assumption was very justified as it totally looked that way at the time and the real type was obscured by those Values not providing any information at all. In the end it turned out it was not Module, but this so the code broke with the additon of meta.
Also the Pact code itself is harder to understand, as a Value can contain virtually anything, so usual "follow the types" to understand the code base breaks.
I would really argue against dropping the guarantees/documentation the type system provides us with, except there are really good reasons for it.

I'm fine however with CommandResult being monomorphic. Generally, I prefer Either in Haskell and customizing the JSON serialization for CommandResult to case on Either to keep the wire format the same.

That does not work unfortunately as CommandResult is only used internally and never sent over the wire.

Thanks! :-)
On 2/4/19 5:26 PM, Emily Pillmore wrote:

We've added the type annotation |CommandValue (Term Name)| to |cValue|
because |CommandFailure a| needs to be typed as |CommandValue (Term
Name)|, which is information that we cannot derive just from the
values |CommandFailure| contains - ergo in the case of
|CommandFailure|, the type parameter of |CommandValue| is a phantom
type, and we're specializing to |Term name| on the fly.

Hi there! What would need to be done to have this merged or is it too controversial? In any case I would need to know, so I am able to adjust pact-web accordingly. Thanks!

This PR is already based on: https://github.com/kadena-io/pact/pull/383 so remote verification does work on this branch.

Closing in favour of #375 which has merge conflicts resolved.


⚠ Jenkins Plugins Artifactory has degraded performance

In 5efc1a5, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1148 ms

Resolved: Jenkins Plugins Artifactory performance has improved in b43ce49 after 33 minutes.


get_parent_terragrunt_dir() returns wrong path

Hi,
I'm currently in the process of upgrading our Terragrunt/Terraform setup.
We have the following folder structure:
.
├── environments
│   └── core
│       └── workloads
│           ├── terragrunt.hcl
│           └── workloads.tf.tpl
├── global
│   ├── foo
│   │   └── bar.ps1
└── terragrunt.hcl

We have hooks in the parent terragrunt.hcl as well as in the terragrunt.hcl in the workloads folder.
Our hook (defined in the child terragrunt.hcl) should run a script in the global folder but fails as the wrong path is provided by get_parent_terragrunt_dir(). This is the hook definition:
before_hook "prepare_workloads" {
    commands = get_terraform_commands_that_need_vars()

    execute = [
      "pwsh",
      "-nop",
      "-nol",
      "-File",
      "${get_parent_terragrunt_dir()}/global/foo/bar.ps1",
    ]

    run_on_error = false
  }

This fails as get_parent_terragrunt_dir() returns /environments/core/workloads instead of /.
I've created a test hook with both interpolations:
before_hook "test" {
    commands = get_terraform_commands_that_need_vars()

    execute = [
      "echo",
      "${get_terragrunt_dir()}",
      "${get_parent_terragrunt_dir()}"
    ]

    run_on_error = false
  }

This is the output:
[terragrunt] 2019/06/18 21:14:55 Executing hook: test
[terragrunt] 2019/06/18 21:14:55 Running command: echo /environments/core/workloads /environments/core/workloads
/environments/core/workloads /environments/core/workloads

I've tried both from the parent file with terragrunt plan-alland with terragrunt in the workloads folder.

Can you share the contents of the include block in the child's terragrunt.hcl file?

Sure, this is from the child terragrunt.hcl:
include {
  path = find_in_parent_folders()
}

Thanks I was able to reproduce this and this is indeed a bug. Will start working on a fix.

Actually, I just realized that this was a misunderstanding on my part. After starting the implementation and digging deeper into the code and rereading our docs, I realized that the get_parent_terragrunt_dir only works in the included config. In other words, this functions is meant to be used in the context of the parent terragrunt.hcl file, not the child's terragrunt file.
Instead, you should actually reuse find_in_parent_folders in combination with dirname (dirname(find_in_parent_folders())) which was included in tg 0.19.4. If you need an absolute path, you can combine it with get_terragrunt_dir: ${get_terragrunt_dir()}/${dirname(find_in_parent_folders())}.

I can see a function that returns the absolute path to the parent terragrunt config folder being useful, so I will keep this open, but since there is a workaround, I will not be implementing that at the moment. I might pick this up again if I have time in a few weeks.

Is this the same issue fixed by https://github.com/gruntwork-io/terragrunt/pull/753?

Ah I think you are right @brikis98 !

Could you give https://github.com/gruntwork-io/terragrunt/releases/tag/v0.19.5 a shot (new binaries should show up shortly)?

Hi,
I think my issue is similar to the one of OP
.
├── environments
│   ├── rc.yml
│   └── dev.yml
│── eu-central-1
│   └── component
│          └── terragrunt.hcl
└── terragrunt.hcl

my parent terragrunt.hcl has something like this:
...
locals {
  env = yamldecode(file("${get_terragrunt_dir()}/${find_in_parent_folders(join("/", ["environments",get_env("ENVIRONMENT", "dev.yml")]))}"))
}

inputs = {
  environment_code = local.env.naming.environment_code
...

as locals are not propagated to child terragrunt.hcl I have to parse the yaml file again. So my child terragrunt.hcl looks like
include {
  path = find_in_parent_folders()
}

...

locals {
  env = yamldecode(file("${get_terragrunt_dir()}/${find_in_parent_folders(join("/", ["environments",get_env("ENVIRONMENT", "dev.yml")]))}"))
}

inputs = {
 app_settings = merge(local.env.vars, {
    "LOG_LEVEL"       = 2
  })
}

when running terragrunt plan within the child working directory everything runs smooth. But when doing a terragrunt plan-all from the parent dir, it breaks (replaced the full path output with placeholders):

Call to function "find_in_parent_folders" failed: ParentFileNotFound: Could not find a environments/dev.yml in any of the parent folders of
<PATH_TO_ROOT_TERRAGRUNT> Cause: Traversed all the way to the root..
[terragrunt] [<PATH_TO_ROOT_TERRAGRUNT_DIR>] 2019/09/25 17:44:20 Encountered error while evaluating locals.
[terragrunt] 2019/09/25 17:44:20 Error processing module at '<PATH_TO_ROOT_TERRAGRUNT>'. How this module was found: Terragrunt config file found in a subdirectory of <PATH_TO_ROOT_TERRAGRUNT_DIR>. Underlying error: <PATH_TO_ROOT_TERRAGRUNT>:47,52-75: Error in function call; Call to function "find_in_parent_folders" failed: ParentFileNotFound: Could not find a environments/dev.yml in any of the parent folders of <PATH_TO_ROOT_TERRAGRUNT>. Cause: Traversed all the way to the root..
[terragrunt] 2019/09/25 17:44:20 Unable to determine underlying exit code, so Terragrunt will exit with error code

Ah there is a known issue where you can't use the terragrunt directory related features in locals, because of the parsing order: include is parsed AFTER locals, not before, so unfortunately, you can't actually use functions like get_terragrunt_dir currently. See https://github.com/gruntwork-io/terragrunt/issues/814#issuecomment-518919484.
The goal is to introduce a concept like globals (https://github.com/gruntwork-io/terragrunt/issues/814) which would address this.

This is now fixed in https://github.com/gruntwork-io/terragrunt/releases/tag/v0.20.0


More intelligent matching for corpus checker

Currently, verify-corpus.pl compares its dumped HTML version of a file's coloration to what's in corpus_html.  This works much of the time, but there can be variations in the HTML that do not result in changes in color.  Adding more intelligence here would be nice.

I just implemented this.


hetzner-01: rename to caliban.nixos.org

We needed a name to create CNAMES to eventually, so here is one proposal, since we're going for pet names.
Caliban is the second largest moon of Uranus, and should somewhat fit into the existing naming scheme. I'm open to alternatives.
https://en.wikipedia.org/wiki/Caliban_(moon)
Once we have the DNS name deployed we can update the deployment address accordingly.
cc https://github.com/NixOS/nixos-org-configurations/issues/269#issuecomment-1693723824

That's fine by me.

I took rhea, haumea and eris and ran with it!

applied!


Add metrics subsystem

Thanks @waitinfuture for the PR.👍🏻  I have some minor comments and please have a look when you are free.

Thanks @waitinfuture for the PR.👍🏻 I have some minor comments and please have a look when you are free.

Hi @TanYuxin-tyx  I have addressed the comments, please take a look. Thanks!


HBASE-28038 Add TLS settings to ZooKeeper client

Looks like HBase has 2 different prefixes for ZooKeeper properties:
hbase.zookeeper.property.
hbase.client.zookeeper.property.
and we also have these:
hbase.zookeeper.quorum
hbase.client.zookeeper.quorum
Unfortunately I couldn't decrypt the meaning of each of every one of them, but we'd like to set system properties here which will be effective for all ZooKeeper clients in the same java process. Hence I only added the following properties which will be set in both client and server modes.
hbase.zookeeper.property.client.secure
hbase.zookeeper.property.clientCnxnSocket
hbase.zookeeper.property.ssl.keyStore.location
hbase.zookeeper.property.ssl.keyStore.password
hbase.zookeeper.property.ssl.keyStore.passwordPath
hbase.zookeeper.property.ssl.trustStore.location
hbase.zookeeper.property.ssl.trustStore.password
hbase.zookeeper.property.ssl.trustStore.passwordPath

Please also note that *passwordPath properties are supported only from ZooKeeper 3.8 (stable version today), but HBase is still on 3.5 meaning these properties are currently ineffective with the default setup.
Doc question: shall I add documentation in this patch or separately?

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
2m 40s
master passed

+1 :green_heart:
compile
0m 30s
master passed

+1 :green_heart:
checkstyle
0m 12s
master passed

+1 :green_heart:
spotless
0m 40s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 27s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 37s
the patch passed

+1 :green_heart:
compile
0m 29s
the patch passed

+1 :green_heart:
javac
0m 29s
the patch passed

+1 :green_heart:
checkstyle
0m 12s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
9m 23s
Patch does not cause any errors with Hadoop 3.2.4 3.3.5.

+1 :green_heart:
spotless
0m 40s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 35s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m  7s
The patch does not generate ASF License warnings.

24m 38s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5370/4/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5370

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 02ccedf229d0 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 89ca7f4ade

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
79 (vs. ulimit of 30000)

modules
C: hbase-common U: hbase-common

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5370/4/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 24s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m  4s
master passed

+1 :green_heart:
compile
0m 31s
master passed

+1 :green_heart:
checkstyle
0m 13s
master passed

+1 :green_heart:
spotless
0m 44s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 30s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 43s
the patch passed

+1 :green_heart:
compile
0m 30s
the patch passed

+1 :green_heart:
javac
0m 30s
the patch passed

+1 :green_heart:
checkstyle
0m 12s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
10m 21s
Patch does not cause any errors with Hadoop 3.2.4 3.3.5.

+1 :green_heart:
spotless
0m 41s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 36s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s
The patch does not generate ASF License warnings.

26m 45s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5370/6/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5370

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 9c5dbabd2c98 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5527dd9453

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
78 (vs. ulimit of 30000)

modules
C: hbase-common U: hbase-common

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5370/6/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9 ptal.


kube-prometheus - Grafana Password Secret can only be admin

What did you do?
Changed password for grafana
What did you expect to see?
I expected to login, but I could only login if I made the password admin
What did you see instead? Under which circumstances?
Environment
Cloud

Kubernetes version information:

1.8.1

Kubernetes cluster kind:

KubeSpray

Manifests:

cluster-monitoring

Prometheus Operator Logs:
Dont even know what this is ^

insert Prometheus Operator logs relevant to the issue here

I can only use value of "admin" for grafana password in kube secrets or else I can'y login.

How did you generate the base64 encoded string for this? You need to make sure to not have a line break in there.
The command we use to generate it is:
echo -n <password> | base64 --wrap=0

Hmmm let me give that a shot, most likely my issue.
On Nov 1, 2017 12:57 PM, "Frederic Branczyk"<EMAIL_ADDRESS>wrote:

How did you generate the base64 encoded string for this? You need to make
sure to not have a line break in there.
The command we use to generate it is:
echo -n  | base64 --wrap=0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/coreos/prometheus-operator/issues/716#issuecomment-341168315,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQTJt21jfQBN8l_5jCouErAgKVa4xVIwks5syKLzgaJpZM4QJ8sV
.

@brancz We shall never talk about this again...
So I do have one last question. I read the grafana watcher README and I'm still not sure what it does. It watches the configmap for changes to the dashboards and reloads them into grafana? Or when you change a dashboard in grafana it saves it to the configmap for persistence?
Great work on this by the way. Super easy to get great monitoring out of the box. I'm going to adjust some metrics and submit a PR.

No worries, it took me some time to figure it out on the first try as well when I was just seeing base64 encoded strings it's hard to catch a line-break :slightly_smiling_face: .
The grafana-watcher actually has nothing to do with Kubernetes, it just plays well into the way Kubernetes does things. The grafana-watcher just watches files and directories for changes and provisions the Grafana instances with the datasources and dashboards from the files. If any files changes, all dashboards are truncated, and re-provisioned. We do this as it allows us to operate Grafana in a declarative way, where we can scale Grafana up and down depending on our need, without the trouble of running a mysql database or equivalent, just to save some dashboard definitions globally. The operational side is one thing and it also allows us to check in the dashboards into version control, that way we can do reviews for the dashboards before applying them through CI or whatever tooling you choose to use. Lastly, we (or rather a very hard working community) recently converted the Grafana dashboard json definitions to use grafanalib, which allows us to define the dashboards as python code, and ultimately allow us to write abstractions above graphs as they are 98% of the time exactly the same with a different query.
Note that we've been talking with Grafana and they're actually working on getting most of this functionality of the grafana-watcher directly integrated into Grafana (slightly different but same end result), so hopefully in the future we won't need to maintain the grafana-watcher, but until then it's a way to fast forward into the future :slightly_smiling_face: .

@brancz
Kube-prometheus has grafana-watcher set to look at a configmap, but I thought that was static, a 1 time mapping when the pod is created? So if you did update the config map you would still have to kill that pod to get grafana-watcher to update grafana?

The grafana-watcher actually watches file-system events, because when a ConfigMap gets updated, it gets re-mounted into a Pod, triggering the file-system events, which makes grafana-watcher re-provision the "local" Grafana instance.

This makes updating Grafana dashboards very easy, as a CI pipeline just applies the ConfigMap with dashboards, and the grafana-watcher ensures the Grafana instance ends up in the correct state, without having to worry about the existing state currently present in the Grafana instance.

@brancz Ahhh I didn't know that config map values got reloaded live into pods. Neat =). Thanks for being so helpful.
I did change some metrics in the included dashboards to be more "fitting"  and my api response time and scheduler charts didn't work at all with kubernetes 1.8.1 one. Should I submit a PR?

@brancz
Before

After

Couple things like that ^

Generally yes, all contributions are welcome :slightly_smiling_face:, the exact queries we'll have to review to ensure that they are still appropriate.


ROS2 Foxy turtlebot3 navigation run fail when set initial pose in rviz2

ISSUE TEMPLATE ver. 0.4.0

Which TurtleBot3 platform do you use?

[ ] Burger
Waffle
[ ] Waffle Pi

Which ROS is working with TurtleBot3?

[ ] ROS 1 Kinetic Kame
[ ] ROS 1 Melodic Morenia
[ ] ROS 1 Noetic Ninjemys
[ ] ROS 2 Dashing Diademata
[ ] ROS 2 Eloquent Elusor
ROS 2 Foxy Fitzroy
[ ] etc (Please specify your ROS Version here)

Which SBC(Single Board Computer) is working on TurtleBot3?

[ ] Intel Joule 570x
[ ] Raspberry Pi 3B+
[ ] Raspberry Pi 4
etc (  Intel Nuc6i7KYK )

Which OS you installed in SBC?

[ ] Raspbian distributed by ROBOTIS
[ ] Ubuntu MATE (16.04/18.04/20.04)
Ubuntu preinstalled server (18.04/20.04)
[ ] etc (Please specify your OS here)

Which OS you installed in Remote PC?

[ ] Ubuntu 16.04 LTS (Xenial Xerus)
[ ] Ubuntu 18.04 LTS (Bionic Beaver)
Ubuntu 20.04 LTS (Focal Fossa)
[ ] etc (Please specify your OS here)

Specify the software and firmware version(Can be found from Bringup messages)

Software version: [x.x.x]
Firmware version: [x.x.x]

Specify the commands or instructions to replicate the issue.
terminal 1.  ros2 launch turtlebot3_bring robot.launch.py
terminal 2.  ros2 launch nav2_bringup bringup_launch.py map:=$HOME/lab_map.yaml
or - ros2 launch turtlebot3_navigation2 navigation2.launch.py map:=$HOME/lab_map.yaml
terminal 3.  ros2 run rviz2 rviz2 -d $(ros2 pkg prefix nav2_bringup)/share/nav2_bringup/rviz/nav2_default_view.rviz

Copy and Paste the error messages on terminal.

error log:
tb3_navigation2_launch_fail_log_20201116.txt
used map create by turtlebot3 cartographer brfore run navigation2 test
lab_map3.zip

Please describe the issue in detail.

I meet a problem is when I set initial pose in rviz, rviz crash and navigation terminal print more error info. Test map create by turtlebot3 cartographer before run navigation2 test.
this fail only meet on real reboot, simulate not meet.

@rafaelhuang-intel
Hi. I'm sorry for the late reply.
I will check this issue and get back to you.
Thank you for your patience

@ashekim
I've updated all ros-foxy-turtlebot3 relate package and tried again, this issue disappear, so I think this issue may fixed. I will close this one.


Add transaction

PR Implements #20 (Add Service Transaction) Use case for the Provider. In doing so I refactored a lot of the code base as well. The core functionality and design of the system remains the same, however there are a few interface changes with entity classes.

Despite the failing checks, I'm going to merge this so everyone can use the updated domain entities. We will have to revisit the code coverage later.

its looking good so far, excellent code quality as usual. still a little fuzzy on the I/O details since it seems you may be making some changes, and while i am not opposed to that i do not see the exact reasoning for why. in either case it makes sense overall and is even a little simpler to understand and read.


Flatten array arguments in legacyMode multi commands

Description
https://github.com/redis/node-redis/issues/2071
In legacy mode multi commands, array arguments are not flattened. This is a bug.
For example, this command that adds two elements to a set:
sadd('a', ['b', 'c'])
when called in a multi will instead add the single element "['b', 'c']".
This bug is a blocker that surfaced while trying to upgrade from node-redis v3 to v4.
The root problem seems to be an inconsistency with the schema of queued multi vs single command arguments.
In a non-multi, the command that gets queued up in our sadd example would be:
["sAdd", "a", ["b", "c"]]
This is later flattened before getting serialized into a redis command.
However, multi commands queue up commands as the tuple: [commandName, args]. In this example:
["sAdd", ["a", ["b", "c"]]]
This is also flattened before getting serialized into a redis command, but only to one level, which is why this issue only surfaces for array arguments.
We fix this inconsistency in this diff by enqueing multi commands in the same format as their non-multi counterparts.
Thank you!

Checklist

[x] Does npm test pass with this change (including linting)?

I am not able to run the entire test suite on my machine (even master), however I have confirmed that the entire client/lib/client/index.spec.ts file passes on this branch, and the newly added unit test fails on master

[x] Is the new or changed code fully tested?
[x] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

No document changes needed

@JPricey Nice catch! Thanks for contributing! :)

@JPricey Nice catch! Thanks for contributing! :)


[Storage] az storage account generate-sas: Migrate to track2 sdk

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Storage

/azp run


Fix install command for clippy, uses rustup

Trying to install with cargo fails with the following error:
Caused by:
  failed to run custom build command for `clippy v0.0.302`

Caused by:
  process didn't exit successfully: `/var/folders/66/180ynjt55z74l338gh0stq0c0000gn/T/cargo-installkNqJBB/release/build/clippy-bcde765a4c9acf93/build-script-build` (exit code: 1)
--- stderr

error: Clippy is no longer available via crates.io

help: please run `rustup component add clippy-preview` instead

Just a tiny doc update

Good catch. Thanks for the help!


Ap 2324 ccms false passported

What
Link to story
Add new attributes to the base.yml
Add the attributes to the false_attributes list for the tests
Add the new attribute sections to the case_add_request.xml file
Checklist
Before you ask people to review this PR:

[ ] Tests and rubocop should be passing: bundle exec rake
[ ] Github should not be reporting conflicts; you should have recently run git rebase master.
[ ] There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
[ ] The PR description should say what you changed and why, with a link to the JIRA story.
[ ] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[ ] You should have checked that the commit messages say why the change was made.

Codecov Report

Merging #2629 (d19272f) into master (79214b1) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #2629   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          528       528
  Lines        10863     10863
=========================================
  Hits         10863     10863

Continue to review full report at Codecov.

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


Will bank-vaults work with Istio already running on k8s?

I am asking to see if running bank-vaults with Istio on k8s would be a problem.
Additional context
We will be running Istio with full mTLS between pods so just seeing if this would work in a setup with FULL Istio deployed.

Hey @nippip - it should not be a problem. Do you have a particular scenario in mind which you think might not work? We semi use them together (with our Istio operator - the BV operator/Vault is running in the control plane and webhook/vault-env in the cluster) and it's fine.
Usually, with Istio, you will have to set up/choose the namespaces where the sidecar injection work and those are different ones anyway.
Please let us know how it goes for you. Btw, how are you deploying/managing Istio, are you using the operator?

Hi @nippip, some users are successfully running BV with Istio enabled clusters as well, please see the following Slack messaging: https://community-banzaicloud.slack.com/archives/DJCC1GQBB/p1557750927001800

@matyix @bonifaido Thank you both for the quick responses and good info / links!  I'll use the slack channel going forward and close this "issue".


Inkscape won't open after install

Under Yosemite 10.10.1, running brew cask install inkscape installs Inkscape, but running the app fails. An Inkscape menu is shown for a split second and then disappears. No error prompts or anything. I have x11 v1.0.1 installed. Any suggestions?
I seem to remember it needing xquartz under recent versions of OS X. Is this still the case?

This issue can be closed since the cask now depend on x11 being installed.


Need a way to check if discord is running, or have a callback run when discord isn't present when calling Discord_Initialize.

In my code, I have a loop like this:
	InitDiscord();
	while (discord_status == DISCORD_WAITING)
	{
		handleevents();
		Discord_RunCallbacks();
	}

Because I need to wait for Discord to be ready, before I can proceed with the rest of the game's startup (since multiplayer connections happen during this phase).
This loop works perfectly fine if Discord is running, it'll wait until Rich Presence is ready, and then proceed. (All the different handlers will change the value of DISCORD_STATUS, including the error callback.)
However, if Discord isn't running, there's no way to check this, not even the error callback fires, and thus the game just hangs here. I'd like to have some sort of callback or return value here.

You can expect that Discord is around after the ready callback, and you can expect that Discord has gone away if you get the disconnect cb, but it's not really safe to assume those states are stable. I would expect those cbs to happen pretty quickly, so perhaps just a timeout would meet your need? I would guess that if Discord is really required you'd bring up UI saying so? I think just in general I'm confused with what you're trying to build.

A timeout would be irritating for people without Discord. (Such as people setting up matches the traditional way without it.)
This is an older game, Duke Nukem 3D to be specific. The host starts the game with a certain command line parameter, which puts them in a "Waiting for players" mode, before initializing anything else in the engine. It waits until the appropriate number of players connect, then loads the game fully and puts everyone together in a match.
During this waiting for players stage is where the host could send invites through Discord.
I need this bit, because joining players launched through Discord need to halt at a certain point in the code until Discord is ready, and the joinGame callback fires, to make sure the joining player's game has all of the right information (IP address to connect to, mods loaded, etc) before proceeding to load the appropriate mods, connecting to the right IP, etc during initialization.

Of course, I could avoid this altogether if there were a way to set what command line arguments are used when another player launches the game through discord.

Not an answer for this repo, but our GameSDK does return specific errors for when Discord is either not running or not installed.
Anyone else looking for an answer for this should move to our GameSDK; it has init callback results like:

NotInstalled
NotRunning

And will fire these if Discord is open but the user is logged out, as well.
https://discordapp.com/developers/docs/game-sdk/sdk-starter-guide
If you need help, come join us in the Discord at https://discord.gg/discord-gamesdk !


java script to do list project

hii i made an javascript to do list project and i have deployed it on heroku server so i wiil add its link to your official website

@harshitsuthar77731Please add your project to the official website also.

how to add my project to the official website please guide me

how to add my project to the official website please guide me

Please refer to #4


signal-desktop-beta: drop

The beta increases the complexity of the signal-desktop package for maintainers and releases of signal-desktop happen very frequently.
Things done
Ran a build to confirm an error message gets thrown.

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/)
25.05 Release Notes (or backporting 24.11 and 25.05 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.

@teutat3s Could you handle the backport?


Crash while editing in interactive window

[Window Title]
devenv.exe
[Main Instruction]
An unexpected error occurred
[Content]
Please press Ctrl+C to copy the contents of this dialog and report this error to our issue tracker.
[^] Hide details  [Close]
[Expanded Information]
Build: 15.8.18131.2

System.NullReferenceException: Object reference not set to an instance of an object.
   at Microsoft.VisualStudio.Language.Intellisense.Implementation.CompletionSession.get_CompletionSets()
   at Microsoft.PythonTools.Intellisense.IntellisenseController.HandleChar(Char ch)
   at Microsoft.PythonTools.Intellisense.IntellisenseController.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
   at Microsoft.PythonTools.Language.EditFilter.ExecWorker(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
   at Microsoft.PythonTools.Language.EditFilter.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.VisualStudio.Telemetry.WindowsErrorReporting.WatsonReport.GetClrWatsonExceptionInfo(Exception exceptionObject)

When Dismiss() is called, it sets _completionSets to null, so you can't call CompletionSets after that without crashing.
We can work around that by checking IsDismissed first, I'm testing a fix that does that.
I filed internal bug 616189 to the editor team, as they should have better handling of that scenario.


Test failure in mrustc 0.9 - enums - Borrow

Calling mrustc -O -L output -o output/local_tests/enums.exe samples/test/enums.rs
Target Load: V V V
(0.00 s) Target Load: DONE
Parse: V V V
(0.00 s) Parse: DONE
LoadCrates: V V V
(0.19 s) LoadCrates: DONE
Expand: V V V
(0.00 s) Expand: DONE
Implicit Crates: V V V
(0.00 s) Implicit Crates: DONE
Resolve Use: V V V
(0.00 s) Resolve Use: DONE
Resolve Index: V V V
(0.00 s) Resolve Index: DONE
Resolve Absolute: V V V
(0.00 s) Resolve Absolute: DONE
HIR Lower: V V V
(0.00 s) HIR Lower: DONE
Resolve Type Aliases: V V V
(0.00 s) Resolve Type Aliases: DONE
Resolve Bind: V V V
(0.05 s) Resolve Bind: DONE
Resolve HIR Markings: V V V
(0.00 s) Resolve HIR Markings: DONE
Resolve UFCS Outer: V V V
(0.00 s) Resolve UFCS Outer: DONE
Resolve UFCS paths: V V V
(0.00 s) Resolve UFCS paths: DONE
Constant Evaluate: V V V
(0.00 s) Constant Evaluate: DONE
Typecheck Outer: V V V
(0.00 s) Typecheck Outer: DONE
Typecheck Expressions: V V V
(0.00 s) Typecheck Expressions: DONE
Expand HIR Annotate: V V V
(0.00 s) Expand HIR Annotate: DONE
Expand HIR Closures: V V V
(0.00 s) Expand HIR Closures: DONE
Expand HIR VTables: V V V
(0.00 s) Expand HIR VTables: DONE
Expand HIR Calls: V V V
(0.00 s) Expand HIR Calls: DONE
Expand HIR Reborrows: V V V
(0.00 s) Expand HIR Reborrows: DONE
Expand HIR ErasedType: V V V
(0.00 s) Expand HIR ErasedType: DONE
Typecheck Expressions (validate): V V V
(0.00 s) Typecheck Expressions (validate): DONE
Lower MIR: V V V
samples/test/enums.rs:12: BUG:src/mir/from_hir_match.cpp:1460: Matching borrow invalid pattern - 1
samples/test/enums.rs:12: note: From here
llvm 3.9.1.

Can you please post the test code?


Add API sample code to AccessibilityGuideline

It should showcase how to use the individual guidelines in a test.

/cc @chunhtai


Releases/v2.0

Hey @dangoslen, the Changelog Enforcer failed. Can you take a look at the error below and correct it? Thanks!
The latest version in the changelog does not match the expected latest version of 2.0.0!

Hey @dangoslen, the Changelog Enforcer failed. Can you take a look at the error below and correct it? Thanks!
The latest version in the changelog does not match the expected latest version of 2.0.0!

Preview of Release Notes to be Created
Breaking Changes

newVersionLineNumber is removed. Instead, if a version is not found, this action now looks for an unreleased version. If neither version can be found, the action will error. Read more in the README.md


MOTR-170 Additional fixes for RC

Key Changes:
Minor responsive styling adjustment on analysis pages for larger mobile device screens.

Merged MOTR-170