any open source website & demo power by sanic?

any open source website & demo power by sanic?

it would be nice to have examples project in README.md

https://github.com/HackerDom/ructfe-2016/tree/master/services/atlablog -- It is web-service based on sonic. It was developed for hacker competitions. It my be helpful for education purposes.
In the project you can see the concept of reusable models paired with Blueprint.
Model Entry -- https://github.com/HackerDom/ructfe-2016/blob/master/services/atlablog/entries/models.py#L19-L25
Entry uses for store in DB BlogPosts, Comments and FIleAttachments.
The key point here: https://github.com/HackerDom/ructfe-2016/blob/master/services/atlablog/main.py#L89-L91
(example of reusable Blueprint for storing Entry model in different DB tables)

https://securitynode.com Internal APIs are powered by sanic. Public API will be released soon as this site is currently under development. DISCLAIMER: I am the developer/owner of the site.

it seen like flask, can it use flask plugin add on?

@decemer No unfortunately while sanic may seem similar to flask in it's outward facing api, it's internals are very different, leading to incompatibility with flask plugin add ons.

We use sanic in our event push notification because of it's speed. And there's no UI so I can't showcase it.

I wrote a PasteBin clone a while ago, as a Flask demo project. I might make a Sanic fork of it if there is interest, but I could use some seasoned feedback in it, given that I have not used Sanic and asyncio extensively and will probably result in some bad usage practices.

Luizberti, the link http://api.cbeam.io/ is wrong or broken and browser get 404.

@KeyJoo as I mentioned in the sidenote, there is no front end on it (yet).
You can go to http://api.cbeam.io/ping to see that the Flask app is really running.
And if you have httpie installed you can use it like:
http --form POST api.cbeam.io/submit Accept:text/plain content='something'

http GET cpi.cbeam.io/<paste id> Accept:text/plain

Update on the Demo Project
I just finished rewriting the Flask demo in Sanic's lingo. The benchmarks against the old version are insane, beat Flask+Gunicorn by a whopping margin (did 760k requests in the same amount of time Flask took to do 16k on a single core). I have a feeling that if I switched to aioredis too performance would go bonkers, but this is good enough for now.
The whole thing is done in roughly 80 tidy lines of code and it reads nicely. Right now it's up and running at http://api.cbeam.io/ and it has replaced the Flask version entirely. I would like some feedback on it if anyone would care to take a look, if you can think of any other use cases I could add that exemplify other features of Sanic feel free to open an issue in the tracker.
Hopefully this can help newcomers get goin' faster faster 😄
link: https://github.com/luizberti/codebeam-api

