yosys: remove pointless patching

(Follow-up to #145193)
This fixes the patch introduced in:
94a047ca74cb (yosys: fix build on darwin, 2021-11-09)
Because we supply yosys with an external build of ABC, the patched
ABCMKARGS variable is without influence. Even if we were building with
in-tree ABC, that part of the patch is wrong, so drop it altogether.
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

@tathougies Let me know if this is okay as-is, or if you want to expand on this in some way.

Result of nixpkgs-review pr 145357 run on aarch64-darwin 1

2 packages failed to build:

glasgow (python39Packages.glasgow)
python38Packages.glasgow

9 packages built:

mcy
python38Packages.nmigen
python38Packages.nmigen-boards
python38Packages.nmigen-soc
python39Packages.nmigen
python39Packages.nmigen-boards
python39Packages.nmigen-soc
symbiyosys
yosys

Thanks for the follow up!

Unfortunately we missed the patches line in the actual default.nix. Fix incoming.

@thoughtpolice What do you mean? What's wrong in default.nix?

Nevermind, it was a complete fluke on my part due to being in the wrong build tree. Forget I said anything, and thanks again!

Okay then. Thank you too for the review!


Improved multicopter actuation API + fixed unittest

This PR removes an unnecessary argument (i.e., n_rotors) in the multicopter actuation API. Note that this information can be extracted by the mapping matrix. Additionally, it adds checks useful for debugging mode. Finally, it fixes a tiny issue in the unittest.
Note that the PR deprecates the current constructor. @PepMS (fyi)

Thanks for the warning @cmastalli , no problem, I'll adapt my code.

Thanks @PepMS for reviewing the code.
I take your comment as approval.


jx-cdx-spring-boot-demo-1 to 0.0.2

Promote jx-cdx-spring-boot-demo-1 to version 0.0.2

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

@joostvdg: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

promotion-build
913e30d3b5703d0b26d6841e04cec0cb05811bb3
link
/test this

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


Adding some more stuff to windows-debug image

Adding some more debugging tools to windows-debug image

I verified the files make it into the debug image with mrosse3/windows-debug:pr-14
/assign @jsturtevant


invalid transaction: Transaction failed to sanitize accounts offsets correctly

Help! To create a token address and call the 'SystemProgram.createAccount' method, what is the meaning of the parameters?
help! 创建token地址调用 'SystemProgram.createAccount' 方法, 参数的意义是什么?
public static TransactionInstruction createAccount(PublicKey fromPublicKey, PublicKey newAccountPublikkey,
long lamports, long space, PublicKey programId)

taking a look, thanks


Python3.11

This PR upgrades the torchsparse repo to 3.11. It should automatically build for the new python version.
I do not expect issues with the release links as it will cp311 in the filename instead of cp38 even if the version number is not changed
For this I installed python3.11 to the ci runners using this branch https://github.com/ZendarInc/RadarProcessor/pull/1523

Yeah that's the goal


Fixed the comment-on-issue workflow

Changed few instances in code to ensure proper working of the workflows.
With the current changes the comment-on-issue raised should work properly.
Fixes #16

Hey @Prajwal0225, please make sure you have the necessary GitHub Actions permissions and that you have correctly configured GitHub Secrets to include the 'GitHub token' for the workflows to function properly.

Do not delete the repository until your pull request (PR) is merged. This will close the PR automatically.

@Prajwal0225, sorry I didn't know that. what should I be doing now?

You have to make a PR again😅.

@Prajwal0225, could you please ensure whether the comment on issue workflow is working properly : )

Yaa, but for that someone has to create an issue on this repo other than owner 😅
can you please create a dummy issue so that we get the idea if it is woking or not

@Prajwal0225, I think it is not working but the code is wrote is valid code. Could you please make sure you have the necessary GitHub Actions permissions and that you have correctly configured GitHub Secrets to include the 'GitHub token' for the workflows to function properly.

ook I will check it


Peer to peer code review day 4

Your Project looks great. ✔

[ ] All functionalities are working.
[ ] Linters are working.
[ ] Correct github flow is followed.

Optional Changes.

[ ] Your input validation is not working correctly. If a user tries to add a book without book name or author name I suggest showing a message that those fields are required. This message should be shown everytime user tries to add an empty input field.

@ShiroYaksha90 , You did a great job but there are some issues I noticed.

[ ]  In the form you used to add a book, you can add a label to the input elements to make it more accessible.
[ ]  You used a table to display the list of books, I suggest you use lists instead of table because lists are more flexible to style than tbale.


Implicit Differentiation with jaxopt.ScipyMinimize

Hi! There is no implicit_diff argument in  jaxopt.ScipyMinimize. Does this mean that jaxopt.ScipyMinimize only supports implicit differentiation but not autodiff of unrolled iterations? Thanks!

Yes exactly, because it’s a black box function we cannot unroll it.
Le mar. 21 févr. 2023 à 01:41, Wenlin Chen @.***> a
écrit :

Hi! There is no implicit_diff argument in jaxopt.ScipyMinimize. Does this
mean that jaxopt.ScipyMinimize only supports implicit differentiation but
not autodiff of unrolled iterations? Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/google/jaxopt/issues/406, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABZB2WJ7Q5U3DSEESHM7STWYQEYRANCNFSM6AAAAAAVCNGZSA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


The .yml templates need to update from base 1.10 kubernetes version to 1.12, 1.13, 1.14 as 1.10 is no longer supported by EKS.

Page: /setup/install/providers/aws/aws-ec2/
Feedback:
The .yml templates need to update from base 1.10 kubernetes version to 1.12, 1.13, 1.14 as 1.10 is no longer supported by EKS.

@edwinavalos
@clareliguori

I'm not sure who owns / has access to those templates in CloudFront.  I think @anshrma created them originally.

This is causing a major issue for me as I made the mistake of upgrading my EC2 instances, but now they can't start because they rely on 1.10

@anshrma or @edwinavalos , can either of you take a look?

I'm not sure if this will fix it yet, as I'm still deploying, but in managing.yml I changed the EksCluster from 1.10 to 1.14 (the version my EKS instance is now). I'll post an update once deployment finishes.

cc @saimadineni

Duplicate of #1546


Return NoSigChainError for id2 of user w/ no sigchain

using KEY_NO_ELDEST status code for import/export of libkb.NoSigChainError.
Also fixed import of NoKeyError which had export, but no import.
r? @maxtaco
cc @strib

👍

retest please

retest this please


[BUG] Library throwing DEBUG log msgs, even when set for INFO Level.

Debug level logs are being thrown by the library, irrespective of what value is supplied for "verbose_console" param for constructor Picamera2(). It happens when log handlers are defined prior to calling of constructor.
Due to above behaviour, Logs are filled up with debug statements being generated by the library.
To Reproduce
import logging
from picamera2 import Picamera2
logger = logging.getLogger("")
formatStr = '[%(asctime)s] [%(threadName)s] [%(levelname)s]: %(message)s'
console = logging.StreamHandler()
console.setFormatter(logging.Formatter(formatStr))
logger.addHandler(console)
logger.setLevel(logging.INFO)
picam=Picamera2(verbose_console=1)
Console Output
[0:33:57.988721016] [91]  INFO Camera camera_manager.cpp:293 libcamera v0.0.0+3866-0c55e522
[0:33:58.091540556] [95]  WARN RPI raspberrypi.cpp:1258 Mismatch between Unicam and CamHelper for embedded data usage!
[0:33:58.093436522] [95]  INFO RPI raspberrypi.cpp:1374 Registered camera /base/soc/i2c0mux/i2c@1/imx219@10 to Unicam device /dev/media3 and ISP device /dev/media0
2022-09-09T06:32:59.722Z | INFO     | Initialization successful.
[2022-09-09 12:02:59,722] [MainThread] [INFO]: Initialization successful.
2022-09-09T06:32:59.723Z | INFO     | Camera now open.
[2022-09-09 12:02:59,723] [MainThread] [INFO]: Camera now open.
[2022-09-09 12:02:59,723] [MainThread] [DEBUG]: <libcamera._libcamera.CameraManager object at 0x7f90beda30>
Expected behaviour
As verbose_console is set to 1, below debug statement should not have generated
[2022-09-09 11:34:32,216] [MainThread] [DEBUG]: <libcamera._libcamera.CameraManager object at 0x7fae3a4bb0>
Hardware :
Pi 3B
OS: BullsEye
Package: python3-picamera2
Version: 0.3.3-1
To avoid the issue, below patch works for now (to be called after initialization of Picamera2)
logger = logging.getLogger('picamera2')
logger.setLevel(logging.INFO)

Hi, thanks for raising this. Unfortunately I don't know much about the Python logging module, but here's what I found out.
Firstly, it's clear the the debug message is being printed by the logger that gets created in the test script, not by the Picamera2's logger (this is clear from the format of the message).
So it seems to me like the Picamera2 logger is correctly not outputting anything but "propagating" the message to the parent, which is logger in the script. (I think logger is actually the "root" logger.)
Anyway, could it be that this "propagation" by-passes filters on the logger, but only heeds filters on the handlers? If you change logger.setLevel(logging.INFO) to consode.setLevel(logging.INFO) then the message no longer appears. In fact, I found this flow diagram which, if I'm interpreting it correctly, demonstrates exactly this behaviour.
Does this sound correct?  But in any case I can't really see that Picamera2 could do anything to provoke this behaviour, it does seem intrinsic to the logging module.

I've been experimenting a bit with this simplified test case:
import logging

class Dummy:
    def __init__(self, level=logging.INFO):
        self.logger = logging.getLogger("Dummy")
        logger.setLevel(logging.DEBUG)
        console = logging.StreamHandler()
        console.setLevel(level)
        console.setFormatter(logging.Formatter('[Dummy]: %(message)s'))
        logger.addHandler(console)

    def do_something(self):
        self.logger.debug("I am doing something")

logger = logging.getLogger()
logger.setLevel(logging.INFO)
console = logging.StreamHandler()
console.setFormatter(logging.Formatter('[Script]: %(message)s'))
logger.addHandler(console)
logger.setLevel(logging.INFO)
logger.debug("This should not appear")

