Use arithmetic operators for arithmetic opperations

Use +, -, *, /, % instead of Add(), Sub(), Mul(), Rem()

Implemented in #25


How to correctly run the demo?

Hey,
I am trying to run the demo, but I only get a crash on Device.init, please give me some help.
Here is the information:
Xcode: Version 15.0 (15A240d)
Device: iPhone 11
OS: iOS 17

try to add any Metal file
u can just create empty .metal file

ok!


Error after updating to 2.9

I just updated my Neo GUI (windows 10) to version 2.9, but i'm receiving the following error:

System.NotSupportedException
An attempt was made to load an assembly from a network location which would have caused the assembly to be sandboxed in previous versions of the .NET Framework. This release of the .NET Framework does not enable CAS policy by default, so this load may be dangerous. If this load is not intended to sandbox the assembly, please enable the loadFromRemoteSources switch. See http://go.microsoft.com/fwlink/?LinkId=155569 for more information.
at System.Reflection.RuntimeAssembly.nLoadFile(String path, Evidence evidence)
at System.Reflection.Assembly.LoadFile(String path)
at Neo.Plugins.Plugin.LoadPlugins(NeoSystem system)
at Neo.Program.Main()

The 2.7.6 version was working without problems.

https://github.com/neo-project/neo-gui/releases/tag/v2.9.0
I fixed this release and re-uploaded the file. Please try it again.

