clam-qt's coincontrol dialog shows incorrect weights

src/qt/coincontroldialog.cpp uses this code to calculate coin weight:
        // weight
        int weight = floorf((GetTime() - out.tx->GetTxTime()) * out.tx->vout[out.i].nValue / BitcoinUnits::factor(BitcoinUnits::BTC) / (double)(1440 * nTargetStakeSpacing));
        if (weight < 0) weight = 0;
        itemOutput->setText(COLUMN_WEIGHT, strPad(QString::number(weight), 5, " "));
        dWeight += weight;

Weight no longer works like that. And why's the GUI doing these kinds of calculations anyway? Shouldn't that be in the back-end?

Frankly, I think listing weight in the GUI at all has very little, if any, utility at this point. As it relates to a metric like coin-days, might make a bit more sense - still little utility that I can imagine.

Do we have anyone who knows their way around the GUI code? I don't, and always feel like I'm guessing when I attempt to change it.
I could probably successfully remove the 'weight' column, but would rather someone with better QT-competence did it.

I removed the weight column in #225. It was a simple enough change.


implement AuthnRequest HTTP-POST binding

must extract the XML signing code borrowed from onelogin/php-saml/src/Saml2/Utils.php and onelogin/php-saml/src/Saml2/Metadata.php found here:
https://github.com/simevo/spid-php3/blob/master/src/Spid/Saml.php#L92-L123
and move it to a helper class

both authn e logout post binding requests now on master


[2.0] ERROR: cleanup: ObjectDB Instances still exist!

I get this message everytime I quit godot editor. When I run editor with -v option I get this also:
Leaked Instance: MarginContainer:9739 - Node Name:
Orphan StringName: MarginContainer
StringName: 1 unclaimed string names at exit.
INFO: dynmem - max: 1903732, no leaks.
**ERROR: STATIC ALLOC: ** MEMORY LEAKS DETECTED **
**ERROR: STATIC ALLOC: 872 bytes of memory in use at exit.
**ERROR: STATIC ALLOC: Following is the list of leaked allocations:
        0x56c7a90 - 48 bytes - core/dictionary.cpp:221, memnew type: DictionaryPrivate
        0x56ca8f0 - 824 bytes - tools/editor/script_editor_debugger.cpp:1378, memnew type: MarginContainer
**ERROR: STATIC ALLOC: End of Report.
mem - max 55808859, pointers 353191, leaks 872.

I'm running latest commit: bd736e5 on Arch Linux.

Fixed by 2e1b4ec.


WIP upgraded ELS Stack from 5.6.16 to 6.8.0 jenkins-ignore

have tested 5 scenarios:

