Update references to loop-invariant-global-usage

Looks like these were missed as part of #27
Was seeing the following error while re-testing #26 at 4bf47e0
$ perflint repro.py
Exception on node <Name.x l.5 at 0x10efcae90> in file '/tmp/repro.py'
Traceback (most recent call last):
  File "/tmp/venv-710/lib/python3.10/site-packages/pylint/utils/ast_walker.py", line 73, in walk
    callback(astroid)
  File "/tmp/venv-710/lib/python3.10/site-packages/perflint/for_loop_checker.py", line 324, in visit_name
    self.add_message("loop-invariant-global-usage", node=node)
  File "/tmp/venv-710/lib/python3.10/site-packages/pylint/checkers/base_checker.py", line 112, in add_message
    self.linter.add_message(
  File "/tmp/venv-710/lib/python3.10/site-packages/pylint/lint/pylinter.py", line 1629, in add_message
    message_definitions = self.msgs_store.get_message_definitions(msgid)
  File "/tmp/venv-710/lib/python3.10/site-packages/pylint/message/message_definition_store.py", line 65, in get_message_definitions
    for m in self.message_id_store.get_active_msgids(msgid_or_symbol)
  File "/tmp/venv-710/lib/python3.10/site-packages/pylint/message/message_id_store.py", line 130, in get_active_msgids
    raise UnknownMessageError(error_msg)
pylint.exceptions.UnknownMessageError: No such message id or symbol 'loop-invariant-global-usage'.

Ouch. Nice spot. thanks


reworded and typo corrected

an---> and

Reworded also, first user should be able to update the web apps then redeploy.

Not redeploy and update..
Request you to merge the PR.
Thanks

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@aftab10662 : Thanks for your contribution to the Azure documentation! The author, @cephalin, has been notified to review your proposed change.

@aftab10662 Thanks for your improvement! #sign-off


fix images with no alt text

fixes sirthias/pegdown#114

Thanks for this patch!
Are we missing a test that should have been red before and is now green?

Just looking into it now but it looks like there isn't a test with an empty alt?

Also, it looks like the recent merge of 1bffa457ed20177c5d766109b4c469831bbb2282 broke some tests so I'll try to get those sorted out.

All Tests passed when I ran them. Sometimes the first run of sbt test after source changes would report failed tests, however these would disappear when sbt test was invoked a second time.

thanks @vsch, I meant that the tests are broken on my branch :)  Your fixes look good.

@sirthias not sure if there's a more clever way of fixing this. Lemme know if you have any better thoughts. Also added an empty image alt test.

Thanks! Looks good!


[BUG]  Address TextMeshPro dependency in Unity 2023

Context

TextMeshPro has been moved into<EMAIL_ADDRESS>Unity 2023 requires<EMAIL_ADDRESS><EMAIL_ADDRESS>is not available in Unity 2021 or 2022
MRTK3 depends on com.unity.textmeshpro v3.0.6  (see package.json file)
MRTK3 needs to work across all LTS versions (Unity 2021, 2022, and eventually 2023)

Because of 1-4 we are unable to satisfy 5.   If we leave  package.json as is, Unity 2023 complains and gives "duplicate type definitions" errors. If we remove  com.unity.textmeshpro from  package.json, Unity 2021 and Unity 2022 complains and gives "unknown type" errors.
In the past, when Unity has moved classes or functionality to a new package, Unity has made "facade" packages and/or classes.  These "facades" redirect to the new package with the new implementations.  Can Unity do the same and create a facade for com.unity.textmeshpro in Unity 2023?
Or can Unity recommend a solution for this problem?  So, we can maintain one MRTK3 version that works across 2021, 2022 and 2023.

Packages with dependency: accessibility, data, uxcomponents, uxcore

@marlenaklein-msft please add the "package: *" labels for all impacted packages.

Need to determine why textmesh pro still gets imported after removing our dependency.  @AMollis  please update.

Updated description and title.   This requires a fix from Unity

We are currently working on a solution for this.
If developers are blocked, they can:

Use the MRTK3 packages directly from the repository.
Replace all references to com.unity.textmeshpro in package.json files with<EMAIL_ADDRESS>Update project manifest.json to remove reference to com.unity.textmeshpro

@AMollis
Something interesting happened in Unity 2023.2.7f1:
https://unity.com/releases/editor/whats-new/2023.2.7#release-notes

TextMeshPro: Introduce a shim version of TextMesh Pro for packages that had a hard dependency on it. (UUM-58068)

I updated our project from 2023.1.16f1 to 2023.2.7f1 and got no issue at all, no need to edit MRTK's package.json, nothing to do.

Thanks for verifying @anonymous2585 ...
We've been working on fix with Unity here.  I'm glad it's working.   Resolving issue as fixed externally


[YUNIKORN-2096] Update e2e README

What is this PR for?
Add how to run a specified e2e test in the e2e README
What type of PR is it?

[x] - Improvement

What is the Jira issue?
YUNIKORN-2096

@wusamzong @FrankYang0529 @pbacsko  please take a look 🙏


Learning Content: Using CesiumSunSky

Create a tutorial that explains how CesiumSunSky differs from Unreal's SunSky actor, how it interacts with the georeference, and how to modify it.
This page should emphasize how the sunsky actor ensures geographically accurate sun position. If possible, include examples of situations where this is important.
This tutorial should additionally cover:

Info on all parts of the SunSky blueprint, but especially:

Shadow draw distance
Light intensity
SkyAtmosphere - why it's important, the settings on it that we changed, how to troubleshoot common issues like black lines on horizon, patchy artefacts from space

Setting the time zone to make sure the time slider matches up, explain how time zone must be set manually

How to troubleshoot common issues like black lines on horizon

Is there a short-term resolution for this before the documentation gets updated? I've noticed this happening a couple times when I start a new level and not sure how to fix it yet. Any insight would be helpful.

Hi @robhawkes,
I haven't been seeing the black lines lately on my end, but if you are I would love to know more so I can properly address it. Could you give me a little more information or steps to reproduce the black lines? Is it right at the start of a new level when you add the CesiumSunSky actor? Do they persist after connecting the CesiumSunSky actor to the georeference?

Do they persist after connecting the CesiumSunSky actor to the georeference?

That was it — I hadn't set the reference to the sky actor! I must have previously been using a level that already had that connected, and didn't realise I needed to manually connect them together when I was creating my own levels from scratch.

Great to hear! There is now a live tutorial on CesiumSunSky on the site - I should actually close this issue now. If you have any other questions regarding CesiumSunSky, though, please let me know and I can address them here or as an update to the tutorial page.


(テスト) WEKO処理修正

Purpose

Changes

QA Notes

Does this change require a data migration? If so, what data will we migrate?
What is the level of risk?

Any permissions code touched?
Is this an additive or subtractive change, other?

How can QA verify? (Through UI, API, AdminApp or AdminAdminApp?)

If verifying through API, what's the new version? Please include the endpoints in PR notes or Dev docs.

What features or workflows might this change impact?
How will this impact performance?
-->

Documentation

Side Effects

Ticket

テスト完了のため閉じます。


testing  issue mail forwarding

this should generate an email

