get_sync_events throws an error when only one event is found and its indices exceed data

traceback:
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-33-5fd9957c9aba> in <module>
     13 #         print(spt)
     14         eventst = yasa.get_sync_events(epochs_data.copy(), sf, detection=spt, center='Peak',
---> 15                                        time_before=0.5, time_after=0.5)
     16         spt["Epoch"] = i
     17         eventst["Epoch"] = i

~/.virtualenvs/sleepy3/lib/python3.7/site-packages/yasa/main.py in get_sync_events(data, sf, detection, center, time_before, time_after)
    210             df_tmp = get_sync_events(data[idx_chan, :], sf, k.iloc[:, :-2],
    211                                      center=center, time_before=time_before,
--> 212                                      time_after=time_after)
    213             df_tmp['Channel'] = c
    214             df_tmp['IdxChannel'] = idx_chan

~/.virtualenvs/sleepy3/lib/python3.7/site-packages/yasa/main.py in get_sync_events(data, sf, detection, center, time_before, time_after)
    237         idx = np.ma.compress_rows(idx_mask)
    238         print(idx)
--> 239         amps = data[idx]
    240         time = rng(0) / sf
    241

IndexError: arrays used as indices must be of integer (or boolean) type

happens when there is only one event (in my case spindle, but that doesn't matter) and that one event is at the beginning or at the end, hence idx_mask = np.ma.mask_rows(np.ma.masked_outside(idx, 0, data.shape[0])) line actually gets rid of anything and idx = []
for now, I manually solve it in the sense, that I am always centered around Peak so I manually check whether my Peak - time_before or Peak + time_after exceed limits of the timeseries and if so, I alter the time_before or time_after.
It'd be nice if yasa offers such an option, or an option to completely skip would be also nice.

Hi @jajcayn! Sorry for the late reply for some reason I didn't get notified. So it seems to me that a quick fix would be to add the following lines in the code, after idx = np.ma.compress_rows(idx_mask):
if len(idx) == 0:
    warning.warn('warning message bla bla bla...Returning None')
    return None  # Or empty pandas DataFrame

We can also, as you say, alter the time before and after but I think I'd prefer the more explicit solution of throwing a warning and returning None. At least the user knows that something is wrong.
What do you think? Thanks!

Dear Raphael,
thanks for the reply! I agree that explicit is definitely better than implicit, i.e. I would just warn the user and return None. As for the returning, I believe that None is better because then you can simply do something like
time_around = 1.  # second
events = None
while events is None:
    events = yasa.get_sync_events(...)
    time_around -= 0.1

I already did a small fix, opening a PR now :)
Best!


[BUG]

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
System.AggregateException: One or more errors occurred. (The process cannot access the file 'C:\Users\USER\AppData\Local\Bloxstrap\Versions\version-97058ca6653344cd\content\fonts\TwemojiMozilla.ttf' because it is being used by another process.)
[Inner Exception]
System.IO.IOException: The process cannot access the file 'C:\Users\USER\AppData\Local\Bloxstrap\Versions\version-97058ca6653344cd\content\fonts\TwemojiMozilla.ttf' because it is being used by another process.

Known bug, make sure Roblox is closed first before you try changing your font.

I have the same issue, and I have ensured roblox is closed before changing fonts. The weird part for me is that this issue ONLY occurs when i attempt to open more than 4 instances of roblox.

The next time you get this error, send the log file and I'll take a closer look.


Import Smart_Image snippets

Each smart image will need to be created as a Scene.
Each map_pin will need to be created as a Note (so that it can link to the relevant JournalEntry).
Do we need to handle the "subset_mask" to modify the image? Can we set a mask on the Scene?
Note.create({
  entryId: journalEntry.id,
  x: 1000,
  y: 1000,
  icon: "icons/my-journal-icon.svg",
  iconSize: 40,
  iconTint: "#00FF000",
  text: "A custom label",
  fontSize: 48,
  textAnchor: CONST.TEXT_ANCHOR_POINTS.CENTER,
  textColor: "#00FFFF"
});

Duplicates #3


Fix/landings typo

:tada: This PR is included in version 1.31.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Use broker name instead of id

Use sm broker name instead of sm broker id when registering a broker at the platform

Coverage decreased (-0.2%) to 91.661% when pulling 03808f2eaa53c670fd9f03ad1b2cc105c2a71f2c on broker-name-notifications into 1fa7df984c23c4d49c6d01c44f8f12b4b7c22778 on notifications.


Point docs from README to Rollbar's site

Keep the basic getting started info, then link to Rollbar's docs.

Done.


MAINT: properly impelment check_scoring

This PR makes check_scoring return a callable as mentioned in the sklearn docs: http://scikit-learn.org/dev/modules/generated/sklearn.metrics.check_scoring.html. It returns the sklearn's check_scoring unless we've implemented it.
It also does two other maintenance items:

Actually performs assert (gosh, did I really make that typo?)
Gets rid of FutureWarnings from SGDClassifier by specifying tol=1e-3.

Thanks!


Change Dockerfile base image to python

Description
We should use python:3.6 image as base image instead of ubuntu to reduce build time.
Screenshots
Files

Dockerfile

To Reproduce
Tasks
Include specific tasks in the order they need to be done in. Include links to specific lines of code where the task should happen at.

[ ] Change base image to python:3.6 in Dockerfile

Hi @matheuscmelo,
Sounds like a good idea to me.Could you knock this out and open a PR?


WIP: Update asynchronous state retrieval

Previously, the gather_state function in the network module used synchronous state retrieval. This commit updates the function to use asynchronous state retrieval by leveraging the retrieve_async method provided by the nispor::NetState struct.
This change ensures that state retrieval does not block the main execution thread, improving the overall responsiveness of the application. Additionally, it paves the way for integrating asynchronous operations into other parts of the codebase.

@jona42-ui Do you still have interest on this? Otherwise, I would like to step in because I want to use tokio/signal to replace ctrlc which is blocking our compiling in Fedora.

@jona42-ui Do you still have interest on this? Otherwise, I would like to step in because I want to use tokio/signal to replace ctrlc which is blocking our compiling in Fedora.

its okay @cathay4t for now for you to step in. though I still have interest.

OK. I will let you know once PR created for this.

@jona42-ui Please check https://github.com/nmstate/nmstate/pull/2676 for my approach.

Closing in the favor of https://github.com/nmstate/nmstate/pull/2676


porting: Add NPL for RIOT

@haukepetersen

note: this is pushed on top of https://github.com/apache/mynewt-nimble/pull/113 due to few build errors (will be rebased after merge)

Very nice!


Support Summer '15 Batch REST API

Unless I am missing the obvious, I don't see that JSForce supports yet the recently released REST API batch function.
Doc link: http://docs.releasenotes.salesforce.com/en-us/summer15/release-notes/rn_api_rest.htm

Batch support is not included yet. By the way, you can use much lower-level call Connection#requestPost() and pass the batch request message.
conn.requestPost('/composite/batch', {
  "batchRequests" : [{
    "method" : "PATCH",
    "url" : "v34.0/sobjects/account/001D000000K0fXOIAZ",
    "richInput" : {"Name" : "NewName"}
  },{
    "method" : "GET",
    "url" : "v34.0/sobjects/account/001D000000K0fXOIAZ"
  }]
}, function(err, res) {
  // ...
})

In future release we may add more excellent support for batch API request sending.

Any plans to implement this yet? Considering we now have Composite, Batch and Tree requests?


Attribute error in torch.nn.modules.module while applying the pretrained error.

I am applying the pretrained models on the images and encounted the following error, "torch.nn.modules.module.ModuleAttributeError: 'DataParallel' object has no attribute 'src_device_obj'". So far I am unable to resolve the error so please help me. Python version is 3.8.5 (I have also tried on 3.6) and torch version is 1.6.0.
Thanks.

What is it that you are doing that causes this error?
by the way, last time I checked 1.5.1 was working fine and 1.6 caused my loss to go to nans! so apart from this, keep this in mind in case you faced nans!

Well, I was trying to generate feature vectors for images. In readme it is written to use megaface,py  in which the model is being loaded like this ,
checkpoint = 'BEST_checkpoint.tar'
print('loading model: {}...'.format(checkpoint))
checkpoint = torch.load(checkpoint)
model = checkpoint['model'].to(device)
I was trying that, and was getting the error. but in lfw_eval.py the model is loaded differently,
checkpoint = 'BEST_checkpoint.tar'
checkpoint = torch.load(checkpoint)
model = checkpoint['model'].module
model = model.to(device)
model.eval()
Well, this works now and I am able to apply the model. Thanks!


Add FZF support for find members

Hi!  Wanted to check if you would be interested in adding FZF support for find members.  Took a quick pass at it based on the work from find usages and it worked.  https://github.com/OmniSharp/omnisharp-vim/pull/621.
Let me know what else needs to be added and I can update the PR.

This looks good, thanks.
Would you mind adding the new g:OmniSharp_selector_findmembers variable to the help (doc/omnisharp-vim.txt)?

Just added, wasn't sure if I got the formatting right.

Thanks very much, @e82eric!

Whoops sorry! I forgot to merge 😂

Beautiful! Absolutely worth the wait! You guys are the best!


Question: how can I filter out resources from an ARM template

I want to iterate over an ARM template, filter out resources of a specific type, and apply the filtered template. How can I do this via the SDK? An example would be nice (btw this repo used to have examples but not anymore?)

Howdy,
To answer half of your question, you can find our examples regarding resources (including ARM Template deployment) here: https://github.com/Azure-Samples/azure-sdk-for-go-samples/tree/master/resources.
That said, I don't believe we have much tooling to support that scenario in our SDK. Could you tell me more about why you want to only deploy a partial template?

