Migrating compound extraction span link tests to weblog

Motivation
Currently, we test the results of compound extraction of inconsistent headers through parametric tests. However, using the parametric app does not truly reflect the behavior exhibited through automatic instrumentation, as we have to manually call extract on the headers passed in, create a span, and add the extracted context/span links to the span. This PR aims to migrate the parametric tests into weblog tests to remove the manual process of imitating this scenario, and have the system-tests represent what actually happens when we have compound extraction on a set of inconsistent headers.
Changes
Add 2 weblog scenarios that take in DD_TRACE_PROPAGATION_STYLE_EXTRACT with values of tracecontext -> datadog -> b3 and datadog -> tracecontext -> b3 respectively. The tests create a span through the make_distant_call endpoint and we verify that we get back a span that has the appropriate span links for different scenarios.
APMAPI-899
Workflow

⚠ Create your PR as draft ⚠
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist

[ ] If PR title starts with [<language>], double-check that only <language> is impacted by the change
[ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
[ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
[ ] A docker base image is modified?

[ ] the relevant build-XXX-image label is present

[ ] A scenario is added (or removed)?

[ ] Get a review from R&P team

Thank's for tackling this @mhlidd . If you strugle to address any comment, or if you'd like  us to do some of them, please reach us on slack !

yes it is. Checking

You may have found a real issue in this tracer, the agent does not like either the data from the lib, here is its response :
msgp: attempted to decode type \"array\" with method for \"str\" at 21/0\\n

I've sent a message on slack to the java guild, let's wait for their response

Waiting for #3587 to move forward. Once it's merged, we'll skip the test for spring-boot-native

The java error has been fixed by DataDog/dd-trace-java#8036


llm chat command

Refs:

#231

Still needs:

[ ] Tests
[ ] Docs

Tests are tricky. Need to cover:

Continuing an existing conversation
Creating a fresh conversation
Logs written to database
System prompt handling
Template handling, including params
quit/exit

I'm not going to tackle this problem in this PR, though it needs to be solved soon:

#73

Decided not to write a test for the template thing, but I tested it manually like this:
$ llm --system 'Your name is $name' --save character
$ llm -t character 'hi' -p name 'GlaDOS'
Hello! How can I assist you today as GlaDOS?
$ llm chat -t character -p name 'GlaDOS'
Chatting with gpt-3.5-turbo
Type 'exit' or 'quit' to exit
> hi
Greetings. How may I assist you?
> what is yoru name?
My name is GlaDOS. I am an artificial intelligence designed to oversee and manage testing procedures. How may I be of service to you?


bundling @angular and rxjs with RC5 makes RC6 failing

Because some dependencies have been updated as part of RC6 and ng2DatetimePicker is actually using RC5 in Npm, this module is breaking.
Easy fix is to remove the package dependency (my current workaround was just to delete node_module in ng2DatetimePicker, npm defaults then on the main node_modules rep).

We use RC6 and everything is allright.

closing. if not works, please open it again.


API picking returns no result

Hi.
The View.pick() seems to always return 0 maybe since the move to FilamentInstance renderables (1.3.0, I think)

Can you give more detail? You can test picking in the desktop gltf_viewer sample app and it works there (top left yellow label).

please reopen with more details. The api is working with our samples.

@romainguy @pixelflinger Sorry for the late answer.
I figured out that it's happening when populating the scene with the asset.getInstance() and not asset.createInstance renderables and not calling asset.popRenderable()
I don't see rendering differences when not calling asset.popRenderable() only the pick returning renderable=0

Actually, I don't really see the meaning asset.popRenderable() anymore since renderable are coming from the instance


Internal error, AssertionError: Unable to add an error, column should be a positive number but -1 given

Probable duplicate of #2145. (Didn't see that one before posting; regrets.)
Following their good lead:

Windows 10 Home, 64-bit, version 1511 (OS Build 10586.164)
Node v5.7.0
JSCS v2.11.0

No complaints; I hope this helps. Anonymized a bit. The first report on this run was for disallowCapitalizedComments (at the same location in my source). This was the second report.
internalError: Error running rule jsDoc: This is an issue with JSCS and not your codebase.
Please file an issue (with the stack trace below) at: https://github.com/jscs-dev/node-jscs/issues/new
AssertionError: Unable to add an error, `column` should be a positive number but -1 given
    at Object.Errors._validateInput (C:\Users\...\AppData\Roaming\npm\node_modules\jscs\lib\errors.js:73:9)
    at Object.Errors.add (C:\Users\...\AppData\Roaming\npm\node_modules\jscs\lib\errors.js:46:14)
    at addError (C:\Users\...\AppData\Roaming\npm\node_modules\jscs\node_modules\jscs-jsdoc\lib\rules\validate-jsdoc.js:166:28)
    at C:\Users\...\AppData\Roaming\npm\node_modules\jscs\node_modules\jscs-jsdoc\lib\rules\validate-jsdoc.js:181:25
    at Object.requireReturnTypes (C:\Users\...\AppData\Roaming\npm\node_modules\jscs\node_modules\jscs-jsdoc\lib\rules\validate-jsdoc\require-return-types.js:17:9)
    at C:\Users\...\AppData\Roaming\npm\node_modules\jscs\node_modules\jscs-jsdoc\lib\rules\validate-jsdoc.js:145:27
    at Array.forEach (native)
    at DocComment.<anonymous> (C:\Users\...\AppData\Roaming\npm\node_modules\jscs\node_modules\jscs-jsdoc\lib\rules\validate-jsdoc.js:144:46)
    at Array.forEach (native)
    at DocComment.forEachTag [as iterate] (C:\Users\...\AppData\Roaming\npm\node_modules\jscs\node_modules\jscs-jsdoc\lib\jsdoc.js:86:19) at test.js :
     1 |/*******************************************************************************
--------^
     2 | * <comment>
     3 | * <comment>

Thanks for doing this valuable and interesting project!

I am getting a similar error, and what seems to trigger it is an @returns block, which is missing its type. Funnily enough, I can put this on any JSDoc comment in the tab, and it will trigger this error.
e.g.
/**
 * <summary>
 *
 * @return <comment>

(Not @return {type} <comment>)

Should work with 3.x

/cc @zxqfox

Huh, we definitely missing tests for jsdoc... ;-(

So is this still valid?

Guess, yes. I'll try to look at it tomorrow.

At this point only major and CST related bugs will be fixed.
You however open ticket on https://github.com/jscs-dev/jscs-jsdoc and keep pinging @zxqfox


Last cell not loading?

i tried everything, I used the latest version,  reinstalled again, checked if i had vram, what am i doing wrong?

apologies, i should've checked first
https://github.com/TheLastBen/fast-stable-diffusion/issues/889\
i also got it load with no errors but now its not even showing the gradio link or anything, its been loading for almost 30 minutes

try removing the extensions and try again

it's still not showing anything :/

ran for 20 minutes and still nothing

run the notebook without the first cell and see if it works

well now i can open the web ui, but it wont load any of the models,  or any other file ifrom my googdrive folders :/

yeah, just double checked, its not loading my files because it didn't run the first cell

let the collab rebuild it and move the files i want from the old model folders into the new install

this is not working for me because it is not connected to gdrive, and the folders are not appearing in gdrive

Alright i just tested it, pcrii, your method worked when i ran it with the first cell, it works now with the drive mounted(i still had to move my files into the reinstalled folders, but that's fine)
thanks @pcrii @TheLastBen
have a great day/night!


Jwt_authn config update: Support per route Jwt requirements

Signed-off-by: Wayne Zhang<EMAIL_ADDRESS>Description:
Based on the requirement discussion from https://github.com/envoyproxy/envoy/issues/2514.
Change the Jwt_authn config to support different requirement based on route match.
Risk Level: Low

@llchan please help to review this config change to see if it works for your cases.
@diemtvu, @lizan  Please help to review this change.

@qiwzhang This works for me, but will you also include the ability to configure at the route level with per-route-filters as discussed on the issue thread? I can see other people wanting that. If we support both it should cover a pretty wide variety of setups.

Not sure how to comment on parts of the file that aren't part of the diff so I'll do it here.
I think the issuer/audience definitions (aka JwtRule currently) need an additional user-specified key/id that is used when specifying the requirements. The issuer url/email alone may not be flexible enough because the audience is also important for validating a token. For example, if I have one issuer https://issuer.com that issues tokens for two audiences foo.com and bar.com, those are two different specs and the user should be able to refer to them independently in the rules.

I added "audiences" in JwtRequirement to override the one in the JwtProvider.  You can set different audiences in the per-route requirement.

@llchan, @lizan, @htuch  PTAL.   I like to get this in.  Thanks

@llchan could you approve this PR?
@lizan @htuch could you PTAL?

OK,  I added 2D  "or_jwt" and "and_jwt" in the JwtRequiement.    PATL

Format failure is from someone else files.
ERROR: clang-format check failed for file: ./api/envoy/config/grpc_credentials/v2alpha/file_based_metadata.proto

@qiwzhang https://github.com/envoyproxy/envoy/pull/3517 should fix the format ci, which is broken on master now.

Overall I think this is reasonable. It feels a bit verbose and seem to be overreaching a bit into authz, but at least it's pretty clear what it's doing.
That said, I'm not 100% sold on it, so I'll suggest another option that isn't quite as flexible, but is far more concise for the common use case while supporting simple AND-ed requirements:
- rules:
  - match: { prefix: "/foo" }
    requires_any: [{ issuer: "iss-A" }, { issuer: "iss-B" }]
  - match: { prefix: "/bar" }
    requires_all: [{ issuer: "iss-A" }, { issuer: "iss-B" }]

Compare that to the equivalent one_of/all_of config:
- rules:
  - match: { prefix: "/foo" }
    requires: { one_of: [{ all_of: [{ issuer: "iss-A" }] }, { all_of: [{ issuer: "iss-B" }] }] }
  - match: { prefix: "/bar" }
    requires: { one_of: [{ all_of: [{ issuer: "iss-A" }, { issuer: "iss-B" }] }] }

This should be doable with a simple oneof between requires_any and requires_all. Or requires_one if that's better.
FWIW I'm also in favor of what @lizan suggested: drop AND-ed requirement support at this layer and just use a simple OR requirement, but validate all JWTs presented. That would certainly suffice for me, but since I'm not familiar with the multi-JWT use case I'll defer that decision to you guys.

@llchan  I like your idea.  I kept "requires_any" and "requiers_all" to support multiple JWT requirements.
Unfortunately,   proto oneof type doesn't allow repeated field.   So "requires_any" and "requiers_all" has to be in this format:
requires_any: { list: [issuer: "issuer1"], [issuer: "issuer2"] }

It feels weird, but they are rarely used. It should be OK.
PTAL.  If it is OK, please approve it.

Last SHA (83b4aa3) made breaking change. It changed "repeated providers" to map<string, provider>.  It will not be backward compatible with older version Istio is using.
Action items for Istio is: to move older version of this config.proto back to Istio/api repo and made Istio/proxy and istio/istio use that version.  This will allow Envoy Jwt_authn config to make breaking change.

Yes. No problem. I'll take a look tomorrow.
On Thu, 14 Jun 2018, 19:26 Wayne Zhang<EMAIL_ADDRESS>wrote:

@nickrmc83 https://github.com/nickrmc83, could you help to review this
jwt_authn file config change to see if it fixes with your use cases?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoy/pull/3381#issuecomment-397393063,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABhXUBFwPCGKT-X6AqZz8jJ5VBTvIhghks5t8qrVgaJpZM4T-pE-
.

@nickrmc83 . It seems that we agreed that your user case is so special, we should not adjust jwt_authn config for that.
You may raise a separate PR for the Authenticator class code change to separate verification with token forwarding

@lizan I addressed your comment of moving "audiences" to RequiredProviider by adding a new field in JwtRequirement onof field:  "provider_with_audiences".   Users can config audiences in JwtProvider, they can also override them in JwtRequirement.

@lizan @htuch  This PR is ready for review.  This PR will just get the config change in first. Its added "JwtRequirement" feature is not implemented yet.  I will use a separate PR to implement it.

Hello guys,
I would like to support multi-tenancy on our microservice. Once the user is authenticated, my JWT will contains a field  like tenant_id: 12323232 . I would like to route the request base on that jwt field. Do you think it will work on what you have done ?

@atali can you open an issue with more details of your use case? Currently it is not possible to route request based on JWT claims (this PR is reverse, route-based JWT configuration).

Is this still being worked on? If not, I would like to help.

@potatop could you be specific on which feature you like to help?
for routing based on jwt fileds,  I am not aware of anybody is working on it.
for supporting per-route requirement: config proto is in,  but implementation did not start yet.
Well, contribution to any of above is welcomed.

Yes, I was talking about the RequirementRule functionality.

yes, it is almost the same, but there are different; the former will not do any jwt verification,  the latter will verify all jwts in the request, but allow fails and missing.

if no requirements matched a path, it will be the same as empty JwtRequirement; not any jwt verification.

@potatop can you open a new issue for further questions/feature requests?


Dynamic NFT with Morty

I'm using Morty images from the Rick and Morty API

https://testnets.opensea.io/collection/morty-v3


README: clarify intro, fix links

@campoy There's a message in master README at the beginning:

master branch is deprecated, please go to v1 branch

with a link to v1 branch. Turns out that we cannot put v1 in master because it would break all our projects that are not migrated to v1 yet, since we just used master before from the builds.


[Community] Broken image link to MeshMate logo

Description
In the meshery.io/community pages, there is a broken link to the MeshMates logo. See example here - https://meshery.io/contributors/subham-rai/
Screenshots

Contributor Guides and Handbook
The meshery.io website uses Jekyll and GitHub Pages. Site content is found under the master branch.

📚 See the Contributing to Meshery.io Website section of the readme.md and other contributing instructions, too.
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

This needs fixed in the readme, too:

@leecalcote Could you please assign this issue to me. Thanks!

Hello sir. Recently when I am creating my portfolio I also fave thus issue and here is my portfolio link: https://codewithnadeem14502.github.io/portfolioo/ you see my work. So I think I fix this in much better. So could you please assign me this .

Hi @leecalcote, I have fixed this issue and have raised a PR. Can you please review it, Thanks.

PS: I didn't work on fixing the logo in README issue as there is an approved PR for it

PS: I didn't work on fixing the logo in README issue as there's already an approved PR for it

Update: Included README file changes as well!

This needs fixed in the readme, too:

This issue has been fixed in the PR #1100

Ok, I saw the PR wasn't merged and it had been mentioned by @leecalcote again in this issue so I included it. Does it mean I'd have to exclude the readme file changes ?

The PR for this issue has been approved and merged to master. You can consider closing this issue @leecalcote

done by #1109


fix: The ‘defaultTest’ option has no effect during evaluation.

The document at ‘https://promptfoo.dev/docs/configuration/expected-outputs/model-graded’ describes how to configure the ‘defaultTest’ option, but it has not been posted yet.

Thank you for fixing this!


Create a footer white-3

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

After building the component please raise a PR with a screenshot of the component and add the component in Footer/footer-white-3.html.
If you need to use any icon please use it from Hero Icons
Good luck.

Will you please assign this issue to me?

Go ahead @issue31


[Plasma][C++] plasma-store-server start with -h , occur bus error

use plasma as a shared-memory ,
start with these args:
./plasma-store-server -m 1000000 -s /tmp/test -d /dev/shm -h
occurs bus error
/home/arrow/arrow/cpp/src/plasma/store.cc:1277: Allowing the Plasma store to use up to 0.001GB of memory.
/home/arrow/arrow/cpp/src/plasma/store.cc:1300: Starting object store with directory /dev/shm and huge page support enabled
/home/arrow/arrow/cpp/src/plasma/store.cc:1328:  hugepage enable , SetMallocGranularity.
/home/arrow/arrow/cpp/src/plasma/store.cc:1349: starting server listening on /tmp/test
/home/arrow/arrow/cpp/src/plasma/store.cc:1154: new PlasmaStore...
/home/arrow/arrow/cpp/src/plasma/store.cc:1163: Memalign...pointer
/home/arrow/arrow/cpp/src/plasma/plasma_allocator.cc:34: Memalign,alignment=64;bytes=997952
internal memalign
internal malloc req = 998056,nb = 997968,min_chunk_size = 32,chunk_overhead = 8
Bus error

env:
[root@02a40e9b75b6 release]# free
              total        used        free      shared  buff/cache   available
Mem:       12979512     6299224     1163552      369948     5516736     5990072
Swap:       4194304       16480     4177824
[root@02a40e9b75b6 release]# uname -a
Linux 02a40e9b75b6 <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Fri Apr 2 22:23:49 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Environment: [root@02a40e9b75b6 release]# free
total        used        free      shared  buff/cache   available
Mem:       12979512     6299224     1163552      369948     5516736     5990072
Swap:       4194304       16480     4177824
[root@02a40e9b75b6 release]# uname -a
Linux 02a40e9b75b6 <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Fri Apr 2 22:23:49 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Reporter: xingwang li
Note: This issue was originally created as ARROW-18000. Please see the migration documentation for further details.

Plasma has been deprecated (https://github.com/apache/arrow/issues/33077) in Arrow 10.0 and subsequently removed in Arrow 12.0 (https://github.com/apache/arrow/issues/33243). Given Plasma is no longer an existing component, closing this issue.
For reference, see this mailing list discussion around possible alternatives: https://lists.apache.org/thread/lk277x3b9gjol42sjg27bst2ggm5s0j2


facing this issue and cause stuck threads

"GC Daemon" #13 daemon prio=2 os_prio=0 tid=0x00007fd2a859f000 nid=0x955 in Object.wait() [0x00007fd290659000]
java.lang.Thread.State: TIMED_WAITING (on object monitor)
at java.lang.Object.wait(Native Method)
- waiting on <0x000000070006f990> (a sun.misc.GC$LatencyLock)
at sun.misc.GC$Daemon.run(GC.java:117)
- locked <0x000000070006f990> (a sun.misc.GC$LatencyLock)
"vfabric-license-heartbeat" #12 daemon prio=5 os_prio=0 tid=0x00007fd2a850b800 nid=0x94e in Object.wait() [0x00007fd290b73000]
java.lang.Thread.State: TIMED_WAITING (on object monitor)
at java.lang.Object.wait(Native Method)
- waiting on <0x000000070006fb20> (a java.util.TaskQueue)
at java.util.TimerThread.mainLoop(Timer.java:552)
- locked <0x000000070006fb20> (a java.util.TaskQueue)
at java.util.TimerThread.run(Timer.java:505)
Please suggest on these thread dumps.

I don't understand your issue.

What thread dump did you analyze?
What did the Thread Dump Analyzer show when you did that?
What did you expect the Thread Dump Analyzer to show instead?

Note that if this your Java app misbehaving you should contact your Java app vendor.


Server crash when killing entities from other mods with Essence Thief enchantment.

I was playing in my 1.18.2 modded server, with a sword with several enchantments on it, when the server crashed and brought me to the multiplayer screen, after killing a mob from the mod Evilcraft. After some more testing, the same happened with entities from the mods Ars Nouveau and Twilight Forest, all three with a Ticking entity error, reading "java.lang.NullPointerException: Cannot invoke "com.verdantartifice.primalmagick.common.sources.SourceList.isEmpty()" because "affinities" is null", which I believe is caused by the Essence Thief enchantment, since the crash won't happen if I use the same swords with the same enchantments except for that one.

This issue should be fixed in the next release, currently planned as v3.2.0.  Thank you for the bug report!


[FEATURE] Show season/episode list instead of search

Describe the feature/change you'd like
For shows with multiple episodes it would be great to be sent to a menu with options such as next episode, previous episode, or replay current instead of being sent back to search after each episode. Even just being sent to the seasons or episode page instead of the search would be amazing.
Additional context
ani-cli (https://github.com/pystardust/ani-cli) has a great way of doing this.

I totaly agree on this, love the way ani-cli doing this... also they have a arg option like "-r 1-100" where it binch watches every episode, besacly start playing episode 1, and when u finnish watching it, it would alrdey scrape and pre prepare episode 2 so that as soon as mpv closes, it launches the next episode, up to 100 if there are that many episode, if I type ( -r "5-32" ) it will start playing episode 5 and then binch it or auto play the next one untill it reaches 32. Or u can type ( -e 4 ) to automaticaly play episode 4, but it wont auto play next episode, instead it returns u to a menue where i can select "next", "prev", "replay" etc.

This will be added. This is the issue you should track --> #229


Create minitests for 7-series DSP

There are fuzzers 100-dsp-mskpat and 101-dsp-pips that target Xilinx 7-series DSP tiles. They should already provide solution for all routing and most of DSP configuration bits. There is an open question whether this is sufficient for adding support for DSPs in f4pga toolchain even in plain multiplication mode.
This issue is about creating a number of simple "minitests". A minitest is a simple design that focuses on using a particular primitive in a meaningful way. Its purpose is to obtain a valid binary bitstream from Vendor tools so that it can be disassembled using open source tools. The goal is to verify if all bits / features that are enabled in the bitstream are properly documented by project X-ray. In another words: "are there any unknown bits".
Initially minitests for DSP should use it as pure multiplier. Once they are complete, subsequent ones may eg. use DSP input / output internal registers. All of them should allow to answer the question: "do we know enough bits to support DSP in F4PGA".
There are existing minitests for other FPGA resources already present under the minitest folder. Those can be used as a reference.

I wanna add minitests about dsp. Is there any suggestions or branches ?


198 cms show language of unsaved changes

Add in validation area.
Unsaved changes to the Post / Tag (parent) document to show in General
Unsaved changes to ContentDto's (child documents) to show per ContentDto language
Rename "Validation errors" to "Validation"
Remove current "Unsaved Changes" label.
Use a yellow "info" icon for unsaved changes

Would also be nice to have some tests to check that the unsaved changes validation is showing as expected.

This is what i did on validations

Hey @ivanslabbert !
There is something that we should have a look at.
Actually, the languages are sorted on EditContent for categories but not in the post. I tried what you said but it didn't work.
I think we need to look a bit at it. Normally it should work everywhere as it is a Generic component.


Save non-ASCII characters

How to save non-ASCII characters? I need to save german and french letters like ö,ü or é.
When I try to save strings with these chars I get error:
Unhandled Exception: HiveError: String contains non-ASCII characters.

You can save non-ASCII characters as value. Keys need to be ASCII.

@lurcas did it work?

Yes it was just the keys. Maybe you should mention that in the docs.
Thanks for the quick help :)

https://docs.hivedb.dev/more/limitations

ahh sorry, my fault


[BUG] onHover not triggered for doughnut chart

The function provided on the onHover option is not being triggered for a doughnut chart. I don't know if this is also affecting other types of chart.
The issue was already reported in #3799, but was closed because nobody presented a test case.
Expected Behavior
The provided function should be called when the mouse hovers a segment of the chart.
Current Behavior
Nothing happens on hover, but the onClick event is working.
Steps to Reproduce (for bugs)
When hovering over on a segment, the tooltip shows but the onHover function is not called.
When clicking a segment, the onClick function is called.
Codepen: https://codepen.io/anon/pen/ZKNGEQ
Context
I was trying to expand (increase outerRadius) a slice of the chart when hovering. If necessary, I can provide a gist with the expansion working for the onClick event.
Environment

Chart.js version: 2.6.0
Browser name and version: Chrome 58.0.3029.110 (64-bit)
Operating System: macOS Sierra 10.12.5

@ricardocosta89 thanks for providing a test case. I quickly looked into this and the cause is that https://github.com/chartjs/Chart.js/blob/master/src/core/core.controller.js#L812-L815 should use options.onHover not hoverOptions.onHover.
I'm happy to look at a PR fixing this. The docs already list that onHover should be at the top level so no updates there are needed

Before we fix this we should make sure that it's the code that's wrong and not the docs. We should look at v2.0.0 and see where the onHover option was placed then and match that since that is the major version

I went back to v2.0.0 and it's always been
options: {
  hover: {
    onHover: function() {

    }
  }
}

So that means we need to update the docs.

Oh bummer. I just submitted the PR fixing the code. 😅

No worries, just change it to update the docs instead 😄

Taking into account that the onClick is provided at the top level, how do you feel about something like this?
var hoverCallback = options.onHover || options.hover.onHover;
// On Hover hook
if (hoverCallback) {
  // Need to call with native event here to not break backwards compatibility
  hoverCallback.call(me, e.native, me.active);
}

It won't introduce breaking changes, but it will keep consistency regarding the onClick.

@ricardocosta89 just proposed that on the PR 😄 Just waiting to see what @simonbrunel thinks but I like that idea since it's cleaner. I would go ahead and make the code change!

I'll wait for feedback, then.
Also, one thing I just noticed is that the callback is invoked for every mouse movement, so we have a lot of calls despite the segment only being hovered once from the user perspective. Is there a way to prevent this?

I believe there is an open issue for that. It really depends on what onHover is being used for. If the raw mouse coordinates are needed then it needs to be called all the time. If the user only cares about the item being hovered it only needs to be called when the hover selection changes. I don't know if there's enough consensus to make a change in the Chart.js internals for this but it could definitely be a new sample that demonstrates how to filter the events

In my case I just wanted to increase the outerRadius, so it will be the second approach (hover selection change). I'll try to see if there's anything stating that the hover is on/off to use in my callback.

We've already written the intersection code. When onHover is called you get the list of internal items that are hovered. You can use that list to determine if the selection has changed

Yes, I managed to do it tinkering around with Chart.js code. I'll try to do it on my side without exposing the lastActive when calling the callback.

I agree, for consistency I prefer having onHover at the options root (similar to onClick). I like var callback = options.onHover || options.hover.onHover; and I would deprecate hover.onHover, which basically means to not update the docs :)

@simonbrunel Another question I have: is options.hover.animationDuration deprecated? I don't remember seeing it in the docs, but there are still references in the code. And this duration is not being respected because the code enters the bufferedRequest branch (core.controller.js#L774).

I don't think it's deprecated, the bufferedRequest branch is entered only if there is already a render in progress, which one should have been requested with options.hover.animationDuration (core.controller.js#L781).

I expected that, but when trying out with hover for the doughnut chart, it was always entering the bufferedRequest branch. Some underlying bug?

I think it's always entering the bufferedRequest branch since something called update() during the event handlers

Ah, indeed. I have to call _chart.update() on my onHover callback for the animation to work. Should I open a new issue about this?

Yup, I think that's a new issue


Indirect recursive call counting varies with function name

Per Slack convo:
When run with: npx elm-review --template jfmengels/elm-review-cognitive-complexity/preview:
This commit fails:  https://github.com/SiriusStarr/elm-review-no-unsorted/commit/5ee691d4c626e36d5cf078ef321c7499d062e1d2
This commit passes:  https://github.com/SiriusStarr/elm-review-no-unsorted/commit/a6a1f8eb45b7e9ef40d8267e8bf5842d686f5576
This seems to be due to more indirect recursive calls being detected in the former case.

Fixed by #6


"ip" module needed for node-red-contrib-homekit-bridged" palette where to install?

Describe Your Problem:

Hi Helander,
would it be possible to use the the "ip" module with this version of node-red? I tried to use the "node-red-contrib-homekit-bridged" palette, but it is missing the "ip" module, when installing node-red parallel to homebridge I could use "npm install ip" inside the "~/.node-red" directory but I don't know where to install this "inside" your plugin.
Logs:
Error: Cannot find module 'ip'
Require stack:

/var/lib/homebridge/nodered-1956/node_modules/node-red-contrib-homekit-bridged/build/lib/cameraSource/index.js
/var/lib/homebridge/nodered-1956/node_modules/node-red-contrib-homekit-bridged/build/lib/utils/ServiceUtils.js
/var/lib/homebridge/nodered-1956/node_modules/node-red-contrib-homekit-bridged/build/lib/HAPServiceNode.js
/var/lib/homebridge/nodered-1956/node_modules/node-red-contrib-homekit-bridged/build/nodes/service.js
/var/lib/homebridge/node_modules/<EMAIL_ADDRESS>/var/lib/homebridge/node_modules/<EMAIL_ADDRESS>/var/lib/homebridge/node_modules/<EMAIL_ADDRESS>/var/lib/homebridge/node_modules/<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>/var/lib/homebridge/node_modules/homebridge/lib/plugin.js
/var/lib/homebridge/node_modules/homebridge/lib/pluginManager.js
/var/lib/homebridge/node_modules/homebridge/lib/server.js
/var/lib/homebridge/node_modules/homebridge/lib/cli.js
/var/lib/homebridge/node_modules/homebridge/bin/homebridge```
Show the Homebridge logs here, remove any sensitive information.

**Plugin Config:**

```json
Show your Homebridge config.json here, remove any sensitive information.

Screenshots:

Environment:

Plugin Version:
Homebridge Version:
Node.js Version:
NPM Version:
Operating System:

Your help is highly appreciated.

Hi, do the installation in /var/lib/homebridge/nodered-1956


Drop Menu For Flashcards

Drop Menu For Flashcards

Issue: Currently the user must navigate to the bottom of the home page in order to explore the flashcards. This causes poor navigation of the site and new users may not be able to find the flashcards or become disinterested.

Solution: Add a dropdown menu in the navigation bar that lists all the categories a user can select for studying.

Dropdown menu has been added.


otLinkSetShortAddress is in link.h, while its implementation is in src/core/api/link_raw_api.c

This gives build issues since we do not enable OPENTHREAD_ENABLE_RAW_LINK_API

@thcu-gp, thanks for raising this issue. I submitted #3348 to resolve this issue. Please review.


Pretty print caveats output

Example:
Installing plugin: ca-cert
CAVEATS:
\
 |  This plugin needs the following programs:
 |  * base64
/
Installed plugin: ca-cert
Installing plugin: debug-shell
Installed plugin: debug-shell
Installing plugin: exec-all
CAVEATS:
\
 |  Usage:
 |    kubectl plugin exec-all <RESOURCE_TYPE> <RESOURCE_NAME> -- <COMMAND>
 |  This plugin needs the following programs:
 |  * jq
 |  * xargs
 |  * bash
/
Installed plugin: exec-all
Installing plugin: gke-credentials
CAVEATS:
\
 |  This plugin needs the following programs:
 |  * gcloud
/
Installed plugin: gke-credentials
Installing plugin: mtail
CAVEATS:
\
 |  This plugin needs the following programs:
 |  * jq
/
Installed plugin: mtail
Installing plugin: node-admin
Installed plugin: node-admin

Codecov Report

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

@@           Coverage Diff           @@
##           master      #99   +/-   ##
=======================================
  Coverage   47.32%   47.32%
=======================================
  Files          12       12
  Lines         598      598
=======================================
  Hits          283      283
  Misses        268      268
  Partials       47       47

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 7225efd...e40d255. Read the comment docs.


Bug fix for code gen and type equality for refinements

Fixes schema 2 graph modeling of type equality.
[{field: Type}[f(field)]] being considered eq to
[{field: Type}[g(field)]] where f and g can't be rewritten to one another.

I'm wondering if there's some work that we should do (now or later) to ensure that a refined handle and unrefined handle should share a base class in KT, but I'm not sure about the semantics of companion objects in KT (i.e. can a subclass have an overridden version of SCHEMA without breaking the parent class?).
Adding @mykmartin for review of schema2graph
@shans for review of schema changes
@jasonwyatt for review of kotlin code
Thanks

@csilvestrini could you look at Kotlin changes? (@Cypher1 jwf is away for a bit)

/gcbrun

Copybara submit to 302819622

/gcbrun
^ Re-running due to timeout in wasm tests.
302819622 has completed successfully


Some entities unavailable

Hi
I have some entities that always appear as unavailable. Any way to solve this?
I do have the data available in multimatic app

Hello,
Do you have any errors in the log ?

I get this message

These are the logs I can find but not sure if relevant
This error originated from a custom integration.
Logger: custom_components.multimatic.coordinator
Source: custom_components/multimatic/coordinator.py:94
Integration: Multimatic (documentation, issues)
First occurred: June 11, 2022, 4:00:49 PM (34 occurrences)
Last logged: 9:26:49 PM
Unexpected error fetching multimatic_zones data: None
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 191, in _async_refresh
self.data = await self._async_update_data()
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 150, in _async_update_data
return await self.update_method()
File "/config/custom_components/multimatic/coordinator.py", line 533, in _fetch_data_if_needed
return await self._fetch_data()
File "/config/custom_components/multimatic/coordinator.py", line 526, in _fetch_data
return await getattr(self.api, self._method)()
File "/config/custom_components/multimatic/coordinator.py", line 94, in get_zones
return await self._manager.get_zones()
File "/usr/local/lib/python3.9/site-packages/pymultimatic/systemmanager.py", line 43, in wrapper
return await func(*args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/pymultimatic/systemmanager.py", line 362, in get_zones
return mapper.map_zones(rooms)
File "/usr/local/lib/python3.9/site-packages/pymultimatic/model/mapper.py", line 265, in map_zones
return _map_zones(json.get("body", []))
File "/usr/local/lib/python3.9/site-packages/pymultimatic/model/mapper.py", line 277, in _map_zones
zone = map_zone(raw_zone)
File "/usr/local/lib/python3.9/site-packages/pymultimatic/model/mapper.py", line 306, in map_zone
func = _map_function(raw_heating, "setting", rbr, quick_veto is not None)
File "/usr/local/lib/python3.9/site-packages/pymultimatic/model/mapper.py", line 371, in _map_function
operating_mode = OperatingModes.get(mode)
File "/usr/local/lib/python3.9/site-packages/pymultimatic/model/mode.py", line 82, in get
return cls._VALUES[name]
KeyError: None
Logger: homeassistant.core
Source: core.py:1583
First occurred: 9:24:47 PM (2 occurrences)
Last logged: 9:24:47 PM
Unable to remove unknown service multimatic/set_ventilation_night_level
Unable to remove unknown service multimatic/set_ventilation_day_level
This error originated from a custom integration.
Logger: custom_components.multimatic.coordinator
Source: helpers/update_coordinator.py:204
Integration: Multimatic (documentation, issues)
First occurred: June 13, 2022, 6:03:26 PM (27 occurrences)
Last logged: 8:29:32 PM
Error requesting multimatic_zones data: [Errno 104] Connection reset by peer
Error requesting multimatic_hvac_status data: [Errno 104] Connection reset by peer
Error requesting multimatic_quick_mode data: [Errno 104] Connection reset by peer
Error requesting multimatic_dhw data: [Errno 104] Connection reset by peer
Error requesting multimatic_quick_mode data: Cannot connect to host smart.vaillant.com:443 ssl:default [Connect call failed ('<IP_ADDRESS>', 443)]

This is something I can't do about it :/ the server is quite unstable at vaillant side.


Inference AUROC on valid set using ModelCheckpoint saved ckpt does not equal valid AUROC from training

🐛 Bug
During training, I have a ModelCheckpoint callback that saves the top 5 models based on valid_AUC computed at the end of validation phase using multiclass_auroc() function. The callback saves the .ckpt file as epoch=X_valid_AUC=0.XXXX.ckpt. When I load the ckpt and run trainer.test() on the same validation set used in training, it returns a different value (see screenshot below). I decided to test if it would work using 1 GPU in ddp, and it works in that case. When using 4 GPUs it does not work.
My validation set size is 543 examples, but in my validation_epoch_end function I use self.log("valid_preds_size", preds.shape[0]) and it outputs 136 which is approx 543/4. Shouldn't it output 543 because validation_epoch_end has gathered all the batch dicts from every GPU? I create my preds tensor like this:
preds = torch.cat(                 [step[head.name]["preds"] for step in valid_outputs if head.name in step]   )

CUDA:

GPU:

Tesla V100-SXM2-16GB
Tesla V100-SXM2-16GB
Tesla V100-SXM2-16GB
Tesla V100-SXM2-16GB

available:         True
version:           10.1

Packages:

numpy:             1.18.1
pyTorch_debug:     False
pyTorch_version:   1.7.1+cu101
pytorch-lightning: 1.1.1
tqdm:              4.46.1

System:

OS:                Linux
architecture:

64bit

processor:
python:            3.7.6
version:           #1 SMP Debian 4.9.210-1+deb9u1 (2020-06-07)

Additional context

This is fixed with v1.2, closing!


Store path of CLDF metadata relative to pylingdocs.cfg

I was running pylingdocs new to create a new project, then cd-ed into the project and ran pylingdocs preview - which apparently picked up the project's pylingdocs.cfg successfully, but failed looking up the CLDF data. So I guess the path given for the cldf option in pylingdocs.cfg should be relative to the config file.

I was not able to reproduce this.

$ pylingdocs new
INFO    Loading configuration
INFO    Loading templates
name [new-pld-document]: tt
...
$ cd tt
$ pylingdocs preview
INFO    Loading configuration
INFO    Loading templates
INFO    Rendering preview
ERROR   [Errno 2] No such file or directory: 'lgr-cldf/cldf/Generic-metadata.json'
ERROR   Could not load CLDF dataset from /home/robert_forkel/projects/pylingdocs/tt/lgr-cldf/cldf/Generic-metadata.json. Please specify a path to a valid CLDF metadata file.

Content of pylingdocs.cfg is
[PATHS]
cldf = lgr-cldf/cldf/Generic-metadata.json

[latex]
template = article

but would work with cldf = ../lgr-cldf/cldf/Generic-metadata.json, i.e. relative to the location of pylingdocs.cfg.

I don't understand -- yes, the path to the cldf dataset is relative to the location of pylingdocs.cfg. What should be different?

It is not relative to pylingdocs.cfg, but relative to the directory in which pylingdocs new was run.

Thus, changing cwd between pylingdocs new and pylingdocs preview causes the error I'm seeing.

For context:
$ la
lgr-cldf
pylingdocs
tt

It is not relative to pylingdocs.cfg, but relative to the directory in which pylingdocs new was run.

OK, so is the issue that at the pylingdocs new stage you entered a path relative to the current directory, but then pylingdocs actually expects a path relative to the created project directory (../bla/bla/bla.json) ? If yes, what behavior would you expect / recommend? If no, I am still not understanding.

Well, at "pylingdocs new" I'm entering an existing path, because that's
supported via tab-completion in the shell. So this path is relative to cwd.
Since pylingdocs knows, where the cfg file will be created, I'd expect
pylingdocs to adapt the path I entered accordingly.
Florian Matter @.***> schrieb am Mo., 19. Sep. 2022,
19:57:

It is not relative to pylingdocs.cfg, but relative to the directory in
which pylingdocs new was run.
OK, so is the issue that at the pylingdocs new stage you entered a path
relative to the current directory, but then pylingdocs actually expects a
path relative to the created project directory (../bla/bla/bla.json) ? If
yes, what behavior would you expect / recommend? If no, I am still not
understanding.
—
Reply to this email directly, view it on GitHub
https://github.com/fmatter/pylingdocs/issues/4#issuecomment-1251357439,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGUOKFQOQW346TSKTVO4ULV7CSQLANCNFSM56YIUIVQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Well, at "pylingdocs new" I'm entering an existing path, because that's supported via tab-completion in the shell. So this path is relative to cwd. Since pylingdocs knows, where the cfg file will be created, I'd expect pylingdocs to adapt the path I entered accordingly.

Right, I understand now! I don't have tab-completion in that context, interesting.

Ah, right, I don't have either. But I copy a path I got via tab-completion.
But maybe the better argument is that paths existing in cwd can be simpler
checked for existence - and would also be simpler to document in help.
Florian Matter @.***> schrieb am Mo., 19. Sep. 2022,
20:37:

Well, at "pylingdocs new" I'm entering an existing path, because that's
supported via tab-completion in the shell. So this path is relative to cwd.
Since pylingdocs knows, where the cfg file will be created, I'd expect
pylingdocs to adapt the path I entered accordingly.
Right, I understand now! I don't have tab-completion in that context,
interesting.
—
Reply to this email directly, view it on GitHub
https://github.com/fmatter/pylingdocs/issues/4#issuecomment-1251401385,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGUOKCWVZHLXIMASQTZR73V7CXHHANCNFSM56YIUIVQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


TimePickerDialog should dispose created ChangeNotifiers.

Description

Fixes https://github.com/flutter/flutter/issues/136260,

Tests

Updates material/time_picker_test.dart to use testWidgetsWithLeakTracking.

Pre-launch Checklist

[X] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[X] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[X] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[X] I signed the [CLA].
[X] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making, or this PR is [test-exempt].
[X] All existing and new tests are passing.

cc @polina-c

@polina-c, thanks for the review!


autoconf: Update autoconf image file to use Ubuntu distro

Port autoconf image from alpine distro to Ubuntu distro.

Hmm https://ftp.gnu.org/gnu/autoconf/autoconf-2.69.tar.gz is unstable.

@erlend-aasland PTAL CI failure
https://github.com/python/cpython-devcontainers/actions/runs/11136086934/job/30947270766?pr=13

@erlend-aasland gentle ping~


How to make flexx app installer?

Hi, learning flexx and loving it. Great work!. Is it posible to create an installer for a flexx app? I haven't found anything about it on the documentation. Do you have any advise about achieving this?

This is covered by #433; I've not put any effort in it yet ...


CustomCommandReplacementMain

Command breaks on some searches

Huh weird, that should not happen.
I am gonna try to reproduce it this week and see if i can find the issue.

I tried it with a few different commands and checked some thing, variable $m is just empty [] which is why it errors out. That's the loop that grabs multiple matching commands, and instead of grabbing something, it seems to grabbing nothing. From what I can tell. Regardless I don't see why the command number would resort to that loop.
It's occurring with a good number of commands. It seems to be commands that are over 2000 characters, which should just output a file, which some commands of mine that are too long are doing, just many of them aren't. One of the commands that isn't displaying is the reaction portion to the CustomCommandReplacement, so that might be a way you can replicate the error.
I hope this helps!

In case it's a command number issue, here's the list of numbers that all don't work. All varying types, cmd, reaction, regex.
20,48,50,65,66,68,77,96,97,103,104,106,108,122,129,130,131,132,133,135,137
Figure the more information you have, the better. But I don't think this part is relevant

I looked a little more into the issue:
I hope my gibberish makes sense
With commands that do attach a file when the output is too long, it checks to see if the "code block" it sends is more than 2000 characters, that works.
However, with the commands that it's failing on, there's no "code block" at all, just the file, so I thought it might be as simple as an "or" statement, {{if or "code block" "file"}} (not proper syntax I know), go to the file loop
But the problem with that solution is when {{$cc := exec "cc" $trigger}} happens, the "file only" situations don't return anything, so then $cc actually has no value. Which then leaves $m empty. It's really a flaw, a call with only a file seems to return empty. It seems like it might be an issue with the bot itself rather than the custom command.


storage location

Hi,
What is the storage location for upload files?

For osfstorage, correct?  The default for that is the /website/osfstoragecache/ directory in your OSF repo.  You can change that here: https://github.com/CenterForOpenScience/osf.io/blob/847824cf86ddd7403a9b18b1ff3278cd8399ab63/website/addons/osfstorage/settings/defaults.py#L15


:truck: update: Add 142nd CatchUp summary

Sorry for the delay in writing this summary! Please review and merge! Thank you!

Reason for merging without review: Team is very busy and the summary is straightforward.


feat: faucet

Resolves https://github.com/ubiquity/pay.ubq.fi/issues/142

You should always link the pull request to the open issue. If its in a different repository, then you paste in the full URL after the "Resolves" keyword.

@Keyrxng

Pls review https://github.com/ubq-testing/faucet/pull/1 and merge if it's fine
I can't run the worker locally via yarn start because I get the Worker exceeded resource limits error all the time. Do you have such an issue?

Sorry for the slow reply been a busy week, trying to scale out of my 9-5 at the minute.
I'll check things this evening.
As for 2, yeah I was able to fire maybe 3 transactions locally before I hit that same issue, restarting the server gave me another few attempts. A side-effect of the local env I assumed, my actual worker doesn't have any such issues.

Sorry for the slow reply been a busy week, trying to scale out of my 9-5 at the minute.
I'll check things this evening.
As for 2, yeah I was able to fire maybe 3 transactions locally before I hit that same issue, restarting the server gave me another few attempts. A side-effect of the local env I assumed, my actual worker doesn't have any such issues.

You mean there is no Worker exceeded resource limits error on a prod worker deployed to cloudflare, right?


MQTT connect loop.

BFUS, 9.0.19*, y21ga flashed
MQTT flood with connected message.
I've tried running the mqttv4 on the camera via ssh:
Starting mqttv4 v0.0.3
key: SOUND_DETECTION | value: sound
Unrecognized config.
Trying to connect... Client yi-cam-3 sending CONNECT
Client yi-cam-3 received CONNACK (0)
Client yi-cam-3 sending PUBLISH (d0, q1, r1, m1, 'foxhome/yi-cam-3/status', ... (6 bytes))
. connected!
Client yi-cam-3 received PUBACK (Mid: 1)
Trying to connect... Client yi-cam-3 sending CONNECT
Client yi-cam-3 received CONNACK (0)
Client yi-cam-3 sending PUBLISH (d0, q1, r1, m2, 'foxhome/yi-cam-3/status', ... (6 bytes))
. connected!
Client yi-cam-3 received PUBACK (Mid: 2)
Trying to connect... Client yi-cam-3 sending CONNECT
Client yi-cam-3 received CONNACK (0)
Client yi-cam-3 sending PUBLISH (d0, q1, r1, m3, 'foxhome/yi-cam-3/status', ... (6 bytes))
. connected!
Client yi-cam-3 received PUBACK (Mid: 3)
Trying to connect... Client yi-cam-3 sending CONNECT
Client yi-cam-3 received CONNACK (0)
Client yi-cam-3 sending PUBLISH (d0, q1, r1, m4, 'foxhome/yi-cam-3/status', ... (6 bytes))
. connected!
Client yi-cam-3 received PUBACK (Mid: 4)
Trying to connect... Client yi-cam-3 sending CONNECT
Client yi-cam-3 received CONNACK (0)
Client yi-cam-3 sending PUBLISH (d0, q1, r1, m5, 'foxhome/yi-cam-3/status', ... (6 bytes))
. connected!
Client yi-cam-3 received PUBACK (Mid: 5)
Trying to connect... Client yi-cam-3 sending CONNECT
Client yi-cam-3 received CONNACK (0)
Client yi-cam-3 sending PUBLISH (d0, q1, r1, m6, 'foxhome/yi-cam-3/status', ... (6 bytes))
. connected!
Client yi-cam-3 received PUBACK (Mid: 6)
Trying to connect... Client yi-cam-3 sending CONNECT
Client yi-cam-3 received CONNACK (0)
Client yi-cam-3 sending PUBLISH (d0, q1, r1, m7, 'foxhome/yi-cam-3/status', ... (6 bytes))
. connected!
Client yi-cam-3 received PUBACK (Mid: 7)
^C

Can you point me where to dig further?

Try the following steps:
1 - disable mqtt-config setting "Topics Prefix for remote configuration" to blank
2 - check if you are using the same client id with 2 or more devices

Crap... the second case was exectly mine... I forgot I have assigned yi-cam-3 already %)


Font was being applied to whole website

Description
I made a PR in which the font formatting at benefits.html was being applied to whole website. The PR was reverted back. Now I have rectified it.
I am really sorry for making such a huge error
Screenshots
No response

I would like to work on this issue. Please assign me this task.


Experience

@abdelrahman-essawy
i didnt know where i should put button (create bids) cuz i confuse with which permission the button for , so if u can make it , it will be helpful


valid for product images?

Hi, is possible use Kaleidoscope for product, as reel.js?
thanks in advance,

Hi @jujes, do you have an example?

hi @thiagopnts
yes, here:
http://www.djangopaths.com/interactive-360-degree-product-views-wagtail-streamfield-block/
looking for use image product rotation on saleor (django based)

oh I see what you mean. No, we dont support group of images, only equirectangular images

ok, thanks! :)


artifact on the creation of invoices

@eriknewland  the invoice will probably works but i just noticed this weird artifact of satoshi price, probably created via the API vs manually in the interface. the manual creation just says satoshis, so maybe there is an extra conversion in the code somewhere?
lnurlp through API

lnurlp on Manual creation

Should be fixed now, also moved the fees into env variables to make it easier to adjust in the future.

er did this get merged without review? @eriknewland
the fee is 100,000 not 10k.
and also 300,000 not 30k


Improve checking for SSW/IDL tests

In GitLab by @wtbarnes on Oct 11, 2019, 23:42
Currently, there is a decorator called needs_ssw that is added to any tests that require SSW to run a comparison between sswidl results and aiapy. The logic behind this is just whether the hissw package is installed. This should be improved, perhaps by just running a simple test IDL script (e.g. print, 'Hello World') and checking if that is successful.
See also #36

In GitLab by @wtbarnes on Oct 29, 2019, 15:07
closed via merge request !34

In GitLab by @wtbarnes on Oct 29, 2019, 15:07
mentioned in commit 59d6d707620d3ff181803f58eda26a535546a023


Ref#4596 Cover different db types for jdbc

Fixes #4596
Last part of expanding JDBC integration tests: covering different db types.
I've disabled some tests because of:

#4750 for postgresql
For derby

I didn't use quarkus-flyway yet, as we discussed in #4730, it can be covered in next PRs.

#4750 was resolved, I've fixed it in the tests.

Well done @ldrozdo . That's good addition. There are few questions in the review.

It seems that we can introduce a group testing for all the supported db.

Interesting idea @zhfeng , maybe it could be another ticket ?

Thanks @aldettinger for your comments, I implemented them. Should I create issue for group testing which @zhfeng suggested?

@ldrozdo yeah, please create a follow up issue for group testing. Thanks!

@aldettinger I'm sorry, the checks are failing and I can't figure out why. Everything works on my side. Can you please have a look, maybe you will spot the issue? Thank you very much.

@aldettinger I'm sorry, the checks are failing and I can't figure out why. Everything works on my side. Can you please have a look, maybe you will spot the issue? Thank you very much.

@ldrozdo The previous failure was just a transient connect failure we have more often those days. I think @essobedo was really nice to relaunch the job :+1:  A bit of fingers crossed and it should pass :)


Improve HistoricalStateProvider.

Describe the feature
I am interested in knowing if there are plans to support StateRootProvider and StateProvider::proof for HistoricalStateProvider.
We need to be able to use the get_proof API on historical data.
Also, we need the capability to perform forks, roll back to an earlier block, and insert a new block on top of a non-latest block using engine_forkchoiceUpdatedV1.
Currently, we are encountering an error "unable to compute state root on top of historical block."  geth allows this operation, and we would like to implement it in reth.
We are very interested in this functionality and could take on its implementation ourselves.
We are interested in hearing your vision of the problem and its possible solutions.
Additional context
No response

Yeah, same here.
Would be great if someone can assist with it a little
Thank you


Add option to throw error when ! (NonNull expression) is used on a non-nullish expression

🔍 Search Terms
"cast", "non null", "nonnull", "@ts-expect-error", "strict", "error"
✅ Viability Checklist

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

⭐ Suggestion
Add in the TSConfig.json a way to tell TS to check when devs are using ! (non null expressions) on variables that are non null
📃 Motivating Example
Mimic @ts-ignore vs @ts-expect-error but for !:
In well maintained codebases, @ts-expect-error can be preferred to @ts-ignore to get rid of those as soon as they become non necessary.
A similar thinking could be applied to ! so that we can keep healthy codebases
💻 Use Cases

What do you want to use this for?

Seeing ! in a codebase should mean that the variable can be nullish, and that we're telling TS that it isn't in practice.
It replicates @ts-expect-error but for !, at one difference: when using @ts-expect-error you can pick if you want this or @ts-ignore. With this proposal, it'd be set in the TSConfig so would get applied to all ! (but this is I think what devs would want)

What shortcomings exist with current approaches?

In large codebases, we can't keep track of why a ! was used, and whether it's still required. So we need to come back to them, and remove them manually to see if TS is happy or not without them. We can use ESLint to warn us against the use of ! but it's a 100% process.

What workarounds are you using in the meantime?

TypeScript-ESLint can be used to either warn / ban ! and instead push devs to use ?. or ??, but sometimes a variable is indeed never nullish, so it modifies / slows down the runtime for no real reason.
As TypeScript-ESLint can also have access to the full TS process, we could also use it to check the type of the variable on which ! is used to see if it's necessary. But this requires spawning the full TS process in ESLint, slowing it down significantly on large codebases (which is why having it directly in TS could be better)

I know that there is already https://github.com/microsoft/TypeScript/issues/46268 that exists, but this other issue also mentions modifying ?? & ?., and here I'm just talking about !

This is https://typescript-eslint.io/rules/no-unnecessary-type-assertion. Re: performance, it largely depends on how your project is set up,

I often see ESLint reporting this assertion due to having stale types, and I have to restart the ESLint server to fix it. For that reason alone I'd like to see this included in TypeScript itself.


feat: compute idle power with core ratio

This patch is for adding an idle power based on ratio of the machine cores over the cores of the machine that is used for training the power model.
This PR can be merged independently but there is no core ratio set yet.
For local regressor core ratio, need the following Kepler PR to solve the TODO.

[ ] https://github.com/sustainable-computing-io/kepler/pull/1684

For sidecar estimator, need the following PR in kepler-model-server to have estimator provides machine spec and server API to put machine spec to weight response.

[ ] https://github.com/sustainable-computing-io/kepler-model-server/pull/405

Signed-off-by: Sunyanan Choochotkaew<EMAIL_ADDRESS>

per Pang's instruction, this PR can be merged independently. Let's have this one first and wait for #1684


FloatQuantization subclass

As I was reviewing https://github.com/pytorch/ao/pull/223
I was reminded of this PR https://github.com/pytorch/ao/pull/214
And I'd be curious what range of floating point numbers we can just express using sublcasses

At some point we probably need to port float_quantize() from https://github.com/Tiiiger/QPyTorch (FP6-LLM use that to do FP16->FP6). The main logic is to handle correct rounding (we cannot just "erase" unwanted bits).
The code for that is actually quite simple (https://github.com/Tiiiger/QPyTorch/blob/f58bba72113e696099ef3e15e06cf421a06ff289/qtorch/quant/quant_cpu/quant_cpu.cpp#L267-L300). I tried out locally and it seems we can implement that in pure PyTorch, and potentially torch.compile() will make efficient CPU/GPU kernels for it (everything is elementwise op).
(NOTE: float_quantize() from qtorch does not handle bit-packing. The output is in original dtype, so it's like fake quantization)

(PyTorch now has uint32 I think, but still no ops, but maybe needed bit ops can just be enabled on uint32 (and other unsigned dtypes))

@vkuzo has an excellent FP32<->FPx conversion functions for MX dtypes (only for FP4_E2M1, FP6_E3M2, FP6_E2M3, but should be straight-forward to extend to other custom FPx). For quantization-aware training/fine-tuning, this should be enough as we probably don't need bit-packing. For inference, @vayuda is working on generic bit-packing, which should be useful (although personally I would prefer explicit bit-packing for each FPx dtype). Not sure if the current state of torch compiler can fuse FPx dequant into triton matmul kernel. If not, we probably need to write custom fused dequant+matmul triton kernel (which can be fun to do 😃)

Prototype FP8 quant using native PyTorch fp8 dtypes: https://github.com/gau-nernst/ao/blob/fp8wo/torchao/prototype/fp8wo/init.py
Llama2-7B-chat on 4070Ti SUPER. tokens/s is measured with torchao/_models/llama/generate.py. PyTorch 2.4.0.dev20240610+cu124

Quant type (weight only)
token/s
Bandwidth GB/s

BF16
46.74
617.64

INT8
88.22
584.05

FP8 E4M3 FN
79.33
1048.31

FP8 E4M3 FNUZ
78.73 (output is gibberish)
1040.44

FP8 E5M2
82.53
1090.65

FP8 E5M2 FNUZ
Error (see below)

Error with FP8 E5M2 FNUZ: Unsupported conversion from f16 to f8E5M2FNUZ with rounding mode rtne
The speed degradation compared to INT8 seems like because torch.compile cannot fuse act_bf16 @ weight_fp8.to(torch.bfloat16) * scales into a single kernel (thus memory bandwidth is also very high).
import torch
import torch._inductor.config

# core dump if either of these flags are enabled
# torch._inductor.config.force_mixed_mm = True
# torch._inductor.config.use_mixed_mm = True

def f(a, b, s):
    return torch.mm(a, b.to(a.dtype)) * s

fp16_act = torch.randn(1, 32).to(torch.bfloat16).cuda()
fp8_weight = torch.randn(32, 32).to(torch.float8_e5m2).cuda()
scales = torch.randn(32).to(torch.bfloat16).cuda()
torch.compile(f, mode="max-autotune", fullgraph=True)(fp16_act, fp8_weight, scales)

Codegen output: https://gist.github.com/gau-nernst/4afd0f5b97368ecf26d54b5f3415b004
When either use_mixed_mm or force_mixed_mm flag is set, I got core dump. (already opened an issue at PyTorch core https://github.com/pytorch/pytorch/issues/128381)
loc("/tmp/torchinductor_thien/h3/ch3inellku5joxa2jz4iwhfnrcquf7fbmuq53uw4vr6kuoctvtzo.py":76:21): error:  size mismatch when packing elements for LLVM struct expected 4 but got 8
python: /root/.triton/llvm/llvm-6f44bb77-centos-x64/include/llvm/ADT/ArrayRef.h:257: const T& llvm::ArrayRef<T>::operator[](size_t) const [with T = mlir::Type; size_t = long unsigned int]: Assertion `Index < Length && "Invalid index!"' failed.

One reason why FP8 quant is not so performant is probably because there is no optimized FP8->BF16 dtype conversion that can be fused with triton (either at triton level or torch.compile level, need to investigate further...)


TypeScript@next type acquisition approach may be broken

Hey folks!
I'm one of the maintainers of ts-loader
We have a nightly build that runs which includes typescript@next in its matrix.  I've noticed this has been failing for a week or so now.  See the travis failure here.  Note that TypeScript 2.3 and earlier all run fine.
TypeScript Version:  nightly (2.5.0-dev.20170618)
Code
The problem is a failing jasmine test here:
https://github.com/TypeStrong/ts-loader/tree/master/test/execution-tests/2.0.3_typesResolution
Expected behavior:
This test should pass - it's not really a runtime test at all; it's purpose is to validate that the @types/jasmine types were acquired.
Actual behavior:
Test fails.  Types don't appear to be being acquired.
I'm afraid I don't have too much time to look further into this but it appears that there is something potentially problematic with the TypeScript@next type acquisition approach and I wanted to give you a heads up.

Does this repro in the 2.4 RC?

Gimme a mo - I'll find out

The build is running now - answer should be yours in the output 😄

So it sounds like the RC doesn't exhibit this behavior, but something we ported over to 2.4 proper might have broken it. Do you know which specific day the build started failing?

@johnnyreilly, is this resolved as of<EMAIL_ADDRESS>per this comment?

Yes it is - closing!

@DanielRosenwasser  looking back through our nightly builds I think this is the one where it first appeared:
https://travis-ci.org/TypeStrong/ts-loader/builds/243198621


Overlays do not resize dynamically

When an app resizes, the overlay does not until it is removed and readded. In this example, navbar-ui had had a banner displayed above it. When I dismissed that banner, the overall size of navbar-ui changed but the overlay is still the original size.

Also happens with page resize, which is most commonly happens when resizing the devtools with the page as well.

ResizeObserver sounds like its perfect for this.

No edge and safari support is tough. Couldn't it be done with just the window resize event? https://developer.mozilla.org/en-US/docs/Web/API/Window/resize_event

web-ext doesn't support edge or safari so I'm not sure there's much value in targeting them.

Oh yea, and also re: window resize, that wouldn't work if the element itself changed in size. For example, my navbar is showing a dismissible banner and if the user does dismiss it, the size of navbar changes but the overlays didn't redraw.

I see. Then yeah sounds like your suggested implementation would be a good one. 👍

I'm convinced as well. Good proposal, @filoxo


closed?

Sorry Joseph, this turned out to be a jQuery problem. Please delete.

Will do, glad you got it resolved!

In the future, you can leave your original issue, and then just provide your solution/fix. That way other's, if they come the same problem, can follow your same solution - if it there is not an issue with the repo code.


--performance flag produces wrong numbers

run the following command: code --performance
run F1 > Show Startup Perf
wait a little ~30sec
open another blank window of VS Code
run F1 > Show Start Perf
🐛 the value of Total seems to include the wait time from step 3

@bpasero fyi I found this because I have the feeling that opening a new window is slower then starting VS Code but I have a hard time debugging/measuring this. Any tips are welcome.

@jrieken yeah this flag currently only makes sense for the first window to open, it is not fit for multi window case, though we could probably add that because we know how many windows are open.

@jrieken only thing I know right now to measure startup time within a window is to run the profiler and do a Reload Window. This should be very similar to the time it takes to open a second window.

You can now always run Developer: Startup Performance even if not providing --performance (it will just have a little less details). Also, for a second window, the start time will be correctly taken as the load time of the window and not the overall program start time.


[TPC-H] Query 21 times out at scale 100

From a preliminary look at the optimized graph, one issue might be that we don't properly push projections into the parquet reads:
Snippet from the graph:
Projection: columns=['l_orderkey', 'l_suppkey']
    FusedIO:
        ReadParquet: path='./tpch-data/scale-10/lineitem' columns=['l_orderkey', 'l_suppkey', 'l_commitdate', 'l_receiptdate'] filesystem=None kwargs={'dtype_backend': None}

I'd expect ReadParquet to only read ['l_orderkey', 'l_suppkey']. Combined with https://github.com/dask-contrib/dask-expr/issues/854, this appears to be fairly catastrophic.

I don't see this as a bug at the moment, we aim to only use one read_parquet call per data source to avoid reading the same columns more than once. This is a specialised example since we actually could separate them out, but I think that this is a little bit of an edge case. I wouldn't focus too much time on this at the moment, although I agree with you that we could be smarter about it. The problem is more that loses in value if there are operations in between read_parquet and the column restriction (like replace, shuffle, ...), since we would do the ops twice in this case. We can certainly make this special case better, but I am not sure if this would help us much in the grand scheme of things

Maybe related to the new CI failure #1363?


Tables/Matrices: Update Matio Library to v1.5.11

Reported by beutlich on 22 Oct 2016 12:40 UTC
Now that matio v1.5.10 is about to be released ModelicaMatIO should be updated to this version.

Migrated-From: https://trac.modelica.org/Modelica/ticket/2092

Modified by beutlich on 22 Oct 2016 12:40 UTC

164e1c2cfe5ad2170c0d08862041279e12f60fe6 updated Matio to v1.5.11. (There was no official MSL release containing Matio v1.5.9 or v1.5.10).


Allow disable app test endpoint auth

Support disable the app test endpoint auth.
Related command
Add arguments --disable-test-endpoint-auth in spring app create and spring app update.
General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[x] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

Hi @yuwzho , please help to review this PR, thanks!

Hi @yanzhudd there's a failure for 'test_asa_scenario.ByosTest.test_persistent_storage' case, since it also happened in other PRs, shall I need fix this failure? please help to review this PR. Thanks!

/azp run

Hi @yanzhudd there's a failure for 'test_asa_scenario.ByosTest.test_persistent_storage' case, since it also happened in other PRs, shall I need fix this failure? please help to review this PR. Thanks!

yes, the failed test is a test of spring module, so please fix the failed test as well.

@yanzhudd this testcase fix has been pushed in the PR validate path exist before all steps, please help approve and merge. Thanks

please resolve the code conflict

Hi @yanzhudd, resolved, please help review. Many thanks


[Event Request] table 5902 "Service Line" - OnBeforeTestQtyToInvoicePositive

Hi, would it be possible to get an OnBeforeTestQtyToInvoicePositive event in table 5902 (Service Line), like this:
        field(17; "Qty. to Invoice"; Decimal)
        {
            Caption = 'Qty. to Invoice';
            DecimalPlaces = 0 : 5;

            trigger OnValidate()
            var
                IsHandled: Boolean;
            begin
                // New event begin
                IsHandled := false;
                OnBeforeTestQtyToInvoicePositive(Rec, CurrFieldNo, IsHandled);
                if not IsHandled then begin
                // New event end
                    if "Qty. to Invoice" < 0 then
                        FieldError("Qty. to Invoice", Text029);
                end; // New event

                if "Qty. to Invoice" > 0 then begin
                    "Qty. to Consume" := 0;
                    "Qty. to Consume (Base)" := 0;
                end;

                if "Qty. to Invoice" = MaxQtyToInvoice then
                    InitQtyToInvoice
                else begin
                    "Qty. to Invoice (Base)" := CalcBaseQty("Qty. to Invoice", FieldCaption("Qty. to Invoice"), FieldCaption("Qty. to Invoice (Base)"));
                    UOMMgt.ValidateQtyIsBalanced(Quantity, "Quantity (Base)", "Qty. to Invoice", "Qty. to Invoice (Base)", "Quantity Invoiced", "Qty. Invoiced (Base)");
                end;
                if ("Qty. to Invoice" * Quantity < 0) or
               (Abs("Qty. to Invoice") > Abs(MaxQtyToInvoice))
            then
                    Error(
                      Text000,
                      MaxQtyToInvoice);
                if ("Qty. to Invoice (Base)" * "Quantity (Base)" < 0) or
                   (Abs("Qty. to Invoice (Base)") > Abs(MaxQtyToInvoiceBase))
                then
                    Error(
                      Text001,
                      MaxQtyToInvoiceBase);
                "VAT Difference" := 0;

                if (xRec."Qty. to Consume" <> "Qty. to Consume") or
                   (xRec."Qty. to Consume (Base)" <> "Qty. to Consume (Base)")
                then
                    Validate("Line Discount %")
                else begin
                    CalcInvDiscToInvoice;
                    UpdateAmounts
                end;
            end;
        }

    [IntegrationEvent(false, false)]
    local procedure OnBeforeTestQtyToInvoicePositive(var ServiceLine: Record "Service Line"; CallingFieldNo: Integer; var IsHandled: Boolean)
    begin
    end;

Thanks!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[FEATURE] View Resume Functionality

An optional section, "View Resume" Needs to be added to all levels. The heading will be "View Resume" and it would hyperlink to the Google Drive/overleaf or any other link user passes in the GitHub Action

Could I work on this issue under SWOC?

Could I work on this issue under SWOC?

@ShivangiRai1310 Okay, you can work on this but do you have some knowledge of Python?

@ShivangiRai1310 Okay, you can work on this but do you have some knowledge of Python?

@ShivangiRai1310 Okay, you can work on this but do you have some knowledge of Python?

@ShivangiRai1310 Okay, you can work on this but do you have some knowledge of Python?

Because the data flow for resume link from actions to data loaders is also needed. You can set up the HTML files by going through the wiki's https://github.com/kaustubhgupta/PortfolioFy/wiki/Setting-Up-Locally and after generating the HTML file, you can add the resume functionality.

Because the data flow for resume link from actions to data loaders is also needed. You can set up the HTML files by going through the wiki's https://github.com/kaustubhgupta/PortfolioFy/wiki/Setting-Up-Locally and after generating the HTML file, you can add the resume functionality.

@ShivangiRai1310 Are you working on this? Let me know if you face any issues!

@ShivangiRai1310 Are you working on this? Let me know if you face any issues!

I have included the frontend part displaying the view resume upon clicking which the pdf opens in another window.
For now, in order to get the resume, I have added a random google drive uploaded link which can be fetched from command line input properties, in case required that way.
Otherwise, let me know how else should I fetch the link. Could you elaborate on that "resume link from actions to data loaders needed"

I have included the frontend part displaying the view resume upon clicking which the pdf opens in another window.
For now, in order to get the resume, I have added a random google drive uploaded link which can be fetched from command line input properties, in case required that way.
Otherwise, let me know how else should I fetch the link. Could you elaborate on that "resume link from actions to data loaders needed"

For getting the link from the user, create a new variable in actions.yml. See the existing parameters to get an idea of how to declare one. Basically, it is :
inputs:
  variable_name:
    description: "<Give the description for resume link, anything would work>"
    required: False
    default: None

After declaring it, pass it as:

make it last argument.
After all this, you will be able to get the variable in the generate_index.py file. Store it in a variable and pass it to both levels and ultimately, using f-strings, pass it to your HTML.

For getting the link from the user, create a new variable in actions.yml. See the existing parameters to get an idea of how to declare one. Basically, it is :
inputs:
  variable_name:
    description: "<Give the description for resume link, anything would work>"
    required: False
    default: None

After declaring it, pass it as:

make it last argument.
After all this, you will be able to get the variable in the generate_index.py file. Store it in a variable and pass it to both levels and ultimately, using f-strings, pass it to your HTML.

Are you able to test this parameter locally?

Are you able to test this parameter locally?

Please do add Swoc label before merging in case required.

@ShivangiRai1310 That's not an issue 😃


tag proposal:  podcast:popular

This could be seen as the ying to the <podcast:gateway>'s yang.  Whereas a gateway list allows the podcaster to specify which episodes a listener should start with, this tag would allow that information to be gleaned from actual listeners by way of host supplied download metrics.
The structure would look like this:
<podcast:popular
  updated="[datestamp of when this tag last refreshed]"
>
[a list of <guid> elements]
</podcast:popular>

Channel
(optional | single)
The node value of this tag should be a list of <guid> elements that reference the most popular episodes in descending order from most to least popular.  The updated attribute should be an RSS standard datestamp of the time this tag's content was last recalculated.  There is no limit on the number of <guid> elements that can be present, but it's suggested to keep it to five or fewer since apps will likely limit it anyway for sane UI behavior.
Example:
<podcast:popular updated="Fri, 08 Oct 2021 18:55:32 +0000">
    <guid>https://example.com/pc20/ep143</guid>
    <guid>https://example.com/pc20/ep66</guid>
    <guid>https://example.com/pc20/ep292</guid>
</podcast:popular>

I think this, too, could be handled better with the better label of <podcast:featured>. "Gateway" just has such bad connotations, and "popular" seems redundant. "Featured" can handle both of these and more.

Do we have tags that aren't controlled manually by the podcast publisher/creator themselves, but are influenced by performance or something outside of the straight publishing flow? This seems like something that could suddenly change on the podcaster in a way they may not want (some episode goes viral, etc), and they have to play catchup/adjust/remove the tag, as they might not want to have that episode at the top -- it might be really old, for example, or feature a co-host that has left.
What are the negative connotations with gateway? Honest question!


Release fixings

Readded money cuz it actually wasn't legacy code.
Also fixing #100 with a simple conversion fix in transfer-funds.js
Clarified Developers.md
Other stuff

I tested this PR, and it seems like sending money doesn't work.

Yeah sorry, that would be due to line 58 in transfer-funds.js
Like I said, I can't test the code for sending money :/

Can you push a fix soon? We're testing now


[c++] Support reading member-to-uri mapping in read and write mode

Issue and/or context:
https://github.com/single-cell-data/TileDB-SOMA/issues/1623
Changes:
Extend caching introduced in #1584 to also store SOMAGroup member-to-uri mapping so that this information can also be accessed in both read and write mode.
Notes for Reviewer:
N/A

Codecov Report
Patch coverage has no change and project coverage change: +25.22% :tada:

Comparison is base (ef50ad3) 65.86% compared to head (8114e06) 91.09%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1627       +/-   ##
===========================================
+ Coverage   65.86%   91.09%   +25.22%
===========================================
  Files         109       34       -75
  Lines        9243     2919     -6324
===========================================
- Hits         6088     2659     -3429
+ Misses       3155      260     -2895

Flag
Coverage Δ

python
91.09% <ø> (ø)

r
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 75 files with indirect coverage changes

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


Vuejs : firebase is not defined

Hello there,
I am new to vuejs and grapesjs, and I was wondering if someone here could help me understand why the plugin doesn't find the firebase variable. I didn't manage to reproduce my local env on a codepen, because I used the vue-cli to install the project. So all I got is the base script.
Here is the error

In the plugin file, the error comes from the firebase.initializeApp. So the first time firebase is called.
If I console.log the firebase variable before the editor.init, the "firebase" variable is set up. Even in previous lifecycle phases.
So my first guess is that the plugin doesn't have access to the variable somehow ?
Other plugins like gjs-preset-newsletter seem to work fine, so I think it's just a problem on my part with the management of vue and the importation of the modules.
Does anyone has an idea ?
// @ is an alias to /src
import { config } from '@/firebase';
import firebase from 'firebase/app';
import 'firebase/firestore';

import 'grapesjs/dist/css/grapes.min.css';
import 'grapesjs-firestore';
import grapesjs from 'grapesjs';

export default {
  name: "editor",
  data() {
  	return {

    };
  },
  methods: {
  	init: function(){

  	}
  },
  beforeMount() {
    console.log(firebase);
  },
  mounted() {
    console.log(firebase);
    const editor = grapesjs.init({
      // Indicate where to init the editor. You can also pass an HTMLElement
      container: '#gjs',
      // Get the content for the canvas directly from the element
      // As an alternative we could use: `components: '<h1>Hello World Component!</h1>'`,
      fromElement: true,
      // Size of the editor
      height: '780px',
      width: 'auto',
      // Managing the storage
      storageManager: { type: 'firestore' },
      plugins: ['grapesjs-firestore'],
      pluginsOpts: {
        'grapesjs-firestore': {
          docId: 'testID',
          apiKey: config.apiKey,
          authDomain: config.authDomain,
          projectId: config.projectId,
        }
      },
      ...
    });
...
};

package.json
{
  "name": "xxx",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "serve": "vue-cli-service serve",
    "build": "vue-cli-service build",
    "lint": "vue-cli-service lint"
  },
  "dependencies": {
    "firebase": "^5.7.0",
    "vue": "^2.5.17",
    "vue-router": "^3.0.1",
    "vuex": "^3.0.1"
  },
  "devDependencies": {
    "@vue/cli-plugin-babel": "^3.2.0",
    "@vue/cli-plugin-eslint": "^3.2.0",
    "@vue/cli-service": "^3.2.0",
    "@vue/eslint-config-prettier": "^4.0.0",
    "babel-eslint": "^10.0.1",
    "eslint": "^5.8.0",
    "eslint-plugin-vue": "^5.0.0-0",
    "node-sass": "^4.9.0",
    "sass-loader": "^7.0.1",
    "vue-template-compiler": "^2.5.17"
  }
}

Hi, it's probably a problem when you made the firebase initialization, can we see the file where you import vue-router etc ?

Can you push a commit in another branch to see all the source ?

Hi @nmarseaud this plugin is not bundled with firebase and firestore libraries so you have to include them manually (eg. https://www.gstatic.com/firebasejs/4.13.0/firebase.js and https://www.gstatic.com/firebasejs/4.13.0/firebase-firestore.js)


BUG: Fix inconsistent C engine quoting behaviour

Add significant testing to quoting in read_csv

Fix bug in C engine in which a NULL quotechar would raise even though quoting=csv.QUOTE_NONE.

Fix bug in C engine in which quoting=csv.QUOTE_NONNUMERIC wouldn't cause non-quoted fields to be cast to float.  Relevant definitions can be found in the Python docs here.

I think that we should expose these csv.QUOTE_* values somewhere in the pandas namespace as well. and/or, maybe accept a string? e.g. quoting='QUOTE_NONE' ? (and case_insensitve), or maybe just quoting='nonumeric','minimal','all' (not really even sure what QUOTE_NONE does)
not sure if this is an issue in the real-world, but seems to be a small pot-hole.
@jorisvandenbossche

@jreback : documentation on the meanings of the csv.* enums can be found in the link I posted in my PR.

@jreback : I'm perfectly fine with name-spacing as I just commented above, though I'm hesitant about accepting strings.  IMO enums are sufficient and will make it easier for users to debug (spelling errors).
Not entirely sure how many people use them as you said (my guess is that many don't, and I certainly have never used them in my work), though it should probably remain as an option since it does play a role in csv.Dialect creation.

[Current coverage][cc-pull] is 84.23%

Merging [#13411][cc-pull] into [master][cc-base-branch] will increase coverage by <.01%

@@             master     #13411   diff @@
==========================================
  Files           138        138
  Lines         50805      50806     +1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          42796      42797     +1
  Misses         8009       8009
  Partials          0          0

Powered by Codecov. Last updated by [3eb4784...383f2db][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pydata/pandas/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pydata/pandas/compare/3eb4784dbf1757f2c3383beff2f70c5da47a93fe...383f2dbefb8c15c5c27398295280315424a96f51
[cc-pull]: https://codecov.io/gh/pydata/pandas/pull/13411?src=pr

Added the io.common enums, and Travis is still happy.  Ready to merge if there are no other concerns.

@jreback : Did some minor tweaking of the docs once more to mention that the csv enums can be found in pandas.io.common.  Ready to merge if there are no other concerns.

I am not really sure that I find pd.io.common.QUOTE_ better than csv.QUOTE_.. (but it is one import less).
I would maybe rather accept strings like 'minimal' or 'nonnumeric', this seems the more user friendly option? (but it makes it of course more verbose to explain all possibilities in the docstring)

@jorisvandenbossche : I disagree about the strings.  See my reasoning above

@jreback : Did some minor tweaking of the docs once more to mention that the csv enums can be found in pandas.io.common.  Ready to merge if there are no other concerns.

@jreback : Did some minor tweaking of the docs once more to mention that the csv enums can be found in pandas.io.common.  Ready to merge if there are no other concerns.

I disagree about the strings.

OK on not adding those, but still, I don't see the added value to users of having the enums in pandas.io.common (it's just more ways to import the exact same thing). So personally I would just not mention it in the docs (also, if we do put it in the docs, we should have a test for it)

What test exactly?  Not sure what you mean.  @jreback thoughts?

What test exactly? Not sure what you mean

Sorry, I meant just a simple test that it is available in the pd.io.common namespace (if we want users to use it from there). But let's first decide what to do with it.

@jorisvandenbossche : Not sure if we're encouraging users to use it from there.  Just providing a pandas-namespace location where those enums are available from what it seems.

Not sure if we're encouraging users to use it from there. Just providing a pandas-namespace location where those enums are available from what it seems.

Adding it to the docstring seems 'encouraging users to use it' to me, why otherwise adding it there? For me it's OK to add it to the pd.io.common namespace if that makes the implementation easier, no problem, I personally would just not add it to the documentation.
@jreback do you have any preference?

I could go either way. I thought it would be more consistent, IOW, a user can just use it directly, not having to think about csv module at all. Though since we don't really document these completely, maybe should just back it out and point for all of this to csv module (for the constants).

thanks @gfyoung

Thanks a lot! This more comprehensive testing is really good


Bump github.com/google/licenseclassifier/v2 to v2.0.0-pre5

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Update to the latest release.
Which issue(s) this PR fixes:

None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

Found a runtime panic in the license classifier, working on a fix in https://github.com/google/licenseclassifier/pull/44


closeButton should be able to set onPressed as backButton

Use case
In our application we want to display a close icon in AppBar with customized behavior.
backButton has onPressed parameter in initializer to enable override of default navigator pop behavior. closeButton should also be able to set it.
Proposal
Add onPressed field to closeButton initializer.

you should be attach your code because I can't direct find problem.


⚠ Bofrost.fr has degraded performance

In d32c7b6, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1505 ms

Resolved: Bofrost.fr performance has improved in e21836e after 7 minutes.


Update Frame.java

ok sure lol

ok sure lol
Literally unnecessary but ok

ok sure lol
Literally unnecessary but ok
this is the best tool


appparts: IsOperationAllowed should supports resource filters in ACLs

https://github.com/voedger/voedger/commit/22a8af0a4548a2419f56dc029b483e4c1f43a883


Generating report

Hello Tananaev,
Thank you for spending time to make your application able to compete with commercial apps. I am completely new to this world of open source and my skills are under zero as far as coding is concerned. I am just a user of those GPS tracking services applications for my professional trucking activity.
So far, I am using a commercial web based application. In fact, 2 web based applications from 2 different companies. Each one of them cover 10 trucks as an experimentation to see what is exactly my requirement. I have read in a recent interview that you are not a user of your application so I'll try to bring my feed back as much as I can.
I am based in Thailand by the way. It is important to say because most of the sensor available on tracking devices are very accurate and very specific but on my perspective I need only to focus on driver's behaviours I can change like "overtime parking engine on" or "overspeed". If I know where my trucks go, how long do they stay at registered delivery spots (which are always the same) and how many kilometers they run, 90% of the job's done as far as monitoring is concerned. The balance 10% remaining uncovered is the uncompressable space of freedom I have to deal with on another ground than with technical devices.
In order to find a open source application to cover my needs, I checked first OpenGTS. I had it running on Ubuntu server but it appears that the standard version doesn't allow to generate trip reports or geofence in/out reports... Even though it is open source, I don't have the skill to enable these reports. So I came to traccar hoping that these reports would be available. However, from your demo web server, I couldn't see where to get them.
Could you please help me with that ? Basically, what I need is a date to date reports showing 1/ time of arrival - time of departure from registered area (geofence) by truck and by selected group of trucks 2/ non moving truck for a certain period of time whether engine is on or off 3/ overspeed.
Are these reports easy to configure on traccar ? Is there a kind of webmin application for traccar that would give access to the core of the application to configure these reports ?
Thank you for your feedback or any feed back from this community.
Best regards
Poobaisle

Thank you for feedback. Currently Traccar only supports a single very basic report, but I plan to implement better report generation very soon.
P.S. This repository is no longer active, so you should use following issue tracker:
https://github.com/tananaev/traccar/issues

Sorry, I am a bit lost with the different repository... I think I left my initial message on traccar then it was moved to traccar-web no ? Traccar-web is no longer active ? Does it mean the server d.traccar.litvak.su is no longer a project ? I found there very interesting reports I should say. Is this new development already over ??

Its still active. You just posted to a wrong issue tracker. Please use this
one https://github.com/vitalidze/traccar-web/issues
I am on vacation currently. My colleague may help you though.
четверг, 14 июля 2016 г. пользователь poobaisle написал:

Sorry, I am a bit lost with the different repository... I think I left my
initial message on traccar then it was moved to traccar-web no ?
Traccar-web is no longer active ? Does it mean the server
d.traccar.litvak.su is no longer a project ? I found there very
interesting reports I should say. Is this new development already over ??
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tananaev/traccar-web/issues/284#issuecomment-232606332,
or mute the thread
https://github.com/notifications/unsubscribe/ACMCNf5fu6gkXjD4a0tjGcGWhMj6PSdoks5qVflsgaJpZM4JMFaQ
.

--
Kind regards, Vitaly Litvak

We have lots of improvements since nice to see a fellow trucker to use this nice system.
I hope you have since found the forum and website. If you have any other questions feel free to open a post there. https://www.traccar.org/forums/ Cleaning up the old issues list here.
@tananaev can be closed.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3220

Questions ?
Please refer to the Backport tool documentation


errorDetail may return dicts

https://github.com/projectatomic/osbs-client/blob/master/osbs/build/build_response.py#L113 expects errorDetail to be a string, while atomic-reactor returns a dict with code and message. As a result koji build may fail:
2016-03-23 09:36:23,571 - atomic_reactor.util - DEBUG - Error: Package: krb5-workstation-1.13.2-10.el7.x86_64 (atomic-reactor-koji-plugin-extras-rhel-7.2-docker-candidate)
2016-03-23 09:36:23,571 - atomic_reactor.util - DEBUG - Requires: krb5-libs(x86-64) = 1.13.2-10.el7
2016-03-23 09:36:23,571 - atomic_reactor.util - DEBUG - Installed: krb5-libs-1.13.2-12.el7_2.x86_64 (@koji-override-0/7.0)
2016-03-23 09:36:23,571 - atomic_reactor.util - DEBUG - krb5-libs(x86-64) = 1.13.2-12.el7_2
2016-03-23 09:36:23,571 - atomic_reactor.util - DEBUG - Available: krb5-libs-1.13.2-10.el7.x86_64 (atomic-reactor-koji-plugin-extras-rhel-7.2-docker-candidate)
2016-03-23 09:36:23,571 - atomic_reactor.util - DEBUG - krb5-libs(x86-64) = 1.13.2-10.el7
2016-03-23 09:36:23,572 - atomic_reactor.util - DEBUG -
2016-03-23 09:36:23,572 - atomic_reactor.util - DEBUG - You could try using --skip-broken to work around the problem
2016-03-23 09:36:23,650 - atomic_reactor.util - DEBUG - You could try running: rpm -Va --nofiles --nodigest
2016-03-23 09:36:26,257 - atomic_reactor.util - ERROR - {"errorDetail":{"code":1,"message":"The command '/bin/sh -c yum install -y ipa-client realmd /etc/selinux/targeted/contexts/dbus_contexts oddjob oddjob-mkhomedir sssd sssd-dbus adcli perl tar \u0026\u0026 yum clean all' returned a non-zero code: 1"},"error":"The command '/bin/sh -c yum install -y ipa-client realmd /etc/selinux/targeted/contexts/dbus_contexts oddjob oddjob-mkhomedir sssd sssd-dbus adcli perl tar \u0026\u0026 yum clean all' returned a non-zero code: 1"}
2016-03-23 09:36:26,257 - atomic_reactor.util - INFO - no more logs
2016-03-23 09:36:26,257 - atomic_reactor.build - INFO - build was unsuccessful!

Should be fixed by #365