and this should generate anouther email (this time also tomyself (as I'm watching)

And this should send notification

Frank, could you open another issue. I've got no mail so far.

and another one ... for me they come through correctly ... very strange

A test from someone who isn't Frank

The SPF record has been removed. This is a personal test, if Google Mail shows a different classification of the received mail.

In case it helps, my gmail shows pass for spf, dkim and dmark I could send details if required but summary is


Entity _options dates

When I am trying to add $_options = ['cast' => []]; to one of my class which contains entity  and additionally  I don't wanna  define  dates configuration...  CI is returning an exception and made me to define it (dates cfg for entity):
Type:        ErrorException Message:     Undefined index: dates Filename:    /home/zdamy/site/system/Entity.php Line Number: 176
(I am a huge fan of writing CLI application but I am sure the same is happening during normal requests).

I know what is causing this error but you should prevent it if in the documentation there is the example like this:
class User extends Entity
{
protected $id;
protected $name;        // Represents a full name now
protected $email;
protected $password;
protected $created_at;
protected $updated_at;
protected $_options = [
    'datamap' => [
        'full_name' => 'name'
    ]
];

}
/sorry there is no way to format code in this form from mobile device.

Just changing the docs is the way to go here :)


About label tool

Great work！
But a little question: Do you know any tools for labeling images of rotating bboxes using eight params(namely four points of bbox)?
Thanks~

labelme @oukohou


[Removal]:

Hotspot Name
alert-lava-moose
Hotspot b58 Address
11eWZoGbmeEAeCnVn9wgYQpo62Y5PfLyHDhAXZgKk8grSLRgrmZ
Discord Handle
No response
Hotspot Manufacturer
bobcat
Removal Reason
i just bought from ebay, and is blacklisted
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information
No response

Proof of purchase after the blacklist date please? If you do not wish to post it here we can handle privately via Discord?

how we can talk on discord?


npm start does not detect modifications in src/*/node_modules

Is this a bug report?
yes
Can you also reproduce the problem with npm 4.x?
yes
Which terms did you search for in User Guide?
node_modules, watch, absolute, relative, import, webpack
(also searched the change logs)
Environment

node -v: v8.6.0
npm -v: 4.6.1 and 5.4.2
yarn --version (if you use Yarn): ---
npm ls react-scripts (if you haven’t ejected):
└──<EMAIL_ADDRESS>
(and I also tried<EMAIL_ADDRESS>and<EMAIL_ADDRESS>versions. There is no problem with 0.9.5)

Operating system: Windows 8.1 and Ubuntu xenial
Browser and version (if relevant): not relevant

Steps to Reproduce

create-react-app test
create file at test/src/node_modules/Test.js with: console.log("Test.js imported");
at test/src/index.js: import "Test.js";
cd test && npm start
modify test/src/node_modules/Test.js;
modify test/src/index.js;

Expected Behavior
at step 5. the project should recompile.
at step 6. the project should recompile with modified Test.js file.
This was the behavior with<EMAIL_ADDRESS>Downgrading fixes the issue.
Braking change was not reported on 0.9.5 -> 1.0.0.
Actual Behavior
at step 5. the project is not recompiled.
at step 6. the project is recompiled, but without Test.js modifications.
The project compiles without errors initially on npm start.
Reproducible Demo
create-react-app test
cd test
mkdir src/node_modules
echo "console.log("Test.js imported");" > src/node_modules/Test.js
mv src/index.js src/index.js.old
echo "import "Test.js";" > src/index.js
cat src/index.js.old >> src/index.js
rm src/index.js.old
npm start
(then make modifications on Test.js file)

https://github.com/facebookincubator/create-react-app/pull/2760

We'll need someone to take over #2760 and champion it. 😄

Should be fixed in<EMAIL_ADDRESS>https://github.com/facebookincubator/create-react-app/releases/tag/v1.0.15


Issue and Pull Request templates

https://github.com/blog/2111-issue-and-pull-request-templates
Will check this out for making the guidelines more prominent.

I would like to take this up.
Please assign me this.

Thanks for the offer, @yash14123, but I plan to do this one to keep our messaging consistent.

We're not moving forward with this for now as discussed in #3637.


inactiveBorderColorList property has no effect

Hi there,
The inactiveBorderColorList property of the ChipList has no effect on the inactive border color. This leaves the ChipList with a default grey border.  Please kindly look into this.
Much thanks, great package btw ❤
Mike

Hey Michael!
Thank you for your kind words :))
Is there an example you can provide to reproduce this problem? It seems to work fine as of now
ChipList(
                listOfChipNames: _dogeNames,
                activeBgColorList: [Theme.of(context).primaryColor],
                inactiveBgColorList: const [Colors.white],
                activeTextColorList: const [Colors.white],
                inactiveTextColorList: [Theme.of(context).primaryColor],
                listOfChipIndicesCurrentlySelected: [0],
                activeBorderColorList: [Theme.of(context).primaryColor],
                inactiveBorderColorList: [Colors.brown, Colors.pink, Colors.teal],
              ),

Results in

Thanks for the swift reply.
Interesting 🤔, perhaps I confused it's usage. I've tested the code snippet you provided and it works as expected.
But isn't there a way to set the color once for all "inactiveBorderColor". Because in my scenario I render the list of chips based on x length of data returned from api, and I do not want to create x length of inactiveBorderColors.
Thanks.

You can provide a list of length 1, it'll use the colour for all chips.
For example:
[Colors.grey]
Every inactive chip will be grey.

Doesn't seem to work. Please check out my code snippet below. inactiveBorderColorList is set to Colors.green but result remains default grey:
               List<String> hashtags = ['Branding', 'Life Style', 'Movies', 'Concert', 'Branding];

                ChipList(
                  shouldWrap: true,
                  supportsMultiSelect: true,
                  showCheckmark: false,
                  listOfChipNames: hashtags,
                  scrollPhysics: const BouncingScrollPhysics(),
                  activeBgColorList: [Colors.red.withOpacity(0.15)],
                  inactiveBgColorList: const [Color(0xffF7F8F9)],
                  listOfChipIndicesCurrentlySeclected: [2],
                  activeBorderColorList: const [Colors.transparent],
                  inactiveBorderColorList: [Colors.green],
                  activeTextColorList: [Colors.red],
                  inactiveTextColorList: [Color(0xff556575)],
                  style: TextStyle(fontWeight: FontWeight.w400, ),
                  borderRadiiList: const [6],
                ),

Result:

I'll check and let you know

Thanks 💯

I'll check and let you know

Hi there 👋.
Any updates on this ?

Very weird...
I pasted in your code

and I get the expected result

Very weird 🤔.
Now I'm officially out of ideas on what the issue could be. At least now I know the issue is not from the package.😅
I guess I'd run some more tests and debugging. ❤🩹
Much thanks 🔥.


[Enhancement] Rename CPU architectures for deb and rpm packages

Describe the bug
The CPU architecture was renamed for archives in opensearch-project/OpenSearch#612 to keep it consistent with plugins and trends on google.
But the deb and rpm packages used amd64 and x86_64.
How do we like to rename them?

Possible issues if we directly rename inside core:
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/1401

@saratvemulapalli We dont really need to do this now as the final release artifact will be rename to the correct name anyway.

Cool. I'll close this one for now. Feel free to re-open if we need it.

I'm reading this issue and it sounds like OpenSearch generates opensearch-min-2.0.0-SNAPSHOT.x86_64.rpm and it should really be opensearch-min-2.0.0-SNAPSHOT.arm64.rpm. To work around this we are doing renaming in opensearch-build? As @saratvemulapalli points out in https://github.com/opensearch-project/OpenSearch/issues/2598#issuecomment-1075590534, we should make this change here and not rename files later in distribution scripts. @peterzhuamazon WDYT? Want to give it a try?

Hi @dblock,
It should be x64 not arm64 for x86_64, tho it really does not matter as that is not the product we will release anyway.
Plus the manifest will also use the existing name of the file so it will not mess up anything is the build.
Thanks.

Also @dblock I fear the changes might somehow cause the min rpm to not able to install like in OSD, and that is why I am very hesitate to even change this, see:

https://github.com/opensearch-project/OpenSearch-Dashboards/issues/1401


Add support for searching for Jobs by query

Add methods to JenkinsServer that support finding matching jobs:
public Map<String, Job> findJobsByTitle(String partialTitle) { ... }
public List<String> findJobTitlesByTitle(String partialTitle) { ... }

What about offering a pull request ?


How to add brightness and focus controls?

How can I add brightness and focus controls as shown here:

Could not find any way to adjust the brightness and focus.

Any updates?

Is there any way to control exposure (brightness) now?


Width of TextInput is wrong with custom font on Android

🐛 Bug Report
The width of TextInput on Android is calculated wrong.
Looks like it takes the width of the default font.
See screenshots below:

With default font (looks like it should be):

To Reproduce
Add a Text component after TextInput (in our case Text with € sign).
Expected Behavior
TextInput should take the width of its content. The distance between TextInput content and the € sign should always be the same. Content shouldn't be cut.
Code Example
import React from 'react';
import { StyleSheet, Text, View, TextInput } from 'react-native';

export default class Test extends React.Component {
  render() {
    return (
      <View style={styles.container}>
        <TextInput
          style={styles.textInput}
          underlineColorAndroid='transparent'
          placeholder='0'
        />
        <Text style={styles.textInput}> €</Text>
      </View>
    );
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    flexDirection: 'row',
  },
  textInput: {
     fontFamily: 'customFont'
  }
});

Environment
React Native Environment Info:
    System:
      OS: macOS 10.14.3
      CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
      Memory: 137.99 MB / 16.00 GB
      Shell: 5.3 - /bin/zsh
    Binaries:
      Node: 10.6.0 - ~/.nvm/versions/node/v10.6.0/bin/node
      Yarn: 1.9.4 - /usr/local/bin/yarn
      npm: 6.1.0 - ~/.nvm/versions/node/v10.6.0/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
      Android SDK:
        API Levels: 23, 24, 25, 26, 27, 28
        Build Tools: 23.0.1, 23.0.3, 25.0.2, 27.0.3, 28.0.2, 28.0.3
        System Images: android-26 | Google Play Intel x86 Atom, android-28 | Intel x86 Atom, android-28 | Intel x86 Atom_64, android-28 | Google APIs Intel x86 Atom_64, android-28 | Google Play Intel x86 Atom
    IDEs:
      Android Studio: 3.2 AI-181.55<IP_ADDRESS>14246
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.6.3 => 16.6.3
      react-native: 0.58.4 => 0.58.4

Also I sight what TextInput on android have min height related with placeholder length. Maybe this is that case. You can tray set empty placeholder while TextInput not empty as a temporary solution.

Also I sight what TextInput on android have min height related with placeholder length. Maybe this is that case. You can tray set empty placeholder while TextInput not empty as a temporary solution.

@Bardimaswift There is another issue with the placeholder, yes. But this one is not related to placeholder. I just updated the screenshots to make it more visible. If I type 1111111 the distance to€ increases. If I type 00000 digits became cut. But with a default font everything looks fine.

@temhota  Any solution ?

Not sure if this will help.  I was having the same issue with a custom font and textinput.
I had to add the '.ttf' to the end of the custom font family (in your case, textInput: {
fontFamily: 'customFont.ttf'
}) and this resolved my issue.  I am still looking at the root cause or why this would have worked, but in case you need a temporary solution, there you go.

Was there an alternative resolution to this aside from adding '.ttf' to the font family? @temhota

Actually, the .ttf just caused all of my custom fonts to disappear and go to the default font of Roboto. Thank you for asking as I forgot to update this accordingly.
To fix this, I actually embedded a  within the  to display the actual value - the system was then able to calculate and I had no more truncation.
So where I formerly had:
                  key={item.itemDisplay}
                  value={item.itemDisplay}
                  style={index === selectedItem ? itemSelectedStyle : itemStyle}
                  caretHidden={true}
                  spellCheck={false}
                  autoCorrect={false}
                  onChangeText={input => this.OnChangeItem(input, item.itemDisplay)}
                  onFocus={() => onFocus(index)}
                  onBlur={onBlur}
                  ref={ref => (this.lastInputRef = ref)}
                />

It became:
                      caretHidden={true}
                      spellCheck={false}
                      autoCorrect={false}
                      onChangeText={input => this.OnChangeItem(input, item.itemDisplay)}
                      onFocus={() => onFocus(index)}
                      onBlur={onBlur}
                      style={index === selectedItem ? itemTextSelectedStyle : itemTextStyle}
                      ref={ref => (this.lastInputRef = ref)}
                    >
                      <Text>{item.itemDisplay}</Text>
                    </TextInput>```

This is really a big problem of TextInput. We are developing the mobile application of a known bank using react native and have custom fonts. We are trying to implement an input Amount component which is always horizontally centralized and font-decreasing after a number character length. Also integer part and the decimal part should be independently changable. When we put the two TextInputs, the seperator Text between them and the currency type indicator Text at the end in flex direction "row". TextInputs can't calculate their character widths correctly on Android and distances between components seem too much if we write thin characters like 1 and too less when we write thick characters like 0 or 8 (even they go to the top of the adjacent Text).

Hey everybody,
We cant handle character size İnci’den textinput without monospace fonts. We have a workaround about this situation but not working effectively and when we change our font set, our scripts arent covered the character size for new font set. We r waiting for this issue with four eyes.

Hey again,
The problem solved @0.59.10 version a hundred percent but maybe after >=0.59 working according to that pull request merged (just a guess btw) https://github.com/facebook/react-native/commit/d2153fc
Please try @0.59 first if you can and then @0.59.10 @temhota


Failure attempting to install vyper >v0.2.8

Environment information

vvm Version: 0.1.0
Python Version: 3.9

What was wrong?
Attempting to install vyper v0.2.10 returns VyperInstallationError: Vyper binary not available for v0.2.10. Attempting to install an older version (v0.2.3) works fine
In [31]: vvm.install.install_vyper("0.2.10", show_progress=True)
---------------------------------------------------------------------------
StopIteration                             Traceback (most recent call last)
/opt/conda/envs/theta/lib/python3.9/site-packages/vvm/install.py in install_vyper(version, show_progress, vvm_binary_path, headers)
    250             release = next(i for i in data if i["tag_name"] == f"v{version}")
--> 251             asset = next(i for i in release["assets"] if _get_os_name() in i["name"])
    252         except StopIteration:

StopIteration:

During handling of the above exception, another exception occurred:

VyperInstallationError                    Traceback (most recent call last)
<ipython-input-31-113d04fb47a1> in <module>
----> 1 vvm.install.install_vyper("0.2.10", show_progress=True)

/opt/conda/envs/theta/lib/python3.9/site-packages/vvm/install.py in install_vyper(version, show_progress, vvm_binary_path, headers)
    251             asset = next(i for i in release["assets"] if _get_os_name() in i["name"])
    252         except StopIteration:
--> 253             raise VyperInstallationError(f"Vyper binary not available for v{version}")
    254
    255         install_path = get_vvm_install_folder(vvm_binary_path).joinpath(f"vyper-{version}")

VyperInstallationError: Vyper binary not available for v0.2.10

How can it be fixed?
The "assets" key is not available in the json dict from github releases starting from v0.2.9. This line raises the exception: https://github.com/vyperlang/vvm/blob/9d1ad9c5986d3b5e8e82859b04636d02e28bd7f1/vvm/install.py#L251
I would also suggest daily CI builds to prevent this from happening again in the future without notice

Vyper 0.2.10 was pulled due to a runtime error we discovered shortly after releasing. So this isn't an issue - the version just doesn't exist.

Still getting the same issue with vyper v0.2.11:
In [8]: vvm.install_vyper("0.2.11", show_progress=True)
---------------------------------------------------------------------------
StopIteration                             Traceback (most recent call last)
/opt/conda/envs/theta/lib/python3.9/site-packages/vvm/install.py in install_vyper(version, show_progress, vvm_binary_path, headers)
    250             release = next(i for i in data if i["tag_name"] == f"v{version}")
--> 251             asset = next(i for i in release["assets"] if _get_os_name() in i["name"])
    252         except StopIteration:

StopIteration:

During handling of the above exception, another exception occurred:

VyperInstallationError                    Traceback (most recent call last)
<ipython-input-8-3f383659896f> in <module>
----> 1 vvm.install_vyper("0.2.11", show_progress=True)

/opt/conda/envs/theta/lib/python3.9/site-packages/vvm/install.py in install_vyper(version, show_progress, vvm_binary_path, headers)
    251             asset = next(i for i in release["assets"] if _get_os_name() in i["name"])
    252         except StopIteration:
--> 253             raise VyperInstallationError(f"Vyper binary not available for v{version}")
    254
    255         install_path = get_vvm_install_folder(vvm_binary_path).joinpath(f"vyper-{version}")

VyperInstallationError: Vyper binary not available for v0.2.11

Was this version also pulled?

@spinoch sorry, still being worked on. We're adding the missing binaries today.

@fubuloubu I guess the files on conda-forge are outdated then: https://anaconda.org/conda-forge/vyper/files

I guess you can ignore my previous comment, I assumed 0.2.11 was pulled and updated from pypi, but it looks like the pypi and conda-forge hashes match

Awesome, so this all works then?

The vvm command works. Didn't do a thorough check if the conda-forge binaries are alright but if there was no change in the pypi tar for 0.2.11 I guess that's fine also


IPK certificate export using thales HSM

Dear,
We have generate issuer public key using hsm and request IPK certificate from Mastercard. And mastercard provide us IPK certificate which we import through perso software.
Now we want to export RSA certificate through Thales 9000 HSM.
Could you please suggest us how to export it.

To generate IPK we use EI command. But after getting certificate from Mastercard/VISA which command we could use to export that IPK certificate.


Work with Azure Service Bus?

This package works with azure service bus? Have support for azure connector?

At the moment, there is no such transport.


Theme toggle

Using the combination of  Android theme and Dark mode preference results to both the names of the conference to choose and its background being dark making it hard for a user to realize which conference he/she is choosing.
Suggestion to correct this:
Make the content(names of conferences) in Choose Conference to be of a brighter color(White) while the background remains dark/black.

Hi @joreilly
How can I join the confetti channel on slack?

Are you setup to use Kotlinlang (https://twitter.com/kotlin/status/1022114553302331392?s=20)....if so, then just search for Confetti on that

ok thanks!!


The type of lookup does not exist in fieldTypes

Versions

@kintone/dts-gen 6.1.16

Current Behavior

The type of lookup does not exist in fieldTypes.
Currently, specifying a lookup property for a lookup field results in a type error.
https://github.com/kintone/js-sdk/blob/master/packages/dts-gen/kintone.d.ts#L227
Expected Behavior

We want to specify the lookup property like this.
record[fieldCode].lookup = 'CLEAR';
record[fieldCode].lookup = true;

@kosuke-k
Thank you for your reporting.
We are checking this issue and will reply to you soon.

@hung-cybo
Thanks for your reply and suggested workaround.
I understand that it is not yet supported.
I hope this will be improved in the future.

@kosuke-k
Thank you for your understanding.
I will close this issue.
If you still have any concerns, please post a new issue.
Have a nice day.


Integration test using content-cache-backends-config charm

Overview
After moving the content-cache-backends-config charm to this repo, the integration test can be done with the packing the content-cache-backends-config charm in the repo rather than download from CharmHub.
Rationale
This allows testing of changes in content-cache and content-cache-backends-config charm together.
Checklist

[x] The charm style guide was applied
[x] The contributing guide was applied
[x] The changes are compliant with ISD054 - Managing Charm Complexity
[x] The documentation is generated using src-docs
[x] The documentation for charmhub is updated
[x] The PR is tagged with appropriate label (urgent, trivial, complex)

Converted back to draft due to some workflow-related issues.


Fix iOS Image Download Bug

TLDR: This PR switches from downloadjs to file-saver to enabled shared images for ios support.
downloadjs does not support ios

Devices without file systems like iPhone, iPad, Wii, et al. have nowhere to save the file to, sorry.

Luckily, file-saver does and it is more widely used.
I tested this on a repl.it instance. I would've gone with glitch.com instead but it has a 128MB disk limit, and this app is 500MB of node components. The thief-prevention screen should still remain (based btw), ~feel free to fork it on replit, or clone it locally and Contend With Npm to see the change working.~ Just noticed the vercel integration! Awesome! Ill be sending it to my ios friends to use for now!
Three of my friends and I have been playing colordle since January and sending the pictures + descriptions over discord. Thanks so much for building this!!! i

@hobgoblina Thanm you for merging!!! It feels good to make a change in something I enjoy, you have made My day 😁
I wish you a wonderful day, week, month, and year!!
PS: Here is my first saved from palette from the live colordle site 🎉❤
Guessed colors:
1: Evening Primrose
2: Flamingo Fury
3: Altdorf Guard Blue
4: Caribbean Blue
5: Zima Blue


spotify-qt: 3.9 -> 3.11

Description of changes

Things done

Updated spotify-qt to version 3.11

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

first time maintainer so let me know if i can change anything

Lgtm. Consider adding yourself to this package's maintainers if you use this package and are interested in maintaining it.

Thanks! I don't use it. I just noticed it was a couple of years out of date and thought it should be updated

nixpkgs-review built this successfully, so do what you will with that information

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

1 package built:

spotify-qt


Fix home timeline pagination 🤞

My theory is that one or both of these is happening:

Filtering on subject_post_id OR subject_post_interaction may cause extra timeline events because type is not taken into account. These could be duplicates, or be outside the requested range, etc.
Ordering by TimelineEvent.id is not a perfect proxy for post or interaction IDs

I think using the appropriate ID based on event type is the way to go, and it seems to be working well on my (tiny) instance. Not sure if it's worth calculating/storing this event_id when creating timeline events.

This may or may not perform badly - I am going to merge it in and then run it on takahe.social to see.


Correctly pass dialog field values through request

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1613935
This is two small changes.
with_indifferent_access added to dialog field lookup
https://github.com/ManageIQ/manageiq/pull/17642 broke our setting values in fields because previously we were setting the names to strings in the lookup
submit_workflow utilized in requests
The submit_workflow option is used to funnel values correctly through requests.

@miq-bot add_label bug
@miq-bot assign @gmcculloug
@miq-bot add_reviewer @eclarizio

@miq-bot add_label blocker

@d-m-u if this will be able to be backported, can you add the gaprindashvili/yes label

@miq-bot remove_label wip

@miq-bot add_label gaprindashvili/yes


[docs] update cloudflare adapter readmes

update the readmes without other reformatting
Closes https://github.com/sveltejs/kit/pull/2820

Does not close 2821


API call for get /api/check/metric_I1 returns null for metric property

When I use the API get call for checking the I1 metric, the return object contains null for the "metric" property instead of the name of the metric as it is for all the other metrics. (Examplaratory return object below)
{   "metric": null,   "score": "1",   "target_uri": "https://doi.org/10.48366/r281265",   "eval_time": "0:00:00.009154",   "recommendation": "\n            You should provide discoverability oriented metadata with one of the following properties:  \n        \n                    dct:title\n                    dct:description\n                    dcat:accessURL\n                    dcat:downloadURL\n                    dcat:endpointDescription\n                    dcat:endpointURL\n                \n                        ",   "comment": "INFO - Evaluating metrics Machine readable format\nINFO - Checking that at least one of the access policy properties is found in metadata:\n\n                    dct:title\n                    dct:description\n                    dcat:accessURL\n                    dcat:downloadURL\n                    dcat:endpointDescription\n                    dcat:endpointURL\n                \nINFO - None of the discoverability properties were found in metadata !\nINFO - Checking if data is structured, looking for at least one RDF triple...\nINFO - 18 RDF triples were found, thus data is in a well structured graph format\n" }

Same observation for
curl -X 'GET' \
  'https://fair-checker.france-bioinformatique.fr/api/check/metrics_all?url=http%3A%2F%2Fns.inria.fr%2Fmeteo%2Fwekg-mf' \
  -H 'accept: application/json'

Obtained:
...
  {
    "metric": null,
    "score": "2",
    "target_uri": "http://ns.inria.fr/meteo/wekg-mf",
    "eval_time": "0:00:00.005862",
    "recommendation": "No recommendation, metric validated",
    "comment": "INFO - Evaluating metrics Machine readable format\nINFO - Checking that at least one of the access policy properties is found in metadata:\n\n                    dct:title\n                    dct:description\n                    dcat:accessURL\n                    dcat:downloadURL\n                    dcat:endpointDescription\n                    dcat:endpointURL\n                \nINFO - At least one of the discoverability properties was found in metadata !\n"
  },
...

Thanks for your feedbacks ! This was a silly bug, it should be fixed in #200

This change will be deployed in the next release (within few days), I keep open the issue.

Indeed, it's fixed, tanks again for your feedback.


Updating the app maintains the previous version entry in windows "Programs and Features"

I had ScreenToGif 2.8.0 installed via ScreenToGif.2.28.Setup.msi installed in win10 x64 20h2.
Today, I wanted to update to the new version, 2.8.1 .
So I tried:

downloading ScreenToGif.2.28.1.Setup and running it,
(uninstalling+reinstalling 2.8.0 and) updating via the program itself,

and in both cases,
in windows "Programs and Features": the old 2.8.0 entry remained and an additional 2.8.1  created:

This shouldn't occur because the updated program still has the same ScreenToGif directory:

Thank you

Did you install with the option to "all users" ticked or not?

There's only one local account in my Windows 10 installation, Administrator [screenshot]
And, during ScreenToGif install/update,
the only option I got related to "all users" was Add the shortcut to all users, and yes, it was ticked (already, by default) :

screenshots

Ok, this issue is related to a flag (ALLUSERS) used in the installer.
When the option to install the app to all users is ticked and later in an update is not ticked, it is treated as another separated app.
So it creates another entry in the App list.
One way to go would be to run a custom action removing the current version before installing it on top.

I'm going to close this issue because I'm tracking it via #721.
I have a working solution for stopping this issue, but you'll need to uninstall the broken version before installing the new version.

I managed to make it to work, you don't need to uninstall manually the previous version.
The installer detects that a previous version was installed on the same folder, but with a different access mode (per-user / per-machine) and it uninstalls before installing the new version.

Thank you for fixing this.


custom-lovelace: bignumber-card.js throttles cpu to 100%

Hi,
I love your script, thank you for providing such knowledge to the community. I am just starting up with home assistant and I have stumbled upon your bignumber-card example. The cards work fine and I love the customisation options, however I cannot really use them in a 24/7 display as it keeps the cpu to 100% all time.
Is it possible that the script enters in a loop somehow? I am not a coder myself, being at most, a merely code tinker.
All the best, and thanks again for this free knowledge 👍

Nvm, please close this thread. Safari seems to have a problem with it as it runs it inside a container. Chrome works just fine.
Thanks again for this awesome script.

@psecto Did you find the root cause of your 100% CPU? Might help other users if you at mentioned also the solution to your problem.

 Javascript code when running in Safari, appears to run into its own app outside of the web browser itself. I don't think it's your script since I also found other javascript intensive pages which are doing the same.

Running Chrome on Mac OS poses no issues.
Best,
psecto

Thank you very much for the detailed comment. Very helpful to other users <3


⚠ Jenkins Plugins Artifactory has degraded performance

In 69f09ac, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1200 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 0ef24f2 after 33 minutes.


[Enhancement][Cherry-Pick] Set Hive string type to a larger length and set null for parse fails column

What type of PR is this：

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

Which issues of this PR fixes ：

Cherry-Pick from https://github.com/StarRocks/starrocks/pull/12986
Problem Summary(Required) ：

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

run starrocks_fe_unittest


Build API Endpoints For F@H/FLDC specific Data

FLDC stats server information as an API + FLDC distribution information
Spec and implementation - 8SP

closing this for now unless it is requested again


IEM Reanalysis too coarse for WEPP usage in the mountains

At the team call today, a comparison was shown with DEP's 0.01x0.01 climate files performing very poorly in the mountains.  At issue was temperatures being too warm and not producing enough snow within the WEPP algorithm for snow generation.  It seems clear to me that the 0.125x0.125 IEMRE domain is too coarse for provision of temperature information.  I have great gnashing of teeth at the moment resolving how I wish to proceed. Step 1 is creating this issue :laughing:

Some debugging cruft here for our CLI file of interest 109.31x033.65.cli
crude yearly means

tmax
tmin
rad
wvl
wdir
tdew
maxr
bpcount
pcpn
rfactor

2007
25.0567
8.20164
511.57
2.87233
0
-0.456986
2.42827
1.74521
2.3314
3.59376

2008
23.8175
7.2847
520.505
3.49973
0
-2.05984
3.72055
1.81421
2.73123
4.46316

2009
24.5534
7.40466
510.373
3.38521
0
-1.51562
1.5854
1.2274
1.50123
1.56463

2010
22.8321
7.19178
523.696
3.46795
0
-0.691233
2.66487
1.54795
2.30995
6.71424

2011
23.5118
6.56219
533.822
3.71068
0
-2.41068
1.42579
1.28767
1.66195
0.982563

2012
25.1112
6.47705
530.781
3.2194
0
-2.23361
1.75945
1.0765
1.47257
1.75355

2013
23.614
5.88904
517.025
3.35068
0
-2.05123
2.32608
1.43288
1.95121
3.34223

2014
24.6447
6.62959
493.271
3.53178
0
-1.56082
2.83931
1.19452
1.66479
4.46618

2015
23.909
6.57205
472.082
3.32
0
1.22795
2.24426
1.50685
2.10649
2.16183

2016
22.2022
4.74126
498.003
3.55383
0
-1.51885
1.44507
1.21311
1.64489
1.46649

2017
23.2589
5.82986
495.323
3.75041
0
-2.46849
1.90286
1.08219
1.40671
2.10515

2018
22.6907
4.59973
492.184
3.70904
0
-2.54548
1.94569
1.20822
1.71553
2.1666

2019
20.9392
5.12904
487.362
3.79068
0
-1.8537
2.61769
1.66849
2.45205
3.90069

2020
22.9153
5.69508
510.09
4.12896
0
-2.85847
1.25308
0.863388
1.07784
1.05372

2021
22.1214
5.58
488.542
4.12329
0
-2.05534
2.86936
1.35616
1.93458
3.33954

2022
21.3655
5.38055
488.619
4.04356
0
-1.53534
3.19398
1.29863
1.8889
2.78217

2023
21.5907
4.34986
508.781
4.0874
0
-2.39863
2.30976
1.05753
1.59189
1.8498

2024
22.8251
5.63033
507.473
4.10765
0
-2.66585
1.73132
1
1.33801
1.38571

I think 2015 was a year of interest, for prism this is grid cell i=231, j=377 , PRISM has an average high of 16.2C and low of 1.6C, yikes.  It also has 656 mm of precip, whereas DEP has 605.9, so that's not horrible, like temperature.


Fix problem of XML parsing empty html string. Fixes #2752

Raise an exception when XML.parse_html called with an empty string.
This is a small PR, probably self-explanatory. Please give any suggestion of better place to put the test or make the check.
Thank you!

Thanks! Can you add the same fix for self.parse? It happens there too.

@asterite sure, doing right now! :)
ps: I didn't add because I thought was only on parse_html, my bad 😉

@asterite the second test (for parse) I added on xml_spec.cr just because makes more sense to me once it's the library name, is it ok?

@fernandes Thank you! 🌟


Collect Cluster Level VMI metrics from controller not handler.

The kubevirt_vmi_phase_count metric is being collected by virt-handler, but virt-handler only sees the VMI once the VMI is marked as scheduled... So we have no visibility into VMIs stuck in Pending, Scheduling, or VMIs that fail before the virt-handler handoff for any reason.
This PR refactors our VMI metrics into two packages, a domain level pkg (domainstats) and cluster level pkg (vmistats)
Cluster level VMI stats (like kubevirt_vmi_phase_count) are now collected by virt-controller because virt-controller has full visibility into all VMIs across their entire lifecycle. Virt-handler is now only responsible for the domain level stats (like kubevirt_vmi_memory_available_bytes) which is scraped from the virt-launcher pod.
Going forward, i'd like to see more cluster level metrics, but until we fix this current issue and merge this refactor we can't accurately capture VMI at a cluster level.
Fixes kubevirt_vmi_phase_count to include all phases, even those that occur before handler hand off.

/cc @sradco

i address the CI failures. The functional tests were attempting to call a virt-handler endpoint directly to verify a metric is reported when that metric has now transfered to virt-controller in this pr.

/lgtm
/approve


Question: VisionOS / Custom Materials

Very cool project!
I noticed you pointed out that it certainly won't work with VisionOS because of "Custom Materials",' I'm a bit out of my depth here, but am curious - is there -any- route you can see towards getting Shape Key animations into RealityKit?  Is this a "simply impossible" thing or "very difficult" or "kinda tricky?"  I'm trying to get a sense of the -scale- of difficulty here. :)
sincerely,
Phil

Thank you! It looks like this package might have been "sherlocked" in iOS 18/ macOS 15/ VisionOS 2. See the docs here: https://developer.apple.com/documentation/realitykit/blendshapeweightscomponent I've not played with the new APIs yet though

Actually, this doc page seems to give the clearest overview of how to use the new API https://developer.apple.com/documentation/realitykit/blendshapeweightsmapping

Once I've had a chance to play with it and confirm it does everything RealityMorpher can I'll update the README to indicate that this is now an iOS 17 backfill

Good find! No shame in being Sher-locked. Just means you're ahead of your time.  I also saw this:
https://forums.developer.apple.com/forums/thread/766484
I haven't had a chance to play with it yet either, I'm very much punching out of my weight class so if you figure it out and are willing to consult I'd be happy to pay you for your time.

What platforms and os version do you need to target? Just Vision OS? And if so, is Vision OS 2 acceptable as the minimum supported version?

Yep, just Vision OS 2

could you DM me please at olidew at gmail dot com?


make shaders work when MAX_CASCADES_PER_LIGHT is undefined

Objective
The post_processing and shader_prepass examples are both currently broken on main. This, combined with https://github.com/bevyengine/bevy/pull/6997 fixes them.
Solution
The same fix as https://github.com/bevyengine/bevy/pull/6997, just wrap the line in an #ifdef so it doesn't cause issues.

I implemented what I think is a better fix in #7419

Using bevy latest (289fd1d0f) I still have problem.


[CAS-1288] Leverage OAuth server support on Spring Security OAuth library and add missing grant types

Add missing grant types :

password credentials
implicit

Reported by: Jérôme Leleu, id: jleleu
Created: Tue, 2 Apr 2013 10:46:24 -0700
Updated: Wed, 19 Jun 2013 10:18:09 -0700
JIRA: https://issues.jasig.org/browse/CAS-1288
As far as the /profile endpoint is concerned, we can define that as a convenience endpoint for the integrator, but it's not strictly required for OAuth 2 compatibility.

@leleuj Shall we tackle this for 4.2? Does it seem like something you can fit into that release?

Yes, I should definitely restart working on a full OAuth server support (same for https://github.com/Jasig/cas/issues/514).

By the way, here is a summary of our last discussion at ESUP, also available on the roadmap doc:

Oauth server support
CAS server can be customized to act as an OAuth server. Presently the OAuth implementation requires that the client receives the TGT to pass to the profile as an access token. Also, the implementation attempts to release all attributes rather than those are allowed due to the limitations in current design. The following alternatives may be used instead:
Rather than passing the TGT directly to the client, we simply encrypt the entire CAS validation response at the time of authorization using a technology such as JWT. The encrypted response is sent to the client, and submitted back again to the profile endpoint at which point it is decrypted again and attributes are released.
Alternatively, rather than encrypting the whole response we could just simply encrypt the TGT and the service and pass it along to the client, which when received again as the access token, would be decrypted to release user attributes.
Or, we could design a new ticket type, that is a true access token made up of random strings that is passed to the client. This would replace the TGT as the token, and does not require any kind of decryption.
Spring Security's OAuth support may be a candidate to review.

Linked to: https://github.com/Jasig/cas/issues/514

Closing as a duplicate of #514


Make Group a type

This changes Group to be a type instead of a function, which means it can support both grouping []Node as well as doing variadic-ish things like Group{n1, n2}.
This also means that Map can just return a Group.
Special thanks to @deitrix for this simple and elegant solution that, for some reason or another, has had me perplexed for a long time.
Fixes #201.

Not sure yet if this is a good idea or confusing, leaving it open for a while.

Why not simply use the variadic syntax?
I know it would break backward compatibility and force people to write things like
g.Group(g.Map(l, f)...)
// Instead of
g.Group(g.Map(l, f))

But that's ok I guess, and the error is screaming (not a silent breaking change)

@EwenQuim thanks for having a look!
Basically because I don't want to break that particular style of usage, with Map inside Group. It's so common everywhere, and I use it myself, and I'd rather not force people to use the spread syntax.
I may just drop this PR and accept that, in the particular use case of returning a list of Nodes without a parent, that one will have to write
g.Group([]g.Node{
…
})

Still thinking about this one. :D

For cases like Map, might it make sense to return a slice type that implements Node, which would effectively be like passing the result to Group?
type NodeList []Node

func (nl NodeList) Render(w io.Writer) error {
	return Group([]Node(nl)).Render(w)
}

func Map[T any](ts []T, cb func(T) Node) NodeList

Alternatively:
type Group []Node

func Map[T any](ts []T, cb func(T) Node) Group

I looove this idea. Really Go-esque and elegant solution

@markuswustenberg It would be breaking change, but most standard uses should just work. Group as a slice type would be a drop-in replacement for callers of the Group function. The only case I can think of where it would throw a compiler error is where the caller expected Group to be a function type, though I can't see that kind of usage being very common.
Group(Map(...)) will also continue to work, with the Group(...) being superfluous if Map's signature is updated to return a Group
I tested the change on my (albeit rather small) project, and didn't get any compiler errors! Might be worth testing on some larger projects, if you have any to hand?

The only case I can think of where it would throw a compiler error is where the caller expected Group to be a function type, though I can't see that kind of usage being very common.

With this in mind, your original PR would also be considered a breaking change 😁

Alternatively:
type Group []Node

func Map[T any](ts []T, cb func(T) Node) Group

Then, you have the option between passing a slice (Group(children)), or varargs (Group{child1, child2, ...}).

@deitrix you're a genius! I didn't understand the full implications of this earlier today. It's so simple and elegant, I can't believe I haven't thought of this solution before. Of course a Group is just a list of nodes. 😅
All tests pass, this looks great. I'm going to leave it open for a few days to ponder implications and test it on my codebases, but I think it's good.

This has been nagging me for literally months!

@eduardolat @JulienTant since you were both involved in the Fragment discussions around #181, I think you'll be pleased to hear that this looks like a final, elegant API for Group that basically solves All The Things.

Looks great, thanks for your work on this fantastic project

I'm just going to merge this, I think it's great. Thanks again, @deitrix, @EwenQuim, and @eduardolat.

This is a super nice change! Thank you all 👍

Also, note that Group must now (in most cases) be used directly as a type, as Go types that are hidden slices can be directly used as it is

@EwenQuim I think it takes some adjusting and documentation, but otherwise I think the Group{…} style is actually quite neat, and what you would expect from a slice of something. It also functions much like the Classes helper (except that one's a map, not a slice). So I don't think I'll add a Fragment helper, I would consider it redundant.
Regarding your last example: you could just put different kinds of Nodes into the slice, as long as they're satisfying the interface. So Div(ID(""), Group{Span(), Div()}, Span()) would work as expected.

I think you're right, it's OK!

Regarding your last example

Yeah that was not a problem, that was just a remark that slices arguments do not work like variadic args (variadic args are more flexible). Gomponents API is really well done :)

@EwenQuim I'm glad we agree.
And thanks! I'm quite proud of it. 😁


Allow the script to understand the file format of ~/.aws/credentials.

The ~/.aws/credentials file is produced by aws configure and has spaces on either side of the equals sign. Given just a single account (or the desired creds being the first set), this RE is still able to pull them out by accepting the spaces.

Nope, makes perfect sense. I'll change it.

Thanks, @nk9!
@elliottfrantz, please merge :-)


Query on handling of DUP event

Hi, may I know how should I present the DUP events in my VCF files which is coming from a population based study? Say for example sample1 has annotated DUP of CN=1 and another sample2 has CN=3 for this stretch of DNA chr1:600342-600816, and the "normal" reference is annotated as CN=2 or hom_ref.
Does the length of the mutation, in this case, number of times the dup event happens affected the prediction?

Hi Solyris,
That is a very good question.
CADD-SV uses annotations in the human reference. So most of the features (like gene based metrics and max values of scores) will remain the same between different copy numbers of a duplication. So I suspect that the scores will not vary much. However, some features are affected like the amount of conserved sequence (will be double for instance in an additional copy number). So that might have a very strong effect.
However, loss of a duplication (CN=1 in sample1) might even best be queried as a deletion of sequence (chr1:600342-600816).
In other words: Length (and choice of SV type) might affect the prediction. As SVs and their effects can be very complex it really depends on the sequence. So I cannot recommend you what would be the appropriate choice in this specific example.

Hi @kleinertp , thanks for the response. Not sure, if I understood you correctly, just to make sure I got you correctly

in CADD-SV prediction of SV's effect, DUP event with CN=3 of higher will not have any difference in what the value of CN is and the correct way to represent these event in the input to CADD-SV will be
chr1  600342  600816  DUP

I see, so these CN=1 (het deletion) or CN=0 (hom deletion) can be represented as below for input to CADD-SV
chr1  600342  600816  DEL

What we are concern about, is the way CNV can be represented to CADD-SV in the right way. In our case, our CNV caller calls DUP and DEL and annotates them with a copy number value, eg CN=[0,infinity) - theoretically, but I guess there would exist some upper limit in real biological system.
Do comment on the best way to represent these kinds of events as input to CADD-SV to maximise the predictive gains.

Hi Solyris,
Yes, Point 1 and 2 are correct.
CADD-SV relies on annotation from the human reference genome. When the duplication is not present in the reference, CADD-SV cannot make use of it. There is no option to tell CADD-SV the exact copy number of a region. I would proceed using these entries:
chr1<PHONE_NUMBER>16 DEL
chr1<PHONE_NUMBER>16 DUP
My comment:
CADD-SV prioritizes SVs, so you will have to manually inspect your results afterwards. So take the copy numbers into account once you received CADD-SV scores and annotations.


Ambiguous iPhone developer certificate

Related to https://github.com/angelXwind/simject/issues/44, but this is not a duplicate certificate.
Copying binaries...
/Users/ahk/Dropbox/Developer/iOS/simject/respring_simulator/.theos/obj/macosx/debug/respring_simulator -> ../bin/respring_simulator
> Making all for tweak simject…
==> Merging tweak simject…
==> Signing simject…
iPhone Developer: ambiguous (matches "iPhone Developer<EMAIL_ADDRESS>(XXXXXXXXXX)" and "iPhone Developer: Corporate name (YYYYYYYYYY)" in /Users/ahk/Library/Keychains/login.keychain-db)
make[3]: *** [/Users/ahk/Dropbox/Developer/iOS/simject/simject/.theos/obj/iphone_simulator/debug/simject.dylib] Error 1
rm /Users/ahk/Dropbox/Developer/iOS/simject/simject/.theos/obj/iphone_simulator/debug/simject.dylib.7876ddb4.unsigned
make[2]: *** [internal-library-all_] Error 2
make[1]: *** [simject.all.tweak.variables] Error 2
make: *** [all] Error 2

I could probably solve this by removing one of them, but I really don't want to do that. Any workarounds?

I was able to export my private certificate, delete it and then make simject. Afterwards it was just a matter of importing it again.


Middy validation causing error

I believe this how to is outdated now:
https://sst.dev/examples/how-to-use-middy-to-validate-your-serverless-api-requests.html
when the library is used now there is an error when the schema is provided as suggested on the the middy website error is obtained when SST stack is deployed:
TypeError: eventSchema2 is not a function
    at validatorMiddlewareBefore
    ...

Code usage:
import middy from "@middy/core";
import { APIGatewayProxyEvent, APIGatewayProxyResult } from "aws-lambda";
import jsonBodyParser from "@middy/http-json-body-parser";
import httpErrorHandler from "@middy/http-error-handler";
import validator from "@middy/validator";

const ONE_WEEK_SECS = 60 * 60 * 24 * 7;

const eventSchema = {
  type: "object",
  properties: {
    body: {
      type: "object",
      properties: {
        sampleNumber: {
          type: "integer",
          minimum: Math.floor(Date.now() / 1000) - ONE_WEEK_SECS,
          maximum: Math.floor(Date.now() / 1000),
        },
      },
      required: ["sampleNumber"],
    },
  },
};

const responseSchema = {
  type: "object",
  required: ["body", "statusCode"],
  properties: {
    body: {
      type: "string",
    },
    statusCode: {
      type: "number",
    },
    headers: {
      type: "object",
    },
  },
};

async function lambdaHandler(
  event: APIGatewayProxyEvent
): Promise<APIGatewayProxyResult> {
  // the returned response will be checked against the type `APIGatewayProxyResult`
  return {
    statusCode: 200,
    headers: { "Content-Type": "text/plain" },
    body: `Hello, got\n${JSON.stringify(event)}.`,
  };
}

const handler = middy(lambdaHandler)
  .use(validator({ eventSchema }))
  .use(jsonBodyParser())
  .use(httpErrorHandler());

export { handler };

when I change the eventSchema to eventSchema2 manually :
const handler = middy(lambdaHandler)
  .use(validator({ eventSchema : eventSchema2 }))
  .use(jsonBodyParser())
  .use(httpErrorHandler());

I get
TypeError: eventSchema is not a function
   at validatorMiddlewareBefore
   ...

Does SST create different version of the exported middy function?

Experienced the same issue. Fixed it by using transpileSchema from @middy/validator/transpile
...
import validator from "@middy/validator";
import { transpileSchema } from '@middy/validator/transpile'
...

const handler = middy<APIGatewayProxyEventV2>(lambdaHandler)
  .use(jsonBodyParser())
  .use(
    validator({
      eventSchema: transpileSchema(eventSchema),
    })
  )
  .use(httpErrorHandler());

Also make sure you're sending correct header i.e Content-Type: application/json
reference: upgrade guide for middy 3 -> 4

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Client Allocation Request for: Net Activity Inc.

Client Allocation Questions
Core Information

Name: Net Activity Inc.
Website / Social Media: https://www.netactivity.us/
Region: North America
DataCap Requested: 5TiB
Addresses to be Notarized: f1xnfcild4h2ux2ympnbjop4kqjsxaz2ruskqdeta
Notary Requested: TimWilliams00

@wcoreneburnn2 Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Hi，thanks for application! I would like to know your information.
Could you please introduce you and your company?
What kind of data do you want to store?
How many miners do you want to store? For how many regions?
What questions would you ask the miners?
What do you do to prevent overallocation?

Hi. I'm Harry. Our company  is to help businesses become more competitive in their industry by utilizing Information Technology. Our business is a service to our clients. We provide services as follows: CLOUD COMPUTING SERVICES---Net Activity’s enterprise cloud services are adaptable, scalable, reliable and affordable; VOIP BUSINESS PHONE SERVICES---Net Activity, Inc. installs and configures on premise based phone systems and IP based solutions; IT MANAGED SERVICES---Our monthly Server and Desktop monitoring solutions at a fixed price that takes the worry and anxiety out of knowing that your systems are up and running; SOFTWARE DEVELOPMENT SERVICES---Our software development services aim at developing solutions in alignment with the clients’ needs and invest in innovation to deliver client satisfaction.
Some data about clients' information and pictures.
2-3miners. We want to store in america. Because it can be convenient.
How long will you spend? Which regions will you do it?
Increase the frequency of inspections.

@wcoreneburnn2 It had finished. I will close this issue.

Thank you for your permission!


New gem release

There is no released version of this gem that doesn't have Redcarpet as a hard dependency.

Released v1.1.0 b7f7c973d24a1c64618acc68171dd35d576b487b, let us know about any issues. Thanks!


 The only way to use a model is via using this awful require syntax, and all the type information is lost and cannot be inferred.
Stop this madness, right now.

Perhaps you can recommend a better way of doing it (or create new PR / repo). I'm using it and it works fine. Not everyone using Sequelize is using TypeScript.

Hello, this repository is indeed quite outdated, I agree with that. I will see what I can do.

#99


Namings

Your namings are terrible! 😭
Read this: https://builtin.com/data-science/variable-names
-rep, Bro(

I'm sorry.
I will try to do better =/


Remove remaining reference to max_db_per_node.

The field was removed from the spec and the last remaining reference
was in an example.

Also testing out of the CLA bot now works (it should, fingers crossed).

Fixes issue #150

LGTM
I'm ok with removing it but if someone wants to keep that info we could move it into the metadata section.

LGTM

LGTM

LGTM

LGTM


[WIP] Add what's new in January

Updated and new articles in January.

/cc @scottaddie Note that I needed to make a single node for January 2020, because a "2020" node with a single "January" child won't be displayed.


Added blameable listener to doctrine_extensions.yml

Thanks for your PR and I'm sorry that there hasn't been any response by the maintainers.
I've closed this PR because it the branch has conflicts that must be resolved. If you still want this PR to be merged, please resolve the conflicts and reopen this PR.


frame may be undefined

I got an error frame undefined, so I added a check here. In all other components that use the frame in aframe repository, we have a condition before using the frame.

Thank you!

Can you please push a new build in the repo so we can use a jsdelivr gh url, or better a new release. Thanks!

Done 3.1.10


What counts toward connections?

In metrics there are server connections and client connections listed. When is each of these types of connection made? I'm trying to predict how many units will be needed.

When you start your app server, each hub will start 5 server connections by default. And client connection will be made after it negotiate with app server and redirect to service.

Ok thanks! So each app server in a server farm will start five connect per hub and each client will start one connection per hub?

I have one single hub, but each time I start my program in visual studio, Azure SignalR uses 20 server connections.  Then if I stop debugging and try to debug again, this number jumps to 30 and maxes out the free plan, so then I receive the "Connection count reaches limit."
I have to completely exit visual studio and close all of the tabs I'm debugging in and wait a couple minutes to restart visual studio for the Azure Signal R server count to go back to 0....am I doing something wrong?  My client connections are always at 0, or match what they should be but it looks like the server connection count is adding in increments of 20 and I can't find any way to shut them all when they max out at 30 without exiting visual studio and re-opening it.  It's like I'm missing a server close hub method for when I stop debugging.

Nice, I close this then.

We use paid tier for production and free tier for staging with with 1 app service and 2 hubs which automatically takes 20 connections on the start (5 per hub, both are asp.net so another 5 per hub)
We made a suggestion to increase that maximum limit of 20 connections. If you guys think it's a good idea and have some spare votes:
https://feedback.azure.com/forums/915436-azure-signalr-service/suggestions/38952187-increase-free-tier-signalr-connection-limit


fysty: Add version 20220202-76f785d5

closes #8005
Fysty is a fork/patch of the SSH and Telnet client PuTTY.
NOTES:

The app is built in 64-bit.
persist is not needed because config is at HKCU\SOFTWARE\SimonTatham\PuTTY.
checkver: I involved release date in version because there will be problem checking which one is newer through commits. For example, 4c207368 comes after a2404931.
bin: Command-line options for putty.exe, etc. can be found in the manual. Also see extras/putty manifest.

The manifest is good - I'm just concerned about the popularity/trust of the repo. Any way to rectify that?


add "have token" indicator

if you have a token, add check to menu item or some other indicator

fixed in v1.1 (build 1275) or later.


Running the server shows an error no module named secret in settings.py file

While running the command python manage.py runserver an error comes up showing no module exists named secret in settings.py file. It should be renamed as
from .secret import getgithubkey and others,  since we are accessing the file in the same directory as of the settings.py file

@anshuldutt21 Are you sure this is not specific to the python version/ virtualenv setup / Django version? because it has beem working fine for us in the past.
Did you setup your local repo using the instruction in README?

@rtgdk I think you are right about this issue being specific to the python version. I tried resetting up and it worked.

Closing this then.


Deep merge with internal arrays does not work as expected

The merge method is not available for arrays (why?) and when objects contain array descendants it causes deep merges not to work as expected.
For example:
// Works as expected without arrays:
var target = Immutable({I: {have: {anObject: {inside: "me"}}}})
var source = Immutable({I: {have: {anObject: {how: "cool"}}}})

console.log(JSON.stringify(target.merge(source, {deep: true})))
// -> {"I":{"have":{"anObject":{"inside":"me","how":"cool"}}}}

// But with arrays, original values in the target may be lost:
var targetWithArray = Immutable({I: {have: {anArray: [{inside: "me"}]}}})
var sourceWithArray = Immutable({I: {have: {anArray: [{how: "cool"}]}}})

console.log(JSON.stringify(targetWithArray.merge(sourceWithArray, {deep: true})))
// -> {"I":{"have":{"anArray":[{"how":"cool"}]}}}
// Expected: {"I":{"have":{"anArray":[{"inside":"me","how":"cool"}]}}}

This seems to be because arrays are not mergable as defined by isMergableObject which causes the merge to get "stuck" once an array is found and replace it instead of continuing the merge as seen here.
We are migrating from Immutable.js where this functionality exists and is useful to us. Is there a design decision behind this or is it a bug?
Thanks.

I'm running into the same limitation. @itaysabato have you found a pattern to use as an alternative?

@gmeans my alternative is to not use this library yet...
I have implemented my own setIn and mergeIn functions that use native objects and arrays.

I ended up using a regular merge within the setIn:
export const saveAddress = (state = initialState, action) => {
  const { address, address2, city, province, postal, country, index } = action;

  const existing = Immutable(state.current.addresses[ index ]);

  return Immutable.setIn(state, [ 'current', 'addresses', index ], Immutable.merge(existing, {
    address,
    address2,
    city,
    province,
    postal,
    country,
  }), { deep: true })
};

Maybe my use case is slightly different since I'm directly setting an element at a specific index, but I experienced the same issue you had. After the setIn the resulting object only consisted of what was passed into setIn, wiping out all existing values that I would expect to be merged in.

@gmeans If the library had a mergeIn method (that creates a new object only if necessary) it would solve my use case and probably yours as well. You can implement it yourself though with code like:
function mergeIn(target, path, source, config) {
    var innerTarget = Immutable.getIn(target, path);
    var innerMerged = Immutable.merge(innerTarget, source, config);

    return Immutable.setIn(target, path, innerMerged);
}

There are probably some edge cases that need to be addressed - this code is based on what I'm doing without seamless-immutable.
Personally, I think this is a basic functionality (the only one I need, in fact) that should be provided by the library.

@itaysabato it is a design decision (at least to my knowledge). You can use a custom merger to control how to handle arrays. Some examples here: https://github.com/crudh/seamless-immutable-mergers

@itaysabato Are you (or anyone else) aware of any good examples of deep merge handling arrays, eg an array a couple of levels down that in turn has various items in it, including arrays and objects with arrays in them? I'm thinking to put together a recursive custom merger to handle arrays, but as it hits anything other than arrays it'd be nice if it could reuse already existing merge logic. Otherwise, as I understand id, the custom merger would have to handle every other kind of item it comes across (objects, primitive types, etc) as well, which seems like a big job.
The example custom mergers seem to be quite simplistic and not cater for recursing down arrays.
I'll keep digging meanwhile.
Sample data
{    aa: {      bb: [       'one',        { xx: { yy: [1, 2, 3] } },        [11, { zz: 123 }]      ]    }  }

@rtfeldman Just pinging you as well. See comment / question above :)


Move this whole thing to a new organisation

@hkir-dev would you mind doing this please?
I think best to just clone this to a new organisation as mentioned by Chris?
If we can get this on cell-ontology.github.io that would be amazing
We can keep this repo until we have addressed all the issues before closing.

New repo: https://github.com/cell-ontology/cell-ontology.github.io
And the new site published: https://cell-ontology.github.io/
Keeping this repo for the possible issues. We can delete afterwards.


adding the label customization

Please, approve the PR

@damien-bernard can you share a little bit about the motivation?

@damien-bernard can you share a little bit about the motivation?

According to the country, we would like to customize the message.
I'll close my request. An enhancement has been pushed https://github.com/arronhunt/highlightjs-copy/pull/7


Updated AWS3provider and added option to add pem certificate while cr…

I have found an issue while using this package. The issue was :
**

[Aws\S3\Exception\S3Exception]
Error executing "ListObjects" on "https://s3.eu-central-1.amazonaws.com/**************?encod
ing-type=****"; AWS HTTP error: cURL error 60: Peer certificate cannot be authenticated with given CA certificates
[GuzzleHttp\Exception\RequestException]
cURL error 60: Peer certificate cannot be authenticated with given CA certificates
[GuzzleHttp\Ring\Exception\RingException]
cURL error 60: Peer certificate cannot be authenticated with given CA certificates

**
Well i found a solution for it which is while making a connection to S3 we haven't option to add PEM certificate so i just added that option and updated readme for future use.

This pull request broke the "push" functionality for those of us that don't specify a .pem certificate. I get an exception with the message Undefined index: http for $this->supplier['http'] in the provider.

@marcaube sorry to hear about that inconvenience and thanks for reporting. Just pushed an update to master that should fix that issue. Can you please try with the dev-master version in your composer and let me know if that fixes it for you as well?


Add Untracked files

Checklist

[ ] Commit Messages follow the Conventional Commits pattern

A feature commit message is prefixed "feat:"
A bugfix commit message is prefixed "fix:"

[ ] Tests for the changes have been added
[ ] CI has been passed. (GitHub actions all turns green)
[ ] CLA has been signed

References to other Issues or PRs

Have you read the Contributing Guidelines?
Brief description of what is fixed or changed
Other comments

I have thought about this situation at the beginning of creating this action:
If we adding new files to our wechaty/grpc and at the same time they need to be deployed to the go-grpc repo, then the existing CI script will not add new files.
I did not add git add . because I think we should manually maintain the new files.
However, to use git add . is ok for me, so let's merge this PR and see if it has any cons.

It seems that you need to bump the version so that the deploy script can publish to a new version tag.


⚠ Orpheus has degraded performance

In b8768f8, Orpheus ($OPS) experienced degraded performance:

HTTP code: 502
Response time: 1342 ms

Resolved: Orpheus performance has improved in 757fd50.


Add turkish AEU accidentals to data.ACCIDENTAL.GESTURAL

This is the sequel PR to #720 :
Since the aeu accidental values cannot be expressed with the already existing values of data.ACCIDENTAL.GESTURAL, it would be very helpful to add them as gestural values too.
I splitted data.ACCIDENTAL.GESTURAL as it has been done for written accidentals in #720 and added the aeu accidentals. To avoid the duplication of data types, I renamed data.ACCIDENTAL.WRITTEN.aeu to data.ACCIDENTAL.aeu.
A corrected version of the aeu accidental value overview has been added as well.

thanks for your work on this, @annplaksin 👍

Sorry for me complaining about this so late, but even if AEU accidentals would have be the same in gestural and visual domain, I don't think we should combine them nto data.ACCIDENTAL.aeu, as this breaks current naming (and basically the current schema). I would prefer to keep data.ACCIDENTAL.WRITTEN.aeu and add data.ACCIDENTAL.GESTURAL.aeu instead.

You have either a break of naming or redundancy. I asked @kepper before I made the pull request if this naming is okay. He didn't complain.

I'm certainly not the the person to decide what is right or wrong here – MEI is a community thing, and we should move forward by consensus whenever possible. I think this is a legitimate question, though, and while I refuse to be any more relevant for the decision than anyone else, I think there are arguments that @annplaksin's implementation is actually consistent with existing MEI.
In CWMN accidentals, we see a certain amount of duplication of visual values: x is two half-tones up, as is ss. For gestural purposes, there's no difference between them, though. This difference legitimates to have two separate data types for the two domains. In the list suggested by @annplaksin, there is no similar duplication of values. Accordingly, we don't need to split this up by domain. I think this is conforming to other situations in MEI. The data.DURATION.cmn data type is used by both data.DURATION (applies to @dur) and data.DURATION.gestural (applies to @dur.ges) – as the values apply to both categories alike.
I think we all agree that MEI is a "grown" model that isn't perfectly consistent across all of its parts. In this particular situation, I think we're safely within the "usual range", though – but this is definitely something to discuss and negotiate…


chore: typecheck example and test files

Summary
It would be nice to typecheck test files in CI. To make sure the test stay up to date and to have more tests for Jest types. For the start these are test files of three packages with types fixed and typecheck set up.
CI runs on the examples folder too, so I added type checks for examples written in TypeScript.
Test plan
If everything is setup correctly, tsc should find two type errors. (Which I will fix, of course.)

Looks like all is working.


GC forces metadata compaction every cycle

The SimpleGarbageCollector class calls .compact(...) (with both flush and wait flags set to true) every cycle, on the operating assumption that the GC performed many changes to the metadata and it'd be good to flush and compact.
However, we probably don't need to compact. We can probably flush instead. If the metadata table's compaction ratio is still set to the default of 1 (set during initialization), this will be enough to trigger a compaction on the changed tablets (but not on the unmodified ones). This is a less aggressive solution than forcing a full table compaction on the metadata, and will also permit users to select a different compaction ratio for the metadata tables, if they choose to do so.
Some other ideas that could be done as separate issues (if necessary): eliminate the automatic flush from the GC entirely, and put this responsibility on the user, or implement a custom compaction strategy that makes use of summarizers to compact the metadata more efficiently (only when there's a clear benefit).

This compact was added in fb54a03d3431deb7c88682284dd9a599457ac70c for https://issues.apache.org/jira/browse/ACCUMULO-1281

Saw this again while testing #1330 . It made it a bit confusing to test -o gc.safemode there, because the set of candidate files skipped from deletion kept growing on every gc cycle, and there was always an extra file in the metadata that didn't show in the SAFEMODE output in the logs (because the compaction was triggered and created a new file after the list of files had already been printed).


how to change font size?

Hello
Many thanks for the great work.
Just wanted to know if it is possible to change the font size in pyexamples?
Mohammad

Hello, do you know how to change the font size?

\tikzset{every node/.style={font=\large }} %font size of nodes


SPLITTER: remove redundant column

Description
Remove split_index column generated from the intermediate step.
Related Issues
#614
Checklist:

[x] My code follows the code style of this project, as detailed in our contribution guidelines.
[ ] I have added tests.
[ ] I have updated the documentation accordingly.

Maybe add a check of the column names in test_python_splitters.


Move error checking for CertPath

Error checking for a missing CertPath would only happen when there was another error to report while creating the listener. Since a CertPath must always be present when using an https stager, this PR moves that validation earlier in the listener creation.

@leesoh Thanks for this!


tsdb/wlog: Only treat unknown record types as failure

I noticed that tsdb/wlog.Watcher.readSegmentForGC treats e.g. histogram records as unsupported, and counts them against w.recordDecodeFailsMetric. I propose that we instead do this only for the record.Unknown type.

I would expect the same change at https://github.com/prometheus/prometheus/blob/6f595c6762d253c0ef7574b93f9795a6e3e40c81/tsdb/wlog/watcher.go#L657-L659

I would expect the same change at

@bboreham thanks, I didn't notice the similarity. I can make a corresponding change there.

How does this PR relate to #13181 ?

How does this PR relate to #13181 ?

I'll try to review that PR, wasn't known to me.

How does this PR relate to https://github.com/prometheus/prometheus/pull/13181 ?

@bboreham after reviewing that PR, as far as I can tell these two PRs are orthogonal to each other.

I would expect the same change at

@bboreham I've made the change also there, PTAL.

@machine424 I think you'll need to review again after I resolved the conflict :)

treats e.g. histogram records as unsupported

Can you clarify this for me please: I see record.HistogramSamples and record.FloatHistogramSamples as cases in the switch statement.

Oh I get it, this was actually a bug in readSegmentForGC only, and my "I would expect" comment caused you to make it a bigger bug.  Sorry about that.


Create web playground

[ ] #216
[ ] #212
[ ] #213
[ ] #214
[ ] #215

NOTE: Possible duplicate!  It has to be better than the one we already have at https://chat.dev/

Hey Can I take up this task?
And I wanted to know if there are any limitations on the technologies I need to be using specifically for building this.
I am a new member to the community, and I am sorry in case I missed out to read something before posting here.
Thanks!

Is someone pursuing this ? If not where do I send the design?

post a design here!

we gotta get chat.dev back online first because otherwise we'll fail to be feature complete

I'll work on that first then do the design then

unfortunately chat.dev is closed source!


Unrecognized arguments base_dir and use_gpt2_tokenizer

create_tfrecords.py: error: unrecognized arguments: --base_dir /content/GPTNeo/openwebtext --use_gpt2_tokenizer
I even tried using input_dir instead of base_dir and without gpt2_tokenizer. I think that worked, but when I got to copying data to the storage bucket it did this instead: No URLs matched: /content/GPTNeo/openwebtext_tokenized
I tried to make folders in the bucket to match the path but nothing worked.

yeah it maked me also some problems, but worked at last. but I couldn't train..there is some bug over there.

yeah it maked me also some problems, but worked at last. but I couldn't train..there is some bug over there.

Same here. Modified the command like this:
!python data/create_tfrecords.py --mode documents --input_dir /content/GPTNeo/$dataset_path --name $dataset_name --output_dir $out_name --write_dataset_config
The command fails with this error:
Traceback (most recent call last):   File "data/create_tfrecords.py", line 205, in <module>     results = create_tfrecords_mp(files, args)   File "data/create_tfrecords.py", line 186, in create_tfrecords_mp     files = split_list(files, len(files) // args.processes)   File "data/create_tfrecords.py", line 67, in split_list     return [l[i:i+n] for i in range(0, len(l), n)] ValueError: range() arg 3 must not be zero

Same here. Modified the command like this:
!python data/create_tfrecords.py --mode documents --input_dir /content/GPTNeo/$dataset_path --name $dataset_name --output_dir $out_name --write_dataset_config
The command fails with this error:
Traceback (most recent call last):   File "data/create_tfrecords.py", line 205, in <module>     results = create_tfrecords_mp(files, args)   File "data/create_tfrecords.py", line 186, in create_tfrecords_mp     files = split_list(files, len(files) // args.processes)   File "data/create_tfrecords.py", line 67, in split_list     return [l[i:i+n] for i in range(0, len(l), n)] ValueError: range() arg 3 must not be zero

Same here. Modified the command like this:
!python data/create_tfrecords.py --mode documents --input_dir /content/GPTNeo/$dataset_path --name $dataset_name --output_dir $out_name --write_dataset_config
The command fails with this error:
Traceback (most recent call last): File "data/create_tfrecords.py", line 205, in <module> results = create_tfrecords_mp(files, args) File "data/create_tfrecords.py", line 186, in create_tfrecords_mp files = split_list(files, len(files) // args.processes) File "data/create_tfrecords.py", line 67, in split_list return [l[i:i+n] for i in range(0, len(l), n)] ValueError: range() arg 3 must not be zero

This happens when the number of documents in the dataset is smaller than the amount of CPUs you have. Set the argument --processes 1 to address this

Same here. Modified the command like this:
!python data/create_tfrecords.py --mode documents --input_dir /content/GPTNeo/$dataset_path --name $dataset_name --output_dir $out_name --write_dataset_config
The command fails with this error:
Traceback (most recent call last): File "data/create_tfrecords.py", line 205, in <module> results = create_tfrecords_mp(files, args) File "data/create_tfrecords.py", line 186, in create_tfrecords_mp files = split_list(files, len(files) // args.processes) File "data/create_tfrecords.py", line 67, in split_list return [l[i:i+n] for i in range(0, len(l), n)] ValueError: range() arg 3 must not be zero

This happens when the number of documents in the dataset is smaller than the amount of CPUs you have. Set the argument --processes 1 to address this

create_tfrecords.py: error: unrecognized arguments: --base_dir /content/GPTNeo/openwebtext --use_gpt2_tokenizer
I even tried using input_dir instead of base_dir and without gpt2_tokenizer. I think that worked, but when I got to copying data to the storage bucket it did this instead: No URLs matched: /content/GPTNeo/openwebtext_tokenized
I tried to make folders in the bucket to match the path but nothing worked.

The project documentation is a little out of date. using input_dir and dropping use_gpt2_tokenizer is the correct way to use the code. Can you elaborate on what goes wrong when you make those changes?

Same here. Modified the command like this:
!python data/create_tfrecords.py --mode documents --input_dir /content/GPTNeo/$dataset_path --name $dataset_name --output_dir $out_name --write_dataset_config
The command fails with this error:
Traceback (most recent call last): File "data/create_tfrecords.py", line 205, in <module> results = create_tfrecords_mp(files, args) File "data/create_tfrecords.py", line 186, in create_tfrecords_mp files = split_list(files, len(files) // args.processes) File "data/create_tfrecords.py", line 67, in split_list return [l[i:i+n] for i in range(0, len(l), n)] ValueError: range() arg 3 must not be zero

As explained by @322997am this is an unrelated error and is caused by the fact that you have more CPUs trying to create TFRecords than you have documents. You should either increase the number of documents or only use some of your CPUs to create TFRecords. The flag --processes n allows you to limit the number of parallel processes being employed.

Hi! Thanks for the feedback. We pushed some updates including a totally revamped notebook, give it a try and let me know if you're still getting this error.


IndexError: too many indices for array

Hi Kostas,
I am using Deepmedic with 2D nifti images. I have changed the kernel sizes setting the third dimension to 1 (ie. size [3 3 1]) and my input images are of size  img: [512x512 uint16]
(all specifications:
       hdr: [1x1 struct]
  filetype: 2
fileprefix: 'processed_nomask'
   machine: 'ieee-le'
       img: [512x512 uint16]
  original: [1x1 struct] )

But during training I got the following error:
Traceback (most recent call last):
File "./deepMedicRun", line 134, in
deepMedicTrain.deepMedicTrainMain(absPathToTrainConf, absPathToCnnModelGivenInCmdLine, cnnInstanceLoaded, filenameAndPathWhereNewlyCreatedModelWasSaved, args.reset_optimizer)
File "/Documents/GitHub/deepmedic/deepmedic/frontEndModules/deepMedicTrain.py", line 496, in deepMedicTrainMain
do_training(*trainSessionParameters.getTupleForCnnTraining())
File "/Documents/GitHub/deepmedic/deepmedic/trainValidateTestVisualiseParallel.py", line 1197, in do_training
reflectImageWithHalfProbDuringTraining
File "/Documents/GitHub/deepmedic/deepmedic/trainValidateTestVisualiseParallel.py", line 770, in getTheArraysOfImageChannelsAndLesionsToLoadToGpuForSubepoch
reflectImageWithHalfProb = reflectImageWithHalfProbDuringTraining
File "/Documents/GitHub/deepmedic/deepmedic/trainValidateTestVisualiseParallel.py", line 123, in actual_load_patient_images_from_filepath_and_return_nparrays
roiMaskData = reflectImageArrayIfNeeded(reflectFlags, roiMaskData)
File "/Documents/GitHub/deepmedic/deepmedic/trainValidateTestVisualiseParallel.py", line 59, in reflectImageArrayIfNeeded
reflImageArray = imageArray[::stepsForReflectionPerDimension[0], ::stepsForReflectionPerDimension[1], ::stepsForReflectionPerDimension[2]]
File "/usr/local/lib/python2.7/site-packages/numpy/core/memmap.py", line 335, in getitem
res = super(memmap, self).getitem(index)
IndexError: too many indices for array
Do you know what is making it give this error?
Many thanks!
Carla

Hello Carla,
I have not previously tested the code on 2D data. I dont think there would be too many changes needed to make it run, probably quite some reconfig of the meta-params though (2d kernels will have far less weights, so some metaparams like learning rate may be needed).
But there is at least one change needed, which is probably the cause of this problem of yours. The data of yours are 2D [x,y] and should be loaded as 3D [x,y,z], with unary 3rd dimension z=1.
The data is loaded in:
trainValidateTestVisualiseParallel.py -> actual_load_patient_images_from_filepath_and_return_nparrays(...)
look for the nib.load() calls and the following get_load() calls. Probably it will just be enough if you reshape the nparray returned by get_load to [-1,-1, 1] dimensions.
Let me know how this goes.
K

Hi Kostas!
thanks for your answer! It proved very helpful and I got over this error by just writing a small function on the py script
def getload(img_proxy):
image = img_proxy.get_data()
if len(image.shape) != 3:
image=np.expand_dims(image, axis=2)
return image
and calling it wherever get_data() was called in the script.
However, when I attempt to train the model it now
gives me the same 'NoneType' error as Zhou Huan is mentioning:
Loading subject with 1st channel at: /Documents/GitHub/deepmedic/examples2/dataForExamples/brats2015TrainingData/train/brats_2013_pat0005_1/niftifile_trial.nii
An error has occured during the function execution
Traceback (most recent call last):
File "/usr/local/lib/python2.7/site-packages/pp-1.6.4-py2.7.egg/ppworker.py", line 90, in run
__result = __f(*__args)
File "", line 94, in getTheArraysOfImageChannelsAndLesionsToLoadToGpuForSubepoch
File "", line 52, in actual_load_patient_images_from_filepath_and_return_nparrays
NameError: global name 'getload' is not defined
Traceback (most recent call last):
File "./deepMedicRun", line 134, in
deepMedicTrain.deepMedicTrainMain(absPathToTrainConf, absPathToCnnModelGivenInCmdLine, cnnInstanceLoaded, filenameAndPathWhereNewlyCreatedModelWasSaved, args.reset_optimizer)
File "/Documents/GitHub/deepmedic/deepmedic/frontEndModules/deepMedicTrain.py", line 496, in deepMedicTrainMain
do_training(*trainSessionParameters.getTupleForCnnTraining())
File "/Documents/GitHub/deepmedic/deepmedic/trainValidateTestVisualiseParallel.py", line 1218, in do_training
labelsForCentralOfSegmentsForSubepTrain] = parallelJobToGetDataForNextTraining() #fromParallelProcessing that had started from last loop when it was submitted.
TypeError: 'NoneType' object is not iterable
I have double checked the data I am inputing (so far only one example) and all my areas, rois and gt lables have the same dimensions. Do you know if there could be something else which is causing this error? I have attached the info of the files in case you want to have a look at them (I am actually using the same file for rois and gt, a plain white square in a black background, since at this stage I just want to have the software run on the cases I am using. I wasn´t able to attach the nifti files, but I can send them via email if that is better.
Many thanks for your help!
Carla

Hello,
During the "do_training" routine, there is a parallel process being spawned at every iteration, that extracts the data for the next training batch. It's the "parallel python" aka "pp" package used for that, to check for documentation that will help for your debugging. When anything goes wrong at runtime during this process, it returns "None" instead of data for a batch. So, the problem is not the NoneType error, but what caused.
And what caused it, is reported in the error message, right before the "traceback" line. In this case, it says "NameError: global name 'getload' is not defined".
Which means that the parallel process does not know about your getload() function. You need to tell to pp that this new function will be used (see pp documentation).
Look for the variable "tupleWithLocalFunctionsThatWillBeCalledByTheMainJob", and add the name of your function in that tuple.
If any more debugging is used, please first check the error message given before the "traceback", roughly 10 lines above the NoneType error.
Hope this helps,
K

Thank you very much for the quick reply. And for indicating the actual error which I had completely missed. It is working perfectly now, both the training and the testing!
Best,
Carla

Glad to here. Let the issue open for a bit, I may try to integrate processing 2D data in the code more naturally soon.
K

This is now supported in v0.6.1, currently on the branch supportTheano09, soon to be merged to master.
Thanks a lot ctaiz96 for feedback on this.
Closing.

Would anyone have a 2D model.cfg to share?

I'd also be interested in having an example 2D model config file. Cheers!