Our template includes NSGs which are getting updated post-deploy by Azure as the LB services are created. When we try to apply the same template during an upgrade, the NSG updates performed by Azure are getting overwritten, resulting in a temporary outage (see https://github.com/openshift/openshift-azure/issues/329) so I wanted to filter out NSGs. Since we use a Golang template to generate the ARM template, I ended up doing the filtering during the generation of the ARM template. Although there have been other times I wanted to do some sort of filtering in Go on a deployment template (eg. another time I wanted to parse a template, filter the scale set, record its CSE value), I don't have any immediate need for this so feel free to close.

Thanks for expanding on this. I think the scenario you're describing warrants some attention from our Azure Networking team. Tools like Terraform work around this by not overwriting existing NSGs, and I'd love to see that same logic reproduced when an ARM deployment is issued.
Mind opening an issue in the Azure/azure-rest-api-specs repository describing the temporary outage that you're seeing, @kargakis? That will help us escalate to the correct team. Feel free to mention me so that I can help follow up on it.

Mind opening an issue in the Azure/azure-rest-api-specs repository describing the temporary outage that you're seeing, @kargakis? That will help us escalate to the correct team. Feel free to mention me so that I can help follow up on it.

@amanohar and @jim-minter should have more info about it

+1 on this. My team would benefit from this as well.

I'm going to close this issue, but we'll note that there is demand for being able to work with ARM Templates in Go.


Support viewing Instagram photos and comments on page

Right now the Instagram photos are thumbnails that simply link to the content on Instagram. Since my new Instagram plugin is connected to the Personal API instance behind this site, I should be able to use the data from the API to create a more enhanced, feature-rich Instagram experience.

I'm toying around with design ideas for this. I want to keep this as light as possible initially, and am leaning towards something like this.

Minimum requirements
[ ] Image, avatar, username, location name and text from Instagram data.
[ ] I'd like the CTA to be large and apparent, and I will implement click event tracking on the button.
[ ] The avatar and username will link to instagram.com/ + username.
Nice to have
[ ] Navigate images from within the modal.
[ ] Render comment count and like count.
[ ] Render created at date.

Updated design idea covering all of the "nice to have" items.


Refactor coil generation

Consider moving the coil generation process into multiple steps

generate spline curve
sweep cross-section

May also want to keep the spline curve after the sweep for possible reuse?

This issue may be null in the context of the new magnet modeling workflow.

I think was either implemented in recent magnet workflow improvements or rendered irrelevant.


[BOT PR] to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: istio-testing
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: wattli
Assign the PR to them by writing /assign @wattli in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
f3dc5f318e30aa10f70722dd0af87991c068f42c
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
f3dc5f318e30aa10f70722dd0af87991c068f42c
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
f3dc5f318e30aa10f70722dd0af87991c068f42c
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

dup of #5807


Fix tests on Windows

Fixes #1174

I updated the pipeline to force Windows tests to pass. Looks like there are still some failures

I am working on an issue in go-ape where "d:" is not recognized as a scheme, which is somewhat related to this.

I don't know what to make of those test failures. It works on my Windows machine. I even set up a D: drive to test it:
PS D:\a\1\s\projectriff\riff> go test ./...
?       github.com/projectriff/riff/cmd/riff    [no test files]
ok      github.com/projectriff/riff/pkg/core    1.152s
ok      github.com/projectriff/riff/pkg/core/kustomize  1.284s
?       github.com/projectriff/riff/pkg/core/kustomize/mocks    [no test files]
?       github.com/projectriff/riff/pkg/core/mocks      [no test files]
?       github.com/projectriff/riff/pkg/core/mocks/mockbuilder  [no test files]
ok      github.com/projectriff/riff/pkg/core/tasks      0.829s [no tests to run]
?       github.com/projectriff/riff/pkg/core/vendor_mocks       [no test files]
?       github.com/projectriff/riff/pkg/core/vendor_mocks/mockbuild     [no test files]
?       github.com/projectriff/riff/pkg/core/vendor_mocks/mockserving   [no test files]
?       github.com/projectriff/riff/pkg/env     [no test files]
ok      github.com/projectriff/riff/pkg/fileutils       1.206s
?       github.com/projectriff/riff/pkg/fileutils/mocks [no test files]
?       github.com/projectriff/riff/pkg/kubectl [no test files]
?       github.com/projectriff/riff/pkg/kubectl/mocks   [no test files]
?       github.com/projectriff/riff/pkg/osutils [no test files]
ok      github.com/projectriff/riff/pkg/riff/commands   2.102s
?       github.com/projectriff/riff/pkg/test_support    [no test files]

Please see: https://github.com/pivotal/go-ape/pull/11/commits/2bfb4c0b0ecf7581776c6254ad753fba6ad7d663
Once that PR is merged, we should get that fix here.

What is the difference between the CI environment and my Windows 10 install? I don't see this issue on the latter.

Looks like https://github.com/projectriff/riff/blob/1376aee9f976a243d60e6a2e6e75587383a19ecb/pkg/core/manifest_test.go#L41 is returning errors we didn't anticipate, meaning the manifest to be nil and causing nil dereference panics.

rebased


[DO NOT MERGE] Raise an error from a notebook to test CI/CD checks

This PR is a spot check to follow up with #66, to verify that CI/CD is working as expected. We add one cell to a notebook that raises an exception, and see if this breaks the CI/CD system.

Raising an exception from a notebook cell does not cause Travis to fail. Something is still broken with the step that checks that the notebooks all run OK.

This is not a bug - this particular notebook is excluded from CI.
https://github.com/HumanCellAtlas/data-consumer-vignettes/blob/77ce685bc841d4d8bedab3ee1fe138c4afbfd93a/test/ignore#L2
In the long term, it should be tested, I just haven't gotten around to tweaking the notebook so that it runs nicely since I've started working on vignettes.


fix: Add missing hpa metric to kubernetes_state and kubernetes_state_core checks

What does this PR do?

Add missing metrickube_hpa_status_target_metric to the kubernetes_state and kubernetes_state_core checks.
Motivation

A user will be able to monitor the state of the target metrics.
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

I will open a PR that only updates kube_state_core


Can You Please explain More, I am Noob and willing to learn.

Just Explain me How Spoofing mail Works and Exactly what and How It's Going to Happen. And guide me so that I can also start. How to get FQDN please help

Added "Email Spoofing 101" section to README.md. Please contact me directly if you need further assistance -- @lunarca_ on twitter.


Marker Position

A question
I want to put a marker that is an arrow that always points to the south and that does not rotate with the camera. It's possible?
I await your response, thanks for your time

Maybe something like this?
map.addMarker({
  position: ...
  bearing: 180,
  flat: true
}, function(marker) {

});


Added responsive support. Added update and destroy public methods.

Public methods added (Issue #21):
//recalibrate the jQuery selection.
$('#target').ellipsis('update');

//remove the jquery-ellipsis plugin from jQuery selection.
$('#target').ellipsis('destroy');

Event binding added to window for responsive support (Issue #20). Destroy method cleans up event binds in addition to restoring text to original.

I tried to use  ellipsis('update')，but It does not work, and when the browser widens, the text is not displayed correctly and is truncated. It is still displayed as the small resolution. Could you give me a concrete  example?

@fxfeiyi are you using the version in this pull request? These changes were never merged.
https://github.com/marcwhitbread/jquery-ellipsis


Remove gazebo_ros_pkgs and dependencies from Rolling

Gazebo classic aka gazebo11 package is not present in Ubuntu Noble nor is the plan of the Gazebo simulation team to package it anymore since the recommended path is to migrate to the new Gazebo, see https://gazebosim.org/docs/latest/migrating_gazebo_classic_ros2_packages.
I'm completely removing in this PR the entries from the rolling distribution.yaml, not sure if there is any better way of dealing with this kind of "retirement".

@j-rivero friendly ping

I'm actually going to close this one out for now, since we've removed some of this stuff separately.  We can always open other, smaller PRs with the individual things that we need to remove.


Fix(?) CRC DNS

In CRC, we can't really configure existing interfaces - they get
overridden by some other process, meaning we can't ensure it won't get
any public DNS.
In order to ensure we use only the local dnsmasq , the best way is
apparently to unmanage the /etc/resolv.conf from within NetworkManager,
and hardcode the content.
Even with that, the comment seems to be removed and replaced by
# Generated by NetworkManager - which is... weird. Since it shouldn't
do anything in there...
We use the CRC "public" IPv4 - apparently <IP_ADDRESS> doesn't work, at
least from the openshift-dns/dns-default pod (throws connections
refused).
In any cases, this change seems to work (for now), ensuring DNS are
properly working.
We might want to amend the DNS configuration on the controller-0 to use
the openshift-dns/dns-default service instead - but that's for a
follow-up.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running

/lgtm

/approve


SIMPLE-6020 sped up syncing operational by removing unneeded API calls

None of the test failures seem to be my fault, so this should be ready to merge.


build 365 render issues,

gas condiuts work fine now but they still render gliched out?

is this confirmed?

#2174


Constructor Returns Subclass of Constructed Class: Extra Newness v2: Deceptively Sensical Looking

I previously fiddled around with the new new Foo('foo')('bar') pattern by extending Function.  However, after working with more reasonable code my brain rebelled once again and remembered an odd fact about the new operator: if the function called as a constructor returns an object, that object becomes the result of the new operator, not the constructed instance!
This means we can write a far more fun example:
class Renewable {
  static get values() {
    return [];
  }

  constructor(value) {
    const Base = this.constructor;
    const prevValues = Base.values;

    return class $Renewable extends Base {
      static get values() {
        return [...prevValues, value];
      }

      constructor(nextValue) {
        return super(nextValue);
      }
    };
  }
}

Which means you can now write the most obnoxious chained call that no one ever wanted:
new new new Renewable('Foo')('Bar')('Zap').values; // => ['Foo', 'Bar', 'Zap']

This is deceptively simple until you look closer, in which case it becomes a recursive pile of dumb:
class Renewable {
  // We want to set an initial value for `constructor.values` just to make
  // later code cleaner.
  static get values() {
    return [];
  }

  // A perfectly ordinary constructor on a perfectly (not) ordinary base class.
  constructor(value) {
    // Begin BS: since this is a base class, we can reference `this` without
    // a call to `super()`.  `this` will be the constructed/ing instance,
    // and as an instance it will have a reference to its constructor.
    // AN IMPORTANT NOTE: If this instance is an instance of a _subclass_,
    // `this.constructor` will be a reference to that _subclass's constructor_,
    // not to the base class!
    const Base = this.constructor;

    // We defined `values` as `static`, so it's attached to the constructor.
    const prevValues = Base.values;

    // Remember, a Class is referenced by its Constructor Function, so we can
    // just use that Constructor off of `this` as the base class for this new class...
    // NOTE: This is a _return statement_!  We're not just defining a subclass
    // of the current instance's class, we're also replacing the result of
    // the `new` operator by returning some object other than `this`!
    return class $Renewable extends Base {

      // In each subclass, we return all the values from the base class plus the
      // next value sent through the constructor.
      static get values() {
        return [...prevValues, value];
      }

      // A seemingly innocuous subclass constructor, which just passes the value
      // on to the super class's constructor...
      constructor(nextValue) {
        // ... except that it's continuing the trainwreck of madness by returning
        // the result of super()!
        return super(nextValue);
      }
    };
  }
}

So what happens each time we call new Renewable('Foo')?

Renewable's constructor function is called with 'Foo'
Base is set to this.constructor, which the first time around will be Renewable itself.
prevValues is set to Base.values, which the first time is Renewable.values, which is [].
A new class $Renewable is defined that extends Base, which right now is just Renewable itself.
That new class $Renewable is returned instead of the newly constructed instance, which results in new returning that subclass rather than the constructed instance of Renewable.

At that point, if we checked .values we'd see an array: ['Foo']
What happens with new new Renewable('Foo')('Bar') then?

First, all of the above occurs, causing the first new operator to return that subclass constructor.
Then, the next new operator calls that returned constructor with 'Bar'.
The fun begins with $Renewable's constructor calling super(nextValue): This goes up to the original Renewable constructor.
In that constructor, this.constructor is now $Renewable's constructor!
And Base.values will then be $Renewable.values, which is ['Foo'].
And then... Renewable's constructor creates another subclass, this time of the first $Renewable class!
... And then returns that sub-sub-class, continuing the unvirtuous cycle.

Thus, from that, new new Renewable('Foo')('Bar').values will be ['Foo', 'Bar'].
Because each class just returns another subclass, this can be done as far as you want!
Edit: The Same Pattern, but Fewer Class Keywords
Despite the trickery to support the class keyword, Classes in JS are still essentially functions, albeit rather funky ones.
This extra-new pattern however is trivial to implement even without this newfangled feature:
const Renewable = (() => {
  function Renewable(value) {
    const Base = this.constructor;
    const prevValues = Base.values;

    function $Renewable(nextValue) {
      return Renewable.call(this, nextValue);
    }

    // Uncomment if you actually care about the prototype chain...
    // $Renewable.prototype = Object.create(Base.prototype);
    // $Renewable.prototype.constructor = $Renewable;

    Object.defineProperty($Renewable, 'values', {
      get() {
        return [...prevValues, value];
      },
    });

    return $Renewable;
  }

  Object.defineProperty(Renewable, 'values', {
    get() {
      return [];
    },
  });

  return Renewable;
})();

Would you like to send a PR with this?

I do apologize for completely missing this for so long, I'll try to get a PR in at some point though life has sadly been a bit preoccupying


#41, #36

As the subject says, this is fixing #41 and #36.
The changes should be relatively straightforward actually, with the non-trivial changes actually being part of the espidf module in embuild.
See this PR for the embuild-related changes:
https://github.com/ivmarkov/embuild/pull/29

Okay, I think this is close to finished.
Still needs some more testing though, and code reviews are appreciated.
I've refactored @ivmarkov 's code pretty heavily and also incorporated the changes I've made in embuild (though the author of the first commit is still, ivmarkov; hope you don't mind).
The biggest change is that EspIdf now uses a closure for the tools to be installed, this allows to decide dynamically which tools should be installed. This change together with the added EspIdfVersion that is parsed from the esp-idf/tools/cmake/version.cmake file now allows us easily to very easily support multiple esp-idf versions.
I've also added a big match statement for selecting the esp-idf detected from the environment, the repo specified with IDF_PATH or everything managed by the installer.
Basically:

If activated esp-idf available and $ESP_IDF_TOOLS_INSTALL_DIR == "fromenv" or unset, use it
If $IDF_PATH is valid, use it as user provided esp-idf repository (but installer manages tools)
Otherwise the installer manages everything and respects ESP_IDF_REPOSITORY and ESP_IDF_VERSION.

There are also some error and warnings for edge-cases.


Accept different HTTP request methods

Introduced an interface for EnvironmentPropertiesInterface to decouple EnvironmentProperties
Can now accept different HTTP request methods by using a RequestInterface
Documentation has been updated to follow ThePHPLeague skeleton

Backwards compatibility has been maintained. If no dependencies are passed, the defaults are loaded.

Codecov Report

Merging #33 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #33   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      3    +1
  Lines          54     60    +6
=====================================
+ Hits           54     60    +6

Impacted Files
Coverage Δ

src/Request/Request.php
100% <100%> (ø)

src/Environment/EnvironmentProperties.php
100% <100%> (ø)
:arrow_up:

src/CliRequest.php
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[WIP] Implementation of SVD

@niketanpansare, @mboehm7, @bertholdreinwald, @iyounus
This initial implementation is keeping in line with the discussion on the dev mailing list.
The CP 'svd' function is implemented and backed by Apache Commons Math.

As per the discussion on the mailing list, the implementations looks good to me. We should definitely consider adding lu, qr, svd, eigen to our Spark backend.

@nakul02 @niketanpansare Can I merge this?

I am OK with merging it in if @iyounus still wants to use it for his distributed SVD script.
@nakul02 Please add Spark SVD instruction when you have time :)

Thanks @niketanpansare. I'll check with @iyounus.
@iyounus @nakul02 Should this be merged?

@nakul02 could you please also add some basic tests for the svd builtin function (e.g., to org.apache.sysml.test.integration.functions.unary.matrix) and remove the wildcard import from LibCommonsMath (in order to prevent name clashes on auto import resolution)? Thanks.

I don't think we should merge this right now. We still need to figure out if this is the route we're taking. What we have here may not be the right choice for SVD.

@iyounus Thanks.

6(!) month check-in. What's the status of this PR? Merge or close?

I think we can just close this. I implemented this based on a recent paper but now I think this method is not very beneficial when it comes to scaling and that paper was somewhat misleading. So at this moment we can close this. This require some more thinking.

@nakul02
Based on the comment from @iyounus, can we go ahead and close this PR?

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/1602/

Yes, I shall close it.

@nakul02 can you rebase this branch of yours. I will clone that. Thanks.:)

I've rebased it - https://github.com/nakul02/systemml/tree/svd
Here on out, you can rebase it yourself. Clone my svd branch on to the clone/fork of the systemml repository and rebase that with master.
If you run into issues doing that, please let me know.


Add: header_included_v6() and set_header_included_v6()

Also, deprecate header_included() and set_header_included() in favor of header_included_v4() and set_header_included_v4()
Close #517

Thanks  @jjnicola

Thanks you @Thomasdezeeuw .

Hi! do you know when this will be available in the crate?


Loop problems when the number of slides is less than slidesPerView * 2

Check that this is really a bug

[X] I confirm

Reproduction link
https://deep-space.blue/testing/js-example/swiper/report-for-github-2023-03-28.html
Bug description
There is a problem with behavior when loop:true, slidesPerView:3 is set and the number of slides is 4 or 5.

When I click swiper-button-next, pagination does not move to the last slide, and after one or two clicks, it does not work.
When swiper-button-prev is clicked, pagination moves to the next slide instead of the previous one. (when there are 4 slides)
When swiper-button-prev is clicked, it moves to the previous two slides. (when there are 5 slides)

Expected Behavior
I am aware that the loop specification has changed since v8, but a method needs to be provided to make it behave as in v8 below when the number of slides is 4 or 5.
https://deep-space.blue/testing/js-example/swiper/card-v8.html
Actual Behavior
No response
Swiper version
9.1.1
Platform/Target and Browser Versions
Windows Chrome 111
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Docs clearly states the requirement


