text:
Cashaddress prefix

Hi Guys, really hope I'm doing this in the right place.. apologies if im not!
We are trying to use cashaddress more frequently to onboard new comers to BCH, there is 1 issue and 1 request here.
Issue: most wallets do not yet support the cashaddress prefix, can we have an option to toggle that off until they do? for example, bitcoin.com wallet can not sweep a cashaddress wallet at the moment.
Request: Can we have a few different options for background graphics for the paper wallet? would be nice to have a few options, especially with the holidays coming up! :)
You guys rock!!

So, I would like to keep the prefix. This is being used to guarantee it scans into the correct wallet. Most wallets have already updated to include this (ABC, Unit, Stash, etc). Ideally that support should be added to the bitcoin.com wallet, as it is a tiny change.
Unfortunately it's harder than it seems for us to add that toggle, as it changes the size of the QR codes that are generated, and throws off the layout considerably.
As for the additional artwork, I am sure that is something we could add. @theantnest how do you feel about that?

totally understand on the toggle prefix.
as for the graphics.. yes please!!

I also support pushing for wallet to support the spec as they are tbe one lagging behind, as discussed on slack.

repo: theantnest/bccaddress


text:
UI-Elements disappear on mouse-hover.

When hovering using a mouse over the left menubar, some icons disappear. It seems like there is a rendering issue. Even though some are not displayed, you can still click them.
I used the latest version from the AppStore, im currently downloading the TestFlight version. Maybe it was already fixed.

This was fixed some time ago.

source: gharchive/issue

repo: thebaselab/codeapp

license: MIT

license_type: permissive

license_source: github-api


text:
Korean Localization

Korean Localization for Version 1.3.3

This is incredible! Thank you!

source: gharchive/pull-request


text:
ci: add CI requirements file

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our commit message format.

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[x] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

We have no Python requirements file for CI, which has made us manually add packages to the install step on our CI.
Issue Number: N/A
What is the new behavior?

This will allow us to freeze and centralize the packages needed for our CI workflow.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

This was just a test

repo: thecesrom/incendium


text:
[BUG] Analytics | House button on graph not clickable

Describe the bug
Under setting -0> Analytics there is a Most Active Times graph and it got a Hours button that some seem to be click able
To Reproduce
Go to dashboard -> Analytics tab
Expected Behavior
Expected tat the button will be clickable
Screenshot/ Video

Code of Conduct

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

@Tamir198 All this page is not functional
Would you like to work on all the functionality of this page?
If you have questions, you are welcome you ask :)

Ye sure, let's start from where I get the data to the house maybe?

@kabir0x23 did someone implement the backend? Or should he create a dummy data json file for now?

dummy is fine for now, backend is not done yet,

@kabir0x23 @ArkadiK94
Do you want me to decide on the structor for the data? and later on, we can refactor it into the backend data?

@Tamir198 yes, if you will have questions about the structure you can ask.

repo: thecyberworld/TheCyberHUB


text:
Publish to nexus mods

Please consider publishing this to nexus mods.

that would be against the permission settings of the original BetterUI mod on nexus https://www.nexusmods.com/valheim/mods/189
while the source code is licences under "BSD 2-Clause "Simplified" License", the permission settings on nexus kind of contradict that. and the original author of BetterUI is sadly no longer around

authors: Goldenrevolver cdobbyn

repo: thedefside/BetterUI


text:
Support JsValue in Callbacks + Prevent Double Free

Allow JsValue arguments in add_callback + dead code cleanup
Prevent double free on set_property failure

Closes #109
Closes #107

Again, spurious internal quickjs error from valgrind. (need to figure out a ignore rule)

repo: theduke/quickjs-rs


text:
10/21 Manual Review - Resources

Section 13.2.1 Quanitfish link goes to a twitter post

Lab meeting time changed

Update lab members photos (madeleine was working on this)

Update Southcoast useR group (still active?) Seems like the last meeting was 2022
Same with Shut Up and Write

@gavinfay will update photo

authors: CataRoman KamranWalsh

repo: thefaylab/lab-manual


text:
Removed comments in numberType.json

These comments were causing errors in our jest tests because comments in json are not allowed.
Please consider merging this as I saw another issue mentioning this.

