PNMP support

Howdy!
I've been trying to get this running on a private repo and quickly ran into the issue that the system only has support for specific package managers. It'd be nice to be able to provide custom commands to add support for PNMP or to have built-in support if that's easier

☝🏻 I was a bit surprised by that this doesn't seem to work with pnpm. I get the following error (though I'm removing parts of the directory names in the message):
Run cypress-io/github-action@v2
internal/fs/utils.js:220
    throw err;
    ^

Error: ENOENT: no such file or directory, open '/home/runner/work/--REPO--/--REPO--/package-lock.json'
    at Object.openSync (fs.js:440:3)
    at Object.readFileSync (fs.js:342:35)
    at Function.module.exports.hasha.fromFileSync (/home/runner/work/_actions/cypress-io/github-action/v2/dist/index.js:18048:54)
    at lockHash (/home/runner/work/_actions/cypress-io/github-action/v2/dist/index.js:6729:26)
    at getNpmCache (/home/runner/work/_actions/cypress-io/github-action/v2/dist/index.js:6739:16)
    at restoreCachedNpm (/home/runner/work/_actions/cypress-io/github-action/v2/dist/index.js:6784:21)
    at installMaybe (/home/runner/work/_actions/cypress-io/github-action/v2/dist/index.js:7365:5)
    at Object.<anonymous> (/home/runner/work/_actions/cypress-io/github-action/v2/dist/index.js:7388:1)
    at __webpack_require__ (/home/runner/work/_actions/cypress-io/github-action/v2/dist/index.js:24:31)
    at startup (/home/runner/work/_actions/cypress-io/github-action/v2/dist/index.js:43:19) ***
  errno: -2,
  syscall: 'open',
  code: 'ENOENT',
  path: '/home/runner/work/--REPO--/--REPO--/package-lock.json'
***

I removed package-lock.json because it was only used by npm, but it seems like there's still a dependency on it here. None of the other code in my workflow makes any reference to package-lock.json.

Now that I've looked into this a little more, it seems like this is a duplicate issue. The solution I found from issue 145 is to add install: false to the with property in my workflow.

@kitsune7 I think you're right! I'll close this one.


Verify the branch when workflow_dispatch is used

Im not sure we have tested this. If a user manually runs scorecard and select a different branch, does the run succeed, and are the results accepted server-side (they should not)?
/cc @azeemsgoogle @rohankh532

Marking as duplicate of #81


8443 port for director is not mentioned for 2.0,2.1 and 2.2 documentation

https://github.com/pivotal-cf/docs-pcf-security/blob/6aa2f4b3a0083fc6f8e28a296ceb9f0f562ae5ff/networking/loggregator-network-paths.html.md.erb#L78
only 25595 is mentioned but user gets this error we are getting the following error in the bosh-system-metrics-forwarder logs on the loggregator nodes:
unable to get token: Post https://<director>:8443/oauth/token: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)

Thanks, @arunavasan  - docs now list both port numbers:
https://docs.pivotal.io/pivotalcf/2-2/security/networking/loggregator-network-paths.html


Make sure GrpcServerConfiguration matches ServerConfiguration

We should clean up and refactor GrpcServerConfiguration to support same usage patterns (builder, etc.) that the existing web server ServerConfiguration class uses.

I'm closing this now the config has been cleaned up. Other config changes such as adding SSL and adding native support etc, can be done in individual issues.


Video Tab - Tilemap, Collapse/Hide

ADD: Tilemap sub-panel (displays the full 80x40 tilemap)
ADD: Collapse/Hide to video sub-panels
ADD: Remember video tab settings
REFACTOR: use general "box-sizing" for everything
REFACTOR: General spacing, shadows, etc

Collapsed

Hidden + Tilemap

Thanks a lot for this feature, merged!


Update README.md

Took me a long long time to figure this out. Might help others making spiderable work

Thanks. Does https is required?

Yes, for SSL + spiderable it is. For just Meteor on SSL it is not necessarily needed.

Could you include that as well. Otherwise, someone might think SSL support is must to use spiderable.

@arunoda done!

@arunoda Done!

Awesome.
BTW: Now mup has SSL support too. I'm testing it right now. Will add the docs :)

Here's the SSL support docs: https://github.com/arunoda/meteor-up#ssl-support

Nice! But Node.js is not that good at serving http / https request right (static files mainly). That's why it is recommended to use a proxy like NGINX / Haproxy for static files when using Node.
If I'm gonna use it: will it also work with Meteor Cluster?

If you are using, Nginx as a static file server. Yes Cluster will still work.
Cluster works on the application layer. So, it's fine.


Fix missing httpd

Could not get httpd to work, so switched to php5

Fixes https://github.com/cywolf/schemaspy-docker/issues/2


Valid Anagram JavaScript

File(s) Modified: [242-Valid-Anagram.js]
Language(s) Used: [javascript]
Submission URL: [https://leetcode.com/submissions/detail/755783365/]

hi @akfisafk ! thanks for contributing!
even though your solution does save some space (though not in terms of Big O), i'd prefer to stick with the original because it more closely mirrors neetcode's solution in his video for this problem. if you'd like to edit the solution to more closely reflect neetcode's video (i think there's a little cleaning up that could be done) you're more than welcome!

Hi @mitchellirvin! Appreciate you reviewing this, that makes sense. I'll take another look and see if I can propose a cleaned up solution based off of Neetcode's original solution.
Thanks again!


Add tests for parser.py

I don't think we have a command line argument named key in our code. Do we need to use this parameter in @mock.patch?

key represents the internal assignment of the api_key destination which is initialized in parser.py.


add language switcher for mobile visible next to hamburger menu

similar to desktop version:

but maybe no icon and just two letter language codes and arrow?

@ttomasz i've just commited on /dev new changes addressing this issue - https://github.com/openstreetmap-polska/openaedmap-frontend/commit/80a5441406b51572b70b7dc7a84a481fa2051eb0
Take a look. It isn't perfect, but i think it's better than previous solution

I've just checked on mobile and it doesn't look good

continued in #63


unbreak OS_Unix::get_executable_path() on OpenBSD

(and possibly also on NetBSD)
On the OpenBSD port, we've carried a local patch to hardcode the path to the godot binary for a long time.  Recently, I had to revise it (since we're now building godot with and without tools enabled) and I'm wondering how the original code could have ever worked.
The current OS_Unix::get_executable_path re-uses OS::get_executable_path (which in turns is argv[0]), does a realpath(3) on it and return the resolved string.  This basically only works when argv[0] is an absolute path and fails badly when Godot is invoked as godot, for e.g. from the shell.  Why don't just re-use argv[0] as in the default implementation?

The aim of this method is to return an absolute path to the executable AFAIK.
Falling back to argv[0] should only be done if it's not possible to get the absolute path. Is there no way to retrieve that path on OpenBSD?

OpenBSD doesn't provide a syscall to retrieve the path to the executable intentionally.  I think that semarie@ reply in https://github.com/ziglang/zig/issues/6718#issuecomment-711134120 explains better that I ever could why something like this is not wanted on OpenBSD.
I'd avoid doing realpath(argv[0]) since that is a real shootgun.  I've tested a few shells (ksh, bash, csh) and they don't put an absolute path in argv[0], they simply set it to "godot" (which is really /usr/local/bin/godot here.)  Now, if I happen to have a file in $PWD called "godot" (not at all surprising) realpath would succeed.
So no, I think that just exec(argv[0]) is the simplest, easiest and safest thing to do.  it already handles for us the case of an absolute path to the executable or the $PATH handling.

So no, I think that just exec(argv[0]) is the simplest

get_executable_path() is used to detect self-contained mode and load GDExtension libraries, it's not just for running new instances of the engine (it's used in OS::create_instance for this purpose as well).

It's not possible to set a default path for GDExtension libraries at compile time?

It is possible to set absolute path in the GDExtenson config, but it's usually relative. Executable path is one of the locations the engine is searching for lib if path is relative (it's a current directory, executable path, and executable path + "../lib"). So non-working get_executable_path won't make it unusable.
Self-contained mode assumes the executable can be located anywhere, all configs, etc. are stored in the same folder as the executable (and this folder can be freely relocated).

OK, so I guess the quickest option is to look at argv[0] and see if we can deduct a working path from it.  ugly, racy, but don't see other way around :)
will look into it tomorrow and update the PR :)

Sorry, it took a while to fix this because of other complications in building godot from the master branch on OpenBSD.
I've updated the PR to do what suggested, if argv[0] looks like a path and realpath(3) likes it, use that, otherwise search for argv[0] in the $PATH.  Tested (in a branch with other misc tweaks so that it builds on OpenBSD) and it seems to work both when argv[0] is a path (./bin/godot-...) and when it's not (as when running godot from the shell)

Might I suggest this?
 (Not sure if Godot actually needs this for anything):
https://github.com/time-killer-games/xproc/blob/9ff65440262a327115923f7c56e9d38f8921b1ad/process.cpp#L907-L1007
Or if you just need for the current executable, this might help things a bit as well for less failure cases:
https://github.com/time-killer-games/libfilesystem/blob/ea009efea523439455327552f12fc71fa30c1e50/filesystem.cpp#L565-L680

I think the way to do this in a portable way is something like:
#include <errno.h>
#include <unistd.h>
#include <string.h>
#include <stdlib.h>
#include <stdio.h>
#include <limits.h>

long get_pathmax(void) {
  long pathmax = -1;

  errno = 0;
  pathmax = pathconf("/", _PC_PATH_MAX);
  if (-1 == pathmax) {
    if (0 == errno) {
      pathmax = PATH_MAX;
    } else {
      fprintf (stderr, "pathconf() FAILED, %d, %s\n", errno, strerror(errno));
    }
  }

  return pathmax;
}

int main(int argc, char* argv[]) {
        char* fullpath = malloc(get_pathmax());
        char* cwd = getcwd(NULL, get_pathmax());

        if (!cwd) {
                fprintf (stderr, "getcwd() FAILED, %d, %s\n", errno, strerror(errno));
        }

        if (argv[0][0] == '/') {
                strncpy(fullpath, argv[0], get_pathmax());
        } else {
                strncpy(fullpath, cwd, get_pathmax());
                strncat(fullpath, "/", get_pathmax());
                strncat(fullpath, argv[0], get_pathmax());
        }

        char* fullpath_canonical = realpath(fullpath, NULL);

        printf("full path = %s\n", fullpath_canonical);
        free(cwd);
        free(fullpath);
        free(fullpath_canonical);
}

this would go in our unix main() function, and we'd have to store fullpath_canonical on the OS:: singleton somewhere I think.
Example output:
[hp@lilly tmp]$ ./test
full path = /home/hp/tmp/test
[hp@lilly tmp]$ /home/hp/tmp/./test
full path = /home/hp/tmp/test
[hp@lilly tmp]$ ../tmp/./test
full path = /home/hp/tmp/test

I think the way to do this in a portable way is something like

This seems reasonable, but it's possible that I might be missing some BSD specific aspects.

@hpvb nope, that's wrong.  Ignoring the issues with strncpy which can overflow called like that (it's strlcpy and/or snprintf that you want to use), argv[0] is still not guaranteed to be a path.  Let me show an example:
% cd /tmp
% cat > me.c
#include <stdio.h>
int
main(int argc, char **argv)
{
        puts(argv[0]);
        return (0);
}
% make me
cc -O2 -pipe    -o me me.c
% cp me ~/bin
% which me
/home/op/bin/me
% me
me
% ./me
./me

In the first case, where me was found on the path, argv[0] contains only the name of the utility and not its path.  It's only when called by its path (as in ./me) that argv[0] may contain the actual path.
The shell in the example was OpenBSD' ksh, but with bash it's the same story:
bash-5.2$ which me
/home/op/bin/me
bash-5.2$ me
me
bash-5.2$ /tmp/me
/tmp/me

So, I still think that something like my PR where we walk $PATH in the general case it's the way to go.

@hpvb nope, that's wrong. Ignoring the issues with strncpy which can overflow called like that (it's strlcpy and/or snprintf that you want to use), argv[0] is still not guaranteed to be a path. Let me show an example:
% cd /tmp
% cat > me.c
#include <stdio.h>
int
main(int argc, char **argv)
{
        puts(argv[0]);
        return (0);
}
% make me
cc -O2 -pipe    -o me me.c
% cp me ~/bin
% which me
/home/op/bin/me
% me
me
% ./me
./me

In the first case, where me was found on the path, argv[0] contains only the name of the utility and not its path. It's only when called by its path (as in ./me) that argv[0] may contain the actual path.
The shell in the example was OpenBSD' ksh, but with bash it's the same story:
bash-5.2$ which me
/home/op/bin/me
bash-5.2$ me
me
bash-5.2$ /tmp/me
/tmp/me

So, I still think that something like my PR where we walk $PATH in the general case it's the way to go.

I think my example is still the most bullet proof solution, even if it still does have failure cases. It will allow you to check the exact ino_t/dev_t numbers and see if they match up with an existing hardlink to the executable, and this will give improper results if more than one hardlink to the exe exists, though very unlikely people would create multiple hardlinks to a game executable. If you really wanted to, you could check if the number of hardlinks is greater than one, in which case the function could either fail or print a warning to console.

@time-killer-games I haven't ignored your suggestion, it's just that i filed the PR beforehand and would prefer if godot didn't link to libkvm just for this.
Also, please note the BUGS section of kvm_open(3):

There should not be two open calls.  The ill-defined error semantics of the Sun library and the desire to have a backward-compatible library for BSD left little choice.

So, yeah, I wouldn't use libkvm for anything new, and especially in a program like godot where OS_Unix::get_executable_path() is called multiple times by different places (possibly by different threads too?)

Thank you, that is important to know. Perhaps make the kvm_t * returned by kvm_openfiles() be a global or a static local would solve this? Also, I doubt using libkvm is absolutely necessary. There should likely be a way to do this purely with libc, though that would require extra research for the implementation.

There should not be two open calls. The ill-defined error semantics of the Sun library and the desire to have a backward-compatible library for BSD left little choice.

I got a second opinion from someone on the FreeBSD discord, and they said this:
I might misread it but I think the bugs section states that ideally the kvm_open() and kvm_openfiles() was one function, but they are both there for compatibility reasons

meaning there really was meant to be one kvm_open* call/function but with the desire to improve the function as well as to keep backwards compatibility they had no choice but to split it into two different functions.


bugfix/naivepnl_facet_plots is still alive.

Codecov Report
Base: 59.01% // Head: 59.32% // Increases project coverage by +0.31% :tada:

Coverage data is based on head (26baeb6) compared to base (cf5b1cc).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #649      +/-   ##
===========================================
+ Coverage    59.01%   59.32%   +0.31%
===========================================
  Files           28       29       +1
  Lines         2991     3044      +53
===========================================
+ Hits          1765     1806      +41
- Misses        1226     1238      +12

Impacted Files
Coverage Δ

macrosynergy/pnl/naive_pnl.py
56.96% <ø> (ø)

macrosynergy/panel/linear_composite.py
77.35% <0.00%> (ø)

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

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


[BEAM-14299] Do not pass null as default value to MoreObjects.firstNonNull

BEAM-14299: if streamName.read() returns null, firstNonNull will throw NullPointerException. We check the emptiness in the next line so it's ok for stream to be empty string, we should use empty string as the default value.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Can one of the admins verify this patch?

LGTM

Run Java PreCommit


When we can expect Infinite scrolling

Hi,
Page menu is very good. But i need infinite scroll. Will you please update the infinite scroll.

Please update this. i need this too


Resources publishing issue

Using PowerBi reporting Services over SQL 2016:
When publishing Resources files (such as images), file extension is lost, and MimeType provided in CreateCatalogItem() is not identified as such by RS. Once uploaded, file is recognized as application/octet-stream.
The $fileName provided to CreateCatalogItem should contains the complete FileName (extension included).
$fileName = [System.IO.Path]::GetFileNameWithoutExtension($_.FullName) + [System.IO.Path]::GetExtension($_.FullName);
Not sure the best to do it in Powershell :)
Still:, nice Tasks, thank you.
A few additional enhancements:

Creating Target folder path would be nice to avoid having to create them manually
Include the rdsFileCount and rsdFileCount as part of the if statement, otherwise we might have an error if path is invalid even though the checkbox is disabled.

@PierrickLux  Next week I have time to review your issue. (got a few deadlines this week). Ill keep you posted

@PierrickLux  You can now check the option  "Add the extension to the resource files" in the advanced section.
Can you let me know if this is now resolved (could not test it 100%)

@PierrickLux FYI, there is a new task, which can deploy the whole folderstructure you have on your local machine :)

@PierrickLux Is this issue resolved?

@ygeelen-tb Yep, perfect thx!
I owe you a beer :)

@PierrickLux You are welcome.
FYI Our address for the beer is:
Tobania Corda Campus - Gebouw I
Kempische steenweg 311 box 6.02
3500 Hasselt


Stuck on reboot or no wifi card

OS Version: macOS Sierra 10.12.1
Mac model: 2010 MacBook Pro 7,1
Dongle or replaced card?: dongle
Used dongle/card: gymble panbt400

Your Issue:
I tried to install continuity on my older MacBook. I booted into recovery and disabled sip. Then rebooted and installed. Everything seemed ok. I did set option to check on boot though not sure if that was right.
Then when I restarted it would hang with progress bar maybe 90 percent the way. If I recovery mode and enable sip, then I can boot fine but now I have no wifi card detected. Disabling sip leads to hanging again.

Having same issue on 2010 Mac Pro 5,1 on 10.12.1.  Been trying to restore it but I can't because sip needs to be off, but if its off, I can't login to uninstall...

I ended up reformatting and reinstalling. Not gonna touch this again.
On Thu, Nov 24, 2016 at 4:49 AM Kevin Segal<EMAIL_ADDRESS>wrote:

Having same issue on 2010 Mac Pro 5,1 on 10.12.1. Been trying to restore
it but I can't because sip needs to be off, but if its off, I can't login
to uninstall...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dokterdok/Continuity-Activation-Tool/issues/362#issuecomment-262746931,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACeYVHJdV2KlfW3BA1e-4Ry9FIOC6Z1bks5rBWuggaJpZM4KgL_Y
.

--
Sent from my iPhone

First of all, thank you to everyone who contributed to this. I have this working fine on a 2011 Macbook Pro running 10.12.1. I would really like for this to work on a Mac Pro 5,1 (upgraded Early 2009) and I'm having the same issues as those mentioned above.
OS Version: 10.12.1
Mac model: Mac Pro 5,1 (upgraded board/flashed - Early 2009)
Dongle or replaced card?: Dongle(s)
Used dongle/card: IOGEAR GBU521 & GYMLE (used separately :-) )

Your Issue: I can get my Mac Pro to recognize both dongles, SIP is disabled, CAT app diagnostics check out, activation "seems" to complete but upon reboot, it hangs on the boot screen. It boots up significantly slower and then completely freezes at about 95% complete bar on the boot screen. The only way to boot back up is by resetting the PRAM. But then my wifi is missing ('No WiFi hardware found' message). I also don't have the option for handoff in my general settings, either. Maybe the activation did not properly work? I noticed a few warning messages about it couldn't make a backup of some kext files because they were read only. Maybe I missed some other errors and this is why it's happening? I didn't think much of it because it didn't stop the process and it 'successfully completed' at the end. I can post the terminal messages if anyone can help out.
Steps done to try to fix the issue:  I tried different dongles. I tried enabling the continuity check on/off after reboot. Same result - Stuck on reboot. Have to reset PRAM in order to boot up successfully. No Wifi Hardware detected after reboot.
The only way to get my computer back to normal, I found, was to boot into Recovery Mode and reinstall Sierra. Even when I can boot into my machine (with no Wifi error), CAT will not let me uninstall because it says SIP is enabled (even though I never re-enabled it). So I was stuck in a bit of a loop that only a reinstall could help me. It's a bit of a time drain but I feel like if I can get this working, it'll breathe new life into the machine.
Thanks!

@YogaFlame
There seems to be an issue with your mac trying to load the patched wifi kext.
Any chance you could boot your mac in verbose mode and capture the output?
P.S.: Resetting PRAM "fixes" the issue since it resets System Integrity Protection to "on", this will cause the kext not to load anymore.. this will fix your mac hanging on the boot but also causes your mac to not be able to detect the wifi card.

@sysfloat
Here's a pic I took while the Mac Pro was re-booting in verbose mode after Continuity Activation. It hangs at this spot: http://goo.gl/tjpzra
Also, here's the log when I ran the CAT tool: http://hastebin.com/raw/ojoludobiw
Let me know what you can find out, thanks