I have the same problem using the new release (same error). However, when I extract it using winrar (instead of copy/paste using windows' native zip program) it somehow works. So this probably is not a Neo GUI problem, but more to do with the way it was zipped or how windows own unzip handles this file.

I have the same problem using the new release (same error). However, when I extract it using winrar (instead of copy/paste using windows' native zip program) it somehow works. So this probably is not a Neo GUI problem, but more to do with the way it was zipped or how windows own unzip handles this file.

@sempiterna I had the same issue unzipping with Windows native unzip program trying to unzip the latest v2.9.0. I then tried unzipping with 7-Zip and it worked. Thanks!


Pushing code review issue to a different branch with a local branch not existing on repository

Hi,
I have an issue when i want to push a code review when I'm on a feature branch not pushed on origin, to another branch pushed to origin.
Exemple: Push to develop from local branch feat/toto
If i push the code review without pushing beforehand the branch feat/toto into the repository, the push is done like this.
git -c credential.helper= -c core.quotepath=false -c log.showSignature=false push --progress --porcelain origin refs/heads/feat/toto:refs/heads/refs/for/develop
When i push the branch feat/toto into origin.
git -c credential.helper= -c core.quotepath=false -c log.showSignature=false push --progress --porcelain origin refs/heads/feat/toto:refs/for/develop

Config:
IntelliJ IDEA 2021.3.1
Gerrit 1.2.5-203
Thanks

@YMasliah-neotys: Do you think this is related or the same as #392?

Yes probably

@YMasliah-neotys: Thanks. Please report if it does not fix the bug. Closing as duplicate of #392 for now.


Improve readability in "open source newbie" article

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding. ✈

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding. ✈


Swift package doesn't work

I just tried to add SpringIndicator swift package but

Also tried to set an exact version to 5.1.0 but still no luck

5.1.0 tag is added now.
please try again.

@KyoheiG3 done and it's working.


fix: import optimization

Thanks for your time on this pull request.
However, I will close this as I prefer the current name. This config package is coming from another repository and we use it in multiple places with the same shortcut.


[GSSoC] Number Guesser Game

Add a number guesser game. The game is simple. You will have to enter a random number and see if the system has guessed the same number

i would like to work on this project

i would like to work on this issue

assigned to @iamvbenz49
Complete this within 1 week and
Include a screenshot or deploy link to assess your project and thereafter include your project in the main code with the following contributing.md guidelines.

@Dharmesh177 Sorry? but I created this issue , with the screenshot... Shouldn't it be assigned it to me?

I'm working on it

Sorry @PoulavBhowmick03 You can make PR Will consider accordingly.

@iamvbenz49 Complete it immediately

ok!! I finished the game just now :), it would be easier for us if there's some contributing guidelines


[AMBARI-24610] [Log Search UI] Show user friendly component name in query input

What changes were proposed in this pull request?
I added the componentLabel pipe to the parameter rendering template.
How was this patch tested?
It was tested manually and by unit tests:
PhantomJS 2.1.1 (Mac OS X 0.0.0): Executed 271 of 271 SUCCESS (9.72 secs / 9.625 secs)
✨  Done in 37.85s.

Please review Ambari Contributing Guide before opening a pull request.

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

The tests failures have no connection with the current change.


Orphaned chromedriver processes

When moving from Serenity-js: 1.2.4 to Serenity-js: 1.3.0. When running more than one scenario (not sharded) there are orphaned Chromedriver instances hanging over at the end of each scenario.
I am running Windows 7, chromedriver 2.28

Sorry this is a duplicate of #56


EditorPlugin.AddCustomType breaks when used custom inheritance

Godot version: 3.2.2.stable.mono
Issue description:
EditorPlugin.AddCustomType not includes custom base script.
If custom base script is passed (Pic1) child script is not shown in "Add Node" Window at all (Pic2).
Expected behavior like GDScript class_name (Pic3).
**Pic1**

**Pic2**

**Pic3**

Steps to reproduce:

Activate plugin in "Plugins" tab.

Minimal reproduction project: custom_node_issue.zip
Thank you!

Workaround: Set the base type to be the closest of the built-in types. E.g. if you have a hierarchy
Node

ParentType

ChildType

Set the base type of ChildType to be Node. This way the type is at least displayed in the list (albeit in a flat hierarchy).

I just got into the same problem with Version v4.0.beta10.mono.official [d0398f62f]. The workaround makes it really difficult for complex systems like BehaviourTrees with many nodes that require to be assembled in the right structure like selectors, decorators and leaf nodes. Everything goes below the Node Type and it becomes very unorganized.  Would be great if this bug could be somehow scheduled to be fixed.


Command R+ is broken?

I downloaded the new one and it would output garbage over and over. Figured it was the quant so I loaded the previous versions. All output repeating nonsense. Tried both textgen and tabbyAPI. Other models appear to work fine. Sometimes I got it to be coherent, but as soon as I tried any sampling it would irreparably break. Don't know who the offender is but tried dev and 0.1.9, happened to both.
built 0.1.8 and issue doesn't appear there.

You need to be more specific. What platform, first of all? And do you have fasttensors enabled?

I'm on linux with torch 2.4.0. I don't use fasttensors because my server caches model weights in ram and direct reading breaks it.

I've seen some sporadic issues with Command-R specifically, but I was only able to reproduce them for a while, then the error spontaneously stopped happening and I'm still trying to trigger it again.
What I was able to determine is that it happens specifically for Command-R and specifically has something to do with tensor storage, since fasttensors reliably eliminated the error. My best guess right now is that safetensors does something radically different when loading to system RAM, as it will when loading the MLP up and gate tensors, since those are remapped in system RAM before being offloaded to the GPU.
I'm still trying to recreate the error so I can maybe try to figure out exactly where it's happening. I've had so many issues with the safetensors library, though, perhaps it's just time to drop it altogether. Idk.

I updated to latest safetensors and in txgui it will output fine with low context. If I hit it with sillytavern API requests it's back to saying RaulRaulRaulRaul.
Getting rid of safetensors is fine as long as i'm not stuck with fasttensors, that's like loading the model first time, every time. I'll enable it and see if it at least gets rid of this behavior.

Yep, on Linux, fasttensors is fast because it bypasses the OS cache, which is great if you have really fast storage but of course otherwise you just lose caching for no good reason. There are other approaches, of course, including a fallback mode I wrote just for Windows where safetensors has some other really annoying issues (with memory management). I could adapt that for Linux and you should get the best of both worlds.
It would be nice to narrow down the problem with Cohere first, though, to make sure it's actually limited to safetensors and not just some unrelated bug that happens to be exposed by something in safetensors.

I tried cuda 12 and 11 and fast-tensors. no difference.

I'd still need a lot more info to be able to reproduce the issue.

I tried both latest TabbyAPI and exllama_hf in textgen. My settings are simply .9 temp and min_P of .03. It's on 3x3090 with flash_attn. Tried 4 and 8bit cache.
Exllama_HF is coherent in notebook mode on ooba, initially. API requests from sillytavern return as above and then the model breaks in ooba too. Once it calls upon the spirit of Raul, it remains on mescaline until reloaded.
Am using this quant: https://huggingface.co/BigHuggyD/c4ai-command-r-plus-08-2024_exl2_4.5bpw_h6
Not sure what else to add.. all prompts do it. Downgrading to 0.1.8 and the exact same stack/prompt/model works.

In TGUI I tried disabling the bos token and still get Raul through the API but in the UI itself I get different nonsense like 0s. It definitely wasn't this way in 0.1.8 because I used the model a lot. I was sorta hoping to try it in TP but some scratch buffer isn't supported. This seems like a slightly worse problem.
I was going to try to checkout before likely commits on 0.1.9 and try to narrow it down if it's indeed this hard to reproduce.

The offending commit is: https://github.com/turboderp/exllamav2/commit/036506f27310153ddd35701013c87608eeb15e17
The one before works. All after call on Raul. Sampling didn't have anything to do with it, you can turn off do_sample.

I don't have a Windows PC that can run this model, but apparently on Linux there's an issue with safetensors. Enabling fasttensors fixes it here.

Enabling fast tensors does not fix it for me and I am on linux. The commit before "Use high priority stream for forward pass" works perfectly no matter how I load it.

Okay, so I think I tracked this down now. It's specific to Tabby because of the way it spawns threads to serve requests. Some state in Torch is thread-local, which could cause ExLlama to use the wrong CUDA streams under specific conditions, causing tensors to end up being moved between devices prematurely.
Anyway, I've pushed a commit to the dev branch that hopefully fixes it. Before:
  "choices": [
    {
      "index": 0,
      "finish_reason": "length",
      "stop_str": null,
      "message": {
        "role": "assistant",
        "content": "RâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâul小行星RâulRâulRâulRâulRâulRâul小行星RâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâul小行星RâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâulRâul",
        "tool_calls": null
      },
      "logprobs": null
    }
  ],

After:
  "choices": [
    {
      "index": 0,
      "finish_reason": "length",
      "stop_str": null,
      "message": {
        "role": "assistant",
        "content": "Raul is a name typically used for males, and it is of Spanish origin. Raul means \"wolf\". The meaning of Raul is \"wolf council\". It is derived from the Hispanicized form of Raoul, a French name that means \"wild counsel\". \n\nRegarding where you can find Raul, it depends on the specific Raul you are looking for. There may be many people named Raul in the world, and their location would depend on their specific circumstances. Some ways to potentially find someone include:",
        "tool_calls": null
      },
      "logprobs": null
    }
  ],

So far, in my testing, it works, at least on Tabby.

Thanks! It's working here in both textgen and tabby. No more raul.


Manage /robots.txt in Datasette core, block robots by default

See accompanying Twitter thread: https://twitter.com/simonw/status/1424820203603431439

Datasette currently has a plugin for configuring robots.txt, but I'm beginning to think it should be part of core and crawlers should be blocked by default - having people explicitly opt-in to having their sites crawled and indexed feels a lot safer https://datasette.io/plugins/datasette-block-robots

I have a lot of Datasettes deployed now, and tailing logs shows that they are being hammered by search engine crawlers even though many of them are not interesting enough to warrant indexing.
I'm starting to think blocking crawlers would actually be a better default for most people, provided it was well documented and easy to understand how to allow them.
Default-deny is usually a better policy than default-allow!

A few options for how this would work:

datasette ... --robots allow
datasette ... --setting robots allow

Options could be:

allow - allow all crawling
deny - deny all crawling
limited - allow access to the homepage and the index pages for each database and each table, but disallow crawling any further than that

The "limited" mode is particularly interesting. Could even make it the default, but I think that may be a bit too confusing. Idea would be to get the key pages indexed but use nofollow to discourage crawlers from indexing individual row pages or deep pages like https://datasette.io/content/repos?_facet=owner&_facet=language&_facet_array=topics&topics__arraycontains=sqlite#facet-owner.

I could try out the X-Robots HTTP header too: https://developers.google.com/search/docs/advanced/robots/robots_meta_tag#xrobotstag

https://twitter.com/mal/status/1424825895139876870

True pinging google should be part of the build process on a static site :)

That's another aspect of this: if you DO want your site crawled, teaching the datasette publish command how to ping Google when a deploy has gone out could be a nice improvement.

At the very least Datasette should serve a blank /robots.txt by default - I'm seeing a ton of 404s for it in the logs.