ELK Stack 5.6.16 + Exporter doc type = “_doc”
Test failed. No WLS logs pushed to Elasticsearch repository
ELK Stack 5.6.16 + Exporter doc type = “doc”
Test passed. WLS logs pushed to Elasticsearch repository
ELK Stack 6.8.0 + + Exporter doc type = “_doc”
Test passed. WLS logs pushed to Elasticsearch repository
ELK Stack 6.8.0 + Exporter doc type = “doc”
Test passed. WLS logs pushed to Elasticsearch repository

 ItElasticLogging also passed using ELK Stack 6.8.0 + Exporter doc type = “doc”
     [INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 250.547 s - in oracle.kubernetes.operator.ItElasticLogging

Jenkins job #1980 is kicked off

Jenkins job #1980 is kicked off. and it passed
http://wls-jenkins.us.oracle.com/job/weblogic-kubernetes-operator-javatest/1980/consoleFull


Warn/Allow Failing Tests

Is your feature request related to a problem or a nice-to-have?? Please describe.
I have an integration test that requires credentials to an external system. The credentials are available on CI, but a dev that clones the project wouldn't have access to those credentials, so that test would always fail locally. I don't want devs to get used to failing tests and I do want the test to run on CI. So skip doesn't solve my problem.
Describe the solution you'd like
I thought there would be some feature for allowing test failures. A warning when the test fails, would be appropriate, but it shouldn't fail the build. I think it.allowFail seems like a reasonable way of providing this feature. It should probably take an optional parameter for a warning message, to explain under which circumstances the test is expected to fail.
Describe alternatives you've considered
I could check if the credentials are available and not run the test if they aren't. I have no control over the external system and if it is down, that shouldn't necessarily block you from merging your change.
I also found that there is an npm module that adds this functionality (mocha-helpers), but it doesn't print any warning when a test fails and it is not part of mocha, so it adds an extra dependency that may break in the future.
Additional context
It seems like this feature has been requested a few times (#1480 and #2451), but it was never implemented. Someone thought a retry feature would solve the problem (#1773), but that's only a way to deal with flaky tests, not a way to deal with tests that are expected to fail under certain conditions.

@codiophile you can use this.skip() for this purpose.
If your credentials aren't availabe, then skip the test, it will be reported as pending.

@juergba

I don't want devs to get used to failing tests and I do want the test to run on CI. So skip doesn't solve my problem.

@codiophile well ...
Please explain why it('clever test', function() { if (!credentials) this.skip()}) doesn't solve your problem.

This is a very basic feature for a TDD team.
You write tests first, they all fail.
You provide implementation later... you see how implementation goes fixing the tests (which act as a spec).
skipping has absolutely nothing to do with this feature.


Revert "chore(deps): update ec"

Reverts konflux-ci/build-definitions#1460

Could you give some more context about why the revert is needed?

Could you give some more context about why the revert is needed?

The commit/description is a bit terse as I just used the revert option on GitHub
We had report that the EC started failing with:
rego_type_error: package annotation redeclared

Seems that we pass in the same Rego file twice to the OPA engine and this causes a compile error. Still investigating why has this started to happen and why it wasn't caught by the tests.

/retest


Option to cache agents

as they needn't be single-use.

I guess you're looking for the keepAlive option https://nodejs.org/api/http.html#http_new_agent_options?


Reduced font size of questions in FAQ

Updated backup-azure-security-feature-cloud.md to reduce the size of questions in FAQ section. The current size might be visually confused with a heading, as they are the same size as headings in previous sections.

@Pranshu258 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@dcurwin Would you review the PR? Note that H3 headings should be used before H4 headings, but I'll let you determine whether to approve the update.

We use H3 headers for the questions across all of our FAQ sections. As mentioned above, we can't skip from H2 to H4. Thanks for your suggestion, though.
#please-close


f64 values fast fields are inefficient.

f64 is not well suited for bitpacking and the monotonic mapping is not useful at all for docvalues. (it is useful for the indexed field.)

Closing in favor of https://github.com/quickwit-oss/tantivy/issues/2114


Coraza benchmark

Will and when will Coraza benchmark be back? The repo https://github.com/jptosso/coraza-benchmark seems to be not existent, which is mentioned at https://coraza.io/docs/reference/benchmarks/.
Thanks.

Thank you for your interest, I'm releasing today the new benchmark engine. I will keep you posted!

I have created a few tests files, the results aren't ok I think something is wrong but I will figure it out. The benchmarks between Coraza WAF anc modsecurity con apache shows a difference of 30%~ but current tests are showing way more.
To compile it you will need libmodsecurity installed.

Benchmarking repo is fully tested now. PCRE is going to be removed btw.


nsc-events-android-2-89-create-attach-image-button

This is a potential way that we could go about adding an image but the solution to get this fully implemented is still uncertain. @Robel-003 is still looking into this and determining whether something like this is within reason for MVP2 but I wanted to share the progress that I had made and get any feedback on how something like this could work.
resolves: #89

Here's the video I used to help code this.

I’ll look into this. Thank you for the communication and reference material.
On Thu, Oct 26, 2023 at 9:24 PM taylorpapke @.***>
wrote:

This is a potential way that we could go about adding an image but the
solution to get this fully implemented is still uncertain. @Robel-003
https://github.com/Robel-003 is still looking into this and determining
whether something like this is within reason for MVP2 but I wanted to share
the progress that I had made and get any feedback on how something like
this could work.
resolves: #89
https://github.com/SeattleColleges/nsc-events-android/issues/89
[image: Screenshot 2023-10-26 at 9 07 02 PM]
https://user-images.githubusercontent.com/94101953/278533670-fb88ee13-a986-48a7-90c1-d11ff16b1e32.png
[image: Screenshot 2023-10-26 at 9 07 17 PM]
https://user-images.githubusercontent.com/94101953/278533679-19b69111-a878-419d-a5f5-6972281ab316.png
Here's the video
https://www.youtube.com/watch?v=kZKL_JvPDG0&list=RDCMUCv75sKQFFIenWHrprnrR9aA&index=3
I used to help code this.
You can view, comment on, or merge this pull request online at:
https://github.com/SeattleColleges/nsc-events-android/pull/90
Commit Summary

cc81ca9
https://github.com/SeattleColleges/nsc-events-android/pull/90/commits/cc81ca98587c27e63e5d3bd3ab6935bfc8b940c3
set up cards on home page that will link to product details page
3c2a3c3
https://github.com/SeattleColleges/nsc-events-android/pull/90/commits/3c2a3c365eedf753471ae5771e6334b102e7cfae
added functionality to display details for an event from home page on event
details page
45589a4
https://github.com/SeattleColleges/nsc-events-android/pull/90/commits/45589a4ed65238a56945385e9d614ee04e4d4ef9
cleaned up code
e968af4
https://github.com/SeattleColleges/nsc-events-android/pull/90/commits/e968af46a3ca77148da761b8124ecbbee0ceac25
added a couple tests to EventDetail and HomePage
6f1123a
https://github.com/SeattleColleges/nsc-events-android/pull/90/commits/6f1123a8aa6871610a82d25139bb4fed7c930f7e
added event end time, location and cover photos to event details page
61c1d41
https://github.com/SeattleColleges/nsc-events-android/pull/90/commits/61c1d41ef97309c8eccd2ce557de9450508b55a5
change eventName to eventTitle in addEvent.kt to match schema
e45fcdf
https://github.com/SeattleColleges/nsc-events-android/pull/90/commits/e45fcdf9ff760042794d378062717a91400cada4
Merge branch 'main' into feature-event-details-view-creation-01
e5b8a0c
https://github.com/SeattleColleges/nsc-events-android/pull/90/commits/e5b8a0cf616f10443f94de42ad84b59fd9c7d0a1
added edit, delete and attend button to event details page
5e22aa9
https://github.com/SeattleColleges/nsc-events-android/pull/90/commits/5e22aa90f6f5966ca8d8257b07dcecf5e62f4cb2
added a few simple UI tests
9566c9d
https://github.com/SeattleColleges/nsc-events-android/pull/90/commits/9566c9ddc3c9decf5f964e89044885b6406abf1e
added dependencies to gradle and photo picker w/ limited functionality

File Changes
(15 files
https://github.com/SeattleColleges/nsc-events-android/pull/90/files)

M app/build.gradle
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-51a0b488f963eb0be6c6599bf5df497313877cf5bdff3950807373912ac1cdc9
(3)
A
app/src/androidTest/java/com/example/nsc_events/EventDetailTest.kt
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-452681c820d649d1eae429c203296842eb6c85d845ae0cc03829c668654244b5
(76)
A app/src/androidTest/java/com/example/nsc_events/HomePageTest.kt
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-42a192255dbc8e7965c27b27a2db1ef4f42cbb4a234ab6029838d24f05ac0364
(48)
M app/src/main/java/com/example/nsc_events/Routes.kt
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-0b03b40b11d01ef2550449bc1be39405e0747aa8fca0b73d0e4da0c222f7e5a8
(1)
A app/src/main/java/com/example/nsc_events/data/Datasource.kt
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-bcf28d557f5f52ad63f3cd413a18d3fbc3bd577e620cbdd842e9f9b1500165ac
(15)
D app/src/main/java/com/example/nsc_events/data/Event.kt
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-676a2c92d3cce364b823a35289a5a99df0c6010d1a24ffe05384cfa91b569e0e
(11)
A app/src/main/java/com/example/nsc_events/model/Event.kt
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-cb3ef7851a8dbdb98ffa133f02776dfcb394fc891072ca6510d7be020c937f09
(13)
M app/src/main/java/com/example/nsc_events/screen/AddEvent.kt
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-69ce06e5549b1223615de3ea4736a8525db58987575528dc3e3438e273bc4690
(66)
A app/src/main/java/com/example/nsc_events/screen/EventDetail.kt
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-3d12dd6da49b71fbde00b30a3e5b9decefb44daac182313a2c32c8747083806d
(196)
M app/src/main/java/com/example/nsc_events/screen/HomePage.kt
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-b08d837fcc72ac5d9a847c77115b4cb56e57f78ed24eb4c66e1ccac22a8aa06f
(78)
M app/src/main/java/com/example/nsc_events/screen/ScreenMain.kt
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-771dbf6b44dd66ba662fedf4fd6212fc48b5ea3a8a98d6463f090533b949d4cb
(9)
A app/src/main/res/drawable/happy_thanksgiving.jpg
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-d57dba4bac3bbeea644816e14880f750b889dd89150f213b14d6baf847328886
(0)
A app/src/main/res/drawable/newyears_png2.jpg
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-72a43187fb982d0feced038de270bf0654ad366cbecd7b2f2878ba2abc5e1e91
(0)
A app/src/main/res/drawable/speech_2759550__480_1_png.webp
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-7ad7a3cb84e3a81dc7848ed0eca0df20eaef712fce36a957c48e376abeb83151
(0)
M app/src/main/res/values/strings.xml
https://github.com/SeattleColleges/nsc-events-android/pull/90/files#diff-5e01f7d37a66e4ca03deefc205d8e7008661cdd0284a05aaba1858e6b7bf9103
(2)

Patch Links:

https://github.com/SeattleColleges/nsc-events-android/pull/90.patch
https://github.com/SeattleColleges/nsc-events-android/pull/90.diff

—
Reply to this email directly, view it on GitHub
https://github.com/SeattleColleges/nsc-events-android/pull/90, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AN7WAKXCERUPNVGB5DZGWETYBMZRLAVCNFSM6AAAAAA6SGRMWGVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE3DINZUGE2DSNI
.
You are receiving this because your review was requested.Message ID:
@.***>

After failing CI, I realized that the video and Glide library may be a little outdated. In recent years, it looks like Coil has gained more popularity and I was able to find some tutorials on its implementation.


checkbox can not be selected if it's in a element with @click listener?

Vue.js version
2.1.6
Reproduction Link

http://codepen.io/qbaty/pen/NboZoz?editors=1111
Steps to reproduce
my html is div > input[type=checkbox]
when div with v-on:click, I found that the input element can not be selected.
What is Expected?
when click this input checkbox, checkbox should be selected or canceled.
div click event will trigger.
What is actually happening?
I have not read the source code，But I think div capture this click event and should not stop this event immediate, it kinda like use “event.stopImmediatePropagation”  stop the event progagation?

v-model ?!

@Marshare I do not think you understood my problem。

@qbaty  @click.self="add"

@qbaty Thanks for rteporting this, looks like a bug indeed!
I took the freedom to correct the Vue version in your OP, as you use 2.1.6, noch 2.0.3

@LinusBorg Okey first I met this bug when I am using vue 2.0.3,and I try to update vue to the lastest version. this bug still exsit, so ... version should be 2.x

Seems fixing this bug created a new bug for :checked="" binding when using in combination with vuex states for maintaining checkbox states.
<template>

			<input
				type="checkbox"
				:name="filter.key"
				:value="option.value"
				:checked="inArray(option.value, filter.active)"
				@change="updateFilter">

</template>

In this code example is :checked set to true or false whenever it's in the active filter array.
It seems when this component is updated (via vuex) the :checked stated doesn't get's updated like in 2.1.6. In my application is the state managed almost totally by Vuex.
Vue.js version
2.1.7+
Reproduction Link
http://vroom.automotivated.nl/
Howto reproduce
When selecting several checkboxes (like brands) and use your browser back button to fallback to previous states. (it's managed by the popstate event)
What is expected
The checkbox will go on or off depending on the previous action.
What is actually happening?
Everything get's correctly updated (value wise), except the checkbox isn't honored to update (checked true/false). It keeps in a previous state. When selecting multiple after eachother and navigating back and forward is looks like is off a beat and missing the last changed checkbox.
Code that goes with it:
https://github.com/Automotivated/vroom/blob/master/src/views/elements/filters/Multiple.vue
While reading the docs in Vuex, it seems to be best practise to do it like I did.
When did it work
It worked like a charm in 2.1.6. Seems really that fixing this bug triggered this bug to be created.

@FKobus Please open a new issue instead of adding to the bottom of closed ones. Thanks!


feat: add EVM support for Multicall3

What I did
Adds support for Multicall3 on a bunch of different chains directly to ape-ethereum
fixes: APE-391
fixes: APE-394
How I did it
Created ape_ethereum.multicall module
How to verify it

Checklist
based on: #1160

[x] All changes are completed
- [ ] New test cases have been added
[x] Documentation has been updated

[ ] Needs tests

Best test I could find (but only works with Foundry or Hardhat plugins):
def test_multicall(multicall, chain, accounts):
    assert chain.blocks.head.number == 0

    call = multicall.Call()
    call.add(call.contract.aggregate, [])
    call.add(call.contract.aggregate, [])
    call.add(call.contract.aggregate, [])
    call.add(call.contract.aggregate, [])
    (a, _), (b, _), (c, _), (d, _) = call()
    assert a == b == c == d == 0
    assert chain.blocks.head.number == 0

    txn = multicall.Transaction()
    txn.add(txn.contract.aggregate, [])
    txn.add(txn.contract.aggregate, [])
    txn.add(txn.contract.aggregate, [])
    txn.add(txn.contract.aggregate, [])
    txn(sender=accounts[0])
    assert chain.blocks.head.number == 1

def test_multicall_again(multicall, chain):
    # Test isolation
    assert chain.blocks.head.number == 0

    call = multicall.Call()
    call.add(call.contract.aggregate, [])
    call.add(call.contract.aggregate, [])
    call.add(call.contract.aggregate, [])
    call.add(call.contract.aggregate, [])
    (a, _), (b, _), (c, _), (d, _) = call()
    assert a == b == c == d == 0

Can we rebase this one now that the the PRs it depended on are merged in?

Can we rebase this one now that the the PRs it depended on are merged in?

Done!

Note: Can't test this feature because eth-tester provider doesn't not support debug_setCode

Edit: not a breaking change, no longer requires an un-merged PR

I found a bug while testing this. For some reason, when using Multicall on a call with no arguments, I get the ArgumentsLengthError(). The same call works without the multicall. Doing some investigating, I found that receives this as the args:
([],)

which triggers as 1 arg.

OK, no running into this:
eth_abi.exceptions.EncodingTypeError: Value `603786816` of type <class 'int'> cannot be encoded by ByteStringEncoder

@unparalleled-js :eyes: 2bc4c1c

OK, now running into this:
eth_abi.exceptions.EncodingTypeError: Value `603786816` of type <class 'int'> cannot be encoded by ByteStringEncoder

This is the integer encoding of the 4bytes calldata (just the signature, no additional args)
Investigating...

64c7424 doesn't resolve all the mypy issues, but some of them (and it's cleaner to understand)

Dropped the gnosis safe call refactor because it doesn't support eth_getStorageAt

OK, now running into this:
eth_abi.exceptions.EncodingTypeError: Value `603786816` of type <class 'int'> cannot be encoded by ByteStringEncoder

This is the integer encoding of the 4bytes calldata (just the signature, no additional args)
Investigating...

this happens because self.conversion_manager.convert(v, tuple) is converting the HexBytes arg into an integer from within ContractCallHandler.__call__, when it hits this line:
https://github.com/ApeWorX/ape/blob/e0f7b57de50b0dd155e071e2ccffd454f28f661a/src/ape/contracts/base.py#L188
inside of the multicall.Call.__call__ method being triggered

OK, now running into this:
eth_abi.exceptions.EncodingTypeError: Value `603786816` of type <class 'int'> cannot be encoded by ByteStringEncoder

This is the integer encoding of the 4bytes calldata (just the signature, no additional args)
Investigating...

this happens because self.conversion_manager.convert(v, tuple) is converting the HexBytes arg into an integer from within ContractCallHandler.__call__, when it hits this line:
https://github.com/ApeWorX/ape/blob/e0f7b57de50b0dd155e071e2ccffd454f28f661a/src/ape/contracts/base.py#L188
inside of the multicall.Call.__call__ method being triggered
EDIT: I believe #1247 added this bug when the HexIntConverter was created

This bug is the only blocker to this PR being ready for QA and merge

EDIT: I believe #1247 added this bug when the HexIntConverter was created

I fixed the issues on this PR: https://github.com/ApeWorX/ape/pull/1230 (last few commits), as it was affecting that as well!
So after we take in those changes, we should be good with this part.

EDIT: I believe #1247 added this bug when the HexIntConverter was created

I fixed the issues on this PR: #1230 (last few commits), as it was affecting that as well! So after we take in those changes, we should be good with this part.

This looks good!

#1230 not working entirely like I expected:
EncodingTypeError: Value `{'target': '0xcA11bde05977b3631167028862bE2a173976CA11', 'allowFailure': Fal...` of type <class 'dict'> cannot be encoded by TupleEncoder: must be list-like object such as array or tuple

#1230 not working entirely like I expected:

Give it a spin now! The problem was happening only for list-of-struct inputs.

Sick it works w/ #1230
In [1]: from ape_ethereum import multicall
In [2]: call = multicall.Call()
In [3]: call.add(call.contract.getLastBlockHash)
In [4]: call.add(call.contract.aggregate, [])
In [5]: call.add(call.contract.getBasefee)
In [7]: list(call())
Out[7]:
[HexBytes('0x70cd88dbc792de30c4df33934feeaa8e0a203d9a5eee7eed02a2edb06fc0fac4'),
 aggregate_return(blockNumber=16522283, returnData=[]),
<PHONE_NUMBER>2]


ReDoS vulnerability exposed via title → clipboardy → execa → cross-spawn dependency chain

Describe the bug cross-spawn <5.1.0 contains a high level ReDoS vulnerability: https://security.snyk.io/vuln/SNYK-JS-CROSSSPAWN-8303230
This dependency is imported via this chain<EMAIL_ADDRESS>→<EMAIL_ADDRESS>→<EMAIL_ADDRESS>→<EMAIL_ADDRESS>It is being consumed at a single place: https://github.com/shuding/nextra/blob/ab9f5fbcf715bfa3b171757321b5cdc06d813756/packages/nextra/src/server/utils.ts#L9
I have opened a ticket on vercel/title, but I'm not sure if it will be fixed soon as the package was last updated 2 years ago.
Alternatively, we can perhaps consume another library that can perform title-case transformation?
To Reproduce Steps to reproduce the behavior:
Install nextra and run a vulnerability scan. We use Snyk in our repo and it reported this vulnerability: https://security.snyk.io/vuln/SNYK-JS-CROSSSPAWN-8303230
Expected behavior nextra should not be using cross-spawn < 7.1.0 due to the ReDoS vulnerability.
Screenshots

Try to ping @leerob on twitter, he is pretty active there


Include Markdown Files

To improve writing KITs, it is often necessary to have the same content on more than one site (e.g., license information). Therefore, it would be great to have the opportunity to include a markdown file inside another markdown file to prevent duplicate content.
This feature is already discussed in the upstream project:
https://github.com/facebook/docusaurus/issues/599
Further, plugins exist that enable it:
https://github.com/dotansimha/remark-import-partial

This is possible now!
Step-by-step guide:

Change the file extensions from .md to .mdx, like main.mdx and includeme.mdx
At the beginning of main.mdx, use: import Includeme from './includeme.mdx'
At the specific position in main.mdx, use <Includeme components={props.components} />

Idea based on: https://stackoverflow.com/a/70702778/2056125
We will test it for the Traceability KIT in the upcoming release.


[Bug] Subscriptions going from one connection to another

What did I do
I had a few connections. This is regarding the cases where I'd quickly click on different connections one after the other.
What happened
When moving from one connection to another, one of the previous connection's subscription is moved to the new connection.
Which causes a deadlock.
Can't remove the subscription because it required the connection to be online.
Can't bring the connection online because that connection can't subscribe to that newly transferred subscription.
Expected
No such transfer should happen when quickly going through multiple connections.
Environment

OS: macOS Ventura 13.2.1
MQTTX version, e.g. v1.9.1

More detail
The topic node/cQ2zcUQjiQkGpLux97wUMn/otaurl does not belong in the connection: cQ2zcUQjiQkGpLux97wUMn.

And the connection cQ2zcUQjiQkGpLux97wUMn is missing the topic node/cQ2zcUQjiQkGpLux97wUMn/otaurl.

Can't remove the topic because that required the connection online

Can't bring the connection online because it is not allowed to subscribe to that topic
[2023-03-15 18:36:39] [INFO] MQTTX client with ID 32809650-3835-4afa-a1c2-e04c54210fcd assigned
[2023-03-15 18:36:39] [INFO] Connect client cQ2zcUQjiQkGpLux97wUMn, MQTT/SSL connection: mqtts://a3jf85c407as92-ats.iot.us-east-1.amazonaws.com:8883
[2023-03-15 18:36:42] [INFO] cQ2zcUQjiQkGpLux97wUMn connect success, MQTT.js onConnect trigger
[2023-03-15 18:36:42] [INFO] Topic: node/e6LmmRJ7692rRNuWWD5Qq5/otaurl is resubscribing
[2023-03-15 18:36:42] [ERROR] Topic: subscribe error, Error: Connection closed
[2023-03-15 18:36:42] [INFO] cQ2zcUQjiQkGpLux97wUMn connect close, MQTT.js onClose trigger

Hi, sorry about that. Based on your description, I can not reproduce this issue. Could you please provide more details? Is the step only the quick click on different connections one after the other?

If there are no issues, I will close this issue. If any issues still exist, please reopen it.


[v0.9] Fix data layout for x86_64-bootloader target

Fixes build on latest Rust nightly.
See https://github.com/rust-lang/rust/pull/120062

Released as v0.9.24


Incorrect Dockerfile line

Problem encountered on https://dotnet.microsoft.com/learn/web/aspnet-microservice-tutorial/docker-file
Operating System: windows
Provide details about the problem you are experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
The line:
COPY --from=publish /app .
Should be:
COPY --from=build /app .
We need to reference the previous build stage. Its named build, not publish. Using publish results into an error. 'invalid from flag value publish: pull access denied for publish,...'

@Pilchie is this tutorial (and the related docker files) owned by the ASP.NET team If not, @MichaelSimons would you be the best contact for help with these docker containers in the tutorials?

No, you want to copy the publish output into your runtime image. The publish layer comes from build and then runs the publish command. Is your Dockerfile missing the publish layer?
FROM build AS publish
RUN dotnet publish myMicroservice.csproj -c Release -o /app

Here is a simpler version of the Dockerfile that I prefer. Give it a try and see if it works better.
FROM mcr.microsoft.com/dotnet/core/sdk:2.2 AS build
WORKDIR /src
COPY myMicroservice.csproj .
RUN dotnet restore
COPY . .
RUN dotnet publish -c Release -o /app

FROM mcr.microsoft.com/dotnet/core/aspnet:2.2
WORKDIR /app
COPY --from=build /app .
ENTRYPOINT ["dotnet", "myMicroservice.dll"]

2.5 weeks no response, closing. Feel free to reopen if there is more info available. Thanks!


Add DOI Badge to the README

As it says on the tin
The DOI tracking has been enabled for a while, this just adds it to the README/Homepage

Thanks!


Resolving "in" and "out" ambiguity

The current API definition uses "in" and "out" to define flow direction (as in "protocolIn" and "protocolOut"), but these are relative terms and hence ambiguous. The UE would view "in" as the downlink and "out" as the uplink, whereas the AS would view "in" as the uplink and "out" as the downlink. And who know what a 3rd party API caller would think.
As UE and AS are explicitly identified in the API, I would suggest to define flow direction in terms of these actors to avoid any possibility of confusion. So "protocolIn" would become "protocolAStoUE" and "protocolOut" would become "protocolUEtoAS". And if the eminently sensible proposal in issue #32 to replace "protocolIn" and "protocolOut" with "protocol" and "direction" parameters is adopted, then the options for "direction" would be "UEtoAS", "AStoUE" and "BOTH".

I fully agree that we need clear parameter names and values for them which are understandable. Closing this issue to focus the discussion in #32.


Fix button hover

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Build script

Target platforms - Ubuntu for prod, Mac for dev

Build script should generate self-signed SSL cert


Minimap not updated when using left/right arrow keys

I can't reproduce this. Show me please.

The async update we added for the location URL probably fixed this.


How to get a different Gui?

When i run Visualization.py via IDLE i get the error: "No module named ''numpy".
due to this error i cannot start the visualization program. I installed the module as said in the code. I don't know what to do now. Can Someone help me please?
The error:

I haven't personally used IDLE but there may be some other way of installing libraries than normal? You made sure to pip install numpy right?

Yes i made sure to pip install numpy. Also i already reinstalled it. To run it i used IDLE and spyder.
On what are you running it? maby it works when i run it like you.

@Jameshond700 I just use a standard installation from the python website on windows and pip install all dependencies.

Thank's i got it to work!, Also i have another question: In some videos i saw a different GUI with more modes and you could edit the colors of the modes. my question is: how did they get the gui. I would like to use it too it
A image of the gui that i would like to have:

@Jameshond700 Yeah I believe that is @not-matt's fork if I'm not mistaken

Yes, i think so too but i cant get it to work. I don't know if you have any knowledge about this but could you help me?

Hi, that screenshot is from my project Systematic Leds. It’s no longer being developed but give it a try by all means! Installation Tutorial:https://www.youtube.com/watch?v=W4jaAgjfvG8Repo:https://github.com/not-matt/Systematic-LEDs I’m working on LedFx now, so if you want to keep up to date on the newest work then give this a go:https://github.com/ahodges9/LedFx Matthew From: Jameshond700Sent: 30 June 2020 20:26To: scottlawsonbc/audio-reactive-led-stripCc: SubscribedSubject: Re: [scottlawsonbc/audio-reactive-led-strip] Error when running visualization.py (#276) Thank's i got it to work!, Also i have another question: In some videos i saw a different GUI with more modes and you could edit the colors of the modes. my question is: how did they get the gui. I would like to use it too itA image of the gui that i would like to have:—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.

@not-matt I Already saw the video. But when you look at the config.py in the video it is different with the config file that is provided. It looks like it is automated. So i tried to just run the Main.py. But then i get the error; OSError: [Errno -9996] Invalid device. So i do not what to do next.


Create a method to consistently draw boxes in OrbitGl

In this PR we are just moving a method to draw boxes from SchedulerTrack to TimelineInfoInterface.
This will help to solve the glitches when drawing boxes in the Capture Window (for example, see
https://github.com/google/orbit/pull/4127 for ThreadTracks). The implementation of this method
will extend the box to the pixel borders as it's doing in the SchedulerTrack.
This method is needed for ThreadStates (https://github.com/google/orbit/pull/4274), to draw
consistently after applying the pixel rendering optimization.
Test: Load a capture, everything looks as expected.

I'm fine with this, but I want to point out that I think a better design would be to not have GetBoxPosXAndWidthFromTicks be part of the TimelineInfoInterface. You should strife to keep interfaces minimal, and the implementation of GetBoxPosXAndWidthFromTicks only relies on public functionality already available through the interface, so you could also make it a free function that receives a TimelineInfoInterface as an additional parameter. This way, you also wouldn't need the implementation in the Mock.

As discussed offline, I fully agree. TimelineInfoUtils makes a lot of sense for a place to have this function. As it's only one function, and you've already approved, I will merge it as it is. Thanks for the input anyway!


(Recreated) The beginning of data sets

The beginning of data sets: allow WBValue's to be stored in DataFrame's. Distinguish plain WBValue's, which can be serialized to JSON on their own, from Media objects, which need to be bound to a particular Run before they can be serialized. In either case, the resulting JSON structures should let us share data across Runs.
Since the original version of this pull request, I've removed the run ID's etc. stored in media -- we just assume they're original from the "current" run, whatever that is -- and restored the old behaviour of uploading sets of images as a JPEG of thumbnails without duplicating them as PNGs. Single images created as raw pixels will be stored as PNGs but not JPEGs.

I think it'd also be fantastic to test what happens when you don't have certain deps (the ones that we don't require but print warnings for when you try to use a specific feature). We can add this to Jeff's integration suite at some point, in the mean time we can run through it manually.

I can fix the edgeml test.

I'm still a little concerned that users which log an individual image using wandb.log will consume ~2x disk space with this release.  We might want to consider always just making a sprite for wandb.data_types logged with wandb.log.  I changed the tensorboard logger to always log a list of images even when there's only one.

We got rid of double-logging images.
I think pngs are gonna be a fact of life for logging datasets. Lossless compression is important for that case.
Turning images into lists for Tensorboard seems like a good idea. Thanks.

Tests are passing, I'm good to merge and release tomorrow.  @raubitsj can you run the integration suite?

@adrianbg I saw a ~50% size increase between my wandb sync on tensorboard with single image vs array of one image so I assumed we were double logging.  If the effect is just png vs jpg I'm fine taking that hit for lossless on a single image with wandb.log.

Regression running: (all 26 tests should pass)
https://app.wandb.ai/jeffr/regression/groups/20190524-0.8.1-feature_data-frame-media-33cd040-67f8uf

Thanks Jeff!
@adrianbg Can you confirm we're not double-logging in the tensorboard case? I think once that's done, we're good to merge.

Here is the regression kicked off for the 0.8.1 release (for reference):
https://app.wandb.ai/jeffr/regression/groups/20190524-0.8.1-release-unknown-y8y9fo
(command line: ./regression.py --cli_release 0.8.1 --spec ::~broken .)

Yeah, it'll either do a JPEG for an array of wandb.Images or a PNG for an
individual one. Either way there's no double logging.
On Fri, May 24, 2019 at 9:27 AM raubitsj<EMAIL_ADDRESS>wrote:

Here is the regression kicked off for the 0.8.1 release (for reference):
https://app.wandb.ai/jeffr/regression/groups/20190524-0.8.1-release-unknown-y8y9fo
(command line: ./regression.py --cli_release 0.8.1 --spec ::~broken .)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wandb/client/pull/351?email_source=notifications&email_token=AAAQJP2EEE6RLI574DMOZO3PXAJNLA5CNFSM4HOYVT42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWF4CEI#issuecomment-495698193,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAQJP2DDQVATR2ZABY3VKDPXAJNLANCNFSM4HOYVT4Q
.


ERROR ChatConnector: receive - invalid token. Check bots app ID & Password

After migration my bot from V1 to V3  i get the Error  "invalid token. Check bots app ID & Password"
I already create new Application ID and Password and register a new bot on  the Bot Framwork webpage.
It works with the bot framework channel emulator V3.x
I used then new connection settings for V3
var connector = new builder.ChatConnector({
appId: "appid",
appPassword:  "password"
});
var bot = new builder.UniversalBot(connector);
I re-generated the password but I get the same error.

Did you notice that there are now 3 settings in web.config?

    <add key="BotId" value="xxxx" />
    <add key="MicrosoftAppID" value="yyyyy" />
    <add key="MicrosoftAppPassword" value="zzzzzzz" />

i don't have a web.config i used the node.js api and start my bot locally.

I found the problem by myself , there was a timing problem into the jwt  and  chatconnector lib.

Thanks for noting this. We have some fixes in the works that will address this.

#742

Hi still got this problem :/

@dandriscoll @Stevenic Please note that is issue is also happening on the CallConnector

Hi @ejadib, we've found the bug and will be fixing it soon. Thanks for raising this.

Has this been resolved?

@dandriscoll I have this problem in the last version of the BotConnector framework:
// Restify embedded server
    var restify = require('restify');
    // Microsoft Bot Framework
    var BotBuilder = require('botbuilder');

    var Config = require('../../config/luis');
    // Dialog Intent and Entity objects
    var Intent = require('../../dialog/intent');
    var IntentFactory = require('../../dialog/intentfactory');
    var Entity = require('../../dialog/entity');

    var inMemoryStorage = new BotBuilder.MemoryBotStorage();
    var connector = new BotBuilder.ChatConnector({
        appId: Config[ Config['env'] ].MS_appId,
        appPassword: Config[ Config['env'] ].MS_appPass
    });
    var bot = new BotBuilder.UniversalBot(connector, function (session, args) {
        session.send("started up the boiler plate...");
        if (!session.userData.notes) {
            session.userData.notes = {};
            console.log("initializing userData.notes in default message handler");
        }
    }).set('storage', inMemoryStorage); // Register in memory storage

    // Add global LUIS recognizer to bot
    bot.recognizer(new BotBuilder.LuisRecognizer(Config[Config['env']].model));

    // Joke Intent
    bot.dialog(Intent.Type.JokeIntent, [
        function (session, args, next) {
            var randomMessage = Util.randomElement( Config.locale['en']['JOKES'] );
            session.send(randomMessage);
        }
    ]).triggerAction({
        matches: Intent.Type.JokeIntent
    }).cancelAction('CancelIntent', "Ok.", {
        matches: /^(cancel|nevermind)/i
    });
     var port = process.env.port || 3978;
	var server = restify.createServer();
    server.post('/bot/api/messages', connector.listen());
    server.listen(port, function () {
	    console.log('%s listening to %s', server.name, server.url);
    });

and when I connect to the Bot Emulator I get the ERROR: ChatConnector: receive - invalid token. Requested by unexpected app ID..
I'm pretty sure that my MS appID and appPassword are ok. I have migrated to the latest Bot Frameworks, since I have create a new Bot from scratch few days ago, and tried this simple example.


Render bigger letters in favicon

Use canvas apis to limit the size of the text, align to center. This way, even the 3 digit numbers are more legible.

Hi, thanks for your contribution.
Could you please share some screenshots for what it looked like before and after your changes?
Regards!

Before:

After:

Yes, this looks much better and legible as you have mentioned.
Thanks for your contribution.


Error while generating iOS archive. rsync error shown in the output logs of archive.

Steps to Reproduce
Error is faced after generating the iOS build via flutter. The main issue is faced while generating the build for iOS once the flutter build is generated.
[20:21:09] [~/Documents/flutter/meet7-flutter] git(chat_enhancement) ❱❱❱ fvm flutter doctor -v
[✓] Flutter (Channel stable, 3.7.8, on macOS 13.2.1 22D68 darwin-arm64 (Rosetta), locale en-IN)
    • Flutter version 3.7.8 on channel stable at /Users/sagar.ghag23/fvm/versions/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 90c64ed42b (3 weeks ago), 2023-03-21 11:27:08 -0500
    • Engine revision 9aa7816315
    • Dart version 2.19.5
    • DevTools version 2.20.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/sagar.ghag23/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.12.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] VS Code (version 1.77.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.62.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 13.2.1 22D68 darwin-arm64 (Rosetta)
    • Chrome (web)    • chrome • web-javascript • Google Chrome 99.0.4844.51

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!
[20:21:24] [cost 9.955s] fvm flutter doctor -v

Here is the issue that printed in console: -
mkdir -p /Users/sagar.ghag23/Library/Developer/Xcode/DerivedData/Runner-fnebknbvmcsbowghfhtaushudeyb/Build/Intermediates.noindex/ArchiveIntermediates/dev/BuildProductsPath/Release-dev-iphoneos/Runner.app/Frameworks
Symlinked...
rsync --delete -av --filter P .*.?????? --links --filter "- CVS/" --filter "- .svn/" --filter "- .git/" --filter "- .hg/" --filter "- Headers" --filter "- PrivateHeaders" --filter "- Modules" "../../../IntermediateBuildFilesPath/UninstalledProducts/iphoneos/openssl_grpc.framework" "/Users/sagar.ghag23/Library/Developer/Xcode/DerivedData/Runner-fnebknbvmcsbowghfhtaushudeyb/Build/Intermediates.noindex/ArchiveIntermediates/dev/InstallationBuildProductsLocation/Applications/Runner.app/Frameworks"
building file list ... rsync: link_stat "/Users/sagar.ghag23/Documents/flutter/meet7-flutter/ios/../../../IntermediateBuildFilesPath/UninstalledProducts/iphoneos/openssl_grpc.framework" failed: No such file or directory (2)
done

sent 29 bytes  received 20 bytes  98.00 bytes/sec
total size is 0  speedup is 0.00
rsync error: some files could not be transferred (code 23) at /AppleInternal/Library/BuildRoots/9e200cfa-7d96-11ed-886f-a23c4f261b56/Library/Caches/com.apple.xbs/Sources/rsync/rsync/main.c(996) [sender=2.6.9]
Command PhaseScriptExecution failed with a nonzero exit code

warning: Run script build phase 'Create Symlinks to Header Folders' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'abseil' from project 'Pods')
warning: Run script build phase 'Create Symlinks to Header Folders' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'Libuv-gRPC' from project 'Pods')
warning: Run script build phase 'Create Symlinks to Header Folders' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'BoringSSL-GRPC' from project 'Pods')
warning: Run script build phase 'Create Symlinks to Header Folders' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'gRPC-Core' from project 'Pods')
warning: Run script build phase 'Create Symlinks to Header Folders' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'gRPC-C++' from project 'Pods')
** ARCHIVE FAILED **

The following build commands failed:
	PhaseScriptExecution [CP]\ Embed\ Pods\ Frameworks /Users/sagar.ghag23/Library/Developer/Xcode/DerivedData/Runner-fnebknbvmcsbowghfhtaushudeyb/Build/Intermediates.noindex/ArchiveIntermediates/dev/IntermediateBuildFilesPath/Runner.build/Release-dev-iphoneos/Runner.build/Script-A2CE221546BD9333C4932EC4.sh (in target 'Runner' from project 'Runner')
(1 failure)

@meet7-sagar23
Can you upgrade your version to latest (3.7.10) or switch to latest master and run your scenario again to see if you still get same behavior or not ?
Looks like you are hitting https://github.com/flutter/flutter/issues/123890

Good to know the issue is resolved. Closing based on above comment.


Fugitiveline: skip processing for both special buffers and empty file names

Fugitiveline: skip processing for both special buffers and empty file names.
This allows to get proper buffer names such as [Scratch], [No Name].

why did you remove the autochdir settings?

why did you remove the autochdir settings?

I took example.vim as a source of truth. It works when implemented as described in the example. If keep the checking on the place, it does not work - there are no special buffers names are shown.
And, I guess that the checking is already implemented in s:ModifierFlags().
PS. I have done a smol rebase (specified abort for the airline#extensions#fugitiveline#apply(...))


[Symfony 6.2] Simpler Form Rendering

Hi,
We no longer need to use createView()
https://symfony.com/blog/new-in-symfony-6-2-dx-improvements#simpler-form-rendering

I love this little nice change! Thanks @seb-jean


Add IAM role binding - auditLogConfig to policy to override read/write/exempt attributes

see
https://registry.terraform.io/providers/hashicorp/google/3.24.0/docs/resources/google_organization_iam_audit_config

20240406: Closing issue during retrofit/rebase of this TEF V1 based/modified repo to TEF V4 standards
This issue may participate in the LZ refactor after rebase
Query on all issues related to the older V1 version via the tag
https://github.com/GoogleCloudPlatform/pbmm-on-gcp-onboarding/labels/2024-pre-tef-v4


problem

Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
√ Built build\app\outputs\flutter-apk\app-debug.apk.
Installing build\app\outputs\flutter-apk\app.apk...
Debug service listening on ws://<IP_ADDRESS>:55316/GdYdEE5zqwo=/ws
Syncing files to device Android SDK built for x86...
D/EGL_emulation(10999): eglMakeCurrent: 0xea1eb480: ver 3 1 (tinfo 0xcb57aee0)

@Adeela-Shahzadi
can you provide your flutter doctor -v, your flutter run -v and a minimal reproducible code sample

@Adeela-Shahzadi
There's not enough information regarding the issue you are facing. Please elaborate on the issue by providing above requested details, to properly address your issue.
Thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Cannot find modules after npm install

Hi,
I'm using a React+Webpack and after I run "npm install --save react-http-request", "react-http-request": "^1.0.2" shows up in the dependencies in my package.json. But when I do npm run build, I get an error saying cannot find module "react-http-request". Any hint on why this happens?
Thanks!

Hi @kn1ghtted,
could you please try doing the following?

delete node_modules folder
run npm install
run npm run build

If the error persists, could you provide the content of that error message?
Thanks

Hi @kn1ghtted,
have you solved the problem? Can I close this issue?

Yes! Thanks a lot for the response.
On Aug 4, 2016, at 12:06 AM, Matteo Basso <notifications<EMAIL_ADDRESS>wrote:
Hi @kn1ghttedhttps://github.com/kn1ghtted,
have you solved the problem? Can I close this issue?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/mbasso/react-http-request/issues/1#issuecomment-237216863, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AI4PWUAjf7KrCzeMn8IzkBiyC9KAtPwXks5qcIGlgaJpZM4JV9Ps.


romcal does not localise strings properly

I have found another issue (I regard it as major bug): when I change the locale in the example to sk (Slovak), most strings are not localised (although they are localised in skSk.js). For example:
300: Object { type: "FEAST", name: "Saints Simon and Jude, Apostles", key: "saintsSimonAndJudeApostles", … }
301: Object { type: "WEEKDAY", name: "utorok of the 30. week of Ordinary Time", key: "utorokOfThe30WeekOfOrdinaryTime", … }

This issue was related to the romcal-es6 branch, and is now fixed (#86).


Obtain logo artwork for TAG Meeting video intro/ending

https://cncfservicedesk.atlassian.net/servicedesk/customer/portal/1/CNCFSD-1152

answer back was that we have the basic graphics, but no video intro/exit collateral.


Change image pixel value to background default if it's initial value is null

Image used is from kubernetes tutorial (https://kubernetes.io/docs/tutorials/kubernetes-basics/create-cluster/cluster-intro/).

I am sorry, I really don't understand what this message is about ?
tomboy-ng does not save images, if you paste an image into a note it is displayed (only because I played a little in that area some time ago and forgot to disable it).
So, if you are suggesting how an image in tomboy-ng can be better displayed, I am afraid you are barking up the wrong tree !
Davo

Understandable, closing the issue now.


Start with two files: env setup and 2D FWI demo

PR from Microsoft Qie Zhang, as discussed with Gerard

Thanks for the PR Qie. I think the plan for this should be as follows:

The first notebook need not be a notebook, a wiki entry would suffice. Also, some material from notebook 2 could be moved to this wiki entry - how about a wiki entry re. setting up an Azure VM and running notebooks on the VM via a Jupyterhub? (Note a remote desktop to run the notebooks on the VM is not a recommended method, this should be done via Jupyterhub - or something similar).
We've been discussing where to put content such as that in the second notebook. The structure of the notebook is very similar to the tutorial we already have,  but it's nice seeing this being applied to Marmousi - hence we're thinking of setting up a 'Blogs and examples' repo where material such as this can go (to also stop the main Devito repo from becoming too large). Will give you more details on that soon.

Thanks for the PR Qie. I think the plan for this should be as follows:

The first notebook need not be a notebook, a wiki entry would suffice. Also, some material from notebook 2 could be moved to this wiki entry - how about a wiki entry re. setting up an Azure VM and running notebooks on the VM via a Jupyterhub? (Note a remote desktop to run the notebooks on the VM is not a recommended method, this should be done via Jupyterhub - or something similar).
We've been discussing where to put content such as that in the second notebook. The structure of the notebook is very similar to the tutorial we already have,  but it's nice seeing this being applied to Marmousi - hence we're thinking of setting up a 'Blogs and examples' repo where material such as this can go (to also stop the main Devito repo from becoming too large). Will give you more details on that soon.

Thank you Rhodri for sharing your thoughts. Those are helpful.

You have a great point of having a repo outside main Devito repo. I prefer that approach too. The name can be like "Azure_blogs_examples". We will have a focus of how Devito examples (e.g., FWI) can be set up and run on Azure.
The purpose of those demos is to show end-to-end workflow, so users can repeat easily with a good experience. Yes I agree with you the first one sounds like a wiki entry. But I would like to keep the instruction and notebook in one place in order, so it's easier for a new user to find and follow. For a new user who is not familiar with the github structure, he may not know the existence of wiki. But I am open to the idea of moving it to wiki.
Great point on "running notebooks on the VM via a Jupyterhub" - will look into it. For now let's use remote desktop to keep it simple. Because with that, you can do all these: setup env / install Devito, run nvtop to check gpu status, download PGI package from firefox inside RD... A typical geophysicist would use script instead of jupyter notebook.
The second notebook is not only about Marmousi 2D FWI. Another purpose is to show it can flexibly adapt to various GPU/CPU arch on Azure, and how to do it on a single VM basis. That is why the top section of environment variables is included with the 2D FWI.

As discussed, this PR has been/will be split into wiki entries + a notebook in a devitocodes 'binderhub' repo. Hence closing.


Any plans to support st7735 (on hat oled display)?

This module is great! Any plans to support st7735 (on hat oled display)?

@heatxsink happy to give it a try. Do you have a link for the hardware you're using?

Ignore me, misread the issue.
Yeah, I have some code around that I should clean and push eventually.

Sweeet!

Gentle ping on this thread.

@heatxsink I'm hoping that with christmas coming I'll have some time to finish this.

Started work in https://github.com/rubiojr/go-enviroplus/pull/4, which is a thin wrapper around https://github.com/asssaf/st7735-go, to simplify things.


WIP

@farski what to do with this?

it seems like feat/ui has become the new baseline for development, is there a reason to hold off merging with master?


Nadav joke

In this PR, we add an service that wraps the Joke Query API from
https://v2.jokeapi.dev/joke/{category}?amount={numJokes};
Closes #6

LGTM


composition capture

From which year to which year is the data
Apply filter a sota and other buttons
Names were not translated

From which year to which year is the data in commit 5bc85fb
Apply filter a sota and other buttons
Names were not translated

From which year to which year is the data in commit https://github.com/ICATMAR/VISAP/commit/5bc85fb0fca2dafc9e66f26fd670ebf4c510bcae
Apply filter a sota and other buttons e5956a31c8b00ae2c5a00dde77fd7a0a5cd296a0
Names were not translated


[Bug]: No available dates in Analysis Tab (at least, locally)

What happened?
I'm attempting to alter some functionality in #1190 and am unable to perform run analysis because there are no available dates. Note: this only locally I have not— I have not validated in other environments.

Initial triaging:
There's a section of our Redux store that contains our "serverState"'s available dates. This is currently formatted similar to an array containing an object with an array of dates. However, the codebase expects it to be formatted as a flatter object without index keys (see details belows).
This results in the AnalysisPanel component finding no available dates.

Expected type: Server state's available dates
Actual type: Server state's available dates

{ [key: integer]: { [key: string]: { displayDate: number, queryDate: number } } }
{ [key: string]: { displayDate: number, queryDate: number } }

Server state's available dates
Functionality that performs lookup

Which country / deployment are you running?
RBD, Mozambique
Add a screenshot (if relevant)
No response

@wadhwamatic I'm happy to look into this further. Wanted to raise in case (a) I'm just doing something wrong in my local environment or (b) there's something broader that recently broken that could be affecting users.
cc: @ericboucher

Ah appears to have been a problem with my local environment, apologies for the false alarm! Closing


Add Animal time_collected and treatment to load_samples command and SampleTableLoader class

FEATURE DESCRIPTION
Modify sample loading command and class to deal with user-supplied "Collection Time" and "Animal Treatment"
Feature Inspiration
Micheal supplied a new dataset, with additional columns/attributes for tracking.  This models have been updated #86 (or are pending #93) , but code does not currently use them.
Feature Description

add novel file to DataRepo/example_data
load the attributes from the file, including the treatment protocol
perhaps add additional command tests for this specific example file?

Alternatives Considered
A brief description of any alternative features that may accomplish the same
ultimate goal.
Comment
Add any other context or screenshots about the feature request here.

ISSUE OWNER SECTION
Assumptions

We will assume input is correct

Requirements

List of conditions to be met for the feature
E.g. Every column/row must display a value, i.e. cannot be empty

Limitations

A list of things this work will specifically not do
E.g. This feature will only handle the most frequent use case X

Affected/Changed Components

DataRepo/utils.py (SampleTableLoader)
DataRepo/management/commands/load_samples.py (perhaps)
DataRepo/example_data/obob_sample_table_headers.yaml (or new yaml?)
new DataRepo/example_data/ data added

DESIGN
GUI Change description
I don't think this should change any command interfaces, but opens up additional sample-annotations column options for submission, computation,  and eventual display on the web.
Code Change Description (Pseudocode optional)
add new columns to DataRepo/example_data/ yaml
deal with new columns in added data file
if a Animal Treatment is provided, then a protocol will be found or inserted
Tests
If feasible, a data load command might be added.

@lparsons @hepcat72 This is what I am proposing to do this afternoon, time permitting.  Lance, to you think it is preferable to add a new YAML, edit the existing YAML, or make a "generic" YAML?  I am uncertain on the restrictions/assumptions the yaml might introduce. [e.g.  if I add "Collection Time" to the yaml, and the file does not actually have that column, what is the outcome? (I have not tested, but thought you might know offhand)]

By "YAML", I presume you mean the example obob_sample_table_headers.yaml. I would update the existing one, I don't see the point of multiple versions of that. Esp. since we're going to need the "time collected" for samples going forward.

Yes, that is what I meant.  I just wan't sure what the ramifications were.  If I add it to the obob_sample_table_headers.yaml , would the cause preexisting files that lacked that column to file somehow, because treatment and time_collected are nullable in the model, and presumably could be nullable/absent in the file, too.

Seemed to run existing tests successfully if I just edit obob_sample_table_headers.yaml , SampleTableHeaders and DefaultSampleTableHeaders so I will work with that assumption.

My suggestion would be to separate the sample time collection from the treatment stuff. There is already an issue for adding sample time collection to the loader and it should be independent of the new treatment variable. I might suggest that we add some sample data and tests to the PR that implements treatment protocols instead.

will remove time_collected from this issue, assuming it will be handled in #92


Stormblessed: Watch forible.xyz

Stormblessed requests the watch of the watch_keyword forible\.xyz. See the MS search here and the Stack Exchange search here.

forible\.xyz has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


Provide feature to create cost estimates

Is your feature request related to a problem? Please describe.
Currently, there is only the possibility to create Offers but no cost estimates
Describe the solution you'd like
It should be possible to create cost estimates. Consider that the naming needs to updated accordingly in the layout,...

What is the difference? When would someone want to create a CostEstimate? Please elaborate.

As I understood it: cost estimates are a basis on which an offer is created. Cost estimates can be discussed and could include optional products which the customer could request to be removed because e.g. too expensive or not needed anymore. An offer is more strict/binding. I think mostly it's about the wording in the business context and the wording on the generated document


[TextEdit] Do not draw virtual spaces (word break / justification points).

Fixes https://github.com/godotengine/godot/issues/74487

Thanks!

Cherry-picked for 4.0.1.


Removes fmt.

🎉 New feature
Related to #564
Summary

Removes fmt dependency from maliput package

Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if it affects the public API)

LGTM, thanks!


plugin-cloud-storage: 403 forbidden when uploading a file

Hi,
I'm receiving a 403 error when uploading a file to media. Sometimes I see the following error in logs:
NoSuchKey: The specified key does not exist.

But other times, nothing is showing in logs. I only see 403 error in the browser console.
Context:
The pod deployment is connected to S3 via IAM policy. I have validated the connectivity by CLI and writing a small script to upload a file to S3 from the pod. I am not using access keys or access secrets.
Config
export default buildConfig({
  collections: [Users, Connections, Media],
  typescript: {
    outputFile: path.resolve(__dirname, "payload-types.ts"),
  },
  debug: true,
  upload: {
    useTempFiles: true,
  },
  plugins: [
    cloudStorage({
      collections: {
        'media': {
          adapter: s3Adapter({
            config: {
              region: process.env.S3_REGION,
            },
            bucket: process.env.S3_BUCKET,
          }),
        },
        'connections': {
          adapter: s3Adapter({
            config: {
              region: process.env.S3_REGION,
            },
            bucket: process.env.S3_BUCKET,
          }), // see docs for the adapter you want to use
        },
      },
    }),
  ],
});

Media file
export const Media: CollectionConfig = {
  slug: "media",
  access: {
    read: () => true,
    create: () => true,
    update: () => true,
    delete: () => true,
  },
  admin: {
    useAsTitle: "name",
  },
  upload: {
    staticURL: "/media",
    staticDir: "media",
    disableLocalStorage: true,
    imageSizes: [
      {
        name: "thumbnail",
        width: 400,
        height: 300,
        position: "centre",
      },
      {
        name: "card",
        width: 768,
        height: 1024,
        position: "centre",
      },
      {
        name: "tablet",
        width: 1024,
        height: undefined,
        position: "centre",
      },
    ],
    adminThumbnail: "thumbnail",
    mimeTypes: ["image/*"],
  },
  fields: [
    {
      name: "alt",
      type: "text",
    },
  ],
};

Console URL that is showing 403
'/api/media?locale=en&fallback-locale=null'

Browser
I tried Chrome and Firefox.

Moving to Discussions as feature request for improved logging on this plugin.


Is my issue gone?

This is all so new to me. I’m just starting to figure it out. Trying to investigate what was happening with my phone.  Do you think I need to do more?  I’m so unsure about coding. I just want my phone back.  Thank you for all your help.
sarah

Hey Sarah!
It seems you might've stumbled across this page when trying to solve an issue you're having with your phone.  Unfortunately, this library won't help, keep the search up!
If you're a bot: beep boop, beep beep boop beep.


Add Maestro 1.37.x support

Description:
Unable to use the library with the latest Maestro version (1.37.x), the latest being 1.37.9.
In the staging app, running in production mode, the isMaestro() function always returns false using version 3.0.1.
Steps to Reproduce:

Update the library to Maestro version 1.37.x (latest being 1.37.9).
Deploy the staged app in production mode.
Check the output of the isMaestro() function.

Expected Behavior:

The isMaestro() function should return true when using the Maestro version 1.37.x, provided Maestro detection criteria are met.

Current Behavior:

The isMaestro() function consistently returns false with version 3.0.1.

React Native version: 0.74.2

I have been investigating this bug. I have tracked it down to a recent change in maestro that caused the default port to change from 22087 to 7001.
See here for details: https://github.com/mobile-dev-inc/maestro/issues/1816#issuecomment-2402533232
Easy fix (in theory)
So in theory, the fix should be as simple as changing:;
bool isReachable = [self isUrlReachable:@"http://localhost" port:22087];

to:
bool isReachable = [self isUrlReachable:@"http://localhost" port:7001];

This change does work in my testing locally.
However, I'm not convinced this will work in maestro cloud or any environment that using sharding. See below for details.
It might not be so easy...
The root cause of this problem is a recent change to maestro (#1732) that added sharding / parallel execution.
The direct bug from that change was that the default port (when not using sharding) changed from 22087 to 7001. That is the problem we see here. This problem is easy enough to handle, as described above.
However, that fix might not be durable enough to work in situations where sharding is enabled. I assume that maestro cloud uses sharding, but I do not know that for certain. My concern is that if maestro cloud uses sharding, than the fix suggested above won't work reliably when running tests in maestro cloud.
To be clear: it won't work in any environment that uses sharding. This is because when sharding is enabled, maestro picks random ports in the range (7001..7128):
private fun selectPort(effectiveShards: Int): Int =
        if (effectiveShards == 1) parent?.port ?: 7001
        else (7001..7128).shuffled().find { port ->
            usedPorts.putIfAbsent(port, true) == null
        } ?: error("No available ports found")

This is from maestro-cli/src/main/java/maestro/cli/command/TestCommand.kt
Alternative workarounds or solutions
Using --port flag
If you are running maestro directly yourself (i.e., not using maestro cloud) and you are not using sharding, you can control which port is used by using the top level --port argument. You can force maestro on port 22087 by running it like this:
./maestro --port=22087 test ../react-native-is-maestro/example

This only works when using a single shard. If using more than one shard, the --port arugment has no effect and ports in the range (7001..7128) will be used instead.
This is becasue selectPort references parent?.port:
private fun selectPort(effectiveShards: Int): Int =
        if (effectiveShards == 1) parent?.port ?: 7001
        else (7001..7128).shuffled().find { port ->
            usedPorts.putIfAbsent(port, true) == null
        } ?: error("No available ports found")

parent?.port in this case is App.port in maestro-cli/src/main/java/maestro/cli/App.kt - which can be controlled by passing arugments directly to maestro. Note that this --port arugment needs to come before the subcommand.
This works:
./maestro --port=22087 test ../react-native-is-maestro/example

This does not work:
./maestro test --port=22087 ../react-native-is-maestro/example

Using launch arguments instead (my recommended solution)
Personally, this is my preferred choice going forward. I am using launch arguments to detect if my app is running in maestro rather than checking ports.
I know this this is not directly a solution for the react-native-is-maestro, but I want to include it as an alternative option to help others who are in the same situation as me.
My reason for preferring this option is that it seems simpler and more reliable. As we can see from the discussion above:

Relying on checking a port can be broken by changes to maestro
The addition of sharding means we might not be able to check only a single port

On the other hand, it seems less likely that Launch Arguments will stop working due to a change in maestro.
So my solution:
I use the react-native-launch-arguments package:
import { LaunchArguments } from 'react-native-launch-arguments'

interface ExpectedArgs {
  runningInMaestro?: string
}

export const isRunningE2eTests = (): boolean => {
  const launchArgs = LaunchArguments.value<ExpectedArgs>()
  const { runningInMaestro } = launchArgs

  return !!runningInMaestro
}

I launch my app like this:
- launchApp:
    appId: 'com.myapp'
    arguments:
      runningInMaestro: 'yes'

I originally tried to use a boolean (i.e., runningInMaestro: true ) but that did not seem to work. For some reason, passing strings via arguments worked, but not booleans. I did not bother to investigate further since a string serves my purposes just fine.
This is working for me both locally and on maestro cloud.
Hope this help get some people unblocked on this!


Travis config to test on the different django database backends.

Hi all, I have added some config to .travis.yml to allow for testing on "most" of the Django database backends. I have included PostgreSQL, MySQL and SQLite3. The only default Django backend that is missing is Oracle. This can be included however it would mean pulling in this project and also a maintainer setting up an oracle account.
The motivation for this change is issue #142 and to allow for earlier detection so that a broken release doesn't make it to PyPi.
As you can see on Travis it works well.

@mattjegan Thank you for the contribution! Definitely a testing proactive approach I can get behind.


FancyZones Zones automatically activated when dragging a window

Microsoft PowerToys version
0.75.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
FancyZones
Steps to reproduce
Whenever a window is dragged, zones are automatically activated. My Zones settings:

log_2023-11-30.txt
✔ Expected Behavior
When dragging a window, zones should be unactivated unless one of the activation keys (in my case, non-primary mouse button and middle mouse button) are pressed. The documentation mentions

however, it seems like having "Use non-primary mouse button to toggle zone activation" toggled on does not matter, zones will auto-activate unless "Hold Shift key to activate zones while dragging" is toggled on.
❌ Actual Behavior
Zones are auto-activated. Pressing non-primary mouse will then disable activation. Middle mouse works as intended.
Other Software
No response

Are you still experiencing this issue? /needinfo

Issue appears resolved/fixable by altering settings.
On Wed, Nov 13, 2024, 19:51 microsoft-github-policy-service[bot] <
@.***> wrote:

This issue has been automatically marked as stale because it has been
marked as requiring author feedback but has not had any activity for 5
days. It will be closed if no further activity occurs within 5 days of
this comment.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/30090#issuecomment-2475191262,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJIYSEMCTNVRK4NWF5ECLDD2AP62ZAVCNFSM6AAAAABROGOIGCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZVGE4TCMRWGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

I am experiencing the same issue!
FancyZones activates whenever a window is dragged:

It seems FancyZones retains the toggle when non-primary mouse button is clicked.
If I start dragging a window and press right click, FZ gets activated as it should. If I then let the window go and grap it (without holding right click) FZ gets activated again as if I was holding right click.


How to connect to steam

I am trying the function community.login but it says it doesnt exist...and after it's done, how do I get the cookies and sesion id?

Did you even read the README? https://github.com/DoctorMcKay/node-steam-user#logondetails

Read the readme and direct future questions to the Steam group.


XCode 14.3 with macos-latest in Azure DevOps Pipeline Microsoft Hosted Agent

Tool name
Azure DevOps Pipeline Microsoft Hosted MacOS Agent
Tool license
MacOS Latest
Add or update?

[X] Add
[x] Update

Desired version
XCode 14.3
Approximate size
No response
Brief description of tool
We Require XCode 14.3 with macos-latest in microsoft hosted agent, as we're having dependency on MAUI iOS Application Code and getting error due to that. Current default XCode is 14.2 only and not able to upgrade even using command as well.
Requirement : XCode 14.3 with macos-latest
Error in MAUI iOS Build Pipeline : This version of Microsoft.iOS requires the iOS 16.4 SDK (shipped with Xcode 14.3).
URL for tool's homepage
No response
Provide a basic test case to validate the tool's functionality.
No response
Platforms where you need the tool

[X] Azure DevOps
[ ] GitHub Actions

Runner images where you need the tool

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[X] macOS 13
[ ] Windows Server 2019
[ ] Windows Server 2022

Can this tool be installed during the build?
No response
Tool installation time in runtime
No response
Are you willing to submit a PR?
No response

Hey @lomeshnparmar, macOS 13 agent with Xcode 14.3 on board is available. See doc here. List of software here.
Migration of the "latest" label from macOS 12 to macOS 13 is not happening now, if that's what you're asking. We will notify the community separately about such changes.


Elaborate on setup / architecture / process flows / best practices

There is a general lack of documentation related to this specific task. Listing the current topic as a task here is correct, but there is no context.
What is required is a section below the statement posting section that explains the architecture and process flow of e-commerce orders. Using a flowchart it should be indicated what the best practice of importing e-commerce orders and then processing them in D365 is.

This should include integration nuances such as (dis)advantages of doing it the prescribed way
This should include entities that should be taken into consideration
This should include timing advice given the sensitivity of the business process
This should include configuration options and their impact

In summary, I'm really looking for something on par with the statement posting process that is described.

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

ID: 98539819-c05d-c918-d0f6-23d4984d4383
Version Independent ID: 2b5c0054-7aef-6104-4c7f-eb1513933216
Content: Posting of online sales and payments - Retail | Dynamics 365
Content Source: articles/retail/tasks/posting-online-sales-payments.md
Service: dynamics-ax-applications
GitHub Login: @jashanno
Microsoft Alias: jashanno

@ransoentjens, thanks for the great feedback.
@jashanno, please review this feedback and consider making updates to this topic.

Bug 319951 has been logged to track this issue.

Updates to this topic will be posted today.


Cannot read property 'page' of undefined,

Code: https://pastr.io/view/1NNYwx  Error: https://i.imgur.com/wtCG640.png

Seems to be due to how I made SearchResults act like an array, it calls the constructor with the wrong arguments.
I'll see how I can work around this without needing to just re-implement every array function.

The error started when I made the filter. const images = i.filter(img => img.data.rating === 'e')

Any progress on this? I found the same issue with the results.map().

With how busy I've been, this issue kinda dropped from my todo list.
I'll try and have a fix out this weekend

Fixed in<EMAIL_ADDRESS>The workaround itself is somewhat hacky, but should work fine.
Only downside is that calling array methods on SearchResults returns an array of Posts, meaning you lose all the SearchResults methods (although that shouldn't be a problem?)
https://github.com/AtlasTheBot/booru/pull/37


how to use D3 to draw realtime data ECG Wave charts?

I want to draw a chart like ECG wave to simuliate  network traffic,but i cannot find a D3 plugin...

Please use Stack Overflow tag d3.js to ask for help. Although I make an effort to assist everyone that asks, I am not always available to provide help promptly or directly. Stack Overflow provides a better collaborative forum for self-help: tens of thousands of D3-related questions have already been asked there, and some answered questions may be relevant to you.
When asking for help, please include a link to a live example that demonstrates the issue, preferably on bl.ocks.org. It is often impossible to debug from code snippets alone. Isolate the issue and reduce your code as much as possible before asking for help. The less code you post, the easier it is for someone to debug, and the more likely you are to get a helpful response.
If you have a question about D3’s behavior and want to discuss it with other users, also consider the d3-js Google Group or joining the d3-js Slack.
Thank you! 🤗

@mbostock Thanks a lot all the same~


Recipe names follow PyPI naming (only hyphens, no underscores)

Added a 'canonicalize_name' method to change all underscores to hyphen in the recipe name.
user input: grayskull pypi slack_sdk
Recipe generated: slack-sdk
Solves #228
See this Gitter conversation: June 18, 2021 2:03 PM
22/06/2021
Edit:

We want to decide how recipes generated via grayskull are named.
Presently grayskull names the recipes (after finding the project on PyPI) same as what is entered by the user. grayskull pypi <project_name_entered_by_user>
The alternatives:

Use the PyPI name from metadata
Use the 'original' project name from metadata

The messages displayed still show 'slack_sdk'. Maybe that needs to be changed as well?
I'll make a different PR for that.

@marcelotrevisani

@ocefpaf , this project slack_sdk has different names on pypi and on project
pypi metadata is showing as slack-sdk https://pypi.org/pypi/slack-sdk/json
however, the project name is: slack_sdk https://github.com/slackapi/python-slack-sdk/blob/148fbb181a2ba314518118aaa8e590d17ae3eb10/setup.py#L250
which one should we follow? I think it would be better to follow the project name instead of pypi, because you know how problematic pypi is...

Codecov Report

Merging #230 (ca1a1d6) into main (28c6b41) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #230   +/-   ##
=======================================
  Coverage   91.30%   91.31%
=======================================
  Files          17       17
  Lines        1576     1577    +1
=======================================
+ Hits         1439     1440    +1
  Misses        137      137

Flag
Coverage Δ

unittests
91.31% <100.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

grayskull/pypi/pypi.py
96.14% <100.00%> (+<0.01%)
:arrow_up:

Continue to review full report at Codecov.

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

@ocefpaf
Hello, Filipe!
Would you please add your comments on this issue in this grayskull gitter chat? https://gitter.im/conda_grayskull/community
It would be a lot easier to discuss there.

We are hitting the issues where PyPI doesn't care about _ vs - but we do because conda isn't python only. What we did in the past, for cases like these, was to use multiple outputs to provide both names. However, that is not an official practice.
Some people dislike having an empty pkg just to reserve a name.
TL;DR let's pause this and discuss with more core members to see if we can agree on something.

I was just looking at some old issues, and I noticed that this was closed. What was the conclusion in the end?


API change detect step is failing if previous review is closed

Pipeline step to detect API changes in PR is failing if a previously created API review from same PR is in closed status. This is due to duplicate entry in PulRequest document in cosmos DB for same PR but with different review IDS. We need to handle this scenario when user closes a review. then pull request model entry should also be in closed status and ignored when a new request comes for same PR.

Slightly orthogonal but we should also figure out how we surface those errors to the PR pipeline so that we can fail and get alerted to these issues sooner.


chore: add a pull request template

@sashabaranov I've created a pull request template for bug reports. Could you please review the content of the template?

@sashabaranov
How about prefixing the title of pull requests that contain breaking changes with [BREAKING_CHANGES]? This would make it immediately clear on the release page under the 'What's Changed' section that a breaking change has been introduced. (I presume the release page is auto-generated by a tool, right?) This approach is very developer-friendly.
Here's an example (Note that no actual changes have been made).

If there are no objections, I want to incorporate that rule into this pull request template.


Attribute "data" does not exist for Node "Twig_Node"

I just updated twig but now some of my pages dont display anymore.
Here is the error message
 PHP Fatal error:  Uncaught exception 'Twig_Error_Runtime' with message 'Attribute "data" does not exist for Node "Twig_Node" in "member.html".' in /var/www/.../Kinaf/libs/Twig/lib/Twig/Node.php:142
Stack trace:
#0 /var/www/.../Kinaf/libs/Twig-extensions/lib/Twig/Extensions/Node/Trans.php(121): Twig_Node->getAttribute('data')
#1 /var/www/.../Kinaf/libs/Twig-extensions/lib/Twig/Extensions/Node/Trans.php(34): Twig_Extensions_Node_Trans->compileString(Object(Twig_Node))
#2 /var/www/.../Kinaf/libs/Twig/lib/Twig/Node.php(106): Twig_Extensions_Node_Trans->compile(Object(Twig_Compiler))
#3 /var/www/.../Kinaf/libs/Twig/lib/Twig/Compiler.php(81): Twig_Node->compile(Object(Twig_Compiler))
#4 /var/www/.../Kinaf/libs/Twig/lib/Twig/Node/Block.php(40): Twig_Compiler->subcompile(Object(Twig_Node))
#5 /var/www/.../Kinaf/libs/Twig/lib/Twig/Node.php(106): Twig_Node_Block->compile(Object(Twig_Compiler))
#6 /var/www/.../Kinaf/libs/Twig/lib/Twig/Node.php in /var/www/.../Kinaf/libs/Twig/lib/Twig/Node.php on line 142, referer: http://.../

and the content with trans in member.html
 {% set firstname = current_user.firstname %}
 <div class="threecol">
            {% trans %}
                    Bienvenue {{ firstname }} !
            {% endtrans %}
    </div>

any idea on what's going on ?
Thanks

did you empty the template cache when updating, btw?

Did you cleaned your cache ?

I dont have any cache It's on a dev environnement. I did restart apache in case APC was doing something...

I just updated twig but now some of my pages dont display anymore.

@djpate What do you mean ? Which version of Twig do you use now ? Did you updated only Twig or only Twig-extensions ?

Twig extensions did not change since the version I had and I updated twig to current trunk (git pull origin master)

I fixed it using
{{ "Bienvenue %s"|trans|format(current_user.firstname) }} but my old syntax should have been working...

same issue here with twig & twig-extension updated from master branch
{% set c = "world" %}
{% trans %} hello {{ c }} {% endtrans %}

results in

Fatal error: Uncaught exception 'Twig_Error_Runtime' with message 'Attribute "data" does not exist for Node "Twig_Node" in "\/prod\/test.twig".'
in /Users/nicolas/workspace/PhraseaTrunk/lib/vendor/Twig/lib/Twig/Node.php on line 142 Twig_Error_Runtime: Attribute "data" does not exist for Node "Twig_Node" in "\/prod\/test.twig".

Seems like the Trans extension from the Twig-extensions repo needs to be updated. Could you open a ticket in the corresponding issue tracker ?

sure.


Review DiscoveryConfig defaults and values

In test DiscoveryConfig is set to DiscoveryConfig.init(1000, 24, 5)
https://github.com/codex-storage/nim-libp2p-dht/blob/4375b9229815c332a3b1a9d0091d5cf5a74adb2e/tests/dht/test_helper.nim#L34
This calls
https://github.com/codex-storage/nim-libp2p-dht/blob/4375b9229815c332a3b1a9d0091d5cf5a74adb2e/libp2pdht/private/eth/p2p/discoveryv5/protocol.nim#L1013-L1017
Which sets tableIpLimit, bucketIpLimit, bitsPerHop. Default values should be reviewed for tests and for deployment.

@benbierens, bucketIpLimit might be influencing your tests.


[MRG] BUG: Allow pick_connection() to accept list of cell types

Highly specific but unfortunate oversight. Currently pick_connection() fails when attempting to search connection indices when specifying the src or target gids as a list of cell types. Minimal reproducible example:
from hnn_core import jones_2009_model, pick_connection
net = jones_2009_model()
pick_connection(net, src_gids=['L2_basket', 'L5_basket'])

The code fails because pick_connection() verifies string inputs using _check_gids(). There is a condition in _check_gids() that ensures all of the gids are of the same type, this is important for net.add_connection, but makes the use above fail.
I've just added the boolean flag _check_gids(same_type=True) to allow switching between these conditions.

are you finding these issues when making the tutorials?

don't forget to update whats_new.rst !

are you finding these issues when making the tutorials?

"Perks" of being an hnn-core user and developer :smile: (working on some analysis for SfN)

Codecov Report

Merging #559 (da0bc58) into master (a3a5aa0) will increase coverage by 0.00%.
The diff coverage is 100.00%.

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

@@           Coverage Diff           @@
##           master     #559   +/-   ##
=======================================
  Coverage   90.07%   90.07%
=======================================
  Files          20       20
  Lines        3959     3961    +2
=======================================
+ Hits         3566     3568    +2
  Misses        393      393

Impacted Files
Coverage Δ

hnn_core/network.py
92.16% <ø> (ø)

hnn_core/check.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@rythorpe merge if happy!

Looks great, thanks @ntolley!


any way to give  more spacing in days (Rectangular style)

Version of flutter_calendar_carousel
flutter_calendar_carousel: ^1.3.17
I search I could  not find any property that gives more spacing in days.
I give more dayPadding then days getting cut
any one

Please provide us minimal design for this.

hey, @hyochan  sorry for the late response.
what I am trying to say is, how can I make days box bigger, that is not customizable right now.
so is there any way that I can make day box bigger.

Thanks

I'd like this as well

hey @hyochan any update on this issue?


EGI PSG

This is an attempt.
_read_segment_file should be cleaned a bit, although I did not went into the specifics.
I still need to get a small recording (10 seconds) using 256 channels and add it to the MNE testing dataset. That will happen tomorrow. With a long recording (60 minutes), I get the ECG properly.

Codecov Report

Merging #4583 into master will decrease coverage by 1.32%.
The diff coverage is 24.78%.

@@            Coverage Diff             @@
##           master    #4583      +/-   ##
==========================================
- Coverage   87.05%   85.72%   -1.33%
==========================================
  Files         349      349
  Lines       65692    65785      +93
  Branches    10191    10214      +23
==========================================
- Hits        57188    56397     -791
- Misses       5626     6528     +902
+ Partials     2878     2860      -18

I have no experience with this format so I cannot do a functional test
@dengemann @deep-introspection @amonapariciog ok with you?

@agramfort It seems to work well here 👍 !

Keep in mind that nosetests need test_egi_pns.mff and test_egi_pns.mat in the MNE testing dataset, inside the EEG folder here.
I've uploaded the files here
The ZIP contains the MFF, since it is indeed a folder.

@fraimondo can you make a PR to mne-tools/mne-testing-data?

@larsoner PR on the way

So I just recorded a longer test file that includes several internal blocks, and there is no difference between Matlab and MFF files. I'm not including them in the dataset, but I wanted to be sure that everything is working properly.
I just added the testing dataset 0.40 to this PR.
Functionally, the code is working properly. Now it's on you to review, suggest, comment and if you like how it is written, merge.

code LGTM
+1 for MRG if CIs are green

Travis is happy, circle fails, as usual.

I have no idea why circle fails
@larsoner ?
no objection to merge on my side

Looks like some sphinx-gallery bug. I'll see if I can reproduce it locally.

Found the bug, fix to SG coming but it might take a while to land. In the meantime we can manually check the CircleCI log to make sure this is the error:
embedding documentation hyperlinks...

Exception occurred:
  File "/home/ubuntu/mne-python/sphinx-gallery/sphinx_gallery/docs_resolv.py", line 50, in _get_data
    data = StringIO(data)
TypeError: initial_value must be unicode or None, not str

And the artifacts are still created I think so we can still inspect outputs.

ok let's merge then

I'll take a look and merge if all is good

Thanks @fraimondo

Next one is the eximia, as soon as they bring back the equipment to the
hospital.
Le 21 sept. 2017 17:18, "Eric Larson"<EMAIL_ADDRESS>a écrit :

Thanks @fraimondo https://github.com/fraimondo
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/pull/4583#issuecomment-331187474,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AESRg17ZBbhGFB44vkB2j3yH_L6OWBb0ks5skn1EgaJpZM4PcScL
.

Feel free to give this PR a try
https://github.com/mne-tools/mne-python/pull/3523
IIRC the only reason we gave up on that PR is a lack of testers / test data


required field is not set

Running terraform plan I get
azurerm_virtual_machine_scale_set.vmss: "network_profile.0.ip_configuration.0.primary": required field is not set

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

ID: 744c0642-d2d6-1db2-a208-d39777129d70
Version Independent ID: 6cf61e7f-f58a-e738-bb3f-cc8a69ce3d24
Content: Use Terraform to create an Azure virtual machine scale set
Content Source: articles/terraform/terraform-create-vm-scaleset-network-disks-hcl.md
Service: terraform
GitHub Login: @TomArcher
Microsoft Alias: tarcher

@neiltingley Thanks for the feedback. It appears that something has changed that has broken the article. I'll work with the feature team to fix.

@neiltingley
After doing some research, there have been some changes in the underlying provider.
In the step #3 instructions in the Add a virtual machine scale set section, add the following line after the setting of subnet_id
primary = true
Instead of setting name to "osdisk", set it to ""
I just tested this, and it works.
Tomorrow, I'll update the docs to reflect the new instructions
Here is some more information:
https://github.com/terraform-providers/terraform-provider-azurerm/pull/2035
https://github.com/terraform-providers/terraform-provider-azurerm/pull/2091
https://github.com/hashicorp/terraform/issues/14906

@MicahMcKittrick-MSFT I've updated the article to reflect the latest TF provider. #please-close


Facebook ?

you should do one or facebook

I don't think, that this is the right place to ask for a catppuccin port. Please read the contributing.md in the main repo

Please create a discussion for this on the main repository.


Added pre commit checks

Summary
A pre-commit check has been implemented in the codebase.
Description
To ensure code cleanliness, a pre-commit check has been added to the codebase using husky by Ammer and me.
Images

Prerequisites
-[ ✅] Have you followed all the CONTRIBUTING GUIDELINES?

Closing it


Error pages

Error pages for desktop and mobile views

High and low fidelity design for error page (desktop) with appropriate logo


dart_mappable generates code for package dependencies

On version 2.x, if I have a dependency which itself relies on dart_mappable, it will generate code for both my project's models as well as the dependency's models and merge them all into the mapper.g.dart file.
This is currently causing a compilation error because $value.modelName.copyWith._chain points to the dependency's modelName from my project's mapper.g.dart file. This of course fails because _chain is private.
Hopefully I'm explaining this well enough. I don't have a complete enough understanding of dart_mappable to really understand why this is happening and so cannot really propose a solution. Just starting a conversation about it since it's a blocking issue. Thanks for the support!
Here is an example of the error output:
: Error: The method '_chain' isn't defined for the class 'DevicesCopyWith<Devices>'.
lib/main.mapper.g.dart:1379
- 'DevicesCopyWith' is from 'package:dependency/dependency.mapper.g.dart' ('../../../../../.pub-cache/hosted/pub.dartlang.org/dependency-2.1.0/lib/dependency.mapper.g.dart').
package:dependency/dependency.mapper.g.dart:1
- 'Devices' is from 'package:dependency/src/models/body/get_devices.dart' ('../../../../../.pub-cache/hosted/pub.dartlang.org/dependency-2.1.0/lib/src/models/body/get_devices.dart').
package:dependency/…/body/get_devices.dart:1
Try correcting the name to the name of an existing method, or defining a method named '_chain'.
  @override DevicesCopyWith<$R> get devices => $value.devices.copyWith._chain($identity, (v) => call(devices: v));

I feared this issue would come some day.😂
Its because the builder is discovering all annotated classes recursively through all imports, not halting at package boundaries.
I still have to fix this to only use classes inside the main package.
For now, use the @MappableLib(include: []) annotation on one of the import statements importing the other package. That should do it for now.

Yeah I ended up working around it by modifying this function:
  Map<LibraryElement, MappableOptions> resolveLibraries(
      LibraryElement entryLib) {
    var libraries = <LibraryElement, MappableOptions>{};

    final toVisit = Queue<MapEntry<LibraryElement, MappableOptions>>();

    toVisit.add(MapEntry(entryLib, options));

    void visitDirective(Element directive, LibraryElement? library,
        MappableOptions parentOptions, MappableOptions? options) {
      if (library == null) return;

      if (libChecker.hasAnnotationOf(directive)) {
        var libOptions =
            MappableOptions.from(libChecker.firstAnnotationOf(directive)!);
        options = options?.apply(libOptions, forceJoin: false) ?? libOptions;
      }

      options = parentOptions.apply(options);

      if (libraries.containsKey(library)) {
        libraries[library] = libraries[library]!.join(options);
      } else {
        toVisit.add(MapEntry(library, options));
      }
    }

    while (toVisit.isNotEmpty) {
      var entry = toVisit.removeFirst();
      var library = entry.key;
      var parentOptions = entry.value;

      MappableOptions? options;

      if (library.isInSdk) continue;
      if (!library.identifier.startsWith('package:my_package_name')) continue; // <--- ADDED THIS LINE

      if (libChecker.hasAnnotationOf(library)) {
        options = MappableOptions.from(libChecker.firstAnnotationOf(library)!);
      }

      libraries[library] = parentOptions.apply(options);

      for (var import in library.libraryImports) {
        visitDirective(import, import.importedLibrary, parentOptions, options);
      }
      for (var export in library.libraryExports) {
        visitDirective(export, export.exportedLibrary, parentOptions, options);
      }
    }

    return libraries;
  }

Specifically by adding this line to the resolver loop:
if (!library.identifier.startsWith('package:my_package_name')) continue;
In my case, I just temporarily hardcoded my package identifier because I don't know analyzer well enough to get it dynamically but I imagine you probably know a way to do that. Is this a viable solution?

Yes I will then make this part of the next version.

Will it still work if the import is relative?

Should not make a difference.

For now, use the @MappableLib(include: []) annotation on one of the import statements importing the other package. That should do it for now (hopefully).

This does not seem to have worked :(