Use Ordering in BinSearch over Bool

Why not use Ordering here rather than Bool?
https://github.com/meooow25/haccepted/blob/d7313e905988a5d1ef0f4666aa2b91169d48f26f/src/BinSearch.hs#L33-L39
Anyways, a very elegant way to do this is to traverse a lazy search tree!
data Bin a = Tip a | Bin a (Bin a) (Bin a)

bsearch ∷ (a → Ordering) → Bin a → a
bsearch f = go where
  go = \case
    Tip a → a
    Bin a l r →
      case f a of
        LT → go l
        GT → go r
        EQ → a

-- Starting with 0, grow exponentially until the
-- left branch is taken, then grow a perfectly balanced
-- binary tree between the extremes
--
ascend = Bin 1 (Tip 0) (go 2) where
  go n | x ← n+n =
    Bin x (interval (succ n) (pred x)) (go x)

interval lo hi =
  case size of
    0 → Tip lo
    1 → Bin lo  (Tip lo) (Tip hi)
    _ → Bin mid (interval lo (pred mid)) (interval (succ mid) hi)
  where
    size = hi - lo
    mid  = lo + div size 2

Here are some usecases:
-- O(log n).

 Ordering only sometimes allows exiting the search early on EQ.

Anyways, a very elegant way to do this is to traverse a lazy search tree!

That's an interesting suggestion. As far as I can see, the main benefit of this is increased flexibility because we are free to build the tree however we want. Choosing the split strategy or building a non-integral tree can't be done with the current code. However, I don't see a use case for this in competitive programming. Have you seen some problems where this is useful? Note that unbounded search (ascend), can be easily be done with the current implementation too.
-- Binary search with only a start index
binSearchUnbounded :: Integral i => (i -> Bool) -> i -> i
binSearchUnbounded f = go 1 where
    go d i
        | f i       = binSearch f (i - d + 1) (i - 1)
        | otherwise = go (2 * d) (i + 2 * d)

Now let's talk about what's wrong.
First, you have a bug, 2 does not exist in your ascend tree. This is easy to fix.
Second, your isqrt sometimes returns $\lfloor \sqrt x \rfloor$ and sometimes $\lceil \sqrt x \rceil$, which is not great. This is harder to fix, try it.
I have not inspected the Farey function.
The tree approach is also more verbose. Which is a price to pay for flexibility perhaps.
But it's also much slower (~4-7x for Ints depending on whether the tree is shared), which is a deal breaker. Benchmarks here. From what I can see this is primarily because GHC is able to unbox all the Ints in the current implementation, which it cannot do with a tree.
Overall, I am not inclined to add this.

Thank you for the elaborate response! I'll look into the performance problem, but for now, I'm pretty sure isqrt can be fixed like this?
isqrt' n | x <- isqrt n = if x*x > n then pred x else x

If I manually unbox the numbers, I beat your version by a landslide:
All
  isqrt0: OK (2.46s)
    813  ms ±  44 ms
  isqrt:  OK (0.57s)
    78.2 ms ± 2.6 ms

And here's your version unboxed, about twice as fast as mine:
binSearch1 f = go where
  go l h
    | 1# ← l ># h = I# l
    | f (I# m)    = go l (m -# 1#)
    | otherwise   = go (m +# 1#) h
    where m = (l +# h) `iShiftRL#` 1#

binSearchUnbounded1 f (I# i0) = go 1# i0 where
  go d i
    | f (I# i)  = binSearch1 f (i -# d +# 1#) (i -# 1#)
    | otherwise = go (2# *# d) (i +# 2# *# d)

isqrt1 n =
  binSearchUnbounded1 ((>n) . join (*)) 0 - 1

  isqrt0: OK (2.46s)
    813  ms ±  59 ms
  isqrt1: OK (0.26s)
    36.2 ms ± 1.4 ms
  isqrt:  OK (0.59s)
    81.4 ms ± 3.3 ms

I'm pretty sure isqrt can be fixed like this?

That's fixing it after the fact, which is an option sure, but it becomes one more thing to add on top of the search.

Here are two more functions to motivate the design:

natsLevelOrder and natsBinaryOrder are not binary search trees so I don't see how this is relevant.

If I manually unbox the numbers, I beat your version by a landslide:
And here's your version unboxed, about twice as fast as mine:
...

You may be benchmarking with optimizations off. GHC does this is kind of unboxing well by itself.
I'll summarize my views on this issue:

I'm open to adding binary searching via a Bin tree if there are competitive programming problems where this is useful. I can't think of any, so if you could share some that would be great.
 Ordering only sometimes allows exiting the search early on EQ.
This is very apparently from the difference in our tree's above. Your version has to walk all the way to the bottom of the tree before making a decision. Is this really faster? compare on numbers are at least in theory a single machine instruction!

This is equivalent to ascend, but interval is quite a bit quicker.
ascend2 = Bin 2 (Tip 1) (go 2#)
  where
    go n = Bin (I# x) (interval2 n) (go x)
      where x = n +# n

-- Creates the j'th left-subtree for ascend
interval2 j = go (iShiftRL# j 1#) j
  where
    go 0# _ = Z
    go n  i = Bin (I# x) (go n' i) (go n' x)
      where
        x  = n +# i
        n' = iShiftRL# n 1#

What flags are you using when you run your benchmark?

-O

Your version has to walk all the way to the bottom of the tree before making a decision? Is this really faster?

I'm not sure what you're comparing against. If it's against the Bin approach then it is faster because the tree is never manifested as data, and it's not related Bool vs Ordering.
I think this is getting off-topic, so I'll address the main points and close this off.

A binary tree approach
Adds flexibility, but no examples of usefulness in competitive programming.
Bool vs Ordering
This is by design. On a range of integers, it simply finds the point of change of the Bool. I consider that being able to exit the search early on EQ is not worth the added complexity and friction at the call site.

Sorry for the spam, I'm basically just using this as an excuse to play with search trees :D
I might open a PR if I make it significant faster!


Togglecase not working as expected

When I select text and invoke the Togglecase command, it changes the text case, then deselects and leaves the cursor at the end of the what I’d selected. So I have to reselect the text and invoke again to toggle the next case.
My only cases in the list are set to:
lowerCase
upperCase
So, it seems it should just toggle the selected case back and forth between upper- and lower-case.
If the text is uppercase, it converts to lowercase.
If the text is lowercase, it converts to uppercase.
If the selected text contains a word that has both upper- and lower-case letters (such as a capitalized word), nothing happens.
Any ideas?
Obsidian 1.2.8
Text Format 2.2.1
Thanks,
Russell

Hi, this issue should be fix in v2.2.2. Feel free to reopen this issue if you have any question.


docs(catalog): Remove WIP

Signed-off-by: Adam Harvey<EMAIL_ADDRESS>Hey, I just made a Pull Request!

Primary intent of this change is to ensure the first sentence of the catalog is not "Work in Progress" which gets displayed on NPM:

Also better aligns it with the catalog-backend in terms of how to really use it. Open for opinions on that wording of course!
No changeset since this doesn't need to be "released".
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

you seem to have missed the DCO for that :) ah the joys of signing

I blame GitHub. 😝

Want to fix plugin-catalog-react shown in the screenshot too? 😅


[9.x] Revert some Carbon::setTestNow() removals

Revert about half of cleanup PR https://github.com/laravel/framework/pull/40790 that removed "useless" calls to Carbon::setTestNow(). Now some tests are non-deterministic and can intermittently fail for new PRs being submitted:
e.g., https://github.com/laravel/framework/runs/5798854285
1) Illuminate\Tests\Cache\CacheFileStoreTest::testIncrementDoesNotExtendCacheLife
Expectation failed for method name is "put" when invoked 1 time(s)
Parameter 1 for invocation Illuminate\Filesystem\Filesystem::put('D:\a\framework\framework\test...da8a33', '1648892146i:2;', true) does not match expected value.
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'1648892145i:2;'
+'1648892146i:2;'

The mocked cache expiry was unexpectedly 1 second in the future.

Carbon::setTestNow(Carbon::now()) keeps time-based mocks and assertions deterministic. Otherwise if the clock's second ticks over mid-test, assertions and mockery checks can be run on an unexpected time of day.
tests/Validation/ValidationValidatorTest.php

Avoid putting Carbon::setTestNow(null) at the end of the test case method. If that test fails, Carbon::setTestNow(new Carbon('2018-01-01')) is now leaking into subsequent tests.
Worst case, wrap it in a try { /* assertions */ } finally { Carbon::setTestNow(null); }.

Ping @lucasmichot just FYI ^

For the record, Carbon 3 will be a bit more strict on this.
Carbon::setTestNow(Carbon::now()); is about freezing the time to make as if the whole test method was running instantly.
Typical case would be things like this:
$repo->getStore()->shouldReceive('put')->once()->with('foo', 'bar', 602);
$result = $repo->remember('foo', Carbon::now()->addMinutes(10)->addSeconds(2), static fn () => 'bar');

(see #41824)
Here the moment to be chosen as the mocked "now" does not matter but it's important that the time is actually frozen during the test.
Between the moment you call Carbon::now() in the test and the moment this date will be read and converted into a number of seconds, few microseconds will elapse. Then difference between this passed DateTime and the new "now" reference will be something like 601.999123 seconds. With Carbon 2 microseconds was brutally truncated, so such a test with no time mock would succeed most of the time and fail if the microsecond unluckily overflow to previous second. In Carbon 3, it will be always take microseconds into account and truncate the result so systematic failure.


error npm install --save @ionic-native/device

npm install --save @ionic-native/device
ionic-hello-world@ D:\Work\Code_NJYDZF\MobileApp
+--<EMAIL_ADDRESS>`-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (cu
rrent: {"os":"win32","arch":"x64"})
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
ionic info
global packages:
@ionic/cli-utils : 1.4.0
Cordova CLI      : 6.0.0
Ionic CLI        : 3.4.0

local packages:
@ionic/app-scripts              : 1.3.0
@ionic/cli-plugin-cordova       : 1.4.0
@ionic/cli-plugin-ionic-angular : 1.3.1
Cordova Platforms               : android 6.1.2
Ionic Framework                 : ionic-angular 3.0.0

System:
Node       : v7.9.0
OS         : Windows 7
Xcode      : not installed
ios-deploy : not installed
ios-sim    : not installed
npm        : 4.2.0

how to solve?please?

npm install --save @ionic-native/device
ionic-hello-world@ D:\Work\Code_NJYDZF\MobileApp
+--<EMAIL_ADDRESS>`-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (cu
rrent: {"os":"win32","arch":"x64"})
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

UNMET PEER DEPENDENCY<EMAIL_ADDRESS>

UNMET PEER DEPENDENCY @ ionic/app-scripts @ 1.3.0

Hello! Thanks for opening an issue with us! As this seems like more of a support question I will urge that you ask this question on our forum or on our slack channel. Thanks for using Ionic!


Button Component

Was adding to the Button Component:
Types:

primary
secundary
ghost
dashed

Variation of types:

danger

Sizes

sm
md
lg
xl

"Behaviors" Styles

focus
disabled
hover

Actions

click

Tests

types
danger
sizes

Don't forget doc in storybook.

You can update your branch and use theme to customize your button. 😄


Fix SSR ReferenceError with decorator metadata

Hi again,
like #853 this PR also solves issue #773. After researching a bit more I found a different solution. Because each solution might have different tradeoffs for you, I will leave both PRs open for now.
I removed the emitDecoratorMetadata option, which is only needed for JIT support, if I am not mistaken: https://github.com/angular/angular/issues/30586#issuecomment-494898597. So with Angular 9 it should not be needed.
The actual issue seems to be a TypeScript limitation, but I opened issue https://github.com/angular/angular/issues/37472 with the angular team, to check whether this is a regression to a workaround they have implemented. From that issue you can find more information from the Angular team in linked issues that describe the original problem very well.
Thank you!

Let's go with your other PR.


Added signup/login page

Added login page
Created states and Submit functions
Fields missing exception handled
Used material UI for inputs
Made Both components responsive
Created Routes(Navbar handling not done as yet)

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # <issue_number_here>

Domain of Contribution 📊

[x] Documentation
[ ] Android (Flutter)
[ ] Backend (Express)

Checklist ✅

[x] I follow Code of Conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GSSOC'22 contributor

Screenshots / Gif 📸
Link showing demo: https://www.recordjoy.com/view/s9lrW3HrTZjmzJmR48Ed

This is not accepted you literally copy pasted the same code which was written for v1. @AdnanMuzavor I am closing this issue.

Okay @vigneshshettyin .Sorry!! I wasn't knowing that you were expecting different code.
If you could have told that I wouldn't have done this.
Sorry for inconvenience.

Please don't repeat it this was mentioned by the GSSOC'22 team as well. @AdnanMuzavor

Yes I won't do it again @vigneshshettyin !!


Children elements listed here seem incorrect

<participants> doesn't seem to exist and it's missing a <profiles> entry that is a parent of <trackingProfile>

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

ID: 014c25d8-5aed-a2e3-b27a-958cc5475e27
Version Independent ID: cfe30baf-c0aa-ed70-d2a5-ef0ad547f547
Content: <tracking> of WCF
Content Source: docs/framework/configure-apps/file-schema/wcf/tracking-of-wcf.md
Product: dotnet-framework
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

@glennc @zhenlan did the participants element ever exist or is just plain wrong here?
This is what I get in my project:

@Lxiamail


Not compatible with IE browser

Not compatible with IE browser

Sorry about that☹. But this project has no plan to support IE browser, so I'll close this issue then.


[Admin.Orders] The invalid export time can be selected

Environment: Windows 10, Google Chrome 101
Reproducible: always
Build found: 17.05.2022
Preconditions
Go to https://ita-social-projects.github.io/GreenCityClient/#
Log in as admin
Steps to reproduce
1.Click the 'UBS-admin' button in pop-up window
2.Click the 'Orders' button
3.Change the 'Order status' in some order to 'Узгодження'
4.In the 'Export date' choose current day
5.In the 'Export time' choose invalid time
6.Fill in all required fields
7.Click the 'Save' button
Actual result
The invalid export time can be selected

Expected result
Only the future time can be selected

@Horokhov Hi, I've thought that this bug was caused due to your time format(you have USA format).Thus, when i try to reproduce it having setted USA format beforehands, format of time does not change on our pop-up.So, you have to add steps to reproduce

Preconditions
1.Go to https://ita-social-projects.github.io/GreenCityClient/#
2.Log in as admin
3.User should have Kyiv time zone on PC, 24 hour clock format.
Steps to reproduce
1.Click the 'UBS-admin' button in pop-up window
2.Click the 'Orders' button
3.Change the 'Order status' in some order to 'Adjustment'
4.In the 'Export date' choose "To" past time.
5.In the 'Export time' "From" choose any time.
6.Fill in all required fields
7.Click the 'Save' button
https://user-images.githubusercontent.com/108956780/217029243-4f4226fc-e6fd-4e5a-b0cb-200efbf80cc3.mp4

cannot reproduce

Bug is fixed


Support numpy/torch/tf/jax formatting for IterableDataset

Right now IterableDataset doesn't do any formatting.
In particular this code should return a numpy array:
from datasets import load_dataset

ds = load_dataset("imagenet-1k", split="train", streaming=True).with_format("np")
print(next(iter(ds))["image"])

Right now it returns a PIL.Image.
Setting streaming=False does return a numpy array after #5072

hii @lhoestq, can you assign this issue to me? Though i am new to open source still I would love to put my best foot forward. I can see there isn't anyone right now assigned to this issue.

Hi @zutarich ! This issue was fixed by #5852 - sorry I forgot to close it
Feel free to look for other issues and ping me or @mariosasko if you have questions :)
Also let us know if we can help find an issue that can correspond to what you're looking for


swagger-ui sends request with List<Map<String,String>> when controller annotated with "@RequestParam Map<String, String> params"

Please take the time to search the repository, if your question has already been asked or answered.

[Yes] What version of the library are you using? Is it the latest version? The latest released version is

What kind of issue is this?

[ ] Question. Is this a question about how to do a certain thing?

[ *] Bug report. If you’ve found a bug, spend the time to write a failing test. Bugs with tests or
steps to reproduce get fixed faster. Here’s an example: https://gist.github.com/swankjesse/6608b4713ad80988cdc9

[* ] spring xml/java config that is relevant
[ ] springfox specific configuration if it makes sense
[ ] include any output you've received; logs, json snippets etc.; and what the expected output should be
[ ] if you have a repo that demonstrates the issue for bonus points! See this example

[ ] Feature Request. Start by telling us what problem you’re trying to solve. Often a solution
already exists! Don’t send pull requests to implement new features without first getting our
support. Sometimes we leave features out on purpose to keep the project small.

Please do consider starring this repository if you do like this project and find it useful.

Not sure, you may need to file a bug with Swagger-ui


Support docker images

Docker images can take up a lot of space. List them and offer removing them (through official, supported ways).
A next step might be to also support volumes and maybe even containers.

I think this is a bit too niche of a use for BCU. Most devs will probably want to use the proper tools for managing these, and you don't end up with random docker images you don't know about.

You're totally right.
Just as with #421, I lost track about BCU's proper use case, sry.
As stated there, the main use case for me is identifying what (apps) take(s) up my disk space, but that's not the main goal of an uninstaller. :D
Sorry for the "noise".


Chore Remove ununsed be rowset CMakeLists.txt

Proposed changes
Remove ununsed be rowset CMakeLists.txt
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Add UFE token

Official announcement: http://ufedefi.world/
Project team background:
The UFE team was established in November 2019 and began design and development. The founder Martin,  graduated from British University with a master's degree in economics. CFA license holder, digital encryption currency enthusiast, one of the early builders of the European Bitcoin community, and angel investor in blockchain projects. He has participated in early investment in famous projects such as Ethereum, Ripple, and Polkadot. As the former head of global economics and strategy and managing director of AllianzGI, Martin has extensive experience and global vision in the fields of financial analysis, asset allocation, equity strategy, asset and bond investment, and digital currency investment.
Project basic information:
UFE is a distributed financial system based on blockchain. It is committed to promoting the development and improvement of the distributed financial DeFi ecosystem, creating on-chain finance, and then building an efficient distributed financial DeFi ecological infrastructure.The UFE open financial system platform aims to develop a complete cross-chain decentralized financial ecosystem, creating a decentralized wallet, lending, financing, transaction, prediction and other DeFi technology facilities, a cross-chain technology financial service ecosystem .Through the consensus of values, UFE DAO gathers a group of people who contribute their self-worth to achieve common goals, stand on the shoulders of their predecessors, and contribute to the Internet of Value.
Recommended Gas Limit (For ERC-20 project teams) for transaction:60,000

欢迎在社区中推荐用户使用 imToken 2.0 ，imToken 支持比特币, 以太坊, Comos, EOS, 波场, Nervos, 比特币现金, 莱特币，波卡等资产管理；同时支持去中心化币币兑换功能 Tokenlon；转账页面支持 ETH, USDT, EOS, BTC OTC 功能；另外对于没有 ETH 矿工费的用户，也提供 ETH 手续费充值功能。


CheckNameAvailabilityResult has field IsAvailalbe instead of IsAvailable for Azure Storage

CheckNameAvailabilityResult for storage accounts has a field called IsAvailalbe instead of IsAvailable.

@tomkerkhove Thanks for catching this issue and for the fix!

Merged. Will be available from v1.2


fix: delete dialog bug

Description
a small undefined variable bug.
Task Item
Closes #4511
Screenshots

Coverage increased (+0.004%) to 54.941% when pulling f955415aa0cd9d6ad71362d755738decfa57c2d1 on liweitian:deleteDialogBug into c4553695d60a09805aae0995f8bb380bbdeb0b44 on microsoft:main.


How to  autoheight UITableViewCell?

When i use Neon，I can't use this gramera to autoHeight UITableViewCell
self.tableView.rowHeight = UITableViewAutomaticDimension
self.tableView.estimatedRowHeight = 300
Could you fix the problem?
Thank you.

Unfortunately, Neon isn't built to handle that type of thing, it's a totally different mechanism.

This means,I can not use Neon in UITableViewCell?
Thank You.

You can use neon to layout the views inside a cell, but you need to calculate the height of the cell yourself.

Thank You，I know.

@Orangerc @mamaral is the a sample code snippet for creating TableViewCell with Neon? I am really struggeling with doing that :-/

What are you struggling with exactly?

I do set the height of the TV in the delegate methods in the viewcontroller. The cell does match the height but I really dont know how to perform something like the "layoutFrames()" function within the tableviewcell .. this is why I am looking for a sample codesnippet for a tableviewcell created with neon :-)

Set frames in layoutSubviews

Works perfect, thank you very much!

Glad to help!


{Question} Time-based expiry mechanism

Hi, I was wondering if you think this library would benefit from time-based cache invalidation mechanism expressed in seconds, something in lines of
    memoize :sleep, expire_after: 60

Basically, if expire_after key is found in method_names (as part of last Hash argument), there would be another variable to do a bookkeeping for, last time certain method was called (possibly with some arguments) and it's cache was skipped. When skipped, that time would be bumped to new Time.now value. That check be performed when resolving skip_cache variable, so that if last_access_time + expire_after <= Time.now skip_cache should be resolved to true, at least from that angle.
I could create PR + tests for that if you think its worthwhile, but preliminary benchmark results do show as much as 25-35% performance degradation (kinda expected, as there are some additional time-based checks). Maybe splitting implementation in memoize + timed_memoize/expirable_memoize could solve this and at least constrain performance degradation to methods memoized with this new approach (old methods would remain as-is).
I know this would increase code surface size or decrease perf (maybe there is a way to mitigate both 🤔 ), hence the question 😄

@github0013
yeah, I guess you are right, thanks for pointing to memery gem, it has exactly what I need 💯


Debounce function

I changed the proposed function type a little to match general tools already available. Also the return type is changed to (...args: Parameters<F>) => void  instead of F
resolves #44

Don't forget to add export to src/index.ts files

Thanks bro


Prepare Azure Monitor Ingestion for release

API change check
API changes are not detected in this pull request.


Remove BCP functionality from here

The package @cosmwasm/bcp was a successful experiment to show token contracts in a multichain wallet. There is no intend to add this functionality into a BCP based wallet like Neuma in production quality. In order to simplify our codebase, we plan the following steps:

[ ] Refactor @cosmwasm/faucet to work with @cosmwasm/sdk directly and remove the dependency on @cosmwasm/bcp. This implies the removal of smart contract token support from the faucet. It is simplifies to just support bank tokens, which is most important for fees.
[ ] Remove CosmWasm specifics (ERC20 smart contract token support) from @cosmwasm/bcp
[ ] Hand over @cosmwasm/bcp to IOV Core (under which target name?)

Mid-term additions

[ ] Split @cosmwasm/sdk into Cosmos SDK and CosmWasm functionality, such that CosmWasm becomes optional (#182)

@davepuchyr Number 2. and 3. is service work for IOV to express our gratitude for being able to build on top of IOV Core and maintain healty code for everyone in the ecosystem. Please folllow this ticket closely and chip in if I get something wrong.

This sounds good.
Happy to hand off the @cosmwasm/bcp experiment to iov-core if they want to maintain it, integrate it in their wallet or in any way make real use of it.
And yes, I also agree on refactoring the faucet to just use @cosmwasm/sdk (bank).

However, I am not sure about

Split @cosmwasm/sdk into Cosmos SDK and CosmWasm functionality, such that CosmWasm becomes optional

We are talking about making a new repo to be modular and extensible, pulling in parts of iov-core and cosmwasm-js and others and making sure this is extensible for other modules. I would leave these as is here (working for all out 0.38-based cosmwasm releases) and do the refactoring on the new branch which will have plenty of redesign tasks as well. Unless you think there is a benefit to refactoring here first. (In any case, this is a separate ticket - bcp stuff is a full :+1:)

As Ethan pointed out correctly, Split @cosmwasm/sdk into Cosmos SDK and CosmWasm functionality, such that CosmWasm becomes optional is independent of this ticket. I created https://github.com/CosmWasm/cosmwasm-js/issues/182 for that. The order of #179 and #182 does not really matter technically.

The line of PRs #183, #184 and #186 does step 4 and 2 first. So we have no ERC20 tokens on BCP and the faucet after that.
I postponed step 1 and 3 since it is not trivial to refactor the faucet. It required one of those two options:

Reimplement the token handling (COMS <-> ucosm) in the faucet
Only work on the native unit (e.g. ucosm) and adapt a lot of documentation

@davepuchyr Under which name would you like to have the BCP Cosmos SDK 0.38 package? @iov/cosmos-sdk?

@davepuchyr Under which name would you like to have the BCP Cosmos SDK 0.38 package? @iov/cosmos-sdk?

Yes, @iov/cosmos-sdk, please.

Package is prepared. Move depends on #202


Async Producer Hang

Versions
Sarama Version: cebb584
Kafka Version: 2.0.0
Go Version: 1.11
Configuration
Producer.Return.Errors = true
Producer.Return.Successes = true
Producer.RequiredAcks = sarama.WaitForLocal
ClientID = "somestring"
Version = sarama.V2_0_0_0
Logs
No logs found
Problem Description
I am running a process that produces messages to a to multiple topics and receives from a single process.  After some time the main async producer hangs indefinetly while sending a ProducerMessage to the input channel:
producer.Input() <- pm
I am able to generate a thread dump included below.  I tried to sort it by the output I thought was the most relevant towards the top and I removed things I did not think were necessary.
Please let me know if you need any additional information.  At this time I am unable to duplicate/correlate the hangs with an event.    Here is the what the mail loop looks like
for {
		select {
                // My internal channel
		case pm := <-p.inputChannel:
			producer.Input() <- pm
			messagesSent.Mark(1)
		case success := <-producer.Successes():
			messagesAcked.Mark(1)
		case err := <- producer.Errors():
			messagesSentErrors.Mark(1)
			log.Errorf("Received Error: %+v: ", err)
		case <-p.producerDone:
		        log.Debug("Stopping the producer loop")
			break Loop
		}
	}

same issue here, can anybody help?

There are some deadlock type issues on both producing and consuming.  The only advice I can give is not to enable producer successes.  Since we disabled that, we have not seen a hang.  I can't confirm that fixes it, but we have not seen a hang in months.  On the consumer side, I would not use async close.  I have found a similar deadlock issue when using that and have not seen it since using close.
I have been testing with the race detector and go does not flag it as a race condition which leads me towards sending/reading a nil channel.  However, I have not had time to debug since its very sporadic.  Hope that helps.

@lechnertech, this is very informative, thanks!

I'm going to close this issue.  Recommend disabling successes and processing input and errors in different goroutines.

I'm going to close this issue. Recommend disabling successes and processing input and errors in different goroutines.

we did by this, but still encounter deadlock. don't know why


Invite @rultor

@rultor, hello

@rultor, hello

@kerelape Hi there! I understand a few simple commands: "merge", "deploy", etc. This page explains them briefly. Any questions or bug reports are welcome. Have fun :)


For apptainer, move /tmp away from tmpfs

It has some issues with it being mounted as "nodev", so revert to using a normal file system for $TMPDIR.
$ limactl shell singularity singularity run -u -B $HOME:$HOME docker://alpine
INFO:    Converting OCI blobs to SIF format
WARNING: 'nodev' mount option set on /tmp, it could be a source of failure during build process
INFO:    Starting build...
Getting image source signatures
Copying blob 213ec9aee27d done
Copying config 29f453b10b done
Writing manifest to image destination
Storing signatures
2022/10/02 07:18:41  info unpack layer: sha256:213ec9aee27d8be045c6a92b7eac22c9a64b44558193775a1a7f626352392b49
2022/10/02 07:18:41  warn xattr{etc/shadow} ignoring ENOTSUP on setxattr "user.rootlesscontainers"
2022/10/02 07:18:41  warn xattr{/tmp/build-temp-98011911/rootfs/etc/shadow} destination filesystem does not support xattrs, further warnings will be suppressed
INFO:    Creating SIF file...
INFO:    Converting SIF file to temporary sandbox...
Apptainer>

Using /var/tmp is the recommended revert, from the Fedora instructions regarding the change (in fc18):
https://fedoraproject.org/wiki/Features/tmp-on-tmpfs
[anders@lima-singularity lima]$ findmnt /
TARGET SOURCE           FSTYPE OPTIONS
/      /dev/vda5[/root] btrfs  rw,relatime,seclabel,compress=zstd:1,space_cache=v2,subvolid=256,subvol=/root
[anders@lima-singularity lima]$ findmnt /tmp
TARGET SOURCE FSTYPE OPTIONS
/tmp   tmpfs  tmpfs  rw,nosuid,nodev,seclabel,nr_inodes=1048576,inode64

This wasn't a problem in Enterprise Linux 8, since it had /tmp on a regular file system:
[anders@lima-apptainer lima]$ df -h /var /tmp
Filesystem      Size  Used Avail Use% Mounted on
/dev/vda1       100G  2.2G   98G   3% /
/dev/vda1       100G  2.2G   98G   3% /
[anders@lima-apptainer lima]$ findmnt /
TARGET SOURCE    FSTYPE OPTIONS
/      /dev/vda1 xfs    rw,relatime,seclabel,attr2,inode64,logbufs=8,logbsize=32k,noquota

Needs rebase


Use multiprocessing in compute_cells_and_kzg_proofs

I'm curious how much faster CI tests finish with multiprocessing. Hoping to reduce CI test time.

make: *** [Makefile:129: citest] Error 137

This is a SIGKILL, which means it probably ran out of memory. Makes sense. There were 36 threads.
This isn't really practical and seems like a bad idea. Going to close.


New test to verify unclosed array

Test confirms bug report https://github.com/stleary/JSON-java/issues/372

Accepted. Merge after stleary/JSON-java#372 is merged.


Should be possible to update typings .lib file used by visual studio extension

We develop our project using visual studio, but use webpack instead of the vs extension to compile typescript. We do use the extension's great support for intellisense. However we recently ran into an issue where the typings used by the extension and the one in npm were different. It would be great if we could either update the typings or point to the version used by webpack.

In VS 2015 you can install the matching version through: https://www.microsoft.com/en-us/download/details.aspx?id=48593
In VS 2017 (currently available in the Preview channel) , this should be available through tools/options menu, see https://github.com/Microsoft/TypeScript/issues/14529 for relevant discussions.

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


chore: rename master branch references to main

Hey @piotr-oles!
If you're happy with this, I'd suggest we follow the instructions here: https://github.com/github/renaming
And then merge this PR.  I'm happy to do this, but I'll hold off until you've approved this PR first.  Or if you'd like to handle it - that's fine too!
✅ Closes: #582

Okay just did the rename.

If you have a local clone, you can update it by running:
git branch -m master main
git fetch origin
git branch -u origin/main main
git remote set-head origin -a

Merged! We should be done...

:tada: This PR is included in version 6.2.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.0.0-alpha.2 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Crashes on form post

https://drive.google.com/file/d/0B8vCgxfNO7E4N1FrUEhQN01DY1E/edit?usp=sharing

Please upgrade to PhantomJS 2. 1.9.x is no longer supported, and known to have many crasher bugs that are fixed in 2 version.


HTTP Error 500 from KeyError when image lacks src attribute

Describe the Bug

When checking the src value for images in pages that were just saved, the server assumes that this attribute exists. This is not necessarily the case.
Steps to Reproduce

Go to any page of any region or go to create a new one
Change the editor from WYSIWYG to HTML by clicking on the View menu and select Source view
Paste in <img>
Click on Update
See error

Expected Behavior

The page form view loads again, and maybe displays a hint that there are empty images
Actual Behavior

The server encounters an uncaught KeyError
Additional Information

Traceback
Apr 02 16:03:53 ERROR django.request - 500 Internal Server Error: /augsburg/pages/de/1/edit/
Traceback (most recent call last):
  File "integreat-cms/.venv/lib/python3.11/site-packages/django/core/handlers/exception.py", line 55, in inner
    response = get_response(request)
               ^^^^^^^^^^^^^^^^^^^^^
  File "integreat-cms/.venv/lib/python3.11/site-packages/django/core/handlers/base.py", line 197, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "integreat-cms/.venv/lib/python3.11/site-packages/django/views/generic/base.py", line 104, in view
    return self.dispatch(request, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "integreat-cms/.venv/lib/python3.11/site-packages/django/utils/decorators.py", line 46, in _wrapper
    return bound_method(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "integreat-cms/.venv/lib/python3.11/site-packages/django/contrib/auth/decorators.py", line 23, in _wrapper_view
    return view_func(request, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "integreat-cms/.venv/lib/python3.11/site-packages/django/views/generic/base.py", line 143, in dispatch
    return handler(request, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "integreat-cms/integreat_cms/cms/utils/tree_mutex.py", line 117, in innermost_function
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "integreat-cms/integreat_cms/cms/views/pages/page_form_view.py", line 305, in post
    if not page_form.is_valid() or not page_translation_form.is_valid():
                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "integreat-cms/.venv/lib/python3.11/site-packages/django/forms/forms.py", line 201, in is_valid
    return self.is_bound and not self.errors
                                 ^^^^^^^^^^^
  File "integreat-cms/.venv/lib/python3.11/site-packages/django/forms/forms.py", line 196, in errors
    self.full_clean()
  File "integreat-cms/.venv/lib/python3.11/site-packages/django/forms/forms.py", line 433, in full_clean
    self._clean_fields()
  File "integreat-cms/.venv/lib/python3.11/site-packages/django/forms/forms.py", line 448, in _clean_fields
    value = getattr(self, "clean_%s" % name)()
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "integreat-cms/integreat_cms/cms/forms/custom_content_model_form.py", line 178, in clean_content
    "Image tag found in content (src: %s)", image.attrib["src"]
                                            ~~~~~~~~~~~~^^^^^^^
  File "src/lxml/etree.pyx", line 2502, in lxml.etree._Attrib.__getitem__
KeyError: 'src'
Apr 02 16:03:53 ERROR django.server - "POST /augsburg/pages/de/1/edit/ HTTP/1.1" 500 20360

@PeterNerlich
I'm thinking what kind of hint message should be shown to users.
Many of them probably don't know what an  tag is and how it should look like. Then it probably doesn't help to say "The text contains a broken  tag. Please check it" or somehting like that if they don't know how to fix it 😓
But the content cannot be saved as long as there is an broken img tag 🤔 Should we remove broken  tags, save the content and tell users, for example, "Did you try to insert an image? Something went wrong. The page/location/event was saved without it. Please try again."

it probably doesn't help to say "The text contains a broken  tag. Please check it" or somehting like that if they don't know how to fix it 😓

It is not anything else than images, right? And an image without a source should be allowed by the HTML spec, AFAIK.
Can we just skip the image tag for media checking if the source is missing to prevent the issue and display a message that an image might be broken?


update mongo adapter

Added Close() method for mongo adapter

Coverage decreased (-0.03%) to 49.04% when pulling 19ea58c629522524f6cf11fd600fe725f7c114b5 on devghosh:mongo_update into ca56a0197fc678d4bfffa06f60f3a738ae1c38f1 on jabong:master.

Coverage decreased (-0.03%) to 49.04% when pulling 19ea58c629522524f6cf11fd600fe725f7c114b5 on devghosh:mongo_update into ca56a0197fc678d4bfffa06f60f3a738ae1c38f1 on jabong:master.


Cargo doesn't respect CARGO_TERM_COLOR for help messages

Problem
I always run with CARGO_TERM_COLOR=never, as I find colorized output very hard to read, nearly impossible to read with default term colors on many systems (both Windows & Linux), and of negligible value even if I tweak  the default term colors.
Color enable/disable for cargo sub-commands is all over the map, which I understand, since each sub-command is responsible for implementing a "--color" option and/or reading and respecting CARGO_TERM_COLOR. However, even cargo itself is ignoring color preferences, at least in help. I've tried in both:
cargo version
cargo 1.74.0 (ecb9851af 2023-10-18)
and
cargo version
cargo 1.76.0-nightly (623b78849 2023-12-02)

Steps

Set CARGO_TERM_COLOR=never
Run the command "cargo help"
Observe that the help message is colorized.

Possible Solution(s)
No response
Notes
No response
Version
Happens with both

>cargo version --verbose
cargo 1.74.0 (ecb9851af 2023-10-18)
release: 1.74.0
commit-hash: ecb9851afd3095e988daaa35a48bc7f3cb748e04
commit-date: 2023-10-18
host: x86_64-pc-windows-msvc
libgit2: 1.7.1 (sys:0.18.0 vendored)
libcurl: 8.4.0-DEV (sys:0.4.68+curl-8.4.0 vendored ssl:Schannel)
os: Windows 10.0.22621 (Windows 11 Professional) [64-bit]

and

>cargo version --verbose
cargo 1.76.0-nightly (623b78849 2023-12-02)
release: 1.76.0-nightly
commit-hash: 623b788496b3e51dc2f9282373cf0f6971a229b5
commit-date: 2023-12-02
host: x86_64-pc-windows-msvc
libgit2: 1.7.1 (sys:0.18.1 vendored)
libcurl: 8.4.0-DEV (sys:0.4.68+curl-8.4.0 vendored ssl:Schannel)
os: Windows 10.0.22621 (Windows 11 Professional) [64-bit]

See #13052 and #9012, and we track that in #9012. Cargo respects NO_COLOR=1 btw.

Thanks. I'll close this in favor of #9012. Probably tweak the issue title to make it more obvious.


before isnt executed

When using the example (kinda; see the simple example with when then; aka the only example with Calculator) from pub.dev
then the calculator isnt initialized and throws
given("emtpy file", (){
    MemoryFileSystem mfs;
    late File file;

    print("rin");
      before((){
        print("before");
        mfs = MemoryFileSystem.test();
        file = mfs.file("/test.json");
      });

      when("storing a valid magician", () async {
        var store = FileMagician(file: file);

        assert(file.lengthSync() == 0);

        then("the file should contain data", (){
            expect(file.lengthSync() > 0, isTrue);
        });
      });
  });

Running the test throws bcs before wasnt executed.
I would expect:
that before gets called.
Deps:
  given_when_then_unit_test: ^0.0.4
  mocktail: 0.2.0

After removing the before call and just calling both constructors,
the then case throws
StateError (Bad state: Can't call test() once tests have begun running.)

StackTrace.current
#0      Eval ()
#1      Declarer._checkNotBuilt (package:test_api/src/backend/declarer.dart:319:5)
#2      Declarer.test (package:test_api/src/backend/declarer.dart:164:5)
#3      test (package:test_core/scaffolding.dart:141:13)
#4      then (package:given_when_then_unit_test/res/then.dart:63:3)
#5      main.<anonymous closure>.<anonymous closure> (file:///test.dart)
<asynchronous suspension>

Replacing
Future<null> body = () async { /*no replication*/}; // body of  "when("storing a valid magician", body)" see above
when("storing magician", body);
//with
group("storing magician", body);

leads to Declarer from test_api-0.4.2 in file /lib/src/backend/declarer.dart to throw an ArgumentError('Groups may not be async.') .

This leads to the issue.
In this case when calls (body)[https://github.com/AndrewPiterov/given_when_then_unit_test/blob/b2bb103b8edccdc95f3733c817ae43b8e8bd257e/lib/res/when.dart#L246] without regard that it might be a future.
In the initial code the test will not be registered against the declarer. Making matters worse the body passed to group will be synchronous and return null.
group will be unable to detect the async body. (see)[https://github.com/dart-lang/test/blob/767dbb5888ab5376ef0319006f7c90b028c1dbf8/pkgs/test_api/lib/src/backend/declarer.dart#L244]
I could imagine two paths. Either we check for async bodys and throw or add an async impl for when.

FileMagician

Hello @Hu1buerger ,
thanks for writing.
Could you please provide full code for this test? I cant find some class and method.


wrong styling on login button while hovering

while we hover on login button it has wrong styling it's border blur should be equal in all dimensions.

Hi Prajjwal, I wanted to work on this issue, Can you clarify it more


Returning the notification in markdown for Gitter

Fixes https://github.com/OpenCollective/OpenCollective/issues/160

I like the new 'format' passing through.

Are you sure gitter notifications can take markdown? I can't find any documentation that the side bar can show links

Can you pls add the format types to constants, instead of hard-coded strings all over the codebase?


Public Domain

Fixes #36

can we change the copyright to be 'contributors' ?

The term "contributors" is very unclear. Does it only include the people who GitHub lists as contributors? Or somebody who contributed and idea or feedback also? Or somebody who did some design work and didn't commit the files?
After some discussion, I think the best solution is to just put everything Open Austin does into the public domain.
So, I'm updating this pull request to use the text from Unlicense.com

:deciduous_tree:
 isn't it breaking the GPL to remove it?</>


Fixes race condition

Wait for server to send next valid order id before sending requests.
Sending requests prior to this may cause the server to close the connection.

Codecov Report

Merging #3 (57664c8) into main (fa6bf08) will decrease coverage by 0.25%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main       #3      +/-   ##
==========================================
- Coverage   18.03%   17.77%   -0.26%
==========================================
  Files           5        5
  Lines         549      557       +8
==========================================
  Hits           99       99
- Misses        445      453       +8
  Partials        5        5

Impacted Files
Coverage Δ

client.go
0.00% <0.00%> (ø)

transport.go
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Need to run play_url twice on first connection

I noticed when executing play_url the first time I only hear the ding from Google Home, a few seconds later I'll run the code again and and it will play the URL. It works then on, but after a while when it disconnects if I want to play the sound again I'll have to run play_url twice. Am I doing something incorrectly? Or is there a way to optimize the pichromecast.py to play the file immediately?
This is the error I get on first execution. It does not show up in subsequent executions and works as expected.
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<stdin>", line 27, in play_url
IndexError: list index out of range

I wanted to update here that first fix doesn't work quite well. I've since discovered a better way to run it. When attempting to send the message to Google to play the url the code loops twice, exiting the first time if successful. The main difference in the adjustment I made is to read_message 4 times instead of 3. On the 4th one look for one of your metadata properties that you wrote to Google. If it's there then exit. If it's not send the message again.
for i in range(2):
self.s.write(pack(">I", len(msg)) + msg)
for _ in range(4):
status = self.read_message()
if status.find(b'"title":"METADATA TITLE PROPERTY"') != -1:
return True #successfully played sound
return False #if we arrive here the sound doesn't work properly

I found a cleaner solution to this.
replace line 27 with this:
transport_data = sock.read(struct.unpack(">I", sock.read(4))[0]).split(b'"transportId"')
if len(transport_data) == 1:
    # if google device idle or in other active session, there is an index length error.
    # This check resolves this issue, and unpacks more data to grab transportId.
    transport_data = sock.read(struct.unpack(">I", sock.read(4))[0]).split(b'"transportId"')
    transport_data = sock.read(struct.unpack(">I", sock.read(4))[0]).split(b'"transportId"')
transport_id = transport_data[1].split(b'"')[1]


use different function for same url path not work in blueprints

when use app.route, it works well:
@app.route('/', methods=["GET"])
async def get(request):
return text('get')
@app.route('/', methods=["POST"])
async def post(request):
return text('post')
curl -XGET http://localhost:8000                 returns get
curl  -XPOST http://localhost:8000             returns post
when use blueprints,
curl -XGET http://localhost:8000/bp           returns  get
curl -XPOST http://localhost:8000/bp        returns  Error: Method POST not allowed for URL /bp
curl -XPOST http://localhost:8000/bp/       returns post
If I want to use http://localhost:8000/bp, what can I do?

@zkdfbb The following code works. What do you have set for strict_slashes?
from sanic import Blueprint, Sanic
from sanic.response import text

app = Sanic()

@app.route('/', methods=["GET"])
async def get(request):
    return text('get')

@app.route('/', methods=["POST"])
async def post(request):
    return text('post')

bp = Blueprint('bp', url_prefix='/bp')

@bp.route('/', methods=["GET"])
async def bp_get(request):
    return text('get')

@bp.route('/', methods=["POST"])
async def bp_post(request):
    return text('post')

app.blueprint(bp)

if __name__ == "__main__":
    app.run(host="<IP_ADDRESS>", port=8000)

$ curl -XGET http://localhost:8000/bp
get
$ curl -XGET http://localhost:8000/bp/
get
$ curl -XPOST http://localhost:8000/bp
post
$ curl -XPOST http://localhost:8000/bp/
post

@scottwernervt
I installed the code from git and it works~~
thank you ~

My bad, didn't realize I was testing against master branch. I recommend following #1170 so you can move off master branch when a new release is cut.


failed to compute cache key: failed to walk /var/lib/docker/tmp/buildkit-mount

edit by j143:
 #4 [2/2] ADD target/comprenhensive-0.0.1-SNAPSHOT.jar comprenhensive-0.0.1-SNAPSHOT.jar
#4 sha256:130e617710b6b80fb66fffd2ef522804becac665c17b1bdb384c76e2364b2839
#4 ERROR: failed to walk /var/lib/docker/tmp/buildkit-mount060609382/target: lstat /var/lib/docker/tmp/buildkit-mount060609382/target: no such file or directory ------
> [2/2] ADD target/comprenhensive-0.0.1-SNAPSHOT.jar comprenhensive-0.0.1-SNAPSHOT.jar: ------ failed to compute cache key: failed to walk /var/lib/docker/tmp/buildkit-mount060609382/target: lstat /var/lib/docker/tmp/buildkit-mount060609382/target: no such file or directory [Pipeline] } [Pipeline] // withEnv [Pipeline] } [Pipeline] // script [Pipeline] } [Pipeline] // stage [Pipeline] } [Pipeline] // withEnv [Pipeline] } [Pipeline] // node [Pipeline] End of Pipeline
ERROR: script returned exit code 1 Finished: FAILURE

error is not relevant to this repository.


Create and download package from remote instance

Imagine:
gradle aemGrab -Paem.grab.filter=/content/some/subtree -Paem.grab.name='Subtree package name'` -Paem.grab.target=backup-dir
this will create package on remote server, build it then download to local directory
will be a base for #51

this will require breaking change, task renaming:
aemBuild -> will create package on remote instance (currently it builds and deploys)
aemDevelop -> will be an alias for build aemDeploy aemAwait

there is no value for separating aemBuild and aemDownload ; aemDownload should build then download package so that actually used aemBuild command will remain untouched and there will be no breaking change

This feature is already covered by simply running aemCheckout aemCompose 😀

in my opinion, aemDownload should extend Gradle ZIP task and produce almost same ZIP as aemCompose but with configured predefined classifier download
aemCompose will produce zip 'example-app-design-1.0.0.zip aemDownload will produce zip 'example-app-design-1.0.0.download.zip
the difference between this files will be that aemDownload will assemble almost empty CRX package with minimal structure, then after uploading and building on AEM, package will be downloaded and extracted into local FS. the content of these package may differ.
what is more aemDownload must consider additional configuration options:

-Paem.download.filterPath=design.xml will cause that basing on file META-INF/vault/design.xml minimal package will be assembled locally, then uploaded to AEM, then build, finally downloaded and unpacked (by default it will reuse default filter.xml which is used to compose package). it will be nice if classifier .download.zip will be updated accordingly to .design.zip (based on filter name)
-Paem.force , download will firstly remove existing paths, then files from downloaded package will replace them, this is unsafe operation so that it must be specified force option to prevent data loss

all these options are negotiable. the intention is to cover as many use cases as it is possible.

Starting development on this one. Taking as a base work already done by @ergo123 https://github.com/Cognifide/gradle-aem-plugin/pull/206/commits/08fa98235ae4d1f7e0a45a8706ac33838774d083

Pull request with finalized implementation: https://github.com/Cognifide/gradle-aem-plugin/pull/230


Sandbox limits increase request for icrate

Crate name
icrate
Build failure link
https://docs.rs/crate/icrate/0.0.2/builds/738837
Requested RAM limit
6GB
Requested timeout
N/A
Requested number of targets
N/A
Why your crate needs the resource increases
icrate contains automatically generated code for many of Apple's libraries/frameworks (the Apple-equivalent to windows-sys), and so takes a lot of RAM to build (in practice, each framework is feature-gated, users wouldn't actually enable all the frameworks at once).
In a local build on macOS I saw the RAM usage go up to about 4.3GB, but I suspect that as the number of frameworks I want to support goes up, so will this number - hence the request for 6GB for now.
Thanks for running docs.rs, it is a very useful service!

I just increased the limit and queued a rebuild


Repositories update; new stats and latest queries, abstract baseclass

Fixes #16
Fixes #18
Fixes #19
Fixes #12

TODO:

Exporters must expect BaseFile objects.

Stats()

count number of files
totalsize total size on disk of files


[Bug] Requires at least two episodes to render site

Expected behavior
Should be able to render the site with less than two episodes.
Actual behavior
With less than two episodes rendering fails with an error message in themes/partials/row.html:112
Reproduction Steps
Starting from scratch, work through hugo quickstart getting started. Install castanet theme, run hugo server -D.  Notice the error. Run "hugo new episode/myepisode.md". Notice the error in the server output. Run "hugo new episode/myepisode2.md". Notice the website now renders.
Operating System and Version
Windows 7 (cygwin)
Web Browser and Version
n/a
Hugo Version
hugo v.0.36..1

Good catch. I'll have to add some catch logic to not try to display the remaining episodes.

I found the bug while building our podcast http://khabardaarpodcast.com/  Fortunately, we decided to launch with an episode zero teaser at the same time as our episode one. (i.e. two episodes at launch).  So we jumped over the hurdle.  But, a fix will be needed for anyone else that wants to start using the theme.

Only fails with row layout, just FYI.


Update to 1.7.4

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Add a drop down for choosing colors.

Presently users have to choose colours from a list of radio buttons. Although it works fine, but the problem with this approach is that the list takes up a lot of space which could have been used otherwise. The goal of this issue is to replace the list of radio buttons by a drop down. As in a drop down the list will appear on demand, space will be saved.
Please refer to the suggestion made by @Aniq55 in #28

hey, thats another one i can fix,
but i would do it not exactly like @Aniq55 mentioned in #28
i think we can save the button, so when u pick a color from the dropdown menu it changes the inputifeld of the hex color to the color choosen in the dropdown, and when u click go it changes the color to the its inside the inputfield, so u can choose color from the dropdown OR choose a 'custom' one.
is that a good idea or do u want the second button?
and what colors the dropdown should have? the same like the radiobuttons or more?

have a look at it, thats how i meant it, if it's fine i would do a pr, should i do another button like Aniq55 mentioned?
didn't get an answer so i just experimented a bit for practice ...
greets

@LoneOne123 can you please send a PR so that we can test it out once, after that we can let you know how it looks like. And apologies for taking so long to reply, was busy with some other stuff.

@LoneOne123 that's fine. Don't forget to add a screenshot in the comments of your PR.


Local Git repository shows "404 The entry or revision was not found in the repository" error.

The repository to the server is working fine, but when I connect it as a project repository, I see a message (see the title). Redmine support recommends changing the "scm_git_command" in "configuration.yml" file, but this cannot be done by the container settings. I took the redmine settings folder out of the container and mounted it in the container. However, after restarting the container, all my configuration changes made in the file disappear. What should I do to use the Git repository as a repository for the project?

Aleksei, does scm_git_command just need to be set to something reasonable or would it need to be customized per person?
Do you have a recommendation for what you'd like it set to?

Aleksei, does scm_git_command just need to be set to something reasonable or would it need to be customized per person?
Do you have a recommendation for what you'd like it set to?

Hi! Thank you for your quick response! I need to write the value "/usr/bin/git "to the parameter "scm_git_command".

Hi! Thank you for your quick response! I need to write the value "/usr/bin/git "to the parameter "scm_git_command".

Huh, it seems odd that the default doesn't just work.
Both the root user and redmine user git commands look at /usr/bin/git
root@75e7de661573:/home/redmine/redmine# which git
/usr/bin/git
root@75e7de661573:/home/redmine/redmine# su - redmine
redmine@75e7de661573:~$ which git
/usr/bin/git

I suspect the problem lies somewhere else.  Especially considering the git version is correctly reported in the admin info page.
SCM:
  Subversion                     1.9.3
  Mercurial                      3.7.3
  Cvs                            1.12.13
  Bazaar                         2.7.0
  Git                            2.29.0

Huh, it seems odd that the default doesn't just work.
Both the root user and redmine user git commands look at /usr/bin/git
root@75e7de661573:/home/redmine/redmine# which git
/usr/bin/git
root@75e7de661573:/home/redmine/redmine# su - redmine
redmine@75e7de661573:~$ which git
/usr/bin/git

I suspect the problem lies somewhere else.  Especially considering the git version is correctly reported in the admin info page.
SCM:
  Subversion                     1.9.3
  Mercurial                      3.7.3
  Cvs                            1.12.13
  Bazaar                         2.7.0
  Git                            2.29.0

I ran through the git repo setup instructions and was able to make it work. https://www.redmine.org/projects/redmine/wiki/RedmineRepositories#Git-repository
Setup test repo
mkdir /srv/docker/redmine/git
cd /srv/docker/redmine/git
git clone --mirror https://github.com/sameersbn/docker-redmine.git
# Make sure permissions are accessible by redmine user, which is user 1000 on my machine
chown 1000:1000 -R .

Mount directory in docker as volume to /git
 $ tail docker-compose.yml
    ports:
    - "10083:80"
    volumes:
    - /srv/docker/redmine/redmine:/home/redmine/data
    - /srv/docker/redmine/redmine-logs:/var/log/redmine
    - /srv/docker/redmine/git:/git

Setup new repo

Repo loaded

I ran through the git repo setup instructions and was able to make it work. https://www.redmine.org/projects/redmine/wiki/RedmineRepositories#Git-repository
Setup test repo
mkdir /srv/docker/redmine/git
cd /srv/docker/redmine/git
git clone --mirror https://github.com/sameersbn/docker-redmine.git
# Make sure permissions are accessible by redmine user, which is user 1000 on my machine
chown 1000:1000 -R .

Mount directory in docker as volume to /git
 $ tail docker-compose.yml
    ports:
    - "10083:80"
    volumes:
    - /srv/docker/redmine/redmine:/home/redmine/data
    - /srv/docker/redmine/redmine-logs:/var/log/redmine
    - /srv/docker/redmine/git:/git

Setup new repo

Repo loaded

Note according to this post, https://www.redmine.org/boards/2/topics/41094?r=44138
A 404 error could happen if the git repo is empty as well.

Note according to this post, https://www.redmine.org/boards/2/topics/41094?r=44138
A 404 error could happen if the git repo is empty as well.

Thank you for reply!

Thank you for reply!


DistributedTests: Error "Value cannot be null. Parameter name: uriString"

I am getting this error when trying to run CodedUITests on a windows 7 virtual machine using the DistributedTests task.  The tests are found but never execute and I am hit with the following error message:
DistributedTests: UnExpected error occured during test execution. Try again.
DistributedTests: Error : One or more errors occurred.--->Value cannot be null.
Parameter name: uriString
TFS 2015 update 3 On Premises
worker_logs.txt
DTAExecutionHost.txt
agent_20170308-230218-utc.txt

@ppptrevoro This is something not expected. Seems like you are using Visual Studio Test Agent 2015 Update 2. Can you please install Update 3 and try running the tests.

Closing this issue for now. Assuming @nigurr 's suggestion worked. Please re open if this doesn't work


Create documentation website

I've established a gh-pages orphan branch to publish generated documentation into.  What is unclear to me at the moment is if these will publish to https://jarad.github.io/pyritas or not.
@jarad , you may need to review the Settings tab -> Pages configuration here and see if you need to set some over-ride for it to work given the CNAME setting in your main website repo.
Or I guess, if a different approach is desired, I would be happy to know current thoughts :)

I've activated the Pages configuration to use the gh-pages repository and it shows up at https://www.jarad.me/pyritas/


UI Modes

There should be a few modes the user can switch between that will govern not only the way panes are added to a screen, but also different shortcuts and the way they behave.
The first mode is tmux mode: https://github.com/imsnif/terminal-grid/issues/2
There should be an easy UI shortcut and visual indication for what mode is on in every screen (a toggle bar that becomes visible when switching between mode - reminiscent of the alt-tab bars?)

Implementation draft:
Every screen has an instance of a mode.
Changing a screen's mode can be done by unreffing the instance and creating a new one for that screen.
Every mode has:
Actions: move pane, resize pane, switch pane directional, add pane, minimize all panes. (maybe another special action or two).
Utilities: import window / export window
constructor: given a screen index and a screen-grid instance, it will rearrange all panes in the screen according to its logic.
Shortcuts will be identical for all actions (so that in every mode, they do something appropriate for that mode - ideally as similar as possible to actions in different modes in order to create flow and usability).
Global actions (these are actions that are not dependent on any pane focus):
Switch focus to pane id, open pane at screen index

Implemented.


Documentation updates

Updates the documentation instructions regarding CUDA and JAX installations and switches the theme to furo

Documentation preview: https://sphericart--99.org.readthedocs.build/en/99/


adding movies

hello
, please tell me movies like are being added, the time of addition is displayed, but the number in the database does not increase.
I made the settings as advised without restrictions. what could be the problem? thank you very much in advance

Здравствуйте я не покупал ключ для добавления всех фильмов Можно ли его употреблять сегодня

Hello @Gagik39, please use English and look at this solved issue, it shows how to add and update movies in CinemaPress.


qt6: 6.4.2 -> 6.4.3

Description of changes
Release note: https://code.qt.io/cgit/qt/qtreleasenotes.git/about/qt/6.4.3/release-note.md

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 221488 run on x86_64-linux 1

1 package marked as broken and skipped:

obs-studio-plugins.obs-hyperion

1 package failed to build:

obs-studio-plugins.obs-ndi

105 packages built:

adwaita-qt6
alice-tools-qt6
bambootracker-qt6
beamerpresenter
beamerpresenter-poppler
calcmysky
calibre
copyq
cudaPackages.nsight_compute
cutemaze
dolphin-emu
duckstation
fcitx5-chinese-addons
fcitx5-configtool
fcitx5-unikey
fcitx5-with-addons
gpxsee-qt6
gqrx
gqrx-gr-audio
gqrx-portaudio
hydrus
jami
khoj
kime
ladybird
libsForQt5.fcitx5-qt
maestral-gui
mediaelch-qt6
mozillavpn
obs-studio
obs-studio-plugins.advanced-scene-switcher
obs-studio-plugins.droidcam-obs
obs-studio-plugins.input-overlay
obs-studio-plugins.looking-glass-obs
obs-studio-plugins.obs-gstreamer
obs-studio-plugins.obs-livesplit-one
obs-studio-plugins.obs-move-transition
obs-studio-plugins.obs-multi-rtmp
obs-studio-plugins.obs-nvfbc
obs-studio-plugins.obs-pipewire-audio-capture
obs-studio-plugins.obs-source-record
obs-studio-plugins.obs-vaapi
obs-studio-plugins.obs-vkcapture
obs-studio-plugins.wlrobs
pgmodeler
pokefinder
prismlauncher
punes-qt6
python310Packages.pyqt6
python310Packages.pyqt6-charts
python310Packages.pyqt6-webengine
python311Packages.pyqt6
python311Packages.pyqt6-charts
python311Packages.pyqt6-webengine
qalculate-qt
qgnomeplatform-qt6
qmmp
qt6.full
qt6.qmake
qt6.qt3d
qt6.qt5compat
qt6.qtbase
qt6.qtcharts
qt6.qtconnectivity
qt6.qtdatavis3d
qt6.qtdeclarative
qt6.qtdoc
qt6.qthttpserver
qt6.qtimageformats
qt6.qtlanguageserver
qt6.qtlottie
qt6.qtmultimedia
qt6.qtnetworkauth
qt6.qtpositioning
qt6.qtquick3d
qt6.qtquick3dphysics
qt6.qtquicktimeline
qt6.qtremoteobjects
qt6.qtscxml
qt6.qtsensors
qt6.qtserialbus
qt6.qtserialport
qt6.qtshadertools
qt6.qtspeech
qt6.qtsvg
qt6.qttools
qt6.qttranslations
qt6.qtvirtualkeyboard
qt6.qtwayland
qt6.qtwebchannel
qt6.qtwebengine
qt6.qtwebsockets
qt6.qtwebview
qt6Packages.kcoreaddons
qt6Packages.poppler
qt6Packages.qtpbfimageplugin
qt6Packages.quazip
qt6Packages.qxlsx
qutebrowser-qt6
retext
stellarium
tdesktop
texworks
zeal-qt6
zint

Tested running telegram-desktop.


401 on start

When i open webpage locally i get error:

https://cds.twilio.com/v2/Streams Failed to load resource: the server responded with a status of 401 (Unauthorized)

screenshot
My credentials in config.php are correct. How can i fix it?

I was getting the same behavior.  I tried the ruby and node "ipm-" demos as well;  same result.
I found this demo, which does work:
https://github.com/twilio/ip-messaging-demo-js

This was a difficult issue to troubleshoot.
I finally found what was the issue. It looks like with the client expects push credentials sid under the grant hash in the JWT token.
I modified the token.php file to the following:
<?php
include('./vendor/autoload.php');
include('./config.php');
include('./randos.php');

use Twilio\Jwt\AccessToken;
use Twilio\Jwt\Grants\IpMessagingGrant;

// An identifier for your app - can be anything you'd like
$appName = 'TwilioChatDemo';
// choose a random username for the connecting user
$identity = randomUsername();
// A device ID is passed as a query string parameter to this script
$deviceId = $_GET['device'];
// The endpoint ID is a combination of the above
$endpointId = $appName . ':' . $identity . ':' . $deviceId;

// Create access token, which we will serialize and send to the client
$token = new AccessToken(
    $TWILIO_ACCOUNT_SID,
    $TWILIO_API_KEY,
    $TWILIO_API_SECRET,
    3600,
    $identity
);

// Grant access to IP Messaging
$grant = new IpMessagingGrant();
$grant->setServiceSid($TWILIO_IPM_SERVICE_SID);
$grant->setPushCredentialSid('CRe9c5eff29e744709d7df875f8a797bf0');
$grant->setEndpointId($endpointId);
$token->addGrant($grant);

// return serialized token and the user's randomly generated ID
echo json_encode(array(
    'identity' => $identity,
    'token' => $token->toJWT(),
));


[Bug] TypeError: Failed to execute 'fetch' on 'WorkerGlobalScope': Failed to parse URL from app/monaco/vs/language/json/jsonWorker.js

https://github.com/microsoft/monaco-editor/issues/4778

seems to maybe have something to do with workers for json and css e.g.
When using sql mode - there is no issue.


Add page models

@akmiller01 if you run the requirements_dev.txt pip install in your local virtualenv you should be able to add functional tests for your app models. I'll be making a test runner on this branch: https://github.com/IATI/preview-website/tree/create-testrunner


Upload fails with "Build has already finished, uploads rejected"

Since few days, we start seeing on one our CI the following message:
{'detail': ErrorDetail(string='Build has already finished, uploads rejected.', code='not_found')}
404

on the command: bash <(curl -s https://codecov.io/bash) -Z -F gpu.
We have multiple CI's running various non-overlapping tests and we upload reports with different tags. Job can end at different times and send reports within 1h.
Is it now an expected behaviour to reject uploads ? How then it is determined the end of the build such that it becomes finished ?
Thanks

Hi @vfdev-5, we have shipped a fix for this now. let me know if you are still running into issues

Seems like it is better now, thanks !


Is this project discontinued?

I don't think it's discontinued, but it's not getting enough market attraction.


fix typo

Codecov Report

Merging #778 (3eba08e) into main (2e3966f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #778   +/-   ##
=======================================
  Coverage   19.96%   19.96%
=======================================
  Files         189      189
  Lines       24542    24542
=======================================
  Hits         4899     4899
  Misses      18788    18788
  Partials      855      855

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 2e3966f...3eba08e. Read the comment docs.


Update persistent-volume.md

update product name, removing Hybrid

@guanghuthegreat Again, thank you -- we are currently working our way through all existing topics with the rebranding. #sign-off


Configure CircleCI workflows

Originates from https://github.com/open-telemetry/opentelemetry-js/pull/39 (Adds CircleCI configuration).

I opted to not set up workflows considering that the repository is still pretty simple and I imagine build/test times will be short for a while. I'll happily spend more time on it when testing parallelization is needed.

Adding some context: When you use workflows, file state is not maintained by default so you have to yarn install at every step, which is not great. Even if you restore from cache, you lose the state in the next build step. You need to set up data sharing using workspaces. The nodejs workflow example CircleCI provides does not work.

Not sure I understand why the CircleCI example doesn't work. Caching can be done at the job level.

It's a case of what's cleaner to do in the config file. You can use workspaces after the initial cache get->install->cache to share your dependencies across all jobs, or add a cache get line to every job description.

It's actually discouraged to share cache across jobs that run in parallel to avoid race conditions. It's also better in general IMO to cache per-job since it avoids having to reinstall everything for a small change.


Some minor doc additions/improvements

Make my original "suggest a feature" text less negative and more welcoming
Add a work in progress warning
Link out to our zulip chat for getting help

pre-commit.ci autofix


PlugInstall fails with Git Version Requirement and missing :endif

This is a new installation on a new Windows 10 computer using Vim 8.2 and Git 2.32.0.   PlugInstall gives the following error:

Type:

[x ] Bug
[ ] Enhancement
[ ] Feature Request
[ ] Question

OS:

[ ] All/Other
[ ] Linux
[ ] OS X
[x ] Windows

Vim:

[ ] Terminal Vim
[x ] GVim
[ ] Neovim

Getting this same issue on a new macOS Big Sur computer.

I had the same problem after updating to Big Sur. I ran :PlugUpgrade then :PlugUpdate while in vim and that fixed everything for me.

I had the same problem after updating to Big Sur. I ran :PlugUpgrade then :PlugUpdate while in vim and that fixed everything for me.

3q，its works for me（It bothered me for two hours)

Please try restarting Vim when you see an error regarding temp files.


mock all loadable libraries

Adding more libraries for lazyloading is not evaluated in the mock of Loadable, my suggestion is to use config.settings.loadables in mock where the lazy loadable libraries are actually registered.
i.e.
https://github.com/RedTurtle/design-volto-theme/blob/master/src/config/italiaConfig.js#L112

@mamico puoi mergiare quando vuoi

Merging, per @sneridagh instructions


Full covariance matrix in predictions

All the GPflow methods can now compute the full covariance matrix at predict time. The GPmodel class has a function predict_f_full_cov.
For convenience I added predict_f_samples, which returns samples from the corresponding distribution.
In addition, the conditionals.py file has been heavily refactored. Old useage exists but raises deprecation warnings.
Since there is nre functionality and also some refactoring, I have increased the version number to 0.1.2, and changed RELEASE.md accordingly.
This fixes #25 . If this is accepted, please file a new issue to remind us to remove the deprecated code in a future release.

Current coverage is 95.91%

Merging #27 into master will increase coverage by +0.33% as of 912376d

@@            master     #27   diff @@
======================================
  Files           20      20
  Stmts         1064    1053    -11
  Branches         0       0
  Methods          0       0
======================================
- Hit           1017    1010     -7
  Partial          0       0
+ Missed          47      43     -4

Review entire Coverage Diff as of 912376d

Powered by Codecov. Updated on successful CI builds.

Hi @jameshensman
looks good in general. I've added a few questions as line comments.
Alex


SPM 3.0.2 compilation fixes: import Foundation, () for Void

This PR addresses two issues I was seeing building with SPM & Swift 3.0.2. Issue #1 was macOS/Linux problem and #2 was Linux only.

The Data/time related components from Foundation were not being found unless Foundation was actually imported.
The LinuxTest function signatures needed to be either (Void) or (). I chose ()

All tests pass mac/Lin. Generated Xcode project still builds and tests pass.

These imports were solved by b605fe2d53a239946ffa81ab7de32daa71fd4aa4


Make more traits support optional

Added "ops" crate feature, and made [std|core]::ops's Deref, DerefMut, Index, IndexMut, and RangeBounds traits optional.

Added "convert" crate feature, and made [std|core]::convert's AsRef and AsMut traits optional.

Closes #42

bors r+


Improve event detail assets view

When browsing the assets in the event details, you can click on the "details" link in the right column, after which the main asset (e.g. Attachments) becomes a tab link above the content and you can further click on more details, which only then appear as tabs:

Suggestion: make the top-level tabs available from the start so you can switch more easily between them and don't have to click on assets again first.

Screencast from 25.06.2024 13:20:27.webm
How do you like this solution?


Imported RKE Cluster displays %3Cnil%3E for NodePorts in Rancher Server UI

Facing a similar issue described here #15482 when importing a cluster created manually with RKE v0.1.16 to Rancher Server v2.1.6.
Not sure if it is a bug or maybe a misconfiguration.
From where comes the information that is used by Rancher UI to display the endpoint of the host? Is it a custom annotation that might be missing? Or is it based on an option to be set during RKE installation in cluster.yml?

Testing note: Tested in v2.1.7 and cannot reproduce.

Can you share your cluster.yml so I could identify some differences to mine?
Further I noticed on the node section in Rancher UI that I can see both the IP addresses and the FQDNs of my RKE node(s):
node01.domain.local  /  <IP_ADDRESS>
When I deploy a custom cluster from Rancher UI I only see the IP addresses of the nodes.
Might this be relevant?

I just set up a simple 1 node cluster.yml with aws ec2 instance (t2.medium in my case)
nodes:
- address: secret_IP
  user: ubuntu
  role: [controlplane,worker,etcd]
  internal_address: <IP_ADDRESS>

What does your cluster.yml look like for your rke cluster? (Hide IP/DNS/sensitive info please)

And my workload:

Played around a bit. Issue seems to be related to what I am setting as address in cluster.yml
With IP address it is working (like in your example): http://<IP_ADDRESS>:31587/
cluster_name: mycluster
nodes:
- address: <IP_ADDRESS>
  user: rke
  role: [controlplane,worker,etcd]

However, when using FQDN mynode01.domain.local instead, Rancher UI shows the wrong url like: http://%3Cnil%3E:31587/
cluster_name: mycluster
nodes:
- address: mynode01.domain.local
  user: rke
  role: [controlplane,worker,etcd]

So, technically my RKE cluster is working with both approaches. However I would prefer to use the FQDN since the IP might be volatile.
I think, Rancher UI should be able to handle both IP addresses and FQDNs.

Thanks @azbpa I'll give it another look using FQDN this time.

Version - master 4/11.
URL encoding needs to be fixed.

just as update: I still have this behavior in Rancher v2.2.2

@alena1108 I think the address is being sent wrong in the endpoints, <nil> is clearly from Go and not the UI (where we have null and undefined, but not nil ).

For testing:
Reference comment https://github.com/rancher/rancher/issues/19208#issuecomment-480225647
Notice that to reproduce we need to have our node use a DNS name rather than static ip. You can do this via AWS - each instance should have a long domain name but this should still work to repro and validate.

Rancher version: master-head (v2.3) (08/21/2019)
I reproduced the issue in Rancher 2.2.2:

Validated the fix in v2.3.

I've used a temporary Instance, so the information in the screenshot won't be valid anymore. As I had to put the full FQDN to demonstrate the fix.


Manually add dotnet8 to base image

Releasing new Host versions.
V3 Release
V2 Release

PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which has an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] If applicable, the PR references the bug/issue that it fixes in the description.
[ ] New Unit tests were added for the changes made and CI is passing.

Prior to applying this change, Host Version 4.31.0 was failing due to missing dotnet8 sdk.  As a result of this change, the build is successful.


[WIP] Ensure all package.json files have a link to the repo

It's missing from some:

[ ]

Codecov Report

Merging #807 into rc0.12.0 will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           rc0.12.0     #807   +/-   ##
=========================================
  Coverage     95.14%   95.14%
=========================================
  Files            73       73
  Lines          2924     2924
  Branches        449      449
=========================================
  Hits           2782     2782
  Misses           46       46
  Partials         96       96

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 732cfde...7163ba7. Read the comment docs.

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ Googlers: Go here for more info.

I signed it!


phoneNumber should not be a constant

Summary
If READ_PHONE_STATE permission is granted at runtime, the phoneNumber constant never get's updated.
This happens because phoneNumber gets set once (initially) and getPhoneNumber just returns the constant, even if it's stale.

Version
all

Affected OS
android

OS Version
all

Current behavior
Start app for first time. Ask user for READ_PHONE_STATE permissions. Then call getPhoneNumber. Returns null, even though you have granted the permissions.
Expected behavior
Start app for first time. Ask user for READ_PHONE_STATE permissions. Then call getPhoneNumber. Returns the phone number (if available).

Definitely, thank you @rcorrie !
Will be shipped in #307

This is implemented on v3 now


WasapiLoopbackCapture raises unhandled exception when i change default audio devices

Hi,
When I use WasapiLoopbackCapture  i get an unhandled exception when i try to change default audio out device ,
How can I handle this exception ?

Look at the Exception property on the RecordingStopped event

Its not raising an event on recording stopped.
Application goes into break state and i get below message
Managed Debugging Assistant 'FatalExecutionEngineError'
Message=Managed Debugging Assistant 'FatalExecutionEngineError' : 'The runtime has encountered a fatal error. The address of the error was at 0x11484265, on thread 0x19ec. The error code is 0xc0000005. This error may be a bug in the CLR or in the unsafe or non-verifiable portions of user code. Common sources of this bug include user marshaling errors for COM-interop or PInvoke, which may corrupt the stack.'
Managed Debugging Assistant 'FatalExecutionEngineError'
Message=Managed Debugging Assistant 'FatalExecutionEngineError' : 'The runtime has encountered a fatal error. The address of the error was at 0x11484265, on thread 0x19ec. The error code is 0xc0000005. This error may be a bug in the CLR or in the unsafe or non-verifiable portions of user code. Common sources of this bug include user marshaling errors for COM-interop or PInvoke, which may corrupt the stack.'

This happens when the recording is inprogress

What happens if you run in release mode (i.e. not debugging from Visual Studio)?

It is the same with the release build also

Hi @markheath  is there any way to handle this exception.
Thanks


Fix: Opacity for Hashed Activity Bars for Gantt (fixes #113)

What is the purpose of this pull request? (Check any that's applicable)

[ ] Feature
[ ] Bug fix
[ ] Dependency updates
[ ] Documentation updates
[ ] Build related changes
[X] Changes an existing functionality
[ ] Other, please explain:

Does this introduce a breaking change?

[ ] Yes
[X] No

What changes did you make? (Give an overview)
Fix: Opacity for Hashed Activity Bars for Gantt (fixes #113)
Before

After

Fix: Border Background bars Gantt for a11y (fixes #123)
Before

After

added 1px border on background which is full opacity so consumers can pass that border against white background for contrast. added 50% opacity on hash background bar so consumers can pass it black to get a high enough contrast against white background.

:tada: This PR is included in version 1.16.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


how can I add logs / store the current logs?

I deployed in an amazon ec2 instance but after a while the application dies. how can I store the logs that appear in console? I tried several options (like winston, etc) but without success.
Thanks!

I had one here: https://github.com/groovybits/gaib/commit/52b9599bb81b65a6aea07d854b77808a9df0fb4a#diff-349e858709586b934dbba2f345b55125c0f2f3eabd3430388c1c3a7cbf40e6b9 I added. Worked okay, logged into .next/ though :( so needs some crafting, and doesn't work with Vercels etc of course.
Streaming output to somewhere is ideal, in Cloudrun it logs the console.log output that is backend. This logged client end too :D.


Service reconcile on services with scale policy triggers deletion of instances.

Rancher Version:v1.1-rc1
Docker Version:1.11.2
Steps to Reproduce:
Have a service with following  scale policy:
"scalePolicy": {
"increment": 2,
"max": 6,
"min": 1,
}
Delete 2 instances of the services.
When service reconcile was triggered , 2 of the existing instances also gets deleted and then recreated.
Finally service gets all 6 containers , but we should not be deleting instances as part of service reconcile.

Tested with latest build from master:
Have a service with following scale policy:
"scalePolicy": {
"increment": 2,
"max": 6,
"min": 1,
}
Delete 2 instances of the services.
When service reconcile was triggered , 2 new instances get created. Existing instances do not get removed.


Coredump related to --support-big-int

When running with --support-big-int, sometimes getting a core dump related to GC compaction.  Here's the operative fn call:
v8::internal::Heap::CheckIneffectiveMarkCompact(unsigned long, double)
If I turn off --support-big-int, no issues with the same dataset.

Full stacktrace

Thu, 20 Jun 2019 01:30:55 GMT | sent 10000 objects to destination elasticsearch, wrote 10000
Thu, 20 Jun 2019 01:31:09 GMT | got 10000 objects from source elasticsearch (offset: 50000)
Thu, 20 Jun 2019 01:31:13 GMT | sent 10000 objects to destination elasticsearch, wrote 10000

<--- Last few GCs --->
io[110236:0x34a3950]   120935 ms: Mark-sweep 1395.5 (1430.2) -> 1394.5 (1431.7) MB, 561.8 / 0.0 ms  (+ 0.0 ms in 59 steps since start of marking, biggest step 0.0 ms, walltime since start of marking 570 ms) (average mu = 0.116, current mu = 0.014) allocatio[110236:0x34a3950]   121974 ms: Mark-sweep 1397.5 (1431.7) -> 1396.4 (1433.2) MB, 1032.1 / 0.0 ms  (+ 0.0 ms in 48 steps since start of marking, biggest step 0.0 ms, walltime since start of marking 1038 ms) (average mu = 0.049, current mu = 0.006) allocat

<--- JS stacktrace --->

==== JS stack trace =========================================

    0: ExitFrame [pc: 0x10fdb31dbe1d]
    1: StubFrame [pc: 0x10fdb31dd1df]
Security context: 0x375c1d19e6e1 <JSObject>
    2: N(aka N) [0x61bec51c5c9] [/usr/lib/node_modules/elasticdump/node_modules/lossless-json/dist/lossless-json.js:~1] [pc=0x10fdb3260cf3](this=0x2e2502e826f1 <undefined>)
    3: O(aka O) [0x61bec51c709] [/usr/lib/node_modules/elasticdump/node_modules/lossless-json/dist/lossless-json.js:~1] [pc=0x10fdb323f4c8](this=0x2e...

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
 1: 0x8dbaa0 node::Abort() [node]
 2: 0x8dbaec  [node]
 3: 0xad83de v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [node]
 4: 0xad8614 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [node]
 5: 0xec5c42  [node]
 6: 0xec5d48 v8::internal::Heap::CheckIneffectiveMarkCompact(unsigned long, double) [node]
 7: 0xed1e22 v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector, v8::GCCallbackFlags) [node]
 8: 0xed2754 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [node]
 9: 0xed53c1 v8::internal::Heap::AllocateRawWithRetryOrFail(int, v8::internal::AllocationSpace, v8::internal::AllocationAlignment) [node]
10: 0xe9e844 v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationSpace) [node]
11: 0x113dfae v8::internal::Runtime_AllocateInNewSpace(int, v8::internal::Object**, v8::internal::Isolate*) [node]
12: 0x10fdb31dbe1d
(core dumped) elasticdump

ElasticDump version
6.1.0

Elasticsearch version
input ES ver: 5.1.1
output ES ver:  7.1.1

Node.js version (note that nodejs v8+ is required for this tool)
v10.14.0

Full command you are having issue with (with inputs and outputs)

elasticdump --params="$PARAMS" --input="$SOURCE_URI/$INDEX" --output="$DEST_URI/$INDEX" --type=data --ignore-errors --noRefresh --support-big-int=true --limit=10000

I'm working on trying to prepare a publishable dataset that triggers this.  It's pretty sensitive data, so it may take a little time.  Hopefully, the error is obvious to more knowledgeable eyes.

This is really a node issue. The limit is too high and the big-int library is doing too much.
Try bumping the max-old-space-size
NODE_OPTIONS="--max-old-space-size=4096"
FYI: There is no version 6.4.1

Sry, I meant elasticdump 6.1.0.  Corrected above.
Yeah exporting NODE_OPTIONS="--max-old-space-size=4096" was the first thing I tried.
Doesn't resolve the issue.

Also what about --maxSockets impact?  Recommendation?

That's totally dependent on you. You can set it to Infinity, but I wouldn't recommend anything over 100. All these though need to be benchmarked, since it depends on how much request Elasticsearch can actually handle.
Adjusting limit relative max-old-space-size is hard, since it also dependent on the amount of properties you have. Through as much memory as you can at it, if that doesn't fix the issue then decrease the limit in increments of 1000.

Reducing the --limit and bumping up max-old-space-size works.  Stabilizes copying the same dataset with --support-big-int.  Wonder if it's possible to put a guard at the framework-level to prevent this behavior.    In any case, it's clearly possible to overflow the heap when pushing elasticdump too hard.
Thanks for the help. 🙏

Been documented with lossless-json.
https://github.com/angular/angular-cli/issues/12645


🚀 Feature: creating custom cli template

🔖 Feature description
Documentation and mechanism to easily plug into the templating mechanism of the backstage-cli. Or being able to override existing templates to create a new plugin.
🎤 Context
Our Backstage for instance has been migrated to MUI 5, new front-end plugins are still generated with old material-ui 4 dependencies. This is just one example of us wanting to be more in control of what is generated for people when they run a backstage-cli template, without having to fork the cli package.
✌ Possible Implementation
No response
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

Sorry, missed this one! This is something we want but not working on immediately in the maintainer team. @minkimcello has been looking at this though

Hi @minkimcello, any updates on this one? Came up recently on Discord again and I've found myself looking for something like this recently as well.

To outline a bit more of our idea for the way forward here: Convert the new templates into standalone definitions that work like simplified scaffolder templates. We'd also make sure that they're portable so that you can for example link to templates on GitHub as well as in the local file system/repo. We then also add a configuration somewhere where you can list the templates that you want to use in your own repo, likely also with the ability to set default/fixed parameters and so on. Overall this would allow adopters to pick the specific templates that apply to their own project and setup, along with the ability to create custom templates for internal needs.

Thanks @awanlin @Rugvip. This is a fairly common use case for us, since we have a steady stream of contributors who need to onboard to our backstage instance and create new plugins.

@minkimcello Thanks a lot for helping out. We'd like to have the new plugin set up with MUI 5 and also we use a different eslint config. So it'd be nice to be able to override the default eslint config somehow.

@minkimcello wanted to check in on your progress as we're looking to work on/ship some of this next week. Would love a draft PR to be able to start up early review/tweaks.


Fix typo in PromiseInspection API documentation.

PromiseInspection.pending() does not exist, should be PromiseInspection.isPending()

Thanks!


[Hopper] Update CGA->CTA tiling heuristic and add support for persistent kernels to improve performance of NUM_CTAS>1 kernels

Improve the logics of determining splitM&splitN
Add necessary support for persistent kernels with NUM_CTAS>1

Thank you for the PR, and thank you to others for reviewing this change!
Please let us know when this is ready for OpenAI to have a look.
I understand that in the past, OpenAI hasn't had a lot of bandwidth to look at nvidia PRs.  It's created a few problems for us, one of which is that nobody over here groks how some of the nvidia-contributed code works!  I'm still new, which means I also don't grok Triton yet, but I also have time to do reviews, because I'm new.  :)  So I'm going to make it a priority of mine to do careful reviews of nvidia code.  Please feel free to ping me here or on Slack if you're not getting the attention you need!
For this PR, would you be willing to include the benchmarks you ran?  I believe you that it's faster, but I want to be able to run the same tests so I don't accidentally regress you in the future.

https://github.com/openai/triton/pull/2638#issuecomment-1806241110
Hi @jlebar, sorry for late reply. I was struggling to enable related modifcations work on latest codes. Unfortunately, I find many NUM_CTAS>1 cases fail after recent refctoring of pipeline pass and other passes (I think ThomasRaoux is working on repairing it). My local branch is based on commit 2217bd2f5 (#2531) and it works for me. So now I can only list some results of persistent warp specialized kernel based on this commit, maybe you can cherry-pick modifications of this PR and replay it.

So maybe this feature need to be delayed until we recover most logics of NUM_CTAS>1, @ThomasRaoux pls feel free to let me know if we are ready or there is anything I can help.

So now I can only list some results of persistent warp specialized kernel based on this commit

No worries on the delay, and thanks for getting back to me!
Are these benchmarks that are checked into the Triton repository?  If so, what are the steps to reproduce?

So maybe this feature need to be delayed until we recover most logics of NUM_CTAS>1, @ThomasRaoux pls feel free to let me know if we are ready or there is anything I can help.

As discussed on slack I would prefer we first re-enable functionality independently of pipelining before adding back pipeline support. Will you be working on this?

Are these benchmarks that are checked into the Triton repository? If so, what are the steps to reproduce?
Yes, they are checked. you can add the following test config in test_full_static_persistent_matmul_kernel of triton/python/test/unit/hopper/test_persistent_warp_specialized_gemm.py and run in respective version (i.e. before and after modification). Then I suppose you can reproduce it. BTW, these selected config is from autotune.
[
       #---before---#
       [1024, 128, 64, 4, 8, 1300, 1800, 3000, False, False, 'none', 'float16', True, 5, True],
       [512, 256, 64, 4, 8, 800, 30000, 10000, True, True, 'none', 'float16', True, 4, True],
       [1024, 128, 64, 4, 8, 1800, 10000, 15000, True, True, 'none', 'float16', True, 5, True],
      #  #---after---#
       [512, 256, 64, 4, 8, 1300, 1800, 3000, False, False, 'none', 'float16', True, 5, True],
       [128, 1024, 64, 4, 8, 800, 30000, 10000, True, True, 'none', 'float16', True, 5, True],
       [512, 256, 64, 4, 8, 1800, 10000, 15000, True, True, 'none', 'float16', True, 5, True],
]

As discussed on slack I would prefer we first re-enable functionality independently of pipelining before adding back pipeline support. Will you be working on this?
Sorry Thomas, I'm not familiar with related modfications and I'm also afraid having no enough time to add back support for pipeline or other passes due to my other tasks. Previously, I thought only pipeline pass needs to complement, however, when I have trouble running persistent warp specialized kernel on latest codes, I realized other passes were also affected by commits after #2531.

I realized other passes were also affected by commits after https://github.com/openai/triton/pull/2531.
@ThomasRaoux @jlebar  The reason causing many fail cases has been found out: some modfication for fp8 make warp specialized kernel fall back to its normal version, which doesn't support NUM_CTAS>1 after pipeline pass refactoring. @PhrygianGates is helping to modify fp8-related codes to avoid this fallback.