Actually it looks like you can send a sitemap.xml to Google using a GET request to:
https://www.google.com/ping?sitemap=FULL_URL_OF_SITEMAP

According to https://developers.google.com/search/docs/advanced/sitemaps/build-sitemap

Bing's equivalent is: https://www.bing.com/webmasters/help/Sitemaps-3b5cf6ed
http://www.bing.com/ping?sitemap=FULL_URL_OF_SITEMAP

I was worried about if it's possible to allow access to /fixtures but deny access to /fixtures?sql=...
From various answers on Stack Overflow it looks like this should handle that:
User-agent: *
Disallow: /fixtures?

I think another thing would be to make /pages/robots.txt work. That way you can use jinja to generate a desired robots.txt. I'm using it to allow the main index and what it links to to be crawled (but not the database pages directly.)

You can generate xml site map from the online tools using https://tools4seo.site/xml-sitemap-generator.


Add Mnemonic::from_entropy and Mnemonic::from_entropy_hex

This is the opposite of get_entropy and get_entropy_hex, it's needed if you want to go from the entropy to the mnemonic.
I noticed there was some related discussion in #5, so I'm not sure if that was ever resolved.

Hi :)
I don't see any issues with this at all, looks good to me.
Please let me know if there are any other changes that would make sense, and I will try to get it merged in quicker than I did this time :)
Thanks!


Deprecate alias_method_chain and use in-memory sqlite3 for specs

The method alias_method_chain no longer exists in Rails 5.1+. The patch uses prepend to replace alias_method_chain, and simplifies the loading of specs with in-memory db.

Merged and released as version 0.1.3


add docker builds

DOCKERHUB_TOKEN and DOCKERHUB_USERNAME secrets need to be created.

DOCKERHUB_TOKEN and DOCKERHUB_USERNAME secrets need to be created.

done :)


Map & properties are responsive and can be toggled in small screens

Description
This PR addresses issue #260
The find page is now responsive and can toggle between the map/properties in small screens (< 640px, following Tailwind's sm breakpoint). The toggling is handled by a state and display: none with Tailwind variants.
I made few styling changes and added some code to handle side effects. Namely, rendering the properties separately affected the pagination's styling on the initial load.
Testing/Proof
Desktop to mobile
https://github.com/CodeForPhilly/vacant-lots-proj/assets/28910543/1df3d7b3-9106-4ba7-b9ab-2a4a12ed1f16
Mobile interactions
https://github.com/CodeForPhilly/vacant-lots-proj/assets/28910543/8295bc55-5157-43e6-ab44-6e761aa6c269
Overflow for x & y
https://github.com/CodeForPhilly/vacant-lots-proj/assets/28910543/bf5eabdd-40c0-40b5-b187-844b2a4d8ede

Hi @thansidwell @Moylena! I was hoping to get some clarifications.
How should the tooltips and filter work for small screens? For the latter, does this mean pressing on filter would swap to the properties component?
A big problem has been the control bar. It overflows for almost all screens < lg-ish and the options aren't visible. This also causes double-scrolling (x & y), which I was able to fix with the exception of a small interval around 750px. At this point, the pagination bar also overflows. I'm not sure if we wanted to handle that here or in a future PR.
Thanks! :)

@amyyeung17 Great work! Here's a video and a list of the issues I noticed. https://share.cleanshot.com/lCsH0zGY

Change the View toggle button to toggle between a map and a grid of four squares to toggle between the map and the cards. I realized we don't have the toggle function adding to the button component, and I think this is approach is better. The icons are attached here.

Stop the content from shifting up and down. If you can.
When you are looking at the map, click on a property, and the property shows up, when you click Back you should go back to the map.
You should be able to click on the filter on the map view. Right now it doesn't work. I'm guessing that's just tricky because of where it's located in the HTML.

@thansidwell Noted, thanks for the feedback! I think I can do 2 now that the navbar is merged. I haven't thought much about the rest, but I don't foresee too many difficulties.

@thansidwell Addressed your comments and took a shot at making the info less cramped. As expected, 2 was resolved with the navbar. :) I also added some hover states for some buttons and property cards.
There are still several things to work on like the nav options, pagination, and odd scrolling behavior. Perhaps these would be best done in new PRs?
Here's the updated version:
https://github.com/CodeForPhilly/vacant-lots-proj/assets/28910543/bc0a994e-8906-4891-b82b-1094986cd016

@thansidwell Realized when working on a different PR that the filter option acted funky if it's applied in-between resizing and added some fixed. Besides that, I think this PR is done.
https://github.com/CodeForPhilly/vacant-lots-proj/assets/28910543/6bd9163a-813a-4cf0-b925-5d236d90bb1e

Hey @amyyeung17! Sorry, I didn't respond earlier. Somehow I missed this. It looks great!
There are a few minor issues that are being resolved with other tickets or I think are best made their own ticket.


add sessionDidStartRequest to TurboNavigationDelegate

oops, sorry, I meant to open this PR against my own fork 😓


Add support for std::num::NonZero integer types

This pull request adds support for various integer types of std::num
Fixes #2650

Please add a news fragment by following the instructions at https://github.com/PyO3/pyo3/blob/main/Contributing.md#documenting-changes Thanks!


Misc logging cleanup + commentary

I played around with the logging module this morning from an end-user perspective. It was a lot of fun! There are some fixes we should definitely merge; some questions are open for discussion. I tried to keep the commits separate + clean, so we could split up and cherry-pick accordingly. This is all in the interest of making dbt look + feel "like dbt" by v1 final.
Critical fixes

#4266 turned on JSON-formatted logging everywhere all the time, except for the legacy logger. Re-enable the choice between create_json_log_line and create_text_log_line based on this.format_json

Touch-ups + cosmetic fixes

For events whose messages were prefixed with \n, instead precede them with fire_event(EmptyLine()). This is much cleaner to look at, since all (non-wrapped) log lines get the same frontmatter (timestamp + log level)
I happened to notice that ConcurrencyLine could be more structured (unless there's a good reason why it wasn't)
If we're going to include log levels on the CLI (more on this below), disable duplicative WARNING prefixes in messages / formatted via warning_tag

Commentary

