ARROW-755: [GLib] Add garrow_array_get_value_type()

garrow_array_get_data_type() is renamed to
garrow_array_get_value_data_type() for consistency.

Oops. I've fixed.

thanks!


Added horizontal scrolling to change spaces, in both directions.

This is great to use the magic mouse/trackpad "Swipe between full-screep apps" feature, with a standard mouse with horizontal scroll buttons/wheel.

Thank you!
You can confirm this change in Karabiner-10.18.5. (the latest beta version)
How to upgrade to the latest beta version:
https://pqrs.org/osx/karabiner/document.html#check-for-updates


make travis work and check all tests for every commit

I don't know travis, so if there is someone willing to help, please do so :)

Fixed by pull requests:
https://github.com/hanickadot/compile-time-regular-expressions/pull/7
https://github.com/hanickadot/compile-time-regular-expressions/pull/8
Thank you @0xdead4ead


Finish the Landing page

add the Quest Board section
make reusable components
add some redirection
fix some responsive problems

gj, the landing page concept work pretty well


Revert Options,

I merged a PR to add a command line option parser, but it was broken so I'm just reverting for now so that the code works. Also fixes my package script.

LGTM


MonadFix instance is needlessly strict

As explained on http://stackoverflow.com/a/34287498/477476 , RevState's MonadFix instance is too strict and is practically unusable as a result. The fix is a trivial matter of using an irrefutable pattern in the instance definition; i.e. going from
instance MonadFix m => MonadFix (StateT s m) where
  mfix f = StateT $ \s -> mfix (\(x, _) -> runStateT (f x) s)

to
instance MonadFix m => MonadFix (StateT s m) where
  mfix f = StateT $ \s -> mfix (\ ~(x, _) -> runStateT (f x) s)

Thanks for the bugfix! I've whipped up a little test suite based on the example in that SO question.


compose: Show avatars for people in typeahead autocompletes.

This PR adds avatars in the typeahead autocompletes next to users. Users without avatars (e.g. @all and @everyone) have an empty placeholder instead so that they line up with the other names.
The typeahead-image class is similar to the emoji class also used by typeahead, but with a few differences in size, positioning, and its border-radius.
Fixes #6635.
Screenshot:

@skunkmb I think we need something to go beside @all in this design; maybe try using a megaphone icon for the purpose?  Also, the vertical positioning looks a bit off; might be a screenshot artifact; can you check it?

@timabbott OK, I made a megaphone icon and added it next to @all, so now it looks like this:

As for the pictures being centered, I checked and they are. There are 4px above and 4px below, and 23px to the left and 23px to the right.
Please let me know if there is more I should fix. Thanks!

For the megaphone icon, check if there's something already in font awesome we can use?  That has perf advantages over making a new one.

(I think you can also probably squash the commits).

@timabbott OK, I used a FontAwesome megaphone icon:

I also squashed the commits. Please let me know if there is more to change. Thanks!

Can you fix the linter issue?  Also, what's the story with messages.swift?

@timabbott OK, the linter issue should be fixed.
As for messages.swift, I added that by accident! I've been tinkering around with making a Zulip library in Swift similar to the zulip-js library in my free time, because Swift is a language that I enjoy using and have worked in before. That was one of the test files. I can show it to you when I'm done if you're interested. I've removed it from this PR, though. Sorry about any confusion.
Please let me know if there is more to change or fix. Thanks!

Looks great!  I'm test-deploying on chat.zulip.org.
I'm not happy with the people.js refactor commit, and split it into 5 commits (with a slightly different result) in what I'm test-deploying on chat.zulip.org.

@skunkmb I merged #8985; you can rebase this on top of that now :)

OK, given the feedback, I went ahead and merged this.  Thanks @skunkmb!
I opened https://github.com/zulip/zulip/issues/9041 for a follow-up to this.


tools/rename.bat uses 'mv' and "move"

The file "tools/rename.bat" uses both "mv" (if available) and the native Windows "move" command.
To use this file on UNIX-like systems, you need to make it executable.

I am surprised. Does this make it work on Windows as on Linux?

Well, it should, in theory.
Tested WinNT 6.1, Ubuntu 22, FreeBSD 14
But it's better to check somewhere else. I just don't have anything else at hand.
It's just that the original required an "mv", and I might not have one.
And the documentation implies that the Vim installer can be made on Unix-like systems, so that one tool can be used.

I think Bram used to use this on Windows with Unix-like tools before running NSIS.
Currently, our CI creates NSIS installers without using this file.

okay thanks.

I think Bram used to use this on Windows with Unix-like tools before running NSIS.

Well, yes, its use before creating the installer is directly described in the documentation. But due to the use of facilities in this tool that are not originally available in MSFT Windows, it is unfortunately not widely used.

Currently, our CI creates NSIS installers without using this file.

Of course, you can do without this tool. And not only in CI (Continuous Integration), but also in "home conditions" when you create an installer for yourself and your friends.
Ultimately, whether this tool is needed or not is up to you to decide. But if it is described in the documentation and is primarily focused on the Windows platform, then I tried to make it "friendly" to this platform and, if possible, universal.


Collection filtering on identifiers gives empty result

Hi,
thank you for this awesome library :)
I just noticed that:
const context = {
  items: [
    {id: 123, title: 'Hello world'},
    {id: 456, title: 'Good bye world'}
  ],
  my: {
    itemId: 123
  }
}

// The following will resolve with an empty array:
jexl.eval('items[.id == my.itemId]', context).then(console.log)

// While this will give the correct subset:
jexl.eval('items[.id == (my.itemId)]', context).then(console.log)

Oh my! That's an alarming bug indeed... thanks for reporting! I'll look into this.


Deprecate order token related queries and fields

Deprecate order.token field. orderByToken query was already deprecated.
⚠
Contains changes from #9281

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-5849 Migrate order IDs to UUID

Task linked: SALEOR-6140 Deprecate order token related queries and fields


Why plan on targeting Windows 8 instead of 10 as per README?

I understand how far back this started, but is it really necessary to have Windows 8 on the roadmap

We don't plan on targeting Windows 8 specifically, it just hasn't been updated yet.

I'll make a PR later


Fix Reward Calculation in example/2022-12-10-textrl-elon-musk.ipynb

In the notebook example/2022-12-10-textrl-elon-musk.ipynb, the reward calculation in the MyRLEnv class should be updated for correct scoring. Specifically, the function get_reward needs modification.
Current Code:
class MyRLEnv(TextRLEnv):
    def get_reward(self, input_item, predicted_list, finish):
        reward = 0
        if finish or len(predicted_list) >= self.env_max_length:
            predicted_text = tokenizer.convert_tokens_to_string(predicted_list[0])
            # sentiment classifier
            reward = sentiment(input_item[0] + predicted_text)[0][0]['score'] * 10
        return reward

The current code concatenates input_item[0] with the predicted text to calculate the sentiment score. However, input_item should be referenced differently to ensure proper reward calculation.
reward = sentiment(input_item['input'] + predicted_text)[0][0]['score'] * 10

feel free to submit a PR to fix this issue~


When using Basic Auth, the request are not cached

If Basic Auth and Cache are both defined, requests will not be cached...
Would this be the intended behavior? Should we enable caching when Basic Auth is defined?

Chris: the auth upstream shouldn't be cached, but the app upstream should be cached

Needs more investigation.
Some scenarios may cause security issues.


Link to Microsoft Trust Center is 404

"The [Microsoft Trust Center] has what you need to understand what we do to secure the Microsoft Cloud." - the link goes to a dead page

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

ID: 7f71c6e6-2454-db53-9cc0-ecc59a7c1cb0
Version Independent ID: 46e9be8a-473c-5c0d-9353-007e25ec0706
Content: Microsoft Trust Center and Azure security information
Content Source: articles/security/security-microsoft-trust-center.md
Service: security
GitHub Login: @TomShinder
Microsoft Alias: terrylan

@kweeket Thanks for taking the time to report this. The link is working for me as expected.
Can you confirm if the link is being translated to this "https://www.microsoft.com/en-us/trust-center"?

Yes, the link goes to https://www.microsoft.com/en-us/trust-center

@kweeket The page is working as expected for me and I checked with a couple of colleagues as well. I believe this must have been a transient issue. I will go ahead and close this issue for now.
If you are still having the same issue, please tag me in the comments and I will get back to you.


Update Day23.md

There was a typo on line 80 where "name" should be corrected to "firstname" as in: ?firstname=Jack&surname=Beanstalk
(The original was ?name=Jack&surname=Beanstalk)

Huge thanks🥳


About the accuracy of adversarial examples

I download the two 'secret model'  from the web url in fetch_model.py,  and load the model weights.  When I use the adversarial examples generated from my own method,  I found the test accuracy of the naturally_trained model is even better than the accuracy of adv_trained model.  I don't know why that happens, can you give some explanation ?

This can happen if you don't construct strong enough adversarial examples. I would recommend comparing your attack to one of the strong baselines already included in the repo.


Update PPO algorithm to use GAE

The PPO algorithm uses GAE (Generalized advantage estimation) to estimate the advantage of an action.  Our current implementation uses basic temporal difference objective.  Switching to GAE will potentially make our learning less reliant on the critic for determining good and bad actions.

Done. It works better.


Azure Data Lake Storage doesn't have a valid alternative

Can you provide valid alternative for azure data lake storage in aws environment?

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

ID: 30d02f16-9621-d204-4132-8ce8db79b0af
Version Independent ID: 65b4385b-79f0-d842-d4a1-5d013615f9b1
Content: AWS to Azure services comparison - Azure Architecture Center
Content Source: docs/aws-professional/services.md
Service: architecture-center
Sub-service: cloud-fundamentals
GitHub Login: @JasRobe
Microsoft Alias: pnp

Thanks for the feedback! We are currently investigating and will update you shortly.

We're currently working on this doc and have added it to the list of things to update. We hope to have changes out soon. Stay tuned!


Update of Starter Guide to 4.11

In Progress

hey @cedricclyburn this PR is now superseded by #50 which contains all these changes of yours. as agreed I commented them out for now until the Atlas stuff is fixed. Closing this one now.


feat: add List Component

Decathlon.design link

https://www.decathlon.design/726f8c765/p/078570-list-beta/b/9115e7
Material.io link

https://material.io/components/lists

We won't develop a template for list components for now


Add a new Cask - gInbox

Add a new Cask

gInbox 0.2.3c

Thank you. Could you please squash your commits?

Thank you. Could you please squash your commits?

It seems that something went wrong here. Do you need any help?

Thank you for the contribution. It needed some fixes, so they were made in f24b139d13f9a63cc5d960c300a1f761affe23c9. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


In-Memory Cache

Why make this change?

Second set of work attributed to #1618 #1617

What is this change?

Implements the design in #1801. The one behavior which isn't in this PR is caching GraphQL "list" queries. That will come in a separate commit once i resolve the errors that arise with generics and the DataReaderHandler delegates.
Creates a DabCacheService which is used in database queries (REST/GraphQL) such that requests and responses are cached in memory.
This PR is not dependent on #1865. However, once that PR and this PR are merged, a third follow-up PR will make a small adjustment in SqlQueryEngine.ExecuteAsync(...) such that the cacheEntryTtl and cacheEnabled values are resolved from runtime config and used determine whether to use the cache and how to use the cache.

