Document the use cases for SerialPort::configure() versus SerialPort::reconfigure()

This also comes out of #28 since it split into a few different issues.
I'd like to suggest that the documentation make more clear the different between configure() and reconfigure() from the user's perspective. As a user of the serial library and looking at these functions and the documentation for them, I would think I'd want to use configure() for the initial configuration and reconfigure() for every configuration afterwards. But it's not clear that a) this is the recommended method and b) that this is the required way to use these. Especially since the example code only ever uses reconfigure().
Maybe it makes more sense to remove configure() and create a second function open_with_config() that takes the port name and the desired initial configuration. And then reconfigure() is always used after initialization. I think that'd make the interface smaller and more clear. This would also compliment #31 by making opening new ports reusing existing port settings require 1 less function call.

@Susurrus The documentation describes exactly what the methods do. The use cases are up to the user. Just use whichever is more convenient for your use case. examples/probe_pins.rs and examples/read_write.rs both use configure(), but it wouldn't hurt to add an example to the API documentation as well.

It described what they do but not why you'd use them. If they're equivalent, then there should be an additional comment like "This is identical to XX() and is provided here as an additional interface to configuring the serial port".
I'm looking at this as a new user, so it'll be very different than your perspective. If you're new to this library and you want to open and configure a serial port and you see both a configure() and a reconfigure() method, you'll likely draw similar conclusions to what I did. This is why it's so important to describe when these functions should be used and why, even if it's just to say they're equivalent functions and to use whichever is more convenient.

I'd also question why reconfigure() exists if all the examples use configure(). What was your reasoning for having two different configuration functions? Considering the extra maintenance and support burden of additional functions, and the confusion having two nearly-identical functions can cause, it seems prudent to reduce configuration to the single right way. It's part of the Python mentality that I think has made that language so easy to pick up and implement good code in and worth adopting elsewhere.


Use of Redis#client method which was removed in redis v4.0

We're using message_bus with Redis as the storage backend and have recently updated our redis gem from v3.3.3 to v4.1.0
In v4.0 of the redis gem the Redis#client method was replaced with Redis#_client which meant it was also necessary for us to update our message_bus gem from v2.0.2 to v2.1.6 as in the older version the after_fork method used pub_redis.client.reconnect
We're all good now but looking through the gem code I did notice that the is_readonly? method is still using pub_redis.client
def is_readonly?
  key = "__mb_is_readonly"

  begin
    # in case we are not connected to the correct server
    # which can happen when sharing ips
    pub_redis.client.reconnect
    pub_redis.client.call([:set, key, '1'])
    false
  rescue Redis::CommandError => e
    return true if e.message =~ /^READONLY/
  end
end

This means that with redis v4.0 onwards, when the method calls pub_redis.client.reconnect it will result in a Redis::CommandError exception with the message ERR Unknown subcommand or wrong number of arguments for ''. Try CLIENT HELP and the method will always return nil.

Nice catch! Can you send a pr through to fix?
On Wed, 5 Jun 2019 at 12:28 am, Chris Hilton<EMAIL_ADDRESS>wrote:

We're using message_bus with Redis as the storage backend and have
recently updated our redis gem from v3.3.3 to v4.1.0
In v4.0 of the redis gem the Redis#client method was replaced with
Redis#_client which meant it was also necessary for us to update our
message_bus gem from v2.0.2 to v2.1.6 as in the older version the
after_fork method used pub_redis.client.reconnect
We're all good now but looking through the gem code I did notice that the
is_readonly? method is still using pub_redis.client
def is_readonly?
key = "__mb_is_readonly"
begin
# in case we are not connected to the correct server
# which can happen when sharing ips
pub_redis.client.reconnect
pub_redis.client.call([:set, key, '1'])
false
rescue Redis::CommandError => e
return true if e.message =~ /^READONLY/
end
end
This means that with redis v4.0 onwards, when the method calls
pub_redis.client.reconnect it will result in a Redis::CommandError
exception with the message ERR Unknown subcommand or wrong number of
arguments for ''. Try CLIENT HELP and the method will always return nil.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SamSaffron/message_bus/issues/208?email_source=notifications&email_token=AAABIXPNFEQK65CE7FCBHETPYZ3YDA5CNFSM4HS46C4KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GXRN5UQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAABIXIKJ4HQJSB2PNNR7KLPYZ3YDANCNFSM4HS46C4A
.


fix: state continuously updated when holding ctrl/cmd

fix https://github.com/excalidraw/excalidraw/issues/3282

🚀🚀🚀


Added property name setters in Color init

It's always mildly annoyed me that the Color instruction didn't support setting with kwargs in python, and there's even an old issue that references it, so this seems like a reasonable fix.

:+1:  super duper


no support for altering the badge number on android

Not sure why we don't support being able to alter the badge number on Android, maybe there's a reason such as there's no official native support.
Seems like other libraries use me.leolin:ShortcutBadger
notifee  only offers support to adjust this via a notification.

@helenaford @mikehardy Is this still correct? I've found this issue and then official Android documentation Set custom notification count.
I might misunderstand something, but I think we are talking about the same thing.
Notification notification = new NotificationCompat.Builder(MainActivity.this, CHANNEL_ID)
        .setContentTitle("New Messages")
        .setContentText("You've received 3 new messages.")
        .setSmallIcon(R.drawable.ic_notify_status)
        .setNumber(messageCount) // <---- THIS
        .build();

Fascinating! Literally had no idea that existed.
https://developer.android.com/reference/androidx/core/app/NotificationCompat.Builder#setNumber(int)
Such vague language, emphasis mine:

On the latest platforms, this may be displayed as a badge count for Launchers that support badging. Prior to Build.VERSION_CODES.O it could be shown in the header. And prior to Build.VERSION_CODES.N this was shown in the notification on the right side.

So, it may be shown in some spot if supported on some new-ish versions, or it could be shown in another spot ;-). Really clears things up.
Does appear to be Android API21 and up based on use word "number" in this file https://android.googlesource.com/platform/frameworks/support/+/refs/heads/androidx-main/core/core/src/main/java/androidx/core/app/NotificationCompat.java
Doesn't look like there are any guarantees though, and it's not a "badge" per se. I could see merging support for this if you made a PR, that would be great, but I would also be really clear in the documentation that it is not shown right on the app icon as people commonly expect, and point to the docs.

I think it's a launcher feature to show budge number or not. I have one that supports it, it works without any changes on my side. This method probably will not show budge on launcher that doesn't support it, it just allows to change counting (don't increment on some notifications) on launcher that can already show these budge numbers.
.
Unfortunately I don't know Java / Android.

https://user-images.githubusercontent.com/842306/144146742-d799763a-b7f7-41b3-bd1c-12808fe94eeb.mp4

Very interesting - I know lots of people want that. I will revise my statement then and say:

I would also be really clear in the documentation that it is not guaranteed to be shown right on the app icon as people commonly expect, and point to the docs.

Believe it or not, android/java is actually not that hard :-) the CONTRIBUTING doc shows how to get the environment all set up and then it's just a little plumbing if an integer down to the right ReactMethod and into where ever NotificationCompat.Builder is called


Add systemd manager configuration command

Signed-off-by: Domenico Chirabino<EMAIL_ADDRESS>Add reload systemd manager configuration command
/kind content
/area documentation
What this PR does / why we need it:
After installation we need to reload the systemd manager so that the new unit is loaded.
Which issue(s) this PR fixes:
#467 (as asked in this comment
/assign @leogr

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: chirabino
To complete the pull request process, please assign lucperkins after the PR has been reviewed.
You can assign the PR to them by writing /assign @lucperkins in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

/remove-lifecycle rotten

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closed this PR.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Update parquet.py

_should_silently_exclude should silently exclude, not print stuff

This seems reasonable, since we're ignoring _SUCCESS and .crc files. _SUCCESS files are empty and .crc files are a data integrity implementation detail that a user would probably almost never want to read in directly.

Looks like a spurious travis failure

Opened a JIRA for this: https://issues.apache.org/jira/browse/ARROW-2137

@kisankansagra could you let us know your JIRA id on issues.apache.org (or create an account if necessary) so we can assign this issue to you?


NIFI-6809 Make use of ASF shortened URL to point to community SLack i…

…nvite link.
Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on both JDK 8 and JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

@apiri looks like @markap14 already merged this.

@MikeThomsen Not seeing it in the commit history.  Am I overlooking something?

NM. It was an early morning screw up on my part.


Asio

Close #22

Asio (at c85476b751ea46fc2c3aa2da0e8d089224be9977) seems to be >50% slower when compared to libuv (at 9a519afa7bdc6e9bb944924094fcad84fcf83a10).
Benchmark results
1a.  I'm not very familiar with asio so any help is very much appreciated 🙏.

@uatuko how can I run this code?

@uatuko how can I run this code?

The easiest if to build and run the "helloworld example" (which is the grpcxx-examples_helloworld CMake target).
If you run make in project root it should create the binary in .build/bin/grpcxx-examples_helloworld (but this will be a debug build).

@uatuko does you project work on windows?
Anyway, I am looking at the code.

io_context::run should generally be called simultaneously from a pool of threads, not by an arbitrary instance. the way you use it means there is only one thread calling io_context. Not sure this is what you intend.

    class IOContextExecutor
    {
        std::shared_ptr<boost::asio::io_context>                                 _ioContext;
        boost::asio::executor_work_guard<boost::asio::io_context::executor_type> _ioContextGuard;
        boost::asio::thread_pool                                                 _pool;

    public:
        IOContextExecutor(std::size_t threadCount = 1);
        ~IOContextExecutor();
        std::shared_ptr<boost::asio::io_context> GetIOContext() const
        {
            return _ioContext;
        }
    };

        IOContextExecutor::IOContextExecutor(std::size_t threadCount) :
        _ioContext(std::make_shared<boost::asio::io_context>()),
        _ioContextGuard(boost::asio::make_work_guard(*_ioContext)),
        _pool(threadCount)
    {
        for (auto i = 0u; i < threadCount; ++i)
        {
            boost::asio::post(_pool, [i, this] {
                try
                {
                    auto threadName = std::string("zzz io_context thread ") + std::to_string(i);

                    os::set_current_thread_name(threadName.c_str());
                    _ioContext->run();
                }
                catch (const std::exception& err)
                {
                    ULTRA_LOG_ERROR("io_context threads crashed {}", err.what());
                }
                });
        }
    }

    IOContextExecutor::~IOContextExecutor()
    {
        _ioContextGuard.reset();
        _pool.join();
    }

