v16.0.0: Cannot perform queries on Ubuntu 20.04, node 14. -- v15.17.0 works

I run wikibase-cli as a Docker container, and have created a wb alias, for a docker run command, with Docker volumes, as mentioned in the documentation.

wikibase-cli version: 15.17.0 does work, v16.0.0 does not. I have upgraded with docker pull
OS version: Ubuntu 20.04.4 LTS Linux 5.13.0-41-generic #46~20.04.1-Ubuntu SMP Wed Apr 20 13:16:21 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

This version of wikibase-cli  works properly
wb -V
15.17.0

wb label Q1
universe

This one v16.0.0. doesn't
docker image ls

maxlath/wikibase-cli  latest be511939e1fa   3 weeks ago     87.8MB

I can reset the config, and then I can run this, and the v16 command doesn't crash (yet):
wb -V
16.0.0

wb config reset

Still possible:
wb label Q1
A Wikibase instance should be specificed.

This can be done in config, ex:
  $  wb config instance https://www.wikidata.org

or with the -i,--instance command option:
  $  wb label Q1 -i https://www.wikidata.org

wb config instance https://www.wikidata.org  # no crash
However:
wb label Q1
Now it crashes:
(node:12) UnhandledPromiseRejectionWarning: ReferenceError: URLSearchParams is not defined
    at stringifyQuery (/usr/local/lib/node_modules/wikibase-cli/node_modules/wikibase-sdk/lib/utils/build_url.js:3:66)
    at queryObj (/usr/local/lib/node_modules/wikibase-cli/node_modules/wikibase-sdk/lib/utils/build_url.js:15:38)
    at /usr/local/lib/node_modules/wikibase-cli/node_modules/wikibase-sdk/lib/queries/get_entities.js:46:10
    at module.exports (/usr/local/lib/node_modules/wikibase-cli/lib/entity_data_parser.js:34:15)
    at module.exports.attribute (/usr/local/lib/node_modules/wikibase-cli/lib/entity_attribute_command.js:19:3)
    at Object.<anonymous> (/usr/local/lib/node_modules/wikibase-cli/bin/wb-label:2:43)
    at Module._compile (module.js:653:30)
    at Object.Module._extensions..js (module.js:664:10)
    at Module.load (module.js:566:32)
    at tryModuleLoad (module.js:506:12)
(node:12) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:12) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

I have no idea where to look, the URLSearchParams is not defined error message is pretty generic.

Current config.json:
{
  "instance": "https://www.wikidata.org",
  "credentials": {
    "https://www.wikidata.org": {
      "oauth": {
        "consumer_key": ".....",
        "consumer_secret": "...",
        "token": "---",
        "token_secret": ".."
      }
    }
  },
  "json": false,
  "sparql-endpoint": "https://query.wikidata.org/sparql",
  "lang": "de"
}

FYI: Annoying npm output
nvm use 14
npm does not support Node.js v14.4.0
You should probably upgrade to a newer version of node as we can't make any promises that npm will work with this version. You can find the latest version at https://nodejs.org/
Now using node v14.4.0 (npm v8.10.0)

ah yes, the Dockerfile wasn't updated after the NodeJS minimal version was moved to 10 (3f4ad7d): now patched (edcbdd7) and published as v16.0.1

Thanks for the ultrafast response! v16.0.1 works now:
wb -V
16.0.1

wb label Q1
universe

A simple docker pull (or removal of the tag number in the bash-alias) did the trick. No fiddling required.
Thaaanks!


[3.0] Make bootstrap wait async export/refer finish

What is the purpose of the change
When there is high qps system call an un-initialized reference, plenty of threads will be blocked until the initialize is done. This may un-acceptable for some user, so here we make the bootstrap wait until all async export/refer is done. Only take the advantage of multi-thread export/refer.

Codecov Report

Merging #8082 (2923421) into 3.0 (7081b5d) will increase coverage by 0.00%.
The diff coverage is 42.85%.

@@            Coverage Diff            @@
##                3.0    #8082   +/-   ##
=========================================
  Coverage     60.84%   60.84%
- Complexity      312      313    +1
=========================================
  Files          1070     1070
  Lines         44835    44829    -6
  Branches       6697     6696    -1
=========================================
- Hits          27278    27276    -2
+ Misses        14538    14536    -2
+ Partials       3019     3017    -2

Impacted Files
Coverage Δ

.../apache/dubbo/config/bootstrap/DubboBootstrap.java
69.76% <42.85%> (+0.61%)
:arrow_up:

...ubbo/rpc/protocol/dubbo/ChannelWrappedInvoker.java
43.47% <0.00%> (-4.35%)
:arrow_down:

.../apache/dubbo/remoting/transport/AbstractPeer.java
63.04% <0.00%> (-2.18%)
:arrow_down:

.../dubbo/remoting/transport/netty4/NettyChannel.java
64.35% <0.00%> (-1.99%)
:arrow_down:

...ting/zookeeper/curator/CuratorZookeeperClient.java
61.57% <0.00%> (-1.48%)
:arrow_down:

...pache/dubbo/remoting/transport/AbstractServer.java
61.42% <0.00%> (-1.43%)
:arrow_down:

...figcenter/file/FileSystemDynamicConfiguration.java
59.04% <0.00%> (-0.48%)
:arrow_down:

...he/dubbo/registry/zookeeper/ZookeeperRegistry.java
57.76% <0.00%> (+0.62%)
:arrow_up:

.../dubbo/registry/integration/RegistryDirectory.java
46.17% <0.00%> (+0.66%)
:arrow_up:

...he/dubbo/registry/multicast/MulticastRegistry.java
67.59% <0.00%> (+0.92%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


Added support to RedProtect, and some improvements

Added support to RedProtect, auto name sign if placed under region and deny self buy/rent.

Oh wow... I didn't expect anyone to actually contribute to the plugin lol.
Thanks for the contribution! I'll test it out and make sure things work as they should, and if they do I'll merge and publish a new version after pushing some minor adjustments.

This additions dont only allows redprotect support but allow any other antigrief additions, because i've added a interface to be easy to add more compatible plugin ^^
I just added my plugin, but if you this plugin to be more popular, you can add GP too.

@FabioZumbi12 Oh, that's great! Thanks man!

@FabioZumbi12 Can I have the RedProtect 2.6.0 b200 JAR file? I am unable to build it from source so I can't test stuff with it nor compile BuyRegion.

Oh shure, you can download from here (ignore the security message): https://host.areaz12server.net.br:8080/job/RedProtect/

After some testing, this seems to work great !
I added a log to show in the console what region plugin is being used, a contributors list when enabling the plugin where I moved your name to (and added that you updated it to 1.14 as well), and fixed a NullPointerException when the plugin was disabled as a result of no economy plugin or region manager being found.
Thank you for the auto name & reload command, didn't even realize you added it until writing this comment, and I even used it to reload my language file xD.
Changed the version to 2.2.0 as it's a minor update, not a patch (has features).
Thanks for the contribution, man!

TYSM 💯
https://www.spigotmc.org/resources/buyregion-1-13.60189/update?update=276640


Incorrect archive link provided for missing cudart64_100.dll

Please make sure that this is a Bug or a Feature Request and provide all applicable information asked by the template.
If your issue is an implementation question, please ask your question on StackOverflow or on the Keras Slack channel instead of opening a GitHub issue.
System information

Have I written custom code (as opposed to using example directory):  No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):  Windows 10 v1809 Build
TensorFlow backend (yes / no):  yes
TensorFlow version:  1.14.0
Keras version:  2.2.5
Python version:  3.7.4
CUDA/cuDNN version:  9
GPU model and memory:  Nvidia Geforce GTX 1050Ti - 4GB

You can obtain the TensorFlow version with:
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
You can obtain the Keras version with:
python -c 'import keras as k; print(k.version)'
Describe the current behavior
The archive link provided for cuda point to 9.0
https://developer.nvidia.com/cuda-90-download-archive
Describe the expected behavior
The download link should currently point to 10:
https://developer.nvidia.com/cuda-10.1-download-archive-update2
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
>>> import keras
Using TensorFlow backend.
Traceback (most recent call last):
  File "C:\Users\drago\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\platform\self_check.py", line 75, in preload_check
    ctypes.WinDLL(build_info.cudart_dll_name)
  File "C:\Users\drago\AppData\Local\Programs\Python\Python37\lib\ctypes\__init__.py", line 364, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: [WinError 126] The specified module could not be found

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\drago\AppData\Local\Programs\Python\Python37\lib\site-packages\keras\__init__.py", line 3, in <module>
    from . import utils
  File "C:\Users\drago\AppData\Local\Programs\Python\Python37\lib\site-packages\keras\utils\__init__.py", line 6, in <module>
    from . import conv_utils
  File "C:\Users\drago\AppData\Local\Programs\Python\Python37\lib\site-packages\keras\utils\conv_utils.py", line 9, in <module>
    from .. import backend as K
  File "C:\Users\drago\AppData\Local\Programs\Python\Python37\lib\site-packages\keras\backend\__init__.py", line 1, in <module>
    from .load_backend import epsilon
  File "C:\Users\drago\AppData\Local\Programs\Python\Python37\lib\site-packages\keras\backend\load_backend.py", line 89, in <module>
    from .tensorflow_backend import *
  File "C:\Users\drago\AppData\Local\Programs\Python\Python37\lib\site-packages\keras\backend\tensorflow_backend.py", line 5, in <module>
    import tensorflow as tf
  File "C:\Users\drago\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\__init__.py", line 28, in <module>
    from tensorflow.python import pywrap_tensorflow  # pylint: disable=unused-import
  File "C:\Users\drago\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\__init__.py", line 49, in <module>
    from tensorflow.python import pywrap_tensorflow
  File "C:\Users\drago\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 30, in <module>
    self_check.preload_check()
  File "C:\Users\drago\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\platform\self_check.py", line 82, in preload_check
    % (build_info.cudart_dll_name, build_info.cuda_version_number))
ImportError: Could not find 'cudart64_100.dll'. TensorFlow requires that this DLL be installed in a directory that is named in your %PATH% environment variable. Download and install CUDA 10.0 from this URL: https://developer.nvidia.com/cuda-90-download-archive

Created PR for it @https://github.com/tensorflow/tensorflow/pull/36489

Closing this as the PR was closed since the version is unsupported now.


[WIP] Use kqueue when running on osx-x86_64

Only 1 test is remained.
com.linecorp.armeria.it.server.GracefulShutdownIntegrationTest > interruptsSlowRequests FAILED
    org.junit.runners.model.TestTimedOutException: test timed out after 20000 milliseconds
        at sun.misc.Unsafe.park(Native Method)
        at java.util.concurrent.locks.LockSupport.park(LockSupport.java:175)
        at java.util.concurrent.locks.AbstractQueuedSynchronizer.parkAndCheckInterrupt(AbstractQueuedSynchronizer.java:836)
        at java.util.concurrent.locks.AbstractQueuedSynchronizer.doAcquireSharedInterruptibly(AbstractQueuedSynchronizer.java:997)
        at java.util.concurrent.locks.AbstractQueuedSynchronizer.acquireSharedInterruptibly(AbstractQueuedSynchronizer.java:1304)
        at java.util.concurrent.CountDownLatch.await(CountDownLatch.java:231)
        at com.linecorp.armeria.it.server.GracefulShutdownIntegrationTest.interruptsSlowRequests(GracefulShutdownIntegrationTest.java:152)

I'm curious kqueue transport is stable enough because it's first time it was released. If the test does not pass completely and it is because of a bug in the kqueue transport, then we may want to disable it by default, although we should keep this code.

@imasahiro Gentle ping ;-)

As you guys said, nobody uses mac servers at production, we cannot achive any merits at this time. So let me close this PR.
Transport type related refacotring will be taken over by https://github.com/line/armeria/pull/600

Thanks, @imasahiro !


[SPARK-41733][SQL][SS] Apply tree-pattern based pruning for the rule ResolveWindowTime

What changes were proposed in this pull request?
This PR proposes to apply tree-pattern based pruning for the rule ResolveWindowTime, to minimize the evaluation of rule with WindowTime node.
Why are the changes needed?
The rule ResolveWindowTime is unnecessarily evaluated multiple times without proper pruning.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing tests. New test case is added to cover SQL usage for window_time.

cc. @cloud-fan @rxin

cc. @viirya as well

The change is only to rebase the fix due to the conflict from #39245. I'm going to merge once the CI passes.

https://github.com/HeartSaVioR/spark/runs/10328690654
Looks like GA fails to pull the test result. Build succeeded.

Thanks! Merging to master.


slidesPerGroup: auto

Is there a way to implement an 'auto' option for the 'slidesPerGroup' option? I'm using this Swiper to create a newspaper view and we use the following structure:

Slide 1 (width 100%)
Slide 2 (width 50%)
Slide 3 (width 50%)
Slide 4 (width 50%)
Slide 5 (width 50%)
Slide 6 (width 100%)

So infact there are 4 views (slide 1, slide 2/3, slide 4/5, slide 6).
When I click the 'next' button from slide 1 I go to slide 2/3. So far so good. When I click 'next' again I go to slide 3/4... this is wrong as I want to go to slide 4/5.
Is this possible?
Set up:
var mySwiper = new Swiper ('.swiper-container',
{
  nextButton: '.swiper-button-next',
  prevButton: '.swiper-button-prev',
  slidesPerView: 'auto'
});

Custom css:
.swiper-container
{
    position: absolute;
    top: 0;
    left: 0;
    right: 0;
    bottom: 0;
}

.swiper-slide
{
    width: 50%;
}

.swiper-slide:nth-child(even)
{
    text-align: right;
}

.swiper-slide:nth-child(odd)
{
    text-align: left;
}

.swiper-slide:first-child,
.swiper-slide:last-child
{
    width: 100%;
    text-align: center;
}

.swiper-slide img
{
    max-width: 100%;
    max-height: 100%;
}

No, sorry, slidesPerGroup can't be like auto or dynamic. But you can just put the content of slides 2/3, 4/5 into single slides, i guess you will got the same behavior

That's a pity.
Yeah that would be possible. But when a user switches to portrait mode I will give the .swiper-slide a width of 100% so the user will see one page per slide.
That's why an auto behauvior for the slidesPerGroup would be great.
For now I will hack around with some JS I think :)

Oh man. I am running into a similar issue where slidesPerGroup: 'auto' would be fantastic as each slide I have is a dynamic width. Being able to slide by slides that will fit in the view port at any given time would be great. A static number can't do this.

👍

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Grab prop (Suggestion)

and a question whether there will be an update that will allow, for example, to raise the prop (I saw something similar in wasabi_boombox), thank you in advance

You mean like grab the prop?

I could work on that even it seems a little strange because of the model i use

yes, that's why I changed it to a regular boombox but if I understand it, there's no need to add it

You just made me thing in something i could add thanks :)

okk

okk


hello,about RealSense and distance

the number meters away  is the groundtruth and how to define it？
RealSense compute the number?
If I want to get the real position of the object，What do I need to do?
thanks！

@mk123qwe

RealSense compute the number?

Yes.
D435 measures the distance with infrared rays, so I think that it is ground truth.
It's measure the distance of the center coordinates of the detected object.
visualization_utils.py
meters = depth_frame.as_depth_frame().get_distance(int_xmin+int((int_xmax-int_xmin)/2), int_ymin+int((int_ymax-int_ymin)/2))

I think that you can understand by seeing the API specification, but there is no explanation.
http://intelrealsense.github.io/librealsense/doxygen/classrs2_1_1depth__frame.html#a5090d69d04ade6dd67175cae9b6bee9c

Thank you again for your detailed answers.
I want to do a project on classification and pick up.
so I appreciate your research very much.
I want to know the cost of hardware compare with TX2 .
The most important problem is that I need to reduces cost.

@mk123qwe
I have no responsibility.
This is just an example.
Japanese yen.
RaspberryPi3 Model B:￥3,000
RealSense D435:￥28,000
HDMI Display 7inch:￥5,000
USB Camera 8 MegaPixel:￥7,000
Total:￥43,000
+α Operation cost

test this:Tutorial to set up TensorFlow Object Detection API on the Raspberry Pi
https://github.com/PINTO0309/TensorFlow-Object-Detection-on-the-Raspberry-Pi
I ran the program successfully, showing the video, but I did not recognize the result and bbox.
and test simple code:
import tensorflow as tf
happy= tf.constant('Happy new year!')
sess = tf.Session()
sess.run(happy)
tensorflow works well

Topics unrelated to this repository will be refused.
Please report the problem to the original fork repository.


Days in monthView are not translated after global moment.locale() change

#103   this issue still exist in one way.
I recocgnized that moment-picker is already changing its language after setting the moment.locale()
but in the monthView that's not the case.
this.headers = moment.weekdays().map((d: string, i: number) => moment().locale(this.$scope.locale).startOf('week').add(i, 'day').format('dd'));
all the other times where you format momentjs your not creating an new moment instance with its own locale
probably changing the language over the global moment.locale() was not an intended feature :D

I have the same problem

Any known schedule when this will be fixed in future version?


Page to show all pledged collectives

It'd be great to have all pledged collectives in one page, to point people there.
We already have a first one and we've found out via twitter.

Edit by @Betree: see detailed specs below

@Mayekita did a page for this but I was thinking that it might be a better idea to show them (and filter them) at the discover page itself.
I asked Maye to do and post a screen on this in addition to what she already did.

Yes, I made a page for this but after we discuss that @cuiki and I agreed on add a filter to the Discover page, so it will be something like this:

Also it matches the issue #1560
Here's the flow you can see how it works:
https://www.figma.com/proto/D9ToLTukHKrcoHq3vNKB9Mea/05.-Discover?node-id=1%3A2&viewport=102%2C310%2C0.788686&scaling=min-zoom

looks good!

To sum up what's need to be done to complete this issue and implement the design:

[ ] This feature should be implemented in the /discover page.
[ ] We need a new PledgedCollectiveCard component to display pledged collectives
[ ] Add a search bar on the page
[ ] Revamp the filters according to design

[ ] Move sortBy, use a StyledSelect instead of a regular select
[ ] Remove the "Show" filter select and instead use links (All collectives, Open source collectives...)

:gift: :money_with_wings: A $100 bounty was attached to this issue. Anyone submitting a Pull Request will be rewarded with $100 when the Pull Request is reviewed, accepted and merged.
:arrow_right: More info

Am interested in working on this issue

Hi @fakela, welcome! First come first served, this issue is yours :slightly_smiling_face:
If you have any question please ask directly on this issue or on Slack.

@fakela I've invited you to the contributors/developers team. You should receive an email to validate this shortly. Once accepted, you'll be able to push code directly on this repository.
Please note that the master branch is protected so you will have to create a new branch to work on.

@fakela As agreed on Slack, we need to move forward with this one and for that we must has a timeframe. If not completed in two weeks from now (2019/04/29) the issue will re-assigned to another contributor or to someone from the team.

@Betree we've created another separate issue to track the last pending sub-task of this. You can find it here: #1929

@cuiki Great! https://github.com/opencollective/opencollective/issues/1929 is out of the bounty scope but ideally we should ship it at the same time.

Cool, that issue is being delivered today.

@Betree Is this still open?

@eloyekunle It is, let's just merge https://github.com/opencollective/opencollective-frontend/pull/1822 before, it will make the changes for it much easier.

opencollective/opencollective-frontend#1822 is now merged, so this issue is up for grabs!

is now merged, so this issue is up for grabs!

@Betree I'd like to work on this issue.

@leonardohof all yours! Ping me if you need anything :)

:trophy: This issue is now completed.
@leonardohof To claim the bounty, you can either:

Submit an invoice to https://opencollective.com/engineering/expenses/new (please mention the issue number in the expense's title)
Or ask for an Open Collective gift card of the same value (to donate to other collectives)

For more info about the bounty process and how to format your invoice, please read https://docs.opencollective.com/help/developers/bounties
If you have any feedback on how we could improve the bounty process or how to better onboard new developers please share it with us, we're always eager to improve :slightly_smiling_face:


Number of Rows not changing in screen saver settings

I want to increase the number of rows shown on the screensaver from the default (4), to 5 due to the image resolution being a little grainy and poor, but it is not letting me. In the settings it shows 5 and previews it but when you actually preview it still shows the 4 rows. is there any fix?

I think the issue is related to #50. You need to killall LegacyScreenSaver processes to update the rows.

I think the issue is related to #50. You need to killall LegacyScreenSaver processes to update the rows.

Did not work at all


general.useragent.locale doesn't work

https://developer.mozilla.org/en-US/Add-ons/WebExtensions/Internationalization#Testing_out_your_extension
D:\Dev\image-picka>npm start -- --pref general.useragent.locale=zh-TW

><EMAIL_ADDRESS>start D:\Dev\image-picka
> web-ext run "--pref" "general.useragent.locale=zh-TW"

Running web extension from D:\Dev\image-picka
Setting custom Firefox preferences: {
  "general.useragent.locale": "zh-TW"
}
Use --verbose or open Tools > Web Developer > Browser Console to see logging
Installed D:\Dev\image-picka as a temporary add-on
The extension will reload if any source file changes
Press R to reload (and Ctrl-C to quit)

Only the message used in manifest is changed.
Now I install portable Firefox in order to test different locales.

This looks work for me.
optionCustomIconLabel and optionCustomIconColorLabel seems to not applied.

They are added after #32 is merged.
Here you can find them on Transifex:
https://www.transifex.com/image-picka/image-picka/translate/#zh_CN/messagesjson/129172120?translated=no