How was this tested?

[x] Integration Tests -> integration tests included which mock the QueryExecutor and test how the DabCacheService handles request metadata and caching method execution. The integration tests aim to test the behavior of the DabCacheService and intentionally do not make private methods public or break out key generation and entry size calculation into separate classes. Additionally, the integration tests are not meant to test the functionality of FusionCache. That library has its own tests which validate functionality.

Sample Request(s)
Until all related PRs are merged, this feature is gated by a feature flag. To add the feature flag, add the following snippet to the AppConfig.json:
// Define feature flags in config file
    "FeatureManagement": {
        "CachingPreview": true, // Turn-On feature
    }

Point requests will be cached with a default 10 second ttl. You can try out whether response times are quicker for your use case.

/azp run

/azp run

/azp run


docs(orderBy): describe your change...

Changed the second example to demonstrate typical sorting behavior (like Finder and Win Explorer) where the first click on a column sorts alpha and the second click on an already sorted column sorts in reverse alpha.  Also shows a simple sort indicator automated using ngShow.

I signed the CLA

I left a couple of minor (style) comments, but in general I like this.

Personally I would rather have this logic in the controller.

I didn't rhink there would be a controller for such simple example, but since there is one already, I too agree it is a better practice to have it on the controller.

How about this:

Sorting predicate = {{predicate}}; reverse = {{reverse}}

  [ unsorted ]

        Name

        Phone Number

        Age

      {{friend.name}}
      {{friend.phone}}
      {{friend.age}}

I like it much better. I wonder if it would make sense to go one step further and ngRepeat over the  (instead of having 3 hard-coded elements with almost identical content).
Not sure if it make the example too complex to focus on the orderByFilter functionality, though.

Either way, but I also wonder if that may complicate the example.  Also, if we did this two this example then it should probably also be done to the other examples that are on this same doc page.  I am happy to make the changes and update the pull request as you see fit.

I'll leave it to @petebacondarwin's judgement :)

I think that @rjenks latest idea is good enough. Do you want to update the PR @rjenks? Then do you want to merge it @gkalpak? Thanks guys

@rjenks, could you squash the commits into one (with an appropriate commit message) ?

@gkalpak - at this stage it is probably simplest to squash and tweak the commit message ourselves when merging. Generally, even if the @rjenks squashes, etc, we will still have to rebase again onto master.

@petebacondarwin, OK, will squash upon merging then :)

Thanks!


A-Frame AR change webcam viewport size (when i tried to change heigth and width, goes back)

I used jquery to try and size the video component that it generated from the a-scene, and it changes the value from the component, but i think that ar.js is triggering on resize event to ajust back to the window size.
I've tried to change the arjs attribute but nothing happens.
Is there a way to set a fixed size, instead of having it full screen/window sized? Or is this a bug from the framework?
I have reported this also to:
a-frame
And redirected to this git:

This is a question more appropriate for the arjs maintainers

A-Frame Version: 0.7.1 and 1.0.4
Platform / Device: Windows 10

let videoComponent = $("video");
videoComponent.height(300);
videoComponent.width(300);

<a-scene embedded arjs='sourceType: webcam; sourceWidth: 600; sourceHeight: 400; displayWidth: 600; displayHeight: 400; canvasWidth: 600; canvasHeight: 400; debugUIEnabled: false;' >

It is an open bug. On the file system-arjs.js

        //////////////////////////////////////////////////////////////////////////////
        //		Code Separator
        //////////////////////////////////////////////////////////////////////////////
        // TODO this is crappy - code an exponential backoff - max 1 seconds
        // KLUDGE: kludge to write a 'resize' event
        var startedAt = Date.now()
        var timerId = setInterval(function () {
            if (Date.now() - startedAt > 10000 * 1000) {
                clearInterval(timerId)
                return
            }
            // onResize()
            window.dispatchEvent(new Event('resize'));
        }, 1000 / 30)

This snippet is a bad trick that triggers a resize event multiple times per seconds. It has to be removed. I tried long time ago to do that, listening to the browser 'resize' event but it is not enough, for some reason.

related: https://github.com/jeromeetienne/AR.js/issues/678

moved this issue in new AR.js repository: https://github.com/AR-js-org/AR.js/


doc: coding_guidelines: replace a whitelist replacement

Nothing in the tree is using the "passlist" term in the "blocklist /
passlist" pair. However, west is using "blocklist / allowlist" already in a
released version.
Recommend blocklist/allowlist instead of blocklist/passlist to avoid
having to add yet another possible replacement, since nothing is using
passlist right now.
Note that blocklist/allowlist is in widespread use throughout many
projects, including Chrome.
Signed-off-by: Martí Bolívar<EMAIL_ADDRESS>

Another reason to prefer allowlist to passlist is that allowlist has the same number of characters as whitelist, so it doesn't affect alignment if you do a big search and replace.

Since I started this firestorm in https://github.com/zephyrproject-rtos/zephyr/pull/29498#pullrequestreview-608625019 I'm just going to respond to https://github.com/zephyrproject-rtos/zephyr/pull/29498#issuecomment-796395830 here then move on:

Zephyr doesn't use blocklist.

That was not intended as a statement of policy; it was an insufficiently clear statement of fact: the term blocklist appears in Zephyr mainline only in the west guide documentation and when paired with "passlist".  It has not yet been used in Zephyr RTOS code.

You're saying "Zephyr" but I disagree with whoever in the community ruled this out, and the fact that west is already using these terms de facto means they're going to be used by Zephyr users. So I stand firm on keeping them.

That's fine.  My position was we had three perfectly adequate names, where the positive and negative verbs are natural antonyms:  "allow/deny", "pass/block", and "accept/reject".
I don't find "allow/block" to be a natural pairing.  I also am tired of all the bikeshed discussions, and figured that since for this case we actually had a documented set of recommended names it was reasonable to propose sticking with them rather than introducing yet another arguably gratuitous variation.
I don't like changing the list to replace pass/block with allow/block, but if that's how people want to go it's fine.  But I don't feel any professional or moral obligation to approve such a change.

I don't find "allow/block" to be a natural pairing. I also am tired of all the bikeshed discussions, and figured that since for this case we actually had a documented set of recommended names it was reasonable to propose sticking with them rather than introducing yet another arguably gratuitous variation.

Given that nobody is using 'passlist', I think that is rather precisely what this PR is doing.


[Bug] Not working on iOS

I run your example on iOS but can't paste image

Sorry my bad, it still work on ios but textfield do not support paste image


Add missing <limits.h> so PATH_MAX would be available

It fixes an error that happens during building on Alpine Linux:
In file included from /tmp/td/tdutils/td/utils/port/path.cpp:7:0:
/tmp/td/tdutils/td/utils/port/path.h: In function 'td::Status td::walk_path(td::CSlice, Func&&)':
/tmp/td/tdutils/td/utils/port/path.h:167:21: error: 'PATH_MAX' was not declared in this scope
   curr_path.reserve(PATH_MAX + 10);
                     ^~~~~~~~

https://github.com/isopen/docker/blob/master/tdlib/Dockerfile.alpine
https://github.com/isopen/docker/issues/1

Thanks, @kossnocorp.

@levlam 👌

Any chance to get it merged?

It will be merged to next release version.

It will be merged to next release version.

Amazing, thanks!

@levlam dynamic update is better
Regards

@levlam dynamic updates are better https://github.com/torvalds/linux/commits/master

This is a warning for @levlam. To avoid changing the standard
@kossnocorp Thank you for understanding :)

@kossnocorp How did you get the error of the current report?
I can not catch it through assembly @isopen. Write the assembly specifications please.

FROM node:9.11-alpine

RUN apk update && apk add python g++ cmake make bash git gperf openssl-dev zlib && rm -rf /var/cache/apk/*

RUN mkdir -p /tmp && git clone -b path-max-fix https://github.com/tdlib/td.git /tmp/td && mkdir -p /tmp/td/build
WORKDIR /tmp/td/build
RUN cmake -DCMAKE_BUILD_TYPE=Release .. && cmake --build . && make install && rm -rf /tmp/td

# ...


WindowsMachineFileCopy Doc States MachineNames are optional, yet pipeline fails with Required error

The Doc states:

Yet when run:

The error is produced:

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

ID: fd94385e-60ae-d9ca-be76-ee7f97f3dfde
Version Independent ID: 03eeae39-e3be-b10f-0ee5-9b8a312aae98
Content: Windows Machine File Copy task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/windows-machine-file-copy.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

@RoopeshNair -- Roopesh, please review this update in private PR 4810.

@cdodrzywolski Thanks for reporting this. This issue is fixed in our new task reference - this task input is marked as required,
https://learn.microsoft.com/en-us/azure/devops/pipelines/tasks/reference/windows-machine-file-copy-v2?view=azure-pipelines
Please check it out and let us know what you think. Thank you.

You're very welcome, thanks for updating!


transport/smtp: Derive Clone impl for AsyncSmtpTransport and AsyncSmtpClient

Maybe I'm missing something, but I don't see why we can't derive these implementations automatically. Well, except for https://github.com/lettre/lettre/pull/995, but that's a different conversation/PR 😂

Hey! Thanks for the contribution. My thinking was that since we don't really need to store the executor inside (Async)SmtpTransport, Clone could be derived without discriminating on E in order not to pollute the trait bounds for the Clone impl.


[SPARK-16340][SQL] Support column arguments for regexp_replace Dataset operation

What changes were proposed in this pull request?
Currently, regexp_replace function supports Column arguments in a query. This PR supports that in a Dataset operation, too.
How was this patch tested?
Pass the Jenkins tests with a updated testcase.

Test build #61800 has started for PR 14060 at commit 0c854b5.

Test build #61800 has finished for PR 14060 at commit 0c854b5.

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

Merged build finished. Test PASSed.

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

@dongjoon-hyun : Thanks for taking the case. It looks good.

Merging in master.

It's my pleasure. Thank you for reviewing, @MukulSoul and @rxin .


Disable first/prev page instead of hiding

Hi,
I like this simple component, but I'd like to improve the UX by disabling first/prev pagination element instead of hiding it. By doing this, the pagination will not 'jump' and it has more or less a fix width.
This could be of course a configurable thing.
Let me know what you think. I could prepare a MR the next days.
Best,
Raoul

Hi, thank you for your issue. I agree that paginator have UX issue that you described. Feel free to create PR.

Would it be ok to trash karma and use mocha/jsdom/enzyme instead?

I wanted to make this change too, just forgot about that haha
@raoulus if you want help with anything let me know 😃

@raoulus it would be great

Please review my PR https://github.com/vayser/react-js-pagination/pull/13
Sorry for the amount of changes, but I updated the dependencies and I also removed some unnecessary code.
@vayser You could setup travis-ci integration. Config is there.

@raoulus thank you for PR, apologize for late response. I will review all changes and merge with travis CI soon

Hi @vayser thank you for this excellent lib, it really save my day.

It changes the way parameters input for custom element (nextPage...etc) and make it overridable.
Thank you!


Allow embedded @responsive directive

Hi,
@responsive directive is just awesome, i could use tailwindcss only for that particular feature !
But it could be even more powerful if it was possible to use @responsive inside @responsive.
I will give you a particular example if you want.
Thanks.

Yes an example would be great! 😄

I was so deep into it, I think I didn't know what I needed exactly !
But I'm glad I asked because otherwise I wouldn't have seen your gif !  I'm still laughing looking at it !
I've got 2 sets of responsive utility classes for my popover component. One for direction and another one for alignment. I needed to override a specific property when a combination of those classes where use together.
@responsive {.popover-top {...}}
@responsive {.popover-left {...}}
@responsive {.popover-right {...}}
@responsive {.popover-bottom  {...}}

@responsive {.popover-align-top {...}}
@responsive {.popover-align-left {...}}
@responsive {.popover-align-right {...}}
@responsive {.popover-align-bottom  {...}}

@responsive {
  .popover-top {
    bottom: 100%;
    left: 50%;
    transform: translateX(-50%) translateY(20px) scale(.8);
    transform-origin: bottom;
    @apply .m-0 .mb-4;

    &.is-visible {
      transform: translateX(-50%) translateY(0) scale(1);
    }

    &:after {
      bottom: -8px;
      left: 50%;
      box-shadow: 1px 1px 0 0 rgba(0, 0, 0, 0.08);
    }
  }
}

@responsive {
  .popover-align-left {
    left: 0;
    right: auto;

    &.popover-top {
      transform: translateX(0) translateY(20px) scale(.8);
      transform-origin: left bottom;
    }

    &.popover-bottom {
      transform: translateX(0) translateY(-20px) scale(.8);
      transform-origin: left top;
    }

    &:after {
      left: 20px;
      right: auto;
    }
  }
}

I didn't want to write each possible combination, so using sass nesting I wanted to do something like that
@responsive {
  .popover-align-right {
    right: 0;
    left: auto;

  @responsive {
    &.popover-top {
      transform: translateX(0) translateY(20px) scale(.8);
      transform-origin: right bottom;
    }
  }

  @responsive {
    &.popover-bottom {
      transform: translateX(0) translateY(-20px) scale(.8);
      transform-origin: right top;
    }
  }

    &:after {
      right: 14px;
      left: auto;
    }
  }
}

I ended up writing this which does what I needed
@responsive {
  .popover-align-left {
    left: 0;
    right: auto;

    &.popover-top,
    &.sm\:popover-top,
    &.md\:popover-top,
    &.lg\:popover-top,
    &.xl\:popover-top {
      transform: translateX(0) translateY(20px) scale(.8);
      transform-origin: left bottom;
    }

    &.popover-bottom,
    &.sm\:popover-bottom,
    &.md\:popover-bottom,
    &.lg\:popover-bottom,
    &.xl\:popover-bottom {
      transform: translateX(0) translateY(-20px) scale(.8);
      transform-origin: left top;
    }

    &:after {
      left: 20px;
      right: auto;
    }
  }
}

I know now it is not the purpose of the @responsive directive. I'm sorry if I wasn't entirely clear 😂.
It was very hard to stay focus writing this comment while seeing your gif ! 😂
Thanks a lot for your time and sorry for the inconvenience.

Ok so my brain has turned to mush going through all that but I THINK what we really would want to support here to solve your problem is applying responsive prefixes to compound class selectors.
So this:
@responsive {
  .foo.bar {
    color: blue;
  }
}

...would generate this:
.foo.bar {
  color: blue;
}

@media (min-width: 576px) {
  .sm\:foo.sm\:bar {
    color: blue;
  }
}

@media (min-width: 768px) {
  .md\:foo.md\:bar {
    color: blue;
  }
}

@media (min-width: 992px) {
  .lg\:foo.lg\:bar {
    color: blue;
  }
}

@media (min-width: 1200px) {
  .xl\:foo.xl\:bar {
    color: blue;
  }
}

I am not sure if we want to support this right now because it probably comes along with complexity that is hard for me to anticipate, but it's probably possible to do it 🤔
I also wonder if there's not another solution to your problem where you just have completely separate utilities for each combination instead of using compound selectors, ie:
.popover-top-left {
  left: 0;
  right: auto;
  transform: translateX(0) translateY(20px) scale(.8);
  transform-origin: left bottom;
}

.popover-bottom-left {
  left: 0;
  right: auto;
  transform: translateX(0) translateY(-20px) scale(.8);
  transform-origin: left top;
}

You could easily make those responsive 👍🏻

Thanks Adam, I fallowed your advise, looks much cleaner in my opinion.

Sounds like we've got this resolved. Closing this, but feel free continue the discussion if needed @lamberttraccard! 🤘


RUMM-2084: Delete API v1 related classes

What does this PR do?
Since we are using API v2 for a while already and it is deployed in all DCs, let's remove API v1 classes, they are not used anyway.
Review checklist (to be filled by reviewers)

[x] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[x] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[x] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #915 (69f6fc2) into develop (54498c8) will decrease coverage by 0.07%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #915      +/-   ##
===========================================
- Coverage    82.89%   82.81%   -0.07%
===========================================
  Files          267      263       -4
  Lines         9057     8967      -90
  Branches      1456     1448       -8
===========================================
- Hits          7507     7426      -81
+ Misses        1147     1141       -6
+ Partials       403      400       -3

Impacted Files
Coverage Δ

...og/android/rum/internal/net/RumOkHttpUploaderV2.kt
100.00% <ø> (ø)

.../android/core/internal/net/DataOkHttpUploaderV2.kt
98.77% <100.00%> (ø)

...android/log/internal/logger/TelemetryLogHandler.kt
75.00% <0.00%> (-25.00%)
:arrow_down:

...tadog/android/core/internal/net/CurlInterceptor.kt
80.95% <0.00%> (-2.38%)
:arrow_down:

...droid/rum/tracking/ActivityViewTrackingStrategy.kt
86.54% <0.00%> (-1.92%)
:arrow_down:

...rc/main/java/com/datadog/opentracing/DDTracer.java
55.65% <0.00%> (-0.42%)
:arrow_down:

.../src/main/java/com/datadog/opentracing/DDSpan.java
52.75% <0.00%> (+1.10%)
:arrow_up:

...android/rum/internal/ndk/DatadogNdkCrashHandler.kt
86.78% <0.00%> (+6.32%)
:arrow_up:


fix: add a mention to the possibility of accessing nested properties

It took me a fair bit of time to figure out it is possible to access nested properties when working with related fields. It is currently not mentioned in the docs it is possible to do so.
This issue brought me on the right track https://github.com/payloadcms/payload/issues/2150#issue-1592868070 - thanks @jpmadrigal07!
Type of change
This change is a documentation update only, the functionality works already.

Thank you @philipgher! Good add!


Request executed twice

Hello!
In my Nuxt app the request to /api/event is emitted twice.
The first request is targeted at my custom host defined in the apiHost property in nuxt.config.js (self-hosted Plausible instance). The second one is sent to https://plausible.io/api/event. Is this intentional?
This code is executed in my layout file when mounted:
    const { enableAutoPageviews } = Plausible()

    // This tracks the current page view and all future ones as well
    enableAutoPageviews()

Hey @gambolputty!
When you call Plausible() like in you snippet, a new instance of the plausible tracker is created with the default settings.
The event duplication is then caused by having two instances of the tracker, one with your apiHost, created by the Nuxt module and configured in nuxt.config.js. The second instance is initialized by the call in your layout file.
If you want to access the plausible tracker instance in any of you components, including layouts, I suggest to use the properties explained in the Usage section of the README.

My mistake. Thanks for clarifying!
Will close this issue now.

Hello!
I'm having the same issue in Nuxt.js too (with default SSR), except that I only use the nuxt.config.js configuration, I don't call it somewhere else :thinking:
modules: [
  'vue-plausible'
],

plausible: {
  apiHost: 'https://plausible.example.com'
},

Any idea ?
Thanks!


Controller - Tapping on code blocks disables styling options, moving the caret not

https://github.com/visual-space/visual-editor/assets/11160141/716fba97-28c4-4845-a28e-8fdcce6e4d59
#192 - The initial implementation

I think this might have been resolved by my recent refactor in #230. Please confirm @DariusMuscalu


Websocket handshake perf

Found a couple spots we could avoid allocating in the websocket handshake.
Would like @blowdart and @Tratcher to take a look to make sure I'm not doing something completely wrong here.
Before:

Method
Mean
Error
StdDev
Op/s
Gen 0
Allocated

CreateResponseKey
1,118.31 ns
20.799 ns
19.455 ns
894,207.5
0.1373
5360 B

IsRequestKeyValid
92.56 ns
1.761 ns
2.163 ns
10,803,358.7
0.0095
400 B

After:

Method
Mean
Error
StdDev
Op/s
Gen 0
Allocated

CreateResponseKey
966.29 ns
18.022 ns
15.976 ns
1,034,883.1
0.0763
3520 B

IsRequestKeyValid
54.61 ns
1.089 ns
1.878 ns
18,311,386.7
-
0 B

With caching SHA1:

Method
Mean
Error
StdDev
Op/s
Gen 0
Allocated

CreateResponseKey
497.52 ns
9.885 ns
10.576 ns
2,009,951.4
0.0153
800 B

New perf numbers in original comment :D

🆙 📅
Removed SHA1 caching to make the PR more enticing :D

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=389131&view=logs

This comment was made automatically. If there is a problem contact<EMAIL_ADDRESS>I've triaged the above build. I've created/commented on the following issue(s)
https://github.com/aspnet/AspNetCore-Internal/issues/3128
https://github.com/aspnet/AspNetCore-Internal/issues/3168
https://github.com/aspnet/AspNetCore-Internal/issues/2483

@BrennanConroy ping


Command 'npms' not found

I installed npms-cli with npm install npms-cli --global.
Then I ran npms -h.
Here is what I got:
Command 'npms' not found, did you mean:

  command 'pms' from deb pms
  command 'npm' from deb npm

Try: sudo apt install <deb name>

I do not think that this is a problem with npm because the globally installed packages ezpr and npx work just fine.
OS: Ubuntu 18.04.3 LTS

For some reason waiting a few days fixed this.
Sorry for any trouble this may have caused.


Added a TestFixture constructor for ITestOutputHelper.

Added a TestFixture constructor for ITestOutputHelper so that TestFixture can be used in a SpecFlow XUnit project.

I'm still trying to figure out how to write tests for this addition. I'm struggling with how to inject an ITestOutputHelper mock.

why can't you just do the following in your derived fixture?
public class MyFixture(IMessageSink s, ITestOutputHelper h) : TestFixture<Program>(s)
{
    protected override void ConfigureApp(IWebHostBuilder a)
    {
        a.ConfigureLogging(l => l.ClearProviders().AddXUnit(h));
    }
}

SpecFlow uses BoDi for dependency injection, the SpecFlow XUnit plugin doesn't have IMessageSink registered. So when I try to run a test with the Fixture injected, I get the following exception:
  Message:
System.AggregateException : One or more errors occurred. (Interface cannot be resolved: Xunit.Abstractions.IMessageSink) (The following constructor parameters did not have matching fixture data: TestProject_XUnitAssemblyFixture assemblyFixture)
---- BoDi.ObjectContainerException : Interface cannot be resolved: Xunit.Abstractions.IMessageSink
---- The following constructor parameters did not have matching fixture data: TestProject_XUnitAssemblyFixture assemblyFixture

  Stack Trace:
----- Inner Stack Trace #1 (BoDi.ObjectContainerException) -----
TestExecutionEngine.FireEvents(HookType hookType)
TestExecutionEngine.OnTestRunStart()
TestRunner.OnTestRunStart()
TestRunnerManager.FireTestRunStart()
TestRunnerManager.OnTestRunStart(Assembly testAssembly, IContainerBuilder containerBuilder)
StarPromiseDateRecalculator_Specs_XUnitAssemblyFixture.ctor() line 19
RuntimeType.CreateInstanceDefaultCtor(Boolean publicOnly, Boolean wrapExceptions)
----- Inner Stack Trace #2 (Xunit.Sdk.TestClassException) -----

The SpecFlow XUnit plugin does have the ITestOutputHelper registered and since the MartinCostello.Logging.XUnit package has a AddXUnit overload for ITestOutputHelper, I thought it would be easier and faster to add a TestFixture overload for ITestOutputHelper than adding IMessageSink to the SpecFlow XUnit plugin code generation.
I also thought this could help other users where they have ITestOutputHelper and not IMessageSink.

available in v<IP_ADDRESS>-beta

Thank you!

From: Dĵ ΝιΓΞΗΛψΚ @.>
Sent: Thursday, November 30, 2023 5:46:37 AM
To: FastEndpoints/FastEndpoints @.>
Cc: xxAtrain223 @.>; Author @.>
Subject: Re: [FastEndpoints/FastEndpoints] Added a TestFixture constructor for ITestOutputHelper. (PR #548)
available in v<IP_ADDRESS>-beta
—
Reply to this email directly, view it on GitHubhttps://github.com/FastEndpoints/FastEndpoints/pull/548#issuecomment-1833609645, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABWNSRVXXCNEIWZNIEIABSTYHBWZNAVCNFSM6AAAAAA76275MWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZTGYYDSNRUGU.
You are receiving this because you authored the thread.Message ID: @.***>


[Rust] Allow recreate index with the same name on the same column

Closes https://github.com/lancedb/lancedb/issues/100

Windows failure seems to be due to vcpkg outage and it is irrelevant,

nit: maybe there should be a flag? replace=True.
Rationale being that some callers may want behaviors like CREATE IF NOT EXISTS instead of replace be default. I guess this can be adding on the lancedb side.

nit: maybe there should be a flag? replace=True.
Rationale being that some callers may want behaviors like CREATE IF NOT EXISTS instead of replace be default. I guess this behavior can be added on the lancedb side.

Makes sense.


Remove typing hint from assert_pods_failed_or_pending

Typing hint failed when running with python 3.8

!verified

!cherry-pick v4.12


It's possible to make a rounded crop mask ?

It's possible to make a rounded crop mask ?

Create pull request.
https://github.com/lykhonis/image_crop/pull/59


BMI calculator updated as per requirement

Title and Issue number

Title BMI CALCULATOR
Issue No. : #159
Code Stack : HTML, CSS, JAVASCRIPT
Close #<159>

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] I have updated the Index.html file for my contribution

why is this modified? This is a React challenge. Why do we have JS here


Advanced Data Structure

A Binary Tree Data Structure is a hierarchical data structure in which each node has at most two children, referred to as the left child and the right child. It is commonly used in computer science for efficient storage and retrieval of data, with various operations such as insertion, deletion, and traversal.

A Binary Tree Data Structure is a hierarchical data structure in which each node has at most two children, referred to as the left child and the right child. It is commonly used in computer science for efficient storage and retrieval of data, with various operations such as insertion, deletion, and traversal.

A Binary Tree Data Structure is a hierarchical data structure in which each node has at most two children, referred to as the left child and the right child. It is commonly used in computer science for efficient storage and retrieval of data, with various operations such as insertion, deletion, and traversal.


Bundle warning about duplicated development dependencies when it should not

Hello.
First of all, thank you for all of your work on bundler! One of my daily tools!
I am working on a gem that can be used in both MRI and JRuby. Most of the dependencies are the same except for a few bits here and there. So I have the following (stripped down) setup:
https://gist.github.com/kaukas/7f91788682be1cfb32f2
When I run bundle install in either MRI or JRuby I get this warning about rake-the-development-dependency:
Your Gemfile lists the gem rake (>= 0) more than once.
You should probably keep only one of them.
While it's not a problem now, it could cause errors if you change the version of just one of them later.

Bundler seems to be happy and silent if I move rake to the runtime dependencies though.
It's just an annoyance but I have quite a few repeating development dependencies and I get the warning politely repeated for every one of them.
Of course, the real package names are not important here. add_development_dependency and multiple platforms declarations should trigger this any time.
Do you think that's a real bug or am I misconfiguring something?
Thank you!

@kaukas Since this is kind of an old issue, can you please check if the warning persists on 1.10.6 and if so, then share your Gemfile as well? Thanks!

Yes, I am still getting that same warning with bundler 1.10.6 and the original setup as specified in https://gist.github.com/kaukas/7f91788682be1cfb32f2.

related https://github.com/bundler/bundler/issues/2822

I'm not sure we can really fix this issue, I think my answer would be use the same gem name for both platform gems (in different gem files), and let bundler auto-select the proper platform

There has been no reply to this issue for a long time so i'm going to close this.


[BUG] Misleading exception message on dataframe types

Describe the bug
I'm using the library for the first time with a Pandas dataframe and ran into an exception that was misleading.
To Reproduce
Steps to reproduce the behavior:

Create a dataframe where the id column is of type int64  e.g. df['id'] = df.index + 1
Create the qrel like this:

qrels = Qrels.from_df(
    df=df,
    q_id_col="id",
    doc_id_col="best_document",
    score_col="score",
)

Observe this error:

[/usr/local/lib/python3.10/dist-packages/ranx/data_structures/qrels.py](https://localhost:8080/#) in from_df(df, q_id_col, doc_id_col, score_col)
    293         """
    294         assert (
--> 295             df[q_id_col].dtype == "O"
    296         ), "DataFrame scores column dtype must be `object` (string)"
    297         assert (

AssertionError: DataFrame scores column dtype must be `object` (string)

Expected behavior
The assertion message should point to the correct column, in this case, it is the ID column that is of the wrong type. From inspecting the code, the assertion message is wrong when the document ID column is of the wrong type as well.

Hi, sorry for that!
I probably copy-pasted or duplicated lines there.
I will fix it in the next release.

Fixed in v0.3.17.
Please, give ranx a star if you haven't yet.


[Get] trap for exotic object should not use assertion for DM

note: Oct 19th (F2F: @dherman and @caridy) Review of proposal
Proposal has one problem, the [Get] trap of the NS exotic object uses an assertion. @dherman found an example where this assertion will affect the runtime:
// a.js (ESM)
import b from "b";
import c from "c";
export default function something () {
   c(); // this hits the assertion
}

// b.js (ESM)
import c from "c";

// c.js (CJS)
module.exports = function c () {}

This is because c is in TDZ. The solution is to make that a dynamic error just like any other TDZ.

closed by https://github.com/caridy/proposal-dynamic-modules/commit/9dcc38f08bf1d328c41fd95936cf7c9775812952


New Networking checks for FTP, Telnet, SQL Server and Kafka

I have defined some new controls to check networking services:

7.134 [extra7134] Ensure no security groups allow ingress from <IP_ADDRESS>/0 or ::/0 to FTP ports 20 or 21 (Not Scored) (Not part of CIS benchmark)

7.135 [extra7135] Ensure no security groups allow ingress from <IP_ADDRESS>/0 or ::/0 to Kafka port 9092 (Not Scored) (Not part of CIS benchmark)

7.136 [extra7136] Ensure no security groups allow ingress from <IP_ADDRESS>/0 or ::/0 to Telnet port 23 (Not Scored) (Not part of CIS benchmark)

7.137 [extra7137] Ensure no security groups allow ingress from <IP_ADDRESS>/0 or ::/0 to Windows SQL Server ports 1433 or 1434 (Not Scored) (Not part of CIS benchmark)

Also I have included this controls in "extra" and "internet-exposed" groups.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thanks @jfagoagas, pretty helpful checks. I will merge them into branch 2.5 for testing.


docs(zh-cn): 33_the-push-to-hub-api-(pytorch).srt

docs(zh-cn): 33_the-push-to-hub-api-(pytorch).srt #390
ISSUE
Subtitles from  00:04:13,380 to 00:04:22,923 are not consistent with the YouTube Video(https://www.youtube.com/watch?v=Zh0FfmVrKX0), further comfirmation is needed.

BTW, the CC caption on YouTube(video ID: Zh0FfmVrKX0) isn't correct during 4:18 to 4:20

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

modified subtitle
timestamp: 00:04:21,113 --> 00:04:22,923
modification: of -->  or


An Admin can Log In with IDIR

Description:
As a Admin user I would like to use my goverment account (IDIR) to login into the MET application so that I can manage engagements within the app.
Acceptance Criteria
Users can login using IDIR

Hey team! Please add your planning poker estimate with ZenHub @saravanpa-aot @nikhila-aot

Hey team! Please add your planning poker estimate with ZenHub @tiagograf-AOT @djnunez-aot @jadmsaadaot

Do we have the IDIR configurations required for KeyCloak setup ?


Tone bug

Description
It seems some tones are too short and is not possible to hear them!!
Steps/Code to Reproduce
mytimer --second=2 --alarm --tone=4

Expected Behavior
Alarm
Actual Behavior
No alarm
Operating System
Windows 10
Python Version
Python 3.8
MyTimer Version
0.7

@sadrasabouri Hi
Would you please check this issue?
Do you have any solution?

I am testing it on Ubuntu and all voices are being played correctly. Although some of them are short, I can hear them clearly.

I am testing it on Ubuntu and all voices are being played correctly. Although some of them are short, I can hear them clearly.

The length of some of them is less than 1 sec!
I think it would be better if we duplicate them to about 1.5 - 2.5 sec.
What's your opinion?
Do you have any interest to handle this issue?

That's a good idea. Sure I'll start working on it ASAP.


Update the integration of the consensus/network layers

See the last commit, the commits before it are preparatory.

Note that GitHub sorts commits by date, which - in this case - does not match
their actual order.
These are preparatory commits:

Hash
Commit

4b80164e
BlockFetch: use headers for the current chain, not blocks

ed3ec370
UpdateLedger: add HeaderState and methods to obtain/advance it

7ae578fa
Switch to AnchoredFragment in OuroborosTag

2d7fe6ec
Switch to AnchoredFragment in the ChainDB

ff5b3619
New ChainSyncServer based on the ChainDB

4f8429e4
New ChainSyncClient

8829523a
Add BlockFetchClient

fab86fdb
Add BlockFetchServer

1ea7da82
Add codecBlockFetchId

e854ee8f
Use the ThreadRegistry for BlockchainTime

de1801a7
Add headerBlockSize to SimplePreHeader

This is the main commit that rewrites Ouroboros.Consensus.Node and updates
the demo:

Hash
Commit

36727156
Header/body split + storage layer integration

The following commits bring down the test-consensus run-time from ~10
minutes to ~1 minute:

Hash
Commit

c3f91fa0
AnchoredFragment: don't validate twice

fb44d224
ChainDB.Model: cache validated ledgers

04cdedf3
Cache the hash of the mock headers

5c243cf0
Add a fast variant of fromChain that doesn't validate

04eb69f7
Disable assertions in IOSim

d92c9b16
ChainDB.Model: more efficient chain construction

@dcoutts: something is going wrong here (maybe the invariant/assertion is not
right to begin with?):

Hash
Commit

63f23569
TEMP: disable failing BlockFetch assertion

There are still a lot of TODOs, e.g., the HeaderState is currently just
(isomorphic to) unit, the ChainSyncClient is very basic and still has many
TODOs left in it.


Support "-alpha remove" to "flatten" the alpha channel

 Issue by tstirrat
Wednesday May 21, 2014 at 21:24 GMT
Originally opened as https://github.com/rmagick/rmagick/issues/103

RMagick appears to support most of the options for img.alpha(type), but it doesn't support the alpha remove command.
from: http://www.imagemagick.org/Usage/masking/#remove
"The best solution is to use the Alpha Remove method to quickly and simply replace the transparency with a background color underlay..."
convert a.png   -background skyblue  -alpha remove -alpha off a_remove.jpg

At the moment I resort to a workaround:
img.border!(0, 0, 'white')
img.alpha Magick::DeactivateAlphaChannel

:+1:  For that flag, just needed it. Thanks for the workaround, too.

Could you add it to RMagick source code and create a pull request with it?

I can certainly give it a try. :smile:


fix bug in method reset() of ByteBufferHashTable.

see issue

@zhangxinyu1 thanks. please fix checkstyle

@leventov thanks, I have fixed it.

@zhangxinyu1 Have you seen this issue in the wild, and if so what error message gets reported? Could you write a test that reproduces the problem and verifies the fix?

@gianm Yes, I recently encountered this problem. The exception is
2017-11-14 19:10:12,201- epinephelinae.GroupByMergingQueryRunnerV2: Exception with one of the sequences!
java.lang.IllegalArgumentException
at java.nio.Buffer.limit(Buffer.java:275) ~[?:1.8.0_121]
...
I set druid.query.groupBy.bufferGrouperInitialBuckets=2147483647, so maxBuckets will always be tableArenaSize/bucketSizeWithHash.
In the bad case,  tableArenaSize=1717986912, bucketSizeWithHash=16.  Then there is:
    tableStart = tableArenaSize - maxBuckets * bucketSizeWithHash;  // tableStart = 0
    int nextBuckets = maxBuckets * 2; //nextBuckets=1717986912/16*2= 214748364
    while (true) {
      // int nextTableStart = 0 - 214748364 * 16 = 858993472. However, long nextTableStart = (long) 0 - 214748364 * 16 = -3435973824
      final int nextTableStart = tableStart - nextBuckets * bucketSizeWithHash;

      //858993472 >  tableArenaSize / 2, but -3435973824 <  tableArenaSize / 2
      if (nextTableStart > tableArenaSize / 2) {
        tableStart = nextTableStart;
        nextBuckets = nextBuckets * 2;
      } else {
        break;
      }
    }
   ...
    bufferDup.position(tableStart);  // postition(858993472)
    bufferDup.limit(tableStart + maxBuckets * bucketSizeWithHash); // 858993472 +<PHONE_NUMBER> = -1717986912, limit(-1717986912) will throw the illegalArgumentException.

@gianm I use git rebase master, the return message is "Current branch fix-reset-bug-in-ByteBufferHashTable is up to date."

@zhangxinyu1 git rebase upstream/master, or whatever https://github.com/druid-io/druid/tree/master is called in your local repo

@zhangxinyu1 and git fetch upstream master before that

@leventov Thanks! I have updated my local repo.

@zhangxinyu1 now you need force push to your fork on github

@leventov I have forced push to my fork.

@zhangxinyu1 no, your branch: https://github.com/zhangxinyu1/druid/tree/fix-reset-bug-in-ByteBufferHashTable is still "305 commits behind master".

@leventov ah thanks! now, https://github.com/zhangxinyu1/druid/tree/fix-reset-bug-in-ByteBufferHashTable is "5 commits ahead of druid-io:master".

It shouldn't be 5 commits ahead of master, it should be 2 or 3 commits ahead of master, only those that you made. Please don't use this thread for free Git & Github usage training.

@leventov I'm so sorry. Thanks for your training, you're so nice. I only know the simplest usage of git. I will close this pr, and reopen another pr.


[bug] Don't show login pop-up on MM Mobile

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
0.5.2
wagmi Version
0.6.0
Current Behavior
https://user-images.githubusercontent.com/56917328/189865243-bd357cb8-c53e-42b1-b96c-d27de2638aa4.MP4
I try to connect MetaMask on mobile from Safari like on your website where it works fine. But when I try with my React app or with your react example, it doesn't work as you can see on the video (where I'm using your react example).
Expected Behavior
Open MM mobile connect popup

Steps To Reproduce
Clone with-create-react-app
Start and try to connect using Safari on IOS
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

Solution : https://github.com/rainbow-me/rainbowkit/issues/420

@LeoClairet05 will there be support for vite as well?

Solution found thanks to : #420
Why it's not working with React (without NextJS)?
It's not working since Webpack 5.0, Buffer is not installed by default anymore so it creates an issue with WalletConnect (Used when you try to connect with MetaMask from another browser than the integrated one in MetaMask mobile app). ⚠ Make sure your react app use https & not http, or it'll not work.
I've created a public repository that you can clone to have a working react app that integrates Rainbowkit, Wagmi without NextJS -> https://github.com/LeoClairet05/wagmi-rainbowkit-react-template
On my template Wagmi is configured to connect on custom chains Fuji & Avalanche.
To make it works on your own project you can follow these steps:
Install "react-app-rewired" -> https://www.npmjs.com/package/react-app-rewired
Change your package.json scripts as following:
"scripts": { "start": "HTTPS=true react-app-rewired start", "build": "GENERATE_SOURCEMAP=false react-app-rewired build", "test": "react-app-rewired test", "eject": "react-scripts eject" }
and add the following devDependencies in your package.json: "devDependencies": { "assert": "^2.0.0", "buffer": "^6.0.3", "crypto-browserify": "^3.12.0", "https-browserify": "^1.0.0", "os-browserify": "^0.3.0", "process": "^0.11.10", "react-app-rewired": "^2.2.1", "stream-browserify": "^3.0.0", "stream-http": "^3.2.0", "url": "^0.11.0" }
create a new file in your directory named "config-overrides.js" with the following content:
` const webpack = require('webpack');
module.exports = function override(config) { const fallback = config.resolve.fallback || {}; Object.assign(fallback, { "crypto": require.resolve("crypto-browserify"), "stream": require.resolve("stream-browserify"), "assert": require.resolve("assert"), "http": require.resolve("stream-http"), "https": require.resolve("https-browserify"), "os": require.resolve("os-browserify"), "url": require.resolve("url") }) config.ignoreWarnings = [/Failed to parse source map/]; config.resolve.fallback = fallback; config.plugins = (config.plugins || []).concat([ new webpack.ProvidePlugin({ process: 'process/browser', Buffer: ['buffer', 'Buffer'] }) ]) return config; } `

Is there a way to undo this process, I added this code and I was able to run it on https,
But even after deleting all the dependencies, removing the node_modules, and reinstalling the packages,
It still goes back to pointing to https

Yes, just remove the HTTPS=true from the package.json in "scripts":{"starts"

I don't know ^^

It was weird to see this unexpected behaviour because even with react-scripts start it was pointing to https
😆 😭 I will try different approaches @LeoClairet05
thank you for your help

Is this issue fixed in the latest update or do I need to do the above work around, because when I followed the above steps above by @LeoClairet05, I got connection not secure on my localhost.

Is this issue fixed in the latest update or do I need to do the above work around, because when I followed the above steps above by @LeoClairet05, I got connection not secure on my localhost.

Local host will not allow you to connect to unsecured connection by default
If you are using Chrome, you'd have to explicitly visit the page by bypass the warning page.
Hope this helps

Yeah that issue is resolved. Thanks for the help. Now it's working as expected.

After struggling for a bit, I finally found the solution for Vite.
I was having another error along the line of: Unhandled Promise Rejection: ReferenceError: Cannot access uninitialized variable.
For anyone having this issue with Vite you need to populate the window with something called Buffer. I have no idea why it works but it works for me.
Just go in your index.html add a script in your  like so:
 <script type="module">
      import { Buffer } from "buffer";
      window.Buffer = Buffer;
</script>

Hope that helps someone :)

@sshmaxime This isn't working now, still getting the Uncaught ReferenceError: Buffer is not defined

if yall are still facing problems in 2024 , w rainbowkit v 1.0> ,  I struggled quite a bit and found a config that magically fixes teh mobile connect issue lol :
next.config.js file :
/** @type {import('next').NextConfig} */
const nextConfig = {
  reactStrictMode: true,
  webpack: config => {
    config.resolve.fallback = { fs: false, net: false, tls: false };
    return config;
  },
};

