GPU support across different kinds of machines

I'm doing very computationally-heavy calculations in the browser for data visualizations. Fortunately, I'm using Electron (Chrome) so the browser features are a known entity. However, people will be installing this on lots of different desktop machines. If those machines don't have the right GPU support and/or slow GPUs, the interactions will fail because CPU fallback computations will be too slow.
Does Weblas run or run quickly on a majority of desktop computers? Or is there a lot of variability? Are there guidelines or checks to see if a GPU on a machine is capable of running Weblas fast?
I'm wondering if I should be thinking of this like the old days of wondering if browsers support a certain feature. Now we've got pretty good browser feature parity (especially with build tools) but as we push computation to the GPU how do we evaluation GPU support?
Thank you for building this library.

Thanks for the feedback and the questions.
I've done tests on a wide range of hardware. In every case I've tested so far, the GPU implementation used here outperforms an optimized javascript implementation.
Most modern consumer devices (including mobile devices) have, at the very least, some form of integrated GPU. While typically much less powerful than a dedicated unit, it usually gives a considerable speed increase.
Variability in a deployment environment is still a concern, however. I'd like to introduce a simple profiling system in the library that allows you, as a developer and user of the library, to get a single number representing the performance class of any individual user. This will help you make better decisions and help me more easily collect statistical information to answer this question in a general way.
This system, which will probably consist of a single function, is in the planning stages now.


Stricter handling of promises/async functions

🔍 Search Terms
"promise", "dangling promise", "async", "floating promise"
✅ Viability Checklist

[X] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[X] This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals

⭐ Suggestion
I think it would be valuable for TypeScript to have an option enabling stricter checks on async functions and promises. Things like:

Disallowing promises that aren't handled or awaited
Disallowing async functions to stand in for functions that are clearly expected to be synchronous. A common bug we run into is passing async functions into Array.forEach():

async function foo() {
  ["foo.txt", "bar.txt"].forEach(async (path) {
    const contents = await fs.readFile(path);
  }); // returns immediately, instead of waiting until all files are read!
}

1) is currently handled by https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/no-floating-promises.md.
2) isn't handled by any common tooling that I'm aware of.
Promises and async/await are common patterns used in JavaScript, and there are a whole class of possible bugs that could be eliminated from stricter TypeScript checking. Some of these are currently handled by third-party linters like ESLint, but misuse of promises are so common, and so likely to be bugs, that I think they deserve to be brought into TypeScript as compile errors. Another benefit is that it prevents users from having to run 2 different tools, each constructing their own ASTs, to get compile-time safety.
To avoid breaking existing TypeScript code, we could gate this change behind a flag for users to decide when/if to opt in.
📃 Motivating Example
Stricter handling of promises to avoid writing bugs in async code.
💻 Use Cases

What do you want to use this for?
Detect entire classes of common async bugs at compile time.
What shortcomings exist with current approaches?
ESLint has a no-floating-promises rule, but it's slow and sometimes has buggy false negatives.
What workarounds are you using in the meantime?
ESLint

see #13376
see #28595.

You should continue to use no-floating-promises.  The TypeScript team will most likely reject this as it doesn't really have much to do with type checking and last time I heard they don't want to add new lint checks to the compiler.

Hm, there appears to be more interest from the team than I suspected, however see discussion starting here, this is apparently a bigger can of worms than it looks at first: https://github.com/microsoft/TypeScript/issues/13376#issuecomment-1456749948

With the new daemon mode, the type-aware ESLint rule is a good move at minimum, but I agree that there’s a serious need here. It’s not in the usual correctness bucket, but in practice dangling promises are one of the major contributors to memory leaks, especially in cases where you expect a short-lived request on a server and "close" it by whatever means, but the promise is still dangling. (Ask me how I know, he says, shuddering to remember the first 4 months of the year.)

@RyanCavanaugh can you clarify which issue(s) this is a duplicate of?


Speed test check & notification?

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
New Monitor
🔖 Feature description
Is there a way to have a notification generated if an internet speed test falls below a certain threshold?   I have the need to monitor an ISP that has a flaky speed rating that changes without warning.   I can't figure out a good way to automate a check for this.  Is it possible with current configuration?
Thanks
✔ Solution
A speed test option and notification trigger
❓ Alternatives
Pinging doesn't really give me good info i don't think
📝 Additional Context
No response

Reference discussion in #744

Something like SpeedTest Tracker that now looks abandoned. It uses SpeedTest CLI and public servers but doing something with netperf would be nice too!

@bam7
I think this is a duplicate of https://github.com/louislam/uptime-kuma/issues/744
If you agree, could you please close this Issue, as duplicates only create immortal zombies and are really hard to issue-manage?
If not, what makes this issue unique enough to require an additional issue? (Could this be integrated into the issue linked above?) ^^

@bam7
We are consolidating duplicate issues a bit to make issue management easier.
I think, we should track this issue in #744 as there is no functional difference (maybe just small naming differences, but nothing that would require a different issue imo)
⇒ I am going to close this as a duplicate.


Add new native method and refactoring method names

Add  a new native method to check the next record availability. Rename existing readTextRecord native method to nextTextRecord while keeping the current functionality.
This resolves https://github.com/ballerina-lang/ballerina/issues/4530

This will fix through a new PR https://github.com/ballerina-lang/ballerina/pull/4553


Added Diameter of Binary Tree in C

Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Yes
Description
I have added the algorithm for finding diameter of binary tree in C. The diameter of binary tree is the number of nodes on the longest path between two leaves in the tree.
(Write your answer here.)
Checklist

[Yes ] I've read the contribution guidelines.
[ Yes] I've checked the issue list before deciding what to submit.
[Yes ] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)
#2716

@ankitaggarwal23 Can I work on this in C++


Bug in the response of Copilot Chat: endless repetition.

Copilot Chat Extension Version: v0.19.2024073102 (pre-release)
VS Code Version: 1.92.2
OS Version: WIN11 23H2
Logs:

Steps to Reproduce:
As this was a non-deterministic response from the LLM, it could be too hard to replicate. But this is the context conversation:
warnov
what wifi bands can the 8266 work?
GitHub Copilot
The ESP8266 microcontroller supports the 2.4 GHz Wi-Fi band. It does not support the 5 GHz Wi-Fi band. Here are the specifics:
Frequency: 2.4 GHz
Standards: 802.11 b/g/n
Channels: 1-14 (depending on the region)
This means the ESP8266 can connect to Wi-Fi networks operating on the 2.4 GHz band and using the 802.11 b, g, or n standards.
warnov
what micros support 5ghz?
GitHub Copilot
Microcontrollers that support 5 GHz Wi-Fi are less common than those supporting only 2.4 GHz. However, some newer microcontrollers and modules do support dual-band (both 2.4 GHz and 5 GHz) Wi-Fi. Here are a few examples:
ESP32:
ESP32-S2: Supports only 2.4 GHz.
ESP32-S3: Supports only 2.4 GHz.
ESP32-C3: Supports only 2.4 GHz.
ESP32-D0WDQ6: Supports only 2.4 GHz.
ESP32-WROOM-32: Supports only 2.4 GHz.
ESP32-WROVER: Supports only 2.4 GHz.
ESP32-WROOM-32E: Supports only 2.4 GHz.
ESP32-WROVER-E: Supports only 2.4 GHz.
ESP32-WROOM-32U: Supports only 2.4 GHz.
ESP32-WROVER-IE: Supports only 2.4 GHz.
ESP32-WROOM-32D: Supports only 2.4 GHz.
ESP32-WROVER-B: Supports only 2.4 GHz.
ESP32-WROVER-IB: Supports only 2.4 GHz.
ESP32-WROVER-KIT: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
...
...
And after many lines lie that, it ended with:
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only 2.4 GHz.
ESP32-WROVER-KIT-V4.1: Supports only
Sorry, no response was returned.

I recommend using the release version when using stable VS Code. It should be chat version 0.18.2

I have this bug too in the release version.
I am often working with large random integers and strings in python. Sometimes copilot will decide to write example code that includes an attempted example of one of these, and it will just keep writing the string or int.


Setup script for Chill

Features

Asks user for the inputs required by chill.
Modifies chill.yml as per the inputs.
Uses pm2 to start the processes.
Displays the addresses where the processes are run.

Limitations

Doesn't add an nginx configuration. User will have to manually point to the addresses displayed.
node/yarn/pm2/git have to be pre-installed for the script to proceed with the installation. (The script however does check if these are pre-installed or not)

Screenshots

:100:

Codecov Report

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

@@           Coverage Diff           @@
##           master      #55   +/-   ##
=======================================
  Coverage   80.59%   80.59%
=======================================
  Files          19       19
  Lines         943      943
  Branches      135      135
=======================================
  Hits          760      760
  Misses        183      183

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 e04bebf...92ccf1e. Read the comment docs.

@kabirbaidhya @romitamgai Someone with Linux should try doing this and then we can merge it.

Also the script creates chill inside chill #chill-ception.
/codehome/leapfrog/chill-project/chill/scripts/chill, /codehome/leapfrog/chill-project/chill/scripts/chill/chill.yml
@leapfrogtechnology/chill-team

@manisacharya You can look at this script and update it. We need a sh file to setup all of chill. This should be like a one click install but from the terminal.
Good opportunity to learn a bit of bash. 😛

@mesaugat I was looking into this. First I had to move chill.yml outside the monitor folder. Now, I will work on this.


⚠ main has degraded performance

In 511180c, main ($MAIN_API_URL) experienced degraded performance:

HTTP code: 200
Response time: 23091 ms

Resolved: main performance has improved in 2aa277d.


Question about ValueDice's nonlinear loss

Hi,
This is a question for the repo for the ValueDice paper, (perhaps @peterjliu or Ilya Kostrikov is the relevant person?)
I see that there is a softmax (instead of a log-sum-exp) involved to compute the J_linear part of the valuedice loss : https://github.com/google-research/google-research/blob/7c7fdac0d57ef262896776785a59d824026a076b/value_dice/value_dice.py#L27
was wondering if you could explain why that is?
Is this something to do with wanting to normalise this part of the loss for some reason?
Would really appreciate any clarification.
Thanks!
Gunshi

I have nothing to do with this

@ikostrikov maybe?

@gunshi see equation 12 in https://arxiv.org/pdf/1801.04062.pdf
In this case the softmax part corresponds to analytically computed gradients of log sum exp.

Hey @ikostrikov, can I also ask for help with the derivation of, or even the intuition behind the replay buffer regularisation version of the objective? It looks to me like in eqn 18 of the arxiv version, we're maximising and minimising the same quantity in the log and linear parts of the expression (for the replay buffer samples).
Thanks!

@gunshi I think it might be easier to explain for the toy MDP that we used in the paper. When the regularization is not used, the values are not defined for certain states and actions. Also it's similar to soft targets used in Improved Techniques for Training GANs.

I see! so is it fair to say that we would actually want the policy roll-outs to be somewhat sub-optimal? Because really good policy roll-outs (that almost look like the expert) are probably sending a contradictory learning signal to the critic, yes?
Thanks for your responses, they're really helpful!

Yes, it's better to have rather suboptimal rollouts with a larger coverage for the replay buffer.

got it, thanks!


Latest from datafusion-contrib

This PR contributes the changes that were made to the project since it moved to the datafusion-contrib github organization.
See https://github.com/datafusion-contrib/datafusion-python/issues/58 for more context.

@jimexist @cpcloud @samkaufman @jychen7 @alamb @matthewmturner Could you please review when you have time.

@andygrove good for me - i just left a governance related question on the original issue proposing this move

👍 to me


Added a fade in animation to every page.

Using a framework I found online called animate.css, I was able to easily add an animation to every page that causes the content in it to fade in. I think this makes the whole site feel a lot smoother and more like an app.
Edit: While I'm making this small change, decided to also change the footer to no longer display "Mock Interview App" and to display our new name.

Added in some more front end changes (the ones discussed on chat).


Procedural Meshes

Questions to be answered:

[ ] Should procedural meshes keep using the tick() function to recalculate the mesh if dirty?

Still work-in-progress. Categorizing StaticMeshes into a separate interface, adding methods unique to them (replaceMaterials, scale...), Creating a README.md, adding sceneLoader/registry support.

InteliJ did bad


Provide type and method extensibility to high level language DSL framework

Currently the high level language DSL framework provides an elegant and powerful approach to do things like flatMap, reduce, filter etc as shown below
    val app = StreamApp("dsl", context)
    val input = "This is a good start, bingo!! bingo!!"
    app.source(input.lines.toList, 1).
      // word => (word, count)
      flatMap(line => line.split("[\\s]+")).map((_, 1)).
      // (word, count1), (word, count2) => (word, count1 + count2)
      groupByKey().sum.log

However developers that wish to add new operations or possibly override operations may find it difficult unless they attempt to extend Stream[T] and provide new methods or override old ones. By using a 'pimp my library' pattern and typeclasses the DSL framework can be extended to be even more powerful. The code below shows how both app.source and app.flatMap could be added dynamically as well as overridden using typeclasses when a specific type binding is required.
object Operations {
  private val LOG: Logger = LogUtil.getLogger(getClass)
  implicit class Source(app:StreamApp) {
    def source[M[_] <: TraverseType[_], T: ClassTag](traversable: M[T], parallism: Int, description: String = null): Stream[T] = {
      implicit val source = TraversableSource(traversable, parallism, Some(description).getOrElse("traversable"))
      app.graph.addVertex(source)
      new Stream[T](app.graph, source)
    }
  }
  implicit class FlatMap[T:ClassTag](stream: Stream[T]) {
    LOG.info("not specific")
    def flatMap[R: ClassTag](fun: T => TraversableOnce[R], description: String = null): Stream[R] = {
      val flatMapOp = FlatMapOp(fun, Option(description).getOrElse("flatmap"))
      stream.graph.addVertex(flatMapOp )
      stream.graph.addEdge(stream.thisNode, Direct, flatMapOp)
      new Stream[R](stream.graph, flatMapOp)
    }
  }
}
object Operations1 {
  private val LOG: Logger = LogUtil.getLogger(getClass)
  implicit class FlatMapArrayDatum(stream: Stream[Array[Datum]]) {
    def flatMap[R: ClassTag](fun: Array[Datum] => TraversableOnce[R], description: String = null): Stream[R] = {
      LOG.info("specific")
      val flatMapOp = FlatMapOp(fun, Option(description).getOrElse("flatmap"))
      stream.graph.addVertex(flatMapOp )
      stream.graph.addEdge(stream.thisNode, Direct, flatMapOp)
      new Stream[R](stream.graph, flatMapOp)
    }
  }
}
object Operations2 {
  private val LOG: Logger = LogUtil.getLogger(getClass)
  implicit class FlatMap1[T:ClassTag](stream: Stream[T]) {
    LOG.info("replaced")
    def flatMap1[R: ClassTag](fun: T => TraversableOnce[R], description: String = null): Stream[R] = {
      val flatMapOp = FlatMapOp(fun, Option(description).getOrElse("flatmap"))
      stream.graph.addVertex(flatMapOp )
      stream.graph.addEdge(stream.thisNode, Direct, flatMapOp)
      new Stream[R](stream.graph, flatMapOp)
    }
  }
}

The example below shows how methods source and flatMap are implicitly added to StreamApp and Stream and how flatMap is overridden for the specific type Array[Datum].
    import Operations._
    import Operations1._
    app.source(new ReplayableSourceTest, 1, "time-replayable-producer").flatMap(metrics => {
      Some(metrics.flatMap(datum => {
        datum.dimension match {
          case CPU =>
            Some(datum)
          case _ =>
            None
        }
      }))
    })

Running the example will result in
[INFO] [04/23/2015 03:25:50.787] [Operations1$] specific

Additionally, if needed the type generic Operations.flatMap could be replaced by an alternative using import exclusion rules
import Operations.{FlatMap => _, _}
import Operations2.{FlatMap}

This issue has been reviewed and we decided that have standard monad operations within Stream like flatMap, map, filter, etc is preferred rather than included all operations implicitly.


[network] [breaking] Remove ping / pong

Breaking Nature
The breaking change is limited to LibraNet communication:

NetworkMessage::ErrorCode definitions changed
NetworkMessage::RpcRequest format changed
NetworkMessage::Ping/Pong removed

Justification

Error definitions were ineffective and practically unusable -- such as reporting the network version in a pre-negotiated channel, returning full length messages as errors, and being incomplete on which message type was being used
Ping/Pong was unused and duplicate of a high level protocol -- there was no value in keeping it around if we never used it properly
Ordering of the fields within RpcRequest required additional parsing to retrieve the protocol which is used for error handling

Details
This PR does a few things:

Improves the error messages to be more useful:

ParsingError: the messaging version isn't relevant since this was already exchanged as part of the handshake, sending the entire message back is bad due to DOS attacks. This returns the first two bytes which typically are the message type and protocol type.
NotSupported: returning back the protocol type isn't that great since it doesn't help us route it to the correct handler, we actually need to first send it to the message type handler, hence we made this a bit more robust

Removes ping / pong as we never got around to migrating the healthcheck service to use this, it was an unimplemented component of the API
Adds an error response on unparseable messages -- which was primarily done to help an integration test
Reorder fields within the RpcRequest so that the first 2 bytes define message type and then protocol type.

It is breaks the LibraNet protocol.

Curious on @phlip9 's thoughts as he worked on the health checking and ping/pong ideas.

Updated the general verbiage on the specification on what must be handled and what it optional. I think we'll still likely need an update on the spec on language, but at least the implementation / code do not diverge.

/land


Welcome screen

