The future of the JSON schema gem

@ruby-json-schema/owners
I have a lot of ideas for refactoring, modernising and slimlining the json-schema gem, as well as adding support for draft 6, 7 and eventually 8. However, I know everyone is busy and reviewing pull requests for json-schema isn't always a high priority.
I'm concerned that the project has become neglected, and with that in mind, I feel that someone needs to step up to take a bit more of a lead to steer the project into a better place. I'd be happy to be that person, and would be prepared to take on the responsibility of making sure the project is well-maintained in the future.
Could you please let me know your thoughts on the idea of me taking the lead on the json-schema project, which would entail cleaning it up and refactoring it as I see fit, and merging pull requests without the agreement of another maintainer. If nobody objects, I'll assume that everyone is busy with other things and is happy for me to push the project forward alone.
Finally, if you're no longer interested in maintaining the project and don't wanted to be "pinged" by contributors on issues or pull requests, let me know and I can change you from being listed as an "Owner" to a "Contributor".

I'm fine with you taking the lead as proposed. Although I probably do not have much time to put into the project (and have been absent for about 1.5 to 2 years for personal reasons), I'd try to help with reviews if possible. Maybe also get back into development but I don't want to promise something I cannot keep.

Hi @iainbeeston Let me know if you need any help.

I've been thinking about this more since I opened the issue, and I've realise that I was being unrealistic. I have a lot of other projects competing for my limited time, on top of work and family life. I would love to bring this project up to date with the latest json schema drafts and refactor it into something more maintainable and see it progress into the future, but I can't spare enough time to make that happen.
Perhaps instead this thread should start the search for new maintainers?

Is anyone interested in becoming a maintainer of json-schema?
There are a lot of open pull requests that need merging, but it's important to understand how they fit into the existing codebase before merging, as they might introduce new issues (the existing test suite is not comprehensive for more complex schema, such as loading from uris or fragments - see #248). Also the gem supports a number of unofficial extensions to the specification (such as default values) that add complexity.
Any maintainer should have a good understanding of the json schema specification. The most recent release of the json schema gem only supports the specification up to draft 05, but draft 06 and 07 have been released since, and 08 is scheduled to be released some time after June 2019. Although the code supports modular validation rules (that can be added and removed for different versions of the specification) and we can use the json schema common test suite to test compliance, I believe that adding support for drafts 6 and 7 will not be a trivial change.
If you're interested in becoming a maintainer please let us know on this issue.

I came across https://github.com/davishmcclurg/json_schemer.
@davishmcclurg what's the rationale for creating a different gem if you don't mind sharing? ☮

There's also https://github.com/brandur/json_schema
There are a lot of json-schema implementations out there in many different languages (although most of them are written in javascript)

Hey y'all. I created json schemer to get better performance and draft 7 support. Performance was a big thing I thought could be improved at the time. I was generating schemas for API validation and the overhead of this gem wasn't acceptable. If those things could be improved here, I'd happily deprecate my gem since I also don't have a ton of time to support it.