I am having the exact same issue on my 2010 Mac Pro. After installing the patch, it hangs on boot-up.
When I installed the patch, it said it had only been tested on OS 10.11. I am running OS 10.12.2.
Is there another patch for OS Sierra, or has anyone found a solution to this issue?

Hello,
MacBook Pro 17" Early 2009. 10.12.x installed using dosdude1 patch. EXACTLY same issue, Additionally can't run using cmd+r because i see only grey crossed circle.
Any help, please?!

Ugh!  Same problem here!  I am able to boot to another partition running Yosemite.  I have access to the Sierra boot partition.  Can't I manually restore the original files?

same issue on mbp 5,1 with patched Sierra , is this project abandoned?

Here's how I fixed it...
(remove BT dongle)

Boot into another partition or boot disk if you have one.  (You'll need to be able to mount and edit the original boot drive.)

Hopefully you have backups of the two original Kext files.  Move them back to /System/Library/Extensions/ replacing the two new ones put there by CAT.

Now reboot.  System no longer hangs or kernel panics, but after a successful boot, all wireless hardware will not be found.  (Thanks to some clever Apple engineers, my wireless Apple mouse is still working despite no Bluetooth.)

Note:  SIP is still disabled.  Run CAT option 3, "Uninstall".  (Even though you've already put the original Kext files in place allowing you to reboot, option 3 rebuilds the Kext caches and a few other important things that will allow your original wireless hardware to get recognized on the next boot.)

Optional: When step 4 is complete and you're prompted to reboot, hold down command-R and enter Recovery Mode, where you can re-enable SIP by entering csrutil enable at a Terminal prompt.

Reboot - everything back to normal without needing to reinstall the OS!

Hello,
This solves the boot hang but doesn't solve the problem of CAT not working. Seems this issues is cropping up in a few threads. Any ideas why the tool doesn't work for Sierra? Is the hang issue related to the backup failure or something else?

I gave up and bought a new Wifi/Bluetooth card so that I won't need CAT anymore.

As I have the very same issue on a 2010 Mac Pro (machine does not boot after CAT, hopefully the Time Machine backup "fixes" the unbootable Mac) I would also not recomment CAT with Sierra. @sparky672 - which Wifi/Bluetooth card did you buy? Will probably go the way you went :) Thank you.

@gpsloco, my procedure above was a lot easier than it may seem and only took a few minutes plus the time to reboot a few times.
I ended up with the upgrade by OSXWIFI dot com and purchased it through Amazon where I saved a buck on shipping.  Price looks different because the item included the Bluetooth antenna extension cable you'll need.  When ordering from OSXWIFI, you'll need to add this item onto it.
After installing the new card & cable....

Delete the Bluetooth prefs file.
Zap the PRAM.
Log out of iCloud.
REBOOT NOW <<- very important or steps 3 & 5 will do nothing.
Log into iCloud.


research spotlight PMID: 30044717

For tomorrow
Previously uncharacterized protein, Ync13, has a role during late stages of cytokinesis
originally published in  Mol Biol Cell

I've done the configuration, ready for the next update.

That new Spotlight is visible now: https://www.pombase.org/archive/spotlight

I have a new figure from the author with correct aspect ration. Includes key.

I've updated the Spotlight image.


Negative batch_limit during calibration on custom dataset

Hello!
I'm currently trying to train a custom dataset of outdoor scenes with the following point data structure: x, y, z, intensity, class. I've used the code and folder structure for the S3DIS dataset as a template when processing the custom data. I ran into trouble when calling the calibration method in train_CustomDataset.py.
If num_workers=10, the for-loop (in the calibration method) that iterates through the dataloader hangs after 20 iterations. Not sure if this is more of a memory issue than a code problem, so I tried setting num_workers to zero. What happens is that the self.dataset.batch_limitvariable turns negative, mainly because b > estim_b, with b = len(batch.cloud_inds) and estim_b += (b - estim_b) / low_pass_T. The values of estim_b and low_pass_T are the same as for the S3DIS dataset.
I'm afraid that it is my lack of understanding of what is going on at a deeper level in your code, especially the potential sampling method, that keeps me from solving the problem. When num_workers=10, the variable b never exceeds 2, but when num_workers is zero b>1000. Could the problem originate from the Dataset object, and thus my preprocessing of my own data?
Any insight on the problem(s) would be very much appreciated :)
Cheers!
Maria

Hi Maria,
First, if you want more details for the potential sampling (and other parts of the code), you can have a look at my PhD thesis, I already mention this in a previous issue: https://github.com/HuguesTHOMAS/KPConv-PyTorch/issues/72#issuecomment-769293888
Try to search for the word "potential" in the previous issues on this GitHub, you will find some info in my answers to the various questions.
Now concerning the calibration method, I pushed a modification recently to make it faster, but at the price that it can be unstable, which could be the reason why your batch_limit turns negative. To make the calibration more stable (but slower), you can decrease the variable:
https://github.com/HuguesTHOMAS/KPConv-PyTorch/blob/aee5494f323aa8db523cfdcc7246562d082add00/datasets/S3DIS.py#L1162-L1163
It should be the same order of magnitude as the value of batch_limit that you expect to have in your network, which is the approximate total number of points in a batch (all input clouds of the batch combined). Can you tell me the value of the following parameters for your experiment:

in_radius
first_subsampling_dl
batch_num
Is the density of points in your data more or less constant or varying a lot?

A final comment: it is very strange that the behavior changes when you change the number of threads... In my opinion, this should not happen, and it could mean there is a bug in your Custom Dataset class implementation. What are the modifications that you made from the original S3DIS dataset? If you follow the same folder structure, you should have nearly no modification to make (only the part where you load the intensity instead of the color).

Thanks for a swift reply! I've read through a number of the closed issues, but that one had slipped my attention.
Based on your comment in #90, the values of the above parameters are

first_subsampling_dl = 0.4 - There are roughly 10 points sampled per square meter, so I chose a value that is a bit higher than the minimum distance between two points. The data comes from a lidar scan and is more or less constant throughout the data
in_radius = first_subsampling_dl *50
batch_num = 6 - Unchanged from the original S3DIS code

The modifications I have made so far are the ones mentioned in issue #72:

The method that prepares the text files to ply files is changed to fit my data, which is stored in the las file format
All of the variables related to the colors, which stores three dimensional data, are now related to the intensity in 1D variables. Meaning that I have made changes in the methods load_subsampled_clouds, potential_item and random_item.
in_features_dim = 2, and thus I have added an extra elif in the following code segment
https://github.com/HuguesTHOMAS/KPConv-PyTorch/blob/7fefb6a8d38fd304775199777ad01d9f1546e2ff/datasets/S3DIS.py#L402-L411
the elif statment reads

elif self.config.in_features_dim == 2:
    stacked_features = np.hstack((stacked_features, features[:, :1]))

Everything else is kept the same, except the variables named 'S3DISxxxxx', they are changed to 'CustomDataxxxxx'
I've tried increasing first_subsampling_dl, but it dosen't seem to have any impact. The same goes for reducing the expected_N variable. The output from the terminale reads
Step     1  estim_b = 0.01 batch_limit =   1501
Step     2  estim_b = 2.11 batch_limit = -60958
Step   158  estim_b = 1.23 batch_limit = -51682
Step   309  estim_b = 1.05 batch_limit = -47182
Step   468  estim_b = 1.01 batch_limit = -41827
Step   629  estim_b = 1.00 batch_limit = -35760
Step   788  estim_b = 1.00 batch_limit = -29133
Step   949  estim_b = 1.00 batch_limit = -21779
ERROR: It seems that the calibration have not reached convergence. Here are some plot to understand why:
If you notice unstability, reduce the expected_N value
If convergece is too slow, increase the expected_N value

Below are the produced error plots, with the first plot being  of debug_in and debug_out and the second of debug_b and debug_estim_b.

While debugging I found that the variable b is the one that causes all of the trouble when going from step 1 to step 2, with its value first being 2 than in the next iteration being 3402. I'm having trouble understanding the big jump in numerical value. Maybe I have to take a third and forth look into how I process the las files to ply files.

update: Had to reduce the expected_N variable with a magnitude of 1e3, which was a lot more than what I first experimented with. I have managed to start the training of the custom dataset! Thank you so much for the feedback that ultimately solved my problem :)
Best wishes!

Hi @marialinea,
Happy to see that your problem was solved.
So it means in the end that you have less than a few hundred points per input sphere, which is really small. Here are a few comments based on my own experiments.

In the case of a sparser dataset like yours, where you end up with very small point clouds, you are free to increase the size of your input (a GPU with more than 10GB should handle batches with more than 10 000 points). That means 3 possibilities:

Decrease the first_subsampling_dl, the network can handle pretty well data even if the density of the input is smaller than the first grid size, and in that way you are sure not to lose any information in the input details.
Increase the in_radius: this will have a positive effect if you are classifying very big objects, as the network will have better information about them. Also, this means a faster test speed because you classify larger chunks of data at each iteration.
Increase the batch_num: larger batch can help accelerate the convergence but in my experiments, I did not see much improvement for values above 10 or 16 depending on the task.

If you still keep very small input clouds, and modify the batch_num, it would be worth playing a little bit with the learning rate decay for faster training. Here:
https://github.com/HuguesTHOMAS/KPConv-PyTorch/blob/cf1f06381ef062344b68286e3d5034b1aa94aecd/train_S3DIS.py#L173

The value 150 means we divide the learning rate by 10 every 150 epochs, but in a smooth way, a little bit at each epoch. You can reduce this value for faster convergence, but when you want to get your final result, set it higher up again to be sure you reach the minimum possible loss.

Thanks for the tips, they'll come in handy!
After some 20 epochs it seems that the network isn't learning at all, a consistent mIoU of 2.1% or less, so I'll have to do some experimenting.

Hello @marialinea . I had the same problem. The batch_limit is negative. But it was solved by reducing the expected_N value into 1e4.
Regarding the consisten low mIoU, did you solve your problem? If so, how did you solve it?

Hi @aldinorizaldy, I don't think I ever solved the problem before my summer internship ended. So, unfortunately, I don't have a solution for you, best of luck!

Hi @marialinea, I solved the problem by changing the first_subsampling_dl and in_radius.  Apparently, my data is too sparse so I need much bigger values for them. Thanks for responding anyway.


Child router-view shouldn't cause update when parent route isn't matched.

Version
3.0.3
Reproduction link
https://codesandbox.io/s/6lpo2oxo4z
Steps to reproduce

Open the console
Click on link "Items"
Click on link "Page B"
Click on link "Page C"

