[Hotfix][Connector-V2][AssertSink]Fix the problem that the data type

Purpose of this pull request
close #4635

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

I don't know what caused the failure.

Because assert sink is a basic component for connectors, there are many test cases involved. If you want to update parameters, please update all config files too. This is not a prod connector so there is little risk of compatibility, and I see anyone submit another issue #4566, IMO we should refactor this connector and make this connector more powerful. Hey guys, what do you think about it? @EricJoy2048 @hailin0 @Hisoka-X


If dependabot and secret is missing: creates prevew comment that PR must be closed and opened by a human first

Allowint secrets for dependabot.


Strider result causes 500 in omnicorp_overlay

strider_result_omnicorp_500.json.txt

i have the fix for this. PR forth coming.


Update license

This PR updates the license to reflect the move to the dask org
cc @jcrist does this seem reasonable to you?

Just noting that the pre-commit failure is unrelated to the changes here. I'm going to merge after other CI builds are done and handle pre-commit separately.


permissions by subdomain or another indicator

We have an app with many subdomains.
A single user could have a role on a single subdomain, or many subdomains.
Is there an easy way to add a 'subdomain_id' column to permissions or roles?
In other words, I've got an app that is organized by companies.  $companies can have $posts.  I want certain users to have different $roles but only based on the $company, so a $user with a $role at company_1 would not have that role at company_2, but might also have that role in company_3.

You'll probably need two things to achive this result:
Make the migrations
In laravel you can edit columns with migrations.
Edit the model
You'll need to edit the model so you can easly attach/retrive subdomain_id value. See Relationship for this.


Param Initialization

Hi, thanks for sharing the code for your insightful work!
From your code I didn't find any explicit param initialization, so do you initialize your weights using the default initialization? (which is kaiming_uniform for convolution layers)
Thanks!

In particular, could you specify the weight initialization of the 1*1 convs in the CrissCrossAttention blocks?


Remove duplicate editor settings definitions

I noticed that EDITOR_DEF is overused in many places. This is a very bad idea, because each time you use EDITOR_DEF, a cat starts crying somewhere you need to provide a default value. This leads to some ridiculous situations like a default value repeated 7 times. The actual value then depends on initialization order. Luckily I spotted only one setting that had inconsistent defaults across definitions.
My absolute favorite was this:

Same thing, defined twice few lines apart.
I went over the codebase and cleaned it up by removing duplicate defs of the same settings and changing them to EDITOR_GET. We have most of the settings defined in _load_defaults() method of EditorSettings, so if there were some additional EDITOR_DEFs, I left the ones in the singleton. Then I left stuff defined in a constructor over inlined settings. If some settings were on equivalent level (i.e. not apparent which one comes first), I moved the definition to _load_defaults().
We should have some policy for using EDITOR_DEF. Repeating defaults multiple times can lead to errors, but also IMO it's just dumb xd
Another things to consider could be:

moving all editor setting definitions to _load_defaults(), so they are all in one place
changing all usage of EditorSettings::get_singleton()->get() to EDITOR_GET (it's the same thing, except it displays error when setting is missing and also it's much shorter)

Also Project Settings likely need the same treatment.

moving all editor setting definitions to _load_defaults(), so they are all in one place

Yeah I have proposed this a couple of times but there was some push back from reduz I think. This is also relevant to this proposal/issue https://github.com/godotengine/godot-proposals/issues/1339#issuecomment-671324409

Thanks!


cargo install build err

i tried installing via cargo install cargo-nextest, but multiple errors occurred:

here is my rust tool chain version:
λ  cargo -V
cargo 1.58.0-nightly (294967c53 2021-11-29)

λ rustc -V
rustc 1.59.0-nightly (0b6f079e4 2021-12-07)

And my OS:
Windows 10 Enterprise 20H2

I see what happened here.
I tried open the powershell as administrator and problem solved (both stable and nightly build)!
Now is ok to run:
cargo nextest run
    Finished test [unoptimized + debuginfo] target(s) in 0.01s
    Starting 26 tests across 1 binaries
        PASS [   0.017s] collection list::vector::test::new_test
        PASS [   0.022s] collection list::linked_list::test::test_contains
        PASS [   0.029s] collection list::linked_list::test::test_get_idx
        PASS [   0.034s] collection list::linked_list::test::test_clear
        PASS [   0.041s] collection list::linked_list::test::test_insert_idx_err
        PASS [   0.046s] collection list::linked_list::test::test_get_idx_err
        PASS [   0.052s] collection list::linked_list::test::test_remove_idx
        PASS [   0.058s] collection list::linked_list::test::test_compiling
        PASS [   0.064s] collection list::vector::test::test_drain
        PASS [   0.070s] collection list::vector::test::test_into_iter
        PASS [   0.075s] collection list::vector::test::compiling_test
        PASS [   0.080s] collection list::linked_list::test::test_iterator
        PASS [   0.086s] collection list::linked_list::test::test_remove_idx_err
        PASS [   0.092s] collection list::linked_list::test::test_insert_idx
        PASS [   0.097s] collection list::linked_list::test::test_push_and_pop
        PASS [   0.103s] collection list::linked_list::test::test_peak
        PASS [   0.091s] collection list::vector::test::test_iter
        PASS [   0.092s] collection list::vector::test::test_iter_mut
        PASS [   0.090s] collection list::vector::test::test_pop
        PASS [   0.090s] collection list::vector::test::test_push
        PASS [   0.089s] collection list::vector::test::test_zst
        PASS [   0.090s] collection tree::binary_search_tree::tests::test_compiling
        PASS [   0.089s] collection tree::binary_search_tree::tests::test_insert
        PASS [   0.089s] collection tree::binary_search_tree::tests::test_pop_min
        PASS [   0.089s] collection tree::binary_search_tree::tests::test_pop_min2
        PASS [   0.088s] collection tree::binary_search_tree::tests::test_remove
     Summary [   0.158s] 26 tests run: 26 passed, 0 skipped

Thanks a lot! ❤


Update Russian docs

Hi, @lex111.
This adds docs/ru/guides/testing-SFCs-with-jest.md, docs/ru/guides/testing-SFCs-with-karma.md, docs/ru/guides/testing-SFCs-with-mocha-webpack.md.
They are not necessary since they are renamed at a below pull request.
https://github.com/vuejs/vue-test-utils/pull/804/files#diff-9d913dee9403ea31f1bae9d21ac68d4a


Getting issue WebKit encountered an internal error when using Fetch API

Describe the bug
When I use Fetch API to fetch data from my API, my app is giving an error that says, "Failed to load resource, WebKit encountered an internal error". Note that it works fine when doing anything else except fetch
To Reproduce
Steps to reproduce the behavior:

Try using fetch API.

Screenshots
If applicable, add screenshots to help explain your problem.
Platform and Versions (please complete the following information):

OS: Gentoo GNU/Linux x86/64
Node:14.16.1
NPM: 6.14.12
Rustc:1.52.1

Can you please share the csp value in your tauri.conf.json and also paste the results of yarn tauri info?

I can use fetch normally on Linux. Check the console to see if it's not a CORS issue.
Here's an example API I used to test it:
const express = require('express');
const cors = require('cors');

const app = express();
app.use(cors())

app.get('/', (req, res) => {
    res.json({
        message: 'Hello World'
    });
});

app.listen(2020, () => {
    console.log('server is listening on port 2020');
});

And this is how I called it:
fetch('http://localhost:2020').then(r => r.json()).then(console.log).catch(console.error)

My CSP on tauri.conf.json is default-src blob: data: filesystem: ws: wss: http: https: tauri: 'unsafe-eval' 'unsafe-inline' 'self' img-src: 'self'.
Please reach us on Discord if it still doesn't work.


Fix FtpRawEvent

Fix fields of FtpRawEvent, FtpJsonOutput.
Fix warning in status.rs when build to release.
Modified collect_records to continue collecting even if error
data is included in the data.

Codecov Report
Patch coverage: 85.45% and project coverage change: +0.24 :tada:

Comparison is base (4303cd2) 67.96% compared to head (8f50def) 68.21%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #456      +/-   ##
==========================================
+ Coverage   67.96%   68.21%   +0.24%
==========================================
  Files          21       21
  Lines        4005     4058      +53
==========================================
+ Hits         2722     2768      +46
- Misses       1283     1290       +7

Impacted Files
Coverage Δ

src/graphql/export.rs
15.98% <0.00%> (-0.19%)
:arrow_down:

src/graphql/status.rs
3.84% <0.00%> (ø)

src/graphql.rs
74.07% <62.50%> (-0.51%)
:arrow_down:

src/graphql/network.rs
87.66% <100.00%> (+0.06%)
:arrow_up:

src/ingest/implement.rs
97.83% <100.00%> (+0.34%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@BLYKIM Version을 0.12.3으로 바꿉시다.


Fix: HotModuleReplacement runtime property bug

HotModuleReplacementPlugin raise warning:
HotModuleReplacementPlugin

The configured output.hotUpdateMainFilename doesn't lead to unique filenames per runtime and HMR update differs between runtimes.
This might lead to incorrect runtime behavior of the applied update.
To fix this, make sure to include [runtime] in the output.hotUpdateMainFilename option, or use the default config.

but when adding the [runtime] parameter to the hotUpdateMainFilename and hotUpdateChunkFilename string, this property is absent in the Data object passed to the filename function
getAssetPath(filename, data) {
   return this.hooks.assetPath.call(
     typeof filename === "function"? filename(data): filename,
     data,
     undefined
   );
}

This PR proposes to add a "runtime" property to the Data object

What kind of change does this PR introduce?

Bugfix
Did you add tests for your changes?

No
Does this PR introduce a breaking change?

No
What needs to be documented once your changes are merged?

No

I signed the license but it is still pending

Pending — Contributor License Agreement is not signed yet.

Can somebody help me?

@sokra , could you please review?

@sokra , If you don't have extra comments could you please merge this fix to master because this bug break HMR for webpack-plugin-serve

I tried hard to create a test case to trigger this problem, but then I noticed this:

The configured output.hotUpdateMainFilename doesn't lead to unique filenames per runtime and HMR update differs between runtimes.
This might lead to incorrect runtime behavior of the applied update.
To fix this, make sure to include [runtime] in the output.hotUpdateMainFilename option, or use the default config.

but when adding the [runtime] parameter to the hotUpdateMainFilename and hotUpdateChunkFilename string, this property is absent in the Data object passed to the filename function

The warning doesn't tell you to add [runtime] to hotUpdateChunkFilename. Only to hotUpdateMainFilename. Adding it to hotUpdateChunkFilename would be wrong, so it's correct that this doesn't work.


Regarding refreshing the access token

Hello,
So it seems the access token provided by Facebook is temporary and it needs to be refreshed each day I think, is that implemented inside the gem, or do I need to handle that from my end?
Thank you,

You have two options:

You can generate a permanent token and have mechanisms to refresh the token.
Refresh the token daily.

Got it. Thank you!


cpp: Check functions in init_array section

This is workaround for zeroed ".init_array" section. This allows instead of crash debug the issue.
Check function pointer before run in section init_array.
Fixes crashes in xtensa ADSP board for cpp tagged tests.
...
 os:  ** FATAL EXCEPTION
 os:  ** CPU 0 EXCCAUSE 12 (instr PIF data error)
 os:  **  PC 0x00000000 VADDR 0x00000000
 os:  **  PS 0x00060820
 os:  **    (INTLEVEL:0 EXCM: 0 UM:1 RING:0 WOE:1 OWB:8 CALLINC:2)
 os:  **  A0 0xbe01406a  SP 0xbe035f40  A2 0xbe034350  A3 0xbe034354
 os:  **  A4 0x00000000  A5 0xbe03574c  A6 0xbe035764  A7 0x00000000
 os:  **  A8 0xbe012fe4  A9 0xbe035ed0 A10 0xbe0317f7 A11 0xbe035f40
 os:  ** A12 0xbe035f20 A13 0x00000004 A14 0xbe0394f0 A15 0x00000080
 os:  ** LBEG 0xbe011521 LEND 0xbe011527 LCOUNT 0xffffffff
 os:  ** SAR 0x00000004
 os: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 0
 os: Current thread: 0xbe0394f0 (main)
 os: Halting system
...

Which "cpp tagged tests" fail because of this problem?  tests/application_development/cpp passes on my ESP32 hardware with 0.11.1 toolchain.

Why are there null pointers in this array? I would expect that to be the root cause of the failure.

I was wondering the same but a quick google learned me that some compilers put null pointers in the array for padding.

The linker might add padding, but if that's reflected in our array start/end symbols then we have a bug in our own linker script.

The proper fix is here: https://github.com/thesofproject/rimage/pull/8
So this workaround is not needed anymore. However i am thinking to add some ASSERT to indicate the problem. Otherwise Zephyr crashes in the beginning in some cases without logs.

OK, so not a bug in the toolchain per se but in the image conversion utility, which is sort of the same thing.  I wouldn't -1 an __ASSERT, but this is an awfully special case here; it would just be tracking for regressions in the image packer.  I think it's not unreasonable for Zephyr to assume that its tooling is generating correct binaries without defensive coding.

Closing since issue is fixed in the 3rd party tool rimage (https://github.com/thesofproject/rimage/pull/8) which copies ".init_array" section for up_squared_adsp board.


Xdebug is enabled but are we able to debug application using an IDE like phpStorm?

I was looking for a good Docker environment for develop Symfony2|3 apps and I think this one fit. :D I have seen that Xdebug is enabled but are we able to debug application using an IDE like phpStorm? Where is that documentation? If not I would like to contribute to this repository by adding that support and maybe continue improving it

Hello @reypm ,
PR are welcome :)
This pull request can help you! #16
Thanks,
Maxpou

@maxpou ok, I will as soon as I have a time for test all of this but I think something is missing since the Xdebug should be listening on the host IP address in other words: xdebug.remote_host = "<host_ip>" and I am not seeing such line at any config files. Maybe I am wrong, the PR and the comments aren't helpful as well

@meron1122 or @cuewizchris: do you have any idea?

@reypm @maxpou This does work because we're using the xdebug.remote_connect_back setting. From the XDebug docs:

If enabled, the xdebug.remote_host setting is ignored and Xdebug will try to connect to the client that made the HTTP request. It checks the $_SERVER['HTTP_X_FORWARDED_FOR'] and $_SERVER['REMOTE_ADDR'] variables to find out which IP address to use.

@reypm @maxpou This does work because we're using the xdebug.remote_connect_back setting. From the XDebug docs:
If enabled, the xdebug.remote_host setting is ignored and Xdebug will try to connect to the client that made the HTTP request. It checks the $_SERVER['HTTP_X_FORWARDED_FOR'] and $_SERVER['REMOTE_ADDR'] variables to find out which IP address to use.

@cuewizchris thanks, didn't notice that before, closing this issue now, no PR needed


SQL FOR JSON Option 1 Example is incorrect

There is in issue with the example shown in Option 1 of the JSON FOR documentation here https://docs.microsoft.com/en-us/sql/relational-databases/json/format-query-results-as-json-with-for-json-sql-server?view=sql-server-ver15
The example shows SQL code to set the JSON key as UnitPrice, but the example data shows the key becomes Price.
The same type of error happens for the next column the code sets the value to Qty, but the data shows Quantity.

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

ID: ffb5c17b-c567-303f-00b8-0c8a0ad15fb5
Version Independent ID: a4ccc766-4a61-5b7a-e5dd-692e3cebb503
Content: Format Query Results as JSON with FOR JSON - SQL Server
Content Source: docs/relational-databases/json/format-query-results-as-json-with-for-json-sql-server.md
Product: sql
Technology: ****
GitHub Login: @jovanpop-msft
Microsoft Alias: jovanpop

@JamesNJep -- thank you for catching this.  I am updating the article.

@jovanpop-msft -- Jovan, this issue is in the image, and there is one reference to UnitPrice in the text (line 36).

I am submitting an internal PR that will update these images. Thanks for the feedback, @JamesNJep.
#assign:@rwestMSFT


Keltner Channel

Hello,
I have add the three indicators for the Keltner channel.

Hey! Thank you! :)


How to use the encrypted model?

Hi,
How to use the encrypted model ?

You can read this article:
<EMAIL_ADDRESS>

OK. Thank you very much.


Added support for regex label

This PR about support regex label #27.
Main idea is to recognise matcher from query and keep matcher instead label in context.
It allow to use multiple namespaces via query like namespace=~kube-system|openshift-monitoring and still works with one namespace without regex.

query parameter for different types:
👍
a bit tricky :)
q.Add("namespace", "monitoring") // MatchEqual
q.Add("namespace", "~monitoring|default") // MatchRegexp
q.Add("namespace=", "!monitoring") // MatchNotEqual
q.Add("namespace!~monitoring|default", "") // MatchNotRegexp: empty value, all in keys

It makes more sense when converted to query URL :)

namespace

I think it makes more sens to have
q.Add("namespace, "!~monitoring|default")

as match not regexp. The syntax then would be namespace=!~monitoring|default but it makes more sense when having query added via function, and also can be used together with features such as https://github.com/prometheus-community/prom-label-proxy/pull/116 -label-value

See somewhat related https://github.com/prometheus-community/prom-label-proxy/pull/171.

From a quick look, #171 should achieve the same goal. Closing this one as it has merge conflicts anyway.


docs: kick off 1.13.7-dev

Commit Message:
Added version history section for 1.13.7.

you need to update VERSION file as well


Return to same filter after creating transaction

If the user has applied any filter on list transaction page, then after creating transaction, the user should be returned back to the same filter. Right now after creating a transaction you end up on list transactions page with no filter applied.
Implementation details:
This can probably be done by using the referrer header from cloudflare.

req.header.referer contains details of who referred the page. create transaction page can be referred by list transactions page with/without filter or by dashboard. This variable has the full url with the filter parameters as well. When the post request is made, the referer is the page itself. So inorder to propogate the original referer from the get request to the post request, the refered is passed as a hidden field to the post request.


What about downloads with method as post and parameters?

Cry for help

see: https://github.com/rndme/download/pull/117/files
Closed by:

#117
#118

download.js is primarily designed to save client-generated data. Dues to popular demand, I added the GET wrapper, but if you're making server responses, you can use content-disposition.


unit test for post_metadata_update receiver implemented in model instance resources

We need to write a unit test for the post_metadata_update receiver that has been implemented in SWAT Model Instance (#1371) and MODFLOW Model Instance (#1471) resources.

Stale.  These resource types deprecated.


Remove empty, misleading Getting Started page

As noted in #13, the Getting Started page is full of blank links. What
started as a documentation page became our intro video script and then
was never executed. This commit removes the page of dead links.
This does not fix the issue report, which should remain open until the
section contains proper getting started documentation.

Do we want to eliminate the Getting Started page totally, or have a placeholder to let people know what's coming? Otherwise this looks 🍰

I don't think we need to say "coming soon." The navigation would be a real downer with notices all over the place for pages that weren't written yet.


KAFKA-12648: tag uses of StreamsException to add Named to

change each StreamsException so that it has the sub topology name in it
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@ableegoldman I tagged all the cases of StreamsExceptions that I think can be attributed to a Sub topology. I may have missed some and some might not be attributable. Also there might be other error types so I am relying on all exceptions being wrapped in a streamsException before they reach the handler. However I know I have seen one thing in soak where that is not the case. But it is a start

Thanks for the draft -- I haven't looked through it in detail but I think I got the gist of it. One thing I’m wondering is whether we need to track down and update each and every individual source of a StreamsException, which is a long list that will only grow over time, or if it would be easier to just catch and wrap/convert these StreamsExceptions at the highest level in which you can still distinguish the source task. I worry that doing it at the lowest level like this is too prone to things slipping through: even if we gathered every single source exception now, new PRs will add new causes of exception and we can't rely on every single person knowing, and remembering, to incorporate this new feature.
Plus, it'll be somewhat unmanageable to percolate this NamedTopology information everywhere throughout Streams. If we can find a way to keep it isolated to a few top-level classes then I think we'll be in better shape, and much more future proof. Let's discuss it during the sync today but I'm thinking the only classes who probably need to know about any of this are StreamThread, TaskManager, StreamsPartitionAssignor, etc

https://github.com/apache/kafka/pull/10544 I have a draft for the other part as well


BRIDGE-636 Redesign tapping activity to support bilateral tapping task

This is the changes to mPower to support bilateral tapping in the app and moving the functionality for archiving results into AppCore for known result types.

Overall seems like nice changes. Erin should review as well obviously, but they looked good to me.

Just the copyright text comment and the translation comment comment, otherwise awesome.


Using a shared node.js for all running agents

Currently I use 4 "agents" to run in parallel with command line script "-t 4"
Is it possible to use shared node.js modules for every agent or will each agent always run separately?
I'd like to try and create a shared queue for dynamically created test users, but at the moment all four agents are overwriting the shared list of used test accounts with the cy.writeFile command
Since the tests run in parallel, when each test creates an account, they write them to the array of users at the same time, resulting in array containing 4-5 users instead of intended 30.
Is there any way to resolve this?

Not relevant.

Not relevant!


Win32 apps launched via interop lack taskbar icons in Win11

Version
Microsoft Windows [Version 10.0.22000.282]
WSL Version

[X] WSL 2
[X] WSL 1

Repro Steps
Since updating to Windows 11, I've noticed that Win32 applications launched from WSL via interop lack their usual taskbar icons.
(It could be argued that this is a regression in the taskbar, but it's also somehow unique to WSL and I didn't already see it reported here.)
Expected Behavior
When launching an app from cmd.exe:
C:\Users\me\AppData\Local\SumatraPDF>SumatraPDF.exe

Taskbar shows as normal:

Actual Behavior
When launching the same exe in the same directory via WSL interop:
 ~/AppData/Local/SumatraPDF $ ./SumatraPDF.exe

Taskbar shows:

Diagnostic Logs
No response

Confirmed repro in WSL2, and updated description.

@0xabu - no repro for me on 22489 or 22000, I tried notepad.exe, explorer.exe, and vscode.

@benhillis I think it needs to be a standalone exe. Try msinfo32.exe

I'm guessing the others you tried are all special, in that the exe you run from interop isn't directly responsible for the window that appears. (notepad surprises me, but I see the same thing.)

msinfo32.exe also works.

In any case, we're just calling CreateProcess, nothing special.  Smells like a shell bug.

Really? Huh. It's 100% reliable repro for me on 22000.282

Left is msinfo32.exe via interop, right is from cmd.

I filed a feedback bug against taskbar: https://aka.ms/AAerscz

My test machine is on 22000.194, I'll install updates and try again.

No repro on 22000.258 rebooting again ;)

Also no repro on a different user's 22000.258. It may be something about my taskbar, or it may be .282

No repro for me on .282 either, strange.

Maybe your icon cache is corrupted?

I tried another machine I had handy, where I went through a similar set of upgrade steps to Win11 then .282, with no repro before or after the .282 update. So it's something about this specific machine/config. (I also tried clearing the icon cache and restarting explorer, with no changes.)

Since a recent update, apps are now appearing with the new WSL logo:

I've also noticed that unrelated win32 apps launched from WSL are all grouped together on the taskbar as one group, so it seems to be picking up the WSL launcher process and using that for the taskbar icons. The mystery of why it repros only on one device remains, however.


reload previous model in a new training

Hello,
I have a model trained with 500 images (Kitti data and ~200 images of my own). Now I want to add new images to improve this model.
My question is, instead of re-train with all 520 images, can I reload this model and add 20 new images to re-train and improve this model? Maybe something like saver = tf.train.import_meta_graph('my_test_model-1000.meta') in tensorflow. How can I achieve it with minimal modification on train.py or tensorvision code?
Thank you very much!

Go to your logdir, increase max_iterations and use 'tv-continue' to continue training.

Thanks!
Do you mean hypes[solver][max_steps] and the continue_training(logdir) function in tensorvision.train? I'm still a bit confused about 'tv-continue'.

I've successfully done what you're asking - in your RUNS folder there should be the saved model, go to model_files/hypes.json and increase max_iterations. Make sure that you have the new images referenced in your train.txt and val.txt files and that the hypes.json file points to these. Then to continue training use tv-continue --logdir RUNS/your_model_.

Oh, if you haven't already make sure you follow Marvin's instructions for installing tensorvision that are toward the bottom of the intro page.

Thank you very much for your help!

thanks guys I could do what I wanted to do after seeing these answers.


fix/fix path behavior

fix: slots were created in the wrong order
fix: push was pushing at the end
test: fix tests for Path

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

2.0.0
GitHub release

Your semantic-release bot :package::rocket:


[Bug]

When using the official tutorial: autotvm_matmul_x86.py, the tuning trials is set to 10,
and the matrix size is set to N, L, M =<PHONE_NUMBER>43. An error occurred during the execution as follows:
Check failed:(code == RPCCode::kReturn)is false: code =1
Check failed:(code == RPCCode::kReturn)is false: code =kShutdown
How can I solve this problem?
My environment is:
Ubuntu 20.04.6
Thanks!

It seems the RPC server timeout when executing, so try to increase the "session_timeout".

Change the session_timeout of rpc.connect(), right？
In which file rpc.connect() needs to be modified?
Thanks!

It seems the RPC server timeout when executing, so try to increase the "session_timeout".
Change the session_timeout of rpc.connect(), right？
In which file rpc.connect() needs to be modified?
Thanks!

@Johnson9009
Change the session_timeout of rpc.connect(), right？
In which file rpc.connect() needs to be modified?
Thanks!

@Johnson9009 Change the session_timeout of rpc.connect(), right？ In which file rpc.connect() needs to be modified? Thanks!

Yes, just try increase the session_timeout of rpc.connect, if you use RPC tracker, it is the session_timeout of "request".
    tracker = rpc.connect_tracker(tracker_host, tracker_port)
    remote = tracker.request(key, priority=0, session_timeout=60)

Please change the tile of your issue, so others can know what the problem about and help you.

OK,  thanks！I will have a try!

@Johnson9009 Change the session_timeout of rpc.connect(), right？ In which file rpc.connect() needs to be modified? Thanks!

Yes, just try increase the session_timeout of rpc.connect, if you use RPC tracker, it is the session_timeout of "request".
    tracker = rpc.connect_tracker(tracker_host, tracker_port)
    remote = tracker.request(key, priority=0, session_timeout=60)

Actually, i am running with 'LocalRunner', and  also get similar problem.


Groups management section of README

I've been using the package and had some troubles with the groups management section.
I think for it to properly work, one has to make sure during the group claim configuration that the user's groups are emitted as roles claims. Seems to me Azure emit this information under a groups  key by default, however the AuthHandler's authenticate method expects  a roles key in the id_token_claims  for handling permissions (i.e. adding user to the mapped Django group based on group membership).
I've prepared a commit for this clarification and opened a PR, hope you don't mind! Please let me know what do you think :)

Looks like you're right, thank you!


Maven with JChess

See what to do with maven on this project

Maven Documentation
Get started
create the project
mvn archetype:generate -DgroupId=sylhare -DartifactId=jchess -DarchetypeArtifactId=maven-archetype-quickstart -DinteractiveMode=false

Build the project (inside jchess)
mvn package

Run and test the project
java -cp target/jchess-1.0-SNAPSHOT.jar com.github.sylhare.App

This command will clean the project, copy dependencies, and package the project (executing all phases up to package, of course).
mvn clean dependency:copy-dependencies package

Generating the site
mvn site


Make shellscripts runnable from the file manager

It would be nice to be able to run a solver directly from the file manager.

An option "Run script" would be useful.

I see your point: it would be convenient to not have to always start a terminal and then start the script. I know that .desktop files can do this, but I would not like to inject and maintain such files.
Do you have any other way in mind?


Allied status overrides reputation

Helper.cs has the following code:
            // Check for allied
            if (State.IsEmployerAlly) {
                cfgIdx = 7;
            }

It seems a little generous for the employer to not distinguish high-rep allies from ones who have "strayed". I propose it be changed to simply give a +1 column shift:
            // Check for allied
            if (State.IsEmployerAlly) {
                ++cfgIdx;
            }

Allying is a binary flag - you either are allied, or you aren't. The game doesn't treat Ally as the end of the spectrum for reputation points, it's a either/or situation. So doing what you propose would lead to weird results, where you are allied in the UI, but using the friendly rep values.
As such, I'm going to leave as is.


Revert "Fix for ocramius 2.1+ file permission bug"

Reverts eMAGTechLabs/cachebundle#31

Coverage increased (+0.8%) to 97.826% when pulling 39454de8dfc8ff2f6d4eaaafaa71b6d648f4ddb0 on revert-31-add_own_filewriter_to_avoid_ocramius_permission_bug_from_2.1_and_up into 1000e2d4607fd6917094bf68fb1ef8a349951a1e on master.


Speeding up the date selection process

Right now it takes 5 seconds to see a new date and then once you click it you need to do a captcha on the newly reloaded page. We could open two pages, one that refreshes every 5 seconds to get the new dates while the other is "staged" where the user has already completed the captcha. When the reloading page finds a date, we can communicate that to the staged page with the date, enable the date and auto click the date via js and then click the next button. The only issue here is that captchas have a few minutes timeout after completing them so the user would need to redo the captcha every few minutes to gain this advantage.

@stilljake Was wondering what your thoughts are on this?

I've got a prototype where we could share dates that become available in realtime using firebase. The problem I see with that is people would be poaching dates of each other so I need a way to lock it down to only users that want specific dates (with some sort of max date limit). I would need to package it separately to not expose the api keys to prevent misuse. I'm not sure if its worth it but if we get enough people using it and sharing data each time they refresh it could increase the odds substantially.