To be honest, it is worse than I wrote last October. I do not have the time (or more honest, have other stuff that is more important to me) to even look at the PRs handed in here. I could do quick review stuff but that wouldn't meet my view of quality standards.
So, I'll do officially what actually already happened, given what I did (not) in the past: I'm stepping back from maintaining json-schema.
The sad thing is, it seems like there is no one who wants to keep up the development. Maybe one of the other two gems can/will go on. Given the architecture of json-schema it is probably easier to improve one of the other to support more standards (if they don't do it already) and have better performance (afair are json-schema benchmarks rather slow).
@iainbeeston  how should we proceed with this?

I'd recommend we at least put a disclaimer at the top of the readme, warning people that the gem is no longer maintained. It's also possible to put a post_install_message in the gemspec to warn anyone installing the gem that it's no longer supported, although I worry that this could be confusing if (in future) someone takes over maintenance and tries to revive it.
My advice to @brandur (author of json_schema) and @davishmcclurg (author of davishmcclurg) would be to add the json-schema common test suite to their tests if they haven't already (we have code in json-schema for doing this and that could be copied), as that's an easy way to be confident that any implementation adheres to the official spec. So long as there are some ruby implementations that match the spec, it shouldn't matter to anyone whether it's json-schema or any other gem, and it should be easy for people to migrate away.

Is there any way we can improve this situation? I've seen this pattern many times before.
It looks like both Voxpupli and Sous-Chefs use this Gem day to day, and are used to running repositories that were once loved but don't have a current active contributor. I wonder whether something similar to their collectives could be done for this and other Gems?
As long as we can decide what good looks like, then I think we can keep this Gem updated and keep the quality as high as desired by @RST-J. All we would need to start is a small group of interested parties to join this org, and start adding in tests.
Sorry if this is a bit presumptuous. But hopefully this stirs something into action.
(see Vox Pupuli and Sous-Chefs for their repos)

@damacus might want to move to https://github.com/davishmcclurg/json_schemer as it's maintained

Hi,
if this helps, Vox Pupuli could keep maintaining the project in their (our, I'm part of Vox Pupuli) namespace. We're happy to help.

@bastelfreak I’d be happy to do that, if no one in @ruby-json-schema/owners objects. If I don’t hear anything from any of the other maintainers I’ll assume they don’t mind, and I will transfer this repo over to the box pupuli organisation.

sounds good to me!

@bastelfreak can we prevent the divide here with @davishmcclurg?

@iainbeeston I would have suggested the same, so just do it. I think we both were the only ones to respond the one or two times when something about the future of this project came up, so I do not expect any reaction and even less objections from others in here.
Would be glad if it is taken over. :+1:

@iainbeeston I invited you to the Vox Pupuli GitHub org so you're able to migrate the repo. @scalp42 sorry, but I'm not sure if I can follow. Divide from what?

@bastelfreak davishmcclurg/json_schemer is already maintained

@bastelfreak Thanks for adding me to the vox pupuli organisation, however I need the permission to create repositories before I can transfer it. Can you set that up for me please?
Can I also confirm which rubygems user I should transfer the gem to? Should it be “voxpupuli”?

can you check the permissions again? You should be able to transfer the repository. The rubygems user is voxpupuli.

@bastelfreak I’ve tried again but I’m afraid I still get the same error when I try to transfer the repo to voxpupuli

can you check your inbox/github settings? You need to accept the invitation for voxpupuli first

@bastelfreak My bad, I assumed I wouldn't need that. Could you please send another invite (the first has expired) and I'll try again

sure, I sent it again.

@bastelfreak I've transferred the repo and invited voxpopuli to be an owner of the ruby gem. Can you confirm both are correct?

Hey again all, I'm the maintainer of another ruby json schema implementation: davishmcclurg/json_schemer. I'm curious if there's any interest here in somehow combining our efforts. I recently released a new version that supports the latest drafts, but it took me a few years before I found the time. I see that work here seems to be picking up a bit and I'm wondering if it makes sense to focus on one gem or the other so that we can better keep up with json schema in the future. I haven't thought too much about the specifics yet, just wanted to gauge interest first. Let me know what you think—and, of course, it's no problem if you prefer to keep the projects separate.
Related: https://github.com/davishmcclurg/json_schemer/issues/142

FWIW, I'm in the process of re-writing json-fuzz-generator from depending on this gem to depending on @davishmcclurg's json_schemer.  Once I'm done with the rewrite I'll rename and re-release the gem.

@davishmcclurg I can't speak for others, but I think at this point this gem isn't receiving the attention it deserves. Vox Pupuli is set up to support a team of maintainers, but at this point I'm not sure people are actively doing so. I think an ecosystem is best served by a single implementation that's well maintained.
Moving forward, I think either people (preferably multiple) should stand up and really own this gem, or formally suggest json_schemer as a replacement.
If people are interested in the former, as a member of the Project Management Committee (see https://voxpupuli.org/elections/) I can help set that up.


Pull/78  COMPRESS-485 + Substituting 'synchronized' with faster and fully thread-safe collections 'ConcurrentLinkedDeque' and iterators.

One ArrayList was synchronized and another was not.
In this concurrent access. Considering wekanesses of memory model this may produce sporadic NPE (so that retrieved Future may not be visible in another thread)
Using synchronization would avoid it but why to use it in such simple critical sections today if we have concurrent collections in Java 1.5+.
So used ConcurrentLinkedDequeue and that's it. Easy.

Coverage increased (+0.07%) to 86.594% when pulling a618b461c97a513bf97bef4d476042bd719fe48a on Tibor17:pull/78 into e127b13c58117a2e26639f5f83b785180ef8cfa2 on apache:master.

Thanks @Tibor17. Could you please do me a favor and rebase this now that I have merged #78?

Before #78 the synchronized would ensure you couldn't spin off new threads and add entries to them while writing or closing. With your change it would be possible to add new threads after writing has started and the result is not really defined (the iterator may see the new stream or it may not), right?
Actually I looked through the history and initially we didn't synchronize the iterator when writing, this has been added as "recommended practice" via https://issues.apache.org/jira/browse/COMPRESS-430 - so we never really cared whether adding new threads while writing would lead to a defined result.

Hi @bodewig , I have to check the code by myself again and rebase the branch since it is 2 months old change.

@bodewig
You have opened two things, but sorry I have to say, they are not related and here is my explanation why:

locking within private/public methods
synchronized iterators

Since now, I am refering to before #78 .
Regarding (1) you mentioned:

synchronized would ensure you couldn't spin off new threads and add entries to them while writing or closing

The methods (addArchiveEntry and submit) and writeTo were not mutually exclusive.
If we want to exclusive then we should fire another PR for that.
If you want to prevent writeTo from double/parallel call, we can do this
if (es.isShutdown()) throw ...Exception.
Locking all 3 methods can be done with AtomicBoolean but that needs to see example.
Regarding (2):
It is old style to use collections wrapped in synchronizedList() and use iterators wrapped within synchronized (streams) {}. In modern collections ConcurrentLinkedDequeue is beneficial because it does not require any external locking.
There was also one issue because the variable futures was not synchronized collection.

Thanks @Tibor17
Actually I've more or less been thinking out loud and not raising issues.
I was trying to figure out whether the synchronized usage was giving the API users - or our code - any extra guarantees that the non-blocking collection code didn't. Keep in mind that I'm not the author of the original code either. I am the one who added the synchronized around the iteration over streams - but completely overlooked to the iteration over futures before that. Doesn't sound as if I was the most qualified person to comment ;-)
What I meant with the first part was that if you added new threads once writeTo was underway then whatever your new threads contributed would not be part of the result - while it now is undefined. Looking at the current code in master I see I've been wrong as writeTo does quite a few things before entering the synchronized block and there is enough leeway anyway.
No, I don't think we need to exclude the methods from each other. The class has a very clear usage pattern of two distinct phases:

add all the things you want to add
call writeTo

and it should be clear that the result of calling writeTo before you are done with the first phase is a dubious idea. In particular as the javadocs of writeTo state it will shut down the executor.
I am aware of the iteration guarantees of ConcurrentLinkedDeque. Back when Kristian added the parallel zip support Commons Compress' baseline has been Java 5 (we bumped that to 6 with Compress 1.12 and 7 in Compress 1.13, which is where we are today). If it had been Java 7 back then, then I'm sure Kristian would have used non-blocking collections instead.

oh, you don't need to rebase at all. I just realized all your changes have been pat of a distinct commit I could pick out of the PR and apply it separately. Sorry.

merged, many thanks again.

@bodewig
I am thankful to you.
btw, great that you mentioned Kristian, he is my colleague in Apache. We have one common friend ;-)
Cheers Tibor

s/non-blocking/lock-free/g :man_facepalming:


Lubricants in file python

This PR resolve #1005

Codecov Report

Merging #1009 (f9086b4) into main (fe72c9e) will decrease coverage by 0.40%.
Report is 9 commits behind head on main.
The diff coverage is 51.56%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main    #1009      +/-   ##
==========================================
- Coverage   85.62%   85.23%   -0.40%
==========================================
  Files          34       35       +1
  Lines        7549     7573      +24
==========================================
- Hits         6464     6455       -9
- Misses       1085     1118      +33

Files
Coverage Δ

ross/__init__.py
100.00% <100.00%> (ø)

ross/fluid_flow/cylindrical.py
45.65% <100.00%> (ø)

ross/fluid_flow/lubricants.py
100.00% <100.00%> (ø)

ross/rotor_assembly.py
93.24% <100.00%> (ø)

ross/units.py
88.88% <62.50%> (-7.73%)
:arrow_down:

ross/results.py
87.70% <37.50%> (-1.98%)
:arrow_down:

... and 9 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


It's nice to have a global terminal buffer

Once this can be done, I'm also interested in implementing this selecting of (some globals) terminal buffers like they did: https://github.com/uga-rosa/ugaterm.nvim

Do you have plan on this issue? I really like this plugin but I also really need this feature, it is too cumbersome to open new terminal buffer for each window.


Global Setting for Safe Links

Please provide some additional discussion about why "tracks user clicks in supported Office 365 apps." is set to true for the Standard and Strict categories and what an organization is supposed to do with the generated data.

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

ID: 0c77e1cb-1509-83cc-7c2a-564d5e3e91ea
Version Independent ID: 9f07278c-37ab-beac-a5dd-fa4f4825b3a4
Content: Microsoft recommendations for EOP and Defender for Office 365 security settings - Office 365
Content Source: microsoft-365/security/office-365-security/recommended-settings-for-eop-and-office365.md
Product: m365-security
Technology: mdo
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

@MSFTTracyP Please help us in resolving this issue. Thanks

@DeanGross, does the explanation of what the setting does over at  Safe Links settings for Office 365 apps.
Even though the setting is turned on by default, if you have no Safe Links policies, then the protection is not applied to anyone.
The setting only applies to links in supported Office documents. The equivalent setting for links in the body of email messages is described in Safe Links settings for email messages.
So, if you understand why to track URL clicks in email messages, the same logic applies for why to track URL clicks in Office documents.

yes that does help, but I am not seeing anything about where the clicks are reported or any guidance about what should be done with the data. Am I missing something?

@DeanGross, probably the best place to see the results of Safe links is the URL threat protection report.

@chrisda Thank you very much for sharing this explanation. @DeanGross Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Password Management: Hardcoded Password

We have used the Firebase podfile as per our need. Here we have attached the screenshots for your reference . kindly help us asap.
Step 1: Describe your environment

Xcode version: 11.3.1
Firebase SDK version:  6.2.1
Firebase Component:  FirbaseinstanceID:4.2.3
Installation method: `CocoaPods '

Step 2: Describe the problem
Hardcoded passwords may compromise system security in a way that cannot be easily remedied.Hardcoded passwords may compromise system security in a way that cannot be easily remedied.

@divyaKalusulingam It looks like a flake or a bug of the tool your are using for analysis, because the provided code has never been a part of Firebase especially in FIRApp.m file.
Another recommendation is to update Firebase in your app if it is possible. There were several stability and security improvements implemented in recent versions that you may benefit from.
I am closing the issue for now but feel free to reopen with relevant code references. Thanks.


[task] Self-Correction

🚀 The Task
When the model summarizes a passage of text or code, it may fail to account for important information that only the user knows. The user should be able to point out what the model is missing, and the model should correct its own summary.
Example
No response
Additional Notes

Creator: @ericyu3
Google Sheets Reference: CarperAI: Instruct-GPT Tasks

Related Issue: #11

Test


When suggesting --port in Getting started, clarify that npm run needs -- to pass additional arguments

When I try to run the dev environment for a project initialized via npm init "@observablehq", it fails with:
Unexpected error: listen EACCES: permission denied <IP_ADDRESS>:3000

Simply because I have indeed another service running on that port. But there is no apparent way of providing an alternative port.
Official docs suggest using --port 3001 but it also fails with:
observable: Unexpected argument '3001'. This command does not take positional arguments. See 'observable help preview'.

Docs also mention this:

... the preview server will choose the next available port, so your actual port may vary ...

But I doubt it does such a detection.
The way to do it is also passing an additional -- so that parameters are passed to the underlying command properly:
npm run dev -- --port

The preview server does indeed check whether the port is available here:
https://github.com/observablehq/framework/blob/d244fa19d771ebc8871f5d10edd39762c8f1b266/src/preview.ts#L80-L89
However, it’s looking for a EADDRINUSE error. The EACCESS error suggests a different issue, which is that the process doesn’t have permission to open this port. That is surprising, since normally ports >1024 are allowed to be used without needing sudo privileges. Perhaps Framework should treat both EACCESS and EADDRINUSE errors the same way?
The docs are correct here in suggesting --port 3001, they just fail to document npm run’s behavior, which is documented here:
https://docs.npmjs.com/cli/v10/commands/npm-run-script
Generally, we don’t try to duplicate the documentation of other tools in our docs because there’s too much to cover. But we could add a note about needing the -- here.

@anacierdem can you give some context, like what operating system you were running? We haven't been able to reproduce an EACCES error on unprivileged ports.


Support Contrib Agents in Executors

Is your proposal related to a problem?
Currently we support contrib agents in the YChaos package but do not support it with the executors (Self/Machine). For this, we take in contrib agents config in the testplan here. We need to support these contrib agents to be run on the target as a part of YChaos
What is the solution you are proposing?
This takes a path where the contrib agents can be defined. When we ssh to the target, we need to copy these files to the target and execute them there on the target host. Modify MachineTargetExecutor to copy these files to the remote host and execute them from the remote.
Alternative
Instead of path based configuration, allow package based import, where

User develops contrib agents as a part of a 3rd party package
User adds configuration in testplan to install that 3rd party package on the remote host
MachineTargetExecutor installs the 3rd party package on the remote. (We should also consider security aspect here)
User defines which is the module to be imported in the testplan config.
We import this pythonically in the package and run the Agent class.

My thought is to support both as a part of YChaos,

Allow path based single python file
Allow Pythonic way of importing contrib agents.

Based on the discussion on this thread, we can take a decision to implement one/both.
TODO
I will be adding technical details(Ansible docs etc) for each of the way in the comments of this thread.

The file based approach uses the Ansible copy module to copy the Python File to the remote machine and executes it while running the agents on the machine.
Here are some of the things to be noted with this approach

The path entered in the testplan will be different than the eventual path of the file on the remote host. This must be taken care of.
We must make sure the file copied is deleted from the host in order to prevent others from accidentally executing the script

The alternative module based approach needs to do this for YChaos

Take in configuration parameters like 3rd party packages to be installed along with YChaos in the venv
The module, classname, config class name to be taken as a part of the testplan
YChaos will use the optional import helpers to import these classes in code and execute it when YChaos is run


Cop idea: first(...) instead of all(...)[0]

https://www.rubydoc.info/gems/capybara/Capybara%2FNode%2FFinders:first
# bad
all('td')[0]

# good
first('td')

I just came to open an issue about that. :smile:
By the way, another case that could be replaced with first is find(..., match: :first). @ydah do you want me to open a new issue about that or you'll handle it here?

@boris-petrov Supported by https://github.com/rubocop/rubocop-capybara/pull/118 . If this is not a problem, I will merge it.

Seems great! Thanks for the quick implementation as usual! :)


2 lines,icon

There are spaces before entering the text
macOS

Does this bit from the troubleshooting guide apply?
https://github.com/bhilburn/powerlevel9k/wiki/Troubleshooting#strange-character--segment-spacing

Yes,Thanks.
I have some icon not work.
POWERLEVEL9K_MODE='awesome-fontconfig'

What font are you using and how did you install it?

follow this

I use this, that works.

To fix the sizing issue, tick "Use a different font for non-ASCII text" and set the font to 13pt DejaVu Sans Mono Oblique for Powerline.

not fix

You can set the Font to DejaVu Sans Mono for Powerline and the Non-ASCII Font to the nerd font to solve the issues or you can use the nerd font for both. The Powerline font does not contain the code points (icons) for the missing icons.

Thanks.


feat: updates multihash table

adds script to auto update
reduces bundle size by having only one explicit table ( 3kb gziped diff )
remove defaultLengths seems to not be used and multicodec tables doesn't have it

closes https://github.com/multiformats/js-multihash/issues/73
BREAKING CHANGE: defaultLengths removed, skein is now lowercase

also breaks by changing case of some of the names, like Skein -> skein
but very good change though! I wondered why this was manually replicated and I guess that's just historical.

@hugomrdias could you please force-push this commit again with an updated commit message that also makes the "Skein" casing change as breaking change (this way it should show up nicely in the changelog). It could be done while merging, but i also forgot doing those things quite often :)

@jacobheun FYI changed the maintainer to me

This contains a breaking change, it should have gone out as v0.5.0.
ipld-git is currently broken as it depends on ~0.4.14 of this module and accesses the defaultLengths property from /src/constants.
Please can you re-release 0.4.19 as 0.4.21 then release 0.4.20 as 0.5.0.


Add missing jdkHome and languageVersionProp properties to DetektCreateBaselineTask

This is a minimal version of https://github.com/detekt/detekt/pull/6214 to fix #6167 only in a point release (1.23.1) given that 2.x will come with major changes in these tasks.

Warnings

:warning:
It looks like this PR contains functional changes without a corresponding test.

  Generated by :no_entry_sign: dangerJS against ff8b924804d590f8a3b46a2c335634b462e22cdf

@cortinico

Note that I set the base branch to release/1.x, not main, with the intention to release this in 1.23.1 only based on my understanding of prior discussions. Please change it if that's not correct.

Let's merge this to main and I'll pick it on the release branch


Geometry Simplification

definition of simplification:

Simplify polygons just enough to meet size limitations of tools like BQ, Tableau etc while still retaining as much detail as possible
The goal is to simplify a very complex geometry “just enough” to hit some size limit.
Sometimes, tools like BQ, tableau, etc. have size/complexity limits when dealing with polygon geometries. This usually happens in the case of country polygons. In these cases, we want to simplify the polygon just enough to be within said limits, but not over-simplifying such that we unnecessarily lose too much detail.

I think we can close this issue you can use geopandas's simplify and setting a meter threshold instead of a file size threshold. A file size threshold is also tricky to compute exactly since it may change based on the hardware and the dependencies


Tweaks

Proposal
Added few tweaks here and there. Most are just styling but also fixed a bug from reports page when "start_time" and "end_time" are not present, which broke the component from rendering. Also not sure if router.get("/reports/customer/:id", ...) route is used somewhere at all or it can be removed?
Also added phone number validation as one of the TA's asked in our first or second presentation about it. I used google-libphonenumber to validate numbers for Gread Britain's format. It validates not only the format (as regular expressions) but also for actual numbers. For whatsapp numbers I skipped it as I suppose it's possible to have whatsapp number from another country? I am also not totally sure if restricting phone number to Great Britain is ok or it is possible to have numbers from other countries?
Also made pay rate optional, so this PR would close #20.
Checklist

[x] I have made this pull request to the master branch
[x] I have run npm run lint and there are no errors

It shouldn't be there, thanks. I will remove it with other PR :+1:

It shouldn't be there, thanks. I will remove it with other PR :+1:


network_ap_config not working as expected

First of all wanted to thank you for creating this software.
Trying it out to manage 33 Samsung TV's.
I was trying to change the network from one SSID to another with network_ap_config, but it doesn't seem to work.
It disconnects the TV from current network, but doesn't change over to the new network.
I have only tested this on a QM55R-B so far with Software Verison: T-KTM2ELAKUC-2330.0, will try with some other models we have when I have some time.
Running Samsung-MDC on Ubuntu 24.04 LTS with Python 3.12.3
samsung_mdc@samsungmdc:~/samsung-mdc$ ./venv/bin/samsung-mdc 0@<IP_ADDRESS>:1515 power ON
0@<IP_ADDRESS>:1515 <POWER_STATE.ON:1>
samsung_mdc@samsungmdc:~/samsung-mdc$ ./venv/bin/samsung-mdc 0@<IP_ADDRESS>:1515 network_ap_config SSID PASSWORD
0@<IP_ADDRESS>:1515 MDCReadTimeoutError: [Errno Response header read timeout] b''

looks like it works exactly as reported before: https://github.com/vgavro/samsung-mdc/issues/10
if you're SURE that device is not changing network (how can you be sure?) - I don't understand reason than. If you eventually do - plz report so I can add to readme. As for now - it LOOKS like it's trying to reconnect whether SSID/PASSWORD right or wrong.
just a little clarification - command SENT SUCCESSFULLY (and confirmed by TCP), command is waiting RESPONSE after sent - response HEADER (exactly first byte) - didn't came in meaningful timeout. Only reason is - TV/Monitor is dropping current network on successfull command receive BEFORE sending acknowledge (as required by protocol).

@YFornasari was your problem resolved? have you tried with wrong SSID/PASSWORD and got same results? are you sure that network wasn't changed (by pinging samsung mdc in parallel, for example) ?

Unfortunately still can't get it to work.
The TV disconnects from the current network, if I reboot the TV I reconnects to the current network.
Looks like it disconnects before sending the SSID and Password to connect to the other wifi.
When I try a wrong password I get the same result.
Tried with another tv (Samsung QB49R), same result.

Van: Victor Gavro @.>
Verzonden: zaterdag 3 augustus 2024 22:12
Aan: vgavro/samsung-mdc @.>
CC: YFornasari @.>; Mention @.>
Onderwerp: Re: [vgavro/samsung-mdc] network_ap_config not working as expected (Issue #16)
@YFornasarihttps://github.com/YFornasari was your problem resolved? have you tried with wrong SSID/PASSWORD and got same results? are you sure that network wasn't changed (by pinging samsung mdc in parallel, for example) ?
—
Reply to this email directly, view it on GitHubhttps://github.com/vgavro/samsung-mdc/issues/16#issuecomment-2267125395, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWXV6VOXKZBBDIMICFR3FP3ZPU2TLAVCNFSM6AAAAABL4EI4HKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRXGEZDKMZZGU.
You are receiving this because you were mentioned.Message ID: @.***>

@YFornasari thanks for your tests!
So what we know for sure:

it disconnects before sending response (ACK) (reported by 2 people from at least 3 devices) - for wrong AND proper  SSID/PASSWORD. "Looks like it disconnects before sending the SSID and Password to connect to the other wifi." - for sure it's the issue, I agree with you.
it connects successfully after that by at least 1 report for proper SSID/PASSWORD...

My guess is - SSID/PASSWORD may be wrong OR wifi connection to this specific ssid unsupported by samsung device for any reason - some unsupported WPA/WPA2 algorithms, other wifi compatibility (newer ieee standard than expected or frequency), samsung bug with tricky password?, wifi is too far away, etc)

Forgot to mention that I can actually connect to the network when I use the remote to type in the password.
Just doesn't work with Samsung mdc network_ap_config.
Double checked the password and it's definitely correct.
To be sure I tried connecting to another different SSID, but still no luck.
After using network_ap_config and looking in the wireless settings of the TV I still have to type the password from the network I pushed. However I can connect to the old network without typing the password.

@YFornasari thanks for your very detailed report, it will be very useful to know this in context of other reports on this matter that may occur later.
Still at this stage - I can't do anything for debug or help unfortunately.
Reset device as last resort, but i don't think it's the case here.
Command is implemented by specification (you can double-check using --verbose option and MDC Protocol specification attached to this repository). For now regarding command itself what I can say for sure - this command ALWAYS returns read timeout (because of network change).
I'll try personally on samsung as soon as i'll be near one, but it may take some time and still may not resolve your issue.

Alright, I'll check once more with --verbose.
If you need me to test anything in the future just let me know.


AdminCommands are loaded/executed although I'm no admin

Hello,
my commands are stored under ./Commands.
my admin commands are stored under ./Commands/AdminCommands
They're loaded as shown in the example bot:
$commands_paths = [
    __DIR__ . '/Commands/',
];
.....
$telegram = new Longman\TelegramBot\Telegram($bot_api_key, $bot_username);
$telegram->addCommandsPaths($commands_paths);

I have an empty list of admin users ($telegram->enableAdmins([]))
and a simple Echo Command (taken from example-bot) that extends AdminCommand and is stored under ./Commands/AdminCommands/AdminEchoCommand.php.
First, when I use /help, the "AdminEchoCommand" is shown under Admin Commands List: section. I supposed it won't be shown since my user id is not in the admins array.
Moreover, I can execute the admin command and I receive a valid result.
Did I make any mistakes? How can I restrict AdminCommands to admins only?
Best regards
xiokax

Can you post your AdminEchoCommand.php file please?

Oh, sure!
<?php
/**
 * This file is part of the TelegramBot package.
 *
 * (c) Avtandil Kikabidze aka LONGMAN<EMAIL_ADDRESS> *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace Longman\TelegramBot\Commands\UserCommands;

use Longman\TelegramBot\Request;
use Longman\TelegramBot\Commands\AdminCommand;

/**
 * User "/echo" command
 *
 * Simply echo the input back to the user.
 */
class AdminEchoCommand extends AdminCommand
{

    /**
     *
     * @var string
     */
    protected $name = 'adminecho';

    /**
     *
     * @var string
     */
    protected $description = 'Show text';

    /**
     *
     * @var string
     */
    protected $usage = '/adminecho <text>';

    /**
     *
     * @var string
     */
    protected $version = '1.1.0';

    /**
     * Command execute method
     *
     * @return \Longman\TelegramBot\Entities\ServerResponse
     * @throws \Longman\TelegramBot\Exception\TelegramException
     */
    public function execute()
    {
        $message = $this->getMessage();
        $chat_id = $message->getChat()->getId();
        $text = trim($message->getText(true));
        if ($text === '') {
            $text = 'Command usage: ' . $this->getUsage();
        }

        $data = [
            'chat_id' => $chat_id,
            'text' => $text
        ];

        return Request::sendMessage($data);
    }
}
?>

It is correct behavior, since your command is under UserCommands namespace
namespace Longman\TelegramBot\Commands\UserCommands;

D'oh! Checked code(s) & structure X times and didn't see it. My focus lied on the extends part. Thank you!
However, as a developer, it's very confusing that namespace plays such an important role. If Command X extends AdminCommand, I assume it is treated as an admin command irrespective of its namespace. Would be worth thinking about it to ease development, I guess.
Thanks again!


NHL Issue

Weird, just pull latest and still the same error
Traceback (most recent call last):
  File "./NHL-Game-Probabilities/NHL/nhl_game_probability_model.py", line 557, in <module>
    main()
  File "./NHL-Game-Probabilities/NHL/nhl_game_probability_model.py", line 551, in main
    date, today_games_df = get_todays_games(param, team_list, team_id_dict)
  File "./NHL-Game-Probabilities/NHL/nhl_game_probability_model.py", line 296, in get_todays_games
    today_games_df = pd.concat([today_games_df, pd.DataFrame.from_dict([{'GameID':games['id'], 'Game State':f"Period {games['periodDescriptor']['number']}", 'Home Team':team_id_dict[games['homeTeam']['id']], 'Home Goals':games['homeTeam']['score'], 'Away Goals':games['awayTeam']['score'], 'Away Team':team_id_dict[games['awayTeam']['id']], 'Pre-Game Home Win Probability':f'{home_win_prob*100:.2f}%', 'Pre-Game Away Win Probability':f'{away_win_prob*100:.2f}%', 'Home Record':home_team_obj.record, 'Away Record':away_team_obj.record}])], ignore_index=True)
KeyError: 'score'

Strange issue here. The code throws no error when I run it now, but I think this happened because the TOR/CGY game was just starting as you ran the program. I think the game state shifted to "live" but didn't show a score because the time was still at 20:00. I'll add something that I think will fix this issue, but it's difficult to tell if it will work because the error is so rare and difficult to reproduce. Try running it again, and let me know if you get an error.

Yeah, running it again worked


ipodtouch0218: Watch kineprohub.com

ipodtouch0218 requests the watch of the watch_keyword kineprohub\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

kineprohub\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Fastnlight in Charcoal HQ


Error displayed in reading mode

In reading mode, the code overlaps with the number of lines.

I can't confirm that. Are you using some custom CSS? What theme are you using? Maybe that's causing problems

After turning off all css code, the problem persists. The theme I am using is AnuPpuccin.

Hi! Unfortunately, I still can't confirm it. I just installed AnuPpuccin Theme, but it is working fine for me.
Try disabling all of your plugins, maybe one of them is causing this behavior. You could send me an example .md file. Maybe there are some invalid characters in the file itself.
Do you see any errors in the console? (Ctrl+Shift+i)
You could also try to create a new Vault and install check if the problem still occurs.
You need to somehow narrow down, what causing this, so I can reproduce it and fix it.

Thanks, I think I found the reason.

Thanks, I think I found the reason.

Just out of curiosity, what was caused this problem?

I apologize for the late reply, it's a problem caused by incompatibility between "better codeblock" and "codeblock customizer".


Add workflow for evaluating predictions

This workflow takes in three parts:

Positive, manually curated mappings
Negative, manually curated mappings
Predicted mappings

And estimates several metrics such as accuracy, precision, recall, and F1 for the predictions. This gives back an estimation of the true metrics, since the positive and negative manually curated mappings likely are not complete and therefore have some bias in which things were curated (e.g., I always curate the easiest first, leading towards a skew that more of my manual curations result in positive calls).
Why is this useful?
Mapping tool competitions don't have to keep writing their own infrastructure for holding their competitions. You just curate (or generate) the target, ask the competitors to generate their predictions in SSSOM, then load it up here
Demonstration
This also comes with a demonstrator by comparing a combination first-party ontology curations combine with third-party Biomappings curations against lexical mapping predictions made by  Gilda. It reports the following when applied to a small number of OBO Foundry ontologies.

prefix
completion
accuracy
precision
recall
$F_1$

chebi
10.8%
98.0%
98.8%
99.1%
99.0%

cl
28.3%
53.7%
90.8%
47.9%
62.7%

clo
52.6%
34.9%
70.0%
38.9%
50.0%

doid
30.1%
26.8%
92.2%
26.3%
40.9%

go
38.0%
80.0%
81.8%
96.8%
88.7%

maxo
44.6%
86.4%
100.0%
86.4%
92.7%

uberon
6.3%
11.2%
98.5%
11.1%
20.0%

vo
66.4%
79.1%
91.7%
77.2%
83.8%

Completion refers to the percentage of predicted mappings that appear in the curated sets (both positive and negative). A higher completion reduces the impact of curation bias. E.g., a completion of 100% means that the metrics are unbiased.
Note that lexical matching has pretty high precision, i.e., most of the predictions it makes are right, but it is more prone to false negatives, so accuracy can vary. Some observations:

This leads to the DOID accuracy being pretty low.
ChEBI has no curations outside of Biomappings, so the number of false negatives is zero, meaning that the accuracy is a less useful metric (TBD, how to communicate that in the table).
CLO has a large number of duplicate terms, which results in an artificially low precision.

[!CAUTION]
Mapping shouldn't be a competition. Make your predictions, curate them, contribute them to Biomappings or directly upstream, then everyone benefits and we don't have to keep playing this game.

Wow this is such a cool idea.. Awesome man!


sctp4nat complie error

Task :sctp4nat:test

connection.AddrUnreachableTest > lostConnectionTest FAILED
java.lang.AssertionError at AddrUnreachableTest.java:202
java.lang.NoClassDefFoundError at AddrUnreachableTest.java:209
connection.SctpChannelTest > sctpChannelTest FAILED
java.lang.NoClassDefFoundError at SctpChannelTest.java:157
core.SctpConnectTest > connectInitTest FAILED
java.lang.AssertionError at SctpConnectTest.java:79
java.lang.NoClassDefFoundError at SctpConnectTest.java:52
core.SctpConnectTest > connectInitTest2 FAILED
java.lang.NoClassDefFoundError at SctpConnectTest.java:84
java.lang.NoClassDefFoundError at SctpConnectTest.java:52
core.SctpInitTest > testAll FAILED
java.lang.AssertionError at SctpInitTest.java:124
java.lang.NoClassDefFoundError at SctpInitTest.java:130
core.SctpServerTest > testAll FAILED
java.lang.NoClassDefFoundError at SctpServerTest.java:42
core.SctpTest > sctpTransmissionTest FAILED
java.lang.NoClassDefFoundError at SctpTest.java:205
7 tests completed, 7 failed
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':sctp4nat:test'.

There were failing tests. See the report at: file:///D:/Workspaces/sctp4nat/build/reports/tests/te
st/index.html

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more l
og output.

Get more help at https://help.gradle.org

BUILD FAILED in 5m 51s
6 actionable tasks: 6 executed

This library is not finished yet, therefore such test errors can occur. Additionally, at the moment, only Linux based systems are supported, because we did not add the .so libraries for windows and mac, yet.
So the question is: what operating system are you using?


Remove redundant require statement from Services module

This should have been removed as part of this commit.

Rebasing against master and force-pushing in preparation for merging.


unclear error for unreachable deployments

Description
env used : main net
version : 2.2.0

unclear error for unreachable deployments when node is down

fixed and verified and improved on devnet

https://github.com/threefoldtech/tfgrid-sdk-ts/issues/1325


Why keep connecting to stanfordcorenlp server

❓ Questions and Help
Hello, I am a new learner in NLP and GNN, I download the project and tried to run this project: (https://github.com/graph4ai/graph4nlp/tree/master/examples/pytorch/summarization)
I follow the guide but it didn't work, Here's What terminnal showed:
**************** MODEL CONFIGURATION ****************
[ Using CUDA ]
out/cnn/gcn_bi_sep_l2_ckpt
Connecting to stanfordcorenlp server...
Connecting to stanfordcorenlp server...
Connecting to stanfordcorenlp server...
Connecting to stanfordcorenlp server...

It kept connecting for hours and nothing happened (no errors, just keep running and never stop). I changed network and used VPN, but it didn't work.
So confused, Can you tell me what should I do?

@uncle-tou , please check this post for the procedure to avoid the "Connecting to stanfordcorenlp server....": https://github.com/graph4ai/graph4nlp/issues/513


Add UMD tests.

Fixes #9

Add RequireJS tests
Add Webpack tests
Add Karma drivers for each
Add lint for new tests
Update documentation
Add code coverage and travis / coveralls support.

/cc @exogen @baer @aisapatino

Testing JS is so complicated! This looks great - :ship:

LGTM :+1:


executor failed running [/bin/sh -c apk add --no-cache bash]: exit code: 1

Starting from a fresh Linux install, the docker compose command of step 4 fails.
The build process of certbot service raises this error:
 => ERROR [evgeniy-khyst/certbot 2/4] RUN apk add --no-cache bash
I couldn't find a way to get around the error. Here is the complete log.
$ docker compose up -d --build
[+] Building 1.6s (27/29)
 => [evgeniy-khyst/nginx internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 91B                                                                                0.0s
 => [evgeniy-khyst/cron internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 91B                                                                                0.0s
 => [evgeniy-khyst/certbot internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 91B                                                                                0.0s
 => [evgeniy-khyst/nginx internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [evgeniy-khyst/cron internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [evgeniy-khyst/certbot internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [evgeniy-khyst/nginx internal] load metadata for docker.io/library/nginx:1.23-alpine                                                                                0.6s
 => [evgeniy-khyst/certbot internal] load metadata for docker.io/certbot/certbot:v1.29.0                                                                                0.6s
 => [evgeniy-khyst/cron internal] load metadata for docker.io/library/alpine:3.16                                                                                0.6s
 => [evgeniy-khyst/nginx 1/7] FROM docker.io/library/nginx:1.23-alpine@sha256:659610aadb34b7967dea7686926fdcf08d588a71c5121edb094ce0e4cdbc45e6                                                 0.0s
 => [evgeniy-khyst/nginx internal] load build context                                                                                0.0s
 => => transferring context: 547B                                                                                0.0s
 => CACHED [evgeniy-khyst/nginx 2/7] RUN apk add --no-cache openssl                                                                                0.0s
 => CACHED [evgeniy-khyst/nginx 3/7] COPY default.conf /etc/nginx/conf.d/                                                                                0.0s
 => CACHED [evgeniy-khyst/nginx 4/7] COPY gzip.conf options-ssl-nginx.conf hsts.conf /etc/nginx/includes/                                                                                0.0s
 => CACHED [evgeniy-khyst/nginx 5/7] COPY site.conf.tpl /customization/                                                                                0.0s
 => CACHED [evgeniy-khyst/nginx 6/7] COPY nginx.sh /customization/                                                                                0.0s
 => CACHED [evgeniy-khyst/nginx 7/7] RUN chmod +x /customization/nginx.sh                                                                                0.0s
 => [evgeniy-khyst/cron] exporting to image                                                                                0.1s
 => => exporting layers                                                                                0.0s
 => => writing image sha256:69c81e0257f83fb06ee9a5efd02ec3e6a2f08f317f56c706ed351002c03a3b7b                                                                                0.0s
 => => naming to docker.io/evgeniy-khyst/nginx                                                                                0.0s
 => => writing image sha256:105657c567975455736bd1d06633a53c56c8def1683b14a58920bf5b7e291ca5                                                                                0.0s
 => => naming to docker.io/evgeniy-khyst/cron                                                                                0.0s
 => [evgeniy-khyst/certbot internal] load build context                                                                                0.0s
 => => transferring context: 91B                                                                                0.0s
 => CACHED [evgeniy-khyst/certbot 1/4] FROM docker.io/certbot/certbot:v1.29.0@sha256:904fd574583ed30b2ebd3e17a4ab953a69589e0d4860c3199d117ad1dd7a4e94                                          0.0s
 => [evgeniy-khyst/cron 1/5] FROM docker.io/library/alpine:3.16@sha256:b95359c2505145f16c6aa384f9cc74eeff78eb36d308ca4fd902eeeb0a0b161b                                                        0.0s
 => [evgeniy-khyst/cron internal] load build context                                                                                0.0s
 => => transferring context: 95B                                                                                0.0s
 => ERROR [evgeniy-khyst/certbot 2/4] RUN apk add --no-cache bash                                                                                0.5s
 => CACHED [evgeniy-khyst/cron 2/5] RUN apk update &&     apk add --no-cache docker-cli docker-cli-compose                                                                                0.0s
 => CACHED [evgeniy-khyst/cron 3/5] COPY renew_certs.sh /etc/periodic/daily/renew_certs                                                                                0.0s
 => CACHED [evgeniy-khyst/cron 4/5] RUN chmod +x /etc/periodic/daily/renew_certs                                                                                0.0s
 => CACHED [evgeniy-khyst/cron 5/5] WORKDIR /workdir                                                                                0.0s
------
 > [evgeniy-khyst/certbot 2/4] RUN apk add --no-cache bash:
#0 0.245 exec /bin/sh: exec format error
------
failed to solve: executor failed running [/bin/sh -c apk add --no-cache bash]: exit code: 1

@rando128, very strange error.
Try running the following command from the certbot folder:
docker build -t evgeniy-khyst/certbot .

Thx!

This error is due to the image being selecting having an incompatible platform (hence exec format error).
In my case, I was running your stack on a Oracle aarch64 (ARM) server. Changing the image to certbot/certbot:arm64v8-v1.3.0 did the trick for me.

Thanks for sharing the solution, @fstolba.
I will update the README

Looks like Certbot added multiarch Docker container  suppory in version 2.4.0 released this month.
After updating to certbot/certbot:v2.4.0 Docker will automatically select the image for yout CPU architecture.


Windows版？

https://github.com/ShirasawaSama/CefDetector
https://github.com/ShirasawaSama/CefDetectorX


becomes not responsive

anyone else having the issue of the program becoming non responsive

I am having the same issue, latest version.


Separate Cython from Python

Separates cython from python as per discussion (https://github.com/github/linguist/issues/5443#issuecomment-879470114).
The color #fedf5b is chosen  as it occurs in the logo and forms a contrast to the python color. Since cython is used in conjuction with python, it would provide better visual  contrast to see this difference. Another option would be #646464 since it also occurs in the logo. However, I personally feel like the fedf5b would be better as it is part of the python logo too.

Thanks for adding the rain for the colour choice to the OP. You're also going to need to update the test by removing the line at
https://github.com/github/linguist/blob/b980593f3fe2bdd1dacb00d918049a87ad9f8f73/test/test_language.rb#L99

Done!


Update apollo-codegen to latest

I'm happy to handle this issue: https://github.com/apollographql/apollo-ios/issues/219 if someone's happy to update this app to using the latest version.

I’ll take care of it!
Sent with GitHawk


feat: e2e sap deployments using stacks

Description
2 stack solutions
SAP ready PowerVS Stack
SAP S/4HANA or BW/4HANA Stack

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline


openbcigui: init at v5.0.9

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

TODO: openbci
java.lang.UnsatisfiedLinkError: Error looking up function 'set_log_level': /nix/store/gjwgyhvydlp2glwihiirvxp8v9hsfmnq-source/libBoardController.so: undefined symbol: set_log_level
	at processing.opengl.PSurfaceJOGL$2.run(PSurfaceJOGL.java:408)
	at java.base/java.lang.Thread.run(Thread.java:831)

Now can be merged, works

Thank you for your work! Could you please change commit and PR titles to app: init at X.X without the v-prefix. Also please split the two packages into two separate commits.

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

1 package failed to build:

openbcigui

1 package built:

neuromore


Visualize the model graph without the docs site

Describe the feature
I would love a way to visualize my dbt graph without launching the docs site.
Describe alternatives you've considered
The docs site is excellent but sometimes I just want to see a quick graph of my model dependencies without spinning up the web app.
Additional context
I worked on a home grown tool that had a command similar to dbt ls but you could pass a flag that would use graphviz to draw the graph.
Who will this benefit?
Anyone developing models and wants a quick look at their graph dependencies
Are you interested in contributing this feature?
I'm interested in contributing. My company uses dbt extensively and I think they could find this useful. Last I checked dbt was using networkx and it does have methods for drawing the graphs so it feels doable.

is there any such tool yet @jsnb-devoted ?

We made one for internal use but I just stumbled upon an open source tool that looks like it does this:
https://github.com/nicholasyager/hotdag
@pablomarambio

@jsnb-devoted
I tried hotdag v 0.7.2 but got
.pyenv/versions/3.9.18/envs/sfdbtsearch/lib/python3.9/site-packages/hotdag/__init__.py", line 36, in _load_graph     compiler.link_graph(linker=linker, manifest=self.manifest) AttributeError: 'Compiler' object has no attribute 'link_graph'
I tried using the https://docs.getdbt.com/reference/artifacts/other-artifacts#graphgpickle with networkx but the results are poor. If anyone knows how the dbt-core code (or if it is done somewhere else) creates the graph, I would like to know which APIs or code parts I should be looking at so I can try and extract the relevant parts.


Bugfix/step attachments

Fix for #95

Codecov Report

Merging #96 into master will decrease coverage by 0.98%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master     #96      +/-   ##
=========================================
- Coverage   88.38%   87.4%   -0.99%
=========================================
  Files          55      55
  Lines        1154    1167      +13
  Branches      113     118       +5
=========================================
  Hits         1020    1020
- Misses         87      98      +11
- Partials       47      49       +2

Impacted Files
Coverage Δ

...mber/json/postprocessors/ElementPostProcessor.java
59.64% <0%> (-7.02%)
:arrow_down:

...java/com/trivago/cluecumber/json/pojo/Element.java
80.67% <0%> (-5.05%)
:arrow_down:

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 50a12a2...6d6fc95. Read the comment docs.


Optimize arguments

use(8 + 8) should be use(16)
But I don't know why it is not optimized.

@mamaar Fixed in #49. Good point.


Show usage for multple disks

Hi again,
Recently I moved my docker-data to another disk to free up space on my boot drive. Now doku shows Docker Disk Space Usage as 13.1 GB but also Disk Usage (21 %) 4.25 GB / 20.6 GB.

Are you able to support multiple disks in the webUI?
Example:

Thanks!

Each of my docker hosts have seperate disks for each docker folder:
Filesystem      Size  Used Avail Use% Mounted on devtmpfs         10M     0   10M   0% /dev shm              24G     0   24G   0% /dev/shm /dev/vda3       5.6G  545M  4.7G  11% / tmpfs           9.5G  1.6M  9.5G   1% /run /dev/vda1       272M   24M  229M  10% /boot tmpfs            24G     0   24G   0% /tmp /dev/vdb         32G   12G   18G  41% /var/lib/docker /dev/vdc        974M   11M  896M   2% /srv/dockercompose /dev/vdd         63G   33G   28G  55% /srv/dockervolumes /dev/vde        9.8G  4.8M  9.3G   1% /srv/dockerlogs /dev/vdf         32G  3.2G   27G  11% /srv/dockerbackups


player.js:327 DOMException: play() failed because the user didn't interact with the document first.

RT.

the same problem here!

Just encountered the same issue:


Deployment cleanup

removed some unneeded files and updated .gitignore

A we talked about we can't remove yarn because that would break deployments, at least as to how the project is currently configured.


Add a library for backoff logic.

The backoff library currently implements two methods:

Exponential backoff.
Fibonacci backoff.

@vishnu-deepsource Probably add tests when Jitter is enabled. The duration returned should be within the acceptable range.

@souvik-deepsource Addressing them.


inconsistent naming of Electrical.Machines.BasicMachines.Components.PartialAirGapDC.flange

Reported by AHaumer on 18 Jan 2009 12:12 UTC
Electrical.Machines.BasicMachines.Components.PartialAirGapDC.flange is erroneously named shaft (doen't match the conversion script!)

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

Comment by AHaumer on 18 Jan 2009 12:21 UTC
solved by ae5d9210a32403891276fdb553d9ceaa7dfcf725, 5477b3bfdf4bf071f3c52dfda612282cc23fdf30 (both maintenance/3.0 and trunk)

Comment by dietmarw on 25 May 2009 08:58 UTC
Assigned to the correct milestone (so it shows up nicely).


support :reload, :reload-all, don't blow away the document

New logic matches ClojureScript master

Thanks again, David!


[clr-datagrid] actions overflow is cut-off at grid boundaries

Select one ...  (check one with "x")
[x] bug
[ ] feature request
[ ] enhancement

Expected behavior
The actions overflow menu should not be cut-off for first/last/any row for any number of actions (at least 5-6 actions should be possible for each row)
Actual behavior
The actions menu is cut-off if each row has more than 1 actions
Reproduction of behavior

https://plnkr.co/edit/XI7L6O?p=preview

Environment details

Angular version: 2.0.X

Clarity version: 0.8.12

OS and version:

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Hi @envyN
I believe this is a duplicate of #37. Please correct me if I am wrong on that.

#37 talks about tooltips and more so within the context of modals. Does the
actions overflow for data grid have the same underlying implementation? If
so then yes this might be a duplicate.
But otherwise, this is specifically for data grid and the actions overflow
within that.
Check out the plunkr I have provided to see the behaviour and triage
accordingly
On 26-Mar-2017 12:34 AM, "Scott Mathis"<EMAIL_ADDRESS>wrote:

Hi @envyN https://github.com/envyN
I believe this is a duplicate of #37
https://github.com/vmware/clarity/issues/37. Please correct me if I am
wrong on that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vmware/clarity/issues/652#issuecomment-289232429, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADPqW_xVx18UQWc9DFHvqdDon11_BoEPks5rpWVGgaJpZM4MpK0F
.

Ok i guess this is being addressed in #649.

It is. Same underlying issue.


clicking the icon is failing often since upgrade to 1.1.9

I have installed today the upgrade to v1.1.9 and since then clicking the icon work first time , and than failed many times. After some time the icon is reacting again.
If I use the short-cut works every time.
I am using MAC book Pro OS 10.13.6
Atom 1.30.0
PlatforIO Home 1.0.2 Core 3.6.1a1

Hmmm @captbaritone do you think you could take this one? It might be related to #239 🤔

Could you tell me which button is failing? Ideally you could show which Atom command the button was bound to.

Hi captbaritone,
I am using platformIO to develop arduino sketch both for Arduino boards and ESP8266 boards.
I do not now how to configure the tool bar, just use it often to compile, upload, open terminal and Serial console.
all this buttons came configured to me when Installed PlatformIO on top of Atom.I am using it for two year now.
The behaviour just changed suddenly this week after I accepted to install some updates , one of them being tool-bar.
First time I click any of this buttons after starting Atom and loading my project, it work OK.
Common sequence would be:
click compile - work OK
click upload - work OK
click Serial monitor - work OK
change some code in sketch...
click upload (will compile and upload) - NOK
if I insist several times (6-10) in a time interval of 1 min, it will eventually work OK and next time will fail again.
I could not yet find a pattern on number of clicks, or elapsed time to have it responde again.
Meanwhile if I use the keyboard shortcut (alt+cmd+U - to upload) it will execute immediately.
let me know how I can help further,
Pedro

@Pedroalbuquerque Thanks so much for reporting this.
@suda I have a fix here: https://github.com/suda/tool-bar/pull/245 Sorry about that.

Just released 1.1.10 with this fix :)

Thank you so much for such a very quick fix. I will test it as soon I get home tonight.

My apologies for breaking your workflow! Thanks for the detailed report.


Edge Case: delegatebw and undelegatebw can cause account limbo (Error: 3080001)

Summary
System contract allows account to unstake tokens if the account has a staking requirement, doesn't clear the bytes consumed, and then places the account in a limbo, where it will not have the bandwidth required to stake the account's tokens. Additionally, it prevents another account from delegating bandwidth as well.
Replicate

delegate bw: cleos system delegatebw eosio testaccount "10 SYS" "10 SYS"
stake bw: cleos system delegatebw testaccount testaccount "10 SYS" "10 SYS"
undelegate bw: cleos system undelegatebw testaccount testaccount "10 SYS" "10 SYS"
unstake bw: cleos system undelegatebw eosio testaccount "10 SYS" "10 SYS"
delegate bw: cleos system delegatebw eosio testaccount "10 SYS" "10 SYS"

The last command throws:
$ cleos system delegatebw eosio testaccount "10 SYS" "10 SYS"
1786650ms thread-0   main.cpp:1083                 operator()           ] act_payload: {"from":"eosio","receiver":"testaccount","stake_net_quantity":"10.0000 SYS","stake_cpu_quantity":"10.0000 SYS","transfer":false}
1786651ms thread-0   main.cpp:419                  create_action        ] result: {"binargs":"0000000000ea30550000000d399ea6c1a0860100000000000453595300000000a086010000000000045359530000000000"} arg: {"code":"eosio","action":"delegatebw","args":{"from":"eosio","receiver":"testaccount","stake_net_quantity":"10.0000 SYS","stake_cpu_quantity":"10.0000 SYS","transfer":false}}
Error 3080001: account using more than allotted RAM usage
Error Details:
account testaccount has insufficient ram; needs 3964 bytes has 0 bytes

Considering delegatebw is from a priviledged account to another account, why would it matter what the target account's RAM allocation is?
If I try to buy ram manually:
$ cleos system buyram testaccount testaccount "100 SYS"
3271366ms thread-0   main.cpp:419                  create_action        ] result: {"binargs":"0000000d399ea6c10000000d399ea6c140420f00000000000453595300000000"} arg: {"code":"eosio","action":"buyram","args":{"payer":"testaccount","receiver":"testaccount","quant":"100.0000 SYS"}}
Error 3081001: transaction reached the deadline set due to leeway on account CPU limits
Error Details:
the transaction was unable to complete by deadline, but it is possible it could have succeeded if it were allow to run to completion

What the account looks like:
$ cleos get account testaccount
privileged: false
permissions:
     owner     1:    1 EOS5q1GZ13iHoDcYE8WSKn6jRe1LDcUpHFKqzLyUNuKUTV97pDheY
     active    1:    1 EOS6MRyAjQq8ud7hVNYcfnVPJqcVpscN5So8BhtHuGYqET5GDW5CV

memory:
     quota:     6.553 Mb     used:     3.871 Kb

net bandwidth: (averaged over 3 days)
     used:               485 bytes
     available:            0 bytes
     limit:                0 bytes

cpu bandwidth: (averaged over 3 days)
     used:             6.676 ms
     available:            0 us
     limit:                0 us

producers:     <not voted>

The balance
$ cleos get currency balance eosio.token testaccount SYS
80.0000 SYS

Note: This account is registered as a producer.
testaccount      EOS5q1GZ13iHoDcYE8WSKn6jRe1LDcUpHFKqzLyUNuKUTV97pDheY  https://testaccount.eos                                       1.0000
someaccount      EOS5q1GZ13iHoDcYE8WSKn6jRe1LDcUpHFKqzLyUNuKUTV97pDheY  https://someaccount.eos                                       0.0000

It's kind of by design - user have to have RAM to run actions that write to DB. In this case someone should buy RAM for this user (cleos buyram).

Actually it's really strange. I cannot explain what's going on here.
@dskvr , did you run any commands except those 4 ? (voting, buying/selling ram)

Note: This account is registered as a producer.

This can explain everything. Producer is eligible for paying for his data. His data changes when someone votes for him. When you delegate bandwidth to someone, his voting power changes. It may potentially affect producers. We have not foreseen that producer's RAM can affect voting/delegation. I need to know more details.
@dskvr, Who has voted for whom in this case?

Now I can reproduce it.
My account has huge RAM quota and low usage:
$ cleos get account anton -j
{
  "account_name": "anton",
  "privileged": false,
  "last_code_update": "1970-01-01T00:00:00.000",
  "created": "2018-05-25T15:28:40.000",
  "ram_quota": "6218815765",
  ...
  "ram_usage": 4292,

And I get this error message:
Error 3080001: account using more than allotted RAM usage
Error Details:
account anton has insufficient ram; needs 3960 bytes has 0 bytes

which contradicts to account.

fixed


Localization for number formats

Ideally the quantity, circulating supply, member count, and holder count numbers should have number formatting with thousands separators.

I think this is already resolved.


Fix titles for many steps

Fixed some Maven warnings.
Shared common versions with parent pom.
Limit the amount of steps displayed on top bar (default -1 which means show all which is the current behavior).
Add getter/setter method for this new parameter.

This is related to issue #27.


README: how to connect via psql

Readers should see the whole flow and know how to connect to their new test cluster in Minikube :smile:

👍

:+1:


[ENH] Implement MetadataProvider, RoaringMetadataFilter, and refactor MetadataFilteringOperator

Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

...

New functionality

...

Test plan
How are these changes tested?

[ ] Tests pass locally with pytest for python, yarn test for js, cargo test for rust

Documentation Changes
Are all docstrings for user-facing APIs updated if required? Do we need to make documentation changes in the docs repository?

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2849
#2848
#2847  👈
#2846
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Sicheng-Pan and the rest of your teammates on  Graphite

Merge activity

Sep 30, 7:32 PM EDT: @Sicheng-Pan started a stack merge that includes this pull request via Graphite.


Support Skin 14

We need to upgrade Skin support from 13 to 14

first problem:
can't import @ebay/skin:
ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve './dist/variables/variables.css' in '/Users/tmanyanov/build/ebayui-core-react/node_modules/@ebay/skin'
<EMAIL_ADDRESS>1:0-41
<EMAIL_ADDRESS> @ ./.storybook/preview.js
 @ ./.storybook/preview.js-generated-config-entry.js
 @ multi<EMAIL_ADDRESS><EMAIL_ADDRESS>(webpack)-hot-middleware/client.js?reload=true&quiet=false&noInfo=undefined ./storybook-init-framework-entry.js<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/preview.js-generated-config-entry.js ./generated-stories-entry.js

@darkwebdev Do you still have the problem? On my branch/PR, it works, yarn build and storybook works

done in v10


Normalized bag of words concept matching not working

Currently the lookup does not insert spaces between words, use the correct bounds of the window subset, or remove punctuation / stop words.

Results of performance test prior to any changes
Precision: 0.5310371075166508
Process Method Duration: '0:00:00.031846'
Recall: 0.6250887784090909

After re-enabling the normalized bag of words:
Precision: 0.3987693291245455
Process Method Duration: '0:00:00.029992'
Recall: 0.64453125
Could be due to over-predicting or the fact that mipacq is somewhat limited in labeled concepts.


Crash reporting for Windows Application Packaging Projects written in C++

I write Windows apps using C++, which rely on use of "Windows Application Packaging Projects" for distribution through the Windows Store. See https://docs.microsoft.com/en-us/windows/uwp/porting/desktop-to-uwp-packaging-dot-net for info on this, if that helps explain how such apps work.
However - it is not yet possible to add AppCenter crash reporting support to C++ apps written in this way to Visual Studio, and I can't see this on your roadmap.
I’ve fed this back to the Visual Studio team probably 3 times in the past year, but nothing seems to have happened.
This would be a game-changer for C++ apps and games for Windows, in my opinion - adding crash reporting support for a potentially vast number of apps.
Thanks for your consideration!

@sseyod thanks for the request! We don't have immediate plans to support crash reporting for Windows C++ apps. We are planning to support crash reporting for WPF/WinForms (.NET Framework) and improve our current support for UWP (.NET native) later this summer. I'll keep an eye out on this thread and we can reevaluate this with enough community support. Thanks!

Note this is related to https://github.com/microsoft/appcenter/issues/436

AppCenter should learn about a company called Microsoft (http://microsoft.com).  For several decades, everyone wrote software for Microsoft using C/C++.  There are thousands or millions of programs written this way. Many developers still write software this way. Maybe it would be helpful for those developers if AppCenter could help Microsoft C/C++ software developers in AppCenter?

Is there any progress? Our whole company is waiting for this matter.

Hello there! Our company is waiting for this too.

We would like to have it too!

Hi @winnie!
Is there any more news on this?
Thanks, Pete

Hey @sseyod! Unfortunately, no updates on this one. We did add a upload custom crashes API so you can now upload your crashes from a Windows C++ app so you can view and manage those reports in App Center. We also support symbolicating those crashes (docs here).
We don't have any immediate plans to build an crashes SDK for Windows C++ apps but I hope this helps a little. I'll update this thread if anything changes. Thanks!

Hi folks,
Just to let you know that Sentry.io have now got crash reporting working for C++ Windows 10 apps. Very well done to them!
More info here:
https://github.com/getsentry/sentry-native/issues/190
@winnie surely it would make sense for AppCenter now to offer a similar feature?
If a 3rd party can do this, you'd have thought that Microsoft could do it too?
Best wishes to all,
Pete

This is still very much needed, especially now that Win32 apps are fully supported on the Windows Store.

I couldn't agree more! I'm still using Sentry.io, which is perfectly fine, but it'd be more convenient to use AppCenter - just like I can already do for my iOS, macOS, Android apps...!
Hi again @winnie - this is my yearly request to ask if somebody at Microsoft could possibly schedule this in ... ASAP!
Pete

Sentry also supports macOS, iOS, Android and most other for t end and backend technologies. Curious why would it be less convenient (disclaimer: I maintain some Sentry SDKS).

Hi @bruno-garcia
I can only answer for myself, but my main reason from a convenience perspective is what we also use AppCenter for Beta App distribution - something not offered by sentry.io (unless you can tell me otherwise), and that just makes AppCenter more convenient. We upload the files, distribute them, and it is all done - apps are distributed for testing (and live use) and we capture the crash reports.
Historically, I've come-over as a HockeyApp user (so I have residual brand loyalty!) - and would like Microsoft to make this a better product! But - I've also done what I can to help sentry.io and support for their Microsoft app crash reporting known about, as it is a great thing to have. Of course: on a related note, AppCenter also doesn't support Beta app distribution for native C++ apps - so that is another "hole" in their offering...
Pete


Embed as UWP Control

Problem Statement
More UWP Control Yea.
Evidence or User Insights
Third Party like Telerik have it.
Third party Telerik
Proposal
Have a sample like Telerik.
Goals
Official UWP Control for the win. Microsoft The best.
Non-Goals
More Choice
Low-Fidelity Concept
High Fidelity from Syncfusion

The calculator is a C++/CX application with a UWP XAML UI, so 100% of the controls you see are already UWP. But I suppose you mean UWP .Net controls (written in C# or VB), in this case it will be a little complicated.
Which control do you miss?

This is a nice idea! You should also post your idea on the Microsoft Community Toolkit page too:
https://github.com/windows-toolkit/WindowsCommunityToolkit/issues
And here:
https://wpdev.uservoice.com/forums/110705-universal-windows-platform/category/193402-uwp-community-toolkit
Someone posted a similar idea some years ago but didn't receive enough upvotes, but Windows changed a lot since Windows 8.1 as well as usage, so you should suggest it again!

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

Thanks for your idea! This is really interesting, and this repo does feel like the right home for discussions on this issue, but I think we need to iron out a few more details first. For example, while we certainly could do this, I think we would want to better understand how many developers would utilize this feature. Leaving this open for discussion to develop the pitch further.

You could enable the Windows UI Library to take in a version of the calculator engine/core and then some common button types/templates that it can use.  I could see it as the equivalent of the Calendar pop-up control, which can be used on a Numeric TextBox - Perhaps a variation of it called CalculatorTextBox.
This Numeric TextBox could also include:

validation for numbers;
arrow up and down to increment the number;
inline calculations | like the value "13 + 5" which when pressing enter changes to 18
extensions for value masking | like telephone numbers, IP addresses, etc

For examples of the incrementing value and inline calculations - just take a look at the Adobe Creative software suite.
Like Date Picker and Calculator controls.

Yeah I agree @mdtauk, I'd see this as more of a special numerical picker type use-case or having a 'smart text-box' like some facet of an excel/one note usage which knows how to process 5 + 3 =  into a result...?

I agree with both of you, it would be a lot easier and faster to create a new user control based on a TextBox and using CalcEngine than create a control using the current Calculator UI (because native, it doesn't support Microsoft Community Toolkit animations/helpers, big refactoring to extract some controls and create a windows runtime component lib, disable access keys, probably some conflicts with localizations, etc...) and CalcViewModel (unfortunately not a Windows Runtime component, so a refactoring will be necessary to be used by a managed UWP app).
Here is how I imagine the feature:
A Textbox allowing users to directly type via keyboard a number or an operation (= 12 + 3)

using this on-screen keyboard layout for tablets and touch screens:

and a "calculator" button displaying a Calculator UI.
I think about 2 different solutions:

My favorite: the button will open a FlyoutPopup hosting a basic calculator UI (similar to the DatePicker control), similar to Telerik/SyncFusion.
Alternative: the button will launch the real Calculator application with Launcher.LaunchUriForResultsAsync(...) but the Calculator UI will display an extra button: "Send result to ". But careful, the calculator isn't available on all Windows 10 platforms (xbox for example).

@rudyhuyn I too would favour a simple calculator UI like DatePicker/Calendar similar to Bing/Google search results too.

However your second suggestion may be better done as a picker.  So like the photo picker on Windows Phone, you tap the control, it opens an instance of calculator, showing only the simple/standard layout, and pressing Enter will offer two results.  One with Order of Operations, and one Iterative Calculation.
You then tap the result you want, and it places the new value into the textbox.  It can be awaited, and all async of course.  Probably should be in a modal dialog with only a close button in the titlebar.

Apologies for the crude mockup (about to update my insider install, so no access to Photoshop, and I wanted to illustrate the thought.
I think a NumericTextBox or NumericBox can have those basic 13+24 calculations as with the Adobe software "spinner" boxes.  As well as the up and down arrow to increment or decrement the value in there.
And then the full on Calculator popup/flyout.
Not sure if there is a need to support Programming or Scientific modes, but I guess it is possible to have that be an option, like when a TextBox hints to the virtual keyboard what layout to use.  CalculatedPicker.CalculatorLayout = CalculatorLayout.Scientific

Thanks everyone for the great discussion on this idea. We had the chance to review this pitch more closely and felt that while this is a great idea, delivering it via the Calculator app may not be the best approach. Coincidentally, microsoft/microsoft-ui-xaml#483 discusses a very similar idea that I think may address the developer need discussed in this thread. I am going to close this issue here, but I encourage everyone to jump into he conversation in the WinUI project.


[Bug]: v29 regression for specifying a tsconfig

Version
v29.x
Steps to reproduce
Baseline, v28

Clone repo: git clone git@github.com:apollo-server-integrations/apollo-server-integration-aws-lambda.git
Install and run tests: npm i && npm t
Note no failures

Errors after upgrading to v29

Checkout reproduction branch: git checkout trevor/jest-v29
Install and run tests: npm i && npm t
Note the following failures:

Expand for error output

ts-jest[ts-jest-transformer] (WARN) Define ts-jest config under globals is deprecated. Please do
transform: {
<transform_regex>: ['ts-jest', { /* ts-jest config goes here in Jest */ }],
},

ts-jest[config] (WARN) message TS151001: If you have issues related to imports, you should consider setting esModuleInterop to true in your TypeScript configuration file (usually tsconfig.json). See https://blogs.msdn.microsoft.com/typescript/2018/01/31/announcing-typescript-2-7/#easier-ecmascript-module-interoperability for more information.

src/index.ts:78:21 - error TS2550: Property 'fromEntries' does not exist on type 'ObjectConstructor'. Do you need to change your target library? Try changing the 'lib' compiler option to 'es2019' or later.
78           ...Object.fromEntries(headers),
~~~~~~~~~~~

Expected behavior
I expect ts-jest to resolve and use the tsconfig that is specified in my Jest config.
Actual behavior
ts-jest does not resolve my specified tsconfig, but instead resolves the default tsconfig.json at the root of the project.
Debug log
Available on request (or can be generated through the reproduction). It is too long to submit the issue.
Additional context
Problem
The errors are a bit cryptic, but I've deduced that ts-jest is no longer resolving the specified tsconfig file in the jest config (below). It is instead always falling back to the default tsconfig.json.
I've also tried updating to the transform configuration format with the same result.
Confirmation steps

Rename the top level tsconfig: mv tsconfig.json _tsconfig.json
Rename the test-specific tsconfig: mv tsconfig.test.json tsconfig.json
Run the tests: npm t
Note no errors

Jest config
import type { Config } from '@jest/types';

const config: Config.InitialOptions = {
  preset: 'ts-jest',
  testEnvironment: 'node',
  roots: ['src'],
  globals: {
    'ts-jest': {
      tsconfig: 'tsconfig.test.json',
    },
  },
  testRegex: '/__tests__/.*.test.ts$',
  verbose: true,
  snapshotFormat: { escapeString: false, printBasicPrototype: false },
};

export default config;

Environment
System:
    OS: macOS 12.3.1
    CPU: (8) arm64 Apple M1
  Binaries:
    Node: 16.17.0 - ~/Desktop/ApolloRepos/apollo-server-integration-aws-lambda/.volta/tools/image/node/16.17.0/bin/node
    npm: 8.18.0 - ~/Desktop/ApolloRepos/apollo-server-integration-aws-lambda/.volta/tools/image/npm/8.18.0/bin/npm
  npmPackages:
    jest: 29.0.0 => 29.0.0

@ahnpnl thanks for the quick turnaround, look forward to trying it out!

I am still getting a bug related to this issue in the newest version v29.0.3 with the patch.
With the globals configuration discussed above, if we ignore the deprecation warning, the correct tsconfig file is sourced.
With the recommended configuration, it instead silently falls back to tsconfig.json.
  transform: {
    "^.+\.m?[tj]sx?$": [
      'ts-jest',
      {
        tsconfig: 'tsconfig.test.json',
      },
    ],
  },

Remove preset will solve your issue.

Indeed it did, thank you @ahnpnl.


versionMapping changes the resolved versions of Gradle projects in POM

Expected Behavior
The version of a project's published JAR is set in the POM as a dependency.
Current Behavior
We noticed after applying versionMapping in Gradle 5.2 to our publications that all the resolved versions of project dependencies (ie: compile project('blah')) have changed in the POM.
Context
This is a bit confusing, hopefully the context makes sense:

We publish versions x.y.z of our project JARs after a successful build, by setting the version attribute in the publication
Before applying versionMapping, x.y.z shows up in the POM dependencies, which is what we want.
After applying versionMapping for api/runtime, the version of our project dependencies in the POM has changed to x.y.
After some digging it appears that x.y is what we set the project.version attribute to.
In other words, we are building release candidates for x.y, so we want all the JARs to be named project-x.y.jar. However when publishing the candidate, we publish all JARs as x.y.z (where z is the build number), so we'd want all POM dependencies to be x.y.z as that is what is resolved/published.

Steps to Reproduce (for bugs)

Unzip attached project
cd to gradle-playground/projectB
gradlew publishMavenJarPublicationToMavenLocal
vi ~/.m2/repository/gradle-playground/projectB/1.0.5/projectB-1.0.5.pom
Observe the project dependency to A:

    <dependency>
      <groupId>gradle-playground</groupId>
      <artifactId>projectA</artifactId>
      <version>1.0</version>
      <scope>runtime</scope>
    </dependency>

Even though in A's build.gradle:
    publications {
        mavenJar(MavenPublication) {
            from components.java
            version '1.0.5'
        }
    }

[optional]
6. Remove versionMapping from B's build.gradle
7. Re-run gradlew publishMavenJarPublicationToMavenLocal
8. Observe the POM now has 1.0.5 version correctly set.
Your Environment
macOS 10.14.2
Gradle 5.2
gradle-repro.zip

This is as expected.
When using versionMapping you ask Gradle to replace declared versions with resolved ones. So Gradle will match all declared dependencies and replace their declared version with the resolved one.
However during publication you change the version that a project is published under. That information is not even known by the dependency resolution engine.
You have two options in this context:

Update the build so that the computed version is not only assigned during publication but also during build, effectively setting the project.version property instead of just the publication version.
Stay away from versionMapping if your build cannot be changed as above.

My answer lacks a bit of context on the difference when using versionMapping or not:

When not using versionMapping, Gradle will identify a project through the available publications in the current build. That is why the correct version is used in that case, since the publication for projectA knows about its publication version.
When using versionMapping, see comment above.

Thanks @ljacomet for the explanation!


OData Client: Retrieving entity by Id results in empty result

Hi,
I tried to create an OData Client using your API.
When retrieving a single entity by ID client.getEntities(...) results in an empty object list.
The generated Link I see in the logs get the correct result, when used in browser.
(See: https://github.com/omniavincitlabor/odata4-test/ (TestClient))

Found out myself.
I should call client.getEntity(...) instead of client.getEntities(...).


Support for rai livestreams

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
This Pull Requests aims to support rai live streams listed here.
On youtube-dl seems to work fine, but other players try to get index m3u8 the next fragments, which leads to an access denied error. (if you need them, mpv logs with the two streams)

Add as separate extractor.

yeah @rabblac , I was also working on this extractor and I noticed that mpv does not handle it good. I don't understand why because it uses ffmpeg as youtube-dl. The only solution for now i found is to use pipes:
youtube-dl http://raiplay.it/dirette/rai3 -o - | mpv -

That's why mpv asks every time for the index m3u8 for new fragments, meanwhile youtube-dl increments by one the fragment.
Anyway, thanks for the fast reply dstftw, I'm working on this.

As already mentioned by @gfabiano you should delegate extraction to RaiPlayIE not copy paste code.

Sure thing, didn't know that was possible. Commit sent.


[CI] add no change diff find comment

SSIA

JaCoCo Code Coverage 76.92% :white_check_mark:

Class
Covered
Meta
Status

  Generated by :no_entry_sign: Danger


[BUG]: ASAN Memory Leak

ASAN is reporting a memory leak with the lapic code when the unit tests are run.

Fixed


Improve mobile view

The portrait and main title were misaligned on some mobile screens. They are now being automatically resized based on the viewport width.

I like this improvement a lot, thank you!


Not usable because outdated Token Action HUD Core 1.4.0 is needed.

Unfortunately, the module cannot be used because it requires Token Action HUD Core 1.4.0. But Token Action HUD is already available in version 1.5.3.

please update to version 0.0.4 of blade runner token hud.


Fix typo in a link to Effectful Handlers doc

The title says it all. Again, very minor; sorry for PR spam.

Thanks!


imv: 3.0.0 -> 3.1.2

Motivation for this change
Update imv from 3.0.0 to 3.1.2
Also, activate the tests
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build imv

Thanks!


VSCode not working due to System.DllNotFoundException: libc.dylib

I updated to MacOS 10.13.6 High Sierra, and after that VSCode started outputting this error:
Starting OmniSharp server at 9/26/2018, 11:50:42 AM
Target: /Users/jon/Development/FlankerApp/FlankerApp.sln
OmniSharp server started with Mono 5.14.0.
Path: /Users/jon/.vscode/extensions/ms-vscode.csharp-1.16.1/.omnisharp/1.32.5/omnisharp/OmniSharp.exe
PID: 3541
[info]: OmniSharp.Stdio.Host
Starting OmniSharp on MacOS 10.13.6 (x64)
[info]: OmniSharp.Services.DotNetCliService
DotNetPath set to dotnet
System.DllNotFoundException: libc.dylib
at (wrapper managed-to-native) OmniSharp.Utilities.PlatformHelper.Unix_realpath(string,intptr)
at OmniSharp.Utilities.PlatformHelper.RealPath (System.String path) [0x0001c] in <2d953b9925794e189d1fc6b0636cb76a>:0
at OmniSharp.Utilities.PlatformHelper.GetMonoRuntimePath () [0x00050] in <2d953b9925794e189d1fc6b0636cb76a>:0
at OmniSharp.MSBuild.Discovery.Providers.MonoInstanceProvider.GetInstances () [0x0000d] in <631a43cefb904ee98b03a9bc55db8282>:0
at OmniSharp.MSBuild.Discovery.MSBuildLocator.GetInstances () [0x0001e] in <631a43cefb904ee98b03a9bc55db8282>:0
at OmniSharp.MSBuild.Discovery.Extensions.RegisterDefaultInstance (OmniSharp.MSBuild.Discovery.IMSBuildLocator msbuildLocator, Microsoft.Extensions.Logging.ILogger logger) [0x00004] in <631a43cefb904ee98b03a9bc55db8282>:0
at OmniSharp.CompositionHostBuilder.Build () [0x0008b] in <631a43cefb904ee98b03a9bc55db8282>:0
at OmniSharp.Stdio.Host..ctor (System.IO.TextReader input, OmniSharp.Stdio.Services.ISharedTextWriter writer, OmniSharp.IOmniSharpEnvironment environment, System.IServiceProvider serviceProvider, OmniSharp.CompositionHostBuilder compositionHostBuilder, Microsoft.Extensions.Logging.ILoggerFactory loggerFactory, System.Threading.CancellationTokenSource cancellationTokenSource) [0x00076] in <595ea3b3353a4f008e6169409d831743>:0
at OmniSharp.Stdio.Driver.Program+<>c__DisplayClass0_1.
b__1 () [0x00117] in :0
at OmniSharp.CommandLineApplication+<>c__DisplayClass11_0.b__0 () [0x0000b] in <631a43cefb904ee98b03a9bc55db8282>:0
at McMaster.Extensions.CommandLineUtils.CommandLineApplication.Execute (System.String[] args) [0x00039] in <344a1e8cd6594b17b1e01f8df6ae8827>:0
at OmniSharp.CommandLineApplication.Execute (System.Collections.Generic.IEnumerable1[T] args) [0x00042] in <631a43cefb904ee98b03a9bc55db8282>:0 at OmniSharp.Stdio.Driver.Program+<>c__DisplayClass0_0.b__0 () [0x00028] in :0 at OmniSharp.HostHelpers.Start (System.Func1[TResult] action) [0x0001c] in <631a43cefb904ee98b03a9bc55db8282>:0
I'm working in Unity so Im using the C# extension in VSCode. Disabling the extension resolves the issue, but I need it to work properly.
I have tried reinstalling VSCode, updating .NET, reinstalling the C# extension.
Any help with this is greatly appreciated! Complete noob here

This should be resolved in the latest version. Please make sure to update your mono too for best experience https://www.mono-project.com/download/preview/


Bounding box distance gradient is missing

There are two issues with the bounding box:

The gradient is missing. I have implemented it the brach box in my fork please see https://github.com/quagla/SdfLib/blob/box/include/SdfLib/utils/Mesh.h#L48
https://github.com/quagla/SdfLib/blob/box/src/sdf/OctreeSdf.cpp#L116

The choice

return mBox.getDistance(sample) + glm::sqrt(3.0f) * mBox.getSize().x;
creates a discontinuity that it's too big and prevents operations such as ray casting. This needs changing to
return mBox.getDistance(sample, outGradient) + mMinBorderValue;
as done in the other getDistance function. Both changes are in my fork, the reason why I am not creating a pull request is that the fork is several commits ahead.

Hi,
I completely agree with the change. I always tested ray casting with the GPU version and didn't notice that the CPU version already had this distance heuristic. You can propose a new pull request with all the changes. I have seen your other changes and am interested in integrating them.
I'm working on a new termination rule (the rule that decides if a node needs more subdivision) to cover more distance field areas. The idea is to have a requested error that depends on the distance to the object. This way, you can have more precision near the model than at further points. This new rule will allow you to compute octrees covering bigger areas without consuming more memory and computation time. If you are interested, I will notify you when it is ready.

Thanks for the quick reply, I will create the pull request. Thank you for looking at the other changes and yes please keep me updated!


chore(deps): update common-dev-assets digest to 27ef988

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
d9b8616 -> 27ef988

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

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

GitHub release
v6.0.0

Your semantic-release bot :package::rocket: