Running from Docker

Hi there
My name is Antonio and I am studying Bioinformatics at University of Copenhagen.
I was directed to this tool by @Maux82 in https://github.com/compomics/moFF/issues/26. I am trying to use ThermoRawFileParser through the Docker image and I am encountering some path issues when mapping the Docker volumes to files in my local filesystem.
I built the image using the command in your README.md
docker build --no-cache -t thermorawparser .
Then I tried running
docker run -v $DATA_INPUT:/data_input -i -t --user biodocker thermorawparser mono /home/biodocker/bin/bin/Debug/ThermoRawFileParser.exe -i=/data_input/$RAW_FILE -o=/data_input/output -m -c=$PXD
where

DATA_INPUT is an absolute path to the directory where my RAW_FILE is
RAW_FILE is the name of my raw file (no path prepended, i.e myfile.RAW)
PXD is a custom string.

I get the following error (please click on the image to get the full picture):

The file I am trying to convert is available here:
https://www.ebi.ac.uk/pride/data/archive/2014/09/PXD000279/20070920_CL_Orbi2_XIC_Hela60_Ecoli10_Offgel_green_Frac4_02.RAW
and it belongs to the dataset used in the MaxLFQ paper http://www.mcponline.org/content/13/9/2513.long
I am running this on an Ubuntu 16.04.
Could it be due to the fact that the path /data_input does not exist inside the container filesystem?
Thanks beforehand for your help and for developing such a useful tool!
Best
Antonio

Hi Antonio,
Thanks for trying out the library, it's still in development so any suggestions/improvements are more than welcome! I'm adding mzML export functionality for the moment.
I've tested the RAW file and it works on my computer. If the output directory exists in $DATA_INPUT, it should be mapped inside the container. I ran the following command (as root user)
sudo docker run -v /home/niels/Desktop/raw:/data_input -i -t --user biodocker thermorawparser mono /home/biodocker/bin/bin/Debug/ThermoRawFileParser.exe -i=/data_input/20070920_CL_Orbi2_XIC_Hela60_Ecoli10_Offgel_green_Frac4_02.RAW -o=/data_input/test/ -m -c=PXD00001
I've added the docker file for biodocker usage. You can also use mono
mono ThermoRawFileParser.exe -i=/home/niels/Desktop/raw/20070920_CL_Orbi2_XIC_Hela60_Ecoli10_Offgel_green_Frac4_02.RAW -o=/home/niels/Desktop/raw/test/
Best regards,
Niels

Hi Niels,
Thanks for your fast answer. I am very happy to see someone addressing the need to get the raw->mgf/mzml in Linux working!
I tried creating a directory output inside $DATA_INPUT  and running Docker with sudo, but still no luck getting it to work. I think it's due to some permission issues between Docker and my host. I can see that if I pass the -m flag:
2018-05-23 22:46:36,915 ERROR ThermoRawFileParser.MainClass - System.UnauthorizedAccessss to the path "/data_input/output/20070920_CL_Orbi2_XIC_Hela60_Ecoli10_Offgel_green_Frac4_02_metadata" is denied.
while if I don't pass -m nor -c
2018-05-23 22:47:25,608 ERROR ThermoRawFileParser.MainClass - System.UnauthorizedAccessException: Access to the path "/data_input/output/20070920_CL_Orbi2_XIC_Hela60_Ecoli10_Offgel_green_Frac4_02.mgf" is denied.
i.e the error is the same just trying to create different files. Could it be that this permission issue is not present when running in your computer?
This is the output of ls when run on $DATA_INPUT
>ls -1 $DATA_INPUT
20070920_CL_Orbi2_XIC_Hela60_Ecoli10_Offgel_green_Frac4_02.RAW
convert.sh
output
Thanks for your time and help
Best
Antonio

Hi @chrishuges
Thanks for your answer! I am sorry I got back to you this late.
I still haven't been able to get it to work with Docker. It would be great if @nielshulstaert  could share a more comprehensive MWE. Could these weird errors be due to my filesystem being NFS mounted?
Moreover, I just got to install the singularity shell, but your link is broken. I tried this instead
https://github.com/kevinkovalchik/RawQuant/tree/qc_dev (I am guessing you just changed your branch name :stuck_out_tongue: ). But whether I install from the contents or this folder, or running
sudo pip install RawQuant I get an error. Maybe I should open an issue in your repo?
Thanks for your input and your help!
Best
Antonio

I just tried setting DATA_INPUT to a folder under /tmp, so it's not under the NFS system, and now I get the following error:
2018-05-30 13:55:06,846 INFO ThermoRawFileParser.CentroidedMgfExtractor - Started parsing /data_input/20070920_CL_Orbi2_XIC_Hela60_Ecoli10_Offgel_green_Frac4_02.RAW
2018-05-30 13:55:07,369 ERROR ThermoRawFileParser.CentroidedMgfExtractor - Unable to access the RAW file using the RawFileReader class!

Hi Antonio,
I meant to use singularity as an alternative to docker, but based on your latest error it seems that this is not your issue. Sorry I couldn't be of more help here.
Yes for the error with our tool please open an issue on our repository with details of the problem. The link was broken because we just pushed that branch to master yesterday actually!
Chris

Hi Niels
No worries! I tried building  the image with the Dockerfile you wrote in the comment above and running the command. I got it to work!!
But it works only if the directory where the RAW file "lives" is not nfs (network filesystem) mounted. My home directory is nfs mounted, so I just placed the RAW in the /tmp folder.
Also, the output directory needs to exist, i.e there needs to be a test directory in /home/niels/Desktop/raw (as you said in a previous comment)
I get as output 2 files in test, the mgf file and the metadata file. It took ~15 seconds to run (pretty fast) :+1:
Thanks for the support and for making this great tool!!
Thanks Chris also for the suggestions.
Best
Antonio


give a way to style the white background by providing containerClassName prop

Is your feature request related to a problem? Please describe.
currently, i can't style the background using anything other than white
Describe the solution you'd like
i always style in dark mode & would love dark background using a prop like containerClassname as i style through tailwind
Describe alternatives you've considered
passing a containerClassname prop
Additional context

i tried className as h-screen p-0 m-0 on the Story & got the above result but still some white remains. see my code:
import WhatsappStories from 'stories-react'

interface IStory {
	title: string
	body: string
}

interface IStories {
	stories: IStory[]
}

const Story = ({ title, body }: IStory) => {
	return (

{title}

{body}

	)
}

export const Stories = ({ stories }: IStories) => {
	const allStories = stories.map((s) => ({
		type: 'component',
		duration: 9000,
		component: () => <Story title={s.title} body={s.body} />,
	}))

	return (
		<WhatsappStories
			width="400px"
			height="600px"
			stories={allStories}
			className="dark:bg-indigo-700"
		/>
	)
}

would love to style it with a simple containerClassName prop & for the 2 div's above WhatsappStories & a className prop for the WhatsappStories as well so the bg-indigo-700 works :)

this is a good usecase. i will add some classnames which allows to update all containers.


add kerberos login

to login there is a hadoop class  UserGroupInformation and method loginUserFromKeytab.
because in ranger we use RangerAdminRESTClient with getServicePoliciesIfUpdated there is MiscUtil.getUGILoginUser() which runs checkTGTAndReloginFromKeytab() on the UserGroupInformation - so the relogin is done in case of credentials expiration.

Codecov Report

Merging #27 into master will decrease coverage by 1.69%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master      #27     +/-   ##
=========================================
- Coverage   85.66%   83.96%   -1.7%
=========================================
  Files          30       32      +2
  Lines         593      605     +12
  Branches       54       57      +3
=========================================
  Hits          508      508
- Misses         85       97     +12

Impacted Files
Coverage Δ

...g/wbaa/airlock/proxy/config/KerberosSettings.scala
0% <0%> (ø)

...airlock/proxy/provider/KerberosLoginProvider.scala
0% <0%> (ø)

Continue to review full report at Codecov.

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


preference.setEnabled(false) does only disable "android:summary"

preference.setEnabled(false) should disable the whole preference, not only the summary

Sadly I have issues on my Android Studio currently, but I've looked at the code of Android, and I can't find what's responsible of changing the style of the textViews, except for this:
/**
 * Makes sure the view (and any children) get the enabled state changed.
 */
private void setEnabledStateOnViews(View v, boolean enabled) {
    v.setEnabled(enabled);

    if (v instanceof ViewGroup) {
        final ViewGroup vg = (ViewGroup) v;
        for (int i = vg.getChildCount() - 1; i >= 0; i--) {
            setEnabledStateOnViews(vg.getChildAt(i), enabled);
        }
    }
}

Yet the style of the original TextViews do not have anything special, so I wonder what's going on...

@Tomblarom Why is it closed? Have you found a way to solve it?

Anyway, I can't reproduce this.

I'm going to color the icon grey ;) Don't know how to solve it.

@Tomblarom This doesn't make sense. Can you please share a minimal project showing this issue?
I don't see this problem.
Also, for icons to have multiple states, you can use a selector, just like any other drawable.

@Tomblarom  How did your change color?

@AndroidDeveloperLB  This happens on "com.lb.material_preferences_library.custom_preferences.Preference"

Found the issue, you need to always return false; on onPreferenceClick.

@jemshit I don't understand what's the issue you are talking about. Please make a new post.

@AndroidDeveloperLB  This issue.When .setEnabled(false) is called, preference title still looks like enabled with black color. Solved.

@jemshit I can't reproduce it. Please create a new post, and show how you did it and what was the fix.


🐛 [Bug report]: Modal in Memory Game lost responsiveness

The modal which open by clicking "how to play" button in memory game play has lost it's responsiveness after adding tailwind i guess.
indeed we need to fix it by reducing the aspect ratio of the image.
anyone who is good at css may fix it i guess.

@Angryman18 I see it working correctly on prod. Have you checked on reactplay.io or localhost?

Yes it works on small screen but on 720p display its broken like the screenshot. Also works well on 1080p.

It looks to be working fine now, otherwise, i'd like to be assigned this issue

hey, @Angryman18  in which device there is a CSS break ..?

hey, @Angryman18 in which device there is a CSS break ..?

768p or 720p. Please check it.

hey @Angryman18 , I checked it is working fine in maximum number of devices but it's breaking for a specific res like this...

Can you fix it. We want it to appear properly on all sort of resolution.

@Angryman18 yeah,
I will try my best to fix it asap..!

Hey can u assign me for this i would like to try solving it ..!


fix(docs): change pipe to curl -s option

What this PR changes/adds
It removes a pipe operator
Why it does that
Because otherwise the -s option for curl is understood as a command line command. Simply copy pasting the example code results in an error in the command line

Done in #149


Email OTP invalid message is not translated properly in other languages

Describe the issue:
The invalid email otp message is not translated properly.

How to reproduce:

Navigate to email_otp.do page
Change to another language
Enter an invalid otp and verify the error message displayed

Expected behavior:
The message should be translated comprehensible
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

Clsoing this issue because the master PR is already merged.


pull request for e2e test at 02/04/2017 17:26:08

:white_check_mark: Validation status: passed
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.


slate.config.js not loading properly

In trying to override/extend some settings in my slate.config.js, I noticed that the settings weren't taking. It seems like it needs to be nested under slateTools based on the way the schema crawls the configs? Not sure which project to submit this to?
module.exports = {
  slateTools: {
    extends: {
      dev: {resolve: {alias}, output: {library: 'Wardrobe', jsonpFunction: 'webpackJsonpWardrobe'}},
      prod: {resolve: {alias}, output: {library: 'Wardrobe', jsonpFunction: 'webpackJsonpWardrobe'}},
    },
  }
};

Fixed in https://github.com/Shopify/starter-theme/commit/23f14812ebedd49490632f2f3cdb805add4cd388


Filter to include every possible verse

Filter to include every possible verse detecting the highest number of verse.
For example, If chapter 1 has 9 verses, the filter should show verses 1 through 9, regardless of the fact whether there is a note for that verse or not.
Currently, the filter shows only those verses in the drop-down that has an existing note.

This is still a problem in  - v1.10 build 228-f43ba9b.

When a new note is added, the filter list will update and show the newly added verse.

This works as expected in translationCore Create - v1.10 build 228-f43ba9b  @birchamp


running manage.py drive get ImportError "No module named 'controller'"

I have update my git clone to the nearest version (v 2.5.1), and I have successfully "pip install -e ." and "pip install donkey[pi]", but still got this error.
By the way, I tried to mix it by changing "manage.py" file in mycar, instand of "import from controller", I set it to be "from donkeycar.parts.web_controller.web import LocalWebController".  This does fix the problem temporarily, but I can't record my data with the recording button pressed on on the website ( I do can control the car on the website).
Thanks a lot if anyone have an idea of this issue, this problem hinders three of my groups from processing on.
From: Beijing University of Posts and Telecommunications， China

I think you are using the dev branch. Switch to master and run pip install -e . again. You should now be able to use the latest version.

I think @tikurahul is right. When you clone donkey, git's HEAD points to dev branch.  Type git branch or git branch -a and you should see what branch you are using. Then if it is dev branch you are using, just type git checkout master in a directory where you have cloned donkey, and run pip install -e . in that same directory. Now it should be fine.

@wroscoe Is that means that support for Joysticks were removed with commit: https://github.com/wroscoe/donkey/commit/311f999da904b17e3021f37e480fbf81bd77effe ?
According to documentation it is still there
https://docs.donkeycar.com/guide/get_driving/
I would recommend to not remove Joystick support, as it will allow to control vehicle without lag.

@gang65 These commit only effects the dev branch. Please use the master branch which has the changes you need.

I did checkout to master and did pip install -e . in the same directory , but I'm still getting the same error.
~/mycar$python manage.py train --tub data/01/ model ./models/mypilot
using donkey v2.5.1 ...
Traceback (most recent call last):
  File "manage.py", line 24, in <module>
    from controller import LocalWebController, JoystickController
ImportError: No module named 'controller'

@smahajan07 Have you activated donkey environment, before pip install -e .? By the way, you could copy your data to some other directory and run donkey createcar ~/mycar  again, but I don't see the reason for setup to not work if you have run the command inside donkey directory that you have cloned.

So an important thing to mention is that after you install it again, you will have to run donkey createcar <name> again and then run the manage.py from it

@igy254 I just saw your message! My bad! And to answer you, the environment was activated , I just had to run the donkey createcar  again. Thank you for your prompt response though!

@all Really sorry that I check this issue after so long, thanks a lot for your help, I will try it in no time.

I don't have a controller module either. I tried running "pip install -e", but it wants an argument (and I don't know what to put). I've also tried "donkey createcar", but it says, "createcar: error: the following arguments are required: path". Does anybody have a solution?

You should do pip install -e . (You are missing a .).

You will also need to rerun donkey create car with the right environment set. Make sure you read all the above comments.

Thanks, creating a new car solved the issue

最新版本的donkey（2.5.1）为什么摇杆控制速度不可以了，也无法通过按键增加或减小速度。

我正在使用谷歌翻译，所以请原谅这个中文的错误 I am using google translate so please excuse mistakes in this Chinese 我相信你所问的与这个帖子无关 I believe what you are asking doesn't relate to this post so please create a new issue here 所以，请 在这里创建一个新问题。


Add Python 3.0.x, 3.1.x, 3.2.x, 3.3.x and 3.4.x to python-versions.

Tool information

Tool name: Python
Tool license:
Add or update? Add
Desired version: Any version of 3.0.x, 3.1.x, 3.2.x, 3.3.x, and 3.4.x on Windows for both x86 and x64
Approximate size:
If this is an add request:

Brief description of tool: A interpreter based programming language.
URL for tool's homepage: https://www.python.org/downloads/
Provide a basic test case to validate the tool's functionality: import sys; print(sys.modules)

Area for Triage:

Question, Bug, or Feature?:

Feature
Virtual environments affected

[x] Ubuntu 16.04
[x] Ubuntu 18.04
[x] Ubuntu 20.04
[x] macOS 10.15
[x] macOS 11.0
[x] Windows Server 2016 R2
[x] Windows Server 2019

Can this tool be installed during the build?

Nope, the build errors because the versions of the tool needed do not exist on some of the virtual environments (windows specifically most of the time).
3.4.x might work for linux / mac since there are versions in python-versions with only linux or mac, however 3.3.x, 3.2.x, 3.1.x and 3.0.x are missing entirely.
Tool installation time in runtime

Not sure.
Are you willing to submit a PR?

I already submitted one to python-versions including an temporary fix for windows only. I will need help since I do not have a working linux and mac to compile for those as well for the python versions I pull requested to add.
https://github.com/actions/python-versions/pull/61
Also I got the sources to the absolute latest versions of those python versions however trying to build the msi installer files for python does not work since I do not know exactly how to find all the things I actually need to create the msi installer for those specific python versions.
The urls I used to download the installers for these in the mean time:
https://www.python.org/downloads/release/python-301/
https://www.python.org/downloads/release/python-314/
https://www.python.org/downloads/release/python-325/
https://www.python.org/downloads/release/python-335/
https://www.python.org/downloads/release/python-344/

Hello @AraHaan , thank you for feature request.
For Ubuntu and MacOS, we build python from source code. It is pretty tricky to build those old versions. Some time ago we have tried but failed and wouldn't like to invest more time into it. For example, Python 3.4 end of life was more than 1 year ago.
For Windows, we don't build from source code and just re-pack existing Python binaries so if it makes sense to you, we can upload Windows binaries for these versions

Alright, however I do know how to build those versions of python as I done so before, however for 3.4.10 it fails to create the msi for x64 since it fails on also the ssl module and the xxlimited module as well, but I did manage to build x86 msi for 3.4.10 however.

Hello @AraHaan !
We've published the following Python versions for Windows to the actions/python-versions repository:

3.4.4
3.3.5
3.2.5
3.1.4
3.0.1

We've also updated the versions-manifest.json file in scope of this pull request, so these Python versions can be installed by actions/setup-python action.
I'm closing this issue, but feel free to contact us if you have any questions.

Just leaving this comment here' not 100% sure if this is the correct place: It looks like the old Python versions were now added (requesting Python version 3.4.4 on Windows x86 does now work), but it seems like there is an error in the install script.
The file that is downloaded contains a "setup.ps1" script that installs and configures the Python stuff, and that script is updating Pip with the following command cmd.exe /c "$PythonExePath -m ensurepip && $PythonExePath -m pip install --upgrade pip".
In my case, this removes Pip 7.1.2 and installs Pip 20.2.4.
However, it looks like Pip 20.2.4 requires Python 3.5 or later, so this Python installation is effectively broken now as it has a Pip version that can no longer run using the installed Python version:
  File "c:\hostedtoolcache\windows\python\3.4.4\x86\lib\site-packages\pip\_vendor\pkg_resources\__init__.py", line 92, in <module>
    raise RuntimeError("Python 3.5 or later is required")
RuntimeError: Python 3.5 or later is required

Can the installation script be changed so it doesn't update Pip to the most recent version, or maybe make it update to the last version that supports the given Python version?

cc: @MaksimZhukov to take a look.

Hello @Leseratte10 ! Thank you for the your comment!
We have rebuilt the recently added Python versions. We've removed Pip upgrading for Python 3.4.4 so the default Pip version will be used. Could you please verify if it works for you?
Please note that Pip is not automatically installed with Python less than 3.4. If you are going to use Pip, you need to install it yourself.

Thanks. I can confirm that Python 3.4.4 now works fine with my app, that was all I needed for Windows XP support.
I tested 3.3 as well and that still display some weird error, but I believe that is due to my application / the fact that 3.3 is very very outdated, and not due to a packaging error.

Thanks for the quick response!
I will close this issue, but feel free to contact us if you have any concerns.


Add a caution about using max_num_segments with low values

Summary of the changes / Why this improves CrateDB
We have seen cases recently where max_num_segments = 1 was used in an attempt to optimize large data volumes (shard sizes of > 100 GB). Query performance was significantly worse after the forced merge.
Elasticsearch has a similar warning: https://www.elastic.co/guide/en/elasticsearch/reference/current/indices-forcemerge.html
Checklist

[X] Added an entry in CHANGES.txt for user-facing changes
[X] Updated documentation & sql_features table for user-facing changes
[X] Touched code is covered by tests
[X] CLA is signed
[X] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Thanks a lot @hammerhead !


wire up cred-ex deletion; delete calls take HTTP DELETE methods

Signed-off-by: sklump<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@af9af82). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #747   +/-   ##
=========================================
  Coverage          ?   98.97%
=========================================
  Files             ?      256
  Lines             ?    13986
  Branches          ?        0
=========================================
  Hits              ?    13843
  Misses            ?      143
  Partials          ?        0

What's up with black?


Move to chainguard static

Move to chainguard static to have zero vuln

With chainguard
grype ghcr.io/guacsec/guac:v0.0.0-local-organic-guac-arm64
 ✔ Vulnerability DB                [no update available]
 ✔ Loaded image                                                                                ghcr.io/guacsec/guac:v0.0.0-local-organic-guac-arm64
 ✔ Parsed image                                                                     sha256:fd5bc85f4446390cfa77f606f63bfa6c6ea2fb58d7b31474322e719f31e76f27
 ✔ Cataloged contents                                                                      4e2577db1e7426f462a0ed67ed8f8148b618556f64e11a954a4eda697d132706
   ├── ✔ Packages                        [890 packages]
   ├── ✔ File digests                    [397 files]
   ├── ✔ File metadata                   [397 locations]
   └── ✔ Executables                     [6 executables]
 ✔ Scanned for vulnerabilities     [0 vulnerability matches]
   ├── by severity: 0 critical, 0 high, 0 medium, 0 low, 0 negligible
   └── by status:   0 fixed, 0 not-fixed, 0 ignored

With alpine
 grype ghcr.io/guacsec/guac:v0.7.2-arm64
✔ Vulnerability DB                [no update available]
✔ Pulled image
✔ Loaded image                                                                                ghcr.io/guacsec/guac:v0.7.2-arm64
✔ Parsed image                                                                     sha256:73d34b8dc03e2227a64256da524798b08ef64177c3e897df811d2a4c512d675d
✔ Cataloged contents                                                                      105648747f79dd2db65573ec29df61147b8a3ea27ef0634befacdc33979ac3c7
  ├── ✔ Packages                        [882 packages]
  ├── ✔ File digests                    [83 files]
  ├── ✔ File metadata                   [83 locations]
  └── ✔ Executables                     [28 executables]
✔ Scanned for vulnerabilities     [46 vulnerability matches]
  ├── by severity: 6 critical, 0 high, 20 medium, 0 low, 0 negligible (20 unknown)
  └── by status:   16 fixed, 30 not-fixed, 0 ignored
NAME                                              INSTALLED   FIXED-IN    TYPE       VULNERABILITY        SEVERITY
busybox                                           1.36.1-r28  1.36.1-r29  apk        CVE-2023-42365       Medium
busybox                                           1.36.1-r28  1.36.1-r29  apk        CVE-2023-42364       Medium
busybox-binsh                                     1.36.1-r28  1.36.1-r29  apk        CVE-2023-42365       Medium
busybox-binsh                                     1.36.1-r28  1.36.1-r29  apk        CVE-2023-42364       Medium
github.com/Azure/azure-sdk-for-go/sdk/azidentity  v1.5.1      1.6.0       go-module  GHSA-m5vv-6r4h-3vj9  Medium
github.com/vektah/gqlparser/v2                    v2.5.11     2.5.14      go-module  GHSA-2hmf-46v7-v6fx  Medium
libcrypto3                                        3.3.0-r2    3.3.0-r3    apk        CVE-2024-4741        Unknown
libssl3                                           3.3.0-r2    3.3.0-r3    apk        CVE-2024-4741        Unknown
ssl_client                                        1.36.1-r28  1.36.1-r29  apk        CVE-2023-42365       Medium
ssl_client                                        1.36.1-r28  1.36.1-r29  apk        CVE-2023-42364       Medium
stdlib                                            go1.22.1                go-module  CVE-2024-24790       Critical
stdlib                                            go1.22.1                go-module  CVE-2024-24789       Medium
stdlib                                            go1.22.1                go-module  CVE-2024-24788       Unknown
stdlib                                            go1.22.1                go-module  CVE-2024-24787       Unknown
stdlib                                            go1.22.1                go-module  CVE-2023-45288       Unknown

Description of the PR

PR Checklist

[ ] All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
[ ] All new changes are covered by tests
[ ] If GraphQL schema is changed, make generate has been run
[ ] If GraphQL schema is changed, GraphQL client updates/additions have been made
[ ] If OpenAPI spec is changed, make generate has been run
[ ] If collectsub protobuf has been changed, make proto has been run
[ ] All CI checks are passing (tests and formatting)
[ ] All dependent PRs have already been merged

We agree on having a static image. However, one concern we chatted about was availability of these images - for example, if a user says they have an issue with an image they've built, we should be able to reproduce it with the hash of the base image that they used. In the case of chainguard images, i don't think this is possible.
Also note that to explain the golang vulns are no longer there not because of the base image, since they are introduced by code, but because dependabot updates have been applied and the old image doesn't have them.

Let's use this image:

gcr.io/distroless/static
https://github.com/GoogleContainerTools/distroless/tree/main/base

Which will have old hashes available.


Library

The __init.py__ file in the dpemu directory is now empty, with all relevant files moved directly under dpemu.
N.B Tests should pass but some imports in the examples may fail!

Codecov Report

Merging #161 into master will decrease coverage by 2.25%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #161      +/-   ##
==========================================
- Coverage   58.69%   56.44%   -2.26%
==========================================
  Files          14       13       -1
  Lines        1276     1265      -11
==========================================
- Hits          749      714      -35
- Misses        527      551      +24

Impacted Files
Coverage Δ

dpemu/dataset_utils.py
0% <ø> (ø)

dpemu/pg_utils.py
100% <ø> (ø)

dpemu/radius_generators.py
86.36% <ø> (ø)

dpemu/ml_utils.py
0% <ø> (ø)

dpemu/problemgenerator/filters.py
75.79% <100%> (ø)
:arrow_up:

dpemu/problemgenerator/series.py
92.85% <100%> (ø)
:arrow_up:

dpemu/plotting_utils.py
23.37% <100%> (ø)

dpemu/problemgenerator/tuple.py
0% <0%> (-50%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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


Windows version detection issue When migrating Media Center to Server 2003 x86 When running Windows Update with Legacy Update

Hi Adam
Do you know the XP2ESD project in the MDL forum?
(Create modern Windows XP installer)
project information
https://forums.mydigitallife.net/threads/tool-xp2esd-create-modern-windows-xp-installer-v1-6-2.82935/
George King Plans to Move Media Center 2005 and Tablet PC Edition
to Server 2003, where I'm also responsible for both the Addon and UpdatePacks.
But I wonder if the program has problematic detection.
After using the Media Center addon
1.legacy update error says Windows XP Pro x64 (it is x86)
2.And I tested the 2nd round without UpdatePacks, more crashes with legacy update, Windows Explorer crashing.
3. The update still installed correctly.
Video of the test https://drive.google.com/file/d/1UqP7AoSoYFEWgmmltLC3c-8xNVHHe0l6/view?usp=share_link

I'm happy to provide you with a test file if you want.

I think it is the media center reg key that interfered the detection of WU

As a Windows XP x86 version 3790.3959 is weird for WU Server so it cannot determine what is that.
You may delete the key registry and check,the restore it.

The OS detection in Legacy Update 1.8 now uses exactly what Windows itself reports - specifically it pulls the string from WMI, which you can also see by running wmic os get caption. I don’t fully understand how this Media Center/Tablet PC on Server 2003 setup works, but given I can see Manage Your Server in the taskbar I’m assuming the OS caption might still be “Windows Server 2003, Standard Edition”.
Note - this doesn’t change which updates you’ll see. The OS label on the website is more of a cosmetic thing, unrelated to what updates Windows Update will deliver you.
Since this is a configuration that was never supposed to be possible originally, it makes sense that WU is confused about it and is probably returning the wrong set of updates. It’s hard to say if this is even a config that can be updated. Someone might need to try triaging which updates do and don’t work.

@XPMilkChan I noticed in the MDL thread that you've been working on an all-in-one update pack for 2003. Do you know which updates do and don't work in this config? There's nothing much Legacy Update can do here currently, because it's a config that was never intended to exist.
In future, when I roll out the custom Legacy Update WSUS server, I could tweak all incompatible 2003 updates to add an "is not Media Center" check. It would be a perfect place to provide your AIO packs too! (The custom server will be optional though, because it requires disabling security checks for Microsoft digital signatures.)

I see that it is also a result of the Media Center Update that George gave me to migrate from Windows XP.
My UpdatePacks are currently being completely revamped.
And George has not uploaded the new version I sent.
I to add as a friend in Discord. I have Update files for XP SP1 SP2 SP3 and also have specific hotfixes.
Server 2003 SP2 x86
I can only find some languages in the list of x64
x64
No missing files  request  form Microsoft I have all the files except TimeZone last update
which is required from UpdatePacks.


Idents do not work if  between open-paren and close-paren with no args

If I have the following line:
def main()

and I try to <CR> between the parens I get:
def main(
)

If there are arguments on the first line, it works as expected. For example:
def main(one,)

If the cursor is right after the comma, a <CR> results in:
def main(one,
         )

@cmcginty I'm not sure, but it's probably supposed to be this way, because if you want this particular feature you need to install something like delimitMate. Notice that if you would press O at the ) it would insert correct level of indentation.

Not sure myself, but closing as works-as-expected for now.


Use OpenAPI to set the default ProviderConfig

Description of your changes

This allows us to avoid an extra update call to the API server when new managed resources are created.
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

I've tested this by building provider-gcp against this commit, and ensuring that spec.providerConfigRef.name is defaulted even when the provider is not running.

Note that many managed resources that opt out of NewNameAsExternalName manually set NewDefaultProviderConfig as initializer. Though I don't think it's necessary to address that now.

I agree. I'm pretty sure for those managed resources the NewDefaultProviderConfig logic will just become a no-op because spec.providerConfigRef.name will never be unset, so I think we should just surface this deprecation in the release notes then let providers update at their leisure. I think our linter config complains when folks try to use a function with a // Deprecated comment, which will help bug folks to update.


Leftover h2 tag in markdown file

/_posts/2013-03-18-ultimate-rem-mixin.md contains a leftover h2 tag after "Mixin to the rescue!"

Thanks! :)

Fixed.


How can I just return the query as JSON?

I can't find this anywhere in the documentation. I just want to return the query as JSON, i.e. convert [[String: Node]] to JSON and return it.

Try
import JSON

let json = try JSON(node: queryResult)

Closing :)


[Test Failed] How to generate NOC chain according to TC-OPCREDS testcases?

Test issue(s)

TC-OPCREDS-3.1
TC-OPCREDS-3.4
TC-OPCREDS-3.5

Platform
raspi
Anything else?
According to TC-OPCREDS-3.1, TC-OPCREDS-3.4 and TC-OPCREDS-3.5, TH needs to obtain or generate the NOC chain using the CSR elements from the previous step. How should I get or generate NOC chain?
TH version: matter-th-fall2023

I am facing the same problem.
@Yinxq
If there are any developments, would you be willing to share the information here?


wide character support

Found this through TWiR :) . It looks like the API assumes each character is one cell wide, but this isn't true of Kanji and other ideographic languages. IIRC, some codepoints even change width based on the locale too (so punctuation is the same width as the rest of the characters, but I can't remember exactly). Any plan to support wide characters or did I skim the API too quickly? Thanks.

Thank you for reporting this! At the moment we do not have wide character support, however I plan on implementing this soon.

Going off the code written in termbox, I don't believe this will be too hard to implement. This should make in into rustty very soon.

Hello, I'm quite interested in this as well. If this is something you're still planning on doing, do you mind giving a ballpark estimate of when? Otherwise I might want to look into this myself, in which case it would be nice to hear any thoughts you've already had about it.

So I've taken a hard look at the issue of supporting wide characters and have encountered a fairly fundamental problem. rustty's abstraction over the underlying terminal relies on the assumption that each Cell occupies only the space which it represents; thus if one cell was to draw a wide character, it would end up drawing into the adjacent cell's space, which would in turn be overwritten when that adjacent cell is drawn.
I'm open to any and all suggestions as to how to handle this, but at the moment I'm drawing a blank.

Another thing is that there are zero-width characters (for things like ZWJ and friends, but also potentially some escape codes as well).
Might be worthwhile to look into how tmux represents these things (since it does support wide characters).


Add support for SAMD21E16

Sorry for the pollution, I didn't mean to open this PR here. These changes were intended to a fork first.


Add an option to keep only N deepspeed checkpoints

Very useful to avoid filling up the disk with hundred of GBs of checkpoints

@afiaka87 what do you think about this one ? I think you have similar space constraints
This PR is ready for review

Yes - I just got bit by not having this last night in fact. I'll give it a test run by end of day.

Did that work for you too @afiaka87 ? Let's merge it ?

Did that work for you too @afiaka87 ? Let's merge it ?

sorry still haven't had a chance to test - give me another few hours. Phil may not have time to check it today in any case.

answered your comments @afiaka87 tell me if you want me to change more things

@lucidrains Could we get this merged soon? Not sure if you've run with DeepSpeed yet; but indeed the problem of filling up your entire disk and breaking training has happened to me a few times now.


Severe Lag caused by Minekea (Its not my computer I promise)

Hello first I wanna say this is a really awesome mod! however while I was updating/rebuilding my modpack for mine and my friends 1.19 server the moment I added Minekea it makes any world (including single player) lag an unbearable amount, you cant even look around its just frozen and jumpy, its definitely not my computer either, and the other mods I have in my pack are the same ones I ran this mod with no problem previously so I'm not sure why suddenly its being strange. this issue only happens when Minekea is added so that's why I decided to contact you about it, sorry to bother you, just thought id let you know and see if there is a solution to this as id really love to continue using this mod in my modpack if possible.
this is the version I'm using
minekea-fabric-1.19-3.3.3.jar

New folder.zip
New folder (2).zip
heres the pics of the mods I'm using, for some reason it made me split them up into two files.


Alert admin when new version is available

It would be very useful for the UI to show some sort of notice when a cluster (or set of clusters) can be upgraded.

@kwwii what would be the UX solution for this?


Project: Exploring a Data Repo

@drphilmarshall made a great start at this here; however, I think we need to go a bit farther and document what is in each dataset rather than just listing the data sets.

@kadrlica is too kind: I'd say I made a minimal start, rather than a
great one :-) Definitely agree with his assessment, though - when I ran out
of time I was starting to think about writing a def that:

Displayed a big bold markdown section heading with the name of the dataset
Printed the README of the dataset directory (or a warning if there is
none...)
Listed the directory contents
Computed some vital statistics (no of visits, sky are covered, no of
objects, no of populated columns in the object table, etc etc
Plotted the sky coverage (that's what I needed Jim's DC2 notebook for)
and so on.
Then, the notebook could either loop over all datasets, and we'd have an
auto-inventory.

@rmorgan10 I just edited my comment above to better reflect my current thinking about what the Exploring A Data Repo and DataInventory notebooks should each do, see what you think. These notes are consistent with the current state of the explorer notebook - I'm currently trying to figure out how to use the butler to check for the existence of various data types.
Some more notes:

In the DataInventory notebook I started trying to download an external notebook that contains functions to plot tracts and patches on the sky and import it as a module - I got close, but not close enough. I think the next step might be to just copy the relevant code from the remote notebook, and provide correct attribution to its author. All that should then happen in the Exploring notebook instead, while the Data Inventory notebook is kept clean.
Jeff Carlin showed some useful-looking visualization code in his tutorial stub on Friday, browse the project branches and commit history to find it.
I think we can get quick code snippets for butler-querying repos from the DM team, either in #dm-newbies or #dm-pipelines - my guess is they have to do this kind of thing all the time.
Gregory Dubois-Felsmann has comments on this project - he's part of the team developing the gen 3 butler, and said he wanted to talk to us! I'll point him at this thread so he can comment.

Since @rmorgan10 showed the data inventory working pretty well at the Holiday Party, I'm going to close this issue and its sibling(?) #130, and issue an upgrade as a new project. Well done Rob! :-)


Version 3.5.0 causes command to fail

Issue
As expected, running the following via the command-line worked successfully using version 3.4.3:
node_modules/.bin/jsdoc -R README.md some-file.js -d ./docs
However, since the 3.5.0 patch the command above fails.
Error Log
The following error is logged utilizing version 3.5.0:
Note: path-to-project instances below represent a real path

path-to-project/node_modules/babylon/lib/index.js:164
bleConstructorReturn(this, _TokenType2.call(this, name, { beforeExpr, binop: p
^
SyntaxError: Unexpected token ,
at Module._compile (module.js:439:25)
at Object.Module._extensions..js (module.js:474:10)
at Module.load (module.js:356:32)
at Object.load (path-to-project/node_modules/requizzle/lib/loader.js:112:15)
at Requizzle.requizzle (path-to-project/node_modules/requizzle/lib/requizzle.js:90:24)
at infectProxy (path-to-project/node_modules/requizzle/lib/loader.js:82:28)
at Module.targetModule.require (path-to-project/node_modules/requizzle/lib/loader.js:101:11)
at require (module.js:380:17)
at Object. (path-to-project/node_modules/jsdoc/lib/jsdoc/src/astbuilder.js:4:15)
at Module._compile (module.js:456:26)
at Object.Module._extensions..js (module.js:474:10)
at Module.load (module.js:356:32)
at Object.load (path-to-project/node_modules/requizzle/lib/loader.js:112:15)
at Requizzle.requizzle (path-to-project/node_modules/requizzle/lib/requizzle.js:90:24)
at infectProxy (path-to-project/node_modules/requizzle/lib/loader.js:82:28)
at Module.targetModule.require (path-to-project/node_modules/requizzle/lib/loader.js:101:11)

Additional notes:
The error above is reported in the following system environments:

nodejs v0.10.28 - running on Mac OSX.
nodejs v0.12.3 - running on Windows Vista.

JSDoc now requires Node.js 4.2.0 or later. Please upgrade Node.js and try again.

Thanks @hegemonic for highlighting the Node.js 4.2.0 minimum requirement for JSDoc 3.5.x - just read the updated README.
Interestingly, several of my projects started failing today after running $ npm install as JSDoc was defined in  package.json as "jsdoc": "^3.4.3" (i.e. It utilizes npm's default save-prefix the caret ^). I think I'll retrospectively set them to to use the tilde (~) to avoid the issue reoccurring on node.js environments < 4.2.0.
I appreciate that the Minor release of JSDocs adds functionality in backwards-compatible manner. However, perhaps the greater Node.js dependency constitues a Major release for JSDocs instead of it being a Minor one. I guess this is a bit of a grey area regarding semver.
Once again, thanks for the prompt reply - much appreciated.

Right now, JSDoc doesn't really follow semver. I know this is a source of frustration for a lot of people, and I may change my policy on this in the future. For now, though, using the tilde for JSDoc is your safest bet.


Icon Request: Empty Notes & Empty Address Book

Hello there,
first of all, I am not sure whether this is the correct place for this kind of request. If not, please redirect me (or the request) to the person in charge.
What we are currently using in our application are the following icons:

What we would also like to have is an icon (or two) to indicate that there are no notes (or addresses).
Quick MS Paint sketch:

(note: because a feature request may take a while, are you aware that you can register custom icons? https://openui5.hana.ondemand.com/#docs/guide/21ea0ea94614480d9a910b2e93431291.html )

Hi @mschleeweiss , is this request still active after >5 years?
I'd assume no and close this ticket, but you can ping me in case it is.
However, I'd consider it unlikely that new icons are added as response to such a request and I gave the alternative to register custom icons above, so probably that's the best way forward...


docs(contributing): add directory structure again (since the previous…

This PR is the same as PR #194 , since the previous PR #194 failed due to conflicts with master branch.
Signed-off-by: Takuma TAKEUCHI<EMAIL_ADDRESS>

The fixed contents of this PR was already approved on PR #194 .
I'm sorry for trouble reviewers, but I appreciate if you approve this PR again.

Merged in #194 and therefore we can close this one which is equivalent.


TextPlugin removes source code HTML on save

I have had something similar occur in two separate projects.
In one, I was making modifications to the source HTML, like adding another style attribute to a tag, and my new attribute would be gone on save, leaving only the previous state of the tag.
In the current one, all tag attributes are getting purged on TextPlugin save.
In both these cases the TextPlugins were created within placeholder fields in the following manner:
block_text_plugin: TextPlugin = add_plugin(
    placeholder=subsection_placeholder,
    plugin_type=text_plugin,
    language="en-us",
    target=block_plugin,
    body=block["body"],
)

Where block['body'] contains HTML elements, including some
.
Before:

After:

djangocms-text-ckeditor 5.1.5

djangocms-text-ckeditor sanitizes any HTML before safe. You might need to configure the sanitizer. See here: https://github.com/django-cms/djangocms-text-ckeditor#configurable-sanitizer

It seems like it would certainly be the root cause, but I can't seem to get the settings to take effect:
I managed to get the Global setting to turn it off altogether to work, but I suspect I want a bit more precision.
Specifically providing "style" inside the settings.py did not preserve the style attributes.
So, now I'm trying the custom parser. I have it successfully hooked up, because I can see its print statements when made within the parse function
from djangocms_text_ckeditor.sanitizer import AllowTokenParser

Keep = bool
class ListAttributeParser(AllowTokenParser):
    def parse(self, attribute: str, val: str) -> Keep:
        if attribute == "style":
            print(val) # -->  list-style-type:lower-roman
            return True
        else: return False

I did add:
'basicEntities': False,
'entities': False,

Have you tried TEXT_ADDITIONAL_ATTRIBUTES = ("style",)?

Does it work? I must say, I am not an expert of htmllib.

I have not gotten the custom parser to work, no. Switching True/False with each other didn't seem to have an effect.


Bump sharp to v0.12

Fixes #70 #81
Requires version bump (1.3.1?) so that the generated project's package.json uses the updated version.

cc @jimmynicol
Sharp 0.13.1 is actually the current version now.
In fact, preinstall.sh is no longer needed on most Linux distributions when using sharp v0.12.0+. Just bumping up to the latest sharp version would save 2-3 steps from my deployment process.
From: https://raw.githubusercontent.com/lovell/sharp/master/preinstall.sh
# This script is no longer required on most
# 64-bit Linux systems when using sharp v0.12.0+

# See http://sharp.dimens.io/page/install#linux

# If you really need this script, it will attempt to
# globally install libvips if not already available.

Hello, if you're planning to upgrade from v0.11.x, please jump to at least v0.15.0 - see https://github.com/lovell/sharp/issues/449 for details.

Since the maintainer is unresponsive, we've decided to maintain our own fork at https://github.com/tripviss/image-resizer


Binary light OFF leads to unexpected behavior (fix or doc)

Hey there!
In my use case I have an aquarium consisting of a light and a "night light". Only the light is controlled via Home Assistant. I'm sure there are more cases where a user want's to give the fixed power for the on and off state of a switch/light?
The following config does not work:
  - platform: powercalc
    entity_id: light.aquarium_beleuchtung
    fixed:
      states_power:
        on: 20
        off: 2

This also does not yield the expected result:
    linear:
      max_power: 20
      min_power: 2

A combination of fixed and standby_power works as expected.
Imho the examples above should either be supported, or the docs should explicitly state that they are not. Wdyt?

This is not supported. You cannot use both modes.
You should use either linear or fixed.
Not sure what you are trying to implement exactly. How do you know this night light is actually on?
Is the night light always on when you turn on light.aquarium_beleuchtung in HA and off when you turn it off?

Sorry about the typo in the example. linear wasn't meant to be part of the first block.
The use case is quite simple. When a device (light or switch) is on, it naturally consumes energy. Some devices do in the off state as well. I suppose this is why you have already implemented standby_power.
Neither states_power nor linear support the "on" and "off" state. This feels counterintuitive.
Long story short: Either they should be supported for usability and to reduce the trial&error hassle; OR it should be document that this is indeed not supported.

Ah ok.
The off state is a "special" state for powercalc. states_power is only needed when you need to have advanced configuration on other states.
The power calculation will first check if the state is off and apply standby_power when this is the case, or 0 when you did not define standby_power. I agree this can be added as a note in the readme.
So in your case you can simply use:
- platform: powercalc
    entity_id: light.aquarium_beleuchtung
    standby_power: 2
    fixed:
      power: 20

Yes that's how I've solved it. Already mentioned that in the initial text :) To carify, I am not looking for a solution to my issue but wanted to raise this usability/consistency issue.

The off state is a "special" state for powercalc

Then my question is: Should it? Intuitively speaking I expect both usage examples from above to work. "on" and "off" are states just as any other, and on and off are two extremes of a linear device (which happens to have only two levels).

I understand this is a littebit confusing.
To "fix" this I need to overhaul a lot of code and restructure the logic.
The standby_power applies to any chosen calculation strategy and is handled before calculation is forwarded to a calculation strategy.
Have no plans to do so.
I have added an extra note to the readme directly under states_power to indicate off is a special case and you should use standby_power for this.

It's rather an edge case and does not justify a bigger overhaul. The readme addition is clear and sufficient. Thanks


Reproduce : BLEU is 10+ lower  than reported

GMA WMT15 German-to-English with Transformer-Base : delta=2.5
@Vily1998
I've followed the guidance and trained the model for about 100 epoch using following set:
delta=2.5
python train.py --ddp-backend=no_c10d ${data} --arch transformer --share-all-embeddings \
 --optimizer adam \
 --adam-betas '(0.9, 0.98)' \
 --clip-norm 0.0 \
 --lr 5e-4 \
 --lr-scheduler inverse_sqrt \
 --warmup-init-lr 1e-07 \
 --warmup-updates 4000 \
 --dropout 0.3 \
 --criterion label_smoothed_cross_entropy \
 --label-smoothing 0.1 \
 --left-pad-source False \
 --delta ${delta} \
 --save-dir ${modelfile} \
 --max-tokens 4096 --update-freq 2

than I evaluate it on the perl script you provided by single checkpoint and average the last 4 cp .
Below is my result :

CheckPointNum
BLEU

5
17.52

10
18.09

20
18.83

25
18.66

40
18.93

50
18.80

60
19.03

70
18.99

80
18.72

90
18.83

Average for 85,90,95,100
18.47

That is lower than reported (BLEU: 31.91) ,But I don't know the reason. So I am here to check something.
Questions

what should the translation reference look like ?
when I generate for BLEU , it would get an error :

It is not advisable to publish scores from multi-bleu.perl.  The scores depend on your tokenizer, which is unlikely to be reproducible from your paper or consistent across research groups.  Instead you should detokenize then use mteval-v14.pl, which has a standard tokenization.  Scores from multi-bleu.perl can still be used for internal purposes when you have a consistent tokenizer.

My translation and reference is just like below :
> head -5 $ref_dir
India and Japan prime ministers meet in Tokyo
India 's new prime minister , Nar@@ end@@ ra Mo@@ di , is meeting his Japanese counter@@ part , Shin@@ zo A@@ be , in Tokyo to discuss economic and security ties , on his first major foreign visit since winning May 's election .
Mr Mo@@ di is on a five @-@ day trip to Japan to strengthen economic ties with the third largest economy in the world .
High on the agenda are plans for greater nuclear co @-@ operation .
India is also re@@ porte@@ dly hoping for a deal on defence collaboration between the two nations .

>head -5 $pred.translation
The prime ministers of India and Japan met in Tokyo .
India 's new prime minister , Narendra Modi , meets in his first major foreign visit since his victory in May , his Japanese counterparts Shinzo Abe in Tokyo to discuss economic and security policy .
Mr Modi is on a five @-@ day trip to Japan to improve economic relations with the third largest economic nation in the world .
Plans for greater nuclear cooperation are high on the agenda .
Reports suggest that India also hopes for a treaty on defence cooperation between the two nations .

I'am not sure that whether it is right ?

How mant epoch it need to train ?
As you can see , I've trained it for about 100 epoches and the BLUE seems has converged .In your experiments at the time, how long /how many epoches did you train with this set of hyperparameters? Using the provided  set of hyperparameters , Is it possible to ensure that the effect is reproduced?

Does torch1.11 support  GMA ?
I noticed many torch operation is for torch old version , and there are many warning like below , I'am sorry that my torch version doesn't match ,I will try an old version later

UserWarning: floordiv is deprecated, and its behavior will change in a future version of pytorch. It currently rounds toward 0 (like the 'trunc' function NOT 'floor'). This results in incorrect rounding for negative values. To keep the current behavior, use torch.div(a, b, rounding_mode='trunc'), or for actual floor division, use torch.div(a, b, rounding_mode='floor').
beams_buf = indices_buf // vocab_size

/GMA/fairseq/optim/adam.py:173: UserWarning: This overload of add_ is deprecated:
add_(Number alpha, Tensor other)
Consider using one of the following signatures instead:
add_(Tensor other, *, Number alpha) (Triggered internally at  /opt/conda/conda-bld/pytorch_1603729138878/work/torch/csrc/utils/python_arg_parser.cpp:882.)
exp_avg.mul_(beta1).add_(1 - beta1, grad)

Ps: I've changed my torch version to the right one and ran it again , the result seemed same as before , so the torch version
may not affect the result.
Env info

NAME
Version

python
3.7

torch
1.7.0+cu10.2

fairseq
0.8.0

Thank you. I've nearly solve it.
The reason why the bleu is lower 10+ is that, When evaluating ,I did not using the detokenized and removed bpe for reference.
here is what I used before

head -5 pred.translation
The prime ministers of India and Japan met in Tokyo .
India 's new prime minister , Narendra Modi , meets in his first major foreign visit since his victory in May , his Japanese counterparts Shinzo Abe in Tokyo to discuss economic and security policy .
Mr Modi is on a five @-@ day trip to Japan to improve economic relations with the third largest economic nation in the world .
Plans for greater nuclear cooperation are high on the agenda .
Reports suggest that India also hopes for a treaty on defence cooperation between the two nations .

head -5 $ref_dir
India and Japan prime ministers meet in Tokyo
India 's new prime minister , Nar@@ end@@ ra Mo@@ di , is meeting his Japanese counter@@ part , Shin@@ zo A@@ be , in Tokyo to discuss economic and security ties , on his first major foreign visit since winning May 's election .
Mr Mo@@ di is on a five @-@ day trip to Japan to strengthen economic ties with the third largest economy in the world .
High on the agenda are plans for greater nuclear co @-@ operation .
India is also re@@ porte@@ dly hoping for a deal on defence collaboration between the two nations .

Actually I should remove bpe for the $ref_dir

head -5 pred.translation
The prime ministers of India and Japan met in Tokyo .
India 's new prime minister , Narendra Modi , meets with the US government 's Japanese counterpart in his first major foreign visit since his May election victory , in May to discuss economic and security policy relations .
Mr Modi is on a five @-@ day trip to Japan to improve economic relations with the third largest economic nation in the world .
Plans for greater nuclear cooperation are high on the agenda .
According to reports , India also hopes for a treaty on defence cooperation between the two nations .

head -5  $ref_dir
India and Japan prime ministers meet in Tokyo
India 's new prime minister , Narendra Modi , is meeting his Japanese counterpart , Shinzo Abe , in Tokyo to discuss economic and security ties , on his first major foreign visit since winning May 's election .
Mr Modi is on a five @-@ day trip to Japan to strengthen economic ties with the third largest economy in the world .
High on the agenda are plans for greater nuclear co @-@ operation .
India is also reportedly hoping for a deal on defence collaboration between the two nations .

I suggest that we should  complete the details of the data preparation and evaluation part .
All in all , The bleu result is reaching 29 now and it shows your code works ! Thank you !
@Vily1998


[Feature] Add possibility to pass initial admin credentials with env variables

📃 Description
This PR adds possibility to pass admin credentials for initial startup. Closes #1597
Docs repo PR: https://github.com/alexjustesen/speedtest-tracker-docs/pull/52
🪵 Changelog
➕ Added

Environment variables

✏ Changed

Initial migration in which the admin user is created
.env.example file

Wouldnt make Sense to accept and already hashed password?
Services like vaultwarden already do this because its way more Secure.

@DenuxPlays it would be overcomplicated, IMHO. By design, you (as the end user) expect the password provided in the .env file to work in the web view.
I agree that it’s more secure, but with the current implementation of user management and the process for creating the default admin password, it would feel a bit odd.

@twinpixxx couple of minor changes, also thank you for linking to the docs PR as well 👍


bundlerApp: don't execute postBuild twice

Motivation for this change
The hook was executed twice, also in the basicEnv.
Things done

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

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

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

maybe I'm stupid, but where is the second (not removed) execution of postBuild?

it's happening via basicEnv, which gets passed the args containing postBuild. So if we instead call it via basicEnv = (callPackage ../bundled-common {}) (removeAttrs args [ "postBuild" ]); we don't get a double execution.
I'm not 100% sure what the original intent was, but I know that at least for the homesick, the double postBuild resulted in a double execution, messing a bit with the executable it was patching.


Add district type to model for frontend

On the "My maps" page, we want to be able to show what type of districts are being drawn (State House, State Senate, Custom, etc.). This information is not currently available on the frontend.

@jfrankl @maurizi Where are we picturing putting this information on the project card? I initially made it
New Jersey · 40 General Assembly districts or Delaware · 5 custom districts
using chamber.name to fill in the type, as it doesn't look like there's a more general type like "State Senate" that each district is then mapped to; however, it looks like most names in chamber in production are very specific, like Florida Senate, Washington Legislative Districts, etc. Those situations look kind of funky in my initial plan, e.g.
Alabama · 35 Alabama State Senate districts
Another thought I had was doing
Alabama · 35 districts · Alabama State Senate
What do you think of either of those? Or should I try to add some kind of logic that uses chamber.name to display a more general type like "State Senate" here?

Here is the relevant design on Figma: https://www.figma.com/file/FYobD6UsJZoNxlpyxUOlR5/Application?node-id=0%3A111
I think the state name should be first, but not sure if the type or number should go next. I think keeping the number and type separate, as opposed to combining them into a sentence, makes it slightly easier to scan, which is important for this page.

Thanks @jfrankl. I can't view the Figma design--it says I don't have permission.

I'm also realizing I may have misinterpreted the scope of this issue--is this card just about making info we already have in the backend accessible to the frontend for this purpose or does it include adding a generic district_type column to the chamber table?

@ktohalloran the intent here isn't to add any new fields - we should just use the existing name field.
I think based on Jeff's wireframes, and given that the name for most state-level chambers starting w/ the state name, it would be best to do somthing like your third option:
Alabama · 35 districts · Alabama State Senate


Update gtest link to point to github

This addresses https://github.com/clMathLibraries/clBLAS/issues/182

Thanks!


Options are not necessary at Application level

Options like HOST PORT are only useful at server level, config them at app level makes no sense and won't work, so remove them from the default config file to avoid confusion.

Thanks for the PR, but these options are required for collecting metrics via Pulse.


[Feature Request] visualizer

https://dzolnai.medium.com/an-alternative-android-visualizer-7747df6c6df8

i cant have multiple fftListeners for a rotating crossfade player collection


Draft: added gpt4all

changes
TBD
why?

instead of using OpenAI's ChatGPT, let users use local gpt4all model. the spec for the gem itself is located here

I was literally just looking at doing this. Nice!

I will fix the gem to be compatible with 3.0, 3.1 and 3.2

let me check. i will rebase

This looks good to go. Will merge it after one other PR and bump the version.


Issues in IE 10 browser

I see following issues when used term.js in IE 10 browser.

Scroll bar does not work in IE 10 browser.

When ever space bar is used to load more information focus is being out from the terminal to enter next command or to click space bar again, focus has to be brought back to the terminal by clicking on the terminal using mouse click.

Proposed a solution: string 1416, instead of
if (parent) parent.appendChild(this.element);
should be
if (parent) {
  parent.appendChild(this.element);
  this.element.focus();
}


Add Anti-Patterns Non Issue NPearson

Did you read the instructions in the README?

Y

Have you read the instructions in CONTRIBUTING?

Y

Have you written this article by yourself? Have you taken inspiration from anywhere and then written it in your words?

Yes the article was written by myself. I did take inspiration from multiple references and then wrote the article in my own words.
References:
http://wiki.c2.com/?AntiPattern
http://wiki.c2.com/?AntiPatternsCatalog
https://sourcemaking.com/antipatterns/software-development-antipatterns
https://sahandsaba.com/nine-anti-patterns-every-programmer-should-be-aware-of-with-examples.html

Nice article @NathanPearson . A few things, however:

Put the article in UX_design
Please carry out proper formatting of the document. I am not able to check if it is plagiarised or not. Perhaps this cheatsheet might help.

Sure thing. I assumed a folder may be appropriate as others could elaborate on individual antipatterns. I'll fix the formatting issue as well.

File moved and the formatting should now be acceptable. If there are any other issues please let me know.

Merging this.


Can't open chrome as mobile

I've already tried using SetMobileEmulation and changing setplatform to android but nothing works. SetEmulation nothing happens (idk if i did something wrong) and SetPlataform Android web driver doesn't open.
Idk if it's a problem or if i'm just dumb enoght to don't realize how to use this version of selenium hahah.
Any ideas?

@SrMilton thank you very much for the feedback.
At the moment, capability mobileEmulation is poorly described, I will add the necessary parameters for working with mobile emulation from the documentation of chromedriver today and add a list of ready-made device list as in google chrome.

@SrMilton, I will not forget to make examples for easy understanding and gtests before building

@SrMilton, You can see the changes here https://github.com/GermanAizek/webdriverxx/commit/5d94d4fac0334b0ee7a49b7fd2ad7e5eec279a34

@SrMilton,
This fixed and supplemented, I am waiting for your response about testing new feature.

@GermanAizek Wow fast af hahah.
Btw, chrome still starting as PC mode.
Code:
  chrome::MobileEmulation me;     me.SetdeviceName(chrome::device::Get("Galaxy Note 3"));     WebDriver gc = Start(Chrome().SetMobileEmulation(me));     gc.Navigate("https://www.google.com.br/");
Result:

What i'm doing wrong?

Very strange behavior chromedriver it cast object JsonObject to double.

The mobile_emulation capability was added to ChromeDriver in 2.11

Do you have a higher version?

@SrMilton,
How the temporary solution should work directly jsonobject generation for chromedriver.
Try my option. I will edit this function definitively. When I'm done, I'll notify you.
JsonObject me; me.Set("deviceName", "Galaxy Note 3"); auto cap = Capabilities().Set("mobileEmulation", me); WebDriver gc = Start(Chrome(), cap); gc.Navigate("https://www.google.com.br/");

@SrMilton,
If you need  parameters for mobile emulation, the code below:
Google Nexus 5
JsonObject dm;
dm.Set("width", 360);
dm.Set("height", 640);
dm.Set("pixelRatio", 3.0);
dm.Set("touch", true);

JsonObject me;
me.Set("userAgent", "Mozilla/5.0 (Linux; Android 4.2.1; en-us; Nexus 5 Build/JOP40D) AppleWebKit/535.19 (KHTML, like Gecko) Chrome/18.0.1025.166 Mobile Safari/535.19");
me.Set("deviceMetrics", dm);

auto cap = Capabilities().Set("mobileEmulation", me);
WebDriver gc = Start(Chrome(), cap);
gc.Navigate("https://www.google.com.br/");

@GermanAizek I'm using the last stable version of ChromeDriver, "ChromeDriver 89.0.4389.23".

JsonObject me;
me.Set("deviceName", "Nexus 5");
auto cap = Capabilities().Set("mobileEmulation", me);
WebDriver gc = Start(Chrome(), cap);
gc.Navigate("https://www.google.com.br/");

I was doing exaclty this before open this issue, but nothing happens, btw we can't use "Start(Chrome(), cap);" this way (at least for me doesn't work your method), the correct way is:

JsonObject me;
me.Set("deviceName", "Nexus 5");
Capabilities cap; //Change here
cap.Set("mobileEmulation", me); //And here
WebDriver gc = Start(Chrome(), cap);
gc.Navigate("https://www.google.com.br/");

But nevermind, still the same, look:

And your last solution defining parameters don't work, absolutely nothing happens in Selenium, my program just open and close.
My configs:
Windows 10 64 bits
Google Chrome 89.0.4389.90 (Official version) 64 bits
ChromeDriver 89.0.4389.23 x32
Visual Studio 2019 (v142)
Windows SDK Version: 10.0.18362.0
MFC in a Static Library
Unicode Character Set
Compiling Release x64
If you have any solution just send to me and i'll test and give to you a feedback asap. Thanks!

@SrMilton,
Have you tried the сommits to roll back?

@GermanAizek i've just rolled back to the version before your last update and still absolutely the same. This is the error i was talking about:

@SrMilton,
Replace Capabilities with JsonObject.

@GermanAizek i hope this is what you meant

@SrMilton ,
const char* url = "http://localhost:4444/wd/hub/";

auto ff = Start(Firefox(), url);

// or
auto ff = Start(Firefox(), Capabilities() /* required */, url);

Did you try specifying the url to the selenium server?

@GermanAizek yes, and i use port 4445, i've changed the client.h to "const char *const kDefaultWebDriverUrl = "http://localhost:4445/wd/hub/";" and EVERYTHING works perfectly, i have more than 400 lines of code using webdriverxx and everything works fine, but i can't open chrome as mobile, this is the last thing i need to finish my project hahah.

@SrMilton,
I understand you, but initially no api was added to webdriverxx to work with MobileEmulation in chrome.
For many browsers, Capabilities are not fully made, too many browsers and for some it is difficult to find documentation.
I'll help you as much as I can. 👍

@GermanAizek Thanks man! Let me ask you something: Are your browser opening as Mobile using those methods?
I really appreciate your help dude! btw sorry if i'm texting something wrong, english is not my major language hahaha.

@SrMilton,
The methods should work, but I didn't check on Windows.

@GermanAizek I'm using AutoIt to press CTRL+SHIFT+I and CTRL+SHIFT+M to open mobile mode while you don't find a solution to this crazy bug hahah. ofc this is the worst way to do what i want, but its good for now

@SrMilton, Sounds brilliant. Genius.

@SrMilton,
You could do without AutoIt, functions that I recently added you can call the mobile mode
Element::KeyDown(), Element::KeyUp(), Element::SendKeys()

@GermanAizek alright, i'll implement this, Thanks!!
Another question: How can i start chrome with some extesion? I know in java we can simple use ChromeOptions(), but idk how to do this in webdriverxx..
Code in java:

ChromeOptions options = new ChromeOptions();
options.addArguments(“–load-extension=” + pathToExtension);
driver = new ChromeDriver(options);

@SrMilton,
Everything is very simple.
Example:
std::vector<std::string> extensions {"/path/to/extension0.crx", "/path/to/extension1.crx", "/path/to/extension2.crx"};
WebDriver ff = Start(Chrome().SetExtensions(extensions));

@GermanAizek guess what..

I've created the .crx of the extension and nothing happens. I need the extension with a specific config. What i'm doing wrong?

@SrMilton,
Have you been able to solve the path problem?

@GermanAizek not yet, any ideas?

@SrMilton,
There is an idea to edit so that the character does not change / to utf8 code

@GermanAizek this is exaclty what i'm trying to figure out..

@SrMilton,
There is an idea to edit so that the character does not change / to utf8 code

@SrMilton,
I got free time and figured out how to make a new function to convert from sequence to utf-8.
In 30 minutes, I'll commit a new feature for testing.

@SrMilton,
Look at the commit to understand the algorithm. This should work.
https://github.com/GermanAizek/webdriverxx/commit/664ecba8a5dde30cf55045bd437226fd95aa131b
Example:
#include "detail/to_string.h"

std::vector<std::string> extensions {
    detail::ToUtf8("/path/to/extension0.crx"),
    detail::ToUtf8("/path/to/extension1.crx"),
    detail::ToUtf8("/path/to/extension2.crx")
};

WebDriver ff = Start(Chrome().SetExtensions(extensions));

@GermanAizek I liked your idea, but not worked hahah.

i += 5;

The program still writing the u002f, but this time your function worked and...

Yes, the path is "C:\extension.crx", and after your function we have "C:\u002fsion.crx" hahahhah
Ps: We need to use \ instead of / in your function.

@GermanAizek alright, i give up hahahahahh. The best i found is set the path to "C:\nextension.crx", selenium print the \n, then i renamed the extension name to nextension.crx, the output in selenium is perfect but this trick doesn't work.. So..We can create a chrome profile with the extension preseted, but how can i select a profile in webdriver, i know in java is possible with:

options.addArguments(“user-data-dir=C:\\Selenium\\BrowserProfile”);

@SrMilton,
I understood the problem. Conversion of UTF-8 to Unicode occurs after JSON is generated using the api, because JSON is required UTF-8 at RFC8259.
Chromedriver began to accept arguments in Unicode. This problem is open, how to pass your string "C:\extens.crx" if the transmission method occurs using JSON.
Unless I am mistaken.

@SrMilton,
You tried a popular way to replace \ with \\ ?

@SrMilton,
Does changing \ to \\ not help?

@SrMilton,
I tested on Linux and I have the result of such
Code:
#include <webdriverxx.h>

using namespace webdriverxx;

int main()
{
	std::vector<std::string> extensions {
		"C:\\adblock.crx"
	};

	WebDriver ff = Start(Chrome().SetExtensions(extensions));

	getchar();

	return 0;
}

@SrMilton,
The example shown in Java is very easy to create.
Example:
WebDriver gc = Start(Chrome().SetArgs({"user-data-dir=C:\\Selenium\\BrowserProfile"}));

So..We can create a chrome profile with the extension preseted, but how can i select a profile in webdriver, i know in java is possible with:

options.addArguments(“user-data-dir=C:\\Selenium\\BrowserProfile”);

@SrMilton,
The example shown in Java is very easy to create.
Example:
WebDriver gc = Start(Chrome().SetArgs({"user-data-dir=C:\\Selenium\\BrowserProfile"}));

@SrMilton,
Does changing \ to \\ not help?

Aru joking with me? hahaha :p Yes, this is the FIRST thing i did, when i do \ the output is c:\u002fextension.crx, when i do \ the output is c:\extension.crx. This is why i'm suffering with this

@GermanAizek omg man, what is going on?? Believe in me, this is the LAST thing i need to do to end this project hahahahahahha
The first one i used \\ and the second one i used /

It's only can be something about configuration in visual studio.. any ideas? also Thanks for your help!

@SrMilton,
I understood the problem. Conversion of UTF-8 to Unicode occurs after JSON is generated using the api, because JSON is required UTF-8 at RFC8259.
Chromedriver began to accept arguments in Unicode. This problem is open, how to pass your string "C:\extens.crx" if the transmission method occurs using JSON.
Unless I am mistaken.

i've just saw this, i'll try to study this case to solve this problem.

@SrMilton,
Under Visual Studio, you can choose between several options Multi-Bytes, Unicode, and a third.
You will find that property in the project properties box,
right click on your project in the "Solution Explorer", select "Properties" (at the bottom), in the option tree go to "general", you will something like:

Select Unicode, Apply and Close.
Now rebuild the project, and it will work.
See more docs:
https://docs.microsoft.com/en-us/cpp/atl-mfc-shared/unicode-and-multibyte-character-set-mbcs-support?view=msvc-160&viewFallbackFrom=vs-2019

@GermanAizek I have bad new to you.. this was one of the first things i checked when i faced this bug..

@GermanAizek I'm implementing the API of the extension, i just need to get a value from the page. Using the DevTools i can use document.getElementsByClassName("test1").valueOf().item("").getAttribute("data-valuekey") this prints something like "aBCsScxdsf548dsas48df5d-" (this is what i want).
How can i get this response? I've already tried something like:

auto title = gc.Eval<std::string>("return document.getElementsByClassName(\"test1\").valueOf().item(\"\").getAttribute(\"data - >valuekey\")");

but it's prob wrong. Do u have any tip?
Ps: i didn't know the extension has an API, sorry hahahah. btw i know i'm asking too much to you and THANKS for answering.

I'm closing this issue because it's "solved". Feel free to open again if you want. I'm finishing the project rn, Thanks @GermanAizek .

@SrMilton,
Hi, there's a way with extensions.
Instead of specifying base64 string, webdriver can decrypt it.


Ignore PID files created by lisa_shell

Since you are at it, we could probably add vagrant too, which appears as untracked file.

@mdigiorgio Ah yes, will add that too. Didn't do it initially because I wasn't sure if it was an artefact of my own setup.

Thanks, I guess it can be merged.


Update pages with latest project structure

We have 4 GitHub repos. Use https://github.com/spring-cloud/spring-cloud-azure to list links of 4 repos.
Add stackoverflow link.
List features for Spring Cloud, Spring Boot and Spring Data to avoid confusion.
Update release versions.
Update links for get-started and reference doc.

@xscript @erickson-doug @bmitchell287 Could you please help review the updates?
I am still working on jekyll to see how to review those pages locally. Please go ahead review the content, the Azure terminology and format. Thanks.

Hi @kyliel - @bmitchell287 has passed this one to me. I've reviewed and flagged a few issues. Other than the comments above, this looks fine to me.

Hi @kyliel - @bmitchell287 has passed this one to me. I've reviewed and flagged a few issues. Other than the comments above, this looks fine to me.


Improve search component focus behavior

Is your feature request related to a problem? Please describe.
It's highly likely user would want to search over available entities.
Describe the solution you'd like
Make it available to enter search query text right after the page is loaded (focus on search component by default).

@arkid15r, Please assign this to me!
Thank You!


now.day() returns zero (0)

So I was trying to build one clock with my RTC and a 16x2 LCD.
When I call now.day() from datetime, it returns 0. So today, 1st May 2021, is 0th May, 2021.
Last I checked it was April 27th, and it was returning April 27th.

Arduino board: Tried both Arduino Uno and Nano

Arduino IDE version (found in Arduino -> About Arduino menu):  **INSERT ARDUINO
1.8.13

Timezone
IST (+05:30)

Time
May 1, 2021

List the steps to reproduce the problem below (if possible attach a sketch or
copy the sketch code in too):

#pragma GCC optimize("Os")

#define ROW 2
#define COL 16
#define SPLITTER_BLINK_DELAY 500

#include <Wire.h>
#include <RTClib.h>
#include <LiquidCrystal.h>

RTC_DS1307 rtc;
LiquidCrystal lcd(12, 11, 5, 4, 3, 2) ;

char daysOfTheWeek[7][4] = {
	"Mon", "Tue", "Wed", "Thu", "Fri", "Sat", "Sun"
} ;

char Months[12][4] = {
	"Jan", "Feb", "Mar", "Apr", "May", "Jun", "Jul", "Aug", "Sep", "Oct", "Nov", "Dec"
} ;

unsigned char centre(char *str, unsigned char len) {
	unsigned char str_len = strlen(str) ;
	if(str_len >= len) return str_len ;
	return (len - str_len) / 2 ;
}

void setup() {
	Serial.begin(57600) ;
	lcd.begin(COL, ROW) ;

	while(!rtc.begin()) {
		char *message = (char *)"No RTC Found!" ;
		lcd.setCursor(centre(message, COL), 0) ;
		lcd.print(message) ;
	}

	while(!rtc.isrunning()) {
		char *message = (char *)"RTC Not Running" ;
		lcd.setCursor(centre(message, COL), 0) ;
		lcd.print(message) ;
		rtc.adjust(DateTime(F(__DATE__), F(__TIME__))) ;
	}
}

char str[100] ;
char *splitter ;
unsigned char secondNow, lastSecond, splitterUnset ;
unsigned long blinkSplitterDelay = millis() + SPLITTER_BLINK_DELAY ;
unsigned long currT ;

void loop() {
	DateTime now = rtc.now() ;
	currT = millis() ;

	secondNow = now.second() ;

	if (lastSecond != secondNow) {
		lastSecond = secondNow ;
		splitter = (char *)":" ;
		splitterUnset = 1 ;
		blinkSplitterDelay = currT + SPLITTER_BLINK_DELAY ;
	}

	if (splitterUnset && currT > blinkSplitterDelay) {
		splitter = (char *)" " ;
		splitterUnset = 0 ;
	}

	sprintf(str, "%3s, %3s %02d", daysOfTheWeek[now.dayOfTheWeek() - 1], Months[now.month() - 1], now.day() + 1) ;
	lcd.setCursor(centre(str, COL), 0) ;
	lcd.print(str) ;

	sprintf(str, "%02d%s%02d%s%02d", now.hour(), splitter, now.minute(), splitter, secondNow) ;
	lcd.setCursor(centre(str, COL), 1) ;
	lcd.print(str) ;
}

Now I am using LCD, but I can omit all the LCD related code, just Serial.print now.day(), and it's 0.

Try using the string example

The string example is totally wrong:

Well, so there was a problem with the RTC itself. I opened the battery, inserted that again, uploaded DS1307RTC > setTime again, and it's fine now!


Optimize the project using news tips from .NET 9.0

Is your feature request related to a new game or help the app to be better? Please describe the feature.
No response

Hello, I tried to solve the issue.
This is what I did:
Implement a new game mode called Continent Challenge, where users guess countries from a specific continent.
You can review changes in this commit: https://github.com/lattaai13/wavepulse-atlas-131/commit/c97b8378e729d86c63a09a1bf6a95c07754eb468.

[!CAUTION]
Disclaimer: The commit was created by Latta AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

This issue was tried to solve for free by Latta AI - https://latta.ai/ourmission
If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


Clean up ci-runner dockerfiles & upgrade to nodejs 14.19.1

Signed-off-by: Peter Zhu<EMAIL_ADDRESS>Description
Clean up ci-runner dockerfiles & upgrade to nodejs 14.19.1.

Remove AL2 runner, as it is not used and deprecated.
Change 14.18.2 to 14.19.1 nodejs version per this: https://github.com/opensearch-project/OpenSearch-Dashboards/pull/1487
Removed nodejs and cypress from the opensearch build runner as it is not needed anymore.

Issues Resolved
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2030 (fa14a5e) into main (b9f83df) will increase coverage by 5.76%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #2030      +/-   ##
=============================================
+ Coverage     94.23%   100.00%   +5.76%
=============================================
  Files           198         6     -192
  Lines          3902       105    -3797
  Branches         29        19      -10
=============================================
- Hits           3677       105    -3572
+ Misses          219         0     -219
+ Partials          6         0       -6

Impacted Files
Coverage Δ

tests/jenkins/jobs/BuildManifest_Jenkinsfile

...est_workflow/test_result/test_component_results.py

...sts/jenkins/jobs/DetectTestDockerAgent_Jenkinsfile

src/system/console.py

src/assemble_workflow/bundle_recorder.py

tests/jenkins/jobs/DownloadFromS3_Jenkinsfile

.../ci_workflow/ci_check_gradle_properties_version.py

src/jenkins/InputManifest.groovy

src/system/os.py

src/test_workflow/bwc_test/bwc_test_runner.py

... and 182 more

Continue to review full report at Codecov.

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

New images:
(Will show up once the build is complete)

https://hub.docker.com/r/opensearchstaging/ci-runner/tags?page=1&ordering=last_updated&name=ci-runner-centos7-opensearch-build-v2
https://hub.docker.com/r/opensearchstaging/ci-runner/tags?page=1&ordering=last_updated&name=ci-runner-centos7-opensearch-dashboards-build-v2
https://hub.docker.com/r/opensearchstaging/ci-runner/tags?page=1&ordering=last_updated&name=ci-runner-rockylinux8-opensearch-dashboards-integtest-v2

All images now use the number v2.

I can perform some sanity check on the new images when they are ready. @peterzhuamazon

Everything is built right now all the images above updated to v2.
Cypress on arm64 can now automatically compile during docker image build time.
Thanks.

Image too big change to multi-stage now.
opensearchstaging/ci-runner               ci-runner-rockylinux8-opensearch-dashboards-integtest-v2   daf9ba3a8c03   43 minutes ago   14.5GB

Size fixed now:
[ci-runner-rockylinux8-opensearch-dashboards-integtest-v2](https://hub.docker.com/layers/ci-runner/opensearchstaging/ci-runner/ci-runner-rockylinux8-opensearch-dashboards-integtest-v2/images/sha256-1054f5782bbf282d9373dba556c943fb7aa9dd2b559d0d69efaee0a30c89b5f3?context=explore)
Last pushed 7 minutes ago by [opensearchstaging](https://hub.docker.com/u/opensearchstaging)
Digest
OS/ARCH
Compressed Size
1054f5782bbf
linux/amd64
1.05 GB
4d3830db21d0
linux/arm64
1.04 GB

[opensearch@c28e1e31fe3e ~]$ cypress run
It looks like this is your first time using Cypress: 9.5.4

✔  Verified Cypress! /usr/share/opensearch/.cache/Cypress/9.5.4/Cypress

Opening Cypress...
[1773:0423/103729.554624:ERROR:gpu_init.cc(453)] Passthrough is not supported, GL is swiftshader, ANGLE is
Could not find a Cypress configuration file in this folder: /usr/share/opensearch
[opensearch@c28e1e31fe3e ~]$ uname -a
Linux c28e1e31fe3e 4.14.273-207.502.amzn2.aarch64 #1 SMP Wed Mar 30 01:13:12 UTC 2022 aarch64 aarch64 aarch64 GNU/Linux

Ignore failure as there is no cypress test present.
This proves that on aarch64 the compiled cypress runs correctly.
Thanks.

Rebuilt with cmake installed so no module compilation failure, still the sandbox test seems error out due to we are building a docker image inside a docker, but binary built.

@tianleh please review.
Thanks.

Hey @peterzhuamazon I see ci-runner is mentioned twice in the image opensearchstaging/ci-runner:ci-runner-centos7-opensearch-build-v2, can we have the image like opensearchstaging/ci-runner-centos7:2.0, would this make simpler ?
Thank you

Hey @peterzhuamazon I see ci-runner is mentioned twice in the image opensearchstaging/ci-runner:ci-runner-centos7-opensearch-build-v2, can we have the image like opensearchstaging/centos7-ci-runner:2.0, would this make simpler ? Thank you

It is always like that and I understand where you come with that question.
It is not ideal but we have been putting all the ci runner images in the same repo without creating multiple other repos.
In the next iteration we can try something else, at least standardize the image names, which is still not there yet.
Let's not do too much change now as release is next week and a lot of coding is depending on these names (yes there are code dependencies on these names).
Plus even with the help of Jenkins these images build in 3 hours due to multiple compilation on the fly through binary translation (which is by design to do so). So I would not start that now.
There is an issue in this repo already pointing that out.
Thanks.

Hey @peterzhuamazon if you have issue for enhancement for ci images can you add this example as well? https://github.com/opensearch-project/opensearch-ci/pull/107
so that we don't have to wait for someone to build the image, whenever a change is pushed to this repo, the ci image can be built automatically.
Thanks!

Hey @peterzhuamazon if you have issue for enhancement for ci images can you add this example as well? opensearch-project/opensearch-ci#107 so that we don't have to wait for someone to build the image, whenever a change is pushed to this repo, the ci image can be built automatically. Thanks!

Not necessarily the same as we have other script to setup env before dockerfile being built, thus cannot directly run docker build from github actions. Please the compiling is intense and it would crash if we do it on the 2 core machine of github runners. It would also take more than 6 hours even if it runs.


analyzer: Prepare SQL queries

SUMMARY
Rather than build SQL queries inline, we should factor them out to live in a single, easily auditable location. Related: #40
ISSUE TYPE

Feature Idea

COMPONENT NAME
analyzer

Example of injection vulnerability found by @lukaw3d


util: Add MemBlockBuilder class to help manage blocks of memory for safe memcpy

Description: Split out from #8779 per suggestion from @htuch -- adds a class to help manage memory blocks, to make it easier to reason about the correctness of calls to memcpy.
Risk Level: low -- this PR just adds/tests a new utility class
Testing: just the new test
Docs Changes: n/a
Release Notes: n/a

@jmarantz looks like legit CI failures on coverage related to this PR, want to take a look?

woo-hoo: this squirrels by the coverage issues matching the expected error message; ptal.

@envoyproxy/maintainers can I have a non-googler review? This is should be relatively simple, as this adds a new class & tests but does not use it yet.


Failing to run the calibrate

Hello,
I am trying to run an int8-quantized model with clip node. But after I run the calibrate.py by the code below, it seems that I get a brken quantized model.
python calibrate.py --model_path V4.onnx  --dataset_path ../../pb --output_model_path model_quantized.onnx --static=True --data_preprocess=None --mode=int8
When I try to execute the systolic_runner example minimaly modified on spike, I keep getting the following error:

Here comes the netron view of the brken quantized model. It seems that I lose all of my clip nodes.

Can you share your model? I can try to replicate this error

Can you share your model? I can try to replicate this error

Thanks a lot for your reply but I have already solved this error. The operator——Relu6 was used in my model, which was regarded as Relu+clip. However, the quantize tool couldn't handle nodes of clip correctly, causing the broken nodes eventually.

Thanks for the update!


core dump

trying to decompress or count lines of compressed tab separated files
~/git/pugz/gunzip -l file.tsv.gz
terminate called after throwing an instance of 'gzip_error'
what():  INVALID_LITERAL
Aborted (core dumped)

the problem is probably due to this (found in README)
Only text files with ASCII characters in the range ['\t', '~'] are supported.

hi yes, is it a ASCII file?


Crashing during activity burst

Looks like there's a race condition during a burst of activity. Thinking maybe opening/closing clients is the issue

Hopefully will see better results after https://github.com/tparker00/bws-cache/pull/7

https://github.com/tparker00/bws-cache/releases/tag/v<IP_ADDRESS>-devel
Has helped a lot, still running into what looks like races with locking but it's much better.

Latest development version has been stable even during frequent updates


Allow setting scalar list default values

Problem

I'm currently migrating a schema which has a non-empty string default. My schema looks like this:
model Person {
  emails String[]
}

This is how it is generated from prisma introspect, decided from #1110. However, when running prisma migrate dev, the generated migration then attempts to remove the default, generating:
ALTER TABLE "Person" ALTER COLUMN "emails" DROP DEFAULT;

Removing this and attempting to add a default using dbgenerated leads to the following error:
Error parsing attribute "@default": Cannot set a default value on list field.

Suggested solution

As far as I can tell, this error is entirely artificial at this point because the release of 2.16.0 introduced a workable syntax for scalar lists via dbgenerated. I'd like to propose simply removing this check:
https://github.com/prisma/prisma-engines/blob/215319b1f112e9e5b5a1f2d2b0553982d2669776/libs/datamodel/core/src/transform/ast_to_dml/db/attributes.rs#L320-L322
Alternatives

None considered, as I haven't found any drawbacks to this proposal.
Additional context

Thanks for bringing this up, I agree we should finally decide on how we can support defaults for scalar lists.

Can you also try adding @default(dbgenerated()) which might prevent the faulty migration.

Can you also try adding @default(dbgenerated()) which might prevent the faulty migration? (We might be actually blocking this no harm in trying)
model Person {
  emails String[] @default(dbgenerated())
}

No dice, same error 😞

This one is a bit painful...
We're running:
version "3.8.1"

Updating migration.sql to include the default doesn't help either are seems like any prisma migrate run also generates a new migration to drop the default...
Any other work arounds?

Last sentence https://www.prisma.io/docs/concepts/components/prisma-client/working-with-fields/working-with-scalar-lists-arrays#null-values-in-arrays states:

To work around this issue, you can set the default value of array fields to {} at a database level

But have been unable to use migrate without it reseting, even if I alter the table in PG itself

Open an issue about migrations not picking that up correct please and provide all the information the issue template asks for @nicklloyd . If that is the case, it is not intended.

Internal note:
There are some more similar issues collected in this column in a project board: https://github.com/orgs/prisma/projects/64#column-17788443


AttributeError: 'ARMPipelineClient' object has no attribute 'send'

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az hcs get-config Extension Name: hcs. Version: 0.4.4.
Errors:
The command failed with an unexpected error. Here is the traceback:
'ARMPipelineClient' object has no attribute 'send'
Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/home/will/.azure/cliextensions/hcs/azext_hcs/custom.py", line 404, in get_config
    cmd, resource_group_name, name, "config")
  File "/home/will/.azure/cliextensions/hcs/azext_hcs/custom.py", line 305, in custom_provider_request
    response = client.send(request)
AttributeError: 'ARMPipelineClient' object has no attribute 'send'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az hcs get-config --resource-group {} --name {}

Expected Behavior
Environment Summary
Linux-4.4.0-19041-Microsoft-x86_64-with-debian-bullseye-sid
Python 3.6.10
Installer: DEB

azure-cli 2.25.0

Extensions:
hcs 0.4.4

Additional Context

route to service team

@pereirawill, could you tell us where you installed this extension from as this extension is not hosted in our public extension repo https://github.com/Azure/azure-cli-extensions? If you installed from other places, please contact proper service team as actually we don't know which team owns this extension.

It's a hashicorp extension for consul managed service.
I've tried to use the following extensions:
az extension add
--source https://releases.hashicorp.com/hcs/0.4.4/hcs-0.4.4-py2.py3-none-any.whl
az extension add
--source https://releases.hashicorp.com/hcs/0.4.9/hcs-0.4.9-py2.py3-none-any.whl

@pereirawill, could you please contact hashicorp for help using the email below as they own the extension?
Summary: Microsoft Azure Command-Line Tools HCS Extension
Home-page: https://github.com/hashicorp/cloud-hcs-cli
Author: HashiCorp Cloud
Author-email<EMAIL_ADDRESS>

thanks @jsntcy

thanks @jsntcy


[API] Support get all labels of the special pods

Feature Request
Is your feature request related to a problem? Please describe:

We can get all labels of the pods and let users select labels instead of filling one. I think this way is more friendly for users.
Describe the feature you'd like:

URL: /api/common/labels?ns=xxx,xxx

return 200:
{
    "app": "xxx",
    "xxx": "xxxx",
}

return 404:
Not Found

return 500 xxx

Describe alternatives you've considered:

Teachability, Documentation, Adoption, Migration Strategy:

Why the URL is URL: /api/common/pods?ns=xxx,xxx? Is this the same API as /common/pods?

Why the URL is URL: /api/common/pods?ns=xxx,xxx? Is this the same API as /common/pods?

Sorry, A typo, URL: /api/common/labels?ns=xxx,xxx

Why the URL is URL: /api/common/pods?ns=xxx,xxx? Is this the same API as /common/pods?

Sorry, A typo, URL: /api/common/labels?ns=xxx,xxx

Ack. But it is also a good idea to put the labels in the /common/pods API, since labels are just pods metadata.