PDF report not working properly

PDF report not working properly even though the system parameter is correctly set.

I already tried the solution suggested in this issue but still can't make the PDF work properly.
https://github.com/Tecnativa/doodba-copier-template/issues/53
Affected versions: 16
Environment: Development in local machine

Well, that's the way to work. Maybe remove the final slash.

@pedrobaeza I removed the final slash and then logged out and then restarted the container. And then re-login but still PDF report is displayed incorrectly.

Hello, the same error here:
2024-07-04 06:02:03,293 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:03] "GET /web/assets/11489-de3508a/web.report_assets_pdf.min.css HTTP/1.1" 404 - 1 0.011 0.056
odoo-1  | 2024-07-04 06:02:03,295 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:03] "GET /web/assets/11507-28e0575/web.report_assets_common.min.css HTTP/1.1" 404 - 1 0.017 0.049
odoo-1  | 2024-07-04 06:02:03,331 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:03] "GET / HTTP/1.1" 303 - 1 0.009 0.022
odoo-1  | 2024-07-04 06:02:03,402 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:03] "GET /web HTTP/1.1" 303 - 2 0.018 0.047
odoo-1  | 2024-07-04 06:02:04,550 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:04] "GET /web/database/selector HTTP/1.1" 200 - 8 0.049 1.097
odoo-1  | 2024-07-04 06:02:05,609 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:05] "GET /web/assets/11489-de3508a/web.report_assets_pdf.min.css HTTP/1.1" 404 - 1 0.014 0.032
odoo-1  | 2024-07-04 06:02:05,617 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:05] "GET /web/assets/11507-28e0575/web.report_assets_common.min.css HTTP/1.1" 404 - 1 0.010 0.038
odoo-1  | 2024-07-04 06:02:06,657 1 WARNING prod odoo.addons.base.models.ir_actions_report: wkhtmltopdf: Exit with code 1 due to network error: ContentNotFoundError

wkhtmltopdf doesn't find some assets, despite report.url being set with https://localhost:8069, in a droplet used for dev and testing, in the past, it was working but now it is not.

@busybee-darmen in your screenshot, I see that you have installed some extra modules to give enterprise appearance. Why don't you try with a fresh install without that complements?

@busybee-darmen in your screenshot, I see that you have installed some extra modules to give enterprise appearance. Why don't you try with a fresh install without that complements?

@pedrobaeza It was working last week on my local machine even though there are enterprise modules installed.
Also, this only happens in the development environment. While in our production server, it does work properly.

Well, you have to check what you have changed in between. Something done broke it.

Hello, the same error here:
2024-07-04 06:02:03,293 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:03] "GET /web/assets/11489-de3508a/web.report_assets_pdf.min.css HTTP/1.1" 404 - 1 0.011 0.056
odoo-1  | 2024-07-04 06:02:03,295 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:03] "GET /web/assets/11507-28e0575/web.report_assets_common.min.css HTTP/1.1" 404 - 1 0.017 0.049
odoo-1  | 2024-07-04 06:02:03,331 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:03] "GET / HTTP/1.1" 303 - 1 0.009 0.022
odoo-1  | 2024-07-04 06:02:03,402 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:03] "GET /web HTTP/1.1" 303 - 2 0.018 0.047
odoo-1  | 2024-07-04 06:02:04,550 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:04] "GET /web/database/selector HTTP/1.1" 200 - 8 0.049 1.097
odoo-1  | 2024-07-04 06:02:05,609 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:05] "GET /web/assets/11489-de3508a/web.report_assets_pdf.min.css HTTP/1.1" 404 - 1 0.014 0.032
odoo-1  | 2024-07-04 06:02:05,617 1 INFO ? werkzeug: <IP_ADDRESS> - - [04/Jul/2024 06:02:05] "GET /web/assets/11507-28e0575/web.report_assets_common.min.css HTTP/1.1" 404 - 1 0.010 0.038
odoo-1  | 2024-07-04 06:02:06,657 1 WARNING prod odoo.addons.base.models.ir_actions_report: wkhtmltopdf: Exit with code 1 due to network error: ContentNotFoundError

wkhtmltopdf doesn't find some assets, despite report.url being set with https://localhost:8069, in a droplet used for dev and testing, in the past, it was working but now it is not.

I got the same error in the logs.

What do you mean by "droplet"?

Digital Ocean Droplet

Well, maybe there's a limitation there. Try on local host if it's working properly. I have no more clues for now.

I'm closing this issue.
Solution: Delete all the local database and restore a new database.


Atom 'Install Shell Commands' links atom instead of atom.sh

Description
Atom 'Install Shell Commands' links atom instead of atom.sh
Steps to Reproduce

Remove /usr/local/bin/atom
Open Atom from the GUI icon or from atom.sh (e.g. /Applications/Atom.app/Contents/Resources/app/atom.sh on Mac OS X)
Select File -> Install Shell Commands
In a terminal check /usr/local/bin/atom
ls -l /usr/local/bin/atom

Expected behavior:
The atom command line tool is expected to link to atom.sh.
Actual behavior:
The atom command line tool links to the atom binary. Subsequent terminal invocation of atom to fail with errors like #10278.
/bin/bash-886>atom ./
2017-10-10 09:14:07.682 atom[1478:13872] *** Assertion failure in +[SQRLDirectoryManager currentApplicationManager], /Users/joshaber/Documents/Development/GitHub/Squirrel.Mac/Squirrel/SQRLDirectoryManager.m:30

Reproduces how often:
Always.
Versions
Atom    : 1.21.0
Electron: 1.6.9
Chrome  : 56.0.2924.87
Node    : 7.4.0
maOS Sierra: 10.12.6
Additional Information
This has occurred since atom 1.18.0 and onward.
Workarounds
Remove the link and recreate it correctly:
sudo ln -s -f /Applications/Atom.app/Contents/Resources/app/atom.sh /usr/local/bin/atom

Just tried removing my atom link and using the Atom menu to reinstate it, looks correct;
 ls -l atom
lrwxr-xr-x  1 damieng  admin  70 Oct 13 11:09 atom -> /Users/damieng/Source/atom/out/Atom.app/Contents/Resources/app/atom.sh

Code also looks correct; https://github.com/atom/atom/blob/master/src/command-installer.js#L44
If you are able to reproduce this error please respond to this issue and I'll reopen it.


Expose logprob of output tokens

Hi, can you help expose logprob of output tokens? This would be helpful for our use cases.

Hi @yunfeng-scale ,
Thanks for your request, we will take it into account and add it to our roadmap. It's not clear to me, yet, when we will have the bandwidth to add it but we will keep you posted.
Thanks,
Julien

Hi @yunfeng-scale, good news!! this work has been completed & is on main! check out return_log_probs in the config.pbtxt


feat(lodash): expose a plugin to rewrite lodash imports for treeshaking

Current dependencies on/for this PR:

main

PR #34   👈

This comment was auto-generated by Graphite.


k8s_event.py: fix typo: merge_type

Hi,
This fixes a - / _ error in the k8s_event example
Cheers,
Pierre

Actually I'm not sure this PR is correct, is merge_type really used anywhere?


特征提取问题

您好，更新之后还存在一点问题：
在SHGD/feature_extract/extractor.py的111行，使用的SHGD/feature_extract/unipose.py中缺少对应的test函数
发生异常: TypeError
test() takes 2 positional arguments but 6 were given
File "SHGD/feature_extract/extractor.py", line 111, in extract_PANDA_skeleton_features
features=model.test(image,0,vis,key[1],key[2]) # 16,2 -> 1,32
File "SHGD/feature_extract/extractor.py", line 149, in
extract_PANDA_skeleton_features()

Hi, thank you for pointing out the mistakes in my code again. The modified part of the Unipose project is uploaded to this project at https://github.com/Jiaoma/SHGD/commit/7ba4f859b3c7d8095921e5098b5acf1e9dc92fb0. To run the skeleton feature extraction and see the visualization of extracted skeleton points, you need to first install visdom by pip install visdom, run python -m visdom.server, and then cd feature_extract, python feature_extract/extractor.py. The visualization results are shown at http://localhost:8097/.

Thanks for your reply！


Update Fast unit tests report comment + fix possible failure in notify function

What does this PR do?
Some adjustements to fast unit tests:

Improve comment. Use expandable section and link to CI Visibility
Fix failing path in unit test notification job

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Change name/path on ColumnIdent to be final

Codecov Report

Merging #4886 into master will increase coverage by 0.12%.
The diff coverage is 95.65%.

@@             Coverage Diff              @@
##             master    #4886      +/-   ##
============================================
+ Coverage     81.97%   82.09%   +0.12%
- Complexity    10645    10658      +13
============================================
  Files          1292     1292
  Lines         44579    44570       -9
  Branches       5639     5636       -3
============================================
+ Hits          36543    36591      +48
+ Misses         5377     5340      -37
+ Partials       2659     2639      -20

Impacted Files
Coverage Δ
Complexity Δ

...nner/projection/AbstractIndexWriterProjection.java
63.09% <100%> (ø)
19 <ø> (ø)
:x:

.../io/crate/executor/transport/NodeStatsRequest.java
100% <100%> (ø)
7 <ø> (ø)
:x:

.../io/crate/planner/projection/WriterProjection.java
73.33% <100%> (ø)
24 <ø> (ø)
:x:

.../crate/planner/statement/CopyStatementPlanner.java
92.94% <100%> (ø)
20 <ø> (ø)
:x:

...rc/main/java/io/crate/metadata/ReferenceIdent.java
87.87% <100%> (ø)
16 <1> (ø)
:x:

...l/src/main/java/io/crate/metadata/ColumnIdent.java
96.84% <93.75%> (+2.61%)
46 <7> (-3)
:x:

...o/crate/analyze/SafeExpressionToStringVisitor.java
83.33% <ø> (-16.67%)
7% <ø> (-1%)

...ate/operation/projectors/SortingTopNProjector.java
82.5% <ø> (-12.5%)
12% <ø> (-2%)

.../java/io/crate/blob/TransportDeleteBlobAction.java
75% <ø> (-5%)
5% <ø> (-1%)

...in/java/io/crate/breaker/RamAccountingContext.java
80% <ø> (-3.34%)
19% <ø> (-2%)

... and 37 more

Continue to review full report at Codecov.

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


Document the headers being introduced

Currently, the ABNF of both the 'Service-Worker' and 'Service-Worker-Allowed' headers are undocumented, as are the contents.
Concretely:

Review RFC 7231, Section 8.3 for considerations when registering new header fields
Review BCP 90 for the technical requirements (cc @mnot )
Add language for both Service-Worker and Service-Worker-Allowed