This seems to be a timing conflict, sometimes not working.
I checked it on a local git repo, but it does not seem to work.

It is ready for use now:

I'm going to release 0.6.0.

This option had been changed to intl.locale.requested but still, i18n API doesn't respect it.

So this option only works if:

Use Firefox stable.
Install required language packs.


2 extra attributes for pakring and garages

We need new attributes to represent all possibilities for carparks and garages for Wüest and IAZI in the mortgage API. Currently, theses 3 options are available:

numberOfUndergroundCarParks
numberOfSeparateGarages --> for garages which are detached from the house but are still located on the same property and which are not included in the building volume of the house (they have a separate building insurance policy)
numberOfExternalParking --> for parking spaces located on an external property

we would also need these 2 new ones:

numberOfAttachedGarages --> for garages which are attached to the house and which are included in the building volume of the house (they do not have a separate building insurance policy)
numberOfExternalSeparateGarages --> for garages located on an external property and which are not included in the building volume of the house (they have a separate building insurance policy)

Celine will create a pull request with the 2 new types


New Version: Typst.Typst version 0.6.0

Finishing Updating the Typst.Typst application.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/111477&drop=dogfoodAlpha

/azp run

@wingetbot waivers Add Validation-Executable-Error

Don't know why the pipelines don't want to work.

@stephengillie Why the Azure Pipelines quotes like that? It also my pull request did not have label.

No commit pushedDate could be found for PR 111477 in repo microsoft/winget-pkgs

Attempting for another retry.


@jordattebayo/theming updates

The income and expense pop out charts were dependent on the inherited body styling. The body styling did not have a fill property and only used the color property. As a result some of the SVG elements were using the default fill properties which caused them to not be visible against dark backgrounds. Adding the fill property in the body styling enabled the SVG elements to inherit the var(--foreground-color).

Looks good!


Fix usage example in README

The explicit fetchall() call is needed since
https://github.com/prestosql/presto/commit/568449b8d058ed8281cc5277bb53902fd044cad7.
Of course, it was strongly recommended even before that, as otherwise
invoking application did not know whether query succeeded, failed or was
considered abandoned.
Relates to: https://github.com/prestosql/presto-python-client/issues/56

CI - https://github.com/prestosql/presto-python-client/pull/58


Support data forwarding for tunnels

As of now, we have supported TCP port forwarding, but the data being forwarded is transported over the unencrypted raw TCP tunnel, sometimes we need a secure tunnel.
The data forwarding tunnel can forward data over any tunnel, such as TLS (like stunnel), KCP, QUIC, SSH, etc. For example:
server side, a TLS forwarding tunnel that forwards data to port 8080, and an SNI proxy:
gost -L tls://:443/:8080 -L sni://:8080
client side:
gost -L :8888 -F forward+tls://server_ip:443
Then all the HTTP(S) requests from client port 8888 will be forwarded to server port 8080 (SNI proxy) and the data is encrypted between client and server.

is there a way to choose the destination from the client?


'module' object has no attribute 'NullTranslations'

Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if any
[x] I have reproduced the issue with at least the latest released version of superset
[x] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
master
Expected results
Actual results
when i run : superset runserver
Traceback (most recent call last):
  File "/usr/local/bin/superset", line 12, in <module>
    from superset.cli import manager
  File "/usr/lib/python2.7/site-packages/superset-0.18.5-py2.7.egg/superset/__init__.py", line 131, in <module>
    security_manager_class=app.config.get("CUSTOM_SECURITY_MANAGER"))
  File "/usr/lib/python2.7/site-packages/Flask_AppBuilder-1.9.1-py2.7.egg/flask_appbuilder/base.py", line 131, in __init__
    self.init_app(app, session)
  File "/usr/lib/python2.7/site-packages/Flask_AppBuilder-1.9.1-py2.7.egg/flask_appbuilder/base.py", line 157, in init_app
    self._add_admin_views()
  File "/usr/lib/python2.7/site-packages/Flask_AppBuilder-1.9.1-py2.7.egg/flask_appbuilder/base.py", line 254, in _add_admin_views
    self.sm.register_views()
  File "/usr/lib/python2.7/site-packages/Flask_AppBuilder-1.9.1-py2.7.egg/flask_appbuilder/security/sqla/manager.py", line 67, in register_views
    super(SecurityManager, self).register_views()
  File "/usr/lib/python2.7/site-packages/Flask_AppBuilder-1.9.1-py2.7.egg/flask_appbuilder/security/manager.py", line 443, in register_views
    category_label=_('Security'))
  File "/usr/lib/python2.7/site-packages/Flask_AppBuilder-1.9.1-py2.7.egg/flask_appbuilder/base.py", line 345, in add_view
    baseview = self._check_and_init(baseview)
  File "/usr/lib/python2.7/site-packages/Flask_AppBuilder-1.9.1-py2.7.egg/flask_appbuilder/base.py", line 297, in _check_and_init
    baseview = baseview()
  File "/usr/lib/python2.7/site-packages/Flask_AppBuilder-1.9.1-py2.7.egg/flask_appbuilder/views.py", line 451, in __init__
    super(ModelView, self).__init__(**kwargs)
  File "/usr/lib/python2.7/site-packages/Flask_AppBuilder-1.9.1-py2.7.egg/flask_appbuilder/baseviews.py", line 628, in __init__
    super(BaseCRUDView, self).__init__(**kwargs)
  File "/usr/lib/python2.7/site-packages/Flask_AppBuilder-1.9.1-py2.7.egg/flask_appbuilder/baseviews.py", line 395, in __init__
    self._init_properties()
  File "/usr/lib/python2.7/site-packages/Flask_AppBuilder-1.9.1-py2.7.egg/flask_appbuilder/baseviews.py", line 679, in _init_properties
    super(BaseCRUDView, self)._init_properties()
  File "/usr/lib/python2.7/site-packages/Flask_AppBuilder-1.9.1-py2.7.egg/flask_appbuilder/baseviews.py", line 423, in _init_properties
    self._gen_labels_columns(list_cols)
  File "/usr/lib/python2.7/site-packages/Flask_AppBuilder-1.9.1-py2.7.egg/flask_appbuilder/baseviews.py", line 405, in _gen_labels_columns
    if not self.label_columns.get(col):
  File "/usr/lib/python2.7/site-packages/Flask_Babel-0.11.2-py2.7.egg/flask_babel/speaklater.py", line 26, in __len__
    return len(text_type(self))
  File "/usr/lib/python2.7/site-packages/Flask_Babel-0.11.2-py2.7.egg/flask_babel/speaklater.py", line 23, in __str__
    return text_type(self._func(*self._args, **self._kwargs))
  File "/usr/lib/python2.7/site-packages/Flask_Babel-0.11.2-py2.7.egg/flask_babel/__init__.py", line 552, in gettext
    t = get_translations()
  File "/usr/lib/python2.7/site-packages/Flask_Babel-0.11.2-py2.7.egg/flask_babel/__init__.py", line 217, in get_translations
    return support.NullTranslations()
AttributeError: 'module' object has no attribute 'NullTranslations'

Steps to reproduce

Looks like something is wrong between flask-appbuilder and flask-babel. Flask-Babel 0.11.1 appears to work fine here.

@xrmx pip install Flask-Babel==0.11.1 it works well, thx

I pushed a PR to bump flask-babel on FAB

@mistercrunch the fix would be ==0.11.1 or >=0.10.0<0.11.2 . Or we can understand what the issue is :)

I experienced the same issue after migrating from RHEL to Oracle Linux.

@jlkokko can you try pip install flask-babel==0.11.1

Same error.  Again this was on a migration from RHEL to Oracle. On a clean Oracle system that did not have python, I was able to successfully install and run without issue.

What i did to fix on Centos, I added the following lines in /usr/lib/python2.7/site-packages/flask_babel/init.py:
        try:             return support.NullTranslations()         except:             import gettext             return gettext.NullTranslations()

Notice: this issue has been closed because it has been inactive for 192 days. Feel free to comment and request for this issue to be reopened.


Exception caught by widgets library:"Class 'String' has no instance method 'where'"

Version:
custom_radio_grouped_button: ^2.1.1
Flutter (Channel stable, 2.5.3, on Microsoft Windows [Version 10.0.19042.1288], locale zh-CN)
my code:
CustomCheckBoxGroup(
  buttonTextStyle: ButtonTextStyle(
    selectedColor: Colors.red,
    unSelectedColor: Colors.orange,
    textStyle: TextStyle(
      fontSize: 16,
    ),
  ),
  unSelectedColor: Theme.of(context).canvasColor,
  buttonLables: [
    "M",
    "T",
    "W",
    "T",
    "F",
    "S",
    "S",
  ],
  buttonValuesList: [
    "Monday",
    "Tuesday",
    "Wednesday",
    "Thursday",
    "Friday",
    "Saturday",
    "Sunday",
  ],
  checkBoxButtonValues: (values) {
    print(values);
  },
  spacing: 0,
  defaultSelected: "Monday",
  horizontal: false,
  enableButtonWrap: false,
  width: 40,
  absoluteZeroSpacing: false,
  selectedColor: Theme.of(context).accentColor,
  padding: 10,
);

Exception:
======== Exception caught by widgets library =======================================================
The following NoSuchMethodError was thrown building _BodyBuilder:
Class 'String' has no instance method 'where'.
Receiver: "Monday"
Tried calling: where(Closure: (dynamic) => bool)

The relevant error-causing widget was:
  Scaffold Scaffold:file:///E:/FlutterDev/YLAib/pages/device_pagee_time_switch/add_single_timeriew.dart:16:12
When the exception was thrown, this was the stack:
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:63:5)
#1      _CustomCheckBoxGroupState.initState (package:custom_radio_grouped_button/CustomButtons/CustomCheckBoxGroup.dart:133:29)

Since it's a group button you need to pass List of defaultSelected so you should pass ["Monday"] instead of"Monday"

@ketanchoyal thank you, According to your solution, the problem is solved. The code above is to directly use the code in your readme. It is recommended to modify the code in the readme.


A better way to compatible Agent-protocol

🚀 Feature Request

Agent-protocol is a universal agent communication protocol. We can implement agent-protocol to build a complex Agent.
I hope all of our Agent can implement Agent-protocol

ref: #530
This PR implement it


enum DbType

Let's bump version in pom.xml to 2.0.0-SNAPSHOT because we've changed the API incompatibly... and also change JavaField.getDbType() to return DbType enum because we've changed the API :-)

The tests fail:

The tests fail:
[ERROR] Failures:
[ERROR]   ColumnTest.annotatedFieldDbTypeTest:60
Expecting ArrayList:
  ["UINT32"]
to contain only:
  ["UInt32"]
element(s) not found:
  ["UInt32"]
and element(s) not expected:
  ["UINT32"]

done

