User roles are being ignored?

Using provided firewall (oauth2_resource), when AccessTokenToken is being created, roles are left as an empty array, therefore in controller, after authentication, $this->getUser() returns null. How do I set proper roles for access_token user?

!? So already solve the issue?

I did a mistake investigating issue I was having.
What really happens is that created access token has user set as string, not as an object.
Do you have any tips how to set user object into token, not user username as string?


select2.org is not working

Hello guys. I tried access select2.org to view the documentation, but i get connection timeout message.

In the meantime you can use https://web.archive.org/web/20210220210734/https://select2.org/

+1 need documentation please 🙏
Thanks @Seb35 for workaround

Good Job 💯
Thanks @Seb35

Seems like it works now!

it works now!

Didn't even notice this. We ran into a DNS resolution issue that was handled. We are still looking in to longer-term solutions that make better use of the Cloudflare cache to keep our docs online at all times.


Clarifications Needed: Test Performance, Batch Size, and Initial Weights

Dear Authors,
Thank you for making your work and code available to the community. We have encountered a few issues while reproducing the results from the paper using the provided repository. We would appreciate your guidance on the following points:

Test Performance Discrepancy:

When using the provided pre-trained weights and running the test script, we observed that the performance is consistently lower by one point compared to the results reported in the paper. Could you please provide insights into any additional steps or configurations that might be necessary to achieve the reported performance?

Batch Size:

In the paper, the batch size is mentioned as 8, while the default batch size in the provided code is set to 16. Could you please confirm which batch size was used for the results reported in the paper? Additionally, if there were any specific reasons for this discrepancy, we would appreciate an explanation.

Initialization of Weights:

It is not clear from the provided documentation whether the initial weights in the training process are randomly initialized or if they use pre-trained weights. Could you please clarify the weight initialization strategy used in your experiments?

Any assistance or clarification on these points would be highly appreciated. Thank you for your time and support.
Best regards,
Amordia

Sorry but I don't know the reason for the test performance discrepancy. Maybe it is becasue the gpu difference? We test the model on RTX 3090.
We have trained the model with different batch sizes and found that the size of the batch does not significantly affect performance. If GPU allows, we recommend using a larger batch size for faster training.
We used the checkpoints pre-trained on ImageNet by MAE in this https://github.com/facebookresearch/mae/issues/8 which is same as CounTR.


Error parameter "boiler_state" nor detected nor configured

Hi,
Firstly thank you for your work.
hen i add a card with type: custom:centrometal-boiler-card
My boiler is a biotec plus
I have the error

Thank you

Do you have centrometal boiler device in devices? And is there an entity like sensor.*_boiler_state ?

That means you entered prefix when adding integration, either define prefix for card (prefix are introduced to support multiple boilers in same HA instance), so you shall have something like:
type: custom:centrometal-boiler-card
prefix: gmx_easy
Od you can remove integration and add again without prefix.
Let me know if this helps.

Wait, you do not have biotec plus, but gmx easy, is this your boiler https://www.chaudieres-morvan.com/chaudiere/gmx-easy/ ?
If yes, then this boiler is not supported and I would need to add support for it, looks it is centrometal boiler under some other brand.
The easiest would be if you contact me on whatsapp +385 91 185 7050 and give me credentials so I can connect using your account and add the boiler.

You can also try following (may or may not work, not sure):

type: custom:centrometal-boiler-card
device_type: biopl
prefix: gmx_easy

Added support for Morvan GMX EASY boilers


Gap between slides and pagination until you reach last slide (IE11)

As far as I can tell this only happens when using lazy load in IE11.
Happens on this page:
http://owlcarousel.owlgraphic.com/demos/lazyLoad.html
On first loading notice the gap:

Scroll to last slide and the gap disappears:

Is there any easy fix?
Cheers

Hi, there... did you ever figure this one out? I'm having the same issue. Please let me know, thank you!

@pshudgins No, sorry. Never had time to look into it.


Fix the issue with role update

Purpose
Partially fix https://github.com/wso2/product-is/issues/13198
Through management console follow below steps.

Add role 1 with some permissions. Then scim level role1 & Internal/system_primary_role1 will be created
Rename rolr1->role2. Then scim level roles will be updated and now role2 & Internal/system_primary_role2 will be available.
Delete role2. There will be two calls goes for deletion one for role2 other one for system_primary_role2. But in scim level Internal/system_primary_role2 will not get deleted because [1] is skipping the useroperation event listeners if the role start with system prefix.
Now create role 1 again, scim level role1 & Internal/system_primary_role1 will be created.
Rename role1 to role2. while updating we will update the system_primary_role1 to system_primary_role2 but in scim level "system_primary_role2" already exists there fore we will end up with unique key violation from scim.

This PR will trigger the pre delete internal roles events for user operation event listeners hence the deletion will happen correctly.
[1] https://github.com/wso2/carbon-kernel/blob/v4.7.0-m13/core/org.wso2.carbon.user.core/src/main/java/org/wso2/carbon/user/core/common/AbstractUserStoreManager.java#L7165-L7166

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2207280444

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2207280444
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2218350203

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2218350203
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2218350203

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2218350203
Status: success


Update tenancy.factory.ts

In my solution, I am hitting a cannot overwrite model after compiled error. By adding this line of logic, it will check if the mongoose model exists and return that for a particular connection rather than trying to overwrite it. Appreciate the great repo.
I borrowed this solution from several blogs that mention the challenge with mongoose like this one: https://nesin.io/blog/fix-mongoose-cannot-overwrite-model-once-compiled-error
Thanks!

Hi @jordanberry Thanks for submitting the PR.

Is working great - thanks for merging!


NSNotificationCenter is not thread safe.

NSNotificationCenter is not thread safe, removing, adding and notifying observers from different threads results in a crash.
On the reference platform NSNotificationCenter is thread safe.

@rajsesh I was hoping I'd get this one :wink:


Filter case prop