Log levels: These are amazing to have in the File. I find they add a bit too much noise on the CLI. I'd be happy with either (a) turning them off for CLI logging, or (b) colorizing warn/yellow + error/red to raise the signal from the noise. (Also, I believe #4266 removed width-fixing for log levels, so I added that back in.)
Colorization: Ideally, we'd have a way to turn this off for both File + JSON-formatted logging. This would require a little bit of plumbing: dbt.ui.color would need information from the events module, beyond just flags.USE_COLORS. Since color methods are mostly called in messages defined on event classes, I wonder if this could work by adding a use_color(self) -> bool method to the File + Cli base types, and reimplementing the color methods to check self.use_color (where self = event class).
EmptyLine: Just like colorization, this is great for CLI + text formatting, but we'd ideally have a way to turn it off for File or JSON logging. (This is what logbook accomplished via the TextOnly() wrapper; I realize I was a bit inconsistent in whether I used this.)

Broadly, I find that I want:

color, concision, clean spacing on the CLI
colorless + compact verbosity in the File + JSON-formatted

Checklist

[ ] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change

@nathaniel-may @emmyoop @iknox-fa Tagging you here for visibility. The critical fix noted above is the only thing we must have for RC2. I know the rest of this is lower priority than the other work we've itemized in #4260.

If we're going to include log levels on the CLI (more on this below), disable duplicative WARNING prefixes in messages / formatted via warning_tag

This is a good reason why I think the warning stuff should live in the events mod eventually.

Closing in favor of https://github.com/dbt-labs/dbt-core/pull/4388


How to use Generated Records?

Hi,
I'm going through the JOOQ tutorial and am having difficulty understanding if I can use Records with your plugin.
Configuration jooqConfig = new DefaultConfiguration().set(SQLDialect.MYSQL);
AsyncClassicGenericQueryExecutor queryExecutor = new AsyncClassicGenericQueryExecutor(jooqConfig, mySQLClientAsync);

Future<Integer> updatedCustom = queryExecutor.execute(dslContext -> {

    //  ## Fails Immediately ## //
    FormRecord formRecord = dslContext.fetchOne(Tables.FORM, Tables.FORM.ID.eq(1));
    formRecord.setIsActive(false);
    formRecord.store();

    // ## This Works ## //
     return   dslContext.update(Tables.FORM)
                                     .set(Tables.FORM.DESCRIPTION, "this works")
                                     .where(Tables.FORM.ID.eq(1));
});

fetchOne fails immediately

org.jooq.exception.DetachedException: Cannot execute query. No Connection configured

Thanks for any help

I was able to get it working by passing another connection to the JOOQ config.
I understand that this negates the purpose of async, but I'm still confused why the generator generates the Record objects.
Being able to chain statements without all of the nested callbacks seems useful.
Is this a use-case for the RX apis?
Sorry if this is obvious, I'm still wrapping my head around Vert.x & JOOQ

Hi @don-yan,
active records do not work directly with vertx-jooq. As you may have noticed, there are three different driver implementations of vertx-jooq:

vertx-jooq-async
vertx-jooq-reactive
vertx-jooq-jdbc
The first two implementations only use the org.jooq.Configuration to determine which dialect should be rendered by jOOQ. The actual query execution is handled by the vertx AsyncSQLClient (PGClient for reactive modules respectively). That means you cannot use any store, update or fetch methods provided by jOOQ at all and you should either use the DAO- or QueryExecutor-methods for this.
The closest you can get is using vertx-jooq-jdbc which should complete your example code without errors as it uses jOOQ not only to generate SQL but also for execution.

Hi @jklingsporn
Is there any way that vertx-jooq-reactive will have the store, fetch and update methods in near future ?


Add "Come back later?" and "Have any feedback?" sections under the pathway page

stems from #33
Figma

What should

"start over"
"export"
"Contact Us"
link to?

Thanks for asking Regina :)
This does need to be implemented:

Contact Us should link to this google form: https://forms.gle/ANBjEcXLut27WAgc7

These parts do not need to be implemented and can be left as empty links:

start over should go back to the Quiz.js page and remove any cache that has been stored -- refer to #100 and #97 .
export should export a PDF of the generated pathway -- refer to #95


bash: nr1: command not found

Description
I typed this in my terminal:
$ git --version
git version 2.11.0
$ npm -v
3.10.10
$ nr1 nerdpack:clone -r https://github.com/philweber/nr1-process-tree.git
bash: nr1: command not found
$
Steps to Reproduce
Expected Behaviour
Relevant Logs / Console output
Your Environment