module.exports = nextConfig;

Can anyone tell me how to fix it for React Vite + TS app?


[Feature] Open Stylebot to the sidebar.

In my opinion, it is more efficient to use the sidebar than to interrupt Stylebot's own UI in the page.
I'm using Firefox.

It's actually pretty non-trivial to implement it as a separate window/sidebar. However, you can adjust the behavior of the editor so it behaves like a sidebar by turning on the "Adjust page layout" option (see below screenshot)
Currently, there seems to be an issue where it is not respected on page reload - I'll work on fixing that.


feat: Add stats block replacement

Description
This PR creates a new block, currently called callout-numbers, which will replace the existing stats block as can be seen on the homepage.
Related Issue
ADB-47
Motivation and Context
The existing block received new design direction. A new block was created to start to decouple the original stats block and jobs block, which were both added to a container on page load.
How Has This Been Tested?
Tested on the preview page: https://sbx-stats-block-adjustment--design-website--adobe.hlx.page/drafts/dev/callout-numbers-draft
Screenshots (if appropriate):
320px screen:

600px screen:

900px screen:

1300px screen:

Types of changes

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] I have read the CONTRIBUTING document.
[x] All new and existing tests passed.

Confirmed. This feature is being removed instead of redesigned. Closing PR.


fix path to .formatter.exs file