Now that we return DbType.DEFAULT instead of "", one fragile test has failed:
[ERROR] tech.ydb.yoj.databind.schema.naming.ColumnTest.plainFieldTypeTest -- Time elapsed: 0.015 s <<< FAILURE!
java.lang.AssertionError:

Expecting empty but was: [DEFAULT, DEFAULT, DEFAULT]
	at tech.ydb.yoj.databind.schema.naming.ColumnTest.plainFieldTypeTest(ColumnTest.java:52)


form-data content-type support?

Hello,
I've recently upgraded to a newer version of this package (0.46.2) and am now running into issues with the server failing to process requests with FormData content-types.
The requests fail prior to reaching our registered handler for the given endpoint with the following error. It seems like there may be a bug with how form-data requests are decoded/parsed prior to being passed on to the endpoint's registered handler:
{
  "_tag": "RequestError",
  "request": {
    "_tag": "ServerRequest",
    "method": "POST",
    "url": "/coi_request",
    "originalUrl": "/coi_request",
    "headers": {
      "authorization": "--redacted--",
      "user-agent": "PostmanRuntime/7.36.0",
      "accept": "*/*",
      "cache-control": "no-cache",
      "postman-token": "0b7ca553-40ef-48aa-94b1-866d638875ec",
      "host": "localhost:8080",
      "accept-encoding": "gzip, deflate, br",
      "connection": "keep-alive",
      "content-type": "multipart/form-data; boundary=--------------------------707399609773474252081923",
      "content-length": "318"
    }
  },
  "reason": "Decode",
  "error": {}
}

Are there any known limitations to using FormData content-types/are you aware of any breaking changes to FormData content-type handling since the big re-write post v0.32.0?
Likewise, is there any way I can register a handler that bypasses the default request payload decoding behavior and handle type validations manually within the endpoint's handler?
Thanks for any help you can provide.

Hey, there was a big rewrite some time ago where the internals started relying on the @effect/platform and the form-data was handling was affected as well. Please refer to the form-data example.
Please be aware that over all the file handling is not handled super well yet. Notably, the OpenAPI spec is inprecise. There is tons of room for improvement and expect some changes in the future. 🙂

Thanks, this helped us fix the problem


(Default)Dict for some attributes

There was some talk about a newer interface taking advantage of the utils.jl/getattr() method.
https://github.com/MakieOrg/GraphMakie.jl/pull/67#discussion_r818534834
It happened that I needed it for some arguments and I developed this PR for a possible implementation.
I implemented it just for nlabels_textsize and node_size.
If you agree with the implementation it's rather straight forward to expand it for all other arguments.
DefaultDict
I didn't really like the idea of bringing in a dependency for DataStructures.jl, but as much I thought about it, I couldn't find a way to leverage multiple dispatch to handle it transparently.
As a result I added one more check to the the getattr() method.
I think overall it is a nice feature though, to have an DefaultDict as an input.
Example
julia> using Graphs, DataStructures
julia> gc = circular_ladder_graph(5)
julia> graphplot(gc; nlabels=repr.(vertices(gc)), nlabels_textsize=Dict(9=>0, 10=>0), node_size=Dict(9=>0, 10=>0))
julia> graphplot(gc; nlabels=repr.(vertices(gc)), nlabels_textsize=DefaultDict(30,Dict(9=>0, 10=>0)), node_size=DefaultDict(5,Dict(9=>0, 10=>0)))

Generating the following plots:

Conclusions
This PR is not meant to be merged without any discussion.
Instead, I wanted to suggest a way of implementing the new interface and talk it over.
Personally I believe that if we want to do exactly this for all attributes, the code could get messy.
So I think in the future it would be good to do all these new lifts inside a function or a macro.

Codecov Report
Base: 80.03% // Head: 79.92% // Decreases project coverage by -0.10% :warning:

Coverage data is based on head (75d6ed8) compared to base (1833710).
Patch coverage: 85.71% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
- Coverage   80.03%   79.92%   -0.11%
==========================================
  Files           4        4
  Lines         556      563       +7
==========================================
+ Hits          445      450       +5
- Misses        111      113       +2

Impacted Files
Coverage Δ

src/utils.jl
96.96% <50.00%> (-1.47%)
:arrow_down:

src/recipes.jl
97.40% <100.00%> (-0.39%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Overall I am in favour of doing this. get_attr was added much later when i needed to handle edge parameters very differently depending on edgeplot using linesegmens or beziersegments. I never took the the time to properly expand the usage to all the arguments. Maybe it would be worth looking how it is done in Makie itself, the problem of distributing attributes which may be given in various formats to their individual recipients must be handled there all the time.
Other things to consider:

Makie can handle observables of single values. I think it would be bad to @lift nodesize=5 to an vector. This might be a problem bad for very large networks.
The code should be extended to relift on size changes, but thats easily doable.

Just brainstroming here, but maybe we should do something like
prep_arguments(x, size, default) = x
function prep_arguments(x::Observable{AbstractDict}, size, default)
    @lift [collect based on $size and $x and $default]
end
function prep_arguments(x::Observable{DefaultDict}, size, _)
    @lift [collect based on $size and $x]
end

so that in the recipe the code becomes something like markersize=prep_arguments(gp.nodesize, size, default).
But thats not good if the arguments come in as Observable{Any}...

Sorry i totally forgot about that PR. And also I am an Idiot: i didn't like how it adds another dep, however, we already implicitly depend on DataStructures (via Graphs and others) so this really is a non-issue.
I am not a huge fan of the implementation of DefaultDict because it does not behave like I'd expect for get. But it is available and used so better to build on that then to cook up something new. So I'd say if you want to add this for more arguments go for it. The very least would be to document its usage for the parts where it is already implemented in that PR and add some tests.

you made some valuable comments above and I didn't have time to process them in detail. Especially looking into the Makie package to see how they are doing it. So if you have more concrete examples of how Makie is doing it please let me know.

So  I'd say if you want to add this for more arguments go for it.

I would prefer that we first decide on the structure of the code, merge this PR as a prototype, and then we can slowly start working on gradually doing the same for all attributes as a non-brainer.
I am already using this feature  locally, so sooner or later I will focus on finalizing & merging this PR..

for a demo you can try the following:
julia> using GraphMakie, GLMakie, Graphs;

julia> gc = circular_ladder_graph(5);

julia> ons = Observable(30);

julia> onf = Observable(30);

julia> graphplot(gc; nlabels=repr.(vertices(gc)), node_size=ons, nlabels_fontsize=onf)

julia> ons[] = 10; # check changes

julia> onf[] = 10; # check changes

julia> ons = Observable(Dict(1=>30, 10=>30));

julia> onf = Observable(Dict(1=>30, 10=>30));

julia> graphplot(gc; nlabels=repr.(vertices(gc)), node_size=ons, nlabels_fontsize=onf)

julia> onf[] = Dict(7=>30); # check changes

julia> ons[] = Dict(7=>30); # check changes

julia> ons = Observable(DefaultDict(70, 1=>30, 10=>30));

julia> onf = Observable(DefaultDict(70, 1=>30, 10=>30));

julia> graphplot(gc; nlabels=repr.(vertices(gc)), node_size=ons, nlabels_fontsize=onf)

julia> ons[] = DefaultDict(20, 10=>70); # check changes

julia> onf[] = DefaultDict(20, 10=>70); # check changes

Other things to consider:

[x] Makie can handle observables of single values. I think it would be bad to @lift nodesize=5 to an vector. This might be a problem bad for very large networks.
[x] The code should be extended to relift on size changes, but thats easily doable.

@hexaeder please let me know what do you think. As already mentioned this PR is a prototype and the other arguments should follow rather easily. The documentation and tests you mention should follow after we reach an agreement :)