@stilljake Was wondering what your thoughts are on this?

I put something together already that takes a similar approach. I have one browser instance scraping dates through various proxies to avoid the rate limits, and then once a date is found the other instance is refreshed and selected. I never thought to simply enable the date and click it. That would probably work much faster.

I'm interested to hear more about how this sharing method would work too.

I agree that it's the delay between refreshed taht is a mojor block to getting a date.  This proposal seems like it would work, and I'd be keen to participate.
Another blockage is having to manually click the reCapcha.  This could be (semi?) automated using SendInput to generate a mouse click.  I've done a successful proof of concept, where I manually position the mouse over the reCapcha, and send a click, the reCapcha is passed.
This could be done in responce to an alert.  Obviously, if reCapcha asks you to solve a picture, you are already too late.  In my experience, if you don't click it too much, then it usually goes straight to the green tick.
I'd be ok with a solution that requires manual positioning of the mouse.  But if the mouse could be moved to the right spot too, that'd be cool.
If this can be made to work, automating the click on "Next", and completing the following page could be automated too.

It's also easy enough to fully automate clicking recaptcha and next. For recaptcha you need to switch page context to the recaptcha iframe, and then you can just do an element.click() on the checkbox. Then switch context back to the main page and click() the next button. I've done a proof of context of this too. As long as recaptchas not asking you to solve anything it's fine.


404 & 505 Routes Added

Pages folder created, 404 & 505 pages added to folder. Routes for 404 & 505 pages created and added to server.js.

Cancelled


Consider using SQLite Json extraction trigger events

https://xeiaso.net/blog/sqlite-json-munge-2022-01-04/

Base cache:
CREATE TABLE IF NOT EXISTS jsonfeed_raw  (
  scrape_date  TEXT  PRIMARY KEY  DEFAULT (DATE('now')),
  raw          TEXT  NOT NULL
);

Working the jq into a select statement may get rough, though.

https://antonz.org/sqlite-generated-columns/
Could use generated columns to enable creation at insertion time.


Where is the perception model zoo?

Where is the perception model zoo?  I want to test all the models on some SoC NPU.

You can find perception models in apollo/modules/perception/production/data/perception/.


Fix multiprocessing freeze support

Kivy and multiprocessing.freeze_support() wasn't working correctly on Windows when launching multiple processes from an app packaged with PyInstaller or cx_Freeze.
The  --multiprocessing-fork sys.argv flag was not being restored in the correct order. It is, surprisingly, hard coded as the first argument in multiprocessing's implementation of this feature.
The patch fixes the check for the --multiprocessing-fork argument, as well as restoring it in the correct position in sys.argv. Tested and verified on windows 7 with an app frozen with pyinstaller.

It looks like this branch now has merge conflicts, and in any case pulls in far more commits than it should (perhaps from a bad rebase). Are you able to fix it up ready for merging again?

Indeed, I've made a mess of it. I'll try to wrangle it back down to a single commit.

I went ahead and made a new, much cleaner pull request ( #4427 ) Thanks for being patient with that!


[topgen] Topgen fails if all source clocks are free running

Description
I was trying no generate a new toplevel in which the source clocks would be external and free running, although derived clocks would still be managed.
The topgen however cannot populate the parent_child_clks dict because of  lines 321 and 322 of util/topgen/clocks.py :
if not clk.aon:
    parent_child_clks[name] = [name];
I'm not sure if the field 'aon' is being used correctly or if external free-running source clocks are not allowed in the design as-is since there is an overlap in the meaning of the hjson field 'aon' for free-running clocks and the defined low-power 'aon' always-on clock.
Would just like to understand the rationale.
Thanks

Assigning @andreaskurth to have a first look at this and then either answer this or to (re-)assign accordingly

Thanks for your question, @AWHoppe.
In the current design, the clock manager manages the main, io, and usb clocks. In that sense, they are not free running. The aon field of the clocks.srcs data structure indicates this:
https://github.com/lowRISC/opentitan/blob/7bf43b1d56a998a465308eb683b7d8413bc5f1e6/hw/top_earlgrey/data/top_earlgrey.hjson#L42-L53
(The field name aon may not be the most descriptive, but the comment above defines its meaning.)

Hi @andreaskurth,
Thanks for the feedback. I'm aware of the HJSON field.
I'm trying to achieve a design where the sources besides 'aon' are also free-running.
I.e., the source clocks will be external to the device (waving any security considerations for the moment).
Basically all the source clocks have 'aon: "yes"' .
However these clocks still have derived clocks of their own.
On the topgen script, the lines I mention above do not let the script populate the dict parent_child_clks with free-running sources, so the scripts gives an errors when it tries to parse the derived clocks.
I'm not sure if the overlap in naming with aon was noticed here, since aon does not have derived clocks of its own, or if there's another rationale behind it.

Ah, I think I understand your point about the difference between 'aon' as the abbreviation for the always-on clock, which is linked to the power-on reset, and aon as the field in the above struct, which defines whether the on-chip clock manager manages the clock or not.
I think there is no fundamental limitation stipulating that the clock manager has to manage at least one clock. (Of course, this would require careful security and implementation considerations, but let's put those aside for the sake of this discussion.)
You could try the following: First, decouple the one always-on clock named aon from the externally-managed aon property:
--- a/util/topgen/clocks.py
+++ b/util/topgen/clocks.py
@@ -318,7 +318,7 @@ class Clocks:

         # Define a list for each "family" of clocks
         for name, clk in self.srcs.items():
-            if not clk.aon:
+            if not name == "aon":
                 parent_child_clks[name] = [name];

         for name, clk in self.derived_srcs.items():

Second, change the aon field of the desired clocks in clocks.srcs of hw/top_earlgrey/data/top_earlgrey.hjson to yes.
util/topgen.py should then no longer fail, unless you do that for all clocks. In that case, it currently fails on some width validations. This will require some more code tweaks.
Disclaimer: I don't currently see a fundamental limitation against what you're trying to do, however it is outside the design space we're currently working in. So it is 'off-label' use, and you're essentially on your own. We may be able to help you and will try to do so in the interest of the community, but we cannot commit to supporting you. If you are interested in a closer collaboration, please contact<EMAIL_ADDRESS>

Thanks for the suggestion, we will maybe add it to our version.
I know we are doing some things not in the design space of Earlgrey or the project per se, but I thought some of these issues, although small, could be of general interest to make the project more robust.
If this is desired I can keep posting whatever I find.
E.g., while trying a test-synthesis for chip_earlgrey_asic after generating the source list with fusesoc, I still had to manually add 3 missing components, and change the order of some of them. Also had to do two fixes in an svh and an sv. (we're using Genus)
Some of these issues might also be on our side due to not so robust understanding of the project structure yet, but I can submit whatever we find after understanding the causes a bit more.

Thanks for your feedback.

I know we are doing some things not in the design space of Earlgrey or the project per se, but I thought some of these issues, although small, could be of general interest to make the project more robust.
If this is desired I can keep posting whatever I find.

Yes, I think those issues and questions are interesting and provide useful feedback about what you and others are trying, which may be inside or outside our current design space.

E.g., while trying a test-synthesis for chip_earlgrey_asic after generating the source list with fusesoc, I still had to manually add 3 missing components, and change the order of some of them. Also had to do two fixes in an svh and an sv. (we're using Genus)

Would you mind creating an issue to describe this in more detail? If you have a fix for this issue, I encourage you to create a PR containing the changes. Thanks in advance!


Remove unused function testCase generating warning when compiling test modules.

Some exercises show a warning when compiling the test module:
Defined but not used: `testCase`

This can confuse users and is not considered good practice.
Unused code:

function testCase
import of Assertion

Affected exercises:

atbash-cipher
binary
crypto-square
hexadecimal
largest-series-product
nth-prime
prime-factors
raindrops
roman-numerals
scrabble-score
triangle

Closes #145

I can't think of a reason to keep the code that (a) we are not using and (b) causes a warning.
@towynlin Is there something here that I'm missing?

Now you are getting me curious as to why Travis doesn't fail without this PR, as we do run tests with -Wall -Werror...
Experimenting quickly on my machine, some raw results:

neither runghc -Wall -Werror not runhaskell -Wall -Werror emit errors
ghc -Wall -Werror fails as you would expect: binary_test.hs:11:1: Warning: Defined but not used: ‘testCase
there are cases in which runhaskell will emit errors: if I have an unused binding in Binary.hs, only then we get something: Binary.hs:7:1: Warning: Defined but not used: ‘a’ - perhaps runhaskell does something to ignore unused errors from the main module... or something? I would benefit greatly from an explanation here!

As for this PR:
The pattern we will see is that we'll have two lines mentioning testCase in the file.
$ git grep testCase | cut -d: -f1 | uniq -c | sort -n
      2 atbash-cipher/atbash-cipher_test.hs
      2 binary/binary_test.hs
      2 crypto-square/crypto-square_test.hs
      2 hexadecimal/hexadecimal_test.hs
      2 largest-series-product/largest-series-product_test.hs
      2 nth-prime/nth-prime_test.hs
      2 prime-factors/prime-factors_test.hs
      2 raindrops/raindrops_test.hs
      2 roman-numerals/roman-numerals_test.hs
      2 scrabble-score/scrabble-score_test.hs
      2 triangle/triangle_test.hs
      3 sgf-parsing/sgf-parsing_test.hs
...

(further entries omitted, we are only interested in entries with 2 lines, that means we also don't touch sgf-parsing)
this seems to match the list in this PR, so this all seems good - it is certain that testCase is unused in the files mentioned in this PR (and if it were being used, Travis would have failed).

I imagine the reason why testCase was present was... copypasta, probably.
The only test files that don't have testCase in them...
food-chain/food-chain_test.hs
house/house_test.hs
octal/octal_test.hs
trinary/trinary_test.hs

So probably some copy paste that includes testCase by default, even though it's not used everywhere (since it was used in a majority).

Same result here regarding runhaskell. It shows some warnings but not others, but I don't have an explanation for that.
I got the error because I have a very different setup here. I don't like the use of a global installation of GHC with the packages provided by _test/bootstrap.sh so I'm using stack projects for all exercises n my repository of solutions - even in Travis CI - to have better isolation, replicability and track dependencies more precisely.
I really prefer stack test over _test/check-exercises.

To me personally, it seems valuable to track dependencies per exercise instead of globally. I definitely took that approach for a different project with multiple self-contained problems. I wonder if one day that would be an appropriate thing to do for this repo.
I realize what I said is not really relevant to this PR so maybe suited for a different issue.

Thanks for pinging me @kytrinyx — haven't found the time to review yet. Good discussion here. Keep it up folks. I'm not hearing a conclusive merge/close call from the discussion yet. If anybody is confident one way or the other, please say so!

For what it's worth, I would support merging this.

We where just discussing others issues in the wrong place. Sorry!
About this PR I believe everyone here agrees.

Cool, in that case... :merge:

I see — finally got a minute to glance over the code. Appreciate your git | cut | uniq | sort data mining @petertseng — that makes very clear that this is the right change. Thanks @rbasso!


(Not an issue) Why the "qq" name ?

Type of issue

[ ] Bug report
[ ] Feature request
[ ] Support request
[x] Existential question

Just wondering ...

Ha! Nice issue type improvisation. This predates my involvement in Fine Uploader (project was created in 2010, I took over in 2012, more history in my Medium article on the subject). Andrew Valums, the original creator, was looking for a short and unique namespace for the library. Apparently, qq fit the bill.

going to close this - but am not opposed to continuing the discussion


Signs don't go away after hunk is staged

Description
When I stage a hunk, the corresponding diff sign no longer disappear.
Neovim version
NVIM v0.10.0
Operating system and version
Ubuntu 22.04.4 LTS
Expected behavior
The sign should disappear after staging a hunk.
Actual behavior
The sign is still there after staging the hunk, with the weird "Hunk 1 of 0" message.

Minimal config
for name, url in pairs{
  gitsigns = 'https://github.com/lewis6991/gitsigns.nvim',
  -- ADD OTHER PLUGINS _NECESSARY_ TO REPRODUCE THE ISSUE
} do
  local install_path = vim.fn.fnamemodify('gitsigns_issue/'..name, ':p')
  if vim.fn.isdirectory(install_path) == 0 then
    vim.fn.system { 'git', 'clone', '--depth=1', url, install_path }
  end
  vim.opt.runtimepath:append(install_path)
end

require('gitsigns').setup{
  debug_mode = true, -- You must add this to enable debug messages
}

Steps to reproduce

git clone https://github.com/octocat/Hello-World && cd Hello-World
Add a line of text in README.
Gitsigns stage_hunk on the added line.
Gitsigns preview_hunk on the hunk that is still there.

Gitsigns debug messages
0.79 D dprintf: Deriving GitSignsAdd from Added
0.83 D derive: Deriving GitSignsChange from Changed
0.97 D derive: Deriving GitSignsDelete from Removed
0.99 D derive: Deriving GitSignsChangedelete from GitSignsChange
1.03 D derive: Deriving GitSignsTopdelete from GitSignsDelete
1.06 D derive: Deriving GitSignsUntracked from GitSignsAdd
1.10 D derive: Deriving GitSignsAddNr from GitSignsAdd
1.12 D derive: Deriving GitSignsChangeNr from GitSignsChange
1.18 D derive: Deriving GitSignsDeleteNr from GitSignsDelete
1.20 D derive: Deriving GitSignsChangedeleteNr from GitSignsChangeNr
1.23 D derive: Deriving GitSignsTopdeleteNr from GitSignsDeleteNr
1.27 D derive: Deriving GitSignsUntrackedNr from GitSignsAddNr
1.31 D derive: Deriving GitSignsAddLn from DiffAdd
1.34 D derive: Deriving GitSignsChangeLn from DiffChange
1.38 D derive: Deriving GitSignsChangedeleteLn from GitSignsChangeLn
1.40 D derive: Deriving GitSignsUntrackedLn from GitSignsAddLn
1.42 D derive: Deriving GitSignsStagedAdd from GitSignsAdd
1.45 D derive: Deriving GitSignsStagedChange from GitSignsChange
1.47 D derive: Deriving GitSignsStagedDelete from GitSignsDelete
1.49 D derive: Deriving GitSignsStagedChangedelete from GitSignsChangedelete
1.51 D derive: Deriving GitSignsStagedTopdelete from GitSignsTopdelete
1.54 D derive: Deriving GitSignsStagedAddNr from GitSignsAddNr

You probably don't have termguicolors set.
GitSignsStagedAdd is linked to GitSignsAdd, so they will appear the same.

Hello, after this commit de18f6b749f6129eb9042a2038590872df4c94a9  - I have similar issues
I have termguicolors set.
When I stage a hunk it becomes less bright, but the issue is that when I start navigating hunks (using keymap from README), the plugin goes crazy in different ways, e.g.

plugin navigates to already staged hunks ( why it should not )
plugin starts to show already staged hunks with unstaged HL
hunk counter in the bottom starts showing crazy numbers, like 1 of 10 ( while I only have 2-3 changes)
then it starts showing like 200 of 200 etc
etc

There's no certain pattern, I am just opening random git file with changes, try to stage one hunk and navigate between remaining ones.

@eshepelyuk  Please do not overload specific issues with your own problems. Please raise separate issues with reproducible steps.
It sounds like you do not like the current behaviour of staged hunks, in which case you can disabled it with signs_staged_enable = false.

It sounds like you do not like the current behaviour of staged hunks, in which case you can disabled it with signs_staged_enable = false.

This is what I was looking for. Doesn't signs_staged_enable = false use to be the default value?


Question

Hi,
I was going through the guzzle code, have a doubt regarding the implementation
https://github.com/guzzle/psr7/blob/0c7997709bb6079658c6091ffec6f1d47240375c/src/MessageTrait.php#L99
Doesn't it need to preserve the case sensitivity? See
https://github.com/php-fig/http-message/blob/f3218d1cdf479cd9a65d568f605edbc17c6df82a/src/MessageInterface.php#L118
While header names are case-insensitive, the casing of the header will be preserved by this function, and returned from getHeaders()

and I am seeing https://github.com/guzzle/psr7/blob/0c7997709bb6079658c6091ffec6f1d47240375c/src/MessageTrait.php#L42
public function getHeaders()
{
    return $this->headerLines;
}

I understand it finally is returning the preserved value. But does headerLines in headers is a bit confusing? I wished the variable name is like
public function getHeaders()
{
    return $this->headers;
}

public function getHeaderLines()
{
    return $this->headerLines;
}

Thank you

This repo hasn't been updated to reflect recent changes to psr-7. We hope to get to it soon though.

On Apr 16, 2015, at 8:37 AM, Hari K T<EMAIL_ADDRESS>wrote:
Hi,
I was going through the guzzle code, have a doubt regarding the implementation
https://github.com/guzzle/psr7/blob/0c7997709bb6079658c6091ffec6f1d47240375c/src/MessageTrait.php#L99
Doesn't it need to preserve the case sensitivity? See
https://github.com/php-fig/http-message/blob/f3218d1cdf479cd9a65d568f605edbc17c6df82a/src/MessageInterface.php#L118
While header names are case-insensitive, the casing of the header will be preserved by this function, and returned from getHeaders()
and I am seeing https://github.com/guzzle/psr7/blob/0c7997709bb6079658c6091ffec6f1d47240375c/src/MessageTrait.php#L42
public function getHeaders()
{
return $this->headerLines;
}
I understand it finally is returning the preserved value. But does headerLines in headers is a bit confusing? I wished the variable name is like
public function getHeaders()
{
return $this->headers;
}
public function getHeaderLines()
{
return $this->headerLines;
}
Thank you
—
Reply to this email directly or view it on GitHub.

Thank you @mtdowling for the fast reply.

Everything should now be up to date


can't install on windows-7-64-bit

it‘s always show the error dialog,like this
i have try rubyinstaller-2.0.0-p451-x64,rubyinstaller-2.0.0-p645-x64,rubyinstaller-2.1.7-x64,rubyinstaller-2.2.3-x64

Sorry, I have no idea what that says. =(

it means

This file version and the Windows version you are running is not compatible. Please check your computer's system information to understand x86 (32-bit) or x64 (64-bit) version of the program, and then contact the software publisher

@gcgan that is a corrupt installer. If you're constantly getting that, try to check if your system has been infected by a virus/trojan or similar.
Download the binaries and compare the md5/sha1 signatures from the announcement email or the companion files.

@luislavena Thanks, I wouldn't have gotten that.


enable and disable cache from runtime config

Describe changes
enable_cache as runtime_configuration
Using cache or not should not only be a choice during setup of steps and pipelines. Instead the user should have the option to disable cache at runtime within the runtime configuration.
Test this by replacing everything in the caching example within the name == 'main' section with this code:
    import random
    r1 = random.randint(0, 10)

    # Initialize a pipeline run
    run_1 = mnist_pipeline(
        importer=importer_mnist(),
        normalizer=normalizer(),
        trainer=tf_trainer(config=TrainerConfig(epochs=1,
                                                lr=0.001+r1*0.000001)),   # to invalidate all previous caches
        evaluator=tf_evaluator(),
    )
    # Run the pipeline

    print('\n \n This should write to cache \n \n')
    run_1.run()
    print('\n \n This should not use the cache \n \n')
    run_1.run(enable_cache=False)
    print('\n \n This should use cache \n \n')
    run_1.run()

    print('\n \n This should use cache \n \n')
    run_1.run(enable_cache=True)
    print('\n \n This should use cache \n \n')
    run_1.run()

I have opted not to add enable_cache to the attributes of the RuntimeConfiguration at runtime_configuration.py.
This means the user will not have enable_cache sugggested to them by their IDE. However, this allows for us to have 3 states:
enable_cache not in runtime config -> use default settings
enable_cache=False in runtime config -> disable cache and overwrite default settings
enable_cache=True in runtime config -> enable cache and overwrite default settings
Small bugfix for cahing logs
Additionally this PR contains a small fix regarding the caching logs. Before this fix the logs would claim that the cache of the current pipeline run were used, which does not make sense. I couldn't quickly find the actual source pipeline run of a cached artifact, so I opted to remove the mention of the run_id altogether from the logs. @bcdurak any ideas how we can access the  original pipeline_run_id that produced an artifact? I have a feeling that at the time of querying the execution_info we have already overwritten some metadata.
Pre-requisites
Please ensure you have done the following:

[ ] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table.
[ ] I have added tests to cover my changes.

Types of changes

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

I have implemented the changes to the logging of the original pipeline run id - however I noticed in the process that using a local stack, pipeline.runs takes between 1-2 seconds to run - as this is called for each step, this would increase teh runtime of cached steps from 0.4 to ~ 2 seconds.

Thats probably too big of a performance hit and is due to the inefficient querying in the post execution workflow. It will only get worse with more pipelines. Lets take it out for now would be my vote

+1 to leaving it out. I'm not sure it adds much to have that information logged. Just knowing that it was cached or not is already a good improvement from what we have currently.

@htahir1 @strickvl How would you feel about moving the code to query for last non-cached run for a step into the PipelineView class as method?

@AlexejPenner so you mean we don't use it for what it was originally written for, but in case someone wants to use it, it's there?


New shared steps capability

Currently, this PR just contains the design doc for the shared steps capability
Checklist

[ ] User's Guide has been updated
[X] Developer's Guide has been updated
[ ] API documentation in the Developer's Guide (api.md) has any new or modified class, method and/or functions listed
[X] Documentation has been built locally and changes look as expected
[ ] Testing comment in the PR documents testing used to verify the changes

@xylar I haven't had much time to work on this, but I figured it was better to get a few thoughts out there while our memories were still fresh from our conversation. Feel free to directly push to this branch with your thoughts. We didn't have a chance to chat about which of us would be writing this design doc, so I just assumed it would be a joint effort. Feel free to take over if you like!

@cbegeman, I added quite a bit more here.  Please have a look when you get back to NM.

@xylar I think this looks great! I'm removing the other documentation checkboxes since they are addressed in other PRs. I'll leave this PR unmerged until the other PRs are approved.

@xylar Feel free to merge this PR at the same time as you merge the implementation PRs.


Display notices top right, closes #107

I have tried a few positions, I thought bottom right would work best. But some pages are very empty, and it looks very odd.
The best place for them I found was a little down, against the right. This should make them a lot less annoying.

Here is a preview:

I don' tthink it' closes #107 but it helps. Thanks!!


File comment should be visible

The old UI of the review system allowed to add file level comment (i.e. comment on the file, not a particular line). The new UI does not display those comments inline. It just leave a mark there is a comment on the file, but when you expand it, you cannot see it.
See for example https://codereview.chromium.org/813503003/

I'm not aware of file level comments in the old UI, but there are comments outside the diff range there. They won't show up until you expand context.


open source supply chain WG Meeting March 19, 2024

Date
day-of-week DD MMM yyyy - time EST / time UK
Untracked attendees

Name
Firm
Comment

Meeting notices

FINOS Project leads are responsible for observing the FINOS guidelines for running project meetings. Project maintainers can find additional resources in the FINOS Maintainers Cheatsheet.

All participants in FINOS project meetings are subject to the LF Antitrust Policy, the FINOS Community Code of Conduct and all other FINOS policies.

FINOS meetings involve participation by industry competitors, and it is the intention of FINOS and the Linux Foundation to conduct all of its activities in accordance with applicable antitrust and competition laws. It is therefore extremely important that attendees adhere to meeting agendas, and be aware of, and not participate in, any activities that are prohibited under applicable US state, federal or foreign antitrust and competition laws. Please contact<EMAIL_ADDRESS>with any questions.

FINOS project meetings may be recorded for use solely by the FINOS team for administration purposes. In very limited instances, and with explicit approval, recordings may be made more widely available.

Agenda

[ ] Convene, roll call, welcome new people
[ ] Approve previous meeting minutes
[ ] Review DevOps SIG project board
[ ] Package Management
[ ] AOB, Q&A & Adjourn (5mins)

Decisions Made

[ ] Decision 1
[ ] Decision 2
[ ] ...

Action Items

[ ] Action 1
[ ] Action 2
[ ] ...

Zoom info
Join Zoom Meeting

https://zoom.us/j/94904595244
Meeting ID: 949 0459 5244
Passcode: 545224
Find your local number: https://zoom.us/u/aesEqmNODb

Github Repo: https://github.com/finos/devops-automation/
Project Board: https://github.com/orgs/finos/projects/33
Mailing List: Email<EMAIL_ADDRESS>to subscribe to our mailing list

Karl Moll / FINOS

JM Walker/Fannie Mae

amol shukla/morgan stanley

Alex Ashley / Liatrio

Ragha Vema | Fannie Mae

Eric Chapman - Liatrio


Add seperate dsync tool extracted from dcmp

This is still very much a work in progress. There is a lot of
duplicated code that exists in both dsync and dcmp, so work needs
to be done to extract the common code and make it available to both
tools. The dsync tool is functional, but needs heavy refactoring.

Thanks, @dsikich.  Looks good.  Wow, there is a lot of code in that tool.  Hopefully, we can find a good set of calls to create some common functions.

Looks good to me if you want to merge this.  In that case, we can open another PR once you get into the refactoring.  We can also go ahead and drop the --sync option and related logic from dcmp to simplify that code a bit.


bots guide: Remove need to subscribe bots to streams.

Includes an additional small doc fix.
I chose not to mention the new mention-from-everywhere feature, since I think it's what users naturally expect to happen.

Merged, thanks @derAnfaenger!  We should probably follow through on doing something with @rht's comment as well.

I think the next thing we'll want to do is split this doc into 2: one for using bots and one for writing bots.  It'll require some pretty significant reorganization, since we kinda have content for both audiences mixed together.  Maybe we should plan that out at tomorrow's bots meeting.


Object doesn't support property or method 'forEach' error IE11

IE version: 11.0.9600.18697 windows 7

I think I solved this issue, but I can't build your project to send you a pull request.
Console show me that error:

This changes works for me on IE.
I just make changes in the functions Init() and onReset(), and put this lines before the forEach

init(): var array = [].slice.call( form.querySelectorAll( tag + exclude ) );

onReset(): var array = [].slice.call( ev.target.querySelectorAll( tag ) );

follow this pictures:

Fixed. Gulp should now also compile correctly.

Thanks for fix this!
npm still installing v2.0.1. Please, can you notice me when v2.0.2 is available in npm?

should be available now

Thanks!


S3 encryption client?

I don't see an equivalent of java's AmazonS3EncryptionClient in C++ SDK. Does it exist already somewhere, or is anyone working on it?

It's in security review as we speak.
Sent from my iPhone

On Oct 13, 2016, at 3:01 PM, Yasushi Saito<EMAIL_ADDRESS>wrote:
I don't see an equivalent of java's AmazonS3EncryptionClient in C++ SDK. Does it exist already somewhere, or is anyone working on it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

That's great news. Do you know an ETA?

It's very close, finishing up some small esoteric bits now.

security review is done. I'm cleaning up some last minute stuff and writing the blog post. It should be available in the next couple of days.

out of curiousity, which platforms will you be using this on?

Thanks for the update. I'm looking forward to trying it.
On Thu, Nov 10, 2016 at 4:39 PM, Jonathan M. Henson <
<EMAIL_ADDRESS>wrote:

out of curiousity, which platforms will you be using this on?

Linux (Ubuntu 14 to be exact). We have a large scientific computing cluster
going, and we want to store the data encrypted since some of them are PII.
—

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aws/aws-sdk-cpp/issues/309#issuecomment-259850352,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAhVs5RJlM8zDPWt633HelWrk2XTMqsEks5q87lKgaJpZM4KWbac
.

--
Yasushi Saito

done and pushed. Blog post and more documentation coming soon. It's running through our release build process now, so it our docs should be updated by mid day today.

https://aws.amazon.com/blogs/developer/amazon-s3-encryption-client-is-now-available-for-c/


[BUG] MNIST test accuracies differ drastically with --gpu flag on and off on Apple M3

Describe the bug
MNIST test accuracies differ drastically with --gpu flag on and off.
To Reproduce

create a clean conda environment with conda==24.7.1

conda create -n mnist python==3.12

install the latest mlx (mlx==0.17.0) with pip

pip install mlx

run mnist example with and without --gpu, I find the results differ drastically (and the GPU version is somehow slower).

Expected behavior
I believe the test accuracy should be exactly the same.
Desktop (please complete the following information):

OS Version: MacOS 14.5
Version 0.17.0
Chip: Apple M3

Additional context
Complete conda info:

Sorry, I think I should put it to mlx-examples


How to build wat for preview?

In addition to generating binary wasm, can the build command generate a readable wat interface?
I tried using wasmer to export wat, but it didn't work:
InvalidWasm { error: "Bad version number (at offset 4)" }

Components are a new binary format that Wasmer chose not to support, which is why you're getting this error message.
To run components you can use Wasmtime. To convert a component into a wat file, you can use wasm-tools print.

Thanks, works good


lastpass-cli: 1.3.4 -> 1.3.6

Description of changes
Update to new version, as the 1.3.4 package currently in the repo is broken because of an old certificate

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/why-is-my-lastpass-cli-not-the-newest-version/34974/4

@marsam could you add a backport label to 1.3.6? the ssl certs on 1.3.4 aren't updated. Thanks so much!


Adds debugging info to test_retries_unchanged_does_not_restart_job

Changes proposed in this PR

logging the full job details when an assertion fails

Why are we making these changes?
We've seen this test fail with:
_____________ CookTest.test_retries_unchanged_does_not_restart_job _____________
[gw1] linux -- Python 3.6.3 /opt/python/3.6.3/bin/python3.6
self = <tests.cook.test_basic.CookTest testMethod=test_retries_unchanged_does_not_restart_job>
    def test_retries_unchanged_does_not_restart_job(self):
        uuid, resp = util.submit_job(self.cook_url, command='exit 0', max_retries=1)
        util.wait_for_job(self.cook_url, uuid, 'completed')
        util.retry_jobs(self.cook_url, job=uuid, retries=1)
        job = util.load_job(self.cook_url, uuid)
        self.assertEqual('completed', job['status'])
>       self.assertEqual(1, len(job['instances']))
E       AssertionError: 1 != 2

And we want to track down why.

This should hopefully be fixed by #838

Closing since @pschorf is addressing this in #838


Doesn't work with soucre.pfm

If I add language-pfm package it doesn't work in that scope

My bad. I missed a , to specify file type


QT wallet unusable on windows 10

this is the current view for the wallet. Its impossible to resize or adjust. can anyone help fix this?

I got the problem solved and this is how I got it.
1: download this file below
https://ufile.io/05x7n
2: Go to command and type %appdata% ( you will find roaming file in the search bar)
3: Go to the: C:\UsersUSERNAME\AppData\Roaming\bitconnect
4: Drag and drop the file into the bitconnect folder
5: Run bitconnect as administrator
6: Once into bitconnect go to settings>network>and choose the default options ( Map port using UpnP
7: Close bitconnect
8: Open bitconnect again with run as administrator
9: Once in bitconnect open help>debug window
10: the number of connection should change to 1 or 3.
11: The progress report should start moving up downloading the blocks from the blockchain.
12: enjoy stacking!
Note: this will not work without the file or without doing step number 6. It took me a lot of time to get the wallet to work. If you have any question please don't hesitate to ask. I will try my best to help you. Only users with PC not mac! If the file is not downloadable please send me a message.

my wallet not connect
help me plz
win10
6: Once into bitconnect go to settings>network>and choose the default options ( Map port using UpnP
disable

Yo need to anable it! Do not use other port use the default one!

mars5838 try changing your resolution or play with the properties (Right Click on the program | Properties |Compatibility tab).  Use some of the older resolutions or older settings.
Win 10 is pretty but not as functional as the older OSes.  Make sure to run this program on an older junk computer and not your main personal computer.  Once your coins have matured past 15 days, you have to leave the program open and the PC on all day 24/7 in order to get interest.  See my HOWTO post.

partizan4 see the old threads and follow the directions carefully.  The solution is for Win 7 but should work for most Windows systems.
BITCONNECT QT WALLET OUT OF SYNC
https://github.com/bitconnectcoin/bitconnectcoin/issues/19
0 Active Connections
https://github.com/bitconnectcoin/bitconnectcoin/issues/13
I attached an updated config file there.  Yona5324 has a similar solution.

I am getting a message that says,
" Assertion failed!
Program:C:\Users\User\Desktop\BCC Wallet\bitconnect-window.exe
File: src\kernel.cpp, Line 368
Expression: pindex->pprev||pindex->GetBlock\Hash.........
Anyone ever have this problem with the BCC QT Wallet?

Can anyone tell me how to solve this error?

@BitcoinLover did you ever find a fix? I've been out of town and upon re-launching, I got the exact same error as you. I'm not find a solution elsewhere so far.
Assertion failed!
C:\Users\user\Desktop\bitconnect-window\bitconnect-window.exe
File: src\kernel.cpp, Line 368
Expression: pindex->pprev || pindex->GetBlockHash() == (!fTestNet ? hashGenesisBlock : hashGenesisBlockTestNet)
[Abort] [Retry] [Ignore]

@JPCryptohouse
Reinstall the wallet and delete registered data you can use Revo-uninstaller to do so!
I got the problem solved and this is how I got it.
1: download this file below
https://ufile.io/05x7n
2: Go to command and type %appdata% ( you will find roaming file in the search bar)
3: Go to the: C:\UsersUSERNAME\AppData\Roaming\bitconnect
4: Drag and drop the file into the bitconnect folder
5: Run bitconnect as administrator
6: Once into bitconnect go to settings>network>and choose the default options ( Map port using UpnP
7: Close bitconnect
8: Open bitconnect again with run as administrator
9: Once in bitconnect open help>debug window
10: the number of connection should change to 1 or 3.
11: The progress report should start moving up downloading the blocks from the blockchain.
12: enjoy stacking!
Note: this will not work without the file or without doing step number 6. It took me a lot of time to get the wallet to work. If you have any question please don't hesitate to ask. I will try my best to help you. Only users with PC not mac! If the file is not downloadable please send me a message.

@yona5324 is there another location for the file?

@JPCryptohouse  https://uploadfiles.io/0hqu7

2017년 12월 11일 (월) 21:59, yona5324<EMAIL_ADDRESS>작성:

@JPCryptohouse https://github.com/jpcryptohouse
https://uploadfiles.io/0hqu7
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bitconnectcoin/bitconnectcoin/issues/25#issuecomment-350717410,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AgF6v1KkQhJXqrbN2yYJ9N1PDtzwJbclks5s_SclgaJpZM4QbIzu
.

한국사람??

JPCryptohouse, I just downloaded the BCC QT wallet on a my windows 7 machine and used the old wallet dat file to restore my BCC wallet.  Thank you all for the responses.

Thanks @BitcoinLover I was able to do the same without using any files I am unfamiliar with (like the ones mentioned in other posts above). My solution was to rename the folder, run the .exe again (creating a new folder and files), and copying over the original wallet.dat file. Had to wait for some hours for the blockchain to download and sync, but I'm back up and running.

I got my wallet working on Windows 10 by changing the compatibility setting to older Windows version. (Right click on the starting icon and select compatibility probs. (I also did the config file thing.)

It has issue in updated version. We will fix issues as soon as possible. Please download older version from here: https://github.com/bitconnectcoin/bitconnectcoin/tree/master/setup/bitconnect-window-wallet


fix: PYTHON_VERSION argument

test: https://github.com/leptonai/leptonai/actions/runs/9474396257

test: https://github.com/leptonai/leptonai/actions/runs/9474460235

test: https://github.com/leptonai/leptonai/actions/runs/9474592860

test: https://github.com/leptonai/leptonai/actions/runs/9474648949

test: https://github.com/leptonai/leptonai/actions/runs/9474719461

test: https://github.com/leptonai/leptonai/actions/runs/9474825503

test: https://github.com/leptonai/leptonai/actions/runs/9474969405

test: https://github.com/leptonai/leptonai/actions/runs/9475943933


Multipage table export uses Stream for first page and Lattice for the rest

In this pdf file: cf-rauld_rp.pdf when I select the tables from pages 422 to 444 and go to Preview and Export data only the first page is correctly detected using the Stream technique, and the rest seems to be extracted using the Lattice technique. If I click in Stream to reload the data, all the pages are displayed correctly in the web, but when I export them to CSV, TSV, zip or other format, only the first page is correctly exported.
I'm using the linux tabula.jar in Solus OS with openjdk-8 as the java engine runtime.
Tomás

Hi Tomás / @Geotig : This is a known issue that's fixed in master, but not in the current packaged version. Can you give this JAR a try and see if it fixes the issue for you? https://www.dropbox.com/s/0gkxj5n64s0hy17/tabula.jar?dl=0

Hi Jeremy,
Thanks for this version, it allows me to export the data I need. I assume that you are aware of this, but it only exports correctly if I manually select the Stream method of extraction before exporting.
I'm available if more testing is needed.

@Geotig: Great, glad to hear it.

it only exports correctly if I manually select the Stream method of extraction before exporting.

Right. Tabula makes a "guess" about which of the two extraction methods to use, and then preselects that. This guess is usually right, but often wrong; improving it is not a high priority right now, since it's easy to switch methods and Tabula's goal isn't to be totally automated.


[#3218] Add ChannelPool abstraction and implementations

Motivation:
Many projects need some kind a Channel/Connection pool implementation. While the protocols are different many things can be shared, so we should provide a generic API and implementation.
Modifications:
Add ChannelPool API and implementations.
Result:
Reusable / Generic pool implementation that users can use.

This is an alternative implementation to https://github.com/netty/netty/pull/3607. @trustin @Scottmitch please check. I kind of like it more in terms of API. WDYT ?

I like the implementation, but if possible I'd like to see it more flexible. Basically a dynamic pool (not just a fixed size) so you can use it easily for on-demand scaling of incoming requests..
Something where you can say pool between N and M channels and clean up after condition x (like channel idle for 1 minute or so).
@normanmaurer do you think that's possible?

@daschl maybe... but I think this is just another implementation, no? This API will work for this. I think we can add more implementations later.

@normanmaurer yeah looks like it.. Just wanted to throw it in for consideration to validate :)

Could you elaborate with me on the advantage of this API over the old one? What benefit does a user get?

@trustin I think it is "cleaner" as it the user not need to hold a reference to the ChannelPool to return the Channel at some point. So it is a bit more inline with what we do with our buffers. Once a user not need the Channel anymore the user just calls PooledChannel.release() and that's it. Also we can store all needed stuff without using Attributes.
You see ?

@normanmaurer Then perhaps we don't need PooledChannel but just a simple wrapper that does not implement Channel? such as:
public Handle<ChannelPool, Channel> {
    Channel channel();
    ChannelPool pool();
    void release();
}

I'm not sure if it's a good idea to add a Channel implementation with pool-related operations because it makes downcasting to a transport-specific type.

@trustin Well the whole Idea was that the user not need to have any extra reference to hold to be able to release the Channel to the pool again.
But I agree It really sucks that we need to wrap all things :/ so maybe the Handle is the best we can do. WDYT?

Am 28.04.2015 um 02:21 schrieb Trustin Lee<EMAIL_ADDRESS>@normanmaurer Then perhaps we don't need PooledChannel but just a simple wrapper that does not implement Channel? such as:
public Handle<ChannelPool, Channel> {
Channel channel();
ChannelPool pool();
void release();
}
I'm not sure if it's a good idea to add a Channel implementation with pool-related operations because it makes downcasting to a transport-specific type.
—
Reply to this email directly or view it on GitHub.

@normanmaurer - I would like to explore the first option more.  Keeping the core simple and not requiring an interface that extends Channel.  Seems like this introduces more complexity and opportunities for things to go wrong, and also may be limiting from a practical usage standpoint as @trustin pointed out.  I think it would be good to explore keeping the core "simple", and providing reasonable extension points (factories, abstract methods, etc...) to let data external to the core which may be useful for users of the pool (i.e. the pool the channel was leased from, etc...).  I think you have identified a good practical use case (getting the pool which the channel was leased from) that we are sure we can support through extensions.

@Scottmitch yep doing this now... Let me close this for now


Question regarding the Position State characteristic

(This is not an ADK question as such but maybe you can answer it or at least direct me where to ask it instead)
Window Covering service includes a Position State characteristic that has three possible values:
0 ”Going to the minimum value specified in metadata”
1 ”Going to the maximum value specified in metadata”
2 ”Stopped”
In the context of the window covering service, I read 0 to mean "Going to fully closed position (0%)" and 1 to mean "Going to fully open state (100%)" - this is what "metadata" (i.e. spec) defines as maximum and minimum values for the target and current positions.
However, it's not clear to me what this characteristic should report when the curtain is moving to an intermediate position - e.g., user moved the slider to 60%, Home app sent a write to target position with value 60 and we are moving now, so we are not stopped. However, we are not moving to the 100% position so we can't say we are moving to the minimum or the maximum value. What should an accessory report in that case?

Interesting question.
Perhaps this feature wasn’t initially meant to set the position to an intermediate state. Maybe we need to update this.
Have you studied the specs and the code to see if there is any indication that an intermediate position is a supported order ?

Le 7 oct. 2020 à 19:59, Deomid Ryabkov<EMAIL_ADDRESS>a écrit :

(This is not an ADK question as such but maybe you can answer it or at least direct me where to ask it instead)
Window Covering service includes a Position State characteristic that has three possible values:
0 ”Going to the minimum value specified in metadata”
1 ”Going to the maximum value specified in metadata”
2 ”Stopped”
In the context of the window covering service, I read 0 to mean "Going to fully closed position (0%)" and 1 to mean "Going to fully open state (100%)" - this is what "metadata" (i.e. spec) defines as maximum and minimum values for the target and current positions.
However, it's not clear to me what this characteristic should report when the curtain is moving to an intermediate position - e.g., user moved the slider to 60%, Home app sent a write to target position with value 60 and we are moving now, so we are not stopped. However, we are not moving to the 100% position so we can't say we are moving to the minimum or the maximum value. What should an accessory report in that case?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Afaik PositionState just says in which direction the window (or window covering) is moving. I think only the language in the spec is a bit misleading.
0: closing
1: opening
2: stopped
May be wrong. But that's how I would have interpreted it (and have implemented it some months back). Can double check what behavior im seeing with one of my certified HomeKit windows.

@rojer ^^ @Supereg is correct: if it's moving (assuming it always tries to get to target position), when current position is greater than target position, position state should be set to 0; if current position is less than target position, positions state should be 1. If current position equal to target position, it should be set to 2.

@maximkulkin this makes sense intuitively but do you have a reference, perhaps?

Would it make sense to modify the code to order an intermediate state ? Like 60% as OP said ?

Le 7 oct. 2020 à 20:29, Clovis Durand<EMAIL_ADDRESS>a écrit :

Interesting question.
Perhaps this feature wasn’t initially meant to set the position to an intermediate state. Maybe we need to update this.
Have you studied the specs and the code to see if there is any indication that an intermediate position is a supported order ?

Le 7 oct. 2020 à 19:59, Deomid Ryabkov<EMAIL_ADDRESS>a écrit :

(This is not an ADK question as such but maybe you can answer it or at least direct me where to ask it instead)
Window Covering service includes a Position State characteristic that has three possible values:
0 ”Going to the minimum value specified in metadata”
1 ”Going to the maximum value specified in metadata”
2 ”Stopped”
In the context of the window covering service, I read 0 to mean "Going to fully closed position (0%)" and 1 to mean "Going to fully open state (100%)" - this is what "metadata" (i.e. spec) defines as maximum and minimum values for the target and current positions.
However, it's not clear to me what this characteristic should report when the curtain is moving to an intermediate position - e.g., user moved the slider to 60%, Home app sent a write to target position with value 60 and we are moving now, so we are not stopped. However, we are not moving to the 100% position so we can't say we are moving to the minimum or the maximum value. What should an accessory report in that case?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

i guess this rather calls for modification of the spec. Maxim's comment makes the most sense to me.

@Clovel you're missing the point. Control of position is done through a different characteristic - target position. Position state is a readonly characteristic that, I guess, is used to show visual indication of accessory doing the motion.

It is exactly as Maxim pointed.
Just look at “state closing” (0) as Decreasing and at “state opening” (1) as Increasing.
Also you have “current position” and “target position”, which are giving the exact position in percentage from 0 to 100.
In the case you have described earlier, the accessory will be seen as “Opening” or “Closing” until it actually reaches the target value (the current value becomes equal to the target value - 60%).
It is that simple.

ok, looks like we have a consensus here. thanks everyone, i made changes to my code to reflect this (https://github.com/mongoose-os-apps/shelly-homekit/commit/eb00f25dafe96f37b7f0904c40f34ebbec951990).


Demo W_IO problem/questions

Hi,
I'm using the demo architecture provided as an instructional basis for my own architecture along with the online documentation.
I am a bit confused about the files provided for the demo/W_IO tile. The fabric.csv points at the folllowing files with BEL commands:
./Tile/W_IO/IO_1_bidirectional_frame_config_pass.v
./Tile/W_IO/Config_access.v
./Tile/W_IO/W_IO_switch_matrix.list
In the "demo" directory the file ./Tile/W_IO/Config_access.v does not exist but ./Tile/W_IO/Config_access.v_safe is there - is this file correct?
The file IO_1_bidirectional_frame_config_pass.v does not seem to use configuration bits, so why is a switch matrix required?
Looking to set my own IO tile up so any explanatory help gratefully received!
Regards,
Rob.

The switch matrix is used to select which signals are used to drive the IO output/tristate, and where the IO input is routed to.
The file in git is named Config_access.v, I'm not sure what is adding the _safe postfix, could it be an antivirus tool or some other script that's running?

Thanks for the reply.
In my copy of IO_1_bidirectional_frame_config_pass.v the parameter NoConfigBits = 0 (in fact it's commented out) - hence my confusion.
Perhaps my demo area is old/corrupted - however I can't find it in the Github repository (sorry, I have very little experience using Github!). Can you point me at it?
Also, are there any other example implementations using FABulous? If so where can I find those?
Many thanks,
Rob.


Parse member pointers correctly

Fix
Fix

[x] Link to issue. If there is no issue please describe it using at least the issue template

[x] Description of the fix

[ ] Added unit test

Fix #78.
Introduce a special handling for member pointers.

@rowanG077 have you started on this already? what do you think about this?


improve configuration control

Each run of the HLSPs shoudl produce a table with information such as date, sector, TICA version, production environment, and any parameters that change, such as

reference cadence
changes to keywords in gdPRF_calc (wingFAC and constrastFac).  These are in wcs_build/step1_get_refimg_ctrlpts.py
That table should be compiled in markdown and checked into this git repository.

Addressed in #12


Additional width added by consecutive w3-dropdown-hover within w3-dropdown-content

I have been seeking to use w3.css in Wordpress, with second level pull-right submenus.  In doing so I have discovered a problem which is somehow down to series of w3-dropdown-hovers within a submenu.
If there are consecutive w3-dropdown-hover divs, for each one additional width is added to the submenu.
If there are non-consecutive w3-dropdown-hover divs, there is no problem.
I have found it impossible to locate the css causing the additional width.
You can 'bodge' a fix by declaring a max-width for the containing w3-dropdown-content, but that's hardly elegant.
I know that w3.css wasn't made to support sub-menus, but it's potentially easily implemented.  The question is, where is the extra width in the menus with consecutive w3-dropdown-hover divs coming from??  I can't see anything in w3.css that could cause this, but something must be there.
I've attached a file that demonstrates the problem ... I've had to rename it to .txt, but just download, rename to .html and you can double-click it to display in a browser.
testpage2.txt

Ok, I've found a solution.  It was to do with the inline-block nature of the w3-dropdown-hover, which combined with the float:right was building up a width which was then not needed when the w3-button part of the dropdown was subsequently displayed as a block.  So the answer is simply this css:
		/* Turn off inline-block and float left for .w3-dropdown-hover within dropdown menus, or else width will accumulate */ 		.w3-bar .w3-dropdown-content .w3-dropdown-hover {display: block; float: none;}
But I've also come up with a solution for adding on-click carets which can be turned on and off depending on when needed, and I've tidied up some of the pull right css so that the pull-right menus work regardless of the menu text size.
The full solution is attached below (again in a .txt file - rename to .html to test):
testpage3.txt

This more generic solution for clickable carets works at every level, but adds an additional div.
(The reason one would want clickable carets is when a menu is collapsed on mobile you can then add accordian opening for submenus ... in case you wondered!)
But this just confuses the more generic issue - the css fix for w3-dropdown-hover within submenus needs to be implemented because it doesn't impact any current w3 advice, but it does mean that others can implement second level submenus without the same hassle.
testpage4.txt


lib/fixed-points.nix: add an example for extends

helped me understand how extends works, hopefully it can help others too

Motivation for this change
Just add an example for how extends unfolds, so one can easier
understand how it works.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

I added some introductory text and mentioned, that I use the g and f as defined in the above comments a few paragraphs above.