ref #79031 Added support for PrestaShop <IP_ADDRESS>

Codecov Report
Patch coverage: 11.11% and project coverage change: -0.03% :warning:

Comparison is base (05e332c) 32.55% compared to head (b9b8347) 32.53%.

:exclamation: Current head b9b8347 differs from pull request most recent head b1d630b. Consider uploading reports for the commit b1d630b to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #217      +/-   ##
============================================
- Coverage     32.55%   32.53%   -0.03%
- Complexity     2664     2666       +2
============================================
  Files           115      115
  Lines          8050     8051       +1
============================================
- Hits           2621     2619       -2
- Misses         5429     5432       +3

Files Changed
Coverage Δ

...rm/lib/exceptions/RetailcrmJobManagerException.php
0.00% <0.00%> (ø)

retailcrm/retailcrm.php
31.98% <0.00%> (-0.24%)
:arrow_down:

retailcrm/lib/RetailcrmHistory.php
57.29% <100.00%> (-0.09%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add cproc to the testsuite

If only to check that fortify-header gracefully handle hobby-scale compilers.

common.h:32:22: error: volatile store is not yet supported


Some small changes to get the latest code base to build on VS2015

Note that while the tsARIBCharset change builds, I'm not sure if it was done correctly.

Hi Aaron,
Thanks, integrated.
The change is ARIBCharset is ok. Mainly it adds / converts a 32-bit Unicode code points into a UString, with UString containing Java-like UTF-16 codes.


sudoku notebook

this PR adds a notebook that explains and visualizes sudoku as a graph coloring problem using networkx
Reference: https://github.com/networkx/networkx/discussions/5085#discussioncomment-1435440
cc: @dschult @rossbar

Thanks @mjhajharia , I'm looking forward to taking a look at this!
In order to facilitate the review process, it'd be really helpful to have a pull request originating from a branch that reviewers (like me) can push to. This makes it easy to fix things that are related to the infrastructure (fixing the failing CI, for example) quickly so that we can focus on the notebook!
This PR looks like it originates from you main branch, which makes it difficult to track your remote branch locally. If you're comfortable doing so, would you mind submitting a new PR from a non-main branch? For example, you could do something like:
git checkout main  # Make sure you're on the main branch
git pull  # Make sure you're up-to-date with your remote
git checkout -b sudoku-notebook  # where "sudoku-notebook" is now the new branch name. You can choose whatever name you'd like!
git push -u origin sudoku-notebook  # create your remote branch

At that point, you should be able to open a new PR from the sudoku-notebook branch instead of main.
LMK if you're comfortable with this - if not, no worries! I'm happy to iterate and find a solution that works for everyone!


Keep controls and refs separate

Controls and variable references were treated as one concept in the case of View and Transform components. This is not correct since they have to be watched on completely different entities. This PR ensures they are treated as completely separate concepts.

Codecov Report

Merging #480 (60171b1) into main (aeebe78) will decrease coverage by 0.04%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main     #480      +/-   ##
==========================================
- Coverage   69.17%   69.13%   -0.04%
==========================================
  Files          78       78
  Lines        9083     9072      -11
==========================================
- Hits         6283     6272      -11
  Misses       2800     2800

Files Changed
Coverage Δ

lumen/transforms/base.py
83.54% <ø> (-0.36%)
:arrow_down:

lumen/views/base.py
64.69% <ø> (-0.44%)
:arrow_down:

lumen/layout.py
78.12% <66.66%> (-0.06%)
:arrow_down:

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


feat(icons): added start/select controller buttons [wip]

Two more for #6 .

[ ] Missing box-shadow for the start button
[ ] Could not make the start button with linear-gradient property, so it's a bit different from the others. Try in a different way to use linear-gradient if it's possible
[ ] Maybe add the bottom label start/select like the example

New classes: .start-label / .select-label


Added get_all_substrings_of_String_in_array(x) in strings.py

Proposed Changes
Add a short description here

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation. (In the main.py it requires to mention a new feature in the menu string)
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] I have checked that my PR is not a duplicate

Please have a look  @SahilKr24  @RKRohk   @sambitraze


[BUG] Unable to find security release for macOS 15.0

Describe the bug
17/09/2024 08:04:18 - INFO (nudge-auto-updater): Loading Nudge config...
17/09/2024 08:04:18 - INFO (nudge-auto-updater): Successfully loaded Nudge config!
17/09/2024 08:04:18 - INFO (nudge-auto-updater): Successfully loaded macOS release data from SOFA!
17/09/2024 08:04:18 - ERROR (nudge-auto-updater): Unable to find security release for macOS 15.0

Expected behavior
15.0 should be processed as a valid release

It looks like in the snippet below we make a comparison to match the security release info with the OS version:
https://github.com/jc0b/nudge-auto-updater/blob/1e29590375064407a4e383f74d16fd04239dccd8/nudge-auto-updater.py#L277
This check fails on the first release of a major version, because:
{
  "OSVersions": [
      {
        "OSVersion": "Sequoia 15",
        "Latest": {
          "ProductVersion": "15.0",
          ...
        }
      }
  ]
}

does not have the exact same ProductVersion as:
"SecurityReleases": [
        {
          "UpdateName": "macOS Sequoia 15",
          "ProductName": "macOS",
          "ProductVersion": "15",
          ...
        }
        ...
]

and 15.0 != 15, so the error is given.


scrollToLocation is not a function.

Describe the bug
I am trying to implement scroll to top function on tap of button to scroll on top position of swipe list view and trying to access method scrollToLocation from SectionList but I am not able to access it using the reference of SwipeListView. So I need help how to achieve it.Below is code from render function.
<SwipeListView
                        useSectionList={true}
                        sections={this.state.data}
                        keyExtractor={(item, section) => item.key }
                        renderItem={this.renderRow}
                        renderHiddenItem={this.renderHiddenItems}
                        renderSectionHeader={this.renderSectionHeader}
                        rightOpenValue={-75}
                        stopRightSwipe={-75}
                        previewRowKey={'0'}
                        closeOnRowPress={true}
                        previewOpenValue={-40}
                        onRowDidOpen={this.onRowDidOpen}
                        stickySectionHeadersEnabled={false}
                        bounces
                        disableRightSwipe={true}
                        enableEmptySections={false}
                        tension={20}
                        scrollsToTop={true}
                        onEndReachedThreshold={0.99}
                        onEndReached={() => {
                          this.setState({ isScrollToTopVisible: !this.state.isScrollToTopVisible })
                        }}
                        ref={(ref) => this.sectionListRef = ref }
        />

Code for scroll to top of list.
scrollToTop() {
   this.sectionListRef.scrollToLocation({ offset : 0, animated : true })
}

To Reproduce
Steps to reproduce the behavior:

Scroll to bottom of SwipeList View.
Tap on button at bottom of screen to perform scrollToTop.

Environment (please complete the following information):

OS: [iOS 12.1]
RNSLV Version: [v2.0.3]
RN Version: [v0.59.8]

I have made mistake for identify reference. Here is proper way to do it.
Replace
ref={(ref) => this.sectionListRef = ref }
with
listViewRef ={ ref => this.sectionListRef = ref }
And then, scrollToLocation works fine.
scrollToTop() {
   this.sectionListRef.scrollToLocation({
            animated: true,
            itemIndex: 0,
            sectionIndex: 0,
            viewOffset: 0,
        });
}

https://stackoverflow.com/questions/58704814/scrolltolocatoin-is-not-a-function-react-native-swipe-list-view

The only function that exists at listViewRef is scrollToEnd()


[Feature request] Regex functionality for find and replace

Regex functionality for find and replace, preferably PCRE2.

is library for PCRE2 secure?
also, this isn't that kind of editor. its meant to be a notepad replacement but within the limits of the chosen platform. So, better than notepad and worse than notepad; in different areas.

regards ratatoeey

Regex search is implemented by @soumyamahunt in this PR: https://github.com/JasonStein/Notepads/commit/5b2c7ed88553f690089bb12b345b18ba5ea03566
It will be available in Notepads v<IP_ADDRESS>


Add paging  and logging support for searching large numbers of issues

Client.SearchIssues() returns all issues, however, some queries have many results and it would be nice to be able to log queries to monitor progress.

Added zerolog usage. Zerolog is popular with over 9k GitHub stars, and slog is not designed to be as performant per the following:
https://github.com/rs/zerolog/issues/571


Changes User Error and Adds color to user display

This is still a beta and does not have the best user end experience.

If not, then the error variable and the stuff related to it is not needed.

If not, then the error variable and the stuff related to it is not needed.

I can not test the code right now. I will report my findings tommorow if possible.

I can not test the code right now. I will report my findings tommorow if possible.

The break is needed in the try statement so that it exits the error loop after entering the correct value.

The break is needed in the try statement so that it exits the error loop after entering the correct value.


Cv2pdf

Refactored CV, added script which generates a pdf file from the latest CV, and added a download button to the CV page to download that pdf.

Deploy is passing.


chore: add secret field in generator::config

Issue Reference(s):
Fixes #2654
/claim 2654

closing in favour of #2656


Q440 keyword type / key is not uppercase

Hey there, really cool extension to flake8 😄
I believe it is reporting some false positives when table field names are type or key.
e.g.
./elite/libraries/launchpad_builder.py:195:0: Q440 keyword type is not uppercase
./elite/libraries/launchpad_builder.py:222:0: Q440 keyword type is not uppercase
./elite/libraries/launchpad_builder.py:247:0: Q440 keyword type is not uppercase
./elite/libraries/launchpad_builder.py:271:0: Q440 keyword type is not uppercase
./elite/libraries/launchpad_builder.py:296:0: Q440 keyword type is not uppercase
./elite/libraries/launchpad_builder.py:343:0: Q440 keyword type is not uppercase
./elite/libraries/launchpad_builder.py:352:0: Q440 keyword key is not uppercase
./elite/libraries/launchpad_builder.py:372:0: Q440 keyword type is not uppercase
./elite/libraries/launchpad_builder.py:396:0: Q440 keyword key is not uppercase
./elite/libraries/launchpad_builder.py:472:0: Q440 keyword key is not uppercase
./elite/libraries/launchpad_builder.py:472:0: Q440 keyword key is not uppercase

And here are some related lines:
Lines 192 - 196:
            cursor.execute('''
                INSERT INTO items (rowid, uuid, flags, type, parent_id, ordering)
                     VALUES (?, ?, 2, ?, ?, ?)
            ''', (group_id, generate_uuid(), Types.PAGE, root_parent_id, page_ordering + 1)
            )