Actually I also thought about a small change that in my opinion will lead to cleaner code (non-disruptive):
Change function getattr(o::Observable, idx, default=nothing) to
getattr(o::Observable, idx, default=nothing) = getattr(o[], idx, default

Add
function getattr(x, idx, default=nothing)
    if x isa AbstractVector && !isa(x, Point)
        return x[idx]
    elseif x isa DefaultDict || x isa DefaultOrderedDict
        return getindex(x, idx)
    elseif x isa AbstractDict
        return get(x, idx, default)
    else
        return x === nothing ? default : x
    end
end

So then the loop can be
@lifft begin
...
[getattr($o, i, $default) for i in 1:$size]
...
end

intead of
@lift begin
...
[getattr(o, i, $default) for i in 1:$size]
...
end

which I always find a bit awkward to handle observables as observables and not values inside a @lift.
What do you think ?

Sure, you may redefine getattr that way. While you're at it: you can use the same issingleargument function ther since it means the same thing.
Have you looked into applying those things to the edge attributes yet? There it is a bit more complicated since edgeplot either calls linesegmens (which needs single attribute or vetor so we need to prep_arguments) or bezierpathes (which makes use of getattr anyway and can handle dicts and defaultdicts). So the prep_arguments shoult probably be applied inside the linesegments path, but i havn't really thought this through.

Also, since getattr, prep_arguments and issingleattribute are all part of the same processing i think they should live next to each other in the codebase. I don't care whether thats recipe or utils. Also, mache prep_arguments should be called prep_attributes or something?

I see. No I still haven't looked into edges.

prep_arguments should be called prep_attributes or something?

yes, sound better.

Since we started this work, I think it's better we might as well finish off all attributes in this PR.
This way also docs/tests can be made more complete.
Following a list of the attributes and which ones can support the new feature:

[ ]         layout = Spring(),

node attributes (Scatter)

[x]         node_color = scatter_theme.color,
[x]         node_size = scatter_theme.markersize,
[x]         node_marker = scatter_theme.marker,
[ ]         node_attr = (;),

edge attributes (LineSegements)

[x]         edge_color = lineseg_theme.color,
[x]         edge_width = lineseg_theme.linewidth,
[ ]         edge_attr = (;),

arrow attributes (Scatter)

[ ]         arrow_show = automatic,
[ ]         arrow_marker = '➤',
[ ]         arrow_size = scatter_theme.markersize,
[ ]         arrow_shift = 0.5,
[ ]         arrow_attr = (;),

node label attributes (Text)

[ ]         nlabels = nothing,
[x]         nlabels_align = (:left, :bottom),
[x]         nlabels_distance = 0.0,
[x]         nlabels_color = labels_theme.color,
[x]         nlabels_offset = nothing,
[x]         nlabels_fontsize = labels_theme.fontsize,
[ ]         nlabels_attr = (;),

edge label attributes (Text)

[ ]         elabels = nothing,
[x]         elabels_align = (:center, :center),
[x]         elabels_side = :left,
[x]         elabels_distance = automatic,
[x]         elabels_shift = 0.5,
[x]         elabels_rotation = automatic,
[x]         elabels_offset = nothing,
[x]         elabels_color = labels_theme.color,
[x]         elabels_fontsize = labels_theme.fontsize,
[ ]         elabels_attr = (;),

self edge attributes (already use getattr so no need ? )

[ ]         edge_plottype = automatic,
[x]         selfedge_size = automatic,
[x]         selfedge_direction = automatic,
[x]         selfedge_width = automatic,
[ ]         curve_distance = 0.1,
[ ]         curve_distance_usage = automatic,
[ ]         tangents=nothing,
[ ]         tfactor=0.6,
[x]         waypoints=nothing,
[x]         waypoint_radius=nothing,

I start to be happy with where this goes and I think we are close to merging.
Have a look at my remarks here where I make a summary and I highlight the new additions.
I took some liberty on some things so feel free to be critical about them.
Summary
Mainly, this PR extends functionality by supporting arguments of Dict.
However there are some other ideas integrated.
Bug Fixes
Testing my previous workflow https://github.com/MakieOrg/GraphMakie.jl/pull/89#issuecomment-1450975745
there were some errors. More specifically updating the Observable properties errored. I fixed the bug in the following lines

https://github.com/MakieOrg/GraphMakie.jl/pull/89/files#diff-8861ed4224a2baebdb2c6efa750e27a9d73b2b84f72f4f46b153786969fd7f8eL259
https://github.com/MakieOrg/GraphMakie.jl/pull/89/files#diff-8861ed4224a2baebdb2c6efa750e27a9d73b2b84f72f4f46b153786969fd7f8eL304
https://github.com/MakieOrg/GraphMakie.jl/pull/89/files#diff-8861ed4224a2baebdb2c6efa750e27a9d73b2b84f72f4f46b153786969fd7f8eR747
The problem was that in the @lift command not the Observable type was used.

Renaming variables
In lines like https://github.com/MakieOrg/GraphMakie.jl/blob/7c2e706a37d29307ca2b0b1cc8dc9d42691d6da2/src/recipes.jl#L206 , https://github.com/MakieOrg/GraphMakie.jl/blob/7c2e706a37d29307ca2b0b1cc8dc9d42691d6da2/src/recipes.jl#L235 ,
https://github.com/MakieOrg/GraphMakie.jl/blob/7c2e706a37d29307ca2b0b1cc8dc9d42691d6da2/src/recipes.jl#L286,
the same name as the recipe Observable is given to represent same reference.
This was in contrast to how node_color, node_marker, node_strokewidth are meant which are different dependent Observable. I think it's more clear to give them different name. I opted for name_m where "m" stood for modified.
Extension of Dict to all supporting arguments
The main work of this PR is modifying get_attr and adding the prep_arguments functions as already discussed some months ago. Now, I just inlined the call to this function to all appropriate recipes.
Use of Dict{Edge} for edge properties
We 've talked in the past about supporting Dict{Edge} for edge properties.
At the beginning I was hesitant to implement something like that since for big graphs it could get slow and memory hungry.
However I think with the current design it's okey.
Already at compile time (or one dynamic dispatch per graphplot) the correct flavor of getedgeinds is called that returns the appropriate indexing. Only when the graph is undirected and an AbstractDict{<:AbstractEdge} is given there is some further computation, which is being done on demand using Iterators rather than saving it in memory.
To further increase efficiency we can use some lazy implementation that caches the result.
SimpleTraits.jl dependency
SimpleTraits.jl is already in the Manifest.toml as it's a dependency of Graphs.jl.
I find it elegant to use it also here.
Docs
I updated the docs to talk about the new feature.
I don't have examples there but I reference the reftests
Reftests
I introduced quite some reftests.
I wanted to introduce them somewhere in the middle of the document as they would be better categorized, but I realized I had to change all the assets numbering. Is there a clever way to do that, or should we always add reftests in the end to avoid such developer overheads ?
Versioning
This is non-breaking but still introduces some obvious new features. Should that be a minor or patch update ?
docs/Manifest.toml
The ./docs should depend on the GraphMakie as dev ... Otherwise it wouldn't follow locally up until the version was published.
Am I right ? How do you develop locally when there are such changes ?
If you like, this could be a separate PR.
Remarks
I just wanted to take the opportunity to highlight some points
element-wise automatic
Currently the Dict funcionality can have a single default value.
However, some arguments like label rotation etc. have different default values per element.
I think we should support that the fallback for Dict and DefaultDict should be these default values.
However, I would advise that to be a separate PR.
Need for benchmarks
As I mentioned I was a bit concerned with performance and currently there is no out-of-the-box way to test how we do.
There are also some other julia packages we can compare to, or even to the python ecosystem.
This can bring in a new dimension to this package, which I think is desirable.

I think it's more clear to give them different name. I opted for name_m where "m" stood for modified.

totally agree

Use of Dict{Edge} for edge properties

see comments on files, i think we can reduce some unecssary updates by splitting prep arguments in two functions for nodes and edges

I wanted to introduce them somewhere in the middle of the document as they would be better categorized, but I realized I had to change all the assets numbering. Is there a clever way to do that, or should we always add reftests in the end to avoid such developer overheads?

No, not yet. There is a bit of a tradeoff, currently we only need to annotate with @reftest and number them automaticially by file name. I have an not-yet-registered package PlotReferenceTests.jl which does things similar but slightly different, there you'd need to give every reftest a name @reftest fig "example". I think it makes sense to switch eventually as it also works much better when interactivly stepping through those tests out of order (as I often do during debugging).

Versioning

I see that it is a big feature upgrade but I'd still opt for non-breaking release. I think there is no reason to make everybody bump there versions.

docs/Manifest.toml

What i normally do is executing docs/localmake.jl from a terminal. It will set up the doc env similar to what happens in CI. And if you have LiveServer in your global env it will serve the docs and you can repeatadly rebuild them without restarting the julia session. Even though I just noticed that there should be a using Revise in there too.

element-wise automatic

I agree, this changes the structure quit a bit but is relatively simple and should be done in a separate PR. Basicially we would first need to expand/prep the arguments  before resolving the automatics, which is currently done first creating those _m version.

benchmarks

Agree. I think there is room for a lot of improvement in the package, currently everything is super dynamic and type unstable.

I'll merge this into a feature branch of the repo to get the doc preview build, you should have all the rights to push to that new branch too


Relations do not change

What happened?
Relations do not change if you navigate with the navigation controlls.
How to reproduce the bug
Make a 1:n relation, a form, enter data open one of the records and navigate with the navigation controlls. The relations do not change.
Package Version
2
PHP Version
8.3
Laravel Version
10
Which operating systems does with happen with?
No response
Notes
No response

Can you provide the context in which you're using the relation? A select field? Or other type?

Related: If you use a relation manager, the related records won't change as you navigate through the records.


Bump read-json version

This is due to https://medium.com/@azerbike/i-ve-just-liberated-my-modules-9045c06be67c. read-json was one of the casualties. Just needs a version bump.

lgtm.


Comments : Ruby : Skip code under multi-line commented portion

To be able to skip the scanning between the commented code written in Ruby I mentioned prefix and suffix commenting style for the same. This change will just skip the matches/samples found under "=begin" and "=end".

Ruby multiline comments appear to be unique in that they must begin at the start of a line. If a multiline comment is started after some code on a line it will actually be a syntax error. I don't think it is worth at this time implementing that exact behavior to only match multi line comments on line start as 1. as it would require a number of changes across the code base so this seems like a good enough solution 2. its not valid ruby anyway.


Add tooltips to show bucket data in time-series

Related to: https://github.com/CartoDB/cartodb/issues/11650
Changes in PR https://github.com/CartoDB/deep-insights.js/pull/582

For acceptance:

Create a TimeSeries / Torque TimeSeries
Hover the buckets and see the tooltips

retest this please

Going back to development.
This works perfectly when the column is number, but when the column is a date the tooltip doesn't show the correct info.

retest this please

retest this please


[release/3.1] Evaluate passing AlwaysPackEvenIfNotServicing=true to Core-Setup for next servicing release

Core-Setup is moving to a package-granular release strategy of its managed packages with https://github.com/dotnet/core-setup/pull/9049. I think we probably need to always build those in source-build so that downstream repos can consume the binaries even when they won't be shipped in the next Microsoft build.
A flag AlwaysPackEvenIfNotServicing is in that PR to let us disable the granularity. I believe this will probably be needed to avoid adding a prebuilt.
Another thing to consider is whether we need to downgrade the produced package's version to match the last-released Microsoft package.

Another thing to consider is whether we need to downgrade the produced package's version to match the last-released Microsoft package.

This is somewhat tied to https://github.com/dotnet/source-build/tree/release/3.1/Documentation/planning/nongranular-servicing-readiness, but for the time being, the default is to match the Microsoft build and we should have this downgrade.

This flag was implemented in the 3.1.107 release here: https://github.com/dotnet/source-build/pull/1702/
We chose not to downgrade the package's version as testing didn't uncover any issues with the version difference.  This does introduce a difference between the source-built build and the "official" build WRT the version of the assemblies built with this flag.
@dagood can this be closed?

Yep, thanks. This issue just tracks the flag to avoid a prebuilt, and https://github.com/dotnet/source-build/issues/1691 tracks doing further validation that enabling this flag without fixing the version difference hasn't hurt any scenarios.


multimon-ng: 1.1.4 -> 1.1.5

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/multimon-ng/versions.
These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 1.1.5 with grep in /nix/store/x3cfglkwjspjg0gv4vlalwzw7wykp2p3-multimon-ng-1.1.5
directory tree listing: https://gist.github.com/f266692b50ec442e1bb40052b1440be8

cc @the-kenny for review

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: multimon-ng
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: multimon-ng
Partial log (click to expand)
installing
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/z24x8rvxfvwcmqnlziff46czmrjm56g4-multimon-ng-1.1.5
shrinking /nix/store/z24x8rvxfvwcmqnlziff46czmrjm56g4-multimon-ng-1.1.5/bin/multimon-ng
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/z24x8rvxfvwcmqnlziff46czmrjm56g4-multimon-ng-1.1.5/bin
patching script interpreter paths in /nix/store/z24x8rvxfvwcmqnlziff46czmrjm56g4-multimon-ng-1.1.5
checking for references to /build in /nix/store/z24x8rvxfvwcmqnlziff46czmrjm56g4-multimon-ng-1.1.5...
/nix/store/z24x8rvxfvwcmqnlziff46czmrjm56g4-multimon-ng-1.1.5

Success on aarch64-linux (full log)
Attempted: multimon-ng
Partial log (click to expand)
installing
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/x6nij8yvdp14h930scg09x7dbbxn6srf-multimon-ng-1.1.5
shrinking /nix/store/x6nij8yvdp14h930scg09x7dbbxn6srf-multimon-ng-1.1.5/bin/multimon-ng
strip is /nix/store/j7d4mr0ikv974ig7yzhknpsq288js4bs-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/x6nij8yvdp14h930scg09x7dbbxn6srf-multimon-ng-1.1.5/bin
patching script interpreter paths in /nix/store/x6nij8yvdp14h930scg09x7dbbxn6srf-multimon-ng-1.1.5
checking for references to /build in /nix/store/x6nij8yvdp14h930scg09x7dbbxn6srf-multimon-ng-1.1.5...
/nix/store/x6nij8yvdp14h930scg09x7dbbxn6srf-multimon-ng-1.1.5


[Query expression] On-conflict expression evaluated to an error; panics

Description:
type TT record {|
    readonly int id;
|};

type TableT table<TT> key (id);

function f1() returns error? {
    return error("Key Conflict");
}

function dummyTest() {
       TableT|error tt = table key(id) from var i in [1, 2, 3]
                          select {
                              id: i
                          }
                          on conflict (check f1());
}

When we pass the check-expression as an argument to addToTable()  function; the expression is evaluated and the error is thrown out.
To solve this; we need to nest this function inside a separate function or with-in on-fail and handle the error gracefully.
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

As per https://github.com/ballerina-platform/ballerina-lang/issues/37310; check-expr's error should be returned to the enclosing invokable


QuerySet not available anymore

Bevy version
latest version after cargo clean and cargo update
hash: f907d67d7ed5cfd8b11b2ad78e9dfd6eeb913cea
Operating system & version
Arch Linux on Kernel: 5.17.1-arch1-1
What I did
Updated bevy version to latest git version and now my Game will not compile if it utilizes QuerySet
And now none of these are available anymore:
bevy::ecs::prelude::QuerySet
bevy::ecs::system::QuerySet
bevy::prelude::QuerySet
bevy::ecs::system::QuerySetState

This has been changed to the more general ParamSet in #2765 :) The migration should be relatively painless, but do let us know if you run into problems.


Dropdown accessibility

I have a dropdown with a custom template and it seems like tabbing into it does not work. When there is no template, it seems to work.
Additionally, the dropdown component doesn't let you use arrow keys to select different options.

Hi, is this project not being maintained anymore? Or am I wrong in assuming that this should be a pretty high-priority item for any scalable component library?

Yes it is still being maintained (primarily by a single person who's had trouble finding extra time to work a lot on it in the last couple of weeks). You're very welcome to fix it yourself and create a PR from it!

Thanks for letting me know!

Hey @tobiasandersen, I created a pull request to add support for this -- https://github.com/react-toolbox/react-toolbox/pull/1718
It's my first pull request to react-toolbox so let me know if I need to change anything.
Thanks!


Returns null for the values in authData,Like email,clientId

Returns null for the values in authData,Like email,clientId and other fields in authData class when done method is called
This happens only with the user not having two-factor authentication to their discord account.
For those who have the fields of AuthData are received and don't remain null

What do you see in the userJson and response properties?

@thepranays Null is returned if authentication fails for any reason or if the user hasn't granted access for those fields. Can't help you without more info. Closing this for now; feel free to reopen anytime.


[folding] Better cursor location when hit enter at a folded line.

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.85.1 (user setup)
OS Version: Windows_NT x64 10.0.19045

Steps to Reproduce:

fold a region.
hit Enter at the end of the folded line.

expect: cursor at end of folded region. (B in the gif)
actually: cursor at the beginning of folded region. (A in the gif)
Version: 1.85.1 (user setup)
Commit: 0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2
Date: 2023-12-13T09:49:37.021Z
Electron: 25.9.7
ElectronBuildId: 25551756
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

Well, the cursor is before the ..., so IMO it makes sense to keep it at the beginning of the folded range.
To insert a new line at the end of the folded region, set the cursor at the beginning the line below the folded range and invoke Insert Line Above

OK, now the behavior makes sense to me.
So, if the folded line is the last line in the file, do I have to unfold it to insert a line below the folded range, or is there a better way?

So, if the folded line is the last line in the file, do I have to unfold it to insert a line below the folded range, or is there a better way?

No in that case you first need to unfold.


doc: shorter links to user-guide

Hi, I need a few shortlinks for the following, which I need for dvc exp init.

https://dvc.org/doc/user-guide/project-structure/pipelines-files
https://dvc.org/doc/user-guide/experiment-management/running-experiments

I'd prefer something like https://dvc.org/doc/pipelines-files or even https://dvc.org/pipelines-files. Or, something else that is much shorter and still readable/understandable.
Here's how we display it right now in dvc exp init:

Looking at the dvc's code, we only seem to have short links for man.dvc.org which redirects to command-reference. We could also have something similar like https://guide.dvc.org/running-experiments (though it's 2 characters lengthier than https://dvc.org/doc/running-experiments).

I'd say "user guide" needn't show up in the URL at all (dvc.org/doc/user-guide/experiment-management => dvc.org/doc/experiment-management)

I'd say "user guide" needn't show up in the URL at all ever

I think I miss your point. What if we have get-started/experiment-management and user-guide/experiment-management

We can also have two letter redirects /gs, /uc, /cr for other sections.

I won's solve the experiment-management/running-experiments problem in the initial @skshetry 's concern.

I still think we need to create a simple custom url shortener which anyone could use to their benefit. I don't see a particular reason to keep it about docs specifically (leak very specific docs structure into it).

We can also have two letter redirects /gs, /uc, /cr for other sections.

I won's solve the experiment-management/running-experiments problem in the initial @skshetry 's concern.

dvc.org/doc/user-guide/experiment-management/running-experiments > dvc.org/ug/exp/run

Seems like the way we're talking about this entails a more elaborate system that just throwing in some redirects-list entries, but considering such a system may take a while, would it be better for now to just use redirects-list entries with some convention that the new solution will follow?

would it be better for now to just use redirects-list entries with some convention that the new solution will follow?

sounds good to me!

dvc.org/ug/exp/run

we are still polluting the global URL space with the docs structure.
Let's use some common third-level URL s.dvc.org or some designated global namespace - dvc.org/s for this please.
After that it will up to the author to put a link they like there. It can be even s.dvc.org/ug/exp/run

I'd say "user guide" needn't show up in the URL at all ever

I think I miss your point. What if we have get-started/experiment-management and user-guide/experiment-management

I was replying to a tangential discussion (https://github.com/iterative/dvc.org/issues/2967#issuecomment-954642910).
In terms of this mostly unrelated issue (#2967, i.e. bespoke one-off shortlink), DVC team should open a PR?

I still think we need to create a simple custom url shortener which anyone could use to their benefit. I don't see a particular reason to keep it about docs specifically (leak very specific docs structure into it)
Let's use some common third-level URL s.dvc.org or some designated global namespace - dvc.org/s for this please.
After that it will up to the author to put a link they like there. It can be even s.dvc.org/ug/exp/run

I'd say that's another separate tangential meta issue, but fully agree.

we are still polluting the global URL space with the docs structure.

Ok. We can use dvc.org/s/ug/... for the time being, and move to s.dvc.org/ug/... when the new domain is ready to serve. Given the time and configuration constraints, this looks like the simplest approach.

s.dvc.org is done (it's fast to create domains like this).

Is it a CNAME for the dvc.org domain? We'll use redirects as usual?

We'll use redirects as usual?

Yep

which I need for dvc exp init.
https://dvc.org/doc/user-guide/project-structure/pipelines-files
https://dvc.org/doc/user-guide/experiment-management/running-experiments

Hi. So is this closed by https://github.com/iterative/dvc.org/pull/3001? I don't see those 2 specific redirects in there.

https://dvc.org/doc/user-guide/experiment-management/running-experiments is now https://s.dvc.org/g/exp/run
https://dvc.org/doc/user-guide/project-structure/pipelines-files is https://s.dvc.org/g/project-structure/pipelines-files
If you need shorter versions to these, please let me know or just submit a PR with s.dvc.org domain. @skshetry


Add Nathaniel-Joe-Unruh to Contributors list

Hi  @Nathaniel-Joe-Unruh, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


I want to have dates not useless information about blocknumber in my history!

I am not a robot. As a user I prefer to you see and use well formmated dates like 03/03 02:51:12, rather than seeing a useless information about block number.

The interesting part is... that this is not only wrong.. but this is not even a consistent:
All History has good date:

There's a simple reason for this: the date is not available in the data from the api but the block number is. We have a component that calculates the date from the block number, but I tested it here and it's inaccurate due to missed blocks, so you end up displaying a date that could be several days off.
I could instead display a date like "4 months ago", which means the accuracy becomes less of an issue. You can then click on the date to see the block where the transaction happened, which does include the actual date. Would that be acceptable?

@svk31 Can you call get_block API to fetch the real timestamp?

By the way on my laptop the timestamp in all history always shows as format "yyyy/m" but not "m/d" or "d/m", very strange. Win7 64bit. Default language in my laptop is set to Simplified Chinese. BitShares GUI language is English.
420.00001428	0.23809523	100.0000	2016/3 02:51:12

And the history box is too narrow in vertical-order-book mode (screen of my laptop is small though). Can it be placed at the bottom in this case (for example aside "my orders" box)?

This issue was moved to bitshares/bitshares-ui#42


Build issue

I follow the UHDM-integration https://github.com/chipsalliance/UHDM-integration-tests to build the toolchain, while when I execute ./build_binaries.sh, there always meet an error
yosys-uhdm-plugin-integration/Surelog/src/DesignCompile/UhdmChecker.cpp:249:42: error: ‘setprecision’ is not a member of ‘std’
and
yosys-uhdm-plugin-integration/Surelog/src/DesignCompile/UhdmChecker.cpp:315:67: error: ‘setw’ is not a member of ‘std’
I think it is because this file forget to
#include <iomanip>
add this line and build will succeed.

Please submit a PR with the change.


Added Send trait to NewEv generic type inside Capability derive macro

Fixes #147 by adding the Send trait to the generic type NewEv inside map_event function of the Capability macro.
In the tap_to_pay example, there is a Delay capability in which the same was done:
impl<Ev> Capability<Ev> for Delay<Ev> {
    type Operation = DelayOperation;
    type MappedSelf<MappedEv> = Delay<MappedEv>;

    fn map_event<F, NewEvent>(&self, f: F) -> Self::MappedSelf<NewEvent>
    where
        F: Fn(NewEvent) -> Ev + Send + Sync + Copy + 'static,
        Ev: 'static,
        NewEvent: 'static + Send, // <------------- Send trait was added
    {
        Delay::new(self.context.map_event(f))
    }
}

Hey @mmannes Thank you for this! Instead of adding the bound in the macro, I added it to at the function level. It seems to make more sense here to me, not sure? Anyway I updated the Delay capability in the tap to pay example and the docs on writing a capability, have a look and LMK if you're happy, and I'll merge.


Bluetooth: gatt: notify application of MTU updates

Add a new callback to struct bt_conn_cb to allow applications to track
changes to the ATT MTU. This callback is only called when the remote
side changes the MTU. The application is already notified of a changed
MTU if it initiated the update via the struct bt_gatt_exchange_params
callback.
Fixes #32035.
Signed-off-by: Jordan Yates<EMAIL_ADDRESS>

This looks like layer violation to me.

True, I didn't consider the layering or alternate protocol implications.
I can see two options for this functionality then.

Define a new callback structure in <gatt.h>, analogous to struct bt_conn_cb, which contains the MTU callback.
Add an MTU callback field to struct bt_gatt_service, so that services that care are notified of MTU changes.

Implementation has moved from the conn files to gatt files.

@JordanYates will you follow this up after the review?

@JordanYates will you follow this up after the review?

Sure. I have no knowledge of EATT thought so I asked @Vudentz for some guidance.
I was waiting on that before implementing something that would probably be wrong.

@jhedberg custom list replaced with sys_slist_t


typo in docs

didnt create a pull request to avoid triggering the CI, not sure if you have some guide on reporting/fixing typos.
anyways, right here the command executed is poe greet, but it looks like it should be poe clean (see line 60 on the same file).

Hi @demetrius-mp, thanks for reporting this. For documentation fixes you can make a PR straight to master, or else I'll make a note to fix this when I have some time.
I also just realized the link from the readme to the CONTRIBUTING.rst was recently broken 😖


empty output for classes in default package

Test case - same as #361 but remove references to TimeUnit.
Output: file is created but is blank. Does not happen if file is put in a package.

fixed in #371


Adding Windows 10/11 Provision for Students

Previously, only users with a multi-tenant hosting rights license could deploy Windows 10/11 virtual machines on Azure. This almost entirely excluded free trial and student users from deploying these types of VMs since the vast majority of these users would not have that type of enterprise license. The Azure portal team has worked with the Licensing Business planning team to include free trial and student subscriptions in the Windows 10/11 licensing program on Azure so that these users are allowed to deploy these images as well. Please reach out to<EMAIL_ADDRESS>if more details are needed.

#sign-off

#sign-off

@rymend

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Add more verbose node alternatives

In init.lua, you define
	s = snip_mod.S,
	sn = snip_mod.SN,
	t = require("luasnip.nodes.textNode").T,
	f = require("luasnip.nodes.functionNode").F,
	i = require("luasnip.nodes.insertNode").I,
	c = require("luasnip.nodes.choiceNode").C,
	d = require("luasnip.nodes.dynamicNode").D,

which is nice and terse, but not super user-friendly. I know that it put me off when I first checked out this plugin and I keep needing to look up what they mean every once in a while.
Since most users will alias those functions anyways (e.g. local s = ls.s), would you be open to adding more verbose variants as well? local s = ls.snippet would be much more readable for most people and barely more verbose in the context of a full config. Of course the current ones would stay, if only for backwards compatibility.
I'd be happy to submit a PR with this and adjusting the documentation if that sounds good to you.
Thanks for LuaSnip btw, it looks like a really solid plugin from what I've seen so far!

Great suggestion, I can't see any downsides to adding those!


Architecture prototype: Try the prototype with realistic data sets

Description of Problem:
Once we have the prototype training and predicting we need to know it actually works properly with realistic data (Sara bot).
Overview of the Solution:
We should try training on some realistic data sets and check predictions work accordingly.
It should also be able to handle large amounts of data.
Blockers (if relevant):
#8412
Definition of Done:

[ ] Training completes as fast as 2.0
[ ] Predictions are correct and are as fast as 2.0

(I had to remove all rules as Sara seams to contain conflicting rules which lead to a failed training)
Training:

rasa train on main: 	40m11.933s
prototype:

Prediction Time
I did 1000 predictions:

rasa on main :
prototype:


Firestore database name customization is not impented.

Thanks for stopping by to let us know something could be better!
PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.
Is your feature request related to a problem? Please describe.
Currently, the php-sdk does not allow us to set a custom database name because it is hardcoded in the constructor (still in 1.39):
    public function __construct(array $config = [])
    {
        $emulatorHost = getenv('FIRESTORE_EMULATOR_HOST');

        $this->requireGrpc();
        $config += [
            'returnInt64AsObject' => false,
            'scopes' => [self::FULL_CONTROL_SCOPE],
            'database' => self::DEFAULT_DATABASE,
            'hasEmulator' => (bool) $emulatorHost,
            'emulatorHost' => $emulatorHost,
        ];

        $this->database = $config['database'];

        $this->connection = new Grpc($this->configureAuthentication($config) + [
            'projectId' => $this->projectId,
        ]);

        $this->valueMapper = new ValueMapper(
            $this->connection,
            $config['returnInt64AsObject']
        );
    }

Describe the solution you'd like
It would be nice to have the hability to change the database name, in case the administrator changed it in GCP:
The change is pretty simple:
    public function __construct(array $config = [])
    {
        $emulatorHost = getenv('FIRESTORE_EMULATOR_HOST');

        $this->requireGrpc();
        $config += [
            'returnInt64AsObject' => false,
            'scopes' => [self::FULL_CONTROL_SCOPE],
            'database' => $config['database'] ?? self::DEFAULT_DATABASE, // <========= This little change
            'hasEmulator' => (bool) $emulatorHost,
            'emulatorHost' => $emulatorHost,
        ];

        $this->database = $config['database'];

        $this->connection = new Grpc($this->configureAuthentication($config) + [
            'projectId' => $this->projectId,
        ]);

        $this->valueMapper = new ValueMapper(
            $this->connection,
            $config['returnInt64AsObject']
        );
    }

(plus Phpdoc updated obviously).
Describe alternatives you've considered
N/A
Additional context
Add any other context or screenshots about the feature request here.

Hello, for some reason when I passed it to the constructor the database stood on "(default)", I'll investigate this behavior on my side. Have a nice day !


DefaultContractResolver broke client get item query

Describe the bug
when using DefaultContractResolver in AddDatasyncControllers (#796), client query items broken, all integration tests fail.
Table GetAsyncItems and GetItemAsync return null or empty from sql statement, but Sqlite is case-insensitive so issue is somewhere but I not able to find where.
To Reproduce
use services.AddDatasyncControllers(new DefaultContractResolver());
Expected behavior
all test pass (after case-insensitive fixes)
Additional
some integration tests can be solved changing comparison of hardcoded system and page properties to case-insensitive

This would be expected.  A lot of the tests basically do something like:

Set up test
Run test
Check that the JSON is correct.

You have changed the semantics of that JSON, so we would have to go through each and every one of the integration tests to ensure that the correct JSON is expected.
The question is - does your application work?  If not, write a few tests (it should not take more than a few tests) in the integration test suite for your specific requirements.

Ok, some tests fail for case sensitivity as expected without case-sensitive refactoring, but that's not my goal now.
I have create a #803 to test basic crud operation with DefualtContractResolver


.6.7 totally breaks zmodem downloads...

shsbbs.net, I can't download from any of my download apps (OLMS2000, the reader or nodeview 2.10) or the files area download protocols.  It just dumps straight out of the download to a garbage full screen.

fixed the zmodem for synchronet - maybe that had side effects. I'll take a look.

Found it - I broke zmodem unit tests because I moved it to another layer. I'll put some work in it to reactivate the unit tests.

0.6.8 fixes that.


Custom column titles for needtable does not work for need_extra_options with "-"

If needs_extra_options definition has a "-" in its name, the custom column title for needtable results in empty table column contents.
needs_extra_options = [
    "short-description"
]

.. needtable::
  :columns: short-description as "Description"

I have an issue somewhat related to this: If you add a % character to the title of a column, the replacement won't work at all:
.. needtable::
    :columns: id;title; modified_conditions as "MCDC Coverage %"

Here the column gets rendered as "Modified Conditions as MCDC Coverage %" and shows no data.
I think this is caused by the regex used to interpret the columns titles:
https://github.com/useblocks/sphinx-needs/blob/c27bc6449c0d2a106d953f4ec9e3051a9dc4500b/sphinx_needs/defaults.py#L202
([\w]+) as "([\w ]+) does not match the inputs properly:

https://regex101.com/r/ZoXhkK/2
https://regex101.com/r/E0mjVI/1


Modal: Consolidate the size attribute to core modal one

The problem
<dds-modal> has expressive-size attribute to support "full width" mode where Carbon core <bx-modal> has size attribute.
The solution
See if a single size attribute can support both cases.

Turned out it's hard or impossible; Hit a limitation where TS doesn't allow extending enum. Closing for now.


targeting bug

Challenge http://www.freecodecamp.com/challenges/waypoint-target-the-same-element-with-multiple-jquery-selectors has an issue. Please describe how to reproduce it, and include links to screenshots if possible.

@impaler1 thanks for posting this issue!
Actually your code is incorrect because:

Use each of these jQuery selectors to target your button element with the class btn and the id target1.
Use the addClass() jQuery function to give the element one new class for each selector: animated, shake, and btn-primary.

So the correct code looks like this:
$('.btn').addClass('animated');
$('button').addClass('shake');
$('#target1').addClass('btn-primary');

or another five combinations.

Please review the Guidelines for Contributing, thank you!.


Stress test app

Stress test app using in the performance report. Currently set up to render 100,000 points in 1000 ScatterplotLayers, but could be easily extended to be more customizable in the future.
Also update probe.gl to latest (there was a bug preventing the reading of frame times).

Coverage decreased (-2.7%) to 80.305% when pulling 8e561cd64270c632c88c966d279f782677c08d77 on stress-test-app into 46ab2abc219269bd95e97c4bbb1ff8200977aa11 on master.

This is great!
Just a generic comment - shall we make this a pure JS app since React can pollute the profiler? You are already breaking the pattern by directly calling deck.setProp.

Yeah, I can definitely do that.

Nit: we have several random data generator utility methods in test with minor differences in data format, at some point we should consolidate all these :
https://github.com/uber/deck.gl/blob/0fec14d85975c5d74c96f698ca3e77d05c19de79/test/apps/data-filter/src/data-sample.js#L2,  https://github.com/uber/deck.gl/blob/0fec14d85975c5d74c96f698ca3e77d05c19de79/test/data/grid-aggregation-data.js#L147

I also added the pico-mercator version of the stress test to make it easy to compare.


Hide Tor addresses by default (option to copy / reveal the address with one click)

See #8

To blur it  & make it non-selectable:
   color: transparent;
   text-shadow: 0 0 10px rgba(0,0,0,0.5); // maybe 15px, depending how readable it is, alternatively change it with placeholder text.
   user-select: none;


fix: Visual things in modals

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #7654

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI;

Copilot Summary

🤖 Generated by Copilot at 5fd393a
Improved the styles and consistency of some modal and identity components. Adjusted the padding, alignment, and color of IdentityItem.vue, ConfirmPurchaseModal.vue, and ListingCartModal.vue.

🤖 Generated by Copilot at 5fd393a

To make the modal header look nice
They removed modal-card-head in a trice
And added some classes
For padding and axes
Like is-align-items-center and py-5

@exezbcz pls visual check

of the top section to k-shade, in dark mode k-hovergrey or k-grey

it's unclear what you mean , i changed the "connected with"  label to have these colors

@exezbcz pls visual check

of the top section to k-shade, in dark mode k-hovergrey or k-grey

it's unclear what you mean , i changed the "connected with"  label to have these colors

I meant the stroke of the section

@exezbcz 🚀

It's a really small detail, but now the x-mark is not aligned on the x-axis.

pay 30 usd
thanks!
small details later 👋

😍 Perfect, I’ve sent the payout
💵 $30 @ 3.73 USD/DOT ~ 8.043 $DOT
🧗 13QUj3pZyFNfYj4AM336hRdyLQbevj5H3sR4PKmLEXLdwZhh
🔗 0x96202e309304ba85b8725e27dedcbca6500f5ef7c517e6c57581c39d569b81de
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues

small details later 👋

its not really small since its the same that was before just on other side..


BlockCheck(Liquid/Stationary_Water wont work) 1.16.5

OTG Plugin or Mod

[X] OTG Mod for Forge

OTG Version
OpenTerrainGenerator-1.16.5-0.0.17r1.jar
Forge or Spigot Versions
forge-1.16.5-36.2.2
What other plugins are you running?
None
Description
BlockCheck() Liquid wont work. Legacy: Stationary_water, Flowing_Water & Water wont work either. (Possibly Stationary_Lava, Lava & Flowing_Lava neither.) Blockchecks with prefix minecraf: wont work either.
Simple Bo3: Just add CustomObject(TestBo3) and it should spawn.
Freq & Rarity set to 25.
Legacy Bo3:
TestBo3.zip
minecraft: prefix Bo3:
TestBo3.zip
Here's a test world that includes the minecraft: prefix bo3
Debug.zip

Fixed this one.


Fixed #34544 -- Avoided DBMS_LOB.SUBSTR() wrapping with IS NULL condition on Oracle.

ticket-34544
Regression in 09ffc5c1212d4ced58b708cbbf3dfbfb77b782ca.
Thanks Michael Smith for the report.
This also reverts commit 1e4da439556cdd69eb9f91e07f99cf77997e70d2.

buildbot, test on oracle.

@msmitherdc Does it work for you?

@msmitherdc Does it work for you?

Yes, this patch works for me. Thanks for the fast turnaround.

Thanks y'all for reviews :star2:


Feature: modify CONNECT response via proxy_connect_response directive

test cases with lua + openresty:
 $ sh runtest.sh
t/http_proxy_connect.t .................... All 22 subtests passed
t/http_proxy_connect_lua.t ................ All 1 subtests passed
t/http_proxy_connect_resolve_variables.t .. All 6 subtests passed
	(2 TODO tests unexpectedly succeeded)
t/http_proxy_connect_timeout.t ............ All 5 subtests passed
	(3 TODO tests unexpectedly succeeded)


vutuv.de/users/{user}/skills_create returns “Page not found” Error

I just created my vutuv account. The confirmation link in my email, brought me to a “get started” page, asking me to fill in some skills. I enter a few keywords and click on the large button directly beneath the skill input field and get this HTTP 404 error.
Update: This is also the case for:
/users/{user}/(edit|((work_experiences|phone_numbers|links|addresses)/new))

This is duplicate with my ticket. #352

Which vutuv account is that? Please include the used URL.

Hello,
having the same issue: https://www.vutuv.de/users/kai.korla
regards
Kai

@GinoHereIam ☺ correct. Maybe this issue can be closed. @wintermeyer One URL was https://www.vutuv.de/users/micha.rosenbaum/skills_create Btw I’m really looking forward to use this app, it looks great so far.

Same issue here with https://www.vutuv.de/users/matthias.gutjahr/skills/new and about any other link where I'm supposed to edit stuff.

Thanks everyone for reporting this error. A hotfix is rolling out, please let us know if the error is fixed.

@cjfreeze Not yet.

@rosetree The second hotfix has rolled out, I am seeing the error fixed on my end. Can you confirm?

@cjfreeze I can now open my edit page, however the skills_create page still returns a page not found.

@rosetree The skills resource is located at vutuv.de/users/<your_slug>/skills. The skills_create is only programmed to accept a POST reqeust for adding several skills at the same time from the user's profile page, so it won't accept a get request.

Closed by 72254b2cd141891573086004d258694551d6db6d

I just logged out and back in, but the still same issue.

Hi,
same Problem. Starting at my page and want to input list of skills. Entering skills and pressing 'Add' results in 'Page not found ' message. See screenshot attached.
regards
Kai

@GinoHereIam @balticer Unlike /edit, The /skills_create is not a link that shows a page, but a link for use by the application. To view and edit your skills, please browse to /skills.

@cjfreeze shouldn't behave the frontend different then? I also can't open followers. Only edit my profile works. Looks like 90% of hyperlinks are dead to me. :(

@GinoHereIam There is actually another bug. Sorry for the confusion, it was just made apparent to us. Seriously, thanks for your patience. Working on it now. Please post related information in #356


Update rpassword to remove dependency on termios

Thanks! I'll look at this asap.
On Dec 14, 2017 11:20 AM, "Jonathan Morley"<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/hwchen/keyring-rs/pull/13
Commit Summary

Update rpassword to remove dependency on termios

File Changes

M Cargo.toml
https://github.com/hwchen/keyring-rs/pull/13/files#diff-0 (2)

Patch Links:

https://github.com/hwchen/keyring-rs/pull/13.patch
https://github.com/hwchen/keyring-rs/pull/13.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hwchen/keyring-rs/pull/13, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADm0i9_Fwbbt-E1lPu4M29H84ntUbVo5ks5tAUrogaJpZM4RCTyw
.


Release v1.0.0-rc2

新增

安全通讯：SDK和Router之间采用TLS协议通讯
资源接口：提供资源访问相关接口
配置文件：新增配置文件，配置TLS证书信息

更新

RPC接口调整

合约调用需要指定签名的账户
合约调用不再需要指定返回值类型列表，因此删除了相关衍生接口

Codecov Report

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

@@            Coverage Diff            @@
##             master      #63   +/-   ##
=========================================
  Coverage     36.40%   36.40%
  Complexity      127      127
=========================================
  Files            47       47
  Lines          1247     1247
  Branches         79       79
=========================================
  Hits            454      454
  Misses          776      776
  Partials         17       17

Continue to review full report at Codecov.

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


sticky filter for cluster/namespace/event type

We love working with Robusta on a daily basis while managing multiple clusters. One pain point that we have when using the UI, though, is that we constantly have to narrow down and focus the context by only selecting the clusters/namespaces we are currently interested in.
In our case, when we want to analyse or troubleshoot something, this is usually (>90% of the time) limited to one cluster and/or a selection of namespaces. And for this, we mainly use and switch between Apps and Timeline views.
BUT, each time we set the filter to see what we would like to see, it gets reset as soon as we switch the view, from Apps to Timeline or vice versa. This is a real bummer.
Instead, we'd love to have the filter sticky and consistent for (ideally) all views (pages) without resetting on a view change. It would be really cool to have the filters be sticky and survive when going from one view to another, maybe even between multiple sessions.

I understand that it might get tricky, given the point that Apps and Timeline have a different set of filters. But the fields not available for Apps (Events Name and Priority) could simply be greyed out (probably reordered, too) and only active for the Timeline.
Thanks for the great product you are delivering!

While thinking about it and using the filters, I figured it would also be great to have another field for the Timeline to filter out the [RESOLVED] events from being listed and only display events that currently don't have the RESOLVED status.

We would like this feature as well.

Thanks for reporting it @kayahk and @Noamstrauss
We're going to implement the parameters presets in the UI.
For now, as a workaround, you can save the URL of a specific filter selection. The parameters are part of the URL, so it can make it easier to access previous filter selection.
Regarding the RESOLVED status filtering, it's on our roadmap as well

I agree with @kayahk as well, hopefully, this will be implemented soon :)

Hey all, we've added presets to the UI. Does the current implementation work for your needs?

Closing this issue as it is resolved, and we do not have new requests.


"Secret" runtimeConfig shown during window reload

Environment

npx nuxi info
Working directory: /home/eriase/Repositories/nuxt-app
Nuxt project info:

Operating System: Linux
Node Version:     v20.10.0
Nuxt Version:     3.10.1
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, runtimeConfig
Runtime Modules:  -
Build Modules:    -

Reproduction
Don't have a repo for this so just providing a tar.gz file instead
nuxt-app.tar.gz
Describe the bug
After building the application and running npm run preview and then refreshing the page I can see the config.hello value flashing before being de-loaded from the homepage.
Additional context
No response
Logs
No response

That's correct as the "private" runtimeConfig is available on the server side. Rendering it in a template will leak it when doing the initial request.


datastore: support conversion of protobuf ptypes directly

Consuming protobufs in an API (gRPC or otherwise) and then storing them in datastore is a really effective pattern, with a catch.
Protobuf supports some interesting ptypes for richer datatypes, such as timestamp, and duration.
These convert directly into similar verbose struct types in golang protobuf, but there exist helper utils to convert to native golang types
https://github.com/golang/protobuf/blob/master/ptypes/duration.go
https://github.com/golang/protobuf/blob/master/ptypes/timestamp.go
A proposal for similar support for structs was rejected
https://github.com/golang/protobuf/issues/370
For these relatively limited number of ptypes, it would be nice if the conversion to golang native type, and then into datastore type was handled more easily, or perhaps an sample showing a protobuf PropertyLoadSaver implementation.

I don't think the proto compiler will be changed to use native types. But writing a sample is definitely do-able.

Moved to https://github.com/GoogleCloudPlatform/google-cloud-go/wiki/Feature-Backlog.

I just store them directly in the data and it seems to work. It will create some weird fields such as XXX_cachsize etc tho..


More resources on enshrined-in-consensus

Hi! Wanted to say great work so far.
This isn't probably the best place to ask this, but...
I noticed you used the term enshrined-in-consensus. I was wondering if you could point me to some resources on this and how it relates to data availability. As data availability is a big concern of ours at Truebit.
Thanks!

Hi there! The term originated here: https://ethresear.ch/t/sharding-phase-1-spec-retired/1407/18?u=jamesray1.

Phase 1 is only enshrining-in-consensus data availability. There is no execution or dapps. The EVM state transition function will be in phase 2, and executors in phase 3. There is no sense of validity because there is no execution. With the latest research the plan in phase 1 will be that notaries try to download a collation body, vote on it, which emits a log that clients read, and if clients see than >= 2/3 of notaries voted for a chunk (see the glossary here then it will be included in the main chain.

Also you can reach us on Gitter: https://gitter.im/Drops-of-Diamond/Lobby.

PS I think I mentioned before but there is a research, Max (maxc on ethresear.ch) who is working on reducing the number of rounds of interactive verification. He would be a good person to collaborate with. I'm partially reading through research and partially reviewing development from my team at the moment. I plan on developing, and reading research as needed to stay up to date as well as for review of research.