Added a property to enable case sensitive filtering (#5)

Hey @andrey-semin
Awesome! We loved your contribution! 🎉
First, I'd like to ask your opinion about renaming the method formatText() to _formatText(), keeping the same pattern. What you think about it?
And second, what about checking the _properties.caseSensitive directly inside of _renderList() instead of passing it as a parameter?
We'd love to hear your opinion about both of these points. 💪 💪

First, I'd like to ask your opinion about renaming the method formatText() to _formatText(), keeping the same pattern. What you think about it?

If it's a pattern across the project then we should keep it consistent =, will update tomorrow

And second, what about checking the _properties.caseSensitive directly inside of _renderList() instead of passing it as a parameter?
I prefer to pass data to functions rather then working with some outside state. It gives more control, testability and is a kind-of-best-practice (see pure functions and functional programming

I prefer to pass data to functions rather then working with some outside state. It gives more control, testability and is a kind-of-best-practice (see pure functions and functional programming

Great! That's why I love the open source community: everyday we have the chance to learn something new. 🎉 So, in this case, I'll probably be applying some changes based on these arguments to the rest of the functions.
Since you said you agree about renaming formatText() to _formatText(), I'll be waiting the rename to accept the pull request.

updated

Awesome, @andrey-semin! Thank you for your Pull Request and thank you for your enriching comments. Yay! 🎉


Drop support for Python 3.7

It's EOL now, as of 2023-06-27 (two days ago): https://devguide.python.org/versions/

One problem: https://glitch.com/ still provides 3.7:
$ python3 --version
Python 3.7.10

Posted on the Glitch feedback forum about this here: https://support.glitch.com/t/upgrade-python-version-from-3-7-which-is-now-eol-to-something-more-recent/63011

Dupe of:

#2097


ensure entire directories are copied for ADDs

Prior to this change, a mojo configuration like
<resource>
  <directory>some/directory</directory>
  <targetPath>/data</targetPath>
</resource>

would generate an ADD /data /data instruction but the some/directory
directory is never copied into the Docker build context directory, so
the actual build will fail.
With this change, the directory is copied to the build context
directory.
Note: it might seem weird that the targetPath is used as the directory
name to use inside the build context as opposed to just the target
path inside the container, but this is to mirror behavior when copying
individual files.
Fixes #146.

@davidxia @gimaker @rohansingh @dawidmalina

awesome :+1:

:+1:

FYI -- I'm using 0.4.0, which should have this fix in it as I understand it.  This is still an issue.  No matter what I do, can't get my application jar into the docker image.

@Derek-Ashmore what does your Dockerfile look like?

Thanks for the quick response -- the last line of this is just me trying to find the jar that's supposed to be there.
dockerfile.txt

@Derek-Ashmore is your build actually putting the jar in target/app? Keep in mind that this plugin uses the <directory> inside the <resource> as the directory in which to find the file.
If you run your build with -X the plugin will spit out a bunch of information about where it is copying files which I think would help you out.
btw, you may find it is much easier to use Maven properties such as ${project.basedir} or ${project.build.directory} so that your build is portable rather than using absolute file paths.

Yes, the build is putting the jar in target/app.  According to the debug messages, it is getting moved to docker/app during the build process.  Thx for looking at this.
Thanks,Derek
-------- Original message --------
From: Matt Brown<EMAIL_ADDRESS>Date: 02/01/2016  8:32 AM  (GMT-06:00)
To: spotify/docker-maven-plugin<EMAIL_ADDRESS>Cc: "Derek C. Ashmore"<EMAIL_ADDRESS>Subject: Re: [docker-maven-plugin] ensure entire directories are copied for
ADDs (#157)
@Derek-Ashmore is your build actually putting the jar in target/app? Keep in mind that this plugin uses the  inside the  as the directory in which to find the file.
If you run your build with -X the plugin will spit out a bunch of information about where it is copying files which I think would help you out.
btw, you may find it is much easier to use Maven properties such as ${project.basedir} or ${project.build.directory} so that your build is portable rather than using absolute file paths.
—
Reply to this email directly or view it on GitHub.

Hi -- I have more information.  The jar file is in the temporary gz file that gets transmitted to the docker machine.  A link to the gz file transmitted is available here:  https://drive.google.com/file/d/0BywOAPVO4FvAcHZEUVRKUHFuVkU/view?usp=sharing
Also, I put a short test together with the Docker Client directly -- I get the same behavior.  The gz (with the jar) goes up, the docker file executes with the same debug output as if I go through the maven plugin, and the jar is not in the resulting container.
Test Docker client code:
@Test
public void test() throws Exception {
final DockerClient docker = DefaultDockerClient.fromEnv().build();
//docker.pull("busybox");
docker.build(Paths.get("src/test/docker"), "moneta-springboot-0.9.2-alpha", new BuildParam[0]);
}

Hi,
I see in the Dockerfile that you are using the ENTRYPOINT, but I don’t see where you are actually adding the jar to the image.  You need a COPY or ADD command e.g.
ADD /app/moneta-springboot.jar /app/
Chris Snider
Senior Software Engineer
Intelligent Software Solutions, Inc.
[Description: Description: Description<EMAIL_ADDRESS>From: Derek C. Ashmore<EMAIL_ADDRESS>Sent: Tuesday, February 02, 2016 4:01 AM
To: spotify/docker-maven-plugin<EMAIL_ADDRESS>Subject: Re: [docker-maven-plugin] ensure entire directories are copied for ADDs (#157)
Hi -- I have more information. The jar file is in the temporary gz file that gets transmitted to the docker machine. A link to the gz file transmitted is available here: https://drive.google.com/file/d/0BywOAPVO4FvAcHZEUVRKUHFuVkU/view?usp=sharing
Also, I put a short test together with the Docker Client directly -- I get the same behavior. The gz (with the jar) goes up, the docker file executes with the same debug output as if I go through the maven plugin, and the jar is not in the resulting container.
Test Docker client code:
@Testhttps://github.com/Test
public void test() throws Exception {
final DockerClient docker = DefaultDockerClient.fromEnv().build();
//docker.pull("busybox");
docker.build(Paths.get("src/test/docker"), "moneta-springboot-0.9.2-alpha", new BuildParam[0]);
}
—
Reply to this email directly or view it on GitHubhttps://github.com/spotify/docker-maven-plugin/pull/157#issuecomment-178510034.

thanks @chris-snider , that would definitely be the cause of the problem then for @Derek-Ashmore .

OK -- I'm officially embarrassed.  Thanks for the assistance - sorry for the red herring.

It is the individual trees we sometimes miss when looking at an entire forest :D
Chris Snider
Senior Software Engineer
Intelligent Software Solutions, Inc.
[Description: Description: Description<EMAIL_ADDRESS>From: Derek C. Ashmore<EMAIL_ADDRESS>Sent: Tuesday, February 02, 2016 7:59 AM
To: spotify/docker-maven-plugin<EMAIL_ADDRESS>Cc: Chris Snider<EMAIL_ADDRESS>Subject: Re: [docker-maven-plugin] ensure entire directories are copied for ADDs (#157)
OK -- I'm officially embarrassed. Thanks for the assistance - sorry for the red herring.
—
Reply to this email directly or view it on GitHubhttps://github.com/spotify/docker-maven-plugin/pull/157#issuecomment-178617260.


Question: Can I use cuda 11, for gpu inference on mms

I tried building mms docker image on top of  nvidia/cuda:11.6.0-runtime-ubuntu20.04
but this failed.
Is this possible ?

It should be possible to run it with cuda 11. Can you post the error that you ran into?

hi @maaquib , even i thoght so, but i am getting this error,

@kaushal-idx This seems like an issue with cuda version incompatibility with your container, not an MMS issue. I'd suggest checking the driver version on your host machine and using an appropriate base container

@maaquib I think it fixed that issue, but now i am getting another one
my driver is 470.103.01 and cuda version on host machine is 11.4

I tried replicating the same in mms-gpu-docker
FROM nvidia/cuda:11.4.0-cudnn8-runtime-ubuntu20.04

ENV PYTHONUNBUFFERED TRUE

RUN apt-get update && \
    DEBIAN_FRONTEND=noninteractive apt-get install --no-install-recommends -y \
    fakeroot \
    ca-certificates \
    dpkg-dev \
    g++ \
    python3-dev \
    openjdk-8-jdk-headless \
    curl \
    vim \
    && rm -rf /var/lib/apt/lists/* \
    && cd /tmp \
    && curl -O https://bootstrap.pypa.io/pip/3.6/get-pip.py \
    && python3 get-pip.py

RUN update-alternatives --install /usr/bin/python python /usr/bin/python3 1
RUN update-alternatives --install /usr/local/bin/pip pip /usr/local/bin/pip3 1

RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install --no-install-recommends -y \
    ffmpeg libsm6 libxext6

RUN pip install  multi-model-server \
    && pip install  mxnet-cu92mkl==1.4.0

RUN useradd -m model-server \
    && mkdir -p /home/model-server/tmp
COPY --chown=model-server dockerd-entrypoint.sh /usr/local/bin/dockerd-entrypoint.sh
COPY --chown=model-server config.properties /home/model-server
COPY --chown=model-server extract_snapshot_details.py /home/model-server
COPY --chown=model-server get_snapshot.py /home/model-server
RUN chmod +x /usr/local/bin/dockerd-entrypoint.sh \
    && chown -R model-server /home/model-server
EXPOSE 8080 8081
USER model-server
WORKDIR /home/model-server
ENV TEMP=/home/model-server/tmp
ENV AWS_PROFILE=textract
COPY --chown=model-server requirements.txt .
ENV PATH="/home/model-server/.local/bin:${PATH}"
RUN pip install -r requirements.txt
RUN mkdir -p /home/model-server/model-store/logs
ENTRYPOINT ["/usr/local/bin/dockerd-entrypoint.sh"]
CMD ["serve"]

Not sure what am i doing wrong, but i am getting the following error

Can you please help me, solve this


Ability to auto-build s2n with AWS CodeBuild Service during Pull Requests

Travis CI Builds can be very slow when there are multiple developers sending PR's to s2n at once since each PR is queued behind one another. The 50 minute build limit in Travis also forces us to break up the build in awkward ways so that different tests can be run in parallel, and limits us to how far we can dial up the Fuzz and SAW tests since they can easily go over the 50 minute limit.
It seems like we would get better stability and more consistent performance if we used AWS CodeBuild.
As of this writing, the latest commit to https://github.com/awslabs/s2n/pull/460 has been stuck in a Travis queue waiting for build hosts for over an hour with no progress. Travis CI is a bottleneck for s2n development in this regard.
Links:

http://docs.aws.amazon.com/codebuild/latest/userguide/getting-started.html
https://aws.amazon.com/blogs/devops/integrating-git-with-aws-codepipeline/
http://docs.aws.amazon.com/codepipeline/latest/userguide/tutorials-four-stage-pipeline.html
http://docs.aws.amazon.com/codebuild/latest/userguide/how-to-create-pipeline.html
https://github.com/svdgraaf/github-codebuild-webhook

Create a Static Website with Build Output Hosted in an Amazon S3 Bucket  appears to solve the issue of other developers not being able to see the build output logs. Its still unclear how people might setup CodeBuild on their forks.

And add a S3 lifecycle rule to delete the logs after a week.

Doing some experimentation supporting CodeBuild will require some changes:

Fix all shell scripts to be posix compliant:

. instead of source
[ = ] instead of [[ == ]]
https://github.com/andrewhop/s2n/commit/09d0ae0566c589e9c499dc359d2c57b6fb14a373

Can't install shellcheck with the default ubuntu repositories

[Container] 2019/05/02 00:29:04 Running command apt-get update -y
Ign http://archive.ubuntu.com trusty InRelease
Get:1 http://archive.ubuntu.com trusty-updates InRelease [65.9 kB]
Get:2 http://archive.ubuntu.com trusty-security InRelease [65.9 kB]
Get:3 http://ppa.launchpad.net trusty InRelease [20.8 kB]
Get:4 http://archive.ubuntu.com trusty Release.gpg [933 B]
Get:5 http://archive.ubuntu.com trusty Release [58.5 kB]
Get:6 http://archive.ubuntu.com trusty-updates/main Sources [531 kB]
Get:7 http://archive.ubuntu.com trusty-updates/restricted Sources [6444 B]
Get:8 http://archive.ubuntu.com trusty-updates/universe Sources [288 kB]
Get:9 http://archive.ubuntu.com trusty-updates/main amd64 Packages [1452 kB]
Get:10 http://archive.ubuntu.com trusty-updates/restricted amd64 Packages [21.4 kB]
Get:11 http://archive.ubuntu.com trusty-updates/universe amd64 Packages [669 kB]
Get:12 http://archive.ubuntu.com trusty-security/main Sources [220 kB]
Get:13 http://archive.ubuntu.com trusty-security/restricted Sources [5050 B]
Get:14 http://archive.ubuntu.com trusty-security/universe Sources [126 kB]
Get:15 http://ppa.launchpad.net trusty/main amd64 Packages [3505 B]
Get:16 http://archive.ubuntu.com trusty-security/main amd64 Packages [1025 kB]
Get:17 http://archive.ubuntu.com trusty-security/restricted amd64 Packages [18.1 kB]
Get:18 http://archive.ubuntu.com trusty-security/universe amd64 Packages [375 kB]
Get:19 http://archive.ubuntu.com trusty/main Sources [1335 kB]
Get:20 http://archive.ubuntu.com trusty/restricted Sources [5335 B]
Get:21 http://archive.ubuntu.com trusty/universe Sources [7926 kB]
Get:22 http://archive.ubuntu.com trusty/main amd64 Packages [1743 kB]
Get:23 http://archive.ubuntu.com trusty/restricted amd64 Packages [16.0 kB]
Get:24 http://archive.ubuntu.com trusty/universe amd64 Packages [7589 kB]
Fetched 23.6 MB in 6s (3823 kB/s)
Reading package lists...

[Container] 2019/05/02 00:29:17 Running command apt-get install -y shellcheck
Reading package lists...
Building dependency tree...
Reading state information...
E: Unable to locate package shellcheck

First pass at this work in my fork codebuild-fuzz, this is almost ready for PR.
(this CodeBuild links are not public)

fuzzer1 arguments here
fuzzer2 arguments here
fuzzer3 arguments here

#1534 moved Sidetrail into Codebuild. Leaving open until we know exactly what tests are required and where they should be done.


Adds validation to mdx plugin

MDX Validation
I found a snippet of code  in the jsx-runtime package that validates the MDX so I stole it.
The reason for adding acorn to the package.json file is because there is a conflicting version with a different api elsewhere in the project that gets picked up unless explicitly set here.
The current implementation silently checks before any changes are written to the disk but if the user tries to save the file then an alert 🚨pops up with a message and prevents the save.
Can you help?
I would like some guidance on how to better inform the user that there is a problem with the formatting both during the edit and also upon save
Thanks for taking the time to look at my PR
-Paul

Cool! Looks good so far. You might want to remove the debugger lol
We might be able to put that check on the field as all. That will let it show an error message.

I would like some guidance on how to better inform the user that there is a problem with the formatting both during the edit and also upon save

Okay, I looked at this more closely and have an opinion.
Consistent Error Messaging
I think for consistency, it would make sense to follow the same pattern that tina is already using to inform users of errors (i.e. throw new Error). I know it's not the most graceful but at least it's consistent with the rest of the error messaging.
Suggested Changes

[ ] move validateMDX into useMdxForm.ts (this way the validation function is collocated with the code & it can live next to checkMdxExists

[ ] nit: change validateMDX to validateMdx (keeps casing consistent)

[ ] move error message in validateMdx function to errors.ts

[ ] update tone/language of error message to align more with errors.s (I love emojis personally but consider removing and then changing the language a bit to match the rest of tina's tone)

ICYDK, to fix this build error:

Error: /home/runner/work/tinacms/tinacms/packages/gatsby-tinacms-mdx/src/validateMdx.ts(19,27): semantic error TS7016: Could not find a declaration file for module 'buble-jsx-only'. '/home/runner/work/tinacms/tinacms/node_modules/buble-jsx-only/dist/buble.cjs.js' implicitly has an 'any' type.
Try npm install @types/buble-jsx-only if it exists or add a new declaration (.d.ts) file containing `declare module 'buble-jsx-only';

We could probably get by with adding an index.d.ts with the line:
declare module 'buble-jsx-only'

Hmm...might need more input on this. So I was able to get it to build locally and fix the ts error by adding the buble-jsx-only.d.ts file to the mdx package, but we added *.d.ts to our .gitignore...
Do we:

remove that line from our .gitignore
add a line to the .gitignore to not ignore the buble-jsx-only.d.ts
something else?

Thanks @jsjoeio I've actioned your points :)
still to do:
[ ] figure out how to update the form/field state  to display that there is an error

* [ ]  figure out how to update the form/field state  to display that there is an error
  _I've spent hours trying to figure this out but it's beyond me unfortunately_

I'm wondering though...do we need to display it in the form/field? Shouldn't throwing it cause the app to crash and show it in the browser, similar to how the other errors do it? (again, not the greatest UX but path of least resistance to getting this merged?)

I'm not sure we want to crash the app, if someone is editing some MDX and they start typing < that won't build so you don't want to save. However you want to let them continue editing <div /> then allow them to save once they are done editing/the mdx is valid

I think the MDX plugin should export a “mdx” field plugin. It would essentially be the same as the Markdown one, but we could add some things.
The things I have in mind are the validate function, and maybe a parse function.
Ideally the mdx field wouldn’t call onChane with invalide MDX. I think the parse function could be used to accomplish that goal. I believe it receives the current and next value, so we could say “if not valid return previous value”

I agree, I've been wanting to get stuck into looking at the editor.
I think it's worth merging the check here in the interim though. I'm not sure if your comment suggests we should close this pr? There's nothing stopping someone using a plain text field so the extra layer of security could help perhaps.

[] I'm going to do another sanity check on this branch today.

Not sure why the build is failing 🤷🏼♂

Ok, this isn't good enough on it's own, you can trigger a submit that just hangs because the check happens after the form submits rather than preventing submission. After your comment @ncphillips I think I have an Idea for how to add draft of the MDX editor. Watch this space...

Closing for now as I don't have time to work on this at the moment

All good, thanks for all the work Paul! Much appreciated


clickToSelect: true + cellEditTableData mode: dbclick not working

Hello,
First: I have configured my table with clickToSelect: true, In the function clickToSelect I give the row another color:
const selectRowTableData = { 		mode: 'checkbox', //radio or checkbox 		clickToSelect: true, bgColor: function(row, isSelect) { 		  if (isSelect) { 			return 'red'; 		  } 		  return null; 		}		 	};
OK, working great.
Now I add celledit with DOUBLE CLICK:
	const cellEditTableData = { 		mode: 'dbclick' // click cell to edit 	};
What happens now:
If I double click on my table it is working, I can edit the table. If I click only once on the table row it should mark the row with the color red, but this isn't working.
I checked the behaviour with the debugger and found out, that the function selectRow (in my case selectRowTableData) will entered, but in the function bgColor nothing is selected. The value is every time false.
I look in the issue tracker and found the same question already in 2015:
https://github.com/AllenFang/react-bootstrap-table/issues/21
There the last entry of the developer was
"Hi stephenrs, so you just only want the user click row for selection and dbclick for editing? If is, I misunderstand before and I'll fix it."
So in my opinion this should work. Do I something wrong?
My full table:
			  <BootstrapTable height={'600px'}  					data={ this.state.tableData }  					options={optionsTableData}  					cellEdit={ cellEditTableData } 					selectRow={ selectRowTableData }  					trClassName={ this.trClassFormat }					 					insertRow  					deleteRow 				 					search={ true }  					multiColumnSearch={ true }  					hover={true}  					condensed  					> 				<TableHeaderColumn dataField='startnummer' isKey>Startnummer</TableHeaderColumn> 				<TableHeaderColumn dataField='vid'>VID</TableHeaderColumn> 				<TableHeaderColumn dataField='titel'>Titel</TableHeaderColumn> 				<TableHeaderColumn dataField='disziplin' searchable={ false }>Disziplin</TableHeaderColumn> 				<TableHeaderColumn dataField='altersklasse' searchable={ false }>Altersklasse</TableHeaderColumn> 				<TableHeaderColumn dataField='kategorie' searchable={ false }>Kategorie</TableHeaderColumn> 				<TableHeaderColumn dataField='wertungsklasse' searchable={ false }>Wertungsklasse</TableHeaderColumn> 				<TableHeaderColumn dataField='dauer' searchable={ false }>Dauer</TableHeaderColumn> 				<TableHeaderColumn dataField='tanzId' searchable={ false }>TanzID</TableHeaderColumn> 				<TableHeaderColumn dataField='anzahl' searchable={ false }>Anzahl</TableHeaderColumn> 				<TableHeaderColumn dataField='land' searchable={ false }>Land</TableHeaderColumn> 				<TableHeaderColumn dataField='schulname' searchable={ false }>Schulname</TableHeaderColumn>
For better reading: JSFiddle

@Paul6552 sorry for lately reply, please try following code, this is not a bug
const cellEdit = {
  mode: 'dbclick'
};
const selectRow = {
  mode: 'checkbox',
  clickToSelectAndEditCell: true
};

@AllenFang
No, it is not working. With double click I can edit the cell. With one click the row is never selected:
	const selectRowTableData = { 		mode: 'checkbox', 		clickToSelectAndEditCell: true, 		bgColor: function(row, isSelect) { 		  if (isSelect) { 			return 'red'; 		  } 		  return null; 		}		 	};
iSelect = never true, every time False!

It's work for me
please describe your problem clear, the problem is the isSelect argument is always false for bgColor function.... NOT ROW SELECTION BROKEN

Following is my example
const selectRowProp = {
  mode: 'checkbox',
  clickToSelectAndEditCell: true,
  bgColor: function(row, isSelect) {
    if (isSelect) { return 'red'; }
    return null;
  }
};

const cellEdit = {
  mode: 'dbclick'
};

export default class ClickToSelectTable extends React.Component {
  render() {
    return (
      <BootstrapTable data={ products } cellEdit={ cellEdit } selectRow={ selectRowProp }>
          <TableHeaderColumn dataField='id' isKey={ true }>Product ID</TableHeaderColumn>
          <TableHeaderColumn dataField='name'>Product Name</TableHeaderColumn>
          <TableHeaderColumn dataField='price'>Product Price</TableHeaderColumn>
      </BootstrapTable>
    );
  }
}

@AllenFang  Thank you for your reply.
I found my error with your provided example. Sadly, I have no idea how to fix this. :-/
isSelect and double click is working if I don't use setState in the function onRowClick.
In my JSFiddle example below, you will find an expanded example of your example above.
Bootstraptable with cellEdit: dbclick, selectRowProp AND NEW options.
In options I set the function onRowClick: this.onClickDoSomething.
isSelect is working (selected Row is true) as long as I don't use setState in the onClickDoSomething function.
So why is isSelect every time false if I use setState in the function onClickDoSomething?
Please open my JSFiddle for better understanding.
Thank you very much
https://jsfiddle.net/qbh9o7px/1/

double click is working if I don't use setState in the function onRowClick.

I think you should point it at begin, this discussion is wasting my time lol
Actually, onRowClick must faster than row selection, os if you call setState in  onRowClick , the table will get rerender so the actually, the selection event is no trigger... I do suggest you not to change state in onRowClick or the other hook events.

Thank you for your time.
Finished


Attachments not rendered, lost upon saving draft

Version 5.007 on Android 5.0.2:
Some types of message content cause existing message attachments to not be rendered while editing messages. If edits are saved, attachments are permanently lost.
From another client

Open Thunderbird* and launch Help > Troubleshooting Information, then choose 'Copy text to clipboard'.
Paste into draft message.
Add file attachment.
Save in drafts.

*It's not clear exactly what HTML content causes this error. Basic entities like bold, tables, & text coloring don't trigger it but the Thunderbird troubleshooting dump does so consistently.
From K9 mail

Locate saved message in Drafts folder view
Observe paperclip icon on draft, indicating an attachment
Open message to resume editing
Observe message apparently has no attachments
Save as draft
Observe paperclip icon is now missing, indicating attachment was lost outright

It's probably worth noting that all HTML is stripped out of the message body when you resume editing with K9 mail.

K-9 Mail can't edit HTML, so there's no point in keeping the original HTML from Thunderbird.
The missing attachment sounds like a bug.
Currently K-9 Mail's draft handling code kind of assumes that you're only editing drafts generated by K-9 Mail. So my advice would be to avoid editing drafts created by other clients for now.

When i create a draft with an attachment and saves it, attachment gets saved. But when i open the draft again and add/update text to it and saves it again, the attachment was lost. So seems like an issue for drafts created by k-9 client as well.

When i create a draft with an attachment and saves it, attachment gets saved. But when i open the draft again and add/update text to it and saves it again, the attachment was lost. So seems like an issue for drafts created by k-9 client as well.

@cketti This seems to a legit issue, i provided the reproducible steps in my previous comment. Can you confirm the behavior?

@swapsmagic I don't have time to look into this issue right now. If you're able to reproduce it please consider submitting a pull request fixing the bug.

This seems to be identical to #2164 and it affects me too. I have version 5.203

Is this a duplicate of #1566 and #2547?
I would also say it is related to #2485, which is a duplicate of #3110.

I wanted to add that for me it is not related to HTML. It happens to plain text mails as well. Here is what I see/have:
Steps to reproduce

Compose a new mail and attache a file in a different IMAP client than K9 (round cube in my case)
Save as draft
Open draft folder with K9 -> Draft mail is displayed with attachment icon
Open the mail with K9 -> attachment is removed
After saving, the attachment icon is gone in the draft folder view
Opening the mail again with another client shows the draft with removed attachment

Environment
K-9 Mail version: 5.600
Android version: 8.1.0
Account type (IMAP, POP3, WebDAV/Exchange): IMAP
It confused my wife and me in several occasions until I decided now to look for issues about this.

@efelon I can't reproduce the bug in version 5.803 with your steps. Can you please check if the bug still exists?

@plan3d I still can. Let me show some screenshots:

Existing draft with attachement:

Opened draft within K9 already without any attachement

...Then make some changes in text...

"Back" on Android asks for saving or discard

After "Saving..." the attachment is gone

Discarding leaves the attachment in place

Version from F-Droid

I have the same issue.
I write a message on roundcube, attach a file, and save it as draft.
In K9Mail E-Mail List view, it shows a mail with an paper-clip. When I open it, no attachment was visible.

I have the same issue for a long time now.
I write a draft on Thunderbird (currently 115.2.0) and attach a file. In K-9 Mail (currently 6.602) E-Mail List view, it shows a mail with an paper-clip. When I open it, no attachment was visible.

Hello,
same issue for me.
opening a draft lose the attachment.
writing a email with an attachment, saving the draft and opening with the email box website (mailbox.org), the attachment are there. If re-opening fro; K-9 android app., only text is there in the draft but no more attachment.
K-9 Mail app : 6.603
Android : 13
Hope it will be fixed.

@cketti @wmontwe Is there any chance to bring some attention to this issue, which is more than eight years old now?

I was about to file a bug on this, but I found someone already filed it, and the issue is still a problem.
An attachment that I had added to a draft message was quietly dropped when I opened the draft message to send it.  Also, my messages are plain text and not HTML, so you can eliminate the HTML variable.

App version: com.fsck.k9 6.801 (from Google Play Store)
Android version: 14 (AP1A.240305.019.A1)
Device model: Google Pixel 6a

Same problem with Version 6.802.
Just went to a priting offfice, sent them an email with PDFs to print, which I prepared with Thunderbird on my computer, and saved as a draft. Attachments lost. Big loss of time. Very annoying.


🐛 Enable CR and CRB for proxy to access kube apiserver (issue 2355)

Summary
Update the rbac kustomization yaml to create the CR and CRB in the KS chart template yaml to allow the proxy to access the kube apiserver
Related issue(s)
Fixes #2355

How about a checked-in test, or a demonstration here, that this works?

How about a checked-in test, or a demonstration here, that this works? (I am curious about why the other two lines do not need to be un-commented.)

Braulio has not provided a pass/fail test but he provided a snapshot of the changes that he needs to pass his scenario here.
My change seems to match what he needs.
Here is a comparison of the chart template yaml before/after the change:

Here is a more detailed report from yesterday:
I gave it a little bit of a functional test. I used the current version of this PR, commit bda95d7ab3bc55e2ece7e38fa5823e6fc23c7d46 . I ran test/e2e/run-test.sh to create a working system. Then I did the following.
mspreitz@mjs13 kubestellar % kubectl config use-context kind-kubeflex
Switched to context "kind-kubeflex".

mspreitz@mjs13 kubestellar % kubectl get -n wds1-system deploy
NAME                             READY   UP-TO-DATE   AVAILABLE   AGE
kube-apiserver                   1/1     1            1           6m42s
kube-controller-manager          1/1     1            1           6m42s
kubestellar-controller-manager   1/1     1            1           4m37s
transport-controller             1/1     1            1           4m10s

mspreitz@mjs13 kubestellar % kubectl port-forward -n wds1-system deploy/kubestellar-controller-manager 8443
Forwarding from <IP_ADDRESS>:8443 -> 8443
Forwarding from [::1]:8443 -> 8443

Then in another shell I did the following and examined the file that was produced; it was good.
mspreitz@mjs13 ~ % kubectl get --raw https://localhost:8443/metrics > /tmp/metrics.txt
mspreitz@mjs13 ~ % open /tmp/metrics.txt

@MikeSpreitzer @francostellari . Here are the steps to reproduce the issue:
Objective: create a simple pod and try to reach the kubestellar-controller-manager /metrics via its kube-rbac-proxy

Create a nginx deployment:

$ kubectl create deployment nginx --image=nginx

Create the below RBAC for the nginx pod to reach a /metrics endpoint:

$ kubectl create -f - <<EOF
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
  labels:
    app: nginx
  name: nginx
rules:
- apiGroups:
  - ""
  resources:
  - services
  - endpoints
  - pods
  verbs:
  - get
  - list
  - watch
- nonResourceURLs:
  - /metrics
  verbs:
  - get

---
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
  labels:
    app: nginx
  name: nginx
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: nginx
subjects:
- kind: ServiceAccount
  name: default
  namespace: default
EOF

Exec into the nginx pod:

$ kubectl get pods
NAME                     READY   STATUS    RESTARTS   AGE
nginx-748c667d99-9fk9t   1/1     Running   0          36m

$ kubectl exec --stdin --tty  nginx-748c667d99-9fk9t -- /bin/bash

Try to reach the kubestellar controller manager /metrics endpoint from inside the pod as following:

$ curl -k --cacert /var/run/secrets/kubernetes.io/serviceaccount/ca.crt -H "Authorization: Bearer $(cat /var/run/secrets/kubernetes.io/serviceaccount/token)" https://ks-controller-manager.wds1-system.svc:8443/metrics

Unauthorized

WARNING: see authorization failure message above

Create the RBAC for the kube-rbac proxy:

$ kubectl apply -f - <<EOF
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
  name: kube-rbac-proxy
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: kube-rbac-proxy
subjects:
- kind: ServiceAccount
  name: kubestellar-controller-manager
  namespace: wds1-system

---
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
  name: kube-rbac-proxy
rules:
- apiGroups: ["authentication.k8s.io"]
  resources:
  - tokenreviews
  verbs: ["create"]
- apiGroups: ["authorization.k8s.io"]
  resources:
  - subjectaccessreviews
  verbs: ["create"]
EOF

From inside the ngnix pod try to reach the kubestellar controller manager metrics endpoint AGAIN:

$ curl -k --cacert /var/run/secrets/kubernetes.io/serviceaccount/ca.crt -H "Authorization: Bearer $(cat /var/run/secrets/kubernetes.io/serviceaccount/token)" https://ks-controller-manager.wds1-system.svc:8443/metrics
# HELP certwatcher_read_certificate_errors_total Total number of certificate read errors
# TYPE certwatcher_read_certificate_errors_total counter
certwatcher_read_certificate_errors_total 0
# HELP certwatcher_read_certificate_total Total number of certificate reads
# TYPE certwatcher_read_certificate_total counter
certwatcher_read_certificate_total 0
# HELP go_gc_duration_seconds A summary of the pause duration of garbage collection cycles.
# TYPE go_gc_duration_seconds summary
go_gc_duration_seconds{quantile="0"} 4.0051e-05
go_gc_duration_seconds{quantile="0.25"} 6.2192e-05
go_gc_duration_seconds{quantile="0.5"} 7.5662e-05
go_gc_duration_seconds{quantile="0.75"} 8.2202e-05
go_gc_duration_seconds{quantile="1"} 0.000243596
go_gc_duration_seconds_sum 3.941643585
go_gc_duration_seconds_count 49909
# HELP go_goroutines Number of goroutines that currently exist.
# TYPE go_goroutines gauge
go_goroutines 433
# HELP go_info Information about the Go environment.
# TYPE go_info gauge
go_info{version="go1.22.2"} 1
# HELP go_memstats_alloc_bytes Number of bytes allocated and still in use.
# TYPE go_memstats_alloc_bytes gauge
go_memstats_alloc_bytes 1.7101256e+07
# HELP go_memstats_alloc_bytes_total Total number of bytes allocated, even if freed.
# TYPE go_memstats_alloc_bytes_total counter
go_memstats_alloc_bytes_total 4.8717670024e+11
# HELP go_memstats_buck_hash_sys_bytes Number of bytes used by the profiling bucket hash table.
# TYPE go_memstats_buck_hash_sys_bytes gauge
go_memstats_buck_hash_sys_bytes 2.309678e+06
# HELP go_memstats_frees_total Total number of frees.
# TYPE go_memstats_frees_total counter
go_memstats_frees_total 1.404821891e+09
# HELP go_memstats_gc_sys_bytes Number of bytes used for garbage collection system metadata.
# TYPE go_memstats_gc_sys_bytes gauge
go_memstats_gc_sys_bytes 3.84372e+06
# HELP go_memstats_heap_alloc_bytes Number of heap bytes allocated and still in use.
# TYPE go_memstats_heap_alloc_bytes gauge
go_memstats_heap_alloc_bytes 1.7101256e+07

WARNING: a success now : )

The above example is assuming that you have a service for the kubestellar controller manager. You can create it as following:
kubectl -n wds1-system apply -f - <<EOF
kind: Service
apiVersion: v1
metadata:
  name: ks-controller-manager
  labels:
    app: ks-controller-manager
spec:
  selector:
    control-plane: controller-manager
  ports:
  - name: metrics
    port: 8443
EOF

@dumb0002 (cc @MikeSpreitzer)
The PR seems to work and solve your problem

I did a run_test
I used your command to create the nginx pod
I used your command to create the RBAC for the nginx pod to reach a /metrics endpoint
I did NOT create the extra RBAC for the kube-rbac proxy:
after logging in

$ k get pods
NAME                     READY   STATUS    RESTARTS   AGE
nginx-7854ff8877-ndjbr   1/1     Running   0          25s

$ kubectl exec --stdin --tty  nginx-7854ff8877-ndjbr  -- /bin/bash

root@nginx-7854ff8877-ndjbr:/# $ curl -k --cacert /var/run/secrets/kubernetes.io/serviceaccount/ca.crt -H "Authorization: Bearer $(cat /var/run/secrets/kubernetes.io/serviceaccount/token)"
https://ks-controller-manager.wds1-system.svc:8443/metrics
bash: $: command not found
root@nginx-7854ff8877-ndjbr:/#
root@nginx-7854ff8877-ndjbr:/# curl -k --cacert /var/run/secrets/kubernetes.io/serviceaccount/ca.crt -H "Authorization: Bearer $(cat /var/run/secrets/kubernetes.io/serviceaccount/token)" https://ks-controller-manager.wds1-system.svc:8443/metrics
# HELP certwatcher_read_certificate_errors_total Total number of certificate read errors
# TYPE certwatcher_read_certificate_errors_total counter
certwatcher_read_certificate_errors_total 0
# HELP certwatcher_read_certificate_total Total number of certificate reads
# TYPE certwatcher_read_certificate_total counter
certwatcher_read_certificate_total 0
# HELP go_gc_duration_seconds A summary of the pause duration of garbage collection cycles.
# TYPE go_gc_duration_seconds summary
go_gc_duration_seconds{quantile="0"} 7.3699e-05
go_gc_duration_seconds{quantile="0.25"} 9.3463e-05
go_gc_duration_seconds{quantile="0.5"} 0.000221975
go_gc_duration_seconds{quantile="0.75"} 0.00063173
go_gc_duration_seconds{quantile="1"} 0.013746327
go_gc_duration_seconds_sum 0.027149926
go_gc_duration_seconds_count 13
# HELP go_goroutines Number of goroutines that currently exist.
# TYPE go_goroutines gauge
go_goroutines 428
# HELP go_info Information about the Go environment.
# TYPE go_info gauge
go_info{version="go1.21.6"} 1
# HELP go_memstats_alloc_bytes Number of bytes allocated and still in use.
# TYPE go_memstats_alloc_bytes gauge
go_memstats_alloc_bytes 7.801344e+06
# HELP go_memstats_alloc_bytes_total Total number of bytes allocated, even if freed.
# TYPE go_memstats_alloc_bytes_total counter
go_memstats_alloc_bytes_total 3.839608e+07
# HELP go_memstats_buck_hash_sys_bytes Number of bytes used by the profiling bucket hash table.
# TYPE go_memstats_buck_hash_sys_bytes gauge
go_memstats_buck_hash_sys_bytes 1.465903e+06
# HELP go_memstats_frees_total Total number of frees.
# TYPE go_memstats_frees_total counter
go_memstats_frees_total 358686
# HELP go_memstats_gc_sys_bytes Number of bytes used for garbage collection system metadata.
# TYPE go_memstats_gc_sys_bytes gauge
go_memstats_gc_sys_bytes 4.656056e+06
# HELP go_memstats_heap_alloc_bytes Number of heap bytes allocated and still in use.
# TYPE go_memstats_heap_alloc_bytes gauge
go_memstats_heap_alloc_bytes 7.801344e+06
# HELP go_memstats_heap_idle_bytes Number of heap bytes waiting to be used.
# TYPE go_memstats_heap_idle_bytes gauge
go_memstats_heap_idle_bytes 2.793472e+06
# HELP go_memstats_heap_inuse_bytes Number of heap bytes that are in use.
# TYPE go_memstats_heap_inuse_bytes gauge
go_memstats_heap_inuse_bytes 1.1395072e+07
# HELP go_memstats_heap_objects Number of allocated objects.
# TYPE go_memstats_heap_objects gauge
go_memstats_heap_objects 64146
# HELP go_memstats_heap_released_bytes Number of heap bytes released to OS.
# TYPE go_memstats_heap_released_bytes gauge
go_memstats_heap_released_bytes 1.114112e+06
# HELP go_memstats_heap_sys_bytes Number of heap bytes obtained from system.
# TYPE go_memstats_heap_sys_bytes gauge
go_memstats_heap_sys_bytes 1.4188544e+07
# HELP go_memstats_last_gc_time_seconds Number of seconds since 1970 of last garbage collection.
# TYPE go_memstats_last_gc_time_seconds gauge
go_memstats_last_gc_time_seconds 1.7229665566040742e+09
# HELP go_memstats_lookups_total Total number of pointer lookups.
# TYPE go_memstats_lookups_total counter
go_memstats_lookups_total 0
# HELP go_memstats_mallocs_total Total number of mallocs.
# TYPE go_memstats_mallocs_total counter
go_memstats_mallocs_total 422832
# HELP go_memstats_mcache_inuse_bytes Number of bytes in use by mcache structures.
# TYPE go_memstats_mcache_inuse_bytes gauge
go_memstats_mcache_inuse_bytes 4800
# HELP go_memstats_mcache_sys_bytes Number of bytes used for mcache structures obtained from system.
# TYPE go_memstats_mcache_sys_bytes gauge
go_memstats_mcache_sys_bytes 15600
# HELP go_memstats_mspan_inuse_bytes Number of bytes in use by mspan structures.
# TYPE go_memstats_mspan_inuse_bytes gauge
go_memstats_mspan_inuse_bytes 256200
# HELP go_memstats_mspan_sys_bytes Number of bytes used for mspan structures obtained from system.
# TYPE go_memstats_mspan_sys_bytes gauge
go_memstats_mspan_sys_bytes 277032
# HELP go_memstats_next_gc_bytes Number of heap bytes when next garbage collection will take place.
# TYPE go_memstats_next_gc_bytes gauge
go_memstats_next_gc_bytes 1.3080688e+07
# HELP go_memstats_other_sys_bytes Number of bytes used for other system allocations.
# TYPE go_memstats_other_sys_bytes gauge
go_memstats_other_sys_bytes 766985
....

Thanks @francostellari
/lgtm

Testing as in https://github.com/kubestellar/kubestellar/issues/2355#issuecomment-2272081946 and above, I also see that the change here makes the difference between success and failure for a client to scrape the metrics of the kubestellar-controller-manager.

/lgtm


[Excel] (custom functions) Minor tweaks

Docs Build status updates of commit f5f0388:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-json-autogeneration.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Twitter: consider splitting larger tweets

Won't improve Twitter support. I might, eventually, remove it.


added pdf merging functionality - please double check if this broke a…

…ny of the working plots

Emma, in all the plots, except for the heatmap, I get the error: "cannot open file '../output/.pdf'". The heatmap has its own field to save it (this was retained from the other app, we could remove it I guess).

@facuxpalacio I think it should be working now - let me know and if so i'll move onto your other task

Now this works great!


Build custom map environment using road runner and Unreal engine

Hello,
I followed the guide mentioned here here
to create my custom environment map. I did not get any error, however, after importing the .fbx file into unreal engine the objects in the environment have no textures.
Why are not the material textures/colors visible in unreal?
Road Runner map:

Unreal engine map:

Hi @ndeshp2s, in case you're still facing issues, please take a look at the thread in #1911. I was facing a similar problem and it has been resolved now.


[FEATURE] Change types to flat_object

Is your feature request related to a problem?
There are a few fields that should have the type flat_object.
What solution would you like?
Change the user_query field to be of the type flat_object.

[Catch All Triage - Attendees 1, 2, 3, 4, 5]


Abdurrehman/video player styling tweaks

Pre-Flight checklist

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

[x] Desktop screenshots
[x] Mobile width screenshots

[x] Testing

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

What are the relevant tickets?
Closes https://github.com/mitodl/ocw-hugo-themes/issues/802
What's this PR do?

Updates the style of video progress bar so it covers the complete width of video player.
Updates toggleable tabs under the video to make them uniform/same. (Related resources and optional tab's UI made same as of transcription tab UI)

How should this be manually tested?

Checkout this branch
Build any course which has videos, transcript, related resources and optional tab.
Verify that video progress bar covers the complete width of video player.
Verify that all the 3 toggleable tabs are same in terms of UI and functionality and they work as expected.
Repeat the above steps on different browsers and different screen sizes.

Screenshots (if appropriate)

@pdpinch I'm thinking of giving some margin from left and bottom to the content of Related Resources and Optional (Lecture Notes) tabs too, similar to what we have in the above transcript tab.

Do you have any thoughts on this? or on the overall design?

@pdpinch I'm thinking of giving some margin from left and bottom to the content of Related Resources and Optional (Lecture Notes) tabs too, similar to what we have in the above transcript tab.

I don't love it, but I we do need some separation, so folks know they can click and also so they have enough space between tappable areas (a mobile requirement).
I think you should indent of contents of the hide/show the same amount.

@gumaerc Thanks for having a look into it.
I have updated the PR with this feedback along with some other tweaks.
Regarding your point of scrollbar: I agree, it doesn't look really good but I think its the default one.
Since I'm using mac so I have this type of default scrollbar:

However, if you think that we should customize this default scrollbar then I'd suggest doing it in other places too (i.e: page scrollbar or any other place where we have a scrolling section or page) for better consistency. And I think this can be done by a separate PR.
What do you think?
Perhaps @pdpinch could also share his thoughts?

my feedback on that was more that it isn't flush with the right side of the container

Ahh right. Thanks for pointing it out. I also did not notice it.
Anyhow, I've updated the PR to push it to the right-most side.
I think we can merge this now!


false offense detected by Style/MixinUsage

Expected behavior
thing = Class.new do
  attr_accessor :value_text, :value_type
  include IsNameValueType
end

RSpec.describe IsNameValueType do
  subject { thing.new }
  # tests
end

expect Mixin usage to be correct
Actual behavior
Inspecting 4 files
.C..

Offenses:

spec/models/concerns/is_name_value_type_spec.rb:3:3: C: Style/MixinUsage: include is used at the top level. Use inside class or module.
  include IsNameValueType
  ^^^^^^^^^^^^^^^^^^^^^^^

4 files inspected, 1 offense detected

Steps to reproduce the problem
create the test as in the expected behaviour and run rubocop <filename>
RuboCop version
Include the output of rubocop -V. Here's an example:
➔ rubocop -V
0.51.0 (using Parser <IP_ADDRESS>, running on ruby 2.4.1 x86_64-darwin16)

It looks like this cop misreports for any case where the class is the first in some set of siblings.
For example:
class A
  include Mixin
end

class B
  include Mixin
end

Result:
❯ rubocop t.rb
Inspecting 1 file
C

Offenses:

t.rb:2:3: C: include is used at the top level. Use inside class or module.
  include Mixin
  ^^^^^^^^^^^^^

1 file inspected, 1 offense detected

Of note is that only the first class has this error reported.
❯ rubocop -V
0.51.0 (using Parser <IP_ADDRESS>, running on ruby 2.3.4 x86_64-darwin16)

I'm also getting false positives for this:
RSpec::Matchers.define :do_something do
  include Mixin
end

Again, strangely, only for the first occurrence in a file. I define three matchers in one file, all with the same mixin, and only get flagged for the first one.
.....
It is a bit unclear from the mixin code itself and the PR what the goal of the cop is. Is it really to prevent mixing into Object? Is it to prevent mixing into main? Is it to prevent mixing in outside of a class or module? Let's state very explicitly what the goal is, and then implement against it more narrowly!

I have the same problem, and another info.
class A
  include module_A
  include namespace_1::module_B
  include namespace_1::module_C
  include module_D
end

class B
   include module_E
end

Rubocop will raise Style/MixinUsage only for module_A and module_D, not the namespaced ones.

Indeed. It's a bit unclear what the implementation is trying to do, but it's clear that it's incorrect. 🙂

Is there any fix on this? It's still popping up for me.

@ZASMan
Can you try updating rubocop to 0.52.0? It has been fixed in 0.52.0 already.
If you still have any bugs, please open a new issue.


Allow padding to the crop area

As per #489, having the cropping area "padded" is a bug that can happen with the library in some scenarios. However, the ability to chose to add a Padding (specifically with restrictPosition=false) would be useful, another customization option.

This can probably be done outside of the library by adding padding to the parent container and moving the "overflow: hidden" one level up.

This can probably be done outside of the library by adding padding to the parent container and moving the "overflow: hidden" one level up.

Can you share a working example of this? I've been playing with CSS and the HTML structure around the cropper and haven't managed to actually get said padding.
I will note that, if I put an image that is SMALLER than the Cropper component, there will be padding by default, as the Crop Area will fit the dimensions of the image and not the cropper. This is rather inconsistent, and I'd say undesired behaviour.

I'm not sure if this matches your expectations but here's an example I just created: https://codesandbox.io/p/sandbox/wonderful-elbakyan-pgcfzc?file=%2Fsrc%2Fstyles.css
See the comments in the css file.

Had to do some slight adjustments to my specific use case, but it did reach the results I was hoping for! I'd still advocate for including it directly in the library as a prop, as I imagine this should be a "enough" commonly needed feature.
For future reference, here's my code that achieved the desired results, wrapped in Mui elements:
<Box
  sx={{
    padding: '100px',
    overflow: 'hidden',
    display: 'flex',
    alignItems: 'stretch',
    justifyContent: 'stretch',
    width: '100%',
    height: '100%',
  }}
>
  <Box sx={{ position: 'relative', width: '100%', height: '100%' }}>
    <Cropper { /* CROPPER PROPS */ }  />
  </Box>
</Box>

I also had to add the css override for the library:
.reactEasyCrop_Container {
  overflow: visible !important;
}

Just adding a prop adds a maintenance cost and isn't this simple as we would need to add an extra div.
I prefer not to add it and have this issue serve as an example.


Remove many Get prefixes in interfaces

These were there to reduce the diff when introducing the interfaces but should be removed.
Will followup for RuleMetadata which will be less mechanical

Codecov Report
Base: 73.60% // Head: 73.69% // Increases project coverage by +0.09% :tada:

Coverage data is based on head (5f4f3fb) compared to base (5b0dfa2).
Patch coverage: 65.93% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           v3/dev     #475      +/-   ##
==========================================
+ Coverage   73.60%   73.69%   +0.09%
==========================================
  Files         146      146
  Lines        6618     6618
==========================================
+ Hits         4871     4877       +6
+ Misses       1472     1466       -6
  Partials      275      275

Flag
Coverage Δ

no-tinygo
73.63% <65.93%> (+0.09%)
:arrow_up:

overall
73.69% <65.93%> (+0.09%)
:arrow_up:

tinygo
72.39% <64.10%> (+0.06%)
:arrow_up:

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

Impacted Files
Coverage Δ

http/http.go
83.78% <0.00%> (ø)

testing/engine.go
58.73% <11.11%> (ø)

internal/corazawaf/transaction.go
66.04% <39.16%> (+0.74%)
:arrow_up:

actions/setenv.go
60.86% <50.00%> (ø)

actions/setvar.go
54.21% <66.66%> (ø)

internal/corazawaf/rule.go
93.80% <93.33%> (ø)

actions/ctl.go
69.00% <100.00%> (ø)

actions/skipafter.go
100.00% <100.00%> (ø)

internal/corazawaf/rulegroup.go
93.90% <100.00%> (ø)

internal/corazawaf/waf.go
96.49% <100.00%> (ø)

... and 2 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Support MinIO In Chart

Sorry-Cypress is very popular in air-gapped networks that doesn't have access to AWS S3.
We need to store our media in different storage option.
The second option that sorry-cypress support is MinIO.
We would be happy to support this in the HELM Chart 😄

Done 🥇


Misc doc changes

Besides other documentation changes, this commit ensures the generated
HTML doc for HexDocs.pm will become the source of truth for this Elixir
library and leverage on latest features of ExDoc.

@edgurgel Any updates on this?

The readme changes can be viewed at:
https://github.com/kianmeng/httparrot/tree/misc-doc-changes?tab=readme-ov-file
The Travis CI build status is broken both in this PR and on main, but that could be resolved in a separate pull request.

It took some time but I'm back :laughing: Sorry for the wait, team

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:


add FixedRateExchange

closes #52
Sample query:
http://<IP_ADDRESS>:8000/subgraphs/name/oceanprotocol/ocean-subgraph/graphql?query={
  fixedRateExchanges(orderBy%3A id){
  id
    exchangeOwner {
      id
    }
    rate
    datatoken {
      id
    }
    baseToken
    active
    updates {
      id
      oldRate
      newRate
      timestamp
      oldActive
      newActive
    }
    swaps {
      id
      by {
        id
      }
      baseTokenAmount
      dataTokenAmount
      timestamp
    }
}
}
Sample result:

tested with @ssallam optimizations, looks good


bindings.properties is not included in release archive?

I was playing with testing the CentOS compatibility and noticed that if I download the release archive then bindings.properties is not in the release so ./bin/ycsb.sh doesn't work.
I think #784 and #785 partially addressed this for the ycsb scripts themselves.
For reference here is what I was running:
docker run --rm -it centos:6 bash
yum -y install git java-1.8.0-openjdk-devel wget epel-release && wget http://repos.fedorapeople.org/repos/dchen/apache-maven/epel-apache-maven.repo -O /etc/yum.repos.d/epel-apache-maven.repo && yum -y install apache-maven
RELEASE=0.11.0; rm -rf /opt/ycsb-$RELEASE /opt/YCSB; wget -qO- https://github.com/brianfrankcooper/YCSB/releases/download/$RELEASE/ycsb-$RELEASE.tar.gz | tar xvfz - -C /opt && mv /opt/ycsb-$RELEASE /opt/YCSB; unset RELEASE
cd /opt/YCSB
./bin/ycsb.sh load basic -P workloads/workloada

The output from the ycsb.sh script is as follows:
# ./bin/ycsb.sh load basic -P workloads/workloada
grep: /opt/YCSB/bin/bindings.properties: No such file or directory
[ERROR] The specified binding 'basic' was not found.  Exiting.

I was able to "fix" the release by doing the following:
wget -O /opt/YCSB/bin/bindings.properties https://raw.githubusercontent.com/brianfrankcooper/YCSB/master/bin/bindings.properties

cc @busbey @kruthar @cmatser


Codec cap helper functions

Part of fixing https://github.com/zephyrproject-rtos/zephyr/issues/62292

@Thalley can you rebase?

Git is acting up, so will need to fix this PR before merge


feat: Refactoring the SD-Factory

Refactoring the SD-Factory to support number of formats (Catena-22-10, GaiaX, FC) as output.
It shall be fully compatible on level of Catena-22-10 with previous versions.
Creates a payload and send it unsigned to the CH

I was running the dash tool and got the following output. Please track the generated IP issue tickets found below in the links:
[INFO] This content is either not correctly mapped by the system, or requires review.
[INFO] A review is required for maven/mavencentral/com.danubetech/key-formats-java/1.6.0.
[INFO] A review request was created https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/10950 .
[INFO] A review is required for maven/mavencentral/info.weboftrust/ld-signatures-java/1.2.0.
[INFO] A review request was created https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/10951 .
[INFO] A review is required for maven/mavencentral/com.goterl/lazysodium-java/5.1.1.
[INFO] A review request was created https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/10952 .
[INFO] A review is required for maven/mavencentral/com.danubetech/verifiable-credentials-java/1.1.0.
[INFO] A review request was created https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/10953 .
[INFO] A review is required for maven/mavencentral/decentralized-identity/jsonld-common-java/1.1.0.
[INFO] A review request was created https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/10954 .

I'll merge this commit but release must not be created from this as the DEPENDENCIES file contains restricted libraries (with IP issue numbers).


Support property graphs on attached databases

What happens?
Currently, you cannot create a property graph on an attached database.
Probably since we only check the main schema.
To Reproduce
ATTACH 'https://raw.githubusercontent.com/Dtenwolde/bluesky-follows-graph/main/bluesky.duckdb';
install duckpgq from community;
load duckpgq;
CREATE OR REPLACE PROPERTY GRAPH bluesky
            VERTEX TABLES (bluesky.account)
;

OS:
macOs 13 - Apple M1 Pro
DuckDB Version:
v1.1.2
DuckDB Client:
CLI
Full Name:
Daniel ten Wolde
Affiliation:
CWI
How did you load the extension?
Community extension version
Did you include all relevant data sets for reproducing the issue?
Yes
Did you include all code required to reproduce the issue?

[X] Yes, I have

Did you include all relevant configuration (e.g., CPU architecture, Python version, Linux distribution) to reproduce the issue?

[X] Yes, I have

Mentioned here: https://github.com/cwida/duckpgq-extension/discussions/166


Docs enhancement: Adding a copy button in the command sections

Adding a copy button in the command sections would allow faster copy-pasta.

I want to work on this issue. Please assign it to me.

@ankitagit11 are you still interested in this issue?

closing since this was already implemented in https://github.com/localstack/docs/pull/719


Ensure tile-loaded event completionCallback is called only once.

The PR also errors when context2D is set after cache creation.

Possible fix for https://github.com/openseadragon/openseadragon/issues/2277

Note that tests are failing probably because there is a new security enhancement on full-screen mode - the test will simply not be trusted since it is not initiated by a user (event).

Hmm not really friendly solution with tests :/

the issue is
            assert.ok( tile.loading, "The tile should be marked as loading.");
            assert.notOk( tile.loaded, "The tile should not be marked as loaded.");
            setTimeout(function() {
                assert.ok( tile.loading, "The tile should be marked as loading.");                <-- here the default completion callback has been already executed
                assert.notOk( tile.loaded, "The tile should not be marked as loaded.");
                callback1();
                assert.ok( tile.loading, "The tile should be marked as loading.");
                assert.notOk( tile.loaded, "The tile should not be marked as loaded.");
                setTimeout(function() {
                    assert.ok( tile.loading, "The tile should be marked as loading.");
                    assert.notOk( tile.loaded, "The tile should not be marked as loaded.");
                    callback2();
                    assert.notOk( tile.loading, "The tile should not be marked as loading.");
                    assert.ok( tile.loaded, "The tile should be marked as loaded.");
                    done();
                }, 0);
            }, 0);

so now it kind of works if the callback is not asynchronous, but does not work vice versa (as intended with tests). Also, the callback is called immediately instead of waiting for the last caller who got a completion callback. Not sure what is better or worse.

Implementation changed to use flags that guard completion (https://github.com/openseadragon/openseadragon/issues/2277#issuecomment-1409481316). Still there might be issue when early completion (completion called immediately), and another plugin attaches context2D. Possibly add also flag that prevents completion before the event finishes?

In the end, I changed the error message as well as docs to be yet more explicit, it will not hurt. Using completion without async has no reason, now early completion can happen only when one hacks the semaphore with two completion calls.

This looks to me like it should work as intended. A couple of thoughts:

This would be a totally natural place to use the Promise API to resolve the loading process exactly one time using Promise.all(). I'm guessing the lack of IE support for Promise precludes this method though? (If IE support has been dropped, I have a Promise-based implementation coded up already).
An advantage of the Promise approach is that it is intrinsically robust to calling resolve more than once - subsequent invocations are simply ignored. This would guard against the cache having the same problem it does now, if someone inadvertently calls the callback multiple times. Even if we can't use Promise, I think it would be good to ensure the "completion" code runs just once, to protect against user-introduced bugs.
I do not think the comment warning against using getCompletionCallback synchronously is needed. This new implementation can be called sync or async with equivalent results. This enables cleaner downstream code and APIs. I do not see the need to scare people off from using it just because its not necessary in the sync case.
Instead, I would be more explicit that this must be used for async processing, and is optional for synchronous processing.

@pearcetm I had just last week or so PR on promise support in events, which would solve this as well (the event could just behave as synchronized await Promise.all()). We decided not to include it at the end since it brings a lot of complexity without a strong motivation why it is really needed to be in the core library (I am still using it in my app by extending EventSource). So I guess it is still not a must-have reason and we rather keep the old API if possible, because compatibility.
Offtopic: to be honest, the first thing I would like with ES6 introduction would be to rewrite manual inheritance on all OSD classes 😄

So I guess it is still not a must-have reason and we rather keep the old API if possible, because compatibility.

I saw that PR and discussion. In this case, though, it would be a purely internal use of Promise which would not change the public API at all; it really only relates to browser support, and does not expose any new complexity.

I do not see the need to scare people off from using it just because its not necessary in the sync case

I do. The filtering plugin that started this PR does pretty wild coding to call the callback at the end of filter application - for no reason (except the bugs 😄 ). It lets them know: do not touch if you don't know you need it.

I don't think the filtering plugin is "wild coding" - it is intended to support arbitrary combinations of sync and async filter operations. Since async is allowed, it gets a callback, and asks each filter to indicate when it's completed, and then calls the callback. To avoid getCompletionCallback for sync filters, it would have to change behavior depending on the exact filter stack. IMO it's a cleaner and more consistent API for developers for this to be allowed both sync and async.

I agree that we don't need the warning against using it synchronously. It'll work just fine with this fix, and it's a valid use case.
We should definitely consider the possibility of introducing ES6. I'll start a new issue for that. Either way, I don't think we need a promise here... We already have an implementation that works, and adding the first promise is a big step, even if it's just internally.
As far as I'm concerned, I think this is ready to land once the "note" is removed from the docs.
It's amusing to me how simple this has become from what seemed like a complicated issue at first! It's great working through these things together :)


Fix Issue 14099 - Promote run.dlang.io to Resources

From the issue:

This is a proposal to make it officially a part of dlang.org: promote the website to run.dlang.org, and add a link to it on the D website (e.g. under the "Resources" submenu).

So this has been sitting here for two months :/
Ping @CyberShadow
Anything blocking this?


Error installing

i have some error when yarn
this is the error
Exit code: 1
Command: ./node_modules/.bin/node-pre-gyp install --fallback-to-build --library=static_library
Arguments:
node-pre-gyp info it worked if it ends with ok
node-pre-gyp info using<EMAIL_ADDRESS>node-pre-gyp info using<EMAIL_ADDRESS>| darwin | x64
node-pre-gyp info check checked for "/Users/krisnailafat/Aplikasi_komo/react-native-mobx-firebase-starter/node_modules/grpc/src/node/extension_binary/node-v72-darwin-x64-unknown/grpc_node.node" (not found)
node-pre-gyp http GET https://storage.googleapis.com/grpc-precompiled-binaries/node/grpc/v1.10.1/node-v72-darwin-x64-unknown.tar.gz
node-pre-gyp http 403 https://storage.googleapis.com/grpc-precompiled-binaries/node/grpc/v1.10.1/node-v72-darwin-x64-unknown.tar.gz
node-pre-gyp ERR! Tried to download(403): https://storage.googleapis.com/grpc-precompiled-binaries/node/grpc/v1.10.1/node-v72-darwin-x64-unknown.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v72 ABI, unknown) (falling back to source compile with node-gyp)
node-pre-gyp http 403 status code downloading tarball https://storage.googleapis.com/grpc-precompiled-binaries/node/grpc/v1.10.1/node-v72-darwin-x64-unknown.tar.gz
node-pre-gyp ERR! Pre-built binaries not installable for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v72 ABI, unknown) (falling back to source compile with node-gyp)
node-pre-gyp ERR! Hit error Connection closed while downloading tarball file

I just cleared my node modules and was unable to reproduce. Sorry for the delayed reply. Did you try clearing modules?


Android SDK to new

I have an Android devices with Android 4.1.2 and it is to old to run this app.

Correct. API v19 is required because I use the Storage Access Framework for payload selection. You could strip this out and get it to run on lower versions, if you really want to.


Update apptainer installation instructions

Closes #9.
The apptainer guide suggests:
sudo apt update
sudo apt install -y software-properties-common
sudo add-apt-repository -y ppa:apptainer/ppa
sudo apt update
sudo apt install -y apptainer

which worked for me. I'm not sure if you want the full block though. Right now, I linked to the official docs and included the missing sudo add-apt-repository -y ppa:apptainer/ppa. But one could argue that the full example above should be included.

Well, since it's going to depend on the linux distribution and/or existing packages anyway, perhaps it's better to just include a link to the installation instructions. That way, if the instructions change, we're automatically up-to-date and don't need to worry about it (or about which statements to include and which are are considered trivial).

Makes sense to me! I agree.


Generate API reference on npm install

Just like we index versions upon install, we should generate the API once

classic merge before CI, love it


Missing information

Description
What steps will reproduce the problem?

pip list
Note: you may need to restart the kernel to use updated packages.
C:\Program Files\Spyder\Python\python.exe: No module named pip
I have just installed Spyder 5.3.1, but it seems I have missing all the moduled installed...I do not know how to run again ...could be posible to return to the previous version ?
 Seems like you are using Spyder from the standalone installer version on Windows. The installer doesn't come with pip so to install or use packages already installed in a python environment you will need to use the custom interpreter preference. For more details about how to achieve that you can check the FAQ page entries on our documentation: https://docs.spyder-ide.org/current/faq.html#using-existing-environment https://docs.spyder-ide.org/current/faq.html#using-packages-installer and
Let us know if that helps!

@haravena seems like you attached an image. However, you will need to directly post it at https://github.com/spyder-ide/spyder/issues/18480 (email attachments get lost in the GitHub interface). Thanks!
Also, just in case the path to put is the one pointing to the Python executable (python.exe) that has access to your packages. However, after we get the image or path you are trying maybe we could check was is happening in your case

Ok I See Daniel
I have put this path:
C:/Users/harav/anaconda3/pythonw.exe
I get this message from the Terminal:
"No es posible reiniciar un núcleo que no fue iniciado por Spyder"
Thanks
On Wed, Jun 29, 2022 at 12:42 PM Daniel Althviz Moré <
@.***> wrote:

@haravena https://github.com/haravena seems like you attached an image.
However, you will need to directly post it at #18480
https://github.com/spyder-ide/spyder/issues/18480 (email attachments
get lost in the GitHub interface). Thanks!
Also, just in case the path to put is the one pointing to the Python
executable (python.exe) that has access to your packages. However, after
we get the image or path you are trying maybe we could check was is
happening in your case
—
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/18480#issuecomment-1170224908,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQDWN467QL5XEPQ77SNJLEDVRR4IXANCNFSM52GDNSYQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks Daniel,
Unfortunately it is not working for me.
!An error occurred while starting the kernel
The following argument was not expected:
C:/Users/harav/anaconda3/envs/spyder‑packages
Run with ‑‑help for more information."
What can I do ?...uninstall everything and start with an older version of
Spyder mayb could help?
On Wed, Jun 29, 2022 at 1:01 PM Daniel Althviz Moré <
@.***> wrote:

I see, I think the easiest approach will be then to create a new env for
you using the conda-forge channel, installing there the packages you want
to use, and then selecting the Python executable from the new env. To
create the new env you will need to run from an Anaconda prompt something
like this:
conda create -n spyder-packages -c conda-forge spyder-kernels
conda activate spyder-packages
Then install with the env activated, you can install the packages you want
to use with something like conda install -c conda-forge
After that, go to Spyder interpreter preferences and set the Python
executable of the new env. Most probably you will set then as custom
interpreter something like this path
C:/Users/harav/anaconda3/envs/spyder-packages/python.exe
I know is kind of tricky to follow all this steps but let us know if the
instructions above help or if you have any other question.
—
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/18480#issuecomment-1170243479,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQDWN45VQNYX5AP6JWYGFE3VRR6PBANCNFSM52GDNSYQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

I would then suggest as you say to try to uninstall the standalone Spyder installation and install it again. Also, from where did you downladed that version?
Another approach could be to directly install spyder in the new env that was created. For that, from an Anaconda prompt, run something like this:
conda activate spyder-packages
conda install -c conda-forge spyder
spyder

With that you will install and launch Spyder from the created env

I have downloaded from https://github.com/spyder-ide/spyder/releases
On Wed, Jun 29, 2022 at 4:15 PM Daniel Althviz Moré <
@.***> wrote:

I would then suggest as you say to try to uninstall the standalone Spyder
installation and install it again. Also, from where did you downladed that
version?
Another approach could be to directly install spyder in the new env that
was created. For that, from an Anaconda prompt, run something like this:
conda activate spyder-packages
conda install -c conda-forge spyder
spyder
With that you will install and launch Spyder from the created env
—
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/18480#issuecomment-1170451825,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQDWN46NPEMVUYD7QLERZNTVRSVFTANCNFSM52GDNSYQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

No problem @haravena , glad to hear that now Spyder is working for you :+1:
Regarding the error running your code, what version of yfinance are you using? The latest one from what I can see is 0.1.72. Maybe you need to update it? Also, there is an issue about json.decoder.JSONDecodeError here: https://github.com/ranaroussi/yfinance/issues/764#issue-941000623 (I guess you need to install at least yfinance 0.1.62 to prevent decoding errors).

Awesome, glad everything is working now for you :+1:
I will close this issue then but if you have any other questions or comments regarding Spyder let us know!

Sure!, thanks again!
On Thu, Jun 30, 2022 at 3:00 PM Daniel Althviz Moré <
@.***> wrote:

Closed #18480 https://github.com/spyder-ide/spyder/issues/18480 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/18480#event-6913316178, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AQDWN47ZDR2YJPFRH5AOD73VRXVGNANCNFSM52GDNSYQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Use import instead of require

As pointed out in #161 if use of ES6+ is encouraged why not use the ES6 import syntax everywhere instead of require?
Happy to take this.

Hey Tim! I think I want to change this example to use fs, request, and or some other async modules. The require example is throwing everyone off.


Tooltip description not the same that in game

Hello,
Look at the joined captured images :

I have got 2 others shields called "Vengeance...", same problem.

This is because Prefix/Suffix/Quality/Style have been added to the item name in the tooltype.
Is it a problem ?

No it's not the problem about the item name.
There are some other differences about tooltip informations in game :

"Block recovery" not present in TQVault
+xx Intelligence : 25 in TQVault --> 26 in game
"15.0% Chance of xx Fire Retaliation" : 35 in TQVault --> 30 in game
But TQVault gives better information like the time of Recharge, not present in game.

This is not a big problem.

An alternative could be to display the full qualified name on "detailed view mode" only (Left-Alt)

No it's not the problem about the item name.
There are some other differences about tooltip informations in game :

"Block recovery" not present in TQVault
+xx Intelligence : 25 in TQVault --> 26 in game
"15.0% Chance of xx Fire Retaliation" : 35 in TQVault --> 30 in game
But TQVault gives better information like the time of Recharge, not present in game.

This is not a big problem.

An alternative could be to display the full qualified name on "detailed view mode" only (Left-Alt)

An alternative could be to display the full qualified name on "detailed view mode" only (Left-Alt)

Hi @QStormDS ,
This is because TQVault display the "base value" of the attribute.
In game, that "base value" is pseudo-randomized from the "Item-seed" value.
TQVault doesn't know that pseudo-random logic, therefore it can't compute back the exact same value.


Enhancement - game filtering

After game list construction, we have enough data from listxml to add the following filters, if users wish to select games in a more controlled fashion:

Manufacturer
Year range

Another useful thing to filter by wold be category. This would be via catver.ini, but this isn't bundled with MAME. So we could have a button which downloads it (perhaps from here?)
Thoughts?

I never knew about catver.ini - that's really cool - yeah I love this idea! 💯 ❤

So what are you thinking - filter the list with some dropdowns at the top or a button which pops open a filter dialog?

I need to have a play. It is nicer if the filter was applied dynamically and was in the same window, but there's very little real-estate (particularly if there are two drop-downs, two input fields, potentially some help text if catver.ini isn't present, and a download button).

Here's an example of my PoC thus far. Key points:

Select all / deselect all are via the header checkbox
If the user deselects individual games, this updates to partial state
The checkbox for filters updates based on the filter state

I should change the select all / deselect all to be based on the filtered set; currently its over the whole collection.

Thoughts?

Wow I ❤ it! I think the checkboxes are fine - although if it's easy you could compare to that filter icon you were talking about. One other thing - I suggest a clear filters button somewhere (maybe on the right side in line with Rebuild List...

Out of the box styles with this library are... not great. Very easy to customise though. The filter icon itself is alright(ish). I think the dot is meant to indicate that the filter has been applied.

Heh you're as pedantic about US as I am 😆 this never ends well hehe
The icons look great but I wonder - could you highlight them (maybe use SystemColors.HighlightColor) when a filter is used? I think they might be more visible - otherwise they are awesome.
I think that the even/odd row coloring is very mac specific and not very Windows personally...

By the way how are you handling the category xml? Mamesaver downloading itself?

Heh you're as pedantic about UIs as I am 😆 this never ends well hehe

No, it's great! It's fantastic to work with someone with opinions - provided they're not diametrically opposed to mine 😉

The icons look great but I wonder - could you highlight them (maybe use SystemColors.HighlightColor) when a filter is used? I think they might be more visible - otherwise they are awesome.

I was wondering that. I'll see if I can coerce the existing styling.

I think that the even/odd row coloring is very mac specific and not very Windows personally...

Easy fix for that!

By the way how are you handling the category xml? Mamesaver downloading itself?

I'm not, yet. The WPF app is currently standalone with dummy data, while I focus on the UI. I was intending for it to be downloaded automatically on first rebuild and perhaps after a 'Refresh category on next rebuild' checkbox on the Advanced tab?
catver.ini is around 330Kb and https://github.com/mamesupport/catver.ini gzips responses, so this automatic approach should be fine.
FWIW, here's the current state, showing an active filter:

Looks awesome! Is there any way to make the rest of the grid white - looks a bit strange currently - like something's not drawing 😆
Maybe we should take a copy of the category file and include an initial one on install and then allow it to be updated? Otherwise the other way might be to give all the games a generic category until it's downloaded or to hide the category column?
Loving the direction...

Is there any way to make the rest of the grid white - looks a bit strange currently - like something's not drawing

Oops, I stopped noticing that! Yes, it's awful and keep meaning to fix.

Maybe we should take a copy of the category file and include an initial one on install and then allow it to be updated?

Not a bad idea. The screensaver isn't too huge. I would like to make the whole process transparent unless users care - an 'update categories' button on the Advanced tab?
This is just a dummy project currently, so I think enough is bedded in now to properly integrate it and migrate the configuration WinForms code?

sounds good to me

Resolved by PR #46.


Free cource

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Looks like you have taken a different base branch to base your changes. Can you take the main branch and update the review?


Fix Swift 6 compatibility

See #190.
Factory is not compatible with Swift 6, so this makes that explicit. Without this, apps compiling against Factory when using Swift 6 will try to compile it with that and fail.

Actually this doesn't seem to help, so this is an Xcode / Swift bug

There's more to it than this. Working on it. Optional cached and shared types are also broken in 6.

There's more to it than this

You mean more errors?

Yep, going to close this as more work was needed. You can see the changes in the swift6 branch.

Sure I appreciate it. This was just the minimum work to at least be able to compile, but looking forward to a more correct solution 👍🏻


media.player.yamaha: some volume levels are unusable

Home Assistant release (hass --version):
0.35.3 (docker image)
Component/platform:
media_player.yamaha
Description of problem:
When calling the service media_player.volume_set with some specific values, the API call made to the receiver are invalid (volume level is out of range by an order of 10 as far as I understand). Such problematic values are 0.55 and 0.56: the rxv library outputs an error (see below) and the volume isn't set. Calling with 0.54 or 0.57 works perfectly fine.
Expected:
The volume should be set correctly, no matter the input value as long as it's in the 0..100 range
Traceback (if applicable):
17-01-07 18:19:42 ERROR (Thread-8) [rxv] Request <YAMAHA_AV cmd="PUT"><Main_Zone><Volume><Lvl><Val>-449</Val><Exp>1</Exp><Unit>dB</Unit></Lvl></Volume></Main_Zone></YAMAHA_AV> failed with b'<YAMAHA_AV rsp="PUT" RC="3"><Main_Zone><Volume><Lvl></Lvl></Volume></Main_Zone></YAMAHA_AV>'

Additional info:
The stacktrace above is the result of a call with 0.55 as parameter. While the receiver can go from something to -80dB to +15dB, the call includes -449, probably instead of -45 or something like that. 0.54 matches -46dB on the receiver, and 0.57 matches -43dB. I couldn't get a log for a functioning call to compare.

I think I know what's going wrong: the receiver only supports 0.5dB increments. You can set -46dB, -46.5dB, but not -46.4dB for example. The receiver's API seems to work with the level * 10 (to avoid floating values I guess).
On the case above, I guess the receiver is expecting -450 (-45dB) or -445 (-44.5dB), but ends up receiving -449 (-44.9dB), which is invalid. Looking at hass's code for the Yamaha component, I guess the issue resides in the way rxv does its computation from -100~0 values to -1000~0 ones.

Sadly, this still doesn't work correctly in 0.40 :-(

i worked around this issue using the yamaha REST API: https://community.home-assistant.io/t/setting-yamaha-media-player-volume/18683
probably, on the long run, it should be considered switching the ymaha component to a library that uses the REST API, instead of the older, XML based API that also doesn’t handle zones well with my RX-V581.

This issue is still relevant

still relevant.

Using the REST API for this component is a non starter, as it would break support for 90% of users out there that have older receivers. Also, the XML-RPC interface includes an open socket which provides basically instantaneous event stream on the device changes. I don't know what the REST polling would be.
This really is a concern in the dependency library, and needs to be handled there. Especially if this is model specific. The rxv library should be doing that filtering and rounding there.

Upstream dependency fix proposed here - https://github.com/wuub/rxv/pull/42

Thanks to you and the power of maths! :-)

Fixed in dev with bump to rxv library.