Sorry I hurried with pull request #31
use absolute path to .formatter.exs file in jobs with cwd

Thanks! ✨


Unexpected behaviour with incremental solving

Hi,
I'm getting some strange behaviour with both version 5.6.2 and the head of master. This is a simplified version of something automatically generated by our system, so apologies if the program looks a bit weird.
#program base.
#show n/1.
{ n(0..4)}.

c :- not n(3).

e1 :- not n(1).
e2 :- not n(2).

:- not n(0); not n(4).
:- e1; not n(3).
:- not n(4); e2.

#program prg_1.

a :- e1, e2.
:- not a.

#script (python).

def main(prg):
  prg.ground([("base", [])])
  print(prg.solve())
  prg.ground([("prg_1", [])])
  print(prg.solve())
#end.

If I take out the first solve (i.e. ground both programs and then solve), Clingo gives the expected solution. If I run it as above, Clingo returns unsatisfiable the second time. Is there something I'm missing, or is this a bug?
Thanks,
Mark

@marklaw, thanks for the report and the nice and small example.
@BenKaufmann, this seems to be a bug in the preprocessor. Running the problem with --eq=0 and --eq=1 produces different results. The first one seems to be the correct one. The problem in aspif format is as follows:
1 1 1 1 0 0
1 1 1 2 0 0
1 1 1 3 0 0
1 1 1 4 0 0
1 1 1 5 0 0
1 0 1 6 0 1 -3
1 0 0 0 2 6 -5
1 0 1 7 0 1 -2
1 0 0 0 2 -4 7
1 0 0 0 2 -5 -1
1 0 1 8 0 1 -4
4 4 n(0) 1 1
4 4 n(1) 1 2
4 4 n(2) 1 3
4 4 n(3) 1 4
4 4 n(4) 1 5
4 1 c 1 8
4 2 e1 1 7
4 2 e2 1 6
0
1 0 1 9 0 2 6 7
1 0 0 0 1 -9
4 1 a 1 9
0