Looks like this has been merged 3 months ago, but not pushed to NPM :(

Hi, sorry for all delay. I'll publish new version soon.

authors: BobDeKort thegamenicorus yuriy-manifold

repo: thegamenicorus/react-native-phone-input

license: mit

license_source: bigquery


text:
Broken download PDF link on publication sites

Discussed in https://github.com/thegetty/quire/discussions/787

Originally posted by zsofiaj July 20, 2023
I am trying again to set up the pdf output on our Italian Paintings project, and I followed the instructions on the documentation page (https://quire.getty.edu/docs-v1/multiformat-output/#pdfprint-output), but the download is not working. I ran quire build, then quire pdf, then I resized the pdf and moved it to a new content/_assets/downloads folder. There was no "downloads" folder there in my template. I changed the resource_link information in my content/_data/publication.yaml file to /downloads/my-file-name.pdf. Then I ran quire build again, and pushed my changes to GitHub. The Netlify deploy finished publishing the site without any problems, but the pdf does not download at the link in the menu.
Does anyone have any idea why? @geealbers @Erin-Cecele Does it seem like I followed the instructions correctly? I wondered if something might be off since I didn't find a "downloads" folder in the spot where the documentation stated. Here is a link to our preview site (and you already have access to the repository):
https://italian-paintings-march2023-updates.netlify.app
I also have some questions about styling the pdf and making it a little neater all around, but I'll post those as new topics once I can get the pdf to download.
Thank you!

@zsofiaj, Thank you for reporting this issue. This also appears to be occurring in our Getty Quire publications. There appears to be a straightforward solution that we are working on now. We will keep you updated!

This has been fixed with PR #799, and will be released in upcoming Quire 11ty version 1.0.0-rc.13.
In the meantime, to fix the issue in existing projects, open the _plugins/vite/index.js file in your project and add the _assets/downloads directory to the list of copy targets there that start around line 55. So, it looks like this:
plugins: [
  copy({
    targets: [
      {
        src: 'public/*',
        dest: outputDir,
      },
      {
        src: path.join(inputDir, '_assets', 'images', '*'),
        dest: path.join(outputDir, '_assets', 'images')
      },
      {
        src: path.join(inputDir, '_assets', 'downloads', '*'),
        dest: path.join(outputDir, '_assets', 'downloads')
      },
      {
        src: path.join(inputDir, '_assets', 'fonts', '*'),
        dest: path.join(outputDir, '_assets', 'fonts')
      }
    ]
  })
]

Thanks so much, @geealbers. I was not able to find a _plugins/vite/index.js file, though. I don't see the "vite" folder in my "_plugins" folder. See attached screenshot. Am I looking in the wrong place?

@geealbers Actually I found that same text in the .eleventy.js file in the main directory. I added the missing _assets/downloads directory to the list of copy targets there, and that fixed it. Our pdf downloads fine now. Is this an okay way to fix it? And will I need to undo this once the Quire 11ty version 1.0.0-rc.13 has been released?

@zsofiaj good sleuthing! I wasn't aware of this but I guess sometime between the version you started your project on, and the latest version of Quire, we moved some stuff from .eleventy.js and into vite/index.js. Whenever you go to update your project the next time, as long as you bring in both the _plugins directory and the .eleventy.js along with everything else (which is what you would normally do anyway), the fix will persist and you won't need to do or undo anything.

Great, thank you @geealbers !

authors: Erin-Cecele geealbers zsofiaj

repo: thegetty/quire


text:
DROP_ME: build with Node.js 10.x.

Signed-off-by: Akos Kitta kittaakos@typefox.io

This was a test PR. I am closing it.

repo: theia-ide/dugite-extra


text:
cell type-aware embedding

Description of feature
Dear DRVI team,
This is a great tool and am enjoying using it a lot. I am just wondering if there are any plans to incorporate cell-type awareness into the model, similar to SCANVI and MRVI?
Many thanks

Dear Bo,
We're delighted to hear that you're enjoying it.
Incorporating supervised information, such as cell types, is indeed an interesting idea. However, DRVI is fundamentally designed for unsupervised discovery in single-cell omics. One of its key strengths is its ability to remain unbiased with respect to potentially imperfect supervised annotations, allowing DRVI to uncover patterns and insights that go beyond existing annotations.
Extending the model to incorporate supervised information is a lower priority than other directions outlined in the discussions section of the paper. Users often incorporate cell-type information to improve integration performance —when the integration performance is measured by the same annotations— introducing a circular logic that we aim to avoid. We’re always open to revisiting our priorities based on compelling use cases. Do you have a reasonable example or scenario where embedding such supervised information could significantly enhance users' analyses? We'd love to hear your perspective.

repo: theislab/DRVI


text:
Failing to set value on view mutates anndata (makes it not a view)

Attributes which derive from the _SetItemMixin mutate the anndata object even when setting the values fails. Here's an example:
import scanpy as sc

pbmc = sc.datasets.pbmc68k_reduced()
subset = pbmc[pbmc.obs["bulk_labels"] == "Dendritic"]

subset.isview  # True
subset.obs["new_val"] = [1, 1]  # Raises ValueError
subset.isview  # False

I think the cause is line 391 from here: https://github.com/theislab/anndata/blob/34f4eb63710628fbc15e7050e5efcac1d7806062/anndata/base.py#L383-L392
The anndata is being modified before the value has been set. This could be averted by making a copy of the attribute to be modified, modifying that, then modifying the anndata object.

Fixed!

repo: theislab/anndata


text:
writeread module does not write to loom format

Hi,
I am sort of confused as to why this function is not working. Any tip on to how resolve this?
Thank you,
O

You're calling this function in the wrong way. Simply type
adata.write_loom('myfilename.loom')

if adata is an AnnData object.
See the documentation of the function.

Hi Alex,
Thank you for your prompt response. I tried what you suggested:

As I have shown you in my previous post adata is an AnnData object.
Olivia

sc.AnnData is the class. You need to call the method of an object, like:
adata = AnnData(...)
[...]
adata.write_loom('./filtered...loom')

Indeed it was enlightening and useful...However, still gives me an error...
type(adata)
anndata.base.AnnData
adata.write_loom('./filtered_gene_bc_matrices_h5.loom')
... writing to '.loom' file densifies sparse matrix
Converting to csc format
Creating
AttributeError                            Traceback (most recent call last)
 in ()
----> 1 adata.write_loom('./filtered_gene_bc_matrices_h5.loom')
/anaconda3/lib/python3.6/site-packages/anndata/base.py in write_loom(self, filename)
1736         """
1737         from .readwrite.write import write_loom
-> 1738         write_loom(filename, self)
1739
1740     @staticmethod
/anaconda3/lib/python3.6/site-packages/anndata/readwrite/write.py in write_loom(filename, adata)
71     if os.path.exists(filename):
72         os.remove(filename)
---> 73     create(filename, X, row_attrs=row_attrs, col_attrs=col_attrs)
74
75
/anaconda3/lib/python3.6/site-packages/loompy/loompy.py in create(filename, matrix, row_attrs, col_attrs, file_attrs, chunks, chunk_cache, dtype, compression_opts)
1019
1020         if scipy.sparse.issparse(matrix):
-> 1021                 return _create_sparse(filename, matrix, row_attrs, col_attrs, file_attrs, chunks, chunk_cache, dtype, compression_opts)
1022
1023         # Create the file (empty).
/anaconda3/lib/python3.6/site-packages/loompy/loompy.py in _create_sparse(filename, matrix, row_attrs, col_attrs, file_attrs, chunks, chunk_cache, dtype, compression_opts)
982                 if ds is None:
983                         logging.info("Creating")
--> 984                         ds = create(filename, matrix[:, ix:ix + window].toarray(), row_attrs, ca, file_attrs, chunks, chunk_cache, dtype, compression_opts)
985                 else:
986                         logging.info("Adding columns")
/anaconda3/lib/python3.6/site-packages/loompy/loompy.py in create(filename, matrix, row_attrs, col_attrs, file_attrs, chunks, chunk_cache, dtype, compression_opts)
1033
1034         for key, vals in row_attrs.items():
-> 1035                 ds.set_attr(key, vals, axis=0)
1036
1037         for key, vals in col_attrs.items():
/anaconda3/lib/python3.6/site-packages/loompy/loompy.py in set_attr(self, name, values, axis, dtype)
561
562                 self.delete_attr(name, axis, raise_on_missing=False)
--> 563                 self._save_attr(name, values, axis)
564                 self._load_attr(name, axis)
565
/anaconda3/lib/python3.6/site-packages/loompy/loompy.py in save_attr(self, name, values, axis)
166                 if self.mode != "r+":
167                         raise IOError("Cannot save attributes when connected in read-only mode")
--> 168                 if values.dtype.type is np.str:
169                         values = np.array([x.encode('ascii', 'ignore') for x in values])
170
AttributeError: 'list' object has no attribute 'dtype'

As you can see from the error output, this is an error within loompy.
I assume you don't have the most recent version of loompy - there used to be a few bugs in it. Try with an update installation of loompy. I'm running 0.2.8 and never had any problem with it.

that did the trick!
Thank you for your patience!

authors: falexwolf flying-sheep oliviamr

repo: theislab/scanpy


text:
Fix values_to_plot literal

@fidelram small doc fix.

@gokceneraslan I'm going to try a tool for doing some back porting here, sorry about the noise!

@meeseeksdev backport to test-backports

authors: gokceneraslan ivirshup


text:
Use for/yield to define routes instead of akka-style nested calls

Some time ago I tried (unsuccessfully) to push akka-http in the direction of for/yield (monadic) DSL.
Here is my rejected PR from 2018.
https://github.com/akka/akka-http/pull/1925
You could consider implementing this. I've been using this monadic DSL in production.

@winitzki thanks for reaching out!
This indeed looks interesting. Did you release that as a library for akka-http?
As for http4-stir I plan first to reach some maturity and only after that start adding new features. My first idea is to include more directivies (http4s-stir-missing) but I could also release your DSL (http4s-stir-monadic).
I'll keep this open, so that we can get back to it in the future.

I never had any time to release this as a library. The code in that PR is reasonably complete and yet quite small. This idea is simple to implement: directives are a monad that needs a flatMap. You can always add more directives in the future, the monadic wrapper works with any directives.

authors: luksow winitzki


text:
Use MAX_LENGTH constant to check buffer size

Replace hard-coded constant for max buffer size check with Buffer MAX_LENGTH when available.

I actually prefer to use my own copy of the constant, which is documented explicitly in the API. If node later decides to increase the limit, I want my API to remain the same. The limit in yazl is based on node's limit, but it is now yazl's limit. If node were to reduce the limit, that would break my assumptions, but I don't think that's ever going to happen.
Thanks for the contribution though! And sorry for the delayed response.

@thejoshwolfe sorry I should have been more explicit in the description: the motivation was that 64 bit versions of node were capable of creating buffers up to 4GB, so valid buffers are triggering this error and I assumed the limit you have here was related to the maximum buffer size in Node. The value of MAX_LENGTH is different on different architectures, so that's why I used it, rather than just changing the constant value here. Since the PR the limit has again been lifted to 8 PiB on 64 bit architectures in v23, which means buffers are effectively limited only by available memory now. Have a look at the API docs to confirm. If this check is indeed related to max buffer size then it might be worth reconsidering since the current limit is below what's possible in Node.
Rereading this some time later though I'm actually not totally clear on what the purpose of the check is? yazl can add files to a zip larger than 1GB via a stream, and it's not possible to allocate a Buffer larger than MAX_LENGTH as Node throws an error. Do you need the check at all?

Huh. Now I'm also not totally clear on what the purpose of the check is. I don't like the idea of zlib compression throwing a runtime error when the compressed buffer would exceed the max size; that would be a pretty bad user experience. But limiting the input is only one of many ways to avoid that problem. (And the current limit doesn't even avoid that problem, I think?) It could also be solved by streaming the compression into the output instead of keeping the compressed buffer in ram. :thinking: (This also relates to #54 .)
It does seem compelling to allow the bigger buffers when they're available and to simply document that the size limit is imposed by node, and link to the API docs you gave. Those docs are pretty clear and include the change history and everything.

Yes I think it makes sense that the principle behind doing the check is to try to avoid the output buffer getting too big by not allowing in huge inputs. Having thought about it, since it's checking an existing buffer that the user is providing, by definition it can't possibly be bigger than MAX_BUFFER, so checking length against that might actually be redundant anyway?
For now, might it be worth removing instead of modifying the check here since it sounds like it won't cause new  problems and would allow bigger buffers to be added?
I'm not across the library enough to talk about the compressed buffer bit and I'm unclear on whether Node is going to throw automatically in that code, but it might already throw a RangeError that could be caught and wrapped with a friendlier message?

authors: gpoole thejoshwolfe

repo: thejoshwolfe/yazl


text:
Docs

[x] Improve README
[x] Improve Wiki
[x] Code documentation required
[x] Ability to comment in Lomda programs

Wiki has been updated.


text:
Test suite improvements

Implements extra testing features, resolves (most of) https://github.com/themaddoctor1/Lomda/issues/14
Useful for testing with respect to memory leaks: https://github.com/themaddoctor1/Lomda/issues/17

Branch has Git issues.


text:
🛑 [Yandex] outbound mail is down

In 28c3d67, [Yandex] outbound mail (smtp.yandex.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: [Yandex] outbound mail is back up in dec61ab.

repo: themanaworld/upptime


text:
User에 GithubLoginId 필드 추가 및 배치 로직 작성

개요
User관련된 데이터 객체(UserEntity, TempUserEntity, )에 githubLoginId필드를 추가했습니다.

githubLoginId는 github에서 login시 필요한 id를 의미합니다. 즉 해당 PR의 작성자의 github login id는 siwony라고할 수 있습니다.

그리고 기존 사용자들은 githubLoginId가 없으므로 githubId로 githubLoginId를 찾아 update하는 batch 로직을 작성했습니다.
AddGithubLoginIdJobConfiguration와 LockupGithubLoginIdService를 참고해주세요
githubLoginId 필드를 추가한 이유
githubLoginId값을 통해 github 프로필 페이지를 만들 수 있습니다.
batch application 수행시점
prod 프로필 환경에서 애플리케이션을 실행하는 시점에 해당 배치 작업이 수행됩니다.
그러므로 master에 merge후 deploy된 후 배치를 비활성화 할 예정입니다.

test 실패

repo: themoment-team/HiRecruit-server


text:
🛑 Image server is down

In 1c958ce, Image server (https://image.themonetizr.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Image server is back up in 2e34a74.

repo: themonetizr/upptime


text:
New keys created by langman should be in default language rather than empty

Langman creates new files with new keys which are empty.
Problem is that translators who receive those files will have to look into the original language files to know what the original message is before they can translate it.
It would be useful to have a default language instead of an empty string.

The point of empty strings is for easily finding them with the langman:missing command, if we fill the new keys we won't be able to identify keys that need translation.

A import/export feature is planned here, I think it'll work for the case you're mentioning: https://github.com/themsaid/laravel-langman/issues/31

Yes, that would work too.

authors: themsaid vpratfr

repo: themsaid/laravel-langman


text:
[Linux]Sending Coins from linux account to windows Account friends Transaction history record not showed

Bug Description
When Sending Points From One Account To friend Account That Transaction Entry Not Shows In Friend Transaction side Only Notified Him
Steps to Reproduce
Steps to reproduce the behavior:
1.Click On Send Points Button
2.Select Friends Account
3.Enter Points & Click on Send Button
4.Go To The Friends Account
5.click on Transaction Tab
Expected behavior
Transaction entry not shown it
Actual behavior
Transaction entry shown it
Screenshots/Recordings
https://prnt.sc/vginl0
OS and Browser

OS: [Linux]

Duplicate issues as its already there:- #425

Duplicate issues as its already there:- #425

Yes its a duplicate issue that issue appears on TNB Ver 26 and now TNB Ver 28 still same issue that's reason posted again..

repo: thenewboston-developers/Account-Manager


text:
Timesheet - 20/04/2021 - Bucky Roberts

Task 1
Sprint planning meeting with Kristy, Justin, and Muhammad.
Proof of Work

Core team feedback
Community feedback
Sprint planning

Time Spent
3 hours

Task 2
Chatted with Truyen and got our GitHub organization plan upgraded so that we could set up proper GitHub action permissions.
Proof of Work

Time Spent
0.5 hours

Total Time Spent
3.5 hours
Account Number
a37e2836805975f334108b55523634c995bd2a4db610062f404510617e83126f

Approved

authors: buckyroberts manishram


text:
Added two new contributers

Resolves #914
Resolves #915
Account Number: 3b59cd93d7cb88fd55caaea77f07328b57cce7e3d1a14b13410291c698dccf2e

@DinShpati Don't club multiple issues in a single PR. One issue, One PR. Can you update your PR?

@DinShpati Don't club multiple issues in a single PR. One issue, One PR. Can you update your PR?

sure

authors: DinShpati thesanjeevsharma

repo: thenewboston-developers/Website


text:
Feature/event restructure

Solves #4

Or an alternative to this: implement all current listeners as middlewares. But rather not. :P

I love me some middleware but not even I'd bother implementing all of these as middleware. :laughing:
In general, I like it. Seems like to cleans up the codebase and reduces the number of concepts a fair bit!
I think you could get rid of the abstract ConsumerAware entirely, it barely saves any code and you're actually working around it for the Wait constructor anyways. A public setter for a dependency is sub-optimal,  $this->consumer->shutdown is just as short and an internal property is hardly worth the bother of a base class. :) So, I'd say chunk that part.
Otherwise, :+1: Nice work!

Regarding the constructor thing: I am thinking about two things:

Make the Consumer a constructor parameter in every listener
Have a setConsumer method

I think having a setter is better otherwise you would have to check the type every time shutdown is called.
Both have pros and cons. Haven't decided which is better yet. Any thoughts about this?
Also, yea, I agree, I am thinking about removing the base class. However the upper problem still needs to be solved. If we stick to the setter solution, it should remain.

I think it'd be better to have the Consumer as a constructor parameter for each one. It gives each class complete freedom on that part, while still abiding by a really simple API.
I'm not sure what typecheck you mean? The only one I see now is the null check but if it's a required parameter of the constructor, then that shouldn't be necessary. If a class does make it optional (for some bizarre reason), then yeah, they have to check but that's no biggy.

@rosstuck can you please check this out? Especially this part. Isn't it a bit vague? (Not just the docs, but the functionality as well)

See #6

For the docs example, maybe break them up per plugin. Here you can drop almost everything before the "// execute maximum of 10 commands" comment, I think.
Otherwise, I think it makes sense, though I'd use a separate eventEmitter, not the eventMiddleware for clarity's sake. That's just me though.

Yeah, breaking it up is a good idea.
The reason I like this EmitterTrait that exposes emitter functions is that
in a DI context you might autoresolve the emitter when resolving the
middleware, thus you might not even have direct imteraction with the
emitter. That said, you can pass an emitter in the ctor if you like that
better.

I really loathe autoresolving DI so that might be the difference in opinion. :)

authors: rosstuck sagikazarmark

repo: thephpleague/tactician-bernard-queueing


text:
creating URL objects that follow https://url.spec.whatwg.org/

This issue is based on https://github.com/thephpleague/uri/issues/113 opened by @kelunik . The premise is to be able to have an URL object which validates the URL string againtst https://url.spec.whatwg.org/

After reading the spec. I think this can be achieved using the League\Uri\create function by updating its algorithm. So we would end up with

the ::createFromString named constructor that would still follow RFC3986
the while the League\Uri\create function would follow to more permissive WHATWG spec.

Currently I see two issues with this:
1°) As it stand the UriInterface used by the league package is similar to PSR-7 UriInterface which imposes some restrictions to URI object like explained in #9 . These restrictions forbids 100% compatibility with WHATWG spec (if for instance one wants to instantiate a File URL).
Why is this important ? Because one would want to have the newly create function pass the URL spec tests defined here https://github.com/web-platform-tests/wpt/blob/master/url/url-constructor.html
using the data from https://github.com/web-platform-tests/wpt/blob/master/url/resources/urltestdata.json without having to modify them.
2°) If more incompatibilities are found shouldn't we move the WHATWG parser to a separate package as some parsing rules are just not compatible with the RFC3986 parser ?

Another issue is that the URL standard is a living one leading to continuous bug fixes being added to the underlying algorithm putting the League\Uri\Factory class in its own repo ensure less useless release for the rest of the package.

repo: thephpleague/uri-schemes


text:
🛑 Website is down

In 24a1a69, Website (https://theswissbay.ch/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Website is back up in 66c77d4 after 4 minutes.

repo: theswissbay/status


text:
Generic exceptions vs Specific exceptions

Description of issue or feature request:
We want to use a shortlist of exceptions that will be thrown from our APIs, so that our users can handle them easily.
On another hand that doesn't stop us from creating other more specific exceptions that derive from those main exceptions.
This allows us to easily test that specific errors were thrown when we expected them.
For example, we are using BadVersionError and LengthOrHashMismatchError which derive from RepositoryError.
Still, there are two use cases that are worth discussing:

There are multiple places inside TrustedMetadataSet where in one API call we are throwing generic and specific exceptions.
That's happening inside each of the functions update* inside TrustedMetadataSet we throw RepositoryError, ReplayedMetadataError, ExpiredMetadataError and UnsignedMetadataError (from verify_delegate()) at the same time.
One idea could be to define new specific exceptions (which derive from RepositoryError and replace the instances of RepositoryError with them.

Read and synchronize with issues:

#1778
#1779
#1714
#1312

Inside the TrustedMetadataSet update* function docstrings we only document that we are throwing RepositoryError without specifying the specific exceptions. If none of them are documented in the Raises section of the method docstrings (they only mention the broad RepositoryError), do we expect anyone to handle them? If we do, we should probably document them?

Inspired by https://github.com/theupdateframework/python-tuf/pull/1725#discussion_r787713776.

Inside _get_session in module tuf/ngclient/_internal/request_fetcher.py we currently throw URLParsingError: https://github.com/theupdateframework/python-tuf/blob/0e26364dfcb54bc7c0eee9f7d2ef7a67b9b53868/tuf/ngclient/_internal/requests_fetcher.py#L135
In my commit https://github.com/theupdateframework/python-tuf/pull/1725/commits/e8f26ebb1e7c68fc9c0ddd5b3a268c0a56a7a304 inside pr #1725 I am replacing it with DownloadError.
The question is do we want to keep the specific exception - URLParsingError and make it derive from DownloadError?
Does my change in this commit make sense?

The question is do we want to keep the specific exception - URLParsingError and make it derive from DownloadError?
Does my change in this commit make sense?

I believe no-one is ever going to handle a URLParsingError so there should be no need for it. If that statement turns out wrong and a need arises later, adding a derived error is easy. The commit is good.

Inside the TrustedMetadataSet update* function docstrings we only document that we are throwing RepositoryError without specifying the specific exceptions. If none of them are documented in the Raises section of the method docstrings (they only mention the broad RepositoryError), do we expect anyone to handle them? If we do, we should probably document them?

Updater does use some of the specific exceptions. I would not object to TrustedMetadataSet having better exception documentation, the reason it doesn't is probably the fact that it's currently private (documenting too much does lead to outdated documentation...)
For updater itself I think documenting just RepositoryError (and not the more specific ones) is reasonable.

I just did a thorough review of all exceptions we raise in api and ngclient and it all looks good modulo some missing documentation (will post related comments to #1785, where they seem more appropriate).
So I agree with what Jussi says about TrustedMetadataSet exceptions and I suggest we remove the following TODO comment?
https://github.com/theupdateframework/python-tuf/blob/cb7bd6aff9be097facecbd27ee5998c48feeb52d/tuf/ngclient/_internal/trusted_metadata_set.py#L62-L65
Let's do that and close this issue?

Yes: PR #1805

authors: MVrachev jku lukpueh

repo: theupdateframework/python-tuf


text:
Neoway technologies

Adding some technologies used in Neoway.

Thanks @Jack3Dz!

repo: thiamsantos/floripa-tech-companies

license: Unlicense


text:
Why don't you inherits from QCheckbox class

I like this plugin, but i'm wondering, why you inherits from QWidgets, instead of a more natural class as QCheckbox?

@mbsanchez, I have done this improvement, I push it soon ;)

authors: mbsanchez thibDev

repo: thibDev/qtCustomPlugins


text:
POC: Reusing AWS IoT as PKI for device registration with Cumulocity IoT

Is your feature request related to a problem? Please describe.

Proof-of-concept: Can AWS IoT be used as PKI for device registration in Cumulocity?
Goal here is to:

Check feasibility of using features from different platform providers to compliment each other
Gain experience on how a local device side pki plugin interface could look like for thin-edge.io to enable multiple PKI providers

Describe the solution you'd like

The following describes the general idea for the proof-of-concept. Please note that I don't know the current process that is used when connecting devices to AWS IoT, so please adapt the sections accordingly to what is possible.
Initial User setup
The exact details will be dependent on what is possible in AWS, e.g. is it possible to export the root/intermediate certificate which is used to sign all device certificates before a device has been registered?
Before the device registration can occur, the root or intermediate signing certificate should be exported from AWS IoT and imported into Cumulocity IoT. This would allow devices to re-use the AWS IoT signed certificates when registering with Cumulocity IoT.
Step 1: Export/Import of root/intermediate signing certificate

Configure then export the root/intermediate signing certificate from AWS IoT
Import the certificate using the Cumulocity IoT Device Management Application under "Trusted Certificates". Enable auto-registration of devices.

Step 2: Device registration with AWS IoT

Device registers itself with AWS IoT (so that it can get a device certificate)

Since I don't know the AWS IoT specifics, I don't know how this exactly works, so there might or might not be a CSR used to do this, if there isn't then just ignore the CSR stuff.
Step 3: Device registration with AWS IoT

Device registers itself with AWS IoT (not sure on the exact specifics, the user might need to pre-register the device in AWS). Hopefully after this process, there is a device certificate on the filesystem that can be used in the communication with Cumulocity IoT
Configure the device to connect to Cumulocity IoT using the same certificate from step 1
Device connects to Cumulocity IoT

The same general process can also be described in the following sequence diagram.
sequenceDiagram
    actor User
    participant device as Device
    participant aws as AWS IoT
    participant c8y as Cumulocity IoT

    Note right of User: Setup
    User ->> aws: Request export Root/Intermediate certificate
    aws -->> User: Root/Intermediate certificate
    User ->> c8y: Import Root/Intermediate certificate as Trusted Certificate

    Note right of device: Device Registration
    device ->> device: Create a private key
    device ->> aws: Send Certificate Signing Request (CSR)
    aws -->> device: Signed Certificate
    device ->> device: Save signed certificate to the tedge cert folder
    device ->> aws: Register Device with AWS

    device ->> c8y: Register Device with Cumulocity

Additional context

There are already two AWS proof-of-concepts (PoCs) which show how thin-edge.io can connect to AWS IoT. This could provide insights on the AWS IoT device registration process.

https://github.com/thin-edge/thin-edge.io/pull/1460
https://github.com/thin-edge/thin-edge.io/pull/1347

Info: Certificate Signing Request (CSR)
A Certificate Signing Request is the process where a locally generated key is generated, but the key is not yet recognised by any external system (nor should the key leave the device!). The device creates a CSR file based on the key, an example of this is shown below:
# Create CSR from private key
openssl req -new -newkey rsa:2048 -nodes -keyout your_domain.key -out your_domain.csr

The CSR file is then used to request to retrieve the public certificate via a request to the PKI. If the PKI request is successful, then the PKI returns a public certificate which is signed by some root or intermediate certificate. The root or intermediate certificate is the one that validates that the device is somewhat "trusted", and can be used to validate the registration requests from a device using a device certificate signed by the "trusted" certificate, that it is ok to communicate with.
This has the benefit that individual device certificates do not have to be individually uploaded to the server (e.g. Cumulocity IoT).

In addition, AWS IoT generates certificates signed by AWS Root CA, so in order to have a private CA, we have to use another service, like AWS Private CA.

This PR proposes an implementation to create a CSR for the device certificate. The question I have regarding this proposal is: "is a CSR generated implicitly along the self-signed certificate" or "would an explicit command less confusing"? Indeed, having a CSR plus a self-signed certificate might be confusing for the users even there is nothing wrong on that.

What i was going for in the POC was having a flag for the certificate create command, e.g. tedge cert create --ca aws, with a value for a specific CA provider, like aws. The command would then handle obtaining a certificate signed by this CA using this CA's preferred method (so for AWS IoT it would be using create_certificate_form_csr function from aws_iot_sdk crate) and then installing the signed certificate in /etc/tedge/device-certs/tedge-certificate.pem path.

What i was going for in the POC was having a flag for the certificate create command, e.g. tedge cert create --ca aws, with a value for a specific CA provider, like aws. The command would then handle obtaining a certificate signed by this CA using this CA's preferred method (so for AWS IoT it would be using create_certificate_form_csr function from aws_iot_sdk crate) and then installing the signed certificate in /etc/tedge/device-certs/tedge-certificate.pem path.

This functionality was implemented in the following commit on pki-poc branch of my fork: https://github.com/thin-edge/thin-edge.io/commit/aedb1c2e456c13081c08bc82a5e3aa4aff26dd6f

Step 1: Export/Import of root/intermediate signing certificate
1. Configure then export the root/intermediate signing certificate from AWS IoT

2. Import the certificate using the Cumulocity IoT Device Management Application under "Trusted Certificates". Enable auto-registration of devices.

I don't think that it's possible to do in AWS IoT. The certificate used for signing client certificates is not viewable anywhere, as far as I know, and is used only for verifying the identity of clients connecting to AWS IoT core itself, so while it could be useful if AWS IoT support ever gets merged, it wouldn't be of any use to Cumulocity.
Similarly with AWS Private CA, the pricing is high, and it's another thing entirely, so I expect this also won't be a good fit for Cumulocity, because instead of just one intermediate CA per tenant that we can import client side, this is just one Root CA that we would have either import globally into Cumulocity, or we would have to manually implement generating 1 intermediate cert per user and making it available for the user to import into their Cumulocity tenant??? Anyway this doesn't make use of AWS IoT at all.
So I would consider AWS dead end in that regard, and maybe look into Azure, although I doubt we will find anything like we hoped for.

However a “not feasible” is just as valuable from a POC. We’ll double check the work before we close the ticket just to be sure. But thanks for the effort :)

Following a discussion with @Bravo555 .

Using create_certificate_form_csr can hardly be used to trust device certificates in the context of a Cumulocity tenant because all these certificates are signed by a generic AWS root certificate.
To implement something along the lines of this request, one needs to use AWSer private CA (The prices make sense, knowing that a single signing intermediate signing certificate is required per tenant).

This can be implemented using aws-sdk-acmpca and notably the issue certificate operation.
In the context of this POC,  this API can be invoked directly from the device. However, this might not be be the best solution from a user perspective and a better solution will have to be designed. Indeed, to invoke this API one needs AWS credentials and it would be cumbersome and non-secure to deploy AWS credentials on a fleet of devices connected to  Cumulocity. A solution can be to use AWS private CA from the Cumulocity microservice acting as a proxy between the devices and AWS private CA: only Cumulocity credentials are then required on the device.

For this POC:

Implement tedge cert create --ca aws using issue certificate aws-sdk-acmpca operation and assuming that a private CA has been created beforehand.

Closing as the original scope of the POC has been fulfilled. The conclusion was that it is not possible to re-use the AWS certificate authority in Cumulocity IoT.
Useful aspects mentioned in this ticket, such as a cli command to create a CSR, will be configured in a new ticket.

authors: Bravo555 didier-wenzek reubenmiller


text:
Migrate profile page to tailwind and Move to app router

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR focuses on refactoring and enhancing the profile and contract management features within the dashboard application, improving user experience and code structure.
Detailed summary

Deleted unused files related to profiles and contracts.
Updated replaceDeployerAddress to replace specific addresses.
Changed useRouter to useDashboardRouter in PlaygroundMenu.
Added onlyMainnet parameter in getSortedDeployedContracts.
Introduced resolveAddressAndEns function for address resolution.
Created PublishedContracts component for displaying contracts.
Modified DeployedContracts to remove isPending prop.
Refactored ProfileUI to include new contract components and metadata handling.
Enhanced EditProfile form with better error handling and layout.
Improved social links handling in PublisherSocials.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#5364  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MananTank and the rest of your teammates on  Graphite

Merge activity

Nov 8, 8:11 PM EST: Graphite disabled "merge when ready" on this PR due to: a merge conflict with the target branch; resolve the conflict and try again..

authors: MananTank jnsdls

repo: thirdweb-dev/js


text:
Händlerformular: Adressfeld aufsplitten

Feld Adresse aufteilen in mehrere Felder:
Strasse
Hausnummer
Postleitzahl
Ort
-> Mit Rebecca schauen wie genau sie es haben will.

Hoi markus
Ich bin dagegen, dieses feld aufzusplitten. Ich kenne keine gründe dafür.
Grüessli
Thömu
Thomas Schlegel
Lorrainestrasse 66
3014 Bern
078 604 59 98

Am 05.09.2015 um 15:52 schrieb Markus Kläsle notifications@github.com:
Feld Adresse aufteilen in mehrere Felder:
Strasse
Hausnummer
Postleitzahl
Ort
-> Mit Rebecca schauen wie genau sie es haben will.
—
Reply to this email directly or view it on GitHub.

repo: thoemel/veloboerse


text:
No Code-Format providers registered for scope:

What's this ? can any body help?

I don't think this issue is related to color-picker. Please let me know if it is. Closing this issue for now.

@vimniky if you have nuclide installed, that module is causing the problem. Try this: apm uninstall nuclide and restart Atom.

There is an unrelated ticket open about this error (still): https://github.com/facebook/nuclide/issues/386
Some might not want to uninstall nuclide, so you can disabled the Keybindings in the color-picker package and rebind them to a new key.
'atom-text-editor':
  'cmd-shift-e': 'color-picker:open'

authors: fabioanselmo gwing33 thomaslindstrom vimniky

repo: thomaslindstrom/color-picker


text:
🛑 Syncthing-KA-BI is down

In f6f00e6, Syncthing-KA-BI ($URL_SYNCTHING_MERZ_NIMBUS_BI) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Syncthing-KA-BI is back up in 1aaac61 after 4 minutes.

repo: thomasmerz/upptime


text:
🛑 Speedtest-v2-HEL1-1 is down

In 5322d6a, Speedtest-v2-HEL1-1 ($URL_SPEEDTEST_V2_HEL1_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Speedtest-v2-HEL1-1 is back up in c2020b6 after 33 minutes.

RCA: https://github.com/thomasmerz/upptime/issues/2990


text:
🛑 Netdata-Karlsruhe.Social is down

In fb71f61, Netdata-Karlsruhe.Social ($URL_NETDATA_KASOCIAL) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Netdata-Karlsruhe.Social is back up in 5d93100 after 8 minutes.


text:
fr: medianBy

Much like you can do a "sortBy" someCollectionOfStuff.sortedBy { it.lum } could the fun functions like medianBy also support that?  I have a large number of pixels, and I'm looking for the median pixel value by luminance, and this would be perfect... if I could get xBy.

@salamanders thanks for the suggestion. I do not see why this shouldn't be included.
In the interim, you should be able to implement this yourself (via extension function) by using the groupApply(). See the source code for many examples on how to do this:
https://github.com/thomasnield/kotlin-statistics/blob/master/src/main/kotlin/org/nield/kotlinstatistics/DoubleStatistics.kt#L39-L40

authors: salamanders thomasnield

repo: thomasnield/kotlin-statistics


text:
Update issue templates

add Bug Report

closes #3


text:
[Swap] Add identifier for ASGARDEX to swap memo

ASGDX ID: 999
Memo: SWAP:{ASSET}:{DESTADDR}:12341999 (slightly adjust limit to id 999)
Depends on #1105

Does it make sense to make this cahnge the part of asgardex-util package?

getSwapMemo has already all data we need to pass: { asset: Asset; address?: string; limit?: BaseAmount }. The identifier 999 will be the last three digits of limit. However, before we can add a slip limit to the memo, we do need to implement a slip limit protection #1105 (we don't have such protection in ASGDX right now)

july 16,2021
is there an issue with your swapping - since the calulations are way off .
did a swap in Asgardex from btc to rune / rune to btc  DID HAVE $3881.57 but after swapping around i now left with $226.40 .. see transaction below ( missing 3655.17 )
CHAOSNET
POOLS
WALLET
$ USD
ASSETS  SHARES  BONDS  HISTORY
DETAIL	DATE	DETAIL
SWAP
INᚱ 7.01576684⚡ 83,337OUT
07/14/2021 19:34
SWAP
INᚱ 710.71562954₿ 0.12373175OUT
07/14/2021 19:16
SWAP
IN₿ 0.12543178ᚱ 717.77828709OUT
07/14/2021 18:53
SWAP
INᚱ 6.67812181⚡ 108,871OUT
07/14/2021 16:48
SWAP
INᚱ 334.88609039₿ 0.05892867OUT
07/14/2021 16:45
SWAP
IN₿ 0.08257076ᚱ 341.7205004OUT
06/15/2021 22:02
paddress
bc1q528rrkmhdhpdhqh0c4ll0nl3rjtx8txgjfa3kd
thor1lfe48vgxxmmmawrsuqe4hpak2qfaetvvsua9su
0x9c02679cca11ad94044083f9b9a161571acf5468
please contact me @ pantheons @ gmail.com

Why did you close this issue, when it’s not resolved yet !!
I still have not got my crypto amount corrected via $4847.868
In eth which you show only .605 crypto !
Please reply
Thxs
Rob
On Thu, Aug 5, 2021 at 9:07 AM veado @.***> wrote:

Closed #1615 https://github.com/thorchain/asgardex-electron/issues/1615
via #1650 https://github.com/thorchain/asgardex-electron/pull/1650.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/thorchain/asgardex-electron/issues/1615#event-5117381044,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUNORVGG6TYVJMGVEATWM53T3KEJDANCNFSM47637GXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Why did you close this issue, when it’s not resolved yet !! I still have not got my crypto amount corrected via $4847.868 In eth which you show only .605 crypto ! Please reply Thxs Rob

Pls note: This issue here (#1615) has nothing to do with #1636, which is still open. You got an info about closing this issue here just because you have added a duplicated message ^ before.

O ok, thxs
On Sat, Aug 7, 2021 at 10:44 AM veado @.***> wrote:

Why did you close this issue, when it’s not resolved yet !! I still have
not got my crypto amount corrected via $4847.868 In eth which you show only
.605 crypto ! Please reply Thxs Rob
Pls note: This issue here (#1615
https://github.com/thorchain/asgardex-electron/issues/1615) has nothing
to do with #1636
https://github.com/thorchain/asgardex-electron/issues/1636, which is
still open. You got an info about closing this issue here just because you
have added a duplicated message ^
https://github.com/thorchain/asgardex-electron/issues/1615#issuecomment-881363939
before.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/thorchain/asgardex-electron/issues/1615#issuecomment-894663609,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUNORVAVPHJNXPQCJZEP56TT3VBGDANCNFSM47637GXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Any update on this issue, notice no change after upgrade?
Thxs
Rob
On Sat, Aug 7, 2021 at 4:57 PM RH @.***> wrote:

O ok, thxs
On Sat, Aug 7, 2021 at 10:44 AM veado @.***> wrote:

Why did you close this issue, when it’s not resolved yet !! I still have
not got my crypto amount corrected via $4847.868 In eth which you show only
.605 crypto ! Please reply Thxs Rob
Pls note: This issue here (#1615
https://github.com/thorchain/asgardex-electron/issues/1615) has
nothing to do with #1636
https://github.com/thorchain/asgardex-electron/issues/1636, which is
still open. You got an info about closing this issue here just because you
have added a duplicated message ^
https://github.com/thorchain/asgardex-electron/issues/1615#issuecomment-881363939
before.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/thorchain/asgardex-electron/issues/1615#issuecomment-894663609,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUNORVAVPHJNXPQCJZEP56TT3VBGDANCNFSM47637GXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

authors: gromgromgrom pantheons1 veado

repo: thorchain/asgardex-electron


text:
THORN-1772 : Update SmallRye FT to 1.1

[x] Have you followed the guidelines in our Contributing document?
[x] [v2] Have you created a JIRA and used it in the commit message?
[ ] [v4] Have you created a GitHub Issue and used it in the commit message?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[x] Have you built the project locally prior to submission with mvn clean install?

Test failure with java.lang.NoClassDefFoundError: org/hamcrest/Matchers

Thank you @mkouba. Apparently I forgot to push my last commit with Hamcrest inclusion :-(

@antoinesd no problem and you're welcome ;-)

authors: antoinesd juangon mkouba

repo: thorntail/thorntail


text:
loader/svg: Remove duplicate if-check in pathAppendArcTo

The if-check to skip-rule when drawing an arc path is already checked in line 476.
In addition, since the float type equal check is performed in the range of 1/256,
unintentional skiped may occur. Therefore, remove duplicate code. https://www.w3.org/TR/SVG2/paths.html#ArcOutOfRangeParameters
test file: SVG_FILE_147893.svg
related issue: https://github.com/thorvg/thorvg/issues/1255

Rebased branch

@JSUYA  Could you please add SVG_FILE_147939.svg to our thorvg/src/examples/resources?

authors: JSUYA hermet

repo: thorvg/thorvg


text:
Review "splashkit camera" guide

A tutorial review of this tutorial has been completed. You have run the code in this tutorial to ensure that it works properly and outputs the results. Fixed an issue in Python code where the wrong number of parameters were passed when fill_rectangle() was called, removed extra parameters, made sure that C# used a top-level statement style, and made sure that the style was consistent with other languages, changed some text descriptions to more clearly express the relationship between game coordinates and screen coordinates, And how to achieve dynamic effects through camera movement.

Good job on this tutorial review and mentioning the fixes you've made on this tutorial.

authors: Srainyyyy breezy-codes

repo: thoth-tech/documentation


text:
Add test suite

Here's something I've been experimenting with lately. The tests/README.md file should, hopefully, explain the changes further. Let me know what you think.

Hey @mptre, this looks super cool!
When I tried compiling and running make test on my Mac this error is printed.
/Applications/Xcode.app/Contents/Developer/usr/bin/make -C tests
make[1]: *** No targets specified and no makefile found.  Stop.
make: *** [test] Error 2

Am I missing something?

Did you run the autogen.sh script prior make test? Your Makefile is
probably outdated.
On 5 February 2016 at 16:21, Calle Erlandsson notifications@github.com
wrote:

Hey @mptre https://github.com/mptre, this looks super cool!
When I tried compiling and running make test on my Mac this error is
printed.
/Applications/Xcode.app/Contents/Developer/usr/bin/make -C tests
make[1]: *** No targets specified and no makefile found.  Stop.
make: *** [test] Error 2
Am I missing something?
—
Reply to this email directly or view it on GitHub
https://github.com/thoughtbot/pick/pull/99#issuecomment-180398531.

--
:wq

This is beautiful.

Can this make use of the autotools TAP test infrastructure, such that the check target (and especially distcheck target) run the test suite?

https://www.gnu.org/software/automake/manual/html_node/Use-TAP-with-the-Automake-test-harness.html#Use-TAP-with-the-Automake-test-harness
https://www.gnu.org/software/automake/manual/html_node/Parallel-Test-Harness.html#Parallel-Test-Harness
It should be as easy as adding TESTS=test/tests.sh to Makefile.am, or thereabout.

@mptre Yeah, I did run autogen.sh. I'm not running GNU make. Could that be it?

My bad, forgot to commit the tests/Makefile. I did attempt to make it automake
compatible but it could most probably be improved. I also had to rename the
test-files since the tests/Makefile.in would be recognized as a test-file
otherwise.
On Fri, Feb 05, 2016 at 07:34:00AM -0800, Calle Erlandsson wrote:

@mptre Yeah, I did run autogen.sh. I'm not running GNU make. Could that be it?

Reply to this email directly or view it on GitHub:
https://github.com/thoughtbot/pick/pull/99#issuecomment-180403589

--
:wq

tests/test is being linked against ncurses. Overwriting AM_LDFLAGS in tests/Makefile.am does not help either. Shouldn't the src directory have a separate Makefile?

This project uses a non-recursive makefile. For some documentation on that, I recommend https://autotools.io/automake/nonrecursive.html .

Thanks for the info, make check is working by now. I still want to make the
test runner more robust to failures prior merging.
On Mon, Feb 08, 2016 at 11:26:41AM -0800, Mike Burns wrote:

This project uses a non-recursive makefile. For some documentation on that, I recommend https://autotools.io/automake/nonrecursive.html .

Reply to this email directly or view it on GitHub:
https://github.com/thoughtbot/pick/pull/99#issuecomment-181464911

--
:wq

The latest commit adds a timeout to the forked pick process in order to terminate if the process gets stuck or if the test case contains to few keystrokes.

Very cool, @mptre! Running this on Mac OS X 10.11.2, I get a "time limit exceeded" error on tests/01.in. I have tried upping the limit on tests/test.c:136 but it times out regardless of the limit. I interpret this as pick not understanding the input and not exiting.
Adding a #!/usr/bin/env bash to tests/test.sh fixes the time out on tests/01.in. My theory is that this has something to do with how echo works across /bin/sh and bash:
sh-3.2$ f=$(grep "^input: " < tests/01.in)
sh-3.2$ echo $f
input:

~/Projects/pick$ f=$(grep "^input: " < tests/01.in)
~/Projects/pick$ echo $f
input: \n

What can we do to make this more portable?

Next timeout is on `tests/03.in`. I think it might have something to do with the `\b` but I haven't found a way to debug this.

Do you have any tips for how to best debug this?

Calle,
On Fri, Feb 12, 2016 at 02:16:17AM -0800, Calle Erlandsson wrote:

Very cool, @mptre! Running this on Mac OS X 10.11.2, I get a "time limit exceeded" error on tests/01.in. I have tried upping the limit on tests/test.c:136 but it times out regardless of the limit. I interpret this as pick not understanding the input and not exiting.
Adding a #!/usr/bin/env bash to tests/test.sh fixes the time out on tests/01.in. My theory is that this has something to do with how echo works across /bin/sh and bash:
sh-3.2$ f=$(grep "^input: " < tests/01.in)
sh-3.2$ echo $f
input:

~/Projects/pick$ f=$(grep "^input: " < tests/01.in)
~/Projects/pick$ echo $f
input: \n

What can we do to make this more portable?

Next timeout is on `tests/03.in`. I think it might have something to do with the `\b` but I haven't found a way to debug this.

Do you have any tips for how to best debug this?

---
Reply to this email directly or view it on GitHub:
https://github.com/thoughtbot/pick/pull/99#issuecomment-183261648

I've manage to get it running on OS X by rewriting the field parser and removing
platform specific constant. The test runner has now been tested on FreeBSD,
Debian and OS X.
--
:wq

Awesome, @mptre! Passes on my machine! After a rebase I think this is ready merge. Maybe we should add these files to .gitignore?
test-driver
test-suite.log
tests/test
tests/test.sh.log
tests/test.sh.trs

Agreed, this is ready to be merged. There is still many test cases
missing but I hope that could be a collaborative effort.
Feel free to add the entries to .gitignore.
On Fri, Feb 19, 2016 at 05:55:22AM -0800, Calle Erlandsson wrote:

Awesome, @mptre! Passes on my machine! After a rebase I think this is ready merge. Maybe we should add these files to .gitignore?
test-driver
test-suite.log
tests/test
tests/test.sh.log
tests/test.sh.trs

Reply to this email directly or view it on GitHub:
https://github.com/thoughtbot/pick/pull/99#issuecomment-186222278

--
:wq

Any plans on getting the test suite to run on Travis? Would be great
since they both support GCC and Clang. While at it: we could make
linting the manpage part of the build process using mandoc(1)[1].
[1] http://mdocml.bsd.lv/man/mandoc.1.html
On Fri, Feb 19, 2016 at 05:55:22AM -0800, Calle Erlandsson wrote:

Awesome, @mptre! Passes on my machine! After a rebase I think this is ready merge. Maybe we should add these files to .gitignore?
test-driver
test-suite.log
tests/test
tests/test.sh.log
tests/test.sh.trs

Reply to this email directly or view it on GitHub:
https://github.com/thoughtbot/pick/pull/99#issuecomment-186222278

--
:wq

Running the suite on Travis would be great, and so would linting the man page be!

I've rebased this, added the mentioned files to .gitignore, added tests/test.sh to tests_test_SOURCES in order to get make distcheck to pass, and merged this as 475e1dc040d9cbe9ed445982b24d354d1b47a078, 88876e04e37596fe9909dd9d120a13cac9507fde, 72987aff761071398c18e3b88afcd04497cadb45.
Thanks for your great work, @mptre!

authors: calleerlandsson mike-burns mptre

repo: thoughtbot/pick


text:
"How to Get Involved" Page

We need a page that highlights the various ways to get involved with ThreeFold.

[ ] Define ways to get involved with ThreeFold
[ ] Create a rough proposal for how to lay this out
[ ] Send to Zia
[ ] Zia execute
[ ] Our team execute

Old issue but want to bring it up with the team

Indeed this should be discussed as if done well could save us a lot of energy and be a proof of how decentralized we can be

We're planning on doing this on the wiki for now but I will keep open to be done at a later date.


text:
Threelang  manual is not hooked in to the mdbook summary

So you can not reach it from the navigation

7264187567864389fc863109b45351d5c7a11e70


text:
High cpu usage when idle

I've noticed that when the farmer bot has nothing to do, it uses a lot of cpu:

This happens whenever the bot has finished performing any necessary actions, or if it simply isn't given any nodes to manage.
Meanwhile, I see get_my_twin_id running over and over again in the logs. I don't understand this behavior, since a twin is permanently associated with an account, as far as I know.

Alright I found the issue. It comes from baobab. Baobab is constantly looking into redis queues for jobs to execute (in a for loop). It looks like talking to redis is not very efficient. I'm looking further into the problem. Adding 100ms of sleep in the loop reduces the CPU usage to 0.6%

I created v0.1.0-rc12. Please check it out and let me know the cpu usage.

This helped, but I still think the approach can be improved: https://github.com/freeflowuniverse/baobab/issues/13

its very normal, in no language we should do an unlimited loop to a network
service,
if the network service is local you will burn through lots of CPU
the faster the language the more CPU will be used
On Sat, Mar 18, 2023 at 2:44 AM Scott Yeager @.***>
wrote:

Closed #9 https://github.com/threefoldtech/farmerbot/issues/9 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/threefoldtech/farmerbot/issues/9#event-8783025014,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABN6FVFVPW72YCZT22XYMNDW4TZNJANCNFSM6AAAAAAV53EBIQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

authors: brandonpille despiegk scottyeager

repo: threefoldtech/farmerbot


text:
Put Mycelium at forefront and wireguard+yggdrasil in advanced section

Situation

We want everyone to use Mycelium over the other network methods.
Mycelium setup should be clearly explained in the network/get started section
Wireguard + yggdrasil should be in advanced more
We should make sure all guides use Mycelium instead of WG/Yggdrasil

Todo

Put Mycelium at forefront and wireguard+yggdrasil in advanced section of networking access

We can close when #622 is merged.


text:
explain the minimum TFTs you need to make a deployment

need more explain the amount of TFT for every deployment
for example if a user need to deploy a zdb with size 10G what is the TFT needed for make this deployment
how to know resource units(CRU, MRU, NRU etc ....) ?

https://info.threefold.io/#/threefold__grid_pricing?id=cloud-unit-pricing
https://info.threefold.io/#/threefold__resource_units
https://github.com/threefoldtech/tfchain/issues/25#issuecomment-892746866

repo: threefoldtech/terraform-provider-grid


text:
Bridge: Learn how links to an outdated documentation

https://manual.grid.tf/threefold_token/tft_bridges/tfchain_stellar_bridge.html#how-to-use-the-tfchain-stellar-bridge

Update:
Investigation:
This issue is blocked on https://github.com/threefoldtech/info_grid/issues/301 and the documentation issue is also currently blocked since:

the instances for testnet and mainnet are not available yet
the steps can't be updated yet since profile manager extension isn't merged yet so the flow is not finalized yet to document.

verified and updated on devnet
version : 433ad62

https://app.testlodge.com/a/26076/projects/40893/suites/234374?expand_section=422339#case_4226598

Verified:

Testnet
2.3.0-rc1

repo: threefoldtech/tfgrid-sdk-ts


text:
Fix reserve action button

Description
Fix the reserve action button.
Changes

Reduce check time of Un/Reserved buttons
Check the twin ID of the rented node when the component is mounted
Reset values after reserve / unreserve process

Related Issues

https://github.com/threefoldtech/tfgrid-sdk-ts/issues/3575

Tested Scenarios

Navigate to any solution
Switch on dedicated nodes
Reserve any node and unreserve it

Updated Scenarios

Reserve from the node page and un-reserve any from the solution page.
Reserve from a solution page, then un-reserve from the node page.

Checklist

[x] Tests included
[x] Build pass
[ ] Documentation
[x] Code format and docstrings
[ ] Screenshots/Video attached (needed for UI changes)

cpd from https://github.com/threefoldtech/tfgrid-sdk-ts/pull/3650


text:
Psuedo random scenery [hackathon]

Hi all,
This is my submission for the hackathon.
Enjoy!

All very good suggestions! thank you!

Looks Great to Me! Good work @DefinitelyMaybe!

authors: DefinitelyMaybe ixxie

repo: threlte/threlte


text:
chore: fix rw-pass doc warnings

this removes nonexistent references.

#482
#481  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @pepyakin and the rest of your teammates on  Graphite

Merge activity

Oct 17, 3:49 PM EDT: A user started a stack merge that includes this pull request via Graphite.

authors: pepyakin rphmeier

repo: thrumdev/nomt


text:
there is memory leak at line number 76

consider breaking the reference to this object during dispose().

@jemisgoti Sorry for late response. I didn't noticed there was issue. I will try to investigate it when I have time.

consider breaking the reference to this object during dispose().

I have updated the code. Please check it if you have time.
Thank you.

authors: jemisgoti thruthesky


text:
error when runing" ./preprocess.sh "

/home/zpf/mycode/HyQuas/src/evaluator.h:117:5: error: array must be initialized with a brace-enclosed initializer
/home/zpf/mycode/HyQuas/src/evaluator.h:117:5: error: array must be initialized with a brace-enclosed initializer
/home/zpf/mycode/HyQuas/src/evaluator.h:117:5: error: array must be initialized with a brace-enclosed initializer
/home/zpf/mycode/HyQuas/src/evaluator.h:117:5: error: array must be initialized with a brace-enclosed initializer
/home/zpf/mycode/HyQuas/src/evaluator.h:117:5: error: array must be initialized with a brace-enclosed initializer
/home/zpf/mycode/HyQuas/src/evaluator.h:117:5: error: array must be initialized with a brace-enclosed initializer
src/CMakeFiles/QCSimulator.dir/build.make:924: recipe for target 'src/CMakeFiles/QCSimulator.dir/compiler.cpp.o' failed
make[2]: *** [src/CMakeFiles/QCSimulator.dir/compiler.cpp.o] Error 1
CMakeFiles/Makefile2:265: recipe for target 'src/CMakeFiles/QCSimulator.dir/all' failed
make[1]: *** [src/CMakeFiles/QCSimulator.dir/all] Error 2
Makefile:90: recipe for target 'all' failed
make: *** [all] Error 2

Seems to be a problem from g++ version. The system can be compiled with g++ 8.3.0.

I modify arrar in the evaluator.h, the error "array must be initialized with a brace-enclosed initializer" is canceled, but I come into another error,"fatal error: cuComplex.h: No such file or directory " and "fatal error: cuda.h: No such file or directory".  but /etc/local/cuda include the two header files.

You can try to write a simple C++ program that include these two headers to debug your environment variables first.

with this command "g++ -I /usr/local/cuda/include  -o test test.cpp" , my helloworld can be compiled, so how to set the environment variables?

i have fixed the error by specifying the environment variables, thanks!

repo: thu-pacman/HyQuas


text:
关于M4数据集的含义

您好，请问可不可以简单的介绍一下m4数据集中各个文件的含义？
比如其中的training.npz\ test.npz\ submission-Naive2.csv\ M4-info.csv这几项，非常感谢！！！

您好，感谢关注！
具体可以参考N-BEATS论文：https://github.com/ServiceNow/N-BEATS

您好，请问如何才能得到格式为npz的training与test文件？我通过GitHub链接访问M4数据集后，发现下载得到train和test文件夹，文件夹内全部为csv格式文件。期待您的回复，非常感谢！

authors: Sunwang-git vaeaea wuhaixu2016

repo: thuml/Time-Series-Library


text:
Questions about evaluating metric ACC

Thanks for your work, when I reproduced the classification problem, I found that the results obtained by the validation set and the test set during the training process were inconsistent, and the results of the test set after the training was completed were also inconsistent with the results obtained during the training process.
https://github.com/thuml/Time-Series-Library/blob/843ee18239b895b3018e7240002e85a0433e7f8b/exp/exp_classification.py#L128
https://github.com/thuml/Time-Series-Library/blob/843ee18239b895b3018e7240002e85a0433e7f8b/exp/exp_classification.py#L129
When I evaluate the validation set twice like this, the results will be different
vali_loss, val_accuracy = self.vali(vali_data, vali_loader, criterion)             vali_loss1, val_accuracy1 = self.vali(vali_data, vali_loader, criterion)             print(val_accuracy,val_accuracy1 )
I don't know if it's due to randomness or other questions, but I'm looking forward to your answers.Thank you.

I'm sorry for not getting back to you sooner. I have tried the code that you posted and got the same accuracy for val_accuracy and val_accuracy1. I guess that this may come from the randomness of devices. You can try to fix all the random seeds with the following codes
def seed_everything(seed):
random.seed(seed)
os.environ['PYTHONHASHSEED'] = str(seed)
np.random.seed(seed)
torch.manual_seed(seed)
torch.cuda.manual_seed(seed)
torch.cuda.manual_seed_all(seed)
torch.backends.cudnn.deterministic = True
torch.backends.cudnn.benchmark = False


text:
关于异常检测的test逻辑

为什么要将训练数据和测试数据concatenate来取异常阈值啊，异常检测是需要依赖训练集么？

您好，如果认为异常检测的场景是一个online系统的话，我们需要更长的序列来判断系统的整体情况，所以用到了训练数据集来寻找阈值。

好的，感谢


text:
Dataset link issue

Sorry but it seems the dataset link has been reset, could you please reupload the dataset? I'm very appreciate if you can do this

Dataset is in the data folder already.

authors: ElijahLai Ji-Xin

repo: thunlp/KNET


text:
config(prow/config,prow/jobs): offline support to repo ti-community-infra/ti-community-bot

ti-community-infra/ti-community-bot is archived.
Signed-off-by: wuhuizuo wuhuizuo@126.com

What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
What is changed and how it works?
What's Changed:
How it Works:

/approve

/approve

repo: ti-community-infra/configs


text:
feat: Adding more granular GitHub permission controls for owners

close https://github.com/ti-community-infra/tichi/issues/590

/hold

[ ] comments
[ ] docs

/close
The API does not work. See also: https://github.com/octokit/octokit.net/issues/2041#issuecomment-751321648

/reopen

/unhold
/assign @Mini256
PTAL~ Thanks!

😭 Deploy PR Preview 506703d failed. Build logs

🤖 By surge-preview

cc: @Mini256

😭 Deploy PR Preview 506703d failed. Build logs

🤖 By surge-preview

cc: @Mini256

This is probably because it was created by my token when it was first created, so it can't be updated after changing token.

This is probably because it was created by my token when it was first created, so it can't be updated after changing token.

Will subsequent PR be affected by this?

/merge

Will subsequent PR be affected by this?

It shouldn't.

repo: ti-community-infra/tichi


text:
Unable to install python-multipart for docker deployment

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
FROM tiangolo/uvicorn-gunicorn:python3.7

RUN pip install python-multipart
RUN pip install fastapi uvicorn
...

Description

On Friday, I can install fastapi and uvicorn without the warning of 'python-multipart' needs to be installed.
The issue starts to appear today.

Operating System
Windows
Operating System Details
No response
FastAPI Version
0.68.0
Python Version
3.7
Additional Context
No response

Resolved with requirments.txt
Docker
RUN pip install -r /app/requirements.txt

repo: tiangolo/fastapi


text:
itsdangerous Coding Compatibility[BUG]

Describe the bug
Under the same key, debuging mode decodes normally and production mode decodes incorrectly
To Reproduce
-------------one----------
from itsdangerous import JSONWebSignatureSerializer
import uvicorn
from fastapi import FastAPI
app = FastAPI()
@app.get("/")
def read_root():
    s = JSONWebSignatureSerializer(secret_key='123')
    a = s.dumps({'uid': 1, 'type': 101, 'scope': 'StudentScope'})
    return {"Hello": a }
if __name__ == "__main__":
    uvicorn.run(app, host="127.0.0.1", port=8000)

result:
{"Hello":"eyJhbGciOiJIUzUxMiJ9.eyJ1aWQiOjEsInR5cGUiOjEwMSwic2NvcGUiOiJTdHVkZW50U2NvcGUifQ.SGxpm7vl1qY4RFrvEGD5IlUrmQS6YQ8prkJdrW0TWgwuzkRskKkhmFa0h_oDLt70YbCAsYK0pZ7oLXJ4PzK5SQ"}
-------------one----------
-------------two----------
from itsdangerous import JSONWebSignatureSerializer
from fastapi import FastAPI
app = FastAPI()
@app.get("/")
def read_root():
    s = JSONWebSignatureSerializer(secret_key='123')
    a = s.dumps({'uid': 1, 'type': 101, 'scope': 'StudentScope'})
    return {"Hello": a }

sell:
uvicorn main:app --reload
result:
{"Hello":"eyJhbGciOiJIUzI1NiJ9.eyJ1aWQiOjEsInR5cGUiOjEwMSwic2NvcGUiOiJTdHVkZW50U2NvcGUifQ.Zch1Snt8meKjZzvACTF6dbqKsYqlYBcYCUH_fjJJlMw"}
-------------two----------
Expected behavior
the one is true
Environment:

OS: [e.g. Linux / Windows]
FastAPI =0.30.0
-python: 3.6.5

Thanks for reporting back and closing the issue :+1:


text:
Fix markdown code section rendering error in SQL tutorial

Currently the Migrations section and below is rendering incorrectly because of unused hl_lines sections. I think this should fix it.

@tiangolo Sorry, hope it's okay if I tag you here. Let me know if I can do anything more

Good catch @vegarsti ! :heavy_check_mark:
I guess it was probably related to a recent version of the docs plugins. I added your fix to the SQL tutorial for Peewee that had the same issue.
Thanks for your contribution :rocket: :cake:

authors: tiangolo vegarsti


text:
Fix models to support pydantic orm_mode

Null (None) values will cause ValidationErrors when used with postgres.
See: https://github.com/tiangolo/fastapi/issues/361#issuecomment-509764272
I made these changes and it works like a charm. (The issue out-of-the-box being the None value for first superuser's Full Name)
FastAPI version: 0.33.0
Pydantic version: 0.30
Example validation error
INFO: ('192.168.0.2', 53732) - "POST /api/v1/login/test-token HTTP/1.1" 500
ERROR: Exception in ASGI application
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/uvicorn/protocols/http/httptools_impl.py", line 368, in run_asgi
    result = await app(self.scope, self.receive, self.send)
  File "/usr/local/lib/python3.7/site-packages/starlette/applications.py", line 133, in __call__
    await self.error_middleware(scope, receive, send)
  File "/usr/local/lib/python3.7/site-packages/starlette/middleware/errors.py", line 122, in __call__
    raise exc from None
  File "/usr/local/lib/python3.7/site-packages/starlette/middleware/errors.py", line 100, in __call__
    await self.app(scope, receive, _send)
  File "/usr/local/lib/python3.7/site-packages/starlette/middleware/base.py", line 25, in __call__
    response = await self.dispatch_func(request, self.call_next)
  File "./app/main.py", line 34, in db_session_middleware
    response = await call_next(request)
  File "/usr/local/lib/python3.7/site-packages/starlette/middleware/base.py", line 45, in call_next
    task.result()
  File "/usr/local/lib/python3.7/site-packages/starlette/middleware/base.py", line 38, in coro
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.7/site-packages/starlette/middleware/cors.py", line 84, in __call__
    await self.simple_response(scope, receive, send, request_headers=headers)
  File "/usr/local/lib/python3.7/site-packages/starlette/middleware/cors.py", line 140, in simple_response
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.7/site-packages/starlette/exceptions.py", line 73, in __call__
    raise exc from None
  File "/usr/local/lib/python3.7/site-packages/starlette/exceptions.py", line 62, in __call__
    await self.app(scope, receive, sender)
  File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 585, in __call__
    await route(scope, receive, send)
  File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 207, in __call__
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 40, in app
    response = await func(request)
  File "/usr/local/lib/python3.7/site-packages/fastapi/routing.py", line 122, in app
    skip_defaults=response_model_skip_defaults,
  File "/usr/local/lib/python3.7/site-packages/fastapi/routing.py", line 54, in serialize_response
    raise ValidationError(errors)
pydantic.error_wrappers.ValidationError: 1 validation error
response
  value is not a valid dict (type=type_error.dict)

I made these changes and it works like a charm

Happy to second that ;)
I think its fairly urgent to merge this PR, since it does break the test suite.

perhaps running CI nightly is of interest, that'll help catching upstream changes
given the in the Pipfile the exact version of the dependencies isn't specified, this might help to catch such breaking changes.

I came across the same issue, thank you for posting a PR @ac3d912. Also thank you @tiangolo for this project.

Just ran into this issue as well. Appreciate you identifying it and submitting a PR @ac3d912. Saved me a lot of time.
FastAPI is awesome @tiangolo. Thanks for all the work you've put into creating and maintaining it.

Here is a :+1: from me too, for this lifesaving tip!  This ought to be merged!

Also regarding @jf--- comment about upstream changes, I hope my recent #69 will be taken into consideration, as it will potentially eliminate such upstream cases...?

Thanks @ac3d912 for debugging and implementing it! :tada: :rocket:
Thanks everyone for the discussion here (and the patience :sweat_smile: ).
@StephenBrown2 , yep, thanks. This was fixed as part of that PR.
I'll close this PR now as the fix is already implemented (from a different PR).

authors: ac3d912 bossjones jf--- stratosgear tiangolo victorbordo

repo: tiangolo/full-stack-fastapi-postgresql


text:
[BUG] Optional multi-value option returns empty tuple instead of None

Describe the bug
Optional multi-value option returns empty tuple instead of None.
To Reproduce

Create a file main.py with:

from typing import List, Optional

import typer

app = typer.Typer()

@app.command()
def main(jobs: Optional[List[str]] = None): # or `typer.Option(None)`
    print(jobs)

if __name__ == "__main__":
    app()

Call it with:

python main.py

It outputs:

()

But I expected it to output:

None

Expected behavior
The expected behavior for an empty parameter annotated with Optional and = None is both documented and commonly known - it should be just None. And that's also how it works in Typer, but for some reason - only for single-value options.
Environment

OS: macOS
Typer version: 0.3.2
Python version: 3.8.5

I encountered the same problem as well. Did you find a workaround?

Confirmed that this still seems like an issue, though the output on the current version of Typer v0.9.0 is [], not () (which is still not the expected None)

PR to fix this issue: https://github.com/tiangolo/typer/pull/664

authors: jaklan jaywonchung svlandeg

repo: tiangolo/typer


text:
best performance training config on nuscenes

VoxelNet

Model
Validation MAP
Validation NDS
Link

centerpoint_voxel_1440
59.6
66.8
URL

PointPillars

Model
FPS
Validation MAP
Validation NDS
Link

centerpoint_pillar
31
50.3
60.2
URL

Hi,  I want reproduce the voxel and pointpillars results above, the CONFIG_PATH should be what?
python -m torch.distributed.launch --nproc_per_node=2 ./tools/train.py CONFIG_PATH

THanks!
:)

this one https://github.com/tianweiy/CenterPoint/blob/master/configs/nusc/pp/nusc_centerpoint_pp_02voxel_two_pfn_10sweep.py
for a slightly better result, you can use https://github.com/tianweiy/CenterPoint/blob/master/configs/mvp/nusc_centerpoint_pp_fix_bn_z_scale.py
it has slightly better data augmentation parameters and achieve 52.4 map / 61.5 nds in my experiments. models are available at https://github.com/tianweiy/MVP#main-results

this one https://github.com/tianweiy/CenterPoint/blob/master/configs/nusc/pp/nusc_centerpoint_pp_02voxel_two_pfn_10sweep.py
for a slightly better result, you can use https://github.com/tianweiy/CenterPoint/blob/master/configs/mvp/nusc_centerpoint_pp_fix_bn_z_scale.py
it has slightly better data augmentation parameters and achieve 52.4 map / 61.5 nds in my experiments. models are available at https://github.com/tianweiy/MVP#main-results

thanks for your reply.
Now i have the answer of best result of centerNpoint based on pointpillars.
could you tell me the config path pf best result of centerpoint based on voxelnet? @tianweiy
thanks!
:)

authors: TianhaoFu tianweiy

repo: tianweiy/CenterPoint


text:
Logger should not be used for improper debugging

As this is used for SLA logging. Any attempt to write as a standard log generates  a bad behaviour.

Thanks @fabienbussinger for your PR,
However I strongly advise you to change this behaviour by extending the DefaultMetrikHandler by your own.
public class BussingerMetrikHandler extends DefaultMetrikHandler {

    @Inject
    public BussingerMetrikHandler(SpringMetrikProperties properties) {
        super(properties);
    }

    @Override
    protected String getObjectProperty(Object bean, String propertyName, String orElse) {
        try {
            return this.formatObject(PropertyUtils.getProperty(bean, propertyName));
        } catch (Exception e) {
            return orElse;
        }
    }
}

And then register this bean in your MetrikConfiguration like :
@Configuration
public class BussingerConfiguration {

    @Inject
    private SpringMetrikProperties props;

    @Bean
    public MetrikHandler defaultMetricHandler() {
        return new BussingerMetrikHandler(this.props);
    }
}


text:
上传大文件在1小时的时候出现：文件分片上传顺序错误，开始重新上传文件

日志.txt
日志太多了 我是从开始报错的地方截取的
每次都是 1小时零几秒的时候就出现 文件分片上传顺序错误，开始重新上传文件
昨天晚上挂了一晚上，早晨起来一看 每次都是1小时零几秒就开始重新上传，一晚上重新上传了N次

问题出在 internal\functions\panupload\utils.go
func IsUrlExpired(urlStr string) bool {
u, err := url.Parse(urlStr)
if err != nil {
return true
}
expiredTimeSecStr := u.Query().Get("x-oss-expires")
expiredTimeSec, _ := strconv.ParseInt(expiredTimeSecStr, 10, 64)
if (time.Now().Unix() - 10) >= expiredTimeSec {
// expired
return true
}
return false
}
我改成
func IsUrlExpired(urlStr string) bool {
u, err := url.Parse(urlStr)
if err != nil {
return true
}
expiredTimeSecStr := u.Query().Get("x-oss-expires")
expiredTimeSec, _ := strconv.ParseInt(expiredTimeSecStr, 10, 64)
if (time.Now().Unix() + 300) >= expiredTimeSec {
// expired
return true
}
return false
}
之后就会在55分左右的时候更新上传地址，然后就没有问题了。已经成功上传了

问题出在 internal\functions\panupload\utils.go
func IsUrlExpired(urlStr string) bool { u, err := url.Parse(urlStr) if err != nil { return true } expiredTimeSecStr := u.Query().Get("x-oss-expires") expiredTimeSec, _ := strconv.ParseInt(expiredTimeSecStr, 10, 64) if (time.Now().Unix() - 10) >= expiredTimeSec { // expired return true } return false }
我改成
func IsUrlExpired(urlStr string) bool { u, err := url.Parse(urlStr) if err != nil { return true } expiredTimeSecStr := u.Query().Get("x-oss-expires") expiredTimeSec, _ := strconv.ParseInt(expiredTimeSecStr, 10, 64) if (time.Now().Unix() + 300) >= expiredTimeSec { // expired return true } return false }
之后就会在55分左右的时候更新上传地址，然后就没有问题了。已经成功上传了
好的，我也统一改成5分钟

有做了优化，请在新版中验证

authors: Adoirsa tickstep

repo: tickstep/aliyunpan


text:
上传文件失败，提示打开上传未完成数据库错误:

[root@centos76 backup]# pwd
/backup
[root@centos76 backup]# ls
2023-03-02.tgz  logs  video
[root@centos76 backup]# /usr/local/aliyunpan/aliyunpan upload --norapid --np -ow 2023-03-02.tgz /Backup/tenxunyun
[0] 当前文件上传最大并发量为: 10, 上传分片大小为: 10.00MB
打开上传未完成数据库错误: panupload.UploadingDatabase.UploadingList: []*panupload.Uploading: panupload.Uploading.LocalFileMeta: Path: readObjectStart: expect { or n, but found ", error found in #10 byte of ...|[{"path":"/home/git/|..., bigger context ...|{"upload_state":[{"path":"/home/git/gogs/gogs","length":50067592,"modtime":1|...
[root@centos76 backup]#

补充 aliyunpan version v0.2.6

通遇到一样的情况

[root@centos76 backup]# pwd /backup [root@centos76 backup]# ls 2023-03-02.tgz logs video [root@centos76 backup]# /usr/local/aliyunpan/aliyunpan upload --norapid --np -ow 2023-03-02.tgz /Backup/tenxunyun
[0] 当前文件上传最大并发量为: 10, 上传分片大小为: 10.00MB 打开上传未完成数据库错误: panupload.UploadingDatabase.UploadingList: []*panupload.Uploading: panupload.Uploading.LocalFileMeta: Path: readObjectStart: expect { or n, but found ", error found in #10 byte of ...|[{"path":"/home/git/|..., bigger context ...|{"upload_state":[{"path":"/home/git/gogs/gogs","length":50067592,"modtime":1|... [root@centos76 backup]#

遇到同样问题，请问解决了吗？

同样的问题。有解决办法吗？
版本:aliyunpan version v0.3.4


text:
Updated DIVA codes

Description
Updated WienerLinien App:

Station search results are not alphabetical ordered.
Small bug corrections.

Copilot

🤖 Generated by Copilot at c4466f4
Summary
🚇📝🌐

This pull request adds a new Tidbyt app called Wiener Linien Info that shows public transit information for Vienna. It includes a manifest.yaml file and a README.md file for the app.

Wiener Linien Info
A Tidbyt app for transit
Manifest and README

Walkthrough

Create a new Tidbyt app called Wiener Linien Info that displays real time public transit information for Vienna, Austria (link, link)
Add a manifest.yaml file that defines the app metadata, such as id, name, summary, description, author, file name, and package name (link)
Add a README.md file that explains the app overview, features, parameters, and legal notices, and shows an example image of the app output (link)

Hey @davidbaco. This PR has some sort of conflict that's making it near impossible to review. As if the app was brand new. I don't know what Github's up to here.
Could you create a fresh fork of the community repo, apply your changes there and cut a new PR? Hopefully that resolves is.

authors: davidbaco matslina

repo: tidbyt/community


text:
schema: Strongly type defaults and config values

Support config values of type other than string. Previously we just
assumed all values were strings, which is kind of crappy for things like
toggles where we were checking stuff like config.get("X") == "true".
Instead, we now check against the schema and provide a value of the
correct type to Starlark. We still serialize everything as a string for
storage, but we deserialize it to the appropriate type.

@betterengineering Turned out to be a bit more code than I imagined... happy to review IRL if necessary.

repo: tidbyt/pixlet


text:
fix argument suffix to name

suffixだと意味が合わない、むしろ混乱の元になるので修正したい
githubで検索しても、${suffix} よりも ${name} のほうが圧倒的多数なので name に変更する
（むしろsuffixを使用しているのはautoware関係のみっぽい・・・？）
参考URL
https://github.com/search?q=name%3D"%24{suffix}_joint"&type=Code
https://github.com/search?q=name%3D"%24{name}_joint"&type=Code
要確認事項
以下、今回のPRに影響を受けるため、修正が必要な各PR
https://github.com/tier4/aip_customized_description.logiee_s-tc/pull/6
https://github.com/tier4/aip_description.iv/pull/2
https://github.com/tier4/crow_sensor_description.iv/pull/2

descriptionはちょっと把握していないので、 @yukkysaito さんにお願いしても良いでしょうか？ :pray:

CAT vehicleを参考にしてsuffixになっていましたが、nameでも良いと思います！

authors: h-ohta kenji-miyake yukkysaito


text:
大神代码逻辑结构真好

赞赞赞

谢谢关注

二分法没有判断就只有一个数据的情况，会出现死循环

是的，已经解决了

如果数据小于一屏的话，缩放不会改变时间轴的时间

大神请问一下，Scale，缩放可以向左边缩小吗
现在是向右边缩小，就是缩小到小于一个屏幕的时候，空白的区域在左边

设计的时候，这个view是从右向左滑动的（刚开始进入的时候查看最新的数据），所以右边是顶部，左边是底部，所以缩放的时候是以右边为基准的，空白就留在了左边部分

authors: billsinla tifezh

repo: tifezh/KChartView