Extarcting KID and signature

Hi,
where did you find documentation to implement this line of code?
https://github.com/floysh/DCC-green-pass-decoder/blob/99344251c7eb9b37103352ae5341c9ad256211f6/src/source.js#L229
Does it exist similar documentation for decoding signature? I would like to manually decode it, rather than using a ready-made "signature.verify()" function copied from somewhere, by comparing its contents to the certificates found here:
https://raw.githubusercontent.com/lovasoa/sanipasse/master/src/assets/Digital_Green_Certificate_Signing_Keys.json

Hi,
I'm not sure I understand what you want to do. You're not supposed to decode a signature, it's just a byte sequence used by the validation algorithm. If you're looking for a way to validate EDCC signatures, there's this documentation from the EU:
https://github.com/ehn-dcc-development/hcert-spec/blob/main/hcert_spec.md#32-structure-of-the-payload
Keeping it short, since EDCCs are regular COSE structures you can verify the signature by following what is stated by the COSE RFC. In section 3 it states that the protected header is a serialized CBOR map, so you can decode it the same way as the payload (which is what I did in the code line you referenced to).
Let me know if I answered your question 😊

My problem is that I cannot use node.js, I need to use pure javascript, and I can't find any port of node.js cose.js to javascript;  I tried importing cose-js into a javascript module using unpkg.com, jsdelivr.net and others but I can 't get it working, I receive errors about missing exports, missing default exports, wrong cbor dependencies,....
Since my original post, I found some info about the signature: it's in P1363 format, which means they are always 64 bytes, representing values R and S concatenated. But that's all, I can't understand how to manage it.
I asked on jsrasign repo if that library could be compatible with DGC signature verification requirements, as I can successfully import it into a module; they replied it should be, because it supports all encrypting algorithms used in DGC, but they didn't provide any clue about how to use it.


Fix #76: Support query strings.

This is a suggested fix for #76. Urls passed to the service should be encoded, especially the query string part, and so the server should decode the received parameter.
That means you should now pass urls at least partially encoded, eg. https://crossorigin.me/http://api.tvmaze.com/search/shows%3Fq=boo (note the %3F, which is ? urlencoded)
Tested locally, works fine and fixes the issue. Cors tests are broken but were so previously.
Oh and, happy new year!

Note that if we want to keep strict backward compatibility we may just want to use the url request parameter instead:
var url = req.url.slice(1);

But I'm finding it rather ugly... What do you think?

Note that if we want to keep strict backward compatibility we may just want to use the url request parameter instead:
var url = req.url.slice(1);

But I'm finding it rather ugly... What do you think?

decoding parameters is usually the considered best practice

Not in this case afaik; since crossorigin acts as a proxy, you want to change the request as little as possible when forwarding it.
When you say request.url is buggy; can you elaborate on that?

Not in this case afaik; since crossorigin acts as a proxy, you want to change the request as little as possible when forwarding it.

Indeed, so that's why you want to retrieve the actual url the users passed to the proxy in the query string, including everything, not a stripped version of it. #76 explicitly describes the issue.

When you say request.url is buggy; can you elaborate on that?

Where did I say that? To me request.url is fine, it's just that we expect too much from it (eg. distinguishing between user provided url qs and its own qs params).

Sorry to be unclear! :)
You said:

I'm finding it rather ugly, edge case error prone

I wonder what edge cases you've found with request.url.
I would argue against URL-decoding; I'd much rather pass on the full request url without transformation (except to remove http://crossorigin.me/ from the beginning!)

I wonder what edge cases you've found with request.url.

Well, with extracting user provided url directily from request.url we're mixing up crossorigin server's query string and user provided url one, which can be confusing, hence error prone.
Also, what if someday you want to deal with options passed as a query string? Let's take an example:

you want to proxy two urls through crossorigin.me: http://foo.bar/ and http://foo.bar/?lol=false
crossorigin.me accepts iself a lol option you can pass as a query string parameter to the service

First query : GET http://crossorigin.me/http://foo.bar/?lol=true
Second query: GET http://crossorigin.me/http://foo.bar/?lol=false?lol=true
Can you see what kind of trouble you can get yourself in? :)
We sure can decide that we won't ever want to support query string parameters, but that's a rather strong decision. Though if we decide so, the request.url approach is then probably the best as it's 100% BC with the previous implementation.
Your call!

Ah, now I follow you :-)
I was assuming we'd never want to support qs parameters, but I know @technoboy10 experimented with that in the last version, so it's probably us to him! :-)

Thanks for the pull request - I've decided to fix this a different way (see 6ac11f79b7944b74c64fe3584da7fcb036a21272).


InstaFlow-0.9B

Will the InstaFlow-0.9B pre trained weights be released?

We are still discussing the form of code releasing. Thank you for your patience!

We are still discussing the form of code releasing. Thank you for your patience!

Thank you very much!


Remove gperftools, use jemalloc

Seems to actually work, requires no container deps, keeping as an optional build target for now.
Signed-off-by: Benjamin Leggett<EMAIL_ADDRESS>

@bleggett: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
10d19a69e70bc33400e9e45c25368201bf0e79a6
link
true
/test test

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

@bleggett: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
d66fb52a9d6a5c4e8a3c91e655ae4aa9074d8e0b
link
true
/test test

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

@bleggett: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
05f2a5346197265e55ee32cf417bb9323a0ba090
link
true
/test test

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


store: keep track of whether a watched path is a dir or a file

@yuindustries this should let you differentiate files and directories in the view code and render them differently, with or without a trailing slash.

@maiamcc

you're not actually putting these in the hud, correct, just laying the framework?

That's right

fwiw I find dirsWatched/pathsWatched to be misleading -- it's not the list of WATCHED dirs/paths, but of CHANGED dirs/paths, right?

Those are separate concepts, see view.Resource.PendingBuildEdits and friends. dirsWatched is the path of directories that we're watching, and they could be PendingBuildEdits that are children of those directories.

Ohhh! Gotcha.


Delete old unused images

When deleting the windows section I noticed that the image folder looked like it needed some cleaning up. This PR deletes all the image files that weren't referenced on any of the doc pages when I grepped for them. Review just requires checking builds are green.

https://ci.openmicroscopy.org/job/OMERO-DEV-merge-docs/306 passed without warnings/errors.


Fix ConsoleBot example

Fixes #714 by adding code courtesy of @alxpereira 's fix to define bot.send
Updates the uptime event hander to properly respond

cc @peterswimm

Thank you for this!
I made these same changes, but in a slightly different place. They're in master now,  and will be included in the next release of Botkit!


More content

Describe your changes
Issue ticket number and link
Checklist before requesting a review

[ ] I have performed a self-review of my code
[ ] The pipeline is passing

Will cover in the Next migration #232


Docs seo fix (duplicate title and description tags)

adds jekyll-seo-tag to the plugin section of the docs/_config.yml file.
The theme we are using (just the docs) comes bundled with this plugin, but not putting the redundant line in the config file results in the duplicate title and description tags in the header (bad for SEO):

This is caused by some code in the header file that checks for whether the plugin is present in docs/_config.yml.
https://github.com/firezone/firezone/blob/65ac01fa94128ddf51e4d8daa543b251f8bf3a80/docs/_includes/head.html#L5-L11
The other option is removing the code in docs/_config.yml.

@gongjason If jekyll-seo-tag handles this and is included with just-the-docs, can we remove the offending lines from that header include, and the entry from plugins? Seems cleaner and less maintenance going forward.


adding retry parameter to URLchecker, will double time between retrie…

This pull request will attempt to address #15 by adding a retry parameter to the action. Specifically:

the retry_count determines the number of retries to do. Each retry starts at 2 seconds, and then is doubled as we advance (2, 4, 8, 16, etc.)
I think #16 is a bug so I moved it out in the loop, let me know if this is intentional or I'm missing something!
I exposed the timeout to requests to the user, with the default being what it already was set to (5 seconds)
I changed the booleans in the action.yml to be all lowercase, mostly for aesthetics because it renders a different color in the editor and it's easier to see the booleans. To allow for either lower or uppcase set by a user, the envar for each is parsed and then made lowercase with .lower().

Currently, we print and add to the list of check_results each retry attempt, the idea being that a retry could have different results. I'm thinking we might want to refactor this so that the failed url is only added after it's failed all retries - that way we don't have the same url in a failed / success list if it worked on retry.
I held off on further implementation because I think @SuperKogito you had a specific idea for error checking that I wanted to ask for more detail about. Take a look here, and let me know what we should do next!
Signed-off-by: vsoch<EMAIL_ADDRESS>

okay I updated the PR so that the act of checking (and determining if we need retry) is separate from adding a final result to a list. I'm not sure what the best way to test this is, I'll wait to discuss with you further.

Awesome! How would you like to test?

the retry_count determines the number of retries to do. Each retry starts at 2 seconds, and then is doubled as we advance (2, 4, 8, 16, etc.)

This was a great idea! :clap:
The testing can be done using the other repo in this organization, I usually just release the updates, edit the action.yml and trigger a build and just observe the results. I also rely on bugs reports from other users like @shahzebsiddiqui. I am open to better alternatives if you have any suggestions.
Concerning the error checking idea, I will be opening a new issue (among others) with a clearer description and we can discuss it there, to see how practical is it and how useful could it be.
Also if you have any ideas of improvements, feel free to open a PR or an issue to discuss it. Right now, my goal is to have a stable version (no bugs or almost none), with the option to commit a report file to the repo (upcoming feature, this is actually the point of putting the errors in a list) and eventually maybe build a bot for the action. In parallel, it would be also important to get more attention from users in order for the project to progress further.
Again thank you for your PR! the newly implemented features will heavily help get more attention :smile:

Great! Yes and we can test it using buildtest and then the usrse repo, likely later today (it’s still wee hours of the morning here) so stay tuned! I’ll also check the other repo in the org later to see if it ran ok, and I can follow up with another PR if any tweaks are needed.

Sounds perfect to me, have a nice productive day ;)


method can in HasRoles trait

@goszowski Since we can now check the permissions of a user by
if ($user->can('permission-name')) {
// Do something
}
I think we don't need this.


TestSingleBinaryEventForChannel and TestSingleStructuredEventForChannel are broken

Describe the bug
These two e2e tests are failing.

For which channel?

I don't recall exactly. I think Kafka and NATSS.

i'll try to reenable them for kafka, they should just work

I think these are reenabled now on master
/close


Xamarin UI Test getting wrong coordinates for item in iPhone XR

I am using
Xamarin UI Test v3.0.11
Xamarin Test Cloud Agent 0.22.1
iPhone XR iOS 14.2
Xamarin iOS <IP_ADDRESS>
app.Tap is not working for me. When I run it in repl() it says its tapping the coordinates but nothing happens.
By playing around with app.TapCoordinates I can get it to tap the button but its seems the coordinates found for the button are wrong.
As anyone else found this issue? I'm now finally able to write tests again in iOS but this is stopping me from creating tests.