Lines 348 - 353:
        cursor.execute('''
            UPDATE dbinfo
               SET value = 1
             WHERE key = 'ignore_items_update_triggers'
        ''')
        self.conn.commit()

Any help is greatly appreciated 😄
Fotis

Thanks so much @pgjones, I can confirm this works as explained 😄


[Cloud Security] Licensing Update

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #
Test URLs:

Before: https://main--prisma-cloud-docs--hlxsites.hlx.page/
After: https://--prisma-cloud-docs--hlxsites.hlx.page/
Worker: https://prisma-cloud-docs-production.adobeaem.workers.dev/?branch=

Content updates provided by Jason & signed-off by Walter


Fix version range for bs-json dependency.

The current version won't build with bs-json 5.0.0.

thanks!


3D Instance segmentation

Hi,
is it possible to modify this repo for instance segmentation? I would like to segment single class and differentiate objects in slices of 3D volume. please advise
thank you

It's not trivial to get image segmentation out of this. It's for semantic segmentation only.


Fix "jumping" when going from home to categories

Apparently, the scrollbar messes with the margin: 0 auto calculation.

Thanks!

Thanks!


[Documentation]: Proxy setup instructions

Rum users may want to set up a proxy server--currently this documentation is missing.

Avoid adblockers since the default rum endpoints is recognized by ublock origin, adblocker, etc.
Setup their own analytics

@williazz Documentation for this would be very handy. We proxy our front end through a caddy server to enable us to white-label our customer portal with client-specific domains but this causes all of our RUM requests fail. As there is no current documentation on this, can you advise how to proxy RUM requests so that AWS will pick them up?

We may also need to add support to include custom headers for the proxy setup. Currently it's not exposed and could be required to support authentication/CSRF .etc


rtv: 1.21.0 -> 1.22.1

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
ran /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1/bin/.rtv-wrapped -h got 0 exit code
ran /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1/bin/.rtv-wrapped --help got 0 exit code
ran /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1/bin/.rtv-wrapped -V and found version 1.22.1
ran /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1/bin/.rtv-wrapped --version and found version 1.22.1
ran /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1/bin/rtv -h got 0 exit code
ran /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1/bin/rtv --help got 0 exit code
ran /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1/bin/rtv -V and found version 1.22.1
ran /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1/bin/rtv --version and found version 1.22.1
found 1.22.1 with grep in /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1
found 1.22.1 in filename of file in /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1

cc @matthiasbeyer @jgeerds for review

No attempt on x86_64-darwin
The following builds were skipped because they don't evaluate on x86_64-darwin: rtv
No log is available.

Success on x86_64-linux (full log)
Attempted: rtv
Partial log (click to expand)
/tmp/nix-build-rtv-1.22.1.drv-0/source
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1
gzipping man pages under /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1/share/man/
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1/lib  /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1/bin
patching script interpreter paths in /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1
checking for references to /tmp/nix-build-rtv-1.22.1.drv-0 in /nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1...
wrapping `/nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1/bin/rtv'...
/nix/store/mpyc1z74lsma9vfc4b04glvvyfs0l40j-rtv-1.22.1

Success on aarch64-linux (full log)
Attempted: rtv
Partial log (click to expand)
/build/source
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/cdrj9mq20cm8hxw4xg7l0mj2jbzab8lf-rtv-1.22.1
gzipping man pages under /nix/store/cdrj9mq20cm8hxw4xg7l0mj2jbzab8lf-rtv-1.22.1/share/man/
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/cdrj9mq20cm8hxw4xg7l0mj2jbzab8lf-rtv-1.22.1/lib  /nix/store/cdrj9mq20cm8hxw4xg7l0mj2jbzab8lf-rtv-1.22.1/bin
patching script interpreter paths in /nix/store/cdrj9mq20cm8hxw4xg7l0mj2jbzab8lf-rtv-1.22.1
checking for references to /build in /nix/store/cdrj9mq20cm8hxw4xg7l0mj2jbzab8lf-rtv-1.22.1...
wrapping `/nix/store/cdrj9mq20cm8hxw4xg7l0mj2jbzab8lf-rtv-1.22.1/bin/rtv'...
/nix/store/cdrj9mq20cm8hxw4xg7l0mj2jbzab8lf-rtv-1.22.1


Use version of debug which comes with dist

versions of debug after 2.4.4 no longer come with a dist folder, which log-behavior relies on.

thx


Issue #123 | Remove mixpanel related  code (master)

Fixes #123 (and add BROWSER_NAME and UPDATE_URL in env files)

Mixpanel is removed and all env variables are working as expected LGTM


Strange Signals from ATV

Getting very high spikes in Current from ATV

Runtime Signals do not make sens

@MrNipple  is this problem still happening?


Add started time to WorkflowInfo

Currently there is no way to find out the workflow start time from the workflow code.
Putting workflow.Now at the beginning doesn't work if the first decision was delayed due to worker downtime.

Also consider other fields recently added to the WorkflowExecutionStarted event

Currently not implemented
https://github.com/uber-go/cadence-client/blob/2d4612d32cf9fd6a0dc45d4e66c2a99492774ee1/internal/workflow.go#L1106-L1129


fail hard when enabling deprecated addons

Now when using the deprecated settings, warning messages would be printed but installation would still proceed. For addons, we remove the charts so we should fail hard because it does nothing when enabling them.
part of the work for: https://github.com/istio/istio/issues/25701

@richardwxn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
b43b07d537b53ec9f4b4ae95700cd27ed040f069
link
/test release-notes_istio

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

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
b43b07d537b53ec9f4b4ae95700cd27ed040f069
link
/test release-notes_istio

unit-tests_istio
b43b07d537b53ec9f4b4ae95700cd27ed040f069
link
/test unit-tests_istio

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

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
b43b07d537b53ec9f4b4ae95700cd27ed040f069
link
/test release-notes_istio

unit-tests_istio
b43b07d537b53ec9f4b4ae95700cd27ed040f069
link
/test unit-tests_istio

integ-multicluster-k8s-tests_istio
b43b07d537b53ec9f4b4ae95700cd27ed040f069
link
/test integ-multicluster-k8s-tests_istio

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

/retest

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
b43b07d537b53ec9f4b4ae95700cd27ed040f069
link
/test unit-tests_istio

integ-multicluster-k8s-tests_istio
b43b07d537b53ec9f4b4ae95700cd27ed040f069
link
/test integ-multicluster-k8s-tests_istio

release-notes_istio
b43b07d537b53ec9f4b4ae95700cd27ed040f069
link
/test release-notes_istio

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

@richardwxn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
485b2c93427890656860d68acb0051f135d182f2
link
/test release-notes_istio

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

@richardwxn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
cb1bae5b4bf6f9a9d1fb7fffcfebf93dc48b2c3f
link
/test unit-tests_istio

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

/retest


Failed to download manifest

i can't update the game or download any dlc ( Failed to Download Manifest )

If you still have the issue come to Discord so we can test few things with more convenient way of communication

me too, what the discord?


Add clock support to BBC BASIC

Via TIME$ virtual variable
In sorry.asm; GETIMS and PUTIMS

Testing:

in mos.c, doesn't seem to wait for the time to be returned from vdp_protocol


Improve iNat names => MO Names

Some fine tuning, including edge and unusual cases:

[ ] Subranks. Ex: https://www.inaturalist.org/observations/215997271
[ ] Map iNat names to MO when MO has homonyms.
Ideas:

Ignore MO Names sensu whatever
If there's only one MO preferred Name, use it.
Can I map using both author from iNat taxon_id (requires 2nd iNat API request) and MO Name.author
Consider a Dictionary to translate iNat Names to MO Names?

[ ] Name s.l. (not an iNat taxon, can never be an iNat identification). Add a Naming if MO Name s.l. exists. (I don't want to create new s.l. because MO homonyms create problems.)
[ ] Improve handling of provsisionals. Things like "Mollisia aurantioviolascens" should import as "Mollisia aurantioviolascens nom. prov." rather than "Mollisia “sp-aurantioviolascens” crypt. temp." Ex: https://www.inaturalist.org/observations/190105416. Possible algorithm: if the epithet is all l.c. letters, then make it nom. prov.

iNat ranks and rank_levels through species for Fungi.
Not complete -- omits some secondary ranks, infrageneric ranks

rank
level

kingdom
70

phylum
60

subphylum
57

class
50

order
40

family
30

genus
20

species
10


Failure in starting the application

Anyone experiencing the following issue? I am encountering this issue now but never experienced this before, I checked online and people said this is an issue with the port:
Listening on http://<IP_ADDRESS>:9987
Error in startServer(host, port, handlerManager$createHttpuvApp()) :
Failed to create server

It seems the error is you already started a webpage using port 9987

Thanks, resolved.


WIP of linux 54 branch

[x]  Hack and Slash to get it to compile
[x] Move gtk and dbus dependency out of chrome (Some technical debt i created)
[ ] Identify what native window work to do.
[ ] Hack & slash to link
[ ] Re-enable widevine
[ ] Re-enable breakpad

Re-enable widevine doesn't need to block for 54 upgrade since it's intentionally disabled on Linux.  The component updater that we use on mac and Windows doesn't have binaries for Linux, so that needs to be added with the alt method of HTTP download location which we haven't done yet.  So basically I'd remove that checkbox for this 54 tracking task.


Name steps

Applied mkpr on .github/workflows/ci.yml
sed s/run: npm run cover --if-present/name: test and coverage\n        run: npm run cover --if-present/

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.842%

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
416

Relevant Lines:
426

💛  - Coveralls


Added Falling Blocks Game

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes #478

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @thakurdiwakar ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@kunjgit check

Thank you  @thakurdiwakar , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Alarms tests and small fixes alarms_api

Alarms tests and small fixes alarms_api.py.

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/pull/69


[PE-D][Tester C] UG: Upper menu bar only briefly mentioned

UG briefly mentions: "6. Export of transactions in csv format. For those who need to share data with stakeholders, Tran$act allows you to generate CSV report." and once more towards the end in the FAQ, but does not actually delve into how to export the information. Briefly touched upon that 'the upper menu bar contains tabs', but export feature not explicitly mentioned. 'Tools' tab should be included as well
You may want to consider user-input commands such as export s/ or export t/ as well.

Labels: type.DocumentationBug severity.Medium
original: phiphi-tan/ped#15

Good to add for CS2101 UG


Fix /me and /announce in PM messaging

Morfent's changes to /invite accidentally broke /me and /announce since
they ran off the single break command that was there before. This fixes
that by giving them back a break command.
Also updated PM /invite to use targetCmdIndex in case other aliases to
/invite are created in the future; these will break with a hardcoded value
in that substring command.

#1406 makes this redundant.


make a home page

Let me know the colour contrast of the homepage(minimal or vibrant) , then I can start working on the homepage by

Adding different sections on the top to redirect
Adding login and signup options
Adding the option to zoom previews before getting redirected to particular pages
Adding a slideshow preview of each club by dividing portions of the page accordingly
Fitting the homepage properly for both desktop and mobile with proper formatting
Adding Slideshow of recent remarkable activities

if you want anything more please inform me as per your convenience

I think keeping the color palette minimal would be better


2016.09.08 engine docs updates

Cherry picks relevant docs updates to 1.12.x branch.

@sfsmithcha @vdemeester @SvenDowideit PTAL
cc @tiborvass @thaJeztah

LGTM
note: update to the man page won't affect anything with the docs build. that only grabs markdown from the docs directory.

LGTM


[Linter] Error running Flake8

I am new to python. Follow a YouTube Video set up Atom, however, when I started coding it keeps popping these error messages:

Python Version is 3.8.2, using a Mac laptop. I installed linter-flake8 in Atom and followed all instructions.
Could someone please give me a step-by-step detailed instruction on how to fix it, please.
I really appreciate all y'all help!

Hi,
Could you tell us what is the output of commands:

which flake8
flake8 --version

Have you installed flake8 for python3.8 with taking care of pip version ?
i.e. using python3.8 -m pip install flake8? (to test if installed: python3.8 -m flake8 --version)
In the options of linter-flake8 atom package, there is the path of bin flake8 to add. You should put the absolute path or adding the path of bin directory containing it in your .bashrc or in your .bash_aliases

I have the same problem, but for a Windows 10 laptop.
When I write command

flake8 --version

I get this:

3.7.9 (flake8-docstrings: 1.5.0, pydocstyle: 5.0.2, mccabe: 0.6.1, pycodestyle: 2.5.0, pyflakes: 2.1.1) CPython 3.8.2 on Windows

But when I write the command

which flake8

It says that

'which' is not recognized as an internal or external command,
operable program or batch file.

But when I write the command
which flake8

It says that
'which' is not recognized as an internal or external command,
operable program or batch file.

Yes sorry, it was for bash language (Unix)

where.exe flake8
in another thread, but what it gives me is this:
INFO: Could not find files for the given pattern(s).

While flake8 --version works, hum, I don't really know how this is possible (perhaps the flake8 file is not an executable, but a python script as on Unix so...)
But normally, if flake8 --version works, then it is in your PATH environment variable so get the absolute path of flake8 and give it to linter-flake8 option Executable Path
https://www.shellhacks.com/windows-cmd-path-variable-add-to-path-echo-path/
(sorry, I'm not a windows programmer ^^)
Could you test just flake8 on an erroneous file for testing if it works properly first ?
Do you use a virtual env ?

Too, verify your python-language-server version with python3.8 -m pip freeze

What I get doing this:

But normally, if flake8 --version works, then it is in your PATH environment variable so get the absolute path of flake8 and give it to linter-flake8 option Executable Path
https://www.shellhacks.com/windows-cmd-path-variable-add-to-path-echo-path/
(sorry, I'm not a windows programmer ^^)

Is this path: C:\Users\arfel\AppData\Local\atom\bin
But I had already put it in the Executable Path option and I am still getting the same error when I open any file.
With this:

Too, verify your python-language-server version with python3.8 -m pip freeze

I get this info:
autopep8==1.5.1
entrypoints==0.3
flake8==3.7.9
flake8-docstrings==1.5.0
mccabe==0.6.1
pycodestyle==2.5.0
pydocstyle==5.0.2
pyflakes==2.1.1
snowballstemmer==2.0.0

Is this path: C:\Users\arfel\AppData\Local\atom\bin
It is not the path of the directory containing the flake8 executable script, but this of atom :p

But I had already put it in the Executable Path option and I am still getting the same error when I open any file.
So normal

Have you just kept flake8  in linter-flake8 option Executable Path ?
I don't understand why flake8 --version in terminal (Powershell ?) works while linter-flake8 can't find flake8.

I get this info:
autopep8==1.5.1
entrypoints==0.3
flake8==3.7.9
flake8-docstrings==1.5.0
mccabe==0.6.1
pycodestyle==2.5.0
pydocstyle==5.0.2
pyflakes==2.1.1
snowballstemmer==2.0.0
But you don't have python-language-server
So python3.8 -m pip install python-language-server

Hi,
Could you tell us what is the output of commands:

which flake8
flake8 --version

Have you installed flake8 for python3.8 with taking care of pip version ?
i.e. using python3.8 -m pip install flake8? (to test if installed: python3.8 -m flake8 --version)
In the options of linter-flake8 atom package, there is the path of bin flake8 to add. You should put the absolute path or adding the path of bin directory containing it in your .bashrc or in your .bash_aliases

Hi,
Could you tell us what is the output of commands:

which flake8
flake8 --version

Have you installed flake8 for python3.8 with taking care of pip version ?
i.e. using python3.8 -m pip install flake8? (to test if installed: python3.8 -m flake8 --version)
In the options of linter-flake8 atom package, there is the path of bin flake8 to add.  Thank you.


Some ferry lines(?) freeze the map layer

Some routes in Norway (I guess if they belong to a ferry line where some keys are missing) freeze the map layer. You cannot drag it with the mouse anymore and clicking in it does not work.
An example is this link: https://brouter.m11n.de/#map=12/60.0736/5.4358/CyclOSM,Waymarked_Trails-Cycling,Smoothness,route-quality&lonlats=5.436859,60.161993;5.335922,59.942872&profile=m11n-gravel
Also, the track remains gray.
For ferry connections in denmark I see no problem, also the heightdata of the ferry line is shown.
Reproduced with Firefox and Qutebrowser (QtWebEngine/Blink).

Seems like a bug with the route quality color coding.
Workaround until someone finds the time to fix it: Disable the route color coding, see screenshot for the button state which works fine.

.


automatic vs manual

When I first downloaded the program I used the settings python main.py -user
A consistant problem I found was that around 10-20% files were "too long" in both vlc and could not be uploaded to youtube. The length of the actual recording did not seem to matter, I converted them in vlc and it worked but took time.
I had the same problem when I used the website tiktokrecorder that eventually stopped working, so its not specific to this program.
I later used the settings python main.py -mode automatic -ffmpeg -user
all files could be opened normally.
Now lives that were lagged would start recording again automaticly.
But I noticed now that several recordings had around 10-30 seconds jumpcuts in them. Recordings that should not have any lag.
I assume the live convertion with ffmpeg is causing the issue since I did not notice anything without it.
In case I have missed any setting, is there any way I can ensure a automatic ffmpeg recording does not have these jumpcuts?
Or is it possible to have automatic recording without the ffmpeg?

The jumps could be caused by lag in your connection or by the live host


Service charters can not be updated in Sunstone

Description
Service charters can not be updated in Sunstone.
To Reproduce
Try to update service charters.
Details

Affected Component: Sunstone
Version: 6.0

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

If the VM charters are updated individually, the service charter is not updated. This should be covered in the issue.

@tinova PR to approve:

Code: https://github.com/OpenNebula/one-ee/pull/1246
Docs: https://github.com/OpenNebula/docs/pull/1851
Tests: https://github.com/OpenNebula/development/pull/1528


[Feature] EPR Checker

Changes
Attributes

Add a module-level attribute #[verifier::epr_check] to run the EPR checker on all proofs in a particular module.

This module also enables options to turn on MBQI for the quantifiers in this module.

Add verifier::inline_only attribute to suppress generation of the function body in the Verification Condition
Add #![no_triggers] trigger annotation to produce an empty pattern. Together with pi.enabled this will effectively eliminate trigger based instantiation for the given quantifier (i.e. MBQI only)

EPR Core Logic

Added new file alternation_check.rs that runs a 3 pass check for quantifier alternation.

First pass to check if all features used in module are supported in EPR
Second pass to collect all recursively mentioned spec functions
Third pass to construct the edges in the quantifier alternation graph

Augment pruning logic in prune.rs to also return a boolean saying all reached types are opaque datatypes.
Add a new EPR-friendly prelude that is produced when the module is decorated with epr_check

Testing

Added epr.rs to test harness to test interesting cases of the quantifier alternation graph generation
Add delmap_epr.rs to examples/ to further test feature and show an example of code organization around the EPR feature

What's the difference between inline and inline_only? Is the original inline conditional on something?

What's the difference between inline and inline_only? Is the original inline conditional on something?

I think the original inline will not suppress the creation of the function definition in the VC. Since we care about reducing the QA graph by eliminating the function defn (defn(x) <==> body), we added this option.


[Question] Compatibility with react-native

Hey! Great package. I was wondering if its compatible for react native!

Hey! Great package. I was wondering if its compatible for react native!

Thank you 😄
Sorry I didn't try


Add an explicit abort listener.

This explicit abort listener will improve handling of errors
and time outs for this OutRequest instance in case the only
"event" that fires is the abort event.
Otherwise we may end up with a "callback not called" situation.
r: @Matt-Esch @rf
cc: @anandsuresh , applying your fix as well in a seperate PR.

👍


Solr indexing branch merge

These are just changes for changed gems from the stash repository for the corresponding branch (solr-indexing).

just restarted that PR build

I"m closing this since it seems like it may not be needed anymore.  My stuff is running OK without it.


feat: add set_file_path and set_base64_path functions

Add set_file_path and set_base64_path functions to allow the use of the [BASE64:xxxx] and [FILE:xxxx] mapping patterns specifying a path that is relative to a base path previously set.
The current behaviour remains if no base path is set, i.e. the path specified in the pattern can be an absolute path or a path relative to the Python process execution path.

Coverage increased (+0.01%) to 88.97% when pulling d0c749af6a713073ecfa4bddb3106835d9b7d41a on feat/add_set_file_path_and_set_base64_path_functions into e00115ad009067203e506eeac12a84b8efb4edd3 on master.


Crashes when trying to restart with --log

forever-d --log log.txt --err errlog.txt ./app &
killall app
-- 2017-Feb-09 09:24:03.61352 -- Process Ended. Exitcode: 0
-- 2017-Feb-09 09:24:03.613601 -- Starting: './app'
object.Exception@/build/ldc-I3nwWj/ldc-0.17.1/runtime/phobos/std/stdio.d(1709): Attempting to call getFP() on an unopened file
----------------

[1]+  Exit 1                  forever-d --log log.txt --err errlog.txt ./app

And app didn't restart, instead forever-d crashed

ok i will look into that, as a workaround you can run it with forever-d ... > server.log &

I think it worked once, not sure if I used pipes when it worked with restarting. But I had this bug basically since I started using forever-d, which was really destroying the point of "forever"

should work now, i could reproduce it and it works now... dont ask me what changed in phobos but it used to work before...


Create SingletonList Operation

Sometimes it is useful to be able to chain operations that produce a single output with operations that expect an iterable input (Selecting the first element from an ordered list, and then using it to seed a subsequent query, for example). Creating a 'SingletonList' operation that wraps an input object in a list and presents it to the next operation would enable this.

Can the current ToList operation not just be used for this?

Merged into develop.

Reopened for missing handler

Merged fixes into develop.


pykafka.Cluster.update() needs to be thread-safe

It's not right now, but there's no telling where/when this is going to be called. We'll have to figure out what the implications of a reconfiguration are and how locking should occur.

Related to https://github.com/Parsely/pykafka/issues/110

Going to mark this resolved.  This was opened because thread-safety for samsa was a challenge, but looking the code most of those issues aren't present here. Those were mostly around using .itervalues() and things like that, which we just don't do on objects that change during an update.
Having the broker be thread-safe and the cluster update in-place really should be sufficient to call this thread-safe.
I'm sure at some point testing will prove me wrong, of course. ;)


Interpolate points of a given mesh from nifti DVF image volume

Hi,
I have generated a volumetric mesh and saved it as a vtkUnstrcturedGrid (i.e. a .vtk file). Here I extracted a volumetric mesh from a nifti image which has a dimension of (203, 127, 161). I have then applied a non-rigid deformation to this nifti file and saved the deformation vector field as another nifti file. The shape of this DVF nifti mage is (203, 127, 161, 1, 3). Here the deformation field is the final position of the voxel after it has been transformed, i.e. it is the initial position of the voxel + the displacement of the voxel.
Now I need to distort the volumetric mesh based on this deformation vector field and save ita gain as a .vtk file. How can I accomplish this using vedo ? Could you please give an example ? It is possible to attach a sample nifti DVF and a voluemtric mesh for your further reference.
I highly appreciate your willingness to help me out on this issue.

Hi, that sounds complicated :)
If you already have the displacements you can simply try:
myugrid.points(myugrid.points()+displacements)
E.g.:

from vedo import *

ug1 = UGrid(dataurl+'ugrid.vtk')
# ug1 = UGrid('ugrid_new.vtk')

ug1.points(ug1.points()+[1,1,1])
# print(ug1.points())

# write it as unstructured grid
# import vtk
# writer = vtk.vtkUnstructuredGridWriter()
# writer.SetInputData(ug1._data)
# writer.SetFileName('ugrid_new.vtk')
# writer.Update()


Update overlay color

Opening pull request

overlay color is already done in the JS does not need to be done in the CSS

LGTM


RobiniaDocs API Explorer

Maybe You will be interested in hosting API Explorer online:
https://www.robiniadocs.com/d/vanara/api/Vanara.PInvoke.html
This is ready-to-go, so above link can be added to README.md for other people to see it.
Or using ready shields.io badge:

[![Static Badge](https://img.shields.io/badge/API%20Documentation-RobiniaDocs-43bc00?logo=readme&logoColor=white)](https://www.robiniadocs.com/d/vanara/api/Vanara.PInvoke.html)

https://github.com/NeuroXiq/RobiniaDocs
Besides above, I saw that Your projects has so many supported libraries that I didn't
included them as an example documentation, I decided to take a few and generate documentation:
Vanara.Kernel32 - RobiniaDocs
Vanara.WebSocket- RobiniaDocs
Vanara.Gdi32- RobiniaDocs
Hope You will find RobiniaDocs useful for this and other projects
Regards
NeuroXiq

Cool project. Thanks for the info!


3D MapView

How can i enable 3D mapview?
@Kristian Fallrø @Nick Italiano

This is an issue tracker, not a place for seeking general purpose help. Use stackoverflow for those things.

Thanks for your quick reply..!
i Checked every where is there any option to enable 3d map View in react-native-mapbox-gl  ?

@HemanthSurimani did you ever figure this out? i've been workin on trying to get 3d maps to work for days.

@taylorjosephgriffin Are you suggesting this is a new feature in the iOS SDK from Mapbox?

@systemlevel what do you mean? I was suggesting that I was having a hard time figuring out how to get a 3D view map with this package. But I got it working using mapbox studio.

@taylorjosephgriffin That's great to hear.  I was curious what you meant by "3D".  Are you just referring to the 3D models that can be placed or are you referring to a new 3D mapping feature highlighted by mapbox :

@systemlevel i was actually just referring to using height data to show 3D buildings

@taylorjosephgriffin  Can you share any link or snip, how and where to add mapbox studio styles in this package.
Thanks in advance.

any solution for this ?
i got this black screen when i use custom style uri


Lower e2e permission check

Codecov Report

Merging #13 (6527937) into main (b2a1bcb) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #13   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           11        11
=========================================
  Hits            11        11

Continue to review full report at Codecov.

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


Copying an activity redirects to the iframed editor [#99311988]

Note: there is a parallel LARA PR to support this feature (https://github.com/concord-consortium/lara/pull/117)

Looks good. :+1:


Bug of update if the replace parameter is true when use FasterEMA

The function setResult In indicator.ts will export wrong value while the 'replace' parameter is true, beacuse 'this.previousResult' always replaced by 'this.result'. If 'replace' is  true in continuous calls to 'update',  you will get wrong result after second call
update(_price: BigSource, replace: boolean = false): Big {
    if (!replace) {
      this.pricesCounter++;
    } else if (replace && this.pricesCounter === 0) {
      this.pricesCounter++;
    }
    const price = new Big(_price);

    if (replace && this.previousResult) {
      return this.setResult(
        price.times(this.weightFactor).add(this.previousResult.times(1 - this.weightFactor)),
        replace
      );
    }
    return this.setResult(
      price.times(this.weightFactor).add((this.result ?? price).times(1 - this.weightFactor)),
      replace
    );
  }

  protected setResult(value: number, replace: boolean = false): number {
    if (replace && this.previousHighest !== undefined) {
      this.highest = value > this.previousHighest ? value : this.previousHighest;
    } else if (this.highest === undefined || value > this.highest) {
      this.previousHighest = this.highest;
      this.highest = value;
    }

    if (replace && this.previousLowest) {
      this.lowest = value < this.previousLowest ? value : this.previousLowest;
    } else if (this.lowest === undefined || value < this.lowest) {
      this.previousLowest = this.lowest;
      this.lowest = value;
    }

    this.previousResult = this.result;
    return (this.result = value);
  }

Got correct result after change to this
    if(!replace){
      this.previousResult = this.result;
    }

Thanks for the hint and the fix. Do you have a little test case in place which I can use? I think I am fixing the issue with #679

Hi @nicewookyes, I found the issue and fixed it: https://github.com/bennycode/trading-signals/pull/679
I also added test cases for the EMA replace behaviour: https://github.com/bennycode/trading-signals/pull/683
Please install<EMAIL_ADDRESS>and let me know if it works. 😀


patches for Windows

With these patches, i was able to run the test script below on Windows, using CUDA 8 and cudnn 5.1. The file RNN.lua would also need to be patched to avoid the LongTensor issue.
require "torch"
local cudnn = require 'cudnn'
local X = torch.rand(320, 3, 240, 240):cuda()
local net = nn.Sequential()
net:add(cudnn.SpatialConvolution(3, 8, 5, 5))
net:cuda()
local output = net:forward(X)

thanks!

Hi @elikosan , will you also patch RNN.lua (and init.lua)?
Hi @soumith , I just want to double confirm that memory was managed by torch by using LongTensor, but now the memory is not managed by torch any more by using ffi. It is not a concern for this kind of temporary integer variables.

@BTNC the memory in question is a few bytes, so i think it's not an issue

Hi @BTNC, i commited a patch and created a pull request.


Nested classes are resolved as IStruct

Libraries
MarkDoc.Members.Dnlib
Describe the bug
Nested classes are resolved as IStruct
Reproducibility
Always
To Reproduce
Steps to reproduce the behavior:

Have a test assembly with the following type structure:

A Parent class
A nested class within the parent class

Resolve the test assembly
Observe the type of the resolved nested class

Expected behavior
The nested class is expected to be resolved as an IClass
Additional context
Revealed in UT.Members tests

Total: 4m


Removing faulty form custom body json

When saving a form custom body json in a list form on Sharepoint online that renders the whole list form blank, there seems to be no way to reset or remove the json again to the Sharepoint default form layout.
I inserted json with a handful of sections listing all list fields and when previewing it rendered a blank form with no list fields and not showing the input field for the json- which I accidentally saved! My bad! Now I don’t seem to have a way to remove the faulty Json, as when opening the form it’s blank with no possibility to see any list fields or any customisation possibilities.
I desperately need access to editing the list data again with the list form.
best regards
Peter

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

ID: 0f6aeedd-a76d-88b0-7ed5-93e20ac218bc
Version Independent ID: 64315895-1103-d5f6-0195-9b5b61211132
Content: Configure the list form
Content Source: docs/declarative-customization/list-form-configuration.md
Product: sharepoint
GitHub Login: @VesaJuvonen
Microsoft Alias: vesaj

Can you clarify what you mean? Are you opening the form from the "New" button, or opening an existing item in your list.

Are you saying the configure layout button is now missing?

Regrettably I've suffered this exact same fault. MS are telling me this is "out of support for SharePoint online" which I am simply dumbfounded by. How can it be possible to nuke functionality of editing/viewing list data through a mistake in the JSON and for support to wash their hands of it?
I'm hopeful that someone here may have some ideas on how to resolve.
To answer @bcameron1231 the configure layout option does appear at the very top of a blank form where previously all my datafields were displayed. Clicking it opens a further blank form over the top, you have to click the X close button twice to get back to the list.
If @djbove was able to solve this I'd be very interested in the method.

I guess I'm trying to repro the issue. Do you have a sample JSON that I could try out?

It was something along these lines, very close to the example directly linked from the site. The tweak I made at the very end was to only add a displayname section without the fields, anticipating it would leave a larger format header in bold text above the last section which lists attachments. I strongly suspect this broke it, but it did of course pass whatever limited parsing of the JSON that the system does to validate it. The tweaks I had made prior to this were working fine. I will add this is my best attempt to recreate it from cut/paste history and isn't exactly as was used - I don't have a copy of that. I was lulled into a safe place just figuring I could play around tweaking it until I got the result I needed! Very appreciative of you taking the time to look at this.
{
"sections": [
{
"displayname": "Pack Details",
"fields": [
"Reference",
"Pack Serial Number",
"Pack No"
]
},
{
"displayname": "Investigation Stage",
"fields": [
"Status",
"Customer Fault Description",
"Steatite Diagnosed Fault/s"
]
},
{
"displayname": "Diagnosis",
"fields": [
"Diagnosis"
]
},
{
"displayname": "More Fields",
"fields": [
"Repairs Reqd",
"Root Cause"
]
},
{
"displayname": "File Attachments"
]
}
]
}

Okay,  great. Just so you know that JSON isn't valid... you have an extra ] bracket after file attachments.
Now, it appears the issue is because you are missing the "fields" values under displayname... so you haven't specified any, thus it is breaking and showing a blank screen.
What it appears, is that the JSON doesn't actually save. But you are right, it renders a blank screen.
But this doesn't stop me from re-configuring my layout. You need to hit the 'X' in the top right, and then select "Configure Layout" again. You can't select Configure Layout from the Configure Layout Screen (frankly, that button should be hidden in my opinion).

As I posted it I figured the JSON was wrong, it's obvious in hindsight of course! The tricky part is that it did save the broken JSON and now has stuck without giving me any way to correct it. Even creating new items in the list using the +New button results in a blank form now that can't be filled in.
Workaround at the moment is to use the Classic Experience which then does allow viewing/editing.
Also note that if the (i) button is clicked when an item is selected to open the details pane, this fails with a "Something went wrong, Cannot read property 'indexOf' of undefined" error.
Customize with Power Apps does work and may be the unpleasant way out of this, by creating a PowerApp to render the form data but this is a huge step up in complexity when all I wanted was to rearrange some fields!

Alright, I have been able to reproduce on another list. It does seem there are scenarios where you add JSON to the page, which is valid JSON but does not match the schema of the formatting required, it can save and it stops it from rending.
@chakkaradeep I'm not sure if you are aware of this issue.... but it does pose a problem.
I've been able to reproduce by adding a JSON format to the body  section of a form. I include a displayname, but do not add FIelds property. It is respected as valid JSON, it will save, but it will completely break the form.
I do notice errors on the browser dev console that show up when loading the broken form.

@markpandrew  and @djbove  I do have a fix.... albeit it's a little convoluted.  But we can fix this using the REST API.

Update the URL to your Tenant and Library Below
Update the GetbyTitle('Demo') to the display name of your list
Grab the content type from your list
a. Go to list, advanced, enable content types
b. Open up content type, and grab the Content Type ID from the URL
c. Replace Content Type in lower command
Open up developer tools in your Browser using F12
Go to the Console, paste in the updated code below with your settings and hit enter.

fetch("https://YourTenant.sharepoint.com/sites/JSON-Formatting/_api/contextinfo", {
  "headers": {
    "accept": "application/json;odata=verbose",
    "accept-language": "en-US,en;q=0.9",
    "content-type": "application/json;odata=verbose",
 },
  "method": "POST",
  "mode": "cors",
}).then(response => response.json())
.then((data)=>{

    fetch("https://yourtenant.sharepoint.com/sites/JSON-Formatting/_api/web/lists/getbytitle('Demo')/contentTypes('0x0100F566899D669FFB4D9CA9135D63C98EAB001D365D0D20AF3E478A1E321ACB5C901F')", {
      "headers": {
        "accept": "application/json;odata=verbose",
        "accept-language": "en-US,en;q=0.9",
        "content-type": "application/json;odata=verbose",
        "x-requestdigest": data.d.GetContextWebInformation.FormDigestValue
      },
      "body": "{\"__metadata\":{\"type\":\"SP.ContentType\"},\"ClientFormCustomFormatter\":\"\"\}",
      "method": "MERGE",
      "mode": "cors",
      "credentials": "include"
    });

}
);

What this code does, is get a digest token in order to be able to update your content type formatting. Then, the second request sets the ClientFOrmCustomFormatter to empty and removes the formatting you have in place.

@bcameron1231 Just wow, what brilliant effort, thank you so much. Not quite there for me, but presumably a mistake on my part. Here's what I did. Went to List Settings from the cog top right, then Advanced settings, turned on "Allow management of content types?" to Yes. Back to List Settings and clicked "Item" hyperlink underneath new Content Type table. URL of this page is:
https://mycompany.sharepoint.com/sites/power/_layouts/15/ManageContentType.aspx?List={29527988-D3BE-4866-9130-1445B696B65A}&ctype=0x010047A5DDD91CE4F5478FBE51E112431EEC
so pasted 0x010047A5DDD91CE4F5478FBE51E112431EEC into second fetch command to replace what is there in the brackets, i.e.  fetch("https://mycompany.sharepoint.com/sites/JSON-Formatting/_api/web/lists/getbytitle('Customer 400 Returns')/contentTypes('0x010047A5DDD91CE4F5478FBE51E112431EEC')", { and so on.
Note my site list name has spaces unhelpfully, should this be replaced with %20 instead?
Anyway, when pasted into F12>Console using Edge or Chrome results in:

Could be my site is a sub-site?  i.e. url is company.sharepoint.com/sites/power/Lists/Customer 400 Returns/All Items.aspx?.....
This error appears when trying to open the form:

Thanks again for your continued patience and support.

Yup. Your site name is wrong. I have mine at json-formatting. Yours should be "sites/power".  ( I Should have clarified you need to replace the site relative url, not the Tenant only)

@bcameron1231 Brilliant, all back. I'm so grateful you've got me out of a real sticky situation. I note you've copied in @chakkaradeep to highlight this beartrap for the less talented Sharepoint meddlers among us, here's hoping something can be tweaked to prevent others getting caught in the seemingly inescapable loop I found myself in. Wish you well.

I tried the above and it didn't work for me.
I had a badly formatted JSON for the view format which I accidentally saved (Whoops!). Whenever that view was accessed, the screen would be completely blank (no SharePoint headers, nothing). I had only one view in the list called "All Items" (default).
I was able to recover my list by basically creating a new view and deleting the old one.

Navigate to the main site
Click the settings (gear icon) -> Site Information -> View all site settings
Under Site Administration, click Site libraries and lists
Click Customize "name of your list"
At the bottom of the page click -> Create view
Give the view a random name and set all of its settings as required
Make the new view the default view (the broken view can't be default otherwise it can't be deleted)
Back in the list settings, click the name of the broken view
At the bottom of the screen, click the delete button

This approach took some work but at least avoided me losing all of my list item data and needing to redo any linking that I had setup in Power Automate.

Some hours later, but @bcameron1231, your hint is still valid and was very useful to help me out.
I've got my JSON broken while i was trying to embed a href link as a section name, no chance to get out of rendering issues via the web interface, form was blank.
During my investigations i was able to use your solution with one little but important change:
→ contentTypes('0x0100F566899D669FFB4D9CA9135D63C98EAB001D365D0D20AF3E478A1E321ACB5C901F') was not validfor my case
fetch("https://YourTenant.sharepoint.com/sites/mysite/_api/contextinfo", {
  "headers": {
    "accept": "application/json;odata=verbose",
    "accept-language": "en-US,en;q=0.9",
    "content-type": "application/json;odata=verbose",
 },
  "method": "POST",
  "mode": "cors",
}).then(response => response.json()).then((data)=>{
    fetch("https://YourTenant.sharepoint.com/sites/mysite/_api/web/lists/getbytitle('mylist')", {
      "headers": {
        "accept": "application/json;odata=verbose",
        "accept-language": "en-US,en;q=0.9",
        "content-type": "application/json;odata=verbose",
        "x-requestdigest": data.d.GetContextWebInformation.FormDigestValue
      },
      "method": "POST",
      "mode": "cors",
      "credentials": "include"
    }).then(response => response.json()).then((data)=>{console.log(data)});
}
);

delivers d.ContentTypes.__deferred.uri which leads me to:
...
<content type="application/xml">
<properties>
<ClientFormCustomFormatter>
...
<Id m:type="SP.ContentTypeId">
	<StringValue>
		0x01000AC41DE60062DB408253A5C45D314A60006AC84011DADFD3409FD1B137E6A22EE9
	</StringValue>
</Id>
...

With this value i was able to reset the JSON formatting:
fetch("https://YourTenant.sharepoint.com/sites/mysite/_api/contextinfo", {
  "headers": {
    "accept": "application/json;odata=verbose",
    "accept-language": "en-US,en;q=0.9",
    "content-type": "application/json;odata=verbose",
 },
  "method": "POST",
  "mode": "cors",
}).then(response => response.json()).then((data)=>{
    fetch("https://YourTenant.sharepoint.com/sites/mysite/_api/web/lists/getbytitle('mylist')/contentTypes('0x01000AC41DE60062DB408253A5C45D314A60006AC84011DADFD3409FD1B137E6A22EE9')", {
      "headers": {
        "accept": "application/json;odata=verbose",
        "accept-language": "en-US,en;q=0.9",
        "content-type": "application/json;odata=verbose",
        "x-requestdigest": data.d.GetContextWebInformation.FormDigestValue
      },
      "body": "{\"__metadata\":{\"type\":\"SP.ContentType\"},\"ClientFormCustomFormatter\":\"\"\}",
      "method": "MERGE",
      "mode": "cors",
      "credentials": "include"
    }).then(response => response.json()).then((data)=>{console.log(data)});
}
);

Thanks again @bcameron1231 !!!

@maexrock Glad it helped you!


Fix getPeerCertificate

From recent Node documentation:
message.socket#

The net.Socket object associated with the connection.

With HTTPS support, use request.socket.getPeerCertificate() to obtain the client's authentication details.

https://nodejs.org/api/http.html#http_message_socket

Fixes #6

This change looks good, but we can't drop support for 0.10. Can you implement this in a backwards-compatible way?
Also, can you add Node 4 to travis.yml here? I'd rather deal with this change in one PR. Thanks!

@jwhitfieldseed I think we should merge this, and release a v2 without support for v0.10. It's going to be EOL soon (along with v0.12) and there seem to be open issues with it still that affect client cert auth.
https://github.com/nodejs/node/issues/5035

Actually it appears this is backward-compatible with 0.10 - the docs led me to the bad assumption that it wasn't.

@jwhitfieldseed updated to specify just version "4". Also, rebased against master

:+1: thanks for the contribution. I've released it as 2.0.0.


Central as plugin

Description
"Central" can be used as Plugin in serving by copying the central jar into plugin-folder of serving.
"Central" is still available as standalone application.
"serving" accepts classes implementing JsonSerializer as plugins.
to be continued.....

I updated your PR.
The JsonSerializer should be be registered int JsonUtils, it has global impact. The customization of json serialization should only be localized in the plugin itself. I removed MetaDataSerializer for now, It's not clear to me why we need this class.
I created a JsonSerializable class, this will work if the class only has one serialization form. If we need to support multiple serialization form, we need have a registration per plugin.

I updated your PR.
The JsonSerializer should be be registered int JsonUtils, it has global impact. The customization of json serialization should only be localized in the plugin itself. I removed MetaDataSerializer for now, It's not clear to me why we need this class.
I created a JsonSerializable class, this will work if the class only has one serialization form. If we need to support multiple serialization form, we need have a registration per plugin.

@frankfliu  Did you test the code after the change? I am sure this breaks the UI and you are not able to view any Model-Info. I expect a javascript error in ModelView.jspx cause no metadata object is defined. (MetaDataSerializer  is needed for this)
The new JsonSerializable-interface violates separation of concerns principle. A class should not know about every way it can get serialized.
Let's think about if HTTP-Request-with-Json-response is the only way that serving will support in support!?
Serialisation is a cross cutting concern, in my experience to ignore this will cause maintainability problems in future.
also adds unnecessary dependency of classes like Classifications to GSON.
You made a valid point when 2 plugins need to serialize the same class in 2 different ways, but I think the JsonSerializable interface doesn't solve this problem neither.
Original PR has a global effects on the particular instance of serving when a plugin is installed, but with the plugin-mechanism you are going to configure serving for your specific purpose anyway. IMO a smaller problem then the SoC-problem at this time.
Open to discuss

@ebamberg
I didn't test UI part, I thought the default Json serialization for Metadata is compatible with MetaDataSerializer.
The problem of using JsonUtils.GSON_BUILDER is it's global, If you changed this builder at one part of the code, it impact the whole system. The original code broke basicdataset testing.
And JsonUtils is a utility class, it should not have the knowledge of individual class. Adding Classification and DetectedObjects in it a really a hack.
JsonSerializable should be just a default for the class, just like Comparable to a Class, Ideally, djl-serving should be able register JsonSerializer at plugin level.

@ebamberg
I added MetaDataSerializer back and now UI is working.
I have some concern about serialize transient field. The Artifact and Metadata has circular reference, with transient field, json serialization will cause a dead loop, and that's why we need a customized MetaDataSerializer. I think the right solution would be create a proper ModelMetadata object instead of using Artifact.

@ebamberg
I added MetaDataSerializer back and now UI is working.
I have some concern about serialize transient field. The Artifact and Metadata has circular reference, with transient field, json serialization will cause a dead loop, and that's why we need a customized MetaDataSerializer. I think the right solution would be create a proper ModelMetadata object instead of using Artifact.

@frankfliu yes fully agreed. I have the same concern. the circular refernce

@ebamberg
I added MetaDataSerializer back and now UI is working.
I have some concern about serialize transient field. The Artifact and Metadata has circular reference, with transient field, json serialization will cause a dead loop, and that's why we need a customized MetaDataSerializer. I think the right solution would be create a proper ModelMetadata object instead of using Artifact.

@frankfliu yes fully agreed. I have the same concern. Out of a glut feeling I would also prophecy the circular reference will cause more issues in future. When I look in the code this reference seems only to exist so the MetaData-object can be re-used to deserialize metadata in the repository class, but not to get used as an object but to access some primitive data-fields. Am I right, or do I miss something ?  agreed that a proper ModelMetaData object is the proper solution. I guess you are talking about swap the current impl with a proper one in the core. Another way for the specific problem could to be add a ModelMataData-DTO to transfer from central-backend to frontend. Both will work. 1 option probably would avoid future issues due a better design, 2nd option would be a quick win for central. So both are fine in my opinion.  btw why do we have all the other transient fields in Artifacts, like Version and so on. I somehow could see why they have to be transient except to avoid decoupling of serializer. Just curious to understand this design.

@ebamberg
When I say ModelMetadata object, I'm actually mean ModelMataData-DTO.
The metadata was original used. as a top level object: https://github.com/awslabs/djl/blob/master/model-zoo/src/test/resources/mlrepo/model/cv/image_classification/ai/djl/zoo/mlp/metadata.json
The transient Metadata field in Artifact class is just a hack to make it easy for code, not meant to be serialized.
We can merge this PR and improve it later.


After training the model with a batch size of 32 with 100 epochs, generator learning rate of 0.00001 and a discriminator learning rate of  0.0000125, I believe the results satisfy the primary condition of producing reasonably clear images. The generator and discriminator loss converged at around 5 epochs, however at around 50 epochs, there was a significant model collapse.
Given the required training time for the model config above, I was unable to adjust the model parameters and retrain the model. If I had the time to rerun the model, I would decrease the learning rate for both generator and the discriminator.

Document

Title:
Good
Address probelm:
Problem mentioned, but not clearly addressed. Why using GAN instead for this particular task.
Working Principle:
Good
Figure visualization:
Good
dependecy:
Good
inputs and outputs description:
No input visualization, Does have output visualization
The output visualization is bit blurry
training plot:
Good
readme using markdown:
Good
preprocessing description:
Good
Description of usage:
very limited descriptio. not very clear on how to use each file

Check the files:

modules.py
Good
dataset.py
Good
train.py
Good
predict.py
Good
Suggestion:

during testing stage, you should disable the training mode of generated when generating images. generator(images, training = False)

Overall:

solve the problem:
Yes
TF/torch implementation:
Yes
Design quality:
Only the AdIN layer is implemented, other techniques such as progressive training, gradient penalty etc. 	are not implemented
code comments:
Good
Difficulty:
StyleGANv1 - hard

Morning Wei,
Thank you for your feedback! I have added your recommended changes to my README file.
As for the results, I am not sure that I will have time to allow the model to run again and produce higher quality images. My choice of hyperparameters were primarily due to time constraints, But I would definitely like to increase the EPOCH count to see some better images.
Thanks again and enjoy your summer!

Marking
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate design and implementation
Good spacing and comments
Header blocks
Recognition Problem
Solves problem
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage
Module present
Commenting
No Data leakage
Difficulty: Hard
Commit Log
Meaningful commit messages
Progressive commits used
Documentation
ReadMe good
Model/technical explanation
Good Description and Comments
Markdown used and PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch)
No Feedback required
Request Description good


Broken page of createCapture.

Description of createCapture contains so many empty lines.
Browser: Firefox 65.0.1

Thanks for reporting @Janglee123. Looks like there is a missing closing tag somewhere on this page that is causing the whole thing to get rendered as a code example, when it should be normal text after the code snippet.

fixed with https://github.com/processing/p5.js/commit/537ccaf34ba5941758673952a06d705e2662ce3c


⚠ SceneTime has degraded performance

In 5be8d5d, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 16707 ms

Resolved: SceneTime performance has improved in 40f0ccc after 6 minutes.


Feature/Mac build

Fix build and link of external shared libraries
Fix CMake target link libraries
Fix bad reference in yac gate impl
Fix includes
Fix type deduction in make_test_subscriber

Codecov Report

Merging #565 into develop will decrease coverage by 0.07%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #565      +/-   ##
===========================================
- Coverage    80.23%   80.16%   -0.08%
===========================================
  Files          305      307       +2
  Lines        12205    12170      -35
===========================================
- Hits          9793     9756      -37
- Misses        2412     2414       +2

Impacted Files
Coverage Δ

test/framework/test_subscriber.hpp
87.75% <ø> (ø)
:arrow_up:

irohad/consensus/yac/impl/yac_gate_impl.hpp
100% <ø> (ø)
:arrow_up:

irohad/model/converters/json_common.hpp
100% <ø> (ø)
:arrow_up:

irohad/model/converters/impl/json_common.cpp
95% <100%> (ø)
:arrow_up:

irohad/consensus/yac/impl/yac_gate_impl.cpp
90.16% <100%> (-0.16%)
:arrow_down:

libs/datetime/time.hpp
100% <100%> (ø)
:arrow_up:

irohad/model/converters/impl/pb_query_factory.cpp
83.63% <0%> (-12.56%)
:arrow_down:

libs/crypto/ed25519_impl.cpp
85.71% <0%> (-5.2%)
:arrow_down:

irohad/model/model_crypto_provider_impl.cpp
38.46% <0%> (-0.67%)
:arrow_down:

...i/interactive/impl/interactive_transaction_cli.cpp
0% <0%> (ø)
:arrow_up:

... and 45 more

Continue to review full report at Codecov.

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

Fixes build on archlinux :+1:

Also fixes protobuf 3.4.0 compatibility issues 👍

Also add that protobuf, grpc  are downloaded by default


add krugerschool

Add Kruger School to list

Hello, Its levan tsotskolauri from georgia
i'd like to ask you add my school in list ill write information below to prove that im student and my school is real
thanks you!
<EMAIL_ADDRESS>ISANI DISTRICT, PUBLIC SCHOOL #216
https://www.facebook.com/სსიპ-ქთბილისის-N216-საჯარო-სკოლა-473256156339558

@sopcommand Please DO NOT comment other folk pull request. Please send an email with your request to<EMAIL_ADDRESS>Please explain why the students have their email addresses in the Government of Georgia domain.

@giojavi04 Please provide us with a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

@philipto Sure, attach the link, image with hois data and the email.

https://krugerschool.edu.ec/
<EMAIL_ADDRESS>

@giojavi04 Thank you for the details. Regrettably, they are insufficient for adding this domain to this repository. The school seems to have another official website by the different domain name. The WHOIS data that you provided are with are correct. However they do not refer to a registrant legitimately representing Kruger School in Ecuador.
I have to close this request without merging. If you are a student or a teacher in this school, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students).


fixed error method && add PanicHandler Support

Fixed error method && add PanicHandler Support

Pull Request Test Coverage Report for Build 72

3 of 8 (37.5%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.06%) to 97.942%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

router.go
3
8
37.5%

Totals

Change from base Build 69:
-2.06%

Covered Lines:
238

Relevant Lines:
243

💛  - Coveralls

Fixed error method && add PanicHandler Support


Fatal pickling exception when using [UNIMOD:nn] identifier in ProForma string with MS2PIPFeatureGenerator and DeepLCFeatureGenerator

I'm getting a mysterious pickling error when trying to use ms2rescore.feature_generators.ms2pip.MS2PIPFeatureGenerator. The error seems to be caused by having [UNIMOD:nn] variable mods in the ProForma string of a peptide.
Here's how I create the environment:
C:\python\python309\python.exe -m venv venv_309_ms2rescore
venv_309_ms2rescore\Scripts\pip3 install ms2rescore==3.0.2

I've written a script that creates a suitable psm_list and MGF file, then the script just calls:
from ms2rescore.feature_generators.ms2pip import MS2PIPFeatureGenerator

...

    fgen = MS2PIPFeatureGenerator(
        model=model,
        ms2_tolerance=ms2_tolerance,
        spectrum_path=mgf_file_name,
        processes=processes,
    )
    fgen.add_features(psm_list)

This works with unmodified peptides (so ProForma sequences like "PEPTIDEK/2"). However, every ProForma sequence with mods ends up with a strange stack trace:
    2024-03-22 09:38:23,962 INFO Running MS²PIP for PSMs from run (1/1) `F981139_1.tsvn36rl336.mgf`...
    2024-03-22 09:38:25,678 INFO Processing spectra and peptides...
    Traceback (most recent call last):
      File "C:\Users\villek\githead\mascot-proj\mascot\www\bin\ML_adapters\MS2RescoreAdapter.py", line 239, in <module>
        main()
      File "C:\Users\villek\githead\mascot-proj\mascot\www\bin\ML_adapters\MS2RescoreAdapter.py", line 205, in main
        _add_MS2PIP_features(
      File "C:\Users\villek\githead\mascot-proj\mascot\www\bin\ML_adapters\MS2RescoreAdapter.py", line 108, in _add_MS2PIP_features
        fgen.add_features(psm_list)
      File "C:\Users\villek\tmp\venv_309_ms2rescore\lib\site-packages\ms2rescore\feature_generators\ms2pip.py", line 207, in add_features
        self._calculate_features(psm_list_run, ms2pip_results)
      File "C:\Users\villek\tmp\venv_309_ms2rescore\lib\site-packages\ms2rescore\feature_generators\ms2pip.py", line 218, in _calculate_features
        for result, features in zip(
      File "C:\Users\villek\tmp\venv_309_ms2rescore\lib\site-packages\rich\progress.py", line 168, in track
        yield from progress.track(
      File "C:\Users\villek\tmp\venv_309_ms2rescore\lib\site-packages\rich\progress.py", line 1209, in track
        for value in sequence:
      File "C:\python\Python309\lib\multiprocessing\pool.py", line 420, in <genexpr>
        return (item for chunk in result for item in chunk)
      File "C:\python\Python309\lib\multiprocessing\pool.py", line 870, in next
        raise value
      File "C:\python\Python309\lib\multiprocessing\pool.py", line 537, in _handle_tasks
        put(task)
      File "C:\python\Python309\lib\multiprocessing\connection.py", line 211, in send
        self._send_bytes(_ForkingPickler.dumps(obj))
      File "C:\python\Python309\lib\multiprocessing\reduction.py", line 51, in dumps
        cls(buf, protocol).dump(obj)
    AttributeError: Can't pickle local object 'create_engine.<locals>.connect'

Here's what I've tried in a fresh venv:

pip install ms2rescore==3.0.1 -- script gives the same stack trace
pip install ms2rescore==3.0.0 -- script gives the same stack trace
pip install ms2pip==4.0.0dev4 -- or 4.0.0dev5, ..., dev8 -- script gives the same stack trace
pip install ms2pip==3.13.0 -- correctly breaks elsewhere because API is incompatible
Change to processes=1 or processes=None or processes=8 -- makes no difference
Create venv with Python 3.10 or 3.11 -- script gives same stack trace

Is the issue the ProForma strings in the input? In this case, there's just one peptide (psm_list.to_dataframe().to_csv()):
,peptidoform,spectrum_id,run,collection,spectrum,is_decoy,score,qvalue,pep,precursor_mz,retention_time,ion_mobility,protein_list,rank,source,provenance_data,metadata,rescoring_features
0,IPAM[UNIMOD:35][-63.998285]TIAK/2,query13_rank1_label1,F981123_1.tsvzrkjbv40.mgf,,,False,0.0475696443353316,,,430.732788,,,,,,"{'query': '13', 'rank': '1', 'label': '1'}",{},{}

The ProForma syntax is correct, or at least accepted by https://pyteomics.readthedocs.io/en/latest/api/proforma.html. If I change it to Unimod titles:
,peptidoform,spectrum_id,run,collection,spectrum,is_decoy,score,qvalue,pep,precursor_mz,retention_time,ion_mobility,protein_list,rank,source,provenance_data,metadata,rescoring_features
0,IPAM[UNIMOD:Oxidation][-63.998285]TIAK/2,query13_rank1_label1,F981123_1.tsvmwgalmq5.mgf,,,False,0.0475696443353316,,,430.732788,,,,,,"{'query': '13', 'rank': '1', 'label': '1'}",{},{}

Then I still get the same exception. If I change the input to use deltas instead of Unimod record IDs, like this:
0,IPAM[+15.994915][-63.998285]TIAK/2,query13_rank1_label1,F981123_1.tsvi5cnqcn8.mgf,,,False,0.0475696443353316,,,430.732788,,,,,,"{'query': '13', 'rank': '1', 'label': '1'}",{},{}

Then MS2PIP is happy. Note, the issue is not having multiple mods at the same site. The fault is clearly caused by the [UNIMOD:nn] field.
From grepping the venv, I can tell create_engine comes from sqlalchemy, and it looks like this has something to do with ms2pip's dlib support:
ms2pip\_utils\dlib.py:94:def open_sqlite(filename):
ms2pip\spectrum_output.py:763:        with open_sqlite(filename) as connection:

But I can't see what I can do at the call site to disable dlib (if it's the issue) or what setting to change.
I will work around the issue by only using deltas in ProForma strings.

I'm getting the same exception if I use [UNIMOD:nn] with DeepLCFeatureGenerator. Using deltas with DeepLCFeatureGenerator kind of works, but it correctly logs warnings like:
2024-03-22 11:31:07,118 WARNING Skipping the following (not in library): [GenericModification('15.994915', None, None)]

Which means DeepLCFeatureGenerator is useless until the fault is fixed.

Same here, tried different platforms.
AttributeError: Can't pickle local object 'create_engine..connect'

Same here, tried different platforms. AttributeError: Can't pickle local object 'create_engine..connect'

Got through by using the docker version.

Looking at the current Dockerfile, it specifies ubuntu:focal (20.04 LTS) as the starting point. I'm not sure which Python version it has; Distrowatch says 3.8.
I've installed Python 3.8.10, the latest 3.8.x with a 64-bit Windows installer, made the venv and and tried pip install ms2rescore. Unfortunately, it gets stuck in compiling statsmodels and there are various errors about ms2pip wheels not being available. So I can't confirm or disprove that using Python 3.8 helps on Windows.

Hi @vrkosk and @ttzzjt,
We are investigating the issue. For now, it seems like downgrading to Pyteomics 4.6 helps to avoid the issue. Just run:
pip install pyteomics==4.6.3

Or update psm_utils, which temporarily has the Pyteomics version restriction in place:
pip install --upgrade psm_utils

I will post here when we know more.
Best,
Ralf

Yes, downgrading to pyteomics 4.6.3 works.

A fix will be included in the next Pyteomics release: levitsky/pyteomics#144


Remove console log statements

console.log statements should not be committed 90% of the time, let's stick to console.debug, console.info etc and generally not commit debug messages unless it makes sense.

Fixed


Fix scrolling to allow larger margins when blocks are smaller

…but still constrain to the workspace.
Fixes #264.

This change is

All cleaned up.  PTAL +@roboerikg

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Suggestion: Rename the operator from validation/main/X to val/main/X

Recent models rely heavily on multiple loss functions. If we want to track them with a PrintReport, the width of the line heading becomes far too wide, containing mostly empty space, mainly resulting from the validation tag in front of the loss.
Could there be a consensus on possible abbreviation?
e.g.
epoch       iteration   main/loss   main/mae    main/gnll   main/weighted  main/VAE    main/VAE_REC  main/VAE_KL  validation/main/loss  validation/main/mae  validation/main/weighted  validation/main/VAE  elapsed_
time
1           356         31.2785     1.39209     3.45184     27.4938        0.332836    32933.7       349.925      26.5542               1.27629              23.1902                   0.32403              114.618

to
epoch       iteration   main/loss   main/mae    main/gnll   main/weighted  main/VAE    main/VAE_REC  main/VAE_KL  val/main/loss  val/main/mae  val/main/weighted  val/main/VAE  elapsed_time
1           356         31.2785     1.39209     3.45184     27.4938        0.332836    32933.7       349.925      26.5542        1.27629       23.1902            0.32403       114.618

We can even decrease the whitespace around the iteration/epoch columns to be numer_of_digits(max_epoch)+2. Alternatively, decrease the default width of 10 to something smaller - 6 or 8.

What is the reason behind the convention of having main in the names. Appears everywhere and takes a good amount of column space. At least when printing can't it simply be train/<loss_name> and val<loss_name>.

I guess it's related to how chainer.training.Updater and its child classes manage optimizer(s) & iterator(s).
For example,chainer.training.StandardUpdater internally manages optimizer(s) & iterator(s) using dictionaries as follows.
https://github.com/chainer/chainer/blob/d297a1787ebb35cfce279dde0df64dd2feba4327/chainer/training/updaters/standard_updater.py#L67-L79
In general, I think reasonable both to shorten validation to val and eliminating main from print :)

Indeed. I've been trying to find where the validation string is used to prefix the optimizer/iterator similar to the dictionary created for main, but couldn't really locate it in the code. (A repo wide search didn't help, just showed all the examples that use PrintReport)
What would the best way to implement your suggestions be in regards to shortening and eliminating?

where the validation string is used to prefix the optimizer/iterator

docs seems to be the answer.

If these values are reported inside the Evaluator extension, 'validation/' is added at the head of the link name, thus the item names are changed to 'validation/main/abs_error' and 'validation/main/squared_error' ('validation' is the default name of the Evaluator extension).

@23pointsNorth validation is the default name of the Evaluator extension.
https://docs.chainer.org/en/stable/reference/generated/chainer.training.extensions.Evaluator.html?#chainer.training.extensions.Evaluator.default_name
https://github.com/chainer/chainer/blob/v6.1.0/examples/mnist/train_mnist.py#L113-L120
You can give it a different name like this:
trainer.extend(extensions.Evaluator(iter, model, device=device), name='val')


Printing on Android device

Hello,
Actually, the printing feature from Android device seems to be not usable. Indeed, if i try to launch a printing from my device when viewer is displayed in iframe, not only the pdf content is sent to the printer but the entire window (with window parent content):
https://jsfiddle.net/300gykh9/
There is a possibility to be usable?
Thx

Closing as duplicate of #7599.


Update SW Distros workflow: use meaningful default for initial_distro

I think it is useful to have in initial_distro  the initial distro we actually want to use, to reduce the possible human errors.

cc @Nicogene this should avoid the error I did in https://github.com/icub-tech-iit/documentation/commit/ac1a5e7144b5339e86dc733c30f2b6526be132f1 .

Nope, the intended use is completely different. This param allows us to not process old distros.

Ah, I see, so leaving "0000.00"  is completely ok.

Yep!


SAP HANA XS Advanced, Creating an HDI Module

Tutorial URL: https://developers.sap.com/tutorials/xsa-hdi-module.html
Please specify the step you are referring to
screenshot's URL are broken, not visible

I see the images. Can you refresh, maybe clear cache?

Thank you for your feedback. We haven't heard from you in the past 30 days, so we are closing the issue.
If you still have questions, feel free to reopen the issue.


WIP: Adds Hide Comment Functionality

References #2370

[x] Hide Comments
[x] Comment List supports hiding comments
[x] Unit-tests

@osilkin98 is this ready for review? If so, mark Lex as reviewer and remove WIP.

@eukreign Do you mind taking a look at this?

Yeah it’s a clunky, I’m gonna put in a dynamic query parser and improve the logic.

On Aug 25, 2019, at 1:02 PM, Lex Berezhny<EMAIL_ADDRESS>wrote:
@eukreign approved this pull request.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.


SushiSwap exchange should support Polygon network

Feature Suggestion
polygon network already available for uniswap exchange please also add same polygon network for sushiswap
Impact
polygon network has very low gas fees. this will help people to do arbitrage between sushiswap and uniswap on polygon network.
Additional context

See https://github.com/hummingbot/hummingbot/pull/5727


[enhancement] add an option to choose whether upload changelog to release info

Is your feature request related to a problem? Please describe
CHANGELOG can be automatically added to the description of a release.
Description of the new feature/enhancement
Give a new option to choose whether add CHANGELOG to the description of a release automatically.
Proposed technical implementation details (optional)

According to tag's name and release name, the CHANGELOG which belongs to that release can be automatically added to the end of the release description. And if the same content is already here, then it won't do anything.
Plan to use PyGithub to achieve this.
Additional context

This feature can depend on the hide-info feature, so that the changelog info will be surrounded by the hide block


feat: add optimism support

What I did
Add optimism support
How I did it
I have used github.com/pandadefi/ape-optimism in order to test
How to verify it
Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

Using the same version of black on my machine I don't have anything that needs formating.

Thank you for getting this rolling!
See https://github.com/ApeWorX/ape-etherscan/pull/34


koordlet: add page cache collector unit test and mend default cold memory config

Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/lgtm


[SPARK-18209][SQL] More robust view canonicalization without full SQL expansion

What changes were proposed in this pull request?
Currently we canonicalize the view definition to provide the context for the database as well as star expansion, this is fragile for the following reasons:

It is non-trivial to guarantee that the generated SQL is correct without being extremely verbose, given the current set of operators.
We need extensive testing for all combination of operators.
Whenever we introduce a new logical plan operator, we need to be super careful because it might break SQL generation.

We use a late binding approach to do view canonicalization, which resolve the dependent views every time the view is used. This PR is expected to achieve the following goals:

Views should reflect changes to the underlying tables and intermediate views. For example:    CREATE TABLE T1(a int, b string)
    CREATE TABLE T2(a int, b string, c int)
    CREATE VIEW A AS SELECT * FROM T1
    CREATE VIEW B AS SELECT * FROM A
    ALTER VIEW A AS SELECT * FROM T2
    SELECT * FROM B

will return data from table T2 (Note that we still need to validate the schema).
2. Make views more robust by not using SQL generation to store the view’s logical plan.
3. The views generated by older versions of Spark or HIVE should still work or throw a meaningful error if the view definition is invalid.
4. A view should throw a meaningful error when the view in an invalid state due to an underlying change. Such errors should be thrown as early as possible.
The follow up works will be:

Remove the SQL generation (in particular the SQLBuilder class) for Spark SQL operators and the test suites(and also golden files) for them.
Disallow cyclic view reference(See SPARK-18389).
View dependency management similar to Postgres.
Underlying table schema change when star is used.
Support create permanent views on non-SQL DataFrames.

How was this patch tested?
Add new test cases in SQLViewSuite.

Test build #69724 has started for PR 16168 at commit aa81a36.

cc @rxin @srinathshankar @nsyca

Test build #69724 has finished for PR 16168 at commit aa81a36.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Test build #69730 has started for PR 16168 at commit 434009e.

also cc @gatorsmile

Test build #69730 has finished for PR 16168 at commit 434009e.

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

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

Merged build finished. Test PASSed.

@jiangxb1987 We need a low level design about the changes you made here, especially document the issue you hit and the solution you choose.

also cc @cloud-fan

@jiangxb1987 - please take a look at my comment on the jira ticket: https://issues.apache.org/jira/browse/SPARK-18209

@hvanhovell @nsyca @gatorsmile @rxin Thank you for your suggestions! I will try to make a better approach ASAP! Thank you！

This is closed because we should achieve the final goal by submitting a sequence of smaller PRs.


Code error for Scene-operations main.ts line 489

In the current source code of /src/main.js of commit aa7c8fb has a code-error in line 489 which prohibits changes of scenes for light-groups.
The reference to the lightScene is not found.
Current code
const sceneObj = await this.getObjectAsync(${channelId}.scene_${commands.scene.toLowerCase()});
it needs to be changed at least to
const sceneObj = await this.getObjectAsync(${this.namespace}.lightScenes.scene_${commands.scene.toLowerCase()});
or better with replacing "lightScenes" with the dynamic object-group-namespace.

You are showing the scenes for a "room". I do not have rooms. Under the hue-instance I do see all objects listed (lights and lightgroups mixed). I am migrating from hue-extended, so I am not familiar with the regular and the legacy-data-structure yet. Do you use the legacy one?

Legacy just adds a useless device object in between instance and first channel. So this should not make a difference.
Can you show me what the light scenes folder

To avoid confusion, I am on beta adapter-version 3.10.1

issue still persists after latest update to

Sorry but what you are writing does not seem right from code perspective or you see something which I do not see. Group scenes are handled in a way that they are created at the corresponding group, else they are created at lightScenes.
https://github.com/iobroker-community-adapters/ioBroker.hue/blob/21194ae683ccb73b26bb2e03108ad1cbcc06f118/src/main.ts#L2161-L2190
And the state change handling is implemented for every dp starting with scene_ like the ones in your lightScenes channel does.
https://github.com/iobroker-community-adapters/ioBroker.hue/blob/21194ae683ccb73b26bb2e03108ad1cbcc06f118/src/main.ts#L256-L273
You should also see the corresponding log message Started scene: .... Please re-check, maybe something has changed with the scenes API but code-wise it looks correct.

Hi @foxriver76 : I think the misunderstanding is about how these scene get triggered.
I am NOT changing the state of a scene datapoint. I am issueing a command in order to set a certain scene to a group of lights. The scene is not defined as a group scene for this group, but as a global scene.
Following example for the data structure:

hue.0.Wohzimmer.* --> the light-group I want to control
hue.0.lightScenes.scene_tagesbeleuchtung --> the scene I want to set for the light
hue.0.Wohzimmer.command --> datapoint I am writing to with the value {"scene": "tagesbeleuchtung","transitiontime": 4}

This code here
https://github.com/iobroker-community-adapters/ioBroker.hue/blob/21194ae683ccb73b26bb2e03108ad1cbcc06f118/src/main.ts#L490-L497
can only handle a scene handed over in the command, which is a scene within the scope of the light/group I want to change. It refers to ${channelId}.scene_${commands.scene.toLowerCase()}.
If you want to set a global scene, which is not defined in the scope of the light/group, but as a global scene under the lightScenes folder, you would need to refer to another namespace, which is ${this.namespace}.lightScenes.scene_${commands.scene.toLowerCase()}

Okay, maybe I do not fully understand which kind of scene this is. AFAIR there were scenes corresponding to a group and the Light Scenes which you can create and also define which lights are connected to the scene https://github.com/peter-murray/node-hue-api/blob/typescript/docs/scenes.md#createscene
So I am not aware how we can set a scene to lights not correlated to the specific scene by its configuration. But if you have a setup running, please feel free to create a PR.

I am afraid I do not know how PRs work and how to interact.
If you want a quick fix, which still favours group-scenes over global scenes, new line 492ff would be:

 var sceneObj = await this.getObjectAsync(`${channelId}.scene_${commands.scene.toLowerCase()}`);
 if(sceneObj === null){sceneObj = await this.getObjectAsync(`${this.namespace}.lightScenes.scene_${commands.scene.toLowerCase()}`);}

And you have tested that this works as expected? ;-)

@foxriver76 I created a PR for this


为什么只支持单类选择器，能不能优化下，变成可选？？

This PR/issue doesn't received response since Weex migrated to Apache two years ago and this repository is no longer active yet.
Feel free to ask question in new repository and thanks for your contribution.

This issue is outdated for a long time and will be closed now. You can create a new one if you still have questions.