Update SQL Managed Instance outbound network security rules

As per Srdan Bozovic<EMAIL_ADDRESS>the outbound network security rule for management traffic has changed from "Internet" to "AzureCloud".

@doherty100 : Thanks for your contribution! The author, @srdan-bozovic-msft, has been notified to review your proposed change.

#sign-off


URL not working.

Issue
import { RedisClient } from 'redis'
new RedisClient({
      url: "redis://rediscloud:pass@redis-ttt.ttt.eu-central-1-1.ec2.cloud.redislabs.com:13180"
});

Why is this not working? It's giving me this error: Error: Redis connection to <IP_ADDRESS>:6379 failed

Environment

Node.js Version: v12.14.1

Redis Version: 4.0.9

Platform: Windows 10 & Ubuntu

@supertiger1234 instead of new RedisClient you should use redis.createClient

@supertiger1234 instead of new RedisClient you should use redis.createClient

I'm using typescript, so i did import { RedisClient } from 'redis'. (I also tried in normal nodejs, and it still doesn't work). I found a new way though by just providing the host, username and password. I still hoped the url to work.

This appears to still not be working how it is described in the docs. Both passing a redis url as a string to createClient('redis://redis-host:6379/0') and passing it in the config object as value for the url key. Both seem to append <IP_ADDRESS> to the url making it invalid.

Can confirm -  still broken

Still broken in 3.1.2

Still broken in 3.1.2

Got any solution @andrewcharnley ?

In my case it was to do with the newer rediscloud:pass syntax which is Redis v6, on earlier version it fails.

what should i do to getthe server working ??

@andrewcharnley have you found another lib that works with this kind of URL? I'm having the same problem here.

@dayvsonsales
In my case, was using a dummy username,  mostly h, removing the username portion for redisURL solved the issue for me

@pawan-bobbili-nutanix I didn't get it. Could you give an example, please?

@dayvsonsales
syntax for redis URL is like [redis[s]:]//[[user][:password@]][host][:port][/db-number][?db=db-number[&password=bar[&option=value]]] right, when upgraded to 3.1.1, my redis connection was failing, on searching , found that the issue i placed the dummy username in the user place of syntax. So, I removed it.
Previously (Not working after updated to 3.1.1)
redis://h:{rest of the url}
Now (Working in 3.1.1)
redis://:{rest of the url}

Working solution for typesciprt:
const client: RedisClient;
client = createClient(redisUrl);

How did you do this? Please show us the new way, I've tried everything and non worked

I connected mine like this:
`import { createClient } from "redis";
const redis = createClient({
url: redis://${process.env.REDIS_HOST}:${process.env.REDIS_PORT},
});`

Still broken, have been stuck on this for 8 hours now. I thought that docker's bridge is assigning wrong aliases, but the issue was here 😞

Still not working

createClient() takes in an options parameter.
import { createClient } from 'redis'

const client = createClient({ url: process.env.REDIS_URL })

createClient() takes in an options object as a parameter, you should set the URL there. For more info look at the type definition found under client.d.ts. -> RedisClientOptions
import { createClient } from 'redis'

const client = createClient({ url: process.env.REDIS_URL })

createClient({url: REDIS_URL}) worked for me. was struggling for 2h with this. thank you! <3

This works for me :
import { createClient } from "redis";
const { Url } = require("url");

const url = `redis://${REDIS_DATABASE_HOST}:${REDIS_DATABASE_PORT}`;
const redisClient = createClient({
  url: Url(url),
  password: REDIS_PASSWORD,
});


fix: private api gateways with vpce need policy

what

create a resource policy for VPCE's

why

if a VPC Endpoint is associated with a private rest api, it still needs
permission to use that endpoint

references

https://docs.aws.amazon.com/apigateway/latest/developerguide/apigateway-private-apis.html

/terratest


Resolve Stack Violations on Create Mint and Group Instruction

Issue
When calling the program locally to run in a solana-test-validator developer receives large stack size violations on mint and group instructions.
Cause
The large use of extension attributes on on the instructions cause heap memory violations.
Resolution
Reduce the use of attributes on extensions to reduce macro generation.
https://github.com/wen-community/wen-new-standard/blob/main/programs/wen_new_standard/src/instructions/mint/create.rs#L38-L48

duplicate https://github.com/wen-community/wen-new-standard/issues/63


[Feature Request] Time format

Can you add an option for the format of the time : 12AM/PM or 24 hours.
Thanks

The hour format now depends on the system format.


(QGIS) ModuleNotFoundError: No module named '_tkinter'

whitebox version:1.5.1
Python version: 3.7.0 (v3.7.0:1bf9cc5093, Jun 27 2018, 04:59:51) [MSC v.1914 64 bit (AMD64)]
Operating System: Windows 10 64bit

Description
I'm running the OSGeo pyqgis distribution
pip install whitebox
works fine. However, my distribution seems to be missing Tkinter. I believe there is something with QGIS that doesn't let Tkinter run/install.
Is there  way to change the backend of whitebox? Or another workaround that will let me run the pyqgis AND whitebox API?
import whitebox
ModuleNotFoundError: No module named '_tkinter'

It works fine on my end. You can also try out the Jupyter frontend https://github.com/giswqs/whiteboxgui
import whitebox
whitebox.Runner()


Ergo error payout failed : Transaction should conform EIP-27 rules

Hello I have an issue with payouts on ergo.
I am on the latest dev branch.
this is the log (I removed the address):
[2022-07-22 16:51:53.3928] [I] [ergo1] [Ergo Payout Handler] Paying 47.33479 ERG to 14 addresses
[2022-07-22 16:51:53.3928] [I] [ergo1] [Ergo Payout Handler] Unlocking wallet
[2022-07-22 16:51:53.6259] [I] [ergo1] [Ergo Payout Handler] Wallet unlocked
[2022-07-22 16:51:53.6259] [I] [ergo1] [Ergo Payout Handler] Current wallet balance is 60.0121 ERG
[2022-07-22 16:51:53.6638] [E] [ergo1] [Ergo Payout Handler] Payment transaction failed: Bad request List(PaymentRequest(,18323300000,List(),Map()), PaymentRequest(,4403600000,List(),Map()), PaymentRequest(,2241300000,List(),Map()), PaymentRequest(,3026900000,List(),Map()), PaymentRequest(,151800000,List(),Map()), PaymentRequest(,List(),Map()), PaymentRequest(,List(),Map()), PaymentRequest(,1439400000,List(),Map()), PaymentRequest(,1670500000,List(),Map()), PaymentRequest(,List(),Map()), PaymentRequest(,816800000,List(),Map()), PaymentRequest(,3795200000,List(),Map()), PaymentRequest(,7860900000,List(),Map()), PaymentRequest(,708800000,List(),Map()), PaymentRequest(,1000000,List(),Map())). Transaction should conform EIP-27 rules
[2022-07-22 16:51:53.6638] [I] [ergo1] [Ergo Payout Handler] Locking wallet
[2022-07-22 16:51:53.6659] [I] [ergo1] [Ergo Payout Handler] Wallet locked
What can I do?

@oliverw this bug is due to an orphaned mined before the payout of the previous block. somehow it interfer with it. Keep this is mind if it comes back.
[2022-07-22 19:01:53.3587] [I] [PayoutManager] Processing payments for pool ergo1
[2022-07-22 19:01:53.4547] [I] [ergo1] [Ergo Payout Handler] Block 799010 classified as orphaned due to pk mismatch
[2022-07-22 19:01:53.4547] [I] [PayoutManager] Processing payments for pool ergo1, block 799010


skipped restart reconciler when create_db or revive_db failed

After create_db or revive_db fails, the re-ip process in the restart_reconciler will also fail. To my understanding, we plan to skip the restart reconciler, and the operator will handle the situation by recreating the database to prevent this failure

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

I checked to ensure that the operator will skip restarting the reconciler. Based on the events I've observed, I haven't seen the operator process restarting the reconciler
Type     Reason          Age    From                Message

Normal   CreateDBStart   5m42s  verticadb-operator  Starting create database
Warning  CreateDBFailed  5m41s  verticadb-operator  Failed when calling VCreateDatabase

@spilchen This is the result I try with my latest change, now we can skip the restart conciler, but we keep re-createdb.
Warning  CreateDBFailed     45s                verticadb-operator  Failed when calling VCreateDatabase
Normal   CreateDBStart      34s (x2 over 48s)  verticadb-operator  Starting create database
Normal   CreateDBSucceeded  0s                 verticadb-operator  Successfully created database with subcluster 'pri'. It took 33.385842687s

@spilchen  this is the result I achieved with my latest change. I can skip restarting the reconciler and continue to recreate the database
Warning  CreateDBFailed     45s                verticadb-operator  Failed when calling VCreateDatabase
Normal   CreateDBStart      34s (x2 over 48s)  verticadb-operator  Starting create database
Normal   CreateDBSucceeded  0s                 verticadb-operator  Successfully created database with subcluster 'pri'. It took 33.385842687s


Detecting whether help was provided via command line arguments to Spectre.Console

If you code simple application Spectre.Console - then you're interested not only on parsing command line arguments, but also you're interested in figuring out whether --help was used as command line argument.
Using .net reflection it's possible:
https://github.com/tapika/swupd/blob/master/cakebuild/CommandLineArgs.cs#L248
but uses bit ugly syntax for that purpose.
Is there a simpler solution to this ?

Apparently help command is built-in, and quite well hidden.
        // Get the command to execute.
        var leaf = parsedResult.Tree.GetLeafCommand();
        if (leaf.Command.IsBranch || leaf.ShowHelp)
        {
            // Branches can't be executed. Show help.
            configuration.Settings.Console.SafeRender(HelpWriter.WriteCommand(model, leaf.Command));
            return leaf.ShowHelp ? 0 : 1;
        }

Can't imagine calling this code as user from outside of Spectre.Console.
In theory you can also override default console output, like this:
            TestConsole console = new TestConsole();
            var actualConsole = AnsiConsole.Console;
            AnsiConsole.Console = console;
            AnsiConsole.Record();

            var app = new CommandApp<ParseCommandLineArgs>();

...
            int exitCode = app.Run(args);
            actualConsole.Write(AnsiConsole.ExportText());

and then display help at the end. (text will be without colors)
Sounds bit complex solution as well.

The CommandExecutor.Execute(...) method (which the first code snippet above from @tapika was taken from) does explicitly handle the -v|--version argument:
        // No default command?
        if (model.DefaultCommand == null)
        {
            // Got at least one argument?
            var firstArgument = args.FirstOrDefault();
            if (firstArgument != null)
            {
                // Asking for version? Kind of a hack, but it's alright.
                // We should probably make this a bit better in the future.
                if (firstArgument.Equals("--version", StringComparison.OrdinalIgnoreCase) ||
                    firstArgument.Equals("-v", StringComparison.OrdinalIgnoreCase))
                {
                    var console = configuration.Settings.Console.GetConsole();
                    console.WriteLine(ResolveApplicationVersion(configuration));
                    return 0;
                }
            }
        }

I do wonder if this kind of explicit handling of command arguments could also be used for the -h|--help help option.

When this issue is triaged/is found to be valid - please assign it to myself and I'm happy to fix.

I would prefer that no need to assume order of arguments - whether help comes first or not.
Previously I have help parameter handling as hardcoded, and it was parsed like this:
var helpOpt = new CommandOptionAttribute("-h|--help");
var isMatch = helpOpt.GetType().GetMethod("IsMatch", BindingFlags.NonPublic | BindingFlags.Instance);
foreach (var arg in args)
{
    if ((bool)isMatch.Invoke(helpOpt, new object[] { arg.TrimStart('-') }))
    {
            displaysHelp = true;
    }
}

But this also implies that help command is the same as "-h|--help" - I can easily think that in future such variations as -?|--? could be added to be more flexible. Help option needs to be centralized and preferably within Spectre.Console.
Could this be coded somehow like this:
var (exitCode, helpWasShown) = app.Run(args);
if(helpWasShown) { Environment.Exit(exitCode); }
... continue with processing command line options...

?

Would you propose to also refactor the code snippet above that handles the -v|--version argument into the same approach used for -h|--help, if for no other reason than consistency ?

Generally API's and functionalities they provide:
API1: app.Run() :
1. parse command line arguments
2. print help if help requested

- bad: combines two api's, not possible to separate
- bad: does inform end-user that help printing occurred

For API2 - consistency, and simplity to use.
Maybe it would make sense to also refactor API 1 to have parse and print as a separate operations, but I think returning two return parameters would be beneficial already.


Clarification on MCTS Runtime Comparison in Table 2

Hello Edward,
In Table 2, for TSP1000, I noticed that for the DIFUSCO model, the SL+G's runtime is 11.86 minutes, and the SL+MCTS's runtime is 24.47 minutes. It can be estimated that the MCTS component alone takes about 12 minutes. However, when comparing this to the ATT-GCN and DIMES models, which also utilize MCTS, I observed that their total runtime (including both inference and MCTS) is only 4.1 minutes and 6.87 minutes, respectively.
This leads me to question the fairness of this comparison. The MCTS runtime for DIFUSCO seems significantly longer than that for the other two models. Given that providing more time to MCTS generally yields higher quality solutions, I am curious whether this difference in allocated runtime might affect the fairness of the comparison.
Could you please clarify whether the runtime for MCTS is consistent across these models, or if there are other factors contributing to this apparent discrepancy?
Thank you for your attention to this matter.
Best regards,
Yifan

Oh, I just realized a fact that I previously overlooked. It appears that the MCTS runtimes for ATT-GCN and DIMES in your table were conducted under a 64-thread setup, as per the hardware specifications mentioned in the DIMES paper. However, it seems that MCTS in your implementation was not executed with 64 threads.
After reviewing your MCTS code, I noticed that the default thread count is set to 16. Therefore, I would like to confirm: were the MCTS algorithms for TSP500, TSP1000, and TSP10000 run and timed under a 16-thread setup in your experiments?
This information would help in understanding the runtime comparison more accurately.
Thank you for your clarification.

Hi Yifan, for your first question, the results of other baselines are taken from the corresponding publications, and may use different hardware, so the runtime may not be directly comparable.
For your second question, the code we released is the code that is used to produce the results in the paper. So the results are obtained with a 16-thread setup.


[Question] <How to set sentinel fallbackUri?>

Question
version：master
The configuration is as follows, fallbackUri = 'http://localhost:9195/http/test/payment'

Response

'http://localhost:9195/http/test/payment'

have this controller  in 9195?

omg, i get it. Thanks


[FEATURE] Ability to change branch yellow color

I tried changing all the color options, but it doesn't change

Four colors added.


Run some tests on FAISS clustering

Basic test to check out FAISS-based KMeans
Visualize clustering based on a num_clusters parameter, with a bit of noise added in

Also, added an environment.yml for Anaconda/Miniconda users!


Timers does not count down IN game, tabbed out they work

As the title say, I can start the test in La Noscea or while raiding Extreme Titania and the same problem occur. It shows up almost like pictures rather than counting down smoothly in game almost as if it's horribly lagging, but when I'm tabbed out of the game it works as usual.
Problem occurred after I had problems updating and thus just did a clean re-download, using the download from advancedcombattracker, trying both OverlayPlugin-<IP_ADDRESS>-x86-full and OverlayPlugin-<IP_ADDRESS>-x64-full, with cactbot-0.10.1 on both attempts.
I also tried once using cactbot-0.10.0. using the La Noscea test, and the same problem occured.

Edit: Problem solved. Either it was updating FFXIV_ACT_Plugin.dll and restarting the pc, or it was just restarting the pc that did it. Now it works anyway.

Problem returned even after restarting PC and updating FFXIV_ACT_Plugin.dll the next day.

I put the Frame Rate Limit in game to 144fps instead of None and now it works, though don't know if it's a permanent fix or if the problem will occur again

I have similar problems with the overlay and FFXIV trying to hog the GPU - setting a frame limit like you did and turning off power management on GPU (like AMD Chill) seems to help. Not sure why it happens but it fixed it for me.

Yeah, @gaogaostegosaurus left some good comments in #309 that might be helpful.

Sounds like you have a workaround for this issue, so I'm going to close this for now.


Pax recon eff

Added cal_rec_efficiency_tf in x1t_sr1.py which is called in photon_acceptance.
Previously, there is only a requirement for the event to have at least 3 photons detected and a flat unity acceptance thereafter.
In cal_rec_efficiency_tf, the PAX reconstruction efficiency is interpolated from a weighted combination of median and lower/upper bounds similar to the logic in BBF, see https://github.com/XENON1T/bbf/blob/master/bbf/gpu_modules/SignalSimCuda.py#L483. This reconstruction efficiency is then applied to the events via photon_acceptance.

This is over a year old now, so I think we can safely close this. Feel free to reopen or make a new PR!


fix: make swagger version locked 0.12.0 rather than lastet

Signed-off-by: Allen Sun<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
In the original Dockerfile, we use command go get -u github.com/go-swagger/go-swagger/cmd/swagger to install swagger binary which is already from the newest code. And we cannot guarantee the quality of the latest code ever.
As we had a rule in hack/generate-swagger-models.sh that we should use swagger 0.12.0 to meet our requirement, we should always download a binary which is locked to version swagger 0.12.0.
As a result, I change the command to be wget --quiet -O /bin/swagger https://github.com/go-swagger/go-swagger/releases/download/0.12.0/swagger_linux_amd64.
Ⅱ. Does this pull request fix one issue?

This issue happens almost in all pull requests which we have modified the swagger.yml.
Like https://github.com/alibaba/pouch/pull/879, https://github.com/alibaba/pouch/pull/878, https://github.com/alibaba/pouch/pull/875 and so on
Ⅲ. Describe how you did it
change a command to finish this
Ⅳ. Describe how to verify it
none
Ⅴ. Special notes for reviews

Codecov Report

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

@@           Coverage Diff           @@
##           master     #881   +/-   ##
=======================================
  Coverage   14.62%   14.62%
=======================================
  Files         115      115
  Lines        6940     6940
=======================================
  Hits         1015     1015
  Misses       5839     5839
  Partials       86       86

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 a2c6ffb...8a4543f. Read the comment docs.

LGTM


How to see elastic search and mongodb data

Hi,
When api-umbrella is running, I can understand that elastic search and mongodb are running in behind to store my data. how can see that data thru terminal in behind screens?

Maybe you should try to connect to mongoDB using CLI, and something like Kibana for ElasticSearch?

By default, the internal MongoDB runs on port 14001 and the internal Elasticsearch database runs on port 14002. So if you're interested in querying these databases directly, you can do so via:

MongoDB: /opt/api-umbrella/embedded/bin/mongo <IP_ADDRESS>:14001/api_umbrella
Elasticsearch: curl "http://<IP_ADDRESS>:14002/"


Documentation of functions related to Cimmino and Bernier 2014

The uniform borehole wall temperature (UBHWT) documentation points to Cimmino and Bernier 2014. Specifically the uniform_temperature function is an implementation of Boundary Condition 3 (BC-III). Could it be placed in the documentation that it specifically applies to BC-III?
Could the documentation of the uniform_heat_extraction function point the boundary condition to which is applies from Cimmino and Bernier (2014)?

Also should be in accordance with Cimmino (2018).

Also should be in accordance with Cimmino (2018).

I agree, this is a good point.

I agree, this is a good point.


Tap on image crashes on Xcode 6 and swift

Though the component works like a charm with iOS7 and objectiveC, using the controller in a swift project result in a severe crash on the image tap. I could not even trap the crash with breakpoints in the handleDismissFromTap: :-/
#update#: As it seems, all gestures crash the app :(

Can you provide more details of your use case or the Swift code you are using? I just tested a demo project where I converted the code in DemoViewController.m to Swift and everything worked fine without any crashes.

Any update as I have the same problem

@BCGray00 I asked @jerometonnelier for more details and/or code regarding the crash he's seeing as I can't duplicate the issue. Can you provide any details about the crash?

Oups sorry I totally forgot about that :( In the meantime, I used another component, so I won't be able to provide any code ou crash log :( Sorry about that.

same, i changed aswell


[WIP] #395 Fixed few overlaps found

[WIP] #395 Fixed few overlaps found

Hi @dinesh-skyach
PR does not fully fixes the issue:

Hi @yivo, I have made the changes and it is fixed now.

@dinesh-skyach Please squash all commits into one and put a name: Fix few UI issues found at profile page & markets (issue #395)..

@dinesh-skyach Still doesn't work:


Ensure this works correctly on Python 3.11 and 3.12

From looking at the source code I think it may have this bug:

https://github.com/datasette/datasette-sqlite-authorizer/issues/3
https://github.com/simonw/sqlite-authorizer-examples/commit/a6f38a2f4c7e6fda4dd7fccca9973c4d74f4e9e2

Failed, as expected: https://github.com/datasette/datasette-sqlite-debug-authorizer/actions/runs/8149112410


Hide days ?

Hi,
Is it possible to hide some days from week in datepicker ?
Exemple: I want to display only working days. So, i want to hide Saturday and Sunday.
Is it possible ?
Thanks

Actually, I do it with this css :
.picker__table th:nth-child(6), .picker__table th:nth-child(7),
.picker__table td:nth-child(6), .picker__table td:nth-child(7) {
    display: none;
    visibility: hidden;
}


Error in typings

@nik0kin There seems to be an issue with the typings:
The following is allowed by the typings but results in strava being undefined during runtime
import { strava } from 'strava-v3' //

So we tried using the default import and now strava is defined during runtime
import * as strava from 'strava-v3' // this results in strava being defined during runtime

But the issue with that:

there are api methods on the instance during runtime
but during dev time there are no api methods available only

defaults
-strava

The typings suggest that strava does not use default exports, or at least in addition also exports the strava constant. Howveer that does not seem to be the case:
Looking at the js there seems to be only a single default export made, hence import { strava } syntax does not work:
https://github.com/UnbounDev/node-strava-v3/blob/a1bfcb3664e6ef4921c8c91b7a3cd295185ce70f/index.d.ts#L158
Should be:
declare const strava : Strava;

And:
https://github.com/UnbounDev/node-strava-v3/blob/a1bfcb3664e6ef4921c8c91b7a3cd295185ce70f/index.d.ts#L160
Should be
export = strava;

This is only a partial fix as this excludes all interfaces from being exported. in order for this to work add the following line at the end:
export as namespace strava;

Which allows users to access the interfaces as follows:
const routes: strava.ActivitiesRoutes

Inspiration taken from moment:
https://github.com/moment/moment/blob/63f3d52945bc773925b862c61ee7a322d4a33308/ts3.1-typings/moment.d.ts#L784

Please submit this as a PR.

@ntziolis @markstos I'm getting an error since this update (v2.0.8):
An export assignment cannot be used in a module with other exported elements.

@ntziolis Please work with @vtimbuc to resolve this. I'm not currently using the TypeScript bindings and trusted that that would change was correct. It could helpl if you both mentioned the version of Node.js and TypeScript that you are using.

I see the same error as @vtimbuc (typescript version 3.7.2)

Will look into this later today. Using typescript 4.0.3. @vtimbuc & @nik0kin Did the bidnings work for you before the update? especially exporting strava itself? See my error description above.
Or where you only using the different input / output types for activities etc? This also worked for me but importing strava itself did not.

Yes, it works with version 2.0.7 using ts-node or tsc compiling to commonjs. tsconfig.json

First off apologies for breaking stuff for you guys. After looking into this a bit further it seems the typing I provided are only supported by later typescript versions.
Using implicit default importing is disabled by policy (as is the default by now) in our projects, hence we could no use:
import strava from 'strava-v3'
After reverting to the previous typings + some fiddling we found the following works well:
import { default as strava, Strava} from 'strava-v3';
So for the time being I'm asking to revert back to the previous typings. By simply reverting the commit made by the PR.
That said there is an issue with the typing, since other imports are also possible but produce runtime errors like:
import * as strava from 'strava-v3'
I suggest calling out the recommended way of importing this into typescript projects on the npm page. It would be helpful to provide an sample that works across typescript versions and does not require changing compiler settings.
@nik0kin Can you check in your setup if the following provides correct typings + has the strava object populated during runtime:
import { default as strava, Strava} from 'strava-v3';

I prepared a PR https://github.com/UnbounDev/node-strava-v3/pull/102 that:

reverts back to previous typings
but also removes export const strava since this is incorrect (looking at the js file there is only a default export no named exports

The second step is key to prevent people from importing this as we did:
import { strava, Strava} from 'strava-v3'

const test: Strava = strava;

This was previously possible previously, but didn't work since the js does not provide a named export strava. Hence it failed during runtime.
@nik0kin Would you be so kind and verify that this doesn't break anything on your end?

importing the default (either way) from strava-v3 works for me with:
declare const strava: Strava;
export default strava;

(There has to a variable named strava to be able to export it as default)

Fixed in v2.0.9, just published.


parent job

Chronos supports parent job,can i add parents job using chronos-python

hi jerrybean,
I don't quite get what you mean - any job with a schedule can be used as a parent for a dependent job, so you can use the client.add function for this. Is that what you meant?

hi Rob-Johnson,you are right! I do something wrong before,thank you!

no problem.


Backported segfault fix in tests and potential badness in general.

Here is the fix for dockercfg_tests.cpp:
https://github.com/dcos/dcos-mesos-modules/commit/31c4d688f9a571b103e73d8ea7be4af02ae109ac#diff-8c7a8f72b4e0dbf2fad3bf8d5bbcd43cR171


Handle 6 decimal places cloud event

Fixes https://github.com/Azure/azure-sdk-for-python/issues/18602

/check-enforcer override


[Bug]: Sometimes cannot connect chromadb.HttpClient()

What happened?
After using chromadb for a while, it won't connect and can't restart the docker container. unable to request service
Versions
chromadb==0.4.24
Versions
chromadb==0.4.24
Relevant log output
No response

@0Xiao0, if you run Chroma in a docker container, can you look at the container logs for errors?

@0Xiao0, if you run Chroma in a docker container, can you look at the container logs for errors?
it didn't see any errors

@0Xiao0, thanks for sharing. Ok, so we can establish that your Chroma server is running and listening.

I didn't do anything, I just mounted local files


Added deleteMethod option (#205)

Hello,
This is the fix I propose for #205. All test passed in testem, for the following browsers:
Chrome 43.0.2357.130 (64-bit)
Safari 8.0.7 (10600.7.12)
Firefox 38.0.5
We'll need to update the documentation at the wiki too (I'd be glad to do it). Should we wait until the next version is released for that?

It seems like the build failed on travis. I'm sorry but I'm not sure what I did wrong :(
What should I do to fix this?

You should update this test: https://github.com/orthes/medium-editor-insert-plugin/blob/master/test/images.js#L306-L323
You can replace post by ajax.
And yes you can update the wiki as soon as this PR will be merged.

@j0k3r it passes now. Thank you very much for your guidance.

@orthes Any objection to this? :smiley:

:+1:

Thanks @bertzzie :smile:


Master2

Add new comments to the test_routes.py file for those new test cases as discussed.

Duplicated PR


Add option to disable puppeteer and run against a static jsdom (faster)

For increased speed and in specific situations, it might be helpful to have an option to disable the headless browser altogether.
Maybe it could use https://github.com/jsdom/jsdom instead.

This repository is now using lodash style issue management for enhancements. This means enhancement issues will now be closed instead of leaving them open.
The enhancement backlog can be found here: https://github.com/filamentgroup/glyphhanger/issues?utf8=✓&q=label%3Aneeds-votes+sort%3Areactions-%2B1-desc+
Don’t forget to upvote the top comment with 👍!

v3.2.0 was released!


Specialize some Folder::consume_iter methods

Fixes the second item of #465.

bors r+

bors r=nikomatsakis


Consider making sample JSON in command agree with samples DTDL

This is very much a for consideration only, because there's pros/cons to what we have today and what I'm suggesting.
In the page we have this blurb.
az iot hub digital-twin invoke-command --command-name reboot -n {iothub_name} -d {device_id} --payload '{"delay":5}'

This does indeed successfully send over JSON.  But assuming we're trying to have that JSON map to what's in the sample we use all over teh pace, this actually isn't what we want.  We'd only want the 5 to be sent across.
Now the pro of leaving as-is is that the JSON as in the document is much more clearly JSON.  5 is of course perfectly reasonable JSON, but it doesn't scream out as JSON.  The pro of changing to just be 5 is that the line here maps to the tempController that lives everewhere else with how device / IoT Explorer send it, which is NOT including the "delay" part from DTDL.
A third approach could be to change the JSON we're sending to be like '{"SomethingElse": 5}.  So it still pops out as JSON, but someone coming to this from the TempController sample won't think they can copy/paste and be done.
I don't have strong opinions on this.  It caused me to trip up momentarily when running sample, though someone new to DTDL might take them longer so I want tocall out.

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

ID: 88ccfa91-20a2-5812-3d58-119f75c271de
Version Independent ID: f7d0bbc4-2cd7-c4ed-0d63-b1014afb344c
Content: az iot hub digital-twin
Content Source: latest/docs-ref-autogen/iot/hub/digital-twin.yml
Service: iot-hub
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

route to service team

update help text in pr, will be available in next release


fix: fix backtraces errors

Deps: #216
This PR fix backtraces erorr for desmos_std.

backtraces is non wasm environment feature, we can ignore this error warning.


Manual entry is no longer supported, please update docs

The Purview docs reference manual entry, but that is no longer supported.  Pleas update the docs accordingly.
"Or, you can select Enter manually and enter a Server name."

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

ID: 7ca3773c-adab-edfa-3cba-66be4cb1a307
Version Independent ID: 6d254ab7-77c1-f920-60d5-30c6a9341afb
Content: Register and scan Azure SQL Database - Azure Purview
Content Source: articles/purview/register-scan-azure-sql-database.md
Service: purview
Sub-service: purview-data-catalog
GitHub Login: @hophanms
Microsoft Alias: hophan

Hello @tifischer,
Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

#assign:viseshag per conversation with Daniel

Thank you. This is now fixed in this PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/147794
#please-close


Upgrade dependencies

Fixes build issues in #487 , and cleans up a few references to deprecated sets.String

/lgtm cancel

Not sure if the current failing e2e tests are related to. Can you rebase, as you had some other PRs with updates there?

/cc @creydr


add realtime logs.

get real time logs for every process

@dk122809 I'm working on a better version of pm2-webui, I looked into your implementation, SSE might be better suited that websockets for realtime log streaming. Will update new version soon. Thanks


fix(tsi): retaining fileset should not copy pointer

Thanks for the submission @foobar! The gotest failures look real (not a spurious failure) to me, I'll wait on the full review until I see they're working. My first impression is that I'm not sure what problem this is meant to solve. Could you describe the goal so I can better understand?


Subscriber doesn't deliver messages with Definitions

I have a topic in a bag where messages include an actionmsgs.GoalStatus field. The subscriber doesn't deliver any message. It seems that the problem comes from the Definitions in GoalStatus.
Here's a simple way to reproduce the issue: in publisher-custom.go and subscriber-custom.go, add msgs.Definitions `ros:"uint8 A=0,uint8 B=1"`  in the definition of TestMessage. Then the subscriber won't show any message.

messages with definitions can now be written and received, thanks.

Thanks for the quick fix!


Error trying to run telepresence on osx (Python 3.4 is too old, need 3.5+)

My environment is osx 10.11.1 and python 3.4. After following the brew installation process, I tried to run telepresence but am getting the following error:
Traceback (most recent call last):
  File "/usr/local/bin/telepresence", line 914, in <module>
    main()
  File "/usr/local/bin/telepresence", line 868, in main
    args = parse_args()
  File "/usr/local/bin/telepresence", line 125, in parse_args
    "Telepresence: local development proxied to a remote Kubernetes "
TypeError: __init__() got an unexpected keyword argument 'allow_abbrev'

Hi,
Thanks for the bug report. Python 3.5 is required, but I neglected to indicate that in the Homebrew dependencies, my apologies. I suggest manually upgrading to python 3.5 or 3.6 with homebrew.
I'll also update the dependencies so that happens automatically in the future.

Oh, and additional bug this found: telepresence's automatic "do you want to file a bug?" code is not triggered if an error occurs during parsing arguments. So need to fix that too.

Fixed in master.

Thank you!

(Unfortunately it doesn't seem like Homebrew gives me any way to force an upgrade to newer Python, so I just added a nicer error on startup.)