I'm having the same issue.


ISPN-8933 NullPointerException in StreamResponseCommand

Added null check for response tracker due to cache shutdown

https://issues.jboss.org/browse/ISPN-8933

Check https://issues.jboss.org/browse/ISPN-8933 for reasoning as to fix.

integrated! thanks @wburns


How to calculate the point-to-plane reconstruction error

Hi,
I want to test some other 3d face reconstruction methods on the FaceScape dataset.
As the predicted meshes from different methods are in different coordinate systems , how can I align the predicted meshes to the groud truth meshes?
Can you share the code to calculate the  point-to-plane reconstruction error as the paper mentioned ?
Also,  Can you share some tips about how to draw the error heatmap like below? I find there is no much reference code.

Any help would be appreciated! :kissing_heart:

Hi, here is a solution to evaluate methods in different coordinate systems:
https://github.com/zhuhao-nju/facescape/tree/master/benchmark
And here is the heat mesh rendering part:
https://github.com/zhuhao-nju/facescape/blob/dd9ecc75eb0292cb46960c441eb744fdbbecf11b/benchmark/code/evaluator.py#L251


Ognl.GetMethods does not return default methods from interfaces implemented by parent class

Same problem as here: https://issues.apache.org/jira/browse/OGNL-249
But fix for issue 249 takes into account only interfaces directly implemented by a class and not an interfaces implemented by a parent classes.
I will prepare a pull request with a test case (and maybe a fix also)

PR #30 merged


Validate (and update) Fidesdemo for fidesctl 1.7.1

Is your feature request related to a specific problem?
fidesctl==1.7.0 released on 2022-06-23! With that in mind, we should validate the demo workflow as is, and update accordingly.
This issue should focus on just the current functionality, not adding new content
Describe the solution you'd like
Walk through the fidesctl demo portion as is, and ensure no issues are present.
Describe alternatives you've considered, if any
Also including new content, but that seems like it would distract from the goal of ensuring the fidesdemo is fully functional
Additional context
https://github.com/ethyca/fides/releases/tag/1.7.0

Let's change this to focus on 1.7.1 which we should release this week.

Updated to 1.8.1

@mfbrown this is all spinning up and the UI is visible for both fidesops and fidesctl - let me know if you'd like to review together as part of acceptance criteria and closure (will open a PR with the changes as well)


Merge develop into master

@bernardbaker @dzx11 Progress you guys have made here while I was away is phenomenal and greatly appreciated. Made some updates to our README and merging develop into master. I've listed you both publicly by name as contributors, please let me know if you'd prefer not to disclose.


Back-porting Dynamic fork bug PR #2433 to 2.31

Pull Request type

[X ] Bugfix
[ ] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

This backports the fix made in this PR that is merged into V3: https://github.com/Netflix/conductor/pull/2443
It fixes a bug with Dynamic workflows possibly not getting a task back due to a task reference name collision somewhere else in the workflow.

@aravindanr any advice here? Not sure how to kick off the above build issue?

@apanicker-nflx or @aravindanr ? Any help here?

@james-deee Sorry about the delay. This slipped through the cracks. The Github workflow is not working correctly. Could you rebase from Netflix:2.31 into your fork and run the checks again?

@aravindanr this build failed. Is it a change to using Java 11 over here in 2.31 land? I bet it used to compile with Java 8? There's now a failure it looks like:
> Task :conductor-contribs:javadoc
                       ^
  symbol:   class PreDestroy
  location: package javax.annotation
/home/runner/work/conductor/conductor/contribs/src/main/java/com/netflix/conductor/contribs/listener/ArchivingWithTTLWorkflowStatusListener.java:53: error: cannot find symbol
2 errors
    @PreDestroy

Looks like it. I switched the ci build to use Java 8. Can you rebase and try again?

@aravindanr that did it. Can you merge this when you get a chance? Thanks.


No default jsDelivr CDN file set

This package doesn't have a default file set. You can set it via jsdelivr, browser, or main field in package.json

Would like to use the library by importing it as:
import chloropleth from 'd3-geomap'

I'm not familiar with all the options you have with regards to JS packages and content delivery networks, What file would I need to set as main and would this by itself allow the library to be imported the way you intend?

The only way I know would be to do: export default class Choropleth extends Geomap, https://developer.mozilla.org/en-US/docs/web/javascript/reference/statements/export. I am not sure if that is backward compatible with pre ES6 js though.

I hope this finally works as expected in version 3.0.0, which I just released.


NaN with HWiNFO plugin

Having an issue where win using the HWiNFO plugin, just getting NaN as the data from HWiNFO. FanControl itself isn't seeing the temp sensor that's connecting to my water temp. Would like to run my radiator fans off what the water temp is, not cpu or gpu temp.
Made sure HWiNFO gadget settings are correct.

HWInfo, while running, exports these sensors valyes into the registry at this address:
Computer\HKEY_CURRENT_USER\Software\HWiNFO64\VSB
Like so:

What do you get?

I exported it out and saved as a txt file.
hwinfo.txt

You exported basically everything? Try and only export the temperature sensors you need like t_sensor.

"Sensor502"="ASUS EC: ASUS "
"Label502"="T_Sensor"
"Value502"="84.2 °F"
"ValueRaw502"="84.2"
"Color502"="004080"

Looks like it's an issue with gpu items. Removed those items from the gadget and it seems to fix the issue
4/15/2023 7:31:00 PM: System.Exception: HWInfo sensors failed: HWInfo/GPU [#0]: NVIDIA GeForce RTX 4080: /GPU Performance Limiters/, HWInfo/GPU [#0]: NVIDIA GeForce RTX 4080: /Performance Limit - Power/, HWInfo/GPU [#0]: NVIDIA GeForce RTX 4080: /Performance Limit - Thermal/, HWInfo/GPU [#0]: NVIDIA GeForce RTX 4080: /Performance Limit - Reliability Voltage/, HWInfo/GPU [#0]: NVIDIA GeForce RTX 4080: /Performance Limit - Max Operating Voltage/, HWInfo/GPU [#0]: NVIDIA GeForce RTX 4080: /Performance Limit - Utilization/, HWInfo/GPU [#0]: NVIDIA GeForce RTX 4080: /Performance Limit - SLI GPUBoost Sync/    at FanControl.HWInfo.HWInfoPlugin.Update()    at FanControl.Domain.BackendProviders.Plugin.PluginBackendProvider.Update()    at FanControl.Domain.ComputerAccessLayer.<>c.<Update>b__33_0(IBackendProvider x)    at System.Collections.Generic.List1.ForEach(Action1 action)    at FanControl.Domain.ComputerAccessLayer.Update()    at FanControl.Domain.ApplicationClock.DoActions() --- End of stack trace from previous location ---    at FanControl.Domain.ApplicationClock.<>c__DisplayClass10_0.<DoActions>b__0()    at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)    at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)

I had a similar issue when swapping out a hard drive (usually requires me to re-select sensors) but this time it got stuck in a loop when I removed the old and new hard drives (leaving just my M.2s). I could interact with most of the interface, but the actions (new configuration, refresh sensor, etc.) didn't seem to do anything.
I managed to resolve it by 1) stopping the process, 2) renaming the active configuration file, 3) starting the program again, and 4) manually "loading" the renamed configuration file. I was then able to re-select the sensors like before.

I had a similar issue when swapping out a hard drive (usually requires me to re-select sensors) but this time it got stuck in a loop when I removed the old and new hard drives (leaving just my M.2s; new hard drive wasn't compatible & old was too big a hassle too put in rn). I could interact with most of the interface, but the actions (new configuration, refresh sensor, etc.) didn't seem to do anything.
I managed to resolve it by 1) stopping the process, 2) renaming the active configuration file, 3) starting the program again, and 4) manually "loading" the renamed configuration file. I was then able to re-select the sensors like before.

I'm also experiencing this issue with using any sensors in HWinfo. It works for a moment but the second it gets a NaN it fails and records an error in the log.

I'm also experiencing this issue with using any sensors in HWinfo. It works for a moment but the second it gets a NaN it fails and records an error in the log.

Try hiding sensors you don't need in hwinfo. I think there is an issue somewhere with how many sensors are being brought in from HWinfo. After I pruned the sensors, this issue went away.

Try hiding sensors you don't need in hwinfo. I think there is an issue somewhere with how many sensors are being brought in from HWinfo. After I pruned the sensors, this issue went away.

So what ended up working so far for me is to fully reinstall HWinfo because I noticed in the registry that the entire key/value pair for the MemTemp sensor was disappearing when I was refreshing constantly. So the NaN was a true reading it was just what HWinfo was reading at the time. So far it's been working since that.


Export/import workspaces

When working in bigger playground workspaces (especially when collaborating with others) it would be very useful to be able to export and import workspaces including all necessary data.

I'd love to see this too. I've been running into an issue a couple of times where a failed subscription has caused all my request tabs to get deleted. I'll open an issue related to that if I can figure out what's causing it, but either way I think it would be valuable to be able to import and export a workspace which could be represented as a JSON file.

Looks like this hasn't had too much work made on it, would there be interest in still supporting this? (I just had to change the port my server listened on, and I lost my graphql workspace and can't manage to get it to be imported by just messing with localstorage)

Really need this too.


Parsing ambiguity related to records.

Dart SDK Version (dart --version)
3.0.5

This program doesn't compile:
void main() {
  final x = 42;
  (x) >= 0; // ok
  (x) > 0; // ko
  (x) >> 0; // ko
}

Error:
A record type with exactly one positional field requires a trailing comma.

It seems to interpret (x) (before >> or >) as some kind of 1-item record type.
(This is not handwritten code but code generated by ClojureDart.)

Looks like something that should work. Possibly the parser committed to an interpretation of (x) being a type a little too soon.

Duplicate of https://github.com/dart-lang/sdk/issues/52365.
@johnniwinther as I recall we talked about if the fix should be cherry-picked and decided it wasn't necessary. Is this still the thouht?

The fix is in the current beta so I don't think it is necessary.
@cgrand Are you blocked on this or do you have a work-around?

@johnniwinther I have a workaround


Unable to play mp4 files stored in local file system using tag

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for an issue that matches the one I want to file, without success.

I'm trying to create a video player using electron.
Video files (mp4) are stored locally in the filesystem.
It is working correctly if I run

electron-forge start

But the video is not loaded if I run the generated executable created by

electron-forge package --platform=win32 --arch=x64

I'm using the latest version of packages (electron-forge 2.8.2 that still uses electron 1.4.15)
How to reproduce:

electron-forge init myprj
cd myprj
download mp4
add follogin tag in index.html:

electron-forge start    <-- this works fine
electron-forge package --platform=win32 --arch=x64
cd out/myprj...
run myprj.exe  <-- this doesnt' work

@sierrodc Can you provide the code that you are using to load the video files.  I believe electron-compile effectively sandboxes your app so that files that aren't precompiled can't be loaded at runtime.  Not sure if this is actually the case without seeing your code though, could be a generic path change issue 👍
/cc @paulcbetts ^^ thoughts