From the spec, it would appear that Service-Worker is a request header, whose syntax is token, and in practice whose value is script (but I don't think the ABNF should be so restrictive; leave that for prose)
From Chrome's implementation (but unspecified in the spec), it would appear that Service-Worker-Allowed is a response header, whose syntax is either URI-reference or, due to the fact that it should only contain a path relative to the same origin (scheme/authority) or is path - although it's unclear whether this is supposed to be path-abempty (is "Service-Worker-Allowed: " a valid header? Chrome appears to allowed this, counter-intuitively), path-absolute (does "Service-Worker-Allowed: " always require a leading /? Chrome's implementation appears to resolve it as a relative URL, and thus doesn't require this), path-noscheme (doesn't seem to fit at all), or path-rootless (prohibits absolute). My gut from the spec is that it's probably meant to be path-absolute, although I'm not sure if @annevk has better language.

Update maxScope to be the result of resolving the URL relative to the document's registering script url, since it incorrectly refers to being the result of parsing serviceWorkerAllowed, which is incorrect, since serviceWorkerAllowed is already parsed (presumably, as a URL)

I think Service-Worker-Allowed should match Location in terms of URL parsing and both should allow basically any relative URL. And I think as far as the contents, the browser will allow "any" for either and not require a specific production.

Agree with @annevk, relative urls are fine here, and are supported by the current spec. As for an empty value, I agree that's an indication of error but the result is the same as omitting the header, so allowing it seems fine.
Excuse my lack of knowledge here, but does the service worker spec count as an open standard as per https://tools.ietf.org/html/rfc2026#section-7, or will we have to define these headers in a seperate doc?
Is the goal to get these headers listed at http://www.iana.org/assignments/message-headers/message-headers.xhtml? I don't see content-security-policy there, is that just an oversight?

Update maxScope to be the result of resolving the URL relative to the document's registering script url, since it incorrectly refers to being the result of parsing serviceWorkerAllowed, which is incorrect, since serviceWorkerAllowed is already parsed (presumably, as a URL)

serviceWorkerAllowed is the result of parsing the header, maxScope is the result of parsing serviceWorkerAllowed as a url.

@jakearchibald The spec should clarify that, then - because it's a non-guaranteed behaviour as implemented in Chrome.
open standard - The SW spec is part of the W3C WebApps WG, right? If so, yes, it's already covered, so  no need for a separate doc. You can see some of the Provisional registrations (e.g. the many CORS headers). The absence of Content-Security-Policy I don't recall whether that was because of some of the IETF vs W3C stuff or just an oversight.
re: maxScope - here's what the language currently reads:

Let maxScope be the result of parsing serviceWorkerAllowed with registration's registering script url.

Here's what it probably should be:

Let maxScope be the result of resolving serviceWorkerAllowed relative to the registration's registering script url.

re: maxScope

Running https://url.spec.whatwg.org/#concept-url-parser passing a URL and an optional base URL returns a parsed URL which is used in the rest of the steps. I don't think it has any problem here.

What does Location with an empty value do? Might just be the same as <a href></a>, no? Not sure why we should disallow that, unless it's already a parse error for Location, which I somewhat doubt.

F2F: Will add ABNF definition (referring to 4.2.4 HTTP new-header syntax).

Dont't worry about the 'open standard' thing.


Merge readable and writable buffer component iteration

Motivation:
It should be possible to iterate the components of a buffer, even when it has no readable or writable bytes.
For the purpose of pointer arithmetic, it should also be possible to get a base pointer for off-heap buffers.
It additionally turns out that unifying ReadableComponent and WritableComponent into a single BufferComponent, allow us to make the iteration entirely allocation free for non-composite buffers.
This in turn allow us to simplify the API by removing the internal iteration methods and only keep a single forEachComponent iteration method.
Modification:

Remove WritableComponent and ReadableComponent, and their processors.
And in their place have a single BufferComponent which contain both of the APIs from the readable and writable components.
Add a baseNativeAddress method to BufferComponent.
Change all usages sites of both the internal and external versions of forEachReadable/Writable, to use the new forEachComponent.
Add firstReadable and firstWritable convenience methods to ComponentIterator, which will skip over components that are not readable or writable, respectively.
Add nextReadable and nextWritable methods to ComponentIterator.Next, which do the same.
Change places like EngineWrapper, IovArray, etc. to no longer rely on internal iteration but use external iteration instead.
Remove processor classes that only existed to be used for internal iteration.

Result:
Iterating the components of non-composite buffers is now always allocation-free; no capturing-lambda or iterator objects are constructed.
The APIs are also simpler; fewer methods and fewer classes.
Fixes #12609 and closes #12611

@timothyklim FYI

Looking into the epoll test failure that showed up after the rebase.


improve storybook buttons styling for better accessibility

Summary | Résumé
Improve Storybook Hide code & Copy button styling for better accessibility.
Button styling before:

Button styling after:

It's looking good, but would you be able to add some hover styles to the buttons?

It's looking good, but would you be able to add some hover styles to the buttons?

Good call - I added them in!


Socket error: { "message": "socket hang up" "code": "ECONNRESET"}

I am facing issue { "message": "socket hang up" "code": "ECONNRESET"},  when using Needle (v1.0.0) as client request to call a https server. After several days researching, it located at the needle.js file,
line 600 ~ 622, where it setup the open_timeout at line 600, and end the request at line 618.  Normally, if the remote server can connect in short time, the open_timeout timer will be clear at line 416, but in case the open_timeout is set very short (I set 10 ms for my case), or remote server is slow and take longer time to  execute the connection, because, we already call request.end() at Line 618, then trying to call request.abort() at line 416, where we will got the  { "message": "socket hang up" "code": "ECONNRESET"}.
here is my JSON response:
{   "error": {     "name": "Error",     "status": 500,     "message": "socket hang up",     "code": "ECONNRESET",     "stack": "Error: socket hang up\n    at createHangUpError (_http_client.js:211:15)\n    at TLSSocket.socketCloseListener (_http_client.js:243:23)\n    at emitOne (events.js:82:20)\n    at TLSSocket.emit (events.js:169:7)\n    at TCP._onclose (net.js:486:12)"   } }
I hope somebody can help to review this issue, as this issue cause an continuous memory consumption in my case, where its keeping creating an object called "TLSscoket" for each call and never release them, even force node GC.

Hi @yangguoyk2015,
Does it also happen on older versions? Say, 0.8 or 0.9?

@tomas , i repeated same error even by v0.8.0, v0.9.0. As i can see they are using the same way to abort request. And going deeper, i found the issue is thrown out from had_error function in needle.js.
  function had_error(err) {     debug('Request error', err);     done(err || new Error('Unknown error when making request.'));   }

I am facing the same issue (v1.1.2).
    at createHangUpError (_http_client.js:252:15)
    at Socket.socketCloseListener (_http_client.js:284:23)
    at emitOne (events.js:101:20)
    at Socket.emit (events.js:188:7)
    at Socket.require.EventEmitter.emit (/Applications/WebStorm.app/Contents/plugins/spy-js/server/lib/tracerNode.js:8:4054)
    at TCP._handle.close [as _onclose] (net.js:492:12)```

and then,
```COOKIE_PAIR is not defined
ReferenceError: COOKIE_PAIR is not defined
    at parseSetCookieString (/node_modules/needle/lib/cookies.js:56:42)
    at /node_modules/needle/lib/cookies.js:32:49
    at Array.reduce (native)
    at Object.parseSetCookieHeader [as read] (/node_modules/needle/lib/cookies.js:29:71)
    at ClientRequest.<anonymous> (/node_modules/needle/lib/needle.js:505:63)
    at ClientRequest.g (events.js:286:16)
    at emitOne (events.js:96:13)
    at ClientRequest.emit (events.js:188:7)
    at ClientRequest.require.EventEmitter.emit (/Applications/WebStorm.app/Contents/plugins/spy-js/server/lib/tracerNode.js:8:4054)
    at HTTPParser.parserOnIncomingClient [as onIncoming] (_http_client.js:472:21)
    at HTTPParser.parserOnHeadersComplete (_http_common.js:105:23)
    at Socket.socketOnData (_http_client.js:361:20)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)
    at Socket.require.EventEmitter.emit (/Applications/WebStorm.app/Contents/plugins/spy-js/server/lib/tracerNode.js:8:4054)
    at readableAddChunk (_stream_readable.js:177:18)
    at Socket.Readable.push (_stream_readable.js:135:10)
    at TCP.onread (net.js:542:20)

Ok I will take a look at this when I have a minute! Thanks for reporting.

@tomas, just for your reference,

seems the socket hangup error is released since node v0.8.20 (my node version is v4.4.7) https://github.com/ether/etherpad-lite/issues/1541

this is the exactly same scenario of our issue. http.IncomingMessage omits events when aborted on client side (https://github.com/nodejs/node/issues/7732), http://www.clock.co.uk/blog/preventing-http-raise-hangup-error-on-destroyed-socket-write-from-crashing-your-nodejs-server, the difference is that my client is node http.clientRequest, the target server is non-node, but a C++ product

from my heap dump, i see there is reference to the needle, attach a screenshot for your reference.

We are seeing the same issue too.

Ok, looking at this right now.

@yangguoyk2015 The issue you linked to refers to something different, which is Node's http.IncomingMessage (server side) not triggering events when the client aborts a request. In your case Needle is aborting the request because you're specifiying a open_timeout which triggers -- that's what causes the ECONNRESET error to be thrown. What are you expecting to happen instead?
@GitTristan Can you please explain better what's your scenario? Are you also setting a low open_timeout via Needle options? What are you expecting to happen when the timeout actually triggers. The second error you posted is something different, by the way. Are you wrapping Needle using Webpack or using a transpiler? Something weird is happening with the COOKIE_PAIR constant.

Ping @yangguoyk2015 and @GitTristan

@maccman Any additional pointers? I'd love to help figuring this one out but I'm kind of stuck.

Have any progress?

@jianzhou520 to be honest this doesn't seem to be an error on Needle's side. You simply get this error because the either the open or read timeouts are being triggered, as discussed on issue #192. So I suggest increasing the open_timeout or read_timeout settings in your code and seeing what happens.
Otherwise please open a new issue with a full example of your case so I can actually replicate the error and find out if there's something wrong. :)

Hey, I've got the same error( But only in deployment in docker container using Cloud Run by Google. When using Heroku hosting I did not have such problem. Also, timeout did  not help, because my server is just waiting for something and....gives me that error.


Restore state not working

please paste your code here

I'm sorry but I don't understand your problem.
Can you better explain it?


Differentiate between onConfirm and onClose calls

I am using SweetAlert2 to alert the user. When I click on the confirm button the behavior is as expected, delete keyword call is made.
render: function() {
return(
<SweetAlert show={this.state.show} title="Context Processor"
                 text="Keyword deletion cannot be undone. Do you still want to continue?"
                onConfirm={this.deleteKeyword}
                showCloseButton={true}
                heightAuto={false}>
                </SweetAlert>
)},

deleteKeyword: function() {
        this.setState({show: false});
        this.props.deleteKeyword(this.state.keyword);
    },

However, when I simply close the alert box call is made then again! I went through the various options provided by the library - https://github.com/kessejones/react-sweetalert2/blob/master/src/ReactSweetAlert2.js but I couldn't find any of interest. Also, added showCloseButton parameter but again when I click on it the delete keyword call is made.
The call is always made to the method passed in onClose and then the method passed in onConfirm is executed. How can I execute different methods for confirming and canceling/closing?

I changed the method to onConfirm={(e) => {this.deleteKeyword(e)}} e tells me if it is confirmed or dismissed.


refactor the organisation of the configuration

refactor the organisation of the configuration folders to solve the ongoing growing number of configurations on each cluster.
The end goal is to refer a config by CLUSTER=<clustername>/<config> make XXX.
both the .sh and .arch files are located in the same folder.
The user can overwrite those folders and also give a separate folder for the .sh and .arch files

[x] create user-defined var CONF_DIR, CLUSTERS_DIR, SUBMIT_DIR, and SUBMIT_CMD
[x] refactor the Makefile to handle properly those variables
[ ] define a variable MOD_FILE to avoid duplication of the module files across configurations
[ ] define a variable SUBMIT_HDR to avoid duplication of the submission headers accross configurations
[ ] make submit needs first to create the submission file and then submit it
[x] add the mpi generic recipe for both mpich and ompi

Due to the profound refactoring proposed and because we all use this branch by default, I will go ahead and merge it to main


Any one used save_map function?

I tried to use save_map function and it does not work?(No error or warning, but no map is saved.)
I use save_map(map_path='path/to/me', data=map_data)

This is for multiplayer support. It saves the map into the temp directory. Hopefully this api will no longer be needed by the time multiplayer is ready to be used.


error Could not find stored procedure Tzdb.Addzone

i have this error on install

Triaging old issues.  Is this still happening?  If somehow this is still a problem for you, and you haven't either solved it or given up, can you send some more troubleshooting information?

What "IDE" are you using (Visual Studio 2015 with SSDT, SSMS directly, notepad++, etc)? (if applicable)
What SQL Server version are you using (both management tools  / client and SQL Engine)? (if applicable)
What operating system and processor architecture are you using?
What did you do?
What did you expect to see?
What did you see instead?


help

How should I add a custom style to the minimap?

How should I add a custom style to the minimap?

Create round mini maps or of any shape, size or style with the Simple Custom MiniMap. No performance heavy render textures or GUI reliance, this system runs perfectly in both Unity Free and Unity Pro. The Simple Custom MiniMap is easy to setup and even easier to customize
Simple Custom MiniMap: Video | Web Demo
Features

Round MiniMaps, square Minimaps, ANY shape
Easily customizable with Unity materials
Use any Unity shader
Easily add particles, UV animations, color lerping, fading etc
Zoom, scale, resize position adjust
MiniMap, World Map or multiple gauges
Does not use GUI or Render Textures!
MiniMap draws in 3 drawcalls
Icons are batched
Control the scene bounds
Fixed rotation or free rotation
Rotation position Speed dampening
5 Demo scenes
Documentation


docs(youtube-player): update loading example to avoid loading API multiple times

Adds some extra logic to the asynchronous loading example that illustrates how to avoid adding the API multiple times.

Hi @crisbeto . Thanks for doing this. I was planning to get to this PR, then work picked up a lot (still pretty busy but I had a chance to come see if anything had happened here!)
I'm slightly concerned that unless I'm missing something, this doesn't actually seem to fix https://github.com/angular/components/issues/20644 or point the implementer in the right direction.
With this implementation it will stop this instance of the component from reloading the script. But if someone has multiple instances of the component on a page, or has this component in a modal that they open and close and re-open then the scope of apiLoaded won't be visible to the other instances of the component. Because of that confined scope we'll still be adding multiple instances of the script to the document body.
I think you're right in terms of cleaning up the example (my proposed solution definitely had a bit of code smell...) - but perhaps where we initiate the variable a comment of // ideally apiLoaded would be a get function that checks the document body for the script and returns a boolean or similar would be beneficial?
I dunno. Maybe as-is is enough. Would be curious to hear your thoughts.

@mbtestfeed the scope in which we have the apiLoaded variable is available to all instance of the YouTube player, even if they've been added at a later point.


Multiline line prompt

I have a feeling this is a really silly question but I cant figure out how to do multi line prompts. I want to press enter to go to the next line but it just submits the prompt. I have tried various other ways but all submit.
Thanks

Hey there!
This is a dup of https://github.com/dustinblackman/oatmeal/issues/16. I’ve attempted implementing a rough version of it myself, but it ends up being pretty complex.
Gonna close this one, we can continue in the other issue. Thanks!


many to many, retrieved data seems not correct

Laravel Version: 5.3.29
PHP Version: 7.0
Database Driver & Version: pdo_mysql mariadb 10

Description:
many to many relationship. query problem
//Aritcle.php
    public function tags()
    {
        return $this->belongsToMany('App\Model\Tag', 'articles_tags');
    }

//Tag.php
    public function articles()
    {
        return $this->belongsToMany('App\Article', 'articles_tags');
    }

Steps To Reproduce:
in the controller, I want to retrieve articles which contains tag name like "this is a tag" which actually does not exist.
$data = Article::with(["tags"=>function($q){
            $q->where('tag_name','this is a tag');
        }])->get();

There are 6 articles in the database, and laravel will give back all these 6 articles, but none of them has any record of tags in the relations attributes by using dd()
It should return 0 record as expected, but it actually returns 6 with no tags.

forgive me. should use Model::whereHas() learning from mistakes


Optimize background activity caching

Goal

Testing

Release Notes

WHAT:
WHY:

#1410  👈
main

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

Join @bidetofevil and the rest of your teammates on  Graphite

Merge activity

Sep 23, 11:21 AM EDT: @bidetofevil started a stack merge that includes this pull request via Graphite.


OnLoadMore is being called on initialize

I used your sample code and added a toast when data is loaded in EndlessScrollListActivity.java, and when the page initializes, the load more is called and unnecessary data is loaded even if user is not scrolled down to bottom of list :(
and also if you try to scroll fast (up and down) and get near to end of the list, you may get stuck into infinite load more!

if there are not enough items, and the end is already in range of loading the next page it will already trigger.
if you scroll really fast. either you load the next page on a much earlier "threshold".

But this is not something that makes sense in production.
Think about a list which is empty and no item is inserted and even if the first request is not made to get first items from server, the second page is triggered to be called and request for the second page data :D

The OnLoadMore listener integrated is a very minimal solution which is ok for most cases. You can easily tweak it or overwrite some logic to adjust it to your needs. You might also want to check out other such listeners, as it is rather independent of the adapter logic itself.
Anyways if you have ideas how to improve it for others, you might want to open a PR

Please open a PR if you would love to improve this. Closing this ticket for the time being

I had the same issue. Solved with a simple tweak.
Initially I disables the listener with disable() method. And called enable() after my data has been loaded in the adapter.


Fancy zone doesn't work on certain apps.

Microsoft PowerToys version
0.75.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
FancyZones Editor
Steps to reproduce
PowerToysReport_2023-10-31-14-53-09.zip
✔ Expected Behavior
Show Fency Zone
❌ Actual Behavior
Even if you press and drag the shift, the fencing zone is not displayed.
Other Software
adobe XD , adobe PhotoShop

having the same problem with 7Zip when I extract via the windows context-menu

having the same Issue with Microsoft To Do App.
fancyZone_nosnap_todo.zip

/dup https://github.com/microsoft/PowerToys/issues/29491


Transitive Provenance

Multiple layers with file downloads from previous layers.
Example:
Job 1 creates two artifacts, 1 jar, and 1 provenance file.
Job 2 is depended on job 1 and downloads the artifacts. We should generate a provenance file based on the existing provenance file from job 1

Provenance files are created on Build Script level. This means one provenance file is create per workflow.
In our assumption 22 hours ago, we though every job created a separate provenance file, so the question how to deal with transitive provenance was valid. Now we know ( #3 ) that multiple jobs create only one file so this issue is not relevant.


[DO NOT MERGE] Modify skopeo tests

The oci name changes in containers/image https://github.com/containers/image/pull/318 caused the skopeo test to fail
Signed-off-by: umohnani8<EMAIL_ADDRESS>

@mtrmac PTAL

@umohnani8 Please update the expected error message:
... value string = "time=\"2017-08-31T15:43:50Z\" level=fatal msg=\"Error initializing destination oci:busybox-latest-noimage:: cannot save image with empty image.ref.name\" \n"
... regex string = "(?s).*Error writing manifest: cannot save image with empty image.ref.name.*"

Thanks!


Documentation: add qingcloud to production user

add qingcloud to production user

LGTM.
/cc @heyitsanthony (can we bypass the commit message checking for this sort of thing? and merge it manually?)

@xiang90 there's no rush to merge these...
@jolestar could you please update the commit message to Documentation: add qingcloud to production user so it matches the commit format in https://github.com/coreos/etcd/blob/master/CONTRIBUTING.md#format-of-the-commit-message? Thanks!

@jolestar
Hi, the commit message needs to be updated not the pull request title.
You can do
git commit --amend

change message to [doucments: add qingcloud to production-users.md]

git push ... -f

@xiang90  done

LGTM. I squashed the two commits into one, and get it merged.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@7b1ccca). Click here to learn what that means.
The diff coverage is 45.69%.

@@            Coverage Diff            @@
##             master    #7426   +/-   ##
=========================================
  Coverage          ?   70.39%
=========================================
  Files             ?      237
  Lines             ?    21226
  Branches          ?        0
=========================================
  Hits              ?    14943
  Misses            ?     5143
  Partials          ?     1140

Impacted Files
Coverage Δ

etcdserver/config.go
77.21% <ø> (ø)

auth/jwt.go
0% <0%> (ø)

embed/etcd.go
67.22% <100%> (ø)

etcdmain/config.go
79.37% <100%> (ø)

embed/config.go
71.16% <100%> (ø)

etcdserver/v3_server.go
62.06% <100%> (ø)

etcdserver/server.go
78.92% <50%> (ø)

auth/store.go
83.81% <59.09%> (ø)

auth/simple_token.go
86.72% <86.66%> (ø)

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 7b1ccca...53705b5. Read the comment docs.


Changes in sponsor

Added extra tier for bronze sponsors
Changed some formatting issues with sponsor-tooltip
Added "current" and "previous" sponsors list
Removed extra bottom-padding in sponsors

LGTM
merge conflicts in Sponsor.jsx, Sponsors.css


Add Zenodo badge

https://zenodo.org/
Allows us to attach a DOI  (Digital Object Identifier) to our package so people can cite us.

Done. Pushed directly to main.


Is it really possible to test the app? How to become whitelisted inside GMS?

Known Issue: "The Exposure Notification API is gonna block you from successfully testing the Application unless you are whitelisted inside GMS."

Dear @gitbrueck,
thank you for reaching out!
Unfortunately this is not something we are able to help you with at the moment, as the authority of whitelisting Google accounts is not within the Android development realm. The general issue of testing the app end to end including the Exposure Notification API should be resolved in near future, as this topic is currently in clarification.
I will leave the issue open until it is resolved, the mentioned known issue will be removed from the list as well.
Best,
Marc

Is the exception "com.google.android.gms.common.api.ApiException: 17: API: Nearby.EXPOSURE_NOTIFICATION_API is not available on this device. Connection failed with: ConnectionResult{statusCode=UNKNOWN_ERROR_CODE(39507), resolution=null, message=null}" an indication that one is not "whitelisted"?

@tallinn1960 Yes, that's the error message you get in this case.
If you want to try out whether your Android device is capable of the service (i.e. if the correct Google Play Services are installed), you could try e.g. the officially published app from Italy. Once such an app is officially published, it doesn't require whitelisting anymore.

To increase visibility to search engines, here is the corresponding stack trace:
W/System.err: com.google.android.gms.common.api.ApiException: 17: API: Nearby.EXPOSURE_NOTIFICATION_API is not available on this device. Connection failed with: ConnectionResult{statusCode=UNKNOWN_ERROR_CODE(39501), resolution=null, message=null}
        at com.google.android.gms.common.internal.ApiExceptionUtil.fromStatus(com.google.android.gms:play-services-base@@17.2.1:4)
        at com.google.android.gms.common.api.internal.ApiExceptionMapper.getException(com.google.android.gms:play-services-base@@17.2.1:2)
        at com.google.android.gms.common.api.internal.zah.zaa(com.google.android.gms:play-services-base@@17.2.1:18)
        at com.google.android.gms.common.api.internal.GoogleApiManager$zaa.zaa(com.google.android.gms:play-services-base@@17.2.1:204)
        at com.google.android.gms.common.api.internal.GoogleApiManager$zaa.zac(com.google.android.gms:play-services-base@@17.2.1:210)
        at com.google.android.gms.common.api.internal.GoogleApiManager$zaa.zaa(com.google.android.gms:play-services-base@@17.2.1:108)
        at com.google.android.gms.common.api.internal.GoogleApiManager$zaa.onConnectionFailed(com.google.android.gms:play-services-base@@17.2.1:75)
        at com.google.android.gms.common.internal.zaf.onConnectionFailed(com.google.android.gms:play-services-base@@17.2.1:2)
W/System.err:     at com.google.android.gms.common.internal.BaseGmsClient$zzf.zza(com.google.android.gms:play-services-basement@@17.2.1:6)
        at com.google.android.gms.common.internal.BaseGmsClient$zza.zza(com.google.android.gms:play-services-basement@@17.2.1:25)
        at com.google.android.gms.common.internal.BaseGmsClient$zzc.zzo(com.google.android.gms:play-services-basement@@17.2.1:11)
        at com.google.android.gms.common.internal.BaseGmsClient$zzb.handleMessage(com.google.android.gms:play-services-basement@@17.2.1:49)
        at android.os.Handler.dispatchMessage(Handler.java:102)
        at com.google.android.gms.internal.common.zzi.dispatchMessage(com.google.android.gms:play-services-basement@@17.2.1:8)
        at android.os.Looper.loop(Looper.java:154)
        at android.os.HandlerThread.run(HandlerThread.java:61)
E/ErrorReportReceiver: [EXPOSURENOTIFICATION]InternalExposureNotificationPermissionHelper 17: API: Nearby.EXPOSURE_NOTIFICATION_API is not available on this device. Connection failed with: ConnectionResult{statusCode=UNKNOWN_ERROR_CODE(39501), resolution=null, message=null}

I installed Immuni officially published from Italy and it worked fine on my device.
Then I rebuild it from source and installed this.
Now I get a message box:
"Google Play Services isn't ready yet
Google Play Services is up to date but cannot enable
exposure notification yet.
Please try again later.
Error code: 39507
GOT IT"
It is not possible to test the sources without being whitelisted.

@tallinn1960 Yes, that's the error message you get in this case.
If you want to try out whether your Android device is capable of the service (i.e. if the correct Google Play Services are installed), you could try e.g. the officially published app from Italy. Once such an app is officially published, it doesn't require whitelisting anymore.

Can you somehow accelerate the whitelisting, so that the app can be tested before being "officially" published. Surely, it's very counterproductive to not allow testers to test the app before it goes public - that's a recipe for failure.

I figured out a limited workaround, that allows you to at least have a peek at what happens after the onboarding process: Use an AVD (Android Virtual Device). It worked with my virtual Nexus 5X API 29. I could do the onboarding flow. I got app-internal warnings, that the Bluetooth API wasn't working, but that was all.

@corneliusroemer After uninstalling a previous build from my Android device, I could build a new version of the app that doesn't crash. (The crash on my device was related to database encryption, so I figured it's easiest to just delete the old database.) Of course, the app still doesn't connect to GMS API, because I'm not white-listed and I don't have the RKI/SAP signing keys.
But when it can't connect to the API, it doesn't crash, it just displays a message.

@corneliusroemer After uninstalling a previous build from my Android device, I could build a new version of the app that doesn't crash. (The crash on my device was related to database encryption, so I figured it's easiest to just delete the old database.) Of course, the app still doesn't connect to GMS API, because I'm not white-listed and I don't have the RKI/SAP signing keys.
But when it can't connect to the API, it doesn't crash, it just displays a message.

@mh- Cool! What did you change to make it not crash? I'm failing to get it to not crash.
@marcmuschko Since you said that the whitelisting problem should be resolved in the near future 6 days have passed. It's now being announced that the app will go live next week. When can we expect the whitelisting issue to be resolved? Saying "not within the Android development realm" isn't good. Please escalate with whomever can do something about this. Otherwise much of the PR is just hot air.

@mh- Cool! What did you change to make it not crash? I'm failing to get it to not crash.

Uninstall on device. git pull. Build. Install again. No crash anymore.


RuntimeError: cuDNN error: CUDNN_STATUS_EXECUTION_FAILED

Hello,
I am trying to reimplementing the pattern branch and have this issue.
There is also a warning saying that "GeForce RTX 3070 with CUDA capability sm_86 is not compatible with the current PyTorch installation. The current PyTorch install supports CUDA capabilities sm_37 sm_50 sm_60 sm_61 sm_70 sm_75 compute_37."
I am using Ubuntu 21.04 and CUDA of version 11.2.
I this the error is due to incompatibility between my rtx3070 and pytorch 1.6. I think I need a combination of pytorch=1.8, torchvision=0.9 to get it work. Do you have an alternative environment for this? ty!

same here +1, seems like that 30 series GPU is not compatible with the environment...

Sorry for the inconvenience (⌣̩̩́_⌣̩̩̀)
I trained and tested the code on RTX 2080Ti, CUDA 10.2, Ubuntu 18.04; I haven't tried any other alternative environment.
So all help is much appreciated and very welcome!

hello，
I'm using RTX 2080Ti, CUDA 10.2, Ubuntu 16.04, cuDNN8.3.0;
and I'm facing the samely issue: a call to cuDNN failed.
Maybe it's not right version of cuDNN?
Can I ask what version of cuDNN do you use?


Audiocity: init at alpha

Project description
Audacity Fork without any Sentry Telemetry or Crash Reporting.
Metadata

homepage URL: https://audiocity.studio/
source URL: https://github.com/AudioCity-Studio/AudioCity
license: gpl2+, gpl-compatiple
platforms: windows, linux, macos

I'm not sure the NixOS team would accept a contribution like this currently.
https://github.com/NixOS/nixpkgs/blob/176b8cce2360c107b9310d3aa8d4c13a1b745d0a/pkgs/applications/audio/audacity/default.nix#L68-L71

I propose to abandon this in favor of https://tenacityaudio.org/, judging by the contribution activity of both forks.


Fix edit theme dialog layout again

Ensures that the layout looks better and doesn't get out of place in the smallest windows and largest fonts, as well as in both English and Chinese environments.

🟢 Before

💙 After

This always increases the bottom height. In fact, I hope to avoid scroll bars as much as possible. We may consider automatic line wrapping.

Try this

Try again

This is reasonable, all we need to do is prevent the UI from being accidentally cut off.


To display a better error message

What happened
When a user deleted a existing process, and kept the duplicate one. Than search the deleted process, the page should display a better error message than display status code 404.
Steps to reproduce

Click drop down list on a existing process and than click "Duplicate", then duplicate a process
Delete the original process
Search the original process, than the page display an error message with status code 404

Expected results
The deleted process page should display a better error message, For example, This page has been deleted.
Actual results
The deleted process page display a message with status code 404

We should send an error message from the backend when a users try to access a process that do not exists

As there are no suggestions made for the improved error message I implemented a simple statement: ```C#
$"Unable to find project id: {query.Id}"

ready for test in test.flyt.equinor.com

ready for test in test.flyt.equinor.com  use link to a deleted process to view the error message, e.g. https://test.flyt.equinor.com/process/514
@QiJin-Bouvet

Tested with commit 65a96ea in TEST env.
Error message shown as below when process not found.


CLI arg for n-survivors to generate a SystemExit

Add a CLI arg e.g. -x that is a tolerance number for survivors. If the survivor count is equal (or greater) than this number raise SystemExit - could allow for CI integration to produce failure state.

https://github.com/EvanKepner/mutatest/releases/tag/0.9.2


imdb dataset cannot be loaded

Hi
I am trying to load the imdb train dataset
dataset = datasets.load_dataset("imdb", split="train")
getting following errors, thanks for your help
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/idiap/user/rkarimi/libs/anaconda3/envs/internship/lib/python3.7/site-packages/datasets/load.py", line 611, in load_dataset
    ignore_verifications=ignore_verifications,
  File "/idiap/user/rkarimi/libs/anaconda3/envs/internship/lib/python3.7/site-packages/datasets/builder.py", line 476, in download_and_prepare
    dl_manager=dl_manager, verify_infos=verify_infos, **download_and_prepare_kwargs
  File "/idiap/user/rkarimi/libs/anaconda3/envs/internship/lib/python3.7/site-packages/datasets/builder.py", line 558, in _download_and_prepare
    verify_splits(self.info.splits, split_dict)
  File "/idiap/user/rkarimi/libs/anaconda3/envs/internship/lib/python3.7/site-packages/datasets/utils/info_utils.py", line 73, in verify_splits
    raise NonMatchingSplitsSizesError(str(bad_splits))
datasets.utils.info_utils.NonMatchingSplitsSizesError: [{'expected': SplitInfo(name='test', num_bytes=32660064, num_examples=25000, dataset_name='imdb'), 'recorded': SplitInfo(name='test', num_bytes=26476338, num_examples=20316, dataset_name='imdb')}, {'expected': SplitInfo(name='train', num_bytes=33442202, num_examples=25000, dataset_name='imdb'), 'recorded': SplitInfo(name='train', num_bytes=0, num_examples=0, dataset_name='imdb')}, {'expected': SplitInfo(name='unsupervised', num_bytes=67125548, num_examples=50000, dataset_name='imdb'), 'recorded': SplitInfo(name='unsupervised', num_bytes=0, num_examples=0, dataset_name='imdb')}]
>>> dataset = datasets.load_dataset("imdb", split="train")

It looks like there was an issue while building the imdb dataset.
Could you provide more information about your OS and the version of python and datasets ?
Also could you try again with
dataset = datasets.load_dataset("imdb", split="train", download_mode="force_redownload")

to make sure it's not a corrupted file issue ?

I was using version 1.1.2 and this resolved with version 1.1.3, thanks.

Hello,
I have the same pb with 1.8.0

Hi ! I just tried in 1.8.0 and it worked fine. Can you try again ? Maybe the dataset host had some issues that are fixed now

Hello,
It works fine now :) !
Thanks !

Ran into the same issue on a different dataset. I workedaround this by passing
    verification_mode='no_checks',

to load_dataset method. Ref: https://github.com/huggingface/datasets/blob/871eabc7b23c27d677bc06ae2cc1ec3a2a04b10f/src/datasets/builder.py#L1141
Note that this is a hack before the root cause is solved.


Italian translation of 1.mdx

Hi @sharkovsky, thanks for the advice, it's my first pull request ever so you know...
I'll try to fix the problems. Don't hesitate to give me feedback! Thank you!

@sharkovsky, I've deleted the files in main. Now I'm not sure how to pull and merge the changes in the original huggingface repo to my fork?? It'd be great if you could give me some hints. Also, I've got another file in its final form, do I create a new pull request each and every time a file is done? Thanks!

Ciao Caterina!
I'd love to help. Are you using the command line for git, or some other tool/GUI?
Git has this concept of remote: you should have two right now, origin which points to your forked repository, and upstream pointing to the original huggingface repository.
By default you usually push and pull from origin, but if you want to integrate the latest changes from huggingface you need to pull the changes from upstream. With command line, this requires:
git pull upstream main

When you git pull it will fetch the changes from the main branch of the upstream remote repository, and try to merge them. I don't expect there to be conflicts, but if there are you can try to solve them by manually updating the files with conflicts and running
git merge --continue

in extreme cases, if you are getting really confused you can always
git merge --abort

If you need, we can set up a google meet or some other quick call!

Hi @CaterinaBi , how are things going? Let me know if i can help in any way!

Hi @CaterinaBi , how are things going? Let me know if i can help in any way!

Hi @sharkovsky , I'm so sorry I ghosted you, I had all sorts of weird things happening to me: I first found myself without any network on the Italian Dolomites for 10 days (thankfully I had planned to take time off from work, so I didn't have any planned meetings!), then I came back to England and realised I forgot my sim card in my mother's phone when she brought me her Italian sim to see whether I'd be able to get some work done using that as a hotspot (that network was terrible too, by the way). So I was locked out of all websites and applications that required a double identification. I should have had GitHub security coded but couldn't find them anywhere... A modern nightmare!
Anyway, I'll be trying to get things right tonight and come back to you if I see I can't update the pull request alone. Because of all that happened I'm also so late with the translations, I feel super guilty!

 Please let us know if you need any help :)

Welcome back @CaterinaBi 🤗 ! Please let us know if you need any help :)

Hello @lewtun! Sorry I disappeared, I feel so bad about this! I've tried to update the pull request, will you please let me know if I've done all correctly this time?
Just to double check: how do you want me to proceed? Do I open a pull request each time I'm done translating a file (and updating the ToC)?

Just to double check: how do you want me to proceed? Do I open a pull request each time I'm done translating a file (and updating the ToC)?

Yes, this would be great! From other translations, I think it's a bit faster if you group a few sections together in a single PR, but of course we'll happily accept single sections 🤗

Thanks for the final fix! I'm now running the CI to check if the docs build and we should soon see the first part of the course in Italian 🇮🇹 !

Great! Thank you very much @lewtun, I owe you a couple of drinks!

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Woohoo it worked 🥳 ! Merging this now :)

Yay! Thanks again!


NTR liver resident NK cells

new term added and refs

#gogoeditdiff

#gogoedidiff

#gogoedidiff

I think this is the last remaining term lost to clashing ID issue.  I'll try to recover.

@gouttegd - great to see this taxon constraints running.  Trying to work out reason for this violation.
I don't see where this is coming from:

hepatic pit cell has:

natural killer cell has

As I understand it 'present in taxon' is very weak.  It just means that some cells of this type exist in the taxon, not all and so doesn't clash with the "never in taxon" on the more specific cell type.

Thanks for tracing - I think we just need an import statement to ensure that the component axioms are visible to editors in Protege.

The hra_subset component is already imported in cl-edit.owl -- its axioms are visible when you open cl-edit.owl in Protégé.
In fact this is probably how the component got changed in commit https://github.com/obophenotype/cell-ontology/pull/2472/commits/735125198519b71827d27daeb33ff7442a8ce5ff. I assume the editor saw the present_in_taxon 'Homo sapiens' annotation in Protégé and simply changed it from there (possibly without realising it came from a component rather than the -edit file itself).


Empty Prometheus cache before each test case

b/281576844
b/281508753
b/281432500

/hold
Need to verify if we can pass an empty match parameter to delete all

/hold
Putting a hold to address Karl's comment

re-lgtm
/lgtm
Feel free to unhold when you're ready to submit

/unhold

/lgtm


Fix link formatting for real

I'm sorry, I didn't have a dev environment set up and thought I could make a quick fix, but I actually made it stop showing up as a link at all. I'm not sure why the syntax isn't being processed the way I expect, and at some point I'd like to make the code of this page less fragile, but for now here's a fix that I've actually tested.

Obviously I just gave it a sniff test as well, mostly because I wasn't clear on how to do anything more useful.  How would I actually check it?

What I did was set up jekyll based on this doc: https://jekyllrb.com/docs/. Of course instead of creating a new blog (step 3) I used this repo. Also, I was doing this on Fedora, and I had to run (checks bash history) bundle install --path vendor/bundle between steps 4 and 5 to get it to work, though I'm not sure if that's always required.
I've done some thinking about a workflow that would allow you to preview the build automatically. Unfortunately, github pages only permits you to build one branch as the site, and even keeping a separate repository for development would be a little awkward because of the CNAME file - if it's in both repos they'll both try to host on www.octotrike.org and only the one with the right DNS records will actually work. Current ideas include:

Separate dev repo that does not have the CNAME file, and slightly annoying git workflows to keep it that way - I believe if new code always goes into dev and you always merge from dev -> production and never the other way around this could be made to work
Dev branch in the main repo, use free CI like https://circleci.com/ or https://travis-ci.org/ to build a preview of the dev branch, then merge into the master branch if it looks good. This seems like less of a hassle to use but has more moving parts and I would need to test that the CI can actually build a preview you can look at in a web browser.

I'm merging without testing myself, because I don't want to hold up bug fixes any longer than I already have.  I'm going to create a ticket about a preview system & workflow so everyone can use it.  In my dreams, more people contribute to the site on a regular basis, and this would help (e.g. I might notice the footer copyright in PR #3).


[Docs] Rename Zeit to Vercel and update links

This started as PR 8923 but I accidentally deleted that PR's repo/branch so it became unable to merge

(https://vercel.com
/docs/configuration#routey)

ping @iansu @amyrlam


Implements the endpoint to get information about your own user

[x] Write tests

Test cases and test recording are the only things changed.


Build Fails when mirroring this project setup: Undefined symbols for architecture x86_64... 201 errors

This isn't directly related to this project, but I was following your example for setting up LibOVR, search paths, libovr.a... but I get some terrible errors when building. Was wondering if you ever encountered this, and what the fix is:

Thanks!

additional screenshots:

Same setup by putting LibOVR inside Applications


(GH-#265) Add Usefule Snippets

This change adds a few useful code snippets that people might want when
doing puppet module development.

https://github.com/lingua-pupuli/puppet-vscode/issues/265

Published Standalone module here: https://marketplace.visualstudio.com/items?itemName=bhurt.puppet-snippets


RDKTV-12808: WebKitBrowser Preconditions

Allow configuring preconditions with CMake flags.
This is to prevent containers from failing to start due to OCDM socket not being present by adding Decryption precondition on some platforms. This ensures we wait until OCDM is ready so that containers will start without error.

Closing older PRs. Please rebase and submit on latest sprint branch.


[SOCCER/TV,,] — Man United vs Nottingham Forest LiVe 𝐒𝐭𝐫𝐞𝐚𝐦𝐬 FrEe BrOaDcAsT oN TV Guide & 𝐒𝐭𝐫𝐞𝐚𝐦𝐢𝐧𝐠 07 December, 2024

Manchester United will look to return to winning ways in the Premier League when they host this season's dark horses Nottingham Forest in their next game on Saturday.
Ruben Amorim suffered his first defeat since taking over as Manchester United manager when his side were defeated 2-0 by bitter rivals Arsenal in their last Premier League match.
Meanwhile, Nottingham Forest come into this game on the back of a 3-0 defeat to Manchester City in their previous league outing. This was their third defeat in their last five Premier League fixtures.
STREAM:Man United vs. Nottingham Forest LIVE in US on Fubo
The Sporting News looks at the key details ahead of this game, including how to watch the match, kickoff times and the latest lineup news.
Man United vs. Nottingham Forest live stream, TV channel
Here's how to watch this Premier League match in the U.S.:
TV channel: NBC
Live stream: Fubo, Peacock, NBC Sports App, NBC Sports website
This game is available for live TV broadcast on NBC. Streaming options are available on Fubo, Peacock, NBC Sports App and NBC Sports website.
For a limited time, you can get your first month of Fubo for as low as $49.99, a $30 savings. Stream ESPN, ABC, CBS, Fox, NBC and 200+ top channels of live TV and sports without cable. (Participating plans only. Taxes and fees may apply.)
WATCH:The best of Premier League action live in US on Fubo
What time does Man United vs. Nottingham Forest kick off?
This Premier League clash takes place at Old Trafford in Manchester, England and kicks off on Saturday, December 7 at 5:30 p.m. local time.
Here's how that time translates across the U.S.:
Date	Kickoff time

Eastern Time	Sat, Dec. 7	12:30 p.m.
Central Time	Sat, Dec. 7	11:30 a.m.
Mountain Time	Sat, Dec. 7	10:30 a.m.
Pacific Time	Sat, Dec. 7	9:30 a.m.
Man United vs. Nottingham Forest lineups, team news
Amorim said after the Arsenal defeat that he was having to manage the work load of some players carefully, so there could be changes to the lineup for Forest's visit.
Mason Mount is one who could drop out, with Marcus Rashford an obvious replacement despite some backlash to his role in the Arsenal loss, and Joshua Zirkzee could also return to the lineup.
Tyrell Malacia may need to keep his place due to Luke Shaw's latest injury, although it's possible that Diogo Dalot will start on the left as the Dutchman continues to build full fitness.
Kobbie Mainoo and Lisandro Martinez are back from suspension, and Leny Yoro could be in line for his first start in the Premier League after impressing off the bench at the Emirates Stadium.
Man United predicted lineup (3-4-3, right to left): Onana (GK) — Yoro, De Ligt, Martinez — Amad Diallo, Casemiro, Mainoo, Dalot — Zirkzee, Rashford, Fernandes
Injured: Lindelof, Evans, Shaw
Suspended: None
Ibrahim Sangare and Danilo are still working on their recoveries and will not likely be involved this weekend.
Anthony Elanga could face his old club at Old Trafford.
Nottingham Forest predicted lineup (4-2-3-1, right to left): Sels (GK) — Aina, Milenkovic, Murillo, Moreno — Anderson, Dominguez— Elanga, Gibbs-White, Jota — Wood
Injured: Sangare, Danilo
Suspended: None
Man United vs. Nottingham Forest Prediction
Manchester United hold the bragging rights coming into this match, having won six of their last seven Premier League home games against Nottingham Forest.
Nottingham Forest have hit a bit of a slump in recent weeks, which Amorim will hope to exploit to regain momentum. Manchester United should win this game, but Forest will give them a challenge.
Prediction: Man United 2-1 Nottingham Forest

cvcv

ghgh


wip: Get most of outlets working

I think this gets a lot of outlets working, am going to keep working on it, but have never worked on an Intellij plugin before. One of the biggest problems I think with this right now is it would be nice to have the outlet have the type of the controller it's mapped to, otherwise it autocompletes inside class and for data attributes for outlets.

Thank you! I will merge the request and complete the feature.


openssl verification failure

Have followed the above guide but when I try to run an openssl connection verification test as below, I get the following error. Note I am running this command from the same directory as where the certs reside.
openssl s_client -connect <edgeFQDN>:8883 -CAfile azure-iot-test-only.root.ca.cert.pem
Start Time:<PHONE_NUMBER>
Timeout   : 300 (sec)
Verify return code: 20 (unable to get local issuer certificate)

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

ID: 6f6b8e5f-7de7-b40b-ad37-dc9b5c8f7d63
Version Independent ID: 10a9b17d-cbf4-ec07-ae39-953d3d0fd411
Content: Create a transparent gateway with Azure IoT Edge
Content Source: articles/iot-edge/how-to-create-transparent-gateway.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

Hi @davebanthorpe, Are you running the openssl command on Linux or Windows?
If on Linux can you execute/provide the results: ls -la azure-iot-test-only.root.ca.cert.pem
or on Windows, can you execute/provide the results: dir azure-iot-test-only.root.ca.cert.pem
Thanks.
James

Running on windows. Here you go:
PS C:\users\hpe\Documents\certs> dir .\azure-iot-test-only.root.ca.cert.pem

    Directory: C:\users\hpe\Documents\certs

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----        11/7/2018   5:25 AM           1245 azure-iot-test-only.root.ca.cert.pem

PS C:\users\hpe\Documents\certs>

@davebanthorpe the MicrosoftDoc looks more related to this doc here instead of the one attached to this thread.  Is that correct?

Hello @davebanthorpe can you take a look at this MSDN thread?
"
To successfully connect to IoT Edge from a downstream "leaf" device, a few things have to be right:

the name that you used for the 'hostname' in the config.yaml file  a) cannot be localhost, b) needs to be lowercase, c) cannot be an IP address, d) has to be EXACTLY the same name you use in your GatewayHostName parameter, and e) MUST be resolvable from your client device.. i.e. you need to be able to ping it by name.  The reason for this is that the hostname in config.yaml is used by the IoT Edge runtime to generate a certificate (under the covers) that Edge Hub exposes to IoT leaf clients that are trying to connect to it.  The name it uses for the Common Name(CN)/Subject of that cert is the hostname from config.yaml.  So, when the client goes to connect to that device, it will do so by name (gatewayhostname in connection string) and it has to be presented by Edge Hub, a certificate that matches the name it's trying to connect to..  otherwise, TLS negotiation will fail.  If any of this is not the case, update your hostname in config.yaml and restart IoT Edge...   If the command "openssl s_client -connect [config host name]:8883 -CAfile /home/pi/work/certs/azure-iot-test-only.root.ca.cert.pem -showcerts" (where [config host name] is the name you used in config.yaml doesn't return a Verified Ok at the bottom, Edge is not set up correctly.
you have to be able to resolve the name you used in hostname in config.yaml from your client, either via a DNS entry, or an entry in /etc/hosts
I assume when you mean you are connecting via HTTP, you mean HTTPS..   can't connect via regular HTTP
ensure, by running "docker ports edgeHub" that Edge Hub is listening on 443  (should also be listening on 8883 for MQTTs traffic and 5671 for AMQP)
Your "client" MUST trust the certificate presented by Edge Hub.  If you used our scripts to generate your certificate, that means it has to trust azure-iot-test-only.root.ca.cert.pem.   With our other SDKs (C#, python, etc), just having that cert in the trusted root cert store (/etc/ssl/certs) is enough.  I don't know about the C SDK.  At a minimum, if you don't see the cert in /etc/ssl/certs, try to install it using the instructions in the transparent gateway instructions under "Installation on the downstream device"

"
Let us know if this helped unblocking your issue?
Thanks!

HI - I have corrected an issue in the config.yaml (edge hostname) and now the openssl test passes OK. I have then used MQTTBOX to verify I can create a connection to the hub from the local host (same place where KepServerEX is running) and that works fine.
However, I am now getting "unable to find valid certification path to requested target" in KepServerEX when I use the same connection and authentication settings. Any idea?

Hello @davebanthorpe I believe that this topic should be discussed on another forum since it is not fully related with the doc. Can I kindly ask you to open a new issue here: https://github.com/Azure/iotedge/issues and let us know the link so that others can follow the troubleshoot there?
Thank you!

new link: https://github.com/Azure/iotedge/issues/532


Signal can only be connected to a node with an attached script

Godot version:
3.2.2
Issue description:
Connecting a signal through the UI can only be done if the node has a script attached. This means that even if you want to connect a signal to a built-in function, you have to:

Attach a dummy script to the node
Connect the signal
Delete the dummy script

For example, it's useful to remove a node after a set period of time by connecting a timeout() signal of a Timer to queue_free(). There shouldn't be a requirement for a script to be attached to do this.
Steps to reproduce:

Add a Timer to a new scene
Select the Node / Signals tab of the Timer
Double-click the timeout() signal to connect it to a method

The dialog will show a message in red text "Scene does not contain any script." You will not be allowed to proceed in connecting the signal.
With the above workaround, you will be able to type the method name into the dialog and double-click the node to make the connection. That makes it clear that this is just a limitation of the UI design.

You don't need to use a dummy script. When the "Connect a Signal to a Method" dialogue box opens up, you can simply press the "Advanced" option next to the Receiver Method input box to allow connection to a scriptless Node (along with giving you the ability to add extra call arguments, set the signal to one-shot mode, and set the signal to deferred mode). You will see that the words "Connect to Script" will change into "Connect to Node", and the warning will disappear, confirming the presence of this functionality. For example, you can type queue_free into the Receiver Method box, turn "Advanced" on, connect a Button's "pressed" signal to a scriptless ColorRect, and be able to free that ColorRect by pressing the button at runtime.

That's perfect. Thank you for the guidance.


Migrate to Solidity 0.6

Can get identical bytecode between CI and local yay

Can get identical bytecode between CI and local yay


Add code for extraction of library dependent repos

Usually it's a good engineering practice to have a short summary of your changes in PR description.
It's also saves time for reviewers when changes have well-defined test plan: exact instructions on how they intended to be used/triggered. I.e in case of CLI it could be a series of shell commands a user must run on his local environment (assuming he is not super-familiar with the project internals) to test the changes: get input data (wget ...), how to process, and what the expected output is.

Yes, this code use https://libraries.io database.
Thank you @bzz for good links and explanation how it should be.
I will check this tool.
I think that now all this code under active development and everything can be changed.
As you see, it is just python scripts. I need to add tests, CLI, README, etc.
I take care of it soon.
My bad that I begin to do it in master, now I switch to develop.

@zurk It is fine to do this project in master branch, as it is experimental.

Ok, I will continue to work in the master branch then.
And this PR is not actual anymore, so I close it.
Here is new one https://github.com/src-d/snippet-ranger/pull/4.


Spring beans transformer

#2142

Current coverage is 40.00% (diff: 48.88%)

Merging #2143 into master will decrease coverage by 0.06%

@@             master      #2143   diff @@
==========================================
  Files          1591       1594     +3
  Lines         55046      55189   +143
  Methods       10840      10859    +19
  Messages          0          0
  Branches       7552       7589    +37
==========================================
+ Hits          22058      22077    +19
- Misses        32988      33112   +124
  Partials          0          0

Powered by Codecov. Last update 519e10c...27f83cb

ping @emeroad please.


Backport of #95 - rspec rails clash

As this is driving me nuts currently, I've backported #95 for the current 2.x branch. Not sure if this was intentionally left for 3.x, but here it is.

Coverage decreased (-0.8%) to 96.0% when pulling 1583344b6f0d3f642434112c2568ff7938c69a83 on bk-one:backport-rspec-rails-clash into 82634afe2ed868cdd7bf3da0e4dfdbcbf46fa754 on philostler:master.

Coverage decreased (-0.8%) to 96.0% when pulling 1583344b6f0d3f642434112c2568ff7938c69a83 on bk-one:backport-rspec-rails-clash into 82634afe2ed868cdd7bf3da0e4dfdbcbf46fa754 on philostler:master.

Coverage decreased (-0.8%) to 96.0% when pulling 1583344b6f0d3f642434112c2568ff7938c69a83 on bk-one:backport-rspec-rails-clash into 82634afe2ed868cdd7bf3da0e4dfdbcbf46fa754 on philostler:master.

Any update on this?

I handled this by monkey patching my codebase as such:
# Monkey patch. See https://github.com/philostler/rspec-sidekiq/pull/108
if Gem::Dependency.new('rspec-sidekiq', '< 3.0.0').matching_specs.any?
  module RSpec
    module Sidekiq
      module Matchers
        if Gem::Dependency.new('rspec-rails', '>= 3.4.0').matching_specs.max_by(&:version)
          alias have_enqueued_sidekiq_job have_enqueued_job
        end
      end
    end
  end
end

I removed the warn since it was writing out the deprecation warnings at all times, even when the old matcher isn't used.

Any progress on this?

👍 Also needed here.

Now that we've released 3.0 is this change still needed?

There may be others who are unable to upgrade to v3 at this time. Might make sense to merge/tag/release it to provide them relief until they can upgrade.

What @baburdick said .. we haven't upgraded yet. I can let you know once we have.

Now that we've released 3.0 is this change still needed?

We upgraded to 3.0 recently; I could remove my monkey patch and everything is working like before.

I won't comment on whether or not a patch-level release should be released in the 2.x series. That's up to the maintainer and those that have issues with it.

Also upgraded some time ago. Just now stumbling across my now ancient monkey patch, removing it, and seeing tests blithely succeed. All good here.


Show release 1.1.1 on the library manager

Upating the library.properties to match the release 1.1.1
This would fix: https://github.com/NorthernWidget/DS3231/issues/74

This library is already registered in LibraryManager list of repositories:

https://github.com/arduino/library-registry/blob/main/repositories.txt#L2991

Instruction for releasing new version says:

Update the version in the library's library.properties).
Tag the library's repository once more and push the new tag (or create a release if the library's Git-hosting site offers a way to do it, for example with GitHub "releases").

So if this merge request doesn't work we can try with the next version: 1.1.2 by doing both at the same time:

Updating the library.properties file
Creating new GIT tag (v1.1.2)

To check if the new version of the library was added we can check the logs from library manager indexer at:

https://downloads.arduino.cc/libraries/logs/github.com/arduino-libraries/DS3231

Hello,
im pretty confident that this PR works, i maintain arduino libs myself and do it the same way.
(just give it some time after the merge to check for the effect)
Could also add a badge to the readme showing the current version:   See here: https://www.ardu-badge.com/DS3231
Example markdown:
[![arduino-library-badge](https://www.ardu-badge.com/badge/DS3231.svg?)](https://www.ardu-badge.com/DS3231)
results in:

Hi @IowaDave, I've seen you merged some recent merge requests.
What do you think about this one? Maybe we could merge it as well?

Hello, @wiiitek and @designer2k2
Thanks for this work. Will you allow me to ask a couple of more questions here:

Are either of you in a position to initiate a roll-back in the event that updating the "release" causes some unforeseen problem with the Library Manager becoming unable to access to the repo at all?
Will you follow through with changing the release number to 1.1.2 if need be?
Will you recommend and push a PR for the "badge" also, if that makes sense to you?

Asking out of an abundance of caution for two reasons: 1) it's not my repo; and 2) I'm not familiar with nor do I claim to understand concepts such as "release" and "badge" well enough to judge these things for myself. My PR merges have been limited to documentation and a couple of example sketches. I have viewed repo maintenance as being above and outside of my scope. I hope you'll understand and tolerate my reasons for asking.
I do see how updating the release would resolve #74, and would agree to pursue it if the Library Manager listing can be rolled-back in the unlikely event described in question 1, above.
Discussion:
Not concerned about code problems in the repo itself. Anything like that we'd fix with a future PR.
I know that individual Arduino IDE users can roll-back the version they install locally. My concern addresses ensuring a valid "most recent" version being available through the Library Manager to future, new users.
If you need me to merge the PR then please offer your thoughts on the questions I asked. I'll check here regularly until we sort this out.

Hello @IowaDave,
answering your questions:
Are either of you in a position to initiate a roll-back in the event that updating the "release" causes some unforeseen problem with the Library Manager becoming unable to access to the repo at all?

Yes, we all are. Rolling back can be either making a new release with reverted changes, or initiating a delete in the library manager for the 1.1.1. See here for a example: https://github.com/arduino/library-registry/issues/2545  This deletes take some time, so i would prefer to then make a new release rolling back the changes, that's much quicker.
Will you follow through with changing the release number to 1.1.2 if need be?

Yes, will do. It would need then also somebody that can generate the release here in this repo. Here is a guide on how to do that: https://docs.github.com/en/repositories/releasing-projects-on-github/managing-releases-in-a-repository
Will you recommend and push a PR for the "badge" also, if that makes sense to you?

For now, i would say no badges. Let's first have the 1.1.1 rolled out and then see about this more cosmetic badge`s.

Thanks, Stephan
I appreciate your responses. I think I have enough information to feel comfortable taking the first step.
Wondering why it has to be me, is all. Could you help me to understand that, when others here obviously know more than I do?
Still, this PR makes sense to me now.
Observation: The latest release is already tagged as 1.1.1, done by Andrew in July 2022. However, the library.properties file still says 1.1.0.  If I understand correctly, what we are doing with this merge is simply to align the properties file with the current latest release tag.
Action: I will go ahead and do the merge as requested. Please continue to follow this topic. I'd like to see the Library Manager's version reflect the work that has been committed since last July also. Will appreciate your thoughts on how to get us all the way there.
Again, please bear in mind my obvious ignorance about managing library repos for access through the Arduino IDE.

@designer2k2 @wiiitek
I merged PR 79 as you requested on March 19.
In a couple of days, pursuant to the discussion in PR 79, I shall look for 1.1.1 to appear in Library Manager.
Question for you: I'm guessing now that the way to reflect the work committed since Andrew published 1.1.1 last July would be by publishing a version 1.1.2 with a library.properties update and a corresponding new release tag?
Thanks!  David

@designer2k2 @wiiitek
I updated 1.1.1 to be the "latest release" also, as wiitek advised.
By the way, the link to downloads.arduino.cc in wiitek's post for checking the library status does not seem to go anywhere useful.
Noticed that we have more than 80 commits since 1.1.1. If 1.1.1 shows up in the Library Manager soon then I'll feel inclined to publish the more recent work as release 1.1.2.
Your thoughts, please?
Thanks,  David

@designer2k2 @wiiitek
Ok, y'all, we seem to have a mess on our hands after merging this PR and will need your help to resolve it.
Two entries now appear in the Arduino Library Manager:
Entry #1: the old one, named "DS3231", having available versions up through 1.1.0.
Entry #2: a new one, at least I do not remember seeing it previously. It is named "DS3231_RTC" and has no version number.
The "more info" link for both of these entries points to the same repo, this one, which is at version 1.1.1 now.
However, both of the entries install version 1.1.0, based on examination of the library.properties file that comes with the download.
Meanwhile, the PR auto-closed issue #74. Turns out that action was premature.
Our initial goal should be to remove the duplication so the Manager lists just one library, named "DS3231", having the capacity to download version 1.1.1.
I did what you asked me to do. You said you'd help to address any problem.  Now I'm asking for your help. Please follow through and help to resolve this matter.
I hope to hear from you soon.

Hello @IowaDave,
first the good things, the point for 1.1.2 is good. All the latest changes should be released!
Now for the bad news, im afraid that 1.1.1 will not work... Its now over 12h after the merge and still no update from the Library Manager.
Background of my negative thoughts ist that the info file inside the zip file from the release here: https://github.com/NorthernWidget/DS3231/releases/tag/v1.1.1  is stating the version to be 1.1.0.  So strictly its tagged correctly, but the actual source code is not matching to it.
Confirming is the link from @wiiitek (it was missing the / on the end) https://downloads.arduino.cc/libraries/logs/github.com/NorthernWidget/DS3231/
2023/03/20 19:04:16 Checking out tag: v1.1.1 2023/03/20 19:04:16 Release DS3231:1.1.0 already loaded, skipping
So i would suggest to go for 1.1.2 right away. I make a new PR, changing the library.properties to show 1.1.2, then once this is merged a new Release is made with tag 1.1.2 by you @IowaDave.  In this sequence the zip file that automatically is generated cointains the matching version information.
"Problem" with this is that the Version 1.1.1 will never show up on the Arduino Library Manager. Arduino IDE Users will see a jump from 1.1.0 to 1.1.2.  I dont see that as a real problem, if somebody needs 1.1.1 it can always be downloaded from github.
I prepare the 1.1.2 PR now to have that ready.

Checking on the 2 Versions in the Library Manager, yes there are 2, thats looking wrong...
The DS3231_RTC will need some further checking, it seems that somebody has just made a copy from this and submitted it again to the library index, even without changing the url, so it points again this this original repo here...
The PR to get the _RTC version added:  https://github.com/arduino/library-registry/pull/1673
And the _RTC repo: https://github.com/affanhanifathtarech/DS3231_RTC
But the library.properties is pointing to here... https://github.com/affanhanifathtarech/DS3231_RTC/blob/main/library.properties#L8
Lets see about that also later, i would consider to open a PR on the library manager to get this copy removed.

Hello @IowaDave , @designer2k2 , thank you for your work.
And forgive me @IowaDave, I should had asked you before mentioning you in my comment.
I dind't reply earlier, because I was away for the weekend and I am happy to see a lot of progress here :)
The new version (1.1.2) is available in Library Manager:

And I was able to install it with arduino-cli :)
As I understand:

there is this DS3231_RTC library, that should probably be:

either updated regarding description and link
or removed from Library Manager

there will be release version 1.2.0 of DS3231 (this repo) created soon

How can I help?

Thanks to you @IowaDave :)
I like the way you approached the duplicated library issue: updates for the description and link. Will watch the other repository.
I don't know why I expected the new version to be 1.2.0 (it was never mentioned). 1.1.2 is also ok. Cheers!


Failing test due to module dependency issue

Failing test on CDash https://slicer.cdash.org/test/18100882. Full testing dashboard can be seen here.
Here you are creating a widget representation of a Slicer CLI module that you are putting in your Slicer ScriptedLoadableModule. However you are not listing antsCommand as a module dependency of antsRegistration.
https://github.com/simonoxen/SlicerANTs/blob/24ede7611e4574785314d6420bfd51ceeef9b340/antsRegistration/antsRegistration.py#L81
https://github.com/simonoxen/SlicerANTs/blob/24ede7611e4574785314d6420bfd51ceeef9b340/antsRegistration/antsRegistration.py#L27

Thanks! added the dependency here a8b3251

The test that doesn’t have a Slicer main window is not passing. See https://slicer.cdash.org/test/18118093

Thanks again. Although I don't quite understand what causes the issue. The error is the following:

When loading module  "antsRegistration" , the dependency "antsCommand" failed to be loaded.

Should I change the order in CMake?
https://github.com/simonoxen/SlicerANTs/blob/master/CMakeLists.txt#L37-L39


advertismentData is always empty in each onDescovery call of startScanning api

Hello. Why in each onDescovery callback all peritherals has empty advertismentData ?
If i scan devices with nrfConnect app i see all devices and also all advertise data. How to get same data with your plugin ?
{
   "type": "scanResult",
   "UUID": "test",
   "name": "test",
   "RSSI": -49,
   "localName": null,
   "state": "disconnected",
   "manufacturerId": 00,
   "advertismentData": {
     "scanRecord": {}
 }

If there are way to get raw advertising package ?

it is because it is an object with getters. logging it like you do reports it as empty when it is not

Thank you


— upgraded kafka to <IP_ADDRESS> - and heavily re-factored  offset-related co...

...de as this area has changed significantly
— fixed issue with incorrect offset being read in the RESTART mode when starting the app for the first time - which caused an infinite loop
— re-factored a lot of exception handling and offset commits - to be more “safety-oriented”: do not try to fix things after severe failures (like errors from Kafka) - but log errors and try to exit in a consistent state instead, and restart the app when the root cause of the issues is fixed. Often the attempts to fix/re-try were causing indefinite loops until you manually kill/fix/re-start the app anyway
— fixed minor bugs related to removing items from Iterators
— added a Main class to be able to run the app as a standalone Java process without ‘jsvc’ - next step would be to refactor the main service class to be a true Service - and be able to run it from any container/driver (from Spring, plain main(), OSGI, etc.)
— made the app less chatty
— analyzed event handling and added TODO items for improvements

I'm closing this Pull request - and will create a new one into the new branch


[ONEToolchain] Introduce ONEToolchain

This will introduce ONEToolchain class as Backend implementations.
Related Issue : #1271
ONE-vscode-DCO-1.0-Signed-off-by: Nakhyun Kim<EMAIL_ADDRESS>

What about adding ONEToolchain.test.ts file?

What about adding ONEToolchain.test.ts file?

That's a good idea!

Since ONEToolchain is also registered as a backend, I think that the test cases checked in the existing test code (ToolchainProvider.test.ts, Backend.test.ts) should be checked for both backends ONEToolchain and MockupBackend.
I thought of two ways below.

Check both 'ONEToolchain' and 'MockupBackend' in the existing ToolchainProvider and Backend related tests, and in a situation where only ONEToolchain is registered in ONEToolchain.test.ts, select and write additional test codes required regardless of ToolchainProvider and Backend.

write additional tests and test cases related to ToolchainProvider and Backend also.

If we proceed as in step 1, the current test code specifies a random backend name and there is a code that compares the expected value with the actual value, so it seems necessary to branch according to the backend name. In this case, I would like to ask for advice on how to write the test.

I prefer step 1. I think you can keep this PR changes as it is and just add new ONEToolchian.test.ts file. This file will be tested for function implementations such as prerequisitesForGetToolchains, getToolchainTypes, etc. Currently, there is no implementation about those functions, so you only need to test the backend name.


Array.prototype.includes fails type-checking when used with Enum subsets

Suggestion
🔍 Search Terms
Enum narrowing, array include
✅ Viability Checklist

My suggestion meets these guidelines:

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

⭐ Suggestion
I've found an edge case where the type of Array.prototype.includes seems to be a bit too narrow.
Given an array that is a subset of enum values, I can't write const check = (item: MyEnum) => ENUM_SUBSET.includes(item); as it claims that MyEnum contains more potential values than what ENUM_SUBSET contains - which is true, but I feel like that being able to perform this check should be valid.
const check2 = (item: MyEnum) => ENUM_SUBSET.find((a) => a === item) !== undefined has the same effect and passes type check but feels much clunkier - In this case I don't think I should need to change my runtime code to satisfy the type system.
📃 Motivating Example
enum MyEnum { First = 'first', Second = 'second', Third = 'third' };

const ENUM_SUBSET = [MyEnum.First, MyEnum.Second] as const;

// It'd be nice to keep the `as const` above so we can infer a type based off ENUM_SUBSET
// This evaluates to `MyEnum.First | MyEnum.Second`
type EnumSubsetType = typeof ENUM_SUBSET[number];

// I would expect this not to raise a type error
const check = (item: MyEnum) => ENUM_SUBSET.includes(item);

// This doesn't raise a type error
const check2 = (item: MyEnum) => ENUM_SUBSET.find((a) => a === item) !== undefined;

// Casting avoids raising a type error it but this seems silly.
const check3 = (item: MyEnum) => (ENUM_SUBSET as ReadonlyArray<MyEnum>).includes(item);

Playground link
💻 Use Cases
I'd like to check if an enum value is in an array. using Array.prototype.includes without encountering type-check errors.
Replacing ENUM_SUBSET.includes(item) with ENUM_SUBSET.find((a) => a === item) !== undefined  works as a workaround but feels clunky.

Tracking the ability to write that kind of function at #14520


Update docs based on feedback from customers and feature PM

After speaking with feature PM Johnathan Garrigues, I have more clarification on how this works. Gradual rollout affects only which set of packages are installed.  It does not affect who can see the PDP (Product Display Page in the store).
Partner Center had questions coming from customers who were unclear as to whether package rollout applies to existing customers or new customers.
To make it clear I have replaced "App's customers" with "customers who have your package installed" where necessary.

#sign-off


Meta is not returned in the TransactionCommonResponse

If I create a new token using Submit result = await xls20client.SubmitTransactionBlob(request); the meta data isnt included in the result.
result.Transaction.Meta is null. There are also other fields that are coming in as null.

[x] Parse the Tx response correctly and include ALL the data.
[x] Write a function that gets the NFT token ID

def get_token_id(meta: dict) -> str:
    if meta['AffectedNodes'][0]['ModifiedNode']['FinalFields']['MintedTokens'] == 1:
        return meta['AffectedNodes'][1]['CreatedNode']['NewFields']['NonFungibleTokens'][-1]['NonFungibleToken']['TokenID']
    return meta['AffectedNodes'][0]['ModifiedNode']['FinalFields']['NonFungibleTokens'][-1]['NonFungibleToken']['TokenID']

https://github.com/Transia-RnD/xrpl.c/pull/6

An example of fetching the last Minted Token ID:
https://github.com/Transia-RnD/xrpl.c/blob/914898fb66bd1f69fd70a1179739e2728c552251/Xrpl.C.Tests/Xrpl.Tests/Client.Tests/TransactionTests.cs#L392


engine build failed

C:\steven\engine\src>ninja -C .\out\host_release
ninja: Entering directory `.\out\host_release'
[1/134] LINK(DLL) flutter_engine.dll flutter_engine.dll.lib
FAILED: flutter_engine.dll flutter_engine.dll.lib
C:/steven/depot_tools/win_tools-2_7_13_chromium7_bin/python/bin/python.exe gyp-win-tool link-wrapper environment.x64 False link.exe /nologo /IMPLIB:./flutter_engine.dll.lib /DLL /OUT:./flutter_engine.
dll /PDB:./flutter_engine.dll.pdb @./flutter_engine.dll.rsp
正在创建库 ./flutter_engine.dll.lib 和对象 ./flutter_engine.dll.exp
runtime.dart_snapshot.obj : error LNK2001: 无法解析的外部符号 kDartVmSnapshotData
runtime.dart_snapshot.obj : error LNK2001: 无法解析的外部符号 kDartVmSnapshotInstructions
runtime.dart_snapshot.obj : error LNK2001: 无法解析的外部符号 kDartIsolateSnapshotData
runtime.dart_snapshot.obj : error LNK2001: 无法解析的外部符号 kDartIsolateSnapshotInstructions
./flutter_engine.dll : fatal error LNK1120: 4 个无法解析的外部命令
[3/134] LINK ./flow_unittests.exe
ninja: build stopped: subcommand failed.

same problem from others:
Trying to build flutter engine on Windows running this:
ninja -C out\host_release
It seems to be working until i hit this error:
[6/7] LINK(DLL) flutter_engine.dll flutter_engine.dll.lib
FAILED: flutter_engine.dll flutter_engine.dll.lib
C:/Python27/python.exe gyp-win-tool link-wrapper environment.x64 False link.exe /nologo /IMPLIB:./flutter_engine.dll.lib /DLL /OUT:./flutter_engine.dll /PDB:./flutter_engine.dll.pdb @./flutter_engine.dll.rsp
runtime.dart_snapshot.obj : error LNK2001: unresolved external symbol kDartVmSnapshotData
runtime.dart_snapshot.obj : error LNK2001: unresolved external symbol kDartVmSnapshotInstructions
runtime.dart_snapshot.obj : error LNK2001: unresolved external symbol kDartIsolateSnapshotData
runtime.dart_snapshot.obj : error LNK2001: unresolved external symbol kDartIsolateSnapshotInstructions
./flutter_engine.dll : fatal error LNK1120: 4 unresolved externals
ninja: build stopped: subcommand failed.

change to debug mode, it can work. Why? Is release mode not support?

Is release mode not support?

In general, there has been no attempt to support release mode for desktop, no. That's tracked in #30672

I'm able to repo this. I've updated the engine build instructions for now to discourage trying to build release for Windows for anything other than gen_snapshot.

This is very interesting, I started to try to solve it.

@stuartmorgan , The reason is that the Windows platform does not link k_vn_snapshot_data.bin.o isolate_snapshot_instructions.bin.o vm_snapshot_instructions.bin.o vm_snapshot_data.bin.o generated by bin_to_coff. How do we add them in the shell/windows link deps?

@stuartmorgan , I initiated a pull request: https://github.com/flutter/engine/pull/11475


Fix setup-go actions

The setup-go actions did not all have check-latest which means they use some cached version of go that currently still resolves to go1.20.4, as seen in a number of recent runs that currently fail at govulncheck because of it:
Run actions/setup-go@v4
Setup go version spec >=1.20
Attempting to resolve the latest version from the manifest...
matching >=1.20...
Resolved as '1.20.4'

Add the check-latest option which should guarantee that this cache is skipped.

Temporary change to backend files added to confirm that the actions now pass. Will revert before merge.

It still resolved to 1.20.4. I can only assume this is a bug in the action.
Setup go version spec >=1.20
Attempting to resolve the latest version from the manifest...
matching >=1.20...
Resolved as '1.20.4'

Opened https://github.com/actions/setup-go/issues/384

Maybe the problems are in https://github.com/actions/go-versions/releases

Let's merge it. Builds should automatically start to pass once https://github.com/actions/setup-go/issues/384 is resolved.


OCSP for multiple intermediate CAs

Hi,
I'm just wondering if it was possible to serve OCSP repsonses for multiple intermediate CAs. How to configure this when there are multiple DBs and CA certs?
Thanks!

No support for this right now but it's something I can probably add in the near future.


[FLINK-8102][docs] Fixed formatting issues in Mesos documentation.

This change fixes formatting issues occurring in the markdown rendering.
It was tested was building the documentation locally.

merging.


sort instances list by updated time

version 留空，走默认值（创建时候时间戳）
编排实例列表按更新时间倒序排序

Codecov Report

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

@@           Coverage Diff           @@
##           master     #593   +/-   ##
=======================================
  Coverage   19.79%   19.79%
=======================================
  Files          29       29
  Lines        1728     1728
=======================================
  Hits          342      342
  Misses       1342     1342
  Partials       44       44

ping @xiaods @cmingxu


Delegates concept in support

The concept of delegates in Kakao.
The concept has been modified by discussions in https://github.com/agoda-com/Kakao/pull/143
What is not completed at the current moment:

incorrect comments
fix todo in WebInteractionDelegate
check DataInteractionDelegate and WebInteractionDelegate
check classes where DataInteractionDelegate and WebInteractionDelegate are using (ListView, RecyclerView and other)
DslMarkers
think about what interfaces/classes are external API
think about com.agoda.kakao.screen.Screen.Companion#onScreen
do full examples
documentation

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: xanderblinov:x: matzukYou have signed the CLA already but the status is still pending? Let us recheck it.

And also @Unlimity please don't create comments for every file about new line. We can figure it out after lint check adding. I suppose that I can do it on this weekend.

@matzuk also I think that it should be merged to master branch. Currently target is -support

And also @Unlimity please don't create comments for every file about new line. We can figure it out after lint check adding. I suppose that I can do it on this weekend.

Yep, I am waiting tuned code style like Detekt for example

Although overall direction is more straightforward now, the implementation is still questionable.
I don't see why there is need to use such complicated approaches with factories and builders.
On top of that with this implementation we leave the responsibility to actually delegate the real call to the real view interaction to the user. Not only that produces a requirement to write a boilerplate, but leaves a huge hole in the stability of the library.
With current implementation I can drop all of the tests simply with 5 lines:
configure {
    onViewInteraction {
        onPerform {}
    }
}

And all of the tests will fail.
I will prepare another solution for your request next week.

The first PR was easier but you desired a more flexible way. I have presented. Yep, the implementation is not trivial but it's not over-engineering.
I can't understand why this approach can leave a hole into the stability of Kakao? Yes, we give developer to reject calls directed in Espresso, but we give full control and it's very convenient. If a developer does something without an awareness of what he is doing it's developer's problem, really.

Closed because of https://github.com/agoda-com/Kakao/pull/155