Rewrite cert sync to use comm, not diff (#146)

Rewrite the cert sync detection:

overly reliant on magic paths (/tmp/existing-certs, etc)
not cleaning up temp files
empty files might be causing diff/grep/awk pipeline to error out (#146)

Changes:

tempfile module used & cleaning up the generated tempfiles
client list still written to disk, but use the ansible internal sort first so there's no additional call
task file name changed from common.yml to more specific cert_sync_detection.yml
simplified the existing cert check pipeline with globs

Decent cleanup of the cert sync & revocation tasks.
Ran fine on a new centos 8 VM.
...
TASK [kyl191.openvpn : [cert sync] Create temporary file for existing certs] ***********************************************
changed: [<IP_ADDRESS>]
TASK [kyl191.openvpn : [cert sync] Create temporary file for expected certs] ***********************************************
changed: [<IP_ADDRESS>]
TASK [kyl191.openvpn : [cert sync] Write expected client list to temp file for comparison with existing certs] *************
changed: [<IP_ADDRESS>]
TASK [kyl191.openvpn : [cert sync] Get existing certs] *********************************************************************
changed: [<IP_ADDRESS>]
TASK [kyl191.openvpn : [cert sync] Find certs that exist but aren't supposed to (on disk, but not in client list)] *********
changed: [<IP_ADDRESS>]
TASK [kyl191.openvpn : [cert sync] Debug: Certs to revoke (skipped if none)] ***********************************************
ok: [<IP_ADDRESS>] => {
    "changed": false,
    "msg": "Will revoke additional certs: x230"
}
TASK [kyl191.openvpn : [cert sync] Cleanup temp files] *********************************************************************
changed: [<IP_ADDRESS>] => (item=/tmp/ansible.wtdse9j2temp)
changed: [<IP_ADDRESS>] => (item=/tmp/ansible.nvvhot9dtemp)
...
PLAY RECAP *****************************************************************************************************************
<IP_ADDRESS>            : ok=65   changed=15   unreachable=0    failed=0    skipped=53   rescued=0    ignored=0


Update hadoop docs

Coverage decreased (-0.01%) to 51.687% when pulling 5f39ca672a4563c32388b4a21bc94c8cbd9f800a on zimiao/hadoop_doc into 22a0e1bf9f4c64d2890127119831121e01f29fcb on master.

Coverage decreased (-0.01%) to 51.687% when pulling 5f39ca672a4563c32388b4a21bc94c8cbd9f800a on zimiao/hadoop_doc into 22a0e1bf9f4c64d2890127119831121e01f29fcb on master.

Coverage decreased (-0.01%) to 51.687% when pulling 5f39ca672a4563c32388b4a21bc94c8cbd9f800a on zimiao/hadoop_doc into 22a0e1bf9f4c64d2890127119831121e01f29fcb on master.


raven.utils.fetchGitSha as requested (#150)

Oh neat, this looks good! Lemme give it a closer look and go from there. Thanks. :)

I'm not sure why the newer versions of node fail (shallow cloning in older git versions?), but it seems I'll need to use the deprecated Buffer API for node 0.x either way.
Let me know if you'd prefer to use some kind of mock repo to test this, or maybe you can make travis use a more recent version of git.

Bleh, I'm not sure off the top of my head. I'll have to look into this a bit. They also seem to be failing in an unrelated test. I'll get back to this later today for sure, just unfortunately, I can't merge this with broken tests until I understand why it's broken. :(

No rush, this needs some additional work to be useful anyway.

No rush, this needs some additional work to be useful anyway.

Biggest issue we have is reading packed refs. After a git gc, the ref doesn't exist anymore and is packed away inside .git/packed-refs.
We need to add this into raven-python too, but shouldn't be too hard.

I just threw up a PR for raven-python to handle parsing the packed-refs file, if you wanted to add that here as well: https://github.com/getsentry/raven-python/pull/761
It was more straightforward than I originally assumed.

There are some tools that can be used to do that (or read directly from git cli) and I don't think it's necessary to add it to the SDK.
Please let me know if it's still desired feature though.


Update files without creating a new package

Hi! I saw you changed some config, cleanup and makevars files in the last updates. I would like to update my package with these without using savvy init. Would it be enough to copy what is here? Or are some of these files platform specific?

Would it be enough to copy what is here?

Yeah, basically that's what you'll want to do. If you want to copy them, you'll also need to rename src/Makevars.win to src/Makevars.win.in as configure.win now generates it. Accordingly, you need to add it to .gitignore.

Or are some of these files platform specific?

What do you mean? If you are talking about those files with or without .win, yes. If the question is whether or not the files of R package can defer depending the platform of the package developer, no.

This should work.
https://github.com/daniellga/harmonium/pull/29

I really meant different files generated for different systems. But you already answered anyway.
I was thinking of something like "savvy-cli reinit" to update these config, makevars, and related files automatically in an existing project but there's the downside of overwriting existing stuff, so the user should be aware of that. I don't know if it's a good idea, probably not.
Thanks for the PR!

Basically, those files are what you are responsible for writing the contents. The template is just a template to be overwritten and customized. So, by its nature, it cannot be updated safely. I apologize that I couldn't figured out the best template earlier.


BODMAS Equation solver is created

Issue #29
Briefly explain your program logic -
I have used javascript functions to evaluate the equation and also handle all types of invalid input errors. Also used react hooks and material ui.
Screenshots(Attach 2 screenshots of your own input and output) -

By raising this PR I affirm that -

[x] My code follows the guidelines of this project.

[x] I have performed a self-review of my own code.

[x] I have commented my code.

[x] My code gives the correct output.

[x] I affirm that I strictly follow contributing guidelines and code of conduct.

@imsoumik03, Great work.

Congratulations 🎉 @imsoumik03 on your contribution.

@imsoumik03,
Adding appreciation points🎉, for doing impressive work and also for taking up admin-choice issue.


FAQ for Password Reset

Included multiple scenarios for password reset requests which should be handled by the respective system administrator of the group / company

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

Hey @ahmaddanialmohd. Just following up on this to find out if I can help with any of the changes. :)

Hi, @justinegeffen @wiersgallak @amyblais .
Sorry for missing out on this. In regards to this suggestion:

I agree with that and we could also add it to the Troubleshooting guide under the Administration section: https://docs.mattermost.com/install/troubleshooting.html#administration-issues as "Resetting user password"?

That sounds like a good idea since System Administrators should be the ones handling this request.

No worries, @ahmaddanialmohd. Would you be open to updating the Troubleshooting guide with this change? :)

No worries, @ahmaddanialmohd. Would you be open to updating the Troubleshooting guide with this change? :)

All set, @justinegeffen Thanks! https://github.com/mattermost/docs/pull/3443

@ahmaddanialmohd Can this be closed now that we have https://github.com/mattermost/docs/pull/3443?

@wiersgallak , yes please. Good to close.


restructure L2/L3s to match ESO Functional Areas

L2 and L3 mapping don't quite map to the core, processing, generic data, and analysis services. update this to match.


$viewContentLoaded and $viewContentLoading not trigger

Can you give some more information about this please? Version of ui-router, example code, etc

I some confused with the doc de UI-Router https://github.com/angular-ui/ui-router/wiki

$viewContentLoaded and $viewContentLoading should found with .run()

ui-router version v0.2.14

They aren't really "found" with .run(). You can assign event listeners to them in the in the run block that are then performed when the events are fired. If you have an example showing they aren't fired, please share it and I'll reopen the ticket.


Allow providing custom arguments to the invocation git blame

In my project I have a graft file to stitch together a history that's broken by rebases.
To allow git blame to track the history, I need to call it as git blame -S grafts.txt -- {path}
AFAIU there is currently no option to provide custom arguments to the invocation of git blame so I can't tell it to use my grafts.
Alternatively, specifying a custom alias for blame would work - I can call git my-blame {path} and have an alias defined in .git/config that my-blame = blame -S grafts.txt.
Either would work for my use case.

Hey mpawlowski-eyeo. Thanks for the feature request!
The revs-file sounds like it is unique per repository. The git blame process is always executed with a cwd set to the directory of the file it is currently blaming. That would mean that the revs-file path would need to be either relative to the .git-folder it was built from or an absolute path. This could possibly interfere with the blame information for sub repositories. Any idea how they interact?

That's a good point!
One way to work around that is to declare the setting like:
"gitblame": {
  "extraCmdArguments": [
    "-S",
    "${workspaceFolder}/grafts.txt"
  ]
}

This makes the path absolute AFAIR?
This likely wouldn't help for the second, alias-based approach, because git my-blame would be an unknown command in a sub repository - unless that alias was a global config or the setting was repeated in the sub repo's .git/config.
The grafts.txt file would not be meaningful for sub repositories because the commit hashes don't exist in their histories, but this wouldn't break anything (unless there was an astronomically unlikely random hash collision).  As long as the hashes from the file aren't encountered in the history traversal that git blame does, they're dormant.
FWIW I found the content/format of the revs-file passed via -S to be poorly documented, and after some googling I found that it looks like this:
$ cat git-grafts.txt
1993fb58c5732068109ce4178ee09bced4973a8a 83a714deb7aeff970e55ef11174b03aa02970d29
3cc64cebef5c028d87495de797630d2e3d56d522 a71c84f53e4e2254f6a34ed50e37ecf5d4362eb9
85b8322d2c1c97e7adfbe2162e7ade16fcd1e019 f2299e697fb99f2a693b405a2043d9622ebfe68b
075b1b8375709eae8a56081a9d4230ab56484ba4 65212eee6e9e6043ad78fc59619cbba31a692f67
e5425abe809325552de52772f2c5f379060f41d8 b78c67a90f7e0ed9ee2529da85520ba9d24185c1
...

When git blame traverses history and reaches a commit 1993fb58c5732068109ce4178ee09bced4973a8a it will skip over it and go to 83a714deb7aeff970e55ef11174b03aa02970d29, then resume traversing history from there. If it then reaches 3cc64cebef5c028d87495de797630d2e3d56d522 it will skip over to a71c84f53e4e2254f6a34ed50e37ecf5d4362eb9 and so on. This is useful if your project has been rebased or resquashed and the "real" history is scattered across many branches. The "skipped" commits are squash commits that would otherwise become a dead end.

Thanks for the explanation of the -S command! It was a bit unclear.
The ${workspaceFolder} idea looks good on the surface, but then you run into a situation like the following:
d  .git
f  .gitmodules
f  grafts.txt
d  a-git-module
    d  .git
    f  module-grafts.txt

(d) directory  (f) file

Perhaps it is not worth figuring out a solution that will work for this case as I get a feeling that this is an extreme edge case.
Workspaces also don't work perfectly as the extension no longer requires a workspace to be able to blame files.
Perhaps the only way to do it is to acknowledge these issues, document them and accept them.

I guess alias support could be a workaround for that case, albeit not super pretty:
One way:
# global scope, "grafted-blame" alias forwards to "blame"
git config --global alias.grafted-blame = blame

# top-level repo, local config overrides global config, the alias now uses grafts.txt
cd repo/
git config --local alias.grafted-blame = blame -S grafts.txt

# submodule, the alias uses module's grafts
cd a-git-module/
git config --local alias.grafted-blame = blame -S module-grafts.txt

# git grafted-blame {path} now behaves sensibly everywhere

You could also use conditional includes in the config files or even run a custom bash script that selects the correct graft file as part of the alias.  Git aliases are pretty powerful, it turns out. But these are all advanced options, the -S {workspaceFolder}/grafts.txt approach is easier to set up and probably handles many common use cases (considering grafts aren't that common in the first place).
I suppose we could have both features: custom command line args for blame and the option to use a custom alias instead of blame

It is luckily quite easy to check if a file exists before adding the -S-flag. I will see when I have time to implement this. Hopefully it will be sometime this week.
Thanks for the dialog. It has been very helpful!

A new version has been uploaded to the marketplaces. It can sometimes take a day or two before it is pushed to everyone. Thanks for the feature request. I hope that it helps with your use case!

Thanks a lot! I'll check it out on Monday and let you know :)
sob., 20 sty 2024, 17:07 użytkownik Albin Jacobsson <
@.***> napisał:

A new version has been uploaded to the marketplaces. It can sometimes take
a day or two before it is pushed to everyone. Thanks for the feature
request. I hope that it helps with your use case!
—
Reply to this email directly, view it on GitHub
https://github.com/Sertion/vscode-gitblame/issues/165#issuecomment-1902163578,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARRMERH4XCXCGSFZIP6WAATYPPTU3AVCNFSM6AAAAABBVGCE6KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBSGE3DGNJXHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

It works fine, great job :)


Add versions of needed packages

One of the comments from the hackathon was to include version numbers of installed software packages that were tried and tested.

Can you clarify which versions this is referring to?

In terms of the javascript dependency versions, this is handled by the package.json files
In terms of the python dependency versions, this should be in the requirements.txt files (though currently we aren't using explicit versions)
Both of these are successfully used to construct the Dockerfile which can be used to build this in a (mostly) device agnostic way.
In terms of the dependent software like NodeJS or Python, we could capture this somewhere, currently the latest versions work and likely many versions older.


Question: Link-Time Optimization

I was browsing Hacker News, and I found this article.
I was astonished when I saw this:
Compilation units

  The most common way of building C projects is to decompose every source file into an object file then link all the objects together at the end. This procedure works great for incremental development, but it is suboptimal for performance and optimization. Your compiler can't detect potential optimization across file boundaries this way.

LTO — Link Time Optimization
  LTO fixes the "source analysis and optimization across compilation units problem" by annotating object files with intermediate representation so source-aware optimizations can be carried out across compilation units at link time (this slows down the linking process noticeably, but make -j helps).

  As of 2016, clang and gcc releases support LTO by just adding -flto to your command line options during object compilation and final library/program linking.
LTO stil needs some babysitting though. Sometimes, if your program has code not used directly but used by additional libraries, LTO can evict functions or code because it detects, globally when linking, some code is unused/unreachable and doesn't need to be included in the final linked result.

I have debugged compilation issues for Dart before, and I never have seen -flto in the commands.
So the question is: Is this something Dart could use, and would it actually do anything useful?

@kaendfinger It'd be better to throw these types of questions at the misc@ alias.
Cheers.

Ok, thanks :)


Add examples of how a simple list and watch is built using client-go

basically how to use resourceVersion correctly. Sorry if this is documented somewhere else ?
From my understanding, we should always do a list and get the resourceVersion from the list
then use a watch with the above resourceVersion
if the watch fails, can we restart the watch with same resourceVersion
if the watch succeeds , how do i get the updated resourceVersion to do the watch again ?
I know if between list and watch , too much time elapses, the buffer can become full, in which case, watch would fail, i guess, do we need to list again there ?

/lifecycle frozen


Current git source fails to build on 20H2 and VS Community 2019 16.11.2

git clone https://github.com/microsoft/Terminal.git
cd Terminal
git submodule update --init --recursive
nuget restore OpenConsole.sln
VS Set to
Release x64  Cascadia Package
Are the updated build requirements?
That process worked though 9/4/2021.
Severity	Code	Description	Project	File	Line	Suppression State
Error	MSB3073	The command "pwsh.exe -NoProfile –ExecutionPolicy Unrestricted C:\Users\apd\devel\WT2\Terminal\tools\GenerateHeaderForJson.ps1 -JsonFile defaults.json -OutPath "Generated Files\defaults.h" -VariableName DefaultJson" exited with code 9009.	Microsoft.Terminal.Settings.Model.Lib	C:\Users\apd\devel\WT2\Terminal\src\cascadia\TerminalSettingsModel\Microsoft.Terminal.Settings.ModelLib.vcxproj	258
Severity	Code	Description	Project	File	Line	Suppression State
Error	LNK1181	cannot open input file 'C:\Users\apd\devel\WT2\Terminal\bin\x64\Release\Microsoft.Terminal.Settings.Model.Lib\Microsoft.Terminal.Settings.Model.Lib.lib'	Microsoft.Terminal.Settings.Model	C:\Users\apd\devel\WT2\Terminal\src\cascadia\TerminalSettingsModel\dll\LINK	1
Severity	Code	Description	Project	File	Line	Suppression State
Error	C1083	Cannot open include file: 'winrt/base.h': No such file or directory	LocalTests_SettingsModel	C:\Users\apd\devel\WT2\Terminal\dep\wil\include\wil\cppwinrt.h	98
Severity	Code	Description	Project	File	Line	Suppression State
Error	C1083	Cannot open include file: 'Profile.g.h': No such file or directory (compiling source file ColorHelperTests.cpp)	UnitTests_TerminalApp	C:\Users\apd\devel\WT2\Terminal\src\cascadia\TerminalSettingsModel\Profile.h	47
Severity	Code	Description	Project	File	Line	Suppression State
Error		PRI175: 0x80070002 - Processing Resources failed with error: The system cannot find the file specified.	WindowsTerminalUniversal	C:\Users\apd\devel\WT2\Terminal\src\cascadia\WindowsTerminalUniversal\GENERATEPROJECTPRIFILE	1
Severity	Code	Description	Project	File	Line	Suppression State
Error		PRI252: 0xdef00071 - File C:\Users\apd\devel\WT2\Terminal\bin\x64\Release\Microsoft.Terminal.Settings.Model\Microsoft.Terminal.Settings.Model.pri not found.	WindowsTerminalUniversal	C:\Users\apd\devel\WT2\Terminal\src\cascadia\WindowsTerminalUniversal\GENERATEPROJECTPRIFILE	1
Severity	Code	Description	Project	File	Line	Suppression State
Error		PRI175: 0x80070002 - Processing Resources failed with error: The system cannot find the file specified.	CascadiaPackage	C:\Users\apd\devel\WT2\Terminal\src\cascadia\CascadiaPackage\GENERATEPROJECTPRIFILE	1
Severity	Code	Description	Project	File	Line	Suppression State
Error		PRI252: 0xdef00071 - File C:\Users\apd\devel\WT2\Terminal\bin\x64\Release\Microsoft.Terminal.Settings.Model\Microsoft.Terminal.Settings.Model.pri not found.	CascadiaPackage	C:\Users\apd\devel\WT2\Terminal\src\cascadia\CascadiaPackage\GENERATEPROJECTPRIFILE	1
Severity	Code	Description	Project	File	Line	Suppression State
Error		PRI175: 0x80070002 - Processing Resources failed with error: The system cannot find the file specified.	TestHostApp	C:\Users\apd\devel\WT2\Terminal\src\cascadia\LocalTests_TerminalApp\TestHostApp\GENERATEPROJECTPRIFILE	1
Severity	Code	Description	Project	File	Line	Suppression State
Error		PRI252: 0xdef00071 - File C:\Users\apd\devel\WT2\Terminal\bin\x64\Release\Microsoft.Terminal.Settings.Model\Microsoft.Terminal.Settings.Model.pri not found.	TestHostApp	C:\Users\apd\devel\WT2\Terminal\src\cascadia\LocalTests_TerminalApp\TestHostApp\GENERATEPROJECTPRIFILE	1

Have you got PowerShell 7 or later installed? I believe that was a recent change to the build requirements.
See here: https://github.com/microsoft/terminal#prerequisites

Sorry somehow I missed that.  PS7.


A question about streaming

Hi.
server/pc_server.go contains
		// set the timeout if we have it
		if len(to) > 0 {
			if n, err := strconv.ParseUint(to, 10, 64); err == nil {
				ctx, _ = context.WithTimeout(ctx, time.Duration(n))
			}
		}

Yet I can't figure out how to call it without passing a duration. Is there a way?

Duration is always set at the edge. The first client will look for the timeout header. If it's not found it will set one to the timeout in the client. The default timeout is 5 seconds. There is no way to make a call without passing the duration. This avoids indefinitely blocking.
Please ask questions in the slack. We maintain issues for bugs.


Extra GPS Data

Is there a way to get extra gps data while using the uGnssPosGetStart, such as the information listed in this structure?
typedef struct {
    uint32_t iTOW;                       /**< GPS time of week of the
                                              navigation epoch
                                              in milliseconds. */
    uint16_t year;                       /**< year (UTC); to obtain this and
                                              the other time-related fields
                                              in this structure as a Unix-based
                                              UTC timestamp, see
                                              uGnssDecUbxNavPvtGetTimeUtc(). */
    uint8_t month;                       /**< month, range 1 to 12 (UTC). */
    uint8_t day;                         /**< day of month, range 1 to 31 (UTC). */
    uint8_t hour;                        /**< hour of day, range 0 to 23 (UTC). */
    uint8_t min;                         /**< minute of hour, range 0 to 59 (UTC). */
    uint8_t sec;                         /**< seconds of minute, range 0 to 60 (UTC). */
    uint8_t valid;                       /**< validity flags, see
                                              #uGnssDecUbxNavPvtValid_t. */
    uint32_t tAcc;                       /**< time accuracy estimate in
                                              nanoseconds. */
    int32_t nano;                        /**< fractional seconds part of UTC
                                              time in nanoseconds. */
    uGnssDecUbxNavPvtFixType_t fixType;  /**< the fix type achieved. */
    uint8_t flags;                       /**< see #uGnssDecUbxNavPvtFlags_t. */
    uint8_t flags2;                      /**< see #uGnssDecUbxNavPvtFlags2_t. */
    uint8_t numSV;                       /**< the number of satellites used. */
    int32_t lon;                         /**< longitude in degrees times 1e7. */
    int32_t lat;                         /**< latitude in degrees times 1e7. */
    int32_t height;                      /**< height above ellipsoid in mm. */
    int32_t hMSL;                        /**< height above mean sea level in mm. */
    uint32_t hAcc;                       /**< horizontal accuracy estimate in mm. */
    uint32_t vAcc;                       /**< vertical accuracy estimate in mm. */
    int32_t velN;                        /**< NED north velocity in mm/second. */
    int32_t velE;                        /**< NED east velocity in mm/second. */
    int32_t velD;                        /**< NED down velocity in mm/second. */
    int32_t gSpeed;                      /**< 2D ground speed in mm/second. */
    int32_t headMot;                     /**< 2D heading of motion in degrees times 1e5. */
    uint32_t sAcc;                       /**< speed accuracy estimate in mm/second. */
    uint32_t headAcc;                    /**< heading accuracy estimate (motion and
                                              vehicle) in degrees times 1e5. */
    uint16_t pDOP;                       /**< position DOP times 100. */
    uint16_t flags3;                     /**< see #uGnssDecUbxNavPvtFlags3_t. */
    int32_t headVeh;                     /**< if the #U_GNSS_DEC_UBX_NAV_PVT_FLAGS_HEAD_VEH_VALID
                                              bit of the flags field is set
                                              then this is the 2D vehicle
                                              heading in degrees times 1e5,
                                              else it is set to the same
                                              value as headMot. */
    int16_t magDec;                      /**< if the #U_GNSS_DEC_UBX_NAV_PVT_VALID_MAG
                                              bit of the valid field is set then this
                                              is the magnetic declination in degrees
                                              times 100; only supported on ADR 4.10
                                              and later. */
    uint16_t magAcc;                     /**< if the #U_GNSS_DEC_UBX_NAV_PVT_VALID_MAG
                                              bit of the valid field is set then this
                                              is the accuracy of the magnetic
                                              declination in degrees times 100; only
                                              supported on ADR 4.10 and later. */
} uGnssDecUbxNavPvt_t;