Hi,
I've uploaded sample source code here:
https://github.com/sierrodc/videoplayer
You need to change mp4 file url in the index.html page.
Thanks.

I think @MarshallOfSound has the right idea, we should probably provide an escape hatch for this in electron-compile, something like a insecurefile:// scheme that will just blindly return a file job

Got it.
Do you know if there's a workaround for this?
If I put the video during compilation time it works fine but this is not what I need.
I've also tried (after adding the video during compilation time) to overwrite using fs the video in output but in this case always the "compiled" video is used.
Is this a problem related to electron-forge or electron? Is there a workaround? what about electron-packager/electron-builder? I really need this functionality. Do you have any suggestions?

@sierrodc The easiest workaround is to register a custom protocol yourself and bypass electron-compile entirely.  In the mean time I'll look into adding an option to the protocol hook in -compile to selectively bypass the compiler for certain URI's.

@sierrodc See https://github.com/electron/electron-compile/pull/199

I'm gonna have a look at this tomorrow, I want to make this a bit harder for people because I don't want there to be a super easy "Copy paste this to disable security" line

Hi, a feedback:

Thank you very much Marshall, your solution works perfectly. I've tested it both in windows and on a raspberry pi 3.
about your "suggestion" to register this function:
addBypassChecker((filePath) => {
return filePath.indexOf(app.getAppPath()) === -1;
});
If I do it, an electron-forge application created using angular-template doesn't work anymore (error similar to "invalid token import" in bootstrap.ts).
But if I add
addBypassChecker((filePath) => {
return filePath.indexOf(app.getAppPath()) === -1 && /.mp4/.test(filePath);
});
everythings works fine for me.

Thank you again.
@paulcbetts what kind of "security" issue are you afraid of? "fs" is anyway available to read/write the filesystem.

@sierrodc
The errors are caused by two things:

In windows OS the apppath has \ as path separator, but the file will have /. Therefore you would need to use app.getAppPath().split("\\").join("/")
in debug mode the app path is like

.....\node_modules\electron-prebuilt-compile\node_modules\electron\dist\resources\default_app.asar

but the filepath is any file served from the application (including the .ts), or external to it.
Both these means that the indexOf will return -1, even for .ts files that it should be compiling, hence the error.
It seems hardcoding the extension is the best way for now, but interested if others have ideas.

For me it plays only when the video file has a low resolution.  No matter the filesize in MB or bitrate, high-res files just won't load for me.


Volar interferes with VS Code's hex color swatches

I ran a VS Code extension bisect and identified Volar as the root cause of this issue.
When a project loads up, I can see that I initially have color swatches next to all hex colors, but once Volar loads, they disappear. See below.
https://github.com/vuejs/language-tools/assets/12532733/b04e5639-381f-4bad-8985-a2294e4411b0

This may be an issue with VSCode, please report it to VSCode. Because the Vue langauge server only provides color infos through the LSP, the Vue extension will not actively push this.


Split tests by language

Release Note
NONE

It seems that go test -timeout 60m -v ./e2e/common/... -tags=integration does not print it's output in progress but it prits everything at the end of the test execution.
Travis CI seems to have an issue with it:
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received
The build has been terminated

Should I change the command to go test -timeout 60m -v ./e2e/common -tags=integration && go test -timeout 60m -v ./e2e/common/languages -tags=integration? What do you thing about it?

@nicolaferraro WDYT?

@nicolaferraro WDYT?

I was thinking about this but did not have time to look carefully.
I remember there was an issue in travis that happened when an execution did not produce any output for a long time. But don't know why this change triggers that issue.
This may be something completely silly, but I remember some issue about the ... notation here: https://github.com/apache/camel-k/pull/1822/files#diff-9aecbcc15d64c3e7a146ee5776a35e65b45fb72461e370c4e295dd90f6d45262R151
I remember a bug about that and maybe the go version in Travis is hitting that. Do you mind trying to run tests on explicit dirs or hack with versions?

@nicolaferraro WDYT?

I was thinking about this but did not have time to look carefully.
I remember there was an issue in travis that happened when an execution did not produce any output for a long time. But don't know why this change triggers that issue.
This may be something completely silly, but I remember some issue about the ... notation here: https://github.com/apache/camel-k/pull/1822/files#diff-9aecbcc15d64c3e7a146ee5776a35e65b45fb72461e370c4e295dd90f6d45262R151
I remember a bug about that and maybe the go version in Travis is hitting that. Do you mind trying to run tests on explicit dirs or hack with versions?

I hope this could help https://github.com/apache/camel-k/pull/1822/files#diff-b2c08647d24f7df76cd7a5f2adc119ab93451ad2f662ded422e55943fd889b04R86

Damn! The https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received is not working in the script that runs the integration test (https://travis-ci.com/github/apache/camel-k/builds/204584739). Then I would go with  go test -timeout 60m -v ./e2e/common -tags=integration && go test -timeout 60m -v ./e2e/common/languages -tags=integration.

@nicolaferraro I am done. Can you recheck/merge this PR?

Thanks


Scroll bottom to top

Description
Closes #89

Related issue(s)

Thank you so much @grraghav120 for contributing :))


Recommended image size

Hi @hiukim, do you have any recommendations about the ideal minimum and maximum size (resolution) of the image that should be compiled as a target for efficient tracking?

I normally use around 800x800. but I don't really have a well tested answer yet.


PoliceQuest3 - De - MT32   (The German Version with MT32 Support) is still in english

PoliceQuest3DeMT32  is still the english version of the game.
You must add the following line:
language=49
to the file:
resource.cfg
so it looks like:
videoDrv=
soundDrv=mt32.drv
minHunk=220
minChip=240
kbdDrv=
mouseDrv=YES
language=49

Thanks for this, will fix!

Super 👍


[Bug]: Unable to Identify github_app_uuid for Deploying Private Repository via Coolify API

Description
I'm trying to deploy a private GitHub repository using the Coolify API on a self-hosted setup. The goal is to automate the deployment process through our front-end app, allowing Coolify to deploy instances on demand with unique environment variables for each.
Minimal Reproduction (if possible, example repository)
Set up a self-hosted Coolify instance.
Create a Team.
Create an Api Key.
Create a GitHub app manually into Coolify resources.
Attempt to use the API endpoint /api/v1/applications/private-github-app to deploy the repository using different IDs (ID, installation ID, client ID).
Exception or Error
"message": "GitHub App not found"
Version
v4.0.0-beta.323
Cloud?

[ ] Yes
[X] No

The Github App related endpoints are still missing from the API.

My usecase demands some kind of deployment with custom envs via front end on X service and i cant find a workaround to deploy them and monitor it at coolify better than this.
In last instance is there a way i can contribute on this with PRs or donations ?

I hope this is now working.
Github App related endpoints will be implemented in the API soon.


WSL terminal launches from command shell, not from icon  (different sympton than 4413)

Environment
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]
Your Distribution version: [On Debian or Ubuntu run `lsb_release -r` in WSL]
Whether the issue is on WSL 2 and/or WSL 1: [run `cat /proc/version` in WSL]

Steps to reproduce

Expected behavior

Actual behavior

The I/O operation has been aborted because of either a thread exit or an application request.
Press any key to continue...

Can launch from bash or wsl from command line.  Launching from start menu or icon yields
The I/O operation has been aborted because of either a thread exit or an application request.
Press any key to continue...

Looks like the environment variables (not sure which one) got corrupted, causing the wslhost not to be found /loaded when launching from windows but found / loaded when launched from the shell.


Consider finding a consistent set of icons

Start here: https://github.com/BitcoinDesign/Bitcoin-Icons

This is now done. We're using Hicon with the Bitcoin Design icons.


Get temperature from motion trigger rather than time lapse?

When computing daily stats and counts for the rolling RAI etc, should the daily temperature value come from the temperature recorded at the daily timelapse image, or the mean of the temperatures from all images taken that day for each species?

If we do this, would need temperature in both effort and motion detection data, then when join them, pick the effort temperature when is.na(species), but the motion detection temperature otherwise.

This can get complicated during different aggregations, would need more thought... leaving as daily value for now.

Confirmed from team that daily timelapse temperature is sufficient, and standardized


Make supabase-js a monorepo

Chore
Describe the chore
Currently, there are various repos for the various JS clients: gotrue-js, postgrest-js, realtime-js, storage-js, and supabase-js which wraps them all. This makes it harder to track issues across the various separate repositories. Issues which are fixed in one of the constituent repositories also require an explicit version bump in the supabase-js repository, which adds another layer of indirection for new users.
Given that most Supabase users tend to just use the main supabase-js client which is also highlighted in the docs, I'd propose converting supabase-js into a monorepo which contains all the separate clients. Additionally, this might also ease the transition into using a more tree-shaking friendly approach as planned for 2.0 releases (#170).

Super late reply: I suppose one blocker is this makes versioning and publishing difficult - we use semantic versioning + conventional commits + semantic-release for most repos.

Issues which are fixed in one of the constituent repositories also require an explicit version bump in the supabase-js repository

