[HW] Nikhil Yadav

**Name: Nikhil Kumar Yadav
**Email<EMAIL_ADDRESS>**Linkedin Profile: https://www.linkedin.com/in/nikhil-yadav-91a79916a/
Attach the homework step I screenshots below:

Attach the homework step II screenshots below:

Hey Nikhil Kumar Yadav, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/RXBPf0HyScyjGi_ZUB4P5A


CASSANDRA-19746: Update CQLSH documentation to reflect compatibility with Python 3.

This patch updates the CQLSH documentation to remove references to Python 2.7 and clarify compatibility with Python 3.
patch by Mohammad Suhel; for CASSANDRA-19746

I think this should say simply Python 3 and not mention a specific Python release.  The current Python driver supports Python 3.8 - 3.12.

I have updated the documentation to state that cqlsh is compatible with Python 3, as requested. Please review the changes.

It looks like this was just a problem with the 4.1 docs, and that trunk had already been fixed.  I think the current (trunk) section on compatibility is fine as-is, having already removed 2.7, and it's not likely worth fixing it on the 4.1 branch of the docs.
I'd therefore suggest we close this PR and Jira as won't fix.

Thank you for the review and feedback.
I understand that the trunk branch has already addressed the Python 2.7 references, and updating the 4.1 branch documentation isn't necessary at this point.
I will close this PR and the associated Jira issue as "won't fix."
Thanks again for the guidance.


geotiff-validator: refactor

rename validateBands to validateBandCount
use constants instead of hardcoded property names
formatting

merging since https://github.com/klips-project/klips-sdi/pull/139 is merged


feat(testing): support symbolic accounts with fresh storage

(opening this draft PR to get initial thoughts and if OK to have this impl in revm, thank you!)
Motivation
Re symbolic testing (atm with Kontrol) in foundry https://github.com/foundry-rs/foundry/issues/4072#issuecomment-1456844543
There's a need for a new freshStorage(address target) cheatcode where storage of target contract should follow rules below:

Every SSTORE that comes through is handled as a normal write.
On SLOAD:

If storage slot was already written, then written value is returned.
If storage slot wasn't already written, a random uint256 is generated and written to storage.

For example, for a contract like:
contract Counter {
    address[] public owners;

    function getOwner(uint256 pos) public view returns (address) {
        return owners[pos];
    }

    function setOwner(uint256 pos, address owner) public {
        owners[pos] = owner;
    }
}

a test like the one bellow should result in PASS
function test_fresh_storage(uint256 a) public {
    a = bound(a, 0, 100);

    Counter counter = new Counter();
    vm.freshStorage(address(counter));

    // Next call would fail with array out of bounds without freshStorage
    address owner = counter.getOwner(a);
    // Subsequent calls should retrieve same value
    assertEq(counter.getOwner(a), owner);
    // Change slot and make sure new value retrieved
    counter.setOwner(a, address(111));
    assertEq(counter.getOwner(a), address(111));
}

Solution

add a new symbolic bool in Account struct, default false
expose Account.mark_symbolic fn to enable symbolic storage
in JournaledState.sload generate random value if account newly created and symbolic

@rakita @mattsse @DaniPopes could you please have a look at let me know if the right way to go, if so will follow up on working the random related failures. thank you!

This does not look like a good fit for Revm. Couldn't you add a logic in step_end that would override the sload value?

This does not look like a good fit for Revm. Couldn't you add a logic in step_end that would override the sload value?

yeah, can do that, just wanted to check if somehow needed here too. thank you, going to close this then.


Quake mode support

@cprog To me it sounds like this can be done by implementing to features:

Global shortcuts so that you can use Control+Tilde to bring Extraterm to the front/back.
Option to hide Extraterm from the taskbar ans only show an icon in the tray area.

I second the addition of a quake mode. I absolutely love being able to just hit a button and have a terminal drop down. I'd prefer it to have a drop down animation, but, basically if it can hide itself and reappear with the touch of a button that would make this way better in my opinion.

It works great for me. Thanks :) Merry Cristmas

@cprog tha's great to hear. Merry Christmas! :christmas_tree: :santa: :gift:


SAK-30075 - Web service for finding and fixing bad attachments in Sam…

…igo pools

There is an easy to use test framework for soap web services it would be nice if tests were add for new web services.

I understand this web service is used to fix a broken attachment condition within question pools but has the root issue been resolved within the question pool logic?

This issue is being addressed at SAM-2257.
El 16 de noviembre de 2015 15:45:05 CET, Earle Nietzel<EMAIL_ADDRESS>escribió:

I understand this web service is used to fix a broken attachment
condition within question pools but has the root issue been resolved
within the question pool logic?

Reply to this email directly or view it on GitHub:
https://github.com/sakaiproject/sakai/pull/1272#issuecomment-157054834

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

I have refactored all the Samigo code and I have put it inside QuestionPoolServiceAPI bean, so I can inject it into webservices and include a mock.
I have also changed it to keep one single method with String parameters, in order that empty strings are treated like null values.


Add boilerplate for CSS Recommendations

Based on the status section of https://www.w3.org/TR/2019/REC-css-contain-1-20191121/

Actually, never mind. Reason best explained by @svgeesus:

We never make the Rec from the ED [1].
Instead, we make the Rec from the Proposed Rec, changing the status and styling to suit. (And if actually needed, we apply any editorial changes which were agreed with the AC, and which we talk about in the Director's Decision).
[1] Because we don't add in whatever changes might have been made to the ED in the couple of months since the Proposed Rec was generated, which the AC has not seen.


Resources list not updated

In case there is a valid resource list it is not updated.
E.g. in case new resources will be available the software is not able to use them.

Thank you for making the resources file optional by 3b39faf45a5a772115dee328aa5d6818f953e823!
I was wondering why we need the resource file at all. What is the point of caching the URIs? Is it just to save one API call?

At the beginning the value of sensors were available only up to 12h after
status change. After that time no data were provided.
Now it seems not necessary because of data remains available.
Removing the resources new resources will be available at startup...
It would be better, i think, to introduce a way to periodically refresh
resources list....
Il Dom 27 Dic 2020, 23:19 Daniel Rheinbay<EMAIL_ADDRESS>ha
scritto:

Thank you for making the resources file optional by 3b39faf
https://github.com/xraver/mercedes_me_api/commit/3b39faf45a5a772115dee328aa5d6818f953e823
!
I was wondering why we need the resource file at all. What is the point of
caching the URIs? Is it just to save one API call?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xraver/mercedes_me_api/issues/15#issuecomment-751522571,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALJM7A6O56ZTOOBBFMGSKZLSW6XHBANCNFSM4VDMSWTA
.


[MINOR] Update release verify script - 'sha' to 'sha512'

Apache Storm's release process has changed to use sha512 instead of sha, but it is not reflected to release verify script. This patch applies to the change.

+1


Telegram tab should be hidden if community doesn't have a telegram configured

Problem
Telegram - if nothing was entered in create/edit community, it shows error message on Telegram tab
Acceptance Criteria
Need to hide telegram tab if community has no telegram configured

Looking at entity.community.header. It seems like this is already fixed. This should be closed

@Tguntenaar there's a catch if you save telegram_handle as an empty string. I'll fix it.


Automating the build

Thanks for inviting me to the repo @MohammadTaseenKhan! What process did you take for generating the .apk file? Is it something that could be automated via GitHub Actions?

no i made it open for the users that what apk are they using
is it safe for them or not
on every version there is a build file available for them

on every month i will build apk for users according to chitchatter layout update
i am unable to create application for ios or mac users because i dont have it

@MohammadTaseenKhan I understand. What I was envisioning for https://github.com/jeremyckahn/chitchatter/issues/98 was a bit different: When Chitchatter is built and deployed to https://chitchatter.im/, .apk files would also be built by the CI/CD system (GitHub Actions) and made available automatically. This is how Farmhand's build process works with web, Linux, and Windows builds: https://github.com/jeremyckahn/farmhand/releases
The key advantage of this is that users that trace their .apk binary to auditable build logs and commit hashes, thereby giving them the ability to verify what is actually in their .apk files. For a privacy-first project like Chitchatter, the contents of build artifacts can't be taken on faith and needs to be tied to source code and reproducible.


Vendor ouroboros 0.4.1

Author of the ouroboros crate yanked 0.4.1 and broke us.  See https://github.com/solana-labs/solana/pull/13555
Once the monorepo recovers, we can bring new Solana crates into SPL and revert this PR

grrr, fine.  I'll just wait for the v1.4.8 crates to ship


Bundle Error<EMAIL_ADDRESS>WatermelonDB

Describe the bug
Getting the bellow error when trying to use this module, with the WatermelonDB usage example. Occurs on bundle.
Error: While trying to resolve module `react-native-flipper-databases` from file `<ProjectFolder>/src/stores/index.ts`,
 the package `<ProjectFolder>/node_modules/react-native-flipper-databases/package.json` was successfully found.
 However, this package itself specifies a `main` module field that could not be resolved (`<ProjectFolder>/node_modules/react-native-flipper-databases/src`.
 Indeed, none of these files exist:

  * <ProjectFolder>/node_modules/react-native-flipper-databases/src(.native|.android.js|.native.js|.js|.android.json|.native.json|.json|.android.ts|.native.ts|.ts|.android.tsx|.native.tsx|.tsx)
  * <ProjectFolder>/node_modules/react-native-flipper-databases/src/index(.native|.android.js|.native.js|.js|.android.json|.native.json|.json|.android.ts|.native.ts|.ts|.android.tsx|.native.tsx|.tsx)
    at ResolutionRequest.resolveDependency (<ProjectFolder>/node_modules/metro/src/node-haste/DependencyGraph/ResolutionRequest.js:65:15)
    at DependencyGraph.resolveDependency (<ProjectFolder>/node_modules/metro/src/node-haste/DependencyGraph.js:287:16)
    at Object.resolve (<ProjectFolder>/node_modules/metro/src/lib/transformHelpers.js:267:42)
    at <ProjectFolder>/node_modules/metro/src/DeltaBundler/traverseDependencies.js:434:31
    at Array.map (<anonymous>)
    at resolveDependencies (<ProjectFolder>/node_modules/metro/src/DeltaBundler/traverseDependencies.js:431:18)
    at <ProjectFolder>/node_modules/metro/src/DeltaBundler/traverseDependencies.js:275:33
    at Generator.next (<anonymous>)
    at asyncGeneratorStep (<ProjectFolder>/node_modules/metro/src/DeltaBundler/traverseDependencies.js:87:24)
    at _next (<ProjectFolder>/node_modules/metro/src/DeltaBundler/traverseDependencies.js:107:9)

To Reproduce
react-native: 0.63.3
using Hermes Engine
Flipper version: 0.64.0
Installed with npm on Ubuntu 20.10,
also with yarn on Windows 10
Code from usage example used.
if (__DEV__) {
  // Import connectDatabases function
  const connectDatabases = require('react-native-flipper-databases').default;

  // Import required DBDrivers
  const WatermelonDBDriver = require('react-native-flipper-databases/drivers/watermelondb').default;

  connectDatabases([
    new WatermelonDBDriver(database), // Pass in database definition
  ]);
}

As well as using,
import connectDatabases from 'react-native-flipper-databases';
import WatermelonDBDriver from 'react-native-flipper-databases/drivers/watermelondb';

....

if (__DEV__) {
   connectDatabases([
    new WatermelonDBDriver(database),
  ]);
}

Hi @Cully-Curwen,
Sorry for the late answer, the reported issue should be resolved with the latest 0.2.1 version.

Hi @panz3r,
No worries, with 0.2.1 all is working on my end.
Thank you for your time and effort.


Add support for more device types like TV and AVR.

Right now the SDK only supports Lights and Accessories. I like to add a TV and AVR as an IP controlled device and use the favorite and volume control functions like we can on the build in devices.

Yes, technically there are no issues but the documentation (#6) is still open

Fixed upstream, i'll close as soon as we released an updated firmware


Uncaught Error: ENOENT: no such file or directory, open '/home/ivlis/.atom/packages/sync-settings...

I've got this error immediatly after updating to 0.7.0
Atom Version: 1.5.4
System: linux 4.4.1-2-ARCH
Thrown From: sync-settings package, v0.7.0
Stack Trace
Uncaught Error: ENOENT: no such file or directory, open '/home/ivlis/.atom/packages/sync-settings/node_modules/github/api/v3.0.0/routes.json'
At fs.js:549

Error: ENOENT: no such file or directory, open '/home/ivlis/.atom/packages/sync-settings/node_modules/github/api/v3.0.0/routes.json'
    at Error (native)
    at Object.fs.openSync (fs.js:549:18)
    at Object.module.(anonymous function) [as openSync] (ATOM_SHELL_ASAR.js:137:20)
    at Object.fs.readFileSync (fs.js:397:15)
    at Object.fs.readFileSync (ATOM_SHELL_ASAR.js:385:29)
    at Object.module.exports (/home/ivlis/.atom/packages/sync-settings/node_modules/github/api/v3.0.0/index.js:22:33)
    at new module.exports (/home/ivlis/.atom/packages/sync-settings/node_modules/github/index.js:184:26)
    at Object.SyncSettings.createClient (/home/ivlis/.atom/packages/sync-settings/lib/sync-settings.coffee:198:18)
    at Object.SyncSettings.checkForUpdate (/home/ivlis/.atom/packages/sync-settings/lib/sync-settings.coffee:46:8)
    at Immediate._onImmediate (/home/ivlis/.atom/packages/sync-settings/lib/sync-settings.coffee:32:8)
    at processImmediate [as _immediateCallback] (timers.js:374:17)

Commands
     -8:37.3.0 core:paste (atom-text-editor.editor.mini.is-focused)
     -8:35.4.0 core:confirm (atom-text-editor.editor.mini.is-focused)
  4x -8:11.7.0 blur (atom-text-editor.editor.mini.is-focused)
     -8:07.3.0 autocomplete-python:go-to-definition (atom-text-editor.editor.vim-mode-plus.normal-mode.is-focused)
 16x -8:07.3.0 blur (atom-text-editor.editor.vim-mode-plus.normal-mode.is-focused)
  2x -3:33.7.0 core:backspace (atom-text-editor.editor.mini.is-focused)
     -3:31.1.0 core:confirm (atom-text-editor.editor.mini.is-focused)
 40x -3:21 blur (atom-text-editor.editor.mini.is-focused)
     -2:17.8.0 settings-view:check-for-package-updates (atom-workspace.workspace.scrollbars-visible-always.seti-ui-no-icons.theme-one-dark-syntax.theme-one-dark-ui)
 33x -2:17.7.0 blur (atom-text-editor.editor.vim-mode-plus.normal-mode.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "linter-flake8",
      "linter-chktex",
      "atom-beautify"
    ]
  },
  "sync-settings": {
    "_analyticsUserId": "XXXXXXXX",
    "_lastBackupHash": "XXXXXXX",
    "gistId": "XXXXXXX",
    "personalAccessToken": "XXXXXXX"
  }
}

Installed Packages
# User
autocomplete-python, v1.7.0
file-icons, v1.6.18
hyperclick, v0.0.35
language-latex, v0.6.1
latextools, v0.8.0
linter, v1.11.3
linter-coffeelint, v1.1.2
open-recent, v5.0.0
sync-settings, v0.7.0
vim-mode-plus, v0.23.0

# Dev
No dev packages

Duplicate of #194.


ModTeamMemberAccessLevel.Statistics same as Manager permission?

The file Runtime/Enums/ModTeamMemberAccessLevel.cs contains these definitions for TeamMember Access:
public enum ModTeamMemberAccessLevel
{
    Moderator = 1, // can moderate content submitted
    Statistics = 4, // moderator access, including read only access to view reports
    Administrator = 8, // full access, including editing the profile and team
}

But the "Edit Team" screen on the modio website lists these permissions:

Moderator - can moderate comments and content attached
Manager - moderator access, including uploading builds and editing settings except supply and team members
Administrator - full access, including editing the supply and team

Is ModTeamMemberAccess.Statistics the same as the "Manager" permission or not? I'm trying to filter mods based on who can upload new builds.

Thanks for bringing this to my attention. The definitions may have changed at some point since the initial implementation.
Yes, Statistics is equivalent to Manager.
I'll create a pull request shortly to update this.

Updated here:
https://github.com/modio/UnityPlugin/pull/44


Migrate ActivityHeatmap to Vue SFC

Follow up to #10188
Migrate ActivityHeatmap to use Vue SFC and compile at build time.
Signed-off-by: Gary Kim<EMAIL_ADDRESS>

@gary-kim eatmap works just fine but "dashboard-repos" are gone ?!?

@gary-kim heatmap works just fine but "dashboard-repos" are gone ?!?

Dont remove alias in webpack.config.js and it works all fine

Dont remove alias in webpack.config.js and it works all fine

Okay then. Added it back in here. At some point, I'll go through to see what's left using compile on the browser and migrate them.

Codecov Report

Merging #10953 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10953      +/-   ##
==========================================
- Coverage   43.62%   43.59%   -0.03%
==========================================
  Files         597      597
  Lines       83923    83923
==========================================
- Hits        36609    36588      -21
- Misses      42802    42820      +18
- Partials     4512     4515       +3

Impacted Files
Coverage Δ

modules/indexer/stats/db.go
50.00% <0.00%> (-9.38%)
:arrow_down:

modules/queue/workerpool.go
58.00% <0.00%> (-2.14%)
:arrow_down:

services/pull/check.go
48.17% <0.00%> (-1.83%)
:arrow_down:

modules/git/repo.go
47.28% <0.00%> (-1.26%)
:arrow_down:

modules/log/event.go
64.61% <0.00%> (-1.03%)
:arrow_down:

models/error.go
30.49% <0.00%> (-0.54%)
:arrow_down:

services/pull/pull.go
34.70% <0.00%> (-0.30%)
:arrow_down:

models/lfs.go
17.27% <0.00%> (ø)

modules/lfs/server.go
40.35% <0.00%> (ø)

routers/swagger_json.go
0.00% <0.00%> (ø)

... and 3 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 2a06d3a...5c6759e. Read the comment docs.

make L-G-T-M work.


[WIP]Release v0.4.0

Codecov Report

Merging #116 into release will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           release     #116   +/-   ##
========================================
  Coverage    97.52%   97.52%
========================================
  Files           59       59
  Lines         1537     1537
========================================
  Hits          1499     1499
  Misses          38       38

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 761a8b8...2591380. Read the comment docs.


Roll Plugins from 64c222d368ba to a5b664db7c7b (7 revisions)

https://github.com/flutter/plugins/compare/64c222d368ba...a5b664db7c7b
2021-12-11<EMAIL_ADDRESS>[tools] fix typo in tools (flutter/plugins#4607)
2021-12-10<EMAIL_ADDRESS>[path_provider] Fix handling of null application ID (flutter/plugins#4606)
2021-12-10<EMAIL_ADDRESS>[in_app-purchase] Updated the ReadMe to remove the deprecated InAppPurchaseAndroidPlatformAddition.enablePendingPurchases  method (flutter/plugins#4597)
2021-12-10<EMAIL_ADDRESS>Add missing return for nullably typed function (flutter/plugins#4598)
2021-12-10<EMAIL_ADDRESS>[shared_preferences] Fix example in readme for mocking values (flutter/plugins#4411)
2021-12-10<EMAIL_ADDRESS>[in_app_purchase] Fix upgrading subscription by deferred proration mode on android. (flutter/plugins#4560)
2021-12-09<EMAIL_ADDRESS>[path_provider] Use the application ID in the application support path (#2845) (flutter/plugins#3077)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Host System monitoring NETTX NETRX

Description
NETTX and NETRX in new host monitoring are None/Zero
Details

Affected Component: [Monitord]
Hypervisor: [KVM]
Version: [6.2]

Additional context
Looks like something happens in new monitoring system.  When I check directly on node, linux_host.rb returns NETTX and NETRX
[root@dc3-node1 monitor]# ./linux_usage.rb
HYPERVISOR=kvm
USEDMEMORY=243410764
FREEMEMORY=20606980
FREECPU=3888
USEDCPU=912
NETRX=41307203246698
NETTX=50104108822222

Progress Status

[ ] Code committed
[ ] Testing - QA
[ ] Documentation (Release notes - resolved issues, compatibility, known issues)

Duplicate #5640, fixed in 6.6.0 and 6.4.3 release

Hi @paczerny, I don't think it was solved in #5640 - it was about VM not HOST. I applied path to 6.2 release and run new VM on host id 2. In host_monitoring db table is not NETTX or NETRX
<MONITORING><TIMESTAMP>1675762479</TIMESTAMP><ID>1</ID><CAPACITY><FREE_CPU><![CDATA[3792]]></FREE_CPU><FREE_MEMORY><![CDATA[20504788]]></FREE_MEMORY><USED_CPU><![CDATA[1008]]></USED_CPU><USED_MEMORY><![CDATA[242982228]]></USED_MEMORY></CAPACITY><SYSTEM></SYSTEM></MONITORING>

Hi @feldsam,
sorry, I haven't notice the difference to previous issue. I'm reopening this.


Adding the docker image for tecan module

We need to create a release & project for this image

Can you add the Makefile?

Makefile should be fixed


How to unit test macros?

Is there a best practice on testing macros? Do you have to have a model sql file or could you directly test what's in the macro folder in test sql file?
Any examples you can point to would be appreciated.
Thanks

So, I tend to reply on the testing functionality from dbt rather than the unit test library. See the example below. Note dbt test will fail if they get back a result hence the actual != expected. Also I added the tag  unit-test to make it run with my other unit tests
{{
    config(
        tags=['unit-test']
    )
}}

with input as (
  select
    '{{ my_macro("my_params") }}' as actual
),
expectations as (
  select actual
  from input
  where actual != 'expected output'
)

select * from expectations

Hi @gaj995 , we currently don't have plans to support tests for macros. You can use @DrValani suggestion 🙂


Added plugin template and scripted test

To run the scripted tests with GitHub actions have a look at https://github.com/playframework/playframework/blob/63c8845ca6392094f079c35652a04a12b153d779/.github/workflows/build-test.yml#L145-L178
The most relevant part is: https://github.com/playframework/playframework/blob/63c8845ca6392094f079c35652a04a12b153d779/.github/workflows/build-test.yml#L168-L176

To run the scripted tests with GitHub actions have a look at https://github.com/playframework/playframework/blob/63c8845ca6392094f079c35652a04a12b153d779/.github/workflows/build-test.yml#L145-L178 The most relevant part is: https://github.com/playframework/playframework/blob/63c8845ca6392094f079c35652a04a12b153d779/.github/workflows/build-test.yml#L168-L176

Made a separate job to run scripted test


Tweaked exception message for invalid formatted SQL file and added to tests DAT-15874

Impact

[x] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Things to be aware of

Things to worry about

Additional Context

Test errors are safe to ignore

Merged on https://github.com/liquibase/liquibase/pull/4870 .


Main method controlling the app

The app needs a main method as a central place for controlling all processes like:

database connections
current window in view - more control over what users views
background processes like e.g. synchronization
suggestion by Jan - variable in globalSetting class as an intermediary solution

Bhupjit Asks -it will be a big overhaul of everything so it might worth pondering over if its something we need to consider along with refactoring and modularisation and converting into a webapp
consider #372

I think a review of whether we should proceed with this work along with change to a web app should be prioritised after we have released version 1.2

This issue also relates to #353 where we have the chance to implement a "one and only one" instance running at the time approach.


Update the existing check to report on the status of Telerik.- Fail if Telerik Installed & Used - Give warning about 10.x upgrade issue- Fail if Telerik Installed & Not-Used - Give message about the removal- Pass if Telerik Not Installed

@daguiler Go for it, greatly appreciated

Fantasic @daguiler - any help we can get will be MUCH appreciated!


omegaconf.errors.ConfigKeyError: Missing key control

python -W ignore generate_samples.py -gpu 0 -dataset landscape -ckpt_path ./models/landscape/stage2/
this is my command,what wrong could you help me ?thanks

Hi, thanks for your interest! Did you download the pre-trained models and place them in ./models/... ?
If this problem still occurs, please share the whole error message here.


[Feature][connector-spark] add a jdbc connector base DataSource V2 API

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
Add a jdbc connector based on Spark DataSource V2 API.
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Read data from multiple database and multiple tables.

I don't know what's different between now and new method, can you explain to me when we make this change come true, what will we get? Thanks.


Split app in reader and holder

Split app in reader and holder
Updated cbor library

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

@googlebot I fixed it.

@davidz25 Sorry, about that. My business email was also in the commit, that is why CLA signed before didn't work. Now it is fixed.

Thanks. Sorry for being slow, I missed this. Now merged.


#2897 - Remove deprecated feature: ClientLibs Manager and friends

Fixes #2897

OK - Decrease in code coverage


Issue40: @LRA: Remove DelayClose, terminal() defaults now to true

The PR contains the changes for API, TCK, and Spec.
Changed the @LRA annotation on a few methods of ActivityController (added terminal = false) so that TCK tests should not be impacted by the change in the default value.
Should be reviewed if it is correct though.

thx @mmusgrov for pointing out where they are used. I didn't found those usages the first time.
Removed the TODOs in the new commit.


Consistent naming for the fermions

What is the current behavior?
The fermion names are of all kind of different conventions.
What is the expected behavior?
They should be unified.
Suggested solutions
The Hamiltonian,  circuit instruction and gate name should have the following names:
FermiHubbard, circ.fhubbard, fhubbard
Hop, circ.fhop, fhop
Interaction, circ.fint, fint
Phase, circ.fphase, fphase
FRXGate, frx, frx

Closed by #43


Troubles with custom value change

Hi, first of all - cool plugin, good job. Anyway got some troubles using it: tried to investigate where is a bug. Let me explain: the problem starts when I change the value manually from outside your plugin, I have connected your plugin with vuex store by using props ":value" (v-model is not used here). On change event I commit and change the value in store. This way I make sure that it is connected to store (sort of two way binding). Everything works as expected until I change "valueFormat" to "object", I keep whole object not only ID, also, there are several levels in a tree.  When I change the value from outside, I'm making sure that the value passed to vue-treeselect plugin has changed as well, but somehow/somewhere in components inside your plugin doesn't keep up with this change. I guess it isn't capable to detecting changes of reference type objects. I tried copying it totally using method: JSON.parse(JSON.stringify(value)) - doesn't help it.
Made some pics:
https://ibb.co/bWyfXH (Example before)
https://ibb.co/cXqSCH (Example after) - Here: after changing the value visually there are no changes in component
Would be helpful if you could help solving this issue, everything else is working perfectly..:

Hi! As the screenshots you provided, I have seen your before/after values are:
Before:
[ {
  id: 1,
  label: 'Konsultacija',
  children: [ {
    id: 2,
    label: 'Konsultacija child 1',
  }, {
    id: 3,
    label: 'Konsultacija child 2',
  }, {
    id: 4,
    label: 'Konsultacija child 3',
  } ],
} ]

After:
[ {
  id: 1,
  label: 'Konsultacija',
  children: [ {
    id: 2,
    label: 'Konsultacija child 1',
  }, {
    id: 3,
    label: 'Konsultacija child 2',
  } ],
} ]

Seems you were attempting to deselect Konsultacija child 3 by removing it from the children array. Actually, this is not the way what vue-treeselect designed to be used (maybe I should add more detail on this in the doc). You should pass only the children as value:
[ {
  id: 2,
  label: 'Konsultacija child 1',
}, {
  id: 3,
  label: 'Konsultacija child 2',
} ],

Remember, when valueFormat="id", you need to pass [ 2, 3 ], right?
Hope this could be helpful! :D

Heya, glad you reviewed this - respect 👍
Mmm, as I understood you are saying, that I should reformat the value structure I'm passing.. mmm. Just extract the values to become first level in array. :|

Remember, when valueFormat="id", you need to pass [ 2, 3 ], right?

And I set valueFormat to "object", so I'm passing full objects instead of id's. Will it work if I:
[ {   id: 1,   label: 'Konsultacija',   children: [ {     id: 2,     label: 'Konsultacija child 1',   }, {     id: 3,     label: 'Konsultacija child 2',   } ], } ]
change value like this:
[ {     id: 2,     label: 'Konsultacija child 1',   }, {     id: 3,     label: 'Konsultacija child 2',   } ]
Anyway will try to sort it out. Will inform you how it end up. Thanks again @riophae :)

Heya, forgot to inform about my trouble here - it works well now. Thanks :)

@GieMik8 Happy to hear that! I am closing this thread.


Compatibility Issues with Slipcover, pytest-forked, and pytest-xdist

Hello maintainers,
I hope you're doing well. I've come across a peculiar issue while using slipcover in combination with pytest-forked and pytest-xdist. I'm facing some unexpected behavior that I wanted to discuss and get your insight on.
Issue Description:

Slipcover and pytest-forked:

When using slipcover in combination with the pytest-forked extension, it appears that code coverage is not being computed correctly. I suspect the forked processes might be causing this, but it's only a guess at this point. I'm expecting to see complete coverage details, but that's not the case.

Slipcover and pytest-xdist:

On the other hand, when using slipcover with the pytest-xdist extension, there's a different issue.
I am seeing an empty output table.
File    #lines    #l.miss    Cover%    Missing
------  --------  ---------  --------  ---------
  py3: OK (7.12=setup[0.10]+cmd[3.80,3.22] seconds)
  congratulations :) (7.47 seconds)

Oh I just saw this issue https://github.com/plasma-umass/slipcover/issues/9.

I guess this can be closed since it's a duplicate of #9


Feat: menu roles

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


changelog for 2.068.1

@CyberShadow you're DAutoTest doesn't rerun when I reopen a PR from the same branch.

Please merge ASAP so I can build the release.

I'm confused, I thought dot-releases were for fixing regressions, not adding new features?

I'm confused, I thought dot-releases were for fixing regressions, not adding new features?

It was a follow-up b/c 2.068.0 was missing those -profile=gc functions.

Nit, the changelog looks like this:
Library changes

(...)

Library changes

(...)

("Library changes" repeats twice and both sections have a list with 1 item in them)

Nit, the changelog looks like this:

You're right, it's missing the $(HR) and the ref to the bugzilla list.
See #1093.


Pack: files with space will contain %20 inside the nupkg

Description
Files with space will contain %20 inside the nupkg. It does not seem to matter if the files are included specifically or through wildcards/globbing.
Repro steps

Clone https://github.com/pchinery/paket-bugreport/tree/master

Run paket pack .

Expected behavior
The file test with space.txt should be in the tools folder of the package.
Actual behavior
The file is named test%20with%20space.txt instead
Known workarounds
none

I started digging in the project and a good candidate for the root is createRelativePath, but it is used in quite a few places and I am still trying to get some orientation in the project ;-)

afaik this is by design and nuget is/was doing the same, do you have an issue with that?

ok, I am quite surprized. Do you know if this will be decoded to spaces again when extracted with Paket/NuGet? If yes, then I can close this again and sorry for bothering you ;-)

Yes at least that's the idea. That's why I asked if you encountered any errors with this.

No, I just came across this and it kept the %20 even after extracting the files, but in restrospect, I found that I extracted them with a regular ZIP tool.

Welcome to NuGet world where everything is magic, even extracting a zip file.


Heatmap used for manual cluster adjustment in 3_Pixie_Cluster_Cells does no match the cell phenotype mask

Dear Pixie developers,
I hope this message finds you well. As you can see in the attached figures, the cell counts in the cell phenotype mask figure does not align with the cell counts in the histograms on the top of the heatmap for manual adjustments. For examples, the biggest cluster should be B cells ( in a greenish blue color) and should occupy the cell positions where light blue color cells (epithelial cells) are currently located. Another examples was that the circle-like region should be epithelial cells but instead labelled as Myeloid immune cells, which I noticed the cell counts is only 8. Basically, what I mean here is cluster per se seemed alright but wrong labelled. We identified these errors by overlaying the masks with the image. I wonder if you guys had encoutered problems like this.

Merry Early Christmas.

Hi @rach-crc, similar question to your other issue - can you double check if you have updated the ark package to the latest version? This was an issue we have encountered previously, but had fixed: https://github.com/angelolab/ark-analysis/issues/1117. Reminder that after you "git pull" to update the Github repo, you must install the latest version with "pip install ."

Hi @rach-crc, similar question to your other issue - can you double check if you have updated the ark package to the latest version? This was an issue we have encountered previously, but had fixed: #1117. Reminder that after you "git pull" to update the Github repo, you must install the latest version with "pip install ."

Hi, @cliu72. Thank you for replying my message during holiday times, happy Christmas Eve.:) I checked my ark analysis version, it's v.0.7.2, whcich I believe should be the newest (see attachment). More importantly, after running the cell cluster notebook, I checked the the cell table with phenotype assigned to each cell. The number of cells with a certain phenotype (e.g., B cell counts) were very similar to what I would expect from the cluster-adjustment heatmap (the heatmap generated in the cell cluster notebook). However, the colored mask do not align with the cell counts in the generated cell table.

Hm I think it's a little strange that you are getting two separate bugs that were already previously described and fixed around the same time (this one and the segmentation one). The behavior that you're describing (cell table is accurate and only the color mask is wrong) is what is described in the previous issue that was solved. We haven't run into this issue in our lab since that fix was implemented.
I'm not sure on what system you're running this on, but I wonder if the version being used is still somehow an older version, even though the version number looks up to date. To help troubleshoot, I would follow the "To Reproduce" part of this issue using the example dataset in the notebook and see if you are getting the issue that is described: https://github.com/angelolab/ark-analysis/issues/1117. You can also look at the specific lines of code that were changed in the fix (https://github.com/angelolab/ark-analysis/pull/1127/files) and compare it with the code that is being used in your environment (go into the exact same environment that you're running into the bug, and you use inspect.getsource to see the source code - compare the source code with what is in Github). As described in the fix (https://github.com/angelolab/ark-analysis/pull/1127), you can also look at the *_meta_cluster_mapping.csv file to see if it looks accurate.
If you still cannot figure the issue, @alex-l-kong can help you troubleshoot in the new year. If there is a time pressure and you need to move forward, you can generate your own color masks in python outside of ark by mapping the segmentation IDs in the cell table to its annotated cell type (and define your own color mapping).


Total collateral field in Babbage transactions

OS
Your OS: Ubuntu 22.04
Versions
The db-sync version (eg cardano-db-sync --version): 13.0.4
PostgreSQL version: 14.1
Build/Install Method
The method you use to build or install cardano-db-sync: cabal
Run method
The method you used to run cardano-db-sync (eg Nix/Docker/systemd/none): systemd
Additional context
Problem Report
While looking into adding support for Babbage transactions I noticed that db-sync doesn't include total collateral field from Babbage spec.
While I'm not sure it's needed, it would be good to get some clarity on the subject.

Is it possible to calculate this as the sum of the collateral inputs minus the sum of the collateral outputs?

Is it possible to calculate this as the sum of the collateral inputs minus the sum of the collateral outputs?

To my understanding, even if CIP mention collateral outputs in plural, there can only be a single collateral output. But yes, the result should be the sum of collateral inputs (lovelace) - lovelace in collateral output. Though, this specific field is part of the tx to my understanding. I'm not entirely sure of its use case, it also doesn't take tokens in collateral into consideration to my knowledge. But just calculating the difference between input and output collateral could any end system system/user do, so not sure I see the value adding it to dbsync to be honest. I raised the issue just because I saw that it was missing, but also unsure if it's actually needed.

Indeed the total colateral is a reduntant and optional field in the transaction. It doesn't add any new information as its values can be computed by the difference coll inputs - coll output. I think it exists for hardware wallets that don't have access to UTxO and can't resolve inputs, but I don't know much about its usability.
In db-sync the field is reflected in the tx.fee of a Tx with a phase 2 failure. If the field is not there, as it's optional, db-sync will compute it. I think this can close?

I'm ok closing this one.


Change contents of readme + fix typo in AnnotationNotFoundException

Codecov Report

Merging #59 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master      #59   +/-   ##
=========================================
  Coverage     98.78%   98.78%
  Complexity      116      116
=========================================
  Files            20       20
  Lines           328      328
=========================================
  Hits            324      324
  Misses            4        4

Impacted Files
Coverage Δ
Complexity Δ

src/HttpApi/AnnotationNotFoundException.php
100.00% <100.00%> (ø)
2.00 <0.00> (ø)

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 d024446...f75f825. Read the comment docs.


Is their any way to ignore response validation?

I don't want to validate response data, is there any option to ignore response validation.
On Swagger tools I had option
_swagger_validate:
      name: swagger_validator
      validateResponse: false

OAS Tools v3 is already out, you can disable response validation through configuration. Check the docs


error on last update google cloud

{ "error": { "errors": [ { "domain": "global", "reason": "required", "message": "Bucket is requester pays bucket but no user project provided." } ], "code": 400, "message": "Bucket is requester pays bucket but no user project provided." } }

Hopefully this will help: https://stackoverflow.com/a/47741739/685706

Closing due to lack of activity


New input and followup action styles

Why
In order to for the user to complete an action like adding a service instance, we need to clearly display the needed actions and next steps.
What

created a new style to apply to an actions box

made the box display full width
made the color distinct from existing page elements
adjusted the display text styles to better highlight the specifics of the action (strong.inline-block)

refined input styles to be closer to the rhythm of the page

reduced padding in the inputs
made input labels stronger
reduced size of selector dropdown indicator
modified spacing around inputs
added a "cancel" button style
added a NONFUNCTIONAL cancel button to the actions box

Known issues

cancel button is nonfunctional; just here to show where it would go
there is no way to dismiss the actions box
users without the proper privileges can access the action button and actions box, but cannot complete the action. these users should not be exposed to actions they cannot perform
these styles are not intended to be responsive

Related Issue
Adjust presentation of followup actions box: https://github.com/18F/cg-style/issues/78

Before

After

Updated the bg color of the FAs box to be a bit more chill.

Ah, whoops! I made that change and didn't save.


q

123

addTextureView是三星等手机在回到非全屏播放的时候原来的TextureView不会create，需要重新new一个TextureView

你好，咱们项目推出数字通证饺子token，https://github.com/lipangit/JZT， 给所有提交过pr的朋友30000个JZT，表示对你过去对项目支持的感谢。
请下载币威钱包https://www.bitcv.app/，把里面的以太坊地址给我，币威的地址站内转账不需要手续费。这东西值不值钱不重要，玩的是信仰。
Hello, let's launch the JiaoZiToken, https://github.com/lipangit/JZT, and give all the friends who submitted pr 30,000 JZT, thank you for your support of the project.
Please download the coin wallet https://www.bitcv.app/?from=bitcvCom and give me the address of the Ethereum. There is no charge for transferring money at the address station, if you do not know Chinese , you can give me other address of the eth. It is not important that the value of this thing is worthless. It is faith.


Homepage update

Pull request checklist
Make sure you:

[x] Signed the Contributor License Agreement
[x] Followed the commit message guidelines

Short description of the change(s)
Update homepage with webhint logo and content updates

Updated CLI instructions.

OK, updated again to get rid of the other two lines in the CLI.


ChocoDsc

Hi
I would like to rename the resource and revise it according to the quidelines from the PowerShell Team.
https://blogs.msdn.microsoft.com/powershell/2017/12/08/dsc-resource-naming-and-support-guidelines/
Would you accept such a PR?

Currently - possibly. We would need to evaluate how that would affect current users of the module.

I'm working my way though the resources, i hope i can present you a first view in a few days.

@Outek it's been a while but where did you get to with this?

Hi
Sorry for this, but i close this issue/request. There is so much to do, maybe it would be better to write it completely new.

@Outek I think this is still a valid request so will re-open.


Update version of Graph App Kit for npm usage

Can you please update the version of graph app kit (currently 1.0.1) so that we can use the latest merges (e.g. https://github.com/neo4j-apps/graph-app-kit/pull/24) via npm?
That would be great. :)
Thank you in advance!

Thanks for updating the npm version of Graph App Kit! :)


Feature: Gradient BG support

Add a support for linear gradient

Add a new column for type of background solid | linear-gradient | ...
For now UI,

Select dropdown, to select type of background default to solid
on Linear Gradient, Show two color input box
Angle of input, valid from 0 to 359°

I think we end up needing more columns in the db.
For now, I think we need:

bgColorType: solid | linearGradient
gradientColorStart: string
gradientColorEnd: string
linearGradientDirection: number (0 - 359)

So making sure we have those in the validation might be good too.


Fix: profile sidebar width issue (#4156)

Hey @TiaApril if you run yarn format it should fix the prettier error that's failing the tests!
Also just FYI usually we fill out the frontend checklist in the PR description to go through the steps to show it's ready.

@nabramow sorry if I ask a lot, what should I do from here? I did run the yarn format from VScode, should I push the branch and do all over it again?


New version

Can you make a new version so we have the latest fixes in a stable build?

Reminder/upvote.

:+1:

I forked and tagged 0.2.8 myself now since I did no longer feel like using dev-master in all composer-files. We're using commando in important cronjobs, and I do not want sudden changes in master to break those. As soon as you tag the version yourself, I will gladly switch back to using the original package/repo.

I think this can be closed as newer versions have been tagged (currently v0.2.9).


Unknown Layer Type: CTCGreedyDecoder

Hi
Having above run time error when running below,
Appreciate any advise,
thank you
C:\Users\optasia\Documents\Intel\OpenVINO\inference_engine_demos_build\intel64\Release>security_barrier_camera_demo  -d GPU -d_va  GPU -d_lpr  GPU  -m vehicle-license-plate-detection-barrier-0106.xml  -m_lpr license-plate-recognition-barrier-0001.xml   -i<EMAIL_ADDRESS>-display_resolution 1200x980
[ INFO ] InferenceEngine:
API version ............ 2.1
Build .................. 32974
Description ....... API
[ INFO ] Files were added: 1
[ INFO ]<EMAIL_ADDRESS>[ INFO ] Loading device GPU
GPU
clDNNPlugin version ......... 2.1
Build ........... 32974
[ INFO ] Loading detection model to the GPU plugin
[ INFO ] Loading Licence Plate Recognition (LPR) model to the GPU plugin
[ ERROR ] Unknown Layer Type: CTCGreedyDecoder

Hi @richardgohth
GPU plugin does not support CTCGreedyDecoder.
I don't know details but maybe you can solve this issue by Custom Layer GPU Extension.
Ask about it in https://github.com/opencv/open_model_zoo.

Noted with thanks


Improve /browse to understand .isolated files and link to the files referenced by it

Repro:
1. Use /browse to load a .isolated file.
2. !!!

Expected:
Clickable links.

Actual:
A nice dump of the json file but it's not clickable. There are cases where this
would be useful; browse google-test results, load a .mp4, whatever. Compression
may get in the way, so the file may have to be decompressed server side even if
we already have client side (javascript based) zlib decompression. Doing it
server side introduces the 32 Mb limit.

I'm really open to any kind of change to improve the usability on this page.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 6 Aug 2014 at 5:04

Implemented.


Send status on PR open

This is an attempt to fully address #3
Hope this makes sense @bradrydzewski. If not, please point me in the right direction.

thanks, LGTM :)

How about a merge then?

@marcinwyszynski I just need to pull down and run some tests and will then get it merged right away. thanks!

one minor request -- could you bump the docker image tag in the .drone.yml to 1.1.0
  docker:
    repo: lgtm/lgtm
-   tag: [latest, 1.0.0]
+   tag: [latest, 1.1.0]
    when:
      branch: master
      event: push

thanks!

@bradrydzewski No bother, done.

Seems like this should be merged already.

Are we there yet?

@marioizquierdo if you're asking me - my work here is done. If you're asking @bradrydzewski he does not seem to frequent these premises any more, so feel free to use my fork.


Field period fix

Fix Job service to work with the date format changes made to resource service

Codecov Report

Merging #9 into master will increase coverage by 0.33%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master       #9      +/-   ##
============================================
+ Coverage     17.41%   17.74%   +0.33%
  Complexity       72       72
============================================
  Files            39       39
  Lines          1068     1048      -20
  Branches         55       57       +2
============================================
  Hits            186      186
+ Misses          879      859      -20
  Partials          3        3

Impacted Files
Coverage Δ
Complexity Δ

...uk/gov/ons/fwmt/job_service/ApplicationConfig.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../ons/fwmt/job_service/rest/dto/FieldPeriodDto.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ob_service/service/impl/CSVParsingServiceImpl.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...vice/rest/impl/FieldPeriodResourceServiceImpl.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...service/data/legacy_ingest/LegacySampleIngest.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ce/impl/totalmobile/TMJobConverterServiceImpl.java
10.88% <0%> (+0.07%)
4 <0> (ø)
:arrow_down:

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 05ab984...9a5805b. Read the comment docs.

codacyyyy failllsss thooo


Fix BinaryData.fromFile scenarios preventing deletion.

Fixes https://github.com/Azure/azure-sdk-for-java/issues/29289 .
BinaryData.fromFile(path).toFluxByteBuffer() and BinaryData.fromFile(path).toBytes() has been changed to not use memory mapped files. Reasons:

The FileInputStream read populates array in one shot usually in toBytes() scenario.
We use small chunks in toFluxByteBuffer, memory mapping small chunks is not advised by JDK docs.
We want these two to work for Windows users without side effects.

The BinaryData.toByteBuffer keeps memory mapping. It's performance wise recommended to do this this way.

API change check
API changes are not detected in this pull request.


How to add section header to uicollectionview

I'm trying to add section header by supplementaryViewFactory, but it never works.
Here's my code:
collectionView.register(MyCollectionHeader.self, forSupplementaryViewOfKind: UICollectionElementKindSectionHeader, withReuseIdentifier: "Section")

let dataSource = RxCollectionViewSectionedReloadDataSource<SectionOfCustomData>()

dataSource.supplementaryViewFactory = { (dataSource, collectionView, kind, indexPath) in
            if kind == UICollectionElementKindSectionHeader {
                let section = collectionView.dequeueReusableSupplementaryView(ofKind: kind, withReuseIdentifier: "Section", for: indexPath) as! MyCollectionHeader

                section.titleLabel.text = "\(dataSource[indexPath.section].header)"

                return section
            } else {
                return UICollectionReusableView()
            }
        }

let sections = [
            ...COPY FROM README...
]

        Observable.just(sections)
            .bind(to: collectionView.rx.items(dataSource: dataSource))
            .disposed(by: disposeBag)

it's running just fine but the headers are not there, is there anything wrong?

I think it's duplicates #127
Feel free to reopen if I'm wrong


fix #196

Added validateForm() to detachFromForm handler.
This fixes the issue where if you unmount an input with an invalid value, the form remains in an invalid state until the next onChange event. Now the form's state.isValid property will update when an input unmounts.

Aha, I see you fixed it :-) This makes sense, thanks!