Hi Michael: as more people are asking about this we should probably extend the uGnssPos API but until we've done that the simplest/quickest way for you to get at the level of detail you would like is to use the uGnssMsg and uGnssDec APIs instead of the uGnssPos API.
Basically you would follow the msg_main example and call to pUGnssDecAlloc() in the callback.  The code might look something like this (I've NOT compiled this code, just a sketch to give you an idea):
char buffer[512];

//  Asynchronous message callback.
void callback(uDeviceHandle_t devHandle,
              uGnssMessageId_t *pMessageId,
              int32_t errorCodeOrLength,
              void *pCallbackParam)
{
    // Don't need the message ID since we asked specifically for UBX-NAV-PVT
    (void) pMessageId;
    (void) pCallbackParam;

    if (errorCodeOrLength > 0) {
        if (errorCodeOrLength > sizeof(buffer)) {
            errorCodeOrLength = sizeof(buffer);
        }
        errorCodeOrLength = uGnssMsgReceiveCallbackRead(devHandle,
                                                        buffer,
                                                        errorCodeOrLength);
        if (errorCodeOrLength > 0) {
            uGnssDec_t *pDec = pUGnssDecAlloc(buffer, errorCodeOrLength);
            if ((pDec != NULL) && (pDec->errorCode == 0)) {
                // Not checking pDec->pId since we only asked for UBX-NAV-PVT,
                // checking for NULL and a zero error code is sufficient

                // The decoded UBX-NAV-PVT message is now hooked
                // into the memory allocated at pDec->pMsg, so for instance:
                uGnssDecUbxNavPvt_t *pUbxNavPvt = pDec->pBody.ubxNavPvt;
                print("iTow is %d.\n", pUbxNavPvt->iTOW);

                // The same rules apply here as to other callbacks:
                // do as little as possible, push any processing off
                // onto an RTOS queue, to somewhere that your
                // application controls the stack-size of
            }

            // IMPORTANT: must *always* free the memory that
            // pUGnssDecAlloc() allocated when done (you can
            // do this at the other end of your RTOS queue if
            // you like).
            uGnssDecFree(pDec);
        }
    }
}

...then in your main code:
int32_t x;
uGnssMessageId_t messageId;

// devHandle is assumed to be set up somewhere above

// Set up the message ID we want to capture: in this case just UBX-NAV-PVT
messageId.type = U_GNSS_PROTOCOL_UBX;
messageId.id.ubx = 0x0107;

// This sets up the message rate for our message ID to be once per "epoch".
// The default epoch of the GNSS chip is one second (you can alter it with
// uGnssCfgSetRate() if you feel a need, but it is usually clearer to leave it
// alone and increase the per-epoch rate for a given message as necessary)
x = uGnssCfgSetMsgRate(devHandle, &messageId, 1);
if (x == 0) {

    // Set up an asynchronous message receiver for this message ID,
    // giving it the callback above.  If you like you can also give it
    // a void * parameter (NULL in this example)
    x= uGnssMsgReceiveStart(devHandle, &messageId, callback, NULL);
    if (x < 0) {
        printf("Unable to start message receiver (%d)!\n", x);
    }

   // callback() will now be called when UBX-NAV-PVT messages arrive,
   // which should be once a second

    ...

    // Call this when you want to stop
    uGnssMsgReceiveStopAll(devHandle);

} else {
    printf("Unable to set UBX-NAV-PVT message rate (%d)!\n", x);
}

Thanks for this Rob, I was able to get it working. There were only a few small things I had to fix. One thing that I don't seem to have working though is the rate setting (just commented it out for now). But I seem to be getting a -4 error.
returnCode = uGnssCfgSetMsgRate(this->devHandle, &messageId, 1);

Ah, yes, sorry, my bad, I should have read my own comments: -4 is U_ERROR_COMMON_NOT_SUPPORTED; M10 has the new configuration API, so you need to use that.  The message rate setting is interface specific (UART1 v UART2 v SPI v I2C v USB), not sure which interface you are using, but you would choose from one of the following:
returnCode = U_GNSS_CFG_SET_VAL_RAM(this->devHandle, MSGOUT_UBX_NAV_PVT_I2C_U1, 1);
returnCode = U_GNSS_CFG_SET_VAL_RAM(this->devHandle, MSGOUT_UBX_NAV_PVT_SPI_U1, 1);
returnCode = U_GNSS_CFG_SET_VAL_RAM(this->devHandle, MSGOUT_UBX_NAV_PVT_UART1_U1, 1);
returnCode = U_GNSS_CFG_SET_VAL_RAM(this->devHandle, MSGOUT_UBX_NAV_PVT_UART2_U1, 1);
returnCode = U_GNSS_CFG_SET_VAL_RAM(this->devHandle, MSGOUT_UBX_NAV_PVT_USB_U1, 1);

Tried that with the correct UART and seem to get the following error, which I don't see in the common errors.
returnCode = U_GNSS_CFG_SET_VAL_RAM(this->devHandle, MSGOUT_UBX_NAV_PVT_UART2_U1, 1);

Unable to set UBX-NAV-PVT message rate (-1025)!

Hmm, that's a NACK from the GNSS chip (see https://github.com/u-blox/ubxlib/blob/master/DEBUG_HINTS.md#reading-error-return-codes).  Are you sure you're on UART2?

Switched to UART1 and it worked but I definitely thought this was configured for UART2.
Heres my config
 //the gps device
        uDeviceCfg_t gDeviceCfg = {
            .deviceType = U_DEVICE_TYPE_GNSS,
            .deviceCfg = {
                .cfgGnss = {
                    .moduleType = U_GNSS_MODULE_TYPE_M10,
                    .pinEnablePower = U_CFG_APP_PIN_GNSS_ENABLE_POWER,
                    .pinDataReady = -1,
                },
            },
            .transportType = U_DEVICE_TRANSPORT_TYPE_UART,
            .transportCfg = {
                .cfgUart = {
                    .uart = UART_NUM,
                    .baudRate = BAUD_RATE,
                    .pinTxd = TX_PIN,
                    .pinRxd = RX_PIN,
                    .pinCts = CTS_PIN,
                    .pinRts = RTS_PIN,
                    .pPrefix = NULL
                },
            },
        };

Where the defined UART_NUM is
#define UART_NUM UART_NUM_2

Ah, sorry, it is confusing: in this case UART2 means from the GNSS chip's internal point of view, rather than the UART interface of the MCU.  Various of the GNSS chip's settings, of which the message rate is one, it remembers on a "per interface" basis, and some of the GNSS chips have two UARTs.
Where they have only one, it is UART1, only a very few have a second UART but UART2 is present in the API signaling of course, you'll just get a NACK from the GNSS chip if you try to use it.  So unless you have a GNSS chip that has two UARTs, or if you have and you are using the first of them, the message rate settings should all be the "UART1" settings.
IYSWIM.

Makes sense! I think I have it all working now. Thanks! It would be nice to have this in the generic API. At least some of the items.

Agreed, I will put it on the list.  Would really not be very difficult to do, since I wrote it just above :-).  Just needs adding.

As always thanks again for the help!


Use 'Rails.root.join' in rails_helper

I updated rubocop-rails gem to v2.17.3 in my project recently.
By this update, Rubocop suggests to use Rails.root.join instead of string interpolation for Rails/FilePath at 'rails_helper.rb'.
It might be better to use Rails.root.join than string interpolation too in rails_helper generator.
How do you think about this?

Released in 6.1.0


SqlAlchemy serializer. Nuberic float field error.

Hello, my friend)
An error occurred during the serialization of float fields.
Error:
  File "/usr/local/Cellar/python3/3.6.1/Frameworks/Python.framework/Versions/3.6/lib/python3.6/asyncio/coroutines.py", line 210, in coro
    res = func(*args, **kw)
  File "/Users/antonsvecov/envs/wl/lib/python3.6/site-packages/aiorest_ws/request.py", line 63, in onMessage
    response = self.factory.router.process_request(request)
  File "/Users/antonsvecov/envs/wl/lib/python3.6/site-packages/aiorest_ws/routers.py", line 131, in process_request
    response.content = handler.dispatch(request, *args, **kwargs)
  File "/Users/antonsvecov/envs/wl/lib/python3.6/site-packages/aiorest_ws/views.py", line 77, in dispatch
    return handler(request, *args, **kwargs)
  File "/Users/antonsvecov/Projects/betronic-white-label/admin/src/utils/decorators.py", line 21, in wrapper
    return handler(request, *args, **kwargs)
  File "/Users/antonsvecov/Projects/betronic-white-label/admin/src/utils/abstract_view.py", line 30, in get
    session, **kwargs)
  File "/Users/antonsvecov/Projects/betronic-white-label/admin/src/utils/abstract_view.py", line 20, in _get_model_all
    return self.serializer(items, many=True).data
  File "/Users/antonsvecov/envs/wl/lib/python3.6/site-packages/aiorest_ws/db/orm/serializers.py", line 635, in data
    ret = super(ListSerializer, self).data
  File "/Users/antonsvecov/envs/wl/lib/python3.6/site-packages/aiorest_ws/db/orm/serializers.py", line 154, in data
    self._data = self.to_representation(self.instance)
  File "/Users/antonsvecov/envs/wl/lib/python3.6/site-packages/aiorest_ws/db/orm/serializers.py", line 585, in to_representation
    return [self.child.to_representation(item) for item in data]
  File "/Users/antonsvecov/envs/wl/lib/python3.6/site-packages/aiorest_ws/db/orm/serializers.py", line 585, in <listcomp>
    return [self.child.to_representation(item) for item in data]
  File "/Users/antonsvecov/envs/wl/lib/python3.6/site-packages/aiorest_ws/db/orm/serializers.py", line 379, in to_representation
    fields = self._readable_fields
  File "/Users/antonsvecov/envs/wl/lib/python3.6/site-packages/aiorest_ws/utils/functional.py", line 17, in __get__
    result = instance.__dict__[self.func.__name__] = self.func(instance)
  File "/Users/antonsvecov/envs/wl/lib/python3.6/site-packages/aiorest_ws/db/orm/serializers.py", line 291, in _readable_fields
    field for field in self.fields.values()
  File "/Users/antonsvecov/envs/wl/lib/python3.6/site-packages/aiorest_ws/db/orm/serializers.py", line 277, in fields
    for key, value in self.get_fields().items():
  File "/Users/antonsvecov/envs/wl/lib/python3.6/site-packages/aiorest_ws/db/orm/serializers.py", line 756, in get_fields
    fields[field_name] = field_class(**field_kwargs)
TypeError: __init__() missing 2 required positional arguments: 'max_digits' and 'decimal_places'

Model:
class UserModel(BaseModel):
    __tablename__ = "user"

    id = Column(Integer, autoincrement=True, primary_key=True)
    first_name = Column(String(60))
    middle_name = Column(String)
    last_name = Column(String(60))
    nickname = Column(String(60), unique=True)
    balance = Column(Numeric(10, 2), default=0)

Problem in the field balance.
Please, fix this bug.
Thanks, dear friend)

Hi, @fantom0005!
Thanks for opening an issue here.
This error occurred in runtime because I forgot to add additional logic for processing cases if passed Numeric field (or some derived class), like we have it for Django DecimalField type.

Fixed in pull request #83


Add MP3 config to config.h.in

config.h.in is missing an MP3 flag which stops MP3 audio from working on cmake projects unless this is overcome/overriden.

Ouch! I didn't notice it! Thanks! 👍


python310Packages.diff-cover: Disable failing tests

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Only a couple of the disabled tests here explain why they're ok to disable. The others just list the errors and assume it's ok, but to me they look like geuinely failing tests.

Package and disabled tests list have since been updated, obsoleting this PR.


Change training data version warning to be a debug log

In the new training data, you can put a version key in the files. If you don't do this, you will see warnings like:
UserWarning: Training data file /Users/christina/rasa/rasa/cli/initial_project/domain.yml doesn't have a 'version' key. Rasa Open Source will read the file as a version '2.0' file. (originating from here)
I'd suggest to not raise those warnings, but rather put them into a debug log.
My reasoning is this (see also the comments in this PR: https://github.com/RasaHQ/rasa/pull/6329#pullrequestreview-462397138):
Adding the version is not expected but rather an optional feature that people can use to get warned about incompatibilities with their Rasa Open Source version. (Or am I mistaken here?) The warnings seem to indicate that something is a problem, which is not the case.
Also, if we want to avoid that users see multiple warnings for each training data file when running rasa init, we would have to add a version key to the init project, too and manually maintain that to stay up to date.

@degiz what do you think?

It depends on how much we want to urge people to explicitly include a version

We want them to at least be aware of the format version. The reason is - once things don't work for them in the future, when the format changes, they will know where to start.
I like the idea that we define the version in example bots, because that's great to show the users that it's a good practice to have the version key.

a warning when a version is provided and it's too high, a debug log when no version is provided.

it makes sense, so let's do that.


Unable to set "data-testid" on the inner input of a Checkbox component.

[X ] The issue is present in the latest release.
[ X] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
It isn't possible to add a "data-testid" on the inner input element for a Checkbox component via inputProps when using typescript 3.8.3. It gives the typescript error "Type '{ 'data-testid': string; }' is not assignable to type 'InputHTMLAttributes'."
Expected Behavior 🤔
The data-testid attribute is set on the inner input element in the same way as for TextField components.
Steps to Reproduce 🕹
Create a Checkbox component which sets inputProps including a data-testid with.:
<Checkbox inputProps={{ "data-testid": "test-checkbox" }} />
See Codesandbox example
Your Environment 🌎

Tech
Version

Material-UI
v4.9.5

React
v3.16.13

TypeScript
v 3.8.3

This is currently a limitation of typescript. We're waiting for https://github.com/microsoft/TypeScript/issues/28960.
For more context see https://github.com/mui-org/material-ui/pull/18902.
Possible workarounds:

@ts-ignore:<Checkbox
  inputProps={{
    // @ts-ignore
    'data-testid': 'test-checkbox',
  }}
/>;

use ByRole instead of ByTestid. We generally don't recommend data-testid. Most of the (interactions) tests can be implemented using ByRole.

As a side note, it's reproducible without Material-UI: https://codesandbox.io/s/material-demo-forked-7gfgi?file=/demo.tsx.
It's also worth mentioning that it's not an issue with the InputBase because of the custom type:
https://github.com/mui-org/material-ui/blob/0ff48309f5429dfed1cfef2c08d8a5d8fce70345/packages/material-ui/src/InputBase/InputBase.d.ts#L236-L240


[OPI - Armbian] Make seccomp disabled by default

Using seccomp fails on orange pi and it's not used on rpi using resinos
Supervisor starts with seccomp profile disabled.
We could default to True if somehow a default seccomp profile file was generated that could be used during addon startup and that is tested to work with at least core addons.
I was not able unfortunately to generate even with the help of several tools proper error mesaages to point out exactly which system call is not working because of seccomp.
I tries to start mosquitto and another custom addon: zigbee2mqtt.
Unfortunately the entry point would not be called for either so the addon would fail silently.

It seems local addons also don't build and it's probably because of seccomp as well, but I can't figure out from where to disable that.

Is seccomp special configured on OPI? I don't see any other platform they have this troubles. But I agree, that we should disable it.

I don’t know. I think it’s how the Armbian build is done.
Sent with GitHawk

@pvizeli any recommendation on how to disable seccomp for local builds?

Based on what I could read on the Armbian forums, they have some issues with SECCOMP and 64-bit.

I get this error running install for local addon:
core-ssh:~# hassio addons install -name local_zigbee2mqtt

ERROR
panic: interface conversion: interface is nil, not string

goroutine 1 [running]:
panic(0x2f8720, 0x442013a140)
	/home/travis/.gimme/versions/go1.7.6.linux.amd64/src/runtime/panic.go:500 +0x390
github.com/home-assistant/hassio-cli/command/helpers.DisplayOutput(0x4420142000, 0x24, 0x200, 0x0)
	/home/travis/gopath/src/github.com/home-assistant/hassio-cli/command/helpers/common.go:118 +0x404
github.com/home-assistant/hassio-cli/command/helpers.ExecCommand(0x343b38, 0x6, 0x44200d2fc0, 0x19, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/home/travis/gopath/src/github.com/home-assistant/hassio-cli/command/helpers/common.go:155 +0x31c
github.com/home-assistant/hassio-cli/command.CmdAddons(0x44200829a0)
	/home/travis/gopath/src/github.com/home-assistant/hassio-cli/command/addons.go:65 +0x714
github.com/urfave/cli.HandleAction(0x2dc640, 0x372fe0, 0x44200829a0, 0x0, 0x0)
	/home/travis/gopath/src/github.com/urfave/cli/app.go:503 +0x118
github.com/urfave/cli.Command.Run(0x343b38, 0x6, 0x0, 0x0, 0x49a2e0, 0x1, 0x1, 0x353ed0, 0x5a, 0x0, ...)
	/home/travis/gopath/src/github.com/urfave/cli/command.go:165 +0x7b0
github.com/urfave/cli.(*App).Run(0x44200e2000, 0x442000a0f0, 0x5, 0x5, 0x0, 0x0)
	/home/travis/gopath/src/github.com/urfave/cli/app.go:259 +0x950
main.main()
	/home/travis/gopath/src/github.com/home-assistant/hassio-cli/main.go:22 +0x150

I work a new version of supervisor to avoid this problem. Can send the log of supervisor by build error?

https://github.com/home-assistant/hassio/commit/cfbaa8cdf9504f498c9767b325e32cc378e6630b

@pvizeli I will try tonight to send some logs.


[dv/top] Regression triage

fixes #14595
change test to invoke auto_mode only, instead of boot and auto mode.
shorten the rng raw bits generation time to speed up the test.

Signed-off-by: Timothy Chen<EMAIL_ADDRESS>

I will merge for now, but will revisit the run time args when #14414 lands.


The config file may not be writable relative to the binary

I packaged SFP_UI as an AUR package, but because it's packaged in a system directory (/usr/bin/SFP_UI), the config file SFP.config is not writable and settings do not save.
The config file appears to be JSON and should have the .json file extension, and be placed within the standard locations depending on the platform if the relative path is not writable.
This issue can affect Windows if the program gets packaged using a third-party installer, or package manager.
(Winget, Scoop, Chocolatey, etc.)
Windows: %AppData%
Linux: $XDG_CONFIG_HOME with a fallback of $HOME/.config when not set
macOS: ~/Library/Preferences

This is something I plan on changing in the next update. Currently I'm waiting for the next update of both avalonia and fluentavalonia to drop and then I'm going to upgrade to net8 and redo the configuration system, and when I do that I will take this into account.

@GrahSebastian having to launch it from the terminal must be some specific problem with the aur package, I'm not too familiar with that. But with the version released on github, you can just double click on the executable to run it from your DE, no terminal necessary.

Yes, the AUR package does not include a desktop entry or icon, I'm still waiting for the programs config file to be moved, so the settings save, until then you either have to touch/chmod the location for the config or manually download and run the binary in a location that is writable.


A few init improvements

This makes init work when you made a deploy to a site manually and want to upgrade to using CI.  It also adds a force flag so you can re-init a site that is already using CI.

Coverage remained the same at 100.0% when pulling c05ac2bf37634f97c4ab59d1a8bc3155c6d7f051 on init-improvements into a2ef0cf19a0cc72fe346d1abf7453c8152a51de0 on master.

Incorporate after https://github.com/netlify/cli/pull/178


hide controllers

hide ui functions are not working

any soloution?

Which functions are you referring to? the web ui can't be modified, as it comes from the IFrame Player.

Which functions are you referring to? the web ui can't be modified, as it comes from the IFrame Player.

i want to hide the media controlls from youtube

This library is a wrapper around the IFrame player, i suggest taking a look at their doc: https://developers.google.com/youtube/iframe_api_reference


V2 Release! First breaking change certificate chains.

Thank you so much @jkralik for this patch, sorry it took me so long :( but we are going to land this now and get in some other stuff!

Breaking changes:

certificate, privateKey was replaced
by certificate (tls.Certificate)
verifyPeerCertificates uses array of bytes for chain
certificate instead of certificate(*x509.Certificate)

Codecov Report

Merging #142 into master will decrease coverage by 0.2%.
The diff coverage is 65.59%.

@@            Coverage Diff             @@
##           master     #142      +/-   ##
==========================================
- Coverage   73.61%   73.41%   -0.21%
==========================================
  Files          54       54
  Lines        3260     3303      +43
==========================================
+ Hits         2400     2425      +25
- Misses        600      610      +10
- Partials      260      268       +8

Impacted Files
Coverage Δ

state.go
66.66% <ø> (ø)
:arrow_up:

server_handlers.go
85.39% <100%> (ø)
:arrow_up:

config.go
100% <100%> (ø)
:arrow_up:

client_handlers.go
82.96% <100%> (ø)
:arrow_up:

conn.go
80% <100%> (-0.05%)
:arrow_down:

crypto.go
58.2% <50%> (-6.5%)
:arrow_down:

util.go
89.21% <66.66%> (+3.5%)
:arrow_up:

handshake_message_certificate.go
76.19% <74.19%> (+5.82%)
:arrow_up:

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 1d51cfe...b8ff2e3. Read the comment docs.

I'm a bit puzzled by something. In a few places I now see this show up:
func xxx(... rawCertificates [][]byte ...) ... {
if len(rawCertificates) == 0 {
		return errLengthMismatch
	}
certificate, err := x509.ParseCertificate(rawCertificates[0])

Since it seems we only ever care/check for/validate the first one, what is the idea behind accepting a [][]byte? And if len(rawCertificates) > 1, what happens to those?

@jkralik hey (no rush though)! any thoughts on @daenney question, I was just fixing merge conflicts haven't really dove in yet.
@hjames9 mind giving it another look over? I tried to use constants everywhere/simplify stuff as much as I possible.

@Sean-Der Sorry for late response but I was very busy. I created PR https://github.com/pion/dtls/pull/152 for this branch to fix issues. And I updated Config  -> Now it is similar as is crypto/tls. But for now it supports only one certificate.


[IOTDB-2745]Fix compaction priority param incorrect bug

The argument "compaction_priority" in iotdb-engine.properties file should be uppercase.

Coverage decreased (-0.05%) to 65.561% when pulling a64384471c7605c6ae2255620e9eba35b5652361 on choubenson:fixCompactionBug into 0d2cad6dbf3f8d9a43242f28d07c8c90fcc4d2c2 on apache:master.


Feature Discussion: Database Migrations

The Aim

Be able to get the database from any state to any other state.

Example 1: from nothing, to all tables created and populated.
Example 2: switch to a different git branch, and new tables and columns are created.
Example 3: switch back to an old git branch, and things roll back naturally.

Be able to discern the current state, and know the steps required to get to a new state.
Be able to run up (eg. create a new table) or down (drop the table)
Take advantage of Haxe's type system, and save creating manual migrations unless absolutely necessary.

The Idea So Far
(Feel free to comment / edit...)
// How do we describe database structure, when the model might not exist at runtime?
// (For example, if we had a model, then rolled back to before it existed, how do we still have a migration that makes sense?)

typedef DBColumn = { name:String, type:sys.db.RecordType };
typedef DBIndex = { fields:Array<String>, unique:Bool };
typedef DBForeignKey = { fields:Array<String>, relatedTableName:String };
typedef DBTable = {
    tableName:String;
    fields:Array<DBColumn>;
    indices:Array<DBIndex>;
    foreignKeys:Array<DBForeignKey>;
};

// What are all the possible actions we'd want to take on our database?

enum MigrationAction {
    CreateTable( name:String, initialFields:Array<DBColumn>, indices:Array<DBIndex> );
    AddField( tableName:String, field:DBColumn );
    ModifyField( tableName:String, before:DBColumn, after:DBColumn );
    AddIndex( tableName:String, index:DBIndex );
    AddForeignKey( tableName:String, index:DBForeignKey );
    CreateJoinTable( modelAName:String, modelBName:String );
    InsertData( tableName:String, id:Null<Int>, properties:{} );
    RunScript( upHScript:String, downHScript:String );
    // And the reverse of all of those...
}
enum MigrationDirection {
    Up;
    Down;
}

// Now, at any given step, we might have several actions we want to do at once, a "migration".
// How do we store these migrations so they can go "up" if code is present,
// and still go "down" if we're on an old version of the code where it doesn't exist.
// Basically, we need to be able to undo a migration, even if that migration does not exist in the current code base.
// So, when a migration runs "up", we'll store info about it in the database that allows us to run it "down" if we need to.

class Migration {
    // These are the actions for this migration
    var actions:SData<Array<MigrationAction>>;

    // And this is information about when we run the migration, which we save to the DB:
    var direction:MigrationDirection;
    var date:SDateTime;
    var id:String;

    function new( actions ) {
        this.actions = actions;
        this.id = Type.getClassName( Type.getClass(this) );
    }
    function runUp();
    function runDown();
}

// For example

class Migration2014121501_Create_User_Table extends Migration {
    public function new() super([
         CreateTable( "User", [...], [...] ),
         ModifyField( "User", ..., ... ),
         CreateTable( "Group", [...], [...] ),
         CreateJoinTable( "User", "Group" )
    ]);
}

var m = new Migration2014121501_Create_User_Table();
m.runUp();

// So, we have a way of creating a Migration class that has actions that can run up and down.
// How do we tell what needs to run up, and what needs to run down?
// Well, imagine we have this API:

typedef MigrationsToRun:Array<{ migration:Migration, dir:MigrationDirection }>;
class MigrationApi extends UFApi {
    function getMigrationsInCode():List<Migration>; // Get all migration classes that were compiled
    function getMigrationsInDB():List<Migration>; // Get all migrations that have been run "up" on the database
    function diffMigrations( inCode:List<Migration>, inDB:List<Migration> ):{ toRunUp:MigrationsToRun, toRunDown:MigrationsToRun }
    function runMigrations( migrations:Array<{ m:Migration, dir:MigrationDirection}> );
}

// So what if we need to do a migration that involves transforming data?
// Other migration systems you just write it in code, but it makes it difficult to "run down".
// Because, if you're on an old version of the code base, you don't have the migration to know what code to run when you're undoing the migration.
// A possible solution is to use hscript, and expose a limited API.

var splitNamesScript = "
for ( u in BasicManager.getAllObjects(User) ) {
    var names = u.name.split(" ");
    u.firstName = names.shift();
    u.lastName = names.pop();
    u.middleNames = names;
    u.save();
}
";
var joinNamesScript = "
for ( u in BasicManager.getAllObjects(User) ) {
    u.names = '${u.firstName} ${u.middleNames.join(" ")} ${u.lastName()}';
    u.save();
}
";
var actions = [
    AddField( "User", { name:"firstName", type:SString<20> } );
    AddField( "User", { name:"lastName", type:SString<20> } );
    AddField( "User", { name:"middleNames", type:SData<Array<String>> } );
    RunScript( splitNamesScript, joinNamesScript );
    RemoveField( "User", { name:"name", type:SString<255> } );
];

// Cool, that looks like it will mostly work - when running up it will create the new fields, run the "splitNamesScript" script, delete the old field.
// When running down, it will create the old field, run the "joinNamesScript" script, and delete the new fields.

//
//
// So.... how do we make this easy to maintain?

//
//

// Let's have a description of the exact state of our DB schema.
typedef Schema = Array<DBTable>;

// And then, add to our MigrationAPI some methods to get a schema:
class MigrationAPI extends UFApi {
    schemaFromModels():Schema; // Based on the models in the code
    schemaFromDatabase():Schema; // Based on the migrations in the DB
    diffSchema( s1:Schema, s2:Schema ):Migration; // Get a migration
    runTempMigrations():Void; // Set up the tables automatically without creating a formal migration.
    undoTempMigrations():Void; // Undo the above step.
}

So in effect, the workflow would be like this:

Create or modify models in your code, and compile.
Use MigrationAPI.runTempMigrations() while you're working on it.
Turn your temporary migrations into a real one.  (undoTempMigrations(), diffSchema() to create new migration).  If you want to change the specifics of the migration or do any custom data seeding / processing, go for it.
Commit the new migration to version control.
When someone else pulls down a new branch or release and it has new migrations, use runMigrations to get the DB in line...

Some questions:

We'll need a tool to help with this.  Does it run as part of the ufront CLI (haxelib run ufront migrate) or does it run through a UFTasks file that is compiled as part of the project.  The tasks file makes sense because in compile time you'd have access to the models.  It might be more bootstrapping required to get an initial setup though...

Should the migrations be a ".hx" file, or should we make them a JSON file or something?  As long as we commit them to source control, and they're available at runtime, it shouldn't matter.

This is a fairly complicated model, but I think it will work for most use cases.  Is there anything I'm missing?  Will this complexity be hard to maintain in the future?  Is their a simpler way to do it?

That's a much better option.  Just let them know which migrations need to be rolled back, and they can figure it out in version control.
Now to just code it.... ;)


Docker files depend on Poetry Version 1.1.11

Install the version of poetry if you want to update any packages:
curl -sSL https://install.python-poetry.org | python3 - --version 1.1.11

Inboard, one of the key dependences, has switched to Hatch (cf. https://github.com/whythawk/full-stack-fastapi-postgresql/issues/23#issuecomment-1470163345) ... do you think it's worthwhile to refactor the stack to switch as well?


Why not my proxy config is useless in the package.json with create-react-app project?

I had node server, so I run it at port 3000, and used axios to request the data.
My react project port at 3200, so I had used the proxy config at package.json.
package.json

request.js

query

network

It looks the proxy is useless, so the request code is 404， how can I fix it, plz help me, thx

Hi @Rain120 , could you add some steps to reproduce this? Adding create-react-app version might also help.

Hi! @kasisnu , srr， It's my fault, I had forgotten it.
The Github repo
Steps

run the node server at port 3000
add request.ts about axios config
add the proxy config at package.json
restart the project

create-react-app version

react version

axios version

It's my fault, bcs my request don't have '/ api' at node server, so I remove the value to / with the property about pathRewrite at package.json.
plz close the issue, thx your help


[ISSUE][CLOSED] [Bug] Per-PR CI check doesn't work when PR is updated with git push -f

 Issue by myungjoo-ham
Tuesday Jul 03, 2018 at 04:36 GMT
Originally opened as https://github.sec.samsung.net/STAR/TAOS-CI/issues/80

Condition:

A PR is already made and Per-PR CI check is done
The PR owner updates the PR with git push -f

Actual:

Per-PR CI does not work. There are two CI pending items and kept pending

Expected:

Per-PR CI restarts and finishes with either "error" or "OK".

Note that open sesame works anyway.

:octocat: cibot: Thank you for posting issue #104. The person in charge will reply soon.

 Comment by git-bot-sec
Tuesday Jul 03, 2018 at 04:36 GMT

:octocat: cibot: Thank you for posting issue #80. The person in charge will reply soon.

 Comment by myungjoo-ham
Tuesday Jul 03, 2018 at 06:02 GMT

ps. This happens "occasionally"... it worked this time..

 Comment by myungjoo-ham
Wednesday Jul 04, 2018 at 03:58 GMT

Not happening anymore. Strange...


Added yii\web\ErrorHandler::getTypeUrl()

Makes it possible for classes that extend yii\web\ErrorHandler to provide their own informational URLs for types/classes/methods.

Overall a good change.

https://github.com/yiisoft/yii2/commit/cf0d582a0b7c9afee92ace1f107a3de483af9dac and https://github.com/yiisoft/yii2/commit/06564db6275c86f7bab704616fdf3b541d40589b. Thanks!

Awesome!


Support for local mongodb

Hi,
does it work for local mongodb, for example, to work in a development/local environment, or I must have a mongodb on azure with cosmosdb? I mean, for production I will use for sure cosmosdb, but I need a local mongodb for development.
Thanks!

Nevermind bro! I already test it and works ok, BTW your Weapsy.CQRS is very cool!

Thanks @vany0114, glad you find it useful :-)


Switch to using ConnectionManager

The goal of Adafruit_CircuitPython_ConnectionManager is to make using sockets easier and the pool (socketpool, adafruit_esp32spi_socket, etc) managed in one place.
Please see the followint Pull Request for more info

branch accidently deleted. Re-opened here


Feat/add google map in hotel bid

resolve #45

https://github.com/lelandrichardson/react-native-maps docs 참고


feat: Allow user to use source images

I have a plan to use multi-stage builds for this instead of overlaying all the modules into a single bind mount. This way we can allow the image name to be the key and allow the use of 3rd party modules that could have the same name and not collide.

Realized my initial implementation was way more complicated than it needed to be. Instead we just use the source directly in the /module bind mount. This makes it so that the source will not overwrite any user defined modules and will only be used for the one module it is defined on.


Fix boolean parse issues

This change is

I got this parse issue too today, would be great to see this fix in the pushed task

I just saw this today and spent 4 hours digging to get to the bottom of this. Please merge this fix. People are wasting countless hours because they expect this to work and then you go and recheck everything in your CI and end up finding it's a bug in the VSTS task itself. Super annoying.

+1


Improved configuration

This pull request will contain changes regarding user configuration.
Currently, I moved morse codes definitions into separate files.
Now editable files by user are in user folder:
config.py - previous userConfig.py
morse_code.py - contains definitions of keyboard and mouse morse codes
morse_code_shortcuts.py - contains definitions of morse codes that used in macro mode.
userPinMap.py - it was just moved into user folder.
This is just first attempt so it can be changed in progress.
About languages support.
Typing letters in other languages can work now but predefined strings won't work with non us language set on target device.
Should I add keymaps structures to make predefined strings work with other languages?
And how should we allow user to change keymaps?
The only options I see now is additional morse command or manually redefining it in config.

that looks good to me. Nice!
re: "Should I add keymaps structures to make predefined strings work with other languages?"
Got any thoughts what that would look like? I cant imagine it..
Re: "And how should we allow user to change keymaps?"
I was wondering - do we have say a UK_keymap.json (or py.. ) - file - like we do at https://github.com/AceCentre/RelayKeys/tree/master/cli_keymaps
and then just add more of them to a keymap directory as needed?
(Keymaps would then be just a user configurable variable. use_keymap = "UK_keymap..")

I'm feeling userPinMap needs to go in root - or x80pinMap needs to go in user (or we somehow combine those files). Its jut not going to be a commonly edited file.

Got any thoughts what that would look like? I cant imagine it..
Re: "And how should we allow user to change keymaps?"
I was wondering - do we have say a UK_keymap.json (or py.. ) - file - like we do at https://github.com/AceCentre/RelayKeys/tree/master/cli_keymaps

Yes, that's will be dictionary in py file or json file, that will be mapping characters to keycodes.

I'm feeling userPinMap needs to go in root - or x80pinMap needs to go in user (or we somehow combine those files). Its just not going to be a commonly edited file.

ok, I will move it to root.

And how should we allow user to change keymaps?
The only options I see now is additional morse command or manually redefining it in config.

Here I wanted to ask how user will be swapping or switching between available keymaps.
Editing them will be no problem.

I cant see why anyone would swap on the fly - so I just think it needs to be a flag in the config. that answer your question?

ok, keymap will be chosen in config or something like that.

I added keymaps for handling predefined strings. For now there are us and uk keymaps.
Keymap can be set here: https://github.com/f1andrew/morAce/blob/1fdbe5b0ae4060a9e767525195cb0166010e63d7/morAce/user/morse_code.py#L4
Also, I changed morse codes structures to be dictionaries.

Great - so If Im right it means you could have a macro of say:
".-"     : "$$",
and it would look up dollar sign and send the correct key code for the defined keymap right?
Looks nicer in a dict

Yes, that's right.

Amazing stuff. Thanks!


Successfully compose Fast-JX and SuperFast

To update the codes of composing Fast-JX and SuperFast. I will update the documentation tomorrow.

Codecov Report

Merging #21 (f54eb68) into main (6005293) will decrease coverage by 71.61%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main      #21       +/-   ##
===========================================
- Coverage   82.99%   11.38%   -71.61%
===========================================
  Files           3       11        +8
  Lines         147     1089      +942
===========================================
+ Hits          122      124        +2
- Misses         25      965      +940

Impacted Files
Coverage Δ

src/composing_models/Building_model.jl
0.00% <0.00%> (ø)

src/composing_models/Fast-JX.jl
0.00% <0.00%> (ø)

src/composing_models/SuperFast.jl
0.00% <0.00%> (ø)

src/SuperFast.jl
100.00% <0.00%> (ø)

src/Lawfunctions.jl
0.00% <0.00%> (ø)

src/RateLawUtilFuncs.jl
0.00% <0.00%> (ø)

src/GasChem.jl
100.00% <0.00%> (ø)

src/Geoschem.jl
0.00% <0.00%> (ø)

src/Ratesfunctions.jl
0.00% <0.00%> (ø)

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


Network: Don't use stable volatile MAC when using bridge.external_interfaces with no bridge IPs

Adds check to avoid using stable volatile MAC addresses when using bridge.external_interfaces and no bridge IPs to avoid possible MAC conflicts in a clustered environent where multiple nodes are connected to same network segment.
Fixes a bug that prevented updating/removing a network's node-specific keys when in non-clustered mode.

Testsuite passed


Black Screen in Text Extractor

Microsoft PowerToys version
0.72.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
TextExtractor
Steps to reproduce
Opening the TextExtractor using shortcut 🪟 + Shift + T.
✔ Expected Behavior
The screen should been partial opaque (translucent) and we have a crosshair which we can drag to select a portion of the screen. I have attached a video regarding the same.
❌ Actual Behavior
Instead of above I am facing a black screen as demonstrated in the video.
https://github.com/microsoft/PowerToys/assets/80639377/ccf51974-9853-47f9-b9de-7e006eae0819
Other Software
No response

possibly related to #28177

Ok will apply the settings and let you know
On Wed, 30 Aug, 2023, 11:15 pm Sean Singh, @.***> wrote:

I had this exact same issue in #28177
https://github.com/microsoft/PowerToys/issues/28177. The problem for me
was that I had a global effect applied in Mica for Everyone.
Set the Global Rule on the left pane to what I have in this screenshot and
that should fix it. Though, this only applies if you have Mica For Everyone
installed and running.
[image: image]
https://user-images.githubusercontent.com/38306929/264428108-2812cf16-91fe-4070-9cca-1d88ce5f6271.png
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/28217#issuecomment-1699597923,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATHHLEOJ5QZ7GJRJAER7BKDXX533JANCNFSM6AAAAAA4EKQOZE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I solved it too following the advice.
Thanks 😊

You don't need to set the global rule, you can set a process exclusion:

Open MicaForEveryone
Click "Add Rule" in the bottom left corner
Enter "PowerToys.PowerOCR" as the process name
Voilá! By default all new rules are default (aka none) configuration.


attempt to index local 'mellifluous_config' (a nil value)

I get this error when I try to change vim.opt.background.
Error detected while processing /Users/strash/.local/share/nvim/lazy/mellifluous.nvim/colors/mellifluous.lua:
E5113: Error while calling lua chunk: ...ifluous.nvim/lua/mellifluous/colors/sets/mellifluous.lua:39: attempt to index local 'mellifluous_config' (a nil value)
stack traceback:
        ...ifluous.nvim/lua/mellifluous/colors/sets/mellifluous.lua:39: in function 'get_bg_light'
        ...im/lazy/mellifluous.nvim/lua/mellifluous/colors/init.lua:68: in function 'get_colors'
        ...hare/nvim/lazy/mellifluous.nvim/lua/mellifluous/init.lua:19: in function 'load'
        ...hare/nvim/lazy/mellifluous.nvim/lua/mellifluous/init.lua:31: in function 'apply'
        .../share/nvim/lazy/mellifluous.nvim/colors/mellifluous.lua:12: in main chunk
        [C]: in function 'nvim_set_option_value'
        vim/_options.lua: in function '_set'
        vim/_options.lua: in function '__newindex'
        /Users/strash/.config/nvim/lua/map_util.lua:114: in function 'toggle_background_color'
        /Users/strash/.config/nvim/lua/map.lua:55: in function </Users/strash/.config/nvim/lua/map.lua:55>

This is very likely due to the same issue I rose just a few hours ago (#54 ). My temporary workaround for that is just comment these lines out:
https://github.com/ramojus/mellifluous.nvim/blob/d17125ff8954fd2ca1db4aaee625f934f9f4d3e4/lua/mellifluous/config.lua#L84-L87
I think this might as well address your problem?

@TinusgragLin it does, thank you!

I've managed to replicate and then fixed the issue, so it should work now. Thanks for reporting!


remove core plugin path

Not used as this constant is available in the core app.

Coverage remained the same at 100.0% when pulling a01b3c6db94a2e6f58c6f9458fff4dadf89deba0 on remove-core-path into 36842ff304dac55dee2cd369f64a7a6a81e7a385 on develop.


Contribution Guidelines

Hey, I notice despite 2.5k stars and 70+ forks at time of writing, there aren't yet any contribution guidelines or published etiquette on how to contribute. Would you be able to add something like this to the README?

The project is out since only one month ^^
We are working on some big refactor.
After that, we will add more documentation on how to contribute.

Perfect, and on that note, please add documentation on how to submit pull requests to documentation :D
I'm happy to help wherever I can, I'd like to start using this tool myself but it's not mature enough yet. Willing to help it get there, though!

You are so nice, thanks a lot.

发件人: Tai Groot<EMAIL_ADDRESS>发送时间: 2020年4月18日 12:11
收件人: monitoror/monitoror<EMAIL_ADDRESS>抄送: Subscribed<EMAIL_ADDRESS>主题: Re: [monitoror/monitoror] Contribution Guidelines (#299)
Perfect, and on that note, please add documentation on how to submit pull requests to documentation :D
I'm happy to help wherever I can, I'd like to start using this tool myself but it's not mature enough yet. Willing to help it get there, though!
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/monitoror/monitoror/issues/299#issuecomment-615552879, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOWJNXZPIN6FIBM3WQHFO4DRNER7NANCNFSM4MEJ4CEQ.

nice !

大家好

i wnat to contribute to opne sourse to so if any project i can contribute so please update me too regarding that

WoW~


Explore circle drawing tool - Places of Interest not working

From https://forum.inaturalist.org/t/places-of-interest-misbehaving-when-using-circle-drawing-tool/42324 I can replicate.

Go to Explore page
Use the circle drawing tool to draw a circular boundary
Once Places of Interest reloads, click on it.
The places listed are not related to the boundary that was drawn. They're places written in Cyrillic script.

eg here's what I see after drawing a circle in the Bay Area:

(the rectangle drawing tool appears to be working fine)

With a circular selection the frontend issues a nearby places query with lat and lon set, but no radius.
The API does not expect these parameters and tries to bail: https://github.com/inaturalist/iNaturalistAPI/blob/main/lib/controllers/v1/places_controller.js#L84-L88
The calling code incorrectly detects the bail (undefined vs null) and issues an unconstrained query for all places: https://github.com/inaturalist/iNaturalistAPI/blob/main/lib/controllers/v1/places_controller.js#L154

I think this just never got implemented.
One option is to always pass swlat, nelng, etc, but to also pass the shape to the API. I haven't looked at the ES API to see whether a circular query is an option.


Translation: Catalan (ca-ES)

Repository: https://github.com/ygneo/opensource.guide
Preview: https://ygneo.github.io/opensource.guide
Locale: ca-ES
On your fork:

[x] Update title, description, and locale in _config.yml, and push the branch to the gh-pages branch of your fork. This should build the website and it should be visible at the preview url above.
[x] Copy _data/locale/en-US.yml to a new file with the name of your locale and translate the strings
[x] Create a new directory in _articles/ for your locale and translate each guide:

[ ] best-practices.md
[ ] building-community.md
[ ] code-of-conduct.md
[ ] finding-users.md
[ ] getting-paid.md
[ ] how-to-contribute.md
[ ] leadership-and-governance.md
[ ] legal.md
[ ] metrics.md
[ ] starting-a-project.md

[ ] Update CNAME to include your locale (e.g. de.opensource.guide)

Once a translation is completed:

[ ] Open a Pull Request on the upstream repository to add your fork to translations in _config.yml

@ygneo Apologies this issue has been neglected for so long. I've merged the first translation to this repository (Spanish) and updated the documentation on the latest translations approach (https://github.com/github/opensource.guide/blob/gh-pages/docs/translations.md). Please submit a pull request when you have translated all articles and checked ./script/test passes and I'll get it merged in ASAP. Thanks!

I would like to contribute to Catalan translation.
I follow this documentation.
This is my fork.

I would like to contribute to Catalan translation.
I follow this documentation.
This is my fork.

@fv3rdugo Please submit a PR when you're done, thanks!


Current issues with hypixel

It seems that our bot is no longer showing the correct status for the mc.hypixel.net server. We are looking into the situation and working on a fix right now. In the mean time, if you find more servers that are reporting incorrect status (after waiting for 6 minutes for discord to update), please put the server URL, expected response, and actual response in a reply below.
Please bear with us as we work through this as it might take some time, we are a couple of college kids working on this in our free time.
Thank you!

This issue seems to have resolved itself. We believe that this may be caused by one of the hypixel servers not sending statuses back as out bot expects. We will continue to monitor this situation but we are going to call this issue fixed for now.


Bracelet of Slaughter charge resets to 30 on every proc

I have been bursting Nechs while wearing a Bracelet of Slaughter (and having a spare in my inv) and have noticed that everytime the bracelet procs and uses a charge, the charge counter resets back to 30.  After it resets to 30 I have to check the bracelet again to correct it, only for it to occur again on the next proc.
Issue:
Bracelet charge count resets to 30 each time it procs.
Reproduce:
Have the correct charge count showing, and have it proc.
Not sure if it only occurs when bursting or if it only occurs when you have a spare in the inventory.
Expected Behaviour:
Each time the bracelet procs the charge count should decrease by the amount of times it procs.  Eg: bracelet has 3 charges displayed, procs once, goes down to 2 charges displayed.
Clip of it in action
Windows 10
Latest version of the client (dunno how to see what version exactly, sorry)

We think this is due to the message which is shown when the bracelet activates having changed?

I play on filtered so I wouldn't know if there is a message for each individual charge change.

Without the filter the prompt says: "Your bracelet of slaughter prevents your slayer count from decreasing.  It has x charges left."


Country select box Improvements

=> country select box should be improved
=> if it is changed to dropdown menu it should be best for the viewing

I woud like to work on this please assign it to me.


modem: cmux: Fix coverity issues

Possible NULL pointer dereferences where discovered by static code analysis, they are addressed in this commit.
Fixes #65367
Fixes #65363
Fixes #65362

@bjarki-trackunit there are three bugs linked to this PR, but I'm not sure why they didn't automatically close when the PR was merged. Will you please review/close them if appropriate?

@MaureenHelm I think they are "false positives", I tried to address them in a way that coverity check would accept, but it just spawned new issues with dead code https://github.com/zephyrproject-rtos/zephyr/issues/66841 instead... and reopened these... working on fixing it :)


Refactor Kubestone labels

Currently, we are setting two labels in the created k8s resources: kubestone-cr-name and app. It would look much nicer if we would be consistent and use our FQDN: kubestone.xridge.io/cr-name and kubestone.xridge.io/app

Hi! @otto-von-bivouac
Can I work on it?

Hi @amitlevy21 !
Of course, and thanks for providing a helping hand, much appreciated.
You can find our dev guide here: http://kubestone.io/en/latest/devguide/
Please let us know if you need any assistance.


Remove metrics from "Checks" section

Hi,
I think should be a nice if all metrics are placed in a new side button on the left. So we will see only checks in the checks sections and only the metrics in the metrics section.
Regards

I tend to believe that we should add some additional filters to the checks view instead, so you can easily switch from non-metrics to metrics checks.

Important to note that a single check can ship metrics and compare those metrics to static alert thresholds.

I'm asking for this because when you have a lot of metrics and checks, uchiwa ui is not clear.

Checks section is all messed up when you have more than 30 checks+metrics definitions, so some kind of filtering or metrics separation would be really helpful.

+1 This would be very useful.

Also looking forward to this feature.
Is there any plan on implementing it in the near feature ?

Related improvement coming in Sensu 0.24: https://github.com/sensu/sensu/issues/1025

great!


Feature request: Detect comment column reach

Hi!
Does JSHint have support for detecting how many columns comments have reached?
/**
 * This is a comment reaching a column number.
 */
function foo() {
  return "bar";
}

Can I configure a max column number for the comments and let JSHint report if there are comments reaching beyond the predefined max? This would be useful for sustaining code quality.
BR
Natanael

JSHint is a tool that helps to detect errors and potential problems in your JavaScript code

Get some good text editor/IDE and configure it to hint you if some line exceeds the X characters limit.

Yeah, that's a proper solution, definitely. Thanks. I still see a value in JSHint warning me my code and its comments doesn't follow the rules I might have about column limit. Especially if there are several people working on the same code and their IDEs aren't configured with auto-wrapping on that column.
As you quoted, if JSHint is about detecting errors and potential problems in my code - shouldn't code style also be a part of that problem?

Hi @nattelog,
No programming/scripting language enforces what programming style you should use. There are recommendations, but the languages themselves won't stop working because of different programming style, e.g "use 4 spaces and not 2/6/8/10/20 spaces".
It's not a program/script error to minify and mangle all the javascript, C, C++ (you name it) language into single line and have it working, thus jshint cannot enforce you what you should and should not do.
Companies, as well group of people contributing to large projects are outlining in the project README.md the guidelines on how you should code, but again they won't reject your commits for exceeding the "spaces" limit.

Ok, I understand. Thank you, @wifiextender. It's just that it would be comfortable having it warn me about style issues. But that's perhaps out of scope for JSHint. Maybe there are other packages that can fix that.

Hey @nattelog,
There is a deprecated rule in jshint that allows you to specify the line limit.
{
  "maxlen": 30,
}

Oh, interesting @wifiextender! Does it on warn too long text in general or only comments?

@nattelog maxlen warns about lines exceeding the specified length, regardless of their content. I would advise against using it though, because (as @wifiextender points out) it is deprecated (JSHint's scope is limited to issues of code correctness)

Alright, thanks for the response, @wifiextender and @jugglinmike !


bug: documentation mismatch between README and Wiki

Did you check docs and existing issues?

[X] I have read all the plugin docs
[X] I have searched the existing issues
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.10.1
Operating system/version
Ubuntu 22.04.4 LTS
Describe the bug
There is one line in the README to install nvim-java (see here) while in the Wiki there are many lines (see here).
Another problem is that the README does not say where to add the line for the installation.
Steps To Reproduce

open the README and read the instructions to install
open the Wiki and read the instructions to install

Expected Behavior

The instructions are consistent.
The README says where to paste the line.

Fixed by
https://github.com/nvim-java/nvim-java/pull/335

The instructions are consistent.

I still see two different instructions, in the README:
return {'nvim-java/nvim-java'}

and in the wiki:
return {
  'nvim-java/nvim-java',
  config = false,
  dependencies = {
    {
      'neovim/nvim-lspconfig',
      opts = {
        servers = {
          jdtls = {
            -- Your custom jdtls settings goes here
          },
        },
        setup = {
          jdtls = function()
            require('java').setup({
              -- Your custom nvim-java configuration goes here
            })
          end,
        },
      },
    },
  },
}

The README says where to paste the line.

The README does not specify where to paste the line.

@lackovic In the read me, it shows how to add nvim-java for custom setup. If you don't know where to put it, then you should learn about the package manager first. In wiki, it shows instruction for Lazyvim distribution.

@s1n7ax in your previous message you wrote that you fixed the problems I reported. I simply pointed out that you did not fix them. What did you fix?

Doesn't matter. Nothing to be fixed


Hello 2017

This is a legit and very important PR. Hope you like it 😃

Awesome, thank you. @skyline75489 💥


Info on when counters were first introduced

Still need to fill in the remaining counters
👀 @josalem

There are some counters in that doc that specify version dependencies in the description of the counter, see line 110 in the doc. We should be consistent in how we specify this requirement. One other consideration: we should differentiate between AspNet v3.1 and .net3.1 since you can run AspNet 2.1/2.2 on top of .net3.1 right?

We should be consistent in how we specify this requirement.

Sure. I'll update everything to the new format

One other consideration: we should differentiate between AspNet v3.1 and .net3.1 since you can run AspNet 2.1/2.2 on top of .net3.1 right?

I don't think it's worth going into that level in detail. Especially given that the only supported scenario is running ASP.NET Core 2.1 on .NET Fx. If you're running ASP.NET Core 2.1 on .NET Core 3.1, you're already in unsupported territory

Works for me 😃


New pip resolver

@ivirshup why Ubuntu 16.04? Isn’t that super old?

AFAICT ubuntu 16.04 is what they use in most of their examples. Considering many of our users will be on academic clusters, I think old-ish versions of linux are reasonable to test against. The alternative would probably be ubuntu-18.04, which we should switch to when 16 is out of support.
Here are the options.

We should have used the newer LTS from the start, 16.04 is out of support in 5 months.


Clean up foot controller and tune walk

This PR does the following:

Moves foot controller functionality into a templated function to make general
Add ability to configure each servo with different proportional gain value
Tune real robot walk
Adds goal position to RawSensors when using Webots

Note: This removes keep level mode, which makes FootController redundant but I intend to add additional modes back in later PR's
Waiting on #1331

who was the walk tuned on


where can i get Power-AI-Engine repository?

i search it in github but i cannot find.

This package is optional, you can skip it if you don't use IBM POWER system.


actually invoke the stemmer

the docs on the snowball stemmer are very limited, but it appears that stem() is what actually runs the algorithm

good catch ;)


python2ruby and more tests

Pull Request Checklist
Is this in reference to an existing issue?
No
General

[x] Update Changelog following the conventions laid out on Keep A Changelog

[x] Update README with any necessary configuration snippets

[x] Binstubs are created if needed

[x] RuboCop passes

[x] Existing tests pass

Purpose
This PR ports the remaining python scripts to ruby, adding tests and additional functionality along the way.  The default execution intends to remain consistent to the original version.
New features

--apipath for everything
--all in metrics-graylog.rb pulls more data and changes naming to be similar to other plugins
--scheme in metrics-graylog.rb allows one to change naming
check-graylog-buffers.rb now understands new 2.10 and up buffer metrics and reports on them simialr to the old version

Known Compatibility Issues
None known.  One caveat is that for metrics-graylog.rb adding --all flag will switch naming.  I think this is clear from description, but please double check me on this.

gentle poke for review

LGTM. Nice on the tests and maintaining backwards compatibility.

Awesome, thank you Eric!  Could you do a release with these changes?  Is there anything else that should be done for that?
Thank you again for your help,
-n

Will cut a release soon.

You're awesome, thank you!  Please let me know if there's anything I can do to assist you or the sensu-plugins project.

1.0.0 is out on rubygems.


Bug: clicking on checkboxes in popup.html emits promise exceptions

To reproduce:
open page with bootstrap elements and click on any checkbox.
Details:
Exception: Uncaught (in promise) Error: The message port closed before a response was received.
Context

Fixed by adding sendResponse() into onMessage listener in content.js. Seems that sendMessage opens a port which expects response. If response is not there it will emits errors.


'Unsupported operand type' error prediction in README example

For Making a prediction part of the README,
r.json() contains this error:

{'message': 'TypeError("unsupported operand type(s) for -: 'str' and 'float'")'}

river and chantilly version:
('0.9.0', '0.3.0')

Hey! That example isn't supposed to work. It's just for the sake example and isn't tested or anything :)


feat/accommodate marker pipe and directive

This change accommodates the possibility for a maker pipe and -directive, separate from the ngx-translate pipe and -directive.
Example directive:
key-value

Example directive 2:
value

Example pipe:
{{ 'key-value' | marker }}

The plan is to fork @biesbjerg/ngx-translate-extract-marker and rerelease it as @colsen1991/ngx-translate-extract-marker with a built-in marker pipe and -directive as well, since @biesbjerg is not responsive in regards to the ngx-translate-extract projects.

Ref. @colsen1991/ngx-translate-extract-marker


fix(auth): Auth pages responsiveness

Fixed the bug you were commenting on the video 😉
Also, from a quick look on the project, you should really look into implementing the inputs decoration inside the input widget instead of using multiple containers (https://dart-lang.github.io/linter/lints/sized_box_for_whitespace.html)

Man this was awesome! Thank you and thank you to @Scount and @jorgelrj as well


Crop support

It may be handy to have a tightened up crop function built into imgscalr.
Has been requested in the past.

Fixed, added in 4.0


'tradeResponse' event never emitted

I noticed my tradeResponse handler was never called when a trade made by the bot was either accepted or declined. The handler looks like this:
client.on('tradeResponse', function(steamID, response, restrictions){
    console.log("Incoming trade response from " + steamID.getSteam3RenderedID());
    //[actual method body]
}

To make sure I wasn't doing something wrong on my end I added an extra console.log() call on top of the emitting method in trading.js:
SteamUser.prototype._handlers[Steam.EMsg.EconTrading_InitiateTradeResult] = function(body){
    // Is trade ID meaningful here? (sidenote: yes it is ;))
    console.log("tradeResponse id emit! "+body);
    this._emitIdEvent('tradeResponse', ..., ...);
}

Unless I'm messing up big-time, I'd venture to say it's broken. =)

Are we talking about trade offers here?

The use case here is:

Bot sends a trade offer to a user
Bot waits for the user to accept or reject, which should trigger this callback.

Unless I'm misunderstanding what this event's supposed to represent?

Yes, you're misunderstanding it. The event is for responses to trade requests, not trade offers. These things:

You might want to look into steam-tradeoffer-manager.

Aaah alright, apologies then! I'll check out the other package, thanks!


Hotfix docs build

Hotfix for #10681 that enables the docs build until we resolved the underlying issue.

Superseded by #10694


MarginContainer improvements

What is introduced by this PR:

Extra margins: top, right and bottom. The custom constant margin has been replaced by margin_left, margin_top, margin_right and margin_bottom (only for MarginContainer). All margins except margin_left are set to 0 by default, to avoid breaking compatibility with the editor.

I am adding the discussion label, since I am not sure if this may be against the original purpose of MarginContainer.

It looks good to me, but as you said I'm not sure either what was the original intention behind the previous margin parameter than only worked as a left-margin. @reduz?

Removed the need of EXPAND size flag to avoid breaking compatibility with the editor.

@reduz This one is ripe for review ;)


Document browser extension error handling

This adds some documentation on how the Bugsnag backend deals with js errors from browser extensions, and gives some suggestions for how to work around it if you have an unusual use case like monitoring an extension.
Refs https://github.com/bugsnag/bugsnag-js/issues/94

LGTM

:star2: Shippin' it! Thanks, @eanakashima.


[$500] Web - Chat - Emoji tooltip isn't disappearing when right clicked

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Have a text message in your chat with a reaction emoji
Hover over the emoji
Right click on it

Expected Result:
It should disappear
Actual Result:
Emoji tooltip isn't disappearing when right clicked
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/26256964-056f-45db-98dc-f4b9cb14de71
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01019598c5c839b930
Upwork Job ID:<PHONE_NUMBER>011954176
Last Price Increase: 2024-02-12
Automatic offers:

bernhardoj | Contributor | 0

Proposal
Please re-state the problem that we are trying to solve in this issue.
The popover is not dismissing the tooltip.
What is the root cause of that problem?
We have common tooltip logic to hide the tooltip if the related popover is shown. This is managed through refs.
What changes do you think we should make in order to solve the problem?
We should establish the ref between the  and  so that https://github.com/Expensify/App/blob/07d97efe708f6c609c52ed7b1740fc2a249a4947/src/components/Tooltip/PopoverAnchorTooltip.tsx#L11 can find the ref and hide the tooltip.
A good example of establishing the ref is https://github.com/Expensify/App/blob/07d97efe708f6c609c52ed7b1740fc2a249a4947/src/components/ThreeDotsMenu/index.tsx#L74C11-L74C20
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
The popover is not dismissing the tooltip.
What is the root cause of that problem?
We have common tooltip logic to hide the tooltip if the related popover is shown. This is managed through refs.
What changes do you think we should make in order to solve the problem?
We should establish the ref between the  and  so that https://github.com/Expensify/App/blob/07d97efe708f6c609c52ed7b1740fc2a249a4947/src/components/Tooltip/PopoverAnchorTooltip.tsx#L11 can find the ref and hide the tooltip.
A good example of establishing the ref is https://github.com/Expensify/App/blob/07d97efe708f6c609c52ed7b1740fc2a249a4947/src/components/ThreeDotsMenu/index.tsx#L74C11-L74C20
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
The popover is not dismissing the tooltip.
What is the root cause of that problem?
We have common tooltip logic to hide the tooltip if the related popover is shown. This is managed through refs.
What changes do you think we should make in order to solve the problem?
We should establish the ref between the <EmojiReactionBubble /> and <Tooltip /> so that https://github.com/Expensify/App/blob/07d97efe708f6c609c52ed7b1740fc2a249a4947/src/components/Tooltip/PopoverAnchorTooltip.tsx#L11 can find the ref and hide the tooltip.
A good example of establishing the ref is https://github.com/Expensify/App/blob/07d97efe708f6c609c52ed7b1740fc2a249a4947/src/components/ThreeDotsMenu/index.tsx#L74C11-L74C20
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
The reaction tooltip is still shown even when the reaction list modal is open which is inconsistent with other elements.
What is the root cause of that problem?
We have a PopoverAnchorTooltip that has the logic to hide the tooltip when the related modal is open, but we use a plain Tooltip for the reaction list component.
https://github.com/Expensify/App/blob/a3943ace4c70470d38a469ed62ab430c92c0e966/src/components/Reactions/ReportActionItemEmojiReactions.tsx#L7
What changes do you think we should make in order to solve the problem?

Use PopoverAnchorTooltip
import Tooltip from '@components/Tooltip/PopoverAnchorTooltip';

Taking over!

@bernhardoj's proposal looks good. I was the author of the PopoverAnchorTooltip component and this bug is the exact reason why we created it in the first place. As such, we should re-use that component again.
🎀 👀 🎀 C+ reviewed

Thanks, @allroundexperts!

PR is ready
cc: @allroundexperts

I think I understand why I lost this proposal. However, I do think I posted the same proposal and the usage of AnchorTooltip.  I didn't just say replace the Tooltip with AnchorTooltip.  I linked the line in which AnchorTooltip will handle this and posted an example of a working component that uses AnchorTooltip.

This PR was deployed to production two weeks ago. @kevinksullivan I think we're waiting on the payment here.

@kevinksullivan bump for the payment

Summary:

@allroundexperts $500
@bernhardoj $500


Is this org.bitcoin.wallet on google-play?

There is app on google-play: "Bitcoin Wallet: by Bitcoin.org"
https://play.google.com/store/apps/details?id=org.bitcoin.wallet
which use appid org.bitcoin.wallet but this repo use appid com.bitcoinwalletmobile , the org.bitcoin.wallet is used in another repo which is not a mobile wallet?! https://github.com/bitcoin-dot-org/Bitcoin-Wallet/blob/896565580cb1fe162f67bfc59124d8924231e347/vue.config.js#L8 so
So the question is this app in google-play yours? legit? if it's legit please point to it in the README of the repo and also git tag releases/version (so easier to test for reproducibility, etc..)

Yes, it's legit. I'm currently just testing for issues with live users from the app stores. Still need to do more work on improving this repo, as you suggested. Thanks for the heads up.

can you point to the commit used to build the 1.0 version on google-play? also it's better to mention the app in google-play in some places like README.md here, as due to missing mentions, no related tag, mismatch in appid, thought that the app in play-store was fake/scam...

https://github.com/bitcoin-dot-org/BitcoinWalletMobile/releases/tag/v1.0
Yeah, I totally get that, I'll update the README soon to point to the app in the Google/Apple store. It's obviously legit, but we're trying to be lowkey about it for now, we need to make sure users aren't losing money before we promote it more.

Thanks @emanuelb for finding this. I updated walletscrutiny accordingly. Personally I think it's asking for hate and trouble to release an "official" bitcoin.org mobile wallet but the current name "Bitcoin Wallet: by Bitcoin.org" shows clearly that ambition.


Document how to tests in-repo engines components.

So there is no way to test the component in the engine?

@knownasilya probably depends on https://github.com/ember-cli/rfcs/pull/60

@knownasilya - For in-repo engines, no not really.  This documentation isn't really about stand alone engines.

I hate to resurrect this issue a bit, but I found that this was necessary for me to integration test my helpers and components.