This is actually not needed - e.g. for a bug fix in postgrest-js npm update @supabase/postgrest-js should be enough. (We do bump the version in supabase-js anyway, though it's kinda redundant.)
Other concerns are valid - difficulty to track issues, tree-shaking. The latter probably needs to wait until there's a proposal on how to go about it - I'm personally not familiar with tree-shaking and how monorepos benefit from it.

@soedirgo , just thought you might want to know that nx workspaces (monorepo tool) provides way to publish npm libraries

+1 on converting to a monorepo.
Many months ago I was against monorepos in general. I didn't see any improves in workflow and versioning was scaring to me.
But I started a project myself with lerna and later changed opinion.
In my little experience I can tell that monorepos are powerful and versioning is wonderful. Not all projects are eligible to be developed in a monorepo but this project it is.
Reasons:
In JS ecosystem is difficult to update indirect dependencies, we all know. So in supabase-js all other repos are direct dependencies and so indirect dependencies in all projects using supabase-js. But supabase-js dependencies are specified with a range and so not always updated correctly. And tracking issues and updates are difficult as mentioned above.
Versioning might be overwhelmed and unnecessary sometimes but helps a lot with updating indirect dependencies.
Overall I think monorepo could be considered.


Quickly dragging tasks across creates duplicates and weird scrolling behaviors

Drag from the first list across to many lists to the right quickly, then back left to the beginning again quickly. You'll notice jarring, odd and incorrect behaviors, namely:

The dragged task is being duplicated in many lists
The Board is scrolling left and right uncontrollably
Sometimes if the above is done quickly, some tasks lose their alpha values, this is less common now
This issue is the continuation of issue #5 since these two issues are slightly different but closely related

This is done as far as I can tell, the only issue I'm not 100% sure is completed is the 1st one where the dragged task is being duplicated, but because we overhauled the entire scrolling system I believe it's been fixed.
In any case, if it is spotted again I'll definitely reopen this, but until then, it's done!


React step 5

My bluemix URL:
http://carbon-tutorial-enoc14.mybluemix.net

Sure!, The PR is alredy


Roll Flutter Engine from 48973d7f9beb to 845dd9d54da9 (1 revision)

https://github.com/flutter/engine/compare/48973d7f9beb...845dd9d54da9
2023-09-29<EMAIL_ADDRESS>Roll Skia from 2d4045f55fd5 to a063eaeaf1e0 (6 revisions) (flutter/engine#46420)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Portainer does not support Docker in Windows container mode

Bug description
We are trying to install empty stack on Docker host managed by Portainer that is running in Windows containers mode.
During the creation of the stack, we receive an error from Docker host:
Error response from daemon: could not find plugin bridge in v1 plugin registry: plugin not found

Expected behavior
We expect the stack to be created just like in Docker/Linux mode
Steps to reproduce the issue:

Install Docker on Windows 2016
Setup TLS connection with Portainer host
Try to create empty stack without any services with docker-compose.yml that looks like that:

version: 2

See error

Technical details:

Portainer version: 1.22.0
Docker version (managed by Portainer): Standalone 19.03.2

Seems the problem is related to defaults encoded in integration/assets/networks/default-network-config.yml in module libcompose. Thankfully the defaults can be overwritten. I'm testing a workaround right now...

The workaround was to include some test service  in composer definition:
'''
version: 2
services:
simple:
image:  mcr.microsoft.com/windows/servercore:ltsc2016-amd64
'''
And execute following on the host Powershell:
'''
Get-NetNat | Remove-NetNat
'''
Then I came to another problem described here: https://github.com/portainer/portainer/issues/2039


Caching of properties, losing intellisense

Often when editing T4 templates, I lose the intellisense so R# start complaining that properties/methods does not exist, looks like this:

If I restart visualstudio, everything is fine again, so I wonder if it is something that either doesn't know when to clear caches, or reload referenced assemblies?
Clearing R# cache does not help..
The t4 templates that this happens for, references assemblies that are built in another instance of visualstudio, and are referenced that way, can that be the problem?
Is there any way I can tell ForTea to clear its caches or reload the assemblies referenced?
Assemblies are referenced like this:
<#@ assembly name="$(SolutionDir)\target\lib\sometool.dll" #>
The t4 generator does not have any problem, the generated code is fine, but the loss of intellisense is not cool... :)

@neslekkim  Did you see this recently with latest versions of ForTea and ReSharper?

That's two years ago, stopped using t4 for other reasons
clever idea, not so god implemenation (T4 that is, I'm not meaning your soution)


[test] fix regex gen with escape issue on single quote

this fixes #11918

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f0b78b4) 61.64% compared to head (537be29) 34.82%.
Report is 2 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12181       +/-   ##
=============================================
- Coverage     61.64%   34.82%   -26.82%
+ Complexity     1153      945      -208
=============================================
  Files          2407     2331       -76
  Lines        130915   127172     -3743
  Branches      20225    19669      -556
=============================================
- Hits          80697    44284    -36413
- Misses        44320    79743    +35423
+ Partials       5898     3145     -2753

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <ø> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <ø> (ø)

java-11
?

java-21
34.82% <ø> (-26.70%)
:arrow_down:

skip-bytebuffers-false
34.77% <ø> (-26.82%)
:arrow_down:

skip-bytebuffers-true
46.50% <ø> (-15.00%)
:arrow_down:

temurin
34.82% <ø> (-26.82%)
:arrow_down:

unittests
46.53% <ø> (-15.11%)
:arrow_down:

unittests1
46.53% <ø> (-0.13%)
:arrow_down:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


lodash: signatures of the method _.slice changed

https://lodash.com/docs#slice
https://lodash.com/docs#_ (description of the chainable wrapper)

thanks mate!


str: use correct cdada comparison function for a regular C string

The cdada_str_ncmp_c function must be used for comparison with regular C string, otherwise str_test aborts.
Fixes: be47ae8cb148 ("str: add cdada_str_ncmp/cdada_str_ncmp_c")

Hi @arkamar,
Thank you for this patch!
Indeed this is the right fix. It seems a copy&paste error from my side. CI has been (incorrectly but) consistently passing (e.g. random example) since https://github.com/msune/libcdada/commit/be47ae8cb14832e3401df4a8bc7a544f17fc170a, so I am a bit puzzled.
What was the environment you were using to make tests fail? It might be a good idea to add it to the CI...

Yeah, it looks like a copy&paste error :)
I found the issue while I was creating a gentoo package for v0.6.0, commit https://github.com/gentoo/gentoo/commit/ac8163603c82d8cdb006edfbacc9cb4440c8355f. The gentoo ebuild is basically a bash script describing how to build the software. We usually run tests, which in this case mean that the package was configured with --with-tests parameter, built with make and then tested with make check. I use pretty bleeding edge environment with GCC 15, glibc 2.40 and Python 3.13.0b4.
It is really weird why CI does not catch it, because cdada_str_ncmp first compares the magic_num for both input parameters, which does not obviously match for s2. Could it be that strncmp will return also -1? Anyway, it returns -78 for me.

Děkuji Petr! (*)
I think I am into something :thinking: ... (fresh debian/testing container in v0.6.0)
root@199d3cd04fa9:~/libcdada/build/test# libtool --mode=execute valgrind ./str_test
==32080== Memcheck, a memory error detector
==32080== Copyright (C) 2002-2022, and GNU GPL'd, by Julian Seward et al.
==32080== Using Valgrind-3.20.0 and LibVEX; rerun with -h for copyright info
==32080== Command: /root/libcdada/build/test/.libs/str_test
==32080==
Running libcdada: 0.6.0, MAJOR: 0, MINOR: 6, PATCH: 0. Build '0ab6a5e863b5e81a20b9a6c6b7f06df4306ea6de'
This is a test test
Another test TEST tEsT
another test test  test test xx yyy xy
==32080==
==32080== HEAP SUMMARY:
==32080==     in use at exit: 0 bytes in 0 blocks
==32080==   total heap usage: 20 allocs, 20 frees, 75,242 bytes allocated
==32080==
==32080== All heap blocks were freed -- no leaks are possible
==32080==
==32080== For lists of detected and suppressed errors, rerun with: -s
==32080== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
root@199d3cd04fa9:~/libcdada/build/test# libtool --mode=execute  ./str_test
Running libcdada: 0.6.0, MAJOR: 0, MINOR: 6, PATCH: 0. Build '0ab6a5e863b5e81a20b9a6c6b7f06df4306ea6de'
This is a test test
Another test TEST tEsT
another test test  test test xx yyy xy
Aborted (core dumped)

And adding a simple trace in line 411:
root@199d3cd04fa9:~/libcdada/build/test# libtool --mode=execute valgrind ./str_test
==32610== Memcheck, a memory error detector
==32610== Copyright (C) 2002-2022, and GNU GPL'd, by Julian Seward et al.
==32610== Using Valgrind-3.20.0 and LibVEX; rerun with -h for copyright info
==32610== Command: /root/libcdada/build/test/.libs/str_test
==32610==
Running libcdada: 0.6.0, MAJOR: 0, MINOR: 6, PATCH: 0. Build '0ab6a5e863b5e81a20b9a6c6b7f06df4306ea6de'
This is a test test
Another test TEST tEsT
another test test  test test xx yyy xy
cdada_str_ncmp: -1, strncmp: -1
==32610==
==32610== HEAP SUMMARY:
==32610==     in use at exit: 0 bytes in 0 blocks
==32610==   total heap usage: 20 allocs, 20 frees, 75,242 bytes allocated
==32610==
==32610== All heap blocks were freed -- no leaks are possible
==32610==
==32610== For lists of detected and suppressed errors, rerun with: -s
==32610== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
root@199d3cd04fa9:~/libcdada/build/test# libtool --mode=execute ./str_test
Running libcdada: 0.6.0, MAJOR: 0, MINOR: 6, PATCH: 0. Build '0ab6a5e863b5e81a20b9a6c6b7f06df4306ea6de'
This is a test test
Another test TEST tEsT
another test test  test test xx yyy xy
cdada_str_ncmp: -1, strncmp: -78
Aborted (core dumped)
root@199d3cd04fa9:~/libcdada/build/test#

I will keep investigating, but seems like a valgrind bug. I've found similar reports in Google :cry:

(*) My son is 50% czech... but I can't speak it :grinning: !

Rádo se stalo :)
It is indeed the valgrind fature, see https://sourceware.org/git/?p=valgrind.git;a=blob;f=shared/vg_replace_strmem.c;h=ae13a2a5f87a34e704e2ceec35a2c712f7248141;hb=HEAD#l709. I think it is technically correct, because the strncmp(3) man page specifies when the return value should be 0, negative or positive but it does not specify the value.
However, the return type is int, how about choosing a different negative value for invalid magic number comparison, something outside of possible char values?

Yup..

However, the return type is int, how about choosing a different negative value for invalid magic number comparison, something outside of possible char values?