NoMethodError: undefined method `[]' for nil:NilClass

Cookbook version
8.0.1
Chef-client version
Chef Client, version 12.11.18
Platform Details
Ubuntu 14.04.4 LTS trusty
Scenario:
I am trying to install a simple database server using the mysql chef cookbook.
Steps to Reproduce:
I am using the cookbook to install a MySQL Database server on a Chef Node running Ubuntu 14.04 Trusty.

I downloaded the latest version of the mysql cookbook (8.0.1), untared and moved the 'mysql/' directory to the 'cookbooks/' directory, and added the dependency to the cookbook metadata.rb file as follows:
depends 'mysql', '~> 8.0.1'

Created a new recipe called database.rb and inserted the following  code:
mysql_service 'default' do
    port '3306'
    version '5.5'
    initial_root_password 'djb5944014'
    action [ :create, :start ]
end

Inserted the database.rb recipe into the default.rb recipe:
include_recipe 'icinga2-crossover::database'

Uploaded the cookbook and its dependencies to the Chef Server:
$ knife cookbook upload icinga2-crossover --include-dependencies

Added the cookbook to the run list of the Chef Node:

$ knife node run_list add chef-node-one "recipe[icinga2-crossover]"

Ran the Chef Client on the Chef Node:
$ sudo chef-client

Expected Result:
I am expecting that the MySQL Server gets installed after running the chef-client command on my chef node.
Actual Result:
chefnode@chef-node:~$ sudo chef-client
Starting Chef Client, version 12.11.18
resolving cookbooks for run list: ["icinga2-crossover"]
Synchronizing Cookbooks:
  - icinga2-crossover (0.1.0)
  - mysql (8.0.1)
Installing Cookbook Gems:
Compiling Cookbooks...
Converging 1 resources
Recipe: icinga2-crossover::database
  * mysql_service[default] action create
    * mysql_server_installation_package[default] action install
      * apt_package[mysql-server-5.5] action install (up to date)
      * apt_package[perl-Sys-Hostname-Long] action nothing (skipped due to action :nothing)
      * execute[Initial DB setup script] action nothing (skipped due to action :nothing)
       (up to date)
    * mysql_service_manager_upstart[default] action create
      * group[mysql] action create (up to date)
      * user[mysql] action create (up to date)
      * service[mysql] action stop (up to date)
      * service[mysql] action disable (up to date)
      * file[/etc/mysql/my.cnf] action delete (up to date)
      * file[/etc/my.cnf] action delete (up to date)
      * link[/usr/share/my-default.cnf] action create (skipped due to not_if)
      * directory[/etc/mysql-default] action create (up to date)
      * directory[/etc/mysql-default/conf.d] action create (up to date)
      * directory[/run/mysql-default] action create (up to date)
      * directory[/var/log/mysql-default] action create (up to date)
      * directory[/var/lib/mysql-default] action create (up to date)
      * template[/etc/mysql-default/my.cnf] action create (up to date)
      * bash[default initial records] action run
        - execute "bash"  "/tmp/chef-script20160728-17026-nl1xuf"
      * apt_package[apparmor] action install (up to date)
      * directory[/etc/apparmor.d/local/mysql] action create (up to date)
      * template[/etc/apparmor.d/local/usr.sbin.mysqld] action create (up to date)
      * template[/etc/apparmor.d/usr.sbin.mysqld] action create (up to date)
      * template[/etc/apparmor.d/local/mysql/default] action create (up to date)
      * service[default apparmor] action nothing (skipped due to action :nothing)

  * mysql_service[default] action start
    * mysql_service_manager_upstart[default] action start
      * template[/usr/sbin/mysql-default-wait-ready] action create (up to date)
      * template[/etc/init/mysql-default.conf] action create (up to date)
      * service[mysql-default] action start

        ================================================================================
        Error executing action `start` on resource 'service[mysql-default]'
        ================================================================================

        NoMethodError
        -------------
        undefined method `[]' for nil:NilClass

        Resource Declaration:
        ---------------------
        # In /var/chef/cache/cookbooks/mysql/libraries/mysql_service_manager_upstart.rb

         42:       service mysql_name do
         43:         provider Chef::Provider::Service::Upstart
         44:         supports status: true
         45:         action [:start]
         46:       end
         47:     end

        Compiled Resource:
        ------------------
        # Declared in /var/chef/cache/cookbooks/mysql/libraries/mysql_service_manager_upstart.rb:42:in `block in <class:MysqlServiceManagerUpstart>'

        service("mysql-default") do
          provider Chef::Provider::Service::Upstart
          action [:start]
          supports {:status=>true}
          retries 0
          retry_delay 2
          default_guard_interpreter :default
          service_name "mysql-default"
          pattern "mysql-default"
          declared_type :service
          cookbook_name "icinga2-crossover"
        end

        Platform:
        ---------
        x86_64-linux

      ================================================================================
      Error executing action `start` on resource 'mysql_service_manager_upstart[default]'
      ================================================================================

      NoMethodError
      -------------
      service[mysql-default] (/var/chef/cache/cookbooks/mysql/libraries/mysql_service_manager_upstart.rb line 42) had an error: NoMethodError: undefined method `[]' for nil:NilClass

      Resource Declaration:
      ---------------------
      # In /var/chef/cache/cookbooks/mysql/libraries/mysql_service.rb

       50:           svc = mysql_service_manager(name, &block)
       51:         when 'sysvinit'

      Compiled Resource:
      ------------------
      # Declared in /var/chef/cache/cookbooks/mysql/libraries/mysql_service.rb:50:in `svc_manager'

      mysql_service_manager_upstart("default") do
        action [:start]
        retries 0
        retry_delay 2
        default_guard_interpreter :default
        declared_type :mysql_service_manager
        cookbook_name "icinga2-crossover"
        version "5.5"
        initial_root_password "djb5944014"
        port "3306"
        instance "default"
        run_user "mysql"
        run_group "mysql"
        socket "/run/mysql-default/mysqld.sock"
      end

      Platform:
      ---------
      x86_64-linux

    ================================================================================
    Error executing action `start` on resource 'mysql_service[default]'
    ================================================================================

    NoMethodError
    -------------
    mysql_service_manager_upstart[default] (/var/chef/cache/cookbooks/mysql/libraries/mysql_service.rb line 50) had an error: NoMethodError: service[mysql-default] (/var/chef/cache/cookbooks/mysql/libraries/mysql_service_manager_upstart.rb line 42) had an error: NoMethodError: undefined method `[]' for nil:NilClass

    Resource Declaration:
    ---------------------
    # In /var/chef/cache/cookbooks/icinga2-crossover/recipes/database.rb

      7: mysql_service 'default' do
      8:    port '3306'
      9:    version '5.5'
     10:    initial_root_password 'djb5944014'
     11:    action [ :create, :start ]
     12: end
     13:

    Compiled Resource:
    ------------------
    # Declared in /var/chef/cache/cookbooks/icinga2-crossover/recipes/database.rb:7:in `from_file'

    mysql_service("default") do
      action [:create, :start]
      updated true
      retries 0
      retry_delay 2
      default_guard_interpreter :default
      declared_type :mysql_service
      cookbook_name "icinga2-crossover"
      recipe_name "database"
      port "3306"
      version "5.5"
      initial_root_password "djb5944014"
      install_method "auto"
      service_manager "auto"
    end

    Platform:
    ---------
    x86_64-linux