NR1 CLI version used:
Browser name and version:
Operating System and version:
s$ uname -a
Linux 123459.cloudwaysapps.com 4.9.0-12-amd64 #1 SMP Debian 4.9.210-1 (2020-01-20) x86_64 GNU/Linux
[

Additional context

Hi, @peefour: Building New Relic nerdpacks requires the New Relic One command-line interface (CLI). I have updated the README file to make this more clear. Thank you!


⚠ Bofrost.at has degraded performance

In 595dff9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1491 ms

Resolved: Bofrost.at performance has improved in 0be5075.


Rebuilding Fuctionality

All written code will soon have a new look.

Now we have done the rebuilding, everything looks prettier


Need more information on Hygieia-jenkins Plugin configuration

Hi Team,
I am trying to setup the Hygieia-Jenkins Plugin. I am able to update the Jenkins with HPI file and able to see the post build configuration settings. but i am not sure what needs to be filled in the testxt boxes for for  Publish Artifact Information and Publish Deployment Results.
Could you please help me with this information.
1.Publish Artifact Information(Artifact Directory,Artifact Filename pattern,Artifact Group,ArtifactVersion)
2.Publish Deployment Results(Artifact Directory, Artifact File Name Pattern, Artifact Group,ArtifactVersion
Thanks
Ganga

Check add build step
Sent from my iPhone
On Apr 13, 2016, at 6:56 AM, gangadharahn <notifications<EMAIL_ADDRESS>wrote:
Hi Team,
I am trying to setup the Hygieia-Jenkins Plugin. I am able to update the Jenkins with HPI file and able to see the post build configuration settings. but i am not sure what needs to be filled in the testxt boxes for for Publish Artifact Information and Publish Deployment Results.
Could you please help me with this information.
1.Publish Artifact Information(Artifact Directory,Artifact Filename pattern,Artifact Group,ArtifactVersion)
2.Publish Deployment Results(Artifact Directory, Artifact File Name Pattern, Artifact Group,ArtifactVersion
Thanks
Ganga
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHubhttps://github.com/capitalone/Hygieia/issues/502

Artifact Directory - with respect to your workspace, where the artifacts (such as .jar) file are
Artifact Filename pattern - "*.jar" for example
Artifact Group - com.capitalone.dashboard for example
Artifact Version - "2.0.0" for example

Closing this as the question is answered.


Add additional OAuth error handling

Hello! We at Think Through Math have implemented this gem for our LTI integration as a Tool Provider. We are currently going through official certification with IMS Global for LTI 1.1. Several of the tests use invalid OAuth credentials. The extra rescue blocks here allow our platform to pass these tests so we wanted to add them to the base gem.
Also, since it looks like this gem is not under active development, we would be happy to take over maintenance of it since we will be actively using it going forward.
Cheers!

Thanks a lot!
Feel free to take over maintenance. This development started for adding LTI to several ruby tools that we no longer support in Universitat Oberta de Catalunya. I would like to follow the project, though.
Cheers!

Closing #1


Keep cursor in source code window when toggling

Hi @hkupty,
In my configuration I have set
visibility = require("iron.visibility").toggle

I observed that when calling :IronRepl the first time, a REPL buffer is opened in a new window, while the cursor was kept in the source code window. Now every call of :IronRepl that gets that window back in the view made the cursor switch focus from the source code window to the REPL.
Not sure if this is by design, but I'd rather like to have the cursor stay in the source code window, so I can send further commands to the REPL without having to switch from the _REPL to the source code window again.
This is what my little PR is doing. So if you like that change, I'd love to have it on the master branch.
Just did some manual testing but
visibility = require("iron.visibility").single
visibility = require("iron.visibility").focus

seem to work as they did before.
Have a good one 🙂

Hi, your fix seems correct. In facilitating, the desired behaviour is to avoid leaving the current window and moving the cursor, so then you can focus on writing the code, not on hopping on and off the source code.
Thanks a lot for the pr!


Remove obsolete apis

Fixes https://github.com/aspnet/Security/issues/1421
https://github.com/aspnet/Security/issues/1631
https://github.com/aspnet/Security/issues/1292
https://github.com/aspnet/Security/issues/1221

Post an announcement.
Not waiting for reviews?

Not if it has to go in today :)  I figure any review feedback I can address afterwards

This has potential to break things, so I'd rather see any issues early in the build before a weekend is the main reason

I'll post an announcement post mondo jump


repository 'https://gitorious.org/tropicssl/tropicssl.git/' not found

submodule update fails with the error message.
fatal: repository 'https://gitorious.org/tropicssl/tropicssl.git/' not found

@swsachini please find some other mirror of tropicssl and change it your submodules

github acquired gitorious so they will be unaccessable

This forks works perfect.
https://github.com/axic/tropicssl
Just change submodule files.


Slider widget, remove refinements when at min and max

I thought there was an issue already opened for this but I am not able to find it.
The slider should remove the refinements when the min and max of the slider are equals to min and max of the dataset.
For wordpress, in instant search page we could have article, page, products. Product is the only one with price so if you refine with the price slider you won't be able to have article and page

@maxiloc I believe this is now fixed, it was not released sorry. Now released with 0.5.0 careful there are breaking changes: https://github.com/algolia/instantsearch.js/blob/master/CHANGELOG.md#050-2015-10-22


[Add][v10.4.3-rx-1.0.4] Added project generation support for new devices

Added project generation support for new devices RX140-256KB, RX23E-A
Signed-off-by: xiao-xuan.ngew<EMAIL_ADDRESS>
Description

Test Steps

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Merge as the part of v10.4.3-rx-1.0.4.


extend FAQ (or somewhere else) to include decision of .js over .jsx

I have wondered why react boilerplate uses .js instead of the commonly found .jsx files. I have eventually found this:
https://github.com/facebook/create-react-app/issues/87#issuecomment-234627904
I guess that this is the reason for the decision. Since I find this quite interesting I suggest to put this somewhere in the documentation.

Closing because this isn't an issue with the boilerplate and no template was filled out.
The boilerplate actually supports using the .jsx extension if you wanted to use it! And yeah, Dan's comment is the reason why most react projects don't need / use jsx anymore


Inconsistent macro expansions

On 0.23.1, this works:
v1, v2, v3, v4 = 1, 2, 3, 4

{% for i in 1..4 %}
  puts v{{i}}
{% end %}

It gives the expected output:
1
2
3
4

but this does not:
a1, a2, a3, a4 = 1, 2, 3, 4

{% for i in 1..4 %}
  puts a{{i}}
{% end %}

Output:
Error in line 2: macro didn't expand to a valid program, it expanded to:

================================================================================
--------------------------------------------------------------------------------
   1.
   2.   puts a{{i}}
   3.
   4.   puts a{{i}}
   5.
   6.   puts a{{i}}
   7.
   8.   puts a{{i}}
--------------------------------------------------------------------------------
Syntax error in expanded macro: macro_94327761232656:2: unexpected token: {{

  puts a{{i}}
        ^

================================================================================

Strangely, using a, b, and c all fail, but I know for sure that when using e and v the macros expand correctly.

It is already fixed, by #4659 . Please close.


Fix extra origin in urls

verbose_name is unique in each project, because we only have one remote.
identifier includes origin for branches and also it can point to the
same commit in tags.
Fixes #5518

Looks like this is going to need a little more of work, the cases of stable and latest don't with verbose_name. We need to use https://github.com/rtfd/readthedocs.org/blob/e84bc96c2fe9f7b04832d9210f2a3026fb4a5b7b/readthedocs/builds/models.py#L150-L173

This is ready for review


Release for ES 7.5.0

Release notes: https://www.elastic.co/guide/en/elasticsearch/reference/7.5/release-notes-7.5.0.html

Do you want a separate issue for 7.5.1?
https://www.elastic.co/guide/en/elasticsearch/reference/7.5/release-notes-7.5.1.html
IDK but if it helps seems fine to skip old point releases and aim to support latest minor release, since Elastic pushes out versions so quickly.

Started looking at this; let's keep reference to this issue for now: https://github.com/elastic/elasticsearch/issues/49787


compiler: Generalize lowering of reductions

Need to try whether this can have any impact on: #1528

no difference


Update ApRESDefs.py

instead of a remote_load arg, always try remotely loading if local file load fails using a try-except statement. This can be propagated through all classes to potentially remove the need for the remote_load arg. I find it very easy to forget that default remote_load=False means that the code will fail with a type error.

This sounds like a good idea. Thanks for working on it.
It does seem like we may run into conflicts as I continue working on #20, as both involve changes to ApRESDefs.py, but I am not really sure how to avoid that. Any thoughts?

Maybe this line does what we want here: https://github.com/leap-stc/LEAP-bootcamps/blob/2a00e294a2cb91c21070bf18ee1051fd4b5d0d0f/Codes/Day2/utils.py#L15

I just merged a PR that should do what this was aiming to do: #53
Thanks again for the idea to do this!!
Let me know if this isnt working as you originally envisioned or you think it can be improved and we can work on improving it! :-)


Fixed typo

Codecov Report

Merging #1295 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1295   +/-   ##
======================================
  Coverage    3.76%   3.76%
======================================
  Files          35      35
  Lines       18261   18261
======================================
  Hits          687     687
  Misses      17448   17448
  Partials      126     126

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 6b00d2a...8f15a31. Read the comment docs.

Thanks @endrec !


install fails

$ pip install knesset-data
Downloading/unpacking knesset-data
  Using download cache from /home/...knesset-data-1.1.0.tar.gz
  Running setup.py (path:/tmp/pip-build-XvqxT3/knesset-data/setup.py) egg_info for package knesset-data
...
Downloading/unpacking pyslet (from knesset-data)
  Downloading pyslet-0.6.20160201.tar.gz (2.8MB): 2.8MB downloaded
Cleaning up...
Exception:
Traceback (most recent call last):
  File "/home/daonb/yeda/Open-Knesset/env/local/lib/python2.7/site-packages/pip/basecommand.py", line 122, in main
    status = self.run(options, args)
  File "/home/daonb/yeda/Open-Knesset/env/local/lib/python2.7/site-packages/pip/commands/install.py", line 290, in run
    requirement_set.prepare_files(finder, force_root_egg_info=self.bundle, bundle=self.bundle)
  File "/home/daonb/yeda/Open-Knesset/env/local/lib/python2.7/site-packages/pip/req.py", line 1198, in prepare_files
    do_download,
  File "/home/daonb/yeda/Open-Knesset/env/local/lib/python2.7/site-packages/pip/req.py", line 1376, in unpack_url
    self.session,
  File "/home/daonb/yeda/Open-Knesset/env/local/lib/python2.7/site-packages/pip/download.py", line 582, in unpack_http_url
    unpack_file(temp_location, location, content_type, link)
  File "/home/daonb/yeda/Open-Knesset/env/local/lib/python2.7/site-packages/pip/util.py", line 644, in unpack_file
    untar_file(filename, location)
  File "/home/daonb/yeda/Open-Knesset/env/local/lib/python2.7/site-packages/pip/util.py", line 575, in untar_file
    path = os.path.join(location, fn)
  File "/home/daonb/yeda/Open-Knesset/env/lib/python2.7/posixpath.py", line 80, in join
    path += '/' + b
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe8 in position 36: ordinal not in range(128)

Storing debug log for failure in /home/daonb/.pip/pip.log

@daonb - try to upgrade your pip:
$ pip install --upgrade pip

Thanks!


[DOCFIX] Add detailed steps in the upgrade document

What changes are proposed in this pull request?
Add detailed upgrade steps in the fourth section of the upgrade document
Please outline the changes and how this PR fixes the issue.
Why are the changes needed?
s the reader more clear specific operation steps
Please clarify why the changes are needed. For instance,

If you propose a new API, clarify the use case for a new API.
If you fix a bug, describe the bug.

Does this PR introduce any user facing changes? no
Please list the user-facing changes introduced by your change, including

change in user-facing APIs
addition or removal of property keys
webui

@alluxio-bot check this please

alluxio-bot, check this please

Thank you for your pull request. In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement (CLA). It's all electronic and will take just a few minutes. Please download CLA form here, sign, and e-mail back to<EMAIL_ADDRESS>
@LiuJiahao0001 pls also follow the instructions here and send your signed CLA to our email. Then your contribution can be merged :)

alluxio-bot, merge this please

alluxio-bot, merge this please


Update readme

I updated the readme to be more instructive on how to fork, clone, run, and edit the docs site.
I took some elements and guides from the Markdoc repo's readme itself.
PS: I'm excited about Markdoc because I'd just gone into a long foray with Markdown and MDX to re-create my company's documentation site Layer0 Docs and Markdoc was the Markdown tool I wished I had.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I noticed I had to sign a CLA. I might just add that to the README ASAP.


about the Aggregator, concatenate two layers'  information, please help me

the Preprocessing output: (B784, 2, 1024), it is calculated below
torch.stack(  [ (B784, 1024), (B784,1024) ], dim=1  )  ,   where  (B784, 1024) and  (B*784,1024) represent the output of MeanMapper for layer2 and layer3
and in the Aggregator,  after a reshape to (B784, 1, 2048), it uses a adaptive_average_pool1d(target_dim=1024)  to get an output of (B784, 1, 1024),   is that to calculate average  about layer2 and layer3 at same dimension, that cannot reach this performance.
a example [1,2,3,4,5], [1,1,1,1,1]   stack to [ [1,2,3,4,5],[1,1,1,1,1]  ]  reshape to [[1,2,3,4,5,1,1,1,1,1]]  and adaptive_average_pool1d, that is computed in line, (1+2)/2=1.5 (3+4)/2,   not to compute (1+1)/2, (2+1)/2, (3+1)/2,  the same place at each.   maybe not good example
mayebe i am wrong, please tell me, thank you for your help!

help me，please

代码好难懂啊，看晕了


feat: init commit svelte-kit

Trying svelte-kit x TS x Tailwind
I am working on my-app directory, so please do these step to run on your local:
cd my-app
yarn
yarn dev

Homepage Route

[x] Move svelte/stores
[x] Move utils
[x] Move statics dir
[x] Move dummy data
[x] Set static adapter for SSG
[x] Hero
[x] Card List
[ ] Pagination
[ ] Menu on Mobile
[ ] Theme switcher

Search Route

[ ] Task...

Tag Route

[ ] Task...

Pros

Easy migration
The DX is getting better than the old sapper
No need to manual generate static paths

Cons

No significant improvement -- maybe worst because we include Tailwind

@itsfaqih if you have a free time, maybe you wanna help me on this page styling?

@itsfaqih if you have a free time, maybe you wanna help me on this page styling?

Sure! Are you doing any styling right now?
Just to make sure there's no overlap

I am DONE, and maybe will not active for few days ahead.


common: allow promises to resolve with a single arg

/cc @schmidt-sebastian

Changes Unknown when pulling 3d0374982dd708b66f6d1937d6cc1a7327ba7aea on callmehiphop:dg--single-promise-resolve into ** on GoogleCloudPlatform:master**.


n_columns and shape return different values for grouped dataframes

n_columns shouldn't return a different value. That's a bug. shape might, but probably shouldn't. In fact... this is definitely a bug. But we should figure out a way to do something similar to dplyr::n(), which permits per-group row counts. This is a really common use case (so much you can use dplyr::count() which is super useful).

Closing in favor of #245.


Remove --network-plugin Kubelet flag for Kubernetes 1.24 clusters

What type of PR is this?
/kind bug
What this PR does / why we need it:

Remove --network-plugin Kubelet flag when migrating from Docker to containerd

New containerd clusters don't have this flag out of the box

Remove --network-plugin Kubelet flag when upgrading from Kubernetes 1.23.x to 1.24.x

This is required because in earlier KubeOne versions we didn't remove the --network-plugin flag when migrating to containerd. Therefore, the flag is still present in the Kubelet config. Upgrading such a cluster to 1.24 causes Kubelet to crash.

This is required since --network-plugin and some other flags got removed in Kubernetes 1.24 (https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.24.md#feature).
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
xref #2010
Special notes for your reviewer:
This is still not tested.
Does this PR introduce a user-facing change?:
Remove the `--network-plugin` Kubelet flag when migrating from Docker to containerd
Remove the `--network-plugin` Kubelet flag when upgrading from Kubernetes 1.23.x to 1.24.x

/assign @kron4eg

/approve
/lgtm


cleanup the list of satdress servers and add sendsats.lol

I have a few questions regarding this PR:

Why were these servers removed? These still provide interesting domain options.
How were the servers to remove choosen? sats.pm for example was not removed
Why was sats4.me removed? It's not a satdress server, but a custom reverse proxy made for LnMe.

I completely misunderstood the list, please ignore this comment. It seems he list is only for the satdress server, sorry :sweat:


add gh workflow for testing coverage

Coverage remained the same at 96.233% when pulling f7af588b813c78a87433c23d1465150c0f1c9a9d on hendrikmuhs:gh-coveralls into c113d13bee8d6e6f7c7ade02a5040b0376af0033 on KeyviDev:master.


[HACKTOBERFEST 2024] - HTML Tables and Forms Quiz

[!IMPORTANT]
This issue is part of our 2024 Hacktoberfest initiative. If you are interested in working on this issue, refer to our top-level tasks to see if this issue is claimed, or to claim it if it is not.

This issue is to create the HTML Tables and Forms quiz.
The file you should be working on is found in /curriculum/challenges/english/25-front-end-development/quiz-html-tables-and-forms.
All quiz questions need to relate to one of these topics:

Form element
Label element
Action attribute
Input type=”text”
For attribute
Input type=”email”
Placeholder attribute
Button element
Button type=“button”
Button type="submit"
Button type="reset"
<input type="button" value="" />
Required attribute
Size attribute
Minlength attribute
Maxlength attribute
Disabled attribute
Value attribute
Readonly attribute
Table element
Thead element
tr element
tbody element
td element
th element
tfoot element
What is an HTML validator?

[!CAUTION]
Questions which do not fall under these topics will be rejected.

You are welcome to include code blocks in your questions, but please keep them short (four lines or less).
Please do NOT open a pull request until you have completed all questions. Your quiz should have exactly 20 questions.

Thank you for your interest in contributing. This issue will be opened for contribution when Hacktoberfest starts on October 1st.

[!CAUTION]
Pull requests which are created before October 1st will not be eligible for Hacktoberfest. We appreciate enthusiasm, but encourage you to hold off on contributing to this issue until the event has begun.

If you wish to be notified when this issue is opened for contribution, subscribe to notifications:

This issue is now open
If you would like to work on this issue, please leave a comment so we can assign it to you.
Be sure to read our contributing guidelines.

I would like to work on the Git quiz issue for Hacktoberfest 2024. Could you please assign this issue to me?

Hi @Dhairya-A-Mehra !
It looks like Naomi is already assigned to this one.
But there are still other open ones

Can you assign this issue to me?

@doosabhulaxmi this issue has already been assigned

This issue has been assigned a contributor. To prevent extraneous notifications, the issue is now locked. If you have any questions, please reach out to me in our Discord community.

I would like to work on this issue.

Could I do this one?

Haha, everyone is so quick! I got distracted checking the quiz material. Good luck to whoever is assigned this issue.

@Adran-Nightshade all yours!

Thank you!

@ilenia-magoni @naomi-lgbt
Is this issue open? Would love to work on this?

Let me assign you, note, usually assigning is not something we do, but it was the decided workflow for these quizzes


Add simon game

Is your feature request related to a problem? Please describe.
No its not a problem. I want to add the simon game.
please assign this issue to me .

Similar project already exist


Font References: Use Google CDN

Rather than hosting the Roboto font locally, it's much easier to reference Google's CDN:
https://www.google.com/fonts/specimen/Roboto

Unfortunately Google falls out with a number of countries that enforce internet censorship - China and the GFW for example - these countries have blocked Google APIs.
 Painful. The Bullhorn Dev teams (very quickly I might say) brought the Google CDN hosted assets - like fonts - back into the Bullhorn application, restoring reasonable page load times.
Cloudflare's CDN works well now, but I fear it's only a matter of time before China and the likes block it to get at services behind it.
Just putting that out there :)
Cheers,
Ben

@BenACorke makes a fair point. When we internalized static resources for B4E and Accel, we briefly considered Cloudfare and CDNJS as alternative CDN sources. In the end we chose to inline because those apps are wired into our Akamai edge caching service, effectively acting as a first-party CDN.
It appears we have a couple of alternative options here:

Do nothing: Pro is that code/product is portable and guaranteed not to break relative to Ben's concern. Con is that deploy package is bloated (the bulk of the compiled package is font resource files) and potentially affects time-to-render performance if the CP host (controlled by CP clients, not Bullhorn) has a slow uplink. Con is mitigated by browser-cache behavior of said static resources.

Externalize to Cloudflare or CDNJS: Pro is a smaller CP package, implying easier upload/upgrade by CP consumers and potentially increased time-to-render perf. Con is CP is now subject to transient CDN unavailability and possible (future) region lockout, requiring an upgrade of all locked-out CP consumers.

Externalize to BullhornCDN: We've recently bootstrapped the BullhornCDN, which is an endpoint covering over our Akamai edge cache service. We need to validate with BH tech ops that the CDN is or will soon be Akamaized. We could drop static resources on the CDN for delivery to CP consumers. Pro is potentially increased time-to-render performance without fear of future lockout; con is transient availability of BullhornCDN is not yet proven.

Finally, we could fork the code and have internalized vs. externalized resources, but IMO product fracture and deploy complexity for CP consumers is not worth the benefit of partial userbase performance increases.
@bryanjroy , @jgodi , @bvkimball , @matthewafischer ... thoughts?

@TheReal This can be modified by the developer to host locally or they can choose to use a different font supported by another CDN.  I would assume that the users that run into this deployment issue, would be the ones big enough to have the resources to implement a work-around.  I am not opposed to putting third-party content into the BullhornCDN but there would be a level of support assumed with that decision, that might not be true.  If we switch to a different Font-Face in the future do we need to continue to host googles font in our CDN forever?
I think a wiki article on how to change your resources to internal is the first step.
Making certain resources available through the BullhornCDN the second step.

I agree with @bvkimball. I don't like the idea of having two projects for internal/external, so we should document how to change and possibly add the font to the BullhornCDN.

Google CDN with local failover implemented with v1.6.0


rebar3_lfe plugin failed to compile on Erlang/OTP 26 due to a a failed call to user:start() in lfe_init

In order to get the plugin to compile correctly I had to comment out the user:start() call below:
start() ->
    case collect_args(init:get_plain_arguments()) of
        {[],[]} ->                              %Run a shell
            user_drv:start(['tty_sl -c -e',{lfe_shell,start,[]}]);
        {Es,Script} ->
            % user:start(),
            %% io:format("es: ~p\n", [{Es,Script}]),
            run_evals_script(Es, Script)
    end. (edited)

Related tickets:

#79
https://github.com/lfe/lfe/issues/474

This is an upstream issue and has been addressed in LFE 2.1.2 (see https://github.com/lfe/lfe/releases/tag/v2.1.2).
I've got a couple of rebar3_lfe releases planned, the first of which will include an update to the latest LFE.

With LFE 2.1.2 and rebar3 3.22.1, this particular issue is solved.


Refactor searching for end of inline (EI) JPEG image streams

This patch changes searching for EI image streams to rely on the EOI (end-of-image) marker for DCTDecode filters (i.e. JPEG images).
Note: This code was initially included in PR #5383, but removed from it based on this comment from @yurydelendik: https://github.com/mozilla/pdf.js/pull/5383#discussion_r21834325.
It should be quite easy to, if necessary, ignore that particular marker sequence, but unfortunately I've been unable to find any information about it.

Okay, I found it at http://www.w3.org/Graphics/JPEG/itu-t81.pdf#page=95&zoom=auto,-89,622 see "F.1.2.3 Byte stuffing". Looks like JPEG encoding takes care of it by itself. Let me think if EOI can potentially appear in other tags, e.g. looking at "B.2.2 Frame header syntax" there is 'Lf' field is present, so we can skip the bytes and avoid false-EOI inside SOF body.

Looks like "Table B.1 – Marker code assignments" has everything we need. Can we use length field for all tags that are not marked with (*) and not reserved to skip bytes?

Since JPEG uses two byte markers, it seemed nice/easy to just use the getUint16 methods of {Stream, DecodeStream, ChunkedStream}. Doing so, I realised that those methods will return garbage values if getByte() === -1 (i.e. when we've reached the end of the stream). Hence the first commit, which I hope is a good way to address that.
After carefully reading through the relevant parts of the marker spec, I hope I've managed to produce a good implement of the searching for EOI. The code will now read the length of the various marker segments, and jump straight to the next marker.
This seems to work very well, and in all cases I've checked this reduces the number of iterations necessary to find the EOI. It also appears that the number of iterations needed are inversely proportional to the length of the image stream, which should be great news for files with many small inline images.
/botio test

Yep, that's what I was looking for. However reading by 16-bit is throwing me off a bit. Can you modify the outer loop to read by bytes and to whitelist tags with length instead of blacklisting know to be without length? Something like this will explicitly specify which marker really have length field and reduce amount of skip(-1) calls:
while ((b = stream.readByte()) !== -1) {
  if (b !== 0xFF) continue; // not a marker
  switch (stream.readByte()) {
    case 0xFF: // padding
      steam.skip(-1); // needing to rewind
      break;
    case 0xC0: // SOF0
    case 0xC1: // SOF1
      var markerLength = stream.readUint16();
      if (markerLength > 2) {
        stream.skip(makerLength - 2);
      }
      break;
  }
}

Thanks for the feedback! How does it look now?
I completely agree that white-listing is better, even though the list became quite long :-)
I also added a special-case for the 0xFF00 "marker", since in many files it's by far the single most common 0xFF.. sequence, and this lets us bypass a bunch of pointless comparisons further down.

/botio test

Looks good, thank you!


Fix parse error on init

Fix parse error on init introduced in 933c61b, by adding missing parentheses.

Oh, I just saw b41f485.
Closing this :)


Is "extra_requirements" option of yml file - ignored right now?

Project URL: https://github.com/diofant/diofant
Build URL: https://readthedocs.org/projects/diofant/builds/4462416/

I expect as it promised in the docs - line pip install -e .[docs] in the build logs, that will install numpydoc, but I see only default pip install -e ..  Also, I'm little surprised why this doesn't break build.
This option does work locally.  See more in https://github.com/rtfd/readthedocs.org/pull/2368#issuecomment-252219610.

It seems, now it works:
http://readthedocs.org/projects/diofant/builds/4723979/


TreeItem does not allow conditional child

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
There is an error
Expected Behavior 🤔
No error
Steps to Reproduce 🕹
https://codesandbox.io/s/wonderful-pond-89ohe
Steps:

Load the sandbox - see that there is nothing displayed.  There should be an error but strangely it is not showing.  If you really want to see it click the refresh

Context 🔦
I should be able to conditionally render
<TreeItem nodeId="parent" label="parent">
          {props.renderChild && <TreeItem nodeId="parent" label="child" />}
</TreeItem>

I thought #19849 fixed this but it seems to only fix TreeView not TreeItem.

https://github.com/mui-org/material-ui/pull/20156
There are more serious bugs than this


Bump jinja2 from 3.1.2 to 3.1.3 (#6967)

Pull request opened by the merge tool on behalf of #6967

Merged in bc0452474d1cd2ac88cb0d8cdd8961a301c56c5c