dummy = Dummy(level=logging.INFO)
dummy.do_something()

which exhibits the same problem - it outputs the I am doing something message which is perhaps not expected.
If I replace logger.setLevel(logging.DEBUG) in the __init__ method by logger.setLevel(level) then the problem goes away. But then, if I also change dummy = Dummy(level=logging.INFO) to dummy = Dummy(level=logging.DEBUG) then the I am doing something message gets output by both loggers!
So it really does seem to be down to the rules of propagation and filters on loggers. If anyone can say what the most correct implementation of the Dummy class should be, then I'll copy that in Picamera2. Thanks!

@davidplowman I think you forgot something in the  DUMMY init function:
    def __init__(self, level=logging.INFO):
        self.logger = logging.getLogger("Dummy")  <---
        self.logger.setLevel(logging.DEBUG)       <---
        console = logging.StreamHandler()
        console.setLevel(level)
        console.setFormatter(loggin
        self.logger.addHandler(console)           <---

Without self you define properties of the global logger.
I have been bitten myself a few times by forgetting self for a class object ;-)

@tvoverbeek Ah, thanks for that, have edited the original in case anyone wants to try it! Though weirdly none of the behaviour seems to change. The rules here seem a bit twisty...  :(

OK, I believe I now understand how it works.
In the original post from @ajonnet there are 2 loggers defined.
The root logger in the main program and the picamera2 logger.
They are arranged hierarchically: any message from the picamera2 logger is also passed to the root logger.
The picamera2 logger has a setting logging.DEBUG.
The picamera2 messages are filtered by the streamhandler(s).
The picamera2 streamhandler blocks DEBUG mesages (console_verbosity=1)
But the root streamhandler has no level defined so it passes everything.
You see it in @ajonnet's output all INFO messages occur twice: first from the picamera2 logger followed by the root logger.
In @ajonnet's code (and in @davidplowman's example) replace the logger.setLevel(logging.INFO) just before
the picam=Picamera2(verbose_console=1) line with console.setLevel(logging.INFO. Similar in @davidplowman's example.
Then only the INFO messages appear twice and no DEBUG message.
However I think the logging concept should be changed.
From the 'Advanced Logging Tutorial', the section on "Configuring logging for a library"(https://docs.python.org/3.9/howto/logging.html#configuring-logging-for-a-library):
It is strongly advised that you do not add any handlers other than [NullHandler]
(https://docs.python.org/3.9/library/logging.handlers.html#logging.NullHandler) to your library’s loggers.
This is because the configuration of handlers is the prerogative of the application developer who uses your
library. The application developer knows their target audience and what handlers are most appropriate for
their application: if you add handlers ‘under the hood’, you might well interfere with their ability to
carry out unit tests and deliver logs which suit their requirements.

The picamera2 logger should only set the logging level and not add any handlers.
console-verbosity becomes logging-verbosity.
Without a NullHandler even when the user does not use a logger, warning and error messages will still be written
to stderr. I suppose this is the behavior we want.
This avoids the double logging messages as in @ajonnet's example.
What do you think?

OK, I believe I now understand how it works. In the original post from @ajonnet there are 2 loggers defined. The root logger in the main program and the picamera2 logger. They are arranged hierarchically: any message from the picamera2 logger is also passed to the root logger. The picamera2 logger has a setting logging.DEBUG. The picamera2 messages are filtered by the streamhandler(s). The picamera2 streamhandler blocks DEBUG mesages (console_verbosity=1) But the root streamhandler has no level defined so it passes everything. You see it in @ajonnet's output all INFO messages occur twice: first from the picamera2 logger followed by the root logger. In @ajonnet's code (and in @davidplowman's example) replace the logger.setLevel(logging.INFO) just before the picam=Picamera2(verbose_console=1) line with console.setLevel(logging.INFO. Similar in @davidplowman's example. Then only the INFO messages appear twice and no DEBUG message.
However I think the logging concept should be changed. From the 'Advanced Logging Tutorial', the section on "Configuring logging for a library"(https://docs.python.org/3.9/howto/logging.html#configuring-logging-for-a-library):
It is strongly advised that you do not add any handlers other than [NullHandler]
(https://docs.python.org/3.9/library/logging.handlers.html#logging.NullHandler) to your library’s loggers.
This is because the configuration of handlers is the prerogative of the application developer who uses your
library. The application developer knows their target audience and what handlers are most appropriate for
their application: if you add handlers ‘under the hood’, you might well interfere with their ability to
carry out unit tests and deliver logs which suit their requirements.

The picamera2 logger should only set the logging level and not add any handlers. console-verbosity becomes logging-verbosity. Without a NullHandler even when the user does not use a logger, warning and error messages will still be written to stderr. I suppose this is the behavior we want. This avoids the double logging messages as in @ajonnet's example. What do you think?

"configuration of handlers is the prerogative of the application developer who uses your library."
-This is the principle that should hold.
As part of my application code, i am already defining the required handlers and logging level.  But when initialize_logger() from picamera2_logger.py is called, it overrides the behavior i expect by defining log level to DEBUG for picamera2. Logs from picamera2, should have merged as per logging configuration defined in application code.
"The picamera2 logger should only set the logging level and not add any handlers."
-Here not even logging level should be defined by the library. It should be possible for application code to control what level of logs are to be generated.
I am naive with logs propagation principle. But expect it to be such that, logs from library principle should be controllable via application code.

I've taken a bit more advice, and the feeling seemed to be that it would be better not to set the logger's level at all. Also, the Picamera2.logger() function is probably unnecessary as folks who understand the logging module would use getLogger("picamera2"), and then configure everything for themselves. So I'll prepare a PR along those lines in due course and then link it to this bug report.


github.com/schneems -> github.com/zombocom

I noticed that the removed had a new URL.
May I ask why? :)

May I ask why? :)

Something like https://github.com/sharpstone/rack-timeout/issues/145?

Do you want this change in the changelog? 🤔

You can add [ci skip] to this PR title to skip the check.
I'm toying with the idea of moving some of my projects out of schneems namespace and adding some people on who might be willing or able to help maintain projects. individually there's not much to do, but when you add up all the projects there's a non-trivial maintenance burdeon.
I'm not sure if it's a good idea or not, but I'm thinking about it. I've also owned this namespace forever and have never used it. I like that it's less specific than github.com/codetriage so it's not limited to just a specific subset of my projects.

You can add [ci skip] to this PR title to skip the check.

Done!


Facing error while installing requirements.txt

Hi,
Don't know if its just me, but when I try to install the requirements.txt, I'm facing an error "Failed to build numpy
ERROR: Could not build wheels for numpy, which is required to install pyproject.toml-based projects
[end of output]"
To find out where the error was originating from, I decided to pip install all requirements individually, and found that this error is coming up while installing scipy.
So just wanted to understand, did I go wrong somewhere or if others faced a similar issue or not.
******Update: Did find that I am able to install newer version of scipy. So is it necessary to to have scipy==1.5.4?
Thanks

"error: Command "x86_64-linux-gnu-gcc -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -fPIC -DNPY_INTERNAL_BUILD=1 -DHAVE_NPY_CONFIG_H=1 -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE=1 -D_LARGEFILE64_SOURCE=1 -Ibuild/src.linux-x86_64-3.1/numpy/core/src/umath -Ibuild/src.linux-x86_64-3.1/numpy/core/src/npymath -Ibuild/src.linux-x86_64-3.1/numpy/core/src/common -Inumpy/core/include -Ibuild/src.linux-x86_64-3.1/numpy/core/include/numpy -Inumpy/core/src/common -Inumpy/core/src -Inumpy/core -Inumpy/core/src/npymath -Inumpy/core/src/multiarray -Inumpy/core/src/umath -Inumpy/core/src/npysort -I/home/namancho/LLM_Forecast/Time_LLM/Time-LLM/trial/include -I/usr/include/python3.10 -Ibuild/src.linux-x86_64-3.1/numpy/core/src/common -Ibuild/src.linux-x86_64-3.1/numpy/core/src/npymath -Ibuild/src.linux-x86_64-3.1/numpy/core/src/common -Ibuild/src.linux-x86_64-3.1/numpy/core/src/npymath -c build/src.linux-x86_64-3.1/numpy/core/src/multiarray/scalartypes.c -o build/temp.linux-x86_64-cpython-310/build/src.linux-x86_64-3.1/numpy/core/src/multiarray/scalartypes.o -MMD -MF build/temp.linux-x86_64-cpython-310/build/src.linux-x86_64-3.1/numpy/core/src/multiarray/scalartypes.o.d" failed with exit status 1
[end of output]
    note: This error originates from a subprocess, and is likely not a problem with pip.
    ERROR: Failed building wheel for numpy
  Failed to build numpy
  ERROR: Could not build wheels for numpy, which is required to install pyproject.toml-based projects
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error
× pip subprocess to install build dependencies did not run successfully.
│ exit code: 1
╰─> See above for output."
This is what I'm getting

What version of python are you using?

What version of python are you using?

3.10.12

I ran into a similar issue when initially trying to use 3.11
On Ubuntu this fixed my issue:
!apt-get install python3.9 python3.9-distutil

I ran into a similar issue when initially trying to use 3.11
On Ubuntu this fixed my issue:
!apt-get install python3.9 python3.9-distutil

Ok, yes running with python 3.9 worked, atleast the requirements are installed now, thanks.


Q&A : compatibility with real Azure DevOps

Is this product compatible with the actual online version of Azure DevOps?
In other words: can it be used to host a local cache of rather big TFVC repo migrated over from on prem older TFS to Azure DevOps? It is especially painful for any newcomers or when branching, as downloading bigger chunks of code inevitably triggers throttling on Azure DevOps side, and the whole process takes unnecessarily long.

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

ID: 8f1b119c-e0a8-5143-bece-d2d0b8556bf6
Version Independent ID: d819bd4f-5272-d350-348e-dd00cc9417c3
Content: How to install Azure DevOps Proxy Server - Azure DevOps Server
Content Source: docs/install/install-proxy-setup-remote.md
Product: devops-server
Technology: tfs-admin
GitHub Login: @aaronhallberg
Microsoft Alias: aaronha

Any information on this?
Ping @aaronhallberg
For the time being, I also logged a request at the developer community: https://developercommunity.visualstudio.com/content/problem/818233/is-the-azure-devops-proxy-server-compatible-with-a.html

Yes, the proxy supports caching files from Azure DevOps organizations. Sorry for the slow response.

What is the process ? I get this issue when setting up a proxy for AzureDevOps

@kishorerv93 You should specify the full url, https://dev.azure.com/org


Introduce ES6 import-statement

Refactor all usages of require to the new import-statement.
So this
var myModule = require("../js/myModule");

becomes this
import myModule from "../js/myModule";

As I see this as a tech-debt issue, I'd suggest to provide a pragmatic explanation to why this is needed with benefits, pitfalls, etc.

@pierlo-upitup Good point. Until this is done, this issue is not implementable.


feat(tooltip): support textColor

Checklist
List of tasks you have already done and plan to do.

[ ] Fix linting errors
[x] Tests have been added / updated (or snapshots)

Change information
The tooltip component supports textColor.
Issues
close #1749.
Related Links
Links related to this pr.

Thanks for your great work!


Edit hdinsight-storm-develop-csharp-event-hub-topology.md

Edit complete: Also asking Larry to check this:
Line 289: The whole table is BOLD. It is a little overwhelming ;)

Hi @ClarityQueen, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (v-diree). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Waiting for Larry's approval.

Induced merge conflict to sort out changes. I'll meet with Larry to confirm.

Merge conflict resolved in another pull request:
https://github.com/Azure/azure-content/pull/3272#issuecomment-90164186


Shell-escaped the project path that we use for xcodebuild and lipo.

Resolves issue #88.
Shellwords is built into Ruby, see the doc for what it does here: https://ruby-doc.org/stdlib-2.0.0//libdoc/shellwords/rdoc/Shellwords.html
In short, a path string like /hello world' becomes suitably escaped for shell use, and transformed into /hello\ world`.
.shellescape was added to string variables passed to both xcodebuild and lipo,  and now things work as expected when there are characters (like spaces) that need to be escaped in the file paths.

Please Note: the failing build has nothing to do with my change...

I can confirm and reproduce same issue in our project, and can confirm this PR fixes it.
Would be great if this PR could be merged @leavez

I can also confirm that it's an issue with projects where the build scheme contains spaces. This PR fixes the problem, please consider merging @leavez

This PR fixes the problem with spaces in build scheme. Please merge it @leavez


Add top level OWNERS for the repo

Adding the top level OWNERS file similar to other maintained kubernetes repos.
This is required for CI test jobs.
/assign @pipejakob
/cc @kubernetes/sig-cluster-lifecycle-pr-reviews

Nice! Looks good in theory, but don't we also need an OWNERS_ALIASES file to define these groups? I don't think our munger supports actual GitHub groups:
https://github.com/kubernetes/test-infra/blob/master/mungegithub/mungers/approvers/README.md

Thanks for your careful review @pipejakob, i somehow thought adding github team would work. Anyway, i have updated the OWNERS file now with individual members. It may not be accurate, but serves the immediate purpose and maintains status quo.
Going forward we can deliberate on the approvers/reviewer list.

Thanks @pipejakob and yes i have raised pr in test-infra to add a pre-submit job which does run on every pr to k8s-anywhere repo against stable versions of k8s and kubeadm. Once the test is passed and pr has an lgtm and approve labels, the pr is gonna be automatically merged by bot.
here is the pr https://github.com/kubernetes/test-infra/pull/5595. it would be great  if you can review the pr.


tools: check-theme: consider background-color too

This is the precise property we want to check.

Thanks!

Thanks!


Refactor: base GUI table on QStandardItemModel

This is kind of necessary as a prelude to wiring the Cell() class up to the table in #221, but it's also something I've been working towards for a while.
Pandas is good for working with tables of numbers & doing calculations, but it's not a great match for what we want in a GUI: it doesn't really know about formatting or images, pulling out individual values to display in the GUI is not super efficient, and adding new rows is a bit awkward. Qt provides the QStandardItemModel to represent rich tables in memory (as opposed to lazily loaded tables), and of course it fits neatly with GUI expectations, so this is what this PR uses instead of pandas.
I also discovered a neat piece of machinery to allow sorting (QSortFilterProxyModel), which means the order within the DamnitTableModel doesn't need to change, and we can always add new rows at the end. However, this does mean the indexes we get from the view have to be translated through the proxy model to find the relevant data.

(I'm dragging in Thomas M and Cammille since I'll be away for the rest of the week :stuck_out_tongue:)

Right, enjoy your time off :beach_umbrella:

@tmichela asked me about performance, so I did some simple timings with a copy of the DAMNIT folder from p6616, which has 320 runs and 6742 variables stored. I measured from running the amore-proto gui ... command to when the window is drawn, in FastX on max-exfl-display.

master: 19 s, 19 s, 18.5 s
this branch: 9.7 s, 9.3 s, 9.6 s

So at least for this case, we've got a nice speedup.
When I've profiled in the past, the DamnitTableModel.data() method has taken a lot of time in total. This is what Qt calls to get information from the model, and even though it's quite fast, it's called so many times that small inefficiencies mount up.

BTW, this is quite an invasive change (and our tests are not as good as I thought they were) so I might suggest deploying it to the beta module for a couple experiments before deploying to the stable module.

Thanks, that's a good idea. And this should be a case where people can freely switch between the stable & beta versions, so it's easy to fall back if something breaks.

If no one else spots anything then I'd say let's just deploy it to beta once this weeks experiments are over and see what happens

I've also looked through the code and tested it without breaking it, so I'd agree to do that. 👍

Thanks! Are you thinking we merge this and then deploy it to beta, or deploy it from the branch to give it some more testing before merge?

Let's merge it and deploy to the beta, otherwise it's a bit confusing to keep track of.


TypeError: expected a character buffer object

I'm running into this repeatedly when I use mentions with a little slack bot I'm building with SlackSocket (great module btw! :-)
Not sure if I'm doing something weird, but I was easily able to "fix" it on my machine by changing the line 42 of models.py, so that SlackEvent._get_mentions converts each mention to unicode instead of str before calling .translate.
Here's the full stack trace with debugging logging enabled:
ERROR:root:error from callback <bound method SlackSocket._event_handler of <slacksocket.client.SlackSocket object at 0x10acc5c50>>: expected a character buffer object
  File "/Users/ses/w/pivo/venv/lib/python2.7/site-packages/websocket/_app.py", line 243, in _callback
    callback(self, *args)
  File "/Users/ses/w/pivo/venv/lib/python2.7/site-packages/slacksocket/client.py", line 367, in _event_handler
    event = SlackEvent(json_object)
  File "/Users/ses/w/pivo/venv/lib/python2.7/site-packages/slacksocket/models.py", line 31, in __init__
    self.mentions = self._get_mentions(self.event['text'])
  File "/Users/ses/w/pivo/venv/lib/python2.7/site-packages/slacksocket/models.py", line 42, in _get_mentions
    return [str(m).translate(translate_map) for m in mentions]

Since I'm not sure if this is somehow my fault I decided to make this issue and not a PR :-)
lib version is 0.6.3
operating system: OS X 10.11
python version:
$ venv/bin/python --version
Python 2.7.10

Thanks for pointing this out @steinnes; this is in fact a py2 vs py3 bug.
I've pushed https://github.com/bcicen/slacksocket/commit/6e043ebda4b2fc229e044505703ef602ed86a105 to coerce the mentions to unicode before translating if using python2 and will include this in the next release. Thanks again for the feedback!

I suspected something like that (ie. I couldn't be the only one using mentions).  Glad to help :-)

fix included in v0.6.5, pushed to Pypi as well


miss schedule cron setting

@tsaeki commented on Fri Aug 02 2019
https://docs.microsoft.com/ja-jp/azure/azure-functions/functions-bindings-timer#java-example
Good
schedule = "0 */5 * * * *

ドキュメントの詳細
⚠ このセクションを編集しないでください。 docs.microsoft.com で必須です ➟ GitHub の問題のリンク。

ID: b5527c3b-a5da-c73d-66c2-20d96003ff7c
Version Independent ID: 42154c7b-b0e0-26db-79bc-3f1673c48e40
Content: Azure Functions のタイマー トリガー
Content Source: articles/azure-functions/functions-bindings-timer.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@srvbpigh commented on Sat Aug 03 2019
Hello, @tsaeki
Thank you for your feedback.
We are actively reviewing your comments and will get back to you soon.
Kind regards,
Microsoft DOCS International Team

Hi team,
I'm moving this here since the user is making a suggestion on the code box

Hi @CeciAc & @tsaeki - Thank you for reporting this issue and helping us drive improvements towards MS Docs. I've created a PR that addresses this issue and the change should reflect within 24hrs.
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


[CR-56] Update machine types for 2022 cloud report run

One possible issue: I couldn't find E*bs_v5 machine types on Azure's official page for E series, neither could I find the type in the Azure console, so likely this one is not available to run for us.
Work item: https://cockroachlabs.atlassian.net/browse/CR-56
Release note: None

TFTR!


Newsletter Subscribe

Description of your changes
Add a feature of newsletter subscribe via MailChimp
"Fixes #801":
I have:

[ x] Read and followed KubeVela's contribution process.
[x ] Update sidebar.js if adding a new page.
[ x] Run yarn start to ensure the changes has taken effect.

Special notes for your reviewer
The official account and official mail should be used to deliver mail and for the API key,

I'll post the effect:

Who is hosting the Mailchimp's service? Does Mailchimp have free plan for open source project?

Sorry but I have to revert the change first and we can keep working on this until everything is clear.


Failed to update Power BI File

Hi,
i have an issue with updating Power BI File from Tabular Editor.
I can open the file, change measures, but when I want to save it, i get an error below.
Please advise me what to do.
Regards,
Borut

Could not save metadata changes to database
Failed to save modifications to the server. Error returned: 'Nepričakovano ime stolpca: stolpec »ObjectID.Expression« je bil prejet v naboru vrstic »ObjectTranslations«. Pričakovan je bil stolpec »ObjectID.Set«.
'.
Save the model as a file to make sure you do not lose any work. In case the database connection was lost, you can still apply your changes to the database by using the "Model > Deploy" option.
V redu

Duplicate of https://github.com/otykier/TabularEditor/issues/287


row count post-mance > row count pre-mance

strangely, it looks like the last row gets copied n times, where n is the # of datasets selected to be added
found this bug while mancing a dummy dataset on state

@tthibo @cathydeng Have either of you observed this since this issue was filed? As far as I can tell, this is resolved.

nope, haven't encountered this recently


Question about updates

I installed rust-analyzer successfully for vscode, following the automated steps in the guide. Is it safe to delete the folder I git cloned now that I've got the extension in vscode? Also will I get updates automatically or should I be git cloning every major release and redoing the steps over again? Thanks for working on RA.

Updates are not automatically. To speedup updating, you can use git pull inside the folder, instead of cloning it again. You can safely delete the folder if you want. The language server itself is found in .cargo/bin inside your home dir and the ext is found inside the vscode config dir.

Updates are not automatically. To speedup updating, you can use git pull inside the folder, instead of cloning it again. You can safely delete the folder if you want. The language server itself is found in .cargo/bin inside your home dir and the ext is found inside the vscode config dir.

Thanks!


Remote-SSH broken after update to 1.46.1

VSCode Version: 1.46.1 (user setup)
OS Version: Windows_NT x64 10.0.18363

Version: 1.46.0 (user setup)
Commit: a5d1cc28bb5da32ec67e86cc50f84c67cc690321
Date: 2020-06-10T09:03:20.462Z
Electron: 7.3.1
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363

Steps to Reproduce:

Update VSCode from v1.46.0 to v.146.1
Remote-SSH: Connect to Host...
Choose host from list of hosts

After this, VSCode will continuously retry connecting (~30 times) to the host and eventually fail. The output has a lot of:
[11:41:50.199] > c775c92b5a1e: running
>
[11:41:50.232] > Acquiring lock on /root/.vscode-server/bin/cd9ea6488829f560dc949a8b2fb789f3cdc05
> f5d/vscode-remote-lock.root.cd9ea6488829f560dc949a8b2fb789f3cdc05f5d
>
[11:41:50.240] > \ln /root/.vscode-server/bin/cd9ea6488829f560dc949a8b2fb789f3cdc05f5d/vscode-rem
> ote-lock.root.cd9ea6488829f560dc949a8b2fb789f3cdc05f5d.target /root/.vscode-serv
> er/bin/cd9ea6488829f560dc949a8b2fb789f3cdc05f5d/vscode-remote-lock.root.cd9ea648
> 8829f560dc949a8b2fb789f3cdc05f5d
> ln: failed to create hard link '/root/.vscode-server/bin/cd9ea6488829f560dc949a8
> b2fb789f3cdc05f5d/vscode-remote-lock.root.cd9ea6488829f560dc949a8b2fb789f3cdc05f
> 5d': File exists
> Installation already in progress...
> c775c92b5a1e##24##
>
[11:41:50.240] Received install output: c775c92b5a1e##24##
[11:41:50.240] Server installation process already in progress - waiting and retrying
[11:41:50.520] "install" terminal command done
[11:41:50.521] Install terminal quit with output:
[11:41:51.243] Running script with connection command: ssh -T -D 56960 <host ip here> bash
[11:41:51.245] Terminal shell path: C:\WINDOWS\System32\cmd.exe
[11:41:51.296] >

I replaced the IP with <host ip here>.

If I revert back to v1.46.0, I can Remote-SSH again to my host.

Does this issue occur when all extensions are disabled?: Yes (with only the Remote - SSH and Remote - SSH: Editing Configuration Files extensions enabled)

I met the same problem

i met the same problem，too！ 100734
but no one responsible for solving this problem!

Is there a bug label or some way to add labels? I don't know why the new release label was removed, nor how to attract attention to this issue for someone to pick up

FWIW, I ran into this issue, and running rm -rf ~/.vscode-server on the remote machine resolved it.

Sorry, this is https://github.com/microsoft/vscode-remote-release/issues/2518, and running the command "Kill VS Code Server on Host" is the suggested workaround


assign mean costs to Kernnetz pipes

Before asking for a review for this PR make sure to complete the following checklist:

[x] Workflow with target rule ariadne_all completes without errors
[x] The logic of export_ariadne_variables has been adapted to the changes
[x] One or several figures that validate the changes in the PR have been posted as a comment
[ ] A brief description of the changes has been added to Changelog.md
[x] The latest main has been merged into the PR
[x] The config has a new prefix of the format YYYYMMDDdescriptive_title

fix: pipes from Kernnetz data in the model are over-proportinally many retrofitted projects as they are longer
do not use own cost data for retrofitted and new build pipes as this would underestimate the investment cost
reconstruct average Kernnetz invest (250€/MW*km) from our costs data

Before:

we only captured 9 Mrd € for roughly 80 % of the Kernnetz investment (0.8*15 = 12 Mrd €)

After:
now we cover 12 Mrd €

the difference in the plots is because of the difference currency years (2020 vs. 2023)


expect on npm is now mapped to facebook/jest not this project

tl;dr Following the installation instructions in the README.md actually installs facebook/jest, not this project.

Explanation
👋 I recently saw greenkeeper create a PR on a project I contribute to asking us to update our expect dependency to 21.0.0. After investigating, expect on npm is now at version 21.0.0 and the link to the source is github.com/facebook/jest, not this project.
@cpojer I saw that you were the last to publish to https://www.npmjs.com/package/expect, did you mean to publish jest instead of this project under the expect name?
If so, I think the README.md Installation section should get changed to match wherever this project is listed on NPM.

Confirming I wasn't crazy
I double checked I wasn't crazy by running  npm  install --save expect as the instructions say to do with a fresh node project and I end up with jest installed not this project:

Please let me know if I can help sort this out. Thanks 🙌
/cc @mjackson @cpojer

See https://github.com/mjackson/expect/issues/226 and https://github.com/mjackson/expect/pull/227

Missed those--thanks.


Is SWRevealVIewController support the device orientation at upside down?

I try to set the project with panel in "Device Orientation" at "upside down".
But When I rotation my iPhone upside down. The effect is not changed.
How can I set the upside down orientation with SWRevealViewController? Thank you very much.

In your ViewController you have to override this function:
override func supportedInterfaceOrientations() -> UIInterfaceOrientationMask
{
    return .All
}

It's okey! Thank you very much~


Avoid auto_parse for values having surrounding quotes

Using version 1.3.3

For the following input:
1,2,"3",4

using following options:
{auto_parse:true, trim: true}

I am expecting: [1,2,'3',4] but everything is parsed to an integer value.

To handle this I wrote a custom auto_parse function:
function autoParse (value) {
    if (value.charAt(0) === '"' && value.charAt(value.length - 1) === '"') {
        // remove surrounding double quotes and skip further parsing
        value = value.slice(1, -1);
    } else if (util.isInt(value)) {
        value = parseInt(value);
    } else if (util.isFloat(value)) {
        value = parseFloat(value);
    } else if (this.auto_parse_date) {
        var m = Date.parse(value);
        value = isNaN(m) ? value : new Date(m);
    }
    return value;
}

with following options:
{auto_parse: csvAutoParse, trim: true, quote: true}

BUT the problem with this is, input like "Hello, World", 2 is parsed to ['"Hello', 'World"', 2] instead of ['Hello, World', 2].

In short:

Need the quoting context like version 2.5.0
Can't update to v2 because of older Node version support.
Finally, avoid parsing values surrounded by quotes.

Simply write a simple auto_parse function and use the second argument which contains the quoting property: https://github.com/adaltas/node-csv-parse/blob/master/test/options.auto_parse.coffee#L62-L76

@wdavidw As mentioned I am using version 1.3.3.
Can't update to v2 because of older Node version support.

Sorry, I missed this part, you can require './lib/es5/index.js'. It is the same as './lib/index.js' but transpiled with babel for backward compatibility.


Refactor backend integration tests

Closes #212

Coverage decreased (-0.002%) to 99.187% when pulling 1b49634f4b6a226421d1d833a3b42843b2136ac5 on JWCook:integration-tests into 38ddcf5425eadc6b174a8b053b2175c4dda00a1f on reclosedev:master.


Can't save PCANet model

When i use "python3 run_mnist.py --gpu 0 train --out result" to train PCANet ,but get this error message
_pickle.PicklingError: Can't pickle <class 'pcanet.PCANet'>: it's not the same object as pcanet.PCANet
How can I solve this problem?
thank you

Thank you for the report.
It was because of the odd behavior of pickle and now the problem should be solved.

It's work, thank you.


Lighting

Lighting

@fooooooooooooooo

lighting looks fine to me

lighting looks fine to me

No it sucks rely bad sorry it is unlit no shading no lighting its really unrealistic

unfortunately i would probably have to rewrite the renderer

skill issue??!???!??

huge i did it


Added my name to contributors list

Trying out setting up Dialtone on my machine, so I added my name to the contributors list to test out my understanding of the process.
@TedGoas , I'm not sure why it's saying I edited some of the css files, I didn't touch them at all. Is there a way to only approve that one change I did and not the rest?

@abasolo-dialpad It's fine. The CSS files are compiled with every build. These CSS files are only used in the Documentation site.


fix toBlob method

on wkwebview engines, the toBlob is called with its context undefined (??)

@Peshou i forgot to compile the files into js, i wonder if that matters at all?

Yea it needed to be compiled, i'll roll another 2.0.2 version as well with the compiled version. Thanks

Aight thanks a lot, I forgot to bind the 'this' operator since I did not call the function immediately, sorry about that. The version is 2.0.2 where this fix is added.


Gradient Descent vs Gradient Ascent

In the introductory section the article describes

This kind of optimization is possible because the networks are differentiable with respect to their inputs: we can slightly tweak the image to better fit the desired properties, and then iteratively apply such tweaks in gradient descent.

My understanding is that such the optimization techniques use Gradient ascent algorithm to optimize the input image. Although, GD can be used with updating input in opposite direction to what GD normally does, but it is nothing but GA. Could you please clarify.
Thank you for making your research available openly. It is of immense help for my learning.

Hi @hegman12, thanks for your interest! You can think of it as gradient ascent, and may even find that many authors play a little loose with these two terms. We may have been used to "descent" because tensorflow's Optimizer class's function for creating an optimization operation is called minimize.
The bigger picture is that you should not end up being confused due to our imprecision. @colah how do you feel about me just changing this particular instance to "gradient ascent"?

It seems like people in the deep learning community tend to frame optimization problems as a loss that you minimize, even when they're actually increasing things. So, descent seems a bit more conventional at this point. Additionally, discussion of optimization algorithms tends to use the descent terminology, so searching for that term is probably more helpful. Implementations, such as TensorFlow, also tend to focus on minimization.
That said, I don't have a strong view and would be happy to defer to others if someone feels strongly.

(I'm going to close this for now, but happy to reopen if anyone feels strongly.)


Event becomes undraggable after using eventRenderer

title: "Undraggable Behavior with eventRenderer in Latest Version"
Issue Description
In previous versions, like 2.5.13, the following code snippet with an eventRenderer function worked perfectly:
eventRenderer={(event) => {
  return
{event.title}
;
}}

I'm aware that it must be {event} in newer versions

Please upgeade to 2.6.7
You will use it like:
eventRenderer={({event, ...props}) => {
  return
{event.title}
;
}}

The code above works fine for month view, but it doesn't work for the week view, can you please check it out?
https://codesandbox.io/embed/custom-event-renderer-forked-hmr69j?fontsize=14&hidenavigation=1&theme=dark

okay got it thanks


quote!  and quote_spanned! macro should take &mut TokenStream

Hey! dtolnay, Thanks for your great work!  ❤
I think quote!  and quote_spanned! macro should take &mut TokenStream instead of creating new one.
Instead of this:
let import = items.iter().map(|item| quote_spanned!(item.span()=> use #item));
quote! {
    #(#import)*
    // ...
}

I think it should look something like this:
// `to_tokens_fn` accept closure and return a struct that implement `ToTokens`

let import = to_tokens_fn(|tokens| {
    for item in items {
        quote_spanned!(tokens, item.span()=> use #item);
    }
});

impl ToTokens for <Struct> {
    fn to_tokens(&self, tokens: &mut proc_macro2::TokenStream) {
        quote!(tokens, {
            #import
            // ...
        })
    }
}

This approach should be more performance, Because it avoid cloning.
Sorry if this is duplicate.

I would prefer not to support this in this crate.


[<IP_ADDRESS>] Could not add 2nd data node in a cluster

I was able to add and run 3 raft nodes + 1 data node, however when I was trying to add the second data node (which is the 5th node in the cluster), the node could not start up and had the following error in the log :
2015/09/17 22:31:48 InfluxDB starting, version <IP_ADDRESS>, branch 0.9.4, commit c4f85f84765e27bfb5e58630d0dea38adeacf543
2015/09/17 22:31:48 Go version go1.5, GOMAXPROCS set to 1
2015/09/17 22:31:48 Using configuration at: /config/config.toml
[retention] 2015/09/17 22:31:48 retention policy enforcement terminating
[monitor] 2015/09/17 22:31:48 shutting down monitor system
run: open server: open meta store: too many peers; influxdb v0.9.0 is limited to 3 nodes in a cluster

Original discussion: https://groups.google.com/forum/#!topic/influxdb/LWWrdJRt3kc
cc: @imlk

CC @jwilder @corylanou
@jwilder - should adding a 2nd data-only node work?

Yes

Closing this since clustering is no longer supported in the open source version.


DataStore doesn't emit ready event.

Describe the bug
Essentially, I'd like to reopen https://github.com/aws-amplify/amplify-ios/issues/1302. Everything is described there.
I can reproduce that the ready event doesn't get emitted unless I re-introduce my old hack, that I published here.
Steps To Reproduce
Steps to reproduce the behavior:
1. Delete App to ensure sync from scratch
2. Run App, triggering initial sync
3. Observe items syncing
4. Observe no more syncing activity, while `ready` not having been emitted

Expected behavior
I expect DataStore to reliably emit the signal.
Amplify Framework Version
1.26.1
Amplify Categories
DataStore
Dependency manager
Cocoapods
Swift version
swift-driver version: 1.45.2 Apple Swift version 5.6 (swiftlang-<IP_ADDRESS>3.62 clang-13<IP_ADDRESS>)
CLI version
8.2.0
Xcode version
Xcode 13.3.1, Build version 13E500a
Relevant log output
No response
Is this a regression?
Yes
Regression additional context
No response
Device
Simulator
iOS Version
15.4
Specific to simulators
haven't tried
Additional context
No response

Hi @anatomybook, for the code snippet from https://github.com/aws-amplify/amplify-ios/issues/1302#issuecomment-906404001
if reconciledReceived == recordsReceived {
   dispatchModelSyncedEvent()
}
else {
    initialSyncOperationFinished = true
}

within onReceiveSyncOperationEvent. Both onReceiveSyncOperationEvent and onReceiveReconciliationEvent will be called in parallel multiple times as the sync operation retrieves more pages of items while the reconciliation applies the mutations. I think your code snippet makes sense because if there is a delay in receiving the finished event from the sync operation, and is emitted after the last reconciled event then the modelSynced event will never get emitted. It won't get emitted because no more items get reconciled to trigger the check, so adding the check in the finished event of the sync process would make sure the event is emitted.

Thanks for bringing this to our attention @anatomybook , I added the PR https://github.com/aws-amplify/amplify-ios/pull/1989 with your code change here.

Thanks for considering my proposal.

this was merged in 1.28.0

I'm still seeing this issue more often than not. I'm using version 2.1.0 of amplify-swift. I always see the log output [AWSDataStorePlugin] [Lifecycle event 6]: ready but way more often than not, the event doesn't come through the hub's datastore publisher.


Add gestures to manage extension bundles

Related: https://github.com/Azure/azure-functions-core-tools/issues/1913
V2 extension bundles are on the way, so an app could be in these 4 states:

No extensions or extension bundles
Manually installed extensions in extensions.csproj
Bundles V1
Bundles V2

Would like to add a gesture to set extension bundles in a project (perhaps a context menu on host.json):

V1 -> V2
V2 -> V1
No extensions -> V1/V2
extensions.csproj -> V1/V2

User should be warned about the consequences and include an option to open bundles docs. The extension also needs to update the launch settings accordingly.
Unlike the Core Tools, VS Code probably doesn't need to consider moving from bundles to manually installed extensions.

@anthonychu can you expand on the motivation for this issue?
AFAIK - the only change required to switch between bundle version is editing host.json. If the user is advanced enough to know they need to change bundle version - would a right click menu to change bundle add value over simply editing the file?

I think there are cases where the user isn't necessarily "advanced" but they need to do it to take advantage of a capability. A concrete example is a JS dev who wants to use Durable Entities, which needs Durable Functions V2 in bundles V2.. The first thing they need to do after creating a new project is to change to bundles V2 before entities would work and they see the entities template (when the template becomes available).
This gesture will help in docs as well, especially in quickstarts. In the case of Durable Functions, one of the first steps in the quickstart will be to switch to Bundles V2, and the simpler we make that, the less likely that we'll lose the person on that step.
I think in general it would be great to make it easier to change the bundle version and minimize mistakes, even for advanced users.
I know there's also an obvious way to solve it and that is to default to v2, but I think we might want to wait on that. In the past we talked about establishing a cadence for updating the defaults for tooling. /cc @mattchenderson @soninaren

I just feel like updating a dependency version is a common concept - in a package.json, requirements.txt, or csproj. I'm not saying these have to be super advanced users, but if they've figured out they need a non-default version of a bundle, it seems on-par for them to update that version in a functions-specific file (host.json).
In terms of the docs, this:

If you're using VS Code, right click on your "host.json" file, select "Update bundle" and choose "V2"

doesn't feel any simpler to me than this:

Ensure the version of your extension bundle in "host.json" is [2.*, 3.0.0)

In addition the second option has the following benefits:

It's the same regardless of CLI, VS Code, or no tooling at all
It teaches the user how the bundle is actually controlled and where to go if they have a problem with bundles
It's not hiding anything. The first option makes me suspicious there's more going on under the covers for a simple version update


Enhance table handling

I think the table behavior can be improved:

When the table gets blurred, remove empty rows
When hitting the backspace / shift-tab on the first cell & first column of an empty row, remove the empty row
On mouse down for a long time on the table should drag the whole table

Can you confirm that holding the mouse down for a longer time doesn't work for you. That behaviour is already in place and works for me currently - however could be there's a problem in a specific browser.
With regard to removing empty rows when they're blurred I don't see any reason not too.

I have not noticed that when you hold the mouse longer it turned into a table drag. Alright, never mind that :+1:

@bfintal @Qqwy OK so I've pushed an update that implements the second suggestion...

When hitting the backspace / shift-tab on the first cell & first column of an empty row, remove the empty row

...but not the first. Deleting all empty rows on blur didn't end up feeling as intuitive as I expected, for one copying into a table from the document becomes hard because creating a new row then selecting text from the page removes the new row.
I have however put in place support for empty rows using the delete key however, so if the row is empty and you press the delete key with any table cell selected the row will be removed. This makes it much easier navigate through and delete empty rows.
We're further looking at table behaviour in a new issue #152 posted by @Qqwy which has some additional suggestions we might try, so for now I'm closing this issue and suggest any further conversation is moved to the new issue.


Oracle Dialect Quote Brackets

I am using this library to generate queries for Oracle (Oracle Database 12c Enterprise Edition Release <IP_ADDRESS>.0 - 64bit Production) and it generates table and column names with square brackets around them. For example:
Actual: Select [column_name] from [table_name];
Expected: Select column_name from table_name;
I did some analysis and found that the brackets are coming from the Quote method in oracl_dialect file. Is there a way to disable the quote application? or to go back to "\"" quote format?
If not please consider fixing it. Thanks in advance.

The reference git commit in which this change was brought in: https://github.com/go-xorm/xorm/commit/b757473de7f547d767fc5425c060026d5c89af5c?diff=split


Document that Pygments styles can be used for --code-theme

The --code-theme configuration option can accept any of the Pygments builtin styles (see #49). I had to dig into the source to discover this, so this PR simply makes that better-documented for users.

Thanks!


Add custom servlet

With Selenium Server there is an option to add custom servlets, which i currently use to manage video recordings. Is there any configuration options to perform a similar setup with selenoid?  This is currently my only holdback from implementing this and dropping selenium server.

I am using vnc2flv for video recording with selenoid. This works pretty good.

Is there any documentation you can provide for this integration?
On Wed, Oct 25, 2017 at 3:43 PM Markus Krüger<EMAIL_ADDRESS>wrote:

I am using vnc2flv for video recording with selenoid. This works pretty
good.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aerokube/selenoid/issues/249#issuecomment-339465205,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGLFrK4kjsd4XppqnrL9xU4d-jj1OlAOks5sv519gaJpZM4QGmjL
.

@rab2215 we plan to integrate video recording directly to Selenoid in next releases. However there is no Servlet functionality because it is not Java. Selenoid is 100% Golang project,

@vania- pooh damn, just spent some days with vnc2flv integration :-D how is the integration planned? Will the cpu of the node/container be affected or the sytem executing the tests?

@vania-pooh I see. So it would be no solution for selenoid on windows machines?

@benzman81 at least no for the first implementation. If you have any proposals of how to do this - please go ahead.

@rab2215 no documentation but code ;-) in the end I start the system process flyrec.py to start recording onTestStart and kill this process with kill -2 onTestEnd. But if you use selenoid with docker containers I would wait for their solution.

@vania-pooh no proposals, but wouldn't it be better to adress this additionally in ggr? no clue how to solve it nicely, but in a big grid using ggr a solution is needed as well.

@benzman81 Ggr already support proxying VNC by session ID. This works since release 1.3.0.

@benzman81 can you share the solution with vnc2flv?

@SergeyPirogov should be something like this: https://stackoverflow.com/questions/45364429/selenoid-recording-video-of-a-running-session/45662657#45662657
@rab2215 created https://github.com/aerokube/selenoid/issues/255 for video recording implementation. Let's continue and video-related discussion there. Closing this one.


qdrant: init at 0.9.1

Description of changes
Qdrant (read: quadrant ) is a vector similarity search engine. It provides a production-ready service with a convenient API to store, search, and manage points - vectors with an additional payload. Qdrant is tailored to extended filtering support. It makes it useful for all sorts of neural-network or semantic-based matching, faceted search, and other applications.
Qdrant is written in Rust crab, which makes it fast and reliable even under high load.
With Qdrant, embeddings or neural network encoders can be turned into full-fledged applications for matching, searching, recommending, and much more!
https://github.com/qdrant/qdrant

Things done

Built on platform(s)

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

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

[x] Tested, as applicable:

and/or package tests

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage

[x] Tested basic functionality of all binary files (usually in ./result/bin/)

22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

On macos 10.15:
running: "/nix/store/md0r2kkc0wg2gk18aaim529krci9kv3w-clang-wrapper-11.1.0/bin/c++" "-O3" "-ffunction-sections" "-fdata-sections" "-fPIC" "-m64" "-arch" "x86_64" "-I" "rocksdb/include/" "-I" "rocksdb/" "-I" "rocksdb/third-party/gtest-1.8.1/fused-src/" "-I" "snappy/" "-I" "." "-Wall" "-Wextra" "-std=c++17" "-Wsign-compare" "-Wsh
  cargo:warning=In file included from rocksdb/cache/fast_lru_cache.cc:10:
  cargo:warning=rocksdb/cache/fast_lru_cache.h:165:3: error: aligned deallocation function of type 'void (void *, std::align_val_t) noexcept' is only available on macOS 10.14 or newer
  cargo:warning=  ~LRUCacheShard() override = default;
  cargo:warning=  ^
  cargo:warning=rocksdb/cache/fast_lru_cache.cc:439:18: note: in defaulted destructor for 'rocksdb::fast_lru_cache::LRUCacheShard' first required here
  cargo:warning=      shards_[i].~LRUCacheShard();
  cargo:warning=                 ^
  cargo:warning=rocksdb/cache/fast_lru_cache.h:165:3: note: if you supply your own aligned allocation functions, use -faligned-allocation to silence this diagnostic
  cargo:warning=  ~LRUCacheShard() override = default;
  cargo:warning=  ^
  cargo:warning=1 error generated.
  exit status: 1

This will be because we still use the macos 10.12 (?) sdk on x86_64.

Hmmm https://github.com/NixOS/nixpkgs/pull/179399/commits/b743bab34c3ea756a2170d3717c68f78073854f8

NIX_CFLAGS_COMPILE = lib.optional stdenv.cc.isClang "-faligned-allocation";

indeed solves the issue.

WFM - could you squash the commits please?

Thanks @risicle :)


Mac desktop builds can only be run once

It looks like running OSX builds of the app causes the app to modify itself and breaking itself somehow.
Here's what I'm seeing:

build full desktop OSX app
before running the app, make a copy of the app
run the app, close it, kill the zombie child process (per this issue), and run the app a 2nd time

Expected behavior: App runs the same both times
Actual behavior: App runs normally the first time, fails to show or populate window the 2nd time
Additional info: If you run the app from the commandline, you can see that the initial host processes are dumping less info to the command line on the 2nd run than on the first run but not reporting errors

if needed, kill zombie child process
run the copy of the app

Expected behavior: Running the app shouldn't mutate the app, so the original and copied versions of the app should behave the same (eg. the copy shouldn't run if the original app no longer runs)
Actual behavior: The copy of the app runs once, then fails to run, demonstrating that running the app modifies the app

Seeing the same behavior on Linux (although no zombie process for me).

Same issue here

Fixed with the new version of Electron.NET.


TemplateMark not rendering complex data types

Context
Consider the template "Late Delivery and Penalty".
The model contains the definition:
import<EMAIL_ADDRESS>Day, TemporalUnit, Duration, PeriodUnit, Period} from<EMAIL_ADDRESS>
@template
asset LateDeliveryAndPenaltyContract extends Contract {
  ...etc...
  o Duration penaltyDuration
  ...etc...
}

The data.json reads as follows:
{
    "$class"<EMAIL_ADDRESS>     ...etc...
    "penaltyDuration": {
        "$class"<EMAIL_ADDRESS>        "amount": 2,
        "unit": "days"
    },
     ...etc...
}

The grammar is defined as:
## Late Delivery and Penalty.

.... for every {{penaltyDuration}}

Expected Behavior
As per olde Template Studio, the text used to renders as:
Late Delivery and Penalty.
----

... for every 2 days

Actual Behavior
The Template Playground renders the same model, grammar and data as:
## Late Delivery and Penalty.

... for every {"$class":"org.accordproject.time@0.3.0.Duration","amount":2,"unit":"days"}

@mttrbrts  I am here for gsoc24
I want to contribute to this project https://playground.accordproject.org/
would you be able to guide me
from where I should start
I will be highly thankful to you

@martinhalford  If possible, kindly mention the steps to reproduce. As it seems a standard procedure while writing down an issue and it helps me a lot while solving the issues.


Support rename_all = "snake_case" for command fn names

@nobane heads up just transferring this to the Specta repo as that is where the macro is defined.


#466 Add fluent assertions across all tests.

Although this touches all the test files, it's pretty uncontroversial. I did about 90% of asserts via regex, and manually fixed the ones that were split over lines.
I haven't made use of any of the more interesting assertions (e.g. assertions on lists/etc), it's all one to one mappings at this stage.
The main thing to point out, is I had to enable multidex, as the truth library caused the instrumented tests to go over the method limit. There might be a way to enable this only on the test project, but I think it's a blanket application as this review stands.

This change is

I want to hold off on this until we figure out why 811e1d6 had 290 tests and the next commit, 690654e (JUnit4), was down to only 263 tests.

While these tests pass in API 24, I'm getting 39 failures in APIs 16 and 19.

@ChrisA89 Add android:name="android.support.multidex.MultiDexApplication" to blocklytest/AndroidManifest.xml, and it fixes the problem.

I've investigated the test case discrepancy:
Before the JUnit change there were 28 instances of testAndroidTestCaseSetupProperly which if run is test classes extend ApplicationTestCase.
After there is only 1 instance as ApplicationTest still extends ApplicationTestCase, but I'm not entirely sure what that test is doing, and I think it should potentially be removed entirely.

Weird.  Yes, ApplicationTest can be removed.
Accepting this in the current state, but it would be great if we gradually replace the .isTrue() and .isFalse() tests with forms that take advantage of the fluent style.  A low priority.
Also, we have several cases of lines longer than 100 characters as a result of this and previous commits.


Added the Verified and Unverified colored cards. Thou they still need…

… some editing required.

Good work!


feat(DIA-109): add represented by section on Artist screen

This PR resolves DIA-109
Description
Adds represented by horizontal list to the Artist screen.
iOS
https://github.com/artsy/eigen/assets/15792853/cb8c422b-f543-44f2-ac2e-303e7193d193
Android
https://github.com/artsy/eigen/assets/15792853/b95dcb64-3c9b-4e3d-b78b-30e0eedff451
PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

add represented by rails to the artist screen - araujobarret

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (add represented by rails to the artist screen - araujobarret)

  Generated by :no_entry_sign: dangerJS against 1e502f472005a0b2f4948116d0d659b686baccfa


Arcane Recipe Book Crash

Crash while opening the Magick Workbench
crash-2023-10-07_14.41.15-client.txt

This should be fixed in the next release, currently planned as v4.0.7.  Thanks for the report!


Lack of documentation for validating karmada images with cosign

What documentation would you like to be added:
Documentation for validating karmada images with cosign.
Why do you think this document is needed:
Document completion for Part3 of https://github.com/karmada-io/karmada/issues/3435
Where do you think the document should be placed:
I think the document could be placed under /Documentation/administrator/verify-artifacts.md

/assign zishen

This duplicates with https://github.com/karmada-io/website/issues/379.


Hint level diagnostics tagged as deprecated only show strikeout on first few characters

As shown in the screenshots. If a hint level diagnostic is tagged as deprecated only the first few characters are struck out. If the diagnostic is at information level then the full range is struck out.
As a comparison, hint level diagnostics tagged as unnecessary are shown too which fades the full range.

Version: 1.40.2
Commit: f359dd69833dd8800b54d458f6d37ab7c78df520
Date: 2019-11-25T14:54:40.719Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Linux x64 4.19.0-6-amd64
Does this issue occur when all extensions are disabled?:
Needs an extension enabled that provides deprecated diagnostics.

Yes - that is somewhat how we have designed this. What is the use case for this?

@jrieken no particular use case other than consistency. I'd expect it to strikethrough the whole range like unnecessary fades the whole range.

Fair feedback - I believe today we use a single editor decoration and for level-hint we crop that to just a few character. With this combination of tags and sev we would need to add at least two decorations...

code pointer: https://github.com/microsoft/vscode/blob/573df438dbb97795ea6ce3c242c27f0dcd8b43dc/src/vs/editor/common/services/markerDecorationsServiceImpl.ts#L189


Added code style within tables

Fixes https://github.com/kubeflow/website/issues/730
Code formatting embedded in text within a table is hard to read - the colour is too light. This change makes the inline code style within a table the same as the style outside a table.
Before:

After:

I'll submit this change to the Docsy theme as well, so that it won't be overwritten next time we get an updated version of Docsy.

This change is

/assign @abhi-g

/lgtm
/approve


bumping js-slang

...to 1.0.12 to enable latest advances in Source Typed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 34.402%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4535

Relevant Lines:
12259

💛  - Coveralls


Removing the Beta

Hello,
How about removing the Beta and warnings from the readme?
It would help me convice teams to use type-route.
Best,

Thanks for pushing on this! I've been using Type Route actively in my own projects since first creating it and as of late haven't needed to make any big changes/fixes to the library so I'd say its pretty stable. I'll find some time beginning of next week to 1.0 it.

btw this lib is actually better than react-router on so many levels 👍

Great thanks!


nomad_plugin.yaml is missing in pip installed package

When you pip install, e.g.:
pip install git+https://github.com/nomad-coe/nomad-schema-plugin-example@main

The nomadschemaexample/nomad_plugin.yaml is not copied to the install folder (e.g. ...python3.9/site-packages/nomadschemaexample/). Without the file the plugin cannot be used.
@schumannj has this problem in her fork too: https://github.com/schumannj/nomad-schema-plugin-fhi-catalysis
For some reason it works for other projects (e.g. https://github.com/FAIRmat-NFDI/nomad-measurements) it works fine. Weirdly enough, the seemingly relevant parts of the respective pyproject.tomls look the same.

@blueraft please have a look as soon as you can. This is currently failing our example oasis deployment, because it includes @schumannj 's plugin.
If you have a solution, please also open a PR for https://github.com/schumannj/nomad-schema-plugin-fhi-catalysis.

@hampusnasstrom Maybe you know more? For your measurments and materials processing plugins the nomad_plugin.yaml gets installed properly.

I've created a PR for this in this repo. Once it gets merged, I can create a PR in fhi-catalysis repo.

Seems like Ahmed solved this? But I did have some issues with this for installing a plugin that's not at the top level of a github repository. However, adding a MANIFEST.in solved this so I usually add one now with the line:
recursive-include * nomad_plugin.yaml


#321 Fix invalid non array styleSheets item.

This will ensure that filter function is called only on array item.

Merged into https://github.com/peterpme/eslint-plugin-react-native


Update .gitmodules with force ignored

Update .gitmodules based on the following fix I found:
https://stackoverflow.com/questions/3240881/git-can-i-suppress-listing-of-modified-content-dirty-submodule-entries-in-sta
https://stackoverflow.com/questions/5126765/how-to-get-rid-of-git-submodules-untracked-status

@learner-long-life This should help untracked all the submodules from previous activities without it hanging on the source control stage like most of what we have now. The picture below shows this


Remove sanity/bounds check on predicted values

I've been in communication with @ledell about this and she's signed off on the changes.

Thanks Bill!
Some notes on the change:  This "bounds check" was used to protect against any bad (very large) predicted values being generated by the base learners.  However, for classification, when the outcome is encoded a factor, the object$ylim was c(NA, NA) so this was causing the predict.h2o.ensemble function to produce NAs.  Rather than changing the code to work for factors/classification, we have decided to remove this truncation step for now.  If it makes sense to add it back in later, we can do that.


Fixed broken link on README.md

https://jira.mesosphere.com/browse/DCOS_OSS-4489
High-level description
What features does this change enable? What bugs does this change fix?
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS- Foo the Bar so it stops Bazzing.

Related tickets (optional)
Other tickets related to this change:

DCOS_OSS- Foo the Bar so it stops Bazzing.

Checklist for all PRs

[ ] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[ ] Read the DC/OS contributing guidelines
[ ] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@adamtheturtle This PR is ready for review.

@mesosphere-mergebot label ready for review

LGTM

Thank you @anithamuthu79 - this change looks good.
Please do fill in the PR description template - that is important for reviewers.
@SarahStegall - when you approve a dcos/dcos PR, please also take a look at the linked mesosphere/dcos-enterprise PR. All DC/OS OSS PRs come with a DC/OS Enterprise PR to at least make sure that nothing gets broken given this change. The DC/OS OSS PR cannot be merged unless that is also approved.


hwameistor-local-storage  CrashLoopBackOff

# kubectl logs -n hwameistor hwameistor-local-storage-8tpsx member
unknown flag: --debug
unknown flag: --debug
Usage of /local-storage:
      --csi-address string        CSI endpoint (default "")
      --drbd-start-port int       drbd start port, end port=start-port+volume-count-1 (default 43001)
      --http-port int             HTTP port for REST server (default 80)
      --kubeconfig string         Paths to a kubeconfig. Only required if out-of-cluster.
      --max-ha-volume-count int   max HA volume count (default 1000)
      --namespace string          Namespace of the Pod (default "")
      --nodename string           Node name (default "")
      --system-mode string        dlocal system mode (default "drbd")
  -v, --v int                     number for the log level verbosity (default 4)

# kubectl logs -n hwameistor hwameistor-local-storage-8tpsx registrar
I0329 17:25:01.795535   94379 main.go:166] Version: v2.5.0
I0329 17:25:01.795621   94379 main.go:167] Running node-driver-registrar in mode=registration
I0329 17:25:01.796774   94379 main.go:191] Attempting to open a gRPC connection with: "/csi/csi.sock"
I0329 17:25:01.796827   94379 connection.go:154] Connecting to unix:///csi/csi.sock
W0329 17:25:11.797552   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:25:21.797490   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:25:31.797462   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:25:41.797211   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:25:51.797723   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:26:01.797340   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:26:11.797854   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:26:21.797675   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:26:31.797289   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:26:41.797400   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:26:51.797928   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:27:01.797231   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:27:11.797889   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:27:21.797075   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:27:31.797422   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:27:41.797067   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:27:51.797390   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:28:01.797302   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:28:11.797830   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:28:21.797633   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:28:31.797319   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:28:41.797749   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:28:51.797405   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:29:01.797486   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:29:11.797584   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:29:21.797242   94379 connection.go:173] Still connecting to unix:///csi/csi.sock
W0329 17:29:31.796966   94379 connection.go:173] Still connecting to unix:///csi/csi.sock

# kubectl get pod -n hwameistor
NAME                                                       READY   STATUS             RESTARTS        AGE
hwameistor-admission-controller-5d55bdddc-lhgjn            1/1     Running            0               8h
hwameistor-apiserver-65fbf6f77-7mr7k                       1/1     Running            0               8h
hwameistor-exporter-5846fdd6cb-824zk                       1/1     Running            0               8h
hwameistor-local-disk-csi-controller-796bfcd496-5gvdl      2/2     Running            0               8h
hwameistor-local-disk-manager-6fj98                        2/2     Running            0               8h
hwameistor-local-storage-8tpsx                             1/2     CrashLoopBackOff   99 (116s ago)   8h
hwameistor-local-storage-csi-controller-6cf756b86d-lhwcg   0/3     Pending            0               8h
hwameistor-scheduler-58cf6b5f8b-z2dwh                      1/1     Running            0               8h
hwameistor-ui-7cff94d45b-9r88p                             1/1     Running            0               8h
hwameistor-volume-evictor-765d9dc8b-pw8w9                  1/1     Running            0               8h

Arg --debug=true  has already removed in v0.9.2. Please replace it with --v=5 and try again.


feat: support legacy gemoji

Fix RM-9702

🧰 Changes
Adds support for our legacy emoji syntax.
Previously, we swapped out our custom emoji library for a more standard one. Unrealized by me, our custom one used dashes instead of underscores. This checks if the same emoji name exists using underscores instead of dashes.
🧬 QA & Testing

Broken on production.
Working in this PR app.

This PR was released!
🚀 Changes included in v6.83.0


question: about encodeURI. server.js:181 (@3c3b47e)

Todd, I'm curious why this "extra" encoding is here.
I've been tacking down an issue where inbound comma, colon etc. are turned into %2C (or appropriate) by util.getUr(), and then this is turned into %252C by the additional
var uri = encodeURI(req.prerender.url.replace(/%20/g, ' '));
i'm curious what this is all about? i've had a look at #121 .. and i removed the extra encodeURI and the utf8 characters are seen correctly by the js on my page (not when encoded twice of course).
Could you explain what drove the addition of this encode step, andy why it may be unsafe/erroneous for me to remove it.
thanks

Can you give an example of a URL that causes this? Or can you email me the URL you're testing and having problems with? I'd like to just test it and see where util.getUrl() is encoding it because I'm not able to reproduce that locally<EMAIL_ADDRESS>And what version of node are you running?

example.com/?foo=1,2,3

I'm on v0.10.33
I put this line:
console.log('encoded URI:', uri);
before req.prerender.page.open(uri...:
I tried both of these and get the same output on my console:
curl http://localhost:1338/http://example.com/?foo=1,2,3
curl http://localhost:1338/http://example.com/?foo=1%2C2%2C3
2015-03-14T00:40:24.333Z getting http://example.com/?foo=1,2,3
encoded URI: http://example.com/?foo=1,2,3
2015-03-14T00:40:24.956Z got 200 in 623ms for http://example.com/?foo=1,2,3

Are you seeing something different?

yes. i suspect you did not start with &escaped_fragment= in your url when triggering pre-render (this is the hint to my nginx config to proxy through my local prerender).
i should have made that explicit.
it when prerender encounters the escaped_fragment url/hash parameter it trims it from the query dict, and then deletes the search attr of the parsed url object (in util.getUrl()), then url.format(parts) is called, which will encode the various parts of the search parameters when reconstructing the url. Thus, the url has already been encoded in this path.
i put these in the analogous location:
            console.log('OPEN', req.prerender.url);
            console.log('open', encodeURI(req.prerender.url.replace(/%20/g, ' ')));

resulting in:
OPEN https://localhost.zumper.com/?foo=1%2C2%2C3
open https://localhost.zumper.com/?foo=1%252C2%252C3

Ah I see, thanks for that. Let me run through that again.

@thoop I've been running tests without the encodeURI and everything seems good. including urls with encodeable characters.
what was the reason for this line of code to begin with? (the /%20/g snippet smells of 'quick fixing some bug, possibly with phantom')

Sorry I haven't gotten to this yet. I still want to check this. The %20 fix was something where phantomjs 1.9.x with a %20 in the URL was (sometimes) causing angularjs to go into an infinite loop.
I'll check it soon and get back to you. I just want to make sure on our end that we don't break anything before changing that on master since we process millions of URLs.

Same problem here, the encodeURI breaks AngularJS routing (with infinite digest loop errors) when there are non ascii characters in the url (tested for accents and Korean characters). Why was the encode/decode initially necessary here?

If you remove the encode, does it work with accents and korean characters as well? Sorry I haven't gotten to this yet. Been insanely busy...

No problem :).
After further testing, removing the encodeURI fixes most extended characters (we randomly tested 2 bytes and 3 bytes utf-8 characters in several blocks), except for the most used part of the latin1 block (not all characters there strangely) and space which still doesn't work. So it's actually not a usable fix.
Changing to PhantomJS 2.0 (without encode) seems to fix all problems for us. Quickly looking at PhantomJS 2 source code it seems like they changed their handling of the url character encoding internally. However this still requires quite a bit of tweaking as they still don't have any proper build for Linux and as such most of the ecosystem is not ready yet and still based on older 1.9 versions.

We are using PhantomJS 2.0 in production. Your notes on PhantomJS 1.9.8 problems when removing encodeURI is why we were using the encodeURI with PhantomJS 1.9.8.
I just pushed a fix for this in version 3.15.2. Since Prerender defaults to PhantomJS 1.9.8, you can set export DISABLE_URI_ENCODE=true to turn off the encoding. It defaults to being left on for now.

encodeURI didn't work for latin1 characters for us either, not sure why. We also had to move to PhantomJS 2.0 in the end. Thanks for your help

Should be fixed in our latest version of Prerender (v4.0.3).


Noting configuration is in duckdb_configuration().

It is in the examples below, but not stated explicitly.

Thanks!


KinematicCharacterControllerOutput doesn't get attached automatically

Hi.
This morning I have been trying to get my player jumping mechanics done and my logic was relying on the grounded field from KinematicCharacterControllerOutput. I followed what the documentation says precisely how to add character controllers and so on but to my surprise nothing had worked as I was expecting. Debugging led me to find that my player has no KinematicCharacterControllerOutput attached to it even tho he has KinematicCharacterController which should attach the output component automatically as it says here. So I don't know if I messed things up somewhere or it's just some intern weird bug.

The applied character motion, and the information of whether the character is touching the ground at its final position, can be read with the KinematicCharacterControllerOutput component (inserted automatically to the same entity as the KinematicCharacterController component).

Here's a demo that demonstrates the issue.

I notice your rigidbody has an incompatible type for the CharacterController, please change RigidBody::Dynamic for RigidBody::KinematicPositionBased.
If you're surprised by the lack of gravity, it's explained briefly here: https://rapier.rs/docs/user_guides/bevy_plugin/character_controller/#gravity ; you can emulate it with the following snippets:
fn control(mut controlers: Query<&mut KinematicCharacterController, With<Player>>) {
    for mut cc in controlers.iter_mut() {
        cc.translation = Some(Vec2::Y * -9f32);
    }
}

app.add_systems(FixedUpdate, control);


Update to go 1.20

We probably want to release on a supported version of go

Codecov Report
Patch and project coverage have no change.

Comparison is base (293b588) 57.76% compared to head (bc91bc1) 57.76%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4021   +/-   ##
=======================================
  Coverage   57.76%   57.76%
=======================================
  Files         108      108
  Lines       10500    10500
=======================================
  Hits         6065     6065
  Misses       3762     3762
  Partials      673      673

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix size_hint for partially consumed QueryIter and QueryCombinationIter

Objective
Fix #5149
Solution
Instead of returning the total count of elements in the QueryIter in
size_hint, we return the count of remaining elements. This
Fixes #5149 even when #5148 gets merged.

https://github.com/bevyengine/bevy/issues/5149
https://github.com/bevyengine/bevy/pull/5148

Changelog

Fix partially consumed QueryIter and QueryCombinationIter having invalid size_hint

We already keep track of which items we iterated over in the QueryIterationCursor, so we might as well use it to compute how many elements there are to iterate over instead of using the full archetype list. It basically cost the same as the current method.
For QueryCombinationIter, we do K binomial coefficients rather than a single one. But I think it's fine, since for each step, we substract 1 from K and N. Also the base case of non-consumed iterator also stays the same as before.

@nicopap can you update this to reflect the changes that are about to be merged in #5148? Then we can clean up the behavior everywhere at once.

@alice-i-cecile Done now. Should be good to go.

Note that ExactSizeIterator is not that much of a performance win. collect used the minimum bound of size_hint already for pre-allocating. It's just (but still pretty good) an ergonomic win. This still doesn't enable optimizing away checks for reallocation (even if we know it will never happen) because it's safe API. TrustedLen could really be helpful here, and the way overflow is handled in the combination code is compliant with the TrustedLen API. However, TrustedLen is still unstable and we cannot use it.

https://github.com/rust-lang/rust/issues/37572

Here are the criterion results https://nicopap.ch/p/bevy_exact_size_iter_criterion_results/all_changes.html