It might be nice to have a welcome screen, maybe with a list of available puzzles or something, rather than just dropping the user into a random puzzle (which currently isn't even included in the repository).

This is resolved by recent commits adding a library view, and the library auto-populates with the default puzzle sets if none are present already.


Clueless halt

After your refactoring, I also got this error. The error appears with no pattern; it just sometimes appear randomly. And I would say it happens quite often.
terminate called after throwing an instance of 'std::system_error'
  what():  Broken pipe

i have no clue why it is happening, hope you could have more insight.

Could you provide the command you run? And Did this error appear during training?

It is happening during training. No python error trace stack is shown.
I'm training with another model. Let me see what happens with pspnet.

https://github.com/donnyyou/torchcv/blob/8432507a910cf0b42366ed7e5a526f454956e9f1/models/tools/module_helper.py#L33
You could change the syncbn type. And you could install with pip according
to https://github.com/zhanghang1989/PyTorch-Encoding.

The error actually appeared when I'm using single gpu, the norm_type is batch_norm. I'm experimenting multi-gpu to see if things change.

You should update to the lastest version of pytorch and do experiments with multi-gpu. And I will fix the single-gpu error as soon as possible!

It still happens when using two gpus with pspnet. My pytorch is up to date.

Could you show me the detailed error log?  And when this error will occur during training, about how many iterations?

Learning rate = [0.00993742924503113, 0.00993742924503113]������Loss = 0.77565169 (ave = 0.75523706)
2019-05-30 00:58:37,509 INFO����[fcn_segmentor.py, 109] Train Epoch: 1  Train Iteration: 550����Time 55.849s / 50iters, (1.117) Data load 0.007s / 50iters, (0.000138)
Learning rate = [0.009931137900563058, 0.009931137900563058]����Loss = 0.59585357 (ave = 0.75127996)
2019-05-30 00:59:33,436 INFO����[fcn_segmentor.py, 109] Train Epoch: 1  Train Iteration: 600����Time 55.927s / 50iters, (1.119) Data load 0.007s / 50iters, (0.000140)
Learning rate = [0.009924842568020915, 0.009924842568020915]����Loss = 0.55061913 (ave = 0.73662124)
terminate called after throwing an instance of 'std::system_error'
��what():  Broken pipe
This is the full log.
I can resume training afterwards, but it will halt again some time in the future

Have you changed the syncbn type? It may lead to this error. And you could replace it with encsync_batchnorm. https://github.com/donnyyou/torchcv/issues/52#issuecomment-497177002

I havent, but as I said it happens when using single gpu, which is not using sync batchnorm. I suspect there should be somewhere else causing this.
Anyways,  I will try using encsync.

Could you email me the detailed log file if convenient? My Email<EMAIL_ADDRESS>

torch=1.1.0 && cuda=9.0. I have successfully run for more than 1000 iters without bugs. You could try to reintall the pytorch.

Can you train longer? Sometimes it takes longer. I'm using cuda10 BTW.

encsync_batchnorm has the same error. I will try cuda9.

FYI, the code before refactoring works fine with current pytorch installation.

Reinstalling the pytorch with cuda-9.0 maybe solve this problem.

It still happens to me with cuda9

2019-05-30 14:44:51,005 INFO    [fcn_segmentor.py, 109] Train Epoch: 1  Train Iteration: 700    Time 21.009s / 50iters, (0.420) Data load 0.011s / 50iters, (0.000229)
Learning rate = [0.009912239908315376, 0.009912239908315376]    Loss = 0.53774828 (ave = 0.54497988)
terminate called after throwing an instance of 'std::system_error'
what():  Broken pipe
[New Thread 0x7fff71fff700 (LWP 29023)]
Program received signal SIGPIPE, Broken pipe.
[Switching to Thread 0x7fff755fe700 (LWP 28666)]
0x00007ffff7bcd69d in write () from /usr/lib64/libpthread.so.0

This is gdb output. Still clueless.

It's not caused by extensions because I'm using extension compiled from previous versions.

It look like it's because of the multi-thread data loader. When I use workers 0, it seems to be working.

https://github.com/donnyyou/torchcv/blob/6b1f113752a702b7180da03e2875aee6fb192ff8/datasets/det/loader/default_loader.py#L17 Comment this line that I add during refactoring and try again.

The code has no bugs for me. I have run for about 2w iterations.

Do you mind replacing, say 4W, with 40k, which is more widely accepted worldwide?

BTW, I finally got some readable error:

Exception in thread Thread-5:
Traceback (most recent call last):
File "/share/data/vision-greg/rluo/local/anaconda/envs/py36/lib/python3.6/threading.py", line 916, in _bootstrap_inner
self.run()
File "/share/data/vision-greg/rluo/local/anaconda/envs/py36/lib/python3.6/threading.py", line 864, in run
self._target(*self._args, **self._kwargs)
File "/share/data/vision-greg/rluo/local/anaconda/envs/py36/lib/python3.6/site-packages/torch/utils/data/_utils/pin_memory.py", line 21, in _pin_memory_loop
r = in_queue.get(timeout=MP_STATUS_CHECK_INTERVAL)
File "/share/data/vision-greg/rluo/local/anaconda/envs/py36/lib/python3.6/multiprocessing/queues.py", line 113, in get
return _ForkingPickler.loads(res)
File "/share/data/vision-greg/rluo/local/anaconda/envs/py36/lib/python3.6/site-packages/torch/multiprocessing/reductions.py", line 292, in rebuild_storage_filename
storage = cls._new_shared_filename(manager, handle, size)
RuntimeError: Broken pipe at /opt/conda/conda-bld/pytorch_1556653099582/work/torch/lib/libshm/core.cpp:99
terminate called after throwing an instance of 'std::system_error'
what():  Broken pipe
Aborted

It never appears before.

https://github.com/donnyyou/torchcv/blob/6b1f113752a702b7180da03e2875aee6fb192ff8/datasets/seg/data_loader.py#L55 Set pin_memory to False and try!

Let me try turning set_sharing_strategy of first and then try pin_memory😊

pin_memory is True before refactoring too.

pin_memory was True before refactoring too.

File "/share/data/vision-greg/rluo/local/anaconda/envs/py36/lib/python3.6/site-packages/torch/utils/data/_utils/pin_memory.py", line 21, in _pin_memory_loop
r = in_queue.get(timeout=MP_STATUS_CHECK_INTERVAL), This error seems to appear in _pin_memory_loop.

My guess is pin_memory will be affected different sharing strategy. It seems that uncomment that line is useful. I'm able to get to 1500 iterations now

It is now working. Able to pass 5000 iterations.

Set pin_memory to False or comment the https://github.com/donnyyou/torchcv/issues/52#issuecomment-497547985 ?

comment share strategy.

Can you train longer? Sometimes it takes longer. I'm using cuda10 BTW.
Ruotian Luo

On May 30, 2019, at 10:52 AM, Donny You<EMAIL_ADDRESS>wrote:
torch=1.1.0 && cuda=9.0. I have successfully run for more than 1000 iters without bugs. You could try to reintall the pytorch.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Is there still any error after commenting the share strategy?

No, it's fine now


add business param pattern match

Describe what this PR does / why we need it
add business param pattern match
Does this pull request fix one issue?

fixes:https://github.com/chaosblade-io/chaosblade-exec-jvm/issues/227
Describe how you did it
Describe how to verify it
Special notes for reviews

@xcaspar

Thank you for your contributions.


Any version above 1.19 crashes instantly

Describe the bug
I cannot start any minecraft versions above 1.19, whenever i try to start any versions above 1.19 it gives exit code 1 and these reports
Desktop (please complete the following information):
OS & version [e.g. Windows 10, 1822]: Windows 10
MSFT Build of OpenJDK Version:
java version "1.8.0_361"
Java(TM) SE Runtime Environment (build 1.8.0_361-b09)
Java HotSpot(TM) 64-Bit Server VM (build 25.361-b09, mixed mode)
To Reproduce:
Steps to reproduce the behavior:

Go to 'Minecraft Launher'
Choose version 1.19.2
Click Play
And it crashes as soon as i see the 1.19.2 window and gives this error

Expected behavior
I expected the game to start normaly
**Sc
hs_err_pid2976.log
latest.log
reens
Additional context

It didn't worked, my drivers are the latest, I'm not using any mods if u know how to read the crash report then u'd know that im using the vanilla minecraft 1.19.2 and its THE OFFICAL launcher

Hi @LenoraFX,
If you are still experiencing the crash with the official launcher and without any mods installed, please submit this issue over at the Mojang bug tracker.
Once Mojang confirms the issue is with our OpenJDK runtime (instead of with any of the Minecraft components), then we will certainly take action.
Thanks,
Bruno


C++ include

Can we have a:
#ifdef __cplusplus
extern "C" {
#endif
.....
#ifdef __cplusplus
}
#endif
in the include file so it can be cleanly included by a cpp file?

Why not specify extern "C" in the C++ file?
https://github.com/ARMmbed/mbed-littlefs/blob/33a2ca519a5b2c3b97db15852721081340a2fdb1/LittleFileSystem.h#L22-L24
Seems a bit strange that a C library needs to support C++ and not the other way around.

That's what I do, I personally just find it neater using an ifdef and extern "C" combo inside header files as it means you can just include it blindly from C or C++ without having to think.
Guess it's personal preference.

On a complete tangent, I was looking at the little fs class in mbed and I couldn't for the life of me see where it allocated space for the file structure, it just uses a void * and doesn't seem to allocate it anywhere.  Don't know if that was your code and I didn't dig that deep so I probably missed the important bit.

Ah, it allocates the file structure in file_open using C++'s new operator, which makes it a bit harder to find:
https://github.com/ARMmbed/mbed-os/blob/master/features/filesystem/littlefs/LittleFileSystem.cpp#L372
One of the trickier parts of file-memory-management is that the file structure must be deleted in file_close, even if an error occurs:
https://github.com/ARMmbed/mbed-os/blob/master/features/filesystem/littlefs/LittleFileSystem.cpp#L394

I'm going to go ahead and close this, feel free to reopen if you want


WIP: fix: 6915

The issue is that swiper.loopFix() doesn't run when the slideToLoop is called, so it gets stuck and doesn't loop. I added swiper.loopFix() to the underlying slideTo function after the transition to fix this.

Changed status to WIP. It doesn't work when you jump more than 1 slide.

Additional change: slidesAppended would return a weird value when going right so slides would be missing so I updated how it's calculated. Works with 3, 5, and 7 slidesPerView.

This breaks autoplay.

Autoplay fixed.

Hi! Thank you but it i think we should avoid fixing it in current version of Swiper. As it is for sure can break something else. This is already fixed in next branch for upcoming Swiper v11, you can check it there

Ah ok thank you!


Error in running openset example

Hello,
I am trying to run inference with openset. I get the following error:
checkpoint, the shape in current model is torch.Size([768, 512]).
size mismatch for tagging_head.encoder.layer.1.crossattention.self.value.weight: copying a param with shape torch.Size([768, 1024]) from checkpoint, the shape in current model is torch.Size([768, 512]).
File "/Users/csv610/Projects/SegmentAnything/RAM/recognize-anything/ram/models/ram.py", line 77, in init
vision_config = read_json(vision_config_path)
^^^^^^^^^^^^^^^^^^
UnboundLocalError: cannot access local variable 'vision_config_path' where it is not associated with a value

Make sure your image size is either 224 or 384


do we have code for america stickers?

We should put them out at the reg table if we have some

Yes LOTS of them!!  Also citycampNC ...since you are the keeper of the swag
does Red Hat have anything to donate for the bags or stickers to sit out :)
On Wed, Sep 21, 2016 at 8:50 AM, Jason Hibbets<EMAIL_ADDRESS>wrote:

Assigned #315 https://github.com/codeforraleigh/CityCampNC/issues/315
to @TaxLadyTracy https://github.com/TaxLadyTracy.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/CityCampNC/issues/315#event-797248838,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMJnMyx8hI8ZfZOCqABHqDaxuN8DrxaPks5qsSgTgaJpZM4KCwsL
.

I'll bring some stickers
On Wed, Sep 21, 2016 at 12:33 PM, TaxLadyTracy<EMAIL_ADDRESS>wrote:

Yes LOTS of them!! Also citycampNC ...since you are the keeper of the swag
does Red Hat have anything to donate for the bags or stickers to sit out :)
On Wed, Sep 21, 2016 at 8:50 AM, Jason Hibbets<EMAIL_ADDRESS>wrote:

Assigned #315 https://github.com/codeforraleigh/CityCampNC/issues/315
to @TaxLadyTracy https://github.com/TaxLadyTracy.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/codeforraleigh/CityCampNC/issues/315#event-797248838
,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AMJnMyx8hI8ZfZOCqABHqDaxuN8DrxaPks5qsSgTgaJpZM4KCwsL>
.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/CityCampNC/issues/315#issuecomment-248668389,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADgoV0SV4uSoprzWkSOuirjEx9h5G7c1ks5qsVxsgaJpZM4KCwsL
.

Thanks!!!!!


[AIRFLOW-2159] Fix a few typos in salesforce_hook

[AIRFLOW-2159] This commit addresses a few typo fixes in salesforce_hook. Here is the Jira ticket for this issue: https://issues.apache.org/jira/browse/AIRFLOW-2159
No tests needed for this PR since it is a comment only change. Tox tests/builds passed for this change.

Codecov Report

