Make dockstring available on conda-forge

This helps conda users install it more easily.

I would say that PyPI would work, too. Here, from the Teams chat.
I did some research on how to publish dockstring on PyPI (for simple installation through pip). Miguel Garcia Ortegon:

Read this: https://realpython.com/pypi-publish-python-package
Create an account on the PyPI test platform: https://test.pypi.org/
Create an account on the real PyPI: https://pypi.org/

Update on this: I looked into this and a big issue is dependencies. If I add our pinned rdkit version and pinned openbabel version as dependencies (install_requires=[...]), then pip installation fails since apparently our specific versions of these packages are not available on pypi. It looks like one would need to manually install the dependencies first (e.g. using conda), then install dockstring. If this is the case I don't really see the point of putting it on pypi; they might as well just install it from github.
We could still put it on conda-forge, but we would need to decide how strict we want to make the python version dependency. Should we force people to exactly use 3.7.10? This would minimize chances of library behaviour changes, but will not be sustainable over time (e.g. python 3.7 will reach end of life next year).
Overall, what are your thoughts on hosting dockstring and pinning it to dependencies @gncs @mgarort ?

Done now. See: https://anaconda.org/conda-forge/dockstring


fixed wrong @linkplain command in Javadoc

Hi Abdull,
Thanks for the fix.  Could you quickly sign our contributor license agreement?  Instructions can be found in the contributor guidelines.

Hi Chris,
CLA signed. My munged confirmation number is 22012080**855.
Cheers

Thanks.  A couple quick additonal things:

Please update your git config (at least for this project) to reflect your real name. (e.g. "First Last<EMAIL_ADDRESS>instead of "Abdull me@mail.com").  This can be done with git config user.name "First Last".  I'll add a note about this in the contributor guidelines.
Please touch up the commit comment to follow the format laid out in the contributor guidelines.

Once complete, please git push -f against your patch-1 branch.

Sorry, can't do this.


Example should be renamed to include simpletest

Example simpleio_map_range_demo.py should be renamed to  	simpleio_map_range_simpletest.py to fit with CircuitPython library standards.
The lines including this file name in the docs/examples.rst file must also be updated to reflect the name change, e.g.:
.. literalinclude:: ../examples/current_file_name.py
    :caption: examples/current_file_name.py
    :linenos:

would be updated to:
.. literalinclude:: ../examples/current_file_name_changed.py
    :caption: examples/current_file_name_changed.py
    :linenos:

great work @ilmanzo !!!


[Event Request] codeunit 12 - "Gen. Jnl.-Post Line" - OnBeforeVendLedgEntryInsert

On the IntegrationEvent for OnBeforeVendLedgEntryInsert, can we add var to the GenJournalLine variable?

I see that the OnBeforeCustLedgEntryInsert already has var added to the GenJournalLine variable.

This will allow us to create the same functionalitiesfor both the purchase and the sales side.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 11076.


Line 362: "the message is clear" ? I do not think so. Fig 15b is very confusing. The "ratio" is undefined. The performance of "1st" is only a few percent worse. Secondly, the result is inconsistent with the earlier Fig. 3. Only modest improvement is expected with waveform analysis and only under some circumstances according to Fig. 3.

https://gitlab.airelinux.org/tjjk/publication/-/issues/9#note_5106


Download Model Problem

Hello, gsutil is installed, but it can't be downloaded for some reasons. Could you provide a download like "https://drive.google.com/uc?Id=19xQK2onIy-3S5W5K-XIh85pAg_RNvBVf&export=download"? Thank you very much.

Sure, I'll look into doing this.
In the meantime, does this help: https://twitter.com/gwern/status/1171869124613591040?s=20

If you navigate to the checkpoint of your choice
@ https://console.cloud.google.com/storage/browser/sf-ctrl/ ,
you can download the individual files in the checkpoint using their public links (The link can be found under the public access column).
For instance, for the seqlen256_v1.ckpt, the links are:
https://storage.googleapis.com/sf-ctrl/seqlen256_v1.ckpt/checkpoint
https://storage.googleapis.com/sf-ctrl/seqlen256_v1.ckpt/model.ckpt-413000.data-00000-of-00001
https://storage.googleapis.com/sf-ctrl/seqlen256_v1.ckpt/model.ckpt-413000.index
https://storage.googleapis.com/sf-ctrl/seqlen256_v1.ckpt/model.ckpt-413000.meta

You can mkdir -p seqlen256_v1.ckpt and then wget the above files to the same effect as the gsutil route.
Closing for now, reopen as necessary.


Include metadata in response similar to feathers-sequelize.

Feather-sequelize includes metadata in the response object, which is helpful for pagination, and I think it'd a good addition to this adapter. With this pull request, the metadata is automatically included, but an config flag could be used to toggle the feature, if that's desired.
Note this is a breaking change. The return type is an object instead of an array and the former response is returned in the object’s data property.

I need to revise the tests before I re-open the PR.


[Enhancement] avoid duplicated local shuffle

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Problem Summary(Required) ：

As for multiple operators who need shuffle property in the same fragment, local shuffle only need insert once after the source operator of this fragment.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[x] 2.4
[x] 2.3
[ ] 2.2

run starrocks_admit_test

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 29 / 30 (96.67%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
19
20
95.00%
[1870]

:large_blue_circle:
com/starrocks/planner/OlapScanNode.java
6
6
100.00%
[]

:large_blue_circle:
com/starrocks/planner/PlanFragment.java
4
4
100.00%
[]

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 29 / 30 (96.67%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
19
20
95.00%
[1870]

:large_blue_circle:
com/starrocks/planner/OlapScanNode.java
6
6
100.00%
[]

:large_blue_circle:
com/starrocks/planner/PlanFragment.java
4
4
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 28 / 29 (96.55%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
19
20
95.00%
[1870]

:large_blue_circle:
com/starrocks/planner/OlapScanNode.java
5
5
100.00%
[]

:large_blue_circle:
com/starrocks/planner/PlanFragment.java
4
4
100.00%
[]

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 28 / 29 (96.55%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
19
20
95.00%
[1870]

:large_blue_circle:
com/starrocks/planner/OlapScanNode.java
5
5
100.00%
[]

:large_blue_circle:
com/starrocks/planner/PlanFragment.java
4
4
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 44 / 45 (97.78%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
30
31
96.77%
[1885]

:large_blue_circle:
com/starrocks/sql/optimizer/RequiredPropertyDeriver.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/planner/OlapScanNode.java
9
9
100.00%
[]

:large_blue_circle:
com/starrocks/planner/PlanFragment.java
4
4
100.00%
[]

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 46 / 47 (97.87%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
32
33
96.97%
[1889]

:large_blue_circle:
com/starrocks/sql/optimizer/RequiredPropertyDeriver.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/planner/OlapScanNode.java
9
9
100.00%
[]

:large_blue_circle:
com/starrocks/planner/PlanFragment.java
4
4
100.00%
[]

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 46 / 47 (97.87%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
32
33
96.97%
[1888]

:large_blue_circle:
com/starrocks/sql/optimizer/RequiredPropertyDeriver.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/planner/OlapScanNode.java
9
9
100.00%
[]

:large_blue_circle:
com/starrocks/planner/PlanFragment.java
4
4
100.00%
[]

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 46 / 47 (97.87%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
32
33
96.97%
[1888]

:large_blue_circle:
com/starrocks/sql/optimizer/RequiredPropertyDeriver.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/planner/OlapScanNode.java
9
9
100.00%
[]

:large_blue_circle:
com/starrocks/planner/PlanFragment.java
4
4
100.00%
[]

run starrocks_admit_test

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 53 / 54 (98.15%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
37
38
97.37%
[1358]

:large_blue_circle:
com/starrocks/sql/optimizer/RequiredPropertyDeriver.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/planner/OlapScanNode.java
9
9
100.00%
[]

:large_blue_circle:
com/starrocks/planner/PlanFragment.java
6
6
100.00%
[]

run starrocks_admit_test


Include attestation for ModelKits

Describe the problem you're trying to solve
ModelKits and the assets they contain can come from any location and be built by anyone. There are no inherent guarantees in any of the existing model / dataset packaging mechanism of provenance or safety. Users want a way to know where the package they are using has come from so they can make their own decision about whether to trust it.
Describe the solution you'd like
ModelKits should be able to include attestations for the package and its contents. We could use something like the SLSA's verification summary and include it with the ModelKit as an option. This would make ModelKits the first packaging for AI/ML that provides provenance attestations.

This will need to be split between signing and attestation.

I'm willing to work on this.


Use long for Connection timeout parameters for NetworkClient

NetworkClient constructor expects Long parameters for conn.timeout:
https://github.com/apache/kafka/blob/2.8/clients/src/main/java/org/apache/kafka/clients/NetworkClient.java#L142-L143

Thank you! addressed in #387 🙇


Jetpack cannot fill with liq.hydrogen

my Jetpack cannot fill with hydrogen and the red bar is not appeared.I was try get him to electrolyser, charging pad, suited or take him out from suit, but nothing happened.The red bar hydrogen is not there, only change color in suit when is on or off.Oxygen bar work correctly and charge pad fill suit with oxygen.
:(     galacticraft suit cannot work on my satelite station :(

SOLVED: I was enabled oxygen using in galacticraft and send Advanced rocketry space suit to trash bin:)

I had to remove all tanks from the suit, put 2 empty ones and stood on H2 charger to get 2 tanks with H2. After that I added 3 empty tanks to the chest piece (before tanks filled with H2) and stood on O2 charger. I have 2 types of gases in my suit now and two bars in HUD.

@matzanivs what version of the mod and minecraft?

Hi, I have 1.7.10 modsGalacticraft and Advanced rocketry, when I switch off
to rewrite Galacticraft suit with AR suit oxygen in config, I still need to
wear AR-suit on my satelite station.
mods are new AR -1.7.10. -1.2.2. and Galacticraft <IP_ADDRESS>.12.500 If you know
how to resolve this "oxygen system mixagge" in both mods, tell me howto
please.(sorry for my bad English, I am from Czech rep.)'Would be great
thankfull<EMAIL_ADDRESS>---------- Původní e-mail ----------
Od: zmaster587<EMAIL_ADDRESS>Komu: zmaster587/AdvancedRocketry<EMAIL_ADDRESS>Datum: 7. 8. 2017 6:27:24
Předmět: Re: [zmaster587/AdvancedRocketry] Jetpack cannot fill with liq.
hydrogen (#783)
"
@matzanivs(https://github.com/matzanivs) what version of the mod and
minecraft?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
(https://github.com/zmaster587/AdvancedRocketry/issues/783#issuecomment-320565961)
, or mute the thread
(https://github.com/notifications/unsubscribe-auth/Ac3Aq-yTbTDttywKxKenb7L6JBjLg0V7ks5sVpIngaJpZM4ObObt)
.
"

solved: put the EMPTY gas tanks in suit inventory
---------- Původní e-mail ----------
Od: martianrock<EMAIL_ADDRESS>Komu: zmaster587/AdvancedRocketry<EMAIL_ADDRESS>Datum: 6. 8. 2017 22:27:02
Předmět: Re: [zmaster587/AdvancedRocketry] Jetpack cannot fill with liq.
hydrogen (#783)
"
I had to remove all tanks from the suit, put 2 empty ones and stood on H2
charger to get 2 tanks with H2. After that I added 3 empty tanks to the
chest piece (before tanks filled with H2) and stood on O2 charger. I have 2
types of gases in my suit now and two bars in HUD.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
(https://github.com/zmaster587/AdvancedRocketry/issues/783#issuecomment-320530436)
, or mute the thread
(https://github.com/notifications/unsubscribe-auth/Ac3Aqw3PIrDuFnyRpXYt5C-NDat59T8Jks5sViGSgaJpZM4ObObt)
.
"

@zmaster587, no, once I fill some with O2 and some with H2, I can refill then fine as long as they are not completely empty. As long as there's some substance left in the tank it will not fill with the different one, but if I let my jetpack consume all H2 and then I stand on O2 pad, empty tanks will fill with O2 and there will be no empty tank left for H2.

@martianrock are you also using AR -1.7.10. -1.2.2?


Add gcloud cli autocompletion module

This may seem odd, but I think we also need to include the license from OMZ if we copy plugins wholesale... this is fine (because it's also MIT) but it's something we need to keep in mind.,

That make senses! I've added their License over here :)

:wave: Hey @belak is it ok now?
Thanks!

LGTM! Thanks!


feat: add options.outputStripAnsi

What I do:
Add new option outputStripAnsi, it can clear output's color and style.
Reason:
Plugin use chalk, when outputTarget is function, it will get unreadable text, someone already asked this question. with outputStripAnsi, outputTarget function can get plain text output.

@stephencookdev Hi, could you take a look at these pull requests? Are you still maintaining the project?

@stephencookdev Hi, could you take a look at these pull requests? Are you still maintaining the project?

Struggling to stay on top of everything definitely 😴 But this change looks good. Let me take a look this weekend and see about getting this released. Thanks for the PR @55utah! 🙂

@stephencookdev thanks for responding! 😊  I was looking at this repo for some time and configured it in the project I work in. I was wondering if you are still present here because I will probably try to contribute and I just didn't want to spend plenty of time preparing a PR and then not be noticed 😅

@stephencookdev @shwarcu  Please take a look.  ❤

Looks legit to me 😅


Support using error message from source

I have a very simple enum that just wraps all other error types:
#[derive(Debug)]
pub enum SomeParseFromEnvError {
    InvalidLogFormat(ParseFromEnvError<InvalidLogFormat>),
    InvalidBool(ParseFromEnvError<ParseBoolError>),
    InvalidInfallible(ParseFromEnvError<Infallible>),
}

The Display implementation I have doesn't do anything fancy - it just relies the fmt directly to the wrapped value:
impl std::fmt::Display for SomeParseFromEnvError {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        match self {
            SomeParseFromEnvError::InvalidLogFormat(err) => err.fmt(f),
            SomeParseFromEnvError::InvalidBool(err) => err.fmt(f),
            SomeParseFromEnvError::InvalidInfallible(err) => err.fmt(f),
        }
    }
}

I need a simple way to derive the same behavior.
Annotation every field with #[error("{0})] is an option that I currently have to use, but it doesn't expand to the same code I write manually.
Also, #[error("{0})] above every field doesn't look good :smile:

I would accept a PR to treat:
#[derive(Debug)]
#[error(...)]
pub enum SomeParseFromEnvError {
    InvalidLogFormat(ParseFromEnvError<InvalidLogFormat>),
    InvalidBool(ParseFromEnvError<ParseBoolError>),
    InvalidInfallible(ParseFromEnvError<Infallible>),
}

as equivalent to:
#[derive(Debug)]
pub enum SomeParseFromEnvError {
    #[error(...)]
    InvalidLogFormat(ParseFromEnvError<InvalidLogFormat>),
    #[error(...)]
    InvalidBool(ParseFromEnvError<ParseBoolError>),
    #[error(...)]
    InvalidInfallible(ParseFromEnvError<Infallible>),
}

Would that work for your use case? You would put #[error("{0}")] once at the top of the enum.

it doesn't expand to the same code I write manually.

Thanks, I filed #13 to track this separately.

I guess #[error("{0}")] is the problem, how about we use #[error_proxy_to_source] or sth?

You can alternatively write #[error("{}", .0)] if that's less bothersome.
I think I wouldn't consider this annoying enough to introduce a new #[error_proxy_to_source] attribute that does the same thing.

Closing in favor of #18.


Deduplication and modules with stateful initialization

As I understand it, deduplication works on a per-file rather than per-package basis so a Browserify build with two dependencies, A and B, one of which uses package C v.1.0 and the other using package C v.1.0.1 with some minor bugfix can end up deduplicating most of the files in C but not all of them.
This reduces the size of the bundle but can cause some very confusing issues if package C has stateful initialization code - see https://github.com/facebook/react/issues/2402#issuecomment-89234482 for an example in the context of a large library.
I'd suggest that it would be safer to deduplicate only at the package level, or perhaps not even deduplicate at all by default and require the user to specifically opt-in to deduplication of packages where they know there are unavoidable version conflicts. Thoughts?

To me this sounds like an issue you should file against the module itself: "Avoid stateful initialization code".

That's a good principle in general, but there are a lot of libraries out there that have some kind of stateful initialization in them. Plus if you move the initialization outside of the module and require the consumer to invoke it, this would be a breaking change for all clients that depend on the module.

You're probably right.

this would be a breaking change
I think in most cases you can get around this, if you really must.

@robertknight I've struggled with this too, though in regards to Angular rather than React. I wrote a plugin to more aggressively dedupe dependencies until there's better support for this in npm/Browserify. It may or may not work for your use case, but perhaps worth a shot: https://github.com/Updater/browserify-resolutions


Use same color mapping for rendered heatmap and heatmap image

The red heatmap is rendered on the client (because of its tiny size) while the bigger one is rendered on the server and shipped to the client as an image. That's fine but they should obviously have the same color mapping.

@sgratzl, can you please take a look at this? will probably not take you very long...
it should be constant. greyscale is fine as well for now.

https://github.com/Caleydo/mothertable/blob/master/src/column/MatrixColumn.ts#L55
explicitly sets the color scale to the redish one. However, the redish one is not supported by the server right now. See also https://github.com/phovea/phovea_server/issues/16

ok, then let's switch the color map back to greyscale (which will also be used in the filter view then).

@bikramkawan, can you please take care of this?

Fixed by switching to grey scale (svg) heatmaps.


Exception while trying to store an array of contracts in Solidity

This was reported by a user who used some interesting code. It compiles in browser solidity and runs properly but doesn't on our stack. Here's the report of what I did to reproduce:
Solidity code:
contract House {
  Person[] persons;

  function addPerson(address personAddress) {
    persons.push(Person(personAddress));
  }

  function getPersonByIndex(uint index) constant returns(address personAddres, string name, uint age) {
    Person p = persons[index];

    personAddres = address(p);
    name = bytes32ToString(p.getNameBytes());
    age = p.getAge();

    return (personAddres, name, age);
  }
  // stolen from: http://ethereum.stackexchange.com/questions/2519/how-to-convert-a-bytes32-to-string
  function bytes32ToString(bytes32 x) constant returns (string) {
    bytes memory bytesString = new bytes(32);
    uint charCount = 0;
    for (uint j = 0; j < 32; j++) {
        byte char = byte(bytes32(uint(x) * 2 ** (8 * j)));
        if (char != 0) {
            bytesString[charCount] = char;
            charCount++;
        }
    }

    bytes memory bytesStringTrimmed = new bytes(charCount);
    for (j = 0; j < charCount; j++) {
        bytesStringTrimmed[j] = bytesString[j];
    }
    return string(bytesStringTrimmed);
  }
}

contract Person {
  string _name;
  uint _age;

  function Person(string name, uint age) {
    _name = name;
    _age = age;
  }

  function getName() constant returns (string) {
    return _name;
  }

  function getAge() constant returns (uint) {
    return _age;
  }

  function getNameBytes() constant returns (bytes32) {
    return stringToBytes32(_name);
  }
  // stolen from: http://ethereum.stackexchange.com/questions/9142/how-to-convert-a-string-to-bytes32
  function stringToBytes32(string memory source) returns (bytes32 result) {
    assembly {
        result := mload(add(source, 32))
    }
  }
}

EPM.Yaml:
jobs:

- name: deployPerson
  job:
    deploy:
      contract: houseAndPerson.sol
      instance: Person
      data:
        - "hello"
        - 12
      wait: true

- name: deployHouse
  job:
    deploy:
      contract: houseAndPerson.sol
      instance: House
      wait: true

- name: insertPerson
  job:
    call:
      destination: $deployHouse
      function: addPerson
      data:
        - $deployPerson
      wait: true

- name: getPerson
  job:
    call:
      destination: $deployHouse
      function: getPersonByIndex
      data:
        - 0
      wait: true

- name: assertPerson
  job:
    assert:
      key: $getPerson.name
      relation: eq
      val: "hello"

and finally the output:
Deploying Contract                       name=House
                                         addr=B7427F27B4E7A444C9765280F2954960BDB5CFBC
Executing Job                                 insertPerson
Executing Job                                 getPerson
Encountered Exception from chain w: %!s(<nil>)

Executing Job                                 assertPerson
Assertion Failed                              $getPerson.name == hello
assertion failed

Could not perform pkg action service: Could not perform pkg action: Container interactive-3ce29cea-ce94-403a-9164-4e5695251a2a exited with status 1

The error occurs at name = bytes32ToString(p.getNameBytes());.
If you change name to a fixed string like name = "A name"; then it does work on the eris stack.
There is probably something happening in the bytes32ToString function that makes it crash. But this isn't the case when using the ethereum browser compiler.

I think this issue is linked to issue #258
The eris stack breaks when trying to assign a value y at index x for memory arrays, like:
bytes memory arr;
arr[0] = 1;

@the-real-aquaman the code you've given actually doesn't compile with 0.4.4. But something similar does: https://gist.github.com/anonymous/2bfbbb87339adf41d946cbf55b555f9b
    function f() {
        uint[] memory arr = new uint[](0x44);
        arr[0x11] = 0x22;
    }

The issue is that the array length is written too the boundary of the memory (the result of the MSIZE call). This is a aspect of the solidity compiler which I believe I have traced to this code:
https://github.com/ethereum/solidity/blob/develop/libsolidity/codegen/ExpressionCompiler.cpp#L842-L851
The reason it does this is still a little obscure to me and I am also missing how this works on other EVM implementations since it is effectively deliberately writing to memory out of bounds.
I have a fix in hand that provides dynamic memory allocation (since we have cases where our 1 MiB memory allocation is insufficient also causing out-of-bounds), but I need to also understand how this behaviour is supported elsewhere (even if it triggers resizes it seems rather odd to be continually writing to the end of memory however big that memory is, and so could cause run-away allocations).


Make MongoDB url customizable through envs

at the time of writting the mongodb url is built using GitHub-Secrets.
I want to have a possibility to use envs for configuring in development.

Hm just some way at all as local development is kinda rough without those secrets


Add an option to always have the tabs visible, if a panel has more than one tab

This new option ensures than a dockable panel will have its tabs visible, even when tabs_visible = false, when it has more than one tab.

Above is a panel with just one tab, and below is a panel with two tabs. This happens when tabs_visible = false and tabs_visible_if_more_than_one = true.
This is something we did in Pixelorama 0.x, because we wanted to avoid tabs being visible when unnecessary, but I never ported the change here so I thought I might as well do that.
This PR also removes set_tab_alignment() and get_tab_align(), as they were unused because the tab_alignment variable already has a getter and setter.
One thing I'm not too sure about is the variable name tabs_visible_if_more_than_one. I feel like this might be a bit too long (although use_hidden_tabs_for_min_size is also about the same length), but I'm not sure how to make it shorter while also ensuring that its purpose is clear.
And I'm also not sure about calling set_tabs_visible() in _resort(). There may be a more optimal way to do this, but without this, tab visibility isn't being updated when a panel gets added/removed/hidden.

Hey @OverloadedOrama , nice stuff, thanks for the addition! Sorry it took me so long to check this PR.
To be honest, for me it makes more sense to hide a single tab, even if tabs_visible == true, instead of showing multiple tabs even if tabs_visible == false. I mean, instead of tabs_visible_if_more_than_one, maybe something like single_tab_visible or hide_single_tab would be better.
What do you think?

This PR also removes set_tab_alignment() and get_tab_align(), as they were unused because the tab_alignment variable already has a getter and setter.

Awesome, thank you!

And I'm also not sure about calling set_tabs_visible() in _resort(). There may be a more optimal way to do this, but without this, tab visibility isn't being updated when a panel gets added/removed/hidden.

Hmm, I think the logic over hiding/showing tabs should be done in DockablePanel instead of the DockableContainer: the container simply forwards the options to the panels, which are responsible for their own tabs. This logic would likely live in the track_nodes method and the options' setter methods.
I wouldn't say it's more optimal, but it seems more organized/clean code-ish this way (either way the flag will be set for every panel during a resort).

To be honest, for me it makes more sense to hide a single tab, even if tabs_visible == true, instead of showing multiple tabs even if tabs_visible == false. I mean, instead of tabs_visible_if_more_than_one, maybe something like single_tab_visible or hide_single_tab would be better. What do you think?

That sounds good to me, I changed it to follow this logic and named the variable hide_single_tab.

Hmm, I think the logic over hiding/showing tabs should be done in DockablePanel instead of the DockableContainer: the container simply forwards the options to the panels, which are responsible for their own tabs. This logic would likely live in the track_nodes method and the options' setter methods. I wouldn't say it's more optimal, but it seems more organized/clean code-ish this way (either way the flag will be set for every panel during a resort).

Yeah that makes sense, I agree. I moved the logic to DockablePanel and added a new _handle_tab_visiblility method which gets called in the setter methods and in track_nodes, and changes in the DockableContainer tab visibility related variables has been limited to just update the DockablePanel variables.
A slight issue I encountered was that, while the DockableContainer variable is named tabs_visible, the DockablePanel variable can't be named the same way, as it is a built-in variable of TabContainer. So I went ahead and named it tabs_hidden, which has the opposite value of DockableContainer's tabs_visible. I'm not too happy about that but I couldn't think of a better name (I thought of should_tabs_be_visible but that seems too long), so ideas are welcome. Alternatively, if we want the DockableContainer and DockablePanel variables to have the same value, we could change DockableContainer's variable to also be tabs_hidden and negate it, at the cost of a small compatibility breakage. I don't have a strong opinion personally and I'm fine either way.

I renamed tabs_hidden to show_tabs. I think it is better now indeed, and since the logic between DockableContainer and DockablePanel's variables isn't being negated, there is no reason to change anything else now.

I think it is better now indeed

I agree! I've tested your branch here and it seems to be working fine, so I guess we can merge it now =]
Thanks again for the addition!


During Offline Embedding. RuntimeError: CUDA out of memory.

Hi, I run the following commond to build the database for small file (all_full.tsv) about 3.17 MB :
SEQDB_PATH=/home/fastMSA/all_full.tsv
MODEL_PATH=/home/Dense-Homolog-Retrieval/Dense-Homolog-Retrieval-main/
OUTPUT_PATH=/home/fastMSA/output
python3 do_embedding.py trainer.ur90_path=$SEQDB_PATH model.ckpt_path=$MODEL_PATH hydra.run.dir=$OUTPUT_PATH

RuntimeError: CUDA out of memory. Tried to allocate 9.59 GiB (GPU 0; 15.72 GiB total capacity; 11.27 GiB already allocated; 2.76 GiB free; 11.31 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.
Predicting:   0%|

Sorry for the late reply. Please try to reduce the batch_size in configuration to fit your memory. The embedding is done batch by batch instead of taking all sequences at once.

Sorry for the late reply. Please try to reduce the batch_size in configuration to fit your memory. The embedding is done batch by batch instead of taking all sequences at once.

Thanks a lot for reply.
After reducing the batch_size in the two configuration file (scale_conf.yaml and pred_conf.yaml), it worked well.


configure CMD-SHELL commands in the Prometheus scrape target

Hello,
I have a health check configured in the docker-compose. I am also using Blackbox to export the metrics to Prometheus. I am not sure how to configure in the black box config for the cmd_shell based commends?
Can you give me an example of BlackBox config and target to be configured in the Prometheus scrape for the below case ??
version: '2.1'
services:
  test-database:
    image: postgres:9.5
    container_name: postgres
    environment:
      - POSTGRES_USER=user
      - POSTGRES_DB=db
      - POSTGRES_HOST_AUTH_METHOD=trust
    healthcheck:
      test: ["CMD-SHELL", "pg_isready -U postgres"]
      interval: 10s
      timeout: 5s
      retries: 5

Thanks for your report. It looks as if this is actually a question about usage and not development.
It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.

Thanks for your report. It looks as if this is actually a question about usage and not development.
It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


consolidate sublist of storage tools

to make list of all tools more organized

Closing based on the above comments.


Enable watchOS for Remote Config and ABTesting

ABTesting is a hard dependency so enable it as well.
Testing: I'm able to get config value without any changes using the watchOS sample app.

Thanks for testing this! 🎉

my clang-format somehow is not in sync even tho it's the same version.

The log might be confusing, since it's actually swiftformat that checks the format of Swift files - not clang-format

Right, I also upgraded swiftformat and I ran both swiftformat and ./script/style.sh. None of them is accepted by CI.

swiftformat should be controlled by the Mintfile and running 0.45.5 both locally and in CI


README.md + License

otherwise it's not trivially easy for people to use a repo

i hate licensing. Cant i just put something somewhere on my github stating "MIT unless otherwise noted" ?

people do it all the time (i used to do it all the time) but it doesn't hold up in court.

closed in #2


Focus Issue when Playback Stops and Returns to prior screen

When you exist the Video Player and return to the prior Browsing screen, the focus is lost and not reset.  Need to keep track of where Focus was at, and make sure to reset.  This may have to been done in the onResume functionality after everything is loaded.  We might want to store this in the Presenter, and restore it from there.

This seems to be fixed now that the app is using the Leanback, HorizontalGridview.  Focus is maintained when coming back from the prior screen now.


(feat)examples: Halo2 Keccak256 Example Circuit

Added a Keccak256 Circuit to mopro-core/examples/halo2/keccak256 and templates/core/circuits/halo2-keccak256.
Added an option to run a halo2-keccak circuit to the template config.
Added a Swift view for halo2-keccak circuit.

The situation with workspace is quite tough.
Both keccak and fibonacci crates are actually called halo2-circuit, as the idea is that whenever anyone wants to add another circuit to the mix, they just create a crate called halo2-circuit, provide a path to it in the config, and it will overwrite it in the dependencies (a hack way to include a dynamic build dependency, I looked for a better way, but could not find anything).
And the idea behind the examples/halo2 was to provide an example of such substitution. However, apparently, as you can not have two same-named workspace members, both examples need to be either their own workspace or one of them should be in mopro workspace, one on its own.
At the same time, if we include a local dependency in another create, that is located inside of the another crate's workspace, Cargo enforces it to be in a workspace, and if it is its own workspace, it throws an error. However, if it is a path override, it is okay...
So the following should work:

fibonacci example to be added to the main workspace
keccak example to be as its own workspace

@ElusAegis how about also setting them different feature flags
e.g. fibonacci and keccak256
so we enable different circuits by enabling different feature flags

I also thought about this. One option is to do place both samples inside of one crate crate, and use features to select between them, similar to FFI. However, that would make the examples complicated and also not quite right, as instead of two independent crates we will get just one shared. This is not ideal as the whole point of examples what in my opinion to show how to have separate crates and do paths overrides.
Another option is to overwrite the dependency in the 'mopro-core' based on the feature flag. However, that seems impossible if both crates have the same name.
I propose to create an issue for that, and figure out something later - if there is a better solution.

@ElusAegis one other idea from @vimwitch is that
we just use the srs, pk, vk in each example
you can maintain the halo2 circuits in either your repo or zkmopro repo
and write down in readme or doc: https://zkmopro.org/docs/halo2/introduction
how to generate the srs, pk, vk
so that we don't have to maintain the halo2 circuits in mopro

I am not sure I understand. The issue is not just in the circuit assets, like srs, pk, vk, but also the logic to generate the proof. It is custom for each circuit afaik (may be there is a way not to have it like that, which I am not aware). So when a user calls generateStaticProof`, it should pick up a custom logic implemented by each circuit crate


Option --build-no-cache appears to use cache

I am trying to understand how (and where) Devcontainers are cached by the CLI. When I make a change to devcontainer.json, VSCode prompts me to rebuild the container. I'm trying to use the CLI to accomplish this same thing: running devcontainer up, and reflecting new changes to devcontainer.json.
Running the following command does not reflect new changes to devcontainer.json, even when all Docker containers are stopped. The only way I can get the CLI to completely rebuild seems to be calling docker rm on all stopped containers. Am I missing something?
devcontainer up \
  --workspace-folder . \
  --workspace-mount-consistency consistent \ # my naive attempt to get the CLI to notice recent changes to `devcontainer.json`
  --build-no-cache # another naive attempt by me to rebuild recent changes to `devcontainer.json`

--remove-existing-container will run a new docker build that should pick up your changes.

That works! Thanks for your help. What is the difference between --build-no-cache and --remove-existing-container?

--build-no-cache only applies when the image is built and that only happens when there is no container or --remove-existing-container is given.


Multi Adapter support

What does this PR do?

Adds multi-adapter training and inference support.
Fixes #211, #208, https://github.com/huggingface/peft/pull/133#issuecomment-1447801733, https://github.com/huggingface/peft/pull/133#issuecomment-1492990881 and https://gist.github.com/philschmid/821c5317d144250feef517aecd390b98

Usage

While loading the first adapter via PeftModel.from_pretrained, you can give it a name using **adapter_name** parameter. Else the default adapter name default is used.
To load another adapter, use **load_adapter()** method of PeftModel, e.g., model.load_adapter(peft_model_path, adapter_name)
To switch between adapters, use **set_adapter()** method of PeftModel, e.g., model.set_adapter(adapter_name)
To disable adapters, use context manager disable_adapter(), e.g., with model.disable_adapter()
Specific to LoRA method: To merge and unload the current active adapter so that the lora weights are added to the base model weights and he injected models are removed to get back the transformers base model with lora weights added, use merge_and_unload() method, e.g.,  model = model.merge_and_unload()

from peft import PeftModel
from transformers import LlamaTokenizer, LlamaForCausalLM, GenerationConfig

model_name = "decapoda-research/llama-7b-hf"
tokenizer = LlamaTokenizer.from_pretrained(model_name)
model = LlamaForCausalLM.from_pretrained(
    model_name,
    load_in_8bit=True,
    device_map="auto",
    use_auth_token=True
)
model = PeftModel.from_pretrained(model, "tloen/alpaca-lora-7b", adapter_name="eng_alpaca")
model.load_adapter("22h/cabrita-lora-v0-1", adapter_name="portuguese_alpaca")

model.set_adapter("eng_alpaca")
instruction = "Tell me about alpacas."
print(evaluate(instruction))

model.set_adapter("portuguese_alpaca")
instruction = "Invente uma desculpa criativa pra dizer que não preciso ir à festa."
print(evaluate(instruction))

with model.disable_adapter():
    instruction = "Invente uma desculpa criativa pra dizer que não preciso ir à festa."
    print(evaluate(instruction))

Link to colab notebook: https://colab.research.google.com/drive/1vrVg8G7AIdCM9qpcfZya0B7OEsPSAxbO?usp=sharing

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Would this work for Inference using LoRAs that were trained by using train_dreambooth.py ?

Hello @younesbelkada, in my tests wherein I ran all the example notebooks and scripts, the only breaking change is loading peft checkpoints from hub which used MergedLinear.
Why remove support for MergedLinear?

The AdaLoRA and MAM papers have shown that the best performance is achieved by incorporating LoRA adapters in FFN in addition to MHA and there is no specific reason to exclude the key layer.  So, I would gradually see more people using custom target modules or AdaLoRA (Adaptive LoRA). This makes mergedLinear a bit redundant.
Maintainance and complexity of MergedLinear are high compared to the functionality. Post removing this, the code is clean and easy to maintain and doesn't disable any model from supporting LoRA.

Other changes:

Now model.eval() no longer merges the weights of the lora into base model. It is now achieved by model.merge_adapter() and model.unmerge_adapter() for unmerging. This is based on user feedback that something explicit like this is better than model.eval() which people normally won't associate with merging lora adapters as they already have different purpose in pytorch. Let me know if this could lead to confusion and if we should support that too with decrecation warnings

Tested in production - works great!

Great timing on this!

disable_adapter() doesn't seem to work in my tests.

How can I just upload the lora and remain the base model？I want to reduce memory usage

this is nice


Loading moment-timezone when using Browserify

Is there a way to load moment-timezone instead of moment when using this module with browserify? Since I'm using browserify, angular-moment loads moment via require('moment'), which won't load moment-timezone.
I can change that line to the following angularMoment(angular, require('moment-timezone')); and it works for me, but obviously would not work for those who just want to use moment.

Did you try requiring moment-timezone before angularMoment? I think it should do the trick...

That did work, thanks!


feat(linter): eslint-plugin-jest/no-duplicate-hooks

part of https://github.com/oxc-project/oxc/issues/492
Rule Detail: link

Currently, many jest rules need to call collect_possible_jest_call_node to glob all nodes first, they will do these things for themselves, I want to change back and avoid calling run_once.
And I intend to let jest rules align to other kinds of rules, It will help other people to review codes too.
I will try it in jest/valid-expect-in-promise, you are always welcome to work on rules except jest/valid-expect-in-promise. :)
After I finish it, I'll ask for your advice!


Not Bound - only able to add  1 "unbound Pie"

I use GP mainly as a way to organize my launcher (great for this). I do not need hot keys for these actions. To save the Pie I need to assign a hot key to it.
It would be nice if i could add more than one Pie without a hotkey

Hi, this was solved in 0.6.1 version.
What version are you running?

Actually, there is a problem: you can only have one pie without a hot key.
Thanks for the report!

version 0.6.2


HIVE-25790 Make managed table copies handle updates (FileUtils)

What changes were proposed in this pull request?
Changed FileUtils.copy() to skip identical files on the destination directory to improve copy performance. FileUtils.copy() originally just removed and recreated the destination directory. This change makes it compare each file and directory, and delete only different files and directories.
Why are the changes needed?
In an optimized replication bootstrap scenario, it copies many files from source to destination. It can copy thousands of files. If it fails during copying process, it retries. Then it has some files already copied, but its implementation removes them and copy all of them entirely. It should skip the already copied ones.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
It introduced few JUnit test scenarios in TestFileUtils and TestCopyUtils. It will be tested with automated regression test suites on the test server.

I found a couple of timeouts on the previous run. I found that CopyUtils.leaveIdenticalFilesOnly should compare the source path with its corresponding path under the destination path, but it actually compared the source path with the destination path. So I fixed it.


Automatically add stations:

Enable the system to add new stations as soon as they are created. We may try to pull all the stations available in thingspeak under tag name = "mamase".

This is automatically done by the backend.


Add generators for common cases

Adds three generators to Kitto:
mix kitto.gen.dashboard
Generates an empty dashboard based ready to be built out. Includes a sample Text widget for developers to get the syntax for adding widgets to dashboards.
Usage:
$ mix kitto.gen.dashboard my_dashboard
# Generates `dashboards/my_dashboard.html.eex`

mix kitto.gen.job
Generates an empty job definition. Uses the passed in argument as the name of the job and broadcasted event.
Usage:
$ mix kitto.gen.job my_job
# Generates `jobs/my_job.exs`

mix kitto.gen.widget
Generates an empty widget. Uses the padded in argument as the name and class of the widget.
Usage:
$ mix kitto.gen.widget my_widget
# Generates `widgets/my_widget/my_widget.js` and `widgets/my_widget/my_widget.scss`

The generators are very simple and currently don't do much to check for validity of the names passed in.

Coverage decreased (-4.7%) to 82.428% when pulling e9ce614dfc2b8cf78d021f937b0d36cda5cbbfbd on generators into d373229a129ae363aca4c65fc131397d5ab951d4 on master.

Looks good, ping me when you add tests to have it merged. :+1:

Coverage decreased (-0.5%) to 86.624% when pulling bdd33097c69160c3974cf0df0212bbbce138b286 on generators into d373229a129ae363aca4c65fc131397d5ab951d4 on master.

Coverage decreased (-0.5%) to 86.624% when pulling ae95d4620886d917aa2b28e764f398756ab262b1 on generators into d373229a129ae363aca4c65fc131397d5ab951d4 on master.

There are only a few minor details to be fixed, otherwise it looks good.

Coverage decreased (-0.9%) to 86.25% when pulling 2e61371b7e01a87cd0939891c0a4e6347e0f061d on generators into d373229a129ae363aca4c65fc131397d5ab951d4 on master.

I have no idea what's up with the failing test... It seems to be trying to ask for confirmation, I assume to overwrite the file, but I don't know a way around that unless I just force overwrite.

Coverage decreased (-0.9%) to 86.25% when pulling 20daa17fb2a571ba539fe60b68189467c768e4b0 on generators into d373229a129ae363aca4c65fc131397d5ab951d4 on master.

Rebased and merged: https://github.com/kittoframework/kitto/commit/2b5104bbc27df3efed46e16b2d9ed7e2baf69463
Thanks @davejlong
Can you also add documentation concerning the generators in the wiki?


Failed validation: #/definitions/QuoteResponse

I'm getting the same set of errors for each of the following tickers: BITF.V, HUT.TO, MSTR, and NCTY when calling:
await yahooFinance.quote(ticker)

An example set of errors for NCTY is:
    errors: [
    {
      instancePath: '/0',
      schemaPath: '#/definitions/QuoteCryptoCurrency/required',
      keyword: 'required',
      params: [Object],
      message: "must have required property 'circulatingSupply'",
      data: [Object]
    },
    {
      instancePath: '/0',
      schemaPath: '#/definitions/QuoteCryptoCurrency/required',
      keyword: 'required',
      params: [Object],
      message: "must have required property 'coinImageUrl'",
      data: [Object]
    },
    {
      instancePath: '/0',
      schemaPath: '#/definitions/QuoteCryptoCurrency/required',
      keyword: 'required',
      params: [Object],
      message: "must have required property 'fromCurrency'",
      data: [Object]
    },
    {
      instancePath: '/0',
      schemaPath: '#/definitions/QuoteCryptoCurrency/required',
      keyword: 'required',
      params: [Object],
      message: "must have required property 'lastMarket'",
      data: [Object]
    },
    {
      instancePath: '/0',
      schemaPath: '#/definitions/QuoteCryptoCurrency/required',
      keyword: 'required',
      params: [Object],
      message: "must have required property 'startDate'",
      data: [Object]
    },
    {
      instancePath: '/0',
      schemaPath: '#/definitions/QuoteCryptoCurrency/required',
      keyword: 'required',
      params: [Object],
      message: "must have required property 'toCurrency'",
      data: [Object]
    },
    {
      instancePath: '/0',
      schemaPath: '#/definitions/QuoteCryptoCurrency/required',
      keyword: 'required',
      params: [Object],
      message: "must have required property 'volume24Hr'",
      data: [Object]
    },
    {
      instancePath: '/0',
      schemaPath: '#/definitions/QuoteCryptoCurrency/required',
      keyword: 'required',
      params: [Object],
      message: "must have required property 'volumeAllCurrencies'",
      data: [Object]
    },
    {
      instancePath: '/0',
      schemaPath: '#/definitions/QuoteCryptoCurrency/additionalProperties',
      keyword: 'additionalProperties',
      params: [Object],
      message: 'must NOT have additional properties',
      data: [Object]
    },
    {
      instancePath: '/0/quoteType',
      schemaPath: '#/definitions/QuoteCryptoCurrency/properties/quoteType/const',
      keyword: 'const',
      params: [Object],
      message: 'must be equal to constant',
      data: 'EQUITY'
    },
    {
      instancePath: '/0',
      schemaPath: '#/definitions/QuoteCurrency/additionalProperties',
      keyword: 'additionalProperties',
      params: [Object],
      message: 'must NOT have additional properties',
      data: [Object]
    },
    {
      instancePath: '/0/quoteType',
      schemaPath: '#/definitions/QuoteCurrency/properties/quoteType/const',
      keyword: 'const',
      params: [Object],
      message: 'must be equal to constant',
      data: 'EQUITY'
    },
    {
      instancePath: '/0',
      schemaPath: '#/definitions/QuoteEtf/additionalProperties',
      keyword: 'additionalProperties',
      params: [Object],
      message: 'must NOT have additional properties',
      data: [Object]
    },
    {
      instancePath: '/0/quoteType',
      schemaPath: '#/definitions/QuoteEtf/properties/quoteType/const',
      keyword: 'const',
      params: [Object],
      message: 'must be equal to constant',
      data: 'EQUITY'
    },
    {
      instancePath: '/0',
      schemaPath: '#/definitions/QuoteEquity/additionalProperties',
      keyword: 'additionalProperties',
      params: [Object],
      message: 'must NOT have additional properties',
      data: [Object]
    },
    {
      instancePath: '/0',
      schemaPath: '#/definitions/QuoteMutualfund/additionalProperties',
      keyword: 'additionalProperties',
      params: [Object],
      message: 'must NOT have additional properties',
      data: [Object]
    },
    {
      instancePath: '/0/quoteType',
      schemaPath: '#/definitions/QuoteMutualfund/properties/quoteType/const',
      keyword: 'const',
      params: [Object],
      message: 'must be equal to constant',
      data: 'EQUITY'
    },
    {
      instancePath: '/0',
      schemaPath: '#/anyOf',
      keyword: 'anyOf',
      params: {},
      message: 'must match a schema in anyOf',
      data: [Object]
    }
  ]

Error confirmed. Thanks for the detailed error report. The yahoo finance API has changed their response schema. This is something very new. This error should happen for almost every symbol, and it seems to do so...
I just opened a PR to fix this: #118. In the meantime, there isn't really much that anyone can do. You could pull down my branch and follow the instructions in CONTRIBUTING.md to use that branch temporarily.
@gadicc We definitely need to accommodate for this in our tests. Maybe regenerate the test files every week or so.

:tada: This issue has been resolved in version 1.10.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Thanks @gb221 for reporting this and @PythonCreator27 for your super fast fix!
@PythonCreator27, agreed.  Maybe a GitHub Action or something.  I wouldn't regenerate the test files but rather just run the tests without using the cache, with:
$ FETCH_DEVEL="nocache" yarn test

and see if any start breaking.
However, currently (recently? or since we now have so many tests), this exceeds Yahoo's limits and your IP gets banned.  I guess this will be easier once I finish #76.


Missing "Development" tab

Hello! I'm trying to following the instructions to disable the storage emulator (https://docs.microsoft.com/en-us/azure/vs-azure-tools-storage-emulator-using#initializing-and-running-the-storage-emulator), however when I right-click on my Azure Functions project in Solution Explorer and click Properties, the resulting window doesn't have the mentioned "Development" tab. I see the normal Application, Build, Debug, etc. tabs but no Development tab. How can I disable storage emulator? It's not working for me (won't create the LocalDB database) so I'm trying to use Azurite instead, but am blocked by this.

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

ID: 90ff8041-c5e9-51de-21fc-17d22eaf901a
Version Independent ID: b18a0b57-ab52-0bd7-2d56-58d0aa05dbba
Content: Configuring and using the Storage Emulator with Visual Studio
Content Source: articles/vs-azure-tools-storage-emulator-using.md
Product: visual-studio-dev15
Technology: vs-azure
GitHub Login: @ghogen
Microsoft Alias: ghogen

@IGx89 We are looking into this and will get back to you shortly.

@IGx89 Thank you for bringing this to our attention! I see the same behavior with my VS too. While I reach out to our internal Team to gather more details around this, I'd like to know the VS version you are using, just so it might help in troubleshooting further.
@ghogen Could you please share your insights?

Sure! Visual Studio 2019 latest stable, 16.4.1

I'll look into this and talk with the engineering team.  In the meantime, try editing your project file by adding the following to disable the Storage Emulator :

netcoreapp2.1
v2
False

That's exactly what I was looking for, thanks! Didn't know about that element.

@ghogen Thanks for that tip!
@IGx89 Please try it out and let us know if you run into issues.

@IGx89 We will now proceed to close this thread as resolved, but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

OK, I will also update the document to add this info.

@ghogen Thanks for the update!


added title to card rows

add title : "Introduction","Graphics","Game Logic","Utilities" to presentation cards
preview : http://dokthar.github.io/jmonkeyengine-ghpage/

plop @Nehon  ( i think you didn't saw the PR )


Add POSIX confstr API

Fixes #3767
Some notes for reviewers:

This PR was successfully tested with my Rust getconf util implementation at https://github.com/rustcoreutils/posixutils-rs
The confstr(3) API was added to unix, because it is POSIX standard, just like sysconf(3), and originates from libc, just like sysconf(3).
The confstr symbol in src/unix/bsd/apple/mod.rs was removed, with Darwin's link symbol notation moved to unix/mod.rs.
The _CS_PATH symbol is POSIX-standard and possibly belongs in unix.  However, it is unclear whether the symbol's value is guaranteed stable across OS's, therefore I kept the MacOS symbols as-is, and added the linux-gnu symbols per confstr man page on Linux.

Status:  all tests except arm-linux-androidabi are passing.  Unclear what is the best solution around this test failure.

Missing functions are either obsolete or explicitly disallowed by SELinux:

a64l/l64a
confstr
crypt/encrypt/setkey
gethostid
shm_open/shm_unlink
sockatmark
ualarm

source

fair enough, even tough I believe musl for example has it too for example but that s fair as is :)

The Platonic ideal would seem to be "confstr on all unix except x, y, and z" but that is difficult to express in the semver tests, I'm guessing.
Further complicating the task is that value of _CS_PATH is not specified and may vary across confstr-supporting OS's.  It is a POSIX-wide API with local-libc-specific values and symbology (thus the difference in Darwin and glibc symbols CS*)


The error message is incorrect for one Azure AD attached queue after removing its associated account

Storage Explorer Version: 1.19.1
Build Number: 20210417.6
Branch: hotfix/1.19.1
Platform/OS: Windows 10/ Linux Ubuntu 16.04/ MacOS Catalina
Architecture: ia32/x64
Regression From: Not a regression
Steps to Reproduce

Select one queue -> Give a RBAC role to it then assign to one account.
Sign in with the account -> Attach the queue via Azure AD.
Remove the account.
The Azure AD attached queue got broken -> Right click the broken queue -> Click 'View Error...'.
Check whether the error message is correct.

Expected Experience
The error message is correct.
The screenshot of the error message for one broken blob container.

Actual Experience
The error message is incorrect.

Merged into main, but opening a new issue to track something I noticed: https://github.com/microsoft/AzureStorageExplorer/issues/4412


Problem with styling

I've problem with styling when component is in the middle of context.

There I have component

And the InfiniteScroll makes a div around him and there is no way to inherit hight from outer component. The only solution which I found is to change line:
From var outerDivStyle = this.props.pullDownToRefresh && this.props.height ? { overflow: "auto" } : {};
To: var outerDivStyle = this.props.pullDownToRefresh && this.props.height ? { overflow: "auto", height: "inherit" } : {};

As you can see height:"inherited" was set in InfiniteScroll props but it inherits from this outerDiv instead of parent.
Also I don't want to specify constant height due to responsiveness of app.

Hi @staho, I am not sure if I completely understand your problem. As far as I can understand you want to style the content, can you do something like this https://codesandbox.io/s/0ykkzx672n
Here I am wrapping the content in my own div and styling it as I want. Does this help? If not can you give me a reproduce-able sample?
<InfiniteScroll
          dataLength={this.state.items.length}
          next={this.fetchMoreData}
          hasMore={true}
          loader={
Loading...
}
        >

            {this.state.items.map((i, index) => (

                div - #{index}

            ))}

        </InfiniteScroll>


Refactor/merge contexts

Merge BookingContext into the SearchContext
Prerender PassengerCards

PLEASE don't merge if it hasn't at least two approvals


Jorwoods/download excel files from views

Built on top of #949 . Adds support for downloading excel crosstabs from views.

Thanks for taking this issue on! I found myself needing to do the same recently, and was exploring whether to just directly use the REST API given the limitation within the Python client. Any guidance on when this might make its way into master?
Really appreciate the work - this will be a big help to many.

@cdonahue-chartis Strange. I see the functions in master, but when I install 0.18 in a venv, I also do not see the excel functions.

@jorwoods Interesting. Glad I wasn't missing something. Please let me know if/when you have any updates on release timing. In the meantime, I'm able to use the dev branch and the Excel export works wonderfully. Thanks again.


How I started out with dbt / Pasha Finkelshteyn @ JetBrains Blog

URL: https://blog.jetbrains.com/big-data-tools/2022/01/25/how-i-started-out-with-dbt/
Review (1-2 sentences): If you ever wanted to try DBT you can do it with Pasha. Many pictures and code snippets were added for free :)

Published in Digest #17.


Where is the code?

I'm so sorry you found it blank. I have added the codes along with pre-built binaries. Should you face problems with those, you can build those from scratch using the build script.

I'm so sorry you found it blank. I have added the codes along with pre-built binaries. Should you face problems with those, you can build those from scratch using the build script.


Drag drop issue in IE

Hi,
Firstly thank you for an amazing tree component. Searched for a while and it really was the best fit for what I needed. Hope you can help with this last issue.
I'm using version 3.1.0 and noticed when dragging a node outside of the tree, it works perfectly in Chrome but not in IE.
When testing in IE, in the callback onExclusionDrop, $event.element returns  null.
Should I rather upgrade to a more recent version?
Kind regards
Sheree

Hi,
Which version of IE ?
Do you have a working plunkr / jsbin to demonstrate the error?
Where do you use onExclusionDrop, is it in actionMapping or in (moveNode) output?
Thanks

Hi Adam
I was testing on Edge.  The page is quite complex at this point and
difficult to replicate in a Plunkr. I hope maybe this snippet of code below
might explain better. I am using the (treeDrop) directive on a  table.
HTML

Name

  Hint:

  Drop node to exclude here.

..........
TypeScript - code behind
onExclusionDrop($event,tree) {
//$event.element is correct when testing in Chrome but null in IE
this.addExclusion($event.element.data.nodeId,$event.element.data.shortName,$event.element.data.nodeType,$event.element.data.uuid,tree);
}
allowExclusionDrop(element) {
return true;
}
Kind regards
Sheree
On Sat, Jun 10, 2017 at 12:20 PM, adam klein<EMAIL_ADDRESS>wrote:

Hi,
Which version of IE ?
Do you have a working plunkr / jsbin to demonstrate the error?
Where do you use onExclusionDrop, is it in actionMapping or in (moveNode)
output?
Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/500tech/angular-tree-component/issues/327#issuecomment-307556382,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXL0MpxT3j-LnY40NBb78eI8CvR6GORRks5sCm3-gaJpZM4N1XEJ
.

Hi Adam
Can I mail a zipped project to you without the node modules?
Kind regards
Sheree
On Sun, Jun 11, 2017 at 8:13 PM, Sheree Kidson<EMAIL_ADDRESS>wrote:

Hi Adam
I was testing on Edge.  The page is quite complex at this point and
difficult to replicate in a Plunkr. I hope maybe this snippet of code below
might explain better. I am using the (treeDrop) directive on a  table.
HTML

Name

  Hint:

  Drop node to exclude here.

..........
TypeScript - code behind
onExclusionDrop($event,tree) {
//$event.element is correct when testing in Chrome but null in IE
this.addExclusion($event.element.data.nodeId,$event.element.data.shortName,$event.element.data.nodeType,$event.element.data.uuid,tree);
}
allowExclusionDrop(element) {
return true;
}
Kind regards
Sheree
On Sat, Jun 10, 2017 at 12:20 PM, adam klein<EMAIL_ADDRESS>wrote:

Hi,
Which version of IE ?
Do you have a working plunkr / jsbin to demonstrate the error?
Where do you use onExclusionDrop, is it in actionMapping or in (moveNode)
output?
Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/500tech/angular-tree-component/issues/327#issuecomment-307556382,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXL0MpxT3j-LnY40NBb78eI8CvR6GORRks5sCm3-gaJpZM4N1XEJ
.

@shereekidson can you try to debug it and see where the problem is?
the $event.element is being pulled from the treeDraggedElement service, which is being set in tree-drag.directive.

Closing this issue due to inactivity. Please open a new issue if the problem still exists.


Move Turso REST API package to a different repository

The Turso REST API package is useful beyond the CLI so let's move it to a github.com/tursodatabase/go-turso repository.

Duplicate of https://github.com/tursodatabase/turso-cli/issues/16 even author is the same :D


Upgrade rrweb-snapshot to support shadow DOM constructed stylesheets

Issue: #AP-5027
What Changed
Just what the title says. Constructable stylesheets in shadow DOM elements will now be displayed in our captures.
How to test

Run yarn test:cypress && yarn archive-storybook:cypress
View the styles render in shadow dom elements and styles render in web components in shadow dom stories in the constructable-stylesheets folder
Verify that the sections are styled with the colors described in the text on those story pages
Run yarn test:playwright && yarn archive-storybook:playwright
Repeats steps 2-3
OPTIONAL: Create your own Cypress or Playwright project that uses constructed stylesheets in shadow DOM elements and verify that they render as expected in the archived Storybook

@andrewortwein I've QA-ed this against both out-of-the-box-Lit and Siements web components (both use constructed stylesheets) and the constructed styles show up in the Storybook!


When registering a namespace, N/A is displayed under Amount.

@garm01 what is the expected result?

Maybe instead of N/A, it would be useful to show the deducted rental fee.
Similar to sending from XYM. The amount without paid fee.
At the moment of the transaction you have a fixed amount. If the namespace is renewed, a new transaction is needed.

I think it's consistent because rental fees are still fees and those are not presented. Maybe an additional column (fee) should be presented (but there is a lack of space). Maybe hidden by default? Someone from UX should look at this.
Also related issue https://github.com/nemgrouplimited/symbol-desktop-wallet/issues/856

User Story: “As a user registering a namespace, I want to always know the cost of the rental fee to pay, so I understand that the process has a cost and how much it is.”
Acceptance Criteria

AC1: When registering a namespace, the cost of the rental fee should be displayed clearly.
AC2: When I have registered namespaces and I can see the transaction in the transaction history, the FEE should be displayed and correctly identified.


Make README and conan recipe independent of oneAPI version

Description
Currently, the README and conanfile.py must be updated for every new release of oneAPI. This PR proposes to remove this dependency and always use the latest oneAPI version.

Output from conan install ../ --profile inteldpcpp_lnx --build missing to verify that conan actually gets the intended package from APT.
Configuration:
[settings]
arch=x86_64
arch_build=x86_64
build_type=Release
compiler=clang
compiler.libcxx=libstdc++11
compiler.version=11
os=Linux
os_build=Linux
[options]
[build_requires]
*: cmake/[>=3.15], ninja/1.10.0
[env]
CC=/opt/intel/inteloneapi/compiler/latest/linux/bin/clang
CFLAGS=-fdiagnostics-color -fdiagnostics-show-template-tree -march=native
CONAN_CMAKE_GENERATOR=Ninja
CONAN_SYSREQUIRES_MODE=enabled
CONAN_SYSREQUIRES_SUDO=True
CPATH=[/opt/intel/inteloneapi/compiler/latest/linux/include]
CXX=/opt/intel/inteloneapi/compiler/latest/linux/bin/dpcpp
CXXFLAGS=-fdiagnostics-color -fdiagnostics-show-template-tree -march=native
LDFLAGS=-Wl,-rpath=/opt/intel/inteloneapi/compiler/latest/linux/lib/
LD_LIBRARY_PATH=[/opt/intel/inteloneapi/compiler/latest/linux/lib:/opt/intel/inteloneapi/compiler/latest/linux/lib/x64:/opt/intel/inteloneapi/compiler/latest/linux/compiler/lib/intel64:/opt/intel/inteloneapi/compiler/latest/linux/../../../tbb/latest/lib/intel64/gcc4.8/]
LIBRARY_PATH=/opt/intel/inteloneapi/compiler/latest/linux/../../../tbb/latest/lib/intel64/gcc4.8/
Version ranges solved
    Version range '>=3.15' required by 'conanfile.py (oneMKL/0.1.0-beta)' resolved to 'cmake/3.17.3' in local cache

conanfile.py (oneMKL/0.1.0-beta): Installing package
Requirements
Packages
Build requirements
    cmake/3.17.3 from 'conan-center' - Cache
    lapack/3.7.1@conan/stable from 'conan-center' - Cache
    ninja/1.10.0 from 'conan-center' - Cache
    openssl/1.1.1g from 'conan-center' - Cache
    zlib/1.2.11@conan/stable from 'conan-center' - Cache
Build requirements packages
    cmake/3.17.3:48e65bdccfe2ff24e5926c49bd4235dc41535d54 - Cache
    lapack/3.7.1@conan/stable:554ab41874f75f88d49da0e471b1162d7a88e3e6 - Cache
    ninja/1.10.0:a6ae3c90901101d61d7824b47d0ce9ad23fec1b8 - Cache
    openssl/1.1.1g:d988447fa516eac7400b2f34e2d4b89e42b4b1a8 - Cache
    zlib/1.2.11@conan/stable:d988447fa516eac7400b2f34e2d4b89e42b4b1a8 - Cache

Installing (downloading, building) binaries...
ninja/1.10.0: Already installed!
openssl/1.1.1g: Already installed!
zlib/1.2.11@conan/stable: Already installed!
cmake/3.17.3: Already installed!
cmake/3.17.3: Appending PATH environment variable: <removed Intel(R) internal path>/vmalia/home/.conan/data/cmake/3.17.3/_/_/package/48e65bdccfe2ff24e5926c49bd4235dc41535d54/bin
lapack/3.7.1@conan/stable: Already installed!
conanfile.py (oneMKL/0.1.0-beta): Applying build-requirement: cmake/3.17.3
conanfile.py (oneMKL/0.1.0-beta): Applying build-requirement: ninja/1.10.0
conanfile.py (oneMKL/0.1.0-beta): Applying build-requirement: lapack/3.7.1@conan/stable
conanfile.py (oneMKL/0.1.0-beta): Applying build-requirement: openssl/1.1.1g
conanfile.py (oneMKL/0.1.0-beta): Applying build-requirement: zlib/1.2.11@conan/stable
conanfile.py (oneMKL/0.1.0-beta): Generator cmake created conanbuildinfo.cmake
conanfile.py (oneMKL/0.1.0-beta): Generator txt created conanbuildinfo.txt
conanfile.py (oneMKL/0.1.0-beta): Generated conaninfo.txt
conanfile.py (oneMKL/0.1.0-beta): Generated graphinfo
conanfile.py (oneMKL/0.1.0-beta):
 LEGAL NOTICE: By downloading and using this container or script as applicable
 (the “Software Package”), and any included software or software made available
 for download, you agree to the terms and conditions of the software license
 agreements for the Software Package, which may also include notices,
 disclaimers or license terms for third party software (the “Agreements”)
 located at https://github.com/conan-io/conan/blob/develop/LICENSE.md, the
 THIRD-PARTY-PROGRAMS file and in the README.md file included with the Software Package.

Running: sudo -A apt-get update
Hit:1 https://apt.repos.intel.com/oneapi all InRelease
Hit:2 https://deb.nodesource.com/node_13.x bionic InRelease
Hit:3 http://archive.ubuntu.com/ubuntu bionic InRelease
Hit:4 http://archive.ubuntu.com/ubuntu bionic-security InRelease
Hit:5 http://archive.ubuntu.com/ubuntu bionic-updates InRelease
Reading package lists...
Package already installed: intel-oneapi-mkl-devel
Package already installed: intel-oneapi-tbb-devel


CPHD Metadata error

In sarpy.io.phase_history.cphd1_elements.blocks.AreaType it is indicated that the AreaType.Polygon field is required, however, within the CPHD specification, everywhere that AreaType is used, the Polygon field is optional.
Example from CPHD.SceneCoordinates

Included in version 1.3.59


Vickel: Watch coinatmradar.com

Vickel requests the watch of the watch_keyword coinatmradar\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

coinatmradar\.com has been seen in 1 true positive, 8 false positives, and 0 NAAs.

Rejected by Jeff Schaller in Charcoal HQ. Reason: 'at author's request: too many fp and on-site hits'


Multiboot

Hello, I was wondering if it is possible to add an additional system with tools like NOOBS or PINN, I want to do a dual system, can it? Thanks.

in theory yes, but it's obviously outside the scope of this project. not to mention it's quite easy to achieve what you want by switching the SD cards.


Is there a Better way to Label Printing?

First, I would like to thank you all @gordol , @momzor @ayiga , @sllobera , @ltrtom  for your hard work.
This is rather a set of questions
(Note that I am new to label printing ...)
1-  Would this https://play.google.com/store/apps/details?id=com.brother.printservice&hl=en_GB
help print directly from an app without this cordova plugin?
I have tried but the Brother Print Service could not find my QL-710w.
2- IMHO, there must be a better way to label printing...
let's say I need to support in my app 10 brands of label printers... with the current approach, I will have to integrate and bundle with my app 10 SDKs and their respective native libraries ... :-(
There must be a better way ...
Thank you very much @gordol , @momzor @ayiga , @sllobera , @ltrtom  and others for your work on this
Arcadius

Hi @arcadius
Glad you got some use from this plugin. It is most definitely rough around the edges... so thanks for your patience.
There are no universal printing solutions for Android at the moment, that I'm aware of. Every manufacturer has proprietary drivers and APIs.
That being said, what are the 10 manufacturers that you need to support? If someone can get me the label printers, I'm happy to try to get them all working with a single library!
Google has been working on a "native" print solution for ages, but as far as I know, it's not come to fruition yet.
Anyway, I wanted to let you know that I have seen your message, and am considering it, but I don't yet have a solution for you.
If you can let me know the manufacturers, I can do a little research and will report my findings here.
Best of luck!

Thank you very much @gordol  for your message.
Most appreciated. :+1:
For now, I need support only for Brother label printers .... I'll focus on that for now.
Thank you very much for starting work on this.
I am currently working on a fork at
https://github.com/MenelicSoftware/cordova-plugin-brother-label-printer
and I will be adding support for Windows and later iOS too.
Later on, we can discuss how we can merge things together so that we have a unique and generic enough plugin for these label printers.
Thank you very much @gordol :+1:

Hi @arcadius just to let you know, on the dev branch, we have base-line support for IOS in progress, and there's now basic bluetooth printing support as well as the ability to set the printer model from JS via setPrinter()

Hello @gordol
Thank you very much for letting know.
I think it would be best if we try merging so that we avoid duplicate of effort
Common JavaSript interface
In my fork,
https://github.com/MenelicSoftware/cordova-plugin-brother-label-printer/blob/master/www/printer.js
I have exposed only one single interface.:
printLabel(base64ImageOrPdfToPrint, {"numberOfCopies":1, "orientation":"LANDSCAPE", "labelName":"W62RB", "modelName":"QL-820NBW", "port":"NET"})

This internally does the printer finding etc ....
So, in the mobile app, one can have a setting page and change the printer model or port on the fly and print straight away without restarting the mobile app (the native Android code in my fork is stateless).
Please, think a bit about having a single JavaScript entry point .... take your time ....
IMHO, Having the printer search hidden from the caller simplifies a bit the system.
We need to work together @gordol  :+1:
Thank you very much for the great work :1st_place_medal:

i don't think a single JS interface is sufficient, firstly, because there may be more than one way to print, and users may want to use use one or another. secondly, we also support pushing PJL to the printer via USB, this can be used, for example, to configure the printer's network settings, etc... from cordova, over usb. see here: https://github.com/gordol/PJL-Generator
that being said, if you want to have a fire and forget print method that encapsulates the others, i see no reason why that couldn't be added, as long as it doesn't break support for the current manual workflows.

Thanks @gordol  for mentioning PJL.
Yes, we could have one interface for sending printer configurations/commands and one for the actual printing operation.

that being said, if you want to have a fire and forget print method that encapsulates the others

This is not a fire and forget as such.
When you try to print and let's say the printer is off or no paper or something, you will still get an error message .... So you know what whent wrong.

Okay, I still want to keep the separate methods, ideally.
If you want to send a PR for an additional method that encapsulates the others, to attempt the first printer that is found, that would be reasonable.
The feature you want should be done fully in JS/Cordova space, I think, to avoid the necessity to port native code to all of the platforms. You can throw in an additional method in www/printer.js. It can run findPrinters, setPrinter, and then printViaSDK. Maybe you can call it something like findSetAndPrint or something along those lines... or printToAny or something along those lines...


The whid has all of a sudden stopped working

My Whid has stopped working. It doesn't do anything if I plug it in my computer. (I tried on multiple computers and it didn't got regonised at all). It came in the mail a few days ago and now it doesn't work anymore.

Allready tried it a few times and followed all the instructions clearly and didn't work. The light aren't even blinking on my WHID when I plug it in.. Thanks for the suggestion though!

if there might be a component problem, maybe because it's a bit loose, you can try it by heating it with a hair dryer, but not too long,


Csm stats tests

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
No issues. Updating functionality
What are the significant changes in functionality due to this change request?
Implements the first round of testing metrics from Colorado School of Mines

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code.
[x] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[x] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[x] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[x] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[x] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[x] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[x] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

@FlanFlanagan there was recently a merge to RAVEN devel that changed the file structure (long story but unavoidable). Now the raven/framework path is raven/ravenframework, so there's some conflicts arising from that.

@FlanFlanagan there was recently a merge to RAVEN devel that changed the file structure (long story but unavoidable). Now the raven/framework path is raven/ravenframework, so there's some conflicts arising from that.

Okay. I'll merge develop and make sure everything is in the right place!

BUILD_ROOT/raven/: ./developer_tools/pylint_check
CONDA
raven_libraries_2.0      /apps/local/anaconda/3.7/envs/raven_libraries_2.0
raven_libraries          /home/moosetest/.conda/envs/raven_libraries
raven_libraries_alt      /home/moosetest/.conda/envs/raven_libraries_alt
Framework
PYLINTHOME is now '/home/moosetest/.cache/pylint' but obsolescent '/home/moosetest/.pylint.d' is found; you can safely remove the latter
************* Module Metrics.metrics.StatsTestsUtils
framework/Metrics/metrics/StatsTestsUtils.py:1:0: C0114: Missing module docstring (missing-module-docstring)

-------------------------------------------------------------------
Your code has been rated at 9.99/10 (previous run: 10.00/10, -0.01)

Whitespace removed!

Thanks for all your work on this, @FlanFlanagan !

@FlanFlanagan Thanks for your contribution. FYI, there are some violation of raven code standard in this PR. Basically, we are not allowed to use underscore between words for variables, and we have specific format for docstrings of functions. You can check the standards here: https://github.com/idaholab/raven/wiki/RAVEN-Code-Standards
If you have time, could you do me a favor to fix these violations in the PR? Thanks in advance.

@FlanFlanagan In addition, is it possible that you can update the raven user manual to reflect the new features that you added in this PR.

@wangcj05 I was too hasty in merging this; I thought I had finished the style review previously. We can fix these up in a follow-on PR.
@FlanFlanagan I've marked some notes in this PR for the sections that need looking at, let me know if there's issues fixing this up. The documentation isn't automatically generated for RAVEN, so we need to update the manual as well.

I'll get on this asap

Will do!

Coming soon! Sorry!
Where should the manual update go?

The manual for metrics is located in raven/doc/user_manual/metrics.tex


Clarification Needed on Python Compatibility

Description of issue (what needs changing)
No response
Clear description
I've recently discovered that Mediapipe has specific compatibility requirements with certain versions of Python, which can be confusing for users who are not aware of this beforehand. As such, I suggest adding clarification in the README file to inform users about the compatible Python versions.
Correct links
No response
Parameters defined
No response
Returns defined
No response
Raises listed and defined
No response
Usage example
No response
Request visuals, if applicable
No response
Submit a pull request?
No response

Hi @shriyakalakata,
Apologies for any confusion. The information about supported Python versions for mediapipe is already available in our documentation. You can find the compatible versions of Python listed there.
Thank you!!

Hi @shriyakalakata,
Could you review the previous comment and confirm if we can consider the matter resolved and close the status?
Thank you!!

@kuaashish Hi, thank you for pointing to the other doc! I do think that it might be good/clearer to have it on the github repo as well. Because someone who's using mediapipe as a package and not contributing to it might not access the document mentioned.

Hi @ktonthat,
Could you please look into this issue? If we can include the Python-supported versions somewhere on GitHub as well.
Thank you!!


Sorry,a newbie here asking basic setup infos

i tried to setup a proper env to generate the demos.
But somehow have some errors.
(```
Hifigan384test) D:\Coding\PYFastCache\PYVenv\Hifigan384test\hifi-gan-master>python inference.py --checkpoint_file vctk_v2\generator_v2 --input_wavs_dir test_files
Initializing Inference Process..
Loading 'vctk_v2\generator_v2'
Complete.
Removing weight norm...
D:\Coding\PYFastCache\PYVenv\Hifigan384test\hifi-gan-master\meldataset.py:15: WavFileWarning: Chunk (non-data) not understood, skipping it.
sampling_rate, data = read(full_path)
Traceback (most recent call last):
File "inference.py", line 94, in
main()
File "inference.py", line 90, in main
inference(a)
File "inference.py", line 54, in inference
x = get_mel(wav.unsqueeze(0))
File "inference.py", line 26, in get_mel
return mel_spectrogram(x, h.n_fft, h.num_mels, h.sampling_rate, h.hop_size, h.win_size, h.fmin, h.fmax)
File "D:\Coding\PYFastCache\PYVenv\Hifigan384test\hifi-gan-master\meldataset.py", line 61, in mel_spectrogram
y = torch.nn.functional.pad(y.unsqueeze(1), (int((n_fft-hop_size)/2), int((n_fft-hop_size)/2)), mode='reflect')
File "D:\Coding\PYFastCache\PYVenv\Hifigan384test\lib\site-packages\torch\nn\functional.py", line 2877, in pad
assert len(pad) == 4, '4D tensors expect 4 values for padding'
AssertionError: 4D tensors expect 4 values for padding
not quite sure what went wrong.Maybe the audio?

here's the layout.
02/12/2020  18:04    <DIR>          .
02/12/2020  18:04    <DIR>          ..
30/11/2020  23:03               762 config_v1.json
30/11/2020  23:03               762 config_v2.json
30/11/2020  23:03               752 config_v3.json
30/11/2020  23:03               394 env.py
02/12/2020  18:00    <DIR>          generated_files
02/12/2020  16:59    <DIR>          generated_files_from_mel
30/11/2020  23:03             2,652 inference.py
30/11/2020  23:03             2,444 inference_e2e.py
30/11/2020  23:03             1,067 LICENSE
30/11/2020  23:03    <DIR>          LJSpeech-1.1
30/11/2020  23:03             6,314 meldataset.py
30/11/2020  23:03             9,905 models.py
30/11/2020  23:03             4,767 README.md
30/11/2020  23:03               113 requirements.txt
02/12/2020  18:04    <DIR>          test_files
30/11/2020  23:03            12,153 train.py
30/11/2020  23:03             1,377 utils.py
30/11/2020  23:03            10,995 validation_loss.png
02/12/2020  16:53    <DIR>          vctk_v2
02/12/2020  16:59    <DIR>          __pycache__
              14 File(s)         54,457 bytes
               8 Dir(s)   7,409,783,296 bytes free

btw,for the audio i just recorded my voice.Not quite sure.What is needed for input.
I assume it just needs some wav audio?

Yes. Put the audio files you recorded in 'test_files' folder. Audio should be sampled at 22kHz.

tried record other audio with 22050Hz.with audacity.
It is downsample to 16bit.
I named it like>Def Tech,My Way,THE FIRST TAKE_22kHz.wav
Just to confirm
pip install torch===1.4.0 torchvision===0.5.0.it is the same version right?

Check if your audio is mono or stereo?
(mono is required)

ahhh stereo...i see ,okok sorry sorry.should we also add that to readme then?

Hmmm.. maybe I will do a pr to include a note for the.audio specs.Well maybe can prevent someone as stupid as me uses a mono audio and resample rate.hahaha had been using it to record instruments.forget to change those settings.Thanks for the guide.Everyone.I did generate the audio. I will research more about how to give text input or mel spectogram.


To many deaths?

Hello,
I am using your synthea:
synthea-with-dependencies.jar
downloaded at 2020 june 3
on Ubuntu 20.04 LTS to generate different patient populations. I am surprised about if I am using the command:
java -jar synthea-with-dependencies.jar -p 1000
I obtain finally
{alive=1000, dead=225}
the number of dead seems to me right. But if I use for example the following command:
java -jar synthea-with-dependencies.jar -a 0-140 -p 1000
I obtain finally
{alive=945, dead=2183}.
That seems to me to many dead people. The same occurs with other ranges of age.
What is going wrong?
Best regards.

My guess is that a significant amount of those people died from covid-19 and just being crazy old. 140 is really really old.
Even if the default max age is 140, the way the ages are generated is different if you specify them and if you do not.
If you specify the age using the -a option, the ages are selected with a uniform distribution. This is going to have a LOT of old people, who are going to die.
If you do not specify the age using the -a option, the ages are selected from the distributions in the demographics file, which are going to skew more towards young and middle aged. There will be LESS very old people, who will die less.

Thank you for your fast response. I understand. It is not an error.
Best regards.

My guess is that a significant amount of those people died from covid-19 and just being crazy old. 140 is really really old.
Even if the default max age is 140, the way the ages are generated is different if you specify them and if you do not.
If you specify the age using the -a option, the ages are selected with a uniform distribution. This is going to have a LOT of old people, who are going to die.
If you do not specify the age using the -a option, the ages are selected from the distributions in the demographics file, which are going to skew more towards young and middle aged. There will be LESS very old people, who will die less.

Hi @jawalonoski , in case not using -a, how do we make sure the generated population is above 18 years old ? I am interested in creating a volume of acute care patients across hospitals in a given city with the generated population is at randomized between 75 - 100 % of the bed capacity for each hospitals.
Similar need is to generate a population for  -

(-a <18 ) Children hospitals and
for LTAC (long term acute care) hospitals and
some for Hospice facilities

Your help will be appreciated.
Thank you.


Module parse failed: Unexpected token

Hi,
I tried to use this library in a lerna project. After I install mui-tel-input  dependancy an error starts to appear in the console when I run storybook.
ERROR in ./node_modules/mui-tel-input/dist/mui-tel-input.es.js 3645:8 Module parse failed: Unexpected token (3645:8) You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders |     if (i && !T && (u.isoCode || e)) { |       const V = +${m2(u.isoCode || e)}`;

  l?.(V, A("input")), g(V), w({

|         isoCode: u.isoCode || e || null,
|         inputValue: V
@ ./packages/react-ui/src/shared/MetadataField/MetadataField.tsx 50:19-43
@ ./packages/react-ui/src/shared/MetadataField/index.tsx
@ ./packages/react-ui/src/index.ts
@ ./.storybook/decorators.js
@ ./.storybook/preview.js
@ ./.storybook/preview.js-generated-config-entry.js
@ multi ./node_modules/<EMAIL_ADDRESS>./node_modules/<EMAIL_ADDRESS>./node_modules/webpack-hot-middleware/client.js?reload=true&quiet=false&noInfo=undefined ./storybook-init-framework-entry.js<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/preview.js-generated-config-entry.js ./generated-stories-entry.js`
Do you know wich loader I need to add in webpack configuration?
Thank you,
Federico

Hello ! Looks like your configuration does not handle Optional chaining .


Fix use of local configurations on windows

solves https://github.com/FZJ-INM1-BDA/siibra-python/issues/292

Codecov Report

Merging #482 (1b4c541) into main (5a0e6b3) will increase coverage by 0.65%.
Report is 134 commits behind head on main.
The diff coverage is 27.30%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #482      +/-   ##
==========================================
+ Coverage   36.81%   37.47%   +0.65%
==========================================
  Files          61       61
  Lines        5421     5641     +220
==========================================
+ Hits         1996     2114     +118
- Misses       3425     3527     +102

Files
Coverage Δ

siibra/configuration/configuration.py
42.35% <ø> (ø)

siibra/core/atlas.py
85.71% <ø> (+1.23%)
:arrow_up:

siibra/core/region.py
45.61% <100.00%> (+1.31%)
:arrow_up:

siibra/features/connectivity/__init__.py
83.33% <100.00%> (ø)

...ibra/features/connectivity/tracing_connectivity.py
75.00% <100.00%> (ø)

siibra/features/image/__init__.py
100.00% <ø> (ø)

siibra/locations/boundingbox.py
19.88% <ø> (ø)

...a/features/connectivity/functional_connectivity.py
55.55% <0.00%> (ø)

siibra/features/image/volume_of_interest.py
68.42% <66.66%> (-0.33%)
:arrow_down:

siibra/features/tabular/layerwise_cell_density.py
36.95% <50.00%> (ø)

... and 28 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


pancakeswap finance website false phishing alert

Phishing detected
You have been redirected because the Polkadot{.js} extension believes that this website could compromise the security of your accounts and your tokens.
https://pancakeswap.finance/swap?chain=bsc
Note that this website was reported on a community-driven, curated list. It might be incomplete or inaccurate. If you think that this website was flagged incorrectly, please open an issue by clicking here.

https://github.com/polkadot-js/phishing/pull/4122


docs: update command for configuring token

Seems like the command was based on the previous name of the binary and needed an update :smiley_cat:

I have read the CLA Document and I hereby sign the CLA


Seemingly unnecessary type annotation of instance variables

Input
→ View on sorbet.run
# typed: strict
class Banana
  extend T::Sig

  sig { params(color: String).void }
  def initialize(color)
    @color = color

    # The error 6002 could be fixed by
    # repeating the type information already
    # given in the sig.
    # @color = T.let(color, String)
  end
end

Observed output
editor.rb:7: Use of undeclared variable @color https://srb.help/6002
     7 |    @color = color
            ^^^^^^
Errors: 1

Expected behavior
No error should be produced.
Analysis
The type of @color can be inferred from the method signature.
I don't know enough to say if the current behavior (requiring the type information to be repeated in a T.let) is necessary, but if so, the repetition is unfortunate.
:heart:
sorbet is my favorite thing about ruby in 2019. Keep up the good work!

Thanks for the report. Sorbet has opted to make declaring a class's API (internal and external). Only local variables are inferred—this is one of the key ways Sorbet achieves really fast type checking speeds, because type checking one method becomes completely independent of all other methods and can be done in parallel.
Some time ago I added an FAQ section about this:
https://sorbet.org/docs/type-annotations#why-do-i-need-to-repeat-types-from-the-constructor
As you mention, this is unfortunate, and we might change this in the future, but we have no immediate plans to.

Thanks Jake. Sorry I missed the docs; nice write-up.

As you mention, this is unfortunate ..

Thinking about this a bit more, java has the same limitation, so we're in good company.
class Banana {
  public int mass;
  public Banana(int mass) {
    this.mass = mass;
  }

Oddly, the exact same level of duplicate-type-annotation never bothered me in java.
Feel free to close this, or maybe keep it open if you think it will keep other people from raising the same issue.

I think this is resolved by #2230?


toggle submissions in and out of competition

This is a crucial functionality for competitive RAMPs, this is the way for a participant to select a particular submission to participate in the competition. The button was kept in the frontend but the view function was missing in this new version of the code.
It is added back in this commit: https://github.com/paris-saclay-cds/ramp-board/pull/276/commits/63d39202c8486a79eb29d6b50bb5cba5abb031ab
The issue can be closed when the https://github.com/paris-saclay-cds/ramp-board/tree/acrobot_dev branch is merged into master.
cc @tomMoral @agramfort if you use this feature in the data camp

yes we saw this too. I attempted a fix
https://github.com/paris-saclay-cds/ramp-board/pull/278 but have no time to
pursue.

I think your fix is more about this issue isn't it? https://github.com/paris-saclay-cds/ramp-board/issues/289


session insert record error

when insert a record to database i got "'NoneType' object is not callable"
`
import tornado.web
from tornado.gen import coroutine
from tornado_sqlalchemy import as_future
from database.tables import *
class UploadFileHandler(BaseHandler):
@coroutine
def post(self):
_file = self.request.files['upload'][0]
_filename = _file["filename"].split(".")[0]
_extension = _file["filename"].split(".")[1]
_mobile = '12345678901'
upload_file = UploadFile(mobile=_mobile,
body=_file['body'],
filename=_filename,
extension=_extension)
with self.make_session() as session:
session.add(upload_file)
yield as_future(session.commit())
`
and the error log is
File "/usr/local/lib/python3.5/dist-packages/tornado/web.py", line 1592, in _execute
result = yield result
File "/usr/local/lib/python3.5/dist-packages/tornado/gen.py", line 1133, in run
value = future.result()
File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
raise self._exception
File "/usr/local/lib/python3.5/dist-packages/tornado/gen.py", line 1141, in run
yielded = self.gen.throw(*exc_info)
File "/srv/www/api/upload_file.py", line 20, in post
yield as_future(session.commit())
File "/usr/local/lib/python3.5/dist-packages/tornado/gen.py", line 1133, in run
value = future.result()
File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
raise self._exception
File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
result = self.fn(*self.args, **self.kwargs)
TypeError: 'NoneType' object is not callable

i have solved the problem, just remove the () after commit

yield as_future(session.commit)

session.commit doesn't return a query object. as_future only works on query objects, hence the exception you receved.