Here are the criterion results https://nicopap.ch/p/bevy_exact_size_iter_criterion_results/all_changes.html

I don't think any of these benches shown above use the query iterator. Do you have results for the benches in this folder? https://github.com/bevyengine/bevy/tree/main/benches/benches/bevy_ecs/iteration

I don't think any of these benches shown above use the query iterator. Do you have results for the benches in this folder? https://github.com/bevyengine/bevy/tree/main/benches/benches/bevy_ecs/iteration

Wow my bad! I updated the page with the actual ECS iteration benches. The bench are a bit more consistent.
It seems I did the benches for the bevy_tasks/iter.rs previously. It's very surprising that a change that supposedly has nothing related would result in bench differences of up to 30% (in both direction). Wouldn't that make the benches unreliable? Would it not be better to delete them altogether if they cannot be trusted?


quotedPrintable.encode should accept Uint8Array as input parameter

You can use the spread operator on the Uint8Array before passing it to encode:
const array = new Uint8Array([0x00, 0xFF]);

String.fromCodePoint(...array) == '\0\xFF'
// → true

I'd like Uint8Array support as well, and Buffer please. I have multiple places in my code where I encode stuff, and thus would like to abstract all of them. So now my code carries a helper function to wrap this lib, but it would be nicer if your lib could do that encoding part for me as well.


ComboBox requires tab twice to navigate over

The issue is that all ComboBox controls require hitting tab twice to cycle over.
I think the cause is the ToggleButton is grabbing focus separately from the control itself.

Yes, that is the cause... I had to make some changes on the default templates to get around it


Allow trace_id to be set from lymph emit/request cli

lgtm

use case ?

"where did my request go?" https://github.com/deliveryhero/lymph/pull/203

it doesn't feel right, but maybe b/c I never need this, but I will not stand against it.


[Flutter Desktop] Remove unneeded settings entries

The following entries don't apply to PC and can be removed:
-Auto Open Keyboard
-Swipe Message Box to Close Keyboard
-Swipe Message Box to Open Keyboard
-Hide Keyboard when Scrolling
-Open Keyboard after tapping Scroll to Bottom

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @scarlett in #desktop-requests.

Done


How to use obs-cli in MAC OS

I have installed OBS Studio 28.0.3 in MAC OS and also installed obs-cli 2.2.3.
When trying to start recording using obs-cli command 'obs-cli StartRecording' no output is generated and recording is not started. Same command worked in Linux and Windows but unable to find suitable command to do the same in MAC OS.
Also tried 'obs-cli record start', 'obs-cli --startrecording'.
Can you please suggest how to use obs-cli commands in MAC OS.

Hi, i have the same issues...any suggestion?

It looks like obs-cli is abandoned and doesn't support the latest version of web socket communation.

obs-cli is abandoned. Use https://github.com/grigio/obs-cmd instead : )

From https://obsproject.com/forum/threads/start-stop-obs-from-a-command-line.165978/
I suppose that project needs to be updated.