Running handlers:
[2016-07-28T23:43:58-04:00] ERROR: Running exception handlers
Running handlers complete
[2016-07-28T23:43:58-04:00] ERROR: Exception handlers complete
Chef Client failed. 3 resources updated in 04 seconds
[2016-07-28T23:43:58-04:00] FATAL: Stacktrace dumped to /var/chef/cache/chef-stacktrace.out
[2016-07-28T23:43:58-04:00] FATAL: Please provide the contents of the stacktrace.out file if you file a bug report
[2016-07-28T23:43:58-04:00] ERROR: mysql_service[default] (icinga2-crossover::database line 7) had an error: NoMethodError: mysql_service_manager_upstart[default] (/var/chef/cache/cookbooks/mysql/libraries/mysql_service.rb line 50) had an error: NoMethodError: service[mysql-default] (/var/chef/cache/cookbooks/mysql/libraries/mysql_service_manager_upstart.rb line 42) had an error: NoMethodError: undefined method `[]' for nil:NilClass
[2016-07-28T23:43:58-04:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)

Resolved?

So how this issue was actually resolved?

I'd like to know as well...

I'm seeing this exact problem now after a week of successfully using this cookbook. The problem doesn't happen on the first chef run, only on subsequent runs.

This is an issue with the Upstart provider when the service is not in a start or stop state or when the output of /bin/status  doesn't match this regex https://github.com/chef/chef/blob/9d09f9e23db45eaf71bc9421507a25fccac1293b/lib/chef/provider/service/upstart.rb#L36
Also I think there is another issue here with the Mysql upstart config, it times out in the post-start section regardless of whether Mysqld is trying to start or erred out immediately. In the case you run chef it hangs on waiting for mysql to come up (although it never will) then when you Ctl-C or the resource times it seems the upstart wait script continues to run, at this point when you rerun chef the state is in post-start which then triggers the bug in the upstart provider.

Opened a ticket for the upstart issue.
https://github.com/chef/chef/issues/5513


Elm API suggestion (breaking change) – use records for most parameters to call and callNoArgs

To aid future code readability, I'd like to request the use of records when specifying the function name and JSON decoder/encoder arguments for call and callNoArgs (since I believe they would always be specified together).
Below are examples of how calls with the current & proposed API would look:
-- current API: I don't find this clear enough at a glance when reading the code
testProcedureConfusing : Int -> Task.Task (Error String) Int
testProcedureConfusing =
  TaskPort.call "testProcedure" JD.int JD.string JE.int

-- proposed API: define my test procedure
testProcedure : Int -> Task.Task (Error String) Int
testProcedure =
  TaskPort.call
    { functionName = "testProcedure"
    , argsEncoder = JE.int
    , bodyDecoder = JD.int
    , errorDecoder = JD.string
    }

-- an example task to call the test procedure with an argument
taskForTestProcedureSpecifying3 : Task.Task (Error String) Int
taskForTestProcedureSpecifying3 =
  testProcedure 3

Here's one way call could be defined to support this:
call :
  { functionName : String
  , argsEncoder : args -> JE.Value
  , bodyDecoder : JD.Decoder body
  , errorDecoder : JD.Decoder error
  }
  -> args
  -> Task.Task (Error error) body
call { functionName, bodyDecoder, errorDecoder, argsEncoder } args =
  callWithJson functionName bodyDecoder errorDecoder <| argsEncoder args

@GabrielBoehme13 I think there is now a critical mass of new features (#7 #6 #3 in particular, plus #8 and #9 todo) that warrant a breaking change. I also adopted the suggestion here in newly added callNS and callNoArgsNS functions added as part of #7 and #6. I will be working on the new public API along the lines of what you had suggested, but in the meantime would appreciate any feedback on the above.

Implemented in v2.0.0


Error reporting while building the project: Cannot find module 'css-animation/lib/Event'

Steps to reproduce

Run npm install docsite -g to install the dev tool.
Run npm i in the root directory to install the dependencies.
Run docsite start in the root directory to start a local server, you will see the website in 'http://<IP_ADDRESS>:8080'.

What is expected?
Build should be done without any error
What is actually happening?
Getting below error
(node:1835) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Cannot find module 'css-animation/lib/Event'

I search for this error and find it may cause by Antd.
Antd' issue about this :Antd Webpack build failing (Can't resolve 'css-animation/es/Event')
I follow the solution above and it make success, the blog can be displayed correctly.
I am not sure if it only happens to me because it's a question about a mouth ago.
If it's a question happen to everyone, please fix it; otherwise, close my issue.

node -v ,  please make sure the version of  your node.js  is 8

node -v , please make sure the version of your node.js is 8

my current version id 8.4.0, but at first it was 12 until I find something wrong.
I'm not familiar with node.js, maybe the reason is this.


Trim and remove spaces from ID with user alert

Summary by CodeRabbit

New Features

Added a new function to improve ID processing across different components, enhancing input handling and consistency.

Bug Fixes

Adjusted ID assignment in various components to use the new ID processing function, ensuring more reliable and standardized IDs.

please reference the related issue in the PR or the PR in the issue.


Simplify docker regex

This is getting out of hand, trying to keep it a bit simpler.

The CI passes, so I guess that it is fine? These regexps are always a hassle to maintain and review

Turns out the kb- part of the regex is currently necessary (unfortunately) because we currently special case KB- in mcr.microsoft.com/windows/servercore images. The specs in in #8442 will make these changes fail specs. Setting as WIP.

I removed the -kb simplification from this PR. Back to ready.


Theme for linux terminals

Hello,
is it possible to create a theme for Linux terminals?
The .terminal file, I believe, only works for MacOSX.
Is there a workaround for Linux users? I personally use gnome-terminal.
Thank you.

Yes, I'll see what I can do when I get a chance. Recently I use macOS heavily :).

I have made a experimental one for gnome terminal: gnome. After running space-vim-dark.sh and restart the terminal, you should be able to see the effect. Let me know if you have any ideas.

I will be happy to create a linux theme for X based terminals and applications (e.g. URxvt, Xterm, and more).

@axvr Great! It would be nice to have these support.


godot 3

This is not a question but a request for help. I am trying to use your custom path finder in a godot 3 project without success. I think there is something I am missing in what you've done. So I tried to create a new project in godot3 replicating yours but it's not working. I would love if you could take a look in case you can quickly see what I have missed. Project is at https://github.com/ttux/GodotIsoCustomPathfinder3

Looked at your project. Thing is that in Godot 3.0, arrays are by default references, if you define an array like Array() inside a function that gets called multiple times or in an iteration, it doesn't reset the array like in 2.1.4. This made some problems in the pathfinder.gd
Also, in the draw.gd, you didn't initialized the fixed frame process. In 3.0 it is now called _physics_process to avoid confusion. So the changes are:
[drawn.gd]
Rename _fixed_process(delta) to:
_physics_process(delta) @ line 21
Initialize the physics_process in the _ready() by putting:
set_physics_process(true) @ line 20
[pathfinder.gd]
Declare an empty array by replacing Array() for [] @ line 36
Declare an empty array by replacing Array() for [] @ line 143
And that's it.

Ah ah, I just found that out and saw your comment now... I created an issue on github https://github.com/godotengine/godot/issues/13157
I updated the project to use priority = [] instead
Should I delete the issue I created on godot github?

Cool. Yes, I was waiting until the first 3.0 demo to do it but I guess I'll push it now, for consistency.


chore: test should treat UserRole as resource

:tada: This PR is included in version 1.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


refactor: use new convenience wrapper and caching

Little refactor following on from the PR which added the python-gitlab wrapper, to try use the common functions (and caching) where possible.
I think caching on get of group/project is fine in these scenarios but there will be some where we need to most up to date state, so we shouldn't use cached result all the time.
What do you think of naming the functions get_x_by_name_cached ? to make it clearer

@gdubicki @amimas - updated following comments


update TLDS_ALPHA_BY_DOMAIN

Codecov Report

Merging #663 (16aa517) into master (089f68b) will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #663      +/-   ##
==========================================
+ Coverage   70.08%   70.12%   +0.04%
==========================================
  Files         125      125
  Lines       11382    11382
==========================================
+ Hits         7977     7982       +5
+ Misses       3405     3400       -5

Impacted Files
Coverage Δ

assemblyline/common/net_static.py
100.00% <ø> (ø)

assemblyline/odm/randomizer.py
95.25% <0.00%> (+0.86%)
:arrow_up:

assemblyline/common/heuristics.py
86.95% <0.00%> (+2.17%)
:arrow_up:

assemblyline/datasource/alert.py
92.30% <0.00%> (+3.84%)
:arrow_up:

Continue to review full report at Codecov.

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


Default Slurmctl port range spans Slurmdb port and ExternalDbd port.

Environment:

AWS ParallelCluster / CfnCluster version:  aws-parallelcluster-2.9.1

Bug description and how to reproduce:
The latest SlurmConf file at /opt/slurm/etc/slurm.conf has the following ctld port range
SlurmctldPort=6817-6827

However the Slurmdb port and DbdPort operate on 6818 and 6819 respectively.
Both /opt/slurm/sbin/slurmdbd (to start the slurm database daemon) and systemctl start slurmctld can therefore not be run simultaneously or they'll error out. Running them with the -D command (foreground) shows this pretty clearly.
It is therefore not possible on the standard settings to connect to an RDS database as shown through the tutorial here
From slurm.conf man page:

SlurmctldPort
The port number that the Slurm controller, slurmctld, listens to for work. The default value is SLURMCTLD_PORT as established at system build time. If none is explicitly specified, it will be set to 6817. SlurmctldPort may also be configured to support a range of port numbers in order to accept larger bursts of incoming messages by specifying two numbers separated by a dash (e.g. SlurmctldPort=6817-6818). NOTE: Either slurmctld and slurmd daemons must not execute on the same nodes or the values of SlurmctldPort and SlurmdPort must be different.
Note: On Cray systems, Realm-Specific IP Addressing (RSIP) will automatically try to interact with anything opened on ports 8192-60000. Configure SlurmctldPort to use a port outside of the configured SrunPortRange and RSIP's port range.

Hi Alexis,
you are right in the fact that with 2.9 version we increased the number of ports used by Slurmctld in order to handle a bigger number of concurrent connections. And yes there is some overlapping with other default ports: the 6818 used by Slurmd can be ignored since slurmd does not run on the head node; regarding 6819 which is the default for slurmdbd you can simply change it by using a different port in slurmdbd.conf with DbdPort. If needed you can also change the ports used by Slurmctld without affecting the system.
In the next version we could change the ports for SlurmctldPort in order not to overlap with the default dbd port, but again you are free to change them in your cluster to support your custom setup.
Francesco

Thanks @demartinofra, I think it's important that those ports don't overlap.
After spinning up my instance (which starts wth slurmdb daemon on the master node  /opt/slurm/sbin/slurmdbd in the post-install script)
I then killed it running
$ ps -ef | grep slurmdbd
ec2-user  1755   863  0 16:01 pts/0    00:00:00 grep --color=auto slurmdbd
slurm     7674     1  0 11:33 ?        00:00:00 /opt/slurm/sbin/slurmdbd
$ sudo kill 7674

I then restarted the control service
$ sudo systemctl restart slurmctld

And then when trying to run sacct on the master node got
$ sacct -j2
sacct: error: slurm_persist_conn_open_without_init: failed to open persistent connection to ip-10-2-1-239:6819: Connection refused
sacct: error: slurmdbd: Sending PersistInit msg: Connection refused
sacct: error: Problem talking to the database: Connection refused

I can then restart the slurmdb daemon and slurm control daemon and successfully run sacct -j2
$ sudo /opt/slurm/sbin/slurmdbd
$ sudo systemctl restart slurmctld
sacct -j2
       JobID    JobName  Partition    Account  AllocCPUS      State ExitCode
------------ ---------- ---------- ---------- ---------- ---------- --------
2                  wrap    compute                     1    RUNNING      0:0

Hi Alexis,
sorry for the late reply. I'm not sure why you had to restart the daemons twice to make it work. What is the slurmdbd config you are setting before starting the daemon in the post-install script? Have you updated the DbdPort in the config?

I restarted it twice here to show the two scenarios, sorry if that was confusing.
The former was with running sacct with just the control service running on the master node.
The latter was running sacct with both.
It shows that to run sacct on the head node, one needs both the control daemon and database daemon running simultaneously.
Yes, in the post installation script I switch the control port range to 6820 to 6830 to not conflict with the daemon.

2.10.0 release is out. Thanks for reporting this.


Fixed reading of .jshintrc

This fixes a problem where the call to fs.readFile would change the context of this, making @docpad undefined, causing .jshintrc not to be read.

Published this fix in v2.2.1


Split this library so we are not forced to install dependencies we do not need.

Is your feature request related to a problem? Please describe.
Improvement.
Our workflow involves typechecking in development builds (watch), but linting, formatting, and other checks are done in either pre-commit hook, or CI (also on the IDE).
We would like to use this package, but it brings a host of dependencies, most of them we don't use (and those can conflict with the versions we will already have).
From package.json:
  },
  "peerDependencies": {
    "eslint": ">=7",
    "meow": "^9.0.0",
    "stylelint": ">=13",
    "typescript": "*",
    "vite": ">=2.0.0",
    "vls": "*",
    "vti": "*",
    "vue-tsc": "*",
    "optionator": "^0.9.1"
  },

Describe the solution you'd like
The ability to only use and install what is actually being used. One way to do this is to split this package into its constitutes, possibly with some core or common package.
Describe alternatives you've considered
This is a design issue.
Additional context
No response
Validations

[X] Read the docs.
[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that asks for the same feature to avoid creating a duplicate.

The dependencies you listed above are peerDependencies but they also are optional which means you only need to install what checker deps you need. Even though we split the packages into such @vite-plugin-checker/core and @vite-plugin-checker/eslint, the core checker dependencies which is ESLint is still inevitable.
https://github.com/fi3ework/vite-plugin-checker/blob/f65aa71a1ac9ab8374085449503174be935658c6/packages/vite-plugin-checker/package.json#L61-L97

Peer Dependencies
As explain here:

Some way of saying, "I only work when plugged in to version 1.2.x of my host package, so if you install me, be sure that it's alongside a compatible host." We call this relationship a peer dependency.

A classic example is a React UI library (like MUI) - it will mark React as a peer dependency as if to say:

"I need this to work, but surely you have it already, so I'm just letting you know which version I work with".

True, yarn and npm don't install them, but they do complain about unmet requirements. So if, as in our case, we only need typescript, we are going to get warnings to our logs (which are noisy anyhow) and figuring out "Oh, this is OK, it is vite-plugin-checker but we can ignore this" makes an exception you need to make time and again.
ESLint
I'm sure there is a good reason for it, but seems a bit odd that in order to do type checking one needs ESLint. Many of us arriving to Vite from Webpack frameworks (RCA, NextJS) are familiar with fork-ts-checker-webpack-plugin.
Is this Approach Scalable?
What if there are (and over the years there will be some) other checks that are needed? Would having a monolith supporting 20 different technologies be a sustainable approach?
Anyhow, this issue is a bit more on the conceptual/architectural side, and being a big and breaking change, happy for it to be closed.

Peer Dependencies
As explained here:

Some way of saying, "I only work when plugged in to version 1.2.x of my host package, so if you install me, be sure that it's alongside a compatible host." We call this relationship a peer dependency.

A classic example is a React UI library (like MUI) - it will mark React as a peer dependency as if to say:

"I need this to work, but surely you have it already, so I'm just letting you know which versions I work with".

True, yarn and npm don't install them, but they do complain about unmet requirements. So if, as in our case, we only need typescript, we are going to get warnings in our install logs (which are noisy anyhow) and figuring out "Oh, this is OK, it is vite-plugin-checker but we can ignore this" makes an exception you need to make time and again.
ESLint
I'm sure there is a good reason for it, but seems a bit odd that in order to do type checking one needs ESLint. Many of us arriving to Vite from Webpack frameworks (RCA, NextJS) are familiar with fork-ts-checker-webpack-plugin.
Is this Approach Scalable?
What if there are (and over the years there will be some) other checks that are needed? Would having a monolith supporting 20 different technologies be a sustainable approach?
What will happen if Typescript introduce breaking changes and so is ESlint, and so will Stylelint? This library will have to decide when to bump a major version, or it'll have to support all permutations (8 in this case). What about testing? Also support in this library all permutations?
Final Thoughts
Anyhow, this issue is a bit more on the conceptual/architectural side, and being a big and breaking change, happy for it to be closed.

100%
in my one-man hobby-project I don't need eslint - all I need is ts-checks for my types.
checker plugin seems neat at first but if I have get these "errors" bc of missing peer dependencies its an straight uninstall

ERROR(TypeScript)  Cannot find module 'eslint' or its corresponding type declarations.
FILE  C:/Projekte/Code/Frontend/mobile/nrop/taya/node_modules/vite-plugin-checker/dist/esm/types.d.ts:3:24

1 | import { ConfigEnv, ErrorPayload } from 'vite';
2 | import { Worker } from 'worker_threads';

3 | import { ESLint } from 'eslint';
|                        ^^^^^^^^
4 | import * as Stylelint from 'stylelint';
5 | import { VlsOptions } from './checkers/vls/initParams.js';
6 | import 'vscode-uri';

ERROR(TypeScript)  Cannot find module 'stylelint' or its corresponding type declarations.
FILE  C:/Projekte/Code/Frontend/mobile/nrop/taya/node_modules/vite-plugin-checker/dist/esm/types.d.ts:4:28
2 | import { Worker } from 'worker_threads';
3 | import { ESLint } from 'eslint';

4 | import * as Stylelint from 'stylelint';
|                            ^^^^^^^^^^^
5 | import { VlsOptions } from './checkers/vls/initParams.js';
6 | import 'vscode-uri';
7 | import 'vscode-languageserver/node';

ERROR(TypeScript)  Cannot find module 'rollup/parseAst' or its corresponding type declarations.
FILE  C:/Projekte/Code/Frontend/mobile/nrop/taya/node_modules/vite/dist/node/index.d.ts:5:41
3 | import * as rollup from 'rollup';
4 | export { rollup as Rollup };

5 | export { parseAst, parseAstAsync } from 'rollup/parseAst';
|                                         ^^^^^^^^^^^^^^^^^
6 | import * as http from 'node:http';
7 | import { OutgoingHttpHeaders, ClientRequestArgs, IncomingMessage, ClientRequest, Agent, Server, ServerResponse } from 'node:http';
8 | import { Http2SecureServer } from 'node:http2';

keeping peer dependency hassle and unnecessary config hell out of one's project is absolute prio number one in modern frontend dev. I hope lib author considers alternative approach and dont just knock it off with "it's just a peer"

Hello, do you agree that when using a bundler such as tsup to generate a common config, then the build will fail because of missing dependencies ? I guess the bundler will try to resolve all the imports, so when vls for example is not installed it will surely fail.
I'm not sure what is the workaround for this.


Fix the generichash state definition

Do not assume anything about its internal structure. This is now an opaque array.

Seems the PPA we were sourcing libsodium from no longer works...

Build fixed in #201, and merged into this.


DistributedClosestPoint should gracefully deal with variations allowed by mesh blueprint

DistributedClosestPoint is limited in what the exact form of incoming data.  Ideally, it should be able to handle any valid blueprint mesh.    Currently, it is limited to double reals and integer integral data and may does exit gracefully if that is not the case.
This was brought up in a project meeting.  It is related to issue https://github.com/LLNL/axom/issues/835.

More detailed group discussion about what we want/need is needed for this.

@BradWhitlock mentioned Conduit topology iteration as a possible model to consider as a starting point.


cli: don't prompt for password twice when using it

PR: https://github.com/cockroachdb/cockroach/pull/15919

On this page, need to remove this note:

After issuing this command, you must enter the password for jpointsman twice.

Need to search for any other mentions of entering password twice when using.


Trailing Space in a comment before the code to be fixed adds unnecessary empty line

Having some code with a potential fix and a comment with a trailing space right before the code, results to an empty line being added in the fix.
If the comment has no trailing space, fixer works as expected.
The PHP version you are using ($ php -v):
=> PHP 7.1.8
PHP CS Fixer version you are using ($ php-cs-fixer -V):
=> 2.10.0
The command you use to run PHP CS Fixer:
=> php-cs-fixer fix a.php
Sample PHP code (a.php):

before running PHP CS Fixer (no changes):

<?php

function foo() : void
{
    // there is a trailing space in this comment!
    $var1 = (
        $var2 ?? 0);
}

with unexpected changes applied when running PHP CS Fixer:

<?php

function foo() : void
{
    // there is a trailing space in this comment!
    $var1 = (
        $var2 ?? 0

    );
}

In the resulted code above, there is an empty line between $var2 ?? 0 and the closing parenthesis

with the changes you expected instead:

<?php

function foo() : void
{
    // there is a trailing space in this comment!
    $var1 = (
        $var2 ?? 0
    );
}

I retried this on 2.2, 2.10 and master and cannot reproduce this issue anymore.
@dbaltas my check might not be all complete, maybe I didn't setup the failing cases correctly so please; if you've the time please retry with any of the latest stable released versions (2.*) of the fixer and let me know if this is still an issue, if so I reopen it :)  thanks for reporting!

The issue persists in 2.12.1
steps to reproduce
rm -rf /tmp/cs-fix
mkdir /tmp/cs-fix
cd /tmp/cs-fix
curl https://gist.githubusercontent.com/dbaltas/18befe7f6582b10168557a0fcdcc234b/raw/18df7964f384b84aabb2ac0649fb8dbd1bd254a9/trailing-space-in-comma-3547.php > a.php
cat a.php
php-cs-fixer --version
php-cs-fixer fix .
cat a.php

logs
5:24:50 ➜  /tmp  rm -rf /tmp/cs-fix
5:24:57 ➜  /tmp  mkdir /tmp/cs-fix
5:25:06 ➜  /tmp  cd /tmp/cs-fix
5:25:11 ➜  cs-fix  curl https://gist.githubusercontent.com/dbaltas/18befe7f6582b10168557a0fcdcc234b/raw/18df7964f384b84aabb2ac0649fb8dbd1bd254a9/trailing-space-in-comma-3547.php > a.php
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   119  100   119    0     0    182      0 --:--:-- --:--:-- --:--:--   182
5:25:20 ➜  cs-fix  cat a.php
<?php

function foo() : void
{
    // there is a trailing space in this comment!
    $var1 = (
        $var2 ?? 0);
}
5:25:30 ➜  cs-fix  php-cs-fixer --version
PHP CS Fixer 2.12.1 Long Journey by Fabien Potencier and Dariusz Ruminski
5:25:44 ➜  cs-fix  php-cs-fixer fix .
Loaded config default.
   1) a.php

Fixed all files in 0.008 seconds, 10.000 MB memory used
5:25:53 ➜  cs-fix  cat a.php
<?php

function foo() : void
{
    // there is a trailing space in this comment!
    $var1 = (
        $var2 ?? 0

    );
}
5:25:57 ➜  cs-fix

Above is the terminal log of creating a file /tmp/cs-fix/a.php with "the before running" version.
After running php-cs-fixer fix, a new empty line has been added after the $var2 ?? 0.
I wouldn't expect the operating system to affect this. (I am running on MacOS 10.13.4, php 7.1.8)

Thanks for getting back!
Seems this is indeed still an issue, my apologies for closing it early :(
After digging around this seems a priority issue between;
method_argument_space and no_trailing_whitespace_in_comment,
When no_trailing_whitespace_in_comment is run before method_argument_space it goes wrong (these both have priority 0)
However this does not happen when not using the rules default configuration;
something like this goes ok:
$ php php-cs-fixer fix tmp/test2.php -vvv --dry-run --diff --rules=method_argument_space,no_trailing_whitespace
but it does go wrong for something like this fails (note: no config file used);
$ php php-cs-fixer fix tmp/test2.php -vvv --dry-run --diff
so the priority issue is somewhere between the two rules and their configuration... bug-hunt continues!


Switch to Basic.Reference.Assemblies

https://www.nuget.org/packages/Basic.Reference.Assemblies.AspNet80
Search for typeof(System.Runtime.AssemblyTargetedPatchBandAttribute).Assembly in the codebase to find the places that should be switched over.

Fixed via https://github.com/dotnet/razor/pull/10416

There are more places this could be done: https://github.com/dotnet/razor/pull/10416#issuecomment-2136969812


Error page improvements

[ ] Close window by hitting "x"
[ ] Make error header darker
[ ] Link issue tracker instead of displaying link

https://pythondiscord.com/error/510 breaks. we should test all the codes.

I'm led to believe that's the internals of flask and that we can't really do much about it, for example, 402 also fails with some socket error.

Regarding the issue with the 510 status codes, upon looking at the error again it seems that werkzeug does not have an error for that specific status code.
  File "/usr/lib/python3.6/site-packages/flask/views.py", line 149, in dispatch_request
    return meth(*args, **kwargs)
  File "/home/joseph/Python/site/pysite/views/main/error.py", line 12, in get
    return abort(code)
  File "/usr/lib/python3.6/site-packages/werkzeug/exceptions.py", line 707, in abort
    return _aborter(status, *args, **kwargs)
  File "/usr/lib/python3.6/site-packages/werkzeug/exceptions.py", line 686, in __call__
    raise LookupError('no exception for %r' % code)
LookupError: no exception for 600
Thu Mar  1 14:56:23 2018 {'REMOTE_ADDR': '::ffff:<IP_ADDRESS>', 'REMOTE_PORT': '48168', 'HTTP_HOST': 'pysite.local:8080', (hidden keys: 25)} failed with LookupError

I'm going to treat this as not an issue with our implementation, this isn't an issue at all really, the few codes that are documented in RFC and exist do not mean anything to us, I know for a fact we won't have to use Not Extended or Payment required at any point on our application.

All enhancements of error pages now complete and in production. Closing issue.


ASK JHUN:

I am getting state set and firebase updating on /contacts.
I am NOT getting anything into store.


startListening don't work on iOS 10

I implemented functions in this order and works ok on Android, but startListening doesn't works on iOS 10.
Functions
Avaliable - (works on android and iOS)
HasPermition - (works on android and iOS)
RequestPermition -  (works on android and iOS)
But startListening -  (works on android, but not works on iOS 10)
I already given permission  on NSMicrophoneUsageDescription and
NSSpeechRecognitionUsageDescription with its string value in info.plist
I call permissions before i call startListening, but doesn´t works on iOS.
I am with:
Cordova CLI: 6.5.0
Ionic Framework Version: 2.0.1
Ionic CLI Version: 2.1.18
Ionic App Lib Version: 2.1.9
Ionic App Scripts Version: 1.0.0
ios-deploy version: 1.9.1
ios-sim version: 5.0.13
OS: macOS Sierra
Node Version: v6.9.2
Xcode version: Xcode 8.2.1 Build version 8C1002
The erros that appears to me is --> error domain=kAFAssistantErrorDomain, same as others issues.
Can you have some idea why this is happen?

sadly Apple provides poor support for Speech API. Just look at the official forum:
https://forums.developer.apple.com/search.jspa?q=kAFAssistantErrorDomain
I really not able to tell you what could be the problem. It works on my device and I can not reproduce this error.

I understand, can you send me your project?
maybe some configuration of your project can be different and work here.

I'm also receiving Error Domain=kAFAssistantErrorDomain Code=1 "(null)" . In some circumstances recognition works, but only after idling for several minutes. More in this picture.

I should mention the above also works flawlessly on Android.

Same error here, with this plugin and with https://github.com/macdonst/SpeechRecognitionPlugin
tested on iOS 10 and both do not work.
On android, no problem.

Here more infos: https://github.com/macdonst/SpeechRecognitionPlugin/issues/88#issuecomment-281378459

Please check last release

How can we update from Ionic?

I am still experiencing issues with using the code that I provided above. Here are the steps I took:

Update ionic-native
Make a button that is bound to the function voiceSearch that is listed in a previous comment.
Tap button and say "What is the time".
a. Wait.
b. No response. Nothing logged in console.
Tap button and say "What is the time".
a. Wait
b. Error logged in the console.
Tap button and say "What is the time".
a. Wait
b. Returns identified words in the console.

Can you please provide a function similar to mine that works on both ios and android? Currently the function voiceSearch that I listed above works fine on android. If it's impossible to write a function that works for both platforms, can you provide a snippet that works for ios only?
Thank you!

@aurothic, How long did you wait?
Seems that needs to wait for about 1 minute for a response back, can you try?

@aurothic you don't need to update ionic-native package.
You have to update the cordova plugin itself.
cordova plugin rm cordova-plugin-speechrecognition
cordova plugin add cordova-plugin-speechrecognition

Thanks @pbakondy now works ...

Great!
Can we close the issue?

This worked for me as well. However, there still is the matter of waiting for a 1 minute timeout before displaying the results. Android detects when the user stops talking, is there any such way to do this on IOS?

For ios use stopListening method.

Yes, but when should I call it? Currently I have a button that starts the listening process. This works very well on android, does it not carry over to ios?

You need use plugin "Platform" to knows if you are on android or ios.
https://ionicframework.com/docs/v2/api/platform/Platform/
On my app I put timer when i call startListening for ios ( 5 seconds ) when time over, the program calls stopListening

Seems like bad practice. Perhaps this plugin could be used with DB Meter.

Error: exec proxy not found for :: SpeechRecognition :: isRecognitionAvailable
console.log(window.plugins.speechRecognition);
this line is working fine
{
getSupportedLanguages: ƒ (successCallback, errorCallback)
hasPermission: ƒ (successCallback, errorCallback)
isRecognitionAvailable: ƒ (successCallback, errorCallback)
requestPermission: ƒ (successCallback, errorCallback)
startListening: ƒ (successCallback, errorCallback, options)
stopListening: ƒ (successCallback, errorCallback)
}
function startRecordingSpeech(){
console.log(window.plugins.speechRecognition);
window.plugins.speechRecognition.isRecognitionAvailable(successCallback,errorCallback);
}
function successCallback(){
console.log(callBackResult);
}
function errorCallback(callBackResult){
console.log(callBackResult);
//Error: exec proxy not found for :: SpeechRecognition :: isRecognitionAvailable
}

In Ionic 4 not working.Getting below error.
TypeError: Object(WEBPACK_IMPORTED_MODULE_1__ionic_native_core["cordova"]) is not a function. (In 'Object(WEBPACK_IMPORTED_MODULE_1__ionic_native_core["cordova"])(this, "isRecognitionAvailable", {}, arguments)', 'Object(WEBPACK_IMPORTED_MODULE_1__ionic_native_core["cordova"])' is an instance of Object)


TypeError

Uncaught (in promise) TypeError: Cannot read properties of undefined (reading '_c')

Duplicate issue
See: #1533

@inlancersystem4 Do you have a reproductive link so we can replicate this?

But also please check what other people have done to fix their issue in #1533, incase that fixes it for you

Check your versions & compatibility, I have had the same issue when using the Vue 2 version of the component in my Vue 3 app.


Split SHAMapTreeNode into leaf and inner nodes.

This reduces the memory requirements of both leaf and inner nodes.
The name SHAMapTreeNode is retained for leaf nodes so as to keep
the public API of SHAMap stable.

This pull-request is purposefully doing as little as possible because it is already too big and it is touching business logic in a non-trivial way.  I've gone to pains to not even make unnecessary white-space corrections to make this easier to review.  I do not think we should lump other cleanups into this P/R no matter how trivial.  Further cleanups should be shuttled to a separate P/R.
I am observing a 25% decrease in memory usage of rippled after 10 minutes of  uptime compared to 0.28.2-b6 (release build).
@JoelKatz @ximinez

This is impressive

25% decrease in memory usage

Nice!
private:
    uint256                         mHash;
    uint256                         mHashes[16];
    std::shared_ptr<SHAMapTreeNode> mChildren[16];
    std::shared_ptr<SHAMapItem>     mItem;
    std::uint32_t                   mSeq;
    TNType                          mType;
    int                             mIsBranch;
    std::uint32_t                   mFullBelowGen;

    static std::mutex               childLock;

Even leaves had all that before this change eh?

This is great. :+1: although I'm not a reviewer.

:+1:

Rebased to b7.

Pulled in David's new test for this work and rebased to b8.

Link failure in MSVC nounity build:
   Creating library build\msvc.debug.nounity\rippled.lib and object build\msvc.d
ebug.nounity\rippled.exp
beast.obj : error LNK2019: unresolved external symbol "public: __cdecl beast::Cr
iticalSection::CriticalSection(void)" (??0CriticalSection@beast@@QEAA@XZ) refere
nced in function "public: __cdecl beast::ArrayAllocationBase<void *,class beast:
:CriticalSection>::ArrayAllocationBase<void *,class beast::CriticalSection>(void
)" (??0?$ArrayAllocationBase@PEAXVCriticalSection@beast@@@beast@@QEAA@XZ)
beast.obj : error LNK2019: unresolved external symbol "public: __cdecl beast::Cr
iticalSection::~CriticalSection(void)" (??1CriticalSection@beast@@QEAA@XZ) refer
enced in function "int `public: __cdecl beast::ArrayAllocationBase<void *,class
beast::CriticalSection>::ArrayAllocationBase<void *,class beast::CriticalSection
>(void)'::`1'::dtor$0" (?dtor$0@?0???0?$ArrayAllocationBase@PEAXVCriticalSection
@beast@@@beast@@QEAA@XZ@4HA)
beast.obj : error LNK2019: unresolved external symbol "public: void __cdecl beas
t::CriticalSection::enter(void)const " (?enter@CriticalSection@beast@@QEBAXXZ) r
eferenced in function "public: __cdecl beast::GenericScopedLock<class beast::Cri
ticalSection>::GenericScopedLock<class beast::CriticalSection>(class beast::Crit
icalSection const &)" (??0?$GenericScopedLock@VCriticalSection@beast@@@beast@@QE
AA@AEBVCriticalSection@1@@Z)
beast.obj : error LNK2019: unresolved external symbol "public: void __cdecl beas
t::CriticalSection::exit(void)const " (?exit@CriticalSection@beast@@QEBAXXZ) ref
erenced in function "public: __cdecl beast::GenericScopedLock<class beast::Criti
calSection>::~GenericScopedLock<class beast::CriticalSection>(void)" (??1?$Gener
icScopedLock@VCriticalSection@beast@@@beast@@QEAA@XZ)
build\msvc.debug.nounity\rippled.exe : fatal error LNK1120: 4 unresolved externa
ls
scons: *** [build\msvc.debug.nounity\rippled.exe] Error 1120
scons: building terminated because of errors.

Fix is in here, first commit
https://github.com/ripple/rippled/pull/1122

Cherry-picked Vinnie's fix for the MSVC link-error.

To be flawless the MSVC link fix should be the earliest commit but its not critical. It'll get sorted in the merge to the upstream.

It looks like the earliest in my local log.  I'm not sure what's up with the order above.

This is what I see: https://github.com/HowardHinnant/rippled/commits/split-node

GitHub weirdness again :-(

Builds successfully on Windows now. The new test is pretty nice. Looks good overall. :+1:

:+1:  So wow!


Goto definition

Using the class-to-file package it should be easy to use the tolerant-php-parser to get the FQN of any class reference and convert it to a file path.

Done


Sign-in error

[Error] Failed to load resource: the server responded with a status of 400 (HTTP/2.0 400) (values:batchGet, line 0)
[Error] [object Object]
	(anonymous function) (cb=gapi.loaded_0:75)

Fixed and duplicate of #6


copy all workdir to Assets dir for FIBSEM

(similar to BRT)

done: https://github.com/niaid/image_portal_workflows/blob/520f83f6c5eb09d380160d9d58feaa9ee4c8d0ee/em_workflows/sem_tomo/flow.py#L309


Error: Base module not found. Maybe try rebuilding the app., js engine: hermes

Hi hello,
Currently i trying to embedded op-sqlite library into the project that i'm working on in react native, typescript using expo format.
Which after npm install upon running via iOS, i got an error
ERROR  Error: Base module not found. Maybe try rebuilding the app., js engine: hermes
ERROR  Invariant Violation: "main" has not been registered. This can happen if:
React version: 18.2.0
Expo: 50.0.15

Did you do a pod install?

Hi @ospfranco , currently i'm using nx, which it resolve after i copy inside the dependency inside /apps folder

I don't know what NX is. It's better you ask on stack overflow

I don't know what NX is. It's better you ask on stack overflow

Thank you. i manage to fix it.
I'm writing some unit test pertaining to database for react native.
code as follows:
`import { open, OPSQLiteConnection } from '@op-engineering/op-sqlite';
jest.mock('@op-engineering/op-sqlite', () => {
const mockedConnection: OPSQLiteConnection = {
close: jest.fn(),
delete: jest.fn(),
attach: jest.fn(),
detach: jest.fn(),
transaction: jest.fn(),
execute: jest.fn(),
executeAsync: jest.fn(),
executeBatch: jest.fn(),
executeBatchAsync: jest.fn(),
loadFile: jest.fn(),
updateHook: jest.fn(),
commitHook: jest.fn(),
rollbackHook: jest.fn(),
prepareStatement: jest.fn(),
loadExtension: jest.fn(),
executeRawAsync: jest.fn(),
getDbPath: jest.fn(),
};
return {
open: jest.fn(() => mockedConnection),
isSQLCipher: jest.fn(),
moveAssetsDatabase: jest.fn(),
};
});
describe('Test Service', () => {
describe('Trigger', () => {
it('should handle the database function correctly', async () => {
const db = open({
name: 'database',
});
const result= db.execute('select sqlite_version();');
expect(result).toBeDefined();
});
});
});
`
it keep give me result undefined. any idea, what's wrong with that? i try setting up location ':memory:' or even put the absolute path '/Users/user/database' also same result.

No idea, any mocks should work.

No idea, any mocks should work.

Weird thing is i create a new project react-native using the same code it have the same result. >.<


modify management of sct_estimate_tracts

change name for: sct_extract_metric.py
in sct_warp_template:

put CSF, WM, etc. into a folder called template to have the same folder tree of the SCT
in this folder, add a txt file that lists all relevant files.

The list.txt file should be CSV format, organized like that:

ID, label_name, file_name
example:
1,left fasciculus gracilis,vol0000.nii.gz

In the sct_extract_metric, the user will be given the flag -f, which will enable the user to choose between folder: 'atlas', 'template', etc.  by default, the folder 'template' will be selected.
in sct_extract_metric, the file list.txt will be read, and all the tracts and file name will be derived from it.

I believe all of this has been done and merge in master in commit 50ffa7c

yup!


Support for https://github.com/insin/inputmask-core?

Would be useful for entering Dates etc. See this in react: http://insin.github.io/react-maskedinput/

Hi,
Is it possible to create mask example for email address?
Eg It should accepts "@" keyword<EMAIL_ADDRESS>Thanks
Nik

Don't know if that's the use case. Because email is not fixed width.
Input masks aren't for validation. That's pretty easy with regex. ember-changeset and its validations if you are looking for this.
If you have to enter lots of dates //__ or fixed width numbers e.g. _, usability is enhanced. Because the user just needs to enter the numbers at the number block. And there remains no room for error.


One question)

        Adapter adapter = new Adapter(getSupportFragmentManager());
        adapter.addFragment(new CheeseListFragment(), "Tab 1");
        adapter.addFragment(new CheeseListFragment(), "Tab 2");
        adapter.addFragment(new CheeseListFragment(), "Tab 1");
        adapter.addFragment(new CheeseListFragment(), "Tab 2");
        viewPager.setAdapter(adapter);

When i debug project its twice enter in onCreateView. Why twice? But i initialize 4 instance of CheeseListFragment?
CheeseListFragment.java
@Nullable
    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        RecyclerView rv = (RecyclerView) inflater.inflate(
                R.layout.fragment_cheese_list, container, false);
       // setupRecyclerView(rv);
        return rv;
    }

@Radzhab
You should read more about ViewPager and its working. ViewPager itself handles above case itself. For instance, if you are currently at item n, the FragmentPagerAdapter calls the onCreateView() of fragments at n-1, n and n+1. You can manually override this with setOffscreenPageLimit()
Read more at http://developer.android.com/reference/android/support/v4/view/ViewPager.html#setOffscreenPageLimit(int)


PLT-5660 Ability to edit a post

Summary
Add the ability to edit the message of a post that belongs to the current user
Ticket Link
https://mattermost.atlassian.net/browse/PLT-5560
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Has UI changes
[x] Includes text changes and localization file updates

Device Information
This PR was tested on:

IOS 10.2.1 iPhone 6s
Android 6.0.1 Samsung J5

Screenshots
IOS

Android

@jarredwitt ready for re-review


[SPARK-23299][SQL][PYSPARK] Fix repr behaviour for Rows.

What changes were proposed in this pull request?
Fix __repr__ behaviour for Rows.
Rows __repr__ assumes data is a string when column name is missing.
Examples,
>>> from pyspark.sql.types import Row
>>> Row ("Alice", "11")
<Row(Alice, 11)>