What is expected?
The router-view inside of Items, should not cause Items to update, when the Items route isn't matched.
What is actually happening?
Notice that going from page B to page C caused an update in Items.

This happens if there is a nested router-view inside a another router-view   keepalive.

This is expected, Items is rendering because the nested router-view is changing

The point isn't that the component shouldn't update. Is that router-view shouldn't change because the parent route isn't matched.

Ah, I see. It's the same as #2739: the component is kept alive and re renders because router-view does (because it's a functional component)

Does it have to be a functional component? Can this be optimized?


Create basic starting point for page.

Added an index.html file with some basic styles.

@matthewsmawfield This is ready for review.

great start!


Add synthesis checks and linting

OpenLane currently has improved checks on Synthesis as well as linting using Verilator. These should be ported to OL2.
Note: may involve Nix.

Done in #114


Address service account token expiration issue

Overview
The service account token expires every 30 days. So we need a solution to rotate the service account token. Here is some supporting documentation. We've landed on rotating the service account token every 23 days, however this is up for debate and the number of days before rotation should be easily configurable. If this process fails we need to be notified, so also investigate a notification mechanism.
https://docs.aws.amazon.com/secretsmanager/latest/userguide/rotate-secrets_turn-on-for-other.html
https://devopstar.com/2023/02/25/automatic-api-key-rotation-for-amazon-managed-grafana/
AC

Service account token is automatically rotated every 23 days
Number of days before rotation is configurable
Investigate and document in confluence findings regarding potential notification mechanisms on token rotation failure
Documentation in confluence is created/updated to reflect any new findings
Manually validate this token rotation process

@PaulHLiatrio  / @pmpaulino  - is this still needing to be done? are y'all going to be doing this one

We have adjusted the codebase so that instead of storing a long lived token and needing to rotate it on a regular basis; we instead have it set up to generate a new API token on the fly which expires after 3m.
This was accomplished by using a lifecycle block in Terraform, and we set the create_before_destroy flag to true.
This ticket is no longer necessary.


Accessibility: Support aria-labels

Addresses issue #124

Thanks @atlidohop! Looks good to me. Just one small nitpick - what's the purpose of making aria props to default to empty string? Shouldn't they stay undefined when not provided?

Good point, @mpowaga . Changes coming up.

Beware, this is untested code, but works for my use case. How can I run index.html?

This should work for all cases now @mpowaga . I also made some changes to make this package more accessible to contributing developers. Let me know if I'm breaking something somewhere else. 😄

I appreciate the effort to make it easier to contribute but React and other packages must be declared as peerDependencies. Can you open an issue where we can discuss how we can improve development experience?

Sure thing. I'll revert the changes in package.json.

@mpowaga Thanks for good comments, much better now. 😄


⚠ Fetchers has degraded performance

In 7d9a858, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 507 ms

Resolved: Fetchers performance has improved in 0680b62.


restore proper definition of width/height

Width/Height for GPU parameter was switched.

/lgtm
/approve


Workingsteps pane now has setup breakpoints

Looks good, I'll be fixing the properties pane for setup steps in my upcoming patch (it's currently blank).


Fixes warning about session reuse.

@matthchr a little of context here. Since msrest 0.4.28, there is an improvement on session handling:
https://github.com/Azure/msrest-for-python#2018-04-18-version-0428
If your signed_session has an optional session parameter, msrest injects its permanent session to be patched, instead of relying on you to create a new one. In practical term, I can re-use the same session instead of creating a new one for each call (which is better).
Should be safe, but please do not hesitate to test it.

FWIW, I did test it before sending the PR :)

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@245328c). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #3613   +/-   ##
=========================================
  Coverage          ?   37.42%
=========================================
  Files             ?     9154
  Lines             ?   275716
  Branches          ?        0
=========================================
  Hits              ?   103193
  Misses            ?   172523
  Partials          ?        0

Impacted Files
Coverage Δ

azure-batch/azure/batch/batch_auth.py
70% <100%> (ø)

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 245328c...5269985. Read the comment docs.

This looks good to me.
@xingwu1, it sounds like @lmazuel is suggesting that we release a patch version of the SDK with this fix. Any objections to that?

It'll certainly make my demo next week easier if I'm using a public release :)
Though there's also the semi-related https://github.com/Azure/msrest-for-python/issues/126 that I'd like fixed too. Not sure whether that will require any coordination with the azure-batch release.

@zooba not coordination needed, and PR done for msrest already:
https://github.com/Azure/msrest-for-python/pull/127

@lmazuel The fix LGTM. However, it is possible to publish the new binary without change the version?

@xingwu1 No, that's against PyPI policy. And anyway, this is a change in code, and then requires a bug fix version update.


in Java programming opens twice times the same source file causing confusion

When I am programing in Java, on clicked in a error/warning in Problems, the Code opens some source files twice times, and it is confusing. This confuses, because the same file is opened without warning, with different versions. This is very bad.
Please, this need to be fixed.

Please report directly against https://github.com/redhat-developer/vscode-java/issues
This is most likely caused by the Java extension which is adding those errors/warnings in the Problems view, but possibly using different URIs to refer to the same files...


_BGate defined as complex conjugate of B gate

The B gate class in two_qubit_to_fsim.py appears to have been defined as the complex conjugate of the B gate as defined in the original reference (unless I'm making a sign error somewhere). It should be XX(a, b)**-0.5, YY(a, b)**-0.25,
https://github.com/quantumlib/Cirq/blob/2b88d34dc2c4dc5dc1c59d21810114112286c225/cirq/optimizers/two_qubit_to_fsim.py#L137
https://arxiv.org/abs/quant-ph/0312193
In a sense this doesn't really matter, since _BGate is only used internally to the module, and B and its conjugate are related by 1-qubit gates. But it might trip someone up down the line.

I think you're right, thank you for pointing it out!!
The XX gate is defined as  and implemented as such in our XPowGate. And YY is similar. I suspect that for the 2qubit decomposition work to FSim it still worked because of some symmetry. However just simply changing the decomposition to the other direction, it breaks all the tests.
@viathor can you confirm our understanding here?

@balopat @gecrooks I agree that cirq's _BGate is the complex conjugate of B gate defined in the paper.
I think the decomposition works because the inverse B gate is locally equivalent to the B gate (e.g. conjugate by ZI), though I think some angles on the single-qubit gates may need adjustment.
We should fix this.
@gecrooks Thank you for catching this! Would you like to have a go at the fix?


[Proposal] Drastic update dependencies and rubies

This project looks used by many projects, I know the one is https://github.com/Shopify/graphql-batch/blob/9038f8907e9d742a2a0ce2c924854a688c2953ff/graphql-batch.gemspec#L24
But the maintenance looks so hard,  docker container run --user=root --volume="$(pwd)/:/usr/src/" --workdir='/usr/src/' -it 'ruby:3.0' '/bin/bash' even fails to execute bundle install the cause is many dependent gems are unmaintained and version mismatched.
So I suggest a way as below.

Drop to support old rubies (including rubinius), cruby 2.4 is ended as https://www.ruby-lang.org/en/news/2020/04/05/support-of-ruby-2-4-has-ended/
Drop non core dependencies (coverage, spec formatter, yard checker)
Update rubocop (Personally I don't basically use rubocop, but this project looks to wanting it 😄 )
Ensure to pass and no warnings in running rspec

How do you think?

This project looks used by many projects, I know the one is https://github.com/Shopify/graphql-batch/blob/9038f8907e9d742a2a0ce2c924854a688c2953ff/graphql-batch.gemspec#L24

Yeah, I created that and I assume that is the primary place this is used.

fails to execute bundle install the cause is many dependent gems are unmaintained and version mismatched.

bundle install seemed to work fine for me on ruby 2.7.2.  Using the same lockfile generated by ruby 2.7.2 seemed to work on ruby 3.0.0, but not once I removed the lockfile and tried to regenerate it on ruby 3.0.0.  Looks like some gems need to be pinned to more specific versions to get this working again.
Looks like checking Gemfile.lock into version control would make it easier to maintain gems, by preserving the known working state.  The main reason to not include the lock file is that we also want to ensure the latest version of dependencies continue working, however, we should really be testing both the earliest supported and latest versions of dependencies on CI anyways.

Ensure to pass and no warnings in running rspec

Looks like the introduction of Object#then introduced a mutation failure for newer versions of ruby.
Of course, getting tests passing on CI is a priority, including fixing warnings to fix them before they turn into failures.

But the maintenance looks so hard

The main pain point has actually been with mutation testing.  I have kind of a love hate relationship with it, where it is normally good at identifying code that is either missing a test case or could be simplified, however, it is a pain when we want less simple code as a performance optimization.  I think we could use a CONTRIBUTING.md file with recommendations for cases like this, where perhaps recommend using an ignore configuration (https://github.com/mbj/mutant/blob/master/docs/configuration.md#matcher) and commenting on why the ignore workaround seems needed.

Update rubocop (Personally I don't basically use rubocop, but this project looks to wanting it 😄 )

I'm all for letting a bot do the nitpicking for me.  I'm used to Shopify's ruby-style-guide, so I wouldn't mind adopting the rubocop-shopify gem.

Drop to support old rubies (including rubinius), cruby 2.4 is ended as https://www.ruby-lang.org/en/news/2020/04/05/support-of-ruby-2-4-has-ended/

Looks like rubinius was already not included in CI.
Dropping old versions of ruby can still be a breaking change and can be annoying when upgrading old dependencies when bugfix releases drop support for unsupported versions of dependencies.  So usually I just bump the minimum version when it actually matters for development of some feature or as part of a cleanup for a major release.  Hopefully using a Gemfile.lock for the minimum supported versions should simplify maintaining support for older versions.

Drop non core dependencies (coverage, spec formatter, yard checker)

Was this just to simplify fixing bundle install or were these a problem during development for other reasons.

I would prefer more focused PRs.

@dylanahsmith

however, it is a pain when we want less simple code as a performance optimization

Can you raise this cases with me? Usually these are interesting and a sign of a bad operator, mutant may want to remove.

This library should probably drop devtools and just use the individual tools that we actually care about directly.  Once we do that, it should be easier to upgrade mutant and see what deficiencies it actually has, compared to which were just from using an older version.  https://github.com/lgierth/promise.rb/commit/c1eeac1a10f2d2a237ca42655755dfee502b91f6 was an example of a commit I made in the past to satisfy mutant, where I changed from using instance variables directly to using the attribute method.  Using the instance variable directly probably makes more sense for performance, even though in non-performance sensitive code using the attribute method would make sense to reduce direct coupling to those instance variables.
I expect there would be more fundamental conflicts with using mutant for performance sensitive code when dealing with fast paths.  In that case, the condition added to provide a fast path could be removed without changing the semantics of the method.  Removing a code paths to see if a code path is tested seems like a pretty fundamental part of the strategy mutant takes to determine if code is sufficiently tested.  For instance mutant's README says

Think of mutant as an expert developer that simplifies your code while making sure that all tests pass.
...
Each reported simplification signifies either:
A) A piece of code that does more than the tests ask for. You can probably use the simplified version of the code. OR:
B) If you have a reason to not take the simplified version as it violates a requirement: There was no test that proves the extra requirement. Likely you are missing an important test for that requirement.

so if the performance is the requirement, then that would require tests for the performance of the code, but that is going to be some combination of slow and unreliable.
I think the way to resolve this conflict is to opt-out of mutant for that code.  Currently, I think that needs to be done on a method level of granularity, although, potentially that could be done in a more fine grained way.


change the today's date background circle color

I want to change the background color of selected date/current date in the calendar.I know I can create Decorator class implementing DayViewDecorator. And adding that decorator while date change happens.It works perfectly.But the thing is when we open the calendar for the first time the current today's background is in default color.

Actually using xml attribute  app:mcv_selectionColor="@color/green" we can change the selection color


Create CraftTweaker-help.txt

I created a little help for anyone who wants to configure chickens and recipes for custom modpacks!

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@gigabit101 @AugustinMauroy


MNT: Re-rendered with conda-smithy 3.1.6 and pinning 2018.06.19

Fixes the current incorrect json-c vs json_c situation in pin_run_as_build, which since there's now a new json-c version breaks installs.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Merging without waiting for Travis, because it's essentially only a metadata change and I'm impatient.


Cosmetic issues

Issues Addressed:
#68
#31
#23
#80

Having fixed the issue of the aggs panel locking and being unable to be removed or re-added, go ahead and do the merge.


MicrosoftAjax.js fails with 'Access is Denied" message on wcf service call over https

Our application was running with AjaxControlToolkit v9.11 over .net2.0 and making calls to wcf web service over http and https and was working properly. application was referring to external script files under scriptmanager (previously ToolkitScriptManager) as below
<asp:scriptmanager id="sm" scriptmode="Release" enablepartialrendering="true" runat="server">

    <services>
            <asp:servicereference path="~/PrintService.svc" />
        </services>
</asp:scriptmanager>

We upgraded our application to .net4.5.2 and since then wcf web service call over secure https is giving the 'Access is denied' error. It works fine over http.
We upgraded AjaxControlToolkit version to 16.1 using Nuget package, however, external script files under "/scripts/MicrosoftAjax/" folder are not updated and so MicrosoftAjax.js file is still the same and we still get the same error over https.
How do I resolve this error? Where can I get the external script files or does the new AjaxControlToolkit version doesn't need to have external script file references?
error from fiddler:
stack
"Error: Access is denied.
   at a.Net.XMLHttpExecutor.prototype.executeRequest (https://securewebtest.vfsgroup.com/TransActionEOS/scripts/MicrosoftAjax/MicrosoftAjax.js:1:80534)
   at a.Net._WebRequestManager.prototype.executeRequest (https://securewebtest.vfsgroup.com/TransActionEOS/scripts/MicrosoftAjax/MicrosoftAjax.js:1:83266)
   at a.Net.WebRequest.prototype.invoke (https://securewebtest.vfsgroup.com/TransActionEOS/scripts/MicrosoftAjax/MicrosoftAjax.js:1:85065)
   at a.Net.WebServiceProxy.invoke (https://securewebtest.vfsgroup.com/TransActionEOS/scripts/MicrosoftAjax/MicrosoftAjax.js:1:88588)
   at a.Net.WebServiceProxy.prototype._invoke (https://securewebtest.vfsgroup.com/TransActionEOS/scripts/MicrosoftAjax/MicrosoftAjax.js:1:87236)
   at PrintService.prototype.PrintDocList (https://securewebtest.vfsgroup.com/TransActionEOS/PrintService.svc/js:14:1)
   at PrintService.PrintDocList (https://securewebtest.vfsgroup.com/TransActionEOS/PrintService.svc/js:38:131)
   at GetDocs (https://securewebtest.vfsgroup.com/TransActionEOS/doclist.aspx:369:21)
   at Global code (https://securewebtest.vfsgroup.com/TransActionEOS/doclist.aspx:468:833)"

@danishtaqvi
Those scripts, MicrosoftAjax.js and MicrosoftAjaxWebForms.js, are a part of the Microsoft.AspNet.ScriptManager.MSAjax NuGet package. Add it to the project, or update if it's been installed earlier. Note that the new scripts are placed to a slightly different location: ~/scripts/WebForms/MSAjax

Thanks @AlekseyMartynov for the quick response and pointing me to the package.
Ok, so I added the package Microsoft.AspNet.ScriptManager.MSAjax to the project and got the scripts folder and files created; updated the reference in my page but got the same error from the new MicrosoftAjax.js file this time. Error in fiddler stack:
"Error: Access is denied.\r\n\n   at Sys.Net.XMLHttpExecutor.prototype.executeRequest (https://securewebtest.vfsgroup.com/TransActionEOS/scripts/WebForms/MSAjax/MicrosoftAjax.js:6:85275)\n   at Sys.Net._WebRequestManager.prototype.executeRequest (https://securewebtest.vfsgroup.com/TransActionEOS/scripts/WebForms/MSAjax/MicrosoftAjax.js:6:88634)\n   at Sys.Net.WebRequest.prototype.invoke (https://securewebtest.vfsgroup.com/TransActionEOS/scripts/WebForms/MSAjax/MicrosoftAjax.js:6:90645)\n   at Sys.Net.WebServiceProxy.invoke (https://securewebtest.vfsgroup.com/TransActionEOS/scripts/WebForms/MSAjax/MicrosoftAjax.js:6:97335)\n   at Sys.Net.WebServiceProxy.prototype._invoke (https://securewebtest.vfsgroup.com/TransActionEOS/scripts/WebForms/MSAjax/MicrosoftAjax.js:6:95743)\n   at PrintService.prototype.PrintDocList (https://securewebtest.vfsgroup.com/TransActionEOS/PrintService.svc/js:14:1)\n   at PrintService.PrintDocList (https://securewebtest.vfsgroup.com/TransActionEOS/PrintService.svc/js:38:131)\n   at GetDocs (https://securewebtest.vfsgroup.com/TransActionEOS/doclist.aspx:371:21)\n   at Global code (https://securewebtest.vfsgroup.com/TransActionEOS/doclist.aspx:470:871)"
Since this was working when the project was on .net2.0, and upgrading it to .net4.5.2 had made changes to web.config file, I want to highlight that some sections of web.config have been removed with the upgrade which looks important;
    <configSections>         <sectionGroup name="system.web.extensions" type="System.Web.Configuration.SystemWebExtensionsSectionGroup, System.Web.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31BF3856AD364E35">             <sectionGroup name="scripting" type="System.Web.Configuration.ScriptingSectionGroup, System.Web.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31BF3856AD364E35">                 <section name="scriptResourceHandler" type="System.Web.Configuration.ScriptingScriptResourceHandlerSection, System.Web.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31BF3856AD364E35" requirePermission="false" allowDefinition="MachineToApplication" />                 <sectionGroup name="webServices" type="System.Web.Configuration.ScriptingWebServicesSectionGroup, System.Web.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31BF3856AD364E35">                     <section name="jsonSerialization" type="System.Web.Configuration.ScriptingJsonSerializationSection, System.Web.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31BF3856AD364E35" requirePermission="false" allowDefinition="Everywhere" />                     <section name="profileService" type="System.Web.Configuration.ScriptingProfileServiceSection, System.Web.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31BF3856AD364E35" requirePermission="false" allowDefinition="MachineToApplication" />                     <section name="authenticationService" type="System.Web.Configuration.ScriptingAuthenticationServiceSection, System.Web.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31BF3856AD364E35" requirePermission="false" allowDefinition="MachineToApplication" />                     <section name="roleService" type="System.Web.Configuration.ScriptingRoleServiceSection, System.Web.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31BF3856AD364E35" requirePermission="false" allowDefinition="MachineToApplication" />                 </sectionGroup>             </sectionGroup>         </sectionGroup>     </configSections>     <system.web>         <pages>             <controls>                 <add tagPrefix="asp" namespace="System.Web.UI" assembly="System.Web.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31BF3856AD364E35" />                 <add tagPrefix="asp" namespace="System.Web.UI.WebControls" assembly="System.Web.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31BF3856AD364E35" />             </controls>
and
       <compilation debug="true" strict="false" explicit="true">             <assemblies>                 <add assembly="Microsoft.VisualBasic.Compatibility, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=B03F5F7F11D50A3A" />                 <add assembly="System.Design, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=B03F5F7F11D50A3A" />                <add assembly="System.Security, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=B03F5F7F11D50A3A" />                <add assembly="System.Web.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31BF3856AD364E35" />
and runtime section referring to System.Web.Extensions. I have attached both config files (2.0 and 4.5.2) to show the difference.
web.config.zip

Changes to web.config seem fine, and it's unlikely that they are related to the "Access denied" error. It rather looks like a cross-domain issue, and may be caused by the fact that WCF Services assemblies have been updated too.
Here are a couple of StackOverflow threads discussing the same problem:

http://stackoverflow.com/q/25889599
http://stackoverflow.com/q/15529328

Thanks for sharing those @AlekseyMartynov .
We did actually visited the cross-domain issue articles earlier and rules out the possibility of this cause as the service is hosted within the same application/domain. Since this error comes in only with .net4.5.2 framework and not with .net2.0, it seem to be internal to working of .net and AjaxControlToolkit.
Eventually, I ran out of options to make it work with AjaxControlToolkit, so tried jquery ajax call and it worked. There was nothing much I had to change for that and below is the working code snippet:
                    $.ajax({                         type: 'POST',                         contentType: "application/json; charset=utf-8",                         url: 'PrintService.svc/PrintDocList',                         data: JSON.stringify({parameter1: value1, parameter2: value2}),                         dataType: "json",                         success: GetDocsSuccess,                         error: GetDocsFailed                     });
Thanks for your kind support, really appreciate your help in this regard.


Add auto shard aware port

This patch makes sessions automatically probe for shard-aware ports
via the CQL OPTIONS request, and if found, tries to use it.
With such a mechanism, users no longer need to remember to manually
set up port 19042 to enjoy shard awareness.
The new algorithm is as follows:

connect to one of the nodes just to send an OPTIONS request and retrieve the shard-aware port

if found, use the shard aware port to connect to the cluster

if successful, continue enjoying the driver
if failed, fall back to the original port (a custom port can fail e.g. due to firewall rules)

if not found, use the original port

Currently marked as draft, because the change was only tested manually in a limited scope. Still, it's ready for preliminary review.
Fixes #255
Pre-review checklist

[x] I have split my patch into logically separate commits.
[x] All commit messages clearly explain what they change and why.
[ ] I added relevant tests for new features and bug fixes.
[x] All commits compile, pass static checks and pass test.
[x] PR description sums up the changes and reasons why they should be introduced.
[x] I added appropriate Fixes: annotations to PR description.

note: the pull request was created with programmer's ergonomics in mind. In particular, if the client already passed the shard-aware port, it will still be probed anyway. It creates a minimal overhead, but I consider it negligible, because creating new sessions is not in any way a most common operation during applications' execution time.

v2:

changed a stray println! used for debugging to info!
tested manually with Scylla, Cassandra and checking the logs:

Tested with RUST_LOG=scylla=trace cargo run --example logging
Cassandra:
Jun 25 09:28:01.085  WARN scylla::transport::session: Unable to establish connections at detected shard-aware port, falling back to default ports
Jun 25 09:28:01.086 DEBUG scylla::transport::cluster: Requesting topology refresh
Jun 25 09:28:01.139  WARN scylla::transport::session: Raw USE KEYSPACE queries are experimental, please use session::use_keyspace instead

Scylla:
Jun 25 09:29:13.493  INFO scylla::transport::session: Shard-aware port detected: 19042
Jun 25 09:29:13.493 DEBUG scylla::transport::cluster: Requesting topology refresh
Jun 25 09:29:13.502  WARN scylla::transport::session: Raw USE KEYSPACE queries are experimental, please use session::use_keyspace instead

For Scylla I also verified that the connection is shard-aware via cqlsh:
cqlsh <IP_ADDRESS> <<< "select address, driver_name, shard_id from system.clients where driver_name = 'scylla-rust-driver' allow filtering;"

 address   | driver_name        | shard_id
-----------+--------------------+----------
 <IP_ADDRESS> | scylla-rust-driver |        2
 <IP_ADDRESS> | scylla-rust-driver |        3
 <IP_ADDRESS> | scylla-rust-driver |        0
 <IP_ADDRESS> | scylla-rust-driver |        1

v3:

preserved the error message from not being able to connect to the shard-aware port
rebased


Fix assume vector key is available

If it is not available, null is an invalid type for our vector property

what do you think about having a NullVector as a fallback throwing that exception? a bit more generic than, but delaying the issue to the point users really want to interact with the vector data.
not sure we need to enforce the vector being around after querying - don't have a use case on my mind at the moment

Superseded by

#114


Implement Send for State.

Since State properly manages the ownership of its contained pointer it should be made Send with unsafe impl Send for State {} so that once wrapped in a Mutex the Lua State can be shared between threads.

I think your proposal is reasonable, since Lua states are thread-safe data; however, you still can't safely use a Lua state in more than one thread since mutating a state from more than one thread at a time causes undefined behavior.

Not using it simultaneously is handled by the Sync trait, which would not be applied.  This just means that it needs to be wrapped in a Mutex.  The problem is that without the Send trait the Mutex will not apply the Sync trait.


"Docs" section is called "Overview" Communication issue

In the header of the site we say "Overview" which links to the docs section. I feel this should be called what it is.. Documentation. Developers are used to looking for a documents section not an Overview section.
Also, if we do decide to step away from the norm and train users for our site to think of Documents as Overview we should change the URL to https://blockstack.org/overview from the current https://blockstack.org/docs See attachment

Closing this issue


Schema Compare file not reusable for Database Projects

Issue Type: Bug
Schema Compare file (.SCMP) not reusable when Database Projects is either source or target.  When starting from Database Projects (icon on left-bar) > right-click project, choose Manage > click Schema Compare > click Open .scmp file > choose .scmp file from location: the Schema Compare window opens properly, however the .sqlproj file that was previously saved as a reference in the .SCMP file is no longer recognized (the source or target side, whichever the .sqlproj was on, is blank).  When you click the ellipse and try to choose your .sqlproj file, it allows you to choose one, but you then receive "Schema Compare failed: Object reference not set to an instance of an object." message.
At this point, the .SCMP file seems to produce the same error even if you to switch a .DACPAC file and then try to compare.  If I close out of the Schema Compare tab, go back to Manage page for the Database Project and try to start the process again, even using an .SCMP file which was save with a db & .DACPAC as it's comparison, results in the same error.
It seems like this error is coming from the database side of the comparison.  Once I change the connection & re-connect to the database, the "Schema Compare failed: Object reference not set to an instance of an object." error goes away.  Only after that step can I go back and point the schema compare at the .sqlproj file I was originally working with.
Extension version: 1.13.0
Azure Data Studio version: azuredatastudio-insiders 1.37.0-insider (4e849f0d601f9e5210f3612458d8f1effae9d527, 2022-04-27T05:28:05.548Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz (8 x 1498)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.59GB (15.74GB free)

Process Argv

Screen Reader
no

VM
0%

@udeeshagautam @Benjin scmp support for sql projects wasn't added when support for schema compare and update project was added, right?

Any updates on this?
I am experiencing the exact same issue (ADS v1.37.0, schema compare tool v1.14.0, SQL Database objects v0.17.1)


Question: Does this work for https servers

Hey there! I had deployed this on one of my https servers and the server didn't start. Any leads as to what I'm doing wrong? I'm using the default settings without changing anything.

This work for https server. I've many production https web servers running with node-ddos

Thanks! I guess something must be wrong on my side then.


500 error with attachments that have multi-byte character filenames.

My helpy is customized and I apologize when this is caused by customize.
I got  an 500 error after submitting reply with attachments that have multi-byte character filenames.
Log is like this.
ActionView::Template::Error (URI must be ascii only "https://{censored}.blob.core.windows.net/helpy/uploads/post/attachments/49/\u{5143}\u{306e}\u{30e1}\u{30fc}\u{30eb}.txt"):

I guess filenames should be encoded before passing CarrierWave.

Update.
The whole log is like this. And CarrierWave (I customize it to use with Azure Blob) stores attachments correctly.
Filenames should be encoded before storing to DB or after getting from DB.
Completed 500 Internal Server Error in 1111ms (ActiveRecord: 768.2ms)

ActionView::Template::Error (URI must be ascii only "https://{censored}.blob.core.windows.net/helpy/uploads/post/attachments/49/\u{5143}\u{306e}\u{30e1}\u{30fc}\u{30eb}.txt"):
    15:         <% if %w(png jpg jpeg gif).include?(att.file.extension) %>
    16:           <%= link_to image_tag(att, style: "width: 125px; height: 125px"), att.url, class: 'screenshot-link', target: 'blank' %>
    17:         <% else %>
    18:           <%= link_to att.url, title: att.url, target: 'blank' do %>
    19:             <%= content_tag(:i, '', class: attachment_icon(att.url)) %>
    20:           <% end %>
    21:         <% end %>
  app/views/posts/_attachment.html.erb:18:in `block (4 levels) in _app_views_posts__attachment_html_erb__2836734086973453099_47349722287320'
  app/views/posts/_attachment.html.erb:14:in `block (3 levels) in _app_views_posts__attachment_html_erb__2836734086973453099_47349722287320'
  app/views/posts/_attachment.html.erb:13:in `each'
  app/views/posts/_attachment.html.erb:13:in `block (2 levels) in _app_views_posts__attachment_html_erb__2836734086973453099_47349722287320'
  app/views/posts/_attachment.html.erb:12:in `block in _app_views_posts__attachment_html_erb__2836734086973453099_47349722287320'
  app/views/posts/_attachment.html.erb:11:in `_app_views_posts__attachment_html_erb__2836734086973453099_47349722287320'
  app/views/admin/topics/_post.html.erb:28:in `_app_views_admin_topics__post_html_erb___3968004254807639614_47349721144820'
  app/views/admin/topics/_ticket.html.erb:20:in `_app_views_admin_topics__ticket_html_erb___2427613158569892413_47349712742540'
  app/views/admin/topics/show.js.erb:1:in `_app_views_admin_topics_show_js_erb___1959363208674178946_47349712510780'
  app/controllers/application_controller.rb:202:in `set_time_zone'

In addition and IMO, should attachments be stored behind the hashed directory?
In the current implementation, crackers may prospect URL to attachments easily.

I see your main language is Japanese, and your language uses non-ASCII patterns. I need to test more the upload methods, but the problem can show to another languages characters patterns.
But looking this log trace, is clear to me. The application don't sanitize the non-ASCII characters correctly.
I need to check this behavior more closer to create a non-intrusive and best-fit solution.

@monaka I've made some test using Kanji characters and all Helpy controllers, and views could render the attachments (images, raw text files). Can you give more details how reproduce this error?

This will be reproduced just sending an attachment file named in Kanji.
And probably also Emoji characters.
I can try your patch on my site if you open it.

First I made a test with a pure instance of Helpy, based on helpy/master. And all Kanji file named, are uploaded, and rendered on your respective views.
Or your instance has a particular modification that causes this problem, or I don't try all cases what your faced. I only tried Kanji characters, and the pure instance of Helpy works well.
In (Brazilian) portuguese we have some few multibyte characters too, and I don't have faced this problem. :smile:

Exactly I have a tiny modification for using Azure Blob with CarrierWave.
But it's hard to explain how my patch causes this issue, IMO.
Okay, I'll try to reproduce and to make new patch. Thank you for your checks.

@rynaro Could you tell me what version of Ruby are you using?
Reffering to README.md, Helpy requires Ruby 2.2+. And URL/URI handlings are changed in Ruby 2.3.
I'm not sure the changes causes this issue. Just possibility.

I got an another 500 exception in displaying avatar in addition to attachments.
(But there is no exception on the logo even thought they are same PNG file. I'm not figure out why...)
app  | ActionView::Template::Error (URI must be ascii only "https://{censored}.blob.core.windows.net/helpy/support/uploads/user/profile_image/1/\u{7e26}\u{7d44}\u{307f}_\u{767d}\u{629c}\u{304d}.jpg"):
app  |     34:

app  |     35:
<%= I18n.t("written_by", default: "Written By") %>:

app  |     36:

app  |     37:         <%=<EMAIL_ADDRESS>size=40) %>
app  |     38:         <%= @doc.user.name.titleize %>
app  |     39:

app  |     40:

app  |   app/helpers/users_helper.rb:84:in `avatar_image'
app  |   app/views/docs/show.html.erb:37:in `_app_views_docs_show_html_erb___2093955696365140870_47010139175040'

And, I found a commit to CarrierWave related on URL encoding.
In addition, I found the article describes the background of this patch writen in Japanese.
So, it seems @Rynaro is right.
This issue is caused not by Helpy.
But CarrierWave (and/or CarrierWave-Azure) and/or Ruby versions.
I close this issue. Thanks for your support.


remnux upgrade is failing at ChildProcess.< (/snapshot/remnux-cli/remnux-cli.js:569:23)

Update returned exit code not zero
Error: Update returned exit code not zero
at ChildProcess. (/snapshot/remnux-cli/remnux-cli.js:569:23)
at ChildProcess.emit (events.js:315:20)
at maybeClose (internal/child_process.js:1021:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:286:5)
and here is the saltstack.log file:
https://drive.google.com/file/d/162Wp3u6-4hg9xGoUsw7qAgwRjecuZK9C/view?usp=sharing

Hi @KhaledEid020 , it looks like you had a temporary loss of connectivity during the installation process. I can see some "Temporary failure in name resolution" errors, and some other connectivity issues.
If you are working behind a proxy, that could cause an issue as well. You can follow the steps here to configure your system to allow REMnux to install behind a proxy. Also, I would recommend testing your internet connection first then trying again.

Thank you, Worked!


New version: LineSearches v7.1.2

Registering package: LineSearches
Repository: https://github.com/JuliaNLSolvers/LineSearches.jl
Created by: @pkofod
Version: v7.1.2
Commit: c8b9f10877952e17c54325420ed810b4c6cd5c42
Reviewed by: @pkofod
Reference: https://github.com/JuliaNLSolvers/LineSearches.jl/commit/c8b9f10877952e17c54325420ed810b4c6cd5c42#commitcomment-81948965
Description: Line search methods for optimization and root-finding

please close this, it's superseded by https://github.com/JuliaRegistries/General/pull/66858


drivers: ethernet: w5500: changed to synchronous TX operation

This PR is a part of my greater WIP effort towards supporting socket offload for W5500 per #76003.
The changes in this PR concern

The structure of the code. I made some helper functions to make my life easier since it takes quite a handful of register read and write to make socket offload work.
To make use of socket offload, I need to assign a smaller socket TX buffer to each socket. There are 16k bytes of TX buffer for all 8 available W5500 sockets. At this moment, since only socket 0 is used in MACRAW mode, it is assigned the entire 16k bytes of buffer. However, if I have to make use of all sockets (use the other 7 sockets for offloading), that's 2k bytes for each socket. This creates great pain in the overall throughput. I identified the major problem to be the semaphore-take in w5500_tx. I changed it into a synchronous loop and saw a great improvement in the performance.

Benchmarking with the echo_server example using 2k bytes TX buffer (with my client written in python reporting the speed)
Before this change
2024-11-25 23:24:56,089: Data receiving rate: 40.3 kB/s
2024-11-25 23:25:04,407: Data receiving rate: 49.7 kB/s
2024-11-25 23:25:12,647: Data receiving rate: 50.2 kB/s

After this change
2024-11-25 23:16:49,762: Data receiving rate: 164.1 kB/s
2024-11-25 23:16:52,370: Data receiving rate: 164.4 kB/s
2024-11-25 23:16:54,980: Data receiving rate: 164.3 kB/s

The performance improves marginally if I use all 16k TX buffer for a single socket (this is the default set in the w5500 driver now, since only Socket 0 is used in MACRAW mode).
I'm aware that this PR won't make too much difference at this point. I'm also happy to combine it with my socket offload implementation if you see fit.

Superseded by #83413.

Superseded by https://github.com/zephyrproject-rtos/zephyr/pull/83413.


OKTA-654783 - Change accessibility for sqlite encryption key to kSecA…

…ttrAccessibleAfterFirstUnlockThisDeviceOnly
Problem Analysis (Technical)
SQLite encryption key should be non-syncable
Solution (Technical)

Change accessibility parameter for encryption key to kSecAttrAccessibleAfterFirstUnlockThisDeviceOnly
Migrate parameters for old key

LGTM, is anything else encrypted besides TOTP?

Username


Remove azure-sdk-tools from the dev requirements

The current state of the repo fails to run tests upon a clean clone.
Removing azure-sdk-tools from the dev-dependencies fixes this (see Github issue).
Re-adding dependency afterward does not break the test pipeline, so I'm wondering if that is the reason this has not been caught.
This fixes #15759

Hey @casperlehmann thanks for the contribution, but we went with a method that keeps the control of this specific dep in the package itself. However, we removed it from the common location as well.
If you pull latest master, I believe you'll find that your experience with tox will be significantly better now.
Apologies for the difficulties. Discussion over here.

Hey @casperlehmann thanks for the contribution, but we went with a method that keeps the control of this specific dep in the package itself. However, we removed it from the common location as well.
If you pull latest master, I believe you'll find that your experience with tox will be significantly better now.
Apologies for the difficulties. Discussion over here.

hey @casperlehmann , I'm closing this PR now. as @scbedd  mentioned, please check out latest code which improved the tox experience.
please feel free to re-open the issue if you still meet with problems :)

Thanks. No worries. I'll have a look if I have the chance to work with it again.


python39Packages.requests-cache: 0.9.3 -> 0.9.4

Description of changes
https://github.com/reclosedev/requests-cache/blob/master/HISTORY.md#094-2022-04-22

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)
[ ] 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.05 Release Notes (or backporting 21.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 170495 run on x86_64-linux 1

2 packages marked as broken and skipped:

python310Packages.tvdb_api
python39Packages.tvdb_api

6 packages built:

cryptop
mnamer
python310Packages.pysychonaut
python310Packages.requests-cache
python39Packages.pysychonaut
python39Packages.requests-cache


static function

Hi there.
In the current version this code:
$routes->group('careers', static function ($routes) {
    $routes->get('/', 'Users\Careers::index');
    $routes->get('create', 'Users\Careers::manage');
    $routes->get('update/(:num)', 'Users\Careers::manage/$1');
    $routes->post('save', 'Users\Careers::save');
});

Is being changed to
$routes->group('careers', function(static $routes) {
    $routes->get('/', 'Users\Careers::index');
    $routes->get('create', 'Users\Careers::manage');
    $routes->get('update/(:num)', 'Users\Careers::manage/$1');
    $routes->post('save', 'Users\Careers::save');
});

Thank you!
Yes, this should be a bug to fix, but, for me to know, what would be here the benefit using "static function"? I've never seen this before. Which framework is it?

Hi, this should be fixed in the latest release (it may take a few ours to update). Thanks!


Add legacy and new SBOM support, with tests

Summary
Add legacy and new SBOM support, with tests. Resolves #7.
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@joshuatcasey The only thing I am thinking about is whether we should roll out the newer SBOM in the first release of the Composer CNB. For the rewrite, we only need to support the legacy. I wonder if it's confusing to also support the newer SBOM functionality in the first release.

@sophiewigmore Ok I can combine the integration tests.
Do we know what might happen if I include those CPEs? They aren't actually listed here: https://nvd.nist.gov/products/cpe, although earlier versions of Composer are listed.

OK that's probably why the CPE wasn't included in the dep-server. The CPE will not be over written by our automation until a new version comes out I think so it should be OK. My main concern is making sure the CPE you added is accurate.


Select-window does not return an error when window 2 does not exist

Describe the bug
The tmux select-window -t 2 does not return an error when there is no window 2. It outputs nothing, does nothing, and has an exit code of 0. It works for all other window numbers except for 2.
To Reproduce
Run the following when there is no window 2.
# Confirm that there is no window 2.
tmux list-windows

# Try to select window 2.
tmux select-window -t 2
echo $?

This outputs the list of current windows and the 0 exit code.
1: bash* (1 panes) [199x49] [layout ead7,199x49,0,0,28] @15

0

Expected behavior
The expected behavior is that it should print out an error message and have an exit code. This happens for all window numbers other than 2.
# Try to select window 3.
tmux select-window -t 3
echo $?

outputs
can't find window: 3

1

Environment

tmuxinator Version: not installed
tmux Version: tmux 2.9a
Ruby Version: ruby 2.6.3p62 (2019-04-16 revision 67580) [x86_64-linux]
OS: Linux 5.1.15-arch1-1-ARCH #1 SMP PREEMPT Tue Jun 25 04:49:39 UTC 2019 x86_64 GNU/Linux

Additional information
Let me know if anything else would be relevant and I'll share it.

Sorry, I was on this repo by mistake because I was reading #713 which seemed relevant. This is obviously not a tmuxinator issue.


Imbued Heart Not Triggering Loot Notifier

Checklist

[X] I'm reporting a problem with Dink
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
I received a Imbued Heart from a Dreadborn Araxyte and Dink did not fire a loot notification. However, I did receive a collection log notification.
Some more information: We do have history of receiving loot notifications for Imbued Hearts & Eternal Gems but these notifications were not from Dreadborn Araxytes. Because of this, I would assume an Eternal Gem would not fire if dropped from a Dreadborn Araxyte either.
Screenshots
In-game message

Dink settings - note: this notification sends directly to Discord

Runelite Logs
client.log
Runelite Version
1.10.42

Does the loot show up in runelite's loot tracker ui?

Does the loot show up in runelite's loot tracker ui?

It does appear to be missing. Runelite problem then?

Runelite problem then?

Yeah, I don't see an existing issue for this so you can report it at https://github.com/runelite/runelite/issues

I spoke to Cook (the wiki dev who gets the Loot Tracker data) and he confirmed he's gotten no tertiary drops in 50,000 kills, so it's definitely broken.


Plugin Market considers plugin incompatible

Plugin is not listed when searching within IDE, on web I see a warning of incompatibility.
But installation from release file is working.
I'm using PyCharm

P.S. thanks a lot, kind man!

Thanks for the report and review! It appears my plugin is still awaiting approval from JetBrains, it should be available in the marketplace within the next few days :)

It should be available now!


Add extend_from_iter into growable trait

Did not achieve expected performance.
growable::primitive::non_null::non_null
                        time:   [2.5629 us 2.5658 us 2.5692 us]

growable::dyn_primitive::non_null::non_null
                        time:   [6.3092 us 6.3195 us 6.3328 us]

After:

growable::dyn_primitive::non_null::non_null
                        time:   [6.0020 us 6.0147 us 6.0300 us]

We can't make extend_from_iter accept generic iterator, it'll make this trait none object-safety.
Fixes #772

packed_simd is broken in the latest rust nightly toolchain.

BTW, packed_simd seems superseded by stdsimd.

BTW, packed_simd seems superseded by stdsimd.

We know. See #747. There still are some blockers on that.

An idea: create a simple function pub fn from_slice_iterator<A: AsRef<dyn Array>, I:Iterator>(&[A], iter: I) -> Box<dyn Array> that matches the arrays' logical type, downcasts them, initializes the static-typed growable, and extend it from the iterator.

This should be done via extension trait to avoid object safety issue.

closing to search for better ideas.


fix(css/modules): Fix parsing and interference

Description:
This PR fixes parsing of css modules and fixes interference with nesting css transform.
Related issue:

Closing as it's fixed by https://github.com/swc-project/swc/pull/6132


Enhanced User Experience with "Try Again" and Hint system for Hangman game

Hi,
I would like to propose a new feature for the Hangman game
Description
This feature aims to improve the user experience by adding a "Try Again" option for players who have lost the game. Additionally, players can request a hint to reveal a letter in the word, helping them progress in the game.
Type of issue

[x] Feature (New Script)
[ ] Bug
[ ] Documentation

Checklist:

[x] I have read the project guidelines.

[x] I have checked previous issues to avoid duplicates.

[x] This issue will be meaningful for the project.

[x] I have proposed a possible solution for the bug

Implementation ideas:

Adding prompt to ask player if they want to try again.
Provide a hint option during the game.
Ensuring hint system is limited to a certain number of uses per game to maintain challenge.

Please assign this task to me. Thank you

I would like to collaborate in this issue, please let me know if is possible.


Multiple mentioning issue fixed in chat

When multiple times mentioned a same user in the chat, all occurrences must be displayed as name.

You need to build this for changes to take effect as a library. Run npm run build on root folder.

Done


Add Flair named entity recognition notebook

I did a test with Allennlp, NLTK and Flair.
For Flair, I tried this model which is the largest English one which identifies 18 different categories of named entities.
For Allennlp, I tried both models found here
For NLTK, I tried the default model used by ne_chunk()
I found that, in general, the set of entities identified by Allennlp and NLTK are proper subsets of the entities identified by Flair. I.e., the recall of Flair is substantially superior to the others.
So here I added the Jupyter notebook I used to extract the named entities using the Flair model.  The notebook depends on an input "data.json" file which contains the full texts of the documents (I will share this file through other channels for privacy reasons). The notebook outputs a .csv file with the entities identified, their category, and which document they occur in.
Notes:

The notebook is very slow when only using a CPU (~60min per document). When I opened it using Google Colaboratory and I activated a GPU there was a vast improvement in performance, it took ~2-3 minutes per document.
I suspect Stanza will work better than NLTK so we may want to try that later (it would be interesting to compare the recall to Allennlp and Flair).
The full texts in data.json concatenate the executive summaries, acknowledgements, forewords and body texts of each document to increase the search space for named entities. I wasn't sure if annexes would be useful for this or not.

Added Word2Vec embeddings and Clustering notebooks.
Spectral clustering first pass with default settings gives strange results. Need to look at other settings.
Kmeans gives interesting results but need to review.
All input and output data for notebooks I will share offline.


updating the overview to point to latest documentation location

and adding additional param required for script in javadoc comments (required since 8_u121)

Why not use https://builds.apache.org/job/incubator-netbeans-html4j-linux/javadoc/overview-summary.html ? Seems to be just fine.

@emilianbold linking to Jenkins builder is not very persistent, the builder always snapshots just the latest state, yet we need an archive for releases (e.g. 1.5.1). It makes sense to co-locate them - bits.netbeans.org is slowly going away, so dukescript.com (as long time supporter of the API) seems like a natural choice.

@eppleton @emilianbold not sure we should be hard-coding third-party links, as friendly as they might be :smile:  The future of bits.netbeans.org is still under discussion, and where things end up, but we could easily add a redirect from an netbeans.apache.org link to the dukescript link in the short term.


Updated hash for default image

I added the new hash for the default ISO file

Closes #7

looks like it's only changed in the code base and not https://bootloader-crimes.de/ because that still shows ebbc79106715f44f5020f77bd90721b17c5a877cbc15a3535b99155493a1bb3f, fails with that and only works if you enter the new c8dbc96b61d04c8b01faf6ce0794fdf33965c7b350eaa3eb1e6697019902945c


LingPy's align method can be used more efficiently

https://github.com/martino-vic/loanpy/blob/b073e367c31ef87c3b3e694b6f2c2adfa7abd2fa/qfysc.py#L405
What you do is to convert the alignment of two strings to a string, then pull out the info. But Pairwise works like this:
pw = Pairwise(["a", "b", "c"], ["b", "c", "a"]) # if you have segmented data, no need to take care of merging
pw.align(distance=True)
almA, almB, dist = pw.alignments[0]

You see? The alignment is accessible via the alignments attribute. Since Pairwise handles more than just one pair, this is in position 0, if only one pair has been passed.

Cool! Replaced the line with leftright = [pw.alignments[0], pw.alignments[1]] and it didn't even break the tests :) Closing this issue now


Set spacing method does not use parameter

The method  in MenuItem class requires a value parameter of boolean type that is not used.
I think the parameter should be removed.

You're right, but instead of just removing the parameter, @Deprecate the current method and have it invoke the correct way to set icon spacing, in order to avoid introducing a breaking change in the public API.


Set ONNX_USE_PROTOBUF_SHARED_LIBS and Protobuf_USE_STATIC_LIBS

The onnx build on windows requires the provision of either static or shared libs for protobuf and to achieve that there are several different flags that need to be set in protobuf, onnx as well as cmake (for find_package). At the very minimum ONNX_USE_PROTOBUF_SHARED_LIBS and Protobuf_USE_STATIC_LIBS need to be opposites of each other, but onnx does not currently set them up that way (onnx doesn't set Protobuf_USE_STATIC_LIBS at all and cmake interprets that as setting Protobuf_USE_STATIC_LIBS to OFF). If/when onnx is updated to correctly set up both variables, this can be removed (see https://github.com/onnx/onnx/issues/3345).

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Jenkins Linux s390x Build #313 [push] Fix protobuf settings (#... started at 17:16

Jenkins Linux amd64 Build #313 [push] Fix protobuf settings (#... started at 16:16

Jenkins Linux s390x Build #313 [push] Fix protobuf settings (#... passed after 42 min

Jenkins Linux amd64 Build #313 [push] Fix protobuf settings (#... passed after 51 min


Check that we still have observations after removing outliers

Ensure we only evaluate_orbits and beyond if we didn't remove all observations as outliers.

This was superseded by https://github.com/B612-Asteroid-Institute/ipod/pull/11


syntax highlight

hi there ,many thanx for this series ,one question though
the below img is from the Tabs Inside Out

could u plz tell me which editor/IDE u were using ?

I was using Sublime Text 2. Hope that helps.

can u tell me which scheme u were using ? ,am trying to implement the same thing but i cant find any scheme/syntax lang that have this scopes.

I am not sure what you mean by scheme/syntax lang. Can you please elaborate?
Sorry for the delay though.

the scheme is the color scheme file u use to get the highlight i.e monokai the syntax language is the actual language u use in ST like html,js,css,etc.. so i was asking were u using a custom package other than the default that came with ST ?

Nothing really fancy and I don't think I had anything apart from the popular Angular plugins like AngularJS/ AngularJS Snippets.
Hope that helps.

unfortunately i cant get the same highlighting as in the screenshot because anything within the double quotes are treated the same so all get the same color which is driving me nuts ,still thanx for ur help :+1: .


Allow for more customized authentication

Allow the user to explicitly set the project ID they want to debug and also pick up the default project ID. This is especially useful when using application default credentials set by gcloud which may not have a project id associated with them.
Fixes #14

It seems like you have push access to the repo. I'll let you go ahead and land this.

@ofrobots thanks for the review!
I looks like I don't have access to merge

Fixed. Let me know if you still don't have permission.

All good, thanks!

I haven't set up any GCP projects on my own workstation yet, so I have two questions:

If you take a normal keyFile, remove the project_id, and authenticate, does anything throw? If not, please you remove invalid_keyfile_1.json from the test fixtures. I incorrectly assumed that the keyFile must contain a project_id.

Is there a one-to-one correspondence between keyFiles and project IDs? If there is, do we need a check to make sure that the projectId passed in corresponds to the keyFile?

Finally, I don't think we should close #14 yet. It might be interesting to have a list of project IDs to keyFiles saved somewhere that this module can read, so that when authorizing, the user can simply pass in a project ID rather than search for a keyFile.


Guide and sample break if Microsoft.AspNetCore.SignalR.Client is updated

This guide and sample break if Microsoft.AspNetCore.SignalR.Client nuget package is updated to version 3.x in the client app. After updating, clients no longer receive the message.

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

ID: 99698458-a6cb-0874-bdda-5a3fcb0988b3
Version Independent ID: 970e5418-c85e-7b4e-adf1-a5ebc411bebc
Content: Azure SignalR Service with Xamarin.Forms - Xamarin
Content Source: docs/xamarin-forms/data-cloud/azure-services/azure-signalr.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @profexorgeek
Microsoft Alias: jusjohns

Apparently this is a known breaking change:
"It is breaking change in SignalR client 3.x. System.Text.Json is now the default Hub Protocol used by SignalR clients v3.x. but the sample send JObject of Newtonsoft.Json.Linq.JObject. Therefore, SignalR client fails to parse the messages.
To mitigate the issue, do the following changes in client side:
add package in your csproj:
create connection and specify json protocol explicitly: var connection = new HubConnectionBuilder().WithUrl(url)*.AddNewtonsoftJsonProtocol().Build();"

Oops, typo:
create connection and specify json protocol explicitly: var connection = new HubConnectionBuilder().WithUrl(url).AddNewtonsoftJsonProtocol().Build();"

Thanks for all the info @jgold6! I'll be investigating and fixing this soon.