Add error messages for unsupported macro features on compilation

Continues the work from #3993
Error messages for weakref and dict.

@davidhewitt this is ready for another review

It looks like there's a compile error on the pipeline still:

It looks like the pipeline is still failing with this same error. I think you should be able to reproduce by running cargo test --lib --features=abi3-py37

I see the pipeline failures. I'll work on them later today.

After making the suggested change, we are getting the following error:

error: cannot find attribute pyo3 in this scope
--> src/tests/hygiene/pyclass.rs:12:46
|
12 | #[cfg_attr(any(Py_3_9, not(Py_LIMITED_API)), pyo3(weakref))]

Do I need to add an import or something to fix this?

Ungh, I realise now that with that suggestion I've blundered right into https://github.com/PyO3/pyo3/pull/2786 (and similar pains around cfg_attr).
I think the solution for now here would be to still use cfg_attr but wrap the whole pyclass in the attribute.
So something like
#[cfg_attr(any(Py_3_9, not(Py_LIMITED_API)), crate::pyclass(name = "ActuallyBar", ..., weakref))]
#[cfg_attr(not(any(Py_3_9, not(Py_LIMITED_API))), crate::pyclass(name = "ActuallyBar", ...))]
pub struct Bar {

Thanks for working on this at the PyCon sprints! 🚀
It looks like there's a compile error on the pipeline still:
error: `weakref` requires >= python 3.9 with the `abi3` feature
  --> src/tests/hygiene/pyclass.rs:16:5
   |
16 |     weakref,
   |     ^^^^^^^

I'd suggest having a look at that file and moving the weakref flag to a separate #[cfg_attr(any(Py_3_9, not(Py_LIMITED_API)), pyo3(weakref))].

I made this change and it gives a compiler error. I don't know rust well enough to even guess how to fix it. Do you have any suggestions?

I pushed a commit to apply something which should work as per https://github.com/PyO3/pyo3/pull/4194#issuecomment-2134555787

Ah, there was a lot of adjustment to conditional compilation to follow up where our own test suite had been silently ignoring the missing behaviour. Good to find that this makes us in a better place! 😂

I think the CI failure arises due to divergence between how we check for abi3 here and how pyo3-build-config activate the limited API for PyPy / GraalPy. I opened #4237 to simplify.

Where are we at with this PR? What can I do to get it over the line?

@codeguru42 great question! After the last updates I pushed, I think this PR should in principle be good to merge.
But naturally, making these error messages emerge properly has uncovered two pain points that I've been fixing separately. First there was #4237 which fixed a divergence between how we were setting the abi3 feature here in macro code versus in pyo3-ffi.
And then now the tests have picked up an old bug that's my fault with the 3.9 implementation, which I opened this morning in #4251.
My hope is that once that PR is merged, we'll be able to merge this one without further modifications 👀

@davidhewitt I merged main into this branch and resolved conflicts. But I have one question for clarification. As you can see in 3c7e898. main has Py_3_9 as the version for cfg but this branch has Py_3_10.  It looks like the change on this branch comes from 7177d537 where you pushed to this PR. I accepted the change from main assuming that it is probably correct, but I want to double-check with you to be sure.

Yep I think main is correct; let's try merging again. Thanks for fixing conflicts 🙏

Hooray, success finally! 🎉 Thanks again @codeguru42 and great to meet you at PyCon 👍

Closes #3945.


SPARK-4300 [CORE] Race condition during SparkWorker shutdown

Close appender saving stdout/stderr before destroying process to avoid exception on reading closed input stream.
(This also removes a redundant waitFor() although it was harmless)
CC @tdas since I think you wrote this method.

Test build #27999 has started for   PR 4787 at commit e0cdabf.

This patch merges cleanly.

Test build #27999 has finished for   PR 4787 at commit e0cdabf.

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

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

Jenkins, retest this please

Test build #28001 has started for   PR 4787 at commit e0cdabf.

This patch merges cleanly.

Test build #28001 has finished for   PR 4787 at commit e0cdabf.

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

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

LGTM. This is similar to #4788. I'm merging this into master and 1.2, and marking it to be back ported into 1.3 later.


Libloading declared most of its API unsafe in a subsequent release

This project depends on libloading = 0.6:
https://github.com/rust-pcap/pcap/blob/8b7bcfe12c79955c228e5657a1e63cdcce0af3f5/Cargo.toml#L43
It’s used in exactly one place:
https://github.com/rust-pcap/pcap/blob/8b7bcfe12c79955c228e5657a1e63cdcce0af3f5/build.rs#L102
But in the subsequent 0.7.0 release, libloading realized that most of its API, including Library::new, should have been unsafe. (The current release is 0.8.3)
“Hiding” soundness issues by using an old libloading version would seem to have a limited impact since only build.rs is affected, but it would be better to upgrade to a current version and actually deal with the unsafe API.

That really not important

That really not important, the only diff would be put the unsafe and close your eye cause we would have no way of ensuring the lib is not doing some static thing that we can't control.

Would you consider a PR that did simply that? Update the dependency to 0.8 and wrap the libloading::Library::new(libfile) file in unsafe {  }? That would acknowledge the situation and avoid the need to use an old libloading version indefinitely – both small but real improvements, I think.

An update would have to happen eventually. A PR would be welcome. Please note that the current pipeline is failing due to some weird Windows issue and some new clippy lints, but outside of that things should pass.


WFLY-11117 Executing legacy operations in mod_cluster subsystem

Jira
https://issues.jboss.org/browse/WFLY-11117

@honza-kasik Do you want to pretest this?

@rhusar Did a manual pretest and it seems to me, that issue is fixed. Thanks!

@bstansberry  Please update your review, thanks! :-)

@bstansberry Thanks!


Crosshair might be off-center

Why do i think that? Well i use a combination of yellow crosshair : and tf2s ingame green dot crosshair and i see some bits of the dot from the : crosshair and i figured out there there is a pos offset on it but idk where to change it.

Unfortunately with how font crosshairs work, there's no way to make them all perfectly centered.
It depends on what crosshair you're trying to use, but for the included one you'd adjust the xpos and ypos values here: https://github.com/rbjaxter/budhud/blob/master/%23customization/_enabled/bh_crosshairs.res#L27-L28
you'd basically adjust these values by one, hud_reloadscheme, see if it's good, and then repeat until it is. change 0.4999 to 0.4998, then 0.4997, and so on


installing gvm fails to update PATH variable in my .profile

On OSX 10.11.6, I tried each of the following:-

bash < <(curl -s -S -L https://raw.githubusercontent.com/moovweb/gvm/master/binscripts/gvm-installer)
curl -s -S -L https://raw.githubusercontent.com/moovweb/gvm/master/binscripts/gvm-installer | bash
zsh < <(curl -s -S -L https://raw.githubusercontent.com/moovweb/gvm/master/binscripts/gvm-installer)
curl -s -S -L https://raw.githubusercontent.com/moovweb/gvm/master/binscripts/gvm-installer | zsh

and in each case, the PATH variable is NOT updated. I ended up having to manually update the PATH variable.
And then, I can't use the use command, I get:-
ERROR: Unrecognized command line argument: 'use'

It turns out that the scripts don't handle aliases of commands used in any of the scripts.

@chrisvelevitch How did you fix the Unrecognized command line argument: 'use' issue?

@alexcurtin I removed/renamed the offending aliases on my machine, removed and reinstalled gvm.


SPA mode for static-adapter is broken

Describe the bug
Switching to SPA mode fails.
Goal is to completely disable svelte kit router and server side rendering to create simple one page application, which still uses $page.url.searchParams to read state from url (like https://gist.github.com/nosovk/c1c7d43071bf2b10bd6b50dd80e2eabc).
Steps to switch Kit app to SAP mode.

modify hooks.js to set ssr to false - https://github.com/nosovk/spa-mode-fails/commit/5c9a9f4ae78947fffe8f56c723e1ca268a439884
modify svelte.config to add kit.browser.hydrate=false - https://github.com/nosovk/spa-mode-fails/commit/242ea161026adb9f4dfa9feb8e69dddc1c40c8f5
build application and observe error: Error: options.hydrate only works if the component was compiled with the `hydratable: true` option`

actually hydrate explicitly disabled in config, but Kit expects to see it in enabled state.
Expected behavior: disabling hydrate should disable hydrate without broking application.
Reproduction
repo vs reproduction: https://github.com/nosovk/spa-mode-fails
deployed url: https://spa-mode-fails.vercel.app/
Logs
proxy.js:15 [HMR][Svelte] Unrecoverable error in <Root>: next update will trigger a full reload
logError @ proxy.js:15
await in logError (async)
handle_navigation @ start.js:832
_navigate @ start.js:425
goto @ start.js:301
start @ start.js:1497
(anonymous) @ (index):14
root.svelte? [sm]:50 Uncaught (in promise) Error: options.hydrate only works if the component was compiled with the `hydratable: true` option
    at Object.claim [as l] (root.svelte? [sm]:50:13)
    at init (index.mjs:1883:40)
    at new Root (root.svelte? [sm]:16:25)
    at createProxiedComponent (svelte-hooks.js:266:9)
    at new ProxyComponent (proxy.js:239:20)
    at new Proxy<Root> (proxy.js:346:11)
    at Renderer._init (start.js:968:15)
    at Renderer.update (start.js:886:9)
    at async Renderer.handle_navigation (start.js:832:3)
    at async Router._navigate (start.js:425:3)

System Info
System:
    OS: Windows 10 10.0.19043
    CPU: (8) x64 Intel(R) Core(TM) i5-8350U CPU @ 1.70GHz
    Memory: 3.83 GB / 31.88 GB
  Binaries:
    Node: 16.9.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.15 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 7.22.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1266.0), Chromium (98.0.1108.50)
    Internet Explorer: 11.0.19041.1202
  npmPackages:
    @sveltejs/adapter-static: ^1.0.0-next.28 => 1.0.0-next.28
    @sveltejs/kit: next => 1.0.0-next.267
    svelte: ^3.46.0 => 3.46.4

Severity
serious, but I can work around it
Additional Information
for my current project switching to kit.browser.hydrate=true is acceptable, but if somebody needs SPA only mode it could be troublesome.

This condradicts each other. You want an SPA, but you disable hydration and the router, which means no JavaScript should be sent to the browser, which means an SPA is impossible. This is also mentioned in the docs: https://kit.svelte.dev/docs/page-options#hydrate

If you want to use Vite without using any of Kit's feature, you may be better served with using a barebones Vite starter template like this one: https://github.com/vitejs/vite/tree/main/packages/create-vite/template-svelte
Closing as this works as designed, you can't disable SSR and hydration/router at the same time.


Arg to minify CSS and JS

Minify the CSS and JS. Use script.min.js and style.min.css.
(Rename js.js to script.js)

https://github.com/juancarlospaco/nim-css-html-minify#css-html-minify

Is it production ready? Cause then we shall implement it :) !
Do you have a JS minifier in pipeline?

I dont know if this should be inside nimwc, most CSS framework provide minified versions,
the only minifiers use NodeJS and Java, thats kind of moving the user away from Nim,
JavaScript syntax change very quickly and can get really weird,
most users will do this aside nimwc anyways.
:thinking:

You right. I personally use https://github.com/tdewolff/minify in an integrated solution for my website. Let's close this.


Implemented redim method on Dimensioned objects

This PR adds the redim method to easily replace dimensions or certain attributes on dimensions in a nested object (following the proposal in #714). The dimension overrides are supplied as keyword arguments to the method with the keyword matching the existing dimension by name and the value supplying one of three overrides:

A simple string to rename the dimension, e.g. curve.redim(x='Time').
A dictionary of parameter settings, e.g. curve.redim(x={'range': (0, 10), 'name': 'Time'})
A new Dimension object, e.g. curve.redim(x=hv.Dimension('Time')).

The objects the redim applies to can be narrowed down with a specification, which is already in use on the traverse, map and select method. This allows only applying the redim to certain objects in  three different ways:

The type of the object, e.g. layout.redim(hv.Curve, x='Time')
A string specification of the form type{{.group}.label}, e.g. layout.redim('Curve.GDP', x='Time')
A function, e.g. layout.redim(lambda x: x.vdims[0].name == 'GDP', x='Time')

The implementation exists in two places, on Dimensioned and on Dataset. The implementation on Dataset is required because a change to the dimension names has to be reflected in the underlying datasource. The rename implementation on all the interfaces so far is pretty trivial and I think it's good functionality to have anyway. Overall I'm pretty happy with this and will be very useful to have, as now it'll be possible to quickly rename dimensions and adjust their ranges.
I still need to work on the docstrings, docs and add unit tests.

I've now added unit tests, so the PR is ready for review.

Looks good to me!

Ok, it makes sense. It definitely has to be on Dimensioned and the reason you need some implementation on the interfaces is also clear.
My only question is why is it called rename on the interfaces too? Why not also call it redim? Maybe I'm missing something but I think it might be more consistent to have one name to consider denoting one concept. Even if the signature isn't identical etc, to me redim makes sense on an interface and well as on a dimensioned object.
Other than that one comment, I like this PR and am happy to merge.

My only question is why is it called rename on the interfaces too? Why not also call it redim? Maybe I'm missing something but I think it might be more consistent to have one name to consider denoting one concept. Even if the signature isn't identical etc, to me redim makes sense on an interface and well as on a dimensioned object.

I think I agree with you and may even say that the signatures should be the same. Some interfaces like iris also supplying units so those should also be updated. I'll go ahead and make that change.

Okay, my latest commit has renamed the interface methods from rename to redim for consistency. I think this is ready to merge now.

Ah, the push build passes but the pr one didn't? Is it a transient error?

Looks like it. Restarting.

So this makes no sense, one of the new unit tests is failing but only the PR build. Both the push build and my local copy work just fine.

Finally passing, turned out to be my fault. Part of the tests in this PR depended on the other iris PR.

Ok. Looks good! Merging.


replace average times with percentiles

Average response times can be misleading with high standard deviations, so I would suggest using either percentiles or mean with percentiles. I'd also add the max response time.
[mean, 50, 95, 99, max]

I can try to implement this @sdsykes

Well, it won't be easy change as I would like to introduce hdr histogram both into lambda, queue and aggregations.

I'd probably forget mean:
[min, 50%, 75%, 95%, 99%, Max]

@wendigo nice to see you too on this project ;)
...cheers from OLX/NaspersClassifieds@Berlin

implemented & merged

@aidylewis please, have a try

@zeph Off work today, but will test tomorrow. Thanks.

This is not in master? This commit seems to be available only in issue#37 branch.

@subwiz I merged https://github.com/goadapp/goad/pull/101 already... I don't get it
and issue#37 's branch is merged too https://github.com/goadapp/goad/pull/99 ...can u pinpoint me the commit u refer to? or make a pull request and I'll review it

I was specifically looking for the -i parameter to compute the percentiles. That seems to be not available in the master?

it's not in the ab (apache benchmark) command line, so I'm asking our contractors to first baseline on that interface and add the extra features as part of a file based configuration (it will include a lot of other things) ...coming soon ;) [a month or two]

Thanks @zeph! Till then we are building branch issue#37 for the percentile computation.

The change that shows latencies in issue#37 corresponds to goad 1.2.0. That version is already very different than master/2.x. It would be a pity to lose this change because averages are often misleading. Would it be possible to incorporate that change back into master?


Remove dead function from the agent command

:wave:
Roaming around the codebase I noticed that the mustfunction in the agent.go file looked pretty dead so I took the freedom to remove it :smile:

You passed the secret test @dottorblaster :smile: ...
Thanks :-)!


Access to intermediate HASH data is need to support ISO7816-8 "5.5 HASH operation"

EVP_MD_CTX_get0_md_data is not deprecated in 3.0.1 but it returns a pointer to hash structures which are deprecated, such as SHA_CTX and SHA256_CTX in our case. I believe this is an oversight.
ISO/IEC 7816-8 "Identification cards — Integrated circuit cards — Part 8: Commands for security operations"  "5.5 HASH operation" says:
"The HASH operation initiates the computation of a hash-code by performing:
 either the complete computation inside the card or
 a partial computation inside the card (e.g. last round of hashing)."
The above requires the intermediate hash data to be accessible to be sent to the card so the card can do last round of hashing.
OpenSC has used OpenSSL hash routines for SHA1 and SHA256 with cards that can do: "a partial computation inside the card (e.g. last round of hashing)."  in previous version but needs a way to get access to at least the SHA256_CTX in 3.0.1.
A proposed patch is to copy the SHA_CTX and SHA256_CTX definition and other defines is here:
https://github.com/OpenSC/OpenSC/pull/2506/commits/1f3df786738431fa00fab23a6a830f32eb3e24e0
But is based on the structures not changing in future versions of OpenSSL.
Can the intermediate hash data for at least SHA256 not be deprecated or made available in some other way?
Thanks.

I proposed solution is to move two #ifndef OPENSSL_NO_DEPRECATED_3_0 statements in sha.h This would then leave the SHA_CTX and SHA256_CTX defined in future versions.
OpenSSL-3-sha.diff.txt

Yes. For the SHA256 case the pseudo code does:
        md = EVP_get_digestbyname("SHA256");
	mdctx = EVP_MD_CTX_new();
	EVP_DigestInit_ex(mdctx, md, NULL);.
       md_data =  EVP_MD_CTX_get0_md_data(mdctx); /* gets pointer to SHA256  parameters */
       EVP_DigestUpdate(mdctx, in, in_len);

https://github.com/OpenSC/OpenSC/commit/1f3df786738431fa00fab23a6a830f32eb3e24e0  is a patch to OpenSC to update the code to work with OpenSSL 3.0 as well as 1.1.1 and LibreSSL.
From the SHA256_CTX the code uses:  SHA_LONG h[8];  SHA_LONG Nl, Nh; and unsigned int num;
(the code could also use SHA_LONG data[SHA_LBLOCK]; but the code calculates from its input.)
This use of the SHA256_CTX data is in the patch at: https://github.com/OpenSC/OpenSC/commit/1f3df786738431fa00fab23a6a830f32eb3e24e0#diff-887c1648a5462ce3cbbcef62689ecd3c7fe57fa0e748ee713a36bd1dfce33cfeR3324-R3346
The code does go on to call EVP_DigestFinal_ex(mdctx, out->hash, &md_out_len); for debugging and to verify that the hash created by the card matched the software hash. The security requirement is the smart card will create the last round of the hash.
I did not write the original code and do not have the smart card to test it. But I did write the patch. OpenSC developers are concerned on how to handle this smart card if this data will not be available in future versions of OpenSSL.
It is also not clear if SHA_LONG Nl, Nh;  and unsigned int num; are really needed or could be derived from the size of the input data and HASH.  But the SHA_LONG h[8]; are needed to seed the card with the intermediate data.

So, if I understand correctly, what you want is to be able to get a blob of some sort that represents the current state, and to be able to feed that same blob back to that same algorithm (card) at a later time to continue the digest operation. Correct?

No - that is not what is being asked for. This is not save-state, restore-state to the same implementation. It is extract the internal state from the algorithm in one implementation and pass that internal state off to another implementation - i.e. one implementation is openssl (off card) and one is something entirely different (on card) and we are passing state between the two.
The "solution" should not be to expose internal structures - it should be to have a defined way to export in a portable format the state without using the internal OpenSSL implementation structure.
Note that this sort of usage is strictly not permitted in a FIPS140 context.

It is extract the internal state from the algorithm in one implementation and pass that internal state off to another implementation

Yes, that is what I am asking for. OpenSSL does the first part, and the card does the final round.

The "solution" should not be to expose internal structures - it should be to have a defined way to export in a portable format the state without using the internal OpenSSL implementation structure.

Then what is the purpose of EVP_MD_CTX_get0_md_data?
It is not deprecated in 3.0.1 and returns a pointer to a SHA256_CTX. when using "SHA256" and SHA_CTX when using "SHA1". (gdb of a test program shows the "H0" matches the initial vector for SHA256 or SHA1.)
Call it what you want, the intermediate SHA256 "H" numbers are needed and could be passed as some blob. They just happen to be the first part of the SHA256_CTX after a call to EVP_DigestUpdate. The Nl and Nh are also used by the implementation, but (I suspect) they can be derived from the input data. The OpenSC software then send any remaining data and values need for the card to do the "5.0 PREPROCESSING", "5.1 Padding the Message" and "5.1.1 Padding the Message" as found in" https://nvlpubs.nist.gov/nistpubs/FIPS/NIST.FIPS.180-4.pdf
I would like to suggest that the "solution" use a SHA256_CTX like structure even if the internal OpenSSL implementation structure is something else.

Note that this sort of usage is strictly not permitted in a FIPS140 context.

That may be the case, but in this case we are relying on ISO/IEC 7816-8 "Identification cards — Integrated circuit cards — Part 8: Commands for security operations" "5.5 HASH operation" calls for: "a partial computation inside the card (e.g. last round of hashing)."

Then what is the purpose of EVP_MD_CTX_get0_md_data? It is not deprecated in 3.0.1 and returns a pointer to a SHA256_CTX. when using "SHA256" and SHA_CTX when using "SHA1". (gdb of a test program shows the "H0" matches the initial vector for SHA256 or SHA1.)

It was not deprecated by mistake. It should have been. We cannot in any reasonable way support its operation with provider implementations.

The "solution" should not be to expose internal structures - it should be to have a defined way to export in a portable format the state without using the internal OpenSSL implementation structure.

So basically, a "blob" with the well defined structure and contents...  I agree with that, and thought I wrote my previous comment to include that possibility.  Looking back, I can see that wasn't quite clear.

Note that this sort of usage is strictly not permitted in a FIPS140 context.

... and considering that there is no well known structure for this sort of thing, it's currently a bit hard to do that anyway.  Of course, nothing stops us from inventing something (and oh joy, specify it in modern ASN.1?), but that's quite an effort...

Not sure why it has to a blob. Why can't we just have params for the individual fields?

Not sure why it has to a blob. Why can't we just have params for the individual fields?

No reason why not ...  "blob" was metaphorical rather than specific

It looks like in 3.0.1 EVP_MD_CTX_get0_md_data is not deprecated, but does return NULL. So my proposed fix from above is not complete.
Since the OpenSC code has control of the calling of EVP_DigestUpdate, it can send a multiple the SHA input so that no data is held over for the  "final" round, thus only the array of "H" values are needed in the "blob" and not a full structure.
If this can not be easily done in OpenSSL, OpenSC has some other options like write a custom provider that can return the data. Or OpenSC could use some other implementation of SHA256 for use for this one card that supports the hash on the card. Or not support this one card if using OpenSSL.
The use of the hash on the card is used to create a signature. If I understand your FIPS140 argument, the verification of the signature is where the hash should not expose the internal data or accept partially provided input. But in our case the creation of the hash would only provide a valid signature if both the software and final round on card results have done the has correctly.

Then what is the purpose of EVP_MD_CTX_get0_md_data?

The purpose is for a (now legacy) backend implementation to access the backend specific data from a EVP_MD_CTX.  It's now legacy, because this isn't at all how things work with providers, which is why you're getting NULL, 'cause providers keep those structures very much more to themselves.
Trying to use this data for higher level purposes is quite a hack, and depends on specific backends; for example, an engine could implement SHA1 or SHA256 using a completely different structure for its own contexts...  you cannot assume that it's a SHA_CTX or SHA256_CTX as OpenSSL used to define them.
In other words, using the contents of what is returned from EVP_MD_CTX_get0_md_data is undefined in all places except the backend that governs those contents.
That being said, previous comments still apply.  It can be made possible to extract relevant bits of data in some sort of supported manner.  This would be a new feature.

This is also needed by libtpms: https://github.com/stefanberger/libtpms/issues/356#issuecomment-1447151135


Release: feature/ProcedureIdValidation

Release: feature/ProcedureIdValidation

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: aniruddha-k:white_check_mark: agnelpraveen:white_check_mark: shalini-peeka:x: SHPEEKASHPEEKA seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add section to README about testing with toolchains

@swift-ci please test

Looks like my bot permissions haven't gone through yet. Someone else want to kick this off for me. Thanks!

@swift-ci please test


ModuleNotFoundError: No module named 'app'

when I am running
>alembic revision --autogenerate -m "train_stations"
I am getting below error

(venv) D:\Codes\fastapi-trains>alembic revision --autogenerate -m "train_stations"
Traceback (most recent call last):
  File "C:\Users\rdhoo\AppData\Local\Programs\Python\Python39\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\rdhoo\AppData\Local\Programs\Python\Python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "D:\Codes\fastapi-trains\venv\Scripts\alembic.exe\__main__.py", line 7, in <module>
  File "d:\codes\fastapi-trains\venv\lib\site-packages\alembic\config.py", line 559, in main
    CommandLine(prog=prog).main(argv=argv)
  File "d:\codes\fastapi-trains\venv\lib\site-packages\alembic\config.py", line 553, in main
    self.run_cmd(cfg, options)
  File "d:\codes\fastapi-trains\venv\lib\site-packages\alembic\config.py", line 530, in run_cmd
    fn(
  File "d:\codes\fastapi-trains\venv\lib\site-packages\alembic\command.py", line 212, in revision
    script_directory.run_env()
  File "d:\codes\fastapi-trains\venv\lib\site-packages\alembic\script\base.py", line 490, in run_env
    util.load_python_file(self.dir, "env.py")
  File "d:\codes\fastapi-trains\venv\lib\site-packages\alembic\util\pyfiles.py", line 97, in load_python_file
    module = load_module_py(module_id, path)
  File "d:\codes\fastapi-trains\venv\lib\site-packages\alembic\util\compat.py", line 182, in load_module_py
    spec.loader.exec_module(module)
  File "<frozen importlib._bootstrap_external>", line 790, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "migrations\env.py", line 5, in <module>
    from app.core import settings
ModuleNotFoundError: No module named 'app'

@skb1129 could please help to where I am doing sorry I am newbee to python

@Dhoot This seems like a problem with your virtual env setup.

@Dhoot I used to have this problem from time to time, so all I did to fix this was to export PYTHONPATH=. environment variable or use it before the command itself like this:
PYTHONPATH=. alembic revision --autogenerate -m "train_stations"

Closing this due to inactivity.


tokenPass & zora edition naming updates

@iainnash here is the pull request for:

changing all references to tokenPass to curationPass
changing all references to "ZORA editions" to "ZORA drops". this was a more substantial change, and would require the deployment of a new SVGMetadataRenderer file so am less set on this one needing to go through. however, I think it would make the naming more flexible as the createDrop functionality rolls out of zora soon

lgtm @0xTranqui


Align EventMap item layout with Figma design

Overview
Update the floor layout in the EventMap page to match the design in Figma.
design

Links

Code: https://github.com/DroidKaigi/conference-app-2024/blob/1b0818c053c5f7c0279da90c54df0c61317b25e3/feature/eventmap/src/commonMain/kotlin/io/github/droidkaigi/confsched/eventmap/EventMapScreen.kt#L82
Design: https://www.figma.com/design/XUk8WMbKCeIdWD5cz9P9JC/DroidKaigi-2024-App-UI

I've finished other tasks, so I'd like to start working on this one. 🙋

Thanks! Assigned 👍


Replace marked with markdown-it

I am waiting for resolve this issues:

[ ] https://github.com/markdown-it/markdown-it/issues/28
[ ] https://github.com/jadejs/jade/issues/1870

Markdown was removed.


What do you mean by this ?

<script src="https://utteranc.es/client.js"         repo="ergo9ine/sfdb_tracker"         issue-term="index"         theme="github-dark"         crossorigin="anonymous"         async></script>

SyntaxError: An invalid or illegal string was specified utterances.10cca43d.js:1

utterances doesn't seem to support repo name with underline
https://github.com/utterance/utterances/issues/1#issuecomment-315630670 https://github.com/utterance/utterances/blob/master/src/repo-regex.ts

@ergo9ine thanks for reporting. The repo regex was not permitting underscores. This is fixed in 5a2aed476eb71c6a5c27757c77a91bca1878d1f0.

\w includes underscore. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions#Using_special_characters

Which browser are you seeing the issue in? Could you provide a link to the page?

https://sf2031.com/
It's settled.
Thanks!

I'm getting this same error. My repo name has a dash in it, is that the problem?


Allow specification of global site settings such as Masthead title, footer text, etc.

Perhaps this setting can be specified in a config file in ini format and passed in cmd-line switch.

#9 will provide this


Upgrade tsup & fix node-18 tests

Motivation

Upgrading tsup to ^5.12.8
Fixes node 18 test matrix

awesome! we'll also want to remove this line: https://github.com/coinbase/cbpay-js/blob/main/.github/workflows/unit-tests.yml#L22

Done!


run statsd on device

statsd seems to require node.js to run and a relatively big project, so maybe we should just build a similar simple daemon in python?

statsd seems to require node.js to run and a relatively big project, so maybe we should just build a similar simple daemon in python?

Yes!

Done, thanks @robbederks!


Packages which depend on NodeJS runtime are categorized under Web

URL: https://pub.dartlang.org/experimental/packages/node_interop
This package can not be used in a Web browser so current classification is a bit misleading.
It should really in the Server category.

To highlight what is happening: the project depends on dart:js and dart:js_util, all of which are designed for the browser. We see that during analysis and classify it as "this is only for the web".
@pulyaevskiy: I haven't checked it, can this library be run in a normal Dart VM server app?

@isoos
This library defines bindings for Node-specific APIs (like IO and FS) so it can't even run in a browser, only in Node.
If the way categorization works is by checking dependencies then I'd suggest to look for dependency on node_preamble. The best I can think of at this point...


Whats the max number of IoTHub instances we can have per subscription

The document mentions that we can have 50 IoTHub instances per subscription. But even if I contact Microsoft Support whats the maximum number of IoTHubs that we can have?
Can we have 1000 IoTHubs per subscription, after raising request with MS Support?

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

ID: ec6fc6a5-9bf6-889e-075d-b5b009e67d1f
Version Independent ID: f3d6fa56-a494-fec8-50e3-f8682c90ac42
Content: Understand Azure IoT Hub quotas and throttling
Content Source: articles/iot-hub/iot-hub-devguide-quotas-throttling.md
Service: iot-hub
GitHub Login: @robinsh
Microsoft Alias: robinsh

@akashperfect Thank you for your query . We will check this and update the thread.

Hello @akashperfect , you can see more information about Azure IoT Hub Limits here: https://docs.microsoft.com/en-us/azure/azure-resource-manager/management/azure-subscription-service-limits#iot-hub-limits
"If you need more than 50 paid IoT hubs in an Azure subscription, contact Microsoft Support." -> Creating an Azure Support Request
We will evaluate case by case and help define the maximum number of IoT Hubs allowed for your Azure Subscription.
Thank you!
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Hello @akashperfect


Use fallbacks when link() fails

System.IO.Filesystem's MoveFile() and ReplaceFile() use link/unlink under some conditions to avoid a copy. This fixes two problems with that:

MoveFile()'s fall-back does not recognize Android's error as non-critical, and bails out on the fall-back with an exception. From logs We know that 'strerror_r(android_error)' is 'Permission denied' so it should be EACCES (we already check for EPERM)
While MoveFile() was using a fall-back, ReplaceFile() wasn't.

@monojenkins backport to mono-2019-04

@monojenkins backport to master


fixed slug generation and image preview #137

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Related tickets
fixes #137

License
MIT

Thanks!


DiactorosMessageFactory createResponse does not rewind strem if created with string body

Q
A

Bug?
yes

New Feature?
no

Version
1.7.2

Actual Behavior
$contents = (new DiactorosMessageFactory())->createResponse(200, null, [], "Hello")->getBody()->getContents();

$contents should be 'Hello', instead it is empty, it only works when a rewind is performed
$stream = (new DiactorosMessageFactory())->createResponse(200, null, [], "Hello")->getBody();
$stream->rewind();
$contents = $stream->getContents();

this is actually really hard to do in some situations when the response is used inside a framework wich uses php-http/message
Expected Behavior
$contents = (new DiactorosMessageFactory())->createResponse(200, null, [], "Hello")->getBody()->getContents();

$contents should be 'Hello'
Steps to Reproduce
simpliest way to reproduce is to use the one liner shown above.
actually i am facing this problem together with geocoder-php so when i create the HttpClient in the code i must add this plugin to the chain like
    $rewind = new class implements Plugin
    {
        /**
         * @inheritdoc
         */
        public function handleRequest(RequestInterface $request, callable $next, callable $first): Promise
        {
            return $next($request)->then(function (ResponseInterface $response) {
                $response->getBody()->rewind();
                return $response;
            }, function (\Exception $exception) {
                throw $exception;
            });
        }
    };
    $decoderPlugin = new DecoderPlugin(['use_content_encoding' => true]);
    return new PluginClient(HttpClientDiscovery::find(), [$rewind, $decoderPlugin]);

removing the $rewind from the plugin list the client will no longer work
Possible Solutions
see above

am i correct in assuming that we can close the issue here, as it is not a problem of php-http/message? it seems something with diactoros vs the psr vs php streams being confusing?

The no PSR define if the stream should be rewinded or not. Implementations are not consistent and I dont see a reason for them to be.
If you want to start using a stream, you should never assume its state.


player should not disable the actuators when a robot is penalized

Describe the Bug
When the robot is penalized, it is set to the side of the field and its actuators are disabled. But when the robot is not in a stable position when it is penalized, it cannot reset itself to a stable position and will fall again when the auto referee stops holding it up. The fall results in another penalty and so on.
Expected behavior
The robot should be able to move its actuators when it gets penalized so that it can recover from unstable positions.

This issue is forbidding some teams to get back into the game once they are penalized. It should then be solved as soon as possible.
To the best of my understanding, waiting 5 seconds after a penalty before starting to penalize the robot should be sufficient to solve the problem.

To the best of my understanding, waiting 5 seconds after a penalty before starting to penalize the robot should be sufficient to solve the problem.

Do you mean waiting 5 seconds after a penalty before starting to disable the actuators of the robot?

Yes, that's exactly what I mean. This would allow robots to set their position properly before disabling the actuators.

This should be fixed now in #148.

@timonegk Could you confirm that this solves your problem?

According to https://github.com/RoboCup-Humanoid-TC/webots/pull/148#issuecomment-854718740, I believe this issue can be closed. Feel free to re-open it if needed.


Allow multiple files in --cp

This fixes issue #14.
I have not written or elaborated upon any unit tests. In my opinion, picocli lends itself poorly to testing - I could not find a way to reasonably verify the results without actually executing them, and I think white-box testing such a small change that can easily be verified via debugger or manually is overkill. Nonetheless, if there are ways to easily access a parsed instance of e.g. AstraMethodInvocation I will augment these changes with respective unit tests.
Additionally, I have not touched anything related to versioning, since it does not mention how releases are handled in the contributing guidelines.

Excellent! Thank you for your contribution, @Arraying.
Agreed - I think manual testing is sufficient for this change.
Thanks for mentioning versioning - we work in snapshot and releases are handled at appropriate milestones by the maintainers, so there's no need for any extra versioning changes for contributions. We'll add something to the guidelines.

All looks good, and tested with ' and ;
Merging now. Thanks again, @Arraying!


The build system doesn't know about SkyLake

I'm trying to compile OpenBLAS on a SkyLake desktop and see many compile errors very similar to the one reported in https://github.com/xianyi/OpenBLAS/issues/332 . I can understand that it would take some time to tune the kernels on a newly release hardware but it would be nice to be able to compile first.

I'm adding TARGET=HASWELL to make it compile now. (Tried BROADWELL and SKYLAKE but they don't seem to be acceptable names yet.)

"TARGET=BROADWELL" should work on the "develop" branch (which you will probably want to try rather than the already somewhat dated 0.2.14 release)

I'm pulling from the default branch from github (which should be develop) and TARGET=BROADWELL doesn't seem to work. The commit I'm on is 40a3fed and the full make command line argument I have is
_config=(
  'SMP=1'
  'USE_OPENMP=1'
  'USE_THREAD=1'
  'MAJOR_VERSION=3'
  'NO_LAPACK=0'
  'NUM_THREAD=8'
  'TARGET=BROADWELL'
  'DYNAMIC_ARCH=1'
)

My mistake, seems the fix from #583 only added the cpuid mapping for Broadwell, not a BROADWELL target as such.

That's what it seems like for me after a git grep as well. =)

@yuyichao , I didn't optimize skylake yet. Therefore, please use haswell kernels. make TARGET=HASWELL

Generic binary with DYNAMIC_ARCH recognize skylake as Prescott. Would probably be good enough for now to just fix this first.

Since this issue is about recognising SkyLake rather than optimize for it, I'll close this now.


Is there any document about remi? I cannot find it

As the tittle said, I cannot find any wiki or document about this great project.
And auto-fill function is not operate appropriate in my IDE (visualstudio code), it just cannot identify remi and report
Import "remi" could not be resolvedPylance[reportMissingImports](https://github.com/microsoft/pylance-release/blob/main/DIAGNOSTIC_SEVERITY_RULES.md#diagnostic-severity-rules).
I've installed it by using command pip install remi and python widgets_overview_app.py works fine.
I don't know what's wrong with it. And as the beginner of programmer, I think I do need a document to help my write some code using remi.

Hello @Matt-wzy , unfortunately there is still no documentation about remi. You can look at the examples, the remi source code and of course you can ask me. Additionally, you can use the Editor to design your interface easily.
Kind Redards

@dddomodossola
Thank for your kind reply.
I do facing a problem here.
I'm dealing with the verification code, which I'll save in a specific location such as '/res/a.png' and then show on web. I want when user( only me exactly) click the image on the web the image refreshes (by saving another verification code image to '/res/a.png').
But according to my search, there are two ways to make web browser refresh image rather than refresh whole web page manually to see the changed image: One is change file name (e.g. from '/res/a.png' to '/res/b.png'), and the other is to change suffix of url (e.g. from  '/res/a.png' to '/res/a.png?123', 123 is random). I want to use the second way to make image to refresh, but I don't know how to do it. Simply change image by xxx.set_image("/my_res_folder:a.png?" + randomNumber) don't work because there is no such image in '/res/a.png?123' . I know it won't work because I'm not programed it to do so. But I don't know how to make it happen.

Hello @Matt-wzy ,
Instead of setting up the filename directly, you can use the load_resource function. It will load the content from file and upload it regardless of the filename:
image = gui.Image(gui.load_resource("./res/mine.png"))

Or
Image.set_image(gui.load_resource("./res/mine.png"))
You can find an example at https://github.com/rawpython/remi/blob/master/examples/resources_app.py
;-)


Signing in error

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
Version 3.30
Describe the bug
When trying to sign in getting message:
"The Sign In code was not found or expired. Please try asking the bot to sign in again".
This happens not all the time and it works after trying again.
To Reproduce
Steps to reproduce the behavior:

Go to aka.ms/Gaia
Try to sign in

Expected behavior
To sign in without errors
Screenshots

Fix should go out in this week's deployment

The fix should be out already. Please reactivate if needed

Thanks!!!


Missing docker-compose-joplin-only.yml file

Any hints for Joplin only  configuration.
Leaving away...
-joplin nginx viewweb
-certbot
...doesn*t work for me,
joplin-vieweb-django-joplin-vieweb-1  | Configuring endpoint tcp:port=8001:interface=<IP_ADDRESS>
joplin-vieweb-django-joplin-vieweb-1  | Listening on TCP address <IP_ADDRESS>:8001
^CGracefully stopping... (press Ctrl+C again to force)

That's similar with my initial purpose

Since I use external proxy manager, I had to configure like that

As suggested in the document, disabling nginx & certbot container helped

Moreover, if you use external proxy manager to be connected to localhost, localhost must be added to ORIGINS

Following is a working sample of docker-compose I'm currently using I hope this can help:

version: '3.4'

x-common-variables: &common-variables
   ORIGINS: "'domain.com', 'ext_ip', 'localhost'"

services:
  django-joplin-vieweb:
    image: gri38/django-joplin-vieweb:latest
    depends_on:
      - joplin-terminal-xapi
    environment:
       <<: *common-variables
    restart: unless-stopped
    volumes:
      - joplin:/root/.config/joplin:ro
      - joplin-vieweb:/root/.config/joplin-vieweb
      - certbot_www:/var/www/certbot/:ro
    networks:
      - YOUR-NETWORK
    ports:
      - <PORT_TO_PROXY_MNGR>:8000

  joplin-terminal-xapi:
    image: gri38/joplin-terminal-xapi:latest
    restart: unless-stopped
    volumes:
      - joplin:/root/.config/joplin
    networks:
      - YOUR-NETWORK

#  joplin-vieweb-certbot:
#    image: gri38/joplin-vieweb-certbot:latest
#    restart: unless-stopped
#    environment:
#       <<: *common-variables
#    depends_on:
#      - joplin-vieweb-nginx
#    volumes:
#      - certbot_www:/var/www/certbot/:rw
#      - certbot_conf:/etc/letsencrypt/:rw

#  joplin-vieweb-nginx:
#    image: gri38/joplin-vieweb-nginx:latest
#    restart: unless-stopped
#    ports:
#      - 80:80
#      - 443:443
#    environment:
#       <<: *common-variables
#    volumes:
#      - certbot_www:/var/www/certbot/:ro
#      - certbot_conf:/etc/letsencrypt/:ro
#    networks:
#      - YOUR-NETWORK

volumes:
  joplin:
  joplin-vieweb:
  certbot_www:
  certbot_conf:

networks:
  YOUR-NETWORK:
    external: true

Thanks for you reply.
My YML look quite similar.
I just want to use it in my local network.
Running it in a Proxmox container.
It always stucks on "Listening to TCP adress...."

I think that it's not a problem of joplin vieweb I just tested another configuration by adding server's local ip in ORIGIN so that it can be added in ALLOWED_HOSTS
For example, if I wan't to connect to http://<IP_ADDRESS>:port/joplin, I set ORIGIN like:
x-common-variables: &common-variables
   ORIGINS: "''<IP_ADDRESS>`'"

After then, I could access Joplin-vieweb in local network.
For what I undertand, this can be a solution for you. Is this right?

Hello.
I'm glad you use the project.
I added the docker-compose file without certbot stuff.
@kestr31 you were right.
2 differences:

I write the ORIGINS like that:

x-common-variables: &common-variables
   ORIGINS: "'localhost', '<IP_ADDRESS>'"

I didn't add

networks:
  YOUR-NETWORK:
    external: true

Do you think it's required ? It worked without for me...
But it won't work directly because the django CSRF_TRUSTED_ORIGINS are built from origins like that:
"https://" + origin
So trusted origins will be, in my example:
https://localhost
https://<IP_ADDRESS>
And it should be, I think:
http://localhost
http://<IP_ADDRESS>
Another way it may fail: if you access with a port, let say 8080, the origin would be
http://<IP_ADDRESS>:8080, which is not in trusted origin (:8080 is not !)
@tpbraini couls you please explin your conf.
For testing purpose I can publish an image dedicated to you config.
I think the solution should be to set the origins like that:
For internal access:
x-common-variables: &common-variables
   ORIGINS: "'http://localhost:1234', 'http://<IP_ADDRESS>:1234'"

For external access:
x-common-variables: &common-variables
   ORIGINS: "'https://domain:1234', 'https://<IP_ADDRESS>:1234'"

But usually for extrenal access it will be the standard https port, not written in url, so we should configure like that:
x-common-variables: &common-variables
   ORIGINS: "'https://domain', 'https://<IP_ADDRESS>'"

Let me know about yuor conf, I'll implement the parse of scheme://address[:port]

Hi
I tried some changes: could you try please to use:

the docker compose file here
and you change it:

set your port in
adapt the ORIGINS (the change is I added http:// to the origins). If you use a specific port, for instance if you access with http://<IP_ADDRESS>:8000/joplin, you should set the port in the ORIGINS.
Change the image tage for django-joplin-vieweb:

services:
  django-joplin-vieweb:
    image: gri38/django-joplin-vieweb:2.0.2-4

It works for me, but I still have issues with settings (get conf fails: I'll have a look to that but for the moment let me know if it's better for you).

Works like a charm many thanks !

Just a side note on the over hand, could you add the e2ee config in your custom settings part?
Synchronisation target: Nextcloud (5)
Starting synchronisation...
Deleted remote items: 1.
Created local items: 8. Deleted remote items: 2. Fetched items: 8/50.
(...)
Created local items: 260. Deleted remote items: 2. Fetched items: 261/261. Completed: 02/05/2022 11:35 (102s)
Downloading resources...
/!\ Your password is needed to decrypt some of your data. Type `:e2ee decrypt` to set it.

Again, kudos for your amazing job till there! 😍

Hi @matbgn
Glad it worked for you.
What did you test: docker-compose.yml or docker-compose-joplin-only.yaml ?
And if you used docker-compose-joplin-only.yaml: with django-joplin-vieweb image :latest or 2.0.2-4 ?
About e2ee, I opens a new issue: #6

docker-compose-joplin-only.yaml with django-joplin-vieweb 2.0.2-4 image unfortunately the latest didn't worked for me as I tried also.
Thanks for the issue I will document it ;-)

Ok thanks for the trials.
I'll commit the 20.02.-4 as 2.0 and 20.0.2 and latest when I'll tested in my env.
It also seems the 2.0.2-4 breaks the setting save: does it still work for you ?

Everything work perfectly by me till e2ee issue

Works fine for me as well.
Thank you for your efforts!


[SHIRO-457] Set SecurityManager in createSubject()

When one does not use a static SecurityManager an exception is thrown everytime a new (unauthenticated) subject successfully logs in. See the associated JIRA issue https://issues.apache.org/jira/browse/SHIRO-457

LGTM! can you add a test for this?


improved JSLint compliance

see individual commits for details

The whitespace issues are going to be merged in with navruzm's changes.


Update with changes from forked repository

Adds the updates from the original forked repository so that we can use the most up to date version of knex here and in the core application.

Created PR in error.


Jekejeke Prolog does not permit back-end compiler usage

Please note that the Jekejeke Prolog Runtime Library usage is limited to
"The library can be distributed as part of your applications and libraries
for execution provided this comment remains unchanged." (*)
Unfortunately the intent here is that the Jekejeke Prolog Runtime Library
does not act as a virtual machine of some sort for a programming language
that is not distant of that of the Jekejeke Prolog Runtime Library.
For the upcoming release 1.3.0 I will try to rephrase more precisely what
usage is granted and what not. Now your NOTES.txt still refers to version
1.0.4, but there is no such version anymore. So I suggest removing
jekejeke.pl from the same directory and taking note, that if customers
want a Jekejeke Prolog Runtime Library virtual machine functionality,
for lets say Logtalk, they should contact me at<EMAIL_ADDRESS>.
(*) Example:
https://github.com/jburse/jekejeke-devel/blob/master/jekrun/headless/jekpro/model/builtin/Flag.java
(**) From here:
https://github.com/LogtalkDotOrg/logtalk3/blob/master/adapters/unsupported/NOTES.txt

Ok I have adapted the license, it now reads:

Restrictions
Only to be distributed with programs that add significant and primary
functionality to the library. Not to be distributed with additional
software intended to replace any components of the library.

It will take a while until it is seen everywhere on GitHub,
but Flag.java shows already the clarified license.

Fixed at a67a1ded804cef005de72976e100d40a01d2a371.


Possible config option for Thundestore mod loader

Not sure if this is the right place, but would it be possible to add an option to revert the color changing aspect via the Thunderstore config?

Sure, I'll find out how I can do that and I'll put that in the next release :)

I just prefer the original solid white/grey


googleapiclient.errors.HttpError: <HttpError 403 when requesting https://www.googleapis.com/content/v2/accounts/authinfo?alt=json returned "Insufficient Permission">

Hi
I tried for several days to establish a Content API connection via Python for Google Shopping but I am getting permission denied. I have been advised to post the issue here as it seems to be library specific.
1- I set up a special directory to dumpy two json files. /home/castnow2019_gmail_com/shopping-samples/content. I dumped 2 JSONs files:
client-secrets.json  merchant-info.json
2- Created a different directory that has Python code in /home/castnow2019/google/env/google-shopping/googleads-shopping-samples/python$
3- I ran pip install -r requirements.txt
4- I ran python -m shopping.content.products.list
but I received this error message:
File "/usr/local/lib/python2.7/dist-packages/googleapiclient/http.py", line 856, in execute
raise HttpError(resp, content, uri=self.uri)
googleapiclient.errors.HttpError: <HttpError 403 when requesting https://www.googleapis.com/content/v2/accounts/authinfo?alt=json returned "Insufficient Permission">
how do I solve this issue?

It sounds like the credentials you are using don't have sufficient permission. Would you be able to provide a code sample?

Sure..you mean Python sample code? it is here: https://github.com/googleads/googleads-shopping-samples/tree/master/python
The client-secret.json file gets created automatically by going to this link: https://console.developers.google.com/apis/credentials

@castnow123 Which sample file specifically are you running?
It looks like you are using python 2.7. Do you know which version of the client library you are using?
In addition, please review the instructions here and here and confirm that you've followed the relevant steps for getting credentials.

@busunkim96
I am running list.py using this command python -m shopping.content.products.list
even if I run it using python3 it gives the error.
The client library is v2.1
I shared some screenshots of client-secrets.json after removing sensitive information.
this same service account is in the users section of my merchange center

@busunkim96  Yes, I am using 1.7.11 I setup OAuth2 client ID as in the screenshot attached. I named the json to client-secrets.json.
but I still rceived this error. do you have skype so I can share my screen and show you? did you try to authenticate using python and you are not getting any errors?

Using application default credentials.
Traceback (most recent call last):
File "/usr/lib/python3.5/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/castnow2019/google/env/google-shopping/googleads-shopping-samples/python/shopping/content/products/list.py", line 48, in
main(sys.argv)
File "/home/castnow2019/google/env/google-shopping/googleads-shopping-samples/python/shopping/content/products/list.py", line 29, in main
service, config, _ = common.init(argv, doc)
File "/home/castnow2019/google/env/google-shopping/googleads-shopping-samples/python/shopping/content/common.py", line 142, in init
retrieve_remaining_config_from_api(service, config)
File "/home/castnow2019/google/env/google-shopping/googleads-shopping-samples/python/shopping/content/common.py", line 179, in retrieve_remaining_config_from_api
authinfo = service.accounts().authinfo().execute()
File "/usr/local/lib/python3.5/dist-packages/googleapiclient/_helpers.py", line 130, in positional_wrapper
return wrapped(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/googleapiclient/http.py", line 856, in execute
raise HttpError(resp, content, uri=self.uri)
googleapiclient.errors.HttpError: <HttpError 403 when requesting https://www.googleapis.com/content/v2/accounts/authinfo?alt=json returned "Insufficient Permission">

yes I did all of the above when I wanted to authenticate using service account but that did not work too.

Hi


Deleting pod has an error on k8s v1.6.1

On my k8s v1.6.1 env, I create a pod using "kubectl create -f scbe_volume_with_pod.yml". After the pod is in running status, I delete the pod using "kubectl delete -f scbe_volume_with_pod.yml". Then u "kubectl get pod --show-all" returns "No resources found". But the volume is still mapped to host on storage, and there is an error "Volume not found" in flexvolume log and "error-backend-not-found-for-volume" in ubiquity log.

The detail log is below:
ubiquity-flexvolume.txt

@midoblgsm
looks like that the bug is in line 173
in case of k8s 1.6 it goes to this else and then the lnPath set to wrong location - Instead of mountRequest.MountPath it set to basename of mountRequest.MountPath (split command).
So the slink created to a wrong place.
Instead of  /var/lib/kubelet/pods/0d63c4dd-812a-11e7-b967-005056b9f233/volumes/ibm~ubiquity/PVC-ID it link to /var/lib/kubelet/pods/0d63c4dd-812a-11e7-b967-005056b9f233/volumes/ibm~ubiquity/WWN
Any reason why there is a split on the InPath?  is this needed for Spectrum Scale?

it is fixed, I verified it on v1.5.6, v1.6.1 and v1.7.1.


Forgot Password Feature

Deliverable:
Add a forgot password form having following field

Email Id
Submit Button

After clicking on Submit button, a new page should come up saying An invitation link has been sent on your registered email, please check your inbox for further steps Along with two buttons 1. To go to login page 2. To go to login page

can you assign me this issue under GSSOC 21

Hi, I would like to work on this feature as I have already implemented this on many projects I can easily do that.
under GSSOC 21
Can you assign me this ?
@jackfrost13

@Shubham-Kumar-2000 Please take up one task at a time
@sukhpreet-singh1 you have been already assigned something, please finish that first

So should I make the frontend only Or should I work on both? @jackfrost13

This issue is only for frontend, hence you need to work only on frontend right now.


[Feature] Ability to set output mp3 bitrate as an option

Description
When mp3 output format is chosen, Spleeter should have either an option of specifying a bit rate say 256K or it should default to a high rate. Currently it exports at 64kbps.
Additional information
When I use this command line:
spleeter separate -i audio_example.mp3 -o output/ -c mp3
on colab or on my mac, the output mp3 files default to 64kbps.
I have tried this on various mp3 with as high input bitrate as 256K and the output mp3 is always 64kbps.
I do not see a way (maybe I have not looked carefully) to pass from the command line or  even modify the ffmpeg wrapper.
The mp3 quality is far lower than wav output because of this low bitrate. Please let me know if this is a default setting on ffmpeg install.

Thanks for the suggestion. Indeed you can't not specify bitrate parameters at the moment.  As you suggest it should so we will add it to the next Spleeter release. Meanwhile you can by pass this by using Separator API from Python code directly, where the provided separation method allows to specify a bitrate in addition of the audio codec to use for export.

Thank you for prompt response and the suggested alternate Felix
This open source is going to revolutionize music as more detailed instrument separation models are built by users.
Very happy to see sigsep stalwarts Fabian participating to extend this too in addition to the excellent breakthroughs they have done already with open unmix.
We are living in a time where python open source deep learning and high storage and compute are giving us heavenly gifts
Big thanks
Ravi

On Nov 14, 2019, at 11:10 AM, Félix Voituret<EMAIL_ADDRESS>wrote:
Thanks for the suggestion. Indeed you can't not specify bitrate parameters at the moment. As you suggest it should so we will add it to the next Spleeter release. Meanwhile you can by pass this by using Separator API from Python code directly, where the provided separation method allows to specify a bitrate in addition of the audio codec to use for export.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Faylixe,
(apologize for mistyping your name on iphone initiated message above, as Felix)
Temporarily, I am converting wave to high bit rate mp3 using ffmpeg so I can update in Colab without my own fork (yet).
You guys have really made a breakthrough.
I will share an example of a mix and unmix from a Tamil legend Ilaiyaraja soon and share why I am saying so.
Ravi

The new version is released and have the new bitrate option for separate command.

Thanks I will check this out
Sent from my iPhone

On Nov 21, 2019, at 8:19 AM, Félix Voituret<EMAIL_ADDRESS>wrote:
Closed #95.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Improve DesmosQuery and DesmosMsg with serde tag

Close #21
This PR applies the serde tag to indicate the route instead of DesmosRoute object. In addition, DesmosRoute is removed since it is never used with this change.

Closes #21


[AMBARI-24687] Upgrade Pack Should Allow Source Information

What changes were proposed in this pull request?
Update upgrade-pack.xsd and UpgradePack.java to contain source information.  This change will be consumed in future patches.
How was this patch tested?
Manual testing.  Automated pending.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4066/
Test PASSed.


Upgrade dependencies

GEOS 3.12 -> 3.13beta
GDAL 3.8 -> 3.9
PROJ 9.1.1 -> 9.4.1

Note to self:

Don't build shell on windows as the sqlite symbols conflict
GDAL requires cpp17 headers that's not available on our CI

GDAL 3.10, GEOS 3.13, and Proj 9.5 are all out now... anything I can help with here?
FYI, there's also a change proposed for both GDAL and PROJ to formalise embedding resources (like proj.db) into the libraries based on your existing approach.

@rcoup Not really, we bumped to latest GEOS separately before DuckDB v1.1.1(I think..). We can't really move forward with latest GDAL because the c++17 requirement is not compatible with our CI setup, and ideally we want spatial to be available on older targets.

By "our CI environment" do you mean https://github.com/duckdb/extension-ci-tools ?
The manylinux2014 toolchain has a newer GCC (v10) from Redhat such that it supports C++17, and there's newer GCC (v9/10/11/13) available from the Ubuntu team for 18.04. So should be possible...

The manylinux2014 toolchain has a newer GCC (v10) from Redhat such that it supports C++17, and there's newer GCC (v9/10/11/13) available from the Ubuntu team for 18.04. So should be possible...

CC @carlopi

Discussed with @Maxxen and @samansmink, I think this is something that would be interesting to look at this in general in the next release cycle.
Extensions (eg. spatial) are free to get ahead and experiment with this, but this comes with some cost, and maybe it's better to do this properly on the tooling side.


Why check for Rails gem to be installed?

Believe it or not, our Rails project does not actually require Rails in the Gemfile. Instead, we require the various sub-frameworks manually, because we don't need them all (notably excluding activerecord). Don't ask, it's a long story...
This means that i18n-js 3.0 will not load properly in our project.
It took me a while to track this down, but I finally found that you are checking not only for Rails to be available and the appropriate version to be returned from Rails.version. You are also checking for the rails gem to be listed in the Gemfile. For us, that obviously causes problems.
IMO (and because it would fix our problem), checking for the correct Rails version to be available in the runtime should surely be enough? You don't need to bother where it comes from, just that it's there.
By the way: seeing that that code was apparently inspired by similar code in the jasmine-gem, you might be interested to know that the jasmine-gem removed that code again to fix other problems with the Gem version resolution.

I will prepare a pull request, but wanted to understand this first. Maybe there is some hidden necessity for this code to be in place? :smile:
P.S.: Relevant XKCD reference: https://xkcd.com/1172 :wink:
P.P.S.: This may very well be the underlying issue for #448.

My pull request indeed fixes both issues.
I would be very happy about a merge and a new release. :heart:


[fixed] shouldCloseOnOverlayClick conflict with text inputs.

Fixes #566.
Changes proposed:

shouldCloseOnOverlayClick must only preventDefault() when
the click is really on the overlay.

Upgrade Path (for changed or removed APIs):

None

Acceptance Checklist:

[x] All commits have been squashed to one.
[x] The commit message follows the guidelines in CONTRIBUTING.md.
[x] Documentation (README.md) and examples have been updated as needed.
[x] If this is a code change, a spec testing the functionality has been added.
[x] If the commit message has [changed] or [removed], there is an upgrade path above.

Coverage remained the same at 86.265% when pulling dc3f140aeb07d91e4c95e257e93e9828ed060c65 on diasbruno:fix/shouldCloseOnOverlayClick-conflict-with-text-inputs into 99f89335030b816f7d029c729b0fd5da930a4063 on reactjs:master.

Released v3.1.5.


Cannot change python interpreter

os: win10
vim: gvim 7.4.1848
syntastic: current latest version.
python: I installed python 2.7 in c:\python27 first and then python 3.5 in c:\python35-32
Problem:
I have this setting in my _vimrc:
let g:syntastic_python_checkers = ['pylint']
now the default interpreter/checker for python is python 3/pylint.
I want to change it to python 2/pylint, so add this:
let g:syntastic_python_python_exec = 'c:/Python27'
But this doesn't work at all.
Another way of setting(In powershell, py -2 -m pylint *.py works fine):
let g:Syntastic_python_pylint_exec = 'py'
let g:Syntastic_python_pylint_args = ['-2', '-m', 'pylint']
Still not work.

OK, figured it out myself, we need these 2 sentence:
let g:syntastic_python_checkers = ['pylint']
let g:syntastic_python_pylint_exe = 'py -2 -m pylint'

Setting the exe works, but only for pylint.  The general solution is to use something like virtualenv or pyenv, create environments for Python 2 and Python 3, install the checekrs you want, and run Vim from said environments.  (I have no idea how you might do that from Windows, but I trust it's possible...)

let g:syntastic_python_python_exec = 'c:/Python27'

On a side note: you need to point the exec to the checker's binary, not to its base directory.

In powershell

You might also want to read :h syntastic-powershell.

@lcd047 point the exec to the checker's binary takes no effect, too. I don't know why.

You might also want to read :h syntastic-powershell

here I just run commands in system environment, which involves nothing with syntastic, but it did remind me of using vim from powershell.
Thanks for your suggestion, I might find out how to integrate gvim with virtualenv.

point the exec to the checker's binary takes no effect, too. I don't know why.

I didn't say it would solve your problem in this particular situation.  g:syntastic_python_python_exec only has effect on the python checker, it has nothing to do with the pylint checker.  No, what I'm saying is, pointing exec to a directory has no chance to work, for any checker.


DOC: clarify installation

cc @stephanie-r-jones let me know if something else needs to be clarified
closes #257

I decided to leave the --upgrade in there because it should work even if it's a first time installation

I decided to leave the --upgrade in there because it should work even if it's a first time installation

Codecov Report

Merging #259 (98cba39) into master (417893f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #259   +/-   ##
=======================================
  Coverage   52.93%   52.93%
=======================================
  Files          23       23
  Lines        2747     2747
=======================================
  Hits         1454     1454
  Misses       1293     1293

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 417893f...98cba39. Read the comment docs.

Codecov Report

Merging #259 (98cba39) into master (417893f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #259   +/-   ##
=======================================
  Coverage   52.93%   52.93%
=======================================
  Files          23       23
  Lines        2747     2747
=======================================
  Hits         1454     1454
  Misses       1293     1293

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 417893f...98cba39. Read the comment docs.

Merged as recommended by @cjayb in #260

Merged as recommended by @cjayb in #260


FR: Save images to predefined folder

Hi
it would be great if one could just capture individual frames to a predefined folder like that can be done with VLC. It isa bit tedious to constantly enter individual file names for saving single frame.

I've added File->Save->Frames to Folder to v0.9.4 beta currently on sourceforge.net (https://sourceforge.net/projects/mrv2/files/beta/).
The first time you call the function, it will ask you the parameters and the directory and name to save the file as usual.  However, the next times you invoke it, it will automatically save the image without popping any options or file name.
To change the file name or options that it is saving under, you should go to File->Save->Single Frame as usual and then go on to keep using File->Save->Frames to Folder.

Thanks, it crashed when I did not provide a filename.
How do I set the frame number?

I tdoes not work on Windows 11, as far as I can tell.
Exr trows error
ERROR: [save] D:/.TMP/test.0001.exr: Unsupported video

It does not work on Windows 11, as far as I can tell.
Exr trows error
ERROR: [save] D:/.TMP/test.0001.exr: Unsupported video

This is the error you get when you don't have Annotations on.  Movie files are encoded in YUV which EXRs do not support.  Turning off Annotations on makes the frame be captured in RGBA_F32 which is compatible with EXRs.

Thanks, it crashed when I did not provide a file extension.

Thanks, I'll check that, as it is likely a bug.

Thanks, it crashed when I did not provide a file extension.

Thanks, I'll check that, as it is likely a bug.

I checked both File->Save->Single Frame and File->Save->Frames to Folder.  With either one when I save with no extension, I get the following error:
ERROR: [cback] File extension cannot be empty.
You will need to give me step by step instructions to reproduce it.

Also it keeps opening the folder browser, then I am manually saving every time I pres "alt-s" which I set to be save single frame. I do not see it saving in hte background like VLC does.

You should not set the shortcut to "Save Single Frame" (which does exactly what it says it does), but to "Save Frames to Folder".

Also it keeps opening the folder browser, then I am manually saving every time I pres "alt-s" which I set to be save single frame. I do not see it saving in hte background like VLC does.

You should not set the shortcut to "Save Single Frame" (which does exactly what it says it does), but to "Save Frames to Folder".

Thanks for the addition, that was it.


test: analyse Analytics spec

Summary by CodeRabbit

Bug Fixes

Updated test specification to focus on the "Analytics" feature instead of the "Fork Template" functionality.

/ci-test-limit-count run_count=100

/ci-test-limit-count run_count=100


Possible strategy for LRU-light

When the in-memory dictionary is full and we add something new (either new kv pair or something pulled from disk) we currently find the largest element in the dict and send it to disk.
This is bad in a few ways

We do this all the time once we fill up our in-memory cache
We find the largest element, this may not be the right thing to push down.  We may want the least recently used
We currently do this through a global max(values, key=nbytes) call which might be expensive if we have many keys, particularly we if we do it at every insertion.

A decent cheap solution would be the following.
Have a dictionary, inmem as we do now.  Have a new set seen-recently.  Whenever we touch a piece of data we add its key to seen-recently and update our total-bytes-in-memory counter.  When this counter goes above available-memory we flush all values corresponding to keys not found in seen-recently.  We then clear seen-recently.
That is to say that the policy becomes that you can stay in memory if you've been used since the last clearing out.
This is amortized cheap, happens less frequently, is cheap to maintain, and easy to build.

One could think of this approach as a generational LRU with history depth equal to 1.

I suppose that this fails for the first round, when all keys are in seen-recently.

I implemented LRU with a heap in https://github.com/mrocklin/chest/commit/24210a5980a2c3691274db1432536921e9d868b5


How can I convert frame while processing record to wav  mp3 file, please help me

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Not enough detail to address


H2 Database to MySQL

Does somebody could tell me how to make Traccar generating MySQL databases instead of H2 database??. Of course I read the help on the Traccar web page (http://www.traccar.org/docs/mysql.jsp), but I need some extra support ;) , I mean, I would like to understand how to solve this:
NOTE: Traccar will create tables for you, but you need to create database with selected name by yourself.
I would be really grateful, Im working in my final project in University, and I am too worried about it :/
Grettings to everyone from Ecuador :)

he intentado sin éxito hacer que trabaje con la base de datos mysql,
luego de instalar mysql, crear el usuario en mysql y configurar el archivo traccar.cfg y le doy iniciar traccar y me sale este aviso:
traccar may have failed to start.
ayuda porfavor!!!

@andyguz08, please create a new thread. Also, check log file (wrapper.log) for a detailed error message.


changes script path on notebook for training

I have changed script path on notebook for training , please review the code


Add instruction to upload binary in release

Some users prefer not to run docker on manager nodes to reduce attack surface. Providing binary would help their deployment.
Signed-off-by: Dong Chen<EMAIL_ADDRESS>

sorry to make life harder for you, but i noticed that in your release for 1.2.5 you made a tar named
swarm-1.2.5-linux-x86_64.tgz
that itself contains
swarm-1.2.5
If you look at what releases in docker/docker do, they have a versioned tar:
docker-1.12.2.tgz
that contain unversioned binaries like
dockerd
I think this is preferable, otherwise any automation scripts will also probably have to rename the swarm binary.

@vincentwoo Agree. The instruction here is correct. The one I uploaded had binary with version in name wasn't following this instruction. I have replaced it with a new one.

LGTM


Publish actual version

The latest version of the npm module has been published 2 years ago.
Why do you not publish new versions? You have implemented step labels and etc but it's available from npm...

Because I've haven't had a RN dev setup for that period of time so its hard for me to QA the release before pushing it. If someone wants to test the current master before a release feel free!


[RESOLVED] High ICMP ping response time

Problem started at 11:52:14 on 2023.04.25
Problem name: High ICMP ping response time
Host: MF-FG
Severity: Warning
Operational data: Value: 196ms
Original problem ID: 19956
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23132&eventid=19956

Problem has been resolved in 7m 0s at 11:59:14 on 2023.04.25
Problem name: High ICMP ping response time
Host: MF-FG
Severity: Warning
Original problem ID: 19956


strange inverse color/highlighting after reverse search in terminal

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.93.0
OS Version: MacOS Sonoma 14.6.1

Steps to Reproduce:

use ctrl+R to perform reverse search in terminal
type search keywords(matched command will pop up)
press left arrow key in keyboard
some text in the command will be highlighted/inversed colored.
even after exit the reverse search, the command line will still be highlighted.

I cannot reproduce. could you pls run the command Developer: Toggle Screencast Mode and record a gif?

looks like all of that text is getting selected. do you have any interesting settings or keybindings?

I don't explicitly set keybindings. This issue happens randomly and only happends when I am connecting to remote linux server with Alma Linux 9. Do you think this cound be caused by the remote server's setting?

Looks like a remote window and local echo is messing with things. You can disable this feature with "terminal.integrated.localEchoEnabled": "off"
Related https://github.com/microsoft/vscode/issues/126209, https://github.com/microsoft/vscode/issues/119103


Update README install instructions for troubleshooting

Description

Updated installation instructions with troubleshooting for common errors.
Type of change

Documentation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4325

Relevant Lines:
12211

💛  - Coveralls


Develop OpenAPI Export for API Definition

In order to make it easy to standup REST APIs atop workers, Golem has a concept of "API Definition", which consists of a number of routes, with binding types, and implemented by the Worker Gateway (worker bridge + worker services).
Currently, two binding types are supported in the Worker Gateway:

Default. In this case, each binding is given by a snippet of Rib, which calls into a function that has been exported from the worker handling the request. The Rib script serves to massage the JSON input into whatever form is required by the function being invoked; and then again, to massage the output of the function into whatever from is required by the JSON response.
File Server. In this case, the endpoint serves static files, which are determined by a Rib script.

Rib is a fully type-inferred language, and type information is available as metadata on existing REST APIs. This type information describes the type of input and output as a type in WASM's Component Model--see WIT for a sense of what these types can represent.
The Rib interpreter has a canonical mapping of values to Component Model types. It's deterministic and predictable and somewhat documented.
For all these reasons, it is possible to know exactly the input JSON and output JSON of every endpoint.
This ticket is to add a new REST API that can export an API Definition into a valid OpenAPI Specification, preserving the request and response types. Separately, the OpenAPI Specification needs to correctly represent security and CORs, which are features of the Worker Gateway. Any client library generated from the OpenAPI Specification needs to be able to work correctly with the real REST API.
/v1/api/definitions/{id}/version/{version}/export

Moreover, the ticket also calls for use of the OpenAPI Spec to generate a Swagger UI; and for a new binding type in the Worker Gateway, which can trigger the Worker Gateway to serve the Swagger UI at the specified endpoint.
Together, all these features will provide very good support for consuming REST APIs built with the Worker Gateway.
To close out this ticket, the following tasks must be completed:

[ ] Full and lossless conversion of any API Definition into a valid OpenAPI Specification.
[ ] Tests at the unit and integration level which demonstrate the correctness of pieces and the whole.
[ ] System tests that generate client libraries from the OpenAPI Spec, and interact to verify the client library works.
[ ] Manual testing for complex scenarios.
[ ] Full documentation, submitted separately in a PR to the docs repository in the golemcloud organization.
[ ] A new binding type for routes called "Swagger UI", which will use serve Swagger UI at the specified route, which are generated during API Definition update (if necessary) and stored for subsequent serving of the endpoint.

/attempt #1178

/attempt #1178

/attempt #1178

/attempt #1178

/attempt #1178


December-2021-take-all

Preparation for December 2021 Shakeup
Preview

Change Log:

feat: take ones for december 12, 2021
feat: moved email sign to bottom of index.html page
feat: re-implemented see all changes button
feat: generated 202112shakeup_en_es.docx from PDF
feat: generated English and Spanish take all PDFs from initial combined PDF.
feat: new mybus-dec-2021 - lines.csv from the https://docs.google.com/spreadsheets/d/1kUy2hog330BVZtm7BSGd-cHVehV54gX8BY5PQSCFMp4/edit#gid=1090510019 this still needs to be verified because i didnt have access to the schedules.pdf
feat: commented out parts of code where other language translations didn't exist

This PR will close #115

@albertkun Oh noes, did I link you to the wrong Google Sheet??  This is the correct one: https://docs.google.com/spreadsheets/d/1EUjBZsCiDYzu3r83Pg3oXCv5_Y0A1wxMLm0QY8NA3yw/edit#gid=1850985258
41 new PDFs have been put here: https://www.dropbox.com/home/Digital Services/Timetables
There will be 70 new timetables.
I added a comment about one minor change - the shakeup goes into effect on December 19th, not the 12th.


test: Add the unit test for chatbot footer component

Description
Added the Unit Test for the Chat Bot Footer with simple tests on buttons, icons, and text.
Closes #158
What change does this PR introduce?
Please select the relevant option(s).

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (changes to docs/code comments)
[x] Unit Test

What is the proposed approach?
Added the unit test for Footer component with jest and react.
Checklist:

[x] The commit message follows our adopted guidelines
[x] Testing has been done for the change(s) added (for bug fixes/features)
[x] Relevant comments/docs have been added/updated (for bug fixes/features)

@tanasebutcaru Hi can you just check the test once and let me know if any changes required

@tjtanjin I have replaced the buttons with the mock buttons

@tjtanjin Sure i have made the changes

@tjtanjin Thank you for guiding me through this


adding a "scroll-to-top button"

Currently, users find it difficult to scroll back to the top of the page. To improve the user experience and make the website more user-friendly, I suggest adding a scroll-to-top button. This will enhance the website's interactivity and provide a more seamless navigation experience..
here is the current photo of the webpage:

Assign this under the Hacktoberfest.

@Luson045 assign this under the hackltoberfest.

okay 👍


All links broken in contents list of Working-at-Wunderkraut/security

http://way.wunder.io/working-at-wunderkraut/security/

Fixed. See https://github.com/wunderkraut/WunderWay/pull/61
Sorry, I forgot to auto-close this issue.


Implement --log-config for all server CLI commands.

This PR implements --log-config on:

tiled serve directory
tiled serve catalog
tiled serve config (where it would override the config)

(It skips tiled serve demo, which is intentionally bare bones and does not accept many of the other options common to the others.)
Previously --log-config was accepted by tiled serve directory only, and it was then ignored. :facepalm:  Expand for interactive tests demonstrating that we got it right this time, for each of the above.

$ tiled serve catalog --temp --log-config example_log_config.yml
Creating catalog database at /tmp/tmpbb9mpn3_/catalog.db
Creating writable catalog data directory at /tmp/tmpbb9mpn3_/data

    Navigate a web browser or connect a Tiled client to:

    http://<IP_ADDRESS>:8000?api_key=0596b6db450d8d128197cf8f5e8b00d8e50f0c32788dd40b5245c8ab828e6af0

[2024-04-25T15:13:26.117Z] [-] INFO:     Started server process [137107]
[2024-04-25T15:13:26.118Z] [-] INFO:     Waiting for application startup.
Tiled version 0.1.0a117.dev57+g0cd0407b
[2024-04-25T15:13:26.120Z] [-] INFO:     Application startup complete.
[2024-04-25T15:13:26.121Z] [-] INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)

$ tiled serve directory /tmp/data --log-config example_log_config.yml
Creating catalog database at /tmp/tmpfbf9dqpk/catalog.db

    Navigate a web browser or connect a Tiled client to:

    http://<IP_ADDRESS>:8000?api_key=f760c7b4b8dff957b22aa69559fe77b301f20334a42bd7079041bf5103526e26

[2024-04-25T15:15:03.062Z] [-] INFO:     Started server process [137221]
[2024-04-25T15:15:03.062Z] [-] INFO:     Waiting for application startup.
Tiled version 0.1.0a117.dev57+g0cd0407b
[2024-04-25T15:15:03.066Z] [-] INFO:     Application startup complete.
[2024-04-25T15:15:03.066Z] [-] INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
[2024-04-25T15:15:03.177Z] [9f868d706a9c17a7] INFO:     <IP_ADDRESS>:49660 - "GET /api/v1/ HTTP/1.1" 200 OK
[2024-04-25T15:15:03.269Z] [3a0e3028f590fb47] INFO:     <IP_ADDRESS>:49660 - "GET /api/v1/metadata/?include_data_sources=false HTTP/1.1" 200 OK
Server is up. Indexing files in /tmp/data...
[2024-04-25T15:15:03.280Z] [283a08dd56c8a025] INFO:     <IP_ADDRESS>:49660 - "DELETE /api/v1/nodes/ HTTP/1.1" 200 OK
[2024-04-25T15:15:03.604Z] [b74009411c58cdb1] INFO:     <IP_ADDRESS>:49660 - "POST /api/v1/register/ HTTP/1.1" 200 OK
Indexing complete.

 TILED_SINGLE_USER_API_KEY=secret tiled serve config example_configs/small_single_user_demo.yml --log-config example_log_config.yml
Using configuration from /home/dallan/Repos/bnl/tiled/example_configs/small_single_user_demo.yml
[2024-04-25T15:16:59.403Z] [-] INFO:     Started server process [137639]
[2024-04-25T15:16:59.403Z] [-] INFO:     Waiting for application startup.
Tiled version 0.1.0a117.dev57+g0cd0407b
[2024-04-25T15:16:59.404Z] [-] INFO:     Application startup complete.
[2024-04-25T15:16:59.404Z] [-] INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)

This PR also adds example_log_config.yml to the repo root

Testing for thoroughness and editing the code:

Test with tiled serve catalog and we see the correlation-ID being inserted

Insert something into the string template and test with serve directory, cause an error

Test with serve config using the toy auth config and it works

All of these work find and wrap the error messages successfully, I'm going to merge it.


SUBMARINE-254. Workbench server support cluster mode

What is this PR for?
Workbench Server is mainly for Submarine Workbench WEB is mainly for algorithm users to provide algorithm development, Python/Spark interpreter operation and other services through Notebook.
The workbench server allows multiple workbench servers to form a service cluster by integrating the cluster components.
The goal of the Submarine project is to provide high availability and high reliability services for big data processing, algorithm development, job scheduling, job scheduling, model online services, model batch and incremental updates. In addition to the high availability of big data and machine learning frameworks, the high availability of Submarine Server and Workbench Server itself is a key consideration.
Design Doc: https://docs.google.com/document/d/1Ax6FQ5CAP-jowm2_Mp2r5kc9r7s1bkFLRfzvvbm5Wzc/edit#
What type of PR is it?
[Feature]
What is the Jira issue?

https://issues.apache.org/jira/browse/SUBMARINE-254

How should this be tested?

CI Pass
TestWorkbenchClusterServer.java

Screenshots (if appropriate)
Questions:

Does the licenses files need update?  No
Is there breaking changes for older versions?  No
Does this needs documentation? Yes

CI Pass: https://travis-ci.org/liuxunorg/hadoop-submarine/builds/601262192


Reduce library size

Hello !
First of all I would like say thank you to all who participate to the development of this great library which work very well.
My issue is not really about the functional aspect of the library but about the size of it. I use the library with an ATMEGA328P and I also use a CAN library. My program is based on the DWM1000Ranging and the program is really too big. I dont know how I could reduce the size of my program by commenting or deleting some unused function of the library. Some ideas ?
Thanks
Matt

Hi @mattweb67 ,
I have two version of library. One for tags, seconds for anchors.  In anchors version I deleted/commented out code for Tags and vice-versa. Try to avoid printf -they're really memory-consuming. I commented debug also (first text with device address & ID, channel etc,)

Hi @prideofisland
Good ideas I try that now, currently I use 110% of the flash memory of the ATMEGA :/
Thanks

@mattweb67 This is a general question. Please ask on https://stackoverflow.com/ or https://forum.arduino.cc/ . Thanks


Added validate on mount option to form fields.

Added prop to form fields. This prop specifies if, the field should be validated onMount, or after it is touched.
Currently waiting for #20

@miq-bot remove_label unmergeable

@miq-bot assign @karelhala


Shared gamma for BatchNorm

I was wondering if it is possible to have multiple BatchNorm layers that use the same gamma variable?
I couldn't find how to define the variable name similar to what is done for convolution layer  here https://github.com/dmlc/mxnet/issues/557

you can do the same thing for batchnorm

How?
I don't see any arguments except for the data.
Can you give me a code example?

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Change default js version to auto

Description:
Change default js version to auto
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#4150
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

It's happening :)


Fixed Django1.8 support (template panel still requires a Django engine t...

...o be defined)

Will this pull request be merged when 1.8b1 is released?

Anyone know when this will go in?

@aaugustin What do you think of creating a specific file to handle the different imports/functions based on the versions of Django? It should help restrict all the if/else logic into a single file and keep the rest of the project ignorant of it.

@tim-schilling Yes, something like debug_toolbar.compat would be useful, especially when removing support for older Django versions.

@tim-schilling Do you intend to work on this within the next few days?

I might not get to it until late this week or this weekend, but yes.

I won't have time before the weekend either.
It would be good to upload a 1.8-compatible release as soon as possible. I suggest the first of us who has time writes the patch and the other reviews it :-)

As of 22473d36346901b3320f1bf25f4f602d8ded6d04 tests pass on Django 1.8 beta 1.


Support Token Credential Backend Roles management

Vault 0.5.2 introduced Token Credential Backend Roles:

Roles can now be created in the token credential backend that allow modifying token behavior in ways that are not otherwise exposed or easily delegated. This allows creating tokens with a fixed set (or subset) of policies (rather than a subset of the calling token's), periodic tokens with a fixed TTL but no expiration, specified prefixes, and orphans.

Docs are here https://www.vaultproject.io/docs/auth/token.html#auth-token-roles-role_name-

We would welcome a PR for this functionality, although it is not on our near term roadmap.

I considered Token Roles, but decided to use AppRoles after all, so I don't need Token Roles anymore. Should we close this issue then?

Token roles appear to be one of the recommended ways of Vault integration with Nomad: https://www.nomadproject.io/docs/vault-integration/index.html