Merging #3085 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3085      +/-   ##
==========================================
+ Coverage   73.26%   73.27%   +<.01%
==========================================
  Files         179      179
  Lines       12491    12491
==========================================
+ Hits         9152     9153       +1
+ Misses       3339     3338       -1

Impacted Files
Coverage Δ

airflow/models.py
87.19% <0%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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

Great.  Thanks for the contribution, Dan!


units in solution table printouts

A couple proposed improvements to make result table printouts even safer:

(carry-over from #831): when looping through key, value pairs in SolutionArray.results_table, use the value instead of key for units printing, or at least double-check that the units agree.

When gpkit's units are disabled (if not gpkit.units), don't print units in results tables. It's misleading.

If item 2 above is implemented, we can remove the if not gpkit.units conditional from test_units_sub in t_solution_array.

reopening due to #921


Uncaught SyntaxError: Cannot use import statement outside a module

Expected behavior
Should load the ChartJS library from the CDN URL.
<script src="https://cdnjs.cloudflare.com/ajax/libs/Chart.js/4.0.1/chart.min.js" integrity="sha512-tQYZBKe34uzoeOjY9jr3MX7R/mo7n25vnqbnrkskGr4D6YOoPYSpyafUAzQVjV6xAozAqUFIEFsCO4z8mnVBXA==" crossorigin="anonymous" referrerpolicy="no-referrer"></script>

Current behavior
The browser shows this error "Uncaught SyntaxError: Cannot use import statement outside a module"
I'm not using a module I'm using the traditional way to load JS Files just using the CDN URL. This is not a node app neither I'm using WebpackJS.
This is totally supported by the previous version 3.9,1
Reproducible sample
https://codepen.io/teocci/pen/vYreREO?editors=0010
Optional extra steps/info to reproduce
No response
Possible solution
No response
Context
No response
chart.js version
v4.0.1
Browser name and version
Chrome Version 107.0.5304.107 (Official Build) (64-bit) and Firefox v107.0(64-bit)
Link to your project
No response

@teocci Hi. You should use https://cdnjs.cloudflare.com/ajax/libs/Chart.js/4.0.1/chart.umd.js

Might be worth putting this in the migration guide

@dangreen Hello why he should use chart.umd.js what is the difference with the other links please ?

The requirement for the UDN version should probably be detailed in the CDN section of the instillation page.


Search by enter button

Closes #100

Coverage decreased (-0.7%) to 70.662% when pulling e57a3bdde9f04354fd2fe24b0557378614bf1c37 on sam-shih:master into 2e46b764454440cda4c626469dbe44a86675d352 on TIRaveler:master.


timezone

Please use the timezone specified in docker-compose.yaml
TZ=Europe/Copenhagen

Hi
I just found an alternative solution
remove
environment:
- TZ=Europe/Stockholm
add
volumes:
- /etc/localtime:/etc/localtime:ro
After the clients first connection the time is shown in your hosts timezone

Thank you 🙏

I had the same issue but couldn't get this to work because the image is missing the tzdata package.
I didn't need to map the additional volume but I did need a Dockerfile with the following
`FROM ngoduykhanh/wireguard-ui:latest
RUN apk update && apk add tzdata`


Add rpm and deb packages to the otelcol-k8s release

We have found a lot of value in the otelcol-k8s release since it has some very useful and common processors without the full contrib package. To get use out of it when it's not being deployed into a kubernetes cluster or if it's being installed into an existing approved base container image, having that release available as an RPM and Deb package would help.

It isn't much work to add the extra artifacts to the release.  We originally started with them but slimmed it down during PR discussions.


Consolidate our color palette and theme object

Done in https://github.com/giantswarm/happa/pull/3557


KAFKA-4228 - make producer close on sender thread death, make consumer shutdown on failure to rebalance, and make MM die on any of the above.

the JIRA issue (https://issues.apache.org/jira/browse/KAFKA-4228) details a cascade of failures that resulted in an entire mirror maker cluster stalling due to an OOM death on one mm instance.
this patch makes producers and consumers close themselves on the errors encountered, and mm to shut down if anything happens to producers or consumers.
Signed-off-by: radai-rosenblatt<EMAIL_ADDRESS>

cc @junrao @hachikuji


touch boot/ssh

touch boot/ssh touch: boot/ssh: No such file or directory make: *** [boot/ssh] Error 1
/Volumes/boot exists
Maybe the Makefile tries to create ssh before /boot is mounted?

No, it's a temporary ./boot/ directory. I guess it's not created in time, and git just ignores empty directories 🤔. I'll fix it.


Ipad issue: unable to open my fills or positions on market page after submitting an order - tabs are unresponsive

I'm still seeing this. Have you been able to test on a tablet device?

works now


add new devices, update tests

Pull Request Test Coverage Report for Build 2142

7 of 7 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 2117:
0.0%

Covered Lines:
1108

Relevant Lines:
1108

💛  - Coveralls


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/44373

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/44833

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/48509

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/62864

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/63250

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/80565


How to properly build it?

Install .NET SDK 6.0
Execute the following command use powershell

git clone https://github.com/loyldg/mytelegram.git
cd mytelegram/source
dotnet restore
cd .\src\MyTelegram.MessengerServer.Abp\
dotnet publish -c Release -r win-x64

Thanks.


Fix Token property name in balances endpoint

Closes #311
The property tokenType in Token (mapped in the balances route) should be named type instead to match the current API

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3732

Relevant Lines:
4085

💛  - Coveralls


[docs] Notistack codesandbox not working

Open https://mui.com/components/snackbars/#notistack
Click the codesandbox edit button
It fails

Raised by https://twitter.com/DongHoangDinh1

Fixed with the releast of<EMAIL_ADDRESS>


godoc: correct fsRoot to prevent misleading message about cmd/

Fixes golang/go#14447

This was reverted due to test failures. Can you send it again and adjust tests as needed?


Langchain vectorstore integration with Kinetica

Description: New vectorstore integration with the Kinetica database
Issue:
Dependencies: the Kinetica Python API pip install gpudb==<IP_ADDRESS>b,
Tag maintainer: @baskaryan,
Twitter handle:

Updated from master

@efriis - Could you please comment on what needs to be done on this PR ?

@efriis - Could you please comment on what needs to be done on this PR ?

Hey there! Are all these methods in the vectorstore implementation necessary? There are a few nonstandard things going on here that will make this vectorstore difficult to use as well:
* assumed dimensionality of openai embeddings -> does that need to be set on initialization?

* documentation is missing - also how to run integration tests / set up a demo instance

I'm going to close this PR for now, and if you add these, feel free to reopen! The lighter weight the implementation is here, the easier it will be to review.

I have followed pretty much the way pgvector vector store implementation was done. Could you please be more elaborate on what is non-standard here ? I am sorry but I am doing this the first time and not sure which one to follow as a reference.

There are differences between how things were implemented before the community split and now - sorry for the confusion!
All of these re-exports from langchain are only for backwards compatibility. New implementations should only exist in community.

https://github.com/langchain-ai/langchain/discussions/14243

There are differences between how things were implemented before the community split and now - sorry for the confusion!
All of these re-exports from langchain are only for backwards compatibility. New implementations should only exist in community.

Thanks for the clarification, will change accordingly.

Modified the vectorstore implementation

I don't think I am allowed to reopen this PR.
@efriis @hwchase17 Do you think you could open it or do I create a new PR ?

Created a new PR - https://github.com/langchain-ai/langchain/pull/18102
Closing this PR.


Calculation of Awario social scores can easily be gamed

What is the problem you are seeing? Please describe.
The Awario Z scores can be gamed to affect current and future ranking.
For example, using the November app mining data, by systematically maximizing the Awario social score of all apps with an original score above zero, the max social Z score drops from 1.74 to 0.84. As a consequence, high ranking apps would loose around 0.05 from their final scores, which would lift top debuting apps several positions in rank (they have no awario score) and give future debuts an edge as the lower final scores would affect the "memory function" for incumbents.
Alternatively, somebody could boost all apps to have a minimum of four in awario social score. This would increase the score of high ranking apps with around 0.02 points, while other would experience a loss of up to 0.3 points. Debuting apps would keep the same score, thus stand to benefit substantially.
How is this problem misaligned with goals of app mining?
It opens up for disturbing manipulation of the scores and rankings. Also, a purpose of the awario social score is to require a minimal level of activity to remain in higher rank, which would be defeated by third party boosting the score of inactive teams.
What is the explicit recommendation you’re looking to propose?
Use the mean social score of an app in place of the Z-score, making it independent of the scoring of the other apps.
The loophole is caused by the awario Z scores depending on average score of all apps and their the standard deviation, which can easily be manipulated by boosting the count of social channels of the apps towards the extremes or towards the average:
if(App score=0, -1, (App score -average App score:App score))/stdev(App score:App score))

Describe your long term considerations in proposing this change. Please include the ways you can predict this recommendation could go wrong and possible ways mitigate.
The purpose of using the standard deviation is to "normalize" the scores so they are comparable across the different reviewers. The proposal normalizes the scores to be in range 0 to 1.
Additional context
#30 #135 #154 #127

@njordhov do you have any examples?

@dantrevino In the November app mining spreadsheet, there are 70 apps with a zero Awario social score, out of 170 apps with a score, leaving 100 apps to have a non-zero score. Now let's boost the latter to all have the max Awario score of five. Calculating z-score for an app with 5 social score:
Mean: 2.94
Standard Deviation, σ: 2.46
Social Z-score: (5 - 2.94)/2.46 = 0.837

Let's take the awario scores of this months front-runner (pDrive) as example and replace its Social Z (1.744150654) with the depressed Social Z-score above. Find the Awario score by averaging its Reach Z (1.365339254) and replaced Social Z (0.837) => 1.101169627 then take the square root for a Awario theta score of 1.04936629782.
Subtracting this from the original final awario score of 1.246894123 gives 0.19752782518 in loss. Divide by 4 to get the resulting loss for their final score: 0.04938.
This demonstrates that the final score of an app can be depressed by strategically boosting the awario social score of selected apps.

The purpose of using the standard deviation is to "normalize" the scores so they are comparable across the different reviewers

Indeed, the reviewers provides scores that reflects the relations between an app and THE "average app". If the average app changes (by boosting the social score of some apps) then indeed the score of all apps changes. And that is a good thing.
In the long run all apps will have awario social Z score of near 0, same for NIL Z score because all apps do their best to receive maximum points. Apps that don't will get a very low Z score. (Maybe that is a barrier for entrance but that is a different issue)
#119 is relevant here describing the same thing for NIL.
So, I don't see anything wrong in the awario social Z score.
I agree, that 3rd parties can and do boost the social score by promoting the app on the social networks. But that is exactly what the awario score is about. The score can only give a vague indication whether the team is active or not. For that we would need a better activity reviewer (#169)

In the long run all apps will have awario social Z score of near 0

If the social awario of all apps are boosted to the max, they will indeed all have an awario social Z score of zero. As a result, using the November app miningdata, high ranking apps stand to loose more than 0.1 points of their final score, while debuting apps keep their scores. The top debuting app would move into 4th from a ranking at 13th. That's considerable motivation to manipulate the scores by spending an hour posting all app names to social channels.

because all apps do their best to receive maximum points.

This is a naive assumption. Participants with multiple apps have reason to let some of their apps deliberately drop in score to boost the score of their high ranking app(s).

3rd parties can and do boost the social score by promoting the app on the social networks. But that is exactly what the awario score is about.

That one app can gain in rank by boosting the score of other apps is a perversion of what the awario score is about.

Although I certainly follow the logic behind what you're describing, I would rather wait until this actually becomes a problem before making big changes to the scoring method. Removing the Z score for social would introduce a ton of other problems, and vastly boost existing apps vs. new apps.
If we ended up in a world where 100% of apps received a 5/5, then maybe we could just drop the social score. I'd also prefer to remove the social score if we suspect that someone is engaging in this 'attack'.


Adding formula to the editor

Hi.
After trying to add a formula to the editor, Quill throws an error in the console:
TypeError: Cannot read property 'render' of undefined
After digging a little bit, I found that the problem is the window.katex.render where katex is undefined.
Are you able to provide a solution?
Thanks.
React-Quill version

[*] master

I also encountered this problem, how to solve it...

You need to have katex available on the window - See the Quill docs: https://quilljs.com/docs/modules/formula/

Anyone solve this problem?

I found the solution for react-quill. Add following code the it work.

Install Katex

npm install katex

Add following 3 line codes to your component
/-----
import katex from 'katex';
import 'katex/dist/katex.min.css';
window.katex = katex;
class xxxxForm extends React.Component {
.......................
}
----/

Then the alarm cleared and formula input successfully


Add an ink_mutex destroy function that tolerates the mutex being locked.

Also converts more calls from strerror() to thread-safe ts::Strerror class.
This is hopefully a work-around for crashes that Yahoo prod is seeing on config reload.

In case it's helpful, here's the Ubuntu build failure:
https://ci.trafficserver.apache.org/job/Github_Builds/job/ubuntu/1034/console
The linking phase has an undefined reference to LogMessage::message in ink_mutex_safer_destroy:
/usr/bin/ld: ../src/tscore/.libs/ink_mutex.o: in function `ink_mutex_safer_destroy(pthread_mutex_t*)':
/home/jenkins/workspace/Github_Builds/ubuntu/src/out_of_source_build_dir/src/tscore/../../../src/tscore/ink_mutex.cc:87: undefined reference to `LogMessage::message(DiagsLevel, SourceLocation const&, char const*, ...)'
/usr/bin/ld: /home/jenkins/workspace/Github_Builds/ubuntu/src/out_of_source_build_dir/src/tscore/../../../src/tscore/ink_mutex.cc:87: undefined reference to `LogMessage::LogMessage(bool)'
libtool: link: ccache clang++ -std=c++17 -g -pipe -Wall -Wno-deprecated-declarations -Qunused-arguments -Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -O3 -fno-strict-aliasing -Werror -Wno-invalid-offsetof -mcx16 -rdynamic -Wl,--as-needed -o traffic_via/.libs/traffic_via traffic_via/traffic_via-traffic_via.o  -L/usr/lib/x86_64-linux-gnu ../src/tscore/.libs/libtscore.so ../src/tscpp/util/.libs/libtscpputil.so -lpcre -lpthread -ldl -Wl,-rpath -Wl,/tmp/ats//lib
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[2]: *** [Makefile:1979: traffic_cache_tool/traffic_cache_tool] Error 1

Yup working on it.


Geometry and others: update calls to use sdf::Errors parameters

Signed-off-by: Marco A. Gutierrez<EMAIL_ADDRESS>🎉 New feature
Work towards https://github.com/gazebosim/sdformat/issues/820.
Depends on: https://github.com/gazebosim/sdformat/pull/1141.
Depends on: https://github.com/gazebosim/sdformat/pull/1263 and its forward port.
Summary
Adds missing Errors structure parameters in a few methods of the following classes:

Box
Capsule
Collision
Cylinder
Ellipsoid
Geometry
Heightmap
Mesh
Plane
Polyline
Sphere
Surface

Adding several classes updates on this PR due to inter dependencies.
Test it
Using the classes above should report all errors through sdf::Errors if the errors parameter is used, it should print them otherwise.
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

When I thought I was almost done adding the ToElement(errors) and Load() method tests for the classes in this PR I found out the ToElement method of the Surface class is missing some elements. I added a PR that fixes this here.
I guess we should keep this PR on hold until that one is merged and ported to sdf13.

The test failures on windows are being investigated in https://github.com/gazebosim/sdformat/issues/1276.


Remove version: SBCL.SBCL version 2.2.3

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: URL works;


live-common 13.5.0

live-common bump: please see full detail in https://github.com/LedgerHQ/ledger-live-common/releases/tag/v13.5.0
some minor libs bump. check integrity of the app on android and ios should be enough (not too deep).

I'm not able to reproduce the issue on TRC20, apparently trongrid fixed the issue, so we no longer need those changes.


Photo Upload failed with the following response: <Response [400]>

What do  I wrong?
cl = Client()
cl.login(username, password)
a = cl.user_info_by_username(username)
media = cl.photo_upload("D:\a\Bild1.png", "First Test Upload")   #ERROR
#Console Output:
Photo Upload failed with the following response: <Response [400]>

@MRV1N2 cannot reproduce

Try to figure out the reasons for yourself, since only you have it reproduced. Perhaps your photo is not the correct resolution, or your IP is in the ban list of Instagram, etc.

Okey, I'm sticking to the research, I'm not giving up that easily. I will post my solution in here.
P.s. adw0rd this is a very cool repository from you. Thank you


Add Solana RPC Canister

Dear @tiamo,
In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA.
If you decide to agree with it, please visit this issue and read the instructions there. Once you have signed it, re-trigger the workflow on this PR to see if your code can be merged.
— The DFINITY Foundation


micro: -version commit hash missing

Issue description
The ISSUE_TEMPLATE of micro want's the commit hash. It is broken on NixOS!
Steps to reproduce
$ nix run -f channel:nixos-unstable micro -c micro -version
Version: 1.4.1
Commit hash: Unknown
Compiled on Unknown

/cc maintainer @dtzWill
Technical details

system: "x86_64-linux"
host os: Linux 4.14.91, NixOS, 18.09.1853.c2950341d03 (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.3
channels(root): "nixos-18.09.1853.c2950341d03, nixos-hardware, nixos-unstable-19.03pre165037.eebd1a92637"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Here's the current behavior:
$ nix run nixpkgs#micro -- --version
Version: 2.0.10
Commit hash: v2.0.10
Compiled on Unknown

And looking at the expression for it, yep we populate our source "revision" into the build, but here it's the git tag v2.0.10 (which in theory could change, more of a ref than a rev).
Anyway.
This seems sufficient for bug reporting, but if this seems like a disservice to our users/their developers, we can look into grabbing the git revision itself.  Offhand I think simplest would be to keep it alongside the source info and bump along with source info when updating to new version, but that's a bit of a pain and not really helpful unless they change their tags often.
(and helpful update bots may miss updating this)
Closing, but if this seems like a problem let me know! Thanks for reporting!

Not sure if that is of any help. It just duplicates the information from version and is not a hash.
A perfect solution would be to have the exact behavior as upstream builds.
There it looks like this: https://github.com/zyedidia/micro/pull/1862#issue-702564388


[SVCS-546] Update Dropox provider to use chunked-uploads

Ticket
https://openscience.atlassian.net/browse/SVCS-546
Purpose
This PR adds support for chunked uploads to Dropbox which will enable us to upload files larger than 150MB.
Changes
Adds three major methods which (1) create a session, (2) upload parts to that session (3) close the session and receives the file's metadata.
Side effects
None that I know of.
QA Notes
Needs osf.io side fix to raise upload limits.
Deployment Notes
None that I know of.

Coverage decreased (-0.03%) to 89.088% when pulling a77b4c532d0b86f30cd67268014780dbefce13ee on Johnetordoff:dropbox-chunked-uploads into ba5573122292d5baa4d62635cab773278e2ee876 on CenterForOpenScience:develop.

Coverage decreased (-0.03%) to 89.088% when pulling 5a8772723179107de15d79a8d431088f35b3f89d on Johnetordoff:dropbox-chunked-uploads into ba5573122292d5baa4d62635cab773278e2ee876 on CenterForOpenScience:develop.

Coverage decreased (-0.03%) to 89.073% when pulling d8db70ce7fc0371660bfdee4d75767d2b9cdea20 on Johnetordoff:dropbox-chunked-uploads into 473191c78c36b6ee63d4609d3952a317ee4ab63e on CenterForOpenScience:develop.

Coverage decreased (-0.03%) to 89.073% when pulling 864d5b234d6ef990fa76c0ad4ec7155e2e9b28e2 on Johnetordoff:dropbox-chunked-uploads into 473191c78c36b6ee63d4609d3952a317ee4ab63e on CenterForOpenScience:develop.

Coverage decreased (-0.03%) to 89.912% when pulling 33abca3bf927a3cc2143db830322e9dde45cffc6 on Johnetordoff:dropbox-chunked-uploads into 26bf2093c15af333e634f14372709e7bf014ccb4 on CenterForOpenScience:develop.


ctags timeout configurable?

Hi,
can we set the ctags timeout configurable? I think the default for large codebases and not so powerful machines, sometimes isn't enough. I have a patch to do it that i could send it via pull request (zoekt-index gets a new argument which is passed to build/ctags.go. if nothing is given, default is assumed)

I'm going to revisit the ctags support soon, in light of
https://github.com/universal-ctags/ctags/commit/5d82ee6e92ee9cdda5cc9dcee91646dd678b4b42
this will likely remove the need for batching files and timeouts.

please try installing universal-ctags (with --_interactive support) as universal-ctags.


[CSR-207] Updated deployment config

@fantasticjamieburns I need some config changes from #10 , i will merge that into here. However, this PR should happen after #10


Apply company name local fallback when SDK config value is null

Jira issue:
Company name local fallback is ignored when Sdk config value is null
What was solved?
Company name local fallback was ignored when SDK config value was null. This issue should be fixed now.
Release notes:

[ ] Feature
[ ] Ignore
[x] Release notes (Is it clear from the description here?)
[ ] Migration guide (If changes are needed for integrator already using the SDK - what needs to be communicated? Add underneath please)

Additional info:

[ ] Tests updated? Added? Unit, acceptance, snapshots?
[ ] Logging for future troubleshooting of client issues added?

@DavDo @Pelkar @AndriiHorishniiMOC Guys, I had to rewrite several functions to be able to add unit tests. That's why I'm asking you to review PR once again.


Liquid Metal Slime has ability slots

I was just updating my parameter mission builds and I saw that the builder is allowing abilities on Liquid Metal Slime. He should just have accessory slots and that's it, I believe!
Thanks as always for the amazing work!

Looks like this one has been fixed, as I'm not seeing it anymore, so I'll close it. Please let us know if it reoccurs, as it could be related to opening new unit tabs when existing tabs are open.


Feature/mib unit tests

:tada: This PR is included in version 1.7.0-develop.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[BUG] LineChart not displaying dataPointLabelComponent and dataPointText properly on RTL Layouts.

When using dataPointLabelComponent on LTR layouts it displays properly:

When using dataPointLabelComponent on RTL layouts it messes up the position of the labels:

When dataPointText on LTR layouts it displays properly:

When using dataPointText on RTL layouts it displays the text mirrored:

I solved this by using transform: [{ scaleX: I18nManager.isRTL ? -1 : 1 }]
However, the datapoints are still not aligned with the line chart... they are off by 1 record as shown in your image

Hi @Golannnnn How did you achieve the RTL layout? Can you please share the code?

Hi @Golannnnn How did you achieve the RTL layout? Can you please share the code?

Hi, I remember it was achieved with the css transform property using either rotate or scale. I unfortunately don't have access to the code anymore because this was at my old company.

For RTL, I had to reverse the array of the data to make it work. Something like that
`const chartData = I18nManager.isRTL
    ? data.map((record, index) => ({
        value: selectedTabIndex === 0 ? record.transactionAmount : record.transactionCount,
        label: record.date,
        dataPointLabelComponent: () => (
            <View style={styles.chartCustomDataTextContainer}>
                <Text variant='labelSmall' style={styles.chartCustomDataText}>
                    {selectedTabIndex === 0 ?
                        AmountFormat.format(Math.floor(data[data.length - 1 - index].transactionAmount))
                        :
                        data[data.length - 1 - index].transactionCount}
                </Text>
            </View>
        )
    }))
    : data.map(record => ({
        value: selectedTabIndex === 0 ? record.transactionAmount : record.transactionCount,
        label: record.date,
        dataPointLabelComponent: () => (
            <View style={styles.chartCustomDataTextContainer}>
                <Text variant='labelSmall' style={styles.chartCustomDataText}>
                    {selectedTabIndex === 0 ?
                        AmountFormat.format(Math.floor(record.transactionAmount))
                        :
                        record.transactionCount}
                </Text>
            </View>
        )
    }));`

   And still had minor issues solved by transform: [{ scaleX: I18nManager.isRTL ? -1 : 1 }].
   Other minor issues such as data points not aligned with the graphy could not be solved.

   It would be great for example to have a property to force LTR even in RTL.


Update pixi lock

I don't understand why %CONDA_PREFIX% isn't parsable here.

I think you need to escape the back slashes, maybe?

@jaimergp I will leave this to you. The plugin system is something unknown to me and probably needs bit more debugging.


Simplify handling of endpoint visibility policy annotations

Now that fetchK8sLabelsAndAnnotations() returns the 'pod' we can
easily get the annotations and ContainerPorts separately. Remove them
from the return values of fetchK8sLabelsAndAnnotations() as half the
callers are ignoring them anyway.
Re-use the pod resolved in the Endpoint's metadata resolver as if the
annotations have changed again in the meanwhile, the watcher will
queue up another visibility policy update.
Let the caller of UpdateVisibilityPolicy() know if the endpoint
regeneration is needed. Use this in Pod watcher to figure out if
regeneration is needed or not.
Regenerate endpoint from endpoint metadata resolver if needed even if
the endpoint's labels have not changed. This likely makes no
difference, as the aim is to run the metadata resolver only if labels
need to be resolved.
Change the behavior in the case of unparsable visibility policy so
that the visibility policy is not changed in that case, rather than
removing the existing visibility policy. The error in visibility
policy parsing is also passed to the caller rather than just
logged. This is more in line with typical error processing where
changes to the running state are avoided on errors and the change
operation is terminated.

test-me-please

Coverage increased (+0.003%) to 44.458% when pulling d4f0d044ecfd5c22ba40505ba372d02e9c90fe73 on pr/jrajahalme/named-ports-afterthoughts into eeedcb59d3456fcf563cee519a2aa7466c5038d4 on master.


Feat: add generic error component to the project

I saw that there’s an open issue (#61) for adding a generic error component to the project and I thought I’d take a swing at it. I hope this is helpful, but at the very least it was good practice. I’m open to any changes or feedback and will keep my eyes peeled for any updates. Here’s gist of the component:

Here are the props available on the component:

error: Displays an error heading/title in bold at the beginning of the component
errorDescription: Which displays a more verbose description after the title (limited to the width of the component body)
hideIcon: which hides the icon, which is visible by default

Awesome!! Thanks, @tangollama That's really exciting! I just sent sent an email to the address listed on your github profile (joel@joelworrall.com).


DCOS-13209: Reduce complex logic in #wrapElement

This PR refactors the ReactUtil#wrapElements method to be more readable. This comes from a comment in a PR: https://github.com/dcos/dcos-ui/pull/1780#discussion_r95982491
Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

run integration tests

Nice @jfurrow!

@MatApple why did you merged this one? Every PR must have two labels before it's ok to hit the merge button.


Empty settlement window query causes UI error

Root cause: calling central settlement GET /v2/settlementWindows with no query parameters returns:
{"errorInformation":{"errorCode":"3100","errorDescription":"Generic validation error - Use at least one parameter: participantId, state, fromDateTime, toDateTime, currency"}}

which propagates through the UI

See also #22 (is that the same?)

Resolved in #59 by using default from and end dates like so:
https://github.com/mojaloop/finance-portal-v2-ui/pull/59/files#diff-1f50adeffe650626116d19665f33e349921aa4da201e36b0499ab33ff4aba979R48-R52


[Bug] KVCache takes up too much memory when running mlc_llm.serve.server

🐛 Bug
KVCache takes up too much memory when running mlc_llm.serve.server, but memory usage is normal when using cli or mlc_llm.gradio
To Reproduce
Steps to reproduce the behavior:

Download model from https://huggingface.co/mlc-ai
Compile the binary (I tried to reduce context_window_size and prefill_chunk_size, but it did not work to reduce the memory taken)
Start Rest API server by

python -m mlc_llm.serve.server --model /home/orangepi/mlc-llm/dist/Llama-2-7b-chat-hf-q4f16_1-MLC --model-lib-path /home/orangepi/mlc-llm/dist/libs/Llama-2-7b-chat-hf-q4f16_1-opencl-1024.so --device opencl:0 --host <IP_ADDRESS>

I tried reduce prefill-chunk-size here, still no help.

Output from executing the code in a conda environment

[2024-03-25 15:35:00] INFO engine.py:238: Estimated KVCacheConfig "max_total_sequence_length": 21275.
[2024-03-25 15:35:00] INFO engine.py:243: Estimated total single GPU memory usage: 14361.14 MB (Parameters: 3615.13 MB. KVCache: 10689.56 MB. Temporary buffer: 56.44 MB)

Expected behavior
That's the KVCache takes when using Gradio.
INFO model_metadata.py:96: Total memory usage: 4077.14 MB (Parameters: 3615.13 MB. KVCache: 0.00 MB. Temporary buffer: 462.01 MB)

Environment

Platform (OpenCL):
Operating system (Ubuntu.):
 To limit the GPU memory taken by KV cache, you can manually specify --max-total-seq-length with a smaller value. Or you can set the environment variable MLC_GPU_SIZE_BYTES to the number of bytes you want the server to use in total (including parameters, KV cache, temporary buffers, etc.).
https://github.com/mlc-ai/mlc-llm/blob/f04cd3e9e81bcd3c02015df6fe0f0eaa9ffd8453/python/mlc_llm/serve/engine.py#L208-L215
Thank you for the feedback! We will provide detailed documentation regarding this.

Hi @gxmlfx, this is intentional because the automatic estimation wants to maximize the KV cache capacity for serving. To limit the GPU memory taken by KV cache, you can manually specify --max-total-seq-length with a smaller value. Or you can set the environment variable MLC_GPU_SIZE_BYTES to the number of bytes you want the server to use in total (including parameters, KV cache, temporary buffers, etc.).您好，这是有意为之的，因为自动估计希望最大化 KV 缓存容量来提供服务。为了限制KV缓存占用的GPU内存，您可以手动指定较小的值 --max-total-seq-length 。或者你可以将环境变量 MLC_GPU_SIZE_BYTES 设置为你希望服务器总共使用的字节数（包括参数、KV缓存、临时缓冲区等）。
https://github.com/mlc-ai/mlc-llm/blob/f04cd3e9e81bcd3c02015df6fe0f0eaa9ffd8453/python/mlc_llm/serve/engine.py#L208-L215
Thank you for the feedback! We will provide detailed documentation regarding this.感谢您的反馈！我们将提供与此相关的详细文档。

@MasterJH5574 Thanks! That works well, I changed int(gpu_size_bytes) * 0.90 to int(gpu_size_bytes) * 0.50, in
https://github.com/mlc-ai/mlc-llm/blob/f04cd3e9e81bcd3c02015df6fe0f0eaa9ffd8453/python/mlc_llm/serve/engine.py#L217-L225
Appriciate for upcoming new document.


[Hub Generated] Review request for Microsoft.ContainerService/aks to add version preview/2023-05-02-preview

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month. 6/10
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month. 6/12
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki. Note that this doesn't apply if you are trying to merge a PR that was previously in the private repository.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

For CI failure Swagger Avocado, it's expected as the reported missing API originates from two files

aks/preview/2019-10-27-preview/openShiftManagedClusters.json, which is deprecated and replaced by redhatopenshift.
aks/preview/2022-09-02-preview/fleets.json, which has been split into an independent service with a separate SDK (package name azure-mgmt-containerservicefleet).

For Breaking Change(Cross-Version), the violated rule 1011 - AddingResponseCode is caused by the change in commit a25bd8a, which fixed the violated rule PutResponseSchemaDescription in CI Swagger LintDiff (based on commit 85209c2). Will seek approval from breaking change review board first and continue with the ARM review afterwards.

For CI failure Swagger Avocado, it's expected as the reported missing API originates from two files

aks/preview/2019-10-27-preview/openShiftManagedClusters.json, which is deprecated and replaced by redhatopenshift.
aks/preview/2022-09-02-preview/fleets.json, which has been split into an independent service with a separate SDK (package name azure-mgmt-containerservicefleet).

Avocado errors are confirmed by the PR authors that's expected and can be ignored.

@FumingZhang - The public main should only be merged when the service is ready. Please fix the API Readiness Check errors or provide a reason why we should ignore it.

Hey @zedy-wj, thanks for reminding. Estimated service readiness is 06/10 (as written in PR description). I won't merge the PR before that.
As for failure in API Readiness Check, it seems that only one operation (providers/Microsoft.ContainerService/operations) was tested, and this operation has not been taken seriously by server/manifest for a long time, I'll see if we could fix it.

Hey @msyyc, could you please help check why CI SDK azure-sdk-for-python-track2 failed? From pipeline log, the error message is

06:38:52.095 cmdout 	[automation_generate.sh] Exception: Found files that do not set api_version:

Hey @msyyc, could you please help check why CI SDK azure-sdk-for-python-track2 failed? From pipeline log, the error message is

06:38:52.095 cmdout 	[automation_generate.sh] Exception: Found files that do not set api_version:

I commit https://github.com/Azure/azure-rest-api-specs/pull/24245/commits/ccbc48f77a2348e832edb4de4e55c909f5ab4946 which shall fix the failure.

    "enabled": {

ARM recommends string enum over boolean for many reasons. Consider something like activation: { Enabled, Disabled } (or similar with better names). Ref: https://armwiki.azurewebsites.net/rp_onboarding/process/api_review_best_practices.html?q=boolean#common-issues-found-in-review.

Refers to: specification/containerservice/resource-manager/Microsoft.ContainerService/aks/preview/2023-05-02-preview/managedClusters.json:7272 in ccbc48f. [](commit_id = ccbc48f77a2348e832edb4de4e55c909f5ab4946, deletion_comment = False)

@mentat9 - I believe we considered using an enum for enabling this feature rather than a boolean.
There were two reasons why we decided not to:

The plan is not to GA this feature. Once it's gone through preview and been vetted, we're planning to enable it by default for all clusters with no api opt-out.
AKS uses enabled booleans all over for enabling optional features (sometimes called Addons) and keeping this a boolean matches the pattern used in the rest of the API. Here are a few examples: #1, #2, #3.

@msyyc - Please check the Python Breaking changes. If there is no problem, please add the Approved-SDKBreakingChange-Python tag, thanks!

I commit ccbc48f which shall fix the failure.

The Python breaking change is caused by change https://github.com/Azure/azure-rest-api-specs/commit/ccbc48f77a2348e832edb4de4e55c909f5ab4946, the removed operations are deprecated.

The service is ready now, please help merge the PR thanks!

/azp run


fix a problem about Multi-Byte character

If the data contains Multi-Byte character,it may be cut by the tcp packets,then the join function's result will contain some unintelligible text,because it turns the object in the array into string before the concat action.So,we should replace it with the Buffer class's member function concat.

Coverage remained the same at 93.296% when pulling 03af7b56f26d0c651ebc0448315b98b1d67936c6 on rhjyy:master into 2bed4c7e99de52cf9b60c74e5f28f393ba596086 on vpulim:master.

Coverage remained the same at 93.296% when pulling 03af7b56f26d0c651ebc0448315b98b1d67936c6 on rhjyy:master into 2bed4c7e99de52cf9b60c74e5f28f393ba596086 on vpulim:master.

Coverage remained the same at 93.296% when pulling 03af7b56f26d0c651ebc0448315b98b1d67936c6 on rhjyy:master into 2bed4c7e99de52cf9b60c74e5f28f393ba596086 on vpulim:master.

Coverage remained the same at 93.296% when pulling 03af7b56f26d0c651ebc0448315b98b1d67936c6 on rhjyy:master into 2bed4c7e99de52cf9b60c74e5f28f393ba596086 on vpulim:master.

Coverage remained the same at 93.296% when pulling 03af7b56f26d0c651ebc0448315b98b1d67936c6 on rhjyy:master into 2bed4c7e99de52cf9b60c74e5f28f393ba596086 on vpulim:master.

@rhjyy please add a test

@jsdevel the test has been added.


Quote all column names

Column names aren't used that much anymore but one place they are is in the order_by parameter.


[Standalone Market Page] Volume calculation in base asset

I wonder whether we calculate the trading volume correctly. Please look at steps and expectations and let me know if I'm missing something.
Steps to reproduce

Find a market that has no trades at all.
Login as 2 users in separate browsers.
Create some resting orders as user 1.
Place some market orders as user 2 (so that trades are placed immediately).
Compare volume data on market's page and on trading view. Focus on the volume.

Expected results

Data on market page matches data on trading page (especially volume in base asset).

Actual results
GF volume is equal to 6 GF,

but in reality only 5 GF were traded

Instead of using the real volume in base currency are we basically dividing volume in quote currency by last traded price? 18 USDT / 3 USDT = 6 GF? And if that's what we do, is it what we should be doing?

Good catch - after evaluating the issue further we have decided to show the volume in quote currency instead.
This issue can be closed once the volume unit has been updated to quote currency.

@ThomasRalee

Resolved in commit https://github.com/InjectiveLabs/injective-dex/commit/fd888ee075d41b54cfa8905e14406a4a4d9d4ca6

Good to go ✅


Add non ASCII encoding schema support to RailsSchemaUpToDate hook

Adds the possibility of adding an encoding config item if you know your schema.rb or structure.sql file contains non ASCII characters, as in the case of the project I'm working on :grin:
It's possible you might also want this on migration files, but I haven't included that in this PR, let me know if that's something you'd like to see to get this merged.

Hey @ttrmw, sorry for the delay in response.
It looks like the Ruby 3 tests are failing with real errors. There are also some spurious errors (timeouts) that can be ignored, but happy to merge once the Ruby 3 tests are passing. Thanks!

Thanks! Will take a look at repro locally and push a fix.

Sorry for the delay. Failures seem unrelated to your changes, going to go ahead and merge.


Cannot run examples or create new projects

I had posted something similar a day or two back, it was closed without the situation being resolved
Running these commands always ends with errors
 ./node_modules/.bin/skit run ./node_modules/skit/example/helloworld
$ ./node_modules/.bin/skit run helloworld --debug
````

util.js:556
ctor.prototype = Object.create(superCtor.prototype, {
^
TypeError: Cannot read property 'prototype' of undefined
at Object.exports.inherits (util.js:556:43)
at Object. (/Users/kokujin/Development/Research/skit_test/node_modules/skit/node_modules/http-proxy/lib/http-proxy/index.js:111:17)
at Module._compile (module.js:456:26)
at Object.Module._extensions..js (module.js:474:10)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:312:12)
at Module.require (module.js:364:17)
at require (module.js:380:17)
at Object. (/Users/kokujin/Development/Research/skit_test/node_modules/skit/node_modules/http-proxy/lib/http-proxy.js:4:17)
at Module._compile (module.js:456:26)

This traceback is not in skit. It looks like your dependencies are not installed properly. This seems to be failing in the http-proxy npm module when including eventemitter3. What is your node version and npm version?

Was excited to fire up the tutorial app, but getting the same error. Here's what I have:
$ npm --version
2.1.8
$ node --version
v0.10.33

on my system:
─$ npm --version
1.4.28
─$ node --version
v0.10.37
On Sun, Apr 26, 2015 at 7:12 AM, Dan Zito<EMAIL_ADDRESS>wrote:

Was excited to fire up the tutorial app, but getting the same error.
Here's what I have:
$ npm --version
2.1.8
$ node --version
v0.10.33
—
Reply to this email directly or view it on GitHub
https://github.com/clusterinc/skit/issues/14#issuecomment-96323754.

I hit this as well, similar problems here:
https://github.com/hipache/hipache/issues/223
https://github.com/jupyter/jupyterhub/issues/237
I think this is due to a change in the version of eventemitter3 that http-proxy depends on, possibly because of this: https://github.com/nodejitsu/node-http-proxy/commit/607f96c00cbda2a6b881b8ff1db05437dbf4ce77#diff-b9cfc7f2cdf78a7f4b91a753d10865a2
Updating Skit's http-proxy version to 1.11 and running npm install seems to fix.


build: change vllm version to github.com/opendatahub-io/vllm

This PR switches to using a hardened branch of vllm that makes it easier to align dependencies on, and has been tested to work well in combination with future the SDG improvements. In the future we may provide a version of these requirements that tracks with the latest upstream

Checklist:

[x] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[ ] Changelog updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

@tiran ah yes good catch. Dropping as an official optional dep makes sense to me for now.

@n1hility Since dropping the optional dependency gets rid of the installation option of pip install instructlab[vllm] for a users vllm workflow, should we document that users need to install the vllm specified here separately?

I was talking with @JamesKunstle and it seems upstream vLLM is only using scipy in one place: https://github.com/vllm-project/vllm/blob/c5832d2ae9431a1672d547c232ec46b1a9051ff0/vllm/attention/ops/blocksparse_attention/utils.py#L22
I think ideally we could pull that requirement (assuming there is an equivalent way to do that single op) and continue to use upstream vLLM instead of a fork. We made a similar ask for the lm_eval lib and got warm reception: https://github.com/EleutherAI/lm-evaluation-harness/issues/2059

I was talking with @JamesKunstle and it seems upstream vLLM is only using scipy in one place: https://github.com/vllm-project/vllm/blob/c5832d2ae9431a1672d547c232ec46b1a9051ff0/vllm/attention/ops/blocksparse_attention/utils.py#L22
I think ideally we could pull that requirement (assuming there is an equivalent way to do that single op) and continue to use upstream vLLM instead of a fork. We made a similar ask for the lm_eval lib and got warm reception: EleutherAI/lm-evaluation-harness#2059

There's a few other differences in that branch that we want. Although, the intention of it is to track closely to upstream. So it's not a full fork, but rather syncing one. I think we'd want upstream support at some point, but for now we are trying to reduce the number of permutations we have while we complete the remaining pieces.

@n1hility Since dropping the optional dependency gets rid of the installation option of pip install instructlab[vllm] for a users vllm workflow, should we document that users need to install the vllm specified here separately?

Yeah I wonder if what we need to do is document a command that looks like:
pip install git+https://github.com/instructlab/instructlab@version -r github.com/instructlab/instructlab/tree/version requirements.txt -r github.com/instructlab/instructlab/tree/version requirements-vllm-cuda.txt

Putting this on hold until I can verify that someone can test by hand ilab working with this version of vLLM.

I'd like if we can have a followup issue (maybe in https://github.com/opendatahub-io/vllm would be appropriate place for this) to get that midstream vLLM implementation to do proper PyPI packaging and publishing so we're not installing a git-based checkpoint - what do you think @n1hility?

Putting this on hold until I can verify that someone can test by hand ilab working with this version of vLLM.

I have verified this PR works with the changes here: https://github.com/instructlab/instructlab/pull/1589

@Mergifyio rebase

@Mergifyio rebase

@leseb updated the commit message.

Yeah I wonder if what we need to do is document a command that looks like:
pip install git+https://github.com/instructlab/instructlab@version \
 -r https:///raw.githubusercontent.com/instructlab/instructlab/version/requirements.txt  \
 -r https:///raw.githubusercontent.com/instructlab/instructlab/version/requirements-vllm-cuda.txt

To install InstructLab with vLLM on CUDA:
VERSION=0.18.0
pip install instructlab==${VERSION} -r https://raw.githubusercontent.com/instructlab/instructlab/v${VERSION}/requirements.txt

You don't need requirements.txt, because that information is baked into the wheel. The path to the vLLM CUDA txt file needs v0.18.0 (with a leading v).

@leseb when you feel as if the PR is ready, I think the only thing preventing it from merging is the hold label you added.


v4 improvement: scoped styles

Is your feature request related to a problem? Please describe.
Hi, first of all I want to say thanks for the good library. I tried to upgrade to version 4 in an existing project and encountered an inconvenience:
In version 4, I can't style the component with scoped styles because the VueFinalModal component uses regular styles and there are no data attributes for scoped styles in the markup. In a project that mostly uses just scoped styles, this becomes a big inconvenience, because some of the styles just don't work and it's not immediately clear why and even more, how to fix it.
A simple example: I have some basic modal window. I want to add a new modal window, which must have a non-standard width. To do this, I add a separate css class and encounter a situation when the rule for this class doesn't work
Describe the solution you'd like
I propose to make VueFinalModal and other components always use scoped styles by default. This would help people like me who always try to use scoped styles, but at the same time it wouldn't hurt people who use regular styling, because they could use special selectors https://vuejs.org/api/sfc-css-features.html#scoped-css
Describe alternatives you've considered
Use a mix of scoped styles and regular styles. Unfortunately, this is a huge discomfort due to the fact that every developer now needs to understand when and what styles to use. This is an additional cognitive load, which I would like to avoid
Additional context
Screenshot of the markup. Perhaps this will help you understand the situation I'm talking about
https://snipboard.io/gFHXP7.jpg

I agree, this would be super helpful

Thanks for your feedback!
I tried to use scoped styles in VueFinalModal, but it doesn't solve the issue (See bd2712e).
I found this issue from vue 3: https://github.com/vuejs/core/issues/2669#issuecomment-734434188, and you can see this CodeSandBox
It seems an issue from the Teleport, and VueFInalModal is based on the Teleport component.
So I think currently we have no way to improve this.

This issue is closed because I still can't find any solution to improve it.


How should I evaluate the model on the test set? You don’t seem to give the code for the test.

No, no code about the test submission is released.

No, no code about the test submission is released.

Can you release the code about the test submission? thank you


Move cron to diem-operator

Currently we're depending on redis-cluster for cron jobs. Redis is an external service that can have outages.
Move the cron job to the diem-operator (single pod) that will use nats to sent a minute message to trigger cron jobs

done via the diem-operator


PT-BR Docs Translate

After talk to Mr. Michael Dowling, the idea is to translate the Guzzle documentation to PT-BR language.
It would be great if any other member could do the same to Spanish and other languages.

Thanks for your efforts, this is great.


Verification Link in Registration Email

The link does not work, has 'None' in URL, which should be a UUID
Edit: Implemented fix with #2 broke Admin user registration email

See edit at top for re-open info.


Expanding selection by line shouldn't go to the next line

Expanding by line (e.g. cmd-shift-left/right) shouldn't jump between lines when hit multiple times.  According to my testing on native, it should go to the start/end of the line, and subsequent presses should do nothing.  This PR matches that behavior.
Fixes https://github.com/flutter/flutter/issues/79291

(PR triage): @justinmc Do you still have plans for this PR?

@goderbauer Yes I do!  I was waiting for https://github.com/flutter/flutter/pull/87133 to be merged.  Coming back to this this week now.

This is blocked on https://github.com/flutter/engine/pull/28008.  There is a bug in the engine that should be fixed before this is fixed here, otherwise I'd be hacking around a bug.

Squashed commits in order to fix svg test.

@LongCatIsLooong Ready for re-review.  This bumps into the problem of not having multiple TextAffinities for a non-collapsed TextSelection, which you mentioned here and I wrote up in https://github.com/flutter/flutter/issues/88135.  I think this PR works in all practical situations though, which I've covered in my test, even though we're not able to distinguish certain non-collapsed selections.


Add AutoscaleImages property to screen1

I've added the Autoscale images property to the Form component. Now the users can use the high-resolution images in their app.
Solves #2015
@ewpatton Please have a look.

Can one of the admins verify this patch?

@jsuyash1514 This is a good start. I was thinking in the issue that this would be a Screen1-only property and have it govern the behavior of the entire app. Right now it would allow people to mix and match behaviors on different screens, potentially resulting in inconsistent/unexpected appearance. Take a look in MockForm.java to see how it is handled for the ShowListsAsJson property.

@ewpatton I've made the required changes. Kindly have a look.
While testing the same, I've encountered the following error:
java.lang.IllegalStateException: no such property: AutoscaleImages

Can you please help me out?
Thank You.

@jsuyash1514 You'll need to add the new property to YoungAndroidSettings.java so that it is defined.

@ewpatton Thanks for pointing me out to the required file. I've made the necessary changes. Kindly review.

@ewpatton I've made the required changes. Kindly review.
For overcoming the new issue, can we improve the way, we calculate the sample size? For example, we can apply either a linear search or a binary search to find the best fit sample size instead of calculating it as a power of 2.


Support for clink completion

Clink introduces multiple modern shell features to Windows cmd.exe, and it supports extending completions through Lua 5.2 scripts like:
clink.argmatcher("foo")
:addflags("-?", "-h", "-n", "-v", "--help", "--nothing", "--verbose")
:addarg("print", "delete")
:addarg(clink.filematches)
:nofiles()
:adddescriptions(
    { "-n", "--nothing",    description = "Do nothing; show what would happen without doing it" },
    { "-v", "--verbose",    description = "Verbose output" },
    { "-h", "--help", "-?", description = "Show help text" },
    { "print",              description = "Print the specified file" },
    { "delete",             description = "Delete the specified file" },
)

and
clink.argmatcher("git")
:addarg({
    "merge"..clink.argmatcher():addflags({
        "--strategy"..clink.argmatcher():addarg({
            "resolve",
            "recursive",
            "ours",
            "octopus",
            "subtree",
        })
    })
})

It would be great if support for generating clink completions (for tools like poetry) could be added. I'm glad to help if the idea gets considered by the team.

So, if I understand that correctly, we would have to generate Lua code for completions, right?

@Secrus Yes, that's the case. And I believe this can be done using a template, just like how  completions are generated for other shells?

Seems like something doable, provided that clink doesn't change their API too often. I am open to considering it, but most likely as a post-3.0 feature. Right now I want to focus on refactoring and improving already existing features.

Glad to see this idea is being considered. Looking forward to it and if there is anything I can do to help, please let me know :)


Random Gradient Generator

Allows users to generate and experiment with random gradients for backgrounds.
Tools and Tech Stacks : JavaScript , HTML & CSS.
Please assign the issue under Hacktoberfest label

@gantavyamalviya can you kindly assign this to me

@gantavyamalviya can you please assign this issue to me


Detect quotechars and doublequotes in csv dialect

Purpose
Closes https://github.com/CenterForOpenScience/osf.io/issues/3817
Changes
Add helper method to augment stdlib's csv.Sniffer for special cases.
Uses regex to look for comma-separated lists inside csv's
Side Effects
None

Ping @lyndsysimon -- Please prioritize this one. @sloria This is a fix we need to get out as soon a possible in support of a user's efforts. Thanks.

I'd normally ask that changes like this be accompanied by a regression test - but in this case, I don't see that the csv_stdlib method is tested at all. That being the case, I'm hesitant to require it.
That said, writing a unit test for detect_quoted_fields would be a reasonable incremental approach.
I'm not going to make this a blocker for me, so... if it's more overhead than it's worth to test it, don't. It wil be caught when a test is written for the csv_stdlib function.

This has been released, merged in hotfix 0.3.7


Update ansible script to include docker-py as one of the dependencies for exporter role

Fixed using commit 141f015bea4ad3673acffd7f6c3a2da24fc25c28

Issue is fixed hence closing it.


Made iPad-compat

Changed the Example project's build settings to target both iPhone and iPad devices.
» iPhone-only makes me cry a little inside.
» With Apple producing iPhones now with a variety of screen size and resolutions, there's little argument remaining not to allow layouting-up to iPad-size too.  A case can still be made for iPad-optimized UIs, of course, but layout-expanded looks a million times better than the built-in iPad 2X scaling for iPhone-only apps.

Cheers!