Yes, we could do that (and yeah, https://github.com/msune/libcdada/blob/master/src/str.cc#L351 seems obvious I was like, whatever). Wondering if it's really worth; strncmp() returns an int, so technically it could return anything < 0.
I don't think anyone would want to use these functions having to treat s/t like CDADA_E_INVALID_STR or something, so I think it's simpler to just leave it as is. Thoughts?

so I think it's simpler to just leave it as is

In the sense of only comparing < 0 or > 0

I don't think anyone would want to use these functions having to treat s/t like CDADA_E_INVALID_STR or something, so I think it's simpler to just leave it as is. Thoughts?

I guess you are right, because it would help only in situations like this one, where functions are used incorrectly.
It's pity that typedef void cdada_str_t effectively silences type-checking warnings, otherwise we would see something like this:
test/str_test.c: In function 'test_cmp':
test/str_test.c:411:40: warning: passing argument 2 of 'cdada_str_ncmp' from incompatible pointer type [-Wincompatible-pointer-types]
  411 |         TEST_ASSERT(cdada_str_ncmp(c1, s2, 10) == strncmp(s1, s2, 10));
      |                                        ^~
      |                                        |
      |                                        char *
test/common.h:8:16: note: in definition of macro 'TEST_ASSERT'
    8 |         if( ! (exp) ) abort()
      |                ^~~

which would probably help to avoid similar mistakes.

It's pity that typedef void cdada_str_t effectively silences type-checking warnings, otherwise we would see something like this:

I have been thinking to change void to something, to enable some basic type-checking for the container ptr. I've trying to remember why I decided to use void in first place, and I can't think of a good reason (I do for keys and values, that's the trade-off for not using MACROs).
So yeah, maybe this is a good moment to change this, to anything other than void. Let me take a look. Do you need v0.6.1 anytime soon for packaging? Still chasing #20

We are fine with v0.6.0 for now. Those issues affect tests only. I can easily fix them with patches if necessary as I already did with str_test.

In case it helps, pushed v0.6.1 with this and the other PR, plus some cleanups on the build system.

In case it helps, pushed v0.6.1 with this and the other PR, plus some cleanups on the build system.

Thanks!


Licensing Intel oneAPI Math Kernel Library PARDISO

do you know can pypardiso that is interfacing Intel oneAPI Math Kernel Library PARDISO be used for open source projects ?

Hi @parshakova,
pypardiso itself is licensed under the BSD-3-clause license: see LICENSE.txt
For PARDISO included in Intel MKL, tto my knowledge the Intel Simplified Software License applies, see https://cdrdv2-public.intel.com/749362/intel-simplified-license-software-october-2022.pdf
see also:

https://pypi.org/project/mkl/
https://anaconda.org/intel/mkl


Added Media Lists to the User's About Tab

[x] Read CONTRIBUTING.md.

Summary of changes:

Added UserMediaListQuery.
Media list integration.
Refactored MediaSmall to reuse it.
UI for user's collection of named lists.

Tested changes:
Huh

https://github.com/imashnake0/Animite/assets/56754616/b07a074c-9c6e-4a5b-aadd-a4fe4b39e070

https://github.com/user-attachments/assets/44009202-73f5-424a-bd5f-002322787c1d


Error: Invalid UTF-8 string in BSON document

Getting this error
Error: Invalid UTF-8 string in BSON document
when I try to access the db by running
deso> db.data.find( ).limit(1)
How to ignore the error and still read the db?
Attaching the screenshot of the error

did you find the solution ?


fix(parser): only show flow error if it's a flow file

Otherwise we get a mixture of confusing error messages.

#5069  👈
main

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

Join @Boshen and the rest of your teammates on  Graphite


Copy-Item : ... The process cannot access the file

Updating 'apifox' (2.2.26 -> 2.2.27)
Downloading new version
Starting download with aria2 ...
Download: Download Results:
Download: gid   |stat|avg speed  |path/URI
Download: ======+====+===========+=======================================================
Download: 9054b5|OK  |    24MiB/s|D:/scoop_user_apps/cache/apifox#2.2.27#http_cdn.apifox.cn_download_2.2.27_Apifox-2.2.27.exe_installer.exe
Download: Status Legend:
Download: (OK):download completed.
Checking hash of Apifox-2.2.27.exe ... ok.
Uninstalling 'apifox' (2.2.26)
Running uninstaller script...
INFO  Backing up user data...
Removing shim 'Apifox.shim'.
Removing shim 'Apifox.exe'.
Unlinking D:\scoop_user_apps\apps\apifox\current
Installing 'apifox' (2.2.27) [64bit] from dev-tools bucket
Loading Apifox-2.2.27.exe from cache.
Running pre_install script...
INFO  Copying user data...
Copy-Item:
Line |
   3 |      Copy-Item "$persist_dir\User Data\*" "$env:AppData\apifox\" -Recu …
     |      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | The process cannot access the file 'C:\Users\13697\AppData\Roaming\apifox\ApifoxAppAgent.exe' because it is being used by another process.
Running installer script...
Linking D:\scoop_user_apps\apps\apifox\current => D:\scoop_user_apps\apps\apifox\2.2.27
Creating shim for 'Apifox'.
Creating shortcut for Apifox (Apifox.exe)
Running post_install script...
'apifox' (2.2.27) was installed successfully!

Although every update is successful, there is always a red report in the middle

My apifox is not set to start, nor is it in use, but this problem is reported

@dong-lufei Try it now


Added check if headers existed on reply

In trying to hook up a C# requester to our Node responder, we came across the issue of:
Potentially unhandled rejection [10] TypeError: Cannot read property 'direct-reply-to' of undefined     at Object.reply (/usr/src/app/node_modules/wascally/src/amqp/queue.js:108:31)
Adding this additional check allowed the two to communicate properly.

@mashu-daishi - thanks for the contribution, I'll merge this in a publish the result to NPM but please note we're not actively supporting this library any more. Have a look at rabbot which supports the same API with a lot of improvements.


Fill InfluxDB with static data aligned with Telegraf measurements?

What products and version are you using?
Telegraf 1.13.3
Where did you look before opening the issue?
https://github.com/influxdata/telegraf/issues/3279
Due to a legacy calculation, I find myself needing to send a piece of static data along with the collections that Telegraf is sending. I've tried searching every way that I can, but I've been unable to find a solution.
Due to a legacy calculation that occurs when polling our InfluxDB data, I'm in need of a field - "Units" in this case - be filled with 1, but aligned with each polling period from Telegraf.
Is there a Telegraf or Kapacitor solution here that I'm not seeing that will for this kind of alignment?

@ajohnson-512 thank you for opening this issue. I'm not sure I completely understand. You might be able to use the influxql FILL option in your query to do this. https://docs.influxdata.com/influxdb/v1.7/query_language/data_exploration/#group-by-time-intervals-and-fill
This looks more like a question, that would be better asked in either on https://community.influxdata.com or the community Slack space: https://influxdata.com/slack. I'm going to close this for now and look forward to seeing you in those spaces.


"Sequence contains no matching element" exception on first run

we get the following error on Azure Devops with the Nukeeper extension on the first run. Rerun fixes the issue.
2019-01-24T13:06:43.6765138Z Task         : NuKeeper
2019-01-24T13:06:43.6765322Z Description  : Version 0.1.54
2019-01-24T13:06:43.6765428Z Version      : 0.1.54
2019-01-24T13:06:43.6765545Z Author       : NuKeeper
2019-01-24T13:06:43.6765685Z Help         : For help please visit https://github.com/NuKeeperDotNet/NuKeeper
2019-01-24T13:06:43.6765868Z ==============================================================================
2019-01-24T13:06:44.6094085Z [command]D:\VstsBuildAgent\externals\git\cmd\git.exe checkout master
2019-01-24T13:06:44.6095453Z [command]D:\VstsBuildAgent\externals\git\cmd\git.exe pull
2019-01-24T13:06:44.6096662Z [command]D:\VstsBuildAgent\externals\git\cmd\git.exe config --global user.name NuKeeper
2019-01-24T13:06:44.6097979Z [command]D:\VstsBuildAgent\externals\git\cmd\git.exe config --global user.email<EMAIL_ADDRESS>2019-01-24T13:06:44.6157316Z [command]"C:\Program Files\dotnet\dotnet.exe" D:\VstsBuildAgent\_work\_tasks\NuKeeper_e5a17a66-903f-46a9-a9b9-94241bcedc39\0.1.54\bin\NuKeeper.dll repo D:\VstsBuildAgent\_work\33\s *** -p 3
2019-01-24T13:06:45.0636762Z You are not currently on a branch.
2019-01-24T13:06:45.0651434Z Please specify which branch you want to merge with.
2019-01-24T13:06:45.0652621Z See git-pull(1) for details.
2019-01-24T13:06:45.0653087Z
2019-01-24T13:06:45.0653340Z     git pull <remote> <branch>
2019-01-24T13:06:45.0655884Z
2019-01-24T13:06:45.4608951Z
2019-01-24T13:06:46.0066827Z Unhandled Exception: System.InvalidOperationException: Sequence contains no matching element
2019-01-24T13:06:46.0068492Z    at System.Linq.Enumerable.Single[TSource](IEnumerable`1 source, Func`2 predicate)
2019-01-24T13:06:46.0068781Z    at NuKeeper.Git.LibGit2SharpDiscoveryDriver.GetCurrentHead(Uri repositoryUri) in C:\Users\marc\Documents\repos\NuKeeper\NuKeeper.Git\LibGit2SharpDiscoveryDriver.cs:line 63
2019-01-24T13:06:46.0069137Z    at NuKeeper.AzureDevOps.VstsSettingsReader.CreateSettingsFromLocal(Uri repositoryUri) in C:\Users\marc\Documents\repos\NuKeeper\NuKeeper.AzureDevOps\VSTSSettingsReader.cs:line 88
2019-01-24T13:06:46.0069492Z    at NuKeeper.Commands.RepositoryCommand.PopulateSettings(SettingsContainer settings) in C:\Users\marc\Documents\repos\NuKeeper\NuKeeper\Commands\RepositoryCommand.cs:line 37
2019-01-24T13:06:46.0069797Z    at NuKeeper.Commands.CommandBase.OnExecute() in C:\Users\marc\Documents\repos\NuKeeper\NuKeeper\Commands\CommandBase.cs:line 84
2019-01-24T13:06:46.0070017Z    at McMaster.Extensions.CommandLineUtils.Conventions.ExecuteMethodConvention.InvokeAsync(MethodInfo method, Object instance, Object[] arguments) in C:\projects\commandlineutils\src\CommandLineUtils\Conventions\ExecuteMethodConvention.cs:line 77
2019-01-24T13:06:46.0070391Z    at McMaster.Extensions.CommandLineUtils.Conventions.ExecuteMethodConvention.OnExecute(ConventionContext context) in C:\projects\commandlineutils\src\CommandLineUtils\Conventions\ExecuteMethodConvention.cs:line 62
2019-01-24T13:06:46.0070629Z    at McMaster.Extensions.CommandLineUtils.Conventions.ExecuteMethodConvention.<>c__DisplayClass0_0.<<Apply>b__0>d.MoveNext() in C:\projects\commandlineutils\src\CommandLineUtils\Conventions\ExecuteMethodConvention.cs:line 25
2019-01-24T13:06:46.0070830Z --- End of stack trace from previous location where exception was thrown ---
2019-01-24T13:06:46.0071124Z    at McMaster.Extensions.CommandLineUtils.CommandLineApplication.<>c__DisplayClass126_0.<OnExecute>b__0() in C:\projects\commandlineutils\src\CommandLineUtils\CommandLineApplication.cs:line 505
2019-01-24T13:06:46.0071307Z    at NuKeeper.Program.Main(String[] args) in C:\Users\marc\Documents\repos\NuKeeper\NuKeeper\Program.cs:line 29
2019-01-24T13:06:46.0775427Z ##[error]Error: C:\Program Files\dotnet\dotnet.exe failed with return code:<PHONE_NUMBER>

Is there anything special about that repo? We do a git checkout "YourMainBranch" does it have a special name or something?

no, it happens on all repos. IMHO you shouldn't do a git checkout with the azure devops extension because that's what the agent does anyway before executing tasks.

The reason why we do it is because the devops extension checks out a detached head from your current master branch. The reason being that with multiple PR's you don't get a weird state, where you do a PR and another person in your team does a PR on the same stuff.
NuKeeper doesn't work with a detached head as it doesn't know which branch it should target. It fails here(https://github.com/NuKeeperDotNet/NuKeeper/blob/master/NuKeeper.Git/LibGit2SharpDiscoveryDriver.cs#L65). If we could make NuKeeper work with a detached head that would be nice, but i'm not sure if you can see if it's a detached head and from which branch that detachead head comes from.

there are environment variables which should help here https://docs.microsoft.com/en-us/azure/devops/pipelines/build/variables?view=azure-devops

issue was asynchronous git commands in the AzD extension. it's fixed now


Updating temporary Reply URL to avoid confusion

Updating the temporary Reply URL provided to avoid confusion and to more directly align with the walkthrough.

@parakhj or @yoelhor Please review.

Looks good
From: Scott Addie<EMAIL_ADDRESS>Sent: Wednesday, July 18, 2018 8:17 AM
To: aspnet/Docs<EMAIL_ADDRESS>Cc: Parakh Jain<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [aspnet/Docs] Updating temporary Reply URL to avoid confusion (#7674)
@parakhjhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fparakhj&data=02|01|parja%40microsoft.com|5b7c453038e845d285bd08d5ecc18879|72f988bf86f141af91ab2d7cd011db47|1|0|636675238319017632&sdata=PrY0ifj%2FkhA18rAUc%2BCU0sJCMFZASHCYUDTtvh%2BDLm0%3D&reserved=0 or @yoelhorhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fyoelhor&data=02|01|parja%40microsoft.com|5b7c453038e845d285bd08d5ecc18879|72f988bf86f141af91ab2d7cd011db47|1|0|636675238319027640&sdata=l2EAL9lTufbcC9OXflxTzB0D9Xfc4zvgvGep4k5enbs%3D&reserved=0 Please review.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Faspnet%2FDocs%2Fpull%2F7674%23issuecomment-405968267&data=02|01|parja%40microsoft.com|5b7c453038e845d285bd08d5ecc18879|72f988bf86f141af91ab2d7cd011db47|1|0|636675238319027640&sdata=zVmmO5QmFleNn5GPK38EmqzX5jOxNLzCVtYXd9DayfY%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAVozEXbrkp7SM1P_vEnHFwl8kn1p_g79ks5uH1F2gaJpZM4VUyzN&data=02|01|parja%40microsoft.com|5b7c453038e845d285bd08d5ecc18879|72f988bf86f141af91ab2d7cd011db47|1|0|636675238319037654&sdata=TgWj1v77uJWGB%2B0L%2FV4VVhWQ5UZ2rfomvm11eBbdekk%3D&reserved=0.

@egbill3eagle Thank you for your contribution! It's very much appreciated.


Possible to have pods-like structure for authenticators and authorizers?

For example, instead of:
app/authenticators/oauth2.js
app/authorizers/oauth2.js
Follow a 'pods-like' structure:
app/oauth2/authenticator.js
app/oauth2/authorizer.js
We could see this cleaning up the file structure of our app a little bit.

That should work - I guess you just have to reference the authenticator as oauth2:authenticator instead of aithenticator:oauth2 then and analogously for the aithorizer.

Tried it, did not work exactly as planned ... Attached is output of resolver after making the changes mentioned above

You can check the "Container" tab in the Ember Inspector to find out what identifier the authenticator was registered for (if at all).

They aren't registered in the container when stored in pods-style directories.

Doesn't seem to be supported by Ember CLI then.

In case anyone was also looking into this, I verified that it is in fact possible to use a pod style structure for you're authenticator/authorizer. I successfully kept both of them in an 'oauth2' folder and have a working app that uses both. I tried what @marcoow suggested first and referenced them with oauth2:authenticator but that was returning an unknown mixin error message. So I tried the traditional way as authenticator:oauth2 and everything worked fine. Not sure if this is due to an Ember CLI update or not but wanted to share my findings.

@aricallen: nice but actually pods are dead and I wouldn't recommend anyone using them right now.
Ember Simple Auth will definitely support the unified module system once that's available.


Multi Browser Testing

Objective
This PR add's Sauce Labs browser testing on Master and Firefox browser testing in Travis due to noticeable failures across the browsers. I've used my private account for the time being since the open source account didn't seem to work for Inferno. It also fixes tests on Safari.
Current Passing Tests

Lookin' good! Anyway we can get any of the old IEs hooked up? Although useless in the real-world, they're still somewhat useful & sometimes targeted in business reqs.
I, for one, have more than a couple clients who need IE9, and so this requirement is talked about often.

@lukeed Support for IE is predictable

@LukeSheard Yup, what I thought too. Now it's documented :)


Setup Duende Identity Server.

Initial setup completed, see https://github.com/KinaUna/Examplium/blob/master/Documentation/Code/IdentityServer/01-Initial-Setup.md


Create a new root type called spec

As an engineer who hates DOORS, I want a configuration managed solution for capturing requirement specification in my model so that I can trace requirements to architecture to behavior in one place.
[ ] Must have a new root type called spec.
[ ] Must have required fields for name(string), and reqs(Requirement[])
[ ] Must have optional fields for description(string), sub-spec(Spec[])
[ ] Requirements must have required fields for ID(int) and shall(string)
[ ] Requirements must have optional field to capture an array of optional attributes

I've implemented a plugin for this in the \plugins\aac-spec directory, but I have not built any business logic yet.
The data model and extensions are in place.
A behavior for spec validation is defined but not implemented.


Chore: fix build_directory_md.yml

Fixes the build_directory_md.yml action. Sets the Nim runtime dir as a forced git exclusion.
git clean removed the nim compiler directory, which jiro4989/setup-nim-action@v2 installs to $(pwd)/.nim_runtime.
This puts the nim install dir outside of the current project directory with:
  - uses: jiro4989/setup-nim-action@v2
    with:
      parent-nim-install-directory: ${{ runner.temp }}

Contradicts this: https://github.com/TheAlgorithms/Nim/pull/78#pullrequestreview-2164377447 from #78 discussion.
Are you sure this change is necessary?

Contradicts this: #78 (review) from #78 discussion. Are you sure this change is necessary?

Ah, sorry, didn't see that PR. Well, some change is necessary for the job to finish.
As I said, there's an alternative approach, which #78 already implements in another script.
I can change this PR for consistency.

Changed the fix to the other approach.


vtcomposite slower than node-mapnik when overzooming points

The benchmarks (using #54), on my machine show that vtcomposite is 3x slower than node-mapnik when overzooming points.
@artemp @millzpaugh - are you able to replicate this? Any idea what might be going on?
node-mapnik
node bench/node-mapnik.bench.js --iterations 500 --concurrency 4

Gives:
4: tiles completely made of points, overzooming, no properties ... 10870 runs/s (46ms)
6: tiles completely made of points, overzoooming, lots of properties ... 6757 runs/s (74ms)

vtcomposite
node bench/vtcomposite.bench.js --iterations 500 --concurrency 4

Gives:
4: tiles completely made of points, overzooming, no properties ... 3676 runs/s (136ms)
6: tiles completely made of points, overzoooming, lots of properties ... 3106 runs/s (161ms)

@springmeyer @millzpaugh - yep, I can replicate this performance degradation even in refactored bbox-check branch.  I've tried to filter out points not within bounding box as early as possible (in point_handler)
void points_point(vtzero::point const& pt)
    {
        CoordinateType x = pt.x * zoom_factor_ - dx_;
        CoordinateType y = pt.y * zoom_factor_ - dy_;
        if (!(x > bbox_.max.x || x < bbox_.min.x || y > bbox_.max.y || y < bbox_.min.y))
        {
            geom_.emplace_back(x, y);
        }
    }

but no significant improvements. I'll keep looking but this hints at vtzero is being slower at reading points.

Found this also #58

Capturing @springmeyer's qs to consider while continuing to investigate this performance issue:

Could it be that node-mapnik is doing something incorrect/buggy?
What does profiling show the bottleneck is in vtcomposite when benching just points?
What does profiling show the bottleneck is in node-mapnik when benching just points?
We should add a flag to run just one of the “rules” in vtcomposite to make ^^ that easier I think

but no significant improvements. I'll keep looking but this hints at vtzero is being slower at reading points.

Interesting....

What does profiling show the bottleneck is in vtcomposite when benching just points?

If I comment all lines in bench/rules.js except "tiles completely made of points, overzooming, no properties" and run with node bench/vtcomposite.bench.js --iterations 50000 --concurrency 4 and sample in Activity monitor I see that the top bottleneck (in the threadpool) on my OS X system is the vector.reserve for points:

Which is slightly odd since that is always going to be vec.reserve(1) in this test, so its hard to believe that is a significant bottleneck.
For "tiles completely made of points, overzoooming, lots of properties" I noticed it would likely speed up property encoding a bit to use the property_mapper (https://github.com/mapbox/vtzero/blob/master/doc/advanced.md). Something like this: /springmeyer/68b7c7bc82138bcc7c3c37fd3b6fb574 speeds things up for me slightly. This test node bench/vtcomposite.bench.js --iterations 50000 --concurrency 4 runs in 3049 runs/s (16397ms)  vs 3746 runs/s (13348ms) without the property mapper.

node-mapnik was significantly faster because we were passing in the same tile twice in the bench script with same layer name.
node-mapnik would grab the first tile that was the same zoom level request as a map request - and thus, did not require overzooming. The second tile did require overzooming but since the layer name already existed in the first tile, it would disregard it and return the first tile.
vtcomposite would overzoom the first tile that required overzooming and then disregard the second duplicate tile.
In sum, node-mapnik wasn't doing any overzooming work which is why it was so much faster.
We've resolved duplicate tiles in the bench.js in #61 and this is now resolved!

👍 to close, but...

For tiles completely made of points, overzoooming, lots of properties I noticed it would likely speed up property encoding a bit to use the property_mapper (https://github.com/mapbox/vtzero/blob/master/doc/advanced.md).

Ticketed this at https://github.com/mapbox/vtcomposite/issues/68


Define workflow for deleting a project and deleting a version

We see more and more tickets about issues that arise around deleting projects and not beeing able to delete versions.
This ticket shall be the base for discussion around this feature.
There are two related parts on this ticket:
Deleting a project
Currently when a project is deleted, the builds for the project's versions are still kept around and publicly accessible. We need to define what shall happen once a user deletes a project and communicate that to the user via docs and via the user interface while deleting a project.
Following use cases might be valid:

[P1] The project merged with a different project or just got out of date and no longer maintained
[P2] The docs are moving to a different hosting platform

Deleting a version
Currently there is no way to delete the builds of a version.
Following use cases might be valid:

[V1] The version was part of a wrong push/invalid version number and should be deleted since the publication was an accident.
[V2] The version is no longer supported, but since it's still listed in google the project's users are still using the version even when it's outdated.

See the Deleting projects and versions milestone for related tickets.

We currently don't want to break links and that's one reason why already built docs are not deleted even when the project goes away. However IMO the project's owner should be in full control of what happens to his docs. So there should always be the option to remove the HTML files from the server somehow.
However deleting documentation should be the last resort and we should offer the user other method of achieving their use cases.
For versions, we could allow the deletion of a version but provide the user with two options:

Mark the version as deleted and create a redirect to a different version specified by the user during deletion (solves use case V2).
Delete the version and the related builds but show someone who tries to access the docs of the deleted version a message "These docs have been deleted, please go to  for the current documentation" (solves use case V1).

In  #1264 a user tried to delete a project and recreated it in order to fix an issue with a build. That did not work as the builds were not deleted. That IMO is counter intuitive.

@gregmuellegger have you arrived at a decision?
I find it incredibly infuriating that I can't delete projects. I moved the docs for reghdfe.readthedocs.org to my own website and I still see people going to the old version which is deprecated, even though I deleted the project entirely.
Either deleting or allowing redirection are better choices than having obsolete documentation. What would it take for this to work? DMCA?

No, we should get approval for one way or the other from @agjohnson or @ericholscher on this first.

I definitely think we should stop hosting deleted docs, but I'd love to save them to S3 or something along those lines, so the links aren't broken forever, and we could link to the archives on the 404 page. I think this is the best solution, and wouldn't be overly hard to build.

Would you zip them in an archive and host this on S3? If we put the raw HTML files there then search engines will still pick up old obsolete documentation.

Just host them normally with a robots.txt disallowing indexing. We could
also keep them hosted on our side at an archive domain, with robots.txt off
as well, though we already have issues with docs taking up too much disk
space, so I'd prefer to offload them somewhere else.
I guess we could just delete them, but that makes me sad, and breaks the
internet.
On Thu, Sep 17, 2015 at 10:19 AM, Gregor Müllegger<EMAIL_ADDRESS>
wrote:

Would you zip them in an archive and host this on S3? If we put the raw
HTML files there then search engines will still pick up old obsolete
documentation.
—
Reply to this email directly or view it on GitHub
https://github.com/rtfd/readthedocs.org/issues/1408#issuecomment-141156520
.

--
Eric Holscher
Maker of the internet residing in Portland, Oregon
http://ericholscher.com

@ericholscher can you do a redirect instead?
e.g. I moved a project from rtfd to my website to put it together with everything else, but now if you google you are at a loss about what link to follow..

The hard part about redirects is that we serve all the HTML directly from nginx, without talking to Python. This is an effort to keep the page loads fast, and keep the docs serving even when our application servers or databases have issues. We do run Python code on 404's, which is where our traditional redirect logic comes in..
So we could delete the files, and do redirects in the 404 handling code. This would require us to keep a Project object in the database, but it would be in a "deleted" or "disabled" state, and have the redirect information on that. This seems like the best option, where a "disabled" project could be given either a redirect to a new site, or an archive listing of the previous content.

As part of this issue you closed Issue 292, but I haven't seen a discussion here about user account management.

Include instructions on signup that username will be made public and not to use PII.

I'd prefer a random string as a profile ID and then optional display name that can be edited after the fact.

Offer ability to delete your own account.

Just wanted to add my vote for a feature that allows users set re-directs from one set of docs to another. We have a couple of a component of a project that has its own set of docs, but want to merge these with a more extensive and complete set of docs.
In the absence of any automatic re-directing, what are folks doing? Just adding a url to the index page and saying "Go here"?

I'm closing this issue because nowadays it's possible to delete a Project and that action will remove all the HTML versions of it also. Besides, a Version can be marked as inactive and its docs are not served.
I consider this issue already done.
In case we want to keep going to the discussion of "archiving deleted projects on S3 or similar" we can open a new one.


No available executors, this is not expected, framework should call createExecutorIfAbsent first  before coming to here.

Pre-check

[X] I am sure that all the content I provide is in English.

Search before asking

[X] I had searched in the issues and found no similar issues.

Dubbo Version
2.7.23
Steps to reproduce this issue
Environment:
Dubbo version 2.7.23
JDK 11
servce provider side
Question:
when I start my spring dubbo service,  below logs occur:
[WARN] [o.a.d.c.threadpool.manager.DefaultExecutorRepository] [] [] []  [DUBBO] No available executors, this is not expected, framework should call createExecutorIfAbsent first before coming to here., dubbo version: 2.7.23, current host: xxx
After a few seconds,  this log no longer occurs.
I am eager to known whether here problems happened, or whether my dubbo service configuration has errors.
Thanks.
What you expected to happen
no this warn log when start dubbo service.
Anything else
No response
Are you willing to submit a pull request to fix on your own?

[X] Yes I am willing to submit a pull request on my own!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Have you tried test on 3.2.x?

Have you tried test on 3.2.x?

Do I have to update dubbo version to 3.2.x ? Due to environmental restrictions, the upgrade cannot be completed in a short time. I want to know if this is a known problem in the 2.7.x version. Can this problem be solved or circumvented in the  2.7.x version?

Have you tried test on 3.2.x?

Do I have to update dubbo version to 3.2.x ? Due to environmental restrictions, the upgrade cannot be completed in a short time. I want to know if this is a known problem in the 2.7.x version. Can this problem be solved or circumvented in the 2.7.x version?

I want to know if this is a known problem in the 2.7.x version.

Yes

Can this problem be solved or circumvented in the 2.7.x version?

No, 2.7.x is EOL now


Only a question...

You have declared jest@latest in devDependencies.json, but when I tried your template, the downloaded jest version from template is the same jest version that is downloaded by the react native init in version 0.56.0.
Why did you still keep the jest in devDependencies.json file?
Btw, awesome work!

Hi!
@lucianomlima I think this and a couple other dev dependencies can be removed, like babel-core. I wasn't able to make jest work with 0.56 yet, were you?

Yeah, in my test don't work too. But in a project without Typescript is working fine with RN 0.56.0.
Yeah! You can put in your package.template.json:
"resolutions": {
  "babel-core": "^7.0.0-bridge.0"
}

To force any package that uses<EMAIL_ADDRESS>to use version 7 instead.
But I not try here xD


Line in bug #1522

Fix it.

Should be working now.

It is now a circle.

This is not a bug, it's a feature.


cilkc in prismjs for intro to cilk programming

new prism.css with theme for parallel-keywords;
use cilkc in intro to cilk programming;
update fib-code image in the intro.

@ailiop you can see cilkc with the deploy preview for this PR. (Ignore "All checks have failed" because I fixed it on Netlify.) I also fiddled with the theme to use our color palette a bit more, and tried the tangelo for parallel-keywords. Thanks for making this possible.

@behoppe Thank you, the Cilk keywords look great!  I think the rest of the theme may need some more tweaking, mainly because the contrast between the lime-green function tokens and the gray background is a bit too low for my eyes.  (Interestingly, the code looks great when I apply my browser-extension dark mode filter.)
Possible improvements:

Use "blue" for keywords and built-in types.
Use "teal" for functions.
Maybe use different colors for keywords and built-in types, possibly adding "lime" or "americanyellow" to the mix.

@ailiop I made several adjustments which improve foreground-background contrast and character-color balance:

Blue keywords
Shift the blue a bit lighter
Shift the lime and the tangelo a bit darker
Shift the background a bit lighter.

I personally like green for functions, if that works, because green on the website associates with clicking links, which I think has a similar notion as a function call. What do you think?
You can easily see the previous theme represented by the image in this intro article btw.

@behoppe Thank you, this looks much better to my eyes!
I think there may still be some benefit to differentiating between keywords (if, return, etc) and built-in types (int, etc), perhaps by using different shades of blue.
And the image at the top of the Introduction to Cilk programming tutorial should be updated once the style is finalized.

Thanks @ailiop. How do I differentiate return from int with prismjs? I could not find a CSS class that distinguished them, and I'm actually convinced there is no distinction in prismjs. (But I could be wrong.)


Update app.py to use open ai API to determine a meaningful filename

Use LLM to create the powerpoint filename instead of using the prompt text.
Previously app.py used the prompt for the filename.  This caused an issue when using a prompt longer than 256 chars due to windows filename limitation.  Now, it takes the prompt, send it to open ai API and gets a meaningful yet short name in return.

Hey @scott------
Thanks for this pr! It's a great feature althrough it should not use GPT-4 for the name generation. That's way to overpowered and too expensive for such an easy task. Let's just use GPT-3.5 Turbo. That should do the job well enough.
Kind regards
Timon Käch

Thanks - I've merged this now.


go 1.5 status unclear to new users on website

On http://getgb.io/docs/install/ it says that it supports 1.4. Perhaps it should say 1.4 or greater? Or is 1.5 not quite supported yet? I only see one open issue involving 1.5 so it seems like it probably supports 1.5.

Gb support go 1.4 and up, thanks for pointing this out. I'll fix the
website.
On Sat, 26 Sep 2015 10:03 Benjamin Atkin<EMAIL_ADDRESS>wrote:

On http://getgb.io/docs/install/ it says that it supports 1.4. Perhaps it
should say 1.4 or greater? Or is 1.5 not quite supported yet? I only see
one open issue involving 1.5 so it seems like it probably supports 1.5.
—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/issues/417.


integ-new-install-k8s-tests_istio broken for "applicationPorts" not defined

Bug description
The test is timeout on every PRs: https://k8s-testgrid.appspot.com/istio_istio#integ-new-install-k8s-tests_istio&width=20
I tried to run it locally, it seems it's caused by failing to deploy the workload (Error creating: admission webhook "sidecar-injector.istio.io" denied the request: template: inject:159: function "applicationPorts" not defined):
Name:           a-v1-66b99f6d85
Namespace:      mirroring-1-69610
Selector:       app=a,pod-template-hash=66b99f6d85,version=v1
Labels:         app=a
                pod-template-hash=66b99f6d85
                version=v1
Annotations:    deployment.kubernetes.io/desired-replicas: 1
                deployment.kubernetes.io/max-replicas: 2
                deployment.kubernetes.io/revision: 1
Controlled By:  Deployment/a-v1
Replicas:       0 current / 1 desired
Pods Status:    0 Running / 0 Waiting / 0 Succeeded / 0 Failed
Pod Template:
  Labels:       app=a
                pod-template-hash=66b99f6d85
                version=v1
  Annotations:  foo: bar
  Containers:
   app:
    Image:       gcr.io/ymzhu-istio/app:ymzhu-dev
    Ports:       7070/TCP, 80/TCP, 8080/TCP, 3333/TCP
    Host Ports:  0/TCP, 0/TCP, 0/TCP, 0/TCP
    Args:
      --grpc
      7070
      --port
      80
      --port
      8080
      --port
      3333
      --port
      8081
      --version
      v1
    Liveness:     tcp-socket :tcp-health-port delay=10s timeout=1s period=10s #success=1 #failure=10
    Readiness:    http-get http://:8080/ delay=10s timeout=1s period=10s #success=1 #failure=10
    Environment:  <none>
    Mounts:       <none>
  Volumes:        <none>
Conditions:
  Type             Status  Reason
  ----             ------  ------
  ReplicaFailure   True    FailedCreate
Events:
  Type     Reason        Age                   From                   Message
  ----     ------        ----                  ----                   -------
  Warning  FailedCreate  61s (x16 over 3m45s)  replicaset-controller  Error creating: admission webhook "sidecar-injector.istio.io" denied the request: template: inject:159: function "applicationPorts" not defined

Maybe caused by https://github.com/istio/istio/pull/17986, cc @howardjohn

https://github.com/istio/istio/pull/18352 fixes it, thanks for noticing

Thanks!


is it possible the battery status display at homekit?

Hallo,
if you ask the ip:port/list?token=xxxx you get
batteryCritical | false
can this information displayed in homekit?
regards

Hello @testpaul999,
this would be a very nice feature!
@lukasroegner If you need some inspiration on how to implement battery status and battery warnings, this plugin just got this feature: https://github.com/meg768/homebridge-ikea-tradfri-gateway.
Cheers
Felix

New version 0.4.0 online and on NPM. I did not add the BatteryService as this service requires percentage of the battery. However, there is a BatteryLow characteristic, which works in HomeKit when added to a lock service. Here are the results:
Normal battery status (nothing visible):

Low battery status (exclamation mark icon + red label):

Details view ("Battery low" in the status label on the top):

Advanced details ("Battery low" at the top):

Danke, werde ich heute Abend testen.
Grüße
Jens-Uwe
Am 22.09.2019 um 12:20 schrieb Lukas Rögner<EMAIL_ADDRESS>
New version 0.4.0 online and on NPM. I did not add the BatteryService as this service requires percentage of the battery. However, there is a BatteryLow characteristic, which works in HomeKit when added to a lock service. Here are the results:
Normal battery status (nothing visible):
[IMG_0004]https://user-images.githubusercontent.com/16758848/65385809-2710e500-dd33-11e9-96cb-80b7c87b2ef1.png
Low battery status (exclamation mark icon + red label):
[IMG_0005]https://user-images.githubusercontent.com/16758848/65385815-3abc4b80-dd33-11e9-849f-aa1cfd71363c.png
Details view ("Battery low" in the status label on the top):
[IMG_0006]https://user-images.githubusercontent.com/16758848/65385822-49a2fe00-dd33-11e9-8e55-1928ed614fe0.png
Advanced details ("Battery low" at the top):
[IMG_0007]https://user-images.githubusercontent.com/16758848/65385823-57588380-dd33-11e9-9242-e0dcbf7e412a.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/lukasroegner/homebridge-nuki/issues/4?email_source=notifications&email_token=AIPS4U3FICVOIZYYVNBAZ3DQK5BINA5CNFSM4IYXC7MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7JC6QI#issuecomment-533868353, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIPS4U3FLOWO3TMURF4UTSTQK5BINANCNFSM4IYXC7MA.

Ich versuche das über config-ui zu installieren und bekomme diesen Fehler.
[cid:2A681CCD-F86F-4C3B-929F-A2B577669FF1-L0-001]
?
Danke
Grüße
Jens-Uwe
Am 22.09.2019 um 12:20 schrieb Lukas Rögner<EMAIL_ADDRESS>
New version 0.4.0 online and on NPM. I did not add the BatteryService as this service requires percentage of the battery. However, there is a BatteryLow characteristic, which works in HomeKit when added to a lock service. Here are the results:
Normal battery status (nothing visible):
[IMG_0004]https://user-images.githubusercontent.com/16758848/65385809-2710e500-dd33-11e9-96cb-80b7c87b2ef1.png
Low battery status (exclamation mark icon + red label):
[IMG_0005]https://user-images.githubusercontent.com/16758848/65385815-3abc4b80-dd33-11e9-849f-aa1cfd71363c.png
Details view ("Battery low" in the status label on the top):
[IMG_0006]https://user-images.githubusercontent.com/16758848/65385822-49a2fe00-dd33-11e9-8e55-1928ed614fe0.png
Advanced details ("Battery low" at the top):
[IMG_0007]https://user-images.githubusercontent.com/16758848/65385823-57588380-dd33-11e9-9242-e0dcbf7e412a.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/lukasroegner/homebridge-nuki/issues/4?email_source=notifications&email_token=AIPS4U3FICVOIZYYVNBAZ3DQK5BINA5CNFSM4IYXC7MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7JC6QI#issuecomment-533868353, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIPS4U3FLOWO3TMURF4UTSTQK5BINANCNFSM4IYXC7MA.

@testpaul999 Image is not attached in your post. Please try to use GitHub and upload the image by dragging & dropping.

@lukasroegner Awesome, thank you for the quick implementation of the battery warning feature!

Sorry please here

Checked by installing latest version on a test setup, works as expected.
Based on the error message I also double-checked the package.json and ran it through a JSON validator, everything is fine.
When I google your issue, people ofter recommend npm cache clean.

hi,
if i install by hand it works.
but the plugin will be installed at the "worng" path.
your plugin /usr/local/lib/node_modules
all other
/usr/lib/node_modules
regards Jens
On Sun, 22 Sep 2019 07:39:22 -0700
Lukas Rögner<EMAIL_ADDRESS>wrote:

Checked by installing latest version on a test setup, works as expected.
Based on the error message I also double-checked the package.json and ran it through a JSON validator, everything is fine.
When I google your issue, people ofter recommend npm cache clean.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/lukasroegner/homebridge-nuki/issues/4#issuecomment-533887660

--
Jens<EMAIL_ADDRESS>

Have you tried to clear the NPM cache?

Sure dos not help. Only at cli install sudo npm install Homebridge-nuki -g runs.
If I’ll try via config-Ui it fails.
Liebe Grüße
Jens-Uwe
Am 22.09.2019 um 17:08 schrieb Lukas Rögner<EMAIL_ADDRESS>
Have you tried to clear the NPM cache?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/lukasroegner/homebridge-nuki/issues/4?email_source=notifications&email_token=AIPS4UYBZN5LQAZ7SGG2AVDQK6C5NA5CNFSM4IYXC7MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7JIGXQ#issuecomment-533889886, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIPS4U5ITCHXBAZEZGAWFETQK6C5NANCNFSM4IYXC7MA.

Hi, have found my issue there was a empty package.json in the path /usr/lib/ .
I'll remove this and now ich can install with the config-ui the package.
Regards
On Sun, 22 Sep 2019 08:08:06 -0700
Lukas Rögner<EMAIL_ADDRESS>wrote:

Have you tried to clear the NPM cache?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/lukasroegner/homebridge-nuki/issues/4#issuecomment-533889886

--
Jens<EMAIL_ADDRESS>


making errors lighter

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[x] Other, please describe: just making it lighter.

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[ ] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[ ] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[x] I have added a convincing reason for adding this feature, if necessary

Other information

I really don't see what this PR is trying to accomplish. It is not making anything ligther, just renaming an import. We're excited to have new contributors, but I hope you understand that we won't merge this.

I really don't see what this PR is trying to accomplish. It is not making anything ligther, just renaming an import. We're excited to have new contributors, but I hope you understand that we won't merge this.

I changed water to H20 and just noticed it now.


Focus is trapped in Datepicker field

Reproduction
https://formkit.link/c1178f7abb660a88b9f5857c5a763908
Describe the bug
Navigating into the date picker field on Safari will keep the focus trapped within the field. I have found no way to escape it – apart from reloading the page.
This is a severe bug, I sincerely hope for quick fix.
Environment
• OS: Mac OS 14.6.1
• Browser Safari
• Version 17.6

@betabong we are looking into it now

@betabong This should be resolved at FormKit Pro v0.127.6

Fantastic, thanks for the quick fix!


Move init-usb-gadget to this repository

The init-usb-gadget script used to live in the TinyPilot repo, now they
live under scripts/usb-gadget/.
This fixes #67 security concerns by changing the file mode to 700 and
owner/group to root.
A few changes were made:

tasks/install_usb_gadget.yml now contains the steps to create the
new /opt/tinypilot-privileged directory and populate it with the
init-usb-gadget file.
templates/usb-gadget.systemd.j2 now points to the new
/opt/tinypilot-privileged location.
defaults/main.yml received new entries to make the job under
tasks/install_usb_gadget.yml easier.

This change is

Whoops this actually caused a break. This repo's test missed it, but TinyPilot caught it:
https://app.circleci.com/pipelines/github/mtlynch/tinypilot/866/workflows/c167a667-6e95-48ad-b653-1cd19d040cea
I fixed it in #69.
The testing gap was because the ansible tests were skipping the tasks related to init-usb-gadget in CI. This was because when I added those tests, I was having trouble getting systemd to run under Docker, but systemd works under the current image, so I removed that workaround in #70, which should catch things like this in the future.

Whoops this actually caused a break. This repo's test missed it, but TinyPilot caught it:
https://app.circleci.com/pipelines/github/mtlynch/tinypilot/866/workflows/c167a667-6e95-48ad-b653-1cd19d040cea
I fixed it in #69.
The testing gap was because the ansible tests were skipping the tasks related to init-usb-gadget in CI. This was because when I added those tests, I was having trouble getting systemd to run under Docker, but systemd works under the current image, so I removed that workaround in #70, which should catch things like this in the future.

Oh, I'm glad to hear you managed to fix it fast. Damn, the fix was something I was already doing but stopped because dest should in theory be able to create the directory if it doesn't exist. I was just skimming through the CI "green" outputs, I should've paid more attention to this. I'm sorry.

Oh, I'm glad to hear you managed to fix it fast. Damn, the fix was something I was already doing but stopped because dest should in theory be able to create the directory if it doesn't exist. I was just skimming through the CI "green" outputs, I should've paid more attention to this. I'm sorry.

No problem, it was a gap in testing, so the green gave false confidence. Generally, we should be able to trust the tests.

No problem, it was a gap in testing, so the green gave false confidence. Generally, we should be able to trust the tests.


Curl-Impersonate support for Chrome Android versions

I see that the impersonate support is available only for chrome99_android and not for any of the succeeding versions of Chrome for Android.
If I would want to use curl-impersonate for Chrome Android for later versions such as v125, v126, v130 and so on, then the only impersonate option available is chrome99_android, which leads to a TLS fingerprint mismatch.
Is there a way to way to resolve this problem with curl-impersonate?

The list of ciphers can be obtained by looking at the Client Hello message in
Wireshark, then converting it using this reference
https://wiki.mozilla.org/Security/Cipher_Suites
after getting the list of ciphers, update curl_chrome99_android file with the new ciphers and rename the file

The list of ciphers can be obtained by looking at the Client Hello message in
Wireshark, then converting it using this reference
https://wiki.mozilla.org/Security/Cipher_Suites
after getting the list of ciphers, update curl_chrome99_android file with the new ciphers and rename the file

Quick question here though. I have a free version of Wireshark installed on my system. In the info section, I never come across the "Client Hello" or "Server Hello" messages. All I see are the "Application Data" messages. Should I have a paid version of Wireshark to obtain the "Client Hello" message? Or am I missing something here?
Additionally, I am trying to obtain the ciphers on a mobile device. Is it even possible to do so using Wireshark?