>>> Row (name="Alice", age=11)
Row(age=11, name='Alice')

>>> Row ("Alice", 11)
<snip stack trace>
TypeError: sequence item 1: expected string, int found

This is because Row () when called without column names assumes
everything is a string.
How was this patch tested?
Manually tested and unittest was added in python/pyspark/sql/tests.py.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

BTW, does non-string field names work in this namedtuple way?

@HyukjinKwon Do you mean something like Row (a=1, b=2, c=3) or Row (1="Alice", 2=11)?  Former works fine, latter fails with SyntaxError: keyword can't be an expression.

I think it still makes sense to produce a repr anyway because we successfully can create the instance for now but .. let me take a closer look within few days for sure.

@HyukjinKwon Here is what I tried:
# Code: return "<Row(%s)>" % ", ".join(fields.encode("utf8") for fields in self)
>>> Row (u"아", "11")
<Row(아, 11)>
# Fails for integer fields.

# Code: return "<Row(%s)>" % ", ".join(str(fields) for fields in self)
>>> Row (u"아", "11")
UnicodeEncodeError: 'ascii' codec can't encode character u'\uc544' in position 0: ordinal not in range(128)

# Code: return "<Row(%s)>" % ", ".join(repr(fields) for fields in self)
>>> Row (u"아", 11)
<Row(u'\uc544', 11)>

# Code: return "<Row(%s)>" % ", ".join(unicode(fields).encode("utf8") for fields in self)
>>> Row (u"아", 11)
<Row(아, 11)>

repr is definitely a better option than str.  But why not unicode?

unicode(fields).encode("utf8"): in this case, we will try to decode it by system default encoding first and then encode it by udf-8 if the input is str (bytes). So, for example, I think unicode("아") case won't work.

Check if it's unicode and convert, etc. might also work ..

@HyukjinKwon return "<Row(%s)>" % ", ".join("%s" % (fields) for fields in self) takes care of everything.

>>> Row ("aa", 11)
<Row(aa, 11)>

>>> Row (u"아", 11)
<Row(아, 11)>

>>> Row ("아", 11)
<Row(아, 11)>

Jenkins, ok to test

Test build #87143 has started for PR 20503 at commit 890aa65.

Test build #87143 has finished for PR 20503 at commit 890aa65.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87143/
Test PASSed.

Merged build finished. Test PASSed.

@HyukjinKwon Should I add more tests covering Unicode?

Can one of the admins verify this patch?

we still need to fix this, right?

ok to test

Test build #91594 has started for PR 20503 at commit 890aa65.

Test build #91594 has finished for PR 20503 at commit 890aa65.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91594/
Test PASSed.

Sure let's add a test with a unicode string to it if there's concern about that and make sure the existing repr with named fields is covered the same test case since I don't see an existing explicit test for that (although it's probably covered implicitly elsewhere).

Gentle ping

Gentle ping again to @ashashwat . Also @HyukjinKwon what are your opinions on the test coverage?

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?

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?

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?

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?

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?

Can one of the admins verify this patch?

Jenkins ok to test.
Gentle ping again to @ashashwat - are you still interested in this PR?

@holdenk I am on it.

ok to test

Test build #98110 has started for PR 20503 at commit 890aa65.

Test build #98110 has finished for PR 20503 at commit 890aa65.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98110/
Test PASSed.

Awesome, thanks. Let me know if I can help :)

ping @ashashwat to update

Can one of the admins verify this patch?

@ashashwat are you able to update this?

Bye the way, this is not just an annoyance for interactive use, I bumped into this issue while trying to understand failing tests (run via pytest).  Having a broken __repr__ is rather annoying in such a situation.
output for a failing test with broken __repr__:
E       assert [<Row(foo)>] == [<[TypeError("sequence ...ect at 0x7fa1d24a4240>]
E         (pytest_assertion plugin: representation of details failed.  Probably an object has a faulty __repr__.)
E         /home/user/projects/foobar/venv/lib/python3.6/site-packages/pyspark/sql/types.py:1552: TypeError: sequence item 0: expected str instance, NoneType found

output for a failing test with fixed __repr__:
E       assert [<Row(foo)>] == [<Row(None)>]
E         At index 0 diff: <Row(foo)> != <Row(None)>
E         Full diff:
E         - [<Row(foo)>]
E         ?       ^ ^
E         + [<Row(None)>]
E         ?

@tbcs would you want to take over updating this PR? You could make a fork?

@holdenk OK, please see #24448.


RFP - Lando

https://github.com/lando/lando
Lando is "a free, open source, cross-platform, local development environment and DevOps tool built on Docker container technology".
Would require https://chocolatey.org/packages/docker-for-windows.

Note (I have highlighted the main points in bold):
Source: https://docs.lando.dev/basics/installation.html#windows
YOU MUST HAVE HYPER-V ENABLED
Make sure that Hyper-V is enabled or lando will not work!
Do not fear the package size!
Note that our Windows installer includes Docker Desktop and this accounts for it's large file size in comparison to our Linux packages.
When going through the installer you can choose to not install Docker Desktop, although we recommend you use the version of Docker Desktop that we ship for compatibility and support reasons!
Make sure you are using at least Windows 10 Professional with the latest updates installed.
Download the latest Windows .exe installer from GitHub
Double-click on lando.exe
Go through the setup workflow
Approve various UAC prompts during install

approve various UAC prompts during install

This doesn't sound like it's going to be silent?

To be tested. Maybe they will not appear if installed with admin rights.


Project Console - New CCLA Template Not Added to Document List

Summary
Project Console - New CCLA Template Not Added to Document List
Background
When a new template is generated after an existing set of CLA templates are present, only the ICLA document details are added to the document list. The CCLA appears to only update the date/time of the new PDF.  Expected behavior is that a new document is added to the list of documents.
Expected behavior
A new document is added to the list of ICLA/CCLA documents.
Screenshots
Template was re-generated several times - CCLA only shows one document:

ICLA shows four:

Environment
Please complete the following information:

Environment: DEV, STAGING and PROD

Acceptance Criteria
The "done" criteria when this feature or problem is resolved. Such as:

Unit Tests added and running in CI
Functional Tests updated to cover feature, if applicable
Demonstrate the set of capabilities to the product team while the code is running in the STAGING environment.

References
(Optional) Provide any code or specification references that would be helpful for the developer implementing this feature.

Verified that the issue is fixed. However there is issue at step3 which is tracked in #1871
Tested for New CLA Group/Existing CLA Group. Also tested for Only CCLA, Only ICLA, Both CCLA and ICLA. However observed issue at step3

Generated templates and observed that files are generated in below format which is as expected

Generate templates again. Observed that 2 elements exists for project_individual_documents and project_corporate_documents fields

3. Observed that recent pdf is selected. However when clicked in PDF icon to view the modal popup, first time generated pdf is selected. If the modal popup is opened again then recently generated pdf is selected. Each time when modal popup is opened, alternatively recently generated and first time generated pdf's are selected. Observed same issue with ICLA pdf as well. If we refresh the page or revisit the page, recently generated file is selected

Observed that while signing CCLA from Corporate Console, recently generated template is used
Observed that while Individual Contributor is signing ICLA from Contributor Console, recently generated template is used
Observed that while Corporate Contributor is signing ICLA from Contributor Console, recently generated template is used
Observed that in S3, templates are stored in format ccla-YYYY-MM-DDTHH-MM-SSZ.pdf or icla-YYYY-MM-DDTHH-MM-SSZ.pdf