Request body assumed always ref

Seems like codegen (0.0.13) expects:
      requestBody:
        content:
          application/json:
            schema:
              $ref: "#/components/schemas/Something"

and does not accept:
      requestBody:
        content:
          application/json:
            schema:
              type: string

Log:
Traceback (most recent call last):
  File "/Users/victor/megad-heated-floor-thermostat/.venv/bin/fastapi-codegen", line 8, in <module>
    sys.exit(app())
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/typer/main.py", line 213, in __call__
    return get_command(self)()
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/typer/main.py", line 496, in wrapper
    return callback(**use_params)  # type: ignore
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/fastapi_code_generator/__main__.py", line 26, in main
    return generate_code(input_name, input_text, output_dir, template_dir)
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/fastapi_code_generator/__main__.py", line 37, in generate_code
    parsed_object: ParsedObject = parser.parse()
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/fastapi_code_generator/parser.py", line 391, in parse
    return self.parse_paths(openapi)
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/fastapi_code_generator/parser.py", line 400, in parse_paths
    return ParsedObject(
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/fastapi_code_generator/parser.py", line 372, in __init__
    operation.arguments
  File<EMAIL_ADDRESS>line 967, in __get__
    val = self.func(instance)
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/fastapi_code_generator/parser.py", line 175, in arguments
    return self.get_arguments(snake_case=False)
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/fastapi_code_generator/parser.py", line 183, in get_arguments
    argument.argument for argument in self.get_argument_list(snake_case)
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/fastapi_code_generator/parser.py", line 197, in get_argument_list
    if self.request:
  File<EMAIL_ADDRESS>line 967, in __get__
    val = self.func(instance)
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/fastapi_code_generator/parser.py", line 113, in request
    type_hint=schema.ref_object_name,
  File "/Users/victor/megad-heated-floor-thermostat/.venv/lib/python3.8/site-packages/datamodel_code_generator/parser/jsonschema.py", line 115, in ref_object_name
    return self.ref.rsplit('/', 1)[-1]  # type: ignore
AttributeError: 'NoneType' object has no attribute 'rsplit'

@synclpz
Thank you for creating this issue.
I will fix it this week.

I'm sorry. I don't forget the issue.
I will try it after I resolve the blocker. https://github.com/koxudaxi/fastapi-code-generator/issues/35

Also running into this issue. Any update on this @koxudaxi?

@Baukebrenninkmeijer
Thank you for posting the comment.
I will work on it this weekend.

@synclpz
I have released a fixed version as 0.0.14


$model->save() generating value for id column

I'm facing a lot of varying issues specially during saving data to MySQL database. All the issues are coming from duplicate or invalid values for the id column. This column has been defined in the migration as:
'id' => $this->bigPrimaryKey(),

The corresponding column definition in the table is:
  `id` bigint(20) NOT NULL AUTO_INCREMENT,
  PRIMARY KEY (`id`),

The corresponding error is typically:
SQLSTATE[22003]: Numeric value out of range: 167 Out of range value for column 'id' at row 1
The SQL being executed was: INSERT INTO `product` (`sku`, ..., `id`) VALUES (NULL, ..., 0)

Please note that id column is included in the resultant SQL and it's value is set to 0. Sometimes, it's set to a value similar to the id of an existing record and then I get duplicate key error.
I don't understand why the id column is being included at all in the generated SQL command?
Additional info

Q
A

Yii version
2.0.44

PHP version
7.4.22

Operating system
Mac OS

It is included only when you provide a value for it that is why I suspect that this might be a cause of your problem.

Here is the code that I'm using:
$product = new Product();
$product->hsn = 'hsn-value';
... // Few more assignments but not id
$product->save();

That's why inclusion of id in the resultant command is surprising.

Here is the exact code in case it helps:
$product = Product::findOne(['wc_product_id' => $data->id]) ?: new Product();
$product->sku = $data->sku ?: null;
$product->name = $data->name;
$meta = array_column($data->meta_data, 'value', 'key');
$product->hsn_code = $meta['_hsn_number'] ?? null;
if ($product->isNewRecord) {
    $product->quantity = $data->stock_quantity ?? 0;
    $product->wc_product_id = $data->id;
}
$product->save();

Here is the latest error generated for it:
SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '2' for key 'PRIMARY'
The SQL being executed was: INSERT INTO `product` (`sku`, `name`, `hsn_code`, `quantity`, `wc_product_id`, `version`, `created_at`, `updated_at`, `created_by`, `updated_by`, `id`) VALUES (NULL, 'Custom Cards', NULL, 0, 490, '0', NOW(), NOW(), 3, 3, 2)

Since no one else is reporting such problem this might be something related to your DB / app. If not, we would need to get a minimal reproducible example to recreate it. In both cases please try it on the fresh project and let us know the results.

Sounds about right. Still we need minimal reproducible case to work on it, otherwise we can only guess what is on your side.

@skills-up  But the code sample you provided is not a batch insert. Check out documentation.

It's not a batch insert code, because the API call is not for a batch insert. The API is called once for each product, and depending on whether a record exists in Yii2 app database, an insert/update is made. Anyway, the issue I've raised is regarding inclusion of id column in the generated SQL command for an insert operation. Curiously, I don't have this issue in an older Yii2 version (about 3 yrs old) used in a different app.

Do you have any rules() on the Model that could be adding the id?

Here is the complete set of rules on the model:
    public function rules()
    {
        return [
            [['wc_product_id', 'initial_quantity', 'adjusted_quantity', 'to_order_quantity', 'version', 'created_by', 'updated_by'], 'integer'],
            [['name', 'initial_quantity', 'adjusted_quantity'], 'required'],
            [['created_at', 'updated_at'], 'safe'],
            [['sku', 'name'], 'string', 'max' => 255],
            [['hsn_code'], 'string', 'max' => 15],
            [['wc_product_id'], 'unique'],
            [['sku'], 'unique'],
            [['version'], 'default', 'value' => '0'],
            [['version'], 'mootensai\components\OptimisticLockValidator']
        ];
    }

As you an see there is no rule corresponding to id.
However, that makes me think: perhaps the following is the reason:
    public function behaviors()
    {
        return [
            'timestamp' => [
                'class' => TimestampBehavior::className(),
                'createdAtAttribute' => 'created_at',
                'updatedAtAttribute' => 'updated_at',
                'value' => new \yii\db\Expression('NOW()'),
            ],
            'blameable' => [
                'class' => BlameableBehavior::className(),
                'createdByAttribute' => 'created_by',
                'updatedByAttribute' => 'updated_by',
            ],
            'uuid' => [
                'class' => UUIDBehavior::className(),
                'column' => 'id',
            ],
        ];
    }

I'll remove the uuid behavior and report back if that solves the issue.

Removing the uuid behavior did solve the issue.
I'm sorry about taking so much of your time. Thanks for all your help.


Add zmin argument passing to plot_field function.

plot_field function is here

After some consideration it seems that there is not need for this.
It is well regulated as it is.


MAINT: Fix tests on Travis

Builds with
Python: 3.5, PANDAS_VERSION=0.18.1 NUMPY_VERSION=1.11.3
fail on travis (linux 64) because conda is unable to resolve dependencies.
https://travis-ci.org/quantopian/trading_calendars/jobs/554823104

@richafrank I guess this issue can be closed.

Ah, thanks!


[SvgIcon] classes prop has no effect on svg styling

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Setting colorSecondary, colorPrimary rules in classes SvgIconProp has no influence on the resulting style.
Expected Behavior 🤔
Setting colorSecondary, colorPrimary rules in classes SvgIconProp should change SvgIcon when color='primary' | color='secondary' SvgIconProp is provieded acording to https://material-ui.com/ru/api/svg-icon/#css
Steps to Reproduce 🕹
https://codesandbox.io/s/sad-euler-xtol8?file=/src/Demo.js
Your Environment 🌎

Tech
Version

Material-UI
v4.9.11

React
6.9.19

Tip, if you want to use a .blue global class name, you need to write the CSS for it.

@oliviertassinari i was thinking i need to provide actual values to classes prop and not a class name... my fault.


[1.19] Sewer keeps

industrial-foregoing-1.19.2-<IP_ADDRESS>-3.jar
titanium-1.19.2-3.7.0-23.jar

buuz how could you not fix this issue it so simple....

It just keeps

fat finger


Settings hosts option results in /etc/hosts: read-only file system

Describe the bug
Not sure if I am doing anything wrong, but I cannot use the hostsoption. When setting it, devenv up asks for the root password but then fails
» devenv up
Building shell ...
[✖] error: open /etc/hosts: read-only file system

which is not surprising because /etc/hosts is a link to the nix-store.
To reproduce
{ pkgs, ... }:

{
  hosts = {
    "mytestdomain.test" = "<IP_ADDRESS>";
  };

  processes.test.exec = "ping mytestdomain.test";

}

Version
0.6.3

Are you on NixOS?

Yes, I use nixos

For NixOS we'd need to print what changes to apply to your profile.
It disallows modifying /etc/hosts so we can't do automatically.


Update README

The "How?" section was incorrect:

UnCSS uses PostCSS instead of css-parse
UnCSS does not extract the stylesheets from the HTML
when used as a PostCSS plugin.

I decided it would be best to just remove point number 2 to make the docs more environment-agnostic.
CC: @mikelambert

Coverage remained the same at 96.266% when pulling f53ca85ebebedb2684a7f7198bc61c68498d0b1c on RyanZim:docs into aea178e95c9215fa98f7b6071dac9969f8be2227 on giakki:master.


Update ForecastSQL.forecast_version columns to accept huggingface commit SHAs

Pull Request
Updates ForecastSQL.forecast_version to 40 characters, allowing for huggingface commit SHAs
How Has This Been Tested?
Tested locally and against India development DB

[x] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

No longer needed


Login redirect url loses part after #

Description of bug
The login redirect mechanism in DNN should ensure that the entire originally requested url is used when redirecting the user to log in and return. I have links using client-side routing (i.e. that include parameters after #) that do not survive this process.
Steps to reproduce
Link that was clicked on:
https://mysite.com/fr-fr/Material/moduleId/665/CompanyId/17/controller/Products/action/Index#/p/25102

Redirects to
https://mysite.com/fr-fr/Login?returnurl=%2ffr-fr%2fMaterial%2fmoduleId%2f665%2fCompanyId%2f17%2fcontroller%2fProducts%2faction%2fIndex#/p/25102

And then
http://mysite.com/fr-fr/Material/moduleId/665/CompanyId/17/controller/Products/action/Index#/

Current behavior
Note the missing p/25102 at the end of the last url.
Expected behavior
We should have the full url. I suspect we should also have the encoding of the forward slashes in the redirecturl correct. That may even be the root cause.
Affected version

[x] 09.06.01

Affected browser
All

I looked into this a bit as well, it looks like we need to properly encode the URL hash, as the /, as is, is a valid character
For Reference: https://stackoverflow.com/questions/2849756/list-of-valid-characters-for-the-fragment-identifier-in-an-url

Right. From my research we go through HttpUtility.UrlEncode, but running this in isolation it works as expected. So we are doing something here that changes the url before the encoding or something.


Creating Baseline of Postgres Database Columns for Future Migrations

This sets up the Flask-Migrate tool and checks in the basic stubs needed to run it. From here out, we can just create migrations and apply them.
I'm particularly interested in feedback about what should be checked in here. I did this largely based on the work here: https://realpython.com/blog/python/flask-by-example-part-2-postgres-sqlalchemy-and-alembic/
Based on those examples and working with Heroku, checking in the actual migrations seems like the right idea so that we don't all have individual copies. However, there are some extra files that come with the setup which may not be necessary. It will likely take some experimentation to figure out what is needed vs not needed.
Anyway, in order to create future migrations (after you've setup postgres locally), you'll just do the following:

Make a change to the models.py file to generate new schema.
Run python manage.py db migrate to generate a migration for the new schema.
Once you've check in your code, this migration will be shared so that other developers and heroku can all pick it up.
Run python manage.py db upgrade to apply the migration to your local database.

Similarly, run heroku run python manage.py db upgrade once the code is checked in and pushed to heroku in order to apply the migration there. I'm working on a means of automating this, by for now the manual approach can be used.
This approach should work for simple migrations, such as adding a new column, renaming a column, or deleting a column (which in my estimate will likely be a majority of what we encounter). If we need to do a complex migration where data might be copied to several different columns or otherwise split in some way, we will have to modify the migration file created under migrations/versions. There are already stubs there to do the basic operations which we would expand. There will be notes with all of this information to follow shortly.

This change is

@jpevarnek Please take a look as well. This is just the basic setup without actually migrating anything yet.

Super cool, thanks for taking care of this!  I am not sure how everything works here exactly.  So, I'm looking forward to the forthcoming documentation and trying this out.

Review status: 0 of 9 files reviewed at latest revision, 2 unresolved discussions.

manage.py, line 8 [r2] (raw file):
Is this a file that you created?  i.e. can we rename it to something more clear, "db_migration_manager.py" or "manage_db_migration.py"?  Can we also have a module docstring to describe what this is for?

migrations/README, line 1 [r2] (raw file):
nit: remove line return

Comments from Reviewable

Since the file manage.py was renamed to db_migration_manager.py, in the sample commands listed on the PR, replace manage with db_migration_manager.

Reviewed 7 of 8 files at r1, 2 of 2 files at r2, 3 of 3 files at r3.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

migrations/README, line 1 [r2] (raw file):
This file was auto generated as part of the migration init process, so I actually didn't even make it. I think you mean add line return here since there wasn't one, but regardless done.

manage.py, line 8 [r2] (raw file):
This was created following the example from https://realpython.com/blog/python/flask-by-example-part-2-postgres-sqlalchemy-and-alembic/ under the Local Migration section, but it will work with any name. Done and done. FYI for the commands to run this later on, replace manage.py with the new name obviously, as in python db_migration_manager.py db migrate .

Comments from Reviewable


Responsive design giraudremi92 branch

Add the Responsive behavior for Laptop screen, Tablet and mobile.
With some changes in CSS and React components
As described here : Add responsive behavior #19
https://github.com/Pyrrha-Platform/Pyrrha-Website/issues/19
Is this related to an open issue?
https://github.com/Pyrrha-Platform/Pyrrha-Website/issues/19
+++ Website ReadMe Review #17
Add Yarn install in the Readme File
https://github.com/Pyrrha-Platform/Pyrrha-Website/issues/17

Thank you @giraudremi92!
I still see some issues when I resize in Chrome or test the mobile/tablet views. But those are things we can iterate on to improve. Merging.

@krook
Ok let me know the issues then i could maybe resolve them
Thank you

Very strange. When I built and ran your fork locally, it seemed like it still had some of the same issues as in production, but when I merged and it pushed a build to https://pyrrha-platform.org/ it shows correctly.
So, very nice job! It looks great.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added method to do ZIP with array of arrays.

:tophat: What? Why?
There is already a zipWith method but i think this one is more useful.
Zip an array of arrays.
this:
@[@[@1, @2, @3], @[@4, @5, @6], @[@7, @8, @9]]

transforms into this:
@[@[@1, @4, @7], @[@2, @5, @9], @[@3, @6, @9]]

:warning: all the arrays should have the same lenght
:ghost: GIF

@masylum @sergigracia

:+1:

I don't see tests nor a PR to the original repo!

The PR to the original Repo is this one:
https://github.com/robb/Underscore.m/pull/60
i'm working in tests, but they use a very old cocoapods version and i can't run them :(

I'll add tests in another moment, I can't even compile their test project.

:+1:


Pandoc Pluging do not find installed pandoc on Mac

Hello,
I am not sure if this is a real issue but after installed Pandoc via brew and checked that pandoc -v returned the installed version, I installed the pandoc Inldrop plugin but at each start of Inkdrop I got the error message saying that pandoc is not installed.
If you can help me on this, I would appreciate it.

As I didn't have a Mac available at the initial time of creation I've been unable to test it on MacOS so it is entirely possible that it simply does not recognize Pandoc on Mac. I do have a Mac now so I should be able to troubleshoot it soon-ish.
I see that there has been some discussion about this on the forum and I think I'll just adopt Lukas' approach of making the pandoc exec path a setting but I'll also try to see if I can get it to work automatically on Mac.

I have taken an initial look at this and have gotten the Pandoc detection working on MacOS by having a setting that allows the user to manually set the full path. Unfortunately I have also discovered that the current method of invoking Pandoc does not work on MacOS as it does not allow for using a custom Pandoc path.

@Shirumipi I have released a new version that should address this issue, at least it did with my limited testing on MacOS. This new version has an option that allow you to manually set the path for the Pandoc executable, similarly to what Lukas described but as part of the plugin settings instead of in the init file.
If you could try it out and see and report back that would be great.

@Thereatra Thanks a million for your quick reply and fix.
I have just tried it and it works perfectly now!!
I wish you a wonderful day.
Shirumipi


imagecreatefromjpeg(): gd-jpeg, libjpeg: recoverable error: Premature end of JPEG file

I am using this package with laravel and for some files that I am trying to resize it happens randomly I get this error:

imagecreatefromjpeg(): gd-jpeg, libjpeg: recoverable error: Premature end of JPEG file"

Stack Frame
/Applications/MAMP/htdocs/laravelweb/vendor/intervention/image/src/Intervention/Image/Image.php

       case IMG_PNG:
       case 3:
       $this->resource = imagecreatefrompng($path);
       break;

       case IMG_JPG:
       case 2:
       $this->resource = imagecreatefromjpeg($path);
       break;

Also see this thread: https://github.com/owncloud/core/pull/21950
Perhaps we should add that here also?

Also, with Laravel, I think the problem is that instead of warnings, that options generates notices. And notices are still triggering the error handler. So you might want to change your error reporting settings to ignore notices.

@barryvdh I am getting this as well. What exactly is the Laravel fix you are mentioning above? How would one specifically ignore this notice. I am finding it behaves differently on different environments. it definitely starts with the image though. In my case it seems to often be images taken via an Android device and then uploaded to our web app. On some deployments it causes:
Intervention\Image\Exception\NotReadableException
And on others I get
imagecreatefromjpeg(): gd-jpeg, libjpeg: recoverable error:
In the second case the images all get created just fine though despite the error getting thrown.

You can set default value of  gd.jpeg_ignore_warning=1 in php.ini
OR
You can set it like this ini_set('gd.jpeg_ignore_warning', true); in your PHP script before calling  imagecreatefromjpeg()
After implementing any of the above, GD Library will ignore the error where it use to fail and  imagecreatefromjpeg()  will return an image resource identifier.
Note: In PHP 7.1.0 the default value of gd.jpeg_ignore_warning has been changed from 0 to 1.
Reference

Thank you. It solve my problem.


flutter web and django restapi

hi guys
i use flutter for my website and django for backend but when i publish my website and place it on real host the data show minified:b6
but in android studio i have no problem and every thing is ok.

how can i solve this!?

Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel beta, 2.9.0-0.1.pre, on Microsoft Windows [Version 10.0.19044.1586], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
[√] Chrome - develop for the web
[√] Android Studio (version 2020.3)
[√] Connected device (2 available)

i have no error in android studio and every thing is ok...i get data from real host in android studio

go to this website
www.afsoone.com

ok i fix the error i have Mixed Content with django CORS


Have "M lvl skill" show your total EXP too

🚀 Feature Proposal
This would help determine your exact experience, as there isnt a way to check that right now. Would also be really handy for skills where you're past 99 and can't check exact xp.
IE: +m lvl slayer
"Your slayer level is 73. You are 100 XP away from level 74. Your total Slayer experience is 734,561"

been added to the bot, thanks lily!


Release v3.0.0

Meta-issue to track the work that is left to be done before a 3.0.0 stable release. May evolve over time.

[x] Specs work. APIs interactions have good coverage, but not the higher level use of the gem (#15 #16 #17)
[x] Async operations. Some requests trigger a background operation that can be followed via a dedicated endpoint. This endpoint is available, but we could propose a method on the response to inspect it directly. (#15)
[x] inspect configuration. Configuration is done globally at the moment, and some things may be missing. We may want to allow per-client configuration as well. (#17)
[x] inspect dependencies. We don't have many, but let's make sure the version we need all of them and that the versions constraint are appropriate. (#17)
[ ] WS/Streaming. Nothing is done related to those matter. We don't want to add more dependencies than needed, so we'll have to make choices on what we want to include in this gem. This may end up being an officially-supported extension.
[ ] Documentation and rdoc. Let's document this gem better. Examples are nice, but they only go so far.
[ ] (3.0 or 3.1) Some endpoints are missing. This is loosely related to the state of our developers documentation, but some endpoints are not publicly documented, some are not in this gem, and some are both of those. (#14)
[ ] Deprecate v2 + post install message, as a mean of advertising that the gem name has changed

I don't know if it's a step you want to add in this list. But when we release the v3, we probably want to deprecate the old gem to force users to move to this new gem, don't we?

Right. We'll push a new minor 2.x version that adds the deprecation/post-install warning, thanks for reminding me.

Closing this. About the unchecked tasks, we'll reconsider them at a later time.

Closing this. About the unchecked tasks, we'll reconsider them at a later time.

I'm kind of afraid these unchecked tasks get lost. What are you planning so that we don't forget about them?

The first 3, nothing. They will be raised again by usage if needed. The fourth one is different, I'll file a ticket for it

OK, thanks :)


osbs option to be deprecated

osbs option '-c' for component name might be deprecated, as this value is anyway overwritten with value from Dockerfile (BZComponent), unless we want to allow not to specify Component name in Dockerfile and rely in that case on this option

Related: https://github.com/projectatomic/atomic-reactor/issues/580

I think it would make sense to make it optional for a start

if this value is really overwritten by component in Dockerfile, that isn't how it was intended (I guess), so if it has to override Component value in Dockerfile it should be optional, but also you should keep in mind the case when it won't be specified and also won't be find in the Dockerfile


Can not add new POSes

Whenever I try and add a new POS since the latest EVE update (either auto or manual), I get this error:
http://pastebin.com/0uLiaXdd
Thanks for any help!

Have you updated the EWS database with the most recent SDE since the latest update?
And can you re-post that bin with formatting? The lack of lines makes it painful to read through.

@illuser I copied it out of the browser window just as it was outputted; I'm not familiar enough with python to parse that out.

Are you the one running the ews app you're using? If so, can you copy the error logs which were generated server-side?

From the traceback you posted, check that RabbitMQ and Celery are running. It appears that kombu was unable to connect to the RabbtiMQ port (ConnectionRefused exception).

RabbitMQ had indeed crashed because the ec2 ran out of memory.  Reconfiguring some processes freed the memory up and got it working again.

Good to hear.
Make sure you change DEBUG set to False in your local_settings.py file. Running with DEBUG=True will cause the async worker to leak memory due to SQL query caching for debugging purposes.


Need to rework Multimodal pipeline for ollama, maybe for other APIs as well

In reworking Ollama support for LLaVa, I found the pipeline for Multimodal chats to be unnecessary. Ollama only requires the image to be attached to the message being sent. The multimodal model takes care of the rest.
Currently the vision chat pipeline seems to be for separate Vision/CLIP models and LLMs (one describes the picture, and then the returned result is put into an LLM).

Any thoughts on what to do for the Vision prompt? It seems to be unnecessary.
I was thinking the default behavior will be to use the normal System prompt, and the Vision prompt could be hidden behind a checkbox, i.e. turn it into a custom Vision prompt.

Also, was the desired UX for vision to be:

Press the "Take Picture" button to append the picture to the next message, then send the message to get a response.
Press the "Take Picture" button to send the picture and get a response, without sending any text to accompany the picture. (Still sending the rest of the chat history, just no new text.)
Type something in the chatbox, then press the "Take Picture" button to send both the picture and message at the same time.

The way the UX works right now, it works closest to to option 2, but it appends the response from the vision model to the previous chat message. (By the way, the text in the chatbox gets completely discarded.)
I think the best way to make Amica be as similar to chatting as possible is to allow both 2 and 1. My chat habits are often to send an image and then type something quickly, send an image alone, or append images to my message before sending.
When I was trying to rewrite the vision pipeline, I ran into trouble deciding how it should be implemented, and I realized that it would depend on the UX. So, any opinions on this matter?

Arbius has a $200 AIUS bounty for this issue!
Brief: Complete the desired UX vision to allow take picture + text and get response as well. Rework pipeline as outlined.
Please read carefully:
To begin work on a bounty, reply by saying “I claim this bounty” - you will have 48 hours to submit your PR before someone else may attempt to claim this bounty.
To complete the bounty, within 48 hours of claiming, reply with a link to your PR referencing this issue and an Ethereum address. You must comply with reviewers comments and have the PR merged to receive the bounty reward. Please be sure to focus on quality submissions to minimize the amount of time reviewers must take.


Production Build won't work on https server

Such a lifesaver having this! Working on a project on local server, I done the build but it don't seem to want to load when using something like Netlify Drops - https://app.netlify.com/drop
Any ideas why Netlify won't take the production build from this repo?

Hi - thanks! I haven't touched this for a while, but something it could be is that by default parcel expects to be at the root of a domain (not sure why, always seemed like a strange choice when relative paths can be used!).
Try adding --public-url ./ to the package.json arguments, like this: parcel build src/index.html --public-url ./
If that works for you I'll probably add it to the actual repo too.
docs: https://parceljs.org/cli.html#set-the-public-url-to-serve-on

Yes that worked! Thanks a lot!


Can't test for no action dispatched?

Is there a way to test that there are no actions? My function may have a condition before dispatching the action.
it('dispatches no actions', (done) => {
    const store = mockStore({}, [], done);

    store.dispatch(
        (dispatch) => {}
    );
});

This test fails with:
Error: timeout of 2000ms exceeded. Ensure the done() callback is being called in this test.

It's tricky, given the nature of asynchronous testing. I don't see how it would be possible since we cannot determine when the code you're testing has finished executing, unless you call done.

You could call done yourself after a brief timeout. Then if you do get an unexpected action, you'll get an error like expected [whatever action] to equal undefined].

My function may have a condition before dispatching the action.

You could export that function and directly test it instead.

I'm using same thought of @bhritchie under Jasmine, test like this:
it('should not dispatch any action', (done) => {
  // I write actions like this because of #12 Jasmine issue
  // https://github.com/arnaudbenard/redux-mock-store/issues/12#issuecomment-182067964
  const expectedActions = [
    act => expect(act).toEqual(jasmine.any(Object)),
  ];

  // pass `done.fail` to mockStore so that
  // any dispatched action will fail this case.
  const store = mockStore(testState, expectedActions, done.fail);

  store.dispatch(myKawaiiAction());

  // if no action dispatched after a proper timeout,
  // then pass it.
  setTimeout(() => done(), 200);
});

Seems fair enough. The only thing that bugs me is the arbitrary timeout value. What is an acceptable time to wait?

New API in 1.0 works well, the problem solved :+1:


Doctests do not remove built files if they fail

In run-tests.py, if the doctests fail locally for some reason, they do not delete the built files and cause problems upon invoking the doctests again. This is because nbsphinx duplicates some of the doctrees upon rebuilds (it is not meant to work with the doctests). The command should be fixed in order to delete the built files upon invoking the doctests, or just the .doctrees folder inside them so that the error logs can be retained.

Hi I would like to work on this issue. I am new to open-source, so I bit of guidance will be helpful.  Kindly assign it to me

Hi @Agnik7, thanks for the help! For more explanation, we have a method in run-tests.py defined like this, which runs these tests through the command line (python run-tests.py --doctest or nox -s doctests)
https://github.com/pybamm-team/PyBaMM/blob/develop/run-tests.py#L61-L97
The issue is with the subprocess.Popen instance, because the process remains open until the terminal window is closed even if the doctests. Therefore, the built files (inside the docs/build/ folder) are not deleted. We should alter it to use subprocess.check_call or subprocess.run with check=True, as appropriate, such that it fails on receiving a non-zero exit code (the doctests are supposed to treat warnings as errors) and closes the process. Try commenting out this line shutil.rmtree("docs/build") and run the doctests again to debug this issue. The ideal functionality is that we should remove the .doctrees/ folder inside docs/build/ regardless of whether the doctests fail or pass so that the doctrees can be re-built.
It would be helpful to set up the project locally using the source installation instructions based on your platform or inside a Docker container (see https://docs.pybamm.org/en/latest/source/user_guide/installation), and then you may try building the documentation and running the doctests by reading through the contributor guide. @Saransh-cpp, could you please assign them this?

Hi @agriyakhetarpal @Saransh-cpp if no one is working on this could you please assign it to me ?Thanks :D

Hi @shubhambhar007, how is this issue coming along?

Hi @agriyakhetarpal ,apologies for the delay , was on call this month :(
Will raise the PR by tonight for review, max by tomorrow morning.
Thanks !!

No worries, thanks for the update!


feat: populate preimage in ledger transactions from lnpayments

Description
This is the first proposal to populate the revealedPreImage property in WalletTransaction objects that eventually get returned out via the api under settlementVia.
The approach here was to:

record the preimage on successful send to lnpayments collection
record the preimage on update of settled pending  payment to lnpayments collection
add the revealedPreImage property to LedgerTransaction type and implement with a join from the LedgerService requests to the lnpayments collection.

Feedback
We decided that this approach isn't ideal for 2 reasons:

the dependency from LedgerService on lnpayments collection is dirty
the join approach may be unnecessary premature optimization

Our first alternative discussed was to simply add a new revealedPreImage property in the metadata for transactions.

this solves the join complexity
this is bad because we would have to edit transactions after-the-fact (on pending payment update) and we'd ideally like to have transactions be immutable

Our second alternative was to create a new transactions_metadata collection to store mutable metadata related to transactions to:

this would live within the Ledger service and solves the crossing-boundaries issue
this would solve the "immutable transactions" issue because we would only update this other collection
we will do 2 separate calls instead of a single join for simplicity to start, and will re-visit alternatives if performance becomes an issue

Conclusion
This PR will be closed in favour a new one where we implement the new transactions_metadata collection and write/read the revealedPreImage property from there.

Note: the first part of this work was originally targeted in PR #1037 but that PR has since been rebased to prep for the new "metadata collection" approach.


implemented stack and queue in Go

This PR implements stack and queue in Go.
Note: code/go/stack-queue.goalso includes a func main where all the tests are written but I'm not sure where to include these (or if they are even required?)

One could think about commenting that behind the declaration to make it clear to beginners what interface{} means in this context

I'm all for making comments in the code if there is some part of an algorithm that can be difficult to understand but I am not a fan of making comments describing language syntax, e.g. what T is in Array<T> in Java.

The code is able to run with go run stack-queue.go on its own that way, that's enough. It doesn't need to be embedded somewhere.

We want to produce executables (as much as possible for all compiled languages) with the build system in construction, so I am not sure this point is valid right now.

@Amaras Does this PR conflict with the build system?

We haven't implemented the Go build system yet, but we need to think about the future.
It doesn't conflict with anything yet, though

@Amaras do I need to make any changes before this can get merged?

go run stack-queue.go should work


ISPN-9020 Unify RemoteQueryDisableIndexingTest and RemoteQueryDisable…

…LegacyIndexingTest
One test was mostly duplicating the other; only one line was different.
https://issues.jboss.org/browse/ISPN-9020

retest this please

c'mon jenkins, you can do it

Trivial enough for me :)


StackBlitz

Tasks

[ ] Create StackBlitz account.
[ ] Connect all examples in the examples directory to StackBlitz.

References

https://stackblitz.com
https://github.com/vidstack/player/blob/main/docs/architecture/overview.md#examples

Closing for now, I'll create a separate ticket that address player examples better.


You can’t skip to the main content when using a screen reader

This fails WCAG 2.1 success criterion 2.4.1: bypass blocks.

See also #112

I also wonder if adding skip links would be good; also just tried to test the skipping with Tab and it "disappears" while you're on the first fold while it goes on highlighting the links below the fold — i.e. it's not scrolling with the skips.

Should’ve noted this here too, but the reason why nothing seems to be highlighted (using tabs) after the top nav, is because it navigates through the links in the slide-out menu that’s brought out by the hamburger menu icon. This is probably the main reason why we need a "skip to content" link so that we can skip not only the main nav items (PRODUCTS / IMPACT / ABOUT / TEAM) but also everything behind the secondary slide-out menu.
A "skip to content" link could be one that is placed at the top of the page and uses the utility class .sr-only so that it doesn’t disrupt the layout but is still available to screen readers.

To add to this, I think we should also use .sr-only-focusable for when the .sr-only element is in focus through keyboard so that a user that navigates via mouse never sees it.
As per this Stack Overflow comment.

Yes! I am not really using Bootstrap, just some components and classes that are useful, like sr-only. As you can see, I only add them one at a time when I realise I need them. Feel free to add it to the utilities: https://github.com/oaworks/oa-works/blob/main/src/_includes/css/utilities.css#L1

Done in https://github.com/oaworks/oa-works/pull/147/commits/89a9a7df37f22b9a6ce8f4805e434c64912f3565
Updated a11y statement accordingly


Adding a HowTo Guide

@roryj Addressed your comments

Uploaded a new commit with an introduction to CloudFormation stacks


add suppress_output

Description
I added an argument to the constructor (and a variable to store that value) that controls whether the other methods (reader_header, read_fields) will print output messages.
Rationale
This feature will make the output look cleaner, e.g., when using DataReader and plotting inside a loop.

Thanks for your contribution @caoliheng !


HAL 32: Use of deprecated go version

Will be fixed with sdk upgrade

Closing as it seems no longer relevant with the current state of the project. Feel free to reopen it.


Fix overview table column headings

Currently, there's an extra blank row for the colum headings

If you try rendering without it, I think the table doesnt work...
Lauren
On Sun, Jul 8, 2018 at 22:17 Brian Lin<EMAIL_ADDRESS>wrote:

Currently, there's an extra blank row for the colum headings
You can view, comment on, or merge this pull request online at:
https://github.com/opensciencegrid/user-school-2018/pull/55
Commit Summary

Fix overview table column headings

File Changes

M docs/curriculum/overview.md
https://github.com/opensciencegrid/user-school-2018/pull/55/files#diff-0
(3)

Patch Links:

https://github.com/opensciencegrid/user-school-2018/pull/55.patch
https://github.com/opensciencegrid/user-school-2018/pull/55.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/opensciencegrid/user-school-2018/pull/55, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADmlRKikUtFB4tRn9MKHi1q0-Rb87wptks5uEstRgaJpZM4VHBfm
.

--
Sent from Gmail Mobile

I just put the dashes below the morning and afternoon headings and it looks better when rendering locally:

This looks good to me.


Adds support for validate_only to the transaction

Issues:
Fixes #653
Problem:
The validateOnly functionality of transactions was not exposed to
the end user. Therefore, the user could never know if the transaction
would pass or fail if they ran it without actually running it
Analysis:
This patch adds support to the TransactionContextManager so that
it allows for the validate_only parameter to be provided.
By default, this parameter is set to False to maintain backwards
compatibility.
Tests:
tests/functional/tm/transaction/test_transaction.py::TestTransaction::test_validate_only_true

tests ran as expected and code looks good to me.


Monolithic board Electrical Assembly Instructions errors

Electrical Assembly Instructions OMITTED D1 and F1.
They're clearly installed in the photos, but no reference in the instructions.
They're the last left-over parts and slots prior to placing the standoffs.
On the PCB J13 (GND +12V) is omitted from the instructions.
On the BOM, J13 is the 5.08mm 2P side entry terminal block.
But that is too large for the holes which match the 3.5mm J6 for the Arduino shield
(not to be confused, but it is, with the J6-7 2x20 40P ribbon cable connectors.)
There are 3 slots on the PCB for the 5.08mm 2P connectors labeled J15 (BATTERY IN), J16 (POWER SWITCH) & again J16 (BATTERY OUT).
Not sure what this J13 is supposed to do, but one of the two 3.5cc 2P terminal blocks is going into it.
Jim Phelan
Mars rover project lead
USAi Labs (formerly Houston Robotics)

@JHPHELAN Sorry I Must have missed this issue. Solving this currently by adding the instructions for which piece goes here. It is just a breakout of the 12V regulator line. I'm sure you had figured that out by now.

Fixed in commit 4b494c4