asio coroutines are nice, but it is easier to profile / trace the standard callback API. I would use callbacks and profile calls which I think suspect might take more time than expected.

Thanks for looking into this @eladm-ultrawis!
The Asio version should be working on Windows, although I don't have access to a PC so haven't verified it. The libuv version doesn't work on Windows due to *nix specific socket duplication code.
I assumed the coroutines approach would use threads internally since the io_context constructor accepts a concurrency hint but documentation isn't great so I could be very wrong here. I'll play around with the callback API a bit as well to see if I could see an improvement.

@uatuko yeah I thought this is what you assumed. the concurrency hint is simply somethin asio passes to the OS API (see this).
Note that io_context isn't creating any threads, it is basically a queue, all the completions (callbacks) are called on the threads that call io_context::run. so the pool threads process all the callbacks.

While I was working on integrating Asio, I had a 💡 moment to take a different approach with libuv. It's still very much a WIP (see #24) but ultimately it should allow integrating Asio and other similar libraries (or just use coroutines) at the application layer without blocking libuv I/O.
Although this wouldn't allow sharing io_context for libuv I/O it can limit it to one thread.
Initial benchmarks seem to suggest this new approach would be much more performant as well. For context, Asio (c85476b751ea46fc2c3aa2da0e8d089224be9977) is 14k req/s while a single-threaded libuv (86e43df5e5de684c8da9f6f83086c3db5d263028) is 40k req/s. Better yet, it doesn't need the *nix only socket duplication and should be more portable as well!

I'm abandoning this PR 😞.
Single-threaded performance using C++20 coroutines and libuv is 3x higher than asio (see #25 for more details). Although asio callback API might give better performance I feel it's taking a step backwards since this project started as libuv callbacks and then moved onto coroutines. Code written using callback APIs aren't the easiest to read either.
I don't have enough knowledge of asio to squeeze more throughput, specially not as much as 3x more than what I managed so far.
If someone could make the effort to get something working with asio that would give a better throughput I'm happy to come back to looking into asio 🙏.

#26 enables switching between Asio and libuv using GRPCXX_USE_ASIO CMake option (or pre-processor macro).


add godoc badge

Codecov Report

Merging #6 (32ae698) into main (b100865) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   52.63%   52.63%
=======================================
  Files           1        1
  Lines          38       38
=======================================
  Hits           20       20
  Misses         12       12
  Partials        6        6

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 b100865...32ae698. Read the comment docs.


Enable Student Profile Tab for Institutions

As an institution users I want to view Student Profile information
Additional Context
https://www.figma.com/file/9Om2tzLAVEoMQrjmrH4fhu/MOD-SIMS?node-id=11337-140498&t=K5C151j6g3EQtqRi-0
Acceptance Criteria

[ ] Convert Form.io to Vuetify.
[ ] Enable authorization paths for Student Profile Tab for institution users.
[ ] Build E2E tests.

The AC Enable authorization paths for Student Profile Tab for institution users is removed as it is scoped as centralized solution to be implemented in #1830

As discussed with team and @JasonCTang , the restriction badge that indicates if a student has any active restrictions is not included in the student details page on institution side as it may not be accurate/misleading to show a no restriction badge to a student who may be having a verification restriction which institution does not have access to.


Refactor import fix

bum! checked out build output. looks like it has to be ../index. tried $lib first which the ts output was good but building docs failed because of it.

Fixed in 36f8eef217c516bf68fc39e91644e8e69b94f685


Warning about app id but Intercom works as intended

Describe the bug
The warning below shows for us in production despite Intercom working as intended (and app id is supplied in the provder).

We set up our provider like this
    <IntercomProvider appId={import.meta.env.VITE_INTERCOM_APP_ID}>
    ...
    </IntercomProvider>

We initialize Intercom like this
const useDDIntercom = () => {
  const { boot, shutdown } = useIntercom();
  const { profileData, hasProfileData } = getProfile()

  useEffect(() => {
    if (!hasProfileData || !import.meta.env.PROD) {
      return;
    }

    const initializeIntercom = (userHash: string | undefined) => {
      try {
        boot({ email: profileData.email, userHash });
      } catch (err) {
        Logger.captureException(err, {
          message: "Intercom has encountered an error to initialize."
        });
      }
    };

    getIntercomHash()
      .then(({ data }) => initializeIntercom(data))
      .catch(error =>
        Logger.captureException(error, {
          message: "Error setting up Intercom. HMAC wasn't generated."
        })
      );

    return () => {
      shutdown();
    };
  }, [hasProfileData, profileData.email]);
};

export default useDDIntercom;

To Reproduce
Codesandbox link
Expected behavior
Not see the warning.
Screenshots
See above.
Desktop (please complete the following information):

OS: Mac
Browser: Firefox, Chrome
Version: Latest

Smartphone (please complete the following information):
Additional context

This is weird indeed. Seems like a warning from Intercom itself, have you tried asking on the Intercom Forum?
I won't consider this as an issue of this library. Feel free to re-open if you can provide a reproduction, so I can further investigate it.


Style resume

A few changes here

Added in logic to the notebook for creating cohort.json to account for resumes. This has appropriate logic to handle file path names (lowercase)
Added in a disable class that greys out an icon / button
Added in JS logic to implement the resume for the modal and main card.. This design groups the "More" option and the resume into a single button to make more room on the card
Added in a fancy ribbon to specify if a student is trying to get hired!
Added in some JS sort and shuffle logic to make sure the students who are trying to get hired appear first in the website

To test:
python3 -m http.server

Make sure that these students have ribbons (hire me) and show up first:

Landry, matt Barnes, Steven Decker, D. Mellow, Christina, Brant Ivey, Tracy Olson, Will McWilliams, Alan Brunlinger, Justinianlawcode, Brandes, Adam Tsuchiyama

Make sure that the carousel doesn't move the page as you scroll
Make sure that the buttons (resume and more) are combined into a single row
Make sure that some social icons are "disabled" - as well as resumes!

Thanks!

ok ship it
On Tue, Jun 23, 2020 at 9:41 PM Taylor Perkins<EMAIL_ADDRESS>wrote:

A few changes here

Added in logic to the notebook for creating cohort.json to account
for resumes. This has appropriate logic to handle file path names
(lowercase)
Added in a disable class that greys out an icon / button
Added in JS logic to implement the resume for the modal and main
card.. This design groups the "More" option and the resume into a single
button to make more room on the card
Added in a fancy ribbon to specify if a student is trying to get
hired!
Added in some JS sort and shuffle logic to make sure the students
who are trying to get hired appear first in the website

To test:
python3 -m http.server
Make sure that these students have ribbons (hire me):
@Landry https://github.com/Landry, @mwbarnesy
https://github.com/mwbarnesy, @steven https://github.com/steven
Decker, @d https://github.com/d. Mellow, @christina
https://github.com/christina, @Brant https://github.com/Brant Ivey,
@tracy https://github.com/tracy Olson, @will https://github.com/will
McWilliams, @alan https://github.com/alan Brunlinger,
@Justinianlawcode, @brandes https://github.com/brandes, @adam
https://github.com/adam Tsuchiyama
Make sure that the carousel doesn't move the page as you scroll
Make sure that the buttons (resume and more) are combined into a single row
Make sure that some social icons are "disabled" - as well as resumes!
Thanks!
You can view, comment on, or merge this pull request online at:
https://github.com/NSS-Data-Analytics-Cohort-2/NSS-Data-Analytics-Cohort-2.github.io/pull/15
Commit Summary

Moved the resume to the second screen, and also highlighted/greyed
out icons based on availability
Updated json, adding in resume logic.
Added in logic to dynamically include the resume
Updated resumes, added in logic for resume button / disabling. Added
in hotfix for carousel

File Changes

M .gitignore
https://github.com/NSS-Data-Analytics-Cohort-2/NSS-Data-Analytics-Cohort-2.github.io/pull/15/files#diff-a084b794bc0759e7a6b77810e01874f2
(2)
M data/Website - Website.csv
https://github.com/NSS-Data-Analytics-Cohort-2/NSS-Data-Analytics-Cohort-2.github.io/pull/15/files#diff-7497247866f485641b93dddf66934145
(61)
M data/cohort.json
https://github.com/NSS-Data-Analytics-Cohort-2/NSS-Data-Analytics-Cohort-2.github.io/pull/15/files#diff-91027afa9b57742097aa78a8ff81de93
(2)
M index.html
https://github.com/NSS-Data-Analytics-Cohort-2/NSS-Data-Analytics-Cohort-2.github.io/pull/15/files#diff-eacf331f0ffc35d4b482f1d15a887d3b
(42)
M scripts/main.js
https://github.com/NSS-Data-Analytics-Cohort-2/NSS-Data-Analytics-Cohort-2.github.io/pull/15/files#diff-e8a233058211c77f51de535cd8ebb8d6
(60)
M styles/main.css
https://github.com/NSS-Data-Analytics-Cohort-2/NSS-Data-Analytics-Cohort-2.github.io/pull/15/files#diff-8c4c6de8f414cb0520c4de4fd2070829
(46)
M utilities/.ipynb_checkpoints/Resizing Images-checkpoint.ipynb
https://github.com/NSS-Data-Analytics-Cohort-2/NSS-Data-Analytics-Cohort-2.github.io/pull/15/files#diff-09f87298bb3e094d0dfe0f8227317d91
(6)
M utilities/Make JSONs.ipynb
https://github.com/NSS-Data-Analytics-Cohort-2/NSS-Data-Analytics-Cohort-2.github.io/pull/15/files#diff-f044f3e86400fc229b9b05f2d13f4838
(405)
M utilities/Resizing Images.ipynb
https://github.com/NSS-Data-Analytics-Cohort-2/NSS-Data-Analytics-Cohort-2.github.io/pull/15/files#diff-95543fe51ab0981f927a187e8d394327
(6)

Patch Links:

https://github.com/NSS-Data-Analytics-Cohort-2/NSS-Data-Analytics-Cohort-2.github.io/pull/15.patch
https://github.com/NSS-Data-Analytics-Cohort-2/NSS-Data-Analytics-Cohort-2.github.io/pull/15.diff
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NSS-Data-Analytics-Cohort-2/NSS-Data-Analytics-Cohort-2.github.io/pull/15,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAAPNISEZZHGDHLDPB727DRYF7XJANCNFSM4OGJEZQQ
.


Support for -ntp/--no-transfer-progress

Is there a possibility to add support for --no-transfer-progress/-ntp in MavenOptions?
Kindly regards.

Sure. If you provide a PR I'll gladly merge it.. ;-)

Closed by #21
Thanks, @natami.

Version 4.6.0 (just tagged and pushed to NPM) contains the new feature.


D3D11 Flush heuristic rework

Aims to fix inconsistent resource readback latencies, and in general just brings flushing behaviour more in line with what a D3D11 driver would do.
The old code was really designed for a world in which interrupting a render pass was a really bad idea due to image layout transitions, but we're no longer doing those when a render pass instance is merely suspended anyway, so flushing in the middle of a draw sequence isn't a problem anymore.
This is kind of scary anyway though so any testing is highly appreciated.

This is kind of scary anyway though so any testing is highly appreciated.

What's Up Mr. Philip.
I tested it on some games, in:
AMD RADV 22.3.3
Proton 7.0-5
Wine-Staging 8.0rc3
GTA Vice City The Definitive Edition ( Rockstar Launcher, Wine-Staging 8.0rc3, futex_waitv )
Far Cry 4 ( Ubisoft Connect, Wine-Staging 8.0rc3, futex_waitv )
Dark Souls Remastered ( Steam, Proton 7.0-5, futex_waitv )
The Evil Within 2 ( Steam, Proton 7.0-5, futex_waitv )
I tested them all for about 13~15 minutes, and didn't notice any anomalies.


dhcp-proxy: correctly exit > 0 on errors

If there is an error during setup we should return the error so that netavark can exit > 0 and systemd will report this as failure.

LGTM

/lgtm


WIP: Date helper refactor

Current coverage is 92.59%

Merging #346 into master will increase coverage by +0.88% as of 5814c06

@@            master    #346   diff @@
======================================
  Files            8       9     +1
  Stmts          181     189     +8
  Branches        39      36     -3
  Methods         90      88     -2
======================================
+ Hit            166     175     +9
- Partial          3       4     +1
+ Missed          12      10     -2

Review entire Coverage Diff as of 5814c06

Powered by Codecov. Updated on successful CI builds.


Keycloack session auth weird behavior

When the session expires, or the login windows expires it causes some weird behavior for the blazor app.
If time, try to find the root cause:
Cookie expiration, logout/login redirect... other stuff.
The auth is working well but is not production ready for sure.

Better now in main branch. Need to implement session management when WASM.


FR-FR translation

Also make sure to add to the toc.yml and have a toc for inside the fr-fr folder as well. Refer to other languages folders for this.

/toc.yml added.
/node/introduction.md added.
/setup.md added.

@Aedelon could you fix the conflicts so we can merge ? Also, contact afong on slack

The confilts are fixed. So now we can merge this pull request. I contacted afong and @edouardhienrichs to continue the translation of NEO docs.

@Aedelon I'm answering you right now. Thank you for reaching out !

I started to review it, but there are really too many corrections to do. I specify that it's not a criticism, this is expected regarding the size of the PR. After all, I really think splitting this PR would help a lot @Aedelon, do you agree ?

@Aedelon @edouardhienrichs Any update on this? Ready to be merged?

Hey guys. This seems abandoned. I speak French natively. I could look at creating new separate PRs starting from scratch (with the latest English documents as I assume it has evolved since then) and use this PR and adjoining comments as a general direction.

Yeah, it seems abandoned. But it isn't. I just need to audit the written texts because we think it could be better. Actually, I currently don't have a lot of time because of the CoZ competition + the Neo contest & my current work so it will wait the 03/10/2018 until I fully come back to the translation. But even with that i'm reviewing the text of Ningjun about "Document for Exchange Developers". That part is already translated.

@Aedelon good to hear ! good luck with the competition.
I'll start doing smaller PRs and if all goes well, we can finish the rest together when you're back on this.


Implement JSON reading and writing methods

Implement these extensions before .NET 5: https://github.com/dotnet/aspnetcore/issues/17160#issue-523751464

Did some of those overloads here https://github.com/davidfowl/FeatherHttp/commit/2bdd49c79143d667f9db8c8c1ac55c50a8b15d2c. Will see if I need more as I write more samples.


Partial fix for #2095

This is a partial fix for #2095. Currently, HMC won't work with vectors, etc. of variables which have differently sized support in the transformed space, e.g.
@model function vector_of_dirichlet(::Type{TV}=Vector{Float64}) where {TV}
    xs = Vector{TV}(undef, 3)
    for i = 1:3
        xs[i] ~ Dirichlet(ones(5))
    end
end

model = vector_of_dirichlet()
sample(model, NUTS(), 1000)

breaks with a similar error to that mentioned in #2095.
This occurs because unflatten, which is used in DynamicPPL.LogDensityFunction, attempts to reconstruct using a VarInfo which has a much larger vector for underlying storage than the one provided as input, e.g. varinfo is working with 3 x 5 dimensional xs but is only using a subset of the indexes, while the provided vector will be (3 - 1) x 5 dimensional since it's in linked space.
We can fix this by simply using the immutable DynamicPPL.link, which results in the "underlying" varinfo also using (3 - 1) x 5 dimensional storage for xs.

Yeah I'll have a look at this later :+1:

Okay, so I figured out what the issue is, and the fix is somewhat annoying.
DynamicPPL.(inv)link does not currently support specification using the Sampler, and so it links the entire VarInfo, even though only a sub-set of  the VarInfo needs linking.
We can add this quite "easily" for TypedVarInfo, but for UntypedVarInfo things become much more annoying :confused:

Okay, so this should now be fixed by https://github.com/TuringLang/DynamicPPL.jl/pull/542

Is there a reason why we're doing a lot of progress reporting during tests @yebai @devmotion ? It makes it quite annoying to determine which tests are actually failing by inspecting the logs :confused:

Is there a reason why we're doing a lot of progress reporting during tests @yebai @devmotion ? It makes it quite annoying to determine which tests are actually failing by inspecting the logs 😕

I don"t recall any previous discussions. I think it would be good to check that progress logging "works" by sampling with progress logging in the tests - but I think this could be limited to a few selected examples and in the remaining sample calls we could disable it. Since generally progress logging introduces overhead and slows down sampling, this could even improve CI times.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

-1 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/mcmc/hmc.jl
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
1368

💛  - Coveralls


Done with Program for Tower of Hanoi

Tower of
My Approach :
Take an example for 2 disks :
Let rod 1 = 'A', rod 2 = 'B', rod 3 = 'C'.
Step 1 : Shift first disk from 'A' to 'B'.
Step 2 : Shift second disk from 'A' to 'C'.
Step 3 : Shift first disk from 'B' to 'C'.
The pattern here is :
Shift 'n-1' disks from 'A' to 'B'.
Shift last disk from 'A' to 'C'.
Shift 'n-1' disks from 'B' to 'C'.
for better understanding
look at this image

issue fixed #496

Please star this repository to show your support. In the meantime I am reviewing your code.


Flaky-test: ClientDeduplicationFailureTest.testClientDeduplicationCorrectnessWithFailure

ClientDeduplicationFailureTest.testClientDeduplicationCorrectnessWithFailure is flaky. It fails sporadically. It was disabled in https://github.com/apache/pulsar/pull/14509 because it was leading to out of memory errors. Ideally, we'll find a way to re-enable it without causing OOM.

btw. The test has been quarantined before. There's not a big difference to completely disabling it.


New install of VSC x64 on Windows hogs CPU renders machine unresponsive

VSCode Version: 1.15.1
OS Version: Windows Server 2012 R2 (2GB Ram - It's a VM)

Steps to Reproduce:

Install VSC x64 or x32
Open Task Manager
Launch
Watch machine grind to a halt

Image shows the issue.  The issue happens with or without the folder opened in VSC (it was an issue directly from first launch).
Can't even close VSC with X or File->Exit, have to end task.

No extensions were installed, it was a clean install.
Running with code --disable-extensions shows the same issue.
I uninstalled and installed the 32 bit version, and I get exactly the same problem.
Have now uninstalled completely.

We just published a new release. Could you tell me whether this happens with 1.16?

I'm not the OP, and I have a pretty decent setup (I7 processor with 32g RAM), but I'm not seeing anything with 1.16.0 - didn't notice anything wrong with the older version either.

Same issue with 1.16.0

It's been running a few minutes and has not yet shown the welcome screen.

Where are you launching it from? So it has never launched successfully?
Can you delete the C:\Users\user\AppData\Roaming\Code folder and try again?

I am launching it from the desktop icon.
I removed the Roaming\Code folder, and relaunched and its still hogging CPU.

What happens if you start it from the command line with --prof-startup? It should create a couple cpuprofile files in your user directory.

And can you just try reinstalling?

Starting with --prof-startup does create a 4d2b_main.cpuprofile.txt
Same issue after uninstalling, deleting Roaming\Code and reinstalling.

Can you upload it? Might shed some light on what it's doing instead of starting up.

4d2b_main.cpuprofile.zip
Sorry, couldn't see the upload option before.  Must not have been looking very hard, cause it's quite obvious!

Looks like it spends 24s on Menu after createBrowserWindow, don't know what that means... then at the very end, it's doing something else, then a couple seconds doing nothing... @jrieken, @Tyriar?

Any way you can try adding more RAM to the VM?

I will have to ask the admin, but yes I can almost certainly bump it up.  Will be Monday now though (UK time here).

+@bpasero

@Mehuge single-CPU VM?

I am not finding the duplicate right now, but running VSCode on a single-CPU VM leads to this.

this one: https://github.com/Microsoft/vscode/issues/17097. We rely heavily on multiple processes which doesn't work well on single core machines.

FYI re #issuecomment-328164482 upped memory to 4GB made no difference.

how about upping cpu count?

I think we know the issue is single CPU on a VM, I don't plan on making any further changes to the VM.

ok. closing as dupe of #17097


WIP: frxETH AMO using minimal changes

If you made a contract change, make sure to complete the checklist below before merging it in master.
Refer to our documentation for more details about contract security best practices.
Contract change checklist:

[ ] Code reviewed by 2 reviewers.
[ ] Copy & paste code review security checklist below this checklist.
[ ] Unit tests pass
[ ] Slither tests pass with no warning
[ ] Echidna tests pass if PR includes changes to OUSD contract (not automated, run manually on local)

Closing as we are no longer using this implementation


⚠ Bofrost.nl has degraded performance

In f41ac3e, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1184 ms

Resolved: Bofrost.nl performance has improved in 2262f26.


Add class when xeditable is disabled

Hi!
Is it possible to add a "disabled" css class to the Xeditable element (when it is disabled)?
It would be really helpful to have a css class like that in order to adjust the appearance of the element.
Right now the style is the same, whenever it is enabled or disabled.

It's possible.  Do you need the styles to dynamically change between enabled/disabled?

Yes, something like "editable-disabled" to be added runtime when edit-disable="true" and to be removed when edit-disable="false"

I tried watching the editable-disabled attribute and changing the classes, but I couldn't get it to work right.
I could give you a one time shot css when the element is loaded the first time.
Or, you could add some styling yourself by adding a class based upon the same criteria you are using for the editable-disabled attribute.

It's pretty useless for us give it css when the element is loaded the firts time.
Right now it isn't a fundamental feature, so we'll work on it when we'll have the time.
Thanks for your time! :smiley:

@MooDySmiles did you figure out a fix for this?

@ckosloski we hacked that by placing a span (with ngIf) when we want xeditable to be disabled.
We have been moved onto a new whole project, so I don't think we will be able to do any progress on this, sorry :disappointed:

@MooDySmiles  Do you want to close this issue then?


feat: create embedder&llm for worker based on model type

What type of PR is this?

What this PR does / why we need it
Which issue(s) this PR fixes

Fixes #268
Special notes for your reviewer

@CodiumAI-Agent /review

PR Analysis

🎯 Main theme: Implementing worker-based model type for embedding and LLM resources
📝 PR summary: This PR introduces the ability to create Embedder and LLM resources based on the model type of a worker. It checks the readiness of the worker and updates the status of the Embedder and LLM resources accordingly. It also includes the necessary error handling and updates to the worker controller.
📌 Type of PR: Enhancement
🧪 Relevant tests added: No
⏱ Estimated effort to review [1-5]: 4, because the PR includes significant changes across multiple files, including the addition of new functions and error handling. It requires a good understanding of the existing codebase to review effectively.
🔒 Security concerns: No security concerns found

PR Feedback

💡 General suggestions: The PR is well-structured and the changes are logically grouped. However, it would be beneficial to add unit tests to verify the new functionality and ensure that it works as expected. Additionally, it would be helpful to include more detailed comments in the code to explain the purpose and functionality of the new methods.

 🤖 Code feedback:

relevant file: controllers/llm_controller.go
suggestion: Consider handling the error returned by the 'Validate' method directly inside the switch case. This would make the code more readable and maintainable. [medium]
relevant line: '+		res, err := embedClient.Validate()'

relevant file: controllers/embedder_controller.go
suggestion: Similar to the previous suggestion, consider handling the error returned by the 'Validate' method directly inside the switch case. [medium]
relevant line: '+		res, err := embedClient.Validate()'

relevant file: controllers/worker_controller.go
suggestion: It might be more efficient to use a map for checking the model type instead of using a loop. This would improve the time complexity from O(n) to O(1). [important]
relevant line: '+	for _, t := range strings.Split(model.Spec.Types, ",") {'

relevant file: api/v1alpha1/model.go
suggestion: The method names 'IsLLMModel' and 'IsEmbeddingModel' suggest that they return a boolean value. However, they return a string. Consider changing the method names to reflect their return type or change the return type to boolean. [medium]
relevant line: '+func (model Model) IsLLMModel() string {'

How to use
 Instructions

Tag me in a comment '@CodiumAI-Agent' and add one of the following commands:
/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.
/add_docs: Generate docstring for new components introduced in the PR.
/generate_labels: Generate labels for the PR based on the PR's contents.
see the tools guide for more details.

To edit any configuration parameter from the configuration.toml, add --config_path=new_value.
For example: /review --pr_reviewer.extra_instructions="focus on the file: ..."
To list the possible configuration parameters, add a /config comment.


API calls are not hidden behind sign in

If sign in is required in config, you can still use API to get any info, for example /api/v1/users/search?q=a

@Unknwon Is there an ETA on this one?

@raunsbaekdk I've put into release 0.6.2.

Fix has been pushed to develop branch. :dizzy:

Close as implemented, feel free to continue discussion! :smile:


Callout on Polyline?

Is this a bug report?
Maybe yes maybe no.
Have you read the Installation Instructions?
Yes
Environment

Steps to Reproduce
Does Polyline support callout the way it does like marker? or Polyline doesnt support callout at all?
I've tried below but callout doesnt seems to appear
          <Polyline
            coordinates={gotoPolyCoord}
            lineCap='round'
            lineJoin='round'
            strokeColor='purple' // fallback for when `strokeColors` is not supported by the map-provider
            strokeWidth={4}
          >
            <Callout style={mapCalloutStyles.calloutContainer} tooltip>
              <MapCallout
                title={'hihi'}
                description={`yo!`}
              />
            </Callout>
          </Polyline>

Polylines don't have callouts.

@rborn: wondering will there be any plan to include a callout for polyline as it's already supported a onPress event?

@isaaclem we're a community so features are added based on what people add in their PRs.
Besides this it has to be supported by the native sdk - which I'm not sure it is.

Take one of the polyline points (the middle one or the one pressed) and construct a marker and a callout based on it.


Adding CVEs for kube-scheduler-1.27

Adding Advisory GHSA-8pgv-569h-w5rw for kube-scheduler-1.27
Adding Advisory GHSA-rcjv-mgp8-qvmr for kube-scheduler-1.27

added in https://github.com/wolfi-dev/advisories/pull/825


extension error

Does this issue occur when all extensions are disabled?: I do not know. (exntension list keep loading with a clock at the left bottom of the extension icon.)

VS Code Version: win 10
OS Version: 1.91.1

Steps to Reproduce:

Error "A shared background process terminated unexpectedly. Please restart the application to recover" appears when starting VSCode while restarting not works.

PS: Extension may be the Key of the error. Several days before, my Copilot expired and I installed activated Codeium extension. After that I did not use VSCode until now and the error appears.
Thanks for your help!

Please use the Start Extension Bisect command to investigate whether the problem is being caused by one of your extensions

Please use the Start Extension Bisect command to investigate whether the problem is being caused by one of your extensions

Thanks for your reply.
I have run the command and how can I check the output of the commad?


Try to fix issues in setup-miniconda

New feature/Bug fix description
As shown in https://github.com/m-rossi/jupyter-docx-bundler/runs/2482291360 the runs fail on Python 3.7 on Ubuntu because the PyPy-version of Python is somehow used.
Checklist

[ ] Tests have passed
[x] New code is covered with tests

Codecov Report

Merging #72 (b32d5dd) into main (4ef9a87) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #72      +/-   ##
==========================================
- Coverage   97.60%   97.52%   -0.09%
==========================================
  Files           6        6
  Lines         459      444      -15
==========================================
- Hits          448      433      -15
  Misses         11       11

Impacted Files
Coverage Δ

jupyter_docx_bundler/__init__.py
95.65% <0.00%> (-0.19%)
:arrow_down:

jupyter_docx_bundler/tests/conftest.py
99.35% <0.00%> (-0.05%)
:arrow_down:

jupyter_docx_bundler/tests/test_bundler.py
100.00% <0.00%> (ø)

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 4ef9a87...b32d5dd. Read the comment docs.

The workaround from here did the trick: https://github.com/mamba-org/mamba/issues/658#issuecomment-760883189


Prompt for choice does not allow default option to be selected

Repro:
Remove-Item test.cs -confirm
The default response when return is pressed without entering any value is 'Y'.
However this is not supported. In VS Mac 8.10 nothing happens. In VS Mac 17.0 a new prompt appears without any action being taken and PowerShell is still waiting for user input.

Fixed in vsmac-17.0 branch.


Add green option to solarized theme

Turns the outer-most section of the statusline Solarized green, making it
look more like classic powerline in normal mode. To enable it:
let g:airline_solarized_normal_green = 1

By default, the theme is unchanged. Only when this option is enabled are any
changes visible.

(Ignore the orange/red stuff on the side. I that showed up before this change and I haven't spent the time figuring out how to get it to go away).

thanks!


[QA-1990] UI integration test slack notification from circleci

https://broadworkbench.atlassian.net/browse/QA-1990
(For context, this is split from https://github.com/DataBiosphere/terra-ui/pull/3287)
Changes:

Retreive CircleCI job artifacts
Process Jest test summary JSON files to retrieve tests information
Compose and send Slack message when test job is finished
Provide a way to run JS code in Terminal window so it can be launched in a CircleCI step (not included in this PR)

To test this:

Set CIRCLE_BUILD_NUM and SLACK_BOT_TOKEN env variables
From integration-tests dir, cmd to invoke script: node --require ../.pnp.cjs ./slack/notify-circleci-test-results.js

Example: this notification was posted to dsp-workspaces-test-alerts today.

@breilly2 @cahrens Hi, can you take a look? I need two approvals. thank you.

I'm deferring to @breilly2 as the assigned reviewer (I'm in the midst of other reviews).


Be able to render any page with an arbitrary pathname in custom server that uses next.

This is a hack I used to implement this, but it would be nice to be able to pass options all the way down to renderToHTML, instead of mutating instance state like this.
const { createServer } = require('http')
const { parse } = require('url')
const next = require('next')
const pathMatch = require('path-match')

const dev = process.env.NODE_ENV !== 'production'
const port =  process.env.PORT || 3000
const app = next({ dev })
const handle = app.getRequestHandler()
const route = pathMatch()

/**
  This is a hack to get around the fact that it _seems_ next
  does not allow us to separate the concepts of pathname and page.
*/
const renderPage = (page, req, res, pathname, query) => {
  app.renderOpts.page = page
  app.render(req, res, pathname, query)
  delete app.renderOpts.page
}

// Then
// renderPage('lesson', req, res, 'course/1/lesson/1', query)

renderOpts is something private. You can simply create a similar function with the use of a HTTP server like express.


4.0.0: Unknown operation named ""."

Hello all!,
after updating a side project to version 4.0.0, the code compile normally, but every query I execute returns: Unknown operation named \"\"."
Rolling back to 3.9.0 solves the issue.
Relevant dependencies of my project:
"@graphql-modules/core": "0.7.17",
"@graphql-modules/utils": "0.4.0",
"fastify": "2.14.1",
"fastify-cors": "3.0.3",
"fastify-gql": "3.9.0",
"graphql-import-node": "0.0.4",
"graphql-tools": "6.0.9",
"reflect-metadata": "0.1.13"

Can you provide an example on how to reproduce the problem? We just bumped graphql-js to the latest so it might be that you need to adjust your schema.

Ciao Matteo,
here's a minimal repro on codesandbox
https://codesandbox.io/s/keen-sara-hs6f5
please start it with npm start or yarn start and open the embedded browser to
https://hs6f5-4000.sse.codesandbox.io/playground
to access.
There are two query:
query {
    getAll {
        id
        firstName
        lastName
    }
}

and
query {
    getOne(id: 1) {
        id
        firstName
        lastName
    }
}

change the id to 1...50

Can you create a repro without the use of all those extra modules? I have a feeling one of them is incompatible with graphql 15.

Excluding the fastify ecosystem, what remains is
"@graphql-modules/core": "0.7.17",
"@graphql-modules/utils": "0.4.0",
"graphql-import-node": "0.0.4",
"graphql-tools": "6.0.9",
"reflect-metadata": "0.1.13"

and those are the bare minimum for using GraphQL-Modules. Once removed these packages and refactored the code, the remaining is a bare fastify server which I'm sure its working because I have another fastify + fastify-gql (without any extra package beside these) project and work like a charm.
I tried to see in my tree if some other package use graphql.js, but the only one seems to be fastify-gql:
$: npm ls graphql
<EMAIL_ADDRESS>/home/user/DEV/graphql
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
So my best guess is something is broken between fastify-gql 4.0.0 and GraphQL-modules. I'll dig through the code as soon as I can.

Keep me posted!

I am also getting this error w/ type-graphql and I reproduced the same with raw graphql-js.
CodesandBox:
https://codesandbox.io/s/fastify-gql-error-r4wlt

I am also getting this error w/ type-graphql and I reproduced the same with raw graphql-js.

I would recommend you to open an issue in type-graphql or graphql-js as it's definitely not related to this module.

I already open am issue in type-graphql repo.
Here it is: https://github.com/MichalLytek/type-graphql/issues/651

I can confirm that the issue is not related to fastify-gql. I forced the package manager with npm shrinkwrap to resolve "graphql": "^15.0.0" but keeping "fastify-gql": "3.9.0" and the error is shown.
Since the other problem is Graphql-Modules, I'll investigate with the guys over here!
Thank you for pointing me on the right direction.

@ardatan in the mentioned issue got the solution for this problem. IDK why now graphql.js mandates the query name

@artecoop Thanks a lot for investigating the issue. Indeed @ardatan solution also fixes the issue with type-graphql.


awk -W interactive is not a useful flag

I have a query around the consul agent start script. https://github.com/cloudfoundry-incubator/consul-release/blob/8aa2c923ff8bb5c058e965871d7357af19531731/jobs/consul_agent/templates/agent_ctl.sh.erb#L9-L10 ; It uses awk -W interactive which whilst a valid flag in mawk, is not any other awk variant. Many awk variants ignore the -W flag, but some throw errors.
This caused us an error earlier today, specifically that consul did not start on our instance of PCF. We are still unsure how that deploy received a different awk than usual, however it did, and at that point we looked into -W interactive, it is intended to cause fflush but only does so on mawk all other awk variants either ignore -W interactive or else throw an error and exit.
Keaty && Dave Goddard

Hi Keaty,
We looked into this and switched from awk to sed. This should solve your problem.
The update can be found at: https://github.com/cloudfoundry-incubator/consul-release/commit/86221316a238992ae2fab8b030647817a6bc0738
Cheers,
Angela and @evanfarrar


language-markup < problem

how i understand plugin dont work with html tags, and i must replace simbol "<" to "& lt;" ?
why you dont add something like this:
$('code').each(function(){
var el = $(this);
el.text(el.html());
});

http://www.sitepoint.com/everything-need-know-html-pre-element/
Please do some research before posting queries.

This is the way
 elements work. It has nothing to do with Prism itself.

Why if it work nice in your example you do something like this:

<p>hi234234 sdfs!</p>

Why you cant write:

hi234234 sdfs!

prism-gh-pages/example.html
Maybe i don't understand something, but prism don`t work with native html. Right?

@iLightGit Did you even go through the link, I posted?.
It's html that does not work like this:

hi234234 sdfs!

It has nothing to do with prism.
You can try it without prism and see for yourself.

I think what @iLightGit means is: Since Prism strips out HTML tags inside  anyway, why not just allow unescaped HTML inside it?
If you add a language-markup class to the above example, and run it through Prism, you will get

hi234234 sdfs!

So it really wouldn’t break anything whether Prism auto-escapes HTML or not.
I don’t agree with this myself, to be honest. I’m just trying to figure out the reasoning behind the comment. Auto-escaping should not be part of Prism in any case, and you can always loop through the code blocks yourself, or just write a plugin to escape the block before Prism process them.

@iLightGit: The point @vkbansal is trying to make is that even without Prism, if you use the following code:

hi234234 sdfs!

the browser will not make the
 tag appear litterally, it will be interpreted as standard HTML.
Prism removes standard HTML from the tags before highlighting. But it could be able to keep it in some cases (see #558 if you're interested in this). So... to be able to keep it sometimes, you have to escape your own code you want to display, so that it is not to be confused with standard HTML.

Even if I escape the opening html tags with "lt;", prism still strips them off.

@StanBPublic Demo or it didn’t happen :)

Did some pretty scientist said demo?
[Problem] (http://jsfiddle.net/javatlacati/6oqdqjh3/)
[Solution] (http://jsfiddle.net/javatlacati/6oqdqjh3/1/)

Nice demo, @javatlacati!
But I think we're specifically trying to see a demo of Prism stripping <? (Correct me if I'm wrong, though!)
Also, we should leave references to physical attractiveness out of this; they aren't relevant.

My bad, I was thinking it was a pre escaping html as html issue. Please continue on your philosophical discussion on the < entity

i'm still having the problem with "<".
forgive me i'm not that much skilled, but here's my problem:
i'm pulling snippets of code stored in mysql. (bash, python,..)
and when i try to display some code with this:
echo "
".$myrow["body"]."
";
the data pulled from mysql look like this:
pass script to ssh:

ssh host -l user $(<cmd.txt)

but in the browser it get displayed like this:
pass script to ssh:
ssh host -l user $(

everything after the < is missing on that line.
what can i do about it ?
thanks

If it's PHP, use htmlspecialchars to escape the output from the database, which you should do anyway for security reasons.

thanks for the quick reply but,
if i do that, the output in the browser is not highlighted anymore. it shows a flat line like this:
pass script to ssh.

ssh host -l user $(<cmd.txt)

no highlight, no newline, just a long raw string.

You should just store the code snippet itself in the DB, not the wrapping pre and code elements.

that's not possible, my whole article is stored in the DB.
that would put the whole article inside  and try to highlight it. that is not reasonable.
what i'm describing here about storing whole blog article in DB is very common thing.

You should escape the actual code before even storing it IMO.

thanks for the reply Golmote, but,
that would mean i would be storing escaped code & text in DB.
i was reading on stackoverflow this is the kind of thing i'd want to avoid.
i have reverted back my site to previous highlighter, the one from Gorbatchev. Prism looked nice, but i cannot seem to be able to use it right now. i will have to see if i can find a workaround all this problem i was describing above.

There are a couple places you could choose to handle this, but you'll have to decide that based on your application's architecture. As mentioned above, auto escaping is not something Prism intends to handle, so you'll need to devise your own solution here. You could escape it in the text editor, save the code snippets separately so you can escape them on their own (what I do), or use JS to escape the characters on the front end before Prism runs or use a Prism hook to do it.

the only thing i can think of, is to grab the body of the blog entry off the DB, and then parse it to isolate bits between  custom tags then i would need to replace the custom tags with

 . this is a bit of an hassle. unless i can figure something easier, i think i'll stick with what i have at the moment.


Added argparse to mplot

We added the arguments -s, --start; -e, --end; and --box. Were not sure
how to exactly use the box values, but everything is reading correctly.
We made start and end required, and box optional.

we will use the box to denote the lower left and upper right of a recangular area within which you can do simple column and row operations to test median filtering etc.  This is useful as long as we have not defined where the treeline is.


Update examples

With all the changes in afire v2.0.0, a lot of the examples need to be updated.

[x] basic
[x] serve_file
[x] routing
[x] data
[x] path_param
[x] header
[x] state
[x] cookie
[x] error_handling
[x] serve_static
[x] middleware (show state access)
[x] logging
[x] rate_limit
[x] threading
[x] trace

Fixed in 549805eadd836f5748331906c09fe4c1a732fd4f


[Bug]: Eslint complaint of unused props passed to Calendar icons

Describe the bug
Think a screenshot is enough

Affected component/components
Calendar
How to reproduce
Just check the source code for the calendar
Codesandbox/StackBlitz link
No response
Logs
No response
System Info
No need to provide this

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

Please this is preventig deployment in my code, How can this be resolved

Bumping!

Encountered the same issue.

Bump

Is this props param used for something? An MR to simply remove the props here would be sufficient?
This affects the usual Next.js process deploying to Vercel, the component out of the box will fail to deploy as it will make the linter fail.
Happy to open the MR if that's all it takes to fix it. I wonder if there is a special reason behind keeping this open?


Doesn't build properly anymore on release 3080 x64

Hello, just tried the latest Sublime3 release (3080) on Win x64 and the Haxe builds doesn't work anymore unfortunately. I get no messages at all in the output window, but in the status bar a message that the build succeeded (no output generated though).
I see in the changelog that some build modifications has been made: http://www.sublimetext.com/forum/viewtopic.php?f=2&t=17471

Great! Is the update available already in package manager?

Yes, it should (Package Manager version = master)

Thanks

Sorry, not sure what you mean with Package Manager version = master?

Package Control should automatically upgrade to latest master.

Ok, I didn't happen yet, but maybe later...?

Unfortunately I'm not 100% sure how Package Control works internally.
You may need to restart ST or "upgrade package" manually through the command panel...

Should be available now.

Yes, it was automatically upgraded now and works fine! Thanks for your work with this.

I'm experiencing the same problem again, also on build 3083. This time the syntax highlighting disappeared, and after using "Set Syntax: Haxe" on one file and pressing Ctrl-B, no output is generated. It find all the hxml files however, giving me a selection, but same for all.
Tried it on multiple projects and on Flashdevelop, so there's nothing wrong with the installation or compilation server. One or more haxe.exe processes are still hanging after I close Sublime.
I have cloned this repo and installed it manually, so let me know if I can assist in debugging.


ABetterBrowser banner is not (or only intermittently) displayed for IE8

CloudFlare support suggested I report this to you.
I defined the CloudFlare aBetterBrowser app for IE 8.0 and below, and tested it from an IE8 browser on Wndows XP.  Although the  section of the source code for a very simple html reply includes the abetterbrowser script, the black abetterbrowser banner is not displayed.  The following screenshot, taken 3+ weeks ago, includes page source, rendered page, and proof of IE8.  Repeating the display today gives the same result.

You can test this yourself by visiting website www.abtuk.org.uk for which the abetterbrowser CloudFlare app is, and will remain, enabled for IE8 and below.  The black abetterbrowser banner is not displayed.  This website has only 1 page, containing a single link (a clickable image).
Further: If that link is clicked, this takes the visitor to multi-page website www.biodanza4me.com which is in the same CloudFlare account and for which the abetterbrowser app is also enabled for IE8 and below.  The black abetterbrowser banner appears intermittently on the pages of this website - I have not yet been able to discern a pattern for the [non-]display of the banner.
During all my testing I did not click the link in the black abetterbrowser banner, nor did I click the big X (to dismiss it).
Regards,
Pete

I'm seeing these errors in console:
SCRIPT5007: Unable to get property 'toString' of undefined or null reference
rocket.js (4,13223)

CloudFlare's rocket loader could be failing?

Thanks for the quick response.
I have de-activated Rocket Loader and re-tested.  IE Developer Tools | Network shows that file rocket.js is no longer downloaded, and within the html source the var CloudFlare= string now contains rocket:"0" (previously was rocket:"a").  However the black abetterbrowser banner is still not displayed.  Please try it yourself.

It seems like CloudFlare.define promise is not actually being fired, so the code is never executed. Could you tell CF support to get one of their engineers to take a look?

I have passed this back to CloudFlare (in their ticket number 436010).  I will let you know their response.

Did CloudFlare get back to you on ticket #436010, bdzpete? I, too, am having issues with A Better Browser.

Hi,
On 26 July I updated my CloudFlare ticket #436010 with the following text:    One month on: Has there been any progress on resolving this?
CloudFlare have not yet responded.  But historically I've found that although CloudFlare first-line support are fairly good at responding, once a topic gets passed back to their developers there is rarely further feedback (whether to say "fixed" or to say "no change planned - working as designed").  This may not be justified comment for CloudFlare's paying customers, but it is certainly true for my cost-free basic account.
Just a thought: since A Better Browser is the first App listed the the CloudFlare "Apps" section, I would have thought it would be in their own interests to ensure that it integrates effectively with CF (it would be bad PR otherwise, for both them and you).  As the app developer, you probably have more influence than a non-paying customer like me has with the CF developers.  Can I suggest you contact them directly?
Regards,Pete Simpson
        From: exegete<EMAIL_ADDRESS>To: xPaw/CF-ABetterBrowser<EMAIL_ADDRESS>Cc: bdzpete<EMAIL_ADDRESS>Sent: Tuesday, 4 August 2015, 13:25
Subject: Re: [CF-ABetterBrowser] ABetterBrowser banner is not (or only intermittently) displayed for IE8 (#68)
Did CloudFlare get back to you on ticket #436010, bdzpete? I, too, am having issues with A Better Browser. —
Reply to this email directly or view it on GitHub.

I work at CloudFlare. The report has been acknowledged and is being examined by one of the developers of the loader script responsible.
John

Thank you, John.
Once the examination is complete, is it possible to update us here?

I hope a solution comes for this problem. I experience the same (the warning banner not showing) in IE 11. It's working fine in Chrome and Safari. I'm also using CloudFlare with Rocket Loader, but turning off Rocker Loader doesn't change anything.


NewClient return error

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[√ ] elastic.v7 (for Elasticsearch 7.x)
[ ] elastic.v6 (for Elasticsearch 6.x)
[ ] elastic.v5 (for Elasticsearch 5.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ ] elastic.v2 (for Elasticsearch 1.x)
Please describe the expected behavior
Connecting to es server successfully and when searching the document, it will return a correct value
Please describe the actual behavior
When connecting to elasticsearch server, it will return a error,

but after I set

to false,  I can connect to it, after that , when I want to search something, I get a 411 error when I try to search a document from es
Any steps to reproduce the behavior?
I don't know, maybe it's the problem of environment, this server is deploy in the cloud server of my company, when I use local elasticsearch, everything is fine. But it's all right when using go-elasticsearch

Honestly, this is the first time I've ever seen HTTP Status 411 mentioned in a bug report. I even had to look it up :-)
Not sure what's going on there. Can you run requests against the ES server with curl or Postman?

Honestly, this is the first time I've ever seen HTTP Status 411 mentioned in a bug report. I even had to look it up :-)
Not sure what's going on there. Can you run requests against the ES server with curl or Postman?

It's all right in Postman, like this

but as you can see below

it will return a 411 error

That 411 is due to a missing Content-Length in the HTTP request. That IIRC happens by default in the Go HTTP client because it uses chunked transfer encoding by default, which omits the Content-Length.
Do you have a proxy between the Go client and the ES instances that requires you to have a Content-Length in the HTTP header? Because ES doesn't require that, hence it works on your machine.
You might be able to work around this by creating a custom http.Client and passing it to Elastic via SetHttpClient(...). The custom http.Client needs to set a HTTP header of Transfer-Encoding: identity, if I read this correct.

That 411 is due to a missing Content-Length in the HTTP request. That IIRC happens by default in the Go HTTP client because it uses chunked transfer encoding by default, which omits the Content-Length.
Do you have a proxy between the Go client and the ES instances that requires you to have a Content-Length in the HTTP header? Because ES doesn't require that, hence it works on your machine.
You might be able to work around this by creating a custom http.Client and passing it to Elastic via SetHttpClient(...). The custom http.Client needs to set a HTTP header of Transfer-Encoding: identity, if I read this correct.

No, it doesn't work,  I  inherite the http.Client and overwrite the method like that

but I still get a 411 error, what confused me is in postman, I don't need content-length

It's a get request...
I don't know why in my golang code it will return a 411 error...

I'm confused about your httpClient. I don't think Elastic is using it, to be honest. Here's how to use your own custom HTTP Transport with Elastic.

From what I see from the screenshots, it doesn't seem to work even without Elastic. So does something like this work at all?
package main

import (
	"fmt"
	"log"
	"net/http"
	"net/http/httputil"
	"strings"
)

func main() {
	body := strings.NewReader(`{"query":{"match_all":{}}}`)
	req, _ := http.NewRequest("GET", "http://localhost:9200/_search?pretty=true", body)
	req.Header.Set("Content-Type", "application/json")
	req.Header.Set("Transfer-Encoding", "identity")
	res, err := http.DefaultClient.Do(req)
	if err != nil {
		log.Fatal(err)
	}
	defer res.Body.Close()
	buf, _ := httputil.DumpResponse(res, true)
	fmt.Println(string(buf))
}

From what I see from the screenshots, it doesn't seem to work even without Elastic. So does something like this work at all?
package main

import (
	"fmt"
	"log"
	"net/http"
	"net/http/httputil"
	"strings"
)

func main() {
	body := strings.NewReader(`{"query":{"match_all":{}}}`)
	req, _ := http.NewRequest("GET", "http://localhost:9200/_search?pretty=true", body)
	req.Header.Set("Content-Type", "application/json")
	req.Header.Set("Transfer-Encoding", "identity")
	buf, _ := httputil.DumpRequestOut(req, true)
	fmt.Println(string(buf))
	res, err := http.DefaultClient.Do(req)
	if err != nil {
		log.Fatal(err)
	}
	defer res.Body.Close()
	buf, _ = httputil.DumpResponse(res, true)
	fmt.Println(string(buf))
}

This will successfully return here. If you can get this to work against your production cluster, we know what happens:
$ go run main.go
GET /_search?pretty=true HTTP/1.1
Host: localhost:9200
User-Agent: Go-http-client/1.1
Content-Length: 26
Content-Type: application/json
Accept-Encoding: gzip

{"query":{"match_all":{}}}

HTTP/1.1 200 OK
Content-Type: application/json; charset=UTF-8

{
  "took" : 0,
  "timed_out" : false,
  "_shards" : {
    "total" : 0,
    "successful" : 0,
    "skipped" : 0,
    "failed" : 0
  },
  "hits" : {
    "total" : {
      "value" : 0,
      "relation" : "eq"
    },
    "max_score" : 0.0,
    "hits" : [ ]
  }
}

it's still 411 error

I'm confused about your httpClient. I don't think Elastic is using it, to be honest. Here's how to use your own custom HTTP Transport with Elastic.

I saw the method like yours in https://developer20.com/add-header-to-every-request-in-go/
but this passage say like below

From what I see from the screenshots, it doesn't seem to work even without Elastic. So does something like this work at all?
package main

import (
	"fmt"
	"log"
	"net/http"
	"net/http/httputil"
	"strings"
)

func main() {
	body := strings.NewReader(`{"query":{"match_all":{}}}`)
	req, _ := http.NewRequest("GET", "http://localhost:9200/_search?pretty=true", body)
	req.Header.Set("Content-Type", "application/json")
	req.Header.Set("Transfer-Encoding", "identity")
	buf, _ := httputil.DumpRequestOut(req, true)
	fmt.Println(string(buf))
	res, err := http.DefaultClient.Do(req)
	if err != nil {
		log.Fatal(err)
	}
	defer res.Body.Close()
	buf, _ = httputil.DumpResponse(res, true)
	fmt.Println(string(buf))
}

This will successfully return here. If you can get this to work against your production cluster, we know what happens:
$ go run main.go
GET /_search?pretty=true HTTP/1.1
Host: localhost:9200
User-Agent: Go-http-client/1.1
Content-Length: 26
Content-Type: application/json
Accept-Encoding: gzip

{"query":{"match_all":{}}}

HTTP/1.1 200 OK
Content-Type: application/json; charset=UTF-8

{
  "took" : 0,
  "timed_out" : false,
  "_shards" : {
    "total" : 0,
    "successful" : 0,
    "skipped" : 0,
    "failed" : 0
  },
  "hits" : {
    "total" : {
      "value" : 0,
      "relation" : "eq"
    },
    "max_score" : 0.0,
    "hits" : [ ]
  }
}

it's still 411 error

So if this still fails, it's not due to Elastic, the library, but somehow related to your infrastructure. Find out what's wrong with your infrastructure first, then we can make it work in Elastic.

OK，Thank you all the same, I will try to get in touch with the admin of es cluster to find out the problem

That's the problem of IDE proxy

@olivere I'm also facing the same issue. I have a proxy in between
Due to Transfer-Encoding: chunked, I'm getting 500 from proxy
How can I disable this? I'm not able to create custom client. Can you please help here
Thanks in advance


what do you mean by "AsyncIO still can't exceed the speed of GIL"

what do you mean by "AsyncIO still can't exceed the speed of GIL", as far as I am concerned,Gil is a machinist ,not a number,so asyncio may be good enough?

Here have a good explanation.
https://www.youtube.com/watch?v=0kXaLh8Fz3k
The GIL don´t allow to use multiple CPU cores


Add support for package comment (#51)

Several linting tool warn about missing package level comments so
write one by default.

Thanks!

That was fast and painless. Thanks!


Please invite me to the Recode-Hive GitHub Community Organization

Name
Vaishnavi Kotturi
Discord Username (if applicable)
No response
Additional Context
No response

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/Recode-Hive/people


Improve description of DFP and AdSense settings.

People keep putting javascript into those fields. Also add a constraint on the fields so people get an error when they paste javascript.

Morning Neil and thanks heaps for this!  Is it ok if I add you to the team so that if you need to make PR you can merge them too?

Have sent the invite anyway... gtg now and get ready for work.  Thanks again and have a great evening!

Awesome, thanks! :heart:


fix: correct css modules hashing to match Webpack when leading digit #8735

Summary
As detailed in #8735 Rspack is currently out of sync with a recent change in Webpack around how css module hashes that lead with a digit are handled.
Current Rspack functionality removes all digits from the start of the hash so that hashed classnames never start with a digit, however this produces an edge case where when the hash is entirely digits it is reduced down to an empty string.
Webpack fixed this issue recently to instead prepend hashes starting with a digit with a underscore character ("_") - https://github.com/webpack/webpack/blob/964c0315df0ee86a2b4edfdf621afa19db140d4f/lib/dependencies/CssLocalIdentifierDependency.js#L81
This commit updates the regex/replace functionality to match Webpack's implementation.
Checklist

[x] Tests updated (or not required).
[ ] Documentation updated (or not required).

 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.

Some test cases failed. If they are fixed, this can be merged.

Sorry had missed an update to a classname in a test, updated now if someone could take a look @ahabhgk @LingyuCoder


Doc: Ascii.doc rendering of some operators (especially involving =) is totally broken

https://sdk.dfinity.org/docs/language-guide/language-manual.html#syntax-ops for some obvious examples.
Looks like APL!

Although I love APL, https://github.com/dfinity/antora-sdk/pull/41 may be the reason? Not sure.

Could be, but I think it was actual us just failing to understand asciidoc.

@chenyan-dfinity would that explain why the examples here look wrong (assignment and equality render badly in the displayed code)

This looks like it’s using a font with code ligatures: https://github.com/tonsky/FiraCode
So it’s not a bug, it’s a feature…

Oh, I misread the calculator code and thought it was rendering := badly, but that's actually /=. Still I don't think stuff like:

is helpful.

Unfortunately, I tried to fix this here https://github.com/dfinity/motoko/pull/2636. I wonder what that will look like now...


Overflow

Added overflow-y for small devices

Issue:overflow content was not visible in small devices

Closing because issue has been resolved


Add JsonContent

Adds a simple JsonContent request body implementation, as sending a JSON body is very common.

@kelunik I made it a static method on BufferedContent. Does this see appropriate or do you think it should be a regular function? If so, do you have a name suggestion? I was thinking createJsonContent.


v<IP_ADDRESS> - /Output flag returns help text

Hi,
So it turns out in this version if you try to use the output flag it just pretends that your syntax is wrong.  I even tried using the exact command from Example 2 of the help text and still got the error.
PS D:\Downloads>
PS D:\Downloads> .\SetupDiag.exe /Output:C:\SetupDiag\Results.xml /Format:xml

SetupDiag v<IP_ADDRESS>
Copyright (c) Microsoft Corporation. All rights reserved.

Usage:
/Output:[Full path and file name for output log file] - Optional - Specifies the name and location for the results log file.
/LogsPath:[Full path to logs]- Optional - Specifies the location of logs to parse. Defaults to checking the current system for logs.
/ZipLogs:[True | False] - Optional - Tells SetupDiag.exe to create a zip file containing the results and all the log files that were parsed. The default value is true.
/Format:[xml | json] - Optional - Specifies the output format for log files to be XML or JSON. If not specified, text format is used.
/Scenario:[Recovery] - Optional - Instructs SetupDiag.exe to look for and process reset and recovery logs and ignore setup/upgrade logs.
/Verbose - Optional - Creates a diagnostic log in the current directory, with debugging information and details about SetupDiag.
/NoTel - Optional - Tells SetupDiag.exe not to send diagnostic telemetry.
/RegPath - Optional - Instructs SetupDiag.exe to add failure information to the registry under the given path.
        Registry paths should start with HKEY_LOCAL_MACHINE or HKEY_CURRENT_USER and be accessible at the elevation level SetupDiag is executed under.
/AddReg - Optional - Instructs SetupDiag.exe to add failure information to the registry on the executing system.
        SetupDiag will by default add failure information to the registry in Online mode only.
        Registry data will go to HKEY_LOCAL_MACHINE\SYSTEM\Setup\MoSetup\Volatile\SetupDiag unless otherwise specified.

Examples:
Example 1 - Offline Mode
SetupDiag.exe /Output:C:\SetupDiag\Results.txt /LogsPath:D:\Temp\Logs\Logs1 /RegPath:HKEY_CURRENT_USER\SYSTEM\SetupDiag
Instructs SetupDiag to parse setup/upgrade log files in the LogsPath directory and output the results to C:\Setupdiag\Results.txt.

Example 2 - Online Mode
SetupDiag.exe /Output:C:\SetupDiag\Results.xml /Format:xml
Instructs SetupDiag to look for setup/upgrade logs on the current system and output its results in XML format to C:\SetupDiag\Results.xml.

Example 3 - Online Mode, no parameters needed
SetupDiag.exe
Instructs SetupDiag to look for setup/upgrade logs on the current system and output the results to the same directory where SetupDiag is located.

Example 4 - Reset/Recovery Offline Mode
SetupDiag.exe /Output:C:\SetupDiag\RecoveryResults.log /LogsPath:D:\Temp\Cabs\PBR_Log /Scenario:Recovery
Instructs SetupDiag to look for reset/recovery logs in the specified LogsPath location and output the results to the directory specified by the /Output parameter.

Example 5 - Reset/Recovery Online Mode
SetupDiag.exe /Scenario:Recovery /Format:xml
Instructs SetupDiag to look for reset/recovery logs on the current system and output its results in XML format.
PS D:\Downloads>

Which of the docs pages is this related to? Maybe this one, linked below?

https://docs.microsoft.com/windows/deployment/upgrade/setupdiag
https://github.com/MicrosoftDocs/windows-itpro-docs/blob/public/windows/deployment/upgrade/setupdiag.md

BTW, if that is the correct page, did you read through the old & closed issue tickets to see if there is anything useful there to help finding out more?

https://docs.microsoft.com/nb-no/windows/deployment/upgrade/setupdiag#tabpanel-issues-closed

I noticed that one of the comments suggested that SetupDiag requires existing and/or failed setup install logs to work (maybe I misunderstood something).

@MrEnigmamgine. I have sorted the problem and found the solution
go tothe  command prompt with administrator rights
then type below command
md "C:\Tools\SetupDiag\SetupDiag Results"
and copy the file to SetupDiag.exe to *C:\Windows\System32*
and then type the command
SetupDiag.exe /Output:C:\Tools\SetupDiag\Results.xml /Format:xml
this software will work when windows upgrade is failed  from the existing version

It seems that your question has been answered by the community and thus we are marking this issue as resolved.
Please note that the issues section of this repository is intended for documentation feedback.
Should you have any further issues similar to this one, please consider opening a product support ticket here:
Windows 10 Support
Feel free to re-open this issue if you feel that there are further suggestions to improve the documentation itself.
Thank you for providing feedback.


where to find the client for krakend API Gateway

Hi all, I am a new beginner with krakend API Gateway. Wanted to know where to find the client to send requests to the krakend API Gateway.

Hi - this isn't something you would have a specific client for. Anything which can send an HTTP request will do.
For example you might have curl  on your command line if running Linux or Mac. Or you could install GUI clients such as Postman or Paw for making these requests. (Or provided you only need to make a GET request, simply use your web browser.)
Just start your Krakend gateway locally and use one of the above to fire a request to whatever URL you have the gateway running on e.g. localhost:8080/some-path or similar.

Thanks for your response.
Thanks
Nagaraj


React 0.16 Support

With React 0.16, it throws an error because PropTypes is no longer included with the main React package. It has a separate package now.

Fixed by #41


FutureWarning: transformers.deepspeed module is deprecated and will be removed in a future version. Please import deepspeed modules direct ly from transformers.integrations

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
Should update deepspeed import
✔ Solution

❓ Alternatives
No response
📝 Additional Context
No response
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

This should not be from axolotl as we don't import this.


Add devops4mecode's file

created a branch, created a file and made a commit, and opened a pull request

Add devops4mecode's file

created a branch, created a file and made a commit, and opened a pull request

created a branch, created a file and made a commit, and opened a pull request


Adding Dynamic Multiphysics simulation of RIA in the GCMR Assembly

Adding a new simulation in the Gas-cooled Microreactor (GCMR) assembly of a postulated reactivity insertion accident (RIA). Similarly, to the previous simulation of flow blockage, a 3-D dynamic Multiphysics simulation was completed by coupling Griffin, BISON, SAM, where the Griffin transport solver DFEM-SN with CMFD acceleration was used.

Closing as addressed by #230


Support for TGE integration

Is your feature request related to a problem? Please describe.
Since 24.08.2024 dynamic electricity prices became available for residential usage in Poland.
All currently-available electricity providers use TGE (Towarowa Giełda Energii) Fixing 1 as reference point.
ENTSO-E, RCE and Nordpool are indexes that display values close to TGE, but not exactly TGE.
That's why TGE custom integration was created: https://github.com/PiotrMachowski/Home-Assistant-custom-components-TGE
This integration provides identical attributes format as ENTSO-E integration so little to no changes in your codebase.
Describe the solution you'd like
EV Smart Charging ConfigFlow suggests TGE Fixing1 sensor if present in Home Assistant instance

This Issue is addressed by PR #335

Pull Request is ready to be merged.
I would be glad if quick merge is possible as I want to popularize EV Smart Charging along Polish Citizens and TGE is only source of truth in terms of electricity price in Poland.


502 Bad Gateway

Hi! When clicking on Open User Interface (Benutzeroberflääche Öffnen) I get a 502 Bad Gateway.
Any ideas?

could you pls post a picture of the config tab?

What for a system?
Which HA installation?
https?
did you try: IP-of-HA-installato:7070 like homeassistant:7070 or 123.456.789.123:7070 ??

Most current HA version, on Docker. I did try up:7070, but it does not reply.

 ports:
      - "7070:7070"
      - "7070:7070/udp"

in docker config for ha?

It is set as —net=host to serve all ports to the container.

ha with https?

Nope.

I have it working, was a config error with evcc. Thanks for your help!

@chheiss can you please explain what was wrong with your config? I'm experiencing the same error with the link in the HA sidebar.


collapsedAttributedLink often disappears upon device rotation

Tested straight from Demo project in iPhone 6 Simulator.
Here's portrait, working fine:

Here's landscape, "... more" disappeared:

should be fixed by #33


[Task]: Restore point for Auto Layout Conversion BE

Terminology:
fixed apps - present day app where widgets are absolutely positioned
auto layout apps - apps that hold widgets in order
As part of Auto layout initiatives, we will be building a way to convert out present day fixed apps to auto layout apps. This conversion will be lossy because of the nature of auto layout.
So in order to make up for that, we will be providing a way to restore to the last point in time where the app was in fixed mode.
Requirements of the BE:

an api to store app data as a snapshot.
an api to use the snapshot to restore the app.
this information should hold even if the app is deployed.
an api to update layouts of all pages at once during app conversion.
add snapshot information as part of application view api.
all apis should with git sync without any issues.

@momcilo-appsmith @rahulramesha the snapshot should only be of the layout(dsl) or all entities like jsobjects, queries, datasources as well?
pls confirm.

Yes, I confirm we will only revert the layout.


Exercise 40's explanation on mutable/variable pointers could be improved

Exercise 40 states:
You can always make a constant pointer to a variable, but you cannot
make a variable pointer to a constant. This sounds like a logic puzzle,
but it just means that once data is declared immutable, you can't
coerce it to a mutable type. It's a safety thing (to prevent mistakes).

It seemed very confusing to me as I first read "You can always coerce a constant pointer to a variable pointer" when what I think was I was meant to understand was: "You can always make a constant pointer reference a variable but not a variable pointer that references a constant value".
It might just be a problem of comprehension (or just me being picky) on my side but I think the explanation could benefit a bit more clarity!

@sawcce You're right. The terminology is off. I checked with the most recent official Zig documentation, and const values are referred to as "const variables".  Obviously that's a bit of an oxymoron, but there you have it.
Instead of "constant" and "variable," to differentiate, we should probably use the terms "mutable" and "immutable".
Maybe:
You can always make a const pointer to a mutable value (var), but
you cannot make a var pointer to an immutable value (const).
This sounds like a logic puzzle, but it just means that once data
is declared immutable, you can't coerce it to a mutable type.
Think of mutable data as being volatile or even dangerous. Zig
always lets you be "more safe" and never "less safe."

What do you think?

@sawcce Thanks for this hint! :smile:


Update root readme

@oliverschuerch, please provide some more information on what exactly needs to be updated.

Update package descriptions for components
Remove beta notice

Update package descriptions for components
Remove beta notice


Search Module "files to include" is case sensitive. Request for case insensitive, or a toggle.

On the search module, the "files to include" portionis case sensitive on the files name
for example I was trying to find something in any of my test files, so I was using
test.* as the search criteria, but I got no results. I had to use Test.* instead
This is contrary to Windows behaviour where files are case insensitive.
for example if I do a dir *Test*.*  vs  dir *test*.* they both return the same results.
I understand that unix is not like this, so can you include a button/setting to turn this on and off, similar to the one on the text search box?

https://github.com/microsoft/vscode/issues/10633


user installation problem

Hi, I really like your project and wanted to dive deeper into it. The installation went well.
Vue - ok
Laravbel - ok
MySQL -ok
I installed default scheme also for CRM.
BUT, how should I set up the users for the first use? I can't login.
Thank you!
Szabolcs

Ohhhh.. I forgot to run the seeders. Sorry. :)

I tried, but the seeds are not working.

Hi, I am very sorry that I didn't answer for so long but I didn't notice your issue.
php artisan db:seed --class=InitSeeder doesn't work?

Hi @szczepanmasny
To me the seeder doesn't work. :(

Did the previous steps succeed?
Have you created database and configured your .env file and run migrations?
After migrations do you have correct table schema in your SQL database?

hi @szczepanmasny
yes i did that and when i try:
php artisan db:seed
or
php artisan db:seed --class=InitSeeder
i get this:
  [ReflectionException]
  Class InitSeeder does not exist

Hi,
Firstly type git pull to get latest api version. Then type composer dumpautoload and then php artisan db:seed --class=InitSeeder
Please let me know if it helps you.

Dear @szczepanmasny
Many thanks for your Feedback.
I did the steps from your previous comment, now i get the following errors. (see screenshot)

You use the crm template, right? This seeder has been updated yesterday, and this error has been fixed in last PR. So if you checkout on master branch and do git pull, it should help.
Sorry for these problems.

Yes i use crm template. I did checkout git pull and it didn't work, unfortunately.

Here i pulled the last commit.

I realized what the problem was. Seeders are placed in templates directory and php artisan load-template should be used only one to initialize project. Then seeders are copied from template to your app. So if you want to have actual seeders in your app you must overwrite them by typing php artisan load-template once again. And then run seeders.
I guess you are already discouraged, but if it was otherwise, it would probably help you

dear @szczepanmasny
That works! Thank you very much.
I also realized that seeders are included in the template directory, but didn't knew that i have to do load-template again, because i already did once. Anyway, it works now and thank you very much!
And no, i am not discouraged. Many thanks for your work, will work with your script!


botbrew basil keep giving me this message

I failed to deploy the package

Any solution? I have same message!


Update Nuget Package Failed because URL to an interal Nuget Feed contains a Space

🐛 Bug Report
We are happy that we found this tool. Our system configuration is special, so we encountered a rare error.
Our internal Nuget-Feed Link (e.g. http:\tfs.local\Our%20Solution_packaging/our-Feed/nuget/v3/index.json) contains a space.
In line 36 (DotNetUpdatePackageCommand.cs) NuKeeper cut the Url in two parts (e.g. http:\tfs.local\Our Solution_packaging/our-Feed/nuget/v3/index.json) while executing "dotnet add" command.
here is the part of the log (--verbosity D) before the exception is thrown:
Nuget restore on local.project.TestConsole.csproj complete In path C:\tfs\OurSolution\local.project\Development\local.project.TestConsole, running command: dotnet restore local.project.TestConsole.csproj -s http://tfs.local/Our%20Solution/_packaging/our-Feed/nuget/v3/index.json In path C:\tfs\OurSolution\local.project\Development\local.project.TestConsole, running command: dotnet remove local.project.TestConsole.csproj package local.package In path C:\tfs\OurSolution\local.project\Development\local.project.TestConsole, running command: dotnet add local.project.TestConsole.csproj package local.package -v <IP_ADDRESS> -s http://tfs.local/Our Solution/_packaging/our-Feed/nuget/v3/index.json
Expected behavior
To add in line 36 masked quotes for parameter "-s"
e.g. dotnet add local.project.TestConsole.csproj package local.package -v <IP_ADDRESS> -s "http://tfs.local/Our Solution/_packaging/our-Feed/nuget/v3/index.json"

What's interesting is that the dotnet restore part escapes the url. I assume that the %20 escaping there is  correct and can be applied to the dotnet add part as well?

Should be fixed in latest release, can you verify?


[Feature Request] Allow customized game launch path

This would make it possible to directly launch custom launchers like the one for elden ring seamless coop mod. https://www.nexusmods.com/eldenring/mods/510?tab=description

ModEngine needs to be in control of creating the game process for injection to work, so making this work is tricky. However, Seamless Coop only requires an external DLL to be loaded. You can do that by editing your modengine configuration like this:
[modengine]
external_dll_discovery = true
external_dlls = ["elden_ring_seamless_coop.dll"]

Leaving this open as a tracker for UX of using external launchers.

Nice! So where should I put the dll to, do I need to specify a path to it?

Yeah man I need to get the mod launcher to kick off ER but with it's filename being "eurotrucker2.exe" +1 to this feature request, sounds like a good addition and could be useful to multiple users. Or just 1