FYI, I've tried this as well, and aioredis murders my performance. I have one system that does event logging to disk that runs Sanic, and it has insane performance. I tried porting my API gateway to Sanic, and it's terrible - almost all because of aioredis.
 I also considered aredis (https://github.com/NoneGG/aredis), but their benchmarks show the same thing.
Having said all of that, if you can make this work at a level above Flask, that would be awesome. Just letting you know that it might be out of your control without rewriting aioredis.

Another example:
https://github.com/stopspazzing/Sanic-Server-with-MDL-Blog-Template

can we close this or: https://github.com/channelcat/sanic/issues/396 ?
No need for both

Closed in favor of #396


client: Strange stats on some errors

https://pkg.go.dev/github.com/nspcc-dev/neofs-sdk-go/client#PrmInit.SetStatisticCallback option allows to enable internal tracktion of op execution. For some errors, submitted stats look like a successful op completed insta
for example, ObjectDelete on ErrMissingSigner
Current Behavior
some errors are nil-ed and still submitted to the stat handler. With this, such results cannot differ from OK in general. At the same time, some homogeneous errors are transmitted as is (e.g. ObjectHash)
Expected Behavior
either

skip stats for invalid input completely
pass any error to the handler as it is

Possible Solution
i can suppose that it was an attempt to implement 1 to exclude user input errors which is not w/o sense. But for this I'd introduce a classification of errors accessible to the external user and let him decide for himself. Although in this case the internal stats lose their meaning becoming completely implementable around the base client
afaik all current errors are exported and documented, so 2 can be done
Steps to Reproduce
see exec statistics tests from https://github.com/nspcc-dev/neofs-sdk-go/pull/641
Regression
no i guess
Your Environment

Version of the product used: 335d9fe90f24494944c2888ef385f06f5f81dcfc

2 is the right thing to do here. It is a call. It has an error and time of some kind.


Remove bitrotted fuzz crate

What was wrong?
The nascent fuzzing stuff in fe/fuzz/ hasn't been touched in a while, isn't going to be touched anytime soon (as we're focusing on other priorities), and doesn't currently build. I suggest we scrap it and resurrect or recreate it at some point in the future.
How was it fixed?
rm -rf

Codecov Report

Merging #686 (45bcc82) into master (a86a9fb) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #686   +/-   ##
=======================================
  Coverage   82.11%   82.12%
=======================================
  Files         125      125
  Lines       10849    10849
=======================================
+ Hits         8909     8910    +1
+ Misses       1940     1939    -1

Impacted Files
Coverage Δ

crates/analyzer/src/context.rs
62.18% <0.00%> (+0.84%)
:arrow_up:

Continue to review full report at Codecov.

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


perf(scanner): store and reuse next_indent_length

Whether an indent/dedent token is generated for a comment depends on whether the indent/dedent is persisted after the comment. This takes O(comment_length) to check. Checking it for every line takes O(comment_length * comment_lines), which is quadratic.
An optimization was made in a901729 which skips this check when there is no possible indent/dedent, such as in the following:
# comment 1...
# comment 1000
print("foo")

However, the check cannot be skipped in the following case:
class Foo:
    def foo():
        print("bar")
    # comment 1...
    # comment 1000
    def bar():
        print("foo")

which comes up when commenting out code.
This PR solves optimizes all cases by storing and re-using the next non-comment indent length and re-using it while still valid (requires TSLexer.get_byte from upstream tree-sitter).
Sample file: https://gist.github.com/llllvvuu/bc80d4df5f39cc27c13297e1d3a39190
Before:
tree-sitter parse ~/test/repro2.py  0.52s user 0.01s system 99% cpu 0.535 total

After:
.../release/tree-sitter parse   0.01s user 0.01s system 88% cpu 0.023 total

Fixes: https://github.com/nvim-treesitter/nvim-treesitter/issues/4839

Required upstream change: https://github.com/tree-sitter/tree-sitter/pull/2648


Is this abandoned?

Title says it all.  There doesn't seem to be activity on this project.

Probably you're looking for https://github.com/Emurgo/cardano-serialization-lib
This is the new project which is compatible with current Cardano mainnet

I am no sure, I'd like to use this sdk to play with Cardano in my Rust app....

@mendesbarreto If you're talking about this library, then no.  If you're talking about the one he linked, https://github.com/Emurgo/cardano-serialization-lib, then yes.  It's got an MIT license.

Shouldn't the repo be archived? It's misleading otherwise if you aren't looking closely

Yes, it should probably be archived.  Anyone have control of this repo?

Yes, it probably should.
@SebastienGllmt @vincenthz @mzabaluev @NicolasDP  -  Do any of you have control of this repo to archive it?


AAP-[5705] Project endpoints update description and name

AAP-5705
Jira ticket: AAP-5705
Small updates for project endpoints:

create endpoint returns a 201 status and only allows unique name parameter
description parameter added to update method (both name and description are optional)

Testing instructions:

Create Project: POST http://localhost:8080/api/projects/

{
    "url": "https://github.com/benthomasson/eda-project",
    "name": "testing",
    "description": "testing"
}

Should receive a 201 status and response of the project object

Create Project: POST http://localhost:8080/api/projects/ with the same body should get a 422 response with the following body

{
    "detail": "Project with name 'testing' already exists"
}

Update Project: PATCH http://localhost:8080/api/projects/{id} (should be able to pass full body or 1 parameter)

{
    "name": "update test",
    "description": "update testing"
}

@cutwater please review

@shannon-donahue super minor, but can you update the PR title to enclose the whole AAP issue in square brackets? (i.e. [AAP-5705])


Change the coordinates(?) of a main menu button with a .JSON file.

Now that I think of it, it's kind of similar to #9058, but anyways, here's this thing I guess.
This YT video shows how it works. (I copied the file from the video.)
https://www.youtube.com/watch?v=hD2aEAMbGb8

you could just make a PR on my repo for it

would help a lot honestly, i've been trying to make it for a while now

actually that sounds like a good idea

I actually saw that in a tutorial

although it doesn't work like that at all, it's not just copying the file and that's it, it's more complicated, you know, modify this, modify that, anyway I'll leave you the tutorial where I got that from, say hi to the guy who did it for me
https://youtu.be/QZQJ701tAqQ

oh okay

by the way, you should see the other videos of the mod, you might be interested

cool

This does literally nothing??

This doesn't do shit.

i agree he should've put the haxe code of the thingamajig, but nope
just the json file thats so lazy

Excuse me I didn't know there was more stuff to make the JSON actually work.

yea, i think you might have to put the json stuff in the main menu state hx file and not just the json file
otherwise its never gonna work

yea, i think you might have to put the json stuff in the main menu state hx file and not just the json file otherwise its never gonna work

okay


Application

Pull request for the application branch

@AliRezaTaheriyoun can you describe a bit in the overview or a comment what is actually in this pull request?

Also I was talking with @grahnavard and I think we want to make a standard folder directory for the examples. In there should probably just be something that looks like this:
examples/
--- Breastmilk/
      --- breastmilk.ipynb
      --- output/
            --- breastmilk_models.pkl
      --- data/
            --- infant_metadata.csv
            --- infant_metabote.tsv
            --- stool_metadata.csv

Please make sure to only push exactly what is needed to reproduce the output!

@alleneross I have changed the appearance of the "Read me.md" markdown file and have fixed the links in it.
In the Application branch I have provided '.ipynb' notebook files concerning the application of waveome onto:

Breastmilk study gut microbiome
Breastmilk study metabolites,
A simple bivariate CD4 count for HIV cohorts.
All the ipython notebook  files provide the step-by-step descriptions from definition of the variables towards the kernel search procedure.
Since sometimes the kernel search requires a considerable time for running, I have also add some lines to show how users can save the output of GPSearch and use it offline. However, I have not followed the standard folder structures yet. So, if it is advisable let me first make the folder structures and then I will push the changes into Application branch and will make a new PULL request.

I believe you should restructure the directory as stated above for the breastmilk and CD4 folders and then push your changes to the remote repository to the same Applications branch and it should automatically update this pull request. You shouldn't need to make a new request.

Thank you. I will do so tonight if I may.


RequestForComment - Enhancement common.states

Hi @SVallant,
we plan to enhance the definition of common.states  definitoipn as described in https://github.com/ioBroker/ioBroker.docs/pull/387/files
Please provide feedback and also an info if you would support this as adjustment in the adapter in the future.
Thank you
Ingo

Hi Ingo,
good job, i will support it
Sepp

it is merged in the "docs" ... have fun :-)


Kanban review

https://github.com/TechnoAndy/Kanban-Board/projects/1

We are two people on our team

TechnoAndy,
henrykgggg

Hi @Team,
Status: Approved :heavy_check_mark: :heavy_check_mark: :heavy_check_mark:

To Highlight 💪🏻

Kanban Board is Ready ✔

Good job implementing the required changes :+1: Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Optional Suggestions  :

N/A

Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag  @ichala in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


[FRE-1068, FRE-1069] add modal animation

https://github.com/user-attachments/assets/665805b5-a297-4fff-932e-8fed4065f081

looks good but I'm not sure if we should have these for all modals or only for drawer, I'm gonna ask jeremy and sonla


:seedling: Add golang version check to verify

This makes sure that we are not exceeding our golang version Supported golang version is 1.22.5

Description

Reviewer Checklist

[ ] API Go Documentation
[ ] Tests: Unit Tests (and E2E Tests, if appropriate)
[ ] Comprehensive Commit Messages
[ ] Links to related GitHub Issue(s)

All passing now

Closing in favor of #1264


Remove and gitignore AssetPackCreator folder

This folder shouldn't be exposed in your repo. Add it to your .gitignore if you need it to remain in the folder but not be tracked by git.

Thanks for the notice! I forgot to commit the changes after publishing the last version to PDX.


Fix broken specs book link

I think specs got moved under amethyst and the book got moved. This should also be an evergreen link 😄

That looks good - thank you! I'll get it merged shortly, and pushed live next time I hit "publish".


allow user to provide a purestake api key

This change allows a user to provide a purestake api key which will be used when submitting network operations.
This is a proof-of-concept approach where the api key is provided directly as an argument down the call stack.
There may be a cleaner design that doesn't require the api key to be a part of so many signatures but I'd like to get feedback at this point before proceeding.

Looks good! The Rust code is stateless, adding the parameter to all the functions that need it is perfectly fine.

Very nice! Before you added the popup I was planning to do exactly that, with the same question mark and everything haha. Didn't know about the tutorial, it's perfect!


Missing implementation of abstract method...

Hi, you forgot to implement method (public function serializeObjectOfType(object $object, string $className): mixed; ) from interface in OptimizedMapper :(
PHP Fatal error:  Class ***\Infrastructure\Serialization\Mapper\OptimizedMapper contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (EventSauce\ObjectHydrator\ObjectMapper::serializeObjectOfType) in /var/www/html/vendor/***/src/Infrastructure/Serialization/Mapper/OptimizedMapper.php on line 16

Hi, I think you need to re-generate your mapper. The generated mapper should contain it: https://github.com/EventSaucePHP/ObjectHydrator/blob/main/src/ObjectMapperCodeGenerator.php#L125 The tests also always validate that the method is there.


tentools

Describe your problem here.
It tells me to install tentools, where do i install tentools
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Nothing edited

Here is tentools https://github.com/TentaRJ/tentools


Convert Interface to Abstract Class

This feature would look for base types of a class which implements an interface.  It checks for any conflicts, i.e. for types implementing the interface that already have base type (an error in C#, which has no multiple inheritance).
This was motivated by a feature found in Resharper 9. I thought this could be some neat feature in Code Cracker

@Kavignon Nice one. But tricky. It has to go through all the compilation to be able to check all the classes and make sure no class is implementing the interface and also already inheriting from another class. This will raise a problem we have with another analyzer (see NoPrivateReadonlyFieldAnalyzer). Also see the related bug: #293.
This would be a refactoring, right?

What problem will it raise ? I thought we could just look for base types of a class, look for those which implements an interface. Or maybe i don't really understand the situation.

@Kavignon You can't "find all references" on an analyzer because you can't (or at least shouldn't) call into async methods, await or .Result. Analyzers are supposed to be fast.
So what we do in these situations is that we would register on compilation start to inspect all class declarations, verify if there is any where the problem happens. We also register on compilation end to verify if there is any interface that would be problem free, and then we could raise the diagnostic. Until the compilation has ended we cannot raise the diagnostic.

Ok, I thought of a way to deal with the problem.
We could raise a hidden diagnostic that would be picked up from a code fix provider. The code fix provider is able to do the async work. If it finds a situation where it cannot offer the fix, it doesn't.
I moved it to Ready, and it is now up for grabs.


Multi engine dbal, support sqlite.

Adding more functionality, factory instance management to the dbal (database services).
Better demo will come tonight with interactive code -> result examples.

@tomvlk  That Design of your Services implementation looks spectacular better! Congratulation!

@LuckyCyborg Thanks, let's get it ready, I will post a way more better example of using databases with and without the dbal tonight. Demo with code => result stuff
:christmas_tree: :santa: :gift:

@tomvlk Yeah, Tom! Let's get it ready! :+1:
With this pull-request of yours, I'm ensured that, deadline-d or not, we arrive to have an almost stable Nova Framework this Year, as in Release Candidate at minimum...  :smile:

@tomvlk How about us to ask Dave to merge this pull-request, and you to open another one, when you are ready to complete the stuff and demos? :smirk:


Run Video-LLaMA on A10 24GB

I was wondering if it is at all possible to run inference and/or finetune the model on one or a bunch of A10 24GB GPUs?
Thanks.

You can run inference with INT-8 on A10 GPUs (7B-sized Video-Llama can definitely work).


Fix crash when SpecialSeparator = "" (empty string) and Fix InPacket for Player (in 1 ...)

https://github.com/NosCoreIO/NosCore.Packets/issues/100

Propably wrong is having 2 defenitions of in packet
I mean 2 Classes have  [PacketHeader("in")].
I think nice idea could be  [PacketHeader("in 1")] but this dont work for me.


BUG: run_openai_gpt.py load ROCStories data error

🐛 Bug
Model I am using (Bert, XLNet....): GPT
Language I am using the model on (English, Chinese....): English
The problem arise when using:

[ 1 ] the official example scripts: run_openai_gpt.py

The tasks I am working on is:

[ 1 ] an official GLUE/SQUaD task: ROCStories

The method of preprocess of ROCStories use ids 0 pad input, while 0 is actually the id of unk_tokens.
The code of method is as fellowing:
def pre_process_datasets(encoded_datasets, input_len, cap_length, start_token, delimiter_token, clf_token):
    tensor_datasets = []
    for dataset in encoded_datasets:
        n_batch = len(dataset)
        input_ids = np.zeros((n_batch, 2, input_len), dtype=np.int64)
        for i, (story, cont1, cont2, mc_label), in enumerate(dataset):
            with_cont1 = [start_token] + story[:cap_length] + [delimiter_token] + cont1[:cap_length] + [clf_token]
            with_cont2 = [start_token] + story[:cap_length] + [delimiter_token] + cont2[:cap_length] + [clf_token]
            input_ids[i, 0, :len(with_cont1)] = with_cont1
            input_ids[i, 1, :len(with_cont2)] = with_cont2

input_ids initialize with 0, which is the id of unk_token, rather than id of pad_token.

Hi @nine09, thanks for the report. Any way you could fix it cleanly and open a pull request?

Hi @nine09, thanks for the report. Any way you could fix it cleanly and open a pull request?

Bien sur! But I need some clues about whether GPTTokenizer already contains some special tokens and how to add special tokens to GPTTokenizer and resize the embedding size of GPTModel.

Hi @nine09, thanks for the report. Any way you could fix it cleanly and open a pull request?

Bien sur! But I want to have some clues about whether GPTTokenizer already have pad_token, otherwise add a new pad_token need resize embedding of GPTModel.

I have fix the bug at #1064, that change add a pad_token to GPTTokenizer so that solved this problem.


Integrate build pipeline with JFrog Build-Info

🧹 Tech Debt
We should define and document a process for integrating with the JFrog CLI and create a Build in Artifactory that includes the NPM packages used to build the package.
We can use this Build to define a security and license violation policy, and use Artifactory to manage the approval and exception process. We can also use it to manage all the third-party licenses used within our Angular apps.
References: https://github.com/marketplace/actions/setup-jfrog-cli
This might allow us to remove npm audit from our pipeline.
We should investigate whether this can replace/augment Dependabot workflows for tracking out-of-date deps.

The current direction is that nimble as a public package would be managed in the public npm registry and we use renovate for dependency updates. So maybe this can be closed @fredvisser ?

Closing as low ROI. We can revive if it becomes a priority.


Upload artifact to GH artifacts?

Since this runs as a reusable workflow, I can't run a step after in the same VM requesting to upload the artifact so a further job can consume it.
Does it make sense to upload the artifact as a step in https://github.com/ianlewis/slsa-byob-template/blob/a047bb0b0f7db5020b6172ed08e922b4cca25377/internal/action-wrapper/action.yml#L40?
rr

Yeah, It could be done here too
https://github.com/ianlewis/slsa-byob-template/blob/a047bb0b0f7db5020b6172ed08e922b4cca25377/.github/workflows/slsa3.yml#L77-L99

I think it makes more sense to do it inside the tool, only because the tool has actual knowledge of the path it writes to (and there are no outputs to the build step)

I believe this is fixed!


Are you using infolink ads on the demo site?

Thank you for your excellent theme.
By the way, I saw demo site. And, many pop-up ads were shown in there.
There seems to be infolink code in js file below.
https://adminlte.io/themes/AdminLTE/dist/js/adminlte.min.js
Is this the intended ad? I felt that demo site has been cracked.
This is just a confirmation.
Thanks again.

Yeah we are. Unfortunately, we enabled the in-text ads by mistake. We didn't mean to add annoying ads like those 👎 .
Thanks for the concern. We appreciate it!


Support multiplication and division as operations in the amount fields

Currently hledger-iadd supports addition and subtraction of numbers in amount fields. This is an enhancement request to also support multiplication and division, since those are very useful for entering added taxes or split bills.
I propose recognizing the standard symbols * and / as the symbols respectively for multiplication and division (as opposed to, say, × and ÷, which are not easy to type).
One issue that will need some thinking through is that of rounding / how many significant digits after the decimal ought to be shown.  I would suggest that the same logic used by hledger while displaying amounts be used by hledger-iadd while entering amounts.

I'm closing this as duplicate of #34 , even though I had to sneakily edit the description of the other issue beforehand :)
(Incidentally, the issue number of this issue is exactly twice the old one...)


flutter 3.13 SyntaxError: Unterminated string in JSON at position 110

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
It's on flutter web, and I wasn't facing this issue on flutter 3.7.11, the moment I upgraded all the web apps started to crash and the thing is they are on prod, and I cannot upgrade them with features anymore.
Pass a long JSON object in query parameters in the URL e.g.
http://localhost:3030/create-promotion?location=New+Orleans&business=%7B%22id%22%3A%22kh5gB31SUBeV_rxoOF-2jg%22%2C%22alias%22%3A%22zee-s-pizzeria-new-orleans%22%2C%22name%22%3A%22Zee%E2%80%99s%20Pizzeria%22%2C%22image_url%22%3A%22https%3A%2F%2Fs3-media1.fl.yelpcdn.com%2Fbphoto%2FWtuNhFFD7qVHIw-OsImMfg%2Fo.jpg%22%2C%22is_closed%22%3Afalse%2C%22url%22%3A%22https%3A%2F%2Fwww.yelp.com%2Fbiz%2Fzee-s-pizzeria-new-orleans%3Fadjust_creative%3D1pM5WZbFxxAwexaBvUdWbA%26utm_campaign%3Dyelp_api_v3%26utm_medium%3Dapi_v3_business_search%26utm_source%3D1pM5WZbFxxAwexaBvUdWbA%22%2C%22review_count%22%3A96%2C%22categories%22%3A%5B%7B%22alias%22%3A%22pizza%22%2C%22title%22%3A%22Pizza%22%7D%5D%2C%22ratings%22%3Anull%2C%22coordinates%22%3A%7B%22latitude%22%3A29.92897794383479%2C%22longitude%22%3A-90.09721863149932%7D%2C%22transactions%22%3A%5B%5D%2C%22price%22%3Anull%2C%22location%22%3A%7B%22address1%22%3A%223914%20Baronne%20St%22%2C%22address2%22%3A%22%22%2C%22address3%22%3Anull%2C%22city%22%3A%22New%20Orleans%22%2C%22zip_code%22%3A%2270115%22%2C%22country%22%3A%22US%22%2C%22state%22%3A%22LA%22%2C%22display_address%22%3A%5B%223914%20Baronne%20St%22%2C%22New%20Orleans%2C%20LA%2070115%22%5D%7D%2C%22phone%22%3A%22%2B15047666056%22%2C%22display_phone%22%3A%22(504)%20766-6056%22%2C%22distance%22%3A3280.8075429286437%7D

And after navigation, simply refresh the page you'll get an exception:
SyntaxError: Unterminated string in JSON at position 111

Now, this is because somehow it won't read the completed JSON object inside the URL for some reason.
"{"id":"kh5gB31SUBeV_rxoOF-2jg","alias":"zee-s-pizzeria-new-orleans","name":"Zee’s Pizzeria","image_url":"https:"

Expected results
It should be able to parse the JSON object, if this was a length issue which I mostly found on the internet it should have been caused in the flutter 3.7.11 version too.
Actual results
An exception:
SyntaxError: Unterminated string in JSON at position 111

With half read JSON object:
"{"id":"kh5gB31SUBeV_rxoOF-2jg","alias":"zee-s-pizzeria-new-orleans","name":"Zee’s Pizzeria","image_url":"https:"

Code sample
Code sample
Class handling my routes
class WebRouter {
  static final navigatorKey = GlobalKey<NavigatorState>();

  static Route<dynamic> onGenerateRoute(RouteSettings settings) {
    final data = settings.name!.getRoutingData;

    switch (data.route) {
      // web-route-start
      case AppRoutes.orderConfirmation:
        final orderId = data['orderId'] as String;
        return _FadeRoute(
          child: OrderConfirmationScreen(orderId: orderId),
          settings: settings,
        );
      case AppRoutes.searchBussiness:
        return _FadeRoute(
          child: const SearchBusinessScreen(),
          settings: settings,
        );
      case AppRoutes.createPromotion:
        final business = Business.fromJson(data['business']); // <-- this is causing exception
        return _FadeRoute(
          child: CreatePromotionScreen(
            business: business,
          ),
          settings: settings,
        );
      case AppRoutes.home:
        return _FadeRoute(
          child: const HomeScreen(),
          settings: settings,
        );

      // web-route-end

      default:
        return _FadeRoute(
          child: const HomeScreen(),
          settings: settings,
        );
    }
  }

  Future<dynamic> navigateTo(String route,
      {Map<String, dynamic>? queryParams}) {
    if (queryParams != null) {
      route = Uri(path: route, queryParameters: queryParams).toString();
    }

    return navigatorKey.currentState!.pushNamed(route, arguments: queryParams);
  }
}

class _FadeRoute extends PageRouteBuilder {
  final Widget child;
  @override
  final RouteSettings settings;

  _FadeRoute({required this.child, required this.settings})
      : super(
          settings: settings,
          pageBuilder: (context, ani1, ani2) => child,
          transitionsBuilder: (context, ani1, ani2, child) {
            return FadeTransition(
              opacity: ani1,
              child: child,
            );
          },
        );
}

Business Model Class
import 'dart:convert';

import 'package:flutter/foundation.dart' as foundation;
import 'package:leaf_for_business/models/category.dart';
import 'package:leaf_for_business/models/coordinates.dart';
import 'package:leaf_for_business/models/location.dart';

class Business {
  final String id;
  final String alias;
  final String name;
  final String imageUrl;
  final bool isClosed;
  final String url;
  final int reviewCount;
  final List<Category> categories;
  final double? ratings;
  final Coordinates coordinates;
  final List transactions;
  final String? price;
  final Location location;
  final String phone;
  final String displayPhone;
  final double distance;
  Business({
    required this.id,
    required this.alias,
    required this.name,
    required this.imageUrl,
    required this.isClosed,
    required this.url,
    required this.reviewCount,
    required this.categories,
    this.ratings,
    required this.coordinates,
    required this.transactions,
    this.price,
    required this.location,
    required this.phone,
    required this.displayPhone,
    required this.distance,
  });

  Business copyWith({
    String? id,
    String? alias,
    String? name,
    String? imageUrl,
    bool? isClosed,
    String? url,
    int? reviewCount,
    List<Category>? categories,
    double? ratings,
    Coordinates? coordinates,
    List? transactions,
    String? price,
    Location? location,
    String? phone,
    String? displayPhone,
    double? distance,
  }) {
    return Business(
      id: id ?? this.id,
      alias: alias ?? this.alias,
      name: name ?? this.name,
      imageUrl: imageUrl ?? this.imageUrl,
      isClosed: isClosed ?? this.isClosed,
      url: url ?? this.url,
      reviewCount: reviewCount ?? this.reviewCount,
      categories: categories ?? this.categories,
      ratings: ratings ?? this.ratings,
      coordinates: coordinates ?? this.coordinates,
      transactions: transactions ?? this.transactions,
      price: price ?? this.price,
      location: location ?? this.location,
      phone: phone ?? this.phone,
      displayPhone: displayPhone ?? this.displayPhone,
      distance: distance ?? this.distance,
    );
  }

  Map<String, dynamic> toMap() {
    return <String, dynamic>{
      'id': id,
      'alias': alias,
      'name': name,
      'image_url': imageUrl,
      'is_closed': isClosed,
      'url': url,
      'review_count': reviewCount,
      'categories': categories.map((x) => x.toMap()).toList(),
      'ratings': ratings,
      'coordinates': coordinates.toMap(),
      'transactions': transactions,
      'price': price,
      'location': location.toMap(),
      'phone': phone,
      'display_phone': displayPhone,
      'distance': distance,
    };
  }

  factory Business.fromMap(Map<String, dynamic> map) {
    return Business(
      id: map['id'] as String,
      alias: map['alias'] as String,
      name: map['name'] as String,
      imageUrl: map['image_url'] as String,
      isClosed: map['is_closed'] as bool,
      url: map['url'] as String,
      reviewCount: map['review_count'] as int,
      categories: List<Category>.from(
        (map['categories'] as List).map<Category>(
          (x) => Category.fromMap(x as Map<String, dynamic>),
        ),
      ),
      ratings: map['ratings'] != null
          ? double.parse(map['ratings'].toString())
          : null,
      coordinates:
          Coordinates.fromMap(map['coordinates'] as Map<String, dynamic>),
      transactions: List.from(map['transactions'] as List),
      price: map['price'] != null ? map['price'] as String : null,
      location: Location.fromMap(map['location'] as Map<String, dynamic>),
      phone: map['phone'] as String,
      displayPhone: map['display_phone'] as String,
      distance: map['distance'] as double,
    );
  }

  String toJson() => json.encode(toMap());

  factory Business.fromJson(String source) => /// << -- EXCEPTION AT THIS POINT
      Business.fromMap(json.decode(source) as Map<String, dynamic>);

  @override
  String toString() {
    return 'Business(id: $id, alias: $alias, name: $name, imageUrl: $imageUrl, isClosed: $isClosed, url: $url, reviewCount: $reviewCount, categories: $categories, ratings: $ratings, coordinates: $coordinates, transactions: $transactions, price: $price, location: $location, phone: $phone, displayPhone: $displayPhone, distance: $distance)';
  }

  @override
  bool operator ==(covariant Business other) {
    if (identical(this, other)) return true;

    return other.id == id &&
        other.alias == alias &&
        other.name == name &&
        other.imageUrl == imageUrl &&
        other.isClosed == isClosed &&
        other.url == url &&
        other.reviewCount == reviewCount &&
        foundation.listEquals(other.categories, categories) &&
        other.ratings == ratings &&
        other.coordinates == coordinates &&
        foundation.listEquals(other.transactions, transactions) &&
        other.price == price &&
        other.location == location &&
        other.phone == phone &&
        other.displayPhone == displayPhone &&
        other.distance == distance;
  }

  @override
  int get hashCode {
    return id.hashCode ^
        alias.hashCode ^
        name.hashCode ^
        imageUrl.hashCode ^
        isClosed.hashCode ^
        url.hashCode ^
        reviewCount.hashCode ^
        categories.hashCode ^
        ratings.hashCode ^
        coordinates.hashCode ^
        transactions.hashCode ^
        price.hashCode ^
        location.hashCode ^
        phone.hashCode ^
        displayPhone.hashCode ^
        distance.hashCode;
  }
}

Screenshots or Video

Screenshots / Video demonstration
Flow Video
https://github.com/flutter/flutter/assets/43790152/817bdeda-51bf-4143-8a01-6f204b22496e
Exception

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.0, on macOS 13.5 22G74 darwin-arm64, locale en-PK)
    • Flutter version 3.13.0 on channel stable at /Users/hamza/Development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision efbf63d9c6 (2 weeks ago), 2023-08-15 21:05:06 -0500
    • Engine revision 1ac611c64e
    • Dart version 3.1.0
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/hamza/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

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

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

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

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 13.5 22G74 darwin-arm64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 116.0.5845.140

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Hi @mhmzdev. Thanks for filing this. I'm unable to test the reproducibility of this with the samples you provided. Please can you provide a minimal complete reproducible code sample that doesn’t include 3rd party plugins or complex production code?
Thank you

@dam-ease you can simply create a model class, and then pass a ModelClass.toJson() into URL as query params, simply refresh the page.
And it doesn't require any third party service.

Hi @mhmzdev. Thanks for your repsonse and detailed explanation. For this issue to be workable, we'd be needing a simple completed and minimal reproducible code sample, if possible a you can share a github repo but kindly remove complex reproduction codes.


Best approach to using npm libs without typings

I am trying to use an npm package that has no corresponding typings file, hence I cannot use an 'import' statement that later most likely participates in the bundling process.
I could not find a documentation to what is the recommended approach by ionic given the new build process in RC0.
The quickest thing I could think of would be to use a 'declare var' statement and an npm task to copy the js file to the www folder.
What approach would you recommend to this?

Hi @morious,
Let's track this discussion in the other tread.  I am happy to chat about it.
https://github.com/driftyco/ionic/issues/8274#issuecomment-250359341
Thanks,
Dan


Updated the call with events

closing this PR,this changes are incorporated in https://github.com/rdkcentral/mock-firebolt/pull/139


feat(helper-cli): Extend path exclude generator by a couple of patterns

Add patterns for common CI configuration, build or test files and dirs.

Commitlint cannot cope with multiple signed-off footers, unfortunately (same with co-authored-by footers). The pragmatic solution is to simply remove the signoff for the non-author, and eventually use the other user as the committer.

Commitlint cannot cope with multiple signed-off footers, unfortunately (same with co-authored-by footers). The pragmatic solution is to simply remove the signoff for the non-author, and eventually use the other user as the committer.

Ok if we just merge regardless of this failing check ?

Ok if we just merge regardless of this failing check ?

I'm not sure if this will result in problems with future PRs... maybe Commitlint looks at all the history instead of just the commits introduced in a PR. So I'd prefer to reword the commit messages, like I also did here.


SKLearnProcessor unable to recognize preprocessing file in sagemaker local mode

Hello, I am experiencing an issue while using SageMaker to build a machine learning model. in Sagemaker local mode, I am facing an issue that says "python3: can't open file '/opt/ml/processing/code/processing_script.py': [Errno 2] No such file or directory." Even though the file is present in the correct location, the same code works fine in the normal mode. I am running my project in vscode dev conatiner and inside the dev container i am trying out the sagemaker local mode. Please let me know what could possibly be wrong with the following code.
Error
INFO:sagemaker.local.image:docker command: docker-compose -f /tmp/tmpx5_z74cs/docker-compose.yaml up --build --abort-on-container-exit
Creating 5ktwpjv1ug-algo-1-5bua0 ...
Creating 5ktwpjv1ug-algo-1-5bua0 ... done
Attaching to 5ktwpjv1ug-algo-1-5bua0
5ktwpjv1ug-algo-1-5bua0 | python3: can't open file '/opt/ml/processing/input/code/processing_script.py': [Errno 2] No such file or directory
5ktwpjv1ug-algo-1-5bua0 exited with code 2
2
Aborting on container exit...
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/sagemaker/local/image.py", line 166, in process
    _stream_output(process)
  File "/usr/local/lib/python3.8/dist-packages/sagemaker/local/image.py", line 916, in _stream_output
    raise RuntimeError("Process exited with code: %s" % exit_code)
RuntimeError: Process exited with code: 2

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "main2.py", line 20, in <module>
    processor.run(
  File "/usr/local/lib/python3.8/dist-packages/sagemaker/workflow/pipeline_context.py", line 272, in wrapper
    return run_func(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/sagemaker/processing.py", line 615, in run
    self.latest_job = ProcessingJob.start_new(
  File "/usr/local/lib/python3.8/dist-packages/sagemaker/processing.py", line 849, in start_new
    processor.sagemaker_session.process(**process_args)
  File "/usr/local/lib/python3.8/dist-packages/sagemaker/session.py", line 1024, in process
    self._intercept_create_request(process_request, submit, self.process.__name__)
  File "/usr/local/lib/python3.8/dist-packages/sagemaker/session.py", line 4813, in _intercept_create_request
    return create(request)
  File "/usr/local/lib/python3.8/dist-packages/sagemaker/session.py", line 1022, in submit
    self.sagemaker_client.create_processing_job(**request)
  File "/usr/local/lib/python3.8/dist-packages/sagemaker/local/local_session.py", line 128, in create_processing_job
    processing_job.start(
  File "/usr/local/lib/python3.8/dist-packages/sagemaker/local/entities.py", line 140, in start
    self.container.process(
  File "/usr/local/lib/python3.8/dist-packages/sagemaker/local/image.py", line 171, in process
    raise RuntimeError(msg) from e
RuntimeError: Failed to run: ['docker-compose', '-f', '/tmp/tmpx5_z74cs/docker-compose.yaml', 'up', '--build', '--abort-on-container-exit']

Code
from sagemaker.local import LocalSession
from sagemaker.processing import ProcessingInput, ProcessingOutput
from sagemaker.sklearn.processing import SKLearnProcessor
sagemaker_session = LocalSession()
sagemaker_session.config = {"local": {"local_code": True}}
role = "arn:aws:iam::111111111111:role/service-role/AmazonSageMaker-ExecutionRole-20200101T000001"
processor = SKLearnProcessor(
framework_version="0.20.0", instance_count=1, instance_type="local", role=role
)
print("Starting processing job.")
print(
"Note: if launching for the first time in local mode, container image download"
" might take a few minutes to complete."
)
processor.run(
code="processing_script.py",
inputs=[
ProcessingInput(
source='./input_data/',
destination="/opt/ml/processing/input_data/",
)
],
outputs=[
ProcessingOutput(
output_name="word_count_data", source="/opt/ml/processing/processed_data/"
)
],
arguments=["job-type", "word-count"],
)
preprocessing_job_description = processor.jobs[-1].describe()
output_config = preprocessing_job_description["ProcessingOutputConfig"]
print(output_config)
for output in output_config["Outputs"]:
if output["OutputName"] == "word_count_data":
word_count_data_file = output["S3Output"]["S3Uri"]
print("Output file is located on: {}".format(word_count_data_file))

What happens if you try to run the example in the same container but in command line?
python3 example.py
I want to rule out VSCode configuration issue.

I have the same issue, how did you resolve?

@VibhavariBellutagi19 can you run it not as part of VSCode, but with python scikit_learn_bring_your_own_container_local_processing.py

@eitansela i was able to run it outside of the dev container successfully just by invoking the local script


Brackets Pricing$$$

Its not an issue yet, but will Brackets remain open source, or will it cost money in the near future?

Official statement from adobe representatives state that it is free indefinitely (https://forums.adobe.com/thread/1235571).
Since the project is open source, you can download it and build the executable yourself. Even if Adobe decided to monetize Brackets 2.0, this open source version would remain free.

As noted above, the MIT license means you will always be able to use this code for free.


PR for #49 Compile build module within Ceylon

This is the pull reqest for #49 .
The Ceylon compiler plugin is executed with
ceylon build [--<compiler-option>=<value>...] <module>/<version> <goal>...

Now always compiles the build module, as I don't know yet how to use the cache properly. For changing that look into file compileModule.ceylon.
Also added a ceylon-build.bat with description. I haven't tested the Unix version of the script though, as it now contains a longer usage explanation. Might been broken now.

Only supports compiler options of pre 1.1 (or 1.0?).
I've deliberately removed the --out compiler option, and defaulted the option --source=build-source

@loicrouchon are you on top of this?

Yes, I'm on it. But no access to a my computer right now. I will take a
look at it on Sunday or Monday evening.
Le 20 févr. 2015 15:46, "Gavin King"<EMAIL_ADDRESS>a écrit :

@loicrouchon https://github.com/loicrouchon are you on top of this?
—
Reply to this email directly or view it on GitHub
https://github.com/ceylon/ceylon.build/pull/50#issuecomment-75248859.

I took a look at your work and I have a few remarks:
First, we should not require the user to input the build module name and version.
Otherwise, you will have to do ceylon build build/1 compile to launch the compile goal which is no good. It should not be longer than ceylon build compile (as long as the build module have the default name).
So I think it makes sense to have an optional parameter for the module name in case you have multiples build modules, but the version should never be needed in input (because it's not needed to compile the build module and so we should take the one that module).
Concerning architecture I think we should not use ceylon.build.tasks.ceylon to compile the build module itself. For me this is requesting a dependency that is not needed.
Calling the compiler should be as easy as below (don't forget import com.redhat.ceylon.compiler.java "1.1.1"; in module.ceylon)
CeylonTool tool = CeylonTool();
tool.setToolLoader(loadJavaModule("com.redhat.ceylon.compiler.java", language.version));
tool.bootstrap("compile", "--source=build-source", "build", "other options, ...");

I noticed that there were modifications in ceylon.build.tasks.ant module and I don't really see what it has to do with this pull request. This might be linked with the fact that you are working directly on master branch.
It would be better to isolate the work related to a pull request by creating its own feature branch.
Here is some documentation about GitHub flow https://guides.github.com/introduction/flow/index.html that explains in a few minutes the feature branch workflow
If you have any questions, let me know, I'll be more than happy to help

Sorry for not answering staight away, but I'm travelling. And I can do the fixes only next Sunday/monday/Tuesday.
So on your comments:
Hmm, thought about it, but then how to destinguish between a module and a goal? And then I didn't knew how to get the version out of a module just compiled. Maybe we can say, if a slash is in the first non-option argument, it's modulename/version otherwise the default is build/1. But this makes the version mandatory, if the module isn't build/1.
Calling the compiler directly might be a better idea.
With the changes in ceylon.build.tasks.ant you're right I should do a different pull request. Thanks for the link.

@loicrouchon: Now changed to use the Ceylon compiler directly. Also added compiler options available in 1.1.1.
module/version is now optional, if default build module build/1 is used. But version is required to distinguish between module and goals.
Compiled binaries go into ./modules, as I couldn't find a way to load the module from a different directory programatically. Maybe some kind of cache should be used, but then the different projects might overwrite or reuse other's build/1 module.


Messages Context

This PR turns state.messages into an object like { entities: Record<string, Message>, ids: string[] } so that we can pass an array of ids into a messages list, which'll just pass a single id to a message item view, and then the message item view can "look up" the actual message object from a(n also new) MessagesContext. For right now it just updates the main reducer and state object and converts back to the old style of "array of message objects" so that the changes are pretty minimal (apart from formatting). I'll add some comments to the PR to add more detail around the interesting bits.

This PR looks great in its current form! (modulo the reducer git conflicts that look trivial). As per your suggestion on GH, I'm going to avoid merging this in until I've had time to properly look over #2 and #3 on your fork so we can merge those into this as a single PR.

This PR looks great in its current form! (modulo the reducer git conflicts that look trivial). As per your suggestion on GH, I'm going to avoid merging this in until I've had time to properly look over #2 and #3 on your fork so we can merge those into this as a single PR.

Fixed those merge conflicts 🤞 tests all pass, updating https://github.com/mysterycommand/azure-mud/pull/2 and https://github.com/mysterycommand/azure-mud/pull/3 now


#789 removing duplicate X-Real-IP header

Commit 4bd4bf3be65eb13166771d375991ccccc3fff664 added setting the X-Real-IP header, which was already added a few lines above. Removing the original proxy_set_header X-Real-IP because the newer one is better grouped with related proxy_set_header commands

This change is

Coverage remained the same at 46.597% when pulling a854dc71b2257fc5cc8ad167482cf743a8906456 on arjanschaaf:duplicate-real-ip into 32f24380ecfe89506070cdb5a99aac6a760af324 on kubernetes:master.

/lgtm

@arjanschaaf thanks!


Add new config for jscs.info and update jshintrc

.jshintrc was updated a while ago (this should be an ongoing process) and probably we will add an ESLint config


Revert back to previous behaviour for setClient

Codecov Report
Base: 59.91% // Head: 59.95% // Increases project coverage by +0.03% :tada:

Coverage data is based on head (53a18e4) compared to base (d9aaace).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #213      +/-   ##
============================================
+ Coverage     59.91%   59.95%   +0.03%
  Complexity      614      614
============================================
  Files           113      113
  Lines          2248     2250       +2
  Branches        249      249
============================================
+ Hits           1347     1349       +2
  Misses          818      818
  Partials         83       83

Impacted Files
Coverage Δ

src/main/java/seedu/address/model/MyInsuRec.java
87.09% <100.00%> (+0.43%)
:arrow_up:

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

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


irps5410 error

I have built and ran the tutorial with no modification successfully but when using Petalinux to configure the Common Software the dpu_sw_optimize.sh fails with:
Auto resize ext4 partition ...[✔]
Start QoS config ...[✔]
Config PMIC irps5401 .../dev/i2c-4: Device or resource busy
Couldn't attach to device 0x43
I included the i2ctool in my Petalinux build but this still errors out. I am using the same base platform as the tutorial.
I have see the problem, but I don't see the solution.
Vivado verision : 2022.2
Broad : zcu104
vitis-ai : 3.0

@FBlueN I am of the understanding that dpu_sw_optimize.sh was developed prior to the inclusion of an IRPS5401 driver in the Linux upstream.  The dpu_sw_optimize implementation leverages IOCTL access to the I2C device which does not function if the IRPS5401 driver has been probed which binds the upstream driver to the hardware.
A potential work-around to this is to disable CONFIG_SENSORS_IRPS5401.  If CONFIG_SENSORS_IRPS5401=y the hwmon/pmbus/ir5401.c is probed and the user space dpu_sw_optimize application will fail.
BTW, the sole purpose of this application is to modify the maximum output power available from the regulator subsystem on the ZCU104.  Without these modifications, the maximum power available to the ZU+ device is insufficient to run some (many) models with multiple threads.

@FBlueN I am of the understanding that dpu_sw_optimize.sh was developed prior to the inclusion of an IRPS5401 driver in the Linux upstream. The dpu_sw_optimize implementation leverages IOCTL access to the I2C device. IOCTL cannot read and write to this hardware device if the IRPS5401 driver has been probed and bound to the hardware instance. The upstream 5401 driver currently does not have the hooks to read/write the necessary registers in the device, so it is not straightforward to modify dpu_sw_optimize to simply leverage the IRPS5401 driver.
A potential work-around to this is to disable CONFIG_SENSORS_IRPS5401. If CONFIG_SENSORS_IRPS5401=y the hwmon/pmbus/ir5401.c is probed and the user space dpu_sw_optimize application will fail.
BTW, the sole purpose of this application is to modify the maximum output power available from the regulator subsystem on the ZCU104. Without these modifications, the maximum power available to the ZU+ device is insufficient to run some (many) models with multiple threads. Given that our use of dpu_sw_optimize is purely for ZCU104 max power adjustment, we have not ourselves invested in modifying the upstream driver to support the required register accesses.

I understand what do you mean.However when I try to disable CONFIG_SENSORS_IRPS5401, I didn't find how to do.So can you teach me how to disable CONFIG_SENSORS_IRPS5401.Thank you.

@FBlueN I am of the understanding that dpu_sw_optimize.sh was developed prior to the inclusion of an IRPS5401 driver in the Linux upstream. The dpu_sw_optimize implementation leverages IOCTL access to the I2C device. IOCTL cannot read and write to this hardware device if the IRPS5401 driver has been probed and bound to the hardware instance. The upstream 5401 driver currently does not have the hooks to read/write the necessary registers in the device, so it is not straightforward to modify dpu_sw_optimize to simply leverage the IRPS5401 driver.
A potential work-around to this is to disable CONFIG_SENSORS_IRPS5401. If CONFIG_SENSORS_IRPS5401=y the hwmon/pmbus/ir5401.c is probed and the user space dpu_sw_optimize application will fail.
BTW, the sole purpose of this application is to modify the maximum output power available from the regulator subsystem on the ZCU104. Without these modifications, the maximum power available to the ZU+ device is insufficient to run some (many) models with multiple threads. Given that our use of dpu_sw_optimize is purely for ZCU104 max power adjustment, we have not ourselves invested in modifying the upstream driver to support the required register accesses.

Hey, I have solved this problem. Thank you.

Thanks @FBlueN we are pleased to hear this.  We will close out this issue but please reopen if needed.


Ability to have theme options per spec

Hello,
I'm using 2 specs on my website and I'd like one of these to not display the full endpoint paths, and the last one to display it.
Would it be possible to allow theme option per spec entry?

Maybe you can try having 2 instances of redocusaurus with different specs, paths, and theme options. See https://docusaurus.io/docs/using-plugins#multi-instance-plugins-and-plugin-ids
Not sure if it will work

I added some to handle multiple instances of presets with different ids. It should work in the next version.

Wow, nice 😎  Thanks a lot 😊


Could not found console object

Description
I just created a empty project on RN 0.70
and got this error on build time.

Version
"@shopify/react-native-skia": "^0.1.145",
Steps to reproduce
just install empty project on react native 0.70 and install react-native-skia and try to build it with react-native run-ios
Snack, code example, screenshot, or link to a repository
none

duplicate of #885

Same thing here.


[Suggestion] Username on wallet should be clickable and redirect to that user's profile

Username on wallet should be clickable and clicking on it should take us on user's profile..

Clicking on followers should show all followers list. And beside of the username there will be a follow button to follow my followers and unfollow button whom i already following.
same for following with unfollow button beside of the username.

Renaming as a part of it is mentioned in #56.


[checkpoints] Sync efficiency (RFC)

The current checkpointing code includes a component that allows a validator to sync to previous checkpoints. It involves:

downloading the checkpoint with all its transaction digests from another node,
download (one by one) the transactions or certificates in the checkpoint,
do a node-to-node sync of all the data required to execute the transaction,

The above is a tragically and needlessly inefficient process. Instead we should:

Package checkpoints as datafiles including all meta-data to authenticate + actual transaction / effects bodies ordered in causal execution order.
Download these files from one or more other peers.
Bulk execute the transactions and double-check their effects.

The flow, in particular, should not involve any network IO except for downloading checkpoints (but not per-transaction download + sync)

Full node can do concurrent certificate download / sync. It is not as efficient as the process @gdanezis describes, but it should be fast enough (currently 800 tx/sec) to hold us over for a while.


disable-access-log have no effect

Set disable-access-log as true in config map.

[root@bd002 ~]# kubectl get cm --namespace=kube-system -o yaml
apiVersion: v1
items:
- apiVersion: v1
  data:
    body-size: "0"
    disable-access-log: "true"
  kind: ConfigMap
  metadata:
    creationTimestamp: 2017-03-26T11:00:41Z
    name: nginx-load-balancer-conf
    namespace: kube-system
    resourceVersion: "1311689"
    selfLink: /api/v1/namespaces/kube-system/configmaps/nginx-load-balancer-conf
    uid: 73d78d7a-1213-11e7-bbf5-f2ecb40a350b
kind: List
metadata: {}
resourceVersion: ""
selfLink: ""

Ingress controller is running with nginx-configmap

[root@bd002 ~]# ps -ef | grep ingress
root      6582 18109  0 08:11 pts/4    00:00:00 grep --color=auto ingress
root     21670 21652  0 07:53 ?        00:00:00 /nginx-ingress-controller --default-backend-service=kube-system/default-http-backend --nginx-configmap=kube-system/nginx-load-balancer-conf

Check nginx.conf for the ingress controller log, found the access_log is not turn off.

access_log /var/log/nginx/access.log upstreaminfo if=$loggable;

@gyliu513 which version of the ingress controller are you using?

I was using 0.8.3, just check the annotation of config.go, seems the 0.8.3 do not include the parameter of disable-access-log as it was added in 17/2/10, so seems I have to use master if want to use this feature, right? @aledbf

@gyliu513 right, this feature was added in 0.9-beta.2

Using 0.9.3 beta, the ingress controller report the following error:
[root@bd003 ~]# docker logs 356280b97b2b
unknown flag: --nginx-configmap
Usage of :
      --alsologtostderr                  log to standard error as well as files
      --apiserver-host string            The address of the Kubernetes Apiserver to connect to in the format of protocol://address:port, e.g., http://localhost:8080. If not specified, the assumption is that the binary runs inside a Kubernetes cluster and local discovery is attempted.
      --configmap string                 Name of the ConfigMap that contains the custom configuration to use
      --default-backend-service string   Service used to serve a 404 page for the default backend. Takes the form
    	namespace/name. The controller uses the first node port of this Service for
    	the default backend.
      --default-ssl-certificate string   Name of the secret
		that contains a SSL certificate to be used as default for a HTTPS catch-all server
      --election-id string               Election id to use for status update. (default "ingress-controller-leader")
      --health-check-path string         Defines
		the URL to be used as health check inside in the default server in NGINX. (default "/healthz")
      --healthz-port int                 port for healthz endpoint. (default 10254)
      --ingress-class string             Name of the ingress class to route through this controller.
      --kubeconfig string                Path to kubeconfig file with authorization and master location information.
      --log_backtrace_at traceLocation   when logging hits line file:N, emit a stack trace (default :0)
      --log_dir string                   If non-empty, write log files in this directory
      --logtostderr                      log to standard error instead of files
      --profiling                        Enable profiling via web interface host:port/debug/pprof/ (default true)
      --publish-service string           Service fronting the ingress controllers. Takes the form
 		namespace/name. The controller will set the endpoint records on the
 		ingress objects to reflect those on the service.
      --stderrthreshold severity         logs at or above this threshold go to stderr (default 2)
      --sync-period duration             Relist and confirm cloud resources this often. (default 1m0s)
      --tcp-services-configmap string    Name of the ConfigMap that contains the definition of the TCP services to expose.
		The key in the map indicates the external port to be used. The value is the name of the
		service with the format namespace/serviceName and the port of the service could be a
		number of the name of the port.
		The ports 80 and 443 are not allowed as external ports. This ports are reserved for the backend
      --udp-services-configmap string    Name of the ConfigMap that contains the definition of the UDP services to expose.
		The key in the map indicates the external port to be used. The value is the name of the
		service with the format namespace/serviceName and the port of the service could be a
		number of the name of the port.
      --update-status                    Indicates if the
		ingress controller should update the Ingress status IP/hostname. Default is true (default true)
  -v, --v Level                          log level for V logs
      --vmodule moduleSpec               comma-separated list of pattern=N settings for file-filtered logging
      --watch-namespace string           Namespace to watch for Ingress. Default is to watch all namespaces

@gyliu513 please check https://github.com/kubernetes/ingress/releases/tag/nginx-ingress-controller-0.9-beta.1
The flag  --nginx-configmap was replaced by  --configmap

Seems the disable-access-log: "true" was introduced in https://github.com/kubernetes/ingress/releases/tag/nginx-0.9.0-beta.2 , I will try this version, thanks @aledbf

@gyliu513 yes, I just mentioned 0.9-beta.1 because since that version the flag --nginx-configmap was replaced by --configmap

cool, thanks @aledbf , will close this if test passed ;-)

Still failed even I was using 0.9-beta.2.
config map:
[root@bd002 ~]# kubectl get cm -o yaml --namespace=kube-system
apiVersion: v1
items:
- apiVersion: v1
  data:
    proxy-body-size: "0"
  kind: ConfigMap
  metadata:
    creationTimestamp: 2017-03-26T11:00:41Z
    name: nginx-load-balancer-conf
    namespace: kube-system
    resourceVersion: "1374122"
    selfLink: /api/v1/namespaces/kube-system/configmaps/nginx-load-balancer-conf
    uid: 73d78d7a-1213-11e7-bbf5-f2ecb40a350b
kind: List
metadata: {}
resourceVersion: ""
selfLink: ""

ingress controller container report the following:
[root@bd002 ~]# docker logs 46575e471f42
I0327 00:40:25.472966       5 launch.go:94] &{NGINX 0.9.0-beta.2 git-7013a52 git@github.com:ixdy/kubernetes-ingress.git}
I0327 00:40:25.473025       5 launch.go:97] Watching for ingress class: nginx
F0327 00:40:25.473235       5 launch.go:242] Error while initializing connection to Kubernetes apiserver. This most likely means that the cluster is misconfigured (e.g., it has invalid apiserver certificates or service accounts configuration). Reason: invalid configuration: no configuration has been provided
Refer to the troubleshooting guide for more information: https://github.com/kubernetes/ingress/blob/master/docs/troubleshooting.md

After upgrade to beta.2, it is working well! Thanks @aledbf


fix(legacy-preset-chart-nvd3): stacked bar charts labels

🐛 Bug Fix
Labels are not being displayed on stacked bar charts with some of the groups disabled in the legend, because the number of series being displayed is being computed incorrectly. I fixed it by ignoring disabled series, and added a new story to test it.

👍


Fix memory leaks (update redis)

Update redis to v4.1.2
Set version 4.0.2

What was wrong?
There's a memory leak on production services that we don't observe on staging. The only difference is the redis version
How was it fixed?
Update redis

Coverage increased (+0.2%) to 88.83% when pulling efa4df0924c483b9d3c4a7df04af02c8e81dd89d on hotfix into 3ade8587622542bfe814e82040159787b179a3e2 on master.


Unable to scroll after opening video post

After expanding most video posts I am unable to scroll until I open either the left or right swipe bar.
This happens on v.redd.it and streamable posts. Could affect more also
Occurs on both main page and when opening video from comments pages.
To Reproduce
Steps to reproduce the behavior:

Go to https://old.reddit.com/r/oddlysatisfying/comments/18ylhs6/creation_of_ceiling_art_using_plaster/
Expand video
Unable to scroll up or down
Close video
Still are unable to scroll
Open and close a side panels (left or right)
Can scroll again

Smartphone

Device: Galaxy S21
OS: Android
Browser Firefox Beta
Version <IP_ADDRESS>7

Reddit page where bug can be easily reproduced
https://old.reddit.com/r/oddlysatisfying/comments/18ylhs6/creation_of_ceiling_art_using_plaster/

Consistently happens to me too:
Device: Pixel 6 Pro
OS: Android 14 (5.10.177)
Browser Firefox 121.0
Version latest from today

Fixed in https://github.com/OctoNezd/oldlander/commit/4130885001dfcb527508cc9a4b7485142bbf79ef


Updated wording

Updated wording

cc: @Justinha
GitHub Issue: https://github.com/MicrosoftDocs/azure-docs/issues/82234

#label:"aq-pr-triaged"

#sign-off

@MarileeTurscak-MSFT: I'm sorry - only the author of this article, @justinha, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


secret syncer

Signed-off-by: Ian Zhang<EMAIL_ADDRESS>

closing for now.
a new PR will be opened with all logic of CLC


i2d_x509 now fails in 3.0.0

I have an application where I create a partial x509 certificate.  It gets sent to an HSM, which adds the public key and signs it.
I do essentially this to build the X509 structure::
X509_new
X509_set_version
X509_set_issuer_name
X509_get_notBefore
X509_get_notAfter
X509_set_subject_name
X509_EXTENSION_create_by_OBJ
i2d_x509

This worked in 1.0.1, 1.0.2, 1.1.1, but fails in 3.0.0.
In debugging, even this simple test case now fails.
X509_new
i2d_x509

I tried including everything except the signature, omitting just
X509_sign

and it also failed.
I also tried replacing i2d_x509 with i2d_re_X509_tbs and it also failed.
It could be user error, but this has been working for many years.

I am afraid with the current 3.0 codebase there are not many options
how to workaround this problem apart from either signing the certificate with a
bogus key - if the HSM is able to re-sign such certificate.
Another (more complicated) option would be to define your own ASN.1
X509 structure where the signature would be optional and thus the
stricter encoder that is now in 3.0 codebase would allow encoding the
incomplete certificate.

The HSM follows ISO-11889 and cannot be changed.  It adds the public key after checking the attributes, and then signs it.
I don't understand the 'revert ... on 1.1.1'.  1.1.1k and previous all worked.  The regression is on 3.0.0.

Is ISO-11889 the same document as document 1A here? I can find references to "partialCertificate", but no clear definition as to what a partial certificate is.
The TBSCertificate structure defined in RFC5280 does not mark the SPKI as OPTIONAL, so simply omitting it does not produce the same structure. Is the expectation to omit it, or to insert some empty placeholder instead?
In order to support this non-certificate structure, I think the bare minimum is that someone has even written down what it is.

Aha, found it in part 3, 18.8.1. Looks like this is an entirely different structure that not only omits the public key, but also the version. And if makes the signature algorithm optional. I think this should be a separate API, for serializing a "TPM Partial Certificate", which is a related but different animal from "X.509 Certificate".

Here's some sample code for how to use OpenSSL's ASN.1 template APIs to transcript the non-certificate structure in the TPM spec. This should work both before and after the bugfix. You also can call i2d_X509_NAME, etc., and assemble it using your favorite DER encoder.
#include <openssl/asn1.h>
#include <openssl/asn1t.h>
#include <openssl/x509.h>
#include <stdio.h>
#include <string.h>
#include <time.h>

typedef struct {
  ASN1_TIME *notBefore;
  ASN1_TIME *notAfter;
} TPM_PARTIAL_CERT_VALIDITY;

typedef struct {
  X509_ALGOR *algorithm;
  X509_NAME *issuer;
  TPM_PARTIAL_CERT_VALIDITY *validity;
  X509_NAME *subject;
  STACK_OF(X509_EXTENSION) *extensions;
} TPM_PARTIAL_CERT;

ASN1_SEQUENCE(TPM_PARTIAL_CERT_VALIDITY) = {
  ASN1_SIMPLE(TPM_PARTIAL_CERT_VALIDITY, notBefore, ASN1_TIME),
  ASN1_SIMPLE(TPM_PARTIAL_CERT_VALIDITY, notAfter, ASN1_TIME),
} ASN1_SEQUENCE_END(TPM_PARTIAL_CERT_VALIDITY)

// Note, per 18.8.1, the signature algorithm is optional while the extension
// list is required.
ASN1_SEQUENCE(TPM_PARTIAL_CERT) = {
  ASN1_OPT(TPM_PARTIAL_CERT, algorithm, X509_ALGOR),
  ASN1_SIMPLE(TPM_PARTIAL_CERT, issuer, X509_NAME),
  ASN1_SIMPLE(TPM_PARTIAL_CERT, validity, TPM_PARTIAL_CERT_VALIDITY),
  ASN1_SIMPLE(TPM_PARTIAL_CERT, subject, X509_NAME),
  ASN1_EXP_SEQUENCE_OF(TPM_PARTIAL_CERT, extensions, X509_EXTENSION, 3),
} ASN1_SEQUENCE_END(TPM_PARTIAL_CERT)

IMPLEMENT_ASN1_FUNCTIONS(TPM_PARTIAL_CERT)

int main() {
  // Fill in some placeholder values, just to demonstrate this works. Missing
  // error-handling and whatnot.
  TPM_PARTIAL_CERT *cert = TPM_PARTIAL_CERT_new();
  X509_NAME_add_entry_by_NID(cert->issuer, NID_commonName, MBSTRING_UTF8, "A",
                             1, -1, 0);
  time_t now = time(NULL);
  ASN1_TIME_adj(cert->validity->notBefore, now, 0, -24 * 60 * 60);
  ASN1_TIME_adj(cert->validity->notAfter, now, 0, 24 * 60 * 60);
  X509_NAME_add_entry_by_NID(cert->subject, NID_commonName, MBSTRING_UTF8, "B",
                             1, -1, 0);

  unsigned char *der = NULL;
  int der_len = i2d_TPM_PARTIAL_CERT(cert, &der);
  fwrite(der, der_len, 1, stdout);

  TPM_PARTIAL_CERT_free(cert);
  OPENSSL_free(der);
  return 0;
}

OTC: there is a proper way how to do this presented above. The i2d should not create invalid encodings. We are not going to revert this change.

Here's some sample code for how to use OpenSSL's ASN.1 template APIs to transcript the non-certificate structure in the TPM spec. This should work both before and after the bugfix. You also can call i2d_X509_NAME, etc., and assemble it using your favorite DER encoder.

t.c:24: warning: no previous prototype for 'TPM_PARTIAL_CERT_VALIDITY_it'
t.c:34: warning: no previous prototype for 'TPM_PARTIAL_CERT_it'
t.c:36: warning: no previous prototype for 'd2i_TPM_PARTIAL_CERT'
t.c:36: warning: no previous prototype for 'i2d_TPM_PARTIAL_CERT'
t.c:36: warning: no previous prototype for 'TPM_PARTIAL_CERT_new'
t.c:36: warning: no previous prototype for 'TPM_PARTIAL_CERT_free'
Was this code complete?
What does "assemble it using your favorite DER encoder" mean?

t.c:24: warning: no previous prototype for 'TPM_PARTIAL_CERT_VALIDITY_it'

Perhaps you're building with more warnings then whatever my GCC had on by default. Try adding DECLARE_ASN1_ITEM(TPM_PARTIAL_CERT_VALIDITY)  and DECLARE_ASN1_FUNCTIONS(TPM_PARTIAL_CERT). Normally these various bits would be split between header and source file, but I skipped all that for brevity. I also haven't checked if this is something that was changed in 3.0.0.
The important bit is that this is a different ASN.1 type and, from the beginning, your code should have reflected that. OpenSSL's X.509 encoder isn't magic. It's just encoding stuff according to DER. That omitting fields worked in d2i_X509 was just a bug.

What does "assemble it using your favorite DER encoder" mean?

Certificates use an encoding called DER. If you're unfamiliar with it, you can read about it here:
https://letsencrypt.org/docs/a-warm-welcome-to-asn1-and-der/
If you have a preferred DER encoder on hand, you can also just ask OpenSSL to encode the various fields, concatenate them and wrap it in a SEQUENCE. (E.g. the library I work on has this imperative encoding library, since I don't like the macro tables business.) But using OpenSSL's macros works just fine. I believe those are public API.


Disable focus on agreement text link

JIRA: https://openedx.atlassian.net/browse/MOB-1553
This disables the focus for the agreement link, however, link still remains clickable.
cc @aleffert  @shahidtamboli

:+1:

:+1:


Remove unnecessary event

General cleanup

Codecov Report

Merging #5631 (738c2d0) into main (70073c0) will decrease coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main   #5631   +/-   ##
=====================================
- Coverage     68%     68%   -1%
=====================================
  Files        401     401
  Lines      26724   26724
  Branches    3893    3893
=====================================
- Hits       18371   18365    -6
- Misses      6831    6835    +4
- Partials    1522    1524    +2

Impacted Files
Coverage Δ

src/client/common/utils/decorators.ts
76% <0%> (-2%)
:arrow_down:

...client/datascience/kernel-launcher/kernelDaemon.ts
53% <0%> (-2%)
:arrow_down:

src/client/datascience/jupyter/jupyterNotebook.ts
74% <0%> (-1%)
:arrow_down:

...active-common/intellisense/intellisenseProvider.ts
74% <0%> (-1%)
:arrow_down:


add nsp support

an imperfect stab, since it's unclear how travis would support separate tasks for report generation. for now, there's a pre-push task that will run nsp locally, then update readme with the badge.

Just kidding, have to muck with Makefile some more. If NSP succeeds, the nspBadge script never runs, and the badge never gets updated.


Update examples

Proposed changes:

fixes https://github.com/RasaHQ/rasa_core/issues/1443
did not update formbot due to https://github.com/RasaHQ/rasa_core/pull/1506
add test for concertbot

Status (please check what you already did):

[x] made PR ready for code review
[x] added some tests for the functionality
~[ ] updated the documentation~
[x] updated the changelog

@MetcalfeTom ready for another review


Binding of array item

Hi guys. I've been using this library to cleanup my codebase but I'm stuck with a little problem. Sorry if my question is stupid, I'm just starting with Swift and there is quite a lot to process.
My settings looks like this:
class Entry: Codable, Identifiable, Defaults.Serializable {
    var id: String
    var displayLogo: Bool = true
}
extension Defaults.Keys {
    static let entries = Key<[Entry]>("entries", default: [])
}

in a View, I'm looping through the entries like this and display a view for each entry.
struct EntryView: View {
      @State var entry: Entry

      var body: some View {
            Text("Inside entry \(entry.id)")
             Toggle("Display logo?", isOn: $entry.displayLogo) // <-- This
       }
}

struct EntriesView: View {
    @Default(.entries) var entries : [Entry];

    var body: some View {
        HStack(spacing: 10) {
            ForEach(entries) { entry in
                 EntryView(entry: entry)
            }
        }
     }

The binding $entry.displayLogo doesn't work. I'd like to be able to update the configuration for a particular entry from inside the view (ie. Update the value of displayLogo).
Any idea how I could achieve this behavior? I know I could pass an index through the sub views and use something like entries[entryIndex].displayLogo but it doesn't seem very clean.
Thanks

Your question is not related to Defaults, but I will answer it.
You need to pass the Binding to the EntryView. Notice the $ in ForEach. That new in Xcode 13.2 or something.
struct EntryView: View {
      @Binding var entry: Entry

      var body: some View {
            Text("Inside entry \(entry.id)")
             Toggle("Display logo?", isOn: $entry.displayLogo)
       }
}

struct EntriesView: View {
    @Default(.entries) var entries

    var body: some View {
        HStack(spacing: 10) {
            ForEach($entries) { entry in
                 EntryView(entry: entry)
            }
        }
     }
}

Unrelated, but Entry should be a struct. And you don't need the type annotation on @Default.
And I would recommend using UUID as id.

Thank you for your help, you saved me a lot of time.
Allow me to buy you a couple of beers.


NTR: reduced folate transmembrane transport

Request on behalf of Hadil Alrohaif;
Can we request the term "reduced folate transmembrane transport"
As a child of GO:0055085 transmembrane transport.
The term GO:0008518 reduced folate carrier activity would be part_of the new term.
Evidence is from PMID:14609557, figures 1 and 2 where the authors show uptake of methylenene tertrahydrofolate, a reduced folate, by ARPE-19 cells (retinal pigment epithelium).
Please add GOC references;
GOC:BHF
GOC:hal
Reported by: rachhuntley
Original Ticket: geneontology/ontology-requests/11801

Hi Harold (@hdrabkin) & David (@ukemi),
Wondering if I can get some advice on biochemistry.  It would be good to link the requested term to ChEBI in order to get some useful classification rather than trying to place by hand.
The paper descibes the function as "transports reduced-folates, such as N5-methyltetrahydrofolate (MTF)."   The best I can do right now is to refer to 5,10-methylenetetrahydrofolic acid (CHEBI:20502), which I believe matches the chemical mentioned in the paper, but clearly the activity is broader than just transport of '5,10-methylenetetrahydrofolic acid'
From a quick look at the relevant wikipedia entry, it looks like everything to the right of folate in this pathway could be referred to as reduced folates:
folate → dihydrofolate → tetrahydrofolate ↔ methylene-THF → methyl-THF
Is this correct?  This would help with the text def, but I’m also wondering if it would make sense to either ask ChEBI for a ‘reduced folates’ term ourselves, or to make a GOCHE union term for these.
What do you think?
Cheers,
David

id: GO:0098838
label "reduced folate transmembrane transport"
definition "The directed movement of reduced folate (dihydrofolate, tetrahydrofolate, methylene-tetrahydrofolate or methyl-tetrahydrofolate) across a membrane." [PMID:14609557]
namespace "biological_process"
is_a: transmembrane transport
is_a: amide transport
It would be good to add some more classification.  folic acid transport also has these classifications (some inferred from ChEBI):
'amino acid transport'
'cofactor transport'
'dicarboxylic acid transport'
'modified amino acid transport'
'vitamin transport'
'dicarboxylic acid transport' doesn't apply to reduced folates.  Do the other classifications?

Closing.  Term added in July.  Think I kept this open for comments on additional classifications.   @hdrabkin @ukemi - any suggestions?


Drcandacemakedamoore/new algos

adding autocombat experiments

needs added explanation about autocombat grouping, and how notebooks can be run with regular or fg_ files for smaller groupings


Align variables section transformer

If you have interesting data that can be used as test data - please share.

You can check also with the optional equal signs.


Avoid log2(), log10(), cbrt(), trunc() on Android

Fixes #1325.

@HouQiming I'd appreciate if you could test I got the fix right :)

Good thing I implemented all those workarounds. :)

That's the good part of supporting C89: it provides a baseline for environments which are C99 but not all the way.

Just saw this. I'm on vacation right now and don't have an Android machine
handy. I'll test it once I get back to work.
Thanks for the fix!
On Tue, Jan 31, 2017 at 6:53 AM, Sami Vaarala<EMAIL_ADDRESS>wrote:

That's the good part of supporting C89: it provides a baseline for
environments which are C99 but not all the way.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/svaarala/duktape/pull/1341#issuecomment-276218212,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHrIF7TeYiOcqPzANw1qz_kuQCXqsCI8ks5rXmnbgaJpZM4LyA7G
.

@HouQiming Sounds good, have a good vacation :)

Just tested it. It works :)


Allow to declare variables of type dtMeshTile

In general this change allows to write code like:
dtMeshTile tile;

With the only private copy constructor it's not possible.
Also it's completely ok to copy and copy-assign object of this type because there is no custom destructor. For example if I need to write custom function to map unsigned char* into dtMeshTile like it's done here I want to write a function:
dtMeshTile asMeshTile(unsigned char* data);

And this is fine because dtMeshTile is only a view to data.
All this is required to check equivalence of two navmesh tiles when one tile is dtNavMesh::getTileAt result and the other is dtCreateNavMeshData output.

And this is fine because dtMeshTile is only a view to data.

This is not true, dtMeshTile owns the data, when you create it with dtTileFlags::DT_TILE_FREE_DATA. Your change would cause UB.
This is managed externally, see here for example. It's a weird design choice that there is no destructor in dtMeshTile itself, but it's probably due to the lack of move semantics.

dtMeshTile owns the data, when you create it with dtTileFlags::DT_TILE_FREE_DATA.

I can't agree with this statement.
dtCreateNavMeshData doesn't not have a flag to say who owns the data. It just provides unsigned char* to somehow allocated and initialized memory. After dtCreateNavMeshData is called, a caller owns the data. And that's where I need to view this data as a structured type. My alternative here is to create a new type that will be mostly a copy of dtMeshTile definition.
Then this data may or may not be passed to dtNavMesh::addTile. There it's possible to say whether the data ownership will be transferred to dtNavMesh or no via flags argument. After that the owner of the data is either the caller or dtNavMesh object. If it's dtNavMesh object it will be deallocated later by one of the following functions:

dtNavMesh::~dtNavMesh
dtNavMesh::removeTile

Ownership means to have some logic to release owned resource. So I don't agree that dtMeshTile is the owner at any point of this chain.

Your change would cause UB.

Could you provide some reference how an absence of a private copy constructor declaration and a copy-assignment operator is UB? Or explain what other code constructions may cause it.

This is managed externally, see here for example.

This contradicts with your first statement.
I assume your arguments are about copying dtMeshTile objects. Note that I never mentioned that I actually want to copy it. But let's say users start to do this after it's allowed. What will happen? Need to take a look at dtMeshTile members. All of them either integral types or pointers. Is it ok to copy integral type or a pointer? Yes, it is. The next question. What if the underlying value of dtNavMesh::getTile result pointer is copied? The copy is valid until dtNavMesh destructor or removeTile function is called for this tile if dtNavMesh owns the data. It will be UB to dereference pointer members of a copy after the data is deallocated. But it's a common practice in C++ to have such situation as a result of a trade-off between safety and performance. Iterators to STL containers is one example. And dtNavMesh::getTile is also an example because it's result may become a dangling pointer after tile data is deallocated. Similar logic applies to the copy-assignment.

Ownership means to have some logic to release owned resource. So I don't agree that dtMeshTile is the owner at any point of this chain.

Thanks for the detailed elaboration! You're right, the dtMeshTile stores the pointer, but the logic (memory management) is external.
So if I understand you correctly, you consider dtMeshTile simply as a POD struct (which happens to be used as the internal storage of nav meshes), and following that argument, it should have value semantics, including copy/assign.
I can only speculate, but I would guess dtMeshTile was never intended to be managed by the user. It's an type used internally for tile storage, however the dtNavMesh API hands out pointers to it (but not values). This abstraction is leaky in many ways and I would argue that this type should have been private in the first place, as many of its fields are implementation details.
You mention you need this to check for equivalence between mesh tiles; what does this involve? You need actual inspection beyond just checking for tile identity (via index/ID)?

You mention you need this to check for equivalence between mesh tiles; what does this involve? You need actual inspection beyond just checking for tile identity (via index/ID)?

I need a partial deep equality. Something similar to:
bool operator==(const dtMeshHeader& a, const dtMeshHeader& b) {
    return /*...*/ && a.polyCount == b.polyCount && /*...*/
}
bool operator==(const dtMeshTile& a, const dtMeshTile& b) {
    return *a.header == *b.header
         && std::equal(a.polys, a.polys + a.header->polyCount, b.polys)
         && /*...*/;
}
bool isEquivalent(const dtMeshTile& a, unsigned char* b) {
    return a == asMeshTile(b);
}

Without this pr I will need to create another type instead of dtMeshTile and convertors from dtMeshTile and unsigned char*:
struct MeshTileView { /* same members as in dtMeshTile */ };
bool isEquivalent(dtMeshTile& a, unsigned char* b) {
    return MeshTileView(a) == MeshTileView(b);
}

And it's not a problem. But it would be nice to reuse dtMeshTile.


Banner for collectives with pending approval

When creating a new collective, the collective is not necessarily immediately activated, it needs to be approved first.
While it's in this state, a banner on top of the collective should explain the status, similar to what we're doing with archived collectives.
API
To enable that, we want to introduce a new flag/date on the API side to know more precisely the current status of the collective: approvedAt.

approvedAt will be set the the current date when the collective is approved by the host or automatically approved (GitHub flow)
approvedAt will be set to NULLwhen the collective change the host or is archived
if not already, the host should be removed when the collective is archived  (member table, hostCollectiveId column)
the approvedAtcolumn needs to be exposed as isApproved in GraphQL (similar to isArchived)

Frontend

If the collective is not archived (isArchived=false) and not approved (isApproved=false) a banner should be displayed on top of the collective page. This happens in the Collective component and should be similar to what is done when the collective is archived.
The banner should not be displayed when the collectiveCreated banner is
The collectiveCreated banner should have a different message regarding if the collective is approved or not. See: https://github.com/opencollective/opencollective/issues/2100

Link to impacted files

approvedAt will be set the the current date when the collective

is approved by the host: https://github.com/opencollective/opencollective-api/blob/master/server/graphql/v1/mutations/collectives.js#L440
automatically approved (GitHub flow): https://github.com/opencollective/opencollective-api/blob/master/server/graphql/v1/mutations/collectives.js#L176

approvedAt will be set to NULL when

the collective change the host: https://github.com/opencollective/opencollective-api/blob/master/server/models/Collective.js#L1502
or is archived: https://github.com/opencollective/opencollective-api/blob/master/server/graphql/v1/mutations/collectives.js#L614

A $100 bounty was attached to this issue. Anyone submitting a Pull Request will be rewarded with $100 when the Pull Request is reviewed, accepted and merged. More info.

I'll like to work on this.

Hey @Shorpee, are you working on this or should I start working on it?

@yellowwoods12 In our bounty program, we prefer if contributors stick to 1 issue at a time. Let's finish https://github.com/opencollective/opencollective-frontend/pull/1960 first?

@znarf Sure. It's almost done.

Hi @znarf, can I start working on this issue if the last one is done?

Sure, let's set a time limit of one week to start.

@yellowwoods12 sorry, the time limit is exhausted, I can't lock this bounty longer for you.
This bounty is available to participants of the OSCA (Open Source Africa) event.

i am shedy from osca PH I will be working on this...

@znarf can I give this issue a shot?


MyGet Plan&Link Change

Url and description of MyGet, changed
Fixes Issue #8386

Actually it's kind of confusing, because they don't state anything about the free plan, even in their pricing page: https://www.myget.org/pricing
Do they really support free plans? How can we redirect people there stating that they do have a free plan if MyGet doesn't even mention it on their site...

@giotavekiari According to the post by someone associated with MyGet (in the linked issue) they do convert trial accounts to free accounts. I agree it does not make that clear on their website, but I don't see why we can't link to it. Even if it was a pay-only service, I don't think it matters. We're just letting users know the options they can use. This is why I was suggesting we simplify the text for MyGet and not mention the free option.

Made some changes as discussed! Please check again! Thank you!


Missing  Documentation

Hi there, I Can't find good documentation at this time for this Gem, it seems useful but no documentation for rails. I want to use it to optimize the image that users upload to my web app.

This gem is for optimising rails assets, optimising uploaded images depends on the way or gem you use for processing files and their transformation. Parent gem parent gem image_optim can be integrated into your pipeline to optimise image files or image data.

This gem is for optimizing rails assets, optimizing uploaded images depends on the way or gem you use for processing files and their transformation. Parent gem image_optim can be integrated into your pipeline to optimize image files or image data.

Good, thank you for the quick reply, I'm using the image_optim now, I thought this Gem does the same work and it's for rails. Now I understand.


Add labels in the example form

Currently, the example form lacks labels in the fields. This makes it harder for screen readers to use the form.
Solution:

Add a id to each field.
Add a <label> element with the for attribute being the same as the id for each field The text of the element is the label.

This is not applicable to the submit button.

Thanks for this. I'll have a look when I can.
If you fancy submitting a PR, the docs are on the gh-pages branch: https://github.com/eduardoboucas/staticman/tree/gh-pages/docs


deploy changes since a commit

deploy changes since a specified commit
grunt shopify:upload-changed --commit=968f3a3299e1ebd268f3109ce02027a164f90803

@patrickleet would you mind adding this new command and flags you created to the README.md so we can reference quickly?


While scrolling, if the mouse gets released outside of the frame the state remains pressed

I know this is more likely a browser issue but this is a nice improvment that could be acheived: could we reset the state or retest the mouse down states when mouse is out of the content?
you can reproduce on https://codepen.io/jakob-e/pen/wBmgYb by clicking the content to scroll and release mouse in the editor area since the preview is in a iframe
maybe somthing using code such as
document.body.addEventListener('mouseleave', function() {
  alert('left body we must recalculate on next mousemove if we still press the mouse button');
})

would help fixing this

@zeachco This si issue for original scroll library https://github.com/cubiq/iscroll

Thanks for redirecting


Failed to match boolean value

Maybe I misunderstood something, but the two behaviors (trivial Boolean logic) below are weird?

https://nikku.github.io/feel-playground/?e=true&c=%7B%0A++%22%3F%22+%3A+false%0A%7D%0A&t=unaryTests&st=true
https://nikku.github.io/feel-playground/?e=false&c=%7B%0A++%22%3F%22+%3A+false%0A%7D%0A&t=unaryTests&st=true

Sounds like a bug indeed. Evaluation should match by value, i.e. this.

Just so that I understand the issue properly.
What about this scenario, what should the unary test return?
https://nikku.github.io/feel-playground/?e=3+<+1&c={
++"%3F"+%3A+false
}
&t=unaryTests&st=true
Do we ONLY match by value in the very specific case where both the value and test are a simple boolean?

I believe it's already fixed up by nikku? The initial problem was comparisons with zero (not the case here). Again, fixed, I guess.

I believe it's already fixed up by nikku? The initial problem was comparisons with zero (not the case here). Again, fixed, I guess.

No, this is not fixed (yet). ? denotes the current value, it is matched against a unary test.
If we enter a test that is always truthy 3 > 1 = true then we match whatever ? is against the result of that test, i.e. true if ? = true, else false.

Looking into our own documentation:

A unary-tests expression returns true if one of the following conditions is fulfilled:

!! The expression evaluates to true when the input value is applied to it.
The expression evaluates to a list, and the input value is equal to at least one of the values in that list.
!! The expression evaluates to a value, and the input value is equal to that value.
The expression is equal to - (a dash).

The issue here is that for booleans, the two scenarios highlighted via !! are ambiguous. A small improvement would be to always compare boolean values to their test by value, but then it leads to other errors:
?: true*, test: < 5
true < 5 becomes 1 < 5 becomes true**
true* === true** becomes true ✔

?: false*, test: < 5
false < 5 becomes 0 < 5 becomes true**
false* === true** becomes false ❌

So I guess the "bug" that exists within the codebase is that we do not discriminate between when the expression itself is driven by the test value vs when it isn't.
An example, if I understand it correctly, is that we expect false, 0 > 1 => true but false, ? > 1 => false @nikku

@Skaiir I'm not sure I fully understand what you mean.
What would help is if you'd file a PR that contributes test cases to show which cases currently conflict.
Even better we could raise this with the DMN TCK to get clarity across all implementors.

Related discussion in feel-scala: https://github.com/camunda/feel-scala/issues/864#issuecomment-2188837091.
TLDR: We want to assess our current handling of such cases.


Suggestion: subcategories and pages

Over time as more resources get added to this site, it seems that it could easily become overwhelming for the current single page format. For example, within a single category such as image generation, there are additional tools and resources such as controlnet, adetailer, civitai.com for models, the forge version of a1111 sd web ui, etc, to name a few.
One idea would be to keep the main page in it's current format, but when the number of resources within a given category grows too much, only show the top X featured items for that category on the home page  (say 5-10 items). Then, make the category title on the home page a link that goes to a page specifically for that category. The category page could have the same format at the main page, but just showing subcategories instead.

Thanks for the suggestion. Yes fully agree we need to introduce the concept of sub(category)-pages, that is accessible under a "more..." link.
Will welcome a technical working concept for this, as have not (yet) started to working on this myself now.
One think am personal working on first is acquiring relevant links in an automated fashion (via github api), which are thousands, and will soon (couple of weeks) need these subpages to land on.

Merged this into a new branch for now.
Still working on an automated way of generating links that will go in to the subpages. When that is done, think we should be able to merge this as-is. Will keep this thread open until that is done, or in case we need to make additional changes.


Webhooks broke reactions

It did kind of work, but reacted only to the message before (since certain things may not be awaited). Now it throw's unknown message.

Okay it still adds reactions but it adds to the previous message. A way needs to be found to wait for the webhook to successfully execute before finding the latest message. Alternatively, a way needs to be found to get the id of only that webhook message. For normal messages you can do that by doing the following:
msg = await ctx.send("Hello world!")
 await msg.add_reaction("<:upvote:767964478570496030>")
 await msg.add_reaction("<:downvote:767964478574690304>")

The problem is that webhook.execute() cannot be awaited nor is it possible to do what is shown above anyways.